Resolve with -lib in TypeScript lib lookup via node-style resolution

Fixes #45993
https://github.com/microsoft/TypeScript/pull/45771 introduced a way to write something like:
{
  "dependencies": {
    "@typescript/dom": "npm:@types/web"
  }
}

This PR changes that to be:
{
  "dependencies": {
    "@typescript/lib-dom": "npm:@types/web"
  }
}

This looks good, though it does make me consider, why do any transform on the lib file name at all? Package names can have dots, so there’s no reason our lookup name can’t be @typescript/lib.dom.d.ts, and then we wouldn’t have to do any of the name parsing.

Maybe leaving on the .d.ts would look confusing to some people (though it’s not actually ambiguous), but I’d be tempted to at least leave the dots instead of dashes. I don’t guess it really matters, though.

I think it's more about the form factor for how I think people would use it - for dom that means we can ship dom and dom.iterable correctly in a single package, instead of via 2 separate npm packages
The same would then apply for 'my runtime's version of es2018' which you can opt-in specific files.

Ah, I forgot about that 👍


Fix R CRAN failures.

Related: https://github.com/dmlc/xgboost/issues/7260#issuecomment-962985358

Remove hist builder dtor.
I did not reproduce the warning using clang-14, might be missing some flags in my config or it's just a bug in clang-12 used by CRAN test.  Anyway, this is a cleanup that removes unnecessary code.
Initialize some variables.

Thanks. May I suggest to hold this PR until we see all the checks for <IP_ADDRESS> on this page: https://cran.r-project.org/web/checks/check_results_xgboost.html. It might take a few days, and we have until Nov 22nd to patch.

There are additional issues with valgrind: https://www.stats.ox.ac.uk/pub/bdr/memtests/valgrind/xgboost/00check.log:

An R test failure: Failure (test_helpers.R:231:11): xgb-attribute numeric precision
Some leaks in the end of the check log.

I didn't remember CRAN ever complains about it, but if it is fixable on our side maybe we can also try?

An R test failure: Failure (test_helpers.R:231:11): xgb-attribute numeric precision

That's just testing the as.numeric in R.  XGBoost is not involved.  Will relax the test.  A longer-term solution might be using JSON instead.

at 0x1E1906EF: xgboost::data::SimpleDMatrix::GetGradientIndex(xgboost::BatchParam const&) (packages/tests-vg/xgboost/src/amalgamation/../src/data/simple_dmatrix.cc:94)

Probably the gpu_id, which is not used in that context.  Will fix.

0x1E21680E: xgboost::DMatrix* xgboost::DMatrix::Createxgboost::data::DenseAdapter(xgboost::data::DenseAdapter*, float, int, std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&) (packages/tests-vg/xgboost/src/amalgamation/../src/data/data.cc:828)

This should be released by R by calling XGDMatrixFree_R.  Not sure why it isn't called.

==2937178==    by 0x1E148116: XGDMatrixCreateFromCSC_R (packages/tests-vg/xgboost/src/xgboost_R.cc:137)

Leak in libgomp?  Not sure if there's anything we can do.

XGDMatrixSetInfo_R (packages/tests-vg/xgboost/src/xgboost_R.cc:203)

std::vector should manage the memory as RAII.  Not sure if there's anything we can do.

Ran valgrind with Rscript testthat.R.  No error or warning from valgrind.

@hcho3 Do you think we can free some budget for running valgrind with gtest on CI once we remove CUDA10?

@trivialfis We can just use GitHub Actions for this?

But then we will skip all GPU code path.

@trivialfis I don't think CRAN tests are run for any GPU code path

running valgrind with gtest

I want to run valgrind with c++ tests.  But there are some warnings whenever GPU code is executed, so we might just limit it to CPU only for now:
==304308== Warning: noted but unhandled ioctl 0x19 with no size/direction hints.
==304308==    This could cause spurious value errors to appear.
==304308==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==304308== Warning: set address range perms: large range [0x10006000000, 0x10106000000) (noaccess)
==304308== Warning: noted but unhandled ioctl 0x49 with no size/direction hints.
==304308==    This could cause spurious value errors to appear.
==304308==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==304308== Warning: noted but unhandled ioctl 0x21 with no size/direction hints.
==304308==    This could cause spurious value errors to appear.
==304308==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==304308== Warning: noted but unhandled ioctl 0x1b with no size/direction hints.
==304308==    This could cause spurious value errors to appear.
==304308==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.
==304308== Warning: noted but unhandled ioctl 0x44 with no size/direction hints.
==304308==    This could cause spurious value errors to appear.
==304308==    See README_MISSING_SYSCALL_OR_IOCTL for guidance on writing a proper wrapper.

I think this PR should contain all the fixes in XGBoost.

@hetong007 I looked into the 2 memory leak reports, first one is from libgomp, the second one is from the std::vector implementation in GCC.  I don't know how to fix them and so far haven't been able to reproduce them using my own setup.  Any chance we can continue the submission?

Got it.  Thanks for guiding the release.

I see most CRAN results are available with no new issue. I'm merging this PR.

Thanks, I will backport it and bump the version

Seems CRAN checks are all finished. Please instruct when to submit (before 22nd).

I will run through all the tests again today

@hetong007 Let me know if there's anything else I can do to help.

It is undergoing checks by the CRAN team. Will update here.

On CRAN now. From the status page I see the column with r-devel-linux-x86_64-fedora-clang has its previous WARN fixed. Appreciated!

Excellent!


v6 appears to be double unescaping regex provided to FindReplaceFilter

