Fix license checker

Remove wrongly formatted and unneeded yaml file from exclusion list (yaml files are excluded anyway)
Slightly improve the recognition and normalization of line comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 41.667%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30

Relevant Lines:
72

💛  - Coveralls


Create Is_prime.cpp

Please accept my request I am a beginner

@Tirtharaj-Talukdar Please open an issue first addressing the work you are going to do

@Tirtharaj-Talukdar Please open an issue first addressing the work you are going to do

"invalid" label will be removed after proper issue followed by PR is made

Kindly perform this algorithm using C, Python and Java

@Tirtharaj-Talukdar Kindly perform this algorithm using C, Python and Java to validate this PR

Heyy I don't know any other language I can only write cpp

I can try to add Modular features but I can write only in cPP

@Tirtharaj-Talukdar just give a try

@Tirtharaj-Talukdar just give a try

Other languages aren't hard enough


UnknownOperationException

I have this small project where I am trying to send some text to Claude. No matter what I do I can't seem to get Claude-v2:1:200k to analyze it. Here is my approach using typescript:

I have installed @anthropic-ai/bedrock-sdk with npm
using ~/.aws/configure for auth
when I run my project with npm run dev :
const anthropic = new AnthropicBedrock();
const data =  await anthropic.messages.create({
model: "arn:aws:bedrock:region:eu-west-1:custom-model/anthropic.claude-v2:1:200k",
max_tokens: 1024,
temperature: 1,
messages: [{"role": "user", "content": "Hello, world! "}]
});
I get this message:
{
Output: { __type: 'com.amazon.coral.service#UnknownOperationException' },
Version: '1.0'
}
There are not so many leads out there that can point me to the right direction, am I missing something ?

Hi @daniel-gofore,
Could you try to run the demo example and tell me if you see the same problem?
https://github.com/anthropics/anthropic-sdk-typescript/blob/main/packages/bedrock-sdk/examples/demo.ts

Hi @dgellow, I have tried the demo one and I get the same result.  These are the models that I have access with AWS on my account:

When I am quering the aws cli for models to see which models I have access to, I get this list : aws bedrock list-foundation-models --region=eu-west-1 --by-provider anthropic --query "modelSummaries[*].modelId"
[
"anthropic.claude-3-sonnet-20240229-v1:0:28k",
"anthropic.claude-3-sonnet-20240229-v1:0",
"anthropic.claude-3-haiku-20240307-v1:0:48k",
"anthropic.claude-3-haiku-20240307-v1:0"
]
When sending the request to Anthropic:
For the models that I don't have access to I get this response:
error: {
message: "You don't have access to the model with the specified model ID."
}
For the models that I have access to but I don't use the specific format like: model: "arn:aws:bedrock:region:eu-west-1:custom-model/anthropic.claude-v2:1:200k" , I get this response:
message: "1 validation error detected: Value 'anthropic.claude-3-sonnet-20240229-v1:0:28k' at 'modelId' failed to satisfy constraint: Member must satisfy regular expression pattern: (arn:aws(-[^:]+)?:bedrock:[a-z0-9-]{1,20}:(([0-9]{12}:custom-model/[a-z0-9-]{1,63}[.]{1}[a-z0-9-]{1,63}/[a-z0-9]{12})|(:foundation-model/[a-z0-9-]{1,63}[.]{1}[a-z0-9-]{1,63}([.:]?[a-z0-9-]{1,63}))|([0-9]{12}:imported-model/[a-z0-9]{12})|([0-9]{12}:provisioned-model/[a-z0-9]{12})))|([a-z0-9-]{1,63}[.]{1}[a-z0-9-]{1,63}([.:]?[a-z0-9-]{1,63}))|(([0-9a-zA-Z][_-]?)+)|(arn:aws(|-us-gov|-cn|-iso|-iso-b):bedrock:(|[0-9a-z-]{1,20}):(|[0-9]{12}):model-gateway/[a-zA-Z0-9-]+)"
}
For the models that I have access to and use the correct format, I am getting :
{
Output: { __type: 'com.amazon.coral.service#UnknownOperationException' },
Version: '1.0'
}

Thank you for the detail, we are looking into it

Hi, any updates on this ? Thank you.

Never mind I have fix it. The isue is that this documentation is wrong. This is the code I have used to get things working, so configure your aws cli with the correct details in order to access the your AWS account, check which AWS Anthropic services you have enabled and drop this code, make sure you have the right nameing of your model:
`import {
BedrockRuntimeClient,
InvokeModelCommand,
} from "@aws-sdk/client-bedrock-runtime";
import { NextApiRequest, NextApiResponse } from "next";
/**

@typedef {Object} ResponseContent
@property {string} text
*/

/**

@typedef {Object} MessagesResponseBody
@property {ResponseContent[]} content
*/

/**

@typedef {Object} TextCompletionsResponseBody
@property {completion} text
*/

/**

Invokes Anthropic Claude 2.x using the Messages API.

To learn more about the Anthropic Messages API, go to:
https://docs.aws.amazon.com/bedrock/latest/userguide/model-parameters-anthropic-claude-messages.html

@param {string} prompt - The input text prompt for the model to complete.
@param {string} [modelId] - The ID of the model to use. Defaults to "anthropic.claude-v2".
*/

export default async function handler(req: NextApiRequest, res: NextApiResponse) {
if (req.method === 'GET') {
try {
const response = await invokeTextCompletionsApi('Hello', 'anthropic.claude-v2');
res.status(200).json(response);
}catch (error) {
console.error('Error handling request:', error);
res.status(500).json({ error: 'Failed to process request' });
}
} else {
res.setHeader('Allow', ['POST']);
res.status(405).end(Method ${req.method} Not Allowed);
}
}
/**

Invokes Anthropic Claude 2.x using the Text Completions API.

To learn more about the Anthropic Text Completions API, go to:
https://docs.aws.amazon.com/bedrock/latest/userguide/model-parameters-anthropic-claude-text-completion.html

@param {string} prompt - The input text prompt for the model to complete.
@param {string} [modelId] - The ID of the model to use. Defaults to "anthropic.claude-v2".
*/
export const invokeTextCompletionsApi = async (
prompt: string,
modelId = "anthropic.claude-v2",
) => {
// Create a new Bedrock Runtime client instance.
const client = new BedrockRuntimeClient({ region: "eu-central-1" });

// Prepare the payload for the Text Completions API, using the required prompt template.
const enclosedPrompt = `Human: ${prompt}\n\nAssistant:`;
const payload = {
  prompt: enclosedPrompt,
  max_tokens_to_sample: 500,
  temperature: 0.5,
  stop_sequences: ["\n\nHuman:"],
};

// Invoke Claude with the payload and wait for the response.
const command = new InvokeModelCommand({
  contentType: "application/json",
  body: JSON.stringify(payload),
  modelId,
});
const apiResponse = await client.send(command);

// Decode and return the response.
const decoded = new TextDecoder().decode(apiResponse.body);
/** @type {TextCompletionsResponseBody} */
const responseBody = JSON.parse(decoded);
return responseBody.completion;

};`

Glad you were able to figure this out!


Implemented #107 - projects per org.

Resolved #107
Ready for 1.1.7
Used the temporary get_big endpoint to

small comment on hardcoded value but otherwise LGTM!


Error running a subcommand of lib/provision.py: sudo scripts/lib/install-node

1)host operating system:
Microsoft Windows 7 Professional
Version=Service Pack 1
2)installation method
Vagrant
3) Log: http://dropmefiles.com/h265I
Good day.
During the installation of zulip with the help of a Vagrant, an error occurs: Error running a subcommand of lib/provision.py: sudo scripts/lib/install-node
I ask to help with elimination. The log file provision is accessible by the link above.

This was fixed in f724900e68d4eb75c93310a8e9cd4b55b9cffff5
Next time, @NikishinRoman, please don't report the same issue in 3 places; pick one of chat.zulip.org, the bug tracker, and the email list :).

Hello
I updated the code, but the error still remains.
provision.txt

can you add set -x to the top of scripts/lib/install-node and try running it again?  That'll make this easy to debug.  You don't need to upload the whole file, just the part starting from + sudo scripts/lib/install-node; I think that should be short enough to make it easy to see where it's failing.

I add set -x to the top of scripts/lib/install-node.
The log file provision is accessible by the link
http://dropmefiles.com/G0vIV

Something isn't right with that log file; it doesn't seem to match the actual code of scripts/lib/install-node in the version of master from 3 days ago.  Are you sure you pulled the new version?  Or is it possible you replaced set -e with set -x rather than just adding set -x?

Good day. Indeed, I replaced set -e with set -x.
Now I did everything right. Here is my log
http://dropmefiles.com/sBJ3Y

OK, this is the (surprisingly interesting) problem:
export NVM_DIR="/usr/local/nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm
+ source /usr/local/nvm/nvm.sh
scripts/lib/install-node: 28: scripts/lib/install-node: source: not found

Error running a subcommand of lib/provision.py: sudo scripts/lib/install-node
Actual error output for the subcommand is just above this.

The source command is a shell builtin that comes as part of bash.  You don't get a similar error when you run source with broken arguments, e.g.:
(zulip-py3-venv) tabbott@zaset:~/zulip$ source /tmp/blah
bash: /tmp/blah: No such file or directory
(zulip-py3-venv) tabbott@zaset:~/zulip$ source ""
bash: : No such file or directory

So I'm tempted to believe that something is very wrong with your system, and /usr/bin/env bash is not a normal version of bash.  Do you have other modifications to the tools?

I did it in the environment of a vagrant.
Vagrant v1.86
VirtualBox v 5.1.22 r115126

OK; can you vagrant destroy; git fetch upstream; git rebase upstream/master; vagrant up to try recreating it?


Update MET to compile against Python 3.6 and coordinate with METplus 2 to 3 transition.

[MET-1080] created by johnhg

Split 2702691 and 2780541

All 8.1 python functionality has been moved from python 2 to python 3.


Android Basics: ViewModel

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-viewmodel?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-viewmodel#4
In which task and step of the codelab can this issue be found?
Task 2, Step 5
Describe the problem
the example code throws the error in Android Studio
Steps to reproduce?

