Adding a field to a shape that already has a field replaces the original field

The behavior should be to add a 2nd field to the shape, allowing for multiple fields per shape.

I agree. I think the problem is simply that I'm copying the vertices and
polygons into the new poly data but not the existing fields. Yes to
incrementally adding fields, no to adding multiple colors -- there can only
be one color.
--Alex
On Wed, Sep 30, 2015 at 12:06 PM, Greg Von Kuster<EMAIL_ADDRESS>wrote:

The behavior should be to add a 2nd field to the shape, allowing for
multiple fields per shape.
—
Reply to this email directly or view it on GitHub
https://github.com/pletzer/icqsol/issues/31.

--
Alexander Pletzer
+1<PHONE_NUMBER> cell
<EMAIL_ADDRESS>

I think the problem is that icqRefineSurface does not interpolate the existing field onto the new grid. The solution would be to use set_attributes in the triangulation call and load the field data when buildng the vtkPolyData object.

Point and cell data are now carried over when refining the mesh. This allows one to add as many fields as one wishes.

@pletzer I don't see this working, at least with our icqsol Galaxy tools.  I'm running the latest code from the icqsol package.  If I create this dataset using the icqsol_create_shape tool:
vtk DataFile Version 3.0
vtk output
ASCII
DATASET POLYDATA
POINTS 24 float
0 0 0 0 0 1 0 1 1
0 1 0 1 0 0 1 1 0
1 1 1 1 0 1 0 0 0
0 0 0 0 0 1 0 0 1
0 1 1 0 1 1 0 1 0
0 1 0 1 0 0 1 0 0
1 1 0 1 1 0 1 1 1
1 1 1 1 0 1 1 0 1
POLYGONS 6 30
4 0 1 2 3
4 4 5 6 7
4 8 16 22 10
4 14 12 20 18
4 9 15 19 17
4 11 23 21 13
POINT_DATA 24
NORMALS Normals float
-1 0 0 -1 0 0 -1 0 0
-1 0 0 1 0 0 1 0 0
1 0 0 1 0 0 0 -1 0
0 0 -1 0 -1 0 0 0 1
0 1 0 0 0 1 0 1 0
0 0 -1 0 -1 0 0 0 -1
0 1 0 0 0 -1 0 1 0
0 0 1 0 -1 0 0 0 1
I then add a surface field named "surface_field" to the shape using the icqsol_add_surface_field_from_expression tool, producing this dataset:
vtk DataFile Version 3.0
vtk output
ASCII
DATASET POLYDATA
POINTS 24 float
0 0 0 0 0 1 0 1 1
0 1 0 1 0 0 1 1 0
1 1 1 1 0 1 0 0 0
0 0 0 0 0 1 0 0 1
0 1 1 0 1 1 0 1 0
0 1 0 1 0 0 1 0 0
1 1 0 1 1 0 1 1 1
1 1 1 1 0 1 1 0 1
POLYGONS 12 48
3 3 0 1
3 1 2 3
3 7 4 5
3 5 6 7
3 10 8 16
3 16 22 10
3 18 14 12
3 12 20 18
3 17 9 15
3 15 19 17
3 13 11 23
3 23 21 13
CELL_DATA 12
FIELD FieldData 1
surface_field 1 12 double
0.0123457 0.197531 0.878371 1.06356 0.0123457 0.197531 0.0123457 0.197531 0.096225
0.3849 1.09623 1.3849
POINT_DATA 24
NORMALS Normals float
-1 0 0 -1 0 0 -1 0 0
-1 0 0 1 0 0 1 0 0
1 0 0 1 0 0 0 -1 0
0 0 -1 0 -1 0 0 0 1
0 1 0 0 0 1 0 1 0
0 0 -1 0 -1 0 0 0 -1
0 1 0 0 0 -1 0 1 0
0 0 1 0 -1 0 0 0 1
I then add a surface field named "surface_field1" to the above dataset using the icqsol_add_surface_field_from_expression tool, producing this dataset.  Notice that the dataset does not include 2 surface fields, but just the last one added.
vtk DataFile Version 3.0
vtk output
ASCII
DATASET POLYDATA
POINTS 24 float
0 1 0 0 0 0 0 0 1
0 1 1 1 0 1 1 0 0
1 1 0 1 1 1 0 1 0
0 1 0 0 0 0 0 0 0
0 0 1 0 0 1 0 1 1
0 1 1 1 0 1 1 0 1
1 0 0 1 0 0 1 1 0
1 1 0 1 1 1 1 1 1
POLYGONS 12 48
3 0 1 2
3 2 3 0
3 4 5 6
3 6 7 4
3 12 10 18
3 18 16 12
3 20 8 14
3 14 22 20
3 19 11 9
3 9 21 19
3 15 13 17
3 17 23 15
CELL_DATA 12
FIELD FieldData 1
surface_field1 1 12 double
0.0335582 0.53346 2.09226 2.37602 0.0335582 0.53346 0.0335582 0.53346 0.261163
1.02062 2.41788 2.67145
POINT_DATA 24
NORMALS Normals float
-1 0 0 -1 0 0 -1 0 0
-1 0 0 1 0 0 1 0 0
1 0 0 1 0 0 0 1 0
0 0 -1 0 -1 0 0 0 -1
0 -1 0 0 0 1 0 1 0
0 0 1 0 -1 0 0 0 1
0 -1 0 0 0 -1 0 1 0
0 0 -1 0 1 0 0 0 1

@gregvonkuster Test testAddSurfaceFieldFromExpressionBolt3 adds a third field to the input file. At least things seem to work on icqsol side.

@gregvonkuster I confirm that adding fields does not work in galaxy.

@gregvonkuster Now fixed in galaxy-csg. I added the cell fields and one point field and got:
vtk DataFile Version 4.0
vtk output
ASCII
DATASET POLYDATA
POINTS 24 float
10 0 0 10 0 1 10 1 1
10 1 0 11 0 0 11 1 0
11 1 1 11 0 1 10 0 0
10 0 0 10 0 1 10 0 1
10 1 1 10 1 1 10 1 0
10 1 0 11 0 0 11 0 0
11 1 0 11 1 0 11 1 1
11 1 1 11 0 1 11 0 1
POLYGONS 12 48
3 3 0 1
3 1 2 3
3 7 4 5
3 5 6 7
3 10 8 16
3 16 22 10
3 18 14 12
3 12 20 18
3 17 9 15
3 15 19 17
3 13 11 23
3 23 21 13
CELL_DATA 12
FIELD FieldData 2
surface_field 1 12 double
86.6149 86.8001 104.801 104.987 0.0123457 0.197531 0.0123457 0.197531 92.4723
98.5344 93.4723 99.5344
surface_field2 1 12 double
86.6149 86.8001 104.801 104.987 0.0123457 0.197531 0.0123457 0.197531 92.4723
98.5344 93.4723 99.5344
POINT_DATA 24
NORMALS Normals float
-1 0 0 -1 0 0 -1 0 0
-1 0 0 1 0 0 1 0 0
1 0 0 1 0 0 0 -1 0
0 0 -1 0 -1 0 0 0 1
0 1 0 0 0 1 0 1 0
0 0 -1 0 -1 0 0 0 -1
0 1 0 0 0 -1 0 1 0
0 0 1 0 -1 0 0 0 1
FIELD FieldData 1
surface_field3 1 24 double
0 1 1 1.22465e-14 0 1.48182e-14 1 1 0
0 1 1 1 1 1.22465e-14 1.22465e-14 0 0
1.48182e-14 1.48182e-14 1 1 1 1


CFP: a11y: Make website navigation readable/accessible

Cilium Feature Proposal
Is your proposed feature related to a problem?
I would like to read the Cilium docs, but I find it hard to navigate, and to know where I am.
It's important to have spatial awareness, because it makes it easier to return to a particular document later in time.
For example, when reading this document "Kubernetes Host Scope", and scrolling up/down, it becomes unclear where in the navigation (left side) you are. The text is bold, but so is the category/subcategories.
Describe the feature you'd like

Use a different color, and perhaps an icon, to clearly indicate "You are here", and perhaps similarly for the section that you are in.
Make the scroll bar always visible, and scroll independently of the main reading area
Make the version selector non-overlapping

Example:

Pinging @zacharysarah on this one, seems reasonable to me, but I am not a tech writer.

For the record, the theme that we use for the docs is in this repo and branch: https://github.com/cilium/sphinx_rtd_theme/tree/v1.0. Then we reference it from Cilium's Documentation/{,requirements-min/}requirements.txt.
@erikschul Thanks for the report, and for the precise suggestions (they sound good!). Is this something you have any desire to work on?
As a side note, there are currently offline discussions about moving away from Sphinx at some point (but nothing confirmed yet as far as I know - Sarah would know best). So I expect little activity on the theme until this has been clarified.

Hmm, I think it would be hard for me to build/run locally.
But I tried using the Chrome extension User JavaScript and CSS, and applied the following CSS:
html body a.reference.internal.current {
	color: rgb(0 115 230);
	text-decoration: underline;
}

html body .wy-side-scroll {
	width: 100%;
}

html body .wy-nav-side {
	padding-bottom: 0;
	margin-bottom: 32px;
}

It doesn't support highlighting the current #section , but that's probably a missing feature in readthedocs.
If you can insert this anywhere in the css files, it should work.
this solves:

more obvious styling for "current" section
scroll bar visible on navigation
bottom of navigiation is visible when scrolling all the way down (not hidden by version selector)


"Chef In Infinite Plane" problem of codechef

Aim
I want to add the code for "Chef In Infinite Plane" problem of codechef (July Cook-Off)
Details
Problem name : Chef In Infinite Plane
Problem Code : CHFPLN
Problem link : https://www.codechef.com/COOK131C/problems/CHFPLN
Please assign me this issue
LGM-SOC Participant
Programming language
C++

[ ] C
[x] C++
[ ] Java
[ ] Python

@ravikr126  Kindly assign me this issue under LGMSOC'21


Merge pull request #1 from scisoft/master

synchronize with Radovan's source

What is this? Why is this a PR?

Sorry, I was trying synchronize my fork with autocmake online, and messed it up. Closing this fake request.
For synchro I will use terminal.


add network strategy guideline to docker quick start

What's the purpose of this PR
add network strategy guideline to docker quick start to support external client to connect to apollo-configservice
Which issue(s) this PR fixes:
Fixes #3565
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.

Codecov Report

Merging #3574 (1d5a7cd) into master (aa4c3a9) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3574   +/-   ##
=========================================
  Coverage     50.95%   50.96%
