Update and rename python-package.yml to unittest.yml

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

Project coverage is 97.45%. Comparing base (dc0f586) to head (09aa2ad).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   98.28%   97.45%   -0.83%
==========================================
  Files           8        5       -3
  Lines         350      236     -114
==========================================
- Hits          344      230     -114
  Misses          6        6

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


Revamp the names of confusing options

There are a few options whose names made sense to me initially, but are in retrospect not carefully named. I'd like to start by adding in aliases for those options, keeping the old ones functional but logging a deprecation message during build time. And then theoretically remove them in the next major revision. (Or keep them working, but remove them from the documentation?)
I was thinking of the following, and I'm open to suggestions:

staticFileGlobs ⇒ filesToCache
dynamicUrlToDependencies ⇒ serverRenderedUrls
navigateFallback ⇒ defaultRoute
navigateFallbackWhitelist ⇒ defaultRouteWhitelist
replacePrefix, stripPrefix, stripPrefixMulti ⇒ try to wrap these up into a single modifyUrlPrefix option

CC: @addyosmani @gauntface @jpmedley for thoughts.

I like all of these except for the navigateFallback and
navigateFallbgackWhitelist.
If I were completely new to the tool, I may not know what a route is. But I
can work out what navigateFallback probably is.
Joe Medley | Technical Writer, Chrome DevRel |<EMAIL_ADDRESS>|
816-678-7195
If an API's not documented it doesn't exist.
On Wed, Aug 31, 2016 at 7:46 AM, Jeffrey Posnick<EMAIL_ADDRESS>wrote:

There are a few options whose names made sense to me initially, but are in
retrospect not carefully named. I'd like to start by adding in aliases for
those options, keeping the old ones functional but logging a deprecation
message during build time. And then theoretically remove them in the next
major revision. (Or keep them working, but remove them from the
documentation?)
I was thinking of the following, and I'm open to suggestions:

staticFileGlobs ⇒ filesToCache
dynamicUrlToDependencies ⇒ serverRenderedUrls
navigateFallback ⇒ defaultRoute
navigateFallbackWhitelist ⇒ defaultRouteWhitelist
replacePrefix, stripPrefix, stripPrefixMulti ⇒ try to wrap these up
into a single modifyUrlPrefix option

CC: @addyosmani https://github.com/addyosmani @gauntface
https://github.com/gauntface @jpmedley https://github.com/jpmedley
for thoughts.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/sw-precache/issues/164, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AH0vizl2YkfX-bZ37QYizoB0p1BD5LRwks5qlZOugaJpZM4JxsjG
.

In other words, an option that explains itself is better than one I have to
read about.
Joe Medley | Technical Writer, Chrome DevRel |<EMAIL_ADDRESS>|
816-678-7195
If an API's not documented it doesn't exist.
On Wed, Aug 31, 2016 at 2:09 PM, Joe Medley<EMAIL_ADDRESS>wrote:

I like all of these except for the navigateFallback and
navigateFallbgackWhitelist.
If I were completely new to the tool, I may not know what a route is. But
I can work out what navigateFallback probably is.
Joe Medley | Technical Writer, Chrome DevRel |<EMAIL_ADDRESS>| 816-
678-7195
If an API's not documented it doesn't exist.
On Wed, Aug 31, 2016 at 7:46 AM, Jeffrey Posnick<EMAIL_ADDRESS>
wrote:

There are a few options whose names made sense to me initially, but are
in retrospect not carefully named. I'd like to start by adding in aliases
for those options, keeping the old ones functional but logging a
deprecation message during build time. And then theoretically remove them
in the next major revision. (Or keep them working, but remove them from the
documentation?)
I was thinking of the following, and I'm open to suggestions:

staticFileGlobs ⇒ filesToCache
dynamicUrlToDependencies ⇒ serverRenderedUrls
navigateFallback ⇒ defaultRoute
navigateFallbackWhitelist ⇒ defaultRouteWhitelist
replacePrefix, stripPrefix, stripPrefixMulti ⇒ try to wrap these up
into a single modifyUrlPrefix option

CC: @addyosmani https://github.com/addyosmani @gauntface
https://github.com/gauntface @jpmedley https://github.com/jpmedley
for thoughts.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/sw-precache/issues/164, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AH0vizl2YkfX-bZ37QYizoB0p1BD5LRwks5qlZOugaJpZM4JxsjG
.

Thanks, @jpmedley. I can leave navigateFallback as-is.
Longer-term, I want to address routing to multiple App Shells via #94, at which point including routing in a parameter name might make sense.

I like the change in name.

I like the renames. I'm personally also fine with:

navigateFallback ⇒ defaultRoute
navigateFallbackWhitelist ⇒ defaultRouteWhitelist

But if consensus is a first time user is going to find it straight-forward to understand what these do as-is then let's for sure keep them as is.

Probably just me but I'd understand what  is intended with 'serverRenderedUrls' if it included the term 'template' somewhere in the name.
'templatesToUrls' would make sense to me more immediately than 'serverRenderedUrls'.

templatesToUrls sounds like a mapping in which each key is a template, and the values are one or more URLs. That doesn't reflect what's being configured here.
Including "template" in the name sounds good, so how about templatedUrls?
While I normally use abcToXYZ to name variables that represent mappings, that's not a hard and fast convention.

@jeffposnick sounds better than my suggestion :+1:

Echoing further support for templatedUrls. I think that's a lot more sensible a name to move forward on.

We've switched to improved names for a number of similar parameters in https://workboxjs.org/
Making a breaking change to rename the options in sw-precache isn't a priority.


A name for pool-1-thread-1's thread  is still running

class ‘PartitionedScheduledExecutor’ got a bug
just work thread exit but not scheduler
@Override   public boolean awaitTermination(long time, TimeUnit unit) throws InterruptedException {     if (isShutdown()) {       if (termination.isDone()) {         return worker.awaitTermination(time, unit);       } else {         long end = System.nanoTime() + unit.toNanos(time);         try {           termination.get(time, unit);         } catch (ExecutionException e) {           throw new RuntimeException(e.getCause());         } catch (TimeoutException e) {           return false;         }         return worker.awaitTermination(end - System.nanoTime(), NANOSECONDS);       }     } else {       return false;     }   }

Hello,
Thanks for your interest in Ehcache 3.
We usually recommend starting a conversation on the users google group before opening an issue.
Could you elaborate on what makes you think there is a bug?


[Feature Request] Tab names in session config

It was partially mentioned in https://github.com/sedwards2009/extraterm/issues/157
Please make it possible to configure the tab name in session types.
From my practice, i use separate session tabs configured for managing exchange / powercli / just powershell, and they all look alike after opening, sure i can rename, but that works only till application restart.

This hasn't been forgotten. I want to get the extension system (more) up and running and then doing this as an extension.

Fixed in v0.54.0.


Problem about Training with Celeba Dataset

Hello, did you use crop on the CelebA dataset for training and testing?

and did you use jpg or png format for images from CelebA

我觉得都没问题吧


fixing the featurization issue

Download the blobs to local before featurizing them.
Sorry for the large diff, should have merged master into my branches before I edited.
The file to look at is azureparallel_tutorial.R

Abandoning the pull request, will submit a clean one


React Native support

Hi , would it support React Native ?  thanks

I don't have any experience with React Native, but a little searching brought me to a React issue about this. Seems like it's not possible at the moment.


Update d3pie.js

Fixing 'ReferenceError: pie is not defined'

Hi @sebinkuriakose, thanks for the PR! Under what circumstances does that error occur? Could you provide a few more details on how to reproduce?

Thanks for the awesome work.
I faced this with following stack trace:
Uncaught ReferenceError: pie is not defined
    at Object.maybeCloseOpenSegment (core-libs.min-a77ef77….js:8)
    at Object.openSegment (core-libs.min-a77ef77….js:8)
    at SVGGElement.<anonymous> (core-libs.min-a77ef77….js:8)
    at SVGGElement.<anonymous> (d3.min.js:2)

To reproduce, click on a segment of a piechart. (Also reproducible on website charts as well)

Hello,
I am facing the same issue.
I have several charts shown in the same page so I can't name them all "pie".
When will this be available in master version ?

Facing the same issue. It can be reproduced when you are trying to use the d3pie object on a div with another ID than "pie":
        var pie = new d3pie("mypie", {
		header: {
			title: {
				text: "A Very Simple Pie",
				fontSize: 30
			}
		},
		data: {
			content: [
				{ label: "JavaScript", value: 264131 },
				{ label: "Ruby", value: 218812 },
				{ label: "Java", value: 157618}
			]
		}
	});

Same problem, any news when the PR will be merged ?

👍🏾

Bump. @benkeen


Use custom time getter by default for chainstate TestFramework

With this PR, tests should be 100% deterministic unless the TimeGetter decides otherwise.
The design of the solution I provided is up for discussion.

I like the idea of a better control over time in the tests, though I'm not 100% happy with the optional time_value field. All these "custom time getter" panics don't look pretty, but I think it is OK for the test code.

Exactly. As long as it's just for the test, it's just for the test. Panics in the tests are acceptable, so we can live with it. I'm happy with changing the design as long as it achieves the same goal.

just curious does it solve the problem seen in bootstrap test?

The implementation of bootstrap had a bug and that was fixed in #503. The issue primary issue I complained about in bootstrap, besides the one that was fixed, is that we get different on every run even when using the same seed value. That doesn't happen anymore. At this point, if you have a bug, you'll get the exact same result every time you run the test if you use the same seed.

Was thinking what could be done about the time getter option issue without going completely crazy with traits and stuff (only slightly crazy). Here's an approach to consider which might work with some modifications:

Parametrize TestFramework by the time getter typepub struct TestFramework<TG> {
    /* ... */
    pub time_getter: TG,

Any operation that needs to get time puts a constraint TG: AsRef<TimeGetter> and uses that to convert it to the actual time getter
Implement AsRef<TimeGetter> for TimeGetter
Create a ManualClock type that contains the time getter and the time value that can be controlled manually
a. expose the fucntions to control the time manually
b. Implement AsRef<TimeGetter> for it
Now TestFramework<TimeGetter> can be used for a custom time getter function or when using system clock, TestFramework<ManualClock> can be used in testing to control time manually
Fun part: since the builder takes ownership of self, it can change the type: // Inside impl<TG> TestFrameworkBuilder<TG>
 fn with_time_getter<TG1>(
     self /*: TestFrameworkBuilder<TG> */,
     time_getter: TG1,
 ) -> TestFrameworkBuilder<TG1> {...}

I leave to your best judgement whether this is worth giving a shot

One thing I've been very aggressively avoiding is making the TestFrameworkBuilder generic. This is because it'll require generic parameters on construction. Unfortunately, it seems that your plan may require that. We had a similar issue with TransactionVerificationStrategy, and that's why you see that ugly enum.

One thing I've been very aggressively avoiding is making the TestFrameworkBuilder generic. This is because it'll require generic parameters on construction.

The TestFrameworkBuilder::new function would just return monomorphic TestFrameworkBuilder<ManualClock> (or whatever the default is). Only TestFrameworkBuilder::with_time_getter would be generic in my original proposal but even that could just return a builder with fixed parametrization.

One thing I've been very aggressively avoiding is making the TestFrameworkBuilder generic. This is because it'll require generic parameters on construction.

The TestFrameworkBuilder::new function would just return monomorphic TestFrameworkBuilder<ManualClock> (or whatever the default is). Only TestFrameworkBuilder::with_time_getter would be generic in my original proposal but even that could just return a builder with fixed parametrization.

I will try it tomorrow. I'll merge this now as I need to run a test loop over night.


Kitsu tracker: HTTP error 401

Steps to reproduce
When I read something, it should mark it as read, but I get an error and it doesn't track it. Website works, but not on Yokai.
Expected behavior
It should track where I'm at when a chapter is completed.

Actual behavior
It gives an error and it doesn't track.

Crash logs
No response
Yōkai version
<IP_ADDRESS>
Android version
Android 13
Device
Samsung galaxy S22+
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, or a request for an extension, I should be contacting the extensions repository's maintainer/support for help.
[X] I am reporting an issue exclusive to this fork. I have also checked that is not an issue on the main version of Mihon
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have filled out all of the requested information in this form.

401 means you need to refresh your login, you can try to unlink & relink your AniList in the app's Trackerror page.
However, AniList has been experiencing some issues recently & have disabled/have unstable API currently. Check AniList for more info/issues, and/or come to the Mihon Discord yokai-j2k channel for basic issues/troubleshooting (in general).

Thank you very much. It worked.


Re-use getVisitAction utility function

The logic for reading an Action from a list of Element instances'
[data-turbo-action] attribute is still scattered across several
modules, in spite of the getVisitAction declared in the util module.
This commit replaces duplicate implementations of the getVisitAction
method with the utility function.
With those calls added, the isAction type checking utility is only
ever invoked by getVisitAction, so this commit moves its
implementation to the util module, and removes all other invocations.

@kevinmcconnell if you're available, I'd really appreciate some code review on this extraction.

@kevinmcconnell great point! I've made that change.


Create FrequencyDomain results class

Currently, the results of the frequency_domain function are stored in a Python built-in dictionary.
It would be a nice improvement if the results were stored in a custom dict-like class with additional useful methods, such as plot.
For instance,  a plot method could depict the power spectral density (PSD) chart like the following:
freq_domain_results = frequency_domain(rri, ...)

freq_domain_results["lf"]
# 134.56
freq_domain_results.plot()

There is something similar already created for time varying analyses:
https://github.com/rhenanbartels/hrv/blob/3f813f6727ff693daccf2fd1a939a30a966f56e1/hrv/nonstationary.py#L13

#29 Look at this


Added deprecation notice to source readme regarding new changes to come

Changes

🧹 Added deprecation notice to source readme regarding new changes to come

/kind documentation

/hold
This is an idea on how to communicate the Source's parameters deprecation, I'll hold it until everyone agrees on a format


Lazily initialize tracer for Jaeger/1.10 compatibility

When running The Jaeger Python Client with this library, the initialization step hangs indefinitely with Django 1.10.
@mikebryant if you take your example OpenTracing/Jaeger Django project and upgrade Django to >=1.10 and try running, the server will fail to start.
I believe this comes down to the way in which Django changed the initialization of Middleware. In versions prior to Django 1.10, the middleware wouldnt be initialized until the first request. In Django 1.10+, the middleware is initialized on start. Initializing the Jaeger tracer before the first request causes the Jaeger client to deadlock -- check out Jaeger Issue 31 and Jaeger Issue 60.
This change simply forces the initialization to happen once on the first request, thus avoiding the above behavior.

@mikebryant ^^

ugh, I'm really sorry. I've been overloaded with other stuff and missed this.
Thanks for pinging me about this!

Ping :)

@carlosalberto, @benjigoldberg if it's any help I was able to get this working against master w/ these changes: https://github.com/rmfitzpatrick/python-django/commit/301045b6a27dcd0ecb87f0a7f743ccbe2f5ec207.

Hi folks,
It would be great to see rmfitzpatrick@301045b cherry-picked into master.
Thank you for the effort, @benjigoldberg and @rmfitzpatrick!

Hi guys, any update 💃


Added ability to specify number of words to display with args.words

This is a simple extension to https://github.com/snare/voltron/pull/184 that respects the --bytes option when displaying words.

@snare

@snare  bump bump

Thank you

Apologies for the delay in looking at this :)


Suggestion: Adding a Multithreading/Processing option

Love this downloader. A option to determine multiple threads to make the downloading process much faster would be really nice to have. Could you add something like this?

The program is already multithreaded for the important parts of the workload.
The downloading process has two steps:

First we retrieve a list of all image-urls that need to be downloaded. This is not parallelized.
Afterward, we download the images. This download is parallelized, but limited to 8 threads to not hammer the image-hosts (which could make them temporarily IP-ban the user) and to not oversaturate the internet connection. 8 threads was chosen arbitrarily and could probably be increased if you're finding it much too slow.

The initial retrieval of image-urls is not parallelized because there is little need for it. Depending on the host (Imgur or Reddit) that we pull data for, we do the following:

Imgur album: Retrieving the required data only takes a single API call.
Imgur account: Retrieving all the images that a specific user uploaded makes one API call per 50 images. (This will abort when Imgur starts rate-limiting us, so a max of 500 requests can be executed here)
Reddit: When pulling data from reddit, we can pull info from up to 1000 posts. The default number of posts that we grab per request is 25, for a total of 40 requests for post information. If every post is a supported Imgur- or Deviantart-link then we do one more API call for a total worst-case of 1040 web-requests.

In the context of the time it would take to download the images from all 1040 links (some of which may be albums with an arbitrary number of images), the time taken during these initial requests is negligible and not worth parallelizing.


NO-JIRA Avoid creation of unnecessary StringBuilder instance in Netty…

…Acceptor#getProtocols(Map)

This isnt NO Jira work. Can a JIRA item be raised

This isnt NO Jira work. Can a JIRA item be raised

@michaelandrepearce this is just code cleanup. Nothing that would bring any value to a release notes...
The commit itself would be enough record of the change here.
If someone is creating a JIRA, it would be a task, as the is not an improvement, not a feature, not a bug fx.. no value on the release notes.

@gemmellr Based on your feedback I cleanedup/simplified the method further.
I have no problem in opening a corresponding JIRA if required.


add py3.12 to testing matrix

ERROR: Could not find a version that satisfies the requirement torch>=1.1.0 (from versions: none)
https://pypi.org/project/torch

The 3.12 test is failing, I think it's because 3.12 is missing from tox.ini under [tox] and [gh-actions].

Travis CI - Pull Request Expected — Waiting for status to be reported Required should probably be removed from the required tests below unless there is an active, paid subscription to Travis CI which is no longer free to open source projects.

I think that adding py312 to tox.ini will not solve the problem which looks like this:
py: install_deps> python -I -m pip install -r /home/runner/work/snntorch/snntorch/requirements_dev.txt
    [ ... ]
        File "/tmp/pip-build-env-vfl9zw5b/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 20, in <module>
      ModuleNotFoundError: No module named 'imp'

https://docs.python.org/3.12/whatsnew/3.12.html#imp so setuptools in requirements_dev.txt needs to be upgraded.

      if: matrix.python-version == '3.12'
      run: pip install --upgrade setuptools

It looks like we're still running into the same error above.

Please re-approve te workflow so we can see if it passes.

https://pypi.org/project/torch is not yet compatible with the current version of Python

We can open this back up once PyTorch catches up to Python 3.12.


[AutoPR resources/resource-manager] Fix ARMViolations for Microsoft.Resources

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3912

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3187


Fix missing RUnlock in Range function

This PR adds missing shard.RUnlock() in the Range function.

@alexandear thanks for the contribution.


Theme Selector does not work

Dev.
In my module I have a shape, shoppingCart.cshtml,  in the MyModule\Views\ folder.
If I duplicate it under the Default site theme, \Themes\MySiteTheme\Views\shoppincart.cshtml, this new version is selected rather than the one in the module.
(Nothing special in any placement file)
But if I fix the theme to another one by a theme selector, this theme owning a dedicated version of the same shape  in \Themes\OtherSiteTheme\Views\shoppincart.cshtml, this last version is not used and it is the version in the module which is called ?
This is already weird, but it is worse because the  layout.cshtml in the OtherSiteTheme is also not used, and neither the dedicated wrapper, document.cshtml.
When a selector changes the current theme we could expect that everything be similar to what occurs in the default theme site. This is clearly different laws that apply.
This is a bug in Dev and certainly in 1.10. May be related to #6514

The layout used is the one in Orchard.Core\Shapes\Views ????? badly broken

If I set the 'laternate' theme as Default Site theme everything works as expected.

i also met it long time ago.
seems related to #5675

I have just tried many configs to have this working.
In the siteTheme for shopping.cshtml

under Views: OK
under Views\MyModule: NOK
under Views\MyModule\Views NOK
under Views\MyModule\Mycontroller NOK
under Views\MyController NOK
layout.cshtml works.

in the theme selected
No one of previous places works and the layout used is always
Layout.cshtml calles is always the one in orchard.Core.
In the actual situation this theme selector feature is just a marketing announcement.

Can you try my PR to see if it resolved your issue? https://github.com/OrchardCMS/Orchard/pull/6517

I will try it as soon as I have VS2015 repaired following the install of an update of GitHub for VS which broke the config (not the first time... no more HitHub for VS for me)

It was Webessentials. Broken by GitHub vsx.
So I tried your patch and no change: when the selector force a different theme than the Site Theme, this theme does not work as a norm

