Add template execution tracing

I am struggling with understanding how Hugo works. Part of the problem is that I cannot see
how templates are being considered and which are chosen for execution.
This small patch is my first attempt at Go. I am confident that I am logging template execution
properly. I seems that there is some need for this (https://discourse.gohugo.io/t/see-which-template-is-used-by-hugo/9983 et al).
Example usage:
t$ ~/Source/hugo/hugo -DEF -cleanDestinationDir --debug
INFO 2020/11/25 09:54:29 Using config file:
Start building sites …
INFO 2020/11/25 09:54:29 syncing static files to /home/kris/hugo/quickstart/public/
DEBUG 2020/11/25 09:54:29 Render page Categories (template _default/terms.html) to "/categories/index.html"
DEBUG 2020/11/25 09:54:29 Execute template: "_default/terms.html"
DEBUG 2020/11/25 09:54:29 Execute template: "partials/site-favicon.html"
...

Using the --debug flag,

the template chosen is printed with the Render message
template.Execute() is being logged, showing also partials

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Kristian Koehntopp seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks. This looks good. Can you

Push the CLA button above
Adjust the commit message according to guidelines (in your case "add => Add"

Email certified
CLA signed
Commit message edited as per request

Commit message edited as per request

I can easily do it for you, but I guess it could be useful for you to know: If you update the title of the PR in GitHub that will not update the title of the commit message itself. To do that you need to do something ala:
git commit --amend
# edit commit message
git push -f

Thanks.
I did the force-push as per your request.
I refrained from doing this before, as I have been taught that amending and force-pushing usually leads to a lot of breakage.

as I have been taught that amending and force-pushing usually leads to a lot of breakage.

In general you are right, but I think one accepted exception to that rule is pull requests.

Since I got your attention: The second half to the question I am trying to answer would be "As a user, I want to be able to trace Hugo as it loads or considers templates/files, and find/does not find them."
As I am learning Go and am familiarizing myself with the codebase, my theory would be that I have to look
into the hugofs part of the codebase? Or where would I have to look?

It's a 2 step operation:

https://github.com/gohugoio/hugo/blob/master/output/layout.go#L66
The above gives a set of candidates for a given page, we then pick the first of we find of those.


Fixup formatting and spelling in the readme

Markdown needs two spaces at the end of a line to know to add a line break. Prior to this change, the type, description, and default lines from each server option were rendered as a single line instead of three lines.

If desired, we could instead use bullet points for the three lines under each heading instead of the double space at the end of lines.


Infinite Redirects when "Authorization" header is not passed explicitly

Version
3.8.5
Context
The OAuth2AuthHandler enters a redirect loop if the Bearer is not present and Sessions are enabled.
What happens:

User requests a protected route.
User is redirected to keycloak.
User is now (or was) authenticated, redirect user to callback route.
Redirect user to "state" route, usually step 1 route
Bearer not found, redirect user back to keycloak
Repeat 3 to 5 infinitely

The "redirect if Bearer not found" can be found here:
https://github.com/vert-x3/vertx-web/blob/8ed1511766697c9ee9e7e5acaad9041f2ca4c384/vertx-web/src/main/java/io/vertx/ext/web/handler/impl/OAuth2AuthHandlerImpl.java#L81-L119
Edit: Also, "setupCallback" redirects user back to the route specified by the "state" parameter
Do you have a reproducer?
https://github.com/LouizFC/vertx-infinite-loop-reproducer
Steps to reproduce

Import the realm-export.json into a new keycloak realm.
Regenerate the "vertx" client secret at the Client > vertx > Credentials tab
Create a test user
Export the OIDC JSON into the app MainVerticle class, removing trailing slashes from the auth url
Run the app and open localhost:8080/user at the browser
Login with the test user
Infinite loop

Thanks for the report, I'll investigate it.

I believe this issue is solved. Please re-open if not.


Query: How to pass multiple images as multichannel input into the model in anomalib?

I have multiple images of the same defect in different lighting conditions. How can I merge them as 1 image having multiple channel in anomalib?

Why would you do this? You can pass multiple image one after another to the anomalib models via a datamodule. One (RGB) image already has 3 channels. If you are working with grayscale image, you try to concatenate 3 single-channel images to one 3-channel image, but I'm not sure if that helps you with anomaly detection accuracy (would be interesting to know though!)

I have 4 RGB images representing the same defect in different lighting conditions. Each lighting condition highlights a different aspect of the defect. In timm models (which is being used in anomalib), we can pass in_channels. In pytorch dataset module, we can customize the channels. How can we do the same in anomalib?

Ahh ok so the timm model loading function is already designed in a way to support more than three channels even with pre-trained models: https://timm.fast.ai/models#How-is-timm-able-to-use-pretrained-weights-and-handle-images-that-are-not-3-channel-RGB-images?
There is no interface to this functionality in anomalib built-in at the moment but you can do it yourself by modifying how the timm models are created in anomalib https://github.com/openvinotoolkit/anomalib/blob/4b1e08ca318a22239c9e396edc513b3a0fc860b7/src/anomalib/models/components/feature_extractors/timm.py#L69
give it a try and tell us how this changed the anomaly detection performance!

If camera and object in fixed position, you can use stereo photometric to generate one image form multiple light.


wamp server refresh and replace cached and create po file

In wamp server I have problem to exec shell command there is a solution for refresh and replace cached version of template and create PO file by php ?
welcome any other suggestions.
Thank you!
EDIT:
I saw that are 2 functions in index.php so I exec from prompt
php ./index.php clear-template-cache -> OK
php ./index.php precompile-templates -> ERROR
Delight\Foundation\Throwable\TemplateNotFoundError: Unable to find template "Z:/ws/www/foundation/views/dashboard.html.twig" (looked into: Z:\ws\www\foundation/views). in file Z:\ws\www\foundation\vendor\delight-im\foundation-core\src\TemplateManager.php on line 162
Stack trace:

Delight\Foundation\Throwable\TemplateNotFoundError->() Z:\ws\www\foundation\vendor\delight-im\foundation-core\src\TemplateManager.php:162
Delight\Foundation\TemplateManager->precompile() Z:\ws\www\foundation\vendor\delight-im\foundation-core\src\App.php:931
Delight\Foundation\App->precompileTemplates() Z:\ws\www\foundation\index.php:111

Thank you for your feedback on this!
The two commands that you tried will refresh the template cache for you, which is needed for all translations to be correctly extracted from the templates. But the commands will not perform the actual extraction. For this, you need a third command, which is ./i18n.sh and will not work on Windows. But you could perhaps use WSL on Windows, or run the underlying commands yourself, which isn’t too hard. Just look inside i18n.sh to see which Gettext tools it calls.
Now concerning the error you described, TemplateNotFoundError, this is caused here. Could you open vendor/delight-im/foundation-core/src/TemplateManager.php in your local application and try to debug this in order to see why it happens. It would be great if you could check which template it tries to load and why that fails, i.e. whether it’s actually missing or just cannot be found due to a wrongly specified path. Perhaps realpath must be called on the path before trying to load the template.

Thanks. For the extraction I try to find some solution as you suggested. About the error I try to debug as soon as I come back. Just because there were different template and I suppose a wrong path I tried to make a test.  I empty the folder view and just put 1 template dashboard.html.twig and run the php command by windows prompt but nothing always the same error.

Perhaps realpath must be called on the path before trying to load the template.

Where I should use realpath ?
My files are

routes

$app->get('/admin/dashboard', ['\App\Controllers\Backend\Core\Dashboard', 'getDashboard']);

class

namespace App\Controllers\Backend\Core;

	use Delight\Foundation\App;

	class Dashboard {
		public static function getDashboard(App $app) {
			echo $app->view('/backend/'. $_SESSION['BE_THEME'] .'/core/dashboard.html.twig', [
				'be_page_identifier' => 'dashboard',
				'be_page_ref' => 'core',
				'be_page_do' => 'dashboard'
			]);
		}
	}

BE_THEME is just for change template in this caxse is "default"
and in folder I put only this template
view/backend/default/core/dashboard.html.twig
Anyway I'll try to debug the template and I'll let you know.

Ok I tried to print the $templateName in foreach and this is the result:
Z:\ws\www\foundation/views\backend\default\core\dashboard.html.twig
problem with \  /  ?

Thanks!
You would need to open vendor/delight-im/foundation-core/src/TemplateManager.php and insert some debugging statements before (or around) this exact line.
For example, what you could try temporarily is
$templateName = \realpath($templateName);

or
$templateName = \str_replace('Z:\\ws\\www\\foundation/views\\', '', $templateName);

or even
$templateName = 'backend/default/core/dashboard.html.twig';

$templateName = \realpath($templateName);
same error
$templateName = \str_replace('Z:\\ws\\www\\foundation/views\\', '', $templateName);
Work.
I fix in this way when I work in local ?
IF YES now show me other error
TemplateSyntaxError: Unknown "ifFileExist" filter.
ifFileExist is a callback function that I use in template for check if file exist or use fallback file
{% set select2_lang = '/assets/select2/js/i18n/'~ session.EZ_LANG ~'.js' %}

{% if select2_lang | ifFileExist %}
<script src="{{ app.url('/assets/select2/js/i18n/'~ session.EZ_LANG ~'.js') }}"></script>
{% else %}
<script src="{{ app.url('/assets/select2/js/i18n/'~ session.EZ_FALLBACK_LANG ~'.js') }}"></script>
{% endif %}

Thanks a lot for testing these two changes and confirming that the str_replace solution works.
The reason was indeed that the template paths were not shortened from absolute paths to relative paths on Windows (due to the backslashes in the paths, which were not handled correctly).
This bug should have been fixed in this commit now.
Can you try running
$ composer update delight-im/foundation-core

to update it to v4.10.1 (or later) and seeing if this solves the issue for you? Thanks!
Your second problem is best discussed in a separate issue, but generally, it means that ifFileExist has not been defined via
$app->getTemplateManager()->addFilter('ifFileExist', function ($filename) {
    // TODO
    // return ...
});

Yes the bug is fixed. About the second issue yes is definied
$app->getTemplateManager()->addFilter('ifFileExist', function($file) {
 ...
});

I try to make some debug if I don't fix I'll open a different issue.
As always you are fast and kindly to offer your professional support.
Thank you very much!

Thanks for confirming the fix!
Let’s discuss your filter in the other issue.


chore: update readme testing directions

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Multi-Tenancy for Routers

Allow ziti-router instances to be used by multiple controllers.

Will likely pursue multi-tenancy via multi-process rather than intra-process. Will re-open if needed


⚠ Bofrost.fr has degraded performance

In 63c7d84, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 916 ms

Resolved: Bofrost.fr performance has improved in 60c7fc5 after 6 minutes.


Only show the first photo when add MWPhotoBrowser as subview

I've created the MWPhotoBrowser object in my viewcontroller as a subview from my self.view:
For 1 to 3
{
[photos addObject:[MWPhoto photoWithURL:[NSURL URLWithString:@"http://farm4.static.flickr.com/3590/3329114220_5fbc5bc92b_b.jpg"]]];
}
self.photos = photos;
MWPhotoBrowser *browser = [[MWPhotoBrowser alloc] initWithDelegate:self];
browser.displayActionButton = NO;
browser.wantsFullScreenLayout = NO;
[self.view addSubview:browser.view];
[self.view bringSubviewToFront:browser.view];
and, so, only show me the first image, but the scroller works fine for the three images. However if I define the MWPhotoBrowser object so:
[self.navigationController pushViewController:browser animated:YES];
The browser show me correctly the three images.
And idea how to fix it with the first method?
Sorry for my english and congratulations for this project. Thanks a lot

In iOS 5 works fine but in iOS 4.3.3 only show the first image

Hi there. The photo browser is designed to be presented within a UINavigationController; you shouldn't add view controller's views to other views (i.e. [self.view addSubview:browser.view]), that is a general principle in iOS — unless you're properly using view controller containment in iOS 5.


feat(rust): decouple Context from Worker

This does the following things, more or less (it's a fairly large refactor):

Introduces a new NodeContext trait.
Makes Worker/Processor generic over <C: NodeContext>
Changes all the workers/processors to behave this way
Changes all the structs that need to reference Contexts to be generic over the Context (for example, Handle<C: NodeContext> rather than Handle)
Moves Handle/Cancel and such into ockam_core
Updates examples, etc (fewer changes here than expected).

Things I'm not happy with (none of these need to be addressed before landing, I think):

I think the api of NodeContext is a bit janky. It's not really minimal. Much of this needs the (forthcoming) generic async work before it can be improved, but after that should be improvable.
Several things that used to take impl Into<Route> or impl Into<String>, can't anymore, at the moment, but this should be fixable in the future (although, it might take true async traits, but we don't need dynamic dispatch so this might be relatively soon...). This mostly impacts internals, rather than the public API used by examples, since Context still defines these as inherent methods.
AsyncTryClone derive needs to be adapted so that it can handle the generic type.

I'm wondering if we can abstract the underlying runtime the same way we currently do with ockam_node in the ockam crate, i.e. by creating the API we need from a runtime (probably spawn, block_on, channels, ...?) in our own crate, which can then include either tokio or async-std or whatever via a feature flag.
There's no reason the async runtime needs to be a generic type on Context (or for Context to even understand that there can be different runtimes)

there are a bunch of style nitpicks in this PR which should probably be pulled out into it's own set of changes. This change set is already pretty huge and some things seem to be changed for style reasons more than functionality.

Ah yeah this is fair, I started by trying to clean up the API to something we could stabilize, but then decided to do that later once the scope of this was more clear.
The Into stuff is unfortunate, I agree. I have some other thoughts about how we can do this (a wrapper around impl NodeContext (or &mut impl NodeContext) that provides the intos prior to entering async) — the issue is that we see this as just an convenience helper on the arguments, but rustc believes that the inputs have to potentially live for the entire length of the async computation, and potentially need to be sent to other threads.
This is unfortunate, and is made worse by the async_trait macro, so I'm hopeful it's not a permanent situation — async trait support is fairly far along, as long as you don't need dyn trait support (which is very unclear when it will happen).

Oh, the other category of "stylistic changes" is "things that fail linting from ../../gradlew lint" (which I had been using as "approximately what CI does") Many of these were things I had thought I had (somehow) caused, but it seems that ../../gradlew lint just fails on develop 🥲

Would be great to revisit in the future


usb port stop send power output after shutdown

I used to be able to charge my other devices using laptop usb port while it was charging but after I try using winutil laptop tweak its stop working

oh seems like I misclick and select desktop which is disable hibernate now I cant hibernate how can I undo that?

I try undo but still not showing


docs: Add note on the behaviour of stop meth

Codecov Report

Merging #40 into kite3 will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            kite3      #40   +/-   ##
=======================================
  Coverage   77.04%   77.04%
=======================================
  Files           3        3
  Lines         305      305
  Branches       36       36
=======================================
  Hits          235      235
  Misses         62       62
  Partials        8        8

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 0881adc...e9241a5. Read the comment docs.


NullBufferedCipher Broken

Hello,
i created a fresh console .net 7 project and used npoi 2.6.1 (newest version).
using NPOI.POIFS.Crypt.Standard;
var x = new NullBufferedCipher();

Than you get the following exception:
Unhandled exception. System.TypeLoadException: Method 'ProcessByte' in type 'NPOI.POIFS.Crypt.Standard.NullBufferedCipher' from assembly 'NPOI.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0df73ec7942b34e1' does not have an implementation.
Can you please fix it?

Why do you wanna use NullBufferedCipher?

@tonyqus we dont want to use NullBufferedCipher but we are doing some reflaction stuff in our UnitTests.
Something like this:
var workbook = new XSSFWorkbook();
var types = AppDomain.CurrentDomain.GetAssemblies()
    .SelectMany(s => s.GetTypes())
    .ToList();

Than you get the same error, so your break with npoi the .net reflaction GetTypes function.

Not every class in NPOI is fully implemented. I have no idea why you wanna iterate all the classes in NPOI.

With version 2.6.0 it's works fine. Its feels a little bit unsafe, if we using a library, where some classes are not working correctly and they are public and not internal.
We are using 87 different nuget packages and no one did something like that, so its feels like a bad habbit.
But if you say its a good practice to do something like that, maybe other libraries will following this trend and we need to accept it.

Some follow up: @tonyqus
That is not working too:
var x = new NPOI.XSSF.Streaming.SXSSFCell(null, CellType.Blank);
x.SetCellValue(DateOnly.MinValue);

You publish a nuget package with .netstandard2.0 and .netstandard2.1, but you use .net 6 feature like dateonly in your project. see https://www.nuget.org/packages/NPOI
#if NET6_0_OR_GREATER
        public void SetCellValue(DateOnly value)
        {
            bool date1904 = ((SXSSFWorkbook)Sheet.Workbook).XssfWorkbook.IsDate1904();
            SetCellValue(DateUtil.GetExcelDate(value, date1904));
        }

This compile condition will never be true since you are using .netstandard2.1.

Some follow up: @tonyqus That is not working too:
var x = new NPOI.XSSF.Streaming.SXSSFCell(null, CellType.Blank);
x.SetCellValue(DateOnly.MinValue);

You publish a nuget package with .netstandard2.0 and .netstandard2.1, but you use .net 6 feature like dateonly in your project. see https://www.nuget.org/packages/NPOI
#if NET6_0_OR_GREATER
        public void SetCellValue(DateOnly value)
        {
            bool date1904 = ((SXSSFWorkbook)Sheet.Workbook).XssfWorkbook.IsDate1904();
            SetCellValue(DateUtil.GetExcelDate(value, date1904));
        }

This compile condition will never be true since you are using .netstandard2.1.

It was already reported in https://github.com/nissl-lab/npoi/issues/1118

Okay so then the NullBufferedCipher will be fixed in #1118 too, like you see here:
https://github.com/nissl-lab/npoi/blob/b4b94feb29641aa3ddd87c763e86ab80c00a371c/main/POIFS/Crypt/Standard/StandardDecryptor.cs#L403
So the answer "Not every class in NPOI is fully implemented. I have no idea why you wanna iterate all the classes in NPOI." is not correct, because you implement the NullBufferedCipher correctly and we can use GetTypes like before.

We are evaluating the impact of the incorrect release of NPOI 2.6.1.  If impact is huge, we will release NPOI 2.6.2. But it looks not many developers are using new features of .NET 6.0 or .NET 7.0

I'm having the same issue too

Same issue here too - any reflection that attempts to load the NPOI assembly throws this:
System.Reflection.ReflectionTypeLoadException: Unable to load one or more of the requested types.
Method 'ProcessByte' in type 'NPOI.POIFS.Crypt.Standard.NullBufferedCipher' from assembly 'NPOI.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0df73ec7942b34e1' does not have an implementation

I'm preparing [NPOI 2.6.2 release]. This issue will be fixed.
Release notes: #1168

NPOI 2.6.2 rc1 released

@tonyqus Thanks for releasing 2.6.2, everything is working fine now.


⚠ Bofrost.fr has degraded performance

In badf4c6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1457 ms

Resolved: Bofrost.fr performance has improved in 0fd5d42.


Test DistortionsListener.prototype.ignorePrimordials

Oops.  This API was not covered!

Blocks #106.

Fixed!


fix: vectorstore pgvector ensure compatibility #3884

Ensure compatibility with both SQLAlchemy v1/v2
fix the issue when using SQLAlchemy v1 (reported at #3884)
langchain/vectorstores/pgvector.py", line 168, in create_tables_if_not_exists     self._conn.commit() AttributeError: 'Connection' object has no attribute 'commit'
Ref Doc : https://docs.sqlalchemy.org/en/14/changelog/migration_20.html#migration-20-autocommit

thanks @hanguofeng!


MercSkinsFix compatibility issues with Inferno Mod

MercSkinsFix does not work with Inferno mod. it gets stuck at 100%. tried everything and nothing works

Thank you for the report! This has been fixed in the newest mod update.


Inverse search opening file in wrong neovim instance

Thanks for writing this plugin, it's a huge time-saver!
I've run into an issue though. If I have two TeX files open in two different Neovim instances with their corresponding PDFs open in Zathura, inverse searching from both Zathura instances will try to open the file in the most recently opened Neovim instance.
I tried looking through the plugin source, but I don't see a place where all the open Neovim instances are searched for the appropriate buffer.
Am I doing something wrong here?

Minimal init.vim:
" Plugin manager
call plug#begin(stdpath('data') . '/plugged')
Plug 'f3fora/nvim-texlabconfig', {'do': 'go build'}

" Set up texlabconfig
lua << EOF
require('texlabconfig').setup({})
EOF

" Configure texlab
lua << EOF
local texexecutable = 'zathura'
local texargs = {
    '--synctex-editor-command',
    [[/home/steven/.local/share/nvim/plugged/nvim-texlabconfig/nvim-texlabconfig -file '%{input}' -line %{line}]],
    '--synctex-forward',
    '%l:1:%f',
    '%p',
}
local servers = {'texlab'}
for _, lsp in pairs(servers) do
  require('lspconfig')[lsp].setup {
    on_attach = on_attach,
    flags = {
      debounce_text_changes = 150,
    },
    settings = {
        texlab = {
            bibtexFormatter = "texlab",
            build = {
                forwardSearchAfter = false,
                onSave = true,
            },
            chktex = {
                onOpenAndSave = true,
            },
            diagnosticsDelay = 300,
            diagnostics = {
                ignoredPatterns = {"You should not use punctuation in front of quotes"},
            },
            formatterLineLength = 80,
            forwardSearch = {
                executable = texexecutable,
                args = texargs,
            },
            latexFormatter = "latexindent",
            latexindent = {
              modifyLineBreaks = false,
            }
        },
    },
  }
end
EOF

This plugin is pretty simple. The behavior described is the one I expected. I understand however it is not optimal. I will try to improve this when I have time.

Thanks for the quick answer! I threw together a workaround for myself here: https://github.com/sdahdah/nvim-texlabconfig/commit/aa82339298c999b7dd334e58f1616faedb172c13
I don't know go at all, so it's not pretty. But is something like that interesting to you? It requires this in init.vim:
local texargs = {
    '--synctex-editor-command',
    [[nvim-texlabconfig -server ]] .. vim.api.nvim_get_vvar('servername') .. [[ -file '%{input}' -line %{line}]],
    '--synctex-forward',
    '%l:1:%f',
    '%p',
}

First, the program tries to open the specified server, then it defaults to looking through the cache.

That seems good to me. Would you like to open a PR?


Use other datasets to train PENet

Hello
I now want to use the PENet network architecture to complete depth completion, because I only need the dataset at hand, so I found the HandNet dataset, which contains depth data and image data collected by the realsense series of depth cameras. I especially want to know where do I need to modify the code? Thank you in advance!
Looking forward to your reply!

Thanks for your interest yuyu! Individually I guess you could write your own dataloader for the HandNet dataset.

Thanks for your quick reply! If I write my own data loader for HandNet, if its depth map and rgb image are already aligned, do I need to use the function load_calib() in kitti_loader? I think I should not need the value of k. Very much looking forward to your reply!

Thanks for your quick reply! If I write my own data loader for HandNet, if its depth map and rgb image are already aligned, do I need to use the function load_calib() in kitti_loader? I think I should not need the value of k. Very much looking forward to your reply!

You might need it if you want to use the proposed geometric convolutional layer, otherwise not.

Thank you very much for your reply!
Thank you very much for your reply!
First of all, I may need to bother you with another question. Is K in the code the internal parameter matrix of the camera?
In addition, I now want to train a 640480 uncropped image dataset, I see that your code is 3521216, so can I directly modify these two numbers?
Looking forward to your reply, thank you in advance! ! !

Runtimeerror:The size of tensor a(288) must match the size of tensor b(1216) at non-singleton dimension3
I have this problem now, do you know what is the reason? thank you very much!

Thank you very much for your reply!
I may need to bother you with another question. Is K in the code the internal parameter matrix of the camera?
Looking forward to your reply, thank you in advance! ! !

Exactly.

alright, thank you very much! I trained the network today. I used the NYU-Depth v2 dataset and changed k to the camera's internal parameter matrix in this dataset, but after 100 rounds of running, the RMSE is still very large. My training set is 1000, the evaluation set is 200, and the test Set 200, could the RMSE reach a few thousand because the training set is too small?

alright, thank you very much! I trained the network today. I used the NYU-Depth v2 dataset and changed k to the camera's internal parameter matrix in this dataset, but after 100 rounds of running, the RMSE is still very large. My training set is 1000, the evaluation set is 200, and the test Set 200, could the RMSE reach a few thousand because the training set is too small?

You could refer to other works (e.g. NLSPN) for the training settings on NYU Depth V2 dataset. After a full training procedure, ENet could reach an RMSE of around 104 mm.

Good! Thank you so much! I'm going to try what you said right away!

Hi, I'm sorry to bother you again.
I have now trained PENet with KITTI, but I still have a question: can this network be trained with NYU depth v2? If not, I'll change my mind. Especially looking forward to your reply.
I changed the format of the NYU depth v2 dataset to the image format required by PENet.Can I do this? Because I ran into some other problems.
Very much looking forward to your reply!

Hi, I'm sorry to bother you again. I have now trained PENet with KITTI, but I still have a question: can this network be trained with NYU depth v2? If not, I'll change my mind. Especially looking forward to your reply. I changed the format of the NYU depth v2 dataset to the image format required by PENet.Can I do this? Because I ran into some other problems. Very much looking forward to your reply!

I didn't change the format of NYU depth V2 and directly rendered the NYU depth v2 dataloader from Park's (NLSPN) repositories. It works normally with a common training setting on the indoor dataset. I hope these files could help.
nyu.json.txt
nyu_loader.py.txt

OK! I will take a serious look at what you have to offer!
thank you very much!


Changed the UI of Yoga Page and added few class

solved issue #299
As proposed earlier I changed the UI of the Page and made it responsive as well. Here are a few of the screenshots of the updated UI: (pls ignore the timer in the screenshots)
Desktop:

Tablet:

Mobile:

@MridulTi Sorry but the current UI is good rather than this. Closing this for now. Thanks for your ideas.


publish: allow enabling/disabling individual post types

e.g. opting into/out of posts, comments, likes, rsvps individually.
i'd like this myself. i'd disable posts but enable everything else.

CMSes can do this themselves by only sending wms for posts they want to publish. so, i'm going to close this.


Path not being reported by shell...? (windows 10)

I don't know precisely what´s going on. I have gulp installed correctly and can run gulp from cmd in my project. The gulpfile is set correctly. But when I run gulp on sublime, the panel reports 'gulp is not a recognized command', even though it is installed globally (and locally on my project).

Sorry for the delay, been a bit busy with a moving this last couple of months.
The problem could be windows not reporting the path correctly, to verify this, try using the exec_args setting, giving it a value like:
    "exec_args": {
        "path": "/bin:/usr/bin:/usr/local/bin"
    }

Let me know!


[vcpkg-tool-meson] Restore the PKG_CONFIG_PATH environment variable to the correct path

Fixes #40305
Paths in the PKG_CONFIG_PATH environment variable should be respected when calling vcpkg_configure_meson(), so remove option --pkg-config-path

[x] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@marcbull @Neumann-A

Restore the PKG_CONFIG_PATH environment variable to the correct path

You currently do not achieve that. adjust the flag correctly instead of removing it.


Upgraded DevCenter to version 1.5.0

Upgrades the formula to install Datastax DevCenter 1.5.0 that improves existing features and adds Cassandra 3.0 features like Materialized views (source)

Thank you for the contribution. It was merged directly as f5f00abd44538713d930998176c55cfef979d7a0 to keep commit history cleaner. Your contribution is still credited to you.

No problem thanks for the update :)


Optimize lv_task function

Hi,
I wanted to optimize its impulses when looking at the lv_task_handler function implementation, because the implementation made it a little puzzling.
The following is the code that I modified to help me see if there is any problem.
1.add lv_ll_ins_pos function
/**

Insert a new node in front of the n_act node

@param ll_p pointer to linked list

@param n_act pointer a node

@return pointer to the new head
/
void * lv_ll_ins_pos(lv_ll_t * ll_p, void * n_act)
{
lv_ll_node_t n_new;
lv_ll_node_t* n_prev;
if(NULL == ll_p || NULL == n_act) {
return NULL;
}
if(lv_ll_get_head(ll_p) == n_act) {
n_new = lv_ll_ins_head(ll_p);
}
else {
n_new = lv_mem_alloc(ll_p->n_size + LL_NODE_META_SIZE);
if(n_new != NULL){
n_prev = lv_ll_get_prev(ll_p, n_act);
node_set_next(ll_p, n_prev, n_new);
node_set_prev(ll_p, n_act, n_new);
}
}
return n_new;
}

2.modify  lv_task_create function
/**

Create a new lv_task

@param task a function which is the task itself

@param period call period in ms unit

@param prio priority of the task (LV_TASK_PRIO_OFF means the task is stopped)

@param param free parameter

@return pointer to the new task
/
lv_task_t lv_task_create(void (task) (void ), uint32_t period, lv_task_prio_t prio, void * param)
{
lv_task_t new_lv_task;
lv_task_t tmp;
//Create task lists in order of priority from high to low
tmp = lv_ll_get_head(&lv_task_ll);
if(NULL == tmp){
new_lv_task = lv_ll_ins_head(&lv_task_ll);
}
else{
do{
if(tmp->prio <= prio){
new_lv_task = lv_ll_ins_pos(&lv_task_ll, tmp);
break;
}
}while(NULL != (tmp = lv_ll_get_next(&lv_task_ll,tmp)));
 if(tmp == NULL){
     new_lv_task = lv_ll_ins_tail(&lv_task_ll);
 }

}
lv_mem_assert(new_lv_task);
new_lv_task->period = period;
new_lv_task->task = task;
new_lv_task->prio = prio;
new_lv_task->param = param;
new_lv_task->once = 0;
new_lv_task->last_run = lv_tick_get();
return new_lv_task;
}

modify lv_task_handler function

inline void LV_ATTRIBUTE_TASK_HANDLER lv_task_handler(void)
{
static uint32_t idle_period_start = 0;
static uint32_t busy_time = 0;
uint32_t handler_start;
lv_task_t* tmp;
int endFlag;
if(lv_task_run == false) return;
handler_start = lv_tick_get();

while(1){
    endFlag = 1;
    LL_READ(lv_task_ll,tmp){
        if(true == lv_task_exec(tmp,tmp->prio)){
            endFlag = 0;
            break;
        }
    }
    if(endFlag) {
        break;
    }
}

busy_time += lv_tick_elaps(handler_start);
uint32_t idle_period_time = lv_tick_elaps(idle_period_start);
if(idle_period_time >= IDLE_MEAS_PERIOD) {

    idle_last = (uint32_t)((uint32_t)busy_time * 100) / IDLE_MEAS_PERIOD;   /*Calculate the busy percentage*/
    idle_last = idle_last > 100 ? 0 : 100 - idle_last;                      /*But we need idle time*/
    busy_time = 0;
    idle_period_start = lv_tick_get();
}

}

