Add return values to type_of API docs

As mentioned in a comment on #799, the type_of docs should list the possible type values, show the return value in the example, and probably link to the data types document.

:+1:


Update description of eslint-plugin-import and eslint-plugin-import-x

eslint-plugin-import now supports both flat config (v2.30.0) and ESLint 9 (v2.31.0) (https://github.com/import-js/eslint-plugin-import/releases), so I removed the sentences that say it doesn't.
Removed "properly maintained" from the description of eslint-plugin-import-x, which is just passive aggressive language from someone unhappy with the way eslint-plugin-import is maintained, and added that eslint-plugin-import-x drops backwards compatibility, which is one of the main reasons for this fork (see https://github.com/un-ts/eslint-plugin-import-x/issues/24).

Thank you!


Add custom builder to reduce op allowing type inference

The PR implements the a custom reduce op builder similar to what we have for mhlo code.
Background
https://github.com/openxla/stablehlo/pull/1869 allows the block arguments of reduce op to have different element types than that of the input arguments of reduce op and the output element type of the reduce op has to equal to those block arguments. As a consequence the output type of reduce op can no longer be inferred from the operand types. The auto-generated builders creates a reduce op with empty block and, as  a result, does not allow inferring the type.
The proposed solution is to create a custom builder which takes the element-type of the block arguments as arguments allowing result type inference.

@sdasgup3 Gently checking-in on this, and appreciate your help here. Unfortunately cherry-picking from your fork doesn't play well with the git submodule setup in torch-mlir which expects a commit on main, so I can't proceed with the fix without this landing.
 Error: fatal: Fetched in submodule path 'externals/stablehlo', but it did not contain f2b6877713027370f26569929fb5d841a79d3589. Direct fetching of that commit failed.


 Convert test_wl_upgrade to go

Backports:
v2.9 https://github.com/rancher/rancher/pull/46911
v2.8 https://github.com/rancher/rancher/pull/46910
v2.7 https://github.com/rancher/rancher/pull/46875

i think this needs a rebase, otherwise lgtm

Yes, I'll rebase again


Archive Format

Have you considered using webarchive format and or pdf for archiving? .war is pretty standard now

Hey,
The current approach is basicially just downloading the html pages (with resources like images, stylesheets & javascript) and putting them in directories.
The goal is to save the websites in their original form, so I didn't want to convert them straight into another format (like pdf). Do you mean something like a button to download a screenshot of the website as pdf file (or a war file of the site) on the details page?

maybe options to convert the html to pdf on download and option to save as warc too


Update zh-CHT translation

Add new translation

Sure, but it seems that @eFiniLan is already working on this.
I'll just close my PR then.
His translation also looks good :)


[SPARK-10707] [SQL] Fix nullability computation in union output

Merged build triggered.

Merged build started.

Test build #44462 has started for PR 9308 at commit f444ae8.

Test build #44462 has finished for PR 9308 at commit f444ae8.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #44468 has started for PR 9308 at commit b27bc75.

Test build #44468 has finished for PR 9308 at commit b27bc75.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class Except(left: LogicalPlan, right: LogicalPlan) extends SetOperation(left, right) \n

Merged build finished. Test FAILed.

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

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

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #44480 has started for PR 9308 at commit 6958a49.

Test build #44480 has finished for PR 9308 at commit 6958a49.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class Except(left: LogicalPlan, right: LogicalPlan) extends SetOperation(left, right) \n

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

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #44482 has started for PR 9308 at commit 319d105.

Test build #44482 has finished for PR 9308 at commit 319d105.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class Except(left: LogicalPlan, right: LogicalPlan) extends SetOperation(left, right) \n

Merged build finished. Test PASSed.

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

@mbautin do you want to update the pull request?

@rxin updated -- sorry for the delay.

Test build #46563 has started for PR 9308 at commit 221f21b.

Test build #46563 has finished for PR 9308 at commit 221f21b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class Except(left: LogicalPlan, right: LogicalPlan) extends SetOperation(left, right) \n

Merged build finished. Test PASSed.

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

thanks - cc @cloud-fan

LGTM except a minor code style comments.

I'm going to merge this. The style thing is good to have, but probably doesn't worth another roundtrip.


Ansible [WARNING]: kubernetes<24.2.0 is not supported or tested

While running the provisioning playbook, the following warning is displayed:
[WARNING]: kubernetes<24.2.0 is not supported or tested. Some features may not work.

This is a warning from the python3-kubernetes version, running on the destination machine. See https://github.com/Homebrew/homebrew-core/issues/164335 for more details.
~$ apt show python3-kubernetes
Package: python3-kubernetes
Version: 12.0.1-1ubuntu1

~$ python3 -c 'import kubernetes; print(kubernetes.__version__)'
12.0.1

The current package version was thoroughly tested and performs as expected.

python3-kubernetes was bumped to version 22.6, in Ubuntu Server LTS 24.04.

I have this warning too on Debian 12.5.


None types are not preserved

As @vmarkovtsev pointed out in #534, None types are currently stripped from the tree.
I don't think the standard has anything to say about this at the moment. Would it be useful if None values are preserved in the tree? This would probably require the definition of a None or Null tag. I need to think through the implications a little more.
cc @perrygreenfield

This appears to be an issue with ASDF and not with the underlying YAML implementation:
>>> import yaml

>>> tree = dict(a=1, b='hello', c=None)
>>> result = yaml.dump(tree)
>>> result
'a: 1\nb: hello\nc: null\n'
>>> yaml.safe_load(result)
{'a': 1, 'b': 'hello', 'c': None}

So this is probably a bug and should be addressed.

I just ran into this issue as well. I think things go wrong when converting the tree to a tagged tree.
>>> import asdf
>>> tree = {"a": 1, "b": None}
>>> af = asdf.AsdfFile(tree)
>>> af.tree
{'a': 1}
>>> tree_tagged = asdf.yamlutil.custom_tree_to_tagged_tree(tree, af)
>>> tree_tagged
{'a': 1}
>>> tree
{'a': 1, 'b': None}

I haven't been able to check, but it seems like in line 138 of treeutil.walk_and_modify there is an explicit check that does not allow dict values to be None. I don't know whether removing this check would cause any other issues that I cannot oversee?

The callback is allowed to return None to indicate that the key should be removed.  It seems like that behavior is intentional, since there's a test for it.
@jdavies-st maybe we should define a special value that can be returned instead of None, something analogous to NotImplemented?

It's not clear why None is being treated as a special case.  Certainly None is used in special ways in python, but it is not clear that asdf should be treating it specially.
The fact that serialization with None values does not round trip is definitely not desirable.

So based on discussions with Perry and an email from Mike, it seems the driver of None being dropped is that it is interpreted as a "missing" value.  This is driven by the fact that ASDF allows the definition of a default value in the schema.  So if the attribute in the tree is missing, but the schema provides a default value, internally None signifies it is missing it gets filled with the default value.
If you're not using schemas to validate your ASDF tree, then this probably unexpected.
I think we need to think more about how we might make None round-trip-able and still indicate a value is "missing" and therefore might need a default value that the schema provides.

This just bit me as I expected None to be stored in the tree but it wasn't and my parser choked on a non-existing key. Would it not be better to perhaps tag missing values with nan and leave None intact?

I think some solution is needed to deal with this. NaN is probably not the right solution but perhaps some asdf-defined special value indicating missing.

Alternatively, would you consider implementing a get() method (similar to dict's get()), so that we could do:
data = asdf.open('file.asdf')
var = data.get('key', None)


data update

Delivery NB included

Missing values are indicated by a colon. Numbers may be accompanied by a 1- or 2-letter code with the following meaning:
b: break in time series
bp: break in time series, provisional
p: provisional
e: estimated
ep: estimated, provisional
For the purpose of this study, this information is ignored and hence the characters removed, facilitating conversion of the data to numeric.

These lines have been pasted twice. Was it intentionally?

In the dataLang DataFrame we have nPopulation | nEmigrants | nImmigrants | nInternetUsers columns. From which year does this data come from? Or is it an aggregate?

Regarding the immigration/emigration/population data line plots: I think it might be nice to use a big enough colour palette s.t. every country can have its own colour. E.g. for the immigration plot, it's hard to know whether the "highest" dark blue line is Sweden, Spain, or in an unlikely case even Luxembourg.


Beaker browser creates very big files

Beaker v0.8.0-prerelease.7
Operating system: Windows 10 x64
This is:

[x] a bug
[ ] general feedback

Hello. I noticed that my C:\ drive ran out of free space very quickly.
found 3 big files. 3.3 GB, 1.5 GB, 1.8 GB
C:\Users\yura\AppData\Roaming\Beaker Browser\Dat\Archives\Meta\e5\7214d9837d16b1f07e57707effccb853dc49acd79caa57c4cb5ca2c572d2d9\content\data
C:\Users\yura\AppData\Roaming\Beaker Browser\Dat\Archives\Meta\6d\27e0e07c06a453fd1676ec2fc51417e3bb9f819a62f77185b5e1d68b03f638\content\data
C:\Users\yura\AppData\Roaming\Beaker Browser\Dat\Archives\Meta\69\72ffe4bb8d24c63c92b91ef3aebf400c4899db8d5020adb96d9c8d46d7609c\content\data
Please, fix or tell me what is going on. :)
ps: cache was cleared. And total size of all seeding sites in a Library is not more then 130 MB.

Is it possible you have some sites that you've created or saved which have a lot of data in them? Currently Beaker retains the history of changes for sites and you may be seeing that.
This is something we need to improve, but haven't had the time to hit yet.

i have 3 seeding sites:

beakerbrowser.com
Radical Networks
my own site, very small and new (no history).
I had also some simple html5 game in seeding list, but then removed it.
I have visited some sites that have been listed in "explore-the-p2p-web" list.  but i think that beaker should clear it with cache clear function.
does it download all the "dat history" even if I'm not choosing to seed a site?

ps: sorry for my English.

but i think that beaker should clear it with cache clear function.

Yes - it's possible that clear cache isn't working as expected, but I'd want to check first whether there are saved sites that could be causing the problem.

does it download all the "dat history" even if I'm not choosing to seed a site?

No, only if you seed or create the site.

@veksha I just deployed a space reducing technique but I'm curious whether it'll apply to your use-case. Could you do this https://github.com/mafintosh/supports-sparse-files#cli and tell me what it says? (I'm curious if your filesystem supports sparse files or not.)

C:\Users\yura>supports-sparse-files
Sparse files are supported :)