The pb seems to be in the fact that the resources in the 'forced theme' have not been scanned, they are not in the resource tables so they can't be solved.

This Theme selector concept is weird or I don't understand its usage

Second option

Ok, then what is this thing  doing ?

For every request, Orchard is able to select which Theme to use among all the enabled ones. This way you could customize the rendering based on the url, the user, the time of the day, whatever. To do so there is a IThemeSelector interface that anyone can implement to provide such behaviors. Each implementation will analyze the current context and decide if it should provide a specific theme or if it doesn't apply. Each result gets a priority, for instance if you want a theme per user, and also have some rule per url, but the current requests matches both, then the priority of your results will disambiguate which theme to use. Selecting a theme will indeed define what template is used to render a specific shape as shapes can be defined or overrided in themes. There are usually two Theme Selector implementation that take action by default. The one that chooses the theme that is defined for the front-end in the settings, and another implementation with higher priority that will return TheAdmin when the request is for an admin controller.

And to be clear, there is no magical theme selector in the system. The default one is implemented using the exact same hooks that anybody can use, and has no special priority.

here is what I did
`
public class MyThemeSelector : IThemeSelector {
private readonly IOrchardServices _orchardServices;
    public MyThemeSelector(IOrchardServices orchardServices) {
        _orchardServices = orchardServices;
    }
    public ThemeSelectorResult GetTheme(RequestContext context) {
        if (AdminFilter.IsApplied(context) || context.RouteData == null || context.RouteData.Values  == null ) {
            return null;
        }
        var rts = context.RouteData.Values;
        if ( rts.ContainsKey("controller") ) {
            var ctrl = rts["controller"].ToString().ToLower();
            var settings = _orchardServices.WorkContext.CurrentSite.As<MySettingsPart>();
            int priority = settings.TSPriority;
            if ( ctrl.Contains("shoppingcart") && settings.ShoppingCartTS && !string.IsNullOrEmpty(settings.ShoppingCartTheme))
                return new ThemeSelectorResult { Priority = priority, ThemeName = settings.ShoppingCartTheme };
            if ( (ctrl.Contains("checkout") || ctrl.Contains("shipping")) && settings.CheckoutTS && !string.IsNullOrEmpty(settings.CheckoutTheme))
                return new ThemeSelectorResult { Priority = priority, ThemeName = settings.CheckoutTheme };
        return null;
    }

}`

So why when I activate it it does not behave as a normal theme ? Why the layout called is the one in Orchard.Core and not the ones in the Theme/View folder ? Why my alternate shapes in the theme view folder are not found and I go back to the shapes in the module ?

Can you put a breakpoint in the service that reolves all theme results and see which one it's using, and why?

I spent 2 days on this. I have put dozen of breakpoints building a kind of breakpoint factory. For sure my theme is selected. Please read the info here.
Ok. I gave up ... again. I will find anothere solution than using 2 different themes, I am late and must deliver.

A simple experiment would be to select those same themes as the active theme, with the default theme selector, to check if the views get rendered as expected. Then, you can break into that default selector, and peek into the theme selector result, to check if the theme name and priority show exactly like what you've put in your settings.
As a side note, however, it seems a bit weird to select an entirely different theme when on the shopping cart or checkout. I'm sure you have your reasons, but it does seem like a few alternates could do the job much more simply.

When I select this 'alternate theme as the site theme, everything is ok. I already write it.
Ok tired of permanent lessons, you know.
Anyway thanks for trying something. I have to progress.

Well, we're also all tired of getting yelled at for trying to help. Your feeling of entitlement is palpable.
You did not do what's after the "then" if I'm not mistaken, which is key to finding out what's going on. Since your theme selector is running, something has to be different in its results. Stepping out of the theme selector up into the logic that makes the actual decision might help as a next step, if looking at the results is not enough to figure it out.

Is your theme enabled? Not default but enabled. The difference is subtle in the UI.

Great day, this theme was not activated. Thanks @Sebastienros.
The world was against my mind but reality was simply a small link under the theme on the theme menu. Classic.
What is the usage of this activated status?
There should be an error when trying to install a 'non activated theme' as the current one.
This is never checked in theme manager.
`public ExtensionDescriptor GetRequestTheme(RequestContext requestContext) {
var requestTheme = _themeSelectors
.Select(x => x.GetTheme(requestContext))
.Where(x => x != null)
.OrderByDescending(x => x.Priority);
        if (requestTheme.Count() < 1)
            return null;

        foreach (var theme in requestTheme) {
            var t = _extensionManager.GetExtension(theme.ThemeName);
            if (t != null)
                return t;
        }

        return _extensionManager.GetExtension("SafeMode");
    }`

Any feature that needs to be used has to be enabled. Themes can be enabled, and set as default, these are two different things. And like features can be not enabled, theme have the same possibility as they can contain custom logic, and be used by some tenants only. Because most of the time it doesn't make sense to just enable them, the link is not prominent.

Do we think there are reasonable cases where a theme selector selects a theme that is not enabled, and that is not an error? How bad would it be to push a warning when the theme that ends up being selected is not found?

Could be logged, WARNING ...

Maybe we can do better than that and have a theme and view resolution diagnostics tool in Glimpse and/or designer tools.

Github is missing a status: closed by concept


Server Group dialog isn't same height as the shell window

SQL Operations Studio Version: master on RHEL 7.3

Steps to Reproduce:

Server Group dialog isn't same height as the shell window

Expected: There should not be a visible gap below the dialog and the bottom of the shell window.

This is fixed now.


[Feature Request] Allow OpenSearch to automatically find java_home without manual definition

As of now, if OpenSearch does not bundle any jdk in work_dir/jdk folder, opensearch will not run even if there is jdk installed on the host.
User needs to manually define OPENSEARCH_JAVA_HOME or JAVA_HOME as env var for opensearch to find the corresponding JDK.
Would be nice if we can have one more layer of detection on this, in the sense:

If no work_dir/jdk as well as OPENSEARCH_JAVA_HOME/JAVA_HOME exist or define, auto use java command or similar to find the java_home location and attempt the start.

% dirname $(dirname $(readlink -f $(which java)))

If work_dir/jdk but has OPENSEARCH_JAVA_HOME/JAVA_HOME exist or define, use values of these env vars.
if work_dir/jdk but no OPENSEARCH_JAVA_HOME/JAVA_HOME exist or define, use work_dir/jdk.

PS: This is specifically for RPM artifact.
We are currently evaluating the possibilities or bundle jdk vs dependency jdk in rpm See this issue. If this behavior not exist then our rpm artifact will have to use post_script to hard code  OPENSEARCH_JAVA_HOME=<> in /etc/sysconfig/opensearch, in order for opensearch to run, if the rpm does not have any bundled JDK.
Thanks.

opensearch's launcher binaries are all bash scripts. It would be rather trivial for it to check if OPENSEARCH_JAVA_HOME is unset (or unfound) and default to a "known good" value.... and in fact, it seems to do this:
from distribution/src/bin/opensearch-env
# now set the path to java: OPENSEARCH_JAVA_HOME -> JAVA_HOME -> bundled JDK
if [ ! -z "$OPENSEARCH_JAVA_HOME" ]; then
  JAVA="$OPENSEARCH_JAVA_HOME/bin/java"
  JAVA_TYPE="OPENSEARCH_JAVA_HOME"
elif [ ! -z "$JAVA_HOME" ]; then
  JAVA="$JAVA_HOME/bin/java"
  JAVA_TYPE="JAVA_HOME"
else
  if [ $OS = "darwin" ]; then
    # macOS bundled Java
    JAVA="$OPENSEARCH_HOME/jdk.app/Contents/Home/bin/java"
  elif [ $OS = "freebsd" ]; then
    # using FreeBSD default java from ports if JAVA_HOME is not set
    JAVA="/usr/local/bin/java"
  else
    JAVA="$OPENSEARCH_HOME/jdk/bin/java"
  fi
  JAVA_TYPE="bundled jdk"
fi

To address this for the RPM case (where JDK is provided as a dependency), if JAVA_HOME is unset (which only matters if OPENSEARCH_JAVA_HOME is unset), have the script check for the existence of the Corretto (JDK dependency) provided directory, and if found, use that, if not, bail out.


Feature/custom sized enums

This resolves #24, adding support for specifying an integer type on enums.

I believe you forgot to consider enum custom sizes when generating structs (that contain fields whose types are custom-sized enums).
For example, the MarshalBebopTo method will just call iohelp.WriteUint32Bytes regardless of the enum's custom size.


Update rancher-version annotation to include 2.7 versions in kubewarden-controller

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When attempting to install the latest kubewarden-controller chart from the UI with a Rancher version v2.7.0, the latest versions of the chart do not appear. This is causing the installation to fail as the kubewarden-crds chart is not being auto-installed along with the controller.
I believe this annotation needs to be updated to include 2.7.0 versions of Rancher, or rewritten to >= 2.7.0-0 <= 2.7.100-0 as v2.7.0 is the first with the extensions support.
Expected Behavior
All of the latest versions should appear and should include the crd auto-install.
Steps To Reproduce

With Rancher v2.7.0 attempt to install Kubewarden from the UI guided steps

Anything else?

Response from chart fetch

@jordojordo you are right, that was it! opened https://github.com/kubewarden/helm-charts/pull/156 to fix it.


Running server for RoboStudio to connect to?

Hi everyone
After a bit of fuss, I've got the SDK compiling and building the sample apps on my RPi Zero W, with an RP-A1M8-R6, and getting some output from
~/rplidar_sdk/output/Linux/Release $ ./simple_grabber --channel --serial /dev/ttyUSB0 115200
Wondering if anyone's ever managed to connect the Slamtec RoboStudio to a TCP or UDP port of another computer in the LAN?
Is that just for "SLAMWare" devices? Is there any useful documentation? (The user manual is not useful).
I see there's some files sl_tcp_channel.cpp, sl_udp_channel.cpp.
It's nice that there's some server code, but it would be pretty cool if there was an basic example, too, like, using the protocol, etc.
I would imagine that connecting the RoboStudio to a headless device running an RPLidar would be a common use case.  (?)
From what I can see, I'm probably just going to end up hacking simple_grabber, publish my own data stream of "theta / dist" values, and write my own visualiser on the other side, by the looks of it.

May I ask what you did to get the SDK to compile on your rp-A1M8 lidar? I'm still stuck trying to fix the files.

