Allow for custom management of "Invalid request URI"

Case
With Mummy 0.4.x, the commit below implements a connection drop on an invalid request URI due to the new parsing of URI elements.
Problem
Running Mummy behind nginx and making a GET request for localhost:8080/badparam?test=% will drop the connection within Mummy with the log-message "Dropped connection, invalid request URI," causing nginx to send a 502 result. In Mummy versions prior to 0.4.x, this type of error was ignored, and the route was served, allowing for a custom-formatted page and logging the bad URI in the backend.
I want control over my webserver and incoming requests. I like that Mummy parses the parameters for me, but I don't want it to automatically decide whether to drop the connection without giving me the option to provide custom management.
Question
Can we make Mummy less restrictive in this approach? Either by passing the data to the route without the parameters or, at least, using the errorHandler in some way?
Blame
https://github.com/guzba/mummy/commit/b05657228add4b152dc9d2fa8dd8847cab781ed2#diff-1e13710d6692a0c54d414e3ba123be6e4b788ce3d95abc4f9d4c9db4cec43c94R823-R829

Are you receiving valid requests with invalid URIs?
I understand the sentiment of wanting to receive all requests, however there are many cases where invalid requests are dropped since there is no way to make sense of them. Previously mummy did not decode the URI before doing routing, which would break for things that included spaces in routes for example, eg "/my path/" would not match "my%20path".
In this case, an invalid URI cannot be properly routed. Calling the errorHandler is reasonable, but I am missing some understanding of a real concrete scenario to base this around. It could appear to be a bunch of pluming and statefulness for a request that no client ever sees?
For example, a request with an invalid chunked encoding is dropped. There is no way to make sense of the body. Passing that invalid encoded body to a handler and expecting it to figure it out seems very unexpected. Each handler would need some sort of "is the body actually valid" check? Is the invalid body stored somewhere other than the valid body? Where is the valid-ness made available? etc. Same goes for .path and URI.
Having a good solid real-world situation will I think clarify what is best to do here -- eg is errorHandler actually good or is maybe path: Option[string] better than path: string so you can know if it isn't present it was not valid?

Relevant: https://github.com/golang/go/issues/21548

Thanks for the feedback and the Go-link, Ryan!
Maybe it's just a me-use-case :) ? I've been accustomed to receiving the requests and managing them all, so suddenly not having that possibility was the origin of the issue.
I don't have a use case for using the test=% in the example above; it's more about the backend management:
1) Log management and incident tracking:
Receiving a 502 error in the Nginx logs raises a flag when it can't be matched with the internal logging system. Is the application down, or was it a bad request? Having authenticated the user and parsing the url /api/v1/user-finance-options?iban=xx&swift=% should raise a red-flag and provide tracking info (instantiationInfo()) and not just a 502.
2) Identifying bad inputs:
Of course, previous fuzzing and testing should have taken care of user inputs, buuuut if something was missed, allowing the request to pass through would enable me to determine whether it was actually a bad request. If there are multiple parameters, some good and some bad, then I would prefer to handle the good ones and just log the bad ones.
And I think you're right, the errorHandler would not be the right place; the Option or a new errorURI/RouteHandler would be better; personally I would prefer a handler. Unless it's just a me-problem ;)


Add way to delete tags

There is no way to delete a tag right now.

https://github.com/CMPUT301F23T01/Sigma-Blue/pull/159


Make set_index(..., sorted=True) handle entirely empty dataframes

[x] Closes #8879
[x] Tests added / passed
[x] Passes pre-commit run --all-files

I think this can be rebased with main, and it shouldn't be difficult to solve conflicts.

Thanks @jorloplaz based on the comments it looks like I was planning to change the approach slightly. Let me see if I can get the new idea working.

@jsignell Did you get a chance to try out the different approach? I'm also happy to take a look next week if it'll help. :)

@jsignell Thanks! I checked this out with @ncclementi. Looks like there's a naming conflict to be resolved, but other than that we agree that this can be merged as is. :)

Ok I fixed the conflict, so should be all good now.

I'll merge main and push to the PR, to fix the failures on CI due to the hanging problem we were having.

I've merged main again, to get #9336 in to fix CI failures. We can merge after CI finishes!


When I set the batch_size larger,the FLOPs became smaller?

When I set the batch_size larger,the FLOPs became smaller,Why?
Please explain it to me.

Hi @hanyuchen8145,
Can you please provide more details? What values did you use, and which method did you try? It would be great if you could show the script and the results you have.

答疑.docx

