Load private SSH key onto host machine dynamically

...needs a few missing things.
in my case one of which is location of the ssh key, but, there may be others.

@dougbtv is this still the case? Can you document which template you're talking about? I'll go and dynamicize it :)

I think this was a quickly added issue when we were at astricon, but.... this is the one I'm speaking of specifically:
https://github.com/redhat-nfvpe/kube-centos-ansible/blob/master/inventory/examples/vms/vms.inventory#L19
That is the ansible_ssh_private_key_file= -- side note that like what I was talking about this morning was that we gen an SSH key on the virthost, then we never pull it down, it's left as a manual action, so... that's probably the bigger picture thing. At any rate, this is maybe something that's all gonna get ripped out. e.g. if we don't create the virtual machines in this playbook, maybe this gets moved into the extracted project.

@dougbtv Thanks man. I think I'm just going to convert this issue to making it so our key is generated automatically for us, and then loaded from the local machine. I'll start working on this, because I think it'll make the documentation a lot easier to consume.
I probably need a flag or option that lets you override all this as well, since some people may be creating their own images, and they may very well have their own keys loaded into that image already.

Also FYI, if you use vm_ssh_key_path in your inventory/virthost/host_vars/virthost.yml file, the template will already generate the dynamic SSH key path for you in vms.local.generated. I did this at AstriCon :)


Update GSoC proposals

Add suggested mentor names, remove Bitmanip, add Pointer Authentication.

I would add email addresses, or point people to the lowRISC mailing list (yes, it still exists!). I'm not sure people get many answers from<EMAIL_ADDRESS>

Either way, merge when you're ready. And let me know if we should publish this right away, or together with the blog post later today.

I would add email addresses, or point people to the lowRISC mailing list (yes, it still exists!). I'm not sure people get many answers from<EMAIL_ADDRESS>
That's a good point. But if someone is interested in a specific project, all potential mentors for this should be in the loop. We will need to forward emails ourselves to the other mentors.

The blog post is now part of this PR as well.


Remove 'use short object names' from SQL preferences

Interface - Editor - SQL Editor - Code Completion
There are two mutually exclusive options:

Use short object names
Use long object names

Looks like 'use short' is not needed
nothing - short
'short' - short
'long' - long
both - long

Major status because this setting is connected with CloudBeaver preferences

We need to create combo box with default/short/long values


in godot version 3.2, will you still have problems with the Samsung SM-G930F mobile phone? It does not work 3D animated bones imported from Blender, it locks the phone and closes the application.

Godot version:

OS/device including version:

Issue description:

Steps to reproduce:
Minimal reproduction project:

Godot version: 3.1.1
OS/device including version: SM-G930F
Issue description:
It does not work 3D animated bones imported from Blender, it locks the phone and closes the application
Steps to reproduce:  animated bones imported from Blender

This is a duplicate of https://github.com/godotengine/godot/issues/28298 which was closed by https://github.com/godotengine/godot/pull/29751


in master, when add team is error

Check if the mysql procedure 'parentTeamIds' is executed correctly.
￼

Team has been remove in 0.3.0-beta-5 version


Fix npm package vulnerabilities in botbuilder-ai

Updated request to 2.87.0 per https://snyk.io/node-js/request

Pull Request Test Coverage Report for Build 1641

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

Totals

Change from base Build 1634:
0.0%

Covered Lines:
2176

Relevant Lines:
3007

💛  - Coveralls


Adds a post about adding the log out button to the app #hackathon2020

A very early WIP but wanted to get it up. I want to use this as an opportunity to discuss the value of hackathons as an opportunity for cross-team collaboration, as well as a way to discuss how early MVPs mature. I'm also using a 5-why's-esque analysis. I've got some ideas for visual assets as well, but those will likely fall out of the Hackathon demo day presentation. Feedback welcome!

Okay! This is ready for review. I've optimistically set the published date for tomorrow 👍

Thanks @mdole – that's all super-valuable feedback! I'll get to that today.


Thermal Expansion calculator under thermodynamics

calculator for thermal expansion

I want to work on this issue under JWOC

Can you assign me in this contributions  as GSSOC21


[BUG] Launching different versions of the App remotely

