unable to run dsSimulate in MATLAB 2019b

Everything works fine when I was using MATLAB 2018a. I updated MATLAB to 2019b and received the following error:
Error: File: dsImportModel.m Line: 236 Column: 43
Invalid syntax for calling function 'varargin' on the path. Use a
valid syntax or explicitly initialize 'varargin' to make it a
variable.
Error in dsGenerateModel (line 241)
[tmpmodel,tmpmap]=dsImportModel(equations,'namespace',PopScope,'ic_pop',specification.populations(i).name,'user_parameters',parameters);
Error in dsCheckModel (line 102)
model=dsGenerateModel(model);
Error in dsSimulate (line 509)
model = dsCheckModel(model, varargin{:}); % handles conversion
when input is a string w/ equations or a DynaSim specification
structure
Error in myFunction(line 155)
data = dsSimulate(s,'time_limits',[dt time_end],
'solver',solverType, 'dt',dt,...

Can you share a minimal example that triggers the issue?

Matlab deems line 236 in dsImportModel to have a syntax error:
[model,map]=dsParseModelEquations(source, varargin{:});

I need a piece of code (e.g., your model) that I can run to trigger the error. Preferentially the example code should be as minimal as possible.
Thank you

I understand, but what piece of which demo file is failing?
Of course I can install Matlab2019b and run all demo files myself (now that I know about them triggering the issue), but this is also about reporting issues properly (please do not take it personally), and having an example of code failing is essential for devs to rapidly reproducing the issue, and eventually fixing it.

Any call to dsSimulate will trigger the error. For example, the Lorentz equation demo will trigger it
eqns={   's=10; r=27; b=2.666'   'dx/dt=s*(y-x);   x(0)=1'   'dy/dt=r*x-y-x*z; y(0)=2'   'dz/dt=-b*z+x*y;  z(0)=.5' }; data=dsSimulate(eqns, 'time_limits',[0 100], 'solver','rk4');
This error did not occur in Matlab2018b and only started after I updated to Matlab2019b. I traced the error to line 236 of dsImportModel. Matlab was not executing any code due to the syntax error it detected in dsImportModel. Sorry that it wasn't clear before.

Thanks, will check it out

Mmm, I am running Matlab 2019b in an external drive and get an error from this piece of code, but the error message is different than in your original report:
Error: Command syntax error see SIM command for details
         in dsSimulate/update_data (line 1197)
         in dsSimulate/tryFn (line 1184)
         in dsSimulate (line 830)
         in testDynaSim (line 9)
Error using dsSimulate/update_data (line 1197)
Command syntax error see SIM command for details

Error in dsSimulate/tryFn (line 1184)
        update_data; % concatenate data structures across simulations

Error in dsSimulate (line 830)
    tryFn(nargout)

Error in testDynaSim (line 9)
data=dsSimulate(eqns, 'time_limits',[0 100], 'solver','rk4'); - Show complete stack trace

So @kfchou, do you also see the same error for this example code? If so, it means that there is more than one error. Could you please confirm?

I fixed the issue triggered by the example code you shared (dev branch), so I'm closing this issue.
If the original issue is not fixed by it, please reopen and attach a new example code specific for it.

hmm that didn't fix the issue for me. I was able to resolve it by adding varargin as an input argument to the subfunction downloadModel() in dsImportModel.m, line 194
function source=downloadModel(ModelID,**varargin**)
similarly, add the input argument to the function call on line 49
source=downloadModel(ModelID,**varargin**);
Seems like there may be multiple syntax errors in the latest Matlab update. Will keep an eye out.

Interesting. What DynaSim version do you use (1.0, master branch or dev)?
Also, is the error that you mention triggered with the same code that you
shared or a different one?

I had the same error and fixed it. The problem was that the newer MATLAB versions require explicit declaration of all the variables that go into a parfor/spmd loops. Otherwise the compiler doesn't even allow for debugging. So, if you are using eval at somewhere, pay closer attention.

Hi there, I just installed dynasim through github on a mac and I'm getting the same error while trying to run the tutorial in matlab R2020b:
eqns={
's=10; r=27; b=2.666';
'dx/dt=s*(y-x)';
'dy/dt=rx-y-xz';
'dz/dt=-bz+xy';
};
data=dsSimulate(eqns,'tspan',[0 100],'ic',[1 2 .5],'solver','rk4');
Error: File: dsImportModel.m Line: 236 Column: 43
Invalid syntax for calling function 'varargin' on the path. Use a valid syntax or explicitly
initialize 'varargin' to make it a variable.
Error in dsGenerateModel (line 241)
[tmpmodel,tmpmap]=dsImportModel(equations,'namespace',PopScope,'ic_pop',specification.populations(i).name,'user_parameters',parameters);
Error in dsCheckModel (line 102)
model=dsGenerateModel(model);
Error in dsSimulate (line 512)
model = dsCheckModel(model, varargin{:}); % handles conversion when input is a string w/
equations or a DynaSim specification structure

Hello, Do you get the same error if you use the development version located
here: https://github.com/DynaSim/DynaSim/tree/dev
On Mon, Jul 26, 2021 at 9:06 AM Margarita Zachariou <
@.***> wrote:

Hi there, I just installed dynasim through github on a mac and I'm getting
the same error while trying to run the tutorial in matlab R2020b:
eqns={
's=10; r=27; b=2.666';
'dx/dt=s*(y-x)';
'dy/dt=rx-y-xz';
'dz/dt=-bz+xy';
};
data=dsSimulate(eqns,'tspan',[0 100],'ic',[1 2 .5],'solver','rk4');
Error: File: dsImportModel.m Line: 236 Column: 43
Invalid syntax for calling function 'varargin' on the path. Use a valid
syntax or explicitly
initialize 'varargin' to make it a variable.
Error in dsGenerateModel (line 241)
[tmpmodel,tmpmap]=dsImportModel(equations,'namespace',PopScope,'ic_pop',specification.populations(i).name,'user_parameters',parameters);
Error in dsCheckModel (line 102)
model=dsGenerateModel(model);
Error in dsSimulate (line 512)
model = dsCheckModel(model, varargin{:}); % handles conversion when input
is a string w/
equations or a DynaSim specification structure
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DynaSim/DynaSim/issues/575#issuecomment-886685571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPSUCJSFKGWK6VXROKZSXDTZVMTZANCNFSM4IYDJ5IA
.

Thank you, when I download the dev version as a zip it works fine (I had used git previously presumably a different version was downloaded through that).


Dependency tune-up of solr-core

Dependency tune-up of solr-core:

Reduce "api" scope of solr-core, thus other modules need to explicitly depend on what they need
Organized dependencies solr-core's build.gradle
Used transitive dependencies much more
Outside solr-core:
Removed some usages of dependencies in favor of JDK
Added some explicit dependencies

Note that there are no changes to the dependency list of the project overall, and thus no new sha1/license.  Some small changes in WEB-INF/lib nevertheless like removal of commons-logging

I made further improvements, especially in SolrJ.
There is one mystery to me.  The license & sha1 check system told me to remove the ones for Janino but I didn't remove Janino; it's merely a transitive dependency now (via Calcite-core).

org.codehaus.janino:janino:3.0.11
|         --- org.codehaus.janino:commons-compiler:3.0.11

WEB-INF/lib has these.
I hope we're fine here without a JIRA issue.  My rationale is that this is just a refactoring.  I set out for the dependencies to be the same in the end but there are a couple minor tweaks I'm aware of as evidenced in this PR -- kerby-kdc isn't needed (transitive to Hadoop stuff), nor is eigenbase-properties (transitive to Calcite stuff).  I suspect these were dependencies that may have been required at some point but after an upgrade and now aren't, or maybe they never were.  Tests pass -- hopefully a good enough check.  We have other transitive dependencies from Calcite that I suspect we don't need either (protobuf and esri stuff) that I could exclude but didn't because my objective here is not to go pruning the project's dependencies, yet I admit I in effect did this with two above.  So; should I declare explicit dependencies to bring back what we might not need, leaving pruning for another issue that could have a bigger scope?

I'm confused about versions.props. I thought this pinned our versions but it seems not because this is what is in there:
org.codehaus.janino:*=3.0.9
Yet Gradle selected 3.0.11.

It's not gradle - it's palantir's consistent versions. "versions.props: lower bounds for dependencies" - it'll select the highest version across all the required versions.


IOException wrapped in InternalError not handled properly

Describe the bug
The call to socket.connect(...) at WebSocketClient:436 can throw a java.lang.InternalError that is currently not handled in the enclosing try/catch as it extends java.lang.Error and not java.lang.Exception
This InternalError wraps a java.lang.IOException that contains useful information to the websocket engine to hande this case.
To Reproduce
Steps to reproduce the behavior:

Install a proxy, i.e. squid
Make it deny connections

Example application to reproduce the issue
Use the previously installed proxy in the ProxyClientExample
Expected behavior
An IOException handled by the underlying websocket engine
Debug log
Exception in thread "WebSocketConnectReadThread-11" java.lang.InternalError: Should not reach here
	at java.net.HttpConnectSocketImpl.doTunneling(HttpConnectSocketImpl.java:181)
	at java.net.HttpConnectSocketImpl.doTunnel(HttpConnectSocketImpl.java:168)
	at java.net.HttpConnectSocketImpl.access$200(HttpConnectSocketImpl.java:44)
	at java.net.HttpConnectSocketImpl$2.run(HttpConnectSocketImpl.java:151)
	at java.net.HttpConnectSocketImpl$2.run(HttpConnectSocketImpl.java:149)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.HttpConnectSocketImpl.privilegedDoTunnel(HttpConnectSocketImpl.java:148)
	at java.net.HttpConnectSocketImpl.connect(HttpConnectSocketImpl.java:111)
	at java.net.Socket.connect(Socket.java:589)
	at org.java_websocket.client.WebSocketClient.run(WebSocketClient.java:436)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at java.net.HttpConnectSocketImpl.doTunneling(HttpConnectSocketImpl.java:179)
	... 10 more
Caused by: java.io.IOException: Unable to tunnel through proxy. Proxy returns "HTTP/1.1 403 Forbidden"
	at sun.net.www.protocol.http.HttpURLConnection.doTunneling(HttpURLConnection.java:2142)
	... 15 more

Environment(please complete the following information):

Version used: 1.4.0
Java version: 1.8.0_181
Operating System and version: Ubuntu 16.04 LTS
Endpoint Name and version: "ws://echo.websocket.org"
Link to your project: ProxyClientExample in the examples folder

Additional context
This is my first time reporting an issue on a open source project.
I'm open to feedback :)

Hello @PearSlice,
thank you for creating this bug report!
Looks good :)
Best regards,
Marcel