May I ask you to create pull request from the changes. Please, also include #98 changes.
Just to be sure I won't miss anything

How to submit changes, Here is my operation, the last error：
git clone https://github.com/littlevgl/lvgl.git
git branch -r
git branch
git branch beta
git checkout beta
git push origin beta
Username for 'https://github.com'<EMAIL_ADDRESS>Password for 'https://601714078<EMAIL_ADDRESS>remote: Permission to littlevgl/lvgl.git denied to chenshijianworkgit.
fatal: unable to access 'https://github.com/littlevgl/lvgl.git/': The requested
URL returned error: 403

Pull request are not git but GitHub features. It's about you request me (in this case) to add your branch to lvgl.
Here you will fing how  to do it using GitHub: https://help.github.com/articles/about-pull-requests/

The code has been submitted to you, and you have a time to look at it.

Finally I merged the the update in dev-5.1.
Can you check them?

Yes, the New Year holiday, a few days did not see littlevgl library.
Start working now and watch your update.

Okay, take your time, and Happy New Year! :)

Happy New Year：
I'm sorry to reply you so late.
just read the update that you submit, the comment is very clear and easy to understand, thank you very much the perfect solution to my problem.
the lv_area_is_on function does not seem to be updated.

Just read the lv_area_is_on function submitted in another issue, then this issue can be closed.

