Bool If statments

Related to https://github.com/davidusb-geek/emhass/issues/154 , https://github.com/davidusb-geek/emhass/pull/174 , https://github.com/davidusb-geek/emhass/pull/175
Did a ruff Grep and found there is quite a flew boolean if statements, If The pr #175 does indeed to be the fix. A pull request could be done to replace all of these with if * == True: (Just to make sure this issue doesn't happen to anything else). If its not, or  unnecessary/irrelevant in any way feel free to close the Issue.
> grep --include=\*.py -Rn -P '(if[\s][^\s]+:)' .
./scripts/load_clustering.py:51:    if data_path.is_file():
./scripts/load_forecast_sklearn.py:59:    if data_path.is_file():
./scripts/optim_results_analysis.py:47:    if get_data_from_file:
./scripts/optim_results_analysis.py:75:    if show_figures:
./scripts/optim_results_analysis.py:77:    if save_figures:
./scripts/optim_results_analysis.py:85:    if show_figures:
./scripts/optim_results_analysis.py:87:    if save_figures:
./scripts/optim_results_analysis.py:109:    # if show_figures:
./scripts/optim_results_analysis.py:111:    if save_figures:
./scripts/read_csv_plot_data.py:133:    if save_figs:
./scripts/read_csv_plot_data.py:147:    if save_figs:
./scripts/special_config_analysis.py:148:    if data_path.is_file():
./scripts/use_cases_analysis.py:75:    if save_figures:
./scripts/use_cases_analysis.py:85:    if save_figures:
./scripts/use_cases_analysis.py:94:    if save_figures:
./scripts/use_cases_analysis.py:105:    if save_figures:
./scripts/use_cases_analysis.py:121:    if save_figures:
./scripts/use_cases_analysis.py:141:    if save_figures:
./scripts/use_cases_analysis.py:162:    if save_figures:
./scripts/use_cases_analysis.py:170:    if save_figures:
./src/emhass/optimization.py:134:        if self.optim_conf['set_use_battery']:
./src/emhass/optimization.py:170:            if self.optim_conf['treat_def_as_semi_cont'][k]:
./src/emhass/optimization.py:190:        if self.optim_conf['set_use_battery']:
./src/emhass/optimization.py:210:            if self.optim_conf['set_total_pv_sell']:
./src/emhass/optimization.py:217:            if self.optim_conf['set_total_pv_sell']:
./src/emhass/optimization.py:233:        if self.optim_conf['set_use_battery']:
./src/emhass/optimization.py:309:            if self.optim_conf['treat_def_as_semi_cont'][k]:
./src/emhass/optimization.py:323:            if self.optim_conf['set_def_constant'][k]:
./src/emhass/optimization.py:351:        if self.optim_conf['set_use_battery']:
./src/emhass/optimization.py:353:            if self.optim_conf['set_nocharge_from_grid']:
./src/emhass/optimization.py:361:            if self.optim_conf['set_nodischarge_to_grid']:
./src/emhass/optimization.py:369:            if self.optim_conf['set_battery_dynamic']:
./src/emhass/optimization.py:453:        if self.optim_conf['set_use_battery']:
./src/emhass/optimization.py:472:        if self.optim_conf['set_total_pv_sell']:
./src/emhass/optimization.py:480:            if self.optim_conf['set_total_pv_sell']:
./src/emhass/optimization.py:487:            if self.optim_conf['set_total_pv_sell']:
./src/emhass/optimization.py:504:        if debug:
./src/emhass/optimization.py:639:        Helper function to validate (and if necessary: correct) the defined optimization window of a deferrable load.
./src/emhass/retrieve_hass.py:105:                    if minimal_response: # A support for minimal response
./src/emhass/retrieve_hass.py:107:                    if significant_changes_only: # And for signicant changes only (check the HASS restful API for more info)
./src/emhass/retrieve_hass.py:181:            if load_negative: # Apply the correct sign to load power
./src/emhass/retrieve_hass.py:188:        if set_zero_min: # Apply minimum values
./src/emhass/retrieve_hass.py:327:        if self.get_data_from_file:
./src/emhass/retrieve_hass.py:334:        if response.ok:
./src/emhass/forecast.py:453:        if set_mix_forecast:
./src/emhass/forecast.py:584:            if self.get_data_from_file:
./src/emhass/forecast.py:608:                if filename_path.is_file():
./src/emhass/forecast.py:614:            if use_last_window:
./src/emhass/forecast.py:658:        if set_mix_forecast:
./src/emhass/web_server.py:126:    if args.addon==1:
./src/emhass/web_server.py:133:        if options_json.exists():
./src/emhass/web_server.py:140:        if use_options:
./src/emhass/web_server.py:144:            if options_json.exists():
./src/emhass/web_server.py:158:    if config_path.exists():
./src/emhass/web_server.py:181:    if args.addon==1:
./src/emhass/web_server.py:219:    if use_options:
./src/emhass/utils.py:54:    if save_to_file:
./src/emhass/utils.py:407:    if use_secrets:
./src/emhass/utils.py:419:    if use_secrets:
./src/emhass/command_line.py:72:        if get_data_from_file:
./src/emhass/command_line.py:104:        if get_data_from_file:
./src/emhass/command_line.py:136:        if get_data_from_file:
./src/emhass/command_line.py:201:    if save_data_to_file:
./src/emhass/command_line.py:237:    if save_data_to_file:
./src/emhass/command_line.py:281:    if save_data_to_file:
./src/emhass/command_line.py:352:        if filename_path.is_file():
./src/emhass/command_line.py:359:    if use_last_window:
./src/emhass/command_line.py:370:    if model_predict_publish:
./src/emhass/command_line.py:413:        if filename_path.is_file():
./src/emhass/command_line.py:446:    if save_data_to_file:
./src/emhass/command_line.py:503:    if input_data_dict['opt'].optim_conf['set_use_battery']:
./src/emhass/command_line.py:631:        if args.debug:
./src/emhass/command_line.py:638:        if args.debug:
./tests/test_forecast.py:50:        if self.get_data_from_file:
./tests/test_forecast.py:207:        if self.get_data_from_file:
./tests/test_forecast.py:282:        if self.get_data_from_file:
./tests/test_optimization.py:32:        if get_data_from_file:
./tests/test_retrieve_hass.py:31:        if get_data_from_file:
./tests/test_retrieve_hass.py:39:            if save_data_to_file:

Yes of course like I said this is basic Python so I use it extensively.
But maybe there are some hidden dragons when retrieving data from flask end points, etc...

It may be good to close this issue and keep an eye open to see if there is any new issue that arrives that seems similar and go from there.
Otherwise if you like we can start transitioning the if statements (ex. Anything in the /scripts that may use passed parameters)

The latest PR #175  did not solve the issue.
I've just tested it using a local compile of the standalone version and it is not taking into account the perform_backtest parameter.
It seems to be a problem with bool only.
Because I tested this and it changed the model type just fine:
curl -i -H "Content-Type:application/json" -X POST -d '{"sklearn_model": "LinearRegression","perform_backtest": "true"}' http://localhost:5000/action/forecast-model-fit

Sorry about that. I'll quickly run some more tests now if I see if I can't diagnose the issue before I head to bed

So the veritable is acting as a string. so  if perform_backtest == "True" or perform_backtest == "true": will work

Maybe try to convert to bool when retrieving runtime params in utils.py?

This might have to be done on all bool runtimes

#177

Sorry about that @davidusb-geek , that was on me again for not testing well enough. The pull request is ruff but hopefully you can run with it and fix the bug. I'm heading to bed 👍


[Phase 2 - Tools] Update Atom plug-in to support Static Extension Methods

The Dart Atom plug-in has a grammar for syntax highlighting Dart code in Atom. This same grammar is also used for syntax highlighting on GitHub. Update this to handle the new syntax.

@a-siva @devoncarew Probably area-intellij is not relevant here?

Agree, we don't seem to have any Atom specific label or a generic plugin label. Removed the area-intellij label, please feel free to assign it to some other area label.

/move dart-atom/dart

/move dart-atom/dart


Added functions to get/set various stockmodel parameters

Added functions to get/set following stockmodel parameters

Tolerance
Stepover
Rest thickness
Detect overhang
Workplane

Thanks. Alpha version should be available to download shortly.


use fully qualified path for superclass

Todo:

[x] Add tests
[x] Add release notes

Release Notes
What Changed

Use fully qualified name for implementation superclass
Output generated files according to their package
Append containing class name for nested Hooks
Add HooksDsl typealias to help with conflicting names

Why
This allows nested classes to be used as the base hook classes as well as top level base hook classes:
abstract class MyControllerHooks : Hooks() {}

class MyController {
    val hooks = MyControllerHooksImpl()
}

vs
class MyController {
    abstract class Hooks : Hooks() {}

    val hooks = MyControllerHooksImpl()
}

📦 Published PR as canary version: 0.10.1--canary.17.183

:rocket: PR was released in v0.10.1 :rocket:

:rocket: PR was released in v0.10.1 :rocket:


Disable soniox

let's disable soniox for now as it is 2x more expensive than deepgram

https://github.com/BasedHardware/omi/commit/d75facc06c4a69da78c92173ca7a1a63909fb7d4#diff-7c62653a4a8a3f0f493b5372a547288fac05a7381701740be55da9a93197d0c9R75


Optimize class weights

Is it possible to tune a classifier's class weight vector using optunity?
Currently the parameter space is defined in optunity's 'minimize' and 'maximize' by passing it the lower and upper bounds. This assumes the parameter is a single number (such as 'C' or 'gamma'). I am wondering whether it is possible to optimize a weight vector, for example, where the length of the vector equals the number of classes.
Thanks

This is currently not supported directly, however you could obtain the same effect by creating an appropriate amount of variables, e.g. w1, w2, ... wN where N equals the number of classes.
This is a little cumbersome, but it will give you the same effect.


[Bug] When Publishing to private feed I get The  element is deprecated.

There seems to be  some braking changes in version 6.0.27
When I publish to nuget private feed I get the following error.
We found the following issue(s):
The  element is deprecated. Consider using the  element instead. Learn more.
We recommend that you fix these issues and upload a new package. To learn more about authoring great packages, view our Best Practices page.

The warning from the server is accurate; you can update your NuGet metadata to include a license element instead of a licenseUrl. NPE lets you view either.


Add missing chart object to dataloder complete call

Problem is that otherwise dataloader complete function has undefined argument!

Thanks for this.  But thanks for bringing this to our attention!


DOC: update multival usage and add docs.fastly.com links

This commit updates the documenation with the following changes:

Replaces docs.signalsciences.net with proper docs.fastly.com URLs
Splits field types into "single" and "multival" with the valid values for each
Expands requestlogging to indicate that it is only valid for rules of type "request" and explicitly calls out that it is not valid for "signal" or "rateLimit"
Fixes "signal exclusion" to be "signal"
Fixes corp rule types to have addSignal, excludeSignal

@daniel-corbett sorry, could you run make docs again so docs/resources/corp_rule.md gets updated? Thank you!


Syntax for Edge.  vertexSteps extends elementSteps from TODO.

See PR https://github.com/mpollmeier/gremlin-scala/pull/81 for the arrow implementation we spoke about in https://github.com/mpollmeier/gremlin-scala/pull/82#issuecomment-129506628

Thank you @dkrieg for your really good idea and your contribution, I merged the PR #81


Make coverage badge flat

Nice! Thanks @onbjerg! 🙂

Nice! Thanks @onbjerg! 🙂
I liked the previous alt text ("Coverage Status") more than the new one ("Coveralls github branch") though. Could you revert that part back?

Sure!


Intel compiler fixes and CI

Build fails with
/home/ruben/workspace/boost-root/libs/mysql/include/boost/mysql/impl/internal/protocol/deserialize_text_field.ipp:95:35: error: explicit comparison with infinity in fast floating point mode [-Werror,-Wtautological-constant-compare]
    if (!ok || std::isnan(val) || std::isinf(val))  // SQL std forbids these values

When building with the newest icpx.

Raised by #173


Topic/complete virtualization

All research garbage is out. We have a clean start now.
Review by @amirsh
Review by @perovic

Adding a unit test which shows how to use Reifierand Staged would be great.

@vjovanov Can you change the version please (e.g. 0.2.1)? Your big surgery is not source compatible :)

This version is not released yet. How can I change the version of the SNAPSHOT branch? Also, this is major change and should be a major version bump.
Can we work on syncing with the new version in the next week and then we can release?

I agree with you. Anyway definitely there should be a version change. Because right now dblab is dependent on version 0.2.0-SNAPSHOT. It would be great if you can release the version before this big surgery (as non-snapshot) and change the version for this one to 0.3.0 (as you've suggested is a major version bump).
The next week is great! I'm totally in for that.


Preprocessing

Using the Leukemia dataset, shiny app preprocessing gives slightly different results from directly applying preproc package. For example, setting mean and variance cutoff to be 0.8. Shiny app gives 205 genes; preproc gives 206 genes. Similar problem is encountered when using 0.3 cutoff for both.

In my case, Shiny gives 206 genes.
Not sure why but I suggest to uninstall preproc and re-install in again for RShiny

Just uninstall and re-install preproc, still get the same problem. Weird!

Found a typo in preproc/R/filter.R.
In line 77. function(z)rank(apply(z[gene.mv,], 1, mean, na.rm=T))). Should the mean be replaced by sd?

How about you fix this problem in your own branch?
I don't want to touch the master branch since all other features are relying on current master.
I will fix the issue when we need to merge everything together.
What do you think? @schwannden

solved


Fixed some minor issues but stuck at Mirror drive

Very impressive stuff you guys are building.
I'm testing
https://github.com/holepunchto/pear-expo-hello-world
and i've encountered some issues.
Some pre-info: I disabled the build processes for iOS because i do not have xcode

On some windows machines it seems like you need to add { shell: true } to the sdk.manager process.

Otherwise I got EINVAL error.

2:
I fixed the issue with execute_process with a hardcoded path ( The previous issue reported did not solve it for me )

I seemed like the build process did not identify (WIN32) for my system by some odd reason ( logged the if/else statement ), and even though I changed the else condition to match the windows setting I still got the same error.

I was using the standard dependency 0.7 of bare dev ~ and it never cloned the prebuilds folder correctly and got stuck here:

I then updated to the latest 0.10.4 and now it gets stuck at here.

Not sure if my hacking has fked something up or if my multiple re-runs of hello-pear --configure has made something break. Anyways hope this issue might help somebody else who got stuck 👍

Thanks for the report! I fixed the shell option, that's indeed needed when executing batch scripts on Windows. The rest seem environment specific though, I couldn't reproduce those. In particular the mirroring can be slow depending on where you are in the world as it downloads a pretty substantial amount of data from the swarm.

Had to roll back the shell option default as it breaks command arguments.

Ok 👍 The mirror / downloading worked btw 🎉

I think that was a problem for me too when I added shell : true to all exec, spawn etc. It was only needed in the end to start en sdk.manager process with execFile 👍

Had to roll back the shell option default as it breaks command arguments.


Fix for gulp watching on windows

Description
Fix for browsersync and file watchers not working correctly on windows. This uses a legacy polling method, so it registers events if the user is on windows (which I assume the windows? is meant to represent.
Fixes #2642 which I've encountered.
Rationale
There are some windows developers around, and cookiecutter-django is a great start for a project whatever OS you use.
Use case(s) / visualization(s)
Not much to visualize sadly, but I think it's a pretty simple PR to understand what it does :)

@Tusky For the CONTRIBUTOR's doc, RST hyperlinks work by having the `@Tusky`_ with the link to your GitHub

@Tusky For the CONTRIBUTOR's doc, RST hyperlinks work by having the @Tusky_ with the link to your GitHub. You can take a look below all the names in the table to know what I mean.

Thank you, forgot to add it at the bottom :(


What did you do to my battery

My battery no longer shows usage, and will not hold a charge. Why did you make these changes to my device? This isn't helpful.

Schok Volt SV55
Model: Schok Volt SV55 (Schok Volt SV55)
Manufacturer: Schok
Board: msm8937
Device: SV55216
Product: SV55216
CPU Cores: 4
CPU Bit: 32-Bit
CPU Model: Qualcomm Technologies, Inc QM215
Memory: Total 1.83 GB, Avail 584.02 MB
Storage: Total 11.45 GB, Avail 10.12 GB
Resolution: 480 * 960 px / 240 dp
Screen Size: 2.4" * 4.9" / 5.5 Inches


Fix regular expression mistake in Python 3.7

Python 3.7 subtly changed the behavior of re.sub() which leads to paths
like ..../.... where they should look like ../...
By matching only non-empty path segments, there is no second-adjacent
empty match that would be replaced.

#304
 was fixed issue on tasks/symlink-shared.yml
But I added similar fix for tasks/rsync-deploy.yml
The issue manifests itself when rcync is used for the current folder.

This was fixed in #304
Not sure if you are running an outdated version

#304
 was fixed issue on tasks/symlink-shared.yml
But I added similar fix for tasks/rsync-deploy.yml
The issue manifests itself when rcync is used for the current folder.

Oh sorry my bad! You are absolutely right and the github diff mislead me


Unit tests fail to download files from dataserver when in homeoffice

When I try to run the unit tests while in homeoffice the download of the needed files from the dataserver is very slow to the point of failing to download some files resulting in multiple errors like:
ERROR ctapipe/io/tests/test_eventseeker.py - requests.exceptions.ConnectionError: HTTPConnectionPool(host='cccta-dataserver.in2p3.fr', port=80): Read timed out.

Apparently this is related to the IP address trying to download the data, since this does not happen when I am in the office (using the same machine).

I also encountered this issue in the past and could only solve it by using a VPN.
@bregeon couldn't help if I remember correctly, since he doesn't have control over the external firewall at in2p3 or whatever else is causing these issues

We actually have a new data server being set up at DESY, so we can at some point drop CCin2p3 and have more control... For now probably there is nothing we can do.  Maybe there is some blocklist set up there now, as for me at home it works fine.

You're in France...

You're in France...

Though it works from GitHub's CI, which probably isn't.  But yes, it's likely they block some IPs (GitHub is likely an exception)

it's likely they block some IPs

It's not completely blocked, just very very slow, I get something like 20 kB/s at home without VPN, and 4 MB/s with VPN.
My home is ~2 km away from the university...


MountVolume.SetUp failed for volume "test" : invalid character 'F' after object key:value pair

I’m deploying FlexVol on AKS according to the procedure:
https://github.com/Azure/kubernetes-keyvault-flexvol
I’m trying option 1 as mentioned “OPTION 1 - Service Principal” and using the sample POD configuration file provided in the same doc.
however the POD doesn’t seem to recognize the flexVolume configuration section and I get the following error:
“MountVolume.SetUp failed for volume "test" : invalid character 'F' after object key:value pair”
My AKS version is 1.12 and I have installed the plugin as mentioned in the doc. I can see the installation POD running on my agents and nothing special in its logs:
NAME                        READY     STATUS              RESTARTS   AGE
keyvault-flexvolume-n44sl   1/1       Running             0          10m
can anyone please advise?
Thanks,

@shayshahak can you please share your pod yaml and kubectl describe pod <pod name>? please redact sensitive information.

@ritazh yes of course.
podyaml.txt
describepod.txt

@shayshahak Thanks for providing these. Few things to check:

I see your pod was created in namespace kv. Please make sure your secret kvcreds was also created in namespaces kv.
ssh into the agent node running the pending pod, get the hostname with kubectl get pod -o wide, or from your describe pod output it's aks-agentpool-94615314-1. then get more detailed logs from the node at: /var/log/kv-driver.log

@ritazh

the secret is in fact resides with the same namespace. when it's not i see the error "MountVolume.SetUp failed for volume "test" : Couldn't get secret kv/kvcreds err: secrets "kvcreds" not found".

please see the requested log, there are some failures in it.
kv-driverlog.txt

thanks

@ritazh

the secret is in fact resides with the same namespace. when it's not i see the error "MountVolume.SetUp failed for volume "test" : Couldn't get secret kv/kvcreds err: secrets "kvcreds" not found".
please see the requested log, there are some failures in it.
kv-driverlog_new.txt

(uploaded again this time without my secrets in it.. :-) )
thanks

got it! from the log i saw "AccessDenied" so i went back to my key vault only to find out the the ServicePrincipalName was suddenly missing on the my Key Vault.
fixed that, redeployed and now the sample pod is running:
NAME                        READY     STATUS    RESTARTS   AGE
keyvault-flexvolume-vzjqf   1/1       Running   0          4m
nginx-flex-kv               1/1       Running   0          2m
thanks @ritazh  for your support :-)


The hitting ghost should be visible in the death state

Currently it's placed behind the pacman and we cannot see it

Maybe you could apply some triumphing animation to it.

 But it should be a separate ticket and, of course, be implemented after fixing this bug because otherwise we won't even see that triumphing animation. :)

Needed for #93


Please add support DHT22 Sensor

Please add DHT22 temperature sensor in this driver.

+1

+1

+1

I think it is not possible to implement a driver for this sensor for the GPIO speed transmission constraints.

