We must be able to receive calculation results from wholesale so actors are able to receive the results

EDI needs to be notified about the results for total production, so we know when to generate the message for the actor.
This is a coop story between Mandalorian and Batman. The purpose is to be able to send a message to actors when a calculation has been performed and they request the message. The message will be sent in CIM-XML.
Acceptance criteria

[ ] Once a calculation has been performed EDI is informed. (Mandalorian) This AC is done when this issue is closed: https://app.zenhub.com/workspaces/mandalorian-5fd22a1a59e4740018f5ab5a/issues/gh/energinet-datahub/opengeh-wholesale/690
[ ] When a calculation for total production has been performed and a actor requests the message. A RSM-014 containing the results for total production can be peeked by the actor in the message format CIM-XML (Batman)

Technical notes
Should happen through an integration event that contains sufficient information so that the result can be identified through the wholesale API. (As agreed between architects and dev)
Test
When this story is complete we are able to start a batch for a specific grid and once the batch is ready we are able to peek the results and receive a RSM-014 containing total production.

@MadsBloendAndersen and @Christian-Meyer, just to be clear: I understand AC2 as solely addressing MDR role and "per grid area" result.

@BjarkeMeier Yes. as I understood it was only these results that where ready.


Fix undefined error for showSunTime / showMoonTime in clock module

Fixes #3143 and adds tests for showSunTime / showMoonTime

Codecov Report

Merging #3146 (f207b99) into develop (46bca1b) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head f207b99 differs from pull request most recent head 9975968. Consider uploading reports for the commit 9975968 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##           develop    #3146   +/-   ##
========================================
  Coverage    25.65%   25.65%
========================================
  Files           53       53
  Lines        11481    11481
========================================
  Hits          2945     2945
  Misses        8536     8536

Impacted Files
Coverage Δ

modules/default/clock/clock.js
0.00% <0.00%> (ø)


[Refac] Add pyq noise

This issue includes several tasks for refactoring and adding noise in Qadence:
Merge run and run_dm  in the backends:

[x] Remove the run_dm method from the abstract class Backend, the Horcrux, Bracket and Pyqtorch classes..
[x] Merge the run_dm and run methods in the Pulser backend class.
[x] Modify the unpyqfy method to represent the output density matrix as tensor.
Closes #428

Add the noise gates from pyq:

[x] Add an optional noise parameter to the PrimitiveBlock class
[x] Add an optional noise parameter to the ParametricBlock class
[ ] Update the Primitive and Parametric block instances class to support the noise attribute.
[x] Add pyq's protocols to the Noise Class.
[ ] Modify the convert_block function:
[ ] Add a constructor in pyq:
Closes #423

Add the noise features:

[ ] Create a function to add a noise protocol after every gate in a circuit.
[ ] Create a function to add a noise protocol after a specific type of gate in a circuit.
(Will open an issue)

Closing as superseded by https://github.com/pasqal-io/qadence/pull/563.
@EthanObadia it was simpler for me to create a new branch since this one was failing, and then progressively change things while seeing what each change was making. Feel free to check the new MR and review it + add comments with stuff that still needs to be added.
I will be on holidays now for 3 weeks so someone will take that MR over :)


fix #2: include machinomy/lib/Payment

Merged on my fork in https://github.com/michielbdejong/ilp-plugin-ethereum-asym-server/pull/1

Thanks! OK to merge, as far as I'm concerned (I don't have permission).


Release 0.4.28

@JuliaRegistrator register(branch=release)

Registration pull request created: JuliaRegistries/General/14727
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.28 -m "<description of version>"<PHONE_NUMBER>f705c85b5e5c3d82b951365c90bc51
git push origin v0.4.28


Session Affinity (sticky sessions) – single revision mode

ETA: Public Preview end of March 2023

Are there any plans to support affinity via http-headers? We have a scenario where we need this for non browser environments.

@dersia This is under consideration but not yet planned.
Would you ever need to configure a single app with both header- and cookie-based session affinity?

Hey @anthonychu, thx for the reply. So no, we are not interested in cookie-based sessions, at least not for this project. We want to run perflog on container apps and scale depending on log load, but we need to make sure, that the service that an logging always log to the same instance. We also thought about running our own logic for balancing and routing, but since there is now way to target a specific instance (there is an open issue for this, but so far it looks like it is not even considered). So best would be to have a header rule that will target to a specific instance of the replica.

@dersia This is under consideration but not yet planned.
Would you ever need to configure a single app with both header- and cookie-based session affinity?

For us it would be an either or proposition. Either session affinity by cookie, which is more for front-end scenarios, and one based on headers to have stickiness based on a routing path (managed effectively through a header set by at reverse proxy in front).

@dersia This is under consideration but not yet planned.
Would you ever need to configure a single app with both header- and cookie-based session affinity?

I was thinking a bit more on it, is the session affinity cookie documented somewhere? Would it be possible to set it as a load balancer key so that requests for the same path segment go to the same container, unless it's not available of course.
The idea being that the cookie would instead be set by the load balancer in a deterministic way. Will explore a bit more, but was not able to find any documentation on what is supported.


Why LightGBMClassifier always shows 0.5 AUC?

I'm trying to use LightGBM for a binary classification and this is my code:
import pandas
import numpy as np
import lightgbm as lgb
from sklearn.cross_validation import train_test_split
from sklearn.metrics import auc, accuracy_score, roc_auc_score
from sklearn.model_selection import GridSearchCV
import matplotlib.pyplot as plt

data = pandas.read_csv('./aggregate_data.csv',delimiter=',', dtype=None)

# Create train and test dataset
y = data['ruptured'].values
X = data.drop(['case_id', 'ruptured'], axis=1).values
lables = list(data.drop(['case_id', 'ruptured'], axis=1).columns.values)
# get rid of some nans
#y = np.delete(y, [34,35], 0)
#X = np.delete(X, [34,35], 0)

X = np.delete(X, [68,69,70,71], axis=1)
lables = np.delete(lables, [68,69,70,71], axis=0)

# Split into train and test set
X_train = X
y_train = y

idx = np.array([74,47])

X_train = X_train[:, idx]
lables = np.array(lables)[idx]

# Centre and standardized model
Xmean = X_train.mean(axis=0)
X_train -= Xmean
Xstd = (X_train.var(axis=0))**0.5
X_train /= Xstd

testData = pandas.read_csv('./test_data_detmer.csv',delimiter=',', dtype=None)

# Create train and test dataset
y_test = testData['ruptured'].values
X_test = testData.drop(['ruptured'], axis=1).values

# Centre and standardized model
X_test -= Xmean
X_test /= Xstd

train_data = lgb.Dataset(X_train, label=y_train, feature_name=lables.tolist())

test_data = lgb.Dataset(X_test, label=y_test, reference=train_data)

gbm = lgb.LGBMClassifier(objective='binary',
                          n_jobs=-1,
                          is_unbalance=True,
                          two_round=True,
                          bagging_fraction=0.9,
                          bagging_freq=1,
                          boosting_type='gbdt',
                          feature_fraction=0.9,
                          learning_rate=0.01,
                          min_child_samples=10,
                          min_child_weight=5,
                          min_data_in_leaf=20,
                          min_split_gain=0.0,
                          n_estimators=10000,
                          num_leaves=21,
                          reg_alpha=0.0,
                          reg_lambda=0.0,
                          subsample=1.0)

gbm.fit(X_train, y_train,
        eval_set = [(X_test, y_test)],
        eval_metric = ['auc', 'binary_logloss'],
        early_stopping_rounds = 5)

This code always gives AUC equal 0.5 no matter how hyperparameters are changed. I wanted to know is it something wrong with my code or not?
Environment info
Operating System: RedHat
CPU/GPU model: CPU: Intel Xeon and GPU: NVIDIA V100
C++/Python/R version: Python 2.7
LightGBM version or commit hash:
Error message
/software/anaconda/5.1.0/lib/python2.7/site-packages/sklearn/cross_validation.py:41: DeprecationWarning: This module was deprecated in version 0.18 in favor of the model_selection module into which all the refactored classes and functions are moved. Also note that the interface of the new CV iterators are different from that of this module. This module will be removed in 0.20.
  "This module will be removed in 0.20.", DeprecationWarning)
[LightGBM] [Warning] min_data_in_leaf is set=20, min_child_samples=10 will be ignored. Current value: min_data_in_leaf=20
[LightGBM] [Warning] feature_fraction is set=0.9, colsample_bytree=1.0 will be ignored. Current value: feature_fraction=0.9
[LightGBM] [Warning] bagging_fraction is set=0.9, subsample=1.0 will be ignored. Current value: bagging_fraction=0.9
[LightGBM] [Warning] bagging_freq is set=1, subsample_freq=0 will be ignored. Current value: bagging_freq=1
[1]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5
Training until validation scores don't improve for 5 rounds.
[2]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5
[3]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5
[4]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5
[5]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5
[6]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5
Early stopping, best iteration is:
[1]	valid_0's binary_logloss: 0.498905	valid_0's binary_logloss: 0.498905	valid_0's auc: 0.5

could you provide test_data_detmer.csv?

My question is not related to my dataset. Due to the code that I pasted above, you could see that still I'm in the training stage. So if it helps you I could remove that test dataset evaluation but keeps getting the AUC of 0.5:
import pandas
import numpy as np
import lightgbm as lgb
from sklearn.cross_validation import train_test_split
from sklearn.metrics import auc, accuracy_score, roc_auc_score
from sklearn.model_selection import GridSearchCV
import matplotlib.pyplot as plt

data = pandas.read_csv('./aggregate_data.csv',delimiter=',', dtype=None)

# Create train and test dataset
y = data['ruptured'].values
X = data.drop(['case_id', 'ruptured'], axis=1).values
lables = list(data.drop(['case_id', 'ruptured'], axis=1).columns.values)
# get rid of some nans
#y = np.delete(y, [34,35], 0)
#X = np.delete(X, [34,35], 0)

X = np.delete(X, [68,69,70,71], axis=1)
lables = np.delete(lables, [68,69,70,71], axis=0)

# Split into train and test set
X_train = X
y_train = y

idx = np.array([74,47])

X_train = X_train[:, idx]
lables = np.array(lables)[idx]

# Centre and standardized model
Xmean = X_train.mean(axis=0)
X_train -= Xmean
Xstd = (X_train.var(axis=0))**0.5
X_train /= Xstd

train_data = lgb.Dataset(X_train, label=y_train, feature_name=lables.tolist())

gbm = lgb.LGBMClassifier(objective='binary',
                          n_jobs=-1,
                          is_unbalance=True,
                          two_round=True,
                          bagging_fraction=0.9,
                          bagging_freq=1,
                          boosting_type='gbdt',
                          feature_fraction=0.9,
                          learning_rate=0.01,
                          min_child_samples=10,
                          min_child_weight=5,
                          min_data_in_leaf=20,
                          min_split_gain=0.0,
                          n_estimators=10000,
                          num_leaves=21,
                          reg_alpha=0.0,
                          reg_lambda=0.0,
                          subsample=1.0)

