msgpack TimeoutErrors

Problems summary
After a while in a session, I start seeing msgpack timeout errors, and was wondering if there was an expectation here.  I am pretty sure it's due to various sources never resolving their completion results.  Is this something source authors need to ensure they handle properly? Or is this an error that ddc should suppress?
Expected
Not to see TimeoutErrors in the message logs
Environment Information

ddc version (SHA1): e392213

OS:
macOS

neovim/Vim :version output:
macVim, 8.2, 1-3376

Provide a minimal init.vim/vimrc with less than 50 lines (Required!)
I am continuing to dig in and see if I can find a specific source of the problem, but figured I would start the dialogue about whether it's a plugin or ddc issue.
Screenshot (if possible)

It seems deno process communication problem than ddc problem ...
But I need the minimal vimrc to reproduce it for me.
We have not ESP skills.
The minimal vimrc and the reproduce instruction is required.
If you don't upload them, I will close the issue unfortunately.

<EMAIL_ADDRESS>The default response timeout = 10000 ms = 10 s.
I think it is long.  Why it does not respond within the time?

It seems ddc-ale problem from your screenshot.  I think the source does not respond.

Sorry, I did not supply a minimal vimrc earlier, I only had a hunch of the problem. Anyways, I think now I've got a pretty clear picture of what is going on, after testing a bunch of stuff today throughout the day.
The minimal vimrc would be:
call plug#begin('~/.vim/bundle')
Plug 'dense-analysis/ale'
Plug 'vim-denops/denops.vim'
Plug 'Shougo/ddc.vim'
Plug 'statiolake/ddc-ale'
call plug#end()
call ddc#custom#patch_global('sources', ['ale'])
call ddc#enable()

However, it's a bit unclear about what the correct fix would be. A summary of the issue is - Ale does not always respond with results, and in cases of rapid typing input, ddc pushes many consecutive requests to ale, but ale only remembers and fires the callback on the most recent one.
The code path starts here:
https://github.com/statiolake/ddc-ale/blob/master/denops/ddc/sources/ale.ts#L35-L41
which calls into here:
https://github.com/statiolake/ddc-ale/blob/master/autoload/ddc/ale.vim#L8-L17
Which in Ale, reaches into here:
https://github.com/dense-analysis/ale/blob/d098124e59ba38b0fcd97b41eec4a6de81bc09ab/autoload/ale/completion.vim#L833
At which point, it overrides a few buffer local variables that hold onto the callback and the request info, BUT only for the last call into GetCompletions (i.e. each subsequent call to GetCompletions overwrites all callback and info data for the previous call):
https://github.com/dense-analysis/ale/blob/d098124e59ba38b0fcd97b41eec4a6de81bc09ab/autoload/ale/completion.vim#L843-L845
https://github.com/dense-analysis/ale/blob/d098124e59ba38b0fcd97b41eec4a6de81bc09ab/autoload/ale/completion.vim#L867-L875
So basically we end up with requests into Ale's completion system, but we ultimately only respond with the latest callback, which leaves a bunch listeners that ultimately time out.
It would probably be a much bigger change to fix Ale to hold onto all requests and have all callbacks fire, but it would also create a lot of additional work for DDC to process requests that might be out of date. I did hack a bit on the ddc-ale plugin to vastly minimize request spam, but it's not ideal since if the number of results are huge, the max number might kick in and hurt your chances of finding what you are looking for:
https://gist.github.com/amadeus/9a438bc725d5892d7eedb754008c9928
From an overall ddc architecture discussion though, it does kinda feel like ddc should be built in a way that the requests from these services may never come back, and at least give plugin devs a way to handle or catch a timeout, instead of just logging to echom.

I will implement that ddc catches TimeoutError.

I think ddc-ale catch the error or SetTimeout() though...

I think ddc-ale can catch the error or SetTimeout() though...

The plugin used to have a setTimeout in place to resolve earlier, but that can cause weird issues if it ends up resolving while you are autocompleting something else, then the results end up getting wiped out or changed

I wonder if DDC should have a way to know the results that have come in are delayed, and no longer relevant, and can just throw them away

I have skipped the timeout.  And I have added the check for it.

Unfortunately it looks like there are still some of these TimeoutErrors triggering through another system. The minimal vimrc posted above I think is still applicable:

However, I think it might have to do with another promise timing out somewhere, not applicable to the fetching stuff we've been discussing here?

Really?????  I have added the exception handler.
Why it does not work?
Please investigate why it is occurred.
And I think ddc-ale should use callback like ddc-nvim-lsp instead.
The current implementation seems blocked.

I think ddc cannot work for it.  No magic.

Still digging, will report if I can find anything

ddc-vim-lsp had the same problem and fixed by source implementation fix.
https://github.com/shun/ddc-vim-lsp

Is the idea that you coalesce similar fetches to re-use the previous promise? I have a suspicion that this issues it not related to fetches anymore, since your fix looks good. I think it might be coming from some other await level code...

Is the idea that you coalesce similar fetches to re-use the previous promise?

Please explain it more clearly.

I think to catch TimeoutErrors is useless.  I should remove the catch.
It should be fixed in source implementation side.

Please check the catch is called or not.
https://github.com/Shougo/ddc.vim/blob/main/denops/ddc/app.ts#L174

I should remove the catch.

I still think it's better user experience to catch timeout error and not emit them (unless perhaps the user enables a debug mode or something). The inherent nature of these async systems is that they may never return, and punishing the user for them seems not great? Turning on debug mode or setting a global variable however, can be very useful to have them emit to catch problems.
As an aside, I am still trying to track down the source of these timeout errors, I do not believe they are coming from gatherResults, but from some other await usage.  They don't seem to happen right away like before, so I think your fixes are still good for the initial but I found here.  Right now, I am just wrapping a variety of different await cases and adding some catches with logs to see if I can find the offending ones (but it seems to take a long time to get the errors, and I haven't figured out a direct way to repro, so it's going slow).
It would be nice if these errors had a bit more descriptive a trace that would let us know where they are emitting from, is there perhaps something on the denops side I could do to make this easier to debug?