Go to...Task 2, step 5, Add backing property to currentScrambledWord
Click on...
See error...private fun updateNextWordOnScreen() {
binding.textViewUnscrambledWord.text = viewModel.currentScrambledWord
}
when it should be:
private fun updateNextWordOnScreen() {
binding.textViewUnscrambledWord.text = gameViewModel.currentScrambledWord

since the instance declared above is precisely of GameViewModel and not of ViewModel
Versions
Android Studio version: 4.1.2
API version of the emulator: Physical Device
Additional information
Include screenshots if they would be useful in clarifying the problem.

Thank you for reaching out to us.
The instance name in the codelab is viewModel not gameViewModel.

You are facing this issue because you might have refactored in your code. Please check again from your end.
No need to update codelab content.


Fixed issue upload large file

I found the bug. You can't upload InputStream because big files seems to plant . You need to use a File type on your files.
I added the class upload(File content), and you need to add the last version of httpclient and httpcore (4.4) , and add httpmime to your pom.xml :

org.apache.httpcomponents
httpmime
4.4

I didn't delete upload(InpuStream content) but she is useless now .

And what was the real cause of the problem?
Is this a bug in apache commons? In redmine implementation? In some http server or proxy? In [anti]virus software on the machine?
Is this an issue with simple vs multipart post? Or is this an issue with Content-Lenght header? Resolutions could be different. Maybe creating multipart request with stream source is a better solution?
I am very suspicious about Content-Length header (and other fixed lenghts in the requests). Usually, reading file length and then sending a file is not an atomic operation. So file could be changed after its length was read. How the guilty component (if any) will behave if Content-Lenght is different from an actual conetnt length?

Content-Lenght is automaticaly set , i didn't do changes about this. But multipart-request doesn't works.
Only application/octet-stream works with REST Redmine api ( see doc on redmine website ) .
I have found this solution because it works with my utilisation, but if you found a general utilisation better than this, you can ! :-)

First of all, there's no pom.xml here - gradle is used to build the project, not maven (makes me wonder where have you found POM file here).
Then, updating httpclient/httpcore to 4.4 breaks the build for me:
D:\projects\git\redmine-java-api-keritaf\src\main\java\com\taskadapter\redmineapi\internal\Transport.java:592: error: exception UnsupportedEncodingException is never thrown in body of corresponding try statement
                } catch (UnsupportedEncodingException e) {
                  ^

UnsupportedCharsetException is now thrown instead. And there are lots of deprecations between 4.2 and 4.4 so I propose a migration to new API.

And could you please explain what is actually broken here?

I appreciate when people help out with this library. but to keep the quality on a certain level we cannot accept pull requests that do not have tests  or detailed explanation of what is wrong and how the fix will address that.
switching to newer Httpclient library (4.5.1) is not an option yet - see problems with pools in https://github.com/taskadapter/redmine-java-api/issues/212


Add PackageFolder support as project property

Scenario: analyzers\cs (for roslyn source generators, for example).
This would mean it's a custom (non-TFM specific) folder/PackageItemKind.
In this sense, PackageItemKind is basically a shorthand for PackageFolder+FrameworkSpecific really.
I don't think we need PackageFolder + FrameworkSpecific project property though. The framework-specific folders are well-known and built-in, so I don't foresee a need for that combination.

This is already supported if you use BuildOutputKind=analyzers\cs (or any string really). Any kind not built-in is considered a custom one and included as-is in the package.


Add sanitizing html content

Resolves #108

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.76%. Comparing base (26313fa) to head (33e40ca).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #110   +/-   ##
=======================================
  Coverage   99.76%   99.76%
=======================================
  Files         160      160
  Lines        2512     2513    +1
  Branches      571      568    -3
=======================================
+ Hits         2506     2507    +1
  Misses          6        6

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


Default user icon should show an anonymous image

Type of issue

[x] Bug
[ ] New feature
[ ] Improvement

Description
See picture

Fixed


CAMEL-19921 - Update default values of kafka client configuration

Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test camel-kafka

@davsclaus I guess we should backport to 4.0.x and probably on 3.x


Add a cursor when we hover on an evaluation result cell

TODO: add more details

fixed in previous issue


Add clickable table of content to "thenewboston blockchain format" to simplify navigation

Here https://github.com/thenewboston-developers/thenewboston-node/wiki/thenewboston-blockchain-format
Be aware the wiki is autogenerate therefore the changes should be made to the template.

@dmugtasimov done, please review.
Will that work If I'll link PR's to issues like that?

@fonar yes

@fonar thank you, merged


Mismatch of dimension in the CNNpolicy

I have a gym env with the obs space Box(8,). When I run PPO with cnn, I get the error as below:
TypeError: Tensor objects are only iterable when eager execution is enabled. To iterate over this tensor use tf.map_fn.
Eventually I find the error is caused by the line 33 in the code common/input.py:
processed_observations = ((processed_observations - ob_space.low) / (ob_space.high - ob_space.low))
The tensor "processed_observations" has shape (, 8) but the pandas variable ob_space.low has shape (8,). The dimensions do not match.
I fix this error by adding a line: ob_space.low.values.reshape(-1, 8). It works. Maybe you have better solution.

Hello,
please fill in the issue template completely.

I have a customised gym env with the obs space Box(8,)
action_space = spaces.Discrete( 3 )
observation_space= spaces.Box( self.src.min_values, self.src.max_values)
When I run PPO with cnn, I get the error as below:
TypeError: Tensor objects are only iterable when eager execution is enabled. To iterate over this tensor use tf.map_fn.
Eventually I find the error is caused by the line 33 in the code common/input.py:
processed_observations = ((processed_observations - ob_space.low) / (ob_space.high - ob_space.low))
The tensor "processed_observations" has shape (, 8) but the pandas variable ob_space.low has shape (8,). The dimensions do not match.
I fix this error by adding a line: ob_space.low.values.reshape(-1, 8). It works. Maybe you have better solution.
Code example
train_env = DummyVecEnv(
[lambda: gym.make('trading-v0')])
agent= PPO2(CnnPolicy, train_env, verbose=1, tensorboard_log="./tensorboard/ppo/")
agent.learn(total_timesteps=20)
Error messages and stack traces are also helpful.
File "C:/Data/Dockers/main_ppo.py", line 119, in
session(config = config, args = args)
File "C:/Data/Dockers/main_ppo.py", line 98, in session
agent= PPO2(CnnPolicy, train_env, verbose=1, tensorboard_log="./tensorboard/ppo/")
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\ppo2\ppo2.py", line 93, in init
self.setup_model()
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\ppo2\ppo2.py", line 126, in setup_model
n_batch_step, reuse=False, **self.policy_kwargs)
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\common\policies.py", line 478, in init
feature_extraction="cnn", **_kwargs)
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\common\policies.py", line 416, in init
scale=(feature_extraction == "cnn"))
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\common\policies.py", line 183, in init
scale=scale)
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\common\policies.py", line 114, in init
self.obs_ph, self.processed_obs = observation_input(ob_space, n_batch, scale=scale)
File "C:\ProgramData\Anaconda3\lib\site-packages\stable_baselines\common\input.py", line 33, in observation_input
processed_observations = ((processed_observations - ob_space.low) / (ob_space.high - ob_space.low))
File "C:\ProgramData\Anaconda3\lib\site-packages\pandas\core\ops.py", line 1585, in wrapper
index=left.index, name=res_name, dtype=None)
File "C:\ProgramData\Anaconda3\lib\site-packages\pandas\core\ops.py", line 1474, in _construct_result
out = left._constructor(result, index=index, dtype=dtype)
File "C:\ProgramData\Anaconda3\lib\site-packages\pandas\core\series.py", line 229, in init
data = list(data)
File "C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 459, in iter
"Tensor objects are only iterable when eager execution is "
TypeError: Tensor objects are only iterable when eager execution is enabled. To iterate over this tensor use tf.map_fn.
System Info

GPU models and configuration, No GPU
Python version: Python 3.6
Tensorflow version: 1.12.0

Why are you using a cnn with a 1d vector? You should use the mlp instead.
As stated in the documentation, the cnn policy is only made for images

I am working on converting the 1d vector to 3d, which is the same as images. My project is to compare the performance of 1d vs 3d. Anyway, I guess even if the input is 3d, the same error will occur. I am testing.

I think your problem may be similar to the one here: https://github.com/hill-a/stable-baselines/issues/259#issuecomment-485263269

Thanks Araffin. My issue is due to the customized gym. I have successful run on the CNN framework. This issue is solved.


QC: Add Unit Tests for new v2 functions

QC Details
Add unit tests for the following functions that are new to gsm v2:

[x] - Input_Rate()
[ ] - Report_KRI()
[x] - Report_Setup()
[x] - Report_MetricCharts()
[x] - Report_MetricTable()
[x] - Report_OverviewText()
[x] - Report_StudyInfo()
[x] - RunQuery()
[x] - RunStep()
[x] - RunWorkflow()
[ ] - Visualize_Metric()
[x] - Visualize_Scatter()
[x] - Visualize_Score()
[ ] - Visualize_QTL()
[x] - Widget_BarChart()
[x] - Widget_ScatterPlot()
[x] - Widget_SiteOverview()
[x] - Widget_TimeSeries()

Additional Comments
Use dev-v2 as the base branch for this issue

An update-
having some trouble with getting Report_KRI() tests to pass- some error with Report_StudyInfo that i wasn't getting last week, and a couple of the viz functions are taking me a while to wrap my head around, but even with those at 0% coverage, we are at package test coverage of 86.55%


Glitchy/unresponsive UI when resizing the window

Godot version
v3.5.beta.3.official
System information
macOS BigSur (11.16.2) - OpenGL ES 2.0 Renderer: Intel Iris Pro OpenGL Engine
Issue description
When resizing the window, everything gets glitchy/unresponsive.
Here's a video showing first 3.5.beta3 and then 3.4.4 (which works fine).
If it may be relevant, I was using @Wcubed's HFlowContainer, but now I've switched to the official HFlowContainer on 3.5. The issue is present using any of them.
https://user-images.githubusercontent.com/481475/161091200-8bfa772e-fbc4-4b19-8526-c10ba5faa2c2.mp4
Steps to reproduce
Open the project, run it and resize the window.
Minimal reproduction project
You can use the Lospec Palette List (develop branch to test 3.5.beta.3 and 1.x branch to test 3.4.4) as reference, which is what I show in the video.

I'll try to test it on my Ubuntu and Windows machines, but it seems likely that it could be related to the HiDPI resolution.

