Chain Index Uses Excessive Memory on Mainnet.

Summary
Syncing the chain index on mainnet requires excessive amounts of memory when it needs to catch up with syncing.
For example, it used approximately 100GB of memory where the last 4% of mainnet was synced recently. (It also uses all available processor cores for extended periods.)
Steps to reproduce the behavior
Run the chain-index on mainnet:
`which time` --verbose plutus-chain-index start-index --network-id 764824073             \
                                                      --db-path chain-index.db/ci.sqlite \
                                                      --socket-path node.socket          \
                                                      --port 9083

Actual Result

AppConfig {acLogConfigPath = Nothing, acMinLogLevel = Nothing, acConfigPath = Nothing, acCLIConfigOverrides = CLIConfigOverrides {ccSocketPath = Just "node.socket", ccDbPath = Just "chain-index.db/ci.sqlite", ccPort = Just 9083, ccNetworkId = Just 764824073}, acCommand = StartChainIndex}

Logging config:
Representation {minSeverity = Info, rotation = Nothing, setupScribes = [ScribeDefinition {scKind = StdoutSK, scFormat = ScText, scName = "stdout", scPrivacy = ScPublic, scRotation = Nothing, scMinSev = Debug, scMaxSev = Emergency}], defaultScribes = [(StdoutSK,"stdout")], setupBackends = [KatipBK,AggregationBK,MonitoringBK,EKGViewBK], defaultBackends = [KatipBK,AggregationBK,EKGViewBK], hasEKG = Just (Endpoint ("localhost",12790)), hasGraylog = Nothing, hasPrometheus = Nothing, hasGUI = Nothing, traceForwardTo = Nothing, forwardDelay = Nothing, traceAcceptAt = Nothing, options = fromList []}

Chain Index config:
Socket: node.socket
Db: chain-index.db/ci.sqlite
Port: 29083
Network Id: Testnet (NetworkMagic {unNetworkMagic = 764824073})
Security Param: 2160
Store from: BlockNo 0

The tip of the local node: SlotNo 52553102
Connecting to the node using socket: node.socket
Starting webserver on port 29083
A Swagger UI for the endpoints are available at http://localhost:29083/swagger/swagger-ui
Syncing (96%)
Syncing (97%)
Syncing (98%)
Syncing (99%)
Syncing (100%)

^C Interrupt
Command terminated by signal 2
        Command being timed: "plutus-chain-index"
        User time (seconds): 489738.46
        System time (seconds): 159428.96
        Percent of CPU this job got: 1556%
        Elapsed (wall clock) time (h:mm:ss or m:ss): 11:34:53
        Average shared text size (kbytes): 0
        Average unshared data size (kbytes): 0
        Average stack size (kbytes): 0
        Average total size (kbytes): 0
        Maximum resident set size (kbytes): 98701420
        Average resident set size (kbytes): 0
        Major (requiring I/O) page faults: 45605
        Minor (reclaiming a frame) page faults: 25476185
        Voluntary context switches: 448477241
        Involuntary context switches: 793936231
        Swaps: 0
        File system inputs: 8
        File system outputs:<PHONE_NUMBER>
        Socket messages sent: 0
        Socket messages received: 0
        Signals delivered: 0
        Page size (bytes): 4096

Expected Result
It's unreasonable to require 100GB+ of memory to run chain index. Ideally, its memory footprint should be under 5 GB.
Describe the approach you would take to fix this

Experiment with the use of --RTS options.
Profile the memory usage of the haskell code.
Break database transactions into smaller units.
Replace SQLite3 with a more performant persistent store.

System info
plutus-apps at commit ce8282d89939852ef752eec43565d5e68715e62d

Could you please check if the situation is still the same? There were several PRs to improve it.


DayView bound offset causes unwanted space between tiles

I want to make set my tiles' height to a constant, and width to match_parent. I mean like putting tiles in a LinearLayout and set each tiles' weight to 1. I tried to do that by calculating screenwidth / 7 and setting it as tileWidth. But this causes unwanted space between tiles. When I read the below code, It seems like there is no way to disable offset. Is there a way to achieve what I want?
https://github.com/prolificinteractive/material-calendarview/blob/3fe9162b695c89e9fb4758e9548d0df2957d6ddc/library/src/main/java/com/prolificinteractive/materialcalendarview/DayView.java#L289-L304

It seems like the reason was not offset. It was radius. I've set radius to Math.max(height, width); offset to 0 and tileWidth to LinearLayout.LayoutParams.MATCH_PARENT and achieved what I wanted. But I had to add the library as a module to do my project in order to edit it :(


Flaky API test: RecordsResourceIT.testConsumeRecordsAsOfTimestamp

Error:  com.github.eyefloaters.console.api.RecordsResourceIT.testConsumeRecordsAsOfTimestamp(Instant, Instant, Instant, int)[1] -- Time elapsed: 1.593 s <<< FAILURE!
java.lang.AssertionError:
1 expectation failed.
JSON path data doesn't match.
Expected: a collection with size <2>
  Actual: <[]>

Error still occurs after #36, need to revisit.

This error hasn't happened in a while. closing the issue for now


How do I add script and stylesheet to global scope?

I just get started and I want to add a script and css styles to global scope, can I just add them to the index.ejs? Or any other better ways to do this?
Ths

You can import css into global scope using the method from line 10 of index.js
It will then be bundled in webpack and exported as a separate .css using the ExtractTextPlugin in the webpack.prod.config
As you state you could also simply add a line in the index.ejs.
😄


Filter env vars configuration

What does this PR do?
Filter env vars configuration.
Motivation
Kubernetes automatically creates a bunch of variables prefixed with TRAEFIK_ (probably based on the image name), and since our config loader was checking for the presence of the above prefix, this situation was obviously creating a conflict.
More

[x] Added/updated tests
[x] Added/updated documentation

Not sure, but you may want to check these too, as services add environment variables by default too:
https://kubernetes.io/docs/concepts/services-networking/service/#environment-variables


Define labels for category: Workflow

