Install with quickstart fails

What happened:
Edit: Seems to be an issue in the Landscaper chart. If chart is installed into a different namespace than "ls-system" (and the namespace ls-system does not exist) the Landscaper pod is broken with the message "service account could not be created in a namespace ls-system, which does not exist" in the logs.
Installing landscaper with the quickstart command fails into a new cluster if no namespace is provided with the error "that a service account could not be created in a namespace ls-system, which does not exist".
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:
Environment:

Suppose you install Landscaper in another namespace namespace than the default ls-system,
let's say in namespace ls-test.
The solution is to use the following helm values with the namespace at two places:
landscaper:
  landscaper:
    deployerManagement:
      namespace: ls-test
      agent:
        namespace: ls-test
    deployers: [container, helm, manifest]

If landscaper.landscaper.deployerManagement.agent.namespace is missing, the built-in agent tries to create service account deployer-default in namespace ls-system.
If landscaper.landscaper.deployerManagement.namespace is missing, it is being attempted to create the installations for the deployers in namespace ls-system.

Both fails if namespace ls-system doe not exist.


Function variable listing standardization

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes  #934
Ref. #84
What are the significant changes in functionality due to this change request?
Standardize the Functions block in terms of variable listing (variables like in ExternalModel)
In addition, it finally allows the Functions class to use the InputData (ref #84)
Since a change of the syntax occurred, the following proposed email has been written:
Dear users, as Pull request #935 , we changed the syntax for inputting external functions (<Functions>) to be consistent
with other objects in RAVEN (e.g. ExternalModel, etc.). Previously, the user needed to separately list each variable to be imported in the function using multiple XML nodes (<variable>). Now, you can just list those variables in a comma separated list within the XML node named <variables>. To ease the transition, a conversion script has been generated and can be run as follows:

python scripts/conversionScripts/toFunctionsNode.py inputFileToBeConverted.xml

Thanks a lot.

RAVEN team

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

I will address the other comments (actually all of them :D ) tonight.
Thanks for the prompt review

@PaulTalbot-INL addressed comments

Code reviewed, continuing with PR QA.

Did the documentation get updated? I didn't see it in the files changed, and the search doesn't turn up any changes in the user manual.

This PR modifies requirements tests, and will need CCB Chair-level approval before merging.

I forgot the manual. Modified.

As chair of the CCB I approve these changes since it touches requirements' tests but does not modify or impact any requirement. Approved by @alfoa

Merging. An email needs to be sent regarding this syntax change.

email has been sent out.


Switch recurrent outputs to match Elixir convention

Recurrent functions currently return {state, seq}. The state is like an accumulator and seq is like an Enumerable. Elixir's map_reduce returns {enum, acc}, so I think it makes sense to use that convention here

Is this for documentation sake? if so, absolutely yes! if the aim is to provide a transparent API for new-to-ai developers alike; this is a fantastic step in the right direction. As someone who has recently done that journey, the semantics of the language was the most important thing for me to be able to work things out.


Pattern mismatch error resolved by adding 'enterprises/' before enterprise_name

added 'enterprises/' before enterprise_name in Create a policy and Provision a device using a QR code to overcome this error: TypeError: Parameter "name" value "LC01gzjgxa/policies/policy1" does not match the pattern "^enterprises/[^/]+/policies/[^/]+$"
for more details please check: https://stackoverflow.com/q/53954963/9752602

Done with signing

Thank you for the pull request.
enterprise_name should already be in the format "enterprises/<enterprise_id>", it is set in this format at enterprise creation, maybe in the cell where you manually paste the enterprise_name you forgot to copy the prefix as well?


fix(fseek&ftell): fix a bug of fseek/ftell on win, which fail to work when file size > 2GB(max bound of long type)

when file size > 2GB, fseek will work wrong, ftell return -1 on windows

Thanks for the contribution! It looks like we already have a CI check that runs tests on Windows. Would you be able to add a test case to ensure this works as expected?

my test env is VS2019 with msvc1929, read a mcap file  >  2GB will reproduce the problem, or you can just test fseek&ftell API derictly on win, i'm sorry for my pr not pass your CI due to my lack of consideration, or it should just be a issue.

in LogTimeOrder will act more bad, nothing will return by readMessages API, in FileOrder still can get data although file size get wrong -1

@no5-aaron-wu Are you able to update this PR to add the unit test requested? It also looks like some other aspects of the PR are failing CI. Consider merging latest main and the swift CI issue might be resolved.

Closing this out since we've not heard back. We are happy to re-visit if you can address the issue around unit tests.


Fix name for Series.cumxxx.

Codecov Report

Merging #1145 into master will decrease coverage by 2.53%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1145      +/-   ##
==========================================
- Coverage   95.19%   92.66%   -2.54%
==========================================
  Files          35       35
  Lines        7075     7075
==========================================
- Hits         6735     6556     -179
- Misses        340      519     +179

Impacted Files
Coverage Δ

databricks/koalas/series.py
96.35% <100%> (-0.14%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.32% <0%> (-72.98%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.72% <0%> (-6.39%)
:arrow_down:

databricks/conftest.py
92% <0%> (-6%)
:arrow_down:

databricks/koalas/frame.py
93.64% <0%> (-3.24%)
:arrow_down:

databricks/koalas/groupby.py
89.64% <0%> (-2%)
:arrow_down:

databricks/koalas/namespace.py
85.97% <0%> (-1.53%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

databricks/koalas/testing/utils.py
78.51% <0%> (-0.75%)
:arrow_down:

... and 2 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 c03b3a6...e8596fa. Read the comment docs.


Is there a way to integrate this with grunt?

The documentation seems to only indicate manual execution.  I originally wanted to push the results to TestRail, but the syncing features looks really cool!
I have Chimp (using cucumber) running and exporting a JSON reports file.
Please advise.

Hi,
You can use something like grunt-exec to run commands.

How about the manual push (https://github.com/SSENSE/node-cucumber-testrail-sync/blob/master/docs/pushing_results.md)?  Does that require some grunt execution?  Or does the hook.js auto run in the context of cucumber to push the file?

Once you setup the hooks.js file, all you have to do is to run the tests with the --push flag: ./node_modules/.bin/cucumber-testrail-sync --push

Once you setup hooks.js, results will be pushed when you run the cucumber tests if you set pushResults: true in your config file.

Couple of questions since I'm unable to do this successfully:

I'm running cucumber through Chimp.  Will this still work?  Have you guys tried it?
My "pushResults" property in .testrail-sync.js looks like this.  It is at the root of my repo.  Whereas the ".feature" files are in some sub-folders (/tests/automation/features).  Is this correct?

module.exports = {
  testrail: {
    host: 'https://xxx.testrail.com',
    user: 'xxx',
    password: 'xxx',  // password or api key
    pushResults: true,
    filters: {
       plan_id: '2407'

I installed "hooks.js" into "/tests/automation/features/support" (cucumber's context?).  Hooks.js is set for cucumber 1.x

Please advise. Thank you.

No we didn't try it. From what I can see, Chimp is used to test UIs but cucumber.js still run the tests, right ? If so, it will work.
Yes. But were the .feature files synced with this module ? Basically, the sync process creates @tcid:xyz tags in each .feature file where xyz is the id of the testcase in TestRail.
Then after each test are ran, the module looks for the value of the @tcid tag and push the test result of that testcase id to TestRail. If the tag doesn't exist, the module has no way to link the test with the correct TestRail's testcase.


success class

It would be great, similar to having the fieldWithErrors class, t to have a fieldWithSuccess class, so you can easier apply some styling when something was entered correctly.

Closing in favor of #229


Duplicate index ls_licensable_products_licensable_type_licensable_id_index

Hey,
Fresh Laravel 9 installation
php 8.0.19
windows 10 x64
Reduce steps

install laravel
install the package
don't publish the migrations
migrate

Screenshot:

Possible work around:
Change prefix in config file before publishing migrations and before php artisan migrate command.
in lincense-server.php
    /**
     * This prefix will be used for all tables created by the license-server.
     *
     * Table name prefixes are used to prevent naming conflicts when using multiple stores
     * or local conflicts with other tables in the database.
     *
     * Example table name: ls_licenses
     *
     * Default: ls
     */
    'default_table_prefix' => 'your_project_name',

This issue was related to MySql's unique index query and running order. Fixed in 8151f1c commit.
Thanks for the feedback 🙌.


Implement /v1/accounts/:address/data-settings

Summary
Changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

19 of 37 (51.35%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 48.481%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/domain/accounts/accounts.repository.ts
0
2
0.0%

src/routes/accounts/accounts.controller.ts
5
7
71.43%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/routes/accounts/accounts.service.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4230

Relevant Lines:
7036

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

19 of 55 (34.55%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 48.37%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/accounts/accounts.controller.ts
5
7
71.43%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
8
0.0%

src/routes/accounts/accounts.service.ts
0
11
0.0%

src/datasources/accounts/accounts.datasource.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4230

Relevant Lines:
7054

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

20 of 54 (37.04%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 48.396%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/accounts/accounts.controller.ts
6
7
85.71%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
9
0.0%

src/routes/accounts/accounts.service.ts
0
9
0.0%

src/datasources/accounts/accounts.datasource.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4234

Relevant Lines:
7046

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

20 of 49 (40.82%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 48.431%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/accounts/accounts.controller.ts
6
7
85.71%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
4
0.0%

src/routes/accounts/accounts.service.ts
0
9
0.0%

src/datasources/accounts/accounts.datasource.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
4234

Relevant Lines:
7041

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

21 of 51 (41.18%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 48.431%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/domain/accounts/entities/upsert-account-data-settings.dto.entity.ts
4
5
80.0%

src/routes/accounts/accounts.controller.ts
6
7
85.71%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
4
0.0%

src/routes/accounts/accounts.service.ts
0
9
0.0%

src/datasources/accounts/accounts.datasource.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
4235

Relevant Lines:
7043

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

21 of 51 (41.18%)  changed or added relevant lines in 8 files are covered.
98 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.2%) to 48.431%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/domain/accounts/entities/upsert-account-data-settings.dto.entity.ts
4
5
80.0%

src/routes/accounts/accounts.controller.ts
6
7
85.71%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
4
0.0%

src/routes/accounts/accounts.service.ts
0
9
0.0%

src/datasources/accounts/accounts.datasource.ts
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/mappers/common/swap-order.mapper.ts
5
27.27%

src/routes/transactions/transactions-view.service.ts
9
17.14%

src/routes/transactions/helpers/twap-order.helper.ts
17
24.44%

src/routes/transactions/mappers/transfers/swap-transfer-info.mapper.ts
17
15.25%

src/routes/transactions/helpers/swap-order.helper.ts
20
32.43%

src/routes/transactions/mappers/common/twap-order.mapper.ts
30
24.73%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
4235

Relevant Lines:
7043

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

21 of 51 (41.18%)  changed or added relevant lines in 8 files are covered.
98 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.2%) to 48.431%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/domain/accounts/entities/upsert-account-data-settings.dto.entity.ts
4
5
80.0%

src/routes/accounts/accounts.controller.ts
6
7
85.71%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
4
0.0%

src/routes/accounts/accounts.service.ts
0
9
0.0%

src/datasources/accounts/accounts.datasource.ts
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/mappers/common/swap-order.mapper.ts
5
27.27%

src/routes/transactions/transactions-view.service.ts
9
17.14%

src/routes/transactions/helpers/twap-order.helper.ts
17
24.44%

src/routes/transactions/mappers/transfers/swap-transfer-info.mapper.ts
17
15.25%

src/routes/transactions/helpers/swap-order.helper.ts
20
32.43%

src/routes/transactions/mappers/common/twap-order.mapper.ts
30
24.73%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
4235

Relevant Lines:
7043

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

21 of 55 (38.18%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 48.134%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/domain/accounts/entities/upsert-account-data-settings.dto.entity.ts
4
5
80.0%

src/routes/accounts/accounts.controller.ts
6
7
85.71%

src/routes/accounts/entities/account-data-setting.entity.ts
5
8
62.5%

src/domain/accounts/accounts.repository.ts
0
6
0.0%

src/routes/accounts/accounts.service.ts
0
9
0.0%

src/datasources/accounts/accounts.datasource.ts
0
14
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4243

Relevant Lines:
7074

💛  - Coveralls

The endpoint PUT /v1/accounts/:address/data-settings was added to AccountsController, protected by AuthGuard.
Associated tests and domain classes were added.


korean language file for 1.0.1 is here!

small translation.
ko_kr.zip
thx for good mod

Thank you for your contribution! The next release will include these.

I noticed "Distance by Cart" was not translated. This is used in the "Statistics" screen of the game:

So I have used "수레 타고 이동한 거리", based on similar statistics.

thank you. I didn't know where that line used. and I think "수레 타고 이동한 거리" is best choice!


cannot start container strapi:latest

Hello I tryed to run strapi + mongodb on docker (making them run from portainer) but it crashes at start
ENV parameters:

parameter
value

DATABASE_CLIENT
MongoDB

DATABASE_HOST
mongodb_strapi

DATABASE_USERNAME
*****

DATABASE_PASSWORD
*****

PATH
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

NODE_VERSION
9.11.1

YARN_VERSION
1.5.1

DATABASE_NAME
strapi

DATABASE_PORT
27017

DATABASE_SSL
false

error from docker logs:
🚀 Start creating your Strapi application. It might take a minute, please take a coffee ☕
Lets configurate the connection to your database:
Database determined by CLI args: MongoDB
(node:6) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'value' of undefined
    at inquirer.prompt.then.answers (/usr/local/lib/node_modules/strapi/node_modules/strapi-generate-new/lib/before.js:111:53)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:182:7)
    at Function.Module.runMain (internal/modules/cjs/loader.js:697:11)
    at startup (internal/bootstrap/node.js:201:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:516:3)
(node:6) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:6) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
./strapi.sh: cd: line 28: can't cd to strapi-app

docker engine version:

parameter
value

Version
18.06.0-ce

API version
1.38

Go version
go1.10.3

OS type
linux

OS
Ubuntu 16.04.5 LTS

Architecture
amd64

Kernel version
4.13.0-1012-gcp

thank you!

Your DATABASE_CLIENT environment variable should be one of:

mongo
postgres
mysql

Same here:
Attaching to strapi_api_1
api_1  | 🚀 Start creating your Strapi application. It might take a minute, please take a coffee ☕
api_1  |
api_1  | Lets configurate the connection to your database:
api_1  | Database determined by CLI args: mongo
api_1  |
api_1  | ⏳ Testing database connection...
api_1  | { Error: Cannot find module '/tmp/strapie0ea507fb540/node_modules/strapi-hook-mongoose/lib/utils/connectivity.js'
api_1  |     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)
api_1  |     at Function.Module._load (internal/modules/cjs/loader.js:475:25)
api_1  |     at Module.require (internal/modules/cjs/loader.js:598:17)
api_1  |     at require (internal/modules/cjs/helpers.js:11:18)
api_1  |     at Promise.all.then (/usr/local/lib/node_modules/strapi/node_modules/strapi-generate-new/lib/before.js:259:15)
api_1  |     at  code: 'MODULE_NOT_FOUND' }

I had the same issue but without docker and yarn global add strapi@alpha solved my problem
:smile:

Don't use yarn with Strapi, not fully supported.

@matteo-bombelli , @BlackDark were you able to solve the problem?
I also just installed a fresh instance of mongo and strapi. I'm getting the same database connection error
Testing database connection...
⚠  Database connection has failed! Make sure your database is running.
My database is running as i've been able to connect to it using navicat.
Env variables
PATH:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
NODE_VERSION:11.1.0
YARN_VERSION:1.12.1
DATABASE_HOST:localhost
DATABASE_PORT:32818
DATABASE_CLIENT:mongo
DATABASE_NAME:strapi
My docker Port is 27017
What could I be doing wrong? I've searched everywhere online. Kindly assist.

@rabira-hierpa Comment as-tu résolu ton probléme ? j'ai le meme souci de connexion... utilises-tu Portainer pour administer tes containers ?

@Guillaume175 can you post/edit your message in english please.

Can you give us an update please.
Based on last version of files.

HI,
We just released a full rebuild of our docker image 🎉
We are closing the issues and open PRs to start fresh.
If you feel like something is missing or want to add things feel free to open new PRs ! 💯

Is that possible to connect strapi with mariadb?

Yap! You will have to use the same config as a MySQL database.
But you set the HOST and PORT of your MariaDB database.
It's because it uses the same driver as MySQL


Loading a pre trained keras model and running in pyspark

Hi,
I am facing an issue with an already trained keras model. When I tried to run the model on files that I read in python, the model gave me results as expected. But, when those files were read as an rdd, it threw the following error:
AttributeError("'Model' object has no attribute '_feed_input_names'",)
Is there any different method of running keras models on spark?

@raviranjan-innoplexus  - Is this still an issue ? Are you using tensorflow back-end for this ?


watch

Model.watch(req) print a debug message :
debug: sails.sockets.id() is deprecated; use sails.sockets.getId() (see http://sailsjs.org/documentation/concepts/upgrading/to-v-0-12)

Thanks @ljacques - there's a fix in place on master and we'll be removing this message in a patch release this week (.watch() formerly called .id() instead of .getId() and we forgot to switch it over during the upgrade.  The only reason for the name change was to make the behavior clearer, and .id() is just an alias- so there's nothing to worry about in the mean time).

Thanks @mikermcneil
I keep testing ;-)


Centos 7 install error

Hello !
This is my first deployment using ceph-ansible and is my first deployment using ansible too.
GATHERING FACTS ***************************************************************
ok: [osd02]
ok: [osd01]
ok: [osd03]
TASK: [ceph.ceph-common | fail on unsupported system] *************************
skipping: [osd01]
skipping: [osd02]
skipping: [osd03]
TASK: [ceph.ceph-common | fail on unsupported architecture] *******************
skipping: [osd01]
skipping: [osd02]
skipping: [osd03]
the full log are on http://pastebin.com/enErnEQT.

@rlljorge from the error it looks like you are missing some config in the group_vars/osd file.
Have you set the following parameters.
journal_collocation:
devices:
regards,

@bdastur I had already set this on config files:
root@deploy:/opt/ceph-ansible# grep "^[^#;]" group_vars/osds
devices:

/dev/sdc
/dev/sdd
/dev/sde

raw_multi_journal: true
raw_journal_devices:

/dev/sda
/dev/sdb
/dev/sda


exposing plugin API to allow in-plugin communication

🚀 Feature
Integrating the possibility to get access to some plugin APIs from other plugins source code and using the yaml file description to expose these functions.
Motivation
Enabling the possibility for plugins to exchange information to enhance automation/processing capabilities and combine existing plugins functionalities.
Pitch
One of the future features of my napari-live-recording would be to integrate a processing pipeline which collects a stream of images, do some computation over each image, and show the result on the viewer / store it. I considered three possibilities:

hardcoding image processing functionalities in the plugin;
loading external scripts;
installing other plugins as dependencies.

Having the possibility to exchange/retrieve functionalities as a feature of napari itself would be beneficial for this - but it would also encourage testing other plugins with less effort.
Alternatives
Adding a documentation page on how to expose some functionalities of one's plugin so that other developers may access it. This is similar to installing other plugins as dependencies as a solution though.

Could you describe what type of API you would like to have?

Supposing that you have a plugin that makes some computation over an image, i.e. grayscale using an external package, it would be cool if one could add a decorator that allows me to get visibility of it whenever I launch my plugin:
On the external plugin side it would be something like:
# grayscale plugin

@napari.plugins.export
def compute_grayscale(image: np.ndarray) -> np.ndarray:
     return compute_my_plugin_grayscale(image)

and on my plugin side it would be:
from napari.plugins import list_apis

class MyPlugin(QWidget):
    def __init__():
        registered_apis = list_apis("name-of-plugin")

Or something along these lines.

If you just would like to have access to list of plugin contributions, then you should take a look at npe2 as this package is responsible for discovering and listing contributions.

You're right, I should definetely move the discussions there. But I thought that as you mentioned npe2 just lists plugins but doesn't give direct access to functions in this way. Maybe I'm missing something.


Fixes resolution change issue #3262

Description

Screenshots (optional)

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have signed the Google CLA https://cla.developers.google.com
[x] My code follows the style guidelines of this project
[x] I have run ./build/all.py and the build passes
[x] I have run ./build/test.py and all tests pass

@TheModMaker I know that you wanted this issue to be fixed by one of the team members later but I think this solution is simple enough to be merged, I am a little doubtful about my implementation of the variable this.isTriggeredByResolution. I would like a review on this. @aditmehta9 I tested for any kind of bugs that might arise, you have been working on this so you kind of would know if any issue would arise, could you please check for any issues if you have time. Thank you.


Integration tests - add support for testing webhooks

Add support to the integration test framework to support testing for received webhooks.
The most important is to test that the holder receives notifications when their credentials are revoked, but would be useful in general to include tests for each type of webhook (credential offer received, credential request received, proof request received etc).
Also can include support for notification of basic messages (which I think are also missing from the integration tests).

We've definitely had issues in the past with the sequence of WebHooks changing unexpectedly when updating ACA-Py. It would be good to be able to catch that.
Not sure if that is easily done for a given protocol, and less so if it can be easily done across all the protocols.


add zone in 3.6.0-ent

https://github.com/vesoft-inc/nebula-ent/pull/3116/files

https://confluence.nebula-graph.io/pages/viewpage.action?pageId=72610208  zone feature updates


feat(query): Added NAS File System Without KMS for Alicloud

Proposed Changes

NAS File System Without KMS for Alicloud

I submit this contribution under the Apache-2.0 license.

Checkmarx AST – Scan Summary & Details – 9e5a6764-72c6-4b2a-b149-31efd4f1cbc4
Fixed Issues

Severity
Issue
File / Package
Scan Engine

HIGH
ALB Listening on HTTP
/assets/queries/cloudFormation/aws/alb_listening_on_http/test/positive3.yaml: 8
CxKICS

Checkmarx AST – Scan Summary & Details – c64a066d-323f-4dcc-93f4-abec92af5846
Fixed Issues

Severity
Issue
File / Package
Scan Engine

HIGH
ALB Listening on HTTP
/assets/queries/cloudFormation/aws/alb_listening_on_http/test/positive3.yaml: 8
CxKICS


Add low kick power calculation

Heyo, I saw pie constantly looking up weights to calculate low kick power and got curious about adding it in
From what i've tested so far seems to work, and should in theory work for enemy mons using low kick on you (showing the power your weight gives them) but the latter part there is hard to test. It still shows the default value "WT" outside of battle.
I'm aware of a similar pull request made (#53 ) so seems they did it at the same time lol but still adding this for your consideration between which to use
Calculation and data were taken from serebii and should also work for grass knot should gen 4 support be added sometime in the future

Thanks for the submission, Fellshadow. Since OnlySpaghettiCode beat you to the punch, we are going to go with theirs instead. I really appreciate the effort and time you put into this and helping to make this tracker better for everyone.

no worries, I felt that'd be the case but figured i'd shoot my shot anyways haha
who knows, maybe another time i'll get there first ;) regardless keep up the great work guys!


Fixed #6707 -- Added RelatedManager.set() and made descriptors' set use it.

Looks good to me.

Thinking about it a bit more, maybe set() should accept an iterable rather than *objs, @collinanderson raised the issue in https://github.com/django/django/pull/2500#discussion_r23572348.
I used *objs for consistency with add() and remove() but those two make sense for a single object, less so for set(). What do you think @akaariai, @carljm, @timgraham?

Did you see https://code.djangoproject.com/ticket/13240 which proposes that for related methods?

@timgraham wasn't aware of it. Can't say I like the idea of flattening all iterables like done in that patch, although I would accept special-casing when there is only 1 object and that object is an iterable (we should keep in mind that it would probably break models with __iter__).
Regarding set(objs) vs set(*objs), I'm 40/60 in favor of the latter :

When I think of it as a shorthand for clear() + add(), then I really want consistency with add().
When I think of a set that I just want to assign, then I find set(objs) nicer.

I also think .set(objs) is better than .set(*objs). I think the strongest parallel for .set() isn't .add(), it's "what you retrieve", which is an iterable.
(It's a tangent, but I also don't like methods that accept either and magically flatten iterables.)

I think the strongest parallel for .set() isn't .add(), it's "what you retrieve", which is an iterable.

That's very true.

I'd also argue that you're more likely to use add() for a single object than you are for set(). Most of the time you'll pass a collection of items when using set() since it's the many part of the relationship.
I think say set() shares more with bulk_create() than add() in this regard.


Add support for custom morph class resolvers

Hey there! This PR adds an optional resolveUsing setting to the morph map generator.
This is a similar idea to what was proposed in #3, but enables users to customize how the "default" alias is resolved by providing a callback.
For instance, you could base the morph names off the table name for each model like this:
MorphMapGenerator::resolveUsing(fn ($model) => Str::singular($model->getTable()));

This would generate names like post or comment depending on the model's table name in the DB (which should be pretty stable even if you rearrange your model classes). This is just an example though, and you could add whatever logic you might want there.
Let me know what you think. Thanks!

Thanks for the PR but I don't think is a feature we want to add? The whole point of a morph map is setting things in stone, with this PR you allow things that can be changed to be used as morph keys and thus your morph map won't be stable again.
In you example, the table name can be changed at a point in time and thus there is no validity in your morph map.
What do you think @freekmurze?

I'm always up for a little flexibility, but I agree there's value in following the strict spirit of this package, and pass up on this.

My thinking is: if you're changing a DB table name, you're already making significant changes to your DB, so you'd probably expect to have to make changes to things like this as well.
There's a lot of functionality in Laravel that derives table/column names from class names, method names, etc. so IMO this fits in with the overall Laravel "philosophy" but I understand your concerns as well :) It's an opt-in feature though, so anyone with concerns about stability wouldn't have to use it.
In any case, I've made the requested changes in case you do choose to merge this. Thanks!!

Let's merge it in with a strong warning 😄

Thanks

Hey, thanks for merging this! Much appreciated 😃


v2: How do I add or remove a dataset to or from the datasets of a chart

Background
There is two way to update datasets of a chart.

Set a new datasets by this.setState
Update the this.state.datasets and call this.forceUpdate

1 works well only update datasets without changing a number of datasets.
2 works well all manipulation.
Implementation
The source code below is effective when 1.
But it is not effective when 2. Because the chart.data.datasets is already updated.

assign all from the next datasets to the current chart
```
// assign all of the properites from the next datasets to the current chart
nextProps.data.datasets.forEach(function(set, setIndex) {
  var chartDataset = chart.data.datasets[setIndex];
for (var property in set) {
if (set.hasOwnProperty(property)) {
chartDataset[property] = set[property];
}
}
});
</details>
https://github.com/jhudson8/react-chartjs/blob/chartjs-v2/lib/core.js#L57

# Question

Which method is better to change the datasets of a chart?

I think this is similar to #38.


option to use night mode without js

title says it all

how?

Cookie?

Set alternative attribute on the style link. Which makes it selectable from View > Page Style. Or for noscript users add a link to a server side script that adds a cookie which then is read by the server side script and serves the correct stylesheet.

this sounds potato af

if you want to change the night mode quickly JS is required. (don't over complicate things)
There will most likely be a setting in the account profile to change your whole account theme.


Issue with Cents

When you set a number with a 0 after the decimal point, for example 1.05, as the amount on the payment slip, it is displayed as 1.50. The reason is because the str_pad in the function getAmountCents() sets the 0 after the 5 and not before. I put a patch file in the zip that i attached here with the fix to this issue
Changes.zip

@FFDani thanks for the report.
Could you create a Pull Request with the changes included?
At best would be if you could also add a test to cover this case so we don't get a regression in the future.
By the way the correct repository for this would be https://github.com/ravage84/SwissPaymentSlip/

Closing as PR https://github.com/ravage84/SwissPaymentSlip/pull/21 is provided.


Standardize config and provide sensible defaults

Currently supported keys, such as printResultSummaryOnly should be deprecated.
Keys that are often the same, such as indentation and fileType, could be moved to a global level or eliminated in favor of a default.

Part of this task should also be to move conifg into a new folder ./.parselt


World Edit (FAWE) Tab Completion broken for setting multiple block types at once

Hello, I have tested this issue, normally with FAWE you can set many blocks in a position for example: //set stone,granite,andesite as well as percentages for each block. But with Suggestion Tweaker installed, tab completion stops working after the first block choice. so only //set stone would tab complete, and I have to type the other blocks out manually, super tedious so I felt the need to post the bug. This is on a Paper Server in 1.20.4 Excuse my unprofessionality.
I'd also like to note that with FAWE you do not use spaces in between each block choice, unlike normal World Edit, I believe it used too or still does use spaces, so keep that in mind.

I've made a quick fix for this, which isn't perfect, unfortunately. The fix only works for suggestions that don't contain separator characters like spaces, commas and so on, which are most of them, thankfully. I'll see what I can do to make a proper fix.


Error: Can’t get every text item of missing value. (-1728)

Without Zoom running, run the script:
./zoom_roster.applescript

When the dialog showing "Please join the Zoom Meeting and press OK." shows up, join a meeting and press OK.
You'll get the Quit dialog showing: "Can’t get every text item of missing value. (-1728)"

Plan to tackle this:

Improve error handling code to retry the getting of UI elements from windows.

Fixed with refactored code and better understanding of Applescript.


Replace asserts with NC_EINTERNAL...

I think we should consider replacing as many asserts as possible with a return value
of NC_EINTERNAL. Crashing the library is generally not a good idea from the user
point of view.

I used to feel this way, then Russ convinced me to give asserts a try, and I believe they have been very successful in netCDF development. Asserts are only used when we know it cannot happen. And I don't believe we are getting a lot of reports of asserts - they are rare.
Right now, when a return happens, it is because a return is possible. When an assert happens, we have made a bad programming error. At that point, crashing is preferable to continuing on and maybe messing up data files.
Asserts are well-understood and do improve code quality. I believe they should not be removed, but if someone wants to compile without them, they should do so. (As is routinely done on HPC builds to improve performance).
With the use of test-coverage tools, it is actually pretty easy to be confident that our asserts are truly impossible. Checking for them with if statements is just not needed.

My analog would be to ENOMEM. If you are out of memory, you
are largely hosed, but we do not crash, we return an error.

NOMEM is quite different in that it is possible.
If I assert(nc) it's because it is impossible for nc to be NULL. If I had to do an if statement instead of an assert, I would just leave it out altogether.
If we did put in such if-statements, it would be reasonable to protect them with a preprocessor flag, so that they could be left out of compiles.
But that just leaves up re-creating the assert that is already built in. I suggest we just continue to use it.

But the fact is that our asserts do get activated. And I suspect that few things
enrage our clients as when the library crashes.

Do we get a lot of reported asserts?

I do applaud any effort to improve code quality, but I don't think this is the way to go. @russrew you may enjoy the irony of seeing me make the same arguments you used to me about 15 years ago. ;-)
I agree that if a user sees an assert, that is a very serious bug that should be addressed immediately. But it is rare. Looking at the 101 open issues, none of them are from asserts. When I look at closed issues, I can see only one that seems to be a user hitting an assert.
As you know I have just done an extensive round of code review, extra testing with test coverage analysis, and refactoring of the libsrc4 code. I am confident that asserts are being used correctly there and would not like to see that changed. In fact, I checked all asserts to make sure we are asserting the maximum we can assert, and checking the minimum with if statements. (This is for performance purposes on HPC.)
If you wish to return error instead, then the way to do this would be to provide your own assert function. This way asserts could be used in a standard way in the code, and existing asserts would not need to change. Your assert function could print an error message, and then return an error instead of halting the process. Your assert function, like the C library one, must go away when the library is compiled with
NDEBUG, so that users can compile without asserts as they do now.
In practice, this will not be an improvement, IMO. Much user code ignores return codes, unfortunately. Even when they check them, the usual result is to then halt the program. Very few science users have code sophisticated enough to deal with such an error in any better way than to just exit. Certainly climate and weather models, written in Fortran, do not have sophisticated error recovery code.
Anyway, what else would be safe to do? If you hit an assert on some libsrc4 code, it means you are missing a necessary pointer. How can anything proceed? Obviously the library is hosed and must be restarted to get it's internal data back into a useful state.
Here's a very typical use of asserts in the libsrc4 code (note that this code is only called for netCDF-4 files):
   /* Find info for this file and group, and set pointer to each. */
   if ((retval = nc4_find_nc_grp_h5(ncid, &nc, &grp, &h5)))
      return retval;
   assert(nc && grp && h5);

In order for the assert to fail, we have called a libsrc4 function on a non-netcdf-4 file (because there is no h5 or grp) or we have no file info struct (nc). There is no reason to continue. If memory is this hosed up, then calling another netCDF-4 function is not a good idea.
So, in summary, asserts have proven valuable to many C projects, and they are valuable in netCDF. Error conditions that can occur should never be checked by an assert, so users should rarely see them. When users do see them, it is hard to think of how a returned error is more useful than just exiting.

Wrong question. The correct question is: do we get a lot of reports of a crash?

I guess I don't understand your question. Perhaps you could elaborate.

I do not recall ever receiving a support question that said
that an assert failed. I have, however, received support questions
that said "netcdf crashed" and it turned out to be an assert failure.
Basically most users are not savy enough to report an assert failure.
They only report "it crashed".

If they included the error message, it would have included the word "assert" which is what I searched for.
I believe most of the crashes were due to bad parameter checking which has been recently improved. For example the rename functions would crash if you passed NULL for the names. Now they return NC_EINVAL, as they should.
Anyway, I guess I have made my case for asserts as well as I can. This matches my usual advice, which is do what all the other well-behaved C projects do.

Didi you search e-support?

By e-support do you mean the github issues? That is what I searched. I don't have access to any other support systems you guys might be using.

Sorry, I forgot that you can not access the esupport (which is the same as
when you worked here).

I hope that I have talked @DennisHeimbigner out of this idea. If so, this issue may be closed.

@DennisHeimbigner, what do you think?

Its ok to close this, but Ed and I will have to agree to disagree.

OK, seems like this issue can be closed.

OK, seems like this issue can be closed.


Incorrect logic in CMS_SIGNED_DATA verify function

Looks like there is a bug in org.pkijs.simpl.CMS_SIGNED_DATA.prototype.verify, where checkCA function is evaluated already when building a checkCA_promises list, i.e. before execution of sequence chain. This results into empty signer_cert object in checkCA function and thus to a failure or incorrect condition evaluation inside the function. I guess it's not intended behaviour?

In fact you are right, could be the case. I have fixed it in latest commit.


[Feature][Module Name] Feature title How do you store an already established graph database in a graph RAG?

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Thank you very much for your work, it has been very beneficial.
But in the demo file DB-GPT-main/examples/rag/graph_rag_example.py, is it possible to store the constructed database locally, so that the next time you ask, you can directly load the local database?
And I can't visualize the database form the web page <IP_ADDRESS>:7687 that the sample builds when I open it during the storage process.
I look forward to your answers in your busy schedule.
Use case
No response
Related issues
No response
Feature Priority
None
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

graph_rag_example只是本地测试，力求简单明了，可以高效走通GraphRAG流程即可。
7687是RPC端口，web访问使用7070/8000访问。

怎么设置？

@yellow123Nike
参考文档：https://docs.dbgpt.site/docs/cookbook/rag/graph_rag_app_develop

@fanzhidongyzby  感谢


document related functions by populating "See Also"

We should list related function in the documentation of all functions a la numpy/scipy. cc @dengemann @agramfort

That's kind of what our main api page does, right? Maybe we can just make
it better.
We should start populating See Also in docstrings, which would also help.

I think I meant populating the See Also :) I'll edit the description of the PR

I am taking this.

YEAHH great call!


Allow disabling PALog macro even in case DEBUG is on

After mobile developers finish debugging PowerAuth integration, our SDK logs are no longer useful to them and clutter the debugging log. Currently, we automatically disable the PALog macro in case DEBUG is not set.
We should also add an extra macro to manually disable the logging, even in the case of DEBUG.

Now it's possible to define DISABLE_PA2_LOG or ENABLE_PA2_LOG to force disable or enable the log.


Remove unused label style from semantic-ui

I really believe that this is not used at all.

Codecov Report

Merging #481 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #481   +/-   ##
=======================================
  Coverage   21.44%   21.44%
=======================================
  Files         166      166
  Lines        1231     1231
=======================================
  Hits          264      264
  Misses        967      967

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 cb25cd6...589e38f. Read the comment docs.


how can I use this admob code with my intel xdk

I'm trying to add a admob advertising to my app, and I see many codes in this site, but how can I integrate it? do i have to creat a file or just copy paste these codes in index.html, help please

where should i insert this code. I inserted before  could you please help me?


Step 4, Add Copilot suggestion from comment: Some jobs were not successful not sure why

Discussed in https://github.com/orgs/skills/discussions/290

Originally posted by Peace-Aielumoh July 18, 2023

![image](https://github.com/skills/.github/assets/95224492/4a7e3b7a-96bc-45bf-a92f-9a0594a532a9)

I'm not sure I understand the question @Vijay1667 but you can drop by https://github.com/orgs/skills/discussions to discuss the courses, or reopen the issue with more details if you think there's a bug in the course.


refactor: #3621 Generic view classes should inherit from TemplateView etc

Fixes #3621
A few comments:

To be consistent, I added get_success_message and get_error_message for classes where it is expected.
I deprecated instance attribute in favour of object which is the standard attribute name used in Django CBV (again for consistency).
I deprecated get_context in favour of get_context_data for the same reason.
Django CBV expect a keyword argument in the URL, while Wagtail always use a positional one. I added the pk kwarg manually if it is not present. I'm not sure what would be the best approach here (keeping it like that, or adding a deprecation warning and redoing all wagtail URLs).
Of course, all current views implementation are still compatible as they are.

Tell me what you think about all that. Thanks!

Looks good, thanks! Merged in bb4ea5922c3989eb486bfbf11ed1092d0719e41e.


frama-c: 20170501 (Phosphorus) -> 20171101 (Sulfur)

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Applied to master (842d8e47660d1600473609f8be5d175825e53a01). Thanks!


feature/remove user info

docs: use the term "removal" for role bindings instead of revocation
docs: add a notice that meshIdB authZ is deprecated

removing the vnext tag for now to unblock the release.


Update networking.md

we can use private domains for internal container apps. So, I suggest to change this docs.

@cachai2
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@cachai2
Can you review this old PR and determine whether it needs to be closed or merged?

emailed author

author is investigating (per email)

@lilyjma, the author directed me to ask you--can you review this PR?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. Or, if the PR should be closed, let us know here.
Thanks. --PR review team

@lilyjma - I accidentally address you in a comment, which I have since deleted. Please ignore--I Iooped you in by accident.

author is investigating (per email)

The change has been integrated in another PR and is now reflected in the live article. Thanks for contributing!


Can't install nbm file, External Browser is requested in impl. version

When I try to install your nbm file, which you've uploaded into your github repo, I got this error:

I use Netbeans 8.1 on Windows 10.
Regards
Chris

The reason behind this issue is that Electron requires Chrome for debugging: http://electron.atom.io/docs/v0.34.0/tutorial/debugging-main-process/
However to use the chrome browser in NetBeans my module had to be either a friend of the browser module or use the implementation. I decided to use the implementation for the beginning.
You get this message because the module requires an old implementation from version 8.0.2. You can fix it by yourself by replacing the implementation version in the module.xml and rebuild the plugin. However I don't think it makes sense to update this plugin. As you already mentioned, the Node.js support in NetBeans 8.1 is much better than it used to be.
So to run an Electron app from NetBeans 8.1 you don't need this plugin at all. Instead extend the package.json for a HTML5 client application like this:
{
"name": "foo-bar",
"version": "1.0.0",
"main"    : "public_html/main.js",
"scripts": {
"electron run": "electron ."
}
}


favicon support for github pages

Putting a favicon.ico in the top directory works when using Jekyll local.  But when I moved my documentation to github pages the favicon.ico did not display.  Adding a link for the favicon in _layouts/default.html resolved the issue of the missing favicon with github pages.

Travis CI failure has nothing to do with my Pull request.  The issue is cuased by an _includes folder not being present.
Error: Invalid theme folder: _includes
There is a file check for the folder _includes in the file: jekyll-theme-minimal.gemspec
When I clone and fork the project the _includes directory is not present.  Most Jekyll templates have this directory (_includes).  So I think an empty _includes folder needs to be added to the repo, Or the check for _includes needs to be removed from: jekyll-theme-minimal.gemspec.
f.match(%r{^((_includes|_layouts|_sass|assets)/|(LICENSE|README)((\.(txt|md|markdown)|$)))}i)


Add global flag to manager and resource CLI subcommands

This PR adds the --global flag to the manager and resource CLI subcommands. Its behavior is identical to the template CLI subcommand's flag of the same name.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "global-flag" git@github.com:nwt/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354406032
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


fix: merge main into rc

:tada: This PR is included in version 2.0.0-rc.4 :tada:
The release is available on:

GitHub release
npm package (@rc dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-rc.5 :tada:
The release is available on:

GitHub release
npm package (@rc dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


RFC: Persisting estimation cache in PostgreSQL database for Backstage users

Feature Suggestion
Backstage is currently the only integration listed in the Integrations section of the Cloud Carbon Footprint website. The supported database for Backstage is PostgreSQL.
Is it feasible to consider a solution where Cloud Carbon Footprint can leverage PostgreSQL for caching estimation data? This would remove the need to support two database instances when integrating the plugin in a Backstage deployment.
Describe the solution you'd like
Cloud Carbon Footprint to have an option to efficiently store the estimation cache data in a PostgreSQL database.
Additional context
I understand that due to the nature of the estimation data that the above scenario may be inefficient.
Would love to get some comment

Switching out MongoDB for PostgreSQL is certainly not a simple task. PostgreSQL does offer support for JSON data types (https://www.postgresql.org/docs/16/datatype-json.html), but I'm not familiar enough with them to determine if they can replace MongoDB's aggregation capabilities. That said, I don't believe PostgreSQL, as a relational database, would work as a suitable replacement due to the points Arik mentioned. We deal with flexible data which might not easily conform to a relational schema.

@mvaltas To clarify, it was my understanding that this would be an added option as an alternative to MongoDB, rather than a complete replacement. But I totally agree with your points

I think this is definitely worth spiking out. Having another integration option that is already more compatible with Backstage seems appropriate, but also considering @mvaltas' reservations, part of the decision might entail a trade of of efficiency or having limitations to how you might fetch and store estimates. @oconnormj, just curious, have you had a chance to explore how this might look without any major updates to the CCF data structure?


python311Packages.folium: 0.15.1 -> 0.16.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.folium is: Make beautiful maps with Leaflet.js & Python
meta.homepage for python311Packages.folium is: https://github.com/python-visualization/folium
meta.changelog for python311Packages.folium is: https://github.com/python-visualization/folium/blob/v0.16.0/CHANGES.txt
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.0 with grep in /nix/store/z1vxx11chplys79x5dzrgl2n4x0qy937-python3.11-folium-0.16.0
found 0.16.0 in filename of file in /nix/store/z1vxx11chplys79x5dzrgl2n4x0qy937-python3.11-folium-0.16.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.folium
python311Packages.osmnx
python312Packages.folium

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z1vxx11chplys79x5dzrgl2n4x0qy937-python3.11-folium-0.16.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.folium https://github.com/r-ryantm/nixpkgs/archive/5d244758663df234131dbafec7b2d072a4c9a18f.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5d244758663df234131dbafec7b2d072a4c9a18f#python311Packages.folium

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z1vxx11chplys79x5dzrgl2n4x0qy937-python3.11-folium-0.16.0
ls -la /nix/store/z1vxx11chplys79x5dzrgl2n4x0qy937-python3.11-folium-0.16.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages marked as broken and skipped:

python312Packages.folium
python312Packages.folium.dist

4 packages built:

python311Packages.folium
python311Packages.folium.dist
python311Packages.osmnx
python311Packages.osmnx.dist

Maintainer pings
cc @FRidh for testing.

Already on master, please open a new PR for the change.


Multiple build agents on same machine (did work with build agent 1.x)

Agent version and platform
Windows Server 2012 R2, agent version: '2.119.1
VSTS type and version
VisualStudio.com
What's not working?
Cannot configure multiple build agents on same machine
Description
When running config.cmd I'm getting a "Cannot configure the agent because it is already configured. To reconfigure the
agent, run 'config.cmd remove' or './config.sh remove' first." error.
I tried this by extracting the second agent to different folder from the first one. With build agent 1.x we had three agents running on same build machine. Is this not supported anymore?

Hi,
you can run multiple agents on the same mashine but you have to use a sepereate folder for each agent. Also the work folder must be seperate for each agent.
Greetings, KirK

After reboot I could configure second agent. No idea what changed.


Projection LSL

Adds constants for controlling projectors via ll*PrimitiveParams*().
Please use this PR for discussing the engineering of the constants.  Eg: whether they should be individual constants like this, or a glob set like most of the commands - or if you'd like to see the naming be different.

Indvidual constants or a glob?  Why not do both?  The glob would be more practical and efficient when changing multiple properties at once.  But individual constants would be more practical and efficient when changing individual properties.
For individual properties, they are even more practical and efficient due to removing the need to copy or store the other properties (done in order to preserve them).

Exactly my thoughts, just decided to hold of on it until I got at least one other opinion! :D

That's really a discussion for the other commit in the implementation (Halcyon) project, but it can almost go unstated that these should apply anywhere the other standard PRIM_* constants apply, and the same way, just different properties.

The values and names seem ideal to me. Merging.


hit the API

In this part of project I implemented:

receiving data from the API and parsing the JSON.
sending data to the API.
Use arrow functions instead of the function keyword.
Use async and await JavaScript features to consume the API.

License part added in Readme file


Isolate background jobs, only run with the server

That's a fix for
https://github.com/opencollective/opencollective/issues/1254

Coverage increased (+0.3%) to 71.631% when pulling 01836e0d898b0b6e5812189908b7eb7b99cd83f7 on fix/background-jobs into cbb57181210d0d26f0c28a21600e27830abc7344 on master.

I don't see the link with that issue.
You are basically just moving the process to update the cache for the complex queries for the homepage into its own background process file. How is that related to contributors.svg? What am I missing?

Ok I understand better with your comment on slack:

According to my inbox, the daily cronjob is not completing since July 17th. The last day we received the ‘Daily ledger health report’
But “Daily Subscription Report” is still there, so there is a problem in between.
Ok, what happened on the 17th of July? We got a deploy https://github.com/opencollective/opencollective-api/compare/19c0fb50...d05e2954 The highlight is:
  setInterval(refreshCache, 1000 * 60 * 60);

-> the setInterval is preventing the script to finish

I've removed the comment and I'll merge and deploy to make sure that the cron job doesn't fail tomorrow morning for the monthly reports.


Vm controller fix update and status update

What this PR does
The controller was not adjusted to /status subresource changes. This means we were trying to use Update even if the change was in Status and would be ultimately ignored.
Some of the changes (to the Status) to vm inside the sync were also ignored because we never passed the vm into updateStatus func. This is now fixed.
On top of these functional fixes, the last 2 commits address a few issues in unit tests and fixes false positive tests.

Fixes #
Why we need it and why it was done in this way
Special notes for your reviewer

Release note
NONE

/hold based on https://github.com/kubevirt/kubevirt/pull/11322

/cc

/test pull-kubevirt-unit-test
At least 2 tests seem flaky on my machine:
[FAILED] Unexpected call to *kubecli.MockVirtualMachineInstanceInterface.Create([context.Background 0xc003a9a000]) at /root/go/src/kubevirt.io/kubevirt/pkg/virt-controller/watch/vm.go:1199 because: there are no expected calls of the method "Create" for that receiver
[FAILED] Timed out after 1.000s.
  The function passed to Eventually failed at /root/go/src/kubevirt.io/kubevirt/pkg/virt-controller/watch/vm_test.go:6143 with:
  Expected
      <[]string | len:0, cap:0>: []
  to have length 1

/close
https://github.com/kubevirt/kubevirt/pull/11836


Add Hedvig Operator: 1.0.0-beta

Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[x] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you considered whether you want use semantic versioning order?
[x] Is your submission signed?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@ashenoy-hedvig can you have at logs in kiwi test (in Travis CI - Pull Request)?

@ashenoy-hedvig can you have at logs in kiwi test (in Travis CI - Pull Request)?

I've fixed the build failure. PR has been updated.


Fix can not createwith some protected exe

fix can not use createwith api to start some unnormal exe, like chinese online、commercial、prorected game dnf.exe https://dnf.qq.com/, but this exe can start run by explorer.

@bgianfo, I have rebased to only include this commit.

Thanks again @sonyps5201314 for all these fixes!

Thanks again @sonyps5201314 for all these fixes!


Enhancement

Detect if " > converts to " < and other encodings
as well as other double URL encodes
or UTF-7 payloads or something :D


Extract Promise utils into own library

The promise utils in this library (abort.ts, extPromise.ts, isPromiseLike.ts, toPromise) are useful, but it would be better if they were a standalone, documented library - specially since #6 will likely make them unused in @consento-org/crypto itself but keep them relevant for @consento-org/api or the mobile app.
Note: its perfectly for this to be an external repository.

Are utils like is-object needed to be implemented in Consento? We might be able use something that already exists. Should I maybe name it consento-utils?

Progress here: Does this look good so far? Should I abstract the buffer utils into there too? https://github.com/consento-org/promise-utils

It looks good for a start, how about the naming, do you think
"promise-helpers" or "promise-workflow" might be a better name? The buffer
methods also probably belong to another (new?) Repo? Do you have npm
publish rights?
On Mon, Nov 2, 2020, 08:16 RangerMauve<EMAIL_ADDRESS>wrote:

Progress here: Does this look good so far? Should I abstract the buffer
utils into there too? https://github.com/consento-org/promise-utils
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/consento-org/crypto/issues/9#issuecomment-720167381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG7FSWA6X6PHKN7TLRR2ZDSNXT57ANCNFSM4TFEDW2Q
.

I like promise-helpers as a name, promise-workflow seems kinda weird. Is there a particular thing about promise-utils that's confusing?
How do you feel about a general consento-utils repo that has all the utils in once place instead of spreading them out?

The thing about utils is coming from this train of thought (just picked randomly a few articles on the topic): https://dave.cheney.net/2019/01/08/avoid-package-names-like-base-util-or-common, https://www.adam-bien.com/roller/abien/entry/util_packages_are_evil, https://softwareengineering.stackexchange.com/questions/284799/semantically-more-appropriate-package-name-than-util-for-the-following-things - I tend to call those packages also lang instead of utils as to point out that these are packages that deal with the general language parts. Aside from this @consento/util suggests that these only apply to consento specific code while I still belive that promise-utils are general purpose, useful even outside of the consento context.

There's already a promise-utils package. :P
Would it maybe make sense to reuse some existing packages for all this stuff instead of adding a bunch of consento-maintained ones?

There are 13000 npm packages  of I am not sure which projects would be good hosts for the code. I obviously would love it, if that code pieces could be accepted by bluebirdjs - most well known. The of the 44 i found on abort-support none really seemed to follow a similar code metapher. Do you think there is a way how we could get that in a project that would enjoy that.
To make the naming conversation simpler: How about using @consento/promise for the time being?

Sounds good to me! 😁 Also @consento/isObject and @consento/exists and @consento/buffer?

I feel like the other utils might have an easier time to find a package that match.

I'm not on the consento NPM yet. :o My username on there is rangermauve. I guess we can figure out replacement libs for the other utils later.

@RangerMauve invitation sent.

Before I add the utils in, mind taking a look at the main branch? I replaced the isObject and exists dependencies with https://github.com/sindresorhus/is

In typescripts strict mode there is a clear difference between objects that may be null and that may not. exists is used to turn an optional typed null or undefined object into a defined one.
function foo (bar?: string): string {
  return bar + 'x' // Error because bar could be null/undefined
}
function baz (bar?: string): string {
  return exists(bar) ? bar + 'x' : 'x' // Works because exists has the "is" operator in the declaration
}
function bazBlock (bar?: string): string {
  if (!exists(bar)) {
    return 'x'
  }
  // after this typescript is SURE that bar can not be null/undefined
  return bar + 'x'
}

What is a good shortcut in is that does the same thing? is.existing?

Cool, I'll extract existing into its own thing, then. 😁

Actually, does the is.nullOrUndefined method serve this purpose?
I currently have something like const exists = (value) => !is.nullOrUndefined(value)

Published as https://github.com/consento-org/promisewith documentation here

Published as https://github.com/consento-org/promisewith documentation here


Project documentation

For docs: fix readme #6 @jkowalleck
Corrected mistakes in the README
Corrected the usage statement

@ptdropper i'd suggest you revert the changes to the python file and stick to the documentation for now,
as this PullRequest's purpose seams to be documentation.
"fixing" the python makes no sense at this point, or does it?

Will fix the python code in a separate specific pull request. This PR is not intended to edit the python file.


changing physical layout to support go install

no it is impossible to install via go install.
recommend changing physical layout to make it possible. that would greatly help adoption.
option A) move to /cmd/goenums/goenums.go and keep main module a "library module"
option B) move to /goenums.go and make whole module a "main module"
I recommend option B).

For option B, I believe to install goenums it will use this command.
go install github.com/zarldev/goenums

How option A would look like when installing?

Thanks, done and implemented. Added to README.md


M3-2772 - Upgrade wdio framework and storybook tests

Fixed duplicate commits.
Updated automated tests to run with lerna commands so that we can still use yarn "xyz" command to run testing scripts
Tests still need to be updated to run with V5 of webdriver
Updated Breadcrumb and AccessPanel tests
Copied over previous V5 fixes
Updated AccessPanel formatting
Updated ActionMenu story
All tests are passing
Updated CircleProgress
All tests are passing
Updated Checkbox test
Updated  ConfirmationDialog test
Updated DebouncedSearchTextField tests
Updated Drawer test
Updated EditableText test
Updated EnhancedSelect test
Updated EntityIcon test
Updated ErrorState test
Updated ExpansionPanel test
Updated HelpIcon test
Updated IconLinkText
Upgraded LinearProgress test
Updated Notice test
Updated PaginationControls test
Updated PasswordInput test
Updated Radio test
Description
Please start the pull request title with the jira ticket corresponding to the work if applicable. Please include a short summary of the feature added, the change, or issue fixed.
Type of Change

Breaking change ('break', 'deprecate')

If the any above types of change apply to this pull request, please ensure to include one of the listed keywords in the pull request title.
Applicable E2E Tests
To run relevant E2E tests, run these commands in 3 separate terminals:

yarn && yarn start
yarn selenium
yarn e2e

Note to Reviewers
Running these tests locally should take less than 2 minutes to run. This upgrades the webdriver io framework, selenium jar, chromedriver, and storybook tests.
These can now be placed back in the pre-commit hook.

Opened a new branch to resolve dupe commits, this one is no longer needed.


#179 - Search people: suggestion user list should be implemented - Implemented

https://github.com/Microsoft/EmbeddedSocial-iOS-SDK/issues/179 implemented

 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.:x: Vadim-Bulavin sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Search files to exclude doesn't work in a multi-root context

VSCode Version: Code - Insiders 1.14.0-insider (15b579d13dbf6817bb493feff6f36cec87577929, 2017-06-27T05:11:33.154Z)
OS Version: Linux x64 4.10.0-24-generic
Extensions:

Extension
Author (truncated)
Version

EditorConfig
Edi
0.9.3

lorem-ipsum
Tyr
1.0.0

sort-lines
Tyr
1.3.0

vscode-svgviewer
css
1.4.2

tslint
eg2
0.15.0

git-project-manager
fel
1.3.2

md-navigate
jri
0.0.1

vscode-scss
mrm
0.6.2

seti-icons
qin
0.1.3

vim
vsc
0.9.0

(1 theme extensions excluded)

#29510

Were any of those files open? I can repo that but not in a general case.

@roblourens I can't remember but I would say so if you can't repro the other case.


Unexpected error with pipelines on Bitbucket

I have an unexpected problem with pipelines on BitBucket. I am getting this error:
ERROR in vendor.js from UglifyJs
Unexpected token: name (chatsBox) [vendor.js:26881,16]
All branches have built successfully. However, when I want to build a pipeline for this one, I got that error.
This is the JavaScript code:
  if (~window.location.pathname.indexOf('/')) {
    let chatsBox = $('.conversations');
    var conversation = [].slice.apply(chatsBox);
    (function chatFeed(index) {

        var divs = Array.from(conversation[index].children);
        var count = 0;

        var cTimer = setInterval(function () {
            if (count === 4) {
                chatFeed(index === 3 ? 0 : index + 1);
                clearInterval(cTimer);
            }
            divs.forEach((e, idx) => {
                e.style.opacity = count >= idx && count !== 4 ? '1' : '0';
            });
            count++;
        }, 1500);
    })(0);
}

On windows everything is alright. However, due to the pipelines built via the Linux server, this script or just that variable is causing a problem.
Does anyone know why and how can I solve this?

I'm sorry, I really don't understand the issue you're describing.
Are you sure you're in the right repo? I see your code is using jQuery to select and loop through a set of divs and change their opacity. This is not really something you want to do in a React project.


Fix code lens in full document synchronization mode

Purpose
Fixes #15375.
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[x] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #15376 into master will decrease coverage by 0.96%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master   #15376      +/-   ##
============================================
- Coverage     22.97%   22.01%   -0.97%
- Complexity     5116     6159    +1043
============================================
  Files          1346     1806     +460
  Lines         69147    84721   +15574
  Branches       9371    12192    +2821
============================================
+ Hits          15889    18651    +2762
- Misses        51483    63038   +11555
- Partials       1775     3032    +1257

Impacted Files
Coverage Δ
Complexity Δ

...alang/langserver/BallerinaTextDocumentService.java
66.9% <0%> (-0.24%)
36 <0> (ø)

...va/org/ballerinalang/stdlib/task/service/Init.java
52.63% <0%> (-47.37%)
8% <0%> (ø)

...ang/stdlib/task/objects/ServiceWithParameters.java
53.57% <0%> (-46.43%)
2% <0%> (-3%)

...lang/stdlib/task/utils/ResourceFunctionHolder.java
40% <0%> (-45.72%)
2% <0%> (ø)

...rg/ballerinalang/stdlib/task/service/Register.java
60% <0%> (-40%)
3% <0%> (ø)

.../ballerinalang/stdlib/task/utils/TaskExecutor.java
45.83% <0%> (-38.79%)
3% <0%> (ø)

...ava/org/ballerinalang/stdlib/task/utils/Utils.java
43.83% <0%> (-38.22%)
14% <0%> (ø)

...lang/stdlib/task/impl/TaskServerConnectorImpl.java
64.28% <0%> (-35.72%)
2% <0%> (-1%)

...a/org/ballerinalang/stdlib/task/service/Start.java
42.1% <0%> (-30.63%)
3% <0%> (ø)

...va/org/ballerinalang/stdlib/task/service/Stop.java
42.1% <0%> (-30.63%)
3% <0%> (ø)

... and 480 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 a20131b...37522d0. Read the comment docs.


Troube installing v<IP_ADDRESS>

Hi,
I'm using Haskell 8.0.2 and I'm facing this on cabal install postgrest.cabal
Preprocessing library postgrest-<IP_ADDRESS>...
[ 3 of 14] Compiling PostgREST.Types  ( src/PostgREST/Types.hs, dist/build/PostgREST/Types.o )
[ 4 of 14] Compiling PostgREST.Parsers ( src/PostgREST/Parsers.hs, dist/build/PostgREST/Parsers.o )
[ 5 of 14] Compiling PostgREST.Error  ( src/PostgREST/Error.hs, dist/build/PostgREST/Error.o )
[ 6 of 14] Compiling PostgREST.DbStructure ( src/PostgREST/DbStructure.hs, dist/build/PostgREST/DbStructure.o )
[ 7 of 14] Compiling PostgREST.Auth   ( src/PostgREST/Auth.hs, dist/build/PostgREST/Auth.o )

src/PostgREST/Auth.hs:54:9: error:
    Variable not in scope:
      validateJWSJWT
        :: ((StringOrURI -> Bool) -> JWTValidationSettings)
           -> JWK -> a1 -> ExceptT JWTError IO a0

src/PostgREST/Auth.hs:58:47: error:
    • Variable not in scope: jwtClaimsSet :: a1 -> ClaimsSet
    • Perhaps you meant ‘jwtClaims’ (line 46)

src/PostgREST/Auth.hs:92:24: error:
    • Couldn't match expected type ‘Base64Octets -> OctKeyParameters’
                  with actual type ‘OctKeyParameters’
    • The function ‘OctKeyParameters’ is applied to two arguments,
      but its type ‘Base64Octets -> OctKeyParameters’ has only one
      In the first argument of ‘OctKeyMaterial’, namely
        ‘(OctKeyParameters Oct (Base64Octets key))’
      In the expression:
        OctKeyMaterial (OctKeyParameters Oct (Base64Octets key))

src/PostgREST/Auth.hs:92:41: error:
    Data constructor not in scope: Oct :: Base64Octets
cabal: Leaving directory '.'
cabal: Error: some packages failed to install:
postgrest-<IP_ADDRESS> failed during the building phase. The exception was:
ExitFailure 1

Has somebody face this problem ?
Regards,

It looks as if you're using the wrong version of the jose library or something. We're currently using package versions pinned at Stackage's LTS-9.6, so compare the package versions there with what cabal has installed locally.
If it turns out that this problem was a dependency version thing I'll tighten up the constraints in our cabal file. Even though we rely on Stackage snapshots for building, I want to ensure it also builds cleanly for vanilla Cabal.

@begriffs so quick ^^
In fact, I'm (still) building binaries to distribute into debian community, and for this I'm using the official cabal way and stackage is not supported ;-)
Are binaries (hosted on https://github.com per release built by stackage ?)

Stackage doesn't build any binaries for us, it just has a server where they test the compatibility of package versions with one another, and publish "snapshots," each of which is recommendations for a set of dependency versions that appear to work properly together.
We make binaries with a continuous integration script that triggers for tagged versions in git. And I personally build it for a few extra platforms like FreeBSD that we haven't automated yet.

Try running cabal freeze and paste the output it writes in cabal.config and we can see what dependency versions it is trying to use.

sorry, I have removed my answer since I am not sure my affirmation is still actual

@begriffs The result of cabal freeze is https://git.io/vbqol
It seems to be a problem with the JWT lib (I think)

OK, I tried building the master branch on a fresh Debian 9.3 machine and it worked for me. This is what I did:
sudo apt-get update
sudo apt-get install ghc cabal-install git libpq-dev libgmp-dev happy zlib1g-dev

# ghc --version
# The Glorious Glasgow Haskell Compilation System, version 8.0.1

# cabal --version
# cabal-install version <IP_ADDRESS>
# compiled using version <IP_ADDRESS> of the Cabal library

git clone https://github.com/begriffs/postgrest.git
cd postgrest

# git rev-parse HEAD
# 70ce1b9329642f4a216826df045844dd2adbc7d8

cabal update
cabal install --dependencies-only

cabal build

Could you see if the build problem has gone away for you with postgrest 0.4.4?

New releases will be buildable with cabal, pinned package versions were missing previously(like jose) which caused some errors. Made this change in https://github.com/PostgREST/postgrest/pull/1177.


Fixed: Reactivating errcheck for tests

This PR removes nolint and reactivates errcheck for tests in datapath and cgnetcls
https://github.com/aporeto-inc/trireme/issues/385

Codecov Report

Merging #387 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #387   +/-   ##
=======================================
  Coverage   55.47%   55.47%
=======================================
  Files          51       51
  Lines        6268     6268
=======================================
  Hits         3477     3477
  Misses       2468     2468
  Partials      323      323

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 a759c1b...76ad9c5. Read the comment docs.


lakectl: support an ignore file for recursive uploads

Allow reading a line-delimited list of file names or patterns that will be skipped by recursive uploads. This should behave similarly to Git (and possibly even allow using .gitignore directly?)

Workarounds can be to use other clients:

If using RClone, you can use its extensive filtering capabilities.
aws s3 cp supports some simpler exclude patterns.


multi module project docs path not found

For a multi module project, run the above command to generate the docs into a sub module. You need to copy to an external module to access it
====== no =======
demo
　├─srv1
　│ 　 └─main
　│　　　　└─docs
　└─srv2
　　　└─main
====== ok =======
demo
　├─srv1
　│  　└─main
　├─srv2
　│  　└─main
　└─docs

you can try run the swag init command in the demo directory


[Observ] Alert channel support Voice/Aliyun

Added in this commit https://github.com/kubegems/dashboard/commit/7140d29bdcd7650aa9c68e3b71607f111d1c6ebe


RFC: use github repo name

Is it a deliberate choice to display [github user]/[Library name] instead of [github user]/[github repo]  on the tiles?
I think this is slightly confusing and missleading. For example, I think the following should say leanprover-community/NNG4 somewhere. What would happen if leanprover-community had two repos defining a lean-lib called Game?

Maybe leanprover-community/NNG4 : Game could be a better title?
(ofc one can argue that all games should just be excluded because they aren't packages in the sense that you would ever require them, but still the question about unique library names stands regardless)

@joneugster It is a deliberate choice to use the package name defined in Lake for the package versus the one used on GitHub. As it currently stands, Lake cannot require two packages with the same name, so having two packages by a single owner with the same name would be just as incompatible for Lake as it is for Reservoir.

I see your point, and I guess the NNG (and any other game) should actually be excluded on reservoir anyways because it is supposed to be a fork of https://github.com/hhu-adam/GameSkeleton
(and its just a technical/historic reason that github doesn't realise that)


Major version when dropping support for Logback version

Describe the bug
We've noticed that you dropped Support for Logback 1.2 in version 7.4 .
This caused our Dependency Update Bot to propose change for 7.4 (Minor version). Can you make this a 8.0 instead of 7.4 instead?
Thanks
To Reproduce
N/A
Expected behavior
N/A
Additional context
Spring Boot 2.7

Unfortunately, I cannot un-release something.  In the future I'll bump the major version when dropping support for logback versions


Improve padding and attention mask handling

This PR includes several closely-coupled changes:

The major change is the introduction of the new PaddingMask and AttentionMask types that replace all uses of seq_lens and tensor-based padding_mask and self_attn_mask parameters. This is also the main reason why this PR touches many files since those parameters are used in many places in the code base. The main benefit of these two new types is the lazy initialization of masks and not materializing them at all whenever possible (e.g. when a global causal attention mask is used with PyTorch SDPA or xformers, we entirely skip constructing the mask). This can significantly improve the performance in the right settings and also reduces memory use (again, sometimes significantly when used with very large context lengths).
MultiheadAttention also needed to be refactored. Mask handling is moved down to SDPA implementations since each SDPA has varying support for different mask types. As part of this work add_bias_kv and add_zero_attn are also removed. Initially we introduced them to have feature parity with fairseq MHA, but turns out that these two parameters are not used anywhere. We can consider introducing them in the future again if requested, but they add needless complexity to mask handling and both parameters are superseded quite a while ago by superior techniques (e.g. pre-LN).
Various nit updates at several places in the code base (like renaming some local variables, updating docstrs) mainly related to the first two points.
Fixes the causal attention mask to support bfloat16.

Tested with LLaMA 2 and SeamlessM4T models in various settings and verified bitwise output parity.

Manually verified clang-tidy check. GitHub runner is (again) out of disk space.


Adding matching brackets exercise (see Elixir track)

I have implemented the matching brackets exercise that I did in the Elixir track.
Perhaps we can include it?

@ErikSchierboom -- just got around to seeing this update! too many updates from too many places! so on it this weekend!

Sure! No rush

i was going to get this PR merged and then start on creating stubs .. is this one ready?
happy to do what is next/needed!

Thanks!


Translate Part 6 into French

Would love to keep going by translating part 5 and 6

@Nicodaube Please strive to make sure you have a separate pull request for each issue. Smaller pull requests is the goal, it makes it easier to review and it a common best practice.


Don't cancel selectstart event on textarea

In our web-app in chrome and IE11 Ctrl+A didn't work on a textarea what was a drop-zone.

Thanks for the contribution! Released in 2.5.1.


[G000550] Rep. Phil Gingrey

Rep. Phil Gingrey:
http://gingrey.house.gov/
Likely contact form URL to start with:
http://www.house.gov/formgingrey/IMA/issue.htm

YMAL file not found. I am working on it.


[BrokenLinksH2] Fixing broken link

Docs Build status updates of commit 49aac6e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

biztalk/core/sb-messaging-adapter.md
:warning:Warning
View
Details

biztalk/core/sb-messaging-adapter.md

Line 59, Column 5: [Warning: file-not-found - See documentation] Invalid file link: 'powershell/module/servicemanagement/azure.service/new-azuresbnamespace'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 40d9f9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/sb-messaging-adapter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Sankey diagram gets reordered on resize/legend change/map roam

I have a sankey diagram in a map like this:

nodes and links are sorted on load, however, when window is resized, legend changed or map roamed, it changes into this:

You can reproduce this issue by resizing chart view in this example: http://echarts.baidu.com/demo.html#sankey-energy
Version: 3.1.10

Hi @jarben, I have fixed this problem, which will be released next version. Now nodes sort by the order in series.data and will not be changed when you resize the diagram. Thanks for your suggestions!

If there any problem after you try the new version, please reopen this issue.

Hi @deqingli , this is great news! Thanks for looking at it! Will test once the release is out:)
Thanks,
Jaro

Hi @deqingli , is it possible to use zoom-in and zoom-out options in toolbox icon view, instead of using in world map  , series --> roam with scaleLimit.
Thanks,
chandru


Search is broken

To reproduce:

Clone master
Try using the top-bar search box

Temp fixed by reverting gem in https://github.com/tactilenews/100eyes/pull/386


[FEATURE REQ]Provide a way to know whether a container has hierarchical namespace enabled when using token credentials

Is your feature request related to a problem? Please describe.
This is a revival attempt of the reported issue https://github.com/Azure/azure-sdk-for-java/issues/26733
Describe the solution you'd like
I'm looking for a way to identify whether the storage account to which the container belongs has hierarchical namespace enabled.
I know about the limitations exposed by @rickle-msft  here https://github.com/Azure/azure-sdk-for-java/issues/26733#issuecomment-1026046740
It would be extremely helpful to know whether we  can use on a hierarchical container DataLakeFileSystemClient /  Data Lake Storage Gen2  techniques or rather or BlobContainerClient/ ABFS on a flat container for listing / deleting its directories and files.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[ ] Expected solution specified

Thanks for filing this issue @findinpath.
I'll start by assuming when you want to make this check you already have a BlobContainerClient, or BlobContainerAsyncClient (and I'm going to write the following answer using the synchronous client but the same / equivalent APIs should exist in the async client).
You can get the BlobServiceClient using BlobContainerClient.getServiceClient(). The BlobServiceClient should then have a method getAccountInfo returning a StorageAccountInfo object. This has an is getter isHierarchicalNamespaceEnabled which will indicate if the Storage account being used has hierarchical namespace enabled.
boolean isHierarchicalNamespaceAccount = blobContainerClient.getServiceClient()
    .getAccountInfo()
    ,isHierarchicalNamespaceEnabled();

Hmm, this one I don't have a good answer for, @seanmcc-msft do you have any additional information on how to use this without a connection string?

Workaround
az storage account blob-service-properties show --account-name my-storage-account
{
.....
  "resourceGroup": "my-resource-group",
....
  "type": "Microsoft.Storage/storageAccounts/blobServices"
}

az resource show --name my-storage-account --resource-group my-resource-group  --resource-type "Microsoft.Storage/storageAccounts"  --query properties.isHnsEnabled

This will show whether HNS is enabled or not for the storage account while using az tool.
Could you please add to the class https://github.com/Azure/azure-sdk-for-java/blob/eee5dc85bf53571289ee1cb5e3a0d0e6bf06df9c/sdk/storage/azure-storage-blob/src/main/java/com/azure/storage/blob/models/BlobServiceProperties.java also the field resourceGroup ?
I'm looking as well whether I have what I need for az resource show in azure-sdk-for-java.

With the above workaround aside, it feels though unnatural to go the above mentioned route just to know whether the container i'm dealing with is hierarchical or not and this only when using service tokens. :(

We found an alternate method that seems to work. It calls this API endpoint directly using the com.azure.core.http.HttpClient. If HNS is enabled for the account then it will return a 401 response status since we aren't authenticated, if HNS is not enabled then it will return a 400 response status that says it is an invalid URI. The upn query parameter is only valid for HNS account types.
private boolean isHierarchicalNamespaceEnabled(String accountName, String containerName)
        throws IOException
{
    URL url = UrlBuilder.parse("https://%s.dfs.core.windows.net/".formatted(accountName))
            .setPath(containerName)
            .addQueryParameter("action", "getStatus")
            .addQueryParameter("upn", "true")
            .toUrl();
    HttpRequest request = new HttpRequest(HttpMethod.HEAD, url);
    try (HttpResponse response = httpClient.send(request).block()) {
        return Optional.ofNullable(response)
                .map(HttpResponse::getStatusCode)
                .map(statusCode -> Objects.equals(statusCode, 400))
                .orElseThrow();
    }
}

Yet, another (maybe more elegant way)
        ClientSecretCredential credential = new ClientSecretCredentialBuilder()
                .authorityHost(authorityHost)
                .tenantId(tenantId)
                .clientId(clientId)
                .clientSecret(secret)
                .build();

        DataLakeDirectoryClient dataLakeDirectoryClient = client.getFileSystemClient(azureLocation.container().orElseThrow()).getDirectoryClient("/");
        boolean isHnsEnabled = dataLakeDirectoryClient.exists();

I'd like somebody from MSFT to confirm whether this solution should be correct in the long run.

@alzimmermsft @seanmcc-msft Just want to check in here, can either of you confirm if any of the suggested solutions above will be correct in the long run?
I would also like to know if you plan to add support for OAuth based access token authentication to the "Get Account Information" endpoint?

/unresolve
It'd still be helpful to answer https://github.com/Azure/azure-sdk-for-java/issues/38912#issuecomment-1972481900

/unresolve


Added a health endpoint

Changed a test_routes.py to add test cases, changed routes.py to add health check

Looks good to me.


Disappearing pseudo-nodes leave a hanging edge

Internet node was present, but then disappeared. The edge to it stayed. Attached report, canvas svg, and /api/topology/containers output, but I doubt they'll be helpful in this case, as when I refreshed the edge went away.

report_missing_node.zip

I have also seen this happen.

Dupe of #1304