@Yasir-Ghunaim
First of all, please forgive me for not replying in time. I'm very sorry.
When I type the following,
python main.py
--dataset 'cifar10'
--batch_size 10
--lr 0.05
--lr_type 'constant'
--workers 6
--method 'ER'
--seed 123
--debug
--validation
--profile_flops
--size_replay_buffer 100
I get
fwd MACs per GPU=108.7GMACs,
fwd flops of model = fwd flops per GPU*mp_size=219.31G
But when I increase the size of batch_size, like when I increase batch_size to 50,
for example:
datasetWithReplay = OnlineDatasetWithReplay(
dataset=strategy.adapted_dataset,
mem_size=self.mem_size,
batch_size=50,
# batch_size=strategy.train_mb_size,
gradient_steps=self.gradient_steps,
batch_delay=self.batch_delay,
seed=self.seed)
    strategy.dataloader = DataLoader(
        datasetWithReplay,
        num_workers=num_workers,
        batch_size=50,
        # batch_size=strategy.train_mb_size,
        shuffle=shuffle,
        pin_memory=pin_memory,
        **other_dataloader_args

Why do the values fwd MACs per GPU and fwd flops of model = fwd flops per GPU*mp_size decrease? Is that reasonable?
If it makes sense, can you explain to me why?

@Yasir-Ghunaim
First of all, please forgive me for not replying in time. I'm very sorry.
When I type the following,
python main.py
--dataset 'cifar10'
--batch_size 10
--lr 0.05
--lr_type 'constant'
--workers 6
--method 'ER'
--seed 123
--debug
--validation
--profile_flops
--size_replay_buffer 100
I get
fwd MACs per GPU=108.7GMACs,
fwd flops of model = fwd flops per GPU*mp_size=219.31G
But when I increase the size of batch_size, like when I increase batch_size to 50,
for example:
datasetWithReplay = OnlineDatasetWithReplay(
dataset=strategy.adapted_dataset,
mem_size=self.mem_size,
batch_size=50,
batch_size=strategy.train_mb_size,
gradient_steps=self.gradient_steps,
batch_delay=self.batch_delay,
seed=self.seed)
strategy.dataloader = DataLoader(
    datasetWithReplay,
    num_workers=num_workers,
    batch_size=50,
    # batch_size=strategy.train_mb_size,
    shuffle=shuffle,
    pin_memory=pin_memory,
    **other_dataloader_args
)

Why do the values fwd MACs per GPU and fwd flops of model = fwd flops per GPU*mp_size decrease? Is that reasonable?
If it makes sense, can you explain to me why?

@Yasir-Ghunaim
First of all, please forgive me for not replying in time. I'm very sorry.
When I type the following,
python main.py
--dataset 'cifar10'
--batch_size 10
--lr 0.05
--lr_type 'constant'
--workers 6
--method 'ER'
--seed 123
--debug
--validation
--profile_flops
--size_replay_buffer 100
I get
fwd MACs per GPU=108.7GMACs,
fwd flops of model = fwd flops per GPU*mp_size=219.31G.
But when I increase the size of batch_size, like when I increase batch_size to 50,
for example:
datasetWithReplay = OnlineDatasetWithReplay(
dataset=strategy.adapted_dataset,
mem_size=self.mem_size,
batch_size=50,
# batch_size=strategy.train_mb_size,
gradient_steps=self.gradient_steps,
batch_delay=self.batch_delay,
seed=self.seed)
strategy.dataloader = DataLoader(
datasetWithReplay,
num_workers=num_workers,
batch_size=50,
# batch_size=strategy.train_mb_size,
shuffle=shuffle,
pin_memory=pin_memory,
**other_dataloader_args
Why do the values fwd MACs per GPU and fwd flops of model = fwd flops per GPU*mp_size decrease? Is that reasonable?
If it makes sense, can you explain to me why?

@hanyuchen8145 Thanks for the explanation. This is a great finding.
The primary focus for our FLOPs measurement is indeed to ensure consistency and facilitate comparisons across different methods while maintaining a fixed batch size. The observed changes in FLOPs due to batch size fluctuations are primarily a result of the replay buffer design.
Let's delve deeper into this:

When we start an iteration with a batch size of 'x,' in the initial step, the buffer is empty. Consequently, the 'OnlineDatasetWithReplay' instance returns 'x' training samples.
However, as we progress to iteration 2 and beyond, the buffer contains samples. This leads the 'OnlineDatasetWithReplay' instance to return 'x' training samples in addition to 'x' buffer samples. These combined samples construct a dataset of '2x' samples for training.

It's worth noting that in this initial iteration, 'x' buffer samples are effectively skipped during training. Therefore, as the batch size ('x') increases, more buffer samples are skipped, resulting in a reduction in FLOPs usage. Refer to this table for an example:

As for resolving this issue, if your specific method requires a consistent batch size, there shouldn't be any concerns with the FLOPs calculation since replay is applied uniformly across all methods. However, if your design necessitates varying batch sizes, you can implement a straightforward adjustment. Simply modify line 164 in avalanche_extend/benchmarks/datasets/dataset_wrapper.py from:
return sample, target, original_index, \
            test_sample, test_target, test_original_index, test_task_label, task_label

to:
return sample, target, original_index, \
            test_sample, test_target, test_original_index, test_task_label,
            sample, target, original_index, task_label, task_label

This change ensures that the first iteration will return both training and buffer samples, with buffer samples being copied from the training set, thereby maintaining a consistent batch size and FLOPs calculation. Alternatively, you could also enable this behavior for the first iteration only when the profiler_enabled is true by passing it to ReplayOnlinePlugin and then using it internally. This way the training is not affected.

@Yasir-Ghunaim
Ok, I understand, thank you for your explanation!

Hi@Yasir-Ghunaim,
According to your advice, I simple modify line 164 in avalanche_extend/benchmarks/datasets/dataset_wrapper py
from:
return sample, target, original_index,
test_sample, test_target, test_original_index, test_task_label, task_label
to:
return sample, target, original_index,
test_sample, test_target, test_original_index, test_task_label,
sample, target, original_index, task_label, task_label,
Why the results are still larger batch_size and smaller flops may sound strange, but they are. Do you know the reason

Hi@Yasir-Ghunaim
According to your advice, I simple modify line 164 in avalanche_extend/benchmarks/datasets/dataset_wrapper py
from:
return sample, target, original_index,
test_sample, test_target, test_original_index, test_task_label, task_label
to:
return sample, target, original_index,
test_sample, test_target, test_original_index, test_task_label,
sample, target, original_index, task_label, task_label,
Why the results are still larger batch_size and smaller flops may sound strange, but they are. Do you know the reason

Hi @Yasir-Ghunaim
According to your advice, I simple modify line 164 in avalanche_extend/benchmarks/datasets/dataset_wrapper py
from:
return sample, target, original_index,
test_sample, test_target, test_original_index, test_task_label, task_label
to:
return sample, target, original_index,
test_sample, test_target, test_original_index, test_task_label,
sample, target, original_index, task_label, task_label,
Why the results are still larger batch_size and smaller flops may sound strange, but they are. Do you know the reason

sample, target, original_index, task_label, task_label

Problem solved! Because I made a mistake somewhere else.

Great, let me know if you have any other issues.


Ignore platform_version on EKS Cluster

Description
This PR ignores the platform_version value of an EKS Cluster. Per documentation, AWS automatically updates existing clusters to the latest version for the corresponding Kubernetes version.
Motivation and Context
At first glance, a proposed TF change for platform_version in a plan is a moment of concern. When you later find out that AWS automatically updates the version, then the concern is minimimized (no-op), I suggest ignoring this output from the AWS API.
Note: I considered submitting this idea to terraform-provider-aws but deemed it was inappropriate because this module is opinionated by the fact that it does not allow consumers to set the platform_version. This signals to me that consumers of this module are 'ok' with just letting AWS do the automatic upgrades.
For example, this is the output of a TF plan

Testing
I am unsure how to test this change because new clusters are automatically at the latest version and I no longer have a test case.

this is a computed only attribute - we do not ignore those. I would suggest revisiting your configurations to identify why its showing up in your diffs

@bryantbiggs understood, no problem! Quick followup if you have a moment?

this is a computed only attribute

I believe I understand, you are referring to arguments vs attributes and you are suggesting that ignore_changes is only applicable to arguments.

I would suggest revisiting your configurations to identify why its showing up in your diffs

I'm not sure what you mean by "configurations" - any hints? My understanding as to why it shows up in diffs is because terraform records platform_version in the statefile (confirmed this is true via terraform state show 'module.cluster.module.cluster.aws_eks_cluster.this[0]'). Is my understanding how how a diff is generated not accurate?
I'm very curious now, terraform documentation does specify that ignore_changes operates on attribute names (not arguments, specifically)

I believe I understand, you are referring to arguments vs attributes and you are suggesting that ignore_changes is only applicable to arguments.

Correct

I'm not sure what you mean by "configurations" - any hints?

Based on how you have configured your Terraform, it may have an effect on showing spurrious diffs like this. I can't say though because I don't have a reproduction

Based on how you have configured your Terraform

Right! That's what I'm asking, I have no idea how to configure this. As far as I know, this is default behavior that I'm showing. Maybe, just maybe, this is a bug in the terraform-aws-provider, afterall??

@bryantbiggs We face the same issue. Every time AWS updates the  platform version for EKS clusters, we encounter drift in Terraform:

It would be nice to see it merged

We're facing the very same situation here...
Would being able to fix such platform_version through a new module property be an acceptable fix?
Either way, we still wonder what configuration doesn't show this behavior.
Thanks,

Would being able to fix such platform_version through a new module property be an acceptable fix?

This is not something configurable.
The image shown here https://github.com/terraform-aws-modules/terraform-aws-eks/pull/3142#issuecomment-2320467888 shows that you are using the cluster_platform_version in some way - how are you using it? are you re-wrapping this entire module?


[SPARK-35902][UI] the auto-generated rdd's name in the storage tab should be truncated if it is too long.

What changes were proposed in this pull request?
the auto-generated rdd's name in the storage tab should be truncated  as a single line if it is too long.
Why are the changes needed?
to make the ui shows more friendly.
Does this PR introduce any user-facing change?
No
How was this patch tested?
just a simple modifition in css, manual test works well like below:

Can one of the admins verify this patch?

cc @gengliangwang and @sarutak FYI

Hi @kyoty, could you show a screen shot in the description from which we can see the effect of this change?

ok to test.

Test build #137455 has started for PR 32191 at commit 7312c99.

Test build #137455 has finished for PR 32191 at commit 7312c99.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42030/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42030/

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

@sarutak  the screenshots have been attached, please help to reveive this, thanks!

Test build #137481 has started for PR 32191 at commit 6e16f16.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42056/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42056/

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

I left one minor comment. Otherwise, it LGTM and I confirmed this change works on my laptop.

Test build #137481 has finished for PR 32191 at commit 6e16f16.

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

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

Test build #137504 has started for PR 32191 at commit b2f2ff3.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42078/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42078/

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

Test build #137504 has finished for PR 32191 at commit b2f2ff3.

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

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

@kyoty
I checked whether the GitHub Actions jobs finished successfully but I see the following message.
Actions jobs for this repository have been disabled by GitHub staff. If you are the repository owner, you can contact support via https://github.com/contact for more information.

Do you have any idea of this?
Anyway, I don't think this change affects the GA failure because it's just a style change and Jenkins passes so I'll merge this.

@HyukjinKwon do you have any idea about sarutak mentioned above ? I have never seen this warning before..

Can one of the admins verify this patch?

Test build #137577 has started for PR 32191 at commit b2f2ff3.

@kyoty I think it's better to ask GitHub stuff as the error message suggests.

Test build #137577 has finished for PR 32191 at commit b2f2ff3.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

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

@sarutak  Thanks for your help and review!


proposed API: separate workspaceFolders and configuration

https://github.com/Microsoft/vscode-languageserver-node/blob/master/server/src/protocol.proposed.ts#L11 contains both workspaceFolders and configuration APIs
As workspace folders and configuration are separate features, which are not necessarily coupled, ideally, they would be defined separate propose.d.ts files
E.g. configuration.protocol.proposed.ts and workspaceFolders.protocol.proposed.ts, or in separate node modules.

Done.


Graph Coloring Problem

Related Issue

Info about the related issue

[x] DevIncept Participant

[x] Contributor

Closes: #60
Describe the changes you've made
The aim was to solve Graph Coloring Problem using backtracking. The code is running and is working perfectly.
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Manually run and tested.
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added test cases that prove my fix is effective or that my feature works.

PR approved. As usual, you are doing great. 👏👏@Manasi2001

Thanks for the support. 😊


fix: versions files

Resolves #924

The reference to oci.home provider will be removed from the network module and cluster module with the next PR.


Additional base physics capability

This adds two BasePhysics capabilities as needed by downstream users:

Adds a way to set a generic primal solution field to an initial value
Adds an index-based accessor for the parameter fields

Codecov Report

Merging #1022 (b511adf) into develop (beb0510) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #1022      +/-   ##
===========================================
- Coverage    89.60%   89.48%   -0.12%
===========================================
  Files          141      141
  Lines        11195    11205      +10
===========================================
- Hits         10031    10027       -4
- Misses        1164     1178      +14

Files
Coverage Δ

src/serac/physics/base_physics.hpp
40.00% <0.00%> (-3.48%)
:arrow_down:

src/serac/physics/solid_mechanics.hpp
83.75% <0.00%> (-1.59%)
:arrow_down:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This should be ready to go.

Combined with https://github.com/LLNL/serac/pull/1011


Java 14: jaxb.Persistence: Unsupported class file major version 58

Environment Details

Helidon Version: 2.0.0
Helidon MP
JDK version: 14
OS: MacOS

Problem Description
We have seen an error when building the database-mp archetype with JDK 14.
Also see the error with io.helidon.tests.integration:helidon-tests-native-image-mp-2 when built and run with JDK 14 (see below). These work fine with JDK 11.
[ERROR] Failed to execute goal org.hibernate.orm.tooling:hibernate-enhance-maven-plugin:5.4.6.Final:enhance (default) on project helidon-tests-native-image-mp-2: Unable to enhance class: Persistence.class: Failed to enhance class io.helidon.tests.integration.nativeimage.mp2.jaxb.Persistence: Unsupported class file major version 58 -> [Help 1]

(Blindly guessing that version 5.4.18 will fix it.)

I did a quick test and Laird guessed well. 5.4.18 appears to fix this.
Also when we fix this, we should update the archetype so that it uses the following properties for the plugin versions (these are already defined in the application parent pom):
                <version>${version.plugin.jaxb2}</version>
                <version>${version.plugin.hibernate.enhance}</version>

The one remaining question is: should/must the plugin version track the "main" Hibernate version?  Probably.

Fixed by PR #2212


VCX - Make a comment field as an optional

According to aries rfcs
https://github.com/hyperledger/aries-rfcs/tree/master/features/0036-issue-credential
https://github.com/hyperledger/aries-rfcs/tree/master/features/0453-issue-credential-v2
the comment field is an optional field.
However, libvcx expects this as a mandatary, so if the comment is empty it causes Json parsing error
Caused by: Cannot deserialize A2A message: missing field 'comment'

This PR fixes this issue.

@Artemkaaas , do you agree this should be merged?

@Artemkaaas
although this PR is correct, just heads up about something we run into. This break compatibility between libvcx issuer with this PR merged and holder running older libvcx, without this PR merged. Issuer will issue cred offer without comment, but older libvcx clients will obviously expect it and fail to parse the message.
So issuer with new libvcx version must set comment in order not to break older libvcx clients.


Updated golint import.

ohai! I see you like pull requests, so I put a pull request in your pull request :)
Hoping this resolves https://github.com/matryer/vice/pull/53 so we can get it merged

actually, nevermind, i could just create a new one with your commit :)