- Complexity     2324     2325    +1
=========================================
  Files           447      447
  Lines         13989    13989
  Branches       1423     1423
=========================================
+ Hits           7128     7129    +1
  Misses         6373     6373
+ Partials        488      487    -1

Impacted Files
Coverage Δ
Complexity Δ

...ervice/service/ReleaseMessageServiceWithCache.java
87.05% <0.00%> (+1.17%)
25.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


Bugzilla 1479656, 1485229 and 1486223 fixes

This pull request also contains a version of the CAVLC B-frame support. As that is merged now, perhaps it's ok though, but it would be nicer to have it on a new plain branch without any extra commits.


ENH: add script to extract BORN file from QE ph.x output

This script is equivalent to phonopy-vasp-born but for QE output.
Edit:
The --primitive_matrix and --supercell_matrix flags are currently not supported

Thanks @flokno!

I have forgotten to ask about phonopy-vasp-born. How can users use this tool?

I have forgotten to ask about phonopy-vasp-born. How can users use this tool?

You simply run phonopy-vasp-born in a folder with OUTCAR or vasprun.xml. It will write the content of the BORN file to screen.
phonopy-qe-born works similarly, but one has to specify the input files as explained via the --help flag, e.g.
phonopy-qe-born pw.in ph.out | tee BORN

Should we document this somewhere?

Thanks @flokno.

Should we document this somewhere?

I will take care of it.


Prepare for Gradle plugin release 3.0.0

#161  👈
main

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

Join @rbro112 and the rest of your teammates on  Graphite


Improve bundling of icon picker

We are including the entire icon picker with all the icon sets in our main bundle :/ This massively increases the size, beyond what is actually needed.
Two problems:

all icons sets bundled but we only use fontawesome-v5
they're all bundled into the main app bundle, instead of one that loads of demand (it's a very infrequently used page)

Fixes for either or both are very welcome!

https://dev.karrot.world/bundlesize.html

I'll have a look...


Can I inject a script before slave run container?

I post this question in Google group, also post here.
Our docker container is depend on AWS EBS. We want to create a EBS volume and mount on slave when launch a task run a container. That require to customize the Marathon's executor, let Marathon's executor to run a script which prepare EBS volume before Docker container run. is possible to do that?  I look through document and find out the 'executor' parameter in API seem can do that. What's mean the 'executor' parameter and how to fill it? thanks

Hey @tnachen can you say something to this?

Hi @mingqi, you like to create a EBS volume on every new task?
Currently there isn't a way to pre-run some command before starting a new task, there is an issue open and I believe someone is working on adding a hook so that you can perform custom action on every docker container task on a per slave basis.

How is this PR going?  If there are preCmd and postCmd in the api, then we can dynamic create volum e for multi instance and other things. Also, after docker stop, we can make some volume clean work.

+1

preCmd and postCmd should be run in the host, not in the docker container.

+1
在 2016-02-25 20:54:01，"AI-PaaS"<EMAIL_ADDRESS>写道：
+1
—
Reply to this email directly or view it on GitHub.

+1

We have hooks avaialble in Mesos now that can run commands before every docker container launched, @aquamatthias is there any Marathon support for pre commands now?

My solution is write a docker script and use it in mesos slave --docker option. The script can do something before docker run every time.

@mingqi
could you pease give more instructoons？
thanks

@tnachen this is currently not supported in Marathon. Can you point me to the proto or an example?