Huh okay.
So I visited each of those sites.

The first one is dat://peer-to-peer-web.com/, which is 2.8GB current and probably more with metadata & history.
I can't access the second one
The third one is dat://dat-cast.hashbase.io/, which is 1.92GB current.

Just to confirm: Are you sure you're not seeding any of those? And, clearing cache doesn't remove them?
(It's odd that your Beaker would have downloaded those full dats without you having seeding them.)

Ok cool. Well we have some ways to improve file usage coming (version history management) and we should get some better interfaces for showing your space-usage, but for now this is all working as expected -- those large files were the sites' data -- and I'm not sure there are any signs of bugs, so I'm going to close.
Thanks for reporting. Please do reopen if you think anything's amiss.


Naming of Authorization Parameters.

Current naming of Authorization parameters is Security:Authentication:*, e.g. Security:Authentication:Mode, etc. It may make sense to rename to Security:Authorization:*

Moving work item tracking to Azure DevOps.


Allow re-ordering of grid columns from settings drop down that lists them

Closed by https://github.com/softlayer/sl-ember-components/pull/344


Errno fix

Here is a fix for #461 and the associated test. I've done separate commits for the test and the bug fix. Not sure if you would prefer them to be squashed?

After a bit of tinkering and some more thought I am withdrawing this PR because I think this is not the correct solution: snmalloc should not be setting errno except for functions that are defined to do so probably the override wrapper is the correct place for this (at the cost of a nullptr check).

I don't want a branch on the fast path of malloc, and setting errno at the outer level will lead to this.  It has to be buried deep inside.

In that case setting errno should become part of snmalloc's API and we will have to save and restore errno in the posix_memalign implementation because it explicitly does not set it. Implementations of the other functions already check for alloc returning null and set errno, although the realloc implementation is a bit broken and I will fix it.

Does posix_memalign require it not to be set, or that you can't depend on it being set?
In particular, I read
https://sourceware.org/bugzilla/show_bug.cgi?id=83#c1
Which implies we can set it.

POSIX2018 is pretty liberal WRT errno:

It is never set to 0 by any POSIX function.
It is undefined after any call to a POSIX function that is not explicitly stated to set it.
It is undefined after a successful call to a POSIX function that is documented to set errno on failure.

posix_memalign is not defined to set errno, so we're free to set or not set it as we desire on failure.  On success we may set it to whatever we want as long as we do not transition it from a non-zero to a zero state.

To get this right, some of the other pals should probably set it.  Like
https://github.com/microsoft/snmalloc/blob/af8ab2daf6455fa1f9d9daedeb65f6abc2d2d50e/src/pal/pal_windows.h#L163-L196
Windows Pal should probably set errno when VirtualAlloc* returns nullptr.


Unable to update 2016-Datacenter-Server-Core to latest version

Build #20240314.03 for !38093 attempted to use the 14393.6796.240302 version of the 2016-Datacenter-Server-Core marketplace image. That failed while replicating the new image, after everything we control completed successfully.
Example error:
2024-03-14 10:55:34Z windows.10.amd64.core, windows.10.amd64.core.rt, windows.10.amd64.core.svc ==> Image creation has succeeded but all replications have failed. please see the imageReplication property output for more details (691ceb13-b6d7-ac5e-0d40-9dbbe0979cdb).
We need to investigate and resolve this problem. Without that, the image will fall behind and should not be considered secure.

Release Note Category

[x] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
TBD :grin:

This PR completed on a rebuild with build (#20240314.11 • VS2022 and base images update). I will verify if we can close this issue.
https://dev.azure.com/dnceng/internal/_build/results?buildId=2405034&view=results

@davfost, so, can this issue be closed❔

The original error has not reoccurred in the pipeline for 2 days and the original PR related to the original failed build shows as merged and complete.
https://dev.azure.com/dnceng/internal/_git/dotnet-helix-machines/pullrequest/38093
Closing the issue.


updated README and yml files

Description
The stand-alone ebpf and flp were implemented against an older version of flp.
This PR updateds the README and yml files to work with the current flp.
Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[ ] Will this change affect NetObserv / Network Observability operator? If not, you can ignore the rest of this checklist.
[ ] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

@jpinsonneau The flp-daemonset-cap.yml doesn't yet work properly. Some permissions probably need to be adjusted.

I'm getting flows in FLP after applying the new changes:
Starting flowlogs-pipeline:
=====
Build Version: -dirty
Build Date: 2024-01-17 17:34

Using configuration:
{
    "PipeLine": "[{\"name\":\"ingest\"},{\"follows\":\"ingest\",\"name\":\"enrich\"},{\"follows\":\"enrich\",\"name\":\"loki\"}]",
    "Parameters": "[{\"ingest\":{\"grpc\":{\"port\":9999},\"type\":\"grpc\"},\"name\":\"ingest\"},{\"name\":\"enrich\",\"transform\":{\"network\":{\"rules\":[{\"input\":\"SrcAddr\",\"output\":\"SrcK8S\",\"type\":\"add_kubernetes\"},{\"input\":\"DstAddr\",\"output\":\"DstK8S\",\"type\":\"add_kubernetes\"}]},\"type\":\"network\"}},{\"name\":\"loki\",\"write\":{\"loki\":{\"labels\":[\"SrcK8S_Namespace\",\"SrcK8S_OwnerName\",\"DstK8S_Namespace\",\"DstK8S_OwnerName\",\"FlowDirection\"],\"staticlabels\":{\"app\":\"netobserv-flowcollector\"},\"timestamplabel\":\"TimeFlowEndMs\",\"timestampscale\":\"1ms\",\"url\":\"http://loki.netobserv.svc:3100\"},\"type\":\"loki\"}}]",
    "MetricsSettings": "",
    "Health": {
        "Address": "<IP_ADDRESS>",
        "Port": "8080"
    },
    "Profile": {
        "Port": 0
    }
}
time=2024-01-19T12:41:23Z level=debug msg=opts.PipeLine = [{"name":"ingest"},{"follows":"ingest","name":"enrich"},{"follows":"enrich","name":"loki"}]
time=2024-01-19T12:41:23Z level=debug msg=stages = [{ingest } {enrich ingest} {loki enrich}]
time=2024-01-19T12:41:23Z level=debug msg=params = [{ingest 0xc000130f00 <nil> <nil> <nil> <nil>} {enrich <nil> 0xc0002bd080 <nil> <nil> <nil>} {loki <nil> <nil> <nil> <nil> 0xc0002bd0e0}]
time=2024-01-19T12:41:23Z level=info msg=using default metrics settings
time=2024-01-19T12:41:23Z level=debug msg=entering SetupElegantExit
time=2024-01-19T12:41:23Z level=debug msg=registered exit signal channel
time=2024-01-19T12:41:23Z level=debug msg=exiting SetupElegantExit
time=2024-01-19T12:41:23Z level=info msg=StartServerAsync: addr = :9090 component=prometheus
time=2024-01-19T12:41:23Z level=debug msg=entering newPipelineFromIngester
time=2024-01-19T12:41:23Z level=debug msg=stages = [{ingest } {enrich ingest} {loki enrich}]
time=2024-01-19T12:41:23Z level=debug msg=configParams = [{ingest 0xc000130f00 <nil> <nil> <nil> <nil>} {enrich <nil> 0xc0002bd080 <nil> <nil> <nil>} {loki <nil> <nil> <nil> <nil> 0xc0002bd0e0}]
time=2024-01-19T12:41:23Z level=debug msg=stage = ingest
time=2024-01-19T12:41:23Z level=debug msg=findStageType: stage = ingest
time=2024-01-19T12:41:23Z level=debug msg=pipeline = [0xc00024c7e0]
time=2024-01-19T12:41:23Z level=debug msg=stage = enrich
time=2024-01-19T12:41:23Z level=debug msg=findStageType: stage = enrich
time=2024-01-19T12:41:23Z level=debug msg=starting kubernetes informers, waiting for synchronization
time=2024-01-19T12:41:23Z level=debug msg=kubernetes informers started
time=2024-01-19T12:41:23Z level=debug msg=starting kubernetes metadata informers, waiting for synchronization
time=2024-01-19T12:41:23Z level=debug msg=kubernetes metadata informers started
time=2024-01-19T12:41:23Z level=debug msg=pipeline = [0xc00024c7e0 0xc0001a2a10]
time=2024-01-19T12:41:23Z level=debug msg=stage = loki
time=2024-01-19T12:41:23Z level=debug msg=findStageType: stage = loki
time=2024-01-19T12:41:23Z level=debug msg=entering NewWriteLoki component=write.Loki
time=2024-01-19T12:41:23Z level=debug msg=pipeline = [0xc00024c7e0 0xc0001a2a10 0xc0001a2d90]
time=2024-01-19T12:41:23Z level=debug msg=pipeline = [0xc00024c7e0 0xc0001a2a10 0xc0001a2d90]
time=2024-01-19T12:41:23Z level=info msg=connecting stages: ingest --> enrich
time=2024-01-19T12:41:23Z level=info msg=connecting stages: enrich --> loki
time=2024-01-19T12:41:27Z level=debug msg=Ingested 875 records component=ingest.GRPCProtobuf
time=2024-01-19T12:41:27Z level=debug msg=Ingested 920 records component=ingest.GRPCProtobuf
time=2024-01-19T12:41:32Z level=debug msg=Ingested 1069 records component=ingest.GRPCProtobuf
time=2024-01-19T12:41:32Z level=debug msg=Ingested 1027 records component=ingest.GRPCProtobuf
time=2024-01-19T12:41:37Z level=debug msg=Ingested 1125 records component=ingest.GRPCProtobuf
time=2024-01-19T12:41:37Z level=debug msg=Ingested 844 records component=ingest.GRPCProtobuf
time=2024-01-19T12:41:42Z level=debug msg=Ingested 1094 records component=ingest.GRPCProtobuf

I'm getting flows in FLP after applying the new changes:

To get rid of these messages, we need to change log-level from debug to info or error.

@jpinsonneau Can you merge your commit with this PR?

It's already part of this PR, we can merge as is to me.
Thanks @KalmanMeth !


Clarify LeaveCriticalSection guarantees

In nf-synchapi-deletecriticalsection.md the following was stated:

The caller is responsible for ensuring that the critical section object is unowned and the specified CRITICAL_SECTION structure is not being accessed by any critical section functions called by other threads in the process.

This puts a note on a CRITICAL_SECTION structure access, but it was never stated, how this structure is accessed by LeaveCriticalSection.
The proposed change ensures that this structure is only accessed until a critical section object ownership is released.

@FooBarrior : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Karl-Bridge-Microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Inheritance chain incorrect

In C#, IBindableObservableVector inherits from IList. So this:
IList -> IEnumerable -> IBindableObservableVector

should be this:
IBindableObservableVector -> IList -> IEnumerable

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

ID: aae2fa96-10e8-9b16-8b8f-a6f00167fcbd
Version Independent ID: 24e07c30-b1fc-d107-6bcf-6f572559cf14
Content: IBindableObservableVector Interface (Windows.UI.Xaml.Interop) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.interop.ibindableobservablevector.yml&version=GBlive&_a=contents
Product: uwp
Technology: winui
Git Login: jwmsft
Microsoft Alias: jimwalk

I think this issue arose from automatic formatting in the API ref docs. I believe it has been fixed since this issue was reported, as I no longer see it.
If there's still a related problem on this page, please open a new issue detailing it. Thank you.

I think this issue arose from automatic formatting in the API ref docs. I believe it has been fixed since this issue was reported, as I no longer see it.
If there's still a related problem on this page, please open a new issue detailing it. Thank you.


[CAMEL-9414] update quickfixj to v1.6.1

quickfixj is now an OSGi bundle and available on marketcetera maven repo
  <repositories>
    <repository>
      <id>marketcetera</id>
      <url>http://repo.marketcetera.org/maven</url>
      <snapshots>
        <enabled>false</enabled>
      </snapshots>
    </repository>
  </repositories>

Is this JAR only in that repo, and not in maven central ?

At the moment yes, if I'm not mistaking there are some plans to upload future version on maven central.

QuickFIX/J  will be made available on Maven central I think next release ( see QFJ-846 ) so I've submitted a task (SM-2773) and PR ( #38) to create an OSGi bundle for 1.6.1.
I'll rework this PR as soon as the bundle will be available.


Docker-compose config

Hello.
I have writtend down
version: 3.8
services:
  jellyfin:
    image: nvllsvm/jellysub
    container_name: jellysub
    JELLYSUB_UPSTREAM_URL: https://my.jellyfin.server
    JELLYSUB_HTTP_PORT: 1234:1234
    restart: "unless-stopped"

in my docker compose but it's not working. I got this error :

Unsupported config option for services.jellyfin: 'JELLYSUB_HTTP_PORT'

How should I name the parameters ?

Environment variables are specified like so in in Docker Compose:
version: 3.8
services:
  jellyfin:
    image: nvllsvm/jellysub
    container_name: jellysub
    environment:
      JELLYSUB_UPSTREAM_URL: https://my.jellyfin.server
      JELLYSUB_HTTP_PORT: 1234:1234
    restart: "unless-stopped"


nou.edu.ng

@chimex61 First stage of verification passed, official domain is nou.edu.ng as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@chimex61 Pull request merged. Thank you.
Please notice that due to a very large number of students of the university (the website says there are over 500 000 of them), we'll closely monitor the license requests from this university. Shall we suspect that some people who are not the university students or teachers access the free license via this university, we'll have to withdraw the university from the trusted list.
Thank you for your understanding.


Fix Coverage

Duplicate PR to test the communication with coveralls.

I was able to get a response from coveralls after sending the request from my machine. I am trying to figure out whether there is an issue with the workflow or with the authentication.

Looks good now, right?

I think this was not successful. If I understand @nkaralis correctly, he pointed out that commit https://github.com/rdf4cpp/rdf4cpp/pull/109/commits/894732eab606eb772bfb5d4d43b5b7ea15943794 reports coverage only because he submitted the results manually.


Add  auto-updates flag to some casks

Add auto-updates flag to the following casks:

appzapper
colloquy
evernote
firefox
hipchat
skitch
skype
slack
spotify
virtualbox

In the future, please submit separate pull requests for each cask, as more often than not this type of pull request brings problems with one or more of them, and it halts the inclusion of the other ones.
In this case, you’ll see the build will fail because you’ve positioned the stanza arbitrarily, and in some cases it does not follow the necessary order.

Sorry, I thought I followed the ordering. I'll close and redo each of these individually.

You can do a single PR, but at least do a commit for each cask.


HSEARCH-1765 Add support for deleting by Query

https://hibernate.atlassian.net/browse/HSEARCH-1765
I hope I fixed all of the issues stated in the old PR. I will have to rebase this PR after HSEARCH-1795 is merged into master, but this is ready to be reviewed again. The duplicated Commits in here will be removed after hardy's patch is applied as well.

uhm either Jenkins is broken or something unrelated to my changes just broke.

rebased :)

any updates on this?

rebased :)

