 Use event.path rather than event.target to get the actual element in which the event occured.
This pull request fixes #245.

Also I just checked and the second test now no longer runs via phantom js
TypeError: 'undefined' is not a function (evaluating 'shadow.createShadowRoot()')

Is this just a matter of updating mocha/phantomjs or is this a known thing?

That's a bit weird. I was sure I had that test running fine in both browser and phantom.
I know the polymer guys use mocha for testing, so I guess it'll work if just the test libs are upgraded. I'll look into it.

What currently is preventing this from being fixed?

It would be amazing if this could be either merged or the underlying issue fixed.
I tried monkey-patching mousetrap.js by adding the following just before the final return statement in Mousetrap.prototype.stopCallback:
// Events originating from a shadow DOM are re-targetted; thus, `e.target` is not always
// the initial event source. If the first element in the event’s composed path is not
// the target, it has a different initial event source than `element`.
if (e.hasOwnProperty('composedPath')) {
    var initialEventTarget = e.composedPath()[0];
    if (initialEventTarget !== e.target) {
        element = initialEventTarget;
    }
}

Is this just a matter of updating mocha/phantomjs or is this a known thing?

@ccampbell I’m trying to write a test for my solution posted above and I get same error:
TypeError: 'undefined' is not a function (evaluating 'shadowHost.attachShadow({ mode: 'open' })')

Mocha version 1.9.0 is used by these tests. That version is very outdated and doesn’t know about Element.prototype.attachShadow (or the createShadowRoot() method from above). My test is passing when opening the tests/mousetrap.html file.

This issue was addressed in #445 and can be closed.


[💡 FEATURE REQUEST]: Ability to set AMQP behavior/management params

Plugin
JOBS
I have an idea!
I trying to listen for AMQP queue and faced following problems:

Is it possible to set AMQP consumer name?
There are many consumers and it would be great to set the name of consumer, e.g. <service-name>-<host name>
AMQP plugin is always trying to create exchange/queue
Usually queue workers doesn't have access to modify RabbitMQ exchange/queue schema by security reasons.
Is there any way to prevent a queue worker from creating an exchange/queue and have it just listen?

Hey @codercms 👋🏻

Not at the moment, but adding a configuration option is easy. I'll add the new configuration options.
Not create, but check if it exists (and only if not - create). This is an idempotent operation (if the exchange exists) to check that configuration fits the exchange configuration. In the latest version, if you do not consume from the queue, RR will not try to check the queue and create it if it doesn't exist. But if you did consume, RR has to check the queue to be aligned with that declared in the configuration. This is also an idempotent operation.


minor code tweak

Since we've moved the INDEX_TABLE to the individual citydb schemas for PostgreSQL (cf. https://github.com/3dcitydb/3dcitydb/commit/5911550054466618ecdb069d46d1395da00e8ce0) , we may need to adjust the following code-line by adding a schema name parameter.
https://github.com/3dcitydb/importer-exporter/blob/788f142ff80f04bc486b4a963d8a95f01b8919ce/impexp-core/src/main/java/org/citydb/database/adapter/postgis/UtilAdapter.java#L281

fixed in e3c1ce6152d8594793b0f1616968629ec9003f62


Create dependency-review.yml

Codecov Report
Patch and project coverage have no change.

Comparison is base (8aa0173) 100.00% compared to head (c2e3a5b) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          255       255
  Branches        55        55
=========================================
  Hits           255       255

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


WIP: Module factory

Related issues
closes https://github.com/DivanteLtd/vue-storefront/issues/2418
closes https://github.com/DivanteLtd/vue-storefront/issues/2406
TODO:

[ ] refactor other modules
[ ] move list of registered modules to factory

yup

ok just docs to be updated. @patzick you can review the code

@patzick changelog and uprgade notes ready ;)


Encyclopedia: description field not visible without inserted text

Hi!
I started a project in the current android app (1.0.7) and added a Encyclopida entry (person). I did not fill out the description field, but I wanted to do this later. I wanted to create all the persons, before going into depth and describing them. When reopening the person entry I was not able to find and edit the field for the description. I had to make a new entry with the description field filled out to see where I have to click.
It would be great if there was an inline text of the empty field saying something like 'no description yet'.
Thank you!
Kind regards!

Fixed! Will be in the next release


Pgbouncer version: Azure Cosmos DB for PostgreSQL. Prepared statement error

I am running into issues regarding the pgbouncer connection pooling in Azure Cosmos DB for PostgreSQL:
https://learn.microsoft.com/en-us/azure/cosmos-db/postgresql/concepts-connection-pool.
The issue is that I get the error: "prepared statement "sqlx_s_1" already exists". I'm wondering if there is something we could do to solve that problem?
Seems like it might have been fixed with pgbouncer version 1.21.0, and I can't find any information about the pgbouncer version currently in use in the Azure Cosmos DB for PostgreSQL.

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

ID: 0be8f4f0-4c94-5301-6cc7-a973c1f90694
Version Independent ID: 6d8ec8d2-347e-049a-8afd-497bb7076f03
Content: Connection pooling – Azure Cosmos DB for PostgreSQL
Content Source: articles/cosmos-db/postgresql/concepts-connection-pool.md
Service: cosmos-db
Sub-service: postgresql
GitHub Login: @jonels-msft
Microsoft Alias: jonels

@pettermoe95
Thanks for your feedback! We will investigate and update as appropriate.

Thank you for reaching out,
Unfortunately, this is the documentation GitHub repository, and we cannot troubleshoot issues, answer product questions, or provide technical support.
For technical support, please open a support ticket in the Azure portal. For questions about the product, please post them in Microsoft Q&A.
#please-close

One suggestion to the documentation would be to add the pgbouncer version, or maybe that also is more of a product issue and should be visible in the Azure portal instead? Anyways, thanks, I will make an issue from the Azure portal.

Thank you for reaching out,
Unfortunately, this is the documentation GitHub repository, and we cannot troubleshoot issues, answer product questions, or provide technical support.
For technical support, please open a support ticket in the Azure portal. For questions about the product, please post them in Microsoft Q&A.
#please-close

Agreed with @pettermoe95, the pgbouncer version is documented for postgres flexible tier, but it's nowhere to be found for the CosmoDB for Postgres offering. Which leads to unknowns about what is supported.

@niklarin, can your team look at this request?

Currently PgBouncer version in Azure Cosmos DB for PostgreSQL is announced only in the product updates. Specifically, currently it's PgBouncer 1.20. To support prepared statements, we'd need to upgrade PgBouncer to the current version. Unfortunately, some engineering work would be required to do so. This engineering work is not currently planned (at least not for the next ~6 months).
https://learn.microsoft.com/en-us/azure/cosmos-db/postgresql/product-updates#august-2023

Currently PgBouncer version in Azure Cosmos DB for PostgreSQL is announced only in the product updates. Specifically, currently it's PgBouncer 1.20. To support prepared statements, we'd need to upgrade PgBouncer to the current version. Unfortunately, some engineering work would be required to do so. This engineering work is not currently planned (at least not for the next ~6 months).
https://learn.microsoft.com/en-us/azure/cosmos-db/postgresql/product-updates#august-2023

Ah that's where the version was hidden. Honestly the flexible tier documentation is way clearer in that regard.
It's a shame that the pgbouncer upgrade isn't planned in the short term. We host a pretty big cosmodb Postgres cluster and the lack of pgbouncer prepared statement support slows us down considerably.

@Aaron-ML sorry about the inconvenience. Unfortunately, at this point the only workaround for your Azure Cosmos DB for PostgreSQL cluster is to put a self-hosted PgBouncer in front of it.


remove uninitialized git submodule

Description
Remove uninitialized Git submodule.
My feeling is that this submodule was just added manually instead of running git submodule add https://github.com/kubewarden/docs/blob/main/.gitmodules themes/projects-theme
The problem that I am facing is while working on an update policy for this repository, Updatecli fails cloning this repository because it doesn't know how to clone the submodule
While running command such as git submodule status, I see nothing neither so 🤷🏾

Test
Current theme shouldn't be affected by removing this file so the preview environment should look like exactly kubewarden.
Additional Information
Tradeoff

Potential improvement

Thanks for the quick review


Contact LA City to see if we can get newer data

I was thinking that since the City of Los Angeles has a more active Open Data initiative than the county does, maybe we could get access to the newer building footprints and/or addresses for the city only? Even if we still had to use older data for the rest of the county, this would be a huge improvement. @cityhubla, remind me, have we discussed this already?

We last talked about reaching out to the city to promote the effort to gain interest. The new dataset is part of the collaborative acquisition program called LARIAC. We could encourage LA City to push their building footprint to data.lacity.org by highlighting the efforts of other cities, (SF, NY, CHI) who have their footprints accessible online and the opportunities they have created. LA could further provide in the words of Mayor Eric Garcetti, "a vast array of useful information to improve life in our city." I think a letter of request furthered by a concerted effort on the part of the OSM community and interested parties can encourage the city to add it, this short of requesting the data by way of the CPRA, (California Public Records Act).

The city open data has a request form, we can try sending it there, I'll craft the message and run it by you, @almccon

@almccon, I made a draft to request the data from DataLA, if you want to take a look, Draft. I'm crafting it to bring excitement to the effort and get some contributors. I'm piecing it from these collaborative efforts, New York City and OpenStreetMap Collaborating Through Open Data. This effort with the city can include address points but maybe have something where lacity can detect changes in OSM to drive updates to their public datasets, like the changewithin tool. We can use this language to spread the word.

@LearningNerd should join in here to bring in support from the academic crowd.