i suggest the hook solution like this:
{
"id": "/develop/test/myapp",
**    "preCmd": "some initial commands on the slave node",                                                              1**
"cmd": "env && cd applications/ && chmod 755 run.sh && ./run.sh",
**    "postCmd": "some post commands on the slave node",                                                                 2**
"container": {
"type": "DOCKER",
"docker": {
...
**                   "preCmd": "some initial commands before the container's CMD or ENTRYPOINT",         3**
**                   "postCmd": "some post commands after the container is running"                                   4**
...
},
"env": {
},
}
preCmd/postCmd  1 && 2 run on the slave node
preCmd/postCmd  3 && 4 run within the container

Is this still evaluated?

Any updates on this? We have a container that we want to run via Marathon but need to execute a script on the host first. The preCmd option above would be perfect - but it needs to run on the host, not in the Mesos agent sandbox.

Any updates on this? I would like to have the preCmd option as given above. Same as Matthew, it needs to be run on the host...

We do not support this at this time. You could in theory have a app with
the same constraints that runs and then sleeps forever and have your app
depend on this, but you'd need to update both everytime either one updated
in order to achieve your desired result.
On Thu, Jan 19, 2017 at 12:54 PM Richard Siebeling<EMAIL_ADDRESS>wrote:

Any updates on this? I would like to have the preCmd option as given
above. Same as Matthew, it needs to be run on the host...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/1001#issuecomment-273895425,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAweZh6nzJVrtoREYj-4DIjmg-J2zITgks5rT81xgaJpZM4DSkyG
.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3931. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Object reference not set error whle deploying .NET 5 elastic beanstalk app

When publishing, we get an error. The logs show the following
Configuring application bundle for a Linux deployment
Writing Procfile for deployment bundle
Unknown error executing command: Object reference not set to an instance of an object.
   at Amazon.ElasticBeanstalk.Tools.EBUtilities.SetupPackageForLinux(IToolLogger logger, EBBaseCommand command, DeployEnvironmentProperties options, String publishLocation, String reverseProxy, Nullable`1 applicationPort)
   at Amazon.ElasticBeanstalk.Tools.Commands.DeployEnvironmentCommand.<PerformActionAsync>d__16.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Amazon.Common.DotNetCli.Tools.Commands.BaseCommand`1.<ExecuteAsync>d__10.MoveNext()

We have no hint as to why this is happening. It has worked for us in the past. The last time we ran a publish was a month ago or so.

Hi @bzuillsmith , does it help if you clean your solution and rebuild it before deploying?
Failing that, in your project's bin\xxx\net5.0\publish folder (where xxx is Debug or Release), can you please check if there is a *.runtimeconfig.json file present?

Sorry, we just recently figured it out. The project had some project properties set that conflicted with the deployment. I don't recall all the names and of the properties, but it was set to target win10-x64 and self-contained was set to true, things like that. These all likely conflict with what eb deploy on a linux platform requires. It would be nice if AWS toolkit could somehow detect these conflicting properties and give a helpful message, but ultimately the issue was we should not have set those in the project file. Feel free to close this, or let me know if you want me to paste the part of our project causing the problems for you to reproduce.


Update sample ADAL dependency

@rustd

Hi @Tratcher, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

It did not work with the version of .NET Core and packages used in the templates in the stable builds for RC2. As long as some tests that scenario we should be good.


No Azure Container Registry service connection settings in Azure Web App for Containers task in Azure DevOps Release Pipeline

I have multi-container application. I have pushed the docker images to ACR through Azure Build Pipeline. From the release pipeline, I'm using Azure Web App for Containers task to deploy my application on the app service. I'm facing unauthorized permission issue while pulling the images from Azure Container Registry. I didn't find option to provide the service connection for ACR in this task. Can anybody help me on this?

is the issue resloved?

Hi, is there any resolution for this? I created a Service Connection using CLI but it is not visible to any pipelines.


Option to output only the filenames with errors when checking?

When checking a directory tree for the first time, a large number of errors can be reported for each file. This causes the filenames themselves to be lost off-screen. I was thinking it might be helpful to have a command-line option to output only the filenames of files with errors.
Thank you for making eclint; it's a helpful tool :-).

I totally agree with @matatk. Error logs can easily be hundreds of lines long for each file, for usually when a source file doesn't comply to a given indentation convention it does so on most lines.
I use files-only filtered EClint reports so often that I've added a custom alias to my shell for this:
alias ecbrief='eclint check 2>&1 | grep  "^[^ ]"'

Having someghing like a --brief option for eclint check, to limit error logs to listing files only, is especially needed for big projects that rely on EClint for continuous integration build tests, where one wishes to keep the failure log as brief as possible.
In my repositories I use a custom script to achieve this, by first redirecting EClint output to a temporary file and then, in case of failure, print a filtered version of it via grep  "^[^ ]" — here's an example from my validate.sh script:
# ==============
# Validate Files
# ==============
# Check that project files meet the code style standards set in `.editorconfig`;
# if not, print only the list of files that failed -- because EClint reports are
# usually too long.

tmpLog=./eclint.log
eclint check 2> $tmpLog || {
    echo -e "\033[31;1m~~~ ERROR! ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~";
    echo -e "\033[31;1mThe following files didn't pass the validation test:\n\033[33;1m";
    cat $tmpLog | grep  "^[^ ]";
    echo -e "\033[31;1m\n\033[31;1mRun ECLint locally for detailed information about the problems.";
    echo -e "\033[31;1m~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~";
    echo -e "\033[31;1m/// Aborting All Tests ///\033[0m";
    rm $tmpLog;
    exit 1;
    }
rm $tmpLog;
echo -e "\033[32;1m/// Test Passed ///\033[0m"
exit

Indeed, having a command line option to just list the files that didn't pass the test would be much better as it would spare the need for such scripts and could achieve the same with a single command on the configuration file of services like Travis CI.


BUG: ycsb options_run not used for run phase

ycsb.options_load was used for both run and load phase, possible copy pasta error. fixing it

Results for minikube_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_ycsb
Pass
0
00:00:40

test_fiod
Pass
0
00:02:37

test_fs_drift
Pass
1
00:00:54

test_smallfile
Pass
0
00:01:20

test_pgbench
Pass
0
00:00:52

test_iperf3
Pass
0
00:01:43

test_sysbench
Pass
0
00:00:34

test_uperf
Pass
0
00:03:55

test_byowl
Pass
0
00:00:32

test_hammerdb
Pass
0
00:01:04

Results for kni_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_ycsb
Pass
0
00:01:07

test_fiod
Pass
0
00:03:14

test_fs_drift
Pass
0
00:01:22

test_smallfile
Pass
0
00:01:33

test_pgbench
Pass
0
00:01:04

test_iperf3
Pass
0
00:02:04

test_sysbench
Pass
0
00:00:45

test_uperf
Pass
0
00:04:03

test_byowl
Pass
0
00:00:57

test_hammerdb
Pass
0
00:01:36


adding queryByName/queryAllByName

Describe the feature you'd like:
Hey
thanks for this project.
I was wondering if there was a strong reason to not support (query|get)AllByName or if this is a missing feature. At the moment, we are adopting dom-testing-library in our project and I've run into concerns where it has forced me to adapt my implementation.
Some pain points

I have to apply data-testid when I don't have to otherwise e.g in forms
unless an external library supports accepting data-testid, I have to wrap component I desire to test and add in data-testid

Suggested implementation:
if there are no reasons against it, I would like to propose we add support to select by name (queryByName)
I tried this out and worked out fine, and would be happy to follow this up with a PR
Describe alternatives you've considered:
on top of wrapping components with e.g <div data-testid="my-id"/>, and using getBySelectText/getByPlaceholderText has fulfilled my needs.
Teachability, Documentation, Adoption, Migration Strategy:
TBD

Is there a reason your forms aren't using labels? A form without a label is not nearly as accessible.

Hello
yes we do use labels 😅
however good call because we do use htmlFor and I simply didn't pay close attention to queryLabelByText 🤦♂
This block gave me the info I was looking for and would have solved a lot of cases.
https://github.com/kentcdodds/dom-testing-library/blob/2cda6a77da4e5375f224acde566cc7a7da840472/src/queries.js#L43-L50
oh well, this was an easy one 😅
see you in react@AMS


Contact Journal AllOver-10Minute Change (EXPOSUREAPP-9236)

This PR addresses Exposureapp-9236.
Change to 10 minute intervalls on every relevant entry inside the Journal.
HowToTest:

Open the Contact Journal
Create some testentries
check the duration picker
check the export function

Screenshots:

shouldn't the duration interval be also updated (15 > 10 ) in this PR?


add ability to install on Magento 2.3.x via composer

Currently it is not possible to install module on Magento v2.3.2 via composer due to this requirement
"magento/framework": "~102.0.5"

Module has been tested on v2.3.2, so I assume this requirement can be updated.
Thanks

Thanks!


bump hip to 5.1.1 to fix corona

After the TCE update the 5.1.0 rocm is no longer available on corona,
but both corona and tioga have 5.1.1, so hopefully this will unblock us.

These should really be using rocmcc rather than clang as the compiler too, but it looks like that breaks uberenv or something else so... yeah, leaving it clang for now.


GetTickersAsync failes for Binance

Hi there
I'm no longer able to execute GetTickersAsync for binance. it failes with the following exception:
2022-09-22 22:18:33,742  - Exception in run: System.IO.InvalidDataException: No market symbol metadata returned or unable to find symbol metadata for AXSBIDR
at ExchangeSharp.ExchangeAPI.ExchangeMarketSymbolToCurrenciesAsync(String marketSymbol)
at ExchangeSharp.ExchangeAPIExtensions.ParseTickerAsync(ExchangeAPI api, JToken token, String marketSymbol, Object askKey, Object bidKey, Object lastKey, Object baseVolumeKey, Object quoteVolumeKey, Object timestampKey, TimestampType timestampType, Object baseCurrencyKey, Object quoteCurrencyKey, Object idKey)
at ExchangeSharp.BinanceGroup.BinanceGroupCommon.ParseTickerAsync(String symbol, JToken token)
at ExchangeSharp.BinanceGroup.BinanceGroupCommon.OnGetTickersAsync()
at ExchangeSharp.ExchangeAPI.b__85_0()
at ExchangeSharp.CryptoUtility.<>c__DisplayClass89_01.<<CacheMethod>b__0>d.MoveNext() --- End of stack trace from previous location ---    at ExchangeSharp.MemoryCache.GetOrCreate[T](String key, Func1 notFound)
at ExchangeSharp.CryptoUtility.CacheMethod[T](ICache cache, Dictionary2 methodCachePolicy, Func1 method, Object[] arguments)
at ExchangeSharp.ExchangeAPI.GetTickersAsync()
Any suggestions?
Thanks,
Jacob

Hmmm. It seems to be working fine when I'm using a clone of the repo instead of the NuGet package. I will close the issue now.
Thanks,
Jacob

A new official nuget will be released soon. For now, you can try using the prerelease nugets: https://dev.azure.com/DigitalRuby/DigitalRuby/_artifacts/feed/DigitalRuby
Those are automatically kept up to date with the main branch of the repo.

Do you know when official nuget coming out?

Ok, I just released it https://github.com/DigitalRuby/ExchangeSharp/releases/tag/v1.0.3

@vslee should be reopened, doesn't work.
No market symbol metadata returned or unable to find symbol metadata for AXSBIDR
   at ExchangeSharp.ExchangeAPI.ExchangeMarketSymbolToCurrenciesAsync(String marketSymbol)
   at ExchangeSharp.ExchangeAPIExtensions.ParseTickerAsync(ExchangeAPI api, JToken token, String marketSymbol, Object askKey, Object bidKey, Object lastKey, Object baseVolumeKey, Object quoteVolumeKey, Object timestampKey, TimestampType timestampType, Object baseCurrencyKey, Object quoteCurrencyKey, Object idKey)
   at ExchangeSharp.BinanceGroup.BinanceGroupCommon.ParseTickerAsync(String symbol, JToken token)
   at ExchangeSharp.BinanceGroup.BinanceGroupCommon.OnGetTickersAsync()
   at ExchangeSharp.ExchangeAPI.<GetTickersAsync>b__85_0()
   at ExchangeSharp.CryptoUtility.<>c__DisplayClass89_0`1.<<CacheMethod>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at ExchangeSharp.MemoryCache.GetOrCreate[T](String key, Func`1 notFound)
   at ExchangeSharp.CryptoUtility.CacheMethod[T](ICache cache, Dictionary`2 methodCachePolicy, Func`1 method, Object[] arguments)
   at ExchangeSharp.ExchangeAPI.GetTickersAsync()
   at BTCPayServer.Services.Rates.ExchangeSharpRateProvider`1.GetRatesAsync(CancellationToken cancellationToken) in C:\Sources\btcpayserver\BTCPayServer.Rating\Providers\ExchangeSharpRateProvider.cs:line 28
   at BTCPayServer.Services.Rates.BackgroundFetcherRateProvider.Fetch(CancellationToken cancellationToken) in C:\Sources\btcpayserver\BTCPayServer.Rating\Providers\BackgroundFetcherRateProvider.cs:line 232
   at BTCPayServer.Services.Rates.BackgroundFetcherRateProvider.LatestFetch.GetResult() in C:\Sources\btcpayserver\BTCPayServer.Rating\Providers\BackgroundFetcherRateProvider.cs:line 72
   at BTCPayServer.Services.Rates.BackgroundFetcherRateProvider.Fetch(CancellationToken cancellationToken) in C:\Sources\btcpayserver\BTCPayServer.Rating\Providers\BackgroundFetcherRateProvider.cs:line 256
   at BTCPayServer.Services.Rates.BackgroundFetcherRateProvider.GetRatesAsync(CancellationToken cancellationToken) in C:\Sources\btcpayserver\BTCPayServer.Rating\Providers\BackgroundFetcherRateProvider.cs:line 205

Can confirm, it still does not work with v1.0.3

Re-opened. We would welcome a pull request from anyone.

It is the same issue as #785. Already fixed with #787.
Everything seems to be fine in the master branch.
I am not sure why it throws exception in the 1.0.3 nuget package.

@vslee Can you double-check why this might not be included in the 1.0.3 release?

https://dev.azure.com/DigitalRuby/DigitalRuby/_artifacts/feed/DigitalRuby

Narrowed down the problem: It seems like the binance.com API which is used in ExchangeSharp does not return the BTCUSD pair, whereas e.g. binance.us does:

https://api.binance.com/api/v3/ticker/24hr?symbol=BTCUSD
https://api.binance.us/api/v3/ticker/24hr?symbol=BTCUSD

I tried replacing the base URL (.us instead of .com) and that works for us. So the issue in general seems more like binance.com removed that trading pair.

@dennisreimann There is no USD currency on BinanceCom

I think we should close this issue.


Support setting of whether txn exec is allowed when TiKV disk full.

Feature Request

When TiKV is disk full, some TiDB operations should be allowed. And the transaction module should provide support.

I think this issue is resolved by #256 ?

it should already been fixed. Please reopen if not.


How to disable all cops except some

You use the --only command line option. For example
rubocop --only Metrics
rubocop --only Style/FormatString,Style/HashSyntax

Thanks, but how to do it with config file?

I'm not sure why you would want to do that exactly, but the normal workflow when starting to use RuboCop is to run rubocop --auto-gen-config, then add inherit_from: .rubocop_todo.yml in .rubocop.yml. It achieves the same goal as disabling all cops, except that the cops that have not found any offenses remain enabled.

I was wondering the same thing. I don't want to inherit all of the default style checks that Rubocop provides by default and would much rather just specify the things we want rather than have to keep going back in to override things

Tried messing around in a fork a bit to see if we can just get rid of the default config, but it looks like we're expecting configuration values to be present for all the cops (as opposed to reading the configs and only loading cops that are declared)
https://github.com/backupify/rubocop/pull/1

Opening this issue again to make a pull request with a solution.

👍

Sweet thanks!

This is a great feature, thanks! Is there any ETA on the upcoming release containing this?

It would actually be nice to have the same option for each category. I'm interested in everything except all the Style/

@pschambacher This issue talks about a few different options, so can you clarify a bit what it is that you want?

@jonas054 this is what I was thinking about:
AllCops:
  DisableByDefault: false

Style/AllCops:
  DisableByDefault: true

This way I get all existing (and future) cops enabled except for any existing (and future) Style cops.
Making it the other way around would be "I only care for Style cops"

That's a good idea. But the syntax should be
Style:
  DisableByDefault: true

Currently, such a setting is silently ignored, which is clearly not what we want.

I'm completely fine with this syntax :)

I came looking for
Metrics:
  DisableByDefault: true

and found this ticket. is anyone working on it?

Most likely no.

For people get here later and interested in this, you can found doc at:
https://docs.rubocop.org/rubocop/configuration.html#generic-configuration-parameters
What you want may look like this:
AllCops:
  DisabledByDefault: true

Style:
  Enabled: true


Rust: JSON post request results in Request-Mismatch with default values

I am trying to use pact-consumer and setup a pact interaction with a POST request and JSON body.
However, I am constantly running into an error 500 with "x-pact": "Request-Mismatch".
I broke it down to a simple example on this repo and I do not see what I do wrong.
So far my guess is that using the following macro combination on line 17 is at fault:
i.request
  .post()
  .path("/")
  .json_utf8()
  .json_body(json_pattern!(like!(json!(Data::default()))));

Though I tried various combinations and none seem to work. The aim is to allow any POST request with arbitrary id values (deserialized from the struct, however).
Am I overlooking something or is there a bug inside pact?

It's because you called json_utf8() which changes the expected content type via content-type header to application/json; charset=utf-8 but reqwest isn't sending it with the UTF charset parameter (just application/json). You just need json_body(...), and removing the utf ones makes the test pass.
When I change the response line to:
.json_body(json_pattern!(like!(json!({}))));

The test errors as you would expect:
thread 'post_test' panicked at 'could not read response body: reqwest::Error { kind: Decode, source: Error("missing field `id`", line: 1, column: 2) }', tests/pact_json.rs:38:46

Side note on Pact framework error messages
I noticed that Pact wasn't printing the actual error message that would have made the underlying problem clear. The real error message is:
  - Mismatch with header 'content-type': Expected 'application/json' to match '^application/json; charset=(utf|UTF)-8$'

Which I think would have helped resolve the issue. I believe this error was being suppressed because of the other assertions before `
The error is only printed when (in this case) pact goes out of scope, however because there are other assertions before that happens I believe that suppresses the real problem - what's the best way to print errors in this case?
The example in the docs would suffer the exact same problem:

The mock server errors will now be written to standard error if the test is panicking because of another assertion that has failed.


fix(ssr): robust regexp to check cjs content

Description
Some dependencies can be externalized
Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Thanks @ygj6 🙌🏼


#3118 add persistent properties to replay search page

https://github.com/FAForever/downlords-faf-client/issues/3119
All the fields used in search are now remembered after closing/opening the client and navigating away/to the replay search page.

One thing we should probably additionally do is set a max date range for the replays. We currently initialize the range to be one year from today due to server load and probably don't want the persistence to undo that.

Any ideas on how you want this to look?

One thing we should probably additionally do is set a max date range for the replays. We currently initialize the range to be one year from today due to server load and probably don't want the persistence to undo that.

Any ideas on how you want this to look?

Yeah I was thinking it would either be defaulting the preferences everytime or setting a limit on the range controller. Setting a limit might be easier.

The code looks good but there are some failing tests

Adding a null check is not the way to fix the tests.
Rather you need to properly mock the filter builder in the test code so it returns mock versions of the controllers

There is still a minor issue, if no preferences have been set (i.e. empty client.prefs file) the range slider low and high values default to 0

In that case if null would be better we can make them Object property and that will be nullable

Yes I believe so


OpenLens launches shell instances, running at 100% CPU which do not close on Lens close

Describe the bug
When starting OpenLens and launching pod shells, multiple shell instances are started, which consume 100% CPU Load. These Shells also do not end once OpenLens is closed.
To Reproduce
Steps to reproduce the behavior:

Start OpenLens
Connect to any cluster
Watch out for shell instances, consuming 100% CPU

Expected behavior
I have no idea what these processes are doing. If this is legitimate processing, the least is to end these when OpenLens is closed and inform the user that legitimate activate is spawned in the background.
Alternatively, do not make this happen at all.
Screenshots
See demo here: https://youtu.be/ZbT7v8P1L9s
Environment (please complete the following information):

Lens Version: 6.5.2-356
OS: MacOS 13.4.1
Arch: arm64 (M2)
Installation method (e.g. snap or AppImage in Linux): Homebrew

Kubeconfig:

Not related to a specific config
Mine too large to share

Additional context
Only way to get rid of the 100% CPU shells is to kill them individually.

Trying to hit the 1K?


6.0 deprecations - rotate warnings and add release notes

Thanks @laymonage! I intentionally skipped over some of the deeper customisation-related things, since it's only really a recap of changes we've documented already - but I suppose field_as_li.html in particular is one that's going to hit a lot of add-ons that we don't maintain ourselves and prompt a flurry of "I upgraded my site and it broke, what's going on?" queries, so it makes sense to mention these briefly too...

Merged in 6b973dca450fd2b25920fe37c3144a48eea88942.


Add missing schema version

The new 1.4.2 version update is causing issues with project config files. When installing the update and applying the project config files I get the following error message:

After digging into the database and the project.yaml file I found that everything is correct (the project.yaml schema version for the redactor-anchors plugin is 2.0; the schema version on the plugins table is 2.0 and the schema version on the projectconfig table is 2.0) but the schema version retrieved by Craft is 1.0.0 (the default, set in craftcms/cms/src/base/PluginTrait.php) because the explicit schemaVersion is missing from the plugin. (The code that checks the schema version is on craftcms/cms/src/console/controllers/ProjectConfigController.php:280)
The issue is preventing every kind of project config update to be applied.
Adding public string $schemaVersion = '2.0'; in the plugin declaration fixes the issue.

Thanks for your contribution! With new Craft 3 installations, the plugin had no issues. With existing installations, however, it did throw that schema warning.
Instead of adding schemaVersion to the php file I find it a bit cleaner to add it directly to the composer.json.
(In fact, it was already specified there in earlier versions)
The fix is published with v1.4.3. Hopefully everything should work again now.

Thank you @heidkaemper the new v1.4.3 fixed the issue and everything works again.


Docs not showing up

See
http://pyansys.readthedocs.io/en/latest/loading_results.html

Fixed.  Added badge.


fix: invalid json syntax in tsconfig.json

Trailing comma is invalid json syntax. This makes some tools like vim breaks while opening this project.
https://github.com/vim/vim/blob/22105fd1fe0dcfe993b5c04c6ebe017a626116e3/runtime/ftplugin/astro.vim#L45-L61

Thanks


Porcupine Issue: Web demo not working

Expected behaviour
Following the readme tutorial at https://github.com/Picovoice/porcupine/tree/master/demo/web I expect the local server to start
Actual behaviour
When running yarn, I get "no such file or directory, lstat '....\porcupine\demo\web\resources\keyword_files\wasm\papaya'
This folder does not exist nor is explained where to get these resources from.
Steps to reproduce the behaviour
run:
yarn
yarn start

The files are located in the repo here, so if the repo has been cloned then the keyword file should be present in that location. Are those files present in your clone of the repo? If so, are you running the yarn commands from the demo/web subdirectory?

@laves thanks. Yes, I cloned the repo and the files are under resources/keyword_files. When running yarn, in demo/web, the files are expected to bei in demo/web/resources/keyword_files though. Should I just copy or have I missed some configuration?
If it matters, I'm running this on Windows.

@ddsky - there was indeed an issue with copy script on Windows. I've addressed it with https://github.com/Picovoice/porcupine/pull/952. Pull latest and it should work for you now.

Reopen if you're still experiencing issues.

@laves works wonderfully, thanks for the quick fix! 👍


HADOOP-16197 S3ATestUtils.translateException to map CredentialInitializationException to AccessDeniedException

HADOOP-16197 S3ATestUtils.translateException to map CredentialInitializationException to AccessDeniedException
Change-Id: Ie98ca5210bf0009f297edbcacf1fc6dfe5ea70cd

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
21
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1103
trunk passed

+1
compile
30
trunk passed

+1
checkstyle
23
trunk passed

+1
mvnsite
35
trunk passed

+1
shadedclient
759
branch has no errors when building and testing our client artifacts.

+1
findbugs
44
trunk passed

+1
javadoc
24
trunk passed

_ Patch Compile Tests _

+1
mvninstall
28
the patch passed

+1
compile
26
the patch passed

+1
javac
26
the patch passed

-0
checkstyle
17
hadoop-tools/hadoop-aws: The patch generated 1 new + 5 unchanged - 0 fixed = 6 total (was 5)

+1
mvnsite
30
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
806
patch has no errors when building and testing our client artifacts.

+1
findbugs
52
the patch passed

+1
javadoc
21
the patch passed

_ Other Tests _

+1
unit
277
hadoop-aws in the patch passed.

+1
asflicense
25
The patch does not generate ASF License warnings.

3397

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-619/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/619

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 13743ef0dc14 4.4.0-138-generic #164~14.04.1-Ubuntu SMP Fri Oct 5 08:56:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 5f6e225

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/1/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/1/testReport/

Max. process+thread count
344 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
22
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1084
trunk passed

+1
compile
31
trunk passed

+1
checkstyle
21
trunk passed

+1
mvnsite
35
trunk passed

+1
shadedclient
761
branch has no errors when building and testing our client artifacts.

+1
findbugs
48
trunk passed

+1
javadoc
23
trunk passed

_ Patch Compile Tests _

+1
mvninstall
30
the patch passed

+1
compile
30
the patch passed

+1
javac
30
the patch passed

-0
checkstyle
17
hadoop-tools/hadoop-aws: The patch generated 1 new + 5 unchanged - 0 fixed = 6 total (was 5)

+1
mvnsite
35
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
766
patch has no errors when building and testing our client artifacts.

+1
findbugs
56
the patch passed

+1
javadoc
22
the patch passed

_ Other Tests _

+1
unit
272
hadoop-aws in the patch passed.

+1
asflicense
25
The patch does not generate ASF License warnings.

3359

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-619/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/619

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux f41d6253cfa4 4.4.0-138-generic #164~14.04.1-Ubuntu SMP Fri Oct 5 08:56:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / ae2eb2d

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/2/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/2/testReport/

Max. process+thread count
340 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
26
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1015
trunk passed

+1
compile
33
trunk passed

+1
checkstyle
23
trunk passed

+1
mvnsite
37
trunk passed

+1
shadedclient
715
branch has no errors when building and testing our client artifacts.

+1
findbugs
40
trunk passed

+1
javadoc
21
trunk passed

_ Patch Compile Tests _

+1
mvninstall
28
the patch passed

+1
compile
25
the patch passed

+1
javac
25
the patch passed

+1
checkstyle
16
the patch passed

+1
mvnsite
31
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
676
patch has no errors when building and testing our client artifacts.

+1
findbugs
49
the patch passed

+1
javadoc
23
the patch passed

_ Other Tests _

+1
unit
276
hadoop-aws in the patch passed.

+1
asflicense
29
The patch does not generate ASF License warnings.

3127

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-619/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/619

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux c1c4f7d7244f 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 002dcc4

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/3/testReport/

Max. process+thread count
410 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-619/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

I arrived a little late but +1 (non-binding)

thanks for all your comments, merging

merged


src folder doesnt work var I18N = require('./../../i18n/index.js');

Ive got this error even I have the folder in top directory or I tried also in src.
error - ./node_modules/.pnpm/next-export-i18n@2.1.0_ld2jel3hspngo3u5lti2kgl2sq/node_modules/next-export-i18n/index.js:5:0
Module not found: Can't resolve './../../i18n/index.js'
https://nextjs.org/docs/messages/module-not-found
Is this because of pnpm or typescript settings?

https://github.com/martinkr/next-export-i18n/issues/32
This is duplicate for this issue sorry for opening this one


Merge openshift online

Merge rh-mobb -> openshift-online

Merged as this is battle-tested in rh-mobb repo already


Feature suggest: Add support for HTTP\2

It would be great to have HTTP\2 support.
Node.JS already supports it:
https://nodejs.org/dist/latest-v14.x/docs/api/http2.html

Hi there, it's already supported.
Do the exact same thing as the https example, but use the http2 module instead of the https module.
Once you create your own server, you pass it into your (main) Polka instance as options.server
Hope that helps~!


Empty value is not supported

It is a historical problem. This client is extracted from TiDB. In TiDB, every key has a non-empty value. In the implementation, the client treats an empty value as the key does not exist, and it also rejects putting a pair with empty value.
But TiKV actually supports a key with zero-length value. The user can put a key with an empty value with another client (e.g. client-rust). Then, they will find the key does not exist when reading with this client.
cc @andylokandy @disksing

The problem is not as simple as it seems, because client's unionstore(memberbuffer) also uses empty value to indicate deletion.

Maybe we can distinguish them via []byte{nil} and []byte("").

seems solved by https://github.com/tikv/client-go/pull/612

now we only support nil value for rawkv API. txnkv still disallows empty value.


luisgen: list entities parsed into a 2d array of strings

Tool
Name: LuisGen
Describe the problem
Why are list entities parsed into a 2d array of strings rather than a 1d array? It seems like it could make sense to have a 2d array if it contained both the entity from the utterance and the value it was resolved to, but right now it seems to just give the resolved value.

All entities in LUIS come back as arrays because a sentence can include more than one instance of an entity--that is why there is a separate entity for "shampoo" and "conditioner".  For list entities the second level is because the resolution can be ambiguous.  Imagine you have "green peppers" with a synonym of "peppers" and "red peppers" with a synonym of "peppers".  If someone says "peppers" then you get [["green peppers", "red peppers"]] because both resolutions are possible for that entity.

Thanks @chrimc62!  Closing.


Dynamic items not working. carousel is not visible

I am using for loop to make n items. after that data showing with any carousel effect

i also faced that issue


Doc: useAuxclasspathFile

I've read the documentation for spotbugs useAuxClasspath, and although it didn't fully make sense to me, it didn't sound like a bad idea. That said, I have no idea what useAuxClasspathFile option does, and how to make use of it.

refs #243


[graphiql] Variable "$episode" got invalid value ""; Value "" does not exist in "Episode" enum."

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
query HeroNameAndFriends($episode: Episode) {
  hero(episode: $episode) {
    name
    friends {
      name
    }
  }
}

=====
{
  "errors": [
    {
      "message": "Variable \"$episode\" got invalid value \"\"; Value \"\" does not exist in \"Episode\" enum.",
      "locations": [
        {
          "line": 1,
          "column": 26
        }
      ]
    }
  ]
}

Expected Behavior
{
  "data": {
    "hero": []
}

Steps To Reproduce
{
  "episode": ""
}

Environment

GraphiQL Version:
OS:
Browser:
Bundler:
react Version:
graphql Version:

Anything else?
No response

this would appear to be an encoding issue in your graphql server, however this is the GraphiQL repo, which is just a web/IDE client.
if you are still stuck, try our generalized #tech-help channel on discord:
https://discord.gg/Ns5kABgg


Multiple-browser: foldername implementation issue

At first I want to say what a great tool Codecept.JS is for me. Thanx a lot for that!
When I test the suite via multiple-browser option, I find 2 bugs, and have 1 feature request.

the first browser-resolution combination I configure will be used in all folder-names for  that browser.
screenshots in these folders are the same size
every device/resolution combination in my config file (for example     {browser: 'chrome', windowSize: '1220x1000'},) results in my mochawesome reporter into 1 report (so 10 browser/res combinations results in 10 tabs with 10 reports). would be cool if the multiple-browser option with mochawesome results in 1 report. somehow ;)

Foldername output when testing via multiplebrowser option:

drwxr-xr-x   7 joel  staff   238B Apr 20 19:25 basic_browser_chrome_windowSize_1220x1000_1
drwxr-xr-x   7 joel  staff   238B Apr 20 19:25 basic_browser_chrome_windowSize_1220x1000_2
drwxr-xr-x   7 joel  staff   238B Apr 20 19:25 basic_browser_chrome_windowSize_1220x1000_3
drwxr-xr-x   7 joel  staff   238B Apr 20 19:25 basic_browser_chrome_windowSize_1220x1000_4
drwxr-xr-x   7 joel  staff   238B Apr 20 19:25 basic_browser_firefox_windowSize_maximize_5
drwxr-xr-x   7 joel  staff   238B Apr 20 19:24 basic_browser_firefox_windowSize_maximize_6
drwxr-xr-x   7 joel  staff   238B Apr 20 19:24 basic_browser_firefox_windowSize_maximize_7
drwxr-xr-x   7 joel  staff   238B Apr 20 19:24 basic_browser_firefox_windowSize_maximize_8
drwxr-xr-x   7 joel  staff   238B Apr 20 19:25 basic_browser_firefox_windowSize_maximize_9
drwxr-xr-x   3 joel  staff   102B Apr 20 19:24 basic_browser_safari_windowSize_maximize_10
drwxr-xr-x   3 joel  staff   102B Apr 20 19:24 basic_browser_safari_windowSize_maximize_11
drwxr-xr-x   2 joel  staff    68B Apr 20 19:24 basic_browser_safari_windowSize_maximize_12
drwxr-xr-x   2 joel  staff    68B Apr 20 19:24 basic_browser_safari_windowSize_maximize_13
drwxr-xr-x   3 joel  staff   102B Apr 20 19:24 basic_browser_safari_windowSize_maximize_14

multi-browser configuration:
      "browsers" :
      [
        {browser: 'chrome', windowSize: '1220x1000'},
        {browser: 'chrome', windowSize: '1024x1000'},
        {browser: 'chrome', windowSize: '768x1000'},
        {browser: 'chrome', windowSize: '320x1000'},

        {browser: 'firefox', windowSize: 'maximize'},
        {browser: 'firefox', windowSize: '1220x1000'},
        {browser: 'firefox', windowSize: '1024x1000'},
        {browser: 'firefox', windowSize: '768x1000'},
        {browser: 'firefox', windowSize: '320x1000'},

        {browser: 'safari', windowSize: 'maximize'},
        {browser: 'safari', windowSize: '1220x1000'},
        {browser: 'safari', windowSize: '1024x1000'},
        {browser: 'safari', windowSize: '768x1000'},
        {browser: 'safari', windowSize: '320x1000'}
    ]

Can someone please provide a working example for this multiple browser implementation? I tried but its not working properly. Some simple example with config pls.

https://github.com/joelgrimberg/demo

Thanks @joelgrimberg !!!

@shrikanthkm07 you're welcome. Just let me know if you have questions.

@joelgrimberg Looks like first two issues is the same (multiple option just starts 5 instances with the same resolution). I'll check it.
About aggregation report feature - Every suite is running in  separate child process and it would be hard to mix all reports in one. I think that we can do some aggregator for reports that will generate one more page, that will contains list of links to reports. Will investigate this

Fix is ready to merge to master.
About reports dashboard: will monitor this issue in mochawesome https://github.com/adamgruber/mochawesome/issues/90. When it will be ready we will use it to automatically generate dashboard

@joelgrimberg
How do I configure multiple suites to run on different Google browsers?I don't know how to config it.Please help to me,thanks
"multiple": {
"suiteName": "",
"suiteConf": "",
"browser": [
"chrome"
]
},


GeistUI with Next 11, Typescript and Yarn 2 with pnp throws a "Module not found" error

Bug report 🐞
I am using<EMAIL_ADDRESS>with Typescript and have set up Yarn 2 in my project.
I proceeded to start next with yarn dev and everything compiled successfully.
I then ran yarn add @geist-ui/react, stopped the app, modified _app.tsx to look like this:
import type { AppProps } from "next/app";
import { GeistProvider, CssBaseline } from "@geist-ui/react";

function MyApp({ Component, pageProps }: AppProps) {
  return (
    <GeistProvider>
      <CssBaseline />
      <Component {...pageProps} />
    </GeistProvider>
  );
}
export default MyApp;

and ran it again.
When navigating to a page I receive a "module not found" error in the console and the browser pointing to the auto-complete.js file in the yarn cache.
Everything works if I remove Geist UI from my app scripts.
I, therefore, inspected the zip files in the following location and found this import
import _extends from "@babel/runtime/helpers/esm/extends";

at line 1 in .yarn\cache\<EMAIL_ADDRESS>to be the culprit.
Running yarn add @babel/runtime fixes the issue.
Version & Environment

Version of browser - chrome 91.0.4472.114
Version of geist-ui/react  - ^2.1.5
Version of Next - 11.0.0
Version of Yarn - 2 (Berry)

Expected Behaviour
The pages should compile properly without the Module not found error.
Actual results (or Errors)
Using yarn berry and PNP I receive the following shell error when navigating to any page in the browser:
wait  - compiling...
error - .yarn/cache/@geist-ui-react-npm-2.1.5-714ca6763e-2ef99fa586.zip/node_modules/@geist-ui/react/esm/auto-complete/auto-complete.js:1:0
Module not found: Can't resolve '@babel/runtime/helpers/esm/extends'
null

Running yarn add @babel/runtime and restarting next fixes the issue.

Thank you for the bug report, the issue has been solved.
This fix will not be merged into the master branch due to the large changes made to the module system. This fix is expected to be added to the next canary release in the next few days.
PS: The next canary release will contain breaking updates, and I'll add a detailed document on how to migrate later.

version v2.2.0-canary.7 is out. release notes


loading modules with coding errors is masked ModuleNotFoundError

This is a follow-up to #374.
There is still a problem when loading components, and the current code hides some things away, making it extremely complicated to debug and find the culprit.
I have a component that did some attribute errors within the code, it asked for forms.field - which is a nonexistent attribute -  instead of forms.BooleanField. So that component was not loaded, and in unicorn_view, line 845ff, it checks first for ModuleNotFoundError (no), and then AttributeError (yes) - BUT: you don't raise the error then, you just save it in last_exception, and the module is NOT loaded, because of the error. So it runs all iterations for all modules, and comes to the last one, where it does not find the module (because it already had found it before, but did not load it).
Here The ModuleNotFoundError is raised, and this overrides the last_exception. This error then is presented to the user - but it tells you nothing about the real cause.
It would be a possibility to remove the catching of AttributeError as a whole - so it is risen and the application stops - which is correct IMHO.
But I don't know what AttributeError you wanted to catch in the first place when you  wrote these lines... I don't have the bigger picture here.

@nerdoc I think you are right that I should just raise the error right away, but I'll dig into this and see if I can find any edge cases that might fail.

Same situation here. I Can't debug the cause of a component not work on staging environment but work on my local notebook.

Sorry about this! I'm going to make the error more clear in the next release, but I won't be able to fix it for the next week or so. But, I'll try to make it better as soon as I can. Thanks for your patience.

This also is true for components that simply raises errors, e.g. in mount().
class FooView(UnicornView):
    namespace=""

    def mount(self):
        """Check if component was called with a namespace attribute:
        {% unicorn "foo" namespace="foo" %}
        """
        if not self.namespace:
            raise AttributeError(
                "Foo component needs to have a 'namespace' attribute"
        )

At least Unicorn should propagate that error.

in https://github.com/adamghill/django-unicorn/blob/6cb0e137dadf64916376147db09854fae87b0f05/django_unicorn/components/unicorn_view.py#L898 and the following lines there is the culprit code IMHO.
You correctly save the AttributeError in the lines before, but do not print it out with the ComponentClassLoadError afterwords.
You could just add the AttributeError in the message, and everything is clearer...

This PR partly fixes the bug. I don't know if other edge cases occur, as there is the second exception raise at the bottom of that code, which still just tells the user a common error without details. I for myself never ran into that. So for me, my PR fixes the issue or at least vastly improves the feedback to the developer...


Unclear support for Teams Noise suppression

Hello,
This document doesn't answer the questions of whether Microsoft Teams supports noise suppression features within Citrix VDI environments.  Would be great if the status/limitations could more clearly provide guidance on that topic.
FYI - This other related document briefly notes "For this option, your computer's processor must support Advanced Vector Extensions 2 (AVX2)":
https://support.microsoft.com/en-us/office/reduce-background-noise-in-teams-meetings-1a9c6819-137d-4b3b-a1c8-4ab20b234c0d
Thanks,
Chris

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

ID: 17880b9d-6b98-3194-cc33-fe8067ce241e
Version Independent ID: 643bcf2f-507b-e449-73db-923d8a368ad9
Content: Teams for Virtualized Desktop Infrastructure - Microsoft Teams
Content Source: Teams/teams-for-vdi.md
Service: msteams
GitHub Login: @serdars
Microsoft Alias: v-mahoffman

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @chrisweis I am looking into this.

Hello @chrisweis
I could find this resource where you can see the feature support per platform, including VDI and the status of noise suppression
https://support.microsoft.com/en-us/office/meetings-and-live-events-5c3e0646-dc37-45ad-84a4-1666fac62d4e


Error at Usage logging policy

I have deployed two Dataspace connectors version 7.0.1 and configured them to point to my local Clearing House.
After the contract negotiation I obtain the data as expected.
The issue comes when I delete the logging of the Dataspace connectors to the Clearing House.
This is changing application.properties file of the DSC to:
## Clearing House
##clearing.house.url=XXX
##clearing.house.path.process=process
##clearing.house.path.log=messages/log

According to the documentation (https://international-data-spaces-association.github.io/DataspaceConnector/Documentation/v6/UsageControl#policy-patterns)

I should not be able to access and obtain the data after the contract negotiation because the DSC's are not logged to the clearing house.
Executing this scenario I am obtaining the data from connector acting as provider.
As far as I understand from the documentation this should not be like this and I should not obtain the data.

Currently, the clearing house configured in the consumer is used to log the data usage. Instead, it should be defined on the provider side to which clearing house the data usage should be logged. For this, it would have to be ensured before or during contract negotiation that this clearing house can be reached by the consumer, so that the policy can be enforced at the consumer. All in all, the whole policy must be implemented anew.
Simply throwing an exception and blocking the data usage wouldn't solve the problem.
We currently not plan to fix this, but are open for contribution.


Restore VR support (openxr) in V-Sekai (Vulkan errors)

Vulkan errors when openxr enabled in project settings

Fixed by merge groups-4.x remotes/v-sekai-godot/openxr_crashfix # pending pr by Saracen 2022-03-06


How to make version 1.0.0 support 'resize' and 'move' event

We need 'event' feature, But it looks does't support

Version 1 doesn't support that. I will do it whenever I have some free time, but a PR would be welcome!

Expect that, Thank you !


Add internal argument to skip pipeline execution

@PhilippeMoussalli, Are you woking on this one or can I take this up?

Hi @satishjasthi :)
Thanks for contributing to Fondant! we're glad to have you in the team,
Indeed this is something that I am planning to tackle as part of a larger ticket. I think it's easier if I take this one up along with all the issues associated with it.
I saw that you recently contributed to adding tests to the components themselves, would be great if you can continue on that ticket then we can look for other possible contributions that can be picked up

Hey @PhilippeMoussalli  :)
Sure, that makes sense 👍