Thank you very much for your help and great ideas!


[collab-galley] Add basic kubernetes file processing utilities

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test integ-mixer-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test integ-mixer-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test integ-mixer-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test integ-mixer-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test e2e-simpleTests-cni

prow/e2e-bookInfoTests-trustdomain.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test e2e-bookInfoTests-trustdomain

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test integ-mixer-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test e2e-simpleTests-cni

prow/e2e-bookInfoTests-trustdomain.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test e2e-bookInfoTests-trustdomain

prow/integ-security-k8s-presubmit-tests.sh
86582849ef39355376df14c696b1641c3573a49b
link
/test integ-security-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [geeknoid,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


wip commit on #75 implementing encoder and decoder for java time classes

So this is a rough first crack I do have a couple questions

How would you like to handle formatters for LocalDateTime and the like
I'm not sure what I'm doing wrong on the decoder side, but most of the added tests fail
What are your thoughts on how I structured the tests (i.e. for enumerated values should I just map on the values or keep it explicit like it is currently?

Thanks and I look forward to the feedback!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Brandon Brown seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for this work, Brandon!

How would you like to handle formatters for LocalDateTime and the like

I think we can restrict most of the formatted date to ISO8601 for interoperability usability’s sake, for example I don’t anticipate every receiving a Zone like America/New_York over the wire, but have seen multiple variants of ISO8601 with up microsecond precision.

I'm not sure what I'm doing wrong on the decoder side, but most of the added tests fail

assert("MONDAY".fromJson[java.time.DayOfWeek])(isRight(equalTo(java.time.DayOfWeek.MONDAY)))

fromJson requires a encoded JSON string - you’re missing the " around your it.

What are your thoughts on how I structured the tests (i.e. for enumerated values should I just map on the values or keep it explicit like it is currently?

They are very readable! Consider moving them to their own files and import java.time._ there.

fromJson requires a encoded JSON string - you’re missing the " around your it.
Haha, yeah that makes perfect sense! I'll work on cleaning this up this morning, thanks for the quick feedback!

[ ] move java tests to their own files
[ ] fix decoder test by properly having strings
[ ] use mapOrFail when calling impure functions
[ ] stick with ISO8601 formatting for all places where a formatter can be used

What should we do about Instant? Other JSON libraries seem to use ISO-8601 to en/decode it

I think ISO-8601 is fine but probably a number (Long) is fine too (epoch millis).


[Bug]: Windows 11 - Resize on drag

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When moving the window around on Windows 11, the vscode window shrinks. Not related to built-in snapping features.
Resolution: 3840 x 2160 pixels
Scale: 150%
Expected Behavior
Window will retain its existing size.
Steps To Reproduce

In Windows 11
Naked installation
4K resolution (16:9) with scale at 150%
Drag the window around

Environment
- OS: Windows 11
- VSCode: 1.72.2
- Extension: 1.1.9

Anything else?
No response

Open issue exists in the old plugin: https://github.com/EYHN/vscode-vibrancy/issues/109

Likely related to scale factor here: https://github.com/illixion/vscode-vibrancy-continued/blob/27803a92af08de4355bdc6d72e8be59c39c14c34/runtime/win10refresh.js#L181

I have the same issue!

Thanks, will check this out. I’m not sure if this Windows 10 fix is even needed still, so I might just remove it entirely when I get a chance to test Vibrancy on a Windows 10 machine.

I have deleted this lines from index.js at VS_CODE_PATH\data\extensions\illixion.vscode-vibrancy-continued-1.1.9\runtime\index.js and run "reload vibrancy" command. It works now!
const win10refresh = require('./win10refresh.js');
win10refresh(window, 60);

I have deleted this lines manually from index.js at VS_CODE_PATH\data\extensions\illixion.vscode-vibrancy-continued-1.1.9\runtime\index.js and run "reload vibrancy" command. It works now!
const win10refresh = require('./win10refresh.js');
win10refresh(window, 60);

Thanks to your comment, I narrowed down the problem.
The problem with this is that disabling it causes window resizing to be incredibly slow, which is apparently part of what these functions were fixing.
You to fix it entirely, go into
VS_CODE_PATH\data\extensions\illixion.vscode-vibrancy-continued-1.1.9\runtime\win10refresh.js
And comment out the entire function win.on('will-move', (e, newBounds) => {
This way we can have our cake and eat it too


Add users.update_user_password() method

Description
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

Current dependencies on/for this PR:

users

PR #191

PR #192

PR #193

PR #195

PR #196

PR #197

PR #198

PR #199

Codecov Report

Merging #207 (69d588e) into feature/usrld-968-fix-authenticate-methods (549c858) will increase coverage by 0.05%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@                              Coverage Diff                               @@
##           feature/usrld-968-fix-authenticate-methods     #207      +/-   ##
==============================================================================
+ Coverage                                       87.64%   87.69%   +0.05%
==============================================================================
  Files                                              32       32
  Lines                                            1149     1154       +5
==============================================================================
+ Hits                                             1007     1012       +5
  Misses                                            142      142

Files Changed
Coverage Δ

workos/users.py
94.91% <100.00%> (+0.22%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Topic/tvm: various bugfixes

This corrects 4 bugs:

update mbc.q and mbc.alpha from the tvm variable in Robot
fix the order of branches in the ternary operator at Robot.cpp::120 avoiding to access bName when it is void
reset refVel properly in PositionFunction
fix the jacobian computation in Frame

Thanks for the fixes. I rebased on mc_rtc master today as I was working so I cherry-picked your commits.
I've reworked https://github.com/gergondet/mc_rtc/pull/2/commits/a5eff5fa3987d30f74d5643db07e07ec7c884a9b to include other functions that were not doing this (and reset refAccel as well). In fact this goes unnoticed in mc_rtc usage as these functions are used through the TrajectoryTaskGeneric wrapper which resets the reference velocity and acceleration in its own reset...


Drawings on the map: Ground overlays

Is there any progress on adding overlays?

Yes, I am also interested in the map overlays. Is there a realistic timescale for it being ready? We would like to move over to Capacitor, but this is the one missing feature and is beyond our skills to create. Thank you for all your hard work and time.


[tool_crash] RPCError

Command
flutter run

Steps to Reproduce

...
...
...

Logs
RPCError
#0      VmService._call (package:vm_service/src/vm_service.dart:1774:7)
#1      VmService.callServiceExtension (package:vm_service/src/vm_service.dart:1726:14)
#2      VmService.callMethod (package:vm_service/src/vm_service.dart:1717:12)
#3      FlutterVmService.callMethodWrapper (package:flutter_tools/src/vmservice.dart:501:28)
#4      FlutterVmService.getFlutterViews (package:flutter_tools/src/vmservice.dart:913:51)
#5      HotRunner.attach (package:flutter_tools/src/run_hot.dart:304:63)
<asynchronous suspension>
#6      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:867:27)
<asynchronous suspension>
#7      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1408:27)
<asynchronous suspension>
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
<asynchronous suspension>
#9      CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#10     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:420:9)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:364:5)
<asynchronous suspension>
#13     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:130:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.24.0, on Microsoft Windows [Version 10.0.22631.4169], locale pl-PL)
    • Flutter version 3.24.0 on channel stable at C:\Users\zygaPG\Desktop\Flutter SDK\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (9 weeks ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at C:\Users\zygaPG\AppData\Local\Android\sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/to/windows-android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Build Tools 2022 17.9.6)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools
    • Visual Studio Build Tools 2022 version 17.9.34728.123
    • Windows 10 SDK version 10.0.22621.0

[✓] Android Studio (version 2024.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)

[✓] VS Code (version 1.94.0)
    • VS Code at C:\Users\zygaPG\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.98.0

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 15 (API 35) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version
      10.0.22631.4169]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 128.0.6613.138
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 129.0.2792.65

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 300451adae589accbece3490f4396f10bdf15e6e
Plugins
path_provider-2.1.4
path_provider_android-2.2.10
path_provider_foundation-2.4.0
path_provider_linux-2.2.1
path_provider_windows-2.3.0
shared_preferences-2.3.2
shared_preferences_android-2.3.2
shared_preferences_foundation-2.5.2
shared_preferences_linux-2.4.1
shared_preferences_web-2.4.2
shared_preferences_windows-2.4.1
uni_links-0.5.1
uni_links_web-0.1.0

Hello @zygaPG
Can you please drop a code snippet, most especially the code that is being called as app begins it's run time ?

Hi @zygaPG, this should have been fixed in 3.24.2, can you upgrade to 3.24.3 to see if you still experience this issue?


Plotter callback

This pull request adds a class PlotGenerator in keras/utils/plotting_utils.py (new file) and a callback Plotter to keras/callbacks.py. The Plotter callback inherits from the History callback and just sends history data for loss and accuracy to the PlotGenerator. The latter one then generates a plot that shows the training loss, validation loss, training accuracy and validation accuracy over time/epochs. The plot is automatically updated at the end of each epoch. The plot can be shown in a window and/or saved to a file. Regression is used by to predict the next couple of values for each line.
I added the plotter callback to both mnist examples.
Example image (mnist):

I deactivated regression in that one, because it's not very helpful when values get close to 100% accuracy.
Example usage:
from keras.callbacks import Plotter
# standard plot
model.fit(X_train, Y_train, callbacks=[Plotter()])

from keras.callbacks import Plotter
# Dont show the plot window, but save to a file
model.fit(X_train, Y_train, callbacks=[Plotter(show_plot_window=False, save_to_filepath="/tmp/last_plot.png")])

from keras.callbacks import Plotter
# No regression
model.fit(X_train, Y_train, callbacks=[Plotter(show_regressions=False)])

Looks cool! But is there an option to disable accuracy plotting? I'm dealing with regression tasks most of the time instead of classification, in that scenario reported accuracy would be 0.

Currently in that case the accuracy chart on the right simply stays empty. There's no flag yet to hide it completely.

The main issue I have with this type of plotting is that it would only be viable when you run your experiments on a machine that has a graphical desktop environment. It's one of the reasons why I would prefer visualization tools to work over the web. Run your experiment on any machine / cluster that has access to the internet, monitor it from any machine that has access to the internet.
Another issue is that the API is both very complex and insufficiently complete.
keras.callbacks.Plotter(save_to_filepath=None, show_plot_window=True, linestyles=None,
linestyles_first_epoch=None, show_regressions=True, poly_forward_perc=0.1,
poly_backward_perc=0.2, poly_n_forward_min=5, poly_n_backward_min=10, poly_degree=1)

That's a lot of arguments (10), and only 3 of them are readily interpretable to me. That's a big UX problem.
Yet, important workflow considerations like showing or hiding accuracy plots are not included.
Thoughts?

Using Bokeh could be a better option for a plotting monitor that could be checked over the web.
The large amount of parameters for regression in the API is indeed not ideal and suggests maybe that it should be decoupled from the plotting object somehow.

+1 for Bokeh

Developing visualization tools on top of Bokeh is a possibility (it was the first name mentioned when we started discussing visualization a couple month ago); if we are going to do that, it would be in a separate visualization app that plugs into Keras. You can join the discussion here: https://github.com/fchollet/keras/issues/254

Closing this as we won't merge it. I do believe this is very valuable code for anybody who does they computation on a desktop with a GDE (e.g. most people who use their own GPU), but I don't think it belongs in master.


Wrapped group identity translation in try/catch and filtered out null tr...

...anslations
Some group identities are not translatable and throws an exception, "System.Security.Principal.IdentityNotMappedException: Some or all identity references could not be translated."
I don't think there is a way to detect if an identity is translatable before attempting it, but we have this issue in our domain. I've run into this error before and I think the only thing that can be done is wrap the translation in a try/catch and eat the exception.

Is it possible to catch just NotMappedException instead of global catch block?

Sure it is and normally I would have allowed other exceptions through or tried to fit into any existing exception handling. But in this use case, failing to translate a group isn't important enough to prevent the user from being imported, crashing the request, and throwing up a ysod - making the app completely unusable to that user. Therefore, I believe it's appropriate to eat any exception from this particular call to Translate. Unless you want to log the exception, then a single catch of System.Exception is sufficient.

From my general point of view it's ok to try to catch a specific exception and do nothing around this specific call. But it's not ok to catch everything - even StackOverflowException, OutOfMemoryException, and others.
What I'm suggesting is doing something like
===
catch(System.Security.Principal.IdentityNotMappedException) { }
instead of the
===
catch {}

I completely understand what you were suggesting. This is my first pull request, so I don't know if there's an etiquette I'm not following. Were you asking me to make that change or are we having a discussion?

No worries :) if you agree that it make sense to change it and I'll merge it.

Not really, but not enough to argue about. Go for it.


On Windows machines with >= 32 cores (more than 64 logical processors) execution fails

Shown as
(venv) my_dir>bump-pydantic --version
bump-pydantic version: 0.7.0

(venv) my_dir>bump-pydantic .
[13:44:38] Start bump-pydantic.                                                                                main.py:61
           Found 10 files to process                                                                                main.py:76
Exception in thread Thread-3 (_handle_workers):
Traceback (most recent call last):
  File "C:\_\py311\Lib\threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "C:\_\py311\Lib\threading.py", line 975, in run
    self._target(*self._args, **self._kwargs)
  File "C:\_\py311\Lib\multiprocessing\pool.py", line 522, in _handle_workers
    cls._wait_for_updates(current_sentinels, change_notifier)
  File "C:\_\py311\Lib\multiprocessing\pool.py", line 502, in _wait_for_updates
    wait(sentinels, timeout=timeout)
  File "C:\_\py311\Lib\multiprocessing\connection.py", line 878, in wait
    ready_handles = _exhaustive_wait(waithandle_to_obj.keys(), timeout)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\_\py311\Lib\multiprocessing\connection.py", line 810, in _exhaustive_wait
    res = _winapi.WaitForMultipleObjects(L, False, timeout)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: need at most 63 handles, got a sequence of length 66
Executing codemods... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╺━━━━━━━  80% 0:00:01Process SpawnPoolWorker-64:

Afterwards it freezes without any processing. Even Ctrl+C doesn't work.
Googling for it suggests that this is a common problem with multiprocessing on  Windows.

This is the issue: https://github.com/python/cpython/issues/89240


Added documentation for review module

Related issues

closes #
Short description and why it's useful

Screenshots of visual changes before/after (if there are any)

Which environment this relates to
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and currently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and I am aware that every new feature should be a module

@kkdg could you re-review this one?

OK, i will finish it today.

@geektutor please commit these suggestions and we can merge that in :)

@geektutor are you willing to finish this?

closing due to long inactivity


Example of "The notify function has been called synchronously! This can lead to frames being dropped."

Please excuse the quality of this elm code :-)
I am reporting this because of the request that appears in the Web Inspector (Region.elm is name of my file):
[Error] Error: The notify function has been called synchronously!
This can lead to frames being dropped.
Definitely report this to <https://github.com/elm-lang/Elm/issues>

	notify (Region.elm, line 3759)
	move (Region.elm, line 3592)

The code is at the end of this post. All it does is allow you to drag a circle around the screen. The cause of the problem lies here (in the step function):
let info = { info | pos <- offset mp (offset info.pos1 info.pos2)}
in Drag info

This, of course, fixes it:
let info' = { info | pos <- offset mp (offset info.pos1 info.pos2)}
in Drag info'

Here's the program. Just drag the circle around to see the problem, with the Web Inspector visible, of course.
import Signal as Signal
import Text as Text
import Graphics.Element as Element
import Graphics.Collage as Collage
import Color
import Mouse

main : Signal Element.Element
main = Signal.map render run

type alias State =
    { pos : Position
    , color : Color.Color
    , region : Region
    , shape : Position -> Color.Color -> Collage.Form
    --, rs : RegionState
   }

shape =
    { pos = (0, 0)
    , color = Color.blue
    , region = crcl 30
    , shape = (\ p color -> Collage.circle 30 |> Collage.filled color |> Collage.move p)
    --, rs = Rest { pos = (0, 0), region = crcl 30 }
    }

type alias Position = (Float, Float)
type alias Region = Position -> Bool

translate : Position -> Region -> Region
translate (x, y) region =
    (\ (x', y') -> region (x'-x, y'-y))

crcl : Float -> Region
crcl radius =
    (\ (x, y) -> x*x + y*y <= radius * radius)

origin = (400, 400)
render : State -> Element.Element
render state =
    Element.flow Element.down
        [ Collage.collage (fst origin * 2) (snd origin * 2) [state.shape state.pos state.color]
        , Text.asText state
        ]

evolve : RegionState -> State -> State
evolve regionState state =
    case regionState of
        Rest rs -> { state | color <- shape.color, pos <- rs.pos } --, rs <- regionState }

        Hover rs -> { state | color <- Color.red, pos <- rs.pos } --, rs <- regionState }

        Drag rs -> { state | pos <- rs.pos } --, rs <- regionState }

