Two makers (eslint/flow) can cause error: E885: Not possible to change sign neomake_err

Using two makers (eslint and flow) sometimes results in inability to move cursor up/down without generating errors that need to be dismissed. Workaround is to switch to using just one maker.
I think it might be because flow tries to add markers to more than just one file even when you just run NeomakeFile?
Expected behavior
It should be possible to press j and k to move your cursor down and up without having to dismiss errors from vim on each keypress.
Steps to reproduce
autocmd! BufWritePost *.js,*.jsx silent NeomakeFile
autocmd! BufWinEnter *.js,*.jsx silent NeomakeFile

My vimrc
I've not come up with a reliable way to reproduce it, but it seems to be triggered when you switch into a different buffer (via C-w h for example) when there's cross-file issues from flow and also local lint errors. Could be a timing issue?
In most cases writing the current file solves it (by triggering NeomakeFile?), but if not then just switching to using just one maker solves it (e.g. via :let g:neomake_javascript_enabled_makers=['flow'])
Here's the errors with verbose=3:
Neomake [156708.084]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Neomake [156709.452]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err

Output of the ":verb NeomakeInfo" command
Neomake debug information
Async support: 1
Current filetype: javascript
Enabled makers
For the current filetype (with :Neomake): ['eslint', 'flow']
NOTE: you can define g:neomake_javascript_enabled_makers to configure it (or b:neomake_javascript_enabled_makers).
For the project (with :Neomake!): []
NOTE: you can define g:neomake_enabled_makers to configure it.
Settings
g:neomake_coffee_enabled_makers = ['coffeelint']
g:neomake_list_height = 5
g:neomake_javascript_enabled_makers = ['eslint', 'flow']
g:neomake_verbose = 3
g:neomake_ruby_rubocop_args = ['--format', 'emacs', '-D']
g:neomake_cjsx_enabled_makers = ['coffeelint']
g:neomake_jsx_enabled_makers = ['eslint', 'flow']
g:neomake_ruby_enabled_makers = ['rubocop']

shell: /bin/zsh
shellcmdflag: -c
Windows: 0

:version

NVIM 0.1.5
Build type: RelWithDebInfo
Compilation: /usr/local/Library/Homebrew/shims/super/clang -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/build/config -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/src -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/usr/local/opt/gettext/include -I/usr/include -I/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/build/src/nvim/auto -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/build/include
Compiled by<EMAIL_ADDRESS>
Optional features included (+) or not (-): +acl   +iconv    +jemalloc +tui
For differences from Vim, see :help vim-differences

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.1.5/share/nvim"

:messages
(trimmed)
"src/lib/integrations.js" 86L, 2121C written
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
"src/components/ConnectIntegration.js" 93L, 2219C written
Error detected while processing function neomake#MakeHandler[44]..<SNR>126_RegisterJobOutput[25]..neomake#ProcessCurrentWindow[7]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err

E121: Undefined variable: g:neomake_verbose
E488: Trailing characters
Neomake [156708.084]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Neomake [156709.452]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
NVIM 0.1.5
Build type: RelWithDebInfo
Compilation: /usr/local/Library/Homebrew/shims/super/clang -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/build/config -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/src -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/deps-build/usr/include -I/usr/local/opt/gettext/include -I/usr/include -I/usr/include -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/build/src/nvim/auto -I/tmp/neovim-20160826-5051-avofzl/neovim-0.1.5/build/include
Type  :quit<Enter>  to exit Nvim
Neomake [156974.035]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Neomake [156978.369]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err
Neomake [156980.356]: Placing sign: sign place 5003 name=neomake_err buffer=5
Error detected while processing function neomake#CursorMoved[4]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   34:
E885: Not possible to change sign neomake_err

See also https://github.com/neomake/neomake/issues/60#issuecomment-260666193

Same problem
.vimrc
let g:neomake_javascript_enabled_makers = ['eslint', 'flow']

:messages
Error detected while processing function <SNR>112_vim_exit_handler[15]..<SNR>112_exit_handler[38]..<SNR>112_RegisterJobOutput[23]..neomake#signs#PlaceVisibl
eSigns[10]..neomake#signs#PlaceSign:
line   30:
E885: Not possible to change sign neomake_err
Error detected while processing function <SNR>112_cursormoved_delayed_cb[2]..neomake#CursorMoved[5]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#Plac
eSign:
line   30:
E885: Not possible to change sign neomake_err
E885: Not possible to change sign neomake_err
E885: Not possible to change sign neomake_err

Same problem as well
.vimrc
let g:neomake_c_enabled_makers = ['gcc', 'clang']

:messages
Error detected while processing function <SNR>97_cursormoved_delayed_cb[2]..neomake#CursorMoved[5]..neomake#signs#PlaceVisibleSigns[10]..neomake#signs#PlaceSign:
line   30:
E885: Not possible to change sign neomake_err

A reproducible test case would help here.
Either as a Vader test directly, or describing it manually.

I have this bug too and find it very difficult to reproduce reliably.
After thinking through about flow, I realise flow is actually a directory maker or project maker instead of the "normal" maker.
After I switched flow as a directory maker, I don't seems to have this bug anymore (cross my fingers) along with a faster and more stable/consistent performance (sometimes my flow errors doesn't show up in a project but it shows up in another).
Hence, my configuration to use flow (and eslint_d) via directory maker:
Plug 'neomake/neomake'
let g:neomake_flow_maker = {
        \ 'exe': 'flow',
        \ 'args': ['--from=vim', '--show-all-errors'],
        \ 'errorformat': '%EFile "%f"\, line %l\, characters %c-%m,%C%m,%Z%m',
        \ 'postprocess': function('neomake#makers#ft#javascript#FlowProcess')
        \ }
let g:neomake_eslint_d_maker = {
        \ 'exe': 'eslint_d',
        \ 'args': ['%', '-f', 'compact'],
        \ 'errorformat': '%E%f: line %l\, col %c\, Error - %m,' .
        \ '%W%f: line %l\, col %c\, Warning - %m'
        \ }

let customFTSettings = ['js']
" Run neomake for all file types except those in customFTSettings (eg. JS)
autocmd BufWritePre * if index(customFTSettings, &ft) < 0 | Neomake
" Run neomake! for JS files
autocmd! BufWritePost,BufEnter *.js Neomake! flow eslint_d

I find it quite ugly to have so much configuration code
but I can't seems to find out how to draw the normal makers for Neomake! usage.
Is there a way to define directory makers for neomake! so that it is a default for users?
Any guide will be good and I can submit a PR for this.

I totally missed the entire existence of directory makers.

@benjie yeah me too and I have living with the problem for quite a while.
I agree that there could be more support for directory maker such as a separate folder to contain makers for directory (currently it looks like it is only for normal makers).
Since users might be using a combination of directory makers and normal makers, I am also not sure if directory makers can be combined with normal makers such as:
autocmd! BufWritePost,BufEnter *.js Neomake! flow
autocmd! BufWritePost,BufEnter *.js Neomake eslint_d

@stevenyap
That should work in general, since they use different lists for errors and counts.
But it might indeed trigger some bug.
I am heavily refactoring stuff currently, so this might get fixed on the way.  I won't investigate into this issue until then (i.e. the autolinting landed - which it depends on 2-3 other branches/PRs).


Request from T0my: Status buttonkey isn't being saved in the profiles, everytime I open tool I need to configure/add again the buttonkey

can proceed to close this request.


[DRAFT] Extend test script for use with Windows/Visual Studio

This pull request, makes testing on the Windows platform slightly easier.  It makes the following changes:

Changes the test script tests\scripts\run-test-scripts.pl to allow the test suite directory to be specified, so the test directory containing the test suite executables doesn't have to be the current directory. This useful when working with Visual Studio project files, that place the test suite executables inside Debug and Release directories.
Changes the test script tests\scripts\run-test-scripts.pl when glob'ing the test suites, to exclude on Windows all files that don't end with .exe to allow build intermediate files to be rejected.
Extends the scripts\config.pl script with a new option of windowsfull to allow all configurations which work or are compatible with Windows, to be enabled in one go, for the sake of testing.

Submitted for review.

Closing as this needs work & has become inactive.


sometimes crashes when throttle fails to convert to int

Don't know yet what is received from Arduino but Python interface throws exception that it can't convert that value to a number and sometimes crashes when throttle is moved.

fix the problem
handle the exception

The actual input content is:
[P0=]

so there is no value at all to convert

Fixed (by ignoring missing data) in a71bc3b


[Question] Get the return value of a command in home assistant

Hello,
I am using this integration in home assistant. I would like for the automation to trigger based on the Power state change.
Manually calling the endpoint "com.webos.service.tvpower/power/getPowerState" works. I see a debug log in the event log whose payload tells me the power state.
Example.
2022-05-22 18:57:23 DEBUG (MainThread) [aiowebostv] recv(<IP_ADDRESS>): {"type":"response","id":51,"payload":{"returnValue":true,"state":"Screen Saver"}}
Home assistant call_service API does not provide a return value so I can't get this output anywhere in home assistant except the debug logs.
I see in the code that we are subscribed to the power state changes and I also do see a log in the debug logs when TV power state changes. Example Active -> Screen Saver
Is there a way to call the getPowerState endpoint and read the data coming back from the TV?
Regards,
Vaibhav

Because I didn't get any response on this, I did it with Node-Red add-on in HA.
Its still not Publish-Subscribe model. I need to invoke the flow in Node-Red to get the latest PowerState of the TV. I have set up a automation in HA to do that so it works OK.
I use node-red-contrib-lgtv 1.1.0.
Use the config node from this to create a config for your LG TV. You need IP address of your TV. I recommend using static IP so your configuration is guaranteed to work.
Set up Node-Red with Home Assistant if you haven't already so you can use the output of the Node-Red flow inside Home Assistant.
The flow to test the API call to LG TV and see the output in the debug window in Node-Red. Copy this and paste in Node-Red import window.

