fix: convert error output to be consistent cross-platform

The error output produced by shell.error() or result.stderr should
not be inconsistent between platforms. This ensures that path separators
are always printed by ShellJS as / instead of as \ on Windows. This
should allow scripts using ShellJS to be more consistent cross-platform.
We had only one test case which relied on the old behavior, and it's in
pushd, which we don't any issues for. Since it's only the error output
that's changed, it's probably a reasonable fix to make.
Fixes #681

Codecov Report

Merging #684 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #684      +/-   ##
==========================================
+ Coverage   92.68%   92.71%   +0.02%
==========================================
  Files          33       33
  Lines        1176     1180       +4
==========================================
+ Hits         1090     1094       +4
  Misses         86       86

Impacted Files
Coverage Δ

src/common.js
94.5% <100%> (+0.12%)
:white_check_mark:

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 346fca4...c90113d. Read the comment docs.

It would be better to just sanitize the paths themselves, but this would be complicated to implement.

Why would that be better? You mean to change \ ➡ / before passing strings to commands? This would have the same effect on output.

For what it's worth, our docs don't actually specify what these error messages may be. We should decide if we really want to consider these part of our public API. In fact, shell.error() is only guaranteed to be truthy or falsy, and .stderr is only guaranteed to be some sort of error message.
The only real motivation we have for this right now is to land tests, which probably isn't enough to justify something on the fence like this. So feel free to push the milestone back to v0.8.

I think you've convinced me. In the error messages, \ characters should only occur in the paths. And because the error messages are not documented, there should be no reason to rely on them. This looks good so I'll go ahead and merge it.

Er, there are merge conflicts.

Rebased off master. Will land this tonight.

I also fixed a TODO which depended on this being landed.


LOC CHECKIN | MSBuild 20160718 | XLFs localized

Hi @v-zbsail, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

Thank you for the contribution!
I tested out the localized xlfs, and they work.
Some notes:

I see that the xlfs under https://github.com/Microsoft/msbuild/tree/master/src/XMakeTasks/ManifestUtil/Resources/xlf are not in this PR. Is there a specific reason for this?
In the meantime, we introduced one new string (in the shared xlf). Do you want us to merge the xlfs you provided now, and then you make a separate, new PR just for the new string? Or do you want to modify the current PR to include the new string (after we commit the updated xlf). The first options lets us test the pipeline of what happens when just one new string gets updated.


metatensor.torch.Labels.to doesn't show correctly in the docs

Here is how it shows now:

The documentation is using @overload, but it could (and I think it should) use Union instead: https://github.com/lab-cosmo/metatensor/blob/master/python/metatensor-torch/metatensor/torch/documentation.py#L430-L436
With overload, this is how the function is supposed to be documented: https://github.com/Luthaf/rascaline/blob/0e3e5e9dadbc8595d3adce4f3ff01a98e456c56f/python/rascaline-torch/rascaline/torch/system.py#L57C1-L97

oh, and the syntax is completely wrong: should be to(self, device: str)/to(self, device: torch.device) anyway for the @overload version. Was this written with ChatGPT/copilot?


Why Download Health Architectures as a Zip?

I saw something curious during Challenge 2 that may be an issue. Is there a reason to have OpenHack participants download the Health Architectures repo as a zip instead of cloning/downloading it themselves directly from that Github repo?
Reasons to send the user to the repo:

Reduces size of OpenHack repository
Reduces need to update the OpenHack repo when Health Architectures repo changes
Exposes users to another repo that may help drive Health Cloud adoption

@mikaelweave This Github used to have documentation to clone Health Architectures repo but to avoid issues when new functionality is added for the sessions planned, this zip was created. Will go back to repo clone after testing code changes.

If we are worried about new changes being introduced, we could download a version like this:
https://github.com/microsoft/health-architectures/releases/tag/v1.0

We did try v1.0, we could ask to create a new tag for the working version. Also planning to make minor changes for now and post it in that repo to get merged. This short-term approach is for the sessions planned.

Ahhh you're already 10 steps ahead of me. 😄  Thanks for the info, really appreciate it!
FYI I've been doing the OpenHack with the main branch of Health Architectures and filling out issues/PRs along the way. Hopefully this will help get to a stable tag we can use for the OpenHack!


Not able to install the plugin

While installing the plugin using (sao kibana-plugin) facing error
plugin-helpers postinstall
'plugin-helpers' is not recognized as an internal or external command,
operable program or batch file.
Is environment issue?

plugin-helpers is another package that the template uses. You can see the repo right here.
Sounds like perhaps the module install didn't happen. You can try running yarn install or npm install, whichever you are using, from the root of your plugin.
If that doesn't work, it would be helpful to know what OS you're using, and what version.

npm install doesn't work.
Node Js Version:- 6.11.1
OS:- Windows 10 Pro
Folder Structure Kibana:- E:/ELK4/TestKibana/Plugin_Dev/kibana/kibana
Folder Structure Plugin:-  E:/ELK4/TestKibana/Plugin_Dev/kibana/whowe

npm install doesn't work.

What happens? Do you get an error or anything when you try to run it? Can you paste the output here?

The error message is given below
<EMAIL_ADDRESS>postinstall E:\ELK4\TestKibana\Plugin_Dev\kibana\whowe
plugin-helpers postinstall
'plugin-helpers' is not recognized as an internal or external command,
operable program or batch file.
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "--save"
npm ERR! node v6.11.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: plugin-helpers postinstall
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'plugin-helpers postinstall'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ce-views package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     plugin-helpers postinstall
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ce-views
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ce-views
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\ELK4\TestKibana\Plugin_Dev\kibana\whowe\npm-debug.log_

Hrm, strange. I'm not sure what's going on here. I had one of the other Windows devs here try it out and it worked fine for them.
Maybe you're in the wrong path? Looks like you're running this from E:\ELK4\TestKibana\Plugin_Dev\kibana\whowe... can you tell me what's in that path? Also, can you share what's in your package.json file?
Also, do you have a node_modules path there? And is there anything in it? In particular I'd like to know if there's an @elastic path in there. I'm thinking maybe you're installing modules in --production mode for some reason, in which case none of the tooling would be installed.

Closing due to lack of response.


Speedup Travis

A few tricks:

Run using docker with sudo: false for faster startup
Cache the result of downloading and extracting neo4j. Even though this is downloading more bytes, this is faster because the cached server is much faster to reach (same network) than the neo4j server.
Cache the result of running bundler

Spurious spec failure is unrelated, and fixed by #535

And, we're green! :green_heart:


Allow access to vector in matrix variable

Addressing: #132
This PR makes possible to create vector variables as row of the matrix variable.
The problem is that reference to the matrix row was considered as a Scalar and not as a Vector.
Example:
integrity_constraints:
    let a = [[1, 2], [3, 4]]
    let b = a[0]
    enf clk' = b[1]

Accessing b as b[1] we assume that b has a vector type, so we parse it like so. On the checking stage we expect value of variable b (that is a[0]) to be a VariableType::Vector, but we have VariableType::Scalar because that's how it was defined on the parsing stage.
I considered two main approaches of solving this:

Work with variable in symbol_table at the verification stage. There are two sub approaches:

Add new variable with name a[0] and value VariableType::Vector([1, 2])
Change existing variable b to have value VariableType::Vector([1, 2])

Work with variable on the stage of adding them to the symbol_table

The first two options don't work due to limitations on working with memory, I can't create another variable and return a reference to it from function because it is a temporary value, and I can't change existing variable because I'll also need immutable reference to symbol_table (to get a type) to do that.
The second is working, and that's what I have done, but I don't like this approach either. The idea is to change variables of considered type (b = a[i] where a is a matrix) so they don't hold Scalar with VectorAccess to the matrix (let b = a[0]), but Vector with values from the matrix, that is, it is generally the same as having let b = [1, 2] in our case. But here is another problem occurs: instead of having a reference to the data stored in the matrix, we just clone it. But it looks like more general problem, as VariableType enum could store only Expression/Vec<>, but not the references to it, so for now it's the only way to handle it.
I assume that there is a better approach for solving this issue, do you have any suggestions?

I think one other way to handle this might be to just store it as a Scalar(vector_access) in the symbol table and then when it is accessed while inserting the expression, handle this case differently in the insert_vector_access function. This would remove the necessity to clone the vector but I don't think cloning is a big problem here (but I could be wrong) so maybe we should go with whichever solution makes it easy for us to handle all the possible cases.