from: androidthings.rocks
In the Native SDK the GPIO speed is 0.15ms , which is still 150μs and nowhere near the 20-40μs required by the sensor.
Android Things and the fact it uses Linux, which is not a real-time operating system.
A conversation with a member of the Android Things team suggested the following:

Your not likely to get consistent sub-1ms timing at the application layer. If the data on the wire is timed the same as a UART, you might be able to write/read using a custom baud rate (thats ~25K baud). Otherwise you’ll probably have to implement it on an MCU.

I tried to do a porting of rasbian c++ code but we do not have the call delayMicroseconds()  and the Gpio::setValue(..)/Gpio::getValue() speed is high (150μs).
I think it's the same as using Native PIO


Structure global validators better

Now the globals validators are defined here ->https://github.com/XRPLF/clio/blob/develop/src/rpc/common/Validators.hpp#L466
We can define them as static inside a struct to better organise them and test them separately from framework.

completed


Add documentation for github issue/PR templates

What makes a good issue template
some examples
how to order templates in the directory for UI ordering

I'm interested in this, will try to take a stab at creating an issue template with some accompanying docs for issue and PR templates. @delucchi-cmu I'm not quite sure if I understand the third bullet point, could you provide some more info on what you had in mind?

I may have a better understanding of point 3 after a little bit of tinkering, the question is just how to control the ordering of this page, right?

Resolved by #376


Autozoom

I am using the library with Symfony 3.2.7 (through the bundle).
I am loading the map, rendered in a template with an ajax request. The initial loading goes perfect.
When I replace the map-container's content with a different map, with the same function, the correct makers are added. Only the  map's auto zoom is not working.
Do you have any idea what could be causing this problem?
// main template

{% block map %}

{% endblock %}

// returned rendered by ajax response

{{ ivory_google_map(map) }}
{{ ivory_google_api([map]) }}

// javascript

var map_container = $('.map-container');

// AUTO ZOOM WORKS PERFECTLY
$(document).ready(function()
{
	loadMap();
});

// AUTO ZOOM DOES NOT WORK AT ALL
$('.obs-select').change(function()
{
	loadMap();
});

var loadMap = function()
{
	var ctl = getControls();

	var routeName = 'map_filter_' + ctl.filter;

	var url = Routing.generate(routeName);

	$.ajax({
		type: "POST",
		url: url,
		success: function(response){
			if (response.length) {
				map_container.html(response);
			}
		},
		error: function(xhr){
			console.log("Oops something went wrong in the controller with route: " + url + " Returns: " + xhr.status + " " + xhr.statusText);
		}
	});
};

@jrswgtr Hey! AFAIR, this is not possible to load multiple times the Google Map API. Do you have errors in your browser console?

ping @jrswgtr


Group Alerts in folders and tags..

Is it a duplicate question?
No
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I'd like to see 2 features added:

Allow Monitors to be sorted into Folders, with a TreeView, where nodes can be closed/opened
Add a Tag(s) attribute to each Monitor, so 1 or many tags can be added, that add a search box or tags-cloud to the main dashboard, which would filter the shown monitors. possibly allowing the filtering to be persistant.

Describe alternatives you've considered
None
Additional context
The intent is to provide means of organising monitorings, and how they are showin

Possibly duplicate of #12 #278


adding blog post section

Description
The section for blog posts and articles about Noir
Summary

It will be possible to add blog posts and articles related to the subject by submitting this pull request. I preferred to place it after the Talks and Workshop section so that someone who follows the repository sequentially can listen to the talks, read the blog posts and then transition more easily to the coding and projects section.
Additional Context
First blog post added https://hackmd.io/XZX9_pZ8Q1aa_ySDPeQopg

Thanks for merge.


Implement configurable label/title/description separator strings

Signed-off-by: Chris Papademetrious<EMAIL_ADDRESS>
name: Pull request
about: Propose changes to fix an issue or implement a new feature

Description
This is a proposed change to make the ". " separator between figure/table title labels,  text, and  elements configurable via a string.
Motivation and Context
Our content uses a colon to separate figure and table titles:
Figure 1: Some Figure
Table 1: Some Table
After creating a plugin to implement this single-character change, I realized it would be more user-friendly to configure this via a single configurable string (in org.dita.base/xsl/common/strings-common.xml). For example,
:
 --
How Has This Been Tested?
I created a testcase to confirm that the default HTML5 and XHTML output remains unchanged, then I reran it with a different string values to confirm that the output was adjusted as desired.
Type of Changes
This new feature was implemented by identifying the places where the title content was generated, then changing the hardcoded text strings to equivalent variables.
Checklist
Please let me know if unit tests are needed for this type of enhancement. (I would need to learn how to add and run them.) Thanks!

Personally - I have rather mixed feelings on this. I think the end result is probably good but it builds on a clunky design. (Admission: that clunky design came from my early code.)
I think with figure/table titles, a better way to go is to use variables like PDF -- which can build up with tokens like the separator, but more importantly, they let you pass in the figure/table number and title text, then determine how to present them as a whole. I like that better than adding several new variables for specific contexts, like desc.separator.
But ... I'm also not sure it's worth going that far. We already have ColonSymbol which combines with other text ... I feel the same way about that, it would be better to get a variable that (for example) returns Note: rather than one to return Note* and another to return a colon (or for French, a space and a colon). This basically uses the design. I think ideally both should use the PDF style, which came to the toolkit well after that Note: process went into HTML.
So, yeah, vaguely uncomfortable with the clunky design, but I agree it follows the current design in HTML and probably gives the right result.
Thoughts from others?

Hi Robert,
I don’t know the PDF style (I’m new to all this) but if the consensus is it would be preferable to adopt that approach, I’m happy to try to copy that approach too!
I basically just wanted an easier way to configure this than overriding entire templates to change one character.

Chris

From: Robert D Anderson<EMAIL_ADDRESS>Sent: Tuesday, January 7, 2020 3:34 PM
To: dita-ot/dita-ot<EMAIL_ADDRESS>Cc: Chris Papademetrious<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dita-ot/dita-ot] Implement configurable label/title/description separator strings (#3438)
Personally - I have rather mixed feelings on this. I think the end result is probably good but it builds on a clunky design. (Admission: that clunky design came from my early code.)
I think with figure/table titles, a better way to go is to use variables like PDF -- which can build up with tokens like the separator, but more importantly, they let you pass in the figure/table number and title text, then determine how to present them as a whole. I like that better than adding several new variables for specific contexts, like desc.separator.
But ... I'm also not sure it's worth going that far. We already have ColonSymbol which combines with other text ... I feel the same way about that, it would be better to get a variable that (for example) returns Note: rather than one to return Note* and another to return a colon (or for French, a space and a colon). This basically uses the design. I think ideally both should use the PDF style, which came to the toolkit well after that Note: process went into HTML.
So, yeah, vaguely uncomfortable with the clunky design, but I agree it follows the current design in HTML and probably gives the right result.
Thoughts from others?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dita-2Dot_dita-2Dot_pull_3438-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAMEXGOPRTFT2XHKMBMDAARTQ4TRJ7A5CNFSM4KCK7ZPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIKFUQI-23issuecomment-2D571759169&d=DwMCaQ&c=DPL6_X_6JkXFx7AXWqB0tg&r=bMg2uJl8U7dqZNrR3_PU2IuKKnQoiod9UfSzAQ9eycM&m=9fkAiBJ2A8UAyVmz68pw_Vy5SLinjx2uznYkS3vaEdM&s=bxwB_aRbR2SR8NraBWRdWC6K2_egvk-hffAF8c2QY2g&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AMEXGOKRGUT3LTZ6BJDNBMDQ4TRJ7ANCNFSM4KCK7ZPA&d=DwMCaQ&c=DPL6_X_6JkXFx7AXWqB0tg&r=bMg2uJl8U7dqZNrR3_PU2IuKKnQoiod9UfSzAQ9eycM&m=9fkAiBJ2A8UAyVmz68pw_Vy5SLinjx2uznYkS3vaEdM&s=YYF4o496_KgbR462C_7D_wn9-eecr-NjXtkOeiYvbmM&e=.


Use a more descriptive selector for our code cells

Describe the problem/need and solution
Context
Currently, we're adding a simple .cell class to our code cells when they are rendered.
However, .cell is a very generic term, and general best-practices for CSS / JS rely on having selectors that are scoped more narrowly to the specific type of object in question.
By using a generic term like .cell, we run the risk of clashing with others libraries that might assuming things have a .cell class that aren't notebook code cells.
I propose that we rename .cell to .nb-cell.
I also propose that we rename the inputs/outputs to follow the Block Element Modifier CSS naming style.

.cell_input to .nb-cell__cell_input
.cell_output to .nb-cell__cell_output

Guide for implementation
Here's where we add the .cell class to each notebook cell:
https://github.com/executablebooks/MyST-NB/blob/6be37cd66d1b7668b16a4bab385222b58f38930d/myst_nb/parser.py#L233-L243
And here's where we define classes for cell input/output:
https://github.com/executablebooks/MyST-NB/blob/6be37cd66d1b7668b16a4bab385222b58f38930d/myst_nb/parser.py#L246-L272
I think all we'd need to do is update those to use the classes above.
Tasks and updates
No response

cc @chrisjsewell - if you're 👍 on this idea would you be willing to fold this into the rewrite at #380 ?

Yeh makes sense 👍


Log operators documentation points to tutorialspoint

Describe the bug:
Log operators documentation points to tutorialspoint and links don't work
Steps to reproduce the behavior:

Go to https://help.sumologic.com/docs/search/search-cheat-sheets/log-operators/
Click on split operator link
See error on tutorialspoint page

Expected behavior:
It should point to sumo docs on the operator
Screenshots

Looks like the breakage landed in #3471.

cc @kimsauce


[AUTHZ] Displays the columns involved in extracting the aggregation operator

Why are the changes needed?

This PR aims to display the columns involved in extracting the aggregation operator.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Could you please take a look if you find time, thanks @yaooqinn @bowenliang123
Although the Aggregate involved columns will sink into the Project, is it necessary to display the extracted agg columns to prevent Spark Optimizer Rule changes?

Make sense to me. Can we also add some tests with having/grouping cases?

I suggest providing relevant tests for evaluation with/without extracting aggregation columns.

As new tests for aggregation pass successfully without extracting columns from Aggregate, I prefer to keep the tests but not the extraction.

As new tests for aggregation pass successfully without extracting columns from Aggregate, I prefer to keep the tests but not the extraction.

Do you mean just add test coverage instead of changing the code?

As new tests for aggregation pass successfully without extracting columns from Aggregate, I prefer to keep the tests but not the extraction.

Do you mean just add test coverage instead of changing the code?

Yes, I think so.

As new tests for aggregation pass successfully without extracting columns from Aggregate, I prefer to keep the tests but not the extraction.

Do you mean just add test coverage instead of changing the code?

Yes, I think so.

In fact, this PR is not a bug fix, existing logic can also extract agg columns, this is because Spark pushes agg columns down to the project operator, while kyuubi overrides the project operator.
Adding this pr is just an optimization in case the agg column is not pushed down to project after the spark optimizer rule is changed

Well noticed, it's good to add ut for extending coverage in aggregation with and without aggregation columns used in output projections.

Codecov Report

Merging #4532 (6a3468f) into master (41e9505) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #4532      +/-   ##
============================================
- Coverage     53.35%   53.32%   -0.04%
  Complexity       13       13
============================================
  Files           571      571
  Lines         31372    31376       +4
  Branches       4224     4225       +1
============================================
- Hits          16738    16730       -8
- Misses        13057    13069      +12
  Partials       1577     1577

Impacted Files
Coverage Δ

.../kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
90.62% <100.00%> (+0.40%)
:arrow_up:

... and 9 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

thanks all, merged to master/1.7


[bug]: error sending all resources

Environment:
Polar v2.0.0; Docker v20.10.21; Compose v2.13.0 (Win 10); network setup: 1 Bitcoin Core v25.0; 2 LND v0.16.2-beta; 2 Taproot Assets v0.2.0-alpha
Steps to reproduce:
First I minted asset type normal.
tapcli assets mint --type normal --name fantasycoin --supply 100 --meta_bytes "fantastic money"
tapcli assets mint f
after mining several blocks on 2 clients, I create an address where ammout == 100% of the mined amount
tapcli addrs new --asset_id {asset_id} --amt 100
I send funds from the 1st client
tapcli assets send --addr {addr}
Quick mine some blocks
now when on 2 clients using tapcli assets list i can see the acquired assets here. The problem is that when I use the same command on 1 client, I see them here as well, I also see them in the Polar GUI
if I now create a new address on client 2 with ammout 1 and try to send resources from client 1, the entire client fails with Error evaluating state (SendStateBroadcast): rpc error: code = Unknown desc = transaction rejected: output already spent and client already cannot be turned back on

Can you try to reproduce with a fresh instance on the latest release? v0.2.2 fixed a number of bugs related to sending and minting.

Tested, the attempt ended with several output already spent errors, turning off the client and I can't turn it on anymore. Unfortunately, I didn't have 2 clients available at the moment, so I tried the whole thing on one. I think (I hope) that it doesn't matter in this case, because it shouldn't be a problem to send resources to oneself indefinitely, it is about sending resources regardless of whether the target destination is a different or the same client. I generated an asset with amt 100, created an address for this asset with amt 100, sent resources, then created an address for amt 10, sent, and subsequent errors occurred during the Broadcasting new transfer tx action. Setup: Ubuntu 20.04; lncli version 0.16.3-beta; tapcli version 0.2.2-alpha; testnet network. After minting, sending, and before generating a new address, I always waited for the transaction to be included in the block to eliminate the effects of an unfinalized state.

#261 should fix this


minor changes

use tm1.cells everywhere and avoid 15s wait on the last iteration.

Thanks for the fix @rkvinoth


turn off the hyperion when send command to turn off Raspbian

After breaking his head and search a lot, read everything here, I could make it work, now I need a light to turn off the hyperion when sending the command to turn off the raspbian, I was never good with linux, I think this is easy, just need the way.

@welitonxd
which Raspian version are you using?

Maybe juste use hyperion remote and use "clear all" with good priority

i dont remember now what i'm did, but when the screen stay black the lights turn off, so, its all ok for me.

alright, thank you for feedback
could be closed

@welitonxd could you close the issue please? :)


