chore: Migrate Python to new directory structure

It would be better if #170, #193, and #195 landed before this change.
 14 files changed, 0 insertions(+), 0 deletions(-)
 rename arrays/python/count-inversions.py => algorithms/Python/arrays/count_inversions.py (100%)
 rename {linked-lists/Python => algorithms/Python/linked_lists}/doubly.py (100%)
 rename {linked-lists/Python => algorithms/Python/linked_lists}/singly.py (100%)
 rename {multiplication/python => algorithms/Python/multiplication}/karatsuba.py (100%)
 rename scheduling/python/interval-scheduling.py => algorithms/Python/scheduling/interval_scheduling.py (100%)
 rename searching/python/binary-search.py => algorithms/Python/searching/binary_search.py (100%)
 rename searching/python/jump-search.py => algorithms/Python/searching/jump_search.py (100%)
 rename searching/python/linear-search.py => algorithms/Python/searching/linear_search.py (100%)
 rename sorting/python/bubble-sort.py => algorithms/Python/sorting/bubble_sort.py (100%)
 rename sorting/python/insertion-sort.py => algorithms/Python/sorting/insertion_sort.py (100%)
 rename sorting/python/quick-sort.py => algorithms/Python/sorting/quicksort.py (100%)
 rename sorting/python/selection-sort.py => algorithms/Python/sorting/selection_sort.py (100%)
 rename {strings/python => algorithms/Python/strings}/palindrome.py (100%)
 rename {strings/python => algorithms/Python/strings}/word_count.py (100%)

Could you add the Readme file?

No.  That is a bad idea because the performance difference would be negligible yet we would create blind spots.  For example, if we create a .github/workflows/write_readme.py script, the tests in that code would never be run.  I find that assumptions about where code will exist in a repo six months in the future are usually wrong.

I put paths: '**.py' in #209

Nice


translations PR 20171127

Ideally this PR should be review by community members for each language.
Note: In the event that you see an english translation instead of the desired language is because this PR was submitted before the translation was made on the Mattermost Translation Server and probably will be submitted for the next PR.

ja LGTM!

Closing in favor of https://github.com/mattermost/mattermost-webapp/pull/402


Serialize List of objects without Root Element?

Consider the following classes:
public class Course
{
    public string Title { get; set; }
    public List<Student> Students { get; set; }
}

public class Student
{
    public string Name { get; set; }
}

The desired output is the following xml:
<Course>
  <Title>Engineering</Title>
  <Student>
    <Name>John Doe</Name>
  </Student>
  <Student>
    <Name>Jane Doe</Name>
  </Student>
   ...
</Course>

Serilizing an instance of a course object generates xml with Students as wrapper element. I tried adding [XmlElement("Student")] to the collection property but it did not work. It still generates a wrapping Students element.
The generated xml also has the following namespaces, even though I enabled implicit typing with EnableImplicitTyping. How do I tell the serializer to remove the namespaces?
xmlns:exs="https://extendedxmlserializer.github.io/v2" xmlns:ns1="clr-namespace:MyProject;assembly=MyProjects" xmlns:sys="https://extendedxmlserializer.github.io/system"
I really do not want to perform post serialization hacks (already thought about using string.replace :) )

Looks like we're 1 for 2 here, @umutesen.  I have added an example showing implicit typing working as expected.  My hunch is that you did not add the List<Student> which was adding all those attributes for you.
The other one is trickier.  It's basically a "default content" property that is applied per object.  System.Xaml has this via the ContentPropertyAttribute and XML does indeed (just verified) have this for XML via the XmlElement as prescribed above.
Unfortunately the v2 collections engine is a bit of a mess.  We're writing v3 now and it should be a lot easier to implement something like this.  Let's see if we can get it in there.

Thank you for creating an example, however, the example seems to test against an output that has <Students></Students> wrapper element. I only require the individual students to be shown without a wrapping <Students></Students> element. Please refer to the desired output in my question.
I am looking forward to the v3 release! :)

Indeed, what I meant by being 1 for 2, @umutesen.  Meaning we can accommodate for one of your requests at the moment (implicit typing) but not the other (default content). 😄


⚠ SolarOscp has degraded performance

In 54fca33, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 304 ms

Resolved: SolarOscp performance has improved in 478b0ee.


bug: timesync is not available in Pop!_OS any longer

Pop!_OS has replaced timesyncd with chrony, causing Ansible to fail.
See: https://github.com/pop-os/systemd/pull/25

:tada: This issue has been resolved in version 1.11.22 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Integrate typeorm + implmenting findAll() function

"@nestjs/typeorm": "^9.0.1",
"typeorm": "^0.3.9",

Branch development is not finished yet, and I'm still confused about some parts of integrating.
This will take more time than planned.


fix(version): update fabric8-stack-analysis-ui to 0.1.27

There were the following issues with your Pull Request

Commit: fd4ca374f68dff70d65e6f00d556ffb52b611a35
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


docker installation not working

Tried to build the android app, but the instructions in the docker installation were raising errors
`ERROR: Analysis of target '//mediapipe/examples/android/src/java/com/google/mediapipe/apps/objectdetectiongpu:objectdetectiongpu' failed; build aborted: no such package '@local_config_git//': Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/4884d566396e9b67b62185751879ad14/external/org_tensorflow/third_party/git/git_configure.bzl", line 64
_fail(result.stderr)
File "/root/.cache/bazel/_bazel_root/4884d566396e9b67b62185751879ad14/external/org_tensorflow/third_party/git/git_configure.bzl", line 14, in _fail
fail(("%sGit Configuration Error:%s %...)))
Git Configuration Error: Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/4884d566396e9b67b62185751879ad14/external/org_tensorflow/tensorflow/tools/git/gen_git_source.py", line 38, in
import six
ModuleNotFoundError: No module named 'six'
INFO: Elapsed time: 1.988s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages l
oaded, 0 targets configured)`

Please make the following changes in the Dockerfile. With those changes, I can successfully build MediaPipe Android APKs in a docker container. We will push out those changes in the coming release.
--- a/Dockerfile
+++ b/Dockerfile
@@ -30,6 +30,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
         unzip \
         python \
         python-pip \
+        python3-pip \
         libopencv-core-dev \
         libopencv-highgui-dev \
         libopencv-imgproc-dev \
@@ -42,9 +43,10 @@ RUN apt-get update && apt-get install -y --no-install-recommends \

 RUN pip install --upgrade setuptools
 RUN pip install future
+RUN pip3 install six

 # Install bazel
-ARG BAZEL_VERSION=0.26.1
+ARG BAZEL_VERSION=1.1.0
 RUN mkdir /bazel && \
     wget --no-check-certificate -O /bazel/installer.sh

Here is the command I run in the docker container to build one of the APKs.
root@d13456f98941:/mediapipe# bash ./setup_android_sdk_and_ndk.sh
root@d13456f98941:/mediapipe# bazel build -c opt --config=android_arm64 mediapipe/examples/android/src/java/com/google/mediapipe/apps/objectdetectiongpu:objectdetectiongpu

# It should print:
# INFO: From Executing genrule @com_github_glog_glog//:logging_h [for host]:
# select bazel-out/host/bin/external/com_github_glog_glog/src/glog/logging.h.tmp to be the glog logging.h file.
# INFO: From Executing genrule @com_github_glog_glog//:config_h [for host]:
# select bazel-out/host/bin/external/com_github_glog_glog/config.h.tmp to be the glog config file.
# Target //mediapipe/examples/android/src/java/com/google/mediapipe/apps/objectdetectiongpu:objectdetectiongpu up-to-date:
#   bazel-bin/mediapipe/examples/android/src/java/com/google/mediapipe/apps/objectdetectiongpu/objectdetectiongpu_deploy.jar
#   bazel-bin/mediapipe/examples/android/src/java/com/google/mediapipe/apps/objectdetectiongpu/objectdetectiongpu_unsigned.apk
#   bazel-bin/mediapipe/examples/android/src/java/com/google/mediapipe/apps/objectdetectiongpu/objectdetectiongpu.apk
# INFO: Elapsed time: 154.928s, Critical Path: 73.21s
# INFO: 2003 processes: 1 local, 1844 processwrapper-sandbox, 158 worker.
# INFO: Build completed successfully, 2072 total actions

Thanks for the quick response. I love this framework more and more!


Link error when using firestore on Win32 desktop

Please fill in the following fields:
Pre-built SDK from the website or open-source from this repo: website
Firebase C++ SDK version: 6.14.0
Firebase plugins in use (Auth, Database, etc.): Firestore
Additional SDKs you are using (Facebook, AdMob, etc.):
Platform you are using the C++ SDK on (Mac, Windows, or Linux): Windows
Platform you are targeting (iOS, Android, and/or desktop): desktop
Please describe the issue here:
(Please list the full steps to reproduce the issue. Include device logs, Unity logs, and stack traces if available.)
I'm trying to use the firestore in our project, unfortunately, I always get the link error when call to firestore function. For example
extern "C" int common_main(int argc, const char* argv[]) {
  App::Create();
  Firestore* db = Firestore::GetInstance();

  return 0;
}

This error is happening in both debug/release mode like bellow:
2>firebase_firestore.lib(8e190590a5ffbe491c9e3724aeefbb9b_fsdep_absl_time_internal_cctz_time_zone.dir_Debug_time_zone_info.obj) : error LNK2001: unresolved external symbol "class std::unique_ptr<class absl::time_internal::cctz::ZoneInfoSource,struct std::default_delete<class absl::time_internal::cctz::ZoneInfoSource> > (__cdecl* absl::time_internal::cctz_extension::zone_info_source_factory)(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class std::function<class std::unique_ptr<class absl::time_internal::cctz::ZoneInfoSource,struct std::default_delete<class absl::time_internal::cctz::ZoneInfoSource> > __cdecl(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &)> const &)" (?zone_info_source_factory@cctz_extension@time_internal@absl@@3P6A?AV?$unique_ptr@VZoneInfoSource@cctz@time_internal@absl@@U?$default_delete@VZoneInfoSource@cctz@time_internal@absl@@@std@@@std@@AEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@5@AEBV?$function@$$A6A?AV?$unique_ptr@VZoneInfoSource@cctz@time_internal@absl@@U?$default_delete@VZoneInfoSource@cctz@time_internal@absl@@@std@@@std@@AEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@2@@Z@5@@ZEA)
2>E:\INDI\3rd\quickstart-cpp\firestore\testapp\build\Debug\desktop_testapp.exe : fatal error LNK1120: 1 unresolved externals

Could you take a look and show me any additional steps I need to do?
Please answer the following, if applicable:
Have you been able to reproduce this issue with just the Firebase C++ quickstarts ? Yes
What's the issue repro rate? 100%

Sorry to hear about your issue. Could you please provide some additional details about your environment?