Looks like your new PR will do the trick!


Discord group highlights are not being cleaned up for IRC

Example:
[discord] <jleclanche> Paging <@&278128453524520970> <@&281210065581506561>

Support added in master.
But the bot needs to have the "Manage Roles" permission for this to work.  (see Server settings - Roles - General Permissions).


Typing error on "ES6: Create an Export Fallback with export default" lesson.

Describe your problem and how to reproduce it:
The test case instruction has a very minor typo error. It say ""Proper used of export fallback".
However this should be "Proper use of export fallback".
Add a Link to the page with the problem:
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/es6/create-an-export-fallback-with-export-default/
Tell us about your browser and operating system:

Browser Name: Google Chrome
Browser Version: 70.3
Operating System: MacOS

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

I would like to work on this 😄 😄


Fix problem with Entry time type

👋 Hello, I'm trying to use your project (great project BTW), but I keep getting the error below:
zerolog: could not write event: error: loghttp.PushRequest.Streams: []*loghttp.Stream: unmarshalerDecoder: Value looks like Number/Boolean/None, but can't find its end: ',' or '}' symbol, error found in #10 byte of ...|n\"}\n"]]}]}|..., bigger context ...|\",\"message\":\"stopping the application\"}\n"]]}]}|...

After a few hours I discovered that the Loki API want's the value as a string, I tested against Grafana Cloud.

@crossworth, thank you for taking a look a this project, sorry for not checking out this before


 Need to see what's different here. Moving to draft for now

I think this is ready, but I'd like to wait for two things:

Testing with a real license, since I think we saw more failures (that it might actually catch!) there
A fix for the git/git-lfs issue, so I don't have to do the workaround in cc07f10

Good news: when we had the not working UNITY_SERIAL, this retried perfectly. I'm going to wait on the git lfs fix, I think, because I don't like the hack here, but this is ready.

Based on https://github.com/Wandalen/wretry.action/issues/164, I unpacked the dotnet-format action so that we can rerun it on those annoying NuGet failures.
Here's a sample, from the testing I was doing on my fork: https://github.com/mikeage/open-brush/actions/runs/9250440297/job/25443964072


Revert to obscuring output from pytest

Pytest is continuing to leak tokens in URLs when tests fail. This PR reverts to the state where we redirect pytest's output to /dev/null. We should iterate on this and move away from pytest completely.
P.S. there's a weird double commit here because I accidentally committed to master on my fork, then did a cherry-pick but I had untracked files so git complained and made me do git commit --allow-empty. Why does everything break when you're trying to be efficient?

Why does everything break when you're trying to be efficient?

That's a great question! :wink:

I guess after this one lands, we can try to put Farallon in auto-deploy mode again (revert #689), correct?

Yes. It may be best to rotate the keys again before we do that too, though I'm not sure how @yuvipanda did that. It would be nice to have some SRE-style for handling these kinds of scenarios

Yes. It may be best to rotate the keys again before we do that too, though I'm not sure how @yuvipanda did that. It would be nice to have some SRE-style for handling these kinds of scenarios

@yuvipanda, can you comment on that?

It would be nice to have some SRE-style for handling these kinds of scenarios

I agree, @sgibson91.


Missing POST Endpoints

Hi,
I've gone through the official developer documentation (https://developer.xero.com/documentation/api/api-overview) for several of the Accounting API resources. There appears to be mention of the POST method for several endpoints, however the API specification here shows that the PUT method is the only one offered.
An example of this would be the /Contacts#bulk endpoint. Should this endpoint, and others like it, have a 'POST' method implemented?
Thanks.

Hi @toby2g
I've made this change for all endpoints that support bulk create.  The one exception is TaxRates which does not have an ID and uses POST to update TaxRates on the /taxrate endpoint.

Thank you so much!


Migrations: Convert enum values to underlying type

When creating a migration to add an enum column, the namespace of the enum is not included in the resulting migration.
EG:
  migrationBuilder.AddColumn<int>(
                name: "Status",
                table: "CommandRecord",
                nullable: false,
                defaultValue: CommandStatus.Pending);

This results in DNX,Version=v4.5.1 error CS0103: The name 'CommandStatus' does not exist in the current context when running dnx ef database update

This is a bug. We shouldn't be referencing the enum value at all here--it may change over time. Instead we should use the int literal.

@bricelam
Actually it has to be a char, since an enum could have a char value.
Ex.:
    public enum MyEnum
    {
        MyEnumItem = 'A'
    }

To get technical, it will take the underlying type of the enum which, in .NET can be byte, sbyte, short, ushort, int, uint, long, or ulong. In your example, the character literal specified is implicitly converted to int--the underlying type of MyEnum--during compilation.

Marking for re-triage. #4353 perpetuates this issue into the model snapshot.

@smitpatel we had another issue where we decided to just convert default values to the underlying type (rather than storing as the enum value)... so that work should resolve this one too.

The other issue (#4286) was not allowing user to set default value for enums. While fixing it, we removed serialization introduced (which was causing the types to be from a fixed list) and converted it to enum value. Since Migrations also have same issue that user-defined enum is used instead of underlying value (this issue), we decided to fix it in model snapshot together.

So does that mean this one was fixed as part of the other change?

The other one is partially fixed. It allows users to set default values now but it introduce this issue in model snapshot. Since migrations is already facing this issue, model snapshot & migrations can both be fixed together.


Choose legend position

Hi Adam,
Using methylartist version 1.3.1, my legend is automatically put on top of the highlighted segment, which is exactly where I wouldn't like it to be (sorry I can't share an image, as this data has not yet been published). Would it be possible to add an option for choosing where to place the legend within the smoothed methylated fraction plot (for example, "left" / "centre" / "right", or in combination with "upper" / "lower" / "center") rather than the default "best"?
Unfortunately, I need to use the region command because my genomic area of interest is too wide for locus, and ideally I would like to keep the legend in the plot rather than hiding it.
Many thanks for your help!
Silvia

Hi Silvia,
Will look into it. In the near term, does what you're doing need to be automated? If it's a small number of figures what I'd recommend is saving as svg (--svg) and editing in Illustrator/Inkscape or equivalent.
Thanks for using methylartist!
--Adam

Good idea, will do! Thank you for this great tool :)


WIP: feat(i18n): Add i18n support (ttag lib) and support for pt-br

Maybe we should let the use choose the language. In this case, there would be a Language choice in both Settings.js and InitWallet.js; and, if the user has chosen a language, it must supersede OS's language. What do you think? @jansegre @obiyankenobi @pedroferreira1

Maybe we should let the use choose the language. In this case, there would be a Language choice in both Settings.js and InitWallet.js; and, if the user has chosen a language, it must supersede OS's language. What do you think? @jansegre @obiyankenobi @pedroferreira1

I think it's good for the user to choose their language but it would be something extra. Only having the OS language would already be ok but I guess it's not the complicated to add this.

Maybe we should let the use choose the language. In this case, there would be a Language choice in both Settings.js and InitWallet.js; and, if the user has chosen a language, it must supersede OS's language. What do you think? @jansegre @obiyankenobi @pedroferreira1

At least on Android I'm pretty used for apps that use the system language and offer no other choice. In fact I don't recall any app that gives me the option to choose.


I would like to be able to sit in a train chair.

Is your feature request related to a problem? Please describe.
Currently, it is not possible to sit on the train.
You can use the shift key to crouch down, but that is all.
Also, sometimes you get swung off the train with it, so I don't want to use it.
Describe the solution you'd like
Like realtrainmod, you should be able to sit tight on the train.
However, if you can't stand up after sitting down, it is meaningless (in the case of realtrainmod, the moment you press the shift key, you will be shaken off.) I thought it would be better to right-click to sit down, and press the shift key to stand up.
Forced disembarkation should be the same as before.

OK, based on how Minecraft Transit Railway works, this is impossible.
Since train model in the mod are not collision box(just some stiker-like images), the only way to make you move on the train is forcing you moving when train moving (if you use third-person camera, you will see you are actually “running” on the train, not standing).
Translate by Google:
Minecraft Transit Railway の仕組みからすると、これは不可能です。
mod の列車モデルは衝突ボックスではありません (ステッカーのような画像のみ) ので、列車で移動させる唯一の方法は、列車が動いているときに強制的に移動させることです (三人称カメラを使用している場合は、 立っているのではなく、実際に電車で「走っている」)。

