How can I update Visual Studio's version of TypeScript?

I bothered @DanielRosenwasser with something similar on gitter yesterday, but I want to reduce the problem to something even easier.
I want to upgrade to the latest version of TypeScript - 1.7.3 - but I'm actually not sure how I can do that when using Visual Studio, which was installed with an older version. It seems like I must be somewhere around TypeScript 1.6, as both the exponentiation operator and this typing don't work.
What I've tried to do to upgrade my version:

Follow the VS2015 download on http://www.typescriptlang.org/

It claims that TypeScript is already installed and there's nothing to do. Just for fun, I tried to remove it and reinstall it anyways. No luck.

npm install -g typescript

This had no effect, as far as I can tell.

npm install typescript

Also did nothing.

Activated Dev Mode: https://github.com/Microsoft/TypeScript/wiki/Dev-Mode-in-Visual-Studio

This stopped the red squiggles from appearing under my exponentiation operator and this type! Promising! However, when I try to compile, I get errors when I try to use features from 1.7 anyways. It's as if Visual Studio is using two separate compilers.
Thanks for your help.

TypeScript 1.7 is part of VS 2015 Update 1, please install VS 2015 Update 1.
If you enabled DevMode, it will always pick the files from this location, so if you are not doing active development on the TypeScript repo, i would disable it.

That is very helpful, thank you! Though I found it strange that I couldn't find a link to that anywhere on the TypeScript site.
Is there a way by which I can upgrade Visual Studio to the next version of TypeScript (that is, the latest off this repository)?

There are two pieces to the typescript plugin in VS; 1. The language service, this is your completion, colorization, signature help, etc.. You can update to the latest using the DevMode as you alluded to earlier. 2. The MSBuild integration pieces, this includes target file, tasks dll and the command line compiler executable (tsc.exe), you can update the manually, but there is nothing like the DevMode currently. We should have a nugget package available in the upcoming weeks, and that should making updating the MSBuild integration and command line compiler straight forward.

hey @mhegazy I seem to be stuck at TypeScript <IP_ADDRESS> (according to "About Microsoft Visual Studio") and I'm not sure how to update it using DevMode. I flollowed this and created the EnableDevMode key with a value of 1, but just don't see how that helps me get TypeScript to version 1.8.6. Maybe this thread is too old or maybe I just need directions?

this thread is probably not the same issue you are running into TS 1.8.34 is the latest TS 1.8.* release. it is the same compiler version as 1.8.10, but some updated VS plugin code, hence the higher version. so having TS 1.8.34 means you are all set with TS 1.8.

Hmm... that throws me off because I've come across a number of forum posts saying that updating to 1.8.6 helped resolve the issue with compileOnSave not working. Plus, when I look here it says <IP_ADDRESS>...
In any case, I've tried everything and can't get VS to compile my typescript unless I do a full rebuild after every change. I've scoured the forums and nothing has really helped. Oddly, it worked again for a little while after I switched to my new dev. machine but then stopped working again. I thought that was fixed? I have tsconfig compileOnSave set to true and have tried just about every other tweak or update I've found online. I can't believe it's still not working.

can you log a new issue, and give us some more information about what you are running into to help diagnose the problem?

Yeah I'll do that if I should. I've hesitated to log an issue because lots of very closely related, though not identical, issues have been logged.

I'm having the exact same issue. The tsconfig page indicates that compileOnSave is avaiable for typescrip TypeScript 1.8.4 and above, but my about page also indicates 1.8.35. Visual Studio doesn't compile the files unless I remove the tsconfig.json file. Seems to be the exact same thing as you, Methodician.

