Many duplicate lint processes can be created

Information
VIM version
NVIM v0.3.0-1336-gcf92a7628 (also replicated on the latest stable build 0.2.2)
Build type: Release
Operating System: mac 10.13.4
:ALEInfo
 Current Filetype: elixir
Available Linters: ['credo', 'dialyxir', 'dogma']
  Enabled Linters: ['credo', 'dialyxir', 'dogma']
 Linter Variables:
let g:ale_elixir_credo_use_global = 0
 Global Variables:
let g:ale_cache_executable_check_failures = v:null
let g:ale_change_sign_column_color = 1
let g:ale_command_wrapper = ''
let g:ale_completion_delay = 500
let g:ale_completion_enabled = 1
let g:ale_completion_max_suggestions = 50
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%code: %%s'
let g:ale_echo_msg_info_str = 'Info'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {'javascript': ['eslint'], 'typescript': ['tslint --fix']}
let g:ale_history_enabled = 1
let g:ale_history_log_output = 1
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 0
let g:ale_lint_on_filetype_changed = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_lint_on_insert_leave = 0
let g:ale_linter_aliases = {}
let g:ale_linters = {'java': []}
let g:ale_linters_explicit = 0
let g:ale_list_window_size = 10
let g:ale_list_vertical = 0
let g:ale_loclist_msg_format = '%code: %%s'
let g:ale_max_buffer_history_size = 20
let g:ale_max_signs = -1
let g:ale_maximum_file_size = v:null
let g:ale_open_list = 0
let g:ale_pattern_options = {}
let g:ale_pattern_options_enabled = 0
let g:ale_set_balloons = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_info = '--'
let g:ale_sign_offset = 1000000
let g:ale_sign_style_error = '>>'
let g:ale_sign_style_warning = '--'
let g:ale_sign_warning = '--'
let g:ale_statusline_format = ['%d error(s)', '%d warning(s)', 'OK']
let g:ale_type_map = {'credo': {'E': 'W', 'ES': 'WS'}, 'tslint': {'E': 'W', 'ES': 'WS'}}
let g:ale_use_global_executables = v:null
let g:ale_warn_about_trailing_blank_lines = 1
let g:ale_warn_about_trailing_whitespace = 1
  Command History:
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/43/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/44/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/45/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/46/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/47/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/48/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/49/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/50/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/51/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/52/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/53/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/54/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/55/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/56/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/57/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/58/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/59/pitch_test.exs''']
(started) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/60/pitch_test.exs''']
(finished - exit code 1) ['/bin/bash', '-c', 'mix help credo && mix credo suggest --format=flycheck --read-from-stdin ''/Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/test/ex_composer_bot/pitch_test.exs'' < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/61/pitch_test.exs''']
<<<NO OUTPUT RETURNED>>>
(finished - exit code 1) ['/bin/bash', '-c', 'mix dialyzer < ''/var/folders/kx/wsz1xqjj5cncxlwzwh1rh_4c0000gp/T/nvimzoPSEi/62/pitch_test.exs''']
<<<OUTPUT STARTS>>>
==> fs (compile)
src/sys/fsevents.erl:2: can't find include file "api.hrl"
src/sys/fsevents.erl:3: undefined macro 'API'
src/sys/fsevents.erl:5: Warning: function find_executable/0 is unused
src/sys/fsevents.erl:8: Warning: function known_events/0 is unused
src/sys/fsevents.erl:13: Warning: function start_port/2 is unused
src/sys/fsevents.erl:17: Warning: function line_to_event/1 is unused
Compiling src/sys/fsevents.erl failed:
ERROR: compile failed while processing /Users/Dylan/Dropbox/Programming/GitHub/ex_composer_bot/deps/fs: rebar_abort
<<<OUTPUT ENDS>>>

What went wrong
Many duplicate linting processes can be created

Reproducing the bug
Steps for repeating the bug:

Open htop and sort by cpu usage
Open vim in a project with lint
As a worst case scenario, type: aa<cr>aa<cr> several times until you see several duplicate linting processes in htop

I have been able to replicate this with credo (elixir) and eslint (js)
This is very annoying because my CPU gets maxed out relatively easily through normal vim usage. My normal vim usage involves going in and out of insert mode pretty frequently,

Right, this does help with CPU usage but it does not actually fix the problem. The problem is that multiple lint processes can be started at the same time, which is a waste of CPU resources.
Perhaps some better solutions would include:

Killing an existing process for the lint just before starting the new one
wait for the previous one to finish before starting the new one

On 7/06/2018, at 6:22 PM, w0rp<EMAIL_ADDRESS>wrote:
https://github.com/w0rp/ale/blob/master/README.md#faq-my-battery-is-sad https://github.com/w0rp/ale/blob/master/README.md#faq-my-battery-is-sad Look at this FAQ entry and change your settings.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/w0rp/ale/issues/1637#issuecomment-395306748, or mute the thread https://github.com/notifications/unsubscribe-auth/AQrCbeBtkJpB7A9TfDHe-pOTbcHC9MS6ks5t6MbBgaJpZM4UdxQo.

Killing an existing process for the lint just before starting the new one

That's what ALE does, NeoVim just isn't good at doing that. NeoVim had a bug where it didn't create processes in a process group and then kill the group like Vim 8 does, but I believe that was supposed to have been fixed in a recent version. Maybe their bug fix didn't work. Try the same workflow in Vim 8 and see if it does a better job. If it does, then there's still a bug in NeoVim they should fix.
Maybe there's a bug in ALE you could try to fix, but I think ALE is doing everything it should. Have a look at ale#engine#RunLinters and follow the logic there.

Okay, sounds good to me. This problem has been raised a few times, and it will thankfully be solved in the next release. (0.3)

Fingers crossed they release before i grow old...

See this FAQ entry: https://github.com/w0rp/ale#5xiii-will-this-plugin-eat-all-of-my-laptop-battery-power
If anything is running too often, configure ALE to run programs less often. If neovim isn't killing processes properly, tell them to fix it, or try Vim instead.

With 2 .ex files open, and g:ale_lint_on_text_changed = 'never' I still see dozens of processes. My CPU seems to max out when I use the mix linter, so I now only have credo set. That being said, when I save a file, dozens of Erlang processes are created. They stick around for a few seconds, then they are killed. So it seems like there might be a couple of issues here... mix linter going off the rails and the credo linter booting up too many processes?
When I run mix credo only one Erlang processes is created.
Maybe neovim/neovim#6530 didn't fix the issue after all? Is anyone else seeing these issues?

Maybe you can configure mix to only run in a single process, or disable the linter completely, if it consumes too many resources.

I've disabled the mix formatter and set the credo formatter to only run on file save.

I'm still seeing a whole bunch of processes with the credo checker enabled, more than I see when I run mix credo from the command line.
I'm guessing that so many multiple processes is actually a bug, but I'm not sure where to report it. I have not seen that behavior in using mix format. Is this a neovim thing, an ale thing or a mix/credo thing? Tough for me to tell. Thanks for your help!

Could this be related to #2084? Compiling an entire project usually spawns a lot of erlang processes.


Example URL in README broken?

Hey JT,
Totally looking at using this for a project and wanted to check out the example URL but I couldn't load the page ;/

Yeah, when I got an AWS bill over $100 / month, I had to rethink my deployment.
The example URL didn't actually include the pull request resource running itself against itself. This was the pipeline it was running.
What were you looking for?

Removed the broken link. Will return.


[feature] generate the report in the desired path

This PR adds the ability to generate a report file in the desired path using CLI and API.

@KostiantynPopovych the new feature looks good to me! i especially like the inclusion of a dedicated logging library!
but i have to say i'm against including CLI functionality in the core depngn function. i believe core and cli should remain separate modules, with cli importing depngn and not the other way around. (i know i'm not an actual maintainer of this package anymore, but i feel strongly about this one, so i figured i'd voice my opinion on it).
Summed up arguments against it:

it violates the "Single Responsibility" principle in SOLID -- depngn should only output compatibility data and cli should handle the reports (and, by extension, validation).
it forces anyone who installs depngn from npm to include ourcli modules, even if they never intend to use them.
(subjective) for some reason, i don't like the idea of a function that can return a value or execute a side effect and return undefined. i feel like that's an unexpected/inconsistent public interface. especially because the return changes based on unrelated arguments passed to it (the absence of reporter and reporterPath and not, for example, an argument like returnData: true or something).
i think it makes navigating the codebase less straightforward. the way it currently is, you can expect functions used in one module to come from that module (with the exception of depngn being called in the cli function). in this implementation, you have functions from cli being imported into core, which could make it difficult for potential contributors to follow the execution flow. i know it's only a few functions now, but i think strictly separating them is a better convention for future maintenance.

besides all that, it was a conscious decision to keep them separate when i initially wrote this so users could import depngn as an npm package to generate the data themselves and then choose what to do with that output.

Hey @kindoflew! Thank you for reviewing! I wrote this after a break, so the initial approach is not perfect, and your points are fair. I rebuilt it, created a standalone entity - report and now I hope it fixed all previous concerns. I didn't update the related docs so that you can review this again, and after that, I will update README.

@KostiantynPopovych -- oohh, i didn't even think of having report be a separate module the user can import! neat!
looking over everything, i wonder if including the file name as part of the reportOutputPath is more work for us and the user than we need?
for example -- in istanbul you can choose the reportDir and that only refers to the directory -- the file type is chosen by reporter and i don't think you can customize the file name. i wonder if that is a less complex option -- both for us maintaining and devs using?
the main reason i think this is because, as it is, a user would have to remember and know that if they specify a filename it has to have the matching extension or it throws an error. not the biggest deal in the world, but maybe a little bit of a clunky user experience?
but reportDir is unambiguous -- there's no way to screw it up as long as it's a valid name for a directory. we could also have a reportFileName option that doesn't need the extension -- that could be inferred from reporter. what do you think?

