Add support for ES6 using Babel

I can contribute a PR for this

Any progress on this? It looks a bit strange to me, that the gulpfile is written in es6 (babel-enhanced) while the application itself still doesn't support es6.

I don't see why not. It's just JavaScript, at the end. :+1:

:+1: push ..

May want to just hold off on any effort here. WSK is undergoing a big change soon now that MDL is released. Let's wait and see what the team has planned for that before putting any effort into rewriting what currently exists.

@Garbee any more info about the big change? Has that happened?

MDL has been made the style setup for WSK already. This was the major change, going from an internally managed component system to external. This means the need for direct babel support internally is lowered greatly since there is no real scripting occurring within WSK (just a small basic starter script.)
I believe it would be best if WSK left this alone and for developers that want babel, they can freely opt-in by editing the setup themselves (as you should be doing anyways.) Since there isn't much logic happening in WSK I don't see why we should force babel on people by default.

It's already happened as per https://github.com/google/web-starter-kit/issues/742


Dev Home GitHub extension should support Enterprise accounts

Suggested new feature or improvement
Support all types of GitHub accounts.
Scenario
Support all types of GitHub accounts.
Additional details
No response

https://github.com/microsoft/devhomegithubextension/pull/286


Version sometimes looses precision

Problem:
I have seen one case where the defined priority of first os-release, then lsb_release, then distro release file, causes version() to return just the major version, when the system actually also has a minor version available. So we do not return all possible version detail, i.e. we "loose some precision".
The case I have seen is Debian 8.2, and that case is actually represented in the new test structure exactly as I have seen it on a real Debian 8.2 system:

/etc/os-release file defining VERSION=8, VERSION_ID=8
/etc/debian-version file containing just "8.2"

The current code simply goes by priority.
Solution:
If the code would be changed to go by best precision (out of all available sources), this could be addressed. The downside of such a solution is that it makes inconsistencies between the multiple sources even harder to understand, particularly if the inconsistency is not just more or less precision but even contradictions in the version parts that are available in multiple sources).
I think however, in the end it would be good to use the highest available precision for the version.

Because this would change the content of version(), I think this should be resolved before we declare the project as being ready.

I don't know.. I tend to want to go with the more explicit, more standardized, less precise solution for two reasons:

precision will have to be backward compatible with things that will not be a part of the common standard at some point. For instance, of Debian decided to drop .2 in os-release, this means that when debian 12 comes out, if they decide to leave behind debian_version for some reason, we will be looking at things that are not a part of the standard they chose to go with.
I'm not familiar with a clear definition of precision in this case. That is, i've seen a debian_version file with only 81 (being 8.1). I'd rather stay away from assumptions. If you're familiar with a unified standard on top of which we can create a precision algorithm, let's consider it.

I just want to note that users have full access to the parsed information anyway so, potentially, they should be able to access the version fields everywhere in the hierarchy. That would make me feel safer as users can code their way into understanding whether they need the most precise one or not based on their needs without us makes assumptions for them.

I see your point about the problem of parsing sources such as debian-version.
So let me change the proposal: We could use only the (up to) 3 information sources we parse right now, and use the version out of those with the best precision.
For example, Debian 8 has this:

"lsb_release -r -s" returns: 8.2
/etc/os-release defines: VERSION_ID="8"
no distro release file

The current "fixed priority approach" causes version() to return 8, the new "best precision" approach would compare the two available versions and return 8.2 as the more precise one. The debian_version file would not be used in that approach.
How does that sound?

debian_version was somewhat of an example. It's not the only problem. The problem is that looking for the best precision is in itself ambiguous. Let's say that you're on RHEL 7 and found the os-release file. Now you rely on os-release to retrieve any relevant information. ld, in that case, looking for the best precision will disregard the fact that every other piece of information was taken from os-release.
The ambiguity begins where you don't know where information will be extracted from. I'd rather have a single source of truth where possible and allow the user to go the extra mile instead of doing it for the user but leaving them scratching their heads.
I might be taking this too far, and I also might be basing this solely on my intuition, but I think it would cause more confusion than value.
What do you think?

I can definitely see both sides here.
How about a compromise: Would you be opposed to leaving version() as it is now, and adding a new best_version() that returns the most precise version, and documents well enough in its description that it may use a different source than the other methods?

That's actually a great idea. I'll go for that.

This issue is addressed by PR #52.


Get HTTP Error 502 while using mriqc web API

Hi, @oesteban @dbirman
After MRIQC (most recent docker version) finished succesfully on my small dataset, I would like to interprete the results based on the distribution of all the IQMs of my dataset and the shared dataset from mriqc web-api. When I am trying the code below, it is giving the below problems after a while. Can you tell me why this is happening? Thanks.
code:
`# T1
df_t1w = get_iqms('T1w', software=None)
df_t1w_unique = df_t1w.drop_duplicates(subset=['provenance.md5sum'])
BOLD
df_bold = get_iqms('bold')
df_bold_unique = df_bold.drop_duplicates(subset=['provenance.md5sum'])`
error message:
`HTTPError                                 Traceback (most recent call last)
 in
1 # T1
----> 2 df_t1w = get_iqms('T1w', software=None)
3 df_t1w_unique = df_t1w.drop_duplicates(subset=['provenance.md5sum'])
4
5 # BOLD
 in get_iqms(modality, versions, software)
24                 query='where={%s}&page=%d' % (','.join(query), page)
25             )
---> 26             with urllib.request.urlopen(page_url) as url:
27                 data = json.loads(url.read().decode())
28                 dfs.append(json_normalize(data['_items']))
/usr/local/miniconda/lib/python3.7/urllib/request.py in urlopen(url, data, timeout, cafile, capath, cadefault, context)
220     else:
221         opener = _opener
--> 222     return opener.open(url, data, timeout)
223
224 def install_opener(opener):
/usr/local/miniconda/lib/python3.7/urllib/request.py in open(self, fullurl, data, timeout)
529         for processor in self.process_response.get(protocol, []):
530             meth = getattr(processor, meth_name)
--> 531             response = meth(req, response)
532
533         return response
/usr/local/miniconda/lib/python3.7/urllib/request.py in http_response(self, request, response)
639         if not (200 <= code < 300):
640             response = self.parent.error(
--> 641                 'http', request, response, code, msg, hdrs)
642
643         return response
/usr/local/miniconda/lib/python3.7/urllib/request.py in error(self, proto, *args)
567         if http_err:
568             args = (dict, 'default', 'http_error_default') + orig_args
--> 569             return self._call_chain(*args)
570
571 # XXX probably also want an abstract factory that knows when it makes
/usr/local/miniconda/lib/python3.7/urllib/request.py in _call_chain(self, chain, kind, meth_name, *args)
501         for handler in handlers:
502             func = getattr(handler, meth_name)
--> 503             result = func(*args)
504             if result is not None:
505                 return result
/usr/local/miniconda/lib/python3.7/urllib/request.py in http_error_default(self, req, fp, code, msg, hdrs)
647 class HTTPDefaultErrorHandler(BaseHandler):
648     def http_error_default(self, req, fp, code, msg, hdrs):
--> 649         raise HTTPError(req.full_url, code, msg, hdrs, fp)
650
651 class HTTPRedirectHandler(BaseHandler):
HTTPError: HTTP Error 502: Bad Gateway`

Sorry, I'm afraid you attempted to access the database at a moment it was going through some hiccups. Please reopen if this is till an issue.


Add sparse_matrix method to PauliSum

There doesn't seem to be an easy way to get the matrix of a PauliSum. It has no matrix method, nor do we have a matrix protocol. Typically one actually wants the matrix as a Scipy sparse matrix rather than a Numpy array. We should make this easy. Should it be a method or a protocol?

At Cirq sync we decided to first add a matrix and/or sparse_matrix method to PauliSum, and only consider making these protocols if there are more use cases. The methods added to PauliSum should take qubit_order as argument.

OpenFermion has an analogous method but it gets quite slow and memory intensive for large matrices and I suspect it can be implemented more efficiently.


Considering integrating StreamPipes and DSS?

Apache StreamPipes is a very great project in the field of tream processing frameworks, and I think it is a good way to enhance the influence of our two projects is by integrating StreamPipes with DataSphere Studio.
DataSphere Studio is a one-stop data application development and management portal open source by WeBank, which meets the needs of the entire process of data application development from data exchange, desensitization and cleaning, analysis and mining, quality inspection, visual display, regular scheduling to data output.
Github address: https://github.com/WeBankFinTech/DataSphereStudio

Hi, thank you very much for the suggestion. I had a look at the project and I think an integration would be great.
Do you have any suggestions how the integration could look like?  I think it would be cool to have Adapters and Sinks in StreamPipes for DataSphereStudio.


Bugfix for #11 Nearest neighbor check

#11 bug fix for neigherst neghbor check
While testing the nearest neighbor check, i found the problem that the loop already overwrites the cell, so the board is simultanously checking neighbors it is alreay party going to the next generation. We need a temporary board where changes will be marked down on.
Create a temporary board where updates for the next generation are stored that will replace the previous board after every cell and its neighbors are checked.

import numpy as np
import random
np.set_printoptions(threshold=10000)
numberOfRows = 7
numberOfColumns = 7
board = np.random.choice([0, 1], size=(7, 7), p=[0.5, 0.5])
temp_board = np.zeros([numberOfRows, numberOfColumns]).astype(int)
Randomly fill the board with 0's and 1's
for rows in range(numberOfRows):
for columns in range(numberOfColumns):
board[rows][columns] = random.randint(0, 1)
#print(board)
2D Array for neighborhood
numNeighbors = 0
print(board)
for _ in range(3):
for rows in range(numberOfRows):
for columns in range(numberOfColumns):
# checks all neighboring cells
if board[rows,columns]:
numNeighbors = np.sum(board[rows-1:rows+2,columns-1:columns+2])-1
else:
numNeighbors = np.sum(board[rows-1:rows+2,columns-1:columns+2])
            # Update the status for the next generation
        if numNeighbors >= 4:
            temp_board[rows,columns] = 0
        if numNeighbors == 3:
            temp_board[rows,columns] = 1
        elif numNeighbors <= 2:
            temp_board[rows,columns] = 0
            # Reset the number of neighbors for checking the next array position

        numNeighbors = 0
board = temp_board
temp_board = np.zeros([numberOfRows, numberOfColumns]).astype(int)
print(board)

Can someone test this code and check if the rules are being followed? So if the board behaves from generation to generation in an expected way?

I'll test tomorrow


GIF branch

Coverage remained the same at 63.111% when pulling faf68687eca9a9c277ff6de1a7d4b7da085e25dd on pfernique:master into bd76679bfe42b2a9503ccfa34f19ebedd42e046e on StatisKit:master.


Further OpenCL kernel improvements

This patch improves OpenCL kernel in the following aspects:

Shrink blake2b_state structure;
Eliminate the cost of table look-up;
Explict data access for efficiency;
Avoid unnecessary initializations;

@Inkeliz, can you please do some comparison between commit c7f05fa19b819c6c3b8c0bd6f55c4e1531db3a47 and this pull request with your setup?

I'm using that code:
func BenchmarkGenerateWork(b *testing.B) {
	b.StopTimer()
	rand.Seed(1) // To make always test the same

	worker, err := NewWorkerGPU()
	if err != nil {
		panic(err)
	}

	DefaultWorkerPool = NewPool(worker) // To only use GPU (removing it will use CPU+GPU)

	for n := 0; n < b.N; n++ {
		b.StopTimer()
		hash, difficulty := make([]byte, 32), V2BaseDifficult
		rand.Read(hash)

		b.StartTimer()
		start := time.Now()
		_, err := GenerateWork(hash, difficulty)
		if err != nil {
			b.Error(err)
		}
		end := time.Now().Sub(start)
		b.StopTimer()

		fmt.Println(end)
	}

	b.StopTimer()
}