@cityhubla Very interesting! I'll share with my team. Our focus isn't on map data right now but we are interested in more examples of how open data initiatives are developing in the cities of LA County.

Thanks, @cityhubla, that's great! Can you enable commenting on that document? Right now it's in read only mode.
I'd also like to loop in @talllguy for his experience in Baltimore. We can point to this article http://technical.ly/baltimore/2015/01/21/elliott-plack-putting-baltimore-openstreetmap-help-out/ to show the nice publicity that DataLA could get by sharing their data.

Thanks for the mention @almccon. I think that it was good to get some press about the import--I heard from a few non-OSM people about the article, "Hey I read this article about you..." types of things. Therefore, in my experience I think that promotion via the media is especially valuable for these kinds of things because it made Baltimore City look good (and it often does not look good), and gave some exposure to OSM.
To prepare I did a little hangout with the OSM.us board to prep for the interview, to make sure I hit some key aspects of the OSM project.
Anyway, regarding getting new data and open data, I think you could show the Baltimore City import, which is nearing completion, the Baltimore County import, or the NYC import as a beacon of how government data can make for an excellent map with a lot of valuable information. For the city of LA, I would see if there is a Chief Data Officer, or a senior GIS person that is open data friendly, to try and get at the latest data. In both of the imports I worked on, the permission didn't have to come all the way from the top, so finding an advocate in the department that dealt with this kind of data was critical.
Let me know if you need more elaboration.

@almccon, it should be editable now, and thanks @LearningNerd, having other LA county cities be aware of OSM can promote the benefits of opendata. And awesome tips, @talllguy, LA does have a Chief Data Officer and a Chief Innovation & Technology Officer we can reach out to.

@cityhubla I'm adding our names and affiliations (please add your own) to the end, to personalize it and add credibility. I got Jon Christensen, who is a UCLA professor and a Stamen partner, to sign too, if we think that's helpful. He's very good at diplomacy and he'd love to help us build contacts with city or county government in any way he can. Who else would be useful to have sign on? Any other allies we have who also represent LA-area entities? Maybe @palewire?