Optimizing Fonts

Feature Request
Summary
I noticed that fonts total to 822 KB when loading a blog post, which is about 3/4 the size of a page.
The fonts on google-webfonts-helper are generally smaller, so I assume some optimization can be done to significantly reduce the file sizes.
Motivation
Faster load times. Only load what's used.
Detailed Description

How many font weights are needed?
What charsets are used?

Additional Context
Should users be able to customize the fonts used? Specifying fonts can be useful especially if system fonts are preferred.

I noticed that tabi takes advantage of the modern variable fonts feature in CSS, so it may be difficult to use non-variable fonts like the 400/600/700 ones.
I also saw the font charsets with FontForge. Overall, I think optimizations can be done on a case-by-case basis and feel like the current fonts are already good for a majority of users.

I'm interested in reducing the size of the resources, so I'll definitely look into some options in regards to fonts.

The google-webfonts-helper link above is pretty good when it comes to font sizes. The biggest downside is that existing font-weights would have to be rounded to the nearest 100.
Potential advantages include more consistent fonts across a variety of browsers. qutebrowser has trouble showing some of the font sizes, for example.

Coding Font
For the mono/code font (Cascadia Code), I've ran:
pyftsubset CascadiaCode-SemiLight.woff2 \
  --layout-features="*" \
  --desubroutinize \
  --unicodes=U+0020-007E \
  --flavor=woff2 \
  --output-file=font.woff2

