Hit an AssertionError when running gen_mean_face.py in SAN

If you are submitting a feature request, please preface the title with [feature request].
If you are submitting a bug report, please fill in the following details.
Which project are you using?
SAN or SBR or TS3 or others?
SAN
Issue description
I ran gen_mean_face.py with the dataset folders in my home directory but I hit this error:
AssertionError: /home/dxy/datasets/300W-Style/300W-Original/afw/4538917191_17.jpg is not a file
I might have misinterpreted what /home/dxy is, but I am sure I placed the datasets folder in my home directory.
Google has not said anything either, only taking it as US dollar.

/home/dxy is the home directory path on my local machine. I will check the path soon

@MANGA-1614  Did you download the snapshot provided by me and then run gen_mean_face.py. If so some file paths in the snapshot is that on my machine, you can (1) generate those snapshots on your machine and then run gen_mean_face.py or (2) modify gen_mean_face.py to correct the path.

Hi,
Thank you for your response. I modified the snapshot files instead. I now
ran into a different problem.
AssertionError: The path : ./snapshots/BASE-CLUSTER-300W_GTB-3/cluster-00-03.lst is not a file.
I looked through your snapshot folder in Google Drive but couldn't find
this. Was I supposed to make this myself?
On Fri, Nov 22, 2019 at 7:39 PM D-X-Y<EMAIL_ADDRESS>wrote:

Closed #52 https://github.com/D-X-Y/landmark-detection/issues/52.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/D-X-Y/landmark-detection/issues/52?email_source=notifications&email_token=AC7NDCU4MLQSXOBCAPQ4RILQVB3S5A5CNFSM4JQTTW3KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVBQ2BEY#event-2824970387,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7NDCWYRXK5YCRJLPHPWGTQVB3S5ANCNFSM4JQTTW3A
.

@MANGA-1614 You can
(1) generate by your own by running bash scripts/300W/300W_Cluster.sh 0,1 GTB 3 as in https://github.com/D-X-Y/landmark-detection/tree/master/SAN#300-w
or (2) use the existing one from https://drive.google.com/open?id=1DtUg0LoZXNuDxeqRtcRxrA4tjrN9rpJa .
I prefer you generate it on your own as this step does not cost much time.

@D-X-Y/landmark-detection
<EMAIL_ADDRESS>I do have that one, straight from Google Drive. I couldn't run it since it
was a shell script and most of my Python was done in Windows.
However, in gen_mean_face.py, it looks out for 2 lists: CLUSTER-300W_GTB-3
and BASE-CLUSTER-300W_GTB-{:d}.
`
save_path = osp.join('cache_data', 'cache', 'clusters',
'300W-{:}'.format(cluster_num), '300W-{:}-{:}.png'.format(i, cluster_num))
calculate_mean(['./snapshots/CLUSTER-300W_GTB-{:d}/cluster-{:02d}-{:02d}.lst'.format(cluster_num,
i, cluster_num)], 68, save_path)
save_path = osp.join('cache_data', 'cache', 'clusters',

'300W-BASE-{:}'.format(cluster_num), '300W-BASE-{:}-{:}.png'.format(i,
cluster_num))
calculate_mean(['./snapshots/BASE-CLUSTER-300W_GTB-{:d}/cluster-{:02d}-{:02d}.lst'.format(cluster_num,
i, cluster_num)], 68, save_path)
`
I got through the first one, but not the second one.
I'm terribly sorry about my inquiries. I'm new to both Python and Computer
Vision.
On Fri, Nov 22, 2019 at 9:40 PM D-X-Y<EMAIL_ADDRESS>wrote:

@MANGA-1614 https://github.com/MANGA-1614 You can
(1) generate by your own by running bash scripts/300W/300W_Cluster.sh 0,1
GTB 3 as in
https://github.com/D-X-Y/landmark-detection/tree/master/SAN#300-w
or (2) use the existing one from
https://drive.google.com/open?id=1DtUg0LoZXNuDxeqRtcRxrA4tjrN9rpJa .
I prefer you generate it on your own as this step does not cost much time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/D-X-Y/landmark-detection/issues/52?email_source=notifications&email_token=AC7NDCQL37PNWLG32DNXOGDQVCJYTA5CNFSM4JQTTW3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE7LOAY#issuecomment-557758211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7NDCVUMQRQNEQUZALCUU3QVCJYTANCNFSM4JQTTW3A
.

I see. The "BASE-C...." is generated by scripts/300W/300W_Base_Cluster.sh. You can try to modify the script to run on Windows.

Ah ok, I see. Sounds good. I'll try to edit that then.
Right now, I've decided to comment it out. I'll return it back if anything
happens.
On Fri., Nov. 22, 2019, 21:50 D-X-Y<EMAIL_ADDRESS>wrote:

I see. The "BASE-C...." is generated by scripts/300W/300W_Base_Cluster.sh.
You can try to modify the script to run on Windows.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/D-X-Y/landmark-detection/issues/52?email_source=notifications&email_token=AC7NDCVKSWHXQY5ZTECWRELQVCK67A5CNFSM4JQTTW3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE7LTPI#issuecomment-557758909,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7NDCT5BC7OCDHJGOCKTPTQVCK67ANCNFSM4JQTTW3A
.


Update dependencies

This PR updates softer defined \ transient dependencies and is supposed to cover for a number of open dependabot PRs in one feel swoop.
Three minor issues discovered:

Axios no longer need data to be set on a request for content-type to be populated. Instead, it now sends the value provided in the data field verbatim and breaks auth flow. Removing previous hack fixes the problem.
fontello-update's dependency open has been defined very broadly and as >=6.0.0 and switched to ESM starting 9.0.0. We do not use ESM on this project and it isn't worth to start because of a single transient dependency.
When adding ST2_PROTOCOL we forgot to add it to our serve task. This creates a problem when you trying to run local version of st2web against dockerized st2.

In general, I have to admit, I'm beyond impressed with how well this project held up for all this years with minimal maintenance it had. Thanks to the community for keeping it afloat.

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

@Kami welcome to the party! 🎉


OpenSource : 5.9 Explain everything related to Kubernetes

Description :

Explain everything related to Kubernetes
How to configure our projects with it

Note :

Changes should be made inside the Open_Source/ directory & Opensource branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR == 1 Contributor
For Contributor please don't create the same issue again, before creating any issue do visit OpenSource Milestone
Doc name should same as the issue title

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.
Happy Contributing 🙌

Can I try it?

Can I work on this?

can i work on this @Aryamanz29 ?

@Harshkumar62367  Assigning this to you, Happy Learning 🎉🎉


Update the theme and take inspiration from the docs-only website

See https://docsyjustdocs.netlify.app/ and https://github.com/lisaFC/justdocs/.
Points to check:

hiding chlidren pages from page footer
vote option at bottom of page (useful? yes/no)
display of current entry in menu

Done by @thebe14 via #244


Please publish API docs in current UI

Thanks for creating an issue! Please fill out this form so we can be
sure to have all the information we need, and to minimize back and forth.

What are you trying to do?