In Windows 10, they both look the same to me. I don't see the shaking behavior I observe in the OP video.
I'll mention that on Windows 10 black bars appear momentarily while resizing (in other words it takes a frame to draw the new areas). This is what Windows 10 does for both version, and for my projects too, even 3D.

I've tested on Windows 10 and it seems to work fine. It must a macOS thing :)

I'll mention that on Windows 10 black bars appear momentarily while resizing (in other words it takes a frame to draw the new areas). This is what Windows 10 does for both version, and for my projects too, even 3D.

It happens to me, too.

It seems like the issue is only present on macOs.
I've tried to reset to the defaults the only values I changed for this project, which are:

run/low_processor_mode=true (default false)
window/dpi/allow_hidpi=true (default false)
quality/driver/driver_name="GLES2" (default GLES3)

And the issue persists.

Do you get the issue on a 1080p monitor / if you switch your Retina display to 1080p (if that's possible)?

Yes, I've scaled my Retina display to 1920x1200
print(OS.get_screen_dpi())
# 295
print(OS.get_screen_scale())
# 1
print(OS.get_screen_size())
# (1920, 1200)

and the 1080p external monitor:
print(OS.get_screen_dpi())
# 83
print(OS.get_screen_scale())
# 1
print(OS.get_screen_size())
# (1920, 1080)

and I've enabled and disabled allow_hidpi and it stills happens.

I've removed my own Theme and the Container with the HFlowContainers and it still happens
https://user-images.githubusercontent.com/481475/161253301-5558ee63-3479-4196-a169-6a732b840d85.mp4

I think that what mac is doing is scaling the image. That is, when the window resized but the new area is not yet drawn, mac draws the old image but scaled. If it didn't scale it, it would be similar to Windows 10 black bars.

I think that what mac is doing is scaling the image. That is, when the window resized but the new area is not yet drawn, mac draws the old image but scaled. If it didn't scale it, it would be similar to Windows 10 black bars.

This is probably what it's doing. I don't think we can do anything about it.

@Calinou but this is something new in 3.5, it doesn't happened in 3.4

For the record, the issue is present on the 3 betas.

Just to discard everything, I've created a new project test godot.zip
3.3.4
https://user-images.githubusercontent.com/481475/161332498-32ee505a-90fb-4043-83f4-05878179d35e.mp4
3.5.beta.3
https://user-images.githubusercontent.com/481475/161332530-7d7e0530-45bc-4315-be03-f02356e67a35.mp4

@bruvzg

@hiulit Does it happen to the Godot window itself?

@theraot I didn't thought about that! And yeah, it happens to the Godot editor as well.
https://user-images.githubusercontent.com/481475/161351729-42843d01-cac7-42a0-840c-53e3d11a5f0f.mp4

While I can reproduce it, when resizing window fast, the same happens in 3.4.4 and any older version I have tested (at least up to 3.0). I can't see any difference.
It probably will need implementing a custom backing layer, and changing the way main loop works to fix it (it's a lot of changes, so probably unfeasible for 3.x).

This is definitely not happening on 3.4.4. You can see in my first video that even when resizing slowly it gets weird on 3.5.beta.3 and it doesn't happen on 3.4.4.
Here's more videos showing the differences:
Showly
3.4.4
https://user-images.githubusercontent.com/481475/161502803-32f7e96e-7b97-4ed6-823e-869d7583ae20.mp4
3.5.beta.3
https://user-images.githubusercontent.com/481475/161502816-0aca68f1-4b87-4c31-b307-4610e1bcedfb.mp4
Quickly
3.4.4
https://user-images.githubusercontent.com/481475/161502826-006beb08-7156-46dc-be53-3c4c0d1de0fc.mp4
3.5.beta.3
https://user-images.githubusercontent.com/481475/161502836-9c0be470-7655-42c2-90ff-1f0b7a261768.mp4

Ok, so I've compiled Godot 3.x branch reverting this particulpar commit https://github.com/godotengine/godot/commit/f997a5f8f6e51db47ca6ecfc11923847ed464094 and it seems like this is what is causing the issue.
Custom compile (3.x without the commit)
https://user-images.githubusercontent.com/481475/161532688-e891caba-49b0-4092-aaf1-3d935ccf7831.mp4
3.5.beta.3
https://user-images.githubusercontent.com/481475/161532716-52421a1b-a30d-4096-b466-1afd70f50b5d.mp4
I think we can safely say there's a big different between the two, right?
I admit that sometimes there are slightly rendering issues on 3.4.4, but they are negligible. I actually never noticed them until now, with all these tests, hehe!

I tested this on Windows 10 I don't see the issue. It has to be a mac specific issue. Very likely not the issue but just to be thorough does this still happen without any text at all? Text has always been buggy for me especially when using font oversampling true as it scales fonts at specific sizes pushing controls around.

I don't have the technical knowledge, but from an end-user point of view, it looks way worse now than before.
At some point, this was introduced https://github.com/godotengine/godot/commit/f997a5f8f6e51db47ca6ecfc11923847ed464094 (which seems to be the "issue"), so I'm guessing something can be done to, at least, find a middle ground.
In 3.4.4, for me, it was working ok. A little bit of lag when resizing the window? Sure, but it was minimal, never noticed before. But at least the content of the window wasn't going bonkers, hehe! Now we have a more responsive window when resizing but the content looks terrible. I don't think the content should be stretching when resizing. Does it look good to you?

@hiulit What are your Mac and GPU models exactly?

@akien-mga MacBook Pro (Retina, 15-inch, Mid 2015)- Intel Iris Pro 1536 MB running on  macOS BigSur (11.16.2)  and I'm using OpenGL ES 2.0 Renderer

For me, it's hard to reproduce, but I think I have found what's wrong, will try to fix it in a moment.

Fixed by #61895.
I assume the fix needs to be forward ported to master too?

why is this closed? it still happens
on both of my windows machines
Laptop with RTX2000 and RTX4090
Godot 4.2.2 / Godot 4.3
when resizing, the UI wobbles back and forth
Windows 10 / Windows 11

Why was this closed? Because this issue was for Godot 3.x, and as far as I can tell it is solved there.
With that said, now that you bring attention to it, I've compared Godot 3.x and Godot 4.x resize side by side, Godot 3.x is much more smooth. I don't think it is as pronounced as it was in Godot 3.x, at least it didn't bother me.
It seems the counterpart of the fix for the master branch didn't happen. Or at least I can't find it.
I notices something else while testing: If Godot 3 is in the background I can click on the border (which gives it focus) and start resizing right away. That works in 4.2.2 but not in 4.3. Godot 4.3 gets focus but I have to release and click again to resize. That seems to be a change on handling out of focus mouse input.

Please continue the discussion in https://github.com/godotengine/godot/issues/84972.

That does not seem right. Under the assumption that you linked the wrong issue, I searched for one, and this seem to be it:

https://github.com/godotengine/godot/issues/99072

This doesn't seem like it's the same, as the problem im facing
The videos above are exactly what I'm experiencing

@Osirisa I think we get out of this if you open a new issue.

Yeah ill do it later

I meant to link to https://github.com/godotengine/godot/issues/39222. Sorry for KDE's clipboard being this unreliable (it's been the case for years) 🙁


Colored version broken on Windows 7

So, last night I found this thing, installed it, and tried the /c switch. Well, its broken:

This is the non-colored version, not broken at all:

Update:
I installed Powershell5.1 and ran the PS version, but all i get is this:

You will need to enable ANSI support to see colors properly - https://github.com/kiedtl/winfetch/wiki/ANSI-Colors
Please search for similar issues before opening a new one.


os/mac/pkgconfig: merge pc files for 11.1 and 11.0

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?

Closes https://github.com/Homebrew/brew/issues/10126.
This combines the pc files for Big Sur 11.0 and 11.1. We no longer distinguish between minor versions of Big Sur in the CLT since https://github.com/Homebrew/brew/pull/10072.

Review period will end on 2020-12-25 at 07:35:58 UTC.

Thanks for this @carlocab!

Review period ended.

Thanks @carlocab!

No, thank you, @MikeMcQuaid. I've been a Homebrew user for such a long time it's a little shocking how long it took me to start contributing.

Thanks again @carlocab!


Capture SSL certs from HTTP Proxy directly

The current implementation of SSL certs capture:

Happens out-of-band, in a distinct capture step
Uses crip

While the current setup is extremely convenient, it would be preferable - both conceptually and from a performance perspective - to pull the certificates directly from the proxy, especially since Portal gives us that flexibility.

Here's the API: https://nodejs.org/api/tls.html#tlssocketgetpeercertificatedetailed
We'll end up with something in ScoopProxy, probably in this block, along the lines of
  onConnected (serverSocket, request) {
    const ip = serverSocket.remoteAddress
    const rule = this.findMatchingBlocklistRule(ip)
    if (rule) {
      serverSocket.destroy()
      this.blockRequest(request, ip, rule)
    } else {
      const exchange = this.exchanges.find(ex => ex.requestParsed === request)
      exchange.cert = response.socket.getPeerCertificate(true)
    }
}

Update: Getting close, but stuck on hard-to-troubleshoot edge cases (https://github.com/harvard-lil/scoop/pull/140#issuecomment-1503646180).
As this feature is not broken at the moment:

Setting PR as draft
Re-prioritization TBD


Use specific trigger lines to define Signal

This insulates simulation production from new triggers which are being added experimentally.

Hi @brownd1978,
You have proposed changes to files in these packages:

JobConfig

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for fe40df195dd36504213cc6bbe31dc59497a1a1f0: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at fe40df195dd36504213cc6bbe31dc59497a1a1f0.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged fe40df195dd36504213cc6bbe31dc59497a1a1f0 at 6c10cbd84d14dc976e0608032ce5bafcf0bc9e9b

build (prof)
:white_check_mark:
Log file. Build time: 10 min 54 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at fe40df195dd36504213cc6bbe31dc59497a1a1f0 after being merged into the base branch at 6c10cbd84d14dc976e0608032ce5bafcf0bc9e9b.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


W10 Post Install Fail - WinRM

Operating System Version: Server 2019
Provider (VirtualBox/VMWare): 6.1.4
Vagrant Version: 2.2.7
Packer Version: NA
Are you using stock boxes (downloaded) or were they built from scratch using Packer? Downloaded
Is the issue reproducible or intermittent? Repro

Please verify that you are building from an updated Master branch before filing an issue. Yes
I pasted the below into issue 21 that was closed, but not sure if that just disappears into a GitHub black hole :D, so made a new issue in case the root cause is different.
I think I might be having a similar issue to Issue 21, I have tripled the cfg.boottimeout and the cfg.winrm timeout for the w10 box, but still get the same issue I'm afraid.
VBox version - 6.1.4
Vagrant version - Vagrant 2.2.7
Host - Server 2k19 x64
DC host is all online and pingable. There is an entry for dc.windomain.local in the host file on w10.
<IP_ADDRESS> dc.windomain.local
Results from third reload / provision:
PS C:\DetectionLab\Vagrant> vagrant reload win10 --provision
==> win10: Attempting graceful shutdown of VM...
==> win10: Checking if box 'detectionlab/win10' version '1.4' is up to date...
==> win10: Clearing any previously set forwarded ports...
==> win10: Fixed port collision for 5985 => 55985. Now on port 2204.
==> win10: Fixed port collision for 5986 => 55986. Now on port 2205.
==> win10: Fixed port collision for 22 => 2222. Now on port 2206.
==> win10: Clearing any previously set network interfaces...
==> win10: Preparing network interfaces based on configuration...
    win10: Adapter 1: nat
    win10: Adapter 2: hostonly
==> win10: Forwarding ports...
    win10: 5985 (guest) => 2204 (host) (adapter 1)
    win10: 5986 (guest) => 2205 (host) (adapter 1)
    win10: 22 (guest) => 2206 (host) (adapter 1)
==> win10: Running 'pre-boot' VM customizations...
==> win10: Booting VM...
==> win10: Waiting for machine to boot. This may take a few minutes...
    win10: WinRM address: <IP_ADDRESS>:2204
    win10: WinRM username: vagrant
    win10: WinRM execution_time_limit: PT2H
    win10: WinRM transport: negotiate
==> win10: Machine booted and ready!
==> win10: Checking for guest additions in VM...
    win10: The guest additions on this VM do not match the installed version of
    win10: VirtualBox! In most cases this is fine, but in rare cases it can
    win10: prevent things such as shared folders from working properly. If you see
    win10: shared folder errors, please make sure the guest additions within the
    win10: virtual machine match the version of VirtualBox you have installed on
    win10: your host and reload your VM.
    win10:
    win10: Guest Additions Version: 6.0.10
    win10: VirtualBox Version: 6.1
==> win10: Setting hostname...
==> win10: Configuring and enabling network interfaces...
==> win10: Mounting shared folders...
    win10: /vagrant => C:/DetectionLab/Vagrant
==> win10: Running provisioner: shell...
    win10: Running: scripts/fix-second-network.ps1 as c:\tmp\vagrant-shell.ps1
    win10: Nothing to do for other providers than VMware.
==> win10: Running provisioner: shell...
    win10: Running: scripts/MakeWindows10GreatAgain.ps1 as c:\tmp\vagrant-shell.ps1
    win10: [02:48] Making Windows 10 Great again
    win10: [02:48] Importing registry keys...
    win10: [02:48] Removing OneDrive...
    win10: [02:48] Disabling automatic screen turnoff in order to prevent screen locking...
    win10: [02:48] Downloading ShutUp10...
    win10: [02:48] Installing ShutUp10 and disabling Windows Defender
    win10: powershell.exe : Invoke-WebRequest : Unable to connect to the remote server
    win10:     + CategoryInfo          : NotSpecified: (Invoke-WebReque...e remote server:String) [], RemoteException
    win10:     + FullyQualifiedErrorId : NativeCommandError
    win10: At C:\tmp\vagrant-shell.ps1:35 char:3
    win10: +   Invoke-WebRequest -Uri "$shutUp10DownloadUrl" -OutFile $shutUp10Rep ...
    win10: +   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    win10:     + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invok
    win10:    e-WebRequest], WebException
    win10:     + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeW
    win10:    ebRequestCommand
    win10:
    win10: . : The term 'C:\Users\vagrant\AppData\Local\Temp\OOSU10.exe' is not recognized as the name of a
    win10: cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path
    win10: was included, verify that the path is correct and try again.
    win10: At C:\tmp\vagrant-shell.ps1:36 char:5
    win10: +   . $shutUp10RepoPath c:\vagrant\resources\windows\shutup10.cfg /quie ...
    win10: +     ~~~~~~~~~~~~~~~~~
    win10:     + CategoryInfo          : ObjectNotFound: (C:\Users\vagran...Temp\OOSU10.exe:String) [], Comm
    win10:    andNotFoundException
    win10:     + FullyQualifiedErrorId : CommandNotFoundException
    win10:
The following WinRM command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

powershell -ExecutionPolicy Bypass -OutputFormat Text -file "c:\tmp\vagrant-shell.ps1"

Stdout from the command:

[02:48] Making Windows 10 Great again
[02:48] Importing registry keys...
[02:48] Removing OneDrive...
[02:48] Disabling automatic screen turnoff in order to prevent screen locking...
[02:48] Downloading ShutUp10...
[02:48] Installing ShutUp10 and disabling Windows Defender

Stderr from the command:

powershell.exe : Invoke-WebRequest : Unable to connect to the remote server
    + CategoryInfo          : NotSpecified: (Invoke-WebReque...e remote server:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError
At C:\tmp\vagrant-shell.ps1:35 char:3
+   Invoke-WebRequest -Uri "$shutUp10DownloadUrl" -OutFile $shutUp10Rep ...
+   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invok
   e-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeW
   ebRequestCommand

. : The term 'C:\Users\vagrant\AppData\Local\Temp\OOSU10.exe' is not recognized as the name of a
cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path
was included, verify that the path is correct and try again.
At C:\tmp\vagrant-shell.ps1:36 char:5
+   . $shutUp10RepoPath c:\vagrant\resources\windows\shutup10.cfg /quie ...
+     ~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\vagran...Temp\OOSU10.exe:String) [], Comm
   andNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Previous issue with vaguely similar problem: https://github.com/clong/DetectionLab/issues/21

I've been seeing a lot of intermittent DNS-related issue coming from Virtualbox lately and I think that's what's causing this. Let me see if I can reproduce.

Hi Chris :)
Sure, do you have a 'known working' VBox version and I can downgrade to that and test? Should be a good nudge towards nailing it down as a VBox issue.

Just installing 6.1.0 VBox as a test now, will report back 👍

All installed and running 'vagrant up' now.
Applied the clipboard fixes detailed here too after getting the --clipboard error.
https://github.com/clong/DetectionLab/issues/374#issuecomment-577920555

Thanks so much for the update! I'm gonna close this one out, but feel free to comment or re-open if it re-surfaces.


Federation Example using v2

The README.md suggests that v2 should be used, but I am finding it difficult to migrate my federated gateway from version 1 to 2, ss some of the packages, e.g. graphql, are missing from v2.
Is there an example of how a federation should be implemented with v2 of the library similar to the federation example in v1 https://github.com/wundergraph/graphql-go-tools/tree/master/examples/federation

Hi @IsabelFreitas-catapult, graphql-go-tools is a low level library. What prevents you from using our GraphQL Federation Router https://cosmo-docs.wundergraph.com/router/intro ?

We haven't updated the examples yet, and this may take some time. In the meantime, I recommend using our router. If you encounter any limitations, we would appreciate your feedback. If you urgently need this library, please reach out for support here so we can address your concerns with priority.

Closing due to inactivity.


Animate player character

Animate player characters
Currently the main characters are just single images. It would be good to animate the character so that when they move their position changes e.g. in a running stance. Currently there are two different player characters in the game for levels one and two. The images have been taken from sprite sheets available at;
http://www.gameart2d.com/temple-run---free-sprites.html
http://www.gameart2d.com/adventurer-girl---free-sprites.html
These sheets have lots of other stances of the character that could be incorporated in to the game.
If you have any questions or suggestions just leave a comment.

Could I work on this as a first timer?

@SnGrimes Great, just leave a comment if you need any help!

The male character is running in a test page. I'm looking at how to implement animations in the main game.js file. Any suggestions?

@SnGrimes Hey sorry for the late reply. Try looking at the web page that the main character image comes from (this can be found in the copyrights file). This site contains a sheet of lots of different character positions. The best way to create an animation is to make the image change when a key is pressed. For example, when the right key is pressed the character image changes to a running stance.
Hope this helps!

I'm sorry I wasn't clear. The male sprite is already animated. The character is running in his own canvas object in a test page. Right now I'm trying to figure out how to integrate his animated run into the game.  I found this link and will try a separate sprite.js file to add animations into the game. If you have any suggestions about where to add the animation code in the Game.js file that would help me a lot.

@SnGrimes Can you show me your code so far? Just copy and paste it in a comment.

I created the loadResources() function to load the images into an array for animating and put loadResouces() under the startGame() function. I originally put it in a IIFE in the Game.js file but that just blew everything up.
function loadResources() {     var maxFrames = 9;     for (let x = 0; x < maxFrames; x++ ) {         let img = new Image();         img.src = "Pictures/Sprite Sheets/Idle__00" + x + ".png";         idleArray.push(img);     } }
I put this animate() function under the this.update function under component and ran it under the if state for image. This prevented the player character and monsters from spawning and the game eventually skips to level 2.
function animate() { ctx.clearRect(this.x, this.y, this.width, this.height);             var msPerImg = 100;             var currentTime = new Date().valueOf();             var idleToDraw = this.image[Math.floor(currentTime/msPerImg) % this.image.length];             ctx.drawImage(idleToDraw, this.x, this.y);
I've never had to squeeze code into an existing program like this before so I'm a bit lost.

@SnGrimes Thanks for putting your code up. HTML canvas starts to get difficult when you want to draw new components during run time. The easiest solution is to manipulate existing ones.
You could start by creating a function that loops through each of your sprite images using a timer,
function animatePlayerCharacter(){
//Timer, set this to the frame rate you want
var timerInterval = setInterval(spriteTimer, 1000);

var spriteImageCounter = 1;

//This would be the number of sprite images you have, 10 is just an example
var numberOfSpriteImages = 10;

function spriteTimer(){

	if(spriteImageCounter < numberOfSpriteImages){

		//If you named each sprite good_guy1.png, good_guy2.png, good_guy3.png etc.....
		playerCharacter.image.src = "Pictures/good_guy" + spriteImageCounter + ".png";

		spriteImageCounter++;

	}
	else{

		//Reset back to the beginning
		sprinteImageCounter = 1;

	}

}

}
You would need to call this when a key is pressed or on screen button. When the key or button is released reset the image.
This is just one way of doing it, feel free to try other methods but hopefully this helps you move forward.

@SnGrimes An alternative would be to use a JQuery plugin. There are a number of different ones available this one might help - http://blaiprat.github.io/jquery.animateSprite/. Remember if you use any existing software, check the license and add it in to the README.txt file.

Thank you! I will look at that one and some other plugins tonight.

Welp, I've created a working running animation with just a difference in one line of code.
This code will create a smooth running animation with an even frame rate but the animation will not stop.
function animatePlayerCharacter () {     var currentTime = new Date().valueOf();     var requestID;     var frames = 9;     var spriteImageCounter = Math.floor(currentTime/100) % frames;     var toDraw = playerCharacter.image.src = "Pictures/Sprite_Sheets/Run__00" + spriteImageCounter + ".png"; }
Once I remove requestAnimationFrame(animatePlayerCharacter); The animation will stop once the arrow key is released but the animation is uneven and not very smooth.
I checked some of the animation libraries and they will either use a separate canvas object or animate a div background. I'm going to keep looking for a solution but any help would be appreciated.

@SnGrimes Could you post your code for the requestAnimationFrame() method?
I agree that using the libraries is difficult because creating multiple canvasses is tricky. A simple solution is best here.

Here is the documentation for requestAnimationFrame.
https://developer.mozilla.org/en-US/docs/Web/API/window/requestAnimationFrame

@SnGrimes Sorry for the delay in getting back to you. Could you create a new PR request with your work so far? That way I can see exactly what code you have added.


Derive proofs for documents signed by more than one key

I am working on a digital contract system where more than one party will be signing a document. Currently, it seems that generating multiple signatures is handled by the underlying libraries, but not for deriving proofs. In the sample app, I can copy:
const signedDocument = await sign(inputDocument, {
  suite: new BbsBlsSignature2020({ key: keyPair }),
  purpose: new purposes.AssertionProofPurpose(),
  documentLoader,
  compactProof: false
});

to
const signedDocument2 = await sign(signedDocument, {
  suite: new BbsBlsSignature2020({ key: keyPair }),
  purpose: new purposes.AssertionProofPurpose(),
  documentLoader,
  compactProof: false
});

And it will generate a document with an array for proof element. Verification also succeeds. The sample continues to:
const derivedProof = await deriveProof(signedDocument, revealDocument, {
  suite: new BbsBlsSignatureProof2020(),
  documentLoader
})

Where it fails in BbsBlsSignatureProof2020 because proof.type is undefined. proof is a hash with keys '0' and '1'. I added the following shim to get by that error:
if(proof['0']) proof = proof['0']

There is then an error that the message and signature don't match unless you add the line:
proof['@context'] = 'https://w3id.org/security/v2'

With these changes, a single proof can be derived and verified. The proper behavior though is to return an array of proofs in the derived document and to verify them all.

Hi @dysbulic thanks for filling this issue, I believe the cause was due to the sample you were using was out of date with the latest release of the library, in #80 I have now updated the sample and tested it with your double proof scenarios with a derivation and it worked as expected, so please let me know if that doesnt resolve this issue.

@tplooker, currently if there are multiple BBS signatures, a derived proof will only be generated for the first one.
Is this the correct behavior? It seems like proof should be an array with a derived proof for each signature.

@dysbulic yes this is correct and by default it is selecting the first proof that matches the supported proof type at the moment. We could potentially modify the logic in deriveProof to generate a derived proof per matched proof, could you elaborate more on why this usage is important, do you want to be able to prove information revealed in a derived proof was signed by multiple issuers?

I am working on a bonded courier service. I want to have the sender, recipient, and courier all sign a JSON-LD contract describing:
• the appearance of the parcel
• where it is coming from
• where it goes
• bond amount staked against loss of the parcel
• bid amount the courier will receive
An escrow covering the courier's bid for the job and a stake of the courier's bond are created.
If a party faults ‒ the sender or recipient is unreachable, the courier absconds with the package, etc. ‒ then a moderator can be employed to decide the disbursement of funds. (I'm working off the OpenBazaar escrow.)
I would like to reveal as little about the contract as possible to the moderator, specifically I want to keep the locations of the parties private while proving that the parties all signed a contract.

I've got code that passes all the existing tests. The sample, however, is failing, and I'm still narrowing down why.
The basic refactor that I did was for studio.deriveProof and studio.verifyProof, I moved the bulk of the computation into a derive (or verify) method which I call appropriately depending on the result of Array.isArray.
It changed the indentation which makes it look like I touched alot more than I actually did.

@dysbulic thanks for the link and description of the usecase, I think we can refactor the deriveProof function to instead of selecting the first valid proof, enumerate through matched proofs and create a derivedProof

@dysbulic we've got this code into master and are fixing the CI pipeline so an unstable release can be used to work with this functionality shortly. Samples are in PR #83 as well if you'd like to look at some examples.
Thanks for filing this issue.

@dysbulic this functionality is now available in the unstable version of 0.7.1-unstable.55c8eaa we will cut a formal release in the next week or two after implementing several other pending features

@tplooker, that's awesome. I'll play around with it some.
You all's turnaround time was great. Thanks for being so responsive to the community.
All this crypto stuff is evolving so quickly, especially the zero-knowledge arena. It's nice to have solid implementations to build on.

@tplooker, that's awesome. I'll play around with it some.

You all's turnaround time was great. Thanks for being so responsive to the community.

@dysbulic you're welcome glad we could help

All this crypto stuff is evolving so quickly, especially the zero-knowledge arena. It's nice to have solid implementations to build on.

Yes watch this space, we have many more features coming down the pipeline.
We will keep this issue open until this feature is cut into a formal release


VCC-2970 VCC-2968 Add a select-only variant; add additional keyboard interaction

When the text input is deactivated, the combobox works like a <select>.
The input is replaced with a div, then all of the appropriate interactions and attributes are added to the div. Notably, this also brings a lot of select-specific behavior to the div.
Even though the text input is not visible, we still have to use familiar keyboard patterns; see https://www.w3.org/WAI/ARIA/apg/patterns/combobox/examples/combobox-select-only/#kbd_label

Thanks @Farsad! Those issues are updated. Now that we're retaining a hidden <input> for form submissions, I'm not sure that "noInput" would be a good fit either.
I came across the "Select-Only" terminology from the W3C.
In another discussion on W3C, this was first called a "read-only combobox" and changed to "select-only" because readonly typically means something different.
I've seen this terminology used from other accessibility bloggers and developers, but usually in reference to the W3C example.
I agree that it's a funny fit when we are adding <select>-initialized comboboxes too, but haven't seen another term commonly used.


Victory components ignore their container width

The documentation mentions that by default the containers are responsive.
What I expected is that while the viewbox would be fixed, the actual svg size would adapt to its container size. But it does not seem to be the case.
class Bar extends React.Component {
  render() {
    return (
      <View style={styles.container}>
        <View style={styles.graphContainer}>
          <VictoryBar />
        </View>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1
  },
  graphContainer: {
    width: 150,
    borderColor: 'red',
    borderWidth: StyleSheet.hairlineWidth,
  }
});

Height as well:
    return (
      <View style={{ flex: 1 }}>
        <View style={{ width: 150, height: 150, borderColor: 'red', borderWidth: StyleSheet.hairlineWidth }}>
          <VictoryBar />
        </View>
      </View>
    );

Workaround: wrap in a VictoryChart and set the width and height attributes (not styles):
<VictoryChart width={150} height={150}>
  <VictoryBar />
</VictoryChart>

I'm running into this issue, too. It seems like our only option is to use absolute values for Victory charts at this point

This is because victory-native defaults the width to Dimensions.get('winow').width. This also is a problem if you want it to resize when the screen rotates.
We solved this by having the width set as an internal state and updating it in onLayout. So in @hwaterke example it would look something like this:
class Bar extends React.Component {
  constructor(props, context) {
    super(props, context);

    this.state = {
      width: 0
    };
    this.measureView = this.measureView.bind(this);
  }

  setDimensions(event) {
    this.setState({
      width: event.nativeEvent.layout.width
    });
  }

  render() {
    const { width } = this.props;

    return (
      <View style={styles.container} onLayout={setDimensions}>
        <View style={styles.graphContainer}>
          <VictoryBar width={width} />
        </View>
      </View>
    );
  }
}

It would be great if Victory native were responsive by default, this is especially problematic when we use it with react-native-web.

Is there a workaround for this today on React Native? Today I need to set the width manually on every chart, it would be very helpful to have the width of the chart be the same as the container.


pgAdmin: Error parsing headers: 'limit request headers fields size'

Setup AppHost with a configuration like this:

var postgresPassword = builder.AddParameter("postgres-password", secret: true);

var myDb = builder.AddPostgres("postgres", password: postgresPassword)
                      .WithDataVolume()
                      .WithPgAdmin()
                      .AddDatabase("catalog-db", "Catalog");

Run appHost

Browse to the Aspire Dashboard

Try clicking the pgAdmin endpoint

You get this page:

The actual backend API that uses Postgres is able to talk to the Postgres container just fine. Yet, pgAdmin just shows that page.

pgAdmin Logs
2024-05-28T17:06:17.8154706 NOTE: Configuring authentication for DESKTOP mode.
2024-05-28T17:06:17.8155531 pgAdmin 4 - Application Initialisation
2024-05-28T17:06:17.8158995 ======================================
2024-05-28T17:06:17.8159099
2024-05-28T17:06:27.2515914 ----------
2024-05-28T17:06:27.2518602 Loading servers with:
2024-05-28T17:06:27.2523020 User<EMAIL_ADDRESS>2024-05-28T17:06:27.2527129 SQLite pgAdmin config: /var/lib/pgadmin/pgadmin4.db
2024-05-28T17:06:27.2529262 ----------
2024-05-28T17:06:32.4395699 Added 0 Server Group(s) and 1 Server(s).
2024-05-28T17:06:33.2835035 postfix/postlog: starting the Postfix mail system
2024-05-28T17:06:38.5229628 [2024-05-29 00:06:38 +0000] [1] [INFO] Starting gunicorn 20.1.0
2024-05-28T17:06:38.5232835 [2024-05-29 00:06:38 +0000] [1] [INFO] Listening at: http://[::]:80 (1)
2024-05-28T17:06:38.5233047 [2024-05-29 00:06:38 +0000] [1] [INFO] Using worker: gthread
2024-05-28T17:06:38.5265404 [2024-05-29 00:06:38 +0000] [121] [INFO] Booting worker with pid: 121
Postgres Logs
2024-05-28T17:06:05.0744682
2024-05-28T17:06:05.0745148 PostgreSQL Database directory appears to contain a database; Skipping initialization
2024-05-28T17:06:05.0745203
2024-05-28T17:06:05.1065780 2024-05-29 00:06:05.106 UTC [1] LOG:  starting PostgreSQL 16.2 (Debian 16.2-1.pgdg120+2) on x86_64-pc-linux-gnu, compiled by gcc (Debian 12.2.0-14) 12.2.0, 64-bit
2024-05-28T17:06:05.1066052 2024-05-29 00:06:05.106 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
2024-05-28T17:06:05.1066093 2024-05-29 00:06:05.106 UTC [1] LOG:  listening on IPv6 address "::", port 5432
2024-05-28T17:06:05.1195364 2024-05-29 00:06:05.119 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2024-05-28T17:06:05.1318991 2024-05-29 00:06:05.131 UTC [29] LOG:  database system was shut down at 2024-05-29 00:06:00 UTC
2024-05-28T17:06:05.1411041 2024-05-29 00:06:05.140 UTC [1] LOG:  database system is ready to accept connections

@DamianEdwards you ran into this. @julioct try clearing your cookies.

That did it, thank you!


Crash in Lexer on '' character

Playing around in the app now, I imagine this is not limited to the app.
Crash occured at backslash in printf("%d\.
Lexer.readCharacter() throws -> UnicodeScalar at Sources/Parse/Lexer.swift:448
Lexer.advanceToNextToken() throws -> Token at Sources/Parse/Lexer.swift:505
Lexer.lex() throws -> [Token] at Sources/Parse/Lexer.swift:349
LexerTextStorage.processEditing() -> () at iOS App/AnnotatedTextStorage.swift:83

Thank you! Yep there are lots of small edge cases like this. I think I'm not handling EOF after Lexington the escape character. 😅

@vdka Try the latest master.

@vdka Is the issue fixed for you?

Sure is :+1:

Thanks for reporting! AFAIK you're the first stranger to download and try this project 😄 I'd love whatever feedback you have!


Add get_point_position method to AStarGrid2D

I think it may be useful, and this method exist in AStar3D/2D too.

Thanks!


Access to XMLHttpRequest at '' has been blocked by CORS policy:

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[X] support request

Current behavior

The unit test on Angular library that uses nngx-formly gives CORS error
Expected behavior

The unit test should pass.
Minimal reproduction of the problem with instructions

The following error is comming in the console.
Access to XMLHttpRequest at 'ng:///DynamicTestModule/FormlyFieldsTab.ngfactory.js' from origin 'http://localhost:9876' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, https.
zone-evergreen.js:2981 Access to XMLHttpRequest at 'ng:///DynamicTestModule/ObjectTypeComponent.ngfactory.js' from origin 'http://localhost:9876' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, https.
ObjectTypeComponent and FormlyFieldsTab are extended type.
What is the motivation / use case for changing the behavior?

Please help me.
Please tell us about your environment:

WIndows 10

Angular version: 2.0.X

Angular 8

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

Node (for AoT issues): node --version =

use stackoverflow.


[request]: Move examples folder out of src.

There are some rust project move example folder out of src, like axum and so on.

This landed in v0.0.24-alpha.


[FLINK-12954] Supports create(drop) view grammar for sql parser

What is the purpose of the change

Add create view grammar like CREATE VIEW view_name(col1, col2) AS select col3, col4 from tbl

Brief change log

Add create/drop view

Verifying this change
See tests in FlinkSqlParserimplTest.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

LGTM
+1 to merge


model problem

hello, I found the attentive-like mechanism in the paper of v1, Whether the enhance(self, c, h) in the unet.py is the attentive mechanism.
also I can't found enhance's usage in the code. Can you tell me where to use attentive mechanism. Thanks!!

same with you about not finding enhance's usage. Has anyone found it?


tbb library linking issue

Summary
Trying to compile oneapi-sample code example  dpc_reducer( https://github.com/oneapi-src/oneAPI-samples/tree/master/DirectProgramming/DPC%2B%2B/ParallelPatterns/dpc_reduce)
Version
oneAPI Toolkit version: l_BaseKit_p_2<IP_ADDRESS>22_offline.sh
oneAPI Sample version:  latest master branch.
Environment
OS: Ubuntu 20.04
Hardware : NUC9
Steps to reproduce
followed dpc_reduce example README file.
export I_MPI_CXX=dpcpp
cmake ../
make
Observed behavior
following compile time error is coming.
**/usr/bin/ld: /tmp/main-585a82.o: in function task_group_context': /opt/intel/oneapi/tbb/2021.4.0/env/../include/tbb/../oneapi/tbb/task_group.h:271: undefined reference to tbb::detail::r1::initialize(tbb::detail::d1::task_group_context&)'**
Expected behavior
should be compile successfully.

this issue is coming due to libtbb.
if in Ubuntu system already libtbb is installed then oneapi environment variable is set that library path instead of oneapi libtbb library path.

OneSmpl_Team3 working on this.


[RFC] Should static key be part of keys allowed to an admin when interaction with action groups via API.

Coming from this discussion thread on the PR: https://github.com/opensearch-project/security/pull/4371.
At the moment, we do not have clear answer neither there is a clear distinction between usages of hidden, reserved and static.
Expected outcome

[ ] An answer has been provided on whether static field should be allowed to be updated for an admin user.

Not only action groups but roles, roles mapping,internal users and tenants

[Triage] Thanks for filing this issue @DarshitChanpura. Swapped to RFC in the title just so more people click and provide their thoughts. I will look into the links you shared and try to offer an opinion later today.

I just saw this by coincidence, so I thought I'll give you my five cents.
As far as I understand it, the meaning of static, reserved and hidden in the config files is as follows:

static: Config entries which are defined in the source code (in https://github.com/opensearch-project/security/tree/main/src/main/resources/static_config ) are marked as such. These are config entries on which you can rely to be everywhere - like the action groups read or crud (except when disabled by a weird config option). Static config entries should not be exposed or editable by the REST APIs or the securityadmin tool. Having this as an attribute in the YAML file feels totally redundant, though, as static entries can be identified as such by being present in the respective collection here:

https://github.com/opensearch-project/security/blob/2b5a811de599f7c7fc2ca2b9246e57fd6cfaf33b/src/main/java/org/opensearch/security/securityconf/DynamicConfigFactory.java#L81

reserved: This is a config entry which can be only modified as a super admin user. It is visible to normal users, though.

hidden: This is a config entry which can be only modifed and seen by a super admin user.

So, to come back to the question:

Should static key be part of keys allowed to an admin when interaction with action groups via API.

My IMHO would be "no" :-)

@willyborankin based on @nibix 's explanation of the keywords and opinion, it seems like _static should not be part of admin actions via REST API.

@DarshitChanpura Agree. Lets convert it to an issue and remove it. Only one thing: Since the functionality is in the public documentation it should be implemented for 3.x version only. Wdyt?

Agreed as I'm not aware of the blast radius of this so 3.x sounds like a safer route to implement this.


also build statement AST for Unknown statements

to provide typeahead and suggestion features
the AST will be incomplete and with syntax errors
take the AST that is longest, where the parser got the furthest

think this is solved


Support Notification for Internet Explorer 8

A lot of brigades liaison with the local government. And since local governments tend to use legacy browsers on Windows then it may be a deal breaker for some if city employees pull up a, uh, less than stellar looking site.
For context, the Bootstrap team won't be supporting IE8 with Bootstrap 4 but do recommend community supported backward compatibility efforts through polyfills and shims.
That being said, it may not be a big deal and I may actually be super wrong and nobody uses XP anymore - or, at the very least, this won't be a problem for a major American city.
Or we don't want to spend the engineering effort and instead encourage brigade website usage on personal laptops instead of government machines.
Anyways, if we choose to not support IE8 then we should probably have a notification that tells users that their browser is not supported.

since we expect even homeless people to be able to access the info via mobile

A lot of homeless and low-income individuals use computers at libraries and a lot of libraries still run Windows XP - which caps out at IE8 and use it as the only available browser. Also, any Code for All brigade outside of the U.S... may have a local city government that still uses XP. I mean, I bet it's pretty likely.

I'd need to check how BS4 handles it, but when BS3 failed (in IE 7, 8) it just collapsed down to a single column. All the content was still there, still accessible.

@therebelrobot as far as the City goes on this, it varies, but it's not as bad as you'd think. For example, I have latest versions of Chrome and Firefox. I use IE only to use my timecard application that requires it because well, who knows why!  And my personal opinion is folks at the City should be using the latest versions of whatever browser they're on if only because it's more secure.
I'd preference some form of graceful degradation and base minimal support on the analytics and users needs. For example, what do the current analytics on the current c4sf site show?
As for lower income users at the Library, you may want to verify the assumption, our Library may be on Windows 7, a lot of the City is, again not everywhere, but in a relatively good portion. Maybe a field trip to the Library ;-) Or I can ping some folks over there.
And I think a good general practice is to build for our context to the best of our ability, document the assumptions made and why and paint the way for others to improve if their context is different, rather than trying to build for every possible use case out of the gate.


Implement CDN loop detection

Title: Implement CDN loop detection
Description:
RFC 8586 [1] proposes CDN providers to develop CDN loop prevention mechanism by using a new CDN-Loop header. While more details are explained in RFC 8586 proposal, here we include a simple TODO list for this feature:

Add a new CDN-Loop request header with configurable value when proxy is processing a request.
Check whether a CDN request has CDN-Loop header with the same configurable value. If yes, deny that request with code 502.

Relevant Links:

https://datatracker.ietf.org/doc/rfc8586/

If I understand correctly, you want to use Envoy to implement detection of the CDN-Loop header (responding with an error) and adding the header to the proxied request (for potential detection later if the request is circular).
I think you can already do this by adding routes that match on the value of the header which perform a direct response with a 502. Request headers can be added to proxied request which covers the forwarding path.

At first glance that sounds like a viable workaround but my intuition says this is something we may want as a core Envoy feature.
For Envoy cloud providers, customers are likely going to specify their own routes, and the likelihood of the majority of them doing this is roughly zero.    Basically I think loop detection ought to be a global configuration option rather than a per-route thing.
@htuch

In general, I'm an API minimalist and would recommend the solution @zuercher expressed. I agree with the point that @alyssawilk makes, but I lean on the API philosophy that we are creating a machine expressible language, and that UX improvements should be reflected in frontends to xDS. We're not exactly consistent with this, and @mattklein123 leans closer towards balancing UX with minimalism.

It could also be implemented as an HTTP filter, I think.

FWIW this sounds like a good filter candidate to me.

I'm happy to work on this feature as an HTTP filter!
Parsing the CDN-Loop header in an RFC 8586-compliant way is somewhat annoying. You can't just split on ',' because the header allows parameter values that are quoted strings, which themselves can contain commas!  The best solution I've come up with is to go to the ABNF in the RFC and write a recursive descent parser.
Here's the approach I'd like to take to solving this problem, which I see forming 3 PRs:

Write just enough of CDN-Loop header parser to pick out the RFC 8586 cdn-id values that appear in the header.
Write a utility function to call the parser and count how many times a particular cdn-id value appears.
Write the CDN-Loop HTTP filter.  The filter shall be configurable with (1) the cdn-id to look for and (2) the number of times it's allowed to be seen.  (It's possible that a CDN wants to let a request transit itself a small number of times before rejecting it.)  Call the utility function from the previous step to simplify the filter's implementation.

Does that make sense to people?

Excellent, thanks!  Depending on how complicated 1) and 2) are it might be fine to fold them all together but otherwise this all sounds fine to me.


Add time to points in PointCloud.

PointCloud holds 4-dimensional vectors so measurement time of individual points can be stored in the fourth entry.
Fixes #573.

Only merge this PR when the respective ROS component change googlecartographer/cartographer_ros#549 is ready.

Thanks for your comments. PTAL at this alternative solution. (It is possible to split it up into two commits.)

All done, please take another look.


:fire: Clean up redundant code

Quick clean up of some redundant/duplicated code.

Nice, thank you! @svlandeg :bow: :cake:


Added custom database table names instructions in readme

Added instructions how to define custom table names for models as optional step in readme file.

:+1: thanks again!


Overlap embeddings

  chunk_overlap: int = 0, overlap should be about 100 to preserve meanings cutoff by chunk_size

It is up to the user to make the text overlap, but let's add that setting to the config.

When you have the time, allow Admin to edit config from Admin UI. no rush on this one, except it's easy to do


Supporting http/2 with server push

Go 1.8 comes with a eagerly anticipated feature: Http/2. One feature of http/2 is server push, which in theory increase the performance of web page delivery.
So I am wondering if by any chance gitea 1.3 will be able to move to go 1.8.
P.S. Since http/2 requires a SSL certificate, do you see that we could also include Let's Encrypt with gitea. I have seen a couple solutions that works with go, i.e. this one from rsc .

We can build the prebuilt binaries with 1.8, but we won't drop support for older go versions as even developers like @strk are still working on 1.6. For the docker image we are using the go version that is available on alpine edge.

So that could be a build tag

Not any build tag at all, http2 is part of the stdlib. It's just working with https connections only

@tboerger After reading more about different version (1.6 - 1.8), I found out that 1.6 actually supports http/2, but not server push, which it is new for 1.8. I guess we could still use http/2 with 1.6 but not server push. Then it should have increase performance AND security by just using http/2 per se.
Reference:
Seven years of Go

I know, but it's up to the user to configure https.

HTTP2 is also definedfor unencrypted transport...
I dont know why golang wont implement this in there std lib.
https://github.com/hkwi/h2c
Clould h2c implemented in gitea with the above library?
That would be great :-)
I want to use apache proxy_http2 without encryption between gitea and apache.
For me i cloud not use gitea directly to port 443 or port 80.