Hey PEsteves8! I'm onto a new project now and it seems to be working
(though I'm working with .net 4.x rather than core ATM) but it was working
to start last time too, then stopped unexpectedly. I was wondering if you
could log an issue? @Mohamed Hegazy suggested I do but I have never before
and not sure where to start and having trouble finding the time to research
the process...
Thanks!
On Fri, Jul 22, 2016 at 8:39 AM PEsteves8<EMAIL_ADDRESS>wrote:

I'm having the exact same issue. The tsconfig page indicates that
compileOnSave is avaiable for typescrip TypeScript 1.8.4 and above, but my
about page also indicates 1.8.35. Visual Studio doesn't compile the files
unless I remove the tsconfig.json file. Seems to be the exact same thing as
you, Methodician.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TypeScript/issues/5995#issuecomment-234577887,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQPtqRpFRU8GNX-HqG14Wv-UZmqMttbks5qYOQfgaJpZM4GxOLc
.

After multiple reinstalls/repairs, cache and temp folder deletions, things eventually started working, after I kind of installed Typescript 2.0 beta. I think. So problem solved.

Mysterious bugs... A bit frustrating when noone can ever quite put their
finger on the root cause of a common problem but that's modern programming
for 'ya. Vast code bases with their own quirks.
On Wed, Jul 27, 2016 at 3:42 AM PEsteves8<EMAIL_ADDRESS>wrote:

After multiple reinstalls/repairs, cache and temp folder deletions, things
eventually started working, after I kind of installed Typescript 2.0 beta.
I think. So problem solved.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TypeScript/issues/5995#issuecomment-235550269,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQPtqOdeckbvqcI5xpjCIs8M5UVyd3vks5qZzX_gaJpZM4GxOLc
.

I guess the issue is that there are multiple tsc.exe on the path and the first one is the old one (in my case 1.0.3):
PS C:\Users\moozz_000> where.exe tsc
C:\Program Files (x86)\Microsoft SDKs\TypeScript\1.0\tsc.exe
C:\Program Files (x86)\Microsoft SDKs\TypeScript\1.0\tsc.js
C:\Users\moozz_000\AppData\Roaming\npm\tsc
C:\Users\moozz_000\AppData\Roaming\npm\tsc.cmd

When you type just tsc the first is matched:
PS C:\Users\moozz_000> tsc --version
Version <IP_ADDRESS>
PS C:\Users\moozz_000> tsc.cmd --version
Version 2.0.3

I updated my VS C:\Program Files (x86)\Microsoft SDKs\TypeScript from 1.8 -> 2.0 via the Angular's VISUAL STUDIO 2015 QUICKSTART,
prereq #4    -->   Download and install TypeScript 2.0 for Visual Studio 2015


Disable rounding for float conversion

Fixes #2336. Currently running into the same problem as https://github.com/calyxir/calyx/pull/2335#issue-2636162273
@ekiwi how did you fix it on the Rust side?

I am currently getting this output:
{
  "inp": [
    15.123900413513184
  ],
  "out": [
    15.123900413513184,
    0.5600000023841858
  ]
}

Which might be fine since that is the closest representable number from the input files? Thoughts @ekiwi?

how did you fix it on the Rust side?

I fixed it by printing out the f32 instead of the f64

Stackoverflow seems to suggest using numpy to work with single precision float. However, it is still unclear to me whether numpy can actually print 32-bit floats.
https://stackoverflow.com/questions/2232362/correct-way-to-emulate-single-precision-floating-point-in-python


Add hasSecond to LocalTime assertions

Check List:

Fixes partially #2541
Unit tests : YES
Javadoc with a code example (on API only) : YES
PR meets the contributing guidelines

Integrated thanks @maxwell142857!

with pleasure~


convertFromHTML merged  tags into one  tag

I'm having some problems with the convertFromHTML, it doesn't render the right data as I expected.
For example,
html:
Paragraph 1
,
Paragraph 2
,
Paragraph 3

editorState:
EditorState.createWithContent(ContentState.createFromBlockArray(convertFromHTML(_html_)), Decorator)
The html at this stage render correctly as expected.

Paragraph 1
Paragraph 2
Paragraph 3

However, when I save data and refresh the page, it displays as below (all in one line)

Paragraph 1 Paragraph 2 Paragraph 3

Then, I clicked on the editor and suddenly the html turned into

Paragraph 1 Paragraph 2 Paragraph 3

Does anyone has similar problem like this? Thanks.

What do you mean, "when I save data"?

@spicyj, sorry for my uncleared explanation.
here is an example,
html:
A

B

C

Basically the result of convertFromHTML(html) is
Object {key: "1lic8", type: "unstyled", text: "A B C", characterList: Array[5], depth: 0}
I expected to get 3 ContentBlock objects as there are 3 paragraphs.
Object {key: "d1m9h", type: "unstyled", text: "A", characterList: Array[1], depth: 0}
Object {key: "4qnh7", type: "unstyled", text: "B", characterList: Array[1], depth: 0}
Object {key: "457nc", type: "unstyled", text: "C", characterList: Array[1], depth: 0}

Thanks, I can repro this.

@spicyj I can take a look at this this week if no one else is.

@CLowbrow Go for it.

@CLowbrow Go for it.

Is there currently a workaround for this?

Workaround: convertFromHTML takes as its optional third argument a "block render map" -- I believe if you specify a map that includes the
 tag as output then paragraphs will be mapped to that block type.
https://github.com/facebook/draft-js/blob/b5d87c2765f7a6e72cd0ae6618682e8d0c5556a3/src/model/encoding/convertFromHTMLToContentBlocks.js#L511

Thanks @spicyj

I tried applying the workaround but without success. Did you figure out a solution @abhoopathy ?
I passed a render map like that:
const blockRenderMap = Immutable.Map({
    'paragraph': {
        element: 'p'
    },
    'unstyled': {
        element: 'p'
    }
 });

Thanks!

Same here, couldn't get that render map to work correctly.  tags just kept getting combined.
My workaround was to replace  tags with  tags in my HTML. Obviously this workaround only works if you have access to the input HTML.

Doesn't seem to work for me either.
I need to customize a convertFromHTML to handle non standard blocks. By 'non standard' I mean some block type which is not listed here.
My plan was to use custom blockRenderMap. But it seems that Draft simply ignores any 'non-standard' record in the blockRenderMap you provide as 3rd arg to convertFromHTML
Is it a bug?

OK. Seems
 tags wont work in DraftJS. Maybe I was doing smth wrong, but it looks DraftJS wraps any text contents into div -> span and then wraps this into whatever element you provide in your blockRenderMap. Since
 element supports only Phrasing content children it throws violation error since
 cannot be a child of
.

@jbrozena22 I tried same approach, but it only worked in case all blocks were
. If there was at least some known block right after
, say
 it combined all
's into one
 with text being concatenated. My use case was copy/pasting content from other editor (Medium).
My 'workaround' was to replace all
 tags with
ones. And then add section to blockRenderMap

@N1kto this workaround works but has a severe drawback: deleting section element requires two backspace ...
there's a function called getBlockMapSupportedTags in convertFromHTMLToContentBlocks.js, should we remove the filter?

@NdYAG well, haven't even noticed that drawback to be honest. Comparing to initial problem that's a minor issue, at least for me.
Not sure that exactly getBlockMapSupportedTags causes blocks combining, since it only returns a list of supported tags, joinChunks seems to be responsible for that. But removing the filter within getBlockMapSupportedTags should make it  possible to map
 elements (or any other) to divs within blockRenderMap, e.g.
Draft.DefaultDraftBlockRenderMap.merge(Immutable.Map({
  p: {
    'element': 'div'
  }
}))

However don't know what would be other consequences of doing that - it has been added there for some reason. Also it might be overwritten by default unstyled record - since it also maps to div element.

This worked for me:
const blockRenderMap = DefaultDraftBlockRenderMap.set('p', { element: 'p' });
const blocksFromHTML = convertFromHTML(html, getSafeBodyFromHTML, blockRenderMap)
  .map(block => (block.get('type') === 'p' ? block.set('type', 'unstyled') : block));
const contentState = ContentState.createFromBlockArray(blocksFromHTML);

@davidchang you've closed both this task and #757 (where you said you were closing that one in favor of this one) and there is no consolidated task created (at least not referenced). Is it intentional?

This issue is still open.

@spicyj @davidchang I am sorry, must have overlooked.

@tomconroy Can you tell me please what getSafeBodyFromHTML looks like? Without that, browser shows an error DOMBuilder is not a function

@ZeroCho Maybe you can use a jsdom to do this behavior on the server. Like this:
function getSafeBodyFromHTML(html) {
    var doc = jsdom(html);
    var window = doc.defaultView;
    global.HTMLElement = window.HTMLElement;
    global.HTMLAnchorElement = window.HTMLAnchorElement;
    return window.document.body;
}

@tomconroy @landrade
Oh oh, never mind. I found out that I can import getSafeBodyFromHTML from 'draftjs'.
Thanks for helping me!

@tomconroy I got this to work with your workaround however it seems to not take into account multiple line breaks (eg.  would just be eliminated for some reason) Is there another option we can pass to the block render map to enable draftjs to take into account line breaks as well?

I just realised this is particularly problematic when copy-pasting Draft.js images from one editor to another. If the unstyled blocks in-between the images are empty, then they will be removed in the new editor... and it becomes impossible to select the images to remove them or add content before/after.
Here's an example from my demo site:

In that example, the content changes from unstyled, atomic, unstyled, atomic, unstyled to atomic, atomic which makes it completely impossible to use the editor.

Here is another illustration from Facebook Notes:

If there was a viable workaround for this, out of all place I imagine it would have one.


[WIP] Feature/dusk tests

don't merge this yet fam

@arcdigital Do you have any Dusk tests so far to push? Otherwise I think we can merge this and then just start adding Dusk tests for all of the pages and features.


Make RtagsFindSymbols use tab-completion

RtagsFindSymbols uses rc -F, which finds symbols based on exact name match. It would be useful to be able to search based on prefix. rc -S lists symbols starting with its argument. It would be useful to make RtagsFindSymbols tab-complete based on the result of rc -S.

AFAIU, the merged change doesn't completely corresponds to the requested feature. @petersohn, correct me if I wrong, you want to be able to auto complete RtagsFindSymbols arguments by intermediately invoking rc -S?

I'm sorry, I've reread the docs for customlist completion and it looks like it indeed matches the desired behavior.

Yes, it seems to do what I need.


about data processing

When processing ct data, run "nnUNet_convert_decathlon_task -i ./DATASET/AMOS22_raw/AMOS22_raw_data/Task01_AMOS_CT
nnUNet_plan_and_preprocess -t 1", you will get two files: nnUNetData_plans_v2.1_stage0 and nnUNetData_plans_v2.1_stage1, but I found that there are 600 files in nnUNetData_plans_v2.1_stage0, including 300 .npz files and 300 .pkl files, which is consistent with the number of CT images, but there are only 326 files in total in nnUNetData_plans_v2.1_stage1. Is this normal?

I think something is wrong because I processed it twice and found that the number of data inside the nnUNetData_plans_v2.1_stage1 folder is different, but still not 600 files

I did it again from scratch and now the data seems correct


Added ExtraTreesRegressor

Fixes #12

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

Codecov Report

Merging #91 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #91   +/-   ##
=======================================
  Coverage   92.80%   92.81%
=======================================
  Files          16       16
  Lines        1001     1002    +1
  Branches      161      161
=======================================
+ Hits          929      930    +1
  Misses         44       44
  Partials       28       28

Flag
Coverage Δ

#unittests
92.81% <100.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

hummingbird/ml/supported.py
77.14% <ø> (ø)

...ummingbird/ml/operator_converters/decision_tree.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Can you please add few tests? Something along the line RandomForestRegressor.

I have added the tests. Please let me know if any other changes/additions have to be made.

Thanks @mmbhatk!


ExtraSpacing does not report unnecessary alignment spacing

While investigating #3801 I spotted the following behaviour of ExtraSpacing (with AllowForAlignment: true):
Expected behavior
> cat --number foo.rb
     1	# bad - additional spacing is unnecessary (also bad for do blocks, or a mix)
     2	[1, 2].any?  { 1 }
     3	[3, 4].any?  { 1 }
     4
     5	# ok - multiple lines are aligned even if the first pair appear to have
     6	# unnecessary spacing
     7	[1, 2].any?   { |x| x > 1 }
     8	[1, 2].any?   { |x| x > 1 }
     9	[1, 2].select { |x| x > 1 }
    10
    11	# bad - additional alignment spacing is unnecessary
    12	foo(1, 2,  3)
    13	foo(4, 2,  3)
> rubocop --except Encoding,FrozenStringLiteralComment foo.rb
Inspecting 1 file
C

Offenses:

foo.rb:2:12: C: Unnecessary spacing detected.
[1, 2].any?  { 1 }
           ^
foo.rb:3:12: C: Unnecessary spacing detected.
[3, 4].any?  { 1 }
           ^
foo.rb:12:10: C: Unnecessary spacing detected.
foo(1, 2,  3)
         ^

foo.rb:13:10: C: Unnecessary spacing detected.
foo(4, 5,  6)
         ^
1 file inspected, 4 offenses detected

Actual behavior
> rubocop --except Encoding,FrozenStringLiteralComment foo.rb
Inspecting 1 file
.

1 file inspected, no offenses detected

RuboCop version
$ rubocop -V
0.39.0 (using Parser <IP_ADDRESS>, running on ruby 2.2.3 x86_64-darwin14)

I think it might be a little tricky to fix, since a glance at the source code suggests ExtraSpacing only considers pairs of lines, but the unnecessary spacing of a group of lines would (I think) require checking all the lines before determining if extra spaces are present on any.
N.B. if I comment lines 3 and 13, I do get the correct warnings - so it's the effect of  AllowForAlignment: true for consecutive lines that is the issue:
rubocop --except Encoding,FrozenStringLiteralComment foo.rb
Inspecting 1 file
C

Offenses:

foo.rb:2:12: C: Unnecessary spacing detected.
[1, 2].any?  { 1 }
           ^
foo.rb:12:10: C: Unnecessary spacing detected.
foo(1, 2,  3)
         ^

1 file inspected, 2 offenses detected

This is what happens in #3081, I don't remember if I had AllowForAlignment or not


Fix broken layout of Table of Contents

Fixes broken layout of ToC in some pages.
Example in JSX page.

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

Thanks!


fix: Merge avaiable modules correctly #15173

What kind of change does this PR introduce?
fix #15173

Did you add tests for your changes?
yes

Does this PR introduce a breaking change?
No, just a bug fix.

What needs to be documented once your changes are merged?
Nothing.

thanks for your appreciation

Thanks


chore: add badge for flame slack channel

A badge for flame slack channel is added in the README.md file.

Codecov Report

Merging #291 (58d73e4) into main (f47b29f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #291   +/-   ##
=======================================
  Coverage   15.10%   15.10%
=======================================
  Files          48       48
  Lines        2820     2820
=======================================
  Hits          426      426
  Misses       2376     2376
  Partials       18       18

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


TypeError: expected str, bytes or os.PathLike object, not NoneType

RUNDIR: runs/fcn8s_pascal/68527
Traceback (most recent call last):
File "train.py", line 229, in
train(cfg, writer, logger)
File "train.py", line 49, in train
augmentations=data_aug,
File "/home/lxc/PythonProjects/pytorch-semseg-master/ptsemseg/loader/pascal_voc_loader.py", line 73, in init
self.setup_annotations()
File "/home/lxc/PythonProjects/pytorch-semseg-master/ptsemseg/loader/pascal_voc_loader.py", line 199, in setup_annotations
path = pjoin(sbd_path, "dataset/train.txt")
File "/home/lxc/anaconda3/lib/python3.7/posixpath.py", line 80, in join
a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not NoneType

2 possibilities of error
1)
I think the issue might  be you didn't add sdb_path in config file properly ?
if not add sdb_path in the config file

The data-set you have, does it contain sbd data as well ?

