fix wrong probe path

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.373%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3891

Relevant Lines:
4123

💛  - Coveralls

Thanks for your help.

@fjogeleit any idea when this will make it to the chart?


[TensorRT-LLM][ERROR] Assertion failed: Size per head is not a multiple of X (unfusedAttentionKernels.cu:1710)

When I quantized int8kv cache on the open source model of openllama 3B, in the last step of run.py,
[TensorRT-LLM][ERROR] Assertion failed: Size per head is not a multiple of X (TensorRT-LLM/cpp /tensorrt_llm/kernels/unfusedAttentionKernels.cu:1710) this error,
I printed out that the hidden size and head nums of the openllama 3B model are 3200 and 32 respectively, so the headsize （Size per head ）is 100, which is not a multiple of 8. The kernel does not support it. What should I do?

you should pad weights' head size to 104 or 128 (note that fmha doesn't support 104 head size), and use the new head size when building the engines.

you should pad weights' head size to 104 or 128 (note that fmha doesn't support 104 head size), and use the new head size when building the engines.

Ask a few more questions,  I understand that before building, should first use a script to pad the weight, and then the quantization (int8 scale) also needs to be redone using the new head size.
Is correct?  and does the unfusedAttentionKernels need to be changed?

I understand that before building, should first use a script to pad the weight, and then the quantization (int8 scale) also needs to be redone using the new head size.

Yes, you can think of that we have a new model checkpoint with padded weights.

does the unfusedAttentionKernels need to be changed?