Use the search.maven.org API but documentation links are only in the old UI (https://search.maven.org/classic/#api). This implies that the API isn't available anymore.

What feature or behavior is this required for?

API client access (Specifically I have an integration test that compares the current artifact to the previously released artifact, although I'm sure there are lots of other use cases)

How could we solve this issue? (Not knowing is okay!)

Add a "API Docs" link from https://search.maven.org/ :)

Anything else?

@justinedelson neither the Bookmarkable URLs or the REST API parts have changed and are accessible for you to use. If there is a specific REST API request not working please let us know.
I do think we could add an page on our new UI with a Bookmarkable URLs section using the urls for the current UI, is that what you were meaning as well?

@SirMaster This is not really about the APIs (or Bookmarkable URLs) per se. It is really about how that information is discovered. At present, to get to the API documents...

Start on https://search.maven.org/
Click on "Classic Search"
Click on "API Guide"

This navigation path was, at least for me, non-intuitive. It also suggests (incorrectly I hope) that the API is somehow related to the "Classic" UI.
To be clear, everything is working and nothing is broken. There's just an opportunity to improve discoverability.

@justinedelson ah ok, clear to me now. Yeah I think that is a very valid point, and I think we missed that totally. I hope we can do something for that in the near future. Thx again for submitting this.


Live reload: app reloads when editing redux action, but doesn't use latest code until manual reload

RN 0.61.5
When editing a redux action that is imported into a component via react-redux connect(), the edit triggers a full app reload. However, the latest action code isn't actually used - for example, if I've added a console.log this won't occur until I manually (CTRL+R) reload the app. This means edits to actions often require two app reloads (one automatic and one manual).

@dancherb have you solved this problem? The hot update is incomplete. Is there a configuration for the code to reload completely. It takes a lot of time to develop the app by hand every time.


let the worker sleep 1 second more when app is down for maintenance

If the queue driver, like sqs, supports long polling, it is better to set --sleep=0 when running queue:work console command. But if it is done so, the CPU will reach 100% when application is down by artisan app:down.
If you find this PR valid, please decide if it is backward compatible or not, and if it should be applied to earlier versions as well.
Thanks.

:-1: Don't use zero for sleeping. Makes no sense.

@halaei can you explain more why you use sleep=0?

Short answer
long polling
Long answer
Sleeping is one of the worst practical solutions to wait for job when there is nothing in the queue. Because you don't want your jobs to be delayed because your workers are currently sleeping.
You only want your jobs to be delayed when all the workers are really busy.
One better solution is called long polling.
Amazon SQS supports long polling.
I also wrote my own Redis-based queue driver that supports long polling.
With long polling option available, it is sleeping that makes no sense any more. Because the queue server does the waiting for you in a better way.


Conflicting names for type and function

Both the function EmblaCarousel and the type are exported with the same name which results in the type not being able to import.
export declare type EmblaCarousel = {
    canScrollNext: () => boolean;
  ...
};
declare function EmblaCarousel(sliderRoot: HTMLElement, userOptions?: EmblaOptions): EmblaCarousel;
export default EmblaCarousel;

Renaming either the type or the function name would fix this.
I tried to do something similar as in react.ts:
import { EmblaCarousel as EmblaCarouselType } from '../vanilla';

But that still give me errors.
As a workaround I could do this:
import EmblaCarousel from 'embla-carousel'
type EmblaCarouselType = ReturnType<typeof EmblaCarousel>

Hello @korsvanloon,
Thank you for your suggestion. I decided to add the Type suffix to all types. So this is how the index.ts file turned out based on your suggestion:
export { EmblaOptions as EmblaOptionsType } from './vanilla/components/options'
export { EmblaEvent as EmblaEventType } from './vanilla/components/eventEmitter'
export { EmblaCarousel as EmblaCarouselType } from './vanilla'

export { default } from './vanilla'
export { useEmblaCarousel } from './react'

So release v3.0.20 should include what you need.
Give it a go and let me know if it helps!
David

@korsvanloon have you had a chance to try the new type exports?

@korsvanloon I'm closing this as resolved. Feel free to re-open this issue if needed.
Kindly,
David


Add right click popout on Shift+F6 pane

for commanders, they can see the time the players are docked in base now from the mini team pane - this would be a good place to allow the comm to right click on a player and choose to "Chat" (starts composing a PM) or "Boot" (raises are you sure popup)
Imported from trac ticket #197,  created by Imago on 08-01-2010 at 18:41, last modified: 08-05-2010 at 17:23

r893
Added Chat, accept, and reject.  Boot is possible, but I think its presence will dissuade use of Chat due to misclicks.  Even w/ the "are you sure box" that could be annoying.  Plus, we don't need more boots.
Trac comment by Xynth on 08-05-2010 at 01:03


BLE test code inop - no RX from module

Receiving from the BLE module isn't working, thus TX isn't easily verifiable either.
When data is being transmitted by the smartphone app, RX doesn't idle high as it should, when probing with a LA. There are short periodic pulses, in the vicinity of where data bytes should be. If the test code is started without a serial object taking control of the I/O pins, and the pin is brought high manually instead, it does go high, so it's not shorted to ground (measuring this with a dmm confirms this anyway).
There's an (undocumented) NPN transistor (Q1, S8050) on the main/carrier PCB connected such that the base is normally connected to the Nano's A2 (mapped to the Propeller's P24 when it's connected instead), the collector is connected to the Nano's D0 (RX), and the emitter to the BLE module's UART_TXD. When the Nano is connected, the voltage output from A2 is about 1.0v. Using a counter-based DAC (duty mode) in the Propeller, the same voltage is achievable, but this has no effect. This would seem to be for transmitting from the MCU, anyway.
Is this being used as some sort of level shifter for interfacing with 5v MCU's (which the Nano is)? The BLE modules's datasheet seems to say it is a 3.3v device.

Update:
After speaking with Elegoo, no, the transistor "arbitrates" access to the TX/RX pins on the Nano between serial upload and BLE module access.
Because the computer uploads the code to the UNO board by using the PIN TX and PIN RX. In the meanwhile, the RX/TX pins are occupied by the Bluetooth module, which will cause the upload to fail. Please download the schematic file for the mainboard from the attachment.

We add a transistor to the RX pin circuit and it solves the problem we mentioned above. The transistor is like a switch and the RX pin is occupied by the Bluetooth module only when there are data sent from the module. The rest of the time, it’s occupied by the computer.


PR: Add a git subrepo for python-language-server

This will allow us to have a local version of the PyLS that is in sync with what we can provide for code completion and linting.
Todo

[ ] Improve Contributing guide.

Hello @ccordoba12! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file bootstrap.py:

Line 213:10: E211 whitespace before '('

In the file external-deps/python-language-server/pyls/__main__.py:

Line 35:80: E501 line too long (82 > 79 characters)
Line 36:80: E501 line too long (86 > 79 characters)

In the file external-deps/python-language-server/pyls/_utils.py:

Line 69:80: E501 line too long (100 > 79 characters)
Line 72:80: E501 line too long (83 > 79 characters)
Line 77:80: E501 line too long (93 > 79 characters)
Line 149:80: E501 line too long (90 > 79 characters)
Line 151:80: E501 line too long (90 > 79 characters)
Line 164:80: E501 line too long (115 > 79 characters)
Line 165:80: E501 line too long (82 > 79 characters)
Line 167:80: E501 line too long (119 > 79 characters)
Line 173:80: E501 line too long (102 > 79 characters)
Line 191:80: E501 line too long (102 > 79 characters)

In the file external-deps/python-language-server/pyls/config/config.py:

Line 39:80: E501 line too long (84 > 79 characters)
Line 48:80: E501 line too long (90 > 79 characters)
Line 49:80: E501 line too long (106 > 79 characters)
Line 50:80: E501 line too long (111 > 79 characters)
Line 55:80: E501 line too long (96 > 79 characters)
Line 66:80: E501 line too long (90 > 79 characters)
Line 107:80: E501 line too long (84 > 79 characters)
Line 114:80: E501 line too long (92 > 79 characters)
Line 128:80: E501 line too long (81 > 79 characters)
Line 129:80: E501 line too long (95 > 79 characters)
Line 140:80: E501 line too long (92 > 79 characters)
Line 153:80: E501 line too long (88 > 79 characters)

In the file external-deps/python-language-server/pyls/config/source.py:

Line 74:80: E501 line too long (85 > 79 characters)

In the file external-deps/python-language-server/pyls/hookspecs.py:

Line 106:80: E501 line too long (80 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/autopep8_format.py:

Line 16:80: E501 line too long (84 > 79 characters)
Line 17:80: E501 line too long (81 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/definition.py:

Line 20:80: E501 line too long (81 > 79 characters)
Line 23:80: E501 line too long (81 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/flake8_lint.py:

Line 93:80: E501 line too long (87 > 79 characters)
Line 122:80: E501 line too long (82 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/highlight.py:

Line 16:80: E501 line too long (84 > 79 characters)
Line 23:80: E501 line too long (104 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/hover.py:

Line 32:80: E501 line too long (100 > 79 characters)
Line 49:80: E501 line too long (81 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/jedi_completion.py:

Line 66:80: E501 line too long (95 > 79 characters)
Line 67:80: E501 line too long (93 > 79 characters)
Line 69:80: E501 line too long (85 > 79 characters)
Line 71:80: E501 line too long (99 > 79 characters)
Line 166:80: E501 line too long (83 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/mccabe_lint.py:

Line 15:80: E501 line too long (82 > 79 characters)
Line 19:80: E501 line too long (81 > 79 characters)
Line 33:80: E501 line too long (83 > 79 characters)
Line 34:80: E501 line too long (102 > 79 characters)
Line 36:80: E501 line too long (113 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/preload_imports.py:

Line 9:80: E501 line too long (83 > 79 characters)
Line 11:80: E501 line too long (90 > 79 characters)
Line 12:80: E501 line too long (90 > 79 characters)
Line 13:80: E501 line too long (83 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/pycodestyle_lint.py:

Line 34:80: E501 line too long (80 > 79 characters)
Line 58:80: E501 line too long (89 > 79 characters)
Line 65:80: E501 line too long (109 > 79 characters)
Line 67:80: E501 line too long (103 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/pydocstyle_lint.py:

Line 18:80: E501 line too long (81 > 79 characters)
Line 32:80: E501 line too long (107 > 79 characters)
Line 33:80: E501 line too long (81 > 79 characters)
Line 38:80: E501 line too long (83 > 79 characters)
Line 39:80: E501 line too long (80 > 79 characters)
Line 61:80: E501 line too long (81 > 79 characters)
Line 66:80: E501 line too long (80 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/pyflakes_lint.py:

Line 24:80: E501 line too long (89 > 79 characters)
Line 66:80: E501 line too long (98 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/pylint_lint.py:

Line 117:80: E501 line too long (83 > 79 characters)
Line 119:80: E501 line too long (84 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/references.py:

Line 10:80: E501 line too long (106 > 79 characters)
Line 19:80: E501 line too long (98 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/rope_completion.py:

Line 21:80: E501 line too long (104 > 79 characters)
Line 22:80: E501 line too long (88 > 79 characters)
Line 23:80: E501 line too long (81 > 79 characters)
Line 34:80: E501 line too long (99 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/rope_rename.py:

Line 24:80: E501 line too long (92 > 79 characters)
Line 31:80: E501 line too long (94 > 79 characters)
Line 38:80: E501 line too long (81 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/signature.py:

Line 24:80: E501 line too long (102 > 79 characters)
Line 39:80: E501 line too long (80 > 79 characters)

In the file external-deps/python-language-server/pyls/plugins/symbols.py:

Line 42:5: E722 do not use bare 'except'

In the file external-deps/python-language-server/pyls/plugins/yapf_format.py:

Line 24:80: E501 line too long (82 > 79 characters)
Line 25:80: E501 line too long (80 > 79 characters)
Line 26:80: E501 line too long (80 > 79 characters)

In the file external-deps/python-language-server/pyls/python_ls.py:

Line 51:80: E501 line too long (80 > 79 characters)
Line 53:80: E501 line too long (85 > 79 characters)
Line 71:80: E501 line too long (94 > 79 characters)
Line 77:80: E501 line too long (83 > 79 characters)
Line 86:80: E501 line too long (85 > 79 characters)
Line 110:80: E501 line too long (99 > 79 characters)
Line 151:80: E501 line too long (88 > 79 characters)
Line 154:80: E501 line too long (110 > 79 characters)
Line 155:80: E501 line too long (93 > 79 characters)
Line 161:80: E501 line too long (82 > 79 characters)
Line 200:80: E501 line too long (111 > 79 characters)
Line 201:80: E501 line too long (118 > 79 characters)
Line 203:80: E501 line too long (81 > 79 characters)
Line 214:80: E501 line too long (97 > 79 characters)
Line 221:80: E501 line too long (108 > 79 characters)
Line 223:80: E501 line too long (99 > 79 characters)
Line 233:80: E501 line too long (94 > 79 characters)
Line 239:80: E501 line too long (80 > 79 characters)
Line 246:80: E501 line too long (82 > 79 characters)
Line 252:80: E501 line too long (87 > 79 characters)
Line 261:80: E501 line too long (97 > 79 characters)
Line 264:80: E501 line too long (87 > 79 characters)
Line 283:80: E501 line too long (87 > 79 characters)
Line 297:80: E501 line too long (110 > 79 characters)
Line 301:80: E501 line too long (93 > 79 characters)
Line 314:80: E501 line too long (99 > 79 characters)
Line 320:80: E501 line too long (87 > 79 characters)
Line 323:80: E501 line too long (87 > 79 characters)
Line 326:80: E501 line too long (95 > 79 characters)
Line 329:80: E501 line too long (82 > 79 characters)
Line 335:80: E501 line too long (87 > 79 characters)
Line 339:80: E501 line too long (97 > 79 characters)
Line 345:80: E501 line too long (105 > 79 characters)
Line 349:80: E501 line too long (101 > 79 characters)
Line 351:80: E501 line too long (82 > 79 characters)
Line 353:80: E501 line too long (91 > 79 characters)
Line 364:80: E501 line too long (93 > 79 characters)
Line 371:80: E501 line too long (90 > 79 characters)
Line 393:80: E501 line too long (102 > 79 characters)
Line 399:80: E501 line too long (85 > 79 characters)
Line 412:80: E501 line too long (80 > 79 characters)

In the file external-deps/python-language-server/pyls/uris.py:

Line 49:80: E501 line too long (81 > 79 characters)
Line 51:80: E501 line too long (80 > 79 characters)
Line 82:80: E501 line too long (95 > 79 characters)
Line 87:80: E501 line too long (89 > 79 characters)

In the file external-deps/python-language-server/pyls/workspace.py:

Line 46:80: E501 line too long (95 > 79 characters)
Line 65:80: E501 line too long (115 > 79 characters)
Line 68:80: E501 line too long (82 > 79 characters)
Line 75:80: E501 line too long (92 > 79 characters)
Line 93:80: E501 line too long (110 > 79 characters)
Line 96:80: E501 line too long (97 > 79 characters)
Line 100:80: E501 line too long (105 > 79 characters)
Line 101:80: E501 line too long (104 > 79 characters)
Line 115:80: E501 line too long (114 > 79 characters)
Line 134:80: E501 line too long (92 > 79 characters)
Line 195:80: E501 line too long (82 > 79 characters)
Line 198:80: E501 line too long (82 > 79 characters)
Line 218:80: E501 line too long (89 > 79 characters)
Line 220:80: E501 line too long (89 > 79 characters)
Line 224:80: E501 line too long (84 > 79 characters)
Line 232:80: E501 line too long (89 > 79 characters)
Line 237:80: E501 line too long (89 > 79 characters)
Line 248:80: E501 line too long (102 > 79 characters)
Line 253:80: E501 line too long (94 > 79 characters)
Line 260:80: E501 line too long (104 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_autopep8_format.py:

Line 3:80: E501 line too long (80 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_completion.py:

Line 12:80: E501 line too long (82 > 79 characters)
Line 13:80: E501 line too long (82 > 79 characters)
Line 107:80: E501 line too long (102 > 79 characters)
Line 111:80: E501 line too long (116 > 79 characters)
Line 121:80: E501 line too long (116 > 79 characters)
Line 127:80: E501 line too long (99 > 79 characters)
Line 128:80: E501 line too long (105 > 79 characters)
Line 144:80: E501 line too long (98 > 79 characters)
Line 156:80: E501 line too long (98 > 79 characters)
Line 254:80: E501 line too long (84 > 79 characters)
Line 288:80: E501 line too long (91 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_definitions.py:

Line 34:80: E501 line too long (94 > 79 characters)
Line 57:80: E501 line too long (94 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_pycodestyle_lint.py:

Line 100:80: E501 line too long (86 > 79 characters)
Line 101:80: E501 line too long (86 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_pydocstyle_lint.py:

Line 7:80: E501 line too long (85 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_references.py:

Line 42:80: E501 line too long (82 > 79 characters)
Line 50:80: E501 line too long (87 > 79 characters)
Line 69:80: E501 line too long (83 > 79 characters)
Line 72:80: E501 line too long (82 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_symbols.py:

Line 48:80: E501 line too long (80 > 79 characters)
Line 69:80: E501 line too long (80 > 79 characters)
Line 72:80: E501 line too long (83 > 79 characters)
Line 73:80: E501 line too long (82 > 79 characters)
Line 82:80: E501 line too long (91 > 79 characters)

In the file external-deps/python-language-server/test/plugins/test_yapf_format.py:

Line 58:80: E501 line too long (108 > 79 characters)

In the file external-deps/python-language-server/test/test_document.py:

Line 31:80: E501 line too long (82 > 79 characters)

In the file external-deps/python-language-server/test/test_language_server.py:

Line 31:80: E501 line too long (98 > 79 characters)
Line 35:80: E501 line too long (108 > 79 characters)
Line 50:80: E501 line too long (108 > 79 characters)
Line 57:80: E501 line too long (98 > 79 characters)
Line 77:80: E501 line too long (101 > 79 characters)
Line 79:80: E501 line too long (88 > 79 characters)
Line 92:80: E501 line too long (107 > 79 characters)
Line 101:80: E501 line too long (80 > 79 characters)
Line 103:80: E501 line too long (86 > 79 characters)

In the file external-deps/python-language-server/test/test_uris.py:

Line 47:80: E501 line too long (103 > 79 characters)

In the file external-deps/python-language-server/test/test_utils.py:

Line 84:80: E501 line too long (96 > 79 characters)

In the file external-deps/python-language-server/test/test_workspace.py:

Line 54:80: E501 line too long (107 > 79 characters)

In the file external-deps/python-language-server/versioneer.py:

Line 564:35: W605 invalid escape sequence '\s'
Line 568:35: W605 invalid escape sequence '\s'
Line 572:35: W605 invalid escape sequence '\s'
Line 613:52: W605 invalid escape sequence '\d'
Line 689:33: W605 invalid escape sequence '\d'


More efficient imports

When importing an icon using a destructuring (i.e. with named imports), it seems necessary (at least for some compilers?) to transverse the entire list of icons. This can lead to a relatively large performance penalty vs default exports. Here is feedback from one compiler and some ideas on how this can be improved.

Apparently someone already changed this repo to make it more efficient, so... PR?

I don't have that code, but it seems pretty straightforward (assuming you'd be happy with esbuild). Are you ok with having all the icons inlined to a single file? (while improving bundle speed, which will effectively duplicate the package size)

Not sure about that. I just checked @mui/material-icons and they also have one icon per file and re-exports in the index file. Webpack handles "pure" modules just fine and can optimize it. Sounds like a bundler issue to me, not like an issue that library maintainers should handle. :thinking:

If this isn't project related, feel free to close.
Would you care to comment on the original issue and see if there is some way forward here?

@mbrevda I did, but the issue over there is already closed. Apparently it's not an esbuild issue.

thanks!


Lost access to Windows Explorer and Control Panel

Description
I was unhappy with the Files app so I tried to remove it from my system but after I did, I could no longer access WinExplorer or Control Panel (accidentally, I found that I could access Explorer after I scanned something and then saved it - Explorer would pop-up but once I shut down or restarted my PC, it was gone again).
I reinstalled Files so I had some file organizer but I still don't like it; it constantly hangs and settings change on their own.  I have searched for solutions online but none of them have worked.  How can I uninstall Files and regain access to Explorer and Control Panel?
Steps To Reproduce

Install Files app
Uninstall Files app
Windows Explorer and Control Panel are inaccessible.

Requirements

e.g. Fix crash that occurs when...
Windows Explorer and Control Panel are still on my system but almost entirely inaccessible.

Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.4046
Log File
Nothing happens when I click on "open log location".

Turn off (if already off turn on then off again) Files settings > Advanced > Set as default and try open Windows File Explorer

@mondotoronto did you by any chance enable the experiential setting to replace File Explorer? If this is the case, you'll need to reinstall Files and toggle the option on & off to revert the registry edits.

Hi,
I appreciate the quick replies.
The "Set Files as the default file manager" setting has always been off (because of the warning!) and I'm not clear if Josh65 is suggesting that I enable it and then disable it.
I don't want to do anything that will make things worse.
Any further clarification or other suggestions are very welcome.
Thank you.Ron.
On Monday, March 11, 2024 at 04:02:26 p.m. EDT, Yair @.***> wrote:
@mondotoronto did you by any chance enable the experiential setting to replace File Explorer? If this is the case, you'll need to reinstall Files and toggle the option on & off to revert the registry edits.
image.png (view on web)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Also email attachments don't work for uploading images. You will have to click the View on GitHub link at the bottom of the email > Copy and paste or click the add attachment button to select the file > Wait until you don't see uploading in the text > Click the green comment button.

Hello,
This all began quite a while ago and, at the time, I didn't realize there was a forum for Files (it's such a generic name that online searches show too many useless results).
I've attached a screenshot of the error message I receive when I try to enter explorer.exe into the Run function.
For the Control Panel, I don't get any error message; it just doesn't open.
Thanks.    On Monday, March 11, 2024 at 05:30:52 p.m. EDT, Josh65-2201 @.***> wrote:
If you haven't ever turned the set as default option on then the issue is probably not from Files. Can you send a screenshot of what error you get when opening them.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Again email attachments don't work. You will have to click the View on GitHub link at the bottom of the email > Copy and paste or click the add attachment button to select the file > Wait until you don't see uploading in the text > Click the green comment button.

Apologies

Thanks, the error isn't caused by Files, but you can try running the below reg script to restart the explorer path
UnsetFilesAsDefault.reg.zip

Thank YOU!  That fix worked and I can now access both applications.
Now, as an amateur computer user, I am in no position to doubt you, but this problem arose immediately after I uninstalled Files, so I am not convinced that that didn't trigger something.
Added to that, your fix is labelled "Unset Files As Default" and it worked for me.
I can't say absolutely (I'm 65 years old!) that I didn't enable that setting but it goes against my normal caution when warned about possible negative consequences, so it's difficult for me to imagine why I would ever do that.  At the time, I didn't like the Files app and wouldn't have wanted to make it the default file manager.
But, anyway, I am glad that those old apps have been restored.  Do you think it's best that I just leave Files on my computer or would deleting lead to more problems?
Thanks very much for your time and attention.  It is appreciated.

Glad it worked and you can uninstall Files if you want. Files doesn't do anything by default to the registry and the only way would have been if the set as default option was turned on at some point.

Thanks again.


failed to parse url if there were special characters

example url:
mysql://root:123456!@$%^&&*()_@@@@<IP_ADDRESS>:3391
A possible solution
encode url

you need to encode each part (host, password, etc.) seperately
example encoded url:  mysql://root:123456!%40%24%25%5E%26%26*()_%40%40%40@<IP_ADDRESS>:3391

decode each part

example: in src/command/authenticator.rs

let password = url_info.password().unwrap().to_string();
let password = urlencoding::decode(&password).unwrap().to_string();

refer to: https://stackoverflow.com/questions/47670078/java-cant-connect-to-mysql-if-password-contain-symbol

fixed


Fixup test

Fixes up the test to work with the new bot config file.
Also refines the test for the bot's existence. We should now be able to have bots in our path and run them in the same way the DebugBot is run (with an empty RootPath).
There are a couple of calls to windows functions that will need to be proxied out.

Please let me know if you notice any mistakes. There's bound to be some.
Regardless, the test now works again on my machine. It would be great if someone could confirm it works on theirs by un-commenting the call to RunTests() in main() and running it.

@alkurbatov As you say, there are a lot of conflicts with #21. If @Cryptyc wants, he could merge one and I'd be glad to resolve the merge conflicts for the other.
The "Utils" cpp and h files in this PR can be merged with the Tools files from #21.

@Cryptyc can't merge #21 until you resolve the merge conflicts in the PR (if I not mistaken GitHub blocks such merges).
Also, AFAIK the common approach is when contributors do it, not maintainers.

Understood.
My normal process it to take care of conflicts by merging master into the feature branch and resolving conflicts before merging back into master. Happy to do so (or side merge one branch into the other in this case) - whatever works.

@Cryptyc No problem.
Now that #21 has been merged, I'm happy to do any required cleanup on this branch. I'll hold off for now in case you're already looking into it, but let me know if you'd like me to do this.

Of corse it will. It has Windows.h and other Windows-specific things inside.All the new code either must be part of Tools.h (probably the best for this PR) or follow it’s approach.

This has been merged in https://github.com/Cryptyc/Sc2LadderServer/commit/1242b2bd54eec17f985f96626b65e15ab2c4100b
I made changes to move things around and keep OSX compatibility, which means it diverges from this PR a bit.  So closing the PR

Also I don’t see why do we keep solutions files after introduction of cmake.

Yes they'll need to be cleaned up and the DebugBot moved into the src folder and cmake configured to generate it as a new project.


Fix up typescriptlang.org 'Learn' link

Figure out how to merge in our .md files the way we did the CodePlex wiki content, host a 'master' MD page that links everything together, or some other way to let people read all the content in an organized fashion.
There should be some tools out there for this already.

Do we want README.md to contain the contents so that people landing on the page can see it? Or do we want an altogether separate Handbook.md to act as the master MD file?
From offhand searches, Pandoc might be the way to go.

@DanielRosenwasser let's get that done then, so that any updates we make are visible on the site. once we have something working, @jonathandturner can help you getting it on the live site.

Hi guys,
As I can see you know that the handbook section of the typescriptlang.org site doesn't show the latest version of the document (the internal module -> namespace change not mentioned). And the latest version itself doesn't mention the ES6 module loading syntax (I read about it from the language spec).  As a beginner with TypeScript (and JavaScript - that's the problem :) ) it was very hard to me (took me several hours) to realize I'm reading old documentation. I believe that the site is supposed to be the starting point for those interested in the language but outdated documentation causes a lot of confusion. I would suggest, as a first step, specifying in the document the language version it reflects and probably link to the latest version at github.

@mhegazy I think it would be appropriate to forward people to GitHub until we fix the handbook.

should be fixed now. the website has been updated see https://blogs.msdn.microsoft.com/typescript/2016/03/24/new-typescript-website/

:thumbsup:


Docs/integrate with main docs

📑 Description
Integrate the template documentation with the main docs at apigear.io

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Only unrelated tests to the docs changes have failed.


Converted popen2 to subprocess

This is the only instance of the popen2 library. What about os.popen? We use that a lot.

Looks to be still allowed but implemented under the hood with subprocess.

@ticoann @amaltaro please review and merge. We will not deal with os.popen, at least now.

Although it seems look fine but this probably need a test.

I'm going to test it today.

@amaltaro @ticoann was this tested? Can it be merged?

I'm gonna run some quick tests on it as well.

I ran the test before, and I didn't see any apparent problem. but It would be nice Alan can run the test as well.

@ticoann good to go


Link to installed path

This fixes #50 for me.
Curious about tests.

I apologise for the delay, I wasn't even aware of your PR's.
I'm happy with this - though typically it will only install as root when the user running the ansible playbook is root. So typically if you run the role as a user (example below), this isn't actually a problem.
Until somebody speaks out, I expect this to work as intended based off the existing fubarhouse_user variable.
Example (source https://github.com/fubarhouse/drupal-vm/blob/master/provisioning/tasks/post.yml)
- name: Include fubarhouse.golang role
  become: yes
  become_user: vagrant
  include_role:
    name: fubarhouse.golang
  when: '"golang" in installed_extras'


The location of  last stride 2 block is different from the paper?

Hi,
In the cfgs of mobilenetv3_large model, the location of the last stride 2 block seems to be different from the paper?
Am i missing anything?
Thanks

could you specify the difference?


Provide minified theme css

Motivation
It is common to have min.css files in distributed npm packages to profit from file size savings.
Description
Currently I am using import 'prismjs/themes/prism-okaidia.css'; but the css file is not minified.

Related to #956.


[Bug] Nav Menu does not work properly on mobile

Items that are too long overflow and aren't positioned properly

I think we could fix this by switching to flex-direction: column on small screen, with menu elements taking the whole width


Speciation leads to an over-representation problem

Speciation process leads to species with few genomes producing a lot of offsprings. This entailed several issues such as stagnation, less diversity and many other.
Potential solutions:

Minimum species size: If a species falls below the minimum size, we can either merge it with another compatible species or remove it from the population.
Offspring limit that depends on species's size

Offspring limit that depends on species's size is implemented but over-representation is still an issue


Add new environ parametr for Travis

Currently build is failing on my PR because there is not a SECRET_KEY in travis environ, so we should setup it to test PR(#75)

Done.


BUG: hull() fails with TypeError

From @z3dev on January 4, 2016 3:42
function main() {
// BUG: If resolution is changed to 18 then there's an error
// "TypeError: this.points[p1] is undefined"
var r = 18;
var a = CAG.circle({center: [-5,0], radius: 3, resolution: r});
var b = CAG.circle({center: [+5,0], radius: 3, resolution: r});
b = hull(a,b);
return b;
}
Copied from original issue: Spiritdude/OpenJSCAD.org#114

Interestingly this version works: (without using the CAG namespace)
function main() {
var r = 18;
var a = circle({ r: 3, $fn: r});
var b = circle({r: 3, $fn: r});
b = hull(a,b);
return b;
}

Not relevant anymore.


Network interface summary does not properly break into multiple lines

If you are having a lot of network interfaces (e.g., virtual bridges for containers), the text interface of below does some very creative line breaks:

It should properly break the columns over to the next line, and it should also indent the next line properly after the Iface (Rx|Tx= label. I'm just guessing here when I say that the same problem probably applies to the block devices (I/O (Rd|Wr)).

This may not be the exact fix you wanted, but #8180 limited the number of devices to 5 so it should generally not overflow to a second line.

Well, that's exactly not a fix but rather a work-around. A fix would properly break that into multiple lines. But it's a start, and especially it's a good work-around for the issue it is meant to prevent (having many many disk devices).
Thanks, I guess. It makes reading the stats less confusing. But I think the layout of the header really needs an overhaul.


Abort process if refcount overflows

This resolves a TODO left in the comments.

Travis fails due to an unrelated problem: associated constants are not supported by Rust 1.18.0.

Thanks so much for this and sorry for the delay. CI has not been a good state. We got CI working on master! would you be able to pull in the latest master?

CI is passing now :)

Thanks!


Frame numbers are in Float

Hi,
From the JSON file I can see that the frame numbers are in float. Can you please tell me the interpretation of this?

It's the video timestamp (https://github.com/antingshen/BeaverDam/issues/132#issuecomment-359611994)


not being able to download

You can download this project from the Green "Code" button and pressing "Download Zip". Alternatively, just use this link
https://github.com/Sycnex/Windows10Debloater/archive/master.zip
Then, you should reference the install guide from the ReadMe.
If you are unable to download the project from the link provided, the issue is likely on the side of your browser/internet connection.


License connection sequence aborted

Hi
I'm unable to connect to any Windows 2003 SP2 server running Terminal Services with no additional security, no NLA, no SSL certificates.
$ xfreerdp  /v:<IP_ADDRESS> /cert-ignore -sec-nla -nego -sec-tls
[12:30:19:189] [23313:-1170782464] [ERROR][com.freerdp.core.connection] - license connection sequence aborted.
[12:30:19:190] [23313:-1170782464] [ERROR][com.freerdp.core] - freerdp_set_last_error 0x2000D

The same command works fine when I try to connect to a Windows 2012 Server.
Testing environment:

Arch Linux x64_64 (package freerdp-git)
xfreerdp version: This is FreeRDP version 1.2.0-beta1 (git 6cd55a5)
Destination host: many Windows 2003 SP2 servers

The previous freerdp 1.1.0-beta1 works fine with the same command line.

Can't connect to the Windows Server 2012, still getting license connection sequence abort:
https://gist.github.com/sitano/bdf2ce7f2af949c960204689df6e77b6 .
FreeRDP /version: This is FreeRDP version 2.0.0-dev (git 3586e33)

Can't connect to Windows 2008R2, because of Terminal licensing server is disconnected. :(
But rdesktop and mstsc.exe connected without any problem (with notification in tray only).
/admin solved the problem, but I would like to connect without /admin, if I can do it by the same script...

I simply solved it using sudo.

@BlackBlade1632 please do not post such advice, using sudo with FreeRDP is a sign you do something wrong.

@akallabeth I never used sudo until this happend. I started using sudo because stopped working.


add focus states

add :focus selectors to <a> and <button> elements Set focus states for <a> and <button> elements to provide parity for mouse and keyboard interactions.

Good call — in Chrome/Firefox the default outlines appear on focus, but not in Safari and likely other browsers.
I'd prefer to add a proper focus state instead of reusing the hover state, and consistency across browsers is highly desirable. I really like how BS4 beta buttons handle this, but we need access to color modifiers to make it happen without setting arbitrary variable values.

Agreed, a unique focus state would be best, but I just went with the idea of "get something in there because reusing hover is better than nothing" rather than "let me try to guess what design aesthetic you would want" :)
totally cool with cancelling this PR if you want to design these yourself.

This was fixed yesterday in the buttons rework. Thanks for the heads up 👍


strings: remove unused var primeRK redefinition

I signed it!

Sorry, the original code was right.


Remove deprecated lifecycle method componentWillReceiveProps

Using the component with a version of React greater than 16.9 will produce the following warning:
 Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

https://github.com/howlowck/react-powerbi/blob/22c21d4c3601e2873a78b5b2f145a22d645952fb/src/index.js#L23
This is due to changes introduced in React 16.3 and can be addressed by appending an UNSAFE_ to the lifecycle method, or refactoring the component to use a different lifecycle method.
React's suggestions on mitigating the changes can be found here.
Suggestions:

Append the UNSAFE_ string to the method.

This will at least allow users to update to react 17.0 when it releases
Can be done via npx react-codemod rename-unsafe-lifecycles

Refactor component state management to remove dependency on componentWillReceiveProps

At first glance this doesn't appear too bad, but I'm not particularly familiar with how the component state is managed, and my own implementation of the component is very simple. Without any test coverage I would be hesitant to introduce any potentially breaking changes.

Thanks.. for now, i'm leaning towards simply appending the UNSAFE_ string to the method.
Do you want to make a PR for that?

Opened a PR. Note that because the "UNSAFE_" addition breaks eslint's camelcase rule present in .eslintrc. I chose to update eslint and use the allow option on the existing camelcase rule, but that can be reverted if you'd rather keep the PR as small as possible.

I merged it and published it to npm<EMAIL_ADDRESS>Thanks for your contribution! Keep it up 😄


UHF-X lock core 10.3.10

Lock drupal core version. 10.4 breaks ckeditor link functionality

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

Project coverage is 17.87%. Comparing base (24833b5) to head (4a68067).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #871   +/-   ##
=========================================
  Coverage     17.87%   17.87%
  Complexity      285      285
=========================================
  Files            36       36
  Lines          1080     1080
=========================================
  Hits            193      193
  Misses          887      887

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


smaller number of bugs in the ntp configuration page

A small text mistake:
http://i.imgur.com/1Ztbsld.png
Change default to 'Generic' we are not pfsense :p
http://i.imgur.com/7fQNOge.png
Defaults don't match actual defaults
http://i.imgur.com/9zuZAh6.png

Update for 'Defaults don't match actual defaults'
Checkboxes and list changed based on selection of type.
All fields aside from baud rate and serial device should be set to readonly unless 'Custom' is selected.

Small addition: On the NTP tab under Timeservers, the "Add another Time Server" picture/button does not load because it does not exist.
http://opnsense.local/themes/opnsense/images/icons/icon_plus.gif

Fixed in https://github.com/opnsense/core/commit/95ba411113a63f0bb4c597c83e686a5f67cd3ed0 except locking baud rate and serial device (the current code is not very nicely written, extending it with more javascript will probably make things worse).

toggling the field to readonly is not an option?

There's quite some messy javascript in that file, fixing event handlers for the checkbox probably costs some extra hours and there's still a lot of other work to do.
If you send in a pull request, I will be happy to merge it in :-)


Model Weights

Hi,
Thanks for open sourcing your implementation of this architecture, do you have any plans to open-source the weights as well, it would be really helpful to quickly perform inferencing and do more research
Thanks again:)

Hi Nishchay,
I'm very glad you are interested in my implementation. I added the weights in this repo., feel free to use it, thanks.

Just wanted to say thank you as well :)

@Line290 I'm a bit new to these models. If I'm not mistaken the word map structure for the structure decoder and the word map structure for the cell decoder (JSON files) are also needed to use the model weights. Would you be able to provide those as well?
    parser.add_argument('--word_map_structure', '-wms',
                        help='path to word map structure JSON')
    parser.add_argument('--word_map_cell', '-wmc',
                        help='path to word map cell JSON')

Also, are the weights you provided for S1S1 or S2S2?
Thanks!


BCMOHAD-5339 : Lookup filter

The lookup filters need to do post deployment

The package.xml deployment  does not support lookup filter . It has been completed by post deployment


markdownlint-rule-github-internal-links

(Until we have Discussion forums everywhere, this seems like the best way to mention the following:)
I just wrote a custom markdownlint rule that checks all internal links according to the anchor generation that Github does in its Markdown rendered (e.g. for README.md files).  Because anchor generation is specific to the Markdown renderer, I figured a custom rule made more sense than a PR to markdownlint itself, but I'm happy to submit a PR if you'd rather.

NPM link
Github link

I think I got everything right, including the NPM keywords, but this is my first contribution "to" markdownlint, so let me know if you see anything I could improve.
I tested it on this pretty complicated Markdown file which includes fun features like <img> tags in the headings. (Testing all the links in this document was my motivation for writing this plugin.)
Anyway, hope this is helpful, and thanks for the great tool!  Writing a plugin was quite straightforward.  Feel free to close this issue.

This is awesome, thanks for sharing! I will give it a try soon and maybe add it to my nightly CI run. I see you have already added the appropriate keyword, so it shows up at https://www.npmjs.com/search?q=keywords:markdownlint-rule  where others will be able to find it easily. Congratulations!￼
FYI, you may want to mention https://github.com/DavidAnson/markdownlint-cli2 in your notes. It's a very new project I started as an alternative to the original CLI.￼￼

I'm not a CoffeeScript person, but a quick scan of your implementation looked perfectly reasonable. Thanks again for doing this!￼

Thanks! I hadn't looked at markdownlint-cli2 much yet, but just figured out how to use it with custom rules, and added corresponding documentation to the README. Nice that you can specify custom rules in the config instead of the command line!


Fix the useless retry when the maximum number of subscriptions is reached

Motivation
When the maximum number of subscriptions is reached, due to the long default value of operationTimeout, the client will keep retrying and get stuck for a long time.
If a NotAllowedException is returned, the client will not continue to retry and will fail directly.
Test
Existing unit tests can be covered
In unit test testMaxTopicsPerNamespace, when the subscription reaches the threshold, the client will retry. The unit test to be stuck for about 3 minutes, and occasionally unit test timeouts occur.

@315157973 Sorry, could you please help add a test to avoid the regression?


Multiple hypervisor nodes

Currently, kubeinit supports deploy HA mode (3 master nodes) but on single hypervisor server. It's actually not really HA if the hypervisor server is down. It's better option where master nodes run on different hypervisor server.

@lapd-viosoft thanks a lot for the feedback, I moved all these issues as cards in a kanban project https://github.com/Kubeinit/kubeinit/projects/1 if you will like to contribute, please feel free to push any PR you will like to have included in the code. When I have some free time I'll try to implement some of them.


Python module documentation

Documentation about adding and testing Python modules is added

@mikhailnikolaev if you think that adding python requirements is beyond of the scope of module documentation then I should add it to docker docs.

Actually, I think that this stuff is about Docker, not about the Python modules. So, maybe, it is more proper to add it in a Docker docs

@mikhailnikolaev if you think that adding python requirements is beyond of the scope of module documentation then I should add it to docker docs.

Actually, I think that this stuff is about Docker, not about the Python modules. So, maybe, it is more proper to add it in a Docker docs

Okay, I had doubts about this, I will do so.

@mikhailnikolaev if you think that adding python requirements is beyond of the scope of module documentation then I should add it to docker docs.

Actually, I think that this stuff is about Docker, not about the Python modules. So, maybe, it is more proper to add it in a Docker docs

I disagree. First, this is about fetching dependencies as a part of CMake configuration/build. Second, this is supposed to be a guide on adding a new python module to ADBench. So, as a developer I would expect, that following steps listed here should be enough to do so. But without CMakeLists that won't be the case. So, I suggest, adding Ilia's paragraph to this doc.


[SMALLFIX] Add javadoc for AlluxioLineage#AlluxioLineage.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13843/
Test PASSed.


Certificate requests too frequent

I'd like to understand this. I'm using the (default) automatic SSL certificate mechanism. Development entails frequent restarts drawing error messages (which cause server to fail) to the effect that that the requests are coming in too frequently.  Could defining an e-mail address certificate ID solve this by informing the certificate authority that its OK to reissue the same certificate?

Do you not define email address to bypass startup prompt or you only use the -agree flag ?
Are you running Caddy inside containers? If yes, I'd suggest you mount a host filesystem path that persists the certificates.

body p { margin-bottom: 0cm; margin-top: 0pt; }

Now I hadn't thought of that? I am running in a container (with
-agree) but I could persist the certificate. Where are the
certificates located?

On 25/1/16 14:18, Abiola Ibrahim wrote:

  Do you not define email address to bypass startup prompt or you
    only use the -agree flag ?
  Are you running Caddy inside containers? If yes, I'd suggest
    you mount a host filesystem path that persists the certificates.
  —
    Reply to this email directly or view
      it on GitHub.

$HOME/.caddy which will translate to /root/.caddy in the container.
I advised same on my docker image https://github.com/abiosoft/caddy-docker#run-the-image-1.

Error message:
caddy_1   | 2016/01/25 15:08:44 [levpoem.co.il] failed to get certificate: acme: Error 429 - urn:acme:error:rateLimited - Error creating new cert :: Too many certificates already issued for: levpoem.co.il
sitescommon_caddy_1 exited with code 1
It worked before and I just ran it after a 24 hour rest. It says too many certificates already issued. **
Any way to erase/reset that?** .caddy is mapped to docker-caddy/dotcaddy but I never saw anything in there
According to your docker setup, mine seems OK. Do you see anything wrong in my setup (attached)

Error message:
caddy_1 | 2016/01/25 15:08:44 [levpoem.co.il] failed to get certificate: acme: Error 429 - urn:acme:error:rateLimited - Error creating new cert :: Too many certificates already issued for: levpoem.co.il
sitescommon_caddy_1 exited with code 1
It worked before and I just ran it after a 24 hour rest. It says too many certificates already issued. Any way to erase/reset that? .caddy is mapped to docker-caddy/dotcaddy but I never saw anything in there
According to your docker setup, mine seems OK. Do you see anything wrong in my setup HERE?

You'll have to wait for some days. Its rate limit error from Let's Encrypt.

Wow! How many days? Would it help if I gave my e-mail address in the
Caddyfile?
On 25/1/16 17:42, Abiola Ibrahim wrote:

You'll have to wait for some days. Its rate limit error from Let's
Encrypt.
—
Reply to this email directly or view it on GitHub
https://github.com/mholt/caddy/issues/534#issuecomment-174547538.

Its a domain level limit.
https://community.letsencrypt.org/t/quick-start-guide/1631/6
5 certificates per domain per 7 days per.

Email address or not, what you have to make sure to do is that the certificates stay on disk so Caddy can re-use them.

Thank you. How can I configure Caddy to run without SSL in the meantime?
I thought it was "tls off" but it seems not. Also, why you don't you
issue -accept in the caddy command?
My Caddyfile:
http://<IP_ADDRESS>, https://<IP_ADDRESS> {
root /var/www/levpoem
tls<EMAIL_ADDRESS>}
http://www.levpoem.co.il {
root /var/www/levpoem
tls off
}
http://www.heartbt.co.il {
root /var/www/heartbt
tls off
}
<IP_ADDRESS> {
root /var/www/heartbt
tls off
}
On 25/1/16 17:52, Abiola Ibrahim wrote:

Its a domain level limit.
https://community.letsencrypt.org/t/quick-start-guide/1631/6
5 certificates per domain per 7 days per.
—
Reply to this email directly or view it on GitHub
https://github.com/mholt/caddy/issues/534#issuecomment-174551281.

body p { margin-bottom: 0cm; margin-top: 0pt; }

Yes, I realize that now too bad I didn't know. How can I operate
without SSL while I'm waiting for the renewal?

On 25/1/16 17:54, Matt Holt wrote:

  Email address or not, what you have to make sure to do is that
    the certificates stay on disk so Caddy can re-use them.
  —
    Reply to this email directly or view
      it on GitHub.

tls off or explicitly specifying http:// in the address should do what you want.

body p { margin-bottom: 0cm; margin-top: 0pt; }

Tnx
On 25/1/16 18:40, Matt Holt wrote:

  tls off or explicitly specifying http://
    in the address should do what you want.
  —
    Reply to this email directly or view
      it on GitHub.

Hopefully that works for you :+1: Sorry about the rate limits, not much we can do about that for now, but while you're still experimenting with getting your setup right, you should use Let's Encrypt's staging server (use the -ca flag to set it to https://acme-staging.api.letsencrypt.org/directory)

body p { margin-bottom: 0cm; margin-top: 0pt; }

thanks again. Does -ca have to be set explicitly or is it st that
way by default? Correct me, but I understand that -ca is meaningless
until 7 days pass to revive my certificates, right?

On 25/1/16 19:59, Matt Holt wrote:

  Hopefully that works for you  Sorry about the rate
    limits, not much we can do about that for now, but while you're
    still experimenting with getting your setup right, you should
    use Let's Encrypt's staging server (use the -ca flag to set it
    to https://acme-staging.api.letsencrypt.org/directory)
  —
    Reply to this email directly or view
      it on GitHub.

No, try it again with the -ca option set to the staging server URL. By default, it is set to Let's Encrypt real production endpoint. The staging server has much higher rate limits, so you should be able to get certificates that way (but they won't be accepted by browsers -- use for testing and experimenting).

body p { margin-bottom: 0cm; margin-top: 0pt; }

Wow. That's great just what I need over the coming week. Thanks

On 25/1/16 20:49, Matt Holt wrote:

  No, try it again with the -ca option set to the staging server
    URL. By default, it is set to Let's Encrypt real production
    endpoint. The staging server has much higher rate limits, so you
    should be able to get certificates that way (but they won't be
    accepted by browsers -- use for testing and experimenting).
  —
    Reply to this email directly or view
      it on GitHub.


Started http server

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v10.16.3

OS
rhel-openssl-1.0.x

Prisma
2.1.3

Description
Started http server

Logs
 }  +3s
  prisma-client { engineConfig:
  prisma-client    { cwd: '/var/www/up-osmp/prisma',
  prisma-client      enableDebugLogs: false,
  prisma-client      enableEngineDebugMode: undefined,
  prisma-client      datamodelPath: '/var/www/up-osmp/node_modules/.prisma/client/schema.prisma',
  prisma-client      prismaPath: undefined,
  prisma-client      engineEndpoint: undefined,
  prisma-client      generator:
  prisma-client       { name: 'client',
  prisma-client         provider: 'prisma-client-js',
  prisma-client         output: '/var/www/up-osmp/node_modules/@prisma/client',
  prisma-client         binaryTargets: [],
  prisma-client         previewFeatures: [],
  prisma-client         config: {} },
  prisma-client      showColors: false,
  prisma-client      logLevel: undefined,
  prisma-client      logQueries: undefined,
  prisma-client      flags: [],
  prisma-client      clientVersion: '2.1.3',
  prisma-client      enableExperimental: [] } }  +387ms
  prisma-client { engineConfig:
  prisma-client    { cwd: '/var/www/up-osmp/prisma',
  prisma-client      enableDebugLogs: false,
  prisma-client      enableEngineDebugMode: undefined,
  prisma-client      datamodelPath: '/var/www/up-osmp/node_modules/.prisma/client/schema.prisma',
  prisma-client      prismaPath: undefined,
  prisma-client      engineEndpoint: undefined,
  prisma-client      generator:
  prisma-client       { name: 'client',
  prisma-client         provider: 'prisma-client-js',
  prisma-client         output: '/var/www/up-osmp/node_modules/@prisma/client',
  prisma-client         binaryTargets: [],
  prisma-client         previewFeatures: [],
  prisma-client         config: {} },
  prisma-client      showColors: false,
  prisma-client      logLevel: undefined,
  prisma-client      logQueries: undefined,
  prisma-client      flags: [],
  prisma-client      clientVersion: '2.1.3',
  prisma-client      enableExperimental: [] } }  +891ms
  prisma-client { engineConfig:
  prisma-client    { cwd: '/var/www/up-osmp/prisma',
  prisma-client      enableDebugLogs: false,
  prisma-client      enableEngineDebugMode: undefined,
  prisma-client      datamodelPath: '/var/www/up-osmp/node_modules/.prisma/client/schema.prisma',
  prisma-client      prismaPath: undefined,
  prisma-client      engineEndpoint: undefined,
  prisma-client      generator:
  prisma-client       { name: 'client',
  prisma-client         provider: 'prisma-client-js',
  prisma-client         output: '/var/www/up-osmp/node_modules/@prisma/client',
  prisma-client         binaryTargets: [],
  prisma-client         previewFeatures: [],
  prisma-client         config: {} },
  prisma-client      showColors: false,
  prisma-client      logLevel: undefined,
  prisma-client      logQueries: undefined,
  prisma-client      flags: [],
  prisma-client      clientVersion: '2.1.3',
  prisma-client      enableExperimental: [] } }  +122ms
  prisma-client Prisma Client call:  +348ms
  prisma-client prisma.users.deleteMany({
  prisma-client   where: {
  prisma-client     phone: {
  prisma-client       equals: '+79000110001'
  prisma-client     }
  prisma-client   }
  prisma-client })  +4ms
  prisma-client Generated request:  +0ms
  prisma-client mutation {
  prisma-client   deleteManyusers(where: {
  prisma-client     phone: "+79000110001"
  prisma-client   }) {
  prisma-client     count
  prisma-client   }
  prisma-client }
  prisma-client   +0ms
  prisma-client Prisma Client call:  +2ms
  prisma-client prisma.users.deleteMany({
  prisma-client   where: {
  prisma-client     phone: {
  prisma-client       equals: '+79000110002'
  prisma-client     }
  prisma-client   }
  prisma-client })  +1ms
  prisma-client Generated request:  +0ms
  prisma-client mutation {
  prisma-client   deleteManyusers(where: {
  prisma-client     phone: "+79000110002"
  prisma-client   }) {
  prisma-client     count
  prisma-client   }
  prisma-client }
  prisma-client   +0ms
  prisma-client Prisma Client call:  +1ms
  prisma-client prisma.users.deleteMany({
  prisma-client   where: {
  prisma-client     phone: {
  prisma-client       equals: '+79000110003'
  prisma-client     }
  prisma-client   }
  prisma-client })  +0ms
  prisma-client Generated request:  +0ms
  prisma-client mutation {
  prisma-client   deleteManyusers(where: {
  prisma-client     phone: "+79000110003"
  prisma-client   }) {
  prisma-client     count
  prisma-client   }
  prisma-client }
  prisma-client   +0ms
  plusX Execution permissions of /var/www/up-osmp/node_modules/.prisma/client/query-engine-rhel-openssl-1.0.x are fine  +0ms
  prisma-client Client Version 2.1.3  +234ms
  prisma-client Engine Version e11114fa1ea826f9e7b4fa1ced34e78892fe8e0e  +0ms

@Kalinkin92 Thanks, can you please share your schema and the Prisma client query that fails? A minimal reproduction would help up triage this quickly.

I'm having the same issue, the error only ever occurs when I'm using prisma.disconnect() with Next.js, w/o the error will never show up.
Versions

Name
Version

Node
v12.18.2

OS
darwin

Prisma
2.3.0

Description
Started http server

Logs
  prisma-client {
  prisma-client   engineConfig: {
  prisma-client     cwd: '/Users/redacted/Projects/Personal/redacted/prisma',
  prisma-client     enableDebugLogs: false,
  prisma-client     enableEngineDebugMode: undefined,
  prisma-client     datamodelPath: '/Users/redacted/Projects/Personal/redacted/node_modules/.prisma/client/schema.prisma',
  prisma-client     prismaPath: undefined,
  prisma-client     engineEndpoint: undefined,
  prisma-client     generator: {
  prisma-client       name: 'client',
  prisma-client       provider: 'prisma-client-js',
  prisma-client       output: '/Users/redacted/Projects/Personal/redacted/node_modules/@prisma/client',
  prisma-client       binaryTargets: [],
  prisma-client       previewFeatures: [Array],
  prisma-client       config: {}
  prisma-client     },
  prisma-client     showColors: false,
  prisma-client     logLevel: undefined,
  prisma-client     logQueries: undefined,
  prisma-client     flags: [],
  prisma-client     clientVersion: '2.3.0',
  prisma-client     enableExperimental: [ 'aggregations', 'aggregations' ]
  prisma-client   }
  prisma-client }  +0ms
  plusX Execution permissions of /Users/redacted/Projects/Personal/redacted/node_modules/.prisma/client/query-engine-darwin are fine  +0ms
  prisma-client Client Version 2.3.0  +94ms
  prisma-client Engine Version e11114fa1ea826f9e7b4fa1ced34e78892fe8e0e  +0ms

I'm having the same issue, the error only ever occurs when I'm using prisma.disconnect() (with Next.js in dev mode), w/o the error will never show up.
Versions
Name	Version
Node	v12.18.2
OS	darwin
Prisma	2.3.0
Description
Started http server

Logs
  prisma-client {
  prisma-client   engineConfig: {
  prisma-client     cwd: '/Users/redacted/Projects/Personal/redacted/prisma',
  prisma-client     enableDebugLogs: false,
  prisma-client     enableEngineDebugMode: undefined,
  prisma-client     datamodelPath: '/Users/redacted/Projects/Personal/redacted/node_modules/.prisma/client/schema.prisma',
  prisma-client     prismaPath: undefined,
  prisma-client     engineEndpoint: undefined,
  prisma-client     generator: {
  prisma-client       name: 'client',
  prisma-client       provider: 'prisma-client-js',
  prisma-client       output: '/Users/redacted/Projects/Personal/redacted/node_modules/@prisma/client',
  prisma-client       binaryTargets: [],
  prisma-client       previewFeatures: [Array],
  prisma-client       config: {}
  prisma-client     },
  prisma-client     showColors: false,
  prisma-client     logLevel: undefined,
  prisma-client     logQueries: undefined,
  prisma-client     flags: [],
  prisma-client     clientVersion: '2.3.0',
  prisma-client     enableExperimental: [ 'aggregations', 'aggregations' ]
  prisma-client   }
  prisma-client }  +0ms
  plusX Execution permissions of /Users/redacted/Projects/Personal/redacted/node_modules/.prisma/client/query-engine-darwin are fine  +0ms
  prisma-client Client Version 2.3.0  +94ms
  prisma-client Engine Version e11114fa1ea826f9e7b4fa1ced34e78892fe8e0e  +0ms

not sure about the issue, but you don't need disconnect prisma on nextjs in dev mode. you can use the same instance all the time:
import { PrismaClient } from "@prisma/client"

let prisma: PrismaClient

declare global {
  namespace NodeJS {
    interface Global {
      prisma: PrismaClient
    }
  }
}

if (process.env.NODE_ENV === "production") {
  prisma = new PrismaClient()
} else {
  if (!global.prisma) {
    global.prisma = new PrismaClient()
  }

  prisma = global.prisma
}

export default prisma

Hey @kldzj Did you issue get fixed after doing what @macrozone suggested?

Hey @pantharshit00, indeed this solved my problem. But I still think that the issue itself persists, because Prisma should've worked even after closing the connection, as the docs state that the client will connect itself lazily.

@kldzj If you explicit call disconnect() it won't connect again.
What the docs meant is you don't need to call prisma.connect in order to establish a connection as client will do that lazily. Client will also reconnect if it crashes. Reconnecting after explicitly disconnecting is not intended.

Makes sense, thank you very much for clarifying @pantharshit00

But what about the situation we do need to disconnect?
According to the docs:
"When deploying your application as a long-running process or when running a Node.js script that uses Prisma Client, it's recommended to always explicitly call prisma.$disconnect() in your code even when an exception is thrown as shown below. Otherwise, the Node.js process will not exit automatically."

@alonbilu That is for exiting the process. If you want to connect again, you can just instantiate the Prisma Client again using new PrismaClient

@alonbilu That is for exiting the process. If you want to connect again, you can just instantiate the Prisma Client again using new PrismaClient


Removing "selectable" field from Edge API docs

I don't know if this field was deprecated or removed or other. When I try to use this field in my project, there is no "selectable" field on a FlowEdge.
The codebase also mentions "selectable" in a few other places, so if this change is valid, I don't know if other places need to be updated as well.

yep, thanks! We mixed something up with the v12 docs here.


Leave -Xfrontend in sourcekit compiler arguments

Various requests fail due to the removal of only -Xfrontend and not the
next argument as well. This shouldn't be needed any more, so just remove
this logic entirely.

@swift-ci please test


[chore] Migrate from Core App type to Kuberetes MeshModel

Description
This PR migrates our old core designs to be compatible with MeshModels.
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

we can hold on this PR, I'm adding more designs in this PR.

🚀 Preview for commit 1dcd0876bf5aadb2b88d6e3e9ed63dd491ea128a at: https://6484149101d6012a5e8be585--service-mesh-patterns.netlify.app

🚀 Preview for commit e747c464b210b2f42cf4a78e6978090223441f8d at: https://64854b406038ce515f4f6753--service-mesh-patterns.netlify.app

🚀 Preview for commit cd6adae382177db04a2fd021e6f9e96153ab5958 at: https://64854c59ad38cc52b51d6dc3--service-mesh-patterns.netlify.app

🚀 Preview for commit e00ecf3a19a1fd09b740c1c73f28a84e38221a35 at: https://64855b6bf88be15b39cb22a2--service-mesh-patterns.netlify.app

🚀 Preview for commit 12a9c4b45af9c679f9f24a2a020ca58923f14d97 at: https://64855e1842bbea5bb83ca1a8--service-mesh-patterns.netlify.app

🚀 Preview for commit 2ada0fab27ee51f232ac8f77361f1fb15b2743c7 at: https://648571e618fa9466af4ad929--service-mesh-patterns.netlify.app

@Revolyssup
This PR is ready for review and merge.

@Revolyssup This PR is ready for review and merge.

Have you tested deploying all these?

I was not able to test IstioSpire design as there is some UI bug due to which CustomResources are not being handled properly in UI. // @Abhishek-Kumar09
Rest all designs are deployable.

Also do we really need three different designs for bookInfo app?

As all the three designs have exact same config. So probably we can remove two of them

I was not able to test IstioSpire design as there is some UI bug due to which CustomResources are not being handled properly in UI. // @Abhishek-kumar09
Rest all designs are deployable.

I was not able to test IstioSpire design as there is some UI bug due to which CustomResources are not being handled properly in UI. // @Abhishek-kumar09
Rest all designs are deployable.


next and previous page links don't go where I'd expect

At the bottom of the Day 0 page, the "next page" link goes to the mini reprohack page, not Day 1 like I would have expected.  The previous page on the mini reprohack page goes back to Day 0.
On the main day 1 page, there is no next or previous page link (this applies to the main page for day 2–4 also)
(part of JOSE review here: https://github.com/openjournals/jose-reviews/issues/260)

Yeah, this is not ideal. I tried looking into this and it seems to be a limitation with the apero theme we're using since their example site has the same problems. I opened an issue upstream to see if this is something that can be fixed site-side or if it should be fixed on their end.


Qutebrowser dark mode

Should Stylix apply a dark mode to qutebrowser depending on the polarity of the current theme, or should these settings remain reserved to user-specific configurations? What about using lib.mkDefault?

Seems like a useful feature. This is something we can do for Firefox too.


docker-machine create generic swarm provision gets "Error while starting container: 500 Internal Server Error: oci runtime error"

When tried to provision swarm master over docker-machine with generic driver I get:
docker-machine create -d generic --generic-ip-address <IP_ADDRESS> --generic-ssh-user ubuntu  --generic-ssh-key $HOME/.ssh/id_rsa --generic-ssh-port 22 --swarm  --swarm-master  --swarm-discovery token://d7a3a1d0a97d6456db9ff01e86f63279  pine64-1

Error creating machine: Error running provisioning: Error while starting container: 500 Internal Server Error: oci runtime error: process_linux.go:291: setting cgroup config for ready process caused "failed to write 0 to memory.swappiness: write /sys/fs/cgroup/memory/docker/1d830978a722d854fd8d1dae4ae89617c5c76b936ae6e939d7b127c83227ba25/memory.swappiness: invalid argument"

Everything seems to work fine until swarm provisioning.
In that moment no extra directory exists inside  /sys/fs/cgroup/memory/docker
Otherwise, this other method  seems to work: Initializing swarm from inside a host previously provisioned with docker-machine
docker-machine create -d generic --generic-ip-address <IP_ADDRESS> --generic-ssh-user ubuntu  --generic-ssh-key $HOME/.ssh/id_rsa --generic-ssh-port 22  pine64-1

docker-machine ssh pine64_1

docker swarm init --advertise-addr <IP_ADDRESS>

Swarm initialized: current node (cjp9erfyw8d3wg75mir61jbk3) is now a manager.

To add a worker to this swarm, run the following command:
    docker swarm join \
    --token SWMTKN-1-1w9xpujer7orhc63pv7nye29u1ek8vse2ya4r6j7y8zhpv6ys7-4gce7oh9fzba79ifzini16qbk \
    <IP_ADDRESS>:2377

To add a manager to this swarm, run the following command:
    docker swarm join \
    --token SWMTKN-1-1w9xpujer7orhc63pv7nye29u1ek8vse2ya4r6j7y8zhpv6ys7-2artsveo5wfwa3remvzkhgzrz \
    <IP_ADDRESS>:2377

Anyway this method also have no directory below  /sys/fs/cgroup/memory
Can anyone indicate, if it's missing any requirement component?
Thanks

Sorry don't know, but did you enable memory and swap accounting in /etc/default/grub on your ubuntu docker server?
https://docs.docker.com/engine/installation/linux/ubuntulinux/

Thanks @jamshid even if it was optional it could have sense in my use case, I've tried but still get similar error
Aug 23 15:39:41 pine64-5 kernel: [  717.331861] IPv6: ADDRCONF(NETDEV_CHANGE): docker0: link becomes ready
Aug 23 15:39:41 pine64-5 docker[3793]: time="2016-08-23T15:39:41.38498Z" level=error msg="containerd: start container" error="oci runtime error: process_linux.go:291: setting cgroup config for ready process caused \"failed to write 0 to memory.swappiness: write /sys/fs/cgroup/memory/docker/4e4bdc309a7bd013232f7773186ae56f2efaccb31a141f3d4cff7797237ee311/memory.swappiness: invalid argument\"" id=4e4bdc309a7bd013232f7773186ae56f2efaccb31a141f3d4cff7797237ee311
Aug 23 15:39:41 pine64-5 docker[3793]: time="2016-08-23T15:39:41.408908000Z" level=error msg="Create container failed with error: oci runtime error: process_linux.go:291: setting cgroup config for ready process caused \"failed to write 0 to memory.swappiness: write /sys/fs/cgroup/memory/docker/4e4bdc309a7bd013232f7773186ae56f2efaccb31a141f3d4cff7797237ee311/memory.swappiness: invalid argument\""
Aug 23 15:39:42 pine64-5 kernel: [  718.481819] docker0: port 1(vethbc862a1) entered disabled state
Aug 23 15:39:42 pine64-5 kernel: [  719.141273] docker0: port 1(vethbc862a1) entered disabled state
Aug 23 15:39:42 pine64-5 kernel: [  719.142472] device vethbc862a1 left promiscuous mode
Aug 23 15:39:42 pine64-5 kernel: [  719.142492] docker0: port 1(vethbc862a1) entered disabled state
Aug 23 15:39:43 pine64-5 docker[3793]: time="2016-08-23T15:39:43.293644000Z" level=warning msg="Couldn't run auplink before unmount /var/lib/docker/aufs/mnt/9a6367ab3048120691ed6534b0240c23496d764a083d7acc3d73168876aaaa94: exec: \"auplink\": executable file not found in $PATH"
Aug 23 15:39:43 pine64-5 docker[3793]: time="2016-08-23T15:39:43.317366000Z" level=error msg="Handler for POST /v1.15/containers/4e4bdc309a7bd013232f7773186ae56f2efaccb31a141f3d4cff7797237ee311/start returned error: oci runtime error: process_linux.go:291: setting cgroup config for ready process caused \"failed to write 0 to memory.swappiness: write /sys/fs/cgroup/memory/docker/4e4bdc309a7bd013232f7773186ae56f2efaccb31a141f3d4cff7797237ee311/memory.swappiness: invalid argument\""

This is the content in the FS
ls /sys/fs/cgroup/memory/docker
cgroup.clone_children           memory.kmem.slabinfo                memory.memsw.failcnt             memory.stat
cgroup.event_control            memory.kmem.tcp.failcnt             memory.memsw.limit_in_bytes      memory.swappiness
cgroup.procs                    memory.kmem.tcp.limit_in_bytes      memory.memsw.max_usage_in_bytes  memory.usage_in_bytes
memory.failcnt                  memory.kmem.tcp.max_usage_in_bytes  memory.memsw.usage_in_bytes      memory.use_hierarchy
memory.force_empty              memory.kmem.tcp.usage_in_bytes      memory.move_charge_at_immigrate  notify_on_release
memory.kmem.failcnt             memory.kmem.usage_in_bytes          memory.oom_control               tasks
memory.kmem.limit_in_bytes      memory.limit_in_bytes               memory.pressure_level
memory.kmem.max_usage_in_bytes  memory.max_usage_in_bytes           memory.soft_limit_in_bytes

and the output of the check config script
/usr/share/docker-engine/contrib/check-config.sh
info: reading kernel config from /proc/config.gz ...

Generally Necessary:
- cgroup hierarchy: properly mounted [/sys/fs/cgroup]
- apparmor: enabled and tools installed
- CONFIG_NAMESPACES: enabled
- CONFIG_NET_NS: enabled
- CONFIG_PID_NS: enabled
- CONFIG_IPC_NS: enabled
- CONFIG_UTS_NS: enabled
- CONFIG_DEVPTS_MULTIPLE_INSTANCES: enabled
- CONFIG_CGROUPS: enabled
- CONFIG_CGROUP_CPUACCT: enabled
- CONFIG_CGROUP_DEVICE: enabled
- CONFIG_CGROUP_FREEZER: enabled
- CONFIG_CGROUP_SCHED: enabled
- CONFIG_CPUSETS: enabled
- CONFIG_MEMCG: enabled
- CONFIG_MACVLAN: enabled
- CONFIG_VETH: enabled
- CONFIG_BRIDGE: enabled (as module)
- CONFIG_BRIDGE_NETFILTER: enabled
- CONFIG_NF_NAT_IPV4: enabled (as module)
- CONFIG_IP_NF_FILTER: enabled (as module)
- CONFIG_IP_NF_TARGET_MASQUERADE: enabled (as module)
- CONFIG_NETFILTER_XT_MATCH_ADDRTYPE: enabled (as module)
- CONFIG_NETFILTER_XT_MATCH_CONNTRACK: enabled (as module)
- CONFIG_NF_NAT: enabled (as module)
- CONFIG_NF_NAT_NEEDED: enabled
- CONFIG_POSIX_MQUEUE: enabled

Optional Features:
- CONFIG_USER_NS: enabled
- CONFIG_SECCOMP: enabled
- CONFIG_MEMCG_KMEM: enabled
- CONFIG_MEMCG_SWAP: enabled
- CONFIG_MEMCG_SWAP_ENABLED: enabled
- CONFIG_RESOURCE_COUNTERS: enabled
- CONFIG_BLK_CGROUP: enabled
- CONFIG_IOSCHED_CFQ: enabled
- CONFIG_BLK_DEV_THROTTLING: enabled
- CONFIG_CGROUP_PERF: enabled
- CONFIG_CGROUP_HUGETLB: enabled
- CONFIG_NET_CLS_CGROUP: enabled (as module)
- CONFIG_NETPRIO_CGROUP: enabled (as module)
- CONFIG_CFS_BANDWIDTH: enabled
- CONFIG_FAIR_GROUP_SCHED: enabled
- CONFIG_RT_GROUP_SCHED: missing
- CONFIG_EXT3_FS: missing
- CONFIG_EXT3_FS_XATTR: missing
- CONFIG_EXT3_FS_POSIX_ACL: missing
- CONFIG_EXT3_FS_SECURITY: missing
    (enable these ext3 configs if you are using ext3 as backing filesystem)
- CONFIG_EXT4_FS: enabled
- CONFIG_EXT4_FS_POSIX_ACL: enabled
- CONFIG_EXT4_FS_SECURITY: enabled
- Storage Drivers:
  - "aufs":
    - CONFIG_AUFS_FS: enabled (as module)
  - "btrfs":
    - CONFIG_BTRFS_FS: enabled (as module)
  - "devicemapper":
    - CONFIG_BLK_DEV_DM: enabled (as module)
    - CONFIG_DM_THIN_PROVISIONING: enabled (as module)
  - "overlay":
    - CONFIG_OVERLAY_FS: missing
  - "zfs":
    - /dev/zfs: missing
    - zfs command: missing
    - zpool command: missing

I get also problems when tried with consult:
docker-machine create \
  --driver generic \
  --generic-ip-address <IP_ADDRESS> \
  --generic-ssh-user ubuntu  \
  --generic-ssh-key $HOME/.ssh/id_rsa \
  --generic-ssh-port 22 \
  --swarm --swarm-master \
  --swarm-discovery="consul://$(docker-machine ip mh-keystore):8500" \
  --engine-opt="cluster-store=consul://$(docker-machine ip mh-keystore):8500" \
  --engine-opt="cluster-advertise=eth1:2376" \
  pine64-1

reconfigures my docker daemon, which suddenly dies on host:
ubuntu@pine64-1:~$ sudo systemctl status docker
● docker.service
   Loaded: loaded (/etc/systemd/system/docker.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Fri 2016-08-26 14:10:22 UTC; 6min ago
  Process: 13133 ExecStart=/usr/bin/docker daemon -H tcp://<IP_ADDRESS>:2376 -H unix:///var/run/docker.sock --storage-driver aufs --tlsverify --tlscacert /etc/docker/ca.pem -
 Main PID: 13133 (code=exited, status=1/FAILURE)

Aug 26 14:10:21 pine64-1 systemd[1]: Started docker.service.
Aug 26 14:10:21 pine64-1 docker[13133]: time="2016-08-26T14:10:21.516005000Z" level=info msg="libcontainerd: new containerd process, pid: 13143"
Aug 26 14:10:21 pine64-1 docker[13133]: time="2016-08-26T14:10:21.568138Z" level=error msg="containerd: notify OOM events" error="open /proc/31265/cgroup: no such file o
Aug 26 14:10:22 pine64-1 docker[13133]: time="2016-08-26T14:10:22.949749000Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
Aug 26 14:10:22 pine64-1 docker[13133]: time="2016-08-26T14:10:22.955414000Z" level=fatal msg="Error starting daemon: discovery advertise parsing failed (invalid cluster
Aug 26 14:10:22 pine64-1 systemd[1]: docker.service: Main process exited, code=exited, status=1/FAILURE
Aug 26 14:10:22 pine64-1 systemd[1]: docker.service: Unit entered failed state.
Aug 26 14:10:22 pine64-1 systemd[1]: docker.service: Failed with result 'exit-code'

but it seems to me that it has something to do with the migration to new 1.12.x swarm mode
After I created a swarm cluster using the env of each host and node syntax work's perfect
Also I readed this:

In the past Docker Machine allowed to set up a Docker Swarm cluster with a number of options for the ‘create’ subcommand.

With Docker 1.12 we now how a much more powerful version of Swarm that is directly integrated into Docker. This feature is called Docker Swarm Mode

source: hypriot blog
So I will take a try


chore: 🤖 remove alias of Atom

Summary

Test Plan

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

this refactor may wait until 0.5 is merged to main, otherwise it will cause huge conflict when we merge 0.5 to main


Weaver doesn't auto fill OnSerialize/OnDeserialize for classes without base.Serialize function

struct: works

class: doesnt work

error is caused by the MessageClassProcessor.CallBase function

strange,  is there an error message?

you should have an override keyword there like this:
        class ArrayIntMessage : IMessageBase
        {
            public int[] array;

            // mirror will automatically implement these empty methods
            public override void Serialize(NetworkWriter writer) { }
            public override void Deserialize(NetworkReader reader) { }
        }


Admin tab

Added an admin tab to display temperature and attendance. Also added another constraint for the temperature to be between 25.0°C and 41.0°C.

Wow looks good! Excited to see how the temperature and attendance display will look like. Good job Yuan Yu!

I see there were significant improvements made to the feature since last review, and the admin details can now be displayed upon command. Great work Freddy! :)


I use the demo--nginx in dockerhub deploy on microk8s, but can't access from browser

I just use microk8s to deploy a demo, and meet some problem:
1、microk8s use k8s 1.16:
2、nginx:

3、deploy like this:

4、pod is ok:

5、but can't access:
I am in the node use browser:
https://localhost:30080
https://<IP_ADDRESS>:30080

@andyzheung, you are accessing it as https.  I think you mean http, i.e. unsecure.

I use http, still not ok.@balchua

I think it has something to do with ipv6 in your system.  There is an existing issue https://github.com/ubuntu/microk8s/issues/498#issuecomment-501088380 cud be related to ur case.

I try , may be I did't use ipv6.

You can use kubectl exec to get into the pod.  Could it be that the nginx container is uaing 8080 port instead of 80?

ould it be that the nginx container is uaing 8080
I enter into the container:

I change my yaml port to 8080, that is ok.Thank you so much.
@balchua


Pretrained model

Hi!
I would like to ask if you can share the pre-trained model for Jetson with the calibration file. I mean if you can share a .zip with the model trained by you and I can prune in my side.
The main problem I have is not achieving the same values in mAP even if I retrain.
Thanks in advance!
All the best,
Franco.

We cannot provide pre-trained model for face-mask-detection.
What is mAP you are achieving currently? What are the errors? Based on you batch size and hyper-parameters, your mAP might drop.

Okay, I'll try with a bigger batch size because I trained with a GTX1070
and batch size 8. Probably that's the problem with mAP in no-masked. By the
way the error with a bigger batch size was an allocation memory error.
So I think if you can't provide the trained model maybe you can help me
with the optimal parameters for an i7 cpu + 16 gb ram + gtx1070. Another
way I think it's possible is using Google Collab or allocating a vm in
azure or aws to use a Tesla V100.
Any suggestions or help is appreciated.
Thanks in advance for your help.
On Thu, Sep 17, 2020, 1:25 PM Amey Kulkarni<EMAIL_ADDRESS>wrote:

We cannot provide pre-trained model for face-mask-detection.
What is mAP you are achieving currently? What are the errors? Based on you
batch size and hyper-parameters, your mAP might drop.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA-AI-IOT/face-mask-detection/issues/12#issuecomment-694345971,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFPNLORJ4QZDL7ZVONO73YTSGIZ7VANCNFSM4RQSKYFQ
.

By the way the error with a bigger batch size was an allocation memory error.
With 24 Batch Size, you will need about 15 GB memory, whereas once you prune model (as in git ~12% prune ratio) you can be okay with 8GB memory.

I have personally tried AWS and used Tesla V100 for this task and it worked out pretty well. I have not tried any other Cloud instances yet so cannot comment on it.

Okay, but I didn't get a point. When you say about prune the model I think you're talking about infer. But my problem is during the training or I misunderstand you sorry for that. Again, if I can train with my own GPU to move then to my AGX Xavier it will be great. Just to put some context, I'm trying to learn how to build the workflow pipeline with my personal computer and the Xavier before moving to cloud. Following the idea of this repo: If I can train the model in my GTX1070 and move the model to infer in the Jetson then mission completed by now. After that I can think in a way to deploy via docker or something like that.
By the other hand, if my GPU memory is too low for this experiment, I think it can be updated as a requirement for the git to prevent future problems with someone else. As a solution for this, I would like to ask if you have any idea (theorical) about if it's possible to make this process using google collab, after all there are GPUs enabled runtimes with Teslas T4 that can be very usefull for my purpouse.

I feel it should be doable even with GTX1070, you need to experiment with other hyper-params as well when you reduce batch size,  in detectnet_v2_train_resnet18_kitti.txt look at model_config . I had better luck with even batch size 16 previously. As batch size reduces your training time will also increase.

When you say about prune the model I think you're talking about infer.

In TLT, we do training first and once we achieve satisfactory accuracy, we prune model and retrain the pruned model.  See 3 to 8 steps on TLT workflow

Oh I got it now! I'll play a little with that this weekend and update this
if I succeed.
Thanks for your help!
On Thu, Sep 17, 2020 at 2:38 PM Amey Kulkarni<EMAIL_ADDRESS>wrote:

I feel it should be doable even with GTX1070, you need to experiment with
other hyper-params as well when you reduce batch size, in
detectnet_v2_train_resnet18_kitti.txt look at model_config
https://github.com/NVIDIA-AI-IOT/face-mask-detection/blob/master/tlt_specs/detectnet_v2_train_resnet18_kitti.txt#L73
. I had better luck with even batch size 16 previously. As batch size
reduces your training time will also increase.
When you say about prune the model I think you're talking about infer.
In TLT, we do training first and once we achieve satisfactory accuracy, we
prune model and retrain the pruned model. See 3 to 8 steps on TLT workflow
https://github.com/NVIDIA-AI-IOT/face-mask-detection#nvidia-transfer-learning-toolkit-tlt-training-flow-
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA-AI-IOT/face-mask-detection/issues/12#issuecomment-694390374,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFPNLOTTSCXSTA25RMKWMCDSGJCRVANCNFSM4RQSKYFQ
.

Okay, working around with the GTX1070 I didn't find a good result in any possible way, I don't know exactly why. My suspicious points to my dataset and probably the batch size, but without luck to find a scientific answer.
So, to avoid this waste of time with my computer, I used a Tesla V100 from a VM, and here comes the interesting part, I built again the dataset as you suggested in the repo, but now the unpruned model hits:
Validation cost: 0.001015
Mean average_precision (in %): 70.6842

class name      average precision (in %)
------------  --------------------------
mask                             81.1487
no-mask                          60.2197

Median Inference Time: 0.005730

after this I think: Okay, let's try retraining the pruned model with the default settings from the notebook and I get this:
Validation cost: 0.000977
Mean average_precision (in %): 65.6631

class name      average precision (in %)
------------  --------------------------
mask                             60.5674
no-mask                          70.7589

Median Inference Time: 0.003357

I think I've misunderstood something, so I decided to make some changes.
First, I changed the amount of the images for each class to 12000 and the results are quite better:
Validation cost: 0.000234
Mean average_precision (in %): 80.8005

class name      average precision (in %)
------------  --------------------------
mask                             84.494
no-mask                          77.1069

Median Inference Time: 0.005580

then I retrained with a tweak in the prune step, I changed this cell:
!tlt-prune -m $USER_EXPERIMENT_DIR/experiment_dir_unpruned_12k/weights/resnet18_detector.tlt \
           -o $USER_EXPERIMENT_DIR/experiment_dir_pruned_12k_test2/resnet18_nopool_bn_detectnet_v2_pruned.tlt \
           -eq union \
           -pth 0.12 \
           -k $KEY

I changed the -pth from  0.8 to 0.12 and I get this result:
=========================

Validation cost: 0.000245
Mean average_precision (in %): 80.6542

class name      average precision (in %)
------------  --------------------------
mask                             84.6146
no-mask                          76.6938

Median Inference Time: 0.004232

At this point I don't know exactly what to change or where point the effort to get closer to your results, If you can help me with this I'll really appreciate your help.
If it's needed I can share my TFRecords to reproduce the training.
Thanks for your time!
NOTE: You should add a warning in the repo about the changes in the path in the config files for those who like me doesn't know much about TLT and stuff like that

@fiv21 Thanks for working through this and recommendations. I am trying to add more detailed steps and examples with sample video soon in upcoming weeks, if that helps.

That's great!
It could work to understand a little bit more about this. It's interesting, because, maybe it's a base example for other cases, perhaps to look if a person uses a security helmet or similar.
I close the issue, waiting for new info in the repo to improve the results or the right way to achieve the same accuracy.
Thanks in advance!