Build from source fails due to undefined references with NCCL v2.9.6

Hi everyone!
Over the last 2 years I have regularly built XGBoost from source. Today, for the first time, I have received an error while building XGBoost v1.4.1 in this environment:

CUDA 11.2
NCCL 2.9.6
GPU_COMPUTE_VER 52;60;61;62;70;72;75;80;86

The error is the following:
-- The CXX compiler identification is GNU 8.4.0
-- The C compiler identification is GNU 8.4.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /cm/shared/apps/gcc8/8.4.0/bin/g++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /cm/shared/apps/gcc8/8.4.0/bin/gcc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- CMake version 3.18.4
-- Performing Test XGBOOST_MM_PREFETCH_PRESENT
-- Performing Test XGBOOST_MM_PREFETCH_PRESENT - Success
-- Performing Test XGBOOST_BUILTIN_PREFETCH_PRESENT
-- Performing Test XGBOOST_BUILTIN_PREFETCH_PRESENT - Success
-- xgboost VERSION: 1.4.1
-- Setting build type to 'Release' as none was specified.
-- Configured CUDA host compiler: /cm/shared/apps/gcc8/8.4.0/bin/g++
-- The CUDA compiler identification is NVIDIA 11.2.152
-- Detecting CUDA compiler ABI info
-- Detecting CUDA compiler ABI info - done
-- Check for working CUDA compiler: /cm/shared/apps/cuda11.2/toolkit/11.2.2/bin/nvcc - skipped
-- Detecting CUDA compile features
-- Detecting CUDA compile features - done
-- CMAKE_CUDA_ARCHITECTURES: 86-real;86-virtual;80-real;80-virtual;75-real;75-virtual;72-real;72-virtual;70-real;70-virtual;62-real;62-virtual;61-real;61-virtual;60-real;60-virtual;52-real;52-virtual;
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found OpenMP_C: -fopenmp (found version "4.5")
-- Found OpenMP_CXX: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- Found OpenMP_C: -fopenmp (found version "4.5")
-- Found OpenMP_CXX: -fopenmp (found version "4.5")
-- Looking for clock_gettime in rt
-- Looking for clock_gettime in rt - found
-- Looking for fopen64
-- Looking for fopen64 - not found
-- Looking for C++ include cxxabi.h
-- Looking for C++ include cxxabi.h - found
-- Looking for nanosleep
-- Looking for nanosleep - found
-- Looking for backtrace
-- Looking for backtrace - found
-- backtrace facility detected in default set of libraries
-- Found Backtrace: /usr/include
-- Check if the system is big endian
-- Searching 16 bit integer
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of unsigned short
-- Check size of unsigned short - done
-- Searching 16 bit integer - Using unsigned short
-- Check if the system is big endian - little endian
-- /root/rpmbuild/BUILD/xgboost/dmlc-core/cmake/build_config.h.in -> include/dmlc/build_config.h
-- Performing Test SUPPORT_MSSE2
-- Performing Test SUPPORT_MSSE2 - Success
-- Using nccl library: /cm/shared/apps/nccl2-cuda11.2-gcc8/current/lib/libnccl_static.a
-- Found Nccl: /cm/shared/apps/nccl2-cuda11.2-gcc8/current/include
-- Using nccl library: /cm/shared/apps/nccl2-cuda11.2-gcc8/current/lib/libnccl_static.a
-- Configuring done
-- Generating done
...
[ 99%] Linking CXX executable ../xgboost
/cm/shared/apps/nccl2-cuda11.2-gcc8/current/lib/libnccl_static.a(enqueue.o): In function `ncclGetCudaGraph(ncclComm*, CUgraph_st**)':
(.text+0x30eb): undefined reference to `cudaStreamGetCaptureInfo_v2'
/cm/shared/apps/nccl2-cuda11.2-gcc8/current/lib/libnccl_static.a(enqueue.o): In function `ncclCudaGraphHostSetup(ncclComm*, CUgraph_st*)':
(.text+0x31ee): undefined reference to `cudaUserObjectCreate'
(.text+0x320c): undefined reference to `cudaGraphRetainUserObject'
(.text+0x3268): undefined reference to `cudaStreamUpdateCaptureDependencies'
/cm/shared/apps/nccl2-cuda11.2-gcc8/current/lib/libnccl_static.a(enqueue.o): In function `ncclEnqueueCheck(ncclInfo*)':
(.text+0x3655): undefined reference to `cudaStreamGetCaptureInfo_v2'
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/runxgboost.dir/build.make:291: ../xgboost] Error 1
make[1]: *** [CMakeFiles/Makefile2:203: CMakeFiles/runxgboost.dir/all] Error 2
make: *** [Makefile:149: all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.HAdsnk (%build)

I think it's important to highlight that I have no issue with this other environment:

CUDA 11.2
NCCL 2.8.4
GPU_COMPUTE_VER 52;60;61;62;70;72;75;80;86

So, I suspect the issue is related to NCCL 2.9.6.
Is this an issue with XGBoost or with NCCL itself?
Thanks a lot for your work and your help.
GT

Hi everyone and sorry for the mess: it looks like I am using a NCCL version built for CUDA 11.3, hence the errors, since cudaStreamGetCaptureInfo_v2 is not available on CUDA 11.2.
Please, ignore this issue.
Have a great day!


Code snippets

Hi Thanks for this extension. It will help me a lot.
I just have one remark. It does not work yet on Code Snippets.
I don't know if you know this functionnality, it's in Tools/Code Snippet Manager, it allows you to make Visual Studio know your most common code parts and automatically write them. I did this for several types of my UPROPERTY, but even with this extension, the tab will still be inserted after UPROPERTY. I think this would be a great improvement of this extension to enable this too.
I looked a bit in your code, I guess the "insert snippet" command is a command event like Paste, but I didn't find it yet and I don't really understand where to look for that. Any clue ?
Have a good one.

I'm not entirely sure and don't have the environment set up at the moment to test it, but you could try VSConstants.VSStd2KCmdID.INSERTSNIPPET.


Hello i am not able to use this package

if i am doing something wrong please help

use asyncio to run async function.
import asyncio
from jiosaavn.Async import searchSong

asyncio.run(searchSong('alone'))

i find the solution


Version change to 0.1.0

r? @maqi
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @Viv-Rajkumar


URLs need better formatting

is there a better way to format the page so the URLs do not wrap around the table columns?

Yes, we should move them to over to html tables rather than jekyll markdown tables.

https://care-connect-api.netlify.com/support_referenceapiservers.html
This is the page with the wrapping issue

Corrected wrapping by moving to html tables


Analyzer: Test class should have test method

Summary
Title: Test class should have test method
Category: Design
Description: Test class should have at least one test method, be abstract or static
Default severity: Info
AB#1950783

From @AliveDevil

Ah, so
[TestClass]
static class GlobalState {
 [AssemblyInitializer] …

would work? If this is the case, the wording "non-abstract or static type […] should contain at least one test method" should not include "static" (or rephrase that it applies to non-abstract and non-static / non (abstract or static)), then the analyzer can be stricter for these types.

I agree with your comment @AliveDevil, the wording is not ideal and maybe we should only mention the exception cases in the rule documentation page but not on the analyzer description.
The following examples are cases where the rule should not trigger:

Test class with no tests but some [AssemblyInitialize] or/and [AssemblyCleanup] methods. Usually this type is static and doesn't contain anything else but some helpers but this is not hard constraint so it's probably easier to just check the presence of the attributes mentioned.

Test class with some [ClassInitialize] or [ClassCleanup] with inheritance mode. The ideal case is that this class is abstract but this is not a requirement so once again we could be looking only at the presence of the attributes.

Note that the rule could apply to derived types from [TestClass] because the related virtual method forces to return derived or instances of [TstMethod].

Correction for point 2, the class doesn't need to be marked as [TestClass] so we only need to avoid raising for case 1.

@Evangelink for rule number one [classInit] and [classCleanup] should be checked as well? they are static too


Added top tips section

CLOSE ON FDR ITEM: https://identitydivision.visualstudio.com/IAM/SSPR/_workitems?_a=edit&id=46151&triage=true

Added tip to test with user not admin

@ajamess,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@ajamess : Thanks for your contribution to the Azure documentation! The author, @MicrosoftGuyJFlo, has been notified to review your proposed change.

Can I sign off for @MicrosoftGuyJFlo? He's OOF until Wednesday. @Curtand #sign-off


libcamera: Rename bbappend to match any version

Upstream package version has changed to 0.0.1.

Closing as this is a duplicate of #1103.


ADBDEV-4943-96: Add assertions to return from PtMDType.

Add assertions to return from PtMDType.
The PtMDType method can return NULL because it returns dynamic_cast which can
return NULL, so I added assertions.

https://github.com/arenadata/gpdb/pull/819


[app] add support for custom openai provider url

atm we only support openai or ollama
add support for custom openai url
definition of done:

[ ] i can drop my url (say llamacpp or github models, or openrouter, or anthropic maybe?) and chat and it works (tool call etc.) on all messages in the empty screen, can still embed videos, etc. (does not fail at tool call)
[ ] consistent UI
[ ] still uses vercel/ai lib (figure out how)

You can just add a "BASE_URL" in the settings, and use litellm.
This will enable you to run any model. Either remote, or local, or both
@louis030195

done


Segfault in daily testing

This just a marker in case this is hard to reproduce.
We're running daily wheel builds on the daily branch of MacPython/scipy-wheels:
https://travis-ci.org/MacPython/scipy-wheels/branches
Last night, there was a segfault on the macOS / Python 3.6 build:
https://travis-ci.org/MacPython/scipy-wheels/jobs/218374427
I'll turn up verbosity on the tests to give more detail on the next run.

While we are at it about building problems coud you please chime in for the Accelerate issue at https://github.com/scipy/scipy/pull/6051 ? It would be nice to have your opinion as you are building SciPy day in day out with MacOS.

Rerunning the tests just now does not reproduce the segfault.  All the code merged since the run with the segfault is in docstrings, so I suspect this is an intermittent problem.

I wonder if there's use for switching on the python's faulthandler, https://docs.python.org/3/library/faulthandler.html for nightly builds and/or bumping the verbosity level.

I bumped the verbosity level to 3 already - and I just now enabled the Python fault handler - let's see what comes up.

Verbose run shows segfault again, now on Python 3.5
https://s3.amazonaws.com/archive.travis-ci.org/jobs/218792747/log.txt
test_kdtree.test_random_compiled.test_points_near_l1 ... config.sh: line 50: 12749 Segmentation fault: 11  python -c "$test_cmd"

@sturlamolden @rainwoodman any ideas what this might be?

I will take a look with valgrind. Is this the head of master or a tag?
On Apr 5, 2017 11:16 AM, "Evgeni Burovski"<EMAIL_ADDRESS>wrote:
@sturlamolden https://github.com/sturlamolden @rainwoodman
https://github.com/rainwoodman any ideas what this might be?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/issues/7262#issuecomment-291949405, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIbTJa9fZJ_N-v6sARkR2ZPZRyVLXZqks5rs9pogaJpZM4Mz1wl
.

The daily branch (where we've seen the segfaults) always builds the latest scipy master.

The fix is in. Sorry for the segfault!

Here is the valgrind log ran with the stock Python 2.7 from Fedora, after 7262.
log.txt
A few jumps in the special functions depend on uninitialized values -- could be an issue in test cases?
The alarming one is an invalid read due to numpy (unlikely scipy related)

A few jumps in the special functions depend on uninitialized values -- could be an issue in test cases?

Probably an issue in specfun, which has been known to do such things.


Add training option with learnit database

Example of running with learnit database:
./trainval_net.py --weight vgg16.pth --imdb learnit_db_train --imdbval learnit_db_val --iters 200000 --cfg vgg16.yml --net vgg16 --set ANCHOR_SCALES "[8,16,32]" ANCHOR_RATIOS "[0.5,1,2]" TRAIN.STEPSIZE "[50000]"

I accidentally tried to base my pull request on this repo, instead of on my fork. Sorry about that.


Raise value equals to None

Hello,
I wanted the moves given by get_available_moves(). I noticed there's a Raise with a value equals to None. Why ? When is this Raise used ?

Heya, for reproduction sake could you paste your code and/or a PGN that reproduces this issue?

Hello,
Here's a simple code :
from texasholdem import TexasHoldEm
from texasholdem.agents import random_agent, call_agent
from texasholdem.game.action_type import ActionType
game = TexasHoldEm(buyin=500, big_blind=5, small_blind=2, max_players=2)
game.start_hand()

while game.is_hand_running():
    for move in game.get_available_moves():
        print(move)
        if move[0] == ActionType.CALL:
            game.take_action(*move)
            break
    if game.current_player % 2 == 0:
        action = random_agent(game)
        print(action)
        game.take_action(*action)
    else:
        action = call_agent(game)
        print(action)
        game.take_action(*action)
    print(game.is_hand_running())


Delete session cookie samesite 3.x

Overview
DELETE /_session does not work if the SameSite attribute is enabled when using browsers that enforce it because we forgot to add the property when deleting the cookie. This PR adds the SameSite attribute if enabled.
Testing recommendations
Enable SameSite, login via the dashboard, then log out. Observe that the AuthSession cookie has a value before log out but not after.
Related Issues or Pull Requests

Checklist

[x] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

Would this also apply when we clear the session on a failed login in https://github.com/apache/couchdb/blob/b59d65e68fa22f8562d84e575f36c6035f303130/src/couch/src/couch_httpd_auth.erl#L448-L451

eep, yes. will make new PR's. good catch.


Add sample on how to consume bounding box

See: https://github.com/Azure/azure-sdk-for-net/issues/12721#issuecomment-644237746

@maririos  - should this be labeled blocking-release?

Added in PR #12828
(not sure why it didn't close)


fix(helm): add trimSuffix to helper functions

A name ending with "-" fails the kubernetes regex tests.

This change is

I signed it!

Thank you! This will go into the 2.0.2 release today.


Sanitize JSON output to enforce snake case and empty string -> null conversions

We need to take into consideration how SAP asserts data types
Based on the data type - null isn't an acceptable value to represent the data and the empty string is at times more appropriate

JSON property names in the idoc formatted json is limited to 30 chars due to a limitation in dynamic data type construction. We'll need to limit remove underscores unless absolutely necessary to uniquely identify the segment

Best way to move forward based on my research ought to be the AJSON library. Had the clearest docs on how to control the serialization behavior.
Will seek to use it for upcoming standalone project

Creating a public static method in the API helper class to manage this globally throughout the project. Default implementation will use the existing /UI2/ namespaced solution with params to switch to alternate means to serialize JSON through other libraries where appropriate


EXSWHTEC-20 - hipFreeAsync negative tests

Added negative tests for hipFreeAsync

nullptr check for pointer parameter
invalid stream handle for stream parameter

Signed-off-by: Marko Veniger

closing since PR is migrated to hip-tests repo


[Android-Wallet-App]Previously Claimed Certificates are Lost after Update

Describe the bug
All previously claimed Certificate swith Android Wallet App 1.2.8-tst (41) got lost when I updated to Android Wallet App 1.3.0-tst (41).
Expected behaviour
Previously claimed certificate should be still there after app update.
Technical details
Android Wallet App Version 1.3.0-tst (41)
Galaxy XCover 4, Android 9

In this specific case it's expected behavior. Considering update is from non-multi-type version, to multi-type. There were critical structural changes. And related migration would break new (multi-type) approach logic

@RalicaY see the comment of Oleksandr here. As in the old app versions the data were stored in general app module, and for multi cert apps it's moved to separate, dcc module.
So please can you close this ticket as it is out of scope.

Closed due to above comments, i.e. behaviour is expected.

@oleksandrsarapulovgl Does this mean that from now on all certificates will be lost when updating to the next version or is this only expected here in this version? That is, what is the expectated behaviour for the versions > 1.2.8-tst (41)
FYI @ltranvan


Loading alpha and TweetCell updates

Hi @patrickreynolds thanks for the contribution - could you please briefly describe what the changes do?

Ah - sorry - I see that your commits have a really good summary :-)


Add support for Apple Password Rules

We should be able to produce a recipe from a password rules string. Both the character and wordlist generators could support this, returning an error if the supplied requirements are outside the constraints of the generator.

Related project: https://github.com/1Password/password-rules-parser


EMB#20 Speed up build by moving code into cpp files

Templates have to be defined in header files, but there is some amount of non-templated code that can be moved into cpp files. Some template classes can also be refactored to abstract away templates such that some of the code can be moved down.
Build times are getting long because every change to a header file causes the whole library to be rebuilt.

Well we are building a standard 'template' library


unable to run example

As in http://amol-mandhane.github.io/htmlPy/ , I created three files exactly the same as the example in hpy_test/.
Then I ran
cd hpy_test
python main.py

I got the following error:
Traceback (most recent call last):
  File "main.py", line 2, in <module>
    from back_end import BackEnd
  File "hpy_test/back_end.py", line 2, in <module>
    from main import app
  File "hpy_test/main.py", line 2, in <module>
    from back_end import BackEnd
ImportError: cannot import name BackEnd

And if I ran
# cd hpy_test
python back_end.py

I got the following error:
Traceback (most recent call last):
  File "back_end.py", line 2, in <module>
    from main import app
  File "hpy_test/main.py", line 2, in <module>
    from back_end import BackEnd
  File "hpy_test/back_end.py", line 2, in <module>
    from main import app
ImportError: cannot import name app

I am using python 2.7 (64bit), pyside 1.2.1 and htmlpy 2.0.3 in 64 bit windows 10.

For some reason, the circular imports are creating problems on Windows machine. The app seems to be working fine on linux. I'll refine the code.

I've adjusted the code to delay circular imports. Thanks for pointing the issue out.

Hi, now I could run the example, but it comes another issue.
The pop-up window shows "click here to say hello world". When I click it, I saw the following error in terminal:
TypeError: say_hello_world() takes no arguments (1 given)

That's a silly mistake on my part. htmlPy is well tested but it's documentation is not. :cry: :cry:
Should be fixed within an hour.

Yet another follow-up issue... (seriously?! :-/ )
After I clicked "click to say hello world", I saw the following error in terminal:
Traceback (most recent call last):
  File "hpy_test\back_end.py", line 8, in say_hello_world
    from main import app
  File "hpy_test\main.py", line 5, in <module>
    title=u"Sample application")
  File "~\venv\lib\site-packages\htmlPy\app_gui.py", line 103, in __init__
    super(AppGUI, self).__init__(*args, **kwargs)
  File "~\venv\lib\site-packages\htmlPy\base_gui.py", line 76, in __init__
    raise RuntimeError("Another htmlPy application is already running")
RuntimeError: Another htmlPy application is already running

I restarted the computer and init the main.py again and got the same error.

The example on the website is cooked up very quickly. I'll replace it with a proper one.
Meanwhile, you can try the ones written in the docs. I've tested them properly.
http://htmlpy.readthedocs.org/

I must be doing something wrong but I cant make the examples work either (same error RuntimeError: Another htmlPy application is already running)
Im using "sample driver file" as the sample_app (http://htmlpy.readthedocs.org/en/master/important.html#use-a-driver-file) and GUI to python calls as the back-end function container (http://htmlpy.readthedocs.org/en/master/tutorials.html#gui-to-python-calls).
I have tried thousand of combinations but there is no way I can import the "app" var without getting this error. All functions work until I try to call "app".
Hope you can spot my error, thanks.

Is it possible to add an example of embedded matplotlib figure in hmtlPy?

Updated the example.


issue fixed #1371 | Medium Issue 2 (CSS)

Pull Request Type
What kind of change does this PR introduce?

Solved an issue/task

What is the current behavior?

Issue Number: https://github.com/fork-commit-merge/fork-commit-merge/issues/1371
What is the new behavior?

a fire animation

fixed

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Use dart pub add to add pub dependency

Signed-off-by: Brett Morgan<EMAIL_ADDRESS>
New Pull Request Checklist

[x] I have read the Documentation
[ ] I have searched for a similar pull request in the project and found none
[x] I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
[ ] I have added the required tests to prove the fix/feature I'm adding
[ ] I have updated the documentation (if necessary)
[ ] I have run the tests without failures

Additional context and info (if any)

Thanks @domesticmouse! Can we keep them both here since we've also provided the latest pre-release badge?

Sure!


vue-tsc failed to infer generic types of custom directives within templates

Vue - Official extension or vue-tsc version
Same as playground
Vue version
3.5.0-beta.1
TypeScript version
5.6.0-beta
System Info
No response
Steps to reproduce
<template>

</template>

What is expected?
vue-tsc should infer the type of T correctly in the template.
What is actually happening?
Type '(newUser: User) => void' is not assignable to type '(newValue: { id: number; }) => void'.
  Types of parameters 'newUser' and 'newValue' are incompatible.ts(2322)

Link to minimal reproduction
Playground link
Any additional comments?
No response

This is a possible improvement. I thought of some methods, but was stuck on why TypeScript can't resolve types like ((a:number)=>number) | ((a:number)=>number). (playground)

@KermanX In the example you provided, f is actually inferred as never.
You can verify this by hovering over type F here: playground

It seems that type annotation has no effect in this case: null! is always inferred as never in TypeScript. You’re essentially telling the compiler that "this is null, but also not null at the same time".

@KermanX In the example you provided, f is actually inferred as never. You can verify this by hovering over type F here: playground.
It seems that type annotation has no effect in this case: null! is always inferred as never in TypeScript. You’re essentially telling the compiler that "this is null, but also not null at the same time".
However, ((a: number) => number) | ((a: number) => number) works as expected. You just need to initialize the variable with something that matches the type (e.g. null! as any).

It seems that type annotation has no effect in this case: null! is always inferred as never in TypeScript. You’re essentially telling the compiler that "this is null, but also not null at the same time".
However, ((a: number) => number) | ((a: number) => number) works as expected. You just need to initialize the variable with something that matches the type (e.g. null! as any). (playground)

Thanks a lot! Adding as any works. However, in the first playground link, hovering f shows const f: ((a: number) => number) | ((a: number) => number) instead of never, which seems a TypeScript bug.

However, in the first playground link, hovering f shows const f: ((a: number) => number) | ((a: number) => number) instead of never, which seems a TypeScript bug.

Reported at: https://github.com/microsoft/TypeScript/issues/59626


Debug Info: Emit debug info for Swift types using the default alignment.

Since Swift doesn't have a source-level mechanism for controlling the
alignment of types there is no point in encoding the default alignment
in the debug info. This change emits debug info for all swift types
with the default alignment value of 0.
This mirrors clang's behavior.
rdar://problem/29007471

@swift-ci test and merge

Swift has an @_alignment attribute.

Emit debug info for Swift types using the default alignment where possible.
This patch emits the alignment for all default-aligned types as 0
which causes the backend to not emit an alignment attribute at
all. This mirrors clang's behavior.

@swift-ci please test and merge

@swift-ci test and merge
(forgot to git-add the new testcase).

@swift-ci test and merge


Benchmark

Hi,
I have created a benchmark suite for Web Components librairies. Polymer is implemented, and you can see the raw results here : https://vogloblinsky.github.io/web-components-benchmark
I will release soon a blog post explaining in detail that, but i want a feedback of many librairies authors before, to be sure i am not wrong in my implementations.
Source code here : https://github.com/vogloblinsky/web-components-benchmark
Can you have a look and give me some feedbacks ?
Thanks

I will open some issues with feedback. Thanks for the benchmark!

Results are updated : https://vogloblinsky.github.io/web-components-benchmark/#todo
Polymer move with your PR from that :

to this for TodoMVC benchmark :


Dev kobeya 1002 2

Ticket Link
Groupings-1002
Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

Investigating the Codacy Reports.

I don't understand the .directive, so can't comment on that. Has Frank reviewed this code?
The text reference "newline characters" is not something you can expect a typical member of the UH community to understand, and that's your audience. Basically, you need to convey that each entry needs to be on separate line. For example: "The file should consist if a list of UH Usernames or UH Numbers, one per line." It is important to note that we have heavily marketed the "UH Username and UH Number" terminology in the UH community and this project needs to consistently use this technology in customer communications, just as it must consistently use "uid" and "uhUuid" in the codes. Two audiences, two sets of terminology.

Okay, I will go and fix these. Frank gave me the directive but I don't think he reviewed this pull request yet. I will message him on Slack.

Did a ticket get created to address my points above?

https://www.hawaii.edu/jira/browse/GROUPINGS-1082, https://www.hawaii.edu/jira/browse/GROUPINGS-1083, and https://www.hawaii.edu/jira/browse/GROUPINGS-1084 were created to address the points brought up in this pull request.


clearing un-needed scripts + better rewiting for the code

make the source code easir to read and understand in addition to clear unused scripts


Infra Failure: Step('install xcode') (retcode: 1)

Infra failure on the install Xcode step:
https://ci.chromium.org/p/flutter/builders/try/Mac plugin_test/7497?

Confirmed this is temporary issue, which succeeded on retry. Keeping this open for investigating root cause.

Actually this is not a transient bug but a bot cache polluted: https://chromium-swarm.appspot.com/bot?id=build866-m9
The same bot has been failing tasks very frequently. It may be related to enabling the xcode cache in the engine with the same named cache as the framework but using different xcode versions.
That was fixed already on the engine but we need to clean the cache on this bot.

I do not have access to clear the corrupted cache on the bot. Can @godofredoc or @keyonghan  please take a look at this.


[MAX32600MBED,MAXWSNENV] Added low-power ticker.

The adding of the low-power ticker API in #1079 caused build failures in the MAX32600MBED and MAXWSNENV platforms. This commit fixes these failures by implementing the low-power ticker for these platforms.

Thanks. I noticed deepsleep quite a change there

Yes! The addition of a ticker based on our RTC greatly simplified things there.


Timesheet - 19/11/2021 - Oleg Tropinin

Task 1
Add support of Bitcoin and Ethereum tickers.
Proof of Work

https://github.com/thenewboston-developers/Blockchain-Scanner/pull/1

Time Spent
30 hours

Task 2
Add pytest. Add first test.
Proof of Work

https://github.com/thenewboston-developers/Node/pull/8

Time Spent
1 hours

Task 3
Migrate README file from Markdown to reStructuredText format.
Proof of Work

https://github.com/thenewboston-developers/Node/pull/5

Time Spent
1 hours

Task 4
Install pre-commit to run mypy, flake8, yapf.
Proof of Work

https://github.com/thenewboston-developers/Node/pull/9

Time Spent
2 hours

Task 5
Resolved isort vs yapf incompability issue and reintroduced isort
Proof of Work

https://github.com/thenewboston-developers/Node/pull/26

Time Spent
2 hours

Total Time Spent
36 hours
Account Number
fa171bd01d13c37dd749ef24e575379033ac1f235aa87410a6f410dd837159a7

@olegtropinin, closing. Please speak to @buckyroberts.


Fix neighbor HT mode reporting

Signed-off-by: Michal Kowalczyk<EMAIL_ADDRESS>

https://jenkins.code.rdkcentral.com/job/github-triggered/14542/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/48780/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/9362/ started

Smoke tests for ['build-tur-bb'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/113510/

Smoke tests for ['build-rpi-broadband'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/113509/

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/9386/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/14567/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/48837/ started.

Smoke tests for ['build-rpi-broadband'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/113602/

Smoke tests for ['build-tur-bb'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/113603/

'build-rpi-broadband' Smoke test SUCCESS.
Prepare image for flashing [SUCCESS]
Flashing RPI_R03S14 [SUCCESS]
Testing 2.4GHz connection [SUCCESS]
Testing 5GHz connection [SUCCESS]
https://jenkins.cmf.code.rdkcentral.com/job/test-rdkb-raspberrypi-wifi-flow/27059/

https://jenkins.code.rdkcentral.com/job/github-triggered/14595/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/48944/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/9415/ started

Smoke tests for ['build-rpi-broadband'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/113759/

Smoke tests for ['build-tur-bb'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/113760/


Display success or error notification after create, update, delete of something.

Expected Behavior
After changing something on a service/user or application, a small popup should be displayed indicating if it was successful or not.
Actual Behavior
Popup does not always appear
Screenshots

I will work on this bug on my forked repo and create a PR as soon as I add the missing popups behaviour.


Clarification of PersistentSubscriptionNakEventAction.Stop

Hey there,
I'm having a PersistentSubscription and while catching up, I come into the situation, that I cannot process a certain Event. I would like to stop the persistentSubscription at the event where it is and continue at a later point (order is important).
I though subscription.fail(event,PersistentSubscriptionNakEventAction.Stop,'Error in sub service') would be the right choice. But Events keep on bubbeling in and get acknowledged to the eventstore.
I tried all combinations of the PersistentSubscription Settings, but weven with a 1000 Events to catch up, the subscription iterates throuh all of them, no matter how often I call sub.fail... Did I misunderstood the concept ? Any clarification is appreciated.

Did you disable autoAck on the persistent subscription?
By default persistent subscription are automatically acknowledge, so that would explain the behavior.
Use false as the last parameters in connectToPersistentSubscription(stream, groupName, eventAppeared, subscriptionDropped, userCredentials, bufferSize, autoAck)

EventStore will essentially just record that an event was not acknowledged. You can then use the Replayed Parked Messages in the admin ui to stream those events back to the subscription?

hey there, thanks for the replay.
Yes i disabled autoAck. I want to have more control over the acknowledgement.

I would also like to park some of the unprocessable entities at time.
And in certain situations I want to Stop the incoming messages in a controlled manner (e.g. a subsequent service is unreachable) and continue at a later point

Isn't this what  PersistentSubscriptionNakEventAction.Stop was thought for, or am i mistaken.
thanks a lot.


Travis-CI gave out multiple FAILED result on R_test, preventing PR

Travis-CI recently gave out multiple build failure , it's very possible that Travis-CI's R environment itself is not maintained well, this happened  in about recent 5 PRs (or more)

@thirdwing @hetong007

Error : .onLoad failed in loadNamespace() for 'mxnet', details:
call: fun(libname, pkgname)
error: could not find function "loadModule"
Some issue in Rcpp?
Looks similar to
We tried adding
Depends: Rcpp;

in https://github.com/dmlc/mxnet/blob/master/R-package/DESCRIPTION and build passed.
based on suggestion here - https://github.com/RcppCore/Rcpp/issues/118
However, using mxnet types in R terminal failed.

Sorry that I can't reproduce this with latest version of Rcpp on OSX.
Can you show me the error message?

OK. I confirm this. It is due to the update from roxygen2
On Fri, Feb 3, 2017 at 3:10 PM, Sandeep Krishnamurthy <
<EMAIL_ADDRESS>wrote:

Error : .onLoad failed in loadNamespace() for 'mxnet', details:
call: fun(libname, pkgname)
error: could not find function "loadModule"
Some issue in Rcpp?
Looks similar to
We tried adding
Depends: Rcpp;
in https://github.com/dmlc/mxnet/blob/master/R-package/DESCRIPTION and
build passed.
based on suggestion here - RcppCore/Rcpp#118
https://github.com/RcppCore/Rcpp/issues/118
However, using mxnet types in R terminal failed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/4870#issuecomment-277349670, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABebVR1ZIs9F5kPGCtI5Kp-AYlKb-3bCks5rY4mugaJpZM4L2oCU
.

--
Qiang Kou
<EMAIL_ADDRESS>School of Informatics and Computing, Indiana University

They claim it's fixed. However I still have trouble with least version. If anyone want to install mxnet then I would suggest clone roxygen2 and checkout 5.0.1

I have this problem with roxygen2 at 5.0.1. What else can I try to make it work?

First time using roxygen2. Upgrading automatically...
[2020-04-08T00:03:57.271Z] Updating roxygen version in /work/mxnet/R-package/DESCRIPTION
[2020-04-08T00:03:57.528Z] Loading mxnet
[2020-04-08T00:03:58.458Z] [1] "Loading local: inst/libs/libmxnet.so"
[2020-04-08T00:04:00.977Z] [1] "Loading local: src/mxnet.so"
[2020-04-08T00:04:00.977Z] Error in loadModule("mxnet", TRUE) : could not find function "loadModule"
[2020-04-08T00:04:00.977Z] Calls: <Anonymous> ... load_code -> <Anonymous> -> run_pkg_hook -> <Anonymous>
[2020-04-08T00:04:00.977Z] Execution halted
[2020-04-08T00:04:00.977Z] Makefile:688: recipe for target 'rpkg' failed
[2020-04-08T00:04:00.977Z] make: *** [rpkg] Error 1

Similar error of loadModule function not found in R
@thirdwing any idea?


Starlight components reading translations from Astro.props.labels property not working from<EMAIL_ADDRESS>

Describe the bug
Starlight 0.14.0 exposes a labels property in Starlight’s route data object for each page -> https://github.com/withastro/starlight/pull/1135 , so all components can read that property instead of using t(). starlight-openapi is not including that property when creating the StarlightPage, so an exception is raised, i.e:  TypeError: Cannot read properties of undefined (reading 'themeSelect.accessibleLabel')
To Reproduce

Install the following dependecies

"@astrojs/starlight": "0.15.0"
"astro": "4.0.3"

Start the project and browse to an operation page

Expected behavior
As i18n is not yet supported, it should generate the operation pages without translating, just using the default lang
How often does this bug happen?
Every time
System Info
No response
Additional Context
No response

Thanks for the report.
Indeed, any usage of the built-in <Page/> component are a bit broken since Starlight v0.14.0. I submitted a PR with a lot of details and an attempt to fix the issue.

As i18n is not yet supported, it should generate the operation pages without translating, just using the default lang

This is unfortunately not something we can do as even the default language UI strings for all Starlight internal components are expected to be passed down as props.

Thanks for the info, I will keep an eye on it!

Thanks for your patience with this issue.
v0.5.0 fixes the issue, e.g. this demo page which uses the latest Starlight version.


Fix outOfBound Exception if the count is bigger than the result size

With a small count of lesser than 50  Direct Message if we call getDmList(50) we have an OutOfBound Exception
To test it use unit test : testGetDmListWithCountAndGetDm() on a twitter account with less than 55 Direct message

Hi @lepit31 , thank you very much for this PR, checking it now.


Update README.md to include control center image

Ahh great, I noticed that todo control center image this morning, but when I came to get a line number to raise it, i see it's done! 💪


Avoid booting providers because they require a Silex\Application instance

Refs https://github.com/silexphp/Silex/pull/1238

looks like some requirements do not work for php 5.3/5.4 which is causing the build failures.
as for the change can you explain why they shoild not be booted?

The build failure is independent from my PR really, it's just that Silex bumped the requirement to php 5.5.9 so we should drop the 5.3/5.4 builds here IMO for 2.0
As for why not booting them, as discussed in the linked issue the boot() method requires a Silex\Application, which Cilex\Application is not, so skipping the boot makes everything work fine it just means we might miss some features that are set up in the boot methods but it is seldom used and surely better than a fatal error.

btw, the order of methods was weird before that. Providers were supposed to subscribe before booting. It was inconsistent with Silex.

The use statement should be removed too

OK I removed the use statement and also did a second commit for dependencies/travis update, it's kinda unrelated but while I'm at it it brings this package fully up to speed with Silex 2.0-dev.


Recover WIP-protected after unenrollment

Hello,
About this step:
"Where "new_location" is in a different directory. This can be on the employee’s device or on a shared folder on a computer that runs Windows 8 or Windows Server 2012 or newer and can be accessed while you're logged in as a data recovery agent."
Can you please explain what does it mean "while you're logged in as a data recovery agent"? How to login like this using Windows 10 Azure AD Join setup + WIP?

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

ID: 16e6e551-830e-f69e-31bc-e109e5b594f4
Version Independent ID: 3396045c-8295-b801-aba5-11a11dd62dc0
Content: Make & verify an EFS Data Recovery Agent certificate (Windows 10) - Microsoft 365 Security
Content Source: windows/security/information-protection/windows-information-protection/create-and-verify-an-efs-dra-certificate.md
Product: w10
Technology: windows
GitHub Login: @DulceMontemayor
Microsoft Alias: dansimp

@lightupdifire - Thank you for your query. To login as a data recovery agent, please see how to create a data recovery agent account by https://docs.microsoft.com/en-us/previous-versions/tn-archive/cc875821(v=technet.10)?redirectedfrom=MSDN
This can also be used for the local device account. You can follow the steps under "To create a local recovery agent" in the document. Once your recovery agent account is setup then you can follow the steps provided in the document.
Thanks
Imran

@joinimran
Thanks for the info. Then I think if we have a local admin account on the computer and EFS DRA certificate created for WIP, for the full cloud Windows 10 this should be enough.


[ADD ME TO THE ORGANIZATION]

**Name: Subham Saha
**Location: West Bengal
**Interest: Web dev
**Discord username(optional): - 𝙁𝘼𝙇𝙇𝙀𝙉𝘾𝘼𝙋#9407
Why do you want to join?

I want to join because I want to contribute in open-source

You can also join our Discord community here
Feel free to check out other cool repositories of the DevsInTech Community here.

@FallenCap Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


Vulnerability issue with tough-cookie

After installing the packet, npm audit was run and it was found the following:
# npm audit report

request  *
Severity: moderate
Server-Side Request Forgery in Request - https://github.com/advisories/GHSA-p8p7-x288-28g6
Depends on vulnerable versions of tough-cookie
No fix available
node_modules/request

tough-cookie  <4.1.3
Severity: moderate
tough-cookie Prototype Pollution vulnerability - https://github.com/advisories/GHSA-72xf-g2v4-qvf3
No fix available
node_modules/tough-cookie

2 moderate severity vulnerabilities

Some issues need review, and may require choosing
a different dependency.

After investigation, accordingly to this https://github.com/advisories/GHSA-72xf-g2v4-qvf3, the vulnerable code was fixed on version 4.1.3 and from the file package-lock.json this package is using 2.5.0
Can the software be updated to this version to avoid this issue?

I will look on removing the dep as its just used for the auth service, still, the auth service is just used once to create the base config, and its not a public facing server nor used in the module by itself


builld failing,

seems like the build is failing.
fatal watch error!!! { Error: Can't resolve 'babel' in '/Users/oriavraham/Development/Playground/react-universal-boiler'
gl

This issues fairly old, so I'm closing it. The issue you ran into was due to updates from Webpack. These are resolved.


Evidence View - Add filter feature to legend

Implement same behavior as in Gene View which allows users to click on concept type icons in the legend to filter the table.

could use a similar approach to https://github.com/Rothamsted/knetminer/blob/master/common/client-base/src/main/webapp/html/GeneView/summariseLegend.js maybe

partially implemented; tests pending...

now; it hides all other evidence rows from table when an evidence icon in top legend is clicked, showing just the clicked evidence type entries.
If users then click another evidence type, it is displayed alongside the 1st clicked one to incrementally show evidence1 OR evidence2. Clicking evidence1 again hides it (as it is visible).


chore(ssa): Implement ir errors

Description

Problem*

towards https://github.com/noir-lang/noir/issues/1473
Summary*

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

I don't think we should convert all panics in the compiler into Result errors. The SSA passes in particular are never supposed to error (unfortunately they sometimes do - this is always a bug). If an error is truly expected to be unreachable then I think it is fine to keep it as an expect/panic.
CC @TomAFrench, @phated for opinions here.

A lot of these errors seem to be something that a user can't trigger by themselves but would require us to have an incorrect compiler implementation so I think we can remove most/all of these.


Fix documentation grammar to be more correct

Some documentation like docstring sometime are hard to understand (I don't know cuz I write it?) but it need to be fix.

Has this been assigned?

@Th3nn3ss No one assigned to it. Do you want to do it?

Yes.


nondeterministic behavior

I would expect lightgbm to produce deterministic results if the parameter seed is set. However I get differences of more than .002 ROC AUC on a validation set using multiple runs of the exact same code.
I am using the following parameters:
lgb_params = {
'objective':'binary',
'boosting_type':'gbdt',
'metric':'auc',
'n_jobs': -1,
'learning_rate':0.01,
'num_leaves': 2**8,
'max_depth':-1,
'tree_learner':'serial',
'colsample_bytree': 0.7,
'subsample_freq':1,
'subsample':0.7,
'n_estimators':80000,
'max_bin':255,
'verbose':-1,
'seed': 1234,
'early_stopping_rounds':100,
}

@aamster Please post the whole snippet. Also, attach the data you use, if it's not sensitive. Or can you reproduce this issue with random data?

Hi @StrikerRUS I found out that if I keep the data set fixed, then results are reproducible. However, if I regenerate the data, i.e generate all the features, which involve a lot of math (division, aggregations, etc) then I get different results each time. Is it possible that Lightgbm will choose different splits based on differences in precision for example, each time the features are generated? There is nothing random about the features that are generated.

Did you change the order of row/column? If yes, it cannot assure the reproducible.

@guolinke yes I noticed that column order was changing. I believe this is the issue

okay, as you set colsample_bytree, the sampled columns at each round will be different, for different column order.


Init superset fail with error 'No module named 'cryptography.hazmat.backends.openssl.x509''

A clear and concise description of what the bug is.
Process

Using Ubuntu 22.04.5, MiniConda, Python3.8
install superset
superset db upgrade

Actual results
Traceback (most recent call last):
  File "/home/chick/miniconda3/envs/py38/bin/superset", line 5, in <module>
    from superset.cli.main import superset
  File "/home/chick/miniconda3/envs/py38/lib/python3.8/site-packages/superset/__init__.py", line 21, in <module>
    from superset.app import create_app
  File "/home/chick/miniconda3/envs/py38/lib/python3.8/site-packages/superset/app.py", line 23, in <module>
    from superset.initialization import SupersetAppInitializer
  File "/home/chick/miniconda3/envs/py38/lib/python3.8/site-packages/superset/initialization/__init__.py", line 33, in <module>
    from superset.extensions import (
  File "/home/chick/miniconda3/envs/py38/lib/python3.8/site-packages/superset/extensions/__init__.py", line 32, in <module>
    from superset.utils.cache_manager import CacheManager
  File "/home/chick/miniconda3/envs/py38/lib/python3.8/site-packages/superset/utils/cache_manager.py", line 24, in <module>
    from superset.utils.core import DatasourceType
  File "/home/chick/miniconda3/envs/py38/lib/python3.8/site-packages/superset/utils/core.py", line 76, in <module>
    from cryptography.hazmat.backends.openssl.x509 import _Certificate
ModuleNotFoundError: No module named 'cryptography.hazmat.backends.openssl.x509'

Environment

Ubuntu 22.04.5,
MiniConda,
Python3.8

Duplicate of https://github.com/apache/superset/issues/22571?

Duplicate of #22571?

Thanks，it works.


Making bot users, sample exam etc

We'll have to make bot users, sample exams, attempt the exams so as to make the analysis part look good and realistic
Not urgent at all ..to be done before the final presentation

Assigned to me...WOW...OH MY GOD


chore(docs): add fselect as user in README

✔ Checklist:

[ ] Commit messages have been written in Conventional Commits format;
[ ] Tests for the changes have been added (for bug fixes / features).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (06355b1) 93.83% compared to head (908ff4f) 93.99%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   93.83%   93.99%   +0.16%
==========================================
  Files          13       12       -1
  Lines         649      633      -16
==========================================
- Hits          609      595      -14
+ Misses         40       38       -2

see 8 files with indirect coverage changes

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


[feature suggestion] be clear about  802.1q management settings in switch config

trunk

The language semantics in the switch config page refers solely to VLAN, which to my understanding presents basically just L2 connectivity whilst port states tag | untag | off and VID/PVID assignment pertain instead to the management of the 802.1q protocol.

Please translate this into actionable items.

Assuming that Enable VLAN functionality just does that, i.e. enabling L2 connectivity, it could be retained as is. If it however also enables 802.1q perhaps amened it to Enable VLAN functionality with 802.1q protocol
Else for VLANs on I would humbly suggest substituting it with 802.1q protocol settings or 802.1q protocol management
Unless it is considered too technical of hair splitting and confusing the user since public domains seem often to refer to 802.1q as  VLAN tagging of sorts and more than often lack the distinction.


MAINT: Merge master and make private/public API changes

cc @rgommers

It looks like missing files are detected one by one:
Exception: /home/runner/work/scipy/scipy/scipy/sparse/tests/test_array_api.py is not installed

so you may want to run that check locally first to speed up the process.

Yeah, fixing them locally

There are many errors mostly due to *.pyx files, similar to:
ImportError while importing test module '/home/smit/Smitlunagariya/scipy/installdir/lib/python3.9/site-packages/scipy/special/tests/test_basic.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/home/smit/anaconda3/envs/scipy-dev/lib/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
../../installdir/lib/python3.9/site-packages/scipy/special/__init__.py:649: in <module>
    from . import _ufuncs
scipy/special/_ufuncs.pyx:1: in init scipy.special._ufuncs
    ???
E   ImportError: /home/smit/Smitlunagariya/scipy/installdir/lib/python3.9/site-packages/scipy/special/_ufuncs_cxx.cpython-39-x86_64-linux-gnu.so: undefined symbol: fellint_RF

fellint_RF was just merged: https://github.com/scipy/scipy/pull/15135. That PR has a small change to setup.py which you'll want to mirror to special/meson.build.

That PR has a small change to setup.py which you'll want to mirror to special/meson.build.

Yeah, I added it. I am trying to push and check if the CI passes since my ubuntu laptop had a hard disc failure(expecting to get it fixed in 2 days) and, so I'm trying from the windows laptop

Please don't mind the commits from an alternative account. I have pushed them from a different laptop.

macOS tests are being run with python 3.10 (see logs under 'Setup Conda Environment' and later). Not sure if this is the source of the failing tests.

No that should be fine - the only failures are coming from:
E   RuntimeWarning: invalid value encountered in reciprocal

which is an issue in numpy where np.reciprocal gives warnings where it shouldn't (LLVM bug) and the test suite settings therefore fail the tests (pytest setting to turn warnings into errors). Apple has fixed the LLVM bug, but we're using a version where that fix hasn't shown up yet I think.

Squashed the last few commits and merged this PR in commit 70ba03eab.

Thanks!!


Return non-zero exit code for failed tests

Currently, oslo validate exits with a 0 status even if the tests fail.  In order to properly fail in CI environments, we would exit with a non-zero status on  validation failure.

this is done


Fix broken stylesheet path for PDFs

The code points to the wrong place after the UI repo split. This changes it to point to the stylesheet under the UI engine.

@miq-bot add_label bug

@hayesr could this be related to https://bugzilla.redhat.com/show_bug.cgi?id=1442067 ?

@imtayadeway I could indeed. Not sure how graphs are generated, this fixes basic stylesheets, but things like images will take more. (I'm also looking into that)

@bdunne perhaps this generator shouldn't try to figure out the path at all. We can just expect the thing that's calling it to use the right path.

Is there an easy fix to make this pluggable? Otherwise, this fixes a bug now and maybe we can punt making it flexible until that's needed?

@hayesr something like this would work:
Vmdb::Plugins.instance.vmdb_plugins.each do |plugin|
  plugin.root.join("app/assets/stylesheets", whatever)
end

You may want to use detect rather than each if you're only looking for one.  And you'll need to see is the file exists.

@bdunne I run into a weird situation where Vmdb::Plugins.instance.vmdb_plugins is empty some times, any ideas?

It can be an empty array if you reload! in dev mode.

@hayesr can you add a before/after screenshot of the PDFs?  Thx, Dan

Restarting CI

@bdunne Ready for another look.

looks good

@hayesr @bdunne can this be euwe/yes? The BZ has the flag set and https://github.com/ManageIQ/manageiq-ui-classic/pull/1332 is euwe/yes as well.

@simaishi No, Vmdb::Plugins doesn't exist in euwe, and the UI wasn't split at that time.  I think we need a separate PR for euwe.

@bdunne thank you.
@hayesr I'll mark as euwe/conflict for now, please remove the label when Euwe PR is ready.

@simaishi If #15131 can be backported it would solve the same problem.

@hayesr In that case, is https://github.com/ManageIQ/manageiq-ui-classic/pull/1332 still needed or just need https://github.com/ManageIQ/manageiq-ui-classic/pull/1363?
With multiple PRs it's getting harder to track what's needed and not. Can you please add/remove fine/yes and euwe/yes labels where needed? I won't backport any until all needed PRs for the BZ are merged, so the labels can be added later if that's easier.

@miq-bot remove_label euwe/conflict euwe/yes
@miq-bot add_label euwe/no fine/no
@simaishi Ok, changing this one to euwe/no fine/no because the other solution is better. (branch-specific versions of #15131) may still be needed, though) Both UI PRs are needed.


[feature-request] Add Covid-19 survey into core codebase

Add the survey below:
https://bphc.hrsa.gov/emergency-response/covid-19-survey-tools-questions
Implement some common medical surveys into CardinalKit that our teams can use as-is, or modify freely.
See StudyTasks.swift
NOTE: interest on features is gaged using reactions on this first post: e.g.  👍

Templated surveys are now available in our ResearchKitOnFHIR library: https://github.com/StanfordBDHG/ResearchKitOnFHIR


Support registering system Ruby

Allows to avoid downloading/installing Ruby interpreter at the expense of losing hermeticity and no Ruby version check.
# WORKSPACE
load("@rules_ruby//ruby:deps.bzl", "rb_register_toolchains")

rb_register_toolchains(
    version = "system",
)

@kigster Does it look reasonable for your case of migrating from Bazelruby?

I am not sure — if by "system" you mean the default Ruby installed by OS-X, then unfortunately it's not the same.
The old rules_ruby supported the local rbenv instalation which typically resides under $HOME/.rbenv/versions/<version>.  There may be multiple versoins there, and each project is allowed to set required ruby version by including the .ruby-version file, which makes rbenv automatically switch to that version.
A mono-repo may have several sub-projects that reference different ruby versions. While this is not something I would recommend anyone to do in practive, the reality is that this does happen quite often within large companies.
Therefore I would suggest differentiating between system ruby, which on OS-X is the pre-installed ruby (and which, unfortunately, is installed on a read-only file system, and does not permit adding any additional gems, and is therefore largely unusable unless you are relying on stdlib and nothing else), and rbenv or even rvm rubies, which are in installed under ~/.rbenv or ~/.rvm folders respectively.

As mentioned in the documentation, "system" means Ruby found on the PATH. If it's registered by rbenv, then it will be picked up respectfully.


Alignment sign off with final query

[ ] Alignment sign off
[ ] Possibly remove some videos from exclude


Add Profile: imvignesh003

A pull request should be created from this issue if not kindly ping @dinxsh ⚠

@all-contributors please add @imvignesh003 for review


add read and write file by fragment and README_ZhCN.md

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-9224.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ NIFI-9224] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ NIFI-9224] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ n] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ y] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ y] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ y] Have you written or updated unit tests to verify your changes?
[ y] Have you verified that the full build is successful on JDK 8?
[ y] Have you verified that the full build is successful on JDK 11?
[y ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ n] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ n] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ n] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ y] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@OIUE Thank you for your contribution.
This pull request in its current form contains multiple merge commits resulting in over 1500 files changed. A new pull request should not contain any merge commits, and changes should be limited to the general scope of the associated Jira issue.
Please open a new pull request with the intended changes squashed into a single commit.  That will provide an opportunity to review the intended changes without the confusion related to merge commits and conflicting files.


The eval of MER

When we adapt the meta-model to a new task, we need to train the meta-model on the training set of the new task. But the eval_tasks function in MER seems to eval on the test  set of new tasks directly. Is that a problem? Does MER still belongs to meta-learning approch?

Hi thank you for your question! Our paper is about a method for applying meta-learning in continual learning settings. As such, our evaluation setting is modeled after prior work on continual learning. Our eval_tasks code comes directly from the GEM project https://github.com/facebookresearch/GradientEpisodicMemory, which was the main point of comparison in our paper. Maybe your confusion is related to meta-training and meta-testing? We do not have any notion of meta-training and meta-testing in or work. In fact, this can be seen as one of the main innovations of our paper. MER applies meta-learning in a generic continual learning setting without the need to break learning up into multiple separate phases. The main.py file is not specific to the MER approach and is almost exactly taken from the GEM repository. The eval_tasks function is simply used for reporting results over time as training progresses -- not for actually training the model. Most of the code provided follows the protocol from the GEM paper with our main contribution being in the meralg*.py files in the model directory. I hope this helps! However, I am not sure if I totally understood your concern, so feel free to clarify if this response doesn't make sense.

Thanks for your reply. In my opinion, your work is extending from Reptile and GEM. And the Reptile does not need to seperate the train-set of train-task into support-set and query-set, which trains on the whole train-set of train-task directly. But in the eval of Reptile, it still need to train on the train-set of test-task, and then test on the test-set of test-task, which adapt the meta-model on the test-task.
Is that your work mainly uses Reptile's framework and focus on  solving the lifelong learning problem without considering the capacity of meta learning to adapt to new task?

Thanks for your reply. In my opinion, your work is extending from Reptile and GEM. And the Reptile does not need to seperate the train-set of train-task into support-set and query-set, which trains on the whole train-set of train-task directly. But in the eval of Reptile, it still need to train on the train-set of test-task, and then test on the test-set of test-task, which adapt the meta-model on the test-task.
Is that your work mainly uses Reptile's framework and focus on  solving the lifelong learning problem without considering the capacity of meta learning to adapt to new task?

Yes. In this paper we are focused on our evaluating our approach in a lifelong continual learning setting. The model must adapt to new tasks when they are presented without forgetting old tasks. The improved adaption can be seen based on improvements on metrics like forward transfer and backward transfer as well as accuracy on tasks directly after learning and retained accuracy after learning all tasks. We did not, however, consider a meta-testing or few-shot learning based evaluation setting. This would be a lot closer to the original Reptile paper, which was not our goal here.


Tensorboard log_hyperparams(params, metrics) seems not to have effect

🐛 Bug
Calling self.logger.log_hyperparams(hparams_dicts, metrics_dicts) in test_epoch_end doesn't have the desired effect. It should show the entries in the Hparams section with hparams and metrics specified but it shows nothing instead.
Looking at the code it seems to be caused by self.hparams and the pre-logging of the hyperparameters at the start of the training. In this way, calls to log_hyperparams won't be able to log the hyperparameters AND the metrics properly since they will clash with the previous log, hence, showing nothing.
To Reproduce
Try to log metrics with self.logger.log_hyperparams.

Code sample
def test_epoch_end(self, outputs):
    avg_recall = np.concatenate([x['recall'] for x in outputs]).mean()
    tensorboard_logs = {'test/avg_ndcg': avg_ndcg, "test/avg_recall": avg_recall}
    ## Log metrics
    self.logger.log_hyperparams(vars(self.params),tensorboard_logs)
    return tensorboard_logs

Expected behavior
Tensorboard should show me the section of Hparams with each entry composed by hyperparameters and metrics.

Environment

CUDA:
- GPU:
- available:         False
- version:           10.2
Packages:
- numpy:             1.18.1
- pyTorch_debug:     False
- pyTorch_version:   1.5.0
- pytorch-lightning: 0.7.5
- tensorboard:       2.2.1
- tqdm:              4.46.0
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.8.2
- version:           #34~18.04.1-Ubuntu SMP Fri Feb 28 13:42:26 UTC 2020

Additional context

yes, hparams seems to not be working for some reason. looking into it.

I played around the issue and I think I may know the problem. When calling log_hyperparams the function adds events to the log file instead of overwrite the existent one (the one created in the pre-train routine). It follows that tensorboard sees one file with different sets of metrics, { } from the first log and the last added with log_hyperparams. This leads to hparams not showing it properly since Tensorboard wants to have all set of metrics to be the same across the experiments.
In order to see if works, you can just have "del self.hparams" at the beginning of your pl.lightiningmodule. This effectively jumps the pre-train logging and correctly shows hparams in tensorboard. Downside? Cannot load from checkpoint anymore ;)

@justusschock is this related to the changes with initializing tb?

Probably it is (we used the writers internal add_hparams for this, which seems to create a separate file). Hopefully fixed this in #1630

Confirmed this is fixed on master,
https://colab.research.google.com/drive/1K6Gxo99O6dEbzzj_lW8jAL74OvjojV9T

The issue is not fixed yet, unfortunately.
Callinglog_hyperparams(vars(self.hparams),{"SOME_METRIC":2012}) in test_epoch_end won't log the metric in hparams. The problem is still the same. The inital log of the hparameters interfers with the last call and won't allow to log metrics in hparams!

def test_epoch_end(self, outputs):
    avg_ndcg = np.concatenate([x['ndcg'] for x in outputs]).mean()
    avg_recall = np.concatenate([x['recall'] for x in outputs]).mean()

    tensorboard_logs = {'test/avg_ndcg': avg_ndcg, "test/avg_recall": avg_recall}
    ## Log metrics
    #self.logger.log_metrics(tensorboard_logs)

    self.logger.log_hyperparams(vars(self.hparams),tensorboard_logs)
    return tensorboard_logs

this is fixed on master. And will be live tomorrow in 0.7.6.

ummmm. this might be a design problem in TB.

there’s never a guarantee your model ends training (cluster interrupt, crash, etc..). In those instances you still want to know the hparams
the TB design assumes your training always completes for metrics...

so, looks like we have to get hacky to get this to work correctly?

I think they may fix this in the future but I don't think in the short time :/
https://github.com/tensorflow/tensorboard/issues/3597

@karapostK  any luck with finding a fix/workaround ?

@singhay I found one but it's not very nicey. In line 840 (run_pretrain_routine function) in Trainer.py there is this code:
    if self.logger is not None:
        # save exp to get started
        if hasattr(ref_model, "hparams"):
            self.logger.log_hyperparams(ref_model.hparams)

which logs the hparams too soon. Simply changing this to:
    if self.logger is not None:
        # save exp to get started
        if hasattr(ref_model, "hparams"):
            pass

does the job. However, remember to call log_hyperparams(hparams,metrics) somewhere in your code in order to properly log the metrics. EIther using the callback on_train_end() or on test_epoch_end.

Have the same problem and I think pl should not log hparams blindly in trainer. What's the point of logging hparams without a metric? It is been saved to disk anyway, so one could look there. If someone can only look at tensorboard (without access to the machine), then it is better to log hparams as text.

most of the time we interrupt training before it completes... a lot of code won’t “complete”  but you still need checkpoints and to know what you ran

+1 Thanks everyone for looking into this. I've also been searching a way to use the hparams tab in tensorboard with lightning. I think @karapostK's solution needs an update: now one should just comment out self.logger.log_hyperparams(ref_model.hparams) in run_pretrain_routine  in trainer.py.
Maybe one solution is to just have a Trainer option like pre_record_hyperparams which is default True, and when False, it turns off this line: https://github.com/PyTorchLightning/pytorch-lightning/blob/325852c6df93f749bb843bff1a3cdba41698722c/pytorch_lightning/trainer/trainer.py#L1077
we can change it to be:
        if self.logger is not None and self.pre_record_hyperparams:

and then the user will manually call log_hyperparams whenever they see fit, and also include the desired metrics.

For anybody trying to solve the same issue for their code, here is how I solved it with a hack:

Change if self.logger is not None: to if False: in trainer.py.
Before training, I have

checkpointer = ModelCheckpoint(filepath='best')

(and add it as a callback to trainer: Trainer(..., checkpoint_callback=checkpointer)).
3. After training, run:
logger.log_hyperparams(params=model.hparams, metrics={'val_loss': checkpointer.best_model_score.item()})
logger.save()

Then the model will appear in your hparams tab with val_loss as a metric.

Closing as this is a TB issue. @versatran01 feel free to reopen if you have any other issues!

Found an answer to this question here:
https://pytorch-lightning.readthedocs.io/en/latest/extensions/logging.html#logging-hyperparameters
(https://github.com/PyTorchLightning/pytorch-lightning/discussions/6904)


suggest adding missing compat entries in bulk

If one happens to forget a [compat] section in a Project.toml altogether, then CompatHelper will create one PR per dependency. It'd be much more helpful to just add a single PR to add them in bulk - unlike compat changes, the new compat entries shouldn't have an effect anyways, so it should be possible to just have them in a single PR.

Seems to be a duplicate of https://github.com/JuliaRegistries/CompatHelper.jl/issues/126.

Duplicate of #126


Character will fall through the ground when ragdoll is active if bTickPhysicsAsync = ture

I find this commit will make the ragdoll fall through the ground. I think it still can not be true.

Reverted these changes. It seems that at low FPS the character still falls through the ground.


Update worker-list and add shift-find

Added shift-find command, related tests and documentation updates.
Resolves #147

Codecov Report

Merging #197 into master will increase coverage by 0.07%.
The diff coverage is 86.95%.

@@             Coverage Diff              @@
##             master     #197      +/-   ##
============================================
+ Coverage     69.29%   69.36%   +0.07%
- Complexity      841      847       +6
============================================
  Files           136      138       +2
  Lines          2983     3000      +17
  Branches        408      410       +2
============================================
+ Hits           2067     2081      +14
- Misses          791      792       +1
- Partials        125      127       +2

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/mcscheduler/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...va/mcscheduler/logic/parser/McSchedulerParser.java
47.05% <50.00%> (-1.43%)
12.00 <0.00> (ø)

...a/mcscheduler/logic/commands/ShiftFindCommand.java
77.77% <77.77%> (ø)
3.00 <3.00> (?)

.../mcscheduler/logic/commands/WorkerFindCommand.java
100.00% <100.00%> (ø)
6.00 <1.00> (?)

...cheduler/logic/parser/WorkerFindCommandParser.java
100.00% <100.00%> (ø)
3.00 <1.00> (?)

...cscheduler/model/shift/ShiftFindCommandParser.java
100.00% <100.00%> (ø)
3.00 <3.00> (?)

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 a9ec971...ee8950e. Read the comment docs.

@wakululuu Thanks for suggestions! They have been committed. All references to cleaner have been swapped to janitor too.


Corona-Warn-App Spanish translation

Avoid duplicates

[x] This issue has not already been raised before
[ ] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation

No Spanish language support.
Suggested Enhancement

Translate the Corona-Warn-App into Spanish language.
Expected Benefits

Spanish community will be able to leverage the application enabling the app to reach a bigger audience in Germany.

I'm already working on this translation. I would like to later send the PR with the issue resolution.

Dear @adrianapintod , thank you for your offer. As noted in corona-warn-app/cwa-documentation#350 translations are done by a dedicated team in close collaboration with the RKI. Hence, we cannot accept community translations at this point.


Fix the bug of generating global dict when using dimension relation based on new load DML

as issue #120 has shown, load table DML will be changed to INSERT INTO and ashok has raise a pr #169 to resolve this, based on thess changes, i design the dict generating from pre-defined dimensions as below:
1.the user should create a dimension table when he wanna use external-column file(no matter single or multiple columns) to generate dict. the DDL statement is just like #169 shows.
2.in the load part,  get the pre-defined dimensions data from dimension table and filter then generate global dict.
3. No using pre-defined dimension data in fact table to generate global dict.
since the most changes have been made in past pr, i make the further little changes based on pr #169 , pls check. @gvramana @Vimal-Das @ashokblend what is your idea?

please do rebase


Added working cocktail shaker sort

Added cocktail sort to the list of algorithms.

Thank you again for your contribution! This time I cloned your repo and everything works as expected, really cool algorithm!
Also, some tips for your next contributions:

When you create the PR, every commit that you made to the branch that you're trying to merge, will be updated in the PR too, so you don't need to create 2 different PRs!
Try to add a bit of info to the PR, for example, if this PR fixes any issue (in this case #3), etc.
If you're solving an issue, you can comment first on the issue thread just in case someone is working on this too, but don't worry because this project is small.

This is just some advice for your next PRs, everything else looks great! Thanks for the contributions and congrats! 🙌


kubectl invoked from incorrect location during bootstrap

What happened:
One command in the cloud-init bootstrap is as follows:
kubectl create secret tls cluster-etcd -o yaml --dry-run=client -n kube-system --cert=/var/lib/rancher/rke2/server/tls/etcd/server-ca.crt --key=/var/lib/rancher/rke2/server/tls/etcd/server-ca.key --kubeconfig /etc/rancher/rke2/rke2.yaml | kubectl apply -f- --kubeconfig /etc/rancher/rke2/rke2.yaml

The problem with this in the first place is that it relies on kubectl, making it a dependency in order to bootstrap.
I wonder if there is a better way to add this secret.
The other problem is that kubectl is not in PATH. This is not supported by RKE2 and it's going to fail on any RKE2 installation.
As the documentation suggests, /var/lib/rancher/rke2/bin/kubectl should be used.
However it's unclear to me if these CLI tools are going to stay or are just there for debugging/testing purposes. Using  /var/lib/rancher/rke2/bin/ may stop working unexpectedly because of that.
Also be aware that this bug is currently silently failing due to #221
What did you expect to happen:
How to reproduce it:
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

rke provider version: v0.3.0
OS (e.g. from /etc/os-release):

Relevant to this issue, is that the /var/lib/rancher/rke2 may not be hardcoded, because it can be set by the user.
I don't know if the data-dir can already be configured on the RKE2 provider, but it may be in the future.
Relevant issue: https://github.com/rancher/rancher/pull/45326

@anmazzotti I believe this was fixed in https://github.com/rancher/cluster-api-provider-rke2/pull/404, I’ll close this issue.


allow server deafening of self on join

also removed a buncha raise commands.errors.CommandInvokeError
they do not work, instead they raise only one type of error, a type error and is not useful at all

raise was used to stop the command from running itself

used your version as a baseplate and modified alot of it


Webviews incorrectly positioned when editors are scrolled horizontally

small/narrow window
open two webview editors
:bug: you can horizontally scroll the editors/headers

https://user-images.githubusercontent.com/1794099/108362852-89a23500-71f4-11eb-9c8c-862dd741a670.mov

What a fantastic feature!

This is actually webview related, since text editors work as expected:

Maybe @mjbvz knows why these webviews aren't moving when everything scrolls left?

Oh wow, I had no idea:

@mjbvz Let me know what needs to be pinged for a relayout here.

Same root causes  as #110450
We'd need to be updated whenever the editor element's position on the screen changes. We may be able to use IntersectionObserver for this
In addition, we'd need to figure out the visible part to the editor element. Because webviews must be at the top level of the dom (in order to avoid being reparented), they will not be properly cropped if an editor is half on the screen and half hidden

We'd need to be updated whenever the editor element's position on the screen changes. We may be able to use IntersectionObserver for this

Where's we? No need for observers since it all happens in JS, we know exactly where the scroll happens, we just need to wire it up.

In addition, we'd need to figure out the visible part to the editor element.

Yeah that's trickier!

The webview code needs to know where to absolutely position the webview. Here's the code that does this:
https://github.com/microsoft/vscode/blob/2e1166cb0c204c048de89eab009aaa8dec616ef2/src/vs/workbench/contrib/webview/browser/dynamicWebviewEditorOverlay.ts#L126
You can look through the codebase to see all the places where layoutWebviewOverElement is called today

@joaomoreno Can you suggest how I can hook into scrolling the editor area?
I think the fix here would be very similar to https://github.com/microsoft/vscode/commit/88c1322a82a2bdf3fe7e5543684b5a36251c87b7 but I can't track down where the scroll events are fired in this case

This is the one:
https://github.com/microsoft/vscode/blob/dff33d73244acced865a2643a62126409cb541a7/src/vs/base/browser/ui/splitview/splitview.ts/#L240-L241
But you've seen this, since it's what eventually gets you here:
https://github.com/microsoft/vscode/blob/dff33d73244acced865a2643a62126409cb541a7/src/vs/base/browser/ui/splitview/splitview.ts/#L240-L241
But that is only at the splitview level. Since the grid is a tree of spltiviews, we probably should join all those splitviews' onScroll events into a global grid onScroll event, which would be the one you could listen to. I've done that here: https://github.com/microsoft/vscode/commit/0e3459b0b9facc5e7a1e95903955f870bc5a532c#diff-ffad104855d5ea285f8b470156736e7325b8892ae4336970a74b8bc9e8110803R215
So you now have that event as in grid.onDidScroll. I suggest to debounce it just a bit, cause it may fire many many times!

Note for testing: this should be fully fixed for webviews but notebooks have may still draw over the side bars when scrolling: #150490

@mjbvz You are right that Notebooks are drawing over side bars as below. Is there an issue open for tracking this?


Won't autofill passwords

With the newest version passwords are not auto filled. Per this stack exchange post I believe you can fix it by changing "?pwd=" to "&pwd=" in launch_zoommtg_url
Example:
zoom save -n [name] --id [id] -p [pwd]
zoom name

Starts zoom and tried to join but does not fill in the password. Same issue if I use --url and -p arguments
When I manually run
open "zoommtg://zoom.us/join?action=join&confno=<id>?pwd=<pwd>"

I get the same behaviour, but changing ? to & the meeting will launch

Can you provide what command you are running to save? Interim fix would be to include password in meeting url when saving.

Including the password in either the & or ? format doesn't seem to help

Hmm I know that zoom hashes the passwords in the URL format for security reasons. Do you have a single zoom link with the hashed password in it, rather than the separate password?

Yes i save one, I tired using zoom save -n test --url [URL with hashed pwd]  when I ran zoom test nothing happens and no error is thrown

@tomginsberg can you try updating to v1.1.6 and see if it works for you? If homebrew is giving you the sha256 mismatch again, untap the cask and re-tap. I found that should do the trick.

great it works!

Nice! Will close this then.


feat: 12 hour time based on Locale

Show 12 or 24 hour time based on user settings. The sell chip takes a lot of space, so this moves the time above it.

Awesome :D


Port About Page From Old Website

Port About Page from the old website over. Make sure to work on the correct branch.
Old Github repo.

 Add About Page

I did this myself, no longer necessary.


Help to execute the EW.py code

Please tell me the correct way to input my spectrum FITS file with the other necessary input parameters that are required by the code  EW.py code of your PHEW (PytHon Equivalent Widths).

Hi, the spectrum format that works best with PHEW.EW() is to have your spectrum data loaded as an array with wavelength, flux, and optional flux error as columns. For instance, you can make a list with two 1d numpy arrays: wavelength as a 1d numpy array and flux as a 1d numpy array. Try using the function read_spec() in the module readspec.py to see if it works with your fits files.
For the rest of the inputs, please refer to section 4.2 of the user guide, and let us know if you have specific questions about those.

Hi, Thanks for helping me out. I have now managed to run the code. Actually, the issue was with my FITS file. As you suggested, instead of loading my FITS file, I loaded my spectrum data as a list containing three 1d numpy arrays of wavelength, flux and error, following which PHEW.EW() worked.


Add observable to indicate if User Profile was loaded

In the main service of this sample we load the User Profile like this:
https://github.com/jeroenheijmans/sample-angular-oauth2-oidc-with-auth-guards/blob/4ee8e2862bb9b825015a9bd5213d07984c9cea13/src/app/core/auth.service.ts#L73-L75
I suspect some people will want to "wait" for the profile to be loaded before doing stuff. So it might make sense to add isUserProfileLoaded$ observable similar to isAuthenticated$ and isDoneLoading$.

However this is solved, this would be very useful for us. We have several components that rely on information coming from the response to loadUserProfile. As it stands, we're dealing with a race condition where a component is loaded before the user profile request completes. We can probably add something akin to your APP_INITIALIZER process. However (and this is untested and could be completely incorrect), I believe that if a user were to logout and log back in (not refreshing/reinitializing the app), then the user profile data would need to be re-requested and I'd be back in the same situation.
Anyway, I'm interested in a solution here and happy to provide more details of our situation to help develop one.

Thanks for the feedback! That sounds like a typical use case indeed.
Don't think too much of the proposed feature by the way, it would probably be a very straightforward boolean observable similar to isAuthenticated$.
I'm not sure if I'll get to introducing this all too soon though, as my open source focus is currently mostly elsewhere.
If anyone ends up doing an implementation feel free to let us know what approach you took!


Add missing fields to billing admin transaction class

We aren't currently displaying the new fields added in #1118 in the admin UI.

Ready for review.

Ready for review.

:+1:


Input in basic starter loses focus after loading script

Hi,
First of all, thanks for building this very exciting piece of technology :)
I'm just trying the simple starter  and I think there is something that's not working as intended. When I first start typing something in the text field it loses focus almost immediately (when the script finishes loading?). But this doesn't happen re-focusing and typing again, check my short video for demonstration. This is surely not the intended behaviour?
https://user-images.githubusercontent.com/376921/147286688-487b0ab0-dd96-4666-8342-036bc240969c.mov

I noticed this issue too

Tried a few different scenarios and looked at the flash of updates in dev tools...
Original example

Put the text in a paragraph (focus issue goes away)

Placing the input as a lone sibling and things work as expected?

Any ideas? 🤔

I found that the replaceNode function causes this by removing the input element before putting the mounted input. So perhaps it should keep focus after replacing. You'll see this by setting a break point when the initial input element is removed.

I looked into it. The issue with this line: https://github.com/BuilderIO/qwik/blob/e4493e57f1e5bdad89b071ab0c824a9efd367e93/starters/apps/starter/src/my-app.tsx#L25
Notice that there is text + {' '}. Thes creates two text nodes. When these text nodes get serialized into HTML they get merged into one. Once the reconciliation algorithm runs the result is that the JSX does not match the HTML and results in the <input being destroyed and re-created. This results in loss of focus.

Fixed by https://github.com/BuilderIO/qwik/commit/7fadd186e2358c95cd5902a800c24b448e6d19c5


implement structured streaming datahub source

What changes were proposed in this pull request?
support datahub source in  structured streaming #226
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)

This branch is out-of-date with the base branch


Auth wall doesn't trigger on personal_website

if signed_in? auth wall logic replaced by @auth_wall boolean
@auth_wall is false on Personal Website workshop so new users can create
GitHub accounts

This is outdated


question about evaluate

When I evaluate the model by phase eva , a error is occured.Could you help me to solve it ?
And I want to ask you how to calculate the ap or map?
Thank you,Best wish

It looks like your ResultMerge_multi_process.py has some issues. You may need to debug the code. For example, did you generate the result_dota folder before you call dsets.merge_crop_image_results(result_path, merge_path) in eval.py? You can use online server https://captain-whu.github.io/DOTA/evaluation.html to calculate AP dota. The hrsc dataset is calculated by ap = dsets.dec_evaluation(result_path) in eval.py.

Thank you very much.Before I call dsets.merge_crop_image_results(result_path, merge_path) in eval.py,It has genrate the result_dota folder

Does the result_dota contain the right files? like Task1_baseball-diamond.txt files contain
P0045__0.5__0___500 0.683183252811 446.0 444.2 465.2 560.3 348.0 578.1 328.8 461.9
P0045__0.5__0___500 0.636462032795 407.4 252.5 429.9 360.8 315.2 383.2 292.8 274.9
P0045__0.5__0___500 0.531519293785 592.8 219.0 615.2 325.4 516.1 345.5 493.7 239.1
P0045__0.5__0___500 0.160389885306 593.1 411.4 614.1 517.1 547.6 532.2 526.6 426.5
P0045__0.5__0___803 0.659127235413 445.5 139.0 467.2 246.9 355.6 267.6 333.9 159.7
P0045__0.5__0___803 0.542766690254 348.1 382.6 370.6 490.0 255.8 513.1 233.3 405.8

yes ,it have

Sorry I have no idea. You may need to debug to see what makes the index out of range.

When I evaluate the model by phase eva , a error is occured.Could you help me to solve it ?
And I want to ask you how to calculate the ap or map?
Thank you,Best wish

我也遇到这个问题了，请问您解决了吗？

By debugging I found the error  causes in DOTA_devkit.ResultMerge_multi_process:line 199-202
but to utilize debugging better,  I recommend using ResultMerge instead of ResultMerge_multi_process in datasets.dataset_dota:line 5
In my case I convert HRSC2016 format to dota format, and results from HRSC2016 as following:
100000849 0.318158239126 53.9 147.5 100.6 211.7 29.2 238.1 -17.4 173.9
100001347 0.189944535494 6.5 336.8 38.7 353.3 18.7 414.5 -13.6 398.0

But in dota, it will use filename to get metadata, like ratio=0.5 x=0 y=500 from P0045__0.5__0___803 0.542766690254 348.1 382.6 370.6 490.0 255.8 513.1 233.3 405.8
so when module re find nothing from my file name, it will cause list index out of range by  x_y_2 = re.findall(r'\d+', x_y[0])...

By debugging I found the error causes in DOTA_devkit.ResultMerge_multi_process:line 199-202
but to utilize debugging better, I recommend using ResultMerge instead of ResultMerge_multi_process in datasets.dataset_dota:line 5
In my case I convert HRSC2016 format to dota format, and results from HRSC2016 as following:
100000849 0.318158239126 53.9 147.5 100.6 211.7 29.2 238.1 -17.4 173.9
100001347 0.189944535494 6.5 336.8 38.7 353.3 18.7 414.5 -13.6 398.0

But in dota, it will use filename to get metadata, like ratio=0.5 x=0 y=500 from P0045__0.5__0___803 0.542766690254 348.1 382.6 370.6 490.0 255.8 513.1 233.3 405.8
so when module re find nothing from my file name, it will cause list index out of range by x_y_2 = re.findall(r'\d+', x_y[0])...

I see. So it looks like if the data is not arranged in the right way (e.g. merge format), it will raise exceptions. Thanks @igo312

I want to ask if the pictures of test set also be split

It looks like your ResultMerge_multi_process.py has some issues. You may need to debug the code. For example, did you generate the result_dota folder before you call dsets.merge_crop_image_results(result_path, merge_path) in eval.py? You can use online server https://captain-whu.github.io/DOTA/evaluation.html to calculate AP dota. The hrsc dataset is calculated by ap = dsets.dec_evaluation(result_path) in eval.py.

Which command to execute can be generated  result_dota folder？

Does the result_dota contain the right files? like Task1_baseball-diamond.txt files contain
P0045__0.5__0___500 0.683183252811 446.0 444.2 465.2 560.3 348.0 578.1 328.8 461.9
P0045__0.5__0___500 0.636462032795 407.4 252.5 429.9 360.8 315.2 383.2 292.8 274.9
P0045__0.5__0___500 0.531519293785 592.8 219.0 615.2 325.4 516.1 345.5 493.7 239.1
P0045__0.5__0___500 0.160389885306 593.1 411.4 614.1 517.1 547.6 532.2 526.6 426.5
P0045__0.5__0___803 0.659127235413 445.5 139.0 467.2 246.9 355.6 267.6 333.9 159.7
P0045__0.5__0___803 0.542766690254 348.1 382.6 370.6 490.0 255.8 513.1 233.3 405.8

When I execute the eval code, the result_dota file is empty. Is there a problem with my storage path?

I want to ask if the pictures of test set also be split

Yes, the dota test set is also split.

Does the result_dota contain the right files? like Task1_baseball-diamond.txt files contain
P0045__0.5__0___500 0.683183252811 446.0 444.2 465.2 560.3 348.0 578.1 328.8 461.9
P0045__0.5__0___500 0.636462032795 407.4 252.5 429.9 360.8 315.2 383.2 292.8 274.9
P0045__0.5__0___500 0.531519293785 592.8 219.0 615.2 325.4 516.1 345.5 493.7 239.1
P0045__0.5__0___500 0.160389885306 593.1 411.4 614.1 517.1 547.6 532.2 526.6 426.5
P0045__0.5__0___803 0.659127235413 445.5 139.0 467.2 246.9 355.6 267.6 333.9 159.7
P0045__0.5__0___803 0.542766690254 348.1 382.6 370.6 490.0 255.8 513.1 233.3 405.8

When I execute the eval code, the result_dota file is empty. Is there a problem with my storage path?

If you are testing dota, make sure your test set is split as the trainval set.

Thank you. I guess my category is wrong. I want to train the model again. But it report errors "AttributeError: 'Namespace' object has no attribute 'resume_train'".
------------------ 原始邮件 ------------------
发件人:                                                                                "yijingru/BBAVectors-Oriented-Object-Detection"<EMAIL_ADDRESS>发送时间: 2020年11月26日(星期四) 凌晨1:10
收件人<EMAIL_ADDRESS>抄送: "王帅帅"<1315975525<EMAIL_ADDRESS>主题: Re: [yijingru/BBAVectors-Oriented-Object-Detection] question about evaluate (#4)
Does the result_dota contain the right files? like Task1_baseball-diamond.txt files contain
P0045__0.5__0___500 0.683183252811 446.0 444.2 465.2 560.3 348.0 578.1 328.8 461.9 P0045__0.5__0___500 0.636462032795 407.4 252.5 429.9 360.8 315.2 383.2 292.8 274.9 P0045__0.5__0___500 0.531519293785 592.8 219.0 615.2 325.4 516.1 345.5 493.7 239.1 P0045__0.5__0___500 0.160389885306 593.1 411.4 614.1 517.1 547.6 532.2 526.6 426.5 P0045__0.5__0___803 0.659127235413 445.5 139.0 467.2 246.9 355.6 267.6 333.9 159.7 P0045__0.5__0___803 0.542766690254 348.1 382.6 370.6 490.0 255.8 513.1 233.3 405.8
When I execute the eval code, the result_dota file is empty. Is there a problem with my storage path?
If you are testing dota, make sure your test set is split as the trainval set.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I guess the catalog is wrong. When I want to train it again, it report error. My image size has been cut to 608*608.
------------------ 原始邮件 ------------------
发件人:                                                                                "yijingru/BBAVectors-Oriented-Object-Detection"<EMAIL_ADDRESS>发送时间: 2020年11月26日(星期四) 凌晨0:24
收件人<EMAIL_ADDRESS>抄送: "王帅帅"<1315975525<EMAIL_ADDRESS>主题: Re: [yijingru/BBAVectors-Oriented-Object-Detection] question about evaluate (#4)
I want to ask if the pictures of test set also be split
Yes, the dota test set is also split.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I want to ask if the pictures of test set also be split

Yes, the dota test set is also split.

您好，我用了ROI Transformer的源码分割了DOTA数据集，但是我没有得到您论文里说的那么多张图片。我最后复现的结果map只有69.7%左右，不知道和图片分割有没有关系。

I want to ask if the pictures of test set also be split

Yes, the dota test set is also split.

您好，我用了ROI Transformer的源码分割了DOTA数据集，但是我没有得到您论文里说的那么多张图片。我最后复现的结果map只有69.7%左右，不知道和图片分割有没有关系。

你好，请问你是分割成608*608大小的图片吗

I want to ask if the pictures of test set also be split

Yes, the dota test set is also split.

您好，我用了ROI Transformer的源码分割了DOTA数据集，但是我没有得到您论文里说的那么多张图片。我最后复现的结果map只有69.7%左右，不知道和图片分割有没有关系。

你好，请问你是分割成608*608大小的图片吗

可能不是，我没有改代码直接分割的，作者的意思是要分割成608*608大小的图片吗

这是论文中的截图，我训练和测试还没跑起来可否请教几个问题？
------------------ 原始邮件 ------------------
发件人:                                                                                "yijingru/BBAVectors-Oriented-Object-Detection"<EMAIL_ADDRESS>发送时间: 2020年11月26日(星期四) 晚上9:36
收件人<EMAIL_ADDRESS>抄送: "王帅帅"<1315975525<EMAIL_ADDRESS>主题: Re: [yijingru/BBAVectors-Oriented-Object-Detection] question about evaluate (#4)
I want to ask if the pictures of test set also be split
Yes, the dota test set is also split.
您好，我用了ROI Transformer的源码分割了DOTA数据集，但是我没有得到您论文里说的那么多张图片。我最后复现的结果map只有69.7%左右，不知道和图片分割有没有关系。
你好，请问你是分割成608*608大小的图片吗
可能不是，我没有改代码直接分割的，作者的意思是要分割成608*608大小的图片吗
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I want to ask if the pictures of test set also be split

Yes, the dota test set is also split.

您好，我用了ROI Transformer的源码分割了DOTA数据集，但是我没有得到您论文里说的那么多张图片。我最后复现的结果map只有69.7%左右，不知道和图片分割有没有关系。

In particular, the images are cropped into 600×600 patches with a stride of 100. The input images have two scales 0.5 and 1.


Any plans of issuance of tokens to fund this project?

It could help to develop it faster.

Yes, possibly. Did you have a platform in mind?

Yes, we're considering various options.  Closing this issue for now; feel free to reopen.


Create ARCHITECTURE.md

An overview of the dojo architecture would be super helpful to new devs and infra-level users. It can probably go to ARCHITECTURE.md in this repo.
I think a great route toward this would be to adapt the DOJO paper (https://sefcom.asu.edu/publications/dojo-sigcse24.pdf) into a living markdown document.

@Scoder12 mentioned https://matklad.github.io/2021/02/06/ARCHITECTURE.md.html, which I am including here since it has been sitting as an open tab since he mentioned it.

Ah, that's where I heard that! I couldn't root-cause its presence in my mind, thanks!

It would be really great for new devs like us. 👍