changed ClientSSLSecurity to ClientSSLSecurityPFX in the readme file

Constructor name is wrong, corrected it.

Coverage remained the same at 93.587% when pulling f0b313b69d592f1eedd263204bdef7e6e6f5b189 on nocubicles:master into fbacc867213fe501fb14efdd775c3e25ee5d12ec on vpulim:master.

Coverage remained the same at 93.587% when pulling f0b313b69d592f1eedd263204bdef7e6e6f5b189 on nocubicles:master into fbacc867213fe501fb14efdd775c3e25ee5d12ec on vpulim:master.

Coverage remained the same at 93.587% when pulling f0b313b69d592f1eedd263204bdef7e6e6f5b189 on nocubicles:master into fbacc867213fe501fb14efdd775c3e25ee5d12ec on vpulim:master.

Thanks!


added test case for directory

This test case generates a file and then reads that file and passess based on the contents of that file. The problem is that if that test passes, the file is generated but never deleted. In future runs, if the generation of the file fails (and the old one is still there), the test will still pass.
Need to have cleaning up for test cases both before the test runs and after it finishes to make sure that any test articiacts are cleaned up.
May also be good to add these artifacts to the gitignore list so that they don't find their way into git.

May be better to use something like this: test https://stackoverflow.com/questions/22627659/run-code-before-and-after-each-test-in-py-test


Can't create installer on 0.36.0, please fix!

My configuration:
'create-windows-installer': {
      appDirectory: _path.join('C:', 'projects', 'desktop', 'build', 'releases', 'win32', 'ia32', 'HackHands-win32'),
      outputDirectory: _path.join('C:', 'projects', 'desktop'),
      authors: 'HackHands',
      setupIcon: _path.join('C:', 'projects', 'desktop', 'fixtures', 'icon.ico'),
      remoteReleases: 'http://desktop.hackhands.com.s3-website-us-west-1.amazonaws.com/win/'+channel,
      loadingGif:  _path.join('C:', 'projects', 'desktop', 'fixtures', 'hh-logo-process.gif')
      //exe: 'HackhandsSetp.exe'
    },

And the output:
[00:02:56] grunt installer --verbose
[00:02:56] Initializing
[00:02:56] Command-line options: --verbose
[00:02:56]
[00:02:56] Reading "Gruntfile.js" Gruntfile...OK
[00:02:56]
[00:02:56] Registering Gruntfile tasks.
[00:02:56] Channel: nightly
[00:02:56]
[00:02:56] Registering "grunt-aws" local Npm module tasks.
[00:02:56] Reading C:\projects\desktop\node_modules\grunt-aws\package.json...OK
[00:02:56] Parsing C:\projects\desktop\node_modules\grunt-aws\package.json...OK
[00:02:56]
[00:02:56] Registering "C:\projects\desktop\node_modules\grunt-aws\tasks\services" tasks.
[00:02:56] Loading "cloudfront.js" tasks...OK
[00:02:56] + cloudfront
[00:02:56] Loading "route53.js" tasks...OK
[00:02:56] + route53
[00:02:56] Loading "s3.js" tasks...OK
[00:02:56] + s3
[00:02:56] Loading "aws.js" tasks...OK
[00:02:56] >> No tasks were registered or unregistered.
[00:02:56] Loading "cache-mgr.js" tasks...OK
[00:02:56] >> No tasks were registered or unregistered.
[00:02:56]
[00:02:56] Registering "grunt-contrib-clean" local Npm module tasks.
[00:02:56] Reading C:\projects\desktop\node_modules\grunt-contrib-clean\package.json...OK
[00:02:56] Parsing C:\projects\desktop\node_modules\grunt-contrib-clean\package.json...OK
[00:02:56] Loading "clean.js" tasks...OK
[00:02:56] + clean
[00:02:56]
[00:02:56] Registering "grunt-contrib-coffee" local Npm module tasks.
[00:02:56] Reading C:\projects\desktop\node_modules\grunt-contrib-coffee\package.json...OK
[00:02:56] Parsing C:\projects\desktop\node_modules\grunt-contrib-coffee\package.json...OK
[00:02:56] Loading "coffee.js" tasks...OK
[00:02:56] + coffee
[00:02:56]
[00:02:56] Registering "grunt-contrib-compress" local Npm module tasks.
[00:02:56] Reading C:\projects\desktop\node_modules\grunt-contrib-compress\package.json...OK
[00:02:56] Parsing C:\projects\desktop\node_modules\grunt-contrib-compress\package.json...OK
[00:02:56] Loading "compress.js" tasks...OK
[00:02:56] + compress
[00:02:56]
[00:02:56] Registering "grunt-contrib-copy" local Npm module tasks.
[00:02:56] Reading C:\projects\desktop\node_modules\grunt-contrib-copy\package.json...OK
[00:02:56] Parsing C:\projects\desktop\node_modules\grunt-contrib-copy\package.json...OK
[00:02:56] Loading "copy.js" tasks...OK
[00:02:56] + copy
[00:02:56]
[00:02:56] Registering "grunt-contrib-less" local Npm module tasks.
[00:02:56] Reading C:\projects\desktop\node_modules\grunt-contrib-less\package.json...OK
[00:02:56] Parsing C:\projects\desktop\node_modules\grunt-contrib-less\package.json...OK
[00:02:57] Loading "less.js" tasks...OK
[00:02:57] + less
[00:02:57]
[00:02:57] Registering "grunt-contrib-sass" local Npm module tasks.
[00:02:57] Reading C:\projects\desktop\node_modules\grunt-contrib-sass\package.json...OK
[00:02:57] Parsing C:\projects\desktop\node_modules\grunt-contrib-sass\package.json...OK
[00:02:57] Loading "sass.js" tasks...OK
[00:02:57] + sass
[00:02:57]
[00:02:57] Registering "grunt-contrib-watch" local Npm module tasks.
[00:02:57] Reading C:\projects\desktop\node_modules\grunt-contrib-watch\package.json...OK
[00:02:57] Parsing C:\projects\desktop\node_modules\grunt-contrib-watch\package.json...OK
[00:02:57] Loading "watch.js" tasks...OK
[00:02:57] + watch
[00:02:57]
[00:02:57] Registering "grunt-electron" local Npm module tasks.
[00:02:57] Reading C:\projects\desktop\node_modules\grunt-electron\package.json...OK
[00:02:57] Parsing C:\projects\desktop\node_modules\grunt-electron\package.json...OK
[00:02:57] Loading "electron.js" tasks...OK
[00:02:57] + electron
[00:02:57]
[00:02:57] Registering "grunt-electron-installer" local Npm module tasks.
[00:02:57] Reading C:\projects\desktop\node_modules\grunt-electron-installer\package.json...OK
[00:02:57] Parsing C:\projects\desktop\node_modules\grunt-electron-installer\package.json...OK
[00:02:57] Loading "index.js" tasks...OK
[00:02:57] + create-windows-installer
[00:02:57]
[00:02:57] Registering "grunt-mkdir" local Npm module tasks.
[00:02:57] Reading C:\projects\desktop\node_modules\grunt-mkdir\package.json...OK
[00:02:57] Parsing C:\projects\desktop\node_modules\grunt-mkdir\package.json...OK
[00:02:57] Loading "mkdir.js" tasks...OK
[00:02:57] + mkdir
[00:02:57]
[00:02:57] Registering "grunt-shell" local Npm module tasks.
[00:02:57] Reading C:\projects\desktop\node_modules\grunt-shell\package.json...OK
[00:02:57] Parsing C:\projects\desktop\node_modules\grunt-shell\package.json...OK
[00:02:57] Loading "shell.js" tasks...OK
[00:02:57] + shell
[00:02:57]
[00:02:57] Registering "tasks" tasks.
[00:02:57] Loading "install-task.coffee" tasks...OK
[00:02:57] + install
[00:02:57] Loading "install-task.js" tasks...OK
[00:02:57] + install
[00:02:57] Loading "mkdeb-task.coffee" tasks...OK
[00:02:57] + mkdeb
[00:02:57] Loading "mkdeb-task.js" tasks...OK
[00:02:57] + mkdeb
[00:02:57] Loading "mkrpm-task.coffee" tasks...OK
[00:02:57] + mkrpm
[00:02:57] Loading "mkrpm-task.js" tasks...OK
[00:02:57] + mkrpm
[00:02:57] Loading "task-helpers.coffee" tasks...OK
[00:02:57] >> No tasks were registered or unregistered.
[00:02:57] Loading "task-helpers.js" tasks...OK
[00:02:57] >> No tasks were registered or unregistered.
[00:02:57] Initializing config...OK
[00:02:57] Loading "Gruntfile.js" tasks...OK
[00:02:57] + build, compile, dist, dist-linux, dist-mac, installer, installer-beta, installer-stable, linux-generic, release-mac
[00:02:57]
[00:02:57] Running tasks: installer
[00:02:57]
[00:02:57] Running "installer" task
[00:02:57]
[00:02:57] Running "create-windows-installer" task
[00:02:57]
[00:02:57] Running "create-windows-installer:appDirectory" (create-windows-installer) task
[00:02:57] Verifying property create-windows-installer.appDirectory exists in config...OK
[00:02:57] Files: C:/projects/desktop/build/releases/win32/ia32/HackHands-win32 -> appDirectory
[00:02:57] Verifying property create-windows-installer.appDirectory.appDirectory exists in config...OK
[00:02:57] Warning: Arguments to path.resolve must be strings Use --force to continue.
[00:02:57]
[00:02:57] Aborted due to warnings.
[00:02:57] Command exited with code 3

On 0.36.0 you need to adjust your config to be:
'create-windows-installer': {
  config: {
      appDirectory: _path.join('C:', 'projects', 'desktop', 'build', 'releases', 'win32', 'ia32', 'HackHands-win32'),
      outputDirectory: _path.join('C:', 'projects', 'desktop'),
      authors: 'HackHands',
      setupIcon: _path.join('C:', 'projects', 'desktop', 'fixtures', 'icon.ico'),
      remoteReleases: 'http://desktop.hackhands.com.s3-website-us-west-1.amazonaws.com/win/'+channel,
      loadingGif:  _path.join('C:', 'projects', 'desktop', 'fixtures', 'hh-logo-process.gif')
      //exe: 'HackhandsSetp.exe'
    }
  }

This is related to the change made in #43

I'm sorry but in the case of incompatibility the release should be a major, right? A release as minor can break all the users using the previous version.