error 6676

I get an error 6676 for all my drives and 0 GB. I have them in the config file as C:\Burst\Burst1 but when I start the miner they show up as C:BurstBurst1 with out slashes. I have tried forward slashes, back slashes as well as trying to use an extra escape slash but it did not work. Any thoughts?

Nobody has reported something like this until now ... please provide more information like screen of error or jminer.properties ... anything that could help understand whats going wrong ... and
ensure you download/use the release from this repository ... to ensure no 3rd party software causes your issues.


Replaced r['comments'][index] with hard-coded 0

We encountered an issue when creating multiple records of the same name with different data.  The code block which executes to create multiple RecordEntry objects was attempting to use the incremented index value despite the fact that there is only ever a single comment associated with the records.
Hard-coding this to the first element corrected the failure and was an effective fix for our use cases.

I know this fix is a dirty hack; the underlying issue is moreso that we can send a list of create requests that link to a single comment.
Depending on your preference, I see a couple ways of mitigating the underlying issue:

Analyze the length of the comment list; if 1, then we handle creating with a single comment, else if length of comment list equals the length of the create requests, process iteratively.

Update the API's strictness in accepting comments

See PR #739 which I believe fixes this problem as well


update cminpack versions

Before adding new features and new modules, please go to issues to submit the relevant feature description first.
Write good commit messages and use the same coding conventions as the rest of the project.
Please commit code to dev branch and we will merge into master branch in feature
Ensure your edited codes with four spaces instead of TAB.

The cminpack package has released a new version, and I kept the latest and some old versions. Test has also been done.

增加新特性和新模块之前，请先到issues提交相关特性说明，经过讨论评估确认后，再进行相应的代码提交，避免做无用工作。
编写友好可读的提交信息，并使用与工程代码相同的代码规范，代码请用4个空格字符代替tab缩进。
请提交代码到dev分支，如果通过，我们会在特定时间合并到master分支上。
为了规范化提交日志的格式，commit消息，不要用中文，请用英文描述。

It seems only the latest version could pass checks. The old version were released several years ago, thus they can be discarded.


Exclude the renderers by feature flag

This PR

Adds a new feature

This includes Phaser.Renderer.Canvas or Phaser.Renderer.WebGL only if the corresponding feature flags CANVAS_RENDERER or WEBGL_RENDERER are true.
For Canvas-only builds this saves a lot.

👍


added link to pageboy

Write your shell scripts in any combination of languages you want.
#!/usr/bin/env pageboy
# ... bash script for bash things

#!/usr/bin/perl
# ... perl script for perl things

#!/usr/bin/env ruby
# ... ruby script for whatever reason

You can mix and match all you want. If your script uses valid shebangs, it will work.

Can you rebase this branch? It seems your commits affect k and pageboy is not included at all.


sparse gemm kernels are not supported in ACL

Output of 'strings libarm_compute.so | grep arm_compute_version':
arm_compute_version=v23.11 Build options: {'Werror': '0', 'debug': '0', 'neon': '1', 'opencl': '0', 'embed_kernels': '0', 'os': 'linux', 'arch': 'armv8a', 'build': 'native', 'multi_isa': '1', 'fixed_format_kernels': '1', 'openmp': '1', 'cppthreads': '0'} Git hash=b'add70ace1e57f65d1ae4d0cedaec6e4578cf87ff'
Platform:
AWS c7g.16xl
Operating System:
Ubuntu 22.04

Problem description:
PyTorch supports sparse tensor formats The request is to provide aarch64 gemm kernels that accept these sparse formatted tensors and does sparse gemm implementation to achieve better performance.

Hi @snadampal
Thanks for raising this. We will discuss the feature request with the team.

Hi @snadampal
We discussed this with the team, we are considering exploring sparse tensors support in the context of GenAI but this is not officially in the roadmap for ACL.  There are no plans to implement this feature.
We would be interested in specific use cases for ACL which you could share with us.
Hope this helps.

Hi @morgolock , isn't ACL targeted for GenAI use cases? My requirement is mainly to accelerate sparse LLMs inference with ACL gemm kernels. Are you planning a different GEMM library for GenAI ?

Hi @snadampal
Apologies if I was not clear. we're exploring ways to accelerate GenAI workloads with ACL. This means that we may consider exploring sparse tensors support in ACL to accelerate these workloads in the future but we don't have any work planned for this feature in our roadmap.

My requirement is mainly to accelerate sparse LLMs inference with ACL gemm kernels.

Could you please share more details about the models you would like to accelerate?
Hope this helps

Hi @morgolock , thanks for the clarification. will share the details with you.

Hi @morgolock, any updates on the plan for sparse GEMM kernels in ACL? This is particularly important as sparse LLMs are able to match performance of dense counterparts on task-specific applications. For ex - https://arxiv.org/pdf/2310.06927

Hi @Arnav0400
There is no work planned to implement this feature at the moment. I'll discuss it again with the team.


Starting Documentation & Demo

Adds basic setup instructions and development guidelines to the README. The goal is to make the amount of effort it takes for new developers to open an interesting demo as small as possible, which means keeping the demo stable is important.

Adds a Sponza demo to Sandbox.

Adds a low-quality Sponza model, associated textures, and font assets to the Sandbox project.
Uses basic phong shader, although Normal maps were excluded for now as the model seems to not have them.
Transparency needs work, but I found a decent compromise which suffices for now.

Adds some OpenGL config to the Init function of Render2D to allow for transparent textures.

This seems to be an bad place for this. Since they were disabled though I assume the upcoming Renderer refactor would have decided where to put these anyway.

Includes a bunch of recent changes from dev

These seem to be mostly refactors, and the demo still works fine, so I'm comfortable including them in master if @Szilard0605 is.

Looks good. I will merge.


job #9369 pathkey sig

This change comes from Keith, I just tested and reviewed it.

This pull request was closed without being merged.


Error in ValidateVR.cs

Hi!
Can't get this to build unfortunately (see image)..
The using UnityEngin.XR.Management; is complaining on .Management
The compiler also complains that the type or namespace name "XRGeneralSettings" could not be found.
I installed this with a project open and pressed "Playcraft Install.unitypackage". Also installed various XR packages.

I haven't updated the Unity plugin for a while, so that may be the cause.  As this points to a core functionality issue, I'll make sure to provide a fix as soon as I can.
In the meanwhile, if that is the only error, you can comment out the script, as it is a minor and independent quality-of-life script (turns on and off a VR setting and has a hook for activating/deactivating objects to save time when switching between flatscreen and VR in editor).

Pushed an update which should fix the issue, please let me know if it works.  Relevant change puts that code inside platform dependent (editor only) compilation.

Thank you! Now it works =D Although I can only move by teleporting?
David Peterson
+46 72 889 5218
Website https://davidpeterson.io/ | LinkedIn
https://www.linkedin.com/in/the-david-peterson/ | Instagram
https://www.instagram.com/thedavidpeterson/
On Wed, 1 Dec 2021 at 03:28, Will Petillo @.***> wrote:

Pushed an update which should fix the issue, please let me know if it
works. Relevant change puts that code inside platform dependent (editor
only) compilation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Will9371/Playcraft/issues/5#issuecomment-983224990,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD772DLOOK4Q7FKMAACUPITUOWB4PANCNFSM5JCEIMVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I assume you are looking at the VR exploration scene.  The XR rig prefab in that scene can teleport and climb.  The climbing feature has a (optional) ability to extend its reach by holding the grip button.  Other VR locomotion systems include an experimental variation on climbing that grabs automatically (Monkey Climb scene) and pull-based movement (as seen in Gorn).  I've been considering a means for refactoring to make the abilities togglable within a single component without sacrificing modularity or efficiency.
I've updated the Readme to better describe the context and intent of this repo.  This is not a commercial asset in development that happens to be free (like VRTK), it's where I put my reusable code built for other projects so I can prototype faster on new projects, publicly shared because "why not".  As such, new features are added on a "whenever" basis.

Got it! Many thanks for sharing and updating this, so cool!
On Wed, 1 Dec 2021 at 21:00, Will Petillo @.***> wrote:

I assume you are looking at the VR exploration scene. The XR rig prefab in
that scene can teleport and climb. The climbing feature has a (optional)
ability to extend its reach by holding the grip button. Other VR locomotion
systems include an experimental variation on climbing that grabs
automatically (Monkey Climb scene) and pull-based movement (as seen in
Gorn). I've been considering a means for refactoring to make the abilities
togglable within a single component without sacrificing modularity or
efficiency.
I've updated the Readme to better describe the context and intent of this
repo. This is not a commercial asset in development that happens to be free
(like VRTK), it's where I put my reusable code built for other projects so
I can prototype faster on new projects, publicly shared because "why not".
As such, new features are added on a "whenever" basis.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Will9371/Playcraft/issues/5#issuecomment-984006722,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD772DJ3X65PEDR6AQXB4WTUOZ5EHANCNFSM5JCEIMVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Error while using module created using Nuitka from python package using numpy.

I am trying to create a standalone module using a simple package I created.
Following is the file structure I have:

/home/user/nuitka_demo/NumpyProject/
    -- ./test.py
    -- ./firstPack/
        -- ./__init__.py
        -- ./firstpack.py

Content of the files:

# __init__.py
from firstPack.firstpack import MyVector

# firstpack.py
import numpy as np

class MyVector:
    def __init__(self, vector_as_list):
        self.vector = vector_as_list

    def vectorize(self):
        return np.array(self.vector)

    def scale(self, scale_factor):
        return scale_factor*self.vectorize()

# test.py
import firstPack
from firstPack import MyVector
vec = MyVector([1,2,3])
print(vec.vectorize())
print(vec.scale(3))

I am using the following command to create the module from the folder /home/user/nuitka_demo/NumpyProject/

python -m nuitka --module firstPack --plugin-enable=numpy --follow-imports --include-package=firstPack --include-plugin-directory=/home/user/anaconda3/envs/nuitka_env/lib/python3.6/site-packages/numpy/core --include-plugin-directory=/home/user/anaconda3/envs/nuitka_env/lib/python3.6/site-packages/numpy/lib

When I run this command in terminal I am getting the following warning:

Nuitka:WARNING:Recursed to package 'firstPack' at /home/user/nuitka_demo/NumpyProject/firstPack' twice.

When I try to use the firstPack.so module created, I get the following error:

$ python test.py
Traceback (most recent call last):
  File "test.py", line 1, in <module>
    import firstPack
  File "/home/user/nuitka_demo/NumpyProject/firstPack/__init__.py", line 1, in <module firstPack>
  File "/home/user/nuitka_demo/NumpyProject/firstPack/firstpack.py", line 1, in <module firstPack.firstpack>
  File "/home/user/nuitka_demo/NumpyProject/numpy/__init__.py", line 142, in <module numpy>
  File "/home/user/nuitka_demo/NumpyProject/numpy/core/__init__.py", line 3, in <module numpy.core>

Details:

$python -m nuitka --version
0.6.5
Python: 3.6.9 |Anaconda, Inc.| (default, Jul 30 2019, 19:07:31)
Executable: /home/user/anaconda3/envs/nuitka_env/bin/python
OS: Linux
Arch: x86_64

I installed Nuitka using pip install Nuitka

I tried the hello world example of Nuitka. It worked for me. There was no issue.

It will be great if someone can point me in the right direction. I am not sure what is it that I am missing and how to resolve this issue.
Thank you in advance.

Okay, I got to a point where I create a module something like firstPack.so using nuitka, and in my test.py module, I import module after importing numpy. Then the code is working. I have shared the updated code below.
# test.py
import numpy as np
import firstPack
from firstPack import MyVector
vec = MyVector([1,2,3])
print(vec.vectorize())
print(vec.scale(3))

Does this mean I cannot create a single module firstPack.so which will include numpy internally? Will have to mention numpy as a dependency always for this module?
Please let me know if I am missing something or any more information is required.


Move lib/togls specs to specs/togls

Why you made the change:
I did this because this is that actual standard for the directory structure and
it is better that we follow the standard as it will allow tooling, etc. to work
properly.

:+1:


[Bug] 4stems mode stops executing without output and exception reported

Description
While 2stems mode works sucessfully, 4stems mode fails. In cmd I use this command:
python -m spleeter separate -i D:\spleeter\src\temp.mp3 -p spleeter:4stems -o D:\spleeter
and after a while it stops and ends with:
OMP: Info #250: KMP_AFFINITY: pid 13616 tid 18408 thread 28 bound to OS proc set 1
OMP: Info #250: KMP_AFFINITY: pid 13616 tid 5420 thread 29 bound to OS proc set 3
OMP: Info #250: KMP_AFFINITY: pid 13616 tid 14956 thread 30 bound to OS proc set 5
OMP: Info #250: KMP_AFFINITY: pid 13616 tid 16504 thread 31 bound to OS proc set 7
PS C:\Users\user>
Step to reproduce

Installed using conda
Run as command in cmd
Got no result and no exception reported error

Output
nothing....

Environment

OS
Windows

Installation type
Conda

RAM available
XGo

Hardware spec
CPU

Additional context

I didn't let the model download automatically in the first run because it's too slow; instead I download from the release and put it in /pretrained_models/4stems.

Hi @Rohan-Kishibe
Not sure what's happening here and I don't know how to help you. Would it be possible to delete the models and try the automatic download again ?

i can concur, if you add the verbose mode --verbose you will certainly see somthing like that :
2020-02-11 07:04:08.024697: W tensorflow/core/framework/allocator.cc:107] Allocation of 681443328 exceeds 10% of system memory.
and my memory was at less than 144mb
where to put the verbose mode:
spleeter separate -p spleeter:4stems --verbose -i /input/the_organ_brother.mp3 -o /output
for the docker container with nvidia :
nvidia-docker run -v $AUDIO_OUT:/output -v $AUDIO_IN:/input -v $MODEL_DIRECTORY:/model -e MODEL_PATH=/model researchdeezer/spleeter separate -p spleeter:4stems --verbose -i /input/the_organ_brother.mp3 -o /output
the bug i have closed :
https://github.com/deezer/spleeter/issues/204


List box, Combo box, Dropdown menus

Latest versions of components also taken into use

This change is

This would be awesome to have right now. Thanks for submitting this PR? I know its that time of year but can we get this merged in?


Setup maven wrapper

Pull Request Template
Description
Add maven wrapper to this project (mvn version 3.6.1).
Certain libraries (for eg. junit 5) require different configurations depending on the version of maven used. By allowing users to have a fully encapsulated build setup we avoid these issues.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[X] Just a regular mvn clean install

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented on my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] Any dependent changes have been merged and published in downstream modules
[X] My changes have no bad impacts on performances
[ ] Any implemented change has been added in the CHANGELOG.md file

Not sure if this needs to be added to the changelog as there were no changes to the library code itself.