@almccon Awesome, I'm getting approval for my affiliation, I'm trying to get backing from my office. I also reached out to @jschleuss from the LATimes for their thoughts.
I made some changes to the draft, but I came across this, [DataLA Policy and Playbook]{http://datala.github.io/od-policy//roadmap/}, its a breakdown of the city's efforts, more importantly there is a roadmap which has the building footprints listed, making the letter more important. The GITHUB author is the Chief Data Officer.

Nice find re: @datala! That's a great thing we can point at. The draft changes look good too. The letter is getting better and better.

Looks like we'll be getting L.A. City data soon. We filed a public records request and the Times will foot the small labor bill to get the shapefiles of buildings. The newest they have is from 2012. Where should I put the file once I get it?

Great news, @jschleuss! You can put the file wherever you like and I can modify the Makefile to download it from there. But if you need somewhere to store it, then @iandees can hook you up with s3 credentials and you can put it here s3://data.openstreetmap.us/imports/la/chunks/

@jschleuss will this include a file of address points too, or will we still need to use addresses from 2008?

Thanks @jschleuss. @almccon, if we are unable to get them, LA County updates their points weekly, LA County Address Points, we can also link to the new 2014 Assessors data, for building uses, #bedrooms and units. Mind you we need to figure out a way to have contributors make sure to do some quality check on if buildings have been built after 2012.

Thanks @cityhubla! I hadn't checked the recentness of the address data. It looks like I had a link for 2012 addresses stored in the Makefile. Looks like your URL for addresses says 2014 at the top of the page, but gives a reference date of 2012 near the bottomt of the page? Your link is a geodatabase, which is interesting.
That's also interesting to see the parcels link. I hadn't noticed that before. Previously I was using data from a UCLA link for 2010 apparently.

And yup, definitely we want to make sure the manual importers check for any newer buildings. That will be one of the most important things they will have to look for. And if possible, they should try to add the correct address to any new buildings that they trace.

I'm not sure what's going to be attached to the shapefile. I asked. Here's the response

@almccon @cityhubla Here are the fields we'll get from the 2012 city shapes: Code, BLD_ID, Height, Elevation, Shape_Area, Shape_Length.
I'm pretty sure we could match some of the IDs with other datasets already released through the portal.

Alright gang. Here is the 2012 building file. I haven't done a difference analysis to see how much this differs from the LA County 2008 file. My initial check doesn't look super promising: I think this could just be an extract from the county's 2008 file. :( Also, we're limited on attributes. BLD_ID is in there, and it appears to match the county's 2008 file.

I did a very quick look and spotted at least one difference... near the corner of Beverly Blvd and N Robertson Blvd:

@almccon, those buildings were built during the 30's and 50's, I found another one by Queens Rd and Sunset Blvd at the Sunset Strip area.

Seems to me that they must have added/modified the 2008 set, albeit not much. I haven't seen anything new in terms of new buildings. My reference would be the W Hotel at Hollywood and Vine. That hotel was built in 2009, the whole block hasn't been reflected in the set given by the city. It has been in the 2014 county set.
Perhaps its been named 2012 based on the very minor modifications to the 2008 set from county.

We can easily identify which buildings have been replaced with the assessors's parcel data.

this is both the 2008 buildings with the 2014 parcels, I styled the parcels with buildings built after 2008 in orange (the image is the Hollywood area, with the W Hotel circled at Hollywood and Vine . When we link the two datasets for the use and age we can identify these for importers to check and redraw the new buildings.

There's a column in the assessors data that says how many buildings are on the property. After a count, there are about 20,996 buildings built after 2008 throughout the county. It's gonna be a fun import!
Here's a screenshot

@cityhubla Yes, we definitely want to start thinking about how to coordinate the manual updating after we do the import. We'd need to make sure that the license for the 2014 data is compatible, right? If so, we could set up a maproulette task to direct people to places on the map that need new buildings traced. We should think about filling this out in the "updates" section in the wiki, and track in issue #6.

Regarding the 2012 data we have at hand, do you think it's still worth using, @cityhubla? Even if it's not truly data that shows all the buildings in 2012, if it's an improved snapshot of 2008 we should still use it.

I would continue to use the 2008 set. The missing data in the 2012 set is the AIN (Assessor Identification Number) attributes are missing. We would need these to link to the AIN on the parcel data to get the building uses. Aside from what you and I found, I don't think its much of an improvement.
I will contact County for licensing and using the 2014 set. I'll draft something and post for the "updates" section on the wiki.

@cityhubla @almccon I agree. Let's do a join to get the assessor data (there now is 2015 assessor data) and combine that with the 2008 file. I wouldn't waste time contacting the county. Mark says it'll be out of contract and posted online in a year. We can do updates then and just use the 2008 file to compare against newer buildings. But either way I say we hit this and make it happen! Yeah! I'm pumped.

LA City's data is the same as the 2008 data from the county. We've decided to move forward with that 2008 data until a new 2014 dataset is released sometime in 2016.


3rd party causing conflict - ng2-validation

Type of Issue
[ X ] Bug Report
[ ] Feature Request

Description
When using, the following package, during the npm run packagr command this will cause failure.
https://www.npmjs.com/package/ng2-validation
This package has a file called
libphonenumber-js/metadata.min.json
I have confirmed this file has valid json, jsonlint
Within this dependency there is a file called
How To Reproduce
Install the ng2-formvalidate, run a packgr build
error:
BUILD ERROR
Unexpected token
Error: Unexpected token
  /frontend/Libraries/form-validation-library/node_modules/rollup/dist/rollup.js:187:14)
    at Module.error (/frontend/Libraries/form-validation-library/node_modules/rollup/dist/rollup.js:8392:3)
    at tryParse (/frontend/Libraries/form-validation-library/node_modules/rollup/dist/rollup.js:8108:10)
    at new Module (/frontend/Libraries/form-validation-library/node_modules/rollup/dist/rollup.js:8149:15)
    at load.catch.then.then.then.source (frontend/Libraries/form-validation-library/node_modules/rollup/dist/rollup.js:10009:20)

Expected Behaviour
A successfull build
Version Information
ng-packagr: v1.x.y
node: v5.x.y
@angular: v4.x.y
npm 3

When the build packgr function command is ran, is it nessecary to include the node_modules folder or can this be ignored by a flag? Other packages I have developed without packagr have not needed to build a node_module folder, also, this doesnt get distributed on publish.

If you are on windows, please change your package.json script to include "packagr": "SET DEBUG=true && ng-packagr -p ng-package.json", run it, then paste the output.

Unfortunately I'm on MAC any assistance on debug for that?
On 26 Oct 2017 10:04, "David Parks"<EMAIL_ADDRESS>wrote:

If you are on windows, please change your package.json script to include "packagr":
"SET DEBUG=true && ng-packagr -p ng-package.json", run it, then paste the
output.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dherges/ng-packagr/issues/224#issuecomment-339600858,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVfK2jO5uWQWj7hOkoJcmgIJmbVL7DFOks5swEsagaJpZM4QHNxX
.

Sorry, I don't have a mac to test commands on, but the idea is that you temporarily set the DEBUG environment variable to true. If you provide a clone-ready reproduction of the issue, it would also be helpful.

Hi David, ill make a clone ready reproduction as unfortunately I cant seem to set debug for you


optimize toplist to have only needed measures

E2E is failing because on changing the time period No comparison is getting changed to Comparison, test is expecting No comparison. This is happening on main as well.

Since we had our code freeze last Friday (but since this will be something we'll want to have in place for Automatad), once we have a final fix in place, what would be the best way to make sure this is part of our 0.40 release?
@ericpgreen2 @begelundmuller Any chance we could merge this with our release candidate or would this require cutting a new hotfix (not familiar with our internal engineering processes)?

Hey, I just happened to look at this and see that it disables some client side code that I wrote last month. Not saying not to merge it if it's causing bigger problems elsewhere, but disabling that code will cause a regression in a bug that was fixed recently, so from a process perspective I think it would have been appropriate to look at the git blame for the code being disabled and flag the author to discuss the impact and alternatives. I only happened to look at this by PR chance, and would have missed the regression being introduced.

Since we had our code freeze last Friday (but since this will be something we'll want to have in place for Automatad), once we have a final fix in place, what would be the best way to make sure this is part of our 0.40 release?
@ericpgreen2 @begelundmuller Any chance we could merge this with our release candidate or would this require cutting a new hotfix (not familiar with our internal engineering processes)?

Andrew, given this PR is considered a release blocker, we'll add it (cherry-pick it) to the release branch once it's merged to main.

@bcolloran that code removal was just a hack to get past the query cancellations that I was seeing. Just reached out to Aditya during IST hours as I wanted to deploy this code on cloud for testing and thats why pushed the change to the branch. Its not supposed to be merged, I will revert that commit. @AdityaHegde mentioned that applications teams will discuss internally on how to implement a proper fix.

@bcolloran removed that commit

@bcolloran that code removal was just a hack to get past the query cancellations that I was seeing. Just reached out to Aditya during IST hours as I wanted to deploy this code on cloud for testing and thats why pushed the change to the branch. Its not supposed to be merged, I will revert that commit. @AdityaHegde mentioned that applications teams will discuss internally on how to implement a proper fix.

Gotcha, thanks @pjain1! Sorry to be sticking my nose in where it doesn't belong in that case!!

@pjain1, the frontend code LGTM.


Odd Scroll Bar Issue on Query Panel

Every So often the query panel will get somehow out of sync. That is, the first couple of line numbers go off screen where they are not visible. You can't scroll to them and no way to see them. A page refresh fixes it all. It also looks like the scroll bar UP Arrow Button is missing so something might be messing with the panel

Expected Behavior

To be able to get to see/edit the top lines of the query panel
Current Behavior

Possible Solution

Refresh the page as a last resort
Steps to Reproduce (for bugs)

Can't come up with a scenario the triggers the issue but happens enough to notice it.

Context

Your Environment

Version used: 1.6.9
Environment name and application type (e.g. Chrome 39, chrome extension v1.4.7):
Firefox 59.0.1 (64-bit)
Operating System and version (e.g. Linux, v1.4.7):
Windows 10 Home

Does the mouse scroll still work?

Yep, just can't scroll to the top lines. Good news is I think I can reproduce it.
Have a long query in the panel, long enough for the scroll bars. Switch tabs, then switch back to the one with the long. It doesn't do it all the time but most of the time. I have about 5 tabs.
Hopefully you can reproduce now :)

Just got an update to 1.7.0 and it seems to not be a problem. Not sure what changed but I think I can close this for now. If I see it again I'll reopen.
Sandy

One more comment. This might have something to do with the Header on the Results panel. In looking at the Query panel vs. the Results panel the scroll bars are offset by the header amount. BUT their is no header on the Query panel.
If you look at the Results panel with a LONG result you will see the opposite effect, the scroll bar Bottom will be chopped off.
I was looking at the CodeMirror-vscrollbar class since it looks like it's used for both panels.
Sandy

I can't replicate the issue in mac at the moment. I'd still look into it though.

Here is a pic just for reference on the bottom scroll bar issue...


Feature Request: RunsAs_userID

Is your feature request related to a problem? Please describe.
IRRSMO00 has a parameter that would allow the XML passed to it to be run under a different User ID.
Describe the solution you'd like
Allowing pyracf to leverage this would be beneficial for automated tasking, and could also allow for a variation on resource profile access checking.
Describe alternatives you've considered
N/A
Additional context
Link to IRRSMO00 Documentation discussing this

Implemented in #57


vim does not build with VisualStudio2019 if PYTHON support is included

Instructions: Replace the template text and remove irrelevant text (including this line)
Describe the bug
A clear and concise description of what the bug is.
(Issues related to the runtime files should be reported to their maintainer, check the file header.)
To Reproduce
Detailed steps to reproduce the behavior:

clone https://github.com/vim/
install MS VisualStudio 2019
execute "vcvars64.bat"
install Python (I used WinPython, 3.8.6, into the folder "C:\Programs")
go to the cloned repo, into the "src" folder, and type
make -f Make_mvc.mak GUI=yes PYTHON=C:\Programs\WPy64-3860\python-3.8.6.amd64 PYTHON_VER=38

This leads to the following output:
Microsoft (R) Program Maintenance Utility Version 14.28.29337.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Python requested (version 38) - root dir is "C:\Programs\WPy64-3860\python-3.8.6.amd64"

        cl -c /W3 /nologo  -I. -Iproto -DHAVE_PATHDEF -DWIN32  -DFEAT_CSCOPE -DFEAT_TERMINAL -DFEAT_SOUND -DFEAT_NETBEANS_INTG -DFEAT_JOB_CHANNEL -DFEAT_IPV6   -DFEAT_XPM_W32   -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 /source-charset:utf-8 /MP -DHAVE_STDINT_H /Ox /GL -DNDEBUG  /Zl /MT /D_CRT_SECURE_NO_DEPRECATE /D_CRT_NONSTDC_NO_DEPRECATE -DFEAT_MBYTE_IME -DDYNAMIC_IME -DFEAT_GUI_MSWIN -DFEAT_DIRECTX -DDYNAMIC_DIRECTX -DFEAT_DIRECTX_COLOR_EMOJI -DDYNAMIC_ICONV -DDYNAMIC_GETTEXT -DFEAT_PYTHON -DFEAT_HUGE /Fd.\ObjGXYAMD64/ /Zi /Fo.\ObjGXYAMD64/ /I "C:\Programs\WPy64-3860\python-3.8.6.amd64\Include" /I "C:\Programs\WPy64-3860\python-3.8.6.amd64\PC" if_python.c
if_python.c
if_python.c(81): warning C4005: 'PyBytes_Check': macro redefinition
C:\Programs\WPy64-3860\python-3.8.6.amd64\Include\bytesobject.h(48): note: see previous definition of 'PyBytes_Check'
c:\Users\thomas\Coding\vim\src\if_py_both.h(120): warning C4013: 'PyString_Check' undefined; assuming extern returning int
c:\Users\thomas\Coding\vim\src\if_py_both.h(123): warning C4013: 'PyString_AsStringAndSize' undefined; assuming extern returning int
c:\Users\thomas\Coding\vim\src\if_py_both.h(259): warning C4013: 'PyString_FromString' undefined; assuming extern returning int
c:\Users\thomas\Coding\vim\src\if_py_both.h(259): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(757): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(764): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(772): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(873): warning C4013: 'PyString_FromStringAndSize' undefined; assuming extern returning int
c:\Users\thomas\Coding\vim\src\if_py_both.h(875): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1076): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1157): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1167): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1641): error C2039: 'ob_type': is not a member of 'DictionaryObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1566): note: see declaration of 'DictionaryObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1832): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(1973): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(2383): error C2039: 'ob_type': is not a member of 'ListObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(2248): note: see declaration of 'ListObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(2468): warning C4133: 'function': incompatible types - from 'PySliceObject_T *' to 'PyObject *'
c:\Users\thomas\Coding\vim\src\if_py_both.h(2770): warning C4133: 'function': incompatible types - from 'PySliceObject_T *' to 'PyObject *'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3111): error C2039: 'ob_type': is not a member of 'FunctionObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(2932): note: see declaration of 'FunctionObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3131): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3302): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3422): warning C4013: 'PyInt_FromLong' undefined; assuming extern returning int
c:\Users\thomas\Coding\vim\src\if_py_both.h(3422): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3427): warning C4047: 'initializing': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3484): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3740): error C2039: 'ob_type': is not a member of 'TabPageObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3691): note: see declaration of 'TabPageObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3793): warning C4013: 'PyString_FromFormat' undefined; assuming extern returning int
c:\Users\thomas\Coding\vim\src\if_py_both.h(3793): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3800): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(3802): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(4129): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(4136): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(4138): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(4183): error C2039: 'ob_type': is not a member of 'WinListObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(4156): note: see declaration of 'WinListObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5130): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5139): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5200): error C2039: 'ob_type': is not a member of 'BufferObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(4841): note: see declaration of 'BufferObject'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5254): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5382): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5390): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(5778): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6386): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6420): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6459): error C2065: 'OutputGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6460): error C2065: 'OutputSetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6461): error C2065: 'call_PyType_GenericAlloc': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6461): warning C4047: '=': 'allocfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6462): error C2065: 'call_PyType_GenericNew': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6462): warning C4047: '=': 'newfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6463): error C2065: 'call_PyObject_Free': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6463): warning C4047: '=': 'freefunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6494): error C2065: 'BufferGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6495): error C2065: 'BufferSetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6496): error C2065: 'call_PyType_GenericAlloc': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6496): warning C4047: '=': 'allocfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6497): error C2065: 'call_PyType_GenericNew': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6497): warning C4047: '=': 'newfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6498): error C2065: 'call_PyObject_Free': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6498): warning C4047: '=': 'freefunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6515): error C2065: 'WindowGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6516): error C2065: 'WindowSetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6517): error C2065: 'call_PyType_GenericAlloc': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6517): warning C4047: '=': 'allocfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6518): error C2065: 'call_PyType_GenericNew': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6518): warning C4047: '=': 'newfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6519): error C2065: 'call_PyObject_Free': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6519): warning C4047: '=': 'freefunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6534): error C2065: 'TabPageGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6535): error C2065: 'call_PyType_GenericAlloc': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6535): warning C4047: '=': 'allocfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6536): error C2065: 'call_PyType_GenericNew': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6536): warning C4047: '=': 'newfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6537): error C2065: 'call_PyObject_Free': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6537): warning C4047: '=': 'freefunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6578): error C2065: 'RangeGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6579): error C2065: 'call_PyType_GenericAlloc': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6579): warning C4047: '=': 'allocfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6580): error C2065: 'call_PyType_GenericNew': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6580): warning C4047: '=': 'newfunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6581): error C2065: 'call_PyObject_Free': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6581): warning C4047: '=': 'freefunc' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6593): error C2065: 'CurrentGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6594): error C2065: 'CurrentSetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6613): error C2065: 'DictionaryGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6614): error C2065: 'DictionarySetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6633): error C2065: 'ListGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6634): error C2065: 'ListSetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6652): error C2065: 'FunctionGetattro': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6680): error C2065: 'vimmodule': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6681): error C2065: 'vimmodule': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6681): error C2224: left of '.m_name' must have struct/union type
c:\Users\thomas\Coding\vim\src\if_py_both.h(6682): error C2065: 'vimmodule': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6682): error C2224: left of '.m_doc' must have struct/union type
c:\Users\thomas\Coding\vim\src\if_py_both.h(6683): error C2065: 'vimmodule': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6683): error C2224: left of '.m_size' must have struct/union type
c:\Users\thomas\Coding\vim\src\if_py_both.h(6684): error C2065: 'vimmodule': undeclared identifier
c:\Users\thomas\Coding\vim\src\if_py_both.h(6684): error C2224: left of '.m_methods' must have struct/union type
c:\Users\thomas\Coding\vim\src\if_py_both.h(6930): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
c:\Users\thomas\Coding\vim\src\if_py_both.h(6867): warning C4047: 'initializing': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(930): warning C4133: 'function': incompatible types - from 'char *' to 'const wchar_t *'
if_python.c(1181): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1185): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1187): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1188): warning C4013: 'Py_FindMethod' undefined; assuming extern returning int
if_python.c(1188): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1248): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1292): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1332): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1352): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1434): warning C4133: 'function': incompatible types - from 'char *[2]' to 'wchar_t **'
if_python.c(1436): warning C4013: 'Py_InitModule4' undefined; assuming extern returning int
if_python.c(1437): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1465): warning C4047: '=': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1469): warning C4013: 'PyString_AsString' undefined; assuming extern returning int
if_python.c(1469): warning C4047: '=': 'char *' differs in levels of indirection from 'int'
if_python.c(1491): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1493): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1497): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1504): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1508): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
if_python.c(1521): warning C4047: 'return': 'PyObject *' differs in levels of indirection from 'int'
NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.28.29333\bin\HostX64\x64\cl.EXE"' : return code '0x2'
Stop.