To list down some possible statements that we need to handle for now:
let a = [[1, 2], [3, 4]]
let b = a[0]
let c = [a[1], a[0]]
let d = [e for e in a[1]]
let f = sum([g for g in a[1])

Please let me know if I missed any.

@Overcastan I think there is an issue with this approach. This doesn't work for statements like:
let a = [[1, 2], [3, 4]]
let c = [a[1], a[0], [5, 6]]

Sorry, I didn't think about this case earlier. I think we will have to make changes to the parser.

Should we manage Vectors with different types, like [5, 6, a[1][0]]? Or, in other words, should we handle statements like
let a = [[1, 2, 3], [4, 5, 6]]
let c = [a[1], a[0], [7, 8, a[1][0]]]

@Overcastan I think so. We already support vectors like [5, 6, a[1][0]]. In my mind, we should just change the parsing rules for matrices to allow rows from other matrices or inlined vectors.
So, we could change this parsing rule:
"[" <matrix_value: CommaElems<Vector<BoundaryExpr>>> "]"

and instead define another enum for matrix rows where they could be identifiers (referring to declared vectors), vector access (referring to row in another matrix) or inlined vectors.
It's possible I'm missing other cases which don't get handled with this or maybe there's a better way to handle this.

Actually, I think this might complicate things because we will have matrices defined as vectors in the AST. For eg,
let a = [1, 2, 3]
let b = [4, 5, 6]
let c = [[1, 2, 3], [4, 5, 6]]
let d = [a, b, c[1]]

This would be defined as a vector at the parser level.

let a = [1, 2, 3]
let b = [4, 5, 6]
let c = [a, b, [4, 5, 6]]
This would be defined as a matrix at the parser level.

Then the IR could get complicated. So, if it makes thing easier, we should probably restrict the syntax here. Maybe we could make it so that matrix could either contain all inlined vectors or all expressions. So, something like `[a, b, [4, 5, 6]]` would be invalid. If users wanted to write a statement like this, they could just define another intermediate variable for `[4, 5, 6]` and use it that way.

@grjte Wdyt?

If we go with restricting syntax here, then we probably don't need to update the parser and only need to add some rules to your current PR.
I think we should support statements like these for now:
let a = [1, 2, 3]
let b = [4, 5, 6]
let c = [[1, 2, 3], [4, 5, 6]]
let d = c[0]
let e = [a + b for (a, b) in (a, b)]
let f = [a, b, c[0]]
let g = [a, b, d]
let h = [a, b, e]

Supporting non-nested list comprehension doesn't require any extra work for this PR since currently we just unroll it and store it as a vector in the symbol table.

Current approach a little bit differently fills variable_roots, and I'm not sure is this a problem or not.

@tohrnii @Overcastan sorry I didn't see the question above about restricting syntax earlier. Yes, I think this is fine, especially for this PR. If we want to allow the more flexible syntax that can be addressed separately as its own issue and PR


[5.0][stdlib][ABI] mark reduce(_:into:) __owned

Pick of #20839 for 5.0. Given the goal of this method is to mutate the initial parameter, it should take it owned since the most common usage is to give up an initial value (like []) to it.

@swift-ci please test


Notifications & notification area

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
Modern operating systems, starting with mobile systems such as Android and iOS, moving to desktop operating systems such as Windows, macOS, and Linux, and even game consoles such as the Xbox and Nintendo Switch have system notification systems. dahliaOS developers would benefit from being able to send notifications to their users.
Describe alternatives you've considered
I was considering a "user feed" that starts with notifications, then proceeds to weather, news, etc.
Additional context
There's already a non-functional "notification area" in the launcher. Perhaps we should use some sort of socket on Dahlia Linux and the native Fuchsia mechanism for Zircon.

Related: #148
I'll keep this issue open to track the progress of a full notification system (in the launcher, etc.) and system API for non-builtin apps that can't access the private Dart APIs.

Closed by https://github.com/dahliaOS/pangolin_desktop/commit/59dbe9db337149a4b737478d4a0a875743fd7ff3


Load llava error

When I try to load pretrained model from local dir, I set llava_groups["mistral"]=local_model_dir and find an error:ValueError: Trying to set a tensor of shape torch.Size([1024, 4096]) in "weight" (which has shape torch.Size([4096, 4096])), this looks incorrect.
Besides, how to set llava_groups["phi"], thanks for your reply.

When I try to load pretrained model from local dir, I set llava_groups["mistral"]=local_model_dir and find an error:ValueError: Trying to set a tensor of shape torch.Size([1024, 4096]) in "weight" (which has shape torch.Size([4096, 4096])), this looks incorrect. Besides, how to set llava_groups["phi"], thanks for your reply.

Sorry for the inconvenience these errors have caused you. I will be dealing with all issues regarding environment configuration in the next few days. And currently there is no need to set llava_groups["phi"].


Clean up solution regen and drain comps

Fixes #29755
About the PR

Fixes a bug in serialization by applying the suggested solution in the linked issue. I could not replicate the issue, but I don't have it after the change either, so, seems fine?
I also tidied up what datafields I could in the components without causing breaking changes.
Why / Balance

Technical details

Media

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

Breaking changes

Changelog

no cl no fun

Oh wait hold on, this needs more tape.

Fixed it


suggestions for improvement

hey guys, not sure how we could improve the repo more, do y'all have any suggestions?
maybe descriptions of algs, website, any good ideas are welcome

you can maybe upload basic algorithms and ask people for optimizations and improvement,
or may be reduce time complexities . just a thought #71

What if you could sort each algorithms based on time complexity?


gltfPack ： Error during model conversion

RuntimeError: memory access out of bounds,RuntimeError: memory access out of bounds

This issue needs more information:

Presumably this is using gltfpack from npm?
Can you specify the command line used in the conversion?
Most importantly, can you attach the model file used?

Since there's no actionable information here I'm going to close this. Feel free to reopen and provide more information requested above (also if this is using node.js I'd need the node version).


Fix URL to Travis CI badge

This makes the badge actually load and links to the builds page. Otherwise it says build|undefined and looks like this is a broken or unmaintained package.

thanks!!!!


can not create .gcmrc file

NERDTree: Node Not Created.

You may have tried to create an already existing .gcmrc file? Did you enable invisible files in your tree?

Not enough information has been provided here. I have tested this on Windows and I cannot replicate the given error message.  I will now close this issue... reopen if you can supply a clear and precise procedure for reproducing your problem.  Thanks!


[BUG] Previous tracks album stays put

Describe the bug
Previous tracks album stays put when skipping track/waiting for track to finish
To Reproduce
Steps to reproduce the behavior:

Play song
Skip to another
Observe

Screenshots
If applicable, add screenshots to help explain your problem.

Extreme example when an ad gets played

Desktop (please complete the following information):

OS: [e.g. iOS]: Windows 11
Version [e.g. 22]: 22463.1000
Spotify version: <IP_ADDRESS>2.g2b11de83

Logs
vendor~xpui.js:1 [Deprecation] Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
(anonymous) @ vendor~xpui.js:1
vendor~xpui.js:1 MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 change listeners added. Use emitter.setMaxListeners() to increase limit
    at c (https://xpui.app.spotify.com/vendor~xpui.js:29:2353172)
    at t.o.addListener (https://xpui.app.spotify.com/vendor~xpui.js:29:2355454)
    at https://xpui.app.spotify.com/vendor~xpui.js:29:2048746
    at Pa (https://xpui.app.spotify.com/vendor~xpui.js:29:2595327)
    at As (https://xpui.app.spotify.com/vendor~xpui.js:29:2619179)
    at t.unstable_runWithPriority (https://xpui.app.spotify.com/vendor~xpui.js:29:2793496)
    at hi (https://xpui.app.spotify.com/vendor~xpui.js:29:2564835)
    at ks (https://xpui.app.spotify.com/vendor~xpui.js:29:2618958)
    at Os (https://xpui.app.spotify.com/vendor~xpui.js:29:2603729)
    at vs (https://xpui.app.spotify.com/vendor~xpui.js:29:2602651)
    at https://xpui.app.spotify.com/vendor~xpui.js:29:2565107
    at t.unstable_runWithPriority (https://xpui.app.spotify.com/vendor~xpui.js:29:2793496)
    at hi (https://xpui.app.spotify.com/vendor~xpui.js:29:2564835)
    at yi (https://xpui.app.spotify.com/vendor~xpui.js:29:2565052)
    at vi (https://xpui.app.spotify.com/vendor~xpui.js:29:2564987)
    at Es (https://xpui.app.spotify.com/vendor~xpui.js:29:2603150)
    at rl (https://xpui.app.spotify.com/vendor~xpui.js:29:2629563)
    at render (https://xpui.app.spotify.com/vendor~xpui.js:29:2632674)
    at https://xpui.app.spotify.com/xpui.js:1:1034172
(anonymous) @ vendor~xpui.js:1
xpui.js:1 Uncaught (in promise) Error: GET request to sp://ads/v1/settings/leaderboard failed with status code 404
    at l.onSuccess (xpui.js:1)
    at onSuccess (xpui.js:1)
vendor~xpui.js:1 cosmos-connector:  fetchAdForSlot endpoint=%o sp://ads/v1/ads/hpto
pubads_impl_2021091501.js:6 [GPT] Invalid argument: PubAdsService.setRequestNonPersonalizedAds("1"). Valid values: 0,1.
(anonymous) @ pubads_impl_2021091501.js:6
pubads_impl_2021091501.js:6 [GPT] Invalid argument: PubadsService.setTagForUnderAgeOfConsent("0"). Valid values: 2,0,1.
(anonymous) @ pubads_impl_2021091501.js:6
ads-iframe-internal-click-tracking.js:6 Spotify Ads Iframe Click Tracking origin URL may have changed: https://xpui.app.spotify.com
adsIframeLog @ ads-iframe-internal-click-tracking.js:6
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/service-worker.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/vendor~xpui.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui-desktop-routes-homepage-takeover-ad.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/754.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui-routes-home.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui-desktop-modals.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED

Looks like I forgot to push the .js file in my latest commit. This should be fixed now

Yeah, just downloaded it and it seems to be fixed. Thanks!


TS Server fatal error:  Maximum call stack size exceeded

Type: Bug
❗❗❗ Please fill in the sections below to help us diagnose the issue ❗❗❗
TypeScript Version: 5.6.0-dev.20240812
Steps to reproduce crash

TS Server Log
❗ Server logging disabled. To help us fix crashes like this, please enable logging by setting:
"typescript.tsserver.log": "verbose"

After enabling this setting, future crash reports will include the server log.
VS Code version: Code 1.92.0 (Universal) (b1c0a14de1414fcdaa400695b4db1c0799bc3124, 2024-07-31T23:26:45.634Z)
OS version: Darwin arm64 23.5.0
Modes:

Please try collecting the TS Server log from the crashed instance:

Set "typescript.tsserver.log": "verbose"

Restart VS Code and reproduce the problem

Open the TypeScript section of the output panel

At the very top, find the first line that prints to path to the semantic error log file. It should look something like:
[Info  - 19:54:59.247] <semantic> Log file: /Users/matb/Library/Application Support/Code - Insiders/logs/20200213T104930/exthost55/vscode.typescript-language-features/tsserver-log-ZT2zau/tsserver.log

That file contains the typescript logs.

Look through that log file for the first error or stack trace you see. If you can share the log, I can also take a look to see if anything stands out

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this


JavalinMithril

Describe the feature
Mithril.js is a javascript SPA framework that comes with a tonne of out-of-the box features. It uses class based and function based components, that can be created without trans-piling or needing  any sort of build chain. It is also wicked fast, faster than vue or react, due to the way it works. The idea is to have something similar to JavalinVue, using Mithril.js as the framework, and doing server-side routing and state injection. We had a discussion about this with @tipsy  on the Javalin Gitter channel, and it seems like a good idea on paper - pending of course, seeing how it works.
Additional context
the implementation is going to be more opinionated than the implementation in JavalinVue, due to the nature of the framework. While JavalinVue guarantees that components are unique, since they use DOM ids and Tag Names, no such guarantee exists in class-based components. So the suggestion we figured out is to use a @import and @package directives at the top of a mithril class file, and use those in order to do dependency resolution, and transform the classes. so a class LoginForm in the package me.kisoft.component would be effectively transformed into the JS Class me_kisoft_component_LoginForm and all references would be replaced in the produced Code. Additionally, we might look at JIT minification of the produced files, just for the sake of making it further optimized. This all introduces interesting challenges, but i think that this has its place in Javalin, because we are all sad sad backend bois who dont like Mainstream JS workflows
However, this means we lose the SPA nature of Mithril.js, which might not be ideal for some, but we gain the simplified workflow we know and love from JavalinVue

I and doing the work on a sepersted repo, for now, until we figure out where to put it.
https://github.com/TareqK/javalinmithril

Since you're basically done, and we've made a repo, I'll close this. Nice work :)


Proposal: R.catchP

I propose a function, catchP, for attaching error handling to promise chains.
const catchP = invoker(1, "catch");

const getJSON = R.composeP(
    doSomething,
    R.catchP(handleError),
    fetch
);

It seems to invite side-effects, so I don't know if it's actually appropriate for compositions. Promise.catch is an important part of promises, however, which may justify the existence of catchP beyond userland (https://github.com/ramda/ramda/issues/1528#issuecomment-158788298).

Thanks for the proposal, @neftaly.
I'd actually like to go in the other direction, and remove the existing Promise-specific functions from Ramda. Promises are flawed. See plaid/async-problem for a better data type. ;)

Fair enough, I definitely prefer approaches like core.async.


Fix headline formatting in README

Github changed the parser, headlines need a space after ## now.

I've noticied, but haven't got the time to do it.
Thank you!


#38 一定数以上のレコードがあるデータのみに絞る機能の追加

Codecov Report

Merging #39 into develop will decrease coverage by 0.46%.
The diff coverage is 20.00%.

@@             Coverage Diff             @@
##           develop      #39      +/-   ##
===========================================
- Coverage    94.57%   94.11%   -0.47%
===========================================
  Files           27       27
  Lines          793      798       +5
===========================================
+ Hits           750      751       +1
- Misses          43       47       +4

Impacted Files
Coverage Δ

kabutobashi/io/__init__.py
87.17% <20.00%> (-9.88%)
: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 dc0c799...7e8359a. Read the comment docs.


[ie11] Nor working on button element

Hi,
It seems that doesn't work on button element on IE11
The issue occurs also on example page, you can test directy.

Commit 3f9ff39 solved this Issue.


Recomendations for tabline/bufferline to use with this?

Spaceline is in my opinion the best statusline for vim. Lighter than lightline and almost as powerful as airline.
I have been struggling to find a decent bufferline/tabline for vim, the best looking one I found was vim-buffet

but when you use it with termguiset it looks like this.

I was wondering what is used by you and the people who contribute to this project use because all of the bufferline/tablines look like an eyesore paired with spaceline.

hmm this is my fork of vim-buffet https://github.com/hardcoreplayers/vim-buffet .

Thank you! I played around with your fork. I replace all instances of let vim_mode = "cterm" with let vim_mode = "gui" and it looks gorgeous. I suspect if you change all instances of if has("gui") to if has("gui") || if has("termguicolors") it would work with termguicolors set


libretro: add thepowdertoy core

Motivation for this change
This adds the The Powder Toy libretro port as a retroarch core.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Looks to be gpl3Only - no relevant mention of "or later" in the source that I can find, and the libretro Discord agrees.


[Question] Can you download users who liked tweets mentioning 'x'?

Description
I'd like to download a list of users who have liked tweets that have, say, mentioned "Brad Pitt" as opposed to users who simply mentioned "Brad Pitt" (along with a link and the tweet they liked)
Is this possible?

There's a limitation imposed by Twitter will allows you to see only the first 25 users that liked a given tweet. This is not suitable in many cases


Possible to omit all traces at specified urls

Introduced a new tracing policy that ignores all traces occuring at
designated paths before sampling occurs to prevent ignored urls from
being sampled.
Fixes #103.

@ofrobots PTAL.

This patch assumes that we use the request url as the name for each root span. This is currently the case for all web frameworks we patch. We would have to refactor the hooks api to expose this information independently of the name. I'm happy to do this if preferable.

@ofrobots Ready for another look.

Retroactive LGTM.


Use C template macros in _c.h.tmpl

As for #216 but for the header template.

Closed in commit https://github.com/FFIG/ffig/commit/fc9b0f625913cc2e97e8e724d87b18efa648a8ea


simple rollback doesn't seem to roll back transaction

As I understand the docs, this code should not print any entries
because the added entries were part of a transaction that is
rolled back. But it actually prints x=y - the Set operation
has actually been committed.
What am I doing wrong?
package main

import (
	"fmt"
	"io"
	"log"

	"github.com/cznic/kv"
)

func main() {
	db, err := kv.CreateMem(&kv.Options{})
	if err != nil {
		log.Fatal("create:", err)
	}
	if err := db.BeginTransaction(); err != nil {
		log.Fatal("begin transaction:", err)
	}
	if err := db.Set([]byte("x"), []byte("y")); err != nil {
		log.Fatal("set:", err)
	}
	if err := db.Rollback(); err != nil {
		log.Fatal("rollback:", err)
	}
	if err := printKeys(db); err != nil {
		log.Fatal("print keys:", err)
	}
}

func printKeys(db *kv.DB) error {
	enum, err := db.SeekFirst()
	if err != nil {
		return err
	}
	for {
		k, v, err := enum.Next()
		if err == io.EOF {
			break
		}
		if err != nil {
			return err
		}
		fmt.Printf("%s=%s\n", k, v)
	}
	return nil
}

Thanks for the report. Fix follows. Please note that afterwards you'll have to update also lldb.
go get -u github.com/cznic/kv should do it.

More changes and testing reveal that the problem was not fixed properly.

@rogpeppe Sorry for the mess, however, I believe now it's finally correct. Please let me know if not.


add ability to tab to autocomplete on attributes

check if other inputs need the same fix

works only when on predefined list and having exactly one result. e.g. works for:

but not for:

you instead have to down arrow and enter
would be nice to have it be consistent


Add support to Unreal Engine 5.1 (Currently in Preview 1)

✅ Working with the current UE5.1 Preview 1 ver.
Branch: https://github.com/lucoiso/UEEOSSDKHandler/tree/UE5.1

Currently working on both UE5.0 and UE5.1 and being used in https://github.com/lucoiso/UEProject_Elementus/tree/feature/EQUIP-26


Distribute Managed Google Play private app to Intune

As a user of App Center I would like to be able to distribute Managed Google Play private apps to Intune.
Currently this is not supported only Android Line Of Business (LOB) to Intune but this is for Enterprise devices not useful.

Bump

Bump

Bump

Adding Bump.

Bump

I am still facing the same, please  I would like to be able to distribute Managed Google Play private apps to Intune.
Currently this is not supported only Android Line Of Business (LOB) to Intune but this is for Enterprise devices not useful.

When can we expect the solution?

Bump

This would be great to have or a documented way to upload to Managed Google Play and submit the link to InTune to add to the Corporate App Portal.

same here

+1

+1

+1


requires-python key in pyproject.toml must consider py3.7 toolchain.

project init
rye init proj --py 3.7.9 --no-readme
pyptoject expected:
requires-python = ">= 3.7"
pyproject actual:
requires-python = ">= 3.8"

I'm aware that we can config that by config.toml or --min-py cli option, but default version behavior when providing toolchain on init, could have improvement.
This also would prevent version conflicts like https://github.com/mitsuhiko/rye/issues/82

possible fix from https://github.com/mitsuhiko/rye/issues/82#issuecomment-1546894668:
> rye init proj --py 3.7.9 --no-readme
new toolchain has conflict with project's required python ">=3.8" - auto fix it? (y/n) > y

pyproject.toml:
requires-python = ">= 3.7"


UI scaling

🐞 bug report
Describe the bug
To me UI scaling does not work. Playing on 4k. UI is to small. Ingame item-evaluate as well as Menu.
If I read the change logs correctly this would have been added on march 15.
🌍  Your Environment

OS: Win10
Version: 0.6.9 non-portable
PoE: Steam 3.10.0 English Windowed Fullscreen

Hi, thanks for creating an issue. The UI does not scale automatically. There is a zoom settings you'd need to modify in the settings menu.

Thanks, this solved the "issue".

Awesome! Closed.


Attachment download with duplicate filename: adding number after extension prevents third-party file manager from recognising file type

K-9 Mail version 5.806 on Android 10.  Opened an email with an MP3 attachment, filename is recording.mp3, pressed ⤓ to download, saved as recording.mp3, tapped on it in Speed Software's Explorer v3.10.2 which opened it in VLC 3.3.4, so far no problem.  Back in K-9, opened a second email which also had an MP3 attachment, and the filename was also recording.mp3, pressed ⤓ to download, this time the suggested name was "recording.mp3 (1)" which I accepted without thinking, and now Explorer didn't recognise it as a file that should be opened in VLC until I manually renamed it to "recording (1).mp3" i.e. the "(1)" should come before the extension to make file managers still recognise the file type.
I did a quick search through the code to see if I could find which part handles the (1), (2) etc suffix, but it seems my 'grep' skills were not quite good enough to find it.

K-9 Mail is using system functionality to save attachments. It passes the original file name to the system and is not involved in the handling of duplicate file names at all. The Android versions on my devices add a number before the file extension, not at the end of the file name. If that doesn't work on your device, it's a bug in the Android version you're using.

Thanks.  For the record (in case anyone is finding this via search), I'm on a Galaxy S9 (SM-G960F), OneUI 2.5, Android 10.  I seem to remember this behaviour did not occur a few months ago (when it did insert the number before the extension) so it must be a recent regression.  Or at least, I don't remember having this exact problem a few months ago, so if Android has not changed its file naming behaviour then perhaps Explorer has lost the ability to open files that have numbers after their extensions, although I can't think why they would drop such a feature.
Also I was slightly incorrect in my description: K-9 only prompted me to save the second recording as "recording.mp3", it was the system that then changed it to "recording.mp3 (1)" but not on-screen (I had to go into Explorer to see it had done this).


Fix for handling SqlBulkCopy options and some bug fixes for BuildMappings.

Handle passing options to BulkWriter and some fixes for BuildMappings.
BuildMappings: shouldMap invert the condition and null-check name/order when column attribute defined but no explicit value for name and order set.
Sorry for combining two issues in one pull request.

Merge conflict :(

I will close this and create a new one.

Thanks! Eagerly waiting for a new release :smile:


Add cosn UFS to provide better ability to read and write cos

Is your feature request related to a problem? Please describe.
COS has poor read and write performance and instability.Compared with COS, COSN is deeply optimized, and its read and write performance is doubled compared with COS, and it also brings better stability.
Describe the solution you'd like
Add CosN UFS
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Urgency
Will be helpful if this is taken on priority.
Additional context
Add any other context or screenshots about the feature request here.

Add cosn UFS to provide better ability to read and write cos #12493

#12493

Add cosn UFS to provide better ability to read and write cos #12493

Add cosn UFS to provide better ability to read and write cos #12493


url-parser: parse HTTP body to _httpRequest

This mirrors the behaviour of expressjs/body-parser and allows the body data to be passed on to error handling if necessary.
Caveat: the request._httpRequest object is changed (body property is added), but if it already exists it is not overwritten.

Should be fine, I think (though it's a little bit redundant for JSON bodies) :-)

Should be fine, I think (though it's a little bit redundant for JSON bodies) :-)

It is (though only for JSON bodies, as you said), but we need an object with (only) the original POST variables. Since flora mixes the JSON keys with the other Request properties, we cannot separate later.


Chem: RDKit minimal lib - R-groups

Datagrok uses RDKit minimal lib for cheminformatics related calculations. Minimal lib is limited to a subset of features presented in RDKit which is usually wrapped and used in Python. Some more features will be useful in the minimal lib. One of them are features essential for R-groups analysis. In Python RDKit they are used as:
Chem.ReplaceCore
Chem.GetMolFrags
Implement these both functions RDKit minimal lib using c++
Add this functionality to JS wrapper and to CFFI wrapper

Implement RDKit Python functions
Chem.ReplaceCore
Chem.GetMolFrags
in RDKit minimal lib
This functionality has to be added to JS Minimal Lib and needs to be implemented in CFFI library

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-14844


whye error occurs

let anyValue:any="mmm";
<string>anyValue="abc";

Type assertion should put before the value you want to assign.
let anyValue: any = "mmm";
anyValue = <string>"abc";

@ikatyang
但是使用分组运算符()之后就正确了
let anyValue:any="mmm";
(anyValue)="abc";
上面的代码不会报错

It is valid even without the type assertion, what do you want to do?
let anyValue: any = "mmm";
anyValue = "abc";

@ikatyang
I have modified the code:
let anyValue:any="mmm";
(<string>anyValue)="abc";
```
the code above is valid，just enclosed in < and >
why？

OK, I see. It seems there's something wrong with the parser for the type assertion.
<string>anyValue1 = "abc"; should be either (<string>anyValue1) = "abc"; or <string>(anyValue1 = "abc");, but none of them are considered.
TS version: 2.4.2
(source)
// case 1
let anyValue1: any = "mmm";
<string>anyValue1 = "abc";
//                ^ [ts] ';' expected.

// case 2
let anyValue2: any = "mmm";
<string>(anyValue2 = "abc");

// case 3
let anyValue3: any = "mmm";
(<string>anyValue3) = "abc";

(generated)
// case 1
var anyValue1 = "mmm";
anyValue1;
"abc";

// case 2
var anyValue2 = "mmm";
(anyValue2 = "abc");

// case 3
var anyValue3 = "mmm";
anyValue3 = "abc";

@ikatyang thank you very much 非常感谢台湾朋友

Please use Stack Overflow for questions


Implement the Aliasize feature.

Related issue:https://github.com/github/orchestrator/issues/991
Description
The Aliasize feature add an icon in the navigation bar.  This icon
allows to swap displaying hostname:port and alias in topology
display (similar to the Anonymize feature).

Thank you. I changed some names (aliasized -> aliased, aliasize -> alias).


[top, dv] Fix rom backdoor symbol overwrite

2 commits in the PR
[top, dv] Fix rom backdoor symbol overwrite
Fix 2 issues

used correct backdoor read/write functions for rom
re-generated and updated rom digest after backdoor overwrite, rom
digest check can't pass

[flash, dv] Fix RMA test backdoor symbol overwrite
Changed to randomize the data and moved the overwrite function invocation to cpu_init
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

@davebwil
With this fix, I can see the kLcRmaUnlockToken is overwritten correctly.

 You will see it even If you run without this fix.
UVM_INFO @<PHONE_NUMBER> ps: (sw_logger_if.sv:516) [flash_rma_unlocked_test_sim_dv(sw/device/lib/testing/flash_ctrl_testutils.c:27)] Transaction end error_codes = 1
UVM_INFO @<PHONE_NUMBER> ps: (sw_logger_if.sv:516) [flash_rma_unlocked_test_sim_dv(sw/device/lib/testing/flash_ctrl_testutils.c:27)] Transaction end error_codes = 1
UVM_ERROR @<PHONE_NUMBER> ps: (sw_logger_if.sv:518) [flash_rma_unlocked_test_sim_dv(sw/device/tests/sim_dv/flash_rma_unlocked_test.c:92)] CHECK-fail: !flash_ctrl_testutils_erase_and_write_page( &flash_state, address, kPartitionId, data, kDifFlashCtrlPartitionTypeInfo, kDataSize)
UVM_INFO @<PHONE_NUMBER> ps: (sw_logger_if.sv:516) [flash_rma_unlocked_test_sim_dv(sw/device/lib/testing/test_framework/test_status.c:34)] FAIL!
UVM_INFO @<PHONE_NUMBER> ps: (sw_test_status_if.sv:51) [tb.u_sim_sram.u_sim_sram_if.u_sw_test_status_if] SW test transitioned to SwTestStatusFailed.
UVM_ERROR @<PHONE_NUMBER> ps: (sw_test_status_if.sv:75) [tb.u_sim_sram.u_sim_sram_if.u_sw_test_status_if] ==== SW TEST FAILED ====


Give graduated feedback for syntax errors.

In an interview setting, students are not going to get feedback that a syntax error has happened "on line X, column Y". To be fair, such syntax errors probably don't matter too much anyway. However, in TIE, the code has to be syntactically correct so that it can run and so that feedback can be given.
A possible middle ground might therefore be to do the following: when a syntax error is found, highlight the line that contains it in red, and say something along the lines of "there's a syntax error here; please find it and fix it", while offering an optional popup hint that shows the exact error and line number. That way, if a student wants to practice finding syntax errors on inspection, they have the opportunity to do so; if they don't want to practice this, they can just open the popup.

A few questions:

Why do we need to highlight the line, instead of letting students find the syntax error by themselves?
Is there a specific reason to use a popup?

For (1), we could do either. It might be worth getting some informal feedback on this; what I proposed was just a suggestion.
For (2), no; it's one way, but there may be others. The main thing was that I wanted the student to be in control of deciding whether to take the hint or not.

Current design and implementation as follows:

Initial view

Run the code with syntax error

take hint

hide hint

No CSS is applied on the button now. I talked with others and it seems acceptable. Any comments? @seanlip @rabidbit

The functionality seems fine to me, except that I'm not sure why you'd want to hide the hint after showing it. Deferring to @rabidbit for design review.

Some feedback:

Change the the button to a link with small font ("Display error details" / "Hide error details"); we want to use buttons very sparingly and try to avoid embedding buttons into the text windows. Append the link inline after the "Try to find and fix it." sentence (as opposed to a new line).
Hide the error messages by default, but once the user toggles it one way or the other, hide or unhide all error message details (so the user won't have to click on links every single time there is an error to display details).

I agree with the first one, but for the second, imagine this scenario:
There might be several syntax errors at different lines. Our functionality can only show the first syntax error it met.
The student wants to practice checking for syntax error, but didn't manage to find it. So he or she will click for error details. Then he or she fixed the first syntax error following the error detail, and run the code again(he/she will assume there are no other syntax errors). The result is that those other errors will be displayed after the run, even though the student doesn't want to look at them. Am I correct?

I think I understand what you mean, and I thought about that. My guess is that the user will see the error messages when they didn't want to, maybe 1-3 times. Then, they will learn that they need to click "Hide error details" when they are done looking at an error message if they don't want to see error details going forward. This would be mildly annoying. If, however, the user has to click to see error details for every single error, then they are forced to click multiple times, whether they like it or not. I suspect this might be more annoying because you may be forced to click on links many times (versus remembering to click a few times). Does that make sense?

Yes I admit that it must be more annoying for the user to click many times, but that fits more to an interview setting, doesn't it? Also, my assumption is that users usually won't have a large number of syntax errors, so they might don't need to worry about "too many" clicks.

That's a good point. Though I would not assume users won't have a large number of syntax errors. I've had interviews where the candidate came up with a solid implementation but wrote with a ton of syntax errors. Also, right now we only have Python, and not everyone is familiar with Python; so, I can see where hiding syntax errors might not be that useful when the user cares more about solving the problem than getting the syntax right. In short, I can see where users will run across a lot of errors, which increases the odds of many clicks. Ultimately, things like this will have to be user-tested, but this is my thinking for the initial approach.


How to access sqlite in our BDD test cases

Trying to Add dbhelper in my project
not able to install database-js sqlite
terminal stuck at
PS C:\ws\N_TEST\NET_Codecept_Automation\NET_UI_Codecept_Automation> npm install --save-dev database-js-sqlite
[..................] - fetchMetadata: sill resolveWithNewModule<EMAIL_ADDRESS>checking installable status
# paste output here

Provide test source code if related

// paste test

In my project, I have SQLite DB. I want to verify some data from database also. So I want to integrate sqlite in my test cases. for that I m following below link
https://github.com/thiagodp/codeceptjs-dbhelper
but not able to install database-js sqlite

CodeceptJS version:
NodeJS Version:
Operating System:
Protractor || WebDriverIO || Nightmare version (if related)
Configuration file:

# paste suite config here

@sumiagar you already asked in the right repo at, in the current repo please only report bugs related to CodeceptJS core, thanks

@sumiagar Sorry, but I will close this issue because the problem is not in CodeceptJS =) Thank you!)


Video support

Support for loading and playing external video files

Note that this might be better implemented outside of the engine, in which case wontfix.


No need for a sorted sentinel

The loop in indiesort that handles the permutation of the elements of the array after the indirect sort uses a sorted condition to detect already sorted elements:
if (current_tuple->original_index != index && current_tuple->original_index != sorted)
{
    // ...
}

The second part of the condition is unneeded if you manage to assign the right values so that all sorted elements satisfy the first condition current_tuple->original_index == index. Boost.Algorithm uses such a trick in the apply_permutation algorithm. In plf_indiesort, it consists in changing this line:
sort_array[destination_index].original_index = sorted;

Into this one:
sort_array[destination_index].original_index = destination_index;

It can avoid a fair number of extra comparisons in some scenarios, and allows the algorithm to accept some exotic iterator types that have a difference_type not compatible with std::numeric_limits.
This permutation algorithm is really close from yours and is explained in details in a series of articles by Raymond Chen.

Thank you.


vue 3 compatibility?

Thx for reporting, I'll take a look tomorrow.

@MDolega20 Please try 0.3.6-beta.0

This is still the issue with v 0.4.0

@scholtz Oh, sorry, please try<EMAIL_ADDRESS>temporarily, that PR has not been released yet because @MDolega20 hasn't confirmed if it is working, can you try and confirm that?

<EMAIL_ADDRESS>produces new error.

I use https://github.com/scholtz/qrcode-vue3 for qr code generator and https://github.com/scholtz/qrcode-reader-vue3 for qr code scanner
npm install --save qrcode-reader-vue3
npm install --save qrcode-vue3

Published<EMAIL_ADDRESS>and<EMAIL_ADDRESS>via vue-demi.
They are still vue 2 and 3 compatible.

JounQin
I got this error
X [ERROR] Could not resolve "qrious"

    node_modules/vue-qrious/lib/index.js:1:19:
      1 │ import QRious from 'qrious';
        ╵                    ~~~~~~~~

  You can mark the path "qrious" as external to exclude it from the bundle, which will remove this
  error.

I guess it's dependency problem

I found that I have to install qrious dependency.. I am not sure why you made code like that..?
Is there any purpose? I guess it's better including denpency package.

It is a peer dependency, so that you can upgrade qrious without changing vue-qrious.

'{ value: string; width: number; class: string; }' is not assignable to type 'IntrinsicAttributes & Partial<{}> & Omit<Readonly<ExtractPropTypes<{ version: { type: NumberConstructor; validator: (version: number) => boolean; }; ... 9 more ...; value: { ...; }; }>> & VNodeProps & AllowedComponentProps & ComponentCustomProps, never>'.
Type '{ value: string; width: number; class: string; }' is missing the following properties from type 'Omit<Readonly<ExtractPropTypes<{ version: { type: NumberConstructor; validator: (version: number) => boolean; }; errorCorrectionLevel: { type: PropType; validator: (level: QRCodeErrorCorrectionLevel) => boolean; }; ... 8 more ...; value: { ...; }; }>> & VNodeProps & AllowedComponentProps ...': type, color


IContextMenuOptions Warning

I'm getting several warnings about a missing IContextMenuOptions, but I'm not too sure of why I'm getting it. Is this module necessary, or should I remove the reference in ./contextMenu.options?

Those are warnings that you can safely ignore.  But those are only logged for me when I compile the library itself - not when I consume the library in an app.  How are you importing the library?  Are you doing import { something } from 'ngx-contextmenu'?

Oh okay, good to know!
This is how I'm importing it in my app,module.ts:
"import { ContextMenuModule } from 'ngx-contextmenu-master/src/lib';"
I used that file path (ngx-contextmenu-master/src/lib) because I get a compile error when I reference 'ngx-contextmenu'.
Thank you!

These warnings will go away in version 2.0.0-beta.1


MudMenu and MudSelect rendering multiple popovers

Bug type
Component
Component name
MudSelect and MudMenu
What happened?
After upgrading to 5.2.0 I am having an issue with the MudSelect and MudMenu components. They are producing double popovers. One seems to be anchored to the component and another that seems to be anchored to the top MudAppBar.
Expected behavior
The expected behavior is that the MudSelect and MudMenu open with only one popover that is anchored in the correct location with the specified width

Reproduction link
https://try.mudblazor.com/snippet/QYGbFbudKouNVGIb
Reproduction steps

Navigate to page with component and activate it
Component then displays a popover anchored to the component and another one that is anchored to something else and is the whole width of the page

Relevant log output
none

Version (bug)
5.2.0
Version (working)
5.1.5
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Are you maybe using  multiple times?

You have more than one <MudThemeProvider\>

I have an issue with that in the past when another dev added it to the MainLayout.razor when it already in App.razor. I'm not sure why it is happening in the try blazor example I posted.
https://try.mudblazor.com/snippet/QYGbFbudKouNVGIb

That was the issue. There was one in MainLayout.razor and one in App.razorThis situation has existed in our application for several months and did not present itself until the upgrade to 5.2.0. I spent several hours looking for other causes and was able to reproduce it on the try.mudblazor.com site and the issue went away when I downgraded to the previous version.
Why am I able to reproduce it here? https://try.mudblazor.com/snippet/QYGbFbudKouNVGIb
I apologize for wasting anyone's time because I believe I have seen this some place before and I should have checked.

Why am I able to reproduce it here? https://try.mudblazor.com/snippet/QYGbFbudKouNVGIb

I think the problem is that try.mudblazor adds a ThemeProvider for each .razor file. We should fix this

I apologize for wasting anyone's time because I believe I have seen this some place before and I should have checked.

No worries :)


Add support to disable S3 Express session auth

Motivation and Context
Adds the ability to disable S3 Express session auth (causing it to use a regular sigv4 session auth instead).
Description
S3 Express One Zone is an opt out feature, and there are three ways to disable it (with the order of precedence as listed):

through the disable_s3_express_session_auth method on an S3 client
through an environment variable AWS_S3_DISABLE_EXPRESS_SESSION_AUTH
through a profile file with a key s3_disable_express_session_auth (won't be supported yet in this PR until https://github.com/awslabs/aws-sdk-rust/issues/1060 is implemented)

If one of the places is set to true/false, then the subsequent places with lower precedence will not be considered.
Something to be aware of when setting the disable option through the environment variable. The environment variable is only checked during a client construction, meaning that if a customer sets it after the client has been created the SDK will not take the environment variable value into account, i.e. the following snippet will NOT disable the S3 Express session auth:
let config = aws_config::load_from_env().await;
let client = aws_sdk_s3::Client::new(&config);

// Set the env variable to true after an S3 client has been created
std::env::set_var("AWS_S3_DISABLE_EXPRESS_SESSION_AUTH", "true");

let _ = client
    .list_objects_v2()
    .bucket("s3express-test-bucket--usw2-az1--x-s3")
    .send()
    .await;

Testing
Added unit tests for S3ExpressRuntimePlugin and integration tests for verifying disabling S3 Express session auth.
Tests for a profile file will be TODO until the said work is implemented.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will re-create a new PR due to many merge conflicts at this point. Will address review feedback in the new PR.


Add GDX Snowplow tracker

link to demo deployment of your smk based app is
https://smk-firewood-rt-b16795-dev.apps.silver.devops.gov.bc.ca
Verify that it looks good, if so merge the pull request which will trigger a prod deployment, otherwise close the pull request without merging
mentions / notifications: @GuyTheFlower, @NicoledeGreef, @franTarkenton

The Vanity URL for the dev deploy is https://smk-firewood-api-gov-bc-ca.test.api.gov.bc.ca
If this is a dev deploy,  verify that it looks good, then merge the pull request to trigger prod deployment.
If this is a prod deployment, view your prod deploy at the url above :)
mentions / notifications: @GuyTheFlower, @NicoledeGreef, @franTarkenton

Closing this. There's one more analytics tracker change to test before putting this into production so I'll create a new PR for it.


Get-UICulture returns wrong result

Steps to reproduce

On Microsoft Windows 10 Home 10.0.18363 Build 18363
Set Settings > Windows region
County or Region Ireland
Regional Format Recommended English Ireland
Language English (Ireland)
On powershell 7.0.3
Run Get-UICulture
Result is
LCID             Name             DisplayName

2057             en-GB            English (United Kingdom)
Expected
6153            6153               English(Ireland)
This is confusing as I set the region to Ireland (in the EU) during PC initial set up.
I'm a bit worried that its detecting a soon to be non-eu country for the UICulture.
As I'd expect values like that are looked at for what data processing laws are followed.
The region as Ireland before I installed Powershell 7
Expected behavior
Run Get-UICulture
Result is
LCID             Name             DisplayName
6153            6153               English(Ireland)
Actual behavior
Run Get-UICulture
Result is
LCID             Name             DisplayName
2057             en-GB            English (United Kingdom)
Environment data

Name                           Value

PSVersion                      7.0.3
PSEdition                      Core
GitCommitId                    7.0.3
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
OS Settings
Settings > Windows region
County or Region Ireland
Regional Format
Recommended English Ireland
Language
English (Ireland)
Windows
OS Name	Microsoft Windows 10 Home
Version	10.0.18363 Build 18363

Note that it isn't the region that determines what Get-UICulture returns, but the Windows display language - have you changed that too? (Settings > Time & Language > Language).
Generally, note that Get-UICulture essentially just calls [cultureinfo]::CurrentUICulture, so if there is an actual problem, you'd have to report it in https://github.com/dotnet/runtime (and it could hypothetically  turn out to be a WinAPI problem in the end).

My language settings have not changed recently. If I go to Settings > Time & Language > Language
My windows display language is English (Ireland)
My Preferred languages in order are
English (Ireland)
English ( United Kingdom)
Gaeilge
日本語 (Japanese)

Close as stale issue. Feel free to follow @mklement0 suggestion.

Close as stale issue. Feel free to follow @mklement0 suggestion.


Question: Why was GLFW not used as the base underlying API?

Hello! This is more of a discussion question, but Discussions aren't enabled on this repository. So I thought I would post an issue here clearly marked as a question. 🙂
My question revolves around the reasons why GLFW was not chosen as the base window API. The README states (enumerated here for reference):

GLFW via LWJGL, SDL2:

Game-oriented
Bad window management
No OS integration
Create one window and go full-screen is the main use-case

(1) is true regarding the main users (I think?), but it doesn't seem overly represented in the API that I can see.
(2) There is no window management in GLFW, so it's not necessarily bad. It just allows you to come up with your own. Do the various platform-specific window APIs come with window management? Don't you have to create your own cross-platform window management anyway?
(3) What OS integration is GLFW missing?
(4) This relates to (2), but there is certainly the ability to create multiple windows and your own window management.
My reason for asking is here is that I am embarking on a project that does cross-platform windowing and drawing (not in JVM land), and so I am genuinely curious why GLFW wasn't picked for JWM. I am currently using GLFW, and I don't see any issue or limitation with it, at least not yet. My assumption would be that one would basically have to rewrite the functionality of GLFW anyway to do cross-platform windowing. So I'd like to better understand where my assumptions are wrong and fill in pieces of my knowledge of what the platform-specific APIs and custom wrapping them might provide over GLFW. I do not have deep familiarity with the platform-specific window APIs (Win32, Cocoa, X11, Wayland).
Thank you so much for any information you can provide! I understand any information I get is a freebie, so I very much appreciate it.

Well, for my use-case (cross-platform UI framework) I needed:

Metal for macOS
Window management
Tight OS integration (e.g. notifications, native fullscreen, window styling, menubar, accessibility, IME, color management etc)
Handling of HiDPI was different between macOS/Windows
I think creating multiple windows on different monitors with different refresh rate was an issue?

See for yourself, maybe if you don’t need all that, it might work for you. After all, it’s a well-know and used library

@tonsky About the refresh rate on different monitors, what do you think is the intended behavior of JWM, when one is 60Hz, and another is 144Hz (very popular for 3rd-party displays recent days).

@bumfo window should re-render at native monitor refresh rate, synchronized with vsync. From my experiments, only macos and wayland support different refresh rates on different monitors, both windows and x11 just set the same rate IIRC

@bumfo window should re-render at native monitor refresh rate, synchronized with vsync. From my experiments, only macos and wayland support different refresh rates on different monitors, both windows and x11 just set the same rate IIRC

Do you mean JWM already supports different refresh rates on macOS, what a good news for me ;). Btw, is that possible to support this as well on windows & x11, e.g. do you know any frameworks that does this correctly.

No, I mean Windows and X11 themselves don’t support different refresh rates

Yes macOS should work already

Thanks for all the information here! Just some small follow-ups, and I'll go ahead and close this issue so that it doesn't clog up your actual issue pipeline.

Metal for macOS

For Skia, it is my understanding that Metal can be had through Vulkan and Molten. Or are you needing or preferring direct access to Metal or using something other than Skia? (Or I guess not wanting to be bound to Skia.) For my uses, I simply can't go any lower than Skia (or some as of yet unknown alternative), so I'm happy to let Skia handle everything (if it can, right now hitting some issues).

Tight OS integration (e.g. notifications, native fullscreen, window styling, menubar, accessibility, IME, color management etc

Is the idea that GLFW doesn't provide these, intentionally of course, and so you would need to write platform-specific wrappers and a generic library anyway for all those needs, so why not just extend such effort to windowing as well?
Thanks again for any information and advice. Even with Skia and GLFW, I've had a lot of issues, but none of them with the actual GLFW library. Currently trying to figure out Skia (namely SkiaSharp) and possible OpenGL driver issues.


FIX #2663 reduce calls to the driver in ls

Current code needs 5 calls to the driver to collect the state, the url and the docker version.
This PR creates a fast path that needs only 2 calls for a sane running machine.
Worst case will require 3 calls.
Signed-off-by: David Gageot<EMAIL_ADDRESS>

\o/
LGTM


Allow cleanup of only locks

I recently had an issue with a few of my jug processes being killed and therefore locks being kept on some active tasks. Normally, I would just delete the lock files in the jugdir, but I was using the Redis backend, so this was not possible. As suggested in the docs, I ran jug cleanup, expecting only the locks to be cleared; I was surprised to find the completed jobs cleared as well.
Is there any way to have jug cleanup only clear the locks? If not, can there be an option for this?

There is jug cleanup --only-locks
Normally, jug cleanup should only clear old files. This is a bug with the redis backend!

I just released 2.2.1 which fixes this.

Thanks so much for your incredibly rapid response!


Chats Controller Added

Controller for the private user chat created and added

Solved #56

@all-contributors please add @MajidKhanBurki for code


Android:- Audio shuts down when move the app to background state

Describe the bug
When we put the app in background during the videocall the audio shuts down in background state
To Reproduce
Steps to reproduce the behavior:
During the videocall keep the app into the background and you will find the audio shuts down and publisher and subscriber are not able to hear anything as the audio is completely disabled in background state.
Expected behavior
The audio should be enabled in background state as in case of iOS the audio is enabled in the background state of the app.
Device (please compete the following information):

Android SDK version: 2.27.1(opentok )
OS and version: Android-12, Android-11

Additional context
Incase of iOS enabling audio in background state can be achieves simply by adding the Background State: Audio capability to app Info.plist but in case of android there is no such thing that can help to enable audio in the background state.

@tusharsharma718, there should be audio. Which sample app did you use to reproduce this?

@v-kpheng  I am using the following sample app to reproduce it :-
Path of sample app:-  opentok-android-sdk-samples/Basic-Video-Capturer-Camera-2-Java

@tusharsharma718, there should be audio. Which sample app did you use to reproduce this?

@v-kpheng Any update on the issue?

@tusharsharma718, apologies for the belated reply.
You should have audio.  We'll investigate.  That said, are you seeing the same issue with other sample apps as well?  Also, which model phones did you use to reproduce?

@tusharsharma718, we couldn't reproduce the issue, even when using https://github.com/opentok/opentok-android-sdk-samples/tree/main/Basic-Video-Capturer-Camera-2-Java.  I think we might be missing a step.  Any ideas?

For context, we tried to reproduce using a Galaxy M51.

@tusharsharma718, we couldn't reproduce the issue, even when using https://github.com/opentok/opentok-android-sdk-samples/tree/main/Basic-Video-Capturer-Camera-2-Java. I think we might be missing a step. Any ideas?

@v-kpheng ,apologies for late reply.
Thanks for providing the input and I tried to test on other devices and it was working fine but it doesn't work on my android device . Given below is the configuration of my android device in which I am facing the issue :
OS and version: Android-12
Device- Samsung Galaxy S10

@tusharsharma718, we couldn't reproduce the issue, even when using https://github.com/opentok/opentok-android-sdk-samples/tree/main/Basic-Video-Capturer-Camera-2-Java. I think we might be missing a step. Any ideas?

@v-kpheng ,apologies for late reply. Thanks for providing the input and I tried to test on other devices and it was working fine but it doesn't work on my android device . Given below is the configuration of my android device in which I am facing the issue :
OS and version: Android-12 Device- Samsung Galaxy S10

@v-kpheng  Any update on this issue?

@tusharsharma718, sorry for the belated reply.  No one on the team has an S10.  The closest(?) we have are S20 and S21.
In any case, can you share some logs?  Maybe it'll shed some light 🤞
Thanks!


Fixed apparent bug with sorting for best fitness

Firstly, thanks for writing this!  I love the way you've laid out the Phenotype trait.
I had difficulty getting my GA to work correctly; the issue appears to be a bug in the MaximizeSelector structure implementation, which sorts elements based on ascending fitness [0].  This should be sorting based on descending fitness ([1] indicates that a better fitness should be > than a lesser fitness).
An alternative resolution for this issue is to just add cloned.reverse() but this should be more performant, though less obvious.
[0] - https://doc.rust-lang.org/std/vec/struct.Vec.html#method.sort_by
[1] - http://m-decoster.github.io/RsGenetic/doc/rsgenetic/pheno/trait.Fitness.html

Thanks!
That is a pretty bad error on my part. Thank you for contributing.

@m-decoster thank you for the quick review and collaboration!


New input AST node: EIsNull

#318 introduced a new node for null buffers, EBufNull. However, nothing was provided in the input AST (as opposed to the AST) to compare a pointer to NULL: polymorphic comparisons seem to be impossible in the input AST.
Thus, this PR introduces a new node for comparing a pointer to null, EIsNull.

Responding here for future reference rather than on the private Slack thread.
You do not need a new EIsNull node: it suffices to generate EApp (ETApp (EOp (Eq, <whatever>), TBuf t); [ <x>; EBufNull t])) to emit <x> != null. It is a quirk of the krml format that a type application of the equality operator is taken to mean polymorphic comparison, where the width argument to EOp is simply ignored (the <whatever> in my snippet above).
If you are adamant that you wish to add one extra node to the krml format, then I will request that you make sure FStar.Extraction.Krml.fs is consistent and uses EIsNull in all cases, so that the temporary workarounds in InputAstToAst.ml can eventually go away. Thanks!

Thanks a lot Jonathan!

You do not need a new EIsNull node: it suffices to generate EApp (ETApp (EOp (Eq, <whatever>), TBuf t); [ <x>; EBufNull t])) to emit <x> != null. It is a quirk of the krml format that a type application of the equality operator is taken to mean polymorphic comparison, where the width argument to EOp is simply ignored (the <whatever> in my snippet above).

This is exactly the confirmation I needed. So let's shelve this PR for the time being. Thank you again!


[BUG] Node Search function: Does not retrieve React Agent

Describe the bug
When searching for nodes via Flowise UI (top left button), search function does not work as expected. Cannot find ReAct Agent
To Reproduce

On Chatflow development screen
Add node button
Search nodes for: rea
Does not return any ReAct agent nodes

Expected behavior
List of search results includes 'ReAct Agent for xxx'
Screenshots

Setup

Installation: npx flowise start
Flowise Version: 1.6.4
OS: Windows
Browser: Firefox

Additional context
NIL

good catch! we still have the name as mrklAgent - https://github.com/FlowiseAI/Flowise/blob/main/packages/components/nodes/agents/MRKLAgentChat/MRKLAgentChat.ts#L30
We'll need to update those as well

Solved with this PR


[Question] compressing the template literals inside JS bundles ?

Hi there, 👋
Thank you for this great project 👏 , especially useful with Tailwind CSS. 😎
I was wondering if there were a webpack / babel / plugin / config that you know which could be used to compress the template literal used by ctl ?
Here is some part of my source code as an example:
...
                    return (
                      <summary
                        className={ctl(`
                          relative
                          block
                          cursor-pointer
                          border-b-1
                          border-navy-blue-100
                          py-10
                          px-2
                          pr-20
                          font-bold
                          dark:border-navy-blue-500
                          md:cursor-default
                          md:p-8
                        `)}
...

This is getting bundled as:
...
(\"summary\", {\n                  className: _netlify_classnames_template_literals__WEBPACK_IMPORTED_MODULE_1___default()(\"\\n                          relative\\n                          block\\n                          cursor-pointer\\n                          border-b-1\\n                          border-navy-blue-100\\n                          py-10\\n                          px-2\\n                          pr-20\\n                          font-bold\\n                          dark:border-navy-blue-500\\n                          md:cursor-default\\n                          md:p-8\\n                          \"
...

I wish it could be "compressed" like this:
...
(\"summary\", {\n                  className: _netlify_classnames_template_literals__WEBPACK_IMPORTED_MODULE_1___default()(\"relative block cursor-pointer border-b-1 border-navy-blue-100 py-10 px-2 pr-20 font-bold dark:border-navy-blue-500 md:cursor-default md:p-8\"
...

Thank you for your feedback

https://stackoverflow.com/questions/57524832/how-to-remove-n-and-t-from-webpack-compiled-template-literal


Auto-close after inactivity setting not available

The setting "Auto-close after inactivity" is described here, but not available on all environments. We can't see it in our...
Is there any tweak to have this available in our environment?

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

ID: fc2d0da5-119d-574e-2a8c-b3ed19df5484
Version Independent ID: f331e4b4-f013-93ce-be71-33b292b1e093
Content: Create and manage workstreams
Content Source: ce/customer-service/create-workstreams.md
Service: dynamics-365-customerservice
GitHub Login: @neeranelli
Microsoft Alias: nenellim

Hello @RobinBronsteede,
As described in the documentation, it is not available for all channels. Can you please clarify for which channel are you not able to see it?
Auto-close after inactivity: Select a time period after which inactive conversations will be moved to the closed state automatically. This option is available for only persistent chat, SMS, social, and Microsoft Teams channels.

Is there a difference in "chat" and "persistant chat"?
We have chat enabled in our environment

Hello @RobinBronsteede , Yes, persistent chat is different from live chat as the name suggests. For more information about persistent chat, see persistent chat.
For information about live chat, see Configure live chat

Hello @RobinBronsteede, I hope that your question is answered. I am closing this issue because no content updates are required.
Best regards,


Add support for doas

doas is an alternative to sudo.

This is unlikely to be implemented, as we are settling on nixos-rebuild as an API for implementing Nixinate. This means that you would need to get nixos-rebuild to support doas, by modifying https://github.com/NixOS/nixpkgs/blob/nixos-21.11/pkgs/os-specific/linux/nixos-rebuild/nixos-rebuild.sh


Use namespaces for future Shaarli v0.11

There is a breaking change for all plugins in Shaarli future release due to the introduction of namespaces.
See https://github.com/ArthurHoaro/shaarli2twitter/commit/c05d044bef8a4bcf4beee6e10eecec970dccf4c0
Shaarli v0.11.0 won't be released anytime soon at its current pace of development, but the plugin isn't currently compatible with the master branch. Maybe you should release the current state of this repo while it's compatible with v0.10.x.

Thanks for pointing that out! I didn't even remember that I didn't made a proper release yet.
That's now done. I'll start adding namespaces for this plugin and the few others I have.


NCD - Subs Visit - Outside Care [6h]

Issue
In the first encounter, we have an activity called "Medical History" with a sub-activity called "Outside Care." In the subsequent encounter, we are getting rid of all but one of the Medical History sub-activities. Even though there are other places in the app that we keep one sub-activity in the drawer, it has been requested that we get rid of the Medical History activity and bump up the Outside Care sub-activity to be a full activity. Outside care is replacing Medical History in the main activity tray and will appear in the same position.
Nothing within the activity is changing.
This is what the main activity screen will look like:

This is what the activity will look like:

Issue

Create Outside Care main activity

@anvmn

nutrition env.

Questions/Found Issues:

Did I understand correctly that the Outside Care sub-activity should become the main activity only in subsequent encounters?
The following confused me:

According to the example in the description: 1. should be "If yes, what?" 2. should be "None of these"

According to the example in the description, the last question is "Were you given medicine?" shouldn't be conditional. But in the application it's conditional. The question only appears if at least one of the diagnoses from the first image is selected and doesn't appear if the diagnoses from the second image (or "Other") are selected:

Checked and Approved:

The Medical History activity (and sub-activity) doesn't appear in the subsequent encounters.
The Outside Care is the main activity in the subsequent encounters.

The Outside Care activity looks as expected except the points in the Questions/Found Issues section:

the second conditional question:

the third conditional question:

the last question:

@LudmilaGoichman
The designs are not 100% accurate. For context, we follow here same design and logic we got at ANC Outside care activity.
1 =>  Yes, only in subsequent encounter.
2 => Per ANC activity.
3 => Same logic as at ANC activity - some dignoses can be treated with medication. Others can not. There's no point in asking if medication was given, if diagnosis is not rtreated with medication.
So, question will show up only if there's at least one diagnosis that can be medicated.


Automate getting bot version

Automate getting the bot version from the poetry/package version at runtime.

Closed with PR #20


Support for Chef InSpec

Please support integration with Chef InSpec. I thought it was already supported, but I can't find it listed on https://github.com/DefectDojo/django-DefectDojo/tree/master/dojo/tools ?

@LOLCATATONIA Do you have an example output file from Chef InSpec you can share?  If so adding it to this repo - https://github.com/DefectDojo/sample-scan-files is a great way to assist getting a new parser added to DefectDojo.
The example file doesn't have to have 'real' data in it - just have reasonable data and keeps the structure of the tools output.  For example, I added a BugCrowd file to that repo but have never worked for 'BigCorp'.  I simply took legit output from a tool and removed/re-wrote anything that might identify the actual source. Some contributors have scanned open source / known vulnerable apps (OWASP JuiceShop) to produce output files as well.
HTH

inspec_audit.log
Here's a sample file. I had trouble pulling that repo.

This issue can be closed @mtesauro


Unnecessary blank line for one-line declarations

When let or var is followed by a value, it is (I think) invalid to have more values following in the same block. In this case, it makes sense to evaluate the line immediately without requiring blank lines
example:
> let x = 1
-

Here there is no need for the - line

I will address in the next release.

Fixed in version 0.1.4


fix: add sharp optimizations

Adds some sharp settings that optimize filesize. sharp recently added these options which previous required something like imageoptim/pngquant

It's new!


Fixed issue #125

Description
Fixes #125
Type of change

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

Codecov Report

Merging #160 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   69.83%   69.83%
=======================================
  Files          12       12
  Lines         716      716
=======================================
  Hits          500      500
  Misses        212      212
  Partials        4        4

Impacted Files
Coverage Δ

ioc.go
34.57% <0.00%> (ø)

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 e9d3266...c6c80b2. Read the comment docs.


How to use caret/clear slots with singleSelect?

v:slot for caret and clear only seem to work with multiSelect. Is there a way to make these work with singeSelect as well?

It should work with single select as well. Please create a repro if it does not for you and I'll have a look. Closing this for now, but feel free to reopen.


Fix changelog divergence for 2.x on main

Description

Fixes changelog divergence for Unreleased 2.x section on main

Issues Resolved

N/A

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/13054/
CommitID: 1e04935
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

#6885

Codecov Report

Merging #6882 (1e04935) into main (7b708c4) will increase coverage by 0.23%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6882      +/-   ##
============================================
+ Coverage     70.52%   70.76%   +0.23%
- Complexity    59112    59274     +162
============================================
  Files          4814     4811       -3
  Lines        283743   283716      -27
  Branches      40915    40915
============================================
+ Hits         200110   200763     +653
+ Misses        67154    66461     -693
- Partials      16479    16492      +13

see 526 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Description fails to render properly for some tags and filters on hover

Describe the bug
In some scenarios if the documentation for a liquid tag or filter includes a warning it ends up getting improperly rendered. For example, image_url contains a warning about requiring at least width or height but it's not clear on hover:

Expected behaviour
The additional description should render on its own line.

would need to figure out what needs to change to make this formatted correctly for VS code and code mirror


Create mborijnland.txt

@Stanleydevv I am sorry, I have to close this request without merging, because you requested to add a non-existent domain to this repository:
$ host mborijland.nu
Host mborijland.nu not found: 3(NXDOMAIN)

Please ensure you are familiar with this repository rules located at the bottom of this page and submit a new request with correct information. Thank you for the understanding!


Updates / cleanup of application help

Application help is inconsistent and should be updated;
The Configurator use the following arguments (similar to argparse):

description: what is the application doing (e.g., Submit model parameter (value, table) through an external interface.
usage: typical usage (e.g. python applications/submit_data_from_external.py --workflow_config <workflow configuration file>
label: program name
epilog: (nowhere used) text after all command line arguments.

Also check that configuration arguments are selected correctly through config.initialize.

Addressed by #399


wslgit stopped working after upgrading to 0.8.0

After switching to https://github.com/andy-5/wslgit/tree/v0.8.0, wslgit stopped working.
Previously I used to use https://github.com/andy-5/wslgit/tree/v0.7.0 (wslgit-mount-at-root.exe). Everything was OK.
lsb_release -a:
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 19.10
Release:        19.10
Codename:       eoan

OS:
Windows 10 Pro 64-bit (10.0, Build 18362) (18362.19h1_release.190318-1202)

Env. variables:
WSLGIT_MOUNT_ROOT: true
WSLGIT_USE_INTERACTIVE_SHELL: smart

Unfortunately, the current documentation is a bit unclear. You need to set the new Windows environment variable WSLGIT_MOUNT_ROOT to / to get the behaviour of wslgit-mount-at-root.exe.
Also, WSLGIT_USE_INTERACTIVE_SHELL=smart is only available in the upcoming v0.9.0 - the current Readme already reflects the state of the master branch.
Does this fix your problem?

Yes, thank you.


Code release

Dear authors, I'm very interested in your work. May I ask when you plan to release the code? Thanks!

I also expect the author to release the code. I added PCN into the pointrCNN, but the running accuracy became worse. I think that sampling the point cloud after completing the false point cloud will affect the border correction

Hi @liuhuaijjin and @jlqzzz, thanks for your interest in our work. The code of SIENet has been released, we hope it will be helpful for your research.


[Feature Proposal] Start containers as soon as Kitematic starts up

It'd be nice if there were a setting to have certain containers start-up automatically as soon as Kitematic starts up. I tend to quit out of Kitematic (and shutdown VM) when I'm not using it. So next time I start it up I have to click on each container and start them individually.
Great work on Kitematic! It's a very helpful app.

As an alternative, you can change the pref for Kitematic not to shutdown the VM on close - That's usually what I do, which keeps the VM up and thus all of your containers up and running as well.
Feel free to look through the code and propose it as a feature. :)
Closing this for now due to the low impact of the feature.


adding some data structures implementation resources

A pull request trying to add some resources about data structures implementation, as mentioned on #26 .

How are those compared to UC Berkeley course?
I do not want to add the list of all available resources on the topic.

This courses teach a lot about implementation of the data structures, as people were saying on the issue I mentioned later. I don't know anything about UC Berkeley course in order to compare both.

Let's merge it then 🎉


Help with MatAutocompleteList

I need help with this I need Scroll here

The whole point of auto complete as a concept is to not have a scroll IMHO. If you need a scroll why not just use a proper dropdown.

@americanslon I agree with you that it should have a height and if it goes past it then a vertical scroll.


Code snippet control indentation

How can I  control unindentation in code snippets?
{
    "Create migration helper execute statement": {
        "prefix": "mhe",
        "body": [
            "\\$helper->execute(<<<SQL",
            "\t$0",
            "\rSQL",
            ");"
        ]
    }
}

When invoking it in indented context:
class Migration {
→   public function up(Helper $helper) {
→   →   mhe<tab complete snippet>
→   }
}

Expected behavior:
class Migration {
→   public function up(Helper $helper) {
→   →   $helper->execute(<<<SQL
→   →   →   |<cursor>
SQL
→   →   );
→   }
}

Actual behavior:
class Migration {
→   public function up(Helper $helper) {
→   →   $helper->execute(<<<SQL
→   →   →   |<cursor>
→   →
→   →   SQL
→   →   );
→   }
}

This is invalid behavior when using heredoc.
I tried to insert \r  to try removing all indentation,
but it is just translated to a newline and indented according to snippet context.

VSCode Version: 1.9.0
OS Version: macOS Sierra

Tricky, today snippets are always adjusted to the indent of the line it is inserted at. Think of the normal for-loop. We might need some special character to signal this behaviour.

I thought that was what I was doing with \r?

That's just interpreted as newline, like the elements in the body array...

Yes I'm aware :) what I meant was, that \r could be that "special" character?

To me it seems more intuitive to be able to use \b in snippets (handle ascii backspaces in the json string).
It would also be handy to have a special sequence to remove n levels of indentation, clear line, and have a reference to the starting indent level the cursor was on when the snippet was inserted.

The question with the unindent is how far to go? Should \b unindent  by one indentation-unit or to the start of the line? Or should another character be use for start of like, like the start of text ascii control character?

I think, if you want to go that way, we need two special characters, one for start of line, and one for unindent one. Both are useful in different contexts. In my particular example I need, start of line in order to support heredoc.

I've came to an issue where I need to unindent the snippet by 1 indentation-unit. e.g.: imagine a lot of elseif or python's elif statements where the indentation is PART OF THE LANGUAGE.
\b support for unindent by 1 unit would be really great

I think support for unindent a single indentation unit will be great too. Sadly it does not solve this particular issue, because I don't how I need to unindent, I simply need to unindent to the start of the line, which \r represents. To be honest I don't really care what the token should be.

Is there any update for current case?

I've put else in both increaseIndentPattern and  decreaseIndentPattern.
When I type else; the indentation works perfectly.
But when using a snippet or auto-complete; it fails.
So I'd be happy if the snippet/auto-complete issue is at least fixed.
Example:

@Hezkore I see your issue, but I don't think it's related to this. This is an issue about "calling" formatters when a snippet is "completed" all placeholders has been replaced or the snippet is "tabbed away from".

@thetrompf Well... my bug report was closed in reference to this issue.
See #76139

This issue is still very much relevant. And I'm still not sure why my issue was closed if it's not similar to this one.
@jrieken any update on my issue (#76139) or why the merge happened?

Maybe related to https://github.com/microsoft/vscode/issues/171439 ?

Maybe related to https://github.com/microsoft/vscode/issues/171439 ?

Maybe. If the issue you're referring to is solved in the way you suggest, it probably/maybe fixes this too.
But I don't agree that there is an issue in 171439. IMO if you select some code and instruct the editor to indent it, the editor shouldn't suddenly not "obey" - it is easy enough to only select the code you want to indent.
In this issue, at least how I imagined the solution, I just want snippets to adhere to the \r (carriage return) character.

But I don't agree that there is an issue in 171439. IMO if you select some code and instruct the editor to indent it, the editor shouldn't suddenly not "obey" - it is easy enough to only select the code you want to indent.

This should be better discussed on that issue, but the thing is, it's very common having to move and indent big chunks of code and it's really annoying having to select and indent the code per blocks manually or to accidentally introduce syntax errors because you oversaw some HEREDOC tags.
You will never want to indent a HEREDOC tag because it simply breaks the code.
Indenting code is a formatting feature, and a formatting feature shouldn't break your code.


Cncf page in meshery.io would look better when Aligned

Current Behavior

In https://meshery.io/programs/meshery/cncf the content are not aligned properly,
Desired Behavior

all the content after properly aligned will give more better look
Screenshots / Mockups

Alternatives

Contributor Guides and Resources

🎨 Wireframes and designs for Meshery Site in Figma
🖥 Contributing to Meshery Website
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

i would like to sove this issues

Hi manik. Could you please annotate/mark the area where you feel the problem is in the screenshot?

@Chadha93 need your suggestion here

@Chadha93 need your suggestion here

@manikmmalhotra What alignment issues are you highlighting? Please propose your suggestions in the PR description for specific alignments of components.

@Chadha93 aligning as shown in the screenshot

@manikmmalhotra @Chadha93
is this what we are trying to achieve ?

// @warunicorn19 views...

The proposed solution looks good. @manikmmalhotra @sourav-sharma7

@warunicorn19 yes, great work @sourav-sharma7


Driverless cannot get the "printer-strings-uri" attribute

Describe the bug
Driverless will generate the localization strings based on the "printer-strings-uri" attributes.
But for the legacy-printer-app, driverless fails to get the "printer-strings-uri" attributes.
To Reproduce
Steps to reproduce the behavior:

In the legacy-printer-app, call the papplSystemAddStringsData() to add a static localization resource.
Run leagcy-printer-app server
Generate driverless ppd by command lpadmin -p foo -v ipp://localhost:8000/ipp/print/foo -m driverless:ipp://localhost:8000/ipp/print/foo
The ppd doesn't show localization strings.

Additional context
Did some investigation:

Driverless connects to pappl and attempts to get the printer attributes: "all" & "media-col-database". But pappl doesn't return the printer-strings-uri attributes.

In cfGetPrinterAttributes5(), remove the "media-col-database" to make Driverless only get the printer attributes "all", pappl can return the printer-strings-uri attributes.

The command ipptool -tv ipp://localhost:8000/ipp/print/foo get-printer-attributes-3.0.test can show the printer-strings-uri.

But the command ipptool -tv ipp://localhost:8000/ipp/print/foo get-printer-attributes-2.0.test doesn't show the printer-strings-uri.

Connect to a hardware device, both ipptool -tv ipp://...  get-printer-attributes-2.0.test and ipptool -tv ipp://...  get-printer-attributes-3.0.test can show the printer-strings-uri.

ra = ippCreateRequestedArray() in function ipp_get_job_attributes() doesn't seem to be returning the correct result... but I didn't investigate further..

What version of CUPS are you linking against?

should be 2.4.x, I don't remember the minor version, will check and reply again in next Monday.

It's my fault. I linked against the CUPS v2.4.2.
After upgrade to CUPS v2.4.6, this issue is fixed.


What texture formats are currently supported?

I couldn't find what texture formats are currently supported for input files.  We should update the README with an exact list.
It would be nice if we could auto-convert textures to glTF friendly formats, perhaps via imagemagick or whatever node-friendly libraries are available.

This FBX2gltf library takes another to unsupported image formats. If the texture isn’t a png or jpg it looks for a texture with the same name but a supported extension.
I’m not sure what the most robust image conversion tool for node is, but the ones I’ve used rely on image magical, which makes installation less straightforward.
https://github.com/facebookincubator/FBX2glTF


sdist is missing suite

The sdist package at PyPI is missing suite directory.  Without the directory testing fails:
_________ ERROR collecting referencing/tests/test_referencing_suite.py _________
/usr/lib/python3.9/vendor-packages/_pytest/runner.py:341: in from_call
    result: Optional[TResult] = func()
/usr/lib/python3.9/vendor-packages/_pytest/runner.py:372: in <lambda>
    call = CallInfo.from_call(lambda: list(collector.collect()), "collect")
/usr/lib/python3.9/vendor-packages/_pytest/python.py:531: in collect
    self._inject_setup_module_fixture()
/usr/lib/python3.9/vendor-packages/_pytest/python.py:545: in _inject_setup_module_fixture
    self.obj, ("setUpModule", "setup_module")
/usr/lib/python3.9/vendor-packages/_pytest/python.py:310: in obj
    self._obj = obj = self._getobj()
/usr/lib/python3.9/vendor-packages/_pytest/python.py:528: in _getobj
    return self._importtestmodule()
/usr/lib/python3.9/vendor-packages/_pytest/python.py:617: in _importtestmodule
    mod = import_path(self.path, mode=importmode, root=self.config.rootpath)
/usr/lib/python3.9/vendor-packages/_pytest/pathlib.py:565: in import_path
    importlib.import_module(module_name)
/usr/lib/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
<frozen importlib._bootstrap>:1030: in _gcd_import
    ???
<frozen importlib._bootstrap>:1007: in _find_and_load
    ???
<frozen importlib._bootstrap>:986: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:680: in _load_unlocked
    ???
/usr/lib/python3.9/vendor-packages/_pytest/assertion/rewrite.py:178: in exec_module
    exec(co, module.__dict__)
referencing/tests/test_referencing_suite.py:27: in <module>
    raise SuiteNotFound()
E   referencing.tests.test_referencing_suite.SuiteNotFound: Cannot find the referencing suite. Set the REFERENCING_SUITE environment variable to the path to the suite, or run the test suite from alongside a full checkout of the git repository.

Please add the suite directory to sdist.  Thank you.
Please note the github tarball contains the suite directory, but it is empty.

Should be available in the next release, thanks.