Player shapeshifting.

Describe your idea, include images if necessary
Ummm, not sure if this is already a feature (could not find) but If it isn’t then I would love it if you could shapeshift into other players. As I said if this is already a thing I apologize but I could not find a way to do it (if it is pls tell me how) imagine you walk up to someone who thinks your their friend and then backstab them mercilessly.

SkinShifter is finally available on CurseForge and Modrinth. I'll upload a ReMorphed update for compatibility soon, too.

YEAH!!!


a few changes

I'm closing this as I've made even more changes to the point that there is not much resemblance to the original send-dataset.


Please stop using Str::headline in Resource class or alow some kind of disable flag as is breaking the bread crumbs and labels

Package
filament/filament
Package Version
v3.0.0
Laravel Version
v10.0.0
Livewire Version
No response
PHP Version
PHP 8.1.10
Problem description
For example if i use the string 'E-mailuri blocate' as label i expect to get that string not

Expected behavior
Don't change my strings unless i say so by a config or some other method
Steps to reproduce
Use in any resource
public static function getModelLabel(): string
{
    return 'My-custom label';
}

public static function getPluralModelLabel(): string
{
    return 'My-custom label';
}

Reproduction repository
https://github.com/filamentphp/demo
Relevant log output
No response

The easiest approach would be to have a private property for the breadcrumb string like with the navigation label and heading where we can override the default behavior which probably works nicely in 80% of the cases. For the other 20% some level of control would be great.

The breadcrumb can be customized separately using the $breadcrumb property on a resource or page, or the getBreadcrumb() method.
I do think however that maybe we shouldn't use the headline() method, but use ucfirst() instead, @danharrin.

The idea is that each word is capitalised based on title case, not just the first
And yes, this can easily be overridden in the breadcrumb, page titles etc

@zepfietje @danharrin if you have a ton of resources is not that fun to do.
Or if you have some plugin installed that doesn't allow you to extend the resource etc.
So please provide a method to turn off this behavior for people that doesn't need/want it (maybe it makes a lot of sense for english language but for some other languages that is a problem).
Also the - are replaced with spaces.
Thanks for your time and understanding.

It is not as simple as you are making it sound, the label is used in other places, not just this.
I will think about adding a new property where you can set the breadcrumb and page titles at the same time.


Fix docker tar generation for release builds

