data integrity test: SIMBAD-resolvable sources?

do we want to test whether sources are simbad-resolvable?

Pretty sure tests will fail — still working on this.

we want to instead do this for the ingest_sources function. will make the tests take too long, and tests will fail if SIMBAD is down.

SIMBAD as part of a set of monthly tests?

todo: make this a ~monthly check.

In SIMPLE, we called it scheduled_checks.py so it's not auto-discovered by test.
Here's also the action file to use for inspiration. This runs every 30 days:
https://github.com/SIMPLE-AstroDB/SIMPLE-db/blob/main/.github/workflows/scheduled-tests.yml

thanks for all the comments!

Tried running in CodeSpaces but this doesn't run as-is, so some more iteration is needed; see the comments above.

Pushed a fix to the various issues I noticed above. The script runs successfully for me in CodeSpaces.

todo: rename the files to be the source names


Filename in title of ui modal

I added a change to show the file name. With the change it would look like this.

Side question: How about  adding those files to .gitignore:
    modified:   js/backend-elm.js
    modified:   js/public/client-elm.js

Solves #171

Hi @andys8. Thanks for the contribution. The two files will be patched when I release a new version.


Python: add BoundMethodValue

Closes https://github.com/github/codeql-python-team/issues/64
I had some trouble convincing myself that adding class BoundMethodValue extends Value was the right thing to do. However, after I adjusted our existing tests, I could see that adding this class did not change the results, so I guess it's all good.
I'm still not super happy that forall(FunctionValue fv, CallNode call | fv.getACall() = call | call.getFunction().pointsTo(fv)) doesn't hold, but today is no the day for trying to change that :stuck_out_tongue:
Note: This PR is not complete yet, since I wanted to show that the tests actually pass BEFORE adding BoundMethodValue -- and this was the easiest way to do so without messing with Jenkins :sweat_smile:

Ready to go :snail: :rocket:
As a reviewer, you might be asking yourself, why isn't BoundMethodValue a subclass of CallableValue? It is not, since it would imply you could use the extra methods that CallableValue defines:

neverReturns
getScope
getParameter
getParameterByName
getArgumentForCall
getNamedArgumentForCall

but they simply don't give meaningful results. We could add special case handling for BoundMethodValue in these methods, but I'm not too keen on that.
I would much rather get to the bottom of this modeling problem, so we're not in a situation where forall(FunctionValue fv, CallNode call | fv.getACall() = call | call.getFunction().pointsTo(fv)) doesn't hold.
Doesn't look good with getArgumentForCall
With this query
from CallableValue callable, CallNode call, string found_arg
where call = callable.getACall() and
    (exists(call.getAnArg()) or exists(call.getArgByName(_))) and
    if exists(callable.getArgumentForCall(call, _))
    then found_arg = "getArgumentForCall exists"
    else found_arg = "getArgumentForCall does not exists"
select call.getLocation().getStartLine(), callable.toString(), call.toString(), found_arg

we get the results

Line
callable
call
found_arg

19
Function f
ControlFlowNode for f()
getArgumentForCall exists

21
Function f
ControlFlowNode for f()
getArgumentForCall exists

23
Function f
ControlFlowNode for Attribute()
getArgumentForCall exists

23
Method(Function f, C())
ControlFlowNode for Attribute()
getArgumentForCall does not exists

24
Function C.n
ControlFlowNode for Attribute()
getArgumentForCall exists

24
Method(Function C.n, C())
ControlFlowNode for Attribute()
getArgumentForCall does not exists

25
Function C.n
ControlFlowNode for Attribute()
getArgumentForCall exists

33
Function D.foo
ControlFlowNode for Attribute()
getArgumentForCall exists

34
Function D.foo
ControlFlowNode for Attribute()
getArgumentForCall exists

37
Method(builtin method append, List)
ControlFlowNode for Attribute()
getArgumentForCall does not exists

37
builtin method append
ControlFlowNode for Attribute()
getArgumentForCall exists

38
Builtin-function len
ControlFlowNode for len()
getArgumentForCall exists

notice that getArgumentForCall doesn't give any results with BoundMethodValue (the ones that say Method(..., ...))
Doesn't look good with getParameter either
from CallableValue callable, CallNode call, int i
where call = callable.getACall()
select call.getLocation().getStartLine(), callable.toString(), call.toString(), i, callable.getParameter(i).toString()

gives

Line
callable
call
i
getParameter(i)

19
Function f
ControlFlowNode for f()
0
ControlFlowNode for arg0

19
Function f
ControlFlowNode for f()
1
ControlFlowNode for arg1

19
Function f
ControlFlowNode for f()
2
ControlFlowNode for arg2

21
Function f
ControlFlowNode for f()
0
ControlFlowNode for arg0

21
Function f
ControlFlowNode for f()
1
ControlFlowNode for arg1

21
Function f
ControlFlowNode for f()
2
ControlFlowNode for arg2

23
Function f
ControlFlowNode for Attribute()
0
ControlFlowNode for arg0

23
Function f
ControlFlowNode for Attribute()
1
ControlFlowNode for arg1

23
Function f
ControlFlowNode for Attribute()
2
ControlFlowNode for arg2

23
Method(Function f, C())
ControlFlowNode for Attribute()
-1
ControlFlowNode for arg0

23
Method(Function f, C())
ControlFlowNode for Attribute()
0
ControlFlowNode for arg1

23
Method(Function f, C())
ControlFlowNode for Attribute()
1
ControlFlowNode for arg2

24
Function C.n
ControlFlowNode for Attribute()
0
ControlFlowNode for self

24
Function C.n
ControlFlowNode for Attribute()
1
ControlFlowNode for arg1

24
Method(Function C.n, C())
ControlFlowNode for Attribute()
-1
ControlFlowNode for self

24
Method(Function C.n, C())
ControlFlowNode for Attribute()
0
ControlFlowNode for arg1

25
Function C.n
ControlFlowNode for Attribute()
0
ControlFlowNode for self

25
Function C.n
ControlFlowNode for Attribute()
1
ControlFlowNode for arg1

33
Function D.foo
ControlFlowNode for Attribute()
0
ControlFlowNode for arg

34
Function D.foo
ControlFlowNode for Attribute()
0
ControlFlowNode for arg

notice that the indexes for getParameter are off by one for the results with BoundMethodValue (the ones that say Method(..., ...))

I just remembered how CallableValue is defined (see below), so trying to avoid BoundMethodValue being a subclass of CallableValue didn't work 😅 (made an edit to the comment above as well).
@tausbn what do think we should do to move this forward? Try to patch up the CallableValue methods for BoundMethodValue?

If it's not too cumbersome, I think it would be worth the effort to support the necessary methods for this to be a CallableValue. Mostly I think this is important for the consistency of the API. It's weird that the getFunction part of a CallNode that you know is callable does not have a CallableValue, even though we know exactly what is called here (this is essentially what your forall is saying).

Right. I'll see if I can tweak it to work :wink:

this is essentially what your forall is saying

forall(FunctionValue fv, CallNode call |
    fv.getACall() = call
|
    call.getFunction().pointsTo(fv)
)