thank you @pvbhanuteja
The reason for the error is the lack of code：
train.py
def train(cfg, writer, logger):
sbd_path = cfg["data"]["sbd_path"]
t_loader = data_loader(
sbd_path,

@pvbhanuteja  thank you !

@li0128 - how did you solve the problem?
I am getting a similar error:
Traceback (most recent call last):
  File "/home/admin-ata168/pytorch-semseg/pytorch-semseg/validate.py", line 167, in <module>
    validate(cfg, args)

  File "/home/admin-ata168/pytorch-semseg/pytorch-semseg/validate.py", line 42, in validate
    img_size=(cfg["data"]["img_rows"], cfg["data"]["img_cols"]),

  File "/home/admin-ata168/pytorch-semseg/pytorch-semseg/ptsemseg/loader/pascal_voc_loader.py", line 73, in __init__
    self.setup_annotations()

  File "/home/admin-ata168/pytorch-semseg/pytorch-semseg/ptsemseg/loader/pascal_voc_loader.py", line 199, in setup_annotations
    path = pjoin(sbd_path, "dataset/train.txt")

  File "/home/admin-ata168/anaconda3/envs/pytorch-semseg/lib/python3.6/posixpath.py", line 80, in join
    a = os.fspath(a)

TypeError: expected str, bytes or os.PathLike object, not NoneType

My config file:
model:
    arch: pspnet
data:
    dataset: pascal
    train_split: train_aug
    val_split: val
    img_rows: 'same'
    img_cols: 'same'
    path: /home/admin-ata168/pytorch-semseg/datasets/VOC/VOCdevkit/VOC2012/
    sbd_path: /home/admin-ata168/pytorch-semseg/datasets/VOC/benchmark_RELEASE/
training:
    train_iters: 300000
    batch_size: 1
    val_interval: 1000
    n_workers: 16
    print_interval: 50
    optimizer:
        name: 'sgd'
        lr: 1.0e-10
        weight_decay: 0.0005
        momentum: 0.99
    loss:
        name: 'cross_entropy'
        size_average: False
    lr_schedule:
    resume: /home/admin-ata168/pytorch-semseg/pretrained_models/pspnet_101_pascalvoc.pth

I downloaded the dataset from:
PASCAL devkit 2012 [Here] and the sbd dataset [Here]
I am using the pretrained model shared here

@mvcaro Do you solve the problem?  I am getting a similar error.

I got the same issue;
I did also mention the sbd_path in the config file.
pascal_voc_loader can get the sbd_path keyword
https://github.com/meetshah1995/pytorch-semseg/blob/801fb200547caa5b0d91b8dde56b837da029f746/ptsemseg/loader/pascal_voc_loader.py#L47
indeed, in the Readme it is mentioned   we can provide  <dataset_keyarg1>:  in the config file. But I do not see where in the training script these additional key:value  arguments are provided to the data_loader
https://github.com/meetshah1995/pytorch-semseg/blob/801fb200547caa5b0d91b8dde56b837da029f746/train.py#L44

@mvcaro @wackxu @jeremyfix
The code in  NO.47 line, "sbd_path=None" was written in the "pascal_voc_loader" file, if you download the project from the github directly. You should code it like this,  sbd_path="/../../../benchmark_RELEASE/(the path your saved) ".

@mvcaro Do you solve the problem? I am getting a similar error.

2 possibilities of error
1)
I think the issue might be you didn't add sdb_path in config file properly ?
if not add sdb_path in the config file
The data-set you have, does it contain sbd data as well ?

dear i'm getting the same error. i don't have sbd data set the folder is bIank. I am running this code on windows 10. i don't know how to define correct path.
runfile('F:/Reasearch/Codes/pytorch-segnet/train.py', wdir='F:/Reasearch/Codes/pytorch-segnet')
RUNDIR: runs\fcn8s_pascal\34014
Traceback (most recent call last):
File "", line 1, in
runfile('F:/Reasearch/Codes/pytorch-segnet/train.py', wdir='F:/Reasearch/Codes/pytorch-segnet')
File "C:\ProgramData\Anaconda3\lib\site-packages\spyder_kernels\customize\spydercustomize.py", line 786, in runfile
execfile(filename, namespace)
File "C:\ProgramData\Anaconda3\lib\site-packages\spyder_kernels\customize\spydercustomize.py", line 110, in execfile
exec(compile(f.read(), filename, 'exec'), namespace)
File "F:/Reasearch/Codes/pytorch-segnet/train.py", line 229, in
train(cfg, writer, logger)
File "F:/Reasearch/Codes/pytorch-segnet/train.py", line 49, in train
augmentations=data_aug,
File "F:\Reasearch\Codes\pytorch-segnet\ptsemseg\loader\pascal_voc_loader.py", line 73, in init
self.setup_annotations()
File "F:\Reasearch\Codes\pytorch-segnet\ptsemseg\loader\pascal_voc_loader.py", line 199, in setup_annotations
path = pjoin(sbd_path, "dataset/train.txt")
File "C:\ProgramData\Anaconda3\lib\ntpath.py", line 76, in join
path = os.fspath(path)
TypeError: expected str, bytes or os.PathLike object, not NoneType

what can I do if I do not add sbd_path?


Support claims with "." character in them

Currently we can't support these because of our glom usage to look up/extract the claims. Meaning, claims/userinfo like:
{
    "foo.bar": "unique-user-id!"
}

cannot be processed, the identity provider needs to remap this.

I am picking this up, can tackle this in a backwards compatible way :)


View statistics of algorithm's requests in a table view

As a HKube manager I want to be able to view statistics of algorithm's requests in a table view, so that I can analyze the usage of HKube and hence its sizing.
User Story implementation details:

_statistics of algorithm's requests means: _
1.1. Average number of algorithm's requests per minute in the chosen time frame.
1.2. Maximum number of algorithm's requests in the chosen time frame.
1.3. Each of the statistics information will be calculated separately per algorithm type.
Table view is detailed in #107

The user story was cancelled


A.16.IR20.IR21.ds.bounds

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
How can you sufficiantly test a bounding box which should be as small as possible if there is no clear definition of what that means? Additionally, this is not considered as crucial for interoperability.
Proposed Change
Remove IR 20 and use only IR 21.

Proposed Resolution
Agreed with the proposed resolution.
Same resolution applies to issues #67 and #70


Change target framework from of 4.5.1 to 4.5

You seams to me not using any of .net 4.5.1 specific feature. So could you please change target framework from of 4.5.1 to 4.5, so exiting project targeting just 4.5 could consume nuget.

Hello, is this still relevant?

Considering Microsoft has ended support for 4.5 and 4.5.1, probably not.
Source https://blogs.msdn.microsoft.com/dotnet/2015/12/09/support-ending-for-the-net-framework-4-4-5-and-4-5-1/


Move the host field to a location in line with ECS or make it configurable

This is related to #40, but not quite the same request.
Please move the host to destination.ip, to be better aligned with ECS and to avoid clashing with host being an object in more and more situations (including, but not limited to ECS).
If that's a no-go, perhaps a first step would be to make the destination field for this information configurable?

The correct ECS field would be source.ip, but I think moving all input-provided fields under [@metadata] makes more sense. This is also more in line with plugins like logstash-input-kafka and logstash-input-rabbitmq.

Yes I could go with that. My goal here is mainly to avoid the clash with ECS host object.
source v dest: I read that as being the same host as the host:port being listened on. Makes more sense if it's the counterparty's IP address, indeed :-)

If that's a no-go, perhaps a first step would be to make the destination field for this information configurable?

I'm ++ on having this configurable with the current value as default. If we break this with a new default, we'll have to maintain two major versions of this plugin for quite some time and also that version will only ship with logstash by default by 7.0.0

I'll open a PR.


improvement: Create util function to restrict slash commands to specific channels

Right now /drop has some custom logic to check its configuration.json entry for the restricted channels, we should move that to a codepath that all slash commands everywhere can use.
Nice idea would be to gather the channel info from the IDs to provide the user with the actual names/link to the channels where the commands are kosher.
I was kinda gung-ho about converting !drop and !drops to slash, so working to make things a brighter happier place for future slash commands we want to implement.

Addressed with #26


gitleaks: refactor

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 303838 run on x86_64-linux 1

1 package built:

gitleaks

Could you also remove the remaining pname references? The executable name also isn't necessarily equal to pname.

The executable name also isn't necessarily equal to pname.

But here it is otherwise the test would fail.


Update vision property of federation entities

Intent
Since Fog of War has been integrated to the gameplay, the vision attribute of the entities has become obsolete. We need to update these values according to the new range.
User Story
As a project owner,
I'd like to amend the vision attribute of the federation entities
so that they would reveal the tiles according to the new built-in masks associated with the new vision ranges.
Acceptance Criteria

Federation entities have their vision attributes set in a predefined range (1-20).

As a desinger I updated the vision property of federation entities


align to react 0.14.x

Removed findDOMNode usages.
Also change React.render to ReactDOM.render in docs.
#42

+1

@u9520107 can you resolve conflicts?

@casesandberg anyway you can get this merged? relying on react color in a project that we want to bump up to 0.14

Merged in here: https://github.com/casesandberg/react-color/pull/47


Agent Self Update: Ignoring the ADO Requests

Dear All,
Background
Our own developed Agent controller is reverting the VM of the Agents on each Job completed.
Once command line argument of the DevOps Agent triggering the revert process when make it offline itself.
The official built-in self update process of the Agent is taking offline the Agent for minutes, this is enough to trigger the revert process.
Inquiry
We would like to prevent this status, that's why the  "Allow agents in this pool to automatically update" function has been disabled on Organization level.
Unfortunately, we are not able to restrict the usage of the built-in "Update all Agents" function for a limited group of users.
Compare to the restricted global settings, this is available for all users with at least Read access to the Pool.
Is there any way to ignore the update requests of the server side on the clients?
Regards,
Gabor Bernics

