Update alter-database-transact-sql.md

ALTER DATABASE MODIFY NAME does not work with Azure Synapse Analytics Dedicated SQL Pool
Test Scenario - Rename SQL Pools.docx
s

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

Hi @WilliamDAssafMSFT ,
ALTER DATABASE MODIFY NAME works for Legacy standalone Dedicated SQL Pools (formerly SQL DW), but it does not work for Dedicated SQL Pools in Azure Synapse workspace.
Document attached.
Test Scenario - Rename SQL Pools.docx

@WilliamDAssafMSFT, if you approve the changes, please add a #sign-off comment to signal that the PR is ready for the review team to merge. Thanks.
#label:"aq-pr-triaged"

Hi @WilliamDAssafMSFT , ALTER DATABASE MODIFY NAME works for Legacy standalone Dedicated SQL Pools (formerly SQL DW), but it does not work for Dedicated SQL Pools in Azure Synapse workspace. Document attached. Test Scenario - Rename SQL Pools.docx

Based on that information the article is accurate. The comments on line 1044 are specific as to what scenarios ALTER DATABASE MODIFY NAME is supported. These changes as proposed would be too broad, as ALTER DATABASE MODIFY NAME is supported in some dedicated SQL pools.

@WilliamDAssafMSFT ,
This appears to be making a confusion here,
Azure Synapse SQL Dedicated Pool and Standalone Dedicated Pools (Formerly SQL DW) are  treated as two.
In this public document, in line 966, its mentioned as follows which can make a confusion -
"In Azure Synapse, ALTER DATABASE modifies the name, maximum size, or service objective for a database."
Since Azure Synapse Analytics Dedicated SQL Pools does not supported on "ALTER DATABASE MODIFY NAME" according to line 1046, the two lines (966 and 1046) having a conflict there.
The problem is not about the Standalone Dedicated Pools (Formerly SQL DW), but the Dedicated Pools in Azure Synapse Workspace.
I hope its clear.

@charithcaldera: I'm sorry - only the author of this article, @WilliamDAssafMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#hold-off
@charithcaldera you should see options beneath each of my suggestions to "Commit suggestion".  You'll know for sure they're in when they are reflected in the Files Changed tab.

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

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

Hi @WilliamDAssafMSFT ,
Done

Thanks @charithcaldera!!
#sign-off

#assign:@MicrosoftDocs/public-repo-pr-review-team

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team