Just creating an gpu_cl_test.go and then execute:

go test -tags cl -bench BenchmarkGenerateWork2 -benchtime 20x -timeout 1h

Since it needs multiples samples, I make that other one to store the results:
func BenchmarkGenerateWork(b *testing.B) {
	b.StopTimer()
	rand.Seed(1) // To make always test the same

	worker, err := NewWorkerGPU()
	if err != nil {
		b.Error(err)
	}

	DefaultWorkerPool = NewPool(worker)

	Start()

	for n := 0; n < b.N; n++ {
		b.StopTimer()
		hash, difficulty := make([]byte, 32), V2BaseDifficult
		rand.Read(hash)

		b.StartTimer()
		start := time.Now()
		_, err := GenerateWork(hash, difficulty)
		if err != nil {
			b.Error(err)
		}
		end := time.Now().Sub(start)
		b.StopTimer()

		Append(n, end)
	}

	b.StopTimer()

	End()

	return
}

var Results = make(map[int][]time.Duration, 1<<11)

func Start() {
	file, err := os.Open("gpu_cl_test_results.json")
	if err != nil {
		if os.IsNotExist(err) {
			return
		}
		panic(err)
	}

	defer file.Close()
	if err := json.NewDecoder(file).Decode(&Results); err != nil {
		panic(err)
	}
}

func Append(i int, d time.Duration) {
	if _, ok := Results[i]; !ok {
		Results[i] = make([]time.Duration, 0)
	}

	Results[i] = append(Results[i], d)
}

func End() {
	fileJSON, err := os.Create("gpu_cl_test_results.json")
	if err != nil {
		panic(err)
	}

	if err := json.NewEncoder(fileJSON).Encode(Results); err != nil {
		panic(err)
	}

	fileMD, err := os.Create("gpu_cl_test_results.md")
	if err != nil {
		panic(err)
	}

	if _, ok := Results[0]; !ok {
		return
	}

	for k := range Results[0] {
		fmt.Fprintf(fileMD, `| %d `, k)
	}
	fmt.Fprint(fileMD, "| ** AVG** | \n")

	for range Results[0] {
		fmt.Fprint(fileMD, `| -- `)
	}
	fmt.Fprint(fileMD, "| -- | \n")

	for i := 0; i < len(Results); i++ {
		total := int64(0)
		for _, vd := range Results[i] {
			fmt.Fprintf(fileMD, `| %s`, vd)
			total += vd.Nanoseconds()
		}

		for j := 0; j < (len(Results[0]) - len(Results[i])); j++ {
			fmt.Fprint(fileMD, `| - `)
		}

		fmt.Fprintf(fileMD, "| **%s** | \n", time.Duration(total / int64(len(Results[i]))))
	}
}

It's messy, but saves the result into gpu_cl_test_results.json and create the table at gpu_cl_test_results.md, which includes the AVG. The time is monotonic, so seems safe for such usage, I guess. I'll post the result using the new and the old one, I couldn't do that before because I can't use the device while computing the PoW. :\
But, thank you for the PR in advance. 👍

WOW. I did some tests now, it's seems to be really faster now. I really need to take a look on that. Because it's far faster than the old one. It's so fast that I'm actually thinking that is something wrong, or it was a really nice job.
It drops from 1.48719817 seconds per PoW to 0.299763834 seconds. Generating 100 PoWs takes 154.860s before, using only GPU, and now (with that new change) took 33.855s. I even put some if !IsValid(), all PoWs is generated are valid and uses the 8x difficulty.

I'll keep testing it futher.

I test much time, and the results are almost always the same, even with other seeds (at rand.Seed()). All generated PoWs are valid against IsValid and also against work_validate. I'm posting here just some of the results. Otherwise, the table will be so big. The "AVG" uses all the data that I have collected, not only those 3 results.

Version
1
2
3
AVG
AVG DIFF
AVG DIFF %

Old
0.862
0.862
0.863
0.861s

New
0.174
0.169
0.172
0.172s
-0.689s
-80.06%

Old
1.206
1.208
1.206
1.204s

New
0.240
0.244
0.240
0.242s
-0.962s
-79.88%

Old
0.830
0.832
0.830
0.829s

New
0.168
0.170
0.169
0.169s
-0.660s
-79.59%

Old
1.557
1.556
1.556
1.554s

New
0.316
0.315
0.311
0.314s
-1.240s
-79.82%

Old
0.988
0.979
0.980
0.980s

New
0.197
0.199
0.196
0.198s
-0.782s
-79.78%

Old
1.645
1.630
1.638
1.632s

New
0.325
0.327
0.326
0.327s
-1.305s
-79.96%

Old
3.100
3.057
3.059
3.063s

New
0.613
0.604
0.606
0.609s
-2.454s
-80.12%

Old
0.537
0.529
0.531
0.532s

New
0.110
0.109
0.109
0.109s
-0.422s
-79.41%

Old
3.214
3.210
3.207
3.208s

New
0.640
0.635
0.637
0.639s
-2.569s
-80.09%

Old
4.412
4.405
4.404
4.405s

New
0.871
0.869
0.880
0.876s
-3.529s
-80.11%

Old
1.854
1.855
1.856
1.855s

New
0.369
0.368
0.378
0.374s
-1.482s
-79.86%

Old
1.311
1.305
1.305
1.306s

New
0.260
0.260
0.279
0.266s
-1.041s
-79.66%

Old
9.644
9.629
9.630
9.631s

New
1.910
1.914
1.950
1.922s
-7.709s
-80.05%

Old
0.106
0.105
0.106
0.106s

New
0.024
0.024
0.024
0.025s
-0.081s
-76.57%

Old
1.556
1.555
1.557
1.555s

New
0.313
0.312
0.312
0.313s
-1.242s
-79.89%

Old
0.655
0.656
0.655
0.655s

New
0.135
0.133
0.134
0.134s
-0.521s
-79.52%

Old
0.157
0.155
0.156
0.156s

New
0.035
0.035
0.035
0.035s
-0.121s
-77.42%

Old
0.831
0.831
0.830
0.831s

New
0.170
0.167
0.175
0.170s
-0.661s
-79.55%

Old
3.299
3.279
3.282
3.283s

New
0.661
0.649
0.675
0.659s
-2.624s
-79.93%

Old
0.688
0.682
0.680
0.682s

New
0.140
0.138
0.142
0.140s
-0.542s
-79.47%

Old
3.779
3.756
3.757
3.760s

New
0.745
0.743
0.767
0.752s
-3.008s
-80%

Old
1.260
1.256
1.256
1.256s

New
0.251
0.250
0.257
0.253s
-1.003s
-79.84%

Old
1.444
1.430
1.430
1.433s

New
0.287
0.285
0.295
0.290s
-1.143s
-79.79%

Old
0.435
0.430
0.430
0.431s

New
0.092
0.090
0.090
0.090s
-0.341s
-79.06%

Old
1.655
1.630
1.631
1.634s

New
0.332
0.325
0.340
0.331s
-1.303s
-79.75%

Old
1.542
1.531
1.531
1.532s

New
0.320
0.305
0.315
0.312s
-1.221s
-79.67%

Old
0.606
0.606
0.607
0.606s

New
0.126
0.123
0.125
0.125s
-0.481s
-79.39%

Old
1.205
1.206
1.205
1.205s

New
0.245
0.241
0.244
0.244s
-0.961s
-79.76%

Old
2.339
2.330
2.333
2.330s

New
0.462
0.466
0.471
0.467s
-1.863s
-79.94%

Old
1.848
1.831
1.831
1.832s

New
0.365
0.369
0.368
0.368s
-1.464s
-79.93%

Old
0.867
0.856
0.856
0.857s

New
0.176
0.175
0.176
0.175s
-0.682s
-79.59%

Old
0.984
0.955
0.955
0.959s

New
0.192
0.195
0.196
0.194s
-0.765s
-79.74%

Old
1.940
1.929
1.931
1.930s

New
0.387
0.386
0.398
0.389s
-1.541s
-79.85%

Old
0.406
0.406
0.406
0.406s

New
0.086
0.085
0.084
0.085s
-0.321s
-79.12%

Old
0.130
0.131
0.130
0.130s

New
0.030
0.029
0.030
0.030s
-0.100s
-76.93%

Old
0.386
0.380
0.380
0.381s

New
0.080
0.079
0.081
0.080s
-0.302s
-79.05%

Old
0.081
0.080
0.081
0.081s

New
0.020
0.020
0.021
0.020s
-0.061s
-75.57%

Old
0.835
0.830
0.830
0.831s

New
0.170
0.167
0.170
0.169s
-0.662s
-79.66%

Old
1.969
1.955
1.957
1.960s

New
0.397
0.390
0.396
0.394s
-1.567s
-79.92%

Old
2.620
2.606
2.606
2.608s

New
0.519
0.518
0.521
0.520s
-2.088s
-80.05%

Old
0.130
0.130
0.131
0.130s

New
0.030
0.029
0.030
0.030s
-0.100s
-76.8%

Old
0.511
0.505
0.505
0.505s

New
0.105
0.104
0.105
0.104s
-0.401s
-79.34%

Old
2.415
2.406
2.406
2.406s

New
0.478
0.478
0.496
0.484s
-1.921s
-79.86%

Old
1.231
1.232
1.230
1.231s

New
0.246
0.247
0.255
0.250s
-0.982s
-79.73%

Old
0.055
0.055
0.056
0.055s

New
0.016
0.016
0.016
0.016s
-0.039s
-71.47%

Old
1.132
1.130
1.131
1.131s

New
0.226
0.226
0.235
0.229s
-0.901s
-79.7%

Old
0.358
0.355
0.356
0.356s

New
0.074
0.074
0.076
0.075s
-0.281s
-78.92%

Old
0.735
0.730
0.730
0.730s

New
0.148
0.148
0.151
0.149s
-0.581s
-79.55%

Old
1.156
1.155
1.157
1.155s

New
0.231
0.232
0.243
0.235s
-0.920s
-79.62%

Old
3.289
3.281
3.280
3.281s

New
0.651
0.650
0.659
0.656s
-2.625s
-80%

Old
0.607
0.605
0.606
0.605s

New
0.124
0.124
0.127
0.125s
-0.480s
-79.36%

Old
1.282
1.279
1.282
1.280s

New
0.257
0.260
0.259
0.259s
-1.021s
-79.77%

Old
0.555
0.555
0.555
0.555s

New
0.114
0.115
0.115
0.115s
-0.440s
-79.28%

Old
0.630
0.632
0.631
0.630s

New
0.128
0.130
0.131
0.130s
-0.500s
-79.4%

Old
2.738
2.731
2.731
2.732s

New
0.543
0.550
0.546
0.547s
-2.185s
-79.98%

Old
0.955
0.955
0.957
0.956s

New
0.193
0.192
0.193
0.194s
-0.762s
-79.74%

Old
0.355
0.355
0.354
0.355s

New
0.074
0.074
0.075
0.075s
-0.280s
-78.91%

Old
1.731
1.731
1.731
1.730s

New
0.350
0.344
0.346
0.348s
-1.383s
-79.91%

Old
1.562
1.554
1.555
1.555s

New
0.316
0.312
0.327
0.316s
-1.239s
-79.65%

Old
2.357
2.357
2.358
2.356s

New
0.476
0.468
0.480
0.474s
-1.882s
-79.88%

Old
1.130
1.130
1.130
1.131s

New
0.232
0.226
0.243
0.232s
-0.899s
-79.47%

Old
1.136
1.129
1.130
1.134s

New
0.228
0.227
0.231
0.230s
-0.905s
-79.76%

Old
0.282
0.280
0.281
0.282s

New
0.060
0.060
0.060
0.060s
-0.222s
-78.72%