Hi @iambernics there is no way to ignore agent update message at the moment, as I understand in general it's not a preferred option to disable it.

Probably one option could be to move such agents to the separate agent pool - if you want to disable agent updates specifically for them.

@iambernics closing this one at the moment due to inactivity - please let us know if you have any further questions.


Allow non-string value in body response for api gateway

Issue #, if available: -
Description of changes:
change data type for body in APIGatewayProxyResponse and APIGatewayV2HTTPResponse struct to interface so it will be more flexible to return the data, instead of only able to return it as a string.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #373 (976b1a8) into master (e5da494) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #373   +/-   ##
=======================================
  Coverage   72.22%   72.22%
=======================================
  Files          19       19
  Lines         738      738
=======================================
  Hits          533      533
  Misses        138      138
  Partials       67       67

Continue to review full report at Codecov.

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

This is already possible with the IsBase64Encoded attribute. Maybe you're looking for a helper func that will marshal an interface{} to a strong value so it can be set in the body? But what that helper would need to know if you're marshaling as JSON, XML, ASN.1, or just encoding raw bytes as Base64.

@MrGossett not like that actually. In my case i want to put a struct in that body, instead of put JSON string to that body

Any news on this being merged?
It's so frustrating to only work with strings

This isn't a valid change, API Gateway will reject the response if the Body field isn't a string.
With this change, any function that tries to set Body to a non-string value, instead of getting their desired JSON, will instead see {"message": "Internal Server Error"}
With API Gateway access logs enabled, I see this error for the failed request: The response from the Lambda function doesn't match the format that API Gateway expects. Lambda body contains the wrong type for field "body"
However, if you find marshaling to a json string to be cumbersome, you also have the option to return any object instead of the envelope struct APIGatewayV2HTTPResponse.
from: https://docs.aws.amazon.com/apigateway/latest/developerguide/http-api-develop-integrations-lambda.html

With the 2.0 format version, API Gateway can infer the response format for you. API Gateway makes the following assumptions if your Lambda function returns valid JSON and doesn't return a statusCode:

isBase64Encoded is false
statusCode is 200.
content-type is application/json.
body is the function's response.

here's the test resources I used:

SAM template.yaml
AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Description: >
  go-http-playground

  Sample SAM Template for go-http-playground

Globals:
  Function:
    MemorySize: 128
    Timeout: 5
    CodeUri: hello-world/
    Handler: hello-world
    Runtime: go1.x

Resources:
  Hello1:
    Type: AWS::Serverless::Function
    Properties:
      Environment:
        Variables:
          HANDLER: "1"
      Events:
        Proxy:
          Type: HttpApi
          Properties:
            ApiId: !Ref HttpApi
            Path: /hello1
            Method: ANY

  Hello2:
    Type: AWS::Serverless::Function
    Properties:
      Environment:
        Variables:
          HANDLER: "2"
      Events:
        Proxy:
          Type: HttpApi
          Properties:
            ApiId: !Ref HttpApi
            Path: /hello2
            Method: ANY

  Hello3:
    Type: AWS::Serverless::Function
    Properties:
      Environment:
        Variables:
          HANDLER: "3"
      Events:
        Proxy:
          Type: HttpApi
          Properties:
            ApiId: !Ref HttpApi
            Path: /hello3
            Method: ANY

  HttpApi:
    Type: AWS::Serverless::HttpApi
    Properties:
      AccessLogSettings:
        DestinationArn: !GetAtt AccessLogs.Arn
        Format: $context.requestId $context.path $context.integration.integrationStatus $context.integrationErrorMessage

  AccessLogs:
    Type: AWS::Logs::LogGroup

Outputs:
  Hello1API:
    Value: !Sub "https://${HttpApi}.execute-api.${AWS::Region}.amazonaws.com/hello1"
  Hello2API:
    Value: !Sub "https://${HttpApi}.execute-api.${AWS::Region}.amazonaws.com/hello2"
  Hello3API:
    Value: !Sub "https://${HttpApi}.execute-api.${AWS::Region}.amazonaws.com/hello3"

hello-world/handler.go
package main

import (
	"encoding/json"
	"net/http"
	"os"

	"github.com/aws/aws-lambda-go/lambda"
)

const HandlerEnvVar = "HANDLER"

// CurrentAPIGatewayV2HTTPResponse is copy-paste of APIGatewayV2HTTPResponse from github.com/aws/aws-lambda-go/events
type CurrentAPIGatewayV2HTTPResponse struct {
	StatusCode        int                 `json:"statusCode"`
	Headers           map[string]string   `json:"headers"`
	MultiValueHeaders map[string][]string `json:"multiValueHeaders"`
	Body              interface{}         `json:"body"`
	IsBase64Encoded   bool                `json:"isBase64Encoded,omitempty"`
	Cookies           []string            `json:"cookies"`
}

// ProposedAPIGatewayV2HTTPResponse reflects the model changed proposed in PR #373
type ProposedAPIGatewayV2HTTPResponse struct {
	StatusCode        int                 `json:"statusCode"`
	Headers           map[string]string   `json:"headers"`
	MultiValueHeaders map[string][]string `json:"multiValueHeaders"`
	Body              interface{}         `json:"body"`
	IsBase64Encoded   bool                `json:"isBase64Encoded,omitempty"`
	Cookies           []string            `json:"cookies"`
}

// Hello returns our handler's response, the caller will handle proxying it to APIGateway
func Hello() interface{} {
	return struct {
		Handler  string
		Messages string
	}{
		os.Getenv(HandlerEnvVar),
		"Hello World!!!",
	}
}

// ProposedV2Handler demonstrates the proposed changes to the envelope. API Gateway rejects this response!!
func ProposedV2Handler() (ProposedAPIGatewayV2HTTPResponse, error) {
	return ProposedAPIGatewayV2HTTPResponse{
		StatusCode: http.StatusOK,
		Body:       Hello(),
	}, nil
}

// CurrentV2Handler demonstrates the construction of an API Gateway HttpApi response envelope containing stringifyed JSON
func CurrentV2Handler() (CurrentAPIGatewayV2HTTPResponse, error) {
	responseBytes, _ := json.Marshal(Hello())
	return CurrentAPIGatewayV2HTTPResponse{
		StatusCode: http.StatusOK,
		Body:       string(responseBytes),
	}, nil
}

// AlternativeV2Handler demonstrates that with APIGateway's HttpApi, Lambda functions can now return json without the proxy envelope
func AlternativeV2Handler() (interface{}, error) {
	return Hello(), nil
}

func main() {
	handlers := map[string]lambda.Handler{
		"1": lambda.NewHandler(CurrentV2Handler),
		"2": lambda.NewHandler(ProposedV2Handler),
		"3": lambda.NewHandler(AlternativeV2Handler),
	}
	lambda.StartHandler(handlers[os.Getenv(HandlerEnvVar)])
}

$ for i in 1 2 3; do echo "handler=$i"; curl -s https://<redacted>.execute-api.us-west-2.amazonaws.com/hello$i | jq; done
handler=1
{
  "Handler": "1",
  "Messages": "Hello World!!!"
}
handler=2
{
  "message": "Internal Server Error"
}
handler=3
{
  "Handler": "3",
  "Messages": "Hello World!!!"
}


If a prompt spans multiple lines, keystrokes result in the prompt label being displayed multiple times

Bug description: If the label of a prompt exceeds the horizontal width of the terminal window, then each keystroke results in the prompt being displayed on a newline.
Expected behavior: If a label spans multiple lines, the behavior is identical to if the label spans one line -- the label is displayed only once.
Observed behavior: When a label spans multiple lines, each keystroke results in a new printed version of the label, i.e. https://asciinema.org/a/hHDOBInyBFwBPXN2ZfaPcULKP

Minimal working example:
package main

import (
        "strings"

        "github.com/manifoldco/promptui"
)

func main() {
        prompt := promptui.Prompt{
                Label:     strings.Repeat("Foo", 100),
        }
        prompt.Run()
}

Asciinema recording of the above: https://asciinema.org/a/hHDOBInyBFwBPXN2ZfaPcULKP

Duplicate of #122


Check if lib target is selected in deployment scripts

Closes #
Introduced changes

Checklist

[ ] Linked relevant issue
[ ] Updated relevant documentation
[ ] Added relevant tests
[ ] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

Do we plan to do that? I think it's worth adding it @THenry14

Do we plan to do that? I think it's worth adding it @THenry14

No, we don't really need this https://github.com/foundry-rs/starknet-foundry/blob/master/crates/sncast/src/starknet_commands/script.rs#L390

@THenry14 we can close it then, right?

@THenry14 imo it's still relevant, the function you linked just loads the compiled code, it doesn't solve the issue with no [lib] target being present in Scarb.toml. E.g. with sncast from main branch when I don't specify the lib target I get:

@THenry14 imo it's still relevant, the function you linked just loads the compiled code, it doesn't solve the issue with no [lib] target being present in Scarb.toml. E.g. with sncast from main branch when I don't specify the lib target I get:

will look into this


Added single quotes on hint query containing mustache substitution fo…

Description

In this PR, single quotes have been added around substitution expression in hint query of Snowflake DB.
This fixes the issue: https://github.com/appsmithorg/appsmith/issues/22132
Single quotes is necessary in hint text because Snowflake DB don't have prepared statements mode yet and with single quotes, the substituted values appear in query without quotes leading to query execution error.

Fixes #22132
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Running a query in Snowflake DB integration, having substitution expression not enclosed in quotes will give an error.
Currently the hint query that has substitution expression has not enclosed then in single quotes. Hence adding single quotes around them to provide user with proper hint.

Manual

Test Plan

Create a new "Create Query" for Snowflake DB. The hint query should contain the sample substitution expression in single quotes.
Run a fetch query for Snowflake DB containing substitution expression without single quotes enclosing it and it should return an error.
Run the query with quotes enclosing substitution expression and the query should run successfully.

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