which version of Visual Studio are you using?
is your build configuration 32 bit or 64 bit?
are you building with /MD or /MT?

Hello @var-const
I'm using Visual Studio 2017 & 2019 to check for this issue. VS 2019 is using as mainly develop an environment for our product.
The above error is happening on:

2019 : 32 bit /MD, 32 bit /MT, 64 bit /MD. I'm getting the other link error with 64 bit /MT is 1>firebase_app.lib(8b06a1cf33ae3e301a1a0b6a784c9d54_flatbuffers.dir_Debug_idl_parser.obj) : fatal error LNK1143: invalid or corrupt file: no symbol for COMDAT section 0x615D. To fix the COMDAT section error, we re-built the library on VS 2019, but in this check, we use the Pre-built SDK from your website, no custom at all. Do you have the plan to add the support for VS 2019?

2017 : 32 bit /MD, 32 bit /MT.

Let me know if you need any additional information.

Hello
I'm trying to compiling the SDK with VS2019 to check if the issue still here. I have some questions, could you take your time to check them?

I don't find the 6.14.0 tag. Is it ok to use the master branch on head revision?
firebase_firestore-d.lib size is only 22MB instead of 390MB since comparing with the one I download from your website. As a result, I get a lot of linking errors when trying to use firebase_firestore lib in our application.  While checking some of them, I find it belongs to some library as (firebase_firestore_core.lib, firebase_firestore_util.lib, ...). I don't find them from CPack Packages. Is there any automation way to have them linked properly inside firebase_firestore lib?

Look forward to seeing your reply.

I have the same issue with this unresolved absl_time_internal symbol.
I am using VS2015 64bit /MD libraries.
The previous linker errors (absl_string_internal related in 6.12) are gone though. The MacOS libraries do not have this issue: they now work fine without having to link any extra absl libraries.
The public git repository is several months old now, and the latest tag is still 6.11. Are you planning on keeping the public repository more up-to-date with development?

A few things:

You should be able to use the master branch at HEAD to build at any time. It may be slightly out-of-date with the most recent releases but we'll get it updated soon and are working on updating it more frequently moving forward.
When you build the open-source version of the SDK, you'll get much smaller libraries than the binary distribution because the dependencies (flatbuffers, grpc, absl, leveldb, etc.) will be linked in separately via cmake. So don't worry if the library size ends up much smaller, ultimately it ends up being the same total library size with all the deps.
The absl time_internal linker issue should be fixed in the 6.14.0 binary release. Are you seeing it in that version?

The absl time_internal linker issue should be fixed in the 6.14.0 binary release. Are you seeing it in that version?

Yes, it is the 6.14.0 binary release. The MacOS build works fine, the windows version has the error above.
The 6.12.0 binary release had a linker error with absl_string_internal. That one is fixed in both windows and MacOS.

Thanks for the info. I believe we've found the root cause. It should be fixed in the upcoming 6.15.0 release.

I'm glad to hear you found the root cause. Could you share with us when 6.15.0 will come? Are there any chances to see VS2019 supported by pre-build SDK?

Did anyone succeed in using the precompiled 6.14.0 or 6.14.1 SDK on windows? I am unable to resolve the unresolved symbols. I tried building several versions of abseil and link the resulting absl_time* libraries, but none of them help.

For those who are interested: I had to actually add absl/time/internal/cctz/src/zone_info_source.cc (and necessary include path) because the symbol is not exported by the abseil C++ library that I built.

The unresolved symbols on Windows will be fixed in the upcoming version of the C++ SDK, 6.15.0, which should be released soon. Sorry for the trouble!

Hi all!
The 6.15 SDK is out now (and pushed to this repository). Has this issue been adequately resolved?

It works for me on both MacOS and Windows. I haven't had any issues since the release, thanks!
Good to know that the public repository was brought up to date.

Thanks for the update. I'll close this issue, but feel free to comment on the closed thread if the linker errors regress.


Fixed height and vertical scroll bar

It's more a question than a bug, I want to know if there are some configurations to manage a fixed height, showing a blank space if there are only a few elements in the side bar, and showing a vertical scroll bar from there if there are to many.
In my test the timeline height grow to always show all the sidebar elements. Handling the scroll from outside don't like me since the horizontal scroll at the bottom them was hidden. I check the "rows" property from auto to an specific value. That slice the sidebar, but the rest was not printed at all, and the vertical scrollbar don't show up like I expected.
If there are a better way to just make questions like this one, please let me know.
Thanks

Hi,
It would be difficult to meet your request as it would require significant changes to the DOM structure of the current timeline instance.
As a hint for our approach, I would suggest that you do a DOM update like the code below.
$("#myTimeline").Timeline("initialized", function (self) {
    let $container   = $(self).find('.jqtl-container'),
        $sidebar     = $(self).find('.jqtl-side-index'),
        $rulerTop    = $(self).find('.jqtl-ruler-top'),
        $rulerBottom = $(self).find('.jqtl-ruler-bottom'),
        $eventContainer = $(self).find('.jqtl-event-container'),
        rowHeight    = $eventContainer[0].clientHeight / $sidebar.children('.jqtl-side-index-item').length,
        rowCounter   = 0
    $(self).parent().css({overflow: 'hidden'})
    $(self).css({overflow: 'hidden'})
    $container.css({overflowY: 'auto', overflowX: 'hidden'})
    $sidebar.children().each(function(){
        if ($(this).hasClass('jqtl-side-index-margin')) {
            $(this).css({ minHeight: (rowCounter == 0 ? $rulerTop.height() : $rulerBottom.height()) + 'px', boxSizing: 'border-box' })
        } else {
            let itemHeight = $(this).height()
            itemHeight = rowCounter % 2 == 0 ? itemHeight + 1 : itemHeight
            if (itemHeight > rowHeight) {
              itemHeight = Math.floor(rowHeight)
            }
            $(this).css({ height: itemHeight + 'px', lineHeight: itemHeight + 'px', boxSizing: 'border-box' })
        }
        rowCounter++
    })
})

This is a way to force overflowing container elements to be updated during instance initialization. However, it requires recalculating and setting the height of the elements in the object, and adjusting them would be very painstaking.
Thank you,


[AutoPR] iothub/resource-manager

Created to accumulate context: iothub/resource-manager

(message created by the CI based on PR content)

@lmazuel please let me know when this can be merged.

@AutorestCI rebase

Rebase done and pushed to the branch

Codecov Report

Merging #2324 into master will increase coverage by 0.07%.
The diff coverage is 73.59%.

@@            Coverage Diff             @@
##           master    #2324      +/-   ##
==========================================
+ Coverage   54.58%   54.65%   +0.07%
==========================================
  Files        5457     5499      +42
  Lines      127938   128501     +563
==========================================
+ Hits        69840    70238     +398
- Misses      58098    58263     +165

Impacted Files
Coverage Δ

...zure/mgmt/iothub/models/iot_hub_sku_description.py
100% <100%> (ø)
:arrow_up:

...zure/mgmt/iothub/models/feedback_properties_py3.py
100% <100%> (ø)

...b/azure/mgmt/iothub/models/operation_inputs_py3.py
100% <100%> (ø)

...b/azure/mgmt/iothub/models/iot_hub_client_enums.py
100% <100%> (ø)
:arrow_up:

...mt/iothub/models/cloud_to_device_properties_py3.py
100% <100%> (ø)

...e/mgmt/iothub/models/cloud_to_device_properties.py
100% <100%> (ø)
:arrow_up:

...t/iothub/models/storage_endpoint_properties_py3.py
100% <100%> (ø)

...e-mgmt-iothub/azure/mgmt/iothub/models/resource.py
100% <100%> (ø)
:arrow_up:

...re/mgmt/iothub/models/iot_hub_quota_metric_info.py
100% <100%> (ø)
:arrow_up:

.../mgmt/iothub/models/iot_hub_sku_description_py3.py
100% <100%> (ø)

... and 123 more

Continue to review full report at Codecov.

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


French typography

