Course Homepage: Text vertical spacing (margins)

The items like platform, title and date have too much vertical space (margins )between them.
This needs to be reduced to improve the visual design.
Expected behavior

Actual behavior

Closed via #548

@byaruhaf @0xTim
This still isn't right. I'm not sure the design relates 100% to the code so I can't give you exact specs but the title shouldn't be that close to the lower metadata (it needs a few pixels of space). i think the space from the top of the title to the platform information should match the space from the bottom of the title to the Date etc. And if possible reduce the line height of the title text (Your First Kotlin Android App) as it seems to have too much space between each line.
Ping me if you need more information. it might be a case of making small tweaks and reviewing each screenshot.

@0xTim
Can someone pick this up?

Hi @luke-freeman
Added  Spacer() so the space from the top of the title to the platform information should match the space from the bottom of the title to the Date.
Please check the spacing below.

@byaruhaf
I think this will do! Close it :)

Thanks @luke-freeman sending the PR now.

Fixed


VSCT - Token replace on build support

In vsct files, it's common to use symbols like package GUID:

 <Symbols>
    <!-- This is out managed package guid. -->
    <GuidSymbol name="PackageGuidString" value="{A4F9D880-9492-4072-8BF3-2B5EEEDC9E68}" />

    <GuidSymbol name="guidDebugTargetHandlerPackage" value="{6e87cfad-6c05-4adf-9cd7-3b7943875b7c}">
      <IDSymbol name="DebugTargetMenuControllerGroup" value="0x1000" />
      <IDSymbol name="DebugTargetMenuControllerFooterGroup" value="0x2000" />

    </GuidSymbol>

Often these same values are declared as constants in the code somewhere:
 [Guid(VsPackage.PackageGuid)]
    public sealed class VsPackage : Package
    {

If the value of a symbol / guid changes, you are having to update / maintain the value in two places - i.e in the code, and manually in the vsct file.
It would be better if we could use tokens in t he vsct file that could be substituted on build.
One suggestion might be to use an attribute on the package, to expose tokens for substitution - i.e:

    [ProvideVsctToken("PackageGuid", VsPackage.PackageGuid)]
    [Guid(VsPackage.PackageGuid)]
    public sealed class VsPackage : Package
    {

Then in the vsct file:

 <GuidSymbol name="PackageGuidString" value="{%PackageGuid%}" />

On build, an msbuild task could discover the attributes and perform the substitution.

I'll create a nugget package to do this. Idea is:
NuGet package will add a targets file to your project.
Targets file will override VSCTCompileDependsOn, to inject its own msbuild target that runs prior to VSCT compilation
  <VSCTCompileDependsOn>$(VSCTCompileDependsOn);ReplaceVsctTokens</VSCTCompileDependsOn>

Targets file will include the ReplaceVsctTokens target, that will do the grunt work.
NuGet package will also add a reference to the project, to an assembly containing the custom attribute - [ProviceVsctToken]. I'll compile this lib for .NetStandard.
The ReplaceVsctTokens target, will reflect on the output assembly to find these attributes. For any it finds, it will perform a token substitution in any VSCT files.
That's the plan.

Thinking about it further, this syntax is probably not the best:
<GuidSymbol name="PackageGuidString" value="{%PackageGuid%}" />

because once the value has been replaced, the token is lost from the source so can't be replaced on future builds.
This is because I forgot the token replacement is happening on the actual source file.
Therefore, it would be better to match to a GuidSymbol's by it's name and then just replace its value.
i.e
 [ProvideVsctToken("Foo", VsPackage.PackageGuid)]
 public class VsPackage: Package {

 <GuidSymbol name="Foo" value="THIS WILL BE REPLACED" />

Code here: https://github.com/dazinator/VsctTokenReplace
I'll close this once I have verified the approach works.

The Extensibility Tools extension by Mads Kristensen approaches this problem the other way around. It views the VSCT file as the 'source of truth' and generates a C# file that contains constants that mirrors all of the guids and CommandIds from the VSCT.
To enable it (with the extension installed) you simply right-click the VSCT file and select Enable Auto-Sync
You can then use these values in your code.

That's an interesting approach.. Thanks for pointing me to it !
To be honest, I hit a snag with my way round, because I first need to compile the assembly before I can reflect on it - to update the vsct source file. This mean, if you change a GUID in the code, then do a build, the vsct source file will then be updated, but the assembly containing the original vsct source file has already been generated in the process - so the newly modified vsctf file won't have any impact until the next build.
I could use Roslyn instead of reflection, to grab the c# attribute values and update the VSCT source file pre-build. If I did this, the built assembly would have the changes. But as it stands that's a lot of work and I am happy with this known limitation for the time being.
To be honest I might just switch to the MAds approach :-)

Just tried out mads extension, and it works great.
The only problem I might encounter is if I had multiple VSCT files, sharing GUID's. For example if menus.vsct and foobar.vsct both used the same symbol guid "PackageGuid" - mads extension would generate two classes for that, and I am guessing the guid value would be duplicated. Meaning if I change the value in one VSCT file, I also have to remember to change it in the other?
To be honest I am not sure how common it is to have multiple VSCT files with shared symbols / guids. I am fairly new to VSIX development.

Yep Mads has got this - works great. Thanks


User settings independent date format

Information on date format is not complete. It seems universal independent format is also supported:
'yyyy-MM-ddTHH:mm:ss' e.g. 2019-12-29T17:05:45
or
'yyyy-MM-ddTHH:mm:ssZ' (UTC) e.g. 2019-12-29T17:05:45Z

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

ID: 710af5f2-8a31-e0d3-4f20-00c79dc54713
Version Independent ID: 1d4d9580-984c-9fee-861a-a02ce828d3db
Content: WIQL reference syntax - Azure Boards
Content Source: docs/boards/queries/wiql-syntax.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@BogdanIoanF Thank you for the feedback, assigning this to the author for review.


[#1] Setup project

Resolve Setup project #1

What happened 👀
Setup base application from scratch.
Insight 📝

Setup Next.js
Support TypeScript

Proof Of Work 📹

Run the application

Web application

@carryall @hoangmirs Could you please re-approve this PR again?
Because I merge into the wrong branch for the previous PR. 🥹


twitter auth Experiment 2b

See what is in the WSGIRequest on the server, to develop new redirect url

👍


update PartitionManager.java

fix wrong output in log when committed offset is too far behind latest offset

+1.

Created STORM-685 for this.

@iBuddha, thanks.  I merged this to master.
Post your name if you would like it in the contributors list along with your github id.
If you would like attribution in STORM-685 on JIRA, please post your JIRA user name too.

@d2r Sorry, my description  "behind lastest offset" is not corrent. It should be "currentOffset", "currentOffset" could be other offset besides latest. But "currentOffset" could still be confused.


Initial impl of state network bridge 1902

I'll go ahead and merge it, since I need it to re-base PR in the main branch & this PR doesn't change anything, just adds new content.


non-standard windows tray icon behavior

LMB click on tray icon is expected to show\hide main windows, it does nothing at the moment.
Closing main window is expected to reduce it to tray with an option to exit the application through RBM click on tray icon => exit.

I'll check, thanks
Not true, closing main window closes applications on everything but OS X.
In the future we will add an option to choose to minimize to tray on clicking X, but closing the app will be the default behavior.

Fixed in #327
Thanks for reporting!


Use requirements.txt for dependencies

See title! Nothing fancy!

Thanks. I should have added this when I switched to more than one requirement 😂


[CC-27365] add user_role_grant resource

Previously, the provider supported user role management but only in an authoritative way.  Using the user_role_grants resource would overwrite all roles for a user.  This caused churn for terraform users trying to manage user roles across multiple projects or via the console UI.  We now add a resource, user_role_grant which allows management of a single user role grant.  The existing user_role_grants resource is left to maintain the previous functionality for those that need it.
As part of this changes, I've also renamed the role_resource files which contain the user_role_grants resource after the full name of the role since that seems to be the convention.
Commit checklist

[x] Changelog
[x] Doc gen (make generate)
[x] Integration test(s)
[x] Acceptance test(s)
[x] Example(s)

@mikeCRL @codingconcepts @fantapop @pawalt @shannonbradshaw
Hi Mike, Rob-
I got word from @shannonbradshaw that you may be able to help with docs for this. A good first step would be to have a look at this PR to see if there are concerns and understand the docs implications.
As you probably know docs are critical for Terraform, as so much Terraform usage begins with examples

I'm waiting on the new SDK tag to be available before merging this down.  We can add documentation revisions to a later PR if any manifest.  Just let me know.


Run DICOM Adapter Docker as non-root user

Description
Fix vulnerability warning CIS-DI-0001 to create a non-root user.
Status
Ready
Types of changes

[x] Non-breaking change (fix or new feature that would not break existing functionality).
[ ] Breaking change (fix or new feature that would cause existing functionality to change).
[ ] New tests added to cover the changes.
[x] All tests passed locally by running ./src/run-tests-in-docker.sh.
[ ] Documentation comments included/updated.
[ ] User guide updated.
[ ] NGC publishing metadata updated.
[x] I have updated the changelog

Closing for now since this requires additional changes on other components as a whole.


Consider moving this project to jazzband

https://jazzband.co/
Even with semi-active maintainers, it might be helpful to have a more open contribution process as well as the Jazzband roadies to help with maintenance. One of my main concerns is the difference in concerns between different users: each user cares about different versions of Django, different existing features that are important to them, and different new features.
I'm not sure how the deployment situation would change. I'm guessing PyPI access would still be fairly restricted.
Thoughts welcome.

Found this issue when assessing the level-of-abandonment of several django model audit/history packages.  Second time I've seen the issue raised in as-many months.  Here's a rough framework for how I'd think about it:

Anything is better than abandoned.
You're delegating control along several dimensions

PRs: who can accept something into the repo
PyPi: who can push something to PyPi
Governance: the process for resolving disagreements about PRs (or PyPi)
Administration: the power to enforce governance by revoking rights

There are several levels of control (with offsetting time commitments):

DIY: Do it yourself (take the time)
DTA: Delegate the activity (take the time to vet contributors)
DTP: Delegate the power to delegate the activity (take the time to vet the vetters)
OPEN: No controls (no time)

You've identified two examples of many possible combinations.  I've added two more for illustration, but you could probably find other variants:

Option
Administration
Governance
PRs

Current State
DIY
DIY
DIY or DTA

Dedicated Org
DIY
DIY or DTA
DTA or DTP

YourLabs*
DTP
DTP
DTP

Jazzband
DTP
DTP (or OPEN)
OPEN

*I was vetted and given commit access to YourLabs after reporting a security issue in a hosted library.  I didn't even ask for access, but the vetting step provides a good contrast with Jazzband.

In any of these models you have to pick a level of control for PyPi, but it should probably parallel the governance model.  It doesn't make sense to have one set of rules for what goes in the repo and another for what is permitted on PyPi.


Add documentation of official Rust configuration files

There are a number of environment configuration repos in the rust-lang group:

rust-mode for Emacs
sublime-rust for Sublime Text
gedit-config for gedit
rust.vim for vim
nano-config for nano
zsh-config for zsh
kate-config for kate

These don't seem to be pointed to or documented anywhere, which is a shame. Seems like something that should be added to the site, although I am not sure where it should be put.

Given the existence of the IDEs page, I am going to go ahead and close this.


add m17n to setting screen, loading screen, planet detail

Hi!
I made some multilingual modifications to the UI in order to translate it into Japanese.
Perhaps people who have resources in other languages will also benefit.

Hi!
I made some multilingual modifications to the UI in order to translate it into Japanese.
Perhaps people who have resources in other languages will also benefit.

I'm more than happy to add this, but I don't know anything about localization, and don't want it ending up breaking every time we change something...I assume other mods add the translations? And if a translation is missing it will default to the string contents?

Hi!
Thank you for accepting my patch.
In the case of Japanese, the DSPJapanesePlugin project provides a function to apply the corresponding translation to unique strings.
Other languages have their own mods.
When the update changes the unique string , the person in charge will create a corresponding translation.
This is done in the same way for the DSP itself and for other mods.
If there is no corresponding translation, the pre-translated string will be displayed as it is.
Remember to  just add ".Translate()" at the end when embedding a unique string(to display) to the MOD, just like I added.

I'll try and remember :)


spider-gazelle.net website cannot access

spider-gazelle.net website cannot access , maybe  you should use the github pages, i think the   spide  is a sensitive word

https://spider-gazelle.net/
Works for me. It's using github pages and being served by cloud-front for speed and SSL.

Are you on a corporate network? Possibly behind a proxy?

Maybe my country block the domain.

China? Weird, I don't think there is anything sensitive on there..
You can view directly on github pages: https://spider-gazelle.github.io/guide/

Thanks ,it works


PHP inconsistencies and suggestions

Hello! I have been using your themes for while and I really like them but I noticed today that there are some inconsistencies when coding in PHP.

For classes I think all of them should be purple so that it matches other languages and the trait keyword should be green like the class, interface (etc...) keywords.
It would be nice if it was possible to make the $ muted for improved readability :)
EDIT: After getting used to it I actually like having the -> arrow and ... splat operator in the muted color and it is consistent with other languages :)
Here's the same code so you can test:
<?php

// Inconsistent color on class names

use ReflectionClass;
use ReflectionIntersectionType;

$foo = new ReflectionClass(SomeClass::class);

// Inconsistent colors on interfaces, classes, traits and enums

interface Example1{}
class Example2{}
class Example3 implements JsonSerializable, Stringable{}
class Example4{}
trait Example5{}
enum Example6{}

// Muted color on spread/splat (...) operator

$a = [1, 2, 3, ...[14, 5, 6]];

// Muted color on "instanceof" operator

if ($foo instanceof ReflectionIntersectionType) {
}

// Muted color on arrow

$foo->getConstant ();

// Muted color on ++ and -- operators

$n = 1;

$n++;
$n--;

Another think I noticed is that the string concatenation operator also becomes muted.

// The dot becomes muted but it should probably be yellow

$string = 'hello' . ' ' . 'world';

// The .= operator also becomes muted

$string .= '!';

thank you for taking the time to make such a complete issue! I agree with you, it will be fixed for the next release :)