Is it possible for us to write tests around this query generation flow so that we never regress on it?

Is it possible for us to write tests around this query generation flow so that we never regress on it?

Yes. Tests can be added on the method that loads these templates from resource files. I will add new tests in this pr. Thanks.

/build-deploy-preview

@tkAppsmith - This is tested. Looks fine. can be closed

/ok-to-test


aliases on singletons return 'bean name already in use'

creating aliases on singleton not possible. See "Alias_on_singleton" in the unit test

turned out to be in the factory where the alias wasn't defined as <alias... but as <bean...


Refactor pytest fixtures

Code Pull Requests
@w4nderlust
Refactored pytest fixture definition per @jaipradeesh recommedation.  Moved fixtures from tests/fixtures/filenames.py to tests/conftest.py.  This takes advantage of standard pytest functionality and allows removing imports for tests.fixtures.filenames.
After making the change, I ran pytest -v tests.  All tests completed succesfully.  Attached is the run log from pytest -v tests.
pytest_fixture_refactor_test_run.txt

Thank you @jimthompson5802  !


Adopt controller-runtime

Problem
Follow controller development best practice and Adopt kubebuilder, controller-runtime and etc.
Persona:
Which persona is this feature for?
Contributors
Exit Criteria
All functions are working as expected
Time Estimate (optional):
7 days
Additional context (optional)
https://github.com/kubernetes-sigs/controller-runtime/

please move to knative/pkg if it is feasible

@MartinForReal To keep consistent with all knative projects, we leverage the structure the same as any other knative project.
The community had this discussion 2 years ago, when operation wg was founded.
The knative common package, knative/pkg, has already established a mature backbone for other knative projects to rely on.
controller-runtime is just one lib to produce kube operator, but it is NOT the only one.

Don't make a beauty contest between controller-runtime and knative/pkg. It is NOT which one beats one another.
They are both great libs. To determine which one to use, it is all based on the time and nature of the project.
For newly created projects, there could be decisions made towards either direction, but for projects, which have been developing for a while, and have already developed the ecosystem in a certain degree, it may be past due to make these decisions, since we departed a long time ago. At the beginning of knative, neither kubebuilder or operator-sdk, based on controller-runtime, was mature enough, to demonstrate the prosperity, so knative made the decision to move on with its own implementation.
By far, in my personal opinion, knative/pkg has got the controller and reconciler wrapped up NOT bad. And in terms of webhooks, I love knative/pkg more, coz it surely takes less effort to add any type of webhook. Last but not least, I am a fan of the tool knative.dev/pkg/apiextensions/storageversion to migrate older version of resources into new version, once the CRDs bump into new version. controller-runtime does not have it by far.
What is good about controller-runtime is that you can either leverage operator-sdk or kubebuilder as the command tool to scaffold, build and publish everything. However, the drawback is that you have keep you golang cod, annotations, tags, manifests, etc, everything in-sync and the project structure has been FORCED to write in a RIGID way.
You have to obey the rules that changing the source code and annotations, and run the commands to generate the rest. You can not tolerate anything NOT in-sync. Once it happened, after running the command to regenerate the manifests, you never know what will overwrite what.
knative/pkg lacks a systematic tool as the command line to generate everything, but on the other hand, it offers some certain degree of flexibility for developers to maintain the project code.
This comment is more of my own opinion and experience, working with both of the libs.
Another example you can refer to is the tekton operator. At the beginning, I was the only one, who insisted to leverage knative/pkg as the lib to write the operator, but all the others went with controller-runtime. However, one year later, they found out that all existing Tekton tools and processes were built based on knative/pkg, if operator did not, they cannot leverage any existing building process, so they decided to change the framework from controller-runtime into knative/pkg, to make tekton operator a real citizen of tekton community: https://github.com/tektoncd/operator.
It is not a beauty contest, it is just what works for the project and the community.

Fair enough , Thanks for the clarification! :)