I still don't know the cause yet, but I've uncovered a bit more info (not sure if it's helpful or not).
I modified the code here:
<EMAIL_ADDRESS>To include more date when it throws, like so:
  async dispatch(name: string, fn: string, args: unknown[]): Promise<unknown> {
    try {
      const plugin = this.#plugins.get(name);
      if (!plugin) {
        throw new Error(`No plugin '${name}' is registered`);
      }
      return await plugin.session.call(fn, ...args);
    } catch (e) {
      // NOTE:
      // Vim/Neovim does not handle JavaScript Error instance thus use string instead
      const logger: string[] = [];
      args.forEach((item) => {
        if (typeof item === "string") {
          logger.push(item);
        } else if (typeof item === 'number') {
          logger.push(`${item}`);
        } else if (typeof item === "object" || Array.isArray(item)) {
          logger.push(JSON.stringify(item));
        } else  {
          logger.push('unknown item')
        }
      });
      throw `${name}:${fn} -- ${logger.join(':')} --${e.stack ?? e.toString()}`;
    }
  }

And the timeout errors I am getting are:
[denops] ddc:onEvent -- TextChangedI --TimeoutError: the process didn't complete in time
[denops]     at https://deno.land/x/msgpack_rpc@v3.1.0/response_waiter.ts:49:25
[denops]     at fire (deno:ext/timers/01_timers.js:477:7)
[denops]     at handleTimerMacrotask (deno:ext/timers/01_timers.js:338:7)

OK.  So please add the catch routine in onEvent and check where the error is occurred.

I have improved TimeoutError catches.
ddc catches all of sources/filters TimeoutError.
Closing.

🙏 yeet! thanks for hearing me out on all this, and thanks for adding those additional timeout safety checks! Muuuuuch appreciated!
Btw, really liking this plugin (despite a few of the growing pains so far, but it's beta after all, to be expected), and really happy you are open to feedback, and excited to see where it leads!
(Given what I am learning, I may have to dig in and make a file autocomplete source since you mentioned it was not something you wanted to take on 🤣)

Btw, really liking this plugin (despite a few of the growing pains so far, but it's beta after all, to be expected), and really happy you are open to feedback, and excited to see where it leads!

You can contribute it!

(Given what I am learning, I may have to dig in and make a file autocomplete source since you mentioned it was not something you wanted to take on rofl)

Yeah, I don't want to implement complex file/path source.
You can create it instead.  I need to focus on plugin core development.

file/path source is not implemented yet by users.
https://github.com/topics/ddc-vim
It means file/path source is very complicated source than others.


autoplot() fails with error indicating call to guides() is broken

On a fresh RStudio session with R 4.3.2, I installed {ggcyto} via Bioconductor and cannot reproduce the first steps of the vignette.
library(ggcyto)
dataDir <- system.file("extdata",package="flowWorkspaceData")
gs <- load_gs(list.files(dataDir, pattern = "gs_bcell_auto",full = TRUE))
data(GvHD)
fs <- GvHD[subset(pData(GvHD), Patient %in% 5 & Visit %in% c(5:6))[["name"]]]
autoplot(fs, x = 'FSC-H')

I get the error:
Error in guides$setup(scales, aesthetics, default = params$guide_default %||%  :
  attempt to apply non-function

The error seems to be from a call to guides_.R in ggplot2, which was updated to v 3.5.0 just 4 days ago.
I first tried downgrading ggplot2 to 3.4.4 (and also 3.4.0) but this didn't solve the problem.
I saw that ggcyto version 1.31.1 on GitHub has a hotfix for something related, so I upgraded ggcyto to this version via devtools; but this gives the same error (with ggplot2 3.5.0). I then tried ggcyto 1.31.1 with ggplot2 3.4.4 and the code above runs OK.
So, there is an incompatibility with ggplot2 3.5.0 which is not fixed by version 1.31.1
Session info for the error
> sessionInfo()
R version 4.3.2 (2023-10-31)
Platform: x86_64-apple-darwin20 (64-bit)
Running under: macOS Sonoma 14.3.1

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/4.3-x86_64/Resources/lib/libRlapack.dylib;  LAPACK version 3.11.0

locale:
[1] en_GB.UTF-8/en_GB.UTF-8/en_GB.UTF-8/C/en_GB.UTF-8/en_GB.UTF-8

time zone: Europe/London
tzcode source: internal

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] ggcyto_1.31.1        flowWorkspace_4.14.2 ncdfFlow_2.48.0      BH_1.84.0-0          flowCore_2.14.1
[6] ggplot2_3.5.0

loaded via a namespace (and not attached):
 [1] gtable_0.3.4        htmlwidgets_1.6.4   devtools_2.4.5      remotes_<IP_ADDRESS>     processx_3.8.3      Biobase_2.62.0
 [7] lattice_0.22-5      callr_3.7.5         ps_1.7.6            vctrs_0.6.5         tools_4.3.2         generics_0.1.3
[13] curl_5.2.0          stats4_4.3.2        tibble_3.2.1        fansi_1.0.6         pkgconfig_2.0.3     data.table_1.15.0
[19] RColorBrewer_1.1-3  desc_1.4.3          S4Vectors_0.40.2    graph_1.80.0        lifecycle_1.0.4     compiler_4.3.2
[25] farver_2.1.1        stringr_1.5.1       munsell_0.5.0       httpuv_1.6.14       usethis_2.2.3       htmltools_0.5.7
[31] urlchecker_1.0.1    hexbin_1.28.3       pillar_1.9.0        later_1.3.2         ellipsis_0.3.2      cachem_1.0.8
[37] sessioninfo_1.2.2   mime_0.12           RProtoBufLib_2.14.0 tidyselect_1.2.0    digest_0.6.34       stringi_1.8.3
[43] dplyr_1.1.4         purrr_1.0.2         labeling_0.4.3      fastmap_1.1.1       grid_4.3.2          colorspace_2.1-0
[49] cli_3.6.2           magrittr_2.0.3      XML_3.99-0.16.1     pkgbuild_1.4.3      utf8_1.2.4          withr_3.0.0
[55] scales_1.3.0        promises_1.2.1      matrixStats_1.2.0   gridExtra_2.3       cytolib_2.14.1      memoise_2.0.1
[61] shiny_1.8.0         miniUI_<IP_ADDRESS>      profvis_0.3.8       rlang_1.1.3         Rcpp_1.0.12         xtable_1.8-4
[67] glue_1.7.0          Rgraphviz_2.46.0    BiocGenerics_0.48.1 pkgload_1.3.4       rstudioapi_0.15.0   R6_2.5.1
[73] plyr_1.8.9          fs_1.6.3            zlibbioc_1.48.0

direct calls to ggcyto::ggcyto with geom_hex or comparable fail as well

unit tests are also failing - I will try to see if I can find out how to fix (although I am no hero when it comes to S3) and if I can will add a PR here. The release notes of ggplot2 3.5.0 give an indication, but mainly from a user-viewpoint and not much from a programmatic viewpoint.

@quantixed things seem a bit more complex than I thought - I hope the package authors/maintainers can come up with a fix, it would cost me more time than I have at the moment - I will (for now) rely on your downgrade suggestion above.

@FMKerckhof no problem. I was planning to take a look and see if I could fix (but am similarly short on time), since you say it's complex, I'm not sure I'll be able to figure it out!

We have a fix on the way, see https://github.com/RGLab/ggcyto/pull/95 to monitor its progress.

thanks @djhammill, the switft fix is greatly appreciated :-) ! Clearly, my understanding of how to initialize ggproto objects was too limited to be of service 😉 I installed the latest github version with latest ggplot2 and it works like a charm.

I also confirm that it's working now. I guess I will close especially since #95
> sessionInfo()
R version 4.3.2 (2023-10-31)
Platform: aarch64-apple-darwin20 (64-bit)
Running under: macOS Sonoma 14.3.1

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/lib/libRlapack.dylib;  LAPACK version 3.11.0

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

time zone: Europe/London
tzcode source: internal

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] ggcyto_1.31.2        flowWorkspace_4.14.0 ncdfFlow_2.48.0      BH_1.84.0-0          flowCore_2.14.1      ggplot2_3.5.0

loaded via a namespace (and not attached):
 [1] gtable_0.3.4        dplyr_1.1.4         compiler_4.3.2      Rcpp_1.0.12         tidyselect_1.2.0    Biobase_2.62.0      cytolib_2.14.1      gridExtra_2.3       Rgraphviz_2.46.0
[10] scales_1.3.0        lattice_0.22-5      plyr_1.8.9          hexbin_1.28.3       R6_2.5.1            RProtoBufLib_2.14.0 labeling_0.4.3      generics_0.1.3      BiocGenerics_0.48.1
[19] graph_1.80.0        XML_3.99-0.16.1     tibble_3.2.1        munsell_0.5.0       pillar_1.9.0        RColorBrewer_1.1-3  rlang_1.1.3         utf8_1.2.4          cli_3.6.2
[28] withr_3.0.0         magrittr_2.0.3      zlibbioc_1.48.0     grid_4.3.2          rstudioapi_0.15.0   lifecycle_1.0.4     S4Vectors_0.40.2    vctrs_0.6.5         glue_1.7.0
[37] data.table_1.15.2   farver_2.1.1        stats4_4.3.2        fansi_1.0.6         colorspace_2.1-0    matrixStats_1.2.0   tools_4.3.2         pkgconfig_2.0.3
`


.col.right-align does not move button to right

I have two divs: .row > .col.[s6,s6] the second one also has .right-align. However the .btn in it does not get aligned to the right.
http://codepen.io/anon/pen/wJwYLw

Shouldn't the blue button be on the right? The docs say:

These classes are for horizontally aligning content. We have .left-align,  .right-align and .center-align

You have to apply the .right class to the button, not the column.
https://codepen.io/anon/pen/evOQZE?editors=1100


Flaky service/valid_source_locations_test/service turned RuntimeError on vm-win-debug-arm64

https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket/8770572234679165089/+/u/test_results/stdout
--- Command "vm" (took 56.000224s):
set DART_CONFIGURATION=DebugARM64 & set DART_SUPPRESS_WER=1 & set DART_CRASHPAD_HANDLER=c:\opt\s\w\ir\cache\builder\sdk\out\DebugARM64\crashpad_handler.exe & out\DebugARM64\dart.exe --sound-null-safety -Dtest_runner.configuration=vm-win-debug-arm64 --ignore-unrecognized-flags --packages=C:\opt\s\w\ir\cache\builder\sdk\.dart_tool\package_config.json C:\opt\s\w\ir\cache\builder\sdk\runtime\observatory\tests\service\valid_source_locations_test.dart

exit code:
255

stdout:
00:00 +0: valid_source_locations_test.dart (VM Service)

** Launching SERVICE_TEST_TESTEE=true out\DebugARM64\dart.exe --disable-dart-dev --write-service-info=file:///c:/opt/s/w/ir/x/t/dart_servicebc5e2257/service_info.json --profiler --sound-null-safety -Dtest_runner.configuration=vm-win-debug-arm64 --ignore-unrecognized-flags --packages=C:\opt\s\w\ir\cache\builder\sdk\.dart_tool\package_config.json --enable-vm-service:0 C:\opt\s\w\ir\cache\builder\sdk\runtime\observatory\tests\service\valid_source_locations_test.dart

>testee>out> The Dart VM service is listening on http://<IP_ADDRESS>:61099/FUDQ5d7Wa7k=/

>testee>out>

** Signaled to run test queries on http://<IP_ADDRESS>:61099/FUDQ5d7Wa7k= (VM Service)

Loading VM...

Done loading VM

Running valid_source_locations_test.dart [1/2]

Paused with PauseBreakpoint

Running valid_source_locations_test.dart [2/2]

>testee>err> ../../runtime/vm/profiler.cc: 1068: error: expected: fp_ != nullptr

>testee>err>

>testee>err> ===== CRASH =====

>testee>err> ExceptionCode=-1073741819, ExceptionFlags=0, ExceptionAddress=00007FF6B8C9AF3C

>testee>err> Aborting reentrant request for stack trace.

00:42 +0 -1: valid_source_locations_test.dart (VM Service) [E]

  NetworkRpcException(Canceling request: WebSocket closed(id: 2540 method: getObject params: {objectId: classes/661/functions/moveNext, count: 100, isolateId: isolates/1391700555245759, _includePrivateMembers: true}))

** Killing script

00:42 +0 -1: Some tests failed.

Consider enabling the flag chain-stack-traces to receive more detailed exceptions.
For example, 'dart test --chain-stack-traces'.

stderr:
Unhandled exception:
Dummy exception to set exit code.

--- Re-run this test:
python3 tools/test.py -n vm-win-debug-arm64 service/valid_source_locations_test/service

If we extended unwinding records setup to win arm64 (https://github.com/dart-lang/sdk/issues/52000) we could have had a better stacktrace for the assert/crash.

Duplicate of #52969


move_and_slide doesn't respect surface velocities

I believe this is a more general problem with all kinematic bodies and not necessarily related to move_and_slide, but it seems it doesn't currently react to surface velocities, like the constant_linear_velocity property of StaticBody3D.
Here's how to reproduce it:

Download this project: character_surface_velocities.zip
Run the main scene
Note how the character falls onto the floor and then proceeds to slide off it
Switch physics engine to Jolt
Run the main scene again
Note how the character only falls onto the floor but doesn't proceed to slide off it

Was this due to a lack of support in Jolt? If so, it's been released in Jolt 4.0 https://github.com/jrouwe/JoltPhysics/releases/tag/v4.0.0

No, surface velocities are already supported by this extension, and were in fact very generously added (or at least expedited) into Jolt in response to them not being supported in this extension at first, as part of #331. That goes for a lot of the features and fixes listed in those release notes.
I've been tracking Jolt's master branch since the start of this project, so its version numbers don't really mean a whole lot as far as this extension is concerned.
The problem with this issue is with regards to how surface velocities affect the movement of kinematic bodies, as opposed to the movement of rigid bodies, which is something that needs to be accommodated for on my side of things.

I am also running into this problem. I have a Character3D which is interacting with a Kinematic StaticBody3D. When I set the constant_linear_velocity and constant_angular_velocity my character can ride on the moving platform, but when I switch to Jolt they stay fixed in global space and fall off.
I see this issue has been reported in #331 , #92.
Is it supportable? I know that #331 references a different technique.

This was fixed in #696, which hasn't been released yet.
I'm aiming to have a new release published within the next day or two.

@mikest Reading your comment a bit closer, I only just now realized what you were trying to do.
You should ideally switch your platform to be an AnimatableBody3D instead, which (despite inheriting from StaticBody3D) is a proper kinematic body. That way you don't have to rely on things like constant_*_velocity for move_and_slide to pick up on the proper velocity of the platform. You can just translate/rotate it as normal and move_and_slide should ride along accordingly.
StaticBody3D is not a kinematic body, but a static body, and you will likely see jank or other weird issues if you try to move one around, especially if it ever interacts with something like RigidBody3D.


After Docker installation FGlab does not see FGMachine

followed installation through Docker to install and run MongoDG, FGLab and FGMachine
When I start the webUI no machine is available. Am I screwing up the URL for the machine? Tried both my_ip:port and port:port neither worked - thanks

I just replicated this, and believe at some point one of the underlying libraries stopped translating addresses without a protocol (e.g. <IP_ADDRESS>:5080) to HTTP addresses - but explicitly adding the protocol (e.g. http://<IP_ADDRESS>:5080) for the FGMachine Docker commands seems to fix this. I've also updated the docs to reflect this.


Expose MsBuild Configuration and Platform setting

This allows choosing other (than default) MsBuild configuration, e.g.
MsBuild:Configuration=Release MsBuild:Platform=x64, especially usable
for projects that are not using default (AnyCPU) platforms.
This relates to #202.
P.S. Do we need to add some tests on that one? Or is there any other particular way to override Configuration/Platform planned?

Do we need to add some tests on that one? Or is there any other particular way to override Configuration/Platform planned?

We don't have great test infrastructure for this yet. It's pretty close, but would require a bit of refactoring to pass in custom options when creating an OmniSharpTestHost.
Eventually, I hope we can add an endpoint to support changing Configuration, Platform, and also TargetFramework.


why are you loading these csv's so many times?

see image

those were just placeholders, I re-added one csv per company for the small-multiples waffle chart!


prefix device-specific distinct IDs with '$device:'

Hey there, we just noticed that on AND, some distinct IDs have a "$device:" prefix. On iOS this is never the case. We are confused as to why this is the case? :D

@carstenhag mixpanel-swift started prefixing $device: in v4.0.5: https://github.com/mixpanel/mixpanel-swift/releases/tag/v4.0.5
The Objective-C SDK started w/ v5.0.2: https://github.com/mixpanel/mixpanel-iphone/releases/tag/v5.0.2
Both were released 5 months ago. If you are using a version >= those, then I would recommend reaching out to<EMAIL_ADDRESS>for help troubleshooting any identity management issues you're experiencing.


Develop 8 evolving conditions

closes #8

@mattldawson This implementation updates the state from emissions as a time-split operation. Another possible update would be as a emissions-source rate applied to the chemistry solver. While the use-case description did not specify the approach, It would be good to consider design elements to allow flexibility in terms of where the emissions are applied.

Sounds good. Maybe this can be included in the next set of requirements for MusicBox.


Add LED1733G7 Trafri light bulb

This weekend I bought a few Tradfri light bulbs from IKEA and added them tot Zigbee2MQTT, but couldn't see them showing up in Domoticz. Turns out, they are supported in Zigbee2MQTT, but not in the pluging yet.
When I added this to adapters/init.py, they showed up and work:
'LED1733G7': DimmableCtBulbAdapter, #TRADFRI LED bulb E14 600 lumen, dimmable, white spectrum, opal white
Could they be added?

Done


PA values are not saved in the settings file

To reproduce:

Launch PTVO
Select Board type as CC2530 + RFX2401 (config control pins!!!)
Click Expert tab
Click P12,P14 or P11,P10
Click Save settings to a file button
Open .ini file and search for TX EN= and RX EN=, they will not be set to selected values

Works correctly if pins are selected manually from the drop-down box.

Fixed in the latest version. Please, check.

Thank you, it is fixed now. Where can I ask some questions about your firmware for CC2530? Only in your blog?

Great! Yes, you may ask in the blog.


npm snapshot process does not end

Hey, I would like to take snapshots every minute, but for some reason the snapshot sometimes takes up to 10 minutes to finish even though the file/photo has already been created.
When the next snapshot starts I then get the following error message:
crond: user root: process already running: cd /app && npm run snapshot
For me it works by adding a timeout like this:
cd /app && timeout 30s npm run snapshot
But of course, that's more of a dirty hack.

Please use the updated version (2.0.0)


[core] add deletedFiles in NewFilesIncrement

Purpose

Linked issue: close #xxx

Tests

API and Format

Documentation

+1

https://github.com/apache/paimon/issues/3057

@YannByron @JingsongLi
Shall we increase the CommitMessageSerializer.CURRENT_VERSION and deserialize the objects according to the version for backward compactibility support ?


Can't retrieve informations

Hi try to use your library like in the example :
python syno.py
Creating Valid API
=== Utilisation ===
CPU Load:   None %
Memory Use: None %
Net Up:     None
Net Down:   None
=== Storage ===
Traceback (most recent call last):
  File "syno.py", line 14, in <module>
    for volume in volumes:
TypeError: 'NoneType' object is not iterable
(venv)iMac:venv sophie$ vi syno.py
(venv)iMac:venv sophie$ python syno.py
Creating Valid API
=== Utilisation ===
CPU Load:   None %
Memory Use: None %
Net Up:     None
Net Down:   None
=== Storage ===
Traceback (most recent call last):
  File "syno.py", line 14, in <module>
    for volume in volumes:
TypeError: 'NoneType' object is not iterable

with value :
api = SynologyDSM("<IP_ADDRESS>", "5000", "xxxx", "xxxxx")

Any idea ?

Hi, interesting please make sure the user has sufficient rights (or try with an Admin account).
Also which version  of DSM are you using?

version : DSM 6.1.3-15152 Update 3
My user "nicolas" is in the "admin" group. "admin" user is disabled on my Synology.

Thanks noticed I did not update my system yet so will do so and try to reproduce the issue, will get back to you.

I just have a test in DEBUG mode:
>>> api = SynologyDSM("<IP_ADDRESS>","5000", "xxxxxx", "xxxxxxxxxxx", True)
>>> api.utilisation
DEBUG: Creating New Session
DEBUG: Requesting URL: 'http://<IP_ADDRESS>:5000/webapi/auth.cgi?api=SYNO.API.Auth&version=2&method=login&account=xxxxxxx&passwd=xxxxxx&session=Core&format=cookie'
DEBUG: Request executed: 200
DEBUG: Failed: {"error":{"code":402},"success":false}

DEBUG: Authentication Failed
DEBUG: Login Failed, unable to process request
<SynologyDSM.SynologyDSM.SynoUtilization object at 0x7f33f1976cc0>

i search in Github SYNO.API.Auth, and i find some code from Synology in Openstack Cinder : https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/synology/synology_common.py
But this API use OTP authentication.

OK. I try with another user it works. I've got two-factor authentication enabled for this user. Sorry for noise.

No problem i will update the documentation to indicate 2 factor should be turned off (was going crazy not being able to reproduce this :-))


Updating version info for PSCore release

Docs Build status updates of commit 58d1a0a:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 58d1a0a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Connect-ExchangeOnline.md
:white_check_mark:Succeeded

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


Adding branch name to commit message

Issue
Hello,
I would like to add the branch name to the commit message when commit is done automatically with aider. We have the ability to change the prompt for the commit message but I didn't find the way to insert the branch name in the message. I have tried to add .git/HEAD to the files but it still not get it correctly. Is there a way to retrieve the branch name from aider?
I am using /commit as a work-around but it you be nice to have it done automatically.
Thanks for help
Version and model info
Aider: v0.67.0
model: gpt-4o

I found another workaround by adding a alias into bashrc like below
alias aider="export GIT_BRANCH=$(git rev-parse --abbrev-ref HEAD) && aider"
Then specifying the branch in AIDER_COMMIT_PROMPT .env with ${GIT_BRANCH}.
This is working however I need to exit aider when I want to change to another branch.


Ensure that hostnames without dots are excluded.

This patch fixes #2347.

Thanks Nissar! @funilrys
This one has been bothering us for awhile.
Merging!

Unfortunately Nissar @funilrys this change removes or truncates all domains containing dashes.
For example, truncations:

The truncation causes a great number of duplicates (deduplication is subsequent) so, in the end, some 40k domains disappear from the base list.
So I'm going to revert this change.

However Nissar @funilrys I'm keeping the new test you added 😄.


Does it makes sense to use wyvox/setup-action-pnpm?

So, setup-action-pnpm is a meta action, which doesn't really do anything special other than try to interpret the package.json correctly.
For example:

When I tried to setup release-plan on ember-sortable, I was getting errors, because it assumed pnpm 8 -- and like, some file should teh source of truth for that sort of thing (package.json#packageManager is a good such source)
thoughts?

here is the then-resulting upgrade to pnpm-8 to fix release-plan: https://github.com/adopted-ember-addons/ember-sortable/pull/534
(note: volta not used on CI, only some folks' machines)

Here is a preview of what release-plan using wyvox/setup-action-pnpm would look like:
https://github.com/adopted-ember-addons/ember-sortable/pull/535
I think one potential blocking feature that would need to be implemented is a fallback for node version lookup (in action/setup-node) -- I've configured it to look in package.json -- but I don't (yet) know if action/setup-node will continue looking other places if not found -- for example -- in the embroider repo, use-node-version is specified in the .npmrc.
We'll need to PR to here: https://github.com/actions/setup-node/blob/b39b52d1213e96004bfcb1c61a8a6fa8ab84f3e8/src/main.ts#L82
to add support for the .npmrc property that pnpm is looking for

I'd very much like to not do this. I find it a lot clearer to just have standard things in github actions, and with some of the pushback I've had about using it in ember-cli I'd rather not introduce it here 👍


Support joda

Is this supposed to be able to support Joda? Or is there no way of extending/adding extra objectMappers?

Nope, you can use Java 8's time classes.


Removes waiting for processing time when releasing iOS app

Introduces a performance improvement to release workflow that may save around 5 minutes, according to the last release workflow logs:

It basically ignores the processing time from AppStoreConnect. It doesn't make sense to wait for it since we'll need to manually release it latter in the portal.

@matuella Yes, but I think we'll need to provide the correctly CHANGELOG to them anyway. I though about using Changelog.txt that I removed in this PR, but I don't think this is going to work unless we keep it updated as we updated the base CHANGELOG.md. What do you think?

@matuella Yes, but I think we'll need to provide the correctly CHANGELOG to them anyway. I though about using Changelog.txt that I removed in this PR, but I don't think this is going to work unless we keep it updated as we updated the base CHANGELOG.md. What do you think?

I think the CHANGELOG.md is too technical for Testflight testers. I think these are good to support, although we will have to update these alongside CHANGELOG.md.
On a side note, how Play Store deals with version changelogs (for testers)?

We can have a similar structure in the Android target. Fastlane Supply supports CHANGELOG txt files as we are doing in iOS today. Do you think we should keep the CHANGELOG and automatically distribute new release to external testers or manually release these versions?

We can have a similar structure in the Android target. Fastlane Supply supports CHANGELOG txt files as we are doing in iOS today. Do you think we should keep the CHANGELOG and automatically distribute new release to external testers or manually release these versions?

I think automatic distributing for releases are nice, honestly - this means that store releases will still required a manual effort - as expected. Given that the work is pretty much summarizing the release in a .txt file, I don't see why not do it.

@matuella Updated this PR to correctly distribute beta builds with their respective CHANGELOG to iOS and Android stores


Updated oci_auto.py with IFTTT Support, Config Gen Tool, Security Fixes, General Typo Support

Added IFTTT integration
Added Config Gen tool that accepts POST data so secure information is not stored in script
Added generalized pathing statements so user directories are not stored in script
Fixed typos

While I think the approach here was correct, this code actually has quite a few bugs and inherent security flaws as well. Feel free to reference for improvements, but this shouldn't be merged into main.

Mainly, the code relies on storing the encryption key in the same dir as the encrypted file..
Secondly, there are no json validation checks and doesn't adequately account for user error
Third, the IFTTT integration still requires editing of the original script and therefor defeats the charter of increasing security by removing sensitive information from the script.

At this point I think the only valuable code worth implementing would be a fortified version of the JSON parser.


Can fd

Work in Progress: Enable CANFD and extended canIds
Feedback welcome;
ToDo:

make can fd configurable;
maybe even enhance configuration dialog for data bit rate
add data sample for extended can id

I've seen the following fork for CANFD support previously, which is more complete but with also contains some refactorings. I think you can check it for further reference until I got the time for fully reviewing that.
https://github.com/TaoWangRUB/plotjuggler-CAN-dbs/tree/can_fd_support


don't redo on deleting f""

Say I started typing a string "blah {foo}", the extension instantly puts an f"...." which is cool
However, If I want to remove the f and do a backspace on it, the extension put back the f"" again.
Can you control this behavior by a setting - which by default does not retry if user removes the f""

Hello. I'll explore if I can control not redoing addition of f in this case with a setting. Thank you for the suggestion.
One way to achieve this currently will be to use undo (instead of removing f) immediately after it adds as f. It will not add the f again.

@bhakta0007,
By default it should allow removing f added now and is controlled by the setting fstring-converter.skipEvaluationPostManualDeletionOfF. However another edit in the same line would add the f back as we have no way of maintaining the version. The f can still be removed.
Please try with v0.1.2 of the plugin and let me know if it solves your problem. Thank you.


Add banner with python versions

This helps users identify which Python versions are supported at a glance (you can see how it looks like on my fork).

Thanks @


DataTemplate in a grid ?

I am having some issues with what I'm trying to do, basically I want to display a list of properties in a grid with the property at the left, the one at the right and a gridsplitter in the middle moving all rows at once, however, if I place my elements like this, I would get a compilation error as DataTemplate expects the elements to be contained in something, the main issue is that if I place them in an other grid it would be bound by the rows of this grid and if I place them in something else like a StackPanel or something like this it would mess with their layout, is there any way to be able to do what I want to do ?
I've been able to get it to compile by placing the grid in the datatemplate but the result is one grid per item, this isn't the same thing, and moving the gridsplitter doesn't resize all the elements

I'm basically trying to get a result similar to a WinForms PropertyGrid generated from a list of items

This should help: https://wpf.2000things.com/tag/sharedsizegroup/

I ended up using DataGrid as it fits more what I was targetting, there's a weird issue though where the first element of the datagrid always stays blue, even when I select something else

I think you mean the alternate coloring of the grid
add more rows to your grid and you'l see what i mean.
Like this

Regards,
Remco

Oh I see, thanks, it was a bit confusing
honestly i'd be ready to pay for a udemy course or something like this simply to have a good introduction to most of at least the basics of avalonia as my main issue all along was that everything assumed we were already a WPF dev etc and didn't take that much time to explain things on the documentation, the tutorial there is only covers the really basic to display a list and add some elements to it but that's all, it doesn't really cover most controls, the different layouts etc

I also tried watching some other projects but most projects are pretty complex, I tried looking at AvalonStudio most of the files are kind of paml something like this, and the few xaml files are almost empty for instance

@SeleDreams
agreed, the Documentation is a bit lacking.
However there are some ok starting examles/blogs out there.
Do a search on google...... This dude https://www.nequalsonelifestyle.com/2019/06/13/avalonia-datagrid-getting-started/ got a few good starting examples
I am also new to VMMV, C# and avelonia, -so I feel your pain- it is a bit of a struggle.
I am planning on making tutorial and examples myself when i figured it out.
Have a good journy,
Remco


[port download failure] shader-slang

Missing hashes identified for port shader-slang
Build: https://github.com/danieljurek/vcpkg-cache-builder/actions/runs/7380927050

[!NOTE]
This list is updated by automation and may change as assets cycle out

627d77d83eef56e25815c30d0e99ae472c9189c760d1a620daebf49407c97e5e4da4b7880781c7788e85c5ccfba3a9e938e50ede2be0e93c0635050b11ad22a0
e898c20c60b4f3e8bbb262fef4cad88fba35b66dafb4784eec8b1603d8ee0293a71faeac1f84a9df96f33e206503ed717f7a068370e40a49bac90c009bbe4336

To fix:

If the SHA is not valid or the port has build problems

Add to ignore-ports.txt or ignore-hashes.txt

Ensure that the port downlaods successful which might include...

Ensuring that a required triplet is present (easiest)
Building the port to download the asset (needs build infrastructure)
Ensure features are properly configured

Missing hashes identified for port shader-slang
Build: https://github.com/danieljurek/vcpkg-cache-builder/actions/runs/7380927050

[!NOTE]
This list is updated by automation and may change as assets cycle out

627d77d83eef56e25815c30d0e99ae472c9189c760d1a620daebf49407c97e5e4da4b7880781c7788e85c5ccfba3a9e938e50ede2be0e93c0635050b11ad22a0
e898c20c60b4f3e8bbb262fef4cad88fba35b66dafb4784eec8b1603d8ee0293a71faeac1f84a9df96f33e206503ed717f7a068370e40a49bac90c009bbe4336

To fix:

If the SHA is not valid or the port has build problems

Add to ignore-ports.txt or ignore-hashes.txt

Ensure that the port downlaods successful which might include...

Ensuring that a required triplet is present (easiest)
Building the port to download the asset (needs build infrastructure)
Ensure features are properly configured

Hash(es) not listed in the current version of vcpkg and not actively requested. Skipping.


Design template for post-meeting overview

We need a Mailchimp template to send notes and details out to members after meetups

This is done! Sitting in Mailchimp :)


Exception in buildAnnotationNode when using future's annotations

Nuitka --version is:
<IP_ADDRESS>
Python: 3.8.3 (default, May 14 2020, 11:03:12)
Executable: /usr/bin/python3
OS: Linux
Arch: x86_64

This is the version from https://nuitka.net/pages/download.html. Originally I had tried with 0.6.8 as found in Debian unstable, with the same result.
Same issue applies to the 0.6.9rc1 snapshot as well.
This does not work:
from __future__ import annotations

from typing import Sequence

def colonjoin(seq: Sequence[str]) -> str:
    return ":".join(seq)

print(colonjoin(["foo", "bar"]))

Removing the from __future__ import annotations makes the example work.
The backtrace I get is:
Nuitka:WARNING:Problem at '<SourceCodeReference to ../ircstream/koko.py:5>' with FunctionDef(name='colonjoin', args=arguments(posonlyargs=[], args=[arg(arg='seq', annotation=Subscript(value=Name(id='Sequence', ctx=Load()), slice=Index(value=Name(id='str', ctx=Load())), ctx=Load()), type_comment=None)], vararg=None, kwonlyargs=[], kw_defaults=[], kwarg=None, defaults=[]), body=[Return(value=Call(func=Attribute(value=Constant(value=':', kind=None), attr='join', ctx=Load()), args=[Name(id='seq', ctx=Load())], keywords=[]))], decorator_list=[], returns=Name(id='str', ctx=Load()), type_comment=None).
Traceback (most recent call last):
  File "./nuitka/__main__.py", line 195, in <module>
    main()
  File "./nuitka/__main__.py", line 188, in main
    MainControl.main()
  File "./nuitka/MainControl.py", line 739, in main
    main_module = createNodeTree(filename=filename)
  File "./nuitka/MainControl.py", line 77, in createNodeTree
    main_module = Building.buildModuleTree(
  File "./nuitka/tree/Building.py", line 1054, in buildModuleTree
    createModuleTree(
  File "./nuitka/tree/Building.py", line 995, in createModuleTree
    module_body = buildParseTree(
  File "./nuitka/tree/Building.py", line 712, in buildParseTree
    result = buildStatementsNode(provider=provider, nodes=body, source_ref=source_ref)
  File "./nuitka/tree/TreeHelpers.py", line 430, in buildStatementsNode
    statements = buildNodeList(provider, nodes, source_ref, allow_none=True)
  File "./nuitka/tree/TreeHelpers.py", line 350, in buildNodeList
    entry = buildNode(provider, node, node_source_ref, allow_none)
  File "./nuitka/tree/TreeHelpers.py", line 311, in buildNode
    result = build_nodes_args3[kind](
  File "./nuitka/tree/ReformulationFunctionStatements.py", line 230, in buildFunctionNode
    annotations = buildParameterAnnotations(provider, node, source_ref)
  File "./nuitka/tree/ReformulationFunctionStatements.py", line 491, in buildParameterAnnotations
    extractArgAnnotation(arg)
  File "./nuitka/tree/ReformulationFunctionStatements.py", line 478, in extractArgAnnotation
    value=buildAnnotationNode(provider, arg.annotation, source_ref),
  File "./nuitka/tree/TreeHelpers.py", line 384, in buildAnnotationNode
    exec(r, m)
  File "<annotations>", line 5, in <module>
NameError: name 'Sequence' is not defined

Thanks for your report, this should hopefully be easy. Unfortunately some exec is needed for this, until Python adds the method to do it. I need to check, if that's not the case yet.

Ah, it's in Python 3.9 only, so I guess, I am adding something to the code, to remind me of that, once it's released.

I am going to make a star import from typing for the exec, but it's not too general. However, what's string, is that I didn't reproduce the issue now.

I am simply getting this:

python3.8 bin/nuitka-run Mini.py
{'seq': 'Sequence[str]', 'return': 'str'}

for this program:
from __future__ import annotations

from typing import Sequence

def colonjoin(seq: Sequence[str]) -> str:
    return ":".join(seq)

print(colonjoin.__annotations__)

I wonder, if this could really be about the minor 3.8 version, other than that I am confused. In my WSL Debian Buster, I have this self compiled 3.8.0 and you got Debian's 3.8.3, for which there is extra option of Debian backports. I will try 3.8.3 once I can.

Oh shoot, I just read the changelog, and this is what truly happens:
https://bugs.python.org/issue39562
The renumbered the future flags, and I was lazy in this branch, and just used the numerical value, that heaven knows, I got it from:
        r = compile(_host_node, "<annotations>", "exec", 1048576, dont_inherit=True)

A comment deemed it unlikely for this to happen, therefore they didn't follow the safe route. No idea, where I took the number from, and why, obviously it would have been better to just add up to flags. My git repo makes this surface all at once. But I can re-construct the actual flags of course.

Ah, this is the value of __future__.CO_FUTURE_ANNOTATIONS only.

A fix should be coming soon, this is simple them. I guess, I wanted to avoid the import of the module at the time, thinking they don't change, as C code is allows to use them, but probably that's too optimistic with the CPython community. For experimental features, they rightfully are ruthless.

This should be fixed on factory https://nuitka.net/doc/factory.html

This is part of <IP_ADDRESS> release that was just done.


Is it possible to build a static build version available for Windows?

Is it possible to build a static build version available for Windows?

There are no plans to support Windows.


Add Crystal support for .cr and .ecr files

Thanks for this plugin, my # is happy again, at least, in ruby.
This PR adds support for Crystal language.
Language definition grabbed from https://github.com/crystal-lang-tools/vscode-crystal-lang/blob/master/package.json#L38-L69 . I'm not sure if the definitions differ how they are resolved. Should only the id and extensions be kept in this PR?
cc: @faustinoaq

Note: the CI is complaining only in a JS test in Windows.

Thank you @bcardiff :tada:


Retry subscribe to Salesforce event when server returns 503 Server too busy

EMP Connector fails retry subscription when SF result in Server too busy response. Instead of dumping need to capture the response and retry again for subscription
Logs that the server returns additional information:
{ext={sfdc={failureReason=503::Server is too busy. Please try your request again later.}}, ...}
As described in the Salesforce documentation here: https://developer.salesforce.com/docs/atlas.en-us.api_streaming.meta/api_streaming/streaming_handling_errors.htm
the 503 Server too busy is a documented error that should/can be retried.
It would greatly add to the resilience of our system if the Salesforce component would automatically retry a subscribe if failed due to a temporary error.

I am stuck with the same error. Did you manage to overcome this @mohitj13 ?


check pytest summary - do not merge

I am trying to figure out why my other package gives multiline output in the pytest short summary on GHA. It should not and I was not able to reproduce it locally, and it seems not to do that on the xarray repo (checking old failed runs). But to be sure I open this dummy PR to see if that is actually true. Sorry for the spam.

=========================== short test summary info ============================
FAILED regionmask/tests/test_test.py::test_a - AssertionError: a long
  multiline
eason
assert False
FAILED regionmask/tests/test_test.py::test_b - ValueError: another
multiline	hingy
FAILED regionmask/tests/test_test.py::test_c - AssertionError:
Not equal to tolerance rtol=1e-07, atol=0

Mismatched elements: 1 / 1 (100%)
Max absolute difference: 1
Max relative difference: 0.5
 x: array(1)
 y: array(2)
====== 3 failed, 881 passed, 22 skipped, 1 xfailed, 30 warnings in 38.21s ======

pytest has stopped truncating for a while now. I believe that only happens if the CI or BUILD_NUMBER environment variables are set? Does GHA set these by default?

Yes that's it
export CI=1
pytest .

triggers the output. Thanks a ton - that was driving me crazy! :bowing_man:

It's since pytest 7.2.0 (2022-10-23) so that's indeed a moment (the CI env var was added around 2020 on GHA). Slightly surprised I never noticed that before...


Add grid-column-collapse to allow nested grids

What would you think about wrapping the collapsing div in a media query for the example?
The way that Foundation illustrates their grid collapse is through media queries. I'm not sure what the most common use case is since I'm pretty sure I've never used the feature, but it would serve as a good example of how it works.

🌵 The code looks good, but I agree that it's still confusing when you should use it. It's a common feature of grid systems, so I'm happy to have it in there, but we should pay special attention to it in the docs

Code looks good! I think the naming of grid-collapse is a bit unclear. It feels like grid addresses the larger grid, as opposed to the column within it.
Would you consider something like grid-column--collapsed?

This was kind of a bad idea on my part. I am closing this in favor of #492


Finished: Mother child difference

Closed #367

Thanks! Fixed lint errors and now I use ancestry variable (didn't notice that we have it)

I see that some conflicts appear in that branch,
Could I ask you to re-create branch with that exercise ?


Fixed sum_intensity_projection docstring

This PR contains:

[ ] a new test-case for the benchmark

[ ] I hereby confirm that NO LLM-based technology (such as github copilot) was used while writing this benchmark

[ ] new dependencies in requirements.txt

[ ] The environment.yml file was updated using the command conda env export > environment.yml

[ ] new generator-functions allowing to sample from other LLMs
[ ] new samples (sample_....jsonl files)
[ ] new benchmarking results (..._results.jsonl files)
[ ] documentation update
[x] bug fixes

Related github issue (if relevant): closes #116
Short description:

Fixes docstring of sum_intensity_projection.ipynb

How do you think will this influence the benchmark results?

Should improve the benchmark quality for this test for many weaker models.

Why do you think it makes sense to merge this PR?

It is good to prompt the LLM with the properly specified goal.

Good catch @ian-coccimiglio !
I'll merge this into the development branch, so that we can use the new prompt in the next big benchmark run.


Add your student profile to 'Student'

Follow the steps and add your 'username.md' file to 'Student'.

hello see u i m new to open source


Add/Update logging in static files middleware

From offline logging discussion:
Add Verbose logging where it can be useful for debugging

Time box to 2 dev-days maximum (across all of k-core)
Use EventIds at each logging statement
Update any existing logging to include an EventId

Relevant logging points we want to include for sure:
        - Served a file - Information
        - Did not serve a file

/cc @muratg

@Tratcher Looks like a good suggestion. Want to take a stab at very early RC1?

Did not serve a file would fire for almost every request, sounds like it should be verbose or debug.  @rynowak Any examples of work already done that uses EventIds?

Yes, not-serving a file would be Verbose.


Completion broken for brew upgrade on Linux

brew doctor output
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output

brew config
HOMEBREW_VERSION: 4.2.4
ORIGIN: https://github.com/Homebrew/brew
HEAD: 1c612baa096c69f2fcb221c74e6f5b9979efdcee
Last commit: 27 hours ago
Core tap JSON: 16 Jan 17:08 UTC
HOMEBREW_PREFIX: /home/linuxbrew/.linuxbrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: :0
HOMEBREW_DISPLAY_INSTALL_TIMES: set
HOMEBREW_EDITOR: vim
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 20
Homebrew Ruby: 3.1.4 => /home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: 20-core 64-bit unknown_0x6_0x9a
Clang: N/A
Git: 2.34.1 => /bin/git
Curl: 7.81.0 => /bin/curl
Kernel: Linux <IP_ADDRESS>-microsoft-standard-WSL2 x86_64 GNU/Linux
OS: Ubuntu 22.04.3 LTS (jammy)
WSL: 2 (Microsoft Store)
Host glibc: 2.35
/usr/bin/gcc: 11.4.0
/usr/bin/ruby: N/A
glibc: N/A
gcc@11: N/A
gcc: N/A
xorg: N/A

What were you trying to do (and why)?
Use tab-completion (in bash) to upgrade a formula without typing its full name (because I'm lazy and/or don't want to misspell it).
What happened (include all command output)?
$ brew uninstall libg[Tab]  # Instantly completes to:
$ brew uninstall libgit2

$ brew upgrade libg[Tab]  # Hangs ~7s then bells. Never completes.

# Apparently still in the middle of typing the command, so:
$ brew upgrade libg[Enter]
Error: No available formula with the name "libg". Did you mean libgr?

What did you expect to happen?
Formula name should autocomplete without error.
Notably, completion works fine for brew uninstall, which I assume should pull from the same list of options.
$ brew uninstall libg[Tab]  # Instantly completes to:
$ brew uninstall libgit2

Step-by-step reproduction instructions (by running brew commands)
[Run homebrew on linux]
brew install [formula]
brew upgrade [partial formula name][Tab]

Bash completion info

what is your bash --version and which bash output?

$ echo "$SHELL"
/bin/bash

$ which -a bash
/usr/bin/bash
/bin/bash

$ /usr/bin/bash --version
GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

$ /bin/bash --version
GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)
...

How have you setup your Bash completion?

The short version is
source /home/linuxbrew/.linuxbrew/etc/bash_completion.d/brew
source /usr/share/bash-completion/bash_completion

The long version is that my `~/.bashrc` has a bunch of conditional logic for different systems, including these bits, which resolve as above on this system:
    if [ ! -r "${HOMEBREW_PREFIX}/etc/profile.d/bash_completion.sh" ]; then
        # If that exists (from `bash-completion` formula), it will have
        # automatically sourced completions. Otherwise, do it manually:
        for COMPLETION in "${HOMEBREW_PREFIX}/etc/bash_completion.d/"*; do
            [ -r "${COMPLETION}" ] && source "${COMPLETION}"
        done

# activate bash-completion
if [[ $PS1 ]]; then  # interactive shell
    if   [[ -r "/usr/share/bash-completion/bash_completion" ]]; then
        . "/usr/share/bash-completion/bash_completion"
    elif [[ -r "/usr/local/etc/profile.d/bash_completion.sh" ]]; then
        . "/usr/local/etc/profile.d/bash_completion.sh"
    elif can_brew && [[ -r "$(brew --prefix)/etc/profile.d/bash_completion.sh" ]]; then
        . "$(brew --prefix)/etc/profile.d/bash_completion.sh"
    elif [[ -r "$HOME/.linuxbrew/etc/profile.d/bash_completion.sh" ]]; then
        . "$HOME/.linuxbrew/etc/profile.d/bash_completion.sh"
    fi
fi

Another example

I have the same experience on another system, with CentOS 7 & Bash 4. It's not what I'm mainly reporting on because it's a non-standard install and all kinds of buggered. But for another example:
centos $ echo "$SHELL"
/bin/bash

centos $ which -a bash
/bin/bash
/usr/bin/bash

centos $ /bin/bash --version
GNU bash, version 4.2.46(2)-release (x86_64-redhat-linux-gnu)
Copyright (C) 2011 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

centos $ /usr/bin/bash --version
GNU bash, version 4.2.46(2)-release (x86_64-redhat-linux-gnu)
...

And the same .bashrc which means effectively:
source /usr/lusers/jacktose/homebrew/etc/bash_completion.d/brew
source /usr/share/bash-completion/bash_completion

Related issue
I reported this in its previous form in #16424. #16452 hid the stderr message, but the failed completion remains an issue.

Ah, that explains it: there are no outdated libg* formulae.
It seems like a clever idea to only complete names of actually outdated formula, but I'd find it a better experience to complete any installed formula name and get the instant feedback of the completion plus the ~2s feedback of the warning:
$ brew upgrade libgit2
Warning: libgit2 1.7.1 already installed

It seems like a clever idea to only complete names of actually outdated formula, but I'd find it a better experience to complete any installed formula name and get the instant feedback of the completion plus the ~2s feedback of the warning

Agreed. Would review and likely accept a PR that did this.

This explains a lot why brew outdated previously didn't do auto-updates: the shell completions use brew outdated!
At a minimum, auto-updates should be disabled within shell completions. Though even then the ~0.7s+ for brew outdated is probably noticeable given brew formulae etc take ~0.05s.

Though even then the ~0.6s+ without auto-updates for brew outdated is probably noticeable given brew formulae etc take ~0.05s.

Yeah, that isn't ideal. Most likely, we cannot hit Ruby code. Which would mean either just switching to installed formula/casks (revert of line: https://github.com/Homebrew/brew/commit/3af16832d9baa6ebe3d85701734720d0fe75bea4#diff-2b40f539194f871ba5fb8069acf4255481b282877bb7019d90f1844252f0651a) or caching the outdated formulae/casks during brew update.

Also, since we use complete -o bashdefault ... this means an empty COMPREPLY (i.e. no outdated formulae/casks) currently outputs default of pwd's files/directories. This probably makes completion look broken.

Though even then the ~0.6s+ without auto-updates for brew outdated is probably noticeable given brew formulae etc take ~0.05s.

Yeh I think brew formulae is worth using.

caching the outdated formulae/casks during brew [update|outdated].

Could do but I worry about an outdated outdated being worse than just using formulae.

The easiest option is then using installed_* (i.e. #16528). This does modify manpages and docs too if that matters.

https://github.com/Homebrew/brew/pull/16528 is merged so should be available in next brew release tag.

Closing with 4.2.6 release.

Thanks all! The experience in 4.2.6 is much nicer, IMO.


Error

xxxx-Pro:django-vue-template zz$ python3 manage.py migrate
Traceback (most recent call last):
  File "manage.py", line 8, in <module>
    from django.core.management import execute_from_command_line
ModuleNotFoundError: No module named 'django'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "manage.py", line 14, in <module>
    ) from exc
ImportError: Couldn't import Django. Are you sure it's installed and available on your PYTHONPATH environment variable? Did you forget to activate a virtual environment?

I use Mac and something wrong.
Could u help me plz?

Sounds like you don't have Django installed.
Did you install the dependencies from the requirements file?

I used Django in another project.
And I install Django again in this project after your suggestion.
But the error looks like to be another one.
Successfully installed django-2.2 pytz-2019.1 sqlparse-0.3.0 zhangzhendeMacBook-Pro:django-vue-template zz$ python3 manage.py migrate Traceback (most recent call last):   File "manage.py", line 15, in <module>     execute_from_command_line(sys.argv)   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line     utility.execute()   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/django/core/management/__init__.py", line 357, in execute     django.setup()   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/django/__init__.py", line 24, in setup     apps.populate(settings.INSTALLED_APPS)   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/django/apps/registry.py", line 91, in populate     app_config = AppConfig.create(entry)   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/django/apps/config.py", line 116, in create     mod = import_module(mod_path)   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/importlib/__init__.py", line 127, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 1006, in _gcd_import   File "<frozen importlib._bootstrap>", line 983, in _find_and_load   File "<frozen importlib._bootstrap>", line 965, in _find_and_load_unlocked ModuleNotFoundError: No module named 'whitenoise'
Am I missing something? Thanks a lot!

Please read through the Readme instructions.
This project has many dependencies:
https://github.com/gtalarico/django-vue-template/blob/master/Pipfile
You will need to use pipenv to get the dependencies installed. Cmds are included in the readme


Robotics_motion Service Abnormal Disconnection

During a cyclic endurance test, after running 3160 cycles, the robot stopped moving. The following error messages were output in the output window:

cycle count:
3158
cycle count:
3159
cycle count:
3160
Execute procedure "hello" failed:
Traceback (most recent call last):
File "/home/boomy/venv/lib/python3.9/site-packages/pyri/sandbox/pyri_sandbox.py", line 290, in Next
self._locself._procedure_name
File "<pyri_sandbox>", line 15, in hello
File "/home/boomy/venv/lib/python3.9/site-packages/pyri/robotics/sandbox_functions.py", line 96, in robot_movej
move_gen = motion_service.movej(robot_name, joint_pos_degree, float(speed_perc))
File "/home/boomy/venv/lib/python3.9/site-packages/RobotRaconteur/RobotRaconteurPythonUtil.py", line 244, in
return lambda self, *args: stub_functioncallgenerator(stub, m1.Name, m1, *args)
File "/home/boomy/venv/lib/python3.9/site-packages/RobotRaconteur/RobotRaconteurPythonUtil.py", line 457, in stub_functioncallgenerator
return GeneratorClient(stub.GeneratorFunctionCall(name, m), type1.ReturnType, param_type, stub, stub.RRGetNode())
File "/home/boomy/venv/lib/python3.9/site-packages/RobotRaconteur/RobotRaconteurPython.py", line 6639, in GeneratorFunctionCall
return _RobotRaconteurPython.WrappedServiceStub_GeneratorFunctionCall(self, FunctionName, args)
RobotRaconteur.RobotRaconteurPythonError.RequestTimeoutException: RobotRaconteur.RequestTimeout Request timeout

Attempting to run Procedures again results in the same error and displays the following message box:

The web page shows that robotics_motion is in a disconnected state:

Attempts to manually start the pyri-robotics-motion-service service fail with the message: "Exception: Identifier UUID or Name already in use":

I have checked the log information but did not find any useful error messages.
pyri-core-2024-01-19--13-33-08.zip
System Version: Ubuntu 22.04 + Preempt RT (please disregard 'VirtualBox' as it is actually running on a physical machine).
Python Environment: pyenv + Python 3.9.5
Installation Method: Followed the instructions in the readme, installed using pip and pyri-cli webui-install
Robot: Realman RML63
Robot Driver: TCP + JSON protocol communication at 100Hz
Could you please advise on how to further investigate this issue to improve system stability? Also, I increased the sampling points to 40 as per this issue. Might this change be affecting the system? Thanks for your help!

Hi @maxwelllls, the procedure runtime sandbox is not intended to have the cycle looped within the produce. If possible, please use the "Main" panel to create a state machine and call the produce repeatedly. You can have it "Jump" at the end of the block to the same block name to repeat.
https://github.com/pyri-project/pyri-core/blob/master/doc/user_programming_guide.md#main-state-machine

Following your suggestion, I switched to using the main component for loop testing. However, after about 1000 iterations, the pyri_robotics_motion_service still crashed, leading to the failure of movej command calls.

By using netstat -an | grep 55921, it can be observed that a TCP server remains in the LISTEN state, and a large number of TCP connections are stuck in the closing process, not closing correctly.

The steps to reliably reproduce the issue are as follows:
Run run_2ur5e_sim, set two groups of joint poses with minimal differences, which I've named ur_ref_pos_1 and ur_ref_pos_1_1, then cycle through them. It takes about 10 minutes to reach 1000 iterations and cause a crash."

Using the command netstat -an | grep 55921 it can be observed that the TCP server remains in the LISTEN state, but a large number of TCP connections are stuck in the CLOSE_WAIT state. This seems to indicate an issue with the server-side program logic, possibly failing to properly handle the logic for closing connections.

After enabling debug mode, the log files obtained continuously indicate failures in IPv6 connections, but there is no information related to the crash.
debug_log.txt

Hi, thank you very much for taking the time to address this issue. I have also been studying the RR source code, hoping to gain a deeper understanding and contribute to the development of this project. I appreciate your efforts and look forward to any insights you may have regarding the problem.

Hi@johnwason，After spending some time reading the source code and debugging, I've been able to pinpoint that void ServiceSkel::CleanupGenerators() starts to clean up outdated Generators after 10 minutes, which then highly likely leads to timeouts when calling the movej function. The code uses mutexes to protect generators, but I suspect there might be conflicts with other resources, such as ASIO.
Given the vastness of the RR framework, my understanding of the code is still quite limited. Could you provide some guidance on directions for further investigation?

@maxwelllls can you run pip list on your venv so I can see what packages and versions are installed?

Also can you look at the developer console on the web browser? It looks like the browser may be creating a lot of connections and closing them.

Following your instructions, I have provided the requested information. The console messages appeared only upon accessing the webpage and did not update when starting or encountering issues with the Procedures. It was a few minutes after the failure of running Procedures that new messages indicating "client closed" appeared. Attached are the results of the pip list command, screenshots of the browser page, and the full browser console output for your review.
pip list.txt
Screenshot
console-export-2024-3-7_9-0-41.txt
I hope this information helps with troubleshooting. Thank you for your guidance and support.

Thanks for the data
I think I have reproduced the problem. I am trying to track down the cause.
The connection errors you see in the console are actually normal behavior. There are several candidate URLs to connect to the services, and the errors you are seeing are the rejected URLs. It looks like the connections are successful.

By shortening the cleanup time for generators to 1 minute and increasing the call frequency of movej as much as possible (unrelated to the length of the movej path, using rapid short paths and 100% speed), the issue can be quickly reproduced. Looking forward to your findings.

I think the problem is in the class TrajectoryMoveGenerator in pyri.robotics.robotics_motion_service.__main__. The cleanup generator function is collecting this class. Something is causing a leak or possibly a concurrency problem. This class is currently implemented as a synchronous generator, where it really should be an asynchronous generator. See https://robotraconteur.github.io/robotraconteur/doc/core/latest/python/api/Generators.html . There is a newer utility in the companion library to help with this design pattern: https://github.com/robotraconteur/robotraconteur_companion_python/blob/master/src/RobotRaconteurCompanion/Util/TaskGenerator.py

It looks like objects from Toppra may be passed to the generator. If Toppra memory is being released by RR it could definitely cause a problem.

The problem was a complex interaction between locks in C++ and the Python Global Interpreter Lock (GIL). In the CleanupGenerators() function, In the function, the generators_lock is used to protect the map during modification. Before, the generators were being destroyed when the generators_lock was still held. Because the objects being destroyed were from Python, the GIL had to be locked. Because of the order of locking, this was causing deadlocks with other threads. The pull request https://github.com/robotraconteur/robotraconteur/pull/179 modifies the destruction process so that the actual deletion in Python happens after the lock is released. This is a complex interaction that is harder to predict and can be even harder to fix.
There was also an issue where the generators were not being deleted immediately once they were complete rather than later with a timeout. That is why the issue was only showing up after a while.
Releasing the core Robot Raconteur library is a little complicated so you will need to build from source until I am able to release it.
There is also a problem where the motion execution will get stuck if you stop the procedure. This is unrelated and I will work on fixing it. For now, if you click "Enable Jog" it will clear the deadlock.

Thank you for resolving this issue. Building from source is feasible, and I've been trying to fix this problem through code modifications this week. Currently, the code uses CleanupGenerators to delete outdated Generators. Could we consider explicitly calling close to clean up these Generators?

After reviewing your pull request, I noticed that SendGeneratorResponse() now explicitly erase the generators.
I’ve successfully run 5000 tests without issues.

https://github.com/robotraconteur/robotraconteur_standards/blob/master/draft/object_protocol.md#generatornext
This is the spec on how the generator should behave. I think there are still a few more places the lifecycle needs to be improved but I think the major errors are fixed.

Then, can I directly base my work on the standard robot, adding functions such as nativeMoveL, nativeMoveC that directly call the robot's own APIs, bypassing the planning part and sending commands directly to the robot?


[HUDI-2262] reduce build warnings

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
This pull request reduce the build warning with minor changes
Build warnings tackled -
[WARNING] The expression ${parent.version} is deprecated. Please use ${project.parent.version} instead.
[WARNING] Parameter tasks is deprecated, use target instead
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[x] Has a corresponding JIRA in PR title & commit

[x] Commit message is descriptive of the change

[ ] CI is green

[x] Necessary doc changes done or have another open PR

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

@data-storyteller : may I know why this PR is in draft? I see CI has passed.

@nsivabalan  12202.1 Unit tests for hudi-spark-client task  from build failed, I will look into it.


Add Boot Environment Integration to OPNsense Web Interface

This pull request introduces a new feature to the OPNsense web interface, allowing users to manage FreeBSD boot environments directly within OPNsense. This integration provides an intuitive and seamless way for users to create, manage, and switch between boot environments, enhancing system management and recovery options.

@sheridans as this is quite some code, we will need time for reviewing and testing. It sounds like a practical addition to OPNsense, probably roadmap 25.1 material.

@sheridans I've created a new branch (be) to work on this, but it does need quite some effort before it will be in an acceptable state for core.
I don't mind restructuring the code and making this happen somewhere on the road to 25.1, but I do want to make sure I'm not running into obstructions while doing so. My first batch of changes (already showing a bit of the direction) is committed in https://github.com/opnsense/core/commit/6ce7644dd9a225f79ef6b71bf968553aafa12fe5
Thoughts?

I can see where you're going with this, and how the code can be cleaned up. It was my first attempt at writing anything for OPNsense, from your changes I can see the direction. I'll take a look at refactoring, I had actually gotten the model working for validation and things, but this can be done in code without any issue. I'd already addressed most of the issue with validation, checking if they exist etc as you branched it. I'll create a PR into that branch, thanks for pointing in the right direction, exactly what I needed

@sheridans I also don't mind finishing the work needed to pull this trough, just don't want to step on anyones toes.

Heh, there's no toes to be stepped on here. Open source is a team sport. I'm happy  to finish it, or let it be finished.
The goal, once functional, was to introduce boot environments with auto recovery features [without a paid subscription, mentioning no company that may or may not have inspired this :)].
If you're schedule is limited, or struggling time wise, happy to help wrap it up. Otherwise, have at it...

let me take another stab at it then, I'll keep you posted.

superseded by https://github.com/opnsense/core/pull/7749


Unexpected failure when preparing tensor allocations

Hi!
I've trained torch EfficientNetB0 model, then converted it to tflite and faced this failure:
Internal error: Unexpected failure when preparing tensor allocations: tensorflow/lite/kernels/conv.cc:332 input->dims->data[3] != filter->dims->data[3] (4 != 3)
I've loaded model as New-Other-TensorflowLite Model and then tried to use:
try{
    MyModel myModel = MyModel.newInstance(MainActivity.this);
} catch (IOException e){
     // TODO
}

but get this error.
Here is part of my code for conversion:
dummy_inputs = (torch.randn(1,3,256,256),)

edge_model = ai_edge_torch.convert(model.eval(), dummy_inputs)
edge_model.export("classifier_best.tflite")

Am I doing something wrong?

Hi, @NazaRik555
I apologize for the delayed response and If possible could you please help us with Google colab notebook with your model along with complete steps to reproduce the same behavior from our end to investigate this issue further ?
Thank you for your cooperation and patience.


Implement xDS circuit breaking interop test

Implements xDS circuit breaking test. See go/grpc-xds-circuit-breaking-interop-tests.
For languages wanting to enable circuit breaking test, add circuit_breaking to the --test_case option and use the new server image projects/grpc-testing/global/images/xds-test-server-3.
/cc @menghanl @ericgribkoff @donnadionne

Closing in favor of #24704.


Add type to Select option button to prevent form submission

The button elements within the Select component options need a type="button in order to prevent a containing form from submitting when clicked.

Dang, I missed the Select too. I will have this fixed tonight.


import/no-unresolved with webpack modules/aliases in the VS Code

Update:
So, there are few separate issues.

To fix issue now, I can adjust my config: replace object spreading with Object.assign, or use babel-register.
Something interpret Unexpected token ... as ESLint error and show no-undesolved, no-duplicate and other.

Which is wrong - it should be SyntaxError in ESLint output.

It happens because node v7.9.0 in VS Code can't launch webpack config with object spread syntax.
May be vscode-eslint can launch .eslintrc.js and webpack config with local node, not with bundles one?

Original text
Faced with such issue.
There is no errors while launching eslint from terminal.

Trying to reproduce error in clean repository, but without success yet.

This was driving me bonkers and I was getting other odd errors from the import plugin. Just for kicks I put:
require('babel-register')

at the top of my .eslintrc and magically everything seems to work now! (I was already using babel-register elsewhere in my project, so it was already installed)
It seems like vscode-eslint runs eslint as a library, and JS imports aren't recognized without babel-register

@alecf
@dbaeumer
No, my webpack config doesn't require babel. It use require() for imports, etc.
But, when I create some simple webpack configuration - it works.
The reason is definitely in webpack config, but it is something else, not the new syntax.
I will try to debug it this week.

@dbaeumer
Found it!
Our project is bundled with node 8.
So, there is some object destructions in webpack configuration.
But there is node v7.9.0 VS Code! Webpack config executed with node 7.9.0 when vscode-eslint try to execute .eslintrc.js with eslint-plugin-import and eslint-import-resolver-webpack.
The wierd thing is, that in most of my files I have strange errors, like at the first image in that issue.
[eslint] Resolve error: Unexpected token ... (import/no-unresolved)
[eslint] Resolve error: Unexpected token ... (import/no-duplicates)

And ESLint output in VS Code was pretty clean

But, when I add one more resolver in my .eslintrc.js
'node': {
    'paths': [
        __dirname,
    ],
},

resolve errors are gone, and I, finally, saw the real error
[Info  - 19:40:18] ESLint server is running.
[Info  - 19:40:19] ESLint library loaded from: /Users/tasogare/Projects/MY_FOLDER/node_modules/eslint/lib/api.js
[Error - 19:40:20] ESLint stack trace:
[Error - 19:40:20] /Users/tasogare/Projects/MY_FOLDER/configs/webpack/browser.js:46
            ...grab(OLD_STYLE_PAGES_DIR, 'browser.js'),
            ^^^
SyntaxError: Unexpected token ...
    at createScript (vm.js:53:10)
    at Object.runInThisContext (vm.js:95:10)
    at Module._compile (module.js:543:28)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/tasogare/Projects/MY_FOLDER/configs/webpack/development.js:9:23)

So, there are few separate issues.

I can adjust my config: replace object spreading with Object.assign.
Something interpret Unexpected token ... as ESLint error and show no-undesolved, no-duplicate and other.

Which is wrong - it should be SyntaxError in ESLint output.

May be vscode-eslint can launch .eslintrc.js and webpack config with local node, not with bundles one?

1.1
Ok, now I'm understand how to solve this with babel-register
require('babel-register')({
    presets: [ 'es2015', 'stage-2' ],
});

It wasn't help me before, because, with default settings, it can't handle spread operator.
With stage-2 it can.

@ValeryVS thanks for the great analysis. Very helpful.
I will keep the item open to look into 2 and 3

https://github.com/Microsoft/vscode-eslint/issues/405#issuecomment-380745912
Still have this issue with VSCode 1.30.1
Project is bundled normally, but eslint plugin underlines me "incorrect" imports in paths with webpack aliases.
Here I found workaround but don't understand how to fix .eslintrc file.
https://github.com/Microsoft/vscode-eslint/issues/464#issuecomment-387205760
No solution to the problem found still?

@alecf Can you show your root .eslintrc workaround?
I need to install babel-register globally?

I changed webpack config from this:
resolve: {         alias: {             '~': path.resolve(__dirname),             '@': path.resolve(__dirname, './src'),         },     },
to this:
resolve: {         alias: {             '~': path.resolve(__dirname),             '@': path.resolve(__dirname, 'src'),         },     },
It seems to be ok. But intellisence not showing fresh added methods in imported file,
but show arguments intellisence after I writing the name of "fresh" method completely.

Thanks @alecf for the workaround. Switching from .eslintrc to .eslintrc.js and adding require('@babel/register') fixed it for me.
Still, I don't think this hack should be necessary, seems like this is a bug in the VSCode plugin, if the CLI works fine without the hack.
By the way, this only started happening to me after upgrading to Babel 7 and switching from .babelrc to babel.config.js, not sure if that's relevant.

Regarding https://github.com/microsoft/vscode-eslint/issues/464#issuecomment-463676765
Can someone provide me with a GitHub repository that I can clone with steps on how to reproduce this?
Regarding using a different node version than the one that is shipped in VS Code. Use the eslint.runtime setting

Maybe this extention should parse jsconfig.json? Users need that file to make IDE understand aliases.

I think if this is necessary then eslint itself or the corresponding eslint plugin should read the jsconfig.json file.

I can provide a PR at this weekend. But if that solutions is OK for the author and community.
Project example with that bug.

I'm not sure about this solution. But here is the project with that bug.

Seems that it's eslint-plugin-import problem
https://github.com/benmosher/eslint-plugin-import/issues/496

I will close the issue. There is now a setting eslint.runtime which lets you point to a newer/different runtime than the one we ship in VS Code.


Clarify "rules of engagement" with arch board

PR: https://github.com/Azure/azure-sdk/pull/983

Link to recording (MSFT INTERNAL): https://msit.microsoftstream.com/video/85df0951-2cd5-4fa9-8f1d-2e3cbc9026c6

By default, aspire to preview every API change, but exceptions to go directly to GA may be made if sufficient business justification exists (for API changes). The Client library should be tested side-by-side with service features.

For API reviews:

Meeting 1 could be brownbag-style on how the feature works, meeting 2 a full API discussion, ad hoc meeting TBD pending a call from architects based on API diff
-- For example, if few small changes come out of meeting 2, third meeting may not be required or done via github issues or email (use azure-sdk-pr for issues tracking internal previews/communications)
Get in the habit of checking final API with system to detect breaking changes
Meeting 2 should be restricted to "controversial" pieces, not line-by-line changes; minimum 1 week to bake with comments closing 1 day before the scheduled meeting

SLA on issues:

Whenever an APIview is given, architects need a minimum of 5 working days to review/comment
Discussion at a higher level is how to handle SLA on Sev1 Azure changes, for example


fix onnx backend cuda cases

Fix onnx backend cuda cases when there is not a gpu device.

@nudles urgent to merge


Not able to define different usage for each links

Problem
I am planning to create this weathermap, example i am taking four nodes and creating two links between them, example from node a to node b and from node c to node d.

then i have elasticsearh index which contains information of outBps and InBps then i create a query which will pull outBps and InBps for each link,

so when i am trying to select this created query on the right side panel of A side and Z side it is not able to differentiate between these query and eventually gives me the same output

Expected behavior
i should be able to select different usage from the queries of different links

like this how is this achievable
VERSIONS! (please complete the following information):

Browser [firefox]
Grafana Version [8.5.11]
Plugin Version [0.3.3]
Additional (Elasticsearch is being used as a data source)

so please help screen shot is attached, Thank you

if you can help with some mock examples of this weathermap collecting data from ealsticsearch?

Yes, if the queries have the exact same name they're going to be selected as the same. I can't really do much about this since there is no other better way to track queries unfortunately. Are you able to just add a some pattern in the "Alias" field of Elasticsearch to make the queries differently named?


publish npm?

Hi, would you mind publishing this loader to npm?

Hi Bline,
Please take back control over the NPM package.
I've added you as author so you should be able to publish it.
Thanks!


Fixed the alignment of lets explore button

issue #475

@Vanshiii203 What have you changed?

@Vanshiii203 What have you changed?

I have changed the alignment of let's explore button
Before -

After -

@Vanshiii203 but it's not reflected in the deploy ppreview. let me check again.


godep restore tries to mess with packages in $GOROOT

The following happens in a project in $GOPATH
$ godep restore
# cd /usr/lib/go/src/golang.org/x/net/html; git pull --ff-only
fatal: Not a git repository (or any of the parent directories): .git
# cd /usr/lib/go/src/golang.org/x/net/html; git checkout 1dfe7915deaf3f80b962c163b918868d8a6d8974
fatal: Not a git repository (or any of the parent directories): .git
godep: restore: exit status 128
# cd /usr/lib/go/src/golang.org/x/tools/go/vcs; git pull --ff-only
fatal: Not a git repository (or any of the parent directories): .git
# cd /usr/lib/go/src/golang.org/x/tools/go/vcs; git checkout 27e692e6ec36d8f48be794f32553e1400c70dbf2
fatal: Not a git repository (or any of the parent directories): .git
godep: restore: exit status 128

Everything in $GOROOT has been installed by my distro's package manager, so I don't want to mess with it manually in any way. IMO, godep should leave everything in $GOROOT alone.

How do I reproduce this?

(I am assuming your GOROOT is /usr/lib/go; you can check this by running go env.)
Godep tries to update packages on disk wherever they are found; if they are not found, it downloads them to the first entry in GOPATH, just like 'go get'.
I would guess that /usr/lib/go/src/golang.org/x/net/html was created somehow outside of godep, and now that it exists, godep finds the package there and tries to update it. Or perhaps the first item in your GOPATH is /usr/lib/go—if so, you should remove it. The value of GOROOT should not be in GOPATH.

@kr
Right, GOROOT is /usr/lib/go and the package via the package manager of my distro. This means no other utility should try to touch that.
My suggestion remains that godep skips every package found under GOROOT.
If the same package is found in GOROOT and GOPATH it should prefer the one from GOPATH since that's what can be controlled by the user.

Commands 'godep restore' and 'godep get' follow the same rules as the go tool for locating and downloading packages. That will remain the case. (In fact, godep doesn't download any packages itself, it just runs the go tool to do that work. Any behavior change there would have to happen in the go tool.)
You should remove the value of GOROOT from your GOPATH, if it's there. I also recommend you delete golang.org/x/net/html from your GOROOT, along with any other packages in GOROOT that aren't part of the standard library. That should get you back up and running, so that both 'godep restore' and 'go get' work properly for all packages.

@kr
This is the folder arch for my development.  Please let me know what is the expected method for using godep.
~/go
~/go/src/company_repo/project
~/go/src/company_repo/project/subprojA
~/go/src/company_repo/project/subprojB
~/go/src/company_repo/project/Godeps
~/go/src/company_repo/project/Godeps/src
~/go/src/company_repo/project/Godeps/src/workspace
Since we are more than one developer, we do not maintain any source in our ~/go/src other than our company_repo.
All the golang sources are vendored into Godeps/src/workspace by using godep
Now the problem is, when I want to now add another dependency, I’m now forced to do the following:
godep restore
godep get  or go get  (both are same)
godep save ./...
Wouldn’t it be easier if godep get will directly save it into Godeps/src/workspace
If that is not the intention of godep, then what is the correct way to make the above work without the need to godep restore.

@sundarv85 rewrite your import paths (the -r option) is the way to work around having to use godep restore for the common cases (you will still need to do this before a godep update though).


PR combining all new commands

Codecov Report

Merging #154 (b4b0889) into develop (1f871a0) will decrease coverage by 2.13%.
The diff coverage is 14.15%.

@@             Coverage Diff             @@
##           develop     #154      +/-   ##
===========================================
- Coverage    32.74%   30.61%   -2.14%
===========================================
  Files           43       43
  Lines         3329     3257      -72
===========================================
- Hits          1090      997      -93
- Misses        2136     2170      +34
+ Partials       103       90      -13

Flag
Coverage Δ

unittests
30.61% <14.15%> (-2.14%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/codeconfig/codeconfig.go
9.14% <0.00%> (ø)

pkg/project/config.go
0.00% <0.00%> (ø)

pkg/provider/pulumi/aws/lambda.go
71.15% <ø> (ø)

pkg/provider/pulumi/azure/azure.go
7.65% <0.00%> (-2.09%)
:arrow_down:

pkg/provider/pulumi/azure/containerapp.go
0.00% <ø> (ø)

pkg/project/types.go
8.64% <8.64%> (ø)

pkg/provider/pulumi/aws/aws.go
61.53% <12.50%> (-1.90%)
:arrow_down:

pkg/build/build.go
36.36% <66.66%> (ø)

pkg/project/function_helpers.go
52.63% <66.66%> (ø)

pkg/project/options.go
80.64% <80.64%> (ø)

... and 2 more

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 1f871a0...b4b0889. Read the comment docs.


Missing License Information

Hi @nikolaik
I see that there is no open source LICENSE information in the repository, is that on purpose or would you like to introduce a license doc ?
Thanks

Went with MIT, see 010db40455491b7f6ffc6efda7bfbb34ffe96fac


Fitbit Weight Data

I'm trying to get Weight Data from fitbit, but the authenticator does not include weight in the scope of the request url. The list of available scopes is here.

I just realized this isn't covered in the wiki but you can add "custom" scopes to the request either through the request type itself or just as a string. You have to add them as a part of the OAuth2Web object before you get the authorization Uri from it. In your case what you would want is the following:
OAuth2Web<Fitbit> oauth = new OAuth2Web<Fitbit>(
    clientId,
    clientSecret,
    callbackUri);

Uri fitbitLoginEndpoint = await oauth
    .AddScope("weight")
    .GetAuthorizationUriAsync()
    .ConfigureAwait(false);

And I just realized after typing this that all of the Fitbit scopes weren't in the current build. So I updated the swagger document and published a new NuGet package 1.3.9. I haven't tested it because I don't have any Fitbit devices that utilize the "weight" scope but it should work.