unit test tracked by issue#1682

What is the purpose of the change
#1682: Enhance the test coverage part-4 : dubbo-common/src/main/java/com/alibaba/dubbo/common/status(store|threadpoolutils) modules
Brief changelog
dubbo-common/src/test/java/com/alibaba/dubbo/common/utils/LogHelperTest.java
dubbo-common/src/test/java/com/alibaba/dubbo/common/utils/LogTest.java
dubbo-common/src/test/java/com/alibaba/dubbo/common/utils/LogUtilTest.java
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #1889 into master will increase coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1889      +/-   ##
============================================
+ Coverage     42.57%   42.71%   +0.14%
- Complexity     4710     4737      +27
============================================
  Files           646      646
  Lines         30657    30657
  Branches       5378     5378
============================================
+ Hits          13051    13096      +45
+ Misses        15594    15511      -83
- Partials       2012     2050      +38

Impacted Files
Coverage Δ
Complexity Δ

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
37.5% <0%> (-12.5%)
3% <0%> (ø)

...a/dubbo/remoting/transport/netty/NettyChannel.java
57.5% <0%> (-8.75%)
19% <0%> (-2%)

...n/java/com/alibaba/dubbo/common/utils/JVMUtil.java
72.22% <0%> (-7.41%)
13% <0%> (-2%)

...libaba/com/caucho/hessian/util/IdentityIntMap.java
25% <0%> (-5.96%)
5% <0%> (-2%)

.../com/alibaba/dubbo/monitor/dubbo/DubboMonitor.java
87.85% <0%> (-1.87%)
15% <0%> (ø)

...baba/dubbo/remoting/transport/mina/MinaClient.java
57.81% <0%> (-1.57%)
8% <0%> (-1%)

...bo/remoting/transport/netty/NettyCodecAdapter.java
53.12% <0%> (-1.57%)
3% <0%> (ø)

.../main/java/com/alibaba/dubbo/common/utils/Log.java
35.13% <0%> (+8.1%)
9% <0%> (+3%)
:arrow_up:

...ba/dubbo/remoting/transport/netty/NettyClient.java
82.75% <0%> (+8.62%)
13% <0%> (+1%)
:arrow_up:

