Consider lld as linker on ELF (Linux)

Issue description
lld (http://lld.llvm.org/) seems to be "mature enough", these days.
With 5.0 release, it might make sense giving a shot as it's much much faster than ld.bfd (which is still the default on every Linux as far as I can tell) and reasonably faster than ld.gold.
ghc recently tried it with reasonable success
https://www.reddit.com/r/haskell/comments/63y43y/liked_linking_3x_faster_with_gold_link_10x_faster/
I'll be happy to work on this (as I also work on lld/llvm) but first I want to know if it's something reasonable (& probably I'll need some guidance on how to actually implement/try this).

This (and more LLVM-based tools in general) might also make cross-compilation more pleasant, cc @Ericson2314
I'd also love to see lld for Mach-O files on Darwin but it's less clear to me how mature it is in that space. We already use LLVM for everything else there, so it wouldn't be too far off.

Duplicates https://github.com/NixOS/nixpkgs/issues/24744

Not quite a dupe, as that one wants lld to work properly, and this one wants it (as I understand it) in the stdenv as the default linker.

@domenkozar  I don't think it's a dupe.
@copumpkin I'm afraid having lld for Mach-O is still a pipe dream as that bit of the linker is quite unmaintained (last I tried, ~ 1 month ago, lld on Mach-O wasn't even able to self-host with a recent'(ish) OS/toolchain [El Capitan]).
Assuming there's  consensus to go forward, I'd love to discuss next steps.
I'm not a NixOS dev, but given I use it on my main workstation these days (and I love it :)) I thought I could help a bit on this side. What's the best way to do this work incrementally.
I'm very willing to debug any bug/mislinks etc..
Something else I'd like to add.
lld/ELF is quite unique as it has LTO support built-in (differently from ld.gold and ld.bfd which need a plugin to work). As LLVM developer I found out this being a killer feature as you just need to pass -flto to clangand lld can read LLVM bitcode files and call the mover/optimizer transparently. This has the great advantage that, if NixOS decides to support LTO at some point in the future, enabling it it's just a matter of passing a flag to clang (well, two, if you consider-fuse-ld=lld`) instead of building an extra shared library and making sure the plugin bits are passed down to the linker (which may be not particularly pleasant in more complex build systems).

@dcci Would you be up for working on #24744, which is a requirement for making this happen?
It would be great to have good lld support across nixpkgs.

Still important to me.

In 2020 a new Mach-O backend was added. According to the homepage, "[t]he linker supports ELF (Unix), PE/COFF (Windows), Mach-O (macOS) and WebAssembly in descending order of completeness." While I cannot personally attest to it, I have heard good reports of people being able to use LLD for Mach-O. That being said the new hurdle is with Apple moving to AArch64 and requiring that executable binaries be code signed on the new macOS machines. (Which the system linker (ld64) does by default.)

In 2020 a new Mach-O backend was added. According to the homepage, "[t]he linker supports ELF (Unix), PE/COFF (Windows), Mach-O (macOS) and WebAssembly in descending order of completeness." While I cannot personally attest to it, I have heard good reports of people being able to use LLD for Mach-O. That being said the new hurdle is with Apple moving to AArch64 and requiring that executable binaries be code signed on the new macOS machines. (Which the system linker (ld64) does by default.)

As relevant as ever.

This can be done using the linker field of the platform config, which is also set by useLLVM = true and pkgsLLVM. I think we are done here!


Trying to use BRL

When I try to post a SELL order with a range of various amounts, example: 10-100 BRL, it fails to create and throw this error :
"The order you just attempted to create has an amount that is too large. Please try again with a smaller amount."
It happend with USD too.

Edit: It happened on web version, several times.

Yes, this happens because the upper limit of your fiat range, 100 BRL would be ~31K sats, which is higher than our current maximum allowed limit of 20K sats.


Update Universal Actions documentation

Increase userIds limit from 5 to 60
Fix responseType for Adaptive Cards - Fixes #388
Point to Teams documentation for details on UserIds.

Microsoft Reviewers: Open in CodeFlow

Docs Build status updates of commit fa3a75e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

AdaptiveCards/authoring-cards/universal-action-model.md
:warning:Warning

Details

AdaptiveCards/authoring-cards/universal-action-model.md

Line 112, Column 816: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/cards/universal-actions-for-adaptive-cards/work-with-universal-actions-for-adaptive-cards#user-ids-in-refresh' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@matthidinger @dclaux - Can you please review these changes and help merge them?

Docs Build status updates of commit d5143d3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

AdaptiveCards/authoring-cards/universal-action-model.md
:warning:Warning

Details

AdaptiveCards/authoring-cards/universal-action-model.md

Line 24, Column 92: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/cards/universal-actions-for-adaptive-cards/overview' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 114, Column 817: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/cards/universal-actions-for-adaptive-cards/work-with-universal-actions-for-adaptive-cards#user-ids-in-refresh' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 293, Column 3: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/cards/universal-actions-for-adaptive-cards/overview' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9d4564b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/authoring-cards/universal-action-model.md
:bulb:Suggestion

Details

AdaptiveCards/authoring-cards/universal-action-model.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fixed launching Kubeflow dashboard in 0.6.1.

Since Ambassador service has been dropped in Kubeflow 0.6.1, update the instruction to access Kubeflow Dashboard via istio-ingressgateway.

This change is

/lgtm
/ok-to-test

@sarahmaddox Seems the IBM Cloud Private for Kubeflow (getting-started-icp.md) is removed from 0.6 in doc after rearrangement? is there any link for those getting started pages? Thanks.

@sarahmaddox Seems the IBM Cloud Private for Kubeflow (getting-started-icp.md) is removed from 0.6 in doc after rearrangement? is there any link for those getting started pages? Thanks.

@sarahmaddox, please ignore this question. @janeman98 find that under Cloud Installation, and point out for me. Thanks!

@janeman98 You can check the preview by here https://deploy-preview-1105--competent-brattain-de2d6d.netlify.com/. I have a quick checking, seems that's OK.

/approve


[FEATURE] Add a NOTICE.md

Since we distribute a fat jar, we should probably have a NOTICE.md file to include licenses for all third-party dependencies and transitive dependencies.

Since we probably have hundreds of transitive dependencies, we should probably start with just the direct dependencies.
Depending on the number of direct dependencies it would probably be good to use a tool of some sort to help generate the NOTICE.md file, at least the first time around.  Although I tried the oss-review-toolkit and couldn't get it to work with the Spark Connector.


Fix styling in Time of Test modal

Related to https://github.com/CDCgov/prime-data-input-client/issues/139 except more than just the cancel button.
See the latest in the engineering-ready prototype

[ ] Modal doesn't match style in design—should be using the same styling as the USWDS card (used many other places on the site).

[ ] The person's name and the add to queue/cancel buttons should be in the card header, and the buttons should be right-aligned

[ ] The spacing of the buttons is off (they're too close together)

[ ] There is weird indentation on the inputs, dropdowns, and "first test" radio options

[ ] We should not be locking in the year on the "date of most recent prior test" question. 2021 is almost here so the idea that all covid tests have happened in 2020 will only be true for a couple more weeks

@hmyers-cms The prototype doesn't show a Cancel, just a Continue. Did you want Cancel and Save?
On the year, that is done automatically by the date input. We have the minimum date for symptom onset set to Febuary 1 2020 and it realizes the year must be the current year so it fills it. Once we get to 2021 that won't happen.

That's my bad, the prototype should be updated now to have a cancel button—I think it just got lost when porting our designs over from Sketch to Figma


abnormal male germ cell physiology

I ended up annotating failure of male germ cell mitotic arrest for J:205334 to abnormal male reproductive system physiology as we don't have a generic term for male germ cell physiology. Please consider adding a term for abnormal male germ cell physiology and/or abnormal male germ cell mitotic arrest as a child of abnormal male reproductive system physiology. If you add abnormal male germ cell physiology, then the term abnormal male germ cell apoptosis should be made a child of the new term.
thanks,
Sue
Reported by: sbello
Original Ticket: obo/mammalian-phenotype-requests/1780

germ cell - MP:0020363
male germ cell - MP:0020362
female germ cell - MP:0020364


minimal not working example producing error on my PC

Hi all,
MnWE: https://gist.github.com/bmer/e1c637385c543b4f3305
When this script is run, I get the following error:
    infer.propagate()
  File "A:\anaconda\lib\site-packages\numba\typeinfer.py", line 565, in propagate
    raise errors[0]
TypingError: cannot unify array(float32, 1d, C) and array(float64, 1d, C) for 'b'

Failed at nopython (nopython frontend)
cannot unify array(float32, 1d, C) and array(float64, 1d, C) for 'b'

I am using 32-bit Anaconda.
Are you able to reproduce this error too?
Kind regards,
Brian

Yes, I can reproduce this error on 64-bit osx.  Hmm... I thought we have implicit cast between arrays of different dtype

Here's a smaller reproducer:

import numba as nb
import numpy as np

@nb.jit(nopython=True)
def f():
    a = np.zeros(2, dtype=np.float64)
    b = np.zeros(2, dtype=np.float32)
    for i in range(1):
        b = a

f()

It turns out we do not allow implict cast for arrays due to a limitation in the compiler's variable definition rule.  The 1st assignment to b is the only definition.  The 2nd assignment will reuse the 1st variable definition.  If implicit cast is allowed, the loop will change the the array type before it.
However, it is not easy to fix this.  Let say the 2nd assignment creates a new variable for b.  We version the old variable as b.1 and the new variable as b.2.  b.1 and b.2 will have two different dtypes, float32 and float64 respectively.  After the loop, if b is referenced, what should be the type of b?  For strict python semantic, it can be either b.1 or b.2.  Implicitly casting can be dangerous.  Another option is to create a different path of different version of the variable.  But we don't know if it will cause code bloat.
Or, we keep disallow this kind of usage.  I think rpython has the same problem.

Note that the simple workaround is to not reuse the same variable name for different things (which is good practice, by the way, from a software engineering perspective).

@pitrou Could you recommend how I can rewrite this? The use case is actually something like this: https://gist.github.com/bmer/66d2485a4727b9eebc62
which is just the bisection algorithm -- I want to update test_coord based on what a and b are every loop, and then update a or b based on some condition, also every loop.

If your goal is to stay in the float32 realm then test_coord = np.float32(0.5) * (a + b) should probably work. The problem is that floating-point literals (like 0.5) are considered to be double-precision by default.

You also don't need to initialize test_coord before the loop, since you immediately replace the value anyway.

Also, if you want float32 precision for speed reasons, you should make sure to cast a and b to float32 before operating on them (which it sounds like they are not).

Ahh, I see! I was switching to float32 for memory reasons -- there's a memory leak somewhere, but I haven't been able to trace it yet. I suppose what i should do then is go back to float64, and redouble my efforts to trace the memory leak, because its unlikely that switching to float32 or float16 will help?

I don't know if we have support for float16 in Numba.  We'll need to take a look at that, though, since NVIDIA's Pascal GPU architecture will introduce native instructions for manipulating float16.  (Someone realized that neural networks do not need high precision, so being able to natively handle float16 effectively gives you a free 2x in memory bandwidth, register space, and floating point throughput.)

No, we don't support float16 yet.

And x86 doesn't have native float16 instruction support anyway, so the motivation is pretty low.

I think we can close this issue.  We're not prepared to support this kind of type polymorphism in Numba's nopython mode.


CloudFormation: Improve behaviour of StackInstances

Fixes #3425
create_stack_instances() now returns the actual OperationId
create_stack_set() now validates the provided name
create_stack_set() now supports the DeploymentTargets-parameter
create_stack_set() now actually creates the provided resources
delete_stack_set() now validates that no instances are present before deleting the set

Codecov Report

Merging #5731 (6938537) into master (42597de) will increase coverage by 0.00%.
The diff coverage is 99.01%.

@@           Coverage Diff           @@
##           master    #5731   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files         748      748
  Lines       73246    73327   +81
=======================================
+ Hits        70464    70544   +80
- Misses       2782     2783    +1

Flag
Coverage Δ

servertests
36.78% <13.72%> (-0.03%)
:arrow_down:

unittests
96.14% <99.01%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/cloudformation/models.py
98.26% <98.78%> (+<0.01%)
:arrow_up:

moto/cloudformation/exceptions.py
100.00% <100.00%> (ø)

moto/cloudformation/responses.py
96.21% <100.00%> (+0.07%)
:arrow_up:

moto/ec2/models/launch_templates.py
99.27% <0.00%> (-0.73%)
:arrow_down:

moto/cloudwatch/models.py
95.27% <0.00%> (ø)

moto/efs/models.py
91.42% <0.00%> (+0.29%)
:arrow_up:

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


Nilable block params does not work consistently

Input
→ View on sorbet.run
# typed: true

extend T::Sig

sig { params(block: T.nilable(T.proc.void)).returns(String) }
def with_block(block)
  'asdf'
end

T.reveal_type(with_block())
T.reveal_type(with_block do 'asd' end)

class Site
  extend T::Sig

  sig { params(a: T.untyped, block: T.nilable(T.proc.void)).returns(String) }
  def self.where(*a, block)
    "String"
  end
end

T.reveal_type(Site.where(id: 1))
T.reveal_type(Site.where(id: 1) { '23' } )

Observed output
editor.rb:10: Not enough arguments provided for method Object#with_block. Expected: 1, got: 0 https://srb.help/7004
    10 |T.reveal_type(with_block())
                      ^^^^^^^^^^^^
    editor.rb:6: Object#with_block defined here
     6 |def with_block(block)
        ^^^^^^^^^^^^^^^^^^^^^

editor.rb:10: Revealed type: String https://srb.help/7014
    10 |T.reveal_type(with_block())
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  From:
    editor.rb:10:
    10 |T.reveal_type(with_block())
                      ^^^^^^^^^^^^

editor.rb:11: Not enough arguments provided for method Object#with_block. Expected: 1, got: 0 https://srb.help/7004
    11 |T.reveal_type(with_block do 'asd' end)
                      ^^^^^^^^^^^^^^^^^^^^^^^
    editor.rb:6: Object#with_block defined here
     6 |def with_block(block)
        ^^^^^^^^^^^^^^^^^^^^^

editor.rb:11: Revealed type: String https://srb.help/7014
    11 |T.reveal_type(with_block do 'asd' end)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  From:
    editor.rb:11:
    11 |T.reveal_type(with_block do 'asd' end)
                      ^^^^^^^^^^^^^^^^^^^^^^^

editor.rb:22: Revealed type: String https://srb.help/7014
    22 |T.reveal_type(Site.where(id: 1))
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  From:
    editor.rb:22:
    22 |T.reveal_type(Site.where(id: 1))
                      ^^^^^^^^^^^^^^^^^

editor.rb:23: Revealed type: String https://srb.help/7014
    23 |T.reveal_type(Site.where(id: 1) { '23' } )
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  From:
    editor.rb:23:
    23 |T.reveal_type(Site.where(id: 1) { '23' } )

Expected behavior
It should work consistently. Either method(block) works in both cases or not work in both cases.

Using &block works in both cases

Hi Harry, the code you've submitted doesn't work in Ruby and Sorbet correctly complains on it:
[1] pry(main)> def with_block(block)
[1] pry(main)*   'asdf'
[1] pry(main)* end
=> :with_block
[2] pry(main)> with_block()
ArgumentError: wrong number of arguments (given 0, expected 1)
from (pry):1:in `with_block'

(I think you're missing & from def with_block(&block))

Note that the "where" example actually does evaluate, it just doesn't do what you seem to believe it does :-) a will be nil and block will be {id: 1}

(I think you're missing & from def with_block(&block))

Yeah, it should have &. However, I was wondering why it didn't complain for the second case.

Note that the "where" example actually does evaluate, it just doesn't do what you seem to believe it does :-) a will be nil and block will be {id: 1}

Why is a nil and not {id: 1}? Will this work properly? sorbet.run

Why is a nil and not {id: 1}?

because that's how ruby works :-)
[7] pry(main)> def baz(*a, s)
[7] pry(main)*   puts "a = #{a}"
[7] pry(main)*   puts "s = #{s}"
[7] pry(main)* end
=> :baz
[8] pry(main)> baz(1)
a = []
s = 1
=> nil

baz(id: 2)
a = []
s = {:id=>2}
=> nil

Will this work properly?

sorbet & pry seem to agree that "yes" :-)

Ah ok. The catchall *a is always assigned later than the block. Ruby
reorders the params and it kind of make sense. Block is treated
differently, thats why &block work.
Sorry for my confusion!
On Sun, Jun 30, 2019 at 9:00 PM Dmitry Petrashko<EMAIL_ADDRESS>wrote:

Why is a nil and not {id: 1}?
because that's how ruby works :-)
[7] pry(main)> def baz(a, s)
[7] pry(main)   puts "a = #{a}"
[7] pry(main)*   puts "s = #{s}"
[7] pry(main)* end
=> :baz
[8] pry(main)> baz(1)
a = []
s = 1
=> nil
baz(id: 2)
a = []
s = {:id=>2}
=> nil
Will this work properly?
sorbet & pry seem to agree that "yes" :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sorbet/sorbet/issues/1147?email_source=notifications&email_token=AAFH4AIAKYBG7NKVLQHU6S3P5F6OXA5CNFSM4H4LY7Z2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY45SDY#issuecomment-507107599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFH4AM2O4ZP7LOVV2PDOHTP5F6OXANCNFSM4H4LY7ZQ
.

No worries, most folks on our team are past the point when we're no longer smarter than Sorbet. At the same time we get all sorts of weird reports so we ourselves get confused constantly ;-)


Let's create a chat room for contributors

Hi there,
I’d like to propose creating a dedicated chat space (I personally prefer Discord, but I’m open to alternatives) to streamline communication between contributors. As a newcomer to the project, I’m eager to start contributing, but I’m a bit unsure where to begin. I found my way here through the Qlib repository and would love to understand how these two projects are connected. I also have a lot of other questions that might be better addressed in a chat room rather than opening multiple issues and cluttering this repo.
Best regards

Hey, great idea to have a chat room! This is no official chatting channel so far, However, welcome to join LLMQuant Discord (many contributors of RD-Agent are there, LLMQuant is an open source AI community!). You can find both the discussion channels for RD-Agent and Qlib:)

Hello @nicolasakf. Thank you for feedback and your will to contribute to RD-Agent!
For an official chatting room, we are thrilled to announce our official Discord channel here: https://aka.ms/RD-Agent-dc. Welcome to join this channel and we are happy to answer your questions in the chatbox!
For your question about the connection between QLib and RD-Agent.
So far we use QLib as a real world simulator in RD-Agent loop to generate real-world backtesting metrics to the automated generated factors and models.
In near future We are planning to make RD-Agent loop a configurable component in QLib to enable automated quant research on existing configurations.

Hi @peteryang1 and @wanghaoxue0  thank you for your responses! Happy to join both servers. I'll close this issue and address further inquiries through there.


chore(deps): use node 16

Description

Related Issue(s)

Checklist

[ ] This PR includes a documentation change
[x] This PR does not need a documentation change

[ ] This PR includes test changes
[x] This PR's changes are already tested

[ ] This change is not user-facing
[ ] This change is a patch change
[x] This change is a minor change
[ ] This change is a major (breaking) change

Changes made

I'm having problems with the minify step when using together with source-map.
Anybody more experienced with typescript who can help?

@smlgbl can you please rebase your PR?

@smlgbl Could you please check if you have build the bundles correctly?
If not, please run npm run bundle

@smlgbl Could you please check if you have build the bundles correctly? If not, please run npm run bundle

Like I wrote before, everytime I do that, it gives me the following, and it's not minified:
ncc: Compiling file index.js into CJS
ncc: Using<EMAIL_ADDRESS>(local user-provided)
An error occurred while minifying. The result will not be minified.
 40kB  dist/sourcemap-register.js
268kB  dist/index.js
315kB  dist/index.js.map
315kB  dist/index.js.map
623kB  [1138ms] - ncc 0.34.0

Maybe I can give you write access, and you run it?

Thanks for your PR @smlgbl

@smlgbl could you please sign your commits?

@smlgbl could you please sign your commits? Also feel free to squash the commits into one commit.

Sorry, hadn't signed anything before, hence it took a while.

@smlgbl could you please sign your commits? Also feel free to squash the commits into one commit.

Sorry, hadn't signed anything before, hence it took a while.

No worries :hugs:


Providing more info on setting up a Oracle databse connection

For oracle database connections, please add some examples of valid connection configurations. Just sending user to find info about "please refer to the Oracle documentation on Data Source attribute of the ODP.NET connection string" is not enough.  A valid configuration examples and screenshots is needed. how about givin more information about an example Oracle db connection via host,port,service name or sid, with username , password? it would be much appreciated.
Thank you!

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

ID: 7e3f95cc-e110-34ad-49bb-e5b58c8cb892
Version Independent ID: 7e3f95cc-e110-34ad-49bb-e5b58c8cb892
Content: Quickstart: Connect and explore Oracle databases - Azure Data Studio
Content Source: azure-data-studio/quickstart-oracle.md
Service: azure-data-studio
GitHub Login: @tdoshin
Microsoft Alias: timioshin

This is a great suggestion. We will add it to our backlog and prioritize it with our other updates.
#please-close


API Automation -Intermittent Error-jSON Parse error: Unrecognized token '<'

Description: Try to hit the same code couple times, it gives error "ERROR - jSON Parse error: Unrecognized token '<'" intermittently. Some times it get passed as well
CMD:

Code:
api_config = {method:'POST', header:{'Accept': 'application/json','Content-Type':'application/json'}, body:{'name': 'test19','salary': 'test4','age': '31'}};
api http://dummy.restapiexample.com/api/v1/create
js obj = JSON.parse(api_result)
echo api_result
js VarId= obj.data.id
echo VarId
api_config = {method:'GET', header:{'Accept': 'application/json','Content-Type':'application/json'}};
api http://dummy.restapiexample.com/api/v1/employee/VarId
js obj = JSON.parse(api_result)
echo api_result
js MsgFetch= obj.message
echo MsgFetch
Error:
ERROR - jSON Parse error: Unrecognized token '<'

Hi @joginderrohilla sorry I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
I can replicate the issue, it seems to be a problem with this API web service. Consider the following code -
api_config = {method:'POST', header:{'Accept': 'application/json','Content-Type':'application/json'}, body:{'name': 'test19','salary': 'test4','age': '31'}};
api http://dummy.restapiexample.com/api/v1/create
echo `api_result`
js obj = JSON.parse(api_result)

Instead of returning a text response of a JSON structure, the following HTML text is returned with error 'Too Many Requests'. This suggests that the web service owner has not provided enough limits for the service to be freely available to all its potential users. Thus, intermittently when there are many users accessing and exceeding the daily / monthly limit, error happens. Since the response is HTML, trying to do JSON.parse() will throw the expected error.
<!DOCTYPE html>

        <meta charset="utf-8">
        <meta name="viewport" content="width=device-width, initial-scale=1">

        <title>Too Many Requests</title>

        <link rel="dns-prefetch" href="//fonts.gstatic.com">
        <link href="https://fonts.googleapis.com/css?family=Nunito" rel="stylesheet">

                429

                Too Many Requests

Closing the issue for now but please lemme know if you have any input or question.

Adding on, by doing the echo step before the JSON.parse() function, you will be able to see the cause of error above, which is the webservice hitting limit and not willing to respond to the API call.

Doing echo after the error will not be able show the response and cause of error, because program quits by then.

Thanks Ken, I found the limit error.


Added iptables switch

We need to have an iptables switch for various environments

I think ACCEPT policies and flush state should always be executed.

Thanks!


Description on how to run tests

What does this PR do?
Just because I sometimes forget how to run the ddp/non-ddp tests...

Before submitting

[ ] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?
Make sure you had fun coding 🙃

📚 Documentation preview 📚: https://torchmetrics--2500.org.readthedocs.build/en/2500/

it seems fine, but somehow, all CPU tests are failing at the stage of collecting tests...

Not sure why the collection is failing, seems independent of the changes in the PR

Not sure why the collection is failing, seems independent of the changes in the PR

I may not agree as master and other PR proceed and here constantly failed...


Feature request: batch interrogate image to txt file

Hey i think this extension would be really useful if it could do batch work on images, and write the interrogation to txt.

Update to the latest version of extension for new Batch tab! 😀

That's great, some prepend and append options for existing txt files should be added too. Great update.


Windows production mode command

All Submissions:

[x] Have you followed the guidelines stated in CONTRIBUTING.md file?
[x] Have you checked to ensure there aren't any other open Pull Requests for the desired changed?

Type of change
Please delete options that are not relevant.

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

Description
Add production mode command for Windows PC.
We can not use Gunicorn on Windows now, so I add Uvicorn command.
I kept the code for the Linux-based machine because it's not better to use only Uvicorn than Gunicorn.

Close #515.

Changed command list. Please check.


Add defaultMaxRecoveryAttempts to RequestDefaults and use for maxRecoveryAttempts in Request.

Codecov Report

Merging #137 (1aace2c) into master (2a27e1d) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #137      +/-   ##
==========================================
- Coverage   94.65%   94.64%   -0.01%
==========================================
  Files          54       54
  Lines        1684     1682       -2
==========================================
- Hits         1594     1592       -2
  Misses         90       90

Impacted Files
Coverage Δ

Sources/Hyperspace/Request/Request.swift
81.81% <ø> (ø)

Tests/RecoverableTests.swift
94.66% <100.00%> (-0.14%)
:arrow_down:

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 2a27e1d...91f944f. Read the comment docs.


New design for iOS screen

Description
The current screens of the app were designed by... well... a programmer.
It's functional but not beautiful, we can do it better.
Screen of Add Anime:

Criteria of acceptance

Do a new design for the Home Screen
Do a new design for the screen of Add Anime
A link of the design prototype must be available on InVision app or similar. It has to contain the colors used, spacing, font type and size. If some image or icon is used, add it references on the file "License - Use of image"

Can you share the UI spec and more details? if I get some time I can try doing this

First, we need a new design prototype (available on InVision).  There are no specifications, just to be more beautiful than actually is hahaha
Then, in another issue, do the implementation of the design following the criteria of the template
@jayahariv are you a developer, right?
1- I can find a designer to do the design and then I call you to do the implementation
2- If you can do a better view without a designer, go ahead! hahaha (KeralaRescueiOS is a beautiful app 😄)

If you can find a designer, that would be great. I am a developer. Let me know when the design is ready.
Thanks, @MaisaMilena, KeralaRescueiOS was a quick application for the awareness of Flood in our state.


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2024-01-26T01:44:51+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
24229ca4c1d62f17f9289944b2ddb8eceec0624e
link
true
/test integ-pilot-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster


Unicomplex ReportStatus not usable from outside unicomplex.

The response to ReportStatus (a tuple) exposes member types that are private to unicomplex.

This is fixed by https://github.com/paypal/squbs/commit/8d8489cb350b87e079753815be2a27ac3d55b9f7.


feature Replace BITMAP INDEX with INVERTED INDEX

Proposed changes
It's time to replace BITMAP INDEX with INVERTED INDEX, since INVERTED INDEX is production ready in 2.0 version.
For compatibility,

already created BITMAP INDEX can still be used BUT users can not create new BITMAP INDEX any more.
If the user use the USING BITMAP INDEX to create index, a INVERTED INDEX is created actually, to avoid error and get a similar query acceleration.

For the further version, BITMAP INDEX will be completely removed.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


optim with normal RAM?

Hi,
So I spent around 24 hours so far training a model on my style images, got the results out by using the model on eval and so far they're not great. When I use the optim function with the styles however the results are pretty decent, however I am limited by my VRAM which is 6GB as to what size images I can output. Having a lot more RAM available, I was hoping I could do pretty decently sized images, but it seems that I can only get much larger images with eval. Does eval use normal RAM instead of VRAM?
I will continue training my model so that I can use eval in the future, whether I can do larger images with optim or not, but no idea how much more training is required to make it anywhere near a respectable result.
What sort of overall loss value should I be aiming for? Does the number of style images make a difference to what I should expect?

It's hard to say the expected loss value, but ideally the model should approximate Gaty's approach pretty well.


Support load balancing for smtp and imap

This would me help to replace all load balancer with traefik.
An advantage is also you can easily use the build in acme(letsencrypt) generation for smtps and imaps.

I'll close the issue due to inactivity but feel free to re-open if needed.

I have no experience with the go language or this project. Would it be feasible for me to look into this?

Does Traefik have or do not support email protocols?
I commented it this week about balancing for smtp and imap and now I see that there is probably of no support.
Such a feature would be interesting for this application https://github.com/hardware/mailserver/issues/83#issuecomment-368767275

@MichelDiz Traefik supports HTTP(S) only.


Add method to correct for Magnetic Deviation

@DanielG86 suggested adding a means for correcting the compass readings for Deviation.  After the built-in magnetic calibration routine runs, there will likely be some residual error. If this error is repeatable then we should be able to correct for it, similarly to using a deviation table with a physical compass.
Depending on the type of error, the correction might be one or more of:

a constant offset
an offset that varies sinusoidally around the complete arc

In the example main.cpp I inserted two transforms: AngleCorrection and CurveInterpolator The first is the most straightforward - it adds a constant offset in radians to the heading value before it is output as Magnetic Heading. The second transform allows one to define an arbitrary set of points that are used to re-map the input to the output values, using interpolation.
Both transforms are able to be edited using the sensor's web interface.

In the example main.cpp I inserted two transforms: AngleCorrection and CurveInterpolator The first is the most straightforward - it adds a constant offset in radians to the heading value before it is output as Magnetic Heading. The second transform allows one to define an arbitrary set of points that are used to re-map the input to the output values, using interpolation.
Both transforms are able to be edited using the sensor's web interface.


Add spec for else without rescue and ensure

Follow up of https://github.com/ruby/ruby/commit/140512d2225e6fd046ba1bdbcd1a27450f55c233.
This PR fixes the following error.
% ../mspec/bin/mspec language/rescue_spec.rb
$ ruby /Users/koic/src/github.com/ruby/mspec/bin/mspec-run
language/rescue_spec.rb
ruby 2.6.0dev (2018-03-23 trunk 62902) [x86_64-darwin17]

1)
The rescue keyword will execute an else block even without rescue and
ensure ERROR
SyntaxError: (eval):3: else without rescue is useless
        else
        ^~~~
/Users/koic/src/github.com/ruby/spec/language/rescue_spec.rb:200:in
`eval'
/Users/koic/src/github.com/ruby/spec/language/rescue_spec.rb:200:in
`block (3 levels) in <top (required)>'
/Users/koic/src/github.com/ruby/spec/language/rescue_spec.rb:199:in
`block (2 levels) in <top (required)>'
/Users/koic/src/github.com/ruby/spec/language/rescue_spec.rb:13:in
`<top (required)>'
[\ | ==================100%================== | 00:00:00]      0F
1E

Finished in 0.007407 seconds

1 file, 40 examples, 70 expectations, 0 failures, 1 error, 0 tagged

Thank you for the new spec


java.lang.UnsatisfiedLinkError: couldn't find "libflutter.so" with ArCore

I'm trying to use ArCore SceneForm in my Flutter app.
The app throws java.lang.UnsatisfiedLinkError in debug mode (flutter run), and it runs fine when I use command flutter build apk then flutter install apk.
Here is my Flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.4.17-pre.10, on Mac OS X 10.14.3 18D109, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 10.1)
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
[✓] Android Studio (version 3.3)
[✓] Connected device (1 available)

! Doctor found issues in 1 category.

If you want any information, please let me know.

Try using the --target-platform flag to specifically select the 32-bit android-arm platform.
By default flutter run will create an APK targeting ARM64 if the attached device is an ARM64 device.  If the app includes other 32-bit libraries, then the process will be unable to load both the 64-bit version of libflutter.so and the 32-bit libraries.  You'll need to build an app containing the 32-bit version of libflutter.so

@phucgaoxam is it still an issue?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


[T4A4][T17-B3]Musa

Ready for review

@ghurabah93 Some comments added. Please close the PR after reading comments. Well done.


event parameter request - codeunit 368 "Format Document" - SetSalesPurchaseLine() - OnAfterSetSalesPurchaseLine() - CommentLine: Boolean

Please add CommentLine as an argument to subscribers of OnAfterSetSalesPurchaseLine(). We can instead check if FormattedWhatever = '' and avoid applying our own formatting if so, but it would be cleaner and more self-explanatory to just get the CommentLine as already determined by yourselves ;-)

So:
        OnAfterSetSalesPurchaseLine(
          CommentLine, Quantity, UnitPrice, VATPercentage, LineAmount, CurrencyCode,
          FormattedQuantity, FormattedUnitPrice, FormattedVATPercentage, FormattedLineAmount);

and
    [IntegrationEvent(false, false)]
    local procedure OnAfterSetSalesPurchaseLine(CommentLine: Boolean; Quantity: Decimal; UnitPrice: Decimal; VATPercentage: Decimal; LineAmount: Decimal; CurrencyCode: Code[10]; var FormattedQuantity: Text; var FormattedUnitPrice: Text; var FormattedVATPercentage: Text; var FormattedLineAmount: Text)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


X_LINK_DEVICE_NOT_FOUND

The corresponding error message is shown below.

Actually, I can find and use OAK-D in the docker according to the official python examples.
The corresponding information about the environment is listed as follows:
Docker container environment: docker run --gpus all -it --ipc=host --net=host --privileged -e DISPLAY=unix$DISPLAY -v /tmp/.X11-unix:/tmp/.X11-unix:rw -e NVIDIA_DRIVER_CAPABILITIES=all introlab3it/rtabmap:focal /bin/bash
Ubuntu: 20.04-LTS
Device: OAK-D-PRO-W
Could you please help me to solve this problem? Many thanks for helping me!

@RendongPi
Hi. Sorry for late.
This code assumes you installed every dependencies outside the Docker, not inside the Docker.
Did you follow the how to install to install everything properly?
Thanks.

@RendongPi
Hi. Did you install every dependencies outside Docker and try again?
Thanks.


Support for EL9

Hi
we would really like to install Zammad on a EL 9 Server, since they support all the Distributions packager.io supports it would be very nice if you could also support EL9. Apparently there are already some things in place, but packaging fails because there is a curl-minimal installed which conflicts with curl: https://packager.io/gh/ombre8/zammad/builds/5/logs/centos-9
As I can't find curl as a (build)-dependency in Zammad, I assume this is something that is built in pkgr?
I'm happy to help, but I somehow can't figure out where to start debugging ;) Any help is appreciated. Thanks

@crohr any hint at all?

@ombre8 I'll have a look, but curl-minimal is not part of the default build dependencies so it must come from somewhere else.

hi @crohr it's the other way around, the EL9 image has curl-minimal pre-installed, and curl seems to be a build dependency, but installing curl fails since it conflicts with the pre-installed curl-minimal

@ombre8 I've force-installed curl in the base image with --allowerasing option. Could you retry and see if it works now?

Nice thanks, now it fails with a missing build pack: https://packager.io/gh/ombre8/zammad/builds/11/logs/centos-9

Closing, as this has been working since quite a while now.


Attempt to overwrite LibEGL.so ...

Hello
Unpacking libglvnd-dev:arm64 (1.0.0-2ubuntu2) ...
dpkg: error processing archive /tmp/apt-dpkg-install-UEUP9Y/072-libglvnd-dev_1.0.0-2ubuntu2_arm64.deb (--unpack):
 trying to overwrite '/usr/lib/aarch64-linux-gnu/libEGL.so', which is also in package libmali-rk-utgard-450-r7p0:arm64 1.6-1ayufan10

Can you do something about it ?

Install libmali-rk-dev.

This dosnt solve problem.
As Im not able to insta qtbase-dev whitch requires mesa gles whitch requires glvnd-dev and this conflicts.


Scientific experiment

See chapter 5

LGTM. We have to resolve the TODO's, but other than that I approve the overal structure!


Use configuration from ProviderSettings in OAuth2AuthorizationServerConfigurer

Fix #182 issue
(rebased version of https://github.com/spring-projects-experimental/spring-authorization-server/pull/184)
ping @jgrandja

Thanks for all the updates @florianberthe ! FYI, I added a polish commit around the tests. This is now in master.

Thanks for all the updates @florianberthe ! FYI, I added a polish commit around the tests. This is now in master.


contact us page

contact us page front-end only, with some styling missing, some one should please check it out and try to fix it, its not in django format, only html and css, just copy the folder and run it on your local machine

Contact -Us page and logic is already done


Removed transition and reverese transition durations on BeamPage

When BeamPage is BeamPageType.noTransition

Seems the test is failing, but not in the code I modified.

Seems the test is failing, but not in the code I modified.

This is fixed in PR #674

This was also merged in #676 so I don't understand why we have 2 :man_shrugging: anyway, we can close.

This was also merged in #676 so I don't understand why we have 2 🤷♂ anyway, we can close.

It was merged on other branch (v1.6) which I didn't know if was gonna merge this branch (master).
Should I remove this PR then?

@busslina Ah, sorry, my bad. I will reopen and merge.
@Sten435 It should have no conflicts with #674 so all good.


Normalize field state

As form state. Undefined --> null

[ ] normalize
[ ] correct types
[ ] docs
[ ] $state to $
[ ] update examples


Activate rails 3 country select

Hi,
I have got an error with a Rails app version 3.2.22 with formtastic 3.1.3 + formtastic_bootstrap 3.1.1 + country_select 2.5.1 with country input : wrong number of arguments (4 for 0).
I check the code in lib/country_select.rb and I see this code :
if defined?(ActionView::Helpers::Tags::Base)
  require 'country_select/country_select_helper'
else
  require 'country_select/rails3/country_select_helper'
end

But when I test in rails console I have :
Loading development environment (Rails 3.2.22)
[1] pry(main)> defined?(ActionView::Helpers::Tags::Base)
=> "constant"

The rails select helper loaded seems to be the wrong one for my rails version. Perhaps the test must be updated?
Best Regards
Luc Donnet

Perhaps this way :
if Rails::VERSION::MAJOR = 3
Do the old thing
else
Do it the new way
end

hmm, yeah perhaps it should be updated. I have a vague memory of choosing not to go down that path, but now I can't remember why.
I'm wondering what your Gemfile looks like. What I've found is some gems open up that module/class path (ActionView::Helpers::Tags::Base) regardless of which version of rails it's running on.
So you can get up and running right now, I'd recommend adding require: false to your Gemfile and creating a rails initializer that looks like:
require 'countries'
require 'sort_alphabetical'

require 'country_select/version'
require 'country_select/formats'
require 'country_select/tag_helper'

#if defined?(ActionView::Helpers::Tags::Base)
  #require 'country_select/country_select_helper'
#else
  require 'country_select/rails3/country_select_helper'
#end

A pain, but at least you can get going until I can fix this.

Thanks for your help and your time!

I've had the same issue and I created a rails initializer like you suggested @scudco, which worked for me. Thank you!

Not an ideal situation here, but I'm closing this because I won't be spending any time fixing Rails 3 issues since a workaround exists.


command line arguments during build are too long

Consider using a response file instead of generating a large command.
Referencing an asp.net core app already breaks the by causing an
C:\Users\martin.ullrich\.nuget\packages\codegeneration.roslyn.buildtime\0.4.11\build\CodeGeneration.Roslyn.BuildTime.targets(20,5): error MSB6003: The specified task executable "dotnet" could not be run. The filename or extension is too long

The command-line for the "GenerateCodeFromAttributes" task is too long. Command-lines longer than 32000 characters are likely to fail. Try reducing the length of the command-line by breaking down the call to "GenerateCodeFromAttributes" into multiple calls with fewer parameters per call.

@AArnott I've created a replacement msbuild target to work around this (it creates a response file and reads the output file). do you think it is better to keep the msbuild task dll around or maybe migrate to pure msbuild?

Thanks for the report and diagnoses. I'm inclined to keep the MSBuild task around. Mostly in the hopes of one day MSBuild actually supporting doing all this in-proc so I don't have to spawn dotnet.exe.


Branch 154885009

Everything passing except for Android Demo app, going to try again and see if it's consistent.  If android passes quickly, I'll merge.
@tensorflow-jenkins test this please

cc @caisq or @yifeif if they have any idea why the Android build is failing with
18:05:18 INFO: Loading package: tensorflow/c
18:05:18 ERROR: /workspace/tensorflow/core/BUILD:1363:1: in cmd attribute of genrule rule //tensorflow/core:version_info_gen: $(PYTHON_BIN_PATH) not defined. Since this rule was created by the macro 'tf_version_info_genrule', the error might have been caused by the macro implementation in /workspace/tensorflow/tensorflow.bzl:1188:7.
18:05:18 ERROR: Analysis of target '//tensorflow/examples/android:tensorflow_demo' failed; build aborted.
18:05:18 INFO: Elapsed time: 2.669s

The only likely culprit is the .bazel.rc commit, but I can't see why that would cause this to fail.

Is it related to this PR? #9547

Maybe, but that PR passed before. @andrewharp thinks it is related to stand alone strategy

Going to try to revert 50aff7a2735ac595199abb67f4c96fa39193007c for now so we can make progress, and then let someone else figure out how to get around this sandboxing issue.

@tensorflow-jenkins test this please


Add --stack-prs option to the request-review command

This option should only be applicable when requesting reviewing of a patch series.
The default behavior of the command should be to create a single branch and cherry pick the entire patch series into that branch and then create a pull request from that branch.
However, if the user includes the --stack-prs option it should instead create a stack of branches, one for each of the patches, and then it should create a pull request for each of those branches. However, the pull requests should be to destined for the previous branch. To understand this lets look at an example where we have a patch series of A, B, C.
We would have to create a branch for A. Let's call it BA based on origin/main and a pull request destined for origin/main. Then we would have to create a branch for B. Let's call it BB based on BA, and a pull request destined for BA. Then we would have to create a branch for C. Let's call it BC based on BB, and a pull request destined for BB.
Note: Ideally the pull request names would have in index as part of them indicating to peer reviewers what order these pull requests need to be reviewed and integrated.
I do have some concerns about this approach as it seems brittle. If a peer reviewer reviews and integrates a patch in the wrong order what happens? It also seems like the once the first patch is merged in and that branch gets cleaned up that the second pull requests will be invalid and therefore it won't work. This needs further investigation, thinking about and discussion. But people are saying this is what graphite.dev does, which is a Git Stacked Branch tool rather than a Patch Stack tool. But, it is also worth noting that they have an extension for GitHub.

Ok, I did some testing with this stacked pull request just in terms of GitHub.
It looks like when you merge the patch PR in. What happens is that the PR for the second that had a merge destination of the branch of the first which is now deleted, ends of falling back to origin/main. This happens to be what we would want. However, it shows two commits in that branch now. I am assuming because I clicked the rebase & merge button so the shas of the commit in main doesn't match that of the one in the branch. That seems a little strange though as normally commits get collapsed by Git if they are same commit just with a different sha.

I think the other question I have is do all Git hosting providers work this way with their PR handling? If not it is yet another point of brittleness. Also it seems a bit like a support nightmare as well.

Also if a reviewer merges a PR in the wrong order it breaks everything. Yet another point of brittleness.

You could create the other PRs as draft which might address the issue with the wrong order.

Draft PRs in GitHub have merge disabled. So you would have to manage switching them from Draft to not draft after each is merged. This seems like madness to me. It seems like you might as well just open the PRs after the dependent one has been merged upstream.

There could be a hook when you integrate to undraft the next PR, but this sounds kind of convoluted.

Another approach would potentially be to have each branch include the dependent patches in it, but have it be targeted for origin/main. This has the same problem of one the first patch PR is rebased and merged the following PRs have conflicts and have to be manually resolved. However, it does kind of solve the merging out of order issue because if someone reviewed and merged the 3rd PR it would include the commits of the first two.
I am not suggesting this is really any better. I think this is still fundamentally horrible in my mind.
It feels like we are fighting against the core model of the tooling instead of just accepting the constraints of the current modeling of the tooling.

I don't think this idea follows the core model of of git-ps "or stacked-diffs" in general. I think there should really be 2 main ways of interacting with e.g. github etc.

As we currently do - isolate a single patch, and push as a PR.
Isolate a series of patches, and push as a single PR.

The use-case you describe above can be handled by 2.. or if the developer really wants to simply submitting each in order, and waiting until each have been integrated before submitting the next.
So if the aim is to break a larger PR into smaller reviewable units, simply use 2 (multiple) commits and encourage people to look at the commit breakdown to understand the PR.

That is how it behaves today. At least with the version in main.
My instinct with this is that people who want stacked branch based PRs actually want a fundamentally different tool than GitHub and they should likely be looking at something else like graphite.dev which provides an experience similar to that.

That is how it behaves today. At least with the version in main.

Yes. Just noticed that recently, it makes the tool essentially perfect for my mental model / use-case.
Stacking branches to implement the "patch-stack / stacked-diff" model is a mistake IMHO. Unless you have some tool that magically manages github directly so all the issues you identified above don't exist.. then fine.. but that then becomes a rather custom github-only tool.. and I much prefer the way you have gone.. which implements this model really well for any "pull-request" based service.

We have decided not to do this until we discover a way that works well with the model GitHub, Bitbucket, or GitLab have. At which point this issue can be reopened.


MaskTextfiel don't update

After validating input and set into TextField-Value the TextField seems not to be Rerendered

On Server-Side the Value is completely not updating

Did you base it on TextField?  I made a recent change to TextField validation to make server-side work
https://github.com/limefrogyank/BlazorFabric/commit/d87b6c5c19d670245e629db4f6bff1367e2a265f#diff-918822d2ff796e2b5e5cb9ffdbb1e5d2R248

Hey @limefrogyank,
Yes I base on Textfield and also merged your stuff into the feature branche befor but it's not worked either.
But I found one error with validation wich I will fix as fast as possible.
Than I will have another deep look into the Texfield-component because of two-way-binding and trigger right Events in right order on textarea and input field and will maybe restructure it a little bit because my feeling is, that it isn't stable enough for use in other components.
I also want to come to the state that Mask-Textfield can inherts from Textfield complete without side-effects.


setup for react and Material-ui

Description
Initial setup for react and material ui
How Has This Been Tested?
Local server ran and there page content was visible in the browser.
Checklist:

[x ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added necessary inline code documentation
[ ] I have added tests that prove my fix is effective and that this feature works
[ ] New and existing unit tests pass locally with my changes

@festus should place redux-thunk in middleware folder. Then import it in store.js


'initialchanges'

test

@cruz-doneduard your test worked! Please can you fork the repository into your account so you make a copy? 😃


YARN-10332: RESOURCE_UPDATE event was repeatedly registered in DECOMM…

2117

 Quality gate: Passed
 Everything looks good
Details


server: create api to query persisted stats by date range

PR: cockroachdb/cockroach#69238
From release notes:
Added a new endpoint /_status/combinedstmts to retrieve persisted and in-memory statements from crdb_internal.statement_statistics and crdb_internal.transaction_statistics by aggregated_ts range. The request supports optional query string parameters start and end, which are the date range in unix time. The response returned is currently the response expected from /_status/statements. /_status/statements has also been updated to support the parameters combined, start, and end. If combined is true, then the statements endpoint will use /_status/combinedstmts with the optional parameters start and end. [#69238][#69238]

@awoods187, I assigned this to SQL since it seems to be about SQL observability, but if this is a new api that we want to auto-generate docs for, we might need to move it to OX Observability where that's handled.
cc @mikeCRL and @itsbilal, who understand the autogeneration approach.

@kevin-v-ngo can confirm but this looks like the new SQL observability items

Yes, this endpoint was created to support the new persisted stats experience in the console by SQL Obs team. The endpoint queries the new virtual table (combining in-memory and persisted stats) and allows filtering based on a specified start and end time.
Users should be able to use this endpoint for programatic access as well so we should probably have docs for this. What's the process for auto-generated docs?

The "Cluster API" docs are the only ones auto-generated right now, as far as I know. For that, we use go-swagger to generate an Open API v2 spec from specially formatted code comments, then use a few other tools to transform the spec into Open API v3 and add example request for each endpoint, and finally use a tool called redoc to build it all as user-facing docs. The process is described here: https://cockroachlabs.atlassian.net/wiki/spaces/PROD/pages/1851719975/API+reference. However, it's just a first iteration, and I believe @itsbilal has a task to document how all of this works on the Eng side and map out a consistent process for other APIs. Bilal, can you please chime in?
cc @ianjevans, who probably needs to be involved for cloud apis as well. And @mikeCRL, who set this all up with Bilal.

Thanks for tagging me @jseldess ! I worked on the current process that you described (openAPI v2 spec generation etc), but given some of its shortcomings, we'd want to eventually move over to having an openAPI v3 spec as the single source of truth, off of which we auto-generate some server-side API code. For other APIs, it would make sense to start off with that approach to begin with. The migration of the existing CockroachDB API is very much a TODO item in the server team's roadmap at some point, and unfortunately I'm no longer in that team to be able to help out with that. But I can guide whoever takes that or any related work up.

cc'ing @erademacher who worked on adding the Cloud API autogen docs.


Don't automatically recapture mouse in borderless window mode

Which versions is window_get_showborder() available in?

function is stubbed but it produces an error anyway on LTS :-\

v_v

We could roll this in using a try...catch?

ehhh inclined to wait until someone reports as a problem ? 🤷♀ my thinking is you need to re-implement dragging in borderless window mode if you want it (i think many games omit it?) and if the window position is updating in realtime the auto-recapture behaviour is undesirable. edge on an edge.

lmk if you want it and i'll add the try/catch

Yeah let's wait until it comes up. Easy enough to implement later.


Add powershell3 support

DIdn't this deserve a shoutout in the release notes for 3.0.1? Or is it not fully tested yet? 😄

Yeah I know I added support to the tracer (what writes the executed lines), but I am not very interested in supporting profiling (the code that processes the trace to get overview of the trace) in PS3, I will write a different tracer for the CC, so I don't collect stuff that the AST did not identify. I was wrong earlier about being able to get rid of AST in CC. CC needs to know both code that was invoked, but even more importantly code that can be invokes but was not. We won't get that from the tracer as it only tracks invoked code.
There is more unsupported stuff I am using, the whole profiler is now based on scriptblock ids which we don't have in PS3 (could be emulated with file:line:column, but we won't be able to use it for unbound scriptblocks).


[Android] Why does the app need location access?

I got the upgrade to 2.7.0 earlier (I was on 2.5.x, if not older, before that) and upon launching the app, it asked me for location access. Why does it need to access my location?

It's actually for Bluetooth to be able to detect nearby devices. Somehow Google requires location for this to work.

Does it now use bluetooth to share files?

It uses Bluetooth to discover other devices, but still uses Wi-Fi to transfer the files.

I see. Like nearby share. Thank you for the clarification.
Good thing landrop works without bluetooth and location access though.


[WIP] Swift 2.0 Support

This currently targets Xcode 7.0b2. Aside from removing the dependency on Box, I haven't made any changes other than what was required to get it working with Xcode 7 and Swift 2.

@mattprowse Many thanks to your continuous work for keeping up with the changes! :sparkling_heart: Since I've started a new PR: #35, which takes over this PR, I'm gonna close this out. :bow:

Oh, I'm really pleased that my work on this ended up getting used! I'm using ReactiveTask in a project and I was impatient to move to Swift 2, so I decided to just fork and do it myself. The fact that I was then relying on my fork made it really easy for me stay motivated about keeping up with the changes to Swift 2.


Allow users to self-enroll

@alejandrogarciasalas

There is no good way to check if the user self-enrolling is using their SID and not someone else's. So we decided make this an instructor task.


Update sources

:robot_face: Updating sources to the latest version.
Commits touching OCaml packages:

ocamlPackages.timed: init at 1.1
ocamlPackages.bindlib: init at 6.0.0
ocamlPackages.dedukti: init at 2.7
ocamlPackages.pratter: init at 2.0.0
ocamlPackages.lambdapi: init at 2.2.1
Merge pull request #190856 from r-ryantm/auto-update/nerdctl

nerdctl: 0.22.2 -> 0.23.0

Merge pull request #190856 from r-ryantm/auto-update/nerdctl

nerdctl: 0.22.2 -> 0.23.0

Merge pull request #190856 from r-ryantm/auto-update/nerdctl

nerdctl: 0.22.2 -> 0.23.0

Merge pull request #190856 from r-ryantm/auto-update/nerdctl

nerdctl: 0.22.2 -> 0.23.0

Merge pull request #190856 from r-ryantm/auto-update/nerdctl

nerdctl: 0.22.2 -> 0.23.0

Merge pull request #190856 from r-ryantm/auto-update/nerdctl

nerdctl: 0.22.2 -> 0.23.0
Diff URL: https://github.com/NixOS/nixpkgs/compare/c00a58c5a8ee11941dbf91c82d254c1992d12806...faa93c4e19e79e7a6de31d6d3492b8f00760ca82

Lots of inits so unless they are broken on a specific version it's probably fine to merge

I ran it manually.

unless they are broken on a specific version it's probably fine to merge

that's what I was thinking.


[FZ-Editor] Custom-Layouts > Auto-select 'Create new custom' and set focus on the edit-button

If no custom layout exists the entry "create new custom" could be automatically selected.
And instead of the button 'Apply' the button 'Edit selected layout' should be selected.
Expected behavior
If I switch to "custom Layouts" and there is no Layout exists, the entry "create new custom" is selected. That indicates that clicking on "edit selected Layout" creates a new one.
Without existing custom layout or when the item "create new custom" is selected the button Apply shouldn't work. Instead only the button Edit selected layout should be selected and should working.

Actual behavior
The entry "create new custom" is not selected. But I can create a new Layout while only clicking at "Edit selected layout".
It also makes no sense, that in this situation the button Apply is selected and works!

Since in Editor v2 the UI will change significantly, closing this as won't fix for this version of the Editor.


Can't create/import pcm from csv, html or wikipedia

On OpenCompare website, when clicking on these links, there is no modal opened to allow import.
In the console we've got errors on loading files:
" GET http://opencompare.org/templates/modalHtmlImport.html 404 (Not Found) "

Yep !
I fixed the issue in the master branch.
I'm currently refactoring the API of OpenCompare for the browser extensions.
Once the API is stable, I will deploy the new version with the fix.

Ah nice ! It's hard to live a day without beeing able to import a pcm...

I know ! I'll try to release it today ;)

The fix is deployed !


change thread label format (#496)

change thread label format  from<EMAIL_ADDRESS>to <user>@<host>.<thread-name>(<thread-id>)

@MaxMoto1702 why are you need that?

@MaxMoto1702 up. Lets discuss a bit and merge this PR

I looked to #496, so lets merge it


chore(main): release 1.3.3

:robot: I have created a release beep boop
1.3.3 (2024-10-06)
Bug Fixes

stop propagation of click events from widget to prevent moodle dialog from closing (#16) (89e4c5a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Annoto/moodle-local-js/releases/tag/1.3.3 :sunflower:


Update 09-aks-cluster.md

variable name fix

Thank you @lfrost1, we are including a variation of this fix in #66 (specifically, changing the variable name when setting the cert.

Going to now close as the fix was merged in #66.  Again, thank you for helping identify this, @lfrost1!  Your PR was much appreciated.


fix: Add Error When No Route is Available

Task linked: CU-8669dafxz No route for the currently selected pair of tokens

Task linked: CU-860pp6kn5 USDC USDT max swap


Compilation on OSX fails due to boost dependency

Trying to build the program from source code seems to fail on OSX (10.15), due to an issue related to the boost dependency licudata. This issue has been reported by others in separate applications as well: Ref. 1, Ref. 2.
Based on some of the comments from those issues, I was able to resolve the installation here by removing the following lines from the CMakeLists.txt file (148–150, primarily the Boost_USE_STATIC_LIBS setting):
if(NOT BUILD_SHARED_LIBS)
	set(Boost_USE_STATIC_LIBS ON)
endif()

However, this seems like a bit of a hack, so I'd like to report the issue here for your review as well.

The trouble here is that the boost libraries do not contain the correct dependency list in their cmake config file. In theory, cmake config files should specify what is needed to correctly link a library. In the case of boost this means you need the library icudata but also the path to this library.
BTW, the lines you commented out were in fact a hack from my side. They are gone in the latest version.


Need time information more directly connected to model input/output

Need simulation date/time ranges passed as explicitly to the simulation engine.  This simulation range should then be forwared to the forcing objects to configure the amount of forcing they load.  Also, the output should use this time information to write an actual time series.
Current behavior
The only time input information is nested in the forcing configuration.  The time range is used by the forcing objects, but is not tightly coupled to the simulation loop, which at this point is a hard coded loop over a fixed number of time steps (assumed hourly).  The outputs are relative to these loop iterations, and are not easily associated with the data/time the output is simulated for.
Expected behavior
Configuration passes a simulation start/end time, as either explicit dates, or start date + simulation time, or some other means.  The simulation start and end times are then passed to the forcing object construction to read the required forcings.  The main simulation loop needs to iterate over this time period, and needs to add the date/time as the output index.

Closing via PR #190 merged into master.


[BUILD] - POM conflates mesos and mesos-utils versions

LGTM, any objections @connordoyle, @brndnmtthws ?

:shipit:, see #406

Cool, this shall close https://github.com/mesos/chronos/issues/406


Add "Standalone Components" section to "Installing Kubeflow"

This PR is intended as an alternative to the proposal from @andreyvelich in https://github.com/kubeflow/website/pull/3724.
The goal of this PR is to reduce confusion, while still meeting the request of @andreyvelich to tell users that some components may be installed separately.
The main change is the addition of a "Standalone Components" section on the "Installing Kubeflow".
This  "Standalone Components" section has a table which lists if each component of Kubeflow is able to be deployed "standalone", that is, without the rest of the platform.
/assign @kubeflow/wg-notebooks-leads @kubeflow/wg-pipeline-leads @kubeflow/wg-training-leads @kubeflow/wg-data-leads @kubeflow/release-team @kubeflow/kubeflow-steering-committee @kubeflow/wg-deployment-leads
Screenshot
Here is a screenshot of the updated "Installing Kubeflow" page:

Closing as https://github.com/kubeflow/website/pull/3724 has been merged.


Google cloud services

Add support for some popular Google Cloud services.
This PR provides the following integration with Google Cloud services:

Authentication via our configuration file and allow to inject a GoogleCredentials CDI bean.
Storage: allow to inject a Storage CDI bean.
BigQuery: allow to inject a BigQuery CDI bean.
PubSub
Spanner: allow to inject a Spanner CDI bean.
Firestore: allow to inject a Firestore CDI bean.

These extensions make the libraries works in Quarkus and manage the needed dependencies.
Documentation is still missing but erything works including native.
Some works still needs to be done around gRPC.
There is already an extension proposal for Firebase here https://github.com/quarkusio/quarkus/issues/8671 maybe we can also provide it but it seems very specific (Firebase admin server).
This is a move of the Google Cloud Services extension from Quarkus core repo to the Quarkiverse, see the original PR here: https://github.com/quarkusio/quarkus/pull/9745

Let's make it io.quarkiverse.googlecloudservices.
That way, we can easily guess the groupId from the repo name. Might be handy for scripting.

Let's merge ? 👍

Great news, good job guys!
Please share link to find the docs for usage, (when available).


Does not work with React

Tried to implement this cursor in my react app but it wasn't compatible. Could this be fixed?

+1
I can't make it work with Gatsby.

Think this means we will have to fix this ourself :/

Think this means we will have to fix this ourself :/

Just put css files from the cdn and everything should work.

Any examples on how to get it working on React/Next.js

`import kursor from "kursor"
new kursor({
type: 2,
color: "#B98FFF",
removeDefaultCursor: true,
})`

Hi guys, I also had the same problem while using this dependency with Next.js.
In Next.js we don't have index.html in public so we have to create a file in the pages folder called "_document.jsx" or "_document.js".
Read this from the documentation to get a better understanding: https://bit.ly/3rMUWQL
"_document.jsx" code:
`import Document, { Html, Head, Main, NextScript } from 'next/document'
class MyDocument extends Document {
static async getInitialProps(ctx) {
const initialProps = await Document.getInitialProps(ctx)
return { ...initialProps }
}
render() {
return (

<script dangerouslySetInnerHTML={{__html:new kursor({ type: 1 })}} />

)
}
}
export default MyDocument
`
Then in your global styles add the following property to remove the default cursor:
cursor: none;

Hi guys, I also had the same problem while using this dependency with Next.js.
In Next.js we don't have index.html in public so we have to create a file in the pages folder called "_document.jsx" or "_document.js".
Read this from the documentation to get a better understanding: https://bit.ly/3rMUWQL
"_document.jsx" code:
import Document, { Html, Head, Main, NextScript } from 'next/document'

class MyDocument extends Document {
  static async getInitialProps(ctx) {
    const initialProps = await Document.getInitialProps(ctx)
    return { ...initialProps }
  }
  render() {
    return (
      <Html lang="en">
        <Head>
            <link rel="stylesheet" href="https://unpkg.com/kursor/dist/kursor.css" />
        </Head>
        <body>
          <Main />
          <NextScript />
        </body>
        <script src="https://unpkg.com/kursor"></script>
        <script dangerouslySetInnerHTML={{__html:`new kursor({ type: 1 })`}} />
      </Html>
    )
  }
}

export default MyDocument

Then in your global styles add the following property to remove the default cursor:
cursor: none;

Hope this helps

It's so easy to use like cutting a cake.
#import
import Kursor from "kursor";
import 'kursor/dist/kursor.css'
you can change the type as you want.
new Kursor({
type: 2
})

my cursor only show on the hero section only not to all body by default it should be on body but not work


UniqueTagList: Remove unused constructors #539

Part of #539

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/1] UniqueTagList: Remove unused constructors

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/542/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v2
@Zhiyuan-Amos submitted v2 for review.

[v2 1/1] UniqueTagList: Remove unused constructors

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/542/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.


This is a bug in JAX's shape-checking rules; please report it!

I'm getting This is a bug in JAX's shape-checking rules; please report it!.
The function that creates this error is the following:
@functools.partial(jax.pmap, axis_name="batch")
def accuracy_and_loss_fn2(params, images, labels):
  logits = resnet_apply(params, images)
  loss = cross_entropy_loss(logits=logits, labels=labels)
  accu = jax.lax.pmean(jnp.argmax(logits, axis=1) == labels, axis_name="batch")
  return accu, loss

When I replace the jax.lax.pmean by a jnp.mean it does not cause the error.  It's a ResNet variant which does work in other calls, including multi-host.
def cross_entropy_loss(*, logits, labels):
   logp = jax.nn.log_softmax(logits)
   loglik = jnp.take_along_axis(logp, labels[:, None], axis=1)
   return -jnp.mean(loglik)

For the offending line, I have also tried jax.lax.all_gather as well as jax.lax.psum and they result in the same, in case that helps.

Okay cool, presumably if we replace apply_resnet(...) with just a constant array of some shape we'll see the same shape error.
It does help that psum raises the same error!
One last question, if it's easy to answer: what's the shape of jnp.argmax(logits, axis=1) == labels here? We might be able to repro with just that being a constant array, so we can clear out everything except the psum/pmean.

It should be (B, C) where B is batch-size and C = 1000 in my case, but I will add a logging and run again just to be sure!

I am confused. I did tmp = jnp.argmax(logits, axis=1) == labels and put print(tmp.shape) just before, and am getting (2,).

Argh, miss-clicked.

Perfect, ty!

This error looks like it's saying we're creating an XLA add instruction on boolean inputs (which isn't supported). (But it's evidently not going through the same code path as jax.lax.add(jax.numpy.array([False]), jax.numpy.array([True])), which throws a better error.)

@skye had a really smart guess: we lower jax.core.unit to Pred[], so maybe there's a partial-eval-inserted unit cropping up here. (It'd be nice to revise things so we lower units to nothing at all in HLO... one of the many things on the todo list.)
I haven't checked yet whether this is pod-specific; on one hand it doesn't sound pod-specific, on the other hand I'm sure we have test coverage for this kind of use case...

@jekbradbury is right, in this case a boolean array is being passed to pmean. I think we should improve the error message, but still make it an error and require an explicit cast in this situation. However, there is an argument to be made for providing some kind of implicit casting functionality (either in psum or even a separate API endpoint?), since that's what numpy does, and even though psum is in lax there isn't a numpy equivalent.
Any thoughts? Otherwise I'll just improve the error message for now.

Oh, hah, I totally didn't get what @jekbradbury 's point was. Sorry! I get it now.
I think we should handle booleans (by promoting to a non-boolean type, like we do for analogous situations in lax_numpy.py).

Ok! I actually like that better, it just felt weird to have some lax primitives do casting and others not. But this way is more useful!

Yeah, it is a bit weird, but I think most lax primitives aren't user-facing. The bool->int promotion in a reduce-sum seems pretty harmless, and this is a nice canonical use case too.

(Also, as per the title of this issue, actually we should do dtype checking in the abstract eval / shape rule, but if we do any it's not currently matching XLA's constraints.)

Wait are you sure the dtype isn't a red herring here? As I mentioned, I get the same error if I do jax.lax.all_gather instead of psum, which I should reasonably expect to work, no?
Regarding casting, my understanding is "jax is numpy" and so I would expect implicit cast of bool to float, which I use all the time in numpy.

jax.numpy approximates NumPy semantics. jax.lax in general does not. Although there's certainly a case to be made that the user-facing parallel operations should promote like NumPy, even if other lax operations do not.

@lucasb-eyer I'm surprised you're getting exactly the same error with an all-gather, since the error specifically talks about add: Expected element type in shape to be arithmetic type for operation add; got PRED. I haven't had a chance to dig into this myself with your code yet though.
Either way, I was able to repro this error with a small psum(bool) unit test, so there's definitely an issue there. I was gonna start by fixing the unit test, then rerunning your code with the fix and see what happens. It's also possible the unit test will expose the more general issue if there is one.

all_gather is just a psum so it makes sense it'd be the same error.

Makes sense, and you are right, when I explicitly cast to float before, the code works as expected in all cases.


Get-PnPTerm: switch "-ParentTerm" not documented and not working

This feature was requested in #468 and subsequently implemented.
However, the implementation is not working and not documented. I can confirm what @smchargue wrote on 8 JUL 2022.

Reopened the previous issue.


UML-1992: Invalid LPA reference numbers should be reported to the user

Purpose
Briefly describe the purpose of the change, and/or link to the JIRA ticket for context
Fixes UML-1992
Approach
Reset the flag to 'false' for api and actor app in local, dev, demo environments
Checklist

[X] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] New event_codes have been documented on the wiki page
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

Using an invalid reference

valid reference but no matching data

LPA Already added


net: ipv6ZoneCache not updated on cache misses

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.11.1 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/michael/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/michael/go"
GOPROXY=""
GORACE=""
GOROOT="/home/michael/sdk/go1.11.1"
GOTMPDIR=""
GOTOOLDIR="/home/michael/sdk/go1.11.1/pkg/tool/linux_amd64"
GCCGO="/usr/bin/gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build554331710=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Creating a listener on a link-local IPv6 address fails on bootup for ≈ 1 minute longer than necessary. This code:
ln, err := net.Listen("tcp", "[fe80::ba27:ebff:fe3b:11b0%eth0]:80")
if err != nil {
  log.Println(err)
}

results in:
1970/01/01 01:00:05 listen tcp [fe80::ba27:ebff:fe3b:11b0%eth0]:80: bind: cannot assign requested address
2018/11/01 18:19:37 listen tcp [fe80::ba27:ebff:fe3b:11b0%eth0]:80: bind: invalid argument
2018/11/01 18:20:11 listen tcp [fe80::ba27:ebff:fe3b:11b0%eth0]:80: bind: invalid argument
[…]
2018/11/01 18:21:11 now listening on [fe80::ba27:ebff:fe3b:11b0%eth0]:80

Notably, listening fails for about 1 minute, which is the time it takes for zoneCache to be invalidated.
This is on a Raspberry Pi 3B+, where the eth0 interface is connected via USB (as is the case on all Raspberry Pi models).
What’s happening here is the following (in order):

System boots, I install a listener on all available IP addresses, including ::1, which fills the zoneCache.
The network interface eth0 is discovered once the USB hub is probed.
eth0 comes up, assigning itself a link-local IPv6 address.
At this point, listening fails with “cannot assign requested address”, because the link-local IPv6 address is marked as tentative while the kernel performs duplicate address detection (DAD).
After a second or so, DAD finishes and the address becomes available.
At this point, listening still fails, this time with “invalid argument”, because the bind call’s sin6_scope_id parameter is set to 0, because the zoneCache does not contain the correct entry for the eth0 interface. This is because zoneCache was filled before that interface existed. In strace, you’ll see: [pid  1716] bind(9, {sa_family=AF_INET6, sin6_port=htons(80), inet_pton(AF_INET6, "fe80::ba27:ebff:fe3b:11b0", &sin6_addr), sin6_flowinfo=htonl(0), sin6_scope_id=0}, 28) = -1 EINVAL (Invalid argument)
After (at most) 1 minute, the zoneCache is invalidated and updated.
Only now does listening succeed. In strace, you’ll see: [pid  1715] bind(7, {sa_family=AF_INET6, sin6_port=htons(80), inet_pton(AF_INET6, "fe80::ba27:ebff:fe3b:11b0", &sin6_addr), sin6_flowinfo=htonl(0), sin6_scope_id=if_nametoindex("eth0")}, 28) = 0

I suggest we update the zoneCache upon cache misses, which fixes this issue. I can send a CL to do that.

It's fine if this proposal focuses on the invalidation using the signal of operation malfunction; otherwise, we should take another approach that working together with the routing information base inside the kernel for accuracy and correctness (upon the fate/nature of IP routing; we cannot avoid any micro routing loop at any time, anywhere.)
Also we need to fix x/net/internal/socket too.

I can send a CL to do that.

Thanks.

Please add Updates golang/go#28535 to the CL description for x/net.


Add Angular Example

Improve the documentation by adding an Angular sandbox.

I'll take that.

@ealush
I see that no one did this issue.
So I would like to take it :)

Yes! @MichalPorag ,go ahead!
You can open up a new sandbox, you cnr use the styling from my react demos. Let me know if you need anything from me. 🙏

Hey @ealush!
It's still open? I can take it

Is this issue still open? PR #761 appears to address the issue's description, but I see it's using Angular v12. I'm interested in creating a sandbox using v16/standalone components if this issue is still open.

@edezekiel, yes definitely.
For reference, I don't know if you've seen this talk by Ward Bell, but it might be a good start

Ok great! And yes that's an awesome talk. I'll incorporate some ideas from that talk in the sandbox example.