I've merged it: looks very good.
Since all commits except the first one were just making corrections to the first one, I've squashed them in a single commit. We don't dislike multiple commits, but only if they help understanding the flow of changes - not needed if they amend mistakes.
I still think the code in DeleteByQuerySupport could use a little more polish, but it's not so bad that it should delay having this feature.. so I've opened a separate issue to do that eventually: https://hibernate.atlassian.net/browse/HSEARCH-1841
Thanks! great to have this

thanks for merging :)


Implement App Mesh v1beta2 timeout

This PR adds support for setting AppMesh listener/route timeouts with canary.spec.service.timeout.
Ref: #577

Codecov Report

Merging #611 into master will increase coverage by 0.00%.
The diff coverage is 55.55%.

@@           Coverage Diff           @@
##           master     #611   +/-   ##
=======================================
  Coverage   55.00%   55.00%
=======================================
  Files          62       62
  Lines        5303     5330   +27
=======================================
+ Hits         2917     2932   +15
- Misses       1959     1967    +8
- Partials      427      431    +4

Impacted Files
Coverage Δ

pkg/router/appmesh_v1beta2.go
84.63% <55.55%> (-2.29%)
:arrow_down:

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 c367e65...37ec07d. Read the comment docs.


Wandering Warrior - Add basic code to make it work

Ok, time to make that basic list:

Create mod helper; - OK
Update libraries; - OK
Add loaded mods in SlayTheSpire.java receiveCustomModeMods(List list); - OK
Update receiveEditStrings() - OK
Update receivePostCreateStartingRelics(...) - OK
Update receivePostDungeonInitialize() - OK
Patch ModHelper to add Wandering Warrior as a Starter Mod.
Update the versions of the libraries used, in the repository

Patch ModHelper to add Wandering Warrior as a Starter Mod. <- Is needed? Time to test, let's go.

WORKS!

Done.


Localize to mandarin

Let's focus on languages spoken in countries that use the EAN/UPC system.
Closed for now.


make l99 command installation optional