I'm sorry but in the case of incompatibility the release should be a major, right?

Only if a major version has already been published, pre-1.0 libraries should not be considered stable between releases.

I see. What you think to do about this incompatibility?

What you think to do about this incompatibility?

You can either stay on version 0.35 or upgrade to 0.36 and update your config, you don't have to upgrade if you don't want to, just stay on 0.35 until you are ready to update.

Okay, thank you!


scrolling-alternative

[x] While scrolling through an image just for editing, a single click on the image pops up a dialog box for image marking, even when image marking is not needed. Please add a specific button for image marking.

1.Once the image part is zoomed in, the image part does not remain stable while moving the cursor.
2.while we single click on the image it continues showing add image table equation tab

we added a button functionality for this.
Resolved in v2.5.1


The 30 day of the example code is wrong,

such as router undefined, authorize_url misspelled.

Greetings,
Thanks for reaching out....
Please mention the article number and we will look into it.
cheers,
team nodejsera

number：30
url：https://www.nodejsera.com/nodejs-tutorial-day30-github-api-with-node.html

Greetings,
Thanks for sharing the details.
We will look into the matter.
Regards,
Team Nodejsera

Greetings,
The issues are resolved.
Please Feel free to check.
Regards,
Team Nodejsera

the code is still not working


Fix conflict errors for impersonation cluster role

Without this patch, requests made on a downstream cluster through the
impersonation proxy can sometimes fail when the impersonation cluster
role fails to update. This is more likely to happen when there are a
high volume of requests. This change addresses the issue in two ways:

Reduce the need to update the cluster role by ensuring that the
resources in the role rules are sorted so that equality comparison is
meaningful.
Retry the update when a conflict is encountered so that a single
conflict error isn't fatal to the request.

https://github.com/rancher/rancher/issues/34671

Added a commit with tests, can either squash it, leave it separate or leave it out for now.

Also, I see we still have 2 commits, so I'm not sure if you're planning to squash them at merge time automatically or decided to keep them separate.

They should stay as 2 commits, one for the changes and one for the tests.


How to do endless?

How to do endless? I don't want to show all activities. This is a very long list. There is a way to make the auto loading of the rest of the parts?

This is beyond public_activity's scope. Google "infinite scroll rails".


Add Support to Donut for Document AI

Add support to Clova AI Donut for real-time document analysis, enhancing Document AI capabilities on Groq Cloud https://huggingface.co/naver-clova-ix/donut-base

@KaifAhmad1 Hi there! 👋 Thanks for the feature request. If you haven't already, please join our developer community on Discord to submit feature requests in the #feature-requests channel so that others can also upvote ones they'd like to see us support! We triage that channel and pass on popular requests to our Product and Engineering teams.


No output from the first code

I'm trying the first code in the getting started docs, but get no output:
package main
import (
        "fmt"
        "log"

        "gorgonia.org/gorgonia"
)

func main() {
        g := gorgonia.NewGraph()

        var x, y, z *gorgonia.Node
        var err error

        // define the expression
        x = gorgonia.NewScalar(g, gorgonia.Float64, gorgonia.WithName("x"))
        y = gorgonia.NewScalar(g, gorgonia.Float64, gorgonia.WithName("y"))
        if z, err = gorgonia.Add(x, y); err != nil {
                log.Fatal(err)
        }

        // create a VM to run the program on
        machine := gorgonia.NewTapeMachine(g)
        defer machine.Close()

        // set initial values then run
        gorgonia.Let(x, 2.0)
        gorgonia.Let(y, 2.5)
        if err = machine.RunAll(); err != nil {
                log.Fatal(err)
        }

        fmt.Printf("%v", z.Value())
}

At the first build that I got this panic message
panic: Something in this program imports go4.org/unsafe/assume-no-moving-gc to declare that it assumes a non-moving garbage collector, but your version of go4.org/unsafe/assume-no-moving-gc hasn't been updated to assert that it's safe against the go1.18 runtime. If you want to risk it, run with environment variable ASSUME_NO_MOVING_GC_UNSAFE_RISK_IT_WITH=go1.18 set. Notably, if go1.18 adds a moving garbage collector, this program is unsafe to use.
So I ran
get -u  go4.org/unsafe/assume-no-moving-gc
and then go mod tidy but now there is no output at runtime. Nor do I get any errors in build or runtime.
How can I fix this?

This looks like a version issue

#523


[BUG] VM created from windows-iso-image template can't start automatically, need the extra manual start effort

Describe the bug

When create a windows virtual machine from windows-iso-image template.
The virtual machine can't start automatically and stay in the off
start virtual machine default/windows-template failed, persistentvolumeclaims "windows-template-cdrom-disk-rvk33" not found

User need to manual start the windows VM to make it running.
To Reproduce
Steps to reproduce the behavior:

Create a windows iso image from file or upload via URL
https://storage.googleapis.com/harvester/windows_server_2019_essentials.iso
Create a VM , select windows-iso-image-base-version template
Edit the appropriate CPU and memory

Select windows iso image with type cd-rom

Click the create button

Expected behavior

VM created from windows-iso-image template can start automatically without error
Support bundle

supportbundle_28479d0f-c787-4ec6-b759-673c63ce6c76_2022-04-21T11-38-33Z.zip
Environment:

Harvester ISO version: master-03ee6213-head (04/18)
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): 1 node harvester on local kvm machine

Additional context
Add any other context about the problem here.

It's not a problem that happens 100% of the time.

@WuJun2016 Could you help us to modify the code for starting VM on creation?
We might need to check the VM CR is created and send start request after that.
thanks a lot

Verified this bug has been fixed.
UI change:

Remove Start virtual machine on creation
Set Run Strategy to RerunOnFailure by default in Advanced Options

Test Information

Environment: qemu/KVM single node
Harvester Version: v1.0.2-rc2
ui-source Option: Auto

Verify Steps:

Follow Steps to reproduce in https://github.com/harvester/harvester/issues/2172#issue-1210911515


Fix typo

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #28   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines        1396   1396
=====================================
  Hits         1396   1396

Impacted Files
Coverage Δ

src/reference.js
100% <ø> (ø)
:arrow_up:

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 eb6737f...c43cc2d. Read the comment docs.


Add exemplars for ochttp plugin similar to ocgrpc plugin.

The ocgrpc plugin records with an exemplar but the ochttp does not.

Will be implemented in Opentelemetry-Go. https://github.com/open-telemetry/opentelemetry-go/issues/559


Improve timeout setting

While adding the config_flow updates in Core Plugwise, I realized we always start with a timeout of 30 secs, so testing a fail takes a long time. Then I thought, we know the password, only if equal to stretch set the timeout to 30 :)

Summary by CodeRabbit

New Features

Introduced improved timeout settings: 30 seconds for the Stretch device and 10 seconds for all other devices.

Bug Fixes

Adjusted timeout assertions in tests to reflect new logic based on username conditions.

Refactor

Simplified constructors by removing the timeout parameter across multiple classes.
Updated connection logic to eliminate timeout-related functionality.

Chores

Updated project version to 1.4.2a0 in the configuration files.
Revised changelog to reflect the new version and improvements.

Closed, not implemented. A better solution has been implemented in the frontend Integration.


Enhance ResGen tool to support a command line argument

Related #3400
Motivation
ResGen tool process all resx files in PowerShell Repo at a time. It is not problem because it works very fast. But updating multiple cs files causes all Repo projects to be recompiled. It really slows down.
We still can not use native resgen from dotnet-cli. See https://github.com/PowerShell/PowerShell/issues/2882#issuecomment-304300044
So we should enhance our ResGen tool, use MSBuild capabilities and process resx files one at a time to avoid unnecessary rebuilds.
Fix
We can specify a resx file path parameter on the command line and process resx files one at a time.
Follow-Up work
We will clean up Build.psm1 and modify csproj files to use the new feature of ResGen tool with MSBuild for faster project build.

I think it's a good idea.

We will clean up Build.psm1 and modify csproj files to use the new feature of ResGen tool for faster project build.

Couple comments: I don't think we should try to be smart and auto-detect set of files for resgen in build.psm1. We should just figure out the way to switch to msbuild resgen sooner or later.
Let's just expose this functionality thru Start-ResGen optional parameter in the same PR.

Yes, it is all for MSBuild - I updated the comment.

oh, do you mean that you know how to plug it into csproj files so it's called automatically?

Yes. You can see sample in #3870

@TravisEz13 Is the PR ready to merge?

LGTM except for a minor comment.


Can i navigate to other page with a button outside the navigation sheet?

Hi this is not an issue just a quick question, can i somehow navigate to the second page by a button which exist outside the navigation sheet? for example this code from your example/lib/tutorial/declarative_navigation_sheet.dart, i want that _ExampleSheet react to the button which outside the navigation sheet, thanks for the great package
class _ExampleHome extends StatelessWidget {
  const _ExampleHome({
    required this.nestedNavigator,
  });

  final Widget nestedNavigator;

  @override
  Widget build(BuildContext context) {
    return Stack(
      children: [
        const Scaffold(
body: ElevatedButton(onPressed: () => 'go to the second page', child: Text('button'))
),
        _ExampleSheet(nestedNavigator: nestedNavigator),
      ],
    );
  }
}
}

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file password: changeme In the installer menu, select "gcc."

nice try


Add tests for parse functions

This PR should be merged after #7 gets in.

Moved to the new repo: https://github.com/MCGallaspy/ka-lite/pull/1


SHW model: no way to get cost of auxiliary energy

It seems that there is no way to get an output variable in SAM for the dollars per year spent on auxiliary energy in the solar hot water model. You can get cost with system, cost without system, etc., but these include pumping energy, etc. All I want to know is how much money is being spent on auxiliary boosting of the tank?

Hi Ben, it's been some time since I've looked at this module.  But I think the "electricity bill" with system is essentially equal to the money spent on auxiliary boosting of the tank. You could set the pumping efficiency to 100% and a very low power as well.


Dockerfile

Automating builds would make TrueCraft more accessible. Mono has a docker image that automatically builds and runs projects.
Ex. Dockerfile (in project root):
FROM mono:4.0.0-onbuild
EXPOSE 25565
CMD [ "mono",  "./TrueCraft.exe" ]

Build with:
docker build -t truecraft .

or, if you want to tag the image with the git commit id:
TAG=`git log --format="%H" -n 1 | cut -c1-10`; docker build -t "truecraft:$TAG" .