@juppin No browser allows for unencrypted HTTP2 though. So even if the spec allows it, it's essentally useless for browser usage

@juppin Well, given the fact that Chrome will begin to block non-https website by showing what insecure https website shows now; and it is now displaying "insecure" on the address bar for the http website, the trend is going to be https for every website. So it does not hurt to use the ssl for proxy, too.

HTTP2 main goal is to optimize the network and reduce the latency while keeping the compatibility with HTTP/1.x.
If you are proxing to a local network or to the localhost I don't think you will see any significant performance gain.
For me, it's definitely not a "useful usecase".

Just adding an autmatic Link header would help out even in HTTP1.1 as those of us who do Cache and Pushing through a web server (Caddy for example) can configure them to push assets that are in the Link header, not only that but people who use Cloudflare would also get that extra benefit because Cloudflare also pays attention to the Link header


add lint/voidDomElementsNoChildren for React

This PR adds linting and autofixing for void elements (br, img, hr, etc.) which have children and/or dangerouslySetInnerHTML properties.
ESLint Rule Reference
#341

Looks good! Thank you!


Improve Semicolons

There is two things we should do:

Allow trailing Semicolons
Implement semicolon inference

For the later, we probably need to move to manual whitespace handling in the parser

See #21


Use atexit(3) to ensure the terminal back to the original state