Old
4.835
4.832
4.831
4.831s

New
0.957
0.955
0.974
0.968s
-3.863s
-79.96%

Old
0.030
0.029
0.030
0.030s

New
0.010
0.011
0.010
0.010s
-0.020s
-66.21%

Old
1.682
1.680
1.683
1.684s

New
0.336
0.334
0.336
0.338s
-1.346s
-79.95%

Old
0.380
0.380
0.379
0.382s

New
0.078
0.079
0.079
0.080s
-0.302s
-79.1%

Old
3.413
3.405
3.406
3.412s

New
0.676
0.676
0.687
0.683s
-2.729s
-79.99%

Old
2.257
2.258
2.256
2.258s

New
0.449
0.455
0.460
0.455s
-1.802s
-79.83%

Old
0.905
0.906
0.906
0.907s

New
0.182
0.185
0.185
0.184s
-0.723s
-79.67%

Old
0.280
0.280
0.281
0.281s

New
0.060
0.060
0.061
0.060s
-0.221s
-78.65%

Old
0.582
0.581
0.580
0.580s

New
0.118
0.120
0.122
0.120s
-0.460s
-79.28%

Old
1.060
1.056
1.056
1.056s

New
0.212
0.215
0.215
0.215s
-0.841s
-79.62%

Old
0.206
0.205
0.206
0.205s

New
0.044
0.045
0.045
0.046s
-0.159s
-77.77%

Old
0.856
0.855
0.856
0.855s

New
0.173
0.172
0.180
0.177s
-0.678s
-79.32%

Old
1.430
1.431
1.430
1.431s

New
0.291
0.286
0.291
0.292s
-1.139s
-79.6%

Old
1.785
1.780
1.780
1.780s

New
0.360
0.355
0.363
0.362s
-1.418s
-79.66%

Old
1.107
1.105
1.106
1.105s

New
0.225
0.222
0.223
0.225s
-0.880s
-79.65%

Old
1.481
1.480
1.481
1.479s

New
0.298
0.296
0.307
0.300s
-1.179s
-79.7%

Old
1.681
1.684
1.680
1.682s

New
0.335
0.335
0.336
0.338s
-1.344s
-79.92%

Old
1.964
1.957
1.957
1.957s

New
0.390
0.391
0.396
0.395s
-1.561s
-79.79%

Old
0.656
0.655
0.655
0.655s

New
0.133
0.133
0.141
0.136s
-0.519s
-79.26%

Old
0.131
0.130
0.131
0.130s

New
0.030
0.030
0.031
0.030s
-0.100s
-76.73%

Old
7.464
7.467
7.457
7.466s

New
1.476
1.476
1.496
1.490s
-5.976s
-80.05%

Old
0.258
0.255
0.255
0.256s

New
0.055
0.055
0.054
0.055s
-0.201s
-78.46%

Old
0.182
0.181
0.181
0.181s

New
0.040
0.039
0.040
0.040s
-0.142s
-78.13%

Old
1.882
1.881
1.881
1.885s

New
0.375
0.378
0.385
0.380s
-1.505s
-79.85%

Old
3.112
3.107
3.104
3.112s

New
0.617
0.626
0.630
0.623s
-2.488s
-79.97%

Old
1.309
1.306
1.306
1.308s

New
0.266
0.264
0.270
0.266s
-1.042s
-79.65%

Old
1.580
1.582
1.582
1.582s

New
0.321
0.316
0.318
0.319s
-1.263s
-79.83%

Old
0.481
0.481
0.481
0.481s

New
0.100
0.099
0.100
0.100s
-0.381s
-79.23%

Old
1.611
1.605
1.606
1.607s

New
0.326
0.321
0.326
0.324s
-1.283s
-79.83%

Old
0.757
0.756
0.755
0.757s

New
0.158
0.154
0.158
0.156s
-0.601s
-79.38%

Old
0.531
0.531
0.531
0.532s

New
0.111
0.108
0.113
0.110s
-0.421s
-79.25%

Old
0.706
0.705
0.706
0.706s

New
0.146
0.144
0.145
0.145s
-0.561s
-79.47%

Old
3.137
3.131
3.131
3.136s

New
0.623
0.624
0.633
0.630s
-2.506s
-79.92%

Old
2.082
2.081
2.081
2.082s

New
0.415
0.415
0.423
0.420s
-1.663s
-79.85%

Old
0.781
0.781
0.781
0.782s

New
0.158
0.158
0.161
0.160s
-0.622s
-79.51%

Old
0.206
0.206
0.205
0.206s

New
0.045
0.045
0.046
0.045s
-0.160s
-77.93%

Old
5.762
5.758
5.759
5.761s

New
1.139
1.139
1.197
1.159s
-4.601s
-79.87%

That is using 8X diffculty. Using 1X drops to about 0.05 seconds per PoW. The "Old" is not the same as #1 , because there I use the GPU+CPU, and now it's just GPU. Switching from CPU+GPU to GPU changes the initial nonce, which changes the time per work.

It's almost 80% faster now. 🥇


Help menu doesn't print when running sct_deepseg command without flags

Description
When running sct_deepseg command without flags, the help menu doesn't print. Instead, the code is run and an error occurs since there are no flags.
Steps to Reproduce
Run sct_deepseg in terminal.
Expected behavior:

Actual behavior:

duplicate of #3154

duplicate of #3154


Additional information
This is an issue about the German language, so I wrote the issue here in German.

@Ein-Tim
Good point!
For the community I think that "du" is fine.
The app itself uses the "Sie" form, so it makes sense to have the FAQs also in the "Sie" form.
The inconsistency is in the new download link, targeted at end users. To be consistent with the reset of the app it should use the "Sie" form.
https://www.coronawarn.app/de/ avoids the issue by not using "du" or "Sie". The text next to the download buttons says:
"App mit!
Jetzt zum Download verfügbar im Apple App Store und auf Google Play."

@MikeMcC399

For the community I think that "du" is fine.

I definitely agree.

The inconsistency is in the new download link, targeted at end users. To be consistent with the reset of the app it should use the "Sie" form.

Right, that also was my starting point for opening this issue. I'll provide a separate PR suggesting to change the text on the download page from the "Du" to the "Sie" form.

coronawarn.app/de avoids the issue by not using "du" or "Sie"

Further down it also uses the "du" form ("Bleib auf dem Laufenden!"). But I think that's okay.
I think this issue can be closed once the form on the download page is corrected.

Fixed in #1095


extracting objaverse renderings failed

Hello,
I followed this issue https://github.com/cvlab-columbia/zero123/issues/19 ↗ to download the Objaverse Renderings dataset using the following command:
wget https://tri-ml-public.s3.amazonaws.com/datasets/views_release.tar.gz
The download process completed successfully, but after several attempts to extract the files, I only managed to extract around 20,000 rendered images of objects. I suspect there might be an issue with the file itself.
Could you please help me confirm if there is any issue with this file?
Thank you.

Hi @kamzero , I've downloaded the views and extracted them multiple times with different environment and systems without problem so I don't think the file itself has any issue.

Thank you for your reply.  I will revisit my download process to ensure that everything is done correctly.


Get submission view count

Describe the solution you'd like
Add support to get a submissions view count.

Describe alternatives you've considered
No response
Additional context
No response

Unfortunately it is not possible to access this information with PRAW.


How to use colmena for new install?

I'm currently in the nixos iso install shell, so I can't just execute colmena apply-local --node <node-name> --sudo because it would install the system on the iso-system.
With plain flakes, I'd execute sudo nixos-install --flake .#<name> --root /mnt. How can I achieve the same with a colmena setup?

Ok, I just found https://github.com/zhaofengli/colmena/pull/209/files. I hope that this will work


Can pycdft method be applied into the systems where the acceptor and donor fragments are different?

Dear Experts,
My research focuses on the simulation of charge transfer processes in solid-liquid interfaces. Recently, I read your excellent paper “PyCDFT: A Python package for constrained density functional theory”, and plan to apply cDFT method into the calculation of outer-sphere reaction rate of organic carbonates on the electrodes.
I have successfully compiled the qbox and pycdft codes, run the example cases, and obtained the same results as demo output files. However, I find that the molecule of acceptor part is the same as donor’s in all cases (He-He+ dimer and thiophene-thiophene+ dimer). Can pycdft method be applied into the systems where the acceptor and donor fragments are different? Additionally, how to set up the parameters in pycdft to achieve the structural optimization with constraint potential? I don’t find the detailed parameter introduction in the pycdft package. Could you please share me a formal manual file of pycdft (my Email address is fangyuan@fudan.edu.cn)? Many thanks!
Best Regards,
Yuan Fang

Hello Yuan, thanks for your interest in pycdft. In answer to your questions

Can pycdft method be applied into the systems where the acceptor and donor fragments are different?
Yes, it is possible to define acceptor and donor fragments that are different from each other. The example cases were chosen so that we could compare to known literature values from previous codes. In your case of interest (organic carbonates on electrodes), things are much more complex and may require some thought about the physics of the system itself; at the very least, you may need to rely on your physical intuition to choose the acceptor and donor fragments and try several configurations. I recommend to also check the constrained charge density as part of this.

Additionally, how to set up the parameters in pycdft to achieve the structural optimization with constraint potential?
pycdft has forces due to the constraint potential implemented and these are fed into qbox, so it would be possible to do structural optimization with a constraint potential. It is largely a matter of passing the desired commands to qbox. The following parameters would be of interest to you:

QboxDriver: you'll need to modify the init_cmd with commands passed onto qbox, e.g., with the choice of atomic dynamics (atoms_dyn) and instead of scf_cmd, set opt_cmd to something where the first number passed to the run command is non-zero (see the Qbox documentation for more info). There is an example of structural optimization in the qbox documentation as well.
CDFTSolver: change job="scf" to job="opt"

