Add refine option to quick open

Splitting this out from #1281,

Some fuzzy-finders have a 'refine' approach - where you could type main.tex, realize there's a bunch of them, but realize you're looking for the one in research folder, so you could then refine the result by typing space + research (so the filter text would be main.tex research). This would turn the filter into a two-pass filter - first, filter all the results with main.tex, then filter all of those results with research. In addition, if we sorted the search results by length, this might be a cheap way to make the fuzzy finder significantly more useful.

I'd certainly find something like this useful, so may have a look into adding it, if it isn't picked up before I get some free time.

@CrossR Where is the content research placed? If it's on the folder you can use the regex strategy and type researchmain.tex.
We could also adjust the regex strategy so that it works similar to other fuzzy finders where if you separate the words with space it tries to match both, e.g. main.text research as you intended. This is the ideal behaviour for the regex strategy for me.

We could also adjust the regex strategy so that it works similar to other fuzzy finders where if you separate the words with space it tries to match both, e.g. main.text research as you intended. This is the ideal behaviour for the regex strategy for me.

Yeah I think having the ability to stick a space between is nice, just because I think a lot of people think along the lines of "I need file X" then realise they get back 20+ results so go "Ooops I mean I know its in this folder", but with the current system we can't really easily add that new folder on, we'd have to delete the query and start again. Being able to just react and type the folder on with a space between makes it a lot easier.
Does it make sense to do it as the original post describes?
I.e. can you see any issue with doing the second terms search on only the results of the first? Or should it be a single search with both terms? Not sure if it changes anything really.

Makes sense and looks simpler to do it that way!
Slightly off topic for this issue but I'm looking at the regex stuff now, and currently we convert every term using the following:
  const filterRegExp = new RegExp(".*" + currentTerm.split("").join(".*") + ".*")
Which would turn my example of research into .*r.*e.*s.*e.*a.*r.*c.*h.*, where I would have expected .*research.* ie anything with the word research in, rather than those letters appearing anywhere in the term in order, but with anything between.
Does it make sense for that to be "fixed"? (I use quotes since that could be the desired behaviour, just seems odd to me). Kinda feels like it may be the root of some of the issues you and others have been having.
Other than that...this was basically just splitting the term on a space and stick a for loop around the current logic. If its that easy for the non-regex (or it makes sense to add it there), I can do it there too. Need to check performance issues of that but I wouldn't expect any changes.