run : Signal State
run = Signal.foldp evolve shape state

state : Signal RegionState
state = Signal.foldp step (Rest { pos = shape.pos, region = shape.region }) (updates origin)

-- pos = center
-- pos1 = position of mouse at start of drag (captured during Hover)
-- pos2 = position of center at start of drag
type RegionState
    = Rest { pos: Position, region: Region}
    | Hover { pos: Position, region: Region, pos1: Position}
    | Drag { pos: Position, region: Region, pos1: Position, pos2: Position}

offset : Position -> Position -> Position
offset (x', y') (x, y) = (x' - x, y' - y)

step : Update -> RegionState -> RegionState
step update state =
    case update of
        MouseMove mp ->
            case state of
                Rest info ->
                    if (translate info.pos info.region) mp then
                        Hover { info | pos1 = mp }
                    else state

                Hover info ->
                    if (translate info.pos info.region) mp
                    then Hover { info | pos1 <- mp }
                    else Rest { info - pos1 }

                Drag info ->
                    let info = { info | pos <- offset mp (offset info.pos1 info.pos2)}
                    in Drag info

        MouseDown d ->
            case state of
                Hover info ->
                    Drag { info | pos2 = info.pos }

                Drag info ->
                    Hover { info - pos2 }

                Rest info -> state