This compresses the font file from 133KB to 25KB. The font includes:

Basic ASCII characters (U+0020-007E): Covers English and most coding symbols.
Ligatures: As layout features are preserved (--layout-features="*"), ligatures commonly used in coding are retained.

I believe this subset should be sufficient for code formatting in the vast majority of use-cases, satisfying 99.9% of users.

Serif & Sans Serif
For the serif and sans-serif fonts, I've ran:
pyftsubset SourceSerif4Variable-Roman.otf.woff2 \
  --layout-features="" \
  --desubroutinize \
  --unicodes="U+0020-00FF,U+0100-024F,U+1E00-1EFF,\
              U+0400-052F,U+0391-03D6,\
              U+4E00-9FFF,U+3400-4DBF,U+20000-2A6DF,\
              U+3040-309F,U+30A0-30FF,\
              U+0600-06FF,U+2000-206F,U+2190-21FF" \
  --flavor=woff2 \
  --output-file=optimized_font.woff2

This subset includes:

Latin and Latin Extended (U+0020-00FF, U+0100-024F, U+1E00-1EFF): Covers most European languages including those with diacritics and accents.
Cyrillic (U+0400-052F): Covers languages like Russian and Ukrainian.
Greek (U+0391-03D6): Includes basic Greek characters.
Chinese (U+4E00-9FFF, U+3400-4DBF, U+20000-2A6DF): A broad set for Simplified and Traditional Chinese.
Japanese (U+3040-309F, U+30A0-30FF): Includes Hiragana and Katakana.
Arabic (U+0600-06FF): Basic Arabic script.
General Punctuation (U+2000-206F): Covers a range of punctuation marks including the bullet point, dashes, and various kinds of spaces.