A space is needed before the exclamation mark in French (see https://en.wikipedia.org/wiki/Exclamation_mark#French).

👍


reviewer-guide

Create a reviewer guide with tips that really helped me to give better reviews.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed, 4 unresolved discussions (waiting on @SchSeba)

docs/reviewer-guide.md, line 21 at r1 (raw file):

* Reasonable info messages

### Other things:

Except for the very first suggestion (insist on tests, that could as well go into the previous "Good to check" section), other entries are quite debatable / open to interpretation / cultural and probably don't belong to written document. I would add though something along the lines of "be compassionate, don't nit-pick, don't block merges for nice-to-have enhancements, ask for a follow-up instead of PR update if possible" if we leave the section.

docs/reviewer-guide.md, line 24 at r1 (raw file):

* Insist on unit and functional tests to avoid regressions (the core case must be tested)
* Prefer readability over code-reuse

You lost me here. Code reuse doesn't hinder readability if done right. If anything, it means less code to review / read / put bugs in.

docs/reviewer-guide.md, line 25 at r1 (raw file):

* Insist on unit and functional tests to avoid regressions (the core case must be tested)
* Prefer readability over code-reuse
* Prefer domain-expression over code-reuse (if something looks 80% the same but is for something completely different, give it its own room; composition is thereore better than inheritence or if/else)

This doesn't sound very specific / clear to me. Can you share an example of what you mean? (Actually, maybe giving / referring to actual examples could help the reader.)

docs/reviewer-guide.md, line 26 at r1 (raw file):

* Prefer readability over code-reuse
* Prefer domain-expression over code-reuse (if something looks 80% the same but is for something completely different, give it its own room; composition is thereore better than inheritence or if/else)
* Prefer to only implement what is needed, Don't plan for the possible future.

Agreed with the first part (hence I often suggest that some change is out of scope of a PR and should belong to a separate one); the message of the second part is a bit controversial though and may be read / interpreted in unhelpful mode.

Comments from Reviewable

I removed the other things section. ( move the unit and functional tests to the other section)
I agree they probably don't belong to a written document


💡[FEATURE]: Predicting Stock Splits and Dividends

Is your feature request related to a problem? Please describe.
Create models to predict stock splits and dividend announcements based on historical data and company performance.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@Akshat111111 pls assign it to me


AmplitudeJS v6.0.0

Working on the latest and greatest AmplitudeJS. In version 6.0.0, we are optimizing a lot of the internal features, making the development process more user friendly, and working to make AmplitudeJS more extendable.
Project
https://github.com/orgs/serversideup/projects/4/views/1
Interactive Elements
Below are all the elements we can interact with in Amplitude version 6.0.0.
Audio Tracker Element

[ ] Documented
[x] Global
[x] Collection
[x] Audio
[x] Audio + Collection

Buffered Progress Element

[ ] Documented
[x] Global
[x] Collection
[x] Audio
[x] Audio + Collection

Container Elements

[ ] Documented
[x] Audio
[x] Collection Audio

Meta Data Elements

[ ] Document: Active Audio Meta Data
[ ] Document: Active Collection Info Meta Data
[ ] Document: Active Collection Audio Meta Data
[ ] Document: Non-Active Audio Meta Data
[ ] Document: Non-Active Collection Audio Meta Data
[ ] Document: Non-Active Collection Info

Mute Elements

[ ] Documented
[x] Global

Next Elements

[ ] Documented
[x] Global
[x] Collection

Pause Elements

[ ] Documented
[x] Global
[x] Collection
[x] Audio
[x] Audio + Collection

Playback Speed Elements

[ ] Documented
[x] Global

Play Elements

[ ] Documented
[x] Global
[x] Collection
[x] Audio
[x] Audio + Collection

Play/Pause Elements

[ ] Documented
[x] Global
[x] Audio
[x] Collection
[x] Audio + Collection

Previous Elements

[ ] Documented
[x] Global
[x] Collection

Progress Elements

[ ] Documented
[ ] Global
[ ] Collection
[ ] Audio
[ ] Audio + Collection

Repeat Audio Elements
Repeat Elements
Shuffle Elements

[ ] Documented
[ ] Global
[ ] Collection

Skip To Elements

[ ] Documented
[x] Audio
[ ] Audio+Collection

Stop Elements

[ ] Documented
[ ] Global

Time Elements

[ ] Documented
[ ] Global Current Time
[ ] Audio Current Time
[ ] Collection Current Time
[ ] Audio + Collection Current Time
[ ] Global Duration
[ ] Audio Duration
[ ] Collection Duration
[ ] Audio + Collection Duration

Volume Down Elements

[ ] Documented
[x] Global

Volume Slider Elements

[ ] Documented
[x] Global

Volume Up Elements

[ ] Documented
[x] Global

Breaking Changes
songs array renamed to audio
https://github.com/serversideup/amplitudejs/issues/504
default_album_art renamed to default_artwork
Making all of the internal verbiage more generic to handle all audio types. (See #504)
amplitude-prev classes renamed to amplitude-previous
It's more verbose and we don't use abbreviations anywhere else.
amplitude-song-slider classes renamed to amplitude-audio-tracker
It makes more sense with the verbiage of the industry and we remove the song token word that allows for better developer experience and flexibility.
Collection Scoping
There were a lot of methods we had that wouldn't make sense outside the scope of a collection. So some of the old functionality that would work on the songs array has been scoped only to a collection. For example:

Next
Previous
Shuffle
These won't work on just a songs (now audio) array. They will only work when scoped to a collection. This will need to be thoroughly documented and in our migration guide.

Documentation Updates
Development Commands
We now have two commands to build AmplitudeJS:

npm run dev - Compiles and starts the watcher command. The mode is defaulted to development with WebPack. This makes it really efficient to develop and watch for changes.
npm run build - Runs a production build and minifies the library

New time_format for config.
Replaces:
HH -> hours
MM -> minutes
SS -> seconds
Tests To Write

[ ] Initialize AmplitudeJS with no audio
[ ] Initialize AmplitudeJS with config url
[ ] Initialize AmplitudeJS with JSON

Hi guys. I've been looking for a player to integrate into a new project. Really love the look and feel of Amplitude but I notice there haven't been any commits in 2 years. Is the project no longer being actively developed?


Group by more types

Signed-off-by: Veeupup<EMAIL_ADDRESS>group by value supoort Int64 / UInt64 / String

@ywqzzy for now, reviewers just need to approve the code and mergify will auto squash commits and merge pr into the main branch : D


Pinned Tags not showing if left-menu is not pinned

Pinned Tags only visilable, if the left menu-bar ist pinned. (picture 1).
If the left menu-bar is not "pinned" the pinned tags are not showing (picture 2).

"""
there we have the salad. its not a bug its a feature.
"""

@B0F1B0
Thank you very much for opening this bug. This is frontend bug, so you opened the ticket in the right repository.

As of 2.1-alpha version the sidebar does not open on mouse hover. Instead sidebar is opened/closed only when user clicks sandwitch menu:

This bug is obsolete.


feat(backend): recover unfinished iterations when starting the server

Codecov Report

Merging #472 (45fdc30) into main (491858f) will decrease coverage by 0.27%.
The diff coverage is 26.92%.

@@            Coverage Diff             @@
##             main     #472      +/-   ##
==========================================
- Coverage   81.68%   81.41%   -0.27%
==========================================
  Files          63       63
  Lines        4182     4208      +26
==========================================
+ Hits         3416     3426      +10
- Misses        766      782      +16

Impacted Files
Coverage Δ

...hestrator/core/state_api/orchestrator_state_api.py
92.45% <20.00%> (-1.76%)
:arrow_down:

label_sleuth/orchestrator/orchestrator_api.py
65.08% <25.00%> (-1.09%)
:arrow_down:

label_sleuth/app.py
59.83% <100.00%> (+0.06%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


#[UIC-1213] Geomap (based on leaflet) is coming on the top of every RVF component

When we want to edit a dashboard with leaflet map on top-right corner, then 'edit dashboard' options are coming behind leaf let map
We are not getting loading icon at Leaf Let Map in explore mode
When we scroll a dashboard with Map Visualisation on top row, then dashboard title is shadowed by the map

(question) Instead of adding inline styles to MAP_CONTAINER wouldn't it be cleaner to add a css class and add the styles inside this class ?

Thanks @patrickracicot  for spending time to review
We discussed the same amongst the team about this fix few days back, and decided that we should fix the problem generically for the whole framework. Right now, OSS (apache) have hard coded all the styles inline. The intent is to contribute the whole feature of applying the css via css classes mechanism for better extendibility so that whole community can take the leverage of it.


pull request for e2e test at 01/13/2017 07:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Develop Jobs()

class should take values from a form (maybe create new route "/jobs/new"?), scrub the values, and create a Jobs() object that can be stored in db.

Jobs() added in this morning's commit


XNXX.com - ERROR: Unable to extract video URL

[x] I've verified and I assure that I'm running youtube-dl 2018.03.10
[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

Full verbose output:
-bash-4.1# youtube --version
2018.03.10

-bash-4.1# youtube -v -j http://www.xnxx.com/video-i7eij1a/non_nude_sexy_teen_tease
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-j', u'http://www.xnxx.com/video-i7eij1a/non_nude_sexy_teen_tease']
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2018.03.10
[debug] Python version 2.6.6 (CPython) - Linux-2.6.32-042stab108.5-x86_64-with-centos-6.9-Final
[debug] exe versions: ffmpeg 2.6.8, ffprobe 2.6.8
[debug] Proxy map: {}
ERROR: Unable to extract video URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube/youtube_dl/YoutubeDL.py", line 785, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube/youtube_dl/extractor/common.py", line 440, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube/youtube_dl/extractor/xnxx.py", line 32, in _real_extract
    webpage, 'video URL')
  File "/usr/local/bin/youtube/youtube_dl/extractor/common.py", line 796, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract video URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description of issue
youtube-dl outputs ERROR: Unable to extract video URL;  errors when trying to download or find links on any xnxx.com videos.
Video used in this example - http://www.xnxx.com/video-i7eij1a/non_nude_sexy_teen_tease
Seems to be the case for all videos I've tested on the site

Can reproduce.


Octave comment block

It looks as though the large comment block doesn't appear to recognize the key characters to initiate a comment when highlighting with octave.

Looking at the octave documentation, it can be done two ways.
Thanks!

Thanks for the review, both types of block comments are now recognized.


fix(pkg/plugin): copy plugins directly to the data directory

Copy plugins from the cache rather than create a symlink.
fixes: #7206

@marckhouzam You wanna take a look at this and verify whether it solves #7206?

@bacongobbler tested and working

Will this be in v3.2.1 release?


Fix the dummy player agent name

This is a minor fix for the dummy player agent name.

We actually have a Player.java too. But didn't commit it not to spoil the contest.
You are right, we should rename the dummy to get a playable offline version.


Using VPN in SWA

Hello,
Is it possible to use Static Web App on VPN?
Because I can't find the network configuration in Azure for SWA.
Thanks,
Escudero

Hi @escudero how are you looking to restrict the SWA? Are you looking to restrict what ip's can access it? Or are you looking to be able to only access it through a particular Azure virtual network?

Hi @miwebst,
Only through a particular Azure virtual network.

This will be achievable when private endpoint support is ready which should be around ~1-2 months from now.


Add In-Memory DB Implementation for Addressable

Fix #2286
Implement previously noop functionality for the memory.Client's methods
for dealing with persisting contract.Addressable types, and
memory.Client constructor.
Update memory.Client methods with noop implementation to panic rather
than return empty data. The reason is that the tests are using the newly
implemented constructor to create instances of the client for easier
maintainability. Previously, we were panicking at the constructor to
protect users from creating an instance of the memory.Client. Now we are
panicking at when an unimplemented method is invoked.
Add tests covering newly added implementation.
Signed-off-by: Anthony M. Bonafide<EMAIL_ADDRESS>

Closing this issue since we have decided to do away with the in memory implementation. See #2340 for more details


deps: trying perfmark 0.23.0. DO NOT MERGE

Trying to see the impact of perfmark 0.23.0 in dependencies.

Unit tests and integration tests succeeded.
The failing checks are about upper bound dependency check. This doesn't cause problem to library users. Good.


Translate in .ts file -> Cannot perform operation, astro-i18n is not initialized.

Describe the bug
I cannot load a translation from a .ts file - something the previous i18n implementation allowed.
Do I have to change this behaviour? Suggestions on how to do it?
I tried to find the minimum attack surface for my project - it is attached.
To Reproduce
Steps to reproduce the behavior:

Clone Repo
Start Astro
Visit Page

Expected behavior
It should work ;)
Mandatory reproduction repository
https://github.com/Nkay/i18n-bug-reproduce-2

your repo is private

oh yes, sorry - corrected.

This is happening because the typescript files runs before the middleware.
But even if the translations were loaded you would still have a problem because the current route is set in the middleware.
Without the information on the current route you could not access page translations.
When using astroI18n outside a template wrap it in a getter, you can retrieve the value once it's properly intialized :
// old
t("my_key")
// new
() => ("my_key")

I will try to document this and add methods to be able to initialize astroI18n outside the middleware.
I'll leave this open until then.

I investigated a bit, the TS modules runs before everything because of vite which apparently initializes every module before the file that imports it runs.
This is really annoying, I also want astro-i18n to be used seamlessly in TS modules but this will need some rework to the core functionality.
Thanks for pointing out this issue, I'm thinking about a good way of solving it. For the moment you can wrap the calls inside getters.

Hi Alexandre,
I changed the occurences (they were mostly in a link element, in the nav and the footer) and so it was only about an hour of work to change it from link.asd -> t(link.asd), and now it fully works again.
Thanks for taking the time and having a look into it!

@perinazzoo If you are using the translations in the template the workaround is using getters instead. Otherwise as of now there is no workaround, as explained before, even if the translations were loaded you wouldn't be able to access the current route's translations because astro-i18n cannot detect it since Astro loads all modules before the request comes in.
This is not a simple fix, first I have to make sure that the library is initialized before the module runs, then I need to add a way to fetch a translation from any group/page and I will probably also need to make every call to the t function return a getter instead.
This issue is on my priority list for sure and I will fix it as soon as I can, I'm however very busy at the moment and I can't afford to work on this, if this is a deal-breaker I'm sorry but you will have to wait.


Mismatches between sample metadata and BIOM table are causing (small?) problems with filtering

https://github.com/biocore/songbird/blob/be66273953f561cc23ab9e1438ba7e1af75ce98a/songbird/util.py#L130-L167
So: if table.ids(axis='sample') contains IDs that aren't in the metadata file, then the metadata file will still include rows for those sample IDs, albeit filled with NaN values.
This makes both metadata_filter() and the first part of sample_filter() effectively not work, since these will be true for every ID in the table (after running the first line with metadata = metadata.loc[table.ids(axis='sample')], at least).
Proof of this with a small DataFrame:
>>> df = pd.DataFrame({"col1": [1, 2, 3], "col2": [6, 7, 8]}, index=["a", "b", "c"])
>>> df.loc['a'] # works as expected
col1    1
col2    6
Name: a, dtype: int64
>>> df.loc['d'] # also works as expected: throws a KeyError (d isn't in df's index)
... long stack trace goes here ...
>>> df = df.loc[['a', 'b', 'd']] # wait, what?
>>> df
    col1   col2
a    1.0    6.0
b    2.0    7.0
d    NaN    NaN

>>> 'd' in df.index # this is what metadata_filter() is doing
True

I don't think this will throw off results (I'd imagine tensorflow can ignore NaNs satisfactorily), but I'm not sure.
I don't currently have the bandwidth to work on a fix for this, but might be able to do so in a week.
I've been thinking recently anyway that it might be worth taking some time to make a library that basically mimics qiime2.Metadata without actually requiring the user to have Q2 installed. This would enable us to handle metadata, do matching with BIOM tables, etc. a lot more confidently in standalone versions of Q2 plugins (as well as in general tool development). Because pandas is great, but sometimes it does some weird stuff.

Hmm. I'm not yet convinced that this would throw an error. If the ids
aren't in the metadata, then they should be filtered out of the table via
metadata_filter.
Do you have a minimal example where this does fail?
On Thu, May 9, 2019, 11:19 PM Marcus Fedarko<EMAIL_ADDRESS>wrote:

https://github.com/biocore/songbird/blob/be66273953f561cc23ab9e1438ba7e1af75ce98a/songbird/util.py#L130-L167
So: if table.ids(axis='sample') contains IDs that aren't in the metadata
file, then the metadata file will still include rows for those sample IDs,
albeit filled with NaN values.
This makes both metadata_filter() and the first part of sample_filter()
effectively not work, since these will be true for every ID in the table
(after running the first line with metadata =
metadata.loc[table.ids(axis='sample')], at least).
Proof of this with a small DataFrame:

df = pd.DataFrame({"col1": [1, 2, 3], "col2": [6, 7, 8]}, index=["a", "b", "c"])>>> df.loc['a'] # works as expected
col1    1
col2    6
Name: a, dtype: int64>>> df.loc['d'] # also works as expected: throws a KeyError (d isn't in df's index)... long stack trace goes here ...>>> df = df.loc[['a', 'b', 'd']] # wait, what?>>> df
col1   col2
a    1.0    6.0
b    2.0    7.0
d    NaN    NaN
'd' in df.index # this is what metadata_filter() is doingTrue

I don't think this will throw off results (I'd imagine tensorflow can
ignore NaNs satisfactorily), but I'm not sure.
I don't currently have the bandwidth to work on a fix for this, but might
be able to do so in a week.
I've been thinking recently anyway that it might be worth taking some time
to make a library that basically mimics qiime2.Metadata without actually
requiring the user to have Q2 installed. This would enable us to handle
metadata, do matching with BIOM tables, etc. a lot more confidently in
standalone versions of Q2 plugins (as well as in general tool development).
Because pandas is great, but sometimes it does some weird stuff.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/songbird/issues/55, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA75VXJAAF22FYXJ25IEIIDPUTSURANCNFSM4HL7RAZQ
.