ob: I prefer not letting apps updating my .bashrc file (as you do in shell/install/cli), therefore I appended my .bashrc with return. I think it could be better to output (in shell/install/cli log) what should be done by the user manually. YMMV.
ob: Well, I think that when we run a program/app without installing it, we should use either absolute or relative path to l99 script. However, when we install a program/app (i.e. copy it to the host file system e.g. into /usr/bin with other data elsewhere, like C libraries should go /usr/lib), then l99 should be found in PATH and thus accessible without path specification.
ob: Moreover, when I run some/path/l99 directly without the env vars defined, it errors out that I have not set it up, albeit it is set up that way. IMO it should run as expected, whether it is run directly (with path specified) or without it and all env vars should be optional (used only when we want to override something).
bb: Ah, that makes sense - okay we should let user run './l99' by itself, or 'shell/install/cli' to their system and run as 'l99'. And should have default values for the envars.

This issue is related to #185.
We can already do something like this:
repo_root='some/path/to/a/folder'
# git clone https://github.com/Ladder99/ladder99 "$repo_root"  # -b v0.10.1
git clone git@github.com:Ladder99/ladder99.git "$repo_root"  # -b v0.10.1
cd "$repo_root"

PATH="$PATH:$repo_root/shell" L99_HOME="$repo_root" l99 use example
PATH="$PATH:$repo_root/shell" L99_HOME="$repo_root" l99 start
PATH="$PATH:$repo_root/shell" L99_HOME="$repo_root" l99 status


[ENHANCEMENT] allow to set revisionHistoryLimit

Describe the use case:
We are using ArgoCD and old replicaset only clutter the interface without any benefit. If we were able to set revisionHistoryLimit to "0" for deployments, replicasets without any pods would be removed.
Describe the enhancement/feature:
Add the following values to the values.yaml and render them into the corresponding Deployment spec.
connectors.revisionHistoryLimit: 10
console.revisionHistoryLimit: 10
identity.revisionHistoryLimit: 10
operate.revisionHistoryLimit: 10
optimize.revisionHistoryLimit: 10
tasklist.revisionHistoryLimit: 10
webModeler.restapi.revisionHistoryLimit: 10
webModeler.webapp.revisionHistoryLimit: 10
webModeler.websockets.revisionHistoryLimit: 10
zeebeGateway.revisionHistoryLimit: 10
Desired outcome and acceptance tests:
revisionHistoryLimit in every Deployment can be set via values.yaml.

@helmecke-nlb Thanks for the suggestion.
Currently, we are considering providing users a generic way to define any extra config they need.
Probably this will be part of the 8.6 release in October (maybe also before it).


External text editor does not open unless Godot quits

Godot version:

master branch commit 65a86ae17c442163c424fe4b229a64d48da0fc89
OS/device including version:

Arch Linux x86_64 - gcc (GCC) 8.2.1 20181127
Issue description:

After trying to run the Finite State Machine example in the demos, which currently has faulty GDScript code, I expected my external editor to run but this was not the case. I placed some debugging code in OS_Unix::execute and turns out that the execvp attempt is actually another command:
Attempting to execute: /opt/godot/bin/godot.x11.opt.tools.64 --path /home/wilson/gitrepos/godot-demo-projects/2d/finite_state_machine --remote-debug <IP_ADDRESS>:6007 --allow_focus_steal_pid 1863 --position 320,180 (null)

For some reason, whether the project was not stopped (i.e. if F8 is not pressed) or not, and Godot quits, the next time Godot opens, it attempts to run my external editor:
Attempting to execute: xterm -bg black -fg white -sl 100000 -e vim +"call cursor(0, 0) /home/wilson/gitrepos/godot-demo-projects/2d/finite_state_machine/player/weapon/sword.gd (null)

For some reason I also had to escape that + symbol with a backslash \ in the argument list within the editor.
For what is worth, here's the code I placed right before the execvp call in OS_Unix::execute:
		fprintf(stderr, "Attempting to execute:");
		for (int i = 0; i < args.size(); i++)
			fprintf(stderr, " %s", args[i]);
		fprintf(stderr, "\n");

Steps to reproduce:

Change external editor after opening project using godot -e from within the project path
Attempt to run Finite State Machine from the official demos (press F5), if the scene runs fine, place some faulty GDScript code.
When the faulty GDScript code is detected, the external editor should have opened at the offending line but it doesn't.
Close Godot
Reopen the project and attempt to run Finite State Machine demo again after opening project using godot -e from within the project path, the external editor still doesn't open after detecting the faulty GDScript code.

Minimal reproduction project:
See the Finite State Machine example in the godot-demo-repository repository at commit eb27a35a2d5212abfc516aa3c8fcb76c907155f0

Is this still reproducible in the current master branch / latest 3.2 builds?

I'll test this next week against a master branch build.

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?

Can confirm this is still happening in the 3.2 branch at commit 68013d23932688e57b489600f4517dd280edc464. The faulty GDScript code detection is still not opening up the external editor. Not even when restarting Godot now. I've updated the steps to reproduce accordingly.


Error running demo

I tried using Bioboxes for the first time using the demo commands.  It ran for about 45 seconds before erroring with the trace below:
$ pip2 install --user biobox_cli
[ lots of output, ran successfully ]

$ /home/jorvis/.local/bin/biobox run short_read_assembler bioboxes/megahit --input SRS015996.denovo_duplicates_marked.trimmed.1.fastq --output megahit_test_dir1only
Traceback (most recent call last):
  File "/home/jorvis/.local/bin/biobox", line 9, in <module>
    biobox.run()
  File "/home/jorvis/.local/lib/python2.7/site-packages/biobox_cli/main.py", line 30, in run
    util.select_module("command", opts["<command>"]).run(args)
  File "/home/jorvis/.local/lib/python2.7/site-packages/biobox_cli/command/run.py", line 26, in run
    ctnr = bbx.run(argv)
  File "/home/jorvis/.local/lib/python2.7/site-packages/biobox_cli/biobox_helper.py", line 42, in run
    self.after_run(output, output_dir)
  File "/home/jorvis/.local/lib/python2.7/site-packages/biobox_cli/biobox_type/short_read_assembler.py", line 36, in after_run
    biobox_output = fle.get_biobox_file_contents(host_dst_dir)
  File "/home/jorvis/.local/lib/python2.7/site-packages/biobox_cli/biobox_file.py", line 9, in get_biobox_file_contents
    with open(os.path.join(dir_, 'biobox.yaml'), 'r') as f:
IOError: [Errno 2] No such file or directory: '/tmp/tmpbSgC_Y/biobox.yaml'

Thanks for the bug report Jorvis. Could you tell me what operating system you are using?
Could you also provide the logs of the container you ran? E.g. docker logs ...

The output of docker info would also be helpful. Thanks.

I'll try to reproduce.  I actually just upgraded that laptop to Ubuntu 16.04, so the environment has now changed.

Ok thank you. If you can diagnose the bug that would be very helpful. We
run unit and feature tests on debian so if you do find the cause of the
problem we can try to update the biobox_cli to cover this.
On 3 August 2016 at 08:51, Joshua Orvis<EMAIL_ADDRESS>wrote:

I'll try to reproduce. I actually just upgraded that laptop to Ubuntu
16.04, so the environment has now changed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bioboxes/command-line-interface/issues/102#issuecomment-237261284,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAEdv1dLtI0HUlwP5xukMb9SYV9puNCks5qcLkVgaJpZM4JZK09
.

I don't want to drop the ball here, but I did test again on my new OS (Ubuntu 16.04) with a new install of bioboxes and the same issue happened:
$ biobox run short_read_assembler bioboxes/megahit --input SRS015996.denovo_duplicates_marked.trimmed.1.fastq --output megahit_test_dir1only
Traceback (most recent call last):
  File "/usr/local/bin/biobox", line 9, in <module>
    biobox.run()
  File "/usr/local/lib/python2.7/dist-packages/biobox_cli/main.py", line 30, in run
    util.select_module("command", opts["<command>"]).run(args)
  File "/usr/local/lib/python2.7/dist-packages/biobox_cli/command/run.py", line 26, in run
    ctnr = bbx.run(argv)
  File "/usr/local/lib/python2.7/dist-packages/biobox_cli/biobox_helper.py", line 42, in run
    self.after_run(output, output_dir)
  File "/usr/local/lib/python2.7/dist-packages/biobox_cli/biobox_type/short_read_assembler.py", line 36, in after_run
    biobox_output = fle.get_biobox_file_contents(host_dst_dir)
  File "/usr/local/lib/python2.7/dist-packages/biobox_cli/biobox_file.py", line 9, in get_biobox_file_contents
    with open(os.path.join(dir_, 'biobox.yaml'), 'r') as f:
IOError: [Errno 2] No such file or directory: '/tmp/tmph9K5zf/biobox.yaml'

What exact log can I help by sending?

Could you paste the output of docker logs ... using the ID of the
bioboxes/megahit container? You can get this ID using docker ps -a.
Thanks for following up.

I had tried 'docker ps' and got nothing before posting, but it seems the -a was the critical missing option.  OK, the output was:
gzip: /bbx/mount/oB4AnDJr8RR2qrrnq3W0V57z/SRS015996.denovo_duplicates_marked.trimmed.1.fastq: not in gzip format
max_read_len must be greater than 1.