Great! I did some more tweaks if you're interested in adding them too :)
[
    {
        "scope": "support.class.php",
        "settings": {
            "foreground": "#AB9DF2"
        }
    },
    {
        "scope": "entity.name.type.class.php",
        "settings": {
            "foreground": "#AB9DF2"
        }
    },
    {
        "scope": "entity.other.inherited-class.php",
        "settings": {
            "foreground": "#AB9DF2"
        }
    },
    {
        "scope": "entity.other.alias.php",
        "settings": {
            "foreground": "#AB9DF2"
        }
    },
    {
        "scope": "storage.type.trait.php",
        "settings": {
            "foreground": "#78E8C6"
        }
    },
    {
        "scope": "keyword.operator.type.php",
        "settings": {
            "foreground": "#FFD866"
        }
    },
    {
        "scope": "keyword.operator.string.php",
        "settings": {
            "foreground": "#FFD866"
        }
    },
    {
        "scope": "keyword.operator.ternary.php",
        "settings": {
            "foreground": "#FFD866"
        }
    },
    {
        "scope": "keyword.operator.increment-decrement.php",
        "settings": {
            "foreground": "#FFD866"
        }
    },
    {
        "scope": "punctuation.separator.decimal.period.php",
        "settings": {
            "foreground": "#FC9867"
        }
    },
    {
        "scope": "punctuation.definition.variable.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "variable.language.this.php punctuation.definition.variable.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "keyword.operator.key.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "keyword.operator.class.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "keyword.operator.nullable-type.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "keyword.operator.spread.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "keyword.operator.variadic.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    },
    {
        "scope": "punctuation.separator.delimiter.php",
        "settings": {
            "foreground": "#C1BCB6"
        }
    }
]

Thanks !
I've fixed the scope of entity.other.alias.php, but for punctuation.separator.delimiter.php, in other languages they don't have colours, so I prefer to leave it as is :)


Add support for @{entity} to the Intellisense language server.

Part of improved intellisense

This work is no longer applicable as it is handled under other feature work for R12.

This work is no longer applicable as it is handled under other feature work for R12.


How to set UITableViewCellSelectionStyle to none?

I have some labels where I don't want the user to be able to tap on the row and get a highlight.
What I've tried:
let row = FormRowDescriptor(tag: "no_image", type: .label, title: "No Image Attached")
row.configuration.cell.appearance = [ "selectionStyle" : UITableViewCellSelectionStyle.none as AnyObject ]

The app crashes when I try to set the selectionStyle.
I can disable selection on the whole table by calling self.tableView.allowsSelection = false but I want buttons in the form to be able to be clicked.

Try setting row configuration value as follows:
row.configuration.cell.appearance = [ "selectionStyle" : UITableViewCellSelectionStyle.none.rawValue]


Why was BusinessTime::Config.reset made private?

One of the things our app needs to do is to reload the BusinessTime configuration to calculate differences for different companies.
We've had to resort to calling BusinessTime::Config.send(:reset) in order to reset the configuration. I'm not sure why it's required, you'd think just over-writing the work_hours config setting would do the trick, but it does not.
Here's a test script that proves the case (Ruby 1.9.3-p448, BusinessTime 0.7.2). The second business_days.after call should pass, but it doesn't until we call reset.
require 'business_time'

business_hours = {
  mon: ['09:00', '17:00'],
  tue: ['09:00', '17:00'],
  wed: ['09:00', '17:00'],
  thu: ['09:00', '17:00'],
  fri: ['09:00', '17:00'],
}

open_24_hours = {
  mon: ['00:00', '00:00'],
  tue: ['00:00', '00:00'],
  wed: ['00:00', '00:00'],
  thu: ['00:00', '00:00'],
  fri: ['00:00', '00:00'],
  sat: ['00:00', '00:00'],
  sun: ['00:00', '00:00'],
}

# April 1, 2014 is a Tuesday
start_date = Time.parse '1 April 2014 12:00pm'

BusinessTime::Config.work_hours = open_24_hours
if 15.business_days.after(start_date) == Time.parse('16 April 2014 12:00pm')
  puts '.'
else
  puts 'F'
end

BusinessTime::Config.work_hours = business_hours
if 15.business_days.after(start_date) == Time.parse('22 April 2014 12:00pm')
  puts '.'
else
  puts 'F'
end

BusinessTime::Config.send(:reset)
BusinessTime::Config.work_hours = business_hours
if 15.business_days.after(start_date) == Time.parse('22 April 2014 12:00pm')
  puts '.'
else
  puts 'F'
end

Hi David,
Is there any improvement since this last post on the multi-tennant support? If not, I'm willing to write some of the code, as long as it's within my technical skill, provided a small guidance.
Regards,
Javier

@jlanatta You might want to check out a similar gem we released recently called biz. It supports multiple schedule configurations in the same thread.
I'm more than happy to help if you have any questions. :)

@craiglittle: thanks for the link! It covers all the functionality I need.


ViewLocator can throw null reference exceptions