Not quite, the problem my forall is illustrating is that fv.getACall() = call isn't the inverse of call.getFunction().pointsTo(fv), which is surprising -- and problematic when you end up wanting the same behavior as getACall but the function is only referenced, not called.
As I talked about above, we could alter Value.getAReference so the following would hold (which it doesn't right now).
forall(FunctionValue fv, CallNode call |
    fv.getACall() = call
|
    call.getFunction() = fv.getAReference()
)

However, I'm not very tempted to do so, as I would rather not introduce an inconsistency between getAReference and pointsTo :grimacing:

I worked on this again now, but needed to change so many things that I created a new PR https://github.com/github/codeql/pull/3407


[ Document ] Add daily build status @open sesame 02/05 10:55

Add daily build status in readme
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #939. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051037220.024228096008301-d574498a930762b5113b0d47fa7f83a697cfbd07/report/misspelling_result.txt.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051037220.024228096008301-d574498a930762b5113b0d47fa7f83a697cfbd07/.

I think it would be good to add a daily-build badge.
http://nnsuite.mooo.com/nntrainer/ci/daily-build/badge/daily_build_badge.svg

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051055570.60605192184448-d574498a930762b5113b0d47fa7f83a697cfbd07/report/misspelling_result.txt.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051057270.53625202178955-76b2886671e23761dc3f9cc166207a16aa67a8e5/report/misspelling_result.txt.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051100040.19646596908569-42b452ac0e4468cc36a679f9a3f1c4bb7229e377/report/misspelling_result.txt.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051101530.072590112686157-ccf6b2d89e0c509305a1244c6a93d5e96044a948/report/misspelling_result.txt.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051103240.4031229019165-8adbac5a398686aefd3abfd306c5da4c51c737db/report/misspelling_result.txt.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/939-202102051105000.69201707839966-fe8ce054bfdb5ec55b182cb1809751fb30886f20/report/misspelling_result.txt.

I think it would be good to add a daily-build badge.
http://nnsuite.mooo.com/nntrainer/ci/daily-build/badge/daily_build_badge.svg

done! Thanks.


Incorrect syntax highlighting in Vim script when omitting space in list of string

Vim syntax highlighting of vim script is incorrect with:
let l = ['a','b','c']

Syntax highlighting is correct if we add spaces after commas as in:
let l = ['a', 'b', 'c']

I understand that vim syntax highlighting is heuristics based
and works well in most common cases. But perhaps we can
make it work in this particular case too.
Steps to reproduce
$ echo "let l = ['a','b','c']" > bug.vim
$ vim --clean bug.vim

And observe the incorrect syntax highlighting.
Screenshots

Environment (please complete the following information):

Vim-8.2.430
Ubuntu 18.04.4

Well seen!
Its are considered as marks from this line which I don't understand.


tests.py34.test._exit_test.ExitTest / tests.py27.test._exit_test.ExitTest

From what I can tell, none of the individual tests are failing, but the sum of the tests is taking too long.  Each test has to go through the c-core memory-leak output because of a server side memory leak with outstanding calls.
https://grpc-testing.appspot.com/job/gRPC_pull_requests/10800/
++ dirname /var/local/git/grpc/tools/run_tests/run_python.sh
+ cd /var/local/git/grpc/tools/run_tests/../..
++ realpath -s py34/bin/python
+ PYTHON=/var/local/git/grpc/py34/bin/python
++ pwd
+ ROOT=/var/local/git/grpc
+ /var/local/git/grpc/py34/bin/python /var/local/git/grpc/src/python/grpcio_tests/setup.py test_lite
I0711 18:29:19.132553797   12219 ev_epoll_linux.c:84]        epoll engine will be using signal: 36
D0711 18:29:19.132598090   12219 ev_posix.c:106]             Using polling engine: epoll
I0711 18:29:19.132627237   12219 init.c:193]                 grpc_init(void)
running test_lite
I0711 18:29:20.860613670   12219 metadata_array.c:42]        grpc_metadata_array_init(array=0x7fa22273f298)
I0711 18:29:21.971739218   12219 metadata_array.c:42]        grpc_metadata_array_init(array=0x7fa2223cae68)
I0711 18:29:21.981306756   12219 metadata_array.c:42]        grpc_metadata_array_init(array=0x7fa22235eef8)
I0711 18:29:22.012173253   12219 metadata_array.c:42]        grpc_metadata_array_init(array=0x7fa222373ac0)
I0711 18:29:22.033328916   12219 metadata_array.c:42]        grpc_metadata_array_init(array=0x7fa22237e688)
Running       _exit_test.ExitTest.test_in_flight_partial_stream_stream_call
[95mTesting gRPC Python...
[0m[92mSUCCESS       _exit_test.ExitTest.test_in_flight_partial_stream_stream_call
[0mRunning       _exit_test.ExitTest.test_in_flight_partial_stream_unary_call
[92mSUCCESS       _exit_test.ExitTest.test_in_flight_partial_stream_unary_call
[0mRunning       _exit_test.ExitTest.test_in_flight_partial_unary_stream_call
[92mSUCCESS       _exit_test.ExitTest.test_in_flight_partial_unary_stream_call
[0mRunning       _exit_test.ExitTest.test_in_flight_stream_stream_call
[92mSUCCESS       _exit_test.ExitTest.test_in_flight_stream_stream_call
[0mRunning       _exit_test.ExitTest.test_in_flight_stream_unary_call
[92mSUCCESS       _exit_test.ExitTest.test_in_flight_stream_unary_call
[0mRunning       _exit_test.ExitTest.test_in_flight_unary_stream_call
[92mSUCCESS       _exit_test.ExitTest.test_in_flight_unary_stream_call
[0mRunning       _exit_test.ExitTest.test_in_flight_unary_unary_call
[92mSUCCESS       _exit_test.ExitTest.test_in_flight_unary_unary_call
[0mRunning       _exit_test.ExitTest.test_poll_connectivity
[92mSUCCESS       _exit_test.ExitTest.test_poll_connectivity
[0mRunning       _exit_test.ExitTest.test_poll_connectivity_no_server
[92mSUCCESS       _exit_test.ExitTest.test_poll_connectivity_no_server
[0mRunning       _exit_test.ExitTest.test_poll_connectivity_no_server_terminate

How'd this get closed? The entire test is still skipped, right? And the message in the skip points directly to this issue?


Even with --title-format, title shows "GoTTY" briefly

The first request GET / returns this:
<!doctype html>
<html>
  <head>
    <title>GoTTY</title>
    <style>body {position: absolute; height: 100%; width: 100%; margin: 0px;}</style>
  </head>
  <body>
    <script src="hterm.js"></script>
    <script src="gotty.js"></script>
  </body>
</html>

This briefly shows GoTTY in my browser title even though I use --title-format. I bet we can make this use --title-format as well?

Hi @ahmetalpbalkan, thanks to the feed back.
Yes, currently the first HTML file sent by the server is a static one stored as a "bin-data". I think I can convert it a template file and fill with the value of --tiltle-format.

Fixed in the master branch. The index file is delivered with the title.


fix for windows path to show in project tree across platforms correctly

This PR fixes # https://github.com/microsoft/azuredatastudio/issues/10986
Converting the relative path to platform safe path before using in uri.
Opened a separate bug for MAC --> SSDT.

Coverage decreased (-0.06%) to 34.36% when pulling eec6f45e64ab0b9a470dcdcbd1b1130a7f3b7967 on bug/WinToMacPath into 87a7c659f23bea483966eba6c3aa54b71b74aaf8 on main.


ADU

The graphics still use the old ADU initialism instead of Device Update for IoT Hub

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

ID: 3e0faeec-4d01-8971-e784-bde6356d5501
Version Independent ID: 0e6ef221-089c-eb86-12c0-12b3c96c1bae
Content: Introduction to Device Update for Azure IoT Hub
Content Source: articles/iot-hub-device-update/understand-device-update.md
Service: iot-hub-device-update
GitHub Login: @vimeht
Microsoft Alias: vimeht

@jlorich Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate. We will update you accordingly.

@vimeht Can you please check and add your comments on this doc update request as applicable

This PR fixes the Import image: https://github.com/MicrosoftDocs/azure-docs-pr/pull/155435

This issue was resolved with Andrew's PR
#please-close


Wrap python code into a tf operation

I have described my issue in the keras group issue 6163 but was re-directed here. I am trying to wrap a non-tf function (actually a class) into a tf operation to use in a custom keras layer. When using py_func it always returns  "ValueError: None values not supported." Is it generally possible to wrap an arbitrary piece of python code, which takes a list of numpy arrays as input and delivers a list of numpy arrays , into a tf op?

I have written custom code.
TensorFlow installed from binary.
TensorFlow version is 1.0.1
No CUDA/cuDNN used.
Currently no GPU.

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!
That said, yes it seems tf.py_func is what you want.

Did you find a solution to this?


Add Interactive formatting extension and add nuget package metadata

This PR

Adds FsSpreadsheet.Interactive as a project that provides Formatters for:

FsWorkbook
FsWorksheet

Adds nuget package metadata to ExcelIO and CsvIO project, as well as the newly added project.

The formatters look like this:

Some dummy data:
let dummyWorkbook = new FsWorkbook()
let dummyWorksheet1 = FsWorksheet("dummyWorksheet1")
let dummyFsCells =
    [
        FsCell.createWithDataType DataType.String 1 1 "A1"
        FsCell.createWithDataType DataType.String 1 2 "B1"
        FsCell.createWithDataType DataType.String 1 3 "C1"
        FsCell.createWithDataType DataType.String 2 1 "A2"
        FsCell.createWithDataType DataType.String 2 3 "C2"
        FsCell.createWithDataType DataType.String 3 1 "A3"
        FsCell.createWithDataType DataType.String 3 2 "B3"
        FsCell.createWithDataType DataType.String 3 3 "C3"
    ]
dummyWorkbook.AddWorksheet dummyWorksheet1 |> ignore
dummyWorksheet1.AddCells dummyFsCells |> ignore

Larger example (read from excel file):

This is a POC, so ill stop here, but here are some notes for further improvement:

Global mutable config settings for formatters. For example maxRows and maxCols, since some sheets might be so large taht you really do not want to render it completely. See Deedle.Interactive for reference.
Once styling can be read from e.g. excel, we can directly map those styles to the html, effectively rendering a preview that looks exactly like opening the file in excel.


Cant install cookbook via supermarket

When trying to install this cookbook it fails with "ERROR: Chef::Exceptions::MetadataNotFound: No metadata.rb or metadata.json found for cookbook apache2 in /Users/karnold/src/SFMOMA-DevOps/chef-repo/cookbooks"
Manually downloading the cookbook and extract it and it does not complain however.
Chef Development Kit Version: 0.9.0
chef-client version: 12.5.1
berks version: 4.0.1
kitchen version: 1.4.2

And the full output from my terminal
it101:chef-repo karnold$ knife cookbook site install apache2
Installing apache2 to /Users/karnold/src/SFMOMA-DevOps/chef-repo/cookbooks
Checking out the master branch.
Pristine copy branch (chef-vendor-apache2) exists, switching to it.
Downloading apache2 from Supermarket at version 3.1.0 to /Users/karnold/src/SFMOMA-DevOps/chef-repo/cookbooks/apache2.tar.gz
Cookbook saved: /Users/karnold/src/SFMOMA-DevOps/chef-repo/cookbooks/apache2.tar.gz
Removing pre-existing version.
Uncompressing apache2 version 3.1.0.
Removing downloaded tarball
No changes made to apache2
Checking out the master branch.
ERROR: Chef::Exceptions::MetadataNotFound: No metadata.rb or metadata.json found for cookbook apache2 in /Users/karnold/src/SFMOMA-DevOps/chef-repo/cookbooks

This seems like more a question for Chef/Supermarket. We do include a metadata with the cookbook (as you can see when you download the tarball with knife cookbook site download apache2

Im curious as to why it only happens with this package.  The issue hasnt surfaced for any other cookbooks ive tried to install via this method

I can't say, other than I can tell you that we don't do anything special when we upload to the supermarket.
You could try adding -VV to the command to see if you get more information.

From what I can tell so far this seems to be an issue revolving around repositories that provide a metadata.json instead of the metadata.rb that is prefered by chef.  Still searching for empirical evidence.


ArgumentError: Package ZygoteRules does not have ChainRulesCore in its dependencies

While executing sherrington_kirkpatrick.ipynb from notebooks folder, I get this error. Does the manifest file require some changes or is there some guide to resolve this issue?
Screeshot attached with more log information:

Review tracking issue: openjournals/joss-reviews#5364

Noted. I'll look into it when I can. Not sure though if it's an issue with my pkg?

Maybe. I checked that based on this, it might be useful to upgrade some existing ZygotRules based functions to ChainRules.

Maybe. I checked that based on this, it might be useful to upgrade some existing ZygotRules based functions to ChainRules.

The thing is, I am actually using ChainRulesCore and not ZygoteRules. So I'm not sure where this error comes from. Has this occured in other notebooks, too? What Julia version are you running?

@timbode It occurs on all the notebooks. I'm using Julia 1.8.5 from VS Code extension.


Added tests for json serialization

#709 works thanks to this, so this PR is no longer needed

I agree, I think #709 covers this.


winget CLI crashes if I try to install, show, search, list, upgrade, uninstall, export

Brief description of your issue
Faulting application name: winget.exe, version: 1.17.2203.10001, time stamp: 0x622a8652
Faulting module name: WindowsPackageManager.dll, version: 1.17.2203.10001, time stamp: 0x622a8613
Exception code: 0xc0000005
Fault offset: 0x0000000000089bf3
Faulting process id: 0x0x2A4C
Faulting application start time: 0x0x1DA1DF86B8EFFE4
Faulting application path: C:\Program Files\WindowsApps\Microsoft.DesktopAppInstaller_1.17.10691.0_x64__8wekyb3d8bbwe\winget.exe
Faulting module path: C:\Program Files\WindowsApps\Microsoft.DesktopAppInstaller_1.17.10691.0_x64__8wekyb3d8bbwe\WindowsPackageManager.dll
Report Id: c4ef8304-6816-4d4d-ab91-cced96aa25c0
Faulting package full name: Microsoft.DesktopAppInstaller_1.17.10691.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: winget
-
-

1000
0
2
100
0
0x8000000000000000

4073

Application
Stians-Laptop

-
winget.exe
1.17.2203.10001
622a8652
WindowsPackageManager.dll
1.17.2203.10001
622a8613
c0000005
0000000000089bf3
0x2a4c
0x1da1df86b8effe4
C:\Program Files\WindowsApps\Microsoft.DesktopAppInstaller_1.17.10691.0_x64__8wekyb3d8bbwe\winget.exe
C:\Program Files\WindowsApps\Microsoft.DesktopAppInstaller_1.17.10691.0_x64__8wekyb3d8bbwe\WindowsPackageManager.dll
c4ef8304-6816-4d4d-ab91-cced96aa25c0
Microsoft.DesktopAppInstaller_1.17.10691.0_x64__8wekyb3d8bbwe
winget

Steps to reproduce
winget upgrade
Expected behavior
That it tries to upgrade my programs
Actual behavior
Immidiatly crashes (it starts to print the spinner, and exists back to the shell)
Environment
C:\Users\StianSkjelstad\Downloads>winget --info
Windows Package Manager v1.2.10691
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.2715
Package: Microsoft.DesktopAppInstaller v1.17.10691.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Out of the box, fresh laptop, joined azure ad, "Microsoft 365 Buisness Standard", and did perform windows update. "Windows Store" icon also went missing.

You’re using an extremely old version of winget and the CDN for that version has been deprecated. I suggest waiting for the store to automatically update Winget or manually updating it yourself

I am probably stuck with that old version, since windows store does not open. Doing wsreset.exe does this:

I will try to do full reset of windows, and this time start windows store and winget before doing windows update.

I performed "Reset Windows" and used winget before doing windows update. It now works as expected and I now have "App Installer" in the list of installed programs. So this means that if you do a windows update before using winget on a fresh Windows installatation, you might end up stuck with a broken version?
The reason for Windows Store missing is likely that the computer is setup with a work account "Microsoft 365 Business Standard" so it missing Intune etc.

FYI, an easy way to update winget without using the Store is to do Add-AppPackage https://aka.ms/GetWinGet. The aka.ms link points to the latest msixbundle available for download from here on GitHub.

So there is a work-around :-) But still out-of-the-box installation of Windows-11 should just work, so something needs to change in what Windows Update installs since the current experience is broken and all howto's you find searching online does not work.

This was related to CDN issues and has been resolved in WinGet 1.7.


Suggest a content animation on landing page

I suggest to add animation to contents when page is loading.
Ex:

Vector image Slide from right
Top content slide from left
grow in and out the "become a member" button

@EANimesha Could you please suggest a one!
cc: @Bawanthathilan @Kushiya @Sewvandiii

Check this out > lottiefiles.com
Bit advance but shape and nice 😍

Anyone working on this ?

Anyone working on this ?

seems like no one

seems like no one

@Shehanka to whom we can assign this ?? who had experience with this ?

@Shehanka to whom we can assign this ?? who had experience with this ?

@nisalrenuja 👀

@nisalrenuja 👀

@Bawanthathilan Can you ?

@Bawanthathilan Can you ?


fix "flat-square" example badge

Logo wasn't matching the used parameter.
fixes #1264

Thank you!


Fix early parse abort in case of missing RMC sentence entries

This fixes #21.
@generationmake can you please test and see if it fixes your problem?

Codecov Report

Merging #25 (25970ba) into master (48ee402) will increase coverage by 3.43%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
+ Coverage   93.54%   96.98%   +3.43%
==========================================
  Files           5        5
  Lines         217      199      -18
==========================================
- Hits          203      193      -10
+ Misses         14        6       -8

Impacted Files
Coverage Δ

src/ArduinoNmeaParser.h
100.00% <ø> (ø)

src/nmea/GxRMC.cpp
95.69% <90.90%> (+6.60%)
:arrow_up:

src/ArduinoNmeaParser.cpp
94.87% <100.00%> (-0.13%)
: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 48ee402...25970ba. Read the comment docs.

@aentinger
fix works. can be merged into master.


Control image pull secrets

Add a ENV var to control image pull secrets:
Eg:
imagePullSecrets:
- name: dockercred

see https://github.com/oceanprotocol/operator-engine/commit/a87ef55d15fc34babeee215576662769a8cc17bf


Refactoring old apps to add new best practices

Many apps contain old constructs that are no longer needed, but might be picked up as good practices by new contributors (or, in some cases, will prevent the app from working at all - though unlikely because generally backwards compatibility is preserved). So, refactoring those would help.
Old and outdated practices include:

[ ] Using Printer(ffs(string, o.cols)) instead of PrettyPrinter(string)
[ ] Redefining __init__ in class-based apps just to set self.menu_name
[ ] Not defining callback in function-based apps as a separate function
[ ] Defining the main_menu inside init_app and using a global to access it from callback
[ ] Adding ["Exit", 'exit'] entries to Menu elements

All of these ought to be grepped for, then removed/replaced.

I should be able to help out with this.

Thank you!


Themes

Added dark mode and a basic theme toggle

Good work
I rebased to latest update, there might be some new styling to add themes to (maybe ctrl+F color:  will help)
No errors and things work smoothly, but lots of deprecation warnings from SASS when using theme
frontend-1  | DEPRECATION WARNING: Sass's behavior for declarations that appear after nested
frontend-1  | rules will be changing to match the behavior specified by CSS in an upcoming
frontend-1  | version. To keep the existing behavior, move the declaration above the nested
frontend-1  | rule. To opt into the new behavior, wrap the declaration in `& {}`.

I think we should also implement saving theme choice like you left a comment on; user auth uses localStorage so we could probably do something similar with theme selection. Also having a more definitive light-dark switch or dropdown selector for the theme choice instead would be better UX


Support Vite 5.0

Hi,
First of all thank you for this super easy-to-use plugin. It has been a joy to use it without any issues and it empowers builds of privacy.sexy.
Vite 5 is released and this package requires maximum of Vite 4.x as peer-dependency.
Can we update it to support Vite 5 so it does not block users' migration?

Absolutely! We're working on an upgrade but are blocked by some backwards compatibility issues between the Vite 4.0 and 5.0 Plugin TS type

How about bumping the major version on @modyfi/vite-plugin-yaml and guide users to right major versions based on which version of Vite they run on README.md? I think it would be clear and user-friendly way matching developer expectations if it's not possible to break backwards compatibility.

I saw that someone started publicly working on this as #24 - is there anything else that would help get vite 5 support soon?

I don't mean to bother you, but did you have a chance to look closer at this, @tonyketcham? Or do you have an idea of when you might get around to it?
I understand if working on this isn't your priority right now, I'm just asking so I can plan around it myself. If you don't have the capacity to work on this right now, that's perfectly fine!

Apologies for the lengthy delay, totally my fault! I'm planning on having this sorted by the end of this week with a release :)

This has been shipped in v1.1.0! 🎉
https://github.com/Modyfi/vite-plugin-yaml/releases/tag/v1.1.0


chore(package): update flow-bin to version 0.60.1

Closes #21

Coverage remained the same at 76.842% when pulling e53936b4084b13f3c0e6525636f805b828803f47 on greenkeeper/flow-bin-0.60.1 into c1599a22be85ba983dce001859372eabf5e1507f on master.


Fix TCP Connections Query

Here is the current query:
let ActiveWorkersInTimeRange = toscalar(                                  StatsDWASWorkerProcessTenMinuteTable
| where TIMESTAMP >= datetime(2017-08-22 12:00:00) and TIMESTAMP <= datetime(2017-08-23 12:00:00) and Tenant =~ "175b633ece1f4f4ba4c6bd2dd2082906"
| where (ApplicationPool =~ "smpwfe-prod-wcus" or ApplicationPool startswith "smpwfe-prod-wcus__")
| summarize makeset(RoleInstance));
StatsDWASWorkerProcessTenMinuteTable
| where TIMESTAMP >= datetime(2017-08-22 12:00:00) and TIMESTAMP <= datetime(2017-08-23 12:00:00) and Tenant =~ "175b633ece1f4f4ba4c6bd2dd2082906" and (ApplicationPool =~ "smpwfe-prod-wcus" or ApplicationPool startswith "smpwfe-prod-wcus__")
| project TIMESTAMP, RoleInstance
| join
(
SystemStats
| where TIMESTAMP >= datetime(2017-08-22 12:00:00) and TIMESTAMP <= datetime(2017-08-23 12:00:00) and Tenant =~ "175b633ece1f4f4ba4c6bd2dd2082906"
| where  RoleInstance in (ActiveWorkersInTimeRange)
| where EventId == 55076
| extend totalandInbound = split(Details,',' ,0)[0]
| extend outboundstring = split(Details,',', 1)[0]
| extend onlyInbound = substring(totalandInbound , indexof(totalandInbound,'Inbound ='))
| project  TIMESTAMP = bin(TIMESTAMP, 10m), ACTUALTIMESTAMP = TIMESTAMP , RoleInstance, Established = EstablishedConnections, TimeWait , Loopback,  Inbound=toint(replace('Inbound =','', onlyInbound)) ,   Outbound = toint(replace('Outbound = ' , '' , tostring(outboundstring)))
)
on TIMESTAMP, RoleInstance
| project TIMESTAMP, RoleInstance, Established, TimeWait, Loopback, Inbound, Outbound
However this ends up in a jagged view like this:

You should replace the last 4 lines of the query with this:
| project  TIMESTAMP = bin(TIMESTAMP, 10m), ACTUALTIMESTAMP = bin(TIMESTAMP ,5m), RoleInstance, Established = EstablishedConnections, TimeWait , Loopback,  Inbound=toint(replace('Inbound =','', onlyInbound)) ,   Outbound = toint(replace('Outbound = ' , '' , tostring(outboundstring)))
)
on TIMESTAMP, RoleInstance
| project TIMESTAMP = ACTUALTIMESTAMP, RoleInstance, Established, TimeWait, Loopback, Inbound, Outbound
Let me know if you have any questions

Thanks @stevenernst131 for the detailed explanation and identifying the issue, I fixed this. I will send a CR shortly. This is how it looks like now


Game crashes after installing Polish fonts

Hello. As in the description, game crashes after adding .POD file with Polish fonts ripped from the original BloodRayne. Could you help?
Help me, Obi Wan Kenobi. You're my only hope.

Oh, sorry, GitHub never sent me a notification about this issue :p
Is this still a problem? How did you install the polish fonts?

Yup. I've cut out font files from the Polish .POD archives, packed 'em into new .POD archive and made it, so this new archive overwrites English fonts. It's strange, that this crashes the game, 'cause every other file, that I've ported from the OG BloodRayne worked like a charm.


Format ExprYield/ExprYieldFrom

Summary
Formats:

yield x
yield from x

expressions
Test Plan
Added fixtures for both expressions, ran black_comparison.
NOTE: This is my first formatter PR. From what I gather there are many many ways that interesting combinations of comments and parentheses can lead to unexpected results. I will try and run the formatter against some large codebases to see if any unexpected results pop up, but figured I'd open it for review in the meantime
Issue link:
Closes: https://github.com/astral-sh/ruff/issues/5916

The cpython check failed on Lib/test/test_asyncio/test_locks.py, which can be shrunk to the following snippet:
def a():
    with (yield):
        pass

Seem like yield can appear everywhere, even in e.g. if. Searching for yield_expression  in https://docs.python.org/3/reference/grammar.html it seems that yield without parentheses is the exception (only for yield statements and right hand sides of assignment), in atom (through group) it has parentheses.

Interesting. I've expanded the test cases with a bunch more examples, and inverted the needs_parentheses() logic, excluding only parents which are some type of assign
You mention yield statements also require no parentheses. How do you mean exactly? Looking at it now yield (yield l), yield from (yield l) yield (yield from l) and yield from (yield from l) all seem to need parentheses)`.

You mention yield statements also require no parentheses. How do you mean exactly? Looking at it now yield (yield l), yield from (yield l) yield (yield from l) and yield from (yield from l) all seem to need parentheses)`.

For
def f():
    yield 1

you get (simplified for readability) StmtFunctionDef { body: [StmtExpr(ExprYield(..))] }. StmtExpr gets formatted with
impl FormatNodeRule<StmtExpr> for FormatStmtExpr {
    fn fmt_fields(&self, item: &StmtExpr, f: &mut PyFormatter) -> FormatResult<()> {
        let StmtExpr { value, .. } = item;

        if is_arithmetic_like(value) {
            maybe_parenthesize_expression(value, item, Parenthesize::Optional).fmt(f)
        } else {
            value.format().fmt(f)
        }
    }
}

so this doesn't become
def f():
    (yield 1)

This is awesome. We may want to consider unifying the implementation in a follow up PR similar to AnyStmtWith (it could also implement NeedsParentheses so that we have all the logic in a single place).
https://github.com/astral-sh/ruff/blob/03018896deebfeb2cf96c28c897dd98a5e1c7699/crates/ruff_python_formatter/src/statement/stmt_with.rs#L15-L114

Good idea @MichaReiser. I'll finish up my outstanding PRs, then move to work on that.


Lint Option: Input parameters not changed within a procedure should be specified with CONST

Been thinking about this. Quite a lot to think about. One case I can think would be tricky is if a parameter gets passed into another procedure, and it's passed by reference and changed inside of it.

I think you have to assume that if its not passed as a constant in the PI then it will be changed in the underlying procedure. This is fine though as Eventually you will get to the bottom of the stack and where the variable is not changed it will suggest it to be a CONST... then the procedure above will do the same... until we get back to the top level

i.e. if a variable can be changed assume it will be changed

Yup that makes sense! Unless the linter has the prototype and then we can tell if it's ref/const/value!

Yeah although the PI/PR have to be aligned right?

Ignore that I see what you are saying, yeah I think you need to look at the prototype for the called procedure, failing that you will need to look at the interface if its in the same source and a prototype is not declared

Thinking about it, I think we also need to hope no one actually uses overloading!! Because then you have to match a signature of the call to one of the overloaded procedures (I'm not entirely sure what the rules are for this as I've never actually used it!)

Good share - forgot about that completely!


Design an user profile page

Issue reference: #46
Should be assigned to @sujitlwale by @tarunnsingh
Originally posted by @sujitIwale in https://github.com/alexa-dev-hub/alexa-community-jaipur/issues/46#issuecomment-715922338

@sujitIwale
Please work and give PR on auth-setup-frontend.


TeX input: {} inside \textrm

\textrm{{}d} produces {}d which is not consistent with real TeX (link to texlive.net)

Because MathJax is primarily concerned with typesetting mathematics, it does not implement TeX's text-mode processing.  So the contents of \text{} is presented verbatim, other than embedded mathematics (e.g., \text{for $x > 1$}).
MathJax v3 has a textmacros extension that allows processing of some macros in text mode, so if you are using version 3, you can load that, and it will handle the example you provide.


Remove box access tokens

Sub-issue of #202
Currently, each DocumentPreview makes a call to /box/token to obtain an access token which is never used. Due to the number of these requests and the time it takes for them to resolve, this drastically slows down our application.

Resolved in #204


Fix NPE on empty filter

Signed-off-by: Niels Thykier<EMAIL_ADDRESS>

I did not use SpringUtils for this as SpringUtils says it is in an "internal API"


Adding Facebook Twitter and Google link

Hey. In footer i noticed facebook twitter and Google icon are not contain any link so I want to add link assign this issue to me under  GSSoC-23 thanks.

Can you please assign me this role? @Vishal-Parasram

@Ijaiswalshivam I'm also contributing. I'm not a owner create any other issues.

You both add what you want I will merge then .

@Anupkjha2601 can I get link which I have to add.

You want my account ? To add.

@Anupkjha2601 social media link then I can add it into their particular icon i don't know your social media or may be you can give your Instagram I'd Twitter mail linked.

+1

Facebook I'd: https://www.facebook.com/anup.jha.108889
LinkedIn I'd: https://www.linkedin.com/in/anup-kumar-jha-36a393201
Instagram I'd: https://www.instagram.com/anup.jha.108889/

@Anupkjha2601 Give Twitter, mail I'd and Instagram link

Twitter I'd: https://twitter.com/AnupKum59920381
Mail id<EMAIL_ADDRESS>

why i'm getting Azure Static Web Apps CI/CD Attempt /build and deploy job failed in 51 seconds ?

@Anupkjha2601 Done added social media link in footer.

@Anupkjha2601 Add GSSoC-23 label to this issue

yes , i will do it .

Twitter I'd: https://twitter.com/AnupKum59920381 Mail id<EMAIL_ADDRESS>
can you give you G+ account link like twitter as email id won't work in place of G+ as when user clicks G+ a google plus profile needs to open like in case of facebook or twitter

Twitter I'd: https://twitter.com/AnupKum59920381 Mail id<EMAIL_ADDRESS>
can you give you G+ account link like twitter as email id won't work in place of G+ as when user clicks G+ a google plus profile needs to open like in case of facebook or twitter
Also, have added the clickable twitter link, please review PR https://github.com/Anupkjha2601/food-recipes-website/pull/219

Twitter I'd: https://twitter.com/AnupKum59920381 Mail id<EMAIL_ADDRESS>
can you give your G+ account link like twitter as email id won't work in place of G+ as when user clicks G+ a google plus profile needs to open like in case of facebook or twitter
Also, have added the clickable twitter link, please review PR https://github.com/Anupkjha2601/food-recipes-website/pull/219
@Anupkjha2601 @Vishal-Parasram @Ijaiswalshivam

Twitter I'd: https://twitter.com/AnupKum59920381 Mail id<EMAIL_ADDRESS>
can you give your G+ account link like twitter as email id won't work in place of G+ as when user clicks G+ a google plus profile needs to open like in case of facebook or twitter
Also, have added the clickable twitter link, please review PR https://github.com/Anupkjha2601/food-recipes-website/pull/219
@Anupkjha2601 @Vishal-Parasram @Ijaiswalshivam


issue raw yum command since we don't have facts in bootstrapping

Closes #1297

ci check this

I'm wondering if our CI uses the centos bootstrap anyways. seems like we'd have uncovered this failure a long time ago if so.


New features

Hello! I use this methods in my matrix-skype project and it would be great if you add it to your project. I have never used typescript before, probably there are some incorrect interafaces or types.

Thanks for the PR.
I am in the process of updating the conversation API (#87) to type-check the responses and simplify it a bit.
It means that your PR will have to be rebased: I'll take care of it this week-end.
Having all the features already working is a huge help. 👍

I am going to rebase and merge this PR. The other PR changes too much things around conversations and I need more time to ensure that everything works fine and there are no problems migrating.
It will be safer to merge this PR first and publish it.

I just finished reviewing your changes: it's pretty good. I just did a small update in createConversation to avoid using an empty thread id and check for the location header.


Ability to retrieve instances for all projects through API

This PR allows for retrieving instances from all projects by using command lxc list --all-projects.

New functions was introduced: GetInstancesAllProjects and GetInstancesFullAllProjects,
so I didn't have to change current functions(GetInstances, GetInstancesFull) signature.
New url parameter was added: all-projects. It would be not convenient to handle this with recursion parameter
because then we should add recursion=3 and recursion=4 for GetInstance and GetInstanceFull respectively.
GetInstanceNamesByNodeAddress and GetInstanceToNodeMap was modified to handle multiple projects.
Colum 'PROJECT' is not displayed by default. We need to add -c e.

Closes #7559

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

So based on the all-projects=true parameter approach instead of the initial recursion=3 plan, I think I'm warming up to the idea of adding Project to api.Instance as that then allows keeping the API otherwise identical.
So let's go with that and add all-projects to the swagger doc, tweak the lxc list behavior a bit and see if @tomponline and @masnax have some opinion on the DB piece.
Thanks!

@turtle0x1 FYI on the slightly different approach (no recursion=3 but instead all-projects=true supported in combination of the 3 different options (no-recursion, recursion-1 and recursion-2).

@stgraber thanks for the ping.
This PR looks like it adds an projects[] param to GetInstanceNamesByNodeAddress which would be a nice feature over the API.
IIRC * is not a valid project name so might it be an idea to change the GET parameter to ?projects=*  so later we can do /1.0/instances?projects=["default", "project_1", ETC] (if we want to filter for some reason)?
Also ill probably open a similar requests for /1.0/images, /1.0/profiles, ETC to have the all-projects=true or projects parameter, not sure if this impacts the overall approach but is worth considering (if you would approve those additions).

Looking good, the only things left are:

Add the GetInstanceNamesAllProjects function
Update swagger to just add the extra parameter to the existing routes instead of adding 3 new routes

@stgraber I don't know if this has any bearing on this PR, but I noticed today that we already have a precedent for requesting an API endpoint to include all projects using project=* for events:
https://github.com/lxc/lxd/blob/dca4643257ff0dfdd90f5c911ac6c5b972162226/lxd/cluster/events.go#L142-L144

@tomponline yeah, I think I'd actually rather we add all-projects=true to the events API than have something else use projects=* :)
The events code that does that was effectively an accident. We internally would store the project as '*' for event listeners that should get all projects (like the cluster ones) and when adding the --all-projects flag to lxc monitor I noticed that I didn't need any API changes and that I could just have the user pass * directly. In hindsight I think I should have added something like the all-projects flag, have the internal struct track it as a boolean and have LXD strictly validate the project field against existing project names. It's probably still worth doing and is a sufficiently rarely used bit of API that we could probably get away with just fixing it (and causing a minor API break).

jenkins: test this please

jenkins: test this please

Testsuite passed

all-projects

Thanks, issue logged https://github.com/lxc/lxd/issues/9647


SDK throwing error while deserializing CSN notification event

When trying to make a call using the CallSessions approach with the C# SDK, I'm getting an error. The call still goes through, but it looks like the SDK is having some issues parsing the JSON response. The error means that we're unable to process the response as we don't get it. The error is: Cannot deserialize the current JSON array (e.g. [1,2,3]) into type 'RingCentral.CallParty' because the type requires a JSON object (e.g. {"name":"value"}) to deserialize correctly.
To fix this error either change the JSON to a JSON object (e.g. {"name":"value"}) or change the deserialized type to an array or a type that implements a collection interface (e.g. ICollection, IList) like List that can be deserialized from a JSON array. JsonArrayAttribute can also be added to the type to force it to deserialize from a JSON array.
Path 'session.parties', line 8, position 19.

Can we get more detail please? Such as code snippet?
It doesn't seem like a notification issue:

When trying to make a call using the CallSessions approach

// https://developers.ringcentral.com/my-account.html#/applications
// Find your credentials at the above url, set them as environment variables, or enter them below
// PATH PARAMETERS
string accountId = "";
// POST BODY
MakeCallOutRequest makeCallOutRequest = new MakeCallOutRequest {
from = new MakeCallOutCallerInfoRequestFrom {
deviceId = ""
},
to = new MakeCallOutCallerInfoRequestTo {
phoneNumber = "",
extensionNumber = ""
}
};
RestClient rc = new RestClient(
Environment.GetEnvironmentVariable("clientId"),
Environment.GetEnvironmentVariable("clientSecret"),
false
);
await rc.Authorize(
Environment.GetEnvironmentVariable("username"),
Environment.GetEnvironmentVariable("extension"),
Environment.GetEnvironmentVariable("password")
);
var r = await rc.Restapi().Account(accountId).Telephony().CallOut().Post(makeCallOutRequest);
// PROCESS RESPONSE

@tylerlong  u r right , this is with callout api call using c# code snippet on the api explorer

I just tired this endpoint and I had no problem

Oh, I forgot I just release the latest 2.0.1 version of the SDK. Please have a try and let me know if you still have this issue.


github: Update CentOS CI check in mergify rules

depends on https://github.com/samba-in-kubernetes/samba-centosci/pull/13

/test centos-ci/sink-clustered/mini-k8s-latest


Scala 2.11 cannot resolve dependencies

sbt file: "com.jcdecaux.setl" %% "setl" % "1.0.0-SNAPSHOT"
result:

Hey @LiuxyEric, what are your Spark and Scala versions? Spark 3.0 may not work with Scala 2.11.
If it's possible, try Spark 3.0 with Scala 2.12

Hey @LiuxyEric, what are your Spark and Scala versions? Spark 3.0 may not work with Scala 2.11.
If it's possible, try Spark 3.0 with Scala 2.12

I am using Spark 2.4 and Scala 2.11. It seems that the three dependencies I mentioned on the picture don't have the right version for scala 2.11.  I updated my scala version to 2.12 and the problem solved. Thanks for you replay!

I am using Spark 2.4 and Scala 2.11. It seems that the three dependencies I mentioned on the picture don't have the right version for scala 2.11.  I updated my scala version to 2.12 and the problem solved. Thanks for you replay!

Otherwise, another temporary fix is to manually import these three dependencies with the following versions in the sbt file when  it's not possible to upgrade from scala 2.11 to 2.12 😄
<dependency>
    <groupId>com.audienceproject</groupId>
    <artifactId>spark-dynamodb_2.11</artifactId>
    <version>1.0.4</version>
</dependency>
<dependency>
    <groupId>io.delta</groupId>
    <artifactId>delta-core_2.11</artifactId>
    <version>0.6.1</version>
</dependency>
<dependency>
    <groupId>com.datastax.spark</groupId>
    <artifactId>spark-cassandra-connector_2.11</artifactId>
    <version>2.5.1</version>
</dependency>

I've added this into the project's readme.
There is indeed something to be improved with the project's pom file and the CI script. I have opened a new issue to continue tracking this problem #173 .

Otherwise, another temporary fix is to manually import these three dependencies with the following versions in the sbt file when  it's not possible to upgrade from scala 2.11 to 2.12 😄
<dependency>
    <groupId>com.audienceproject</groupId>
    <artifactId>spark-dynamodb_2.11</artifactId>
    <version>1.0.4</version>
</dependency>
<dependency>
    <groupId>io.delta</groupId>
    <artifactId>delta-core_2.11</artifactId>
    <version>0.6.1</version>
</dependency>
<dependency>
    <groupId>com.datastax.spark</groupId>
    <artifactId>spark-cassandra-connector_2.11</artifactId>
    <version>2.5.1</version>
</dependency>

I've added this into the project's readme.
There is indeed something to be improved with the project's pom file and the CI script. I have opened a new issue to continue tracking this problem #173 .


Fix capitalization for pirates and rats

About the PR

When rats and pirates speak their first word in all caps, it'll now properly not lowercase the word. Additionally, the prefix added will also be in all caps. For rats, if they get a suffix, and their last word was in all caps, that word will also be in all caps.
Fixes #24996
Why / Balance

Let the pirates scream YARGH.
Technical details

LINQ
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://github.com/space-wizards/space-station-14/assets/32827189/7a838717-351e-4d39-9ba1-25e32ae59367
Breaking changes

Changelog

:cl:

fix: Pirate Accent and Mobster accent will respect capitalization better now

Tried to make it a little smarter by checking only the first and last words and changing caps based on that.


Generated constructor do not have annotations

When properties have annotations, AutoValue puts their respective annotations to the parameters in the constructor. This library does not, this causes nullity problems with some tools. It is a problem with kotlin too.
Here as an example the generated code from AutoValue and this library respestively.
$AutoValue_Section(
      @Nullable String title,
      @Nullable String featuredBarColor,
      @Nullable String featuredRealtorLogoUrl)

  AutoValue_Section(String title, String featuredBarColor,
      String featuredRealtorLogoUrl) {
    super(title, featuredBarColor, featuredRealtorLogoUrl);
  }

Fixed by #144

I checked the issues but couldn't find it. Sorry for duplicate issue. 👍


Multiple dev drive filters require quotes

Type of issue
Typo
Feedback
The documentation has example commands to set filters.  However when multiple filters are specified, they must be surrounded by quotation marks (as-per documentation, testing on Windows CMD and Powershell):

The following command is found in the documentation, and does not work as-is:
fsutil devdrv setfiltersallowed PrjFlt, MsSecFlt, WdFilter, bindFlt, wcifs, FileInfo, ProcMon24

Techinically the example should have quotes too:
fsutil devdrv setfiltersallowed Filter-01, Filter-02

Page URL
https://learn.microsoft.com/en-us/windows/dev-drive/
Content source URL
https://github.com/MicrosoftDocs/windows-dev-docs/blob/docs/hub/dev-drive/index.md
Author
@mattwojo
Document Id
bfda7965-bc34-ab88-fdc9-efef4a5d8a9e

Checking on this. Thanks for filing!

@ChandKV - I believe this is correct and there is a doc bug. Can you confirm?


Expose extra attributes + initialize

Expose the extra attribute in docs for all types. This is a fairly important attribute to be aware of since moderngl types cannot be extended using inheritance. I personally went down an unecessary rabbit hole of making wrappers though composition before I saw the extra attribute.

Initialize all extra attributes on instance creation. program.extra will trigger AttributeError unless the extra attribute has previously been assigned a value. I think it's a lot more user friendly to ensure the attribute is present with a default None value.

This default value is also advertised in most initializers, so most IDEs and code inspection tools believe the attribute is present with a default value. (They don't care about the TypeError())
https://github.com/cprogrammer1994/ModernGL/blob/a2cf2e1b20939331b25e63531f128b4f9008ab62/moderngl/program.py#L26-L34

Thank you!


Bot doesn't catch <@#####> mentions

Discussed in detail in the Dev Discord, basically Modmail fails to catch any mentions that were derived from doing <@#############> which defeats the purpose of that addition in that code so we need to re-write and add to that part so it can catch them and search for those users so it works correctly

This feature was removed entirely in v1.0, so this is now out of scope.


Test GitHub integration

Testing github actions -- this will not be merged.

@sdavidp is it intended to have 113 commits? :D

@sdavidp is it intended to have 113 commits?

@marcinwadon I don't think it matters? I just want to trigger the github action.
However, it seems to have unexpectedly passed. I'm going to kill this PR and start a new one based on current develop, then "break" the code and verify it fails locally, then push it to github and see what happens.
cc: @AlexBrandes  @ryle-goehausen


Fix: keyring permissions where world readable

Reported in https://bugzilla.suse.com/show_bug.cgi?id=920926
by  Andreas Stieger<EMAIL_ADDRESS>Before thsi fix to umask for keyring handling, After execution
of ceph-deploy, all keyrings have mode 644.
The documented ceph-deploy procedure by creating a dedicated
admin user, and keys will be readable to all other (non-admin)
users as well, thus leaking authentication credentials.
The fix uses umask to resolve the writing of keyfiles.
Signed-off-by: Owen Synge<EMAIL_ADDRESS>

Can one of the admins OK this pull request so I can run a build?

OK to test

ok to test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1529/
Tests passed for this pull request.

@osynge This looks good to me. Thanks for the fix!
Some changes recently went in that make this need to be rebased off of master.  Can you do that?  Then I'll merge it it.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1537/
Tests passed for this pull request.

@trhoden I think this is rebased now? If anythign is wrong please let me know.

@osynge Looks like you did a merge instead of a rebase? I wouldn't expect to see a merge commit (there are now two commits in this PR).
Assuming that you had set up github.com/ceph/ceph-deploy.git as your upstream repo, you would normally do something like the following:
git fetch upstream
git checkout wip_key_perms
git rebase upstream master
git push -f origin wip_key_perms

Did you do something along these lines?
I'm happy to chat about this on #ceph-devel too if you like.

  More than I wanted to document here in the comments.  So instead I just created a new PR for it.


regarding Paths.

As puppeteer is used in linux/macos too and memorizing those paths are usually not necessary I have made a package to assist it. Please find it here https://github.com/shirshak55/edge-paths.
we can use it like this:
const edgePaths = require("edge-paths");

// On Windows

edgePaths.edge; // 'C:\\Program Files (x86)\\Microsoft\\Edge Dev\\Application\\msedge.exe'

Currently supports windows. Once underlying package support macs and linux i will add support asap :)
Thanks Microsoft for making fastest browser in the universe.

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

ID: aa0e4d86-12ac-7f7e-1c06-66377987487e
Version Independent ID: 52fa943b-5a87-0d35-1d89-a2a627486061
Content: Puppeteer - Microsoft Edge Development
Content Source: microsoft-edge/puppeteer.md
Product: microsoft-edge
Technology: devtools
GitHub Login: @zoherghadyali
Microsoft Alias: zoghadya

This is for @zoherghadyali to weigh in on. This is beyond my technical knowledge.

Hi @shirshak55! Thank you for creating the edge-paths package on npm and making it easier to get the executable path to Edge! I tried it out on Windows with this snippet:
const edgePaths = require("edge-paths");
console.log("Windows Edge path", edgePaths.edge);

but what I see logged is: Windows Edge path C:\Users\zoghadya\Documents\My_Apps\DevTools-todomvc\node_modules\edge-launcher\dist\x86\MicrosoftEdgeLauncher.exe which isn't what I expected. I expected edgePaths.edge to point to my installation of stable Microsoft Edge: C:\Program Files (x86)\Microsoft\Edge\Application\msedge.exe.
Please let me know if this is a bug and I'm happy to try again! I would also like to request a way to provide the paths to Edge Canary, Dev, and Beta as well as the Stable Edge channel. Thanks!

@zoherghadyali hey there can u give me list of all paths. I don't have windows so it is abit hard for me to figure it out
Windows
Edge Stable: C:\Program Files (x86)\Microsoft\Edge\Application\msedge.exe
Edge Canary: C:\Program Files (x86)\Microsoft\Edge SxS\Application\msedge.exe
others (Confirm edge canary path)
On OSX the path i receive are
Beta: /Applications/Microsoft Edge Beta.app/Contents/MacOS/Microsoft Edge Beta
Canary: /Applications/Microsoft Edge Canary.app/Contents/MacOS/Microsoft Edge Canary
Dev: /Applications/Microsoft Edge Beta.app/Contents/MacOS/Microsoft Edge Dev
Stable: /Applications/Microsoft Edge Beta.app/Contents/MacOS/Microsoft Edge

I have tried to check windows paths currently it returns
C:\Program Files (x86)\Microsoft\Edge Beta\Application\msedge.exe
C:\Program Files (x86)\Microsoft\Edge Canary\Application\msedge.exe
C:\Program Files (x86)\Microsoft\Edge Dev\Application\msedge.exe
C:\Program Files (x86)\Microsoft\Edge\Application\msedge.exe

I hope its correct.

Would you mind reconfirming
Typescript
import { getEdgeBetaPath, getEdgeCanaryPath, getEdgeDevPath, getEdgePath } from "edge-paths"

console.log(getEdgeBetaPath())
console.log(getEdgeCanaryPath())
console.log(getEdgeDevPath())
console.log(getEdgePath())

Or Javascript
let { getEdgeBetaPath, getEdgeCanaryPath, getEdgeDevPath, getEdgePath } = require("edge-paths") // Replace with require("edge-paths")

console.log(getEdgeBetaPath())
console.log(getEdgeCanaryPath())
console.log(getEdgeDevPath())
console.log(getEdgePath())


getter called before constructor

in the skeleton navigation welcome.js:
export class Welcome {
  heading = 'Welcome to the Aurelia Navigation App!';
  firstName = 'John';
  lastName = 'Doe';
  previousValue = this.fullName;

  x = {};

  constructor() {
    console.log('Constructor');
    this.x.y = 'z';
  }

  get fullName() {
    console.log('fullName', this.x);  // <-- this.x === undefined. neither x= {} nor constructor has been called yet
    return `${this.firstName} ${this.lastName}`;
  }

Try changing your logging line to console.log(this.x.y);

right. had some typos. i've update the sample

Seems to be working fine for me https://gist.run/?id=e30321988c8f08ab6fb5af37a3f91b56

yeah, but who knows, whats different with gist, run-time transpilation, bundled ...
try the same in the skeleton and it'll be
welcome.js:22 fullName undefined
welcome.js:12 Constructor

wrong button, so again:
yeah, but who knows what happens with gist, run-time transpiled, bundled ...
try with the skeleton and it'll be
welcome.js:22 fullName undefined
welcome.js:12 Constructor

Not a bug with Aurelia. If anything, it's a transpiler issue.

@EisenbergEffect
hmm. i tried the same with es6 bundled, typescript and webpack. it's always the same. that doesn't hint at a transpiler issue
ps: one can remove the fullName 'call' in the view. still the same.
in case it says anything. following calls the fullName getter before the constructor
at Container.invoke (webpack:///./~/aurelia-dependency-injection/dist/commonjs/aurelia-dependency-injection.js?:565:13)
    at StrategyResolver.get (webpack:///./~/aurelia-dependency-injection/dist/commonjs/aurelia-dependency-injection.js?:126:35)
    at Container.get (webpack:///./~/aurelia-dependency-injection/dist/commonjs/aurelia-dependency-injection.js?:500:21)
    at eval (webpack:///./~/aurelia-templating/dist/commonjs/aurelia-templating.js?:3831:71)

It shouldn't be possible to call a getter before the ctor has been run unless it is detached from the constructor somehow. @jdanyow Do you know anything about this?

ah, stupid me. found the error.
that's run before the constructor:
previousValue = this.fullName;
so, it's not a bug, it's a feature :smile: for real though
issue closed. sorry for the confusion


fix: adjusted tab height

SUMMARY
The scroll bar on query history disappeared based on a redesign that we did for the results page to add rows returned.
I adjusted the tab height of rows returned, in order to not have a scroll appear there, and added the scroll back to query history.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

/testenv up

Thanks for the PR!

can we add Before vs After screenshot, and manual test plan to the descprtion?
might not be related to the changes in the PR, but please see attached screenshot of the disconnected dropdown

@junlincc hey! Thank you! I added before shots.
I will happily look into that, thank you for bringing it up.

I think #2 isn't related to this fix, I'm seeing it on master already. Created a separate ticket here: https://github.com/apache/superset/issues/13842

@yousoph and @junlincc played around with it today, I know why it exists, working on fix now.

I think #2 isn't related to this fix, I'm seeing it on master already. Created a separate ticket here: #13842

is regression caused by https://github.com/apache/superset/pull/13694

@eschutho making the tab heights the same, gives the results tab a scroll bar

Looks good! @steejay if interested, here's a video.
https://user-images.githubusercontent.com/5186919/113434461-df423380-9395-11eb-808c-e95939f924fa.mov


Block edge 4.3.40

4.3.40 needs to be disabled:
3 clusters currently failing (14%),  3 gone (14%),and 15 successful (68%), out of 22 who attempted the update over 7d
Currently failing classification:
* Down nodes: 1
* Logging issues: 2
* Node pressure: 1
* Single master: 1
* Slow etcd: 1
* Version retarget: 1

4.3.40 can be enabled again:
2 clusters currently failing ( 8%),  3 gone (12%),and 17 successful (71%), out of 24 who attempted the update over 7d
Currently failing classification:
* Down nodes: 2
* Logging issues: 1
* Node pressure: 1
* SDN issue: 1
* Single master: 2
* Version retarget: 1
* volumesnapshots v1alpha1: 1


Combobox - On select dropdown does not disappear using phx-change on form

When using to_form function on a phx-change event like a validation event, the dropdown does not disappear when the item selected once. If the same item is selected twice the dropdown disappears. Whats weird is this does not occur when the first item is selected
def handle_event("validation", %{"group_id" => group_id}, socket) do
    # Do some validation
    {:noreply, assign(socket, form: to_form(%{"group_id" => group_id}))}
end

def render(assigns) do
    ~H"""
    <div>
      <.form for={@form} phx-change="validation">
        <.combo_box
          label="Group"
          placeholder="Filter down by group"
          options={@groups)}
          field={@form[:group_id]}
        />
      </.form>
    </div>
    """
end

Thanks just pushed a potential fix. Can you try with 0.4.8?

Sweet it works now, thank you


kasthuri catmaid broken

Most of these projects seem to not work.  Having a catmaid or leaflet project would aid my demos next week and also enable science.  :) openconnecto.me/catmaid/?dataview=13

We need to re-org the catmaid projects. Can you send me a list of tokens /
channel combinations for all the kasthuri data we want public? That info
isn't as readily apparent from current catmaid as it may seem.
On Thu, Sep 3, 2015 at 8:58 AM William Gray<EMAIL_ADDRESS>wrote:

Assigned #141
https://github.com/openconnectome/open-connectome/issues/141 to
@alexbaden https://github.com/alexbaden.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/open-connectome/issues/141#event-400017761
.

Yea:
Image project is always kasthuri11cc, except where noted:
reference data:  ac3, ac4
annotations:  kat11segments, kat11synapses, kat11vesicles, kat11mito
3 cylinders:  kat11redcylinder, kat11greencylinder, kat11mojocylinder
low res data:  kasthuri14Maine  (standalone image project)
s1 column:  kasthuri14s1colEM linked to kasthuri14s1colANNO

On Thu, Sep 3, 2015 at 9:04 AM Alex Baden<EMAIL_ADDRESS>wrote:

We need to re-org the catmaid projects. Can you send me a list of tokens /
channel combinations for all the kasthuri data we want public? That info
isn't as readily apparent from current catmaid as it may seem.
On Thu, Sep 3, 2015 at 8:58 AM William Gray<EMAIL_ADDRESS>wrote:

Assigned #141
https://github.com/openconnectome/open-connectome/issues/141 to
@alexbaden https://github.com/alexbaden.
—
Reply to this email directly or view it on GitHub
<
https://github.com/openconnectome/open-connectome/issues/141#event-400017761
.

—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/open-connectome/issues/141#issuecomment-137433013
.

I believe I have fixed all the existing projects. Take a look and close if
I'm correct?
On Thu, Sep 3, 2015 at 9:52 AM William Gray<EMAIL_ADDRESS>wrote:

Yea:
Image project is always kasthuri11cc, except where noted:
reference data: ac3, ac4
annotations: kat11segments, kat11synapses, kat11vesicles, kat11mito
3 cylinders: kat11redcylinder, kat11greencylinder, kat11mojocylinder
low res data: kasthuri14Maine (standalone image project)
s1 column: kasthuri14s1colEM linked to kasthuri14s1colANNO

On Thu, Sep 3, 2015 at 9:04 AM Alex Baden<EMAIL_ADDRESS>wrote:

We need to re-org the catmaid projects. Can you send me a list of tokens
/
channel combinations for all the kasthuri data we want public? That info
isn't as readily apparent from current catmaid as it may seem.
On Thu, Sep 3, 2015 at 8:58 AM William Gray<EMAIL_ADDRESS>wrote:

Assigned #141
https://github.com/openconnectome/open-connectome/issues/141 to
@alexbaden https://github.com/alexbaden.
—
Reply to this email directly or view it on GitHub
<

https://github.com/openconnectome/open-connectome/issues/141#event-400017761

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/openconnectome/open-connectome/issues/141#issuecomment-137433013
.

—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/open-connectome/issues/141#issuecomment-137453643
.

Thanks!  Looks good!


Fix backend isues

These issues aren't part of this repo, but need to be fixed before this milestone is complete, so here for visibility:
https://github.com/MoveOnOrg/mop/issues/24
https://github.com/MoveOnOrg/mop/issues/41
https://github.com/MoveOnOrg/mop/issues/60

Added a new issue, so re-opening this.


make ClientDetails.getAuthorities() return a set which extends GrantedAuthority

@sofior Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Method75

Method75 is not working

Very informative.
How do you compile this project at first.

¯_(ツ)_/¯ guess you are filtered by project readme. It works!


Initial build

Alaska Airlines Pull Request
This PR represents the initial build of the auro-loader repo
Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

@blackfalcon
I think loaders are a big opportunity to surprise and delight. More of an airline theme would be fun! (takeoff/landing?)
Looks pretty neat though and I'd love consistency in this space. The only real feedback is that the non-spinners (ping/ripple/pulse) move a little too fast for me to feel calmed down that I am waiting for something to load, and kinda make me amped up.

@blackfalcon
I think loaders are a big opportunity to surprise and delight. More of an airline theme would be fun! (takeoff/landing?)
Looks pretty neat though and I'd love consistency in this space. The only real feedback is that the non-spinners (ping/ripple/pulse) move a little too fast for me to feel calmed down that I am waiting for something to load, and kinda make me amped up.

Points are taken. In response, these loaders are pure CSS with simple HTML (inside the CE). I'd like to stay away from adding dependencies like SVGs if possible or having to greater gifs for these.
Also, in regards to timing, that can be adjusted.

I like your logic on componentizing this for consistency, but I also like the idea of a more brand-style design. Why the hesitation on an SVG dependency?

Could something like this be done with CSS shapes & masks?

@clockeby

Why the hesitation on an SVG dependency?

I guess I could bake the SVG into the CE. Something like clouds wouldn't be part of the Auro Icon lib, so sure, SVGs back on the table!

A couple of the loader types made me think of WCAG flashing guidelines (flashing can cause seizures). I don’t know all the details around the guidelines but I’m including it here to review: https://www.w3.org/TR/WCAG20/#seizure-does-not-violate

Homepage has a loading plane and loading cloud. The plane gif actually demonstrates why we added “cloud”: we didn’t want skyscraper backgrounds to create a 9/11 effect. 😬

@amatlock1 The CSS that drives the animation is responsive to users who prefer reduced motion. This is the main reason why I want to get the loaders into code so that we can control this. Gifs would need to be wrapped in other code to support this feature and that is too hard to police.
https://github.com/AlaskaAirlines/WebCoreStyleSheets/blob/master/src/_core.scss#L36-L40

@amatlock1 took your a11y point to the next level. By default, Auro will over-ride all animations when reduced motion is turned on. But in the close of loaders, there is something missing there with the animations just being stuck. So, see this.
https://github.com/AlaskaAirlines/auro-loader/commit/797248ef616d9f18293169a0fc1db221e314318c
The idea here is that when reduced motion is checked, the loader animation will be replaced with the string Loading...
Standard view

With reduced motion

Thoughts on linear vs non-linear animation speeds? Asking I’m from curiosity, I’m okay if we defer this.
The cloud on homepage has non-linear interpolation (not sure if it’s ease-in, ease-out or otherwise).

Thoughts on linear vs non-linear animation speeds? Asking I’m from curiosity, I’m okay if we defer this.
The cloud on homepage has non-linear interpolation (not sure if it’s ease-in, ease-out or otherwise).

This is the design feedback I am interested in. The cloud loader I built for this demo didn't really take a whole lot of nuance like that into account. There is a meeting scheduled with designers to discuss options.
thanks for the feedback!

@blackfalcon I don't see a use case for the cloud loader currently. However; here is an example of the fluidity I would like to see in our circle loaders. https://codepen.io/jczimm/pen/vEBpoL

@blackfalcon also, here is what we use for a content loader on food https://codepen.io/bloom-dan/pen/OXYQay

@blackfalcon  also, there are different types of loaders...i'm wondering if we need to separate them here in github. Content loader, data loader, interstitial loader, etc.

@blackfalcon I don't see a use case for the cloud loader currently. However; here is an example of the fluidity and sequencing I would like to see in our circle loaders. https://codepen.io/jczimm/pen/vEBpoL

This loader was inspired from @littleninja posting of a use case on the home page.

@blackfalcon also, here is what we use for a content loader on food https://codepen.io/bloom-dan/pen/OXYQay

Yes, but I would argue that the shimmer is a different custom element.

@blackfalcon also, there are different types of loaders...i'm wondering if we need to separate them here in github. Content loader, data loader, interstitial loader, etc.

I am open to having categories of loaders much like how we have them in auro-icon

I want to thank everyone for their feedback on this. It was decided to go with a limited set of loaders for the v1.0 version of auro-loaders. Moving forward @vidalmenAS as offered to take the lead to help this new component grow and fir the needs of the design org as a whole.
Current candidate for consideration v1.0
http://auro-loader.surge.sh/components/auro/loader

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update to Bevy 0.12

Do you need some help with this? :)

Heyho,
Thanks for reminding me!
I actually haven't had much motivation to work on this project, especially since I lacked any feedback.
The update to 0.12 opens up a lot of possible simplifications and rewrites since many important parts of the rendering pipeline were changed.
I think I'll update the project to 0.12 first without rewriting any major logic in the next days!
I also saw a couple of people forking this project in an attempt to update the project to 0.12 but seems like no one got very far.
However if someone else wants to try their luck feel free to do so.
Have a great day and thanks for the reminder (and motivation boost) :)

Hehe, well my feedback is that I love love looove the way your crate looks and would like to continue using it for a long time :)

I gave you collaborator rights to Foxtrot so you can patch it in whenever you want even when I'm too busy to review 😄

I'm new to Bevy, currently learning Rust along with it, and I was very happy to find a working grass plugin. Then, it suddenly stopped working with the 0.12 upgrade. I did not check why until now. I hope some day I'll be able to contribute, but currently, I'm relying on it will just work after a while. So your work here is much appreciated, even if we don't comment :)

Thanks everybody for your support :)
The new release is out!


examples: fix (2020-10)

Catch up examples for crystal-0.35.1.

checker: https://github.com/maiha/crystal-examples
compiler: Crystal 1.0.0-dev [0fa955075] (2020-10-09)
last commit: 0fa955075 Make abstract def return type warning an error (#9810)

Best regards,

@bcardiff
Nice! It's easy to understand with a clear structure of "Definition," "Setup," and "Use.
Log.define_formatter TestFormatter, "#{source(before: '[', after: "] ")}#{message}"
Log.setup(:info, Log::IOBackend.new(formatter: TestFormatter))
Log.for("foo.bar").info { "Hello" } # => - [foo.bar] Hello

I'll push this commit after test.


add cmake testing to our circleci setup

We are currently not testing using cmake with circleci...
https://github.com/lemire/simdjson/blob/master/.circleci/config.yml

@TkTech : do you know a cheap and efficient way to do that? I know how to do it easily on travis... but not on circleci.

Sure, I can flesh out the circleci tests after work if you'd like a PR.

@TkTech That'd be great.
You live in Ottawa?

I do indeed!

So we are almost neighbours. (I was in Ottawa all week last week.)

Put up a PR. We should also remove travisci, or at least disable running the tests and explicitly specify an arch that supports AVX2 for building (ex: -march=haswell).
Not all travisci hosts support AVX2, so builds will continue to randomly fail when it tries to execute on a machine without it (ex: #60) All circleci v2 hosts are Haswell so AVX2 is available.
Could probably also get rid of drone as well, no reason to have so many ci running, doesn't really offer anything. Need to keep appveyor as it's necessary for Windows builds.

@TkTech I removed travis.


Add checkbox visual element

What would that feature address
Add checkbox visual elements into Taipy: checkbox

Parent issue: #436
Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%
[ ] Create related issue in taipy-doc for documentation and Release Notes
[ ] Check if a new demo could be provided based on this, or if legacy demos could be benefit from it
[ ] Ensure any change is well documented

With Checkbox as added bonus (and row classname in stylekit)
from taipy.gui import Gui, State

val = None

def on_change(sdtate: State, var: str, val: str):
  print(f"on_change(sdtate: State, var: {var}, val: {val})")

Gui("""
#test Selector Radio
<|{val}|selector|lov=Item 1;Item 2;Item 3|label=Selector|class_name=row fullwidth|hover_text=This is on the way|mode=radio|>
#test Selector Check
<|{val}|selector|lov=Item 1;Item 2;Item 3|label=Selector|class_name=row fullwidth|hover_text=This is on the way|mode=check|>
""").run()


Not able to launch AWS SAM LOCAL in MAC because of NPE

Description
Unable to launch AWS SAM LOCAL on MAC with eclipse due to a NPE.

Create the Hello World App in Eclipse, with out any modification.
Follow the steps to install SAM Local
Re launched Eclipse
Right click on the java file and select "Run as" -> "AWS SAM Local"
waited for a bit and then I get a NPE.

Steps to reproduce
Provide steps to replicate.
Observed result
I noticed that I can correctly invoke the app by launching it from the SAM cmd line as such:

sam local start-lambda -t serverless.template
aws lambda invoke --function-name "HelloWorld" --endpoint-url "http://<IP_ADDRESS>:3001" --no-verify-ssl out.txt
Output is:{
    "StatusCode": 200
}

Details of software:

OS: 10.14.5 (18F132)
sam --version: AM CLI, version 0.22.0
DOCKER-desktop: <IP_ADDRESS>

Eclipse LOG
eclipse.buildId=4.12.0.I20190605-1800
java.version=12.0.2
java.vendor=Oracle Corporation
BootLoader constants: OS=macosx, ARCH=x86_64, WS=cocoa, NL=en_CA
Framework arguments:  -product org.eclipse.epp.package.jee.product -keyring /Users/Angel/.eclipse_keyring
Command-line arguments:  -os macosx -ws cocoa -arch x86_64 -product org.eclipse.epp.package.jee.product -keyring /Users/Angel/.eclipse_keyring

com.amazonaws.eclipse.lambda
Error
Wed Sep 18 22:43:27 EDT 2019
Failed to launch SAM Local.

com.amazonaws.eclipse.core.exceptions.AwsActionException
	at com.amazonaws.eclipse.lambda.launching.SamLocalDelegate.launch(SamLocalDelegate.java:159)
	at org.eclipse.debug.internal.core.LaunchConfiguration.launch(LaunchConfiguration.java:862)
	at org.eclipse.debug.internal.core.LaunchConfiguration.launch(LaunchConfiguration.java:720)
	at org.eclipse.debug.internal.ui.DebugUIPlugin.buildAndLaunch(DebugUIPlugin.java:1020)
	at org.eclipse.debug.internal.ui.DebugUIPlugin$2.run(DebugUIPlugin.java:1234)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: java.lang.NullPointerException
	at com.amazonaws.eclipse.lambda.launching.SamLocalDelegate.buildSamLocalCommandLine(SamLocalDelegate.java:243)
	at com.amazonaws.eclipse.lambda.launching.SamLocalDelegate.launch(SamLocalDelegate.java:105)
	... 5 more

This seems strange, do you have the latest version of the sam eclipse toolkit as well?

I should have since i just set it up and I didnt use the eclipse market place. I will check to tonight and post the plugin and feature version

Any updates..? I am feeling like a sitting duck here..

Any updates..?

@arvera This is an IDE issue with AWS Toolkit. I created an issue that links out to here but they should be able to help you more, as that is where the NPE is coming from. This might come back to us is there is an specific issue but will need them to debug from their side.
Closing this for now given the NPE is coming from the AWS Toolkit not SAM CLI


[JP] Add description of special syntax of map

Apply #452 changes to Japanese translation

#481

Thank you @hinamiyagk!  @3100 could you take a peek at these changes?

@hinamiyagk Thank you!

Thank you @hinamiyagk and @3100! 👍


[Client encryption]: Add .NET8.0 target

Pull Request Template
Description

This is preliminary step to enable Brotli compression on Cosmos.Encryption.Custom.
.NET8.0 target was added to the project
New compiler complaints were addressed

To be processed after #4757
Type of change
Please delete options that are not relevant.

[] New feature (non-breaking change which adds functionality)

Closing issues
Contributes to #4678

/azp run


Investigate optimizing using multithreading

When analyzing multiple files, it might be beneficial to use a thread pool given that the work per file is fairly self-contained. The only shared state are the global stats. These stats number don't have to be atomic, there could be a set per pool thread that is collated when analyzing is finished.

Implemented with 3aadbe0


How to segment everything in the image use onnx ?

I use onnx model follow this note , and I find no description of  everything in official demo .

I also have this problem.

DId anyone can solve this problem?

I also have this problem.

I also want to realize the "everything" mode by using this library, but actually the masks it generate can't cover the whole image. I want to make every pixel in the image correspond to a mask, what should i do? Is there any parameters to be changed?

Same issue here.

Also trying to accomplish this


Add new Node js course at free-courses-id.md

What does this PR do?
Add resource(s)
For resources
Description
Added Web Programming UNPAS's course on the topics of Node.js
Why is this valuable (or not)?
Node.js is a Runtime environment which can be used for Web development using Web Framework such as React, Angular, Vue, Svelte, and Express JS. Web Programming UNPAS makes easy to learn and understanding about Node.js, Creating a Website with Node.js and Express JS.
How do we know it's really free?
The creator has uploaded the course at YouTube for everyone. It's available and free to learn.
For book lists, is it a book? For course lists, is it a course? etc.
They are courses.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

order it with alphabetical.
Make it the first one will fix the checks error.

order it with alphabetical.
Make it the first one will fix the checks error.

Thanks for reminding me! I'm fixed it now

Thanks, and welcome to free-programming-books!


#18 create gui for search results (cshtml file)

The gui is a partial view that uses the general layout file (_Layout), the Model(Product Class )is needed before the complete implementation can be completed.
Note: the model is null and the images are hyperlinked as placeholders
Time: 2 hrs

Goodjob. Tested and approved.
-Still, uncomplete as the models are null please make changes to that when the model code has been added


[App] Format client error ApiExceptions without a traceback

What does this PR do?
The lightning CLI uses the lightning APIs to create, read, update, and delete resources such as apps and clusters.
If an API call fails due to client error (e.g. invalid input, unauthorized, unauthenticated) the API will return an HTTP status code in the 400s, along with a custom message in the response body.
The CLI (usually) does not directly handle these exceptions. As a result the exception bubbles up through the click framework (which does not recognize the exception type) and to the python runtime, which produces a long traceback and dumps the raw exception to the user.
This PR fixes the user experience. When our API fails on a client error, the CLI will display the message from the server without a traceback.
Before
➜ lightning create cluster \
        lf-001 \
        --external-id blablabla \
        --role-arn blablabla \
        --edit-before-creation
Traceback (most recent call last):
  File "/Users/lucafurst/.pyenv/versions/lightning-2/bin/lightning", line 33, in <module>
    sys.exit(load_entry_point('lightning', 'console_scripts', 'lightning')())
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/cli/lightning_cli.py", line 79, in main
    _main()
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/cli/lightning_cli_create.py", line 79, in create_cluster
    cluster_manager.create(
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/cli/cmd_clusters.py", line 136, in create
    resp = self.api_client.cluster_service_create_cluster(body=new_body)
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/utilities/network.py", line 92, in wrapped
    raise e
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/utilities/network.py", line 80, in wrapped
    return func(*args, **kwargs)
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/api/cluster_service_api.py", line 66, in cluster_service_create_cluster
    (data) = self.cluster_service_create_cluster_with_http_info(
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/utilities/network.py", line 92, in wrapped
    raise e
  File "/Users/lucafurst/go/src/github.com/Lightning-AI/lightning/src/lightning_app/utilities/network.py", line 80, in wrapped
    return func(*args, **kwargs)
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/api/cluster_service_api.py", line 133, in cluster_service_create_cluster_with_http_info
    return self.api_client.call_api(
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/api_client.py", line 362, in call_api
    return self.__call_api(resource_path, method, path_params,
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/api_client.py", line 173, in __call_api
    response_data = self.request(method,
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/rest_client.py", line 75, in wrap
    raise err
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/rest_client.py", line 69, in wrap
    response = func(*args, **kwargs)
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/api_client.py", line 408, in request
    return self.rest_client.POST(url,
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/rest.py", line 307, in POST
    return self.request("POST",
  File "/Users/lucafurst/.pyenv/versions/3.8.12/envs/lightning-2/lib/python3.8/site-packages/lightning_cloud/openapi/rest.py", line 237, in request
    raise ApiException(http_resp=r)
lightning_cloud.openapi.rest.ApiException: (400)
Reason: Bad Request
HTTP response headers: HTTPHeaderDict({'access-control-allow-headers': 'Content-Type,X-Grid-Key,X-Grid-User,Authorization', 'access-control-allow-methods': 'GET,POST,PUT,PATCH,DELETE,HEAD,OPTIONS', 'access-control-allow-origin': '*', 'content-type': 'application/json', 'vary': 'Accept-Encoding', 'date': 'Tue, 25 Oct 2022 23:37:00 GMT', 'content-length': '54', 'x-envoy-upstream-service-time': '2', 'server': 'envoy'})
HTTP response body: b'{"code":3, "message":"name is required", "details":[]}'

After
➜ lightning create cluster \
        lf-001 \
        --external-id blablabla \
        --role-arn blablabla \
        --edit-before-creation
Error: name is required

Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda

Love it!


Search keyword in results page

I would like to be able to search for a keyword (that I know it has executed) on an execution, but I don't want to find out under which keyword it is hiding.
More ideas. Search box with:

Search only keyword names
Search on Fully Qualified Keyword Names
Search on visible log level
Search on all log levels

Search in log would be great. We already have #3799 about that so let's continue discussion there.

Search in log would be great. We already have #3799 about that so let's continue discussion there.


Strip non alpha numeric characters from the beginning of a comment before validating it

In addition to documentation I use comments visually break a source file into sections.  This means that many of my comments begin with a row of "****" or "----".  However this runs afoul of the  "comment on export  should be of the form ....".
If those characters could be stripped before the comment text is checked, as the symbols are obviously not documentation that would be helpful.
Another solution would be (maybe in conjunction with godoc) to have the comment of "private" comments.  Original comments in the code were for developers, then someone got the idea that those comments could also be used for documentation, which is good in general.  However sometimes you still want to provide "internal" documentation that external developers don't need to see.  Often when I want to provide internal documentation for other developers or the future me as to what I was thinking, things to look out for, or the internal workings of a method or object.
Go appears to have taken over the source comments by saying "Comments on public object and public functions are always available for public consumption."  I believe we still need a way to say hey this comment is for future developers working on this function or object.

For my own personal use I've done the change and placed in pull request #226.

I believe a much easier solution to just put the row of **** with an extra line break before the next docstring.
For example:
// Bar1 does nothing.
func Bar1() {}

// Bar2 does nothing.
func Bar2() {}

/**********************************/

// Foo1 does nothing.
func Foo1() {}

// Foo2 does nothing.
func Foo2() {}

Does this solve your use case? I'm not in favor of stripping alpha-numeric characters from a docstring since it leads to worse issues.

I guess that would work...it just comes down to coding practices.  After using Go a little more i'm just starting to feel like Go's coding guidelines conflict with mine too much to even try to appease lint.

The guidelines in Effective Go discourage the use of rows of stars or such as separators.
https://golang.org/doc/effective_go.html#commentary
Comments do not need extra formatting such as banners of stars. The generated output may not even be presented in a fixed-width font, so don't depend on spacing for alignment—godoc, like gofmt, takes care of that.

Thanks for pointing to that, @frasergraham. Closing this.


Support specifying many manifest directories to install in test harness.

What type of PR is this?
/kind feature
What this PR does / why we need it:
Support specifying many manifest directories to install in test harness.
We add a new setting, manifests-dirs and do not remove manifest-dir to maintain backwards compatibility. We may consider removing manifest-dir in the future.
Does this PR introduce a user-facing change?:

Test harness now supports passing multiple directories for installation prior to running tests.

@jbarrick-mesosphere the test harness was only recently added and is still evolving. What would we break if we decided not to maintain backwards compatibility?

Most likely nothing (except maybe our integrity, as it is a part of the configuration format). It did just now occur to me that there is no reason for me to add a new CLI parameter though, I should just make the existing manifests-dir flag a string slice which will retain the old behavior.


HARP-7729: MapViewUtils zoom and distance calculations give wrong value

Codecov Report

Merging #1203 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1203      +/-   ##
==========================================
+ Coverage   47.63%   47.63%   +<.01%
==========================================
  Files         246      246
  Lines       22720    22727       +7
  Branches     5060     5060
==========================================
+ Hits        10822    10826       +4
- Misses      11898    11901       +3

Impacted Files
Coverage Δ

@here/harp-map-controls/lib/MapControls.ts
23.58% <ø> (ø)
:arrow_up:

@here/harp-mapview/lib/Utils.ts
69.17% <100%> (+0.51%)
:arrow_up:

@here/harp-mapview/lib/MapView.ts
58.93% <100%> (ø)
:arrow_up:

@here/harp-test-utils/lib/TestUtils.ts
77.64% <0%> (-3.53%)
: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 e657abc...f69a5af. Read the comment docs.