type Update = MouseDown Bool | MouseMove (Float, Float)

updates : Position -> Signal Update
updates (x_org, y_org) =
    Signal.merge
        (Signal.map (\ (x, y) -> MouseMove (toFloat x - x_org, y_org - toFloat y)) Mouse.position)
        (Signal.map MouseDown Mouse.isDown)

Many thanks
Rolland

Is it fair to say that this may be a version of #873? If so, can we close this one and follow along there?

That seems appropriate, Evan. Many thanks.

Cool, thanks for the report!


Issue with "Contribution Design" in @site/docs/tutorials/tut-design.md

Describe the problem

Page: Contribution Design
Source: https://github.com/demisto/content-docs/tree/master/docs/tutorials/tut-design.md

The links in the following section no longer work.
https://xsoar.pan.dev/docs/tutorials/tut-design/#classifiers-mappers-types-fields-and-layouts
Screenshots

Closing as this was fixed by https://github.com/demisto/content-docs/pull/850


I would like to see a description of rate-limited sampling in Java.

When using the Java Application agent to send telemetry to Application Insights, rate-limited sampling works by default.
https://learn.microsoft.com/en-US/azure/azure-monitor/app/java-standalone-config#rate-limited-sampling

On the other hand, "Sampling in Application Insights" does not mention rate-limited sampling.
https://learn.microsoft.com/en-US/azure/azure-monitor/app/sampling?tabs=net-core-new