Currently, we are calling LeaveAlternateScreen and disable_raw_mode only when the process successfully finishes. However, there are many situations to abruptly abort the process. Aside from SIGKILL, I want this program to make the terminal state back to the original state as it can. I think atexit(3) can be used to solve this problem. libc::atexit seems to be good.

FYI https://doc.rust-lang.org/nomicon/ffi.html

I'm considering replacing the implementation of invoke with execvp(2), but this API cannot be used with atexit(3). So far, this issue is not prioritized.


Init repo failed

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] puppy => You're not submitting a puppy. I already have one and he's adorable

What is the current behavior?
init repo failed, it always show "No Gitflow", but acctually succeed, because in sourcetree this repo do not need init again

Is this a bug? Sorry about that. If so give me explicit details how to reproduce:

Open Webstorm
GitFlow Actions -> Init repo
Notifications shows 'Initialized gitflow in repo xxx', but this plugin still shows No Gitflow

What is the expected behavior?
Init success so I can start  a feature

What is the motivation / use case for changing the behavior?

Please tell me about your environment:

Gitflow4idea version: 0.7.0

Gitflow version: 1.12.3(AVH Edition)

IntelliJ Help -> about > click copy icon and paste here. Should look like this:

  WebStorm 2019.2.2

Build #WS-192.6603.19, built on September 3, 2019
Licensed to xxx
You have a perpetual fallback license for this version
Subscription is active until November 16, 2020
Runtime version: 11.0.3+12-b304.56 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 7 6.1
GC: ParNew, ConcurrentMarkSweep
Memory: 2008M
Cores: 4
Registry: bigger.font.in.project.view=true, ide.tree.ui.experimental=false, ide.balloon.shadow.size=0
Non-Bundled Plugins: com.chrisrm.idea.MaterialThemeUI, Gitflow