Run with:
docker run -it --rm -p 25565:25565 truecraft

I agree that a Dockerfile would be useful, but I don't want to maintain one. Are you interested in taking on that responsibility?

I can do that. https://github.com/mikenon/truecraft-dockerfile is what I've got.

Cool. How do you edit the config.yaml?

Ah, yep, that's an issue. I'll get it sorted.

Okay, I think I've come up with a better way for handling things.

Nice, looks good. What's next?


Whoops json error response uses status code 200

Hi,
I noticed that the Whoops json error response uses status code 200 when returning error.
Shouldn't it be 500 or something like that instead?

Yes, it should. I had assumed that was something done by Zend\Stratigility\Middleware\ErrorHandler, but, in fact, it's done within the error response generators. We have logic to do that in the production error response generator, but it's missing from the whoops variant.
Essentially, all we need to do is add an import statement:
use Zend\Stratigility\Utils;

and one line within __invoke():
$response = $response->withStatus(Utils::getStatusCode($e, $response));

If somebody would like to tackle this, adding unit tests to verify, I'd appreciate it!

@weierophinney I was just looking at this.

Fixed with #476


Exception when importing pyserini

Hi I follow the instructions of pyserini to set up the environment and run the example script.
The exception I have when running teh example pyserini script copied from readme is:
Exception                                 Traceback (most recent call last)
<ipython-input-18-fed52f7bef12> in <module>()
----> 1 from pyserini.search import pysearch
      2
      3 searcher = pysearch.SimpleSearcher('lucene-index.robust04.pos+docvectors+rawdocs')
      4
      5 # To additionally configure search options, such as using BM25+RM3:

2 frames
/content/anserini/src/main/python/pyserini/setup.py in configure_classpath(anserini_root)
     35                                    'anserini-*-fatjar.jar'))
     36     if not paths:
---> 37         raise Exception('No matching jar file found in target')
     38
     39     latest = max(paths, key=os.path.getctime)

Exception: No matching jar file found in target

I installed and ran it on colab, python 3.

Did you follow the instructions to run and build the jar? https://github.com/castorini/anserini#getting-started

Did you follow the instructions to run and build the jar? https://github.com/castorini/anserini#getting-started

I follow the demo colab notebook to set up the environment.
To reproduce it, I have create a notebook here.
https://colab.research.google.com/drive/1gAtadmMw-4hAsYM-YmM-q2VJR6vnTquT

Check out: https://colab.research.google.com/drive/1GOCyWyYW_fwJMKy5FVWdYHEC2IlvpqZN
Reopen issue if you're still having problems.


bug fixed allowing Body to be specified with omega

For a body, if omega is provided, now just self.pomega is set, leaving omega as a derived property.

Oh good catch! A few comments inline.


#27 fixed

made some UI changes.

Hey @Dishi-Gautam, could you please mention the issue tag that you resolved?

add this in the description


CSV writer

A community user asked:

With some non-Zeek NDJSON data sources we turn into ZNG and store, we also may want to write to a SQL db (JSON works but prefer CSV) to populate visualizations of the data in say like an Apache Superset (which I actually have plans to integrate for Zeek data, probably back out in Zeek JSON from ZNG).

I think the CSV writer should handle heterogenous records, though I assume that'd require writing to a temporary file to build the union of all the record types. That may be more than what the community user asked for, if their target was a sql db import.

@alfred-landrum, agreed.  I will write up a new issue for this.  There are a couple ways to do it.

We also need to change the search endpoint with csv output (as part of the larger refactoring there) so the front-end doesn't have to have its own csv writer and record unflattener in javascript.

Verified in zq commit 4bce00d.
Output as CSV currently works for any data that can be represented by a single "descriptor" (i.e. they all have the same record type). For instance, the contents of a single Zeek log:
$ zq -f csv stats.log.gz
_path,ts,peer,mem,pkts_proc,bytes_recv,pkts_dropped,pkts_link,pkt_lag,events_proc,events_queued,active_tcp_conns,active_udp_conns,active_icmp_conns,tcp_conns,udp_conns,icmp_conns,timers,active_timers,files,active_files,dns_requests,active_dns_requests,reassem_tcp_size,reassem_file_size,reassem_frag_size,reassem_unknown_size
stats,2018-03-24T17:15:20.600725Z,zeek,74,26,29375,-,-,-,404,11,1,0,0,1,0,0,36,32,0,0,0,0,1528,0,0,0
stats,2018-03-24T17:20:20.60102Z,zeek,281,6435390,3052165142,-,-,-,2575313,2575317,5818,187,1070,338620,5126,9490,1301330,33086,48528,27,61,0,77448,53168,0,0
stats,2018-03-24T17:25:20.601054Z,zeek,282,4556854,1920350471,-,-,-,1958516,1958507,4401,110,1010,211123,4600,2354,1006374,27351,41930,10,5,0,90752,0,1672,0
stats,2018-03-24T17:30:20.601101Z,zeek,281,8112284,3883978461,-,-,-,1586683,1586685,2795,137,648,207754,4666,31554,928834,19429,29547,26,3,0,126712,395584,1504,0
stats,2018-03-24T17:35:20.601137Z,zeek,282,5467567,3398705931,-,-,-,1535999,1535998,4239,146,305,193639,4731,2510,879701,25895,35230,88,6,0,455128,0,0,0

It's then trivial to put this output into my paste buffer and, say, enter it into a Google Sheet and select Data > Split Text To Columns.

If the data trying to be output requires multiple descriptors, the user will receive an error message and the output will stop when the first record of a different type is encountered. For example, if you try to output two Zeek logs at once via CSV:
$ zq -f csv stats.log.gz weird.log.gz
_path,ts,peer,mem,pkts_proc,bytes_recv,pkts_dropped,pkts_link,pkt_lag,events_proc,events_queued,active_tcp_conns,active_udp_conns,active_icmp_conns,tcp_conns,udp_conns,icmp_conns,timers,active_timers,files,active_files,dns_requests,active_dns_requests,reassem_tcp_size,reassem_file_size,reassem_frag_size,reassem_unknown_size
stats,2018-03-24T17:15:20.600725Z,zeek,74,26,29375,-,-,-,404,11,1,0,0,1,0,0,36,32,0,0,0,0,1528,0,0,0
csv output requires uniform records but different types encountered

However, we do have a plan to address this case as well, and that's tracked in #1271.
Finally, I noticed that csv is not yet listed in the -f output formats in the zq help text, so #1300 tracks getting that added.
Thanks @mccanne!


Cursor changed to pointer

Changes proposed
cursor changed to pointer
Screenshots

@bishal7679, it is always recommended to first open an issue & then make the necessary PR for that! (even if its a small one)
Please follow this approach for your next contributions
I hope you understand :)

Thanks for contributing @bishal7679 !!

@verma-kunal extremely sorry for that. Actually I have created 2 issue and by mistake without creating the third issue I send the PR. Thanks a lot for reminding me.


Consistent "SSH: Temporary failure in name resolution" when attempting to utilize docker plugin

Attempting to utilize the Docker plugin is resulting in the following errors no matter what I seem to try.
Error logs
PLAY [Converge] ****************************************************************

TASK [Include webserver config role] *******************************************

TASK [my_role_name : Debug] *****************************************************
ok: [rhel-9] => {
    "msg": "debug from tasks/main.yml"
}

TASK [my_role_name : ansible.builtin.include_vars] ******************************
ok: [rhel-9]

TASK [my_role_name : Create apps folder] ****************************************
fatal: [rhel-9]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh: ssh: Could not resolve hostname rhel-9: Temporary failure in name resolution", "unreachable": true}

I have reproduced this error in both my local workstation and in CI/CD. So this is either something with molecule/the plugins or my configuration somewhere. I have tried various roles to ensure it's not something specific to the role I am testing.
converge.yml
- name: Converge
  hosts: all
  gather_facts: false
  tasks:
    - name: Include webserver config role
      ansible.builtin.include_role:
        name: my_role_name
        tasks_from: main.yml

molecule.yml
prerun: false
dependency:
  name: galaxy
driver:
  name: docker
lint: |
  yamllint .
  ansible-lint .
platforms:
  - name: rhel-9
    image: redhat/ubi9:9.1.0
    # platform: aarch64
    dockerfile: Dockerfile.j2
    privileged: true
    cgroupns_mode: host
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:rw
    command: /usr/sbin/init
provisioner:
  name: ansible
  log: true

Versions
ansible [core 2.15.5]
molecule 6.0.2 using python 3.10
    ansible:2.15.5
    azure:23.5.0 from molecule_plugins
    containers:23.5.0 from molecule_plugins requiring collections: ansible.posix>=1.3.0 community.docker>=1.9.1 containers.podman>=1.8.1
    default:6.0.2 from molecule
    docker:2.1.0 from molecule_docker requiring collections: community.docker>=3.0.2 ansible.posix>=1.4.0
    ec2:23.5.0 from molecule_plugins
    gce:23.5.0 from molecule_plugins requiring collections: google.cloud>=1.0.2 community.crypto>=1.8.0
    podman:23.5.0 from molecule_plugins requiring collections: containers.podman>=1.7.0 ansible.posix>=1.3.0
    vagrant:23.5.0 from molecule_plugins
Docker version 24.0.7, build afdd53b

@patrickblackjr
Could you please provide the Dockerfile.j2?
And have you tried other docker image?


Ignores line breaks (\n)

Hi, thanks for releasing this great streamlit extension. I found the link breaks (\n) in the message are ignored. Can you please make the message display the line breaks?

yes，i also found it，

I've created a pull request to fix the line breaks (\n) problem #23


React & Redux group project - Space Travelers' Hub

Use React documentation.
Use React components.
Use React props.
Use React Router.
Connect React and Redux.
Handle events in a React app.
Write unit tests with React Testing Library.
Use styles in a React app.
Use React life cycle methods.
Apply React best practices and language style guides in code.
Use store, actions, and reducers in React.
Perform a code review for a team member.

@KDlamini thank you for your revision, it is an Optional suggestion. Why you did not approve the project?

@KDlamini thank you for your revision, it is an Optional suggestion. Why you did not approve the project?

Hey @franclobo,
My deepest apologies. I have no idea how missed that.
Kindly ignore my review and request another review.
I apologize for any inconvenience on my part.

TO NEXT REVIEWER:
Please ignore my required change. It was an optional suggestion that I missed by mistake.
Thank you.