I understand. It's a matter of specifications.
I will delete the post.
Thank you.


Clear pk context and other minor changes in *_free() procedures

Description
These changes were made after closer look into existing context freeing and zeroing functions.
pk.c: ctx->pk_info could remain non-zeroed
pem.c: minor optimization
rsa.c and dhm.c : As in coding standards requirements, every code statement should be on its own line
Status
READY
Requires Backporting
Yes
Which branch?
development
Migrations
NO

retest

@RonEld - This looks quick to review. Can you take a look?

Jenkins is failing on the FreeBSD timing test which is a known issue, and as good as a pass.


After updating to react-native 0.40 and react-native-svg 4.6.1 strokeWidth property no longer default to 1

After updating to react-native 0.40 and react-native-svg 4.6.1 strokeWidth property no longer default to 1.  Have to add a strokeWidth='1' to each Svg component.

We will fix that in next version

Fixed in 5.x


Can client ID and secret be the same as the ones for the whole project?

I've been using the OAuth2 library to access the Search Console API, since this API not in the "Advanced Google Services" list. I now notice in my Developer Console that I have two set of credentials, one for the Apps Script project (which is a Google Sheets add-on) and one for the OAuth2 library.
Would there be any problem to use the Apps Script project credentials for the OAuth2 library? Would that mean that when the user installs the add-on and gets the OAuth window, he/she won't need to do the extra authorization step for the Search Console API?

Thanks!

In general we recommend that you don't touch the "Apps Script" client ID, since if it gets deleted or broken there isn't any way to restore it. Even if you did re-use it, you would still need to go through two authorization flows, since the normal Apps Script authorization dialog wouldn't know to ask for the Search Console API scopes.

Actually I did delete the Apps Script client ID :) Luckily, changing to a new Developer Console project recreates the credentials.

That will work, but all your existing users will need to re-authorize the script, so it's a last resort.

Yeah, my mailbox spiked quite a bit :)
Hope that Google either adds more APIs in the Advanced Google services section or allows adding scopes manually to the project to avoid double authorization .


deploy to bluemix button

Hello, thanks for sharing the project, it is very useful. could you add a deploy to bluemix button:
https://console.bluemix.net/docs/services/ContinuousDelivery/deploy_button.html#deploy-button
it would help to very quickly deploy more than 1 app for testing purposes. it would also be great if there would be a web online form to enter the data of the .env and manifest.yml ... hope it helps. thanks & greets.

@stevenpkg Could you provide information on this issue.

The preferred option is to use 'ibmcloud app push' command, We did used to have Deploy to IBM Cloud button, I will think about if we should bring it back.

@stevenpkg is right we don't plan to support the deploy to IBM Cloud


[Addition]: spoofed clusters of heltec miners in Philippines (owner 13SwpgUJHRRvxsj5suiyAoPVeZPUfo5EmBpdHnmsYJVYat8LuRJ)

Hotspot b58 Addresses
11Gd5f66NXZdroqVecfYjxJetBkxzGPBRwgFE2hKEiErfhKs8b5
112gmxZC1spiTFQFGC7B39tvB223HBKqp6kQ9Sb4fhvYSK5pLbKW
11DmxnXsHM34qLG5TAb9irLZj2t2GzmNf4PBmJZUZT5KXEhQP3D
11mEGiVfiWZqH1aDskt7eGSuy5vgRjN3WsRAqKKBXXawtJiUviN
11yj9u3JyDnDwsd7Jp6arHHCZzydfGL1FiiVtKcgjrxtKUnUGgT
112QzKk51G6Hy9DmEvTdkNAwqFDbKyEN7uiJNC3wSa4juM6qSj13
112QzKcQmeKyTPrR6PVhc7gf9QgmQ7esim6Bza6heAJKLXMBBkzR
112ChVJoAMgts5NmU7Ra1W7Fm1zDxLe6HM7oYzdPve7SCWACqahY
1121wcq76Ti2WqMpNFPW5xBLbW8SzUhAHFZPAVCrnwYXkdqtu7gp
Discord Handle
screwdriver#9187
Reason(s)
spoofed cluster of Heltec miners with Hongkong IPs in a remote province in the Philippines

We have taken into account this addition request in the next iteration of the denylist. Some or all of the Hotspots listed may be added to the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Pull case which maybe starts a job when added to its own function

+0 Food for thought: http://number-none.com/blow/john_carmack_on_inlined_code.html


Duplicate tasks in check.yml

The files core/common/tasks/check.yml and core/cluster/tasks/check.yml contain the same tasks. As all roles depend on core/common these tasks are redundant, and can be merged into a single file. Running these tasks twice simply increases runtime, and multiple definitions of the same variable are error-prone.
Duplicate tasks:

"check | Count daemon processes"
"check | Check if daemon is running"
"check | Set default daemon nodename"

@acch Thanks. I was also thinking this but here we will have one issue and that we need to fix. if we are going to merge core/cluster/tasks/check.yml in the core/common/tasks/check.yml   then  core/cluster can not be used without core/common role but then this dependency always required SE package and scale_version variable but currently this independent role is getting consumed by ESS deployment and was also getting used by CNSS team independently without passing SE package or scale version because scale_version variable is not getting used in the core/cluster role. so we will have ton fix this before merging this into single. I also agree to remove this  duplicate tasks. Thank you


MUMUP-2334 : Silent login on load