It is not needed since the head size is multiple of 8 now. TRT-LLM should only know it is 104 (it doesn't matter whether it is padded or not).

I understand that before building, should first use a script to pad the weight, and then the quantization (int8 scale) also needs to be redone using the new head size.

Yes, you can think of that we have a new model checkpoint with padded weights.

does the unfusedAttentionKernels need to be changed?

It is not needed since the head size is multiple of 8 now. TRT-LLM should only know it is 104 (it doesn't matter whether it is padded or not).

thanks！
I think this kernel is transpose for kv cache, so this error will appear in int8 kvcache.
So this problem does not exist for GPTQ?  I can temporarily bypass this problem if I do GPTQ？

I think this kernel is transpose for kv cache, so this error will appear in int8 kvcache.
So this problem does not exist for GPTQ? I can temporarily bypass this problem if I do GPTQ？

The generation MHA kernels also have the limitation of multiple-of-8 head size，so padding with zeros is the best solution for now.

I think this kernel is transpose for kv cache, so this error will appear in int8 kvcache.
So this problem does not exist for GPTQ? I can temporarily bypass this problem if I do GPTQ？

The generation MHA kernels also have the limitation of multiple-of-8 head size，so padding with zeros is the best solution for now.

Hello, I have completed padding and can run it, but the sample result is wrong. I would like to ask, if my weight size is (32000, 3200), pad 3200 to 3584, size is (32000, 3584), so that each row A large number of 0s will be padded, but when the memory is read, the data is read row by row continuously. As a result, the data originally accessed in the second row now becomes 0. Therefore, the result is incorrect. Is this analysis correct?

You need to pad the head size, not the hidden size to multiple of 8.
so, it should be like 100 + 4 zeros + 100 + 4 zeros + .....
At the same time, you need to modify the model config (like hidden_size or head_size to the padded one). TRT-LLM will take everything as head size 104, and have no ideas about the zero paddings.

by default, head size = hidden_size // num_attention_heads.
for qkv-projection weights, you need to pad the head size as I described.  for the dense weights (after self-attention), you need to also pad each head size in the K dimension of GEMM operation while only need to pad zeros to the end of hidden size in the N dimension.  by doing that padding strategy, we can make sure the results are correct.
Then, you can pad zeros to the end of hidden size for all other weights (forget about head size).
We are working on making this padding strategy part of the engine builder, but it needs extra weeks to have that feature as we have too many features in the list.  hopefully, we can release that soon.

@PerkzZheng  In my opinion,
@
void invokeTranspose4dBatchMajor(const T* kSrc, const T* vSrc, KVCacheBuffer& kvTable, const int localBatchSize,
    const int seqLen, const int attentionWindowSize, const int sizePerHead, const int localHeadNum,
    const KvCacheDataType cache_type, const float* kvScaleOrigQuant, const int* sequence_lengths, cudaStream_t stream)
{
    // Block handles both K and V tile.
    dim3 blockSz(128, 2);
    constexpr int x = (sizeof(T) == 4) ? 4 : 8;
    dim3 gridSz((seqLen * sizePerHead / x + blockSz.x - 1) / blockSz.x, localBatchSize, localHeadNum);

    TLLM_CHECK_WITH_INFO(sizePerHead % x == 0, "Size per head is not a multiple of X");

sizePerHead  from  https://github.com/NVIDIA/TensorRT-LLM/blob/v0.8.0/cpp/tensorrt_llm/plugins/gptAttentionCommon/gptAttentionCommon.cpp#L459
mHeadSize from onnx node attribute, we can modify onnx node attribute to change mHeadSize to 104, and padding zeros for a series of weights accordingly.
On the other hand, we can close the check
TLLM_CHECK_WITH_INFO(sizePerHead % x == 0, "Size per head is not a multiple of X");

and overload transpose4dBatchMajorKVCache for sizePerHead % x != 0  case.


Use released react-router-dom version instead of removed @next

Use released react-router-dom version instead of removed @next

Thanks @ConneXNL


Imported VDB is very tiny!!

the scale of imported VDB is very tiny against of real world scale ...
it would be better if 100 times bigger ...

I am sorry that my response is so later.
I implemented Scale Factor parameter in VDB Inspector and the default value is 0.01.
Did you change this value?

so ... i can't change this guy !!

I know what you mean!
This inspector is a bug. I will fix it this weekend.

Unity's experimental feature ScriptedImporter is buggy...
Assembly Definition Files cannot use for ScriptedImporter...
I will try to remove the dependence of Assembly Definition Files.

Fix it
https://github.com/karasusan/OpenVDBForUnity/pull/6


Add MQTT support to SDK

Add MQTT messaging support to SDK

No need for this now, standard Paho library can be used for messaging.


Bower distribution Issues

Hi, I'm not capable to get the last version (currently the 1.6.8 according to the bower.json file) through bower. I think this is because the last version released is the 1.6.6. That version is not compatible with last version of angular material 1.1.0. Have you planned release the 1.6.8 version?

@AndreeVela hello, you might saw the development bower.json file, which usually lives in the master branch or most likely on a separate feature- branch.
You can never get the version specified in the master or feature- branch, it needs to be released in order to do that.
Now 1.6.8 is released, you can use it.


Solution of Helping chef in C++

Related Issue
https://www.codechef.com/problems/FLOW008
Solved issue number: #248
Describe the changes you've made
I have given a solution of this problem in C++.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C++
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Nice work @kanak22 :)


[Enhancement] Support relation alias name case insensitive

Fixes #issue
#14825
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 8 / 8 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/parser/SqlParser.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/Field.java
3
3
100.00%
[]

:large_blue_circle:
com/starrocks/qe/ConnectContext.java
4
4
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 11 / 11 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/parser/SqlParser.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/Field.java
3
3
100.00%
[]

:large_blue_circle:
com/starrocks/qe/ConnectContext.java
4
4
100.00%
[]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0


Add the option to add multiple ssh public keys for Azure infrastructure

This give users the option to define multiple ssh public keys when
deploying the base infrastructure on Azure.

I signed it


Don't crash if an environment directory is missing spec.json

Hey, thanks for your contribution! However, the issue you are trying to fix actually lies in the findBaseDirs function, which should never return invalid environments. I suggest the following:
diff --git a/cmd/tk/other.go b/cmd/tk/other.go
index f160d8c..fc86aa5 100644
--- a/cmd/tk/other.go
+++ b/cmd/tk/other.go
@@ -20,7 +20,10 @@ func findBaseDirs() (dirs []string) {
 	}

 	if err := filepath.Walk(".", func(path string, info os.FileInfo, err error) error {
-		if _, err := os.Stat(filepath.Join(path, "main.jsonnet")); err == nil {
+		if _, err := os.Stat(filepath.Join(path, "main.jsonnet")); err != nil {
+			return nil
+		}
+		if _, err := os.Stat(filepath.Join(path, "spec.json")); err == nil {
 			dirs = append(dirs, path)
 		}
 		return nil

Would you adapt your PR? Thanks!

Hey, are you still on this? Let me know if I should pick up :D

Hi @sh0rez -- I came across this when evaluating how much work it'd be to get my old ksonnet setup working with tanka; after fixing this and getting a little further I realized it was more work than I had time for (and I haven't gotten back to it).
Anyway, I added a new commit based on your feedback. I fixed it a little differently than what you suggested but I think this way shows the intent more clearly.


add pipeline for micromasters tables

closes https://github.com/mitodl/ol-data-pipelines/issues/142
Test locally by running pull_micromasters_data_pipeline from dagit
This doesn't have a schedule yet-I will add one once the rest of the pipeline for generating reports is built out

I think that before we merge this and start adding all of these new datasets to S3, it's worth writing the integration with the Glue data catalog to handle the registration of the tables and their schema. Otherwise it's a manual and error-prone process.


Prevent GNA image tag defaulting with K8s version

How to categorize this PR?

/area usability
/kind regression
What this PR does / why we need it:
https://github.com/gardener/gardener/pull/9477/files#diff-38d05f084de4de6e6424bae70053842cf593786e9dd357a4262025ff078810a4L53-L69 accidentally introduced a bug (see code document in the changes) which caused gardenlet to try deploying gardener-node-agent with a wrong image tag.
This PR fixes this regression.
We haven't noticed this in the e2e tests because we use image vector overwrites for the Gardener images (since they get freshly built by skaffold). This bug only occurs when no image vector overwrite for gardener-node-agent is used.
Which issue(s) this PR fixes:
Fixes #9577
Special notes for your reviewer:
Thanks @lotharbach for reporting 🚀
Follow-up of #9477
Release note:

A bug has been fixed which caused `gardenlet` to try deploying `gardener-node-agent` with a wrong image tag.

/cherry-pick release-v1.92

/cc @oliver-goetz


[Feature][create flink cluster] 是否可以通过dinky来创建一个flink集群

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
No response
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This feature has been implemented
The steps are:

Create a cluster configuration (guaranteed to be available)
Click Start Cluster in the Operation column (currently only yarn session can be started)
You can see it in the flink instance management


fmt: inconsistent padding of integers with and without f.sharp

go1.4 go1.6 (see playground)
http://play.golang.org/p/C8nFpvFtCa
package main

import (
	"fmt"
)

func main() {
	fmt.Printf("|%#6x|\n",1)
	fmt.Printf("|%#06x|\n",1)
	fmt.Printf("|%06x|\n",1)
	fmt.Printf("|%6x|\n",1)
	fmt.Printf("|%#.6x|\n",1)
}

Output
|   0x1|
|0x000001|
|000001|
|     1|
|0x000001|

I would have expected the length of the output of %#6x and %#06x to be the same, namely 6 characters/runes as the width flag suggests.
A few places in the standard library use e.g. %#08x instead of %#.8x.
There are 37 places outside of fmt_test.go for grep "%#0.*x" of which 19 seem to be error message formatting.
The fmt_test.go tests that fail were added to check overflow of the internal buffer.
Interestingly besides these only the decode test that uses %#08x in https://github.com/golang/arch/blob/master/arm/armasm/gnu.go#L70 seems to fail if i change the behavior of the 0 padding to be the same as the space padding. ( only a small change in the integer function of fmt when f.sharp is set )
Is it to late to make them behave the same with padding as to much code may already rely on this? Then maybe the documentation should note this deviation from normal padding behavior instead.
( Maybe this is something for go vet. when %#0Lx is used it was maybe intended to be %#.Lx .)

CC @robpike

comparison to c:
printf("%#06x", 0xffff)
prints 0xffff not 0x00ffff as go currently does.

Will not change since it would break existing code.


add option to pause build and prompt instead of auto install

For security reasons (though it may just be security theater) it might make sense to show a prompt instead of auto installing. This is to prevent a typod package for being auto installed.
As per this conversation on Twitter: https://twitter.com/tikotzky/status/682050451467038720

Based on #5, this may end up there (https://github.com/renke/auto-install-webpack-plugin).
Moar suggestions!

Timer (e.g. 5...4...3....2...1), kinda like Gmail's "undo" that a rebuild will cancel, or, if the terminal is open, a keypress will cancel it.

But, what I'm wondering is if introducing a keypress action would be no different than what we have today with:

Start the build & watcher in Terminal A.
Open Terminal B & run npm install whatevs.
Add import whatevs from "whatevs" in your editor & "save".
Terminal A picks up changes & rebuilds.

These projects would only improve the developer experience by cutting out 1 tab/window & a few keystrokes.
What sold me on this was that I would keep coding after I wrote the import through the implementation a ways before ever saving. (I mean, who saves right after putting in require #amirite!?)
Anyway, still digging the suggestion, just wanted to think through the goals again & align any work with them.

Just did some more research on this and I don't think this will work with regards to #5, as prompting is inherently asynchronous, and the script currently runs synchronously.
Going to close until I experiment with #5 some more and determine if there's a way around this (possibly using generators, since they do while loops).


Window Title Not Instantly Updated when showTabs Enabled

VSCode Version: 1.3 insider
OS Version: Win7

Steps to Reproduce:

Enable tab in user settings => "workbench.editor.showTabs": true
Open many files.
Select other tab.
The window title doesnt change instantly. Unless you resize/change other window or redraw occur?.

@ijat the window updating should happen in any case also when tabs are not enabled. you say you only see this when tabs are enabled?

A number of days has past without answer on the questions asked. I am thereby going to close this issue to clean up our backlog of issues. Once you can answer on the questions, the issue can be reopened again. Thanks for understanding this process!


Errors

Description of the issue
When i connect the Addon on the MQTT Integration i get these errors
Addon version
1.35.0-1
Platform
Core
2023.12.4
Supervisor
2023.12.0
Operating System
11.2
Frontend
20231208.2
Logs of the issue (if applicable)
1:
`Logger: homeassistant.components.mqtt.mixins
Source: components/mqtt/mixins.py:599
Integration: MQTT (documentation, issues)
First occurred: 13:20:10 (1 occurrences)
Last logged: 13:20:10
Erroneous JSON: online`
2:
`Logger: homeassistant.helpers.template
Source: helpers/template.py:2260
First occurred: 13:20:10 (1 occurrences)
Last logged: 13:20:10
Template variable error: 'value_json' is undefined when rendering '{{ value_json.state }}'`
3:
`Logger: homeassistant.helpers.template
Source: helpers/template.py:751
First occurred: 13:20:10 (1 occurrences)
Last logged: 13:20:10
Error parsing value: 'value_json' is undefined (value: online, template: {{ value_json.state }})`
4:
`Logger: homeassistant.components.recorder.db_schema
Source: components/recorder/db_schema.py:582
Integration: Recorder (documentation, issues)
First occurred: 13:20:10 (4 occurrences)
Last logged: 13:20:10
State attributes for switch.zigbee2mqtt_bridge_permit_join exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored
State attributes for sensor.zigbee2mqtt_bridge_permit_join_timeout exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored
State attributes for sensor.zigbee2mqtt_bridge_version exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored
State attributes for select.zigbee2mqtt_bridge_log_level exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored`

New Version exist


Link to OSM

When I click a POI, some details come up. But I know that back in OSM there are many more details. So, please provide a way for user to jump to the equivalent OSM link. Or, provide a link for opening the same map location on OSM for editing etc.

This is already possible.
When you click a POI on Qwant Maps the url looks for instance like:
https://www.qwant.com/maps/place/osm:node:1009073534
The last part of the url refers to the OSM id.
You have to concatenate this id with the OSM url to get the OSM link:
https://www.openstreetmap.org/ + node/1009073534
However we will consider to add a link that directly proposes the correct "concatenated" link.

However we will consider to add a link that directly proposes the correct "concatenated" link.

Yep, that's the request. Thanks!

:+1: for the request, thanks !

Implemented in https://github.com/QwantResearch/erdapfel/pull/321
and now deployed to production: https://www.qwant.com/maps/place/osm:way:309089736


Core requirements for Registry/Gateway Document Review & Rework

Back Story
Rajaneesh has sent this for review to Ravi and Binu & Tanya from ONDC (Binu has added some comments and those have been resolved). The review request has been sent on Sep 6.
The deliverable for this is part of the document here: https://docs.google.com/document/d/1JlE-JTgQCPVPtFUqs3dIfBma3Koi_pVsO4LwB4hGcM8/edit#heading=h.m1mzlygez2v3
Reference Card
#96

@vbabuEM - This card is a duplicate of #226 hance closing this card.
@viraj89 @emmayank @faizmagic


"Connect with SSH by using PuTTY" - incorrect description

[Enter feedback here]
The description for "Connect with SSH by using PuTTY" is somehow mixing public and private keys. Public SSH key is mentioned as prerequisite, but path to private key is requested. Following the description is leading to an error message, that publickey authentication is missing.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 04ae6dca-f225-c076-4b79-3ee5f76f530f
Version Independent ID: 25d0c989-e093-d2bc-c152-6d73bb44dc13
Content: How to use an SSH public key with Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/azure-stack-dev-start-howto-ssh-public-key.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@Langenbuch Thanks for your feedback. I'll update. Thanks!
#in-progress

@Langenbuch Sorry about the confusion in the titling and the intros in this topic. I updated those, so hopefully it won't be so confusing. Thank you for your feedback and taking the time to point this out.
#please-close

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


could not import name 'jpeg8_encode' from 'imagecodecs'" after packaging Django project with PyInstaller

Dear author
I am using tiffslide for my Django project. Every function works well when I run my code directly. But when I create a package with PyInstaller, running the package .exe throws the following error:
"could not import name 'jpeg8_encode' from 'imagecodecs'" when using PyInstaller
I tried specifying a hidden import like this::

And I also found the dist directory imagecodecs folder has only 3 file _shared.cp39-win_amd64.pyd, zlib.cp39-win_amd64.pyd to imagecodecs folder of dist
But it didn't work either. Could you fix the problem?
my env:
windows 11 64bit
Python 3.9.19
tifffile==2024.8.30
tiffslide==2.4.0
imagecodecs==2024.6.1

The hiddenimports = ... line looks wrong (extra underscore). Try:
hiddenimports = ["imagecodecs." + x for x in imagecodecs._extensions()] + ["imagecodecs._shared"]

Or try add the _jpeg8*.pyd, _jpeg2k*.pyd, and _imcd*.pyd files to datas


Implement a UI for multiple phrase search

PR #5579 implemented multiple phrase search, but as of now this is only accessible through the #search= URL parameter. It would be nice have a UI for this. We need to adapt our current find bar to add a checkbox for phrase search, but this also involves integration for the Firefox add-on.

See #7870 for corresponding pull request.

Given that this, as outlined in https://github.com/mozilla/pdf.js/issues/7368#issuecomment-425377443, is basically blocked on similar functionality existing for regular searching in Firefox itself, is it really meaningful to keep this issue open indefinitely?


Update appwrappers to 0.27.0

Changes:

Append autopilot anti-affinities to existing matchExpressions array
Add labels to aggregate appwrapper editor/viewer role RBACs
Bump Kueue from 0.8.1 to 0.8.3

@varshaprasad96 -- This includes the fix for the appwrapper bug fix with affinity injection that we want to be sure to get into 2.16.

Upgrading the Makefile KUEUE_VERSION from 0.8.1 to 0.8.3 just is the cleanest way to deal with the go dependencies of the upstream.  There's already a replace directive replace sigs.k8s.io/kueue v0.8.3 => github.com/opendatahub-io/kueue v0.7.0-odh-2 that back levels it for ODH.


[SPARK-3444] Provide an easy way to change log level

Add support for changing the log level at run time through the SparkContext. Based on an earlier PR, #2433 includes CR feedback from @pwendel & @davies

Merged build triggered.

Merged build started.

Test build #31340 has started for   PR 5791 at commit d9d03f3.

Test build #31340 has finished for   PR 5791 at commit d9d03f3.

This patch fails Scala style tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

final class Word2Vec extends Estimator[Word2VecModel] with Word2VecBase

trait HasStepSize extends Params

This patch does not change any dependencies.

Merged build finished. Test FAILed.

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

Merged build triggered.

Merged build started.

Test build #31352 has started for   PR 5791 at commit fac14a0.

Test build #31352 has finished for   PR 5791 at commit fac14a0.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

final class Word2Vec extends Estimator[Word2VecModel] with Word2VecBase

trait HasStepSize extends Params

This patch does not change any dependencies.

Merged build finished. Test PASSed.

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

Looks great, but left one small comment.

Merged build triggered.

Merged build started.

Test build #31596 has started for   PR 5791 at commit 338d7bf.

Test build #31596 has finished for   PR 5791 at commit 338d7bf.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #31606 has started for   PR 5791 at commit 338d7bf.

Test build #31606 has finished for   PR 5791 at commit 338d7bf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Merged build triggered.

Merged build started.

Test build #31628 has started for   PR 5791 at commit 3bf3be9.

Test build #31628 has finished for   PR 5791 at commit 3bf3be9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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


Can i minimize RAM setting of DectectionLab

Operating System Version: Windows11
Deploying via VMWare
Vagrant Version (if applicable): 2.2.19b\

Please verify that you are building from an updated Master branch before filing an issue.
Description of the issue:
My computer has more than 90% RAM workload, so i wanna know can i minimize RAM setting of DetectionLAB. Ex: logger from 4GB to 3GB and more. And how can i do it, edit the VM manually or change Vagrantfile file?
Thanks!

You're gonna be in for a bad time if logger runs out of RAM and needs to page to disk (from personal experience). But if you want to change the RAM usage, you can modify the VM without having to re-provision the whole box. If anything, I'd lower the RAM usage on the other machines that get spun up or just halt them depending on what you're doing at the time.

Like @n0leptr said, you can, but the lab was already designed with minimum RAM requirements in mind. Lowering RAM on any of these machines is likely to cause some significant slowdowns or issues, but yeah, all you have to do is modify the Vagrantfile


Move Rake tasks to rufus

So, we need to move the rake tasks that we have to use the new rufus system.  From the install guide
The following rake command needs to be put into Rufus.
# Updates the budgets for projects
rake upkeep:update_budgets

We don't care about the other 2 jobs at the moment.

@chriskacerguis - do you think that 1 hr intervals are good to run this on?

I'd do, 15 min

This was pretty much already done, it just needed the line added to the task_scheduler.rb file.

Added some feedback to the PR, I think update_budget needs some extra love before we schedule it out.


fixes bug when delete default handler was returning 200

fixes #408

Thank you - how bizarre, that function was never being invoked? The things hidden in codebases...
:beers:

hehe yep, didn't notice that they were square brackets at first :stuck_out_tongue_closed_eyes: easy to miss, would have left unnoticed if not the bug


remove saxpy; add note on prophet

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[x] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

Codecov Report

Merging #582 (ec3c63b) into master (d040213) will decrease coverage by 32.53%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #582       +/-   ##
===========================================
- Coverage   86.80%   54.27%   -32.54%
===========================================
  Files         119      117        -2
  Lines        5739     5691       -48
===========================================
- Hits         4982     3089     -1893
- Misses        757     2602     +1845

Impacted Files
Coverage Δ

etna/analysis/__init__.py
100.00% <ø> (ø)

etna/analysis/outliers/__init__.py
100.00% <ø> (ø)

etna/transforms/__init__.py
100.00% <ø> (ø)

etna/transforms/outliers/__init__.py
100.00% <ø> (ø)

etna/commands/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

etna/commands/backtest_command.py
0.00% <0.00%> (-96.43%)
:arrow_down:

etna/commands/forecast_command.py
0.00% <0.00%> (-92.00%)
:arrow_down:

etna/commands/__main__.py
0.00% <0.00%> (-87.50%)
:arrow_down:

etna/commands/resolvers.py
0.00% <0.00%> (-80.00%)
:arrow_down:

etna/analysis/outliers/density_outliers.py
22.44% <0.00%> (-75.52%)
:arrow_down:

... and 70 more

Continue to review full report at Codecov.

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


Nomenclature Help

Hello guys, Today I successfully integrated with X10A. Since I'm no expert in this field i might be missing some names:

DHW -> domestic hot water
R1T - R2T etc?
RT Temp?
LW?
BUH?

:)

Hi @vekexasia
R1T, R2T etc. are conventional names for specific temperatures in the circuits of external or internal unit. But the definition is in the name.
RT Temp is the return temperature when it comes back from your house to the heater.
LW is the target temperature from your water logic (function of the external temp)
BUH is backup (electric) heater.
You'll find some info there (and can contribute too! :) )
https://github.com/raomin/ESPAltherma/wiki/Information-about-Values


fix hide some properties in show create catalog stmt

the use_meta_cache and create_time property should not be visible in
show create catalog stmt.
These properties are not for user.

run buildall


Removed sql.Convert usage, moved everything to casts

This removes our reliance on sql.Convert and changed everything to make use of casts. Casts have three contexts: explicit (value::type), assignment (INSERT and UPDATE statements), and implicit. In addition to moving to casts, I've also added assignment casts.
This move also comes with a massive change to how function overloads are handled. Overload resolution is handled through available casts, so by removing the hacky behavior that was in place, we're now able to unskip a number of tests that just could not be handled through the previous logic. There were other tests that were also incorrect, and those have been fixed.
Overall, this has resulted in a fairly substantial decrease in code throughout the codebase, in addition to allowing for types to finally have "full" support.
Companion PR:

https://github.com/dolthub/go-mysql-server/pull/2523

Skipped two bats tests as they're using comparison operators. Right now we're still using GMS' comparison operators, so they'll fail if the types don't match. Doltgres already would handle the implicit conversions, so whenever we add all of the comparison operators, we can unskip the tests.


Hard freeze with blinking cursor when window.zoomLevel is -1 or below

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
Version: 1.69.0-insider
Commit: f361c5b71d6676cfc6de97cdb1cc40b08bf7d994
Date: 2022-06-22T05:19:57.057Z
Electron: 18.3.4
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.14.21-150400.22-default
Other version also affected, see What happens below.

OS Version: Linux/Windows

Steps to Reproduce:

Disable all plugins and all default settings in setting.json
Set zoom level to -1 from keyboard or by modifying settings.json
Open 1 text document or file in the project
Keep hovering mouse just over the edge of toolbar and EXPLORER window close to Explorer, Search, Source Control icons

What happens:

Cursor keeps flickering between default cursor (arrow) and pointer cursor (hand) when on the edge of the toolbar

Reproducible on Windows (at least with version 1.67.*) and Linux with all versions starting with 1.65.2 onward

All Visual Studio Code windows freeze (become unresponsive to any input), cursor continues flickering when it is withing Visual Studio Code window, VSCode application needs to be killed to continue work.
* Reproducible on Linux starting from version 1.66.2 onward (at least these version are also affected: 1.68.1 and 1.69.0-insider f361c5b7...)
* Not reproducible on Linux with 1.65.2 (no freeze, no blinking cursor outside of "critical region")
* Not reproducible on Windows (no freeze, no blinking cursor outside of "critical region")

What should happen:

Cursor should not flicker
Visual Studio Code should not freeze

Additional information:
While this sound like a very strange corner case, hard freeze repeatedly happens during normal work when I just drag mouse to the toolbar to switch between Explorer and Source Control and happen to cross that "critical region". Today it happened to me at least 3-4 times in an hour until I downgraded the version to continue normal work without regularly force shutting down VSCode processes. Getting zoom level at -1 or below is also very trivial especially when trying to fit more information on a small screen (for example, laptop).
"Critical region" is within the red box on the screenshot below:

Please try to CPU profile this. Hit F1 > Toggle Developer Tools. In the overflow menu select 'More Tools > JavaScript Profiler'. In there select start, let it profile for 30 to 60 seconds, stop it, save it to a file and attach the file here. Thanks.
https://user-images.githubusercontent.com/1794099/120606621-a97e0180-c44f-11eb-8e32-9fdafdc949ec.mov

@jrieken: Tried that. I can start recording, but there is no way to save, since everything gets locked up: no buttons, no shortcuts, no window resizing.  If there is a way to limit profile length and autosave in the end (without pressing anything), there is a chance I can get that profile, otherwise I cannot really follow instructions above.

Okay, figured it out: code --remote-debugging-port=9222, then connected with Chrome and captured profile from there (had to .zip it otherwise GitHub is not happy):
cursor_blinking_freeze_CPU-20220622T194825.cpuprofile.zip.

It looks like interacting with the DOM is really slow for you...

@deepak1556 Do you have an ideas why that would be?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

I also have this issue, the same "critical zone".
Its a serious bug, I often let code running on interactive jupyter notebook, and all of a sudden this bug happens and I have to close vscode and lose my progress

I have the same issue on Ubuntu 22.04. Also works with zoom level 1.

The same happens to me when I'm using the debug. I'm working with Django, Chrome and FastAPI all running together on my debugger.

Same issue on Linux Mint 20 and Linux Mint 21.
It goes away if you wait for a few minutes, but the time is variable.

I had the same problem in Manjaro wit Vscode, but I detected the same problem in others apps who use Electron. So I think it's a problem with Electron. For example I got blinking cursor and freeze with the app Hamsket when I'm trying to select an emoticon.

 Have just tested this again and I still get blinking cursor, but no crash on this version:
Version: 1.72.2
Commit: d045a5eda657f4d7b676dedbfa7aab8207f8a075
Date: 2022-10-12T22:16:26.920Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.14.21-150400.24.21-default
Sandboxed: No
So from my side it seems that this issue can be closed. Can anyone confirm?

Hey, fellow VS Code users! Have just tested this again and I still get blinking cursor, but no crash on this version:
Version: 1.72.2
Commit: d045a5eda657f4d7b676dedbfa7aab8207f8a075
Date: 2022-10-12T22:16:26.920Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.14.21-150400.24.21-default
Sandboxed: No
So from my side it seems that this issue can be closed. Can anyone confirm?

Closing per https://github.com/microsoft/vscode/issues/152843#issuecomment-1304795968, can reopen if the issue persists.


Project : Word Counter

General Requirements :

Create a word Counter.
It should count the exact number of words written on an element.
You can implement in Any form.
Be little more creative.

Project Issues will be assigned on First Come First Serve Basis after October 1

Hi... I'm new to open source. Can I contribute to this issue?. I think I can do it.

@KaveenLaksitha go ahead 🕺

@soumik2012 Thanks. If you don't mind, can you tell me how to start?

Step to Contribute

Fork this repository
Clone the repository in your computer
Create a branch projectname-username-branch
Add your projects to a new directory, the name of the directory should be your ```Word_Counter``
Add One project at a time
Add & commit
Push changes to GitHub
Compare & Submit a Pull Request

@soumik2012 Thanks

I want to contribute on it.

@soumik2012 I've submitted a pull request.

@soumik2012  task is completed and pr ready to merge.


I'd like to use bwa-meth instead of bismark BAM files, is it possible?

Hi @sorjuela ,
great package! I was looking for a differential ASM detection tool like yours, however I have BAM files generated by bwa-meth and I do not want to use bismark, is there any possibility to use bwa-meth output as input of your tool?
Best,
Katarzyna

Dear @katwre,
@sorjuela is away for a couple days, so I'll give a brief answer. I think 'no', because under the hood we use methtuple from @PeteHaitch and on the main page of his repo, it says "Currently, the SAM/BAM file must have been created with Bismark." But, to be honest, I don't know what the differences are in what gets into the BAM; maybe a simple conversion can be done.
Best, Mark

methtuple parses the XM-tag of Bismark. From memory, bwa-meth doesn't produce this tag, so that's why it's limited to Bismark

@markrobinsonuzh and @PeteHaitch ,
Thank you very much for your quick reply!
methtuple-based approach looks great, but I think for now I will give a try with SNP-based one.
So then I have a question, if I go for using a bam file + output from BisSNP/biscuit then "MD" and "XG" tags in the alignment could be sufficient? I am asking because it looks like in DAMEfinder code  "MD","XM","XR","XG" tags are extracted from a BAM file (e.g. like here https://github.com/markrobinsonuzh/DAMEfinder/blob/87f43acb506131eadfc289b054f3813bc46fc74f/R/split_bams.R#L90), but actually only MD (mismatching positions/bases) and XG (genome conversion state for the alignment, it's CT or GA from what I've just read) are used and in my bwa-meth alignment I can see MD tag and YC tag that is always either GA or CT...
ST-E00351:144:HF3:6:2219:28412:67164      163     chr1    9990    0       8M2D108M        =       9990    118     TATCATTCATAACCCTAACCCTA
ACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACCCTAACC    AAAFFJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJJ
JJJJJJJJJJJJAJJJJJFJFJJAJJFJJJJJJJJJJAJJJFJJJAJ<A-<AJF--A-AAF7AFF-AAJA    MC:Z:8M2D108M   YC:Z:GA MD:Z:1G2T0G2^CG108      YD:Z:r  PG:Z:MarkDupl
icates     RG:Z:XXX-RUNID-0144-FLOWCELL-AHF3-LANE-6__val_ NM:i:5  AS:i:108        XS:i:107

Hi @katwre, thanks for your interest in the package!
You’re right, for the SNP-mode I actually only use the MD and XG tags. The problem here is that bismark has a specific type of MD tag, which could look like this for example: ‘ 2C0C3C7C2C7C1C0C1C1C1C2C2C0C1C2C2C0C6C7C0C0C0C0C0C4C3C1C0C7C5C0C1C5C1C5C1C8‘, unlike a typical MD tag that looks more like the one in your bwa bam file. Looking at your read makes me think that maybe we could combine the MD with that MC tag, but would have to check what that is next week. My current MD parser only works for the bismark MD, sorry.
Best,
Stephany

Hi @sorjuela ,
Thank you for your response! If you would have any idea how to use MD or combination of MC and MC tags or how actually Bismark creates MD tags that would be very useful!! (e.g. do MD tags have only Gs or Cs in their string? are MD tags calculated from CIGAR strings?) Should I use samtools calmd to recreate MD tags in order to mimic bismark-like MD tags? I tried already few tools for differential allelicly methylated regions calling that use SNP information and unfortunately they don't really work for me, so that's why I would be open to help make your tool work with bwa-meth and SNPs called from WGS / biscuit SNPs
Cheers,
Katarzyna

Several things:

Why don't you want to use bismark? I know it requires a lot of memory, but this is really a problem if you have veeery large files.
The MD tag in bismark is like an addition to the CIGAR, because I can use it to do SNP calling, AND to call (allele-specific) methylation. Reason why you see all those Cs, these are mismatches generated by the bisulfite conversion, meaning if it's a mismatch, the original C was unmethylated. As PeteHaitch mentioned above, he uses the XM tag, which is a string with methylation calls (easier and maybe even better than using the MD). I could also use this, but I chose to use the MD because of the above (SNP and meth calling from one place).
The read tags from your output don't seem to contain any methylation information, but it's hard to say, and I can't find any documentation on bwa-meth outputs. Extracting allele-specific methylation from there doesn't seem very straightforward...what tool do you normally use to call methylation?
(side question) What other tools do you know for differential ASM? as far as I know no other tools perform differential ASM. However there are other ASM callers that do not perform differential analysis.
Best,
S

I have rather big files .fast.gz files (paired-end data, around 45G for each fastq.gz corresponding to a mate), it's cancer patient-derived data and it looks like bwa-meth performs better on this data than bismark
Thank you a lot for explanation of your implementation and having a look at read tags information! Since it doesn't look that there is a straight-forward and fast solution, then probably I will just give up with it for now. I call methylation using methylDacker (https://github.com/dpryan79/MethylDackel) for reads aligned using bwa-meth
I tried biscuit (https://github.com/zwdzwd/biscuit) and https://github.com/astatham/snappymeth
I think this issue can be closed then
Thanks,
Katarzyna


Profile email and picture in GUI (left menu)

When someone new gets the code, do they have to login with their email?  If so, something to capture in the instructions & also what do they do with the email?  Since it's INL, they may be sensitive about us doing something with their emails.
@mikevasan98 @tylerstern0

prio:low

When someone new gets the code, do they have to login with their email? If so, something to capture in the instructions & also what do they do with the email? Since it's INL, they may be sensitive about us doing something with their emails.
@mikevasan98 @tylerstern0

We expected to have login because we wanted to manage every execution based on the individual user, i'm still working on that part..We are not going to collect their email, database control has been owned by themselves hence no need to worry about it

TY.  We should ensure them some how that their emails are not being sent to a foreign power, their activities and files monitored, etc.
Maybe we can just use usernames rather than emails?  That will partially assuage them re: security concerns.

TY. We should ensure them some how that their emails are not being sent to a foreign power, their activities and files monitored, etc.
Maybe we can just use usernames rather than emails? That will partially assuage them re: security concerns.

yeah sure..


Add validator for AWS::RDS::DBCluster.ServerlessV2ScalingConfiguration

I run the make regen and commit the file. Not sure if it is needed or not.

Thank you for the PR.


[D1] add a page for remote development

I wasn't actually sure where this page should live, but the content should definitely be in our docs.

@rozenmd do you have permission to merge this?

@Maddy-Cloudflare - yep!


update training for CYOA if space in option

I should add linting for this...

I wanted to be sure what I found by try/test was the good way to do. Thanks @hexylena


Visual bug on Windows

After refresh and pointing the cursor to the right of the bar, the first two bookmarks have flickering stripes at the top. Couldn't reproduce it on Linux. Browser is the same - latest Waterfox. Visible only with glass style.

That has to do with the browser, firefox handles opacity and blur differently than chromium. It seems to happen to me on firefox as well, but if I place my cursor over the bookmark, it goes away.
I don't have the ability to fix this since this has to do with how the browser renders things, which I can't control.

I couldn't reproduce this on FireFox 127.0.2, windows 10

Other extensions aren't allowed to load on the new tab page, it should just be a bug with how firefox renders stuff


Agent command buffer not being reset

Describe the bug
The agent will continue to act with the same command after a env.reset, indicating that its command buffer has not been reset. This spills into the pre start steps, causing the agent to move about during the pre start steps
To Reproduce
env = holodeck.make("InfiniteForest-MaxDistance")
env.reset()
env.step([0, 0, 100, 100])
env.reset()
for _ in range(100):
    env.tick()

Expected behavior
The agent should have no action after a reset.

Fixed in BYU-PCCL/holodeck#258


Android SplashScreen implementation

Hi,
i think you should consider to modify the sample to this approach: https://medium.com/android-news/the-complete-android-splash-screen-guide-c7db82bce565 (1. Using a Launcher Theme (The good))
Because sometimes the SplashScreen (in another activity) isn't show. It depends how android restarts the app.
Reference:
https://stackoverflow.com/questions/50789501/xamarin-android-splash-screen-doesnt-work-on-resume
https://developer.android.com/topic/performance/vitals/launch-time

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

ID: a5a4e93b-006f-176e-cf02-108f48bff807
Version Independent ID: 2d677054-ae70-1eb1-903d-70b4f6cdfc50
Content: Xamarin.Forms splash screen - Xamarin
Content Source: docs/xamarin-forms/user-interface/splashscreen.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @profexorgeek
Microsoft Alias: jusjohns

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and it's documentation can be found here.


Getting proxy image error on macOS Ventura 13.3

In our environment, all Internet access must pass thru our Corp firewall...we have our proxy info set in .zshrc (http_proxy, https_proxy, HTTP_PROXY & HTTPS_PROXY all set to http://<proxy.xxxxx>:8443), and continue to receive this error:

Even removing all of our proxy settings and using our guest network (not proxied), still produces the same results
The lima.log file shows this:
2023-04-10T18:36:02.670Z: > limactl copy /var/folders/5n/f2m9m8b16jz24ng8dprpm3qr0000gp/T/rd-nerdctl-e2Bcx7/nerdctl 0:/tmp/rd-nerdctl-e2Bcx7.nerdctl
2023-04-10T18:36:02.682Z: Lima: executing: chmod 755 /tmp/rd-nerdctl-e2Bcx7.nerdctl: Error: spawn /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura EACCES
2023-04-10T18:36:02.683Z: > /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura shell --workdir=. 0 sudo rm -f /tmp/rd-nginx.conf-QbZ2zf.nginx.conf
2023-04-10T18:36:02.683Z: Error starting lima: Error: spawn /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura EACCES
at ChildProcess._handle.onexit (node:internal/child_process:283:19)
at onErrorNT (node:internal/child_process:478:16)
at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {
errno: -13,
code: 'EACCES',
syscall: 'spawn /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura',
path: '/Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura',
spawnargs: [
'copy',
'/var/folders/5n/f2m9m8b16jz24ng8dprpm3qr0000gp/T/rd-nginx.conf-QbZ2zf/nginx.conf',
'0:/tmp/rd-nginx.conf-QbZ2zf.nginx.conf'
]
}
2023-04-10T18:36:02.795Z: > /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura shell --workdir=. 0 sudo rm -f /tmp/rd-nerdctl-e2Bcx7.nerdctl
2023-04-10T18:36:02.796Z: Error trying to start/update containerd: Error: spawn /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura EACCES:  Error: spawn /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura EACCES
at ChildProcess._handle.onexit (node:internal/child_process:283:19)
at onErrorNT (node:internal/child_process:478:16)
at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {
errno: -13,
code: 'EACCES',
syscall: 'spawn /Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura',
path: '/Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl.ventura',
spawnargs: [
'shell',
'--workdir=.',
'0',
'chmod',
'755',
'/tmp/rd-nerdctl-e2Bcx7.nerdctl'
]
}
We cannot remove proxy info for application to function.  All connections to the internet must be proxied...

opened in wrong forum...sorry...


TinyMCE 400

Mezzanine version: 3.1.1
Django version: 1.6.2
Python version: 2.6
OS: CentOS 6.5
I'm fairly new to Mezzanine, but I have it mostly running here.  However, "Rich text" pages are not currently working in that any pop ups (Add, image, "HTML", link, etc.) return a "Bad Request (400)".  From the debug logs (domain redacted):
[19/Apr/2014 18:48:30] "GET /asset_proxy/?u=http://www.domain.com/static/grappelli/tinymce/jscripts/tiny_mce/plugins/advlink/link.htm HTTP/1.1" 400 26
[19/Apr/2014 18:48:33] "GET /asset_proxy/?u=http://www.domain.com/static/grappelli/tinymce/jscripts/tiny_mce/plugins/advlink/link.htm HTTP/1.1" 400 26
[19/Apr/2014 18:48:37] "GET /asset_proxy/?u=http://www.domain.com/static/grappelli/tinymce/jscripts/tiny_mce/themes/advanced/source_editor.htm HTTP/1.1" 400 26
I'm using a ProxyPass/ProxyPassReverse to localhost from Apache as I have some other projects served from this server.  The fact that it's getting to Mezzanine debug means the Proxy is working through Apache, but failing on the builtin web server.  I have the "domain.com","www.domain.com","localhost", and "<IP_ADDRESS>" in ALLOWED_HOSTS.  For good measure, I tried * to see if that was the issue and it was the same result.

I had the same issue. My setup is also same as you using hostname proxy.
Your fix worked. Thanks

I should note that I've since used nginx for dev and deployment with Mezzanine and the stock code has worked fine with nginx, though the above snippet worked fine for me when I was using Apache.

Fix apache configuration with:
ProxyPreserveHost On
Now all will work without patching.


How to create an Embeddable js code for other website to Embeed

I have a simple widget in react js using zoid im stack on how to create the share embeed code for other website to embeed. I need some help

i thought simply provide one line javascript

How do i go about that? do u have any example I can look at?

<script
    src="https://yoursite.com/js?client-id=SB_CLIENT_ID">
  </script>

I serve that static js file using nodejs where I can process SB_CLIENT_ID to lookup database and pull necessary information. Then the script which generated by webpack earlier will be serve using ejs package to rewrite all the ejs variable with the data I have from database. That's it. I can serve react app then


OAuth status?

Hi, I see stuff in the code for OAuth - is it all ready to use?
I ask as there's no mention in the main readme or the examples.
Folks on the Flickr API group are getting antsy about the potential end of the old authentication mechanism.
I'm ready to move my stuff over if/when Flickraw supports it.
If it is all working, some examples would be really useful.
Thanks, Darren.

Hello Darren,
Flickraw uses only oauth authentication since version 0.9.
You use methods flickr.get_request_token, flickr.get_authorize_url and flickr.get_access_token.
See https://github.com/hanklords/flickraw/blob/master/examples/auth.rb
Regards,
Mael

Hello Darren,
Flickraw uses only oauth authentication since version 0.9.
You use methods flickr.get_request_token, flickr.get_authorize_url and flickr.get_access_token.
See https://github.com/hanklords/flickraw/blob/master/examples/auth.rb
Regards,
Mael

Ah, that'll teach me to read properly. :-)
Will upgrade (was on 0.8.2) and see if I can figure it out.
Thanks.

Ah, that'll teach me to read properly. :-)
Will upgrade (was on 0.8.2) and see if I can figure it out.
Thanks.


Field Guide appearance should match "Map" appearance

in the recent release (todays)  the Map looks great, very organized and easy to see.  The field guide requires a popup to see other articles.  It would look good if it matched the layout of the "Map" page.

Considering that we will soon have maybe hundreds of entries in our field guides, I think it will look a lot nicer if we stack them in a modal so they're out of site of normal browsing. Also, this means that the footer is still accessible without a ton of scrolling. We will definitely consider doing a map-like presentation in the future though. I hadn't even thought of this possible approach.


nginx 正向代理再增加一层squid做正向，如何指定使用squid的IP和端口 // how to specify next-hop ip address of proxied squid server?

$connect_host
host name from CONNECT request line.
$connect_port
port from CONNECT request line.
$connect_addr
IP address and port of the remote host, e.g. "<IP_ADDRESS>:12345". IP address is resolved from host name of CONNECT request line.
是使用以上的变量么，我试了一下，都报错 nginx: [emerg] unknown directive "connect_addr，请问具体如何使用，多谢？

You can use proxy_connect_address directive to specify IP address of the squid.

Example of proxy_connect_address:
        proxy_connect;
        proxy_connect_allow all;
        proxy_connect_address <IP_ADDRESS>:8082;

I have tested it, but it still shows the IP of the server. Here is my configuration：
   # forward proxy for CONNECT request
    proxy_connect;
    proxy_connect_allow            all;
    proxy_connect_connect_timeout  10s;
    proxy_connect_read_timeout     10s;
    proxy_connect_send_timeout     10s;
    proxy_connect_address <IP_ADDRESS>:23333;

    location / {
      proxy_pass http://$host;
      proxy_set_header Host $host;
    }

Check at httpbin.org:
{
"args": {},
"headers": {
"Accept": "/",
"Accept-Encoding": "gzip, deflate",
"Host": "httpbin.org",
"User-Agent": "python-requests/2.22.0"
},
"origin": "<IP_ADDRESS>, <IP_ADDRESS>",
"url": "https://httpbin.org/get"
}

but it still shows the IP of the server

I dont know what your problem is.

Check at httpbin.org:

What is your network model?
And please describe what you expected to happen.

proxy_connect_address <IP_ADDRESS>:23333;

This directive means it will proxy CONNECT-tunnel traffic to endpoint <IP_ADDRESS>:23333.
Note that your client should send CONNECT request first then send raw stream from the connection.

@berny9015 解决了这个问题吗
 server {        resolver <IP_ADDRESS>;        listen                         8355;        proxy_connect;        proxy_connect_allow            all;        proxy_connect_connect_timeout  10s;        proxy_connect_read_timeout     10s;        proxy_connect_send_timeout     10s;        proxy_connect_address <IP_ADDRESS>:3128;        location / {            proxy_pass http://$host;            proxy_set_header Host $host;        }     }
这是我的配置<IP_ADDRESS>:3128是下一跳squid的地址输入命令
curl -k https://github.com/ -v -x <IP_ADDRESS>:8355
显示错误：
`

Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8355 (#0)
Establish HTTP proxy tunnel to github.com:443

CONNECT github.com:443 HTTP/1.1
Host: github.com:443
User-Agent: curl/7.47.0
Proxy-Connection: Keep-Alive

< HTTP/1.1 200 Connection Established
< Proxy-agent: nginx
<

Proxy replied OK to CONNECT request
found 148 certificates in /etc/ssl/certs/ca-certificates.crt
found 592 certificates in /etc/ssl/certs
ALPN, offering http/1.1
gnutls_handshake() failed: An unexpected TLS packet was received.
Closing connection 0
curl: (35) gnutls_handshake() failed: An unexpected TLS packet was received.
`

hi @liuchenx

gnutls_handshake() failed: An unexpected TLS packet was received.
Closing connection 0
curl: (35) gnutls_handshake() failed: An unexpected TLS packet was received.

The connect-tunnel worked well for your case, in which case the connect tunnel has been established and raw stream( https request for github.com) has been proxied to backend server (squid).
But the squid server failed to do SSL handshake, may your squid server is not configured to server SSL request.

@berny9015 ping?
Has it been solved?

think it resolved. Feel free to comment if you still have this problem.

I have the same problem.
I need add "Certificate Authentication" to SQUID and I have chosen to put an nginx in front of the squid. My SQUID have SSL Bump configured.
My NGINX conf:
`
upstream _squid {
server <IP_ADDRESS>:31288;
}
server {
listen 33334;
error_log  /var/log/nginx/error.33334.log debug;
access_log  /var/log/nginx/host.33334.log  main;
proxy_set_header Host            $host;
proxy_set_header X-Real-IP       $proxy_protocol_addr;
proxy_set_header X-Forwarded-For $proxy_protocol_addr;
    proxy_connect;
    proxy_connect_address   <IP_ADDRESS>:31288;
    proxy_connect_allow            all;
    proxy_connect_connect_timeout  10s;
    proxy_connect_read_timeout     10s;
    proxy_connect_send_timeout     10s;
    location / {
            proxy_pass       http://_squid;
            proxy_set_header Host            $host;
            proxy_set_header X-Real-IP       $proxy_protocol_addr;
            proxy_set_header X-Forwarded-For $proxy_protocol_addr;
            #proxy_pass http://_squid;
            #proxy_set_header Host $host;
    }

}
`
My SQUID config:
`
http_port 31288 ssl-bump cert=/etc/squid/ssl_cert/AbitabCA.pem key=/etc/squid/ssl_cert/Abitab.pem generate-host-certificates=on dynamic_cert_mem_cache_size=4MB
sslcrtd_program /usr/lib64/squid/ssl_crtd -s /var/lib/squid/ssl_db -M 4MB
sslcrtd_children 8 startup=1 idle=1
always_direct allow all
ssl_bump server-first all
ssl_bump bump all
sslproxy_cert_error allow all
sslproxy_flags DONT_VERIFY_PEER
`
Curl output:
`
curl -x <IP_ADDRESS>:33334 https://vermiip.es -v -k

Rebuilt URL to: https://vermiip.es/
Trying <IP_ADDRESS>...
TCP_NODELAY set
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 33334 (#0)
Establish HTTP proxy tunnel to vermiip.es:443

CONNECT vermiip.es:443 HTTP/1.1
Host: vermiip.es:443
User-Agent: curl/7.54.0
Proxy-Connection: Keep-Alive

< HTTP/1.1 200 Connection Established
< Proxy-agent: nginx
<

Proxy replied OK to CONNECT request
ALPN, offering h2
ALPN, offering http/1.1
Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
successfully set certificate verify locations:
CAfile: /etc/ssl/cert.pem
CApath: none
TLSv1.2 (OUT), TLS handshake, Client hello (1):
error:1400410B:SSL routines:CONNECT_CR_SRVR_HELLO:wrong version number
stopped the pause stream!
Closing connection 0
curl: (35) error:1400410B:SSL routines:CONNECT_CR_SRVR_HELLO:wrong version number
`


Update email template for new style

I've updated the email template but haven't tested it in an actual email sending flow with real data.
Could someone help with that?
Resolves #215

@SheldonCheng overall looks good. But the box highlighted in the red should not be bold right?


Nos social deploy

This merge updates the existing traefik and nos_social roles so that the resulting service fits the requirements outlined in #66 .
It modifies traefik to use cloudflare dns-challenges for it's certresolver, instead of a single-file http challenge.  This allows for arbitrary wildcard domains.  It also configures traefik to create certificates for {{ domain }} and *.{{ domain }}.
On the nos.social side, there are several updates to the docker compose:

adding a redis service, with a persistent docker volume
updating the routes to direct traffic to the api, webflow, or the traefik dashboard accordingly,
refining the rules and tasks to be more succinct.

The host rules in the docker compose are quite verbose.  This has two advantages: a blunt clarity (hopefully), and an built-in priority.  Traefik sets priority based on the length of the rule, so we check to see if the traffic should be handled by the api first before redirecting to the webflow site.  I will likely revisit this though to see if i can say this more clearly and briefly.

Looks good!


Wrap sendAlert() in a try/except block.

Fixes #10464
Status
Ready
Description
While using the old mechanism for sending alerts from MSTransferor we were not trying to handle exceptions at place, when it comes to sending alerts. But this was breaking the creation of the Rucio rules for a  given data block. Or more precisely it was calling Rucio successfully and creating all the necessary rules, and only when it was about some large blocks(which were exceeding the so configured threshold), it was trying to send alerts which from times to times may break, due to a plethora of reasons [1]. In result the function call self.makeTransferRucio() was never able to succeed and the call for the given DID was retried every  MSTransferor polling cycle with one and the same RSE list, while the workflow was sitting in assigned. In general Rucio is able to protect us from situation like this because it does not recreate already existing rules. A side effect of such behavior, though, is that in case any of the RSEs which are to be used for these rule submissions gets out of quota (i.e. for a reason not related to our broken workflow) MSTransferor drops it from the list of RSEs for this DID and in result the new rule to be created is not considered as duplicate and then Rucio creates a brand new rule, which leads to the over replication.
The current PR aims to enclose the the call to the alertManager in order to avoid situation as the one explained above.
[1]
https://github.com/dmwm/WMCore/blob/5da7f3c898bb39cdb7efb64070af182419501602/src/python/WMCore/MicroService/MSTransferor/MSTransferor.py#L727
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/83
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/84
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/85
and configuration for MSOutput
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/86
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/87
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/88
External dependencies / deployment changes
In order to fix the actual issue we also need to provide the proper PR in the gitlab K8 configuration repository for adding the correct AlertManagerUrl to the MSTransfferor configuration

Hi @amaltaro I am getting there (to the PR with the proper configuration changes) really soon ... The problem is I cannot manually generate an alert with the AlertManaterAPI and the correct url (which is now known). That's why I've put status "Need more testing". I suspect some more changes to the code may be needed here and there.

Hi @amaltaro,
Now we have the full picture. I just made the three PRs in the K8 service-config repository and please take a look at my second commit to the current PR which reflects  a change that we need to make in the current alertManagerAPI class in order to use the proper api version, as it is explained in the prometheus documentation  [1]. I decided to split the Url from the Uri and keep the  former in the service config, while the later to be in the code, but if you or @goughes  think otherwise, just let me know.
[1]
https://www.prometheus.io/docs/alerting/latest/clients/

@todor-ivanov these changes look good to me.
I have two questions though:
*) did you manage to run a basic test with these changes? If so, then I think we can run a final test in the production POD, and if things go through, then we make a new release and ask Imran to push that in as soon as images are available.
*) MSOutput also relies on AlertManager. Perhaps you could update those 3 merge requests in gitlab for ms-output as well? Creating new MR is fine as well.

Thanks @amaltaro ! Yes I am done with all the tests needed. Sorry for not reflecting this in the PR description.

Thanks Todor.


Fixes #16 - Add support for Travis CI linting

@tompscanlan No worries on the delay, Hope your holiday was good. Here is the PR.

No problem. It usually takes me several shots to get travis working the first time

@tompscanlan i have to change some of the modules around that are no longer supported in version 1.x to the 2.x modules. Do you want me to include that in this pr or open a separate one then make Travis work?

Whichever is most convenient.  I'm not too picky ;)


Doesn't it work with nuxt 2 and nuxt bridge?

The docs did mention that this package is nuxt 3 ready.
I believe it was not mentioned that it isn't compatible with nuxt 2 and bridge

May I clarify if this is intended for nuxt 3 only?
Thank you

It's hard to create a project compatible with Nuxt 2 and Nuxt 3, because they are very different from each other. Here you can preview if the module is compatible: https://modules.nuxtjs.org/?q=icon. If you need a solution for Nuxt 2 I recommend https://www.npmjs.com/package/@nuxtjs/svg-sprite.

For every module that has to be switched at the same time as upgrading to Nuxt 3, it magnifies the effort and risk needed to make that upgrade. I wish more attention was put into making the upgrade path a smooth one.


package garnet-server as a .NET tool

fixes https://github.com/microsoft/garnet/issues/774
preview (to check package details etc) is here - I've separately initiated a handover of that package to the Garnet account.
Usage:

   15:54:40  GarnetServer net-tool  3   8.0.300   153ms 
➜ dotnet tool install --global garnet-server --version 1.0.36-preview1
You can invoke the tool using the following command: garnet-server
Tool 'garnet-server' (version '1.0.36-preview1') was successfully installed.
   15:54:45  GarnetServer net-tool  3   8.0.300   2.637s 
➜ garnet-server
    _________
   /_||___||_\      Garnet 1.0.36 64 bit; standalone mode
   '. \   / .'      Port: 6379
     '.\ /.'        https://aka.ms/GetGarnet
       '.'

* Ready to accept connections

Here is where our CI pipeline does a NuGet pack of the .nuspec in order to create the nupkg, then sign it, and then push it to nuget.org --
https://github.com/microsoft/garnet/blob/main/.azure/pipelines/azure-pipelines-external-release.yml#L101
Push happens here: https://github.com/microsoft/garnet/blob/main/.azure/pipelines/azure-pipelines-external-release.yml#L191
Seems we need to add an additional step here to create this garnet-server tool. Assuming it generates a nupkg, the above pipeline should automatically sign and push to nuget.org, as it does this for all .nupkg files found in the build artifact directory.

Will merge this and we can make whatever changes we need to get this to push automatically, thanks!


Banner not visible

### Tasks

Please try clearing your browser cache and checking the banner in incognito mode (or on any other device/browser you might have). The banner is displayed correctly for me and other users, as demonstrated in the comment of the linked pull request.


Desktop screen share failing with TypeError: e.split is not a function

Hi,
I'm trying to get desktop screen sharing to work using Firefox 59 on Ubuntu/Linux 17.10 and OpenFire server 4.2.3, ofmeet 0.9.3. When I click on the screen share button, I get an red popup box saying Screen sharing extension failed to install and a few JS console errors:

[react/features/base/tracks/functions.js] <t.a/<>:  Failed to create local tracks Array [ "desktop" ] TypeError: e.split is not a function
[conference.js] <_handleScreenSharingError>:  failed to share local desktop TypeError: e.split is not a function
[JitsiMeetJS.js] :  UnhandledError: null Script: null Line: null Column: null StackTrace:  TypeError: e.split is not a function

I thought there was no need for an extension anymore using Firefox > 52, am I wrong ?
What could be the problem here ?
Thank you very much,
Kind Regards.
Stack traces here:
20180326-openfire-meetings-desktop-share.jsconsole.txt.gz

Well, this upstream issue seems related https://github.com/jitsi/jitsi-meet/issues/2350


[New game]: Bulls Eye Game [GSSoC'23]

🎮 Game Request
Game logic and basic description

This is multi player game
This is made by using HTML, CSS and JS languages
This is best suited to get the feel of archery
Just hit the target and get the points

Point down the features
Game points

If just hit the board, +1 point
If hit the bulls eye of the board, +1 ( +1 for bonus point )

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Avdhesh-Varshney , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#1572

Hey @Avdhesh-Varshney !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign


[Question] - How to handle extensibility of a model

This is just a general question about my use case, and EF7.
My application has an EF7 model and db context. I am calling this the "Core" Data Model.
My application allows third party "extensions" to be installed.
These extensions need to be able to "extend" the existing data model (tables etc) to do things like:

Add additional columns
Add foreign keys that reference existing core tables.

The issue I have, is how to allow a third party extension to do these things, without breaking the existing "Core" EF context that is in use.
Extending existing tables with Additional  Columns
Am i right in thinking the correct path for this is:

The developer would create a new DbContext.
Derive their own entity from the core entity.
Map their entity to the existing Table name.
Generate a migration - and make sure its empty (does nothing).
Add their additional properties, and generate a migration that adds those columns to the table.

The rule would be, never add a new [Required] property without a default value to an existing table. As that would break the existing DbContext. ?
Adding a Foreign Key that references an existing table.
If a developer wanted their new table to relate to an existing core table:

Create their own DbContext
Create their own Entity for the new table as usual.
What does their new entity look like in terms of adding a navigation property to the existing table (i.e the Foreign Key relationship?_
Does the developer need to Ignore() the navigation property (type) from their model? It will already exist in the Core model / DbContext.
How to avoid FK violation, when the existing core DbContext tries to delete an entity from its table, after the new FK is in place? Would the rule be, all new FK relationships have to be added with "Cascade on Delete"?

I'd love to head some feedback thoughts,, guidance on this topic!

The approach you described is one that we have seen folks use. The foreign key scenario is really the same as the first one, you just need to generate a migration (with all the base types in the model) before you create the new relationship. Cascade delete would work, or optional relationships with SetNull for the cascade behavior.


The documentation for prefix let repeats itself twice

The documentation for prefix let  repeats itself, and the same code example is shown twice.

That might be on purpose. It's documented twice: on Variables tutorial and on Operators tutorial
But when you look at the docs just for let it's showing all the docs it can find, which in this case are the two duplicated sections from two different tutorials.

Still, as #1924 says, it's probably better to rephrase one of them. The origin of some docs is copypasta, but that does not need to show.


Symlinks for textures

Contact Details (Optional)
Frisser on the server.
What is the context of the feature/improvement?
I like to split my ingested files into folders depending on what level in game and if its structural or a prop. This means I don't have to scroll through tons of files to find something so its much easier. The base game is quite large at 5gb so lots to replace and I use the same textures repeatedly.
Please describe the feature/improvement?
If instead of packaging a mod with the same texture multiple times, you could create a symlink to one of the textures with the same hash, it would save a lot of space.
What would be your solution? (optional)
No response
Version
2024.4.1
Media
No response

Hi @Frissj, thanks for filing. I've got a few questions to clarify here:
Are you running into problems when using symlinks to organize your textures? Or is this a request to provide a symlink option somewhere?

@nxkb Hi. As I understand, because I ingest files to different folders depending on the layer, textures when packaging the mod are duplicated as its copies the absolute file path? I like to ingest to different folders to make it a lot quicker to select the replacement and because it allows me to use the same name if the replacement is on different layers which avoids complicated file names. Anyway, instead of copying the texture twice, if it has the same hash as a previous texture, could a symlink be created instead so the mod file size is not twice as large as it needs to be?

Thanks for filing @Frissj
We've created an internal ticket for tracking: REMIX-3589


Different number system issue

Describe the bug

Calcultions should not be performed on two different number systems simultaneously else it would result in errorneous result and the user should also be prompt in case of such error

Possible solution

Solution would be to take the input over a single number system that would be selected by the user once before the calculation. By default decimal number system would be considered.

Screenshots

Kindly assign this issue to me @sairish2001

Sir @sairish2001,
Kindly assign this issue to me.
Thankyou

Go ahead
Deadline 9 April,2021 11:59 PM IST

is this issue still open? can i work on this issue?
as deadline is over


"Goto Usage"

In a similar vein as #9, would like to be able to quickly determine if, how much, and maybe even where/jump to for sub/functions.

This finds all the references in the current file.  Still need to add support for include files.


Master

No changes... Why are you submitting this pull request?

Nyr, can I get the ability to close requests that keep coming up? I've
learned most of your reasoning, and could help if you want
On Mar 13, 2017 2:28 PM, "Nyr"<EMAIL_ADDRESS>wrote:

Closed #274 https://github.com/Nyr/openvpn-install/pull/274.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Nyr/openvpn-install/pull/274#event-998051822, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGcIFjmdknvpzVvGm5a-Jh8ByAH0Hd8oks5rlbTvgaJpZM4Mb0I9
.

@lukebarone you are always helpful and I really appreciate that. Unfortunately the free version of GitHub doesn't provide a way to set granular permissions for collaborators: you either have zero permissions or you have full control over the repo. It's a shame.
Anyway, @1511dchan was banned from the repository after this third pull request, since communication seemed to be impossible, probably due to the language barrier.


MM 20404 Migrate 'components/announcement_bar/version_bar' module and associated tests to TypeScript

Summary
Migrate 'components/announcement_bar/version_bar' module and associated tests to TypeScript.
I am a beginner, not sure I did it correctly, please let me know where I did it wrong.
I can't find where is "version_bar" in my browser to check is my code actually working.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/16167
JIRA https://mattermost.atlassian.net/browse/MM-20404

Hello @tianlangwu,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Hi, thanks for the advice! I have fixed the three problems you mentioned. Could you tell me where is "version_bar" in the web app preview?

@tianlangwu The version bar appears when a new version of the server is detected. I think you can make it appear by updating your sever to a new version without refreshing the browser.

/update-branch

/update-branch


A feature that automatically updates Psych Engine when a new version is released.

What feature do you want to get added on the base engine?
There should be a feature that automatically updates Psych Engine when a new version of it is released so that you don't have to install it from the Gamebanana or GitHub page.
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

Unless HaxeFlixel can download files from web sources and we just never knew about it or something, I'm pretty sure this can't be implemented

Wait, maybe it should be something that downloads the latest version itself.

this mod has a cool way to update psych engine
https://gamebanana.com/mods/471076

If it helps, I was working on a mod manager built in Haxe that could successfully download and install mods
Here is the code for the downloader portion
It can be used like this:
    downloader = new Downloader();
    downloader.addEventListener(DownloadEvent.DOWNLOAD_COMPLETE,downloadComplete); // Add downloadComplete

    var file:File = new File(destination); // Replace destination with where you want the file to download to
    downloader.download(url,file); // Replace url

The code for the downloader can definitely be improved, but for now it works
Haxe also has built-in functions to extract .zip files, here is a working example for doing so.
I might end up making a PR of this at some point, but at the moment I don't have time.

I'm gonna stop updating Psych Engine soon, so it is too late to implement this in my opinion.

Wait a minute, Psych Engine is being discontinued soon?


[CMT-536-842] Bug: fix what is wrong with french article half showing

What happened 🆘

A french article is showing up on the front page when it (probably) shouldn't be showing.
To Reproduce 🔂
see at french.blog.yearn.finance
More 🔎

Browser : vivaldi

I would like to take this.


HyperFastCGI threading/IO configuration vs. ASP.NET configuration.

Hi,
we're having  an ASP.NET web service with ServiceStack which itself calls multiple other HTTP web services at self - around 3 HTTP calls to an external service per request to our service.
We're suffering from very bad performance there. The called external webservice should be able to handle 100k requests per second.
We cannot send more than around 5 per second, altough we're having 1000 requests per second to this service as well.
So it seems we're encountering deadlocks and full queues.
I found this: https://support.microsoft.com/en-us/kb/821268
It seems like this is exactly the symptoms we're having.
The question now is:
The HFC configuration let's us define all of these parameters: <threads min-worker="40" max-worker="0" min-io="4" max-io="0" />.
Is this related to the configuration recommendation Microsoft is making on their page?
Or is this totally distinct and am I supposed to add maxWorkerThreads, maxIoThreads, minFreeThreads, minLocalRequestFreeThreads etc to our Web.config as well, because otherwise the (low) default values would be in affect for our web application?
Thanks!

ThreadPool config values just an example and you should tune these values according to your application usage. If it creates alot of parallel network requests you should increase number of IO threads. If application creates large number parallel threads via ThreadPool.QueueUserWorkitem you should increase number of worker threads.  Also you can set all these values to "0" to get default ThreadPool settings.
When the new request arrived ThreadPoolchecks if there is available thread in pool and if thread is not available not it tries to create new one. This is very costly operation and degrades performance alot. If number of created threads hit the max threads value then ThreadPool stops to create new Threads and wait when previous was completed. You should monitor you app to be sure that threads are not created on every request and total number of threads in normal conditions is unchanged (if you do no create threads directly with Thread.Start).
My experiments shows that default number of worker threads is very low for high-load application and should be increased. These values from sample hfc.config are good for 2-core processor without active ThreadPool usage in services. So if you have more cores or actively use external calls you are definitely have to change these values to more large values.
Settings
<threads min-worker="40" max-worker="0" min-io="4" max-io="0" />
points to minWorkerThreads, maxWorkerThreads, minIOThreads and maxIOThreads settings in ThreadPool.
Finally, you should monitor threads numbers (via counters for example) in your application and find the best settings according to your usage.
Also it could be the situation when application does not return threads to the threadpool, if you have a simple example which shows the issue I will look in it.


Consider to allow non-inventory dirs

I don't have a concrete usecase, but I stumbled upon this while addressing #519 and checking recursive directory removal:
We could allow for plain dirs (no .anchor) that therefore are not inventory-tracked items. This would be valid if they only contained .onyoignore'd files, so: no assets or inventory dirs underneath.
Low priority, though, just something to consider when we have some experience with .onyoignore usecases to begin with.

Good news, this was already implemented at some point since the issue was opened! We currently have bin/ without an .anchor; the dir is listed in .onyoignore, and onyo continues to work just fine.
I will thereby close this issue now, feel free to reopen it in case I missed something :)


addon-a11y — Not working with config withInfo({ inline: false })

Our team recently discovered that that you can only get accessibility testing working if withInfo has the option { inline: true }
I would like to either:

Add a note of this on the readme.md for the plugin, or
Write a fix to make this work with { inline: false }

Adding this so there is a location to add my pull request when it is ready, and to get any correction or feedback in case I am approaching this incorrectly.
Please specify which version of Storybook and optionally any affected addons that you're running
{
    "@storybook/addon-a11y": "^4.1.13",
    "@storybook/react": "^4.1.13",
}

Upgrade to 5.0.x/5.1.x-alpha if you can. There have been a lot of good a11y improvements recently.

Hoping to replace info with Docs which won't have this issue: https://medium.com/storybookjs/storybook-docs-sneak-peak-5be78445094a


update composer versions across all packages

To deal with issue #80

updated all composer versions across all packages, bumping to ^0.11.0

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


Send Money - "Looks like you're offline" message displayed on top of the Send button

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify.com
Log in with expensifail account
Click on Fub menu and Select Send money
Put amount and select Continue
Turn off the internet connection

Expected Result:
"Looks like you're offline" message displayed under Send button
Actual Result:
"Looks like you're offline" message displayed on top of the Send button
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

I don't think this is a bug. It is probably intentional and if we move the offline text below the button the position of the button will change.
cc: @Luke9389  @roryabraham

This should be on hold until we touch offline functionality like other issues.

Holding on publishing this until I get clarification on the above.


Can't seem to control guest access

I have set this all up and it seems to be working, except for controlling guest access.
I have sensitivity labels that prevent sharing, but when users create Teams/Groups with these labels, they can still share the team/group externally. Are there additional settings I need to change within Office 365 Groups? Or is this something that hasn't fully been implemented yet.
Sorry if I have missed this anywhere in the documentation!

I think I opened this in the wrong location. Sorry all!


Unify DNS for local dev

** Experimental! **
This is a draft PR to get some feedback from the team about usability.
We've been having some trouble getting playwright to work properly when run against local dev due to some inconsistencies in the way DNS works inside a docker network, and on our laptops.
When containers connect to each other, they use service names defined in docker-compose.yml (for example, "backend", and "database"), but when we connect to the containers from our laptops, we use the 'localhost' name with different ports for each container. This means that OAuth redirects inside the docker network need to be different than redirects for browsers running on our laptops.
This PR makes the container names consistent inside and outside of the docker network by adding /etc/hosts entries to our systems to create aliases for {service_name}.corporanet.local, and using docker network aliases inside the docker network with the same name.
After pulling this branch down, engineers will need to re-run make local-init to apply all the necessary changes to their environment.
The corporanet.local suffix allows us to use a Samesite=Lax policy for the session cookie. Without domain suffixes, browsers don't allow sending cookies to the backend on redirects.
Reviewers
Functional:
Readability:

Changes

add
remove
modify

Definition of Done (from ticket)
QA steps (optional)

Hi @jgadling ! Just wondering when this PR will be ready to merge 😄 Thank you so much again for the fix!

Hi @jgadling ! Just wondering when this PR will be ready to merge 😄 Thank you so much again for the fix!

I just haven't had a chance to sort out the test failures, but it's otherwise good to go. I probably won't have time for that until tomorrow. If someone else has the bandwith, go for it.

Ooh perfect! OK I'll take a look at the failed tests now. Thank you!

@Bento007 not urgent, but if you have time could you help take a quick look at why the unit test on session cookie didn't get Secure for cookie even though, True seems to be passed? https://github.com/chanzuckerberg/corpora-data-portal/pull/1001/files#diff-63893bfe0ed0a90f3c17e64ae8d17d4a41894ecb4a296f9146fa94b24a053d14R61
Thank you!

Hi @jgadling ! Quick update: @Bento007 and I chatted about the failing unit test.
It seems like the reason it's failing is because unit tests run inside the containers, which always has DEV_MODE_COOKIES=1, thus require_secure_cookies = False, but the session cookie unit test still expects Secure, thus failing.
So we're wondering if it's easy to run unit tests on HTTPS, using the already generated cert? Because if that works, then we can remove the code:
if os.getenv("DEV_MODE_COOKIES"):
            require_secure_cookies = False

?
Context: https://chanzuckerbergteam.slack.com/archives/C01CPJURT0S/p1616699680039000

So we're wondering if it's easy to run unit tests on HTTPS, using the already generated cert? Because if that works, then we can remove the code:

I think there might be a simpler option ... PR in a sec.

LGTM!! Thanks so much for all the fixes, @jgadling 🤩 👏 ✨ !
Love the solution to reset DEV_MODE_COOKIES to undefined in test command, so we don't need to alter the build env variable config 💡

I think we need a better solution soon... but this should unstick us for the moment :'(


overriding styles

Hi,
this is probably a newb/general hugo question but I can't seem to override the stylesheets.
in my site root i made an /src/scss/chunky-poster.scss
also tried /static/src/scss/chunky-poster.scss
...but no joy. Can't even change the background colour!
Do I need to build out the scss files first or is that handled by Hugo?
Cheers,
Pete.

Yeah, sorry, I haven't added support for custom styles yet. You can do it like https://github.com/puresyntax71/hugo-theme-chunky-poster/issues/32#issuecomment-572669559. Will try to add support for it this week.
That is for CSS though. If you want SCSS and not fork the project (for recompiling), you can use Hugo pipelines instead. This seems easiest.

ok got it :)  (so not insane after all)
Cool, this is day 1 of hugo for me today so this might be a project for another time... but with pipelines that would be modifying the stylesheet links in the header to do the conversion on the fly right?

Yeah, that is right. It would build the SCSS/SASS files for you and include them in the head as long as you've followed the documentation from the link.

Got it at #68 . Just copied over the feature from the Event theme.


getting webpack-dev-server warning on resqInjector.js attaching the error.

Compiled with problems:X
WARNING in ../../node_modules/cypress-react-selector/src/resqInjector.js 12:15-22
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
I am stuck here and i see a fix for this in webpack-dev-server but however, along with cypress/webpack-dev-server it didnt work. Kindly help with here.  Attached screenshot of the overlay which blocks the cypress event handlers. and after click on X mark, we can see the actual screen.
cypress-error.pdf

could it be that webpack-dev-server is displaying an error?
https://webpack.js.org/configuration/dev-server/#overlay

module.exports = {
  //...
  devServer: {
    client: {
      overlay: {
        errors: true,
        warnings: false,
      },
    },
  },
};

Applying this configuration to a webpack build will remove the warning from being displayed.
but it will not mask the warning from cli:
WARNING in ./node_modules/cypress-react-selector/src/resqInjector.js 16:13-20
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted


Slack bot should send scheduled summary messages of requests that are dragging

The #community-needs channel can get overwhelming, so we can teach the bot how to post a scheduled summary message to bump the requests that are dragging.
BSS does this and it rules, so we can ape their design (big ups @maybejustmaybe & @cubeghost)

implemented here: https://github.com/bedstuystrong/firebase-functions/blob/899d2e15c0bf651c02702adb9ae983e25f517997/functions/messages.js#L143

It should include any requests in the "Delivery Needed" status that are not assigned to a cluster delivery already.
Since our data model doesn't emphasize priority we can use the time since it was first posted to prioritize requests.[1]
Maybe as a reach the bot can pin the messages itself?

[1] Or should it use the time since the request was created in the first place?

Ugh oh no I duplicated https://github.com/crownheightsaid/mutual-aid-app/issues/63...
Closing this one


problem parsing multi-line definition

Most VCD files I have, use a multi-line definition for each timestamp:
#1
b00 !
b1100 "
#2
b11 !
b1110 "

You mentioned PyVCD, the example of that VCD writer also generates that kind of format.

When looking into your code, I notice that you only parse for $command and #time_value.
The third option would be a line not starting with $ or #. e.g. to call a new function process_signal_change() in that case. You would also have to remember the last valid time definition to have it available for that function.
process_time_value has to learn parsing time definitions without signal change.
In case there is also a signal change (like in your test1.vcd) you could call the new function on the rest of the line.

@EtlamGit i changed the code.
It should work for you.
just try and tell me the result.
BTW:  you should download the code from github. I have not updated the pypi.

Great, very fast reaction.
Now it is working better, but fails in your new check routine in line 124.
My VCD files use a syntax for single bit signals to omit the space:
#1
b011011010100 c
b011101010101 f
1j
0k
0o
1p
#2
b011010100010 c
b011101110001 f
0l
1o

so there is the space you expect in case of wider signals, but it is missing for 1-bit-wide signals.
(I always use the code directly from here, installation via pip and package seems to be broken currently. I will open another issue or even pull request when I'm able to figure out why this is the case.)


[main] Update actions

Cron -knative-prow-robot
/cc we don't use Prow
/assign we don't use Prow
Produced by: knative-sandbox/knobots/actions/update-actions
Details:

Codecov Report

Merging #255 (b2040e5) into main (a37c1dd) will increase coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
+ Coverage   80.39%   80.65%   +0.26%
==========================================
  Files          20       20
  Lines         765      765
==========================================
+ Hits          615      617       +2
+ Misses        136      134       -2
  Partials       14       14

Impacted Files
Coverage Δ

pkg/vsphere/adapter.go
64.34% <0.00%> (+1.55%)
:arrow_up:

Continue to review full report at Codecov.

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