Note: for support questions, please ask on StackOverflow: https://stackoverflow.com/questions/tagged/reactiveui . This repository's issues are reserved for feature requests and bug reports.
Do you want to request a feature or report a bug?
Bug
What is the current behavior?
DefaultViewLocator.AttemptViewResolutionFor can throw a null reference exception if ToggleViewModelType returns null.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
What is the expected behavior?
Doesn't throw an exception (either, ToggleViewModelType doesn't return null, or there's a null check around it in ResolveView).
What is the motivation / use case for changing the behavior?
Not throwing exceptions when they shouldn't be thrown.
Which versions of ReactiveUI, and which platform / OS are affected by this issue? Did this work in previous versions of ReativeUI? Please also test with the latest stable and snapshot (http://docs.reactiveui.net/en/contributing/snapshot/index.html) versions.
7.2
Other information (e.g. stacktraces, related issues, suggestions how to fix)

I am currently able to reproduce this. Going to have to roll back to 7.1 due to a significant change in behavior (it was not throwing the null reference before 7.2).


No longer works

realtor.com changed their site... so this no longer works

@alloylab yeah, this happened in the first week i was working on this once released, few days later it stopped working and i noticed they keep changing the layout, class names ids and positioning of the elements.


Fieldset inside form group collapses margin on legend

Margin's on legend's collapse when inside fieldset
GOV.UK Elements (app) contains legend { overflow: hidden;} however this doesn't appear to be in GOV.UK Elements itself.
Therefore examples in the App render differently than in the the kit / when you include elements.

After a bit of sleuthing we've worked out that this is due to a difference between the documentation on govuk-elements.herokuapp.com and what's currently in the prototype kit – the prototype kit isn't up to date with this change.


portal-bridge: Do not require --epoch-accumulalor-path cli flag for beacon bridge

Require --epoch-accumulator-path flag only if the history network is enabled in portal-bridge. The bridge uses the --network flag to enable the desired networks.
Blocked by https://github.com/ethereum/trin/pull/822.

Hey @ogenev, could you please assign me to this issue


Step 3.2 is absolutely vague -

[Enter feedback here]
Maybe it is repetition of the previous step.

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

ID: aab7c843-301a-0f60-bb52-dbdaad4f18f0
Version Independent ID: aab7c843-301a-0f60-bb52-dbdaad4f18f0
Content: Tutorial: Use dataset code components
Content Source: power-pages-docs/configure/tutorial-pcf-code.md
Service: power-pages
Sub-service: ****
GitHub Login: @GitanjaliSingh33msft
Microsoft Alias: gisingh

Thank you for your feedback and for calling that out. It was a repetition of that step, and that particular documentation issue has been rectified.


prisma-query-engine is not closing when the program is finished

prisma-query-engine is not closing when the program is exited. and it is taking space in memory even though our dart program is done. You have to manually kill the process in task manager... is there a way to close the engine

https://prisma.pub/getting-started/setup.htm
This is a known problem. At present, I haven't found a better solution. Maybe the CABI engine will not have this problem in future versions.

Hello,
This resolves the issue for me :
main() {   AppLifecycleListener(     onExitRequested: () async {       PrismaClient client = // your Prisma Client ;       await client.$disconnect();       return AppExitResponse.exit;     },   );   runApp(const YourApp()); }


Omit auth header

Looks good once the additional request properties we talked about are serialized

I guess I'll also need to rev the version and publish


formContext in HTML web resources

How can we access the formContext in HTML web resources. GetGlobalContext only returns Xrm.Page.context, not the full form context.

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

ID: 6a6b587f-31d1-2615-6d76-767210c452fd
Version Independent ID: 96054a5b-0b47-f7da-8291-68a3a9366992
Content: Client API Xrm object for Dynamics 365 Customer Engagement
Content Source: ce/developer/clientapi/clientapi-xrm.md
Service: crm-online
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

@wintg-zns Full formContext isn't available in HTML web resources. You can use parent.Xrm.Page as the form context.
Closing this issue.


Debug Build Issue - Opencv + Windows

When building a debug build on windows platform, for example for face_mesh, build fails with several unresolved symbol errors
demo_run_graph_main.lib(demo_run_graph_main.obj) : error LNK2019: unresolved external symbol "void __cdecl cv::imshow(class cv::String const &,class cv::debug_build_guard::_InputArray const &)" (?imshow@cv@@YAXAEBVString@1@AEBV_InputArray@debug_build_guard@1@@Z) referenced in function "class absl::lts_2020_09_23::Status __cdecl RunMPPGraph(void)" (?RunMPPGraph@@YA?AVStatus@lts_2020_09_23@absl@@XZ)
bazel-out\x64_windows-dbg\bin\mediapipe\examples\desktop\face_mesh\face_mesh_cpu.exe : fatal error LNK1120: 14 unresolved externals
Target //mediapipe/examples/desktop/face_mesh:face_mesh_cpu failed to build

Conmand line for this was
bazel build -c dbg --define MEDIAPIPE_DISABLE_GPU=1 --action_env PYTHON_BIN_PATH="C://Users//username//AppData//Local//Programs//Python//Python39//python.exe" mediapipe/examples/desktop/face_mesh:face_mesh_cpu

Build is successful for -c opt.
Reason for this is, in mediapipe/third_party/opencv_windows.BUILD file, the dll/lib files for opencv are not w/ debug symbols. Modifying this file to include following solves the problem -
srcs = [
        "x64/vc15/lib/opencv_world" + OPENCV_VERSION + ".lib",
	"x64/vc15/lib/opencv_world" + OPENCV_VERSION + "d"+".lib",
        "x64/vc15/bin/opencv_world" + OPENCV_VERSION + ".dll",
	"x64/vc15/bin/opencv_world" + OPENCV_VERSION + "d"+".dll",
    ],

where the d suffix points to the required files.
It wasn't too clear from the output error that this was required, hence creating this issue for future reference.

Hi @rahulchaphalkar, Could you please explain the steps followed to reproduce the above issue. Thanks!

I followed the steps for installation on windows listed HERE
Downloaded opencv 3.4.10 from https://opencv.org/releases/ as recommended, modified WORKSPACE file to ensure it points to the opencv location.
Hello world ran as expected.
Built Mediapipe face_mesh model for cpu according to instructions HERE ]
(replaced hand_tracking with appropriate face_mesh parameters)
It builds and runs successfully with the flag -c opt, so installation and build is successful. However, since i needed debug symbols, i built it with bazel's debug flag i.e. -c dbg, that's when i encountered the above errors.
They are resolved when I add the d suffix opencv library files to the mediapipe/third_party/opencv_windows.BUILD file.
I'm assuming the d suffix files, like opencv_world3410d.dll and opencv_world3410d.pdb are required for additional symbols for debug build.

Does https://github.com/google/mediapipe/issues/1718#issuecomment-794353452 help?

@jiuqiant ah i believe it is the same issue, which already has a PR. I'll close this one. Thanks.


Fix attribute error for send/recv and ray put/get in base_module

This PR fixes the issue where param doesn't have attribute "data" for send/recv and ray put/get in base_module.py. The issue is introduced by PR #92 .

LGTM


Postgrel graph issue

Issue Description
I experience an issue when I first made a graph of phish results and now logging and dashboard graphs are messed and the results I expected are normal function.
Reproduction Steps

run a graph
start King Phisher

Environment Details

Host OS: Ubuntu 18.04.2 LTS
King Phisher Version: 1.14.0-beta(rev: e07132639c7e
Error Details / Stack Trace
Error Type: king_phisher.errors.KingPhisherGraphQLQueryError
Error Details: KingPhisherGraphQLQueryError()
Error UID: e34fd0d0-d083-4d06-b356-df70941799cc
RPC Error: N/A (Not an RPC error)
King Phisher Version: 1.14.0-beta (rev: e07132639c7e)
Platform Version: Linux: Ubuntu 18.04 bionic
Python Version: 3.6.7
Gtk Version: 3.22.30
Timezone: America/New_York
Thread Information:
=> MainThread (alive=True daemon=False)
Thread-7 (alive=True daemon=True)
Thread-6 (alive=True daemon=False)
Thread-11 (alive=True daemon=True)
Thread-12 (alive=True daemon=True)
Stack Trace:
Traceback (most recent call last):
File "/home/saurus/king-phisher/king_phisher/client/application.py", line 488, in do_server_connected
campaign = self.get_graphql_campaign() if campaign_id else None
File "/home/saurus/king-phisher/king_phisher/client/application.py", line 371, in get_graphql_campaign
campaign = self.rpc.graphql_find_file('get_campaign.graphql', id=campaign_id)
File "/home/saurus/king-phisher/king_phisher/client/client_rpc.py", line 241, in graphql_find_file
return self.graphql_file(path, query_vars=query_vars)
File "/home/saurus/king-phisher/king_phisher/client/client_rpc.py", line 224, in graphql_file
return self.graphql(query, query_vars=query_vars)
File "/home/saurus/king-phisher/king_phisher/client/client_rpc.py", line 205, in graphql
query_vars=query_vars
king_phisher.errors.KingPhisherGraphQLQueryError
GraphQL Exception Information:
GraphQL Errors:
table
table
table
GraphQL Query:
retrieve a campaign by it's ID
query getCampaign($id: String!) {
db {
campaign(id: $id) {
id
created
name
description
expiration
maxCredentials
  credentialRegexUsername
  credentialRegexPassword
  credentialRegexMfaToken

  campaignType {
    id
    name
  }
  company {
    id
    name
  }
  user {
    id
    name
  }
}

}
}

Reproduced, bug is related to the recent upgrade of SQLAlchemy from v1.2.6 to 1.3.3.

Apparently the sqlalchemy.relationships.orm.RelationshipProperty.table attribute has been deprecated in favor of the sqlalchemy.relationships.orm.RelationshipProperty.target attribute. When table was removed, this broke king_phisher.server.graphql.types.database.sa_get_relationship which caused a cascading failure through the AuthorizationMiddleware up to GraphQL.
I'll have this fixed today so we can continue the upgrade to SQLAlchemy v1.3.3.

Fixed in commit 437dbae2ea04d864580f02acd818012661c20dcd.


nixos/nextcloud: Update recommended nginx settings

Motivation for this change
Update default configuration to match the latest recommendations from Nextcloud: https://docs.nextcloud.com/server/15/admin_manual/installation/nginx.html
To get rid of all the warnings the imagick PHP module is enabled too.
Also update to PHP 7.3 as this is officially supported by Nextcloud 15: https://docs.nextcloud.com/server/15/admin_manual/installation/system_requirements.html
Things done

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

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

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

Yes! Which is great for reproducible systems!

@GrahamcOfBorg test nextcloud.basic nextcloud.with-mysql-and-memcached nextcloud.with-postgresql-and-redis


ZIP-211: Validate Disabling Addition of New Value to the Sprout Value Pool

Motivation
We need to add the disabled sprout pool consensus rule. Resolves https://github.com/ZcashFoundation/zebra/issues/1564 when merged.
Specifications
https://zips.z.cash/zip-0211
https://zips.z.cash/protocol/protocol.pdf#joinsplitdesc
Solution
Implement the rule, add a test. The test can be improved.
Review
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

I updated the title of this PR so it says "Disabling Addition of New Value to the Sprout Value Pool".
It is still possible to remove value from the sprout value pool.


Allow to create junction point instead of symlink for win users.

yarn-audit-fix --symlink=junction

:tada: This issue has been resolved in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v1.6.0

Your semantic-release bot :package::rocket:


Abhijith Ganesh

Please Review my website: https://abhijithganesh.github.io
and review my github readme thank you.

@AbhijithGanesh , in this repo, you need to add your profile readme img to data.json for others to vote on. I think you should open up an issue in https://github.com/EddieHubCommunity/support/issues for others to review your github profile. Thanks.

Thanks for updating

Let us know if you need any help. 🙂

**@abhijithGanesh

you push your work.

Get screenshot of profile.

Then you upload the image.

And github changes your image in to a link, which will be like
Click here for link

Add data.json with your data. You need to fork the repository, implement your changes, and submit a pull request 🙂🎉**


add reset password feature

[x] Fixes issue #26:"Add option to recover password "
[x] Other checks

Reason for pull request:
"New Feature : reset password"
Description of changes or additions:
"Gmail id has to be configured to sent reset mail"

@selvamanikannan I have reviewed your PR for sure you have made a lot of efforts. But I want you to make some changes:

Please remove the Forget Password option from navbar add in it the login page beside SignUp option.
Please remove the remove password options from the navbar if you want this to be available for users good to have it in the profile update page just ask for old password and new password to reset it.
I am getting error when I tried the forget password option. You can have a look over the error

@selvamanikannan I have reviewed your PR for sure you have made a lot of efforts. But I want you to make some changes:

Please remove the Forget Password option from navbar add in it the login page beside SignUp option.
Please remove the remove password options from the navbar if you want this to be available for users good to have it in the profile update page just ask for old password and new password to reset it.
I am getting error when I tried the forget password option. You can have a look over the error

pls add ur mail id and password in seetings.py
EMAIL_HOST_USER = 'your  gmail id'
EMAIL_HOST_PASSWORD = 'your  gmail pass'
line no 137,138
the allow low application at here https://myaccount.google.com/u/0/lesssecureapps?pageId=none
ensure ur gmail is not having two step authentication
then run  the app it will work.
i want to update you that i have tested properly
waiting for your reply

@selvamanikannan I have reviewed your PR for sure you have made a lot of efforts. But I want you to make some changes:

Please remove the Forget Password option from navbar add in it the login page beside SignUp option.
Please remove the remove password options from the navbar if you want this to be available for users good to have it in the profile update page just ask for old password and new password to reset it.
I am getting error when I tried the forget password option. You can have a look over the error

pls add ur mail id and password in seetings.py
EMAIL_HOST_USER = 'your gmail id'
EMAIL_HOST_PASSWORD = 'your gmail pass'
line no 137,138
the allow low application at here https://myaccount.google.com/u/0/lesssecureapps?pageId=none
ensure ur gmail is not having two step authentication
then run the app it will work.
i want to update you that i have tested properly
waiting for your reply

Still getting same error. I will look over it. Till that can you please update the PR with other changes.

Good to me. Thanks :)


Create examples and tests for run_list.go and run_list_item.go

in examples update bin, cmd and Inspec

The go tests are adequate. This code doesn't really hit the api.


Cannot serve AdminJS in production environment.

Describe the bug
Cannot serve AdminJS in production with NGINX.
Installed libraries and their versions
"adminjs": "6.2.0",
"@adminjs/express": "5.0.0",
"@adminjs/nestjs": "5.0.1",
"@adminjs/typeorm": "4.0.0",
Expected behavior
It should download the global.bundle.js and app.bundle.js but currently, it's not accessible.
Screenshots
Production

Local

Desktop (please complete the following information if relevant):

OS: macOS

Browser brave
Version 1.43.89

Are you sure your nginx server is not blocking AdminJS routes? For example global bundle is served from:
/admin/frontend/assets/global.bundle.js

I'm pretty sure I'm not blocking the /admin/ routes.
in this NGINX config, you can see that, I'm reverse proxying all of the naked routes to nestjs backend.

And in below photo, the design-system and components bundle are served from the same /admin/frontend/assets/ folder, but they're served properly.

Another possibility is that this path here is being resolved incorrectly in your production environment for some reason:
https://github.com/SoftwareBrothers/adminjs/blob/master/src/backend/utils/router/router.ts#L51
Have you considered using @adminjs/bundler to pre-bundle browser files?


Add right-click menu management

The menu is too long
I want to be able to categorize by extension

This is on our roadmap pls see https://github.com/microsoft/vscode/wiki/Roadmap#workbench


1.5.55

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@cderv I got us past a few failures to a point where the build completes but fails to test with:
ERROR: Error executing '/home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/tools/x86_64/typst': No such file or directory (os error 2)

Stack trace:
    at execProcess (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:4717:15)
    at checkVersions (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:101118:37)
    at eventLoopTick (ext:core/01_core.js:153:7)
    at async check (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:101066:13)
    at async Command.actionHandler (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:101389:5)
    at async Command.execute (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:8017:13)
    at async Command.parseCommand (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:7907:20)
    at async quarto (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:118219:9)
    at async file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:118239:9
    at async mainRunner (file:///home/conda/feedstock_root/build_artifacts/quarto_1722356633693/_test_env_placehold/bin/quarto.js:118123:9)

I only had a little bit of time to focus on this so I'm going to set it down for now.

@conda-forge-admin, please rerender

@mfisher87 this is quite cryptic error to me.
First is the long path with placehold multiple times expected ? Just to be sure.
Then, what is quarto_1722356633693/_test_env_(...truncated...) ? Is this where Quarto is installed ?
I don't know how this works for other tooling bundled, but checkVersions() is the place in Quarto's code where each tool is run with --version to see if we get what is expected.
For typst, the binary used is
export function typstBinaryPath() {
  return Deno.env.get("QUARTO_TYPST") ||
    architectureToolsPath("typst");
}

First is the long path with placehold multiple times expected ? Just to be sure.

:laughing: yeah, that's a normal Conda thing. I forgot why, I learned at some point and only kept the knowledge that it's normal :)

Then, what is quarto_1722356633693/test_env(...truncated...) ? Is this where Quarto is installed ?

This is where Quarto is installed during the build process. It's one of the final steps of installing the package and then running the items under the test: key in meta.yaml.

My understanding of conda packaging here is that our downloaded tools are not used but conda-forge equivalent are, and I see you modified this PR to use typst from conda-forge. So maybe just setting QUARTO_TYPST is missing ?

I think this is a perfect description of what's going on. You've accurately described what I'm trying to do and filled in exactly the gaps that were missing in my mind! Thank you. I should remember this detail by now! :laughing: :bow:

You've accurately described what I'm trying to do and filled in exactly the gaps that were missing in my mind!

😄 Good team work ! 🤝

Getting closer! @cderv any chance you have Windows hardware or experience that could help identify the problem? May be a bit until I can set up a VM.

The slash is the thing that jumps out the most, but I followed the pattern in bld.bat.

I don't think this is it. Usually it is handled ok. Like this, nothing jumped out so I tried on my windows machine to understand how typst conda package was working and I found the problem.
conda install -c conda-forge typst=0.11.0

Nice catch, thanks @cderv! Do you think there can be an upstream release that upgrades the typst dependency to 0.11.1? That way we won't need to change the recipe once to work around this and then change it again once Quarto upgrades typst.

Shoot! Thanks for considering it. Let's try this :)

@cderv if things look good to you, feel free to merge! I'll probably click merge sometime tomorrow if I don't hear anything :)

You did good by merging. I was off today - start of summer break for me. Thanks for the update on this !
I am glad we manage to build it !
Just did not have time to warn you that we did a new patch release today 😬 - sorry.
We do one usually first of each month, so that you know.
Thanks again !

Happy to help, but looks like things didn't go quite perfectly :grimacing:
After releasing, looks like #53 also impacts 1.5. When I built it locally, the file was present, but somehow it's not present in the automated build. I'm really confused.

I'll think about how to debug this 🤔
Maybe something is wrong with the copy to what is bundled ?


DBeaver v22.1.1 // Export Tasks does not respect ${table}

System information:

Operating system (distribution) and version: Windows 10 Enterprise
DBeaver version: 22.1.1 x64 Portable
Additional extensions: Microsoft Office Integration & Darkest Theme

Connection specification:

Database name and version: DB2
Driver name: DB2 JCC (Type 4) <IP_ADDRESS> FP000
Do you use tunnels or proxies (SSH, SOCKS, etc)? Yes

Describe the problem you're observing:
I have an export task with multiple queries
As export type I have Microsoft Excel (Microsoft Office Integration)
As an output, I have specified ${tables}
On execution, I expect to get get 4 different xlsx files.
However, I get a single file with the name "export.xlsx", consisting on the export of the last query.
Steps to reproduce, if exist:
Reverted to v22.1 and was able to to 4 different xlsx files named each one based on ${table}

Hello @valsamis-d,
Perhaps you have the highlighted option enabled?

@ShadelessFox no. It's disabled. I've always had this disabled as my export consists of 4 different queries and I want to have district output files for each one.

It seems to be broken by #15695.
Thanks for the bug report. Unfortunately, I'm unable to think of any workaround.

It seems to be broken by #15695.
Thanks for the bug report. Unfortunately, I'm unable to think of any workaround.

Thanks for confirming @ShadelessFox !
Until this is resolved, I will stay in v22.1 :) Thanks for following-up!

Hello @ShadelessFox @serge-rider  @emironovDB
I downloaded 22.1.2 and the issue has been resolved, at least with getting different export files. I see that it was worked around in #16990.  Are they any specific rules in order to help the engine to resolve table name? My current flows are based on some naming conventions, which assume that the first comment in each query has a specific name and date. For example:
-- ENTITY_A_2022_07_10
Query for Entity A for 2022_07_10
-- ENTITY_B_2022_07_10
Query for Entity B for 2022_07_10
etc.
Thank you
Dimitrios

DBeaver attempts to resolve the name of the table from a query.
It works fine for simple queries but fails for others because it expects to find just a single table name.

Thanks for the response @ShadelessFox !
Is it difficult to resolve the table name based on the comment that is prior to the query, such as "-- ENTITY_A_2022_07_10"?
This will some flexibility when the output files are passed into so sort of further automated processing.
Thank you
Dimitrios

Thank you so much @ShadelessFox !


Option to add custom inputAccessoryView

`
let nextButtonItem = UIBarButtonItem(title: NSLocalizedString("NEXT",
comment: ""),
style: .done,
target: self,
action: #selector(pinCodeNextAction))
    let toolbar = UIToolbar()
    toolbar.items = [UIBarButtonItem(barButtonSystemItem: .flexibleSpace, target: nil, action: nil),
                     nextButtonItem]
    toolbar.barStyle = .default
    self.codeTextField.inputAccessoryView = toolbar`

The above code throws error saying Cannot assign to property: 'inputAccessoryView' is a get-only property.
Please resolve this.
.

Thanks for reporting the issue. I'll fix this in the nearest days.

@todesengelS I've added inputAccessoryView support, please check the new version. You have 2 options for setting inputAccessoryView:

I've added sample toolbar as an accessory view to Example project, please look at ViewController.swift (viewDidLoad method). One thing I wanted to note — you should call sizeToFit() on your UIToolbar instance so it will have correct bounds.
I marked inputAccessoryView property with @IBOutlet attribute, so it's possible to set it in .xib or .storyboard file


OpenVPN client export custom config text area not showing multiple lines

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[yes] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[yes] I have searched the existing issues and I'm convinced that mine is new.
Describe the bug
In OpenVPN > Client Export there is a text field to add a custom config that will be returned in
the output file.
When I add 3 commands on 3 different lines to this field, all 3 commands are shown in one line separated with spaces, when there should be seen as 3 consecutive lines of text.
To Reproduce
Steps to reproduce the behavior:

Go to 'VPN > OpenVPN > Client Export'
Click on 'Custom config text field'
Write some line of text and press Enter to continue writing on the next line
The problem is that the new line is shown as a space so the following line is shown just next to the previous one, so all entered commands are shown on one long line

Expected behavior
The various entered lines of custom config should be seen one on multiple lines of text.
Screenshots
This is how I see 3 command:

fragment 1250 mssfix 1250 tun-mtu 1500

This is how it should be seen:

fragment 1250
mssfix 1250
tun-mtu 1500

OPNsense 19.7.6-amd64
FreeBSD 11.2-RELEASE-p14-HBSD
OpenSSL 1.0.2t 10 Sep 2019
Running on VMWare ESXi 6.7

I can't reproduce this on my end (using Chrome):

The generated config file shows multiple lines as well.
Maybe a browser issue?

You are right, I'm using Firefox. I've just tested with Chrome and it shows fine.
Exported config shows multiple lines on both browsers.
May be it can be fixed by using the same textbox attributes that are being used in Advanced Config of the OpenVPN configuration.

Its' not the input itself, for some reason the content seems to be cleansed in firefox.

ok, looks like this https://bugzilla.mozilla.org/show_bug.cgi?id=1137650 should be fixed with https://github.com/opnsense/core/commit/fde54d978e82bdb1c4460715f9f1f4a5e1025a47 (opnsense-patch fde54d9 on a console)

@AdSchellevis ... tested and now it's ok... thank you


Add Redis implementation for CacheHandler

Implemented here: https://github.com/eneajaho/ngx-isr/commit/d4a11ceeb3223bd5fe74477bdeead926ce55c564


Configure this repo to cache image layers between GHA runs for the push-rdev hook

Currently, docker images are being built from scratch each time in GHA (at least for push-rdev.yml)
@edulop91 recommends incorporating this: https://github.com/chanzuckerberg/github-actions/tree/main/.github/actions/docker-build-push

@atarashansky Can you confirm whether this is still needed?


Refactor KeystoreUtils

Summary
Related issue https://github.com/wso2/product-is/issues/22117

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12537876363

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12537876363
Status: success


Integrate Loki with Traefik ingress

Issue

This PR fixes #116 and #117 (since they are related)
Solution

This PR integrates ingress_per_unit lib with Loki
Context

Loki send over relation data its endpoint (internal cluster IP) so client charms can set up it, but this is not useful for clients that are outside the cluster. That why we are integrating ingress_per_unit lib from Traefik charm
Testing Instructions

Deploy Grafana agent charm
Deploy Loki charm
Deploy Traefik charm
Add a relation between Grafana agent and Loki
Check in /etc/agen/agent.yaml that Loki's endpoint is present.
Relate Loki with Traefik
Check in /etc/agen/agent.yaml that Traefik based endpoint is present.

Release Notes

Integration test would be great. The one from Prometheus could probably be mostly reused.

Integration test would be great. The one from Prometheus could probably be mostly reused.

Absolutely, I was reading just that one


Supporting reveal.js 4.x

I tried to install with reveal.js 4.1
I cloned revealjs to the sphinx project root and used the following custom config in conf.py but it did not produce valuable output:
revealjs_static_path = ['revealjs/dist']
revealjs_css_files = ['reveal.css']
revealjs_script_files = ['reveal.js']
revealjs_style_theme = 'theme/black.css'

Is current recommendation to clone the 3.9 stable branch of revealjs?

In currently version, implement for sorting css files is incorrect order.
If use this conf, this write link tag of theme style prior to link of revealjs core css, browser does not render correctly.
Workaround.
Set theme css file path into revealjs_css_files.
revealjs_static_path = ['revealjs/dist']
revealjs_css_files = ['reveal.css', 'theme/black.css']
revealjs_script_files = ['reveal.js']

Is current recommendation to clone the 3.9 stable branch of revealjs?

Yes. I reccomend to use revealjs 3.9 yet.
reveal.js 4.x has changes  of behavior of using plugins from version 3.x .
sphinx-revealjs cannot support new writing syntax for using plugins, then it does not work plugins.
(I am development for this issues in feature/reveakhs-multi branch )

On current version (1.0.1) , use Reveal.js 4.x to build presentation.
If you use version 0.x and want to change 1.x, please see migration document.


[Bug]: Progress Indicator page

Prerequisites

[X] I have searched the backlog for duplicate or closed feature requests

Your test device
iPhone 10
OS Version
iOS 15.3.1
App version
0.2.0
Describe the issue
Position of the progress indicator is left aligned and should be centre aligned

Expected Behavior
align progress indicator to the centre

this was tested ok


[Bug]: % not passing file path

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Run :FlutterRun -t %
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS:
Flutter version: 2
Is flutter in $PATH: yes
neovim version: 0.6

Additional context
You can pass a % to most commands in neovim and it will expand to the file path, but flutterun takes it as a literal argument.
Generally I'd offer to fix this, but I don't what could possibly cause this.

Hi @joshpetit, so this isn't native behaviour for most user defined commands, the only commands that automatically expand the percent symbol are native vim commands. I don't actually think this is expected behaviour since it's not normal for user-defined commands to just do this. Usually a plug-in author has to do the work of expanding special symbols to emulate this native functionality. I actually have another plugin where I've had to do this and it can be quite involved.
I think in order to get this functionality I recommend mapping the percent symbol or %%, for example in your config to expand to the file path. You can use cnoremap %% expand('%') or something like that.

That's beautiful, thank you

Just as in FYI doing cnoremap wouldn't work for this, I'm just doing  % to get the path from % buffer.


Compiler just dies on include Comparable(T)

struct Int
  struct Null < Int
    include Comparable(T)

    def to_s(io)
      io << ""
    end
  end
end

renders:
/usr/local/bin/crystal: line 6: 64408 Segmentation fault: 11  "$INSTALL_DIR/embedded/bin/crystal" "$@"

works for me. Make sure that you are not using buggy llvm version. For reference, see #540.

Yes, must probably be the buggy llvm version, it doesn't crashes for me either.

@alex-fedorov Could you confirm if this was because of the buggy llvm version?

No response in months, closing this for now. Please reopen if you can still reproduce this issue.

Yeah, this was already resolved, sorry for not closing..


We need a Scene server that holds all information about scenes

@pravusjif are you working on this?

Can you provide more details? Probably we have it in the catalyst, but it is hard to know based on the title only

@pravusjif are you working on this?

No, it's the first time I read about this, kernel normaly fetches the scenes data hitting some lambdas in the catalysts i think, mayve a more centralized way could be faster and more optimized.

catalysts are fast enough for our use cases, they act as central servers, we just have many of them available. please describe the need and what needs to be improved, that would yield better results

I think this is related to where we are going to store the unpublished scenes (projects) that were going to appear in the builder in-world panel. Going to update the description to add more details.


Enable dict round trip test for json

Codecov Report

Merging #875 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #875      +/-   ##
==========================================
+ Coverage   63.58%   63.62%   +0.04%
==========================================
  Files          88       88
  Lines       14775    14775
  Branches     3285     3285
==========================================
+ Hits         9394     9401       +7
+ Misses       3226     3216      -10
- Partials     2155     2158       +3

Impacted Files
Coverage Δ

vm/src/stdlib/json.rs
63.93% <0%> (+5.73%)
: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 9503449...e6f08cf. Read the comment docs.

Thanks for enabling this!

This doesn't modify any Rust code...

So it doesn't; I could've sworn the build was failing. Weird.


As a player, I would like to save my game state and be able to load my game state for a better playing experience

Implement a save functionality
The saving state should be kept in local storage persistently.
Provide an ability to load a saved game state.
Implement a menu to saving/loading.
There should be options to determine the saving location.
Do not use any object serialization mechanism or do not use any binary format to save the data. Come up with a model to represent the game state as a text file and serialize the current state into a such text file. Also, come up with a way to verify the content of the file (such as a CRC check) to prevent players to modify the game state file content manually.

I want to work on this issue.
Mert Can Gönen
181101038

Sana gönderdiğim daveti kabul etmen gerekiyor. Epostanı kontrol et.
Bu arada projeyi bilgisayarında çalıştırmayı denediğini ve bir sorun yaşamadığını varsayıyorum.


Feature/upgrade v8 5.7.492.77

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2977/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2989/

diff --git a/lib/ApplicationFeatures/V8PlatformFeature.cpp b/lib/ApplicationFeatures/V8PlatformFeature.cpp
index be00b7448a..f48ee843e1 100644
--- a/lib/ApplicationFeatures/V8PlatformFeature.cpp
+++ b/lib/ApplicationFeatures/V8PlatformFeature.cpp
@@ -188,15 +188,15 @@ void V8PlatformFeature::unprepare() {
v8::Isolate* V8PlatformFeature::createIsolate() {
v8::Isolate::CreateParams createParams;
createParams.array_buffer_allocator = _allocator.get();

createParams.oom_error_callback = oomCallback;
createParams.fatal_error_callback = fatalCallback;
+/// TODO  createParams.oom_error_callback = oomCallback;
+///   createParams.fatal_error_callback = fatalCallback;

Is something missing here?

diff --git a/lib/Basics/FileUtils.cpp b/lib/Basics/FileUtils.cpp
index bf9bcfb395..9938f5fdc6 100644
--- a/lib/Basics/FileUtils.cpp
+++ b/lib/Basics/FileUtils.cpp
@@ -324,9 +324,9 @@ bool copyDirectoryRecursive(std::string const& source, std::string const& target
std::string rcs;
std::string flt = source + "\*";

UnicodeString f(flt.c_str());

icu::UnicodeString f(flt.c_str());

handle = _wfindfirst(f.getTerminatedBuffer(), &oneItem);

handle = _wfindfirst((wchar_t*)f.getTerminatedBuffer(), &oneItem);
if (handle == -1) {
error = "directory " + source + " not found";
@@ -335,7 +335,7 @@ bool copyDirectoryRecursive(std::string const& source, std::string const& target
do {
rcs.clear();

UnicodeString d((wchar_t*)oneItem.name, static_cast<int32_t>(wcslen(oneItem.name)));

icu::UnicodeString d((wchar_t*)oneItem.name, static_cast<int32_t>(wcslen(oneItem.name)));
d.toUTF8Stringstd::string(rcs);
fn = (char*)rcs.c_str();
#else
@@ -418,8 +418,8 @@ std::vectorstd::string listFiles(std::string const& directory) {
std::string rcs;

std::string filter = directory + "\*";

UnicodeString f(filter.c_str());
handle = _wfindfirst(f.getTerminatedBuffer(), &oneItem);

icu::UnicodeString f(filter.c_str());
handle = _wfindfirst((wchar_t*)f.getTerminatedBuffer(), &oneItem);

c style cast

diff --git a/lib/V8/v8-conv.cpp b/lib/V8/v8-conv.cpp
index 1aebf2496f..6871747c31 100644
--- a/lib/V8/v8-conv.cpp
+++ b/lib/V8/v8-conv.cpp
/// @brief converts an V8 object to a boolean
-bool TRI_ObjectToBoolean(v8::Handlev8::Value const value) {
+bool TRI_ObjectToBoolean(v8::Isolate* isolate, v8::Handlev8::Value const value) {
if (value->IsBoolean()) {

return value->ToBoolean()->Value();

return value->IsTrue();
}

really?

+++ b/lib/V8/v8-globals.h
+#define TRI_IGETC isolate->GetCurrentContext()
+
+#define TRI_GET_INT32(VAL) VAL->Int32Value(TRI_IGETC).FromMaybe(0)
+
+#define TRI_GET_UINT32(VAL) VAL->Uint32Value(TRI_IGETC).FromMaybe(0)
+
+#define TRI_GET_DOUBLE(VAL) VAL->NumberValue(TRI_IGETC).FromMaybe(0.0)
+
+#define TRI_GET_STRING(VAL) \
+  VAL->ToString(TRI_IGETC).FromMaybe(v8::Local<v8::String>())
+
+#define TRI_GET_OBJECT(VAL) \
+  VAL->ToObject(TRI_IGETC).FromMaybe(v8::Local<v8::Object>())
+
+#define TRI_OBJECT_HAS_PROPERTY(OBJECT, PROPERTYSTRING) \
+  OBJECT->Has(TRI_IGETC, TRI_V8_ASCII_STRING(isolate, PROPERTYSTRING)).FromMaybe(false)
+
+#define TRI_OBJECT_HAS_V8_PROPERTY(OBJECT, PROPERTYSTRING) \
+  OBJECT->Has(TRI_IGETC, PROPERTYSTRING).FromMaybe(false)
+
+#define TRI_OBJECT_HAS_OWN_V8_PROPERTY(OBJECT, PROPERTYSTRING) \
+  OBJECT->HasOwnProperty(TRI_IGETC, PROPERTYSTRING).FromMaybe(false)
+
+#define TRI_OBJECT_HAS_OWN_PROPERTY(OBJECT, PROPERTYSTRING)                       \
+  OBJECT->HasOwnProperty(TRI_IGETC, TRI_V8_ASCII_STRING(isolate, PROPERTYSTRING)) \
+      .FromMaybe(false)
+
+#define TRI_OBJECT_GET_PROPERTY(OBJECT, PROPERTYSTRING)                \
+  OBJECT->Get(TRI_IGETC, TRI_V8_ASCII_STRING(isolate, PROPERTYSTRING)) \
+      .FromMaybe(v8::Local<v8::Value>())
+
+#define TRI_OBJECT_GET_V8_PROPERTY(OBJECT, PROPERTY) \
+  OBJECT->Get(TRI_IGETC, PROPERTY).FromMaybe(v8::Local<v8::Value>())
+
+#define TRI_OBJECT_DELETE_PROPERTY(OBJECT, PROPERTYSTRING) \
+  OBJECT->Delete(TRI_IGETC, TRI_V8_ASCII_STRING(isolate, PROPERTYSTRING)).FromMaybe(false)
+
+#define TRI_OBJECT_DELETE_V8_PROPERTY(OBJECT, PROPERTY) \
+  OBJECT->Delete(TRI_IGETC, PROPERTY).FromMaybe(false)
+
+#define TRI_OBJECT_TO_V8_STRING(OBJECT) \
+  OBJECT->ToString(TRI_IGETC).FromMaybe(v8::Local<v8::String>())
+
+#define TRI_TO_OBJECT(VAL) \
+  VAL->ToObject(TRI_IGETC).FromMaybe(v8::Local<v8::Object>())
+

I do not like the new macros. I think passing something like isolate->getCurrentContext() into a function is not too bad and makes much more obvious what is going on.

js/client/authentication/user-access-right-task-update-view-arangosearch-spec.js
js/client/authentication/user-access-right-task-update-view-arangosearch-spec_1.js
js/client/authentication/user-access-right-task-update-view-arangosearch-spec_2.js

Is this work in progress?

Some functionality is missing and I do not like the marcos.

@ObiWahn Which macros in particular?

Some functionality is missing and I do not like the marcos.

@ObiWahn Which macros in particular?

he is talking about the new macros in lib/V8/v8-globals.h that keep object property access a one-liner. The new V8 api has even more neccity for boiler plate code for simple things, that I decided to throw more macros at it.

Ah, I see. The way I was looking through the diffs, they didn't appear as new code for some reason. I would also be somewhat against introducing new macros when we can avoid it. We already have enough to be confused about without hiding what's happening.

Blue: http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr-linux/4307/


Question - Is it a successor to AlpacaTag?

Hi. With AlpacaTag being last updated 9 months ago, can this solution be considered as its successor? I'm asking because I've tried to install the AlpacaTag. It worked for the basic labelling, butthe Active Learning Server is always off and I see you don't respond to issue raised by other people iin AlpacaTag. Therefore the question if this is the new AlpacaTag.
Thanks

in some ways yes. in other ways no. The main thrust of this framework will be incorporating explanation capture and training of downstream models using these explanations. I'm currently working on refactoring all model training code, after doing a big refactor of the front-end code over the summer. @JimGong
Active Learning is not a priority of this system,

Thank you @Rahul-Khanna though I was hoping it would be. I'm on a lookout for an open source labelling/annotation tool with Active Learning to increase labelling productivity. Something like Prodigy, but free. Thanks for answering anyway!

for sure, but hopefully with this tool you won't need to label much to begin with. The goal is to use additional information (an explanation of why you labeled something the way you did) to train machines. The next release (projected 1/14/21) will allow you to train a relation extraction model with this new methodology. you can check out this paper to learn more: https://openreview.net/pdf?id=rJlUt0EYwS


Improve cluster edit workflow for clusters created via RKE2 template

on v2.6.0-rc1
Repro steps:

create a cluster via rke2 templates
go to cluster management -> edit config for the newly created cluster

Issue:

(see screenshot above) for now, we are just giving this warning when a cluster is deployed via RKE2 template.
Solutions:

Option 1 : add a feature flag or some sort of toggle for admin to set, like this:

Allow users to edit clusters created via RKE2 template (changes that users make in the cluster will be overridden with the next change to your RKE2 template)

with this disabled, the edit button will not be shown for clusters created via RKE2 templates (we should be able to detect this, since we can add the warning for rke2 cluster templates)
with it enabled, the work flow will remain the same. We should add to the warning … please contact the rancher admin to make changes to the template for this cluster

If we were to do something about this issue it would be to detect the situation, find the corresponding app that deployed it and bring you to the upgrade for that instead so that you could do what you "meant" (change the config of how this cluster was deployed) instead of what you "said" (edit this specific cluster resource).
The banner about it being managed is a generic thing that applies to, and is shown for, all resources created from a helm install; it has nothing to do with clusters specifically.  Any resource that was created by helm release will normally be overwritten if you use helm again to update it.  If you still want to edit that resource directly you can, that is a valid thing to do.  We do not normally have the UI pretend to enforce that you not do things that are still valid to do in the backend.

detect the situation, find the corresponding app that deployed it and bring you to the upgrade for that instead so that you could do what you "meant" (change the config of how this cluster was deployed) instead of what you "said" (edit this specific cluster resource).
I agree that this would work, however for RKE2 templates only admin has access to update the template(s)/chart that deployed a cluster.  So I was trying to think of a solution that would prevent unwanted updates from happening while also clarifying to users how to update the template vs. individual cluster updates.
My idea for this was, if user is admin we could expose a new field in the ... hamburger menu on each RKE2 template-deployed cluster, something like Edit RKE2 Template (screenshot below shows the menu I'm referring to)

We do not normally have the UI pretend to enforce that you not do things that are still valid to do in the backend
I understand that editing a cluster created via rke2 templates is valid for a user, however (please correct me if I'm wrong) I don't understand how this would be useful in terms of the RKE2 template feature. These are the counters I can think of:

a user installs a cluster from a template and later decides to update it (say, the k8s version).  later, the admin updates the template, say from 1 node to 3.  Now, the user has. a cluster with nodes on an older version than expected, if the template is even able to upgrade due to the user's previous edits
say I am admin, and I want to use cluster templates to ensure some level of uniformity in the clusters that are deployed. I can limit/hardcode variables through the template. However, after the user goes through the template workflow, they can treat the cluster however they want.
This isn't quite the same comparison, but we do have some valid features (i.e. the legacy flag) that we can toggle through the UI

Again, I see that the current workflow is valid in terms of what can be done with clusters/templates but I think the intended uses  of RKE2 templates might differ with that (especially considering people who have used  rke1 templates and are used to the idea of locking in certain values)

Closing based on age. There have been changes around this page since 2.6.0 so if there are still questions, file a new ticket and link back.


Add documentation

There is little documentation, and where this is, it's incomplete. Need to complete the documentation before publishing to the Powershell Gallery.

It would be good to externalise the help (rather than using comment-based help). This would help with localisation support if it's ever wanted/needed.
The PlatyPS project would certainly help with this. However, it would require a build process on the CI server (or a separate project completely) to ensure the help is updated with each commit. Note: this will probably be needed in the future to digitally sign the module's files when publishing to the gallery anyway..
It would also make it possible to support updateable help...

In addition I think it would be good to update the Readme to include any and all blogs on Lability as they in 1 way or another do provide a form of documentation

I started play with

AppVeyor - for continues integration
Pester - for testing help structure
for my OffCATcmd repository .

Currently for any commit help is parsed. In the next step I'll add invoking PlatyPS for build/update markdown files and pushing back to GitHub.

Ping: @lazywinadmin
François-Xavier I would like add a code based on your blog post Using Pester to test your Comment Based Help to the Lability repository. Do you agree ?

@it-praktyk No problem Wojciech. Thanks for your comments btw, will check this week-end.

Thank you @lazywinadmin !
@virtualengine , I put the first version of Pester tests for help in my Lability fork - as Tests/Help.Tests.ps1 .
I also generated the first version of markdown files generated by platyPS v. 0.5.0 module - in the Docs subfolder.
Discussion is needed

In what directory should I put Help.Tests.ps1 file before I will create pull request to your repository?
If any help structure - specially for a .NOTES section - was discussed by your team/contributors?
who should update existing functions? - some help parts are missed. You can see the tests results in the file Docs-Errors\Pester_Results.html

Thanks @it-praktyk! The rendered Markdown looks pretty impressive :+1:
How do you see PlatyPS being used here? Is the intention to produce Markdown files for online help/documentation? Or is the ultimate goal to move the existing help to Markdown, so that we can generate the MAML help files (that could be localized)? If it's the latter I wouldn't put too much effort into the comment based help tests..

Thanks @iainbrighton :-)

The rendered Markdown looks pretty impressive
It's not my merit. Thanks @vors !

How do you see PlatyPS being used here?
I don't know better solution to produce markdown helps what can be translated to MAML next.

As my "sandbox" I use my OffCATcmd project. My goal is prepare solution to regenerate help files via AppVeyor if Pester tests pass.
What do you think about opening (automatically via GitHub API) issue when Pester tests for help structure fail?
BTW, I tried parse Pester results to text using Format-Pester but currently this doesn't work as expected for the text output. I opened issue . Your PScribo is used. Could you look ?

I just added pull request #117 . I hope that failed Pester test will allow improve quality of help for the Lability module.

Lability is quire rough at the first place, but then it turned out to be a masterpiece. Thank you!


Add example of template usage

Could be a nice way to also showcase list rendering.

Thanks for raising this issue!
Do you have an example of what you have in mind? Not sure if it's better fitted for gtk4.cr or libadwaita.cr instead

I fixed some bugs in gi-crystal, so now it's possible to use Gtk::ListView with Gio::ListModel 🎉
I added an example: https://github.com/hugopl/gtk4.cr/pull/62
it needs gi-crystal >=0.20.0 to not crash.

Awesome!


[Feature] support set_var hint for DML

Why I'm doing:

The SET_VAR hint is only viable for query, but not DML

What I'm doing:

Support set_var for insert/update/delete sql

Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-3.2

https://github.com/StarRocks/starrocks/issues/32024


New Game Survivor Added

PR Description 📜
I have Added A playable HTML + CSS + JavaScript remake of a space-based "shoot-'em-up" arcade game for Atari / Commodore 64 from 1982, including a level editor and design tool.
Fixes #2592

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @AbhishekPSingh07 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit I have already Done it

Thank you  @AbhishekPSingh07 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Add "performer" to DiagnosticReportDisplay

This change will help our customers compare DiagnosticReport results in Medplum to a 3rd party lab

Coverage: 93.927% (+0.001%) from 93.925% when pulling f8f0e7de9946244c956c64b8e2466497715e0be7 on rahul-dr-display-performer into 3312e5e83964e01c76c0d3b216a51462ebfc20e0 on main.


Add Christian as code owner

Pull Request Test Coverage Report for Build 190357802

0 of 0   changed or added relevant lines in 0 files are covered.
190 unchanged lines in 4 files lost coverage.
Overall coverage increased (+2.4%) to 34.485%

Files with Coverage Reduction
New Missed Lines
%

pkg/zips/env.go
3
0%

pkg/zips/types.go
25
54.76%

pkg/zips/client.go
75
3.96%

pkg/zips/template.go
87
26.32%

Totals

Change from base Build 189432167:
2.4%

Covered Lines:
978

Relevant Lines:
2836

💛  - Coveralls


preview to source jump not working after inserting new text in the source file

Adding a new theorem to the document by adding #theorem[] will reproduce this issue
typstpreviewerror.zip


SMTP authentication with trusted IP address (without username and password)

So, we have some application that only allow SMTP IP to be set and we cannot specify username and password there. In postfix, there is an option to add our trusted IP's in mynetworks in main.cf file but I am not getting this option in Postal.
Can we achieve this?

That certainly seems like a reasonable request :)

So can we expect something in near future? :)

+1 for this. Configserver CSF firewall is like this,  you can not set username / password only smtp IP

This was added.

@adamcooke
is it possible to use SMTP-IP with credentials (That means that credentials works only when email receive on postal request from particular IP)
Thanks

No


mistake in wiki page 'Building'

In section 'How to get the MacVim source code': https://github.com/macvim-dev/macvim/wiki/Building#how-to-get-the-macvim-source-code
$ git clone git://github.com/macvim-dev/macvim.git

should be
$ git clone git@github.com:macvim-dev/macvim.git

or some error may occurs:

The unauthenticated git protocol on port 9418 is no longer supported.
Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.

Hi @mzlogin
Nice catch! Could you make a PR to fix this issue?

Hi @eirnym
I'd like to make a PR, but as I know, GitHub doesn't support pull requests for the wiki repository, only the main repository.
ref https://stackoverflow.com/questions/10642928/how-to-pull-request-a-wiki-page-on-github and https://github.com/isaacs/github/issues/333

It's a shame that GitHub still not allow to fix wikis via PRs

I agree 😢
So would you please fix it?

Thank you, fixed it.


Make RuboCop runner composable and capture any stdout message from it

Motivation

RuboCop is still too chatty and breaks the protocol by printing to stdout despite our best attempts. Today I was having problems with the formatting on Tapioca and realized it was because of the:
An error occurred while Layout/BlockEndNewline cop was inspecting /Users/ufuk/src/github.com/Shopify/tapioca/lib/tapioca/static/rbs_converter.rb:166:8.
To see the complete backtrace run rubocop -d.

problem (which is in pre-1.34 versions of RuboCop).
I am not exactly sure why this was causing a problem, since that specific message should be going to stderr, but the LSP communication was still broken because of it.
Regardless, I realized that we should start capturing the stdout from any RuboCop runs.
Implementation

In order to do this in an encapsulated and uniform manner, I realized that we were missing a layer of abstraction between RuboCop::Runner and our request classes that depend on it. I then realized that the problem was stemming from the fact that we've been using inheritance rather than composition for that part of the codebase.
So, I did a refactor to extract a composable RuboCopRunner that both the RuboCopDiagnosticsRunner and RuboCopFormattingRunner are able to use. That resulted in a much better and easier to read implementation for both of those classes.
Afterwards, I implemented a capture_output method in the RubocopRunner which I made all invocations to RuboCop::Runner#run wrapped in. This method does the following:

Captures (and throws away) $stdout
Captures (and throws away) $stderr
Turns off Ruby $VERBOSE flag so that Kernel#warn messages are never printed.

This should make sure that we never print anything that we don't control to stdout/err anymore.
Finally, I think showing any errors that were encountered (and handled) during RuboCop processing is helpful, so I made RuboCopRunner explicitly print uniq errors to stderr.
Automated Tests

I haven't added any but it might be nice to maybe add one.
Manual Tests

Not sure how, but if you can recreate the conditions of https://github.com/rubocop/rubocop/pull/10878 on a repo that is using RuboCop 1.33 or earlier, then it is enough to trigger the broken behaviour and to see that this PR fixes it.

TIL about the STDERR difference. To add a unit test for this, you can register a handler in the queue_test that prints something and verify that it didn't actually print anything.

@st0012 Yeah, I know about the difference between STDERR/STDOUT and $stderr/$stdout. However, my goal is to catch legitimate uses of puts in general, which end up going to $stdout.
In any case, I changed my approach and implementation to limit the capture to only RuboCop and to capture only $stdout, which seems to working well for me. FYI @vinistock.

Ok, it ended up being much more tricky and I had to change the $VERBOSE global as well. That flag controls if Kernel#warn is going to have an output or not and I propose that we set $VERBOSE = nil for the ruby-lsp executable overall in any case. I don't think we want random warning messages being sent to the Output tab.
WDYT?

I propose that we set $VERBOSE = nil for the ruby-lsp executable overall in any case

I think that makes sense 👍
Regarding the implementation, I was a bit worried about swapping $stderr/out will affect the other thread's output as we can run 2 requests at a time now. But right now it's not going to happen because language_server-protocol-ruby uses STDOUT instead of $stdout.
However, it's still quite dangerous because if one day the STDOUT thing is changed, we can start experiencing weird race condition issues.


Add alternative route when using heading

When requesting a route with heading, we could think about calculating a second route without heading. If this route is different, we can add it as an alternative route.

Was added in #18


NumberBox Font size not change the text inside the editor

Describe the bug
I cannot change the font size of the NumberBox
Steps to reproduce the bug
Steps to reproduce the behavior:

Add NumberBox to Page
Set the attribute FontSize to any size.

Expected behavior
The size of the UI and the number text will be change.
Actual behavior
Only the UI box changed and the number text nothing change.
Screenshots

<muxc:NumberBox x:Name="WaterLevelBox" Header="Water Level:"                 SmallChange="1"                 LargeChange="5"                 Minimum="0"                 FontSize="72"                 SpinButtonPlacementMode="Inline"/>
Version Info
UWP target version: 10.0.18362
Microsoft.UI.Xaml version: 2.4.2 (from nuget)
Gallery version (found on Settings page):
<IP_ADDRESS>
Windows 10 version:

[ ] Insider Build (xxxxx)
[ ] November 2019 Update (18363)
[x] May 2019 Update (18362)
[ ] October 2018 Update (17763)
[ ] April 2018 Update (17134)
[ ] Fall Creators Update (16299)
[ ] Creators Update (15063)
[ ] Anniversary Update (14393)

Device form factor:

[x] Desktop
[ ] Mobile
[ ] Xbox
[ ] Surface Hub
[ ] IoT

Additional context

Seems like we are just missing a template binding. Should be an easy issue to fix.

@MikeHillberg Does FontSize, FontFamily, etc. flow down the tree or do we need to explicitly template bind them?

Yes, generally all of the text properties should inherit down the element tree.


Different performance when model is reloaded

Hi
I am training alexnet on my own image data. approximately 10,000 images and 3 classes. I ran the training procedure and saved the models at the end of every epoch. train and test log files show accuracies of > 90% . But when I load the model and test it on the same training and testing data I get very poor results.
img = trainHook(base_dir .. file)
preds = model:forward( img:cuda() )
_, pred_sorted = preds:sort( true )
predictions[ file ] = pred_sorted[1]
TrainHook takes care of cropping and mean,std normalization.
Do you have any idea why this might happen? I can provide more information if it is not clear

In your test script, before testing, make sure you call model:evaluate() which puts all the batchnorm modules into testing mode and switches off dropout.

Forgot to mention, I do that right after loading the model.

Forgot to mention that, I call model:evaluate() right after loading the model

the test labels, do you make sure they are in the same order as when trained?
for example, while training they could be: {cat, dog, horse}, but in your test scripts they might have been repermuted as {dog, cat, horse}

Yes, They are in the same order. I also tried getting predictions only on images from a particular class (base_dir points to a particular class), even this gives poor results

@dileepc I really am not sure. reproducing the results with a loaded model hasn't been a problem for us. It might be something simple like image normalization being different in your test script.

I think I found whats going wrong here. After image is loaded and passed through trainHook the tensor needs to be reshaped from (3,224,224) to (1,3,224,224) , because the model:forward() usually takes 128 images of a batch at once as a (128,3,224,224) tensor input.

Thanks for pointing this out. Fixed the readme via commit https://github.com/soumith/imagenet-multiGPU.torch/commit/710bace7b0d0342b87a4d9ad590c0bec3d6de8ad


Calculate a better MSBuild tools/extensions path

Need to fix the FileUtilities class and extract logic to a new class. These are the scenarios I want to consider.
Tools / Extensions Paths

When running in MSBuild.exe:

Set the Tools Path to current directory (expanded, not using ".").
Assume we're running in <Root>\MSBuild\15.0\Bin and choose <Root>\MSBuild for Extensions Path (ToolsPath\..\..).

Running as a dependency and that folder is "complete":

Use the reference folder location as the Tools Path.
Assume we're running in <Root>\MSBuild\15.0\Bin and choose <Root>\MSBuild for Extension (ToolsPath\..\..).

Running as a dependency that is not "complete":

Discard that folder and try to fallback to a well known location.
For Dev15 (VisualStudioVersion >= 15'), this would be %VSINSTALLDIR%\MSBuild\15.0\Bin for Tools Path and  %VSINSTALLDIR%\MSBuild for Extensions Path.

Note: By "complete" I mean contains enough of MSBuild to do a build. I will probably check for msbuild.exe and msbuild.exe.config as a minimum bar. This will cover the case that you create an app that references Microsoft.Build.*.dll and it gets copied to your output. Your output should not be considered a tools path unless it contains those files.
Issues:
If %VSINSTALLDIR% doesn't exist

We could throw an Exception and tell the user to run from a Visual Studio command prompt.? We no longer have the ability to determine a valid location.
Use global Visual Studio API to determine a location for Visual Studio (there may be multiple) and pick one of them.

Running Open Source Build

We don't want to pick up the Visual Studio Tools Path or Extensions Path in this scenario. Note that Extensions Path would be broken by the above logic because our build layout is like bin\x86\Windows_NT\Debug\. The Extensions Path would point to x86 or something similarly wrong.
This isn't really solvable without producing a build layout in the right format and pulling in all our run-time dependencies (what the bootstrap proj does now).

This is related to #702 which is a temporary workaround.

Related to: #114


demo and blog post gone?

Was very excited to see a modern working example but sadly it appears the blog post and demo are no longer up (I'm getting a 502 error)?

Thanks for notifying me.
It should be back online. Feel free to close the issue if it works for you.

Yes it's back up. Many thanks for the quick response.

Hi, is the demo offline?


Dinosaur game is not opening [GSSoC'23]

Describe the bug
Dinosaur game is not opening after clicking on Start Game
Expected behavior
It should have taken me (user) to the page where the game starts.
Instead it is showing error.
Screenshots

Would like to fix this. Please assign me this issue under GSSoC'23

can u assign me this isssue


Update template_shelly_monitoring.yaml

Updated version of template

README seems to be somehow autogenerated, What is the correct way to update this (I haven't seen or might have missed info about READMEs in guideline)? Overview seems to be copied multiple times in this README and tables that are autogenerated might be out of date as well, but I don't think I will go trough them and manually fix them all, I don't have that much free time.

Hello @UniverseDevel,
The readme is indeed automatically generated, and will be updated on successful pull request when CI/CD will run.

Great then how to update just the Overview section of it?

Hello @UniverseDevel,
OverView Section is copied from the template description field. If you wish, you can use MarkDown formatting.

Fantastic, I will later update my templates description. Thank you


Publish nightly builds

Each commit to main branch (or in fact any branch, for easier development) could publish the library to some repo, not Maven Central.

to make this work with jitpack do this:

add jitpack.yml to the repo

jitpack.yml
install:
  - echo "Running a custom install command"
  - ./gradlew clean :library:publishToMavenLocal -xtest -xsignMavenJavaPublication

see https://github.com/NikkyAI/github-actions-kotlin-dsl/blob/main/jitpack.yml

use this in your script

@file:Repository("https://jitpack.io")
@file:DependsOn("com.github.krzema12:github-actions-kotlin-dsl:main-SNAPSHOT")

make sure that the jitpack.yml file exists on whatever branch/commit you request from jitpack, without it will fail to build the artifacts

Left: mention using nightly builds in the docs.

Thanks @NikkyAI for the great idea with JitPack!


Movies Genre Classification

Deep Learning Simplified Repository (Proposing new issue)
🔴 Project Title : Movies Classification
🔴 Aim : Classify different genres of movies using deep learning approaches.
🔴 Dataset : https://www.kaggle.com/datasets/akshaypawar7/millions-of-movies
🔴 Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

Assign me


fix: Fix NaN and empty objects in getNonDefaultConfiguration

Is there an issue number associated with this PR? Should tests be added for this use case?

No issue, but I’ll add some tests

Is there an issue number associated with this PR? Should tests be added for this use case?

Done!


Add printer whitelist

Based on existing blacklist code regions.
If the whitelist is empty then it is disabled.
The blacklist overrides the whitelist.

The filterBlacklistPrinters function was duplicated in cups.go and
winspool.go, so instead of creating two more almost identical
filterWhitelistPrinters functions I factored out the common code in
lib/printer.go.

Ignore the appveyor stuff, I left it half-configured by mistake.

Looks good, thanks!
Just needs unit tests for lib.FilterWhitelistPrinters and lib.FilterBlacklistPrinters.

Ok, I haven't looked into testing yet. I'll see what I can do.

I've committed simple tests for those functions. Hopefully reflect.DeepEqual() isn't verboten. ; - )

DeepEqual is no problem. It's useful for tests!
It's looking good, thanks for the tests.
I was thinking about the whitelist, I would like to avoid using a zero-length list to disable whitelist processing. I haven't figured out an optimal way to do this though. Maybe a separate whitelistEnabled option? But that has issues too.

My current idea is this:
In JSON, printer_whitelist and printer_blacklist could be one of:

a non-empty list
the string "all"
empty (omitted)

I am not sure if it is easy to do this in Go without a lot of hassle.
Thoughts?

I got the idea from reading this:
https://developers.google.com/protocol-buffers/docs/reference/go-generated#oneof

I think I see where you're coming from - to have one symmetric configuration concept for both lists, right?
You're kind of going through the steps I considered (last comment in #261 - "magic entry or separate option") before I concluded that they're not necessary and it's already simpler and symmetric without them. I agree the "all" string (vs magic entry in list) is probably an implementation hassle though, and for minimal gain if we divide the functionality up:

Share all / block none
Share some / block some
Share none / block all

All users want is 2 (and maybe 1, though not on Windows) so config for 2 would hopefully be as simple as possible - such as simply deciding which list to use. I only see connector management software wanting 3. Allowing entries be printers that don't exist is kind to users (in the case of printer deletion or renaming) and support for case 3 falls out of that naturally. In fact it allows putting effectively a comment in either list, such as "This connector is being managed by foo".
Anyway, I've written more than I intended. ; - ) From a user's point of view, and seeing how it works in the connector management software that I'm writing, I'm happy with what I checked in.
Let me know what you decide. : - )

Re "I would like to avoid using a zero-length list to disable whitelist processing".
Before all my assumptions and reasoning above, I should have been more polite and asked: Why? ; - )

Ok. I am happy with this as is. If needed, we can later add a special directive that means "blacklist all". For now, you can whitelist a non-existent printer (like "") which should work.
The wiki is out of date for configuration, but I filed #269 for that.

Thanks. This should also close #261.
Cheers,
Nick.


Fix table names inside database

Won't be fixed for now. And is not a really big problem. Just for maintainability.


Coverage artifacts in js code when stringifying a function at runtime.

I learned about how Istanbul measures coverage in a really cool way thanks to my current problem. My current task is to write a javascript function and then return the function as a string. My unit test attempts to eval the string function and fails because of the following error:

ReferenceError: __cov_HnbYpbOoQM4MaLYAY_l6Ig is not defined

When I take a look at the value of my function string, I see this:
function (browser,tabTitle){
    __cov_HnbYpbOoQM4MaLYAY_l6Ig.f['3']++;
    __cov_HnbYpbOoQM4MaLYAY_l6Ig.s['10']++;
    var currentTabId=browser.getCurrentTabId();
    __cov_HnbYpbOoQM4MaLYAY_l6Ig.s['11']++;
    browser.getTabIds().forEach(id=>{
        __cov_HnbYpbOoQM4MaLYAY_l6Ig.s['12']++;
        browser.switchTab(id);
        __cov_HnbYpbOoQM4MaLYAY_l6Ig.s['13']++;
        if(browser.getTitle().indexOf(tabTitle)>-1){
            __cov_HnbYpbOoQM4MaLYAY_l6Ig.b['2'][0]++;
            __cov_HnbYpbOoQM4MaLYAY_l6Ig.s['14']++;
            return false;
        }
        else {
            __cov_HnbYpbOoQM4MaLYAY_l6Ig.b['2'][1]++;
            __cov_HnbYpbOoQM4MaLYAY_l6Ig.s['15']++;
            browser.switchTab(currentTabId);
        }
    });
}

Cool huh? Now I understand how instanbul is able to see what code was visited during unit test execution. In the coverage context, I'm sure __cov_HnbYpbOoQM4MaLYAY_l6Ig is defined and works fine. But, in my unit test context, it doesn't exist. :)
Here are a couple of gulp tasks that I think are relevant:
gulp.task('pre-test', function () {
  return gulp.src(['build/**/*.js', '!build/src/features/**/*'])
    .pipe(istanbul({includeUntested: true}))
    .pipe(istanbul.hookRequire());
});

gulp.task('unit-tests',** ['clean-coverage', 'pre-test'], function(){
    return gulp.src(['build/src/**/*.spec.js'])
        .pipe(mocha({reporter: 'spec'}))
        .pipe(istanbul.writeReports({
            dir: './coverage',
            reportOpts: {
                dir: './coverage'
            },
            reporters: ['text-summary', 'html']
        }))
        .pipe(istanbul.enforceThresholds({ thresholds: { global: 80 } }));
});

So, I think I need a way to prevent istanbul from adding coverage code to this function. I have tried /* istanbul ignore next */ in various places with no luck (coverage code still present). I would gladly provide my unit test context with a defined __cov_HnbYpbOoQM4MaLYAY_l6Ig but that slug changes each time.
Not sure what to try next. Any help would be appreciated!

This (https://github.com/gotwarlost/istanbul/issues/310) helped me resolve the issue at runtime.  Maybe it will help you with Grunt tasks, too?


Automated Package Upgrade

:tada: This PR is included in version 28.2.46 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


TypeError: undefined is not an object (evaluating 'progressRef.current.reAnimate')

i have same problem please check

I'm having the same issue.