During loading if a recent login is not detected (last 4h) then it does a silent login via /portal/Login?silent=true.
The response is JSON, which we store in the $sessionStorage.portal object. username, lastLoginMills, and theme.
During a fetch to session.json it gets a different username that is stored in $sessionStorage.portal.username it will throw a redirect to the login page
By default the silent login on load is disabled (as most apps won't do this, but angularjs-portal will)
Documented all new app-config.js entries in configuration.md. Lucky enough, if you don't want to use this feature, then just don't configure any of the new entries.

Lucky enough, if you don't want to use this feature, then just don't configure any of the new entries.

:heart:

:+1:

Sounds like a new, backwards-compatible feature, so a minor version bump might be in order?

Logical. will do @paulerickson


Benchmark GeoIP lookup

[*] Test single thread perf of DB
[ ] Test multiple thread perf of DB

Single threaded performance of MaxMind DB ~68K RPS for random IPs. Performance is acceptable so not testing multithread performance for now.
Script located in ~/scripts/BenchmarkGeoDB.jav


Add files via upload

please add my universty name to your list
Jordan University of Science and Technology
https://www.just.edu.jo/

@razan-mardini cit.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Consider piggybacking on the same requestAnimationFrame for all cbs

Tried it, tricky, couldn't get perf improvement, because you end up shuffling new stuff in memory (in heap in addition to stack?) in order to keep track what's queued and what's cancelled.


WIP: remove pandas dependency from 1.0.0

#79

Coverage decreased (-3.6%) to 93.35% when pulling 90c9fdcd0790dc193787b4b90b0ef8a1b919a49c on arcondello:numpy_response into 82a1f857037be737adf2065ace90cbc8eb490392 on dwavesystems:master.

Coverage decreased (-3.6%) to 93.35% when pulling 90c9fdcd0790dc193787b4b90b0ef8a1b919a49c on arcondello:numpy_response into 82a1f857037be737adf2065ace90cbc8eb490392 on dwavesystems:master.


tox 4.x and tox-pyenv: ImportError: cannot import name 'hookimpl' from 'tox'

Looks like the current release 1.1.0 does NOT play nicely with tox 4.x.
In addition to the bug documented here:
https://github.com/tox-dev/tox-pyenv/issues/21
On adding the "py" package, the next error uncovered is:
(tox) root@9cb1a471f174:~# tox --version
ROOT: No tox.ini or setup.cfg or pyproject.toml found, assuming empty tox.ini at /root
Traceback (most recent call last):
File "/root/tox/bin/tox", line 8, in
sys.exit(run())
File "/root/tox/lib/python3.8/site-packages/tox/run.py", line 19, in run
result = main(sys.argv[1:] if args is None else args)
File "/root/tox/lib/python3.8/site-packages/tox/run.py", line 38, in main
state = setup_state(args)
File "/root/tox/lib/python3.8/site-packages/tox/run.py", line 53, in setup_state
options = get_options(*args)
File "/root/tox/lib/python3.8/site-packages/tox/config/cli/parse.py", line 38, in get_options
guess_verbosity, log_handler, source = _get_base(args)
File "/root/tox/lib/python3.8/site-packages/tox/config/cli/parse.py", line 61, in _get_base
MANAGER.load_plugins(source.path)
File "/root/tox/lib/python3.8/site-packages/tox/plugin/manager.py", line 83, in load_plugins
self._register_plugins(inline)
File "/root/tox/lib/python3.8/site-packages/tox/plugin/manager.py", line 37, in _register_plugins
self.manager.load_setuptools_entrypoints(NAME)
File "/root/tox/lib/python3.8/site-packages/pluggy/_manager.py", line 287, in load_setuptools_entrypoints
plugin = ep.load()
File "/root/.pyenv/versions/3.8.13/lib/python3.8/importlib/metadata.py", line 77, in load
module = import_module(match.group('module'))
File "/root/.pyenv/versions/3.8.13/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 843, in exec_module
File "", line 219, in _call_with_frames_removed
File "/root/tox/lib/python3.8/site-packages/tox_pyenv.py", line 48, in
from tox import hookimpl as tox_hookimpl
ImportError: cannot import name 'hookimpl' from 'tox' (/root/tox/lib/python3.8/site-packages/tox/init.py)

I was able to solve it:
https://warchantua.hashnode.dev/how-to-use-tox-4-with-pyenv-and-poetry


Improve lift documentation to be more intuitive

By using different orders of magnitude, lift's combinatorial nature is a bit more clear

Funny, I thought we'd done this long ago.
It's definitely worth doing.


Inventory profile products for VIPs

Look into + document current profile offerings & plans from Bounties, Gitcoin, Kauri, Meridio

What is the deliverable here, a doc Notion with screenshots, research and conclusions?


Reducer can spend too long on initial pass to remove large statements

Sometimes the largest statements that can be removed are not that large, and the reducer can spend a long time managing to remove them, without making much difference in the size of the shader.
Place a lower bound on the size of a statement for it to be considered worth removing in this phase.

Fixed via #358


libedit: 20180525 -> 20190324

Motivation for this change
New version.
Since determining what changed isn't easy, I made a repo with the
releases for comparing more easily.
For example, changes from version we now have to the latest
(in this PR):
https://github.com/dtzWill/libedit-releases/compare/libedit-20180525-3.1..libedit-20190324-3.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Eval problem appears unrelated.

@GrahamcOfBorg eval


fix concurrency issue in go to definition

If you go back and forth between files quickly using go to definition and go back, then extension features like go to definition, hover, etc. stop working. It gets fixed once you close the file and load it back again.
The following changes fixes the problem.

Oooh nice, thanks. I guess we could make the filename just the UUID but given we shouldn't be near any kind of file length limit I like leaving the name in in case it's ever useful for diagnostic purposes.

Yes I agree, the file name should be there for diagnostic purposes 🙂


add snyk ignores for the node-gyp tar vuln

Fixes #7959
Fixes #7960
Fixes #7961
Fixes #7962
Fixes #7963
Fixes #7964

We're using dependabot to notify us of vulnerabilities. Any chance we can also ignore this specific tar issue there?


Vulnerability roundup 85: micronaut-1.3.0: 1 advisory [9.8]

search, files

[ ] CVE-2020-7611 CVSSv3=9.8 (nixos-20.03)

Scanned versions: nixos-20.03: a84b797b28e. May contain false positives.
Cc @moaxcp

20.03 is frozen and EOL.
Closing issue.


Healing Spirit improperly calculates the number of uses of the healing feature

This implementation of healing spirit calculates the number of uses as min(2, spellMod), but it should be max(2, 1 + spellMod).

This will be fixed on the next update.


online play

I know the title sounds dumb but with mod engine can i just launch er from steam and not worry about being banned for having it installed

Duplicate of https://github.com/soulsmods/ModEngine2/issues/46.


JSON needs to include the quality flag

JSON currently does not output the quality flag.

Pull request include d in #21


DataContainer should probably have a save method

Description
Issue I came across trying to make callbacks which can store calculation state. Best situation would be that DataContainer would have a (possibly abstract) save method which would save the data using an appropriate format. My current workaround is to save the .array using numpy's own saving tools, but that might be very silly with some datasets; could still be a sensible default though.
Environment

23.1.1.dev350+g5d569dc0 g5d569dc0 3.12.7 | packaged by conda-forge | (main, Oct  4 2024, 16:05:46) [GCC 13.3.0] linux

Thanks @manchester-jhellier - This is something we have talked about a lot recently. Our sister package SIRF, does indeed have a save method for data containers.  Definitely something we need to discuss again


ML-260 Initial

Signed-off-by: johnholt<EMAIL_ADDRESS>Initial LDA implementation for topic estimation.  C++ attributes should be reviewed by @jamienoss and @ghalliday  as these are attributes that should be replaced with improved Child Dataset processing Jira https://track.hpccsystems.com/browse/HPCC-14803

@johnholt I have only really looked at the c++.  The code looks solid. Some very minor style comments - it is coded more as old style C than c++, but none are significant.

@gavin please take a quick look.  Mostly using size_t to avoid future size issues as suggested.  Moved declaration to first use.

Ooops.  Closed in error.

@ghalliday changes completed on review comments and a few additional fixes made. Please take another look and let @richardkchapman know if code can be migrated

@ghalliday Over to you

@johnholt a couple of minor comments.  Probably worth squashing the commits before merging.  I suspect the merge of upstream/master should have been a rebase instead.

@richardkchapman @ghalliday Changes made, commits squashed to a single commit.


Jest tests not working with serveStatic

I created a simple project with hono-minimal
I then added
import { serveStatic } from "hono/cloudflare-workers";
app.use("/*", serveStatic({ root: "./" }));
app.use("/favicon.ico", serveStatic({ path: "./favicon.ico" }));

now when I run npm run test I get this error:
    Cannot find module '__STATIC_CONTENT_MANIFEST' from 'node_modules/hono/dist/cjs/adapter/cloudflare-workers/server-static-module.js'

    Require stack:
      node_modules/hono/dist/cjs/adapter/cloudflare-workers/server-static-module.js
      node_modules/hono/dist/cjs/adapter/cloudflare-workers/index.js
      src/index.ts
      src/index.test.ts

https://github.com/lane711/hono-minimal/blob/main/src/index.ts

Hi @lane711 !
You need to add a stub for __STATIC_CONTENT_MANIFEST by referring to the following:
https://github.com/kriasoft/cloudflare-starter-kit/pull/22

Related to #1127

Resolved by #1984


Feature/apple health

What has been done

Switch lat / lon to make gpx files exported from Apple Health work correctly (this will break other gpx files, but I only use those, so for now that's fine with me)
Added basic Dockerfile and docker-compose for easy usage

Sorry, this was not ready as a PR, was only a small change so my use case was working

Sorry, this was not ready as a PR, was only a small change so my use case was working


Gevent / invoke compatibility & support for Match exec SSH config

Describe the bug
Origin: https://github.com/Fizzadar/pyinfra/issues/569
Essentially gevent monkey patching of thread & subprocess modules breaks using invoke, which paramiko uses to implement Match exec statements in SSH config.
This looks the same as: https://github.com/gevent/gevent/issues/1268
To Reproduce
Put the following in SSH config:
Match exec echo

Then try to use pyinfra with a SSH target and get lots of error spam w/:
BlockingIOError: [Errno 35] Resource temporarily unavailable

Expected behaviour
It shouldn't error! Unsure on how to fix this, or whether it's possible.

I figured out a fix for this - it's pretty horrible but gets the job done, and beats the current exception that is raised: https://github.com/Fizzadar/pyinfra/commit/a7964e525a9c632b1d7b515c6a85cdaf749c342c.
Released in v1.6.2.


JCR_ROOT Path Breaks Upon Project Import

Create AEM project using Developer Tools for Eclipse, Maven Archetype 7.
Delete project from Eclipse, but do not delete contents on disk.
File->Import->Maven->Existing Maven Projects, select the project you created in step 1.
Look at 'problems' tab, and note that jcr_root is not correct.
Look at ui.apps project properties, note that path under 'AEM' tab is not correct (only has jcr_root).
Uncheck 'Content Project' facet in project properties.
Rick click on ui.apps, Configure->Convert To...Content Project.
Check Problems & 'AEM' tab, note that path is now set correctly.

Yup, it's a bit awkward now to set up. Also tracked in SLING-3766 - Store the project's content sync root property in the filesystem

This issue has been fixed in the Sling IDE Tooling project ( see SLING-3766:Store the project's content sync root property in the filesystem ) and will be incorporated in the next AEM Developer Tools public release.


Unbuffered output needed?

System-nvidia-smi script
At first I tried your version and it literally printed $1 each iteration, like the piped awk didn't handle well quotes and doublequotes on output variables:
# $1 %
# $1 %
...

If I only kept "$1" without the labels it would work, but I still needed custom labelling, and multiple print parameters brought buffer problems (the output didn't even show up on polybar).
The only way I made this work was by emulating python -u with fflush() (not so efficient):
nvidia-smi --loop=10 --query-gpu=utilization.gpu --format=csv,noheader,nounits | awk '{ print "GPU",""$1"","%"; fflush() }'
Now I'm interested, what is wrong with my setup? Something having to do with python maybe?

I'm working on this right now.
@Neflux
Did you find out why you need fflush()?
Your script works fine for me with fflush or without it but maybe I should keep it for compatibility.
@x70b1
I was thinking about including other queries, like fan.speed or memory.used as comments inside the script. And then every user can duplicate the module with the queries he wants using the script as template. Is that okay?

I tried to remove fflush() in my script, and while the output doesn't change on console (if I try to launch the single script), it doesn't show up in my polybar anymore, so I think I have to keep it.
This is my module config anyway:
[module/system-nvidia-smi]
type = custom/script
interval = 0.2
exec = ~/.config/polybar/scripts/system-nvidia-smi.sh
tail = true

That .sh file only contains the line of code I previously wrote here.

You're right. I had not tested with polybar yet.


Feature requuest: add support for Craph QL

I've been playing around with the Craph Ql plugin, any thoughts about adding Imager support?

As far as I can tell, this is something that Mark has to build in support for in CraftQL. It might be that you can use imager by enabling the useForNativeTransforms config setting, but that depends on how transforms were implemented in CraftQL.

Thanks, I commissioned Mark to do this since opening this issue, it's now in the imager branch https://github.com/markhuot/craftql/tree/imager


Database connection timeouts when called via shell/queue.php

For my site, I have multiple queue workers using Beanstalkd as a queue backend, and I start them with shell/queue.php --watch [queue-name].
These queues seem to work fine, but after some time (I'm guessing after MySQL's wait_timeout has been reached on the database connection), there's an exception logged:
Next Zend_Db_Statement_Exception: SQLSTATE[HY000]: General error: 2006 MySQL server has gone away in /var/www/html/lib/Zend/Db/Statement/Pdo.ph
p:241
Stack trace:
#0 /var/www/html/lib/Varien/Db/Statement/Pdo/Mysql.php(110): Zend_Db_Statement_Pdo->_execute(Array)
#1 /var/www/html/app/code/core/Zend/Db/Statement.php(291): Varien_Db_Statement_Pdo_Mysql->_execute(Array)
#2 /var/www/html/lib/Zend/Db/Adapter/Abstract.php(479): Zend_Db_Statement->execute(Array)
#3 /var/www/html/lib/Zend/Db/Adapter/Pdo/Abstract.php(238): Zend_Db_Adapter_Abstract->query('SELECT COUNT(*)...', Array)
#4 /var/www/html/lib/Varien/Db/Adapter/Pdo/Mysql.php(428): Zend_Db_Adapter_Pdo_Abstract->query('SELECT COUNT(*)...', Array)
#5 /var/www/html/lib/Zend/Db/Adapter/Abstract.php(825): Varien_Db_Adapter_Pdo_Mysql->query('SELECT COUNT(*)...', Array)
#6 /var/www/html/lib/Varien/Data/Collection/Db.php(225): Zend_Db_Adapter_Abstract->fetchOne(Object(Varien_Db_Select), Array)
#7 /var/www/html/app/code/local/Model/Event.php(122): Varien_Data_Collection_Db->getSize()
#8 /var/www/html/app/code/local/Model/Queue/ReportWorker.php(52): Model_Event->sendEvents(1)
#9 /var/www/html/app/code/community/Lilmuckers/Queue/Model/Queue/Task.php(427): Model_Queue_ReportWorker->sendEvents(Object(Lilmuckers_Queue_Model_Queue_Task))
#10 /var/www/html/app/code/community/Lilmuckers/Queue/Model/Queue/Abstract.php(214): Lilmuckers_Queue_Model_Queue_Task->run()
#11 /var/www/html/shell/queue.php(194): Lilmuckers_Queue_Model_Queue_Abstract->runTask(Object(Lilmuckers_Queue_Model_Queue_Task))
#12 /var/www/html/shell/queue.php(60): Lilmuckers_Shell_Queue->_watch(Array)
#13 /var/www/html/shell/queue.php(231): Lilmuckers_Shell_Queue->run()
#14 {main}

It seems that there was a fix for this issue in 2014 (see these lines specifically, moving the db connection handling outside of the while() loop: https://github.com/lilmuckers/magento-lilmuckers_queue/commit/715c75a786e918024a6b97bb6fdf1eea350ce315#diff-06ed6044f3f923b9c36f7a31eb80e753R180), which was part of the issue: https://github.com/lilmuckers/magento-lilmuckers_queue/issues/5
The code for the db connection handling seems to have moved back inside the while() handler since the fix was applied (see: https://github.com/lilmuckers/magento-lilmuckers_queue/blob/master/shell/queue.php#L183-L210), but I'm wondering if that was an accidental reversion?
Or might I just be reading the new code wrong?

So the old code closed the connection as soon as the process ran - and then tried a reconnect when the process failed. The problem with that was that every iteration through the while loop (depending on the beanstalkd timeouts) was closing and reopening the DB connection - which was causing issues further down the pipeline with MySQL when you scaled out the workers.
Basically it caused the main magento stack to start shitting itself.
The most recent change was watching for these disconnect events, throwing an exception for them - and reconnecting based on that exception event. The idea there being that instead of constantly flipping the MySQL connection - it's only being flipped when absolutely required.
The exception is logged either way - so it the process failing, or is it just filling up the exception logs and proceeding as expected?
If it's dropping the mysql connection completely - the solution could be to adjust how it's catching the "went away" exception and triggering the reconnect logic.

It seems like after observing a few threads that I ran seperately, the code is doing what it should—after wait_timeout is reached, I see an exception in the log, and I see in the database there’s a new connection set up (using SHOW FULL PROCESSLIST;).
Since we have a number of workers per queue, and our wait_timeout is 3600 sec/60 minutes, there are a number of these exceptions logged.
Do you recommend setting a much higher wait_timeout, or are the exceptions mostly harmless in your experience?

Unless i'm misunderstanding - it sounds like it's doing precisely what is expected:
IE: Connecting, processing, then when it dies - reconnecting.
Extending the wait_timeout on the MySQL side in my experience is not the best idea, because most of your DB load (at scale) will be your customer connections - and a longer wait_timeout will result in your connection count staying at an unreasonably high level - requiring more resources thrown at your SQL cluster.
That error is an expected error - because the connection is potentially idling for a long time during processing tasks and waiting for queue items and such. Which is why it's meant to recover "invisibly" from it (but, as i say, it does log the error).

Yeah, I did some more testing with the queue watchers today, and can confirm that when I start them all at once, I can see zero of the 'gone away' errors for the full wait_timeout period (60m in my case), and after that, I start seeing individual workers throwing the 'gone away' exception here and there (as some queues are idle that whole time, while others get jobs to process and so aren't idle and don't necessarily hit wait_timeout until they're idle later).
I'm going to close this issue, as I think it's more of a 'known non-bug'—I wonder though if there's any way it could be handled more gracefully, as any persistent queue watcher that's not restarted regularly would run into wait_timeout at some point (assuming the queue doesn't get any jobs within that period of time). Though with MySQL's default of 28800 (8 hours), it's likely that most queues would have a job within any given 8 hour window (more so than with a 1 hour window), so wouldn't run into the exception as often.


Ability to Toggle Rainbow Brackets on/off

As a VZCode user, I want to be able to turn off Rainbow Brackets if it's annoying.

This could go into the Settings modal


Polymorphic Example Not working

Hi there.
I did everything like your example but i didnt manage to make it work.
This post could not be saved. Please try again.
Is that the correct structure of the array?
Array
(
    [Image] => Array
        (
            [0] => Array
                (
                    [model] => PropertyGroup
                    [attachment] => Array
                        (
                            [name] => zinwn-01.jpg
                            [type] => image/jpeg
                            [tmp_name] => C:\xampp\tmp\php6F.tmp
                            [error] => 0
                            [size] => 71437
                        )

                )

        )

)

Is it because iam using xamp?

Closing as there hasn't been movement on this in months. Please comment if you are still having this issue.


feat: Integrate OpenTelemetry for Distributed Tracing

Integrated OpenTelemetry for distributed tracing in the Golang application.
Configured the OpenTelemetry SDK to send traces and spans to Sentry for improved monitoring.
Added necessary dependencies and initialization code for OpenTelemetry.

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

57 of 63 (90.48%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 83.38%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/mycarehub/application/common/helpers/helpers.go
20
26
76.92%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
17674

Relevant Lines:
21197

💛  - Coveralls


make highlights tappable

highlights of inline feedbacks are now tappable, allowing them to be edited in the related file
(Artemis Servers are finally live again, you need to switch to token based authentication in order to test!)

https://user-images.githubusercontent.com/79016456/210264998-742a7fd7-54c2-48e6-93f8-10c43b8d1f6d.MOV

Also, it's impressive that this just works - well done!

I'm slightly unhappy that there are two places where a EditFeedbackView is used (and that there is no central place that manages showing it), but I don't see any besser solution than this myself

I mean we could create a separate ViewModel that manages this sheet, but I don't think this would help for readability 🤷♂

Besides, there is a bug that if you edit a multi-line comment it will somehow split the comment into two or maybe more. Please take a look at it.

I can't reproduce that bug. Can you specify when exactly it occurs?

I can't reproduce that bug. Can you specify when exactly it occurs?
Im not able to reproduce it myself anymore 😅. So maybe it's magically fixed, very rare or I just did something weird. So this is fine then.

I tried around a lot now, but the ListView won't get updated if I make changes in a single feedback of the AssessmentResult. I think it has to do with how the ForEach decides to redraw its content, as it actually redraws stuff when I change the UUID of the edited Feedback or when a feedback gets added.
That's why I now use the hash of the Feedback as Identifier for the ForEach, which works perfectly fine and requires barely any changes.

Will still do the requested change with just passing the id

Looks way better now and exactly like I imagined it to be implemented. Good job. I will test and take a more detailed look tomorrow and then approve.

I tried around a lot now, but the ListView won't get updated if I make changes in a single feedback of the AssessmentResult. I think it has to do with how the ForEach decides to redraw its content, as it actually redraws stuff when I change the UUID of the edited Feedback or when a feedback gets added.
That's why I now use the hash of the Feedback as Identifier for the ForEach, which works perfectly fine and requires barely any changes.

Probably because ForEach redraws are very expensive they will try to reuse older already rendered views. So this solution should definitely be fine.


bloxstrap bootstrap not showing

my bloxstrap  bootstrap is the normal roblox loader please help

https://github.com/pizzaboxer/bloxstrap/wiki/Switching-between-Roblox-and-Bloxstrap


Modify UX/Output

https://issues.redhat.com/browse/OSD-5676

@katherinelc321 Please wait for the bot to merge PRs

@iamkirkbater , didn't squash in the end :(

Can we revert this and squash?


update api docs on Placement.Constraints for services

Signed-off-by: Lars-Magnus Skog<EMAIL_ADDRESS>- What I did
I noticed that when inspecting a service with constraints, the output of Placement doesn't correspond with the api docs, i.e. Placement is not an array of constraints, but rather contains a Constraints property which holds the array of constraints.
- Description for the changelog
Document Placement.Constraints as the array of constraints and give some simple examples of what it can look like.
- A picture of a cute animal (not mandatory but encouraged)

Thanks @ralphtheninja
Could I have one question that if the spaces in "node.role == worker" are necessary?

Thanks @ralphtheninja
Could I have one question that if the spaces in "node.role == worker" are necessary?

I don't think they are, it's just the output you get when doing docker service inspect foo.

@allencloud Or are you requesting me to remove them? :)

It looks like this is only updating an old version of the API docs. What's the process for updating the current version? I believe it's generated by swagger now.

@aaronlehmann I'd like to know that myself. This was the only thing I could find about the api. I'd be happy to move the changes to the correct place.

The current API docs are in swagger indeed (https://github.com/docker/docker/blob/v1.13.0/api/swagger.yaml#L1934-L1941, and https://github.com/docker/docker/blob/v1.13.0/api/swagger.yaml#L2047-L2056), but I checked those, and I think they are already correct (https://docs.docker.com/engine/api/v1.25/#operation/ServiceCreate)
(although no "example" data is shown, not sure if that's possible in the swagger specs)

Thanks again for your answer. @ralphtheninja
I am just having some confusion about that. Now more clear. Thanks.


remove force_static from textinput module

Summary:
changelog: [internal]
force_static doesn't need to be in here, let's remove it.
I change one module per diff. It makes it easier to land it and pinpoint where build failures are coming from.
Reviewed By: christophpurrer
Differential Revision: D56678530

Merged as https://github.com/facebook/react-native/commit/bb120ff663ae56ca825b7b32873127ea91d52720


Clarify inputs/outputs on IBM Circuit Function page

URL to the relevant documentation
https://docs.quantum.ibm.com/guides/ibm-circuit-function
Select all that apply

[ ] typo
[ ] code bug
[ ] out-of-date content
[ ] broken link
[X] other

Describe the fix.

The Overview section talks about abstract PUBs as the input and links to PUBs description under primitives. But it doesn't say anything about what abstract PUBs mean (e.g. how it's different from what Qiskit Runtime Estimator takes).
There is no explicitly section on output format. It's only mentioned under the example.

Will check in with author of page!


Use a deployment based enum in the node configuration to indicate the genesis block to use.

Currently each node loads the genesis block from a serialised hex string contained in the node's config.yaml. An improvement, suggested by @Geod24, is to use an enum in the config.yaml and load the correct genesis block from the binary executable.

For reference, it doesn't have to be in the config. But we should guard against misuse. One such misuse is to use a mainnet key on testnet. Another is doing some testing with a node, then restarting it to be on Mainnet, but not wiping out the cache.

How about we use a folder name for each network. For example:
mainnet/config.yaml
mainnet/.cache
testnet/config.yaml
testnet/.cache

That would work indeed. But also, having different keys gives us free replay protection.

Yes each config.yaml has a different GenesisBlock.

What I mean is that, if the keys have the same values, you could end up accidentally using the same key on both network.
That's why most blockchains have this scheme.


CUDA_VISIBLE_DEVICES=0,1,2,3  python -m torch.distributed.launch --nproc_per_node=4 --master_port=${PORT:-300}    tools/train.py --config configs/foodnet/SETR_Naive_768x768_80k_base_RM.py  --work-dir  checkpoints/SETR_Naive_ReLeM  --launcher pytorch

Again, I have a question about the process of train, when I use your guidline ,and  there are some error:

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.

Traceback (most recent call last):
File "tools/train.py", line 167, in
main()
File "tools/train.py", line 98, in main
init_dist(args.launcher, **cfg.dist_params)
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/mmcv/runner/dist_utils.py", line 20, in init_dist
_init_dist_pytorch(backend, **kwargs)
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/mmcv/runner/dist_utils.py", line 34, in _init_dist_pytorch
dist.init_process_group(backend=backend, **kwargs)
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py", line 422, in init_process_group
store, rank, world_size = next(rendezvous_iterator)
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torch/distributed/rendezvous.py", line 172, in _env_rendezvous_handler
store = TCPStore(master_addr, master_port, world_size, start_daemon, timeout)
RuntimeError: Permission denied
Traceback (most recent call last):
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torch/distributed/launch.py", line 261, in
main()
File "/home/dongxiaoxiao/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torch/distributed/launch.py", line 257, in main
cmd=cmd)
subprocess.CalledProcessError: Command '['/home/dongxiaoxiao/anaconda3/envs/open-mmlab/bin/python', '-u', 'tools/train.py', '--local_rank=3', '--config', 'configs/foodnet/fpn_r50_512x1024_80k_RM.py', '--work-dir', 'checkpoints/FPN_r50_RM', '--launcher', 'pytorch']' returned non-zero exit status 1.
And I remember that about 20 days ago,I saw some said "--launcher pytorch" may cause some questions, but I don`t know, hope your reply ,Thanks a lot!

@Mark1Dong sorry for replying late since I am super busy recently. Can you first paste your environment (OS, GPU etc.)?

Thanks for your reply, and I have solved this question. The main problem is the command-line format

also , for the port, '-300' is not suitable, so I change the port to ' -36900' ,and the question is solved


[Bug]Bosses not despawning after Tesla Invasion

What Title says, Tesla invasion ended around 11:32pm Eastern and quite a few of the bosses remained.

I've not seen this happening since, so going to close.


SQL Server | Linked Server

Didn't work for me with user DSN, only with system DSN.
The regedit does not seem to have any impact

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

ID: 96736d22-b3e0-290d-3615-5468859d0c9c
Version Independent ID: c13dfbf7-0335-586d-2a04-d9d05ddb1f3a
Content: Connect to Azure Cosmos DB using BI analytics tools
Content Source: articles/cosmos-db/odbc-driver.md
Service: cosmos-db
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@dudu-markovitz Thank you for bringing this to our attention. We will review this for accuracy and update the documentation accordingly.

@dudu-markovitz Can you provide specifics as to what did not work with regard to User versus System DSN? I have no System DSN entries yet all the testing I have done does not indicate issues with using User DSN.

@dudu-markovitz Spent a little bit more time looking at this, and believe you are correct bit also, the doc does indicate system data source already. It is subtle but is does imply System DSN:

I also did want to show that User DSN functions when not creating a Linked server connection:

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


ToC broken

The ToC link to get to this page ("Frequently asked questions" under Concepts) is broken.
It goes to this link: https://docs.microsoft.com/en-us/azure/purview/frequently-asked-questions.yml
Please have that fixed so that it points to https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/frequently-asked-questions

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

ID: 8e54b028-10da-903b-9c18-5b6173e6bfa3
Version Independent ID: e879342f-0169-500f-3e47-66c947990045
Content: ARM template frequently asked questions
Content Source: articles/azure-resource-manager/templates/frequently-asked-questions.yml
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@dcbrown16 Thanks for the feedback. We shall evaluate and update the document as appropriate.

@dcbrown16 - thanks for making us aware of this issue. I have submitted a fix for the links. It should get published today.
#please-close

@tfitzmac Thanks for the update!
@dcbrown16 The changes are now live in the original doc.
We will now proceed to close this thread as resolved, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


different results on colab and anaconda on windows

Hello,
I am getting accurate results on google colab but wrong results on anaconda installed on windows. Issue on windows is I am getting wrong results (though getting correct on colab). Although I tried simple installation in different environment.
Can you please help me how I can install it correctly on windows.

I am using tf 2.2.0.
Could you delete .pkl file manually and run find function again? btw, I don't understand what is wrong in the command prompt.

Here you can find the keras discussion having same trouble: https://github.com/keras-team/keras/issues/4875
Some solved the trouble with uninstalling and reinstalling keras, tf and h5py.

closed due to inactivity

@serengil
This has worked for me for saving and loading weights
use model.to_json() to save your architecture config only without weights, and training configuration
save weights using model.save_weights()
load model using model_new = tf.keras.models.model_from_json(model_json)
compile model using model_new.compile()
load weights using model_new.load_weights()
After doing above, I was able to get the same level of accuracy

@yuh8, where you done these changes?


Q: How to see all issues?

How can I see which 4 issues I have to fix for the following result?
>>>> Newer Versions available <<<<
 Namespace         Name                                         Kind           API Version (Current Available)   Replace With API Version (Latest Available)   Migration Status
 my-namespace   my-ingress                                 Ingress        networking.k8s.io/v1beta1         networking.k8s.io/v1                          4 issue(s): fix issues before migration

If it's too much to print detailed output in this tabular form, maybe we can have a different output format (e.g. json)?

@wrdls issues are present in the table below this table, currently json output is not supported though data is already in a format so that it can be printed in json easily. I will try to pick it up if you would have bandwidth then can you pick it up?
Thanks

Oh, I see. Not sure why I missed that those two tables are related.
About JSON output, I wouldn't mind getting involved but I'm not sure when I'd be able to work on this.


TestAssertionFailure should implement Error

I'm trying to incrementally adopt Googletest. I have a test helper that returns anyhow::Result<()> and I want to start replacing its inner logic with verify_that!(...)?. verify_that! doesn't return an anyhow::Result of course, and usually in Anyhow I solve that by slapping a .context() call onto the end. But this doesn't work:
 2  error[E0599]: the method `context` exists for enum `Result<(), TestAssertionFailure>`, but its trait bounds were not satisfied
     --> src/test.rs:1297:57
      |
 1297 |             verify_that!(notif.status, eq(want_status)).context("assertion failed")?;
      |                                                         ^^^^^^^ method cannot be called on `Result<(), TestAssertionFailure>` due to unsatisfied trait bounds

So, basically you cannot mix TestAssertionFailure with other types of error (be it via Anyhow or just like Box<dyn Error> or whatever).
Maybe truly orthopraxic  (just added that word to my vocabulary, thanks ChatGPT) test helpers wouldn't wanna do this and the fact that I have this problem is a sign that I haven't properly absorbed the googletest-rs doctrine? Like, when something that isn't exactly a test failure goes wrong, instead of returning errors in the failure case I should be failing an assert_*?
But, it seems unnecessary to enforce that doctrine? And maybe there are more important reasons that I haven't thought of why TestAssertionFailure should implement error?

Hmm, or maybe the solution here is to go the other way around: instead of getting TestAssertionFailure into a std::Result<_, E: std::Error> you should just squash your non-googletest result into a googltest result using IntoTestResult?

This is the same reason why anyhow::Error does not implement Error (see https://github.com/dtolnay/anyhow/issues/153).

Oh yeah I actually discovered that after posting this issue, I should have updated it!

I like to think of googletest::Result<> as the anyhow::Result<> for tests and testonly code.

That sounds sensible to me. I think the most important thing Anyhow does is the Context trait which allows you to tack on extra info as you propagate errors, so you end up with error messages like the ones Go code tends to produce ("couldn't do the thing: couldn't read the config file from /path/to/config: no such file or directory") instead of the ones C++ code tends to produce ("no such file or directory" or "couldn't do the thing"). Not sure if googletest::Result already has something like that?


How should the password be encrypted in the redisson.yml

How should the password be encrypted in the redisson.yml
Like using jasypt to encrypt, but it doesn't work
         password: ENC(bupn8NNqnG8WVA8DyeBw11G6doJQoPd3sEt/UYtGzqSeh1AGFSArenBF7oCaDddl)

passwords aren't encrypted


Mention the requirement to install iron-input using bower

The documentation skips this part and hence if someone is following the documentation to the point they are unable to get the final step.
Screenshot:

This issue was moved to Polymer/docs#1237


Improve releasing during 0.8.2

Thanks to @wingyplus's excellent improvements to the Elixir SDK, we are now able to make it part of our release process. We are now able to produce the following SDK releases at the same time:

🐹 Go
🐍 Python
⬢ Node.js
🧪 Elixir

🎉🎉

Push a fix for incorrect env name in Elixir.

Thank you @wingyplus!


huge repository

bfg --strip-blobs-bigger-than 1M g2

Using repo : /Users/druf/projects/g2/.git

Scanning packfile for large blobs: 11974
Scanning packfile for large blobs completed in 520 ms.
Found 3 blob ids for large blobs - biggest=3230456 smallest=1514634
Total size (unpacked)=6261188
Found 461 objects to protect
Found 3 tag-pointing refs : refs/tags/3.0.4, refs/tags/3.0.5, refs/tags/3.0.6
Found 72 commit-pointing refs : HEAD, refs/heads/master, refs/heads/tests/add-nodejs-10, ...

Protected commits
-----------------

These are your protected commits, and so their contents will NOT be altered:

 * commit d6fe44eb (protected by 'HEAD') - contains 1 dirty file :
	- demos/data/top2000.json (3,1 MB)

WARNING: The dirty content above may be removed from other commits, but as
the *protected* commits still use it, it will STILL exist in your repository.

Details of protected dirty content have been recorded here :

/Users/druf/projects/g2.bfg-report/2018-10-11/14-25-23/protected-dirt/

If you *really* want this content gone, make a manual commit that removes it,
and then run the BFG on a fresh copy of your repo.

Cleaning
--------

Found 1509 commits
Cleaning commits:       100% (1509/1509)
Cleaning commits completed in 1.917 ms.

Updating 30 Refs
----------------

	Ref                                           Before     After
	-----------------------------------------------------------------
	refs/heads/master                           | fdaa120a | e25f1695
	refs/heads/tests/add-nodejs-10              | d6fe44eb | f8dc015b
	refs/remotes/origin/feat-statGrammar        | a70acb51 | 92488e9a
	refs/remotes/origin/fix-axisTitle           | decf238a | 36eea723
	refs/remotes/origin/fix-legend              | fdaa120a | e25f1695
	refs/remotes/origin/master                  | fdaa120a | e25f1695
	refs/remotes/origin/refactor-axisController | 3ee1e5f5 | c93618f4
	refs/remotes/origin/tests/add-nodejs-10     | d6fe44eb | f8dc015b
	refs/tags/3.0.1                             | 876ca619 | cd099540
	refs/tags/3.0.10                            | 6ad57f47 | 37df6d14
	refs/tags/3.0.12                            | ebfb7d27 | 6f3d99f7
	refs/tags/3.0.2                             | 990c7184 | b38f8fa5
	refs/tags/3.0.3                             | f1330102 | 00037413
	refs/tags/3.0.4                             | 9ce96f26 | 41d389c9
	refs/tags/3.0.5                             | 8f632f4a | 08ade9a7
	...

Updating references:    100% (30/30)
...Ref update completed in 45 ms.

Commit Tree-Dirt History
------------------------

	Earliest                                              Latest
	|                                                          |
	........DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD

	D = dirty commits (file tree fixed)
	m = modified commits (commit message or parents changed)
	. = clean commits (no changes to file tree)

	                        Before     After
	-------------------------------------------
	First modified commit | 1ae5279e | 48573e89
	Last dirty commit     | decf238a | 36eea723

Deleted files
-------------

	Filename       Git id
	---------------------------------------------------
	heatmap.png  | 7cce8196 (1,4 MB), acdfa5a6 (1,4 MB)
	top2000.json | 7517a259 (3,1 MB)

In total, 2610 object ids were changed. Full details are logged here:

	/Users/druf/projects/g2.bfg-report/2018-10-11/14-25-23

BFG run is complete! When ready, run: git reflog expire --expire=now --all && git gc --prune=now --aggressive

deleted files:
7cce819697a78989b1c9b09f973edc9dc88dbe2c 1516098 heatmap.png
acdfa5a676a48ec44af8560fe5467ca0a6dabef0 1514634 heatmap.png
7517a259182d04be6ee25c5980e2244b1535e38b 3230456 top2000.json

In version 4.0, we will remove some dependencies and support packaging on demand.


SAK-45336 Assignment resubmission attempt after due/accept until date

https://sakaiproject.atlassian.net/browse/SAK-45336

20.3p 33d863816d


Support int and uint as datatype for template formatting

Fixes GH-9679
Signed-off-by: Jacques-Etienne Beaudet<EMAIL_ADDRESS>

Converted as a draft until I think about https://github.com/prometheus/prometheus/issues/9679#issuecomment-962161263

Thanks for your PR. For legal reasons, we require that all commits are signed with a DCO before we can merge them. See this blog post for considerations around this.
This means that the last line of your commit message should read like:
Signed-Off-By: Your Name<EMAIL_ADDRESS>If you are using GitHub through the web interface, it's quickest to close this PR and open a new one with the appropriate line.
If you are using Git on the command line, it is probably quickest to amend and force push. You can do that with
git commit --amend --signoff
git push -f $remote $remote_branch_for_pr

As always, be careful when force-pushing.

@roidelapluie Done for the DCO, mismatched my professional email for my personal email in the signed off thing, that's why it didn't match before.

thanks!


Error status 1 on mac OS ?

Hi,
I freshly installed pandoc & pandoc-citeproc on my mac using Homebrew and I always get that error message:

$ pandoc --filter pandoc-citeproc --bibliography biblio.bib -s intro.md -o intro.pdf
pandoc-citeproc: error running filter.
"stdin" (line 8813, column 23):
unexpected "\"
expecting letter, white space or "{"
CallStack (from HasCallStack):
error, called at src/Text/CSL/Input/Bibtex.hs:121:32 in pandoc-citeproc-<IP_ADDRESS>-H5BEGFTT3MoA89hux6ES4G:Text.CSL.Input.Bibtex
Error running filter pandoc-citeproc:
Filter returned error status 1

I use macOS Sierra 10.12.6. I tried to reinstall via the .pkg but the error remains. Any solutions ?
Thanks,
Vincent

This error is generated by pandoc-citeproc in trying to
parse your bibtex bibliography.
Try looking around line 8813 of this file, and you may be
able to narrow down the problem.  Note: the error messages
don't always give the most useful locations; the real
problem may occur earlier.
It would be most helpful if you could create a minimal
example -- smallest possible bibtex file that causes the
problem.  (You can also test using pandoc-citeproc -y biblio.bib, which just converts it to YAML and will raise
the same error.)

My .bib file is automaticly updated by Zotero thanks to BetterBibTeX.
This is the part of the file which causes the error:
@comment{jabref-meta: groupstree: 0 AllEntriesGroup:; 1 ExplicitGroup:@Inbox\;0\;; }
Apparently, this is a list of all the folders and subfolders of my collection. Works fine when I remove it. Do you know how to force BBT to remove it when updating .bib file?

I don't know anything about BBT.  However, we should try to improve the parser so that it doesn't have problems with this comment.