This change allows release builds to complete successfully.
The baseline docker targets used to build images as $(IMAGE) (which implies $(IMAGE):latest) and yesterday a change was committed that changed this to $(HUB)/$(IMAGE):$(TAG) but didn't update the tar-generation rule to reflect this name change.  This change primarily updates the tar-generation rule to reflect the change.
The release build doesn't care about the HUB (it just wants the tar files), but I can't set an empty hub (the Makefile doesn't permit one, but even if it did the image name would probably end up with a '/' prefix).  The Makefile's default hub is "istio", and this name is also now baked into the tar so I had to update release/store_artifacts.sh and release/publish_release.sh to reflect this change.
I also changed release/cloud_builder.sh to specify a HUB.  This is just to help insulate the build from any change to the default HUB in the top-level Makefile.  Although "docker.io/istio" is specified the docker tool seems to implicitly change this to just "istio".
I tested that a release build can complete successfully with this change, though I did no testing of the artifacts to see if there's any 2nd order side effects.

@mattdelco: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
2f9a1e29b7ec7ccdadd3a6e80a924a4ee68a4af7
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e

/lgtm
To prevent further breakages - we can add a "make tar" (and copy result to artifacts) in the nightly build from CI.


Rss to html

Step 1 of the pipeline, looking to incorporate a feature to read xml files in addition to rss feeds.

Looks great! the column names that we need for SQL compatibility are "publication, url, article_title, date"


Retrieve the peer certificate from the certificate array

Fixes #1076
The SSLContextGrpcAuthenticationReader reads the last certificate from the peer certificates array, however I believe the intent was probably to retrieve the peer certificate, not an intermediate certificate.
https://github.com/grpc-ecosystem/grpc-spring/blob/de71ce3deaa48c220bcade928268806c5e971656/grpc-server-spring-boot-starter/src/main/java/net/devh/boot/grpc/server/security/authentication/SSLContextGrpcAuthenticationReader.java#L56
The Javadoc of javax.net.ssl.SSLSession#getPeerCertificates specifies that it returns:

an ordered array of peer certificates, with the peer's own certificate first followed by any certificate authorities.

If there are no intermediate CA then the array with have length 1, and there will be no difference in behavior. This is probably why this bug has not been reported before (I don't think).

@ST-DDT I've added tests, it was a bit of a pain because there were no tests for the certificates/TLS yet

@ST-DDT I've added tests, it was a bit of a pain because there were no tests for the certificates/TLS yet

There are integration tests:
https://github.com/grpc-ecosystem/grpc-spring/blob/de71ce3deaa48c220bcade928268806c5e971656/tests/src/test/java/net/devh/boot/grpc/test/setup/SelfSignedMutualSetupTest.java#L40-L45

There are integration tests:

Ah I didn't see those, but there are no tests or certificates with an intermediate AFAIK. Are you OK with the approach I took?

Ah I didn't see those, but there are no tests or certificates with an intermediate AFAIK. Are you OK with the approach I took?

Yes, I played with your tests a bit to ensure, that we don't just test the mock certificate arrays.
All seems to be working as expected. (Although I must admit, that I know very little about certificates especially in Java).

@ST-DDT is there something else we need to do to get this merged?

@ST-DDT is there something else we need to do to get this merged?

More reviews.

@ST-DDT is there something else we need to do to get this merged?

More reviews.

This has been sitting here a few weeks now, what is required to get more people to review this? Can we tag someone?

@yidongnan @stanley-cheung @fengli79 Could one of you review this so it can be merged? I am not sure who to tag, just found you guys as recent reviewers of closed PRs.


⚠ Ion has degraded performance

In efb3e4c, Ion (https://ion.tjhsst.edu) experienced degraded performance:

HTTP code: 200
Response time: 5705 ms

Resolved: Ion performance has improved in eec3163.


[stable] Add back Manifest::targets_mut

It is needed by cargo-c, it was removed in df5cb70e7bc8872216af736f108f5a959a6d2302.
This is a stable backport of #8494. It's technically fixing a stable/stable regression, so this seemed like a good idea. If this is accepted I can also prepare a backport to 1.46 (current beta).

Hm, highfive didn't assign for some reason, let's r? @ehuss

Would y'all also be prepared to make a point release of Cargo, and if so that version bump should probably happen here too. This seems like a harmless backport but this isn't something that we've done before.

Oh, yeah, including the version bump makes sense to me. I've pushed up a commit doing so, though please check (not sure if that's the only place I need to update).
I was thinking we'd potentially just release it to crates.io without even inclusion in upstream Rust, since it won't affect that, but happy to do either -- we haven't yet built artifacts for rust stable point release so there's still time.

I was thinking we'd potentially just release it to crates.io without even inclusion in upstream Rust, since it won't affect that

Agreed. This doesn't need a point release of Rust binary artifacts, because anything using cargo-the-crate rather than cargo-the-binary should be using cargo from crates.io.

To be clear we're doing a binary promotion anyway, so it doesn't matter whether this is in it or not to me.
I would like to get a hard yes/no on that by end of this week though if possible, so we can get artifacts built and ready for shipping next Thursday. Let me know if I should do anything further or can help coordinate.
(It also looks like CI here is failing, but I suspect that's spurious?)

We talked about this in the Cargo meeting today and our conclusion was that it's fine to backport this. More generally, though, we're willing to review these sorts of API-level backports but none of us really wanted to manage the work to actually do the backport. @Mark-Simulacrum are you ok signing up to do all that? We were thinking it includes things like:

The stable backport (this PR)
The version bump (this PR)
The publish once merged
The beta backport
The stable submodule update
The beta submodule update

Would you be ok being responsible for this? So long as someone's signed up to do all the work we're ok backporting changes like this.

I can do all that. I seem to recall hearing we should just merge this PR, rather than invoke bors?
I'll post beta backports and such tomorrow.

Yeah bors doesn't listen to azure and nightly is broken for unrelated reasons so manual merge is fine


Adding NuGet Tool Installer task

i had a small comments only. looks good

@pspill , @ericsciple I've addressed all of your comments.


made changes for M2 mac support

I've updated package versions in requirements.yml to use conda packages available to M2 macs.  Because of an update to flask-sqlalchemy it also required a one-line change to minerva_analysis/server/models/database_model.py.  This is explained in the following link:
https://stackoverflow.com/questions/73961938/flask-sqlalchemy-db-create-all-raises-runtimeerror-working-outside-of-applicat

I tested on my home Ubuntu 22.04 desktop at it seems to run fine.

I found a less invasive change set that doesn't require touching the app code. Rob can you verify whether this works for you too? Here are just the few dependency specs I changed. I also removed the pip section entirely since opencv is available in conda-forge (I got some missing library errors from the pip-installed opencv-python wheel on my mac, anyway, so that wasn't even usable).
- python
- sqlalchemy<2
- opencv=4.5.3

Yes, that works for me on my M2 laptop.  You're right, that's better than modifying existing code.  Thanks!


fix: switches hvac modes properly in dual mode

Fixes #5

Thanks @swingerman - I'll check this out soon 👍


Add cfg for event feature to event tests (#45)

This change fix the cargo test error with default feature

@Berrysoft here

Is it ok now?

Thank you! However by thinking twice, I think a more suitable way is to change Cargo.toml and add required-feature to this [[test]].

❯ git diff
diff --git a/Cargo.toml b/Cargo.toml
index 47786a3..2bcfc87 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -92,6 +92,8 @@ once_cell_try = []
 read_buf = []
 nightly = ["read_buf", "lazy_cell", "once_cell_try"]

+test_event = ["event"]
+
 [[example]]
 name = "tick"
 required-features = ["time", "signal"]

Do you mean this? I feel it's over complex, you need to enable test_event when testing, but to enable event when building.

https://doc.rust-lang.org/cargo/reference/cargo-targets.html#tests

yes, it looks much better :)


Remove dependency on std

Please refer to #6 for more information.

Hi,
@lthiery, @IGBC I was looking at options to remove std from the core part of the library while still providing to_string as optional part of the library - if people want to use it (and similarly to move some other parts of the library to separate modules). What I believe should work is extract those to separate modules and do the implementation of to_string, etc in such additional modules. My thinking is that this way only those additional modules will depend to std and the core part of the library will not. It seems to me that this will solve your issue as you would depend only on the core part. As I do not have tons of experience with rust modules and embedded implementations for now, could you confirm that this plan sounds good to you?
Thanks in advance! If you or someone else provides feedback that this will be a working solution, I should have something testable some time next week.

You could also gate them behind feature flags, this is done by a lot of other libraries who aim for std/no-std compatibility.
IMO there's no right or wrong way to do it as long as how to use it in both configurations is clearly documented with examples.
(emailed in from a phone, exuse my brevity)
8 Mar 2020 13:36:54 Ivaylo Petrov<EMAIL_ADDRESS>
Hi,
@lthiery [https://github.com/lthiery] , @IGBC [https://github.com/IGBC] I was looking at options to remove std from the core part of the library while still providing to_string as optional part of the library - if people want to use it (and similarly to move some other parts of the library to separate modules). What I believe should work is extract those to separate modules and do the implementation of to_string , etc in such additional modules. My thinking is that this way only those additional modules will depend to std and the core part of the library will not. It seems to me that this will solve your issue as you would depend only on the core part. As I do not have tons of experience with rust modules and embedded implementations for now, could you confirm that this plan sounds good to you?
Thanks in advance! If you or someone else provides feedback that this will be a working solution, I should have something testable some time next week.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [https://github.com/ivajloip/rust-lorawan/issues/7?email_source=notifications&email_token=AB57LQ7Q36EL6AMM7BJM6U3RGONPLA5CNFSM4K7IEB22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOEWINY#issuecomment-596206647] , or unsubscribe [https://github.com/notifications/unsubscribe-auth/AB57LQ2WVQUNVPFRD2WONUDRGONPLANCNFSM4K7IEB2Q] . [https://github.com/notifications/beacon/AB57LQYZRHFMIF52BHTGCJDRGONPLA5CNFSM4K7IEB22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOEWINY.gif]

Thank you very much for your reply! I ended up having a mixture of separate module + flags for now. @lthiery, please let me know if you face any issues with this solution. The version is currently released as 0.5.0-rc0.

Awesome! I have some Rust bindings for the Semtech SX12xx drivers that I hope to connect to this soon. Thanks @ivajloip


fix badges in readme.md

What does this MR do?

fixes broken links in README.md

Pull Request Test Coverage Report for Build 5

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

Totals

Change from base Build 2:
0.0%

Covered Lines:
195

Relevant Lines:
253

💛  - Coveralls


K8SPXC-1165 - Fix big-data, proxy-protocol and self-healing-chaos tests

CHANGE DESCRIPTION
big-data test is failing on openshift because we don't wait enough.
self-healing-chaos is checking proxysql vars which has little value and we constantly need to update this so decided to remove this part.
proxy-protocol test was not updated for openshift when the PR was merged for default authentification plugin.
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?
[ ] Are OpenShift compare files changed for E2E tests (compare/*-oc.yml)?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PXC version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
failure

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
failure

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1439/commits/565d67f8a50906fbc7cfe2a3950af62ef4d82aac
image: perconalab/percona-xtradb-cluster-operator:PR-1439-565d67f8

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1439/commits/565d67f8a50906fbc7cfe2a3950af62ef4d82aac
image: perconalab/percona-xtradb-cluster-operator:PR-1439-565d67f8


I can't run the script which named "backdoor_badnet_fedavg_convnet2_on_femnist.yaml"

Hello! I know fs from your paper "Revisiting Personalized Federated Learning: Robustness Against Backdoor Attacks".And I am studying fs in order to set up an experiment related to backdoor attacks using fs. However, I found that I cannot find relevant documentation, and the sample scripts in the code cannot be executed. Is this functionality in place, or is there an issue with my configuration environment?
Other scripts like "femnist.yaml" and "CRA_fedavg_convnet2_on_femnist.yaml" can run normally.

I have modified the .yaml file according to your advice, but it seems there is more than one error in the file.

"batch_or_epoch: epoch" is what I modified based on femnist.yaml, but I'm not sure where the dataset should belong.

I checked the file you provided, but I still did not find which category "dataset" belongs to, and I also found that there were no categories of "optimizer" and "trainer" in the file，however these two are present in the.yaml file.is it the fault of the file you provided or the fault of the.yaml file?

I checked the file you provided, but I still did not find which category "dataset" belongs to, and I also found that there were no categories of "optimizer" and "trainer" in the file，however these two are present in the.yaml file.is it the fault of the file you provided or the fault of the.yaml file?

Hello! Thank you for your interest in our paper. I just want to verify which branch you are currently on. If you are in the master branch, could you please switch to the branch named "backdoor-bench"? The related scripts to conduct the backdoor attacks are under "scripts/backdoor_scripts/attack_config".

We apologize for any confusion caused by the documentation being located only in the master branch（ https://github.com/alibaba/FederatedScope/blob/master/benchmark/Backdoor-bench/README.md ）. We appreciate you bringing this to our attention and will work to rectify the issue as soon as possible. Thank you for your patience and understanding.

Hello! I am currently using the master branch. I've checked the files you provided, but I have some concerns. As stated in the files, FS is an 'open-sourced library that updates frequently,' and I'm worried whether the existing official documentation is still valid for Backdoor-bench after many updates. If I intend to use custom datasets like MNIST, can the current official documentation serve as a reference?

sorry,the errors that exist in the master branch seem to also exist in the Backdoor bench。

I have checked all the. yaml files in the "backdoor_scripts \ attack_config" directory and I am sure that they all have the same issue. @Osier-Yi @rayrayraykk

I'm sorry for arbitrarily assuming it was an issue with the. yaml file before carefully checking. I didn't realize that the. yaml file format in the backdoor bench is different. I have other questions, but to help others who see this issue, I will create a new one. Thank you for your patience in answering.

Hi, @tntstation
Thank you very much for your question.  It helped me solve the issue with running the specified config.  However, as a beginner, I still have many questions I’d like to ask.  If possible, I’d love to connect privately for further discussion.  My email is<EMAIL_ADDRESS>and I look forward to your response.


More Confirmed iGPUs/dGPUS

AdvancedMap working fine in Sonoma on the following iGPUs/dGPUS
Lenovo T530, with Intel HD 4000 (patched with OCLP):

Lenovo T490, with Intel UHD 620:

iMac12,2 with NVIDIA QUADRO K1100m (patched with OCLP):

Does the 2011 even support Metal? So does it mean that AdvancedMap doesn't even need Metal support?

Does the 2011 even support Metal? So does it mean that AdvancedMap doesn't even need Metal support?

No, because it uses an AMD Terrascale card. But I upgraded mine with an NVIDIA Quadro K1100m.

GA-Z490 Vision with Intel I9-10850K and AMD RX580 4GB using AdvanceMap 4.0:


Add 'after_on_activate()' call

This pull request adds a way to extend the on_activate function without making public mob_activate() and without totally replacing it. I understand that calling mob_activate() is essential for the API to work, so instead of adding a way to override it, I have added the ability to do an extra function call after mob_activate() has been called.
The extra result paramter in api.lua I think it is unnecessary; I left it because the original function does return mob_activate(), but I have seen that the function returns nothing. This can be removed and will make the code cleaner.

We already have on_spawn functions that are called after initial first activation, how would adding another function benefit mobs ?

This is very useful when a one-time adjustment to the mob has to be done every time is activated. on_activate is called every time the object is reloaded, so this includes map reload, game restart, etc.
The one reason I want this for is to be able to restore the last animation the mob had. Jn mu case, if one of my NPCs lay or sit, and I go far away or restart the game, the NPC is invariably standing. That is done in mob_activate. With this, I can set the animation state to whatever it was. Quite sure it can have other uses as well, it is a way to extend mob_activate


No Windows Installer XML after install

After installing current version in Visual Studio 2015 professional  (spanish language), there is not such Windows Installer XML template.

The template comes with the WiX Toolset (http://wixtoolset.org/), not with the Wax editor.


5.0 (@next): Error for Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export #4700

Error for Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export #4700
Occurs on import of ConnectedRouter. Fresh install of 5.0 using Yarn.
I note that the issue #4700 is closed, but people are still getting the problem.
Code to reproduce:
import React from 'react';
import PropTypes from 'prop-types';
import {Switch, Route} from 'react-router';
import { ConnectedRouter, routerReducer, routerMiddleware, push } from 'react-router-redux'

class App extends React.Component {
  render() {
    const {history, ...rest} = this.props;

      return (<ConnectedRouter> <div/></ConnectedRouter>

    );
  }
}

App.propTypes = {
    history: PropTypes.object.isRequired
};

export default App;

There is something wrong with your build tooling or your install. That error happens when you pass a non-component to React.createElement, which is most commonly the case when there is an error in imports or the system doing those imports under the hood.

Thanks Tim
Do you have any suggestions as to how I can debug this - we are using an ejected create-react-app (was based on react-15)?

Try cleaning your package manager cache and remove the node_modules directory.
if you are using yarn run yarn cache clean. Then install the packages normally, it should fix the issue.


[Event Request] Codeunit 99000832 Sales Line-Reserve - CallItemTracking

Hi Team,
In codeunit 99000832 we need possibility to replace and avoid using standard code. Having event in the end of procedure CallItemTracking, would be a perfect solution :

[IntegrationEvent(false, false)]
local procedure OnCallItemTrackingOnBeforeItemTrackingLinesRunModal(var ItemTrackingLines: Page "Item Tracking Lines"; SalesLine: Record "Sales Line")

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Added support for http user basic authentication

Added support for http user basic authentication

Can you explain the goals of this PR, why is not enough http.BasicAuth

close due to inactivity


Java scalar default to 'any' instead of 'Object'

Describe the bug

To Reproduce
Steps to reproduce the behavior:

My GraphQL schema:

# Put your schema here
scalar Date

schema {
  query: Query
}

type Query {
  me: User!
  user(id: ID!): User
  allUsers: [User]
  search(term: String!): [SearchResult!]!
  myChats: [Chat!]!
}

enum Role {
  USER,
  ADMIN,
}

interface Node {
  id: ID!
}

union SearchResult = User | Chat | ChatMessage

type User implements Node {
  id: ID!
  username: String!
  email: String!
  role: Role!
}

type Chat implements Node {
  id: ID!
  users: [User!]!
  messages: [ChatMessage!]!
}

type ChatMessage implements Node {
  id: ID!
  content: String!
  time: Date!
  user: User!
}

My GraphQL operations:

# Put your operations here

My codegen.yml config file:

# Put your YML here
generates:
  src/main/java/my/app/generated/Resolvers.java:
    plugins:
      - java-resolvers

Expected behavior

Java code with scalar is not defined in codegen.yml should show as of class Object
public interface ChatMessage {
    public DataFetcher<Object> id();
    public DataFetcher<String> content();
    public DataFetcher<Object> time();
    public DataFetcher<Object> user();
  }

But instead it shown as any
public interface ChatMessage {
    public DataFetcher<Object> id();
    public DataFetcher<String> content();
    public DataFetcher<any> time();
    public DataFetcher<Object> user();
  }

Environment:
use https://graphql-code-generator.com/

OS:
@graphql-codegen/java-resolvers/@graphql-codegen/java : 1.13.3
NodeJS:

Additional context

Fixed in v1.13.4


Revert "Revert "Log ban hits in DB. (

Without broken migration this time (thanks EFCore).

What's the status on getting this merged?


Store model type and number of classes as metadata in the '.pt' file

🚀 Feature Request
Manual work has to be done to get a custom model working when it could be completely automatized.
self.model = models.get(
    <type_of_yolo_model>,
    num_classes=<the-number-of-classes-in-your-model>,
    checkpoint_path =<path-to-your-weights.pt>
)

Proposed Solution (Optional)
You could save the model and its associated metadata like this:
torch.save({
    'state_dict': model.state_dict(),
    'metadata': {'model_type': 'yolo_nas_s', 'classes_count': 80}
}, '/pth/to/your/model.pt')

Then you could load it like this:
checkpoint = torch.load('/pth/to/your/model.pt')
metadata = checkpoint['metadata']

self.model = models.get(
    metadata['model_type'],
    num_classes=metadata['classes_count']>,
    checkpoint_path ='/pth/to/your/model.pt'
)

Avoiding any manual work on custom model loading. I get quite a lot of requests for this here

This is a great add-on and makes it much more user-friendly. Any ideas about what's planned for the next releases?

Any ideas about what's planned for the next releases?

I don't know but this should be high on their list IMO :rofl:

https://github.com/mikel-brostrom/yolo_tracking/blob/master/examples/detectors/yolonas.py


test: bridge call tests

Summary by CodeRabbit

New Features

Added a new testing function for bridging calls in the CrosschainTestSuite.
Introduced a method to retrieve multiple token pairs in the Erc20TestSuite.
Implemented a new test function in the IntegrationTest suite to ensure functionality of bridging calls to Fxcore.

Tests

Enhanced testing capabilities in various suites to include new functionalities and integrations.

mg


There is no function named "object" (v0.13.3)

Error: Call to unknown function
on .terraform/modules/coginto_route53/modules/records/main.tf line 15, in resource "aws_route53_record" "this":
15:   for_each = var.create && (var.zone_id != null || var.zone_name != null) ? local.recordsets : object({})
There is no function named "object".

resolved by #14

Fixed in #14.
v1.5.0 has been just released.


Result structure for REST GET /v2/schemas/keyspaces (and /v2/schemas/keyspaces/{keyspaceName}) different from Swagger model in "simple" topology case

Looks like results one gets for "Keyspace" entries (whether individual or List) does not conform to what Swagger API suggests. Individual keyspaces are expected to look like:
{
  "name": "{KEYSPACE_NAME}",
  "datacenters": [
    {
      "name": "{DC-NAME}",
      "replicas": 0
    }
  ]
}```

but in actuality for "simple" keyspaces (ones with topology other than networked) no `datacenters` is included at all:

{
"name": "{KEYSPACE_NAME}"
}

Similarly I think "NetworkTopologyStrategy" would produce different, but incompatible, output (looks like that'd give `"dc-name":replica-count` instead of Object with `name` and `replicas`).

The API is correct but the swagger needs updated. For SimpleStrategy we're returning just the name which doesn't match the Swagger and this is the current response fo NetworkTopologyStrategy which does match.
{
            "name": "testks",
            "datacenters": [
                {
                    "name": "dc1",
                    "replicas": 3
                }
            ]
        }

Thank you @dougwettlaufer for verifying this.

Due to conversion to Quarkus, OpenAPI, I think this as specific individual issue can be closed; we need to verify OpenAPI definitions as a whole.


Replace deprecated Matchers to ArgumentMatchers

What changes are proposed in this pull request?
org.mockito.Matchers is deprecated in org.mockito 3.4. This class will likely be removed in version 4.0 It's recommended to change it to org.mockito.ArgumentMatchers.
Why are the changes needed?
Replace deprecated org.mockito.Matchers to org.mockito.ArgumentMatchers
Does this PR introduce any user facing changes?
no

reference https://www.javadoc.io/doc/org.mockito/mockito-core/3.4.0/org/mockito/Matchers.html

alluxio-bot, mere this please


support for Qt VS Tools?

Is there any way to use Qt VS Tools with vcpkg-built qt binaries?
It seems that files built with qmake and vcpkg have different folder structures. Qt VS Tools cannot find qmake.exe and other required files when using vcpkg built ones.
My projects use Qt VS Tools intensively. Without it, I don't know how to use qt (moc, qt designer, etc) on windows. :(
I have successfully built and installed all qt 5.9.2 modules(x64-windows), on Windows 10 machine, using VS 2017 15.5.4, vcpkg 0.0.103.

Excuse me for bringing that up again, but this qt.conf doesn't work for me on Windows.
Can anyone share his setup of vcpkg/Qt5.11.1 or close versions?
VS Tools are just erroring over and over with '..\Vcpkg\installed\x64-windows\tools\qt5\bin\qmake.exe cannot be found'.
Any help would be appreciated.

I have found that the tools try to add a /bin/ to the folder you specify so you need to create a bin folder inside the {whatever}/tools/qt5 and copy all the files there. Then it will complain about not having a qmake.conf file inside such bin folder, and then it is trying to find a qmake.conf file in the {whatever}/mkspecs/win32-msvc. If you create the file there, it causes an error, object reference not set to an instance of an object. That's as far as I have been able to go so far.

In Vcpkg\installed\x64-windows\tools\qt5
I see qt_debug.conf  and qt_release.conf which are remind me the one posted above.
Assuming it worked for that author of that message, and that maybe someone made a fix for that and put those files in there, there's a question: how to make them work?
Or maybe it's worth to contact Qt VSTools guys and ask them if they can fix that.
Not sure options do we have here.

Or maybe it's worth to contact Qt VSTools guys and ask them if they can just support vcpkg build.
Not sure what options do we have here.

No. This issue should be reopened. Qt devs have decided to set it as non-issue on their end. We don't have much choice but to set directory structures right.

@JackBoosY Please add blocked by upstream to this issue. I played I bit with it and only Release build can currently be correctly setup using the QtVsTools.

I'm wondering if there has been any movement on this. Seems the qt vs tool people have made it so the tool will work fairly easily with vcpkg. Works fine in release but in debug I keep getting linker errors related QT5Networkd.lib and libcrypto.lib. Any thoughts?

Hello, from what I get as error, it seems that the qt vs tool assume the debug libraries are located in the same directory as the release ones. The solution would be to change the path for the debug libs

@lukadriel7 Could you provide some details about your workaround?

@lukadriel7 @evakili If you use the debug versions of the binaries for debug builds they automatically know where to find the debug binaries. QT appears to have resolved this.

@dahubley Where you tell VS tools to use debug binaries?

You can specify the binaries on this screen

@dahubley  I try this before, but the following error occurred: C:\Users\____\AppData\Local\QtMsBuild\qt_globals.targets(268,5): error MSB4181: The "QtRunWork" task returned false but did not log an error.

@evakili This is how I did:

Open your project property manager tab
Add a new property sheet for the Debug and Release configuration of your project
Open the property sheet and add a new macro containing the paths to the libraries (Debug libs for debug config, release libs for release config), save and exit
Go to your project properties(all configurations) -> linker -> input and add your newly created macro as well as other dependencies(user32.lib, kernel32.lib, ...)
uncheck inherit from parent or default project
save your config and enjoy compiling in debug mode

A actually think this issue is closed?

IMO the above workarounds are not real solutions, but some hacks. So this issue should be kept open till the integration works without handy hacks.
Also, the above workarounds have limitations in other scenarios, like debugging, using Qt tools in build events, and etc.

You may set QMake_QT_INSTALL_LIBS_ = " ....\vcpkg\installed\x86-windows\debug\lib" in your project file.
Has to go before 'qt.props' - so place it after

Close this PR until upstream fixes this.

After a lot of frustration I was able to get my environment to build a dynamic Debug configuration. To help others here is my situation:

I wanted both dynamic and static builds for both debug and release.
I also wanted to us Qt Visual Studio tools and vcpkg with Visual Studio 2022.
I needed Qt5 libraries built in vcpkg.

I found that dynamic-release and static-release built and worked fine but dynamic-debug and static-debug needed tweaks.
For static-debug the tweak was as listed above: Change "Qt Project Settings" -> "Paths" -> "Qt Binaries" from bin to Debug/bin.
For dynamic debug you do the same as static-debug but if you use the resource compiler or the user interface compiler (rcc.exe or uic.exe) you must also add to the system path the path where the vcpkg debug dlls are located which is \installed\x64-windows\debug\bin

This issue is closed?or just soveld in qt5 but not qt6?
I install PCL via vcpkg ,and as its dependency,qt6 was installed automatic,but I Failed to create a new qt project.The error info show as the first picture.
When I open the .vcxproj file I created,the error info show as the second picture.

When I open the .vcxproj file I created,the error info show as the second picture.

Then I  found there was no mkspecs folder in the Qt6 folder which is exist bin folder.

I would not say it is solved but the work-around works for me. There is something confusing about Qt5 + Qt Visual Studio tools and the overlap of using Qt5 by vcpkg versus Qt5 by separate library (not vcpkg). Here are the main problems I faced:

If you use vcpkg + manifest file then updates to vcpkg will rebuild any time there is a Qt5 update/patch/fix. This takes > 2 hours on my system to rebuild.
It is confusing/not easy with vcpkg to freeze a certain commit or version to hold Qt5 and stop rebuilding.
The combination of Qt5 and vcpkg is clumsy to use and you have to find the right tweaks of Qt5 to get dynamic/static along with release/debug configurations.

So now I build Qt5 using vcpkg command line and export to a folder for both static and dynamic. Then I configure VS .props files for the Qt5 version using Qt Visual Studio tools and point to the release/debug and static/dynamic export builds. But I still have to configure the .props files properly to point at the correct configuration (Release/Build)


Fix bug where subpaths weren't prepended to violation messages

Addresses SubpathValidator doesn't prepend violation messages with prefix

:+1:

@arothuis

:+1:


Updated OWNERS file for release-2.7

https://issues.redhat.com/browse/ACM-2363
Signed-off-by: dislbenn<EMAIL_ADDRESS>

@marcolan018 can you approve?


Remove Created field when adding a new work item using the dialogue box (#643)

Remove Created field when adding a new work item using the dialogue box #643

Current coverage is 53.88% (diff: 100%)

No coverage report found for master at 223acc3.

Powered by Codecov. Last update 223acc3...8819c1c

[test]


V3: firestack.ServerValue.TIMESTAMP

Is this implemented? I get undefined for firestack.ServerValue.TIMESTAMP using the current v3 branch from this repo.
Or has the syntax for this changed?
Thank you.

okay, if anyone else stumbles upon this… syntax is different from js SDK. firestack.ServerValue returns a promise.
return firestack
      .ServerValue
      .then((ServerValue) => {
          // you can use ServerValue.TIMESTAMP now
       })

@florianbepunkt I have this working locally the same as the web api, synchronously - as per your first example. Internally uses the built in FB ref .info/serverTimeOffset to keep it up to date.
I need to push up my local sometime, just not sure it's in a safe state yet as i've done quite a lot on it.  Working on it though!

@Salakar Great. I'll try it when you push it.

@florianbepunkt keeping open for v3 milestone

@florianbepunkt
Pushed this up, syntax is as follows now:
// very random example
const db = firestack.database();
const ref = db.ref('users/boatymcboatface');

ref.child('sessions').push(db.ServerValue.TIMESTAMP);

As you can see its now like the web sdk, no need to use a promise to get server val. Same with push.
Give it a whirl when you can :)

RIght away i get firestack.database is not a function when i try to declare it.


Vulnerability Assessment removed from SSMS 19.1

Hi,
Vulnerability assessment for SQL Server - SQL Server | Microsoft Learn was removed from SSMS 19.1. What would you like to do with this article? Add a note on the removal as people may still be using older versions now?

See Release notes for (SSMS) - SQL Server Management Studio (SSMS) | Microsoft Learn

PS. PowerShell cmdlets for managing SQL Vulnerability Assessments | Microsoft Learn seem to be still available with PowerShell Gallery | SqlServer 22.1.1

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

ID: d2d785ce-ee98-7cbf-a016-b88f6947dd8e
Version Independent ID: 2ecb2905-5c58-f768-3487-5af00d983102
Content: Vulnerability assessment for SQL Server - SQL Server
Content Source: docs/relational-databases/security/sql-vulnerability-assessment.md
Service: sql
Sub-service: security
GitHub Login: @DavidTrigano
Microsoft Alias: datrigan

@ingebeumer -- Inge, thank you again for your feedback.
@DavidTrigano -- David, please look into this issue.

@ingebeumer Discussing this with @erinstellato-ms and we'll close this when we've created an internal PR. Thanks!

Replaced by https://github.com/MicrosoftDocs/sql-docs-pr/pull/27935.


kong ingress controller go wrong

Summary
there are many error logs showing Unauthorized
Steps To Reproduce
I do not know how it comes
Additional Details & Logs

Kong version ($ kong version)
2.0.4
Kong debug-level startup logs ($ kong start --vv)
0724 09:25:54.363400       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190819141724-e14f31a72a77/tools/cache/reflector.go:98: Failed to list *v1.Service: Unauthorized
E0724 09:25:54.753406       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190819141724-e14f31a72a77/tools/cache/reflector.go:98: Failed to list *v1.KongIngress: Unauthorized
E0724 09:25:54.808929       1 leaderelection.go:324] error retrieving resource lock kong/kong-ingress-controller-leader-kong-kong: Unauthorized
E0724 09:25:55.106217       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190819141724-e14f31a72a77/tools/cache/reflector.go:98: Failed to list *v1.KongConsumer: Unauthorized
E0724 09:25:55.156996       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190819141724-e14f31a72a77/tools/cache/reflector.go:98: Failed to list *v1.KongPlugin: Unauthorized
E0724 09:25:55.158741       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190819141724-e14f31a72a77/tools/cache/reflector.go:98: Failed to list *v1beta1.Ingress: Unauthorized
E0724 09:25:55.365033       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190819141724-e14f31a72a77/tools/cache/reflector.go:98: Failed to list *v1.Secret: Unauthorized
Kong configuration (the output of a GET request to Kong's Admin port - see
https://docs.konghq.com/latest/admin-api/#retrieve-node-information)
Operating system
AKS

Can you share the entire logs of the controller?

How was this instance deployed? Had you made any changes/upgrades before this started occurring? We've observed some cases where, for example, a botched Helm upgrade will clobber some of the RBAC configuration. Removing everything other than the Kong CRDs (KongPlugin, KongConsumer, etc.) and re-installing will often work as quick fix (if using Helm, a delete/install will handle this for you, as it doesn't touch the CRDs).
The errors appear to indicate that your ServiceAccount and associated ClusterRole/ClusterRoleBinding (see https://github.com/Kong/kubernetes-ingress-controller/blob/master/deploy/single/all-in-one-postgres.yaml#L488-L587 and https://github.com/Kong/kubernetes-ingress-controller/blob/master/deploy/single/all-in-one-postgres.yaml#L781 for examples of what they should like) are no longer correctly bound.
What ServiceAccount is shown in your Kong+controller deployment, and can you see a ClusterRoleBinding for it? If so, does the ClusterRole in the binding still contains the expected permissions, e.g. for listing KongPlugins:
  resources:
  - kongplugins
  - kongclusterplugins
  - kongcredentials
  - kongconsumers
  - kongingresses
  - tcpingresses
  verbs:
  - get
  - list
  - watch

Closing this due to lack of activity. Please re-open if needed.


Preserve metadata on project references

In our build we extend the expand project references and would like to have metadata flow to this point.

:+1:

Thanks!


B03900 update manifests on edit

Coverage decreased (-1.6%) to 49.269% when pulling 35e95df53a422e9d105b9f83e190f9f88a8fcac6 on b03900-update-manifests-on-edit into 7739832eedd8e6afe5b90677cbca085b7768b294 on sprint-tcdl-staging.


concurrency: support ThreadContext lookup without CDI

In order for mp-concurrency to replace Reactive Contexts which is currently used by RESTEasy to provide context propagation, I suspect we will need to be able to obtain a TreadContext implementation without CDI, because RESTEasy-core does not depend on CDI.
Keeping the dependencies of mp-concurrency would allow it to be more broadly used. A CDI implementation could be provided as an optional dependency.
Copied from https://github.com/eclipse/microprofile-sandbox/issues/18

Added commit https://github.com/eclipse/microprofile-concurrency/pull/8/commits/e3ae382bb1a61a011a530992ddae6ff6ca3b8414 under pull #8 to experiment with solving this with a builder pattern, as mentioned in the sandbox issue.


Temperature and Data-oriented Orbits Class

Implemented temperature calculations and saved data into PKL file. Enabled the Orbits class to be super simple and fast since it only has to contain data and modulate Parameters instead of having to re-calculate temperature for arbitrary orbits.

Most of the actual changes were accidentally committed with git merge which is why the PR doesn't contain much information.


SMTPat generates unexpected output from Z3.

Hello,
I obtain the following error from Z3 when I use a SMTPat in one of my functions. I found another issue #1796 still open where they obtain the same error message but I don't know if the situation is exactly the same.
Sem.Term2.fst(37,0-38,120): (Error 276) Unexpected output from Z3: (error "line 148910 column 50: Wrong number of arguments (1) passed to function (declare-fun Sem.Term2.occurs.fuel_instrumented (Fuel Term Term) Term)")

I tried to make the example as minimal as possible:
module Sem.Term2

(****** Interface *******)

type funsymbol = int

type variable =
  {
    v_name: string
  }

type name = int

type term =
  | Var : v:variable -> term
  | Name : n:name -> term
  | Func : s:funsymbol -> l:list term -> term

(** Occurrence function **)

val occurs: variable -> term -> bool
val occurs_list: variable -> list term -> bool
let rec occurs x t= match t with
  | Var v -> v = x
  | Name _ -> false
  | Func _ args -> occurs_list x args
and occurs_list x l = match l with
  | [] -> false
  | t::q -> occurs x t || occurs_list x q

assume val spec_occurs_list_exists: x:variable -> l:list term -> Lemma
  (ensures (occurs_list x l <==> FStar.List.Tot.existsb (occurs x) l))
  [SMTPat (occurs_list x l)]

assume val get_vars_list: l:list term -> list variable -> list variable

val spec_get_vars_list: l:list term -> acc:list variable -> Lemma
  (forall (x:variable). (FStar.List.Tot.mem x (get_vars_list l acc) <==> (FStar.List.Tot.mem x acc \/ occurs_list x l)))

Note that if you change the type variable to type variable = int then it works. Don"t know if that can help you debug.
Best,
Vincent

Thanks @VincentCheval for the report!
The issue is same as #1575 which is still a problem. I have added the example there, with some explanation for the behavior. Closing this one.


[BUG] Bindings do not propagate correctly

Description
Apalache reports
Input error (see the manual): SubstRule: Variable x' is not assigned a value E@14:32:43.998

when run against Next (but not NextOk). Strange interplay between skolemization and short-circuiting is suspected.
Input specification
---------- MODULE test ----------

EXTENDS Integers

VARIABLE
  \* @type: Int;
  x

\* @type: () => Bool;
Oops ==
  IF FALSE
  THEN x' = 42
  ELSE x' = 99

Init == x = 1

Next == /\ \E v \in {1}: Oops
        /\ x' /= 0

NextOK == /\ Oops
          /\ x' /= 0
====================

The command line parameters used to run the tool
check --next=Next test.tla

and
check --next=NextOk test.tla

Not present anymore.


Validation message to check balance is not shown when user taps on Withdraw button even though user does not have enough balance.

Frequency: 100%
Repro on build version: Test Flight build v1.5.0 (29)
Repro on devices: iPhone XR (13.2.2) , iPhone 11 (14.0)
Testing Account:<PHONE_NUMBER> / Backup Key
Pre-condition:
launch the deeplink celo://wallet/pay?address=0xd13eee43e38f65fe3839bef6297928b43de38a02&amount=.1&token=CELO
Make sure user doesn't have enough CELO balance
Repro Steps:

Launch the application & login with a test account.
Copy the given link and paste to Safari browser.
Tap on Open button from "Open this page to valora" pop up
Make sure the Withdraw CELO screen is open.
Tap on the Withdraw button.

Current Behaviour:  No validation message and nothing happens when the user taps on withdraw button
Expected Behaviour: Users should see a validation message to check balance.
Impact: User will get the idea of what went wrong.
Attachment: IOS_NoValidationMessage.mp4	(Bug Observed at 12 sec)

Hi I have verified this issue using latest Test Flight Build V1.15.2(65) & observed that following.
Observations:
"Transaction Failed" Error Message displayed when user taps on Withdraw button.
Device: iPhone 6+(12.4.5)


Is custom emoji still supported

Cant figure out how to do it, but read in another post that said it was deprecated. Any help?

Custom emoji should still work but it was only really effective in one server if your not a nitro user so it’s like meh which is why I dropped support Sent from my iPhoneOn Dec 18, 2022, at 2:48 PM, VSeaV @.***> wrote:
Cant figure out how to do it, but read in another post that said it was deprecated. Any help?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

what line am i supposed to uncomment out?

Custom emoji should still work but it was only really effective in one server if your not a nitro user so it’s like meh which is why I dropped support Sent from my iPhoneOn Dec 18, 2022, at 2:48 PM, VSeaV @.> wrote: Cant figure out how to do it, but read in another post that said it was deprecated. Any help? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.>

what line am i supposed to uncomment?

where did u change in the code? i'd like to use a custom emoji too


Alien bullet creation logic isn't correct

Running through the demo game the logic that decides which column is going to fire is wrong.
Probably resetting the counter incorrectly

Now correct for the first demo game after reset. Later demo games are wrong.

Also when starting a game the bullets seem a bit fast.


Update to use UptimeRobot APIv2

This PR is to update this gem to support UptimeRobot APIv2. The APIv1 will be retired by 1 Oct 2017. This change includes a commit from @diegoadt that did most of the grunt work to get APIv2 up and running. I worked on updating the readme and a couple of variable name changes to make the change more understandable.
This is my first open source PR, so let me know if there is anything I should clean up or additional information I should provide.

I see that we missed updating the tests. Working on getting those passing now.

Tests are passing with minimal updates to reflect the changes in APIv2

:clap: :clap: :clap: thanks @avondohren, I reviewed and for me, your PR can be merged.

Uptime Robot did change the actual endpoints and parameter names, so apps that update will need to make those changes in their own code as well. Not sure how exactly to talk about that in the PR / in the README.md (maybe?).

But by the PR, the gem is in another minor version, right? So, if apps uses this gem they won't update to this specific version without manual intervention.

@winebarrel any suggestion?

Also tested it in production, works very well 👍

Sorry for the late reply.
@avondohren Thanks you for your PR! 👍
I will write the following note in the README.
If you want to use V1 API, please specify as follows.

> gem 'uptimerobot', '~> 0.1.6`

v0.2.0. has been released. 😃


From private repo

@sethmanheim : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Added Reset Button Underneath Run

This will now display a danger button showing "Reset" for anyone who wants to start over from the original server sent code.

This seems great but risky. I haven't pulled the code and tried it out; is there some pop-up that prevents the user from accidentally reseting their code and possibly causing major distress?
And just a heads up, quoting from the README: "As always, the staging branch is the appropriate place to branch off of to fix/add something!" So you should be forking and merging into staging, not master.

LumenTeun, I have 3 pull requests submitted that all function together. They're all related to making an automatic saving feature for bonfires; I don't want anyone loses their progress accidentally. There is a modal added to a different .jade file, but the correct file, that asks the user for confirmation before they reset their code. This button makes that modal pop up when clicked. My additions all follow the same styling as FCC's theme, and I have already discussed this with Quincy. He'll be taking my reset button and adding it to the button group with "help, pair, bug", not as a separate button below run.
-The only reason I didn't submit to the Staging branch was because I didn't have access to it for some reason.

Cool, hope to see it in production soon :)


use uint8 for distances to reduce memory use

resolves #9

adding an overflow check didn't affect the benchmarks, so I used uint8 for the distances - it throws if a distance is too large for the int type.

LGTM! Positively surprised that the overflow check is free!
We don't need to do an overflow check for every operation - each pairwise distance is summed using 32-bit ints, then there's a single check that it fits into a uint8 before casting it. If we had to check every integer addition it would almost certainly add some non-trivial overhead.


(PDB-3641) Standardize package GC command on gc_packages

Test PASSed

Waiting for CLA signature by @scottyw
@scottyw - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

CLA signed.

CLA signed by all contributors.


Lack of document for php extension of grpc

Hi,
I'm working with grpc, when I write client code with PHP, I found little document about grpc php extension.
I only found: http://www.grpc.io/grpc/php/namespaces/Grpc.html.
I'm really need some document about what is Grpc\ChannelCredentials, Grpc\\ServerCredentials, etc, and the methods, properties is has.
Because I am not familiar with c, so it is hard to read the source code under grpc/grpc/src/php/ext.
Really need some help from grpc php group!!!
Thanks

@stanley-cheung

@stanley-cheung any progress? thx

I'll start writing docs about the class like Channel and error codes which implement in c extension, or extract the comment from codes directly and update into grpc.io php API.

I didn't find a good tool to generate API document automatically from C extension. However, I noticed that someone has already make the document as grpc.php. With the same idea, we can do the same thing -- upload such file into packagist named as grpc-ext-for-api-doc, and then there are tools generating the document like apigen.

This has been updated.
grpc.io (main website) -> Docs -> Languages -> PHP -> API Reference, which will land you to https://grpc.github.io/grpc/php/namespace_grpc.
We have updated the generated documentation to include both the .c|.h classes from the PECL extension, as well as the .php classes from our composer package.


Keyboard shortcuts

Could I suggest to just blatantly copy the keyboard shortcuts from this other video platform?
Shift + ? to show a map of the keyboard shortcuts available:

In particular I miss the Shift + ./> to increase playback rate, but hey, why not just copy all the shortcuts that apply to LBRY... for consistency :)

Thanks for the issue and write up, can you please add it to https://github.com/lbryio/lbry-desktop/issues/1395


known issue: high severity vulnerabilities tracing to jpeg-js

first of all: thanks a lot for this plugin!
when installing it via npm, it gives a warning for 6 high severity vulnerabilities.
i'll probably use it anyway, but maybe it would be a good idea to fix them at some stage :)

Thanks for the note!  When I wrote this, there were no vulnerabilities, but it looks like jpeg-js (from png-to-ico) now has an infinite loop vulnerability.
This should not be unsafe for running this plugin, as the plugin is only active at build time, so it wouldn't cause a DoS on a website.
Regardless, I will look into fixing this shortly.  Thanks again for raising it.
$ npm install --save-dev eleventy-plugin-gen-favicons                                                                                [19:07:01]

added 118 packages, and audited 119 packages in 15s

9 packages are looking for funding
  run `npm fund` for details

6 high severity vulnerabilities

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.

$ npm audit                                                                                [19:07:28]
# npm audit report

jpeg-js  <0.4.4
Severity: high
Infinite loop in jpeg-js - https://github.com/advisories/GHSA-xvf7-4v9q-58w6
No fix available
node_modules/jpeg-js
  @jimp/jpeg  <=0.12.0 || >=0.16.1
  Depends on vulnerable versions of jpeg-js
  node_modules/@jimp/jpeg
    @jimp/types  <=0.11.1-canary.891.908.0 || >=0.16.1
    Depends on vulnerable versions of @jimp/jpeg
    node_modules/@jimp/types
      jimp  0.3.6-alpha.5 - 0.11.1-canary.891.908.0 || >=0.16.1
      Depends on vulnerable versions of @jimp/types
      node_modules/jimp
        png-to-ico  >=2.0.1
        Depends on vulnerable versions of jimp
        node_modules/png-to-ico
          eleventy-plugin-gen-favicons  *
          Depends on vulnerable versions of png-to-ico
          node_modules/eleventy-plugin-gen-favicons

6 high severity vulnerabilities

Some issues need review, and may require choosing
a different dependency.

There is a related issue in the png-to-ico project: https://github.com/steambap/png-to-ico/issues/19
and the related issue in jimp: https://github.com/oliver-moran/jimp/issues/1088
It sounds like the png-to-ico library has not been updated yet, so I can't simply bump the version.
However, as the author confirms there, only the png part of the library is used, so hopefully the library is not susceptible to this issue.
I will investigate whether I can manually force a different version of jpeg-js, or maybe at least swap out jimp-compact as described in the issue above.  Ideally, this would be fixed by the jimp project updating to a newer version of jpeg-js.  Not sure what the timeline is on that :( .

png-to-ico has been updated.  I will release a new version picking that up this week.
https://github.com/steambap/png-to-ico/pull/21

I have released version 1.1.1 which now shows no vulnerabilities (commit c4c9572f9d7d46427d631682faac5c7d251bcdf9 ).  Thanks again!


Various Path FFI improvements

See commit messages for details.
The first is effectively not changing anything other than adding comments in the code, using expect() instead of unwrap() with more details, and not lossy converting non-UTF8 paths from GLib to Rust on Windows but fail instead on invalid UTF-8 (otherwise we might access the wrong file! potential data loss, etc, and it's a bug in the C library if that ever happens)

Please carefully review, @EPashkin and @GuillaumeGomez :)
The second commit is also untested on Windows, if someone can give it some testing... you can get the annoying paths by using Path::canonicalize().

Seems good to me, just a bit sad for all the new panics... Otherwise, huge :+1: for the very complete comments, tt helps a lot for understanding!

The second commit is also untested on Windows, if someone can give it some testing... you can get the annoying paths by using Path::canonicalize().

Works fine on Windows

Question: should we add tests for these functions? I think it might be a good idea (even more since it's testable!).

Question: should we add tests for these functions? I think it might be a good idea (even more since it's testable!).

I was wondering about that, but is there a proper way of doing things with files during tests? How/where could we create random files and use them during the test?

There are great crates for that like tempdir (then you create your files in the temporary directory).

How would I add that as a dependency only for tests? dev-dependencies?

dev-dependencies indeed. :)

Ok, writing something

There

Thanks!
@EPashkin: We don't have a cargo test ran in travis nor appveyor?

Travis don't have :(
https://github.com/gtk-rs/glib/blob/master/appveyor.yml#L19

@sdroege Can you add a cargo test in .travis.yml please?

Sure, done

Thanks! Just waiting for CIs now and then I merge (unless you have something @EPashkin?).

@GuillaumeGomez All looks good and reasonable.
@sdroege Thanks

There many GLib-GObject-CRITICAL **: g_value_unset: assertion 'G_IS_VALUE (value)' failed in travis log

From another test though, but I also can't reproduce it here. All clean here
Can you reproduce?

I'm not: it on linux.
IMHO better just merge as warning seems not related to this PR and can be fixed later.

I don't get these on Linux either, that's where I'm usually testing.

Then I merge. Thanks!

Errors don't repeated on Ubuntu 16.04 LTS too.


[US] As a Dev, I want to pass the private key to the SDK instance through its constructor

As a Dev,
I want to pass the private key to the SDK instance through its constructor,
So its easier to initialize the SDK

At the moment, in a backend setup: the private key is passed via a .env file.
Goal: pass this private key via a param at the SDK's constructor level.

Fixed via #424


jugglinglab: 1.2.1 -> 1.4.1

Motivation for this change
Things done

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

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

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

Actually, it turns out that the new version uses OR-tools, which seems to be unavailable for Java in NixOS. I guess we need to package OR-tools first...

Result of nixpkgs-review pr 120757 at 60c83133 run on aarch64-linux 1

1 package failed to build:

jugglinglab

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

Actually, it turns out that the new version uses OR-tools, which seems to be unavailable for Java in NixOS. I guess we need to package OR-tools first...

I've noticed that on aswell. I also naively tried adding 'jkd' and 'maven' to the dependencies of or-tools and adding 'package_java'  as a build target. Sadly I does not seem to be that easy :(

nix-build or-tools with jdk and maven
[INFO] Scanning for projects...
[INFO]
[INFO] --------------< com.google.ortools:ortools-linux-x86-64 >---------------
[INFO] Building com.google.ortools:ortools-linux-x86-64 8.1.9999
[INFO] --------------------------------[ jar ]---------------------------------
Downloading from central: https://repo.maven.apache.org/maven2/org/apache/maven/plugins/maven-source-plugin/3.2.0/maven-source-plugin-3.2.0.pom
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  0.247 s
[INFO] Finished at: 2021-04-27T14:29:53Z
[INFO] ------------------------------------------------------------------------
[ERROR] Plugin org.apache.maven.plugins:maven-source-plugin:3.2.0 or one of its dependencies could not be resolved: Failed to read artifact descriptor for org.apache.maven.plugins:maven-source-plugin:jar:3.2.0: Could not transfer artifact org.apache.maven.plugins:maven-source-plugin:pom:3.2.0 from/to central (https://repo.maven.apache.org/maven2): Transfer failed for https://repo.maven.apache.org/maven2/org/apache/maven/plugins/maven-source-plugin/3.2.0/maven-source-plugin-3.2.0.pom: Unknown host repo.maven.apache.org: Name or service not known -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException
make: *** [makefiles/Makefile.java.mk:401: temp_java/ortools-linux-x86-64/timestamp] Error 1
make: *** Waiting for unfinished jobs....
builder for '/nix/store/bv0y4zdk9yjy4kwjzndan6wma85lh70a-or-tools-8.1.drv' failed with exit code 2
error: build of '/nix/store/bv0y4zdk9yjy4kwjzndan6wma85lh70a-or-tools-8.1.drv' failed

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/updating-jugglinglab-packaging-a-maven-repository/13726/1


PP-9148 Cache Cypress with version in key

In our workflow for running tests on PRs, we cache Cypress so that it doesn't need to be downloaded and installed for each test run, as this takes a number of minutes. This cache is persisted between action runs.
We were caching with the key Linux-cypress not taking into account versioning. This means that if we tried to update our Cypress version, we would probably end up with multiple Cypress versions in the same cache due to restoring the existing version from the cache and then installing the new version when we run npm install.
To ensure that we only cache and restore the required version, retrieve the Cypress version number from the package.json file and append this to the cache key.

Yep, looks good to me 👍 Didn't realise jq came for free in workflows...

Yeh must come in the linux image. I couldn't find anything that said this exactly anywhere but saw that Paas were using it in workflows without having to install it.


Allow any implementation of Pool to be passed to JedisLockProvider

I was not able to pass JedisSentinelPool as constructor argument to JedisLockProvider, which made it impossible to use it in a Redis slave-master cluster.

Thanks, will release it tonight or tomorrow.

Released as 0.13.0


⚠ Bofrost.at has degraded performance

In 3b0c5db, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2093 ms

Resolved: Bofrost.at performance has improved in fff0f1a.


Implement support for the bwc tests to run in parallel for multiple plugins

Implement support for the bwc tests to run in parallel. This will require implementing bwc tests in another plugin to test the parallelization, currently bwc tests are implemented for anomaly-detection.

[Triage] @VachaShah could you describe where this work would be done, is this per component or something that would need to be done in the CI workflow?


Colors by prefix?

conf.ttl states that there is an option to choose colors randomly or to assign color by class. Is it also possible to assign color by prefix?
For example, resources under http://dbpedia.org/resource/ should have a different color then resources under http://dbpedia.org/ontology/.
If it does not exist yet I propose it as a new feature.
This could also be implemented by allowing regular expressions for "colors by class", e.g.:
<http://dbpedia.org/ontology/.*>            conf:hasColorPair       "#c3a116-#ac8c13".

hi,
this use case never really happened to me ... but I can see the point
are you going to publish data using LodView? I mean, do you need this feature for some new publication?

I am going to publish the SNIK ontology, which models textbook knowledge about Hospital Information Management. Each textbook has a different prefix and we refer to them by their color. E.g. there is the "blue book" and the "orange book".

I try to implement it myself I don't understand the configuration code. In ConfigurationBean.populateBean(), the color list is parsed but I don't find the code that processes conf:byClass.

I try to implement this myself now by changing ConfigurationBean.java. If you are interested, I can send you a pull request when it is finished.

Made a pull request with the prefix color feature.


Database and gf_Annot todo fixes

Some fields in "category"/"source" columns are empty

A549-Foxa1-Haib.txt - all NaN in the score field


Setup GitHub Actions

Description
GitHub Actions is providing a feasible alternative to CircleCI for running tests on branches and PRs. It has been implemented in Vocabulary creativecommons/vocabulary#245 and along similar lines must also be implemented in this repository.
Problems solved
GitHub Actions has similar features, better integration and higher usage limits as compared to CircleCI. Switching would be the best option.

Hello @dhruvkb I would like to take this up

Hello @dhruvkb I went through the GitHub actions in vocabulary repository for reference
So, can you please help as if the same steps and jobs need to be present for the fonts repository
Thanks

Check the current CircleCI config in Fonts for the jobs that you need to setup here.

Okay @dhruvkb will follow accordingly


LandDetector Bug

Can't take off after the first flight in Altitude mode.
Because of this line:
https://github.com/PX4/Firmware/blob/master/src/modules/land_detector/LandDetector.cpp#L206
This line will never works until we leave landed state:
https://github.com/PX4/Firmware/blob/master/src/modules/land_detector/MulticopterLandDetector.cpp#L214
So,if we arm in altitude mode ,this line will be the only chance to takeoff:
https://github.com/PX4/Firmware/blob/master/src/modules/land_detector/MulticopterLandDetector.cpp#L239
It's looks good for the first time to takeoff.
But,when we landed and disarmed after the first flight.
_min_trust_start !=0 after we landed,because we returned from https://github.com/PX4/Firmware/blob/master/src/modules/land_detector/MulticopterLandDetector.cpp#L204  ,and we did not reset "_min_trust_start"  to zero.
In this case ,we can't takeoff after the first flight if we are in altitude mode with no position lock.
Maybe we could never takeoff if we are in altitude mode and have position locked?

should be fixed by https://github.com/PX4/Firmware/pull/7081
if not, please reopen


Create setonsmarts.txt

Scholl

@avetyansos I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the abovementioned issue. Thank you for the understanding!


Fix docker support and update github actions

Ensure that docker is fully functional with the new changes

Determined that docker, unlike podman, only assigns the host IP
after the instance has been created

Updated the GitHub Actions to test both podman and docker

Used a simple acceptance test for validation since the upstream beaker
tests are wired for Vagrant

Fail if the container is in an inaccessible state instead of trying to
SSH forever

@genebean Yes. The podman tests won't work without it (it's almost merged). The updates in core beaker aren't being worked to my knowledge since I just discovered the issue.