"Sampling in Application Insights" shows that sampling does not work by default in Java. However, rate-limited sampling seems to work.
I think "Sampling in Application Insights" is the Docs on sampling.
Could you please add a note to this document about Rate-limited sampling in Java?

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

ID: 2f4c906d-7abe-eab7-331a-120230344583
Version Independent ID: ab24ade5-5e4c-b430-c3f8-2a00073e9e81
Content: Telemetry sampling in Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/sampling.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@jun110
Thanks for your feedback! We will investigate and update as appropriate.

@jun110 Thanks for your feedback.
Assigning to content author @AaronMaxwell   for review.

@AaronMaxwell that would be great. It is in fact on-by-default since the 3.4.0 release last year.


Regression Defect - Course Progress Report

Course Progress:

[ ] Choose course dropdown having more width

[ ] if select the last 30 days option from the time range, if does not have data then should be displayed as 'No data found' or 'No data available' instead of showing an empty screen

[ ] if any heat chart cell does not tooltip (white empty cells) does not show any tooltip earlier cQube

[ ]  Choose course dropdown having more width - Not Fixed

[ ] if select the last 30 days option from the time range, if does not have data then should be displayed as 'No data found' or 'No data available' instead of showing an empty screen - Resolved

[ ] if any heat chart cell does not tooltip (white empty cells) does not show any tool tip earlier cQube - Not Fixed

Retested defects

[ ] Choose course dropdown having more width - Resolved

[ ]  if any heat chart cell does not tooltip (white empty cells) does not show any tool tip earlier cQube - Resolved

This issue is fixed and closed


Kudos!

Hey man, amazing extension! I have been looking for it for a while, even thought to do it myself. Hahah. Anyways. Just wanted to praised your good work!
I just noticed that if there is an underscore signal in the property, it doesn't convert.

Hey @raphaklaus, thanks! I was really tired of dealing with JSONs so I had to do it myself 😆
About the issue, it relates with #1, definitely need a better regex for the atom map


cmake-js v2.0.0 print-configure does not work

Hi,
After update from cmake-js from v1.1.1 to v2.0.0, I'm getting the following output:
$ cmake-js print-configure
info CFG Applying CMake.js config from root package.json:
info CFG {"runtime":"node","runtimeVersion":"0.12.7","arch":"x64"}
undefined
Any idea?

I'll look into it tonight. Expect a fix arrive on tomorrow morning.

Thank you!

Fixed as of 2.0.2.


feat(code): add project details

Before this PR:
Generic Project Details
After this PR:
Project Materializes Tables as desired to BigQuery
Reason that prompted this change:
Learn DBT

FYI, Apologies for the PR / Branch. I updated origin to avoid this in the future


[FEATURE] Draggable Columns

@HenryVonfire i've decided to tackle this on! I got mostly everything working (a bit of code taken from your PR) but it feels kinda laggy / slow. Plus, the column width change while dragging a column over.... if you can help me out with that, we can ship this feature out soon!
@taras have you had any experience with ember-sortable? It feels a bit off tbh. Maybe it might be more optimal to do it via dnd-helpers...

[x] Work with simple headers
[x] Work with grouped headers
[ ] Work with footer
[ ] Might need to rework how column groups and sub columns are rendered to get footer logic working
[ ] Fix laggy feel when dragging a column

Looking forward to it!

As far as I know, it's like it adds the width of a column when you start to drag a column (i.e. if you have 3 columns it places the first and the second correctly but the third in the place of a fourth column when you drag one). I was looking at the update method in the sortable-group component but I haven't found the problem yet.
I also removed the CSS of ember-light-table to be sure it was because of it but the problem still persists.


[bug]: make fails to build at mser object

With the latest code from master branch if you run the make command it will fail with this error message:
make: ***  No rules to generate the objective «obj/Debug/mser.o», needed for «out_debug».  Stop.
This is because mser.cpp file doen't exist, if you remove $(OBJDIR_DEBUG)/mser.o from the Makefile it will compile.
This is correct?

I modified the Makefile. "obj/Debug/mser.o" is omitted.

I modified the Makefile. "obj/Debug/mser.o" is omitted.

Perfect! thank you!


create a function to print the key properties of a numpy array

create a function to print the key properties of numpy array are .ndim, .shape, .size, .dtype

defining a function to print the array properties.
def array_key_properties(array):
print(f" number of dimensions in an array (axes). :-- {array.ndim}")
print(f" size of the array in each dimensions. :-- {array.shape}")
print(f" total number of elements in the array. :-- {array.size}")
print(f" data type of the element in the array. :-- {array.dtype}")

The file is uploaded in the Numpy Array folder.


fix: concurrent modification error in update method

Contains
Fixes #2469
How to test
Make a deer fall off a cliff
Outstanding before merging
Should check for null pointers in other areas of the code, I didn't notice any while testing.

Had to edit the two empty lines that I forgot to delete, that's why I closed and reopened this PR.


Small spelling update in code sample

updating app-center to appcenter

@glennwester, reviewed?

@jwargo The fix is correct, compare https://github.com/Microsoft/appcenter-cli#commands


feat: add support for new surface operations client

add a new method createNewSurfaceOperationsClient (private) while we migrate the previous classes over.
This is because we accidentally shipped GA "new surface" clients which use the previous surface's operations clients. We don't want to break GA clients, so we can only change the client classes for clients which are NEW_SURFACE_ONLY (e.g. they do not contain legacy clients).

I think this direction is not a good idea, and it inflates an already-inflated GapicClientTrait.
Better to generate the logic directly in the GAPIC client.

Closing in favor of https://github.com/googleapis/gapic-generator-php/pull/714


Plugin can break valid queries by converting text inside quotes too

If the double quote is inside the value you are querying against, like this:
SELECT *
FROM table_name
WHERE table_name.some_value = 'te"s"t';

Then the plugin will break the query by converting it to:
SELECT *
FROM table_name
WHERE table_name.some_key = 'te's't';

So either the values inside quotes should be ignored (which is a preferred and easier approach), or escaped properly.

Thanks for reporting the issue @slaFFik
@Umtcnlbn can you take a look at this?


(not an issue) how to extend arrow?

Arrow is fantastic, I do not use anything else for date/time manipulations.
I now would like to expand it a bit, by adding a method for the Arrow object. I saw there was a previous discussion about a similar subject, and I tried a suggestion from there:
import arrow
import arrow.arrow

class MyArrow(arrow.arrow.Arrow):
    def __init__(self, year, month, day, hour=0, minute=0, second=0, microsecond=0, tzinfo=None):
        super().__init__(year, month, day, hour=0, minute=0, second=0, microsecond=0, tzinfo=None)

    def nextday(self):
        print("tomorrow is {day}".format(day=self.replace(days=+1).day))

f = arrow.factory(MyArrow)
print(f.now())
print(f.now().nextday())

It does not work as arrow.factory() is not implemented.
Traceback (most recent call last):
  File "C:/Users/yop/.PyCharm50/config/scratches/scratch_38", line 10, in <module>
    f = arrow.factory(MyArrow)