Expected behavior
The build should run through. (Note that the build DOES work without the PYTHON options!)
Screenshots
If applicable, copy/paste the text or add screenshots to help explain your problem.
Environment (please complete the following information):

Windows 10 Education N, Version 10.0.19041

try the official python: https://www.python.org/ftp/python/3.9.0/python-3.9.0-amd64.exe That is what is used for the vim-win32-installer repository and this one works.

You are specifying PYTHON and PYTHON_VER options, but you should use PYTHON3 and PYTHON3_VER options instead.
Adding DYNAMIC_PYTHON3=yes might be better.

Hello, I am trying to do the same thing, but without the dynamic option and the compilation fails on both python2 and python3. These are my commands for the compilation using VS:
Python2:    nmake -f Make_mvc.mak GUI=yes PYTHON=C:\Users\gou\Miniconda3\envs\py27 PYTHON_VER=27
Python3:    nmake -f Make_mvc.mak GUI=yes PYTHON3=C:\Users\gou\Miniconda3 PYTHON3_VER=37
...
if_python3.obj : error LNK2001: Nicht aufgelöstes externes Symbol "__imp__PyEval_RestoreThread".
gvim.exe : fatal error LNK1120: 105 nicht aufgelöste Externe
NMAKE : fatal error U1077: ""C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\bin\HostX86\x86\link.EXE"": Rückgabe-Code "0x460"
Stop.
There are many LNK2001 errors, but i cut them off to not spam the window. The win32 installer provides Python support, but only dynamic, which doesn't seem to be acceptable for some plugins, i.e. vimspector.
Does anyone have an idea how to fix the problem? It seems to be a Windows issue (surprise surprise) because i compiled vim fine on linux. The win32 installer provides Python support, but only dynamic, which doesn't seem to be acceptable for some plugins, i.e. vimspector.
Thanks in advance.

can you please try with official python package and not some other distrution?

The win32 installer provides Python support, but only dynamic, which doesn't seem to be acceptable for some plugins, i.e. vimspector.

I don't think this is true. you should be able to simply load the library and then it should work. Ping @puremourning please correct me if I am wrong. Thanks!

@chrisbra you're not wrong... I only ever use your builds of vim for windows and they work fine with vimspector and YCM

Hello,
thanks a lot for the prompt responses, and amazing work @puremourning :).
Unfortunately for me, Vim installed via the Windows installer (with both versions of Python included dynamically) gives me a warning that vimspector needs the +python3 option. I tried doing some basic things like setting a breakpoint or installing an adapter after closing the warning, but nothing happens, so i assume that the plugin is not loaded. Can it really be due to the fact that my python installation is not in C:? I'd prefer to stick to the conda environment if it's possible.
Thanks in advance.

you may need to set the :set pythonthreedll=c:\path\to\python\dll. Just make sure that if you use a 32bit vim you also use the  32bit python dll. I am not sure if miniconda/anaconda will always work.

start vim with gvim -Nu NONE and run :py3 pass. If this doesn't work, then python3 plugins won't work.
It's likely that either:

your python does not match the architecture of your vim (both must be 32 or both 64 bit)
your python dll is not in your PATH
your python is not python  3.6, which is the version that vim is compiled against in your screenshot.

The simplest way to double-check this and make it work is to :set pythonthreedll=<path> where <path> is the path to the actual python36.dll in your python installation.

I got it working :) :) Thanks a lot.
Now i'm having an issue with the vscode-cpptools adapter. During installation the archive is extracted, then removed, and then the installation crashes because a file in the now removed directory doesn't exist.

If you know the answer right away that'd great, but if this should be put in a new issue, then i'd be happy to do that.

Awesome, thank you. :)


Display historical price data in Treasury transfers list

The Treasury design calls for an asset price below the asset amount in the historical transfers list.
The Gnosis API doesn't provide this data, so we'll need something else, possibly coingecko:
https://www.coingecko.com/en/api/documentation
/coins/{id}/history
Though it looks like collating that API with the token's ID might be tricky

We're in talks with Consola finance for potentially using their API for a revamped Treasury page.
No point in doing further work on Treasury until we get that settled and/or have user feedback requiring this.


Can't get "declare module ..." to work

I'm trying to make some type definitions for some libraries I'm using, but I can't get it to work. I've been reading http://flowtype.org/docs/declarations.html but it doesn't seem I'm understanding it correctly.
Clone https://github.com/AgentME/flow-test/, run npm install (to install lodash locally), and then run "flow check". The flow check should fail because of misuse of lodash, but it does not.
flow-libs/lodash.js (pretty bare-bones so far)
declare module lodash {
  declare function map<T,U>(array: T[], cb: (value: T, key: number|string, array: T[]) => U, thisArg?: any): U[];
  declare function filter<T>(array: T[], cb: (value: T, key: number|string, array: T[]) => boolean, thisArg?: any): T[];
}

index.js
/* @flow */
// jshint ignore:start

import {map} from 'lodash';

// should work
var list = map([5, 6], x=>x+1);
console.log(list);

// should fail flow check
var list2 = map(123, 'foo');
console.log(list2);

Possibly the same issue as #447

@AgentME Sorry, couldn't repro. I made a dir called test, added index.js to it, made a subdir called lib, added lodash.js to it, made a .flowconfig in test/ with the lines
[libs]
lib
and ran flow check test, got the expected errors:
/home/avik/test/index.js:11:13,27: function call
Error:
/home/avik/test/index.js:11:17,19: number
This type is incompatible with
/home/avik/test/lib/lodash.js:2:36,38: array type
/home/avik/test/index.js:11:13,27: function call
Error:
/home/avik/test/index.js:11:22,26: string
This type is incompatible with
/home/avik/test/lib/lodash.js:2:45,91: function type

If you have lodash installed (after running "npm install lodash"), then flow check fails to fail.

Yeah, I can repro the same behavior, but only when the node module exists.

Merging with #447


feat(vinxi-react-server-dom): open vite peerDeps up to version 6

Originating from https://github.com/TanStack/router/issues/2931, where the vite's peerDeps for the @vinxi/react-server-dom package was at ^4.3.9.
This change opens vite's peerDeps range from ^4.3.9 to ^4.3.9 || ^5.0.0 || ^6.0.0.

Hi! I'm the autofix.ci troubleshooter bot.
It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃

Closing. Will open a PR without changes to the lockfile.


👌 IMPROVE: Disable Font Smoothing Disabler

Disable Font Smoothing Disabler in macOS Mojave. It's a new issue and over 500 people have found my solution to be working great for them on social media and blog comments.
Thanks for the awesome repo!