See also extforce and run command in the Qbox documentation (http://qboxcode.org/doc/html/).
The restart functionality (as shown in the 01_he_coupling tutorial) may also be of interest to you.
Structural optimization with constraint forces has been not been benchmarked as much with larger systems, so please let us know if there are strange relaxations happening.

Could you please share me a formal manual file of pycdft?
The *rst files are the build files for generating a manual for pycdft. You can generate *html files that can be viewed in your browser by installing sphinx; the instructions are in <path_to_pycdft>/doc/docify.sh. After installing sphinx, run the docify.sh script.

Dear Dr. Wang,
Thank you for your great suggestions about construction of molecular fragments and structural optimization with constraint potential in the pycdft package. And according to the detailed PyCDFT manual, I tried constructing a simplified mode (Au-thiophene, a thiophene molecule with a gold atom in Fig.1) and calculating the energies of diabatic electronic states of the Au-thiophene model (Au-thiophene and Au-thiophene(-)). The Franck-Condon and relaxation processes of Au-thiophene(-) were considered.

Fig.1 the structure of Au-thiophene model.
For Franck-Condon calculation,
the part of QboxDriver was written as “
sample=sample,
init_cmd=”load gs.xml\n”
“set xc PBE\n”
“set wf_dyn JD\n”
“set scf_tol 1.0E-8\n”,
scf_cmd=”run 0 200 3”
“,
CDFTSolver part was written as “
job=”scf”, optimizer=”brenth”, sample=sample, dft_driver=qboxdriver, maxscf=800
“,
and ChargeConstraint part was written as “
sample=solver1.sample,
fragment=Fragment(solver1.sample,solver1.sample.atoms[1:10]), #ps: locating at thiophene frament
N0=27, #ps: one electron more than the sum of valence electrons of thiophene
N_top=1E-5
“.
For relaxation calculation with constraint potential,
the QboxDriver part was modified as “
...
init_cmd=”...”
“set atoms_dyn CG\n”
“set force_tol 1.0E-3\n”,
opt_cmd=”run 100 200 3”
“,
and CDFTSolver part was modified as “
job=”opt”, ..., maxstep=100, F_tol=1E-3
“.
However, the program prompted an assertion error “<class ‘AssertionError’>, AssertionError(), <traceback object at 0x2b3ebf0a41c0>” in the former calculation and the program blocked at QboxDriver part in the latter calculation. I don’t know what wrong is in the PyCDFT input files of both calculations. Could you please teach me how to modify the input files in order to achieve the simulation of electronic states of Au-thiophene(-) in Franck-Condon and relaxation processes? Many thanks!
To conveniently confirm, I have sent the input and output files of Au-thiophene, Au-thiophene(-) Franck-Condon and relaxation calculations in the attachment.
Au-thiophene.tar.gz
Due to the limit of storage, I deleted the ONCY files of elements and the output file, gs.xml.
Best Regards,
Yuan Fang

Dear Dr. Wang,
I have solved the problems above!
For the Frank-Condon calculations, I set up the wrong the n1, n2, and n3 values in pycdft.common.sample class. Actually, the value should be obtained from the np2v variable in the output file of qbox pre-calculation. Once I set up the exact value (252 in this case), the scf calculation with constraint potential will start to run and converge.
For the geometry optimization calculations, except the wrong n1-3 values, I also set up the wrong variables in the pycdft.cdft and pycdft.dft_driver.qbox_driver class. After replacing solver1.solve_opt() to solver1.solve() (If I use the solver1.solve_opt(), the calculation will crash directly and report an error "core dumped") and adding the scf_cmd variable, the calculation can be run. However, the calculation stops and can't optimize the structure with constraint potential when the first scf calculation converges. Could you please teach me how to modify the python input file to start the step of geometry optimization in pycdft? Many thanks!
To conveniently confirm, I also sent the python input file of geometry optimization calculation in the attachment.
Best Regards,
Yuan Fang
Au-thiophene.zip

Dear Dr. Wang,
Did you receive my messages above? Actually, I still look forward to extending the pycdft method  from qbox to the more common periodic DFT software, such as VASP or QuantumEspresso. Is it possible? And could you please teach me how to exactly use pycdft code in VASP or QuantumEspresso software? Many thanks!
Best regards,
Yuan Fang

Hi Yuan,
For future reference, please open a new issue for a new question and close the old issue. This will help us keep track of issues.
Additionally, please follow standard practices in posting to forums for electronic structure codes (i.e., signing with your name and affiliation).
In answer to your last question, it is in principle possible to interface pycdft with other electronic structure codes. The other code would at minimum need to be able to take in external forces and external potentials, and a way to communicate with pycdft (this is the client-server mode for Qbox). You would also need to write a child class in dft_drivers for the code you specifically want and methods for parsing the charge density and wavefunction.
I will try to get back to your other question but I will be delayed as the next couple of months will very busy for me.


Add tsconfig, more linting to common package

Just gradually turning on more checks when it's easy and fixing up trivial issues.
There are lots of interesting typing problems related to "contracts that have some properties" which I don't know how to solve yet, so I am not attempting to turn on no-explicit-any until I am a better Typescript programmer and I understand the best solution to these problems.

@akrolsmir Just letting you know that this is the sort of thing I am up to in my spare time, since you seemed interested in the idea of "turning on more linting stuff".

There are lots of interesting typing problems related to "contracts that have some properties" which I don't know how to solve yet, so I am not attempting to turn on no-explicit-any until I am a better Typescript programmer and I understand the best solution to these problems.

I've been looking into a few of those, work so far is on https://github.com/ForrestWeiswolf/manifold/tree/remove-any, but I figured I should hold off on making a PR until this is merged.


Make sure it works in a typical Steal project

Probably just adding some tests here would suffice.

@matthewp does https://github.com/stealjs/system-npm/pull/123 close this one? Or should I add similar tests here? Maybe make the current demo files actual tests.

I'm going to close this one, please reopen if there's anything missing


client-go should support install.istio.io/v1alpha1

client-go should allow handling of IstioOperator.

@nrjpoddar I created this Issue here because client-go doesn't have an Issue Tracker.  I was not sure how to label this Issue.  area/networking?  A new label area/networking/client-go?

Not sure, you can add area/networking but client-go is a cross WG concern.
Solution is simple though you need to add these annotations (comments) on proto files for client-go to pick them up automatically:
https://github.com/istio/api/blob/master/networking/v1beta1/destination_rule.proto#L189

It looks like the annotations are there in operator proto, there's probably some filter in client-go side.

Created https://github.com/istio/client-go/pull/209/

There's an underlying issue https://github.com/istio/istio/issues/24829 which will prevent us from adding Operator CRDs in client-go

Labeling as area/networking; if this is incorrect let me know.

Not stale.  Adding area/user experience because UX merged with the config and must own config issues.

I don't know. It's good question for @jasonwzm.

@jahvon Yes, it is still an issue. client-go relies on DeepCopy method working. I am following up in the blocking issue.


fix code highlighting in concepts-models.md

fix code highlighting - currently highlighted lines of code are out of sync with linked files
pics below show how it looks before the changes

Relevant file: https://github.com/Azure-Samples/azure-digital-twins-getting-started/blob/main/models/advanced-home-example/IRoom.json

@baanders
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@baanders all looks good, I think it can be merged

Hi @BartekSTRX, thank you for reviewing my comments! In order to merge this pull request, you'll need to implement the requsted changes by adding a new commit.
Alternatively, I can make the updates myself in a separate PR and close this one if that is easier for you. :)

fixed according to suggestions

#sign-off


Added misses tier key/value for broom_handle

FYI: Broom handle should be tier 1 of Neutral items
Referece dota2 client

This also needs to be applied in the updateconstants script

This is already done in the script, it just wasn't run at the right time


Add Web5 Additional Steps To Docs

https://github.com/TBD54566975/web5-js#additional-steps

On hold this is failing with MDX escaping...

Additional steps for React Native or Node 18

This SDK relies indirectly on the `@noble/ed25519` and `@noble/secp256k1` packages. Therefore, in certain environments, you'll need to perform additional steps to make it work.