TypeError: 'module' object is not callable

Does that mean that the idea was implemented in another way?
I am trying to wrap my head around the ArrowFactory and Arrow classes. My naive understanding is that calling ArrowFactory().one_of_the_methods() will return an Arrow object. In that case I do not understand how to solve the problem of

creating my own Arrowclass, with the relevant method
creating my own ArrowFactoryè class which inherits Arrow` (from the docs)
and have the capacity to use get(), now(), etc. which are methods of ArrowFactory

The TypeError is trying to tell that arrow.factory is an module so you can't call it directly. But apparently it failed haha.
import arrow
import arrow.arrow

class MyArrow(arrow.arrow.Arrow):
    def __init__(self, year, month, day, hour=0, minute=0, second=0, microsecond=0, tzinfo=None):
        super().__init__(year, month, day, hour=0, minute=0, second=0, microsecond=0, tzinfo=None)

    def nextday(self):
        print("tomorrow is {day}".format(day=self.replace(days=+1).day))

f = arrow.factory.ArrowFactory(MyArrow)
print(f.now())
print(f.now().nextday())

This will work.

@philiptzou thank you for the help and for the fully working example. It made my day.
The condescending comment  was not useful, I mentioned the error in the context of the linked example but you may have missed that obvious part.

@wsw70 If my words make you feel condescending I'm really sorry about that. My intention is to complain the ambiguous error message that comes with TypeError means nothing and no one can understand it. Unless you have been f***ed by that message before.
But it will be another long story if we want to make Python changing the message.

No pb,  we're good :) And thanks for the working example.  I just now need to understand why arrow.now() correctly handles hours,  minutes and seconds, while f.now() does not,  setting them to 0. I know that these are the default values but why the day, month and year is correctly set is a mystery for me.

@wsw70 Perhaps you should not set "hour=0, minute=0, second=0, microsecond=0, tzinfo=None" in your constructor when calling super().__init__.


[stable/dex] Add tolerations, affinity and nodeSelector to grpc-certs/web-certs jobs

What this PR does / why we need it:

Add tolerations, affinity and nodeSelector to grpc-certs/web-certs jobs

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

@galindro thank you!
/approve
/lgtm

@vi7 I fixed the code. Can you review it again?

/lgtm


chore: add smoketest for Loop and RTL in actions

There are no new changes, but rebased to include the Fulcrum smoketest


Unstable test: Submodule_RecognizedWithCorrectVersion

The VersionOracleTests.Submodule_RecognizedWithCorrectVersion test frequently fails with this output:
    VersionOracleTests.Submodule_RecognizedWithCorrectVersion [FAIL]
      System.ComponentModel.Win32Exception : The system cannot find the file specified
      Stack Trace:
           at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
           at System.Diagnostics.Process.Start()
           at SevenZipNET.WProcess.Execute()
           at SevenZipNET.SevenZipExtractor.ExtractAll(String destination, Boolean overwrite, Boolean keepStructure)
        src\NerdBank.GitVersioning.Tests\TestUtilities.cs(70,0): at Nerdbank.GitVersioning.Tests.TestUtilities.ExtractRepoArchive(String repoArchiveName)
        src\NerdBank.GitVersioning.Tests\VersionOracleTests.cs(18,0): at VersionOracleTests.Submodule_RecognizedWithCorrectVersion()

It causes PR builds to fail frequently.

Seems like this can be closed now with your recent commit.

Closed by e1be2046c9680029c68b974c357e635e847a5dd8

It's still happening.


Prevent interpolation of gun location when ISM switches to actor

With slomo 0.1 and pm.SkipFindingTargets 1:
https://github.com/HaywireInteractive/OnAllFronts-Public/assets/108254625/bba9863d-3e1f-4dca-a8b7-bd16d0e5152a

DevNote: When frustum for Characters is fixed change it to lower value for easier replciation.

Blocked:
FPP


MAID-966

test creates a mesh of connection_managers sending message to each other.
note: the test fails as the NewConnection event is called two times the expected number. to enable simpler investigation of the problem, the test avoided creating many tasks in parallel during connect and send.
I extend the test to  a more parallel one once the issue above is resolved.

Coverage remained the same at 88.03% when pulling 30463392bb4784af26c8612af6fd2f7068eb8c71 on mmoadeli:master into 51ee987ad09f9f7f2281a0aca43e570c9ec49837 on dirvine:master.


not working in local ttest

i have download mobile angular UI. That sample not working in local

Are you trying to open it directly, or are you opening it in your local web server (localhost)?


wayland-scanner: fix typo

s/Intented/Intended
Signed-off-by: Michel Alexandre Salim<EMAIL_ADDRESS>

This change is

Thanks!


slightly clean up CircleCI config

We only need one branch of googleapis
There's no need to "install" toolkit; we only need the jar.

Codecov Report

Merging #2325 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #2325   +/-   ##
========================================
  Coverage      86.8%   86.8%
  Complexity     5168    5168
========================================
  Files           454     454
  Lines         20511   20511
  Branches       2209    2209
========================================
  Hits          17805   17805
  Misses         1932    1932
  Partials        774     774

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 4aa4a5d...1b91cf9. Read the comment docs.


Custom JsonConverter fails to deserialize object when using inner properties values

Source/destination types
      public class Parent
        {
            [JsonConverter(typeof(TwoPointsDecimalJsonConverter))]
            public decimal Total => Child1.total + Child2.total;

            public Child Child1 { get; set; }
            public Child Child2 { get; set; }

            public class Child
            {
                public decimal total { get; set; }
            }
        }

Source/destination JSON
{"Total":5.00,"Child1":{"total":2.0},"Child2":{"total":3.0}}

Expected behavior
Json to be serialized correctly. This works on version 10.0.3 but fails on 11.0.X. Also removing custom JsonConverter will work too.
Actual behavior
Newtonsoft.Json.JsonSerializationException: 'Error getting value from 'Total' on 'ConsoleApp3.Program+Parent'.'
NullReferenceException: Object reference not set to an instance of an object.
Steps to reproduce
  var parent = new Parent()
            {
                Child1 = new Parent.Child() {total = 2},
                Child2 = new Parent.Child() {total = 3}
            };

            var serializedObject = JsonConvert.SerializeObject(parent);
            var deserializedObject = JsonConvert.DeserializeObject<Parent>(serializedObject);

 class TwoPointsDecimalJsonConverter : JsonConverter
    {
        public override object ReadJson(
            JsonReader reader,
            Type objectType,
            object existingValue,
            JsonSerializer serializer)
        {
            throw new NotImplementedException();
        }

        public override bool CanConvert(Type objectType)
        {
            return (objectType == typeof(decimal) || objectType == typeof(decimal?));
        }

        public override bool CanRead => true;

        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
        {
            writer.WriteRawValue(((decimal)value).ToString("F2", CultureInfo.InvariantCulture));
        }
    }

I'm facing the same issue since moving from 10.0.3 to 11.0.X.
Here the work around when deserializing the object :
var deserializedObject = JsonConvert.DeserializeObject<Parent>(serializedObject, new JsonSerializerSettings() { ObjectCreationHandling = ObjectCreationHandling.Replace });

@TylerBrinkley CanRead was intentionally set to true along with NotImplementedException  to indicate that the NullReferenceException  i am getting is happening prior of the call to ReadJson

I see the issue. Since Child1 and Child2 are not included as parameters to the constructor they're null by default. And since Total doesn't handle when they're null it's throwing a NullReferenceException trying to get the child's total. When deserializing Json.NET tries to get the existingValue of the Total property your code throws the NullReferenceException.
It seems more like a problem with your code if you're allowing the Parent class to ever be in a state where it throws a NullReferenceException. I'd suggest you fix your code by either including the Child1 and Child2 properties as parameters to the constructor and make the properties read only or else make Total handle when Child1 or Child2 are null perhaps with the null propagator operator.


One Call API 3.0 Support

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Are there any plans to support the 3.0 API soon? For the time being I used patch-package to get it working
diff --git a/node_modules/openweather-api-node/dist/constants.js b/node_modules/openweather-api-node/dist/constants.js
index 045baa7..35f6651 100644
--- a/node_modules/openweather-api-node/dist/constants.js
+++ b/node_modules/openweather-api-node/dist/constants.js
@@ -55,7 +55,7 @@ exports.SUP_LANGS = [
 exports.SUP_UNITS = ["standard", "metric", "imperial"];
 exports.API_ENDPOINT = "https://api.openweathermap.org/";
 exports.GEO_PATH = "geo/1.0/";
-exports.DATA_PATH = "data/2.5";
+exports.DATA_PATH = "data/3.0";
 exports.WEATHER_PATH = exports.DATA_PATH + "/weather";
 exports.FORECAST_PATH = exports.DATA_PATH + "/forecast";
 exports.ONECALL_PATH = exports.DATA_PATH + "/onecall";

Hello!
I'm planning on making version 3.0 of the package (which will have support for the 3.0 API). Currently I don't have enough time but I will start working on it in a couple of weeks.

The<EMAIL_ADDRESS>is out and it has support for version 3.0 of the onecall api.


Bug in calculation of oauth_body_hash parameter. Fix is trivial

Summary
oauth_body_hash parameter is incorrectly calculated
Simplest Example to Reproduce
request({
    method: "POST",
    headers: [
      {
        name: "content-type",
        value: "application/xml"
      }
    ],
    url: "any_valid_url",
    body: "Hello World!",
    oauth: {
      body_hash: true,
      consumer_key: "jisc.ac.uk",
      consumer_secret: "secret",
      transport_method: "header",
    }
  }),

Expected Behavior
oauth_body_hash should be equal to "Lve95gjOVATpfV8EL5X4nxwjKHE"
Have a look at example here: https://tools.ietf.org/id/draft-eaton-oauth-bodyhash-00.html#anchor15
Or using an online tool: https://www.online-convert.com/result/df6e64e9-bc92-4ff3-b849-0c8f003c7e3c
Current Behavior
oauth_body_hash is calculated incorrectly. When using request-debug:
REQUEST make request http://ltiapps.net/test/tc-outcomes.php
{ request:
   { ...,
     headers:
      { ...,
        'content-length': 12,
        Authorization: 'OAuth oauth_body_hash="MmVmN2JkZTYwOGNlNTQwNGU5N2Q1ZjA0MmY5NWY4OWYxYzIzMjg3MQ%3D%3D",oauth_signature_method="HMAC-SHA1",..."' },
     body: 'Hello World!' } }

Possible Solution
PR is linked
Context
This issue fails my requests to IMS LTI Outcome service which requires POST requests and body hashes.
@simov FYI

@mikeal any ideas how is it possible to make some progress here? Thanks


Multiline entry summaries

This is a very extreme suggestion, but I thought it's worth mentioning to see what other people think.
I've been putting longer descriptions alongside each of my entries, and it occurred to me it'd be nice to be able to integrate the whole thing with markdown, such that the klog format is a subset of markdown.
Here's one way it could work: an entry is a bulleted list with a time-code, and the entry-description is anything which is attached to that bullet (including sub-bullets, etc.)
# 2021-03-22 Monday
- 07:00-08:30 #writing, wrote to Arash, Jon, Per.
- 08:30-10:30
   - Found Hall's marriage theorem, which says saxophone and diagonal representation of inequalities are equivalent, which allowed me to prove the diagonal lemma & so the theorem.
   - Was able to cut out a bunch of extra material from the paper.
      - Including cutting all the corollaries.

- 13:00-18:00 Read
   > "It is in the chemistry of these subtle substances, these curious precipitates and explosive gases which are suddenly formed by the contact of mind with mind, that James is unequalled"

# 2021-03-23 Tuesday
Terrible hangover.

- 14:00 coffee
- 14:00-13:00 read my letters, wrote to my mother.

I have been thinking about that as well already and also find myself using bullet points in the summaries. (Not indented, though.) I agree the summaries could allow for more flexibility, so it’s good that you bring this up so that it can be discussed further.
A few general upfront thoughts: I think there is a natural overlap between the three concerns of time tracking, todo’s and notes keeping. It’s a tricky balancing act to decide where to draw the line, and I’d like to keep klog focussed on time tracking primarily. What I like about the current summary style is that it feels “cheap” (read: simple to memorize and understand) with a quite straightforward (read: basically non-existent) syntax. I also don’t want to force users into a certain usage scheme but rather allow them to make up their own according to individual needs. There might be people who don’t use summaries at all, but who only want to enter the “naked” time data.
However, on the other hand, users shouldn’t feel unnecessarily limited by the summaries, and there are indeed a few shortcomings at the moment. I feel that making klog a subset of markdown probably goes a bit too far, but it should be possible to use something like markdown/todo.txt inside the summaries more easily.
Two things have been floating around in my head:
Multiline entry summaries
As you pointed out, the entry summaries (the text directly behind the time values) can only be one-liners. It could be possible, though, to have multi-line text as well, which could be achieved by expanding on the python-style indentation like so:
2020-01-01
    4h Was able to cut out a bunch of extra material from the paper
        including cutting all the corollaries
    1h
        - this
        - that

So you would basically be able to continue the text on the next line, with increased indentation level.
Allow indentation in record summaries
Currently there can’t be indentation in the record summaries (the one underneath the date), because indentation is reserved for the entries. There could be some sort of marker though, similar to blockquote syntax in markdown:
2020-01-01
| - Did this and that
|    - Includes this other thing
|    - And this one
    8:00 - ? Just started

You could do this right now already, but of course the | (or whatever character is used) would be part of the copy then. But the spec could reserve a special marker that gets stripped away by the parser. I’d find it important for this to be optional, in order to not loose the simplicity for use-cases where this kind of flexibility is not needed.

Happy to hear that -- and happy to give thoughts on any ideas you might have.
You mentioned in a previous post "I feel that making klog a subset of markdown probably goes a bit too far."  Could you expand a bit on what the drawbacks of that would be?
There would be many advantages (I think): you could use existing markdown highlighting, processors, & you'd naturally have all the features of markdown.
It certainly would break backwards-compatibility, and it would probably be require reasonable amount of work to implement, but are there drawbacks specific to the format?

I think building on top of markdown would change the notion of klog. Currently, klog is basically a data format that happens to be human-readable. If it was markdown, I think it would feel more like a journal that allows for time tracking additionally. That’s also what I meant earlier: I think there is a natural overlap between time tracking and journaling, but it’s a philosophical question where to put the focus on.
From a technical point of view it might be tricky to define the file format so that it isn’t ambiguous or hard to memorize, because with markdown you generally have the full freedom to structure the document as you please. So a klog-flavoured markdown would either a) have to be very restrictive and narrow down what is allowed and how it’s supposed to be structured. In this case I think you’d lose a lot of the benefits of using markdown, however. Or b) the rules would be broad and permissive, but that might lead to complications or ambiguities. There could be text in between sections, or other headlines, links, footnotes, etc. Consider the following example: which of the data would be understood? And would the rest be silently skipped, or would klog raise errors or warnings? (Because it might have been done on purpose, but it also could have been by accident).
# 2021-03-22 Monday
- 07:00-08:30 #writing, wrote to Arash, Jon, Per.
- [1h Tennis](https://my-tennis-club.com)

# Tuesday, 2021-03-23
1h Workout
8:00 - 9:00 Tennis again

## 2021-03-24 (Wednesday)
- I did the following things:
  - 09:00-10:00 Breakfast
  - 1h Workout

I found your initial example quite interesting, because I use klog completely differently. My summaries (if I write them at all) are rarely longer than 5 words, and I treat them more like a reminder than like a journal with detailed descriptions. So I perceive summaries basically as short annotations of the data for future reference. If I want to take more extensive notes about something, I usually do that separately. It’s interesting anyway to see how the habits and preferences are different.


network, Istio: Fix outbound test

What this PR does / why we need it:
The negative test checking that Envoy refuses to
connect to external service was missing .Should(Succeed())
so when we changed from using "kubevirt.io" as an external
service to another cluster vmi, the test was broken without
us knowing about it.
This commit properly registers the server vmi (there are two VMIs, server VMI without proxy
sidecar and the tested client VMI which does have the proxy sidecar) as a service by defining
VirtualService, destinationRule and Gateway.
The vmi HTTP server is now accessed via Istio ingressgateway, and when
Sidecar disallowing all external services is created, the Envoy proxy properly
refuses to forward the request.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/retest

/approve
Thanks!


Switch to pyfiglet

Description

Switch to an ASCII art tool (pyfiglet) that's already available as a conda package.
Resolves #3

Closing in favor of #11


show_info is not displayed in thread_worker functions

🐛 Bug

When using a thread_worker decorator to define a threaded function, the calls to show_info in that function are not routed to the
To Reproduce
Steps to reproduce the behavior:
Execute the following code from the napari IPython prompt

import napari
from napari_plugin_engine import napari_hook_implementation
from magicgui import magic_factory

from napari.qt import thread_worker
from napari.utils.notifications import show_info

show_info('Test from main')  # This is displayed in the activity window
import time

@thread_worker
def test_thread():
    print("started thread")
    show_info("in thread info")  # This prints "Notification" to stdout
    return 0

test_thread().start()

Expected behavior
The message in a show_info call in the separate thread should be displayed

Environment

Please copy and paste the information at napari info option in help menubar here:
napari: 0.4.12.dev51+g87f1ccac
Platform: Linux-5.13.0-7614-generic-x86_64-with-glibc2.33
System: Pop!_OS 21.04
Python: 3.9.7 | packaged by conda-forge | (default, Sep 14 2021, 01:17:55) [GCC 9.4.0]
Qt: 5.15.2
PyQt5: 5.15.4
NumPy: 1.21.2
SciPy: 1.7.1
Dask: 2021.09.0
VisPy: 0.8.1

OpenGL:

GL version: 4.6 (Compatibility Profile) Mesa 21.2.1
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1920x1080, scale 1.0
screen 2: resolution 1920x1080, scale 1.0

Plugins:

console: 0.0.4
napari-useg: 0.0.1
scikit-image: 0.4.12.dev51+g87f1ccac
svg: 0.1.5

Here is a script version1:
import time

from skimage import data
import napari
from napari.qt import thread_worker
from napari.utils.notifications import show_info

napari.view_image(data.astronaut(), rgb=True)
show_info("Test from main")  # This is displayed in the activity window

@thread_worker
def test_thread():
    print("started thread")
    time.sleep(1)
    show_info("in thread info")  # This prints "Notification" to stdout
    return 0

test_thread().start()

This segfaults on my machine -- I suspect due to newest Qt? -- but this is unrelated to the bug, it happens when I just try to run python -c "import napari, skimage; viewer = napari.view_image(skimage.data.astronaut(), rgb=True)"


Fix link

@yoichinak : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good (verified). Thanks for this fix @yoichinak.


feat: add support for injectScript

Noticed the option was missing, would be great to get this added!

Hey - any way this can be prioritized?

sure, here you go!

Whoops! One more quick change, @mbrevda to fix the tests and then we can merge.
Looks like we need to update the fixture here: https://github.com/WebPageTest/webpagetest-api/blob/5889c0d3ecdad071bfdde59b3c525548fe1acf38/test/fixtures/command-line/help-test.txt#L35
So that it uses <injectScript> instead of <script>

like that?

Perfect! Thanks @mbrevda!

Pleasure - eta on version bump and github action update to use this?

Any ETA on when this will be published?

ping - can we get this published please?

Done! Sorry it took a bit. We'll have to roll the other updates out in another release, but this way we get your updates out the door. Thanks for doing this!

Awesome, thanks! Can we please bump the github action to use it? https://github.com/WPO-Foundation/webpagetest-github-action/pull/9