Single quotes in attributes lead to bash syntax error in knife bootstrap

If the initial attributes used in with knife bootstrap contain an apostrophe / single quote, then a bash syntax error is throw when chef client is started on the remote machine.
Running with a "nice" set of attributes. Bootstrap connects to the node and tries to trigger the recipe (which fails as expected as the chef server isn't running):
$ knife bootstrap host-redacted -j "{\"x\":\"y\"}"
Bootstrapping Chef on host-redacted
host-redacted Starting Chef Client, version 12.1.2
host-redacted [2015-09-03T02:11:37-07:00] ERROR: Connection refused connecting to http://<IP_ADDRESS>:8889/nodes/host-redacted, retry 1/5
ˆC

Then, running the same thing with an apostrophe in a valid location:
$ knife bootstrap host-redacted -j "{\"x\":\"y'\"}"
Bootstrapping Chef on host-redacted
host-redacted bash: -c: line 80: unexpected EOF while looking for matching `"'
host-redacted bash: -c: line 84: syntax error: unexpected end of file

Spotted in versions 11.8.2 and 12.1.2.

It looks like the issue may be with the lack of single quote escaping when generating the template at https://github.com/chef/chef/blob/master/lib/chef/knife/bootstrap.rb#L390 as the default template (https://github.com/chef/chef/blob/master/lib/chef/knife/bootstrap/templates/chef-full.erb) uses single quotes.
This opens up for code injection attacks:
$ knife bootstrap localhost -j "{\"a\": \"b'; echo Hello there; pwd; '\"}"
Doing old-style registration with the validation key at /root/.chef/chef-validator.pem...
Delete your validation key in order to use your user credentials instead

Connecting to localhost
localhost bash: line 80: warning: here-document at line 79 delimited by end-of-file (wanted `EOP')
localhost Hello there
localhost /root
localhost bash: line 85: ","run_list":[]}
localhost EOP
localhost
localhost echo "Starting first Chef Client run..."
localhost
localhost chef-client -j /etc/chef/first-boot.json -E _default: No such file or directory

Since you're bootstrapping the host you've already got all the rights in the world to inject arbitrary code as the user you're bootstrapping the host with, so not sure I'd consider this an 'attack'.
This will be fixed by #3920 which will eliminate the horrible bash command.

We'll want the sustaining team to try to reproduce this.

closing in favor of #8859 tracking the actual work.


bug: Issues with split behavior

Did you check the docs and existing issues?

[X] I have read the docs
[X] I have searched the existing issues

Neovim version (nvim -v)
0.10.2
Operating system/version
Windows 11 23H2
Describe the bug
I overwrote the default split keymaps like so:
{
  ["<C-h>"] = false,
  ["<C-s>"] = { "actions.select", opts = { horizontal = true }, desc = "Horizontal [s]plit" },
  ["<C-v>"] = { "actions.select", opts = { vertical = true }, desc = "[V]ertical split" },
}

Both split keymaps split vertically though. What am I missing?
What is the severity of this bug?
minor (annoyance)
Steps To Reproduce
Overwrite keymaps with the ones in the description.
Expected Behavior
<C-s> does a horizontal split, <C-v> does a vertical split.
Directory structure
No response
Repro
-- save as repro.lua
-- run with nvim -u repro.lua
-- DO NOT change the paths
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "runtime", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  {
        "stevearc/oil.nvim",
        config = function()
            require("oil").setup({
              columns = {
                "permissions",
                "size",
                "mtime",
                "icon",
              },
              keymaps = {
                ["<C-h>"] = false,
                ["<C-s>"] = { "actions.select", opts = { horizontal = true }, desc = "Horizontal [s]plit" },
                ["<C-v>"] = { "actions.select", opts = { vertical = true }, desc = "[V]ertical split" },
              },
            })
        end,
  },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Did you check the bug with a clean config?

[X] I have confirmed that the bug reproduces with nvim -u repro.lua using the repro.lua file above.

The user's config and the default config is merged with vim.tbl_deep_extend().
https://github.com/stevearc/oil.nvim/blob/52cc8a1fb35ea6ce1df536143add7ce7215c63c0/lua/oil/config.lua#L358
The keymap for <C-s> already exists in default config which vertical = true, so merging with user's keymap which horizontal = true will make both horizontal and vertical true.
https://github.com/stevearc/oil.nvim/blob/52cc8a1fb35ea6ce1df536143add7ce7215c63c0/lua/oil/config.lua#L65
It looks like neovim splits vertically if both is true. (lua vim.cmd.sbuffer({ mods = { horizontal = true, vertical = true } }))
IMO keymaps should not be deep merged.

@Foo-x is correct, this is because of some unfortunate side effects from the deep merging. The fix is a bit inelegant, but very simple.


fix: apply types/address.go from<EMAIL_ADDRESS>

Description

This PR enable to apply types/address.go from<EMAIL_ADDRESS>Major changes:

types.AccAddress, types.ValAddress and types.ConsAddress have changed from string type to byte array type.
types.ValidateAccAddress(STRING) + types.AccAddress(STRING)→types.AccAddressFromBech32(STRING)
types.ValidateValAddress(STRING) + types.ValAddress(STRING)→types.ValAddressFromBech32(STRING)
types.ValidateConsAddress(STRING) + types.ConsAddress(STRING)→types.ConsAddressFromBech32(STRING)
types.AccAddress(STRING)→types.AccAddressFromBech32(STRING)
types.ValAddress(STRING)→types.ValAddressFromBech32(STRING)
types.ConsAddress(STRING)→types.ConsAddressFromBech32(STRING)
types.BytesToAccAddress(BYTES)→types.AccAddress(BYTES)
types.BytesToValAddress(BYTES)→types.ValAddress(BYTES)
types.BytesToConsAddress(BYTES)→types.ConsAddress(BYTES)

Additional changes:

Fix and improve lbm-sdk

Fix invalid sumcheck address

https://github.com/line/lbm-sdk/pull/603/files#diff-ceb5513cfac0e7a49b0b5d44db5eea090c8d644d57946c64720aac8195363380L71-R73

Fix the bug that should be used operator address when to is nil on CreateFTClass

https://github.com/line/lbm-sdk/pull/603/files#diff-8bde5886bff65fcafc1d529eec14b27ef07528c9d92fd2ce8469ffbb4a35c3eeL299-R437

Fix incorrect address type

https://github.com/line/lbm-sdk/pull/603/files#diff-a124b23c2ecfa30a15bbae43eb33ca9552635de7c40286b0e693aa9278dd9fc2L252-R252

Add address validation if address are used without validation.

Fix the following cosmos-sdk bug

https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/x/slashing/keeper/querier.go#L51
https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/x/staking/legacy/v043/store_test.go#L31
https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/x/genutil/collect.go#L162

closes: #578
Motivation and context

How has this been tested?

Screenshots (if appropriate):
Checklist:

[x] I followed the contributing guidelines and code of conduct.
[x] I have added a relevant changelog to CHANGELOG.md
[ ] I have added tests to cover my changes.
[ ] I have updated the documentation accordingly.
[ ] I have updated API documentation client/docs/swagger-ui/swagger.yaml

Please add this changes to CHANGELOG.md

I found a minor diff.

https://github.com/line/lbm-sdk/pull/603/files#diff-a124b23c2ecfa30a15bbae43eb33ca9552635de7c40286b0e693aa9278dd9fc2R378

TypeURL: "/lbm.bank.invalidversion.MsgSend",

https://github.com/CosmWasm/wasmd/blob/v0.27.0/x/wasm/keeper/handler_plugin_encoders_test.go#L377

TypeURL: "/cosmos.bank.v2.MsgSend",


Add support for Amazon ECR

Docker credentials can't be static, have to obtain an authorization token.

:+1:

:+1:

For now we can run this periodically to generate credentials: https://github.com/rlister/ecr-login
But it would be more slick to build in ECR support as a config option.


Does this tool support latest gradle android project structure?

Hi guys, thanks for such cool tool.
When I try to run it with lint XML file it fails. It says that manifest file can't be found.
IOError: [Errno 2] No such file or directory: '/Projects/Project/app/AndroidManifest.xml'

Use below snippet to workaround until this is fixed:
android-resource-remover --app app/src/main --xml `pwd`/app/build/reports/lint-results-yourBuildVariant.xml


Sherpa crew mini support

Sherpa mount might work (edit. won't work), but would be great to separate mount with matching silhouette
https://github.com/jrlomas/Sherpa-Crew-Mini
Thanks for your work!

That thing is pretty awesome, but I think i'll have to pass on including it in the lineup of mounts.  I just dont have time at present to work something up.  you could drop the sherpa crew cad into the step file from my repo and see if it lines up with any of the other extruders with diagonally oriented motors.  you might find one close enough to work.  you may need to mirror the bracket on X, but my parts are designed to be able to do that.  I hope you figure something out.  if you use my geometry to design a new bracket, post it in a usermod folder and ill approve the commit.  closing this issue.


Cannot get back the full object from the relational field after saving an object

We set a complete object to object's relation field, but there is only id leave after saving. Following is the testing code and the output comparison between parse.com and parse-server.
The testing cloud code:
Parse.Cloud.define("testQuery", function(request, response) {
  var objectQuery = new Parse.Query(ObjectClass);
  var Object;

  objectQuery.include('user');
  objectQuery
  .get("_<object_id>_", {useMasterKey: true})
  .then(function(_object) {
    object = _object;
    object.set('user', request.user);
    console.log("1 ======> " + JSON.stringify(object.get("user")));
    console.log("1 ======> " + object.get("user").getUsername());
    return object.save(null, {useMasterKey: true});
  })
  .then(function() {
    console.log("2 ======> " + JSON.stringify(object.get("user")));
    console.log("2 ======> " + object.get("user").getUsername());
    response.success("");
  }, function(error) {
    response.error(error);
  });
});

console log of parse.com:
1 ======> {< full user object >}
1 ======> username
2 ======> {< full user object >}
2 ======> username
console log of parse.com:
1 ======> {< full user object >}
1 ======> username
2 ======> {"objectId":"< user id >"}
2 ======> undefined

Which of the two outputs is cloud code, and which is from parse-server. You wrote parse.com twice :)

Also, please provide which version of the Parse SDK you are using (console.log(Parse.CoreManager.get('VERSION')))

@andrewimm sorry for confusion! I just updated my comment :P
The first one is parse.com and second one is parse-server
The parse SDK version is js1.8.1

Very peculiar. Saving the object shouldn't have side effects on its children. I'll investigate this

Parse.Cloud.define("testQuery", function(request, response) {
  var objectQuery = new Parse.Query(ObjectClass);
  var Object; // <-- upperCase `O` and a reserved Javascript constructor

  objectQuery.include('user');
  objectQuery
  .get("_<object_id>_", {useMasterKey: true})
  .then(function(_object) {
    object = _object; // <-- lowerCase `o` and a leaked global since it´s not declared with var
    object.set('user', request.user);
    console.log("1 ======> " + JSON.stringify(object.get("user")));
    console.log("1 ======> " + object.get("user").getUsername());
    return object.save(null, {useMasterKey: true});
  })
  .then(function() {
    console.log("2 ======> " + JSON.stringify(object.get("user")));
    console.log("2 ======> " + object.get("user").getUsername());
    response.success("");
  }, function(error) {
    response.error(error);
  });
});

I'm guessing your parse server configuration isn't injecting globals even though Parse.com probably does. Try to run this code with 'use strict';

@carmenlau do you have a beforeSave function defined for the ObjectClass class? And what Parse.com SDK version did you use to run the test cloud code?


feat(nodes): generalised metadata handling for nodes

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Description
Add two new nodes to handle metadata generation, plus a couple new fields.
MetadataItem field

label: str: the label for the metadata item
value: Any: the value of the metadata item

MetadataItemInvocation node
Inputs

label: str: the label for the resultant metadata item
value: Any: the value of the resultant metadata item

Outputs a MetadataItem
MetadataDict field

data: dict[str, Any]: a dict of metadata

MetadataInvocation node
Inputs

items: list[MetadataItem]: a collection of metadata items, they are collected by a collect node

Outputs a MetadataDict
Usage

(note the output - a dict of the metadata items)
Notes

You can't do anything with the output metadata dict just yet. Nodes currently have a metadata field that is hidden and "reserved" for use by the linear UI. Because no workflows use this, we may safely repurpose it without breaking existing workflows. Will address this later, @ebr nerd-sniped me and I had to get an MVP working.
The existing MetadataAccumulator node can be totally replaced by this for linear UI graphs. May need to create an additional intermediary node to facilitate this accepting both direct and connection values. Hard to explain but anyways its not hard.
One change in this PR is the introduction for Any field types. This is already handled by the backend, but it just wasn't supported in the UI. OpenAPI schema generation doesn't provide a "type" for Any, so I've added UIType.Any to be explicit.
MetadataItemInvocation uses Any for its value field.
Whenever an Any type field is involved, connections are valid. You may connect an Any to anything, and anything may connect to an Any. With great power comes great responsibility, so I don't plan on letting any other core nodes use this. Note that it is only used as an input for MetadataItemInvocation. Probably, we should never ever allow anything to output an Any. Community nodes are free to use this as they wish.
You may not directly edit the value field of MetadataItem, because it is an Any type. How could we represent this in the UI? Well, it's technically doable - you could have a drop-down to choose any supported input field type and get an input component for it, but that feels wrong. In the workflow editor, you must provide the value with a connection. When creating a graph programmatically, you may give it any serializable value directly.

Related Tickets & Documents

Closes #4495

QA Instructions, Screenshots, Recordings
Though this is a draft, and you cannot do anything with the metadata dict just yet, you can still try it out. Follow the example in image.

Updated validation for collect nodes to allow Any inputs to accept Collect outputs
Added tests for Any (and polymorphics)
Made MetadataInvocation.items polymorphic

So @ymgenesis 's test case now can work like this:

If you want multiple MetadataItems, you'll still need to use a Collect:

feat(nodes): add more tests

Add test, currently xfail'd, for collect -> iterate node connections
Add more polymorphics tests

feat(nodes): refine validation for collector outputs

Remove the check for Any inside the validation function, instead only run the validation function if the collect is not outputting to an Any or list[Any] (both are valid)
Add validation for Collect -> Iterate, and refine the test for this

feat(nodes): improved api for workflow and metadata embedding

Invocations that want metadata or workflows now must inherit from WithMetadata and WithWorkflow models (in addition to BaseInvocation). These provide the appropriate workflow and metadata fields to the invocation. The alternative API is to use the @invocation decorator, but this doesn't give type hints in invocation definitions and is also much harder to understand. I tried and scrapped it.
Add MergeMetadataDictInvocation, which takes a collection of MetadataDict and merges them together. This adds additional flexibility to the construction of metadata. Without this, the linear UI would need a separate MetadataItemInvocation for every piece of metadata - with this merge node, far fewer nodes are needed to construct metadata.
Update all invocations to use this new setup
Update UI with new field types and logic to handle this new way of specifying workflows and metadata should be used

closed in favor of #4686


Quit unexpectedly

Pock will somehow quit by itself, i'm on MacOS Catalina (Ver 15.4). Not sure why

Experiencing the same 10.15.3

Experiencing the same on MacOS 10.15.3 and Pock 0.7.2.

I think might have something to do with app switching, or app switcher, like Context

It seems to crash silently, I'm having the same issue. Is it keeping logs somewhere?

Same here. Just dissapears randomly.

Same here. Is pock still being maintained? No commit since january...

Same here. Is pock still being maintained? No commit since january...

Hi! As stated in the README:

I’m currently migrating all default widgets to make Pock independent from them. At the end of the process, each widget will have its dedicated repo, where you'll be able to open new PRs. I hope to publish the new sources soon alongside PockKit.
The PRs that are currently opened will be merged manually in the dedicated widget repository
Thanks for your patience!


fixcompatible with lookup timestamp types

Proposed changes
Issue Number: close #xxx
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Thanks for @vinlee19, can you write  the description of pr


Fix cstyle error in ./src/istgt_crc32c.h

Repo: openebs/istgt
Branch: replication
File: ./src/istgt_crc32c.h
Steps to compile in: README.md
PR subject should be like: fmt(cstyle): cstyle fixes in
./src/istgt_crc32c.h: 29: #define followed by space instead of tab
./src/istgt_crc32c.h: 38: #define followed by space instead of tab
./src/istgt_crc32c.h: 39: #define followed by space instead of tab
./src/istgt_crc32c.h: 40: spaces instead of tabs
./src/istgt_crc32c.h: 40: #define followed by space instead of tab
./src/istgt_crc32c.h: 41: #define followed by space instead of tab
./src/istgt_crc32c.h: 47: line > 80 characters
Makefile:73: recipe for target 'cstyle' failed
Resources:
-> Countributers guide: https://github.com/openebs/openebs/blob/master/CONTRIBUTING.md
Complexity: Easy

I will take this up.

sure @Pensu go ahead. Let me know if you need any help.

@ashishranjan738 Sure, how do you suggest shortening the line from 80 characters? Shall I break it and add \, like we do in shell scripts?

ya you can try.


Remove urlgravatar field from account_info response

Summary
There's a weird special-case in the account_info response that adds a field named urlgravatar when the account has the EmailHash field set. We should remove this.
Reproducing

Example account_info with urlgravatar in the account_data
ledger_entry doesn't do this when you fetch an AccountRoot that way.

Solution
This field's contents are always http://www.gravatar.com/avatar/{EmailHash as lowercase}, which is pointless. Anyone who wants to look up a Gravatar can probably do that using the EmailHash without the server taking its precious time to inject an HTTP URL (not even HTTPS!) into a low-level ledger object.
Technically, this is (*sigh*) a breaking change, even though it's not documented and as far as I can tell even the old desktop client and web client don't have code to use it. So it should probably be on a version switch, even though that means this pointless little bit of code will continue to sit around for a long time yet.

My humble opinion: This can just be removed.
That said, if whoever removes it prefers to do that in api_version 2 (or 3, or whatever number makes sense at the time it gets removed), that's absolutely fine as well.


Add generic source plugin for testing purposes

To be used for mocking events in EDA e2e tests.
Closes AAP-11535.

New release of ansible-rulebook needed for the tests to pass.


Trakt TV collection does not display

Expected Behavior
Trakt TV Shows collection should list my trakt collection.
Current Behavior
Couple of days ago, I started to get the following error upon entering TV Shows - Trakt > My Collection.
Error message displayed:
BAD STATUS GETTING TRAKT COLLECTION FOR SHOWS: 500
Other menu entries related to trakt seem to work. Even My Collection for Movies in Elementum is working. Seems like the problem is only on TV Shows collection menu entry.
Possible Solution
Not aware of any.
Steps to Reproduce (for bugs)

In Elementum plugin, go to TV Shows - Trakt > My Collection
Note the error message that appears on the top.

Context
I cannot see my Trakt collection.
Your Environment
Kodi 19.3 on Android arm.

Version used:
0.1.86

Environment name and version:
Kodi 19.3

Operating System and version:
Android 7

Link to Kodi debug log:
https://paste.kodi.tv/rejujeqavi

@asmerovski
hm, i do not see error in log file. try to reproduce it again, maybe.

kodi.log
Cannot add log to paste.kodi.tv, it is too large, so here it is.

i still do not see related errors in log.
"TV Shows - Trakt > My Collection" works for me,
but i guess you added to your collection something new that breaks elementum.
2 options:

remove new things that you added to collection and check again. then please report what item caused issue.
if it will not help - share your collection, e.g. https://trakt.tv/users/YOUR_NAME/collection , it might help to understand what causes the issue. but please try 1st option first.

Hi, i removed all recently added shows as instructed in step 1. Stil happening.
This is my collection: https://trakt.tv/users/asmerm/collection

unfortunately, i do not know how to help you with this issue.
i do not see any errors in 1st log and in 2nd log i see only unrelated errors (like error b/c torrent was not chosen and then error b/c of failed log upload).
i expected that some "tricky" new item in collection caused that - but no. and i also do not see any "tricky" items in your current collection.
as a "wild guess" - try to disable "Show number of unwatched episodes in Shows/Seasons" option in "Appearance" tab.
this feature had bug, but it was fixed, and that bug was very visible in log file.


the meaning of label data

@cnlinxi sorry again for my response. You can find the data at https://codeforthailand.s3-ap-southeast-1.amazonaws.com/attacut-related/data.zip
Please unzip and make sure the root directory is at ./data. The content of the archive contains

Only the first two are relevant for training; sampling-0 means all the dateset, while sampling-10 means only 10 files are used. You can use sampling-10 for quick training.
Before running the training command below, make sure that you have the ./artifacts directory.
python ./scripts/train.py --model-name seq_sy_ch_conv_concat \
 --model-params "embc:8|embs:8|conv:8|l1:6|do:0.1" \
 --data-dir ./data/best-syllable-crf-and-character-seq-feature-sampling-0  \
 --output-dir ./artifacts/model-xx  \
 --epoch 2 \
 --batch-size 1024 \
 --lr 0.001 \
 --lr-schedule "step:5|gamma:0.5"

Originally posted by @heytitle in https://github.com/PyThaiNLP/attacut/issues/20#issuecomment-670847406

@cnlinxi sorry again for my response. You can find the data at https://codeforthailand.s3-ap-southeast-1.amazonaws.com/attacut-related/data.zip
Please unzip and make sure the root directory is at ./data. The content of the archive contains

Only the first two are relevant for training; sampling-0 means all the dateset, while sampling-10 means only 10 files are used. You can use sampling-10 for quick training.
Before running the training command below, make sure that you have the ./artifacts directory.
python ./scripts/train.py --model-name seq_sy_ch_conv_concat \
 --model-params "embc:8|embs:8|conv:8|l1:6|do:0.1" \
 --data-dir ./data/best-syllable-crf-and-character-seq-feature-sampling-0  \
 --output-dir ./artifacts/model-xx  \
 --epoch 2 \
 --batch-size 1024 \
 --lr 0.001 \
 --lr-schedule "step:5|gamma:0.5"

Originally posted by @heytitle in #20 (comment)

Hi heytitle:
If i want to eval the word accuracy, can you tell me how to use "xxx.label" file with "xxx.txt" file ? do you know the link of syllable in  "xxx.txt" file  with "0\1" symbol in "xxx.label"?
what's the meaning of "0\1" symbol in "xxx.label"?

@cnlinxi sorry again for my response. You can find the data at https://codeforthailand.s3-ap-southeast-1.amazonaws.com/attacut-related/data.zip
Please unzip and make sure the root directory is at ./data. The content of the archive contains

Only the first two are relevant for training; sampling-0 means all the dateset, while sampling-10 means only 10 files are used. You can use sampling-10 for quick training.
Before running the training command below, make sure that you have the ./artifacts directory.
python ./scripts/train.py --model-name seq_sy_ch_conv_concat \
 --model-params "embc:8|embs:8|conv:8|l1:6|do:0.1" \
 --data-dir ./data/best-syllable-crf-and-character-seq-feature-sampling-0  \
 --output-dir ./artifacts/model-xx  \
 --epoch 2 \
 --batch-size 1024 \
 --lr 0.001 \
 --lr-schedule "step:5|gamma:0.5"

Originally posted by @heytitle in #20 (comment)

Hi heytitle:
If i want to eval the word accuracy, can you tell me how to use "xxx.label" file with "xxx.txt" file ? do you know the link of syllable in  "xxx.txt" file  with "0\1" symbol in "xxx.label"?
what's the meaning of "0\1" symbol in "xxx.label"?


can use local LLM? not chatgpt

It is also possible to use open-source local models, and the corresponding prompt must be modified to ensure compatibility.


i18n.languages is always an array of ['en']

Describe the bug
Using the react-i18next "useTranslation" hook on the client gives me access to the i18n instance which contains i18n.languages. However this is always an array of ['en'], even though my supportedLng is ['en', 'ja'] and I have both json files in the correct paths. ('/locales/en/translations.json', '/locales/ja/translations.json')
Does remix-i18next lack support for understanding the supported languages potentially?
Your Example Website or App
Github repo
Steps to Reproduce the Bug or Issue

Add remix-i18next as per the installation instructions, with more than one language.
Log i18n.languages from the useTranslation hook on the client
See it doesn't return all the viable languages.

Expected behavior
It should list all the languages available listed in supportedLng
Screenshots or Videos
No response
Platform
macOs, chrome
Additional context
No response

Fixed - uses fallbackLng not supportedLng, which I find strange.


① [Currency selection] Remove currency picker

📲 What
Paired with @Scollaco to remove the currency picker from SettingsAccountViewController.
🤔 Why
We've decided to replace the picker behaviour with a table view that will be pushed onto the navigation stack.
🛠 How
Deleted all references to the picker.
✅ Acceptance criteria

[ ] Project builds successfully
[ ] Tests pass
[ ] Currency picker is gone

🤖Related PR's for this feature
#569

 Change Currency Picker to new view with TableViewCells

Looks like case SettingsCurrencyPickerCell in Nib.swift can be removed as well..

Also from visual perspective there are 2 questions that come into my mind:

Should we add disclosure indicator to the change currency cell to imply that this will push user to a different screen?

Should we change the text color to something more subtle?
Using green color makes the text look like a button, but since we want the whole cell to react to taps we don't explicitly need to make this look like a button (the disclosure indicator from first point should make that clear). Cc @dannyalright

@dannyalright sure - just to be clear we add a chevron disclosure icon in a later PR, but are you saying the text should not be green?

that's right. It should be the default cell 'value' colour. Here's the exact screen just to be safe. All good RE chevron in later PR


Smoothing is piling-up time lag between screen and LEDS

Bug report, debug log and your config file (FULL LOGS ARE MANDATORY)
https://pastebin.com/ny5c5VwW
Steps to reproduce
Activate HyperHDR without smoothing --> no time lag
Activate smoothing --> Time lag increases the longer HyperHDR is running
What is expected?
No major influnce of smoothing on time lag between screen and LEDs
No increase of time lag with run time of HyperHDR. It seems that HyperHDR is piling up the delay time.
What I addionally recoginzed: Even if I specify an update frequency of 20 in the web interface the log reports "interval: 40ms (25Hz)"
What is actually happening?
Time lag is constantly increasing with run-time of HyperHDR
System
Behaviour can be reproduced under Windows with DirectX grabber and RasberryPi 3b+ with external grabber. Logs have been created on windows system
HyperHDR Server:

Build:           (HEAD detached at dec81c0) (Awawa-2a2ed8d/dec81c0-1631541363)
Build time:      Sep 15 2021 15:59:04
Git Remote:      https://github.com/awawa-dev/HyperHDR
Version:         <IP_ADDRESS>
UI Lang:         en (BrowserLang: de)
UI Access:       default
Avail Capt:      Windows (Microsoft Media Foundation)
Database:        read/write

HyperHDR Server OS:

Distribution:   Windows 10 Version 2009
Architecture:   x86_64
Kernel:         winnt (10.0.19043 (WS: 64))
Qt Version:     5.15.2
Browser:        Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.69 Safari/537.36 Edg/95.0.1020.44

Forgot to mention, that it worked as expected in verion 16.1.

But DX grabber was not present in v16. Can't see a problem frankly. Diagnostic can a problematic for particular setup external network LED instances such as WLED over wifi because a lot depends on the connection stability and even 25Hz could be too much sometimes. You can lower delay to from 150ms to ~50ms in the smoothing settings to have better response time also.

Hello awawa-dev
Thanks for your quick response!
I used the raspberry pi version with 16.1 on my "productive" system. Also understand that set-up with WLED is hard to judge on.
I tried to lower the update frequency to 10 fps and still in the log it is reported as 25Hz. According to the log I am not able to lower it less than 25Hz, whatever I enter. I can increase > 25Hz, but not decrease.
Could you maybe please check if the update frequency I enter into the webgui is correctly applied?
2021-11-12T10:05:16.120Z [SMOOTHING0] Selecting config (0) => type: Alternative, dirMode: false, pause: false, settlingTime: 50ms, interval: 40ms (25Hz), antiFlickTres: 32, antiFlickStep: 1, antiFlickTime: 250
Thanks!

although web-gui allows to input as low as 1Hz, then in the smoothing procedure it's verified and 25Hz is set as minimum. I could to set same limit in both units but very doubt it will change anything in your situation and very low refresh rate makes the smoothing looks bad anyway. Lets say 25Hz is lowest usable.
if the wifi connection is lost from time to time even for a very short time it could result in such behavior as your. Other possibility is the router itself. But it's hard to judge if the previous version works better for you: major changes in smoothing was made in v16, in v17 are only cosmetic and they couldn't alone change behavior definitely in such dramatic way. And I'm very confident about it. WLED network protocol on the other hand changed a bit to fix one problem but then it should affect also no-smoothing scenario.
You can assess the maximum WLED throughput but enabling 'continues output'  and settings rather high refresh rate (for example 50Hz) in the smoothing (video source or some effect must be active anyway). Then in WLED's statistics panel you can read the final refresh rate. Refresh rate in HyperHDR should be set lower than maximum achieved refresh rate in WLED, But it is not a scientific method and you can miss drop in the packages delivery (and WLED refresh rate) if they appears only from time to time.
Other border-possibility that could affects smoothing is completely running out of system resources (CPU/RAM) but it's unlikely in your case as that happens for both your OS/devices. But WLED/network is one common thing that connects the problem so I would dig deeper around it.

Your are correct. My wifi seems to be the bottleneck though I did not thought newwork through-put for wled would be that high.
Also I seem to understand, why it is not a problem without smoothing being activated. If smoothing is off I assume the LED update frequency is equal the capture frequency and in my case it is 20fps, what my wifi can perfectly manage. By activating smooting with 20 Hz the LED update frequency is going up to 25 Hz as minimum and I run into delay.
For me 20 fps also for smoothing is absolutely enough. I compared it to Hyperion-NG, where lower rates are possible. Of course I would like to benefit from HyperHDR (3dLUT, HDR and anti-flickering). Could you please take into account to let lower smoothing rates than 25Hz to be set. Thanks a lot!

What was your best result when testing max throughput with WLED and continues output in HyperHDR? I'm asking because there is a very small chance that for some configuration/setups WLED may be limiting your performance. Especially since you have over 300leds (what model?).
It's not problem for me to unify web page settings with internal smoothing module and set for example 15Hz as minimum for both. But I would suggest first to set grabber to 25Hz and disable smoothing just to be sure it will help anyway.

I cannot go over 20fps, but the network througput is minimal (~18KB/s). I am using APA102 LEDs and of course it could be WLED is limiting the performance with 302 LEDs. Need to do some research.

18Kb/s indicates that hyperhdr is sending just 20fps (~3B*302  *  20). You must enable smoothing with continues output and have video/capturing source (not effects because some have their own smoothing settings independent from the user...don't remember which exactly, but should be visible in the log when the effect start to play)

If I activate smoothing with 20fps & continues output, network goes up to 23KB/s. But wled does not work anymore, since 25 fps are sent in reality, I also recognized that in that case wled becomes unresponsive with an error (connection to light failed).
Unfortunately 20 fps seems to be the max for my set-up.

Which WLED version and ESP32 or ESp8266? Do you use default WLED pinouts for SPI? Seems a bit like WLED runs out of resources but it shouldn't because it's just a SPI for APA102...unless it works under software emulation.

It is an ESP8266. Data GPIO = 2 and clock GPIO = 4. For clock default would be GPIO 0. Reason is, that I use a board where GPIO 4 is passed through for the clock.

If it's not a hardware SPI outputs (or not correctly detected by the library if it's non-reference design) for that boards then WLED is probably is using bit bang instead of hardware interface. ESP8266 contrary to ESP32 is not capable of pinout remapping
https://github.com/Makuna/NeoPixelBus/wiki/NeoPixelBus-object#dotstarspimethod
So the root of the problem lies probable in the ESP hardware, its configuration or WLED. Anyway since Smoothing lower limit was different in the GUI than in the Smoothing module I changed that with the newest commit (lower limit is set to 20Hz globally).

Hi awawa
Thanks again for the excellent support and for bringing HyperHDR to the community!
Your hints made me dig deeper into the ESP8266, WLED and APA102 and your conclusion was correct: For hardware SPI on the ESP8266 you need to connect GPIO14: SCLK for the CLK and GPIO13: MOSI for data. I can easily go up to 50 fps, even I understood that WLED is limiting to 42 Hz maximum.
WLED is suggestion GPIO2 as default data pin and actually I could not find any consistent documentation on how to use pins for APA102 leds.
I will also post it in the WLED forum. It may help others.


Accessing child elements of XML sequences return unexpected results

Description:
Refer to the below code example. Note that accessing the first element of the sequence using xmlSequence.children().get(0) returns an empty XML value and xmlSequence.children().get(1) returns the 0th element, as shown below.
public function main() {
    xml xmlSequence = xml `
                    <child1>
                        <grandchild1>"Hello1"</grandchild1>
                        <grandchild2>"World1"</grandchild2>
                    </child1>
                    <child2>
                        <grandchild1>"Hello2"</grandchild1>
                        <grandchild2>"World2"</grandchild2>
                    </child2>
                  `;
    var child1 = xmlSequence.children().get(0);
    var child2 = xmlSequence.children().get(1);
}

values of child elements

parent sequence

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

The child1 variable is not empty, it is whitespace. This is the expected behaviour and not a bug. This is the whitespace between <child1> and <grandchild1>.
io:println(child1 is xml:Text); // true
io:println(child1 == xml ``); // false - not empty

If child1 is
<child1><grandchild1>"Hello1"</grandchild1><grandchild2>"World1"</grandchild2></child1>

you will not get the whitespace XML text when calling children().

The child1 variable is not empty, it is whitespace. This is the expected behaviour and not a bug. This is the whitespace between <child1> and <grandchild1>.
io:println(child1 is xml:Text); // true
io:println(child1 == xml ``); // false - not empty

If child1 is
<child1><grandchild1>"Hello1"</grandchild1><grandchild2>"World1"</grandchild2></child1>

you will not get the whitespace XML text when calling children().

Ah I see.. thanks for the clarification @MaryamZi 🙂
In that case, I think what we should do is improving the debugger variable presentation to clearly represent whitespaces and empty values so that the debug users won't get confused in such contexts.

Closing the issue since the observed behaviour is expected.


Document and run benchmarking

Please describe the feature you'd like to see

Document the config.json
All the esoteric features
@pankajastro should be able to run benchmarking without any support

Describe the solution you'd like
A clear and concise description of what you want to happen, if possible with code examples.
Are there any alternatives to this feature?
Is there another way we could solve this problem or enable this use-case?
Additional context
Add any other context about the feature request here.
Acceptance Criteria

[ ] All checks and tests in the CI should pass
[ ] Unit tests (90% code coverage or more, once available)
[ ] Integration tests (if the feature relates to a new database or external service)
[ ] Example DAG
[ ] Docstrings in reStructuredText for each of  methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
[ ] Exception handling in case of errors
[ ] Logging (are we exposing useful information to the user? e.g. source and destination)
[ ] Improve the documentation (README, Sphinx, and any other relevant)
[ ] How to use Guide for the feature (example)

I have tried to document it here https://docs.google.com/document/d/1ZW4IkwqMTOKmDYD6unrFb3-Ii49X35MEwLpcPQuhzhU/edit?usp=sharing


[elm-package] Fix elm package service

With the release of Elm 0.19.0, the package file changed (from elm-package.json to elm.json), which breaks the badge.
(This replaces #1979 due to a refactor.)

Warnings

:warning:

This PR modified service code for elm-package but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @Y0hy0h!

  Generated by :no_entry_sign: dangerJS

I noticed that it queries the http version. Is it possible to use the https version?

The CI failed initially because I told you the wrong tag. The format just changed and I'm still getting the hang of it!

There still seems to be an issue with the service name.

Argh, I guess I had that title format right to begin with.
When I run the tests locally using npm run test:services -- --only=elm-package, I get these two failures. It looks like the tests need to be updated to reflect the formatting change.

  1) ELM PACKAGE
       gets the package version of elm-lang/core

	[ GET /v/elm-lang/core.json ]:
     ValidationError: child "name" fails because ["name" must be one of [elm-package]]
      at Object.exports.process (node_modules/joi/lib/errors.js:196:19)
      at internals.Object._validateWithOptions (node_modules/joi/lib/types/any/index.js:675:31)
      at module.exports.internals.Any.root.validate (node_modules/joi/lib/index.js:138:23)
      at Object.pathMatch.matchJSONTypes (node_modules/icedfrisby/lib/pathMatch.js:303:9)
      at _expect (node_modules/icedfrisby/lib/icedfrisby.js:563:10)
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1261:26)
      at start (node_modules/icedfrisby/lib/icedfrisby.js:1244:12)
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:1131:16)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.EventEmitter.emit (domain.js:442:20)
      at Request.<anonymous> (node_modules/request/request.js:1161:10)
      at Request.EventEmitter.emit (domain.js:442:20)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1083:12)
      at IncomingMessage.EventEmitter.emit (domain.js:442:20)
      at endReadableNT (_stream_readable.js:1081:12)
      at process._tickCallback (internal/process/next_tick.js:63:19)

  2) ELM PACKAGE
       invalid package name

	[ GET /v/elm-community/frodo-is-not-a-package.json ]:

      AssertionError: expected { Object (name, value) } to deeply equal { Object (name, value) }
      + expected - actual

       {
      -  "name": "elm package"
      +  "name": "elm-package"
         "value": "invalid"
       }

      at Object.pathMatch.matchJSON (node_modules/icedfrisby/lib/pathMatch.js:138:38)
      at _expect (node_modules/icedfrisby/lib/icedfrisby.js:583:10)
      at IcedFrisbyNock._invokeExpects (node_modules/icedfrisby/lib/icedfrisby.js:1261:26)
      at start (node_modules/icedfrisby/lib/icedfrisby.js:1244:12)
      at Request.runCallback [as _callback] (node_modules/icedfrisby/lib/icedfrisby.js:1131:16)
      at Request.self.callback (node_modules/request/request.js:185:22)
      at Request.EventEmitter.emit (domain.js:442:20)
      at Request.<anonymous> (node_modules/request/request.js:1161:10)
      at Request.EventEmitter.emit (domain.js:442:20)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1083:12)
      at IncomingMessage.EventEmitter.emit (domain.js:442:20)
      at endReadableNT (_stream_readable.js:1081:12)
      at process._tickCallback (internal/process/next_tick.js:63:19)