* **Other information** (e.g. detailed explanation, stacktrace, related issues, suggestions how to fix, links for me to have context words of praises, pictures of puppies (again with the puppy??) )

java.lang.NullPointerException
  at gitflow.actions.InitRepoAction.update(InitRepoAction.java:39)
  at com.intellij.openapi.actionSystem.ex.ActionUtil.performDumbAwareUpdate(ActionUtil.java:176)
  at com.intellij.ide.util.gotoByName.GotoActionModel.updateActionBeforeShow(GotoActionModel.java:296)
  at com.intellij.ide.util.gotoByName.GotoActionModel$ActionWrapper.lambda$getPresentation$0(GotoActionModel.java:645)
  at com.intellij.ide.util.gotoByName.GotoActionModel.lambda$updateOnEdt$4(GotoActionModel.java:449)
  at com.intellij.openapi.application.TransactionGuardImpl$2.run(TransactionGuardImpl.java:312)
  at com.intellij.openapi.application.impl.LaterInvocator$FlushQueue.doRun(LaterInvocator.java:433)
  at com.intellij.openapi.application.impl.LaterInvocator$FlushQueue.runNextEvent(LaterInvocator.java:416)
  at com.intellij.openapi.application.impl.LaterInvocator$FlushQueue.run(LaterInvocator.java:399)
  at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
  at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:776)
  at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:727)
  at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
  at java.base/java.security.AccessController.doPrivileged(Native Method)
  at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
  at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:746)
  at com.intellij.ide.IdeEventQueue.defaultDispatchEvent(IdeEventQueue.java:878)
  at com.intellij.ide.IdeEventQueue._dispatchEvent(IdeEventQueue.java:827)
  at com.intellij.ide.IdeEventQueue.lambda$dispatchEvent$8(IdeEventQueue.java:466)
  at com.intellij.openapi.progress.impl.CoreProgressManager.computePrioritized(CoreProgressManager.java:704)
  at com.intellij.ide.IdeEventQueue.dispatchEvent(IdeEventQueue.java:465)
  at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
  at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
  at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
  at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
  at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
  at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Thanks for the report.