Workflow labels define the position of an issue or PR in out process:
Needs action labels
color: gold (#FFD700)

split into tasks: Issue voted as a Project
discussion needed: Needs agreement on how to solve it
review backend
review frontend
review product

Approved labels
color: pale green (#98FB98)

review backend 👍
review frontend 👍
review product 👍

Rejected labels
color: firebrick (#B22222)

review backend 👎
review frontend 👎
review product 👎
blocked: Depends on something else before it can be acted on.
needs gif: On interface changes the PR should include a gif with the new behavior

Situation labels
color: whitesmoke (#F5F5F5)

on test: It's on the test server
in progress: Someone is working on it
sprint ready: Ready to estimate

Special Label
color: forestGreen (#228B22)

ready for merge 🕶 : Title says it all

@JannKleen Is it possible to have a deployed label for things that have been already deployed, that would make a lot easier to compile this data to communicate that to the rest of the company. Can it be automated?

Also, is it possible to auto-set and unset a 'currently-on-test' label?

ready for deploy, too?

Do you need the ready for deploy @JannKleen @nsb ?
I guess you need it, right @JanNash ?

No emoji @ all? 😞

Yeah, not sure if I need the ready to deploy label. @JannKleen do you think we could benefit from that in the tables repo?

They don't allow it @JanNash 😢
This is clearly not targeted at millenials

😭

@JanNash @menor i didn't find a ready-made solution to tag PRs once they're merged into master but feel free to come up with a script to do this.
We'd also be using the 'deployed' tags once for the weekly update email and once for the monday standup (the two times a week we need to report them).
Maybe there's a better way to do this

We can filter by is:pr is:merged in the PR list, but I think having the tag is better since I guess that also shows PR that have been merged to a branch different from master.
What about putting the label manually after merging or deploying? That way we would be able to filter things that have been merged but not deployed and have a clear picture of the PRs we are deploying.
I know manually is not ideal, but we need to label everything else manually anyways.


feat(cdk/tree): implement typeahead for TreeKeyManager

Based on #26586. Followed by #26588.

I think I fixed the tests; failing ones seem to be unrelated?

Yes, can you just rebase to make sure it's not related to your PR? the main branch is green now

I'll need someone to rebase the cdk-tree-revamp branch, I don't have the ability to do that


Waffle is closing its doors

Hi.
Thank you for NDbUnit2.
I just noticed that Waffle is closing its doors in less than 2 months (May 16, 2019).
https://waffle.io/closing-its-doors
One of the last steps in the migration guide indicates:

Remove any Waffle badges or graphs from your README.

https://github.com/savornicesei/NDbUnit2/blob/master/README.md is where I noticed Waffle is referenced.
Thank you.

@icnocop Thank you. I will remove the waffle references.


Added support for voice/list

We were getting 501 error while executing compliance test VoiceListConformance without this change.
With this change compliance test is passing with 200; below response from AH212 RDK Reference device:
$ python3 main.py -v -b <deviceIP> -I <deviceID> -c VoiceListConformance

testing voice/list   {} ...
voice/list Latency, Expected: 200 ms, Actual: 73 ms
[ PASS ]
{
  "status": 200,
  "voiceSystems": [
    {
      "enabled": true,
      "name": "AmazonAlexa"
    }
  ]
}

TestCase1: when Alexa is disable on platform:
python3 main.py -v -b <deviceIP> -I <deviceID> -c VoiceListConformance

testing voice/list   {} ...
voice/list Latency, Expected: 200 ms, Actual: 52 ms

[ PASS ]
{
  "status": 200,
  "voiceSystems": [
    {
      "enabled": false,
      "name": "AmazonAlexa"
    }
  ]
}

TestCase2: when Alexa is enabled on platform:
python3 main.py -v -b <deviceIP> -I <deviceID> -c VoiceListConformance

testing voice/list   {} ...
voice/list Latency, Expected: 200 ms, Actual: 101 ms

[ PASS ]
{
  "status": 200,
  "voiceSystems": [
    {
      "enabled": true,
      "name": "AmazonAlexa"
    }
  ]
}

Abandoning because this is included in https://github.com/device-automation-bus/dab-adapter-rs/pull/20


hand encoding json is fraught with peril, just use stdlib, it knows how.

fixes #241 and I also am assuming any other case of json encoded as string inside json.

You probably want to rebase off of master for this.

It's worth noting that applying this patch makes the marshalling process roughly 20% slower:
# before
$ go test -bench . ./internal/protocol/json/jsonutil/
PASS
BenchmarkBuildJSON	   50000	     32727 ns/op
BenchmarkStdlibJSON	  100000	     12064 ns/op
ok  	github.com/awslabs/aws-sdk-go/internal/protocol/json/jsonutil	3.516s

# after
$ go test -bench . ./internal/protocol/json/jsonutil/
PASS
BenchmarkBuildJSON	   50000	     39859 ns/op
BenchmarkStdlibJSON	  100000	     12254 ns/op
ok  	github.com/awslabs/aws-sdk-go/internal/protocol/json/jsonutil	3.930s

Given that the specification for string is fairly simple (http://www.json.org/), I'm not sure it warrants the overhead.

Cool, your last commit actually fixes #241 too, so im fine either way.
Might want to have a spec that does some crazy json inside json inside json inside json and have it actually hit some services. Never know what people will try and shove in dynamodb.

Yea that is a good idea. I'll take a look tomorrow at beefing up the test suite to make sure we're handling cases like yours.

:+1:  thanks @lsegal


validation should be added for add function in CRUD where each word should be trimmed for spaces, else search results will fail for same.

resolved with last commit efab238


Added DL580 G6 and DL580 Gen8

Added hardware survey for two HPE ProLiant models, DL580 G6 and DL580 Gen8.

Thanks @tylerwoodall. I see one system has encrypted filesystem. It's good to get that into the database. I presume no issues setting that up or you would have mentioned it.

Thanks @tylerwoodall. I see one system has encrypted filesystem. It's good to get that into the database. I presume no issues setting that up or you would have mentioned it.

That is correct. No issues encountered.


[Feature] Spark writer supports insert overwrite

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Spark writer supports insert overwrite
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

can you assign me this task?

@tobehardest Just create a PR for this.

Unassigned because that haven't replied for a long time


Elastic cloud provider version 0.8.0 adding apm_secret_token everytime pipeline runs

I have recently upgraded the Elastic Terraform provider from version 0.4.1 to 0.8.0 and encountered an unexpected issue during the process. After successfully importing the ec_deployments resources, I performed a plan to verify the upgrade, but I noticed that the creation of apm_secret_token is pending in the plan. Surprisingly, this situation persists even after a successful apply.
Upon further investigation, I noticed that the apm_secret_token value is not being written to my state file during the apply process, and it remains set to the default value, i.e., null. This issue seems to be specific to clusters where the Integrations Server (APMs) feature is enabled.
I am using Terraform version 1.5.
Readiness Checklist

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I am reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior

I expect that this vale of apm_secret_key will be added to my terraform state file and won't be pending in plan constantly.
Current Behavior

Steps to Reproduce

Create a ec_deployment resource with elastic provider 0.4.1 and add integrations_server configuration
Upgrade provider from 0.4.1 to 0.8.0
Import ec_deplotment and apply changes
Make terraform plan again

Context

We are trying to upgrade from elastic provider 0.4.1 to 0.8.0
Your Environment

Version used: v0.8.0
Running against Elastic Cloud SaaS or Elastic Cloud Enterprise and version: Elastic Cloud SaaS 8.8.2

Whilst we can fix the endless diff loop, this value is only returned by the API during resource creation, we're not able to set this value to a useful value.
To avoid this in 0.8.0 you could manually edit state, populating this with a non-null value. I'll look at avoiding the diff when the value is null as well.
In practice, usage of this field is potentially error prone in integrations_server resources. I'm going to open an issue to deprecate it and eventually remove it from integrations_server resources entirely.


Support thrift server in Astro

Since Astro uses HiveContext we can make Astro to support thrift server. Users can use JDBC driver to connect to Astro if we can support thrift server in Astro.

Added PR https://github.com/HuaweiBigData/astro/pull/50 . Please review


Module rebooted after 5 minutes

As module can receive and work on time update command send by orange pi, but i noticed after last code update when module send data to thingsboard it reboot.

Second issue is, ttgo is not sending or correct temperature and humidity to thingsboard local server.

Issue got fixed in #129


Unable to run darwin binary.

Whenever I am running the command "./grok_exporter -config ./example/config.yml", I am getting following error.

Thanks for the bug report. I must say I have never seen this error.

I have a Mac Book as my development machine, it never occurred there.
For each commit I run automated OS X tests on https://travis-ci.org/fstab/grok_exporter, the error never occurred there.
I even set up a new virtual machine with an original high sierra os, to make sure there are  no unexpected dependencies. There was no error running grok_exporter there.

Can you provide some more info that could help me reproduce this error?

similiar error
grok_exporter-0.2.4.darwin-amd64 renweibo$ ./grok_exporter -config ./example/config.yml
SIGILL: illegal instruction
PC=0x438e59a m=10 sigcode=1
signal arrived during cgo execution
goroutine 1 [syscall, locked to thread]:
runtime.cgocall(0x4384990, 0xc420053600, 0x4486e6f)
/Users/fabian/golang-1.9.3/go/src/runtime/cgocall.go:132 +0xe4 fp=0xc4200535c0 sp=0xc420053580 pc=0x40033d4
github.com/fstab/grok_exporter/exporter._Cfunc_onig_new(0xc420049200, 0x5800000, 0x580076e, 0x0, 0x46ee2d8, 0x46ee178, 0xc4202b4a40, 0x0)
github.com/fstab/grok_exporter/exporter/_obj/_cgo_gotypes.go:294 +0x4d fp=0xc420053600 sp=0xc4200535c0 pc=0x437cc2d
github.com/fstab/grok_exporter/exporter.(*OnigurumaLib).Compile.func1(0xc420049200, 0x5800000, 0x580076e, 0x0, 0x46ee2d8, 0x46ee178, 0xc4202b4a40, 0x53)
/Users/fabian/go/src/github.com/fstab/grok_exporter/exporter/oniguruma.go:77 +0x19f fp=0xc420053660 sp=0xc420053600 pc=0x437f24f
github.com/fstab/grok_exporter/exporter.(*OnigurumaLib).Compile(0xc42000e088, 0xc420209800, 0x76e, 0x0, 0x0, 0x0)
/Users/fabian/go/src/github.com/fstab/grok_exporter/exporter/oniguruma.go:77 +0x179 fp=0xc4200536f0 sp=0xc420053660 pc=0x437d509
github.com/fstab/grok_exporter/exporter.Compile(0xc420064070, 0x6d, 0xc42000e0e0, 0xc42000e088, 0x0, 0x1, 0x4043224)
/Users/fabian/go/src/github.com/fstab/grok_exporter/exporter/grok.go:31 +0x9c fp=0xc4200537a8 sp=0xc4200536f0 pc=0x4373c5c
main.createMetrics(0xc420132000, 0xc42000e0e0, 0xc42000e088, 0x0, 0x0, 0x0, 0x0, 0xc420013e70)
/Users/fabian/go/src/github.com/fstab/grok_exporter/grok_exporter.go:178 +0x15f fp=0xc420053b10 sp=0xc4200537a8 pc=0x4382a4f
main.main()
/Users/fabian/go/src/github.com/fstab/grok_exporter/grok_exporter.go:62 +0x14d fp=0xc420053f80 sp=0xc420053b10 pc=0x438106d
runtime.main()
/Users/fabian/golang-1.9.3/go/src/runtime/proc.go:195 +0x226 fp=0xc420053fe0 sp=0xc420053f80 pc=0x402dc56
runtime.goexit()
/Users/fabian/golang-1.9.3/go/src/runtime/asm_amd64.s:2337 +0x1 fp=0xc420053fe8 sp=0xc420053fe0 pc=0x40597b1
rax    0x0
rbx    0x1
rcx    0x5
rdx    0x700003543be4
rdi    0x46ee2d8
rsi    0x1
rbp    0x700003543d90
rsp    0x700003543aa0
r8     0x0
r9     0x0
r10    0x467cfb0
r11    0x6ffffeec65fc
r12    0x700003543bb0
r13    0x1
r14    0xc4202b4a40
r15    0x4c00020
rip    0x438e59a
rflags 0x10297
cs     0x2b
fs     0x0
gs     0x0

grok_exporter-0.2.4.darwin-amd64 renweibo$ go version
go version go1.10 darwin/amd64

grok_exporter-0.2.4.darwin-amd64 renweibo$ cgo -V
cgo version go1.10

This is weird, I have never seen the error on my system. Could you provide the output of uname -a, and if you have otool installed also the output of otool -L ./grok_exporter?

otool -L ./grok_exporter
./grok_exporter:
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1252.0.0)
	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1451.0.0)
	/System/Library/Frameworks/Security.framework/Versions/A/Security (compatibility version 1.0.0, current version 58286.41.2)

Here is uname
uname -a
Darwin WeideMBP.lan 17.4.0 Darwin Kernel Version 17.4.0: Sun Dec 17 09:19:54 PST 2017; root:xnu-4570.41.2~1/RELEASE_X86_64 x86_64

The problem seems weird to me. If you need more information, please give more instruction. I need this tool to work with some of the server.

This is very strange. You have exactly the same OS and library versions as the machine I am using for developing grok_exporter. Do you have any idea how I could reproduce this error? The error doesn't occur on my machine.

Currently for me to reproduce it, it's very simple. Download the archive, unzip it, execute it then failed.
I'm not familiar with golang. What's your idea about following questions?
It seems that these lines in grok_exporter.go cause problem
	metrics, err := createMetrics(cfg, patterns, libonig)  # line 62
	exitOnError(err)                                                              # line 63

Q1: Is the code I pointed is where the real error occurs?
Q2: What's the real work in createMetrics?
Q3: Can you provide more information about the error code itself?

You are right, from the stack trace we can see that createMetrics(...) is the call where the error occurs. However, the actual error seems to be a little further down the stack, in oniguruma.go, line 77, near r := C.onig_new(...).
A little background on what this does: Go's built-in regular expressions don't support backtracking (see re2 for more info). As grok patterns heavily rely on backtracking, grok_exporter use an external regular expression library called Oniguruma. Oniguruma is built into the Ruby programming language, which is used for Logstash, which is the origin of grok patterns.
Oniguruma is a C library. Go supports calling C libraries from go programs using cgo. This is the magic C in C.onig_new(...). The function onig_new is not a go function, it's a C function defined in oniguruma.h.
For some reason, the call to onig_new fails with the illegal instruction error. The error message sounds like there is an illegal assembly code involved, but it could be something completely different. I don't know why this happens, especially as I have never seen it on my machine.

You could try to build grok_exporter from source, as described in README.md. This will create a grok_exporter binary that is dynamically linked against Oniguruma on your system. Maybe this helps. (the official release is statically linked, so that there is no need for having Oniguruma installed)

@fstab Get it. I's clear to me now. I'll try to build it later on my system and ping here back for anything useful to dig the problem.

@fstab  Here is the update. Build from source works. Following is the steps
brew install fstab/oniguruma/oniguruma-5.9.6
go get github.com/fstab/grok_exporter
cd $GOPATH/src/github.com/fstab/grok_exporter
git submodule update --init --recursive
./release.sh darwin-amd64  # at first, it failed for the requirement of go 1.9.3 but I have go 1.10
# manually disable go version check in release.sh
./release.sh
cd dist
unzip grok_exporter-0.2.5-SNAPSHOT.darwin-amd64.zip
cd grok_exporter-0.2.5-SNAPSHOT.darwin-amd64
./grok_exporter -config example/config.yml

Then I check http://localhost:9144/metrics. It show some output in browser.

@fstab  Here are some questions:

I don't found the detail step to build dynamically linked version grok-exporter. How to get it?
Now with this new result, what conclusion can we get?
Do you have any suggestion what's the next thing we can try to find the reason?

The release is built exactly with the same commands as you described above. I have no idea why it works when you run these commands on your machine and it doesn't work when I run the commands on my machine.
The dynamically linked version is created when you just run go install github.com/fstab/grok_exporter instead of using the release.sh script. You can run otool -L $GOPATH/bin/grok_exporter to verify that it's dynamically linked against oniguruma.
It would be interesting to see if the release you built on your machine crashes on my machine. Could you send me the ZIP file to<EMAIL_ADDRESS>?

The release is sent to you and let's see what happen on your machine.

Dynamically linked version here. And http://localhost:9144/metrics show output.
$ go install github.com/fstab/grok_exporter
$ otool -L $GOPATH/bin/grok_exporter
/Users/renweibo/project/go/bin/grok_exporter:
	/usr/local/opt/oniguruma-5.9.6/lib/libonig.2.dylib (compatibility version 3.0.0, current version 3.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1252.0.0)
	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1451.0.0)
	/System/Library/Frameworks/Security.framework/Versions/A/Security (compatibility version 1.0.0, current version 58286.41.2)
$ cd dist/grok_exporter-0.2.5-SNAPSHOT.darwin-amd64
$ $GOPATH/bin/grok_exporter -config example/config.yml
Starting server on http://localhost:9144/metrics

Your release works fine on my machine.
With 'it doesn't work when I run the commands on my machine.' I meant that if I build a release on my machine using exactly the same commands that you described above, the release doesn't work on your machine.
I have no idea why this happens.

Please ping me when you release next version, I will try it again. if the problem disappear, I suggest you close the issue and move on.

@renweibo released v0.2.5. If you are still looking into it, I would be interested if the problem is resolved. Updated to golang 1.10.2, maybe that helped.

It still report error.

What if you build the release in your machine with ./release.sh darwin-amd64? Does it work?

After disable go version check, rerun release.sh
renweibo$ ./release.sh darwin-amd64
?   	github.com/fstab/grok_exporter	[no test files]
ok  	github.com/fstab/grok_exporter/config	0.007s
ok  	github.com/fstab/grok_exporter/config/v1	0.008s
ok  	github.com/fstab/grok_exporter/config/v2	0.011s
ok  	github.com/fstab/grok_exporter/exporter	2.891s
ok  	github.com/fstab/grok_exporter/tailer	6.716s
ok  	github.com/fstab/grok_exporter/templates	0.011s
Building dist/grok_exporter-0.2.6-SNAPSHOT.darwin-amd64.zip
sed: oniguruma/oniguruma.go: No such file or directory

Oh, you tried between two of my commits while I was creating oniguruma/oniguruma.go. It should work now. I also removed the version check. Could you try again?

It seems that oniguruma/oniguruma.go is missing.
renweibo$ git pull
Already up-to-date.
renweibo$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean
renweibo$ ./release.sh darwin-amd64
?   	github.com/fstab/grok_exporter	[no test files]
ok  	github.com/fstab/grok_exporter/config	(cached)
ok  	github.com/fstab/grok_exporter/config/v1	(cached)
ok  	github.com/fstab/grok_exporter/config/v2	(cached)
ok  	github.com/fstab/grok_exporter/exporter	(cached)
ok  	github.com/fstab/grok_exporter/tailer	(cached)
ok  	github.com/fstab/grok_exporter/templates	(cached)
Building dist/grok_exporter-0.2.6-SNAPSHOT.darwin-amd64.zip
sed: oniguruma/oniguruma.go: No such file or directory

This is weird, the file is right here: https://github.com/fstab/grok_exporter/blob/master/oniguruma/oniguruma.go Could you check if you see the file on your disk after git pull?

This time I can build it. And I found the reason for build fail. I git clone the repository in one directory not under GOPATH. So it failed for the release.sh assume it current path is under right path in GOPATH like ${GOPATH}/src/github.com/fstab/grok_exporter
The file I build local, it can run without problem.


Unit | Component | sl pagination: previousPage action decrements currentPage

not ok 546 PhantomJS 1.9 - Unit | Component | sl pagination: previousPage action decrements currentPage
    ---
        actual: >
            null
        message: >
            Died on test #1     at test (http://localhost:7357/assets/test-support.js:3025)
                at testWrapper (http://localhost:7357/assets/test-support.js:6192)
                at test (http://localhost:7357/assets/test-support.js:6205)
                at http://localhost:7357/assets/tests.js:25589
                at http://localhost:7357/assets/vendor.js:152
                at tryFinally (http://localhost:7357/assets/vendor.js:33)
                at http://localhost:7357/assets/vendor.js:158
                at http://localhost:7357/assets/test-loader.js:60
                at http://localhost:7357/assets/test-loader.js:51
                at http://localhost:7357/assets/test-loader.js:82
                at http://localhost:7357/assets/test-support.js:6024: 'undefined' is not a function (evaluating 'pages.find(function (page) {
                            return page.active;
                        })')
        Log: |
    ...

Will likely be resolved by https://github.com/softlayer/sl-ember-components/issues/1503

Resolved by https://github.com/softlayer/sl-ember-components/issues/1503


Fail to expand the attached services for Azure Germany accounts

OS: All
Build Version:20190211.1
Repro Steps:

Sign in with Azure Germany account.
Right click one storage account --> Select 'Copy Connection String'.
Attach with the copied connection string in Azure Storage Explorer with 'Azure Germany' environment.
Expand the attached storage account in Storage Explorer --> Check the services.

Expect:
Succeed to check the services without error.
Actual:
Pop up an error.

More Info:

This issue also reproduces for Azure China accounts.

This issue doesn't reproduces for Azure accounts.

More Info:
There is no any extra info in Connection String to indicate that the account belongs to Azure China & Azure Germany accounts.

Need to do the same thing as this:
https://github.com/microsoft/vscode-azurefunctions/blob/ec65cbf9d0f4a2b7a45c250bd7c5c56587daafe1/src/utils/azure.ts#L79

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!


Implement Action and Mutation subscribers

Updated the lib to follow more of an OOP design.
Now supports { action: "myAction" } and { mutation: "myMutation" } watchers:

Both an action and mutation callback function is passed two arguments: the event's payload and the store's state before the action occurs and after the mutation occurs.

I didn't think it was worth including the action or mutation type in the watcher's callback because when defining the watcher, type -  the specific action or mutation name for which to invoke the callback - is already known.

Rad, Looks exactly like we had discussed.
Great additions, I like the OOP approach. This should make it easier to extend in the future.

Sorry about the delay. I will be reviewing and potentially pushing this into prod this weekend.


Feature: Displaying only the month

Hey,
I like this project so far. But I now need the possibility to add a picker where one can only choose between month and year.

Thank you for your appreciation!
Since this is intended a polyfill, I'm not likely to add features that aren't available to the native input[type="date"] spec. I'm sorry!

Actually there exists a input[type="month"]. That was what I was thinking about

That is an interesting idea, to support more input type values. Will think about doing that this week! Thank you for the suggestion!

Note that this requires some refactoring in order to match the standard behavior, so it probably won't release for a bit.


why my memory always

my conditions: linear kernel ; six features; Binary classification（good or bad）
my question: when my points reach to 15000（good）and 15000（bad）；at this time its training，the Memory is increasing all the time，and then Memory crash，my PC memory is 8G

For libevm, it stores kernel elements. That's why your memory is not enough. If you use liblinear,  it does not store kernel values so you won't get memory problems


Ionic serve to Firefox fails

I did:
ionic serve -w firefox
and got:
Error opening the browser -  [TypeError: undefined is not a function]

I see the issue. I'll patch it today.

Out in 1.5.1.


New RP Microsoft.AzureTerraform.Management - export Azure resources to Terraform HCL

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[x] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[ ] Other, please clarify:

edit this with your clarification

This PR introduces a new RP Microsoft.AzureTerraform.Management which allows user to export existing Azure resources into a Terraform HCL.
This RP has been merged to RPSaaSMaster branch of the private (-pr) repo. Check via git log -- specification/terraform.
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Armstrong testing
https://github.com/gerrytan/azure-terraform-armstrong (private repo)

The detected lint diff errors are false positives.

API TEST ERROR REPORT

Rule
Message

Coverage Report

Operation
Tested properties
Total properties
Coverage

All
25
25
100.0%

Operations_List
0
0
100.0%

ExportTerraform
25
25
100.0%

OperationStatuses_Get
0
0
100.0%

The main.tf file used to ran API test (armstrong): https://github.com/gerrytan/azure-terraform-armstrong/blob/fd630c1274c398990b7aafa7438529dde091cf09/Microsoft.AzureTerraform/main.tf

API TEST ERROR REPORT

Rule
Message

Coverage Report

Operation
Tested properties
Total properties
Coverage

All
25
25
100.0%

ExportTerraform
25
25
100.0%

OperationStatuses_Get
0
0
100.0%

Operations_List
0
0
100.0%


Set tag for MenuBar

If MenuBar replaces a different (stateless) widget in the view for an application it causes a downcast error because the state is never initialised correctly: since tag() is not implemented for it, diff() gets called instead of state()/children().

you closed it before i could merge it?


Maintance

Custom route/ navigation
Structured

@AarushThapa
Please rebase to master branch and try to squash all the commits

@poudyalanil squash merge & delete the branch once reviewed.


RewriteMultiTerm cannot set N for TopTermsN, TopTermsBoostN or TopTermsBlendedFreqsN

RewriteMultiTerm exposes the multi term query rewrite options as an enum. An issue with this is that no value can be passed for N for any of

TopTermsN
TopTermsBoostN
TopTermsBlendedFreqsN

This affects all major versions.

Will be in the next 2.x and 5.x release


Modify Scholar to work with newer packages

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


libvirt_network terraform import crash

System Information
Linux distribution
Archlinux
Terraform version
Terraform v0.13.3
+ provider registry.terraform.io/dmacvicar/libvirt v0.6.2
+ provider registry.terraform.io/hashicorp/template v2.1.2
+ provider registry.terraform.io/hashicorp/vault v2.14.0

Provider and libvirt versions
0.6.2

Checklist

[ NO ] Is your issue/contribution related with enabling some setting/option exposed by libvirt that the plugin does not yet support, or requires changing/extending the provider terraform schema?

[x] Is it a bug or something that does not work as expected? Please make sure you fill the version information below:

Description of Issue/Question
Create in the destination machine QEMU a network, then reproduce the same configuration in terraform and try to import into the current state.

I don't remember if we support the operation of import. I remember in past we had similar issue
Are you willing to drive this research task and help the provider?
So we would need:

research if it is possible with the provider to do it for all resource, track down some issue
research if we can implement and HOW we would do it.

I can give additional pointer in case.
Tia

I've been trying to run an import as well, and had reasonable success with
$ terraform version
Terraform v0.13.4
+ provider registry.terraform.io/dmacvicar/libvirt v0.6.2
+ provider registry.terraform.io/hashicorp/template v2.2.0

I've hit a couple of problems though:

I have no idea what the id should be for libvirt_ignition resources. I see that the id is composed of a filesystem path and a UUID but don't see how the UUID is generated or to which libvirt resource it is associated
after import, all resources are marked as to be replaced for various reasons

It gets me closer since I am at least able to get the outputs from terraform, but it's not working perfectly. In case it matters, I ran the import after (somehow) the terraform state stored on an s3 bucket became unusable.

@MalloZup If import is not supported, then is now way to keep a configuration like mine. Because I cannot create neither maintain later, so seems is needed to add more parameters inside the module.
If you don't mind, I'll take a look of how to do it to prepare a pull request with the item supported in creation. One day we could add the import functionality.

sure thx @Nassiel :love_you_gesture:

Hi @MalloZup I have the pull request almost ready. But I'd appreciate some help, I want to avoid some combinations like add extra interfaces and put bridge at the same time. On the tests, the test fail if I do that and the condition is being evaluated correctly but for some reason, Terraform plan says "yeah, all fine, go on" and of course later the apply fails.
Any idea?? Here is the branch.

Hi @MalloZup I have the pull request almost ready. But I'd appreciate some help, I want to avoid some combinations like add extra interfaces and put bridge at the same time. On the tests, the test fail if I do that and the condition is being evaluated correctly but for some reason, Terraform plan says "yeah, all fine, go on" and of course later the apply fails.
Any idea?? Here is the branch.

Has there been any progress on this? Would be great to create a "base infrastructure" and then import this into other projects that depend on it. For example, in my homelab server I have one virtual machine with multiple network interfaces attached to it. It has Nginx running that forwards requests to the appropriate virtual machine on a different subnet. A new project could then create resources on this existing infrastructure e.g. create a virtual machine on a different subnet. I would be interested to help but not sure where to begin.

@abdullahchaudhry @MalloZup Many things have happened since then, but, AFAIK, the issue on this topic wasn't addressed.  I'll update it with the latest commits and see if it's solved. If not, I'm happy to retake it if someone can also take a look at the error.

Thank you @Nassiel

Hi, @abdullahchaudhry seems solved, now they use a function to iterate lookupMachineand there it searches by canonical name or name. So I'd consider this solved.


Various "C+" improvements

Some possibilities:

import instead of #include when not referring to system headers
string type with helper syntax
basic polymorphic containers (dist, list, slice)
range based for loop to go with containers
range notation

Obviously first would need way to opt in per file/per section, and a way of cleanly integrating/generating down to normal C.

I didn't try it, but https://github.com/red0124/sgc looks straightforward, and could be not-too-verbose with some helper syntax sugar.

Some random scribblings on that idea (user syntax at bottom)
#pragma luv
// #includes all of SGC.
// reserves: `vector, `str, `list, `map, `set, `hashmap, `hashset, ...
//
// at parse time `vector(int) v:
// 1. inserts SGC_INIT_HEADERS(VECTOR, int, _luv_vector__int_) at top level
// 2. adds SGC_INIT_DEFINITIONS(VECTOR, int, _luv_vector__int_) in a temporary ltcg.c
// 3. declares _luv_vector__int_ v
// 4. calls v..init();
//
// where .. syntax is also specially handled and does:
//
//   x..something(a, b, c) <==> <static_type_of_x>_something(&x, a, b, c)
//
// e.g.
//
//   v..push_back(1)  ==>  _luv_vector_int__push_back(&v, 1)
//
// .. is chosen because it's not too ugly, and works with clang-format.
//
// :: also mostly works, makes me think of static calls (sort of what it is, so maybe good?)
// doesn't format quite as reliably as ..

// :, @, #, ! don't work for format
// $ doesn't work because it's sorta an identifier character in C
// ^ and | maybe plausible but always get spaces which looks a bit weird
//
// Single . could maybe be made to work since we have to know the static type of the lhs anyway
// - ambiguous for member access, but prettier
// - possibly more confusing what's happening?
// - ? if rhs of . exists on structure, then it's just a field access, otherwise member invoke
//
// What about pointers? declaration seems fine, but then more magic required in
// ., .., -> etc.

#if 0
SGC_INIT(VECTOR, int, vec)

int main(void) {
  vec v;
  vec_init(&v);

  vec_push_back(&v, 1);
  vec_push_back(&v, 5);

  *vec_at(&v, 0) = 2;

  for_each(i IN v AS vec) { printf("%d ", *i); }

  vec_free(&v);
  return 0;
}
#endif  // Would be written as vvv

int main() {
  `vector(int) v;

  v.push_back(1);
  v.push_back(5);

  v.at(0) = 2;

  foreach (i, v) {
    printf("%d ", *i);
  }

  v.free();
}

No need for #pragma, or wacky prefixes. Instead, just use_Vec, _Str`, etc. Those could almost be macros but not quite because some stuff it would expand to needs to be injected at top-level or in a single translation unit.
I think the operator overloading for . is fine too then because we have a bunch of known types so we can just not have any "public" fields, or special case any if necessary.

apparently reserving _Str isn't going to work so well since we want to use Microsoft's std*.h:
C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.34.31933\include/vcruntime_string.h:71:     _In_z_ char const* _Str,
                                                                                ^ error: expected ','

One of __str, @str, $str, __Str, @Str, $Str maybe

I started by implementing an __attribute__ on structs that makes them callable method-like: https://github.com/sgraham/dyibicc/commit/496d31c8e52c0712056c6026101e99cdbf12d77b#diff-5b99019f6d3db6895298385f15e166a8f07ed2b61cffb0b3fc7d0569571f6f62 .
Then I thought the rest might just be a matter of having the preprocessor map $vec(int) (the user syntax) to _Vec$int (the generated instantiated container) and then as a side-effect, doing the #include of the container type (as in something like https://github.com/stclib/STC#usage which seems like a nice container library), along with a bit of caching/unique-ing/whatever.
And that was almost enough for it to work! One problem that comes up is where to insert the header when it's magically instantiated. Since the preprocessor is doing the work, the only two possible locations at the head and tail of the current file (because we have no lexical context to know where "global scope" is). The tail seemed wrong because we need (e.g.) the struct definition early. So the head seemed like it'd work, but the problem is that by the time we see $vec(int) other files have been included, say, stdint.h that might be needed by the container. But when the container header sees an #include of stdint.h the include guard will avoid the contents of stdint being included again. So if the tokens for the container are injected at the head of the file they won't be in the correct order. So we sort of want the container injected ... at global scope ... after "some stuff" ... and before "some other stuff". Which doesn't sound great.
I could force include some headers (but it'd probably have to be always, not only when containers are in use, which is no good).
Perhaps rewrite the containers to need to include any files?
A related problem that I haven't thought too much about is what to do when the $vec(int) is in a header/forward-declaration situation where it shouldn't be defining anything yet. Or at least not re-defining when done multiple-y. It might not matter (if each definition only gets instantiated once per translation unit) or... if it does, maybe some sort of hacky attribute that allows struct redefinition, as long as it matches the previous one (which feels like encroaching on C++ ODR nose-demon territory 🤮).

Containers are actually working (?) in a branch now. This code https://github.com/sgraham/dyibicc/commit/3c3ad8e0b6875fc901507ca1fcfa2d6d8b9fac39 doesn't feel too awful, though the implementation is maybe a bit unpleasant (and of course ignoring the lack of wisdom in extending C at the syntax level)!
The extensions are:

__attribute__((methodcall(PREFIX))) which allows the .. syntax that adds the "self" and follows pointers if necessary
supporting and baking in $vec(T) and $map(K,V) which are the first two container types supported (embedded hacked versions of https://github.com/stclib/STC)

(I think I'm probably going to remove this as it's too hacky even for me, and put this type of functionality into a more extensive alternate "front-end".)


We need more questions for the certifying exam... put your ideas here...

We have made a new database of OSPAT certifying questions, there is currently one question in the database we probably need at least 20. Please use this thread to submit any questions you have. For instance...
Which of the following psychedelics has the shortest duration of action?
=DMT
LSD
Psilocybin
Mescaline
Ketamine
Which of the following psychedelics has the quickest onset of action?
=Smoked DMT
Ingested LSD
Ingested Psilocybin
Ingested Mescaline from Bridgesi
Ingested Ketamine

Which of the following is a correct statement regarding the effects of LSD on the brain?
Stimulates the activity of dopamine
Inhibits the activity of serotonin
=Stimulates the activity of serotonin
Imitates the activity of glutamate
Imitates the activity of dopamine
How was LSD discovered?
Ancient civilizations have eaten their leaves for thousands of years.
A German scientist was looking for a new stimulant drug.
=A Swiss scientist was looking for a new headache cure.
The CIA was looking for a new truth serum
What is one of the most dangerous aspects to eating psychedelic mushrooms?
They frequently cause heart attacks, even on the first use.
=Poisonous mushrooms can easily be mistaken for them.
They cause extreme overheating and dehydration.
Some people have profound allergies to them that can lead to death.
Where does mescaline come from?
A German scientist looking for a new stimulant drug discovered it.
It is found in the leaves of the mescalinus sativa plant.
=It is found in part of the peyote cactus.
It was developed in the 1920s as an anaesthetic.
Which of the following is true about PCP?
It was developed as a cure for migraine headaches.
It was developed as a truth serum for the CIA.
It was used in the 1950s as a blood pressure medication.
=It was used in the 1950s as a surgical anaesthetic.
In the brain, PCP:
Stimulates the activity of dopamine
Imitates the activity of dopamine
Disrupts the activity of serotonin
Stimulates the activity of glutamate
=Disrupts the activity of glutamate

These are great questions so far, although they all probably fall within the topic of molecules. We definitely need to have a wide range of questions focused on clinical skill and interventions for therapists on top of these.

Something like this?
Behaviour analysis is based upon the principles of:
~Classical conditioning.
=Operant conditioning.
~Dream analysis.
~All of the options.

Sorry for the delay. Yes that is getting warmer, although I should have been more specific. We need to have questions focused on clinical skill and interventions within the context of psychedelic therapy.
Some examples would be...
What is the best example of the appropriate use of therapeutic touch?
~Whenever the client experiences affect.
=Only in cases agreed upon by the client during preparation sessions.
~When the client is elated or overwhelmed and asking for physical touch.
~All of the above.
How many therapists should participate in sessions according to the MAPS MDMA-Assisted Therapy protocol?
~1
=2
~3
~As many as possible.

Would anyone else like to contribute?

I've been thinking about this lately and am starting to believe we should hold off until there are more contributors. In my mind it makes sense that a dedicated team or committee would take the lead on developing the questions but I don't think we have enough contributors for that yet.
Thoughts?

Well thats the main problem atmo. There's not enough contributors to this repo...

I agree.


basic windows support (-uname)

Basic windows support by side stepping os.uname() on exception

Coverage decreased (-0.6%) to 93.68% when pulling e0afba36e5a048f966847f5011efd5e3b0467b1e on feature/basic_windows_support into 91cf58beb45a877322b2fe9453258a885ea4f043 on master.


Multi-Language Problem

Hi Is there a way to enter content into another language when adding categories and products.

I don't think that we support content localization. @thiennn correct me if I'm wrong

now I want to write the description part of the product title turkish english and russian but when adding content it does not have a field

@fetullahyldz I sorry but this feature hasn't been supported yet

May be it's a good feature for upcoming releases

Will  work on it this weekend

We should schedule everything and focusing to stabilizing the platform rather than adding new feature
I think we should add this after the 1.0.0, let's stabilize and improve current localization module

Worked, basically

Thank you very much has been really great health and your project will be much better

Hello, opera vs google chrome does not choose language in mobile browser and there seems to be a problem


strman : static import issue

Hi Shekhar,
The other day I was playing with your code for strman: java library but I got some error.
My sole class comprised of main() and static import statement on top. Could you please assist on what went wrong. Is it coz of java version issue, m using java 8.
error: static import only from classes and interfaces
Regards
Sudip Nayak

@Sudip7 can you share code via gist or pastebin? There is stackoverflow issue that mentions the issue https://stackoverflow.com/questions/11782076/static-import-only-from-classes-and-interfaces

Please find the source code below:
`import static strman.append;
public class strmanJava {
public static void main(String args[]){
System.out.println("testing new java library");
}

}`
Pom.xml
`

4.0.0
<groupId>com.javalib</groupId>
<artifactId>Java8JsonLib</artifactId>
<version>1.0-SNAPSHOT</version>
<dependencies>
    <dependency>
        <groupId>com.shekhargulati</groupId>
        <artifactId>strman</artifactId>
        <version>0.1.0</version>
        <type>jar</type>
    </dependency>
</dependencies>

`

@Sudip7 sorry its my bad. README.md was incorrect. You have to import import static strman.Strman.append as all functions are inside Strman class. Also, if you face any issue can you raise  issues in the library Github project https://github.com/shekhargulati/strman-java

Now it's resolved. Thanks for quick assistance


Correct an eslint rule, ensure dependency is installed

This PR fixes the following:

airbnb eslint rule is to be installed
makes an exception (hopefully the last one) that parenthesis is as-needed
correct code accordingly

We can merge #9 when this is done


Buttons with image?

I have if I'm not mistaken (maybe I'm wrong now there is no access to the pc to check) xna texture2d and I want to display this texture on a button I just need a code I have no idea about the format of textures imgui.net / imgui

for some reason i cant display the texture instead of white square i use https://github.com/0x0ade/ImGuiCS/blob/master/ImGuiFNA/src/ImGuiXNAState.cs

already done (I used new ImGuiXNAState when the state already existed and it was a black / white square)

I can't provide support for another ImGui-based library. If you try to use this library (ImGui.NET) and have a specific question, preferably with a more complete description of what you've tried and why it didn't work, then feel free to open a new issue.

I can't provide support for another ImGui-based library. If you try to use this library (ImGui.NET) and have a specific question, preferably with a more complete description of what you've tried and why it didn't work, then feel free to open a new issue.


Uncaught TypeError: _reactDom.default.createPortal is not a function

i get this error when i try to use Dialog in material-ui v1
Uncaught TypeError: _reactDom.default.createPortal is not a function
at Portal.render (Portal.js:106)
at eval (ReactCompositeComponent.js:796)
at measureLifeCyclePerf (ReactCompositeComponent.js:75)
at ReactCompositeComponentWrapper._renderValidatedComponentWithoutOwnerOrContext (ReactCompositeComponent.js:795)
at ReactCompositeComponentWrapper._renderValidatedComponent (ReactCompositeComponent.js:822)
at ReactCompositeComponentWrapper._updateRenderedComponent (ReactCompositeComponent.js:746)
at ReactCompositeComponentWrapper._performComponentUpdate (ReactCompositeComponent.js:724)
at ReactCompositeComponentWrapper.updateComponent (ReactCompositeComponent.js:645)
at ReactCompositeComponentWrapper.performUpdateIfNecessary (ReactCompositeComponent.js:561)
at Object.performUpdateIfNecessary (ReactReconciler.js:157)

[x] This is a v1.x issue (v0.x is no longer maintained).
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

the expected behavior is to diplay the dialog like the exmple
Current Behavior

Error:
Uncaught TypeError: _reactDom.default.createPortal is not a function
at Portal.render (Portal.js:106)
at eval (ReactCompositeComponent.js:796)
at measureLifeCyclePerf (ReactCompositeComponent.js:75)
at ReactCompositeComponentWrapper._renderValidatedComponentWithoutOwnerOrContext (ReactCompositeComponent.js:795)
at ReactCompositeComponentWrapper._renderValidatedComponent (ReactCompositeComponent.js:822)
at ReactCompositeComponentWrapper._updateRenderedComponent (ReactCompositeComponent.js:746)
at ReactCompositeComponentWrapper._performComponentUpdate (ReactCompositeComponent.js:724)
at ReactCompositeComponentWrapper.updateComponent (ReactCompositeComponent.js:645)
at ReactCompositeComponentWrapper.performUpdateIfNecessary (ReactCompositeComponent.js:561)
at Object.performUpdateIfNecessary (ReactReconciler.js:157)
Context

Your Environment

Tech
Version

Material-UI
v1.0.0

React
15.4.1

browser
chrome

etc

Which bundler do you use?

we are using "webpack": "2.6.1"

I had the same issue my React version is 15.4.0 and webpack is 3.3.0

Check the installation warning. We only support React 16.3.0. You can always use the last v1 beta release if you can't upgrade React.

Upgrading my react version to 16.3.0 or greater fixed the issue.

I got the same error My reactJS version is 16.13

I found this will work and no warnings in Chrome console:
import React from 'react'
import ReactDOM from 'react-dom/client'
import PortalReactDOM from 'react-dom'

//...

class Modal extends React.Component {
  //...

  render() {
    return PortalReactDOM.createPortal(this.props.children, this.el);
  }
}

//...
const root = ReactDOM.createRoot(appRootEl);
root.render(<Parent />);

That is, use react-dom to createPortal, and use react-dom/client to creatRoot.


Tooling Questions

I have a few questions regarding the WinObjC project in general and its closed-source compiler in particular. Is it possible for this compiler be used to compile Objective-C code that does not necessarily target the UWP? (I don't care about the libraries or the runtime, just the compiler.) This would be a big help for me in some of my desktop projects.
Also, is it possible to integrate the compiler into a third-party IDE which I am thinking of writing? If so, how would be the best way to do this? I find that the appeal of WinObjC is greatly reduced when I realize that currently I must use Microsoft's IMO sub-par tooling, so I have decided to create my own. Thanks!

closed-source compiler

The compiler is Clang/LLVM, which is Open Source software.

Is it possible for this compiler be used to compile Objective-C code that does not necessarily target the UWP? (I don't care about the libraries or the runtime, just the compiler.)

Yes (see above).


Change to log warning (for now) as hosted applications are using this…

… wrong.
@hmusum or @toregge please review

This is a followup of https://github.com/vespa-engine/vespa/pull/23478.


[#4992] support credential vending framework

What changes were proposed in this pull request?
support credential vending framework
Why are the changes needed?
Fix: #4992
Does this PR introduce any user-facing change?
no
How was this patch tested?

add UT
propose a draft PR in #4966 , and could run pass S3 token with Gravitino IcebergRESTServer

@jerryshao comments are addressed, please help to review again


MAISTRA-554 Don't error when pruning object that can't even exist

For example, if there's no Jaeger CRD in the system, deleting a Jaeger
resource will fail with NoKindMatchError. The pruner should not treat
this as an error, because the object can't exist if the CRD isn't there.

This should be fixed too https://github.com/Maistra/istio-operator/blob/maistra-0.12/deploy/maistra-operator.yaml#L198
jaegers => jaeger

@aljesusg No, jaegers is correct (must be plural).

ok Thanks @luksa


codecov ignore explicit paths not with regex

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

Current dependencies on/for this PR:

main

PR #709   👈

This comment was auto-generated by Graphite.


DistributedLockInterceptor doesn't use transactions

The logic inside the DistributedLockInterceptor does all logic without a transaction around it which results in race conditions. This should wrap the logic inside a transaction or should use advisory locks (https://www.postgresql.org/docs/current/explicit-locking.html#ADVISORY-LOCKS).

The Interceptor is not aware of any Databases so handling transactions in there would be a nightmare.
A Race Condition in this case should always end with the catched excpetion
I am open to have that in a more Transactionally way as it would help the redis implementation a lot!

PS: the jdbc implementation is catching a DataAccessException which is thrown in case of a race condition

Save being insert is just guaranteed for micronaut-data-jdbc and not for other micronaut-data projects, therefore this seems dangerous.
There are many possible race conditions like instance 1 getting an old lock and instance 2 setting a new one. Instance 1 will happily delete the fresh lock, as it just searches by name without another identifier. (See https://github.com/uberall/micronaut-distributed-lock/blob/c706beb4b85a3035fdbd803d4dbbb353715c4764/core/src/main/java/com/uberall/interceptors/DistributedLockInterceptor.java#L59-L63)
You can only eliminate these race conditions by moving the logic inside the transaction by either moving it into a more complex repository method or using logic which sets and checks in one query. I know this is a bigger refactoring, but otherwise you can't fix these issues. They are so complex, you may not be able to find them all, leave that to PostgreSQL developers.

We could overwrite the Interceptor in the jdbc implementation and simply add the @Transactional annotation.
That would mean that Instance 2 is still seeing the not yet deleted lock (transaction is not yet commited) and would try to save a new one too. It would fail as Instance 1 would have created it's lock again in the meantime (transaction was commited + lock.name has a unique key).
I do not want to force users of this library to use Postgre so "leave it to the Postgre" developers is exactly what we need to avoid here.
Feel free to create a MR if you have ideas!


Have a clear story around serialization and code limitations

Via @jeffposnick:

i was fooled by the "it's just TypeScript" nature of working on a board in VS Code, and didn't internalize the fact that there are limitations due to the eventual board serialization step. there's a (TODO: talk about isolation and inability to reference to other variables outside of code.) comment in the doc that i missed, and because i didn't see that, it was a lot of trial-and-error to figure out why i couldn't, say, define a function in my .ts file and then call it from within a board. even once i got the hang of using code definitions, i was not sure why i couldn't call things that can presumably be serialized properly, like core.fetch(), from within a code definition

Need to:

[ ] have a clear story of how code works and why it can't contain nodes.

Another bit from exadev folks:
import { WebClient } from '@slack/web-api';

const slackClient = code(({slackToken}) => {
  const wc = new WebClient(slackToken)
  return {client: wc}
})

Maybe I didn’t install the package? No, I did. My IDE is perfectly happy with it too. Looking in the developer tools I see the error in the resultant javascript.

function fn_3({ slackToken }) {
  const wc = new WebClient(slackToken); <- error here
  return { client: wc };
}
self.onmessage = () => self.postMessage({ result: JSON.stringify(fn_3({"slackToken":"djdj"})) });self.onerror = (e) => self.postMessage({ error: e.message })

@seefeldb


Feat celery threading #194

Overviewer now runs in a celery worker thread. Button in world browser starts the render. Show button does nothing. Use /database route to see renders. Previews will only work on linux out of the box.
Closes #194

App name is still 'Minecraft Madness test'
Isn't it possible / better to just check if preview exists than to store arbitrary value in db?
Maybe we should make iterative super_safe_join so that full path relative to app.root can be set in config.
Order in README is faulty.

super_safe_join is now live in master

I just wrote a join function too :P


223 Store User Job

Functionality is as follows

If you are not signed into an account submitted job ID's are saved to the session (temporary)
If you are signed in then the job ID is added to a database (permanent) linked to the account

These are two different tables in the "Jobs" tab so the user can tell which saved job ID's might be lost (in session) and which will persist forever (in database)

I tested this both on Firefox and safari and the log-in is working great.   e.g. logging in to a new browser (e.g. new session) jobs are displayed for logged-in user.   Merge request approved!  I'm going to merge it in to dev so I can rebase my own branch.


Update fr.json : better translation for storage_rental

Description, Motivation & Context

Related issues

Links and data
Relevant OSM Wiki links:

…

Relevant tag usage stats:

…

Checklist and Test-Documentation Template
Read on to get your PR merged faster…
Follow these steps to test your PR yourself and make it a lot easier and faster for maintainers to check and approve it.
This is how it works:

After you submit your PR, the system will create a preview and comment on your PR:

🍱 You can preview the tagging presets of this pull request here.
If this is your first contribution to this project, the preview will not happen right away but requires a click from one of the project members. We will do this ASAP.

Once the preview is ready, use it to test your changes.

Now copy the snippet below into a new comment and fill out the blanks.

Now your PR is ready to be reviewed.

## Test-Documentation

### Preview links & Sidebar Screenshots

<!-- Use the preview to find examples, select the feature in question and **copy this link here**.
     Find examples of nodes/areas. Find examples with a lot of tags or very few tags. – Whatever helps to test this thoroughly.
     Add relevant **screenshots** of the sidebar of those examples. -->

<!-- FYI: What we will check:
     - Is the [icon](https://github.com/ideditor/schema-builder/blob/main/ICONS.md) well chosen.
     - Are the fields well-structured and have good labels.
     - Do the dropdowns (etc.) work well and show helpful data. -->

### Search

<!-- **Test the search** of your preset and share relevant **screenshots** here.
     - Test the preset name as search terms.
     - Also test the preset terms and aliases as search terms (if present). -->

### Info-`i`

<!-- **Test the info-i** for your fields and preset and share relevant **screenshots** here.
     The info needs to help mappers understand the preset and when to use it.
     [Learn more…](https://github.com/tordans/id-tagging-schema/blob/main/CONTRIBUTING.md#info-i)
 -->

### Wording

- [ ] American English
- [ ] `name`, `aliases` (if present) use Title Case
- [ ] `terms` (if present) use lower case, sorted A-Z
<!-- Learn more in https://github.com/openstreetmap/id-tagging-schema/blob/main/GUIDELINES.md#2-design-the-preset -->

Hey, thanks for your contribution.
However, please note that everything in the dist folder is read-only.
So, how to contribute translations, then? You can do this in transifex, see https://github.com/openstreetmap/id-tagging-schema/blob/main/CONTRIBUTING.md#Translating


Feature/aggregates as windows

Which issue does this PR close?
Closes #865
Closes #688
Rationale for this change
This function allows the user to turn any aggregate function into a window function. Previously using aggregates as windows was only expressed through functions.window() which took the aggregate's name. This only supports built in window functions or functions that have been registered with the session context. While this worked, it's a bit of an anti-pattern for people who don't need to register functions into the session context.
What changes are included in this PR?
Adds .over() to an expression which indicates it is to be used as a window function.
Are there any user-facing changes?
During testing I noticed that the default window frames for f.window() are set to unbounded preceeding to unbounded following. With .over() it uses the default frames, so if ordering is set it goes from unbounded preceeding to current row. This does match user expectation.
Example
Old version:
f.window(
    "first_value",
    [column("a")],
    order_by=[f.order_by(column("b"))],
    partition_by=[column("c")],

New version:
f.first_value(column("a")).over(
    partition_by=[column("c")],
    order_by=[column("b")]
)

During testing I noticed that the default window frames for f.window() are set to unbounded preceeding to unbounded following. With .over() it uses the default frames, so if ordering is set it goes from unbounded preceeding to current row. This does match user expectation.

So does this PR also close https://github.com/apache/datafusion-python/issues/688?

It does! Updated the description

I had just verified with a test - if this test is not duplicative then maybe we should add it.
def test_window_frame_defaults_match_postgres(ctx):
    # ref: https://github.com/apache/datafusion-python/issues/688

    # create a RecordBatch and a new DataFrame from it
    batch = pa.RecordBatch.from_arrays(
        [pa.array([1.0, 10.0, 20.0])],
        names=["a"],
    )

    df = ctx.create_dataframe([[batch]])

    window_frame = WindowFrame("rows", None, None)

    col_a = column("a")

    # Using `f.window` with or without an unbounded window_frame produces the same results
    no_frame = f.window("avg", [col_a]).alias('no_frame')
    with_frame = f.window("avg", [col_a], window_frame=window_frame).alias('with_frame')
    df_1 = df.select(col_a, no_frame, with_frame)

    expected = """DataFrame()
+------+--------------------+--------------------+
| a    | no_frame           | with_frame         |
+------+--------------------+--------------------+
| 1.0  | 10.333333333333334 | 10.333333333333334 |
| 10.0 | 10.333333333333334 | 10.333333333333334 |
| 20.0 | 10.333333333333334 | 10.333333333333334 |
+------+--------------------+--------------------+
""".strip()

    assert str(df_1) == expected

    # When `order_by` is set, the default window should be `unbounded preceding` to `current row`
    no_order = f.avg(col_a).over(Window()).alias('over_no_order')
    with_order = f.avg(col_a).over(Window(order_by=[col_a])).alias('over_with_order')
    df_2 = df.select(col_a, no_order, with_order)

    expected = """DataFrame()
+------+--------------------+--------------------+
| a    | over_no_order      | over_with_order    |
+------+--------------------+--------------------+
| 1.0  | 10.333333333333334 | 1.0                |
| 10.0 | 10.333333333333334 | 5.5                |
| 20.0 | 10.333333333333334 | 10.333333333333334 |
+------+--------------------+--------------------+
""".strip()
    assert str(df_2) == expected


Better parsing for Layouts with int/bool type.

todo

[ ] throw error if throwConfigExceptions is true

This change is

Current coverage is 82% (diff: 0%)

Merging #1815 into master will increase coverage by <1%

@@             master      #1815   diff @@
==========================================
  Files           276        277      +1
  Lines         18373      19918   +1545
  Methods        2848       3238    +390
  Messages          0          0
  Branches       2107       2462    +355
==========================================
+ Hits          14996      16314   +1318
- Misses         2897       3105    +208
- Partials        480        499     +19

Powered by Codecov. Last update 9c1d7b4...35a0b67


Show Labels bugs

Thanx 4 almost the best Twitter extension, I think latest updates cant's let labels disappear & only show on hover. Pleaze fix it.

Hi @Tageel83 thanks for the report, this is a new bug. Will prioritize this.

https://github.com/typefully/minimal-twitter/commit/df7dac42d2f85a57fbb42aa343596c4dd40deba5

https://github.com/typefully/minimal-twitter/pull/129/commits/24b51322f46f03bb0b7120c461745ff4d22734e7


feat: convenience construction of a threefry

Hmm, I think the test failures are probably due to a change in JAX. If you are able to fix that then I'd be happy to merge this.

I can't reproduce the error locally.

Which version of JAX are you using? I suspect it's probably due to an update on their end.
(I haven't checked this myself yet.)

Which version of JAX are you using? I suspect it's probably due to an update on their end. (I haven't checked this myself yet.)

0.4.25

Okay, so possibly an issue with the latest version of JAX! If this passes with an earlier version then that will be the culprit, and we'll need to figure out how to use JAX's changes.

Now that jax has a key type that isn't just an Array and can be used for annotations, this PR isn't necessary.


Implement a rolling least squares method.

In the example notebook we see rolling OLS done in the .rolling() context, and the text itself says it's a little slow. Well, polars_ols has an optimized version of least squares regression using an updating algorithm instead of reestimating at every window.
I know from reading previous issues that work on OLS is low priority, but this seems like a pretty straight forward job of porting the algorithm to this module's structure, so maybe we should give it a go? For economics and finance work, rolling OLS is one of the most used models, so it would be a very nice feature to have integrated here.

Of course! Hands are tied on KNN right now unfortunately...
Just curious, what is in polars_ds that is not in polars_ols? And what makes you want to port polars_ols's functionality over? Do you think some consolidation will be great for the polars + data science community?

Yeah, I don't think it's too healthy to have hyperspecific polars extensions like something for OLS itself, but more so domain specific packages, like data science, time series analysis, econometrics, etc...
I could see an extension separate from polars_ds being focused around econometrics, for example, and then integrating more stuff specific to OLS, like statistics on the coefficients themselves, panel econometrics... Something like plm does on R, as opposed to forecast/tidyverts.
Nothing against that package, though, I currently have it running on production in my job and it's been great. I just miss native and intentional integration with more of the data science toolset.

I will start with L2 and L1 regularization, and then move on to rolling regression

L1, L2 are done. Will start working on rolling this weekend. The first version will probably more restricted, e.g. no null values input columns... There is a small chance that it will be in the v0.5.1 release. Stay tuned.

A very early implementation: https://github.com/abstractqqq/polars_ds_extension/pull/221
Only supports very basic null policy right now. Skip null is not supported yet.

I am closing it for now. Some more skipping options, rolling l2, etc,. will be added.


[ISSUE #8046] Fix authentication context build for no extFields

Which Issue(s) This PR Fixes

Fixes #8046
Brief Description
Fix authentication context build for no extFields
How Did You Test This Change?
start producer/consumer without ak/sk

Codecov Report
Attention: Patch coverage is 0% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 42.85%. Comparing base (0f0324a) to head (b476281).
Report is 1 commits behind head on develop.

Files
Patch %
Lines

...n/builder/DefaultAuthenticationContextBuilder.java
0.00%
1 Missing and 1 partial :warning:

...on/builder/DefaultAuthorizationContextBuilder.java
0.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8102      +/-   ##
=============================================
- Coverage      42.88%   42.85%   -0.03%
- Complexity     10358    10360       +2
=============================================
  Files           1270     1270
  Lines          88689    88692       +3
  Branches       11398    11400       +2
=============================================
- Hits           38030    38013      -17
- Misses         45968    45987      +19
- Partials        4691     4692       +1

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

If the client sets ak/sk, then all request headers will have ak/sk information, including heartbeats.
And if ak/sk is not set on the client, those configurations should also be turned off on the server. Maybe this is not a bug @dingshuangxi888
authenticationEnabled=false
authorizationEnabled=false

If the client sets ak/sk, then all request headers will have ak/sk information, including heartbeats. And if ak/sk is not set on the client, those configurations should also be turned off on the server. Maybe this is not a bug @dingshuangxi888
authenticationEnabled=false
authorizationEnabled=false

The old client has no ReqeustHeader, such as Heartbeat api, if the ak/sk is not set, it will get NPE error.

If the client sets ak/sk, then all request headers will have ak/sk information, including heartbeats. And if ak/sk is not set on the client, those configurations should also be turned off on the server. Maybe this is not a bug @dingshuangxi888
authenticationEnabled=false
authorizationEnabled=false

The old client may has no ReqeustHeader, such as Heartbeat api, if the ak/sk is not set, the extFields will be null and it will get NPE error.

OK

@dingshuangxi888  Some unit-tests maybe need to be resolved.


Adding database schema to MVCForum tables to allow easier integration with other applications.

I've updated all of the implementations of EntityTypeConfiguration<> to call ToTable() setting the name of the tables and the schema name so that it is easier to integrate MVCForum with other MVC apps and have a combined schema. If there is an appSetting with a key of MVCForum:SchemaName [1] present in web.config, that name is used when the tables are created, otherwise the default schema name of "dbo" is used.
[1] web.config changes
<appSettings>    <add key="MVCForum:SchemaName" value="MVCForum"/> </appSettings>

Thanks for the PR. I can't add this I'm afraid as it will break existing sites. Will keep it in mind for the v2.0 change though.


Fix integration tests to be executable on any Kubernetes cluster

Fix tests to run on any Kubernetes.

Use port forward to access applications.
Use hostname for loadbalancer if ip is not available

Tested on EKS.

Integration tests in GH runners are executed using model testing: tox -vve integration -- --model testing
In our test in this PR, when we try to setup port-forward, we assume model kubeflow (see in-line comments).
If we use ops_test.model_name instead it will make tests more portable and with no disturbance to workflows. Unless mlflow requires kubeflow model, the the workflow needs to change to use kubeflow model.


Update cloudwatch, sdk-core, ses, sns, sqs, ssm to 2.20.111

About this PR
📦 Updates

software.amazon.awssdk:cloudwatch
software.amazon.awssdk:sdk-core
software.amazon.awssdk:ses
software.amazon.awssdk:sns
software.amazon.awssdk:sqs
software.amazon.awssdk:ssm

from 2.20.110 to 2.20.111
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #2811.


How to search for Chatter discussion id in Bread search bar

Version information

Laravel: v5.5.48
Voyager: v1.3.1
PHP: 7.4.1
Database: MySQL 5.7.28
Description of problem
I integrate Voyager with Chatter and created Chatter discussion bread and Chatter post bread.
I want to get all post from a specific discussion. Any way to do it?
Screenshot:

As the issue template says questions should be asked on Voyager Slack group, in the right channel.
We prefer to keep github for bug reports and feature requests.


Address PR comments in the refactor branch

This PR is part of the refactor series #481. This addresses the review comments for the following PR:
#516
#521
#531
#534

Thanks @baothientran!


K8s

Additions/changes based on work done for ez crypto devops

Add k8s minikube section
Add k8s gcp section
Upgraded docker files

I agree about the readme. I was going to make one but there will be an instant conflict with the other PR. I can make one after that's merged in if you're agreeable to that