TabStore onTabUpdate gives an exception

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to Chrome. Open a new tab.
Open background.html devtools console.
Type google.com (website url doesn't matter here). Type any url.
Check console to get the below mentioned error
See error

Expected behavior
There should not be a console error before I even start to use my accessibility Insights extension.
GIF

Extension (please complete the following information)

OS: Windows 10
Version:1.0.4
Powered by axe-core 3.2.1
Chrome Version: Version 74.0.3729.40 (Official Build) beta (64-bit)
Target Page: N/A

Are you willing to submit a PR?
Yes
Did you search for similar existing issues?
Yes
Additional context
Add any other context about the problem here.

@haonliu is this resolved?

@haonliu is this resolved?

Yes. Should the person who opened this issue close it or I should?

great!
the person who open it. @Shobhit1 could you please verify?

Fixed. I dont see the console error anymore. Closing the issue.


[FEATURE REQUEST] : Request for Out of Office (OOO) Status Feature

Description:
We need to implement a new feature that allows users to request Out of Office (OOO) status. Currently, users are directly becoming OOO without any formal request process. The proposed feature aims to introduce a request-and-approval mechanism to manage OOO status changes.
Key Features:

User Initiated Request:

Users should have the ability to submit a request for OOO status.
This request could include details such as the start date, end date, and a brief reason for the OOO request.

Superuser Approval/Rejection:

Superusers (administrators or designated personnel) should be able to view and manage OOO requests.
Superusers can either approve or reject OOO requests based on company policies or workload considerations.

Multiple Requests Handling:

The system should be designed to handle multiple OOO requests simultaneously.
Clear visibility into all pending, approved, and rejected requests should be provided to both users and superusers.

Implementation Considerations:

Ensure proper user interface elements for submitting and managing OOO requests.
Implement a secure and auditable system for tracking OOO status changes.
Maintain a log of all OOO requests, including their approval or rejection status.

Additional Tasks:

Flow Diagram for Current Implemented User Status:

Create a detailed flow diagram illustrating the current implementation of user statuses, including the transition between different states (e.g., active, idle, OOO).
Identify key decision points and interactions that affect the user status.

Design New Schema for User Status, OOO Status Requests, and Logs:

Design a new database schema that accommodates the introduction of the OOO status request feature.
Clearly define attributes for user status, OOO status requests, and logs.
Ensure the schema is scalable, maintainable, and adheres to best practices.

UI Implementation for User:

Develop user interface elements to allow users to initiate and manage OOO status requests.
Include a user-friendly form for submitting OOO requests, displaying current status, and viewing request history.
Ensure a seamless and intuitive experience for users.

UI Implementation for Superuser:

Design a dedicated interface for superusers/administrators to review and manage OOO status requests.
Provide clear visibility into all pending, approved, and rejected requests.
Implement actions for superusers to approve or reject requests with ease.

Logging UI:

Include a user interface for viewing logs related to user status changes and OOO status requests.
Ensure that logs provide detailed information, including timestamps and reasons for status changes.


Display selected question's details

Changes

Add API to retrieve question by id
Add route for viewing question details
Add in link 🔗 from home to individual question
Rename file from View.js to QuestionPage.js
Add css 🎨 to question page
Add back navigation 🔙 to question page
Update error page
Add loading animation for question page
🆕 Add PageNotFound component & Update usage of it
🆕 Extract out firebase configuration from backend index.js to firebase.js — usage of it is as per normal in index.js

Preview of the page

lgtm love the spinner

lgtm :shipit:


[utils] Add usePossiblyOpaqueIdentifier

Avoids cascading updates when useOpaqueIdentifier is available.
Uses useOpaqueIdentifier if available, otherwise falls back to useId.
Unfortunately, some useId usage did not stay compatible with the original vision so we need to do some work in the callsites first.

Oh my, host attributes being verified with PropTypes.string is going to be fun to deal with :exploding_head:

This will not be part of v5. It acts as a test to find out which current callsites of useId are incompatible. These will be fixed for v5 so that we have an upgrade path during v5. I just don't want to experiment with this hook in production just yet (even though one would need to install experimental React versions).

Looks like we can use the built-in hook in React 18 and fall back to our shim in React 17 and earlier: https://github.com/reactwg/react-18/discussions/111

Causes hydration mismatches when integrating into https://github.com/mui-org/material-ui/pull/29171.
Not clear why but there seems to be at least an issue with Emotion (https://github.com/reactwg/react-18/discussions/111#discussioncomment-1581040) so I'll hold this PR until we know why.


Broken link in README

Under the Quickstart section, the link in the line
"Runs in all supported browsers: Glitch" goes to a Glitch project that does not exist.
https://i.imgur.com/uS7gHzx.png

@azakus this link is gone now. The ticket could be closed


PROBLEMS filter line edit misses standard navigation shortcuts (HOME/END)

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.84.2
OS Version: Suse SLES 15 SP5

This bug report describes a problem regarding the PROBLEMS view (workbench.actions.view.problems)
Steps to Reproduce:

write a filter string into the PROBLEMS filter line edit

Press the HOME button
Nothing happens
Expected behavior: the Cursor moves to the beginning of the string

Steps to Reproduce:

write a filter string into the PROBLEMS filter line edit
move the cursor to the beginning of the string
Press the END button
The input focus leaves the line edit (and is now on the 'View as Table' button)
Expected behavior: the Cursor moves to the end to the string

It works for me on Mac with cmd+left and cmd+right keys. Could be keyboard related, hence assigning to @ulugbekna

duplicate of https://github.com/microsoft/vscode/issues/180869


when will be the next cohort of 2020

Hi,
I want to ask when will be the next cohort of 2020 and how the mentees will be chosen?

Hello @slim-hmidi  at this moment the mentorship initiative is evaluating previous cohorts and improving the process to asserting pairings.
As soon as we finalize this process we will update the README and notify everyone about the next cohort
please fell free to join our slack channel mentorship on http://www.nodeslackers.com/


Display issue when used from IJulia

I open a new notebook, first cell is
using RCall

Second cell is
"x <- 5" |> rcopy

This returns an empty graph as the first output, and only then the real output of
1-element Array{Float64,1}:
 5.0

If I then add a third cell that has purely julia code, it will also add this empty graph in the out cell.
I attach a screen shot:

I'm on Windows, julia 0.3.11, IPython/Jupyter 3 and all packages updated to the latest version. This happends with RCall from master or the latest registered version.
Any idea?

Oh, and I just tried this on the sb/robject branch, same thing.

Thanks, I'll look into it.


Linux client not working on Linux Mint

Describe the bug
Outline client does not connect on Linux Mint.
To Reproduce
Add a server, click connect. Does not work.
Expected behavior
Should work.
logs
./Outline-Client.AppImage
loading web app from file:///tmp/.mount_OutlinUWm6eX/resources/app.asar/www/electron_index.html
Checking for update
Outline is starting
Update for version 1.0.11 is not available (latest version: 1.0.11, downgrade is disallowed).
connecting to 11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000...
a helper has exited, disconnecting
all helpers have exited
disconnected from 11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000

could not connect: ServerUnreachable ()
Promise "DNS lookup" resolved before 10000 ms.

Screenshots
.
I am happy to provide more info
Desktop:

OS: Linux Mint 18.1 Serena
Google Chrome 76.0.3809.100

same
OS: Linux Mint 19.2
Google Chrome  Version 78.0.3904.70 (Official Build) (64-bit)

Hi Guys, same here on Ubuntu 16.04
I verified nothing logged looking through journalctl

@dban0001
hello guys;
please check the open port on your server.
best regard.

same problem on Mint 18.1

same problem on Mint 18.1

use command for run outline
./Outline-Client.AppImage --no-sandbox

Same problem on Ubuntu 16.04.6 LTS. --no-sandbox did not help. Server is available from any other device in my local network (android, ios), only laptop has a problem (all are connected to same wifi)...

use command for run outline
./Outline-Client.AppImage --no-sandbox
Sometimes the sandbox does not allow connection and needs to be turned off.

Thanks for the advice, unfortunately --no-sandbox didn't resolve it. The situation here is similar to @lexin4ever : iOS and Android clients located on the same LAN as my Mint PC both connect successfully to that same server that i'm trying to reach from linux.

Thanks for the advice, unfortunately --no-sandbox didn't resolve it.
@AshkanSecure (( you are sooooper ))

First, turn off the firewall on your server and then double-check.
If you have a successful connection, the problem is from the server and you should fix the problem there.

No firewall on the web host (digital ocean), and firewall (iptables) rules on the machine are flushed.
I am connecting from China to Europe, so not sure what the national firewall is doing in between. If there were any connectivity tests with wikipedia.org or google.com, they would be blocked.
A quick network dump seems to indicate that the TCP socket was open (SYN), then my client requested it to be closed (FIN).

@AshkanSecure

Same error, Ubuntu 16.04 LTS, Outline
Try to change port for new Key to the same as for other devices in the same WIFI network with no success.
All other devices (Android iOS and Mac OS) working OK with this Outline server.
$ telnet xx.xxx.xxx.xxx 45780
Trying xx.xxx.xxx.xxx...
Connected to xx.xxx.xxx.xxx.
Escape character is '^]'.
But no connection for the same IP and port from Outline Beta 1.4.0


Main font looks as if it has no anti-aliasing or hinting applied

Hi, the app looks really smooth, but the font doesn't! One user on the Play Store wrote "the fonts look too grainy, better fonts would be seriously appreciated". From your reply to that review I gather you don't experience this yourself. For me, I'm seeing it both on my device and in the Android Studio emulator, which emulates a "Nexus 5X API 28 x86".
I don't know much about Android development, but I managed to change the font in the source code to other system fonts, but they have the same problem.
Here's a screenshot from the emulator. When you look closely or zoom in you can see that the issue is only with the list items, all the other UI text (e.g. app name, list names, add new item) looks smooth. Maybe you can find out what you did differently for the list items?

This is right, the grainy fonts never jmped to my eyes before I zoomed on your screenshot. Now I can see them.
To be able to swipe the items the system "draws" the text on a canvas. I don't think I can do much for it as this is handled by a lib I'm using (advancedrecyclerview). I'll see if I can make it look bettter.

I have an even worse experience than the OP:

It's a pity, because I would love to use 1List, but it puts too much strain on the eyes to work with the list.

@lolo-io Isn't the issue fixed in the develop branch by commit 4f6b2c65f5a2888835bba8f34ffc58eae597e94e ?


Upstream merge request.

Netty version was bumped.

https://github.com/WaterfallMC/Waterfall/commit/641c851948a3752b3c9f045c4657aa95a642e336


33513 pull replication by document id

Do pull replication with a subset of document IDs.
The user configures clientFilterDocIds on the CDTPullReplication object; this is an array of document IDs.
Internally, the sets clientFilterCurrentSetDocIds (doc IDs for inclusion during pull, for which we already have at least one rev in the database), and clientFilterNewDocIds (doc IDs for inclusion during pull, which we don't yet have in the database) are used.
Lots of things are missing right now, especially an updated remoteCheckpointDocID method whose input to the hash function should the doc ID list.

Suggest we close this one for now @tomblench ?


Pezzotti on server => subsample to 5K => Pezzotti on client

Tech to try for sub-sampling

[ ] (BBI/FredHutch ?)
[ ] projected support points (PSPs)
[ ] ? Hamiltonian Monte Carlo with Energy Conserving Subsampling

Forked out to separate repo, see: https://github.com/reconstrue/single_cell_on_colab/issues/1


Multicast UDP traffic appears to cause Booster to encounter watchdog resets

While testing out some multicast UDP traffic, I observed booster reboot twice. The service then indicated that a watchdog occurred. It's not clear if this was related to the Multicast traffic or not, but seemed to only occur when executing the Stabilizer streaming HITL with a streaming IP of <IP_ADDRESS>
> service
Version             : Unspecified [release]
Hardware Revision   : v1.5
Rustc Version       : rustc 1.72.0 (5680fa18f 2023-08-23)
Features            :
Detected Phy        : W5500
Panic Info          : None
Watchdog Detected   : true

>

This was running on 1ea1124170383a787d8dce644a263fde21fc279d

n.b.
I'm also streaming multicast in the same net on <IP_ADDRESS>.
I would not use multicast in <IP_ADDRESS>/8. Let's stick to the admin scoped <IP_ADDRESS>/8.

Upon further investigation, I'm not certain that the watchdog was a result of multicast. It could have been a watchdog from earlier testing I was doing that was still latched.

This appears pretty reproducible with <IP_ADDRESS> in my network. I'm looking into what's happening

Disabling the watchdog indicates that booster remains operational throughout the whole event and doesn't encounter a true lockup. What I believe may be happening is that there's so much traffic incoming on the PHY that smoltcp and the W5500 need to process that it slows down processing of the application in general. I wonder if there's a way we can handle events like this where there's excessive ethernet traffic.

It should not be seeing that traffic at all. It doesn't flood.

I see the green ethernet LED remain consistently on during the Stabilizer stream period, indicating that Booster is indeed receiving excessive packets.
I opened https://github.com/smoltcp-rs/smoltcp/issues/848 around this, but this could be due to a cheap switch that isn't handling multicast properly as well.

That doesn't seem to be the case here. I'm seeing close to zero traffic towards stabilizer (while another one is streaming multicast). As expected.

This is likely then a W5500 or w5500 specific bug.

Do you have a router and/or managed switch in between? The switch I'm using with my local network is just some cheap unmanaged unit. I suspect that it's forwarding the multicast traffic to all of the ports regardless of subscription to the multicast group.

There is a cisco 2960L in between. Whether that one can be called "managed" or not is debatable. But it does behave properly regarding non-flooding of multicast.

We have found a similar issue when using the booster with an unmanaged switch with other traffic. The booster seems to reboot when it receives invalid packages on the MQTT port 1883. After that, when trying to enable channels, they only seem to change to powered instead of enabled and stay this way until the booster is powercycled.
This behavior can be reproduced by sending invalid packages via netcat, e.g. nc -u <booster-ip> 1883 < /dev/random.
We will use a vlan for now, but it would be nice if this could be fixed to make integration into existing network structures easier.

@aferk if you connect to the device when it's in this state and run the platform service command and attach the output here, that would be helpful.
That will tell us why the device is resetting.

Running service before sending packages:
> service
Version             : v0.5.0 [debug]
Hardware Revision   : v1.6
Rustc Version       : rustc 1.76.0 (07dca489a 2024-02-04)
Features            :
Detected Phy        : W5500
Panic Info          : None
Watchdog Detected   : false

Sometimes I just get the Watchdog Detected: true as above, but when re-trying a few times, I do also get the more informative panic info below:
> service
Version             : v0.5.0 [debug]
Hardware Revision   : v1.6
Rustc Version       : rustc 1.76.0 (07dca489a 2024-02-04)
Features            :
Detected Phy        : W5500
Panic Info          : panicked at /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/w5500-0.4.1/src/raw_device.rs:92:13:
attempt to subtract with overflow

Watchdog Detected   : true

Booster HW is version 1.6 (non-HL) from Creotec. We tried to upgrade the firmware because of this issue https://github.com/sinara-hw/Booster/issues/393, which both of our non-HL boosters regularly experience.

@aferk The old firmware is dead and rotten. You will hopefully understand that this firmware here is bound to have the same fate as long as people don't invest into it or contribute or buy where the development is funded.

The panic referenced above should be fixed in the latest release of the w5500 crate, which was updated recently (in https://github.com/quartiq/booster/pull/373). I'm currently testing a fix for the watchdog event in smoltcp


Update trace4cats-fs2, trace4cats-testkit to 0.12.0-RC1+162-70070fb2

Updates

io.janstenpickle:trace4cats-fs2
io.janstenpickle:trace4cats-testkit

from 0.12.0-RC1+156-7ea07b63 to 0.12.0-RC1+162-70070fb2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.janstenpickle" } ]

labels: library-update, semver-build-metadata

Superseded by #4.


Move GKE basic operations to a separate package

As talked with Chao, the current clustermanager library contains too many logics that are specific to integration testing, so it's not a good idea to reuse it for cluster management for performance testing.
This PR moves GKE basic operations to a separate package, and make some small refactors.
Bonus:
Fixed a bug by setting the InitialNodeCount to be MinNodes for GKE creation request, otherwise we will get a 0-node cluster.
/cc @adrcunha
/cc @chaodaiG

/hold
I'm considering if we can call wait directly in CreateCluster, DeleteCluster and SetAutoscale.

@chizhg , thank you for the work.
Regarding the wait, I would prefer adding separate APIs such as WaitCreateCluster, WaitDeleteCluster, etc.. So that the current APIs are conforming with GKE SDK  in terms of async operations. The nice thing about using this async operation vs. gcloud is that we can have the luxury of not waiting on the creation/deletion etc..
Async delete is already been used, and  when the needs come we can leverage it for creation, for example we can start cluster creation operation, building container images while waiting, then come back waiting for cluster creation, which can largely alleviate the pain of time spent waiting on cluster creation.

Also, there will be a big merge conflict between this one and #740, which removed SetAutoscale, and have some other clean ups

@chizhg , thank you for the work.
Regarding the wait, I would prefer adding separate APIs such as WaitCreateCluster, WaitDeleteCluster, etc.. So that the current APIs are conforming with GKE SDK in terms of async operations. The nice thing about using this async operation vs. gcloud is that we can have the luxury of not waiting on the creation/deletion etc..
Async delete is already been used, and when the needs come we can leverage it for creation, for example we can start cluster creation operation, building container images while waiting, then come back waiting for cluster creation, which can largely alleviate the pain of time spent waiting on cluster creation.

I didn't make much change to the wait logic since I didn't find a super clean way.

Unit tests are failing, fixing...

@chizhg , thank you for refactoring!

Not sure if it's possible to add unit tests for client.go, let's discuss it tomorrow...

/hold cancel
This PR is now ready for review.

Looks cool!
/lgtm

/approve

/lgtm

/lgtm


support SceneConfigs for navigator replace.

We need to create navigation something like this:
a -> b -> c -> d back button on b, c, d should take you back to a at all times.
We are using replace for navigation from b->c->d However there is  no animation support for replace.
Is there a better way to do this with animations?
@vjeux

@ericvicenti

Instead of using replace, use push to animate to b->c->d
Then, instead of doing a pop on the back button press, do popToTop(), so pressing back will always take you to a

@ericvicenti a here was just for example and not the top of the stack. Actually there is one more view before a.

Have you tried popToRoute(route)?
http://facebook.github.io/react-native/docs/navigator.html#navigation-methods

popToRoute give me Invariant Violation: Calling pop to route for a route that doesn't exist!"

So here is the hierarchy: x -> a -> b -> c -> d.
A back on b, c or d should take you back to a. b to you to c though. I use replace in b to take you to c and so on for d
I hope the context is clear.

I'm not sure, but it seems you're trying to access re-create the same route, when you need to pass exactly the same route. So maybe storing a link to the route object and using it may help.

nope. I'm not trying to create the same route. I just want that when you are on c and you press back it should jump to a instead of b.

Yeah, I got your case. Could you provide gist with example or smth?

Codebase is too long for a gist etc.
This is what i do in b.js
if(this.state.runNext) {
      return this.props.navigator.replace({
        component: require('./c'),
        sceneConfig: Navigator.SceneConfigs.FloatFromRight,
      });
    }

if(this.state.runNext) {
  this.props.navigator.popToRoute({
    component: require('./c'),
    sceneConfig: Navigator.SceneConfigs.FloatFromRight,
  });
}

Doesn't work?

@Kureev, close, but that creates a new route object. The navigator will look in the route stack for that object but it cannot be found. You need a reference to the exact same route object, you cannot create a similar one.
@chirag04, if you want an animation then you should use push. Replace will swap out the scene immediately.
If I understand correctly, then you want to make it easy to get back to the x scene. At some point you surely call navigator.push(xRoute) in order to load the x scene in the first place. You need to save a reference to xRoute so you can later call navigator.popToRoute(xRoute). Does that make sense?
If you put your code in a simplified gist, I can show you how to do that.

@ericvicenti popToRoute makes sense and like you said i will need to store reference to the parent Scene. Is there a set way of keeping a reference to the parent Scene or should i just pass that as prop to the child scene.

You could pass as a prop, or if you only want to render that scene once, you could put the route in a module.

Sure, That seems to work here.
Just a suggestion: just like parentNavigator, can we support parentRoute in the navigator?

I am hoping to move away from parentNavigator, because it is very confusing when child scenes have direct access to the parent hierarchy.
Can you use parentNavigator.route for that?

Ah i see. Makes sense. I din't try parentNavigator.route yet. Will let you know.
Thanks for taking the time out to help me. You guys are awesome.

i have similiar problem with error message "undefined is not a object evaluating sceneConfig..."
I'm using .popToPop() and it's working, but after change route app crash with above error, so problem was I did't bind this on my function with popToPop method...
sry for eng


Slider doesn't update width without triggering component rerender.

The issue is the same as here pretty much: https://github.com/akiran/react-slick/issues/809 ,- without resize event, or any other trigger for components to rerender (modification of the props for example) the images inside slider do not adjust their size. Here is an example:

I took a look into this and was able to fix this using css calc() property, will PR soon.

This one should also fix all of these: https://github.com/akiran/react-slick/issues/600, https://github.com/akiran/react-slick/issues/543 .

Just add a ref in wrapper:
<Slider {...settings} ref={slider => this.slider = slider}> ... </Slider>

and doing this.slider.innerSlider.onWindowResized() after setting states in wrapper, should take care of the updates.

Will try to come up with a better js solution for this.

https://github.com/akiran/react-slick/commit/8591dace983fa1bc103964459ab99eaa34eed8de solves the problem. Changes will be released soon.


Fixed headers, since some tutorial steps were showing up as separate sec...

...tions in the article ToC.
A small change, but it fixes an issue with the "In this article" section (article ToC) that shows up on the right side of the page.

Please fix merge conflict and then we can accept your PR. Thank you -- Monica


Local files that start with file:///

Hello, Thanks for the extension, it is exactly what I was looking for, but I can't seem to get it to work with local urls that start with "file:///". Is there a special syntax for this?

Hey @ndeubert,
It only works on websites because it's based on [location.href](https://github.com/sylouuu/chrome-tab-modifier/blob/release/0.5.0/src/js/content.js#L9). Moreover, the extension can't access to local files except if you have a website with a virtual host. Something like "localhost/...", "my-website.local"...

I reopen this issue because I've found something about file:// access.

Hi @ndeubert,
Acutally, it's possible, see this commit.
This feature is planned for release 0.6.0 at the end of the month.
Bests

Excellent, merci beaucoup! I tried making the same patch to the
manifest.json file in my google-chrome config folder but that still did not
work for me. So I also tried replacing the js and html folders with the
latest ones from git but that also did not seem to work for me either. For
reference the URL I am trying to match is like this:
"file:///scratch/svn/ssh1/index.html?<IP_ADDRESS>&debug"
and my tab_modifier.json file has this:
{
"file://": {
"icon": "http://localhost/logos/soundtouch_32x32.png",
"pinned": true
}
}
On Wed, Sep 16, 2015 at 6:50 AM Syl<EMAIL_ADDRESS>wrote:

Hi @ndeubert https://github.com/ndeubert,
Acutally, it's possible, see this commit
https://github.com/sylouuu/chrome-tab-modifier/commit/5a64229ab9d44625edbaf3b6b7963a2cb06c462b
.
This feature is planned for release 0.6.0 at the end of the month.
Bests
—
Reply to this email directly or view it on GitHub
https://github.com/sylouuu/chrome-tab-modifier/issues/13#issuecomment-140703929
.

Your JSON file is OK but if you wan't to make it works just check the developer mode in chrome://extensions/.
Clone the repository & go to release/0.6.0.
git checkout release/0.6.0

Import the chrome-tab-modifier/dist folder in a developer extension & add your JSON.

Got it working! Thank you again.
On Wed, Sep 16, 2015 at 11:55 AM Syl<EMAIL_ADDRESS>wrote:

Your JSON file is OK but if you wan't to make it works just check the
developer mode in chrome://extensions/.
Clone the repository & go to release/0.6.0.
git checkout release/0.6.0
Import the chrome-tab-modifier/dist folder in a developer extension & add
your JSON.
—
Reply to this email directly or view it on GitHub
https://github.com/sylouuu/chrome-tab-modifier/issues/13#issuecomment-140785066
.

Supported in 0.6.0.

Hello. Just wanted to point out a caveat: not working in conjunction with SSL encrypted websites.
E.g. I have the entry:
"""
{
"mail.google.com": {
"icon": "file:///path_to_my_favicon.png"
}
}
"""
but the javascript console says:
"""
Mixed Content: The page at 'https://mail.google.com/mail[...]' was loaded over HTTPS, but requested an insecure favicon 'file:///path_to_my_icon.png'. This content should also be served over HTTPS.
"""

@pierrethibault
You can't load a local favicon, you have to upload it somewhere.
This is related to detect local URL ie.:
{
   "file:///path_to_my_file.html": {
        "unique": true
    }
}

Alright sorry for the confusion.

Quick note to say that this case is listed in the README with few workarounds.


Improve readme descriptions and instructions.

Wording could use improvements across the board so that setup/usage is easier to follow/understand.

This will probably never be good enough. But I at least got it caught up to a reasonable point.


Use sambamba for duplicate marking.

In porting the FDA metrics pipelines to the WDL workflows I ran into that those had been updated to switch duplicate markers but these had not.  This carries that change over to the CWL version.  I haven't tried running it on the immuno.cwl yet--thought I'd open this and see if it's something we wanted to do before running a test.

I'll let John and others review, but I support this change as one that increases the speed through parallelization while using the same duplicate marking strategy and producing essentially identical results.


Values validation doesn't work as expected

Describe the Bug
Validation of availability_zone_count were introduced not far ago. It looks valid but fails if I passing
  availability_zone_count    = 2
  instance_count                 = 2

in terragrunt inputs with error:
│ Error: Invalid value for variable
│
│   on variables.tf line 115:
│  115: variable "availability_zone_count" {
│
│ The availibility zone count must be 2 or 3.
│
│ This was checked by the validation rule at variables.tf:120,3-13.

But if I do not provide value at all, default value 2 working good. I was able to fix it by adding tonumber() func:
variable "availability_zone_count" {
  type        = number
  default     = 2
  description = "Number of Availability Zones for the domain to use."

  validation {
    condition     = contains([2, 3], tonumber(var.availability_zone_count))
    error_message = "The availibility zone count must be 2 or 3."
  }
}

Environment (please complete the following information):
Terraform v1.1.4
on darwin_arm64

provider registry.terraform.io/hashicorp/aws v3.73.0
provider registry.terraform.io/hashicorp/null v3.1.0

Additional Context
I'm not sure where error is, in terraform which doing casting of variables in contains function or somewhere else.

I guess this is related to my finding mentioned here:
https://github.com/cloudposse/terraform-aws-elasticsearch/pull/122


Untie a field type and required attrib

Feature Request
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
 pydantic version: 1.5.1
            pydantic compiled: False
                 install path: /Users/user/projects/pydantic/pydantic
               python version: 3.7.6 (default, Dec 30 2019, 19:38:26)  [Clang 11.0.0 (clang-1<IP_ADDRESS>)]
                     platform: Darwin-19.4.0-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions', 'email-validator', 'devtools']

In this moment field "required" attribute is tied on two field parameters: field type and default value.
For example:
class SomeClass(BaseModel):
    a: Optional[int]
    b: Optional[int] = ...
    c: int = None
    d: int = ...

a - not required
b - required
c - not required
d - required
First point. Using Ellipsis by default for set required=True on a field is not transparent.
Second point. allow_none attribute. What with him?
a - allow_none
b - allow_none
c - allow_none
d - not allow_none
In my current project I have the goal: make "not required" and "not allow_none" field.
I can't see opportunity for making this kind of field. For what I need this field? PATCH queries. Some attributes can be None, but only if they define directly in query body. If not - do not make them None by default.
I think, problem in the dependence between Type/required/allow_none attributes.
What if we untie this dependency? May be field Type should not affect to "required"? I create PR (unfinished) for demonstrating my thought. Main points this PR:

Management on a required attribute through config;
"default" value can affect on a required attribute: defined -> required=None (if don't say opposite in config);
a field Type can't affect "required";

Yes. This way makes base model definition little complex. But it's price for more transparent and independence with type and defaults (partial from last).
What are you thinking about it?
P.S. I didn't make test for PR, but Pydantic tests was adapted.
In this moment field "required" attribute is tied on two field parameters: field type and default value.

duplicate of #990. This will change in v2.
If you don't like Ellipsis you can use Required
from pydantic import BaseModel, Required

class SomeClass(BaseModel):
    a: int = 4
    b: int = Required

@samuelcolvin I think, it's not a duplicate #990 issue.
I talking about unrequired + not_none combination and untie Type and Required field attributes. In #990 talking about new type for required + none field attributes combination, or some tie with default value.
Also, not that one is messages in #990 didn't give unrequired + not_none combination.
Please correct me, if exsit way to define unrequired, not_none field without default value (with #990 or without).


Release with PostCSS 8 compatibility

It seems like a PR was made for PostCSS 8 compatibility (#38) but a release hasn't been made since then.
Is there any chance we can get a release cut, for those that are trying to migrate to PostCSS 8?

Just to scratch my own itch, I pushed a 4.0.0 of this package under my own username on npm
https://www.npmjs.com/package/@alexlafroscia/postcss-color-mod-function

I using<EMAIL_ADDRESS>with latest webpack v5.64.2 and latest postcss v8.3.11 and it seem it is working well with PostCSS 8. Or Im missing something?

Support for PostCSS 8.0.0 has be released as  v4.0.0
Thank you for your patience 😅


use web-serial-polyfill to run web root on some device don't support webserial

some device (such as Android) don't support webserial but webusb. use https://github.com/google/web-serial-polyfill to implement a wevserial api over webusb. so that able to root a stick with only a phone
demo of it: https://androidserialport.web.app

I don't understand the reason for this, why would one use Android to root an SFP GPON stick?
The solution to this is to simply use a PC with chromium based browsers, I'm totally against making things more complicated for Android.
Firefox and Safari don't even support webusb so there is no interest in using polyfill to add support for these  desktop browsers

I am absolutely against supporting devices without webserial as far as I'm concerned, if you want you can do a PR but it shouldn't bring too much new code to maintain and above all it should be just a fallback in case webserial isn't supported and not replace webserial

What do you mean you only have an android computer? But have you already used the serial in this android computer?
But does it by chance also work in chromebooks or computers with unusual operating systems?
In any case, I agree with @ErnyTech, if you think this functionality is useful and easy to do (no new dialogues, no new code, just a <script src>, remember that this site is currently being developed without the use of npm, so it must be self-updating).
There is no hidden code all the code is always visible in the source code, and in any case there is the v2 version if the device supports failback mode and a mixture of v2 and v1 if the device does not support it, which is currently quite up to date, there would be editing. A v3 manual could be made based on python scratch

sorry for my misleading sentence. i means "Android phone" is just a regular linux computer. if there is a more generic/manual tutorial, android user can use the normal linux way (such as picocom)

both me and @ErnyTech are also linux users (he much more than me, and the procedure has been tested on numerous linux distros), now apart from android which distros have problems? I don't even understand what picocom has to do with it...

i said im not familiar with front end. but maybe i can have a try

good luck, I don't really understand what needs to be done and I have more doubts than questions about the code you linked....

sorry for the misleading sentence again. the "hide" is actually means they are in the js code. to the end user which don't familiar with js, they may unable to find it. that's what "hide" means

Ok...

i didn't check the steps what v2 did is same as v3 or not. but the page said "Do not use the V2 and V1 versions under any circumstances" without explaining why. that make me think v3 have some change on procedure, and v2 / v1 not up to date

the reason is simple at present most of the Italian people who are in the telegram group despite the warnings still use it as soon as the web procedure gives an error without even asking or anything else, at present V2 and V1 have been kept up-to-date and congruent with V3 (as far as possible, I remember that they are currently mainly used for consultation), but in the future this might change.

i means "Android phone" is just a regular linux computer.

Absolutely not, Android is not a normal linux computer. A linux computer generally means a computer with a GNU/Linux distribution installed, Android has just the kernel and some programs in common.
In fact in a GNU/Linux distribution it is sufficient to install chromium to use webserial and there is no problem, all tests of the root procedure with web serial have been developed in a Linux environment and I can guarantee that there is no problem.

i means "Android phone" is just a regular linux computer.

Absolutely not, Android is not a normal linux computer. A linux computer generally means a computer with a GNU/Linux distribution installed.

i means termux (or other terminal emulator) and chroot to a arm based linux rootfs

i means "Android phone" is just a regular linux computer.

Absolutely not, Android is not a normal linux computer. A linux computer generally means a computer with a GNU/Linux distribution installed.

i means termux (or other terminal emulator) and chroot to a arm based linux rootfs

It's such an unusual usage that it's not useful to modify our tool to cover such a small number of users, if you can use the serial from termux you should be able to use the root python script V2.

then you should know it would be better to explain how to solve the problem to others (like v1), rather than embedded everything to a script and tell others to run it (v2/v3). that what i means here. i want to use picocom to access the stick and run command by myself. rather than run some script fron internet.

Yes this is true, it is not documented what to do if it gives an error.
This could be PR.
In general:

if it gives the error step 2 is to use the flash procedure with ymodem
if it gives the error step 1 you have to change TTL adapter or MA5671A

@ErnyTech correct me if I'm wrong

v2/v3 is using sfp ttl adapter. v1 is not. "write a manual version of v3" i mentioned before is means i want a v1 style manual way but using sfp ttl adapter just like v2/v3

yes it too, the only thing it requires extra is tweezers to short-circuit the eeprom.

ok. but please also add a comment on v1/v2 to tell that it's still up to date. so that user can still read the step in v1/v2 as a reference.

No, because there is no point in saying that this procedure is up-to-date if it is not, it is easy to tell if it is by the date it was last modified on github.
In the sense that if I write today that it is up to date and if I edit v3 but not v2/v1 it is no longer up to date but there is a note that it is up to date.

i also noticed that the gpio config in v2 is same as v3. but different than v1. is that because v2/v3 using a sfp tll adapter and v1 using some jump wire to access ttl directly?

is the same between v1/v2/v3, only we realised that some batches of MA5671A have some unconnected pins.

2. a manual version (v1 like) procedure but using sfp tll adapter just like v2/v3

A manual version of V2/V3 is not recommended as you need perfect timing to stop uboot and then there is no serial output until you complete the unlock.
For this reason a manual version has never been made and we made V2 with python scripts and V3 with webserial.

A manual version of V2/V3 is not recommended as you need perfect timing to stop uboot and then there is no serial output until you complete the unlock, it is not easy to do manually.

I would add that a manual version of v2 could seriously damage the ma5671a by causing it to lose its optical calibration.

any news?