gbm.fit(X_train, y_train)

Any idea?

sorry, I just want to reproduce your problem with your data, since we didn't meet such an error.
You can try this data with your parameter: https://github.com/Microsoft/LightGBM/tree/master/examples/binary_classification . If this can work, you should check your data carefully.

this is a kind of bug i have also meeted, might cause by using ctrl+c to stop the running of model!!!!!


Get-DbaAgReplica - Initial Version

Fixes #1712 (partly)
Changes proposed in this pull request:

new command

How to test this code:

[ ] Have basic pester test included you can run.
[x] Get-DbaAgReplica -SqlInstance server (On that is SQL Server 2008 R2 or less)
[x] Get-DbaAgReplica -SqlInstance server (One that does not have an AG)
[x] Get-DbaAgReplica -SqlInstance server (One with an AG configured)
[x] Get-DbaAgReplica -SqlInstance server -AvailabilityGroup Ag1 (Server with multiple AGs)
[x] Get-DbaAgReplica -SqlInstance server -AvailabilityGroup Ag1 -Replica Replica1 (Server with multiple replicas)

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2012

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10

Question. What is the last test for?  Would you have an AG with <2 replicas?

@gbargsley It does not mean you have less than 2 replicas. The command is to return info on just one of the replicas in an AG.

@wsmelton
I have run through the list of tests below.  I did find that when connecting to my AG listener and not using the custom port it gives me an error about version 11 required.  All other tests return data as expected.
Could it be possible to list the primary replica in the output?

hey @wsmelton - can you get this fix in within the next 20 minutes?

@potatoqualitee I can't fix that, it is because of the login in Connect-SqlInstance it is not related to this command.

In order to fix that you need to resolve issue #1686

lemme know

@gbargsley any further issues found or we good to merge this one?

The fundamental tests I ran showed what was expected.


Throw SecurityTokenSignatureValidationFailedException when none of the provided keys verify the signature

Fixes  #1268

Throw SecurityTokenSignatureKeyNotFoundException when a security token does not contain a key identifier and the signature validation is not successful with keys found by the runtime.
Modify current logic to also resolve encryption keys when security token does not contain key identifier.

The duplicated code is not all from this PR. I looked into reformatting ValidateSignature method in Saml/Saml2SecurityTokenHandlers. One thing I found was Saml/Saml2Assertion does not have a base class.

@RojaEnnam looks good, only change i have to suggest is to use 'KeyInfo' instead of 'kid' for saml tokens.
The reason that there is not base class to Saml1 and Saml2 is that they are very different objects.
We could have ValidateSignature across all tokens perhaps in the future.


FormControlLabel

Hello Team,
I'm Using a FormControlLabel with a Switch.
Sample :
<FormControlLabel
    label={this.props.displayedName}
    labelPlacement="start"
    control={
        <Switch checked={this.props.value === true} onChange={() => this.props.onChange()} />
    }
/>

The problem is when I'm trying to test the component.
I can't simulate the onChange function.
To get props like "checked" , I've done something like this:
expect(
container.find(FormControlLabel).props().control.props.checked
).toEqual(true);
But container.find(FormControlLabel).props() allows me to read properties, not to simulate any action.

@NguyenJeremy69 I would suggest you try to run this question on StackOverflow. In your case, I woud use the mount API of enzyme and target the Switch component directly.

Hi, I ran across this issue recently and looked up several posts without a definitive solution (Here are some posts that are closest to the issue, for anyone interested:
https://stackoverflow.com/questions/56823431/trying-to-test-formcontrollabel-in-enzyme-wont-trigger-click-when-simulate
https://github.com/enzymejs/enzyme/issues/1283)
I assume there are a few people using mui and enzyme. So these problems can be relevant for many.

The first problem is to access the Switch or whatever you're using for the control prop inside FormControlLabel. mount is definitely viable, as Olivier mentioned above. Since you will be able to find() the Switch.

However, if anyone wants to use shallow, here's how you can trigger the onChange:
let label = wrapper.find(FormControlLabel).at(0);  // wrapper is shallow
label.prop('control').props.onChange({ target: { name: 'item1', checked: true }});

p.s.: remember to provide the same "shape" of event as required in your handler, I used the example pattern provided in mui demo components.

My remaining issue is that although I can see the handler being called, the state doesn't get updated. (label.prop('control').props.checked remains false)
I've tried wrapper.update() with no luck. And, to my knowledge, there's not a great way to access states in tests if you're using functional components, so I cannot actually see whether the state has been updated.

This is really confusing for me and has taken me a lot of time. If anyone has a good solution, please let me know. I'd really appreciate it.

@CristianoYL Consider using testing-library over enzyme, you will face less unexpected issues like this one.


Add powershell plugin

Hi there,
thx for publishing Athena, this is a great implant !
I added two powershell plugins:

powershell-command: to run one (or more) commands
powershell-script: to upload a ps1 file and run it.

Also the same powershell runspace is re-used, which means you can define functions, variables at some point and re-use in later commands/scripts.
If you are interested to push it to main I would be happy to make any modification you think is needed.
There are couple of restrictions:

the powershell SDK in .NET core does not have all the functions/modules that .Net Framework has
it does not work on Linux (and I guess Mac)
Due to an open bug in Powershell SDK it does not work for single file build. Hopefully this will be fixed at some point.

Still I think this can be very useful for windows. Most commands and scripts works. As an example I attached a screenshot where I run PowerUp.ps1.
Any feedback would be appreciated.
Antoine

happy that you like it :)  I will submit a pull for v0.2-dev then.
Regarding the Powershell SDK not working on single file build, the issue is documented on their github repo: https://github.com/PowerShell/PowerShell/issues/13540
Next I am thinking to implement basic windows shellcode injection.

closing this one for #9


Edit pass on ASP.NET Core migration eBook PR

Ping @nschonni
Can you help us with a fix for the markdownlint warning in this PR? It looks like there is an issue interpreting an example regular expression as an invalid markdown link.

Ping @nschonni
Can you help us with a fix for the markdownlint warning in this PR? It looks like there is an issue interpreting an example regular expression as an invalid markdown link.