@kindoflew, hmmm, I thought having an additional option for file name would be too much, but it will definitely be less complicated for us to maintain and probably for users to get used it. So, yeah, I will split the logic into two separate options (the idea of this PR under the hood is to be able to create a report with some specific file name)

@kindoflew, updated, two options are in place)


Create stripped-down output with only key columns

Create an output with only columns that can be used to perform the crosswalk

This can easily be done by dropping all other columns from the output.


[#139179971] Upgrade cf-cli to 6.25.0 in cf-acceptance-tests

#139179971 Upgrade CF to >= v252
What?
While upgrading to CF v253, the acceptance tests use the command cf run-task[1] which was included in cf-cli 6.23.0[2]
We decided upgrade to the latest client version 6.25.0
[1] http://cli.cloudfoundry.org/es-ES/cf/run-task.html
[2] https://github.com/cloudfoundry/cli/releases/tag/v6.23.0
[3] https://github.com/cloudfoundry/cli/releases/tag/v6.25.0
How to review?
Code review shall be enough
Who?
Anyone but @keymon or @csaliceti

works fine, merging


Improve keyboard handling

Never been 100% happy with the way the keyboard debounce/repeat is handled in MOS, esp. when returning keycodes to BASIC. Look at improving it so that INKEY$ will always show the keycode, rather than the debounced version used in MOS/GET$

Testing:

KEYBOARD_1.BAS: GET$ - passed


babel.min.js

<EMAIL_ADDRESS>https://cdnjs.com/libraries/react/
https://github.com/facebook/react/issues/5497
https://facebook.github.io/react/downloads/single-file-example.html

https://facebook.github.io/react/docs/hello-world.html
http://codepen.io/gaearon/pen/ZpvBNJ?editors=0010
https://developer.mozilla.org/en-US/docs/Web/JavaScript/A_re-introduction_to_JavaScript

https://learning.xgqfrms.xyz/Front-End/React/React/app01/


Fix TO Go cachegroups PUT/POST API endpoints

The Go version of the cachegroup API was looking up parent cachegroup
IDs using the parent names in the PUT/POST request. This is
fundamentally different from the Perl version which only accepts the
parent IDs and doesn't look them up by name.
This commit also:

adds cachegroup client methods that use Nullable structs for
input/output (to have something usable for the API tests - the
incorrect implementation of the API was holding the tests up)
updates all API tests to use the new Nullable cg methods
fixes the cachegroup API test to verify the proper API behavior for
PUT/POST

Fixes #2559

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2125/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2126/
Test PASSed.


Add Replicate demo and API

Hey @Xiangtaokong  ! 👋
This pull request makes it possible to run your model inside a Docker environment, which makes it easier for other people to run it.  We're using an open source tool called Cog to make this process easier.
This also means we can make a web page where other people can run your model! View it here: https://replicate.com/cjwbw/swin2sr
Replicate also have an API, so people can easily run your model from their code:
import replicate
model = replicate.models.get("cjwbw/swin2sr")
output = model.predict(image="...")

You are more than welcome to modify the Replicate page (e.g. Example Gallery), let me know and I can transfer ownership to your Replicate account or https://replicate.com/mv-lab.
In case you're wondering who I am, I'm from Replicate, where we're trying to make machine learning reproducible. We got frustrated that we couldn't run all the really interesting ML work being done. So, we're going round implementing models we like. 😊

@chenxwh @Xiangtaokong sorry for the late reply. We appreciate this PR and the replicate demo.
Please transfer the ownership of https://replicate.com/cjwbw/swin2sr to my Replicate account or https://replicate.com/mv-lab.

Hi @mv-lab,
Thanks for merging the PR, the page now is under mv-lab account, view it here: https://replicate.com/mv-lab/swin2sr :)


fugue metadata upgrades

In the new Fugue digital archive, older items have metadata of contributors and editors. More recent items 2017+ have no metadata (except date and issue_number). Upgrading the metadata for the newer items would be a good short-ish metadata project (about 18 items).

[ ] extract list of editors to editors column
[ ] extract list of contributors to creator column
[ ] check title (I just generated these from the number and date)

@mkoepele this would be a good project for you, and get to read UofI poetry!

Added metadata to the more recent items!


[MRG] Py37 build artifacts

This PR removes the explicit version for cython, which was preventing build artifacts for Python 3.7 (#76), and adds a check to make sure our build artifacts were created before deployment.
I added the check in the before_deploy section of the build as noted here, so it will only get run on a tagged release (i.e., it won't run when we merge this in). I initially had it in the after_success phase, and I am more than happy to move it back, but it made more sense to me here.

If y'all would like to see a build where the assertion ran correctly (since it won't run unless we are on a tagged branch), see here

Codecov Report

Merging #77 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #77   +/-   ##
========================================
  Coverage    98.51%   98.51%
========================================
  Files           37       37
  Lines         1746     1746
========================================
  Hits          1720     1720
  Misses          26       26

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 1d17960...cf1dd36. Read the comment docs.


Adds LMEvalJob resource

Short description:
Adds LMEvalJob resource to the wrapper
More details:
CRD needed to run an evaluation job on a language model using lm-evaluation-harness
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

Summary by CodeRabbit

New Features

Introduced the LMEvalJob class for managing evaluation jobs with customizable parameters.
Added functionality to convert job instances into a structured dictionary format.

Bug Fixes

Implemented error handling for missing required attributes in job configurations.

/verified


RHIDP-4362 - Add VM images to Using Dynamic Plugins docs

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s):
1.3
Issue:
RHIDP-4362 - Add VM images to Using Dynamic Plugins docs
Link to docs preview:

Reviews:

[ ] SME: @debsmita1
[ ] QE: @gustavolira
[ ] Docs review: @hmanwani-rh
[ ] Additional review: @mention assignee (by writer)

Additional information:

Closing as this content was published in 1.3


Using "call" command to open VSCode on Windows 10 doesn't close the prompt

VSCode Version: Code 1.11.1 (d9484d12b38879b7f4cdd1150efeb2fd2c1fbf39, 2017-04-06T13:58:12.954Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions: none

The following commands  running from a batch file on windows 7, opens a new instance of visual code and exits the prompt. On windows 10 the prompt remains open until the VSCode instance that was opened is closed.
call code
exit 0

What is your intent?

To open some workspaces automatically at logon

You can point automatically to Code.exe, bypassing code.cmd altogether.

I had already worked around it, the point of the report was just to notify that on the referred environment vscode is holding on to the execution instead of returning to the calling context as one would espect using the "call" command  and as is currently happening in windows 7.
Thanks


Link is not rerendered when location changes, causing active state to go out of sync.

Version
2.0.0
Steps to reproduce
Expected Behaviour
Link render() is called when the location changes.
Actual Behavior
Link render() is not called when the location changes.

I'm sorry the issue request is so vague. I'm setting up an example as we speak!

Okay,
sorry for the preemptive issue. I figured it out. Apparently, when there's a component anywhere in the tree above the Link that has the PureRenderMixin, it doesn't re-render.
In hindsight, this is pretty obvious but it took me a while to figure out. Have some semantics considering context changed? I guess so because before, I never had this problem.
I'm using pure-render-decorator to decorate my classes, I'm guessing they should add a test for context as well.

See https://github.com/reactjs/react-router/issues/470. It's tricky.


Upload is not working anymore

After recent update "upload" command is not working anymore. It just doing nothing.

I have noticed this also.

Same problem. Thought I was going crazy.

Also can confirm this.

as version 0.7.20 upload works fine for me.

version 0.7.20
I can download but can't upload . I am new to remote-ftp. No error messages.

+1

+1

Is anyone still having this issue? As I cannot seem to replicate it

Thanks for posting. Reopen if you have any issues with the latest version.


Totally Wrong. Just Wrong

First, there is no such option under Policies. The only option under policies in Access Control.
There is something under Sites > Active Sites > <Select Site> > Sharing
Not only is the UI incredibly stupid (Do we really, really, need five separate places to enable this?) but the docs are just plain wrong.

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

ID: 6ed46514-8036-a543-d690-73d48acef690
Version Independent ID: 8542c3ce-3012-3d23-c3e4-4331dac033d2
Content: Turn external sharing on or off
Content Source: SharePoint/SharePointOnline/turn-external-sharing-on-or-off.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@officedocsbot assign @scanum

@boomalator thank you for reaching out and helping to improve MS docs. We value your contribution.  Could you maybe suggest the changes that you think should be put in place to improve the UI?

@boomalator Sorry for the confusion. We recently rolled out a new Sharing page and updated this article to correspond, but not everyone had the new page yet. I have since updated this article with new screenshots, and I hope you're now seeing the page shown in the screenshots.


fix: list workflows from cache during initManagers. Fixes #13303

Fixes #13303
Signed-off-by: Liang Zheng<EMAIL_ADDRESS>

Trying to understand the current state of this PR with respect to the comments that @agilgur5 added. Github doesn't show me the comment dates, and the commit history was rewritten, so I can't tell which comments are still outstanding and which ones aren't. Thanks.

Trying to understand the current state of this PR with respect to the comments that @agilgur5 added. Github doesn't show me the comment dates, and the commit history was rewritten, so I can't tell which comments are still outstanding and which ones aren't. Thanks.

hi @juliev0, This PR is currently not making progress, and I am unable to correctly understand the solution mentioned in the comments.


feat: Allow creating VPC routes for already existing or shared TGW

Description
These changes allow the creation of VPC routes for already existing or shared TGWs via tgw_id that already exists without changing the input variables.
Motivation and Context
Currently, VPC routes can point only to the transit gateway created by the module itself.
Breaking Changes
No
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

Given that this PR is currently blocking our production deployment, could we please move forward and merge the PR?

@antonbabenko ping

@antonbabenko ping

I am starting to use this module now, is the project dead? I am having the same issue


[Bug-12956] fix incorrect java path

Purpose of the pull request
Fix the problem reported by this issue #12956。
This pull request  update java path。
Brief change log
fix incorrect java path
Verify this pull request
This pull request is code cleanup without any test coverage.

@zhangfane You could either run mvn spotless:apply or rebase your code on the latest dev branch to pass CI.

@EricGao888 done

@zhangfane Something wrong with CI, could u plz rebase on the latest dev branch? It will help make CI pass and I could merge this PR. Thx : )

@EricGao888 done


Treasury Report - Unblock testing of file-generation in staging environment for USDR_ADMIN users

Context
While some of the initial file generation pieces are being merged into staging, there is no way to easily test this feature piece-meal via the staging environment. The objective of this ticket is to be able to create a way for a USDR_ADMIN user to directly call each of the new lambda functions that are being added download the resulting files being generated.
Current State
N/A
Expected State
USDR_ADMIN users see the following buttons at their disposal on the homepage. Note: these buttons should not appear for any other user-role and will be discarded once the Treasury Report feature is fully built-out.
Form Field 1 - Accepts a long text containing payload from the user.
Button 1 - Kick-off file generation
Button 2 - Download Project 1A file
Button 3 - Download Project 1B file
Button 4 - Download Project 1C file
Button 5 - Download Subrecipient file
All the "Download ***" buttons should create a GET signed-url for the s3 object containing the project/subrecipient file for the current reporting period of the organization.
The "Kick-off file generation" button should send any payload that the user has added in "Form Field 1" to all the lambda functions.
Implementation Details

Add a Redwood Form
All download buttons can follow the same pattern as the existing Upload file download button except the s3 key will be different.
Add documentation for users on how to fill out the form and structure the payload

Note for context - 295 & 296 & 320 are permanent solution for file generation. This issue (348) is a workaround to allow generation of files before those are done


Fix the Coveralls badge in the README

The badge was showing coverage for the cli-refactor branch instead of master.

Coverage remained the same at 96.262% when pulling baa698c3e4b7b39204f438d618b09ae9d6f2e6f7 on fix-coveralls-badge into 6354ad8817a96eaea136d33bd0607709753bbdab on master.


Support Google Cardboard

Dropdown for VR integration should support both Oculus and Google Cardboard
Should also support full screen and vr camera selection via URL params, so can be put into QR code.

Added in 9a79e1a2a20f03c17ede4ce88f6f46b37e4354a5


Crash when running tests

Hi!
When I run the tests in Xcode-beta / Swift 6, I get the following crash:
Thread 3: Fatal error: 'try!' expression unexpectedly raised an error: Error Domain=NSCocoaErrorDomain Code=257 "Het bestand 'Inputs' kon niet worden geopend, omdat je geen bevoegdheden hebt om het te bekijken." UserInfo={NSUserStringVariant=(

Unfortunately, I can't see which test triggers this.
Kind regards,
Maarten

Thank you, I'll check this later.
Additionally, we don't run tests for Xcode 16 beta on GitHub Action, so I'll prepare a CI environment to run tests for Swift 6.

Umm, I can't reproduce this issue on my environment with 16 beta 2.
How about modifying SwiftTestingBridge.swift?
#if compiler(<6.0)

// Below Xcode 15.x, we can't run swift-testing directly. I believe it will be unnecesarry since Xcode 16.
final class AllTests: XCTestCase {
    func testAll() async {
        await XCTestScaffold.runAllTests(hostedBy: self)
    }
}

#endif

This bridging is not required for Xcode 16 and Swift 6 compiler.

I guess your repository is in a not readable/writable path

its kinda weird: I gave Xcode-beta full disc access. And also tried with the repo in a different path.
Might very well be an Xcode-beta bug though. I'm going to try if I can run the tests somewhere else.

Could you see the full path of the error message?
According to the screenshot, Operation is not permitted. RevolutionaryKitTests read fixtures from a file system, but It seems to be failed.

Just tried running the tests after moving the repo to /tmp . Now the tests run correctly.
I'm writing this one up to an Xcode bug. So I'll create a feedback message for Apple.

FYI: it seems this is fixed in the new Xcode beta 3.


document.body doesn't exist. Modernizr hyphens test needs it.

This happens only after upgrading to <IP_ADDRESS>. Reverting back to <IP_ADDRESS> is fine.
Also in the log:
Uncaught TypeError: Cannot call method 'appendChild' of null

So one of the builder option requires a special loading? It seems like it cannot support loading in head tags anymore?

I am having this issue in Modernizr 2.8.3.
'webkitAudioContext' is deprecated. Please use 'AudioContext' instead.
modernizr.min.js?body=1:5 document.body doesn't exist. Modernizr hyphens test needs it.
modernizr.min.js?body=1:6 Uncaught TypeError: Cannot read property 'appendChild' of null

Might need to provide more context on how to reproduce the error, if you want to get help with this (unless other people are already having the same error, and know how to fix it).  You can also try poasting your issue in https://github.com/Modernizr/Modernizr, if it's just an issue with Modernizr (this is just the gem to incorporate Modernizr as a rails asset)

Thank you


Remove Beta Banner from collections publisher

What
Remove beta banner in Collections Publisher
Why
This is incorrect as the product is not beta and is not being rebuilt - this may be considered misleading to the user.
https://trello.com/c/aFpyXJdm/415-remove-beta-banner-in-collections-publisher
Before

After

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

It looks like this change means that the UI no longer shows 'Collections Publisher' anywhere obvious for the user to know which app they're in – since the beta banner is what showed the app name in a prominent position.
Is this okay? Do users benefit from knowing which Publishing app they're using, or is that not useful information?

Good spot - I've added the product name in


Update BaseModel.ts Typo

This is a potential typo here I noticed when using the SDK.
Did you mean deep clone?

Thanks, it was a typo.
btw. If you are using node17+ you may want to use the native structuredClone() method instead of the "hacky" clone().


Bitfinex :: IOT, BCH, OMG, XMR, ...

failed crypto assets
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository

Action taken

Expected result
more crypto assets

Actual result (unexpected outcome)

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

ill work on that during the day

the same would be nice for Kraken. For example I would like to trade in EUR / XRP (Ripple), Kraken allows it, but I cannot choose the option in the bot.

Hey.
Currently at work. Im consideribg kraken updates as well.
-------- Ursprüngliche Nachricht --------Von: kalzso<EMAIL_ADDRESS>Datum: 20.08.17  20:35  (GMT+01:00) An: askmike/gekko<EMAIL_ADDRESS>Cc: decoye<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Betreff: Re: [askmike/gekko] Bitfinex :: IOT, BCH, OMG, XMR, ... (#1011)
the same would be nice for Kraken. For example I would like to trade in EUR / XRP (Ripple), Kraken allows it, but I cannot choose the option in the bot.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/askmike/gekko","title":"askmike/gekko","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/askmike/gekko"}},"updates":{"snippets":[{"icon":"PERSON","message":"@kalzso in #1011: the same would be nice for Kraken. For example I would like to trade in EUR / XRP (Ripple), Kraken allows it, but I cannot choose the option in the bot."}],"action":{"name":"View Issue","url":"https://github.com/askmike/gekko/issues/1011#issuecomment-323603118"}}}

Uploaded a patch to my fork.
Gekko patch 2. Wait for mike to implement it.

This has been fixed by @decoye (see #1015).

@decoye
thx for you patch, but this won't work.
ERROR: Gekko does not support this currency/assets pair at Bitfinex
i think the problem is this (IOTA vs. IOT).
{ pair: ['USD', 'IOTA'], minimalOrder: { amount: 0.01, unit: 'asset' } },
after change this to "IOT" and then this will work but with a other problem.
`
2017-08-21 17:05:55 (INFO):	Setting up:
2017-08-21 17:05:55 (INFO):		 Paper Trader
2017-08-21 17:05:55 (INFO):		 Paper trader that simulates fake trades.
/gekko/plugins/paperTrader/paperTrader.js:14
this.fee = 1 - (calcConfig['fee' + calcConfig.feeUsing.charAt(0).toUpperCase() + calcConfig.feeUsing.slice(1)] + calcConfig.slippage) / 100;
^
TypeError: Cannot read property 'charAt' of undefined
at new PaperTrader (/gekko/plugins/paperTrader/paperTrader.js:14:57)
at load (/gekko/core/pluginUtil.js:100:22)
at /gekko/node_modules/async/dist/async.js:1156:9
at replenish (/gekko/node_modules/async/dist/async.js:1030:17)
at iterateeCallback (/gekko/node_modules/async/dist/async.js:1015:17)
at /gekko/node_modules/async/dist/async.js:988:16
at /gekko/node_modules/async/dist/async.js:1158:13
at load (/gekko/core/pluginUtil.js:68:14)
at /gekko/node_modules/async/dist/async.js:1156:9
at replenish (/gekko/node_modules/async/dist/async.js:1030:17)
at iterateeCallback (/gekko/node_modules/async/dist/async.js:1015:17)
at /gekko/node_modules/async/dist/async.js:988:16
at /gekko/node_modules/async/dist/async.js:1158:13
at load (/gekko/core/pluginUtil.js:68:14)
at /gekko/node_modules/async/dist/async.js:1156:9
at replenish (
/gekko/node_modules/async/dist/async.js:1030:17)
at iterateeCallback (
/gekko/node_modules/async/dist/async.js:1015:17)
at /gekko/node_modules/async/dist/async.js:988:16
at /gekko/node_modules/async/dist/async.js:1158:13
at load (/gekko/core/pluginUtil.js:68:14)
at /gekko/node_modules/async/dist/async.js:1156:9
at replenish (
/gekko/node_modules/async/dist/async.js:1030:17)
`

@netcoding It seems like you are running Gekko over the CLI. In that case please update your config so that is has these 2 properties: https://github.com/askmike/gekko/blob/stable/sample-config.js#L203-L204

Ohh yes, you are faster as i am ;-)
I've also see this and added this in my config.
'feeMaker: 0.15,
feeTaker: 0.25,
feeUsing: 'maker','
But the the failure in the string description "IOT != IOTA" will keep.

Hey hey,
im not a user of bitfinex, so i could not test it.
just went to the website and tried to figure out how they name their asset pairs.
So IOTA is infact IOT?

@decoye
Yes, IOTA is infact IOT.
See here, https://bitfinex.readme.io/v2/docs/ws-general .
Anyway, thanks so much to @askmike & all other submiters for this fantastic tool. i love it.


Keep extra log attributes (fixes #276)

What was changed
Fix for #276, removing the calls to super().process(...) so that we do not overwrite the extra attributes for the log record.
Why?
So that we can properly support structured logging with extra attributes
Checklist

Closes #276
How was this tested: Tested against the patched "hello_activity.py" described in #276

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

There is an issue w/ latest Python static linking. Unrelated to this PR, but I will have to fix it first then I'll get back to this.

Thanks for merging @cretz! When can we expect a release with these fixes?


getCachedBounds should use the identity matrix

Sorry but I do not have time to make a clean PR (I do not use git on a daily basis so it always takes me half an hour to make it work) but we found and fixed a bug in the cacheAsBitmap feature.
When you calculate the width and height of a cached container, the values are wrong because the cached sprite multiply its bounds by the worldtransform matrix. It shouldn't.
How to fix :
/**
* Calculates the bounds of the cached sprite
*
* @private
*/
DisplayObject.prototype._getCachedBounds = function ()
{
    this._cachedSprite._currentBounds = null;

    return this._cachedSprite.getBounds(core.math.Matrix.IDENTITY);
};
```

at https://github.com/pixijs/pixi.js/blob/master/src/extras/cacheAsBitmap.js#L252

Regards.

Offtopic. So glad you're here! Do you want to join our team slack?

Merged, thanks as always @JiDW  and @ivanpopelyshev.


[bug] Hosted Gorilla:

Exception: Error communicating with OpenAI: HTTPConnectionPool(host='zanino.millennium.berkeley.edu', port=8000): Max retries exceeded with url: /v1/chat/completions (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7a33981022f0>: Failed to establish a new connection: [Errno 111] Connection refused'))
Failed model: gorilla-7b-hf-v1, for prompt: I would like to translate 'I feel very good today.' from English to Chinese

Thanks for raising this @TaoAthe The provider shut down the Gorilla server for scheduled maintenance, and is now up!


feat (Mathlib/RingTheory/MvPowerSeries/NoZeroDivisors) : power series with constant coeff unit is a unit

Prove properties of the inverse of a power series over a general Ring.
We prove that a multivariate power series is a unit iff its constant coefficient is a unit.
The proof uses the construction of the inverse (which is a priori only a left inverse) and proves it is a right inverse.
To that aim, we prove the more general result that if the constant coefficient of a multivariate power series is not a zero divisor, then the power series itself is not a zero divisor.

Shouldn't the PR title refer to Mathlib/RingTheory/MvPowerSeries/NoZeroDivisors? That's where most of the changes/contributions seem to be.

You're right. Having an inverse in the general case was my motivation, but given what was there, all I had to do is proving that constant coeff invertible (or non zero divisor) implies non zero divisor.

Perhaps the file NoZeroDivisors should be NonZeroDivisors, given the name of the structure?

Late on, I will prove here that MvPowerSeries satisfies NoZeroDivisors… Either name is fine to me.

Then I agree with this naming.

Thanks!

maintainer merge

bors r-

We should really fix the bib linter. I think you have to put the reference in alphabetical order wrt to the author manually.

The problem with the bib file with that one should not format it by hand, but just execute bibtool…

Thanks!
bors merge


Slow completions in Python + Jupyter: Code process pegged at 100% CPU

Type: Bug
I am using VS Code, Python and Jupyter on Linux Mint. (Details included.) After some amount of usage, code editing in Python cells in a Juypter notebook becomes very slow. Auto-completion suggestions take a long time (even though I didn't even need them). The cursor refuses to select text for 5-10 seconds until a completion hint appears. A code process appears to be using 100% of a single CPU (my system has 8.)
Specific example: suppose we have this:
    x1 =
    x = np.concatenate([np.asarray(x1), np.asarray(x2)])

We want to select np.asarray(x1), cut it, insert x1 and paste past the x1 =. Position the cursor before np. Hold down the mouse to select. No response. Drag the cursor past the (x1) and release. No response. Wait for 10 seconds. Eventually, the np.concatenate help appears, after which the desired text is selected.
This is then repeated on every edit. Type some, VS Code decides this would be a good point to be helpful and the world stops for 5-10 seconds.
Multiple code processes are active. One of them consumes 100% of a single CPU even when I leave VS Code idle for some period of time as when typing this report.
There are multiple other bug reports similar to this one. I reported a similar issue some weeks ago as a comment on some other bug report. There is often a question of PyLance. Does PyLance run inside a code process, inside a python process or as its own process? I ask because there is no Python process consuming CPU, though multiple are running due to open Jupyter sessions. Only the one code process has run amok. All other processes on the system are basically idle.
In case there is useful information, the command line arguments for the runaway process are:
--type=utility --utility-sub-type=node.mojom.NodeService
--lang=en-US --service-sandbox-type=none
--dns-result-order=ipv4first --inspect-port=0
--crashpad-handler-pid=1177399
--enable-crash-reporter=b8373450-4de6-4c25-9955-af1fa298bdef,no_channel
--user-data-dir=/home/paul/.config/Code
--standard-schemes=vscode-webview,vscode-file
--enable-sandbox --secure-schemes=vscode-webview,vscode-file
--cors-schemes=vscode-webview,vscode-file
--fetch-schemes=vscode-webview,vscode-file
--service-worker-schemes=vscode-webview
--code-cache-schemes=vscode-webview,vscode-file
--shared-files=v8_context_snapshot_data:100
--field-trial-handle=0,i,7313903016689858413,10432611641339754399,262144
--enable-features=kWebSQLAccess
--disable-features=CalculateNativeWinOcclusion,SpareRendererForSitePerProcess
--variations-seed-version

The workaround is to exit VS Code and restart. This gives snappy performance for some hours before things bog down again. So, a habit of restarting VS Code several times per day will prevent the issue at the cost of having to restart notebooks and get them back into the desired execution state.
Broader Issue: Excessive "Help"
The shame is that performance is slow for something I don't need or want in this case: I don't need that pop-up intellisense help for this simple edit.
If each mouse movement or key press has to wait for various add-on processes to complete, then lags are inevitable: the slightest performance issue anywhere slows down the most basic typing tasks. Since VS Code is primarily about editing, secondary actions should occur outside the respond-to-event-and-update-editor loop, IMHO. So, a design suggestion would be to separate completions from the I/O path: do completions in the background and present them once they are ready. If the user has moved on, then the user didn't need that suggestion, so discard it. By having tasks block the edit loop, VS Code is bringing back memories of good old fashioned dial-up terminals to a busy timeshare system with multi-second lags per keystroke.
More generally, the constant pop-up "help" is often a nuisance: the pop-ups block text I'm trying to edit. Since I don't need the dialogs, I've gotten into the habit of making them go away by frequently press the Esc key. In Jupyter, however, pressing Esc when no pop-up is actually open turns out to move focus from the code to the cell. This means I'm using brain cycles to figure out if I need to press Esc so I can see my work, or if doing so will switch focus so I have to reach over the the mouse and refocus on the code.
Further, once some pop-ups occur, they capture the arrow keys. In Python, type 'foo {}'., then decide you want to use the arrow key to move up a line to copy something. Unfortunately, now, the arrow key scrolls through the list of choices, so, after a dot, one has to remember to hit ESC Up-Arrow, but if I had not typed the dot, then there is no ESC first. Or, if I typed foo. (not a quoted string but with a dot), then the capturing pop-up does not appear, and I can use the arrows. It is exhausting to remember all this!
The result is a cumbersome experience. The lags, when they occur, just make a tedious situation worse.
One way to avoid these issues is wait some time, 1/2 sec, say, of no activity before popping up the text. This is enough time to indicate the user wants the help. Otherwise, the user would just keep typing and the window would not pop up, and there would be no need to hit Esc to make it go away.
Or, have some keystroke that turns off pop-ups temporarily. Re-enable them on restart or when the trigger key sequence is hit again. Extra credit for an indicator in the status footer.
Or, have a mode where popups do not appear unless the user presses some key asking for one. That is, rather than typing ' ESC foo {}' ESC . ESC format ESC ( ESC bar ESC, I just type 'foo {}'. <HELP> (bar if I want help in remembering format and type nothing I just type format myself. Maybe this functionality already exists and I just have not found it?
A general solution to the excessive pop-up issue would not only improve user experience in general, it would take the pressure off the cases where some task or other takes a long time to display its popup. That one dialog would take 10 seconds to appear, but I'd care only if I wanted to wait for it.

Just happened again. There seems to be no definite trigger. VS Code has been running for a few days, in use for roughly 5-10 hours during that time. I was working on a notebook with roughly 30 MatPlotLib charts. Performance was good until suddenly it wasn't. I'd been working for a couple of hours on this same notebook. The performance hit when I was doing pretty much the same thing as for the past hour: fiddling with charts and seeing how they look. The notebook also has a few hundred lines of code split across a dozen cells, in case that matters. This notebook uses the standard MatPlotLib backend, no fancy visualizations or animations. All editing is run-of-the-mill: typing, copying, pasting; nothing fancy at all.

Happened again after another few hours of work and about 2 days of up time. This time I get a window saying that the window is not responding and asking if I want to restart. This is a change from the earlier behavior where the system would just become unresponsive, so thanks for this fix.
The problem now occurs when typing strings. In a notebook, using Python, in a code cell:
plt.axhline(0, color='gray', linestyle='|')

Cursor is at the "|" caret. About to type "--". However, the UI pauses for 5-10 seconds, then pops up a selection with things like "ymax=" and a list of files. When I type the first "-", there is another few second pause as the system reduces the list to just the list of files that contain a dash.
As noted in the original post, this is not as helpful as one might think: I almost never enter file names, or argument names, into the string I'm typing.
I also observe that some part of the intellisense seems to have broken. Start typing the following in the same code cell:
plt.

Prior to the CPU hitting 100% in a code process, I would get a popup of symbol names, which is actually helpful here. After fiddling around to create this report, auto-complete has stopped working. It also now takes many seconds for the colors to change, for the red squiggly line to disappear, etc.
Finish typing the line above:
plt.axvline(0.5)

It now takes 30 seconds for the 'plt part to turn green (indicating a constant) and the 'axvline to turn yellow (indicating a function). This suggests the slowness is in whatever bit of the product responsible for the fancy formatting. The good news is that this "broken" functionality now prevents the system from stalling as I type: I can type full speed and let the system eventually catch up, which is an improvement over the behavior in the original description, so thanks for that fix!
I also notice that, in the "broken" state (CPU at 100%), I get a message in the status bar saying "28 files to analyze". This appears multiple times as I type. For example, type:
np.as

The popup with asarray, etc. no longer appears (as mentioned above). Instead, on each character 'a', 'r', 'r', etc., the status message "28 files to analyze" appears. This is odd as the 'np.` refers to NumPy which isn't one of my files, so it is not clear why the system wants to analyze 28 files (which, I assume, are my various Python files). And, it is not clear why the files need to be analyzed on every keystroke since none of those files changed from one keystroke to the next.
Sometimes, as I play with this use-case, the auto-complete does appear, but at this point it is a crapshoot.
As before, the workaround is to exit VS Code and restart, after which things are back to normal (for a while at least.)
Latest version info:
Version: 1.88.1
Commit: e170252f762678dec6ca2cc69aba1570769a5d39
Date: 2024-04-10T17:34:12.840Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Linux x64 5.15.0-101-generic

Happened again in the latest version, this time after only a few hours of usage.
Version: 1.89.0
Commit: b58957e67ee1e712cebf466b995adf4c5307b2bd
Date: 2024-05-01T02:08:25.066Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Linux x64 5.15.0-101-generic

Happened yet again. Same scenario. As far as I can tell, the situation seems to occur after some amount of time working in Jupyter, perhaps with larger notebooks, or just after some number of unlucky code completions. The problem seems worst when dealing with Numpy symbols, but then that is mostly what this notebook uses.

Happened again after another few hours of work and about 2 days of up time. This time I get a window saying that the window is not responding and asking if I want to restart. This is a change from the earlier behavior where the system would just become unresponsive, so thanks for this fix.
The problem now occurs when typing strings. In a notebook, using Python, in a code cell:
plt.axhline(0, color='gray', linestyle='|')

Cursor is at the "|" caret. About to type "--". However, the UI pauses for 5-10 seconds, then pops up a selection with things like "ymax=" and a list of files. When I type the first "-", there is another few second pause as the system reduces the list to just the list of files that contain a dash.
As noted in the original post, this is not as helpful as one might think: I almost never enter file names, or argument names, into the string I'm typing.
I also observe that some part of the intellisense seems to have broken. Start typing the following in the same code cell:
plt.

Prior to the CPU hitting 100% in a code process, I would get a popup of symbol names, which is actually helpful here. After fiddling around to create this report, auto-complete has stopped working. It also now takes many seconds for the colors to change, for the red squiggly line to disappear, etc.
Finish typing the line above:
plt.axvline(0.5)

It now takes 30 seconds for the 'plt part to turn green (indicating a constant) and the 'axvline to turn yellow (indicating a function). This suggests the slowness is in whatever bit of the product responsible for the fancy formatting. The good news is that this "broken" functionality now prevents the system from stalling as I type: I can type full speed and let the system eventually catch up, which is an improvement over the behavior in the original description, so thanks for that fix!
I also notice that, in the "broken" state (CPU at 100%), I get a message in the status bar saying "28 files to analyze". This appears multiple times as I type. For example, type:
np.as

The popup with asarray, etc. no longer appears (as mentioned above). Instead, on each character 'a', 'r', 'r', etc., the status message "28 files to analyze" appears. This is odd as the 'np.` refers to NumPy which isn't one of my files, so it is not clear why the system wants to analyze 28 files (which, I assume, are my various Python files). And, it is not clear why the files need to be analyzed on every keystroke since none of those files changed from one keystroke to the next.
Sometimes, as I play with this use-case, the auto-complete does appear, but at this point it is a crapshoot.
As before, the workaround is to exit VS Code and restart, after which things are back to normal (for a while at least.)
Latest version info:
Version: 1.88.1
Commit: e170252f762678dec6ca2cc69aba1570769a5d39
Date: 2024-04-10T17:34:12.840Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Linux x64 5.15.0-101-generic

Sorry for the delay here, thanks for the command line it indicates to be the extension host. Can you follow the steps here https://github.com/microsoft/vscode/wiki/Performance-Issues#profile-the-running-extensions to capture the CPU profile when you see this issue again. Thanks!

Thanks for the link. I'll report the results as requested. The problem occurs intermittently, so it might take a few days for it to occur again.

It seems that this problem may have been resolved in a recent update: I've not seen the issue in some time. Closing for now to avoid clutter. I'll reopen it if the issue reappears.


Typo in function for the workflow item

https://github.com/equinor/fmu-dataio/blob/081f5860fc9c94974986665a28e57c233767bcab/src/fmu/dataio/dataio.py#L246
*reference

Included in #56


Update setup.py

fix typo in org-name path-component of setup url arg
This pull request fixes: no issue created for tiny typo.

[ ] I've added tests to cover the proposed changes

No, but hoping I can get a pass for what is almost  only a documentation typo.
Changes proposed in this pull request:
-Make link to the repo from PyPI work! :)
Thanks for sharing your code!!

@akariv just to say i've reviewed and this is good - just a typo fix.

This has since been fixed in a74e1cd, closing PR. Thanks for the review anyway though @rufuspollock, I'm honored even if it was only for a typo fix! 😄


Error prevented the integration from working at all until it was reloaded.

I noticed today that none of the Subaru functionality was working properly. All the entities related to it said unavailable.
I was only able to fix it by reloading the integration manually.
I checked the logs, and the following error was reported:
Logger: homeassistant.config_entries
Source: config_entries.py:594
First occurred: June 10, 2024 at 10:29:22 PM (1 occurrences)
Last logged: June 10, 2024 at 10:29:22 PM

Error setting up entry<EMAIL_ADDRESS>for subaru
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/subaru/__init__.py", line 102, in async_setup_entry
    vehicles[vin] = await _get_vehicle_info(controller, vin)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/subaru/__init__.py", line 210, in _get_vehicle_info
    VEHICLE_HAS_LOCK_STATUS: await controller.has_lock_status(vin),
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 334, in has_lock_status
    await self.get_data(vin)
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 448, in get_data
    await self.fetch(vin)
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 608, in fetch
    result = await self._fetch_status(vin)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 1120, in _fetch_status
    raise err
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 1113, in _fetch_status
    await self._locate(vin)
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 1155, in _locate
    js_resp = await self._remote_query(vin, api.API_LOCATE)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/subarulink/controller.py", line 1022, in _remote_query
    raise SubaruException("Remote query failed. Response: %s " % js_resp)
subarulink.exceptions.SubaruException

How did you reload it manually?  Download the files again or run the configuration again?

@tweas you can reload the integration through the UI:
Settings -> Devices & services -> Integrations tab -> Subaru (HACS)
Once you navigate to that page, there should be a small ... option on the right next to 'CONFIGURE' under "Integration entries".
If you click the ... there should be a Reload option at the top of the resulting popup menu. Hope that helps!

Yes, aware of the how's.  I was just asking which option you meant when you said reload.  Not everyone uses the actual terminolgy from the app when commenting.  So, wanted to confirm if "reloading" was the actual "Reload" option.  OR, did you have to download and effectively reinstall the custom configuration in order to resolve.
In any case, the issue I was having was that mysubaru itself was down when I ran the latest core update making me think the update had broken the integration.


[macOS] Fix curl error "argument list too long"

See https://github.com/VBA-tools/VBA-Web/issues/396 for details.
The requestbody.txt file is being saved to /Users/MYSUSER/Library/Containers/com.microsoft.Excel/Data and will be removed once the curl command has been executed.

Surely the file should be placed in a temporary directory instead? What if the local directory is read-only?

Thanks @HLFH!

Surely the file should be placed in a temporary directory instead? What if the local directory is read-only?

@mohawk2 /Users/MYSUSER/Library/Containers/com.microsoft.Excel/Data is indeed one of the few writable directories with modern masOS Excel (since 2016/365): since Excel is sandboxed, external directories (such as /tmp) would require approval first.


[FLINK-32712][format/json] enhance flink ogg-json format

What is the purpose of the change
The BEFORE field in ogg-json can be configured to contain no table fields, as in the ogg-json example below.
{
  "table": "ZBZZZ",
  "op_type": "U",
  "op_ts": "2023-07-20 21:45:34.860817",
  "current_ts": "2023-07-21T05:45:36.615000",
  "pos": "00002564940142073691",
  "before": {},
  "after": {
      "ID": 1461242,
      "PROPERTY_01": "tc",
      "PROPERTY_02": null,
      "PROPERTY_03": null,
      "PROPERTY_04": "K",
      "PROPERTY_05": "5",
      "PROPERTY_06": null,
      "PROPERTY_07": null,
      "PROPERTY_08": null,
      "PROPERTY_09": null,
      "PROPERTY_10": null
  }
}

For this case, ogg-json format should not send a Record of type UPDATE_BEFORE downstream。 Worse, it can sometimes cause the flink job to crash out.
Brief change log

add logic for handling "before":{} case.
add the test data for this case.

Verifying this change
This change is already covered by existing tests, such as OggJsonSerDeSchemaTest.
so I added the test data for this case.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no) no
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no) no
The serializers: (yes / no / don't know) no
The runtime per-record code paths (performance sensitive): (yes / no / don't know) no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know) no
The S3 file system connector: (yes / no / don't know) no

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure


ABC view in machine folk session

Should not strip out S: field, or C: field.

It deliberately only passes the information fields that make sense to render as part of the score. Currently, that's set to M, L, Q and K.
https://github.com/tobyspark/folk-rnn-webapp/blob/5cd5c70131ec55c16de884ffed4b0794ec9366cf/folk_rnn_site/folk_rnn_site/models.py#L187
It definitely shouldn't pass S. This is effectively a tune attribution, which should be displayed with the text of the rest of the page. And rendering a long text field as part of the score just looks super screwy. Currently the S text only appears in the full ABC seen lower in the page, in 'create setting', and on download of the ABC. It would perhaps make sense to either render this as one more tune attribution paragraph on the left, or actually create a tune attribution from the S field on submission.
Perhaps it should pass C. Generally, I think metadata text should be rendered with the other metadata on the page. So pulling out any C: field and putting it there would be my vote. But if we were to have it render as part of the score, at least it's a text field that isn't going to e.g. go off the page.

True. What I'm worried about is lost attributions... which I guess is a
problem with real tunes as well! As long as it's in the downloaded ABC,
shouldn't be a problem.
On 11/07/2018 10:51, Toby Harris wrote:

It deliberately only passes the information fields that make sense to
render as part of the score. Currently, that's set to M, L, Q and K.
https://github.com/tobyspark/folk-rnn-webapp/blob/5cd5c70131ec55c16de884ffed4b0794ec9366cf/folk_rnn_site/folk_rnn_site/models.py#L187
It definitely shouldn't pass S. This is effectively a tune
attribution, which should be displayed with the text of the rest of
the page. And rendering a long text field as part of the score just
looks super screwy. Currently the S text only appears in the full ABC
seen lower in the page, in 'create setting', and on download of the
ABC. It would perhaps make sense to either render this as one more
tune attribution paragraph on the left, or actually create a tune
attribution from the S field on submission.
Perhaps it should pass C. Generally, I think metadata text should be
rendered with the other metadata on the page. So pulling out any C:
field and putting it there would be my vote. But if we were to have it
render as part of the score, at least it's a text field that isn't
going to e.g. go off the page.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tobyspark/folk-rnn-webapp/issues/80#issuecomment-404094349,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJy6ZK6QbyP1S73Ojp-z-GOf4JpI-vZ-ks5uFbx9gaJpZM4VKoQJ.

--
Cheers.
-Bob L. Sturm

Bob L. Sturm, Lecturer of Digital Media
School of Electronic Engineering and Computer Science
Centre for Digital Music
Eng. 111
Queen Mary University of London
Mile End Road London E1 4NS
United Kingdom
http://www.eecs.qmul.ac.uk/~sturm/
https://highnoongmt.wordpress.com
https://soundcloud.com/sturmen-1
https://www.youtube.com/channel/UC7wzmG64y2IbTUeWji_qKhA

Yeah we don't lose anything. But you have highlighted an arbitrary division between the ABC info fields and machine folk's TuneAttributions aka notes. I'll make an issue to join them up.


Add mc-sgx-panic crate for common panic handling

Current dependencies on/for this PR:

main

PR #14

PR #13

PR #15

PR #22

PR #24

PR #30   👈

PR #31

PR #21

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #14

PR #13

PR #15

PR #22

PR #24

PR #30   👈

PR #31

PR #21

This comment was auto-generated by Graphite.

Codecov Report

Merging #30 (b4249a1) into panic-log (dfb54d8) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##           panic-log      #30      +/-   ##
=============================================
+ Coverage      87.80%   87.90%   +0.09%
=============================================
  Files              4        5       +1
  Lines            123      124       +1
=============================================
+ Hits             108      109       +1
  Misses            15       15

Impacted Files
Coverage Δ

panic/src/lib.rs
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


evaluation_strategy -> eval_strategy

evaluation_strategy is deprecated

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Related: https://github.com/huggingface/trl/pull/1559


Feat/ Update node version used by project to match Electron

Description
In this PR the node version advertised as used by the project is bumped to match what is packaged with previously updated electron. Also, httpsnippet  is updated, as it claims node 20 compatibiliy since version v3.0.4.
fixes: #2710
relates to: #2668

@lohxt1 @sanjai0py could you take a look? It seems this should have been done as part of #2668.


 is not working anymore after upgrading to react-router-native v5

I'm trying to update to react-router-native to v5.1.2. I'm getting this error TypeError: Cannot read property 'Consumer' of undefined when I use <AndroidBackButton />.

You have a different version of react-router in your project somewhere. You can use npm ls react-router to find it and upgrade it to the same version as react-router-native.


[CI] Upload Test Log for CI failure shall contain trino engine log #2094

Why are the changes needed?

Upload Test Log for CI failure shall contain trino engine log.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

thanks, merged to master


Logout

In gitlab by @CalumTowers on Aug 1, 2016, 17:47
When logging out of the forum, the user is redirected to here: https://core.vatsim-uk.co.uk/mship/auth/logout?returnURL=http://community.vatsim-uk.co.uk&ssoKey=Community Forums.
The user is asked whether they want to terminate their session and log out of Core but neither of the buttons on this page do anything.

In gitlab by @CalumTowers on Aug 4, 2016, 19:28
Changed title: {-Forum -}Logout → Logout

In gitlab by @CalumTowers on Aug 4, 2016, 19:28
It is worth noting that the same issues are present with the RTS

In gitlab by @NFarrington on Aug 4, 2016, 20:27
mentioned in commit 3f33aa70bf8d1b6808ba279579bf8b620c4ff406


Add field to define color selection

Add a field which defines what selects color, or make a selector within the formatting settings which defines what selects color.

This is also fixed with the code but I need to merge it in.


Reproducing on Tanks and Temples dataset

Hi,
Thanks for sharing the code!
I'm trying to reproduce the reported numbers on the Tanks and Temples dataset. Is it possible for you to share the configs for the tanks and temples dataset or even the pre-trained models? That would be very helpful!
Thanks in advance!

Hi,
Thank you for your interest in our work! I'm currently working on cleaning up the code for NeRF-Synthetic and Tanks&Temples datasets. Hopefully I will update the code before the ICLR conference.

Thanks for the reply and effort!

Hi, any update on this? Thanks!

I just updated the instructions for running our pretrained model on the TNT dataset.

Great, thanks!


errors happen when building on windows with vs2019

I build ngransac following the direction of https://docs.microsoft.com/en-us/visualstudio/python/working-with-c-cpp-python-in-visual-studio?view=vs-2019, all the libs and dlls are correctly setting and the building are successful. No import error happens in importing he built module of ngransac, but when calling ngransac.find_essential_mat an error arises: find_essential_mat(): incompatible function arguments. The following argument types are supported: 1. (arg0: at::Tensor, arg1: at::Tensor, arg2: int, arg3: int, arg4: float, arg5: at::Tensor, arg6: at::Tensor, arg7: at::Tensor) -> float Invoked with: ...
Someone says  the header file <torch/extension.h> should be added in the first of .cpp file, but link error happens: unresolved THPVariableClass.
I am trapped in it and cannot struggle out, please help.

solved, thanks all the same.


ImportError:~/DCNv2/_ext/dcn_v2/_dcn_v2.so: undefined symbol: THCudaTensor__nDimension

After I have runned '. mask.sh', and then 'python test.py', This error produced.Why?

I also meet the  error when I run ‘test.py’, in line 113
https://github.com/CharlesShang/DCNv2/blob/99a8293352df1840cc7c7f1ab8945286a7a8d57f/test.py#L113
the error is
TypeError: initializer for ctype 'struct THCudaTensor *' must be a pointer to same type, not cdata 'struct THCudaDoubleTensor *'

@zccoder
This is built on torch 0.4.1
run
import torch
torch.__version__
# '0.4.1'

to check

@LZP4GitHub
you can swith between double and float in dcn_v2_func.py manually
from _ext import dcn_v2 as _backend
# from _ext import dcn_v2_double as _backend

Double precision is only for gradient check, it's not recommended because it's poorly optimized by Nvidia.

I've met the same issue. I found there are typos in src/dcn_v2_cuda.c.
To change THCudaTensor__nDimension to THCudaTensor_nDimension may resolve this.

@everjesse Thanks for reporting this. That's weird, I'll check whether both function names exist.

THCudaTensor_nDimension corresponds to dim(), THCudaTensor__nDimension to _dim().
dim() is the new ATen API that supports zero dimension, thus it’s recommended.
I’ll change it later.  @everjesse

Thanks for fixing this!
Your update with PyTorch 1.0 is pretty cool, and it works well! @CharlesShang

Good to know it, thanks! @everjesse


Support void/Task-returning mutations if using mutation conventions and AddQueryFieldToMutationPayloads

Product
Hot Chocolate
Is your feature request related to a problem?
In the following, assume mutation conventions and AddQueryFieldToMutationPayloads are enabled, i.e., all mutation payloads have at least a query field.
Consider this mutation:
public class Mutation
{
    public void Foo()
    {
    }
}

This mutation is not picked up by HotChocolate, presumably because it returns void. It's the same if it returns a non-generic Task:
public class Mutation
{
    public Task Foo() => Task.FromResult(1);
}

However, I can create an empty FooPayload class and return that:
public class FooPayload;

public class Mutation
{
    public FooPayload Foo() => new();
}

This now works, and the query field is automatically added to the otherwise empty FooPayload.
The solution you'd like
My desire is that if I use query conventions and AddQueryFieldToMutationPayloads, then my mutations can return void or a non-generic Task, and the effect is the same as if I had defined and returned an empty payload class for that mutation, as demonstrated above.

I think that is not a good pattern :) A mutation should return the affected object. If there is no effected object on your graph return success. We will not add support for void as it would lead to bad practices.
I will discuss it in the team however and report back.

Thanks for your insights! I agree that the affected object should be returned as a general rule. However, say for example that you delete an object, or otherwise that there is no affected object (that still exists, at least). What should be returned then? What do you mean "return success"? Do you mean returning success: true? If so, I'd argue that is pointless and just duplicates information (the presence/absence of errors).

There could also be other operations where normal APIs would return 204 No Content or 202 Accepted (instead of, say, 200 OK or 201 Created).

Closing this issue as we do not plan to implement this

Thanks for considering it! I accept your decision, though I would be very grateful for a short explanation of why, in light of my questions above.


Using TunneLS to make client TLS enabeld

Hi,
More a question than an issue:
Is it possible to use TunneLS in a way that it unwraps TLS for a non TLS-enabled client?
I have a TLS-secured REST-API in a docker environment. One API-Consumer has no TLS-Support.
Is it Possible to place TunneLS between this client and the API?
I tried this, but it seems it does not work?
Do I have to use client mode?
my config:
{
        "Nodes": [
                {
                        "Name": "tls-wrapper",
                        "Mode": "client",
                        "Accept": "<IP_ADDRESS>:5002",
                        "Connect": "<IP_ADDRESS>:3376",
                        "Cert": "/etc/docker/cert.pem",
                        "Key": "/etc/docker/key.pem",
                        "CA": "/etc/docker/cert.pem",
                        "InsecureSkipVerify": true
                }
        ],
        "StdErrLogging": true,
        "StdErrPrefix": true
}

Some help would be highly appreciated...

What do you mean it doesn't work?

Oh, you needed to use server mode.


Turkish translations added to homepage and messages

Thanks @muratbsts 😀

Hi, @MadLittleMods you're welcome. How to change the language of gitter.im website for an overview?

@muratbsts This won't be available until we update the dependency in the webapp and push out a release.
I assume it picks up your locale from your OS/browser preferences.

Thanks @MadLittleMods I understand.


Usage questions

Hey Simon, great repository!!
Totally want to try that with my heat pump.
https://github.com/simonlmn/can-wifi-gateway-stiebel-eltron/blob/e0e3c47e6d6c3a481ba210b32dd40fb486632b08/README.md?plain=1#L106
Before I go full in with that solution I want to understand of how one can include it in e.g. home assistant.
Could you please provide sample-data of the MQTT topic or your specific setup to include your stiebel eltron device?
How could I use that translate-file from can2mqtt in with your project?
https://github.com/Hunv/can2mqtt/blob/a9b2cb4717d23e6bac56b9b4344bb17feb73fceb/can2mqtt_core/can2mqtt_core/translator/stiebel_eltron/StiebelEltron.json
Thanks in advance!
Dan

Hi Dan, thank you for you interest in the project!
My setup is based on OpenHAB and uses the standard HTTP binding to actually poll the gateway every 30 seconds to get new data. It is not very sophisticated but seems to work pretty good. It is running for about 2 years now with the occasional hiccup solved by resetting the gateway (but has been pretty stable for months now without any interruption).
I had added basic MQTT support on request by someone else, but I am not using it myself, so keep that in mind. It does work and I just made a small change where each datapoint/parameter is published to a separate sub-topic. You probably have to read or potentially change the MQTTClient.h to see what it does or adapt it to your needs.
Regarding the can2mqtt translate-file: The gateway currently uses are compile-time "definitions" table (see ValueDefinitions.h) which it supports and knows how to interpret. I always wanted to switch that for a run-time configuration similar to what you linked, but since what I have now works for me, it did not happen yet. Looking at that file, I think my value definitions should be mostly the same, but maybe some things are missing or different (depends on your heatpump model and what you want to read/write).
So maybe have a look into that if it is sufficient. Of course, you can also try changing the implementation, but I know that this comes with some initial friction (e.g. the current build setup is not that great and I also want to reduce the nesting of projects using sym-links).

Quick update: as I wrote in the other issue #5, I am working on supporting "runtime-configurable" definition tables to make this project really usable on any heatpump/device which speaks the same basic CAN protocol. May take a moment, but maybe this is also useful for you.


purdue Add files via upload

adding Purdue Global University

@mhogue44 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Use self.ArgParameter for init argument list parameters

Not the perfect solution, but cleaner than cast(RangeExpr, initParam). This is the current workaround for #98.
Also removes the implicit promotion from float to Range, which while convenient, really probably isn't safe. Still allows range-float operations, eg range minus float.
Also a few additional cleanups in the standard library.

The tol parameter is necessary for a float -> range conversion. 1 * Volt (I think) still produces a float-like, the difference now is that there's no implicit promotion (in the static typer, at least) from float to range.

The underlying reason (other than 'it feels magical and maybe a bit dangerous') is that the float-like and range-like types overlap, so this change of removing implicit promotions removes that overlap.


Horizon API: "transactions" function should have a parameter to skip XDR content

What problem does your feature solve?
Some transactions (claimable balance....) have HUGE XDR load. Like 10Mb per transaction. If you do not need XDR info it would be good to provide parameter to skip this data in return. We have seen "transactions" responses that were 800MB for a single wallet due to complete overload of XDR fields.
What would you like to see?
Something as "xdr=true(default)/false" that would not return value of result_xdr and result_meta_xdr
What alternatives are there?

@alien2108 are you running your own Horizon instance or using SDF's? If the former, there is a configuration value that removes XDR output from all Horizon responses (SKIP_TXMETA=true). There is no ability to set this on a per-request basis. If you are using SDF's instance, then txmeta will be removed from all API responses starting at Protocol 22.
If you'd still like to request this feature (per-request txmeta removal), can you please refile it over in the Horizon codebase? That is over here: https://github.com/stellar/go


Tab Broken Down into smaller Elements

Enable the designer to decide about the amount of tabs and allow text style definition, e.g. all caps. Compare to XD too. See survey results https://docs.google.com/presentation/d/1ekou3J2SY6XXj1asQVgnNJT4UXDhz6hAR_bmSnPmEbc/edit#slide=id.g18d0eacc774_0_60

Related to https://github.com/trimble-oss/modus/issues/480


Refactor volunteer view of event form

Issue #100 : The UI view for users who are volunteers still has input boxes even though they can not edit the contents. This is an enhancement of the view.
The following is how it looks like before.

The following show the view looks like after.

Test:

Login in as a student and navigate to /1/1/edit_event
The view should be the same as the second uploaded picture.
Login as a Celts students staff or Celts admin and navigate to /1/1/edit_event
The following view should be displayed.

Do we want the facilitators field to be required?  I forgot what we decided on that.  If so, the selectpicker for that field needs a required tag.  If not, then the facilitators field should not show up on the volunteer view when there are no facilitators to display.


Rename all instances of fisherman to watcher

Objective
Rename all instances of fisherman to watcher
Origin Document
https://github.com/pokt-network/pocket/issues/956
Goals

Make sure that Pocket networks specs are up to date with the new terminology of 'Watcher'

Deliverable

[x] Rename all instances of fisherman

General issue deliverables

[ ] Update the appropriate CHANGELOG(s)
[x] Update any relevant local/global README(s)
[ ] Update relevant source code tree explanations
[ ] Add or update any relevant or supporting mermaid diagrams

Creator: @0xRampey
Co-Owners:
Also probably cue to merge in ADR https://github.com/pokt-network/pocket-network-protocol/pull/38 @dylanlott

@0xRampey Makes sure to update the metadata of PRs as well as issues. Going to update this one so PTAL.

@Olshansk I've updated the gameplan in the origin document and linked to it here. Lmk if that clears it up.

@Olshansk we still need this ticket I think, right?

@Olshansk we still need this ticket I think, right?

Yea, that's correct. And one for renaming Servicer to Supplier as well.


Azure SDK Review - [IC3 ACS Calling]

New SDK Review meeting has been requested.
Service Name: IC3 ACS Calling
Review Created By: Bob Gao
Review Date: 12/14/2023 06:05 AM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET, iOS, Android,
Description: ACS calling Dec review
Detailed meeting information and documents provided can be accessed here

Calling SDK
Android

2.7.0

https://apiview.dev/Assemblies/Review/95c984e7d94a4961bd724d0d07394cf9?revisionId=35a0683839ce4e5e9ba32acecb9bca25&diffOnly=True&doc=False&diffRevisionId=03424c177c804986a8ebcc09896a36f8

iOS

2.9.0

https://apiview.dev/Assemblies/Review/80768c5191c748a69f83e4e2d42bd97d?revisionId=174b2969ca8e4d5da1670ca170a62c3f&diffOnly=True&doc=False&diffRevisionId=0f3982c1a91c42df9fb8505c29cd232a

.Net

1.5.0

https://apiview.dev/Assemblies/Review/db8d6cbd1678473da111b1967350b837?revisionId=b3d837bd2162473fbc9b5a47fbda70f9&diffOnly=True&doc=False&diffRevisionId=e906e51d520542739b77fedc076e9fa8


updated LastReadAt value immediately when a document retrieved

updated LastReadAt value immediately when a document retrieved
Issue: https://github.com/aldarisbm/memory/issues/12

We should make sure that the object that is saved to the databased is updated too

we should probably add something like this to the data sourcer interface
	// updateLastReadAt updates the last read at timestamp for the given document
	updateLastReadAt(document *types.Document) error

https://github.com/aldarisbm/memory/blob/d785ee6a65e2a29d84e5283fa2d5876b7ae88e11/datasource/local_store.go#L17

Hi @aldarisbm
I have worked on your review comments, please review.

will be looking at this shortly, I was away for a couple of weeks, and my computer died L(

@aldarisbm
please review .


Blog post before and after event

Reassigning based on PMRs request during meeting and also the fact I ended up not going!

Blogged it


Discussion: [Title Bar Consistency]

** Discussion: [Title Bar Consistency]**
I have been looking at various title bar sizes and each one of them are not consistent in either: title bar size, fon't size, vertical alignment of the text as well as the vertical alignment of the minimize, maximize, or close buttons. I think some work needs to be done to keep these things consistent here is a photo of three four different applications. Notepad - a Win32 app, Xbox (beta) An Electron app, Calculator which is a UWP app, Edge Chromium, and Grove Music.
@bitcrazed spoke on the issues an challenges involved with this process,

@mdtauk Depending on what UI framework an app is built, you may see system buttons of slightly different sizes and shapes. For example:
Win32 apps like Notepad rely on the OS (GDI/DComp) to draw system controls and so get a standard sized title bar
Chrome / Edgeium draws its own non-client area in order to get its tabs up into the title bar, and draws its own system buttons ... aligned to the top edge of its Window for ... 'reasons'
Terminal also draws its own non-client area to get its tabs up into the title bar region, and in order to make the tabs fit, we have a taller title/tab bar.
In Terminal, we chose to scale & center the system buttons in the available title bar space, which results in them being a little bigger, but still square and in proportion:

I think going forward, these differences need to be eliminated by using the same control or the appearance of the same control to create the exact same result.

Related Links
https://github.com/microsoft/terminal/issues/1375#issuecomment-526275909
https://github.com/microsoft/terminal/issues/1375#issuecomment-526284633

I wouldn't say that any difference "needs" to be "eliminated", in your screenshot on the left the caption buttons were intentionally centered. At the end of the day the caption buttons are also an item of design for the app developer.
Imo, UWP has already made it easy to customize the title bar with ExtendViewIntoTitleBar that retains the caption buttons, whereas doing the same thing in win32 is exceedingly hard. So I'm not sure what this issue is about. Is this a request to standardize caption buttons across inbox Windows apps?

There are a number of different Frameworks being used, and each of those does handle its own Titlebar drawing.
Win32 frameworks, like MFC or WinForms - will most likely not make any changes, so as not to introduce compatibility problems with classic apps running on Windows 10/Core/Modern OS.
Apps that use the Ribbon control, also override the window controls because the ribbon interacts with that region.
WPF has a Window object, and the community could opt to make any changes to their TitleBars, but many apps opt out of the standard window controls, and "roll their own".
UWP apps can extend into the TitleBar, and re-colour the window controls - the sizing of which is designed for High DPI and the 4px grid.
Then there is Edge, Electron Apps, which all do their own thing.
WinUI Desktop may be able to use the new Windowing APIs, a XAML Window object, and updated shell Interops - to provide easier ways for users of these frameworks, to update their TitleBars to use the default control sizes, glyphs, but with the ability to customise or extend into the TitleBar area, so these customised versions can be replaced, and consistency brought in over time

I wouldn't say that any difference "needs" to be "eliminated", in your screenshot on the left the caption buttons were intentionally centered. At the end of the day the caption buttons are also an item of design for the app developer.
Imo, UWP has already made it easy to customize the title bar with ExtendViewIntoTitleBar that retains the caption buttons, whereas doing the same thing in win32 is exceedingly hard. So I'm not sure what this issue is about. Is this a request to standardize caption buttons across inbox Windows apps?

Well, that is part of the reason I am bringing it up. All of these are Microsoft apps. Each one of these are inconsistent in some way or form with the list in the original post. It wasn't just a case of the one all the way to left being centered and others aren't. Each one is different.
We know that a Win32 programs won't be updated, but at the same time a program like Notepad was put in the Microsoft Store, so things might change for that particular app.
At the same time, as posted in the related links, a new program like Terminal is having issues with having a consistent experience.
Of course third parties are going to do what ever they want, but if everything is consistent, they really shouldn't have to unless they want to.
This all about consistency,  which is why I bought it up.

This is a discussion topic @adrientetar  Not strictly a Feature Request.  But I would be happy to see formalised Window objects and APIs for WinUI Desktop apps (be they XAML, Win32, or MFC) frameworks, which make it easy and simple to customise the TitleBar in a consistent and robust way.

I think the most consistent way to make app titlebars consistent is to allow easy creation of the Windows Runtime's CoreWindow from non-UWP project types.

There is design guidance, but the WinAppSDK Titlebar does not yet contain all the properties to make these consistent with what is suggested.

Specific call outs:

Tall mode for the Window Caption Buttons
App Icon toggle, along with Release/Preview subtext
There are no windowing events for styling the TitleBar control when Inactive.


Error when Running SampleApp

When I run the sample app using coreclr 1.0.0-rc2-16357 I get the following error:
KestrelHttpServer/src/Microsoft.AspNet.Server.Kestrel/Http/Frame.FeatureCollection.cs(23,34): .NETPlatform,Version=v5.4 error CS0535: 'Frame' does not implement interface member 'IHttpConnectionFeature.IsLocal'

New bits were just pushed to the dev feed so this is fixed.


Turn by turn API call service

This issue states that only Graphhopper is available currently for turn by turn api calls #166 . Correct me if I am wrong. Is it possible to use another service like OSRM for turn by turn API calls?

Currently, the application is only set up to use Graphhopper for creating snapped-to-street geometries for trip patterns. It likely is possible to use some other service, but we have not configured the application for alternative services and I think we're somewhat wary of introducing alternatives because of any increased complexity in the already complicated pattern shape manipulation/creation.

@landonreed Is Graphhopper open-source (hosted on own servers) also supported, or only the hosted version of Graphhopper (paid plans?)

Graphhopper open-source is not currently supported, but I think this would be a relatively simple fix. I think we would just need to make the URL in this request come from a config variable from env.yml (similar to GRAH_HOPPER_KEY).

@landonreed Hi, we would be ready to submit PR for this (done already by @a1ex4nd3r ) but we noticed this is linked to another issue #385 - and this indirectly blocks the full functionality - once gtfs without shapes is imported, shapes are then generated using open-source Graphhopper or indeed commercial Graphhopper and then such gtfs with shapes is exported, the zip file doesn't include the shapes.txt file at all). Is there anyone working on it ?

@wkulesza, I had noticed your question was unanswered when going through old issues today. #385 has in fact been closed, but I'm not sure if you still need this functionality. Let me know if this is no longer needed and I will close (otherwise, still happy to accept a PR!).


Build monitor is not creating issues for failed pipeline runs

Build monitor should've opened issues for the multiple failed builds in the helix-machines-daily definition according to the configuration in https://github.com/dotnet/dnceng/blob/58603e03089aae816e1a06065e104a1a6a81cb3f/src/DotNet.Status.Web/DotNet.Status.Web/.config/settings.json#L104-L107
It seems like no issues have been created in a while. We should investigate and see if the service is running correctly.

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

There is an alert for this functionality here: https://github.com/dotnet/dnceng/issues/3786. Unclear if the alert is related, but investigating that would be a good first step.

investigation in #3786  has shown that the webhook was disabled after multiple failures.

investigation in #3786 has shown that the webhook was disabled after multiple failures.
#3786 (comment)

I enabled but it but it got disabled again after some time because of many failures
https://dotneteng-status.azurewebsites.net/api/azp/build-complete returns an error, is it possible that the url is not the right one?

We've been opening issues through this automation. Taking a quick look at the status of the webhooks to see if we can close this.

We have 4833 successful deliveries and 20 failed deliveries in the last 7 days. The failures I've examined resulted in 500 status codes from the status website. Will see if we have any logs on the service side for these failed requests.

The failures I've examined were transient failures trying to access the GitHub APIs that lasted throughout 5 retries. I think we can close this as the functionality is working except when GitHub goes down


bug: file: package protocol not supported

Admittedly, this might be more WAI then a bug, and we can likely work around it, but filing as a bug in case y'all decide it's a necessary feature.
We have local packages which we reference via a file: protocol. When we have them in our pnpm-lock.yaml and execute a bazel sync with<EMAIL_ADDRESS>we get the following error:
ERROR: An error occurred during the fetch of repository 'npm':
   Traceback (most recent call last):
        File "/private/var/tmp/_bazel_vasilios/96c9699731d9b99e83af4f7d36096194/external/aspect_rules_js/npm/private/npm_translate_lock.bzl", line 319, column 33, in _impl
                lockfile = _process_lockfile(rctx)
        File "/private/var/tmp/_bazel_vasilios/96c9699731d9b99e83af4f7d36096194/external/aspect_rules_js/npm/private/npm_translate_lock.bzl", line 145, column 43, in _process_lockfile
                return translate_to_transitive_closure(lockfile, rctx.attr.prod, rctx.attr.dev, rctx.attr.no_optional)
        File "/private/var/tmp/_bazel_vasilios/96c9699731d9b99e83af4f7d36096194/external/aspect_rules_js/npm/private/transitive_closure.bzl", line 86, column 17, in translate_to_transitive_closure
                fail("unsupported package path " + package_path)
Error in fail: unsupported package path file:packages-dist/admin-bunsen

Similar to #166, the error here is in recognition of the listed packages, which in this case begin with file: and have a resolution.directory entry. We can/will prune these from the pnpm-lock.yaml, as we'd prefer local packages to reference each other via local bazel targets. But it is possibly worth an explicit comment that local packages are not allowed, or some feature allowing them to be ignored when processing pnpm-lock.yaml?

I'll think about this one but I think you are correct that we should either prune them or support it


refactor(typing): use Literal type hint for conflict attribute

I've made the type hint for the conflict attribute more precise by using Literal["rej", "inline"] instead of str.
@yajo Is refactor(typing) the correct change type in your opinion? It seemed the most appropriate to me, but I couldn't find a guideline for committing only typing changes.

I relabeled the commit to the build category. Refactor commits appear in the changelog, and I don't think this one deserves such a honor. 😜