Regular expressions that are supplied to the FindReplaceFilter now appear to be double unescaping so that a regex supplied in config such as \[x is now treated as an unclosed character class, e.g. [x rather than an escaped square bracket etc.

This is as a result of #881 and is expected behaviour


createWebHistory with relative path

What problem does this feature solve?
I have a tricky question and a working example is difficult as you also need a reverse proxy… but I hope you get my point when you try understanding following setup (vue3 with vue-router4)
Configure vue app with  in index.html
The router is configured with createWebHistory()
The App is served from a subpath like example.com/my-app/
I want to have relative base url as I only want to build one image and use it on different paths.
What is expected:
The base routing path is appended to the current path, so a about page on /about looks like example.com/my-app/about
What is happening:
The route prefix “my-app” is deleted and navigation is internally working but the urls is wrong as it is example.com/about
When i configure my base with createWebHistory('/my-app/) i loose the possiblity to deploy my container on different context-paths with only one container. I would always have to rebuild the container what I want to avoid.
What does the proposed API look like?
createWebHistory works with relative path
Alternative would be a inspection of the base-href

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.

I posted to the forum and no applicable answer and I described that it's currently not possible - so why it's not a feature/bug request?

Alternative would be a inspection of the base-href

you mean like this?
createWebHistory(document.querySelector('base').href)

But I would define in the base href './' and not the application path itself. Possible I would need to inspect the location.pathname somehow?!

The webHistory can't work reliably on a relative path. That's just a technical fact. See this:

You have a route for / and one for /a,  and a relative base path.
You deploy this app in a subfolder called /a.
You directly visit the page in your browser by typing in yourdomain.com/a.

Should the router match the / or /a route? Without knowledge about the absolute base path, that's impossible for the router to determine.

I don't know how angular does ist, but there the router can work with a relative base path like <base href='./' />
When I define this relative base path <base href='./' /> in vue the router just deletes the subpath from the url...

I can't say I know Angular intimitely, but all examples of using <base href> in Angular use an absolute path. Likely for the same reason as I gave for VueRouter.

From my understanding the router should start from the index.html and from there take over all the routing path.
I have an angular application with <base href="./" /> up and running and it just works with one container and multiple deployments. That's why I'm searching this possibility now in vue.
But you could be right that are some limitations when the deployment path is somehow equivalent ot a route... Perhaps I can somehow build a demo with a dev-server and included the reverse proxy to tryout some more thougts.

The issue is that a relative paths works as long as you start the app from that URL, and then navigate to other routes.
When you start at a nested route (i.e. by having clicked on a link to yourappp.com/sub-path/actual-route/1, then the relative path will mess up things, because now the base is yourappp.com/sub-path/actual-route/1 where you would want it to be yourappp.com/sub-path/.

Interesting explanation, thank you! I will try to verify that and come back to you, so we could improve the documentation about this?
Am 15. Dezember 2021 15:01:35 MEZ schrieb "Thorsten Lünborg" @.***>:

The issue is that a relative paths works as long as you start the app from that URL, and then navigate to other routes.
When you start at a nested route (i.e. by having clicked on a link to yourappp.com/sub-path/actual-route/1, then the relative path will mess up things, because now the base is yourappp.com/sub-path/actual-route/1 where you would want it to be yourappp.com/sub-path/.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/vuejs/vue-router-next/issues/1245#issuecomment-994819108

sure

@LinusBorg I verified your description - thanks again.
I found also a lot of discussions on stackoverflow.com where also a proposal was made to set the base path dynamically in the index html https://stackoverflow.com/a/39084260 same idea I posted here and I think that would work also with the vue Router but you must improve it's just a bit for development.


ARROW-8881: [Rust] Add large binary, string and list support

Similar to other implementations, this creates binary, string and list arrays with i64 offsets instead of i32. Behaviourally, everything's the same as the i32 counterparts, except for the larger array offsets.
I'll look at the relevant integration tests separately.
Code's a bit repetitive, but given that we currently don't have much review bandwidth; I'd rather not spend a lot of time on it now, as I want to try complete other integration-related tasks in time for 1.0.0.

I'll look at the relevant integration tests separately.

Separately as in another commit to this branch, or is there a JIRA already for enabling these integration tests?

I'll look at the relevant integration tests separately.

Separately as in another commit to this branch, or is there a JIRA already for enabling these integration tests? Do you expect that the tests would fail if you just unskipped them?

As a separate PR, was planning to use https://issues.apache.org/jira/browse/ARROW-8883. I'm unfortunately having PC issues, so I'm making slow progress with debugging the failures

@nevi-me Are we trying to get this in before the 1.0.0 release? if you are in a hurry we can quickly review this and merge. However IMO the code duplication can largely be avoided by having some base classes. We can do it in a follow up to replace the current code.

Yes @sunchao, I wanted to merge this as part of integration support.
I had tried finding a way of using a base, but when I struggled I opted for the repetitive way for now.
There's also Datatype::Decimal, but I'm struggling to understand the spec, so I don't think I'd be able to complete it in time for 1.0.0.


Google assistant climate mode fix

Description:
Various thermostat sensors have a set of supported modes that differ from the globally supported Google Assistant climate modes in letter case only ("Cool" vs "cool"). The difference in case would prevent the Google Assistant component from reporting that those modes are supported and would typically cause it to report the thermostat as off. This fix converts all modes to lower case.
Related issue (if applicable): fixes #10713
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
Lint passed. Flake passed. Pydocstyle passed.
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Since you convert all the mode names to lower case before handing them to google assistant, they will also come back in lower case in determine_service, and you will call the SERVICE_SET_OPERATION_MODE with the lower-cased version in line 256 - 258, right? That won't work with devices depending on getting the right capitalization of their mode constants, if I'm not mistaken?

Shouldn't we just standardize the operation list with constants? 🤔 Aren't you looking into this @tinloaf ?
The operation list should contain constants, not English words as that would prevent translation.

You're right that the lower-cased version is what will be sent to the
device. All I can say is that for the cases I've used, it works. The Google
API requires that the mode be lowercase, so unfortunately we can't use the
case of the device. Regardless, there really should be some sort of mapping
that would take care of the non-standard name cases.
On Tue, Nov 21, 2017 at 11:22 AM, Lukas Barth<EMAIL_ADDRESS>wrote:

Since you convert all the mode names to lower case before handing them to
google assistant, they will also come back in lower case in
determine_service, and you will call the SERVICE_SET_OPERATION_MODE with
the lower-cased version in line 256 - 258, right? That won't work with
devices depending on getting the right capitalization of their mode
constants, if I'm not mistaken?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/10726#issuecomment-346079861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANovFTIzDuzlB25ZBur6sMYN2l0739T3ks5s4vjHgaJpZM4Ql7d8
.

Yep, I'm currently trying to get a plan for this @balloob . I've tried to get some input over in https://github.com/home-assistant/home-assistant.github.io/pull/3910 and will probably start the refactoring soon.
Basically, "operation mode" will probably be replaced by "control" and "action", with each having a small set of possible values.

Let's merge this for now. When we migrate our climate platforms over to use standardized operation modes, we will remove this code again.


[FEATURE REQ] Building Custom Models with Multiple pages

Library name and version
Azure.AI.FormRecognizer
Query/Question
I'd like to build a custom model which should be able to process multiple pages in a document.
To better illustrate,  when a person needs to submit his pay stubs for say 3-6 months, he could very well create a pdf document with paystubs for 6 months placed in it.
Is it achievable using Form Recognizer? I understand it may not be available right now, but is it in plans for the future releases? If so, when can I expect this feature?
Environment
Dot Net 5.x, Windows Server 2019 and VS - 2022

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@ctstone Could you help answering this question about the service's capabilities?

Ping @ctstone

@ctstone @anrothMSFT @vkurpad Could you take a look at this question?

Hi! I've joined a new team. I've pointed the team to this thread on an offline mail.

Hi @castasint, currently the model will only process a single page within a document that matches a template. The feature you need is to split a file into multiple documents and then process each of the documents. This is in our roadmap and you should expect to see it by the end of the year.

This feature is highly important to my team, we've had to develop around it. Is it possible there is a roadmap regarding this and other features?

@Stephen-Zealous As @vkurpad mentioned, the requested feature is in the service's roadmap and it's expected to be available by the end of the year. I'm leaving this issue closed since the question has been answered, but if you feel you have more questions, feel free to open a new issue.


internal/singleflight: flaky TestDoDupSuppress?

See http://build.golang.org/log/683c92f5ad9b397c36b9e6bcd3d457e70a2eca59.
The singleflight.Group.Do didn't return?
panic: test timed out after 2m0s

goroutine 33 [running]:
testing.startAlarm.func1()
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/testing/testing.go:702 +0x132
created by time.goFunc
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/time/sleep.go:129 +0x3a

goroutine 1 [chan receive]:
testing.RunTests(0x5d7908, 0x6606c0, 0x3, 0x3, 0x1)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/testing/testing.go:561 +0x8ad
testing.(*M).Run(0xc82003fef8, 0xc82000e640)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/testing/testing.go:493 +0x70
main.main()
	internal/singleflight/_test/_testmain.go:58 +0x116

goroutine 8 [semacquire]:
sync.runtime_Semacquire(0xc82000e7ac)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/runtime/sema.go:43 +0x26
sync.(*WaitGroup).Wait(0xc82000e7a0)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/sync/waitgroup.go:126 +0xb4
internal/singleflight.TestDoDupSuppress(0xc8200101b0)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/internal/singleflight/singleflight_test.go:69 +0x1c6
testing.tRunner(0xc8200101b0, 0x6606f0)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/testing/testing.go:455 +0x98
created by testing.RunTests
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/testing/testing.go:560 +0x86d

goroutine 10 [chan receive]:
internal/singleflight.TestDoDupSuppress.func1(0x0, 0x0, 0x0, 0x0)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/internal/singleflight/singleflight_test.go:49 +0x7d
internal/singleflight.(*Group).doCall(0xc82000e780, 0xc8200bc000, 0x58b150, 0x3, 0xc820016060)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/internal/singleflight/singleflight.go:93 +0x2c
internal/singleflight.(*Group).Do(0xc82000e780, 0x58b150, 0x3, 0xc820016060, 0x0, 0x0, 0x0, 0x0, 0x0)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/internal/singleflight/singleflight.go:63 +0x284
internal/singleflight.TestDoDupSuppress.func2(0xc82000e780, 0xc820016060, 0xc8200101b0, 0xc82000e7a0)
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/internal/singleflight/singleflight_test.go:57 +0x81
created by internal/singleflight.TestDoDupSuppress
	/tmp/gobuilder/dragonfly-amd64-69f0d4c6bebc/go/src/internal/singleflight/singleflight_test.go:65 +0x156
FAIL	internal/singleflight	120.020s

I can't reproduce this (tried many runs, and varying timing, GOMAXPROCS), the code looks fine, and this has been testing for years inside Google's build system for dl.google.com.  I'm inclined to blame dragonfly somehow, even though that's lazy of me.
Hey, @dvyukov, have you ever reviewed this code? Maybe it is buggy.

@bradfitz I guess one of the goroutines in test did not block in 100ms.
It's easy to reproduce if you comment out the sleep.
The following sequence does not do what it is meant to do. If calls > 1, then wg.Wait won't return.
	wg.Wait()
	if got := atomic.LoadInt32(&calls); got != 1 {
		t.Errorf("number of calls = %d; want 1", got)
	}


Fixing #6123

When gym names have unicode names, bot crashes when using GymPokemon task. This fix makes the eventmanager more robust.

👍

I'm afraid when arriving at gyms with unicode names, the bot still crashes
I changed this

self.formatted = str(formatted).encode('utf-8', 'ignore').decode('utf-8')
self.formatted = str(formatted).encode('ascii', 'xmlcharrefreplace')

into this

    try:
        self.formatted = str(formatted).encode('utf-8', 'ignore').decode('utf-8')
    except UnicodeEncodeError:
        self.formatted = str(unicode(formatted).encode('unicode_escape'))

After this change, no more crashing. Does this change make sense?

@ChiefM3  thanks for feedback. I understand issue now. Will fix the fix now :). Solution is almost similair like yours but a little different.
    #Fixing issue 6123 for gym names that are in utf format.
    try:
        self.formatted = str(formatted).encode('ascii', 'xmlcharrefreplace')
    except UnicodeEncodeError:
        self.formatted = str(unicode(formatted).encode('unicode_escape'))

Pull requesting now.


add title component

I think this will need the ability to inject HTML into the title, or something more fancy like the ability to parse some formatting within the component, right? Some of our example preprints utilise this already, but I'm pretty certain it's a base level expectation that the title can involve some formatting.

this is doable but we probably want to come up with a list of allowed html elements and decide what shape the data passed to the component will look like.

It would be amazing if we could apply filters to parameters. Here is a library that we could configure to allow only the permitted html into the template: https://www.npmjs.com/package/sanitize-html

It would be amazing if we could apply filters to parameters. Here is a library that we could configure to allow only the permitted html into the template: https://www.npmjs.com/package/sanitize-html

It is considered bad practice to just put raw html into a component and would probably be better to construct it from what we parse out of the xml than to render it as html and and put it in that way (even if its sanitised).

I was imagining something similar to this on the server - a function that transforms a format with encoded and limited text transformations to HTML equivalent. I'm not necessarily suggesting that we use the encoda-formatted JSON as there (though it is ready, and type-defined), but a similar structure with similar purpose that can encoded enough of the JATS standard formatting we'd expect would work also. This transform function could then be used by different react components for rendering the limited-format content we feed it across different parts of the UI, but taking just "text" for others where we don't expect or want formatting.
This would then need to feed back into the server code that transforms the XML to store in the DB so that we only transform and store the format-encoded output, and we transmit that to the client via the API.
From a chat with @will-byrne this is similar to what he was thinking - but we both think it's worth further discussion and prototyping together to understand the problem space better.

It would be amazing if we could apply filters to parameters. Here is a library that we could configure to allow only the permitted html into the template: https://www.npmjs.com/package/sanitize-html

It is considered bad practice to just put raw html into a component and would probably be better to construct it from what we parse out of the xml than to render it as html and and put it in that way (even if its sanitised).

That would be a server app concern presumably.


Consider baseline % for burstable instances

Amazon EC2 burstable performance instances provide a per type baseline CPU utilization and, are able to go beyond that threshold based on credit availability.
Given this specific behavior, special consideration must be taken when processing their metrics.
Referenced family types: t2, t3, t3a and t4g.
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/burstable-performance-instances.html

Possible solution could consider assuming that full CPU Utilization of these instances types is when they are at (or over) their specified baseline performance


meta tags in blog posts

Is it possible to add meta tags to a blog post generated html? I'd like to add twitter meta tags so my post links generate a preview.
Thank you!

You can add arbitrary HTML to your posts, but if you close the HTML head in your header.html (which I would expect), these tags will  show up in the body, which to my knowledge violates the HTML spec.
So I don't think there is a ready made way to to this, but you could modify the write_page function to do what you want. I imagine something like

filter lines that match <meta.*> from your blog post
replace a META_TAGS_HERE marker in your header.html with the filtered lines

That is what I figured out, hoping I could have missed something to make it work.
Thank you for your answer, I will try and hack something.


Order/filter configurable presets

Result Set Order/Filter settings should persist or one should be able to save the current version as a default preset
use case 1:
I always want all tables sorted by id DESC on all tables I open to see the newest rows first - even though I have not explicitly saved this behaviour through the 'Result Set Order/Filter settings' window, but rather in some general settings for that Database.
use case 2:
I always want some_random_nonstandard_column to be sorted DESC when I open table X. this will override the column(s) specified in use case 1.

New action will be added in results viewer toolbar:

It saves current filters state (criteria and ordering). This state will be restored on next table open.

@serge-rider I found and used this feature today intuitively without remembering it was my feature request. You are truly a master of great UX!
[offtopic]
Could you recommend any good books/podcasts/learning resources, so I could become an UX god too? :)
[/offtopic]

@serge-rider I tried with different SGBDs, databases, SOs, but the button always is disable for me.

This works only for table data editor (not for custom SQL queries)

Ah... OK.

Maybe this is a subject for a separate feature request. Although it won't be a bit more complicated (because it is not so easy to identify custom SQL queries)


Blazor client class Program not public

the "Program" class created by new Blazor project (not hosted) not set as public on VS 15.7preview2 - Blazor 0.1.0 ... I can't find other issue for this, I hope is not a duplicate.

@danroth27 @SteveSandersonMS Solved with PR #574 if and when is merged

@danroth27 I just checked, and the default new console app also doesn't use the public modifier on either the Program class or its static Main method. So to be consistent with that, don't you think we should instead remove the public modifier from the "Hosted" template?

Oh wait, I remember why it’s there. In the hosted template, it has to be public so the server middleware can reference it. Yes let’s update the standalone in case people try to add it as a reference from a server project.

Yes, this is my case, I've used standalone project to an existing webapi project


[REF-3243] Recharts allow_decimals does not work as described

Describe the bug
The documentation for Recharts XAxis allow_decimals parameter says:

Allow the ticks of XAxis to be decimals or not.

Naively I would expect that when set to False, then values for tick-marks shown on the X-Axis are not shown with full decimal representation and instead are truncated to integers in some manner, however, this is not the case.
To Reproduce
Minimal example:
"""Welcome to Reflex! This file outlines the steps to create a basic app."""

import numpy as np

import reflex as rx
from rxconfig import config

x = np.linspace(-20, 20, num=500)
y = x**2
data = []
for xd, yd in zip(x, y):
    data.append({"x": xd, "y": yd})

def index() -> rx.Component:
    return rx.recharts.line_chart(
        rx.recharts.line(
            data_key="y",
            stroke="#8884d8"
        ),
        rx.recharts.x_axis(data_key="x", allow_decimals=False),
        rx.recharts.y_axis(),
        data=data,
    )

app = rx.App()
app.add_page(index)

Expected behavior
I expect that the data shown on the XAxis will not display the full decimal and rather be truncated in some way to integers
Screenshots
Here is the plot shown when using allow_decimals=False. Note, the X-Axis is rather busy and hard to read due to the extended decimal data being shown.

Specifics (please complete the following information):

Python Version: 3.10.12
Reflex Version: 0.55
OS: Kubuntu 22.04
Browser (Optional): Chrome (latest for this Kubuntu release)

REF-3243

Looks like the current implementation aligns with the documentation. By default, the type property of XAxis is considered category, rather than a numeric value. Consequently, the graph was interpreting the data as a string. Specifying the type as number resolves this issue, and the graph behaves as expected.
"""Welcome to Reflex! This file outlines the steps to create a basic app."""

import numpy as np
import reflex as rx
from rxconfig import config

# Generate x values from -20 to 20 with 500 points
x = np.linspace(-20, 20, num=500)
# Compute the y values as the square of x
y = x**2

# Prepare the data in the required format
data = []
for xd, yd in zip(x, y):
    data.append({"x": xd, "y": yd})

def index() -> rx.Component:
    return rx.recharts.line_chart(
        rx.recharts.line(
            data_key="y",
            stroke="#8884d8"
        ),
        rx.recharts.x_axis(data_key="x", allow_decimals=False, type_="number"),
        rx.recharts.y_axis(),
        data=data,
    )

# Create and configure the app
app = rx.App()
app.add_page(index)

I hope this clarifies the situation and helps with your implementation.


Version control logo

Problem
Some people use source tree and other version control systems that can pick up logos! Let's supply one!
Solution
Add logo to base directory.
Screenshots

Paired with
Nobody.

TIL! cool


How to remove the Inherited class

Hi Team,
I would like to removed Inherited class from my command object. After i updated to latest version its broken
public class RemoveDefinitionsDocumentFilter : IDocumentFilter
{
public void Apply(SwaggerDocument swaggerDoc, DocumentFilterContext context)
{
swaggerDoc.Definitions.Remove("IResult");
swaggerDoc.Definitions.Remove("ILoggedOnUserProvider");
swaggerDoc.Definitions.Remove("Command");
}
}
New Version, I cant use SwaggerDocument swaggerDoc,

So the new Apply function signature is: public void Apply(OpenApiDocument swaggerDoc, DocumentFilterContext context)
I can see it doesn't contain a definitions member, so you should probably dig into it's structure deeper. I would suggest placing a breakpoint and then inspecting the object in the debugger.
Additionally, I think you want to hide some schemas, the DocumentFilterContext contains the schema repository, which contains a dictionary called Schema's, I think you need to remove them from there.
Seems like it's something you can try, don't know if it will work though

As of 5.0.0, the object model has changed to reflect Swagger/OpenAPi v3 as opposed to v2. This is described in the "Breaking Changes" of the release notes:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/releases/tag/v5.0.0

As of 5.0.0, the object model has changed to reflect Swagger/OpenAPi v3 as opposed to v2. This is described in the "Breaking Changes" of the release notes:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/releases/tag/v5.0.0


Initial implementation based on dpl

LGTM

@tboerger looks like it didn't grab the right logo file ... try http://svgporn.com/logos/rubygems.svg

other than that, LGTM

LGTM

@Bugagazavr verified that it's working, merging it now.


Maintained?

Does it still work? Any plans to maintain it any further? Is there any documentation? :kissing_heart:

Nope to all of those questions, sorry 😕


Deploy website

Deploy website version based on v0.10.0-rc

Maybe @ylemkimon could write a document about the release process. 😄
I just took a quick look at a checkout of this, and the page seems good. (In particular, the CDN is working.)

The website looks great! 🎉

Sorry for late responses. I'm not sure it's a GitHub issue or my ISP issue, but GitHub is currently very slow.

I wonder if there's some way to tell circleci and netlify to ignore PRs against gh-pages.

There seems to be no way for Netlify to ignore it. For CircleCI, we can use dummy config or it seems that master branch configuration can be used to control it.

What command did you run to generate this PR?

I ran ./release.sh -p 0.10.0-rc, but as gh-pages is a protected branch (#1412), I had to push to another branch and open a PR.

Maybe @ylemkimon could write a document about the release process.

Really I should 😄

Now that we're reviewing the documentation changes in PRs to master we probably don't need gh-pages to be a protected branch anymore.

gh-pages is no longer a protected branch.

@kevinbarabash I'm worried that someone may gain access to an account with write access and push malicious code.

Good point.  gh-pages is now a protected branch.


Extension.value is a DataType, and using it will raise errors.

Caveat: I haven't fully researched this one, but it seems to me that choice types are again the culprit here:
  Patient.ethnicity E
    return Tuple {
      codes: { E.ombCategory } union E.detailed,
      display: E.text
    }

In this fragment from CMS SupplementalDataElements.cql, we see E.ethnicity and E.ombCategory, which are aliases for an extension defined in QICore.  The value of an Extension (in our POCO model) is DataType, and somewhere along the line, when these extension values are used, we are getting an invalid cast from DataType to string (for display).
I have for now added a new cast that will cast a DataType to type string when a string is expected. This might actually work, since the CQL->ELM compiler has already checked that the assignment is valid, so we might as well assume it is.

In general, the FHIR TypeConverter is the place to know that DataType actually means: a choice type from the underlying model. So it is its responsibility to try to convert (with as) a DataType to one of its subclasses (or return null otherwise).


dssp error when run the example

Hi,
I've installed all the packages in requirement.txt. When I run the examples by this command:
python  run_GeoPacker.py --inputPATH example/ -i 3MPC_A.pdb --pdbname 3MPC  --chainID A
the dssp error appeared as below:
Traceback (most recent call last):
File "/MYPATH/GeoPacker/run_GeoPacker.py", line 85, in
evaluate(feature_path, pdbname, chainID,seq_tobe_designed,seqname,packered_outPATH,outputfile)
File "/MYPATH/GeoPacker/run_GeoPacker.py", line 30, in evaluate
AA_label, data = load_Data(feature_path, inputfile,pdbname, chainID ,seq_tobe_designed)
File "/MYPATH/GeoPacker/MainScripts/utils.py", line 215, in load_Data
geometric_f, distance = pre_fea(feature_path,inputfile,pdbID,chainID)
File "/MYPATH/GeoPacker/MainScripts/utils.py", line 273, in pre_fea
t1_r = t1.result()
File "/root/mambaforge/lib/python3.9/concurrent/futures/_base.py", line 446, in result
return self.__get_result()
File "/root/mambaforge/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result
raise self._exception
Exception: DSSP failed to produce an output
My mkdssp version is 4.1.0, installed by compiling the source code in https://github.com/PDB-REDO/dssp
I'm newer to this area and I have no idea what happened...

I noticed that you have installed the newer version ‘mkdssp’, however, the biopython uses the default executive command  ‘dssp’.
An easy way to check the aforementioned issue is to see which dssp or mkdssp you are using.
If the system fails to find dssp, but mkdssp, an alternative way for you is typing ‘cp /path/mkdssp  /path/dssp’, where the /path/ is the path including the executive command ‘mkdssp’.

I noticed that you have installed the newer version ‘mkdssp’, however, the biopython uses the default executive command ‘dssp’.
An easy way to check the aforementioned issue is to see which dssp or mkdssp you are using.
If the system fails to find dssp, but mkdssp, an alternative way for you is typing ‘cp /path/mkdssp /path/dssp’, where the /path/ is the path including the executive command ‘mkdssp’.

Hi,
I've installed all the packages in requirement.txt. When I run the examples by this command:
python run_GeoPacker.py --inputPATH example/ -i 3MPC_A.pdb --pdbname 3MPC --chainID A
the dssp error appeared as below:
Traceback (most recent call last): File "/MYPATH/GeoPacker/run_GeoPacker.py", line 85, in evaluate(feature_path, pdbname, chainID,seq_tobe_designed,seqname,packered_outPATH,outputfile) File "/MYPATH/GeoPacker/run_GeoPacker.py", line 30, in evaluate AA_label, data = load_Data(feature_path, inputfile,pdbname, chainID ,seq_tobe_designed) File "/MYPATH/GeoPacker/MainScripts/utils.py", line 215, in load_Data geometric_f, distance = pre_fea(feature_path,inputfile,pdbID,chainID) File "/MYPATH/GeoPacker/MainScripts/utils.py", line 273, in pre_fea t1_r = t1.result() File "/root/mambaforge/lib/python3.9/concurrent/futures/_base.py", line 446, in result return self.__get_result() File "/root/mambaforge/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result raise self._exception Exception: DSSP failed to produce an output
My mkdssp version is 4.1.0, installed by compiling the source code in https://github.com/PDB-REDO/dssp
I'm newer to this area and I have no idea what happened...

Hi,
I've installed all the packages in requirement.txt. When I run the examples by this command:
python run_GeoPacker.py --inputPATH example/ -i 3MPC_A.pdb --pdbname 3MPC --chainID A
the dssp error appeared as below:
Traceback (most recent call last): File "/MYPATH/GeoPacker/run_GeoPacker.py", line 85, in evaluate(feature_path, pdbname, chainID,seq_tobe_designed,seqname,packered_outPATH,outputfile) File "/MYPATH/GeoPacker/run_GeoPacker.py", line 30, in evaluate AA_label, data = load_Data(feature_path, inputfile,pdbname, chainID ,seq_tobe_designed) File "/MYPATH/GeoPacker/MainScripts/utils.py", line 215, in load_Data geometric_f, distance = pre_fea(feature_path,inputfile,pdbID,chainID) File "/MYPATH/GeoPacker/MainScripts/utils.py", line 273, in pre_fea t1_r = t1.result() File "/root/mambaforge/lib/python3.9/concurrent/futures/_base.py", line 446, in result return self.__get_result() File "/root/mambaforge/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result raise self._exception Exception: DSSP failed to produce an output
My mkdssp version is 4.1.0, installed by compiling the source code in https://github.com/PDB-REDO/dssp
I'm newer to this area and I have no idea what happened...

I do this command "cp /path/mkdssp /path/dssp" and now the system can find mkdssp and dssp both. But that error still appeared.
Now if I type "which dssp" or "which mkdssp", I will get a line “/usr/local/bin/dssp” or "/usr/local/bin/mkdssp"

I think the problem that biopython fails to find the executive command ‘dssp’ or ‘mkdssp’ is still existing. I recommend you an empirical way to install ‘dssp’ using ‘conda’ shown in the ‘requirement.txt’, and then ‘cp mkdssp dssp’(please add the path).  Before doing this, delete dssp4.0 executive command you installed.

I wish I could... Actually, I cannot install dssp using conda on my server, that is why I built the package by compiling the source code, which is more difficult. I think it may be because of the network conditions of the server.
Now I'm trying to build this project on another server in conda way, everything worked well, and this command "cp /path/mkdssp /path/dssp" is also runned. But when I run the example, that error keeps appearing...
Exception: DSSP failed to produce an output
Do I need to align all the packages in requirement.txt? Now I use tensorflow==2.6.0, numpy==1.19.5, pandas==1.1.5, three differences.

Please use an example to try whether the installed executive dssp or mkdssp is useful.
Try to add the absolute path  of the executive dssp or mkdssp to the DSSP module in line 188 of MainScripts/single_f.py.
eg. dssp = DSSP(model,inputfile,dssp='/usr/bin/dssp')

Please use an example to try whether the installed executive dssp or mkdssp is useful.
Try to add the absolute path of the executive dssp or mkdssp to the DSSP module in line 118 of MainScripts/single_f.py.
eg. dssp = DSSP(model,inputfile,dssp='/usr/bin/dssp')

Hi,
I used a simple case to test the dssp, and this error also occurred.
code
class Test_Dssp():
    def __init__(self):
        self.parser = Bio.PDB.PDBParser()

  def test_dssp(self, inputfile, pdbname):
      structure = self.parser.get_structure(pdbname,inputfile)
      for model in structure:
          dssp = DSSP(model,inputfile, dssp='/MYPATH/mambaforge/envs/Geopacker01/bin/mkdssp')
          print(type(dssp))

  if __name__ == "__main__":
      inputfile = '/MYPATH/GeoPacker/example/3MPC_A.pdb'
      pdbname = '3MPC'
      if 1:
          Test = Test_Dssp()
          Test.test_dssp(inputfile, pdbname)
      else:
          dssp_tuple = dssp_dict_from_pdb_file(inputfile)
          dssp_dict = dssp_tuple[0]
          print(dssp_dict['A',(' ', 1, ' ')])

error

/root/mambaforge/lib/python3.9/site-packages/Bio/PDB/DSSP.py:252: UserWarning: parse error at line 1: This file does not >seem to be an mmCIF file
warnings.warn(err)
Traceback (most recent call last):
File "/MYPATH/GeoPacker/my_scripts/test_dssp.py", line 26, in
Test.test_dssp(inputfile, pdbname)
File "/MYPATH/GeoPacker/my_scripts/test_dssp.py", line 18, in test_dssp
dssp = DSSP(model,inputfile)
File "/root/mambaforge/lib/python3.9/site-packages/Bio/PDB/DSSP.py", line 438, in init
dssp_dict, dssp_keys = dssp_dict_from_pdb_file(in_file, dssp)
File "/root/mambaforge/lib/python3.9/site-packages/Bio/PDB/DSSP.py", line 254, in dssp_dict_from_pdb_file
raise Exception("DSSP failed to produce an output")
Exception: DSSP failed to produce an output

It's weird that both ways of installing dssp got the same error, I think there must be some other problem.
I have also added this issue to dssp repo, hope someone can handle this...

I have tested on different python versions including 3.7 and 3.9, and everything goes well. Attached screenshot is based on your provided python version for test. I hope it can provide you with some help and reference.

Thanks for this information. It's helpful. I found that mkdssp worked well with orders like that mkdssp 6tiz.pdb, while it takes about 20 seconds, I don't know if it's normal. And I noticed that you are using an earlier version of mkdssp, I'm trying to install one, which maybe works well together with biopython.


<EMAIL_ADDRESS>hash check failed

virtualhere-client: 5.5.1 -> 5.5.2
Updating one outdated app:
Updating 'virtualhere-client' (5.5.1 -> 5.5.2)
Downloading new version
vhui64.exe (5.2 MB) [======================================================================================================] 100%
Checking hash of vhui64.exe ... ERROR Hash check failed!
App:         extras/virtualhere-client
URL:         https://www.virtualhere.com/sites/default/files/usbclient/vhui64.exe
First bytes: 4D 5A 78 00 01 00 00 00
Expected:    5ef86c76ac193d12a7d2c023f911007e5eadde48
Actual:      a4f2dabddb67f6e4c5cdbbcd854c9e13cd8d2316

https://github.com/ScoopInstaller/Extras/commit/8da169a9521f5e5bc0c10e48ab275e038d8229d0


Fix potential deadlock when creating a ServiceClient instance

By adding missing ConfigureAwait(false) calls

@0xced We cannot accept the merge but will pick this up and add this to our main repro.  We will close the pr when we push this fix.
thanks !

Have you considered turning this GitHub repository into your main repository? Pretty much all teams at Microsoft are now operating on GitHub and accepting pull request the GitHub way.

Fair question :)
Unfortunately this is not possible at this time as this client depends on non public assets from our underlying server infrastructure to build that are undergoing significant redesign and changes.
It is out goal to get there at some point as the dependencies reach the point where we can post them, using this repro for our primary source and build test publish on our internal ado systems.  We are still a ways off on it.  This challenge is primarily responsible for the delay in getting to a 1.0 for us.
Again, it is our goal to get there. We have taken the git as primary source with several of our newer tools that rely on this lib and will continue to do that going forward.


Comments in function invocation chain are removed

The "set headers" comment is being removed.
         // Create response
         let response = Response::with((status::Ok, body))
-            // Set headers
-            .set(Header(headers::ContentType("application/json; charset=utf-8".parse().unwrap())))
-            .set(Header(headers::CacheControl(vec![headers::CacheDirective::NoCache])))
-            .set(Header(headers::AccessControlAllowOrigin::Any));
+                           .set(Header(headers::ContentType("application/json; charset=utf-8"
+                                                                .parse()
+                                                                .unwrap())))
+                           .set(Header(headers::CacheControl(vec![headers::CacheDirective::NoCache])))
+                           .set(Header(headers::AccessControlAllowOrigin::Any));

Might be related to #406 and #410.

resolved: fixed


sslmode should be a type safe const

This will reduce the possibility to make failures in API usage and clarifies possible values.

Codecov Report

Merging #29 into master will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   81.53%   81.67%   +0.14%
==========================================
  Files           8        8
  Lines         509      513       +4
==========================================
+ Hits          415      419       +4
  Misses         50       50
  Partials       44       44

Impacted Files
Coverage Δ

postgres.go
83.33% <100.00%> (+8.33%)
:arrow_up:

testing.go
80.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


mui-rte 2.0.1 not compatible with versions after @mui/material 5.0.0 and onwards?

Checklist

[x] This is not covered in the repository examples.
[x] I am using the package latest version.
[x] This is a bug report or an enhancement proposal.

Description
I'm using @mui/material 5.1.0, and installed newest version of mui-rte (2.0.1)
Getting this error when trying to render the editor TypeError: theme.spacing is not a function. I am wrapping the editor to theme provider, but still no luck. But after changing @mui/material back to 5.0.0 it works. Is this a bug in newer versions of the package or what?

Ok it seems that I was importing ThemeProvider incorrectly. For me at least, it worked when I switched the import from
import { ThemeProvider } from '@mui/material/styles'
to
import { ThemeProvider } from '@mui/styles'
Docs. suggested to import it from @mui/material/styles which didn't work for me. Just curious if others have had the same problem.


Update for Lexer API Changes

EnterTokenStream now requires an additional argument, update the usages.
See also: git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@361007
91177308-0d34-0410-b5e6-96231b3b80d8
See also: https://github.com/apple/swift-clang/commit/a3c314617033ec86c5a4eac093e301b359c48c9d

@swift-ci please test

CC: @bob-wilson @DougGregor @dexonsmith

Tests are failing because swift/include/swift/Basic/LLVM.h needs fixing... working on a change for that too.

Duplicate of https://github.com/apple/swift-clang/commit/3044c4bc86175dab4d94aca2ed8815720c47f5fc


adds AioRedisHealthProvider

Adds new healthcheck provider with tests

Thanks, this is awesome (and apologize for the delay).
Checking.


Shadow system - 2 / 6

This is the second.
2/6 -> Add implementations for nearFar, allocation, directional, filter.
See https://github.com/libgdx/libgdx/pull/3347

Don't worry I'm used to this remark ;-)
But here it doesn't pertub the review so I think it's not important ! Indeed it's the eclipse formatter.

Thanks! I'll have to look at it in depth, but my first impression:
I don't think that restricting the implementation to ModelInstance is a good idea (use RenderableProvider instead). Also, it looks like the bounding box calculation/caching doesn't belong there. If we need to know the bounds, then I think it is time that we add this to the 3D api (which on itself has nothing to do with shadows, it can also be used for frustum culling, ray picking, etc.).
My first idea would be to add a few member to Renderable, like Renderable#center, Renderable#size/halfExtents and Renderable#radius. Ofcourse that also means that we need to keep track of that in the model/modelinstance. Then it might be best to move that kind of information to MeshPart (because that is what actually defines the shape), although ofcourse that is untransformed.
Currently Renderable doesn't use MeshPart, because MeshPart didn't exist when Renderable was created. I never changed that to avoid breaking the API. Perhaps this is a good reason to break the API and add a MeshPart member to Renderable. In that case we might not necessarily have to add (and manage) the center/size/radius members to renderable because it can be (cheaply) derived from the transform and meshpart members.

If we need to know the bounds, then I think it is time that we add this to the 3D api (which on itself has nothing to do with shadows, it can also be used for frustum culling, ray picking, etc.).

I like that! :+1:
Maybe we will even finally add some simple default frustum culling to ModelBatch? :D

I like that too! What about this pull request? Do I wait for your API change?

I've created PR #3483 for it. Let me know what you think about it. Because it's an api breaking change, I'll wait a few days with pulling, in case someone (or myself) finds anything that I overlooked or alike. If everything is fine, I'll pull it in, so you can update this PR as well.

Ok so I'm trying to use the new API. I am wondering how to manage renderable pool. Indeed, if AABBNearFarAnalyzer takes an array of RenderableProvider, I need to  provide a pool. It would be good to use the ModelBatch RenderablePool but it's protected. How would you do ?

In ModelCache I've practically copied over the code. Perhaps it is useful to add that class (FlushablePool) to the utils and make it part of the lib?
Besides that, I have some mixed feelings about this. In practice it means that the getRenderables method most of the time is called at least twice in the same render call. One for the shadows and once for the model batch. And while methods are relatively cheap, they typically involve traversing the node tree and multiplying the matrices.
We probably should "fix" that some time. Perhaps with a layer that caches the renderables. Some kind of system where you plug in things that do stuff with the renderables (sounds like a perfect javadoc description ;). However, one thing at a time, let's keep that in mind and call the getRenderables method twice for now.

Xoppa, I updated AABBNeaFarAnalyzer with the new meshPart API.
I would like to rename Sphere to BoundingSphere to be more consistent. Can I do it ?

Wouldn't it make more sense to rename BoundingBox to just Box? Rectangle isn't called BoundingRectangle as well.

Yes maybe

Thanks, looks good. Only one thing that I noticed: Can you explain why you have some values in the constructor? E.g.
public AABBNearFarAnalyzer (Array<RenderableProvider> renderableProviders)

This means that you either rely on the array (by reference) to be updated outside the class or that the caller recreates a new instance every time. The array itself is only used (through the getRenderables method) in the analyze method. So why not make it an argument of that method? (is there any implementation of the interface method thinkable that doesn't need the renderables?) Alternatively you could wrap it in a begin/end call, like so:
analyzer.begin(light. camera);
analyzer.add(instance);
analyzer.add(instances);
analyzer.end();

Note that you probably should not use Array<RenderableProvider> here, but instead Iterable<T extends RenderableProvider>.
BoundingSphereDirectionalAnalyzer has a similar constructor argument. Perhaps move it as argument for the analyze method or add a getter/setter for it. My guess (not sure though) is that every implementation of the DirectionalAnalyzer will need the "main" camera just as much as it needs the light.
Btw, having a cam far 10,000 times the cam near is not a good idea for precision. Better make it more like 1f and 100f (which is Camera's default).

Ow, it looks like you're never calling RenderablePool#flush

Xoppa, I updated everything according to your comments.
There is no constructor anymore for AABBNearFarAnalyzer and BoundingSphereDirectionalAnalyzer.
I added the call too renderablePool.flush()

Thanks!


Remove unused specs and pieces of code

What makes you believe the tests are "unused"?

@michaelklishin I had strong doubts about it, of course :)
I'm making an attempt to fix specs in master branch, but in fact I shouldn't have sent PR this early

@killthekitten the thing is API has changed, so the new way to do same thing is via execute:
(client/execute session
                (client/build-statement "SELECT * FROM users")
                :retry-policy (:downgrading-consistency policies/retry-policies)
                :consistency-level (:any policies/consistency-levels))

I'd leave the with-consistency-level test, just port them to the new API.
What was wrong with test-select-where-without-fetch?

@ifesdjeen in previous commit I've just misunderstood the real purpose of test-select-where-without-fetch being on it's place.
Does the fix looks ok now? I've also removed with-consistency-level macro, since I didn't find any other usages in the code.
And thank you for your help.

Ok, much better now :)


[release-4.16] OCPBUGS-36138: perform operator apiService certificate validity checks directly

Manual cherry-pick of https://github.com/openshift/operator-framework-olm/commit/56226d5893af4be48862a4d42375da09d90cc8a6 to 4.16

/jira refresh

/jira refresh

/jira refresh

/retest

Test pass, see: https://issues.redhat.com/browse/OCPBUGS-36138
/lgtm
/label qe-approved
/labe cherry-pick-approved

/label cherry-pick-approved

/retest
/label backport-risk-assessed

/retest

/cherry-pick release-4.15


restaurant/feature/add_cuisine

Change pull request from main to develop !

yoyo close pull request because u made it on main branch and change it to develop @aya-seif


Tensorflow DLL leaks references to itself, preventing the DLL from unloading

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes (see below).
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 Pro 64bit (version 2004)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A.
TensorFlow installed from (source or binary): Binary.
TensorFlow version (use command below): 2.3.1 (CPU only).
Python version: N/A.
Bazel version (if compiling from source): N/A.
GCC/Compiler version (if compiling from source): N/A.
CUDA/cuDNN version: N/A.
GPU model and memory: N/A.

Describe the current behavior
We link to the Tensorflow DLL dynamically (i.e., using LoadLibrary() directly on tensorflow.dll, or on another DLL that itself links to Tensorflow). We use the Tensorflow C API to run some data through a model. Eventually, when processing is complete, the DLL is unlinked (i.e., using FreeLibrary() directly on tensorflow.dll, or the other DLL that itself links to Tensorflow). For some reason unknown to us, at this stage the operating system (Windows) still believes the DLL is in use (the reference count is non-zero), thus any static resource allocated internally by Tensorflow is (probably?) not freed.
The code example below illustrates this problem.
If, within the same process, and under certain conditions, the procedure above is repeated a second time (link again to TF, run a model, etc.), we experience an access violation error in TF_SessionRun(). We are not able to produce a sample code that reproduces this last error, as it only occurs when our software is used by a third party application (the source code of which we do not have access to). However, we noticed that the error would no longer happen if we forcefully unlink the Tensorflow DLL by repeatedly calling FreeLibrary("tensorflow.dll") until the OS holds no reference to it (this requires some 30-40 calls to FreeLibrary()!). This implies that some internal state of Tensorflow from the first "link/process/unlink" run is still in use, but is somehow invalid.
Describe the expected behavior
A cycle of  LoadLibrary() / run TF / FreeLibrary() should leave the Tensorflow DLL reference count unchanged (i.e., equal to zero if TF is not used anywhere else in the program). On FreeLibrary(), any internal resource held by TF should be freed (provided TF is not in use by other parts of the program). Loading the DLL again should result in a blank new internal state of TF.
Standalone code to reproduce the issue
#include <iostream>
#include <exception>
#include <string>
#include <windows.h>
#include <tlhelp32.h>

// Helper function to count the number of references to tensorflow.dll
std::size_t GetTensorflowRefCount() {
    auto handle = GetCurrentProcessId();
    auto s = CreateToolhelp32Snapshot(TH32CS_SNAPMODULE, handle);

    std::size_t value = 0;
    MODULEENTRY32 me32;
    me32.dwSize = sizeof(MODULEENTRY32);
    Module32First(s, &me32);
    do {
        if (std::strcmp(me32.szModule, "tensorflow.dll") == 0) {
            value = me32.GlblcntUsage;
            break;
        }
    } while(Module32Next(s, &me32));

    CloseHandle(s);
    return value;
}

// Manually import Tensorflow some types and functions
typedef enum TF_Code {
  TF_OK = 0,
  TF_CANCELLED = 1,
  TF_UNKNOWN = 2,
  TF_INVALID_ARGUMENT = 3,
  TF_DEADLINE_EXCEEDED = 4,
  TF_NOT_FOUND = 5,
  TF_ALREADY_EXISTS = 6,
  TF_PERMISSION_DENIED = 7,
  TF_UNAUTHENTICATED = 16,
  TF_RESOURCE_EXHAUSTED = 8,
  TF_FAILED_PRECONDITION = 9,
  TF_ABORTED = 10,
  TF_OUT_OF_RANGE = 11,
  TF_UNIMPLEMENTED = 12,
  TF_INTERNAL = 13,
  TF_UNAVAILABLE = 14,
  TF_DATA_LOSS = 15,
} TF_Code;

using TF_Status = void;
using TF_Graph = void;
using TF_SessionOptions = void;
using TF_Session = void;

TF_Code            (*TF_GetCode)(TF_Status*);
TF_Status*         (*TF_NewStatus)();
TF_Graph*          (*TF_NewGraph)();
TF_SessionOptions* (*TF_NewSessionOptions)();
TF_Session*        (*TF_NewSession)(TF_Graph*, TF_SessionOptions*, TF_Status*);
void               (*TF_DeleteSession)(TF_Session*, TF_Status*);
void               (*TF_DeleteSessionOptions)(TF_SessionOptions*);
void               (*TF_DeleteGraph)(TF_Graph*);
void               (*TF_DeleteStatus)(TF_Status*);

void ImportTensorflow(HINSTANCE handle) {
#define LOAD_FUNCTION(Function) Function = reinterpret_cast<decltype(Function)>(GetProcAddress(handle, #Function))
    LOAD_FUNCTION(TF_GetCode);
    LOAD_FUNCTION(TF_NewStatus);
    LOAD_FUNCTION(TF_NewGraph);
    LOAD_FUNCTION(TF_NewSessionOptions);
    LOAD_FUNCTION(TF_NewSession);
    LOAD_FUNCTION(TF_DeleteSession);
    LOAD_FUNCTION(TF_DeleteSessionOptions);
    LOAD_FUNCTION(TF_DeleteGraph);
    LOAD_FUNCTION(TF_DeleteStatus);
#undef LOAD_FUNCTION
}

// Helper function to throw on error
void TFStatusCheck(TF_Status* status) {
    if (TF_GetCode(status) != TF_OK) {
        throw std::runtime_error("Error in tensorflow");
    }
}

int main() {
    // Load Tensorflow DLL and import functions
    std::cout << "initial: " << GetTensorflowRefCount() << std::endl;
    auto handle = LoadLibrary("tensorflow.dll");
    std::cout << "DLL loaded: " << GetTensorflowRefCount() << std::endl;
    ImportTensorflow(handle);
    std::cout << "functions imported: " << GetTensorflowRefCount() << std::endl;

    // Create a dummy session
    TF_Status* status = nullptr;
    TF_Graph* graph = nullptr;
    TF_SessionOptions* options = nullptr;
    TF_Session* session = nullptr;

    try
    {
        status = TF_NewStatus();
        std::cout << "TF_NewStatus: " << GetTensorflowRefCount() << std::endl;
        graph = TF_NewGraph();
        std::cout << "TF_NewGraph: " << GetTensorflowRefCount() << std::endl;
        options = TF_NewSessionOptions();
        std::cout << "TF_NewSessionOptions: " << GetTensorflowRefCount() << std::endl;
        session = TF_NewSession(graph, options, status);
        TFStatusCheck(status);
        std::cout << "TF_NewSession: " << GetTensorflowRefCount() << std::endl;
    }
    catch (const std::exception& e)
    {
        std::cout << e.what() << std::endl;
    }

    // Free allocated resources
    if (session) TF_DeleteSession(session, status);
    std::cout << "TF_DeleteSession: " << GetTensorflowRefCount() << std::endl;
    if (options) TF_DeleteSessionOptions(options);
    std::cout << "TF_DeleteSessionOptions: " << GetTensorflowRefCount() << std::endl;
    if (graph)   TF_DeleteGraph(graph);
    std::cout << "TF_DeleteGraph: " << GetTensorflowRefCount() << std::endl;
    if (status)  TF_DeleteStatus(status);
    std::cout << "TF_DeleteStatus: " << GetTensorflowRefCount() << std::endl;

    // Free DLL
    FreeLibrary(handle);
    std::cout << "DLL unloaded: " << GetTensorflowRefCount() << std::endl;

    return 0;
}

Other info / logs
Simply compile the above on Windows with clang++ test.cpp -o test.
Output on my computer:
initial: 0
DLL loaded: 1
functions imported: 1
TF_NewStatus: 1
TF_NewGraph: 2
TF_NewSessionOptions: 2
2020-10-26 13:38:48.388408: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  AVX2
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2020-10-26 13:38:48.398942: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x1ffcd6eeec0 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-10-26 13:38:48.399288: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
TF_NewSession: 41
TF_DeleteSession: 40
TF_DeleteSessionOptions: 40
TF_DeleteGraph: 40
TF_DeleteStatus: 40
DLL unloaded: 39

Notice the following:

TF_NewGraph generates a new reference to the DLL
TF_NewSession generates 39(!) new references to the DLL
TF_DeleteSession removes one reference
We are left with 39 leaked references, which we did not create explicitly in our program.
We were able to reproduce this with a number of TF versions (1.15, 2.1.1, 2.3.1), with or without GPU support.

This is intended behavior. Several internal shared objects in TF provide functionality while they are being loaded. This is achieved by static initializers and global data.
We do not support unloading TF shared libraries. Doing so would lead to a lot of bugs and sudden behavior changes.

I am not sure I understand this reasoning. Static initializers and global data are destroyed when a process is terminated, and this does not lead to bugs and sudden behavior changes, because this data cannot be used afterwards anyway. The same holds for DLL unloading. From the point of view of Tensorflow, unloading the DLL should have the same effect as terminating the process, static data should be deleted automatically on unload without you having to do anything. Loading the DLL back again should have the same effect as starting a new process.
See this simple DLL example (compile with clang++ -shared dll.cpp -o dll.dll):
#include <iostream>
#include <windows.h>

struct foo {
    foo() { std::cout << "construct " << this << std::endl; }
    ~foo() { std::cout << "destruct " << this << std::endl; }
    void baz() { std::cout << "use " << this << std::endl; }
};

extern "C" {
__declspec(dllexport) void bar() {
    static foo tmp;
    tmp.baz();
}
}

BOOL APIENTRY DllMain(HMODULE hModule, DWORD dwReasonForCall, LPVOID lpReserved)
{
    return TRUE;
}

... and a program that uses it (compile with clang++ test_dll.cpp -o test_dll.exe):
#include <iostream>
#include <string>
#include <windows.h>
#include <tlhelp32.h>

// Helper function to count the number of references to dll.dll
std::size_t GetDLLRefCount() {
    auto handle = GetCurrentProcessId();
    auto s = CreateToolhelp32Snapshot(TH32CS_SNAPMODULE, handle);

    std::size_t value = 0;
    MODULEENTRY32 me32;
    me32.dwSize = sizeof(MODULEENTRY32);
    Module32First(s, &me32);
    do {
        if (std::strcmp(me32.szModule, "dll.dll") == 0) {
            value = me32.GlblcntUsage;
            break;
        }
    } while(Module32Next(s, &me32));

    CloseHandle(s);
    return value;
}

void (*bar)();

void ImportDLL(HINSTANCE handle) {
#define LOAD_FUNCTION(Function) Function = reinterpret_cast<decltype(Function)>(GetProcAddress(handle, #Function))
    LOAD_FUNCTION(bar);
#undef LOAD_FUNCTION
}

int main() {
    // Load dll DLL and import funcitons
    std::cout << "initial: " << GetDLLRefCount() << std::endl;
    auto handle = LoadLibrary("dll.dll");
    std::cout << "DLL loaded: " << GetDLLRefCount() << std::endl;
    ImportDLL(handle);
    std::cout << "functions imported: " << GetDLLRefCount() << std::endl;

    // Use DLL
    bar();
    std::cout << "DLL used: " << GetDLLRefCount() << std::endl;

    // Free DLL
    FreeLibrary(handle);
    std::cout << "DLL unloaded: " << GetDLLRefCount() << std::endl;

    return 0;
}

This prints, on my machine:
initial: 0
DLL loaded: 1
functions imported: 1
construct 00007FFF3BBC41D0
use 00007FFF3BBC41D0
DLL used: 1
destruct 00007FFF3BBC41D0
DLL unloaded: 0

See how the destructor of the static variable bar::tmp is automatically called when the DLL is unloaded. This is the expected behavior, and allows me to re-load the DLL afterwards if I need to, with a brand new static state as if I just started the process.

FYI, I just got around to test an alternative library to Tensorflow, PyTorch, and it does not suffer from this problem. I can load/unload their DLL as in the simple example above. This suggests to me that Tensorflow is doing something special, possibly incorrect.

TensorFlow also takes into account the security aspect of the framework. Hence, it has a much safer design, that prevents scenarios which seem to work but later would be exploitable/cause hard to debug bugs.
If you don't care about these aspects and other frameworks allow you to use an unsafe design, by all means.

Thank you for your reply. Could you elaborate (or point me to a link/code) on that additional safety, and how this relates to not being able to unload the DLL? It might help us solve the problem on our end.
Ironically, it was precisely Tensorflow's inability to unload that has caused us a hard-to-debug bug. To this date we also have no correct solution for it either, because our own design relies on that ability for all our dependencies.

Imagine code taking a pointer to objects from the DLL and what happens when the DLL gets unloaded

That would be a programming error, or a breach of contract. Much like using a TF_Session* after it has been deleted by TF_DeleteSession(). In both cases the user is in control of when these pointers are used, and when they are no longer valid (when deleted explicitly, or when unloading the DLL), so any misuse is solely the user's fault.
This safety risk didn't prevent you from exposing the TF_DeleteSession() function (and other functions to delete resources), because resources should be freed when no longer used. The same goes with DLLs.

We no longer use Tensorflow, so this is no longer relevant to us. However, if anyone else stumbles upon this problem and is willing to do something about it, here is a pointer to a possible solution inspired by this blog article.
I just learned today that, on Windows, std::thread increases the refcount of the DLL it is created in, until the thread terminates (see here for the rationale; I guess that's a common thing people do in the Windows world, and is not limited to just std::thread?). I assume Tensorflow has some sort of global thread pool, but a) it does not handle DllMain events or any other resource management mechanism to automatically close the thread pool when no external reference to Tensorflow exists, and b) it offers no way to manually close the thread pool from the public C API either. Therefore, it is not possible to delete these thread objects, hence the DLL refcount can never go down to zero.


Broker do not compute stats data when send delay message

BUG REPORT

Please describe the issue you observed:

Environment:  rocketmq 4.8.0
When I use rocketmq-exporter + prometheus + grafana monitor the rocketmq cluster, I found a question:
I can`t see the rocketmq_producer_tps of the topic that used to send some delay message.
When I debug rocketmq-exporter , I know it collect metrics by TOPIC_PUT_NUMS from broker, code as following:
 //how many messages has sent for the topic
                        bsd = mqAdminExt.viewBrokerStatsData(masterAddr, BrokerStatsManager.TOPIC_PUT_NUMS, topic);
                        String brokerIP = clusterInfo.getBrokerAddrTable().get(bd.getBrokerName()).getBrokerAddrs().get(MixAll.MASTER_ID);
                        metricsService.getCollector().addTopicPutNumsMetric(
                            bd.getCluster(),
                            bd.getBrokerName(),
                            brokerIP,
                            topic,
                            Utils.getFixedDouble(bsd.getStatsMinute().getTps())

When I debug broker source code, I found that when broker process delay message, it will use SCHEDULE_TOPIC_XXXX replace the original topic in the message, but when compute the result for broker send stats data, it still use SCHEDULE_TOPIC_XXXX as the topic which be sent message. Code and my comment as following:
            // MessageStore put message will set  SCHEDULE_TOPIC_XXXX to the topic of msgInner
            putMessageResult = this.brokerController.getMessageStore().asyncPutMessage(msgInner);
        }
        // so when handlePutMessageResult, the msgInner`s topic is SCHEDULE_TOPIC_XXXX
        return handlePutMessageResultFuture(putMessageResult, response, request, msgInner, responseHeader, mqtraceContext, ctx, queueIdInt);

@xxd763795151 it seems this issue duplicated with the https://github.com/apache/rocketmq/issues/2683，could you confirm the https://github.com/apache/rocketmq/issues/2683 related PR can meet you request first？and please feel free to reopen this issue again.


Update User Guide based on the PED feedbacks

Fix #115, #124, #125, #126, #129, #145, #151, #153, #154, #158, #161, #162, #170
Following tasks(changes required to be made) to be done is completed.

Remove exit duplicate (/)
Remove yellow block (/)
State in UG that multiple contacts could have the same phone number and email as they might sharing. (/)
Add 'linux' in the 'installation' part of the UG (/)
Explain in UG, what is the price field of a person refers to (/)
Remove edit g/GROUP in the UG (/)
Explain in UG that 'ungroup' feature is coming soon (/)
Add square bracket to "NAME2" under 'group' feature (/)
Add "..." after [MORE_KEYWORDS] and [MORE_TAG] in UG under "find" feature (/)
Update screenshots for "undo" feature, including "add" features(/)
Add spaces for the "undo" feature screenshots (/)
Update the command summary section (/)

Codecov Report

Merging #175 (2df71fa) into master (fc5096a) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master     #175   +/-   ##
=========================================
  Coverage     45.75%   45.75%
  Complexity      470      470
=========================================
  Files           115      115
  Lines          2402     2402
  Branches        270      270
=========================================
  Hits           1099     1099
  Misses         1237     1237
  Partials         66       66

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Changed source code to Kotlin, and updated build scripts for the new API

Code rewrite in Kotlin and updated Gradle build scripts to the new API
#10

Thank you for your contribution. But some of the parts that the library uses is missing.
You've changed app module.
But that library's modules is in here.


Should we check for FF version before using MediaRecorder?

There is a problem with Firefox MediaRecorder on some versions so should we check for those versions and warn or something like that? Related link below.
https://bugzilla.mozilla.org/show_bug.cgi?id=1250984

We can warn users if you're facing same issues in the latest Firefox standard release?

Maybe you should make a pull-request for exact versions? E.g.
Firefox on Android (version: xyz) seems NOT supporting video-tracks recording.
......
......
etc.

Will look at this next week. Don't have any time this week. If someone has time and is willing to do this before me, let me know, so we don't do the same thing same time :)

For me, recording using the MediaStreamRecorder on Firefox 45.0.1 on Mac is REALLY choppy. Video continuously flickers in and out on playback. Is this similar to your issue or is it new?


"Unexpected timeout from process 1. Process will no longer be monitored" While Egressing Trace in Kubernetes (DM 8)

Description
When egressing a trace in Kubernetes using .NET Monitor 8, the following messages show up in the logs:
2023-03-09T19:43:27.5549626Zwarn: Microsoft.Diagnostics.Tools.Monitor.ServerEndpointInfoSource[52]       Unexpected timeout from process 1. Process will no longer be monitored. 2023-03-09T19:43:27.5581934Zinfo: Microsoft.Diagnostics.Tools.Monitor.CollectionRules.CollectionRuleService[44]       => TargetProcessId:1 TargetRuntimeInstanceCookie:f29dfb3c42de421eb8cdbb65fec4b124       Stopping collection rules. 2023-03-09T19:43:27.5588928Zinfo: Microsoft.Diagnostics.Tools.Monitor.CollectionRules.CollectionRuleService[45]       => TargetProcessId:1 TargetRuntimeInstanceCookie:f29dfb3c42de421eb8cdbb65fec4b124       All collection rules have stopped.
Checking the /processes route while egressing shows that there are no processes present - after completing the egress operation, the /processes route once again shows the default process (with a persistent UUID).
It's also worth noting that the egress is successful, and that dotnet-monitor continues to work as expected after this issue.

Configuration
DM - image: mcr.microsoft.com/dotnet/monitor:8.0-preview
Target - image: mcr.microsoft.com/dotnet/samples:aspnetapp
ConnectionMode - Listen
API Operation - curl.exe -v "http://localhost:52323/trace?pid=1&durationSeconds=5&egressProvider=monitorBlob"

Regression?

This issue appears to be a regression - the .NET Monitor 7 image in Kubernetes with the exact same configuration does not have this issue.
Other information

Let's check if the 7.1 image exhibits the same issue; if it does, it's likely due to some change in the NETCore.Client library from dotnet/diagnostics.

Let's check if the 7.1 image exhibits the same issue; if it does, it's likely due to some change in the NETCore.Client library from dotnet/diagnostics.

Confirming that the 7.1 image does have this issue.

This has been fixed in the 8.0 Preview 3 and 7.1.1 releases.


Icepak.create_pcb_from_3dlayout(component_name, project_name, design_name, extent_type='Polygon') still creates bounding box

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
The function Icepak.create_pcb_from_3dlayout(component_name, project_name, design_name, extent_type='Polygon') doesn't  create a polygonal PCB successfully. Instead, AEDT uses bounding box. The AEDT window shows the following error.

It seems that it works only if I give the correct name of the outline polygon in the parameter of this function.
When I create a PCB manually in AEDT GUI, the outline polygon is automatically selected. But in scripting, I can't get the outline. In a compatible script, I can set "Use3DLayoutExtents:=" to True like the code below to avoid this problem. However, in PyAEDT, there is no such parameter.
oEditor.InsertNativeComponent(
	[
		"NAME:InsertNativeComponentData",
		"TargetCS:="		, "Global",
		"SubmodelDefinitionName:=", "PCB",
		......

		[
			"NAME:NativeComponentDefinitionProvider",
			"Type:="		, "PCB",
			"Unit:="		, "mm",
			"Version:="		, 0,
			"MovePlane:="		, "XY",
			"Use3DLayoutExtents:="	, True
		]
               ......
	])

Steps To Reproduce

Open AEDT and import an EDB file
Insert Icepak design
Run the script below.

from pyaedt import Icepak

ipk = Icepak()
project_name = ipk.project_name

oEditor = ipk.modeler.oeditor
ipk.create_pcb_from_3dlayout(
    component_name='PCB',
    project_name=project_name,
    design_name='main',
    extent_type='Polygon'
)

ipk.release_desktop(close_projects=False, close_desktop=False)

Which Operating System are you using?
Windows
Which Python version are you using?
3.7
Installed packages
attrs==23.1.0
certifi==2023.5.7
cffi==1.15.1
charset-normalizer==3.2.0
click==8.1.5
click-plugins==1.1.1
cligj==0.7.2
clr-loader==0.2.5
colorama==0.4.6
cycler==0.11.0
et-xmlfile==1.1.0
Fiona==1.9.4.post1
fonttools==4.38.0
geopandas==0.10.2
idna==3.4
imageio==2.31.1
importlib-metadata==6.7.0
kiwisolver==1.4.4
matplotlib==3.5.3
networkx==2.6.3
numpy==1.21.6
openpyxl==3.1.2
osmnx==1.1.2
packaging==23.1
pandas==1.3.5
Pillow==9.5.0
platformdirs==3.9.1
plumbum==1.8.2
pooch==1.7.0
psutil==5.9.5
pyaedt==0.6.85
pycparser==2.21
pyparsing==3.1.0
pyproj==3.2.1
python-dateutil==2.8.2
pythonnet==3.0.1
pytz==2023.3
pyvista==0.38.0
pywin32==306
requests==2.31.0
rpyc==5.3.1
Rtree==1.0.1
scikit-rf==0.28.0
scipy==1.7.3
scooby==0.7.2
Shapely==1.8.5.post1
six==1.16.0
SRTM.py==0.3.7
typing_extensions==4.7.1
urllib3==2.0.3
utm==0.7.0
vtk==9.2.6
zipp==3.15.0

Hi Rungenwagen,
I had the same issue - however I managed to overcome it by searching for the outline polygon the HFSS 3DL and use that input when importing PCB in Icepak.
Using something like the lines below.
""
Outline = h3d.modeler.objects_by_layer("Outline")
ipkapp.create_pcb_from_3dlayout(component_name="PCB1",project_name=h3d.project_name, design_name=h3d.design_name, extent_type='Polygon',outline_polygon=Outline[0])
""
I have not seen a project where the outline did not excist in HFSS 3DL - so I do not know what will happen in that case.
Best regards,
Kim

Hi Kim,
thank you for the tip. I managed it.
Best regards,
Rungenwagen

Fixed, please have a  look


Arming denied due to GeoFence despite GPS Available

Today I wanted to run some test in the basement. So I got a lot of On and Off GPS Locks. Still at some points the GPS Reception was very acceptable. I wanted to Arm and shortly start the Copter to test the behaviour of the Home Position Altitude of the MinimOSD. But the Copter denied Arming with a Warning that a valid Home Position has to be available when using Geofencing RTL. But GPS Reception was ok, I got a valid Lock on my Home having 6 Sats available. Also Dis- / Rearming IO or Power Cycling did not help.
So either GPS Quality did not reach some criteria that I don't know of? Or there is a Problem with the Arming when using Geofence RTL. As soon as I switch Geofence Violation to any other than RTL I can arm.
Currently on 1.5.2 Stable.

Interesting experiment. The system correctly thinks that your GPS is potentially sketchy. Let us know if that test fails outdoors. But we're not going to change anything for indoor operation - its inherently unsafe and GPS is not designed to work there.

I agree. Any hint about the criterias taht leads the System to think that GPS is unreliable. An older uBlox 6 might as well only reveal 6 Sats with the HDOP from the M8N above. So what is the Limit?

Is this what you're looking for? https://github.com/PX4/Firmware/blob/master/src/modules/commander/commander.cpp#L2083

Yes. I was looking around in this file and tried to figure out what makes a HomePoint valid. I figured it should be HDOP < 2.5 (AFAIK). In the OSD I could see that the Homepoint Alt was reset to 0 as Baro alone (which is active as long as GPS does not have a Lock) actually is about -310m off.
I'll try to see how this behaves Outdoor, but for me it is still not clear why with this GPS Reception the Homepoint was not set valid.

There are two parameters which specify the minimum gps quality:
https://github.com/PX4/Firmware/blob/master/src/modules/commander/commander.cpp#L1210
https://github.com/PX4/Firmware/blob/master/src/modules/commander/commander.cpp#L1800
You'll have to check the values for your config to see what they are.

Those:
static float eph_threshold = 5.0f;
static float epv_threshold = 10.0f;
I assume this corresponds to HDOP/VDOP?

that's actually not clear to me from QGC; would have to look at the code.


update pd dependencies

What problem does this PR solve?
migrate pd to tikv/pd
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects
Related changes

LGTM


Update codeql-analysis.yml to include Python

Motivation and Context
Apply CodeQL scanning to Python as well as csharp.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

thank you @brendandburns


Align with recent changes on Gradle plugin

Extract CI logic to a utility class
Read Teamcity data from a generated build configuration file

Is this ready to review?

I incorporated some changes from https://github.com/gradle/common-custom-user-data-gradle-plugin/pull/123 which is not yet merged. It would probably be better to agree on this one first.

This is ready for review @runningcode
Here is a build scan from TC 2022.10.1

Thanks for breaking this up in to smaller PRs. So that I can best review this, can you outline the planned PRs to align the CCUD Maven Extension with the CCUD Gradle Plugin so that I know where to review and what will be covered in a future PR?

You can review the splitted PRs in this order:

#97
#98
#99
#100


MARP-364 GitHub Actions Pipeline for WebTests based on new 22.04 Ubuntu images

Hi @ivy-rew, I checked the readme of the latest Ubuntu and found that it has the Firefox Browser now.
https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md
I also test in DocFactory dev-build and success.
Please take a look at this PR.


resize docker container disk fail!

hi,i refer to http://jpetazzo.github.io/2014/01/29/docker-device-mapper-resize/,when i resize docker container disk,it is fail,pls help me.
My docker version is:
16:17:30 # docker version
Client version: 1.3.2
Client API version: 1.15
Go version (client): go1.3.3
Git commit (client): 39fa2fa/1.3.2
OS/Arch (client): linux/amd64
Server version: 1.3.2
Server API version: 1.15
Go version (server): go1.3.3
Git commit (server): 39fa2fa/1.3.2
My docker info is:
Containers: 1
Images: 4
Storage Driver: devicemapper
Pool Name: docker-8:3-3408905-pool
Pool Blocksize: 524.3 kB
Data file: /var/lib/docker/devicemapper/devicemapper/data
Metadata file: /var/lib/docker/devicemapper/devicemapper/metadata
Data Space Used: 944.2 MB
Data Space Total: 536.9 GB
Metadata Space Used: 1.04 MB
Metadata Space Total: 8.59 GB
Library Version: 1.02.84-RHEL7 (2014-03-26)
Execution Driver: native-0.2
Kernel Version: 3.10.0-123.el7.x86_64
Operating System: CentOS Linux 7 (Core)
My docker config is:
16:25:20 # cat /etc/sysconfig/docker
OPTIONS=-H fd:// --insecure-registry docker.ops-chukong.com:5000 --iptables=false --storage-opt dm.blocksize=512K  --storage-opt dm.loopdatasize=500G --storage-opt dm.loopmetadatasize=8G
i create and run docker container,it name is test1.and i resize this container disk from 10g to 20g.
before the change:
16:15:07 # dmsetup table /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
0 20971520 thin 253:0 6
after the change:
16:15:49 # echo $((2010241024*1024/512))
41943040
root@docker-test2:/tmp
16:16:37 # echo "0 41943040 thin 253:0 6"|dmsetup load /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
root@docker-test2:/tmp
16:16:57 # dmsetup table /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
0 20971520 thin 253:0 6
root@docker-test2:/tmp
16:17:00 # dmsetup resume /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
root@docker-test2:/tmp
16:17:13 # dmsetup table /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
0 41943040 thin 253:0 6
root@docker-test2:/tmp
16:17:15 # resize2fs /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
resize2fs 1.42.9 (28-Dec-2013)
resize2fs: Device or resource busy while trying to open /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa
Couldn't find valid filesystem superblock.
so what is the problem?

I don't know how to help, but have you tried to use aufs istead of devicemapper ?

thanks you for replay，I always use devicemapper.I found this problem is in
the centos 7,and it's ok in the centos 6.
2015-04-21 21:16 GMT+08:00 Glaudiston Gomes da Silva <
<EMAIL_ADDRESS>
I don't know how to help, but have you tried to use aufs istead of
devicemapper ?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/11153#issuecomment-94790169.

Looking at the above, I don't think this is a bug in docker, so I'm going to close this issue, but feel to comment if you think I closed this by mistake.

+1

USER POLL
The best way to get notified when there are changes in this discussion is by clicking the Subscribe button in the top right.
The people listed below have appreciated your meaningfull discussion with a random +1:
@kriss9

@dl528888  if you are use centos7,you can try:xfs_growfs /dev/mapper/docker-8:3-3408905-8f99c3f825b13351b7a21802e4341cb669ae0e939c3ccc0f788edc5827065eaa


Return the length of the array/slice

Write a function that returns the length of the array/slice. For example:
number := lengthArray(myArray)

[ ] Implement the function as part of the internal/utils pkg
[ ] Write unit test
[ ] Make sure there are testable examples
[ ] For logging use https://github.com/Sirupsen/logrus
[ ] in case of specific errors please store them separately to internal/errors pkg

Hints

As an exercise, try to write the test first. Let it fail. Then try to fix it, by writing the actual implementation of the function.

Extra help

https://gopher.gr/arrays-me-tin-golang/
https://gopher.gr/basic-unit-testing-me-golang/

For logging https://linuxhint.com/golang-logrus-package/

len and cap are built-in methods. Not needed.


Add .HasCodeBlock

Hugo already has a .HasShortCode that returns true if a given shortcode was used in the page. I't useful when, for example, we want to include some CSS or JS in the head to handle the content of the shortcode. It would be nice to have a .HasCodeBlock as well, for when we need to include some CSS or JS in the head to handle the content of particular code-blocks.
For example, mermaid diagrams are typically implemented with code blocks like this:
```mermaid
graph TD;
    A-->B;
    A-->C;
    B-->D;
    C-->D;
\```

It would be nice to be able to add something like this in the head:
{{ if .HasCodeBlock "mermaid" }}
...
{{ end }}

It's already possible to do something similar by setting a variable in the codeblock hook, and then adding the JS in the footer or something. But there is currently no way to add it in the head, before the {{ .Content }} is called. I think it would be nice to have this option, as it's typically recommended by the packages to add things in the head of the page. And it's nice to keep all the resources in the same place in the head of the page. (Without using some variable in the front matter like hasMermaid = true...)

I agree, but it needs to wait a little, and I'll spend a few lines to explain why:

We do lazy rendering of .Content, mostly to avoid doing the work if it's never used.
We have some methods that depends on .Content (e.g. .WordCount) which leads to a rather clumsy init setup. Also, the root Page API has become rather big and is difficult to document (we're working on a auto generated reference).
This situation became even more visible in the recent Hugo edition where we added .Fragments (where you now can do .Fragments.Identifiers.Contains "my-page-heading") and split the parsing and rendering of the content (calling just .Fragments is very effective).
I have propsed that we add an RenderContent that returns a struct. With .Fragments in mind, we should probably also have a ParseContent method. When I'm getting around to wrapping up my million pages branch that should also allow us to pass a context (cache) identifiers to these, e.g. RenderContent "home".
With that done, it should be much easier to do .ParseContent.CodeBlocks.Identifiers.Contains, .ParseContent.ShortCodes.Identifiers.Contains, .ParseContent.Fragments.Identifiers.Contains etc. and get a much simpler and easier to understand API.

But there is currently no way to add it in the head, before the {{ .Content }} is called.

You can force content to be rendered anywhere with something like:
<head>
  ...
  {{ $noop := .WordCount }}
  {{ if .Store.Get "mermaid" }}
    ...
  {{ end  }}
  ...
</head>

Any of these will force content rendering:
Content
FuzzyWordCount
Plain
PlainWords
ReadingTime
Summary (regardless of whether or not it is defined in front matter)
TableOfContents
Truncated
WordCount

I just chose WordCount because it is the smallest value, probably less than 5 bytes.

Oh, that's a nice workaround. Thanks!

okay I think is the reason why I was seeing https://github.com/gohugoio/hugoDocs/issues/2751#issue-2624562322 and its quite flaky as well because sometimes the store gets populated sometimes it doesnt.
I will update my head to use {{ $noop := .WordCount }} thanks @jmooring


lxd/backups: Use backups dir for unpack

Closes #7561
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Is this instead of using storage.backups_volume or does that get symlinked to it?

backups is symlinked to the value when the config key is set.


[TS] - Ultica Support

Mod Names Needs Improved
All or Majority
Needs Improved
Add Ultica Support to mods

ill do this later on tonight

Thanks.


Consider using System.Reflection.Metadata

Instead of using IMetaDataImport and our own PEFile implementation, consider using System.Reflection.Metadata in its place, if possible.

Additionally, PEImage needs to be threadsafe.

What are the benefits of using S.R.M in place of IMetaDataImport?

I'm not sure if there are any.  It's more of something I wanted to investigate if there were benefits or if we should maintain the status quo.


-FPS

My osiris kills ~ 50-60% FPS. Is there any way to optimize it because if I choose to play according to Legit at 25-35 FPS or play without cheat but with stable 62 (fps_max 63) I would choose the second !!!

What options are you using?

My osiris kills ~ 50-60% FPS. Is there any way to optimize it because if I choose to play according to Legit at 25-35 FPS or play without cheat but with stable 62 (fps_max 63) I would choose the second !!!

did u use fix bone matrix in misctab?

clear osiris

I just inject not including any functions and I’m eating a bunch of FPS

To me happens the same but i have 300 fps when i inject stay with 20/30 fps and i have only activate skins and bomb time, If someone can help i will be grateful.

I no longer have this problem after updating the hook component

Thanks

Shuvi-moro what did you made get the error out?


checkPresence and PubNub

Hi,
First, thank you for your tremendous work on this project.
I am trying to use PubNub as my signaling service. When I try to use checkPresence or anything that relies on it, I get "undefined" returned. Is checkPresence supported on PubNub, and if not, could you suggest an alternative to see if a room exists so I can determine whether to do an open or join?
Thanks,
Paul

I get the same problem, only that I'm not using PubNub, did you find a solution?


[chore] cpu 和 memory 指标返回pid, pname, bcwd

Description:

Link to tracking Issue:
Testing:
Documentation:

已废弃回滚


Import Custom ABI

Allow import of custom ABI at error message Contract not found. Check your contract address & network

Good suggestion.
Right now, it pulls ABI's from Etherscan. If you verify your contracts there, it'll be available on this UI here.
Importing directly from here would definitely be a nice-to-have.
For the time being, I'll add a link to the verification page on Etherscan.


Can not use the visualize utils on the remote Linux server.

Can not use the visualize utils on the remote Linux server. There is no GUI.
I tried a lot of things but it didn't work out.
Any advice?
Thanks.

Maybe have a look at my point cloud viewer: https://github.com/MartinHahner/LiDAR_fog_sim/blob/main/pointcloud_viewer.py
You can fork the repo and adjust it to your needs.


Out of memory

I use a 12GB titanxp. It runs out of memory.
How to avoid this problem ? Is there a bug or my gpu memory is too small? What gpu do you use ?
Thank you

The number of trainable parameters of your model may be too large. See the console ouput for more detail.
I'm using 11G GTX1080Ti, there is nothing wrong with me.

Thanks for your response. I will check it.


Every few seconds i need to restart stream

Hello My issue is i put a stream and start a user to watch this stream. everything work withou transcoding. than after few second i need to restart the stream but i dont now the reason. If i start sthe stream direct without in vlc it works fine.

Transcode type
Video: Copy Audio: Copy
It works fine for me

and for massive start/stop streams show the msg
An error occurred.
Sorry, the page you are looking for is currently unavailable.
Please try again later.
If you are the system administrator of this resource then you should check the error log for details.
Faithfully yours, nginx.

This has been fixed


Add more logging and proper permissions to image scan

@JaredBurck @sabre1041 @pabrahamsson

@etsauer nice! For those reviewing, I ran a the image scanner and it took a couple of hours. Not knowing the percentage complete is annoying when you have a large amount of images being scanned.

@bbeaudoin see updates.

@bbeaudoin when I look at line 115 and 118, they match. are seeing something different?

@etsauer Running through this resulted in partial success

Docker configuration raised by @bbeaudoin appears to be set up properly as images from the registry is pulled
Images from the Red Hat registry cannot be retrieved as the cluster that I am using leverages images from registry.redhat.io which requires authentication. This would require a subsequent enhancement


Implement props for platform

At the moment there is no way to pass properties to the script, such as:

data-use-service-core
data-elfsight-app-lazy

It would be nice to be able to add them

Sure, it would be nice to have the package supporting the latest platform features. I plan to take care of this next week.

Implemented in 1.1.0.


A question about the tutorial of DLPFC dataset

I am highly intrigued by your innovative work. The tutorial you shared on the DLPFC dataset only covered the processing of 4 slices. I am curious to know how I can modify the code to incorporate all 12 slices for processing.

Hi. Since only 403 genes shared after the intersection of top 5000 HVGs of all 12 slices, we suggest you increase the n_top_genes to 10000, to ensure enough common HVGs in the combined input matrix. We also used the sample names (not slices) as "batch_name" to simplify the integration order. Tutorial can be found here: https://staligner.readthedocs.io/en/latest/Tutorial_DLPFC_12S.html