So it appears the megahit biobox requires the input to be in compressed format. It worked after I changed this.  Is there a way to get the error to propagate up rather than just showing the YAML error?

Thanks for posting the log and working out what the problem was.
Yes, I agree it would be ideal if the error message was more descriptive. Generally when the biobox.yaml cannot be found it means that something has gone wrong inside the container. A more descriptive error message from the would be useful, perhaps directing the user to look at the container logs if there is a non-zero exit code?

Also to follow up on .gz requirement for the input files, this is one we have added to the biobox specification. This is for expediency as the less input formats we have to support the easier it is to create more bioboxes as less time is required for each.


Fix AT_ constants for DragonFly

@bors: r+

:pushpin: Commit abdeff3 has been approved by alexcrichton

:hourglass: Testing commit abdeff3 with merge 50c5a33...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 50c5a33fb69a0cae0ad448fbcee31733dd11b26d to master...


Suggestion - App naming - Avoid any legal conflict or 'war' with Google

Hi, i think you should consider to rename your application to comply with the guidelines for Android app naming. I know that the app is not in the PlayStore - Still you should consider to change it, because it could trigger a conflict with google resulting in a possible block of all third-party AA apps.
Since you said yourself on your blog that google might have blocked your previous apk this could be a reason..
https://play.google.com/about/developer-content-policy-print/
https://stackoverflow.com/questions/28755825/android-app-naming-guidelines-first-word-as-company
Youtube Auto especially with that logo is probably an impersonation.
For example you could name rename it:

CarPlayer for YouTube
whatever for YouTube

That Package name was a desperate attempt to unblock the app, but I guess google got bored of this now. So we are good.


Example does not properly demonstrate command

The sample has formatting issues.
If you run the commands in the sample, it will not produce the results shown.   Not clear if this is just outdated or sample expects some external configuration change.

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

ID: f0d5ae0d-56da-7ea2-4628-00727dc340d8
Version Independent ID: 1ded5d7f-1df5-48b8-abd7-4c78c90ad538
Content: Reset-IISServerManager (iisadministration)
Content Source: docset/windows/iisadministration/reset-iisservermanager.md
Product: w10
Technology: powershell-windows
GitHub Login: @coreyp-at-msft
Microsoft Alias: brianlic

@officedocsbot assign @e0i


ALM Toolkit - Error Deploy

When trying to do a simple deploy, I am making an error. Any deploy, from any database, I get an error message and depoloy is not performed. Can you help me ?
Erro Deploy AS.zip

Hi, this is a known bug in AAS. It is being investigated and hopefully will be fixed soon.
It is caused by related to special characters in object name references (e.g. Perspective table/column names that use certain special characters). You may be able to workaround this by deleting the DB, removing special characters like '.', '[', ']', ':', '$' and redeploying, but we’re still validating the details of the problem and will hopefully rollback the regression soon…

ok. Thank you, Chris.


jssrc2cpg + kotlinsrc2cpg: Refactor the local node creation logic

Current creation of Local nodes in JS & kotlin dosen't have columnNumber, lineNumber.
We used same method to create LocalNodes as in other frontends.

Looks good otherwise, just the print and comment that seem misplaced


Newbie question creating sidebar links

Hello,
I'm trying to figure out how to create links in the sidebar then once the link is clicked on it should load the content in the center. Please advise.
Thanks,
Dan

@dk4210 this is just a template, how did you code the content loading? Are you using a framework?
Please post an example on gist so we can get a look.

Hi,
This is a very broad question and difficult to answer. There are multiple ways to do this and it all depends on your preference. To get you started, you may want to check out AJAX.
Thanks!

Actually I'm using Ajax. Thanks
On Jan 16, 2016 11:07 AM, "Abdullah Almsaeed"<EMAIL_ADDRESS>wrote:

Hi,
This is a very broad question and difficult to answer. There are multiple
ways to do this and it all depends on your preference. To get you started,
you may want to check out AJAX http://www.w3schools.com/ajax/.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/almasaeed2010/AdminLTE/issues/871#issuecomment-172219030
.


components loaded, but not styled? Using Webpack

I am able to load the component and see it , but it is not styled. Am I missing something or did i not configure it correctly?
I used webpack and here is my config.js
module.exports ={

  entry: {
    welcomeForm: './public/app/app.jsx'
  },

  output: {
    path: "./public/js/",
    filename: "bundle.js"
  },
  module: {
    loaders: [
      { test: /\.jsx$/, loader: 'jsx-loader?insertPragma=React.DOM&harmony' },
      { test: /\.css$/, loader: "style-loader!css-loader" },
      { test: /\.scss$/, loader: "style!css!sass!" },
    ]
  },

  resolve: {
    modulesDirectories: ['web_modules', 'node_modules'],
    extensions: ['', '.js', '.jsx']
  }

};

I followed the example and my app looks something like this
 mui = require('material-ui'),
  RaisedButton = mui.RaisedButton;

 render: function() {
    return (
      <RaisedButton label="Default" />
    );
  }

I see a button that says default, but it is not styled in the material way.

You mush require css in the component.
That webpack can bundle css for you
+ require('./main.less');
 mui = require('material-ui'),
  RaisedButton = mui.RaisedButton;

 render: function() {
    return (
      <RaisedButton label="Default" />
    );
  }

In main.less, you must import the material-ui stylesheets
@import "~material-ui/src/less/scaffolding.less";
@import "~material-ui/src/less/components.less";

And because material-ui using less, so you need to add less-loader in your webpack.config.js
    { test: /\.jsx$/, loader: 'jsx-loader?insertPragma=React.DOM&harmony' },
    { test: /\.css$/, loader: "style-loader!css-loader" },
+   { test: /\.less$/, loader: 'style-loader!css-loader!less-loader' },
    { test: /\.scss$/, loader: "style!css!sass!" },

I'm not familiar enough w/ webpack to compile the CSS.
I ended up doing it manually and loading the main.css file in my default layout.
but its working now, thanks.

@paOol, the less loader documentation helped me understand how/why @waynelkh's answer works.

webpack provides an advanced mechanism to resolve files. The less-loader stubs less' fileLoader and passes all queries to the webpack resolving engine. Thus you can import your less-modules from node_modules. Just prepend them with a ~ which tells webpack to look-up the modulesDirectories

@import "~bootstrap/less/bootstrap";

It's important to only prepend it with ~, because ~/ resolves to the home-directory. webpack needs to distinguish between bootstrap and ~bootstrap because css- and less-files have no special syntax for importing relative files. Writing @import "file" is the same as @import "./file";

...and now with 0.8 out, all styling is done inline so you don't need the less any more.
I suppose this can be closed?

Thanks @wmertens
@paOol Please reopen if this is still an issue.


[Preview 9] Update EventArgs

React to https://github.com/aspnet/AspNetCore/pull/12660.

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

ID: be4c4d0c-149c-bde1-c7fa-f082916390ca
Version Independent ID: 9d945996-ad02-bd4b-58ea-c465cc5cc74c
Content: Create and use ASP.NET Core Razor components
Content Source: aspnetcore/blazor/components.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

I assume this is why the link "EventArgs classes in the reference source" now 404s?

@mv10-work Just a month ago I fixed that on https://github.com/aspnet/AspNetCore.Docs/pull/13617 ... you can see how long that lasted! :smile: I'll update the link again now. Thanks for letting me know.


503 after enabling Profiler

Hi!
I have a small .net core app. This app is hosted as a docker container in Kubernetes. When I enable Application Insights Profiller  then all request to this application falls in 503 error (Bad Gateway)
I tryed to enable profiling using both methods  "ENV ASPNETCORE_HOSTINGSTARTUPASSEMBLIES Microsoft.ApplicationInsights.Profiler.AspNetCore" and "services.AddServiceProfiler();"
But  the error  still remained. How can I investigate this problem?
I see in application logs only:
{"Timestamp":"2018-05-03T12:08:38.1261884+00:00","Level":"Information","MessageTemplate":"Service Profiler session started.","SourceContext":"ServiceProfiler.EventPipe.Client.Schedules.TraceSchedule","ConversationId":"(none)","ProfilingContext":"(none)"}

@ins4n333 Thank you for using Service Profiler. I am sorry to hear that it doesn't work for you and also for the delayed reply. Kubernetes hosted container should be working.
By the description, the app is working properly without enable Service Profiler but returns 503(Server Internal Error) when has Service Profiler Enabled, is that correct?
You said you are using both environment variable and calling services.AddServiceProfiler(). I am not sure if you are using both at the same time or you mean you tried both method. Because of bug #6 , you can use either but please do NOT use both at the same time. (Well, until the next release that contains the fix.)
Secondly, please set the log level to Debug to see if you could see some more traces:
    "LogLevel": {
      "Default": "Warning",
      "ServiceProfiler":"Debug"
    }

Please use gather the following details:

kubectl logs
kubectl describe

Please paste the result here, I will take a look.

Hi @xiaomi7732 ! I have already analazyed this problem. It turns out when I deploy application with enabled profiler my pods fall in OOM... It depends on the fact that I had had pods which are limited in CPU and MEMORY.  But the profiler itself requires some amount of memory for correct work. After that I increased memory limit of pod my application started fine and profiler was able to collect some traces. But when I tryed to open these traces using portal I received "Issue loading trace with correlation id: xnlayhmq".
Does this fact correlated with that I use .NETCORE 2.0 runtime (not 2.1)?