I was working in a remote Jupyter notebook and updated FiftyOne to 0.7.2. My local version was still on 0.7.1, and when I launched the App in the notebook I got the following error.
Exception in thread Thread-7:
Traceback (most recent call last):
  File "/home/erich/sw/conda/envs/test/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/home/erich/sw/conda/envs/test/lib/python3.9/threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/home/erich/sw/conda/envs/test/lib/python3.9/site-packages/fiftyone/core/client.py", line 108, in run_client
    io_loop.run_sync(connect)
  File "/home/erich/sw/conda/envs/test/lib/python3.9/site-packages/tornado/ioloop.py", line 530, in run_sync
    return future_cell[0].result()
  File "/home/erich/sw/conda/envs/test/lib/python3.9/site-packages/fiftyone/core/client.py", line 96, in connect
    self._data = self._HC_ATTR_TYPE.from_dict(
  File "/home/erich/sw/conda/envs/test/lib/python3.9/site-packages/fiftyone/core/state.py", line 126, in from_dict
    stage = fost.ViewStage._from_dict(stage_dict)
  File "/home/erich/sw/conda/envs/test/lib/python3.9/site-packages/fiftyone/core/stages.py", line 183, in _from_dict
    stage = view_stage_cls(**{k: v for (k, v) in d["kwargs"]})
TypeError: __init__() got an unexpected keyword argument 'tags'

I'm not sure how a version mismatch could cause in issue. The Jupyter notebook should be only using the remote install. Perhaps clear your cache?
You were using a match_tags() stage? It's parameter name was changed from tags to tag. Hmmm

I'm not sure how a version mismatch could cause in issue. The Jupyter notebook should be only using the remote install. Perhaps clear your cache?
You were using a match_tags() stage? It's parameter name was changed from tags to tag. Hmmm

It just occurred when I installed the new version, it could very likely have been a cache issue.
The strange thing is that I was only trying to visualize the entire dataset, there was no match_tags() stage

It just occurred when I installed the new version, it could very likely have been a cache issue.
The strange thing is that I was only trying to visualize the entire dataset, there was no match_tags() stage

I think this can be closed. We haven't been able to reproduce, right?

I think this can be closed. We haven't been able to reproduce, right?

I haven't, closing.

I haven't, closing.


Publish multiarch manifest with releases

Publish multiarch manifests with each release.
https://github.com/kubevirt/kubevirt/issues/9756
Release note:
Publish multiarch manifests with each release

/lgtm
/approve
/hold
Can you try out the build_release_artifacts on a custom registry and  see if evertying is fine? Then feel free to remove the hold.
We can then push a beta.1 release I think.

/unhold


GTK+3 support for mozc_renderer

mozc_renderer with --enable-gtk-renderer is built for gtk+-2.0, could you support gtk+3?
This is not a bug, mozc_render well work with gtk+-2.0, just feature request.

Duplicate of #492


Successfully creating an agent shows a red dot instead of blue

Since this isn't an error I was expecting a blue dot that fades away.

Browser cache issue.


Fix: change lamports by getting mintInfo

Pull Request Description
Related Issue
Fixes #123
Changes Made
This PR adds the following changes:

Fixed token decimal handling in trade function to support tokens with different decimal places
Added proper scaling of input amounts based on token's decimal places
Added getMint import from @solana/spl-token

Implementation Details

Fetches mint info using getMint to determine token decimals
Calculates correct scaled amount using Math.pow(10, inputDecimals)
Supports tokens with any number of decimal places (e.g., USDC-6, SOL-9)

Transaction executed by agent
Example transaction: [Insert your test transaction hash here]
Prompt Used
Trade 0.0001 SOL for BONK
Additional Notes
This fix ensures proper handling of token decimals, which is crucial for:

USDC (6 decimals)
SOL (9 decimals)
Other SPL tokens with varying decimal places

Checklist

[x] I have tested these changes locally
[x] I have updated the documentation
[x] I have added a transaction link
[x] I have added the prompt used to test it


Allow a BoxDecoration to be specified in ButtonThemeData

Flutter doesn't have a good way of decorating a button with something that isn't a flat color. This is because the underlying Material widget doesn't allow for it.
Right now we just stack a DecoratedBox behind a MaterialButton with a transparent background. This works for our larger buttons, but fails for our smaller buttons, since the MaterialButton pads the height up to 48dp. This makes the button look like a regular sized button and there's no reasonable way to workaround it.
A proper fix would be to allow a BoxDecoration to be specified in ButtonThemeData. Then we can remove the Stack + DecoratedBox workaround.
Related to or same as https://github.com/flutter/flutter/issues/21125

b/141787699

Hi @jkurtw, we shouldn't support this type of modification since that would violate the Material specification.
However, I think what you could be looking for is creating a custom button composed of a GestureDetector and a Container, similar to this S.O. post.

@shihaohong Does it "violate" the Material spec or is it "unspecified" by Material spec?

I believe that modifying components to have an appearance outside of the design language is a violation of the design language.

/cc @willlarche for your thoughts


Use Jersey3 libs in service

@chenchals Only did the changes (since they were small) but it may need testing on the vumc deployment

@chenchals does the client generate code in javax (old, for spring 2.x) or jakarta (new, for spring 3.x) namespace. This can set like this - https://github.com/DataBiosphere/tanagra/blob/main/client/gradle/open-api.gradle#L34


Added new project anmed AudioPhile

Issue reference:
Proposed changes:
Added a new project to your  html-archive , please review my PR and let me know if you need any changes
Type of change:
Please delete options that are not relevant.

[x] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Additional info (if any):
Screenshot

Great work @shikhar13012001
Kudos to you 🙌


Improve perfomance

Purpose
This pull request resolved :

Problem selection tree take more time because reason recursive
Scale down time to execute methodinfor - Normally I estimate >3s
Improvement font size is better, at the moment , it too small from user feedback.
Add support Revit version 2024
Search supported in pervious version.

Description

(FILL ME IN) Brief description of the fix / enhancement. Mandatory section
Declarations
Check these if you believe they are true

[ ] This PR fix bug
[ ] This PR for new feature
[ ] The codebase is in a better state after this PR
[ ] The level of testing this PR includes is appropriate
[ ] User facing strings, if any, are extracted into *.resx files
[ ] Snapshot of UI changes, if any.
[ ] This PR modifies some build requirements and the readme is updated

Reviewers
@weianweigan
FYIs
(FILL ME IN, Optional) Names of anyone else you wish to be notified of

Great! Now we support revit 2024. Thanks!


enforced condition begin <= INVALID_POS_LOC has failed: (no message provided)

Input
$ srb init

Observed output
Generating /var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.rbi with 39624 modules and 274 aliases
Printing your code's symbol table into /var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/from-source.json
Printing /var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.rbi's symbol table into /var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.json
Traceback (most recent call last):
        6: from /Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/bin/srb-rbi:237:in `<main>'
        5: from /Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/bin/srb-rbi:196:in `main'
        4: from /Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/bin/srb-rbi:121:in `init'
        3: from /Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/bin/srb-rbi:232:in `block in make_step'
        2: from /Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/lib/hidden-definition-finder.rb:38:in `main'
        1: from /Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/lib/hidden-definition-finder.rb:47:in `main'
/Users/mbajur/.rvm/gems/ruby-2.5.3/gems/sorbet-0.5.5449/lib/hidden-definition-finder.rb:149:in `write_constants': /var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.rbi had unexpected errors. Check this file for a clue: /var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.json.err (RuntimeError)

And the mentioned file:
Exception::raise(): ./core/Loc.h:74 enforced condition begin <= INVALID_POS_LOC has failed: (no message provided)

Backtrace:
sorbet::Exception::enforce_handler(std::__1::basic_string_view<char, std::__1::char_traits<char> >, std::__1::basic_string_view<char, std::__1::char_traits<char> >, int) (in sorbet) + 42
sorbet::core::Loc::Loc(sorbet::core::FileRef, unsigned int, unsigned int) (in sorbet) + 109
sorbet::parser::Builder::Impl::constGlobal(ruby_parser::token const*, ruby_parser::token const*) (in sorbet) + 84
(anonymous namespace)::constGlobal(void const*, ruby_parser::token const*, ruby_parser::token const*) (in sorbet) + 31
ruby_parser::bison::typedruby25::parser::parse() (in sorbet) + 27153
ruby_parser::typedruby25::parse(void const*) (in sorbet) + 40
sorbet::parser::Builder::build(ruby_parser::base_driver*) (in sorbet) + 32
sorbet::parser::Parser::run(sorbet::core::GlobalState&, sorbet::core::FileRef, std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > >) (in sorbet) + 807
sorbet::realmain::pipeline::runParser(sorbet::core::GlobalState&, sorbet::core::FileRef, sorbet::realmain::options::Printers const&) (in sorbet) + 433
sorbet::realmain::pipeline::indexOneWithPlugins(sorbet::realmain::options::Options const&, sorbet::core::GlobalState&, sorbet::core::FileRef, std::__1::unique_ptr<sorbet::ast::Expression, std::__1::default_delete<sorbet::ast::Expression> >) (in sorbet) + 1122
sorbet::realmain::pipeline::index(std::__1::unique_ptr<sorbet::core::GlobalState, std::__1::default_delete<sorbet::core::GlobalState> >&, std::__1::vector<sorbet::core::FileRef, std::__1::allocator<sorbet::core::FileRef> >, sorbet::realmain::options::Options const&, sorbet::WorkerPool&, std::__1::unique_ptr<sorbet::OwnedKeyValueStore, std::__1::default_delete<sorbet::OwnedKeyValueStore> > const&) (in sorbet) + 332
sorbet::realmain::realmain(int, char**) (in sorbet) + 7157
main (in sorbet) + 11
start (in libdyld.dylib) + 1

/var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.rbi: Exception parsing file: `/var/folders/xs/wzbp_g3s2q54_mwq3pbrj4gc0000gn/T/d20200317-75684-1sknp18/reflection.rbi` (backtrace is above) https://srb.help/1001

Expected behavior
Initialization should be finished

Ruby 2.5.3

Does this happen for an empty project with 2.5.3, or are you able to share a minimal case to reproduce it?

Sadly, it happens on pretty big commercial closed-source project. Is there anything i can do to try to reproduce that? Backtrace of that is not telling me much

I am newly running into this issue as well. Unfortunately I also work on a large proprietary codebase.

Do you have files of >8M of size?
This error could be triggered by this

https://github.com/sorbet/sorbet/pull/2766 might help when it lands

I'm not fully aware of how sorbet works, but if you're asking if i have ruby files larger than 8M - than i don't ;)

My hidden.rbi is 6.5MB but since that's deleted I assume that's not the problem?

Can someone offer guidance on how to fix, work around, or debug this issue?

@mbajur This is fixed for me on the latest release; you should update and give this a try when you have a chance.

Perfect! Can confirm it now worked :) Thanks everyone


Can't train smaller models, bb = 0, 1, 2

Trying set one of 'vgg16', 'vgg16bn', 'resnet50', changing self.bb to 0 or 1 or 2, but getting RuntimeError: Given groups=1, weight of size [64, 3712, 3, 3], expected input[1, 1856, 128, 128] to have 3712 channels, but got 1856 channels instead even in case of changing input from 1024 to 512 and etc.
Set to 3 or 6 works as normal.

Hi, I'm sorry that only Swin and PVT are currently supported for the backbone (ref). These channel numbers cannot be defined by ourselves, which is defined by the backbone networks, no matter if it's VGG16 or Swin Transformer.


About the cuda version code.

Hi, I noticed that you updated some parameters, will you release the cuda version of the code next?

We do not have any immediate plans of developing a GPU friendly version of the code as reproducing results using CPUs is fairly quick. We shall let you know if this is in the works. Thanks.

I also have a question about MSL or SMAP datasets, does each training train each subset of the dataset (e.g. C1, C2) separately, or does it train all the data at once?

We train each file independently.


Use a  for serializations instead of a

After talking it through with @Becojo, we (checkout) opted to use a div to store serialized values, using data-serialized-id as the attribute for the serialization's ID, and data-serialized-value for the JSON-encoded serialized value. We opted for this because it just feels safer to avoid a script tag here, even if we provide a non-JavaScript type. Core also has some validations that complain about the markup we use in react-html for these serializations.
I made this change to our own version of react-html: https://github.com/lemonmade/quilt/commit/199f77739b7c20762a2d562cde2885e1522ab850#diff-fa2b0a55752ecf9bd270b9c4167b97f8, and it's probably a good idea to pull it into mainline too. I don't think anyone should be depending on the exact structure of these serializations, but that would be a note to call out in the changelog.
cc/ @TheMallen

This seems simple enough, but I'm not sure how impactful it is as a change. If I understand the concern here is just that it could potentially be flagged by linters / browser security in the future? Or just that it's possibly slightly more secure to use div than script type="text"?

The value is both that it’s safer and that it didn’t complain about the markup when we added an implementation of serialization to core. That second part doesn’t matter if it’s a bogus rule in core, but I think core is right that we should just prefer the safer markup here. I think it may also mean the dependency on serialization-JavaScript could go away, since HTML attributes have special escaping rules, but not 100% sure on that bit


Be more aggressive in finding the selected document

fix #253

Warnings

:warning:

Source/dom/models/Document.js changed, but not:

📚 its docs
🧪 its tests

That's OK as long as you're refactoring.

  Generated by :no_entry_sign: dangerJS


RGBA: channel 'A'

RGBA for foreground: Does channel 'A' indicate the mask?

Yes, A means alpha.

Thank you!


read_iiasa() - filter also by meta information

It would be helpful if read_iiasa() could also filter for meta information like climate assessment category.
Use case: I tried to download data for all C1 scenarios from the AR6-DB. This works with the below, but when extending the query to more than just global and selected variables the memory requirements get very large. Adding the option to add meta information to the query (as commented line) would remove the need to first download and store a large file in memory.
df = pyam.read_iiasa("ar6-public",
    region = "World",
    variable = ['Emissions|CO2', 'Primary Energy|Coal'],
    # Category = "C1",
    meta=['Category', "IMP_marker"])

df.filter(Category="C1", inplace=True)

(With thanks to @jkikstra for confirming that this is currently not possible.)

Indeed, this was possible (I believe) but was lost a while back due to a regression in the RestAPI of the ixmp package.
We will implement this feature as part of the migration to the ixmp4 package for hosting IIASA scenario databases, see http://docs.ece.iiasa.ac.at/ixmp4

I have the same needs as you. Following your example, I tried to use a filter to categorize, as shown.
But I have two question:

Why the filtered IamDataFrame can not be assigned to the new variable, It seems to be an empty table.
How can I filter out multiple categories at once, like df.filter(Category=["C1","C2","C3"], inplace=True)
@danielhuppmann, @jkikstra  Some ideas about addressing it? Thank you!

This is an unrelated question to the initial issue, and this is purely a mistake in your python code. You use the argument inplace=True, so the filter-operation happens directly on ar6_gas (which as you can see in the following cell now only has the C1 category).
When you call ar6_gas.filter(Category="C1", inplace=True), the method does not return any object, hence ar6_gas_c1 is None - see the error message in the last cell of the screenshot.
This behavior directly follow the equivalent pandas behavior for inplace, so please read the docs there.

This is an unrelated question to the initial issue, and this is purely a mistake in your python code. You use the argument inplace=True, so the filter-operation happens directly on ar6_gas (which as you can see in the following cell now only has the C1 category).
When you call ar6_gas.filter(Category="C1", inplace=True), the method does not return any object, hence ar6_gas_c1 is None - see the error message in the last cell of the screenshot.
This behavior directly follow the equivalent pandas behavior for inplace, so please read the docs there.

Thank you for your reply. It is my negligence.
I have already solved this problem.


Redis Pub-Sub Global Break Cache

To support a multi-node setup, we should delete any cache data for shortened urls per Redis pub-sub

Implemented in 16823abfb2d35399a1256f33b608173922cbc10b & d65a5f96c2af76a15c15fb489180b8475b4ee19c


2020.1 debug mode build fails

Debug-mode build of dldt is failing on CentOS 7 / gcc 4.8.5. Here are my steps to reproduce:

git clone
git submodule update --init --recursive
mkdir build && cd build
cmake -DCMAKE_BUILD_TYPE=Debug ..
make

[  0%] Building CXX object ngraph/src/ngraph/CMakeFiles/ngraph.dir/attribute_adapter.cpp.o
In file included from /usr/include/c++/4.8.2/x86_64-redhat-linux/bits/os_defines.h:39:0,
                 from /usr/include/c++/4.8.2/x86_64-redhat-linux/bits/c++config.h:2097,
                 from /usr/include/c++/4.8.2/bits/stl_algobase.h:59,
                 from /usr/include/c++/4.8.2/vector:60,
                 from /home/me/dldt/ngraph/src/ngraph/attribute_adapter.cpp:17:
/usr/include/features.h:330:4: error: #warning _FORTIFY_SOURCE requires compiling with optimization (-O) [-Werror=cpp]
 #  warning _FORTIFY_SOURCE requires compiling with optimization (-O)
    ^
cc1plus: all warnings being treated as errors
make[2]: *** [ngraph/src/ngraph/CMakeFiles/ngraph.dir/attribute_adapter.cpp.o] Error 1
make[1]: *** [ngraph/src/ngraph/CMakeFiles/ngraph.dir/all] Error 2
make: *** [all] Error 2

It appears that both ngraph and ade are setting _FORTIFY_SOURCE=2, which is great, but apparently gcc emits warnings unless combined with optimization flags. Both libraries also have hard-coded -Werror (treat warnings as errors) in their builds, which would be okay if they did in fact compile without warnings.

Update: The culprit is actually cmake/os_flags.cmake, which adds the -Werror flag to dldt and all sub-projects. For windows builds, this respects TREAT_WARNING_AS_ERROR, but not so for Linux/Mac. I see there are several other reported issues related to this, and an unmerged pull request going back to Oct 2019 that fixes it.

Hello @wlfgang
From the documentation available for dldt cmake build on CentOS we have this article - https://github.com/opencv/dldt/blob/2020/build-instruction.md
However, -DCMAKE_BUILD_TYPE=Debug is not listed there as validated option, so this might be not officially supported.
Please check all the available information we have for additional build options here https://github.com/opencv/dldt/blob/2020/build-instruction.md#additional-build-options
Also, we'll let you know if we get any information from developers team about it.
Thanks.
Best regards, Max.

@wlfgang

You can compile DLDT with cmake -DTREAT_WARNING_AS_ERROR=OFF and it's not respected for Unix systems as well.
Now, ngraph SDL flags are not used in compilation (see https://github.com/openvinotoolkit/openvino/blob/master/CMakeLists.txt#L98) while IE ones are used only for Release mode.


Custom postgresql port is ignored

https://github.com/opscode/opscode-omnibus/blob/master/files/private-chef-cookbooks/private-chef/providers/pg_database.rb#L21 does not honour custom port configured in chef-server.rb
Similar applies to database_exist? method below

As part of work to move into a consolidated repository, this ticket has been moved to:
https://github.com/chef/chef-server/issues/190
Thank you for taking the time to file it!


This commit handles method ambiguity in classes

The problem we were facing was that overloaded methods were not
correctly resolved. Only the first occurrence of the method was
inspected.

Thanks for the fix - I forgot about method overloading. I'll try to get a new release out in a few hours after I do some testing.

Available in 0.10.2, which I just pushed to PyPI.

Thanks for the quick turnaround! And although not really related, thanks for putting this together and maintaining it! Works great and helped a lot!


example keyboard.py connect stopped.

I try to connect Logica Master 3,  the process is stopped.
sudo ../venv/bin/python keyboard.py device1.json  usb:0 connect DA:BA:FC:8D:3E:37
DEBUG:asyncio:Using selector: EpollSelector
DEBUG:bumble.transport.usb:loading libusb library at /home/minyeli/gitprjs/blueZ-prjs/bumble/venv/lib/python3.10/site-packages/libusb_package/libusb-1.0.so
DEBUG:bumble.transport.usb:USB Device: Bus 003 Device 005: ID 33fa:0001
DEBUG:bumble.transport.usb:selected endpoints: configuration=1, interface=0, setting=0, acl_in=0x82, acl_out=0x04, events_in=0x81,
DEBUG:bumble.transport.usb:auto-detaching kernel driver
DEBUG:bumble.transport.usb:current configuration = 1
DEBUG:bumble.transport.usb:starting USB event loop
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_RESET_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_RESET_COMMAND
  return_parameters:       HCI_SUCCESS
DEBUG:bumble.drivers.rtk:USB device (33FA, 0001) not in known list
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_READ_LOCAL_SUPPORTED_COMMANDS_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_READ_LOCAL_SUPPORTED_COMMANDS_COMMAND
  return_parameters:
    status:             HCI_SUCCESS
    supported_commands: bfffff03ccffefff3ffffc1ff20fe8fe3ff78fff1c00040061f7ffff7f3800007effffffffffff07000000000000000000000000000000000000000000000000
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_READ_LOCAL_SUPPORTED_FEATURES_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_READ_LOCAL_SUPPORTED_FEATURES_COMMAND
  return_parameters:
    status:      HCI_SUCCESS
    le_features: fff8010000000000
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_READ_LOCAL_VERSION_INFORMATION_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_READ_LOCAL_VERSION_INFORMATION_COMMAND
  return_parameters:
    status:             HCI_SUCCESS
    hci_version:        12
    hci_subversion:     3
    lmp_version:        12
    company_identifier: 2279
    lmp_subversion:     3
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_SET_EVENT_MASK_COMMAND:
  event_mask: ffffffffffffff3f
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_SET_EVENT_MASK_COMMAND
  return_parameters:       HCI_SUCCESS
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_SET_EVENT_MASK_COMMAND:
  le_event_mask: ffffffff00000000
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_SET_EVENT_MASK_COMMAND
  return_parameters:       HCI_SUCCESS
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_READ_BUFFER_SIZE_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_READ_BUFFER_SIZE_COMMAND
  return_parameters:
    status:                                HCI_SUCCESS
    hc_acl_data_packet_length:             1021
    hc_synchronous_data_packet_length:     255
    hc_total_num_acl_data_packets:         9
    hc_total_num_synchronous_data_packets: 4
DEBUG:bumble.host:HCI ACL flow control: hc_acl_data_packet_length=1021,hc_total_num_acl_data_packets=9
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_READ_BUFFER_SIZE_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_READ_BUFFER_SIZE_COMMAND
  return_parameters:
    status:                           HCI_SUCCESS
    hc_le_acl_data_packet_length:     251
    hc_total_num_le_acl_data_packets: 6
DEBUG:bumble.host:HCI LE ACL flow control: hc_le_acl_data_packet_length=251,hc_total_num_le_acl_data_packets=6
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_READ_SUGGESTED_DEFAULT_DATA_LENGTH_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_READ_SUGGESTED_DEFAULT_DATA_LENGTH_COMMAND
  return_parameters:
    status:                  HCI_SUCCESS
    suggested_max_tx_octets: 27
    suggested_max_tx_time:   328
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_WRITE_SUGGESTED_DEFAULT_DATA_LENGTH_COMMAND:
  suggested_max_tx_octets: 251
  suggested_max_tx_time:   2120
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_WRITE_SUGGESTED_DEFAULT_DATA_LENGTH_COMMAND
  return_parameters:       HCI_SUCCESS
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_READ_BD_ADDR_COMMAND
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_READ_BD_ADDR_COMMAND
  return_parameters:
    status:  HCI_SUCCESS
    bd_addr: 04:7F:0E:7C:48:C8/P
DEBUG:bumble.device:BD_ADDR: 04:7F:0E:7C:48:C8/P
DEBUG:bumble.keys:JSON keystore: /root/.local/share/Bumble/Pairing/04-7f-0e-7c-48-c8-p.json
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_WRITE_LE_HOST_SUPPORT_COMMAND:
  le_supported_host:    1
  simultaneous_le_host: 0
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_WRITE_LE_HOST_SUPPORT_COMMAND
  return_parameters:       HCI_SUCCESS
DEBUG:bumble.device:LE Random Address: F0:F1:F2:F3:F4:F5
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_SET_RANDOM_ADDRESS_COMMAND:
  random_address: F0:F1:F2:F3:F4:F5
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_COMPLETE_EVENT:
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_SET_RANDOM_ADDRESS_COMMAND
  return_parameters:       HCI_SUCCESS
DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_EXTENDED_CREATE_CONNECTION_COMMAND:
  initiator_filter_policy: 0
  own_address_type:        RANDOM
  peer_address_type:       RANDOM_DEVICE_ADDRESS
  peer_address:            DA:BA:FC:8D:3E:37
  initiating_phys:         LE_1M_PHY
  LE_1M_PHY.scan_interval:           96
  LE_1M_PHY.scan_window:             96
  LE_1M_PHY.connection_interval_min: 12
  LE_1M_PHY.connection_interval_max: 24
  LE_1M_PHY.max_latency:             0
  LE_1M_PHY.supervision_timeout:     72
  LE_1M_PHY.min_ce_length:           0
  LE_1M_PHY.max_ce_length:           0
DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_STATUS_EVENT:
  status:                  PENDING
  num_hci_command_packets: 1
  command_opcode:          HCI_LE_EXTENDED_CREATE_CONNECTION_COMMAND

I am not know which kind of problem happened.

I found the mouse required encryption link.  But I have no idea how to implement it with Bumble Library. Is there any encryption-related demo for the connection?

All the Bumble connect processes will stop on the HCI_LE_EXTENDED_CREATE_CONNECTION_COMMAND.
I am not sure whether it is caused by the USB dongle or other reasons.
=== Connecting to FC:D0:33:FF:2F:20... DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_CREATE_CONNECTION_COMMAND:   le_scan_interval:        96   le_scan_window:          96   initiator_filter_policy: 0   peer_address_type:       RANDOM_DEVICE_ADDRESS   peer_address:            FC:D0:33:FF:2F:20   own_address_type:        RANDOM   connection_interval_min: 12   connection_interval_max: 24   max_latency:             0   supervision_timeout:     72   min_ce_length:           0   max_ce_length:           0 DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_STATUS_EVENT:   status:                  PENDING   num_hci_command_packets: 2   command_opcode:          HCI_LE_CREATE_CONNECTION_COMMAND

I found the mouse required encryption link. But I have no idea how to implement it with Bumble Library. Is there any encryption-related demo for the connection?

If you need an encrypted connection, you should 1/ pair the two devices that communicate (you can use the pair.py app for that), then, once you've paired and the pairing keys are stored in the key store (you only need to pair once, the key store will remember the keys), you can enable encryption on the connection by calling await connection.encrypt()

All the Bumble connect processes will stop on the HCI_LE_EXTENDED_CREATE_CONNECTION_COMMAND.
I am not sure whether it is caused by the USB dongle or other reasons.
=== Connecting to FC:D0:33:FF:2F:20... DEBUG:bumble.host:### HOST -> CONTROLLER: HCI_LE_CREATE_CONNECTION_COMMAND: le_scan_interval: 96 le_scan_window: 96 initiator_filter_policy: 0 peer_address_type: RANDOM_DEVICE_ADDRESS peer_address: FC:D0:33:FF:2F:20 own_address_type: RANDOM connection_interval_min: 12 connection_interval_max: 24 max_latency: 0 supervision_timeout: 72 min_ce_length: 0 max_ce_length: 0 DEBUG:bumble.host:### CONTROLLER -> HOST: HCI_COMMAND_STATUS_EVENT: status: PENDING num_hci_command_packets: 2 command_opcode: HCI_LE_CREATE_CONNECTION_COMMAND

Most likely here you're trying to connect to a random address that doesn't exist. If the device you're connecting to has LE privacy enabled (most phones and devices would have), their BLE random address changes periodically (typically every 15 minutes or so, plus every time a connection is established). So unless you're paired the devices (in which case the scanner can resolve those changing random addresses back to a fixed one), you'll need to scan and find out the current address of your target device each time before connecting to it.

I tried pair
`sudo ../venv/bin/python pair.py --mode le --sc true --mitm false  device1.json usb:0 DF:85:B6:69:32:A6
<<< connecting to HCI...
<<< connected
=== Connecting to DF:85:B6:69:32:A6...
<<< Connection: Connection(handle=0x0E01, role=CENTRAL, address=DF:85:B6:69:32:A6)
***-----------------------------------
*** Pairing starting
***-----------------------------------
Traceback (most recent call last):
File "...bumble/apps/pair.py", line 404, in pair
await connection.pair()
File "...bumble/venv/lib/python3.10/site-packages/bumble/device.py", line 811, in pair
return await self.device.pair(self)
File "...bumble/venv/lib/python3.10/site-packages/bumble/device.py", line 2661, in pair
return await self.smp_manager.pair(connection)
File "...bumble/venv/lib/python3.10/site-packages/bumble/smp.py", line 1970, in pair
return await session.pair()
File "...bumble/venv/lib/python3.10/site-packages/bumble/smp.py", line 1288, in pair
await self.connection.abort_on('disconnection', self.pairing_result)
asyncio.exceptions.CancelledError: abort: disconnection event occurred.
During handling of the above exception, another exception occurred:
`
I found the mouse will disconnect , and make active connect aagain. (I am not sure for this.)

If runs without connect
sudo ../venv/bin/python pair.py device1.json usb:0 <<< connecting to HCI... <<< connected ^C
Nothing happened.


[Feat]: Buttons 3 in a row and look the same.

Description
Hello, I am using your script in a project of mine.
I would like it if you could use 3 buttons instead of 2.
It would also be good if you could specify the color in the config.
The problem in Germany is that you have to make all buttons the same size and they must not differ in color either.
If you could then get the 3 in a row I would be perfectly happy. currently I've customized it using my own CSS.
Proposed solution
No response
Additional details

Hi @schickermineral,
a 3 button layout  (https://github.com/orestbida/cookieconsent/issues/79) has been implemented in v3. You can check it out in the playground.
Unfortunately there is no plan to bring this to v2.

Love the look of the v3 beta - can't wait until it gets released!
Love that "playground" link. Any possibility of having it spit out the "current" configuration that has been selected? That would allow a sort of WYSIWYG visual configuration!

@Dekker500,

Any possibility of having it spit out the "current" configuration that has been selected

Yes, that's the objective!


Recurbate Extractor returns 403 Error

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[x] I've read about sharing account credentials and I'm willing to share it if required

Region
Europe
Provide a description that is worded well enough to be understood
When trying to run yt-dlp with Recurbate I always get a 403 error. I'm able to play the video in my browser using the same IP with absolutely no problems. Downloading the video via direct url using the developer console also works.
Even when providing the correct login data via cookies.txt the extractor doesn't seem to work properly.
I've tried:

different IPs / proxies
different URLs / videos
different browsers including chrome, chromium and firefox (always used respective cookies)
different parameters etc.

I'm unable to find another recent/open Issue for Recurbate. As I've tried alot of options and since I'm able to play the video in my browser this must be a bug and not a mistake on my side.
Happy to provide more info if needed.
Thanks in advance!
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--verbose', '-vU', '--cookies-from-browser', 'firefox', 'https://recurbate.com/play.php?video=60488255']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.8.10 (CPython x86_64 64bit) - Linux-5.15.0-79-generic-x86_64-with-glibc2.29 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.07.22, mutagen-1.46.0, secretstorage-2.3.1, sqlite3-2.6.0, websockets-11.0.3
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/praes/.mozilla/firefox/wvwksmnk.default-release/cookies.sqlite"
[Cookies] Extracted 374 cookies from firefox
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[Recurbate] Extracting URL: https://recurbate.com/play.php?video=60488255
[Recurbate] 60488255: Downloading webpage
ERROR: [Recurbate] 60488255: This video is only available for registered users; Set your authenticated browser user agent via the --user-agent parameter.. Use --cookies-from-browser or --cookies for the authentication. See  https://github.com/yt-dlp/yt-dlp/wiki/FAQ#how-do-i-pass-cookies-to-yt-dlp  for how to manually pass cookies
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 710, in extract
    ie_result = self._real_extract(url)
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/recurbate.py", line 29, in _real_extract
    self.raise_login_required(msg=SUBSCRIPTION_MISSING_MESSAGE, method='cookies')
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 1170, in raise_login_required
    raise ExtractorError(msg, expected=True)

From the error message:

Set your authenticated browser user agent via the --user-agent parameter

Did you try doing that? It's not in your log

@bashonly
Yeah, I've tried that one with no success.
Output using the --user agent parameter:
[debug] Command-line config: ['--verbose', '-vU', '--cookies-from-browser', 'firefox', '--user-agent', 'firefox', 'https://recurbate.com/play.php?video=60488255']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.8.10 (CPython x86_64 64bit) - Linux-5.15.0-79-generic-x86_64-with-glibc2.29 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.07.22, mutagen-1.46.0, secretstorage-2.3.1, sqlite3-2.6.0, websockets-11.0.3
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/praes/.mozilla/firefox/wvwksmnk.default-release/cookies.sqlite"
[Cookies] Extracted 374 cookies from firefox
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[Recurbate] Extracting URL: https://recurbate.com/play.php?video=60488255
[Recurbate] 60488255: Downloading webpage
ERROR: [Recurbate] 60488255: This video is only available for registered users; Set your authenticated browser user agent via the --user-agent parameter.. Use --cookies-from-browser or --cookies for the authentication. See  https://github.com/yt-dlp/yt-dlp/wiki/FAQ#how-do-i-pass-cookies-to-yt-dlp  for how to manually pass cookies
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 710, in extract
    ie_result = self._real_extract(url)
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/recurbate.py", line 29, in _real_extract
    self.raise_login_required(msg=SUBSCRIPTION_MISSING_MESSAGE, method='cookies')
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 1170, in raise_login_required
    raise ExtractorError(msg, expected=True)

firefox is not a valid user-agent string
it should look like "Mozilla/5.0 (Windows NT 10.0; rv:118.0) Gecko/20100101 Firefox/118.0"
but find your browser's actual user-agent. you can type "my user-agent" into any major search engine

@bashonly
My bad, just tried the correct user-agent:
[debug] Command-line config: ['--verbose', '-vU', '--cookies-from-browser', 'firefox', '--user-agent', 'Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/116.0', 'https://recurbate.com/play.php?video=60488255']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.8.10 (CPython x86_64 64bit) - Linux-5.15.0-79-generic-x86_64-with-glibc2.29 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.07.22, mutagen-1.46.0, secretstorage-2.3.1, sqlite3-2.6.0, websockets-11.0.3
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/praes/.mozilla/firefox/wvwksmnk.default-release/cookies.sqlite"
[Cookies] Extracted 374 cookies from firefox
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[Recurbate] Extracting URL: https://recurbate.com/play.php?video=60488255
[Recurbate] 60488255: Downloading webpage
ERROR: [Recurbate] 60488255: This video is only available for registered users; Set your authenticated browser user agent via the --user-agent parameter.. Use --cookies-from-browser or --cookies for the authentication. See  https://github.com/yt-dlp/yt-dlp/wiki/FAQ#how-do-i-pass-cookies-to-yt-dlp  for how to manually pass cookies
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 710, in extract
    ie_result = self._real_extract(url)
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/recurbate.py", line 29, in _real_extract
    self.raise_login_required(msg=SUBSCRIPTION_MISSING_MESSAGE, method='cookies')
  File "/home/praes/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 1170, in raise_login_required
    raise ExtractorError(msg, expected=True)

Have you visited the site in your browser within the last 30 minutes? The site is Cloudflare-protected, it needs fresh cookies from within the last 30 minutes with a matching user-agent

Keeping this issue open to track it for when curl-cffi support is added.
In the meantime, here is a guide for anyone experiencing this issue:
https://github.com/yt-dlp/yt-dlp/wiki/FAQ#im-getting-http-error-403-and-the-site-has-an-open-issue-on-the-tracker-thats-labeled-cloudflare-related-what-can-i-do

ERROR: [facebook] : Cannot parse data ...
I have been following this discussion. I logged into the facebook page, and executed these commands within 30 minutes:
TRIAL 1:
yt-dlp -vU --cookies-from-browser firefox https://www.facebook.com/727806382/videos/pcb.10161220512376383/621925246461522[debug] Command-line config: ['-vU', '--cookies-from-browser', 'firefox', 'https://www.facebook.com/727806382/videos/pcb.10161220512376383/621925246461522']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-6.2.0-26-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, certifi-2020.06.20, pyxattr-0.7.2, secretstorage-3.3.1, sqlite3-2.6.0
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/kevin/.mozilla/firefox/1tngxtjz.default-release-1/cookies.sqlite"
[Cookies] Extracted 2477 cookies from firefox
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[facebook] Extracting URL: https://www.facebook.com/727806382/videos/pcb.10161220512376383/621925246461522
[facebook]<PHONE_NUMBER>61522: Downloading webpage
[facebook]<PHONE_NUMBER>61522: Downloading webpage
ERROR: [facebook]<PHONE_NUMBER>61522: Cannot parse data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
File "/home/kevin/.local/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 710, in extract
ie_result = self._real_extract(url)
File "/home/kevin/.local/lib/python3.10/site-packages/yt_dlp/extractor/facebook.py", line 707, in _real_extract
return self._extract_from_url(real_url, video_id)
File "/home/kevin/.local/lib/python3.10/site-packages/yt_dlp/extractor/facebook.py", line 654, in _extract_from_url
raise ExtractorError('Cannot parse data')
TRIAL 2:
yt-dlp -vU --cookies-from-browser firefox --user-agent 'Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/116.0' https://www.facebook.com/727806382/videos/pcb.10161220512376383/621925246461522
[debug] Command-line config: ['-vU', '--cookies-from-browser', 'firefox', '--user-agent', 'Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/116.0', 'https://www.facebook.com/727806382/videos/pcb.10161220512376383/621925246461522']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-6.2.0-26-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, certifi-2020.06.20, pyxattr-0.7.2, secretstorage-3.3.1, sqlite3-2.6.0
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/kevin/.mozilla/firefox/1tngxtjz.default-release-1/cookies.sqlite"
[Cookies] Extracted 2477 cookies from firefox
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[facebook] Extracting URL: https://www.facebook.com/727806382/videos/pcb.10161220512376383/621925246461522
[facebook]<PHONE_NUMBER>61522: Downloading webpage
[facebook]<PHONE_NUMBER>61522: Downloading webpage
ERROR: [facebook]<PHONE_NUMBER>61522: Cannot parse data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
File "/home/kevin/.local/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 710, in extract
ie_result = self._real_extract(url)
File "/home/kevin/.local/lib/python3.10/site-packages/yt_dlp/extractor/facebook.py", line 707, in _real_extract
return self._extract_from_url(real_url, video_id)
File "/home/kevin/.local/lib/python3.10/site-packages/yt_dlp/extractor/facebook.py", line 654, in _extract_from_url
raise ExtractorError('Cannot parse data')

@kevirost wrong issue


Enhancement: Collaboration with PKNCA

I just learned of this package, and I'm glad that you're taking it on. I am the primary author of the PKNCA package which I see you're using under the hood. If there are things that would make life easier to have in PKNCA, please let me know (ideally by creating/linking to an issue over there). One specific item that has been on the PKNCA roadmap for a while is to import SDTM (PC and EC/EX) or ADaM (ADPC/ADEX or ADNCA) with less effort (similar to #65).
I'm going to comment on some of the issues where PKNCA already has some support for the desired features and link them here.

FYI, I'm not yet familiar with the working of this project, overall. I will comment on other issues where I think that existing (or desired) PKNCA features can help so that we can more effectively collaborate. Please let me know if I miss the mark about how features are being implemented.

Hi @billdenney thank you so much for joining the team! We welcome any advice you can give to us to help implement the PKNCA functions better, and also would love to collaborate with you. I have sent you a message via the Pharmaverse Slack to see if we can set up a meeting- it would be good to give you a run down of the app and its features so we can collaborate more effectively.

Sounds like a good plan @js3110. I will answer all open suggestions in the issues by @billdenney and we can perhaps also address them deeper during our meeting!


refactor support command to remove promptui, and add test coverage

The support subcommand used promptui, and when introducing tests, I encountered that the race detector was detecting race conditions left and right that I couldn't really resolve.
I refactored the command to now use positional args as inputs. It's less interactive (sorry), but it has 100% test coverage to my eyes.
Signed-off-by: Jose R. Gonzalez<EMAIL_ADDRESS>

Failed make tidy. Just added to my pre-commit hooks.

Coverage increased (+2.2%) to 62.763% when pulling 3064568f35bfabc3eda78541bfe01e39fec5d7f1 on komish:refactor-support-cmd into b4aa0ae77057a288adcbe90f13d7eb31af1c2455 on redhat-openshift-ecosystem:main.


Natural Expression

About

Status

[x] These changes have been tested against Discord API or do not contain API change.
[x] This PR includes only documentation changes, no code change.
[ ] This PR introduces some Breaking changes.

I think the old one was better, cause "without code change" sounds more weird.

ok close pr


Tooling: Can we come up with another way to edit CHANGELOG and schema-next.yaml?

I apologize if this is already tracked somewhere.
If we have a long running PR in this repo and we add the changelog and schema changes early, we have to painstakingly resolve merge conflicts on those over time because we're all editing the same file. Is there any way we could find a pattern that didn't require editing a monolithic file together?
I think the best way would be to have a folder for future changelog and schema changes respectively, which each have a collection of individual files for the changelog and schema changes for the next release. Then the release process would take care of squashing together all the files we have in those folders each release. This would have two benefits:

No more merge conflicts by all editing the same files; each change would have its own file
If things change while you're working on your PR, it is very easy to edit your expected schema and changelog changes
The file will only exist in the folder on main once the PR is merged, meaning if a PR lasts beyond the release it was created for, we don't have to worry about continuously moving the change back up to next when we resolve merge conflicts

However I am open to any other suggestions that would help resolve the toil of resolving merge conflicts in these files.

I have a workaround - I add my change to the changelog not to the end but before the last entry (will not work if you are the first one after release but this happens rare). In this case you will not have a conflict as changes appear in different parts of file.

There's the possibility to automatically create release notes (with change logs) from the commit history: https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes
However, this requires discipline with naming and labeling. Also, it would still be some manual work to get the changelogs from different releases into a single file.

I like how they setup in the collector repo https://github.com/open-telemetry/opentelemetry-collector/blob/main/CONTRIBUTING.md#when-to-add-a-changelog-entry
Perhaps we can do something similar?
For the schema files I'm not sure we can do it in a automated way🤔

I like how they have it in the collector repo

This is what I was hoping as well. It is a much nicer system. I think it would be reasonable to come up with something similar for schema-next.yaml, given that it is already a structured document we could merge other structured documents into the main one. I feel like it's a solvable problem, but there's no existing tooling for it yet.

There's the possibility to automatically create release notes (with change logs) from the commit history

I like this in other repos I'm a part of, however the CHANGELOG.md seems to require significantly more detail than release notes so I'm not sure if there is an easy automated way to encode all of that information into a commit.

I will try and work on adopting https://github.com/open-telemetry/opentelemetry-go-build-tools/tree/main/chloggen in the semconv repo.


Repo size is big (~1.2 GB), need to clean up

@xylar reported that the repo size is about 1.2 GB large, we may need a clean up.

Here's a little bit of info on how to do that:
https://stackoverflow.com/questions/2116778/reduce-git-repository-size

I'm working on #242 and noticed again how huge the repo is.  It takes a long time to build a python package for e3sm_diags because of this.  I just want to make sure it's still on the radar to be fixed sooner or later.

I tried the stackoverflow a while ago and didn't fix the size issue somehow. Yes, this issue is still on my radar. I will try again before our v2.0.0 release. Thanks for the reminder...

Duplicate of #306


Feature group all

Summary
We're considering adding a new feature for the test scenario. Now, if we want to run our test using all the nodes in a certain group, we have to count the number of nodes available in that group and then write it to the toml file.
This feature allows us to directly write the keyword 'max_avail': GROUP_NAME:max_avail, and cloudAI will directly select the maximum number of nodes available in the group in question.
Test Plan
example_test.toml
name = "nccl-test"

[Tests.1]
    name = "nccl_test_all_reduce"
    nodes = [
        'defq:DEFQ_M25:max_avail',
        'defq:DEFQ_M27:2',
        'defq:DEFQ_M29:max_avail'
    ]
    time_limit = "00:03:00"

example_test_2.toml
name = "nccl-test"

[Tests.1]
    name = "nccl_test_all_reduce"
    nodes = [
        'defq:DEFQ_M25:1',
        'defq:DEFQ_M27:2',
        'defq:DEFQ_M29:3'
    ]
    time_limit = "00:03:00"

Based on offline discussions -

This is a good feature to add instead of counting number of nodes manually.
Discussion is required on the right keyword. all is confusing because it would mean we need all the nodes in SU but there will be other jobs running, incl ones from the current test scenario.

There is a ruffer check error about the name of a function. However, this function already existed in the code. Should I ignore this error?

There is a ruffer check error about the name of a function. However, this function already existed in the code. Should I ignore this error?

It is not about the name, your modification increased the complexity of the function, see here for more details. I suggest we extract some blocks to make it easier to read, it will help fixing this issue as well.
Please also add tests for this change.

Discussion is required on the right keyword. all is confusing because it would mean we need all the nodes in SU but there will be other jobs running, incl ones from the current test scenario.

As we discussed, I changed the keyword 'all' to 'max_avail'.

There is a ruffer check error about the name of a function. However, this function already existed in the code. Should I ignore this error?

It is not about the name, your modification increased the complexity of the function, see here for more details. I suggest we extract some blocks to make it easier to read, it will help fixing this issue as well.
Please also add tests for this change.

The function has been split into small blocks.

The function has been split into small blocks.

Thank you! Please don't forget about tests.

@ybenvidia @amaslenn what's left to merge this one?

@ybenvidia please also add example of how to use this feature - ask @TaekyungHeo where to place it

@amaslenn what's left to merge this one?

We need unit tests to cover new behaviour.

Have you had a chance to test it on a Slurm system? If so, please add the test results to the test plan section of this PR.

@amaslenn what's left to merge this one?

We need unit tests to cover new behaviour.

@amaslenn Done

@ybenvidia , we need two approvals for major features. https://docs.google.com/document/d/1fqD_hBXmj0ikXX91iAVGvCLZk2tZdUKqK08TZdT7E0g/edit#heading=h.d8yv92wslwgf

@TaekyungHeo Damn ... I didn't know this PR was considered a major feature.
Sorry about that. I still don't have access to this document. I'll be careful next time.


Clarification on the removal of EnableAsync()

The "enable" step defined by the UPOS spec is done implicitly whenever a particular command requires that the device be enabled. For other peripherals, the caller must ensure that they call EnableAsync() before sending any commands that have an {enable} prerequisite.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hey! We decided to add this remark to the ClaimedLineDisplay class. Thanks for the pull request!


Use O_NONBLOCK for tty stdin.

Signed-off-by: Yang, Wei<EMAIL_ADDRESS>

@xs3c ， first you should create an issue for the PR, then add Fixes: # xxx in you commit content

@xs3c - when creating the matching Issue for this PR, please add details why you need this change :-) thx

@xs3c You need to add proper description in your commit message to explain why the change is needed. Otherwise it looks good to me.

@xs3c any updates? Thx.  Can you add the details? 😄

@raravena80 @bergwolf @grahamwhaley sorry for late, I was busy with other works, I would add it this week.

/test

Another CI fail due to the mount issue @amshinde:
[2]     should be empty [It]
[2]     /tmp/jenkins/workspace/kata-containers-runtime-ubuntu-18-04-PR/go/src/github.com/kata-containers/tests/integration/docker/run_test.go:295
[2]
[2]     Expected
[2]         <string>: [    0.501821] systemd[71]: tmp.mount: Failed to execute command: No such file or directory
[2]         [    0.502146] systemd[71]: tmp.mount: Failed at step EXEC spawning /usr/bin/mount: No such file or directory
[2]     to be empty
[2]

/test

/test-ubuntu

/test-fedora

/test

restarting ci-vsocks and metrics.  I suspect these are just noise and this should be okay.


GO:0015420 ABC-type vitamin B12 transporter activity, reaction update

I just looked at this GO and saw that the reaction does not correspond to what we have in Rhea. The Rhea reaction is RHEA:17873 (EC <IP_ADDRESS>)
an alkylcob(III)alamin(out) + ATP + H2O = ADP + an alkylcob(III)alamin(in) + H+ + phosphate
an alkylcob(III)alamin = CHEBI:140785
@deustp01 In Reactome R-HSA-3000238.1, this GO-term is linked to LMBD1. This protein (according to Swiss-Prot Q9NUN5 is not a transporter but a chaperone that targets ABCD4 to the lysosome (pmid:27456980, pmid:25535791 and pmid:33845046). R-HSA-3000238.1 is linked to RHEA:32832, which is wrong

FYI, RHEA:17874 + EC:<IP_ADDRESS> is now annotated in ABCD4 (internal version) and should be out next release.

Hi @kaxelsen
Is this a Reactome issue?
Because I think current GO mappings are OK, GO:0015420 = EC:<IP_ADDRESS> = RHEA:17873 - right?

The cited reaction in the definition does correspond to REHA:17873 (I looked at QuickGO) so that part is also a GO issue

I fixed the ontology issue. @deustp01 Can you please fix the Reactome annotation?
Thanks, Pascale

Right. We were relying on the earlier observation that mutations in LMBD1 prevent cobalamin transport. The later papers cited at the top of this ticket show that ABCD4 mediates the transport, which is ATP-driven, and that LMBD1 enables ABCD1 localization to the lysosome. So we need to create a new reaction for LMBD1-mediated ABCD1 localization and change the enabler on the existing cobalamin transport reaction.

The Reactome annotations, both for the normal process R-HSA-196741 and a relevant disease counterpart R-HSA-3359462 were fixed a while ago and the fixes have propagated to our public web site (see links), so I will close this ticket.


Wrapping documentation around FAble test : mthodology, findings & next steps

add references on documentation to share back methodologies & results on the last round and have it as a shareable piece of info

@Lu-stack-art to add the documentation link here

Documentation is here https://docs.google.com/presentation/d/1VqghzfD74_7WjejkVySsFMcFxa3BONAZ1zYTA4f98xo/edit#slide=id.g11014e75a59_0_0

Some reviews and edits still need to be done  and we are time boxing the efforst left so we will be good to consider done by end of week


Reproducing Overfitting

Hi, I am trying to reproduce the results on the PCN dataset (i.e., Table 3 in the paper). While I was following the instructions for training GRNet, I got something similar to overfitting. I was wondering if this is normal
"""bash
CUDA_VISIBLE_DEVICES=0,1 bash ./scripts/dist_train.sh 2 13232
--config ./cfgs/PCN_models/GRNet.yaml
--exp_name example
"""

Yes, the loss on trianing and evaluation phase is different. (BTW, why not try AdaPoinTr or PoinTr)

Thanks for answering the question! That said, if the curve looks fine, what is the potential problem that makes it overfitting?
What I was trying to do is to reproduce GRNet, PCN, and PoinTr results using the PCN dataset (i.e., Table 3 in the paper) but couldn't achieve a comparable performance for GRNet and PCN using the default config files. Is there anything I missed?
Yes, I did try AdaPoinTr and PoinTr and they were looking good.


The SRTG part may won't be update

As the code and paper show, if TG gate keep close,oprations on the input of pool and lstm part won't move forward. And it means when we do back  propagation, parameters of lstm part and TG gate won't be updated, which  can easily happen. So how  did you solve the problem? or did I missing something?

Yes, the main mindset is to only use information that is cyclic-consistent. Therefore, the LSTM parts of the network are only updated if and only if they are consistent with the spatially pooled version of the activations. (You can think of this as worse-case scenario with only using the backbone CNN). For most experiments, the gates were inactive for the first 30~50 epochs, thus the weights of the LSTM would not be completely random when the gates are indeed active for the rest of the training.
Best,
Alex

Yes, the main mindset is to only use information that is cyclic-consistent. Therefore, the LSTM parts of the network are only updated if and only if they are consistent with the spatially pooled version of the activations. (You can think of this as worse-case scenario with only using the backbone CNN). For most experiments, the gates were inactive for the first 30~50 epochs, thus the weights of the LSTM would not be completely random when the gates are indeed active for the rest of the training.
Best,
Alex

Ha, the result seems very reasonable. Thanks for your reply and your work(both the code and paper) did inspire me much.


85 update analyzers to use the new trace call stack graph implementation

What does this PR do?
Update the open source HTA analyzers to use the new trace call stack implementation. This PR makes the following updates:

Refactor CuptiCounterAnalysis to use the new Trace CallGraph implementation.
Update CriticalPathAnalysis to use the new implementation.
Update get_frequent_cuda_kernel_sequences to use the new implementation.
Add back CallStackGraph.dfs_traverse() method.
Correct the assertion on the value of botton_level_op in test_get_cupti_counter_data_with_operators().
Update infer_device_type() to use pid and id columns in addition to the stream column.
Move the unit tests for CuptiCounterAnalysis and CriticalPathAnalysis to separate test files.

Before submitting

[X] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[ ] N/A

[ ] Did you write any new necessary tests?

[X] N/A

[ ] Did you make sure to update the docs?

[x] N/A

[] Did you update the changelog?

[X] N/A

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (c1235b4) 91.35% compared to head (324f3a3) 89.99%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #86      +/-   ##
==========================================
- Coverage   91.35%   89.99%   -1.36%
==========================================
  Files          38       41       +3
  Lines        3619     3668      +49
==========================================
- Hits         3306     3301       -5
- Misses        313      367      +54

Files
Coverage Δ

hta/analyzers/critical_path_analysis.py
90.56% <100.00%> (ø)

hta/analyzers/cupti_counter_analysis.py
94.33% <100.00%> (+0.68%)
:arrow_up:

hta/common/trace_call_stack.py
87.66% <100.00%> (+1.44%)
:arrow_up:

tests/test_trace_analysis.py
100.00% <100.00%> (+0.37%)
:arrow_up:

hta/common/trace_call_graph.py
91.35% <90.90%> (+0.59%)
:arrow_up:

tests/test_cupti_profiler.py
96.00% <96.00%> (ø)

hta/analyzers/cuda_kernel_analysis.py
83.33% <93.33%> (-7.65%)
:arrow_down:

hta/common/types.py
91.66% <87.50%> (-8.34%)
:arrow_down:

tests/test_critical_path_analysis.py
98.23% <98.23%> (ø)

hta/utils/checker.py
46.66% <46.66%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


MM-T386- Invite new user to closed team with "Allow only users with a specific email domain to join this team" set to "sample.mattermost.com"

Summary
This test creates a closed team, ensure that only users with a 'sample.mattermost.com' email domain can join the team, send out an invite, verify the email and make sure that the new user is successfully able to join the team.
Test case: https://automation-test-cases.vercel.app/test/MM-T386

Thanks @saturninoabril for the feedback! I created a file with the helper functions that you mentioned, and added some that I thought would be useful since they are being reused in several places. Please let me know if I should change anything

/update-branch

We don't have permissions to update this PR, please contact the submitter to apply the update.

@josephk96 Would you mind rebasing and updating this PR so I can merge it? That would be great! Thanks in advance!

Hi @prapti , thanks for the review!
I synced and merged this PR = :)


Contributing Guidelines

I wrote a simple CONTRIBUTE.md from the Python project and adapted it to Nim.
I advise for the use of snake_case for file_names, while I recommend camelCase and PascalCase for objects to follow NEP1 specification of the language.
Third party modules is an empty file, but I add it in case, we need third party modules in the future.
I removed references to Gitter and check that the Discord link is correct in each file.
I added a MIT License similar to any of TheAlgorithms project with the correct year.
The README.md is minimalistic. If you have any enhancement that I can do, it might be better to take care of this in another PR, just so we show that the repository is already open to contributions.

At this point Nim-specific coding style requirements are listed in three separate paragraphs: Algorithms should, Coding Style and Other Requirements for Submissions. I suggest resolving the reviews above and reorganize the paragraphs.
I propose:

Algorithm implementation requirements  # for general stuff
Nim coding style # language-specifics
Submission requirements
Optional suggestions

I am not done, it is a little bit too soon to merge it. It still lacks:

a better document structure
the use of Option
a relaxation on variable naming. We need to define use cases.
Effect system: use of funcs and raises pragma
Use of unittest and runnableExamples
I think it also lacks links to Nim resources like Nim 2.0 documentation.

I did not find it mentioned in the guidelines but I realized that the site needs to have a way to understand which file solves this algorithm (in order to associate the various implementations of the algorithms together) so I guess names for algorithm are standardized, correct? I assume the Python version should be used as reference for name of algorithm? I guess this should be mentioned somewhere.

Should we enforce the use of the export marker *?
It enables to distinguish the helper functions from those that really solves the problem and are part of a user API.
Also, I noticed this sentence in the original CONTRIBUTING.md:
"Algorithms should be packaged in a way that would make it easy for readers to put them into larger programs.".

Which version of the compiler is better?
Should we simply recommend to compile against the stable version?
Should we put the Nim version in source code?

Learning materials should definitely target stable.
Marking code with the compiler version numbers just adds another thing to manage. Ideally, automated testing should make it unnecessary.
Exporting the main function is fine as a suggestion, in my opinion. No harm if it's omitted.

I think titles and document structure can still be greatly enhanced. I have put every information in there. I have tried to follow @ZoomRmc recommandation, but it is not easy to separate Nim information from general recommandation. Especially from a non-native English speaker like me.

What's the third_party_modules.md file for?

That's for optional packages not in the standard library.

Well @pietroppeter approved these changes right before those last small markdown fixes.
I would like to get @ZoomRmc review before merging just in case.
It is indeed important to merge it quickly so that the repository has a front page and to attract more contributors.
Specific refinements of CONTRIBUTING.md can be made later.


I need check if tag to be used before delete her

Actually exists any configuration?

You can search for tagged objects If you don’t find any then you delete it.
Article.tagged_with("ruby").any?


Bugs on Android 10

https://ibb.co/hfLSJFT
Hi, i want to report because the app doesn't run at all on my phone.
Device: Realme RMX1821
OS: Android 10
App version: 2.0.5

Go to the about tab and click "Reconfigure for Android 11 (or similar, forget the exact wording)".

Fixed 2.0.7


'More action' button is not actionable using Keyboard while it is actionable using 'mouse'

Describe the issue

Open designer standalone (nx serve).
Search for an action.
Hover over the tooltip (i) icon -- observe tooltip opens up. Observe no way to open the tooltip with keyboard.
Select an action.
On the action page, hover over the (i) icon, observe that tooltips opens up. Observe no way to open the tooltip with keyboard.

Are you using new designer or old designer
New Designer (With action panels)
Screenshots or Videos
On mouse hover, tooltip pops up. No way to access using keyboard.
On search results:

On action selection:

Browser
Edge
Additional context
Related internal work item: https://msazure.visualstudio.com/One/_workitems/edit/17554171
AB#25639340

It gets read by the screen reader when the keyboard is over the operatoin

Based on my testing with nvda screen reader, looks like the tooltip text is not being read by screen reader. Is it possible the result is different with different screen readers?
Example screenshots of screen reader output below

Could be, I use the screen reader built into MacOS as well as ChromeVOX for testing and both of those read it. Being on MacOS, I don't have access to NVDA or JAWS so if it's not working on those, I may need your help with fixing it.


NStringTypeHandler handles only String and not NString

The NStringTypeHandler seems to only handle strings as the NString methods are commented out. What is the right solution for handling Unicode characters when the NStringTypeHandler is written in this way?

I suspect that the calls to the NString methods comment out was due to those calls were introduced in Java 6 and the type handler was done when mybatis still supported older versions of Java.  Now that mybatis requires Java 6, it seems safe to make the correct NString based calls.

Thank you both for the report!
I will uncomment the lines.


[🐞] useTask$ not available for import

Which component is affected?
Qwik Runtime
Describe the bug
When trying to import useTask$ per the documentation, this function does not seem to be available. Should I rely fully on useResource? Love using Qwik btw.
'useTask$' is declared but its value is never read.ts(6133)
Module<EMAIL_ADDRESS>has no exported member 'useTask$'.

Reproduction
npm create qwik@latest
Steps to reproduce
import { useTask$ } from<EMAIL_ADDRESS>System Info
Apple M1 Arm64 via vsCode

Additional Information
No response

thanks for the report @shaunsurfing !
the version that useTask$ is part of is not released yet, so the docs will be reverted to the old lifecycle hooks for now
soon it'll be released and will be available to import and to use
thanks for the awareness!

looks like we wrote this in the same time @jordanw66 😅
Thanks!

Thanks both! Amazing responsiveness. Can't wait for the next release. Will be covering much of Qwik on the new YT channel.

sure thing @shaunsurfing !  and great to hear!
make sure to add it to the media page once it'll be ready

@shaunsurfing v0.16.0 which was just released contains the expected functionality 🎉  have fun trying it out 💪

@zanettin amazing timing! Was just about to do a separate video series with Solid JS first but now with this working I can use Qwik instead!

@shaunsurfing yeah! sounds fantastic! go wild and please share the video 😍

This was a bit messy! sorry for all the inconvenience!
New 0.16.1 already has the useTask$() API!

No problem at all! The video will be up today. Really pleased with Qwik.
On Mon, 19 Dec 2022 at 12:17, Manu MA @.***> wrote:

This was a bit messy! sorry for all the inconvenience!
New 0.16.1 already has the useTask$() API!
—
Reply to this email directly, view it on GitHub
https://github.com/BuilderIO/qwik/issues/2444#issuecomment-1357583827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKY5CFJYFDHUWNCCHFBKSKDWOBG6BANCNFSM6AAAAAAS7QYC54
.
You are receiving this because you were mentioned.Message ID:
@.***>


skate.js.org doesn't work

in Chrome, Version 54.0.2840.41 beta (64-bit)
Just the following error in the console:
Uncaught TypeError: Cannot assign to read only property 'customElements' of object '#<Window>'
    at http://skate.js.org/dist/index-with-deps.min.js:10:13938

Thanks for reporting this, it looks like there's something wrong with some polyfill used by the library, probably this? https://github.com/skatejs/skatejs.github.io/blob/master/package.json#L42

This should be fixed once it's updated to use the latest skatejs-web-components version. Safari 10 should also be fixed shortly (https://bugs.webkit.org/show_bug.cgi?id=160331).

+1 site shows up blank on iPhone

I've been trying to find time to work on this. I've got it mostly fixed.
The problem with Safari is Safari's fault. The have a pretty nasty bug I
reported before release and they didn't fix it before it was released to
the public: https://bugs.webkit.org/show_bug.cgi?id=160331. We've tried
hacking a fix in the latest skatejs-web-components but it only seems to fix
itnin certain scenarios and seems to still be broken on the website. It's
fixed in the latest tech preview, so hopefully it goes out in their next
updates.
I'll try and get the site working today.
On Fri., 14 Oct. 2016, 00:34 Simon Tsvilik<EMAIL_ADDRESS>wrote:

+1 site shows up blank on iPhone
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/skatejs/skatejs/issues/836#issuecomment-253514211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIVbB2W1qvs2cRS7tyzLBYoWKI2-NLTks5qzjNUgaJpZM4KPSI3
.

Wow, that bug is nasty. Did any of the old IE-style recalc hacks work i.e. zoom:1, or toggling display from none to initial?

I've fixed the Safari issue in skatejs-web-components. The other issues were because of a syncing issue that I've worked around in the router.
I've disabled the todo list example for the time being because it requires #863 be solved. The workarounds don't work due to spec limitations.


Missing compile step in release.yml

Describe the bug
When doing a release the current version of release.yml is missing a compilation of the java code before trying to create a docker image.
We need to add a mvn compile or mvn test step after checking out the repository and before running docker build.
The current Dockerfile expects compiled classes in target/classes folder.
In the future we might switch to a multistage docker build file to be able to build everywhere and using jlink or graalvm for faster startup times but at the moment we don't.

I tried the release action in a separate repository to prepare for tomorrow @phteffe and found that we might need a mvn compile step in the release.yml file before we can build docker images with some content in them. :)


Panic in autorest/adal/token.go

I'm getting a panic in go-autorest when trying to use NewAuthorizerFromEnvironment (source) from the keyvault/auth package. This happens if I do not set up any environment variables before, so I would expect it to just fail with a 401.
It takes 30 seconds before it crashes, so I'm guessing there's some timeout somewhere which doesn't get aborted on auth failures?
Versions
go-autorest: v10.11.2
azure-sdk-for-go: v17.3.0
Repro case
package main

import (
	"context"
	"log"

	"github.com/Azure/azure-sdk-for-go/services/keyvault/2016-10-01/keyvault"
	"github.com/Azure/azure-sdk-for-go/services/keyvault/auth"
)

func newKeyVaultClient() (keyvault.BaseClient, error) {
	var err error
	c := keyvault.New()
	c.Authorizer, err = auth.NewAuthorizerFromEnvironment()
	if err != nil {
		log.Printf("Failed to create Azure authorizer: %v\n", err)
		return c, err
	}

	return c, nil
}

type VaultKey struct {
	VaultURL string
	Name     string
	Version  string
}

func Encrypt(data string, key VaultKey) error {
	c, err := newKeyVaultClient()
	if err != nil {
		return err
	}
	p := keyvault.KeyOperationsParameters{Value: &data}

	// Crash here
	res, err := c.Encrypt(context.TODO(), key.VaultURL, key.Name, key.Version, p)
	log.Println(res)
	return err
}

func main() {
	log.Fatal(Encrypt("foo", VaultKey{
		VaultURL: "https://my-vault-url.vault.azure.net",
		Name:     "test-key",
		Version:  "a2a690a4fcc04166b739da342a912c90",
	}))
}

Stack trace
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x825680]

goroutine 1 [running]:
github.com/Azure/go-autorest/autorest/adal.retry(0xd18f80, 0xc420205dd0, 0xc420214500, 0x0, 0x0, 0xc764b9)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/adal/token.go:887 +0x1a0
github.com/Azure/go-autorest/autorest/adal.(*ServicePrincipalToken).refreshInternal(0xc42019d200, 0xd21c00, 0xc420028178, 0xc60e96, 0x17, 0x0, 0x0)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/adal/token.go:814 +0x27f
github.com/Azure/go-autorest/autorest/adal.(*ServicePrincipalToken).EnsureFreshWithContext(0xc42019d200, 0xd21c00, 0xc420028178, 0x0, 0x0)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/adal/token.go:704 +0x1c8
github.com/Azure/go-autorest/autorest.(*BearerAuthorizer).WithAuthorization.func1.1(0xc420214300, 0xc4201bb050, 0x8466ba, 0xd19920)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/authorization.go:118 +0x100
github.com/Azure/go-autorest/autorest.PreparerFunc.Prepare(0xc4202463e0, 0xc420214300, 0xc4201bb098, 0x846647, 0xd19920)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/preparer.go:52 +0x30
github.com/Azure/go-autorest/autorest.WithNothing.func1.1(0xc420214300, 0x2, 0xd19920, 0xc420246400)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/preparer.go:93 +0x39
github.com/Azure/go-autorest/autorest.PreparerFunc.Prepare(0xc420246400, 0xc420214300, 0x2, 0xd19920, 0xc420246400)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/preparer.go:52 +0x30
github.com/Azure/go-autorest/autorest.Prepare(0xc420214300, 0xc4201bb300, 0x2, 0x2, 0x0, 0x0, 0xdf8475800)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/preparer.go:85 +0x8f
github.com/Azure/go-autorest/autorest.Client.Do(0xd180c0, 0xc420230470, 0xd18f60, 0xc420205da0, 0x0, 0x0, 0xdf8475800, 0xd18c2e2800, 0x3, 0x6fc23ac00, ...)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/client.go:207 +0x133
github.com/Azure/go-autorest/autorest.DoRetryForStatusCodes.func1.1(0xc420214300, 0xd19960, 0xc420075f40, 0x0)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/sender.go:225 +0x11e
github.com/Azure/go-autorest/autorest.SenderFunc.Do(0xc420075f40, 0xc420214300, 0xc4201bb760, 0x1, 0x1)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/sender.go:38 +0x30
github.com/Azure/go-autorest/autorest.SendWithSender(0xd198c0, 0xc4200ea600, 0xc420214300, 0xc4201bb760, 0x1, 0x1, 0x0, 0x0, 0xb226e0)
        /$GOPATH/src/github.com/Azure/go-autorest/autorest/sender.go:80 +0x75
github.com/Azure/azure-sdk-for-go/services/keyvault/2016-10-01/keyvault.BaseClient.EncryptSender(0xd180c0, 0xc420230470, 0xd18f60, 0xc420205da0, 0x0, 0x0, 0xdf8475800, 0xd18c2e2800, 0x3, 0x6fc23ac00, ...)
        /$GOPATH/src/github.com/Azure/azure-sdk-for-go/services/keyvault/2016-10-01/keyvault/client.go:1085 +0x10c
github.com/Azure/azure-sdk-for-go/services/keyvault/2016-10-01/keyvault.BaseClient.Encrypt(0xd180c0, 0xc420230470, 0xd18f60, 0xc420205da0, 0x0, 0x0, 0xdf8475800, 0xd18c2e2800, 0x3, 0x6fc23ac00, ...)
        /$GOPATH/src/github.com/Azure/azure-sdk-for-go/services/keyvault/2016-10-01/keyvault/client.go:1041 +0x2f9
main.Encrypt(0x70815b, 0x3, 0x712052, 0x21, 0x709592, 0x8, 0x71163b, 0x20, 0x0, 0x6659d8)
        /tmp/azure-auth-crash.go:37 +0x20a
main.main()
        /tmp/azure-auth-crash.go:43 +0x59
exit status 2

Panic doesn't repro on Windows but it does on Mac, investigating further...

Here's what I have so far.  Lack of environment vars means NewAuthorizerFromEnvironment will default to case number four which is MSI authentication; in this case the authorizer will attempt to connect to a locally-running server to obtain a token.  Since the server isn't running the TCP dialer fails to make a connection so sender.Do here returns a nil response with a url.Error; this error wraps a net.OpError, and on Windows that wraps a os.SyscallError however on Mac it wraps a poll.TimeoutError.  For Windows SyscallError.Temporary() returns false however on Mac TimeoutError.Temporary returns true (see here).  As a result, when running on Mac we think this is a temporary error and attempt to dereference a nil response.
I will fix the nil dereference however the rest is IMO a defect in the standard library.

No panic on Linux, must be specific to Mac.

This has been fixed in https://github.com/Azure/go-autorest/releases/tag/v10.11.3

@jhendrixMSFT Hm, I'm actually on Linux, though, so definitely not Mac only. I'll see if it works better on Linux too with the fix

With the fix applied I do not get a panic, but a timeout trying to reach the metadata endpoint (testing on my laptop, so expected that it cannot be reached)


Update Android Things support library version to 1.0

Hi.
I thought I had, maybe it is in progress on my desktop and then I would have gotten distracted (months ago).
I will look at it later today.

Done, looks like I did it and then didnt commit. Also submitted to androidthings.withgoogle.com (also did the pca9685 driver).


'cloud_firestore/CloudFirestorePlugin.h' file not found. | Flutter run & build --release works fine. But archiving gives me many errors.

I'm still kinda new in the app development. And this is my first flutter app I want to publish in the Play & Appstore.
But I'm keep getting this error. When trying to archive the app to the appstore.

I made a flutter app which works just fine when running
flutter run --release   flutter build ios --release
But when archiving I keep getting this error. I have seen a few other issues for this error, but all the provided solutions do not work.
This is my flutter doctor:
`flutter doctor -v
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.15.1 19B88, locale en-NL)
• Flutter version 1.9.1+hotfix.6 at /Users/lau/Developer/flutter
• Framework revision 68587a0916 (3 months ago), 2019-09-13 19:46:58 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[✗] Android toolchain - develop for Android devices
✗ Unable to locate Android SDK.
Install Android Studio from: https://developer.android.com/studio/index.html
On first launch it will assist you in installing the Android SDK components.
(or visit https://flutter.dev/setup/#android-setup for detailed instructions).
If the Android SDK has been installed to a custom location, set ANDROID_HOME to that location.
You may also want to add it to your PATH environment variable.
[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.2.1, Build version 11B500
• CocoaPods version 1.8.4
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/setup/#android-setup for detailed instructions).
[✓] VS Code (version 1.40.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.6.0
[✓] Connected device (1 available)`
For the last three days I have been cleaning flutter, reinstalling cocoapods, changing the podfile and so on. But no improvements.
I'm (kinda) sure it has to do with the podfile since that is what everybody is saying. But I dont know how to fix it.
Please help!

NOTE:
The app works perfectly on Android and iOS devices.

Found a solution!
It is nothing with the podfile or flutter!
You should open the  project/ios/Runner.xcworkspace directly from the Finder. Do not open the iOS folder completely with xcode, since you will not have the right settings.


build(deps): bump sub-dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
508

Relevant Lines:
508

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
508

Relevant Lines:
508

💛  - Coveralls


Question about the process of ViT-Comer

Thank you for your excellent work. I have a small question here. In the forward function of class ViTCoMer in vit_comer.py, there is a deform operation on input features. Why do we need this operation before patch embedding? Could you explain this operation in more detail? Thank you!

@ShirleySyh It will be used in MSDeformAttention modules, more details are as follows:
https://github.com/Traffic-X/ViT-CoMer/blob/1dd3ada6e3568626c5b17a3fbb732dd8c06a39f3/detection/mmdet_custom/models/backbones/comer_modules.py#L188

Thank you for your reply. I have another question. In your paper in Section 3.3, features from ViTs are added to the F4 to get F'4 and then F3, F'4, F5 are unified through Multi-Scale Deformable-Attention. However, in class CTI_toV in Line 351 https://github.com/Traffic-X/ViT-CoMer/blob/main/detection/mmdet_custom/models/backbones/comer_modules.py, it seems that features from ViTs are added to the multi-scale features after the multi-scale faetures pass through the Multi-Scale Deformable-Attention. Features from ViTs are not unified with multi-scale CNN features, which is different from the discription in paper. I'm not sure if it's my misunderstanding of the code or paper. Looking forward to your answer. Thank you!


[POC] feat: added async getSession method

What kind of change does this PR introduce?
This is a proof of concept for a new async getSession() method. This allows users to get a valid session (not expired) as opposed to the synchronous session() method, which may return an expired session.
Additional context
I know we're planning to discuss this further, but since I already had most of the code, I thought I'd throw something together quickly. It will hopefully make our discussion clearer if we have something to look at!
Once gotrue-js has a method like this (or similar) it will allow supabase-js to work something like:
async function getAccessToken() {
    const {session} = await auth.getSession()

    return session?.access_token ?? null
}

const postgrestClient = new PostgrestClient({ getAccessToken, ... })

Then whenever postgrestClient is about to send a query, it can call the getAccessToken() method, which will return an up-to-date token.
Note: While this isn't a breaking change in gotrue-js (purely additive), it will be a breaking change once we start doing the above in supabase-js, as some users may rely on the first request possible sending an expired token. We may consider ignoring this use case though, as it is a strange one!

@alaister instead of getAccessToken in postgrest-js and storage-js, wdyt about using this approach instead?
// SupabaseClient constructor
this.rest = new PostgrestClient(url, {
  ...,
  fetch: async (input, init) => {
    const headers = init?.headers ?? {}
    const accessToken = await getAccessToken()
    if (accessToken) {
      headers['Authorization'] = `Bearer ${accessToken}`
    }
    return this.fetch(input, { ...init, headers })
  },
})

@soedirgo That works too!
Did you see this: https://github.com/supabase/storage-js/pull/67? It essentially makes all of the headers async. What are your thoughts on the benefits/drawbacks of all three approaches?

Ah yup, that works as well.
I think the only real drawback of .getAccessToken()/.getHeaders() is that it's kind of specific to us, and it feels weird for people using postgrest-js/storage-js directly that headers are set using a callback.
Meanwhile the .fetch() override kinda stretches the functionality of custom fetch - it's not really how it's meant to be used.

I agree that a custom fetch is the most flexible, but you're right does feel a bit dirty.
Personally, I could go either way between the custom fetch and async getHeaders. One thing nice about getHeaders is we can expose it to users who want to get an access token some other way, and it feels like a first-party solution. That being said, we can just document how to use the custom fetch like that without adding any additional API surface area.
@inian would love your thoughts here too.


ForInputTypes should be IEnumerable

Add validation to assert valid types? What is a nice way to validate that only types deriving from UIElement and ContentElement are used, IIRC those are the types that works with Validation?

Maybe a TypeConverter


Convert Enumerable.Sum to a for[each]

The compiler can optimize a for loop over an array very well, but cannot optimize LINQ (and LINQ does not have overloads to special handle arrays today). This results in CallVirt methods that cannot be inlined and add additional performance cost (this can be seen in AzProfiler logs internally).

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ndrwrbgs sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@doubleyewdee this seems like a reasonable thing to merge and release. Any ETAs when it will be worthwhile ;) ?