This configuration should cover a broad spectrum of languages and scripts, aiming to satisfy most use-cases.
This brings the sans serif font, Inter-4.0-beta9h, from 326KB to 191KB. (Kept layout-features)
The serif font, Source Serif 4, went from to 429KB to 196KB. (Removed layout-features; saw no downside)
Aggregated, this takes the font sizes f rom 888KB to 412; less than half of the original size.

I will be doing a PR with these changes soon.

That sounds awesome! Thanks for working on this!

Done!
Thanks for requesting this, I had been meaning to look into it.
PR: #186. You might be interested in the last section, which includes instructions to create your own—potentially much smaller— subsets without needing to modify the theme's files.


Convert all materials in the demo game to .tres format

Currently "2 minute adventure" uses .material format for materials. However, it is difficult to track changes in this type of files. Converting all materials to a text format (.tres) would solve this issue.

This is a very low priority task. On the other hand, it shouldn't take long :)

This will take some time, and it's not relevant for now.


Final improvements - fix events

Save Reward, Licking, AirPuff, Light stimulus sensor output from PicoScope as TimeSeries
link the time series from the TimeIntervals table
Also contains the updates after running conversion from fresh environment (frozen_dependencies.txt)

@alessandratrapani example files

@alessandratrapani can you check these example files? I've rerun the conversion and awaiting this approval to upload the files 🙈 let me know if you see something else that could be improved.

Thank you!


Templates: Upgraded to MudBlazor 7 and Latest Blazor Project Templates

Description
This update upgrades the MudBlazor templates to use the latest Blazor .NET 8 templates and MudBlazor 7 (preview 4).  This update also uses MudBlazor components wherever possible for all pages.
How Has This Been Tested?
TODO
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

Hello @ScarletKuro @henon @danielchalmers, here's a draft PR for updated MudBlazor template projects, made from scratch based on the latest aspnetcore Blazor templates.  There's of course a bit of work to do to understand the newer template code, as well as to upgrade pages to use MudBlazor components.
This was PR'ed early so you can see the raw template changes before I dig into upgrading to MudBlazor components and restoring some of the original custom MudBlazor template code.
Also, did you have a workflow to let you quickly test changes as you worked?  It looks like I'll need to install templates, make a solution, then run it each time I need to test something.  Thanks for the review!

@mikes-gh will you help with reviewing this?

See #465


Create Problem Flow

After creating the problem with the title -> go to problem editor page

This will be resolved in the linked issue.


0.9.2

There was a typo in protocol.js, it was put as Thift.TProtocolExceptionType when it should be Thrift.

This has been taken care of in trunk, no references to 'Thift' can be found


Add interactive graphic - how batch size affects gradient accuracy