Node.js 18 or earlier needs globalThis.crypto polyfill:
```js
import { webcrypto } from "node:crypto";
\\ Or @ts-ignore
if (!globalThis.crypto) globalThis.crypto = webcrypto;
```
React Native:
Needs crypto.getRandomValues polyfill and sha512
```js
import "react-native-get-random-values";
import { hmac } from "@noble/hashes/hmac";
import { sha256 } from "@noble/hashes/sha256";
import { sha512 } from "@noble/hashes/sha512";
ed.etc.sha512Sync = (...m) => sha512(ed.etc.concatBytes(...m));
ed.etc.sha512Async = (...m) => Promise.resolve(ed.etc.sha512Sync(...m));
secp.etc.hmacSha256Sync = (k, ...m) =>
hmac(sha256, k, secp.etc.concatBytes(...m));
secp.etc.hmacSha256Async = (k, ...m) =>
Promise.resolve(secp.etc.hmacSha256Sync(k, ...m));
</details>

Add this under the import statement in Step 3 of Installation

Additional import for Node 18
/*
Needs globalThis.crypto polyfill. This is *not* the crypto you're thinking of.
It's the original crypto...CRYPTOGRAPHY.
*/
import { webcrypto } from "node:crypto";

Additional imports for React Native
/*
React Native needs crypto.getRandomValues polyfill and sha512.
This is *not* the crypto you're thinking of.
It's the original crypto...CRYPTOGRAPHY.
*/
import "react-native-get-random-values";
import { hmac } from "@noble/hashes/hmac";
import { sha256 } from "@noble/hashes/sha256";
import { sha512 } from "@noble/hashes/sha512";
ed.etc.sha512Sync = (...m) => sha512(ed.etc.concatBytes(...m));
ed.etc.sha512Async = (...m) => Promise.resolve(ed.etc.sha512Sync(...m));

secp.etc.hmacSha256Sync = (k, ...m) =>
  hmac(sha256, k, secp.etc.concatBytes(...m));
secp.etc.hmacSha256Async = (k, ...m) =>
  Promise.resolve(secp.etc.hmacSha256Sync(k, ...m));


Rename utility classes to avoid ambiguity between breakpoint and non-breakpoint specific classes.

In order to move forward with the expanded set of sizing and media breakpoint specific utility classes, it is necessary to rename the following, to remove ambiguity between media breakpoint specific classes and general sizing utility classes using the same naming convention. This will also make it easier to increase the number of breakpoints in v6.0.0, which may support up to 20 content based breakpoints:

All -xs, -sm, -md, -lg, and -xl suffixes, where the meaning is a media breakpoint size, will be replaced with:
-bp-1-under
-bp-1-over
-bp-2-under
-bp-2-over
-bp-3-under
-bp-3-over
-bp-4-under
-bp-4-over
-bp-5-under
-bp-5-over

Class names using -xs, -sm, -md, -lg, and -xl suffixes, where the meaning is NOT related to media breakpoints, will remain unchanged.

Completed.


Create Dockerfile_base_amazonlinux

This dockerfile will be used to create daemon image with the amazonlinux base image
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Rollback Dockerhub image base to AmazonLinux PR #130

Hi, can I have why the image is rollbacked?
Is there any problem on the scratch based image?

Hi @shogo82148
The Dockerhub image was switched back to use the amazonlinux base image. When using the scratch base image, we unintentionally broke some customers who were relying on some bash tools which are shipped with amazonlinux but not with scratch. Please see these related issue for more details: https://github.com/aws/aws-xray-daemon/issues/119, https://github.com/aws/aws-xray-daemon/issues/128
Currently, the Dockerhub image is based on amazonlinux and the ECR image is based on scratch to maintain the existing customer experience. Are you facing any issue with these images?

@srprash Thank you for your reply. I understood.
I'm migrating my service from EC2 to ECS, and looking for a pre-built image of X-Ray daemon.
I have currently no problem, but I was just wondering.
I'm going to use the ECR image, thanks again!


Invalid path reference articles.title; Associations can only be pointed to directly or via their id property

I am using spring-graphql along with spring-data-mongo and querydsl and faced with the below issue
org.springframework.data.mapping.MappingException: Invalid path reference articles.title; Associations can only be pointed to directly or via their id property
	at org.springframework.data.mongodb.core.convert.QueryMapper$MetadataBackedField.getPath(QueryMapper.java:1275) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.convert.QueryMapper$MetadataBackedField.<init>(QueryMapper.java:1134) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.convert.QueryMapper$MetadataBackedField.<init>(QueryMapper.java:1111) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.convert.QueryMapper.createPropertyField(QueryMapper.java:365) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.convert.QueryMapper.lambda$mapFieldsToPropertyNames$0(QueryMapper.java:234) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at java.base/java.util.Map.forEach(Map.java:713) ~[na:na]
	at org.springframework.data.mongodb.core.convert.QueryMapper.mapFieldsToPropertyNames(QueryMapper.java:232) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.convert.QueryMapper.getMappedFields(QueryMapper.java:220) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.QueryOperations$QueryContext.getMappedFields(QueryOperations.java:363) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.MongoTemplate.doFind(MongoTemplate.java:2573) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.ExecutableFindOperationSupport$ExecutableFindSupport.doFind(ExecutableFindOperationSupport.java:177) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.core.ExecutableFindOperationSupport$ExecutableFindSupport.all(ExecutableFindOperationSupport.java:135) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.data.mongodb.repository.support.SimpleMongoRepository$FluentQueryByExample.all(SimpleMongoRepository.java:391) ~[spring-data-mongodb-4.1.1.jar:4.1.1]
	at org.springframework.graphql.data.query.QueryByExampleDataFetcher$ManyEntityFetcher.getResult(QueryByExampleDataFetcher.java:751) ~[spring-graphql-1.2.1.jar:1.2.1]

Here is my set up :
@AllArgsConstructor
@NoArgsConstructor
@Getter
@Setter
@Builder
@EqualsAndHashCode
@QueryEntity
@Document(collection = "articles")
public class Article {

    @Id
    private String id;
    private String title;
    private Integer minutesRead;
    @DBRef(lazy = true)
    private User author;
}

@AllArgsConstructor
@NoArgsConstructor
@Getter
@Setter
@EqualsAndHashCode
@Builder
@QueryEntity
@Document(collection = "users")
public class User {

    @Id
    private String id;

    private String name;
    private Integer age;
    private Date createdAt;
    private String nationality;
    private List<String> friendsIds;
    @DBRef(lazy = true)
    private List<Article> articles;
}

@GraphQlRepository
public interface UserRepository extends MongoRepository<User, String>, QuerydslPredicateExecutor<User> {
}

here is my graphql schema:
type Query {
    users: [User]
     ...
}

👍 So while the following graphql seems to work fine:
query {
  users {
    id
    age
    name
    articles {
      id
    }
  }
}

❌  adding extra properties on the nested articles object does not:
query {
  users {
    id
    age
    name
    articles {
      id
      title
    }
  }
}

throwing org.springframework.data.mapping.MappingException: Invalid path reference articles.title; Associations can only be pointed to directly or via their id property.
Can you please shed some light as to what is the problem here ?
Also, I have noticed that when using MongoRepository along with QuerydslPredicateExecutor  then QueryByExampleDataFetcher takes precedence over QuerydslDataFetcher. Is this expected behaviour?

Inline code snippets don't really help here. Can you share the full sample application that we can use to replicate the problem?

Also, I have noticed that when using MongoRepository along with QuerydslPredicateExecutor then QueryByExampleDataFetcher takes precedence over QuerydslDataFetcher. Is this expected behaviour?

We generally don't expect both, and if both are implemented the order is not defined.

Inline code snippets don't really help here. Can you share the full sample application that we can use to replicate the problem?

Thanks for the quick response. Sure. here is a sample project to reproduce it. https://github.com/kliarist/boot-mongo-graphql

Also, I have noticed that when using MongoRepository along with QuerydslPredicateExecutor then QueryByExampleDataFetcher takes precedence over QuerydslDataFetcher. Is this expected behaviour?

We generally don't expect both, and if both are implemented the order is not defined.

Thanks. I see.. I need the QuerydslPredicateExecutor present tho, as I will need the predicate builder functionality in the future.

This is a Spring Data problem when attempting to project properties from an association. I filed spring-projects/spring-data-mongodb#4453 to address the issue.

@mp911de I see.. thanks


Task: Update styles for theme

screenshots:

^The config sets a max-width: 1000px, and the playground preview has a width of 400px plus padding and margin, which leaves less than 300px of free space for the code arena.
If you scrunch the browser you’ll see them stacked, which can we can set as the default if you prefer:

/cc @boygirl

Um, looking at those screenshots, I am wrong. Something weird is happening and .Interactive is not getting set to the full-width. Looking into it now.

Ok, fixed!

so good!


Fix IP record sync

This can also be achieved more laboriously by doing match function on external id but then updating the IP address during the update method. However it will need logic to determine conflicts before save, and order the updates such that there will not be dupe IPs in the IP pool during save. If record X has switched to IP B and record Y has switched to IP A, then during the update method there is a deadlock which will need managing as well. This proposed solution attempts to simplify this by syncing on IP and updating external id, which seems odd but may work better.

Verified works with IPv6 and IPv4


:seedling: Changing md5 to sha256 in tests and default templates

What this PR does / why we need it:
This PR is added to fix following gosec issue:
G501 (CWE-327): Blocklisted import crypto/md5: weak cryptographic primitive (Confidence: HIGH, Severity: MEDIUM)

/test-centos-e2e-integration-main
/test-ubuntu-integration-main

/hold

/test-centos-e2e-integration-main
/test-ubuntu-integration-main

/retitle :seedling: Changing md5 to sha256 in tests and default templates

/unhold

/test-centos-e2e-integration-main
/test-ubuntu-integration-main

/test build

/test-centos-e2e-integration-main
/test-ubuntu-integration-main

/hold
e2e integration is failing because we are still getting m5sum from dev-env. we need to change defaults in dev-env too.

/test-centos-e2e-integration-main

/test-centos-e2e-integration-main

/lgtm

/keep-test-centos-e2e-integration-main

/keep-test-centos-e2e-integration-main

/keep-test-centos-e2e-integration-main

/unhold

/test-centos-e2e-integration-main
/test-ubuntu-integration-main

/test-centos-e2e-integration-main

/lgtm

/cherry-pick release-1.4
/cherry-pick release-1.3
/cherry-pick release-1.2
/cherry-pick release-1.1

/cherry-pick release-1.4 /cherry-pick release-1.3 /cherry-pick release-1.2 /cherry-pick release-1.1

I wont recommend cherry picking to release-1.1


Update documentation for state machine diagrams

There have been some minor grammar updates to support state machine diagrams.

Done!


Add leaderboard component to ClimaLand's long runs

Similar to leaderboard component in ClimaCoupler, a leaderboard component can be added for ClimaLand.
The leaderboard component would plot the bias of the simulation and observational data for monthly data. See the plot below for what it could potentially look like. This will be done for et, gpp, and lwu. More variables can be added later.

The spin up time can be estimated using the plot below.

The plot is generated from the following script:
import ClimaAnalysis
import GeoMakie
import CairoMakie
import Dates

output_dir = "land_longrun_gpu/global_diagnostics/output_0004/"
sim_dir =
    ClimaAnalysis.SimDir("land_longrun_gpu/global_diagnostics/output_0004/")
short_names = ["gpp", "ct", "swc", "swa", "si", "lwu", "et", "sr"]
plot_dict = Dict()
for short_name in short_names
    @info "Current var: $short_name"
    var = ClimaAnalysis.get(sim_dir, short_name)

    if ClimaAnalysis.has_altitude(var)
        var = ClimaAnalysis.slice(var, z = var.dims["z"][end])
    end
    if short_name == "swa"
        var = ClimaAnalysis.set_units(var, "unitless")
    end

    var = ClimaAnalysis.replace_nan(var, 0.0)
    var = ClimaAnalysis.apply_oceanmask(var)

    var.attributes["start_date"] = "2021-01-01T00:00:00" # add start date since they are not in the OutputVars

    # Getting dates
    var = ClimaAnalysis.shift_to_start_of_previous_month(var)
    output_dates =
        Dates.DateTime(var.attributes["start_date"]) .+
        Dates.Second.(ClimaAnalysis.times(var))
    @info "Working with dates:"
    @info output_dates

    yr_a, yr_b = ClimaAnalysis.times(var)[1:12], ClimaAnalysis.times(var)[13:24]

    rmse_arr = zeros(length(yr_a))
    for (idx, (t_a, t_b)) in enumerate(zip(yr_a, yr_b))
        var_a = ClimaAnalysis.slice(var, time = t_a)
        var_b = ClimaAnalysis.slice(var, time = t_b)
        g_rmse = ClimaAnalysis.global_rmse(
            var_a,
            var_b,
            mask = ClimaAnalysis.apply_oceanmask,
        )
        rmse_arr[idx] = g_rmse
    end
    plot_dict[short_name] = rmse_arr
end

fig = CairoMakie.Figure(size = (1800, 600))
short_names_plot = [["gpp", "ct", "swc", "swa"], ["si", "lwu", "et", "sr"]]

for (row, short_name_vec) in enumerate(short_names_plot)
    for (col, short_name) in enumerate(short_name_vec)
        ax = CairoMakie.Axis(
            fig[row, col],
            title = "RMSE for $short_name",
            xlabel = "Month",
            ylabel = "Global RMSE",
            xticks = (1:12, Dates.monthname.(1:12) .|> x -> x[1:3]),
        )
        CairoMakie.lines!(ax, 1:12 |> collect, plot_dict[short_name])
        CairoMakie.scatter!(ax, 1:12 |> collect, plot_dict[short_name])
    end
end
CairoMakie.Label(fig[0, :], "RMSEs of 2021a against 2021b", tellwidth = false, fontsize = 30)
CairoMakie.save("RMSE_plot.png", fig)


Expid 158488, tile 5181 on 20221216 QA shows one petal missing, but Nightwatch positioning diagnostics look fine

Suspect this is caused by https://github.com/desihub/desispec/issues/1940

Let's leave as unsure and tag and assign to Marcelo.  When this is resolved
we'll reQA.
On Sun, Dec 18, 2022, 21:24 Dustin Lang @.***> wrote:

Suspect this is caused by desihub/desispec#1940
https://github.com/desihub/desispec/issues/1940
—
Reply to this email directly, view it on GitHub
https://github.com/desihub/desisurveyops/issues/86#issuecomment-1356868658,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2E4JFX67PYFOPUKXWYBLWN5XGJANCNFSM6AAAAAATCWGPH4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

for info, from @marcelo-alvarez on slack:
"I could not find the cause of a pipeline failure during preproc traceshift for exposure 158488 on camera b4, so I marked sp4 as bad and consequently tile 5181 was processed without it (see desipec/1940)."
https://desisurvey.slack.com/archives/C01HNN87Y7J/p1671322332283719

This should now be done @schlafly please have a look.

Great, I've assigned this back to myself and will close when we have changed the QA to good.  Thanks!

I've marked this as good.  It should be picked up by the next archiving and then included in MTL updates.  Closing.


Implement "Forgot password?"

The button is there on "Login"

Will need an email service.

Lowering priority, removing from milestone.

The button is no longer there.
Possible alternatives:

Only use third-party/social logins (GitHub, Google, Twitter)
Login with email only (ala Medium)


Document Prometheus labeling guidelines and recommendations

Document Prometheus labeling guidelines and recommendations, e.g. label with namespace, pod name, etc for Kubernetes environments.

Also see:

https://prometheus.io/docs/practices/naming/
https://github.com/OpenObservability/OpenMetrics/blob/master/specification/OpenMetrics.md#metric-naming-and-namespaces

@odeke-em let's discuss this - could use your help to itemize these guidelines and add to the readme for the Prometheus receiver.

It will be the end user's responsibility to add the correct labels. But these guidelines still need to be written and documented in a readme or in multiple places. The Prometheus receiver readme is a good place, the Operator readme is another place this should be documented and any other docs used for configuring the exporter or Prometheus pipeline.

@jmacd can you take a crack at an initial set of guidelines for labeling that we can provide in OTEL documentation.

10/29: At this point, we assume it will be end user's responsibility to add the right labels they need. Take a look at https://github.com/OpenObservability/OpenMetrics/blob/main/specification/OpenMetrics.md#metric-naming-and-namespaces for good practices.


Chain doesn't work with nested groups

If a group has chain set as true, the commands of a sub-group can't be executed.
Minimal exmaple cli.py
import click

@click.group(chain=True)
def cli():
    pass

@cli.group()
def gp():
    pass

@gp.command()
def cmd():
    pass

cli()

Then
$ python cli.py gp cmd
Usage: cli.py [OPTIONS] COMMAND1 [ARGS]... [COMMAND2 [ARGS]...]...

Error: No such command "cmd".

Maybe this isn't resolvable, but the docs say "When using multi command chaining you can only have one command (the last) use nargs=-1 on an argument. Other than that there are no restrictions on how they work." at http://click.pocoo.org/5/commands/#multi-command-chaining
Using Python 3.4.3 and Click 5.1

This is actually something that cannot be supported given the guarantees subcommand invocations have.  Click now gives a proper error message when trying to do something like this instead of doing the unexpected thing.

It actually turns out that some of the guarantees that were given did not really make sense.  There were actually cases where the current behavior accidentally caused bugs as well.  So in Click 6 the following things are true:

chain commands can nest
if a chain command is below a chain comment it will eat up all arguments and returning to the outer chain is impossible
optional arguments on a group are no longer supported.

For the changes see 0a2919f34fcbc635d8530b4c5b60bf119b2bcedb and earlier commits.


What's the meaning of 'phc.layers[-1].add_shape(legume.Circle(eps=1.0, r=r))'?

In the example of 'Bands of a photonic crystal slab', there is a command 'phc.layers[-1].add_shape(legume.Circle(eps=1.0, r=r))',
Could you please explain the meaning of 'layers[-1]' in the command? Thank you very much!

The phc.layers variable is a list of layers of the photonic crystal object, not including the upper and lower claddings. Here a circle is added to the "-1th" entry of that list. In python a negative index counts backwards from the end of the list or sequence. So -1 indexes the final entry of the list.
See https://realpython.com/lessons/indexing-and-slicing/ for more details on slicing and indexing sequences in python.

Thanks @StephenJamesCashen


Max retry exceeded when using DeltaTable with Azure Blob Storage

Environment
Delta-rs version: 0.16.0
Environment:

OS: Windows 11
Python: 3.11.8
Pyarrow: 13.0.0
adlfs: 2024.4.1

Bug
What happened: I'm facing an issue when using the deltalake lib to save / loading data to Azure Blob Storage. Sometimes, I'm getting the following error:
DatasetError: Failed while saving data to data set CustomDeltaTableDataset(file_example).
Failed to parse parquet: Parquet error: AsyncChunkReader::get_bytes error:
Generic MicrosoftAzure error: Error after 10 retries in 2.196683949s, max_retries:10,
retry_timeout:180s, source:error sending request for url
(https://<address>/file.parquet):
 error trying to connect: dns error: failed to lookup address information: Name or service not known

What you expected to happen: I expected to load the data from the Delta table and convert it to a Pandas DataFrame without any errors.
How to reproduce it:
from deltalake import DeltaTable

datalake_info = {
    'account_name': <account>,
    'client_id': <cli_id>,
    'tenant_id': <tenant_id>,
    'client_secret': <secret>,
    'timeout': '100000s'
}

# Load data from the delta table
dt = DeltaTable("abfs://<azure_address>", storage_options=datalake_info)

More details: I was looking for a parameter like max_retries but couldn't find anything related. Does anyone know a solution or workaround for this issue? I didn't find an approach in the docs: https://docs.rs/object_store/latest/object_store/azure/enum.AzureConfigKey.html

Has anyone ever faced this problem?

I'm also getting this error lately:
Generic MicrosoftAzure error: Error after 10 retries in 3.296507315s, max_retries:10, retry_timeout:180s, source:HTTP status server error (503 Service Unavailable) for url (https://onelake.blob.fabric.microsoft.com/xxxxxxxxxxxxxx/Tables/dddddddddddd/_delta_log/_last_checkpoint).
The path I am using is<EMAIL_ADDRESS>

this should work, using 0.18.2 and above
from azure.identity import ClientSecretCredential, AuthenticationRequiredError
credential = ClientSecretCredential(
                client_id = "appId",
                client_secret="secret",
                tenant_id= "tenantId"
                )
access_token =       credential.get_token("https://storage.azure.com/.default").token
storage_options=     {"bearer_token": access_token, "use_fabric_endpoint": "true"}
from deltalake import DeltaTable
scada = DeltaTable('abfss://workspace@onelake.dfs.fabric.microsoft.com/Lakehousename.Lakehouse/Tables/xxxxx',storage_options=storage_options)


Set 404 header

On the error route. If at all possible

This is only possible server side.
Vercel might have integration.


Update aspnetapp.yaml

updated to use the new api version networking.k8s.io/v1 instead of deprecated api extensions/v1beta1 which is getting removed in 1.22

Checklist

[ ] The title of the PR is clear and informative
[ ] If applicable, the changes made in the PR have proper test coverage
[ ] Issues addressed by the PR are mentioned in the description followed by Fixes.

Description

Fixes

Closing as docs/examples have been updated.


Update "DC Source 3" to improve resizeability

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
"DC Source 3"does not resize nicely.
To Reproduce
Steps to reproduce the behavior:

From the shapes "Electrical / Sources" add the DC Source with "+" and "-".
Resize to smaller size
"+" and "-" nearly vanish when resized to 20x20.

Current Shape Code
<shape aspect="variable" h="60" name="DC Source 3" strokewidth="inherit" w="60">
    <connections>
        <constraint name="N" perimeter="0" x="0.5" y="0"/>
        <constraint name="S" perimeter="0" x="0.5" y="1"/>
    </connections>
    <background>
        <ellipse h="60" w="60" x="0" y="0"/>
    </background>
    <foreground>
        <fillstroke/>
        <path>
            <move x="27" y="55"/>
            <line x="33" y="55"/>
            <move x="30" y="2"/>
            <line x="30" y="8"/>
            <move x="27" y="5"/>
            <line x="33" y="5"/>
        </path>
        <stroke/>
    </foreground>
</shape>

Suggested updated Shape Code
<shape aspect="variable" h="60" name="DC Source 3" strokewidth="inherit" w="60">
    <connections>
        <constraint name="N" perimeter="0" x="0.5" y="0"/>
        <constraint name="S" perimeter="0" x="0.5" y="1"/>
    </connections>
    <background>
        <ellipse h="60" w="60" x="0" y="0"/>
    </background>
    <foreground>
        <fillstroke/>
        <path>
            <move x="22" y="55"/>
            <line x="38" y="55"/>
            <move x="30" y="2"/>
            <line x="30" y="18"/>
            <move x="22" y="10"/>
            <line x="38" y="10"/>
        </path>
        <stroke/>
    </foreground>
</shape>

draw.io version (In the Help->About menu of the draw.io editor):

draw.io desktop version 14.1.8

Desktop (please complete the following information):

OS: Win 10 Enterprise LTSC (1809)

Additional context

dc-source-3.drawio.zip

A new shape will be in next release, which should address this issue


ValueError: not enough values to unpack (expected 4, got 2)

ERROR:root:!!! Exception during processing !!!
ERROR:root:Traceback (most recent call last):
File "/home/ec2-user/SageMaker/ComfyUI/execution.py", line 154, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "/home/ec2-user/SageMaker/ComfyUI/execution.py", line 84, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "/home/ec2-user/SageMaker/ComfyUI/execution.py", line 77, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "/home/ec2-user/SageMaker/ComfyUI/custom_nodes/ComfyUI-3D-Pack/nodes.py", line 334, in run_3DGS
gs.training()
File "/home/ec2-user/SageMaker/ComfyUI/custom_nodes/ComfyUI-3D-Pack/gaussian_splatting/main_3DGS.py", line 163, in training
out = self.renderer.render(ref_cam, bg_color=bg_color)
File "/home/ec2-user/SageMaker/ComfyUI/custom_nodes/ComfyUI-3D-Pack/gaussian_splatting/main_3DGS_renderer.py", line 1044, in render
rendered_image, radii, rendered_depth, rendered_alpha = rasterizer(
ValueError: not enough values to unpack (expected 4, got 2)
Above is the error, this was casue by the calling:
rendered_image, radii, rendered_depth, rendered_alpha = rasterizer(
            means3D=gaussians_xyz,
            means2D=screenspace_points,
            shs=shs,
            colors_precomp=colors_precomp,
            opacities=gaussians_opacity,
            scales=gaussians_scales,
            rotations=gaussians_rotations,
            cov3D_precomp=cov3D_precomp,
        )

looks like only return 2 values but required is 4.
How to fix it? thanks in advance

Sorry, I don't have enough information to determine what's the problem, are you running the example workflow & data I provided? or did you change the inputs data? If so, may I have a screen shot of your workflow pls?
Thanks for testing! I'll make those edge case detection more restrict 👍

worklow name: 3DGS_Multi-View_Renders_to_ply
I didn't change any inputs

Sorry, I couldn't reproduce the error you got, could you pull the newest change set and use this [example workflow] (https://github.com/MrForExample/ComfyUI-3D-Pack/blob/main/_Example_Workflows/Using_Stack_Orbit_Camera_Poses_to_conditioning_StableZero123.json)
And see if error still exist?
Please make sure to copy all the example inputs into ComfyUI's input directory 👍

Good news: the orbit camera workflow has no error.
Bad news: 3DGS_Multi-View_Renders_to_ply still complains the same not enough value error!
my env:
*A10 linux machine, notebook
*PyTorch 2.0.1
*Python  3.10.13

Thanks for testing and letting me know, I'll do a double check on that 👍
Is there other functionality or algorithm you think will benefit most people that you know who are using ComfyUI?

Oh! 😁
I know why you got that error, I saw your another comment where you suggested to install the original Differential Gaussian Rasterization, so I assume that you also installed the original one, but this project, as state in readme, used a updated version of Differential Gaussian Rasterization which supports depth & alpha rendering (both forward and backward), and that is exactly what error trying to tell us what is missing!
So to fix this, delete the original Differential Gaussian Rasterization, and install & build a new one using:
git clone --recursive https://github.com/ashawkey/diff-gaussian-rasterization
pip install ./diff-gaussian-rasterization

Let me know if there is any other issue 👍

Very nice, i will test it.
BTW, your work is really cool!

This is my working installation steps:
`
!git clone --recursive https://github.com/NVlabs/nvdiffrast
!git clone --recursive https://github.com/ashawkey/diff-gaussian-rasterization
!git clone --recursive https://github.com/facebookresearch/pytorch3d
!pip install ./diff-gaussian-rasterization
!pip install ./gaussian-splatting/submodules/simple-knn
!pip install ./nvdiffrast
!pip install ./pytorch3d
`

Glad to hear this issue is solved!


Users need input box lengths to be relative to how much information they need to enter in them

Assigning this to Sanjay to change box lengths.
For a real life example please see below


Delete manifests/j/Jackett/Jackett/0.20.834.0 directory

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Where is the app package in the master branch??

There is no "app" package in the master branch. I can found it in the other branch (core-refactor), but in the master branch that package is missing.
By the other hand in the "core-refactor" branch, the class "DaggerApplicationComponent" is missing too. So I can't compile the project.
Great project and great tutorial.
Thanks

@Mun0n It's not missing, it has just a different name according to the layers of the architecture. Instead of app there is the presentation module which is the same.

@Mun0n About missing DaggerApplicationComponent: You should build the project then it appear.

+1 @lenguyenthanh

Thanks all!


Doc issue with second example

On the home page, after the example on top, there is a Single Date section with text:
For a single date, use the <calendar-date> component, and place inside it a <calendar-month> component.

And then the next section is Date Range (https://wicky.nillia.ms/cally/#date-range) and it has the same text + sample code.

Thanks. That'll teach me not to copy and paste sections 😅 if you fancy making a PR the docs are under the /docs folder. Otherwise I'll fix tomorrow

I was curious what the syntax was, so, I don't know what the fix is. ;) But I'll take a quick look see.

I think it should just read

For a date range, use the <calendar-range> component, and place inside it a <calendar-month> component.

Did you fix it already? I'm looking now and it looks right. It's possible I didn't read clearly as well.

Oh heh, yeah, it says 'for a single date'. Yes, I can do a quick pr.


Update the progress status with the ticker

Apparently, the progress reporter already uses a ticker initialized at 1second, however, the updateProgress function is called every time new stats are added. I just tried to update the progress with the ticker only (60fps), to see if it has an effect on cpu usage on mac OS X. I may miss some points but it doesn't looks like the root cause of issue #263.

Thanks for trying to fix this! Unfortunately, this isn't a good fix. It updates the status 60 times per second although there may be nothing to update. I'd suggest leaving the ticker at a second (so that the progress is updated at least once per second). Then add a new field lastUpdate time.Time to the Progress struct, check this in Report() and only call updateProgress() when since the last update more than 16ms have passed. e.g.:
func (p *Progress) Report(s Stat) {
	[...]
	needUpdate := false

	p.curM.Lock()
	p.cur.Add(s)
	cur := p.cur
	if time.Since(p.lastUpdate) > 16 * time.Millisecond {
		p.lastUpdate = time.Now()
		needUpdate = true
	}
	p.curM.Unlock()

	if needUpdate {
		p.updateProgress(cur, false)
	}
}

Ok, your code worked as it is :)

Cool, I didn't expect that. I'll merge this later today. If you have the time, you could squash both commits.
Maybe @rubenv can give this code a try.

Looks like a perfect fix. CPU usage is back where it should be: within restic.
LGTM


Add Slow Zones

This PR

Adds slow zone information at /slow-zones
Has two graphs, one for total "slow" time per week, and one for all slow zones.
There is also a dropdown bar on the top right that allows you to save the charts as png/jpg

Notes

You'll have to add all_slow.json to public/static/slow-zones locally to test (let me know if you have any troubles)
I'd love some input on design / copy
A decent amount of these change are just prettier changes
I could add a date filter if needed. Right now I'm starting at 1/1/2021

Gifs

Two other quick notes:

We might want some README updates as well WRT this new feature.
Perhaps after it is merged we should do a minor version bump? (i.e. v2.1) @mathcolo @idreyn lmk what y'all think WRT to that


Enable pluggable components on Windows

Description
Removes the restriction on use of pluggable components on Windows.  Updates the Unix Domain Socket syntax and detection logic to handle Windows' quirks (namely using unix: instead of unix:// and not relying on the ModeSocket flag).
In the case of Windows, as the ModeSocket flag is not set, the Dapr runtime instead relies on looking for "socket shaped" sockets which, by default, have the .sock extension. If the user desires to use another extension, they can set the DAPR_COMPONENTS_SOCKETS_EXTENSION environment variable to an arbitrary extension. (This environment variable is already supported in the Java and .NET pluggable component SDKs.)
With this change, the Dapr runtime will also respect temp directories specified via $TMPDIR, whereas it previously used /tmp.
Issue reference
Please reference the issue this PR will close: #6082 (closed earlier for lack of comments)
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@philliphoff please can you re-enable the socket integration test for windows in this PR? :) they are also quite limited so consider extending them a bit if you want
https://github.com/dapr/dapr/blob/53ceb4aa59a57a25de9d76b61b298584f5339354/tests/integration/suite/daprd/pluggable/basic.go#L48

please can you re-enable the socket integration test for windows in this PR? :) they are also quite limited so consider extending them a bit if you want

Enabled those tests, which seem to pass on Windows and other platforms (though looks like there's some flakyness in other tests).

@daixiang0 Anything more you'd like me to add?

Yesterday, version 0.3.0 of the dapr-sandbox/components-dotnet-sdk was released which includes the initialisation of Unix Domain Sockets on windows, can you enable the search in the component folder on windows as well so that people using Visual Studio can also develop on it?

Yesterday, version 0.3.0 of the dapr-sandbox/components-dotnet-sdk was released which includes the correct initialisation of Unix Domain Sockets on windows, can you enable the search in the component folder on windows as well so that people using Visual Studio like me can also develop on it?

@m3nax Can you be a bit more specific about what you're trying to do?  I'm not sure what you mean by "search in the component folder" with respect to pluggable components.  (Usually the UDS is placed in a temp folder separate from the Dapr components folder.)

@philliphoff It's quite simple, I'm using Visual Studio Dapr Extension for Visual Studio 2022 whitch starts debugging on my pc, whitch is Windows. The task manager confirm it.

Then, when I try to register the binding to use it in another application I get the current message:
level=debug msg="the current OS does not support pluggable components feature, skipping initialization" app_id=myapp

This makes it impossible to debug pluggable components with Visual Studio 2022. Is there any other way?

@m3nax Until this PR gets committed, likely not.  the VS Dapr extension does support limited debugging of applications within WSL, but I've not tried to add a pluggable component into the mix so I can't say for certain whether that would work.

Not stale

(Just a note that I'll be unlikely to have time in the near term to freshen up this PR, also given that there have been some significant changes for component hot-reload that intersect these changes.  I welcome any interested people to take that up if they have time/interest, else I'll probably just close this PR.)


Prepare Role Descriptions for Validators and Proofreaders

There are no (I think) current role descriptions or help provided for proof readers or validators.
Recruiting for validators (both for new coords to undertake, or to work alongside a coord) needs to take plae more urgently than recruiting for proof readers.
E0 U7 S 8 C8 P23

From note of 27 August meeting: use the ones in the handbook

The objectives for Proofreading and Validation are included in the Handbooks.
This suggestion proposes that we need a duty statement. If so how will it be applied?
Geoff


[testng-report] The chronological view HTML block is not correctly constructed

TestNG 6.8
The Chronological View part of the testng HTML report is missing an HTML tag which breaks the entire report page. There is a  tag which is never closed, causing the page to not be displayed correctly.
Since this piece of HTML is at the end of the page the impact is not too big, however it is noticeable if there are multiple suites run at once. The Chronological View of the subsequent suites will never be visible, since they are considered part of the first (hidden) chronological panel.
See below the extract from the testng index.html report.

        Methods in chronological order

        ...

      [here! missing the closing tag for div class="chronological-class"]

This doesn't seem to be a problem no more in 7.0.0
Heres a snippet

test.mixed.JUnit3Test1

    testA
    0 ms

    testB
    0 ms


Balance magic bean

Version 1.8.13
First issue is that magic beans can replicate themselves using the right click harvest function that some mods add. So whenever i right click the plant i keep the plant and get an extra magic bean. This doesnt happen with the cyclic harvesters.
Second issue is the balance within it. From my point of view all items have equal chances. I have my cyclic farm running for around 2-3 days on a 11x11 field. I have 130 netherstars laying around. Even without the bean dupe above the initial costs are fairly low..
I would suggest if it is possible to balance drop rates within the magic bean harvest. Let us get more clay/glass but less of valuable items such as nether stars/ghast tears. Or add a config to disable certain drops from the magic bean.

" the right click harvest function that some mods add." is this the one from harvestcraft, or a different standalone one?  I know there are a few.  By default, the bean gives you a seed item and the regular drops if broken normally so you can replant it.
The harvester tries to be smart about it and it looks up the seed item for the plant and removes one of those from the drops before replanting, so that part in particular might be a bug with the right click harvest mod.
You are correct, all items currently have equal chance.  I should add config file for the dropped items.  Now that I think of it, this would also allow for items from other mods to be dropped.
Another thing i can add to config is whether bonemeal affects them.  For a long time i had this disabled, but i allowed bonemeal by request.  This again makes it a bit OP so that should be a config toggle


Add [MaxTime] Attribute

We could add, similar to NUnit a MaxTime attribute. in order to Assert on Test performance.

Any assert fails will take priority
If a test passes but the time it took was too long then it will fail showing the allowed time in ms and then the time it took.

There is already a timeout attribute (even though it does not work very well), would this be sufficiently different from that?
Is this also possible to get this functionality on assertion level, for example by using FluentAssertions. I am not sure if we have any guidance on this, but personally like the xUnit approach where they keep the Assertion library minimal to only provide the core assertions, and allow it to be replaced by a library that does just assertions.

So this in turn is different from Timeout attribute.
[TimeOut] has the behaviour kill the test process with no TestCleanup() method being called. i believe its primary focus is to stop rogue test processes clogging up CI builds.
[MaxTime] is more of an assertion it runs all the TestMethod and looks at the execution time and reports assertion failure overtime with TestCleanup(). This makes it more of a performance tool for test execution. This works well with AAA pattern.
I can't think of any other way to implement a timing assertion apart on a Attribute.
However they have both of these concepts on NUnit

The intended behavior of TimeOut is to stop the currently running test and call the clean up, I see I forgot to paste the link, I meant to link to #522.
I think that if TimeOut worked correctly it would be very similar to MaxTime, while implementing MaxTime we would face the exact same problems as with TimeOut.

Ok then fixing that is the best course.


Update Tooltip to use @xstate/fsm

I'm not sure about this anymore. While building Carousel I've realised that xstate really over-eagerly re-renders. I'm not usually bothered by this but triggering an event that gets swallowed by the machine still causes a re-render even tho state hasn't changed so, if you're triggering lots of events in an irrelevant state your component re-renders like crazy regardless.
This seems to defeat the point of using a state machine for me. If I want to take advantage of the swallowing, the swallowing should effectively behave as if nothing happened. I don't want perf to be impacted by swallowed events 😬


Update PlatformIO to latest (4.4 as of writing) (IDFGH-6723)

https://registry.platformio.org/tools/platformio/framework-espidf should be update to the latest release of esp-idf (4.4 as of writing)

If you notice the owner of that package is platformio so I don't think espressif can update it

@negativekelvin I filed a ticket here as well, after looking for a long time to find a more appropriate place to file: https://github.com/platformio/platformio-registry/issues/7

Afaik Submitting packages to platformio registry is moved from platformio to espressif.
(Same as for the toolchains). So requesting here seems the right place.
@pedrominatel  is this correct for idf releases?

@Jason2866 I don't think so... @me-no-dev does have a way of uploading the toolchain packages (they show up in espressif namespace in platformio registry), however other packages (openocd-esp32, platform-espressif32) are under platformio namespace in platformio registry, so AFAIK we can't update them directly.

@igrr There is no restriction in uploading. If you upload the package it will be usable under ownership espressif. Older packages are still under the ownership platformio.
Same as it is for the toolchains. Old ones are still under the ownership of platformio.
So @me-no-dev or anyone who is a member in the platformio team account can upload packages in platformio registry (as owner espressif).

@Jason2866 Right, but then the resulting package would appear at https://registry.platformio.org/tools/espressif/framework-espidf and not https://registry.platformio.org/tools/platformio/framework-espidf, which I think is what the issue reporter is asking for?

@igrr Since the newer toolchains are already with the ownership espressif t would be imho no problem to have all "other" from espressif in platformio registry.
For the platformio maintained framework it is just a change in the  platform.json of the owner. I think Platformio crew would be grateful if the have less package pack work

@Jason2866 sorry for the late response. I have confirmed with PlatformIO folks that Espressif can perform updates of tools and libraries in PIO registry. Updates of development frameworks is a different matter which requires a contract with PIO. We are in the process of sorting out the details.

I am closing this ticket as expired. Original discussion was already answered. Thanks for understanding.


vendor chunk contains manifest file [since 1.1.2]

When using chunk-manifest-webpack-plugin with CommonsChunkPlugin, it seems my manifest file is "leaking" into the common chunk's files. I'm including new ChunkManifestPlugin({filename: 'wpManifest.json'}) in my plugins, as well as
new webpack.optimize.CommonsChunkPlugin({
        name: 'vendor',
        minChunks: ({resource}) => /node_modules/.test(resource)
    })

Looking in stats.compilation.chunks, I see that the vendor chunk now has the manifest listed in it!
// from console.log of parts of stats.compilation.chunks
foo [ 'fdc9b7b128.js', 'fdc9b7b128.js.map' ]
bar [ '2f10fcf981.js', '2f10fcf981.js.map' ]
app [ 'app-36a1f801d3.js', 'app-36a1f801d3.js.map' ]
vendor [ 'wpManifest.json',
  'vendor-8d40f60f58.js',
  'vendor-8d40f60f58.js.map' ]

Surely that shouldn't be there?

Probably duplicate of #47 and #48

Quite possible 😄


FieldRelate

Description
gen.FieldRelate  use check.BaseStruct which is in an internal package.
So end users will get use of internal package gorm.io/gen/internal/check not allowed

yeah, this is expected behavior. u can get *check.BaseStruct from method GenerateModel
    b := g.GenerateModel("banks")
    cc := g.GenerateModel("credit_cards", gen.FieldRelate(field.BelongsTo, "Bank", b, nil))

Hum I see. thank you for quick answer. Sorry about it !


Update listappt command to show all appointments within a specific timeframe

listappt currently displays all appointments. It is not that useful on its own.
Suggested implementation: listappt s/12/02/2024 9am e/14/02/2024 12pm.
This will display all appointments that fall within 12/02/2024 9am to 14/02/2024 12pm.

Not worth to do. We already have the trace feature.


create Helm repo for NE deployment chart

Any service providing HTTP server would do (see).
This can be done for example here on GH.

This was done in GH pages here https://github.com/nuvlaedge/deployment/settings/pages. The helm repo will be served from gh-pages branch, to which a separate step in the release GH action proposed as part of this PR https://github.com/nuvlaedge/deployment/pull/96 will be publishing the helm repo index.


Make Chef work with latest curl CA bundle and AWS S3 certificates

Right now Chef ships an old version of CA Certificates to get around the s3.amazonaws.com being issued by a Root CA that is no longer shipped in the CA Cert bundle. However, this creates other problems when Chef can't trust certificates that are generated by newer root CAs that are not included in the older CA bundle.
Reference: https://feedback.chef.io/forums/301644-chef-product-feedback/suggestions/9148300-front-s3-downloads-with-cloudfront-and-custom-ssl
Let's get a release of Chef that uses the newer CA bundle and modifies it to also work for AWS S3 certificates.

Since the only issue we current have and have had with the cert bundle is with the specific AWS S3 cert, the minimum viable product that will solve this issue is most likely to fix is just for the AWS certa and to download the current cacert bundle and append the cert they are removing.
We could import the perl script and hack it up to our liking but that will be more work, and while that would solve any future issues as well so far we've only ever seen this problem with this one cert, so YAGNI.


Issues w Requirement.txt

Please assign this issue to me under GSSoC'21
I'll do it and also correct Python 3.7 to 3.9

No, it's python 3.7.


Make certain structs derive Copy

I'm working on https://github.com/qdot/libappindicator-sys, which will use gtk-sys for some of the symbols it needs to fill in for function arguments. I'd like to use other g*-sys crates to fill in things like GObject/GType/GTypeClass/etc that are used as struct field types.
The problem is that I'm using bindgen for my library, and it creates all structs as #[derive(Debug, Copy)]. Since none of the G* structs impl Copy, I'm stuck using the versions that bindgen creates. It works, but it'd be nice to have everything provided from the g*-sys crates if possible.
Here's the commit where I brought this up on the project
https://github.com/qdot/libappindicator-sys/commit/e3f9ce63f2a811cd00f0a1d6b1a937df49e846b3#commitcomment-21061033
cc @jonhoo

From a very cursory glance at the codebase, it looks like this is something that would need to be implemented in https://github.com/gtk-rs/gir. Maybe @EPashkin can confirm? In particular, I'm guessing Copy can't be derived for all the generated types (e.g., things with pointers)?

@qdot, as @jonhoo says types for that Copy derived must don't contains pointers.
On fast glance I found only 3 of these in gtk-sys: GtkBorder, GtkPageRange, GtkRequisition
While I can add generating #[derive(Debug, Copy)] for them IMHO better do it on upper level (ex. not in GdkRectangle, but in gdk::Rectangle).
Currently Gir don't support generating upper level struct from these "direct" so all these struct currently manually written when needed, and seems Border, PageRange, Requisition not present in gtk-rs\gtk crate.

@EPashkin Problem with doing it at the gdk::* level is that we'd still have problems with what I'm trying to do for bindgen, since I'm running lower than that level. For instance, with libappindicator, bindgen just hands me something like this:
#[repr(C)]
#[derive(Debug, Copy)]
pub struct _AppIndicator {
    pub parent: GObject,
    pub priv_: *mut AppIndicatorPrivate,
}

I'd still need the gobject-sys to provide me a Copy derived version, since that struct is supposed to be a direct representation of the C struct. Right now, my bindings file just has its own bindgen-generated GObject implementation that does that.

I don't like idea copying classes structure's without using glib-provided functions.
Can you show example when it really needed?

I suppose it's not really needed in that things work as is right now, and I really don't expect anyone to be playing with the innards of the bindgen-generated structs in libappindicator-sys. It just seemed like it might be nice to not have bindgen redefine types provided by gobject-sys and other crates if I didn't have to.


Name of SSM parameter to store DAPA lambda function needs to be unique

The following ssm parameter name does not allow for multiple deployments in the same AWS account as the following error is encountered during deployment
Error: error creating SSM parameter (/unity/unity-ds/api-gateway/integrations/collections-dapa-function-name): ParameterAlreadyExists: The parameter already exists. To overwrite this value, set the overwrite option in the request to true.

  on .terraform/modules/unity_cumulus_lambda/main.tf line 214, in resource "aws_ssm_parameter" "cumulus_collections_dapa_ssm_param":
 214: resource "aws_ssm_parameter" "cumulus_collections_dapa_ssm_param" {

resource "aws_ssm_parameter" "cumulus_collections_dapa_ssm_param" {
  name  = "/unity/unity-ds/api-gateway/integrations/collections-dapa-function-name"
  type  = "String"
  value = aws_lambda_function.cumulus_collections_dapa.function_name
}

resource "aws_ssm_parameter" "cumulus_collections_create_dapa_ssm_param" {
  name  = "/unity/unity-ds/api-gateway/integrations/collections-create-dapa-function-name"
  type  = "String"
  value = aws_lambda_function.cumulus_collections_creation_dapa_facade.function_name
}

resource "aws_ssm_parameter" "cumulus_collections_ingest_dapa_ssm_param" {
  name  = "/unity/unity-ds/api-gateway/integrations/collections-ingest-dapa-function-name"
  type  = "String"
  value = aws_lambda_function.cumulus_collections_ingest_cnm_dapa.function_name
}

resource "aws_ssm_parameter" "cumulus_granules_dapa_ssm_param" {
  name  = "/unity/unity-ds/api-gateway/integrations/granules-dapa-function-name"
  type  = "String"
  value = aws_lambda_function.cumulus_granules_dapa.function_name
}

The new lambda param for fastapi has a prefix which allows multiple deployment.
The old parameters are being removed in PR #204


Harbor cfg migrator should be rerun-able on "up"

Currently, if i upgrade from 1.3 to 1.5, it only success on the first time. But always fail at the following times.

fixed in #4535


Possible additional async methods

Ok, I took a look. I'm remembering now that we started the async push with just adding files and data. Here are the remaining methods that could be changed if they're causing issues (I'll put a star next to the ones that we should do ASAP):
Adding data

AddMessage*
AddLike*
AddComment*
AddIgnore*
AddFlag

Invites

AcceptInvite*
AcceptExternalInvite*
IgnoreInvite*
AddInvite
AddExternalInvite

Profile

SetName*

Schema

AddSchema

Contacts

AddContact
RemoveContact

Notifications

AcceptInviteViaNotification*
IgnoreInviteViaNotification*

Threads

AddThread*
AddOrUpdateThread*
RenameThread*
RemoveThread*
SnapshotThreads*

Originally posted by @sanderpick in https://github.com/textileio/go-textile/pull/892#issuecomment-523962700

Original comment from @andrewxhill:

I'm still seeing some blocks in Dev at least.
Two clear places.

Join a Cafe with some photos already in your wallet. In this case, the UI will block until all photos have finished syncing. Ouch for more than a couple photos.
Send a message (text is fine). The keyboard will lock until the message is posted in the thread. Makes a boggy feeling keyboard for message->message->message rattling.

Can I fix this with something in the app / rn sdk?

I didn't want to lose this in a closed PR, so moved here.

I'm curious about the UI blocking when adding a cafe. I'd be surprised if it's actually the photos syncing (uploading to the cafe) that is actually blocking the UI.


docs: improve custom query handler docs

What kind of change does this PR introduce?

Describe the limitations for custom query handler code
Keep all of the detailed explanation in only one part of the docs instead of two

The second point is subjective but I hope you will agree that at least the first one is good. 🙂
Summary
It took me like an hour or two today to realize what's going on, when I finally looked at how puppeteer processes query handlers. 😅
I would like to make it easier for other people in the future.

@johanbay could you PTAL?


Donation page error

I followed the donation link to apply some appreciative sats, touched the lightning button, but got an error.

Thanks for the report .. its a reminder from the universe to get the Integrate Tallycoin Connect feature for RaspiBlitz done see #1450

Someone donated on-chain yesterday, off-chain still not working.

Because for legal/tax reasons it makes sense to handle donations like mentionen here #2292 anyway.

Just one thing, you are going to use their node, not yours, and "IRS".
DM with Ben Prince from OpenSats (abitcoinperson on twitter, did not find his username here), this is his responses, I cut my questions:

Our model, as of now, would be running our own btcpay instance. Each project has its own “store” and any donations to that store are tracked individually. 100% of the Sats would be passed on to the project with no fees. So generally same outcome with us in between (for 501c3 purposes) plus we add in fiat donations which will be converted and distributed in Sats
Unfortunately the irs has so many * rules
One of which prevents us from “directly” passing through donations. We need to make sure that we get projects to ensure they are helping fulfill our mission, which is very broadly written
Already kinda playing in a grey space here
That being said, I’m definitely gonna ask about the idea of ppl hooking in directly to the platform with their nodes
I think we might let them directly open channels and do payouts that way
At least that was my v1 idea.
Since we’re only paying out Sats, it’d make sense to use L2 when possible

Enphasize: This was what he described as a initial proposal, not definitive.
For legal/tax reasons, donations on a hiddenservice receiving with a nym :)
If being donated bitcoin, don't handle it to the mafia, I would be a indirect contributor to this atrocity by donating with this rules. Blitz and Bitcoin are not obliged to comply to constitutions, it just runs the code.
Please reconsider this.

Hmm .. OK sounds it adds more problems to the mix then it solves.


Ebola Outbreaks portal items need more information

Ebola Outbreaks items are missing Access and Use Constraints.  These items are planned to be released under different users. If the Web Map and Web App are released under the "Released" owner, the Feature Layer that is used by them will need to be released under "Released" owner, but will be released as a "Demo" owner. I've combined the following portal items into the same issue because they will most likely have the same Access and Use Constraints. These Ebola Outbreaks items need more information:
Ebola Outbreaks feature service id:8592ab25641446a0829964daf45a5548

[ ] Access and Use Constraints
[ ] Item is identified to be released under a Demo owner, please check to see if this should be a Released owner instead
Note: The summary and description are the same and should be modified; however, that will not hold up the release.

Ebola Outbreaks in West Africa (DEMO) id:f7026a1e91814f1caa1fac5a5ee656fa

[ ] Access and Use Constraints
[ ] Item will be released as a "Released" owner, please check to see if this should be a "Demo" owner instead
Note: The summary and description are the same for this Web Mapping Application and should be modified; however, that will not hold up the release.

Ebola Outbreaks in West Africa (DEMO) id:934c8c749bff49248a3eb23512793281

[ ] Description
[ ] Access and Use Constraints
[ ] Item will be released as a "Released" owner, please check to see if this should be a "Demo" owner instead

@nfeuerstein @jrweakland @karlterrey @sschwoppe @lfunkhouser @elinz
Based on the review from legal I am going to delete all Ebola content at 1100 today.

I deleted this item.


fix: normalize import

avoid to raising error when there is a foler or python file that has same name in working directory

https://github.com/PaddlePaddle/PaddleClas/pull/2207