If you look at the rendered page, it's tripping on the unescaped | in (?:\["'\](?<1>\[^"'\]*)["']|(?<1>\S+)), which I think @DavidAnson flagged in another PR was going to start breaking in the new version as that breaks markdown tables in general

If you look at the rendered page, it's tripping on the unescaped | in (?:\["'\](?<1>\[^"'\]*)["']|(?<1>\S+)), which I think @DavidAnson flagged in another PR was going to start breaking in the new version as that breaks markdown tables in general

Right, here is the PR @nschonni is referring to: https://github.com/dotnet/docs/pull/21977

Right, here is the PR @nschonni is referring to: https://github.com/dotnet/docs/pull/21977


Folding memos close when using check boxes

Describe the bug
When there is a check box in the folding section of a memo, clicking it also collapses the memo.
Steps to reproduce

Create folding memo
Add check box below ---

Screenshots or additional context
No response

Already fixed.


openapi(engine): log verbose request/response info for debug

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/7316
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/run-engine-integration-test

Codecov Report

Merging #7321 (c88945b) into master (2843d67) will increase coverage by 0.2015%.
The diff coverage is 58.6676%.

Additional details and impacted files

Flag
Coverage Δ

dm
51.9765% <36.6666%> (+0.0714%)
:arrow_up:

engine
60.7982% <53.6379%> (-0.2593%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #7321        +/-   ##
================================================
+ Coverage   59.6477%   59.8493%   +0.2015%
================================================
  Files           790        798         +8
  Lines         91016      91448       +432
================================================
+ Hits          54289      54731       +442
+ Misses        31982      31947        -35
- Partials       4745       4770        +25

/merge


fix crash while creating post

duplicate of #2316

It's duplicate actually, the first one related to assessing the issue only, not fixing it.

@noumantahir
Great find, would be good time to check all props of editor screen and optimize it then or rewrite new editor screen and replace old one, move important methods and keep structure clean 👍

Some quick fixes to implement

[ ] reduce frequency of validity check
[ ] reduce frequency of local draft save
[ ] optimise username check
[ ] update modals to unmount content after closure


RFC: Provider Caching

Providers should be able to specify what files are necessary for the install step to allow for better Docker layer caching.
For example, the NPM provider could specify that package.json is all that is necessary to install. If that files does not change between deploys, a cached layer can be used.
Provider caching can be split up into a few different types

Specify the files necessary to install so that only these files are keyed against when computing a cache hit/miss
App/framework level caches (e.g. .next directory). These caches should be re-used even if the source code that generated the cache changes. For this level of caching we cannot take advantage of built in Docker layer caching and will likely require a persistent volume that can be shared between builds.

This RFC will mainly focus on the first type.

I'd love to work on this, I have some ideas about how this can be made extremely modular.

I was initially thinking of allowing providers to specify what files are necessary for the install step. This would allow us to take advantage of the Docker layer cache if these files do not change between builds.
fn install_files(&self, _app: &App) -> Result<Vec<String>> {
  Ok(Vec::new())
}

Alternatively install_cmd could be extended to return a list of (command, files) pairs, but ideally if the provider does not specify any install files, all files are presumed to be needed.
For example, in the NodeJS case, only the package.json file is necessary to run npm install. If the NPM provider was extended to
impl Provider for NpmProvider {
    // ...

    fn install_files(&self, _app: &App) -> Result<Vec<String>> {
      Ok(vec!["package.json".to_string()])
    }

    fn install_cmd(&self, _app: &App) -> Result<Option<String>> {
        Ok(Some("npm install".to_string()))
    }

   // ...
}

Then when creating the Dockerfile we can first only copy over the install files (package.json) file, run the install step (npm install), and then copy over the remaining files. If only the source code changes between builds then the dependencies installed with npm can be pulled from the cache.
Obviously, there are more advanced use cases with different languages and frameworks. Please share them here or bring up any concerns you see with this approach.

How would this handle for example the .next folder with NextJS? That isn't needed at install time but isn't (but can be used) at build time.
PS: This approach shouldn't error if the file doesn't exist.

Hmm at the moment it wouldn't handle that case and caching for specific frameworks (next, remix, etc) will require a bit more involved solution. Happy to hear thoughts/suggestions here.

I do think this should happen in multiple steps. First step to just get deps cached between builds (easy win). Next step can be to give providers and API for more advanced fine grain cache control.

Hmm at the moment it wouldn't handle that case and caching for specific frameworks (next, remix, etc) will require a bit more involved solution. Happy to hear thoughts/suggestions here.

I kinda struggled with this a tiny bit when I took a swing at the Rust and Zig providers
I think we need a way to have postbuild actions which allow you to insert 0 - N "COPY" calls into the dockerfile to move them into the right place

I think we need a way to have postbuild actions which allow you to insert 0 - N "COPY" calls into the dockerfile to move them into the right place

I really like the idea of doing it like this. Not sure if COPY is the write call, more should be ADD but can be disclosed later.

The providers sort of have that ability now. They can define files that are necessary for each phase and only those files are copied over when running that phase so we can make ues of the build cache. However it doesn't work all the time and there are definitely some holes and I would like to see it fleshed out some more.
Honestly I think we will need some semi-custom caching solution in the not too distant future because relying on the Docker layer cache isn't great and it would be nice to have more control over what is cached.

Where would cached files be stored if we don't use Docker caching? IMO, just using Docker cachnig is a better idea.

Where would cached files be stored if we don't use Docker caching?
That is what we have to figure out 😆

The problem with Docker caching is that it is linear. A changed env var before the build phase will break the cache for every command below (which is fairly common on Railway because we pass in the github commit sha to every build). It would also be nice to benefit from the cache even if the files change. For example you can still benefit from the .next cache even if the source files you are building from change.
We will see that. Can optimize for the Docker cache now and see where the issues are and if/what we can do to improve it.


Kraken Streaming exchange cannot establish connection

This is due to a problem in XChange that is being fixed. knowm/XChange#3803

A PR has been merged into the develop branch of XChange that fixes this issue. I have been able to confirm that the fix does work for Arbitrader.
I am not comfortable releasing a version of Arbitrader based on snapshot builds of XChange, so we will need to wait until 5.0.4 is released before this can be closed. Assuming they follow their typical release schedule it should be coming in the next week or so.

Looks like XChange 5.0.4 is coming soon. knowm/XChange#3805

Just merged #264. Will confirm this issue is fixed and close.

Confirmed that the latest version can connect to Kraken websocket successfully. 👍


Error: method 'isBlogAuthorized' Error: Did not check() all arguments during call to 'isBlogAuthorized'

When I try to navigate to add blog post it bounces off and on the server console I get the below error.
Anyone come across such error? Know how to fix/bypass it?
Thanks,
Praney
Here is the error code:
I20150411-21:40:57.265(10)? Exception while invoking method 'isBlogAuthorized' Error: Did not check() all arguments during call to 'isBlogAuthorized'
I20150411-21:40:57.266(10)?     at [object Object]..extend.throwUnlessAllArgumentsHaveBeenChecked (packages/check/match.js:357:1)
I20150411-21:40:57.266(10)?     at Object.Match.failIfArgumentsAreNotAllChecked (packages/check/match.js:112:1)
I20150411-21:40:57.266(10)?     at maybeAuditArgumentChecks (packages/ddp/livedata_server.js:1614:1)
I20150411-21:40:57.266(10)?     at packages/ddp/livedata_server.js:648:1
I20150411-21:40:57.266(10)?     at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20150411-21:40:57.266(10)?     at packages/ddp/livedata_server.js:647:1
I20150411-21:40:57.267(10)?     at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20150411-21:40:57.267(10)?     at [object Object]._.extend.protocol_handlers.method (packages/ddp/livedata_server.js:646:1)
I20150411-21:40:57.267(10)?     at packages/ddp/livedata_server.js:546:1

If you leave both Blog.settings.authorRole and Blog.settings.adminRole empty then it will let anyone into the admin area and at least you can try to debug it from there. I'm not familiar with that error code unfortunately so that's all I can tell ya.
Also I'm pretty sure that check only runs if authorRole is set so if the above works then try setting Blog.settings.adminRole but keep Blog.settings.authorRole undefined until you figure it out.

This package doesn't pass the requirements of the audit-argument-checks package. I'll try to investigate precisely why that is, but for now, I suspect @praneybehl you can remove that package and it will work fine.

Thanks, Yes I've dropped it.


doc: Add Fedora dependency packages

This PR adds build dependency package on Fedora.
On my system (Fedora 35) no additional runtime dependencies are required after successful build against system packages.

@jarolrod

I can pick this up in a follow-up if you don't want to retouch.

Cool!
Closing in favor of #93.


Regarding AOT Dataset

Hai @tusharsangam, Great work.
Did you use complete #part1 of the AOT dataset, or partial?
When downloading the dataset, is partial=True?

@Sairam13001 we used videos from the first part but not all the videos are used. We keep the partial=False while downloading & training.


Remove depedency on VideoInput layer

I am trying to build live demo taking on fly sample to classify. How can I remove the dependency on  the VideoInput layer?

@moaljazaery if you have some video loader written in python, and perform the transformations (re-scale, crop, normalize etc. as done in VideoInputOp), then we can do inference completely in caffe2 (python land). But I suggest to be very careful to have the same set of transformation (color channel order, etc) so that you can have the best accuracy.


Fix assertion value modification

This PR fixes the following bug in GUI:

Assertion operator falls back to "eq" when user starts changing assertion value.
Steps to reproduce:

Create any assertion that requires value, except "eq" (for example "neq")
Start changing the value

Expected: Operator doesn't change
Actual: Operator falls back to "eq"

Thanks for taking care of this @Sl-Alex !!
Merged.


Edit the scrobble cache.

The ability to edit the music before it's sent to Last.fm/Libre.fm

Done and done.


Repeating songs don't scrobble, it only scribbles it the first time

I'm using Phonograph, and the scrobbler seems to only detect once the song when I play it on repeat.
It seems that this happens both when I set the repeat button to repeat 1 song and when I queue the same song several times. I'm using Phonograph on Android 6.0.1 in a Samsung Galaxy S7 SM-G930FD

#324

@HumbleBeeBumbleBee I have the same issue with Google Play Music.
On another app, I saw that they were able to fix it: https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1061
I don't get what you mean on the #324 ticket by saying "Shuttle Player may solve this problem." Is your solution to use another music player?

@HumbleBeeBumbleBee I have the same issue with Google Play Music.
On another app, I saw that they were able to fix it: https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1061
I don't get what you mean on the #324 ticket by saying "Shuttle Player may solve this problem." Is your solution to use another music player?


Failover of RADOS monitor when intergrate Cloudstack with CEPH RBD

Hello,
When integrating CEPH RBD as the primary storage for Cloudstack, we can only input one IP address of the node running the monitor service into the 'RADOS monitor' field, while typically, CEPH cluster has at least 3 monitor nodes. It seems that there is no HA capability if this monitor node stops working.
When I turn off the monitor service:

The running VMs still function normally and can be rebooted.
However, if VM stopped, it cannot start again, and the agent reports an error of not finding the domain in Virsh.
Also, I cannot create new VMs
So, is there any way to achieve HA by inputting multiple IP addresses of RADOS monitor nodes? Will this issue be considered for resolution in the future?

Hi @tuanhoangth1603
this issue has been fixed by #6792
You can test <IP_ADDRESS>-RC3 which includes the fix.

Thanks so much !


Credentials are not set when using BUNDLE_GEM__PUSH_KEY

bundle env is able to notice it:
circleci@9d7f73ed1f7d:~/project$ bundle env

Environment
Bundler       1.16.1
  Platforms   ruby, x86_64-linux
Ruby          2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux]
  Full Path   /usr/local/bin/ruby
  Config Dir  /usr/local/etc
RubyGems      2.7.6
  Gem Home    /usr/local/bundle
  Gem Path    /home/circleci/.gem/ruby/2.5.0:/usr/local/lib/ruby/gems/2.5.0:/usr/local/bundle
  User Path   /home/circleci/.gem/ruby/2.5.0
  Bin Dir     /usr/local/bundle/bin
Tools
  Git         2.11.0
  RVM         not installed
  rbenv       not installed
  chruby      not installed

Bundler Build Metadata
Built At          2017-12-21
Git SHA           0034ef341
Released Version  true

Bundler settings
path
  Set via BUNDLE_PATH: "/usr/local/bundle"
app_config
  Set via BUNDLE_APP_CONFIG: "/usr/local/bundle"
silence_root_warning
  Set via BUNDLE_SILENCE_ROOT_WARNING: true
bin
  Set via BUNDLE_BIN: "/usr/local/bundle/bin"
gem.push_key
  Set via BUNDLE_GEM__PUSH_KEY: "**removed**"
gemfile
  Set via BUNDLE_GEMFILE: "/home/circleci/project/Gemfile"

But when I run bundle exec rake release I have:
circleci@9d7f73ed1f7d:~/project$ bundle exec rake release
gate 0.7.1 built to pkg/gate-0.7.1.gem.
Tag v0.7.1 has already been created.
rake aborted!
Your rubygems.org credentials aren't set. Run `gem push` to set them.
/usr/local/bundle/gems/rake-12.3.1/exe/rake:27:in `<top (required)>'
/usr/local/bundle/bin/bundle:104:in `load'
/usr/local/bundle/bin/bundle:104:in `<main>'
Tasks: TOP => release => release:rubygem_push
(See full trace by running task with --trace)

@jandudulski You closed this issue without a resolution. I'm running into the exact same problem. How did you resolve?

@alexkates it wasn't an issue with bundler but my misunderstanding how RubyGems works. You have to create ~/.gem/credentials file with yaml content like:
---
:rubygems_api_key: KEY

And push key is not the value of a key but its name, i.e. rubygems_api_key in my example.


job localmount

implements a job local-mount as commented on #58

Thanks for the PR!
I've thought about this some more, and unfortunately I don't think I can accept this change. It would introduce a second way of providing mounts, which is not something that I think should exist. I prefer to keep things simple and only have a single way to specify each resource.


Tags context menu entry doesn't appear when right-clicking an unselected bookmark

When a right click causes the current active bookmark selection to change (aka. right-clicking an unselected bookmark), the 'Tags' menu item appears and then immediately disappears. This seems to be a result of the state changing while the context menu is visible. I've been able to disable side effects of this behaviour in most places by simply not having any menu items be conditional on the selection when shown in the context menu, but it's not possible with the tags menu which includes all the tags for the selected items.

This looks to be a SwiftUI bug that may be addressed in SwiftUI 3, but it's being exacerbated by the custom approach to updating the selection when the user displays the context menu (something which isn't supported out-of-the-box in SwiftUI 2 at least).

Since upgrading to Monterey, this issue manifests differently: now the tags menu is simply empty on first display.

Fixed.


AF doesn't work with Dark Age Of Camelot

Hi,
first thank you for your work.
When you plan to push next release? I'm really interested in the anisotropic filtering commit.
Thanks

I am not sure when I will release the next update.  There are still a few things I wanted to do before releasing.  There was a crash on two games that never happened before that I wanted to troubleshoot.  But I have been busy with other project recently.
Anyways, here is the latest build.  It is pretty stable:  dxwrapper.zip

Wow, nice! Thanks.
How can i force AF in the settings? Need to put 1 or AF level (4, 8 16)?
Because it seems to not work for me on d3d9

How can i force AF in the settings? Need to put 1 or AF level (4, 8 16)?

1 = highest your video card can support.  2, 4, 8, or 16 = specific AF settings you want to configure.

Because it seems to not work for me on d3d9

Interesting.  I don't actually change the texture filtering, but just enable the AF option.  I can do an update that actually changes the texture filtering in addition to enabling the AF option.
Have you tried using the AF option in your video card's Control Panel?

uhm, seems that i cannot force AF with my control panel (intel hd620) in the game i play (Dark Age Of Camelot).
I set it to forced 16x in intel control panel but in game there is no change... I also set 16x in your dxwrapper but no change.
only way i found to force it succesfull is using another kind of d3d9 wrapper (dgvodoo), but it has kinda horrible performance. (with or without AF)...
could you try to code the AF forcing in your awesome wrapper?
Thanks :D

Because this is about compatibility and not enhancements.
Graphics cards control panels are already good enough for that.

This issue may be related to a fix I put in here: Fix AF with multi-stage textures.
Can you try with the latest build and see if that helps:  dxwrapper.zip

Closing this issue.  You can reopen it if the issue still exists.


Remove version: Freeplane.Freeplane version 1.12.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


Upgrade Wox to .net framework 4.5?

Since I'm implementing Auto upgrade function. I noticed quirrel.Windows, which sounds a good choice. Atom uses this to upgrade itself. But this library needs .net framework 4.5, and Wox is now targeting to .net framework 3.5.
.net framework is installed in win8 by default. So for win8 users, it's not a problem if Wox upgrading to .net framework 4.5. But for win7 users, they may need to install .net framework 4.5 (quirrel.Windows will auto install .net framework 4.5 if it find user didn't install that).
I don't know how many Wox users still use windows 7 (or didn't install 4.5 in windows 7)? So, I'm not sure if we should upgrade Wox to .net framework 4.5, or should we find another auto upgrade solution.

I think need add something like Google Analytics to Wox for collect statistic, and on the basis of this information take decide about switch to .net framework 4.5.

That's fair.

I know that .net 4.5 would help with plugin development, at least for what I am doing with the chocolatey plugin.
It should however be fueled by customer adoption and adding a way to send machine metrics back for collection is probably the best thing to do in the short term.


Update Composer dependencies (2021-07-02-00-12)

Visual regression test passed!
View the visual regression test report


feat: bump helm-diff to v3.9.0

blocked by: https://github.com/databus23/helm-diff/pull/519

@mumoshu ping


test(netlify): check deployment of a branch from ouds/main

Description
Just a branch/PR from ouds/main with a modification to check if a Netlify deployment is triggered.

It works! Branch has been deployed to https://deploy-preview-2579--boosted.netlify.app/


Update dependencies and imports

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: andkononykhin:x: mzk-vctYou have signed the CLA already but the status is still pending? Let us recheck it.


Clarify package upgrade policy.

It would be nice to formalize our package upgrade policy.   In other words: When packages may be installed side-by-side, and when they are mutually exclusive?
A straw-man policy proposal, which seems to yield intuitively correct results: "The (target, channel, major, minor) tuple must be unique per-machine".    Some examples of what this would mean:

Any 32-bit installation will co-exist with any 64-bit one (target is different).
Installing a nightly replaces previous day's nightly (they only differ in build revision).
Nightly, beta and stable with the same major.minor version will co-exist.
Installing 1.0-alpha.2 will uninstall 1.0-alpha.1.  Installing 1.0-beta.1 uninstalls any 1.0-alpha.  And so on...
1.0.x-stable will co-exist with 1.1.x-stable, but 1.0.1-stable will replace 1.0.0-stable.

(obviously, this only applies to platforms, where installer packages are capable of enforcing such rules)
cc @brson, @alexcrichton, @pcwalton

Any 32-bit installation will co-exist with any 64-bit one (target is different).

The windows and unix situations are different here in multiple ways: on windows installation naturally goes into its own directory, so multiple rustc's won't clobber each other. On windows we don't currently do cross compilation as easily as on unix.
My ideal is for any channel, to have a single rustc installation that can target either architecture, which is achievable.
On Windows I agree this is ok.

Installing a nightly replaces previous day's nightly (they only differ in build revision).

Yes.

Nightly, beta and stable with the same major.minor version will co-exist.

I don't know how to do this on unix in a way that is ergonomic. Perhaps they could come with a tool that is like multirust-lite that swaps the active channel. Everything could be installed as e.g. rustc-1.0.0.

Installing 1.0-alpha.2 will uninstall 1.0-alpha.1. Installing 1.0-beta.1 uninstalls any 1.0-alpha. And so on...

Yes.

1.0.x-stable will co-exist with 1.1.x-stable, but 1.0.1-stable will replace 1.0.0-stable.

I have not considered this before, but it will definitely require some changes. Similar problems to the previous on unix.
It would be worth considering how various Linux package managers could support this.
I feel fine following the rules you've laid out on windows at least, though I want to think more about which stable compilers might coexist next to each other.
They are harder to support on unix, but on unix it's common to see multiple versions of e.g. gcc installed together. Distros will probably want it eventually, so we should figure out how to do it.


Add new script that troubleshoot public folder deletion failures

Issue:
The script is working to validate the below conditions to investigate public folders items/public folders deletion operations failures & propose FIXes with mitigations.
Checks run across public folder:

validate content PF MBX across both PF & its dumpster
validate EntryId &DumpsterEntryID values are mapped properly
validate public folder mailbox TotalDeletedItemSize value hasn’t reached its RecoverableItemsQuota value
Validate parent public folder health
Validate Dumpster flag
validate if it's PF perm sync issue
validate PFstats size if it was met or not
validate if dumpster folder has children
validate MEPF health if DebugMEPFproxyguid was empty/null

The script is creating new directory containing 2 log files for logging errors for the script & the public folder report containing issues causing deletion failures & their proposed FIXes.

Also the script has a logging capability over the PowerShell screen presenting issues encountered with proposed FIXes for Admins to follow & mitigate in-question public folder.

Finally there’s a feedback form provided over the script to collect ideas/feedback from admins about improving the script.
Reason:
Support cases/bugs deflection over that specific topic
Fix:
NA
Validation:
Working on EHLO blog to discuss the above points clearer

Please run .\build\CodeFormatter.ps1 -Save to correct the format errors found by the build pipeline

It should have caught a lot more than that. You might need to merge from our repository and try again.

It should have caught a lot more than that. You might need to merge from our repository and try again.

@dpaulson45 I tried to switch to main and run the codeformatter but it didn't share any results then switched back to my branch same behavior no issue spotted, sorry if my question seems stupid as am still newbie to Github can you share what should I do if possible?
Thanks!

@bill-long would you like to review this as well

Commit time on mkdocs.yml needs to be updated so we can merge. Easy way to do this is just make a change to the file, commit it, change it back, and commit it. Another option is to rebase the whole branch, but that is more complicated.

Commit time on mkdocs.yml needs to be updated so we can merge. Easy way to do this is just make a change to the file, commit it, change it back, and commit it. Another option is to rebase the whole branch, but that is more complicated.

Done.

Run .\.build\CodeFormatter.ps1 -Save to address the formatting rules.


Problem displaying the menu

Hi,
I'm very pleased of your excellent Astrarium Software. Good luck for increasing development for that.
After installing and executing, the Menu software display "{Menu.Map}" althoug the name it self. How can I correct this ? Thnak you in avdance.

Hello,
thank you for the positive feedback about the app!
Seems that Astrarium was not installed correctly, try to reinstall it. Also could you be so kind to attach a screenshot with the problem.

This is a capture of the problem:

This is the problem :

I also have this problem. A solution is to change language to Russian and then back to English. However, this needs to be done every time Astrarium is restarted.

Finally I've found the root cause. This will be fixed in the upcoming release. Thanks everyone for reporting the problem.


Add facebook button

Now you could find our bot just clicking this button in README.
Also it looks great.

Coverage remained the same at 90.654% when pulling 989e2d21f06e4b0b21467777ff9c291f5a2a6659 on Facebook-button into e7d3b1a53a917837e901015fbbdf6e6d0277ccff on master.

Coverage remained the same at 90.654% when pulling 989e2d21f06e4b0b21467777ff9c291f5a2a6659 on Facebook-button into e7d3b1a53a917837e901015fbbdf6e6d0277ccff on master.


@maasdal.com

Title
Rutger Truyers
Bluesky profile url
https://bsky.app/profile/maasdal.com
Twitter profile (url or username)
No response
Category
security
Type
microsoft

TS Security Member in Belgium


Old GFX card still being reported after installation of new one on Mac

Description
Just swapped out a Radeon HD 7850 for a Nvidia GTX 970, but Neofetch is still reporting the old card (Radeon HD 7xxx).
Neofetch version
Neofetch 5.0.0
Screenshot
Neofetch output:

system_profiler SPDisplaysDataType output:

Verbose log
https://gist.github.com/benmadethis/7de54535abe79fca61f6614ac64ac492

Delete your /Library/Caches/neofetch/gpu file as a workaround, but I don't know if this is a bug or not. Ping @iandrewt

Thanks @konimex that did the trick.


Complete Seed overview

missing fields in /seeds are:

[ ] plant name (needs to be added)
[x] additional name
[ ] harvest year
[ ] amount
[ ] origin

cosmetic problems:

[ ] "My Seeds" partly hidden #266

new functionality:

[ ] edit seed entries
[ ] remove seed entries

@buenaflor What is the status of this issue? Please tick what is done in the top-post. Which PRs implement it?

I updated above to match the use case in #366

There is a separate issue now for the last point: #878


It get stock

when a dynamic body is covered by many static bodies.
use mouse to draw many lines，a line is a static body.
then the dynamic body will be blocked by the lines.
when the lines cover the dynamic body,it will get stock.
How to solve this problem?

What behaviour do you want in this scenario?
On Sun, 10 Feb 2019, 06:02 xiajiaonly<EMAIL_ADDRESS>wrote:

[image: image]
https://user-images.githubusercontent.com/4102643/52530133-83426e00-2d3b-11e9-8e7d-a972e8ec09da.png
when a dynamic body is covered by many static bodies.
use mouse to draw many lines，a line is a static body.
then the dynamic body will be blocked by the lines.
when the lines cover the dynamic body,it will get stock.
How to solve this problem?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532, or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvlwbaTzsH3QwKbCJi_MaPrJNSrPpeks5vL7XbgaJpZM4ay6EG
.

I think the dynamic body  should be stuck and in a static state.

I'm not sure I understand. If it's static, it won't move. If it's blocked,
it won't move.
On Sun, 10 Feb 2019, 08:26 xiajiaonly<EMAIL_ADDRESS>wrote:

I think the dynamic body should be stuck and in a static state.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532#issuecomment-462113465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvl4p0E7iC_JBysMPTcrDD74MGc4NEks5vL9fKgaJpZM4ay6EG
.

It jumps in box2D.
In reality, it should be an illegal state.
Whatever the state it is, it should not increase the consumption of world's Step ().

Ok, so it's not "stuck".
If a dynamic object is over another object, it will push away from the
centre of the other object. Most physics engines will do the same. If you
want your object to be static, make it static. Otherwise it will have
dynamic behaviours. If you want to pretend the object is on another plane,
so they shouldn't interact, you should use the category bits and mask bits.
There are great tutorials at www.iforce2d.net/b2dtut/
On Sun, 10 Feb 2019, 11:01 xiajiaonly<EMAIL_ADDRESS>wrote:

It jumps in box2D.
In reality, it should be an illegal state.
Whatever the state it is, it should not increase the consumption of
world's Step ().
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532#issuecomment-462122684,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvlw_GHEudPJn6XLsPNNYF1bnmk0vkks5vL_v0gaJpZM4ay6EG
.

T.T，You misunderstood me.I don't care about the state of the object. What I care about is that the Step() function will be stuck.Maybe this needs a GIF to explain it more accurately.

Ok, so you call Step and it locks up?
How many iterations are you asking it to perform?
On Sun, 10 Feb 2019, 12:56 xiajiaonly<EMAIL_ADDRESS>wrote:

T.T，You misunderstood me.I don't care about the state of the object. What
I care about is that the Step() function will be stuck.Maybe this needs a
GIF to explain it more accurately.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532#issuecomment-462130530,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvl5dZiWfR_vfiRLMmTBcUzC6xdzZbks5vMBcAgaJpZM4ay6EG
.

m_pWorld->Step(1.0f / 60.0f, 4, 2);

It would seem strange for this not to have been discovered before. You
could use iforce2d's RUBE to model the same scene (export a breaking
example or create the objects in RUBE, save it, and change the file in
notepad to have the same object properties) and see if that suffers the
same issue. If it does, you have something reproducible. Otherwise, it's
likely something peculiar to your code.
 - Hugh

"Bureaucracy is the epoxy that greases the wheels of progress."

James H. Boren

On Sun, 10 Feb 2019 at 13:25, xiajiaonly<EMAIL_ADDRESS>wrote:

m_pWorld->Step(1.0f / 60.0f, 4, 2);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532#issuecomment-462132668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvl9YPvqv7t5VHeKO3Wq4lw5CmEgE2ks5vMB3ZgaJpZM4ay6EG
.

the code modify from the testbed,is here:https://github.com/xiajiaonly/TestB2D

Because of the limited time, I haven't written a typical example. I will revise this example in the next step to make the problem more typical.

I looks like the complexity of your static model is the problem. Simplify, then simplify again.

Yeah, I was going to say that looks pretty crazy. How many objects do you
have there?
On Sun, 10 Feb 2019, 23:28 Erin Catto<EMAIL_ADDRESS>wrote:

I looks like the complexity of your static model is the problem. Simplify,
then simplify again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532#issuecomment-462191727,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvl8kw78YEOHHKzFIP1a2pUnYLC2eRks5vMKslgaJpZM4ay6EG
.

It doesn't need a lot. In some cases, just a few. In the above examples, some are unnecessary, just to illustrate the problem. I will revise the example above. Yes, the problem is more prominent. I just got up. Wait a minute.

I have updated the code.

“body sum” just the  lines sum，in up pic.

“body sum” include lines sum and balls sum,in up pic.

Are you trying to make a solid body? You should use Polygons, not Lines...
On Mon, 11 Feb 2019, 03:48 xiajiaonly<EMAIL_ADDRESS>wrote:

I have updated the code.
[image: image]
https://user-images.githubusercontent.com/4102643/52545456-9ebc8000-2df2-11e9-8ae5-0ea7db23cdf8.png
“body sum” just the lines sum，in up pic.
[image: image]
https://user-images.githubusercontent.com/4102643/52545478-ce6b8800-2df2-11e9-8380-00f06dc932b3.png
“body sum” include lines sum and balls sum,in up pic.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/erincatto/Box2D/issues/532#issuecomment-462213098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZZvl6rDFGYw042chTECNZzVQnaoJdagks5vMOf6gaJpZM4ay6EG
.

Yes, it's the same.

I experimented with this test case. The performance problem is definitely SolveTOI. The over-constrained bodies are trying to prevent tunneling. The solver fights harder for circles than polygons.


Upgrade version of airbase from 41 to 46.

Upgrade version of airbase from 41 to 46.

@electrum Is there any chance to merge it and then to release the airlift?

Merged, thanks!


chore: Add JetBrains folder to .gitignore

Personally, I don't want to favor each contributor's IDE of choice but recommend creating a repo-specific ignore or a global ignore
See https://stackoverflow.com/a/5724499

Personally, I don't want to favor each contributor's IDE of choice but recommend creating a repo-specific ignore or a global ignore
See https://stackoverflow.com/a/5724499

Cool! I didn't know you could do that.


Remove deprecated code

Merge after: https://github.com/packit/requre/pull/285 -> then record again failing tests
Merge after: packit/packit-service#2532 packit/configuration-migrations#1

recheck


Update README.md

Added the last Context

Please add some value add, if you're trying to be added to the contributors. Check the issues :)


Parameter documentation?

@KeeganShaw-GIS Soroush @tarcadius - your thoughts on the below?
A method to update documentation as we go might be helpful, and make that documentation part of the code base so changes come in with both code changes, and the changes to documentation. You all have done fantastic work with first round, but having something part of the code base could be an enhancement?
Generally, there are probably 3 sets of documentation:

method level - documentation for parameters, what the method does, etc. - the way to do this is probably sphinx: https://www.sphinx-doc.org/en/master/   - parameters, defaults, units, etc. for each method in the docstring at the start of the method. This is probably already a PEP8 thing already, I'm not certain. This documentation is aimed at people taking the model apart and putting it back together again.

parameter level - documentation for model parameters in the input parameter files. This defines what columns need to be in the input spreadsheets, and what every single parameter does. This is aimed at people who want to run the model but aren't taking it apart. I think you all have already made this - but should it be part of the code, so it can be updated? Possibly markdown or something that can be auto-rendered to a pdf?

web-app level - documentation for web app people - this is kind of a separate deal and probably will be part of web app code.

Thoughts on this? I'm suggesting possibly doing this way so documentation is kept current and part of pull requests instead of being a big task that someone needs to do later.

I will add this to the meeting agenda for this week. I like the idea of sphinx.

Is this related to what @c-vollrath is investigating with GitWiki? I may request to lean on someone else to lead this issue and let them teach me as they go, but sounds like a good idea to have everything well documented and cross referenced internally. Will save much future pain.

Closing this one for now - I think we are mostly there with the new markdown. Docstrings still need some work here and there, but we can clean this up with time.


MSAL Angular query string is lost after redirected back.

Library

[x]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Framework

[x] Angular 8
[ ] React
[ ] Other

Description
In Angular APP, hit exactly the same issue as described here.
The query string appeared for a while and then got lost.
https://stackoverflow.com/questions/63707766/capturing-query-string-before-angular-constructor-is-called
We are using 1.4.0, tried 1.4.8, same issue
Error Message

MSAL Configuration
{
auth: {
clientId: clientId,
authority: AAD_ENDPOINT + (this._tenantId || 'organizations'),
validateAuthority: true,
redirectUri: window.location.origin,
postLogoutRedirectUri: window.location.origin + baseHref,
navigateToLoginRequestUrl: true
},
cache: {
cacheLocation: window.navigator.userAgent.indexOf('Edge') > -1 ? 'localStorage' : 'sessionStorage'
},
system: {
loadFrameTimeout: 30 * 1000
},
framework: {
isAngular: true
}
}
// Provide configuration values here.
// For Azure B2C issues, please include your policies.

Reproduction steps
// Provide relevant code snippets here.
// For Azure B2C issues, please include your policies.

Expected behavior
Identity Provider

[x] Azure AD
[ ] Azure B2C Basic Policy
[ ] Azure B2C Custom Policy
[ ] ADFS
[ ] Other

Browsers/Environment

[x] Chrome
[x] Firefox
[x] Edge
[ ] Safari
[ ] IE
[ ] Other (Please add browser name here)

Regression

[ ] Did this behavior work before?
Version:

Security

[ ] Is this issue security related?

Source

[x] Internal (Microsoft)
[ ] Customer request

@fhljys Can you please share steps to reproduce and verbose level logs from msal? It's unclear from the stackoverflow issue linked, is this an issue with your own custom query string or the hash that contains the tokens returned from AAD? Can you clarify?

Hi @tnorling , thanks for your prompt reply.
Here is to provide more details

I start a page with https://atlas-preview.cloudapp.net/en-us/authoring?factory=xxx,
I was been redirected to login.microsoft and when redirect back I got https://atlas-preview.cloudapp.net#id_token=, note, all the path and query string are lost
Then at some point I got   https://atlas-preview.cloudapp.net/en-us/authoring?factory=xxx#id_token=. This looks perfect to me. All the query and path are recovered
Then I got https://atlas-preview.cloudapp.net, noting is left..(But at this moment, my app is already logged in.

I shared you a gif in teams as well.

@tnorling , after more investigation, I think we found the root cause.
Issue happens here:

The original code removed hash, but an extra # is left. Angular will treat this as a location change, so it will cancel the original navigation and trigger a new one. The cancel operation somehow triggered an extra pop state.
So the issue here is why MSAL provides me an extra # after the navigation? Could you fix that?

@fhljys Please update to the latest version of msal, as this issue (extra # remaining in url) was addressed in 1.4.2: https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-core/changelog.md#142

Thanks @jasonnutter. Verified. It works.


ci: Change Ruby setup to standard

What problem is this PR intended to solve?
Currently, Actions CI is using non-standard branches for Ruby setup.  When the branches were used, changes had not been merged to the various actions repos (MSP-Greg/setup-ruby-pkgs, ruby/setup-ruby).  Updates have been merged.
Have you included adequate test coverage?
When tested in my fork, updated CI workflows ran as they do in main
Does this change affect the behavior of either the C or the Java implementations?
na, as only workflow files have been changed

@flavorjones Some days I really wish MSYS2 had more resources.  The error in the CI workflow job windows (disable, head) is caused by a slow connection:
error: failed retrieving file 'msys.db.tar.zst.sig' from mirror.umd.edu : Operation too slow.
Less than 1 bytes/sec transferred the last 10 seconds

It did pass in my fork.  I've recently seen similar errors in other repos, involving other files...

@MSP-Greg No worries, we see that failure relatively often. Merging! And thank you again for everything you're doing behind the scenes to make Windows work well.


[TEP-0076] Add indexing into array for taskrun params reference

Changes
This commit provides the indexing into array for params and gated by
alpha feature flag. Before this commit we can only refer to the whole
array for taskrun params, with this feature we can refer to array's element such as
$(params.param-name[0]).
/kind feature

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Indexing into array for taskrun params is now an alpha feature, element of array params can be accessed via $(params.param-name[i]).

@ywluogg

#4723

/assign @ywluogg

/retest

/retest

/retest-required

/retest-required

/retest-required

/retest-required


Regression: filter_parse type json is dropping events

My environment

Fluentd version: 1.1.0
Docker image: official fluent/fluentd:v1.1.0
OS on the Host: Ubuntu 16.04.3 LTS
Linux kernel: 4.13.0-32-generic
Docker version: 17.12.0-ce

How to reproduce
Config file
$ cat fluent.conf
<source>
  @type tail
  path /x.log
  read_from_head true
  tag docker.*
  <parse>
    @type json
    time_key time
    time_format %Y-%m-%dT%H:%M:%S.%N%Z
    localtime true
  </parse>
</source>

<filter docker.**>
  @type parser
  format json
  key_name log
  reserve_data true
  emit_invalid_record_to_error false
</filter>

<match **>
  @type stdout
</match>

Test data
This is the log file I am reading with tail:
$ cat x.log
{"log":"+ cat /etc/resolv.conf\n","stream":"stderr","time":"2018-02-20T14:26:51.656915333Z"}
{"log":"nameserver <IP_ADDRESS>\n","stream":"stdout","time":"2018-02-20T14:26:51.666966175Z"}
{"log":"options timeout:1 ndots:0\n","stream":"stdout","time":"2018-02-20T14:26:51.666988017Z"}

Expected behavior
Fluentd 1.0.1 shows the expected behavior:
$ docker run --rm -ti -v $PWD/x.log:/x.log -v $PWD/fluent.conf:/fluentd/etc/fluent.conf -e "FLUENTD_OPT=-qq" fluent/fluentd:v1.0.1
2018-02-20 14:26:51.656915333 +0000 docker.x.log: {"log":"+ cat /etc/resolv.conf\n","stream":"stderr"}
2018-02-20 14:26:51.666966175 +0000 docker.x.log: {"log":"nameserver <IP_ADDRESS>\n","stream":"stdout"}
2018-02-20 14:26:51.666988017 +0000 docker.x.log: {"log":"options timeout:1 ndots:0\n","stream":"stdout"}

Actual behavior
Fluentd 1.1.0 drops 2 events:
$ docker run --rm -ti -v $PWD/x.log:/x.log -v $PWD/fluent.conf:/fluentd/etc/fluent.conf -e "FLUENTD_OPT=-qq" fluent/fluentd:v1.1.0
2018-02-20 14:26:51.666966175 +0000 docker.x.log: {"log":"nameserver <IP_ADDRESS>\n","stream":"stdout"}

Cause
Looks like the problem is caused by a 3rd-party dependency (oj). It has been introduced in
https://github.com/fluent/fluentd-docker-image/commit/a5792ee5a3fe716e2222ebefa8ce3b0e960f0e33#diff-106c3e6808cb07c5ed182eed25142b97
Diff:
- && gem install oj -v 2.18.3 \
+ && gem install oj -v 3.3.10 \

With oj 2.18.3 I see the expected behavior with oj 3.3.10 events are dropped.

I confirmed this behaviour.
The problem is oj changes exception handling.
Will fix soon.


Prevent handshake based signature forgery

Fixes #2983.
Intuition:

an attacker could locally craft a preimage that matches the ChallengeRequest::nonce value range.
the target then signs the nonce during the handshake.
the attacker is able to reuse the signed nonce in any other non-handshake interactions as an authenticity proof.

In order to mitigate this issue, this PR removes the possibility of brute forcing the preimage locally by generating a response_nonce that is signed alongside the initial nonce and sent in the ChallengeResponse so the signature can be verified.

Note: This will require a version bump for the Event and Message types, because this will break the current handshake protocol

Targeting this to mainnet branch given it contains breaking changes


Incorrect commandClass returned for Multilevel Switch CC targetValue ValueID

I'm not too familiar with the code, but I'm pretty sure this command class value is incorrect?
https://github.com/zwave-js/node-zwave-js/blob/fdbea264b00b480b5894789b5fac9d486ba72864/packages/zwave-js/src/lib/commandclass/MultilevelSwitchCC.ts#L101-L107

Uhh yeah... that is definietly wrong.

Uhh yeah... that is definietly wrong.


Remove jQuery dependency

Thanks for your library :)
Any chances that you will remove the jquery dependency one day?

No problem, glad it helps!
Yep, the jQuery dependency is maybe extraneous for a small library like this... But as most angular projects included it by default, I didn't bother too much to remove it.
But you're right, it should get rid off.
I'll try to do that soon

Thanks so much for your quick fixes!
I will check them out soon.

No problem!


I cant select hitbox on mobile

Describe your bug here. Mind you, if you are modding, please try testing your bug in a clean version of the engine instead, this ensures the bug is actually caused by the engine itself.
The two buttons to change my control on android is not working, can you fix that please? (The build target is Android)
Command Prompt/Terminal logs (if existing)
Doenst exist

Are you modding a build from source or with Lua?
Lua
What is your build target?
Neko, HashLink, or other build system
Did you edit anything in this build? If so, mention or summarize your changes.
No
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?

So well if you guys not fixing that, so just leave the hitbox

what i mean is click or tap everywhere and spamming it until it changes what ever mobile control you want. its currently bugged but doing what i told actually works

I dont wanna hurt my fingers spaming bruh

YES FINNALY, after almost breaking my screen i got hitbox

told ya

now close this issue

don’t make duplicates


[opentitantool] Avoid orphaned OpenOCD processes

If a opentitantool or other process invokes Jtag.connect() but terminates for any reason before invoking Jtag.disconnect(), the current code may leave an orphaned openocd process, which may hold on to USB resources.
The stock OpenOCD code does not seem to have provisions to terminate e.g. when stdin closes.  This CL insteads patches OpenOCD to have it request a SIGHUP from the Linux kernel, whenever its parent terminates for any reason.

Indentation in the patch should be tab to match upstream OpenOCD

I don't think that is worth stalling this patch. Merging.


Docker SCC Verifier image uploads artifacts twice

When trying to publish verifier stubs to Nexus releases via the Docker image, I stumbled across a problem, that stubs are uploaded twice. This was an issue, because the Nexus that I am using is set up in a way, that prevents updating anything in release repository. It caused a failure every time.
SCC version: 2.2.0-BUILD-SNAPSHOT from September 25th 2019, freshly built from master.
Reproducer: Try running provider verification using docker with PUBLISH_ARTIFACTS: "true" into a release repository, that does not allow updates.
Proposal (or possibly the solution) is available in #1223.

Fixed via https://github.com/spring-cloud/spring-cloud-contract/commit/f436d7fca9413cc7ecc3f6b41debe9e8b37c63cd


Compile classes in loose mode

This is low-risk and will give us a little better bundle size and
runtime performance. More information:
https://babeljs.io/docs/en/babel-plugin-transform-classes#loose

Coverage remained the same at 84.494% when pulling 4073d24c9b76966156866cf4972faa444bbd9cfc on loose-classes into 888f650565b4256dde39210fdba7faadba969516 on master.

<EMAIL_ADDRESS>


Is it possible to build at runtime

Can the gui be built at runtime, and bound to JSON

Do you mean this?
https://acrodata.github.io/gui/playground

yes, great!!
I also want to load this library directly in the browser, how can I do that ?

yes, great!!
I also want to load this library directly in the browser, how can I do that ?

I'm sorry, it doesn't support now. It's better to use in a web app.


Removal of 2.0 deprecations

We need to remove the following:

[x] Deprecated grid file, and any lingering uses of old grid
[ ] Deprecated old form module (Cory)
[ ] Old form styles (Cory)
[ ] Remove examples of clr-checkbox in button groups (Cory)
[ ] Remove old datepicker layout option (Matt)
[ ] Unnecessary gemini tests (Matt)

Have majority of the deprecated forms module and styles removed. Everything builds but I need to go back and check all three projects and gemini for any possible issues.

Is the older .toggle-switch part of the old forms deprecation? I see that the new component was added a couple of months ago.


Point in GraphicsLayer with PictureMarker not working

Reported on Discord by user addri
The following code block was not displaying the point:
<Map ArcGISDefaultBasemap="arcgis-navigation">
    <GraphicsLayer @ref="_graphicsLayer">
        <Graphic>
            <Point Longitude="_lon" Latitude="_lat" />
            <PictureMarkerSymbol Url="/images/marker.svg" />
        </Graphic>
        <Graphic Attributes="_graphicAttributes">
            <Polygon Rings="@(new[] { _mapRings })" />
            <SimpleFillSymbol Color="@(new MapColor(81, 46, 132, 0.8))">
                <Outline Color="@(new MapColor(255, 255, 255))" Width="1" />
            </SimpleFillSymbol>
        </Graphic>
    </GraphicsLayer>
</Map>

however, moving the point graphic to the MapView made it work.

I tried to reproduce this here, and it seems to work ok, closing.


Fix the buildifier test.

buildifier outputs diffs, but returns an exit code of 0.

cc @mattmoor you might want something like this in rules_docker.

Should we make it a generic script test.sh and run all the tests there?
I was also wondering if this should be done through bazel test, although I couldn't find any rules that did that already.

Should we make it a generic script test.sh and run all the tests there?

Most tests will be caught in the bazel test ./... we'll add back in once we get tests.

I was also wondering if this should be done through bazel test, although I couldn't find any rules that did that already.
I thought about that, but part of the point of this is to catch invalid BUILD files as well as mis-formatted onces, so it's a bit meta to use bazel test to execute it.


[js-testing] write tests for javascripts/dashboard/components/SliceAdder.jsx

Notice: this issue has been closed because it has been inactive for 331 days. Feel free to comment and request for this issue to be reopened.


JSX function component autocomplete adds parenthesis

Type: Bug
Currently if you autocomplete function component in JSX it will add parenthesis which is wrong and they shouldn't be there. As you can see function component adds parenthesis, but const component  don't which is a correct way. I expect function component in JSX skip parenthesis if I'm editing code inside JSX return like in the screenshots below.
Function component (invalid behavior) 🐛

Const component (correct behavior) 🟢

VS Code version: Code 1.76.2 (ee2b180d582a7f601fa6ecfdad8d9fd269ab1884, 2023-03-14T17:53:46.528Z)
OS version: Darwin arm64 22.3.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
3, 3, 3

Memory (System)
32.00GB (2.80GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

where-am-i
ant
0.2.0

vscode-eslint
dba
2.4.0

gitlens
eam
13.4.0

prettier-vscode
esb
9.10.4

auto-rename-tag
for
0.1.10

copilot
Git
1.78.9758

vscode-graphql
Gra
0.8.6

vscode-graphql-syntax
Gra
1.1.0

fluent-icons
mig
0.0.18

symbols
mig
0.0.9

vscode-react-native
msj
1.10.2

color-highlight
nau
2.5.0

vscode-thunder-client
ran
2.5.1

vscode-fileutils
sle
3.10.0

highlight-matching-tag
vin
0.11.0

vscodeintellicode
Vis
1.2.30

vim
vsc
1.25.2

quokka-vscode
Wal
1.0.528

(1 theme extensions excluded)

Thanks a lot @mjbvz it was really annoying issue! Also, I see we gonna have JSX linked editing which is very cool, I'm very happy! 🎉


Callback definitions are repetitive

Consider the following factory definition:
  factory :reservation do
    after(:stub) do |reservation, evaluator|
      if reservation.price_cents > 0
        build_stubbed(:payment, reservation: reservation)
      end
    end

    after(:create) do |reservation, evaluator|
      if reservation.price_cents > 0
        create(:payment, reservation: reservation)
      end
    end
  end

The only difference between the two callbacks is the strategy used to build the payment yet I had to duplicate a bunch of code to achieve that. I have this problem often - namely I want the same exact callback for several strategies but I want the code in the callback to use the same build strategy that triggered the callback.
It'd be great if the evaluator exposed the strategy used. Something like:
  after(:create, :stub) |reservation, |evaluator|
    if reservation.price_cents > 0
      evaluator.build_strategy(:payment, reservation: reservation)
    end
  end

Evaluator has an (private) instance variable which gets you an instance of the strategy. But we'd need to get from that to the name of the method to send in order to repeat that strategy on another object. If that makes sense.

I'd find this really useful as well, particularly for use with has_many associations and using build_list or build_stubbed_list in a single after(:build, :stub), rather than having two separate hooks where the only difference is the build strategy.

+1

Closing this in favor of https://github.com/thoughtbot/factory_girl/issues/458


Module not found: Can't resolve '../../../lerna.json'

Compilation error when running yarn start.
Failed to compile

./node_modules/blocks-ui/dist/header.js
Module not found: Can't resolve '../../../lerna.json' in '/home/daniel/code/screen-editor/node_modules/blocks-ui/dist'

Seems like the issue was introduced on this commit:
https://github.com/blocks/blocks/commit/5333f4b1a82d0ef78ececfdf9a7471f090885bea#diff-2b4d43d51f360ddfd79c8596b9a871feR5

Should be fixed in 0.0.54, thanks for opening the issue!


Support synchronous methods

There are some methods which only gives asynchronous option.
Like delete method in JMailTM class, or save method in Attachment class.
For me it looks like a good idea to give a synchronous version of this method.
It doesn't look a hard task to do since these methods have similar format(just some code snippet surrounded by new Thread(new Runnable() { ... }).start();).
So I'll be happy to do some contribution myself, but I'm not sure what is best option to achieve the goal.
1. Just make new synchronous method named like deleteNow

code would be like this :

    /**
     * (Synchronous) Deletes the Self Account
     */
    public void deleteNow(){
        try{
                IO.requestDELETE(baseUrl+"/accounts/"+id , bearerToken);
        }catch (Exception e){
                System.out.println("|EXCEPTION IGNORE | "+e);
        }
    }
    /**
     * (Asynchronous) Deletes the Self Account
     */
    public void delete(){
        new Thread(this::deleteNow).start();
    }

This would be a easy option, not effecting legacy code.
It may cause naming confusion, XX(Asynchronous) <-> XXNow(Synchronous) naming pattern doesn't match existing code like asyncXX(Asynchronous) <-> XX(Synchronous) (see fetchMessages)

2. Follow existing naming pattern

code would be like this :

    /**
     * (Synchronous) Deletes the Self Account
     */
    public void delete(){
        try{
                IO.requestDELETE(baseUrl+"/accounts/"+id , bearerToken);
        }catch (Exception e){
                System.out.println("|EXCEPTION IGNORE | "+e);
        }
    }
    /**
     * (Asynchronous) Deletes the Self Account
     */
    public void asyncDelete(){
        new Thread(this::deleteNow).start();
    }

There will be no naming confusion to users.
This does effect legacy code, making many existing code/project broken.

3. Make delete method returns Future object

Make a Thread Pool, and submit all asynchronous tasks(deleting account, saving attachments, fetching mails, etc..) to it.
In this case, submit method will return a Future object.
So it will be easy to wait for the thread, and WorkCallback class will be easily changed to Future<Boolean>
It is likely not to effect legacy code, since only return type of the method is changed. But, some rare case(e.g. method referencing like jmailtmObjectList.stream().foreach(JMailTM::delete)) will result an error.
A Thread Pool is used, so we should be careful to avoid bottleneck.(but Thread Pool looks like a better option than making new thread every time calling a method)

What would be the best option?

sorry i accidentally clicked closed lol
well i was planning to shift to future task as you said its not a good practice to create new thread all time
but right now i don't have free time so i would try to go with the legacy one
still you can make a pull request if you want.

Maybe could use shutdown hook to terminate Thread Pool in the end of the program, but It more looks like a punt than a solution.
I'll choose option 1, and add some synchronous versions of some methods.
I will make pull request when work's done :D


dart2js: remove methods that delegate to super-method

If all a method does is delegate to its super-method, the method can be removed, relying on the property-based dispatch to execute the super-method directly. The Flutter framework toString() methods have this property. Let us call these 'trivial' methods.
The requirement is that the optimization applies to methods containing assertions and other debug code that gets optimized away, leaving only a supercall. We can't tell prior to optimization that the method is trivial (and this is the whole point, otherwise the method would not have been written!)
Recognizing a trivial method is not straight-forward. The generated JavaScript does not have sufficient information to distinguish a trivial method from other methods. We would have to recognize trivial methods with a CFG pass just prior to JavaScript codegen. The cost of an extra pass can be ameliorated by detecting a super-call in an earlier pass.
Making the program work after the trivial method is removed is tricky because function bodies are compiled independently (for scalability). The problem is that subclasses may target the method we want to remove, but the super calls in the subclasses can't tell until too late that the target function has disappeared. It is impossible to go back and 'fix' the method that references one we removed.
class A {
  toString() => debugMode ? 'A' : super.toString();
}
class B extends A {
}
class C extends B {
  toString() => 'C:' + super.toString();
}

Typically C.toString will call a member via direct lookup
C.prototype = {
  toString$0: function() {
    return 'C:' + A.prototype.toString$0.call(this);
  }
}

or via an alias
    return 'C:' + this.super$A$toString$0()

where alias is created via an assignment prior to the prototypes being linked for inheritance
    A.prototype.super$A$toString$0 = A.prototype.toString$0

(dart2js currently uses both mechanisms, although the alias one has better performance.)
An additional source of references to trivial methods is tear-offs. Consider the expression super.toString in class B or C - that would be a call to the super-getter that creates a tear-off for the super-method.
I think the alias mechanism could be made to work for removing more trivial methods since the list of aliases is generated later, on the basis of calls using the aliases.
We can remove a trivial method that has no references. This would only be leaf trivial methods because trivial methods themselves contain references. We could account for this by tracking references from within trivial methods separately to other methods, and removing the references due to the removed trivial methods.  This approach would 'unpeel' the super-chain from the leaf class until a non-trivial method, but not remove any trivial calls that have non-trivial overrides.
To remove other trivial methods we would have to remove the references redirecting them to the non-trivial method, i.e.
    A.prototype.super$A$toString$0 = A.prototype.toString$0
-->
    Object.prototype.super$A$toString$0 = Object.prototype.toString$0

JavaScript ES6 has a super-call feature which we don't use because it is not supported by IE11.
What is nice about ES6 super is that it makes the call-chain references implicit. We can simply remove trivial methods that have no tear-off getter.
ES6 super-calls use the syntactic positioning of the method to determine the [[HomeObject]], which limits the JavaScript syntax available for creating instance methods. One limitation is that you can't inline a method containing super except into a method of the exact same class, otherwise super means something else. This might be important - currently over half of the super calls in dart2js generated code are inlined.

TL;DR: Given the complexity, special case nature, and relatively low impact of the optimization I think we should defer working on it until we are making better use of ES6.
/cc @ferhatb @sigmundch @vsmenon

Thanks for the detailed analysis - for the specific case of toString, I wonder then if we should instead push for the annotation-based tree-shaking approach (already tracked by https://github.com/dart-lang/sdk/issues/35303).

I looked at the flutter gallery and found there are 68 toString methods that directly call the super method after optimization. If all of these methods could simply be deleted, the code size saving would be ~0.12%.


Unable to load servers.json if oauth2 enabled only

Hello, we tried with your chart and for some reason unable to load pre-configured db definitions from servers.json to all users .
If I enabled internal mode it works but only for initial user.
Would be nice to have option load servers.json for oauth2 users.

Can you perhaps post your configuration so I can take a look at how you've set it up and what might be incorrect?

Hi @rowanruseler. Here is our config.
`kind: Secret
apiVersion: v1
metadata:
name: pgadmin4-config
type: Opaque
stringData:
config_local.py: |-
MASTER_PASSWORD_REQUIRED = True
AUTHENTICATION_SOURCES = ['oauth2']
OAUTH2_AUTO_CREATE_USER = True
OAUTH2_CONFIG = [
{
'OAUTH2_NAME': 'Azure',
'OAUTH2_DISPLAY_NAME': '',
'OAUTH2_CLIENT_ID': '',
'OAUTH2_CLIENT_SECRET': '**',
'OAUTH2_TOKEN_URL': '/oauth2/v2.0/token',
'OAUTH2_AUTHORIZATION_URL': 'oauth2/v2.0/authorize',
'OAUTH2_API_BASE_URL': '/oauth2/v2.0/',
'OAUTH2_USERINFO_ENDPOINT': 'https://graph.microsoft.com/oidc/userinfo',
'OAUTH2_ICON': 'fa-database',
'OAUTH2_BUTTON_COLOR': '#0000ff',
'OAUTH2_SCOPE': 'openid'
}
]
apiVersion: v1
kind: Secret
metadata:
name: pgpassfile
type: Opaque
stringData:
https://www.postgresql.org/docs/9.4/libpq-pgpass.html
pgpassfile: |
server.uat:#{password.uat}#
server.dev:#{password.dev}#
server.prod:#{password.prod}#`

replicaCount: 1

## pgAdmin4 container image
##
image:
  registry: docker.io
  repository: dpage/pgadmin4
  tag: "5.7"
  pullPolicy: IfNotPresent

## Deployment annotations
annotations: {}

service:
  type: ClusterIP
  port: 80
  targetPort: http
  # targetPort: 4181 To be used with a proxy extraContainer
  portName: http

  annotations: {}

serviceAccount:
  # Specifies whether a service account should be created
  create: false
  # Annotations to add to the service account
  annotations: {}

  name: ""

strategy: {}

serverDefinitions:
  ## If true, server definitions will be created
  ##
  enabled: true

  servers: |-
    "1": {
      "Name": "UAT",
      "Group": "Servers",
      "Port": 5432,
      "Username": "db",
      "Host": "server.uat",
      "SSLMode": "prefer",
      "MaintenanceDB": "db",
      "PassFile": "/var/lib/pgadmin/storage/pgadmin/file.pgpass"
    },
    "2": {
      "Name": "DEV",
      "Group": "Servers",
      "Port": 5432,
      "Username": "db",
      "Host": "server.dev",
      "SSLMode": "prefer",
      "MaintenanceDB": "db",
      "PassFile": "/var/lib/pgadmin/storage/pgadmin/file.pgpass"
    }

networkPolicy:
  enabled: true

ingress:
  enabled: true
  annotations:
    kubernetes.io/ingress.class: nginx
    kubernetes.io/tls-acme: "true"
    cert-manager.io/cluster-issuer: letsencrypt-prod
  hosts:
    - host: host.net
      paths:
        - path: /
          pathType: Prefix
  tls:
    - secretName: pgadmin-tls
      hosts:
        - pgadmin.host.net

# Additional config maps to be mounted inside a container
# Can be used to map config maps for sidecar as well
extraConfigmapMounts: []

extraSecretMounts:
  - name: config-local
    secret: pgadmin4-config
    subPath: config_local.py
    mountPath: "/pgadmin4/config_local.py"
    readOnly: true
  - name: pgpassfile
    secret: pgpassfile
    subPath: pgpassfile
    mountPath: "/var/lib/pgadmin/storage/pgadmin/file.pgpass"
    readOnly: true

existingSecret: ""

env:
  # can be email or nickname
  pgpassfile: /var/lib/pgadmin/storage/pgadmin/file.pgpass
  email<EMAIL_ADDRESS>
  enhanced_cookie_protection: "False"
persistentVolume:

  enabled: true

  annotations: {}

  accessModes:
    - ReadWriteOnce

  size: 10Gi

securityContext:
  runAsUser: 5050
  runAsGroup: 5050
  fsGroup: 5050

livenessProbe:
  initialDelaySeconds: 30
  periodSeconds: 60
  timeoutSeconds: 15
  successThreshold: 1
  failureThreshold: 3

readinessProbe:
  initialDelaySeconds: 30
  periodSeconds: 60
  timeoutSeconds: 15
  successThreshold: 1
  failureThreshold: 3

VolumePermissions:
  ## If true, enables an InitContainer to set permissions on /var/lib/pgadmin.
  ##
  enabled: true

## Additional InitContainers to initialize the pod
##
extraInitContainers: |
  - name: add-folder-for-pgpass
    image: "dpage/pgadmin4:4.23"
    command: ["/bin/mkdir", "-p", "/var/lib/pgadmin/storage/pgadmin"]
    volumeMounts:
      - name: pgadmin-data
        mountPath: /var/lib/pgadmin
    securityContext:
      runAsUser: 5050

containerPorts:
  http: 80

resources:

  limits:
    cpu: 300m
    memory: 500Mi
  requests:
    cpu: 150m
    memory: 400Mi

autoscaling:
  enabled: false
  minReplicas: 1
  maxReplicas: 100
  targetCPUUtilizationPercentage: 80

nodeSelector: {}

tolerations: []

## Pod affinity
##
affinity: {}

## Pod annotations
##
podAnnotations: {}

## Pod labels
##
podLabels: {}
  # key1: value1
  # key2: value2

init:
  ## Init container resources
  ##
  resources: {}

## Define values for chart tests
test:
  ## Container image for test-connection.yaml
  image:
    registry: docker.io
    repository: busybox
    tag: latest
  ## Resources request/limit for test-connection Pod
  resources: {}

  securityContext:
    runAsUser: 5051
    runAsGroup: 5051
    fsGroup: 5051

@IvanKolbasiuk you are missing Shared key in your definitions.
  servers: |-
    "1": {
      "Name": "UAT",
      "Group": "Servers",
      "Port": 5432,
      "Username": "db",
      "Host": "server.uat",
      "SSLMode": "prefer",
      "MaintenanceDB": "db",
      "PassFile": "/var/lib/pgadmin/storage/pgadmin/file.pgpass",
      "Shared": true

But I couldn't get username and passwords to be shared. So oauth2 users has the same server list but no username and no passwords set.

Sry for ping @rowanruseler but was wondering if you had similar issues? :bow:

Yeah I am not really sure what is going wrong here. I'll try to see if I can get some time available this week to test this out and see where it goes wrong.

I am also facing the same issue .. After adding Shared: True my Oauth2 users get prompted for both username and password, despite that they are present in the pgpass file. My configs are exactly as above.

can also attest to Oauth2 users needing db username and password

I'm seeing the same issue. It works as expected with the internal authentication but the Servers fail to load with an OAuth2 user authentication.

same here. seems like oauth2 users only have access to their homedir (/var/lib/pgadmin/storage/email_domain/ for me)

related pgadmin issue: https://github.com/pgadmin-org/pgadmin4/issues/5824

I think this issue is related to the general issue: https://github.com/rowanruseler/helm-charts/issues/193
This should be fixed in the docker 6.20 version.

Same here with both OAuth2 and internal users on multi-user installation. I have managed to share servers via "Shared": "true" JSON variable + managed to feed a password via pgpass file, but PG username is still required for all imported servers for all pgAdmin users except the one which is importing JSON file.
If a user sets username for at least one time then it gets stored in their preferences and won't be required anymore.
I guess this has something to do with the way pgAdmin encodes usernames/passwords in a database. Maybe something related to master password.
P.S. Note that pgpass file can only be used to set the password. First four parameters in that file, including username, is used only for locating a right password, not to set in the connection string.

For me it only asks for username, which is related https://github.com/pgadmin-org/pgadmin4/issues/6229

@bitnik When Shared is true and SharedUsername is set, the logged in oauth-user still gets asked for password. How do you provide it for the shared configuration? Passfile? How? Relative or Absolute? In which key?


centos 7: uwsgi install succeeds, but can't be run, manual pip commands can't be run

I've added a cookbook so it can easily be tested. Just run a vagrant up inside the cookbook and you should be able to produce the following easily:
[vagrant@pootle-berkshelf ~]$ sudo su pootle
bash-4.2$ cd ~
bash-4.2$ source /opt/pootle/bin/activate
(pootle)bash-4.2$ uwsgi
uwsgi: error while loading shared libraries: libiconv.so.2: cannot open shared object file: No such file or directory
(pootle)bash-4.2$ pip uninstall uwsgi
Traceback (most recent call last):
  File "/opt/pootle/bin/pip", line 7, in <module>
    from pip import main
  File "/opt/pootle/lib/python2.7/site-packages/pip/__init__.py", line 21, in <module>
    from pip._vendor.requests.packages.urllib3.exceptions import DependencyWarning
  File "/opt/pootle/lib/python2.7/site-packages/pip/_vendor/requests/__init__.py", line 62, in <module>
    from .packages.urllib3.exceptions import DependencyWarning
  File "/opt/pootle/lib/python2.7/site-packages/pip/_vendor/requests/packages/__init__.py", line 29, in <module>
    import urllib3
ImportError: No module named urllib3

Manual pip commands (directly with pip or with python -m pip) all fail with either this error or an unknown exception error.
Does anybody have any idea why this happens?
cookbook-pootle.zip

This is because the default install provider on RHEL and CentOS is the SCL packages, which have to be used via the scl enable command line tool. Within Chef, the cookbook hides this from you because the install source is modular. If you want to use system packages, you can configure that either in node attributes, via the python_runtime_options pseudo-resource, or on the python_runtime resource itself.

Thanks @coderanger for the quick reply. Pip does work again now, still running into a problem with uwsgi, but that's something outside of this cookbook I think.
However, this behaviour, wouldn't it make sense to document this somewhere? E.g. under a known problems section in the readme. Since SCL is the default on centos it seems and a virtualenv is used a lot.

The scl provider installs Python using the Software Collections packages. This is only available on RHEL, CentOS, and Fedora. SCL offers more recent versions of Python than the system packages for the most part. If an SCL package exists for the requested version, it will be used in preference to the system provider.


Import Rao et. al data into Schema Destroyer

Try out data wrangler / data submission stuff on the new schema.

Finally done.
Files are not imported, this is the only missing piece


Can't delete older versions of solutions

When we refactor our kata solutions, we should be able to delete old versions from our profile!

Duplicate of #540


Inserted all rounds

I have inserted all rounds of serie A for the season 2021-22

Thanks for the updates. Keep it up. Mille grazie.


[FEATURE] Minimum cost MVP including perimeter account for personal/contractor shadow PBMM environment

Required Basic Info
To properly assess the enhancement request, we require information on the version of the Accelerator you based this request upon:

Accelerator Version:  (eg. v1.1.6)
Latest

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
No
Describe the solution you'd like
A clear and concise description of what you want to happen.
this is more of a documentation verify during a minimally viable SEA install on a clean account that includes the perimeter account.  Once I have the system up i will iterate down to the minimum cost MVP
Initially documenting on http://wiki.obrienlabs.cloud/display/DEV/AWS+Secure+Environment+Accelerator
I will port or mirror the details as I go to the wiki/readme here
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Be warned - I approach this piece of work as an overly enthusiastic but humble developer - or at least I am tentatively hopeful that I can complete the install.

Hey!

did you take a look at option #5, which was add last month: https://github.com/aws-samples/aws-secure-environment-accelerator/blob/main/docs/installation/customization-index.md - this is a start in this direction? :-).
also, it is difficult to iterate down, easier to iterate up - some object types cannot be downsized (i.e. MAD), other objects cannot be easily removed (regions, VPC's, accounts), etc.
"The question is can I combine the 2 accounts via AWS Organizations?" - Which 2 accounts?  Please keep in mind AWS accounts generally cost nothing, the resources you run in accounts that cost $.  Minimum deploy is 3 accounts per the ultra-lite example.

sounds good, I cleaned one of my accounts and will enable AWS organizations there - I asked to upgrade the limit from 4 to 6 accounts there and just got the limit approved.  I'll start with the light option - I see you recently added ultra-light - very nice.  Ideally i'd use the minimum config that keeps the perimeter account above the ultra-light - but will MVP the ultra-light to start.  The core test cases we are implementing are around direct S3 PUTs and SDK REST calls around the S3 Transfer Manager SDK and VPC Gateways leveraged via the transit gateway for APIGW/DynamoDB traffic between separate app accounts

With the launch of v1.5.0 and updating example #5 to leverage NFW instead of 3rd party firewalls, I believe this meets the original ask with respect to PBMM.  If you were not using MAD as your identity source, it could be further streamlined between example 5 and example 3.