This is a good idea for anyone with a non-Retina screen.  I've found that it helps with some but not all apps, and (in my experience) hurts none.


Update sensorfilter world position calculation and new FSR readings

ngl... not sure how to do the rest @KipHamiltons


Recovery Services Vault can no longer be enabled for CMK without role assignment to Key Vault

Cannot use user-assigned managed identity instead of system-assigned managed identity because it does not (yet) support firewall restrictions or private endpoints on the Key Vault.
Error when creating the vault first, then assigning RBAC on the KV, then creating another vault deployment with encryption enabled because supposedly backup items are already present.


test: use FeatureContract in create_test_state and tests which uses it

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (828c57b) 69.64% compared to head (92ae6a3) 69.62%.
Report is 2 commits behind head on main-v0.13.1.

Additional details and impacted files
@@               Coverage Diff                @@
##           main-v0.13.1    #1556      +/-   ##
================================================
- Coverage         69.64%   69.62%   -0.02%
================================================
  Files                59       59
  Lines              8173     8168       -5
  Branches           8173     8168       -5
================================================
- Hits               5692     5687       -5
  Misses             2015     2015
  Partials            466      466

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


File name is not excessively encoded for the AjaxFileUpload validation message (Resolve #364)

Now we have 2 places where filename appears not-encoded:

AjaxFileUploadEventArgs.get_fileName - change in public API!
A line near "preparing upload data" - is it safe to skip encoding here?

@AlekseyMartynov

AjaxFileUploadEventArgs.get_fileName - change in public API!

No. This _fileName is not affected by getFileName(). The callstack is as follows:

onFileSelectedHandler
addFileToQueue(fileItem)
Sys.Extended.UI.AjaxFileUpload.Item constructor

A line near "preparing upload data" - is it safe to skip encoding here?

Yes. This fileItem is not affected by getFileName(). The file name is encoded later, at the line 766.


refactor: Make project manifest loading DRY and consistent

I have extracted drive-by refactors from https://github.com/prefix-dev/pixi/pull/1485

Defer much of Project::load to Project::from_manifest for consistency, and to avoid repetition.
Rename Project::load to Project::from_path, to be consistent with Manifest::from_path
Remove unused method Manifest::solve_group, as well as unnecessary collection

Thanks!


"php artisan route:cache" should produce the same file when nothing changed in the (unnamed) routes

Laravel Version: 7.30.0
PHP Version: 7.2.31
Database Driver & Version: MySql 8.0.16

Description:
I expect the routes-v7.php to be determenistic, giving the same result for the same set of routes, when the routes did not change.
php artisan route:cache generates each time a different bootstrap/cache/routes-v7.php file, when there are unnamed routes. Unnamed routes are currently assigned a random name in Illuminate\Routing\AbstractRouteCollection
    /**
     * Get a randomly generated route name.
     *
     * @return string
     */
    protected function generateRouteName()
    {
        return 'generated::'.Str::random();
    }

Because part of my workflow is to always call the route:cache before a git push there is always a different bootstrap/cache/routes-v7.php build. The changed routes-v7.php file is committed and is deployed from the git repository, because there is no SSH access on the production server. The production server in on shared hosting.
So I have many unnecessary commits, because there were changes in the routes-v7.php file, while there was no change in the routes ( routes/web.php ).
P.S. the route:cache is automated so I don't have to think about rebuilding the route cache when something changes.
Steps To Reproduce:

have a routes/web.php with unnamed routes e.g.

<?php

    Route::get('login', 'Auth\\AuthController@getLogin')->name('login');
    Route::post('login', 'Auth\\AuthController@postLogin');
    Route::get('logout', 'Auth\\AuthController@getLogout');
    Route::post('logout', 'Auth\\AuthController@getLogout')->name('logout');

call php artisan route:cache
copy bootstrap/cache/routes-v7.php to bootstrap/cache/routes-v7.php.first
call php artisan route:cache
compare bootstrap/cache/routes-v7.php with bootstrap/cache/routes-v7.php.first. They are different, but should be the same.

Other
See also my question in discussions which yielded no answer: https://github.com/laravel/framework/discussions/33963

This isn't really a "bug" in the framework.

In what category does this issue belong , if this is not considered a bug?
Should I file it somewhere else ?

The changed routes-v7.php file is committed and is deployed from the git repository

Because this is your specific workflow which is different from the framework's default setup this isn't considered a bug. We can't help you here.

Sorry to hear Laravel does not support hosting without SSH access.
Well I guess I have to live with this ever changing routes-v7.php file then.
Because this really bugs me, if I would create a pull request regarding this issue, would that be a candidate for being merged. Or has such an attempt no chance ?

You could just name all your routes, it's not really a good practice to have unnamed routes anyway

Mathias E. sent me an email: You could just name all your routes, it's not really a good practice to have unnamed routes anyway

Thanks for this suggestion. I just named my unnamed routes "unnamed.1" to "unnamed.259". Yep there are 259 unnamed routes, because this project originates from Laravel 4.2 and was shifted to Laravel 7.
After naming everything (don't forget api.php and other files in the routes folder), the php artisan rout:cache is finally determenistic.

Sorry to hear Laravel does not support hosting without SSH access.

That doesn't really have anything to do with this. You can't edit the cached routes file. We can't control what env you choose to host your project. If you don't have SSH access then some things like route caching is unavailable for you.


Part 3 - 2 race unchecked though I selected other so maybe OK

Yeah there's no "other" box for race


added media_stop

VLC was missing the media_stop. The pause was present, but starting the same file result in resuming the file instead of start over new
Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

I corrected the style issues and missing flag!

last style fix!

Yes, I removed the trailing whitspace


[SelectionManager] Select the word underneath the cursor on contextmenu

If a user right clicks on a word or url in the terminal, the word should be automatically selected, so it can be copied without having to select it first. This would mimic the default behaviour of Terminal.app, Chrome and most other UIs that allow text selection. I can try to create a PR with this feature if desired.

I think it would be good to support this for macOS only, a PR would be awesome! One thing to keep in mind is that it generally doesn't seem to select whitespace in the programs I tested, currently SelectionManager._selectWordAt will select whitespace so you'll need to add an extra parameter to that function or something.

I was under the assumption that this would be the default behaviour for Windows and Linux too. How about making it platform dependant by default, but configurable through a config flag?

@mofux sounds good if it's default on macOS only. It would be cool to have opt-in support on other platforms.

Is it really blocked? I would have thought you'd check MouseEvent for whether it was a right click and/or listen to contextmenu?

@Tyriar I thought I would just hook into the mousedown handler of the selection manager (the part that is changed by #780), check if options.rightClickSelectsWord is true, selection is currently empty and evt.button is 2 (right mouse button) and basically trigger the same action that is done for double-click (which should select the word underneath the cursor). Then, term.defaults.rightClickSelectsWord would only be true if Browser.isMac

Right now the right click logic is contained within Clipboard.ts which is called by listening to contextmenu or mousedown depending on the browser. This should probably go inside rightClickHandler which already has the SelectionManager passed in.

Fixed with https://github.com/xtermjs/xterm.js/pull/1114


grid-row mixin with ONLY collapse behaviour does not work

The mixin grid-row should accept behaviour: collapse according to the documentation from _row.scss.

@param {Keywords} $behavior [null]
Modifications to the default grid styles. nest indicates the row will be placed inside another row. collapse indicates that the columns inside this row will not have padding. nest collapse combines both behaviors.

But it isn't working because of the wrong if condition (I assume) and currently collapsed behaviour works only in combination with nest value.
@if index($behavior, nest) != null {
    @include grid-row-nest($gutter);

    @if index($behavior, collapse) != null {
      margin-left: 0;
      margin-right: 0;
    }
  }
  @else {
    max-width: $width;
    margin-left: auto;
    margin-right: auto;
  }

How can we reproduce this bug?
use : @include grid-row($behavior: collapse);
What did you expect to happen?
I expect unset max-width and:
margin-left: 0;
margin-right: 0;

What happened instead?
max-width: $width;
margin-left: auto;
margin-right: auto;

@andycochran or @ncoden can one of you take a look at this?

Hmm... I'm not sure (because in fact I don't like the way we handle columns margins), but I think @samuells is right.

We should have:
  @if index($behavior, nest) != null {
    @include grid-row-nest($gutters);
  }
  @else {
    @include grid-row-size($size);
  }

  @if or index($behavior, nest) != null index($behavior, collapse) != null {
    margin-right: 0;
    margin-left: 0;
  }
  @else {
    margin-right: auto;
    margin-left: auto;
  }

But there is probably a way to implement it better than this.

Edit: it appears that for:
<html>
  <div class="row>
    ...
  </div>
</html>

... the normal behavior is to create a basic container (centered with a maximum width). I don't like this behavior (that's a row, not a container), but it's works like this for now.
Maybe we could use a  new behavior or option container as default (that would be incompatible with the nest behavior), and generate .row with.

Yeah, rows currently are their own containers, and there is no separate concept of a container. @gakimball do you know what the reasoning behind that was?

Related to: https://github.com/zurb/foundation-sites/issues/9569

Was this actually solved with https://github.com/zurb/foundation-sites/pull/9568?


Custom error, "responses.map is not a function"

Hi,
I'm using graphql-server-express and my server is set up like this:
server.use(
  GRAPHQL_ENDPOINT,
  authenticate,
  bodyParser.json(),
  graphqlExpress((req) => ({
    context: createContext(req),
    formatError,
    schema
  }))
);

The authenticate middleware throws an error if the JSON web token doesn't validate:
import jwt from 'jsonwebtoken';
import {JWT_SECRET} from '../../config/auth';

export default (req, res, next) => {
  if (req.headers && req.headers.authorization) {
    const parts = req.headers.authorization.split(' ');
    const authToken = parts[1];

    try {
      const decodedToken = jwt.verify(authToken, JWT_SECRET);
      req.decodedToken = decodedToken;
    } catch (error) {
      return res.json({
        data: null,
        errors: [{
          message: 'Invalid or expired authentication token.'
        }]
      });
    }
  }

  return next();
};

However, when apollo-client is trying to parse this on the client, it fails with Network error: responses.map is not a function. Is it possible it respond with a GraphQLError outside of the GraphQL server? Or any ideas how I can solve this?
Thanks!

are you using transport batching by any chance?

Yes.

Had the same problem, switching to normal network interface solved the problem.

@richardkall actually, after looking at your code I realized that this is due to an invalid response from your server. The response should be an array of GraphQL results [{ data: ..., error: ... }, ...]. Your server just returns a single response, which is not a valid JSON response for a batched request. Your middleware will have to check if the request expects a single response or an array.
I made sure it throws a more useful error message in #893.


Support for '--dart-define-from-file' via Command Line

I'm currently using the Flutter Patrol package and have encountered an issue. The package does not support the '--dart-define-from-file' option via the command line. This feature is crucial for my project's workflow, and I'm wondering if there are any plans to implement this functionality.
Could you please let me know if this feature is on the roadmap, and if so, when we might expect it to be available in the package? Any information or guidance you can provide would be greatly appreciated.
Old PR Add dart defines from file

Why is it crucial for your project? What is possible with --dart-define-from-file that is not possible with --dart-define?

It works, but we have 24 variables and, for now, 5 whitelabels. Sometimes, the variables are changed and not the Test variables. If we have the --dart-define-from-file option, we can use the same files for whitelabels and for testing. We currently have 5 whitelabels, but we will be adding more.

Closing as duplicate of https://github.com/leancodepl/patrol/issues/1742


Collapse borders

Using the technique from https://ratatui.rs/how-to/layout/collapse-borders/

There's a couple of difficult T junctions in that layout to hit just using the layout and block border sets. It would probably be easiest to manually draw them (someday we'll have a real collapsible border approach I think).
P.S. in your layout code you can drop the .as_ref() calls and you might be able to use Min(0) instead of Max(some calculation based on leftover sizes)