@ins4n333 Thanks for looking into the issue. Really appreciate it. With the correlation id you provided, I checked the telemetry on our end and it is the typical CLR 2.0 run-time failure.
I just want to mention here is that there is a new issue in our back-end to analyze CLR 2.1 traces because of a recent CLR change. The issue is tracked by #11 . Please hold it a bit if you want to try it on CLR 2.1.

@xiaomi7732 Thank you! I belive that AI profiler will be very useful tecknology for diagnostic perfmonce problems. Existense Linux tools for dotnet profiling (such as perf) are not very convenient..

Very pleased to hear that :-) Thanks again for using the profiler and give us the feedback.
I'll lock this conversation for references.
Feel free to open up new issues at anytime if there's more info.


user is receiving an invoice for less amount than what he donated

https://opencollective.com/finpo/transactions
he made a $250 donation, he's dowloaded invoice is for $215 (this is probably related to fees)
See email subject: "I can't find Receipt anymore ... since 2018/3" to see the invoices (they have PI)

cc/ @clarete

I think this is just an issue of using wrong field - when a collective of type USER or ORG, we need to use netAmount, instead of Amount.

I also need to test generating the invoice for a collective too, but here's what I've gotten so far:
https://github.com/opencollective/opencollective-website/pull/790

Looks like the bug is elsewhere. This is pulling the invoice for the wrong transaction. The collective is the user/org instead of the collective that received the money. I'll look into it more.


Attempted to fix the performance

PR is currently in a broken state due to the fact that I do not have the time to fix all the config.json dependency issues that arise from packaging.


Change SIL ref_element_addr getFieldNo() to return a unique index.

The fix is needed to have a unique AccessStorage object for each property. The AccessPath commits will contain test cases for this functionality.
If you're looking at the diff, only look at the first commit.

@swift-ci test

@swift-ci test source compatibility

@swift-ci benchmark

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
2626
2913
+10.9%
0.90x (?)

FlattenListLoop
946
1031
+9.0%
0.92x (?)

Code size: -O
Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

NSStringConversion.UTF8
603
661
+9.6%
0.91x (?)

Improvement
OLD
NEW
DELTA
RATIO

FlattenListLoop
1543
1415
-8.3%
1.09x (?)

Code size: -Osize
Performance: -Onone

Improvement
OLD
NEW
DELTA
RATIO

TypeFlood
165
154
-6.7%
1.07x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac mini
  Model Identifier: Macmini8,1
  Processor Name: 6-Core Intel Core i7
  Processor Speed: 3.2 GHz
  Number of Processors: 1
  Total Number of Cores: 6
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 64 GB

I can't reproduce the SCK failures.

@swift-ci test source compatibility

SCK failures appear unrelated to this PR


Evaluated performance of the nf1 genotype classification model

In this pr, the nf1 genotype classification model is evaluated on Null and WT single cell morphology data and the results are stored.  This is after feature-selecting and normalizing on only Null and WT plate data. Please let me know if you have any suggestions.

Sounds good, thanks @jenna-tomkinson, merging now!


Port to QT?

Is it possible for this to be ported to Plasma/QT?
I can contribute if you're interested in pursuing it.
Thanks

I think you miss understood this tool about. It didn't depend in any GUI toolkit. It only changes configuration file. If it have configuration file, it can be styled.


Error while initiating a sst-app

Hi
I'm following the instruction on this page to initiate sst-app https://serverless-stack.com/chapters/building-a-cdk-app-with-sst.html
However, I keep encountering this error npm ERR! Unexpected end of JSON input while parsing near '...kedSize":449192,"npm-'
How could I fix this?

Can you also post your npm version? And maybe upload the debug log that was generated?

Hi, my npm version is 6.14.5 and here is the generated log file
2020-10-12T18_45_15_312Z-debug.log

Thanks. I couldn't find anything specific about this issue in the logs. But a quick Google search turned this up:
https://stackoverflow.com/a/47896884/3369702
Can you give that a try?

Hey @devfromfinland is this still an issue? I'm going to close the issue for now - let us know if you'd like to reopen it


Add back template

It seems that the template file was deleted by a recent new HIP.

Ah thank you, must've slipped in


Text in FoldableStyledArea or GenericStyledArea looks blurry

Hi,
Issues:
Devices:
MacBook Pro (Retina, 15-inch, Mid 2015)
this is retina HD screen, macbook；
another display device  monitor DELL P2417H
this is a normal dp screen, link it to mac.
When I has the same text shown in area, code likes:
FoldableStyledArea area = new FoldableStyledArea();
VirtualizedScrollPane vpane = new VirtualizedScrollPane<>(area);
//add vpane to root node.
area.replace(xxx)
show on retain screen mac looks good:

but dragged the javafx app to second monitor, it will looks blurry and wild colorfu:

Suggest:
https://github.com/javafxports/openjdk-jfx/issues/225
https://stackoverflow.com/questions/23728517/blurred-text-in-javafx-textarea
I've read some pages, tells that maybe a javafx issue for scrollPane,  or 'cache' , 'Snap To Pixel' skin etc.
and still not fix on openfx16.
While  the GericStyledArea implements Region. Maybe we can fix or has workaround by this project.
Please try it and feedback in free time...
Best wishes.

Could you please try this with the latest version of RichTextFX 0.10.7 and see if it still happens.
I tried on Windows 10 with 150% scaling, JRE 11 as well as 17, and it was crisp.

I've with richtextFX 0.10.7 already, JDK is 16.0.1, openfx 16.
Maybe you could try it on retina mac os; and link another monitor.
Drag to main mac to the monitor.
I think it maybe javafx default issue. I'am seeking for a workaound.
Still I want to report to openjfx.
Because the other control or label, editText is same with this issue

I will try it and feed back.

Finally, i figure it out.
Two ways to fix it:
define two config:  boolean  hd1, hd2.
1.
if (hd1) {
scene = new Scene(rootView, info.width, info.height);
//scene = new Scene(rootView);
} else {
//according to your config isDefault to use it to new scene
scene = new Scene(rootView, info.width, info.height, true, SceneAntialiasing.BALANCED);
}

if (!hd2)) {
System.setProperty("prism.lcdtext", "false");
//System.setProperty("prism.subpixeltext", "false");
}


Checksum checking feature has very verbose logging

processing ....
Skipped ... (Checksum: ...)

Can these logs be behind the verbose flag?

Thank you for the follow up.
This indeed happens when first running the watch command.
I understand wanting to give feedback during this process but for example the download already logs differently.
A less verbose option would be to follow the already existing functionality of showing a progress bar :
104|104 [==============================================================]  100 %

How would a progress bar work for watch, a command the has nothing to do unless you do something. And then when you do something it uploads a single file. A progress bar is meaningless in this context.
The output of the watch command is consistent with other tools that have similar behaviour

Looked through the source code it seems to be the result of how the file watcher polling works.
The watcher (github.com/radovskyb/watcher) keeps an in memory state of when a file was last modified.
On the first run this state is missing and emits a first series of events ,causing the theme watch command to log once for each file.
Looking at it more closely I do not think a distinction can be made between these initial events and actual file change events.
Just to clarify :
I have absolutely no issue with logging actual events during the watching process.
It are the initial logs when starting the command that bug me.
This seemed an obvious issue to me and I did not take the time to fully explain my case.

Closing this.
Thank you your feedback :)


added setBuffer to getBuffer example

Very nice, thank you for the contribution.
Please remove the old ImageGetBuffer.cs file that this example replaces.
Also, looks like we have a configuration error here:
https://open.cdash.org/viewBuildError.php?buildid=5945378

This looks like a great contribution. Are you able to fix the build errors?


Add compact density for Dialog

[x] React

[ ] Design

[ ] Documentation

As discussed in meeting with the team, all US on epic compact is closed if they do not have any additional information.


Update enable-archive-mailboxes.md

@markjjo , we are starting to add deep links for inline diagnostic to articles for self-help. Can we add the following at the bottom of this article:
This test will check and identify issues about mailbox archiving;
Run Tests: Archive Mailbox
It should be similar to the diagnostic link in this article :https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/use-dkim-to-validate-outbound-email?view=o365-worldwide

Need to add reference to inline diagnostic.

@lindabrGithub I'm going to close this PR and then add the requested content in the private branch. I think I will also create a new section (called something like "Run Diagnostics") so that this is more discoverable. In the future please submit PRs in the private repo. Here's how: https://review.docs.microsoft.com/en-us/office-authoring-guide/contribute-get-started-setup-quick-update?branch=master

@lindabrGithub A new archive mailbox diagnostics section has been added to this topic.
https://docs.microsoft.com/en-us/microsoft-365/compliance/enable-archive-mailboxes?view=o365-worldwide#run-diagnostics-on-archive-mailboxes


Anime_Title no longer working with Animepahe downloads

Whenever I am downloading from animepahe the episodes do not have the anime title in the name even though I have that set in the config.

@Blatzar I've handled it.

Should work now, closing issue


Added handling for non-standard RPC errors