This is a bug in the GUI update logic, in the meantime until this gets sorted out you can simply restart WebStorm after the gitflow init and the plugin widget should work properly

Thanks for the report.
This is a bug in the GUI update logic, in the meantime until this gets sorted out you can simply restart WebStorm after the gitflow init and the plugin widget should work properly

No, restart don't work.


Twill Media Library generating incorrect image url when using Glide with AWS S3

Description

Incorrect image url generated when using Glide with AWS S3
Steps to reproduce

Config .env file as below
MEDIA_LIBRARY_ENDPOINT_TYPE=s3
MEDIA_LIBRARY_CASCADE_DELETE=true
MEDIA_LIBRARY_LOCAL_PATH=uploads
MEDIA_LIBRARY_IMAGE_SERVICE=A17\Twill\Services\MediaLibrary\Glide
S3_KEY=foobar
S3_SECRET=foobar
S3_BUCKET=foobar

Expected result

Below is a working url generated when using A17\Twill\Services\MediaLibrary\Local
https://infomate.s3.ap-southeast-2.amazonaws.com/6dde5274-a0af-482d-a367-6207ce2ebe6d/500x500.png
Actual result

When using S3, shouldn't the url be pointing to S3 endpoint?
https://infomate.test/img/6692f470-b9f5-425c-9df6-be124f142118/500x500.png
https://flareapp.io/share/OmVpJ437#F40
Versions

