rename Sky to Flutter and add clarifying comment to template

Fixes flutter/flutter#7162
Requires merging flutter/engine#3293

Do existing customers need to take any action?  Or are all the old classes still around?

FYI @tvolkert this will likely require changes to internal apps during the next release.

It should be backwards compatible.
On Tue, Dec 6, 2016 at 20:22 Eric Seidel<EMAIL_ADDRESS>wrote:

FYI @tvolkert https://github.com/tvolkert this will likely require
changes to internal apps during the next release.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/7165#issuecomment-265352040, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYGiXB96ukje0o9sIDQP8mk38tpm1Xlks5rFjR5gaJpZM4LE6Jg
.


Fix for marshall() and demarshall() generated methods. In my project …

…resulting js code size went down from 433kb to 380kb, and demarshall() performance few times better now.
Fixes both:
https://issues.jboss.org/browse/ERRAI-1055
https://issues.jboss.org/browse/ERRAI-892

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

@slavap Also there seem to be test failures now. Unfortunately the PR build isn't setup to show codegen stack traces.
You should try running this locally to see if you can reproduce the issues:
mvn clean install -Pintegration-test -Derrai.codegen.details=true

jenkins retest this

@mbarkley

But more importantly, I have concerns about changing from append to concatenation.
Does this change impact runtime performance when marshalling/demarshalling?
Are we sure that the code generated from concatenation is as efficient as using the StringBuilder?

StringBuilder is not native to javascript, so could not be optimized by js vm.
From java7 string concatenations are automatically processed by JVM as StringBuilder under the hood.
So IMO all this append() stuff makes sense for java6 only.
Generated js code is very clean and much smaller after my fix.

In what environment did you test the code-size difference? More specifically: How big was the project? How many portable types? Did you do a full compile with optimisations and obfuscation?

I've tested in my current project. Full optimization and obfuscation of course.
About 270 portable classes, some classes with more than 100 properties.

I can try and find time this week to help do some profiling.
Just look at resulting js code (full optimization, pretty style) and it will be clear how better newly generated js code is.