Sorry for not following up on this. I don't have a good excuse for forgetting about this, except "grad school". (While I was already investigating #98, I figured I should take time to address this also.)

If the ids aren't in the metadata, then they should be filtered out of the table via metadata_filter.

This is the problem -- the line of code in match_and_filter() that reads
metadata = metadata.loc[table.ids(axis='sample')]

will insert every sample ID in the table into the metadata, resulting in the not-originally-in-the-metadata sample IDs having a row in the metadata DataFrame containing all NaN values. It's straight up impossible for metadata_filter() (or the first clause in sample_filter()'s if statement) to return False.
I don't really know if this throws a concrete error, and it honestly doesn't really matter. What matters is that the metadata DataFrame is incorrectly set1, and this is (potentially) a problem for the output correctness -- or at minimum this means that the code is misleading, since it's not doing what it seems like it should be doing.
1 To be less dramatic, this problem is limited to the case where the metadata samples are a subset of the table samples. Now that I think about it, I think this is usually the opposite way around with most studies I've worked with.
Example

Do you have a minimal example where this does fail?

Sure. Here are a minimal sample metadata TSV file and BIOM table, respectively.
sample_metadata.txt
mt.biom.txt
The metadata contains sample IDs Sample1, Sample2, Sample3, Sample5, Sample6, Sample7 (note the lack of a Sample4).
The table contains sample IDs Sample1, Sample2, Sample3, Sample4, Sample5, Sample6 (note the presence of Sample4 but the lack of Sample7).
Here is code that documents this bug using the above datasets (in multiple places I've straight up copied from Songbird's match_and_filter() in order to mimic the behavior of that function exactly).
>>> import biom
>>> import pandas as pd
>>> # Load in the data
>>> table = biom.load_table("mt.biom.txt")
>>> metadata = pd.read_csv("sample_metadata.txt", sep="\t", index_col=0)
>>> # What's in the table?
>>> table.ids(axis="sample")
array(['Sample1', 'Sample2', 'Sample3', 'Sample4', 'Sample5', 'Sample6'],
      dtype=object)
>>> # Looks good. What's in the metadata?
>>> metadata
           Metadata1  Metadata2  Metadata3
Sample-ID
Sample1            1          2          3
Sample2            4          5          6
Sample3            7          8          9
Sample5           13         14         15
Sample6           16         17         18
Sample7           19         20         21
>>> # Still looking good. What happens when we try to subset
>>> # the metadata in the exact same way Songbird's code does?
>>> metadata = metadata.loc[table.ids(axis='sample')]
           Metadata1  Metadata2  Metadata3
Sample-ID
Sample1          1.0        2.0        3.0
Sample2          4.0        5.0        6.0
Sample3          7.0        8.0        9.0
Sample4          NaN        NaN        NaN
Sample5         13.0       14.0       15.0
Sample6         16.0       17.0       18.0
>>> # So, although we filtered out Sample7 (in metadata but not in table),
>>> # we actually *inserted* Sample4 (in table but not in metadata).
>>> # This means that stuff like metadata_filter() won't work the way we
>>> # expect it to.
>>> def metadata_filter(val, id_, md):
...    return id_ in metadata.index
>>> supposedly_filtered_table = table.filter(metadata_filter, axis='sample')
>>> supposedly_filtered_table.ids(axis="sample")
array(['Sample1', 'Sample2', 'Sample3', 'Sample4', 'Sample5', 'Sample6'],
      dtype=object)
>>> # Yeah, we didn't filter the table at all.

So that it's easily rerunnable, I've included a copy of the above code without commentary/formatting here:
minimal_code_to_reproduce.py.txt
Solution
I think if you take the set union of the table and metadata IDs, and filter to that (instead of just throwing all table IDs into metadata.loc[], this seems to solve the problem:
>>> metadata.loc[set(table.ids(axis="sample")) & set(metadata.index)]
           Metadata1  Metadata2  Metadata3
Sample-ID
Sample6           16         17         18
Sample3            7          8          9
Sample2            4          5          6
Sample5           13         14         15
Sample1            1          2          3
>>> # The metadata is filtered correctly now!
>>> # That being said it's out of order, so if this is a problem I guess it'll require
>>> # re-sorting in some way.

TL;DR
ANYWAY sorry for the 6-month delay in explaining this bug report. Usually I'm faster with this sort of stuff!!!
I honestly do not have the time to personally submit a PR for this, but I think implementing the proposed solution above shouldn't be too terrible. As mentioned, I think this should be a pretty rare bug, but 1) it does provably exist in Songbird's code, and 2) I think it'd be a good idea to resolve it sooner rather than later.
Thanks!

Ahh. Yes that is a bug - we should be doing a set intersection rather.
Thanks for reporting! This should be an easy fix.
On Fri, Nov 15, 2019, 12:35 AM Marcus Fedarko<EMAIL_ADDRESS>wrote:

Sorry for not following up on this. I don't have a good excuse for
forgetting about this, except "grad school". (While I was already
investigating #98 https://github.com/biocore/songbird/issues/98, I
figured I should take time to address this also.)
If the ids aren't in the metadata, then they should be filtered out of the
table via metadata_filter.
This is the problem -- the line of code in match_and_filter() that reads
metadata = metadata.loc[table.ids(axis='sample')]
will insert every sample ID in the table into the metadata, resulting
in the not-originally-in-the-metadata sample IDs having a row in the
metadata DataFrame containing all NaN values. It's straight up impossible
for metadata_filter() (or the first clause in sample_filter()'s if
statement) to return False.
I don't really know if this throws a concrete error, and it honestly
doesn't really matter. What matters is that the metadata DataFrame is
incorrectly set1, and this is (potentially) a problem for the output
correctness -- or at minimum this means that the code is misleading, since
it's not doing what it seems like it should be doing.
1 To be less dramatic, this problem is limited to the case where the
metadata samples are a subset of the table samples. Now that I think about
it, I think this is usually the opposite way around with most studies I've
worked with.
Example
Do you have a minimal example where this does fail?
Sure. Here are a minimal sample metadata TSV file and BIOM table,
respectively.
sample_metadata.txt
https://github.com/biocore/songbird/files/3849884/sample_metadata.txt
mt.biom.txt
https://github.com/biocore/songbird/files/3849886/mt.biom.txt
The metadata contains sample IDs Sample1, Sample2, Sample3, Sample5,
Sample6, Sample7 (note the lack of a Sample4).
The table contains sample IDs Sample1, Sample2, Sample3, Sample4, Sample5,
Sample6 (note the presence of Sample4 but the lack of Sample7).
Here is code that documents this bug using the above datasets (in multiple
places I've straight up copied from Songbird's match_and_filter() in
order to mimic the behavior of that function exactly).