This PR ports this back here so that we fail on non-standard RPC errors. E.g.:
RPC response: {
    "jsonrpc": "2.0",
    "error": "input could not be validated: {ResourceID:rs_1CAWtHWtinN2uDdc1ou8g7NaE1q Rules:0xc00045eb00}: has a primitive type that is NOT VALID -- given: // Expected valid values are:[array boolean integer number null object string]"
}

[2018-10-28 16:20:16.725] [INFO] RPC CALL schema-service.ResourceSchema.SetRules
  duration: 3
  error: null

Could you add a test?


Please tag/release the 1.7+ master branch

We're using glide for dependency management and would like to have a semver to use in our glide.yaml.
Could you please tag master with the 1.7+ context changes with 2.0 or something?

Done! ✨


System default file explorers

As an end user
I want loading and saving files to use my system default file explorer
So that I can save and load files while having to interact with swing as little as possible
Description
Replace the current save and load file dialogs with something that doesn't use swing. If the user doesn't have a graphical file picker on their system for whatever reason I guess falling back onto swing makes sense.

I found two ways of accomplishing this idea:
use the native file explorer (via awt) if it exists:
new java.awt.FileDialog((java.awt.Frame) null).setVisible(true);

or theme the Java GUI to have the look and feel of the OS:
try {
    UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
}catch(Exception ex) {
    ex.printStackTrace();
}

Should this be closed as wont fix?


[Bug]: Duplicate collections are possible (nodejs)

What happened?
We were bulk writing data to 15 collections yesterday. Our script was doing this rapidly in a high throughput environment. We are using the rest server with client.getOrCreateCollection().
We observed during this process that it was possible to create more than one collection with the same name. From a clickhouse table perspective it looked like this. As you can see we have two collections with the same name but differerent collection_uuid values.
collections table

uuid
name
metadata

123-123-123-123
foo
null

234-234-234-234
foo
null

This resulted in records being added to the embeddings table for both collection_uuid values. When attempting to delete or search records with these differing UUIDs, we were seeing unpredictable behavior. We had to solve the issue manually by "merging" the collection_uuid recordsets.
Also, for anyone interested, it is critical when manually manipulating these tables that the metadata field (at a minimum) contains the string null. Leaving it empty (actually null), caused Chroma to return an error for all records (even if those records were properly populated in the collection.
I think it would be wise to add a unique key to the collections name field.
Versions
Chroma v0.3.26 (client and server) NodeJS 16.
Relevant log output
No response

thanks @wmbutler - this will be fixed with the refactor we are landing this week.

Closing this as it is stale and we have made substantial improvements since this issue was posted (a whole new backend). Please let me know if anything else pops up here and we can re-open it.


fix: ignore top-level function declarations from antfu/eslint

related to https://github.com/rrd108/vue-mess-detector/issues/81

Thank you for the PR, it was quick! :smile:


Glossary anchor links, intro to chialisp edits, typo fixes

Mostly adding some details to the Intro to Chialisp page to make it more explicit.
I also added anchor links to the glossary so we can link to definitions.
Docusaurus doesn't directly support in-line anchor links so I added the links manually. The only real drawback with my solution is that there's no visible anchor allowing to copy the id, but this is a good compromise that keeps the compact and readable list format.

Thanks for this! HTML <a> tags do indeed seem to be the only way to add anchor links in Docusaurus.
I made a couple of minor changes, but looks good to me.


Fix code scanning alert - Uncontrolled data used in path expression

Tracking issue for:

[ ] https://github.com/rickcollette/peaceful/security/code-scanning/1

fixed in c1afca9


Dutch language

Could you please also include the Dutch language in Strautomator.
Thanks!

https://github.com/strautomator/core/commit/cf04deed846f58a7866e48fff4c57fccca3ec1f4

@igoramadas Fast reply! 🤯
If you tell me what files need to be translated I can make a PR for when I have time.

I added 1 remark to that commit cool -> koel. The others seem fine 👍

Good idea and pretty easy to implement :-)

Good idea and pretty easy to implement :-)

I think this reply is intended for my other ticket ;) #30
Closing this one as you implemented it and it is ready for next release.


Fixed outdated docker images FROM and KAFKA_VERSION