Hi,  I got it compiling after changing this bit https://github.com/Slamtec/rplidar_sdk/pull/66/files


Reshape arrays inplace

TODO: in the case of rtensor

assign the shape to m_shape
prevent dimension change.

Great!


exp/services/recoverysigner: add docs for how to setup unique signing keys

What
Add docs to the README for how to setup unique signing keys in recoverysigner using the new logic in the recoverysigner-unique-signing-keys breanch.
Why
Someone should be able to get pretty far using the usage text but it will be helpful to have some clear docs that step-by-step how to setup the recoverysigner with unique signing keys.

Done in #2807.


Keep chapter height consistent when tabbing (Chapters page)

The height of each tab content is different. So when tabbing, the layout changes (medium-width screens). It should not. Best way to solve this is to place add a container with a the min-height set to the taller height of both tab contents. This requires us to render both (also the one not shown to the user at the moment). Not elegant (let me know if you have a better proposal - fixing the height does not work across responsive screen sizes and dynamic content), but the current layout jump is pretty unpleasant.
Current behavior:

Expected behavior is that the height remains the same, even when tabbing to the challenges.

I have created a PR for this issue.


Implement the new revamped CSV data module design

There is API design changes in the APIS of the CSV data module.
The new design can be found in here.
https://docs.google.com/document/d/18r8PIBqI8C3H71VwJMFDb3xe9yCFgm9rS2Vx7tdTVaQ/edit?usp=sharing
This issue is created to track the new implementation

Implemented by https://github.com/ballerina-platform/module-ballerina-data.csv/pull/22


go-licenses command is not recognized

go-license command is not recognized even though go files were moved to /usr/local.

You can refer to https://go.dev/doc/tutorial/compile-install


App Only Authentication

Hi,
Does it lack something for using app only Authentication with your package ?
Could you provide to me an example of use ?
According to the documentation, the body of the request must have "grant_type=client_credentials"
I can't find where this parameter is added to the request ...
Thxs

@RouxRC Thanks for this workaround. I just want to note that this line:
tokenclient.oauth2.token(grant_type="client_credentials")
returns a string of a dict instead of a dict itself. So it needs to be passed into json.loads() to work.

Ha yes indeed, I also do that in personal pieces of code, I'll edit the code, thanks for the feedback!


Fix sorting when displaying provider's hosts (list view)

We need to post the ajax request to correct url:
post /ems_infra/show/...

rather than:
post /ext_management_system/show/...

https://bugzilla.redhat.com/show_bug.cgi?id=1215272

@h-kataria You're right, I fixed the other occurrences of the problem as well.

@h-kataria Fixed the containers part as well.
We need to use "#{@layout}", since we're in a javascript section and a plain @layout would just
give us a syntax error.

verified in UI. merging


[cryptotest] Add P-384 targets to cryptotest.

These were initially excluded because we didn't support P-384 in cryptolib yet, but now that P-384 support is present we should test it. Note that there were some tests with SHA-256 in the original test suite that was commented out, but I removed them: since cryptolib doesn't let you use a hash function with collision resistance weaker than the curve security level these will always fail.
Verified locally that the ECDH and ECDSA targets pass on FPGA.
Follow-up after noticing it was missing while making some changes, see https://github.com/lowRISC/opentitan/pull/25391

Backport depends on https://github.com/lowRISC/opentitan/pull/25391 which depends on https://github.com/lowRISC/opentitan/pull/25390


fix(filesystem): UNC paths are not supported

Towards #1175

@rudolfix, that's a couple of cunning fixes. I'd say I didn't find ideological errors - it's just about a couple of symbols (normal ones), which make fsspec (not our code) to turn the path into something weird. Can't say I like it, but it worked. Pushed a test in verified-sources: https://github.com/dlt-hub/verified-sources/pull/423

@rudolfix, hm, I don't think it actually works. Surprisingly, when you try to read one file, it goes fine. But if you give a glob, fsspec does path translation of some sort, and always adds a disk name into the start, creating a pattern like:
(?s:[\\\\/][\\\\/]localhost[\\\\/][\\\\/]C\\$[\\\\/]git_reps[\\\\/]dlt[\\\\/]tests[\\\\/]common[\\\\/]storages[\\\\/]samples[\\\\/].*)\\Z
Here:
https://github.com/fsspec/filesystem_spec/blob/37c1bc63b9c5a5b2b9a0d5161e89b4233f888b29/fsspec/utils.py#L742
So, basically, it can see files by UNC path (I logged, it really does), but it filters them out, not returning anything. Not sure if it can be overridden...
However, if I give a file path (not a glob), it reads the file correctly, not going to this part of code, which does the path translation.
 But UNC - no.