Tracking additional refinement proposals by @CrossR (https://github.com/onivim/oni/issues/1266#issuecomment-368906703):

I've had a look into this in other editors, and I think there is a few things we can potentially do, put I'm not fully sure of their cost.
By the looks of VSCode they use a big scoring algorithm to apply scores to the results, then that is used for the sorting : https://github.com/Microsoft/vscode/blob/master/src/vs/base/parts/quickopen/common/quickOpenScorer.ts
Looks to basically just give higher scores depending on if certain criteria are met.
We could potentially do something like that, but it could also just slow down the search or cause more issues when we have to add edge cases after edge case. Though, there were still some issues complaining it wasn't as good as FZF still.
That said, it would improve the search further, since currently I don't think we do anything like this.

@bryphe A simpler way to achieve this would be by just adding the word separated by space as a new term in the regex, no performance penalty and pretty predictable. I would love that as an alternative to the scorer.

@badosu from my understanding, the scorer would work along side any filtering, with it just being used to sort the results from the filter. The refinement is useful, but I still sometimes get weird results.
I'm also not 100% on the regex strategy or if I should add multiple types. Ie the way I've done it currently I do each as a separate term, such that the second term is only searching the result of the first and so on. This feels pretty natural to me since I whittle down the set I can currently see.
Would you also want a mode that you just add on terms with a space and they are appended to the first search term? Or am I misunderstanding?

A simpler way to achieve this would be by just adding the word separated by space as a new term in the regex, no performance penalty and pretty predictable.

Regardless I'll stick up a WIP PR for what I have now, and have a look. It feels a lot better now, and I think with some more intelligent sorting of the results we should be great! Though that said adding this with multiple search terms could make scoring a whole bunch more awkward, at least if we attempt to take the VSCode bit as is.

I'm also not 100% on the regex strategy or if I should add multiple types. Ie the way I've done it currently I do each as a separate term, such that the second term is only searching the result of the first and so on.

Yeah, it's the same thing. As I said, if the terms are exclusive, searching terms separated by space (e.g. 'index controller' => /index|controller/) is the same thing as searching for one then the other.
For scoring I usually prefer to have it simple and predictable, e.g. frequency and order of the terms. Unless you're managing a really huge and complex file tree I don't see the overhead of having the scorer as an advantage.

The filtering aspect of this has been addressed, but there is still the scoring aspect to be considered here.

I've moved this out into its own ticket to make it a bit more focused.


cfn_nag fails for CloudWatchLogs for KinesisStreamToS3

cfn_nag is enforcing rules for CloudWatchLog for retention period to be specified in days and supply a KMS key for encryption. The construct should add rules to suppress the following:
Reproduction Steps
Create a construct using
const firehoseToS3 = new KinesisFirehoseToS3(this, 'KinesisFirehose', {
    kinesisFirehoseProps: defaultKinesisFirehoseProps,
    existingBucketObj: props.s3Bucket
});

Error Log
| WARN W84
|
| Resources: []
| Line Numbers: []
|
| CloudWatchLogs LogGroup should specify a KMS Key Id to encrypt the log data
| WARN W86
|
| Resources: []
| Line Numbers: []
|
| CloudWatchLogs LogGroup should specify RetentionInDays to expire the log data
Environment

**CDK CLI Version      :1.85.0
**CDK Framework Version:1.81.0
**AWS Solutions Constructs Version     :1.81.0
**OS                   :MacOS
**Language             :Typscript

Other

This is :bug: Bug Report

As a workaround, I am adding the following code to suppress cfn_nag.
  (firehoseToS3.node.findChild('firehose-log-group').node.defaultChild as CfnLogGroup).cfnOptions.metadata = {
      cfn_nag: {
          rules_to_suppress: [{
              id: 'W84',
              reason: 'Log group data is always encrypted in CloudWatch Logs using AWS Managed KMS Key. For customers wanting to us CMK for \
              CloudWatchLogs should customize the solution further to add encryption options'
          }, {
              id: 'W86',
              reason: 'Log Groups are set to \'Never Expire\'. Customers should customize the retention policy based on their organization\'s \
              retention policies'
          }]
      }
  }

As a workaround, I am adding the following code to suppress cfn_nag.
  (firehoseToS3.node.findChild('firehose-log-group').node.defaultChild as CfnLogGroup).cfnOptions.metadata = {
      cfn_nag: {
          rules_to_suppress: [{
              id: 'W84',
              reason: 'Log group data is always encrypted in CloudWatch Logs using AWS Managed KMS Key. For customers wanting to us CMK for \
              CloudWatchLogs should customize the solution further to add encryption options'
          }, {
              id: 'W86',
              reason: 'Log Groups are set to \'Never Expire\'. Customers should customize the retention policy based on their organization\'s \
              retention policies'
          }]
      }
  }

Fix for this has been pushed out in the latest release v1.82.0

Fix for this has been pushed out in the latest release v1.82.0


Feat: Add collection

Fixes #
Changes

Check how many nfts are left to mint
Add a button to mint 1 token in the verify page

Checks

[X] Have you catchup your branch with the latest state in base?

[X] Have you reviewed your proposed changes and removed debris?
[X] Have you successfully ran due tests with your changes?

Closes #38, closes #83

@cpl121 any idea why the preview is failing?

Build solved and merge done @begonaalvarezd


Read Data From Zip Files

Summary
Allows for mod assets to be loaded from inside a zip file. Implements #236
Changes

Implemented a ZipFileHandler class
Added logic to Game.getInternal() to detect if a zip file is part of the path and returns a ZipFileHander object.
Added support to load assets from an assets.zip file
Deprecated Game.getInternal()
Renamed Game.getInternal() to Game.resolveFile()

Testing
I've currently only tested this with the Guns mod zipped up and put in my repo's mod folder.

Discussed this on Discord. Issue with list and order of folder resolvment.


community: Change post-filters to pre-filters in BigQueryVectorStore

PR Description
BigQueryVectorStore's _create_filters are post-filters. Now that BigQuery Vector Search supports pre-filtering on stored columns (https://cloud.google.com/bigquery/docs/vector-index#stored-columns), we can change all post-filters to pre-filters.
Note that this requires storing the columns that are referenced by the pre-filter. This cannot be done in Langchain yet (have to use the BQ CLI/UI directly). If the required columns are not stored, then the pre-filters will act as post-filters.
Relevant issues

Type

🆕 New Feature
Changes(optional)
Change the vector search query from:
SELECT *
FROM VECTOR_SEARCH(
  Table <table>,
  ...)
WHERE <filter>

to
SELECT *
FROM VECTOR_SEARCH(
  (SELECT * FROM <table> WHERE <filter>),
  ...)

Testing(optional)
Run unit and integration tests.

Note(optional)

@lkuligin for final approval

we should start increasing the test coverage, please. It should have at least a unit test.

we should start increasing the test coverage, please. It should have at least a unit test.

I know we should have more unit and integration tests, but can we do that later once we have a owner for BigQueryVectorStore? I struggled for 2 hours to write a unit tests using mocks and was not able to make it work (I'm neither proficient in Python nor Langchain infra).

@lkuligin What do you think? Can we add more testing later?

@eliasecchig merging for now, but, please, plan to improve the testing coverage for this component in the future


Fix responsiveness

On small screens it doesn't look nice

It looks like it only happens on my Firefox Nightly

media query was added.


upgrade spring boot version to 2.4.3 to make our projects support spring-native 0.9 beta

spring-native released a beta version, support spring boot 2.4.3, so our company plans to use it, so hope the next version of redisson boot starter upgrade to the spring boot version.

We have exactly the same plans

Done


run tests on 7.4

@freekmurze seems build is failed because of old symfony/console package
see
https://github.com/symfony/console/commit/93261ed9e543031667431102d3a4846269632dae
it has been fixed in v4.3
Not sure how to deal with this

We'll be moving to GitHub Actions, and we'll be sure to test on PHP 7.4 too.


Aquarea heat pump frequently reporting as "No response" in homekit

Describe Your Problem:
My heat pump's heating setting range from -5 to 5, the integration seems to be able to se it even higher or lower than that, uppon which it becoms unresponsive. Since heating season is almost over at our location, i have it manually set to -5 (lowest possible to maintain temperature at night), and when that is set, it reports as no response in homekit. when i set it to the dot, in the thermostat entity, it sets it to the 0 value on heat pump, and is then reponsive again. but then its heating too much for my needs. the entities are responsive in homebridge though.
Logs:
[3/25/2024, 7:40:51 AM] Homebridge v1.7.0 (HAP v0.11.2) (Homebridge D9B1) is running on port 51574.
[3/25/2024, 7:40:55 AM] [PanasonicHeatPumpHomebridgePlugin] Authenticated to Aquarea Smart Panasonic
[3/25/2024, 7:40:55 AM] [PanasonicHeatPumpHomebridgePlugin] Adding new accessory: Pralnica
[3/25/2024, 7:40:55 AM] [PanasonicHeatPumpHomebridgePlugin] Adding new accessory: Pralnica
[3/25/2024, 7:40:55 AM] [@hernas/homebridge-panasonic-heat-pump] This plugin generated a warning from the characteristic 'Active': Characteristic not in required or optional characteristic section for service Thermostat. Adding anyway.. See https://homebridge.io/w/JtMGR for more info.
[3/25/2024, 7:48:19 AM] [PanasonicHeatPumpHomebridgePlugin] Authenticated to Aquarea Smart Panasonic
[3/25/2024, 7:52:12 AM] [Homebridge UI] [denisbobovnik] Accessory layout changes saved.
[3/25/2024, 7:52:29 AM] [Homebridge UI] [denisbobovnik] Accessory layout changes saved.
[3/25/2024, 7:52:43 AM] [Homebridge UI] [denisbobovnik] Accessory layout changes saved.
[3/25/2024, 7:53:06 AM] [Homebridge UI] [denisbobovnik] Accessory layout changes saved.
[3/25/2024, 8:04:49 AM] [Homebridge UI] [denisbobovnik] Accessory layout changes saved.
Plugin Config:
"email": "removed",
"password": "removed",
"enableOutdoorTempSensor": true,
"enableEcoModeSwitch": false,
"enableComfortModeSwitch": false,
"refreshTime": 5,
"platform": "PanasonicHeatPumpHomebridgePlugin"
Screenshots:

Environment:

Plugin Version: 2.0.1
Homebridge Version:  1.7.0
Node.js Version:  20.9.0
NPM Version:  10.1.0
Operating System:  macOS Sonoma

I used to have same issue with random "no response" in homekit but after upgrading to 2.01. problem disappeared

@denisbobovnik Hey, I know about the issue but I could never really debug it correctly. Even before Panasonic changed their API I am randomly getting "No response" for the heater (but not for the water tank).
At least now I have a clue, that it happens with -5, so maybe my calculations in the code are incorrect which breaks the homekit integration somehow. As I don't have 100% reproducible scenario, it is hard for me to find the issue. The code looks ok.

and now, have no idea why the same issue is reported in homekit

Can you post the logs? I will take a look at this on weekend. I feel something is off with the accessory setup, but I tried adding caching and always returning the value on get method so that is surprising.

[25/03/2024, 14:10:33] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [7080972b-acc5-4ee0-be5e-95907e876663]
[25/03/2024, 14:10:33] [@hernas/homebridge-panasonic-heat-pump] Updating outdoor temp: 8
[25/03/2024, 14:10:42] [@hernas/homebridge-panasonic-heat-pump] Readings: {
"temperatureNow": 40,
"heatingCoolingState": 1,
"targetHeatingCoolingState": 1,
"outdoorTemperatureNow": 8,
"tankTemperatureNow": 45,
"tankTemperatureSet": 45,
"tankHeatingCoolingState": 0,
"tankTargetHeatingCoolingState": 1,
"isActive": true,
"ecoModeIsActive": false,
"comfortModeIsActive": false,
"tankTemperatureMax": 65,
"tankTemperatureMin": 40,
"targetTempSet": -1,
"targetTempMin": -5,
"targetTempMax": 5,
"tempType": "heat"
}
[25/03/2024, 14:10:42] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [3c7ca285-dfb4-4f5f-8116-324ca4389cbf]
[25/03/2024, 14:10:42] [@hernas/homebridge-panasonic-heat-pump] Updating TargetTemperature of Floor Heater: {"minValue":35,"maxValue":45,"minStep":1}
[25/03/2024, 14:10:42] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [7080972b-acc5-4ee0-be5e-95907e876663]
[25/03/2024, 14:10:42] [@hernas/homebridge-panasonic-heat-pump] Updating outdoor temp: 8
[25/03/2024, 14:10:51] [@hernas/homebridge-panasonic-heat-pump] Readings: {
[25/03/2024, 14:10:51] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [3c7ca285-dfb4-4f5f-8116-324ca4389cbf]
[25/03/2024, 14:10:51] [@hernas/homebridge-panasonic-heat-pump] Updating TargetTemperature of Floor Heater: {"minValue":35,"maxValue":45,"minStep":1}
[25/03/2024, 14:10:51] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [7080972b-acc5-4ee0-be5e-95907e876663]
[25/03/2024, 14:10:51] [@hernas/homebridge-panasonic-heat-pump] Updating outdoor temp: 8
[25/03/2024, 14:11:00] [@hernas/homebridge-panasonic-heat-pump] Readings: {
[25/03/2024, 14:11:00] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [3c7ca285-dfb4-4f5f-8116-324ca4389cbf]
[25/03/2024, 14:11:00] [@hernas/homebridge-panasonic-heat-pump] Updating TargetTemperature of Floor Heater: {"minValue":35,"maxValue":45,"minStep":1}
[25/03/2024, 14:11:00] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [7080972b-acc5-4ee0-be5e-95907e876663]
[25/03/2024, 14:11:00] [@hernas/homebridge-panasonic-heat-pump] Updating outdoor temp: 8
[25/03/2024, 14:11:09] [@hernas/homebridge-panasonic-heat-pump] Readings: {
[25/03/2024, 14:11:09] [@hernas/homebridge-panasonic-heat-pump] Updating: Trojanki [3c7ca285-dfb4-4f5f-8116-324ca4389cbf]
[25/03/2024, 14:11:09] [@hernas/homebridge-panasonic-heat-pump] Updating TargetTemperature of Floor Heater: {"minValue":35,"maxValue":45,"minStep":1}

Sorry my mistake wrong naming, I have randomly heat pump ‚no response’ With water it is always  ok.

I get the feeling that depending on the settings of the heatpump we are getting wrong/missing values from the API
I can't reproduce now, but I saw this
[03/04/2024, 16:58:36] [@hernas/homebridge-panasonic-heat-pump] Readings: {
    "temperatureNow": 27,
    "heatingCoolingState": 1,
    "targetHeatingCoolingState": 3,
    "outdoorTemperatureNow": 10,
    "tankTemperatureNow": 47,
    "tankTemperatureSet": 48,
    "tankHeatingCoolingState": 0,
    "tankTargetHeatingCoolingState": 1,
    "isActive": true,
    "ecoModeIsActive": false,
    "comfortModeIsActive": true,
    "tankTemperatureMax": 65,
    "tankTemperatureMin": 40,
    "tempType": "comfort"
}
[03/04/2024, 16:58:36] [@hernas/homebridge-panasonic-heat-pump] Updating: Wärmepumpe [100d7607-3a4d-4e7d-a0da-b8a102c5414b]
[03/04/2024, 16:58:36] [@hernas/homebridge-panasonic-heat-pump] Updating TargetTemperature of Floor Heater: {"minValue":null,"maxValue":null,"minStep":1}

When I checked the official app / aquarea-smart.panasonic.com it was set to -5 and comfort mode (unknown how this happened, I never set it to that value manually)

@mastaab  Please check v2.0.2


[BugFix] Should not collect tablet stats of shadow index

Fixes #issue
What type of PR is this:

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

Checklist:

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

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

Bugfix cherry-pick branch check:

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

[x] 3.1
[x] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/TabletStatMgr.java
2
2
100.00%
[]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0


RDD Graph (spree part)

This PR adds functionality to display RDD DAG in Stage and Job views, very similar (essentially copy of Spark UI visualization) with some minor tweaks for React.
Should be merged together with similar PRs for json-relay and slim.

There is a bug with not displaying skipped stages correctly.

thanks @sadikovi, sry for the continued delay, I am trying to page all this stuff in ASAP, but still waylaid; had to move apartments immediately when I returned from traveling on Sunday! Thanks for your patience, I look forward to reviewing all these!

There is also bug with displaying stages in Job page, stages should be sorted by id, otherwise all DAGs become messed up.
Spree DAG:

Spark DAG:

@ryan-williams thank you very much!
I hope that with my PRs did not break spree functionality:)

query used to test this job:
val df1 = sqlContext.range(0, 100).filter($"id" > 3)
val df2 = sqlContext.range(0, 10000).filter($"id" > 12)
val df = df1.join(df2, "id").cache()
df.select($"id", lit(1), lit(true)).cache().count

Fixed sorting problem and marking skipped stages, note that these changes are valid only with corresponding changes in spark-json-relay and slim.

ping @ryan-williams


Not working for iOS

Once I install this plugin and try to run the ios app it hangs, the app just gets stuck with this as the last line.
[   +4 ms] executing: /usr/bin/xcrun simctl launch B879ED90-4B32-4B79-A329-473935AFCD2D com.example.app
--enable-dart-profiling --enable-checked-mode --verify-entry-points
[ +236 ms] com.example.app: 69799
[        ] Waiting for observatory port to be available...

The android app is still able to run.

If anyone comes across this issue where ios simulator gets stuck, it maybe because of the incompatibility of the package and the connectivity package. I used data_connection_checker instead and it worked.


create llama_hub package

this requires moving existing folder (loader_hub) to llama_hub, which requires changes on the llamaindex repo: https://github.com/jerryjliu/llama_index
Currently I'm still building the packages manually, will take a TODO to fix

@jerryjliu I'm getting an error trying to use this loader. I'm using it like this:
# load documents from s3 directory using OpendalS3Reader
OpendalS3Reader = download_loader("OpendalS3Reader")
loader = OpendalS3Reader(
    bucket='cr3po-test',
    access_key_id='[access_key_id]',
    secret_access_key='[secret_access_key]')
docs = loader.load_data()

 docs = loader.load_data()
  File "/Users/lol/anaconda3/lib/python3.10/site-packages/llama_index/readers/llamahub_modules/opendal_reader/s3/base.py", line 65, in load_data
    loader = OpendalReader(
  File "/Users/lol/anaconda3/lib/python3.10/site-packages/llama_hub/opendal_reader/base.py", line 44, in __init__
    self.op = opendal.AsyncOperator(scheme, **kwargs)
pyo3_runtime.PanicException: must be valid hashmap: PyErr { type: <class 'TypeError'>, value: TypeError("'NoneType' object cannot be converted to 'PyString'"), traceback: None }

Any insight would be appreciated.

Also sorry I'm not entirely sure where to reach out for issues with the loaders.
Additionally I cannot get the following loader to work with locastack: https://github.com/emptycrown/llama-hub/tree/main/llama_hub/s3, I try and set the s3 endpoint and it does nothing.
It seems to still attempt to just use the default s3 endpoint. Any help is appreacited.


Minimal working docker install guide w/ py3.7/py3.8/py3.9?

Hi guys,
Is there any chance you could provide a minimal working install for a docker file with pip and python >=3.7? Currently the docs only show how to do this with pip but the install doesn't work with for example the following dockerfile:
FROM python:3.7.10-slim
# INSTALL BLAS
RUN apt-get update && apt-get install libatlas-base-dev liblapack-dev libblas-dev -y
# THIS FAILS
RUN pip install cvxpy

I've tried a number of ways to install with the openblas package but nothing seems to work. On a mac 10.15.7, I manage to install cvxpy with python 3.7.10 and numpy 1.19.2 but not numpy 1.20 (latest). But the docker file above does not work.
Having a minimal working install for the higher python versions >= 3.7 would be tremendously helpful, both on docker with pip/poetry or on a mac, e.g. 10.15 with numpy 1.20.

@matzhaugen You closed the issue, so I hope you already found a solution.
If someone else finds this issue, below is a minimal working Dockerfile. However, working with conda might be an option to consider as well.
FROM python:3.7.10-slim
RUN apt-get update && apt-get install build-essential -y
RUN pip install cvxpy

There is usually no need to install openblas, as it is the default blas version that comes with the numpy wheel (see here under "NUMPY PACKAGES & ACCELERATED LINEAR ALGEBRA LIBRARIES".
Indeed, we can check this by running:
docker build . -t cvxpy
docker run -it cvxpy bash
python

import numpy
numpy.show_config()
>>> openblas_info:
    libraries = ['openblas', 'openblas']
    library_dirs = ['/usr/local/lib']
    language = c
    define_macros = [('HAVE_CBLAS', None)]

Hey @phschiele yes, I found a solution, which was basically your solution. It seems like the numpy build system is complicated with the mac accelerate backend, which was why I was resorting to docker. But it seems like the ECOS and and SCS are having similar issues.
https://github.com/embotech/ecos-python/issues/29
https://github.com/bodono/scs-python/pull/38

@matzhaugen We are working on wheels for Linux at the moment, such that you can directly pip install cvxpy in the Dockerfil without compiling. The PRs you linked also will fix some conflicts with certain numpy versions.

Excellent, thanks!

New wheels for ECOS are online, going up to Python 3.9.


TraceWriter breaks serialization

Ran into an issue where setting a TraceWriter on my serialization settings breaks serialization.
Similar to issue #793
I setup a small repoducable repo here as requested: https://github.com/volak/jsonNetIssue
Exception: Token String in state Object would result in an invalid JSON object. Path ''.
I found reproducing the issue is dependent on defining a TraceWriter AND the custom json converter.  In my project I use this Id class to represent a string, guid, or long id value - and I added a json converter to help json.net serialize it.
Without the TraceWriter it serializes fine - with TraceWriter and without the custom converter it serializes but obviously doesn't properly handle the Id field.

Thanks for the repo. I have fixed this issue - https://github.com/JamesNK/Newtonsoft.Json/commit/64f6fd7b39f9371122144eba0bed5eb19e061ec0


[SPARK-18886][Scheduler][WIP] Adjust Delay scheduling to prevent under-utilization of cluster

What changes were proposed in this pull request?
This is a significant change to delay scheduling to avoid under-utilization of cluster resources when there are locality preferences for a subset of resources.  The main change here is that the delay is no longer reset when any task is scheduled at a tighter locality constraint.  Instead, each task set starts the locality timer the first time it fails to utilize a resource offer due to locality constraints.  One task set never tightens the locality constraints, even if subsequent offers are made that utilize tighter constraints.
A more complete description of the issues w/ the previous scheduling method can be found under the jira.
How was this patch tested?
Added unit test for original issue.  Ran all unit tests in o.a.s.scheduler.* manually.  Full tests via jenkins.
TODO

[ ] add more unit tests, especially for recompute locality levels.
[ ] code cleanup (especially all the logging added)

Test build #70412 has started for PR 16354 at commit 449ba20.

@mridulm @markhamstra @kayousterhout
This is not ready to merge -- it needs some cleanup and more tests -- but I thought that seeing an implementation might help think through the design.  I think the discussion should still center on the overall approach, and that discussion should probably happen on the jira, not here.  (I'll happily fix code issues if that would help.)
Obviously, this is a pretty big change to the way delay scheduling works; I'd gladly consider alternative approaches that dont' involve such a large change in behavior, but I don't see them.  IMO this problem is serious enough that it merits the large behavior change.

Test build #70412 has finished for PR 16354 at commit 449ba20.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70412/
Test PASSed.

closing this since on the jira we decided that a different approach is needed.


needs.sh: use relative service directory for dependency check

This allows full use of the needs.sh dependency check (brilliant idea btw).
Now service directory is checked rather than service name, which means
that it works with any service directory like /var/service,
instead of just /service.

For some systems, dependency resolution is already present at the resource management level (which is just above service supervision, and typically coordinating it).  The ./needs.sh script is really for those systems that have "dumb" resource management, aka SysV-styled rc.d scripts that use sequencing to emulate dependency resolution by making sure "everything is up in the correct order".
The idea was, if a service absolutely needed another service to be up, and the system had no way to accommodate that, then ./needs would step in and fill that gap.  If you are using native runit, which has no dependency manager, then this script makes sense.  The runit examples have hard-coded calls for other services in them, but that's hardly portable, so it makes sense that we abstract that away with ./needs, which smooths over this problem while making it portable.  If you are using s6 for supervision along with s6-rc to manage it, then while it will technically still function, it will be redundant, as s6-rc will bring up dependencies before even launching the ./run script.  That last example of s6-rc is why ./needs is disabled by default, because at the moment, we have no way of knowing if dependency management is installed at run-time.
It's nice that the calling sequence of ./needs self-organizes into a dependency tree, but it has a weakness.  It can only bring all needed services up, but it will never bring them down.  So, I view ./needs as a stop-gap measure for systems that cannot support true service dependency management.

Agree with you @apayne . Using runit here, the needs framework is welcome. Especially with fast SSDs and services like lightdm which fail hard without dbus being started correctly beforehand.
Dependent services not being brought down is not an issue here. Something that would be further appreciated is dependency ordering. While writing this post I got an idea that it could be done with your needs framework the same way its done with sysv-rc, ie, via naming like 01-sv1, 02-sv2, etc. :)
Appreciate the detailed response.


[FR] Jimp node

@RafaPolit What do you think about the idea to make a generic jimp node?
I use now a function node to resize images, but maybe a "jimp node" would be helpful for more people...
Here is the sample code:
JIMP.read(msg.payload).then(function(image) {
    msg.image.width = image.bitmap.width;
    msg.image.height = image.bitmap.height;
    image.resize(newwidth,JIMP.AUTO);
    image.quality(quality);
    msg.image2.width = image.bitmap.width;
    msg.image2.height = image.bitmap.height;
    image.getBuffer(image.getMIME(), onBuffer);
}).catch(function (err) {
    // handle an exception
    if (err) throw err;
});

function onBuffer (err, buffer) {
    if (err) throw err;
    msg.payload = buffer;
    node.send(msg);
}

I think it would be interesting, but since someone can just set jimp as a global and have access to it in functions, I'm not sure what one would gain from having a node that just exposes jimp?

Sure, but for a novice it's not that easy:
First you have to know about jimp - but the node could provide the keywords "scale","crop","resize", "image processing" and so on.
One has to install the library manually - as node, the installation routine takes care of the dependencies.
One has to fiddle in the settings file...handle the callbacks, parameters, etc
I could try it myself, but i dont have experience with javascript and node creation - so i decided to ask you before.


feat(FMP-333): overhaul flow states

closes FMP-333
closes FMP-334
closes FMP-372

Current dependencies on/for this PR:

main

PR #66   👈

This comment was auto-generated by Graphite.


Incompatible version 2.2-2.1

I continue to receive this error of version incompatibility. Should i try to downgrade to version 2.1?
thanks, Dan

You can modify the script, update the version check and see if it works.
Find the line for compatible_major_version and change the value:
compatible_major_version = "2.2"

Hello, we are planning to update the scripts according to the API changes in version 2.2 along with the full release of Metashape 2.2 which will happen presumably in late December or early January.
Like @jasonrmh said, you can workaround Incompatible Metashape version: ... error with manual fix in the script (with replacing 2.1 -> 2.2).
After that the script will try to execute, but may fail with errors in case it uses something from the Python API that has changed. If so - please provide full log with encountered errors.

Should i try to downgrade to version 2.1?

Alternatively, it's the most reliable option.

You can modify the script, update the version check and see if it works. Find the line for compatible_major_version and change the value: compatible_major_version = "2.2"

Thanks!

Hello, we are planning to update the scripts according to the API changes in version 2.2 along with the full release of Metashape 2.2 which will happen presumably in late December or early January.
Like @jasonrmh said, you can workaround Incompatible Metashape version: ... error with manual fix in the script (with replacing 2.1 -> 2.2).
After that the script will try to execute, but may fail with errors in case it uses something from the Python API that has changed. If so - please provide full log with encountered errors.

Should i try to downgrade to version 2.1?

Alternatively, it's the most reliable option.

Thanks!


SW3: Expand cents color scheme

When selecting "Cents" color scheme, user can input two things:

the edo number (color wheel repeats on every 1\n interval, current color scheme is practically 1edo)
the color phase (which color is 1/1 on?)

This will visualize the approximation strength of an edo against a scale.

The "Cents" coloring scheme is focused on 1edo, but it is not strictly periodic.
A new builtin should be made for approximation strength viz: https://github.com/xenharmonic-devs/sonic-weave/issues/141

Meanwhile, here's how you do it manually
ed(53) // Some scale

// Color everything based on 12edo
i => i hsl(360 * i % 1\12, 255, 50)


Return nextTick Promise from wrapper interactions

What problem does this feature solve?
We recently updated our Vue test suite to beta 33 from beta 29, and a lot of our tests broke because we were expecting many DOM changes to take effect immediately, when now they happen after nextTick. This is fine: it's well documented and mimics how a component behaves in "real life", so no problems there.
We had to pepper our tests with a lot of await wrapper.vm.$nextTick, and figured we might be able to drop a few lines if we can have the interactions with our wrapper objects return a promise instead of the tests having to create and await them. Normally we would call it after each setData, setProps or trigger call to the wrapper, but I'm sure there are other methods that could benefit from it as well.
Maybe this is not too disruptive, since those methods currently return void, so I'm guessing most workflows would be unaffected. It would just offer an alternative to those who prefer a shorter syntax.
The Vue function is already imported in test-utils/src/wrapper.js, so I ran a quick test and this worked fine:
/**
 * Dispatches a DOM event on wrapper
 */
trigger(type: string, options: Object = {}) {
  // ...

  const event = createDOMEvent(type, options)
  this.element.dispatchEvent(event)

  return Vue.nextTick()
}

What does the proposed API look like?
Taking the example listed in the current documentations for using nextTick:
// Before (would still work as is)
it('button click should increment the count text', async () => {
  expect(wrapper.text()).toContain('0')
  const button = wrapper.find('button')
  button.trigger('click')
  await wrapper.vm.$nextTick()
  expect(wrapper.text()).toContain('1')
})

// After
it('button click should increment the count text', async () => {
  expect(wrapper.text()).toContain('0')
  const button = wrapper.find('button')
  await button.trigger('click')
  expect(wrapper.text()).toContain('1')
})

Doesn't seem like much of a trim, but in our case at least it adds up to quite a few lines.
I'd love to know what you guys think, and thanks for your effort on this library.

Hey, it's in the pipeline for V1. We will probably ship it very soon.

That's great to hear, thanks for letting me know! Looking forward to it.


After updating @angular to rc5, can't import router module

Can't import module router.
I've checked that this module is actualy exists in node_modules/@angular/router.

Mac OSX El Capitan

Version: angular-cli: 1.0.0-beta.10

import { Routes, RouterModule } from '@angular/router';

Log:
Error: Typescript found the following errors:
/Users/dao/dev/heroes/tmp/broccoli_type_script_compiler-input_base_path-38jMyUeg.tmp/0/src/app/app.routing.ts (1, 38): Cannot find module '@angular/router'.
at BroccoliTypeScriptCompiler._doIncrementalBuild (/Users/dao/dev/heroes/node_modules/angular-cli/lib/broccoli/broccoli-typescript.js:120:19)
at BroccoliTypeScriptCompiler.build (/Users/dao/dev/heroes/node_modules/angular-cli/lib/broccoli/broccoli-typescript.js:43:10)
at /Users/dao/dev/heroes/node_modules/angular-cli/node_modules/broccoli-caching-writer/index.js:152:21
at lib$rsvp$$internal$$tryCatch (/Users/dao/dev/heroes/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1036:16)
at lib$rsvp$$internal$$invokeCallback (/Users/dao/dev/heroes/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1048:17)
at lib$rsvp$$internal$$publish (/Users/dao/dev/heroes/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1019:11)
at lib$rsvp$asap$$flush (/Users/dao/dev/heroes/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1198:9)
at nextTickCallbackWith0Args (node.js:420:9)
at process._tickCallback (node.js:349:13)

I met the same issue.

I solved it by moving to WebPack.
It is working but you'll get some warnings in your IDE though.
I'm encountering other problem there though:
http://stackoverflow.com/questions/38896349/angular-2-rc5-angular-router-3-0-0-rc-1-invalid-configuration-or-a-bug
But anyway your issue can be solved by moving to WebPack version.

Thx, I'll try it.

Let me know if the webpack version didn't fix it. You can find instructions to install it here: https://github.com/angular/angular-cli#webpack-preview-release-update


Fixes #1191 by implementing the RSS feed of PlantUML releases

As far as my code is concerned, there is no more violation

As usual, I 've let Travis find violations for me. Does it make me a bad person ?
You are right: by doing that, you keep feeding The Travis, it calms the beast :smiling_imp:


AMBARI-24950 - Logsearch: use os timezone in Logfeeder

What changes were proposed in this pull request?
cherry picked a part of https://github.com/apache/ambari-logsearch/pull/41
Logfeeder sets the default timezone to GMT at start up. Most service logs' logtime are in os timezone. If os timezone is set to other than UTC Logfeeder does not convert the logtime values to UTC when sending the log entry to Solr and Logsearch portal shows invalid log time values.
Fix:
Remove explicitly set the default time zone to GMT. This case the default time zone going to be the os time zone
Or it can be overridden by JVM option.
How was this patch tested?
Manually:

Deploy Ambari and a cluster: Zookeeper, Infra-Solr, Logsearch
Set the time zone en each node to America/Chicago:
timedatectl set-timezone America/Chicago
Restart Amabari server/agents and all services
Check Ambari server log and notice that the log time values are at the new time zone in the end of the logfile while the values in the beginning are in UTC

grep 'State of service component' /var/log/ambari-server/ambari-server.log
2018-11-23 15:04:53,058  INFO [agent-report-processor-0] HeartbeatProcessor:647 - State of service component LOGSEARCH_SERVER of service LOGSEARCH of cluster 2 has changed from INSTALLED to STARTED at host c7401.ambari.apache.org according to STATUS_COMMAND report
2018-11-23 15:05:19,535  INFO [agent-report-processor-0] HeartbeatProcessor:647 - State of service component LOGSEARCH_LOGFEEDER of service LOGSEARCH of cluster 2 has changed from INSTALLED to STARTED at host c7401.ambari.apache.org according to STATUS_COMMAND report
2018-11-23 09:18:35,669  INFO [agent-report-processor-0] HeartbeatProcessor:647 - State of service component LOGSEARCH_SERVER of service LOGSEARCH of cluster 2 has changed from STARTED to INSTALLED at host c7401.ambari.apache.org according to STATUS_COMMAND report
2018-11-23 09:18:42,652  INFO [agent-report-processor-1] HeartbeatProcessor:647 - State of service component LOGSEARCH_LOGFEEDER of service LOGSEARCH of cluster 2 has changed from STARTED to INSTALLED at host c7402.ambari.apache.org according to STATUS_COMMAND report

Check that all ambari server log entries logtime is converted to UTC in Solr

{
        "log_message":"State of service component LOGSEARCH_SERVER of service LOGSEARCH of cluster 2 has changed from INSTALLED to STARTED at host c7401.ambari.apache.org according to STATUS_COMMAND report",
        "id":"606519d6-a315-4553-8f49-dda2eba7437f",
        "logtime":"2018-11-23T15:04:53.058Z"},
      {
        "log_message":"State of service component LOGSEARCH_LOGFEEDER of service LOGSEARCH of cluster 2 has changed from INSTALLED to STARTED at host c7401.ambari.apache.org according to STATUS_COMMAND report",
        "id":"97760f62-ec8b-447f-92e4-b49f616ed897",
        "logtime":"2018-11-23T15:05:19.535Z"},
      {
        "log_message":"State of service component LOGSEARCH_SERVER of service LOGSEARCH of cluster 2 has changed from STARTED to INSTALLED at host c7401.ambari.apache.org according to STATUS_COMMAND report",
        "id":"19aefe4f-ab22-43b5-a952-4bd355a0f6b3",
        "logtime":"2018-11-23T15:18:35.669Z"},
      {
        "log_message":"State of service component LOGSEARCH_LOGFEEDER of service LOGSEARCH of cluster 2 has changed from STARTED to INSTALLED at host c7402.ambari.apache.org according to STATUS_COMMAND report",
        "id":"3601f700-7272-4c38-bd7d-5d89552bdcc2",
        "logtime":"2018-11-23T15:18:42.652Z"},

Check that the logtime values are converted to the specified timezone on Logsearch portal

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5205/
Test FAILed.
Test FAILured.

Retest this please!

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5207/
Test PASSed.


how to run example shaders?

Is there a list with example/gallery shaders that one can go through, inside the program, or must they be copy/pasted into the main shader file one at a time?

Is there a list with example/gallery shaders that one can go through, inside the program, or must they be copy/pasted into the main shader file one at a time?

You can  drag and drop the example ones directly onto the window ( if they're valid), or you can just save things as the myshader_2d.wgsl (not 2m obviously when you're on a 3D shape)


Pytest config

[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I have searched the documentation and believe that my question is not covered.

Question

Hello, can I write settings for pytest (for example) in pyproject.toml? I've tried the next one to put in pyproject.toml, but it does not work:
[tool.pytest]
addopts = "--cov=JWTAuth -v -rxXs"

The Poetry project does not provide technical support for usage of pyproject.toml, which is a Python standard and not Poetry-specific.
Your question should be directed to the pytest maintainers: https://github.com/pytest-dev/pytest/issues/1556


[BUG] Repo name still wrong

Expected behaviour
My eyes don't bleed
Current behaviour
They do

LGTM, it's not r****

Fixed, repo still needs to be updated in code tho


Fix UnusedBlockArgument's IgnoreEmptyBlocks parameter from being ignored

In 8f01fbe1 a very old patch was applied to a new tree.  Because of this, the
changes made in that commit to the default config actually added the same key
for the Lint/UnusedBlockArgument cop a second time.  Because of how YAML
parses, the previous definition, which defined the IgnoreEmptyBlocks
parameter was lost and because of that, any configuration that defined it
would now throw a warning.
Also brought up in #3095

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it)
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

👍


When name/ password is empty, login button appears as loading

(this is my bug, just assigning to you in case you want to pick it up)

Deployed


unread notif badge never go over 100

this is a feature atm; we've had to optimise the matrix.org synapse to cap the badge at 100 to avoid perf problems.


Parameter Configuration shows too much info

I believe that parameter configuration is a bit confusing.

What is the meaning of be able to select "is top concept in scheme" or "is in scheme" ?
Also Info bubble states that "if a dimension contains empty values it should not be selected for further analysis". How could I know that the dimension contains empty values and I should not select it?

Attributes have been reduced to label and key.
You might want to check the preview of the data you send to avoid sending null values.

You might want to check the preview of the data you send to avoid sending null values.

Can't this be added as a validation? It would help a lot.


[CODE] Issue#3 Sort output

The output list can be sorted asc/desc by setting the params to history() func.
The combined list of the entries can now be sorted either in ascending or descending order by specifying the choice as parameters to the history() function.
Fixes #3
Type of change
[CODE]
Please delete options that are not relevant.

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

Checklist:

[ ] I have read the contribution guidelines and followed it as far as possible.
[ ] I have performed a self-review of my own code (if applicable)
[ ] I have commented my code, particularly in hard-to-understand areas (if applicable)
[ ] I have made corresponding changes to the documentation (if applicable)
[ ] My changes generate no new warnings
[ ] Any dependent and pending changes have been merged and published

LGTM!
(BTW you can check the points in the checklist by changing the [ ] to [x])


refactor: move from io/ioutil to io and os packages

The io/ioutil package has been deprecated in Go 1.16 (See https://golang.org/doc/go1.16#ioutil). Since this project has been upgraded to Go 1.17 in #7029, this PR replaces the existing io/ioutil functions with their new definitions in io and os packages.
Don't bother creating a PR until you've done this:

[x] Run make pre-commit -B to fix codegen, lint, and commit message problems.

Create your PR as a draft.

Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it
does not need to pass.
Once required tests have passed, you can make it "Ready for review".
Say how how you tested your changes. If you changed the UI, attach screenshots.

Tips:

If changes were requested, and you've made them, then dismiss the review to get it looked at again.
Add you organization to USERS.md if you like.
You can ask for help!

I'm sorry, I don't think we should accept this PR. The reason is that we often have to back port fixes, and changes to package imports will create a lot of work for anyone doing the back-porting. I don't think that cost outweighs the benefits.


Merge scaling branch to dev and create a release

cc @gzuber
Tracking item to move release scaling changes

Closing as completed.


Performance Counters do not Increment by value. Value is always 0

Runtime version : ASP.NET 4.8

Describe the bug
I have the following code:
namespace Utilities
{
    public class AppInsightsPerfCounters
    {
        private static PerformanceCounter avgCounter64Sample;
        private static PerformanceCounter avgCounter64SampleBase;

        static AppInsightsPerfCounters()
        {
            configure();
        }
        private static bool SetupCategory()
        {
            if (!PerformanceCounterCategory.Exists("AverageCounter64SampleCategory"))
            {

                CounterCreationDataCollection counterDataCollection = new CounterCreationDataCollection();

                // Add the counter.
                CounterCreationData averageCount64 = new CounterCreationData();
                averageCount64.CounterType = PerformanceCounterType.AverageCount64;
                averageCount64.CounterName = "AverageCounter64Sample";
                counterDataCollection.Add(averageCount64);

                // Add the base counter.
                CounterCreationData averageCount64Base = new CounterCreationData();
                averageCount64Base.CounterType = PerformanceCounterType.AverageBase;
                averageCount64Base.CounterName = "AverageCounter64SampleBase";
                counterDataCollection.Add(averageCount64Base);

                // Create the category.
                PerformanceCounterCategory.Create("AverageCounter64SampleCategory",
                    "Demonstrates usage of the AverageCounter64 performance counter type.",
                    PerformanceCounterCategoryType.SingleInstance, counterDataCollection);

                return (true);
            }
            else
            {
                Console.WriteLine("Category exists - AverageCounter64SampleCategory");
                return (false);
            }
        }

        private static void CreateCounters()
        {
            // Create the counters.

            avgCounter64Sample = new PerformanceCounter("AverageCounter64SampleCategory",
                "AverageCounter64Sample",
                false);

           avgCounter64SampleBase = new PerformanceCounter("AverageCounter64SampleCategory",
                "AverageCounter64SampleBase",
                false);

            avgCounter64Sample.RawValue = 0;
            avgCounter64SampleBase.RawValue = 0;
        }

        private static void CollectSamples(System.Collections.ArrayList samplesList)
        {

            Random r = new Random(DateTime.Now.Millisecond);

            // Loop for the samples.
            for (int j = 0; j < 20; j++)
            {

                int value = r.Next(1, 10);
                Console.Write(j + " = " + value);

                avgCounter64Sample.IncrementBy(value);

                avgCounter64SampleBase.Increment();

                if ((j % 10) == 9)
                {
                    OutputSample(avgCounter64Sample.NextSample());
                    samplesList.Add(avgCounter64Sample.NextSample());
                }
                else
                {
                    Console.WriteLine();
                }

                System.Threading.Thread.Sleep(50);
            }
        }

        public static void CollectButtonClick()
        {
            Random r = new Random(DateTime.Now.Millisecond);
            int value = r.Next(1, 100);
            value = 5;
            System.Threading.Thread.Sleep(value);
            avgCounter64Sample.IncrementBy(value);
        }

        private static void CalculateResults(System.Collections.ArrayList samplesList)
        {
            for (int i = 0; i < (samplesList.Count - 1); i++)
            {
                // Output the sample.
                OutputSample((CounterSample)samplesList[i]);
                OutputSample((CounterSample)samplesList[i + 1]);

                // Use .NET to calculate the counter value.
                System.Diagnostics.Debug.WriteLine(".NET computed counter value = " +
                    CounterSampleCalculator.ComputeCounterValue((CounterSample)samplesList[i],
                    (CounterSample)samplesList[i + 1]));

                // Calculate the counter value manually.
                System.Diagnostics.Debug.WriteLine("My computed counter value = " +
                    MyComputeCounterValue((CounterSample)samplesList[i],
                    (CounterSample)samplesList[i + 1]));
            }
        }

        //++++++++//++++++++//++++++++//++++++++//++++++++//++++++++//++++++++//++++++++
        //    Description - This counter type shows how many items are processed, on average,
        //        during an operation. Counters of this type display a ratio of the items
        //        processed (such as bytes sent) to the number of operations completed. The
        //        ratio is calculated by comparing the number of items processed during the
        //        last interval to the number of operations completed during the last interval.
        // Generic type - Average
        //      Formula - (N1 - N0) / (D1 - D0), where the numerator (N) represents the number
        //        of items processed during the last sample interval and the denominator (D)
        //        represents the number of operations completed during the last two sample
        //        intervals.
        //    Average (Nx - N0) / (Dx - D0)
        //    Example PhysicalDisk\ Avg. Disk Bytes/Transfer
        //++++++++//++++++++//++++++++//++++++++//++++++++//++++++++//++++++++//++++++++
        private static Single MyComputeCounterValue(CounterSample s0, CounterSample s1)
        {
            Single numerator = (Single)s1.RawValue - (Single)s0.RawValue;
            Single denomenator = (Single)s1.BaseValue - (Single)s0.BaseValue;
            Single counterValue = numerator / denomenator;
            return (counterValue);
        }

        // Output information about the counter sample.
        private static void OutputSample(CounterSample s)
        {
            System.Diagnostics.Debug.WriteLine("\r\n+++++++++++");
            System.Diagnostics.Debug.WriteLine("Sample values - \r\n");
            System.Diagnostics.Debug.WriteLine("   BaseValue        = " + s.BaseValue);
            System.Diagnostics.Debug.WriteLine("   CounterFrequency = " + s.CounterFrequency);
            System.Diagnostics.Debug.WriteLine("   CounterTimeStamp = " + s.CounterTimeStamp);
            System.Diagnostics.Debug.WriteLine("   CounterType      = " + s.CounterType);
            System.Diagnostics.Debug.WriteLine("   RawValue         = " + s.RawValue);
            System.Diagnostics.Debug.WriteLine("   SystemFrequency  = " + s.SystemFrequency);
            System.Diagnostics.Debug.WriteLine("   TimeStamp        = " + s.TimeStamp);
            System.Diagnostics.Debug.WriteLine("   TimeStamp100nSec = " + s.TimeStamp100nSec);
            System.Diagnostics.Debug. WriteLine("++++++++++++++++++++++");
        }

        public static void configure()
        {
            SetupCategory();
            CreateCounters();
        }

        public static void measure()
        {
            var samplesList = new System.Collections.ArrayList();
            CollectSamples(samplesList);
            CalculateResults(samplesList);
        }

    }
}

###Image description
The counters are reported but no value appears. I am incrementing the counter like this:
public static void CollectButtonClick()
        {
            Random r = new Random(DateTime.Now.Millisecond);
            int value = r.Next(1, 100);
            value = 5;
            System.Threading.Thread.Sleep(value);
            avgCounter64Sample.IncrementBy(value);
        }

However the values in Application Insights appear all as 0.
query_data (1).csv

To Reproduce
To reproduce the issue investigate the following repo:
Repo
It contains these two Projects:
*Utilities, where are the PerfCounter functions
*SampleWebAppChanged, where these functions are called
This is the link to the file from the Utilities nampespace: https://dev.azure.com/linkestagiodiogomoura/_git/SampleWebAppTransfer?path=%2FUtilities%2FAppInsightsPerfCounters.cs.
This is the link to the place where the function is called (Contact page) SampleWebAppChanged namespace:
Contact.aspx.cs
Contact.aspx
To reproduce the issue click the "TrackSinglePerfCollect" button at the bottom of the Contact page of SampleWebAppChanged, without using IIS Express (Use IIS because counters arent reported with IIS Express to Application Insights).

This is related to the pull model of performance counter collector. So every minute the counter measures are collected. If the measurement is 0 at the time it will be reported as it is. So the counters increments are not reported.


Using ZenHub

Hola! @saadq has created a ZenHub account for the saadq organization. ZenHub is the only project management tool integrated natively in GitHub – created specifically for fast-moving, software-driven teams.

How do I use ZenHub?
To get set up with ZenHub, all you have to do is download the browser extension and log in with your GitHub account. Once you do, you’ll get access to ZenHub’s complete feature-set immediately.
What can ZenHub do?
ZenHub adds a series of enhancements directly inside the GitHub UI:

Real-time, customizable task boards for GitHub issues;
Multi-Repository burndown charts, estimates, and velocity tracking based on GitHub Milestones;
Personal to-do lists and task prioritization;
Time-saving shortcuts – like a quick repo switcher, a “Move issue” button, and much more.

Add ZenHub to GitHub
Still curious? See more ZenHub features or read user reviews. This issue was written by your friendly ZenHub bot, posted by request from @saadq.

If anyone is interested in contributing to latexresu.me, I recommend installing the ZenHub extension, as that is what I'll be using to manage the project. I haven't been that active on this project for awhile, but I plan to start fixing the remaining issues and adding features quite soon.

On second thought, I'm just going to go with using GitHub Projects as it's an already built-in feature. Anyone who wants to see a high level overview of what is being worked on can go check that out. I've been busy with summer classes for awhile, but as they start winding down I'll be able to put more time into continuing the development of this project.


Add modify functions

I wanted to modify my AST and saw that there were no helper utility functions in this library.  I wrote a couple basic ones.

glsl.mod.find(node)
glsl.mod.remove(node)
glsl.mod.replace(node, newNode)
glsl.mod.add(node, newNode, after)
glsl.mod.addBefore(node, newNode)
glsl.mod.addAfter(node, newNode)

Fantastic! :)
Would you be up for writing tests for the mod functions too?

I wrote some basic ones to ensure that they work.  They are not comprehensive.
Side note: might be worth breaking your tests up in to a couple different files d:

Yes, breaking them up into manageable bits is long overdue :p

Thank you!
v1.1.0 in npm now.


Unable to use defined JSON subschemas

Issue Type: Bug
Steps to Reproduce:

Create a "definitions" member for a JSON schema, and put a subschema there.
Attempt to refer to this subschema in settings.json.

I'm in a folder called "json" with a subfolder called "schemas" with the schemas, and a subfolder called ".vscode" for the settings.json.
Here is the definitions schema I am using:
{
  "$schema": "https://json-schema.org/draft/2019-09/schema",
  "type": "object",
  "definitions": {
    "items": {
      "type": "array",
      "items": {
        "$ref": "items.json#"
      },
      "minItems": 1
    }
  }
}

If I just make items.json into an array and use "url": "./schemas/items.json" works fine, but that isn't very conducive to my current work case.

So I tried using this:
"schema": {
	"type": "array",
	"items": {
	  "$ref": "./schemas/items.json"
	}
  }

Got this error: "Problems loading reference 'vscode://schemas/custom/schemas/items.json': Request vscode/content failed with message: cannot open vscode://schemas/custom/schemas/items.json. Detail: resource is not available".

Tried using this:
"url": "file:///C:/Users/BLATANT-DOXX-RISK/Source/Repos/json/schemas/types.json#/definitions/items"
And it just used types.json, ignored the definitions part, and looked for nothing but a blank object.

Finally, I tried using this: "url": "./schemas/types.json#/definitions/items".
Got this error: "File not found (c:\Users\BLATANT-DOXX-RISK\Source\Repos\json\schemas\types.json#\definitions\items))".

@aeschli came up a lot when looking through Github issues.

VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:35:15.005Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.80GB (7.40GB free)

Process Argv
C:\Users\ZACH-GAMING\Source\Repos\Cataclysm-DDA-Testing\data\json

Screen Reader
no

VM
0%

Extensions: none

We don't support refering to subschemas from settings.json


Update yarn build command to be yarn package

This is in correlation with https://github.com/apache/daffodil-vscode/pull/232

@stevedlawrence Actually this will still need the yarn install for it to be able to compile the code and run the webpack command I will add it back in

@stevedlawrence Actually this will still need the yarn install for it to be able to compile the code and run the webpack command I will add it back in

The current main branch of the vscode repo works fine for me:
git clean -xdf && yarn package

I don't get any webpack errors, and it looks like the vsix file was built successfully.

You are right I actually run in prepackage so it can be removed will do that one oversight on my part


Issue with banned player

Reproduction:

Ban a player
Player try join after banned
Un-ban it.
Player try join and got You a already connected to this proxy

Then the player can't join anymore. No issue with origin BungeeCord.

plugins, Waterfall version?

Probably related to https://github.com/WaterfallMC/Waterfall/issues/69

@kamcio96 Latest Waterfall. Would happen with no plugin. Minecraft server version spigot-1.8.8

It's possible its an issue with https://github.com/WaterfallMC/Waterfall/blob/master/BungeeCord-Patches/0027-Improve-connection-closing-fixing-the-kick-delay.patch

@jamierocks I revert UserConnection.java. No effect.

It's Event4j's issue. I hava a plugin that query which target player disconnect from but while it banned null returned and an NPE thrown. I add debug info into BungeeCord.removeConnection and see it called in BungeeCord but no called in Waterfall.
Seem like Event4J not catch all exception and break logic. @jamierocks @Techcable

Interesting @Techcable are you able to fix anytime soon?

I've removed the Event4J patch temporarily - https://github.com/WaterfallMC/Waterfall/commit/80bed7979d707563283635998cc326f6aa0ad5b0

This issue as it is is resolved, Event4J will be reapplied when that is fixed. Resolved.


Standardize sendgrid endpoint

Pull request for the topic https://github.com/slingr-stack/platform/issues/13419 created by @agreggio-slingr
What does it do?
This ticket implements the use of slingr-helpgen, by which sendgrid endpoint was created.
How to test it?
Make calls from the platform for example using the console using the information provided in the documentation.
Implementation notes
Create new version of the endpoint, it is not compatible with previous versions.
Implementation notes
Create a new tags changing the major version and deploy to the platform.

#approve

#approve


Puppet Agent

This adds in Puppet Agent as the default (rather than the deprecated individual packages).

Thanks @grahamgilbert!
Does the OS_VERSION Puppet-Agent recipe Input matter (i.e. is it really important to install the right agent pkg version)? The recipe's default is still 10.10, which can be changed, but I wonder if it's better to just set this automatically.
It would also be good to have this explained in the README docs that puppet_version refers to the 3.x line.

We deploy the 10.10 version here on 10.11 and suffer no il-effects. I'll sort out the README

Thanks!


clickhouse: Nil pointer dereference in metrics exporter

After upgrading to ClickHouse Operator v0.21.2, the following error occurs at the start of the installation.
Though it seems to self-heal after few restarts.
Error Log:
I0707 05:05:52.512308       1 rest_server.go:37] Starting metrics exporter at ':8888/metrics'
I0707 05:05:52.529129       1 exporter.go:294] Add explicitly found CHI default/signoz-6hjc6q1prm-clickhouse
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x198 pc=0x1370d8e]

goroutine 1 [running]:
github.com/altinity/clickhouse-operator/pkg/apis/metrics.(*Exporter).DiscoveryWatchedCHIs(0x16c6661?, {0x18f6338, 0xc00022dce0}, 0xc0004d60c0)
	/clickhouse-operator/pkg/apis/metrics/exporter.go:295 +0x3ee
github.com/altinity/clickhouse-operator/cmd/metrics_exporter/app.Run()
	/clickhouse-operator/cmd/metrics_exporter/app/metrics_exporter.go:108 +0x36b
main.main()
	/clickhouse-operator/cmd/metrics_exporter/main.go:23 +0x17

Ref: https://github.com/Altinity/clickhouse-operator/issues/1187

@spenczar This has been fixed in the main branch of the clickhouse-operator, and will be part of >=v0.21.3 release.
However, we wouldn't be bumping up operator version anytime soon, since it would involve migration steps for CRDs once again.


Fix content_id mismatches between draft and live content stores

Yet another task we need!

For people like myself who don't know why there are mismatches, could you summarise how they got there?

@jamiecobbett We updated the PR description above. We can clarify further if needed. Let us know.


[FLINK-28929][table] Add built-in datediff function.

What is the purpose of the change
Add built-in datediff function
Brief change log

Syntax:

DATEDIFF(expr1,expr2)

Returns:

returns expr1 − expr2 expressed as a value in days from one date to the other. expr1 and expr2 are date or date-and-time expressions. Only the date parts of the values are used in the calculation.
This function returns NULL if expr1 or expr2 is NULL.

See more:

mysql: https://dev.mysql.com/doc/refman/8.0/en/date-and-time-functions.html#function_datediff

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: ( no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

hi @twalthr  do you have time to review it? thanks very much!

hi @snuyanzin could you help review it?


test: Remove unnecessary excludes

CP is empty on 1.3 and makes tests fail on 1.2, so I drop backporting for this one


规约有Markdown 版本的吗？

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

阿里巴巴内部有md版本，对外则使用PDF输出。你可以使用工具转化一下。


Show number of added, changed, and updated outages in commit title

Similar to simonw's git scraper article for PGE outages.

fixed in a32231c


:3:1-19: error: unsafe variables in:   head_pred(Stop,1):-[#inc_bias].

Hi,
Thanks for this great work!
I'm trying to use Popper in my custom domain.
Specifically, I'm trying to induce rules about the actions of some agents and their relationships.
I start by providing the bias.pl, bk.pl, and exs.pl files.
The bias.pl looks like:
%% domain ontology
head_pred(Stop,1).
body_pred(IsPedestrian,1).
body_pred(IsCar,1).
body_pred(IsAmbulance,1).
body_pred(IsBus,1).
body_pred(IsPolice,1).
body_pred(IsTiro,1).
body_pred(IsReckless,1).
body_pred(IsOld,1).
body_pred(IsYoung,1).
body_pred(IsAtInter,1).
body_pred(IsInInter,1).
body_pred(IsClose,2).
body_pred(HigherPri,2).
body_pred(CollidingClose,2).
body_pred(LeftOf,2).
body_pred(RightOf,2).
body_pred(NextTo,2).
body_pred(See,2).

 Thanks!

head_pred(Stop,1).
body_pred(IsPedestrian,1).
body_pred(IsCar,1).
body_pred(IsAmbulance,1).
The arguments in the head_pred and head_body atoms must be ground.
Specifically, your predicate symbol names need to be lowercase, e.g.
head_pred(stop,2).
On Wed, 20 Mar 2024, 19:50 Bowen Li, @.***> wrote:

Hi,
Thanks for this great work!
I'm trying to use Popper in my custom domain.
Specifically, I'm trying to induce rules about the actions of some agents
and their relationships.
I start by providing the bias.pl, bk.pl, and exs.pl files.
The bias.pl looks like:
%% domain ontology
head_pred(Stop,1).
body_pred(IsPedestrian,1).
body_pred(IsCar,1).
body_pred(IsAmbulance,1).
body_pred(IsBus,1).
body_pred(IsPolice,1).
body_pred(IsTiro,1).
body_pred(IsReckless,1).
body_pred(IsOld,1).
body_pred(IsYoung,1).
body_pred(IsAtInter,1).
body_pred(IsInInter,1).
body_pred(IsClose,2).
body_pred(HigherPri,2).
body_pred(CollidingClose,2).
body_pred(LeftOf,2).
body_pred(RightOf,2).
body_pred(NextTo,2).
body_pred(See,2).

Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/logic-and-learning-lab/Popper/issues/84, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC2RSXRADCAAJM65W32CILYZHR6HAVCNFSM6AAAAABFAEJWC6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4TQNBSHE4TMMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks for the swift reply!
Lowercase worked!
May I also ask how Popper handles disjunction?
Say, my rule is like:
stop(V0):-(iscar(V0)^colliding(V1))v(iscar(V0)^isatinter(V0)^isininter(V1))

I tried the default parameter, but can't find a solution. Not sure if this is because I didn't enable disjunction.

Popper will learn two rules, which form a disjunction:
stop(V0):-iscar(V0),colliding(V1).
stop(V0):-iscar(V0),isatinter(V0),isininter(V1).

I see, thanks for the clarification.
I find the outputs are:
16:46:21 Generating programs of size: 2
16:46:21 Generating programs of size: 3
16:46:21 Generating programs of size: 4
16:46:23 Generating programs of size: 5
NO SOLUTION
Num. programs: 55806
Test:
        Called: 55806 times      Total: 16.28    Mean: 0.0003    Max: 0.053      Percentage: 58%
Constrain:
        Called: 55806 times      Total: 5.45     Mean: 0.0001    Max: 0.525      Percentage: 19%
Generate:
        Called: 55807 times      Total: 4.11     Mean: 0.0001    Max: 0.026      Percentage: 14%
Load Data:
        Called: 1 times          Total: 1.77     Mean: 1.7666    Max: 1.767      Percentage: 6%
Init:
        Called: 2 times          Total: 0.01     Mean: 0.0069    Max: 0.014      Percentage: 0%                                                                                16:52 20-Mar-24
Total operation time: 27.62s
Total execution time: 28.60s

I have changed the default arguments MAX_VARS=3, MAX_BODY=4, in this case, what does the "program size" mean? Does it mean the number of rules that form a disjunction?


Added -f option (for single file with file name, instead of directory).

This adds -f or --file for single file import and supports file instead of directory uploads.
Note: I tested in firefox, and the dialog is not coming up automatically on enter.  Manually clicking browse (by shrinking the iframe temporarily) works as expected.  We will need to figure out how to handle this.
@rginda PTAL

LGTM after naming nitpick.


Version 2.14.1

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

Comparison is base (b5b61d7) 89.91% compared to head (1755326) 89.91%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #836   +/-   ##
=======================================
  Coverage   89.91%   89.91%
=======================================
  Files          82       82
  Lines       14767    14767
=======================================
  Hits        13278    13278
  Misses       1489     1489

Flag
Coverage Δ

unittests
89.91% <ø> (ø)

unittests-rust
89.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Fix workflow triggers to fire on any source change

Fixes #39

Merging this even though it's naughty to not have approval in order to unblock others.


Page transition delay

While navigating from one page to other, sometimes there is delay (sometimes even more than 2 seconds!)
I understand pages are server rendered but this is local host and I only have two pages. One is root and doesn't take any params. Other page has only one param. Root page has only 1 component (container) and is heavily lightweight. When I press on header logo which is link to /, sometimes delay goes up to 3 seconds which is unacceptable, and user can't even see that any loading is being done. I am already using prefetch attribute.
Is there a way to detect that a page is about to be loaded and show some GUI loader for that case? Are there some other optimizations that I have to take care of? Does this appear in production build?

You're probably seeing a delay due to Next.js’s on-demand compilation in development.
I've used NProgress to show a progress loader based on listening to Router events. See: https://github.com/CityOfBoston/registry-certs/blob/develop/client/RouterListener.js
Note that currently the Next Router can only have a single listener per event, so make sure that you only set one set of listeners in your code.

@finneganh Thank you very much.
So this is as expected, due to development mode. These are very good news. I kinda have issues finding explanations to some cases in Next.js due to lack of docs. (I don't say that current docs are bad, they aren't just detailed enough to explain all cases).
I didn't know about the listener and never used any middleware as suggested, can you please tell me more about it?
Meantime, this issue is now resolved and therefore may be closed.

I've taken the pattern of wrapping all my Page components in a common higher-order component to do consistent client-side initialization, regardless of which page is loaded by the browser first.
You can see here: https://github.com/CityOfBoston/registry-certs/blob/develop/client/page.js
where I attach the RouterListener based on whether browserInited is set.
That page HOC wraps my components like this: https://github.com/CityOfBoston/registry-certs/blob/develop/pages/death/index.js


How to switch keyboard layout on the welcome screen?

How to switch keyboard layout on the welcome screen?

Specify layout on client side.

How can I enter the password in English if at the time of connection the keyboard was in the Russian layout?

+1, I have same problem

Enter your password not in welcome screen but client will help you.
For example,

We have no plans to implement such feature.
Specify the layout at client side is the only option for the meantime.


Fixing error with Resend integration

Fixed a syntax error on API for Astro v5 changes.

Created new PR #88 that fixed all the errors and cleans up issues.


MRG, DOC: Document anonymization better

Closes #7575
@drammock feel free to review and merge if you're happy.

thx @larsoner


ModuleNotFoundError: No module named 'azure'

Hi,
it seemed that in planetary-computer 0.4.8 a change is causing import error. Issue does not exist in 0.4.7.
Thanks for looking into it.

Huh, sorry about that. I could have sworn azure.storage.blob was a required dependency.
I'll fix this and make a new release.

0.4.9 is out on PyPI and should fix this: https://pypi.org/project/planetary-computer/


Get the WitnessPubKeyHash from a Payment

Hi,
I was wondering something.
I am making P2WPKH payments with an application, and from another one I try to read the blocks and retrieve any payment made by the other application.
data := []byte(msg)
x := bytes.NewBuffer(data)
b, err := block.NewFromBuffer(x)
if err != nil {
	fmt.Println(err)
	return
}
fmt.Println("-- BLOCK")
for tx := range b.TransactionsData.Transactions {
	fmt.Println("---- TXN")
	for output := range tx.Outputs {
		fmt.Println("------ OUTPUT")

		if output.Script == nil || len(output.Script) == 0 {
			return
		}

		switch address.GetScriptType(output.Script) {
		case address.P2WpkhScript:
			fmt.Println("------ P2WPKH")

			p, err := payment.FromScript(output.Script, &network.Regtest, nil)
			if err != nil {
				fmt.Println(err)
				continue
			}

                        // same outputs
			fmt.Println(p.WitnessPubKeyHash())
			fmt.Println(p.WitnessScriptHash())

		default:
			fmt.Println("------ OTHER")
			continue
		}
	}
}

So my first question is why the output of the two lines:
			fmt.Println(p.WitnessPubKeyHash())
			fmt.Println(p.WitnessScriptHash())

is the same?
And my second question is that somehow the output is the same, but actually it's not what I am expecting.
I get a long address of 75 bytes, while the payments are made on addresses of 43 bytes. So I am expecting an address of 43 bytes.
Looking more carefully at the output, I noticed that the addresses are matching minus the "checksum":
Original address:      ert1q989xa8zz0a4fs7zrsmfsacwkvnup8vyd 27xnhv
p.WitnessPubKeyHash(): ert1q989xa8zz0a4fs7zrsmfsacwkvnup8vyd 989xa8zz0a4fs7zrsmfsacwkvnup8vydul8w9d

But a checksum of 38 chars for WitnessPubKeyHash not sure, that makes sense.
So isn't there too much data in the p.WitnessHash? It looks like it.
I have been able to recompute the address I was expecting with the right checksum, by using bech32.ConvertBits on p.WitnessHash then cutting the results to keep the 32 first bytes.
So that kind of comforts me into that idea that there are too much data in WitnessHash, or (most likely) just something out of my knowledge.
Sorry the lengthy message, thanks for your help

Sonny

Hi @SonnyAD
Thanks for the detailed report.
I was able to replicate what you're experiencing and after inspecting the payment pkg I found the bug. I'll open a PR soon to fix it.


Use root user in FileProxy.remove

Avoids encountering permissions issues.

Moved to https://github.com/ChrisTimperley/dockerblade


[FEATURE] try mergin test variations

Another attempt at faster builds: merging variations on the same function together in one test.

Fixed more or less


Skip transpilation

Summary
backend.run() doesn't support un-transpiled circuits, so it needs to set the skip_transpilation flag to avoid re-transpilation.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 38.391%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2591

Relevant Lines:
6749

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 38.391%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2591

Relevant Lines:
6749

💛  - Coveralls


Add support for JSDoc types

{type} isn't highlighted
example:
/** @type {string} */

@iddan @jwhitmarsh @Alaneor @saidmoya12 I've just pushed a commit - tag name addDocblock. If any one wants to try this and let me know if it does the trick. apm install language-babel@addDocblock


buffer local variable `conda-anaconda-home' not working

If I set conda-anaconda-home as a buffer local variable, conda-env-activate can not activate the correct conda env.
Steps to reproduce the problem
Configuration

In init file:
(use-package conda)

My python project is organized as:
.
├── .dir-locals.el
└── src
    └── main.py

In .dir-locals.el:
((python-mode . ((conda-anaconda-home . "/home/user/programs/miniconda3/"))))

Operations

open main.py
M-x conda-env-activate, choose environment base, enter
*Message* buffer shows:if: Error: executing command ""/home/user/.anaconda3/bin/conda" ..activate "bash" "/home/user/programs/miniconda3/"" produced error code 127

Specifications

conda-20200818.1614
emacs 27.1
conda 4.9.2
Arch Linux
x86_64 Linux 5.9.14-arch1-1

I did some debugging and found out what caused this error.
As the code shown below, in the line 190, the statement (with-current-buffer standard-output in function conda--get-path-prefix will override the buffer local variable conda-anaconda-home.
https://github.com/necaris/conda.el/blob/9f7eea16e9ad3eb34fe3d1cbd9d6162b8046c2f8/conda.el#L185-L197
If you change the code into this:
 (defun conda--get-path-prefix (env-dir)
   "Get a platform-specific path string to utilize the conda env in ENV-DIR.
 It's platform specific in that it uses the platform's native path separator."
   (message "%s" conda-anaconda-home)  ;;; <========= /home/user/programs/miniconda3/
   (s-trim
    (with-output-to-string
      (with-current-buffer standard-output
        (let* ((conda-executable-path
                (concat (file-name-as-directory conda-anaconda-home)
                        (file-name-as-directory conda-env-executables-dir)
                        "conda"))
               (tmp (message "%s" conda-anaconda-home))  ;;; <========= /home/user/.anaconda3/

The first message prints: /home/user/programs/miniconda3/, and the second prints: /home/user/.anaconda3/

I did some debugging and found out what caused this error.
As the code shown below, in the line 190, the statement (with-current-buffer standard-output in function conda--get-path-prefix will override the buffer local variable conda-anaconda-home.
https://github.com/necaris/conda.el/blob/9f7eea16e9ad3eb34fe3d1cbd9d6162b8046c2f8/conda.el#L185-L197
If you change the code into this:
 (defun conda--get-path-prefix (env-dir)
   "Get a platform-specific path string to utilize the conda env in ENV-DIR.
 It's platform specific in that it uses the platform's native path separator."
   (message "%s" conda-anaconda-home)  ;;; <========= /home/user/programs/miniconda3/
   (s-trim
    (with-output-to-string
      (with-current-buffer standard-output
        (let* ((conda-executable-path
                (concat (file-name-as-directory conda-anaconda-home)
                        (file-name-as-directory conda-env-executables-dir)
                        "conda"))
               (tmp (message "%s" conda-anaconda-home))  ;;; <========= /home/user/.anaconda3/

The first message prints: /home/user/programs/miniconda3/, and the second prints: /home/user/.anaconda3/

@AirManH thanks for the report and the debugging! It looks like capturing the current value of conda-anaconda-home outside the with-current-buffer block will fix this -- any interest in submitting a PR?

@AirManH thanks for the report and the debugging! It looks like capturing the current value of conda-anaconda-home outside the with-current-buffer block will fix this -- any interest in submitting a PR?


--dump-json missing information for .mp4s from twitch.tv

Seems like running the --dump-json command doesn't return all information for twitch.tv clips
running
youtube-dl https://clips-media-assets.twitch.tv/24708907632-offset-5632-854x480.mp4 --dump-json

outputs
{"upload_date": "20170306", "protocol": "https", "extractor": "generic", "direct": true, "fulltitle": "24708907632-offset-5632-854x480", "id": "24708907632-offset-5632-854x480", "playlist": null, "title": "24708907632-offset-5632-854x480", "_filename": "24708907632-offset-5632-854x480-24708907632-offset-5632-854x480.mp4", "playlist_index": null, "webpage_url_basename": "24708907632-offset-5632-854x480.mp4", "display_id": "24708907632-offset-5632-854x480", "format": "mp4 - unknown", "requested_subtitles": null, "format_id": "mp4", "url": "https://clips-media-assets.twitch.tv/24708907632-offset-5632-854x480.mp4", "extractor_key": "Generic", "vcodec": null, "http_headers": {"Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Accept-Language": "en-us,en;q=0.5", "Accept-Encoding": "gzip, deflate", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "User-Agent": "Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20150101 Firefox/47.0 (Chrome)"}, "ext": "mp4", "webpage_url": "https://clips-media-assets.twitch.tv/24708907632-offset-5632-854x480.mp4", "formats": [{"protocol": "https", "format": "mp4 - unknown", "url": "https://clips-media-assets.twitch.tv/24708907632-offset-5632-854x480.mp4", "vcodec": null, "http_headers": {"Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Accept-Language": "en-us,en;q=0.5", "Accept-Encoding": "gzip, deflate", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "User-Agent": "Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20150101 Firefox/47.0 (Chrome)"}, "ext": "mp4", "format_id": "mp4"}]}
it is missing the duration info. is this an error in the encoding of twitch's mp4s, or something on the youtube-dl side?
thanks!

https://clips-media-assets.twitch.tv/24708907632-offset-5632-854x480.mp4 is a direct link where is it supposed to get duration from?

what about something like https://clips.twitch.tv/RoundLitigiousMangetoutKippa ?

There is no duration available.

Ah, so a limitation of the twitch api? I think I had the assumption that youtube-dl would download create a stream of the video file extracted from the url and pass it into ffprobe or something to get info.
But it probably calls w/e 3rd party api to get information based on the url

There is no API. It parses a webpage.

Oh! Thanks for clearing that up


metanorma/metanorma-build-scripts#66 Drop ruby 2.4 support

As title.
Generated by Cimas.

Failure doesn't related to PR


Func wrapper resolving

var container = new Container(rules: Rules.Default
                .WithUnknownServiceResolvers((request) => new DelegateFactory(_ =>  null)));

//container.Register<A>(); //UNCOMMENT THIS LINE TO GET EXCEPTION
var fFunc = container.Resolve<Func<IFoo>>();
var f0 = fFunc();//can be resolved as null because of unknown service resolver
container.Register<IFoo, Foo1>(reuse: Reuse.Singleton, ifAlreadyRegistered: IfAlreadyRegistered.Replace, setup: Setup.With(asResolutionCall: true));
var f1 = container.Resolve<IFoo>();
var f2 = fFunc();

Assert.IsNull(f0);
Assert.IsNotNull(f2);//EXCEPTION
Assert.AreEqual(f1, f2);

internal interface IFoo { }
internal class Foo1 : IFoo { }
internal class A
{
public A(Func<IFoo> func) { }
}

Why f2 is null after uncomment the line: container.Register<A>()

I am not sure, the A is not resolved by your example. Seems unrelated. Probably a bug. Will check.

Hi @raV720,
The problem is the corner case with the cache, particularly the first added A registration triggered the caching of the first resolved IFoo.
Thanks for finding this issue, I was able to come with a more generic solution to control the cache for factories produced by UnresolvedServiceResolvers and dynamic registrations.

fixed.


Macro: undo/redo does not toggle snake mode

Steps to Reproduce

Go to Macromolecules mode
Add peptide chain on canvas
Click on 'Snake mode' button on left tool panel
Click on 'Undo' button on left tool panel

Actual behavior
Chain position changed on previous one(before snake layout), however snake mode still turned on and bonds drawn as snake ones.
Expected behavior
A clear and concise description of what you expected to happen.
Snake mode turned off and bonds drawn as straight lines.

Autotests added to the table.


Add last to OnTick - Add missing timeframes

There are 2 items in the pull request:

The Advisor returns last on the OnTick. This can be useful on CME futures for example
The MQL5 file does not recognize all the supported timeframes. Added them.

Ok, sounds like a good idea. Thanks for adding this!


Auto generate ibstat file for IB performance validation under topo-aware …

An enhancement for topo-aware IB performance validation #373
This PR will auto-generate a required ibstate file ib_traffic_topo_aware_ibstat.txt which is used as input to build a graph
Tested on the 4 A100VM Clsuter.

Having this auto-generation functionality, I think we also need to update the way we handle the --ibstat argument. It can be optional. If users specify one in config, we won't need to re-generate it; otherwise, we auto-generate it.

@RyoYang please resolve conversation if you have addressed the corresponding comments :)

@RyoYang please resolve conversation if you have addressed the corresponding comments :)

Updated, thanks for the reminder. Please have a check.

/azp run


Include dcos auth login as part of CLI install snippet

Currently the UI does not include the auth login. I think most users have to do this. Why not include in default snippet?

Lets include it! Closing this issue, creating one for UI team.


tailscale commands fail to connect to tailscaled

Example:
[/share/CACHEDEV1_DATA/.qpkg/Tailscale] # uname -a
Linux XXXX-ts451 4.14.24-qnap #1 SMP Mon Jul 26 01:49:01 CST 2021 x86_64 GNU/Linux
[/share/CACHEDEV1_DATA/.qpkg/Tailscale] # ./tailscale version
1.14.0
tailscale commit: 62a458f7f4474ac2e26676ed8f06d43e80e5a34a
go version: go1.17
[/share/CACHEDEV1_DATA/.qpkg/Tailscale] # ./tailscale ping
Failed to connect to tailscaled. (safesocket.Connect: dial unix /var/run/tailscale/tailscaled.sock: connect: no such file or directory)

Should the tailscale command be changed to reference var/run/tailscale/tailscaled.sock (relative pathname) instead of /var/run/tailscale/tailscaled.sock? I just added a symbolic link from /var/run/talescale to /share/CACHEDEV1_DATA/.qpkg/Tailscale/var/run/tailscale/, and now I can execute commands. But I assume after I reboot the QNAP I'll have to reinstall that link.

Because the root filesystem is overwritten on every firmware update, then the installation script does not use /var/run/tailscale/tailscaled.sock for the socket, but instead uses the local path.
Instead of creating a symlink, you can append -socket option to the tailscale command with the location of the UNIX socket. This is also documented in the readme.


/news/ can be incredibly slow and is inconsistent (by event type) how far it goes back – clean up

I removed /news/.


revisiting FPS

I can see that the FPS issue has gone through a fair bit of processing over the years (#32, #117,  #371, #418, #453, #656, #680 ...  )
If I understand the evolution correctly, there was even a time when there was 15fps as a low frame rate option, but now there is only 30 or 60...
Is there a technical reason that it is difficult to capture less frames? or was this a decision to try to simplify the UI?
I would like to repeat and clarify my (pre)proposal from #453:

More options for capturing less frames during the recording phase
(not millions, just a couple more... )
like 10,15,30,60 would be fine:

The possibility to export at up to 2x the frame rate
(and to set a preferred default in general preferences)
(but not by adding more frames - by setting the frame duration)

Visibility of the fps of the recording fps, the playback fps, and the multiplier in the export gui bottom bar.

Plugin vs core:
I can see that 2. can be easily handled by a plugin that controls ffmpeg settings,
and maybe if you offer space in the bottom bar to plugin generated jsx (as is being discussed) then 3. could also be covered by a plugin.
I assume that plugins could also have their own preferences tab, so the options could also be added in that way.
BUT, I don't see why. It just feels like a very basic core functionality of a screen capture program to offer more than 2 options for frame rate and to allow for playback frame rates at least 2x the capture rate.
Maybe I missed some of the old discussions, or maybe I am just stubborn and demanding ; ) but I'd really love to understand the current status, and I'd also love to get coherent with the project leaders (either to surrender my opinion because you already thought it through and convince me that these features are superfluous and/or belong in a plugin.... OR more hopefully coherent in the way that we can craft a feature proposal that I can offer a PR to solve)

Thank you for the detailed issue @gotjoshua!

We reasoned that we would always prefer to capture at the highest possible quality, which gives you more wiggle room in post. However, not all hardware is capable of capturing as many as 60 frames per second, so we defaulted to 30. After recording we let you set a custom FPS for the export, which is remembered, so if you always want to export at 15 FPS, you can set it once and leave it. It'd be great to hear your use case for capturing at a lower FPS.

Kind of relates to #453, being able to change the playback speed – which benefits from having a higher quality source as mentioned in 1. This is indeed something that a more advanced plugin system could allow for, maybe by passing ffmpeg options @karaggeorge?

These are features that we have talked about but, but decided not to implement in order to focus on the core experience. I'd recommend opening separate issues for each feature request so we can gauge the interest and discuss potential solutions for each.

Cool, thanks for the props @skllcrn !
So use case:
I want small file size gifs that only have 5-15 fps of data and are set to play faster than they were recorded (1.2x-2x)
This is for quick little examples of UI interaction.
Example:
I record 10 seconds of UI interactions at 10fps
I export a gif that will play all 100 frames in 7.5 seconds (at 1.5x) by setting a frame delay of 0.06s
I guess its also possible to record at 30 or 60 drop 60-90% of the frames until you have 10fps of data left and then set the playback to 15fps. I am not so familiar with ffmpeg options, but i guess all this is possible via post processing.
However, I think that it fits better to adapt the core ui (very slightly) to include the possibility to change playback fps in a way that results in faster or slower playback.

I understand, right now your best option is to capture a 60 FPS recording and use the Save Orginal feature in the File menu, then edit the playback speed in a third party tool. We'll consider supporting this in #453, and I'll be closing this issue on favour of that one. Thank you!


Ligh: Add hs null check to avoid crash

Avoid a crash on the hs light check if none is set.
Fixes #746

Checked in my home assistant installation with the L530 light bulbs and this null checks results that the lights are controllable again and do not show as unavailable.


Black images

When I run test_SRGAN.sh, everything was fine except the results I got were blacked-out images (I mean all the images; inputs, outputs, and targets).
Please check if the pre-trained model hasn't any issue.

img_003-targets

img_003-outputs

img_003-inputs

img_001-targets

img_001-outputs

img_001-inputs

It is the case with all images

Mine is fine. Maybe download the model and try again?

I think this is a problem with python 3.7, I have same problem when running python 3.7 on CUDA 10.1
I downdated to python 3.6 with CUDA 10.0 and it working :)


[T3A5][W11-A2] Zhi Jiang

Ready for Review

Accepting without review since T3A3 and T3A4 were reviewed in detail. You are advised to cross check those comments with your code for this activity too. We also discussed the right way to do this exercise in the tutorial.


The email address is not working and returning errors when you send an email<EMAIL_ADDRESS>

[Enter feedback here]

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

ID: 303075ec-ed9b-7ba4-15c5-242f93d53eb2
Version Independent ID: 15d3653d-af25-8ee7-2e94-01057a734de4
Content: Publish a Microsoft Edge extension - Microsoft Edge Developer documentation
Content Source: microsoft-edge/extensions-chromium/publish/publish-extension.md
Service: microsoft-edge
Sub-service: extensions
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

AB#49073986

Thank you for reporting this issue, I'll bring it to the right people to get it fixed as soon as we can.

Our team is currently working on fixing this issue. So it's a temporary bug that will get resolved soon. I'll update this issue again once I know this has been fixed.

The email should be working again. Closing this issue. Feel free to get in touch again if you see another issue.


ease precision tolerenace for iosys tests

The tests in the new iosys module comparing resulting time series are too tight for some build systems. Easing them a little bit allows for a clean build.

Coverage decreased (-2.7%) to 81.851% when pulling f4915eb44c45b17a9ac271e7acec58470243b5fa on bnavigator:ease-iosys-tests into a7f5f1ce47e7504848c8b0bccb3186a27fc57a64 on python-control:master.

Coverage decreased (-2.7%) to 81.851% when pulling f4915eb44c45b17a9ac271e7acec58470243b5fa on bnavigator:ease-iosys-tests into a7f5f1ce47e7504848c8b0bccb3186a27fc57a64 on python-control:master.