Is shift+click or command+click supposed to do something different than regular click? (right now it doesn't)

The documentation says ⌘+click and ⇧+click is to select multiple items. But the same happens with regular clicking (without any keys).
I would say ⇧+click (shift+click) is supposed to select a whole range, for example if you click item number 3 it selects that, and then if you ⇧+click item number 8, it selects all items 3 through 8. And then next if you click item number 14 it will select that (in addition to 3~8 that were already selected). And then ⇧+click number 10, it should select items 10 through 14 (while also keeping 3 thru 8 selected).
Hopefully this description clarifies how it's supposed to work.
Furthermore if you want ⌘+click to have the same behavior as in macOS Finder, then regular clicking an item should only select that item, deselecting any previously selected items. Or if you click an item that was already selected, then deselect it (so it's actually toggle). Whereas ⌘+clicking an item should toggle that item while leaving any other selected items selected.
But the ⇧+click is more important imho.

@ganeshrvel can you please add this feature

I would say ⇧+click (shift+click) is supposed to select a whole range, for example if you click item number 3 it selects that, and then if you ⇧+click item number 8, it selects all items 3 through 8.

I tried to replicate the default key stroke behaviour as seen in the macOS finder. The macOS finder does not select range of files on ⇧+click
i will look into this.

I also cannot use keyboard shortcuts to select multiple files or multiple folders. I am able to replicate the issue on re-install of the program as well.

+1 for Shift+click for selecting range. I wanted to copy range of photos from "Camera" folder. Clicking 250 single files is not what I was expecting :) Reverting back to Android File Transfer.
Speaking about Finder. Shift+click works in List and Column view mode.

I tried to replicate the default key stroke behaviour as seen in the macOS finder. The macOS finder does not select range of files on ⇧+click

In finder it does. Click the first item, hold Shift and click a second, the items inbetween will be selected (range). Click first item, hold Cmd and click a second, the first and the second will be selected (multiple). Actually, I cannot us openMTP reasonably without - having 10,000 files, want to select around 800...


adds back max-rpc-payload-size option

This got removed in feature branches but is still in master. Need it for larger bundles.

lgtm after CI passes


How to run custom training loops?

Hi TorchMD-Net,
Following up from a twitter post of Guillem.
I'm interested in running 'custom' training loops. As I understand it, currently the torchmd-train routine will learn a mapping between atoms and per-atom energies, and maybe molecules <-> per-molecule energies (i.e. summing atom energies into a per-molecule level). I'd like to try other things that don't fit into a PyTorch Lightning format - to think of some examples:

using ranking loss functions over a batch, instead of a direct mapping to an energy
learning on relative energies rather than absolute. Perhaps for sets of tautomers, where each molecule has 1 to 10 tautomers, and the absolute energy of each is not known or important, but the free energy difference between them is known. Similarly one could imagine learning relative binding free energies in a congeneric series

Anyway, question is: is it suitable to just instantiate a model like TorchMD_ET, run a forward pass over it, and slot that into any custom training&loss loop?
Thanks for the tool and congrats on the nice publication
lewis

Hi! Thanks for the kind words. Guillem is our executive public relationships officer indeed.
The principal "model" abstraction in TorchMD-Net is just a regular torch.nn.Module, which you can train as you would any other module:
https://torchmd-net.readthedocs.io/en/latest/models.html#available-models
The model can be created with this function:
https://torchmd-net.readthedocs.io/en/latest/generated/torchmdnet.models.model.html#torchmdnet.models.model.create_model
Where the parameters are any of the ones supported by torchmd-train:
https://torchmd-net.readthedocs.io/en/latest/torchmd-train.html#configuration-file
Your feature request (custom training loops), while well supported, could be better documented. So let me go ahead and add a section for it. I will link it to you here when its done.
Having said that, you should now that:

You are free to create a dataset providing anything other than energies/forces. Although we do not currently provide a way to configure custom loss functions it is something in our roadmap.
We do support training on relative energies, perhaps you can make use of it for your purposes https://torchmd-net.readthedocs.io/en/latest/models.html#training-on-relative-energies

I added a section on custom training loops to the docs, check it out! https://torchmd-net.readthedocs.io/en/latest/models.html

Depending on your usecase you might also just use the representation model (i.e TensorNet, Equivariant Transformer...) which will return (N_atoms, hidden_features) scalar and/or vector features.
See for instance  https://torchmd-net.readthedocs.io/en/latest/models.html#equivariant-transformer
You can then manipulate these features however you want, either by adding an OutputModel as the TorchMD_Net class does or by incorporating the representation model as part of a bigger model.

Brilliant, this covers everything thoroughly! Thanks so much for the quick response and example documentation. Looking forward to getting stuck in!
All the best
Lewis


fix typo for prosody contour

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

#sign-off


Tensorflow Python3

i recently installed tensorflow in my linux machine with pip.
when i try to import tensorflow it show me this
Python 3.6.4 (default, Dec 19 2017, 14:09:48)
[GCC 7.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
/usr/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning: compiletime version 3.5 of module 'tensorflow.python.framework.fast_tensor_util' does not match runtime version 3.6
  return f(*args, **kwds)

i installed tensorflow cpu version with this command pip install tensorflow
this warning don't affect the script but its annoying
can anyone explain to me what is this
OS: Linux ubuntu
Python Version: 3.6.4

It's been discussed in #14182. You can find the solution on StackOverflow.


Add code format enforcement to the build process.

Codecov Report

Merging #149 into master will not change coverage.
The diff coverage is 68.18%.

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   62.63%   62.63%
=======================================
  Files         100      100
  Lines        9504     9504
=======================================
  Hits         5953     5953
  Misses       2911     2911
  Partials      640      640

Impacted Files
Coverage Δ

webhook/aws/sns_handler.go
78.31% <0%> (ø)
:arrow_up:

device/transactions.go
100% <100%> (ø)
:arrow_up:

secure/validator.go
86.17% <70%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


How to configure @MessageMapping handler for two queues from properties

I can configure statical message handler for two queues using annotation @MessageMapping like this MessageMapping({"Q1", "Q2"}), right?
I'd like to provide queues names from the properties file. Test and prod environments have different queues names.
Is it possible?

Hi @ulisses79 this should be possible indeed. You can use a property place holder for that.
Just write @MessageMapping({"${myQueue1}", "${myQueue2}"}), both should resolve to the actual queue names that are defined in your property file.
I am closing this ticket for now as this is not a bug/feature, please re-open in case you run into troubles.

Thank you, it helped a bit. What if I'd like to provide one property name which is a comma-separated string containing queues names?
myQueues=Q1,Q2....,Qn
something like this
@MessageMapping({"${myQueues}")


🛡 Denial of Service (DOS) via Front-Running Leads to Law Stone Initialization Failure

[!NOTE]
Severity: Critical
target: v5.0.0 - Commit: cde785fbd2dad71608d53f8524e0ef8c8f8178af
Ref: OKP4 CosmWasm Audit Report v1.0 - 02-05-2024 - BlockApex

Description
The instantiation process of the Law Stone contract is susceptible to a front-running vulnerability when interacting with the Objectarium contract for storing .pl files. This vulnerability stems from the public visibility of transaction data in the mempool, which allows attackers to intercept and replicate the initialization parameters. The core issue arises during the instantiate function's call to store_object in the Objectarium. If an attacker captures and submits the same data/program to the Objectarium ahead of the legitimate transaction, the Law Stone's initialization will fail, leading to repeated Denial of Service (DoS)
Impact
This vulnerability exposes the Law Stone contract to a persistent threat of initialization failure, which can be systematically exploited to prevent its deployment.
Recommendation
Ensure that the Objectarium is aware of the Law Stone's dependencies and enforces checks that the calling contract matches expected parameters.
Ref

contracts/okp4-law-stone/src/contract.rs
contracts/okp4-objectarium/src/contract.rs

Agree with the analysis, however this case has already been fixed in a version post-audit: https://github.com/axone-protocol/contracts/pull/528


Update Spark-connector-starrocks.md

Why I'm doing:

The back slash in creating the schema with StructType is redundant.
The PySpark code can't be used directly with the annotation.
A redundant right bracket in the code, this shows error in IDE.

What I'm doing:

Delete the redundant back slash
Wrap the PySpark code with bracket, so that the code can be used directly with annotation in the original place.

Fixes #issue
What type of PR is this:

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

Does this PR entail a change in behavior?

[] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

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

Bugfix cherry-pick branch check:

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

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

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


what-is-monero: improve wording

closes #2160

ci for ruby 2.7 fixed with #2167

The phrase privacy enhancing is usually used to indicate that a given privacy system is practically sufficient but is not "perfect" and may have flaws, so I would keep it.
Although, the next sentence is "ensure the anonymity of its users" so... ¯\(ツ)/¯


mutually_exclusive_ranges test breaks when using --store-failures on BigQuery

Describe the bug
When running a test suite which includes the mutually_exclusive_ranges test and using the --store-failures option, dbt fails when trying to insert rows into the target database, specifically BigQuery. Here are some example error messages that we've received:
Database Error in test dbt_utils_mutually_exclusive_ranges_qb_payment_filter_not_allowed__dbt_valid_from__qb_payment_id_invoice_payment_id__dbt_valid_to (models/staging/filters/qb/qb_payment_filter.yml)
  CREATE TABLE columns must be named, but column 1 has no name at [9:6]

Database Error in test dbt_utils_mutually_exclusive_ranges_qb_project_balances_filter_not_allowed__dbt_valid_from__qb_project_id_qb_account_id__dbt_valid_to (models/staging/filters/qb/qb_project_balances_filter.yml)
  CREATE TABLE columns must be named, but column 1 has no name at [9:6]

Steps to reproduce

Have a mutually_exclusive_ranges test
Run dbt test --store-failures

Expected results
The failures would be logged into a table in BigQuery and there would be no "Database Error" error messages
Actual results
We get the errors above
Screenshots and log output
Database Error in test dbt_utils_mutually_exclusive_ranges_qb_payment_filter_not_allowed__dbt_valid_from__qb_payment_id_invoice_payment_id__dbt_valid_to (models/staging/filters/qb/qb_payment_filter.yml)
  CREATE TABLE columns must be named, but column 1 has no name at [9:6]

Database Error in test dbt_utils_mutually_exclusive_ranges_qb_project_balances_filter_not_allowed__dbt_valid_from__qb_project_id_qb_account_id__dbt_valid_to (models/staging/filters/qb/qb_project_balances_filter.yml)
  CREATE TABLE columns must be named, but column 1 has no name at [9:6]

System information
packages:

 - package: dbt-labs/dbt_utils
   version: 0.7.3

 - package: tnightengale/dbt_meta_testing
   version: 0.3.3

Which database are you using dbt with?

[ ] postgres
[ ] redshift
[X] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
<installed version: 0.20.2
   latest version: 0.20.2

Up to date!

Plugins:
  - bigquery: 0.20.2
  - snowflake: 0.20.2
  - redshift: 0.20.2
  - postgres: 0.20.2

Additional context

Are you interested in contributing the fix?
I could try... :) I'd need to learn the build and testing toolchain but am interested in doing so

I think the issue is here, a snippet of the compiled SQL:
    select

        qb_project_id || '|' || qb_account_id,

        dbt_valid_from as lower_bound,
        dbt_valid_to as upper_bound,

        lead(dbt_valid_from) over (
            partition by qb_project_id || '|' || qb_account_id
            order by dbt_valid_from
        ) as next_lower_bound,

        row_number() over (
            partition by qb_project_id || '|' || qb_account_id
            order by dbt_valid_from desc
        ) = 1 as is_last_record

    from my_project.sandbox.qb_project_balances_filter

There needs to be an alias on that first line         qb_project_id || '|' || qb_account_id,
This might only happen in cases where the mutually exclusive test is using a composite/constructed "partition" from the data - this error probably wouldn't be triggered in a case where the "partition" field in the .yml configuration is a single column.
Here's the test config for us that causes the error:
      - dbt_utils.mutually_exclusive_ranges:
          lower_bound_column: dbt_valid_from
          upper_bound_column: dbt_valid_to
          partition_by: qb_project_id || '|' || qb_account_id
          gaps: not_allowed


RuntimeError: autograd_mapper input's constituents shouldn't require gradients

When I want to train model with nestedtensor, it raise this error, how can I solve it?

Hello @linyangsdu,
Do you have a small code snippet that reproduces this error?
Thanks,
Christian


cli: fix panic/segfault in dry-run compose

Reported by Don: happens because w/ --dry-run we don't return an error, but also don't return a project, but we'd still try to print the endpoints. Fixed by returning a specific dry-run error.

LGTM, might be beneficial to add some integration test for dry-runs int he future since it is very unlikely be tested by us during development.

I thought about it, but it would be some hacky cobra code, because this is fundamentally a command test. Maybe once we get rid of cobra.


Upgrade aws-sdk-go to v1.34.14+ to support overriding EC2 metadata service endpoint by setting AWS_EC2_METADATA_SERVICE_ENDPOINT

We need to use alternative EC2 metadata service endpoint for security reasons. aws-sdk-go introduced AWS_EC2_METADATA_SERVICE_ENDPOINT environment variable in v1.34.14 allowing to use a custom endpoint. Would be great to bump the version of aws-sdk-go to v1.34.14 or newer.
Last version of the daemon (3..2.0) was released over 15 months ago. Just wondering when can we expect a new version to be available?

Hi @ashevtsov-wawa ,
Thanks for opening this issue. We can bump the aws-sdk-go dep version to the latest however we might need to some changes in order to bump the dependency so I will mark this as an enhancement. Feel free to contribute on this issue. Moreover, we are also prioritizing daemon 3.3.0 release.


Package for CMSSW releases?

See #4, sort of. This will require restructuring the directory layout so that scram knows how to put things in the right places. Then "treemaker.py" (or perhaps just a "treemaker" script, or whatever it ends up getting called), winds up on the path and the library ends up on PYTHONPATH.

Done!


Newest commit breaks website

Newest commit makes the website redirect to a google 404 page

I dunno how to revert commits

commit 51c29c33539c3081b5572f4929e008b5dc1bc170 fixed this


Import package under personal account

I am wondering why https://github.com/g4s8/go-bundle is created and then import to degitx.  Why not create it as a public package directly in degitx? If other packages want to use it, they can import it from degitx.
What might be the benefits doing this way?

@jeffzhu503 this package is a general purpose package, it is not related to degitx project somehow, since it's responsible for arbitrary data binary marshalling.


linux-amd64.tar.gz v1.11.10 displays  v1.11.9

Describe the bug
A clear and concise description of what the bug is.
I am having some issues with the body of the email not being transferred to mattermost.
I've downloaded the "latest version"  here: https://github.com/Virtomize/mail2most/releases/tag/v1.11.10
./mail2most -version shows: v1.11.9
To Reproduce
Steps to reproduce the behavior:

Go to '...'
https://github.com/Virtomize/mail2most/releases/tag/v1.11.10
Click on '....'
linux-amd64.tar.gz
./mail2most -version
See error
v1.11.9
Expected behavior
should see v1.11.10

Release version
The version tag of the used version.
v1.11.9   should be v1.11.10
noticed the timestamp on the mail2most file is
3/6/2024 @ 6:10:21 AM
Architecture
linux-amd64.
Additional context
some section of the email body is not being displayed

Thank you @eyebank for reporting this. I updated the release with the correct binaries


Incorrect documentation in people.peopleconnections

I was looking for documentation on the people API after running the Quickstart and foud my way to: https://github.com/google/google-api-php-client-services/blob/master/src/Google/Service/People/Resource/PeopleConnections.php
That wasn't very helpful because the information there seems to be wrong. Page size actually goes up to 2000 not @opt_param int pageSize The number of connections to include in the response.     Valid values are between 1 and 500, inclusive. Defaults to 100.. And the personFields optional parameter doesn't seem to be documented at all, yet is used in the Quickstart

That is a great point, in fact, all the code in this repository is auto-generated and some times it shows. I use either API Explorer or the API's reference page as the source of truth when I am looking for info on a particular API.


Add Tycho Orbital Shipyards weaponry from SpaceDock

This is a semi-automated commit on behalf of sashan13

I reacted to the identifier on this seeing as TOS is a rather widespread acronym on its own and My first thought was not a KSP mod, how hellbent are we on the identifier should match installed content policy?

I'm happy to change it to TychoWeaponry, which was what I first thought to set it as.

#rebuild


Support Queue subscribers for Topic construct

Requested by Julien and Sione from Slack

Requested by Julien - https://twitter.com/jgx

Seth implemented this using native CDK code, and it looks quite complex https://gist.github.com/SethThomas/fa769197be405e3ec5c147933e5dd021
We should definitely have built-in support for this use case. I think we can really simplify this.

You can now add sst.Queue subscribers to a sst.Topic. An example here https://docs.serverless-stack.com/constructs/Topic#configuring-queue-subscribers


Filtering options of Ladybug do not look nice

See figure:

This is a limitation of the Echo2 framework. Work is in progress to replace Echo2 with Angular. This will among other things make it possible to do this the way you would expect it.

I understand. Will we leave this issue open until we are using Angular and have this issue fixed?


fix: improve reliability of screen view tracking

fixes: https://github.com/customerio/issues/issues/6616
QA test this PR from RH PR builds: https://github.com/customerio/RemoteHabits-iOS/pull/69
Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

[x] Assign members of your team to review the pull request.

[x] Wait for pull request status checks to complete. If there are problems, fix them until you see that all status checks are passing.

[x] Wait until the pull request has been reviewed and approved by a teammate

[ ] After pull request is approved, and you determine it's ready add the label "Ready to merge" to the pull request. A bot will squash and merge the pull request for you after the label is added.

[ ] Merge this branch into in-app as it will increase the reliability of it.

Here's the test run: https://app.qase.io/public/report/6866e04c03ad8d8be8fe529601a0adb68fe9b49c

PR unblocked as we have chosen to not prevent duplicate events. We will use customer feedback to see what use cases occur and decide on a solution at that time.

:tada: This PR is included in version 1.2.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Feature Request]: allow creating workbench envs from secrets

Feature description
Currently, data to be provided as env variables in notebooks is manually typed-in and then converted into a secret.

apiVersion: v1
kind: Secret
metadata:
  name: accesskey-secret
type: Opaque
data:
  username: blabla
  password: blablapassword

There is the other way around, though, that is used much more often:
There are secrets in the namespace, be it with database connection strings or certain api keys, that we want to have available inside a running notebook via env variable. A secret can have multiple keys, such as username and password above. In that case, the container with the spec below would have two env vars from secret database-secret inside it
username
password
The mechanism for that on a notebook container level is envFrom in the container section. Multiple secrets from the namespace can be read in.
spec:
    containers:
    - env:
      - name: ServiceBindingOperatorChangeTriggerEnvVar
        value: "31793"
        envFrom:
        - secretRef:
            name: database-secret
        - secretRef:
            name: accesskey-secret

I do not suggest odh dashboard create secrets (which is fine already), I am in addition suggesting using existing secrets, as often happens in corporate environments. The cluster admins are aware of which secrets they provide and create in a data science projects namespace. So having the values of those secrets available inside the container as env is ok.
@lucferbux Maybe your architects can give an opinion, also,
Describe alternatives you've considered
No response
Anything else?
No response

cc @kywalker-rh @andrewballantyne

UI:

I think that SimpleDrownDownSelect is already used at i.e. Accelerators, for a dropdown list of Accelerators:
https://www.sketch.com/s/d8550d2f-fae4-44fd-82df-9f400563d459/a/YZyg3q0

Hi @shalberd, @andrewballantyne
I have designed some mocks for this, which reflect the comments too, but before I go ahead and do anything else, I just want to make sure I'm heading in the right direction, and would appreciate your feedback on this to see if i'm missing anything (which I most definitely am!)
Here are the mocks in progress on Figma, where you can leave a comment, or you can comment here too :) Thanks!
https://www.figma.com/file/whP8hi2FeYWtvrGP5MYMag/Allow-creating-workbench-enviroments-from-secrets?type=design&node-id=0%3A1&mode=design&t=22fwkf4jMJnPMIsK-1
cc @kywalker-rh