import biom>>> import pandas as pd>>> # Load in the data>>> table = biom.load_table("mt.biom.txt")>>> metadata = pd.read_csv("sample_metadata.txt", sep="\t", index_col=0)>>> # What's in the table?>>> table.ids(axis="sample")
array(['Sample1', 'Sample2', 'Sample3', 'Sample4', 'Sample5', 'Sample6'],
dtype=object)>>> # Looks good. What's in the metadata?>>> metadata
Metadata1  Metadata2  Metadata3
Sample-ID
Sample1            1          2          3
Sample2            4          5          6
Sample3            7          8          9
Sample5           13         14         15
Sample6           16         17         18
Sample7           19         20         21>>> # Still looking good. What happens when we try to subset>>> # the metadata in the exact same way Songbird's code does?>>> metadata = metadata.loc[table.ids(axis='sample')]
Metadata1  Metadata2  Metadata3
Sample-ID
Sample1          1.0        2.0        3.0
Sample2          4.0        5.0        6.0
Sample3          7.0        8.0        9.0
Sample4          NaN        NaN        NaN
Sample5         13.0       14.0       15.0
Sample6         16.0       17.0       18.0>>> # So, although we filtered out Sample7 (in metadata but not in table),>>> # we actually inserted Sample4 (in table but not in metadata).>>> # This means that stuff like metadata_filter() won't work the way we>>> # expect it to.>>> def metadata_filter(val, id_, md): ...    return id_ in metadata.index>>> supposedly_filtered_table = table.filter(metadata_filter, axis='sample')>>> supposedly_filtered_table.ids(axis="sample")
array(['Sample1', 'Sample2', 'Sample3', 'Sample4', 'Sample5', 'Sample6'],
dtype=object)>>> # Yeah, we didn't filter the table at all.

So that it's easily rerunnable, I've included a copy of the above code
without commentary/formatting here:
minimal_code_to_reproduce.py.txt
https://github.com/biocore/songbird/files/3849924/minimal_code_to_reproduce.py.txt
Solution
I think if you take the set union of the table and metadata IDs, and
filter to that (instead of just throwing all table IDs into
metadata.loc[], this seems to solve the problem:

metadata.loc[set(table.ids(axis="sample")) & set(metadata.index)]
Metadata1  Metadata2  Metadata3
Sample-ID
Sample6           16         17         18
Sample3            7          8          9
Sample2            4          5          6
Sample5           13         14         15
Sample1            1          2          3>>> # The metadata is filtered correctly now!>>> # That being said it's out of order, so if this is a problem I guess it'll require>>> # re-sorting in some way.

TL;DR
ANYWAY sorry for the 6-month delay in explaining this bug report. Usually
I'm faster with this sort of stuff!!!
I honestly do not have the time to personally submit a PR for this, but I
think implementing the proposed solution above shouldn't be too terrible.
As mentioned, I think this should be a pretty rare bug, but 1) it does
provably exist in Songbird's code, and 2) I think it'd be a good idea to
resolve it sooner rather than later.
Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/biocore/songbird/issues/55?email_source=notifications&email_token=AA75VXLHJOXLLEQRNQYP5ITQTYYK7A5CNFSM4HL7RAZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEELSLI#issuecomment-554219821,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA75VXMOHNWBZVFOYO4PZ7LQTYYK7ANCNFSM4HL7RAZQ
.


AZCOPY_MSI_RESOURCE_STRING does not work with MSAL

Which version of the AzCopy was used?
v10.21.0
Which platform are you using? (ex: Windows, Mac, Linux)
Linux (specifically shell extension in ev2)
What command did you run?
export AZCOPY_AUTO_LOGIN_TYPE=MSI
export AZCOPY_MSI_RESOURCE_STRING="$resourceId"
azcopy sync "$src" "$dst" --recursive --delete-destination=true

What problem was encountered?
2023-10-01 09:58:19.345119: Failed to perform Auto-login: ManagedIdentityCredential authentication failed
2023-10-01 09:58:19.345131: GET http://<IP_ADDRESS>/metadata/identity/oauth2/token
2023-10-01 09:58:19.345144: --------------------------------------------------------------------------------
2023-10-01 09:58:19.345156: RESPONSE 400 Bad Request
2023-10-01 09:58:19.345168: --------------------------------------------------------------------------------
2023-10-01 09:58:19.345180: Failed to make request: '\''Identity not found: '\''ACI_MI_DEFAULT'\'''\''
2023-10-01 09:58:19.345192:
2023-10-01 09:58:19.345204: --------------------------------------------------------------------------------
2023-10-01 09:58:19.345215: To troubleshoot, visit https://aka.ms/azsdk/go/identity/troubleshoot#managed-id.'

How can we reproduce the problem in the simplest way?
Use any bash shell with User assigned managed identity, in my case it was in ev2
Have you found a mitigation/solution?
Use client ID instead of resource string
export AZCOPY_MSI_CLIENT_ID="$clientId"

The option has been deprecated :
https://github.com/Azure/azure-storage-azcopy/releases/tag/v10.21.0-Preview

Deprecated support for object IDs in MSI. Client ID or Resource ID can be used as an alternative.

Please forgive me, I misread your issue.
I thought that you use the deprecated system variable.
In fact, I have the same issue #2398 but azcopy uses the system identity as fallback instead of the provided resource string.

Hi, I believe this issue was fixed by the following PR https://github.com/Azure/azure-storage-azcopy/pull/2406
This should be released as part of our 10.22 release

The 10.22.0 version was just released. Closing this issue as a result.


[JENKINS-68025] hudson.plugins.jobConfigHistory.FileHistoryDaoTest fails on Java 17

As described in JENKINS-68025 (comment), once the proper Add-Opens directives are put in place, this test fails with a variety of errors trying to serialize java.util.concurrent.locks.ReentrantLock.serialVersionUID. The reason for this is the test tries to serialize a reference to a mock object. This is not a realistic production use case, and there is no production bug. To fix the test, we simply use the Jenkins test harness to create a real agent rather than a mock one. That makes the test more realistic and fixes the test bug when running on Java 17. With this change, mvn clean verify (with the generic tooling changes I have been working on) passes on Java 17.

thanks for your PR!

I've fired up a test run using Java 17 on ci.jenkins.io: https://ci.jenkins.io/job/Plugins/job/job-config-history-plugin/job/testing%252Fpr187%252Fjava17/

Why? I already wrote that "with this change, mvn clean verify (with the generic tooling changes I have been working on) passes on Java 17."

Hopefully, all tests are passing now 👏🏻

Did you not read "with the generic tooling changes I have been working on"? Your test run will not work without plugin POM 4.40 (not yet released). You can clone plugin-pom and install a snapshot build of the latest commit if you want to play around with this.

Your test run will not work without plugin POM 4.40 (not yet released).

Oic, I wasn't fully aware of that. Thanks for the clarification.


Assign a policy initiatives with role assignment

Hi,
I  managed to assign a policy initiative but it doesn't apply the role assignment to the manged identity.
My Policy initiative is referring 4 policies definition and each ones have the role definition ids :
                    "roleDefinitionIds": [                         "/providers/microsoft.authorization/roleDefinitions/749f88d5-cbae-40b8-bcfc-e573ddc772fa",                         "/providers/microsoft.authorization/roleDefinitions/92aaf0da-9dab-42b6-94a3-d43ce8d16293"                     ]
My code is here : JamesDLD/azure-polices
The message I got :

Thanks in advance,
Jamesdld

Just found the issue!
Adding below both requirements that solved it if it can help others -->
When doing Policy assignment

Make sure to fill the policy roleDefinitionIds if you are using the effect DeployIfNotExists on the < Policy >.json file, cf the following sample.

"roleDefinitionIds": [ "/providers/microsoft.authorization/roleDefinitions/749f88d5-cbae-40b8-bcfc-e573ddc772fa", "/providers/microsoft.authorization/roleDefinitions/92aaf0da-9dab-42b6-94a3-d43ce8d16293" ]

Make sure to fill in the App Registration detail into the following brackets on the assign.< Policy >.json file

"identity": {
  "principalId": "The Identity principalId",
  "tenantId": "Your Tenant Id",
  "type": "SystemAssigned"
}

Just found the issue!
Adding below both requirements that solved it if it can help others -->
When doing Policy assignment

Make sure to fill the policy roleDefinitionIds if you are using the effect DeployIfNotExists on the < Policy >.json file, cf the following sample.

"roleDefinitionIds": [ "/providers/microsoft.authorization/roleDefinitions/749f88d5-cbae-40b8-bcfc-e573ddc772fa", "/providers/microsoft.authorization/roleDefinitions/92aaf0da-9dab-42b6-94a3-d43ce8d16293" ]

Make sure to fill in the App Registration detail into the following brackets on the assign.< Policy >.json file

"identity": {
  "principalId": "The Identity principalId",
  "tenantId": "Your Tenant Id",
  "type": "SystemAssigned"
}


Add minRowHeight or minElementWidth

It will be very nice to add one of this options

Hi @DenislavParvanov, sounds great! Could you comment how do you imagine those options would work in the following scenarios?

When specifying a minRowHeight, what happens if, with the given elements, the minRowHeight cannot be reached, thus a regular mosaic cannot be created?
When specifying a minRowHeight, how jQuery Mosaic should react if minElementWidth has also been specified and the aspect ratio for a certain element cannot be attained as a result?

I think of jQuery Mosaic primarily as a responsive way of creating mosaics. I think the responsive nature of the generated mosaics is a primary goal of the project, so giving too many specific sizes and limits starts to defeat the responsive logic of the plugin pretty fast. Do you think implementing those two options, if the conflicts between those kind of limiting parameters could be solved, and the complexity of managing exceptions was acceptable, would go against this essence?
Thanks!

Since I'm not getting any answer and it's been a long time, I'm closing this issue.


chore: Changed tableName to memoryType as it is mapping to memories table in database

…the type field in the memories table in the DB.  This improves code clarity as there were many places where a MemoryManager is created and took tableName and the code said this would act on a different database table, but in fact all of those lead to the memories table in the database, and tableName was just being mapped to type there, so memoryType is more appropriate name

Relates to:
No issue