I will look into this tomorrow. ;)

The tests are passing now. However, I'm not entirely sure on the following naming inconsistency:
The service is called elm-package, but the badge will display elm package. I don't know if it is even possible to rename the service to elm package (because of the whitespace).
I guess this depends on whether there is a guideline for this sort of naming issue. I personally think it's ok, but it's not perfect.
Is there a simple fix to this I'm not seeing?


Log rotation limited by maximum timeout length

When researching #59 and #60 I found the maximum timeout length is limited to<PHONE_NUMBER> ms. Implementing timeout chaining would allow for longer timeouts so that monthly and yearly logfile rotation works properly.

My concern with this is the potential to leave a lot of un-managed setTimeouts floating around as you have to chain more and more of them together.

I have a version that uses the long-timeout module to implement timeouts longer than<PHONE_NUMBER> ms, but I'm having trouble rewriting the test rotates logs on the specified interval so that it passes.
In the original version, Math.min was overloaded so that it would trigger a rotate in 100ms even though it's set to daily. Not sure what to do in this case since I no longer use min.
internals.setUpRotate = function (reporter, period) {

    var now = Moment.utc();

    period = period.toLowerCase();
    now.endOf(internals.timeMap[period]);
    var timeout = now.valueOf() - Date.now();

    reporter._state.timeout = Lt.setTimeout(function () {

        internals.rotate(reporter, period);
    }, timeout);

};