@mbarkley I've added changes according to your request. Unfortunately I still see test failures :-( Tried to run: mvn clean install -Pintegration-test -Derrai.codegen.details=true
on my PC without success, it fails in testPointToPointMessageInClusterAfterClientChangesNodes, but I cannot figure out what/where the problem is :-( I need help with that.
mvn clean install - works just fine, no errors.

@slavap
I have added an inline comment in DefaultJavaMappingStrategy about the cause of the test errors.
I see what you are saying about StringBuilder in the JVM, but I am less certain about what code gets generated by GWT using StringBuilder versus concatenation (as clearly the code must be different, or else it wouldn't have been larger with StringBuilder). I will try to do some tests of my own to empirically verify that there is no runtime performance regression.

@mbarkley I will check how using cast will affect generated js code. Not sure it's a good idea. Would you like to see the example of generated js code? Actually performance after my fix even visually noticeable, first I've thought that marshaling started passing empty :-)

@mbarkley Replaced: for (String key : obj.keySet()) { ... }
by: final Set keys = obj.keySet();  for (String key : keys) { ... }
That way no cast is needed and integration tests should pass.

@mbarkley At last "All checks passed"!
By the way, on my PC it is not that good :-(
For example Errai::Bus::JSR-356::WebSocket::Weld ............... FAILURE with:
Running org.jboss.errai.bus.server.websocket.test.jsr356.cdi.filter.FilterTest
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.232 sec <<< FAILURE! - in org.jboss.errai.bus.server.websocket.test.jsr356.cdi.filter.FilterTest
org.jboss.errai.bus.server.websocket.test.jsr356.cdi.filter.FilterTest  Time elapsed: 1.232 sec  <<< ERROR!
org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: filtertest.war
Caused by: java.io.IOException: Invalid response
God knows what that means :-(
And integration tests are painfully slow, even on decent 4 cores PC with SSD.
Final results of my fix - resulting js down from 433kb to 362kb, i.e. > 16% reduction.
Next step is data bindings optimization :-)


Make admin git gc tasks run in background

The admin git gc and git fsck tasks should run in the background rather than run on the request scope.
Any errors should then be put in to the notices.
Fix #10299

Now should we be putting a system notice to say that we've completed the gc/fsck?

None of the other tasks seem to do this. So perhaps it would be better to do it as another PR

@zeripath I would like to create a task with a new type and track the status of them.

You could leverage the process table - there's nothing that requires that to be an actual running external process

Now should we be putting a system notice to say that we've completed the gc/fsck?

I'd go for it. Given that it can only be started manually, a notice would not be unexpected.

We can just run it with WithUnique(...) like we do for Cronable events.

Tried this locally, UI showed GC was started but in the log I see this error (no stack or anything):
2020/03/06 19:21:35 ...uters/admin/admin.go:186:func1() [E] Error whilst running git gc: 0x1c97a40

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@345515a). Click here to learn what that means.
The diff coverage is 41.3%.

@@            Coverage Diff            @@
##             master   #10629   +/-   ##
=========================================
  Coverage          ?   43.61%
=========================================
  Files             ?      588
  Lines             ?    82502
  Branches          ?        0
=========================================
  Hits              ?    35980
  Misses            ?    42060
  Partials          ?     4462

Impacted Files
Coverage Δ

routers/admin/admin.go
10.21% <0%> (ø)

modules/repository/check.go
0% <0%> (ø)

modules/cron/cron.go
50.49% <61.29%> (ø)

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 345515a...9af8901. Read the comment docs.

@silverwind There was a bug in the previous version try now

OK. We have two options - keep this a simple just background the GC tasks PR - or properly sort out these admin and cron tasks.
That is:

Admin and cron task should appear in Processes - with cancellable contexts
They should report when they're done - presumably into system notices?

As an extension cron tasks need restructuring - they should probably be merged with the admin tasks - and their config moved to the db?

They should report when they're done - presumably into system notices?

Yes, please. I don't feel like one should have to dig into logs for this. I'd say also report when admin actions start in the system notices.
Additional "nicities" I could see:

Add the username who started it to the start notice
Add either a report that lists repo size before/after gc or just the raw CLI output to the finish notice.

See #10745

How is the status here? My hope was to get this in 1.12.0.


Panic when using shatter tool.

Hi @IvoryDuke
This work of yours is thoroughly impressive and I have learned a lot from studying it. Thank you.
I will use some of the concepts for my own (non-game) editor.
How long did it take for you to make this?
I noticed that there is a panic every time I do the shatter operation. I am not sure if I am doing something wrong, or if its an issue with the shatter tool itself or just the item selection logic.
I have created a pull request for this:
https://github.com/IvoryDuke/HillVacuum/pull/7
But I could be misdiagnosing the issue.

Thank you for the opened issue I have just finished reviewing it.
If I remember correctly I started HV in November-December 2022 and have been working on it in my free time.
It is my first Rust project so it has taken me a lot of time since I have been learning the language as I added new features.
It is also my first major coding project.
Thank you a lot for your time! seeing people taking interest in it is motivating me to add new features.

Wow this is your first large project? That's hard to believe! I was expecting you to work for an IDE company of some sort :P
Kudos to you!


Add evaluation tool and documentation

Purpose
This PR uses the new Azure AI Generative SDK evaluate() and generate() functions to create an evaluation workflow for testing changes to settings.
I tested several prompts:

our default prompt
Chris's prompt (a variation on our default)
A trimmed version of Chris's prompt
weak prompt: "You are a helpful assistant"
ignore-sources prompt: Designed to encourage ungrounded responses.
pig-latin prompt: Designed to get responses to be somewhat garbage.

Very interesting results:

I've shared observations on the metrics and the SDK itself in an internal doc:
https://microsoft-my.sharepoint.com/:w:/p/pamelafox/EW12LIs7l9RFn8jyICVWQS4BhLE6c5WvP0oXz-uH7T8z8g?e=iAZ55A
There are a few improvements pending to the SDK, and to the groundedness metric in particular (which you can see was too generous in my runs above), but overall, it's a useful tool and framework for developers of our app. I expect developers may add their own metrics, like the "has citation" one that I added.
Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Follow instructions in evaluation.md doc

Looks like there are typing errors, I'll take a look at those tomorrow.

Update: We're going to move this into its own repo to make it more modular and reusable by more developers.

Closing in favor of https://github.com/Azure-Samples/azure-search-openai-demo/pull/1061 which links to https://github.com/Azure-Samples/ai-rag-chat-evaluator


Code Coverage Reports

Is there a way to get coverage reports for our onceover tests?

Possibly, but the code coverage tools that exist wouldn't work, here's why:
Onceover is designed to work with the roles and profiles pattern, in which each machine gets only one role, which includes many profiles, which call out to forge modules, which do the work. Onceover only tests roles, not profiles (normally) so if you were to look at the code coverage it would be low. However given that each node only gets one role, and nothing should be getting profiles assigned directly to it (only through a role) this means that if you test every role you essentially have 100% coverage, because each node only gets one role and that's it meaning all permutations are covered.
Does that make sense?

Yeah, thanks for the response.
I currently have onceover testing roles to make sure they include the expected profiles with the appropriate parameters, and I also test the profiles themselves to make sure their sub-resources are called correctly. I was thinking of the logic I have in my profiles for executing different code paths based on profile parameters or hiera data, and making sure I covered all those branches.

That's true, but you'd have to certainly modify the way the existing code coverage tools work as they would be expecting a test for every resource. To be honest I'd argue that testing for roles to contain certain profiles is just a duplication of effort, unless you want to globally enforce that "profile::base" included in every role or something similar. As for profiles I'd also argue that unless a resource is wrapped in some logic that you want to make sure works, testing for them is redundant, it just creates a second place you have to change code if you ever refactor it.
I think a really interesting code coverage report would involve not classes that don't have tests, but classes which exist in the repo (in the controlrepo, not in modules) where not compiled as part of any test. That would make it very easy to ensure you had enough factsets to cover all permutations of your if-else logic etc.

Closing due to inactivity


[WIP] exec async bug?

Hi there,
Say, I ran this code:
var sh = require('shelljs');

sh.exec("sleep 0.5", {async: false}, function() {
    sh.echo("print first");
});

sh.echo("print last");

...as output,  I will get:
print last
print first

Your documentation on shelljs.org seems to imply that if you pass in an {async:false} as options, you still get to run code synchronously. Am I misreading the docs?
I think you are overwriting options.asyncin here: https://github.com/vise890/shelljs/compare/arturadib:master...vise890:master#diff-c3bfabb5e6987aa21bc75ffd95a162d6L168
Check out the diff, tell me what you think. Cheers. M.

I saw your changes and I think you don't need to change any logic there, if you just change the default value of async:
  // Callback is defined instead of options.
  if (typeof options === 'function') {
    callback = options;
    options = { async: false };
  }

Then if will only be true if there's a callback specified
  // Callback is defined with options.
  if (typeof options === 'object' && typeof callback === 'function') {
    options.async = true;
  }

I believe this is what was intended, but it have a typo on the initialisation of the async property.

@thegiantbeast I don't think that your first change would be beneficial. Then every time you provide just a callback, it will be run syncronously. Which is not what the docs imply.
If you provide options like {async:false} and a callback the bug will still persist.

ok so I started from scratch an made a new PR in #184

@vise890 I think you should review your logic, your latest change wouldn't work either.
options.async = options.async || true;

You won't be able to set async as false as false || true will always return true...
I checked the code again and the logic is right (ignore my previous comment about the typo).
So if you don't pass a callback (either on 2nd or 3rd param) async will always be false, but if you do it means it will run asynchronously and it's not configurable because that's how it's supposed to work (the sync method doesn't even support a callback).
What are you trying to achieve?

You're right in pointing out that my logic is wrong.
What I'm trying to achieve is that when you pass {async:false} as options
and a callback the task will be run synchronously.
If that's not possible then you shouldn't say that options.async is true
by default if you provide a callback. It made me believe that there was a
possibility to set it to false. You should say that options.async is true
when a callback is provided and that any value passed is ignored.
Here's the specific point in the docs I'm referring to BTW:
http://documentup.com/arturadib/shelljs#command-reference/exec-command-options-callback
I'll have another look tomorrow.
@vise890 https://github.com/vise890 I think you should review your logic,
your latest change wouldn't work either.
options.async = options.async || true;
You won't be able to set async as false as false || true will always return
true...
I checked the code again and the logic is right (ignore my previous comment
about the typo).
So if you don't pass a callback (either on 2nd or 3rd param) async will
always be false, but if you do it means it will run asynchronously and it's
not configurable because that's how it's supposed to work (the sync method
doesn't even support a callback).
What are you trying to achieve?
—
Reply to this email directly or view it on GitHub
https://github.com/arturadib/shelljs/pull/182#issuecomment-72387970.

I think you misunderstood what is said (maybe it needs some clarification), but the documentation what's trying to say is that in case of a callback you don't need to see async as it's set automatically for you.
And the option is there for the case where you don't have/need a callback but you still want to run the command asynchronously, ie .exec('ls /', {async: true}).
But the feature is correct and the documentation is also "correct", but maybe it should clarify a little bit better.

Ok have you had a look at #185 ? I think the docs aren't very clear when they state:

[async] Defaults to true if a callback is provided.
To me, saying that it "defaults" to true, implies that you can still set it... I don't know, maybe it's just me.

Yes, I understand what you're saying and that's why I said documentation needs some clarification (maybe replace default with:

[async] automatically sets to true if a callback is provided.

I think this will explain correctly the functionality of it.


Failed to load message bundle in debug function

Behaviour
Expected vs. Actual
i use the source code to generate the vsix file.
open a python file and first time click "Run and Debug"
the quick pick window should shows "Select a debug configuration" "Debug Configuration" "Python File" and so on.
but every item shows "Failed to load message bundle. See console for details"
if i press "F5", everything is ok.
Steps to reproduce:

git clone https://github.com/microsoft/vscode-python
cd vscode-python
npm ci
pip install black
npx gulp installPythonLibs
npm run package
install the vsix
open a python file
click "Run and Debug"
the quick pick window shows many "Failed to load message bundle. See console for details"

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.9.12
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Conda
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

languageServer: "Jedi"

linting
• enabled: false
• flake8Args: "<placeholder>"
• flake8Enabled: true

sortImports
• args: "<placeholder>"

formatting
• provider: "black"

Im closing this issue, because with the new localization API  this error no longer exists.

Im closing this issue, because with the new localization API this error no longer exists.


Plugins with satellite assemblies

One issue that has hit both msbuild and roslyn with AssemblyLoadContext is that they're custom loaders did not find satellite assemblies.
https://github.com/dotnet/roslyn/issues/29014
https://github.com/Microsoft/msbuild/issues/2203
Is that handled automatically by this project?

It's one of the known issues I haven't addressed yet. It's one I definitely want to fix.

Nice. :)


Routing based on dates

What is the context of this PR?
As a user
I want to only answer questions which are applicable to me based on a date I have provided
So that I can skip any questions to reduce burden and to ensure I am able to answer them
Date compared with previous answer, metadata or hard coded date including today.
Offset_by allows for you to compare against a date in the past or future. Offset must allow for days, months and years.
How to review
test schemas:
test_routing_date_equals.json
test_routing_date_greater_than.json
test_routing_date_less_than.json
test_routing_date_not_equals.json

FYI: Schema titles don't match the schema itself.

@nealedj, @shahi645  Ready for re-review. I've added some comments as well.


add sphinx documentation

This PR introduces code documentation built with sphinx. Implements part of #30 (missing additional usage examples).

Hello @czab092! Thanks for the great work! I will change the PR merge target from main to docs/add_contributing_file. We will only change the html_theme, update some minor details (like version) and finally merge it to master!


[fix #1395] add new git Actions to auto check for changes to third-party jar dependencies

What is the purpose of the change
add new git Actions,that will check whether there are new third-party jar
Brief change log
add new shell file regenerate_konwn_dependencies_txt.sh,run the shell to regenerate known-dependencies.txt
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
Anything that affects deployment: no
The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: no

Documentation

Does this pull request introduce a new feature? no

Is it reasonable to put it into the existing License check Action process?
I think it's ok to create a new action separately, but you need to rename the Acton name(eg:Third-party jar check) to distinguish it from the existing one.

Every time we need to enter the detailed action process page to see the specific exception information, is there a easy way to display some failure information of the action on this page?  Can something like git robot be used, of course this is another feature.

I think it's better use new git Actions to check third-party dependencies,and I had renamed the Acton name.

Is it reasonable to put it into the existing License check Action process? I think it's ok to create a new action separately, but you need to rename the Acton name(eg:Third-party jar check) to distinguish it from the existing one.

xml-apis-1.3.04.jar is not in file known-dependencies.txt!
Error: Process completed with exit code 1.

xml-apis-1.3.04.jar  is missed in known-dependencies.txt!


feat: Fail-fast more often

NOTE: This is meant as a discussion thread, at least at first. How far do we want to go on the "keep on trucking" ←→ "fail fast" scale?
Description
In general, I think it would be best if flutter_soloud started to Fail Fast more. Right now, the plugin is very forgiving, and "keeps on trucking" even when there's a severe problem.
Although I agree that an audio plugin can be a bit more lenient (after all, for most apps and games, not hearing one of the sounds is preferable to experiencing a crash), I think a nudge towards more fail-fast approach would be helpful.
For example, the AssetsManager.getAssetFile() method keeps on trucking when there's any error during asset loading, and merely returns a null (and logs a severe warning). Here. This makes it really hard to debug — the developer might not notice the warning (or it might not even get printed to the console). And a problem when loading an asset seems like something the developer would want to address ASAP.
More tenuously, I'd say that some PlayerErrors should not be returned normally, but should be thrown. The developer can always use the standard try / catch / finally block, instead of reimplementing it manually (with if / else).
// Proposed API usage.

try {
  setState(() => _playerReady = false);
  final sound = await soloud.loadFile('...');
  await soloud.play(sound);
} on SoloudFileException {
  setState(() => _error = "File couldn't be loaded!");
} on SoloudException catch (e) {
  setState(() => _error = "Couldn't play: $e");
} finally {
  setState(() => _playerReady = true);
}

This kind of approach also means that the returns from various methods can be much simpler. For example, SoLoud.loadFile() can "just" return SoundProps? (or even non-nullable SoundProps). The various exceptions are thrown instead of being returned as part of a record.

This was addressed with #47.


Home page has some elements that are not refreshed

For some reason not all elements on my home page are refreshed properly (I think)

There is a broken image for the single trace that I uploaded via the app
3 other traces are still processing after a couple of days.

The processing part is normal if there were a lot of people uploading images this week(hint there was)
For the other problem it's a duplicate of #10


add vlsiffra notebook

Upstream:
https://github.com/antonblanchard/vlsiffra
Hard-Dependencies:
yosys (but can use https://github.com/amaranth-lang/amaranth-yosys)
Soft-Dependencies:

OpenROAD
ORFS
open_pdks.{sky130a,gf180mcuc)

Usage:

as a tool: https://github.com/antonblanchard/vlsiffra#example-usage
as an amaranth Elaborate: https://github.com/antonblanchard/vlsiffra/blob/main/vlsiffra/adder.py#L6

Also potentially explore packaging it natively with conda-eda: https://github.com/hdl/conda-eda/issues/248


Document link checker in CONTRIBUTING.md

Description
This PR adds a section to our contributor guide to document our use of starlight-links-validator and show how to run checks locally.

size-limit report 📦

Path
Size

/index.html
5.21 KB (0%)

/_astro/*.js
21.42 KB (0%)

/_astro/*.css
13.04 KB (0%)


Space between instruction arguments is error

I'm not sure what compiler the plugin is designed for, but Macroassembler AS is perfectly happy with spaces, and a large disassembled source I'm working with has spaces, which makes my IDE light up with this plugin :-).
Is it possible to be a little more lenient with argument validation?

This is currently not supported yet, see https://m68kplugin.yanncebron.com/known_issues.html#misc


UI issue on "Node settings" screen.

-When we go to enter the node address in the node settings, the "note" is coming above the port textbox.

[It is happening on SAMSUNG Galaxy A50,A70.I think this issue is coming only on big screen devices. ]

-Still able to see this issue on staging app V1.4.1(191)

Noting that this is the same issue we're dealing with in #2386.
Also, https://github.com/facebook/react-native/issues/6785 might be relevant.

Noting that this is the same issue we're dealing with in #2386.
Also, https://github.com/facebook/react-native/issues/6785 might be relevant.

-Verified this issue on Dev app v1.4.1(193).

-Verified this issue on Dev app v1.4.1(193).


Open changeset in iD

I'm submitting a bug report
Brief Description
iD shows all existing mappings of all mappers.
What is the motivation / use case for this feature?
Using iD to check a changeset.
What  is the current behaviour, (attach relevant screenshots) ?
Opening changeset through OSMCha in iD shows all existing mappings, including mappings of other changesets
What  is the expected behaviour ?
Showing only mappings belonging to changeset that currently is being inspected in OSMCha (as happens with opening in JOSM)
When does this occur ?
Every time when using "Open changeset in iD"
How do we replicate the issue ?

Click "Open changeset in iD"
Check iD for mappings

Please tell us about your environment:

Windows 10
MS Edge Version 104.0.1293.47
Firefox 103.0.2

Other Information / context:
N/A

Did you open an issue for iD well? I don't think, iD can handle changeset downloads out of the box. Maybe you need to check with them first.


Hacktoberfest2022 Webpage Bugs

Description
Hey @SauravMukherjee44 ,
There are bugs header section and footer section in the Hacktoberfest2022 webpage, I have figured out the bugs and solving them right now. Please asign me this task.
Thank you!
Screenshots

Hey @IAmTamal @SauravMukherjee44,
Please check the PR of #577.
Thank you!

I want to contribute to this issue.


compile error: response has no member named 'body'

I'm unable to set the body of the server response after its construction:
typedef boost::network::http::server< http_request_handler > server;
void http_request_handler::operator()( const server::request& rq, server::response& rsp )
{
   rsp = server::response::stock_reply( server::response::ok );

   rsp.body = "hello world!";
   body(rsp, "hello world!");
   rsp << body("hello world!");
}

The 3 lines attempting to set the response body yield the following compilation errors respectively:

‘boost::network::http::sync_server_base<boost::network::http::tags::http_server, dedupe::manage::http_request_handler>::response’ has no member named ‘body’
boost/network/message/directives.hpp:25:5: error: ‘struct boost::network::http::basic_response<boost::network::http::tags::http_server>’ has no member named ‘body’
boost/network/message/directives.hpp:25:5: error: ‘struct boost::network::http::basic_response<boost::network::http::tags::http_server>’ has no member named ‘body’

Running with cpp-netlib 0.9.3 and gcc 4.6.1 in C++0x mode.

Closing this because it's old, and the server response object has been replaced.


Fix X64CallingConventionResolver

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 28.207%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

Cpp2IL.Core/Utils/X64CallingConventionResolver.cs
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.004%

Covered Lines:
3352

Relevant Lines:
10112

💛  - Coveralls


Fix typo verified to approved

Sorry for the 2 PRs, you might wanna squash it. I edited directly on GH.

Closing this as I made another commit in #40.


Refactor: use from and first when available

Stack:

#1163 ⬅
#1162

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (pr/giladchase/gilad/small-refactor-rust-1.74/4f026e58@2b352bb). Click here to learn what that means.

:exclamation: Current head 233bec9 differs from pull request most recent head 19993fd. Consider uploading reports for the commit 19993fd to get more accurate results

Additional details and impacted files
@@                                   Coverage Diff                                    @@
##             pr/giladchase/gilad/small-refactor-rust-1.74/4f026e58    #1163   +/-   ##
========================================================================================
  Coverage                                                         ?   67.26%
========================================================================================
  Files                                                            ?       52
  Lines                                                            ?     6739
  Branches                                                         ?     6739
========================================================================================
  Hits                                                             ?     4533
  Misses                                                           ?     1791
  Partials                                                         ?      415

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adding SDKs umbrella page

🚀 Deployed on https://657752fc56128f23e4b6df7c--cozy-fox-0defec.netlify.app


Invalid user name or password

Hello,
No matter which username/password combo I use, I can't seem to pass the connection test.
For API access I use the same username/pass combo as the admin logon to Zabbix.
Do I need to enable the API for my user somewhere else?
Grafana 2.5.1 and Zabbix 2.2.1, API
Thank you

Hello! What kind of error you got? Invalid password or Connection failed?

Invalid password.
I confirmed the Admin password on the Zabbix side.
Here is a screenshot.
http://imgur.com/Rzrv4ZD

Can you try another user? Are you installed plugin first time or updated it from previous version?

I created a user api/api with full admin permissions and it's being denied.
Plugin install is a first time install.
Here is the screenshot showing all settings.
http://imgur.com/IPmn0Ax

Strange. What plugin version are you use?
Page should look like this:

I deleted the zabbix folder, downloaded everything again and it's working now. Not sure what happened. Probably user error on my end.
Thank you very much.

Ok.


OpenGL shader precision

How to make ( FS) shader support highp precision in bgfx?
I found that bgfx only supports medium precision for fs shader.

@bkaradzic

For the uniform qualifier,due to VS shader is highp precision but FS shader is Mediump precision, but types and precisions of uniforms with the same name must match across all shaders that are linked into a single executable, so program will crash due to
bgfx/src/renderer_gl.cpp (4148): BGFX 0: L0001 The fragment floating-point variable testUniform does not match the vertex variable testUniform.
    	The precision does not match.

@bkaradzic

Aren't those only default precisions? Can't you override it for specific uniforms?
uniform highp vec4 u_color;

Aren't those only default precisions? Can't you override it for specific uniforms?
uniform highp vec4 u_color;

@attilaz  but tool shaderc will remove the precisions highp when bgfx convert the shader.


Update the proto files from upstream.

Updated commits to not update the files that have moved to editions, reached out to the protobuf team to check on the behavior change I'm seeing.


Spurious diff with mist_device_gateway after each terraform apply

upstream: https://github.com/Juniper/terraform-provider-mist/issues/35

fixed in latest release


fix(mini.map): winblend description

DOC: Fix the reverse description of winblend on mini.map
0 makes it transparent:

100 makes it opaque:

[x] I have read CONTRIBUTING.md
[x] I have read CODE_OF_CONDUCT.md

DOC: Fix the reverse description of winblend on mini.map
0 makes it transparent:
100 makes it opaque:
* [x]  I have read [CONTRIBUTING.md](https://github.com/echasnovski/mini.nvim/blob/main/CONTRIBUTING.md)

* [x]  I have read [CODE_OF_CONDUCT.md](https://github.com/echasnovski/mini.nvim/blob/main/CODE_OF_CONDUCT.md)

Thanks for PR!
However, the current documentation is correct. Documentation is taken almost verbatim from corresponding 'winblend' option. And here where it is used directly.
Here is my experience with winblend = 0:

And this is with winblend = 100:

I am not sure why you experience this. Assuming no mixup with actual behavior, probably you have your hi MiniMapNormal highlight group with guibg=NONE. Together with transparent transparent terminal emulator it makes "transparent".

Strange,
I get same behaviour even if i keep my terminal non transparent and with nvim colorscheme non transparent
maybe this is only for me,
Well in the end it works for me on reverse, Thanks!!

Strange, I get same behaviour even if i keep my terminal non transparent and with nvim colorscheme non transparent maybe this is only for me, Well in the end it works for me on reverse, Thanks!!

Strange indeed.
Are you sure that 'winblend' is actually set to what you think it is set? If you execute :lua print(MiniMap.config.window.winblend), does it show the value you expect it to?
If the answer is yes, what is the output of :hi MiniMapNormal?

Yes, doing :lua print(MiniMap.config.window.winblend) outputs 0 which is what i set

If the answer is yes, what is the output of :hi MiniMapNormal?

Very strange... No clue at this point as to why you experience this. Good thing you can work around this.
If in future you detect a change in behavior (maybe after changing theme or terminal emulator), would you mind writing here about this?

If in future you detect a change in behavior (maybe after changing theme or terminal emulator), would you mind writing here about this?

Currently i am using Tokyo-night color scheme and Alacritty terminal emulator.
I will write about this if the behavior changed..


feat 💥: Remove ytdl support

What type of pull request is this?

[ ] Non-breaking code change
[x] Breaking code change
[ ] Documentation change/addition
[ ] Tests change

Description

Removes the built-in ytdl support, in the interest of not getting smited
Changes

Remove YTDLAudio
Remove yt_dlp from extras

Checklist

[x] I've formatted my code with Black
[x] I've ensured my code works on Python 3.10.x
[x] I've tested my code

I suggest moving this to the ext package

Moving what? Im removing it 😅


Add Lottie animations

Add suitable lottie animations that should appear when search results are loading, movie details are loading and at any other relevant places.
https://lottiefiles.com/getting-started

Hi @ken1000minus7,
I'm interested. Can you please assign it to me.. Also if you have a specific animation in mind, then please share it's link. Else I'll try to find out any suitable one.
Thanks!

@Devendra34 assigning you 😇, there are no particular animations decided on yet, they just have to be suitable enough for the place they are used at. An animation for searching is already in the app, you can check that for reference if you want😊

Hi,
The developer seems to be inactive on this issue! Can I be assigned to this?

Ok sure @its-me-debk007 , assigning u 👍


[Feature Request] Add support for SDXL-Lightning

Thank you for this great work! I really appreciate the continuous improvements and new features being added to this project. I would like to request the addition of support for SDXL-Lightning.

Currently, I am utilizing the UNet from sdxl_lightning_4step_unet.safetensors and two text encoders from SG161222/RealVisXL_V4.0. With the prompt "a cute dog with a hat on a skateboard" and a guidance scale of 1, I generated the following result:

Unfortunately, the quality of the generated image is not as expected. Could you please provide any suggestions or changes that could help improve the output quality?
Thank you!


Android: Playing Videos don't fit fully in screen window

Android (and maybe iOS too?): Playing Videos don't fit fully in screen window
Original: https://jira.numenta.com/browse/WEB-285
cc @jsfowles

@jsfowles good one for you


what's the meaning of 训练结束, 您可查看控制台训练日志或实验文件夹下的train.log ?

i completeㅇ all preparation steps and pushed the train model button but it just printed that message soon.
didn't activated tensor board.
what should i do?

"training is over, you can check the console training log or train.log under the experiment folder"


Have an article listing best practices for Blazor projects

Depending on the content we may choose to have a single or two separate articles for this for Blazor Client and Blazor server.
The content here should be provided by engineers.

We've already covered many aspects of Blazor apps in different articles including detailed perf and security guidance.


New package SaferNetworking.RegAlyzer.1 version <IP_ADDRESS>

Microsoft Reviewers: Open in CodeFlow

One or more of the installer URLs doesn't appear valid.
This may happen for sites with policies prohibiting distribution or use by third parties.
This may happen for URLs pointing to domains that do not align with the publisher domain or package domain. If you could provide supporting evidence from the publisher that the URLs for the installer are correct, that would help us to validate and approve this PR.

The URL is legit. The installer URL is https://download.spybot.info/RegAlyzer/regalyz-<IP_ADDRESS>.exe, and the parent domain https://spybot.info redirects to the publisher URL https://www.safer-networking.org/. If not possible to approve this, it would also be possible to change the package URL to https://download.spybot.info/RegAlyzer/. That URL is still dedicated to the package even though not the homepage of the package.

@wingetbot waivers Add Validation-Domain


Dunfell branch does not build due to new override syntax being used

Hi!
I am not sure whether I misunderstand something about the Yocto versioning here...
It seems like it is not possible to build the dunfell branch starting from the following commit:
https://github.com/aws4embeddedlinux/meta-aws/commit/6d4bd338b7fa2c535f84c1be1efbc841fe64fd25
It is because some recipes contain overrides with the new syntax. Is it possible to somehow use those recipes on dunfell? Am I missing something?

depends on the version on Dunfell you're using. After last August, Dunfell was made to use both old and new syntax. Also when doing so ensure to change the local.conf setting for conf version to 2.

Thanks for the info @rpcme. I knew I was missing something! Will update our dunfell version! Thanks!!


Handle array properties correctly (next branch)

Same as #243 but for next branch

hal-json-vuex 2.0.0-alpha.11 is published.

hal-json-vuex 2.0.0-alpha.11 is published.

@carlobeltrame Can you have a look if the package was properly built before publishing 2.0.0-alpha.11? I struggle to get things working when downloading the package via npm. When building locally and adding via yalc, everything works as it should.
First I thought it could be a caching issue, but removed cache and redownloaded from npm multiple times - still doesn't work.

Hmmm good question. I thought the publishing tool np should build automatically before releasing, but looks like it doesn't. I just published alpha.12, can you try with that?
I'll have to show you soon how to release so you can also do it whenever you need it.

Yes, much better 😃
PR is here: https://github.com/ecamp/ecamp3/pull/2337


[Bug 66184] Ensure that all root loggers have a level or parent

Since Java 8 the default level of a root logger is no longer set in the
constructor. This PR sets the level to INFO (LogManager's default) if it
is absent.
This patch should work on all the branches.

Thanks for the bug report and PR. I just want to clean up a few minor points (e.g. add a change log entry) and then I'll back-port this.


Add option to download assets with original filename

Currently stactask will download assets and name the file as the <asset_key>., thereby keeping the extension, but possibly (probably) renaming the file.
Sometimes a subprocess will expect the file to be named the same as it was originally.   Present as option.

@matthewhanson @philvarner @gadomski  -- I'd like to motion this move from "add an option" to "make original filenames The Behavior™".  But any valid argument for f"{asset_key}.{ext}" file names being preferable will easily convince me otherwise.
And yes, I've run into some external science code which depends on filenames .... again.

This is supported by stac-asset, although don't think it's the default.
What we really need to do is drop fsspec here in favor or stac-download.

Ok, that makes sense. I'll keep a fork for immediate use, and consider #76.

But any valid argument for f"{asset_key}.{ext}" file names being preferable will easily convince me otherwise.

I'm not sure which should be the default, but one benefit of f"{asset_key}.{ext}" is that you're guaranteed to not have name clashes when downloading all assets to a single directory (as long as you don't have duplicate dictionary keys). There's cases for both — IMO the "I just want to see what's there" use-case wants key-based, whereas the "I'm using this in a data pipeline" use-case wants orig-file-name.

name clashes when downloading all assets to a single directory

OK, that's the trick: asset-key (or any) file naming can run amuck if path_template doesn't separate items out by ${id}.  Seems like the PR may be useful until #76 can be addressed.  Incoming.


Mayhem missing?

Can't seem to find Mayhem in the package list, though it appears on the "Included software" list on the website. Has it been left out?

Hi. I think you missed the note at the very top of the README file for this archive. Nothing here is expected to work anymore since about Retropie 4.3 when they made major changes to how everything worked and I didn't have time/desire to work on it any longer thanks to some extremely entitled people in the RetroPie community.
But good news, after being away from Retropie for a few years, I just bought a Pi 4 and am in the middle of setting it up so I'll have the bug to get this stuff working again. I may not do everything, but we'll see. No promises.

Oh, I see!
Very sorry to hear that. I saw some of your posts on the forum, and gee,
seems like a hostile environment.
Anyway, I'd love to try some of the stuff if you do end up fixing it for
the new versions!
Keep up the good work!
man. 24. aug. 2020 kl. 21:10 skrev zerojay<EMAIL_ADDRESS>
Hi. I think you missed the note at the very top of the README file for
this archive. Nothing here is expected to work anymore since about Retropie
4.3 when they made major changes to how everything worked and I didn't have
time/desire to work on it any longer thanks to some extremely entitled
people in the RetroPie community.
But good news, after being away from Retropie for a few years, I just
bought a Pi 4 and am in the middle of setting it up so I'll have the bug to
get this stuff working again. I may not do everything, but we'll see. No
promises.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zerojay/RetroPie-Extra/issues/315#issuecomment-679312080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPHBUS7GD3MTXLKUF3SJ7LSCK3KLANCNFSM4QJFNIQA
.

Here's the reason it was moved: https://github.com/zerojay/RetroPie-Extra/commit/a34ef08709861bb402402fd8339515059b8107a3#diff-85d09d266b45472bfc9f5c862f07becc
I'll give it a shot though.

Mayhem is still broken and still needs to be fixed upstream as it asserts during startup. The script that checks how much ram is allocated to the GPU is also broken, but the main issue is Allegro asserts on start. As mentioned in issue #280, this will need to be fixed upstream as there is nothing I can do for this.


Virus Total URL Enrichment API Error

Describe the bug
API error when attempting to use URL enrichment playbooks
-URL Enrichment - Virus Total domain report
-URL Enrichment - Virus Total report
To Reproduce

Enable & Configure Playbook
Open Logic App Designer
Expand 'Get Domain report' or 'Get URL Report'
See error: "Operation not found under connector '/subscriptions//providers/Microsoft.Web/locations//managedApis/virustotal'
After execution, step will show error output:
"{
"statusCode": 404,
"message": "Resource not found"
}"

Expected behavior
No error and playbook execution completion
Screenshots

Desktop (please complete the following information):

OS: Win 11
Browser Chromium Edge
Version 97

Additional context
No issues running other VirusTotal Enrichment Playbooks, such as File Hash or IP.

@Dan156478  seems like the connection didnt deploy.  if you click on API Connections do you see the connection resource?  like this:

Hi @dicolanl, Looks to be be deployed & connected. I've tried using an existing connection that we are successfully using for the VirusTotal FileHash & IP playbooks, and also tried creating a new connection, both result in the same error.

@Dan156478  found the issue that get url report action doesnt exist anymore and has been replaced.  i updated the logic app and submitted a new PR.

@dicolanl Perfect, thank you!

Hi, the same happens for me with Get IP report (same error, and it worked until last week)


5-fold cross-validation results

Hey there,
thanks a lot for the repo! It's really useful to have all these methods in one place.
When looking through the results I was a bit confused though: The lesion scores are way higher and the Liver scores are way lower than what we have in our internal validation of most of these models. Also the 2D model is usually a lot worse than the 3D nnU-Net. Can you be more specific how these scores were computed? Are you using the same folds as nnU-Net and averaging over all five folds? How are the metrics computed?
This is what we are getting with the current nnU-Net 3d_fullres for Liver and Lesion. Dice computed on each image separately and then averaged over images:
fold0: 0.9651	0.6547
fold1: 0.9613	0.5669
fold2: 0.9509	0.6783
fold3: 0.9574	0.7061
fold4: 0.9515	0.5955
Average: 0.9572	0.6403
These numbers are consistent with the results reported in our supplement:  0.9576 0.6253
Just an idea: It would be really useful to extend the table with computational requirements to that users know what they are getting into. GPU hours required for training and VRAM requirements can help a lot to put results into perspective
Best,
Fabian

Hi Fabian,
First off, I would like to thank you for the invaluable contributions of nnU-Net and its flexibility, which have greatly benefited our hybrid human-AI annotation process within our dataset.
Currently, I'm working with the LiTs evaluation code, and I've observed that it reports the lesion dice as an average across all lesions. To avoid confusion, we've made this note in our ReadMe, along with a reference link to the full model_summary_metrics.csv.
In our analysis, the standout model based on lesion dice per case is MedNext-L-5, with a score of 0.5780 (±0.0635). The 2D nnU-Net model has more competitive results than its 3D counterparts, with the latter scoring 0.5117 (±0.0405), which is beneath the benchmark of 0.6403.
I speculate that the discrepancy might stem from the split as well as the different training durations, specifically, running the benchmark for 100 epochs rather than 1000, possibly leading to under-convergence in the 3D nnU-Net. I have also highlighted this in the ReadMe model training.
I've uploaded the output of running nnUNetv2_plan_and_preprocess for your reference here.
Would you be able to confirm the plan generated and the specific data splits follow your internal validation?
I should extend the table with the computational requirements.
Thank you for your time and support.
Warm regards,
Marawan

Hey Marawan,
thanks a lot for the clarifications, also in the readme. Since the repository is advertised as a starting point for participating in challenges the provided benchmarks will help users a lot in putting their results into perspective. In that context it might be sub-optimal though to make model recommendations based on very short runs, without clearly outlining that immediately next to the table. You probably don't want to point your participants in a wrong direction by suggesting that a 2D model can be competitive on your data. It could also be useful to directly link the evaluation code next to the csv so that one can immediately see what was being done.
One thing I am not happy with is the fact that the standard nnUNetTrainer was modified in this repository to only run for 100 epochs. This is no longer the nnU-Net and should not be listed in a way that would suggest it is. Users will use this repository thinking it is the nnU-Net and then be dissatisfied with the performance. It would have been more transparent to leave all the nnU-Net trainers untouched and to create new ones for each specific purpose: nnUNetTrainer_100epochs, nnUNetTrainerSAMed, nnUNetTrainerSAMed_100epochs, .... These could then also be listed in the benchmarking table to make it perfectly transparent what is being measured.
Thanks a lot for putting all of this together! The challenge looks super interesting and you might see a submission from our side as well ;-)
Best,
Fabian

Aloha Fabian,
To eliminate any confusion among users, I've made necessary adjustments to the trainers and added the epoch count next to the relevant table. Given the request for a benchmark set at 1000 epochs, it's clear that this process is both time and compute-intensive (the purpose of this repo is to assist users, not to promote a competitive environment with an optimal performance). Currently, I'm allocating time to run all models at 1000 epochs.
Notably, after completing the 5-fold for 3D nnU-Net in about 60 hours, roughly equating to 2.5 days, we observed higher KPIs on our data split compared to the results you've reported.
metrics_summary: liver dice per case: 0.9628 (±0.0062) and lesion dice per case: 0.6698 (±0.0560)
This outcome prompted another table in the ReadMe, reflecting these findings. We plan to update this table further with the results, model weights, VRAM for the all baselines once those are finalized.
Thank you for your suggestions and interest, and I look forward to your participation in the challenge.
Warm regards,
Marawan


Cache Issue

I'm getting the following errors with the plugin which breaks the entire site.
Fatal error: Uncaught Error: Class 'Google\Auth\Cache\MemoryCacheItemPool' not found in /var/www/content.homesliceapp.com/public/wp-content/plugins/wp-stateless/lib/Google/src/Google/Client.php on line 1025
Error: Class 'Google\Auth\Cache\MemoryCacheItemPool' not found in /var/www/content.homesliceapp.com/public/wp-content/plugins/wp-stateless/lib/Google/src/Google/Client.php on line 1025
Has anyone else had an issue like this?

If you are using git ignore then check if you are ignoring the Cache folders. If so, then make it so that it does not ignore the following path: wp-content/plugins/wp-stateless-new/lib/Google/vendor/google/auth/src/Cache

Hello @seanhudson
Is issue still relevant?

Closing the ticket.
@seanhudson  Please re-open the ticket if you still have this issue.


Refactor ReactPanel

ReactPanel is a class that creates vscode WebView panel from Ethcode main React app. This class can be moved out from extension.ts

after v2 merge ethcode will not use react panel anymore


jobs: fix effect/gain/you dont trigger self gains buffs

This bug would break something like BuffTracker that
relies on the event, so buffs would not correctly trigger on the event
when the player gains some effect but when the others gains effect
from the player.
This patch fixed this. But I am a bit worrying whether the name is
clear enough for what is it used for.

This patch fixed this. But I am a bit worrying whether the name is clear enough for what is it used for.

The name is clear to me, for what it's worth.


add sorting to bus parsing

Makes the Indexing Dict match a sorted order of the buses.
Closes #218 and addresses #125
@chriscoey This solves inconsistencies between the PhaseShifter and TapTransformer Models in PowerModels/Matpower and the one in PowerSystems.jl

Codecov Report

Merging #217 into master will increase coverage by 1.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #217      +/-   ##
==========================================
+ Coverage   57.88%   58.91%   +1.02%
==========================================
  Files          51       51
  Lines        3842     3838       -4
==========================================
+ Hits         2224     2261      +37
+ Misses       1618     1577      -41

Impacted Files
Coverage Δ

src/parsers/pm2ps_parser.jl
92.7% <100%> (+0.03%)
:arrow_up:

src/utils/ybus_calculations.jl
97.29% <100%> (+97.29%)
:arrow_up:

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 1e2fcf7...aeb517c. Read the comment docs.

Codecov Report

Merging #217 into master will increase coverage by 1.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #217      +/-   ##
==========================================
+ Coverage   57.88%   58.91%   +1.02%
==========================================
  Files          51       51
  Lines        3842     3838       -4
==========================================
+ Hits         2224     2261      +37
+ Misses       1618     1577      -41

Impacted Files
Coverage Δ

src/parsers/pm2ps_parser.jl
92.7% <100%> (+0.03%)
:arrow_up:

src/utils/ybus_calculations.jl
97.29% <100%> (+97.29%)
:arrow_up:

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 1e2fcf7...aeb517c. Read the comment docs.


Preview pickup chat

Bruno from Solikyl said:

A friend of mine using Karrot here also suggested to me, when I commented the new [pickup chat] feature, that the chat should be open for others to read at least before joining the pickup, so other people would see how what is being discussed and feel motivated to join. I'm still unsure whether that's a good option or not

It would be a bit like slack channel previews - and the user would not become part of the chat, but just have read-only view of the chat.

Thanks! I have the feeling it makes sense to think about this in the context of store conversations & subscriptions, so I'll close this in favor of #986


NL validation not working properly

A recent change only allows the #### @@ format for The Netherlands. I can also see DHL describes this format. Unfortunately it's very common in The Netherlands to write it as ####@@ and all companies can accept this format. Would you be open to a PR to adjust this behavior again or will this library stick to the DHL formats?

The validator can ignore whitespace by passing a value of true in the 3rd argument to isValid()


Different gamemodes? [question]

Hello,
I saw a lot of photos of different maps/gamemodes in the photos folder and I was wondering if those were planned or not yet implemented?
I would love to help with the development of the maps if needed.
Thanks

Write me in discord

Write me in discord

I don't know what your discord it.
Here's mine so you can add me: ThruBeing#0554


Cannot access protected property cpanelAPI::$scope

I can't seem to find helpful info on this error - hope someone can help.
The troublesome line is $uapi->scope = $username.'_Mysql';  where $username is my cpanel username.
I also had to change the example instantiation line from
$uapi = new cpanelUAPI(  to
$uapi = new cpanelAPI(  because there is no cpanelUAPI class in cpaneluapi.class.php
any ideas?

@Brent72 - what are you specifically trying to achieve?
The usage is as follows:
$api = new cpanelAPI($username, $password, $url);
$api->uapi->Mysql->get_server_information();

You can't access scope directly, instead you set it by treating it like an object property.

Thanks for the quick response.
The example in the repo seems to be incorrect. Line 26 is as follows:
$uapi->scope = 'Mysql'; // set the scope to the module we want to use. NOTE: this IS case sensitive.
That was where I was tripping up.

@Brent72 Ah, that's my bad. The readme has been updated since those changes were made, but i guess the examples haven't been.
I'll add it to the list of things to sort ASAP.
Any other issues? or can i close this one? :)

Yeah - thanks close this one.
Kind regards...
Brent.
On Fri, 8 Mar 2019 at 00:46, Toby New<EMAIL_ADDRESS>wrote:

@Brent72 https://github.com/Brent72 Ah, that's my bad. The readme has
been updated since those changes were made, but i guess the examples
haven't been.
I'll add it to the list of things to sort ASAP.
Any other issues? or can i close this one? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/N1ghteyes/cpanel-UAPI-php-class/issues/13#issuecomment-470530696,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALR8DMnKNnsRyPnrqcchbANu-0nHIxSiks5vURgWgaJpZM4bgHii
.

@Brent72 Ah, that's my bad. The readme has been updated since those changes were made, but i guess the examples haven't been.
I'll add it to the list of things to sort ASAP.
Any other issues? or can i close this one? :)

Hello, will you update when?
I'm in doubt of what is correct ... lol


Support arrays in CLI arguments

Closes #1267
Allows TSTL to receive arrays for its parameters via the CLI from a comma separated value
tstl -p tsconfig.json --noResolvePaths patha,pathb

Let me know if there's some other edge cases I may be missing
Edit:
Also supporting arrays of objects parameters (only effects luaPlugins) by parsing a JSON array instead of a comma separated string
tstl -p tsconfig.json --luaPlugins '[{ "name": "my-plugin" }]'

Can we add objects too? It would be really nice to specify plugins as command-line options too. (I have a crash debugger plugin that is only conditionally turned on when debugging.)

How would you like to see this on the CLI?
Current option I'm thinking of is
$ tstl -p tsconfig.json --luaPlugins my-plugin-1,my-plugin-2@my-plugin-import
                                     |  name   | |  name   | |    import    |

But this could become more complex if we add more options to plugins
Initially I thought JSON object as both name and import can be specified for a single plugin and we can have multiple plugins. This helps if we support more options in the future
$ tstl -p tsconfig.json --luaPlugins '[{ "name": "my-plugin-1" }, { "name": "my-plugin-2" }]'

It looks quite verbose but at least grants the ability to control these plugins via CLI
Perhaps name only might be a solution
$ tstl -p tsconfig.json --luaPlugins my-plugin-1,my-plugin-2

Or maybe we need some other CLI options to include/exclude plugins so users don't have to deal with a potentially tricky CLI syntax
TypeScript doesn't have any examples to pull from and the last time I needed to specify an object via CLI I just used JSON 🤔

Ready for review @Perryvw!


Navigation bar not showing opened page

The navigation bar does not make the currently opened page text (nav-bar) bold. Home is always bold no matter the page.

Is this a duplicate to #17 ?


Example of CMS.registerIcon in the doc causes error

I am afraid that an example of CMS.registerIcon shown in the doc seems to cause error. I had to make a small modification to make it work.
Unhandled Runtime Error
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object.

Does not work
CMS.registerIcon('house', <FontAwesomeIcon icon={faHouse} size="lg" />);

Works
    CMS.registerIcon('house',()=><FontAwesomeIcon icon={faHouse} size="lg" />);

Note: an arrow function of () => added.
Please update the documentation. Thank you.

The docs have been updated. Thanks for catching that.


Could you help remove the vulnerability introduced in your package?

Hi, @xzyfer @nschonni, there is a vulnerability introduced in your package node-sass:
Issue Description
A vulnerability CVE-2021-33623 is detected in package trim-newlines(<3.0.1,>=4.0.0 <4.0.1) and<EMAIL_ADDRESS>is transitively referenced by<EMAIL_ADDRESS>We noticed that such a vulnerability has been removed since<EMAIL_ADDRESS>However, node-sass's popular previous version<EMAIL_ADDRESS>(2,589,226 downloads per week) is still transitively referenced by a large amount of latest versions of active and popular downstream projects (about 8,154 downstream projects, e.g., @codecademy/styleguide 41.12.3, @spryker/oryx-for-zed 2.11.1, @pnp/spfx-controls-react 3.2.1, nuxt-cookie-control 1.9.7, liferay-theme-tasks 10.5.0, smc-hub 1.5.2, @igor-lemon/antd-scss-theme-plugin 3.0.0, etc.).
As such, issue CVE-2021-33623 can be propagated into these downstream projects and expose security threats to them.
These projects cannot easily upgrade node-sass from version 4.14.1 to (>=6.0.1). For instance<EMAIL_ADDRESS>is introduced into the above projects via the following package dependency paths:
<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>(2<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>......
The projects such as jstransformer-scss and scss-to-json, which introduced<EMAIL_ADDRESS>are not maintained anymore. These unmaintained packages can neither upgrade node-sass nor be easily migrated by the large amount of affected downstream projects.
On behalf the downstream users, could you help us remove the vulnerability from package<EMAIL_ADDRESS>Suggested Solution
Since these inactive projects set a version constaint 4.14.* for node-sass on the above vulnerable dependency paths, if node-sass removes the vulnerability from 4.14.1 and releases a new patched version<EMAIL_ADDRESS>such a vulnerability patch can be automatically propagated into the 8,154 affected downstream projects.
In<EMAIL_ADDRESS>you can kindly try to perform the following upgrade:
meow ^3.7.0 ➔ ^6.0.0;
Note:
meow@6.0.0(>=6.0.0) directly depends on<EMAIL_ADDRESS>which has fixed the vulnerability (CVE-2021-33623)
Thank you for your contributions.
Best regards,
Paimon

This CVE is pretty much a non-issue here. It's a package used by the CLI portion of the library. If something has access to try and DDoS you server that way, you've got other problems. https://overreacted.io/npm-audit-broken-by-design/
I believe this was asked on several other threads, but we have not done multiple supported releases in the past


:white_check_mark: Support HD Fonts [done, requires next inav release]

Since now the Digital Systems have grown up and support full OSD, we get more and more HD fonts that would match our screen resolution.
Do you see any chance to support a nice HD font? Could be helpful also to showcase them.

HD OSD support is implemented.
It will be released after next inav release, because it requires changes in inav.
Related PR to inav: https://github.com/iNavFlight/inav/pull/9327

HD OSD is suported in Inav 7.0 RC1 and plugin v 1.4.0


Fix auto scaling - don't set inactive write scaling on active tables

Fixes #749
Most of this PR is setting up the unit test.

Seems reasonable from your description, but I have not spent much time in this area of the code.


[BUG] Video Pauses Every Time You Try to Play It on IOS (CapacitorCookies + Datadog Integration Issue)

Description:
When using Datadog's automatic tracking with CapacitorCookies: { enabled: true } in a Capacitor app on iOS, video playback for video tags is interrupted or fails to function properly. This issue affects all video elements and any libraries that use them under the hood.
Through testing, it was identified that Datadog’s automatic tracking of user interactions (e.g., route changes, button clicks) conflicts with video playback on iOS. Disabling CapacitorCookies or setting trackViewsManually in Datadog can resolve the issue temporarily, but the underlying cause seems related to how Capacitor handles cookies in combination with Datadog’s tracking.
Environment:
IOS device/emulator 17.5 ( any version bug is reproducable )
Plugin
CapacitorCookies with capacitor 6.0.0( any version bug is reproducable )
Step to reproduce
1)Initialize a Capacitor iOS app with the following configuration:
datadogRum.init({
trackViewsManually: false,
...
});
2)Enable CapacitorCookies in capacitor.config.json:
{
"CapacitorCookies": {
"enabled": true
}
}
3)Add a video tag to any component or page:

  Sorry, your browser doesn't support embedded videos.

4)Run the app on an iOS device/emulator and attempt to play the video.
Expected Behavior:
The video should play without any interruptions or issues.
https://github.com/user-attachments/assets/0e83965c-a979-473d-b9a2-f806d630cb23
Actual Behavior:
The video fails to play or stops unexpectedly.
https://github.com/user-attachments/assets/eaa38f81-af82-4f68-b325-00d8b1da13d7
Minimal reproducible example (please use your own Datadog credentials)
https://github.com/bulletbrand/safe-area-demo-repo

I’ve already created a similar issue on the Capacitor repo, but if you have any ideas or the opportunity to take a look, I would greatly appreciate it. https://github.com/ionic-team/capacitor-plugins/issues/2190
There seems to be a problem with integrating these tools

We are also seeing the issue, but do not have Datadog, disabling CapacitorCookies solves the issue, but as we depend on it, that is not a viable solution.


Clarification regarding hand tracking data returned by HAND_MOVE event

@geyang
I am getting hand tracking data from a meta quest 3 and wasn't able to find information regarding the following in the documentation:

What values of from the 25x16 values of each hand correspond to the respective wrists? I am currently working off the assumption that the first 16 values correspond to the wrists.
The hand tracking data is relative to the wrist transformation, does that mean the wrist is always at origin of the world frame because the position values in my "assumed" wrist matrix are always 0,0,0? Shouldn't the headset be at the origin?

Thanks for the question. I have updated the documentation here:

Hand Tracking Example

Important: Please refer to the newest documentation here: latest | Hand_tracking
I reproduce the content below: (Last Update: 2024-10-19)
Hand API
You can get the full pose of the hands by listening to the HAND_MOVE event.
You can add flags left and right to specify which hand you want to track.
The returned data looks like the following:
/**
 * Left and right pose are relative to the wrist transformations.
 */
export type HandsData = {
  left?: Float32Array;       // 25 * 16 values.
  right?: Float32Array;      // 25 * 16 values.
  leftState: HandState;
  rightState: HandState;
};

export type HandState = {
  pinch: boolean;
  squeeze: boolean;
  tap: boolean;

  pinchValue: number;
  squeezeValue: number;
  tapValue: number;
}

    from vuer import Vuer, VuerSession
    from vuer.schemas import Hands
    from asyncio import sleep

    app = Vuer()

    @app.add_handler("HAND_MOVE")
    async def handler(event, session):
        print(f"Movement Event: key-{event.key}", event.value)

    @app.spawn(start=True)
    async def main(session: VuerSession):
        # Important: You need to set the `stream` option to `True` to start
        # streaming the hand movement.
        session.upsert @ Hands(fps=30, stream=True, key="hands")

        while True:
            await sleep(1)

Matrix format
All 4x4 transform matrices used in WebGL are stored in 16-element Float32Arrays.
The values are stored in the array in column-major order; that is, each column is
written into the array top-down before moving to the next column to the right and
writing it into the array. Therefore, for the array [a0, a1, a2, …, a13, a14, a15],
the matrix looks like this:
                                  ⌈  a0 a4 a8 a12  ⌉
                                  |  a1 a5 a9 a13  |
                                  |  a2 a6 a10 a14 |
                                  ⌊  a3 a7 a11 a15 ⌋

For details, refer to the MDN documentation on XR Rigid Body Transformation
Hand Landmarks
We follow the XR Hand
conventions, and return the landmarks in a single array of 25 * 16 values in
the following order:

Hand joint
Index
Hand joint (continue)
Index

wrist
0
middle-finger-phalanx-distal
13

thumb-metacarpal
1
middle-finger-tip
14

thumb-phalanx-proximal
2
ring-finger-metacarpal
15

thumb-phalanx-distal
3
ring-finger-phalanx-proximal
16

thumb-tip
4
ring-finger-phalanx-intermediate
17

index-finger-metacarpal
5
ring-finger-phalanx-distal
18

index-finger-phalanx-proximal
6
ring-finger-tip
19

index-finger-phalanx-intermediate
7
pinky-finger-metacarpal
20

index-finger-phalanx-distal
8
pinky-finger-phalanx-proximal
21

index-finger-tip
9
pinky-finger-phalanx-intermediate
22

middle-finger-metacarpal
10
pinky-finger-phalanx-distal
23

middle-finger-phalanx-proximal
11
pinky-finger-tip
24

middle-finger-phalanx-intermediate
12
-
-


iox-#2167 iox-#2156 Update SpscFiFo

Pre-Review Checklist for the PR Author

[x] Add a second reviewer for complex new features or larger refactorings
[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[x] All checks have passed (except task-list-completed)
[x] All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
[x] Assign PR to reviewer

Notes for Reviewer

See discussion
Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #2167
Closes #2156

@albtam something weird happened with your last force push. It seems the e-mail is now unverified

@albtam you do not need to close the PR just of the e-mail address :)


updated line 345

Changed command New-CsOnlineVoiceRoute to Get-CsOnlineVoiceRoute so that it correctly correlates with line 342

@nmurav Ken wanted to make sure you were kept in the loop for this update.


Fixes for new survey changes

Motivation and Context
acceptance test changes for survey service
What has changed
changed tests to acomodate changes
How to test?
run ATs
Links
https://trello.com/c/GvoKYQUd/1077-merge-in-ryans-acceptance-test-changes-after-survey-service-changes-have-made-it-to-master

Happy with this, though would personally like to see a reply to @necrophonic

spoke to him about it, hardcoding the 0001 because its a form type not an RU ref. Also most of the codebase uses 0001 type for forms. Only real alternative would be to random a formtype form an array of them but there's really no benefit to it.


refactor: generate data-loader only by getData

fix https://github.com/ice-lab/ice-next/issues/513
不以 getAppData 作为是否生成 data-loader 的依据
用户声明 getAppData 可能只是为了获取全局性的数据，data-loader 只受是否定义了 getData 决定

Codecov Report
Base: 82.12% // Head: 88.32% // Increases project coverage by +6.19% :tada:

Coverage data is based on head (29a9745) compared to base (5d10f02).
Patch coverage: 95.20% of modified lines in pull request are covered.

Additional details and impacted files
@@                          Coverage Diff                          @@
##           chore/remove-consola-from-runtime     #514      +/-   ##
=====================================================================
+ Coverage                              82.12%   88.32%   +6.19%
=====================================================================
  Files                                     90      109      +19
  Lines                                   7669     9445    +1776
  Branches                                 971     1298     +327
=====================================================================
+ Hits                                    6298     8342    +2044
+ Misses                                  1364     1095     -269
- Partials                                   7        8       +1

Impacted Files
Coverage Δ

packages/plugin-rax-compat/src/index.ts
70.40% <0.00%> (ø)

packages/ice/src/config.ts
77.57% <33.33%> (+1.99%)
:arrow_up:

packages/runtime/src/Document.tsx
82.08% <33.33%> (ø)

packages/rax-compat/src/intersection-observer.ts
79.44% <42.85%> (ø)

packages/ice/src/createService.ts
95.12% <93.33%> (+0.27%)
:arrow_up:

packages/ice/src/esbuild/cssModules.ts
88.34% <100.00%> (+19.49%)
:arrow_up:

packages/ice/src/esbuild/transformPipe.ts
78.34% <100.00%> (+3.83%)
:arrow_up:

packages/ice/src/service/runtimeGenerator.ts
83.75% <100.00%> (ø)

packages/ice/src/tasks/web/index.ts
100.00% <100.00%> (ø)

packages/ice/src/utils/renderExportsTemplate.ts
92.10% <100.00%> (ø)

... and 58 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

更倾向于增加配置项，现阶段 data-loader 还存在重复打包 request 等库的问题


Microsoft Defender Antivirus blocked AntiAlexa due to Backdoor:Win32/Bladabindi!ml

Is this false alarm?

Yes it is!
You can see and compile the source code of my program and, as you can see, there is no malicious code in it.
The false positive is probably caused by the the program's attempt to copy itself into Windows startup folder.
Please, if you can, report the case of false positive to Windows Defender to prevent other users from running into the same problem, thank you!

I reported the false positive of Windows Defender to Microsoft Security Intelligence and they answered me in the following way:

At this time, the submitted files do not meet our criteria for malware or potentially unwanted applications. The detection has been removed. Please follow the steps below to clear cached detections and obtain the latest malware definitions.

 1. Open command prompt as administrator and change directory to c:\Program Files\Windows Defender
 2. Run “MpCmdRun.exe -removedefinitions -dynamicsignatures”
 3. Run "MpCmdRun.exe -SignatureUpdate"

Alternatively, the latest definition is available for download here: https://docs.microsoft.com/microsoft-365/security/defender-endpoint/manage-updates-baselines-microsoft-defender-antivirus

Since the detection has been removed you shouldn’t have any more problems 👍


chore: improve post integration action

Improve post integration action to get rid of test run Fatal Error:
<--- Last few GCs --->

[5519:0x5766980]   289914 ms: Scavenge (reduce) 3913.7 (4142.6) -> 3913.2 (4142.6) MB, 6.6 / 0.0 ms  (average mu = 0.207, current mu = 0.220) allocation failure;
[5519:0x5766980]   289929 ms: Scavenge (reduce) 3913.9 (4142.6) -> 3913.4 (4142.8) MB, 6.5 / 0.0 ms  (average mu = 0.207, current mu = 0.220) allocation failure;
[5519:0x5766980]   289943 ms: Scavenge (reduce) 3914.2 (4142.8) -> 3913.7 (4143.1) MB, 6.4 / 0.0 ms  (average mu = 0.207, current mu = 0.220) allocation failure;

<--- JS stacktrace --->

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0xb6e500 node::Abort() [/opt/hostedtoolcache/node/18.12.1/x64/bin/node]
 2: 0xa7e632  [/opt/hostedtoolcache/node/18.12.1/x64/bin/node]
 3: 0xd47f20 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/opt/hostedtoolcache/node/18.12.1/x64/bin/node]
FAIL src/features/nfts/context/__test__/context.test.tsx
  ● Test suite failed to run

    A jest worker process (pid=5519) was terminated by another process: signal=SIGTERM, exitCode=null. Operating system logs may contain more information on why this occurred.

      at ChildProcessWorker._onExit (../../node_modules/jest-runner/node_modules/jest-worker/build/workers/ChildProcessWorker.js:3[70](https://github.com/input-output-hk/lace/actions/runs/8420025827/job/23053879871#step:5:71):23)

<--- Last few GCs --->

[5518:0x62df9[80](https://github.com/input-output-hk/lace/actions/runs/8420025827/job/23053879871#step:5:81)]   299414 ms: Scavenge (reduce) 3[91](https://github.com/input-output-hk/lace/actions/runs/8420025827/job/23053879871#step:5:92)3.5 (4140.5) -> 3913.3 (4140.7) MB, 6.5 / 0.0 ms  (average mu = 0.213, current mu = 0.199) allocation failure;
[5518:0x62df980]   299422 ms: Scavenge (reduce) 3914.3 (4141.1) -> 3913.9 (4141.1) MB, 7.0 / 0.0 ms  (average mu = 0.213, current mu = 0.199) allocation failure;
[5518:0x62df980]   29[94](https://github.com/input-output-hk/lace/actions/runs/8420025827/job/23053879871#step:5:95)32 ms: Scavenge (reduce) 3915.1 (4141.6) -> 3914.9 (4141.6) MB, 7.8 / 0.0 ms  (average mu = 0.213, current mu = 0.199) allocation failure;

and clean-up a bit output logs

Changes unknown
when pulling b27919ee51e2863877cbd9d72f6fad9f86b1fa29 on chore/post-integration-action-improvements
into ** on main**.


JW-79 Packet memory management

I've addressed the issues. Please review again.

Fixed memory leaks which happened in certain circumstances and corrected some formatting issues.

Bump.


cmake can not find python3 (missing: Development Development.Module   Development.Embed) (found version "3.8.9")

Please create a discussion in https://github.com/microsoft/vcpkg/discussions/new instead.
Hi, I want to install openeb which so I got the following code:
PS C:\Users\masiy\openeb\build> cmake .. -A x64 -DCMAKE_TOOLCHAIN_FILE=C:\Users\masiy\openeb\cmake\toolchains\vcpkg.cmake -DVCPKG_DIRECTORY=C:\src\vcpkg -DBUILD_TESTING=OFF
But I terminal reports an issue:
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19044.
-- Building METAVISION_SDK_MODULES_OPEN modules : base;core;driver;ui
-- Building METAVISION_SDK_MODULES_PROFESSIONAL modules : analytics;calibration;cv;ml;3dview;cv3d
CMake Error at C:/Program Files/CMake/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
Could NOT find Python3 (missing: Development Development.Module
Development.Embed) (found version "3.8.9")
Call Stack (most recent call first):
C:/Program Files/CMake/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
cmake/Modules/FindPython/Support.cmake:2993 (find_package_handle_standard_args)
cmake/Modules/FindPython3.cmake:391 (include)
C:/Users/masiy/vcpkg/installed/x64-windows/share/python3/vcpkg-cmake-wrapper.cmake:67 (_find_package)
C:/Users/masiy/vcpkg/scripts/buildsystems/vcpkg.cmake:754 (include)
cmake/custom_functions/python3.cmake:63 (find_package)
CMakeLists.txt:211 (include)
-- Configuring incomplete, errors occurred!
See also "C:/Users/masiy/openeb/build/CMakeFiles/CMakeOutput.log".
See also "C:/Users/masiy/openeb/build/CMakeFiles/CMakeError.log".
Could someone tells me how to solve this problem? Thanks very much.

@MariaSabrinaMa Thanks for posting this issue,
Do you have multiple versions of pythone3 and make sure that  you using the function target correctly in CMakeLists.txt.
find_package(Python3 COMPONENTS Development REQUIRED)
target_link_libraries(main PRIVATE Python3::Python)

-DCMAKE_TOOLCHAIN_FILE=C:\Users\masiy\openeb\cmake\toolchains\vcpkg.cmake
This looks wrong

@MariaSabrinaMa Is still work for this issue?

We haven't been able to repro this; if more information comes up, or this issue appears again, please reopen.

Hi, the issue is still not solved...

@MariaSabrinaMa Did you run the command .\vcpkg.exe integrate install ?

Yes I did and it did not work. I am pretty sure I have multiple versions of python on my PC. I found the official document from prophesse (the software I want to install) stating that:
In that case, we can see that cmake found some version of the Python interpreter, but it then fails because it does not find other required components from this specific version. The solution is to specify to cmake which version of Python to use with the option -DPython3_EXECUTABLE=<path_to_python_to_use>.
So I added that but I got a new issue:
PS C:\Users\masiy\openeb\build> cmake .. -DBUILD_TESTING=OFF -DPython3_EXECUTABLE=/usr/bin/python3.8
CMake Error: The source directory "C:/Users/masiy/openeb/build/.8" does not exist.
Specify --help for usage, or press the help button on the CMake GUI.
PS C:\Users\masiy\openeb\build> cmake .. -DBUILD_TESTING=OFF -DPython3_EXECUTABLE=C:\Users\masiy\AppData\Local\Programs\Python\Python38
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19044.
-- Building METAVISION_SDK_MODULES_OPEN modules : base;core;driver;ui
-- Building METAVISION_SDK_MODULES_PROFESSIONAL modules : analytics;calibration;cv;ml;3dview;cv3d
CMake Error at C:/Program Files/CMake/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
Could NOT find Python3 (missing: Interpreter Development Development.Module
Development.Embed) (found version "3.8")
  Reason given by package:
      Interpreter: Cannot run the interpreter "C:/Users/masiy/AppData/Local/Programs/Python/Python38"

Call Stack (most recent call first):
C:/Program Files/CMake/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
cmake/Modules/FindPython/Support.cmake:2993 (find_package_handle_standard_args)
cmake/Modules/FindPython3.cmake:391 (include)
C:/Users/masiy/vcpkg/installed/x64-windows/share/python3/vcpkg-cmake-wrapper.cmake:67 (_find_package)
C:/Users/masiy/vcpkg/scripts/buildsystems/vcpkg.cmake:754 (include)
cmake/custom_functions/python3.cmake:63 (find_package)
CMakeLists.txt:211 (include)

Have you added python3 to your system environment variables?

Yes it was added to my system environment variables

@MariaSabrinaMa Since I have no way to reproduce your problem, I can provied the following solutions:

Keep only one version of pyhton in your local
Can you manually run pyhton.exe("C:/Users/masiy/AppData/Local/Programs/Python/Python38/python.exe")
Add cmake -DPython_EXECUTABLE=<full_path_to/python_interpreter>...

Do not forget to cleanup your build directory before running this command. :)

You don't need to pass any additional arguments to cmake find python. If Development is in the list of components, then Interpreter will be the python3 executable provided by vcpkg.
But openeb does something very fishy here:
https://github.com/prophesee-ai/openeb/blob/92aa7c1fc0df52ab53a0b62517255737689f1134/cmake/custom_functions/python3.cmake#L43
So I would say that this is not a bug in vcpkg but in openeb.

@Osyotr Thank you for your reply, I would like to say that currently we are not getting the correct reply and information from the user.
@MariaSabrinaMa

Your toolchanin is not provided by vcpkg
 -DCMAKE_TOOLCHAIN_FILE=C:\Users\masiy\openeb\cmake\toolchains\vcpkg.cmake
According to the log, there is no runnable python interpreter locally, and the user has not replied whether can run python3 normally  Interpreter: Cannot run the interpreter "C:/Users/masiy/AppData/Local/Programs/Python/Python38"
There may be file permission problems, you can try other disks. :)

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!

Uninstalled version Python3.11 midway and installed version Python3.10. Using find_package(Python3 3.10 COMPONENTS Interpreter Development NumPy REQUIRED). A similar issue occurred. Then clean up the build directory and use find_package(), the issue disappeared.


Continious Integration Build Script

Create ruby build script to launch required docker container for the
desired version of couchdb, build android-sync for the desired platform
and tear down the docker container.

+1 once you change those instances of "colocal" to "cloudant local" in comments


Derive Reference Number

Derive reference number  from ISO document number + Stage. Why does the Rice Model have "(E)"?

E for English, F for French. ISO Standards must have E and F versions.

And that means there will need to be a French version of the template once I create it, right?

I think we need to support a document that is in French. I’m not sure if we should support translations inherently (locale per phrase) or on a document level (locale per document). Maybe the latter is easiest.

If the reference number is derived from the ISO doc number, part number, year, and language, then it does not need to be included in the Asciidoc or the XML: the downstream tools can derive it.

Yes, you are correct. Lest forget not the "Stage" abbreviation (e.g., "DIS", "WD", "CD", "NWIP", "FDIS", "IS"...). We just need to set the authoritative number for the downstream tools, right?


Dependency seems to have made a breaking change (vscode-languageserver-node)

After following the instructions here, I ended up with a broken build.
src/bin/server/session/synchronizer.ts:51:88 - error TS2345: Argument of type 'number | null' is not assignable to parameter of type 'number'.
  Type 'null' is not assignable to type 'number'.

51     this.documents.set(document.uri, LSP.TextDocument.create(document.uri, languageId, document.version, content));

@jchavarri  pointed out that this is likely due to this change: https://github.com/Microsoft/vscode-languageserver-node/blob/ca1dc1e523b90348c8c4b8af6b740087d3e00ac0/types/src/main.ts#L662

i'm not sure why when calling npm link it picks the latest version of vscode-languageserver-types (3.8.2) instead of 3.7.1, that vscode-languageserver-protocol depends on.
The repro steps provided by @hcarty where:
git clone https://github.com/freebroccolo/ocaml-language-server
cd ocaml-language-server
yarn install
cd ..
git clone https://github.com/reasonml-editor/vscode-reasonml
cd vscode-reasonml
yarn install
cd ../ocaml-language-server
npm link

It seems the issue with npm link goes away by using yarn link instead. If @hcarty confirms this works, we could update the instructions to only use yarn.

For the record I only ever use yarn so I would recommend using that for the best results. Probably a good idea to mention that in the instructions.

@jchavarri It does work with yarn link - thanks!
I wondered why the instructions switched between yarn and npm but I'm not at all familiar with either tool so I thought there must be a reason to use both since the instructions did.


Feature Request: Option to use tabs instead of spaces when aligning property value pairs

Is it possible to add option to align property value pairs with tabs instead of spaces?
  // Align assignment statements in a hashtable or a DSC Configuration.
  "powershell.codeFormatting.alignPropertyValuePairs": true,

Currently when formatting hashtable the tabs are replaced with spaces:

Would love to have the option to use tabs for alignment instead:

Given
https://github.com/PowerShell/PSScriptAnalyzer/blob/128a9a225b4567ea828c18e9766519bf6afc9eb2/Rules/AlignAssignmentStatement.cs#L34
it looks like this rule needs to allow for configuration of the whitespace character it uses and adjust its alignment count appropriately

Yes, see #1055

Deduplicating to https://github.com/PowerShell/PSScriptAnalyzer/issues/1055


#101: fix scbe acceptance test with namespace and align PVC ymls with…

… new storageClassName param

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Updating youtube-dl throws 'requirement already up-to-date', but 'youtube-dl --version' returns out of date version

Checklist

[ ] I'm reporting a broken site support issue
[x ] I've verified that I'm running youtube-dl version 2020.05.03
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar bug reports including closed ones
[x ] I've read bugs section in FAQ

Verbose log

youtube-dl -v https://www.youtube.com/watch?v=4OJDeeyHoK4
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/watch?v=4OJDeeyHoK4']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.06.08
[debug] Python version 3.6.2 (CPython) - Darwin-19.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.2.2, ffprobe 4.2.2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 4OJDeeyHoK4: Downloading webpage
[youtube] 4OJDeeyHoK4: Downloading video info webpage
WARNING: Unable to extract video title
ERROR: This video is unavailable.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 2077, in _real_extract
    raise ExtractorError(error_message, expected=True)
youtube_dl.utils.ExtractorError: This video is unavailable.

Description

When trying to download Youtube videos, I'm getting this error:
WARNING: Unable to extract video title
ERROR: This video is unavailable.

I went to update Youtube-dl by executing:
sudo pip install --upgrade youtube_dl
Which throws:
WARNING: The directory '/Users/[my name]/Library/Caches/pip' or its parent directory is not owned or is not writable by the current user. The cache has been disabled. Check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Requirement already up-to-date: youtube_dl in ./.local/lib/python3.6/site-packages (2020.5.3)

So even though I'm told that 'Requirement up-to-date' for version 2020.5.3, running this command tells me my version is out of date:
youtube-dl --version
2019.06.08

What am I missing here?

Most likely you have multiple youtube-dl instances installed.

How do I remedy this? I am new to Python and package managers. Thank you.

@dstftw

@psrpsrpsrpsr Either uninstall youtube-dl from APT or from PIP.
Apt: sudo apt remove youtube-dl
You just need to remove one. Follow the instructions in the README for pip.


Tweak some icons

Hiya, I've recently been playing with this mod and it's awesome, now some items are actually useful and take up less space in the accessories bar :D, thanks for developing it!
The only pet peeve I have are some icons that don't really fit the aesthetic of the game :sweat_smile: So I tweaked them a bit to what I think would make them fit better.
If you don't like them, no problem, of course!

I like the changes. Finally a arrow texture that fits.


opencv-python-headless in requirements seems to create conflict?

Sometimes it has different version to opencv-python can cause issue. It is only used in top-down-rendering. Can we change this dependency to opencv-python?

I think it is because we have the requirement for opencv-python-headless somewhere like drivingforce, which causes strange conflict.


bug: Quote escaping termination failure

v0.3.0 of quamina, go 1.19
In building a toy, line oriented cli for quamina, I fed the library JSON data from https://www.gharchive.org/ and generated failures with the following message:
at line 1 col 15: truncated string. The following reductionist (but valid) JSON triggers the error
{ "message" : "docs: change \" -> \\" }

The quamina rule being processed is:
{"type": ["PublicEvent"]}

although I think the pattern is immaterial.
Adding a space after the last \ in the example JSON allows for successful processing.
I have a longer chunk of probably awful, newbie Go code that implements the cli binary, but I'll abstain from posting here unless requested.

Hey
This happens only in v0.3.0 since we introduce fast way to skip string values, I did a PR to fix this issue - https://github.com/timbray/quamina/pull/151


[enhancement]: BicepParam support for AzureResourceManagerTemplateDeployment

Task name
BicepParam file support
Describe your feature request here
The AzureResourceManagerTemplateDeployment task is supporting BICEP template files but does not support BicepParam files.
Currently it generates this error:   ##[error]Error: Ensure the Parameters file ( '/home/vsts/work/1/s/..../..../xxxxxxx.bicepparam' ) is valid. Task failed while parsing with following error: Unexpected token u in JSON at position 0
Cold you please support this.  BicepParam files are so much cleaner to author.

Hi @BertusV thanks for reporting! We are working on more prioritized issues at the moment, but will get back to this one soon.

Thanks.

Any updates on the support for .bicepparam files - it would be nice to NOT have to use the .json file in this task


Update KTO example to use better model and ChatML support

This PR tweaks the KTO example to use a small Qwen model that is better at following the chat template than the previous one. I've also added support for ChatML so users can align the base model directly.

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


Update UI for v0.2 demo

Resolves #39, by @fanwgwg.

After you manually fixed those conflicts, then do a git add . then git commit

@fanwgwg Take a look at the Travis logs and search for "Failed"
You'll see a bunch of things that failed in this branch such as:
guitests.AddCommandTest > add FAILED
    java.lang.IllegalStateException: Name not found: meet Alice Pauline
        at guitests.guihandles.TaskListPanelHandle.navigateToTask(TaskListPanelHandle.java:107)
        at guitests.AddCommandTest.assertAddSuccess(AddCommandTest.java:45)
        at guitests.AddCommandTest.add(AddCommandTest.java:34)

Could you investigate why it might be?

@fanwgwg Tests are failing for me locally to. This is the command that travis runs:
./gradlew clean headless allTests coverage coveralls -i
Can you try running that script locally?

Yay tests pass now :) :)

@fanwgwg The one test that failed in latest build was guitests.HelpWindowTest > openHelpWindow FAILED. Could you look at that?

@rachx @INCENDE This branch is looking good to review now

I just checked locally by running headlessly, the helpwindow passes test.

Coverage decreased (-2.9%) to 83.383% when pulling 490cff77d69d8f2f9e05f6c00c7d12169e82f25b on change-ui into a5ec10d051f477b8bea997ffaf9a0ffd27321173 on master.

This PR is all green now. @fanwgwg are you confident to master this? (Don't push merge yourself, I'll help you do it because of the potential conflict issues from other branches)

Coverage decreased (-2.2%) to 83.631% when pulling c00db2f2d216959168e4d505f5e06ea088636235 on change-ui into 58671b0c7128b07f30b115aa8b0c6967dbc2ea42 on master.

Passing! Yay!

@fanwgwg Confident to merge yet?

So the branch #79 doesn't matter now?

@fanwgwg Ideally yes: we should re-open 79, clean it up, merge it first, then update this PR then merge it. Do you wanna quickly get that done?

Since Rachael says that though the code in #79 works, it is quite messy, so she has rewrote some codes in another branch. Is it okay if we just merge this branch into master first?

Yeah we could just merge this branch into master now (it's due time, this PR is getting very very big), then can create a new branch to fix whatever messiness we want later then PR it. @rachx thoughts?

Just merge then. I think we can choose to keep the new files from my existing pr later?


Turbolinks not working on dropdown links, refreshes the page like normal

So, I have dropdown links like so:

        {{ __("Finance") }}

                {{ __("Created Invoices") }}

                    {{ Auth::user()->invoices->count() }}

and normal links like
link

The dropdown links do no work, they just refresh like normal, but the non-dropdown links load as I expect with this package. I have my javascript in the head of my site with the following code on the script:
 data-turbolinks-track="reload" data-turbolinks-eval="false"

Any ideas what could be causing it,or how to fix it?

What do you mean they refresh like normal? Refreshing the page is what you don't want right?
I think you just need to remove the href="#" and replace with role="button" so you still get the cursor icon while hovering