Risks
Low
Background
What does this PR do?
This improves code clarity and understandability.  Previously the code strongly implied that the MemoryManager would act on different tables in the underlying DB, whereas all instantiations of the MemoryManager actually write to the same table in the DB but write a string to the 'type' field to distinguish (and namespace) different types of memories.  This PR renames 'tableName' as 'memoryType'
What kind of change is this?
Code improvement
Why are we doing this? Any context or related work?
As the code is evolving quickly, changes to improve understandability and semantically clear naming of variables and parameters will help us scale and improve overall developer productivity.
Documentation changes needed?
documentation changes are included
Testing
Tested changes.
Where should a reviewer start?
Discord username
ASI Carbon

This is going cause problems for the 70 or so PRs we have open. I'm not sure it's value is worth the cost

always a trade off yes... a bit luckily i just did a quick search through the open PRs and looked at in the "files changed" for those containing "tableName"... was a manual check but only seemed to affect about 4-5 PRs out of all of them.  I suspect this is because with many of the PRs their changes are well isolated in the code base.
On the value side of this change, I would say that this caused me quite a bit of confusion until I finally dug into the code and found that tableName was not in fact a reference to a table name at all.  When I asked in the forum unfortunately nobody was able to answer me.  Given the project is still so early... and more developer coming... the value of more rapidly understanding the system multiplied by all those developers reaching productivity a bit faster is real value.  I know there is alot of interest in the memory system as well so as part of the core it should be as clear as possible IMHO.

This is going cause problems for the 70 or so PRs we have open. I'm not sure it's value is worth the cost. Though if we're going to do it, we better do it sooner than later.

How do you think this will affect the v1 agents in production? Those agents that get an upgrade can be a headache to find out if their is not a public announcement on this type of change when it gets merged.

I’m just looking for some additional input from Eliza devs before we make a decision. Thanks for the analysis asicarbon, it is helpful.

I’m just looking for some additional input from Eliza devs before we make a decision. Thanks for the analysis asicarbon, it is helpful.

sure thing!


Bridge actor doesn't have "outbox" property

The outbox property is required by ActivityPub standard: https://www.w3.org/TR/activitypub/#actor-objects (the collection can be empty).
This causes the problem ‘json parse error: missing field outbox’ with some fediverse servers such as mitra. https://codeberg.org/silverpill/mitra
Example https://momostr.pink/users/npub1hgclq59xd44wuulmykzqtfz0elaxvt3y0p8xv5y5f78u95cgjsnshhy0we

Fixed.


Should the parser include the "option" for unions?