This change looks great! It's been merged.
Thanks!


GPU Temp showing "none"

Hello, I am trying the command "print(WinTmp.GPU_Temp())" and get CPU temperature but GPU returns "None". My PC has a dedicated Nvidia RTX 3060 Ti

Help would be appreciated!
(Edit: it isn't about running the program as administrator, just had another issue with that but already resolved it but the problem is still the same)

Thank you so much for using the library and sorry about now getting to this issue sooner.
Does your GPU temperature show up when in the Task Manager under the performance tab? While CPU temps are usually never reported in this section, I have found that GPU temps usually appear under this section.

Yes, also I'm on Windows 10 if that's relevant.

I just published a new version of this library (v1.1.0) that uses the latest version of LibreHardwareMonitor. Could you try to install and test it out to see if it resolves your issue?

Yes, I updated the version and now it works! Thanks.


Change message ephemeral

Changed all statements by Auriga to EphemeralMessage.
I considered refactoring as well, but since the differences would be significant, I decided to limit the changes to the above for now
https://moneyforward.slack.com/archives/C03CB6NF2G6/p1720156080683339

@sho0126hiro 🙏


Can you troubleshoot database problems with the plugin?

I think it's about the miner break event.
Do you have problems with characters other than English? But the problem didn't happen at other job events like woodcutters or architects
I found out what was wrong yesterday. I hope the problem is solved quickly.
Server Type (Spigot/Paperspigot/etc):
Paperspigot
Server Version (using /ver):
This server is running Paper version git-Paper-235 (MC: 1.14.4) (Implementing API version 1.14.4-R0.1-SNAPSHOT)
  Miner:
    fullname: '광부'
    shortname: M
    description: '광부는 석재나 광물을 캐서 돈을 벌 수 있습니다.(주의*원석을 다시 놓으면 돈과 경험치가 동일하게 환불됩니다.)'
    ChatColour: DARK_GRAY
    chat-display: full
    max-level: 200
    #slots: 10
    leveling-progression-equation: 10*(joblevel)+(joblevel*joblevel*4)
    income-progression-equation: baseincome+(baseincome*(joblevel-1)*0.01)-((baseincome+(joblevel-1)*0.01) * ((numjobs-1)*0.05))
    points-progression-equation: basepoints+(basepoints*(joblevel-1)*0.01)-((basepoints+(joblevel-1)*0.01) * ((numjobs-1)*0.05))
    experience-progression-equation: baseexperience-(baseexperience*((numjobs-1) *0.01))
    rejoinCooldown: 10
    Gui:
      Id: 270
      Data: 0
    Break:
      stone:
        income: 0.88
        experience: 46.26
      granite:
        income: 0.88
        experience: 46.26
      diorite:
        income: 0.88
        experience: 46.26
      andesite:
        income: 0.88
        experience: 46.26
      coal_ore:
        income: 6.13
        experience: 323.29
      iron_ore:
        income: 14.00
        experience: 740.09
      gold_ore:
        income: 61.25
        experience: 3237.90
      lapis_ore:
        income: 9.63
        experience: 508.81
      redstone_ore:
        income: 9.63
        experience: 508.81
      emerald_ore:
        income: 560.00
        experience: 29603.66
    Place:
      coal_ore:
        income: -6.13
        experience: -323.29
      iron_ore:
        income: -14.00
        experience: -740.09
      gold_ore:
        income: -61.25
        experience: -3237.90
      lapis_ore:
        income: -9.63
        experience: -508.81
      redstone_ore:
        income: -9.63
        experience: -508.81
      emerald_ore:
        income: -560.00
        experience: -29603.66
  ##```

We don't have any database problems, just other reported issues. Also, other chars are works correctly. You should insert the console error if have about jobs.

There was no console error. Only miner had problems.

Can you please describe it again what's your problem?

good
after setting in jobconfig
I have verified that the events of the miner job are working fine in the sqlite repository.
But when I converted to mysql repository state the miner job event didn't work
Other job events have worked
Oddly, only the miner didn't work.

Could you please paste the whole jobconfig file, to hastebin.com site?

https://hastebin.com/vamuyubeme.coffeescript
check cfg

Isn't it possible to send only miner data? Other occupations work normally. "Woodcutter" "Builder"

Isn't it possible to send only miner data? Other occupations work normally. "Woodcutter" "Builder"

Why you want to save only one job? We need to save all jobs, because if we don't save the jobs data, then it will loss the player jobs.

https://pastebin.com/wkxWAUUz - jobconfig
https://pastebin.com/WDVYXRct - gconfig
check cfg

Well, recently there has been an update to the database that uses ids instead of job names for easier searching. The problem with this is that it is not completely finished and causes errors.
Here are some things you can try:

Save your database to other folder, so save the file to another location.
Rebuild the database, shut down the server, delete the current database and restart the server.
Log in to the server and join the miner job and do something and see if it works properly.

shit....
Are you still converting item id to item name?
When will it be released? If there's a plug-in problem, there's nothing more to do.
If you still try to create another database, the miner will not work the same.

Isn't it possible to send only miner data? Other occupations work normally. "Woodcutter" "Builder"

Why you want to save only one job? We need to save all jobs, because if we don't save the jobs data, then it will loss the player jobs.

https://pastebin.com/wkxWAUUz - jobconfig
https://pastebin.com/WDVYXRct - gconfig
check cfg

Well, recently there has been an update to the database that uses ids instead of job names for easier searching. The problem with this is that it is not completely finished and causes errors.
Here are some things you can try:

Save your database to other folder, so save the file to another location.
Rebuild the database, shut down the server, delete the current database and restart the server.
Log in to the server and join the miner job and do something and see if it works properly.

I forgot to say it before.
Could it be a problem with the paper bucket or the latest version of 1.14.4?

https://github.com/Zrips/Jobs/commit/3e0ed72a3fca6fe6e01b1985dda5ac5fe9b33ccf


Add Resource Limits to Backups and Object Storage

Description
Four new Resource Types have been added. Admin can configure corresponding resource limits for the tenants at different levels (domain, account, project)
User dashboard's Storage section will show the new resources, their limits and current usage.

backup - No. of backups used by the account
backup_storage - Backup storage allocated for the account
bucket - No. of buckets used by the accounts
object_storage - Object storage allocated for the account.

Some other related changes done to BnR framework:

Maximum number of Backups to retain can be specified while creating Backup schedules, similar to Scheduled snapshots.
Oldest Scheduled backup of the same interval type will be deleted once the number reaches the configured max Backups value.
Code refactor: Moved syncBackups method from BackupProvider to the framework BackupManagerImpl, as it is a common functionality and all providers were using duplicated code.

Changes done to the Object Storage Framework

Quota parameter is made mandatory while creating a bucket. Bucket quota is considered to be the allocated space and will be used to enforce Resource limits.

Schema Changes:

New Column max_backups added to backup_schedule table
New Column backup_interval_type added to backups table

Api Changes:

createBackup: new Parameter scheduleid. It should be specified whenever a scheduled backup is created. This will translate to the backup_interval_type in the backups table.
createBackupScheduke: new Parameter max_backups. To specify maximum number of backups to retain for the given schedule.

Configurations:

Setting
Scope
Default Value
Description

backup.max.hourly
Global
8
Maximum recurring hourly backups to be retained for an instance

backup.max.daily
Global
8
Maximum recurring daily backups to be retained for an instance

backup.max.weekly
Global
8
Maximum recurring weekly backups to be retained for an instance

backup.max.monthly
Global
8
Maximum recurring monthly backups to be retained for an instance

max.account.backups
Global
20
The default maximum number of backups that can be created for an account

max.account.backup.storage
Global
400
The default maximum backup storage space (in GiB) that can be used for an account

max.domain.backups
Global
-1
The default maximum number of backups that can be created for an domain

max.domain.backup.storage
Global
-1
The default maximum backup storage space (in GiB) that can be used for an domain

max.project.backups
Global
20
The default maximum number of backups that can be created for an project

max.project.backup.storage
Global
400
The default maximum backup storage space (in GiB) that can be used for an project

Setting
Scope
Default Value
Description

max.account.buckets
Global
20
The default maximum number of buckets that can be created for an account

max.account.object.storage
Global
400
The default maximum object storage space (in GiB) that can be used for an account

max.domain.buckets
Global
-1
The default maximum number of buckets that can be created for an domain

max.domain.object.storage
Global
-1
The default maximum object storage space (in GiB) that can be used for an domain

max.project.buckets
Global
20
The default maximum number of buckets that can be created for an project

max.project.object.storage
Global
400
The default maximum object storage space (in GiB) that can be used for an project

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11678

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11742

@blueorangutan test

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✖ el8 ✖ el9 ✖ debian ✖ suse15. SL-JID 11777

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11824

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11830

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✖ el8 ✖ el9 ✖ debian ✖ suse15. SL-JID 11833

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11835

@blueorangutan test

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11841

[SF] Trillian test result (tid-11932)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 52504 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10017-t11932-kvm-ol8.zip
Smoke tests completed. 141 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

[SF] Trillian Build Failed (tid-11941)

[SF] Trillian Build Failed (tid-11942)

[LL] Trillian Build Failed (tid-7045)

@blueorangutan package

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11891


Account for bank fees when confirming pending order

User story
As a host admin, I need to input the actual amount that was received via bank transfer (or other means not integrated into the platform) when confirming a pending financial contribution on the host dashboard. For example, someone makes a contribution of $100, but actually what arrives in the bank is $95 due to bank transaction fees. When I confirm the transaction, it should credit the Collective $95 not $100.
Best solution for this problem
I think the designers could come up with a better solution, but what I thought of was a box where you confirm the amount actually received. This would default to the amount the transaction was for originally, but could be edited to reflect the exact amount received, and that's what would be credited to the Collective balance.

MVP
The current workaround for this is to either confirm the order and then submit an expense to the Collective for the difference, or cancel the pending order and use Add Funds to manually credit the correct amount. Both of these are a lot of work for the admin and prone to human error.

This seems to be a duplicate of https://github.com/opencollective/opencollective/issues/4069 and if no objections I am closing this.


catkin_prepare_release fails for a protected target branch

I don't know if any clean solution is available for this issue. Just describing an issue I have.
Looks like https://github.com/ros/catkin/issues/463 can happen when I run catkin_prepare_release from a branch where certain configuration is not yet done. And from my experience that kind of branch is a "feature branch", one that's created temporarily and will be deleted once its purpose is done.
$ catkin_prepare_release
Prepare the source repository for a release.
Repository type: git
Found packages: pkg_a, pkg_b,...
Prepare release of version '1.0.6' [Y/n]?
<string>:12: (WARNING/2) Inline emphasis start-string without end-string.
<string>:12: (WARNING/2) Inline emphasis start-string without end-string.
Trying to push to remote repository (dry run)...
Could not determine git remote: Command '['/usr/bin/git', 'config', '--get', 'branch.sample_branch.remote']' returned non-zero exit status 1

May be the branch 'sample_branch' is not tracking a remote branch?

$ which catkin_prepare_release
/opt/ros/kinetic/bin/catkin_prepare_release
$ apt-cache policy ros-kinetic-catkin
ros-kinetic-catkin:
  Installed: 0.7.11-0xenial-20180222-175501-0800
  Candidate: 0.7.11-0xenial-20180222-175501-0800
  Version table:
 *** 0.7.11-0xenial-20180222-175501-0800 500
        500 http://packages.ros.org/ros/ubuntu xenial/main amd64 Packages
        100 /var/lib/dpkg/status

Manual workaround is to run a following command as posted on https://github.com/ros/catkin/issues/463#issuecomment-20703183
git branch %YOUR_BRANCH% --set-upstream remotes/%YOUR_REMOTE%/%YOUR_BRANCH%

Btw, this may not be an issue of catkin_prepare_release but I'm just sharing. Reason why I run from a temporary branch is on some repos the target branch is protected and doesn't allow direct commit, which catkin_prepare_release wants to do. So I open a PR to the target branch and have it merged.
A problem of this flow is that tag gets created based on the temporary branch's status at the time catkin_prepare_release succeeds, which is never the same with the status when the PR is merged. To work this around, I delete the tag that's just created both on local and on remote. Then once PR is merged I git tag %TAG% locally then push to the remote.

I am not sure I understand what you are asking for. catkin_prepare_release does two parts:

modify the local versions and commit the changes locally
push the changes as well as a tag upstream

If you local branch is not tracking a remote branch the tool won't be able to do the second step. Probably the same is true when the remote branch is protected.
Do you want an option to only perform the first step?

I am closing this for now due to no response. If you have additional information please create a new ticket in the catkin_pkg repo since the functionality has been moved as of #941.


chore: remove obsolete info about payment method type

Removed the fifth entry that describes to select the payment method type.
As discussed in slack:

In the past we had this as a direct field on the payment method. But it doe snot exist anymore. So we just have to remove it from the public docs.

Sorry for the waiting time, completely forgot about it:(


Docs, test, CI and repo set up for future use

Thanks for this! Quick question.
The second flow chart might be a bit verbose. I guess it's automatically generated and we an modify in the future?
[ic_experiments/classes_graph/packages.png] - is the one I mean.

Thanks for this! Quick question.
The second flow chart might be a bit verbose. I guess it's automatically generated and we an modify in the future? [ic_experiments/classes_graph/packages.png] - is the one I mean.

Hi, this is automatically generated, I agree not the most helpful view for our use. It shows how the packages call each other which often looks confusing


Support multiple input sources

In some cases you might need to convert 2 reports. This would result in calling the report2junit twice and also results in 2 output reports. By accepting multiple input sources we could remove the need of calling report2junit twice. And we can generate a single output report.
In order to implement this we first should remove the need of the --source-type option, see: #1.

The logic to support consolidation is implemented. We still need:

Cli support for multiple files
Documentation update with examples


fix(overlay): restore previous host element before attaching

The first time when PositionStrategy.attach() is called, the overlay is already added to the DOM. However, after detaching and attaching again, it is not yet added to the DOM.
This PR changes the order and first restores the host element back, and then invokes the attach() method.

Is there a bug this is fixing? Are you able to add a unit test that captures the desired behavior?

Yes, I am trying to calculate the size of the overlay in my own PositionStrategy. However, after detaching and attaching, the size is 0,0 because it is still outside the DOM.
I will add a unit test for that.

I added the test. I double-checked it; without my fix it fails as expected.

Pushed the requested change.


Generation of HDBLCM config file

Two node install of SAP HANA using RHEL for SAP Collections fails because the config for node 2 uses the hostname of node 1
Error message
fatal: [dhcp200-237.redhat.com]: FAILED! =>
{
  "changed": true,
  "cmd": [
    "./hdblcm",
    "--ignore=check_diskspace,check_min_mem",
    "--hdbinst_server_xs_engine=off",
    "--hdbinst_server_import_content=off",
    "--configfile=/tmp/ansible.ia7cu4lchanaconfig/configfile.cfg",
    "-b"
  ],
  "delta": "0:00:04.955358",
  "end": "2022-07-21 12:25:10.811444",
  "msg": "non-zero return code",
  "rc": 1,
  "start": "2022-07-21 12:25:05.856086",
  "stderr_lines": [
    "Running in batch mode",
    "  IP address '<IP_ADDRESS>' (host 'dhcp200-241.redhat.com') is not assigned to a local network interface"
  ],
  "stdout_lines": [
    "",
    "",
    "SAP HANA Lifecycle Management - SAP HANA Database <IP_ADDRESS>.1655123455",
    "************************************************************************",
    "",
    "",
    "Scanning software locations...",
    "Detected components:",
    "    SAP HANA Database (<IP_ADDRESS>.1655123455) in /software/hana/extracted/SAP_HANA_DATABASE/server",
    "Log file written to '/var/tmp/hdb_DB1_hdblcm_install_2022-07-21_12.25.06/hdblcm.log' on host 'dhcp200-237'."
  ]
}

PR has been reviewed and rejected, each of these Ansible Tasks are run on the remote node / are run independently. The dynamically generated config file is generated from the Jinja file inside of the Ansible Role, and pushed to sap_hana_install_configfile_directory. As shown in the Ansible Role defaults file, this directory is on the remote/target/managed node/s:
# Directory where to store the hdblcm configfile template and the Jinja2 template:
sap_hana_install_configfile_directory: "{{ sap_hana_install_software_extract_directory }}/configfiles"

The only reason for hitting this problem was when the default values were used and using NFS
Looking at the default values
sap_hana_install_software_directory points to /software/hana [default value]
The path /software/hana is typically a NFS folder in most or some use cases
We generate the sap_hana_install_software_extract_directory from sap_hana_install_software_directory, so it becomes /software/hana/extracted
For configfiles, it would become /software/hana/extracted/configfiles (sap_hana_install_configfile_directory: "{{ sap_hana_install_software_extract_directory }}/configfiles")
Since /software/hana can be a NFS folder, /software/hana/extracted/configfiles can be the same on both the nodes. Without the PR, installation will fail.
If not sap_hana_install_software_extract_directory has to be mandatory and a check needs to be done before proceeding


[XAI] improve abn attention map

This is a plugin for attention map.

Already merged at https://github.com/sony/nnc-plugin/pull/35


Create account on pypi

Goal
We need an account on pypi to publish the packages of python-streaming organization.

closing this one because we are using a personal pypi account


Should nvidia-exporter deployed as Daemonset instead of Deployment

Hi @BugRoger
From the Readme, we can see that you deploy the nvidia-exporter as a Deployment, it may not reasonable if I have more GPU worker nodes in my cluster. Should this be updated as Daemonset ?
Another question is :  actually I find your work from grafana dashaboard, and want I would like to do is to expose nvidia metrics to grafana via kube-prometheus monitoring system.
However, seems I can NOT find your metrics in Prometheus service discovery after deploying your service. Seems I have to deploy service account and service monitor for it. Do you have any guidance or suggestions?
Best Regards!

/close


Workflows analysis

I split the different step in its own Snakefile
Snake_download
Snake_metadata
Snake_crop
Snake_seg
Snake_morph
There are 2 SLURM_Snake
SLURM_Snake_download that takes config file INPUTCSV, therefore this one is used only by Snake_download
SLURM_Snake_generic works with other Snake_*** (metadata, crop, seg, morpho)
There is a README.md with the usage syntax.
This folder is what I have been using to test during development and to process the data step by step, this could be useful to other!

It looks like most of the logic in the various Snake_download, Snake_metadate, Snake_crop, etc files are just bits copied out of BGNN_Snakemake. For example:
https://github.com/hdr-bgnn/minnowTraits/blob/f567c720f2020b01c9543d24b0ff628d0af53e8c/Workflows_minnows/Step_by_step/Snake_download#L7-L19
vs
https://github.com/hdr-bgnn/BGNN_Snakemake/blob/502d285fe89374926cfe1b7d4240388d9946c27e/Snakefile#L6-L18
LIST = config['list']

print(f"your are using this list {LIST}")

df = pd.read_csv(f'{LIST}')
split_df = df['original_file_name'].str.split('\\.', expand=True)
split_df.columns=['original_file_name','2']
samples_df = df.drop(columns=['original_file_name'])
df = pd.merge(split_df['original_file_name'],samples_df, how="left", left_index=True, right_index=True)
samples_df = df.set_index('original_file_name', drop=False)

NAMES = list(samples_df['original_file_name'])
NAMES = [os.path.splitext(name)[0] for name in NAMES]

Are we abandoning the use of https://github.com/hdr-bgnn/BGNN_Snakemake as part of this project?

@johnbradley
could you review this and make the change for the export SBATCH_ACCOUNT=$SLURM_JOB_ACCOUNT in SLURM_Snake_download and  SLURM_Snake_generic you have suggested.
Cheers

@thibaulttabarin Are we abandoning the use of https://github.com/hdr-bgnn/BGNN_Snakemake for Minnow_Traits? Why not make these fixes/re-organization in the BGNN_Snakemake repo?


Jwilaby/#725 theme unthemed areas

Added a few items missed in the last commit :)

Pull Request Test Coverage Report for Build 430

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

Totals

Change from base Build 427:
0.0%

Covered Lines:
3732

Relevant Lines:
6242

💛  - Coveralls


#1074 Adding VANN vocab class (again - see PR number 1075!).

Signed-off-by: Pat McBennett<EMAIL_ADDRESS>This PR addresses GitHub issue: #1074 .
Briefly describe the changes proposed in this PR:

Added the VANN vocab to the collection of vocabularies already defined within RDF4J.

Looks good to me


workaround case for etcdctl not yet being installed

Fixes #789

Please reopen from your fork instead of from kubernetes-incubator/kargo repo


TensorFlow: remove no longer needed third_party/boringssl files.

(I couldn't find any reference to the build files anymore, so I'm assuming they can be deleted)

Yep, I think this completes the work started in 4af1fb42d1c84d8493561089d710b2f4a059a279. LGTM.

LGTM


feat(cluster): added support for enablePDB option

Add support for enablePDB configuration option
Fix https://github.com/cloudnative-pg/charts/issues/395

@mbugeia Could you add the option to the non default configuration test that verifies that options are passed correctly?
https://github.com/cloudnative-pg/charts/tree/main/charts/cluster/test/postgresql-cluster-configuration
You'll need to set it to false in the cluster values and add the option in the assertion.

@itay-grudev Thanks for the review, I've added the test. Unfortunatly I'm not sure if it works as I don't know how to run the tests locally :see_no_evil:


Linker Error (SDL2)

I've installed sdl2, corrade and magnum via from the arch community repository.
When I tried to build the bootstrap (base) project I received the following error:
[ 50%] Linking CXX executable MyApplication
/usr/bin/ld: /usr/lib64/libMagnumSdl2Application.a(Sdl2Application.cpp.o): relocation R_X86_64_32 against `.rodata.str1.8' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Nonrepresentable section on output
collect2: error: ld returned 1 exit status
make[3]: *** [src/CMakeFiles/MyApplication.dir/build.make:101: src/MyApplication] Error 1
make[2]: *** [CMakeFiles/Makefile2:86: src/CMakeFiles/MyApplication.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:98: src/CMakeFiles/MyApplication.dir/rule] Error 2
make: *** [Makefile:118: MyApplication] Error 2

I've also tried to add -fPIC to the linker flags but it didn't work.
Do you have an idea what the problem might be and how I can fix it?

I've installed hardening-wrapper which overwrote the c++ compiler in my path.


Hello-world-nginx fails on Kitematic

Expected behavior

I clicked on the  hello-world-nginx  which comes up as recommended  in Kitematic
I just wanted to check my Docker install to see if it is working
It failed -- see below
I then clicked [Create] for the plain-old official "hello-world" image
That fails also

As a check:

On command line:

D:> docker run hello-world

Hello from Docker!
This message shows that your installation appears to be working correctly.

To generate this message, Docker took the following steps:
 1. The Docker client contacted the Docker daemon.
 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
    (windows-amd64)
 3. The Docker daemon created a new container from that image which runs the
    executable that produces the output you are currently reading.
 4. The Docker daemon streamed that output to the Docker client, which sent it
    to your terminal.

To try something more ambitious, you can run a Windows Server container with:
 PS C:\> docker run -it microsoft/windowsservercore powershell

Share images, automate workflows, and more with a free Docker ID:
 https://cloud.docker.com/

For more examples and ideas, visit:
 https://docs.docker.com/engine/userguide/

Working out of the box

Actual behavior
**hello-world*9
(just a BLACK screen)

when I click Start (again)..
Some kind of error or diagnostic message flashes and it is then
Occulted by the blackness.

IF you MUST black-out the page at least make it a bit transparent so I can tell you what it said

hello-world-nginx
failed to register layer: re-exec error: exit status 1:
output: ProcessBaseLayer C:\ProgramData\Docker\windowsfilter\611772198a73c434e8f0ca623036094c9ef56c6e09c8649735c09aeb47bf1d30:
The system cannot find the path specified.

Information

Diagnostic ID from "Diagnose & Feedback" in the menu.

OK this was very confusing since this is Kitematic issues
I suggest you change this text to say: "Docker widget Diagnose & Feedback" (please)

a reproducible case if this is a bug, Dockerfiles FTW
page URL if this is a docs issue or the name of a man page

From  the docker diagnose & feedback page:
A diagnostic was uploaded with id: 56AAF1A8-0491-4F9F-8FF4-F87957E2AF5E/2018-02-21_10-54-25
host distribution and version (Windows version, build number, etc)

Windows 10 64-bit
Docker version 17.12.0-ce-win47 (151139)
Engine:  17.12.0-ce
Machine:  0.13.0
Compose:  1.18.0
Notary: 0.4.3
Credential helpers: 0.6.0

Steps to reproduce the behavior

Open  kinematic
Click on either image mentioned

Related information:

The kitematic/hello-world-nginx fails on command line as well.

Normal Docker hello-world works satisfactorily.
And Not from Kitematic

I've filed a bug report, here:

Hello-world-nginx fails on Docker-for-Windows (command line) #3599

I'm unable to reproduce - Is this only happening in Docker for Windows?

Still fails on the Windows command line.  Seems OK on Linux.

I had the same issue but sorted it by switching to using Linux containers by right-clicking the docker sys-tray icon.

Docker Desktop Community, v<IP_ADDRESS> (31259)
Still happening with Server Version 18.09.2, with a "Context Error" popup message when you click on the "Create" button.
Note: I did log into Docker Hub before running Kitematic
Still works from command line


RC Test: Gene Expression Essentials 1.0.0-rc.3

Gene Expression Essentials v1.0.0-rc.3 is ready for RC testing. This is the 3rd RC for this sim, and resolves a couple minor issues found during the preceding RCs.  The changes to the code were quite localized, so verification of these issue and a spot test on our major platforms should suffice.
Link to sim
Link to iFrame
Issues to Verify

[x] https://github.com/phetsims/gene-expression-essentials/issues/114
[x] https://github.com/phetsims/gene-expression-essentials/issues/117

Thanks! Closing.