@ralombar I requested access with my gmail account

@shalberd I didn't receive an email for this, could you message me with your email and i'll try to add you as a viewer, thanks!

Hi @shalberd until we get you access to the actual mocks, here are screens of the flow, beginning point is the user adding a variable under Environment variables.
I was in the process of adding the same flow for Config Maps too. Will take @andrewballantyne feedback on board too, as he has left comments on Figma. I'm interested to receive your feedback also, thanks!

Post-GUI, but still GUI-related in that "which section of the env vars GUI should be visible":
For a View-Type user, that environment variables - secret dropdown should be disabled, hidden.
Commented here as well https://github.com/opendatahub-io/odh-dashboard/issues/1706#issuecomment-1788805529

Thanks for your comment @shalberd, @andrewballantyne, @shalberd to clarify, are you saying that the user should not be able to select existing secrets and these capabilities are only allowed for the admin? if this is the case, I can come up with the admin view (being able to select existing secrets) and user view (having the option disabled).
With Config Maps, it will be the same capabilities for all types of users, so there will be the one design option for this.

@ralombar No, don't split hairs here. If they have permissions to show the resource, we should manage that. You don't need to worry about that with this work. Let us not work to create an artificial layer of permissions, it is what k8s says it is and no further or less than that. If it is less than that today, it's a lack of us writing code to support it, not by intent.
Let them select as many secrets and configmaps they want from their project. If we need something higher than that, shared secrets, configmaps, or something else, we can cross that bridge when we need to.

Moved to Jira: https://issues.redhat.com/browse/RHOAIUX-25

@ralombar is there a way to make the Jira issue viewable for / visible for non-Redhat employees with an access.redhat.com login? In general, I'm unsure what the new way of interacting via Jira is for the ODH community.

@shalberd Just want to let you know we're working on this. I believe the RHOAIENG board is already public, but we need to make the RHOAIUX board public. I've figured out how to do it, just need to jump through a couple hoops before we can, though. Should be there in a couple weeks.

@kywalker-rh very nice, thank you. Let me know if, post-UI, you want me to contribute anything regarding the logic in dashboard for this feature or if you've got people working on those aspects, too, once UI is set.

Hi @kywalker-rh in addition to UI https://issues.redhat.com/browse/RHOAIUX-25 (access still pending for non-RedHat employees logged in via access.redhat.com), is there a RHOAIENG Jira ticket for this as well?

I do not believe there is one currently -- nothing is linked. Are you still not able to access the UX tickets @shalberd ?

@andrewballantyne  correct, UX access not yet possible Jira

I'm itching to help with a f/ branch here cause getting env Variable values in via a secretRef is extremely important in my environments ... Github Tokens, S3 Access Credentials and much more ...


updated version

TL;DR
Bumped version number from 0.1.13 to 0.1.14 in Cargo.toml.
What changed?
Updated the version number in the Cargo.toml file from 0.1.13 to 0.1.14.
Why make this change?
This version bump likely indicates new features, bug fixes, or improvements have been made to the project. Updating the version number is crucial for proper versioning and helps users and developers track changes between releases.

#43  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mAmineChniti and the rest of your teammates on  Graphite

Merge activity

Oct 17, 5:14 AM EDT: A user started a stack merge that includes this pull request via Graphite.


Shopware 5.7.1 - whiteout

if installing it into the new shopware 5.7.1 version, its shoing only a whitepage after opening the menue tab.
no entrys in the console log.
PHP Error:
Fatal error: Uncaught Symfony\Component\DependencyInjection\Exception\InvalidArgumentException: The parameter "shopware.app.rootdir" must be defined. in

Has been fixed with https://github.com/FriendsOfShopware/FroshAdminer/commit/e9fd484ad92508da50bbb7a0a01ffbbfcf4559a6


ci: trigger bee-factory builds on release

This PR is part of an automatization initiative to stream-line updating Bee versions across our JS land. This action triggers automatically build of Bee Factory images that will be then propagated to our JS repos with PRs.

This change is

Codecov Report

Merging #2654 (621a338) into master (014b476) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2654      +/-   ##
==========================================
- Coverage   63.73%   63.52%   -0.21%
==========================================
  Files         234      236       +2
  Lines       26100    26473     +373
==========================================
+ Hits        16634    16817     +183
- Misses       7970     8138     +168
- Partials     1496     1518      +22

Impacted Files
Coverage Δ

pkg/api/api.go
64.30% <0.00%> (-10.81%)
:arrow_down:

pkg/debugapi/router.go
97.27% <0.00%> (-2.73%)
:arrow_down:

pkg/localstore/subscription_push.go
77.19% <0.00%> (-1.76%)
:arrow_down:

pkg/localstore/mode_put.go
58.90% <0.00%> (-1.15%)
:arrow_down:

pkg/localstore/mode_set.go
54.10% <0.00%> (-1.12%)
:arrow_down:

pkg/retrieval/retrieval.go
64.06% <0.00%> (-0.32%)
:arrow_down:

cmd/bee/cmd/start.go
2.35% <0.00%> (-0.03%)
:arrow_down:

pkg/debugapi/debugapi.go
100.00% <0.00%> (ø)

pkg/auth/handler.go
0.00% <0.00%> (ø)

pkg/auth/auth.go
59.80% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

This will also trigger bee-factory for prereleases (rc builds), is that the behavior we want?

From our end - yes.

Maybe it would be nice to include it in separate file (release_triggers.yaml or something like that) with event release to keep release file on its own and it looks that you already have tag available as GITHUB_REF

@vandot like this?
I originally thought it is not possible to do this as I need to have all the Docker images published by the time the Bee Factory workflow is triggered, but in the end, I think I found a way.


Adding make_moons, make_circles, make_blobs functions

Here you have some functions to generate basic datasets. I hope it will be usefull for inspecting clustering and classification methods.

Codecov Report

Merging #136 into master will decrease coverage by 5.25%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage    76.3%   71.05%   -5.26%
==========================================
  Files          21       22       +1
  Lines        1110     1178      +68
==========================================
- Hits          847      837      -10
- Misses        263      341      +78

Impacted Files
Coverage Δ

src/datasets_synthetic.jl
0% <0%> (ø)

src/datasets.jl
87.5% <0%> (-6.25%)
:arrow_down:

src/measures/loss_functions_interface.jl
94.73% <0%> (-5.27%)
:arrow_down:

src/equality.jl
50% <0%> (-5%)
:arrow_down:

src/measures/finite.jl
96.57% <0%> (-2.86%)
:arrow_down:

src/data.jl
95.45% <0%> (+0.85%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for suggesting this, it would be useful functions to have; a few points:

please do use the conventions n and p
try  to stick with the general coding style, for instance don't indent docstring
avoid using push! and particularly so with a [] which is an array of Any
use range instead  of your own linspace
add tests (e.g.: that points are  generated, in the right proportion, with the right dimensionality), you check code coverage to guide you

once that's done I'll do another pass, thanks!

PS: one note, while I would insist that the internal code uses n and p, feel free to add a keyword argument n_samples (which is what is used in sklearn for instance), e.g.:
function foo(n, p; n_samples=n, n_features=p)
    # ...
end

Thanks for suggesting this, it would be useful functions to have; a few points:

please do use the conventions n and p
try  to stick with the general coding style, for instance don't indent docstring
avoid using push! and particularly so with a [] which is an array of Any
use range instead  of your own linspace
add tests (e.g.: that points are  generated, in the right proportion, with the right dimensionality), you check code coverage to guide you
add  the functions to  export

once that's done I'll do another pass, thanks!

please do use the conventions n and p

Done

try to stick with the general coding style, for instance don't indent docstring

Done

avoid using push! and particularly so with a [] which is an array of Any

I feel the code is more clear this way and it will not affect performance, ( I do it one chunk at a time not one instance at a time).

use range instead of your own linspace

Done (LinRange instead of my own linspace)

add tests (e.g.: that points are generated, in the right proportion, with the right dimensionality), you check code coverage to guide you

Need to do this one

add the functions to export

Done

@tlienart Before pulling this, be sure to rebase onto dev first. Currently this PR is a request on to master.

Oh,  last thing, please use the code coverage helper indicated above to show how  you can add tests to fully  cover your code; since MLJBase is a base/abstract package, we really want to  strive towards 90%+ coverage. Thanks!

Hopefully all issues but the test part are done.
I am not sure hot to test for the error cases. I´ve added abstract types as input to avoid users to break the methods.
I still have the issue with n or p if introduced as negative but I don't want to write n::Unsigned because it is quite annoying. Now, as n::Int a user can write n=-1 and this should not be allowed.

Thanks for all your effort david, I'll finish the last minor things & merge it

Last try before 2020, @tlienart go and rest a little bit :D

superseded by #155


Issue26:Convert html to react -> Add Course

I have worked on conversion of Add Course Page to react

Closed by #93


Update virtualized list header to be floating

Adds expo-blur for universal backdrop blur of the header
Add universal useThemeSetting from tamagui/next to get current system theme
Adds h-100dvh css utility class and overrides tamagui default body height
Tamagui currently uses 100vh which is larger than the screen size on mobile.
Dynamic viewport-percentage units were created to address that problem.
https://caniuse.com/viewport-unit-variants
https://www.w3.org/TR/css-values-4/#viewport-variants

Latest push reworks things to drop the animation connected to onScroll and addresses the other comments. Thanks.

Thanks again for your work on this.
I've been inspired by your expo blur integration to create a global navigation layout for the stack.
For the most part, I've removed scroll listeners from the master branch for performance reasons. However, your work on the virtualized list layout has helped greatly with fitting the sample car data into the list.

Thanks again for your work on this.
I've been inspired by your expo blur integration to create a global navigation layout for the stack.
For the most part, I've removed scroll listeners from the master branch for performance reasons. However, your work on the virtualized list layout has helped greatly with fitting the sample car data into the list.

Thanks again for your work on this.
I've been inspired by your expo blur integration to create a global navigation layout for the stack.
For the most part, I've removed scroll listeners from the master branch for performance reasons. However, your work on the virtualized list layout has helped greatly with fitting the sample car data into the list.

Awesome. Thanks! Looks like I can close this. 😄


make the waves more detailed?

maybe you could adjust the waves to use FTT instead of a basic sine? this website details what FTT is way better than i could:
https://slembcke.github.io/WaterWaves
it would be really cool to see realistic waves implemented natively

bro thats minecraft...

WYM?

bro thats minecraft...

evan did implement water waving effects so whats stopping us/him from making it better?

bro thats minecraft...

evan did implement water waving effects so whats stopping us/him from making it better?

any idea what he meant by "bro that's minecraft"?
(i'm NIDNHU btw, same guy different account")

bro thats minecraft...

evan did implement water waving effects so whats stopping us/him from making it better?

any idea what he meant by "bro that's minecraft"? (i'm NIDNHU btw, same guy different account")

yeah this is me

bro thats minecraft...

evan did implement water waving effects so whats stopping us/him from making it better?

any idea what he meant by "bro that's minecraft"?

probably that this is too complex for a voxel game like minecraft (or its clones). i mean the waves already are a pretty cool addition and making them more detailed just for the flexes? /shrug


Work with operations to include new Extracts app in Cloudfront config

Figure out a path for it to live; currently thinking https://mapzen.com/data/metro-extracts-alt/.

Closed in https://github.com/mapzen/operations-engineering/issues/324.


Added SPLIT_V op to micro

Added split_v to micro, supports cases where the output tensors were pre-allocated in the model.
Ported tests from TFLite

@yair-ehrenwald  Can you please check @rthadur's comments and keep us posted. Thanks!

@yair-ehrenwald Can you please resolve conflicts? Thanks!

@gbaned Unless I'm missing something the only conflict is in
tensorflow/lite/micro/kernels/all_ops_resolver.cc
which has the addition of the split_v registration?

@petewarden Can you please assist on above comments from @yair-ehrenwald. Thanks!

@yair-ehrenwald To proceed, you should just sync your master branch to tensorflow HEAD, rebase your change on top of that, then push your change branch back.

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@yair-ehrenwald, Any update on this PR? Please. Thanks!

@gbaned  Should be done shortly, thanks :)

@yair-ehrenwald  Can you please sign CLA. Thanks!

@googlebot I fixed it

@gbaned  Should be done now, with no conflicts :)

@yair-ehrenwald Can you please sign CLA. Thanks!

@yair-ehrenwald Can you please sign CLA. Thanks!

@gbaned Hi, I signed it, should be ok.

@googlebot I fixed it.

@yair-ehrenwald  It still shows CLA is pending , can you please sign CLA. Thanks!

Could someone give some insight on replacing  and dynamic memory allocation in Micro implementation?

@yair-ehrenwald It still shows CLA is pending , can you please sign CLA. Thanks!

@yair-ehrenwald, Any update on this PR? Please. Thanks!

@yair-ehrenwald, Any update on this PR? Please. Thanks!

@gbaned  I think I understand why it says no CLA, I will fix the issue and open a new PR (and link to this one) shortly.

@yair-ehrenwald, Any update on this PR? Please. Thanks!

https://github.com/tensorflow/tensorflow/pull/42452


Revert fetch_by_ensembl_gene to release/108 version

Description
This change reverts method fetch_by_ensembl_gene to its release/108 version, and in doing that restores the ability to query the homology/id/:id endpoint by versioned gene stable ID.
Use case
Until Ensembl 108, queries of the homology/id REST endpoint would return a result for a versioned gene stable ID, even if the initial stable_id_lookup failed, because of a fallback query of Compara databases to verify that the gene could indeed be accessed by the given versioned gene stable ID. As part of the process of deprecating Compara method MemberAdaptor::fetch_by_stable_id, that fallback query was removed, which effectively removed support for querying homologies by versioned gene stable ID.
For example, these queries return equivalent results via the Ensembl 108 REST API:

https://rest.ensembl.org/homology/id/ENSG00000186092?content-type=application/json;compara=vertebrates;target_species=equus_caballus
https://rest.ensembl.org/homology/id/ENSG00000186092.7?content-type=application/json;compara=vertebrates;target_species=equus_caballus

However, the latter query currently fails on the Ensembl 109 test REST site.
With the extension of the deprecation period of MemberAdaptor::fetch_by_stable_id, it is possible to reinstate the fallback query and thereby restore the ability to query homologies by versioned gene stable ID.
Benefits
Users will be able to continue using the homology/id/:id endpoint to query homologies by versioned gene stable ID for at least one more Ensembl release. Development is in progress to replace this functionality in release 110.
Possible Drawbacks
This solution is necessarily temporary, since MemberAdaptor::fetch_by_stable_id will be removed from the Compara Perl API in Ensembl 110. But in the worst case, support for querying homologies by versioned gene stable ID would end in Ensembl 110 rather than release 109.
Testing
A unit test has been added to homology.t which queries the homology/id/:id endpoint with horse gene stable ID ENSECAG00000014890. This is not a versioned stable ID, but it can serve as a proxy for one, because it is present in the homology Compara unit-test database but not in the stable_ids test database used in stable_id_lookup.
All Compara-related unit tests passed locally.
Changelog
Are you changing the functionality of an endpoint? If so, please give a one line summary for the public facing changelog.
No change as such, only a partial reversion of the changes in PR #542 to restore functionality present in the Ensembl 108 REST API.

Coverage: 44.433% (-0.004%) from 44.437% when pulling 9849f7f852916402c22e738cb5f1860b8b1c9a1f on twalsh-ebi:release/109 into d46dd52e2c115281759a10dc31a8362f25358152 on Ensembl:release/109.

@twalsh-ebi - for the records these changes have been reverted from 'release/109', but they are still in 'main' and shall go into 'release/110' as soon as we branch.
This is the expected behaviour we agreed on. Thx

@twalsh-ebi - for the records these changes have been reverted from 'release/109', but they are still in 'main' and shall go into 'release/110' as soon as we branch. This is the expected behaviour we agreed on. Thx

Thanks @sgiorgetti, I understand. An update to the reverted method is in development for the 'main' branch.


Allow configuring according-padding as a percentage of screen size?

I have my accordion padding set to 400, which works to my liking on a 32" monitor. However, if I'm out and I'm using my Mac with only its built in display, this padding is too much. I find myself having to temporarily edit my config to adjust this setting until I'm back to having my big monitor. Is there a feature I'm missing?

Related to: #397
I don't mind to have percentages for accordion paddings. I won't be working on it. PRs are welcome


Can't create a tag when its name is already in the description of another

I wanted to create the tag #swedish but I was not able to because AS-1663 already contains "swedish" in its description.
As a workaround, I was able to create a tag #xxx and rename it to #swedish but this is unobvious and cumbersome.

Also, I cannot now remove the tag AS-1663 which I assigned to faun.se by mistake.

Happened again; I had to resort to trickery to create #japanese.

I'm not sure if we should call this a bug or a feature request. It's not what a user expects, but it's definitely the intent behind the current code.
The ability to have the non-matching text become a new entry option is something that's done with JavaScript added by MS. It's not a standard part of the bootstrap-select plugin. The code, as it's currently written, doesn't make any attempt to allow adding anything as a tag other than text which has no matches.


Screen reader detection

At least one tool of this kind detects if a screen reader is running, and automatically invokes certain options within its overlay without asking.
If it is known that someone is using a screen reader, it is known that they have a disability. That is personal data, and there are laws that prohibit the collection and use of personal data without informed consent.
I've reached out to friends who specialise in privacy for their thoughts.
This may be a topic we should include.

Agree


It worked on win10 with C&C general!

Thank you very much! It worked on win10(21H2 LTSC) with C&C general(mod of fivestar) and perform well! The game often make amd display driver offline and leave me blake screen with offical d3d8.dll before.But I still have some questions:

How can I test d8vk with 3dmark2001se on win10?  3dmark2001se can't work ,when I putting your d3d8.dll in it's directory.
Why dose the game load official d3d11.dll and d3d12.dll,while it's really load 'C:\Windows\SysWOW64\vulkan-1.dll'
The following log and capture may helpful:
info:  Game: game.dat
info:  DXVK: d8vk-v1.0
info:  Built-in extension providers:
info:    Win32 WSI
info:    OpenVR
info:    OpenXR
info:  OpenVR: could not open registry key, status 2
info:  OpenVR: Failed to locate module
info:  Vulkan: Found vkGetInstanceProcAddr in vulkan-1.dll @ 0x70b0b540
info:  Enabled instance extensions:
info:    VK_KHR_get_surface_capabilities2
info:    VK_KHR_surface
info:    VK_KHR_win32_surface
info:  D3D9: VK_FORMAT_D24_UNORM_S8_UINT -> VK_FORMAT_D32_SFLOAT_S8_UINT
info:  AMD Radeon 780M Graphics:
info:    Driver: 2.0.250
info:    Vulkan: 1.3.237
info:    Memory Heap[0]:
info:      Size: 3840 MiB
info:      Flags: 0x3
info:      Memory Type[0]: Property Flags = 0x1
info:      Memory Type[4]: Property Flags = 0xc1
info:    Memory Heap[1]:
info:      Size: 13922 MiB
info:      Flags: 0x0
info:      Memory Type[1]: Property Flags = 0x6
info:      Memory Type[3]: Property Flags = 0xe
info:      Memory Type[5]: Property Flags = 0xc6
info:      Memory Type[7]: Property Flags = 0xce
info:    Memory Heap[2]:
info:      Size: 256 MiB
info:      Flags: 0x3
info:      Memory Type[2]: Property Flags = 0x7
info:      Memory Type[6]: Property Flags = 0xc7
info:  Process set as DPI aware
info:  Device properties:
info:    Device name:     : AMD Radeon 780M Graphics
info:    Driver version   : 2.0.250
info:  Enabled device extensions:
info:    VK_AMD_memory_overallocation_behavior
info:    VK_EXT_attachment_feedback_loop_layout
info:    VK_EXT_conservative_rasterization
info:    VK_EXT_custom_border_color
info:    VK_EXT_depth_clip_enable
info:    VK_EXT_full_screen_exclusive
info:    VK_EXT_hdr_metadata
info:    VK_EXT_memory_priority
info:    VK_EXT_non_seamless_cube_map
info:    VK_EXT_robustness2
info:    VK_EXT_shader_stencil_export
info:    VK_EXT_transform_feedback
info:    VK_EXT_vertex_attribute_divisor
info:    VK_KHR_external_memory_win32
info:    VK_KHR_external_semaphore_win32
info:    VK_KHR_pipeline_library
info:    VK_KHR_swapchain
info:  Device features:
info:    robustBufferAccess                     : 1
info:    fullDrawIndexUint32                    : 1
info:    imageCubeArray                         : 1
info:    independentBlend                       : 1
info:    geometryShader                         : 1
info:    tessellationShader                     : 0
info:    sampleRateShading                      : 1
info:    dualSrcBlend                           : 0
info:    logicOp                                : 0
info:    multiDrawIndirect                      : 0
info:    drawIndirectFirstInstance              : 0
info:    depthClamp                             : 1
info:    depthBiasClamp                         : 1
info:    fillModeNonSolid                       : 1
info:    depthBounds                            : 1
info:    multiViewport                          : 1
info:    samplerAnisotropy                      : 1
info:    textureCompressionBC                   : 1
info:    occlusionQueryPrecise                  : 1
info:    pipelineStatisticsQuery                : 1
info:    vertexPipelineStoresAndAtomics         : 1
info:    fragmentStoresAndAtomics               : 0
info:    shaderImageGatherExtended              : 0
info:    shaderClipDistance                     : 1
info:    shaderCullDistance                     : 1
info:    shaderFloat64                          : 0
info:    shaderInt64                            : 0
info:    variableMultisampleRate                : 1
info:    shaderResourceResidency                : 0
info:    shaderResourceMinLod                   : 0
info:    sparseBinding                          : 0
info:    sparseResidencyBuffer                  : 0
info:    sparseResidencyImage2D                 : 0
info:    sparseResidencyImage3D                 : 0
info:    sparseResidency2Samples                : 0
info:    sparseResidency4Samples                : 0
info:    sparseResidency8Samples                : 0
info:    sparseResidency16Samples               : 0
info:    sparseResidencyAliased                 : 0
info:  Vulkan 1.1
info:    shaderDrawParameters                   : 0
info:  Vulkan 1.2
info:    samplerMirrorClampToEdge               : 1
info:    drawIndirectCount                      : 1
info:    samplerFilterMinmax                    : 0
info:    hostQueryReset                         : 1
info:    timelineSemaphore                      : 1
info:    bufferDeviceAddress                    : 0
info:    shaderOutputViewportIndex              : 1
info:    shaderOutputLayer                      : 1
info:    vulkanMemoryModel                      : 1
info:  Vulkan 1.3
info:    robustImageAccess                      : 0
info:    pipelineCreationCacheControl           : 1
info:    shaderDemoteToHelperInvocation         : 1
info:    shaderZeroInitializeWorkgroupMemory    : 0
info:    synchronization2                       : 1
info:    dynamicRendering                       : 1
info:  VK_AMD_shader_fragment_mask
info:    extension supported                    : 0
info:  VK_EXT_attachment_feedback_loop_layout
info:    attachmentFeedbackLoopLayout           : 1
info:  VK_EXT_conservative_rasterization
info:    extension supported                    : 1
info:  VK_EXT_custom_border_color
info:    customBorderColors                     : 1
info:    customBorderColorWithoutFormat         : 1
info:  VK_EXT_depth_clip_enable
info:    depthClipEnable                        : 1
info:  VK_EXT_extended_dynamic_state3
info:    extDynamicState3AlphaToCoverageEnable  : 0
info:    extDynamicState3DepthClipEnable        : 0
info:    extDynamicState3RasterizationSamples   : 0
info:    extDynamicState3SampleMask             : 0
info:  VK_EXT_fragment_shader_interlock
info:    fragmentShaderSampleInterlock          : 0
info:    fragmentShaderPixelInterlock           : 0
info:  VK_EXT_full_screen_exclusive
info:    extension supported                    : 1
info:  VK_EXT_graphics_pipeline_library
info:    graphicsPipelineLibrary                : 0
info:  VK_EXT_memory_budget
info:    extension supported                    : 1
info:  VK_EXT_memory_priority
info:    memoryPriority                         : 1
info:  VK_EXT_non_seamless_cube_map
info:    nonSeamlessCubeMap                     : 1
info:  VK_EXT_robustness2
info:    robustBufferAccess2                    : 1
info:    robustImageAccess2                     : 1
info:    nullDescriptor                         : 1
info:  VK_EXT_shader_module_identifier
info:    shaderModuleIdentifier                 : 0
info:  VK_EXT_shader_stencil_export
info:    extension supported                    : 1
info:  VK_EXT_swapchain_colorspace
info:    extension supported                    : 0
info:  VK_EXT_hdr_metadata
info:    extension supported                    : 1
info:  VK_EXT_transform_feedback
info:    transformFeedback                      : 0
info:    geometryStreams                        : 0
info:  VK_EXT_vertex_attribute_divisor
info:    vertexAttributeInstanceRateDivisor     : 1
info:    vertexAttributeInstanceRateZeroDivisor : 1
info:  VK_KHR_external_memory_win32
info:    extension supported                    : 1
info:  VK_KHR_external_semaphore_win32
info:    extension supported                    : 1
info:  VK_NVX_binary_import
info:    extension supported                    : 0
info:  VK_NVX_image_view_handle
info:    extension supported                    : 0
info:  Queue families:
info:    Graphics : 0
info:    Transfer : 2
info:    Sparse   : 0
info:  DXVK: Read 309 valid state cache entries
info:  DXVK: Graphics pipeline libraries not supported
info:  D3D9DeviceEx: Using extended constant set for software vertex processing.
info:  D3D9DeviceEx::ResetSwapChain:
info:    Requested Presentation Parameters
info:      - Width:              2560
info:      - Height:             1600
info:      - Format:             D3D9Format::A8R8G8B8
info:      - Auto Depth Stencil: true
info:                  ^ Format: D3D9Format::D24S8
info:      - Windowed:           false
info:  Presenter: Actual swap chain properties:
info:    Format:       VK_FORMAT_R8G8B8A8_UNORM
info:    Color space:  VK_COLOR_SPACE_SRGB_NONLINEAR_KHR
info:    Present mode: VK_PRESENT_MODE_IMMEDIATE_KHR
info:    Buffer size:  800x600
info:    Image count:  3
info:    Exclusive FS: 0
info:  Setting display mode: 2560x1600@0
info:  Setting display mode: 2560x1600@0
warn:  D3D9DeviceEx::SetRenderState: Unhandled render state 164
info:  DXVK: Using 16 compiler threads
info:  Presenter: Actual swap chain properties:
info:    Format:       VK_FORMAT_R8G8B8A8_UNORM
info:    Color space:  VK_COLOR_SPACE_SRGB_NONLINEAR_KHR
info:    Present mode: VK_PRESENT_MODE_FIFO_KHR
info:    Buffer size:  2560x1600
info:    Image count:  3
info:    Exclusive FS: 0
info:  D3D9DeviceEx::ResetSwapChain:
info:    Requested Presentation Parameters
info:      - Width:              2560
info:      - Height:             1600
info:      - Format:             D3D9Format::A8R8G8B8
info:      - Auto Depth Stencil: true
info:                  ^ Format: D3D9Format::D24S8
info:      - Windowed:           false
info:  Setting display mode: 2560x1600@0
info:  Setting display mode: 2560x1600@0

Thank you very much! It worked on win10(21H2 LTSC) with C&C general(mod of fivestar) and perform well! The game often make amd display driver offline and leave me blake screen with offical d3d8.dll before.

Glad to hear it helps. The unmodded base game and its addon are known to work fine with d8vk, save for some minor performance problems in the menus (see #151).

How can I test d8vk with 3dmark2001se on win10?  3dmark2001se can't work ,when I putting your d3d8.dll in it's directory.

I'm fully on Linux these days, so can't advise here, but we've had reports of 3DMark2001 SE working just fine on Windows, so I'm not entirely sure why you are running into issues.

Why dose the game load official d3d11.dll and d3d12.dll,while it's really load 'C:\Windows\SysWOW64\vulkan-1.dll'

Good question. You'll have to ask whoever wrote the mod, I guess. d8vk will only ever need vulkan-1.dll because it translates d3d8 to Vulkan for rendering and the base game certainly won't load either d3d11 or d3d12 dlls.

I 'm sorry to report the issues of 3dmark2001se.I try again, it works well.And much fast than official dx8!


UpsertTest::testUpsertWithMultipleSet fails

Related #6600
See also https://github.com/codeigniter4/CodeIgniter4/pull/6690#issuecomment-1279611246
There was 1 failure:

1) CodeIgniter\Database\Live\UpsertTest::testUpsertWithMultipleSet
Failed asserting that a row in the table [user] matches the attributes
{
    "email"<EMAIL_ADDRESS>    "name": "Jarvis",
    "country": "2022-10-14"
}

Found similar results: [
    {
        "id": "5",
        "name": "Jarvis",
        "email"<EMAIL_ADDRESS>        "country": "2022-10-15",
        "created_at": null,
        "updated_at": null,
        "deleted_at": null
    }
] and 2 others.

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/DatabaseTestTrait.php:280
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/Live/UpsertTest.php:641
phpvfscomposer:///home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/phpunit:97

https://github.com/codeigniter4/CodeIgniter4/actions/runs/3253661191/jobs/5341121215

https://stackoverflow.com/questions/381371/sqlite-current-timestamp-is-in-gmt-not-the-timezone-of-the-machine

I'll put a fix in in a minute

https://github.com/codeigniter4/CodeIgniter4/pull/6692

Closed by #6692


warning--filled 16px icon update

updating warning--filled.svg 16px to be non-reponsive

@conradennis this is this resource @1x:

this icon was updated today in #438


Report Writer support is missing

Thank you for the product.
I noticed that the Report Writer for COBOL feature is missing. I could not locate any reference to this in the limitations documentation: https://ibm.github.io/zopeneditor-about/Docs/knownissues.html#cobol
It would be nice to at least not make the editor think the the code represents an error. Maybe choosing some color to indicate that the feature is not supported.

Hi,
I am currently studying textmate grammars and VS Code language extension.
I saw that it was possible to define extensions to manage languages which make it possible to extend an already defined language: they are "injectors".
Report Writer defines an extension to the COBOL language and I think this should be handled with an injector type language extension, complementary to Z Open Editor, so that the specifics of Report Writer does not impact the support of the COBOL language by itself, (for those who do not use Report Writer).

I assume that this may get implemented in the future. Anyway, thank you.

Thanks for submitting this issue. We have to see if there is enough demand for this. We did get the request to be more agnostic for other pre-compiler content as well and that might be a first goal to achieve.

For my part, I did some experiments.
Z Open Editor has two COBOL source language processing mechanisms:

a COBOL language server
a COBOL TextMate grammar : ./cobol-textmate/cobol.json

I don't know the relationship between TextMate grammar and Language Server. The use of COBOL grammar always seems active, even in the presence of the COBOL language server.
By analyzing the COBOL TextMate grammar, I saw that it supported elements specific to the Report Writer syntax.
By default, the COBOL language server is active, but it does not fully manage the syntaxes related to Report Writer, and in particular does not know how to identify the role of Report Writer elements in the COBOL program as a whole. This is what generates all the problems in the editor:

If we deactivate the COBOL language server, (Ctrl + Shift + P / Pause the COBOL language server), then only the TextMate grammar is used. In this case, the Report Writer elements seems to be well recognized:

On the other hand, when the COBOL language server is deactivated, all the facilities that depend on it are lost:

search for the definition of a data element
search for uses of a data element
renaming of an identifier
...

What I can suggest to you is to work on your COBOL Report Writer sources by disabling the COBOL language server to avoid reporting problems.
While there is a command to suspend the COBOL language server, I have not found a command to re-enable it ... it seems that it is enough to close the source file and then reopen it to re-enable the language server COBOL at its level (this seems to be a mechanism attached to the source being edited).
The COBOL language server is developed in Java, and runs in an Eclipse context, code sharing with IDz / ZOD, confer ./language-server/plugins. Some of these plugins would have to be supplemented / rewritten to include Report Writer elements.
You might be better off getting Report Writer taken into account in Z Open Editor, a free of charge solution, by posting an RFE on the IBM RFE Community at the level of IBM Developer for z/OS, including Enterprise Edition, charged solution, which will benefit in waterfall to Z Open Editor.
For information, RFE regarding Report Writer:

87995 - Support Cobol Report Writer syntax in RDz, product: IBM Developer for z/OS, including Enterprise Edition, status: Declined
70792 - Support Cobol Report Writer syntax in RDz, product: IBM Developer for z/OS, including Enterprise Edition, status: Declined
146921 - Support for Cobol Report Writer statements, product: Application Discovery and Delivery Intelligence, status: Uncommitted Candidate

Thank you very much for the detailed answer.

Thank you.

From: Peter Haumer @.>
Sent: August 18, 2021 2:04 AM
To: IBM/zopeneditor-about @.>
Cc: egkareem @.>; Author @.>
Subject: Re: [IBM/zopeneditor-about] Report Writer support is missing (#145)
Thanks for submitting this issue. We have to see if there is enough demand for this. We did get the request to be more agnostic for other pre-compiler content as well and that might be a first goal to achieve.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IBM/zopeneditor-about/issues/145#issuecomment-900835862, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADXKN7CAK74WKFJHHYXEXTTT5NEN7ANCNFSM5CGVSAEA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.


Remove deprecated WinGet package for June 2024 GDK Update 1

Remove deprecated WinGet package for June 2024 GDK Update 1
Microsoft Reviewers: Open in CodeFlow

@microsoft/xbox-developer-experience-toolbox for review


Changing the netmask of a Virtual IP does not apply change

Consider the following configuration:

This results in the following IP addresses set on the LAN interface:
        inet6 fe80::1%vtnet0 prefixlen 64 scopeid 0x1
        inet6 fe80::53%vtnet0 prefixlen 64 scopeid 0x1

If you change the subnet mask, and then hit Apply, nothing on the system changes:

The system is still configured as:
        inet6 fe80::1%vtnet0 prefixlen 64 scopeid 0x1
        inet6 fe80::53%vtnet0 prefixlen 64 scopeid 0x1

Removing and adding the aliased addresses again works as expected.

for some reason ifconfig doesn't accept changes here, example output when changing 128 to 120 (or anything else):
# /sbin/ifconfig vtnet2 inet6 'ffff:ffff:fffff::4f31' prefixlen '120' alias vhid '3'
ifconfig: ioctl (SIOCAIFADDR): Invalid argument


[terra-alert][fusion theme] Custom Box Shadow Overrides Prop Defined Color

Bug Report
Description
In the fusion theme, the custom box-shadow color that is defined via the customColorClass prop is being overridden by the fusion class var --terra-alert-custom-box-shadow value.
In the default style sheet, the color was not included in the custom variant's box shadow to allow the color class to be inherited: https://github.com/cerner/terra-core/blob/master/packages/terra-alert/src/Alert.module.scss#L127-L129
In the fusion theme, a color is defined for the box shadow, should it is not inheriting the custom color value.
Steps to Reproduce

Go to https://engineering.cerner.com/terra-core/components/terra-alert/alert/alert
Look at the custom color class example
Change the theme to the orion-fusion-theme
See the custom color not being honored

Additional Context / Screenshots

Expected Behavior

Solution
Update the theme var to remove the color:
    --terra-alert-custom-background-color: #f4f4f4;
-  --terra-alert-custom-box-shadow: inset 2px 0 0 #9b9fa1;
+  --terra-alert-custom-box-shadow: inset 2px 0 0;

@neilpfeiffer @jeremyfuksa Could either of you confirm the suggested solution is correct?

It looks to me that the theme var could be removed completely.


vim: interface colorscheme

Scope
This PR exposes the base16 colourscheme by using a Vimscript dictionary
Motivation
This change allow users to acess the colours within their Vim/NeoVim config, which
is useful for example to set highlight groups for specific plugins
Changes Made

Defined a base16 colour scheme in a Vimscript dictionary
Exposed the color scheme via a global Vimscript variable

Example usage
g:colors['base00'] " Yields the hex code of base00

Or using lua
vim.g.colors.base00 -- Yields the hex code of base00

I just noticed that some indenting is wrong in my formatting commit, this is due to the fact that my own neovim config is currently broken and is not displaying tabs correctly. However, I did  check that this does not affect the functionality of the code in this pr whatsoever, which is why I'm not making another PR. This comment is just to point out this minor error on my part here

I renamed the colors to stylixColors, as from what I've looked at in the stylix codebase, it mostly seems to be camel case  which is why I've opted for this name out of the options you've kindly given me.
Again the indentation might be messed up, but I tried fixing it again with my new fix indendation commit, it should be the right amount of indentation but with the state of my current Neovim I cannot confirm if that is the case. If it is still messed up I'm assuming someone with a less broken neovim will fix it.

I renamed the colors to stylixColors, as from what I've looked at in the stylix codebase, it mostly seems to be camel case which is why I've opted for this name out of the options you've kindly given me.

I was actually referring to the VIM naming convention rather than the Nix (Stylix) convention.

Again the indentation might be messed up, but I tried fixing it again with my new fix indendation commit, it should be the right amount of indentation but with the state of my current Neovim I cannot confirm if that is the case. If it is still messed up I'm assuming someone with a less broken neovim will fix it.

You can also start nvim without loading your configuration:
-u vimrc    Use  vimrc  instead  of  the default ~/.config/nvim/init.vim.  If
            vimrc is NORC, do not load any initialization files (except
            plugins).  If vimrc is NONE, loading plugins is also skipped.  :help
            initialization

I renamed the colors to stylixColors, as from what I've looked at in the stylix codebase, it mostly seems to be camel case which is why I've opted for this name out of the options you've kindly given me.

I was actually referring to the VIM naming convention rather than the Nix (Stylix) convention.

Again the indentation might be messed up, but I tried fixing it again with my new fix indendation commit, it should be the right amount of indentation but with the state of my current Neovim I cannot confirm if that is the case. If it is still messed up I'm assuming someone with a less broken neovim will fix it.

I checked the vim variable naming convention and will convert it to snake case

You can also start nvim without loading your configuration:
-u vimrc    Use  vimrc  instead  of  the default ~/.config/nvim/init.vim.  If
            vimrc is NORC, do not load any initialization files (except
            plugins).  If vimrc is NONE, loading plugins is also skipped.  :help
            initialization

Alright I'll do that to check if the indentation is good or needs fixing
Thank you

I changed the name to snake case as per vim variable naming convention and double checked the indentation with vim by using vim -u NONE -U NONE -N -i NONE and it looks good

Assuming you are configuring Vim/NeoVim through Home Manager, could you not simply use ${baseXX} to substitute the colors directly into your config?

Assuming you are configuring Vim/NeoVim through Home Manager, could you not simply use ${baseXX} to substitute the colors directly into your config?

Initially, I had the same thought. Unlike NixVim users, I assume plenty of VIM and NeoVim users simply symlink their Vimscript or Lua configuration files with Nix. In that case, it might be nice to have these color variables injected. At least, that is my speculation.

Assuming you are configuring Vim/NeoVim through Home Manager, could you not simply use ${baseXX} to substitute the colors directly into your config?

I for example use mostly Lua in my NeoVim config which means I'm unable to directly inject the variables directly from stylix. Even if that was not the case I still believe it is a good idea to be able to access these colours directly in Vimscript or Lua. This would especially be needed for custom vim plusings which don't have access to the stylix variables as plugins are usually purely just a collection of Lua or Vimscript files without nix


docs: update import asset and import label tutorials

split the original "importing assets and labels" tutorials into two separate tutorials
also added some images

waiting for @xczuba review


Fix issue converting map to sorted map

Type of change

Bugfix

Description

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

is this a bug fix to released functionality.  If so, it should at least a have a short description explaining how the defect impacts this user?

is this a bug fix to released functionality. If so, it should at least a have a short description explaining how the defect impacts this user?

No, it shouldn't have any impact on the user. I only noticed the issue, when copying the pattern for another functionality. And there it broke the logic. I still is wrong and fixing it seems the right thing to do.


Py3ews - Activated the legacy section in the openssl.cnf

Status
Ready (Reason for hold)
Related Content Pull Request
Related PR: link to the PR at demisto/content
Related Issues
Two bugs:
Related: XSUP-20359,
Related: XSUP-20247.
An old bug that I think is also related:
XSUP-19122.
Description
In the above mention bugs, EWS Mail Sender integration failed after upgrading the docker image to: demisto/py3ews:<IP_ADDRESS>044.
The error in the logs is:
  File "/usr/local/lib/python3.10/hashlib.py", line 123, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type md4

It appears that some hashing algorithms with MD4 among them were disabled as obsoleted, which caused that error.
To solve that we activated the legacy section in the openssl.cnf file (re-enabled md4 to hashlib) - based on the solution described here: https://github.com/ecederstrand/exchangelib/issues/608.

@JudahSchwartz - I'm not sure why the verify.py failed - by testing locally from inside the docker it passes, and also I used the created dev/demisto image in the integration itself and it seems that it solved the bug.
Any idea?

@ShacharKidor you modified the file: docker/crypto/verify.py (in the crypto dir). You want to modify: docker/py3ews/verify.py.


Notifications in Voctogui

The GUI should be able to show some kind of notifications (like the stream pause/blank notifications). Those could be triggered via IRC (if the name of the room is mentioned) or via a http/rest API.
In this way the Voctomix operator can be alerted to look at the chat or do certain actions.
An advancement of this feature could be the remote control of Voctomix via IRC e.g. set the stream blanker via an specific command, in this case access to the IRC-Channel should be restricted.

I like the idea. We could also think of adding a hole chat and maybe intercom feature top the gui. This would be more screen space efficient than putting the chat below.
Not sure if this fits here but i would also like to add the intercom (either mumble or SIP based) to the GUI.

This is a duplicate of #93.
Implementing a custom Chat would be another Story. @derpeter please open one some explanations about the benefits above the current solution.


Simplify TODO PAY-7

Remove the return statement and commented code. The students paste all of this in from the codelab instructions.

Thanks for the PR!


Update DG

TODO:

[x] Target user profile, value proposition, and user stories
[x] Use cases
[ ] Non-functional requirements
[x] Glossary

Link to the task.

@Zhou-Jiahao-1998 The NFR section needs to be updated or at least modified. Reopening this issue!


Context menu with Current build status

As a programmers i would like to see the build status of all jobs when i click the right mouse button at the system tray icon because the main editor should not hide by another window.
#enhancement

As a ~family-busy~ maintainer of the project, kindly provide a PR so i can further assist you with it. Cheers!

Sorry, i dont have knowlege of c# development. But i think you could use the property fullDisplayName of jenkins api response in your file JenkinsService at line 122 in
https://github.com/zionyx/jenkins-tray/blob/master/JenkinsTray/BusinessComponents/JenkinsService.cs

Sorry, I don't have knowledge of cs and don't have an cs develop environment.
But I have used the time and started an project in Java. Currently I missing my time to write the docu but the code is working fine for me at now.
https://github.com/Huluvu424242/jenkinsmonitor


NETOBSERV-1473: migrating netobserv-agent to TCx for new kernels

Description
Migrate ebpf agent from TC to TCX "TC eXpress" if the kernel support it, this PR fix #230.

how to check TCX is enabled in ur kernel

zgrep CONFIG_NET_XGRESS /boot/config-$(uname -r)
CONFIG_NET_XGRESS=y

how to check if TCX hooks are attached to all interfaces

sudo bpftool net

xdp:

tc:
wlp0s20f3(2) tcx/ingress tcx_ingress_flow_parse prog_id 3923 link_id 214
wlp0s20f3(2) tcx/egress tcx_egress_flow_parse prog_id 3922 link_id 213
br-68b237cee3d3(3) tcx/ingress tcx_ingress_flow_parse prog_id 3923 link_id 216
br-68b237cee3d3(3) tcx/egress tcx_egress_flow_parse prog_id 3922 link_id 215
docker0(4) tcx/ingress tcx_ingress_flow_parse prog_id 3923 link_id 218
docker0(4) tcx/egress tcx_egress_flow_parse prog_id 3922 link_id 217
enp9s0u2u1u2(1736) tcx/ingress tcx_ingress_flow_parse prog_id 3923 link_id 220
enp9s0u2u1u2(1736) tcx/egress tcx_egress_flow_parse prog_id 3922 link_id 219

flow_dissector:

netfilter:

testing

[X] using 6.6.2-201.fc39.x86_64 run standalone agent code as well as e2e test and both passed with the new TCX. Waiting for OCP build with rhel9.4 to be able to resume the testing on OCP cluster

[ ] pkt drop

[ ] DNS

[ ] RTT

[ ] SRIOV secondary interface

[ ] PCA

[ ] performance and scale test

[X] on older kernel we will get this warning and fall back to the legacy TC hooks

time="2024-02-05T12:56:55Z" level=info msg="interface detected. trying to attach TCX hook" component=agent.Flows interface="{1b684d9681b6cb8 106 NS(none)}"
time="2024-02-05T12:56:55Z" level=warning msg="can't attach to TCx hook flow ebpfFetcher. fall back to use legacy TC hook" component=agent.Flows error="failed to attach TCX egress: tcx not supported (requires >= v6.6)" interface="{1b684d9681b6cb8 106 NS(none)}"

Note: GH use ubuntu-latest which is currently 6.2.0 so doesn't have TCX support yet
Dependencies
OCP with rhel9.4 kernel
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[ ] Will this change affect NetObserv / Network Observability operator? If not, you can ignore the rest of this checklist.
[X] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[X] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[X] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

Codecov Report
Attention: Patch coverage is 1.88679% with 156 lines in your changes are missing coverage. Please review.

Project coverage is 32.77%.  Consider uploading reports for the commit 7f4c0af to get more accurate results

Files
Patch %
Lines

pkg/ebpf/tracer.go
0.00%
133 Missing :warning:

pkg/agent/packets_agent.go
0.00%
8 Missing :warning:

pkg/ebpf/bpf_x86_bpfel.go
0.00%
8 Missing :warning:

pkg/agent/agent.go
12.50%
6 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #262      +/-   ##
==========================================
- Coverage   33.84%   32.77%   -1.07%
==========================================
  Files          47       47
  Lines        3865     3988     +123
==========================================
- Hits         1308     1307       -1
- Misses       2469     2593     +124
  Partials       88       88

Flag
Coverage Δ

unittests
32.77% <1.88%> (-1.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

verified with https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=59954477

/ok-to-test

Code looks good; any tips to test this on a cluster ?

yes detailed instructions on how to test this on OCP is in the comment section for
https://issues.redhat.com/browse/NETOBSERV-1473

/ok-to-test

/ok-to-test

did multiple rebase to keep with latest code

/ok-to-test

discussed with @Amoghrd and he is fine running QE verification post merge


Near Spawner penalty not working correctly

*Detailed description of the issue:
On our server players can get spawners and spawneggs from lootboxes, so we set permission jobs.nearspawner.-0.98 to default group. Now players get 1/50 of base income for killing mob from spawner, thats correct. Problem starts, when players are using boost items, or at night, when we have scheduled night bonus +100% income. It seems like every bonus/penalty add up to final bonus. So at night we have -98% near spawner and +100% night bonus, that equals +2%, so player get more than base income. But we want to have 98% of boosted income (200% of base income), thats 1/25 of base income.
I think equation for bonuses is now:
income = base income x sum of every bonuses
But logically it should be:
income = base income x sum(scheduled bonuses, boost items, dynamic payment bonus, permission bonuses,...) x near spawner penalty
Because this feature is for getting certain part of income for killing mob spawned naturaly, when you are near spawner, not just to decrease it "randomly" depending on other bonuses.
Jobs version:
<IP_ADDRESS>
Server Type (Spigot/Paper/etc):
Purpur
Server Version (using /ver):
git-Purpur-1610 (MC: 1.18.2)

I can add option to pick one of two behaviors here. Will default to old one. Look for multiply-boosted-extra-values in general config file with next update
b69a4f1397a79b7e2ddfaea2cafc434939106b72


#M19. Simplify and speed-up rendering scores

Description

Use custom SQL query to pre-aggregate scores instead of looping through all scores
Use pandas to speed up calculations

Pull request type

[x] #M01 (Miscellaneous User Story #01)
[ ] #P02 (Participant User Story #02)
[ ] #S03 (Staff User Story #03)
[ ] #A04 (Admin User Story #04)
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Related Issue
#176
Configuration instructions

Update requirements using pip install -r requirements.txt

Testing

All tests passing (apart from broken tests for the custom Slack provider which have not been working from the start)

Closed accidentially


Publish to Cucumber Pro

This will automatically push our docs to Cucumber Pro after a succesful build, meaning they're always up to date.
Based on https://docs.travis-ci.com/user/deployment/custom/ and
https://docs.travis-ci.com/user/encrypting-files/
Let's see if this works...

@mattwynne Perhaps this is exactly what Build Stages could be good for?

@mattwynne Also: COOL CHANGE, you!

@olleolleolle I was just thinking to myself "this is going to run for all those different Ruby versions in the matrix, that's stupid". Would build stages fix that then?

@mattwynne The example they put in their thing was TEST_SUITE=none (being another ENV variable which wouldn't add to "matrix width" (if that's a word)).

Once this build has run and I've got some feedback I'll have a play with it.

Interesting. @mattwynne When this works, I'm interested to use this for aruba as well.

Cucumber-JVM has for some time now used Build Stages to publish snapshot releases; .travis.yml

Won't work now we're on Circle!


Files for deploying to GOVUK PaaS

We've added 2 files that make it easier to deploy a prototype to the GOVUK PaaS.
.cfignore lists all the files and dirs that don't need to be copied across to the PaaS when deploying. It is a replica of the .gitignore file. This makes deployments quicker for users.
The manifest.yml file contains some config items for a prototype. In particular the name which ends up making up part of the URL. Using a manifest in this way makes it simpler for users to deploy prototypes to the intended location. It reduces the complexity of the cf push command and reduces the opportunity to accidentally push a prototype to an incorrect location. Two things noticed during testing.

ideally, would the cloud foundry app name be in config.json, or use the name already there?

Cloud Foundry looks for a manifest.yml file in the home dir of you project when you issue the cf push command. It uses the attributes in there to configure the app it creates on the PaaS.
The name attribute is used to make the route. So for example if name was set to govuk-proto-kit it would create an app on the PaaS by that name and the URL would be https://govuk-proto-kit.cloudapps.digital
So to answer your question I'm not sure Cloud foundry can use any other file for the name attribute. And my understanding is that the name attribute it uses in manifest.yml has a slightly different purpose from the serviceName  attribute in config.json. One is used to make the route and is in the form of word-word. The other is the name of the service used across the prototype.

@colmjude Is this the point that the user picks the url, or have they done that prior to this step?
Either way - can it be more obvious that this is the url? Can the .yml include comments?

@colmjude how does it work if you have multiple apps?

User picks the url they'd like to use at this point.
Just checked with the team and the manifest.yml can have comments in it.
You need to have a manifest.yml per prototype/app.
If you want to push multiple versions of a prototype, such as for versioning, a user can override the name in the cf push command.
For example name in mainfest.yml is set to govuk-proto-kit deploys using cf push would write over that. However if you'd like to keep an old version running, maybe as an archived version after a round of testing, a user could deploy with something like cf push govuk-proto-kit-v2. This would make an app at https://govuk-proto-kit-v2.cloudapps.digital

For reference, this is the guidance we give users in the heroku guide.
I feel like it would be good to have a comment or something that explains this will be the url, in the form name-of-the-thing.cloudaps.digital - and that it needs to be unique.

Sorry for the delay on this!
Going to close this as it's out of date with latest code (our fault).
Will talk to the PaaS team - it may be that we include this kind of code, or that they include it in their guidance.


[WFCORE-1620 2.x] Update slf4j to 1.7.22

Full integration - Windows Build 75 outcome was FAILURE using a merge of e8e49dfb808c066e850f7dfe80db4061a9954f64
Summary: Tests failed: 1 (1 new), passed: 3371, ignored: 194 Build time: 01:15:13
Failed tests
org.jboss.as.test.integration.domain.mixed.eap620.MixedDomainDeployment620TestCase: java.lang.RuntimeException: java.lang.RuntimeException: java.lang.RuntimeException: Could not start container
	at org.jboss.as.test.integration.domain.management.util.DomainLifecycleUtil.awaitHostController(DomainLifecycleUtil.java:466)
	at org.jboss.as.test.integration.domain.management.util.DomainLifecycleUtil.start(DomainLifecycleUtil.java:256)
	at org.jboss.as.test.integration.domain.mixed.MixedDomainTestSupport.startAndAdjust(MixedDomainTestSupport.java:165)
	at org.jboss.as.test.integration.domain.mixed.MixedDomainTestSupport.start(MixedDomainTestSupport.java:91)
	at org.jboss.as.test.integration.domain.mixed.MixedDomainTestSuite.getSupport(MixedDomainTestSuite.java:96)
	at org.jboss.as.test.integration.domain.mixed.MixedDomainTestSuite.getSupport(MixedDomainTestSuite.java:59)
	at org.jboss.as.test.integration.domain.mixed.eap620.MixedDomain620TestSuite.initializeDomain(MixedDomain620TestSuite.java:43)
	at org.jboss.as.test.integration.domain.mixed.eap620.MixedDomainDeployment620TestCase.beforeClass(MixedDomainDeployment620TestCase.java:37)
------- Stdout: -------
21:26:08,374 INFO  [org.jboss.modules] (main) JBoss Modules version 1.5.2.Final
21:26:08,559 INFO  [org.jboss.as.process.Host Controller.status] (main) WFLYPC0018: Starting process 'Host Controller'
[Host Controller] 21:26:09,051 INFO  [org.jboss.modules] (main) JBoss Modules version 1.5.2.Final
[Host Controller] 21:26:09,214 INFO  [org.jboss.msc] (main) JBoss MSC version 1.2.6.Final
[Host Controller] 21:26:09,261 INFO  [org.jboss.as] (MSC service thread 1-7) WFLYSRV0049: WildFly Full 10.2.0.Final-SNAPSHOT (WildFly Core 2.2.1.Final-SNAPSHOT) starting
[Host Controller] 21:26:09,861 INFO  [org.jboss.as.controller.management-deprecated] (Controller Boot Thread) WFLYCTL0028: Attribute 'permgen-size' in the resource at address '/host=master/jvm=default' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:09,862 INFO  [org.jboss.as.controller.management-deprecated] (Controller Boot Thread) WFLYCTL0028: Attribute 'max-permgen-size' in the resource at address '/host=master/jvm=default' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:09,900 INFO  [org.jboss.as.host.controller] (Controller Boot Thread) WFLYHC0003: Creating http management service using network interface (management) port (9990) securePort (-1)
[Host Controller] 21:26:09,902 INFO  [org.xnio] (MSC service thread 1-1) XNIO version 3.4.0.Final
[Host Controller] 21:26:09,909 INFO  [org.xnio.nio] (MSC service thread 1-1) XNIO NIO Implementation Version 3.4.0.Final
[Host Controller] 21:26:09,965 WARN  [org.jboss.as.domain.management.security] (MSC service thread 1-7) WFLYDM0111: Keystore C:\BuildAgent\work\93ecd9ad966b2aa5\full\testsuite\mixed-domain\target\domains\MixedDomain620TestSuite\master\configuration\application.keystore not found, it will be auto generated on first use with a self signed certificate for host localhost
[Host Controller] 21:26:09,981 INFO  [org.jboss.remoting] (MSC service thread 1-2) JBoss Remoting version 4.0.21.Final
[Host Controller] 21:26:10,006 INFO  [org.jboss.as.remoting] (MSC service thread 1-6) WFLYRMT0001: Listening on [::1]:9999
[Host Controller] 21:26:11,509 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0060: Http management interface listening on http://[::1]:9990/management
[Host Controller] 21:26:11,510 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0054: Admin console is not enabled
[Host Controller] 21:26:11,510 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0025: WildFly Full 10.2.0.Final-SNAPSHOT (WildFly Core 2.2.1.Final-SNAPSHOT) (Host Controller) started in 2843ms - Started 48 of 52 services (18 services are lazy, passive or on-demand)
INFO  [org.jboss.as.test.integration.domain.management.util.DomainLifecycleUtil] HostController started in 3619 ms
[Host Controller] 21:26:13,690 WARN  [org.jboss.as.messaging] (management-handler-thread - 1) WFLYMSG0071: There is no resource matching the expiry-address jms.queue.ExpiryQueue for the address-settings #, expired messages from destinations matching this address-setting will be lost!
[Host Controller] 21:26:13,691 WARN  [org.jboss.as.messaging] (management-handler-thread - 1) WFLYMSG0072: There is no resource matching the dead-letter-address jms.queue.DLQ for the address-settings #, undelivered messages from destinations matching this address-setting will be lost!
[Host Controller] 21:26:14,936 INFO  [org.jboss.as.controller.management-deprecated] (management-handler-thread - 4) WFLYCTL0028: Attribute 'default-stack' in the resource at address '/' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:15,614 INFO  [org.jboss.as.controller.management-deprecated] (management-handler-thread - 1) WFLYCTL0028: Attribute 'stack' in the resource at address '/' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:16,194 INFO  [org.jboss.as] (MSC service thread 1-8) WFLYSRV0050: WildFly Full 10.2.0.Final-SNAPSHOT (WildFly Core 2.2.1.Final-SNAPSHOT) stopped in 14ms
[Host Controller] 21:26:16,195 INFO  [org.jboss.as] (MSC service thread 1-3) WFLYSRV0049: WildFly Full 10.2.0.Final-SNAPSHOT (WildFly Core 2.2.1.Final-SNAPSHOT) starting
[Host Controller] 21:26:16,211 INFO  [org.jboss.as.controller.management-deprecated] (Controller Boot Thread) WFLYCTL0028: Attribute 'permgen-size' in the resource at address '/host=master/jvm=default' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:16,212 INFO  [org.jboss.as.controller.management-deprecated] (Controller Boot Thread) WFLYCTL0028: Attribute 'max-permgen-size' in the resource at address '/host=master/jvm=default' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:16,215 INFO  [org.jboss.as.host.controller] (Controller Boot Thread) WFLYHC0003: Creating http management service using network interface (management) port (9990) securePort (-1)
[Host Controller] 21:26:16,222 WARN  [org.jboss.as.domain.management.security] (MSC service thread 1-3) WFLYDM0111: Keystore C:\BuildAgent\work\93ecd9ad966b2aa5\full\testsuite\mixed-domain\target\domains\MixedDomain620TestSuite\master\configuration\application.keystore not found, it will be auto generated on first use with a self signed certificate for host localhost
[Host Controller] 21:26:16,251 INFO  [org.jboss.as.remoting] (MSC service thread 1-5) WFLYRMT0001: Listening on [::1]:9999
[Host Controller] 21:26:16,380 INFO  [org.jboss.as.controller.management-deprecated] (Controller Boot Thread) WFLYCTL0028: Attribute 'default-stack' in the resource at address '/profile=full-ha/subsystem=jgroups' is deprecated, and may be removed in future version. See the attribute description in the output of the read-resource-description operation to learn more about the deprecation.
[Host Controller] 21:26:16,504 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0060: Http management interface listening on http://[::1]:9990/management
[Host Controller] 21:26:16,504 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0051: Admin console listening on http://[::1]:9990
[Host Controller] 21:26:16,504 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0025: WildFly Full 10.2.0.Final-SNAPSHOT (WildFly Core 2.2.1.Final-SNAPSHOT) (Host Controller) started in 307ms - Started 48 of 52 services (18 services are lazy, passive or on-demand)
21:26:17,337 INFO  [org.jboss.modules] (main) JBoss Modules version 1.3.0.Final-redhat-2
21:26:17,484 INFO  [org.jboss.as.process.Host Controller.status] (main) JBAS012017: Starting process 'Host Controller'
[Host Controller] 21:26:17,916 INFO  [org.jboss.modules] (main) JBoss Modules version 1.3.0.Final-redhat-2
[Host Controller] 21:26:18,046 INFO  [org.jboss.msc] (main) JBoss MSC version 1.0.4.GA-redhat-1
[Host Controller] 21:26:18,092 INFO  [org.jboss.as] (MSC service thread 1-6) JBAS015899: JBoss EAP 6.2.0.GA (AS 7.3.0.Final-redhat-14) starting
[Host Controller] 21:26:18,737 INFO  [org.xnio] (MSC service thread 1-4) XNIO Version 3.0.7.GA-redhat-1
[Host Controller] 21:26:18,745 INFO  [org.xnio.nio] (MSC service thread 1-4) XNIO NIO Implementation Version 3.0.7.GA-redhat-1
[Host Controller] 21:26:18,760 INFO  [org.jboss.remoting] (MSC service thread 1-4) JBoss Remoting version 3.2.18.GA-redhat-1
[Host Controller] 21:26:18,977 INFO  [org.jboss.as.remoting] (MSC service thread 1-8) JBAS017100: Listening on [::1]:19999

@jamezp is this ok in a community micro release (2.2.1.Final)?
There are no EAP implications.