While submitting some of my changes upstream, the maintainer said he would rather include all the information available on the AST (see https://github.com/eleme/thrift-parser/pull/6#discussion_r113278627).  This would mean our compiler would need to know to throw away the option property on Union types (I think this makes it a little more complex).
Should we follow the guidance by the upstream maintainer to get the changes upstreamed?

Another recommendation by a community member on the upstream PR is to force each "option" property to "optional".

So what's your opinion.  To me if the thrift binary considers it invalid syntax we shouldn't support it but on the other hand we would have to manage our own fork which doesn't seem productive.

The upstream maintainer hasn't responded to my followup questions.  I think it might be best to make the changes as he requested and push the invalid syntax checks into the compiler (the thrift binary has the luxury of handling all parts of the process so they can error wherever they like).
I'll create a PR for the changes as suggested by the upstream maintainer.

Merged into develop.


Add focusable variants of widgets

This PR adds a set of actors to the scene2d extensions that mimic scene2d's default widgets, but automatically declare themselves to be focusable on setStage

Can you add a paragraph to https://github.com/MrStahlfelge/gdx-controllerutils/wiki/Button-operable-Scene2d ?
(You don't have access, but please add a comment here and I will copy it to the wiki entry when merged)

Sure thing! I'd place this after the ControllerList description.
Focusable actors
Most of the scene2d widgets have been extended to automatically call ControllerMenuStage#addFocusableActor. They follow the original naming, prepended by Focusable (FocusableButton, FocusableImage, and so forth).

Would it be possible to release a new version with these?

Sorry for not replying so long. I wanted to release a new version, but at the moment the project is not compatible with Java 11. Gradle needs to get upgraded and this needs other changes to the build files as well. As I am really busy right now and did not follow the general changes on libgdx builds at the moment, I would really appreciate a PR updating the build environment so that I can do a release quickly. Thanks :)

Ask and you shall receive :grin: #29

Great, thanks. 2.4.0-SNAPSHOT is available now, release builds do not work (yet), more in the PR.


For  metrical-tracker

In metrical-tracker,
lmk_path = imagepath.replace('images', 'kpt').replace('.png', '.npy').replace('.jpg', '.npy') is kpt7.npy ?
lmk_path_dense = imagepath.replace('images', 'kpt_dense').replace('.png', '.npy').replace('.jpg', '.npy') is kpt68.npy?

Not really. 'kpt' and 'kpt_dense' are folder names under 'input/<id_name>/', which will be generated during the tracking process. I don't konw the exact reason why you can't run metrical-tracker and you should turn to its issues to find some clues.

Hello, @Tiandishihua  , Excuse me，
Have you solved this problem?


Allow to dynamically add custom functions at runtime

This is very helpful for scenarios where the custom function is very domain-specific and does not make sense to add to the Luckysheet codebase.
Developer will have to be cognizant of locales and needs to handle providing the appropriate text based on the current locale.

Thanks


Update MD file on new branch 2017-11-08_09-26-58

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Unable to launch with custom profile

Description of the new feature/enhancement
I want to have a dynamic profile which windows terminal can execute from command line. wt -p "<pathToProfile.json>" instead of giving it a set profile name and it only looking for the global settings.json. I am making a command line program and really don't want to have to edit a global settings.json every time a new background color / icon gets changed as editing that file may not be possible if it's already running in another instance.
Proposed technical implementation details (optional)
I want to have a custom profile json on command line boot which I looked for hours and it doesn't exist.

Could you maybe elaborate more on what you're trying to do with

I am making a command line program and really don't want to have to edit a global settings.json every time a new background color / icon gets changed

Fragments  might fit the bill for what you're doing here. FWIW, the terminal's settings.json file can totally be written to while the Terminal is running. The Terminal should even be able to pick up those changes and hot-reload them. This has been used before to hilarious extent

I need to dynamically detect the terminal exe icon it's executing and other settings as a custom profile. For example cmd as the terminal Java as the icon.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Mike Griese @.>
Sent: Wednesday, May 25, 2022 6:39:58 AM
To: microsoft/terminal @.>
Cc: jredfox @.>; Author @.>
Subject: Re: [microsoft/terminal] Unable to launch with custom profile (Issue #13175)
Could you maybe elaborate more on what you're trying to do with
I am making a command line program and really don't want to have to edit a global settings.json every time a new background color / icon gets changed
Fragments https://docs.microsoft.com/en-us/windows/terminal/json-fragment-extensions might fit the bill for what you're doing here. FWIW, the terminal's settings.json file can totally be written to while the Terminal is running. The Terminal should even be able to pick up those changes and hot-reload them. This has been used before to hilarious extenthttps://www.hanselman.com/blog/adding-reaction-gifs-for-your-build-system-and-the-windows-terminal
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/13175#issuecomment-1137134805, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACKJBMOTX674NPYFHXYKD53VLYGQ5ANCNFSM5W4YTXAQ.
You are receiving this because you authored the thread.Message ID: @.***>

basically I launch it check if my custom json profile exists if not create it. Then I launch it using wt -p "directory/to/my/CustomProfile.json

Wait that's not how the commandline arguments for the Terminal work at all. You can't specify the path to another file of settings to use.
For your scenario, you definitely want to create the profile as a fragment profile, stick it in one of the fragments paths (whatever's more relevant for your type of application), and then you should be able to invoke that profile with wt -p {name of the profile you generated}

I know it's not in there yet that's why I wanted to specify a path or directory of paths to search for profile jsons and color jsons
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Mike Griese @.>
Sent: Monday, August 1, 2022 12:39:39 PM
To: microsoft/terminal @.>
Cc: jredfox @.>; Author @.>
Subject: Re: [microsoft/terminal] Unable to launch with custom profile (Issue #13175)
Closed #13175https://github.com/microsoft/terminal/issues/13175 as not planned.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/13175#event-7103561608, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACKJBMLTMPRUUDWU65TQ4JLVXADVXANCNFSM5W4YTXAQ.
You are receiving this because you authored the thread.Message ID: @.***>

We're tracking that specific request in /dup #6687. I suspect that fragment profiles will work for your use case today though.


What about Plugins ?

it would be great if there were plugin management

Hello,
What is the idea ?
Areas ?
Moduler design with Autofac or Castle Windsor ?

I think is good idea, but not for the future plans, this sample is going to be a Micro-services app running in Docker or cloud or local.


Previews not visible behind cloudflare proxying

Hi guys,
I'm having an issue where the stl previews don't seem to be rendering correctly, they just show as a grey box as seen below.

I can download the files fine and the .png file preview seems to work. the files inside the container are root:root 775.
Sometimes when i move from page to page i will see the stl files render for a split second but them disappear again. could it be maybe something different about my stl files thats causing the renderer to crash? they are all exported from tinkercad if that makes any different? here's an example stl
wyse_bottom.stl.zip
checked the /logs URL but don't see any specific errors that stand out
logs.zip
running from the latest container image on ubuntu 20.04.03 lts, 5.4.0
docker ver 20.10.2:2291f61
my compose
version: "3"
services:
  app:
    image: ghcr.io/floppy/van_dam:latest
    container_name: 3dp
    ports:
      - 3214:3214
    volumes:
      - "/var/lib/docker/containers/3dp/libraries:/libraries"
    environment:
      DATABASE_URL: postgresql://username:password@<IP_ADDRESS>/van_dam?pool=5
      SECRET_KEY_BASE:
      GRID_SIZE: 260
    depends_on:
      - db
      - redis

    networks:
      new-macvlan-network:
        ipv4_address: <IP_ADDRESS>

  db:
    image: postgres:13
    container_name: 3dp_postgres
    volumes:
       - "/var/lib/docker/containers/3dp-db/data:/var/lib/postgresql/data"
    environment:
      POSTGRES_USER: username
      POSTGRES_PASSWORD: password
    ports:
      - 5432:5432
    restart: always
    networks:
      new-macvlan-network:
        ipv4_address: <IP_ADDRESS>

  redis:
    image: redis:6
    container_name: 3dp_redis
    restart: on-failure
    networks:
      new-macvlan-network:
        ipv4_address: <IP_ADDRESS>

volumes:
  3dp_db_data:
networks:
  new-macvlan-network:
    external:
      name: new-macvlan-network

Any other logs I can look at to help narrow down the issue? or things to test out?
Thanks.

Bit more digging about and i think it's related to cloudflare proxy settings as when i jump into the container host, the stl previews are fine, and when i disable auto-proxy for that subdomain, it renders correctly.
Seems like the "rocketloader" is mangling the application.js in someway preventing the previews from rendering. could possibly be fixed by rearranging the order with which content is loaded?
workaround is to create a new rule in your cloudlfare dashboard(Rules>Page Rule>New rule)
put in your domain: sub.example.com/*
and select the below option

Oh, that's interesting! Are you running it on a cloud server then that's behind cloudflare? I've only ever run it locally, so this is a whole new potential place for problems 😁
Can I ask - where did you find the software? I've never had anyone using it before I don't think, and suddenly there are a bunch of issues coming in (unsurprisingly!)

Oh, that's interesting! Are you running it on a cloud server then that's behind cloudflare? I've only ever run it locally, so this is a whole new potential place for problems 😁

I have it self hosted on a local machine with nginx proxy manager in front of it, and I have cloudflare proxying it again to hide my local IP. but seems it's doing a lot more than just proxying by default.

Can I ask - where did you find the software? I've never had anyone using it before I don't think, and suddenly there are a bunch of issues coming in (unsurprisingly!)

David over at DBTech done a nice video about it here - https://www.youtube.com/watch?v=FWj7WNPjOCw

Thank you for the link! It's amazing that it's been found! I'll try to take a look into what might be happening...

Rocket Loader™ is known to break some pages, but you should be able to set an exclusion rule for VanDAM.

@DarrenF-G did you ever get this working with rocketloader?

For people who come here having the issue that the previews seem to load, but never show: Try disabling your Darkmode extension. I use Dark Reader, and with it on, all the STL preview things were just gone.
Not a bug on Van DAMs side, but I found this trying to figure this out, and then realized it might be the dark mode extension messing with it.
PS: @ Floppy, love the software, thanks for making it. Setup is a bit rough, but the actual experience using it is great. Only thing I needed to do, is setting a password so I can make it available from the outside without exposing anything.

I have a similar issue. However I am using CloudFlare tunnels to expose manyfold onto the internet. The previews do not load correctly (with one preview black box showing in background stretched out). Forcing a browser refresh does correct the issue problem. this issue also appear to just be and Edge problem as i cannot replicate in firefox

Closing this due to inactivity, but if it's still a problem, please reopen it!


Use IP instead of hostname for etcd bind address

Specify the IP address that etcd should bind, instead of allowing it to choose one given the machine's hostname.
Fixes #145

LGTM


how to remove mouse pointer from my record_video,thank you~

See. https://github.com/aerokube/selenoid/issues/1163#issuecomment-998780905


Maintenance

Landed in https://github.com/metarhia/impress/commit/3a1c774a415c55f25b23d4bf8b2af2243bef83a5 and https://github.com/metarhia/impress/commit/25abc449d13f38282072211dfdae6080c722aaaa


Many missed optimizations.

Highly redundant MIR is optimized into this.
    7ff13cd670f0:	48 83 ec 18          	sub    $0x18,%rsp
    7ff13cd670f4:	48 89 1c 24          	mov    %rbx,(%rsp)
    7ff13cd670f8:	48 8b c7             	mov    %rdi,%rax
    7ff13cd670fb:	33 c9                	xor    %ecx,%ecx
    7ff13cd670fd:	48 c7 00 1c 00 00 00 	movq   $0x1c,(%rax)
    7ff13cd67104:	c7 40 08 ff ff 01 00 	movl   $0x1ffff,0x8(%rax)
    7ff13cd6710b:	c7 40 44 00 00 04 00 	movl   $0x40000,0x44(%rax)
    7ff13cd67112:	c6 40 51 00          	movb   $0x0,0x51(%rax)
    7ff13cd67116:	40 88 48 52          	rex mov %cl,0x52(%rax)
    7ff13cd6711a:	40 88 48 53          	rex mov %cl,0x53(%rax)
    7ff13cd6711e:	c6 40 54 00          	movb   $0x0,0x54(%rax)
    7ff13cd67122:	c6 40 55 00          	movb   $0x0,0x55(%rax)
    7ff13cd67126:	48 8b 1c 24          	mov    (%rsp),%rbx
    7ff13cd6712a:	48 83 c4 18          	add    $0x18,%rsp
    7ff13cd6712e:	c3                   	ret

Of note is that a stack frame is being made, without need for it. Also, a register is being spilled when it is not being used. This is due to the hard register information being computed before other passes remove the need to have the register be spilled.
Also of note is the use of RAX instead of RDI, for the first parameter.
Finally, the constant zero is written to RCX, when it could have been merged with the two moves at offsets 52 and 53.
Many of these strike me as low hanging fruit. I'm on the bbv branch. Input code available on request, by email.

Sorry for delay with the answer and thank you for reporting this problem.
I am really interesting in the source code.  Could you,  please, post it here.  I'll try to investigate it and write when and what could be fixed after that.

Gladly. The code is very long, and I do not wish to share it here. Do you have a mail address that I could use?

Gladly. The code is very long, and I do not wish to share it here. Do you have a mail address that I could use?

<EMAIL_ADDRESS>


Add an extra Kotlin dependency. Set KotlinTest version as Kotlin ver…

Also set KotlinTest version as Kotlin version is set.
cf. Issue #140.
All tests pass, and the message as shown Issue #140 has gone. Also the plugin loads into IDEA 2017.1.

I've merged this into the develop branch https://github.com/kingsleyh/DLanguage/commits/develop

So the plot with this repository is that master has only the releases and all work is on develop?

I see you reverted something, can you explain why so I don't do something wrong again.

it was putting the version number in gradle.properties. kotlin_version is in there as it's used multiple times in the build.gradle file and should be same each time. For most dependencies such as kotlintest there's no need to set a variable for the version number.

I had thought the general policy was to either use a properties file or not. You do not need a properties file just because you want to use it twice, you can set that in the build file directly. Thus the current situation seems a mix to me and so I went for the properties file for all things that change with versioning.

Main reason for the gradle properties file is integration with the build server. the variables can be configured in the travis build matrix. Currently it's only ideaVersion that's being used. It lets run the builds against multiple versions of intellij, as done here. likewise we may want future builds to be done with jdk8 and jdk9. or set the version in travis prior to auto deploy to jetbrains repo. I think there should be very little in the gradle properties.

It still leave the inconsistency of kotlin_version being defined in the properties file: on the argument above it should be defined in the build.gradle file. I would argue the opposite of the last line. Either everything should be in the build.gradle file or, if a properties file is to be used, then all the variables should be in it so that the build.gradle file only ever changes if the structure of the build changes, updating the dependencies only requires changes in gradle.properties.


feat: Do not encode payloads in cassettes as base64 by default

Resolves #1413
TODO:

[x] Parametrize existing tests with "--cassette-preserve-exact-body-bytes"
[x] Re-check coverage
[x] Write docstrings & comments

Codecov Report

Merging #1450 (d579168) into master (ce6cfd4) will decrease coverage by 0.25%.
The diff coverage is 95.52%.

@@            Coverage Diff             @@
##           master    #1450      +/-   ##
==========================================
- Coverage   99.14%   98.89%   -0.26%
==========================================
  Files          88       88
  Lines        6785     6778       -7
  Branches     1185     1169      -16
==========================================
- Hits         6727     6703      -24
- Misses         30       39       +9
- Partials       28       36       +8

Impacted Files
Coverage Δ

src/schemathesis/models.py
99.05% <ø> (-0.02%)
:arrow_down:

src/schemathesis/cli/callbacks.py
97.22% <50.00%> (-2.78%)
:arrow_down:

src/schemathesis/cli/cassettes.py
99.42% <98.36%> (-0.58%)
:arrow_down:

src/schemathesis/cli/__init__.py
99.18% <100.00%> (ø)

src/schemathesis/specs/openapi/validation.py
76.47% <0.00%> (-11.77%)
:arrow_down:

src/schemathesis/extra/pytest_plugin.py
94.11% <0.00%> (-5.89%)
:arrow_down:

src/schemathesis/utils.py
97.78% <0.00%> (-0.89%)
:arrow_down:

src/schemathesis/specs/openapi/loaders.py
98.31% <0.00%> (-0.85%)
:arrow_down:

...rc/schemathesis/specs/openapi/negative/__init__.py
94.11% <0.00%> (-0.17%)
:arrow_down:

... and 22 more

Continue to review full report at Codecov.

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


fix: app does not crash while creating new ticket or event

Fixes #638
Changes: app does not crash while creating new ticket or event
Screenshots for the change(UPDATED):

Done with the changes


Power consumption

Dear team,
I would like to get access to my power consumption in eve app and homekit
Is it possible ?
Thanks in advance,

Yes. You need something like https://www.home-assistant.io/integrations/neurio_energy/

Thank you for your answer. I would like to use the Smartlife outlet I have and use it in this plugin. I can view the power consumption in the Smartlife app but not in Home app or eve app ..

This is technologically impossible with the api that this plugin uses.


Remove some requires

Maybe some of these made more sense if we were requiring spec_helper in the past.

Yep.


resolve #52

add '--insecure' option to curl command when retrieving kubernetes's info from 'https://kubernetes.default.svc.cluster.local/api/v1/namespaces/'

@nmatsui Thanks a lot!
I'd be slightly inclined to do some research whether there is a AKS side workaround for this instead of lowering the security alltogether. Any general opinions on that? Have other projects the same issue with AKS?
I commit to research this, but can't do it right now.

I rewrote README & the comment of bin/vernemq.sh. Please check them.

LGTM, thanks!


Generic with capture returned from lambda parameter results in incorrect circularity error.

TypeScript Version: 4.0.0-dev.20200509 && 3.9.0-beta && 3.8.3

Search Terms:
generic capture implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer
Code
declare function getApple<T>(input: T, fun: () => void): T;
const apple = getApple('apple', () => apple)

Expected behavior:
No error.
Actual behavior:
Error on line two:

'apple' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.

Playground Link:
https://www.typescriptlang.org/play?ts=3.9.0-beta#code/CYUwxgNghgTiAEAzArgOzAFwJYHtXwHMQMBBABzIhAB4AVAPgAotUzkMAueWgGiTS6MAlPAC89eADccWYEK60A3ACgA9KvgByKBSqb4WALaUsYLBggBPeAAsoAZ3gZLZBNtSX9AI3BRk9hHN4YBwQR1QcDFsoSQQoJxc41AiMKGw8eChUYANHOEQQOHQQHOAsOEwreBwYA2zy8AtrFgMMRxwAd3wWcywoCCwAL0KAOmUwPHsonUoEUUJiclnGbV0QTT5hMQkZqiEgA
Related Issues:
Yes, though most appear to be a correct error and people don't understand (maybe this one is too and I'm one of those people!)
Workaround:
Wrapping the lambda in curly braces so the captured variable is not returned resolves the issue.  Also, if the function is not generic the issue is resolved.
declare function getApple<T>(input: T, fun: () => void): T;
const apple = getApple('apple', () => {apple}) // ok

declare function getApple(input: string, fun: () => void): string;
const apple = getApple('apple', () => apple)

const apple = getApple('apple', () => {apple}) // ok

This should probably be () => ({apple}) to be an equivalent example. Here the function returns an object, in your example it returns nothing.

@MartinJohns Returning void was specifically the fix I was trying to illustrate.  If the function doesn't return the captured value, then the type is inferred correctly.  If the function does return the captured value, it infers incorrectly.
Using your suggested code, returning an object with the captured value as a property results in inference failure again.
declare function getApple<T>(input: T, fun: () => void): T;
const apple = getApple('apple', () => ({apple})) // error

We issue errors on circularity, and can sometimes avoid doing so. The fact that we can sometimes avoid hitting a circularity error when one is present should be taken as a carve-out; if you have a circularity, you may receive a circularity error even if you don't immediately see the path the checker to get into that state.

With some help in TS Discord I think I understand the problem here.  TypeScript is trying to figure out what the type of each of the parameters to getApple are to see if they match the getApple signature prior to figuring out what the type of apple is.  So first the compiler figures out that the type of the first parameter is 'apple' and then it tries to figure out what the type of the second parameter is but runs into a problem because it doesn't yet know the type of apple.  At this point it throws an error, because it has given up on type checking the function call at this point.
As a human, I am starting with the signature types and then checking to see if the provided parameters match, but the compiler is first trying to fully resolve the provided parameter types and then comparing them to see if they match.  This means that the compiler doesn't short-circuit with "we don't actually care what the return type of () => apple is because the function's signature calls for => void" like the human does.


How to remove an user from all groups in signalr core 2.2?

Hey, i send group names by javascript signalr lib. And each requests send new groups list for user so i should remove past group subscribes.
I couldn't find any example in google. So i am asking here.
How to remove all subscribed groups in one request and add new subscribes of group list? I don't know when i send new groups list to signalr server that user has already in other groups or not.
Please show me some examples about managing or mapping groups with connection ids or user names.
Groups: over 20k for now and it will be more (maybe over 50k)
So i need scalable solutions.
Thanks in advice!!

@asimus90 If this issue is related to content in one of the topics, please close this and re-open using the This Page feedback button+form at the bottom of the topic. It will add metadata to track the issue and automatically ping the author of that topic.

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select This page .


feat: Update Generic SAML Draft to return SP Metadata

Description of the change
This PR updates the Generic SAML Draft terraform resource (and the dataSource) to return the fields recently added in the API for the Entity ID, Both ACS URLs with their respective index and the SLO URL.
Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklists
Development

[x] Lint rules pass locally
[x] The code changed/added as part of this pull request has been covered with tests
[x] All tests related to the changed code pass in development

Code review

[ ] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] Jira issue referenced in commit message and/or PR title

Testing
Created a file with these changes:
resource "cyral_integration_idp_saml_draft" "test_saml" {
  display_name = "example-okta-integration"
  idp_type     = "okta"
  attributes {
    first_name = "some-first-name"
    last_name  = "some-last-name"
    email      = "some-email"
    groups     = "some-group"
  }
  disable_idp_initiated_login = false
}

data "cyral_integration_idp_saml" "test_source" {
  # These are the filters you can apply.
  display_name = ""
  idp_type     = ""
}

And connected to a CP using the latest API for generic SAML.
I was able to check the resources with terraform show after the terraform init / apply

The SonarCloud is failing due to the code replication in defining schemes (data source for listing Generic SAML and resource for Generic SAML 'crud'). However,  we usually go with Schemes this way, so I don't know the best approach here.
FYI @wcmjunior / @jrich523


Add code action to replace if with bool::then method

This would apply to if expressions with a single else branch, where the consequent is of the form Some(expr) and the alternative is None.
Before:
let exclamation_mark = "Hello!".chars().enumerate().find_map(|(idx, c)| {
    if$0 c.is_ascii_punctuation() {
        Some(idx)
    } else {
        None
    }
});

After:
let exclamation_mark = "Hello!"
    .chars()
    .enumerate()
    .find_map(|(idx, c)| c.is_ascii_punctuation().then(|| idx));

Edit: I realised that Iterator::position is a better way to do what I was doing in this example, but then the if would disappear, so the example still stands :)

Duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/8413 ( just realized I called it Option::then in there by accident)

Ah, makes sense then why my search didn’t turn up any results.


Problems when parsing  tags (and more)

newsboat version:
newsboat 2.21.0
System: FreeBSD 12.2-RELEASE-p2 (amd64)
Compiler: g++ 4.2.1 Compatible FreeBSD Clang 8.0.1 (tags/RELEASE_801/final 366581)
ncurses: ncurses 5.9.20140222 (compiled with 5.9)
libcurl: libcurl/7.74.0 OpenSSL/1.1.1d zlib/1.2.11 nghttp2/1.42.0 (compiled with 7.74.0)
SQLite: 3.34.0 (compiled with 3.34.0)
libxml2: compiled with 2.9.10
Issue
This issue doesn't seem to be OS or terminal specific, but it's been reproduced on FreeBSD and Arch Linux with st and xterm.
Although I haven't tested it with many articles, except my own, and it'd be rather weird to be article specific, I've noticed that newsboat eats up parts of the text when it's wrapped in a
 tag.
I've noticed two things:

It deletes blank lines inside the

It might delete parts of a line, or the whole line even. This seems to be happening with lines containing code.

My suspicion for the latter is that the parser gets confused if there's "actual code" to be parsed.
Blank line deletion
On newsboat

Same part of the same article, but on Feedbro (Firefox add-on)

C code inside

newsboat

Feedbro

Bonus: Random weird positioning of
 and more line deletion inside

newsboat

Feedbro

Thanks! Can you provide the actual XML that led to this output, so I can reproduce? Note that you can make Newsboat read local files with file://, e.g. put file:///home/user/reproducer.xml into your urls file, and Newsboat will treat it as a feed and parse it just as if it was an http:// URL.

Thanks! Can you provide the actual XML that led to this output, so I can reproduce? Note that you can make Newsboat read local files with file://, e.g. put file:///home/user/reproducer.xml into your urls file, and Newsboat will treat it as a feed and parse it just as if it was an http:// URL.

Can you provide the actual XML that led to this output, so I can reproduce?

https://christosmarg.xyz/rss.xml

Note that you can make Newsboat read local files

I've tried it but got the same result.

Can you provide the actual XML that led to this output, so I can reproduce?

https://christosmarg.xyz/rss.xml

Note that you can make Newsboat read local files

I've tried it but got the same result.

The problem with C code is that the ampersand is escaped only once. When XML parser processes the document, it turns the & into &; when HTML renderer sees &, it assumes that what follows is a name of an HTML entity, doesn't find it, and freaks out. I will accept any patch that makes the parser more resilient, but I don't consider malformed input to be a bug in Newsboat.
The other two are less obvious, I'll take a look at them tomorrow.

The problem with C code is that the ampersand is escaped only once. When XML parser processes the document, it turns the & into &; when HTML renderer sees &, it assumes that what follows is a name of an HTML entity, doesn't find it, and freaks out. I will accept any patch that makes the parser more resilient, but I don't consider malformed input to be a bug in Newsboat.
The other two are less obvious, I'll take a look at them tomorrow.

The mystery of blank lines is solved too: Newsboat expects empty line to contain a single character, newline, but the feed has some whitespace before the newline: https://github.com/newsboat/newsboat/blob/c282d28504f47d8b2c92009285f5db2481545fcf/src/htmlrenderer.cpp#L911-L925
That'd be easy enough to fix.

The mystery of blank lines is solved too: Newsboat expects empty line to contain a single character, newline, but the feed has some whitespace before the newline: https://github.com/newsboat/newsboat/blob/c282d28504f47d8b2c92009285f5db2481545fcf/src/htmlrenderer.cpp#L911-L925
That'd be easy enough to fix.

Ah, the bonus (weird
 positioning and missing lines) is caused by the same thing as the C code problem: HTML entities are only escaped once. For example, in the following part of the <description> tag:
        <pre>
        for (pc = (unsigned short *)buf, pos = 0; pos < len; pos++) {
                switch (src[pos]) {
                        .
                        .
                        .
                        .
                }
        }
        </pre>

The less-than sign in the loop condition is encoded as <; XML parser turns it into <, which makes HTML renderer look for a tag name which isn't there. Again, the correct encoding would be &lt; — XML parser will turn that into <, which HTML renderer will process as expected.
The closing angle brackets don't stump the HTML renderer, so > is rendered as >. But < always confuses the poor program :)
So again: I'll accept a patch making the renderer (or the parser) more resilient, but I don't consider malformed input to be a Newsboat bug.
I'll close this issue once I fixed the missing blank lines.

Ah, the bonus (weird
 positioning and missing lines) is caused by the same thing as the C code problem: HTML entities are only escaped once. For example, in the following part of the <description> tag:
        <pre>
        for (pc = (unsigned short *)buf, pos = 0; pos < len; pos++) {
                switch (src[pos]) {
                        .
                        .
                        .
                        .
                }
        }
        </pre>

The less-than sign in the loop condition is encoded as <; XML parser turns it into <, which makes HTML renderer look for a tag name which isn't there. Again, the correct encoding would be &lt; — XML parser will turn that into <, which HTML renderer will process as expected.
The closing angle brackets don't stump the HTML renderer, so > is rendered as >. But < always confuses the poor program :)
So again: I'll accept a patch making the renderer (or the parser) more resilient, but I don't consider malformed input to be a Newsboat bug.
I'll close this issue once I fixed the missing blank lines.

@christosmarg, BTW, consider using CDATA blocks for this — it's a neat way to avoid all this escaping (although CDATA can't be nested, and you still have to look out of ]]> in your posts).

@christosmarg, BTW, consider using CDATA blocks for this — it's a neat way to avoid all this escaping (although CDATA can't be nested, and you still have to look out of ]]> in your posts).

I updated my whole RSS feed and blog script to wrap all
s in a CDATA block and it all works fine now!

I updated my whole RSS feed and blog script to wrap all
s in a CDATA block and it all works fine now!

I was wrong about what code causes this — it's TextFormatter which turns an array of rendered strings into a markup for our UI library, not HtmlRenderer — but anyway, #1431 fixes this bug. Hopefully it'll pass review and will be merged in a few days, and then released as part of 2.23 in March.

I was wrong about what code causes this — it's TextFormatter which turns an array of rendered strings into a markup for our UI library, not HtmlRenderer — but anyway, #1431 fixes this bug. Hopefully it'll pass review and will be merged in a few days, and then released as part of 2.23 in March.


VRAM needed

Hey, how much VRAM is needed to create a 144 frames video?

We test at least 16G VRAM is needed.


Added more components in frontend

Added more components in frontend

Description

Added components such as footer, search, registors.

Additional context

Improved design

Closes

#16

@Ammoniya please check this pr.