Came up with a hokey solution, but it works! Submitting a pull request now...
    timeout = Math.min(timeout, timeout); // for test compatibility

@arb Why is it an enhancement ? Looks a lot like a bug to me.

Because as it was, it would just chop of the time to the max time setTimout could support. This was the documented behavior. Changing it so it now works would be considered an enhancement.

IIRC, overflow on setTimeout is strictly equivalent to setting it to 0, so it's immediate, probably the cause of the IO exception on #63.

This code allows using timeout beyond the capabilities of existing good-file, so it's an enhancement.

Closed via https://github.com/hapijs/good-file/pull/62


any plans to remove the IndexConfigurator and merge it into models?

as we all know that ES 6.0 and beyond are removing multi-type index, and this will cause every model will need a single index in ES, so if we still need a IndexConfigurator class, and say , we have about 20 models need to be indexed with es, then we will need 20 IndexConfigurator class.
This is some annoying setting, for ES 6.0 and later version.
So I'm wondering if there is a chance to merge the IndexConfigurator class's getSetting and getMapping function into individual model class?

Hey, this is actually a very good question! I was thinking about that, but currently ES6 supports multiple types in one index for migrated instances from ES5. So if I drop it, some of the guys, who uses multiple types and migrated to ES6 will lose some functionality. I will think about making IndexConfigurator deprecated and moving all the configurations to model, but it will take some time.