[{"id":"a87c1f750a91bdbd","type":"inject","z":"0ae12e835f14d4d6","name":"","props":[],"repeat":"","crontab":"","once":false,"onceDelay":0.1,"topic":"","x":570,"y":200,"wires":[["d615edf81c390e96"]]},{"id":"d615edf81c390e96","type":"function","z":"0ae12e835f14d4d6","name":"getPowerState","func":"return {\n    topic: 'ssap://com.webos.service.tvpower/power/getPowerState',\n    payload: {}\n};","outputs":1,"noerr":0,"initialize":"","finalize":"","libs":[],"x":820,"y":200,"wires":[["8469f7793527dc01"]]},{"id":"8469f7793527dc01","type":"lgtv-request","z":"0ae12e835f14d4d6","tv":"","name":"requestPowerState","x":1030,"y":200,"wires":[["e0be6dfa88eafaba"]]},{"id":"e0be6dfa88eafaba","type":"debug","z":"0ae12e835f14d4d6","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"payload.state","targetType":"msg","statusVal":"","statusType":"auto","x":1290,"y":280,"wires":[]}]


add ChatGLM3 to support Chinese RAG

Includes：

fix a error on conversation history of llm chat notebook
switch chatglm2 to chatglm3 for better RAG support
update the RAG template of Chinese LLM
add stop token for remote model on rag chat notebook

@eaidova could you help to review, thanks a lot.


Intended behaviour?

The following code produces this config file when ran without a existing config file,
as im defining a default value for the boolean in the getBoolean method,
shouldn't wildCanFlee be set to this default value when saved?
# General settings for the world
general {
    # Wild Pokemon can try and flee from battles
    wildCanFlee=null
}

		HoconConfigurationLoader configurationLoader = HoconConfigurationLoader.builder().setFile(new File("newpixelmonconfig.hocon")).build();
		CommentedConfigurationNode mainNode = configurationLoader.load();

		CommentedConfigurationNode generalNode = mainNode.getNode("general");
		generalNode.setComment("General settings for the world");
		allowWildPokemonFleeing = generalNode.getNode("wildCanFlee").setComment("Wild Pokemon can try and flee from battles").getBoolean(false);

		configurationLoader.save(mainNode);

Also is there a way to make configurate not sort the map before writing?, alphabetical order kinda ruins internal ordering

alphabetical sorting is impossible to fix with the hocon lib being how it is.
however, the defaults thing is something that can be fixed, probably with a ConfigurationOptions field

The defaults issue has been fixed in 760cf6c

https://github.com/lightbend/config/pull/557 looks promising with changing the key sort order


CI deployment

Let's get the api in some sort of a tagged deployment cycle mabye

No tags, deployment to heroku enabled from master after CI tests pass.


Implementing the stop-propagation helper

Closes #9.

Thanks for taking a look, @rwjblue! I've added the test for capture. Once we have ember-on-event-helpers (or whatever it may be called), I'll be happy to move this over there.

Hey @gmurphey, thanks so much for implementing this! I would love, if you could reopen this PR against ember-event-helpers. ❤

Closing in favor of buschtoens/ember-event-helpers#4.


Parameters are null when AFJSONRequestSerializer is used for a POST

I'd like to add a value in Header for Authorization so that I'm using AFJSONRequestSerializer before my POST. Here is my code:
AFHTTPRequestOperationManager *manager = [AFHTTPRequestOperationManager manager];
manager.requestSerializer = [AFJSONRequestSerializer serializer];
[manager.requestSerializer setValue:self.apikey forHTTPHeaderField:@"Authorization"];

[manager POST:url parameters:@{REST_PARAM_USERID:    userId}
      success:^(AFHTTPRequestOperation *operation, id responseObject) {

} failure:^(AFHTTPRequestOperation *operation, NSError *error) {

}];

But  the failure is always called. I found the parameters I passed to server is always null although it's not in my Xcode debug. I don't know what's the reason. Can anybody help? Thanks

When My service running on Local server that time I wrote down AFJSONRequestSerializer


[CHORE] [Catalogs] [Delta Lake] Add test coverage for Delta Lake reads on Azure.

This PR adds test coverage for reading Delta Lake tables from Azure Blob Storage (or more specifically, Azure Data Lake Storage gen2 on ABS). Since the ABS emulating server only runs on Windows, we run a session-scoped Docker container containing the cross-platform Azurite emulator. We use a Docker container rather than a background process in order to not introduce another heavy dev dependency (node.js/npm).
Closes #1971

@samster25 @jaychia Note that starting the azurite container + the contained mock server takes ~15 seconds(!), although this is amortized over all tests using the fixture since it is session-scoped. This roughly doubles the time taken for the Delta Lake test suite, from 25 seconds to 50 seconds, but I think that this is well worth the coverage.
If we ever want to eliminate this 15 second baseline, we could make node.js/NPM a dev dependency, where the mock server fixture assumes that the azurite-blob program is installed and is on the PATH, and skips the tests with the appropriate reason if not, including directions to install node.js/NPM and to npm install -g azurite.


Add editing

This adds the option to make the grid editable, similar to a spreadsheet.

wonderful, thanks for the feedback @mattrothenberg!

🚢 :shipit:


fix: PostgreSQL compatibility issues

Fixes a few PostgreSQL compatibility issues.

Fix documented here: https://github.com/dbt-labs/dbt-utils/issues/192#issuecomment-761029060
Another issue regarding long identifiers. Documented here: https://github.com/dbt-labs/dbt-utils/issues/629. The solution is just a temporary hack which makes the identifier shorter.

Did not meant to open a PR against this project. Was only intending to open one in my own fork.


Error after newest SteamOS update

Hey there, a few extensions have been broken since SteamOS updated to 3.5.19 and this is one of them
The error decky displays when trying to view the plugin says
TypeError: Cannot read properties of undefined (reading 'Field')
    at eval (eval at importReactPlugin (http://localhost:1337/frontend/chunk-ecb27f3b.js:6994:43), <anonymous>:831:68)
    at eval (eval at importReactPlugin (http://localhost:1337/frontend/chunk-ecb27f3b.js:6994:43), <anonymous>:1129:3)
    at PluginLoader.importReactPlugin (http://localhost:1337/frontend/chunk-ecb27f3b.js:6994:43)
    at async PluginLoader.importPlugin (http://localhost:1337/frontend/chunk-ecb27f3b.js:6969:17)

Followed by
Loading DeckSettings caused an exception as shown above. This usually means that the plugin requires an update for the new version of SteamUI. Check if an update is present or evaluate its removal in the Decky settings, in the Plugins section.

+1

An update has already been submitted to the store to resolve this issue, but it has not yet finished going through testing/approval.

Same issue here. Looking forward to the update being publicly released. Thanks!

This plugin (v1.3.0-1) doesn't seem to work on my deck running SteamOS v3.5.19, are we waiting for a version newer than 1.3.0 or should that version work for me?

Looks like the update PR has been approved and merged but the build action failed to upload due to a duplicate version number. I'm not sure whose responsibility it is to fix that.
https://github.com/SteamDeckHomebrew/decky-plugin-database/actions/runs/10269774024
As an aside the plugin seems to have disappeared from the store now entirely. I've been trying to get to the bottom of exactly why it's missing but the documentation for the store isn't great


[podresapi 1/3] agent,config,helm: make agent runtime configurable.

Add configuration for the agent itself. Currently the only control enables access to Node Resource Topology Custom Resources and it is on by default. Update the agent to honor its configuration.

/cc pfl

/cc @pfl


[PLUGIN-1715] Implement retry feature for BigQuery Execute Plugin

Implement retry feature for BigQuery Execute Plugin
Jira : PLUGIN-1715

In general internal errors should be retried.
Do an exponential back-off retry on errors with reason jobBackendError , jobInternalError.
Backoff requirements (exponential from 1 to 32 seconds with a multiplier of 2)

Code Changes

Add a new set that contains all reason we should retry on.
A new custom exception that is raised when the error should be retried.
A new function to generate the exponential back- off retry policy
A new dependency is added dev.failsafe that handles the retry logic based on the exception raised.

Unit Tests

Use mocks to test the behavior of the retry.

Changes Squashed !

Thanks @tivv , i have made the changes and squashed them , hopefully you can approve this now.
I will get this QA tested before merging  👍

New changes include

Reverted changes in docs/BigQueryExecute-action.md as the UI fields are hidden.
Removed filter retryOnBackendError with condition retryOnBackendError == true as this will not be required now.
Made all the ui fields hidden
Retry On Backend Error is TRUE by default , on widget and backed.

Resolved conflicts caused by PR: https://github.com/data-integrations/google-cloud/pull/1344 in bigquery/action/BigQueryExecute.java


Update sloth to version 1.7, added appcast

Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Thank you, as always.


Hover Effect [Feature Request]

Description
Example from vuetify
Why Oruga need this feature
To grow the functionality of oruga

Hey, @zhaolinlauI am interested in this issue..
please assign to me this issue.

Hey, @zhaolinlauI am interested in this issue.. please assign to me this issue.

You just open a pull request... and @jtommy will be going to review it

what should I do please suggest me?

Hi, @KetanParmar02 . Are you still working on this?


Visual Studio 2019 uses the actual version for the ARP table now instead of the build version

Since the version number in the ARP table used the actual version instead of the build version winget might think that this is for an older version of Visual Studio 2019 since the build version for the previous release was 16.10.31410.357 and the actual version for the current version of Visual Studio 2019 is 16.10.3
Same thing also happened with build tools (And most likely other versions that the Visual Studio Installer has)

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Did this change recently? I could've sworn VS used to used the build number in the ARP table.

Yes it seems to have been changed in the last release and the version displayed is the actual version on both the computer where I updated and in the sandbox where I installed the newer version.

If this only happens for the most recent version I wonder why. Seems like a regression. We'd have to move all of the other ones for it to work but then they won't match correctly.
I wonder if we should wait on microsoft/winget-cli#1073 to do this so we can put the shorter version number in the PackageVersion box and just set the version in the programs array to whatever it identifies itself as in the ARP table.
Another moderator got an opinion? @OfficialEsco @ItzLevvie

I did try to install an older version in a sandbox using winget after accidentally starting a install for the newest version but it installed the newest version and it seems like installations still on an older version still had the build version in the ARP table based on when I had kept Apps and Features open while upgrading my installation

Yeah I see that on my other box that has the last build of VS Pro on it. This is odd.

Think you need to get a answer from @ms-jb or @lszomoru
@denelon contacted them to fix the VS Code version number and since then they have been updating a few things

I am managing the publishing pipeline for Visual Studio Code, and this seems to be a question related to Visual Studio.

@denelon Can you put a Blocking-Issue or something on this so it doesn't get merged until we can figure this out? Thanks!

I've added the "Blocking Issue" label. I've reached out to the Visual Studio team. They are holding for us to implement the package ownership feature to begin automated manifest publishing. We're expecting that feature in the next week or so to hit production. I'm not sure how soon they will be prepared to automate. I'll also ask about the version number changes in ARP.

Hey @msftbot, we're still waiting on stuff!

bump

@msftbot don't close this!

@jedieaston  I have confirmed with Visual Studio that the versioning change was intentional.  The PM that drove the decision is out of the office, so I do not know the rationale.
I think we should move forward with this being their pattern from here on out.

@coehlrich I did all of them and they are merged, if you want to close this.


Update deckgl to 8.1.5 version

The version 8.1.5 fixes the clipping issue

The build was failing due to the typescript version 3.9.2 has a bug https://github.com/microsoft/TypeScript/issues/38535
I've fixed the version 3.8.3 which is prior to 3.9.2.

TS version updated and pinned for all packages.
I've also updated all examples to use deckgl 8.1.5

@manmorjim Hey, I was also checking / updating the examples and we need to update ALL deck.gl components to 8.1.5 in package.json files (luma, geo-layers, etc) because otherwise the examples will fail

@manmorjim Hey, I was also checking / updating the examples and we need to update ALL deck.gl components to 8.1.5 in package.json files (luma, geo-layers, etc) because otherwise the examples will fail

All deckgl components were updated to 8.1.5 except lumaGL due to deckgl 8.1.5 uses lumaGL version 8.1.2.

Don't merge this please, I would mix this with my PR on examples

Done in https://github.com/CartoDB/toolkit/pull/110


46912 sqlcipher attachments

Closing this PR for now, awaiting the refactoring of the commits on top of the feature-encryption branch.


Remove packages which are not building on any platform

This is unreleaing the 4 packages that are continuously failing to build on all platforms.
https://discourse.ros.org/t/preparing-for-kinetic-sync-2020-07-21/15517/2

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/preparing-for-kinetic-sync-2020-07-21/15517/3


not support cert made by openssl?

i make cert and key by this way:

This is not websocket issue, your browser rejects cert. You need to explicitly trust it first somehow.
You might try to open websocket's address in browser directly, check browser's warning and follow up to your HTML page again. But it'll not work for all other users, this is why self-signed certs are good for testing only.
Real certs are free nowadays I just don't know if regulation in certain countries or US trade regulations could prevent users from obtaining them from sites like Letsencrypt.

OK! Your suggestion should be nice. At first, I'll make sure that my brower does not reject self-signed certs. Secondly, whether that works or not, I will get cert from Letsencrypt and have a try with it. Thank you!


expecting to read a single byte for receiving credentials but read zero bytes

打开APP之前：
➜  frida-ios-dump ./dump.py XXXX
Start the target app XXXX
unexpected error while probing dyld of target process
打开APP之后：
➜  frida-ios-dump ./dump.py XXXX
Start the target app XXXX
expecting to read a single byte for receiving credentials but read zero bytes

You are probably using a jailbreak that doesn't support Frida (like unc0ver). Try changing to the newest  Electra jailbreak

+1
8plus iOS 12.0.1
unc0ver b37

https://github.com/frida/frida-core/issues/120#issuecomment-447981159

+1 7Plus iOS 12.1.1
unc0ver b40

+1

@r3ggi do you know any others ways to dump app with unc0ver

+1
Unc0ver

You are probably using a jailbreak that doesn't support Frida (like unc0ver). Try changing to the newest Electra jailbreak

iPhone5s + iOS9.1

@r3ggi do you know any others ways to dump app with unc0ver

Yeah, try https://github.com/BishopFox/bfinject

I was able to decrypt iOS app with unc0ver (iOS 11.2.6, Frida 12.8.6)
According to the https://github.com/frida/frida-core/issues/120#issuecomment-469902599 the latest Frida (12.4+) supports unc0ver now. Try updating Frida through Cydia to the latest release.
You can also refer to this blog post to make sure you have not forgotten about anything.

Try to install "Open for ios 11". Or just launch app while executing manually.


Bug: It always automatically types what I previously typed in when I quit KeyCastr

As the video shows, I was typing some words in the text. After I quited KeyCastr, it automatically repeated what I had previously typed in, even if I did nothing.
https://github.com/user-attachments/assets/be49fb4e-9dda-4d8b-9080-434eef422f45

Hi, thanks for opening an issue about this with a video demonstration. This bug has been around for a while, referenced also in #72 .
I have been able to figure out what is causing this, but I'll need to think a bit about the best way to handle it. It should be fixed in the next release.


Make It an ACTUAL programming language.

Description
To make it a serious programming language, we'll need to add atleast barebones for a lang.

Static types bhai ye hai a = 0 to
bhai ye shabd hai a = "strings"
Runtime features
dekh $PATH for getting current PATH.
Add more data types.

Type
Syntax

String
shabd

Number
ginti

Array
dhara

Object
Could be determined by the class

Object oriented features like class(for very least) and inheritance(for mainstream langs)

Domain
Bhai Lang
Type of Contribution
Enhancement
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of bhai-lang.

mai ispe contribute karu bhai??

mai ispe contribute karu bhai??

If you are sure in every way(like optimization, size, complexity) then just go ahead 😋

added a PR for Funda, it will introduce functional programming
you can check it in preview build https://6234ac37ec8e100008f579b5--bhailang-preview.netlify.app/#playground

added a PR for Funda, it will introduce functional programming
you can check it in preview build https://6234ac37ec8e100008f579b5--bhailang-preview.netlify.app/#playground

Bhai API toh batao😅. Kaise implementation hoga function ka?

added a PR for Funda, it will introduce functional programming
you can check it in preview build https://6234ac37ec8e100008f579b5--bhailang-preview.netlify.app/#playground

⭐ bhai you are awesome!
Lekin homepage me bhi toh funda daal do.

Should I close this issue now?

I am working on adding class

How does this look??
these features are built on top of the changes in the last PR, I can add  a PR once the last PR is accepted

How does this look?? these features are built on top of the changes in the last PR, I can add a PR once the last PR is accepted

Bhai, you are absolutely talented person. And the API is great. You earned my issue my first badge help wanted.
Absolutely genuine 🥲

You are just awesome!

How does this look?? these features are built on top of the changes in the last PR, I can add a PR once the last PR is accepted

Hey wait. 😐 Where is new keyword implementation?
What you have implemented is just sane as getting output of a function. I think just refactor and everything will be just fine.🥲

new is not necessary for this (take python for example), by default it creates a new object, and it's different than a function, functions will not have methods of it's own or it's own data members,
see the  example below:

new is not necessary for this (take python for example), by default it creates a new object, and it's different than a function, functions will not have methods of it's own or it's own data members,
see the example below:

I've only worked with JS/TS and C#. Both of them use new so I thought same for bhailang. But python's implementation is also good.

naya keyword can be used

No need for unintended naya if @thecodacus has already implemented it.

#257  Added a pr for class feature

#257 Added a pr for class feature

My highschool board exams are going on. It will over on 12 Apr. Till then, keep making improvements.🥲

#257 Added a pr for class feature

If you have time: Are you an IITian?


Pin sphinx < 5.2.0

Summary
Latest version of Sphinx 5.2.0 is failing documentation. Pin for now
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.708%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
642

Relevant Lines:
848

💛  - Coveralls


[USB host] failed to run on S3, works fine on S2 (IDFGH-6402)

Here is simple usb host app i am using to test plug in/out
https://gist.github.com/chegewara/8f497fe8d8cf6b4597ea74f601307ff4
This app is working perfectly fine on S2, but it fails on S3
I (29) boot: ESP-IDF v5.0-dev-676-g5c33570524-dirty 2nd stage bootloader
I (29) boot: compile time 13:25:17
I (29) boot: chip revision: 0
I (33) boot.esp32s3: Boot SPI Speed : 80MHz
I (38) boot.esp32s3: SPI Mode       : DIO
I (42) boot.esp32s3: SPI Flash Size : 2MB
I (47) boot: Enabling RNG early entropy source...
W (52) bootloader_random: RNG for ESP32-S3 not currently supported
I (59) boot: Partition Table:
I (63) boot: ## Label            Usage          Type ST Offset   Length
I (70) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (78) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (85) boot:  2 factory          factory app      00 00 00010000 00100000
I (93) boot: End of partition table
I (97) esp_image: segment 0: paddr=00010020 vaddr=3c020020 size=096c8h ( 38600) map
I (112) esp_image: segment 1: paddr=000196f0 vaddr=3fc90fe0 size=027b8h ( 10168) load
I (116) esp_image: segment 2: paddr=0001beb0 vaddr=40374000 size=04168h ( 16744) load
I (126) esp_image: segment 3: paddr=00020020 vaddr=42000020 size=1cf78h (118648) map
I (152) esp_image: segment 4: paddr=0003cfa0 vaddr=40378168 size=08e6ch ( 36460) load
I (160) esp_image: segment 5: paddr=00045e14 vaddr=50000000 size=00010h (    16) load
I (166) boot: Loaded app from partition at offset 0x10000
I (166) boot: Disabling RNG early entropy source...
W (169) bootloader_random: RNG for ESP32-S3 not currently supported
I (187) cpu_start: Pro cpu up.
I (188) cpu_start: Starting app cpu, entry point is 0x40375104
0x40375104: call_start_cpu1 at /home/chegewara/esp/master/components/esp_system/port/cpu_start.c:164

I (166) cpu_start: App cpu up.
I (202) cpu_start: Pro cpu start user code
I (202) cpu_start: cpu freq: 160000000 Hz
I (202) cpu_start: Application information:
I (205) cpu_start: Project name:     usb-host-test
I (210) cpu_start: App version:      1
I (215) cpu_start: Compile time:     Dec 11 2021 13:25:10
I (221) cpu_start: ELF file SHA256:  1fc60f079b2b978b...
I (227) cpu_start: ESP-IDF:          v5.0-dev-676-g5c33570524-dirty
I (234) heap_init: Initializing. RAM available for dynamic allocation:
I (241) heap_init: At 3FC941A0 len 0004BE60 (303 KiB): D/IRAM
I (247) heap_init: At 3FCE0000 len 0000EE34 (59 KiB): STACK/DRAM
I (254) heap_init: At 3FCF0000 len 00008000 (32 KiB): DRAM
I (260) heap_init: At 600FE000 len 00002000 (8 KiB): RTCRAM
I (267) spi_flash: detected chip: generic
I (271) spi_flash: flash io: dio
W (275) spi_flash: Detected size(8192k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (289) sleep: Configure to isolate all GPIO pins in sleep state
I (295) sleep: Enable automatic switching of GPIO sleep configuration
I (303) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (383) : install status: 0
I (383) : client register status: 0
usb_host_lib_handle_events event_flags: 0
E (693) HUB: Short device descriptor transfer failed, USB_TRANSFER_STATUS_ERROR

assert failed: enum_stage_cleanup_failed hub.c:453 (dequeued_enum_urb == p_hub_driver_obj->single_thread.enum_urb)

Backtrace:0x40375a62:0x3fcf41100x40378f85:0x3fcf4130 0x4037f4fd:0x3fcf4150 0x4200a264:0x3fcf4270 0x4200a39b:0x3fcf4290 0x4200a6e2:0x3fcf42b0 0x42006120:0x3fcf42d0 0x420059e3:0x3fcf42f0 0x4201cda9:0x3fcf4330 0x4037bc4d:0x3fcf4350
0x40375a62: panic_abort at /home/chegewara/esp/master/components/esp_system/panic.c:403

0x40378f85: esp_system_abort at /home/chegewara/esp/master/components/esp_system/esp_system.c:123

0x4037f4fd: __assert_func at /home/chegewara/esp/master/components/newlib/assert.c:85

0x4200a264: enum_stage_cleanup_failed at /home/chegewara/esp/master/components/usb/hub.c:453 (discriminator 1)

0x4200a39b: enum_handle_events at /home/chegewara/esp/master/components/usb/hub.c:515

0x4200a6e2: hub_process at /home/chegewara/esp/master/components/usb/hub.c:652

0x42006120: usb_host_lib_handle_events at /home/chegewara/esp/master/components/usb/usb_host.c:490

0x420059e3: app_main at /home/chegewara/demos/tests/usb-host-test/build/../main/usb-host-test.c:47 (discriminator 15)

0x4201cda9: main_task at /home/chegewara/esp/master/components/freertos/FreeRTOS-Kernel/portable/port_common.c:127 (discriminator 2)

0x4037bc4d: vPortTaskWrapper at /home/chegewara/esp/master/components/freertos/FreeRTOS-Kernel/portable/xtensa/port.c:130

Im not sure what may be the problem, is it chip revision 0 or maybe pins are not configured properly:
I (289) sleep: Configure to isolate all GPIO pins in sleep state
I (295) sleep: Enable automatic switching of GPIO sleep configuration

esp-idf commit: 5c33570524118873f7bd32490c7a0442fede4bf8
board: esp32 S3 wroom 1, N8R2
Thanks

Looks like its pins misconfig. Adding prevent crashing, but didnt solve problem completely:
void app_main(void)
{
    gpio_reset_pin(GPIO_NUM_19);

@chegewara It's probably something to do the difference in how the USB PHY is setup on the S2 vs the S3 (there's an internal MR pending review already). There are an extra couple of bits to set on the S3 in order to switch the USB PHY from being connected to the USB Serial JTAG Controller to the USB OTG Controller.
There's a USB PHY driver that should already abstract away these differences, but that USB Host stack hasn't been updated to use the USB PHY driver (will be part of the MR). In the mean time, you can try adding modifying usbh_ll_internal_phy_conf() to the following:
#include "soc/rtc_cntl_struct.h"
static inline void usbh_ll_internal_phy_conf(usb_wrap_dev_t *hw)
{
    //Enable internal PHY
    hw->otg_conf.pad_enable = 1;
    hw->otg_conf.phy_sel = 0;
    // phy_sel is controlled by the following register value
    RTCCNTL.usb_conf.sw_hw_usb_phy_sel = 1;
    // phy_sel=sw_usb_phy_sel=1, USB_OTG is connected with internal PHY
    RTCCNTL.usb_conf.sw_usb_phy_sel = 1;
    //Set pulldowns on D+ and D-
    hw->otg_conf.pad_pull_override = 1;
    hw->otg_conf.dp_pulldown = 1;
    hw->otg_conf.dm_pulldown = 1;
}

Thanks @Dazza0
I thought it may be related to JTAG, as it is something extra on S3
Also thanks for that snippet, will check it, and im glad there is MR already, no rush i am just testing it on S3, no really needed yet

Ok, that seems to solove problem, even if USB host seems to be just a bit less stable on S3, but dont know why yet.
Thanks

Just to add my results, when I update usbh_ll.h to include the RTCCNTL configuration as suggested by @Dazza0 the code does indeed function, but as @chegewara mentioned it is somewhat unstable. I can have a completely successful OTG Device insertion and removal like this
create async task
I (7522) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (7522) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (29332) USBH: Device 1 gone
I (29332) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (29332) : DEVICE gone event
I (29362) : install status: 0
I (29362) : client register status: 0
I (33692) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (33692) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (36722) USBH: Device 1 gone
I (36722) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (36722) : DEVICE gone event
I (36752) : install status: 0
I (36752) : client register status: 0

or I can have Root port reset and reboot fails like this
create async task
I (2023) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (2023) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (4843) USBH: Device 1 gone
I (4843) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (4843) : DEVICE gone event
I (4873) : install status: 0
I (4873) : client register status: 0
E (6533) HUB: Root port reset failed
E (6783) HUB: Root port reset failed
I (7203) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (7203) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (9903) USBH: Device 1 gone
I (9903) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (9903) : DEVICE gone event
I (9933) : install status: 0
I (9933) : client register status: 0
I (12253) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (12253) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (15603) USBH: Device 1 gone
I (15603) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (15603) : DEVICE gone event
I (15633) : install status: 0
I (15633) : client register status: 0
E (17413) HUB: Root port reset failed

abort() was called at PC 0x4200b600 on core 0
0x4200b600: root_port_handle_events at C:/Users/spenc/esp/esp-idf/components/usb/hub.c:222

Backtrace:0x40375aba:0x3fce15a00x4037907d:0x3fce15c0 0x4037f43a:0x3fce15e0 0x4200b600:0x3fce1650 0x4200c0bd:0x3fce1670 0x420071c3:0x3fce1690 0x42005ef0:0x3fce16b0 0x4037bd0d:0x3fce16e0
0x40375aba: panic_abort at C:/Users/spenc/esp/esp-idf/components/esp_system/panic.c:402

0x4037907d: esp_system_abort at C:/Users/spenc/esp/esp-idf/components/esp_system/esp_system.c:121

0x4037f43a: abort at C:/Users/spenc/esp/esp-idf/components/newlib/abort.c:46

0x4200b600: root_port_handle_events at C:/Users/spenc/esp/esp-idf/components/usb/hub.c:222

0x4200c0bd: hub_process at C:/Users/spenc/esp/esp-idf/components/usb/hub.c:649

0x420071c3: usb_host_lib_handle_events at C:/Users/spenc/esp/esp-idf/components/usb/usb_host.c:490

0x42005ef0: client_async_seq_task(void*) at C:/Users/spenc/esp/esp32-usb-host/components/usb_host/host/usb_host.cpp:40

0x4037bd0d: vPortTaskWrapper at C:/Users/spenc/esp/esp-idf/components/freertos/port/xtensa/port.c:130

ELF file SHA256: e9a22024db2aa6f2

Rebooting...
ESP-ROM:esp32s3-20210327

I'm hitting this issue as well. #8122

Just to add my results, when I update usbh_ll.h to include the RTCCNTL configuration as suggested by @Dazza0 the code does indeed function, but as @chegewara mentioned it is somewhat unstable. I can have a completely successful OTG Device insertion and removal like this
create async task
I (7522) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (7522) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (29332) USBH: Device 1 gone
I (29332) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (29332) : DEVICE gone event
I (29362) : install status: 0
I (29362) : client register status: 0
I (33692) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (33692) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (36722) USBH: Device 1 gone
I (36722) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (36722) : DEVICE gone event
I (36752) : install status: 0
I (36752) : client register status: 0

or I can have Root port reset and reboot fails like this
create async task
I (2023) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (2023) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (4843) USBH: Device 1 gone
I (4843) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (4843) : DEVICE gone event
I (4873) : install status: 0
I (4873) : client register status: 0
E (6533) HUB: Root port reset failed
E (6783) HUB: Root port reset failed
I (7203) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (7203) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (9903) USBH: Device 1 gone
I (9903) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (9903) : DEVICE gone event
I (9933) : install status: 0
I (9933) : client register status: 0
I (12253) USB_HOST_CLIENT_EVENT_NEW_DEV: client event: 0, address: 1
I (12253) : device speed: USB_SPEED_FULL, device address: 1, max ep_ctrl size: 64, config: 1
config: 0[1], interface: 0[1], intf class: 255
E (15603) USBH: Device 1 gone
I (15603) USB_HOST_CLIENT_EVENT_DEV_GONE: client event: 1
W (15603) : DEVICE gone event
I (15633) : install status: 0
I (15633) : client register status: 0
E (17413) HUB: Root port reset failed

abort() was called at PC 0x4200b600 on core 0
0x4200b600: root_port_handle_events at C:/Users/spenc/esp/esp-idf/components/usb/hub.c:222

Backtrace:0x40375aba:0x3fce15a00x4037907d:0x3fce15c0 0x4037f43a:0x3fce15e0 0x4200b600:0x3fce1650 0x4200c0bd:0x3fce1670 0x420071c3:0x3fce1690 0x42005ef0:0x3fce16b0 0x4037bd0d:0x3fce16e0
0x40375aba: panic_abort at C:/Users/spenc/esp/esp-idf/components/esp_system/panic.c:402

0x4037907d: esp_system_abort at C:/Users/spenc/esp/esp-idf/components/esp_system/esp_system.c:121

0x4037f43a: abort at C:/Users/spenc/esp/esp-idf/components/newlib/abort.c:46

0x4200b600: root_port_handle_events at C:/Users/spenc/esp/esp-idf/components/usb/hub.c:222

0x4200c0bd: hub_process at C:/Users/spenc/esp/esp-idf/components/usb/hub.c:649

0x420071c3: usb_host_lib_handle_events at C:/Users/spenc/esp/esp-idf/components/usb/usb_host.c:490

0x42005ef0: client_async_seq_task(void*) at C:/Users/spenc/esp/esp32-usb-host/components/usb_host/host/usb_host.cpp:40

0x4037bd0d: vPortTaskWrapper at C:/Users/spenc/esp/esp-idf/components/freertos/port/xtensa/port.c:130

ELF file SHA256: e9a22024db2aa6f2

Rebooting...
ESP-ROM:esp32s3-20210327

I received the same error due to board mistake. Problem was D+ short with ground, resolved by clearing  short circuit.


tensorflowlite.dll is huge (compared to other platforms)

Opened on behalf of @sztomi, Issue is current as of this writing. Previous Issue: https://github.com/tensorflow/tensorflow/issues/48118
Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): Source
TensorFlow version: https://github.com/tensorflow/tensorflow/commit/5d6cc7bf97a226c1e6a73ad4fc391c154dd622ac
Python version: n/a
Installed using virtualenv? pip? conda?: n/a
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): MSVC 2019 (cl: 19.28.29334)
CUDA/cuDNN version: n/a
GPU model and memory: n/a

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
bazel build -c opt //tensorflow/lite/tensorflowlite.dll

This yields a 15MB statically linked binary when compiled for 32-bit windows (i.e. the 64-bit version will be even larger). Compared to other platforms, this is a 5x increase that is difficult to explain, especially in lieu of tools like bloaty on Windows. Unfortunately I don't know where to start poking this at all - but I do believe that even accounting for platform differences, a 5x size difference is unexpected. The same issue was noted here: https://github.com/tensorflow/tensorflow/issues/33634#issuecomment-620645664 (with a suggestion to use the C API instead, but the C API is a binding that cannot be used independent from the C++ binary).
Any other info / logs
Let me know if I can include any more info.

It looks like you're facing a build/installation issue related to TensorFlow. The problem you're describing is about the size of the statically linked binary when compiling TensorFlow Lite for Windows.
The issue seems to be regarding the size of the compiled tensorflowlite.dll binary, which is significantly larger on Windows compared to other platforms. This is causing a 5x increase in size, which is unexpected and might be impacting the overall application size.
Here are a few steps you could consider to investigate and potentially address the issue:

Debug Symbols: Check if the compiled tensorflowlite.dll binary includes debug symbols. Debug symbols can significantly increase the size of the binary. Ensure that you're building with optimizations (-c opt) and without debug symbols if size is a concern.

Linker Flags: Review the linker flags and options used during the build process. Sometimes, specific linker flags can lead to larger binary sizes. Ensure that you're using appropriate linker flags for size optimization.

Compiler Flags: Review the compiler flags used during the build process. Certain compiler flags might result in larger code size. Make sure you're using flags that encourage size optimization.

Unused Code Elimination: Ensure that unused code elimination is enabled during the compilation process. Unused code can contribute to the binary size. You can explore options like "tree shaking" to remove unused code.

Platform Differences: Windows might have some inherent differences in how it compiles and links binaries compared to other platforms. Check if there are specific platform-related factors that might be contributing to the larger binary size.

Build Configuration: Make sure you're building the TensorFlow Lite library using the appropriate configuration for Windows. Different build configurations can affect the size and behavior of the generated binary.

Dependency Management: Check if there are any unnecessary or redundant dependencies being linked into the binary. Removing unused dependencies can help reduce the size.

Compiler Version: Sometimes, using different compiler versions can result in different binary sizes. You might want to try different versions of MSVC and see if it affects the size.

TensorFlow Versions: The issue might be specific to the version of TensorFlow you're using. Consider trying different TensorFlow versions to see if the problem persists or if it's resolved in a newer version.

Since this issue seems to be related to the internal build process of TensorFlow Lite and the specific platform (Windows), it might be beneficial to engage with the TensorFlow community directly. You can post your issue on the TensorFlow GitHub repository, following their guidelines for reporting build/installation issues. This way, the developers and maintainers of TensorFlow will be able to provide you with more targeted assistance and solutions.

Hi, @pkgoogle
Thanks for raising this issue. Are you aware of the migration to LiteRT? This transition is aimed at enhancing our project's capabilities and providing improved support and focus for our users. As we believe this issue is still relevant to LiteRT we are moving your issue there. Please follow progress here: https://github.com/google-ai-edge/LiteRT/issues/153
Let us know if you have any questions. Thanks.


No item click effect like RecyclerView

Describe the bug
My TreeView row using CardView, I set it to “selectableItemBackground” but no click effect work. It works fine with default RecyclerView (using same layout.xml).
To Reproduce
Steps to reproduce the behavior:

Create TreeView with custom tree node view
Add android:focusable="true", android:clickable="true" and android:foreground="?attr/selectableItemBackground" to custom tree node view
Click on 1 tree node
No click effect performed

Expected behavior
TreeView item should has ripple effect when clicked.
https://drive.google.com/file/d/116yVaXkOfKYCWkDr6ZSc7rb4KdzsEnVd/view?usp=sharing
Video
https://drive.google.com/file/d/118hr4uv6pyGhtSHLSnOu_EO6UP6h6vD8/view?usp=sharing
Smartphone (please complete the following information):

Device: Realme 5i
OS: Android 13
Version 1.1.3

Hello @ptu1710,
Thank you for reporting and creating sample for this issue, i have solved it and it increase the performance on the adapter, I will publish bug fix release now and you can try it after 1h.
https://github.com/AmrDeveloper/TreeView/assets/23631699/88e4980d-0f18-4272-9556-ed568f8b8df5
Thank you and Enjoy TreeView
Amr Hesham

Thanks for this quick fix! I will try it then.


GacUtil in v5.0.18

I'm running v5.018 with VS2010/BizTalk 2010 Beta, and when I run the 'Gac output of selected project' from the tools menu, I get the following error:

Microsoft (R) .NET Global Assembly Cache Utility.  Version 3.5.30729.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Failure adding assembly to the cache:   This assembly is built by a runtime newer than the currently loaded runtime and cannot be loaded.

I double checked that I have the newest SDK and what not, any thoughts?
This work item was migrated from CodePlex
CodePlex work item ID: '6916'
Assigned to: 'tfabraham'
Vote count: '1'

[tfabraham@6/29/2010]
It's wrong all right.  I used the same SDK registry key for VS 2010 as for VS 2008 because at the time I didn't have a VS2010 machine and didn't know where it was located.  By the time I did have VS 2010, it got missed.  Thanks for the report.

[UnknownUser@6/29/2010]

[UnknownUser@6/29/2010]
Resolved with changeset 48112.

[tpm2112@6/30/2010]
Thanks for the quick turn-around.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


Instant username autocompletion broken

It seems switching to trix has broken instant username autocompletion.

Closing in favor of the #69 PR.


DOC: move 0.7->0.7.0, rebuild master

Two things combined, if it is too complicated to review I can redo them as two separate PRs:

move 0.7 documentation to 0.7.0 in prerapration for 0.7.1 doc creation once there is a v0.7.1 tag. Maybe that should be v0.7.0 instead (with a 'v')?
rebuild documentation ~on~ off master, to see what the 0.7.1 docs will look like.

hmm. "build error" - I guess that is expected on this PR to a branch.


Autoflagging reportbacks with 0 quantity

Would make reviewing reportbacks a whole lot easier for reviewers.  We don't allow 0 on web, but SMS reportbacks currently don't seem to check.  Ideal fix here is to prevent saving a Reportback on SMS until the user enters a valid quantity.

Chatted with campaigns team and they'd rather just go through these. So we're not going to do this work.


Error parsing the parameters

The commands
astataudit -c test.wav
astataudit -f test.wav
astataudit -x test.wav
astataudit -g test.wav

do not give the expected error messages
Option -c requires an argument
Option -f requires an argument
Option -x requires an argument
Option -g requires an argument

because test.wav is considered the parameter.

The help doc suggests 'y' or 'n' as the parameter to these options, but each requires an argument and your example provides one for each, though it doesn't use the expected vocabulary.
https://github.com/amiaopensource/astataudit/blob/2b61e49cd4b9bf86139494ee96b16008a4bcbea2/astataudit#L31-L34
imho, the expectation Option -c requires an argument is unfair as your example does provide an argument for the option. Perhaps a function could validate the parameter.

Perhaps a function could validate the parameter.

Indeed, that is what I tried to say.

Perhaps a function could validate the parameter.

Indeed, that is what I tried to say.

Another solution could be to add -i (aut similia) for the sound file.

Closing as there is no interest.


What if CONDA is already installed but not from Homebrew

Hi, thanks for your time and effort you put into creating and sharing this work! :)
I was just wondering, before looking into the script and hacking away, if there's a specific need for conda to be installed from brew...
(base) ➜  Deep-Live-Cam-Mac-Installer git:(main) ./deep_live_cam.sh
Apple Silicon (arm64) detected.
Homebrew is already installed.
Homebrew Conda not found at /opt/homebrew/Caskroom/miniconda/base/bin. Please ensure Miniconda is installed via Homebrew.
(base) ➜  Deep-Live-Cam-Mac-Installer git:(main) conda --version
conda 24.4.0
For M1 & M3
Thank you!

I'm replying to myself:
I just put the path where my conda install is and the script worked and even asked for camera perms which I accepted. The UI launched but there's no camera nor a green led notifying that camera is active.

Hi @CeKMTL - Can you test this again - hopefully it will now use an existing conda install if it already exists without trying to install the homebrew version

Assume fixed as no further reports of this issue.


Accept key backups as usable if they're signed with the master cross-signing key

Accept key backups if they're signed by the master cross-signing key.
Used matrix-js-sdk as a reference which accepts it as usable if either the signing device is verified or it's the cross-signing key. https://github.com/matrix-org/matrix-js-sdk/blob/develop/src/crypto/backup.ts#L426-L433
Signed-off-by: Brad Murray<EMAIL_ADDRESS>Pull Request Checklist

[x] Pull request is based on the develop branch
[x] Pull request contains a changelog file in ./changelog.d. See https://github.com/matrix-org/matrix-ios-sdk/blob/develop/CONTRIBUTING.md#changelog
[x] Pull request includes a sign off

Codecov Report

Merging #1492 (5081a33) into develop (f6f1f15) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #1492      +/-   ##
===========================================
- Coverage    12.05%   12.04%   -0.01%
===========================================
  Files          512      512
  Lines        83831    83839       +8
  Branches     35790    35792       +2
===========================================
- Hits         10103    10097       -6
- Misses       73360    73374      +14
  Partials       368      368

Impacted Files
Coverage Δ

MatrixSDK/Crypto/KeyBackup/MXKeyBackup.m
0.00% <0.00%> (ø)

MatrixSDKTests/MXCryptoBackupTests.m
0.00% <0.00%> (ø)

MatrixSDKTests/MXHTTPAdditionalHeadersUnitTests.m
71.73% <0.00%> (-13.05%)
:arrow_down:

Continue to review full report at Codecov.

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

Strange, these tests seem to pass locally, any ideas?

Strange, these tests seem to pass locally, any ideas?

Unfortunatelly crypto + integration tests are very flaky, we are in the process of enabling + fixing them. I'd say make sure that they pass on your local branch, and optionally you can rerun the GitHub tests a few times. We cannot just turn them off because locally they are not as flaky and thus hard to spot the misbehaving ones.


Use SAML with kibana and another http authentication mechanism

Hello,
We have setup the SAML authentication (was not simple).
We tried to use SAML authentication and http basic authentication (via ldap or internal users) but as the SAML redirects directly to our Azure Active Directory.
Is it possible to have both ? If yes, great, how could it be setup ? If not, I will create a feature request ticket.
Many thanks for your work and this great tool.
Kind regards,
Julien Nix

@zengyan-amazon , can you provide your comments here?

@zengyan-amazon , can you provide your comments here?

@juliennix Currently security Kibana plugin only support 1 authentication type, while Elasticsearch security plugin supports chaining the auth handlers. If you want Kibana to authenticate users using both SAML and basic auth, it is not supported.
if you want Kibana users to login using SAML, and your systems can make calls to Elasticsearch using basic auth, that is supported. To do this, in your Elasticsearch security plugin config, you can set basic auth order to 0 and challenge to false, then config SAML auth order to 1, and challenge to true.

@juliennix Currently security Kibana plugin only support 1 authentication type, while Elasticsearch security plugin supports chaining the auth handlers. If you want Kibana to authenticate users using both SAML and basic auth, it is not supported.
if you want Kibana users to login using SAML, and your systems can make calls to Elasticsearch using basic auth, that is supported. To do this, in your Elasticsearch security plugin config, you can set basic auth order to 0 and challenge to false, then config SAML auth order to 1, and challenge to true.


Optimization: Remove redundant list call in check_file in Build/check_indents.py

In the check_file function of Build/check_indents.py, it is performing a redundant list call, on f.readlines() which itself returns a list, in line 28:
def check_file(filename : str) -> bool:
    ...
    with open(filename, 'r') as f:
        contents = list(f.readlines())
    ...

This can have impact on performance, if the file is really large. This does not have any benefit over the plain f.readlines() either.
I propose the following change:
-        contents = list(f.readlines())
+        contents = f.readlines()

Would be happy to add a PR, if the above proposal sounds reasonable.

Thanks for raising this, and for your patience! Would be happy if you'd still be interested in opening a PR!

Sure, I will add one today.


Question, how to make a standalone excalidraw

Disclaimer: I am not a JS or CSS developer, I have a bare understanding of React framework, or build tools like yarn.
What I'd like is the ability to open a local index.html with Excalidraw (an offline version). But I have some difficulty to piece things together. The context of this is to load excalidraw in a Chrominium Embedded frame (JCEF) for an IntelliJ plugin.
I wanted to test what is described here : https://www.npmjs.com/package/@excalidraw/excalidraw#in-browser
<script
  type="text/javascript"
  src="https://unpkg.com/<EMAIL_ADDRESS>></script>

This is likely wrong, as nothing happens.
I can probably use whatever is built from the yarn directive in the dockerfile:
https://github.com/excalidraw/excalidraw/blob/6dee02e32062057cb5b18702f0b5fe1a3ef571fd/Dockerfile#L15
But isn't there a better / simpler alternative ? A prebuilt lib like the one on unpkg ?
Thnaks in advance

And many thanks for excalidraw it's awesome

Ok I think I lacked react and node related stuff knowledge a lot.
still I think the project’s documentation and example could be improved to be more precise, typically the package’s readme lithe code listing could add file paths and stress out that going in the code sandbox is critical to understand what should be done to get something working.

Hey @bric3.
What did the problem you were facing turn out to be, so that we can improve our documentation? Also, feel free to open a PR or make suggestions on what we should improve.
Thanks! ❤

@dwelle Hi,
Please keep in mind I'm not a frontend developers, so I probably lack many reflexes.

The first thing I looked at was to be able to make a simple index.html that load the necessary JS (and any other necessary) assets from a standard.
And for that I went to the repo's Readme, however this readme doesn't point to the documentation that I should use (as a consumer of the library). In fact I searched the Web and found the npmjs page mentioned above. Only to find a tad later it was available here in the src/packages/excalidraw.

Then in this readme, I believed I found what I was looking for in just the following snippet (in-browser section).
<script
  type="text/javascript"
  src="https://unpkg.com/<EMAIL_ADDRESS>></script>

However it doesn't work that way. It's true this section mentions react and react-dom, yet this didn't kicked in immediately in my head. I believe this section should be rewritten with the index.html as the first snippet, and then eventually hidden details section with the index.js and finally the mention about production or development JS urls.
One thing that confused me in the codesandbox example is the presence of package.json, and that led me to believe I needed tools that I didn't understand well at this point (node, npm, yarn, ...).
So I think this example may need some more explanation.

After some iteration I finally got it working ...as part of a yarn lifecycle, this might have been avoided, but I went through this path. And then as searched over internet some examples were using the real react App approach.

So I tried to upgrade my simple HTML page to use react, however I didn't immediately caught that the section Web Bundler, Webpack was actually the right section as I already had been using yarn and react-scripts. Especially since the full example is collapsed (as well as the codesandbox link).
I had issues to migrate the in browser code to actual web bundler code. Typically I didn't expect the App function to be rewritten (from React.createElement style to <Excalidraw style), maybe that's expected with react, but for me, a novice, it was a surprise.
Also when looking at the codesandbox, I noticed that I still wonder about

There's a window.EXCALIDRAW_ASSET_PATH = "/"; in the public/index.html that I am not sure if I should use it in my app.
There's the public/excalidraw-assests-dev with fonts, and other stuff, that I am not sure if I need those, and I don't know how to grab them easily.

At this stage I have a working minimal React app with excalidraw. But I still wonder if my setup is correct.

Thank you @ad1992 for the feedback !

Unfortunately github markdown we can't control the size of dropdown, we might move the examples to a different folder altogether so full example is at one place.

Maybe moving the link to the codesandbox outside the <details> element could help.

Yeah in the browser we are just using React API in vanilla js and not JSX syntax hence it looks different. We can use jsx in browser with babel plugin but it will become slow and not suitable for production.

Actually this a very interesting aspects, I think it's work mentioning, I say that as a novice, but I think it's useful to explain the difference.

Not sure if you went through the Installation section, but I will explain here.

Yes I did, but you explained this part better ! Actually since it's an IntelliJ plugin I need to download the assets to make them available "locally".
Actually I have one left question: What is the best option to access these assets? I just inspected the unpkg link
https://unpkg.com/browse/<EMAIL_ADDRESS>and noticed the same structure in node_module/@excalidraw/excalidraw/dist. Should I just copy these files manually to my specific destination dir ?
Or is there a package.json trick to copy them where needed ?

Actually I have one left question: What is the best option to access these assets? I just inspected the unpkg link

As mentioned above by default it will work out of the box and get served from unpkg so you need not make any changes at all unless you want to host it yourself.
If you want to host it yourself then you have to move the folder excalidraw-assets and excalidraw-assets-dev to the path (basically your CDN path ) where your assets are served.

OK, thank you @ad1992 !
I've got another question is there a public API or a way to load a scene from an image file (SVG or png). The IDE infrastructure have it's own file infrastructure, which means I cannot use Excalidraw relying on the browser filesystem api. The documentation only points to API that accepts elements to load a scene. I've found this code.

https://github.com/excalidraw/excalidraw/blob/5c73c5813ce92d3c7b0610530f78ccb06a47d983/src/data/blob.ts#L82-L113
https://github.com/excalidraw/excalidraw/blob/5c73c5813ce92d3c7b0610530f78ccb06a47d983/src/data/image.ts#L106-L136

It's easy to re-implement, but this might be out of sync.

So I would like to use loadFromBlob, at this time hacking around didn't work.
Hacky that doesn't work :
import Excalidraw, {
    exportToSvg,
    exportToBlob,
    getSceneVersion,
    loadFromBlob,
} from "@excalidraw/excalidraw";

export default function App() {
    const excalidrawRef = React.useRef(null);

    window.loadBlob = (blob) => {
        return loadFromBlob(
            blob,
            excalidrawRef.current.getAppState());
    };

    window.exportAsSVG = (exportParams) => {
        return exportToSvg({
            elements: excalidrawRef.current.getSceneElements(),
            appState: {
                ...excalidrawRef.current.getAppState(),
                ...exportParams
            },
        });
    };

Export as SVG works fine, but with loadBlob I'm encountering a Uncaught TypeError: Object(...) is not a function. I somewhat expected an error since this function do not appear public or exported.

@bric3 the API doesn't expose loadFromBlob hence you are getting the above error. We don't support Importing images yet.
But we do embed scene data if "Embed scene" is checked as shown below (this will increase your export file size)

We will be exposing loadFromBlob as well very soon, there is a similar request here as well post which you will be able to use it.

Cool thanks a lot.

@ad1992 Hello again, I have a question again on this scene embedding. I'd like to do that programmatically, However I can't seem to get excalidraw embed the scene when calling exportToSvg
import React from "react";
import Excalidraw, {
    exportToBlob,
    exportToSvg,
    getSceneVersion,
    serializeAsJSON,
} from "@excalidraw/excalidraw";

// placeholder for functions
window.saveAsSvg = null;

...

export default function App() {
    const excalidrawRef = React.useRef(null);

    ...

    window.saveAsSvg = (exportParams) => {
        return exportToSvg({
            elements: excalidrawRef.current.getSceneElements(),
            appState: {
                ...excalidrawRef.current.getAppState(),
                ...exportParams
            },
        });
    };

    return (
        <div className="excalidraw-wrapper">
            <Excalidraw
                ref={excalidrawRef}
                initialData={{ appState: {
                        exportEmbedScene: true
                    }
                }}
                ...
            />
        </div>
    );
}

I also tried to pass the exportEmbedScene to the exportToSvg directly (as part of the appState). But this doesn't seem to work. The scene never gets embedded. Looking at the code
https://github.com/excalidraw/excalidraw/blob/5cd921549a7e2b67219ee3f10d98228e23103c0f/src/scene/export.ts#L68-L134
I believe exportToSvg cannot embed the scene. It's likely the same for the png / jpg exports. Is it expected ?

@ad1992 Hello again, I have a question again on this scene embedding. I'd like to do that programmatically, However I can't seem to get excalidraw embed the scene when calling exportToSvg

@bric3 do you mind asking this and #3752 as a QnA in https://github.com/excalidraw/excalidraw/discussions/categories/q-a ? We are gradually moving to discussions for any queries related to excalidraw npm package so that everything is well documented at one place and it will be helpful for future users as well.

Not at all. Thx for the pointer ! @ad1992

Done, question is asked here https://github.com/excalidraw/excalidraw/discussions/3756


Update Marketo User Audit Data Stream Docs for event structure change.

CET-1021
Event structure change, wrapped event details in the body element.
Remove the user field.
Add the userId to support easy track of the user details.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@bejohnso I added the equivalent changes in this repo based on feedback from @rperelma
@kgera, please let me know if any else I need to do before merging these changes.

@kgera, I don't have write access to merge. Would you please help me to merge this pull request?


Metadata files have a double dot in the name

E.g. result-160f4730af01d4b2f1bb1c924764ef77.json..metadata.

This was happening in older metadata filenames and the recent one seems perfect.

@santoshmahto7 Are we going to rename ?

Fixed with this MR https://github.com/iocost-benchmark/iocost-benchmarks/pull/64 (merged).

Merged : https://github.com/iocost-benchmark/iocost-benchmarks/pull/64


Limit # of imports at the same time per user

Fixes #2132

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

retest this please

Frontend tests were OK :+1: (details)

@rafatower CR of the backend please

@CartoDB/frontend: can you review the js part, please?
I modified the code to show the 429 error message that comes from the backend:

https://github.com/CartoDB/cartodb/commit/d569ed565f31f25beddb1a2a0bfcfed95e6d8c0b
https://github.com/CartoDB/cartodb/commit/ebce67c4a291a3d48e5acdb6fb4caf62beb56f96

And here's the result, for @CartoDB/design:

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

backend looks good :+1:

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

@CartoDB/frontend PING again, sorry to insist but I need to push this live as soon as possible and is only pending the FE CR

Checking

Frontend tests were OK :+1: (details)


raidboss japanese localize

btw, is there any way to host overlay url with github pages? I wanna share my rep to friends instantly. but it doesn't work...

This looks great, can you fix the two lint errors from travis ci? https://travis-ci.com/quisquous/cactbot/jobs/221345257
npm run lintfix should fix them automatically if you have them installed
I am using github pages to share cactbot master branch, e.g. having people use https://quisquous.github.io/cactbot/ui/raidboss/raidboss.html as their raidboss url.  You may be able to do the same with your fork's master branch?

oops, fixed it.
thanks ill try it!

Sorry for bothering you. maybe it should be ok...

Sorry for bothering you. maybe it should be ok...

It's no problem at all! Thank you for all the changes.  I'm happy to merge any more translations you would like to do.


Compile error under Xcode 15: "Multiple methods named 'setProgress:' found with mismatched result, parameter type or attributes"

In Xcode 15.0 and 15.1, there is a compilation error at MRProgressOverlayView.m:811:
        [((id)self.modeView) setProgress:self.progress];

Multiple methods named 'setProgress:' found with mismatched result, parameter type or attributes

There is no issue in Xcode 14.3.1.

Hi I had the same problem I adjusted this line to be
[((id)self.modeView) setProgress:self.progress animated:animated];
The App compiled and worked


Windows forms support

Any example on how to add it to a windows forms? I tried and i failed multiple times appreciate if could get an example.

Show what you are doing. There is nothing wrong with the Overlay. It will work with WinForms

But there’s no example to do it

no example needed it is exactly the same


Differences between GPT4 and Llama tokenizer leads to mismatch in token count during prompt pruning.

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Windows 11 Pro
- Continue: v0.8.12
- IDE: VS Code
Model: Codellama 70b (Free Trial), or any Llama-base models that rely on the the Llama tokenizer.

Description
Current methods to estimate prompt token count use the GPT-4 tokenizer. However, Llama-based models use a different tokenizer. This difference leads to mismatch between estimated token count in Continue versus what is actually received by the model. The Llama tokenizer consistently produced ~30% more tokens than the GPT-4 tokenizer. Depending on the configuration of the LLM server, this can lead to inference errors from the prompt exceeding the maximum number of allowable tokens. See this Discord discussion on the topic.
Differences between the GPT-4 tokenizer and Llama tokenizer can be explored using these links:
https://platform.openai.com/tokenizer
https://belladoreai.github.io/llama-tokenizer-js/example-demo/build/
Current token count estimation is done around here: https://github.com/continuedev/continue/blob/c8d793ec4599b954c4ec41fe4187d8e676e0b048/core/llm/countTokens.ts#L12
@sestinj has identified a js llama tokenizer that may be worth exploring: https://github.com/belladoreai/llama-tokenizer-js.
To reproduce

Select the Codellama (Free Trial) option from list of default models (or any Llama-based model)
Determine max prompt token length for that model
Create a prompt that that size or knowingly exceeds it. To produce this I usually pass in a whole C++ source code file as reference via the "@" prompt operator.
Submit prompt to LLM and observe error resulting in token count differences despite prompt pruning being performed by Continue

Log output
Continue error: HTTP 500 Internal Server Error from https://node-proxy-server-blue-l6vsfbzhba-uw.a.run.app/stream_complete

Error in Continue free trial server: 403 Input validation error: `inputs` tokens + `max_new_tokens` must be <= 4097. Given: 5437 `inputs` tokens and 1024 `max_new_tokens`

@rastna12 this is now available in pre-release. Here's the commit that did it: https://github.com/continuedev/continue/commit/e8bbdc06a192a9d6576b7019a164393c16019306
Let me know how it looks and I'll wait to close the issue until you've verified

I think this has been resolved given conversations in Discord. If I'm mistaken please re-open!


Switch to a pyproject.toml

Replacing setup.py and requirements.txt with pyproject.toml

We need to change all the exactly pinned dependencies to lower bounds, but that's for later.

resolves #110
We should also document developers should install the dev packages too.

Codecov Report

Merging #110 (5571028) into main (9564e61) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 5571028 differs from pull request most recent head 7e0a653. Consider uploading reports for the commit 7e0a653 to get more accurate results

@@           Coverage Diff           @@
##             main     #110   +/-   ##
=======================================
  Coverage   85.89%   85.89%
=======================================
  Files          55       55
  Lines        4139     4139
=======================================
  Hits         3555     3555
  Misses        584      584

I've made a few additions:

Changed the instructions in the developers documentation to install all the dependencies.
Changed the license tag to license-files and moved it to tool.setuptools as apparently this is best practice.

I have asked some colleagues about how license and readme files are used. The readme file is parsed by PyPi so if we ever publish it to PyPi then I think the Readme displays on the front page maybe?
However. nobody seemed to understand how the license file was used. As we have a seperate license file incorporated in the repository anyway, this should be included in any distribution i think. .
Anyway, I think its good to go for now.
@mhauru - Do you think we should all try and do a full install before merging? There are endpoints in this file. . . Or do we trust to CI to do this?
I think regardless, @GiorgioCerro, it would be good if you could test the pip install your end. Instructions are here.

Thanks Edwin, looks good. I was heartened to see that the unit tests action fails if we make the package uninstallable, but would be happy to get any possible comments from @GiorgioCerro

I am testing this now with a Windows laptop. If that works I think it is going to be a super thumbs up!
Just checking, I saw @mhauru is working with a Macbook. What about you @EdwinB12 ? I hope all the packages are compatible with the three different OS

Everyone in REG runs a Mac, so everything has been mostly tested on MacOS, although I would expect things to go smoothly on Linux too since it's so similar. Now Windows on the other hand I'm very interested to hear how it goes, I have very little understanding of how development works on Windows and this would be good feedback. If you have access to the Linux Subsystem for Windows thing that might come in handy if things don't work out of the box.

Yeah, there's a problem with the frozen scipy version: https://github.com/alan-turing-institute/DTBase/issues/91
Will hopefully sort that out and increase the range of compatible versions soon, as we go through relaxing dependency constraints in general.


Human-friendly errors status update?

Hey folks - one common-thread-of-annoyance that I've had from many-a-sails-app over the last year has been validation errors and their respective messages. Specifically: the ability to override the "programmer-friendly" validation errors like undefined should be a string (instead of \"null\", which is a object) with our own custom errors like [Input] is a required field.
It was originally on the forecast for "0.10.0-rc6", which has now (as far as the Trello indicates) been pushed back to "0.10.x".
I am interested in helping get this out the door, or at the very least being able to fork sails and edit my own copy if you've already got it under way :)
So, how goes this functionality?
Also, I'm interested in attempting this on my own regardless, would one of you Sails devs (or @robwormald) know where I could find the place in the code where the actual response containing the WLValidationError is sent? I poked around but couldn't find it :)
Edit: forgot that I made a repo and might be making a library (separate from sails) thats a human-friendly-error-producing validation library. Take a look at my proposed implementation, that's the format that I think the responses should be sent back to the client.

For sails v0.11.X you may use this sails-hook-validation

Neat stuff @lykmapipo .

@derekbasch +1

+1


Small parser patch

IDK why this was broken or what this is actually fixing.

Actually this looks like it's stemming from a merge issue in

https://github.com/HazyResearch/snorkel/commit/10f23e6ca0742ba0ed05e6393db94bd515e4778d
https://github.com/HazyResearch/snorkel/commit/35e714902e04c58118ce2445fbaef3ad7d138af4

@bhancock8 can you check this out and make the appropriate changes and/or rollback parser.py to ab7b23

Yep--your fix is exactly what was needed, and it makes sense. Looks like the get function being dropped was just a transcription error. The original line looked like this:
parts['text'] = ''.join(t['originalText'] + t.get('after', '') if i < L - 1 else t['originalText'] for i,t in enumerate(block['tokens']))
And after we worked out the bug in CoreNLP being caused by PTB characters, that extra "if" statement wasn't needed. This patch looks good to me.

Ok cool, can you run a diff on the rest of the file to make sure everything else was preserved?

Yep--diff looks good. That's everything.


nixos/inadyn: init

Description of changes
Adds a NixOS module for inadyn.
inadyn does not use a standard configuration file format, so some custom logic is needed to render nix options into a configuration file. Every current option should be correctly handled.
Rather than running as a daemon, inadyn is used in one-shot mode at user-defined intervals, as it will be idle most of the time. The default interval is set to every minute, which is the case when using daemon-mode. I have also tried to harden the service as much as I can.
I have tested this successfully with my personal simple inadyn configuration, but would appreciate it if other users can confirm this module works for them.
Closes #242330, and if merged before 23.11 stable NixOS users shouldn't be without a dyanamic dns client :partying_face:
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Here is another attempt https://github.com/ahbk/my-nixos/blob/master/inadyn/default.nix. It has passwordFile and uses the systemd settings from this template https://github.com/troglobit/inadyn/blob/master/inadyn.service.in. Let me combine.

@ahbk (replying in the main thread rather than a code review comment) your module is quite different from this one: it runs inadyn as a daemon rather than a timer (though reasonable minds could disagree on which is preferable), and is not RFC42 compatible. I also think passwordFile should be renamed as other modules use it purely as a file containing a password only (rather than additional included config options).
An advantage of Nix to me is to minimise the context switching between different config formats, which is why I went through so much effort here to allow configuration purely in Nix. Though, I appreciate the resulting logic to render the config file is a bit involved.
I'll try to get some eyes on this to see if there's a consensus on which module is preferred.

Yes good points.
I didn't delve on the systemd settings I just assumed the templets would be good. Is it not right to run it as a service?

No, both are valid. inadyn can be run as an always-active daemon or on a timer using oneshot mode.

Other than squashing my commits, are there any blockers here?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2980

For now this module can be used via my NUR

Rebased on master + moved release notes to 24.05

Does anything else need to be done?

LGTM. just squashing and fixing the conflicts.

Conflicts resolved and commits squashed.

Not sure how those files made it in, sorry!

Fixing conflict in release notes

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3583

Hi. I've used this now for my own IPv4 and IPv6 configuration, and I've a few remarks I wanna add.
In summary the issues I ran into were:

AF_NETLINK missing from service (useful for IPv6 through ip command)
debug log is still enabled producing long logs in relative short amount of time
in order to hide the secret I needed to use configFile, as DynamicUser is set, defeating the purpose of using the nix language

Hi. I've used this now for my own IPv4 and IPv6 configuration, and I've a few remarks I wanna add.

Thanks for the feedback!

In summary the issues I ran into were:
* `AF_NETLINK` missing from service (useful for IPv6 through `ip` command)

Noted, I will add it in my next push!

* debug log is still enabled producing long logs in relative short amount of time

I kind of agree, but on the other hand inadyn logs too little with debug logging disabled imo. This is the full output for running inadyn  with the standard logging level and it discovering on ip change is needed:
inadyn[2110999]: In-a-dyn version 2.12.0 -- Dynamic DNS update client.

* in order to hide the secret I needed to use `configFile`, as `DynamicUser` is set, defeating the purpose of using the nix language

Does this approach not work? You would use something like sops-nix to manage auth.txt. Agree with SuperSandro it could be better telegraphed.

Does this approach not work? You would use something like sops-nix to manage auth.txt. Agree with SuperSandro it could be better telegraphed.

At least for me, it did not, as the access permissions to auth.txt were not right. Or at least I think that was the issue. As far as I know auth.txt would by default be stored in a way that only root can read it. This wouldn't be an issue with systemd's LoadCredential, if auth.txt would be the only one provided to it. LoadCredential opens the given file as root and then puts it in a secure, but for the process accessible storage.
As LoadCredential has no overview over the existence of auth.txt, it won't read that in, so auth.txt would still be only readable for root. Your usage of DynamicUser here I think conflicts with that, as that user won't be able to read auth.txt.
It was late at night when I set this up, so I'm not 100% sure what steps I took to end up writing the whole config file as a sops-template. But I think we tried the include directive.

Summary of latest changes:

Rebased on master
Service no longer runs as DynamicUser, this seemed to be the simplest way of letting include directives work
Include directives work
Log level is configurable
AF_NETLINK capability added
Use of systemd timer format documented

@SuperSandro2000 what else do you think would be useful to add other than the existing warning to point users in the direction of secure credentials management?

Looks good enough for now. Everything else can be done in another PR.

This broke the manual build: https://github.com/NixOS/nixpkgs/pull/305842


fix: fix typo to refer to settings.polyfills

I set settings.polyfills as in README, but it did not load.
This is due to a typo in the code for polyfills.
This PR will fix the typo so that settings.polyfills is referenced correctly.

You're rught, Thanks!

https://github.com/azu/eslint-plugin-typescript-compat/releases/tag/v0.3.1


Documentation - create an highlight of what has been released in 1.0.0

using the engineer blog to post it

blog post in draft right now: https://github.com/Capgemini/blog-private/pull/936


prevent invalid enumerator state

I believe it will close https://github.com/tapwork/HeapInspector-for-iOS/issues/19
Though I don't have extensive tests yet

sounds reasonable

@haifengkao could you check the latest version 0.4 if it fixes #19

@tapwork No, the version 0.4 doesn't work at all.
I did a little more research:
Normal Block1

Normal Block2

Corrupted Block

It seems that EXE_BAD_ACCESS is a result of an invalid zone pointer. If we would verify the content of zone, we can detect if the zone object is valid or not.

thanks. looks good

@haifengkao right


Avoid eager loading of jschardet

This PR is part of https://github.com/Microsoft/vscode/issues/30685 and also used to get my feed wet. The jschardet module is loaded eagerly and heavy although actually rarely used (its usage depends on the auto-detect-encoding-setting). This PR makes loading lazy

Cool trick with the overload on detectMimeAndEncodingFromBuffer(readResult: stream.ReadResult, autoGuessEncoding?: false), did not know about that.

TypeScript-Ninja-Skillset


actualizing readme

Okay - we'd probably also want to update travis.yml in this pull request too, to make sure we're actually testing against the versions we claim to support.
More importantly, we ought to upgrade the supported Django versions.


can not connect internet with websocket

Describe the bug
It is ok while run my program in unity editor,  but it seem can not connect internet after deploy to hololens.
It is no errors while in unity editor and visual studio while deploying to hololen .
I donot know the question.
The output is "已加载“C:\Windows\System32\Windows.Networking.Connectivity.dll”。无法查找或打开 PDB 文件。" while program is debuging.
Or my way is wrong of using websocket in hololens.
Screenshots

below two images is debuging output.

Your Setup (please complete the following information)

Unity Version [e.g. 2019.2.6.f1]
MRTK Version [e.g. v2.1]

Target Platform (please complete the following information)

HoloLens

@IfnoLee, thanks for filing an issue. But it doesn't seem like you are using any MRTK component. If you have an issue with the HoloLens platform, you should reach out in that community. It'll be easier and faster for you to get an answer to resolve any platform-specific issues (irrelevant of MRTK)
https://support.microsoft.com/en-us/hub/4338666/hololens-help
Can you also provide more details about the hololens? Is it 1 or 2? What OS version do you have on there?

@IfnoLee, thanks for filing an issue. But it doesn't seem like you are using any MRTK component. If you have an issue with the HoloLens platform, you should reach out in that community. It'll be easier and faster for you to get an answer to resolve any platform-specific issues (irrelevant of MRTK)
https://support.microsoft.com/en-us/hub/4338666/hololens-help
Can you also provide more details about the hololens? Is it 1 or 2? What OS version do you have on there?

The WebSocket SDK is ok ,but the token is not be parsed because of  Newtonsoft.Json is not work in UWP . Everything is ok after I replace Newtonsoft.Json by Json.Net.Unity3D.

I have same issue when I had used Newtonsoft.Json.  I hope that Microsoft office don't close bug. I looking forward to  fixing this bug for MRTK.

@IfnoLee, thanks for filing an issue. But it doesn't seem like you are using any MRTK component. If you have an issue with the HoloLens platform, you should reach out in that community. It'll be easier and faster for you to get an answer to resolve any platform-specific issues (irrelevant of MRTK)
https://support.microsoft.com/en-us/hub/4338666/hololens-help
Can you also provide more details about the hololens? Is it 1 or 2? What OS version do you have on there?

The WebSocket SDK is ok ,but the token is not be parsed because of Newtonsoft.Json is not work in UWP . Everything is ok after I replace Newtonsoft.Json by Json.Net.Unity3D.

this version is old ? author  think about update Json.Net.Unity3D?


pull_request_template.yml is not working as expected

After clicking Compare & pull request button after making changes on my branch on the forked repository, I see the following screen.
It's not templated, it is just whole content of pull_request_template.yaml.
Perhaps this is not working as expected.

@AlexSCorey
Just to inform, I'm tagging you since you've added that file.

Thanks @kurokobo.  Yes, it is not working as expected.


Update free-programming-books.md

Fix a broken link

Thanks for the fix, and welcome to free-programming-books!

Thanks for the fix, and welcome to free-programming-books!


Incorporate cursorline highlighting in sign column

Fixes #563
WIP as I'm not sure what is required for extmarks and I haven't had a chance to see what needs adding to tests and documentation.

[ ] Have you read CONTRIBUTING.md?

Really excited for this feature, let me know how I can help


Use consistent cases when hashing

Inconsistent cases can cause the hashes to "miss", which may cause messages to be discarded. By using lower case everywhere, this is avoided. It might be desirable to create a wrapper for this hash function that just appends all arguments.

@damancusonqn @nenadilic84


fix(upload): S3ArtifactUploader NPE

Fixes #1762
Context
An attempt to solve the problem with uploading artifact to custom S3 that doesn't response with owner in GetBucketAclResponse
Checklist

[x] Review Contribution Guidelines.
[x] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Update documentation when applicable.

Thank you 😄