...java/com/alibaba/dubbo/common/utils/LogHelper.java
40% <0%> (+40%)
15% <0%> (+15%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Merge sort ,selection sort, bubble sort

Merge sort. selection sort ,bubble sort
Merge Sort is a Divide and Conquer algorithm that divides an array into two halves, recursively sorts both halves, and then merges the sorted halves to produce a sorted array . using language java

@indrakishore   "I would love the opportunity to contribute and work on this project.
Could you please assign the issue to me? I'm eager to dive in and make a meaningful impact. Thank you for considering my request!"

/assign


“Unable to extract title” error

The command and output are as follow, using the latest version of youtube-dl:
$ youtube-dl http://www.pornhub.com/view_video.php\?viewkey\=XXXXXXXX --verbose
Output:
[debug] System config: []
[debug] User config: [u'-f', u'bestvideo+bestaudio/best']
[debug] Command-line args: [u'http://www.pornhub.com/view_video.php?viewkey=XXXXXXXX', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.08.09
[debug] Python version 2.7.6 - Darwin-14.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.7.2, ffprobe 2.7.2
[debug] Proxy map: {}
[PornHub]<PHONE_NUMBER>: Downloading webpage
ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 655, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 286, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pornhub.py", line 67, in _real_extract
    video_title = self._html_search_regex(r'<h1 [^>]+>([^<]+)', webpage, 'title')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 577, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 568, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Post the full output of youtube-dl -v --dump-pages ... with concrete URL.

@dstftw It is very strange, according to your parameters it works! Can you tell me why?

Do you state it doesn't work with youtube-dl -f bestvideo+bestaudio/best URL? If so post the output with concrete URL.

Nvm, you didn't remove the video id. youtube-dl -f bestvideo+bestaudio/best http://www.pornhub.com/view_video.php?viewkey=1531821647 works just fine for me.

I used a default setting in ~/.config/youtube-dl/config which downloaded best quality automatically.
But I didn't get what you mentioned about that video ID. For other websites I copied the address everything works fine, even few days ago PornHub is operational as well. I was thinking some protocol changed on PornHub.

Most likely a temporary issue. Closing.


build(django): install and setup Django

PULL REQUEST
Description
Please describe the reason related to the changes:
Breaking Change
Does your changes introduce a breaking change?

[ ] no
[ ] yes

Information (issue, links, logs, etc.)
Please list any other useful information:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
36

Relevant Lines:
45

💛  - Coveralls


Maven Repository is out of date

Cannot use the latest version because it is not present in the Nexus repository.
I manually browsed the Nexus repository and only up to version 1.9.1 is present.
I can fork the repository and compile install it manually, but that isn't ideal especially since there is a Nexus repository available.
Looking forward to incorporating JSON based storage for GlobalWarming, but would like to see the latest artifacts uploaded. Thanks!

You're right, the repo is outdated.
This is because LightningStorage has migrated to JitPack.
The description of the GitHub page is wrong.
Thanks for the hint!
Here are the correct dependencies again:
Can also be found here: https://jitpack.io/#JavaFactoryDev/LightningStorage/2.3.8

       <repository>
                <id>jitpack.io</id>
               <url>https://jitpack.io</url>
	</repository>

       <dependency>
	    <groupId>com.github.JavaFactoryDev</groupId>
	    <artifactId>LightningStorage</artifactId>
	    <version>2.3.8</version>
	</dependency>


Run tests on CI

They currently seem to be run manually. I haven't had a chance to take a stab at this, but do you have a preferred public CI toolchain that you'd like me to use?

I don't have a preference.  I know that Jenkins is popular with open source packages but I'm open to anything.

Well something like travis or circleci is what I meant.

Either travis or circleci is great.  I have a slight preference for travis.

Any idea why it might be hanging here/what I can do to fix the hangs? https://travis-ci.org/josegonzalez/blackbox/builds/213205514

Sure. Will the passphrase be the same each time?

The password will either be "a" or "b" (yes, just 1 letter, lower case). There will be text that says what it will be shortly before the prompt.
If I remember the "expect" language, you could set a variable to "a" any time you see the text "lowercase letter a" and "b" any time you see the text "lowercase letter b".  Then any time you see the prompt, send what is in that letter.
Or, just remember that the first 3 prompts are "a", the next three are "b". (I think it is 3 and 3.  Not 100% sure).
Tom

FYI: "make test" is no longer interactive (i.e. it no longer pauses to ask for passwords, etc.)
It should be pretty easy to set up CI now.
I recommend it do:
make test
make packages-rpm
make packages-deb

If you are still willing to volunteer to set that up, I'd be forever grateful.

Closer, but the gpg-agent definitely asks for a password: https://travis-ci.org/josegonzalez/blackbox/builds/237347010
I'm not too familiar with expect, any chance you have a moment to dig at that?

Is gpg-agent in the PATH?   It may be in a separate package depending on the distro.
Line 273-274:
tools/confidence_test.sh: line 27: gpg-agent: command not found
tools/confidence_test.sh: line 33: gpg-agent: command not found

Line 429:
gpg: gpg-agent is not available in this session

I added gnupg-agent as an installable package, but that didn't seem to help :|

It looks like that did help.  The new error is due to pinentry-tty and pinentry not being found.
Hmm... maybe that script should first check to make sure all the tools it needs are found.
Tom

I just committed 7c4fa4f4bed91a1324b95001fb7cb9c646fc1291 which does an explicit test to make sure that pinentry, gpg-agent, and other programs are in the $PATH.  This should help with that situation, plus prevent future problems. Also, people running the tests in their own environment will find fewer surprises.


typeshed that built in Language Server is outdated.

Environment data

VS Code version: 1.44.1
Extension version (available under the Extensions sidebar): 2020.3.71659
OS and version: MacOS 10.15.4
Python version (& distribution if applicable, e.g. Anaconda): XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Relevant/affected Python packages and their versions: XXX
Relevant/affected Python-related VS Code extensions and their versions: XXX
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info microsoft/vscode-python#3977): XXX
Value of the python.languageServer setting: XXX

Expected behaviour
XXX
Actual behaviour
XXX
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
XXX

https://github.com/microsoft/python-language-server/issues/1371


Define roles for launching openulmus

I 'think' we want to figure out the roles/resources needed to build out the MVP and also maintain and sustain ongoing. Should we continue to work this out in the MVP strategy doc?

Maybe commit that doc you have on google docs to github so we can reference it better / increase transparency that there's something going on :)

Maybe commit that doc you have on google docs to github so we can reference it better / increase transparency that there's something going on :)

Note you don't see my name as a team lead. I wanna focus on onboarding, outreach, community, fundraising, etc. That will be done mainly on behalf of Open Producer, Drupal Dojo, etc.
In terms of the project and process, the ideal would be for ELMS leads to run the show and use the entire development experience as an opportunity for people to contribute and learn.
So ya - whatever the most transparent format for scoping in the open... let's do it!

@btopro Per our discussion regarding Open Ulmus as a Drupal Dojo project. I think we'll be able to direct resources to launch and maintain; also provide opportunities for some of the same people/partners work on multiple projects.
What would be a better way to list out these shared needs than our current system of scattered google docs?

Hi @gusaus -- @btopro sent me here to help out with this task. From what I understand -- we are looking for a better way to organize role information for the community? I'd be happy to help out with this. I like the idea of centralizing some of that google doc here in the repo since the community gathers mostly on here, twitter, and slack.

Per my long winded comments in the doc, I think once we clearly define our needs we can point potential contributors to this issue. Especially if @btopro can reference the doc in this issue's description https://docs.google.com/document/d/11g4tAxHIAu_LuTcDPx-9bFcBqBIxKt85KuKJeT1LvDM/edit?usp=sharing

@gusaus -- This sounds good to me. I'm currently going through the document to try and establish some people for these roles and see if there are any additional positions that should be added. We may be able to get some younger (in college or just graduated) devs involved with the project soon -- so I'm going to keep this in mind for the apprentice roles.
I'm happy to help mentor people looking to get involved with ELMS:LN as well -- so if you have any people looking for more involvement...send them my way!

There are several new people coming into #drupalopenlearning and #drupal-dojo in http://drupalslack.herokuapp.com/
As ELMS / Open Ulmus is going to house the curriculum and training materials from those programs, those rooms should also be a great source for contributors at all levels.
See everybody there?!?

https://github.com/elmsln/openulmus.org/issues/3 is resolved, but it's looking like there are some gaps with regards to managing and sustaining https://docs.google.com/document/d/11g4tAxHIAu_LuTcDPx-9bFcBqBIxKt85KuKJeT1LvDM/edit?usp=sharing
In addition to being able to fire up Drupal Dojo's Twitch.tv at any time, we can begin to do a proper job creating new content https://github.com/DrupalOpenLearning/administration/issues/8 as a way to showcase and direct resources to ELMS and Open Ulmus.
We need to know what, who, how much $$ is needed to sustain.

project manager is going to be the biggy. Envisioning what it should provide to the rest of open collective, where there's overlap with dojo, what courses / trainings to pursue. From there, identifying technologies and personnel needed to properly author those courses/trainings.
This needs a PM who has technical and training space knowledge, having worked with Drupal / other technical communities would be a plus. I'd imagine this could be a part time gig, maybe 5-10 hours a week to get up and going. If needs are identified not provided by the platform or other collectives then getting those functionalities pegged for being worked on in the elmsln/drupal community.

I dono how much you pay a PM to get things up and going but you've gotta have a paid coordinator with a goal of getting at least 5 different communities to have trainings delivered via the system. From there you can figure out your next steps. This is currently on free tier though I'm paying personally since I already use free tier for other things.

Per our follow-up in #drupalopenlearning (anyone not there already should visit http://drupalslack.herokuapp.com to join), the PM you're describing here is essntially the same as the ID role I was blindly asking about here :grinning: https://github.com/DrupalOpenLearning/administration/issues/8
What is poorly described could be the first of many trainings for Drupal Open Learning/Drupal Dojo.
I think we're just beginning to see the potential opportunity for providing services to other communities in OpenCollective -
https://webpack.academy/
https://reactiveui.academy/
Focusing back on this issue regarding maintaining/sustaining Open Ulmus...

Is there a need for PM/ID to manage Open Ulmus or does payment come from projects and trainings that run on top.
Do we have people with interest/availability to assume this type of role? @kat-wehr?
Would ELMS collective be the service provider?

Apologies if I'm asking confusing questions - most of this is uncharted OSS territory.

I'm only confused in that your asking me lol.

payment question is a good question... maybe people can donate for courses / training w/ a % of that going to the collective and a % going to the elmsln collective to cover costs of the service existing? Payment workflow / gateway would need built. We're more the once you get through that stuff sorta scoping thing

Webpack provides a good example of how it could work https://medium.freecodecamp.org/introducing-webpack-academy-bad0a4e23deb

If you do decide to pay for these courses, 30% of any funds generated from this platform will go directly to our webpack open collective.

We should probably have a separate issue or follow-up discussion on the topic.
With regards to roles needed to manage and maintain the site - is @mmilutinovic1313 working to identify? With the Drupal Dojo site now live, there may be similar needs for those potentially interested in working on multiple projects https://github.com/DrupalOpenLearning/drupaldojo/issues/5

I have begun building out a "Getting Started with ELMS:LN course for OpenUlmus. This can serve as a starting point for other courses that we build for other projects.

There's some conversation in https://github.com/elmsln/openulmus.org/issues/1#issuecomment-331521363 pertaining to shared roles on multiple projects.
As an instance of Open Ulumus, the roles needed to manage and maintain Drupal Open Learning could potentially be identical https://github.com/DrupalOpenLearning/administration/issues/1#issuecomment-318171817
Many of the long terms roles for Drupal Dojo https://github.com/DrupalOpenLearning/drupaldojo/issues/5#issue-248103504 could also be shared.
@mmilutinovic1313 If you're still the goto person for sorting out these roles, would it be helpful to recreate https://docs.google.com/document/d/1LrShmkSJ4ewaUNSDtNX0UD1P-W7d0fvsw1blrdmkRn8/edit?usp=sharing as a wiki similar to what I did here? https://github.com/DrupalOpenLearning/drupaldojo/wiki/Team

@mmilutinovic1313 We definitely should follow up with @danielrose28 and @djfusco regarding current interest/availability regarding roles discussed in https://github.com/elmsln/openulmus.org/issues/3

@mmilutinovic1313 per our slack discussion, I'm wondering if some of the traditional technical project management responsibilities pertaining to the websites could be assigned out to somebody else.
i.e. what if we found a dedicated PM to handle open ulmus, drupal open learning, & drupal dojo sites... would that make your responsibilities as described by @btopro more doable? https://github.com/elmsln/openulmus.org/issues/2#issuecomment-327961621

What would you envision the other PM's roles being as compared to mine?  Would they be handling creating and managing tasks? Or building out the sites? Or something else? I'm assuming the roles you are referring to for me would be identifying areas that we need to address to improve the platform then?

This may not be a bad idea as well as I only have weekends and evenings to dedicate to the effort. If someone was more full time for some of the logistical parts of this - I can focus on cordinating between groups and helping build out functionalities we need to sustain the collective etc.

I do think we should have a working session to define roles via skype or google plus.

@mmilutinovic1313 Again, I 'think' your role would be following what @btopro is describing https://github.com/elmsln/openulmus.org/issues/2#issuecomment-327961811

I dono how much you pay a PM to get things up and going but you've gotta have a paid coordinator with a goal of getting at least 5 different communities to have trainings delivered via the system.

That seems to be more focused on coordinating classes on Open Ulmus, Drupal Open Learning, and other ELMS powered platforms. If we're able to identify 5 or so communities with budget, that probably would fill up your available time.
I think that would justify the need for a separate PM mentor/lead to manage open ulmus, drupal open learning, drupal dojo and potentially other partner sites. Especially if that person is responsible for scoping out the cost for developing and maintaining each site. IMHO that still is our biggest need.

@mmilutinovic1313 @btopro Assuming the roles needed to build/manage Drupal Open Learning (an instance of Open Ulmus) would be identical...possibly we can use the Drupal Dojo and Drupal Open Learning projects as a way to generate additional visibility and resources by locking down the roles here? https://github.com/DrupalOpenLearning/drupalopenlearning/issues/2

yup

@gusaus I'm going to copy the roles section of the drupal open learning wiki here too if you are saying that the roles overlap

@mmilutinovic1313 Regarding https://github.com/elmsln/openulmus.org/issues/2#issuecomment-336341779 I think the mentors/advisors/leads will be identical. Each site would provide an opportunity to train up a variety of apprentices.
Again, I'm going on the assumption that the Drupal Open Learning site would be an instance of Open Ulmus. So essentially that would be one of the ongoing, revenue generating, projects managed by the Open Ulmus team.
So ya - copy over the wiki!

@gusaus g2g

@gusaus -- Updated. Copying over the wiki now!

@mmilutinovic1313 In addition to adding or revising roles (like your specific PM role https://github.com/elmsln/openulmus.org/issues/2#issuecomment-333592503) previously discussed... possibly we should list out all typical roles/requirements/incentives for producing a class. I believe this is something @btopro described before such Slack transcripts disappeared.
Instructional Designer? Developer? Writer/docs? Someone to produce videos? I have no idea...
Is there with all the job descriptions?

Instructional designer (someone to ensure there are meaningful objectives / the training plan builds knowledge in a logical way)
Content Expert (whatever community is producing a course at that time)
Developer (add functionality to the system)
PM (make it all happen)
Doesn't necessarily have to be 4 people here but those are the 4 hats.

@mmilutinovic1313 Can you add those roles and descriptions to wiki? Would also be good to see who else is interested/available to take on some of those roles.

@mmilutinovic1313  Possibly we should have a separate wiki breaking down the operational roles we're defining here -
https://github.com/DrupalOpenLearning/administration/wiki/Drupal-Open-Learning-program-staff
As Drupal Open Learning is essentially an instance of Open Ulmus we can further define roles on this side. Mase sense?

@gusaus -- so something like a "core operational roles" template for open ulmus. That way instances (such as Drupal Open Learning) can use this template to fill roles according to the recommended practices?

As mentioned in https://github.com/DrupalOpenLearning/administration/issues/10#issuecomment-340582115 we're still trying to define roles with consistency across multiple projects and repos.
With Open Ulmus essentially the umbrella learning platform we should include a master list of roles, responsibilities and definitions in the scope of the project.
The most efficient way to collaborate on refining these roles might actually be to focus our attention back on this google doc
https://docs.google.com/document/d/11g4tAxHIAu_LuTcDPx-9bFcBqBIxKt85KuKJeT1LvDM/edit?usp=sharing

The following bits from @cleverington may also help us define these roles and responsibilities.

As a side-note the Project Owner / Project Sponsor are definitely items we’ll want to get filled in.  Even if it is a single line (for example) with “Open Producers”, it at least gives a control chain to start marking up an Authority Matrix for project sign-offs and approvals moving forward.

In a buzzword reduction: The Site Builders, Designers, and Project Managers will probably have a solid level of autonomy, but (and especially with an endeavor this big), they’ll want guidance for “Big Vision” questions.  Pie vs Cake. Pirates vs. Ninjas. Polymer vs. React. etc.

@btopro probably won’t ever care about what Readability Testing is done against the verbiage within the various Courses Hosted within the Drupal Open Curriculum, but the Program Manager should be able to reach upwards to ????? at Drupal Open Learning, who would touch base with ???? at Open Ulmus about getting a standardized testing method (such as the Flesch-Kincaid Grade Level Testing) used on all Open Learning Environments.

Somewhat related, can we add @cleverington as contributor to this repo?

@gusaus @cleverington - Check out my modifications and comments I made to the role descriptions on the doc. https://docs.google.com/document/d/11g4tAxHIAu_LuTcDPx-9bFcBqBIxKt85KuKJeT1LvDM/edit#

Added @cleverington as a contributor as well

As @mmilutinovic1313 @cleverington @ikit-claw and myself should be able to get the program up and running, I created a shared 'program manager' role and consolidated our multiple hat wearing responsibilities. We can refine this as we go, but I think combining all leads into one role will make it easier to allocate our time across projects https://github.com/elmsln/openulmus.org/issues/5#issuecomment-340910677
https://docs.google.com/document/d/11g4tAxHIAu_LuTcDPx-9bFcBqBIxKt85KuKJeT1LvDM/edit?usp=sharing

Sounds good! I put what was on the doc into the wiki earlier and am cleaning up the formatting now. Should the doc be the same in each of the other projects we are collectively involved in?

Does Budgeted time per month for the role then quadruple? 40 = 160?

So who is the program manager responsible for budgets?!? If we hit the goals between 3 projects https://github.com/elmsln/openulmus.org/issues/5#issuecomment-340910677 that would be 160 hours total. Those hours would then be allocated to the 4 or so program managers. So roughly 40hrs each per month.

Okay that's what I thought - Just wanted to clarify on the wiki / doc. With this info I may be able to close off some issues :)

Also, @gusaus which one of us is the one responsible? I know I have one issue that deals with budgets, but we have some that are unassigned.

Open Ulmus Core
Individual Project Team (Drupal Dojo, Drupal Open Learning, etc.)

Closing this task and changing https://docs.google.com/document/d/11g4tAxHIAu_LuTcDPx-9bFcBqBIxKt85KuKJeT1LvDM/edit?usp=sharing to 'Suggesting' format.
Further changes should be completed via Suggestions and reviewed by Program Team.
https://github.com/elmsln/openulmus.org/wiki/2.-Open-Ulmus-Core-Team updated as of 11/5/2017 version of document.

Not sure we should consider this issue resolved until we're sure have enough staff/team members/person hours/whatever to manage the platform. Especially if there's going to be a customer/client facing service offering in the mix.
I think we should start by getting a realistic sense of roles and availability from @kat-wehr @michael-collins @btopro @djfusco and other currently involved with the projects. Aware that some of this work (like on the ELMS:LN platform side) is potentially in-scope with day job responsibilities, we probably need to get to a point where there's enough budget to cover 40hrs/wk of total person time dedicated to managing the platform.
Based on that I've reopened this issue and made some tweaks to the google doc above https://github.com/elmsln/openulmus.org/issues/2#issue-242679456

I'm happy to lend support in the form of platform development. I have a community to lead. While Open Ulmus was a dream of mine, I think I can best serve that dream by building out the next gen content authoring and presentation things we're currently building. They'll make Open Ulmus / anything produced here worth using because of how superior their UX is. I'm always here to further ideas and occasionally do some heavy lifting but I view myself more as support for @mmilutinovic1313 getting things going then me directly doing the building. I think I can reasonably put in 1-5 hours a week to keep conversations moving and big lifts as needed.

After talking to @btopro today -- I'd be happy to take the lead in the direction that this platform should head toward. I have a clearer idea as to the short and long-term goals of Open Ulmus and would like to work with @cleverington, @kat-wehr, and @gusaus to get this platform off the ground as soon as we can.

I can go 5-10 hrs/week as needed based on project scope month to month. I have an 8-5 job that in reality does not involve any ELMS stuff, so I need to make sure I'm doing my best to keep day job hours and consulting hours separate in order to stay out of trouble. My 8-5 job runs me ragged at certain times of the year (like right now - gearing up for new semester) but then at other times (spring/summer months) I have a lot of mental bandwidth post 5 pm to bang out additional work.
The work of an ID and efficiency is based a lot on the client as well and how complete of a picture they have going into a project, and also how organized/timely/communicative they are. If the ID has to hold their hand and work through their ideas with them from ground zero through completion then it's a lot more time consuming than working with a client who knows what they want right away, can provide a detailed plan and let the ID do the work and just provide timely feedback.

My availability is roughly along the same lines as @kat-wehs.
As for the existing Roles, if the current version of the Wiki feels insufficient, we need to separate the Role-requirements into Phases and create a separate issue.
I closed out the task because continually re-hashing Roles at this stage is Scope Creep. Let's schedule a revisit of this topic early January and focus our efforts on MVP items. With the knowledge that we are on 'paid' time (once sponsorship arrives), we need to focus our hours unto tasks within the current projects.
We have confirmed contributions and efforts of @kat-wehr, @mmilutinovic1313, @gusaus, and @cleverington for the Core Team.
Also, I back-tracked some of @gusaus changes:

The formatting and headers were intentional as preparatory formatting to match standard business documentation. One of those checkbox items people look for.  It is not the individual which is important, it is the Role.
Header-order is a Section 508 requirement and WCAG 2.0 requirement. H3 should only follow H2, H4 should only follow H3, and so-on.

Personally I think today we made some huge steps forward 'cuz team members like @btopro and @kat-wehr provided clarity in terms ideal scope of responsibilities (sorry if I'm just making up clever terms) and realistic short term and long term commitment towards managing, maintaining, and sustaining Open Ulmus. That in turn should provide us with a better sense of who/what is needed to scope out and launch the Open Ulmus MVP  https://github.com/elmsln/openulmus.org/issues/1
Trying to avoid too much of a groundhog day situation, I'd just refer back to my comment from yesterday regarding my reasoning for reopening this task https://github.com/elmsln/openulmus.org/issues/2#issuecomment-346133430
With that, I'd recommend we consider the google doc or wiki more of an internal working doc as we scope out the product.

Like the others, I also have a day job and also got a new assignment in my job at Penn State.  My time, like @kat-wehr goes in highs and lows, depending on the time of year.  I could give 5-10 hours per week.  I'm most likely able to help with server-side stuff, including cloud hosting, as well as some data exchange scripting and xAPI things.  I'm better when you let me know 'we need X by date Y' and some direction on where you want to go.

With clarity from the entire team, I'd like to suggest the project manager (@mmilutinovic1313, right? :raised_hands:) head up the following -

[ ] revise the Open Ulmus roles google doc as we match the actual team requirements (and compensation) to whatever is figured out in the project scope #1
[ ] update the Open Ulmus roles wiki based on whatever we figure out in the previous step
[ ] make sure all team members interested in DOYOT (do on your own time) projects (this is one of them) reference/comment on the following https://github.com/openlearninglabs/administration/issues/2 https://github.com/openlearninglabs/administration/issues/5#issuecomment-346238440 and craft their ideal role as a member of Open Learning Labs.
https://docs.google.com/document/d/1BEPnpPC8ovTnVy9vEAiw3clHWMH2Ufn_fyPgAg3lmJc/edit?usp=sharing
[ ] 🤘

@gusaus -- what is a DOYOT project?

We touched on this during our last meeting #55 , but only a few team members have reviewed and provided input in the discussion thread. As it's important to gauge current interest and availability from everybody who has done so previously in this issue, I'd suggest the following 'before' the next meeting.

[ ] Review this issue regarding Open Ulmus service scope #58  (including issues that are referenced)
[ ] Review all referenced issues and run through the suggested action items ☝ https://github.com/openulmus/openulmus.org/issues/2#issuecomment-346530920
[ ] Leave any followup comments, thoughts, questions in this issue, the discussion thread from last week, or just leave a 👍 here if you've read.

If we do that, we should have a better idea for what we need to review and decide during our next meeting.

Ok - based on our decision to go for 501c3 status #57 and alter our mission and scope #58  to ensure we're approved, I think we do need to make some adjustments to how we’re defining and trying to budget for the Open Ulmus Core Team
First it would be good to gauge current interest and availability from everybody who has done so previously in this issue https://github.com/openulmus/openulmus.org/issues/2#issuecomment-366797773
Based on the 5-10 hrs/wk range of total availability that everybody noted previously, I think we should consolidate all roles needed to manage Open Ulmus under ‘Program Team’  https://github.com/openulmus/openulmus.org/wiki/2.-Open-Ulmus-Core-Team#program-team
Platform Team is sort of a mishmash of roles https://github.com/openulmus/openulmus.org/wiki/2.-Open-Ulmus-Core-Team#platform-team
Aside from time needed for platform updates (which we could estimate as part of the hosting cost), the other roles are more about developing the platforms #1 or creating content. Funds for that type of work should actually flow through ELMS:LN, Open Learning Labs https://github.com/openlearninglabs/administration/wiki/Budget-and-Sustainability#sustainability, or some other partner collective.
Does that make sense?


_private_notes are lost on author update (form)

Not only a few fields are affected. It seems to be a structural issue with the author update, that it completely discards all information not present in the form (@hoc3426 reports wikipedia links also disappear). The update mechanism should probably be revisited to update the existing record rather than overwriting everything.

Not only a few fields are affected. It seems to be a structural issue with the author update, that it completely discards all information not present in the form (@hoc3426 reports wikipedia links also disappear). The update mechanism should probably be revisited to update the existing record rather than overwriting everything.


missing hvalsBuffer and hgetallBuffer

The redis.hvalsBuffer() and redis.getallBuffer() methods, while existent are marked as nonexistent when using tyescript.
My setup:
ioredis: 4.17.3
typescript 3.9.5

I think the issue belongs to https://github.com/DefinitelyTyped/DefinitelyTyped so can you create a ticket there?

ioredis v5 provides official TypeScript declarations so closing this. Sorry for the late response!


Manager failing to load mods into launcher

If you watch this video you can see the issue pretty easy. Video isn't best quality but still visible. - https://youtu.be/5cvlfb4GBpY
For some reason Stellaris just began to crash out of the blue. The UI 1080 mod or w/e it's called was causing some serious issue, so I deleted everything Stellaris related. And I mean everything. All files in documents, Appdata, Steam, were deleted. And launcher files.
Did a fresh install for game and downloaded mods. Mods have not changed. I've been working on it for days. I hope this is an easy fix, but my gut is telling me that isn't a possibly.

https://youtu.be/5cvlfb4GBpY - here is the video

I'm thinking maybe the manager isn't generating the load.json or w/e?

Hi,
Since the Paradox launcher brings playsets, the launcher always override the last mod used list.
The only workaround for now is to start the game directly with the game executable after any list application.

Yo,
Could you pls explain what you mean by list application.
Thanks

I think I have already done what you are suggesting, but it resulting in a crash, or sometimes I could get in game but they're would be pink errors on screen.


DEVAI-31: Custom Namespace Tekton Task

What does this PR do?:

Adds a Tekton Task to the RHDH namespace during run of configure-pipelines.sh so that users can target different namespaces for their apps and the Tekton Task will setup up that target namespace with the resources it needs to manage the deployments.
Which issue(s) this PR fixes:

resolves missing part of https://issues.redhat.com/browse/DEVAI-31
PR acceptance criteria:
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

[ ] Tested and Verified

[ ] Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

How to test changes / Special notes to the reviewer:

Cool feature!


Change managing of output FASTQ format

By default, the FASTQ format is gzipped and -dgz option allows to disable. This option should disappear in favor of one enum which allows to use a different kind of compression with FASTQ format (bgzip/gzip/none). This is interesting for store files in HDFS, and control it better.
For beta.

Default should be gzip.

ReadsToFastq use an enum for the output (PLAIN or GZIP). Other formats could be implemented easily, but requires some work into the writer that is not going to be done for beta.
Moving to version 1.

Closed in favor of #82.


Support for other currencies

Would be great to have support for international websites and other currencies.
I'm based in the uk, and access Farnell through uk.farnell.com for instance.
Would it be possible to implement this? How could I help?

Linked to https://github.com/xesscorp/KiCost/issues/65

Linked to #65

@romain145, discussion re-open in #95.


[Popper] Change default strategy to use position: fixed

Fixes #370
Fixes #781
Fixes #1464
We originally stayed away of position: fixed because it isn’t the most efficient way when scrolling (as it doesn’t take advantage of not having to update the transform and let the scroll do the job) but it does fix all of the positioning caveats we currently have.
The only caveat from changing this is performance when scrolling (slightly lagging behind feel of the transform udpating).
That said:

This is a non-issue when the component is modal as scrolling is disabled
Tooltip closes on scroll, we have also discussed about matching this for all other non-modal popper-backed components, which if we do would also negate the issue.

This has sadly uncovered some other holes with regards to new stacking contexts created on fixed element when a transform is involved, so we likely won't be able to go ahead with this.
We will do a spike on using floating-ui as a replacement.

Hey @benoitgrelard  @andy-hook,
You are doing a really great job with providing all of those components!  Just wondering, Is there any chance you could provide some temporary fix until floating-ui spike will be ready?
Thank you for all your work!
 Just wondering, Is there any chance you could provide some temporary fix until floating-ui spike will be ready?

Glad you're finding the work useful! if you have a working fix then I suggest sticking with it for now. We're unlikely to merge a halfway solution in the meantime.
Feel free to post your problem and current solution in one of the linked issues though and we can follow up there.


fix: Add beacon roots touched slots into state_access with native tracer

Tested against blocks 20175000 through 201750010.
closes #350

Note: the Jerigon Integration failure is unrelated to this PR, as we don't have yet an image for cancun. This has been raised in https://github.com/0xPolygonZero/jerigon-test-network/issues/1.


Update gRPC library

This changed since I think this was last updated 😄
Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Thanks!


Unclear error while rollback

Description
tried to deploy gw with tcp as backend ip:22
and I got the error  ContractNotExists
the expected error
due to 255 failed to validate backend '<IP_ADDRESS>:22': failed to parse backend with error: parse "<IP_ADDRESS>:22": first path segment in URL cannot contain colon

i got the correct error after adding some delay before applyAllExtrinsics in grid_client/src/high_level/twinDeploymentHandler.ts
Steps to reproduce
deploy a gw with  <IP_ADDRESS>:22 as backend
Full Logs/Alerts
Omar Kassem, [6/7/23 5:55 PM]
deploying gateway monNetwork on node 11'
Validation succeed
Start creating the gateway deployment with name wireguardssh
Merging workloads
Validation succeed
Deploying on node_id: 11
Check the name contract for the workload with name: wireguardssh
Lock acquired
2023-06-07 16:51:00        API/INIT: Not decorating runtime apis without matching versions: TransactionPaymentApi/2 (1 known)
phase: {"applyExtrinsic":1}, section: balances, method: Withdraw, data: ["5HHRg2nVfTXD9StGihrjHre5LhC2xVRZhBYvVqoorvsqo5F5",10225]
phase: {"applyExtrinsic":1}, section: smartContractModule, method: ContractCreated, data: [{"version":4,"state":{"created":null},"contractId":30483,"twinId":800,"contractType":{"nodeContract":{"nodeId":11,"deploymentHash":"0x3961653736636539666564396438633632396531616134373234373037396138","deploymentData":"0x7b2274797065223a2267617465776179222c226e616d65223a22776972656775617264737368222c2270726f6a6563744e616d65223a22227d","publicIps":0,"publicIpsList":[]}},"solutionProviderId":null}]
phase: {"applyExtrinsic":1}, section: utility, method: ItemCompleted, data: []
phase: {"applyExtrinsic":1}, section: smartContractModule, method: ContractCreated, data: [{"version":4,"state":{"created":null},"contractId":30484,"twinId":800,"contractType":{"nameContract":{"name":"0x776972656775617264737368"}},"solutionProviderId":null}]
phase: {"applyExtrinsic":1}, section: utility, method: ItemCompleted, data: []
phase: {"applyExtrinsic":1}, section: utility, method: BatchCompleted, data: []
phase: {"applyExtrinsic":1}, section: balances, method: Deposit, data: ["5EvbAkU2fMiWXGrAmnUTi4zdVNXTa9cpamnhakuQcSMYSnpT",10225]
phase: {"applyExtrinsic":1}, section: transactionPayment, method: TransactionFeePaid, data: ["5HHRg2nVfTXD9StGihrjHre5LhC2xVRZhBYvVqoorvsqo5F5",10225,0]
phase: {"applyExtrinsic":1}, section: system, method: ExtrinsicSuccess, data: [{"weight":{"refTime":2157129688,"proofSize":0},"class":"Normal","paysFee":"Yes"}]
Lock released
Sending deployment to node_id: 11
Rolling back deployments
Deleting name contract with name: wireguardssh and id: 30484
Deleting contract id 30483
Deleting contract id 30484
Lock acquired
2023-06-07 16:51:06        API/INIT: Not decorating runtime apis without matching versions: TransactionPaymentApi/2 (1 known)
phase: {"applyExtrinsic":1}, section: balances, method: Withdraw, data: ["5HHRg2nVfTXD9StGihrjHre5LhC2xVRZhBYvVqoorvsqo5F5",10139]
phase: {"applyExtrinsic":1}, section: balances, method: Deposit, data: ["5GgRLHTmSKBUhqE34DHVgBWZrRbG4xRWLxiCxTPeq37jDHax",10139]
phase: {"applyExtrinsic":1}, section: transactionPayment, method: TransactionFeePaid, data: ["5HHRg2nVfTXD9StGihrjHre5LhC2xVRZhBYvVqoorvsqo5F5",10139,0]
phase: {"applyExtrinsic":1}, section: system, method: ExtrinsicFailed, data: [{"module":{"index":12,"error":"0x07000000"}},{"weight":{"refTime":434338204,"proofSize":0},"class":"Normal","paysFee":"Yes"}]
Lock released
/home/kassem/tfgrid-sdk-ts/packages/tfchain_client/dist/node/client.js:320
            result = await this._applyExtrinsic(extrinsic, resultSections, resultEvents);
                     ^
Error: Failed to apply {"args":{"calls":[{"args":{"contract_id":"30,484"},"method":"cancelContract","section":"smartContractModule"},{"args":{"contract_id":"30,483"},"method":"cancelContract","section":"smartContractModule"},{"args":{"contract_id":"30,484"},"method":"cancelContract","section":"smartContractModule"}]},"method":"batchAll","section":"utility"} due to error: ContractNotExists
    at /home/kassem/tfgrid-sdk-ts/packages/tfchain_client/dist/node/client.js:312:19
    at async TFClient.applyExtrinsic (/home/kassem/tfgrid-sdk-ts/packages/tfchain_client/dist/node/client.js:320:22)
    at async TwinDeploymentHandler.rollback (/home/kassem/tfgrid-sdk-ts/packages/grid_client/src/high_level/twinDeploymentHandler.ts:373:5)
    at async TwinDeploymentHandler.handle (/home/kassem/tfgrid-sdk-ts/packages/grid_client/src/high_level/twinDeploymentHandler.ts:523:7)
    at async GWModule.deploy_name (/home/kassem/tfgrid-sdk-ts/packages/grid_client/src/modules/gateway.ts:82:23)

Omar Kassem, [6/7/23 5:55 PM]

    at async GWModule.descriptor.value (/home/kassem/tfgrid-sdk-ts/packages/grid_client/src/modules/utils.ts:14:12)
    at async GWModule.descriptor.value (/home/kassem/tfgrid-sdk-ts/packages/grid_client/src/helpers/validator.ts:23:12)
    at async main (/home/kassem/tfgrid-sdk-ts/packages/grid_client/scripts/wireguard_vm.ts:97:27)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
➜  grid_client git:(development_add_wireguardExample) ✗

I got the correct error after adding some delay before applyAllExtrinsics in grid_client/src/high_level/twinDeploymentHandler.ts

I tried to add the delay but keep getting the same error, can't reach any valid solution yet
the issue surely in
https://github.com/threefoldtech/tfgrid-sdk-ts/blob/0ab73db5448ca61f484d836584b94f904f0fb050/packages/grid_client/src/high_level/twinDeploymentHandler.ts#L373

Verified on devnet.
I created a name gateway with the same backend that was mentioned above <IP_ADDRESS>:22. and it failed with the expected error.
due to 255 failed to validate backend '<IP_ADDRESS>:22': failed to parse backend with error: parse "<IP_ADDRESS>:22": first path segment in URL cannot contain colon
the rollback was also completed successfully and the contract was deleted.


TensorFlow 2 and current status

Hey guys, I have more of a question than an issue: Does this FRCNN implementation works with TensorFlow 2 as well? Also, is this repo being actively maintained as of today (2020-04-03)?

@haimat
The development is quite idle right now, since I'm mostly developing in pytorch.
Did it produce any errors in TensorFlow2? Would be very helpful if you can post any pitfalls you guys have experienced in the new version.

Not yet I wanted to ask first.
Will drop you a line when I have tested it!

@haimat did u faced any errors in TF2?

Frankly I haven't tested in yet - I have originally asked here because I didn't want to invest time only to find out that TF2 is not supported, so I thought I will ask you guys first ...


SDK/CLI: Disabled AWS accounts shouldn't be onboarded

Update the zanshin organization scan_target onboard_aws_organization command to exclude AWS accounts that are suspended (in AWS Organizations for the customer's account). The AWS CLI for this would be aws organizations list-accounts --query 'Accounts[?Status=='SUSPENDED'][Id]

Fixed here: https://github.com/tenchi-security/zanshin-cli/pull/49


UX improvements for Emojii errors

Uploading a reportback with an emojii in the caption seems to regularly fail. Would be nice to detect this before submitting to the API and display a better error message, if the API doesn't support emojii (we should confirm this before writing any code)

Confirmed that we receive a 500 error when we submit Emojis within the reportback caption:
{"error":{"code":500,"message":"Could not create\/update reportback."}}

@tongxiang Adding checks to the fields in User Register form sounds good.

Not an issue for this codebase, but the 500 error upon submitting an emoji feels more like it should be a 400 (in which case we'd be displaying the "Could not create/update reportback" message). Ideally if we did any Northstar work on this, that message could be tweaked with a possible "Invalid characters" message.
cc @jonuy

Closed by #681.

Possibility of allowing :cat2: :tiger: 's for Reportbacks in https://github.com/DoSomething/phoenix/issues/6167


feat(postgres): add root password parameter

Apparently a compliant root_password value is required, if there's a password policy set, due to the fact that:
NOTE on google_sql_database_instance:
- Second-generation instances include a default 'root'@'%' user with no password. This user will be deleted by Terraform on instance creation. You should use google_sql_user to define a custom user with a restricted host and strong password.

Otherwise there's the following error:
 Error: Error, failed to create instance postgre-dev-01: googleapi: Error 400: Invalid request: Root user's password is not policy compliant: Length of password cannot be less than 8. ., invalid
│
│   with module.postgre.google_sql_database_instance.default,
│   on .terraform/modules/postgre/modules/postgresql/main.tf line 50, in resource "google_sql_database_instance" "default":
│   50: resource "google_sql_database_instance" "default" {

Note source: https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/sql_database_instance
Example from google: https://cloud.google.com/sql/docs/postgres/samples/cloud-sql-postgres-instance-pvp

@the-veloper thanks for the PR. Lint test failed.
Thanks

/gcbrun

/gcbrun

This will addres #214


ValueError during loading of dataset

❓ Questions and Help
Hi
I tried loading a custom dataset in coco format and followed the descriptions regarding the symlink for the dataset. As a background, I labelled the images on Labelbox, exported the annotation file in JSON WKT format, downloaded the images saved as URL in the annotation file, split the annotation file and the images into train, validation and test data and converted the annotation files to JSON COCO format.
For using maskrcnn_benchmark, I configured my own .yaml file so that it has the correct paths:
...
DATASETS:
  TRAIN: ("coco_nuclei_train", "coco_nuclei_val")
  TEST: ("coco_nuclei_test",)
...

I also configured my own paths_catalog.py file, where I specified the correct paths to the symlinks in maskrcnn-benchmark/datasets/coco_nuclei which looks like this:
...
DATASETS = {
        "coco_nuclei_train": {
            "img_dir": "coco_nuclei/train",
            "ann_file":	"coco_nuclei/annotation/train_coco.json"
        },
        "coco_nuclei_val": {
            "img_dir": "coco_nuclei/train",
            "ann_file":	"coco_nuclei/annotation/val_coco.json"
        },
        "coco_nuclei_test": {
            "img_dir": "coco_nuclei/test",
            "ann_file":	"coco_nuclei/annotation/test_coco.json"
        }
    }
...

Still when I run the training, it reads in the annotation files without problem but apparently cannot load the training/validation data as it raises a ValueError:
loading annotations into memory...
Done (t=0.01s)
creating index...
index created!
loading annotations into memory...
Done (t=0.03s)
creating index...
index created!
[<maskrcnn_benchmark.data.datasets.concat_dataset.ConcatDataset object at 0x2acd4fd43160>]
Traceback (most recent call last):
  File "tools/train_net.py", line 172, in <module>
    main()
  File "tools/train_net.py", line 165, in main
    model = train(cfg, args.local_rank, args.distributed)
  File "tools/train_net.py", line 60, in train
    start_iter=arguments["iteration"],
  File "/home/max/github/maskrcnn-benchmark/maskrcnn_benchmark/data/build.py", line 159, in make_data_loader
    sampler = make_data_sampler(dataset, shuffle, is_distributed)
  File "/home/max/github/maskrcnn-benchmark/maskrcnn_benchmark/data/build.py", line 63, in make_data_sampler
    sampler = torch.utils.data.sampler.RandomSampler(dataset)
  File "/home/max/anaconda3/envs/pt_mask_Rcnn_env/lib/python3.6/site-packages/torch/utils/data/sampler.py", line 64, in __init__
    "value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integeral value, but got num_samples=0

I checked the paths multiple times, but maybe this issue occurred with someone else as well, help will be MUCH appreciated! If I am missing important information please let me know.
Thank you in advance!

I believe that the dataset that you loaded might end up being empty.
Here is what I'd recommend:
add a
from IPython import embed; embed()

in this part of the code
https://github.com/facebookresearch/maskrcnn-benchmark/blob/d3fed42afe59910dd650308c426640d183d044b5/maskrcnn_benchmark/data/build.py#L46
and check that the dataset that is returned there is actually valid. For example, check that len(datasets[0]) is what you would expect, and also that datasets[0][0] returns the image and the annotation in the BoxList format properly.
Also, do this without torch.distributed.launch, i.e., in a single GPU.
My guess is that your dataset is empty for some reason.

Thank you very much for your answer! The dataset is indeed empty, len(datasest[0]) gives 0 with single and multiple GPU training. Will now try to dismantle the code and find out why the images could not be loaded. Could the image size be a reason (my images are 2048x2048)?
I will obviously post updates if I find a solution.

@maxsenh no, the image size wouldn't be a problem.
It's probably because your JSON is not in the format that COCO is expecting

Thank you again for your answer! I found that the issue was because of wrong IDs given by Labelbox. After I changed them, it worked fine.

Cool, glad to know that you found the problem!
Closing this issue, but let me know if you have further questions

Thank you again for your answer! I found that the issue was because of wrong IDs given by Labelbox. After I changed them, it worked fine.
Hey,
Can you please tell me what worng ID has been given by Labelbox ,
I am having a similar problem.

Sure, so what I did is I downloaded the annotation file in JSON WKT format and then used the tool labelbox2coco to convert the JSON file into coco format. Then I changed in 'images' per image the value of the 'file_name' to the correct filename and removed the URL given by Labelbox (I downloaded all the images first) and changed 'id' to an integer corresponding to the filename. Then I changed per image in 'annotations' the 'image_id' to the integer corresponding to the filename.
It is maybe not the best way but it worked! Hope it helps


Is there any way to run this on windows?

Is there any way to run this on windows? I see that this require SpConv (PyTorch Spatially Sparse Convolution Library) which currently does not support windows.

Yes, maybe you should first solve the problem of SpConv. But for some models which do not need spconv (PointRCNN, PointPillar), maybe you could try to build it in Windows, but I never tried it.


feat: update Electron v10 and add v11

Updates ABI information for new Electron v10 stable and v11 beta.
To be merged on August 25th.
cc @MarshallOfSound

We should be good to go as of 2.19.1 (I added auto-updating capabilities in #83 and #92)


Mouse wheel sensitivity for non freeMode

Hello! well, actually I'm not reporting a bug, but just asking about the possibility to have mousewheel sensitivity when the freeMode is set to false. I tried to modify your code but had no success.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Additional surface flux parameters

Adds semi-empirical stability constant \zeta_a and \gamma (exponent required in calculation of stability parameter \zeta). Sources for default values are:
Gryanik et al. (2020) : 10.1175/JAS-D-19-0255.1 (Eq. 64)
Gryanik et al (2021) : 10.1029/2021MS002590 (Table 1)
modified:   src/SurfaceFluxes/universal_functions_parameters.jl
modified:   test/old/surface_fluxes.jl

PULL REQUEST
Purpose and Content
The parameters added in this PR enable the option to apply a non-iterative solution to the MOST equations in the stable boundary layer regime. A parallel PR in SurfaceFluxes.jl will follow that then uses these parameters in the updated obukhov_length function.
Benefits and Risks
Benefits : Having access to these parameters provides an alternative to the iterative solution procedure in stable b.l. in SurfaceFluxes.jl
Risks: Risks are expected to be mitigated via the added tests. There are no deletions in this PR so this is not expected to affect other code components.
Linked Issues
https://github.com/CliMA/SurfaceFluxes.jl/issues/62
PR Checklist

[x] This PR has a corresponding issue OR is linked to an SDI.
[x] I have followed CliMA's codebase contribution and style guidelines OR N/A.
[x] I have followed CliMA's documentation policy.
[x] I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
[x] I linted my code on my local machine prior to submission OR N/A.
[x] Unit tests are included OR N/A.
[x] Code used in an integration test OR N/A.
[x] All tests ran successfully on my local machine OR N/A.
[x] All classes, modules, and function contain docstrings OR N/A.
[x] Documentation has been added/updated OR N/A.

We should probably just remove the other modules, now that no other packages are using it. We can save that for a separate PR, though (since it'll technically be breaking)

We should probably just remove the other modules, now that no other packages are using it. We can save that for a separate PR, though (since it'll technically be breaking)

Noted; wasn't sure what the suggested scope for change was (for now I have updated the parameter .toml file but also updated the old tests). Changes with updated docs have been pushed now.

bors r+

Ah @charleskawczynski could you please merge this and register the new version? It seems I can't r+ to this repo even with you having reviewed / approved it. Thanks!

bors r+


.Net: Microsoft.SemanticKernel.Plugins.Memory.dll is delay-signed or test-signed assembly

Describe the bug
The assembly Microsoft.SemanticKernel.Plugins.Memory.dll fails to load on a Windows Server machine:
Could not load file or assembly 'Microsoft.SemanticKernel.Plugins.Memory' or one of its dependencies.
Strong name signature could not be verified. The assembly may have been tampered with, or it was delay
signed but not fully signed with the correct private key. (Exception from HRESULT: 0x80131045)

Checking the assembly with sn.exe -v yields:
sn -v Microsoft.SemanticKernel.Plugins.Memory.dll

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Microsoft.SemanticKernel.Plugins.Memory.dll is a delay-signed or test-signed assembly

To Reproduce
Steps to reproduce the behavior:

Verify that verification skipping is disabled, i.e. sn.exe -Vl for your architecture (x86/x64) shows

sn.exe -Vl

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

No verification entries registered

Load the assembly into a runtime that verifies strongname signatures.

Expected behavior
The assembly should load without issues.
Platform

OS: Windows
Language: C#
Source: beta1, beta4

The Microsoft assemblies signed only in the release packages.
To get a strong name, you need to follow this steps:

Download the nuget package Microsoft.SemanticKernel.Plugins.Memory 1.0.0-beta6

Rename microsoft.semantickernel.plugins.memory.1.0.0-beta6.nupkg to microsoft.semantickernel.plugins.memory.1.0.0-beta6.zip
Extract it
Get the assembly file Microsoft.SemanticKernel.Plugins.Memory.dll from the lib/netstandard2.0 folder
Check for the strong name like you did.

F:\check-assembly-strongname> sn -v Microsoft.SemanticKernel.Plugins.Memory.dll

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Assembly 'Microsoft.SemanticKernel.Plugins.Memory.dll' is valid

Closing this issue, for now.
Thanks for the feedback.

Hi @RogerBarreto  I've just tried that and I still get:
C:\Users\WernerWopienka\Downloads\microsoft.semantickernel.plugins.memory.1.0.0-beta6\lib\netstandard2.0>sn -v Microsoft.SemanticKernel.Plugins.Memory.dll
Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.
Microsoft.SemanticKernel.Plugins.Memory.dll is a delay-signed or test-signed assembly

@Wopienkaatwork , please what was the Nuget Package you attempted to download from ?
Ensure you got it from here: Microsoft.SemanticKernel.Plugins.Memory 1.0.0-beta6

@RogerBarreto ,  yes I took the link you provided, downloaded the nuget file, renamed it to zip, extracted it and used the vs2022 cmd prompt:
C:\Users\WernerWopienka\Downloads\microsoft.semantickernel.plugins.memory.1.0.0-beta6\lib\netstandard2.0>
sn -v Microsoft.SemanticKernel.Plugins.Memory.dll
Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.
Microsoft.SemanticKernel.Plugins.Memory.dll is a delay-signed or test-signed assembly
I also tried it via the nuget explorer with the same result.

I second this, just downloaded beta6 and tested:
> sn -v Microsoft.SemanticKernel.Plugins.Memory.dll

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Microsoft.SemanticKernel.Plugins.Memory.dll is a delay-signed or test-signed assembly

$ sha256sum Microsoft.SemanticKernel.Plugins.Memory.dll
76a701d0aced588b01eeaaf9ca4a187766911d0d0b79e3102fb9d68c7d3417ee  Microsoft.SemanticKernel.Plugins.Memory.dll

Ok, will look deeper into this.


Change azdata executions to point to kube config and cluster contexts

Description:
This change depends on #13551. Change #13551 adds the ability to capture and store the path to the Kube config file and the corresponding cluster in which an arc DC is located. This PR makes the following changes:

Modifies azdata command-line executions to start setting the necessary environment variables.
Changes the calls to the azdata command-line API to pass in the added information about Kubr config file and the cluster context.
Changes to the notebooks to set the necessary environment variables.

Tests:
Modified existing tests so they work with changes
Validated all extension tests.
In Progress: E2E testing and validation
Work items and Design:
This PR is next in a series of changes needed for #12727
Here is the full scope of planned work. MultipleControllerSupportInAdsAzureArc

Pull Request Test Coverage Report for Build 413437174

13 of 38 (34.21%)  changed or added relevant lines in 10 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 41.542%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/arc/src/ui/components/filePicker.ts
0
1
0.0%

extensions/arc/src/ui/dashboards/miaa/miaaDashboardOverviewPage.ts
0
1
0.0%

extensions/arc/src/ui/dashboards/postgres/postgresOverviewPage.ts
0
1
0.0%

extensions/arc/src/ui/dialogs/connectControllerDialog.ts
0
1
0.0%

extensions/azdata/src/azdata.ts
11
12
91.67%

extensions/arc/src/models/miaaModel.ts
0
2
0.0%

extensions/arc/src/models/postgresModel.ts
0
2
0.0%

extensions/arc/src/providers/arcControllersOptionsSourceProvider.ts
0
4
0.0%

extensions/azdata/src/api.ts
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/arc/src/ui/dashboards/postgres/postgresOverviewPage.ts
1
3.93%

extensions/notebook/src/jupyter/serverInstance.ts
4
73.81%

Totals

Change from base Build 412800721:
-0.02%

Covered Lines:
22098

Relevant Lines:
48817

💛  - Coveralls


[FEATURE] Mobile-friendly / responsive UI

🚀 Feature

Fix how responsive the web pages are so they can be viewed and used effectively on mobile devices.
Motivation

I would love to be able to use LLM Studio while on the go. The web GUI makes it so nice to setup experiments, monitor them, and tweak that I found myself this weekend checking on my phone often. However, it was challenging considering the pages are not designed for mobile.

I'd be up for contributing this one.

Thanks for opening and being open to contribute here @Glavin001.
I think in general it is possible to make this work, needs changes in underlying Wave styling.
But I think it might be far from trivial making everything mobile friendly - up to you to give it a spin.

Thank you for working on this issue @Glavin001
Actually, you can define breakpoints for the layout.
layouts=[
    ui.layout(breakpoint="0px", width="1430px", zones=zones_small),
    ui.layout(breakpoint="768px", width="1430px", zones=zones_medium),
    ui.layout(breakpoint="1024px", width="1430px", zones=zones_large),
],

Having the app more mobile friendly would be huge, so looking forward to what you will come up with.
If there is anything that you see missing feature wise in H2O Wave, please also drop a note over at the Wave repository https://github.com/h2oai/wave


Fullscreen fixes and smart sizing resuscitation

make smart-sizing work again which was killed in previous commits
renamed xfc->width and xfc->height to sessionWidth / sessionHeight since several contributers in the past repeatedly made the error to treat these as equal with the x11 window width and heigth
removed unused variables from xf_detect_monitors()
removed xfc->desktopWidth and xfc->desktopHeight context variables since they were basically unused (from the contexts's point of view) and only served as temporary storage
removed some weird/unnecessary/ugly code and workarounds
miscellaneous small fixes
new feature: restore previous window position when toggling out of  fullscreen mode
new feature: if /f is specified in combination with /smart-sizing:WxH we run the session in the /smart-sizing dimensions scaled to full screen

@nfedera First, thanks for taking the effort to try to bring this altogether. As far as I can tell, the functionality from my PR is still intact. I do not really know the state of smart-sizing before all of your changes was but it seems to not behave as I would expect from the QA document.
Two examples:

xfreerdp /v:x.x.x.x /smart-sizing:640x480 s/ize:1920x1080
I would expect to see a 1920x1080 desktop scaled down to a 640x480 window. The window is 640x480 and the desktop is 1920x1080, but no scaling happens. This looks like an initialization issue cause if I toggle fullscreen then back then it behaves as expected.

xfreerdp /v:x.x.x.x /smartsizing:640x480 /size:1920x1080
I would expect my session to be running at 1920x1080 and scaled to my fullscreen dimensions, instead it looks like my session is running at 640x480... Or maybe my session is being scaled down then displayed fullscreen afterwards.

Let me know if this makes sense or if you are seeing different results. I just did some quick testing with CentOS and metacity.

@bjcollins if you got a moment could briefly describe your test setup and your tests?

@bjcollins Thanks for testing.
Re 1):
Definitely a bug which is fixed in the last commit fd5f474
Re 2):
This is different. In general (indep. of smart-sizing) /f has always nullified the /size parameter.
So your /smartsizing:640x480 /size:1920x1080 /f options are actually the same as  /smartsizing:640x480 /f.
Now /f in combination with /smartsizing:WxH is handled special now (based on a feature request) and I'm quoting the QA page here:

if /f is specified in combination with /smart-sizing:WxH we run the session in the /smart-sizing dimensions scaled to full screen
Example xfreerdp /v:servername /f /smart-sizing:320x200 This should create a 320x200 rdp session scaled to your full screen dimensions

I just confirmed that it is working exactly like this and the result you've described it is the correct behavior.

@bmiklautz I agree that testing documentation is needed for fullscreen, and I will try to get that started. I think we should write up and verify appropriate behavior with lack of Xinerama and/or lack of NET_WM_STATE_FULLSCREEN and NET_WM_FULLSCREEN_MONITORS hints available.I will try to cover the correct functionality with those things available in the next few days. I think there may be a bug in the absence of those things and I would want to look at it again and test to try to determine what I would consider to be desired functionality in those situations.

@nfedera  re 2) sorry, should have read the docs closer :) Very confusing to me to have what seems like dual meanings for that parameter. In the non-fullscreen case /smart-sizing is used for the local window sizing. In the fullscreen case, it is used for the remote desktop sizing.
Totally outside the scope of your PR, but just stating my opinion below based on what little I know about the whole smart-sizing thing...
I think it would be better if...

Without /smart-sizing and without /f then /size would be used for both the remote desktop size and local window size.
Without /smart-sizing but with /f then /size would be totally ignored and the remote desktop size would be calculated based on the local screen size
With /smart-sizing but without /f then /size would only refer to the local window size and /smart-sizing would set the remote desktop size
With /smart-sizing but with /f then /size would be totally ignored AND /smart-sizing would set the remote desktop size

So, my case 1) above would scale a 640x480 desktop up to a 1920x1080 window and my case 2) above would still work the same. Basically, /smart-sizing would always have a dual purpose of enabling smart-sizing as well as setting the remote desktop size that will be scaled from.

@bjcollins I'd propose to create an issue where we can discuss better or more intuitive options that can be changed after this PR is merged - d'accord ?

@nfedera nice work! Will try to get my test-environment updated this weekend.Will report back.

had time to update my test-environment. could only do some quick test for
/f /smart-sizing:640x480
/f /smart-sizing:1920x1080
Display resolution is 1024x768 (thinclient VM). Both is working as expected. Fullscreen RDP scaled to lower res and scaled to higher res.
Thx for your efforts! If you need further testing, please let me know what you would like to have tested and I try to find some free time.

@Hobby-Student cool, Thx for doing the tests!

@nfedera I'm currently testing wfreerdp for some windows clients (high DPI no fullscreen smart-sizing in Microsoft RDP-client).
Smart-sizing in general is working in wfreerdp working in window mode. E.g. "/size:1024x768 /smart-sizing:640x480" starts a "1024x768" window which can be resized and the resolution of "1024x768" is stretched.
Should there be the feature like in xfreerdp to scale up/down to the "smart-sizing" resolution?

@Hobby-Student AFAIK this is not implemented in the windows client but I guess that it should be quite easy to implement if the basic scaling is already there and working.


[websocketpp] Add features to build without Boost

This creates features for websocketpp so that the consumer can choose the ASIO library and whether openssl and zlib are used (since these are all optional).
This should be a transparent change for existing users.

This has been requested in issue #9404

@joefocusrite, thanks for the PR!
Could you update the baseline version via the following command, and then summit the changes?
./vcpkg x-add-version websocketpp --overwrite-version

@joefocusrite, could you execute './vcpkg x-add-version websocketpp --overwrite-version' and summit the changes?
Error: While reading versions for port websocketpp from file: C:\a\1\s\versions\w-\websocketpp.json
       File declares version `0.8.2#1` with SHA: 02d3d2638964b690dafd98788c4a035c11611077
       But local port with the same verion has a different SHA: 0ca3e13e38a50cb0da259618ab3bd19a8eaace05
       Please update the port's version fields and then run:

           vcpkg x-add-version websocketpp
       to add a new version.
To attempt to resolve all errors at once, run:

    vcpkg x-add-version --all

Hi @PhoebeHui. Thanks for getting back to me! I've just re-run that command and pushed 🙂

/azp run

LGTM, thanks!


Add support for icons with custom tags

you should consider adding support for creating icons with custom tags in game, considering the mod can display everything i threw at it so far. for example:
custom heads

enchantment glint

right now you have to manually edit the menu.json, and with a simple python script i can pull and format the data from a /give command generated with an online tool, making it a relatively easy process.
with that being said, i don't know how you'd be able to do it, but i think you're smarter than me so chances are you could figure it out.
not having access to custom heads feels restricted, so really give it a thought, with like the 1.21 update or something
tested on 1.20.4 fabric

i figured i'd share the script for anyone who wants mess around with custom heads and stuff
script.py
just paste the give command to standard input and it will poop out the proper "{Count:1...}" shebang. you can pipe it to wl-copy on wayland or something and paste it in the menu.json
it might not be pretty or neatly organized but it does convert all edge cases i could think of ¯\_(ツ)_/¯

Added now, with the same format as it would be in the json :)
Had to change functionality a bit for the icon pick screen to achieve this, but nothing too major.


⚠ Pihole-NBG has degraded performance

In e0f07d0, Pihole-NBG ($URL_PIHOLE_DE) experienced degraded performance:

HTTP code: 200
Response time: 13532 ms

Resolved: Pihole-NBG performance has improved in 68d215b.


Pie chart

Version
4.9.0
Steps to reproduce
Pie charts working well on PC browser but when i change to mobile it all mix up
What is expected?
Pie chart to be responsive
What is actually happening?
it miss up

Hi @HaniGerges , we've optimized the pie labels display in the coming 5.0

And https://echarts.apache.org/en/tutorial.html#Responsive Mobile-End might help.


Refactor SslHandler internals to always use heap buffers for JDK SSLE…

…ngine implementation
Motivation:
We should aim to always use heap buffers when using the JDK SSLEngine for now as it wants to operate on byte[] and so will do internal memory copies if a non heap buffer is used. Beside this it will always return BUFFER_OVERFLOW when a smaller buffer then 16kb is used when calling wrap(...) (even if a very small amount of bytes should be encrypted). This can lead to excercive direct memory usage and pressure for no good reason.
Modifications:
Refactor internals of SslHandler to ensure we use heap buffers for the JDK SSLEngine impelementation
Result:
Less direct memory usage when JDK SSLEngine implementation is used

/cc @gmcatsf @tbrooks8 WDYT ?

@normanmaurer, my understanding is that bytes in heap buffer will be copied into native memory later in the pipeline and could you comment if this change would not increase pressure down the pipeline?

@gmcatsf at the end you will native memory and there is no way around this requirement. That said with this change the needed native memory should be minimised as it is very unlikely that you will always need 16kb buffers.

I wonder if you could also test / verify / benchmark

What are you interested in? Like a JMH comparing usage of the SslHandler with direct buffers vs. heap buffers?
I assume there are already unit tests around the SslHandler?

@tbrooks8 I was just interested if you have some sort of benchmark that involves SslHandler with the JDK SSLlEngine that you used for previous optimisations. We have some as well :)

@normanmaurer My prior PR was motivated by the fact that I saw direct buffers being allocated (assertions triggering) in an environment without direct buffer pooling enabled. Not like a specific microbenchmark.
I can download a netty snapshot against my build and check that the test assertions are not triggering anymore?

@tbrooks8 I guess that is a good start :) Should I generate one for you or can you do it by yourself ?

I can do it. I'll pull your branch and comment with results later today.

@tbrooks8 <3 ... thanks a lot!

I built the jars today and ran our Netty module test suite and everything looked good to me.
We do have some full Elasticsearch benchmarks that use TLS.  But unfortunately it's not the type of thing that I can easily run against a snapshot. I can monitor after the next release and reach out if I see any regressions though.