yes, I know this is the breaking change after all, but the only way to use multi-type in ES6 is by creating the index in ES5. So, you can create to branch, for example , current 3.x will still use the olde IndexConfigurator class, and new 4.x will support ES6.
and by supporting ES6, I think another feature is needed, that is create the index with some prefix. and that is easy.

I'd prefer to keep IndexConfigurator, but make it deprecated and get rid of it completely only with ES7 release. I'll try to do that in the next release.

The configuration in an own class is good practice imho. Separation of concerns.

@ametad it's good practice, but require bit more tweak to get the decoupling going.
With ES6.2, I now write just as many indexConfigurator as model, as well as mappings.
And it would be good to consider parent-child join mapping together using Eloquent's relation setup.

@fzhan I understand the whole inconvenience of adding configurator for each model, but now we have that separation in ES6, so it makes sense. Also if I rewrite how it works now, we will get not backward compatible version or lots of deprecated stuff, because the changes will be huge. That's why I want to wait for ES7 and get rid of some things entirely.

Closing ticket until ES7 is released.


Transforming Point Sets Without Writing to File

Thanks for the awesome package!
Given a transformation calculated by simpleelastix, I want to to apply this to a point set. While this works great, the issue is I need to perform this operation thousands of times, and would like to implement via parallel runs on a cluster.
The documented implementation: https://simpleelastix.readthedocs.io/PointBasedRegistration.html takes the point set as input using SetFixedPointSetFileName which expects a file written to disk and outputs the result in the file "outputpoints.txt". This reading/writing to files causes problems/inefficiencies in parallellization.
So I was wondering