Add an interactive version of the 1D gradient graphic that highlights how batch size affects gradient accuracy.
Initial settings result in this plot

A batch size of 1

A batch size of 256

Resolves #17

This is another fantastic change!


Can I perform Code coverage for IBMi for the programs with screen?

No, it if not possible currently. Eventually, you'll be able to import coverage results, which will then support coverage from anywhere.


"Replace anything" functionality

Hi, thanks for this great extension! One of my main use cases is the "Replace Anything" feature which changes the background https://github.com/geekyutao/Inpaint-Anything#-replace-anything. Any chance this will be added/is easy to add? (I saw a related issue for "Remove Anything" but that's the inverse and is implemented differently.)
I think this would be useful for a lot of people, who are just trying to use the vanilla inpaint function for this right now!

Proposed Thank you. I have reviewed the "Replace Anything" implementation that you linked to, and it seems to be using the same Diffusers StableDiffusionInpaintPipeline as this repository. Therefore, the difference is in the mask specification. However, since the current implementation covers it well, I don't think it's necessary to add the feature.


Postgres 12 - Array Editor does not work properly

Refers to this issue: #403
System information:

Windows 10 Pro (Version 10.0.19041 Build 19041)
DBeaver version 7.2.2

Connection specification:

PostgreSQL 9.6.12 and PostgreSQL 12.2.
PostgreSQL JDBC Driver

Describe the problem you're observing:

for new arrays everything works fine
for existing arrays a double-click does not let me edit values
for existing arrays I can only add elements with NULL-values or remove elements

Steps to reproduce, if exist:

Have a table.
Create new column with, for example, _varchar or _int2 type.
add a new data-array with arbitrary values (works fine) and persist it
reload the table (F5)
try to edit some values of the just created array (does not work)
try to add new elements to the array
try to remove elements from the array
try to persist the changes (results in the Errors mentioned below)

Include any warning/errors/backtraces from the logs
Error creating struct
  Unable to find server array type for provided name _varchar.

Error creating struct
  Unable to find server array type for provided name _int2.

Another "wish" from my side, if someone will work on this soon:

Please make sure, I can handle NULL values inside arrays
This can be done with an additional option (set NULL-value) on the right-click action
Similar to right click on simple varchar or text data cells

Thanks in advance for your work on this.
For me, DBeaver is the best Open Source solution that wipes out all the disadvantages of pgAdmin4.

Thank you for the report. I have an error in Error log only when I delete the data from array cell (brackets remain in it).
But the changes are saved. Also adding, editing array values work for new and existed rows in 7.2.3 DBeaver PostgreSQL 12.

Is it possible to send DDL example to reproduce other cases?
Could you check 7.2.3 version https://dbeaver.io/download/?

I checked the new version 7.2.3.
I also created a new column (_varchar) for testing.
Still the same issues are coming up as mentioned already.
I made a video for you. Will send it per mail.

This bug was recently fixed as a part of #9964 .
Fix will be included in 7.2.4


Folder won't open if path contains apostrophe (')

System info:

Neovim 0.6.0
Kubuntu 20.04.3 LTS
Latest nerdtree

I don't know if anybody else has experienced the same issue since there are no other complaints, but when I open a folder say "Foo's Bar" it won't open. At first I thought it was because of that space between words but when I remove it the error still occurred. I then remove the apostrophe and it worked.
Maybe this is the expected behavior since punctuation or special characters are usually not used in paths but I figure it is worth mentioning it. (Or it could be a bug)

And does the folder open if it contains a space? Have you tried? with "Foos Bar" name? I am getting a similar issue and my hypothesis is the space presence, becaue I do not have any (other) special character in the name.

Tried opening a directory that conatains blank and it didn't list what within it.

I cannot reproduce this issue, Since this issue has been stale for a long time I'm going to close it, Feel free to reopen it with a minimal .vimrc / init.vim file that can produce it.


proxy set handler returned false for property "'toast'"

export interface IStore {
  toast: ToastProps;
  resetToast: () => void;
}

export const store = proxy<IStore>({
  toast: {
    success: null,
    error: null,
    open: false,
  },
  resetToast() {
    this.toast = {
      success: null,
      error: null,
      open: false,
    };
  },
});

This is not the first time the error message confuses people. #178
I understand the problem is serious.
To my knowledge, it's hard to detect this at runtime.
But, we don't need to give up. Let's try two challenges.

explore if we can customize the error message (this requires deeper understanding in valtio and proxy-compare)
add a new rule to warn this usage in eslint-plugin-valtio. Though this is valid for some use cases, we can't detect good one and bad one (would be nice if it could).

I'd like to ask for volunteers.
cc @Aslemammad @YPAzevedo @TheEhsanSarshar @israelidanny @a-eid @barelyhuman @LXSMNSYC

I wouldn't mind going through both the recommendations and let you know what my thinking of this is