Experienced errors when following the documentation from the branch-1.0
docker build -t devenv devenv/.`
The build failed because the FROM images was oudate and apt repository were moved to archive.
__Ref: https://unix.stackexchange.com/questions/508724/failed-to-fetch-jessie-backports-repository

Wheezy and Jessie were recently removed from the mirror network, so if you want to continue fetching Jessie backports, you need to use archive.debian.org instead:

Doing a quick research, I found the official openjdk:8 and updated the kafka version to match the download page url.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Just noticed that master already had some fix applied to it.
My change might be irrelevant now.

Feel free to reopen and merge if you want to apply the fix to the branch-1.0


React-router migration to inferno-router

How do I make the equivalent inferno-router code of this:
ReactDOM.render(
  <Provider store={store}>
    <Router history={history}>
      <Route path="/" component={Layout}>
        <IndexRoute component={Home} />
        <Route path="room(/:room)" name="room" component={Room} />
      </Route>
      <Route path='*' component={NotFound}/>
    </Router>
  </Provider>,
  document.getElementById('app')
)

Oh, it's on the backlog of 1.0


Initial memory visualizer for Vulkan

This actually does two things;

It has an optimization for the callstack resolution which now only happens when it needs to instead of eagerly doing it all the time. This saves significant time in dbghelp.dll and saves some memory too since we only need to store the stack pointers instead of the string
It includes a new allocation overview window that is just a flat list of all allocations that are currently being held by the allocator, including their name. This makes it super easy to track down where memory is being wasted.

@MarijnS95 I restored the branch because this was causing internal build failures; which is why this repo is set not to delete branches when merging in the first place :-)

IMHO we should reference the main branch more eagerly after merging, or push a tag for as long as this rev needs to stick around, instead of leaving unused branches cluttering up which we don't have as a workflow anywhere else (and this kind of thing happened before, rev references to branches aren't reliable enough).
(Besides, it's quite tricky to get this right in the GitHub app which still jitters the "Load more items" and "Delete merged branch" button around a lot)
Now that the internal revision is updated, can the branch be re-removed? :pleading_face:


Fix answer bundle for rejected media

Description
When rejecting media, the identification-tag (i.e. media ID) must
be removed from the bundle in addition to setting the m-line port
to zero.
See https://tools.ietf.org/html/draft-ietf-mmusic-sdp-bundle-negotiation-39#section-8.3.4
Reference issue
Fixes #840

Codecov Report

Merging #841 into master will decrease coverage by 0.11%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master     #841      +/-   ##
==========================================
- Coverage   76.76%   76.65%   -0.12%
==========================================
  Files          60       60
  Lines        4257     4258       +1
==========================================
- Hits         3268     3264       -4
- Misses        716      718       +2
- Partials      273      276       +3

Impacted Files
Coverage Δ

peerconnection.go
72.98% <40%> (-0.07%)
:arrow_down:

dtlstransport.go
74.43% <0%> (-2.28%)
:arrow_down:

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 d8b7905...36a6180. Read the comment docs.

Sorry for the slow review!
@thinkski mind adding a test? Don't worry about full coverage, just to assert if you have one disabled Transceiver it properly generates.
You don't have to go through the full signaling, you can inject the state if that is easier. I approved the PR so you aren't blocked on my approval!


log sample won't accept input with '<'

The log sample seems to treat less than ('<') as a wildcard, so pasting any log content with such character (e.g. a weblogic server log: https://tekslate.com/weblogic-logging-filtering) will fail.
dissect pattern '####<%{Timestamp}> <%{Severity}> <%{Subsystem}> <%{MachineName}> <%{ServerName}> <%{ThreadID}> <%{UserID}> <%{TransactionID}> <%{ContextID}> <%{RawTimeValue}> <%{MessageID}> <%{MessageText}>'
You can manually retype the log text - this will work once, but if you click elsewhere then back to log sample it will be reformatted to replace 'special' chars with e.g. > < etc. which will then not parse
probably the same is the case with other 'special' chars.
I note this is problem with "log sample" but not with the dissect pattern

@pharrison007 Thanks for opening the issue. I believe this is related to the use of a div with contenteditable="true" to emulate the textarea tag we were using before. I need to find a better way of handling this case.

@jorgelbg you likely already know, but just in case:
This appears to be triggered by the clearContentEditableLine function, but I guess the escaping is nessecary?
Maybe using textContent instead of innerHTML would alleviate the need for escaping those characters?
Again, I am a complete novice in regards to HTML / JS, so please feel free to hit me with a cluestick in case that makes no sense :smile:

FYI I deployed the new image to https://dissect-tester.jorgelbg.me/. I did some testing using the examples in this issue and in the linked blog post but it would be useful if someone else gives it a try. Feel free to reopen the issue.

@jorgelbg this solves the issue I have been observing, thanks!

Nice @paketb0te thanks for confirming!


Extended core support by changing arguments interface for SET command

Closes #992

Coverage: 87.47% (+0.01%) from 87.458% when pulling 6375cfbe1177db7af9573b7eb9e0bdef2f47a029 on vladvildanov:vv-992-set-arguments-update into a77a43913a74f9331f637bb12867eb8e274814e5 on predis:v2.x.

This would be a breaking change and should go into v3.x, eh?

@tillkruss Looks like, there's a lot of command interface changes since Redis 6.2, so I assume it should be 3.x for all of them

Yes, we can't merge breaking changes into v2.

Folks, I don't think we're having the same conversation here. If an interface is extended, meaning existing user usage doesn't change - I'd argue we can continue. If we break  an interface, then yes.
@tillkruss This is similar to many other projects so WDYT? Again the proposal I'm making is that when extending the interface - meaning no user facing change or behavioural change unless actively used, we continue within this code path.

Folks, I don't think we're having the same conversation here. If an interface is extended, meaning existing user usage doesn't change - I'd argue we can continue. If we break an interface, then yes.

Correct. I do see several changes to existing test cases. If we can leave the current ones in place and add new ones that prove the old functionality works, it's not a BC.


[FLINK-20675][checkpointing] Ensure asynchronous checkpoint failure could fail the job by default

What is the purpose of the change
Compared with https://github.com/apache/flink/pull/14656, this PR add another commit to refactor interfaces to decline checkpoint with CheckpointException instead of previous Throwable.
Currently, no mater how many times of async checkpoint failure, job would not trigger failover. This PR ensure the mechanism of failing the job by default when async phase of checkpoint failed, which was broken in FLINK-12364.
Brief change log

Let CHECKPOINT_ASYNC_EXCEPTION could also be treated to fail the job.
Ensure decline checkpoint with specific CheckpointException instead of previous throwable.
If task is not running, never decline the checkpoint on task side to avoid unexpected failover again.
refactor interfaces to decline checkpoint with CheckpointException instead of previous Throwable.

Verifying this change
This change added tests and can be verified as follows:

Added CheckpointFailureManagerITCase#testAsyncCheckpointFailureTriggerJobFailed to ensure job would failed once async checkpoint failed.

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

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

CI failed due to FLINK-21104

CI failed due to FLINK-21104

@rkhachatryan I use checkpointException.getCause() when printing decline checkpoint message to avoid duplicate information in fixup commit.

@rkhachatryan I use checkpointException.getCause() when printing decline checkpoint message to avoid duplicate information in fixup commit.

@rkhachatryan thanks for your review!
@pnowojski do you have more comments?

@rkhachatryan thanks for your review!
@pnowojski do you have more comments?

Merging...

Merging...

Hey @Myasuka, the PR broke the current master. Could you help fixing it?

Hey @Myasuka, the PR broke the current master. Could you help fixing it?

I will push a fix.

I will push a fix.

@dawidwys Sorry for the late reply. As discussed offline before, thanks for your kind hotfix.

@dawidwys Sorry for the late reply. As discussed offline before, thanks for your kind hotfix.


update ratingBadge borderRadius

Per Wes design request, reducing the ratingBadge borderRadius from 2 to 1

Codecov Report

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

@@          Coverage Diff          @@
##           master   #276   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          39     39
  Lines         439    439
=====================================
  Hits          439    439

Impacted Files
Coverage Δ

src/RatingBadge.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 5221b0b...7048288. Read the comment docs.


Protocol method when longPress started

Hi,
How about protocol's method to inform delegate when longPress has started?
Amazing work! Thank you.
Thomás

Do you really need it?

Yes, I need! My tableview is update each minute, but I need to invalidate the NSTimer while user is reordering some cells. Did you understand?
Thank you.
Thomás Calmon

Ok, I will add this and release it tomorrow.

Thank you so much!
Cheers,
Thomás Calmon


groupBy node doesn't have expected behaviour

I defined this tick script:
var aggregate = stream
	|from()
		.database('sampledb')
		.retentionPolicy('default')
		.measurement('sample')
	|window()
		.align()
		.period(5m)
		.every(5m)
	|groupBy('node')
	|mean('metric1').as('metric1')
	|influxDBOut()
		.database('sampledb')
		.measurement('actual')

My expectation is that when this script ran, it would produce the equivalent of:
select mean(metric1) as metric1
into actual
from sample
where time >= '2016-03-08 05:00:00' and time < '2016-03-08 08:00:00'
group by time(5m), *

However, what actually happened is that only one of the "node" groups was written into "actual" - the other groups were discarded.
Is my expectation of what the groupBy node is meant to do incorrect or is this a bug?
I have a Vagrantfile project that includes test data and reproduces the problem available here:    https://github.com/wildducktheories/influxdata-testcase
I was using kapacitor 0.12 and influxdb 0.11

Ok, so I tweaked the test to log after the window() node and sure enough that's what the issue is - the data in the recording is sorted by node first, then time and so the window node skips all the records from subsequent nodes.
@nathanielc - is there a way to ensure that queries are stored in timestamp sorted order?

@jonseymour Hmm, is seems odd that a recording is not sorted by time. I thought that was impossible. I'll have to go check.
Any way you probably want to groupBy before the window anyway since then the windows will be windowed per group. It shouldn't make a difference but under the hood it will be more efficient.
var aggregate = stream
    |from()
        .database('sampledb')
        .retentionPolicy('default')
        .measurement('sample')
        .groupBy('node')
    |window()
        .align()
        .period(5m)
        .every(5m)
    |mean('metric1').as('metric1')
    |influxDBOut()
        .database('sampledb')
        .measurement('actual')

Ok, so I was deceived by the documentation here.
The distinction between groupBy() the property of a stream node and groupBy() the chaining method wasn't clear from reading the documentation.
(The groupBy() as property isn't really documented, although the example for the chaining method does, somewhat confusingly, include an example of the use of the property syntax).

@nathanielc  I've created a PR against the documentation https://github.com/influxdata/docs.influxdata.com/pull/379 to fix the documentation issue.
Even if this solution works, it isn't obvious to me why the other solution didn't work.
Do you want to leave this issue open until you have had a chance to check why the window() first technique didn't work?

Yes, let's leave it open.Thanks
On Apr 7, 2016 6:00 PM, "Jon Seymour"<EMAIL_ADDRESS>wrote:

@nathanielc https://github.com/nathanielc I've created a PR against the
documentation influxdata/docs.influxdata.com#379
https://github.com/influxdata/docs.influxdata.com/pull/379 to fix the
documentation issue.
Even if this solution works, it isn't obvious to me why the other solution
didn't work.
Do you want to leave this issue open until you have had a chance to check
why the window() first technique didn't work?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/influxdata/kapacitor/issues/423#issuecomment-207142965

@jonseymour very interesting test-case thanks.
Can you tell me how you got the query that get produced by the tick script?
I try to debug my scripts but I find it very difficult.


DllNotFoundException: Unable to load DLL e_sqlcipher.dll

Hi,
Been trying to solve this issue - but still havent got the way out. Appreciate some help on this matter.
"DllNotFoundException: Unable to load DLL 'C:\Users\AppData\Local\Temp\bin\runtimes\win-x64\native\e_sqlcipher.dll' or one of its dependencies: 0x80070005 (E_ACCESSDENIED)"
Thanks!

We are also experiencing this issue on net4.x CacheDatabase Issue


Typo fix in "Use Jupyter notebooks in Galaxy" tutorial

Changed mentions of the RNA-seq file suffix from ".count" to ".counts" to match up with the actual linked file.

Thanks @Borkan1 !


Syntax highlight support for Markdown

The syntax highlight for Markdown files (.md) is rather weak, I had to switch to the VSCode default solarized dark theme (which has surprisingly good at markdown syntax highlight) whenever I edit a markdown file.
For other filetypes, I really love your theme!!
Keep it up, just wish it could be even more perfect!

@ianpan870102 Other than #1 what is weak about it... ? Just need to try and figure out what you're expecting, I'm guessing you're wanting better inline-code, list highlighting?

Dear ginfuru,
Thank you for the email!
I hope that the attached two screenshots can successfully elaborate the
syntax highlighting differences in VSCode-solarized-dark, and your
Better-Solarized dark (which seems to be quite 'grayed-out' in 80% of my
Markdown code).
Best,
Ian
Ed Heltzel<EMAIL_ADDRESS>於 2018年9月13日 週四 下午8:34寫道：

@ianpan870102 https://github.com/ianpan870102 Other than #1
https://github.com/ginfuru/vscode-better-solarized-dark/issues/1 what
is weak about it... ? Just need to try and figure out what you're
expecting, I'm guessing you're wanting better inline-code, list
highlighting?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ginfuru/vscode-better-solarized-dark/issues/2#issuecomment-420990596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ai_MNOxTEghFaf9VqKL6G8hsMCJnYGkqks5ualDsgaJpZM4WmeB5
.

@ianpan870102, would you mind showing the screenshots as part of the post? Attaching them to the email probably doesn't work.

@ianpan870102
What version of the ther theme are you using?

I'm using the latest version that comes with the VSCode built-in themes. Yes, I am showing the '.md' code -- not the preview -- because I'm discussing the syntax highlight.

This is what blockquotes looks like with Dank Mono


Validation upon object creation.

I had the same problem as issue #124 (validation not working on object creation). This commit resolved it for me on WordPress 4.5.2 but I've not performed tests of all affected code.

Thanks for doing this. In the conditional, note that "id" is not always the name of the primary key, so please see the comment and ideally re-submit the pull. This will be very helpful in finally actually validating the save action! Much appreciated.

That makes total sense - duly changed.
Apologies for the delay in updating the pull request.

Thanks!