Is it possible to provide the Fixed Point Set in memory, i.e. not as a file on disk?
Is it possible to change the default output file from outputpoints.txt (or ideally not write to a file at all)?

Thanks!

It is not possible at the moment because elastix itself only supports reading point files from disk.

It is not possible at the moment because elastix itself only supports reading point files from disk.

Thanks for the response. This seems like a strange choice by elastix ...
I'm mostly working with rigid transformations, so I guess I can extract the transformation parameters and implement myself. So I'll mark the issue closed.
Thanks!


$('#Div').bind("ready.jstree", function() is not being called

Hello,
Tree is not loading some times and  hence .bind("ready.jstree) function is not invoked.
in our application,we are generating the JSON dynamically and this issue happens only when the JSON object is huge in size.are there any limitation of the JSON size that JSTREE supports?.
The issue i mentioned happens some times and  few occurances tree is loading fine. and .bind("ready.jstree) invoked
when we expeience this issue,we see we are getting below script error.Note that this issue we are facing only in IE ,but not in chrome.
SCRIPT0: Script Error
Line: 0
Please can you help this resolved.
We are using version v 3.3.9

There is nothing I can do to fix this error - the error is not jstree related - you are probably hitting a browser limit.


Can i Use your toolkit with Google Maps  api ?

Can i Use your toolkit with Google Maps  api ?

Hi!
Yes, you could write an adapter for the Google Maps API, to use it for routing and create a map widget for the UI. However, currently there is no support for Google Maps, because afaik the Terms of Service of Google Maps prohibit the usage for real time navigation.
I hope this helps!

I understand that the routing in only the service you support for google
and no nteractive GPS driving directions .
Or I can do all the services by FFWDME
On June 6, 2017 at 8:41:53 PM, Christian Bäuerlein<EMAIL_ADDRESS>wrote:
Hi!
Yes, you could write an adapter for the Google Maps API, to use it for
routing and create a map widget for the UI. However, currently there is no
support for Google Maps, because afaik the Terms of Service of Google Maps
prohibit the usage for real time navigation.
I hope this helps!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ffwdme/ffwdme.js/issues/31#issuecomment-306579544, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQ0Zp2joJnnUTAVgrdo8INRVMut4zODdks5sBZ1vgaJpZM4Nxn-y
.

Sorry, but I did not understand this last message. Could you rephrase it please?

how to use it by google , Can you give me an example ?
2017-06-30 17:52 GMT+02:00 Christian Bäuerlein<EMAIL_ADDRESS>
Sorry, but I did not understand this last message. Could you rephrase it
please?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ffwdme/ffwdme.js/issues/31#issuecomment-312304596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ0Zp3TCD8_dOUaWUdyn9r06G9z4mmxUks5sJRnLgaJpZM4Nxn-y
.

Sorry I can not help you with this, as this would be against the Terms of Service  of Google Maps.

Hello, The New Google Maps T&C (eff June 11th 2018) do not fully prohibit real time navigations anymore. Are you planning to add gMap support to your great code ?
Thanks

Interesting! Do you have a link to this?

https://cloud.google.com/maps-platform/terms/
Look at 3.2.4 (c). Terms have changed. Turn by Turn real time navigation is
allowed except if it is 'substantially similar to the functionality
provided by the Google Maps for Android mobile app'. To be honest, those
terms are pretty vague and subject to interpretation. The important point,
IMO, is that they went from a 'no' to a 'yes but'...
2018-05-15 9:33 GMT+02:00 Christian Bäuerlein<EMAIL_ADDRESS>
Interesting! Do you have a link to this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ffwdme/ffwdme.js/issues/31#issuecomment-389072077,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG7E-6sS1S73rxdYoDSdv1o6WMF-KD2Gks5tyoTTgaJpZM4Nxn-y
.

Cool! Right now I have no plans to add support for Google Maps myself, but I am happy to assist/review PRs. :)


Use cookie for authentication

This pull request:

adds issue_token / cookies authentication + tests
fixes the is_online key

+1 for this please.

Thanks @nicoinch, I hope to have some time this weekend to review and test.

When is it expected to be merged? Thanks

@benceh90 I will have a look at the pull request when it is ready for review. In the mean-time you can already use this great work of @nicoinch and we will merge and release when it is widely tested.
I will be out of office from December 15th to January 15th, so I expect mid january that it will be released the earliest.

@nicoinch https://github.com/iMicknl/ha-nest-protect/issues/122 seems to be relevant to this PR, but fortunately has a promising traceback :)

@nicoinch would you need some support on this PR? I don't have time yet, but happy to have a look in a few weeks. Seems there are still some issues with this implementation, around not receiving updates.

@iMicknl I am very sorry but I do not have an Home Assistant instance anymore 😥 I won't be able to do fixes or changes to my PR as I won't be able to test them.

@iMicknl I am very sorry but I do not have an Home Assistant instance anymore 😥 I won't be able to do fixes or changes to my PR as I won't be able to test them.

I'm surprised someone deep enough in it to write a PR left the ecosystem! What did you switch to, if I might ask?

Replaced by #167, thanks for your contribution @nicoinch. Much appreciated.


Request for adding an explanation what would happened for SaaS subscription after canceling Azure subscription.

For the below MS Doc, that would be helpful if there is an explanation that SaaS subscription is not stopping charge if users cancel Azure subscription. SaaS subscription is totally different that Azure subscription. For example, users purchase SendGrid service through Azure Marketplace, and it means that users are not purchasing Azure service. This section explains only Azure service not Marketplace product which is SaaS subscription. Because of the documentation, users misunderstood if they cancel Azure subscription means that SaaS subscription also stops charging. It is not correct understanding, and it would be charging even though they think that SaaS subscription has been canceled.
What happens after subscription cancellation?
https://learn.microsoft.com/ja-jp/azure/cost-management-billing/manage/cancel-azure-subscription?source=recommendations#what-happens-after-subscription-cancellation

@Hirokato1
Thanks for your feedback! We will investigate and update as appropriate.


Use different Markdown parser

Preferably mdast. Make sure it supports HTML.

As far as I can tell, mdast can't parse HTML out of the box and I didn't find a plugin for it. But I think this is not that essential. But maybe @naehrstoff can give a few examples where this would have been useful in the past?
However, mdast seems to be the most extensible option out there. It has some cool plugins:

https://github.com/wooorm/mdast-slug (useful for #28)
https://github.com/wooorm/mdast-zone (could be used for #29)

Ping me for help! mdast too, however, doesn’t care about strings of HTML (or their validity), and in fact I’m not very interested in adding that as it’s beyond the scope of markdown, and possible even quite dangerous.
Most importantly, mdast is indeed quite extensible, and I’m always interesting in more ideas :smile:

We've mostly encountered this in polishing styleguides:

We're not able to use text like "Me & Co." because this will be turned into and displayed as "Me & Co." or something. Somewhat solved by using "Me + Co."
Ensure nice typography as in Mrs. Ripley → Mrs. Weaver. Might be able to use Unicode for this instead
Add special elements unique to a given styleguide as in Wow

These are small annoyances and not that important individually, but they accumulate and differ from what is to be expected from Markdown, so they are quite unexpected to novices and slightly annoying to more experienced writers.
@wooorm Thanks for offering your help! If we were to tackle this issue, we'd have to be able to somehow turn HTML elements within, say, a Markdown paragraph into React components. But this will probably add too much unnecessary complexity … I guess what it means for mdast is that the AST would have to go deeper and turn HTML within said paragraph (e.g. the & or ) into a syntax-tree-thing. We'd then need to write a plugin that turns these things into React components.
Markdown usually doesn't have to concern itself with this transformation because it's just HTML, so content within the paragraph will get parsed by the browser just fine, but in our case we'd need to be able to turn it into React components. Sigh.
Yeah, not sure this is worth pursuing, just wanted to explain a bit of the reasoning behind it :smile:

I did some research and experiments, and came to the conclusion that currently it's not worth it to change the Markdown engine.
I mainly looked at remark wit remark-react (formerly mdast) but it has two major drawbacks:

Just replacing marked with remark adds 91KB to the minified bundle size
I added remark to the benchmark suite at https://github.com/markdown-it/markdown-it, and it's ~30 times slower than marked (maybe I did something wrong?).

These are quite the deal breakers for me.
Another good contender would be markdown-it but it's also a lot bigger than marked. Its main advantage is full CommonMark compliance but I don't think that's super-important for us at this moment.
Therefore I suggest we just keep marked (it's not very actively developed but it still works).


Adding new functions to account.py

New functions

account_set_private -> It makes the account private
account_set_public -> It makes the account public

@bilakos26 thank you!