@dai-shi I wouldn't mind playing around with option (1). I think it's a better approach than Option (2) personally both in ease of implementation and practicality (we can't guarantee that someone is running a great & fully functioning linting config)
I'll get back with feedback in the next day or two if nobody gets there before me.

@israelidanny I gave up option 1 in a nice way, so it might require some tricks. One thing to note is we don't want to modify proxy-compare for valtio-only problems.
@barelyhuman So, would you primarily take option 2?
I appreciate your helps. Please also refer discussions in #178.

This was confusing to say the least. The error was pointing to the store. But it was my fault.
I was trying to call resetToast on snap
const snap = useSnapshot(store);
snap.resetToast();

After that fix. It's been working great.

If one prefers using this, they need to make sure calling the method on store not snap.
store.resetToast();

Another discussion in #205.

If one prefers using this, they need to make sure calling the method on store not snap.
store.resetToast();

Another discussion in #205.

I made the change. Now I'm using store for mutation and snap for reading and reacting to changes.

There is the option of binding all the methods to the correct this while building the snapshot, but it may not be the approach we want here. This way there wouldn't be any error, but it does introduce a little bit of "magic". What do you think @dai-shi ?

There is the option of binding all the methods to the correct this while building the snapshot, but it may not be the approach we want here. This way there wouldn't be any error, but it does introduce a little bit of "magic". What do you think @dai-shi @barelyhuman ?

While that would help with the developer experience but then we start assuming what the developer wants, someone might use this inside a function and expect this to have a different context, plus the support would then have to be handled for all types of function definitions.
As in, context of this for arrow is going to be different and context for for lamda, named, object method is going to different, I think we'd over engineer it and then call for more problems, I still have to do the above while adding to the linter rules but that is to warn the user of the keyword, and not assume the context of this so it's still a viable solution.
There can definitely be better solutions , i'm open to any.  Since I've only written like 2-3 lines on the eslint plugin as of now

There is the option of binding all the methods to the correct this while building the snapshot, but it may not be the approach we want here. This way there wouldn't be any error, but it does introduce a little bit of "magic". What do you think @dai-shi @barelyhuman ?

The current behavior is intentional to make it as pure JS as possible.
this on snapshot is useful for getValue function.
const state = proxy({
  count: 1,
  getDoubledCount() {
    return this.count * 2
  },
})

console.log(state.getDoubledCount())

const Component = () => {
  const snap = useSnapshot(state)
  return <div>{snap.getDoubledCount()}</div>
}

This doesn't work if we bind getDobuledCount().


[Question] v6 - How to dynmically set height

Description
In previous version I could only specify width to 100% and LottieView would automatically have the correct height, keeping the json ratio.
I read #1078 but I think I am missing something here.
In my scenario the content team can manage the content of our pages from a CMS and they upload animation's json to it.
We then fetch them from URL, having no way to know their size, or ratio.
The current README states we can use this code
  return (
    <LottieView source={require("../path/to/animation.json")} autoPlay loop />
  );

but it does not work except if width and height are set, which seems coherent with the v6 changelog.
Can you please explain if there is a way to achieve this in v6?
Lottie Version
Version: 6.4.0

Hi,
You can store width and height of each animation alongisde their json files in your CMS. Then use those information to build your style based on your server response.

 This data can be used to create styles based on the server's response.

This functionality was really appreciate. It is bizarre that no more developer create tickets about it.
I had this problem too, btw https://github.com/lottie-react-native/lottie-react-native/issues/1078
I start a function to get proportional size but it need some adjustments:
/**
 * Get proportional size from the other reference.
 * @param source
 * @param type
 * @param reference
 * @returns
 */
export const getProportionalSize = (source: any, type: 'width' | 'height', reference?: number | null) => {
	let defaultSize: any = { h: 0, w: '100%' };
	if (type === 'height') defaultSize = { h: '100%', w: 0 };

	if (!source) return defaultSize;
	if (!reference) return defaultSize;

	const { h, w } = source;

	if (type === 'width') {
		if (reference >= w) return { h, w };

		const difference = (reference * 100) / w;
		const cHeight = (h / 100) * difference;

		return { h: cHeight, w: reference };
	}

	if (reference >= h) return { h, w };

	const difference = (reference * 100) / h;
	const cWidth = (w / 100) * difference;

	return { h: reference, w: cWidth };
};

getProportionalSize('./path/to/json', 'width', 420);

In this example, 420 is my screen width, and I want to get proportional height.
The source need to include w and h keys like this:
{"v":"4.8.0","meta":{"g":"LottieFiles AE 3.0.2","a":"","k":"","d":"","tc":""},"fr":60,"ip":0,"op":120,"w":800,"h":1300, ...
I think, it is by default, when your export your animation.
It's working only with Lottie JSON file, not this DotLottie file.

PS : any plans to add this functionality back? Or some king of helper to get the animation ratio ?

When I find some time. I will try to expose a native method so that you can get the width and height of the animation file. The previous default behavior could cause a lot of problems in terms of API. With dotLottie coming up, it's not possible anymore to read width and height directly from the file itself unless I read their protocol for its binary implementation.

I think we should probably pin this issue for future reference. Sadly, the API that we used to provide for this functionality was extremely flawed and relied too much on Lottie only using JSON and that JSON file being formatted in a specific way.

@matinzd Any update on exposing a method for this? With v6 being necessary for Apple guidelines sometime soon this is still a blocker for us to upgrade.

@matinzd Really need a guideline for this issue. If you can suggest a solution, I think if you don't have time to do it, there will be plenty people willing to help. We have upgraded to v6, and all of the sudden, all the lottie views disappear.


Fix: page transition animation is not working when using version 1.4

Hi! This module is great work. I like this.
But, page transition animation is not working when using AngularJS 1.4.
(maybe, caused by below breaking changes - https://github.com/angular/angular.js/blob/master/CHANGELOG.md#breaking-changes-19)
So, I fixed it. Please check and merge.
Best regards, frontainer

Great, thanks frontainer


Does not compile with -Werror and -Wundef set

While attempting updates to support this warning flag configuration use in our build, it was found that nothing defines gsl_CONFIG_CONTRACT_VIOLATION_TERMINATES yet it is used in the defining of gsl_CONFIG_CONTRACT_VIOLATION_THROWS_V.

However, gsl_CONFIG_CONTRACT_VIOLATION_TERMINATES is only used in defined(...) and I do not see it reported due to -Wundef with g++ (GCC) 5.2.0 on Windows.
I can see lots of other cases related to -Wundef though.

In release 0.23.0.


Support for 'infinity'

Postgres supports infinity timestamps.
=# select 'infinity'::timestamptz;
 timestamptz
-------------
 infinity
(1 row)

However, this adapter doesn't:

It is, however, a valid and useful range to support, particularly in the context of temporal tables.

Interesting. I actually think this is a problem with Harlequin core, or more specifically, the data table implementation. We use Apache Arrow to store the data returned by the query, and I don't think Arrow supports infinity timestamps (they have an Arrow.max constant, I believe). We probably just need to catch this error and replace the infinity value with Arrow.max.


Documenso cannot be started on NixOS

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

compute the out path of Documenso (e.g., via nix repl)
cd to that path.
Try to run npm run start

Expected behavior
Documenso should work locally on port 3000.
Additional context
It looks like it's working first, but when you try to interact with it, it spits out this error.
Key log segment:
PrismaClientInitializationError: Query engine library for current platform "linux-nixos" could not be found.
You incorrectly pinned it to linux-nixos

Notify maintainers

@happysalada
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.8, NixOS, 23.11 (Tapir), 23.11.20231221.0b3d618`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/etc/nixPath`

Add a :+1: reaction to issues you find important.

hey, thanks a lot for reporting this.
Unfortunately that doesn't ring a bell for me.
Maybe we could try to open an issue upstream about building from source.
Also one of the more recent versions might have fixed this already.
This might not be a lot of help, but if you find out a fix for this, I'd be happy to help you get it merged.

I'd love to help with this, although I'm not very familiar with the NPM ecosystem, and any issues I open may be poorly framed as a consequence. Upstream's documentation on 'development running' is pretty similar to our use case of grabbing the files, selecting the important ones, and eventually calling nix run -- so maybe that's a good starting point? The README.md instructs you to copy the .env.example file to .env, and possibly edit it with whatever configuration you want. I was wondering, why does that step seem to not show up in the default.nix?
(Apologies if this is a red herring. As I said, I'm pretty unfamiliar with the NPM ecosystem.)
I'm also not sure what the difference is between npm start dx and npm start run, but I assume it's some sort of build/run script target similar to make?
I'm going to go ahead and try to prepare it standalone environment, and see what happens.

hey, thank you for your work on this!
Just to give you some context on nixpkgs.
It's common to not set environment variables on the binary itself since those are a setting.
What usually happens is that the binary should be built properly and then a service is created where the proper environment variables are set to run it correctly. The binary itself should be independent of any settings so that anyone can later set them as they please.
if you want an example of a service where this is exactly what is done (just setting the environment variables to run a node js service), you can grep the following line
      systemd.services.lemmy-ui = {

lemmy-ui is an example of just that.

Yeah, setting it in the package, since it's immutable, would usually mean rebuilding the package each time it changes I imagine. Although, for now, I've just been trying to figure out how to get it working in the first place. I am not exactly sure at what point the NPM bootstrapping process injects the environment in, or how -- and I don't even know if it's literal environment variables or something else. I was just playing around to see if I could start with a working state, and then have something to compare to my broken state to hint as to what's going wrong.
(It's also notable that there are some oddities -- like, if I recall properly, the Wordpress package does bake the config.php in with the package. I was actually considering making a PR to have that be a symlink to a data directory, which is then optionally either separately managed by a different, smaller package -- or managed imperatively if the user likes to roll that way.)

completely understand, thank you for your efforts in trying to fix this!


Responsive Footer

Description

Make some necessary changes to make footer responsive.
Robotics club section can be moved to the right side, logo and social networks section can be aligned to the center (You can also use your alignment to make it better) .

I am interested. plz assign it to me

Hi! Can you assign it to me?

@Avtech04 I am facing some issues in setting up the project.
RuntimeWarning: Pillow 8.1.0 does not support Python 3.12 and does not provide prebuilt Windows binaries. We do not recommend building from source on Windows. which python version should i use?

@chow870 Try Python 3.9.13

@Avtech04 these are the changes that I have made.

Are these okay? any further changes?

In the first image you have attached, i guess its full screen size. So why its different from current website UI. and it is not looking good in first image. (Useful links and Robotics Club section)

Sure, I shall do it today itself.
On Wed, 9 Oct 2024, 12:20 Ayush Varshney, @.***> wrote:

In the first image you have attached, i guess its full screen size.
So why its different from current website UI. and it is not looking good in
first image. (Useful links and Robotics Club section).

image.png (view on web)
https://github.com/user-attachments/assets/01b945fe-163a-44f6-80db-1cc1e8e0d988

In the last image, all links under useful links should start from
same vertical line, which is currently not.

image.png (view on web)
https://github.com/user-attachments/assets/c5c5938c-87c8-4a3b-a665-7f2bf867e400
Complete it by today itself, else i will assign it to someone else.
@chow870 https://github.com/chow870
—
Reply to this email directly, view it on GitHub
https://github.com/ContriHUB/Robotics_Club_Website/issues/3#issuecomment-2401317137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD22NR2E5FMNDOWKSLYNQ4TZ2TG5BAVCNFSM6AAAAABPPFRJHSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBRGMYTOMJTG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Avtech04

Ok, Now its fine!

Ok, Now its fine!
where should I generate PR?

I guess your completion time is finished,  make request again !

it says :

it says :

Ask abhishek and tell you issue on discord, he will resolve the issue.

@Avtech04 i shall send the request again in some time today itself. Plz dont assign this project to anyone else in the meantime

Please assign this issue to me

@Avtech04 i have sent you the request. Plz accept it. I will create the PR asap.

@Avtech04 I am getting this error while creating the PR :

i have followed all the steps of setting the project properly yet cant figure out, why am i getting this issue.

Hey you can`t push to the main branch right? You have to push your code in your branch, and then make a PR for the same.

@Avtech04 I again got the same issue even though I had created my own branch and had pushed my code in that branch.

@Avtech04 I had also tried with forking and then cloning. yet got the same issue

@Avtech04 I had also tried with forking and then cloning. yet got the same issue

I have DM you on discord. Share detailed query there.


Option dialogClass → crash

http://plnkr.co/edit/XXFKwHHcZD4Ecjh8twaW?p=preview — official demo

Uncomment line 41 (dialogClass)
Uncaught TypeError: Cannot read property 'innerHTML' of null

:(

It works only when using an existing class. Demo fixed now.


[ci skip] adding user @adrianinsaval

Hi! This is the friendly automated conda-forge-webservice.
I've added user @adrianinsaval as instructed in #70.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #70

@conda-forge/core please merge


The application isn't for Shared Hosting (access to ssh with composer command)

I did everything according to the instructions (https://github.com/domrafacz/website-monitor/blob/master/docs/install.md) "Traditional installation" - I didn't do "yarn install" and "yarn build" because these commands are not allowed to be used by ssh on my hosting account. The application installed correctly, this is confirmed by:
$ composer dump-autoload
Generating optimized autoload files
Generated optimized autoload files containing 6950 classes

After entering the address in the browser: https://xyz.mydomain.tld/ I was automatically redirected to the page:
https://xyz.mydomain.tld/en/login
where is a 404 error.
The app probably has an error creating correct paths.
In my case I unpacked the app (.zip ) in the ../public_html/ directory, and in Apache I indicated that the default directory for my domain: https://xyz.mydomain.tld/ is ../public_html/public
I'm sad that I couldn't test the app. I also tried to contact Mr. Dominik (Panem Dominikiem) creator of the application but without success.

Thanks for your quick reply. I don't have access to the command:
nvm
My hosting account only allow:
https://wiki.hitme.pl/pomoc/hosting-www/php/hosting-php-composer-jak-zaczac/

My .env file
ALLOW_PRIVATE_NETWORKS=0
TRUSTED_IPS='localhost'
DEFAULT_LANGUAGE=en
APP_ENV=dev
APP_SECRET=06f2***48
MESSENGER_TRANSPORT_DSN=doctrine://default?auto_setup=0
DATABASE_URL='mysql://xyzuser:abcdefwww@localhost:3306/xyzdb_1'

The app still should run without building assets, because they are already bundled in public_html directory. Docker installation runs without building assets just fine, so I don't see any reason why app shouldn't run on shared hosting.
Installation guide is missing command that you have to run
php bin/console doctrine:migrations:migrate
You can try to also share logs or screenshot

Yes I use MariaDB:

I see that your last command allowed you to create tables in the Database. Unfortunately, when I go to the page address, I still get a 404 error:

They are located in build directory, later i will try install on shared hosting, you can also check logs in var/log directory.
Maybe there is some problem with apache configuration

@domrafacz Thank You for your support. I wrote you an e-mail from<EMAIL_ADDRESS>and on facebook. I can send you my apache server logs on e-mail or on fb, ok?

@domrafacz Are You planning on working on the problem today? I could help with testing Your app, which is a great idea (it's one of the few LAMP stack apps that monitor websites/services for logged in users)

I was able to reproduce the issue, it was cause by lack of proper .htaccess file since, the project was primary developed to run on nginx and caddy. I pushed fix that should resolve this problem.

Thank you for the information. I will start testing and edit this message with the test results.

In my free time I created a logo (vector graphics) for Your project.

If You're satisfied with this graphic, let me know how to send it to You.
This topic can be closed

I'm not sure if you're interested in the graphic, so I've included it below for your reference. I'm closing this report. Thank you for your assistance 🥇
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<!--


enter: pass nsactions array rather than path array

Before this commit, we used to construct shared namespace paths purely from string operations; that is, --share pid,mnt=/foo would be reconstructed as --share pid=/foo/pid --share mnt=/foo/mnt. The problem with this approach is that this doesn't work well with magic links to file descriptors in /proc.
Instead, we now open the directory path and use its file descriptor to open further nsfs files.

I tried using the auto-merge but it looks like it broke the build. I have a fix locally but I'm not authorized to push it to Snaipe's fork. Probably better to use a rebase than a merge commit anyways.