Hi @fborriello!
This is generated code (https://github.com/takari/maven-wrapper) and I would not recommend changing it.

I know, but this means that every time there is a new maven release and we want to upgrade it, we need re-generate the code from takari project, isn't it?
But, if apply the below changes, the maven version upgrade can be performed by simply update a value in the properties file.
What do you think?

Hi @fborriello!
This is generated code (https://github.com/takari/maven-wrapper) and I would not recommend changing it.

I know, but this implies that every time we need to upgrade the maven version, we need to use the takari project to re-generate the maven-wrapper with the latest version.
But, if we apply the suggested changes, the upgrade will be done by simply modifying a value in a properties file. what do you think?

Hi @fborriello!
This is generated code (https://github.com/takari/maven-wrapper) and I would not recommend changing it.

I know, but this implies that every time we need to upgrade the maven version, we need to use the takari project to re-generate the maven-wrapper with the latest version.
But, if we apply the suggested changes, the upgrade will be done by simply modifying a value in a properties file. what do you think?

I agree it would make the upgrade process a bit simpler, but it would require further changes.
The properties file is handled by org.apache.maven.wrapper.WrapperExecutor which unfortunately does not seem to handle placeholders.
It results in this error:
Exception in thread "main" java.lang.RuntimeException: Could not load wrapper properties from '/Users/dmartinsalbuqu/Development/Workspaces/bull/.mvn/wrapper/maven-wrapper.properties'.
        at org.apache.maven.wrapper.WrapperExecutor.<init>(WrapperExecutor.java:72)
        at org.apache.maven.wrapper.WrapperExecutor.forWrapperPropertiesFile(WrapperExecutor.java:56)
        at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:60)
Caused by: java.net.URISyntaxException: Illegal character in path at index 68: https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/${maven.version}/apache-maven-${maven.version}-bin.zip

Hi @fborriello!
This is generated code (https://github.com/takari/maven-wrapper) and I would not recommend changing it.

I know, but this implies that every time we need to upgrade the maven version, we need to use the takari project to re-generate the maven-wrapper with the latest version.
But, if we apply the suggested changes, the upgrade will be done by simply modifying a value in a properties file. what do you think?

I agree it would make the upgrade process a bit simpler, but it would require further changes.
The properties file is handled by org.apache.maven.wrapper.WrapperExecutor which unfortunately does not seem to handle placeholders.
It results in this error:
Exception in thread "main" java.lang.RuntimeException: Could not load wrapper properties from '/bull/.mvn/wrapper/maven-wrapper.properties'.
        at org.apache.maven.wrapper.WrapperExecutor.<init>(WrapperExecutor.java:72)
        at org.apache.maven.wrapper.WrapperExecutor.forWrapperPropertiesFile(WrapperExecutor.java:56)
        at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:60)
Caused by: java.net.URISyntaxException: Illegal character in path at index 68: https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/${maven.version}/apache-maven-${maven.version}-bin.zip

if you don't mind, I would try to figure out a solution for this, do you have an available branch that raises the above error?

Hi @fborriello!
This is generated code (https://github.com/takari/maven-wrapper) and I would not recommend changing it.

I know, but this implies that every time we need to upgrade the maven version, we need to use the takari project to re-generate the maven-wrapper with the latest version.
But, if we apply the suggested changes, the upgrade will be done by simply modifying a value in a properties file. what do you think?

I agree it would make the upgrade process a bit simpler, but it would require further changes.
The properties file is handled by org.apache.maven.wrapper.WrapperExecutor which unfortunately does not seem to handle placeholders.
It results in this error:
Exception in thread "main" java.lang.RuntimeException: Could not load wrapper properties from '/bull/.mvn/wrapper/maven-wrapper.properties'.
        at org.apache.maven.wrapper.WrapperExecutor.<init>(WrapperExecutor.java:72)
        at org.apache.maven.wrapper.WrapperExecutor.forWrapperPropertiesFile(WrapperExecutor.java:56)
        at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:60)
Caused by: java.net.URISyntaxException: Illegal character in path at index 68: https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/${maven.version}/apache-maven-${maven.version}-bin.zip

if you don't mind, I would try to figure out a solution for this, do you have an available branch that raises the above error?

You can replace the contents of the properties file with:
maven.version=3.6.1
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/${maven.version}/apache-maven-${maven.version}-bin.zip
maven.wrapper.version=0.5.5
wrapperUrl=https://repo.maven.apache.org/maven2/io/takari/maven-wrapper/${maven.wrapper.version}/maven-wrapper-${maven.wrapper.version}.jar

And try to run maven, for example:
./mvn -version


[Introspection] Cannot set a default value on a relation field

Magento:
error: Error parsing attribute "@default": Cannot set a default value on a relation field..
  -->  schema.prisma:1828
   |
1827 |   design_change_id Int       @id
1828 |   store_id         store     @default(0)
1829 | }
   |
error: Error parsing attribute "@default": Cannot set a default value on a relation field..
  -->  schema.prisma:1874
   |
1873 |   name      String?
1874 |   region_id directory_country_region @default(0)
1875 |
   |

This happens in many other schemas as well:
https://github.com/prisma/introspection-engine-output/pull/5/files#diff-c05bafbfe94261397d420d74896f7bdc

Decision
We should support static default values on relation fields.
Step 1) Make sure Introspection and Query Engine does not reject it in the PSL
Step 2) Query Engine should use the default value