"area17/twill": "haringsrob-patch-1-dev"

 "laravel/framework": "^10.0",

PHP 8.2.5 (cli)

mysql  Ver 8.0.32

Hi @theia-kevin, no, it should not generate a S3 url, because Glide is a dynamic image rendering service. With Glide, image requests need to go through your Laravel application for the parameters to be processed. From there it can return the cached image transformation.
If the URL generated was pointing to S3, adding parameters, e.g. w=1000, would have absolutely no effect.
Because of the nature of Glide, Glide + S3 is not optimal without using a CDN on top of the Glide /img route, because there's too much overhead in having your Laravel application return an S3 file, even when Glide already generated a cached image.
However, you can decide to use S3 as your Glide source while using local as your Glide cache, which is more performant but requires space on your server.
Ultimately, this is why Twill also offers dedicated image rendering services compatibility, like Imgix and Twicpics.
Finally, if you get an error on the generated Glide image URLs, check your logs, you might be missing the GD or Imagick PHP extensions.

The error you shared is actually saying that your Laravel application isn't able to write to the log file, so you'll need to fix that before seeing the actual error.

Hi @theia-kevin, no, it should not generate a S3 url, because Glide is a dynamic image rendering service. With Glide, image requests need to go through your Laravel application for the parameters to be processed. From there it can return the cached image transformation.
If the URL generated was pointing to S3, adding parameters, e.g. w=1000, would have absolutely no effect.
Because of the nature of Glide, Glide + S3 is not optimal without using a CDN on top of the Glide /img route, because there's too much overhead in having your Laravel application return an S3 file, even when Glide already generated a cached image.
However, you can decide to use S3 as your Glide source while using local as your Glide cache, which is more performant but requires space on your server.
Ultimately, this is why Twill also offers dedicated image rendering services compatibility, like Imgix and Twicpics.
Finally, if you get an error on the generated Glide image URLs, check your logs, you might be missing the GD or Imagick PHP extensions.

@ifox Thankyou so much, what you explained make sense.


r2dbc-pool 0.9.0.RELEASE (was 0.8.8.RELEASE)

Updates io.r2dbc:r2dbc-pool from 0.8.8.RELEASE to 0.9.0.RELEASE.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.r2dbc", artifactId = "r2dbc-pool" } ]

labels: library-update, commit-count:1

Superseded by #187.


Intercepting Publish on server does not return the correct status code to the client

I am trying to put an authorization functionality around publishing topics. The code below will stop the topic being published but the client gets a Success status instead of NotAuthorized
Is there anyway for the client to know the topic was not rejected by the server or is this a bug ?
Below is a test to show the issue
    [Test]
    public async Task TestCancelPublish()
    {
        var testTopic = "test/topic";
        var mqttFactory = new MqttFactory();
        var mqttServerOptions = new MqttServerOptionsBuilder().WithDefaultEndpoint().Build();
        using (var mqttServer = mqttFactory.CreateMqttServer(mqttServerOptions))
        {
            string clientId = Guid.NewGuid().ToString();
            await mqttServer.StartAsync();
            bool messageInterceptedOnServer = false;
            mqttServer.InterceptingPublishAsync += async (args) =>
            {
                // intercept server message and set reason code to NotAuthorized

                args.Response.ReasonCode = MqttPubAckReasonCode.NotAuthorized;
                args.Response.ReasonString = "Not Authorized";
                args.ProcessPublish = false;
                messageInterceptedOnServer = true;
            };

            using (var mqttClient = mqttFactory.CreateMqttClient())
            {
                var mqttClientOptions = new MqttClientOptionsBuilder()
                    .WithTcpServer("localhost")
                    .WithClientId(clientId)
                    .Build();
                await mqttClient.ConnectAsync(mqttClientOptions, CancellationToken.None);

                // publish the message
                var res = await mqttClient.PublishStringAsync(testTopic, "hello");
                while (messageInterceptedOnServer == false)
                {
                    await Task.Delay(300);
                }

                // This Fails
                // response status code says Success, I would expect NotAuthorized ???
                Assert.AreEqual(res.ReasonCode, MqttClientPublishReasonCode.NotAuthorized);
                await mqttClient.DisconnectAsync();
            }

            await mqttServer.StopAsync();
        }
    }

You must change a few things in order to get it running:

You must use protocol version 5 which can be set in the client options builder
Do not set ProcessPublish to false because then the message will basically treated as never sent
You must use QoS 1 so that the client will receive a response. QoS 0 will not receive and response from the server
When using protocol version 5 you must have a subscriber. Otherwise you will get a "NotMatchingSubscribers" response

I adjusted the unit test based on the guide above and I get the same results of success being returned when I expect Not Authorized
`      [Test]
public async Task TestCancelPublish()
{
var testTopic = "test/topic";
var mqttFactory = new MqttFactory();
var mqttServerOptions = new MqttServerOptionsBuilder().WithDefaultEndpoint().Build();
using (var mqttServer = mqttFactory.CreateMqttServer(mqttServerOptions))
{
await mqttServer.StartAsync();
bool messageInterceptedOnServer = false;
mqttServer.InterceptingPublishAsync += async (args) =>
{
// intercept server message and set reason code to NotAuthorized
              args.Response.ReasonCode = MqttPubAckReasonCode.NotAuthorized;
              args.Response.ReasonString = "Not Authorized";
              //args.ProcessPublish = false;
              messageInterceptedOnServer = true;
          };

          using (var client1 = mqttFactory.CreateMqttClient())
          {
              var client1Options = new MqttClientOptionsBuilder()
                  .WithTcpServer("localhost")
                  .WithClientId(Guid.NewGuid().ToString())
                  .WithProtocolVersion(MQTTnet.Formatter.MqttProtocolVersion.V500)
                  .WithWillQualityOfServiceLevel(MqttQualityOfServiceLevel.AtLeastOnce)
                  .Build();
              await client1.ConnectAsync(client1Options, CancellationToken.None);

              using(var client2 = mqttFactory.CreateMqttClient())
              {
                  var client2Options = new MqttClientOptionsBuilder()
                      .WithTcpServer("localhost")
                      .WithClientId(Guid.NewGuid().ToString())
                      .WithProtocolVersion(MQTTnet.Formatter.MqttProtocolVersion.V500)
                      .WithWillQualityOfServiceLevel(MqttQualityOfServiceLevel.AtLeastOnce)
                      .Build();
                  await client2.ConnectAsync(client2Options, CancellationToken.None);
                  await client2.SubscribeAsync(testTopic);

                  // publish the message
                  var res = await client1.PublishStringAsync(testTopic, "hello");
                  while (messageInterceptedOnServer == false)
                  {
                      await Task.Delay(300);
                  }

                  await client2.DisconnectAsync();
                  // This Fails
                  // response status code says Success, I would expect NotAuthorized ???
                  Assert.AreEqual(res.ReasonCode, MqttClientPublishReasonCode.NotAuthorized);

              }
              await client1.DisconnectAsync();
          }

          await mqttServer.StopAsync();
      }
  }

`

That is a good idea. But would check for success instead. So that it will never return NoMatchingSubscribers only if it is basically a success and no error etc.


R# for RAR, CopyTask, copy tests

Please stop emailing me
On Wed, May 16, 2018, 9:07 PM Erik Mavrinac<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/Microsoft/msbuild/pull/3321
Commit Summary

R# for RAR, CopyTask, copy tests

File Changes

M README.md
https://github.com/Microsoft/msbuild/pull/3321/files#diff-0 (2)
M src/Shared/UnitTests/MockEngine.cs
https://github.com/Microsoft/msbuild/pull/3321/files#diff-1 (109)
M src/Tasks.UnitTests/Copy_Tests.cs
https://github.com/Microsoft/msbuild/pull/3321/files#diff-2 (1042)
M src/Tasks/AssemblyDependency/ResolveAssemblyReference.cs
https://github.com/Microsoft/msbuild/pull/3321/files#diff-3 (471)
M src/Tasks/Copy.cs
https://github.com/Microsoft/msbuild/pull/3321/files#diff-4 (200)

Patch Links:

https://github.com/Microsoft/msbuild/pull/3321.patch
https://github.com/Microsoft/msbuild/pull/3321.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/msbuild/pull/3321, or mute the thread
https://github.com/notifications/unsubscribe-auth/AgLdKKU32vhbwtt-FICO6V9ebvVfrPMKks5tzNtVgaJpZM4UCTOi
.


enable onboard temperature register

Looks like the adc_read_temp is implemented by the constant to enable it is missing.

Tested on pico.


App Icons for hyprland/window

This PR introduces App Icons for the hyprland/window module (as requested in #1925)
Since this required almost no changes from the code for sway/window and I didn't want to duplicate the code, I introduces a new AAppIconLabel class which now both hyprland/window and sway/window inherit from. The sway implementation uses app_class if app_id is not set (appears to be the case if the window is running under xwayland) but I wasn't really able to find a suitable replacement for that with hyprland (app_id seems to correspond to hyprlands class though).
Tested on both hyprland and sway

Nice!
Thanks :)


In case of unsuccessful cp of module artifacts to remote path, restore the original path

Codecov Report
Patch coverage has no change and project coverage change: +0.04 :tada:

Comparison is base (8df9082) 55.74% compared to head (e7fa93e) 55.79%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #418      +/-   ##
==========================================
+ Coverage   55.74%   55.79%   +0.04%
==========================================
  Files          60       60
  Lines        5462     5463       +1
==========================================
+ Hits         3045     3048       +3
+ Misses       2417     2415       -2

Impacted Files
Coverage Δ

redisbench_admin/run_remote/standalone.py
51.56% <0.00%> (-0.82%)
:arrow_down:

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Integrate welcome bot

I can add a welcome bot config file having a proper message that will show up when any user will open up an issue or pull-request for the first time.
For reference, check out:  https://github.com/apps/welcome
If you feel this is a great idea!
Please assign it to me. @Saikat-98

Go ahead start working on it, I will assign it to you.

Go ahead start working on it, I will assign it to you.

I would like to work on this.

I would like to work on this.