@rudolfix, the only way I see how to fix it, is to write our own filesystem, inheriting the fsspec.LocalFileSystem and override the glob method to fix this translation. As there are no conditions in the translation, there is no way to skip it for globs (it's skipped only for precise paths). What do you think?

@IlyaFaer

@rudolfix, the only way I see how to fix it, is to write our own filesystem, inheriting the fsspec.LocalFileSystem and override the glob method to fix this translation. As there are no conditions in the translation, there is no way to skip it for globs (it's skipped only for precise paths). What do you think?

heh this is so bad :) could you check first if regular Python glob works correctly? then my take would be to use it for file:// and just translate the results to something fsspec understand. but let's check it out.

@rudolfix, yes, Python works fine, I tried it even before started to look at fsspec

translate the results to something fsspec understand

I don't think it's possible. Because, if we read localhost, it's okay, we can translate the path. But what if we read a network entity? It'll start with an ip address - it can't be translated into a path, which starts with a disk name. So, fsspec will add it into the pattern by itself.

@rudolfix, uf-f, that's quite a big re-writing. But I think I understood it

@IlyaFaer it was actually impossible to correctly implement all file paths styles on all platforms using fsspec. what this PR does is to use Python glob for all file:// uris. it also converts all file:// uris to native paths before opening. somehow fsspec is able to open UNC paths, but it is not able to do that when they are in file:// form.
as usual tests are way more complicated than code...


Base32 decryption and ROT13 decryption cannot occur at the same time

Base32 decryption and ROT13 decryption cannot occur at the same time

Closing this because it's a duplicate of #606. There's a bug where Caesar doesn't work with encodings.


syslog not availble for windows

i try to build krakend for windows, but building failed.
I just recognise, as far as i tested it, the only reason why krakend-ce build failed for windows is, that the logging is using syslog, which is not availble for windows and plan9.
I would like to have a krakend as a windows binary.
(I just dig a little deeper in this issue and implemented a little work around, 2 src files with different build tags, one for windows without syslog and one for the rest.)

Hi @willie68, couldn't you issue a PR for your changes because it is useful to have krakend-ce running on Windows?

Hi,
i already did this, see https://github.com/devopsfaith/krakend-gologging/pull/4

Sorry, my bad. I didn't notice it because your PR is closed already. You closed it because the maintainer doesn't want to support Windows?
Anyway, it does seem like syslog support have been removed from log.go for non-Windows built, as mentioned in https://github.com/devopsfaith/krakend-gologging/pull/4#issuecomment-601115294.

Yes, that was the main reason.
But can't see any changes on the master branch... But, if you need this, you can simply use the files on my fork.

before closing this, I want to extend my point at the linked discussion: we don't have the resources to keep all the features available on every OS/arch or to manage several production-ready versions with different sets of features/behaviours. that's why we decided to just support gnu/linux environments for production and gnu/linux and osx envs for devel.
this is not me telling you "you should not use windows", this is me suggesting you "if you're on windows, try krakend via docker".

Thats ok for me. No problem. The idea behind windows support for me ( and i think i'm not the only one)  is, that i had to develop software on windows (Company policy). On our side it is not planned to run this software productively on Windows. Our productive cloud stack is based on kubernetes/docker, so everything is fine.


Ability to encrypt to a per-row symmetric key

Hi again,
We want to symmetrically encrypt certain data to a key that's provided by the model object.  Particularly, we're encrypting it to a secret answer known only to the user.   We'd like to encrypt it without the asymmetric encryption - looks pretty easy to adjust in Strongbox::Lock.  Would you entertain a patch for that?
For background, we're going to encrypt the secret answer using the normal method, and in the security perimeter we'll be able to then conduct operations that require the decrypted answer - like encrypting new bits of data using it as a symmetric key.  Then, at times when the user chooses to enter their secret answer, we can provide some limited decryption for that user's need, while preventing mass exposure of decryptable content to an attacker.
Thoughts?
Randy

This sounds really useful, and it's similar to a change I'd likely need to add myself, to a project.  Anything ever come of this?

David, the changes are there in my fork.  I suggest reviewing the changes in detail to determine whether they meet your needs.


面包屑中的一级菜单建议设置成不可点击

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
A clear and concise description of what you want to happen.
点击 详情信息 跳转至404
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
src\layout\components\sidebar\breadCrumb.vue
Additional context
Add any other context or screenshots about the feature request here.

src\layout\components\sidebar\breadCrumb.vue

这属于你的需求 自行更改


[Event Request] report 86 "Adjust Add. Reporting Currency" - OnSetFilterOnClosedVATEntries

Hello, can you please add a new Event OnSetFilterOnClosedVATEntries in report 86 "Adjust Add. Reporting Currency" ?
It is for the trigger OnPreDataItem on the "VAT Entry" Dataitem.
        dataitem("VAT Entry"; "VAT Entry")
        {
            DataItemTableView = sorting(Type, Closed) where(Type = filter(Purchase .. Sale));

            trigger OnAfterGetRecord()
            begin
                VATEntryCount := VATEntryCount + VATEntryStep;
                if VATEntryOldCount div 100000 <> VATEntryCount div 100000 then begin
                    Window.Update(1, VATEntryCount div 100000);
                    VATEntryOldCount := VATEntryCount;
                end;

                "Additional-Currency Amount" := ExchangeAmtLCYToFCY("Posting Date", Amount, false);
                "Additional-Currency Base" := ExchangeAmtLCYToFCY("Posting Date", Base, false);
                "Add.-Currency Unrealized Amt." := ExchangeAmtLCYToFCY("Posting Date", "Unrealized Amount", false);
                "Add.-Currency Unrealized Base" := ExchangeAmtLCYToFCY("Posting Date", "Unrealized Base", false);
                "Add.-Curr. Rem. Unreal. Amount" := ExchangeAmtLCYToFCY("Posting Date", "Remaining Unrealized Amount", false);
                "Add.-Curr. Rem. Unreal. Base" := ExchangeAmtLCYToFCY("Posting Date", "Remaining Unrealized Base", false);
                Modify();
            end;

            trigger OnPostDataItem()
            begin
                Window.Close();
            end;

            trigger OnPreDataItem()
            var
                GLSetup: Record "General Ledger Setup";
            begin
                Window.Open(Text002Txt);
                if Count > 0 then
                    VATEntryStep := 10000 * 100000 div Count;

                GLSetup.Get();
                IsHandled := false;
                OnSetFilterOnClosedVATEntries("VAT Entry", IsHandled);
                if not IsHandled then
                    if not GLSetup."Unrealized VAT" then
                        SetRange(Closed, false);
            end;
        }

    [IntegrationEvent(false, false)]
    local procedure OnSetFilterOnClosedVATEntries(var VATEntry: Record "VAT Entry"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13591.


Redirect users to overview page if the lobby was closed

https://github.com/sopra-fs23-group-40/Client/commit/43f8bd1cc60afa5e034239d422d736d2fee4b486


Add file that was missing from 8496

When #8496 was merged, it was missing an image file. This change adds it.

/azp run mrtk_docs


Camera / audio is still in use even after the stage is closed or leave

Hi,
i am using the demo app and its working fine only one issue we noticed during the testing, the camera / audio is till in use even after we leave the stage i am using the basic broadcast sample and then while testing our tester found even after we exit or leave the call the camera/audio is in access, the problem is in the sample (sample code) also. How we tested is that after leaving the stage and backpress and closed the stage activity, After that  we open phone camera or screen recorder and try to record a video then the system showing we cant record voice as its used by our app, so after destring the app completely from task bar we were able to record the video with sound, So please help to solve this issue.
on destroy we used the following codes
override fun onDestroy() {
      clearPreview()
      leaveStage()
      viewModel.release()
      super.onDestroy()
  }

clearPreview()
private fun clearPreview() {
      binding.participantListLayout.removeAllViews()
      binding.opponentView.removeAllViews()
      binding.userView.removeAllViews()
  }

leaveStage()
   private fun leaveStage() {
        viewModel.leaveStage()

    }

@mboulin Please give a help as we are at deadline of our release

The issue you are facing is likely due to the camera and microphone resources not being properly released after leaving the stage.
Could you try releasing the DeviceDiscovery instance: The DeviceDiscovery class manages the local device resources like camera and microphone. You should release this instance when you are done using it. Add the following line in your onDestroy() method:
override fun onDestroy() {
    clearPreview()
    leaveStage()
    viewModel.release()
    deviceDiscovery?.release() // Add this line
    deviceDiscovery = null // And this line
    super.onDestroy()
}

You may try to also release the Stage instance in your leaveStage() method.


Why requestWillBeSent and responseReceived has no "timing" datas, or why they have -1 data if there are "timing" datas?

Component
Version

Operating system
Windows 7 Ultimate, 64bit

Node.js
Node.js 8.9

Chrome/Chromium/...
Chrome 69

chrome-remote-interface
0.26.1

Is Chrome running in a container? YES
Hello, Im getting all requests of a website with "requestWillBeSent" method, all the request are come in but timing datas got "-1" value. Why this is happening? How can i calculate or get all timing datas successfully?

I don't see any timing data in Network.requestWillBeSent.
The ResourceTiming object (response.timings in Network.responseReceived) have its values set to -1 when they are not available/do not apply, e.g., proxyStart will be -1 if no proxy is used.
Also this FAQ concerns Chrome running in a container.

I don't see any timing data in Network.requestWillBeSent.
The ResourceTiming object (response.timings in Network.responseReceived) have its values set to -1 when they are not available/do not apply, e.g., proxyStart will be -1 if no proxy is used.
Also this FAQ concerns Chrome running in a container.

OK, thanks for your response. It helped too much. I have a new question about this timing issue;
I want to do a waterfall chart with reliable datas like, Google DevTools Network Waterfall but I don't get all timing datas that Google gets. Why is this happening? How can I get that datas like Google's Waterfall for the same target URL request/response?

The timing computation is not a straightforward process, see my attempt here in chrome-har-capturer.

hello ,I'm a novice，I want to get all data of network tab in chrome dev,header,response,etc..,Use the Chrome DevTools Protocol method to obtain the data,not inchrome

@chen1-CN take a look at chrome-har-capturer.


Allow the replacement of execpath

In order to use expand_location we need to have a dep on the input.
https://docs.bazel.build/versions/master/skylark/lib/ctx.html#expand_location
Private module maps used in swift library that were previously not
deps, and they are specified now specified as header, so they remain
private. If we fix swiftc_inputs to be expandable in Bazel this would
be unnecessary.
For xcode project generation we need to call expand_location and the
aspect will fail if it doesn't have it as a dep. The corresponding code is added
to Tulsi and XCHammer https://github.com/bazelbuild/tulsi/pull/157

Would it be good to address the failures in your Tulsi PR before landing this?

Hey @garrettmoon - we should be good to land as tulsi takes these build files as an input, so as long as they match what it's expecting it will work ✅


CPCIR-Alvis: (some) jobs are not terminating until time limit is reached

The current jobs I am submitting to Alvis to process CPCIR can experience the following: processing completed, but the job does not finish, so it keeps the resources unavailable for further processing or other users until the time cap is reached. The stats below show an example:

(That the GPU memory usage is this low is related to how the job script is organized and that I was processing --failed files. When processing 'a first pass' the memory usage is very high.)
I think there might be some 'ghost' processes running preventing the job to terminate, particularly because there is a small bump in memory usage around 09:45 without an increase in CPU in the example shared.
I need to highlight that, while I detected this to happen usually, I also saw this to not happen... In the approach I used in the past, which was 100% offline, I didn't experience this. Therefore I will first investigate using only files available locally, given that our connections to the CPCIR data server already caused issues elsewhere (issue #65).
Since this issue keeps the resources busy but unused, I halted the processing of CPCIR until this is resolved.

I tried doing processing offline in the cluster (code not pushed to the GitHub main repo, but in a fork here). The same scripts did not suffer and did suffer this issue. Therefore it can be difficult to identify what is causing this issue, but at least that it's something depending only on "the Internet connection" can, in principle, be ruled out.

Replying to my question above: I should probably have noticed the use of a lock, introduced as a hard-coded value with 9fb0aa49541358eeefa7c23d0a501b8e73408663, which prevents the GPU to execute more than one prediction simultaneously, even if --n_processes is greater than 1. I don't remember why this was introduced, but maybe to avoid CUDA OOMs?
Since there's a new higher time limit in the cluster, I re-wrote the slurm bash scripts making use of it as well as not imposing a GPU device lock (i.e. ccic.bin.process.py:533 set to None), and running with --n_processes 3.
For now it seems to work, but this needs to be run for a while to detect if this solves the issue. However, in the meantime, we get more files processed without underusing the resources (currently, the average GPU usage is almost 100%, with 54% GPU mem used).

Related, to not open a new issue at the moment.
The processing can halt with neither any processing happening nor the job terminating if, for any reason, an input file is present in the --input_path but this file cannot be opened, i.e. it is corrupt. The following is the error message:
[ERROR] (ccic.bin.process): Downloading of input file 'merg_2016010102_4km-pixel.nc4' failed.
Traceback (most recent call last):
  File "/opt/conda/envs/ccic/lib/python3.10/site-packages/ccic/bin/process.py", line 319, in download_files
    input_file, clean_up = input_file.get()
  File "/opt/conda/envs/ccic/lib/python3.10/site-packages/ccic/processing.py", line 107, in get
    return self.file_cls(output_path), False
  File "/opt/conda/envs/ccic/lib/python3.10/site-packages/ccic/data/cpcir.py", line 243, in __init__
    with xr.open_dataset(self.filename) as data:
  File "/opt/conda/envs/ccic/lib/python3.10/site-packages/xarray/backends/api.py", line 553, in open_dataset
    engine = plugins.guess_engine(filename_or_obj)
  File "/opt/conda/envs/ccic/lib/python3.10/site-packages/xarray/backends/plugins.py", line 197, in guess_engine
    raise ValueError(error_msg)
ValueError: did not find a match in any of xarray's currently installed IO backends ['netcdf4', 'scipy', 'gini', 'zarr']. Consider explicitly selecting one of the installed engines via the ``engine`` parameter, or installing additional IO dependencies, see:
https://docs.xarray.dev/en/stable/getting-started-guide/installing.html
https://docs.xarray.dev/en/stable/user-guide/io.html
Exception in thread Thread-1 (download_files):
Traceback (most recent call last):
  File "/opt/conda/envs/ccic/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/opt/conda/envs/ccic/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/opt/conda/envs/ccic/lib/python3.10/site-packages/ccic/bin/process.py", line 325, in download_files
    processing_queue.put((input_file, clean_up))
UnboundLocalError: local variable 'clean_up' referenced before assignment

The processing of CPCIR is moving forward, but using the hack from https://github.com/SEE-GEO/ccic/issues/67#issuecomment-1883218373 is not solving the issue, below an example:

The job has completed the processing of all files it had to process, I have not identified any warning or error in the logs, but the job does terminate; it's consuming a considerably large amount of memory.
I think there is definitely something happening with the way we have structured the code that causes this to not exit nicely, and it's difficult to asses what is it in a non-interactive setup such as the cluster.
It is probably related the step behaviour in the memory that is seen in the previous screenshot as well as in the one below (which will end up in an OOM if it takes one more step):
.
I honestly have no idea what can be causing this issue (looks like a memory leak or zombie processes). Instead of spending more time trying to fix this, I think I prefer to spend some time "babysitting" the cluster process and killing them manually, if needed.

Logging a bit more information:
Perhaps the large amount of memory is not an issue. I 'read online' (I lack the source now to reference it) that this consistent increase in memory can be normal when working with large datasets and it might be due to how Python-the OS manage the memory space. Below screenshots of three jobs that have been running for almost 4 days now:

Looks like the system is smart: no 'Killed.' messages in the logs or any other error related to this memory consumption near the edge.

I had a look at the code and suspect that this  this line can cause a race condition which may cause the download thread to not terminate. Can you try removing this line and see if the problem persists?

Thanks for the hint. I have removed this line and I am trying it.

Removing that LOC seems promising. Three jobs that terminated finished as they should. We need to wait more, though.

And we waited more and removing that LOC does not completely solve the issue, at least for one job 1783582, but during this job 301 files failed to download!
I still think though that was an improvement, so it should end up in main.

Update: after all CPCIR data for 1998-2023 has been processed, I confirm that removing the suggested LOC (in #87) does not completely solve the issue but it seems to mitigate it; there were less idle jobs when removing this LOC. It cannot be ruled out that the root of the problem is somewhere else.


Update Threat-Modeling-Cloud-Migrations.md

Improved existing content, added new...

nice :)

@dpaic I just tweet about it https://twitter.com/DinisCruz/status/871756615434854400


Error ReturnAmountToDrop():

Hi I just wanted to see if you have any info on this error when I try to start the airdrop ?

Hi @paranormal39 did you receive any feedback on this error? I am also replicating it

No I have not I reached out to the developers with no luck I can try again here shortly and ill update when I hear somthing


ALTER DATABASE ... WITH NO_WAIT

I am learning more about ALTER DATABASE [AdventureWorks2017] SET RECOVERY options. When I open the database properties dialog box, go to options page, change the Recovery model from Simple to Full, click the Script action to new query window, SQL Server 2017 SSMS v18.2, creates a script:
ALTER DATABASE [AdventureWorks2017] SET RECOVERY WITH NO_WAIT
I didn't see "WITH NO_WAIT" in the ALTER DATABASE documentation.  It would be nice if it was there.
Best regards,
Jody

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

ID: 8d76ab6e-5a3d-94eb-26f9-b2805b35ea85
Version Independent ID: 2ac43d27-c318-3e32-1e78-55229975eb4e
Content: ALTER DATABASE SET Options (Transact-SQL) - SQL Server
Content Source: docs/t-sql/statements/alter-database-transact-sql-set-options.md
Product: sql
Technology: t-sql
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

In this file: https://docs.microsoft.com/en-us/sql/t-sql/statements/alter-database-transact-sql-set-options, search for "no_wait"

#please-close

Augh! Sorry I missed that! Thank you for your quick response.


Upgrade to NUKE 8.1

This gives new, non-deprecated, GH actions workflow versions in workflow generation.

Thank you!


Wanted: isCrawler()

For reasons similar as https://gist.github.com/Stanback/6998085 I would love to see an isCrawler() function in this package, so that I can code some special exceptions in my single page app.
Thanks!

I would like to use this module, but i need isCrowler too.Are the function planned to enter in future versions?
Thanks

This would be helpful for me, too. I think, for now you could use family: 'Spider', right?
e.g.
> useragent.parse('Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)').device.family === 'Spider'
true

> useragent.parse('Mozilla/5.0 (compatible; bingbot/2.0; +http://www.bing.com/bingbot.htm)').device.family === 'Spider'
true

yeah, something like that would be cool

There is isbot, but I think this library would be more accurate.

thx, yes it is


Explicitly set DetailSlider slot content to fill available space below buttons.

Temporary workaround intended to allow post editor to fill vertical space until we revamp slider to use flexbox or something else cleaner for extensibility.
I do intend to explore flexbox instead in the short term.
Part of https://github.com/PolymerLabs/arcs/issues/823

@sjmiles already landed a patch that does this with flexbox.


Timeout should be handled by the http client not the library

Having timeouts with requests doesn't propagate through links and in general has caused weird behaviors such as causing timeout on app start (even though manually setting timeout to 60 sec). As it's not getting passed through the links as an exception it's no reasonable way to have it trigger a RetryLink either.
https://github.com/zino-hofmann/graphql-flutter/blob/59bb1d5bac4890c0025f38c385dce20a3b600c03/packages/graphql/lib/src/core/query_manager.dart#L260 removing the timeout here and controlling the timeout on the http client gave the correct response with it propagating through the links for retries and error reporting.
To Reproduce (MUST BE PROVIDED)
Make a request that is longer than the timeout (you can set the timeout to 1 sec) and check if ErrorLink can print it.
Expected behavior
timeout should be controlled through the http client and timeouts should pass through the links.
device / execution context
iOS/Android

Also, having 5 seconds as a default timeout duration is a very weird choice. Any request on a slow network will cause timeouts that is not logged through ErrorLink etc.

I agree! I was surprised by this today.
But my timeout implementation was done as an HttpLink. If anyone is interested (as is):
import 'dart:async';
import 'dart:math';

import 'package:common_dart/core/offline/connectivity_cubit.dart';
import 'package:graphql/client.dart';
import 'package:graphql_flutter/graphql_flutter.dart';
import 'package:http/http.dart' as http;
import 'package:logging/logging.dart';
import 'package:rxdart/rxdart.dart';

class HttpTimeoutRetry extends HttpLink {
  HttpTimeoutRetry(
    super.uri,
    this._connectivityCubit, {
    this.requestTimeout = const Duration(seconds: 15),
    super.defaultHeaders = const <String, String>{},
    super.useGETForQueries = false,
    http.Client? httpClient,
    super.serializer = const RequestSerializer(),
    super.parser = const ResponseParser(),
    super.followRedirects = false,
  }) : super(httpClient: httpClient);

  final Logger _loggerService = Logger('HttpTimeoutReTry');

  final Duration requestTimeout;
  final int maxRetries = 3;
  final ConnectivityCubit _connectivityCubit;

  @override
  Stream<Response> request(Request request, [NextLink? forward]) async* {
    Duration currentTimeout = Duration(microseconds: requestTimeout.inMicroseconds);
    int retries = 0;

    final Stream<Response> timeoutStream = RetryWhenStream<Response>(() {
      // only retry queries, to avoid issues with the current mutation/command logic
      if (request.isQuery) {
        return super.request(request, forward).timeout(currentTimeout);
      } else {
        return super.request(request, forward);
      }
    }, (Object error, StackTrace stack) {
      if (error is! TimeoutException) {
        return Stream<void>.error(error, stack);
      }

      if (retries < maxRetries) {
        retries++;
        // after the backend fixes, the timeout should still exists, but only for extreme scenarios
        // so, we put only 3 re-tries, adding 15 seconds on each.
        //if (retries > 3) {
        currentTimeout = Duration(seconds: currentTimeout.inSeconds + 15);
        // } else {
        //   currentTimeout = Duration(seconds: currentTimeout.inSeconds + 2);
        // }
      } else {
        return Stream<void>.error(error, stack);
      }

      // we will retry
      _loggerService.fine(
          'RETRYING, waiting: $requestTimeout, current $retries of $maxRetries - ${request.operation.operationName} - online?: ${_connectivityCubit.state}');

      // check for connectivity
      if (_connectivityCubit.state is OfflineState) {
        _loggerService.fine('waiting connectivity');
        // if offline, only retry after it is online again
        return _connectivityCubit.stream.where((ConnectivityState e) => e is OnlineState);
      }

      // if online:
      // avoid herd issues (if a server goes down, some clients might wait different moments to retry) -random, from 0 to 500ms
      return Rx.timer<void>(null, Duration(milliseconds: Random().nextInt(500)));
    });

    yield* timeoutStream;
  }
}

I just spent all day trying to figure out why DevTools was showing me a query response but the client had an exception with no data. Thank you a million times for helping point me in the right direction!

Please, publish the "graphql-v5.2.0-beta.9" version to the pub.dev, to allow us to change the timeout duration, ASAP.

The addition of this timeout has caused most of my unit tests to now fail regardless of the timeout set.

There is someone that can open a PR to fix this issue?

5 seconds as a default is indeed way to low. I really had to dig deep to get here
you can fix it for now by forcing graphql to beta.8 in pubspec.yaml:
  graphql: 5.2.0-beta.8

i am using graphql_flutter: ^5.2.0-beta.6 still getting that 5 second issue. Unable to process graphql queries. Any solution?

@woutervanwijk @mohammad-quanit what about https://github.com/zino-hofmann/graphql-flutter/issues/1457#issuecomment-2370960923 ?
It is working with the latest one?

@vincenzopalazzo  I am using graphql_flutter package and installed the latest version but still getting this issue.
QueryResult(source: QueryResultSource.network, data: null, context: Context({}), exception: OperationException(linkException: UnknownException(TimeoutException after 0:00:05.000000: No stream event, stack:<…>

@mohammad-quanit probably you have to change the default timeout?

@vincenzopalazzo  can you guide me how? where to change timeout?

Try to see if this solve your problem https://github.com/zino-hofmann/graphql-flutter/pull/1447/files
if now a PR will be welcome that specify your specific use case :) I am trying to be responsive in PR review and library release.
Hoping to drop the v5.2 at the end of the year and drop the beta

Any update here? Sounds like the options are to ditch the graphql-flutter package in favor of the graphql package (since it offers a timeout) or ditch graphql and go rest for dart. It doesn't sound like Apollo is going to be working on a proper dart package anytime soon so it would be great for this to get fixed.

@marc-wilson is https://github.com/zino-hofmann/graphql-flutter/pull/1475 solving your problem?
If yes I could release another beta version today

@vincenzopalazzo It doesn't appear there has been any new versions released within the last 23 months for either graphql_flutter or graphql packages. If you could make the version available on pub.dev, I will pull it down and give it a shot.


MWPW-147683 - CSS Cleanup

Final pass on the css
Edits to markup where necessary

Resolves: MWPW-147683
Test URLs:

Before:  https://quiz-entry-block--milo--adobecom.hlx.page/drafts/colloyd/quiz-entry/?martech=off
After: https://card-and-input-css--milo--colloyd.hlx.page/drafts/colloyd/quiz-entry/?martech=off

Testing notes:
The css should match the comp (find the link in the Jira ticket) in mobile, tablet, and portrait. This includes card states (selected, keyboard focus, etc.), the ML input, with it's autocomplete box and all of the text sizes and spacing. I did do some work to bring the carousel more in line. I'm not sure how animating it is possible, given how the cards are dynamically added or removed when the slides are advanced, so that not present. Finally, I tested this in FF and Safari to be sure that everything  holds up and it does.

When I select the card, I found the while small stuff on the top right corner of the card, do you know why?

 When I select the card, I found the while small stuff on the top right corner of the card, do you know why?

Because it's in the figma design for button selected states.

I see two border here, can we have only one?

 I see two border here, can we have only one?

The blue "border" is the outline from keyboard navigation. I've opted for the same style, offset by 2px outside of the element. Technically, there's 2 borders on buttons, selected cards etc. but that's the point of a keyboard nav indicator. This is not an issue, it is expected.

 I see two border here, can we have only one?

The blue "border" is the outline from keyboard navigation. I've opted for the same style, offset by 2px outside of the element. Technically, there's 2 borders on buttons, selected cards etc. but that's the point of a keyboard nav indicator. This is not an issue, it is expected.

You check this, https://helpx.adobe.com/search.html, I like the clear border

The quiz-directions is not changed when we switch to tablet view

under mobile view, I only got two cards, per design, we will show the same as tablet and desktop

 The quiz-directions is not changed when we switch to tablet view

To be clear, I did not change anything about how the carousel functions, the number of cards shown at different device widths and the core functionality are unchanged. In short, these are not bugs related to this pr.

 I see two border here, can we have only one?

The blue "border" is the outline from keyboard navigation. I've opted for the same style, offset by 2px outside of the element. Technically, there's 2 borders on buttons, selected cards etc. but that's the point of a keyboard nav indicator. This is not an issue, it is expected.

You check this, https://helpx.adobe.com/search.html, I like the clear border

Addressed the border-offset on the field so it's only got one border. I gave similar treatment to the carousel arrows.


Add page for difference between free/paid cluster and enterprise plan on bmx

The website has been archived; closing.


[5.1] Adding support for StreamOptions

5.2 ist already supporting this. Since 5.1 uses the same SwiftMailer Version, we should support this too.
Needed for PHP 5.6 because in combination with OpenSSL PHP 5.6 denied using self-signed certificates, as long as it was not specified separately.

We don't normally backport features into 5.1, but I think in this case, we should probably make an exception. // cc @taylorotwell


ntp: added translations

Asana Task/Github Issue:
Description

Testing Steps

Checklist

Please tick all that apply:

[ ] I have tested this change locally
[ ] I have tested this change locally in all supported browsers
[ ] This change will be visible to users
[ ] I have added automated tests that cover this change
[ ] I have ensured the change is gated by config
[ ] This change was covered by a ship review
[ ] This change was covered by a tech design
[ ] Any dependent config has been merged

#1279  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Simple calculator

Type of change

[x] Added new project
[x] Bug fix
[x] New features
[x] Enhanced documentation

Changes proposed in this pull request

Project name :

Simple calculator

Short description :

Simple calculator with simple operations that can recieve more functions :)
Checklist

[x] The code is Python 3 compatible
[x] I have added relevant comments in my file to make code understandable
[x] I have added my project in a folder which includes all the necessary files

Already exists : https://github.com/Mrinank-Bhowmick/python-beginner-projects/tree/main/projects/Calculator


Fixes NeuralQuery Schema

Description
Fixes NeuralQuery definition based on the documentation (and usage of OpenSearch).
Issues Resolved

query_image: format is not required since this is Base64 encoded image.
model_id: is required if there is no default. thus we cannot enforce it here.
query_text or query_image: at least one of them needs to be specified (both allowed).

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

Hello @dblock & @Xtansia , I been trying to add a test for query_image. The query is not a problem, but was not able to find any OpenSearch-provided pretrained model with image processing support that I can use.
If anyone know how what I can use here will appreciate it.

Hello @dblock & @Xtansia , I been trying to add a test for query_image. The query is not a problem, but was not able to find any OpenSearch-provided pretrained model with image processing support that I can use. If anyone know how what I can use here will appreciate it.

Let's ask @navneet1v and try the #ml channel on the public slack.
Is that required to make the API call successfully? I understand it's required to get some search results, but as long as we can exercise the actual change here (fails before this change, passes after this change) we will be able to merge it.

There is no pretrained model from Opensearch side that can process images as per my knowledge. @martin-gaievski during your testing for neural search do you know any Opensource model you used? or we were always using some closed source models.

Thanks @kolchfa-aws for pointing us to https://github.com/opensearch-project/ml-commons/issues/2598

Hello @dblock & @Xtansia , I been trying to add a test for query_image. The query is not a problem, but was not able to find any OpenSearch-provided pretrained model with image processing support that I can use. If anyone know how what I can use here will appreciate it.

Let's ask @navneet1v and try the #ml channel on the public slack.
Is that required to make the API call successfully? I understand it's required to get some search results, but as long as we can exercise the actual change here (fails before this change, passes after this change) we will be able to merge it.

There is no pretrained model from Opensearch side that can process images as per my knowledge. @martin-gaievski during your testing for neural search do you know any Opensource model you used? or we were always using some closed source models.

We used closed source models only

Thanks everyone for your answers.
Based on the current conditions, I do not know what I can use to test query_image in neural query due to lack of available model to use.
I created a test, but logically fails using the existing model in tests/default/ingest/pipeline/neural_search.yaml.
If anyone knows what I can use to add a test great, I can give it a try.
Otherwise, not sure what else can be done to complete and merge this PR.
Thank you

Otherwise, not sure what else can be done to complete and merge this PR.

How about writing a negative test that only does search with a model that doesn't exist?
              - script_score:
                  query:
                    neural:
                      passage_embedding:
                        query_image: ...data
                        model_id: does-not-exist # TODO: OpenSearch needs ..., see https://github/...
                        k: 100

This should respond with a predictable error, but one that's not a validation error for query_image which is your fix.

This should respond with a predictable error, but one that's not a validation error for query_image which is your fix (test should fail without your fix).
Thanks, good idea. I tested:

If we have a wrong field name we get a 400 error parsing_exception.
If we set and invalid model_id we get a 404 error status_exception (Failed to find model).
Will create a test based on this.

@dblock , added the test case. I'm getting the following response and do not know how to skip the RESPONSE PAYLOAD SCHEMA.

The underlying issue is described in https://github.com/opensearch-project/opensearch-api-specification/issues/445. For now you should be able to add a 404 response to the search request/response like here. A trivial type: object schema will suffice.


Unknown compiler option '-c'.

After running yarn install and yarn start, this is the error i get:
> tsc-watch --onSuccess 'rollup -c'
error TS5023: Unknown compiler option '-c''.

I think I found the cause of the problem. After removing the -c flag, command 'rollup' is not recognised....
I am using windows and linux alongside and it seems to work on linux.
After replacing "tsc-watch --onSuccess 'rollup'" with "tsc-watch --onSuccess \"rollup\"" it suddenly started working.

Hi @foreggs! Is working here for me in MacOs as well... Could you send a PR with the solution that works on Windows as well? Also, if you can check if the rest of the commands are working on Windows that'd be great :)

These commands don't work on windows:
 {
    "start": "tsc-watch --onSuccess 'rollup -c'",
    "report-coverage": "cat ./coverage/lcov.info | coveralls"
  },

And here is how to fix that:
 {
    "start": "tsc-watch --onSuccess \"rollup -c\""
  },

When it comes to the 'cat' command, there is the 'type' command on windows. There is no cross platform equivalent. I'm not familiar with coveralls so I can't help you with that one.

The report coverage is meant to run only on a CI server (i.e.: Travis)  so I'm not worried about that one.
Thanks for your help ;)

#65

Many many thanks! Closed by #65


Chroot immediately shuts down seconds after booting

Please paste the output of the following command here: sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20190902190258~master:9a594a66
release: xenial
architecture: amd64
xmethod: xorg
targets: gnome
host: version 12499.66.0 (Official Build) stable-channel candy
kernel: Linux localhost 4.4.190-16770-gb3c36ab5af98 #1 SMP PREEMPT Thu Nov 14 23:21:08 PST 2019 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Please describe your issue:
When starting the chroot (sudo startgnome) it starts normally and gives the black screen that normally signifies it booting into the chroot, however, after a few seconds on the black screen, instead of loading the chroot GUI, it goes back to Chrome OS and the console is shutting down the chroot.
If known, describe the steps to reproduce the issue:

Start chroot

Log of what happens when starting chroot:
(notes demarked with angle brackets)
chronos@localhost / $ sudo startgnome
Entering /mnt/stateful_partition/crouton/chroots/xenial...
Unknown username "whoopsie" in message bus configuration file

_XSERVTransmkdir: ERROR: euid != 0,directory /tmp/.X11-unix will not be created.
< VT Switch Black Screen starts >

X.Org X Server 1.18.4
Release Date: 2016-07-19
X Protocol Version 11, Revision 0
Build Operating System: Linux 4.4.0-128-generic x86_64 Ubuntu
Current Operating System: Linux localhost 4.4.190-16770-gb3c36ab5af98 #1 SMP PREEMPT Thu Nov 14 23:21:08 PST 2019 x86_64
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 3584000 verity payload=PARTUUID=7f7ea7ec-b52e-3e4e-8ec9-dadf3ff7f4c1/PARTNROFF=1 hashtree=PARTUUID=7f7ea7ec-b52e-3e4e-8ec9-dadf3ff7f4c1/PARTNROFF=1 hashstart=3584000 alg=sha256 root_hexdigest=01643fea75e10ac954711c515c04d5f8852c96de5aac105c7a4e5930b9039adb salt=f22dff874df894f2136a998325612307f08a894201e26005c962b320bd24ef05" noinitrd vt.global_cursor_default=0 kern_guid=7f7ea7ec-b52e-3e4e-8ec9-dadf3ff7f4c1 add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic
Build Date: 10 August 2018  09:33:05AM
xorg-server 2:1.18.4-0ubuntu0.8 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.33.6
        Before reporting problems, check http://wiki.x.org
        to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
        (++) from command line, (!!) notice, (II) informational,
        (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Sun Dec 15 13:50:25 2019
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
Error org.freedesktop.DBus.Error.ServiceUnknown: The name org.chromium.LibCrosService was not provided by any .service files
xf86EnableIOPorts: failed to set IOPL for I/O (Operation not permitted)
Error executing command as another user: Not authorized

This incident has been reported.
crouton: version 1-20190902190258~master:9a594a66
release: xenial
architecture: amd64
xmethod: xorg
targets: gnome
host: version 12499.66.0 (Official Build) stable-channel candy
kernel: Linux localhost 4.4.190-16770-gb3c36ab5af98 #1 SMP PREEMPT Thu Nov 14 23:21:08 PST 2019 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
xrandr: Output eDP1 is not disconnected but has no modes
xrandr: Output DP1 is not disconnected but has no modes
xrandr: Output HDMI1 is not disconnected but has no modes
Kicking xrandr again
xrandr: Output eDP1 is not disconnected but has no modes
xrandr: Output DP1 is not disconnected but has no modes
xrandr: Output HDMI1 is not disconnected but has no modes
Kicking xrandr again
Running exit commands...
/usr/bin/xinit: connection to X server lost

waiting for X server to shut down Hangup
Error org.freedesktop.DBus.Error.ServiceUnknown: The name org.chromium.LibCrosService was not provided by any .service files
< VT Switch Black Screen Ends; I can see Chrome OS again >
(II) Server terminated successfully (0). Closing log file.

Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
chronos@localhost / $

@SirNapkin1334,
There was a previous issue with this, see:

https://github.com/dnschneid/crouton/issues/2923#issuecomment-280820569

But a fix was merged that resolved it. I guess it may be cropping up again, not sure, but the crouton devs will take a look at it when they get a chance.
In the meantime, you should be able to use the 'xiwi' xmethod to get around this issue, see below to add 'xiwi' to your chroot:

sudo crouton -n xenial -t xiwi -u

You'll also have to add the crouton integration extension.
Hope this helps,
-DennisLfromGA

Is there any way to make xiwi run in full screen or something? Because I don't like having to deal with Chrome OS on the outside, and would much prefer the VT-style OS switching (the only reason I use crouton is because I absolutely hate Chrome and Chrome OS). Perhaps I should compile crouton to get that fix, but there's no information about compiling crouton?

Alright then, updating the chroot fixed it. It also fixed audio and video playback, so I can watch Youtube on my chromebook again. However, I'm hijacking this issue because playback of Reddit videos (often embedded mp4, gif, gifv, etc.) does not work - the first frame loads but the video does not start when pressing the Play button. Do you know why this might be happening?

Actually, I'll just make a new issue. #4188


deploy(docker) fix(docker) - Updated Dockerfile production

Equals to #1494 but in production.

Coverage remained the same at 73.224% when pulling 796e929c76ea856bceb3fb65dd053d5bbd21126a on FedeG:update_docker_production into e3eafa6131ad799b77ada6e3bf1202fc686e0735 on meanjs:master.

I haven't tested either, but LGTM.
@codydaig Feel free to merge as long as nobody else has any feedback.


⚠ Bofrost.be has degraded performance

In 10533a0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 856 ms

Resolved: Bofrost.be performance has improved in 837a656.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3106

Questions ?
Please refer to the Backport tool documentation


DropdownTrigger now requiring onClick prop is a breaking change in 1.1.3

After upgrading we receive the following error
Warning: Failed propType: Required prop onClick was not specified in DropdownTrigger. Check the render method of ProFilterStatus
1.1.3 sets the prop for onClick in the DropdownTrigger.jsx file to be required. It was not previously required and we handled the events ourselves.
Sticking with 1.1.2 for now. Thanks!

I just released https://github.com/Fauntleroy/react-simple-dropdown/releases/tag/1.1.4 to resolve this issue. Looks like cloneElement and propTypes do not interact as I expected them to.


New FPGA oneAPI Sample for Dynamic Profiler tool

Signed-off-by: alisanikiforova<EMAIL_ADDRESS>Description
This commit adds a new FPGA sample that demonstrates the functionality of the Intel(R) FPGA Dynamic Profiler for DPC++ tool.
This tool can be used on FPGA designs to collect profiling information about the design as it runs on FPGA hardware in order to assist with design optimization.
The tutorial includes a README that explains how the tool can be set up and what data it can collect, as well as a simplistic design example that demonstrates how the data it collects can be used to locate areas for optimization.
Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[X] Command Line

Testing pipeline
Expected to fail:
INTEL/intel namespace/filename incompatibility of beta09/beta10

All test failures are expected due to beta09/10 syntax differences.


Constructor Injection?

Hey there,
Firstly it's been a great effort in producing such a great library, thank you
I was wondering if you have considered supporting constructor injection? (as opposed to only supporting property injection)?
I do realise it complicate things on how you use constructors for options etc but I think it would be so beneficial going forward
Reading about property injection in general a lot of people often say to avoid it:
https://stackoverflow.com/questions/38459625/property-injection-in-asp-net-core
_"Please note that property injection is considered bad in 98% of all scenarios, because it hides dependencies and there is no guarantee that the object will be injected when the class is created.
With constructor injection you can enforce this via constructor and check for null and the not create the instance of the class. With property injection this is impossible and during unit tests its not obvious which services/dependencies the class requires when they are not defined in the constructor, so easy to miss and get NullReferenceExceptions.
This might also help with my other issue:
https://github.com/bilal-fazlani/commanddotnet/issues/42
Let me know your thoughts on this?
The only valid reason for Property Injection I ever found was to inject services into proxy classes generated by a third party library, i.e. WCF proxies created from an interface where you have no control about the object creation.
Avoid it everywhere else."_

I agree. I never liked Property Injection either because of the same reasons as you mentioned. And as you pointed out, the only blocked in implementing constructor injection is global options.
Just thinking, if we could swap these two..? So use properties for global options and constructor for dependency injection.. That would be nice I think..

Yes i would be in favour of this change. Especially when the default Microsoft DI system only supports constrictor injection

I haven't started on this yet. Will make an attempt in next 2 weeks hopefully.

Awesome, appreciate your efforts

Hi all.  I like the direction this is heading.  It would be great to be able to let a DI framework ctor the command objects even when there are global args defined.  I was thinking about the global args defined as Properties and I think there's one downside to that approach.  If there are multiple global args and they're used to configure the system, a class won't know when all of the properties have been provided.   Using a method to capture the global args allows us to perform logic with them.   The use case of #55 is a good example.  The ctor was able to then configure a proxy based on multiple parameters.  Instead of properties, what if a method could be decorated with a GlobalAttribute, similar to the DefaultAttribute?  I think this gives us the best of both worlds.  And since this would be a breaking change, this design will make it easier to fix existing ctors.  The devs simply convert the ctor to the decorated method.

If there are multiple global args and they're used to configure the system, a class won't know when all of the properties have been provided.

@drewburlingame With current impl, a constructor can only have options and not arguments.
And when no value is provided to an option it falls back to its default value. Such as 0 for Int, false for Boolean etc. With this, we still wont know if all values are explicitly passed by user or not.
I am not completely sure of either approach. @drewburlingame would a pre command exec hook help the below use case?

Using a method to capture the global args allows us to perform logic with them.

@drewburlingame With current impl, a constructor can only have options and not arguments.
And when no value is provided to an option it falls back to its default value. Such as 0 for Int, false for Boolean etc. With this, we still wont know if all values are explicitly passed by user or not.

I misspoke.  You can replace all 'global args' with 'global options'

@drewburlingame would a pre command exec hook help the below use case?

Good question.  With #55, the pre-hook wasn't needed because the ctor served the same purpose.  With this change, the ctor would no longer work for that and a pre-hook would have value here.  That method could also be used to declare global options.
Since this would be a breaking change, you might want to toggle the feature with a feature flag in AppSettings and give people a version or two before removing the old feature.

With #55, the pre-hook wasn't needed because the ctor served the same purpose. With this change, the ctor would no longer work for that and a pre-hook would have value here. That method could also be used to declare global options.

I agree. Right now, none of the options seem "small changes"
Just thinking out loud now.. would it be sane (or even possible?) to use ctor for both global options and DI?

Hmmm...  The ctor will be called either by this framework or a DI container.  How will this framework supply the global options to the ctor when called by the DI?  I think they'd always have to be part of an IArgumentModel and that IArgumentModel would have to be registered with the DI framework.   In this case, we'd need PR #56 and we'd need to extract the logic that ctor's the command so it can be replaced by the DI container.    We'd have to collect all of the global options before resolving the command from the container to ensure the IArgumentModel is populated when passed into the ctor.
I keep going back and forth on this.  Overall, I think it's a reasonable approach when using a DI container.  Using ctor-for-both vs a pre-hook method seem like trade-offs in complexity and principal of least surprise.  It would help to write up a document of the pros & cons of each approach.

Just want to say this is a great discussion, and it's an important lesson on how various changes can impact different use case scenarios. I think there are other commandline NuGet packages out there, perhaps it might worth taking a look on how they might of solved the issues?

I too would be in favor of this change. Need to have a look at the code to get an understanding of the overall architecture and the changes needed to me made.

I've been thinking about this more.  I've been working on a design to implement a middleware pipeline.  The signature would take a context object with an Action next delegate.   I think we should move away from the ctor options and replace it with a "middleware" method in the class.  If the method exists, options can be defined by additional parameters in the method.    We can look for the method with the context parameter.  Only one would be allowed per class.

This is handled in the beta3/master branch.  You can see an example of the new command middleware method in the test here https://github.com/bilal-fazlani/commanddotnet/blob/beta-v3/master/CommandDotNet.Tests/FeatureTests/HostCommandMiddlewareOptions.cs#L245
I removed support for ctor options and try to resolve command classes via DI before creating with Activator.

@drewburlingame Interesting, will have to look at this further. Would be good to see an example of the constructor injection and this together say with the calendar sample. Perhaps injecting a calendar service etc

@dotnetshadow I hope to be working on the documentation next week.  In the meantime, here's a quick example.
public class Calendar
{
    private ICalenderService _calenderService;

    public Calendar(ICalenderService calenderService)
    {
        _calenderService = calenderService;
    }

    public void AddEvent(AddEventArgModel addEventArgs)
    {
        _calenderService.AddEvent(addEventArgs)
    }
}

and if in your example you previously used the ctor for ctor options, then the example would now look like

public class Calendar
{
    private ICalenderService _calenderService;

    public Calendar(ICalenderService calenderService)
    {
        _calenderService = calenderService;
    }

    public Task<int> Init(CommandContext ctx, Func<CommandContext, Task<int>> next, string someOption)
    {
        _calendarService.SomeInit(someOption);
        return next(ctx);
    }

    public void AddEvent(AddEventArgModel addEventArgs)
    {
        _calenderService.AddEvent(addEventArgs)
    }
}

the Init method will be detected as a middleware method because of the Func<CommandContext, Task<int>> parameter.  It will be executed for every command defined in the class.  With this pattern, you can wrap call to next with any exception handling and cleanup.  As an alternative to the Init middleware method, you can create a middleware component that would perform this init for any class that takes an ICalenderService in the ctor.

ctor injection is supported in 3.0.0