Downloading feature for Bilibili is not working properly in terms of sequential videos

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.06.02. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.06.02

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v http://www.bilibili.com/video/av4808130/
[debug] System config: []
[debug] User config: [u'-f', u'bestvideo+bestaudio/best']
[debug] Command-line args: [u'-v', u'http://www.bilibili.com/video/av4808130/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.06.02
[debug] Python version 2.7.10 - Darwin-15.5.0-x86_64-i386-64bit
[debug] exe versions: avconv 11.4, avprobe 11.4, ffmpeg 3.0.2, ffprobe 3.0.2
[debug] Proxy map: {}
[BiliBili] 4808130: Downloading webpage
[BiliBili] 7802182: Downloading video info page
[download] Downloading playlist: 【长篇】哆啦A梦443【钉铛】
[BiliBili] playlist 【长篇】哆啦A梦443【钉铛】: Collected 4 video ids (downloading 4 of them)
[download] Downloading video 1 of 4
[debug] Invoking downloader on u'http://ws.acgvideo.com/b/35/7802182-1.flv?wsTime=1464819935&wsSecret2=46d27ce17d9c45ed34c501f1f9bd20e3&oi=1445658136&appkey=86385cdc024c0f6c&or=1167325994'
[download] Destination: 【长篇】哆啦A梦443【钉铛】-7802182.flv
[download] 100% of 46.71MiB in 00:37
[download] Downloading video 2 of 4
[debug] Invoking downloader on u'http://ws.acgvideo.com/b/35/7802182-2.flv?wsTime=1464819935&wsSecret2=a5caf416c7f8c9267f7e5a94068afdfe&oi=1445658136&appkey=86385cdc024c0f6c&or=1167325994'
[download] 【长篇】哆啦A梦443【钉铛】-7802182.flv has already been downloaded
[download] 100% of 46.71MiB
[download] Downloading video 3 of 4
[debug] Invoking downloader on u'http://ws.acgvideo.com/b/35/7802182-3.flv?wsTime=1464819935&wsSecret2=6adbdee4f951614edc56800a8ca1566e&oi=1445658136&appkey=86385cdc024c0f6c&or=1167325994'
[download] 【长篇】哆啦A梦443【钉铛】-7802182.flv has already been downloaded
[download] 100% of 46.71MiB
[download] Downloading video 4 of 4
[debug] Invoking downloader on u'http://ws.acgvideo.com/b/35/7802182-4.flv?wsTime=1464819935&wsSecret2=f1ea44300218d2ae35bc0bbe2ff6e4ae&oi=1445658136&appkey=86385cdc024c0f6c&or=1167325994'
[download] 【长篇】哆啦A梦443【钉铛】-7802182.flv has already been downloaded
[download] 100% of 46.71MiB
[download] Finished downloading playlist: 【长篇】哆啦A梦443【钉铛】
<end of log>

Description of your issue, suggested solution and other information
The URL of that video has multiple pieces, which is common in Bilibili, the early versions of youtube-dl downloaded these pieces and merge automatically, while the recent versions download these pieces without merge, but after the fix on last version, the file names are not carrying part_x suffix thus after first piece been successfully downloaded, the following ones cannot be downloaded since this naming issue.
Suggest that add the part_x back or even better, add auto merge back since I need to write a bash to use ffmpeg anyway.

AFAIK auto merging in youtube-dl is never implemented. Which version are you using and how did you enable it?

_partX suffixes will be added back in the next version. Sorry for my previous mistake.

@yan12125 I don't know the auto merging you've mentioned. But Bilibili download feature did merge all clips together. Probably version in last year, and I reported this when I found out:
https://github.com/rg3/youtube-dl/issues/8314
Like if I download best quality for YouTube it'll merge audio and video into a single mkv file rather than audio track and video track alone. For Bilibili it also should merge clips into a single file.

From the description at #8314, I guess in BiliBili, videos were originally served as single files, and some day they decided to convert them to multi-part files. Handling multi-part videos correctly is much harder than merging video- and audio-only files, like the case in YouTube. There are lots of pull requests aim to implement it (#2844, #2723, #995, etc.), but none of them is good enough to get merged.

@yan12125 Hi, actually it was split into several files as well, if my memory is correct, it downloaded several files and merge at the end.
Since the old version has _partX suffixes, I have a bash for that, and if you can implement it into your code make it auto-merge, that'll be some great contribution to me and probably someone else. 😄
Here's what I have before:
printf "ffconcat version 1.0\n" > videolist.txt
for i in {1..4}
do
    printf "file '%s'\n" "【长篇】哆啦A梦442【梦蓝】-7668669_part$i.flv" >> videolist.txt
done
ffmpeg -f concat -safe 0 -i videolist.txt -c copy "【长篇】哆啦A梦442【梦蓝】-7668669.flv"
rm videolist.txt

Using ffmpeg concat feature.
So far, I used it for Bilibili, and for this gist https://gist.github.com/starkshaw/ec3cba780100bbb4050d 😏 before. But the code above is for newer ffmpeg version.

Dunno. Maybe you were using a fork but not rg3's version.


Best Practices for List Views and Lenses

Hi,
I am taking inspiration from the list example to build a slightly more complicated example but I run into a few conceptual issues. I am especially interested in the last part where list has shared data to be able to remove elements (redacted a bit for clarity):
List::new(|| {
    Flex::row()
        .with_child(
            Label::new(|(_, item): &(Vector<u32>, u32), _env: &_| {
                format!("List item #{}", item)
            })
        )
        .with_child(
            Button::new("Delete")
                .on_click(|_ctx, (shared, item): &mut (Vector<u32>, u32), _env| {
                    shared.retain(|v| v != item);
                })
        )
}))
.vertical()
.lens(lens::Id.map(
    |d: &AppData| (d.right.clone(), d.right.clone()),
    |d: &mut AppData, x: (Vector<u32>, Vector<u32>)| {
        d.right = x.0
    },
)

(A) First, this list only depends on AppData.right, so I'd like it to be globally lensed to that, i.e. either add .lens(AppData.right) at the end or replace lens::Id.map with AppData.right.map. This works:
.lens(
    |d: &Vector<u32>| (d.clone(), d.clone()),
    |d: &mut Vector<u32>, x: (Vector<u32>, Vector<u32>)| {
        *d = x.0
    },
)
.lens(AppData::right) // or AppData::right.map in the lens above

But is there a way to "automate" this unit Vector<T> => (Vector<T>,Vector<T>)? If I have several dynamic list I'll rewrite it many times.
(B) Let's say now AppData.right is a Vector<Stuff>. I have an impl Wiget<Stuff> object that I want to use in lieu of the Label of the example. How can I lens the state (Vector<Stuff>, Stuff) into just Stuff more elegantly that by writing a dedicated lens from scratch?
Once solved I would add it to the list.rs example I think it'd be worth sharing!

May have found a way for (A):
fn dupli_lens<T: Data>() -> impl Lens<T,(T,T)> {
    lens::Id.map(
        |d: &T| (d.clone(), d.clone()),
        |d: &mut T, x: (T, T)| {
            *d = x.0
        },
    )
}

Then after .vertical() it's simply:
.lens(AppData::right.then(dupli_lens())
// or
.lens(dupli_lens())
.lens(AppData::right)


Add trss-node to Jenkins

Request to finish adding trss-node to Jenkins, see https://ci.adoptopenjdk.net/computer/trss-node/
Details:
Would like to complete the setup (and someone with visibility of which would be the appropriate credentials to use).
We will add a Jenkins job that will allow us to update the trss server via the --watch option (rather than depend on the small set of people to log on to the machine to manually update).

If this isn't running as the jenkins user I believe we'll have to create a duplicate credential with the trss username

If this isn't running as the jenkins user I believe we'll have to create a duplicate credential with the trss username

Looks like it is running as a jenkins user so I have enabled it (the IP address on that agent definition was also incorrect) and made a suitable java available to run the jenkins agent.

thanks @sxa !


"Screen-shots here." is not a URL

See README

Great, thank you for the reminder, we will add them when we have them.


EC2 Instance-Tags with a "." (dot) cannot be inserted into mongodb

We are using tag-names to mark EC2 instances with additional information, some of them contain a dot, which Amazon can handle just fine, however mongodb seems to reject these as key:
2013-07-10 15:27:25.087 - ERROR - [MongoDatastore.scala:339] failed to upsert record:
...
"tags": [{
			"Client.E-mail"<EMAIL_ADDRESS>		},
...
scala.actors.Actor$$anon$1@ab81c86: caught java.lang.IllegalArgumentException: fields stored in the db can't have . in them
java.lang.IllegalArgumentException: fields stored in the db can't have . in them
	at com.mongodb.DBCollection._checkKeys(DBCollection.java:1087)
...

Fixed in #94 .


Support band selection for multi-spectral imagery

The band parameter of get_leaflet_tile_layer() is an integer type. I would suggest adding support for band selection when there are more than 3 bands, e.g., band=[4, 3, 2]
Sample dataset: landsat.tif (7 bands)
from localtileserver import get_leaflet_tile_layer, TileClient
from ipyleaflet import Map

url = "/home/qiusheng/Downloads/landsat.tif"
# First, create a tile server from local raster file
tile_client = TileClient(url)

# Create ipyleaflet tile layer from that server
t = get_leaflet_tile_layer(tile_client)

# Create ipyleaflet map, add tile layer, and display
m = Map(center=tile_client.center())
m.add_layer(t)
m

In QGIS

This is absolutely doable but will require a bit of work. I don't have a timeline for this but I absolutely will implement this


Adding permissions

fixes: #20

Codecov Report
Patch coverage: 74.46% and project coverage change: -0.38 :warning:

Comparison is base (b6ff0a9) 80.45% compared to head (9589d8e) 80.07%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
- Coverage   80.45%   80.07%   -0.38%
==========================================
  Files          20       22       +2
  Lines         706      753      +47
==========================================
+ Hits          568      603      +35
- Misses        138      150      +12

Impacted Files
Coverage Δ

...cks_sdk_python/api_client/workspace/permissions.py
42.85% <42.85%> (ø)

...tabricks_sdk_python/api_client/workspace/client.py
72.22% <100.00%> (+3.47%)
:arrow_up:

...icks_sdk_python/resources/workspace/permissions.py
100.00% <100.00%> (ø)

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

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


Fixed #521 - AltGr combinations not working

Summary of the Pull Request
Let's start fixing #521 and the currently broken state of AltGr combinations, as it heavily affects many non-US users of this project.
PR Checklist

[ ] Closes #521
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #521

Detailed Description of the Pull Request / Additional comments
I'm about 100% sure that this PR is not going to fix #521 by itself, but it's a start. 🙂
If others wan't to contribute to this PR, please feel free to do so. I'll gladly grant push permissions to my fork.
If you want to view the diff here on GitHub I recommend enabling "Hide whitespace changes" in the diff viewer.
Validation Steps Performed
This PR fixes all immediate/obvious issues in regards to AltGr for me in all types of shells I know.
I think it's fairly easy to see why it works, since I simply delegate the handling to WM_CHAR instead.

Alright... I've spent 5 hours on this thing today. Dammit. I hope it was worth it. 😄
This PR should now be far more robust than my basic solution in #521.
I've changed the Terminal class to actually propagate the left/right states of Alt/Ctrl separately down into the TerminalInput class. That way we can continue (or... technically start...) using keyEvent.IsAltGrPressed() in the HandleKey method.
This also has the nice side effect of this PR allowing others to continue pressing Ctrl+Alt without it being confused for a AltGr key press.
For instance if I press AltGr+@ on my german keyboard it will now insert a @ character, BUT if I press Alt+Ctrl+Q it will properly send an escape sequence instead, just like it used to.
I've also fixed the input tests I believe, which now...

Reflect the changes to AltGr being handled by WM_CHAR instead
Added the missing Ctrl+/ sequence check - without it the tests are failing on systems with non-US keyboards

This is exciting. Thanks so much for looking at it! We’ll probably get to reviewing this after the weekend. 😄

@miniksa Thank you for your kind review! I've reverted most parts of the changeset and tried to be as minimalistic as possible this time. You'll find that the diff should be easier to read now.
I could optimize the code structure here and there to use e.g. switches (like I did originally), but I feel like I might not be following any plans you guys might already have in mind for this project. For now I'd like make this PR simple and leave any larger refactorings to you. 🙂
Except of course... Regarding the modifiers bitfield... I also don't think my current approach is acceptable either.
I'm slightly unsure what you mean with "expand it [the Settings::KeyModifiers structure] to have a preference" though.
What I'd like to suggest is to write a class that abstracts over the exact same bitfield used by the TerminalInput module, but provides a simplified API to query for Alt/Ctrl/AltGr states, similar to how KeyEvent does it already. That way this class would stay compatible with the DWORD based APIs. 🙂
Since the diff is relatively minimalistic such changes can be done at any point in time though IMO.
Either way is fine for me (i.e. me writing this class or not as part of this PR).

Please, can you review this issue ? For most people in Sweden, Germany and Switzerland the terminal with WSL is unusable at the moment without this patch.

Also relatively unusable for French users, given that this prevents using all sorts of keys. #{[|\^@]} all require pressing AltGr for French AZERTY keyboards.

Same for Finnish users. Cannot navigate paths as cannot type path separators.

@miniksa Michael Niksa FTE Thank you for your kind review! I've reverted most parts of the changeset and tried to be as minimalistic as possible this time. You'll find that the diff should be easier to read now.
I could optimize the code structure here and there to use e.g. switches (like I did originally), but I feel like I might not be following any plans you guys might already have in mind for this project. For now I'd like make this PR simple and leave any larger refactorings to you. 🙂
Except of course... Regarding the modifiers bitfield... I also don't think my current approach is acceptable either.
I'm slightly unsure what you mean with "expand it [the Settings::KeyModifiers structure] to have a preference" though.
What I'd like to suggest is to write a class that abstracts over the exact same bitfield used by the TerminalInput module, but provides a simplified API to query for Alt/Ctrl/AltGr states, similar to how KeyEvent does it already. That way this class would stay compatible with the DWORD based APIs. 🙂
Since the diff is relatively minimalistic such changes can be done at any point in time though IMO.
Either way is fine for me (i.e. me writing this class or not as part of this PR).
Edit: Ah you're already done reviewing. 😅

If you could write a follow on issue or PR to make it into a nice class that does what you say... that would be ideal.
For everyone else, I'll try to get someone else to 2nd this and we can correct anything that @zadjii-msft finds disagreeable later when he gets back.

@miniksa Do you believe it might be possible to get this PR through until the weekend? Coincidentally I'm going to have some free time then and could work on that. But don't worry if it's not possible. We'll find the necessary time/solution either way, right? 🙂

@lhecker, I think we could merge it today if I can find someone with 15 minutes to read it. Let me keep trying. We have had meetings this afternoon.

@lhecker Just noticed this was merged. Thank you so much for submitting a PR and fixing this issue!


ESlint rule for design token usage

Need acceptance criteria

Closing as not planned. Duplicate https://github.com/MetaMask/metamask-extension/issues/22712


Allow to configure a bit more parameters for Compute VM

This change allows to set CPU, RAM and disk size values for a compute.
In order to get a better integration with ci-framework, some more parameters are now available, in order to tweak the output directories for various artifacts loaded or created by the script.
It also makes a small change as to how the VM disk is created, using image layer (disk image chain). It allows to get rid of some qemu steps. In order to still be able to get the wanted disk size at the end, we now inject a couple of "firstboot-command" in order to resize the disk live.
The whole will allow to run multiple compute nodes, leveraging the EDPM_COMPUTE_SUFFIX parameter, and an integration with the ci-framework by allowing to output files in the common location defined therein.
Also, we ensure we're cleaning things from the correct location, by allowing to pass the same kind of parameters to the edpm-compute-cleanup.sh script.

Guess I can merge it? Or shall I wait for anyone else to do it? @dprince and concerns? :)


list_fields() bug fix

Prb: cannot see list_fields() results, got NULL everytime.
Sol: Change only the path to the result, linked to Elasticsearch 7.4.0

Maybe you could make the logic in process_response conditional on the particular version of ES?

Well, [another repo]{https://github.com/ropensci/elastic} is doing the "all-in-one" package.
My point of view is we could make one specific branch for each version, following the ES versions.
Managing conditions into code could be problematic and hard to fix.


Tiny Toon Adventures 2 - Montana's Movie Madness (U) [!]

Could someone do Tiny Toon Adventures 2 - Montana's Movie Madness (U) [!]?

Hi there,
I've now created a patch for this. Give it a try and let me know if any issues :)

Thanks a lot! I will!

Works perfect!!! Thanks a lot!! Could you please make a video on Youtube how to do it yourself? Thanks again!


Duplicate data

How can i fix it? please.

Can you please provide code examples of what you're having troubles with?

Closing. Please create a new issue with code examples and/or a code sandbox to demonstrate the problem you are facing.


After running build the LoginAuthentication starts to give {authResponse: null, status: 'unknown'} yet when running in development mode its running fine

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

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi. Can you describe clearly step by step your problem? Did you follow my instruction on README?


Wasm failing after unexpected panic

Hi envoy team!
I'm using the proxy-wasm rust sdk (an older version as of now, 1.4.0).
I have an issue where some of my replicas reached an invalid state, after running fine for multiple hours.
[critical][wasm] [source/extensions/common/wasm/context.cc:1181] wasm log rtcdp debatch-egress-1 vm_id: panicked at 'unexpected status: 2', /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/proxy-wasm-0.1.4/src/hostcalls.rs:261:23\n"
[error][wasm] [source/extensions/common/wasm/wasm_vm.cc:38] Function: proxy_on_http_call_response failed: Uncaught RuntimeError: unreachable\n"
" 0: 0x2c01a - __rust_start_panic\n"
" 1: 0x2bffd - rust_panic\n"
" 2: 0x2bfcd - _ZN3std9panicking20rust_panic_with_hook17he2a025723e105e28E\n"
" 3: 0x2b5c1 -_ZN3std9panicking19begin_panic_handler28_$u7b$$u7b$closure$u7d$$u7d$17hd9f8c213ec91b9d5E\n"
" 4: 0x2b500 - _ZN3std10sys_common9backtrace26__rust_end_short_backtrace17h6efd730283875809E\n"
" 5: 0x2bc3f - rust_begin_unwind\n"
" 6: 0x2d3d4 - _ZN4core9panicking9panic_fmt17hb02133958c1e7d35E\n"
" 7: 0x24323 - _ZN10proxy_wasm9hostcalls13get_map_value17h8f76686c8737a7b4E\n"
" 8: 0xa473 - _ZN63_$LT$debatch..Filter$u20$as$u20$proxy_wasm..traits..Context$GT$21on_http_call_response17h38adc080707d07a5E\n"
" 9: 0x1f8ee - _ZN10proxy_wasm10dispatcher10Dispatcher21on_http_call_response17h32cc94c5cad9dc17E\n"

As far as I understand, when I attempted to get the status header from a dispatched http call, the response header map was not actually ready.
fn on_http_call_response(&mut self, _token_id: u32, _num_headers: usize, body_size: usize, _num_trailers: usize) {
  self.get_http_call_response_header(":status").unwrap_or(String::new());
  ....
}

This would not be such a big of an issue by itself, but once this panic showed up, every future request resulted in 503 wasm failed. I have to restart every faulty pod to make them work again.
Debug logs not showing much (the panic stopped showing after the first time):
[2023-03-31 11:59:37.648][65][debug][http] [source/common/http/filter_manager.cc:953] [C68190][S14693529014648669634] Sending local reply with details wasm_fail_stream
[2023-03-31 11:59:37.648][65][debug][http] [source/common/http/filter_manager.cc:930] [C68190][S14693529014648669634] Sending local reply with details wasm_fail_stream directly to the encoder

Is this some known issue with older proxy-wasm crate?

As an update, I tried bumping to the latest proxy-wasm crate, which did not help.
Next I will inspect the num_headers field to see if it's 0, in which case I will stop trying to get the response header map. Will come back with updates.

The behavior should not be to fail all consecutive requests in an unrecoverable state.
Deleting the wasmplugiin and/or rotating the pods is not a viable solution.

Checking for the num_headers field before trying to access the header map seems to have helped, not seeing any more wasm_fails.
However, envoy should handle this situation and not end up in an invalid filter chain state.

I am seeing the same issue.
The panic of the wasm VM made all coming requests fail as they could not go through the wasm filter anymore.
Is there a way that Envoy can limit the issue inside a single HTTP stream instead of at the VM level?

see...

It's possible to re-snapshot VM to a working state, but unfortunately that logic is not implemented in Envoy. Currently, a crash in VM is effectively a downed Envoy, and you need specific probes to detect this condition.


Start New Development Cycle

Start new development cycle.

Forgot to create a branch, no PR for this one, direct commits to develop see 6bb34b8 and 98e89c2


Fix merge conflict from branch-22.04

Fixes the merge conflict noted in #5198.  ElementAt was unshimmed in branch-22.06 which caused the merge conflict, and this was resolved by ignoring the changes to the shim-specific files and only taking the common changes to GpuOverrides.
NOTE: This must be merged with a merge commit, not squashed!

build

build


Investigate skipped tests in Ix

Share4, Finally6.

I think Share4 was disabled because e1.Dispose has no effect on the IEnumerator as it is implemented via a yield return state machine. I don't know if it is even possible or not.
This could be fixed via a manual state machine.
As for Finally6, the Finally() part does not run. I see 3 cases:

I don't know how long a cancellation token is supposed to be valid for a MoveNext so the test might be too fast completing the first task such that Cancel has no effect.
The token could be reacted to until the next MoveNext is invoked thus even if a value is ready, cancel still propagates.
Finally could be registering with the token which it doesn't do currently.

Thanks @akarnokd.
I agree that's the cause for Share4. In the interest of correctness, it's probably worth turning this into a state machine.
Not sure what the right thing is yet for Finally6

I'll post a PR shortly for the two.


python311Packages.opensearch-py: 2.2.0 -> 2.3.1

https://github.com/opensearch-project/opensearch-py/releases/tag/v2.3.1
https://github.com/opensearch-project/opensearch-py/releases/tag/v2.3.0
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 254926 run on x86_64-linux 1

12 packages built:

pentestgpt
pentestgpt.dist
python310Packages.awswrangler
python310Packages.awswrangler.dist
python310Packages.langchain
python310Packages.langchain.dist
python310Packages.opensearch-py
python310Packages.opensearch-py.dist
python311Packages.awswrangler
python311Packages.awswrangler.dist
python311Packages.opensearch-py
python311Packages.opensearch-py.dist


PIC 9(06) type is ignoring the space (hex: 40).

Describe the bug
The Datatype for a field is PIC(06). The hex value is 'F940F0F0F0F0'. The equivalent value is supposed to be '9 0000'. But while reading the file itself cobrix ignoring space and giving '90000'.
Code snippet that caused the issue
  spark.read
   .format("cobol")

Copybook (if possible)
         01  RECORD.
           05  FIELD1    PIC 9(06).

Read_options:
{'encoding': 'EBCDIC', 'ebcdic_code_page': 'cp037', 'string_trimming_policy': 'none', 'debug_ignore_file_size': 'true'}
Could you please confirm the behaviour of cobrix when we have spaces in PIC 9 fields.?
We've seen other instances where Cobrix ignores trailing white spaces in a similar way. We'd like to understand if these are simply ignored from PIC 9 fields and if there is an option to enforce enable / disable that.
@yruslan

Cobrix converts COBOL numeric fields to Spark numeric types (e.g. int, long, double etc). Spark numeric types do not have a notion of spaces between numbers since they are stored in binary representation. If you want to retain spaces in numbers you should use A(06)  or X(06) as the PIC.

@yruslan
One more scenario,
Datatype: PIC 9(02)V9(02)
Hex: 40404040
Since it is repetitive of 40(space), we are expecting null. But cobrix converting this to 0.00
How can we achieve the above result ?

Will check the behavior for all spaces. Meanwhile, could I ask you to make the above scenario a separate issue?

The null detection should be fixed at the master branch now. You can compile it from the source and reference '2.5.0-SNAPSHOT'. Let me know if it fixed the issue for you.


[Docs]: document debugger configuration for common IDEs

Feature scope
Other
Description
As per this comment, it might be beneficial to provide example debugger configuration for 1 or 2 more IDEs (in addition to vscode added in #968) as a precursor to adding config files (contingent on a new question) to the cookiecutter templates.

We do have PyCharm docs now: https://sdk.meltano.com/en/v0.30.0/dev_guide.html#pycharm-debugging


[MRG] Layout created for spatial_colors if not found.

Closes https://github.com/mne-tools/mne-python/issues/3411
Now the layout is generated from channel positions if it is not detected when plotting with spatial_colors.

Looks good to me, thanks @jaeilepp !

[Current coverage][cc-pull] is 86.67%
[][cc-pull]

No coverage report found for master at 0fbc4cd.

Powered by Codecov. Last updated by [0fbc4cd...f23963e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/mne-tools/mne-python/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/mne-tools/mne-python/compare/0fbc4cd0b54e3e51b3451e2c9f69fea194f3a633...f23963e479b901b3568f1971a87ca72771b2a797?src=pr
[cc-pull]: https://codecov.io/gh/mne-tools/mne-python/pull/3414?src=pr

I can confirm it works for me

thx @jaeilepp you deserved your tennis game...

I sure did.


fix: support deprecated draftMode preview URL config

Should fix #2136, and will now console.warn if the deprecated previewUrl.draftMode is used, and nudge towards using previewUrl.previewMode 🙌

@rdunk Great catch! 🤦♂ fixed on main! 🙌


Use real names and email as display names for managing access

In the access management interface use the users real name followed by email in parentheses to identify users.  See illustration

This makes all users email addresses public. I think you should strongly reconsider this. Most users will (rightly so) consider this an egregious privacy violation.

This will get addressed with the redesign of the sharing interface.  But, we should make sure we address any privacy concerns before we implement something.

Is there an alternative that better respects privacy.  @alvacouch what do you think?  I thought quite hard about this while writing the document about usernames and emails that is at https://github.com/hydroshare/hydroshare/blob/master/docs/DesignDocs/UserNameEmailRules.md.  We need a way for someone wanting to share a resource with someone to be able to identify and find them.  In the document I stated that emails in HydroShare are not private, because I think it is impossible to guarantee their non disclosure in a system of this nature where they can be part of metadata records for resources that may be public.  However we do try make it hard for spammers.  In the document I suggested a messaging system rather than emails for directly contacting other users, but given the non uniqueness of names, the arbitrariness of usernames, I think that seeing an email address is necessary to make sure a resource is being shared with the right person.

Lets use Firstname Lastname, organization (username) as the identifier string in all places on this interface consistently and not display emails.
Other recommendations

Avoid the multi-select.
List with check boxes for the permissions being given (owner, change, view)
Have a definitive remove icon for removing users
@ironfroggy  will post a mockup

See #201.  @mauriel133 this relates to the branch you are working on

This is implemented in 1.6.0 and 1.6.1 deploys.


Fix the style of displaying drive sizes

Resolved / Related Issues
This pr fixes the style of displaying drive sizes (Property and Tooltip).

In the tooltip, the background of the ring was missing.
In the tooltip, the 3 texts did not use exactly the same style.
In light mode, the color of the free space legend did not match the color used by the ring. The color of the ring has been used because it is more readable and this corresponds to the progressbar of the Drive widget.

Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)
Before Property

After Property

Before Tooltip

After Tooltip

LGTM.


From and To labels should be translatable

The other way its to pass label on props
<DateRangeCalendar labelFrom="From" labelTo="To"/>
And change DateRangeCalendar.tsx line 145
    const fromLabel = this.props.labelFrom;     const toLabel = this.props.labelTo;
And
npm run build


android studio build got error : duplicate entry: module-info.class

how to fix it

This library is not ready for android, but we have a slightly different alternative


frost: Isolate creating shares and creating PoP

After discussion today, we do not need to know the other partys' polynomials in order to create their secret shares, PROVIDED we have some pre-existing way to assign the other parties an index
    pub fn create_shares(
        &self,
        party_indexes: impl Iterator<Item = usize>,
        scalar_poly: &Vec<Scalar>,
    ) -> Vec<Scalar<Secret, Zero>> {

    pub fn create_proof_of_posessions(
        &self,
        keygen_id: &[u8],
        scalar_poly: &Vec<Scalar>,

This PR is a step towards passing non-standard participant indexes into frost key generation as opposed to 1, 2, 3, ....
I also added   create_shares_and_pop() which behaves similar to the existing usage before this PR.
I did this primarily because the iterator of indexes is not so friendly to pass in 1..=n_parties, and we will soon be adding a sorting algorithm.

TODO: Use BTreeMap in new_keygen and elsewere, preventing duplicate party indexes