[ConfigTransformer] Reproducer Expected to find class AppTestsBehatDemoContext in file

The following package is currently failing:
https://github.com/symfony/recipes-contrib/tree/main/friends-of-behat/symfony-extension/2.0
With the error:

friends-of-behat/symfony-extension/2.0/config .................................................................................................................................................................................... FAILED
The command "'/opt/homebrew/Cellar/php/8.1.8/bin/php' '/Users/tmp/Documents/Projects/symfony-php-recipes-contrib/vendor/bin/config-transformer' 'switch-format' '/Users/tmp/Documents/Projects/symfony-php-recipes-contrib/friends-of-behat/symfony-extension/2.0/config'" failed. Exit Code: 1(General error) Working directory: /Users/tmp/Documents/Projects/symfony-php-recipes-contrib Output: ================ Error Output: ================ In FileLoader.php line 196: Expected to find class "App\Tests\Behat\DemoContext" in file "/Users/alexan derschranz/Documents/Projects/symfony-php-recipes-contrib/friends-of-behat/ symfony-extension/2.0/tests/Behat/DemoContext.php" while importing services from resource "../tests/Behat/*", but it was not found! Check the namespac e prefix used with the resource. switch-format [-n|--dry-run] [-c|--config CONFIG] [--]

I actually never saw the asterisk in the PHP syntax:
-"../tests/Behat/*"
+"../tests/Behat"

Would this help?

I used the wrong syntax for commit fix :) my bad. I'll handle this one locally. Just need to know, what is expected output.

I get it. The conversion should happen as provided, just now it crashes on error half way.
The YAML tries to autoload directory that does not exist.

Thats correct. Sorry forget to update the description after I did found out what is happening 🙈

I actually never saw the asterisk in the PHP syntax:
-"../tests/Behat/*"
+"../tests/Behat"
Would this help?

Think there is no difference but as you found out the issue is another place. The * is used in symfony in the docs: https://symfony.com/doc/current/service_container.html strangely not in the recipe services.yaml recipe file: https://github.com/symfony/recipes/blob/main/symfony/framework-bundle/6.2/config/services.yaml but some other packages using the asterisk.

Think there is no difference but as you found out the issue is another place.

Yea, I've tried it locally in PHP config for this pacakge and this part is irevelant.

Thats correct. Sorry forget to update the description after I did found out what is happening see_no_evil

No worries, it's always digging unexpected results :)
Btw, just 1 fixture at a time makes it easier to process for me. If there are 2, I start to think there are 2 or 3 issues at once and confused :confused: :laughing:

@TomasVotruba actually It are 2 issus. The services_test.yaml throws an unexpected class error. While analysing that I thought that it should also not error if the resource was not found (services_ignore_none_exist_resource.yaml) and just convert that without erroring.

PS: Found some discussion about the asterisk here: https://github.com/symfony/recipes/pull/802#discussion_r455856843 and created a update pull request for the symfony docs:  https://github.com/symfony/symfony-docs/pull/17167 to remove the not requried asterisk.


Separate Adform adapters descriptions

Separating descriptions about Adform Prebid.js and Adform Prebid Server adapters

@Pupis - please help us understand why AdForm wants to separate these files.
One of the things publishers want to do is A/B testing: checking performance of a bidder client-vs-server side. To support this, we highly encourage bidders to accept the same set of parameters for their Prebid.js and Prebid Server adapters.

Hi,
The reason for this separation is - Adform Prebid Server adapter does not support video and this documentation is misleading.


Player location not updating

Fresh installation of mod. Map loads, but player location appears to be fixed to its initial state in the save file. Loading a few different saves moves the cursor, but fails to update. Vehicle locations are also static and appear wherever they are in the loaded save file.

It sounds as though the web app lost contact with the game. This can happen if the game is unresponsive for a while and the webapp gives us trying to contact it.
Al @OrihalconZA has said, open the menu and ensure "game updates" is enabled.


Add logger option

Motivation
Right now Lodestar uses js-libp2p-gossipsub but there is no way to incorporate gossipsub log into Lodestar's log which make it hard to debug
Description

Right now js-libp2p-gossipsub uses this log https://www.npmjs.com/package/debug#output-streams
Would be great if it has a logger option following an interface so that application can go with its own log, if not then fallback to a default logger

@tuyennhv to enable gossipsub's logger you need to run lodestar with an ENV variable like DEBUG=libp2p:gossipsub such that
DEBUG=libp2p:gossipsub node lodestar.js

@tuyennhv to enable gossipsub's logger you need to run lodestar with an ENV variable like DEBUG=libp2p:gossipsub:* such that
DEBUG=libp2p:gossipsub:* node lodestar.js

@dapplion yeah that's what I did, I successfully incorporated gossipsub logs into lodestar's log files in my branch


Dialog Primitive: Invalid hook call. Cannot read properties of null (useRef)

Bug report
Current Behavior
When importing the Dialog.Root Primitive anywhere inside the React App, I'm getting the following errors:

Expected behavior
No runtime error.
Reproducible example
Freshly generated Vite (React) + TypeScript project using:
"react": "^18.2.0", "react-dom": "^18.2.0",
Your environment

Software
Name(s)
Version

Radix Package(s)

React
x

Browser

Assistive tech

Node
n/a

npm/yarn
npm

Operating System

How did you fix this?


dataset errrors

There is a minor problem with the provided data, /data/Off-Road/testing/y0613_1242/dense_depth/1623721591686.png are missing

I got the same problem. The author may fail to upload some data due to the baidudrive’s limit.
I recommend that the author compress the entire folder and upload it.

There is a minor problem with the provided data, /data/Off-Road/testing/y0613_1242/dense_depth/1623721591686.png are missing

Sorry.
I have uploaded the compressed ORFD dataset.


Dynamic entities/menu/dashboard

It seems impossible to configure ng-admin after the page has loaded.
I'm trying to do the following workflow:

Have the user authenticate with https://github.com/akagadovskiy/ng-admin-jwt-auth
Request the menu and entities (+ dashboard) the user has access with with the logged in api
Then display ng-admin

I tried the other tickets:

https://github.com/marmelab/ng-admin/issues/1026
https://github.com/marmelab/ng-admin/issues/1022
https://github.com/marmelab/ng-admin/issues/875

Which all seems to result in the same errors:

ng-admin-jwt-auth doesn't load as I haven't loaded ng-admin yet
ng-admin doesn't load because baseApiUrl is not set
ng-admin loops indefinitely in a loop
ng-admin doesn't know about the menu/entities

It might have to do with me missing some knowledge as I'm new to angular (I hoped I didn't need to read all the in depth documentation of Angular before being able to use ng-admin). Is the above scenario possible?

If I follow: https://github.com/marmelab/ng-admin/issues/1026
It won't add the menu as the menuList is empty when the for loop happens, or when you put the for loop in the callback, they won't get displayed until you click on home.

I got it working in the end with angular-deferred-bootstrap and window.location.reload (to re-bootstrap the application). It might be good to have a full example in the demo application for less experienced javascript developers?

Hello, part of the solution was described in https://github.com/marmelab/ng-admin/issues/1029#issuecomment-209285741.
However this is more an angular issue than ng-admin itself. Using $http injection in the config section does not solve the problem correctly because the asynchronous call is resolved too late to be usable.
Please do post your code snippet in this issue .We should compile a list of recipes in  the FAQ, this one being one coming back quiet often.
I have found that the best and more secure way to solve this however is to move the login page in its own application:

Bootstrap on the login App. This only loads the login app required javascripts
On successful login, GET the required user preferences to customize the menu and view ..etc..
Manually boostrap ng-admin and only at this point load ng-admin javascripts

Here is my code, this uses a slightly modified version of https://github.com/akagadovskiy/ng-admin-jwt-auth but I think it is still clear how it works. I can also post the code for building the menu from JSON is that is interresting for the docs @Phocea .
var angular = require('angular');
var deferredBootstrapper = require('angular-deferred-bootstrap');

var apiUrl = 'http://localhost:8080/v1';

var app = angular.module('adminApp',
    [require('ng-admin'), require('ng-admin-auth')]
);

app.config(['NgAdminConfigurationProvider', 'ngAdminJWTAuthConfiguratorProvider', 'NG_ADMIN_CONF', function (nga, ngAdminJWTAuthConfigurator, NG_ADMIN_CONF) {
    ngAdminJWTAuthConfigurator.setJWTAuthURL(apiUrl + '/auth/');
    ngAdminJWTAuthConfigurator.setCustomAuthHeader({
        name: 'Authorization',
        template: 'Bearer {{token}}'
    });
    var admin = nga.application('Admin').debug(false).baseApiUrl(apiUrl + '/');
    ngAdminJWTAuthConfigurator.setLoginSuccessCallback(function() {window.location.reload();});

    admin.header(require('../header.html'));

    if (NG_ADMIN_CONF) {
        // construct menu from the JSON in NG_ADMIN_CONF
    }
    nga.configure(admin);

}]);

app.run(['$location', 'NG_ADMIN_CONF', function($location, NG_ADMIN_CONF) {
    if (NG_ADMIN_CONF && $location.path() == '/login') {
        $location.path('/dashboard');
    }
}]);

deferredBootstrapper.bootstrap({
  element: document.body,
  module: 'adminApp',
  resolve: {
    NG_ADMIN_CONF: ['$http', '$q', function ($http, $q) {
        if (localStorage.userToken) {
            return $http.get(apiUrl + '/config.json', {'headers': {'Authorization': 'Bearer ' + localStorage.userToken}}).error(function (response) {
                localStorage.removeItem('userRole');
                localStorage.removeItem('userToken');
                localStorage.removeItem('user');
                window.location.reload();
            });
        } else {
            return $q(function(resolve, reject) {resolve();});
        }
    }]
  }
});


Architecture Diagram

For comment, to be added to the architecture page

Maybe add web browser to the left-most orange box?  "e.g. web browser, curl, HTTPie"
Meaning the raw output, rather than the tiled web UI.  It can be a simple baseline debugger / get the metadata...but it can open up a can of worms with auth involved.  Thus I'm a bit conflicted about recommending it.

Do the ASGI dotted lines at the top make use of a httpx AsyncClient / starlette TestClient (or something similar)?

Yes, thought I might put it the other way around:  httpx AsyncClient / starlette TestClient make use of ASGI.
I have the same conflicting feeling about web browsers. It tells a nice story, but may get messy with auth.


Programmatically inserting custom objects?

Hello,
Is it possible to insert custom models into this simulation via the python interface?
Thanks

This is not currently possible via the Python interface, but it would be possible with a bit of effort. It would involve getting your custom objects imported as uasset files into our Unreal project (i.e., SpearSim.uproject), writing a new C++ function to spawn a user-specified asset at a user-specified location, and then exposing the C++ function to our Python interface. If you're interested in giving this a try, the best place to start would be going through our Getting Started and Building SpearSim tutorials. Once you're finished these tutorials, post here and we can guide you from there.


ApplicationProtocolNegotiationHandler should drain buffer messages on channel close

Motivation
ApplicationProtocolNegotiationHandler buffers messages which are read before SSL handshake complete event is received and drains them when the handler is removed. However, the channel may be closed (or input shutdown) before SSL handshake  event is received in which case we may fire channel read after channel closure (from handlerRemoved()).
Modification
Intercept channelInactive() and input closed event and drain the buffer.
Result
If channel is closed before SSL handshake complete event is received, we still maintain the order of message read and channel closure.

/cc @hyperxpro

I will pull this in once the CI pass... thanks @NiteshKant


add versions with constraints

Fixes #470
@kaniini can you validate that this actually resolves what you were looking for? Specifically, look at the test case for version resolution here.

Got it rebased, it messed up the order of package installation. I will figure it out, resolve, and push it out.

Sounds good! :+1:

OK, fixed that. Letting CI have a run and tell us, and good for your review @kaniini

It's a really good thing I added that examples/alias-only.yaml, and added testing all examples in CI. It found a regression in this change (which I have fixed, of course).

I always like "clean and green" CI 😁


'spike-in   organism count' is resuing  MIXS:0000103 'organism   count'  with different semantics

If the food-animal and food-farm package submitters would like a 'spike-in   organism count' term, they must create a new term wholesale, not attempt to situationally override the fundamental semantics of MIXS:0000103, 'organism   count'
Of course, we can help them, and all of the MIXS:0000103, 'organism   count' attributes can be copied over, but a new MIxS ID must be assigned.
Or they can reuse  MIXS:0000103, 'organism   count' as-is
Only Example, Requirement and Preferred unit can be customized in the packages sheet

Not edited for now

The did a good job regarding these two different serovar/serotype terms:

MIXS:0001157
MIXS:0001168

'spike-in organism count is: MIXS:0001335
To Edit:
update MIxS ID is 'food-animal and animal feed' and 'food-farm environment' packages.
Cheers,
Lynn


In one suite can be parsed only 2 and more testcases, but not only one

Try use json output for better handling with object.


Use tdays from estimators for time in title

Pull Request Test Coverage Report for Build 272

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 58.391%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

artistools/estimators.py
0
4
0.0%

Totals

Change from base Build 270:
-0.03%

Covered Lines:
2947

Relevant Lines:
5047

💛  - Coveralls


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1636 ms

Resolved: Bofrost.at performance has improved in 854dd83 after 6 minutes.


CIP-110 Ceramic Anchor Contract

cip: 110
title: Ceramic Anchor Contract
author: Sergey Ukustov<EMAIL_ADDRESS>discussions-to: https://github.com/ceramicnetwork/CIP/issues/110
status: Draft
category: Standards
type: Core
created: 2021-10-14
edited: 2021-10-14

Simple Summary
We can open other anchor service operators to participate in Ceramic anchoring using an allowlist as a smart contract on Ethereum blockchain.
Abstract
This improvement proposal presents a Ceramic Anchor Contract. It is a novel way to make anchors on Ethereum network.
Motivation
As of today, anchoring on Ceramic is done independently by each Ceramic Anchor Service (CAS). An anchor is a transaction from a wallet to itself with data, that represent a CID of a Merkle root.
While being low cost, such a construction presents a security risk, especially for Ceramic mainnet. A malicious CAS might withhold IPFS data necessary for proving an anchor (i.e. Merkle tree elements put on IPFS), thus making a Ceramic stream effectively invalid. We would like to make anchor services available for running by anybody, but this risk we would like to mitigate. Thus, until a validator network is in place, we would like to have a known set of trusted anchor services. We would like though the list to be open for anybody to join.
Specification
We can achieve that using a smart contract on Ethereum network (as our main network used for anchoring) that contains an allowlist of trusted anchor services. Only an allowed CAS can make an anchor transaction. The allowlist can be modified, preferrable by a DAO.
Contract Features

contract should be "ownable"

an owner can transfer ownership to other address
this allows us to start with just a single externally-owned address as an owner, and later transfer ownership to a DAO

contract should maintain an allowlist of allowed CAS accounts

only the contract owner can modify the allowlist
"modify" means "add" or "remove"
the contract should emit appropriate events for both adding and removal
an allowed address can remove itself

contract has "anchor" function

only an allowed address can call the function
the function should accept a CID of the merkle root
the function should emit an appropriate event

Implementation details
The contract should be written in Solidity language, using Truffle or Hardhat toolboxes. It might use OpenZeppelin library.
For "ownable" feature we are going to extend our contract Ownable from OpenZeppelin. The functionality provided there is good enough, so we do not elaborate on that topic further.
The allowlist is maintained as mapping (address => bool). We use two functions to modify the allowlist: addCas and removeCas, both emit a respective event:
mapping (address => bool) allowlist;

event DidAddCas(address indexed _service);
event DidRemoveCas(address indexed _service);

// Only owner can add to the allowlist
function addCas(address _service) public onlyOwner {
    allowlist[_service] = true;
    emit DidAddCas(_service);
}

// Removal can be performed by the owner or the service itself
function removeCas(address _service) public {
    require((owner() == _msgSender()) || (allowlist[_msgSender()] && _msgSender() === _service), "Caller is not allowed or the owner");
    delete allowlist[_service];
    emit DidRemoveCas(_service);
}

The "anchor" function can be called by an allowed service only. Its responsibility is limited to just emit an event:
// Only an address in the allowlist is allowed.
modifier onlyAllowed() {
    require(allowlist[_msgSender()], "Allowlist: caller is not allowed");
    _;
}

event DidAnchor(address indexed _service, bytes _root);

// Here _root is a byte representation of Merkle root CID.
function anchor(bytes calldata _root) public onlyAllowed {
    emit DidAnchor(_msgSender(), _root);
}

Anchoring Process
We maintain an authoritative list of anchoring contracts across Ethereum networks: Ropsten, Rinkeby, mainnet. This list is shared by CAS and Ceramic nodes.
CAS should use a contract on a network of choice for anchoring. When validating an anchor record, Ceramic node should check if a transaction happens on the authoritative contract, and the emitted event contains a merkle root.
Backwards Compatibility
Contract-based anchoring is not backwards compatible. We have to delineate between version of the anchors. We add a version property to AnchorCommit payload. Contract-based anchors use version: 1. If version is omitted, we expect the proof to be a raw transaction, and we check if it originated from one of the old well-known CAS addresses. After some time, to be defined later when CIP is accepted, Ceramic network only allows version: 1 proofs.
Rationale
An allowlist is for CAS instances, it allows them to do blockchain transactions. Thus, it is natural, that it is implemented as a blockchain smart contract.
Mainly we use Ethereum network now for anchoring, so Ethereum-based allowlist is a natural choice. It does not prevent to use other blockchains in future.
Single Ethereum transaction could be considered expensive. With the outlined design we effectively equalize the costs through multiple co-existing CAS instances.
Implementation
No implementation yet.
Security Considerations
Ideally, the contract should be independently audited to mitigate smart contract risks.
We consider allowed CAS to behave well, which is only enforced on social consensus layer until fully decentralized validators network is in place.
Copyright
Copyright and related rights waived via CC0.

CIP-110 is now Final. Closing this issue.


Suggestion: Need for proper documentation

There should be a site showing documentation on how to use this tool

Yes, indeed, we are still working on the landing and documentation pages


Main branch is not protected from pushes

Hi,
Apologies. I accidentally did a git push to the main branch from some features I was working on and it wasn't rejected. I then tried to force push back to where we were before my push but that was rejected :upside_down_face:. I ended up just adding 3 revert commits for what I was doing so we should be back to a stable position. At some point it would be nice if we could add a rule to only allow merges to the main branch from PRs.

I think (now that we are in a GitHub organization) you should have the necessary admin permissions to change these settings on the repo, and I don't mind if you do. For now, I left direct pushes allowed, but also configured it so that force pushes are allowed. If you think it's better to disable both, I'm fine with that.

Neat, didn't notice. I think for a multi-user org we should have rules in place preventing any one person from going rogue and doing something to the detriment of other users. I've gone ahead and disabled force pushing and enabled the option to only allow main branch pushes from pull requests.

Sounds good to me.


GPU process isn't usable. Goodbye.

Unable to start the app on 6.1.2-arch1-1 with the latest Nvidia 525 drivers.
[3242:0103/084128.559210:FATAL:gpu_data_manager_impl_private.cc(445)] GPU process isn't usable. Goodbye.
Trace/breakpoint trap (core dumped)

It does seem to run when using ./sixgrid --no-sandbox but that doesn't seem like an ideal solution.

Seems like this is an electron related issue, and from looking at issues on other electron projects that have the same error using --no-sandbox does fix it for Electron 11.x
I also found that (maybe) adding --in-process-gpu might fix this, but I'll see if that does when I get home. (src)

Can confirm that ``--in-process-gpu``` also works

Is that just with ./sixgrid --in-process-gpu or ./sixgrid --no-sandbox --in-process-gpu?

Just with ./sixgrid --in-process-gpu

@faucetlol2 Can you try running sixgrid with no arguments with this release?
https://github.com/SixGrid/sixgrid/releases/tag/v0.3.6-3-bug-gpuprocessisntusable

That works!


[BUG] Cannot print a string represented List[str]

Describe the bug
I really like how this module works, so I use it in my script to get better output when running in the shell. But today, I found that rich seem to have some problems when rendering.
When I tried to use rich to print subprocess.TimeoutExpired , I did not get the output I expected. Is this a bug?
except subprocess.TimeoutExpired as err:
        rich.print(f"TimeoutExpired: {str(err)}")

To Reproduce
from rich import print
print("'['arg1', 'arg2']'") # will only get ''
print("['arg1', 'arg2']")   # will get nothing

Platform
platform : Linux (ubuntu 18.04)
terminal : bash
Python : 3.6.9
rich : 4.0.0

Anything in square brackets is interpreted as console markup. See the docs for details how to escape, or disable console markup.


Is there a way to know the name of the chapter where the highlight comes from?

Not that I am aware of.

@speric Hi Eric
Where would I ideally start to implement this feature in order to contribute here?


Farthest territory calculation not working

This is a huge problem, but most of the cases it happens.
Only if Háromszék is the farthest territory in the real game, it finds it, but otherwise it finds a wrong county instead, which can be even itself (!), but if not, then also one of its neighbours.

Undefined would be "farthest territory from". And the furthest territory from itself is itself, it says. So 'tere're some problems, that's for sure.
Also this undefined thing is not pleasure, because I've no idea, why doesn't it finds the translation.

A few commits ago, I thought I solved the problem with adding the criterium when returning with the farthest territory that it shouldn't be itself.
But it seems, it didn't solved the problem completely, because it can select still a neighbouring county, like here:

The problem was that on each neighbouring territory, the function broke from the loop, and stopped checking the distance between other points of the outline of the examined territory and the solution territory's outline, BUT it also stopped getting the dimensions of the examined territory, and hence, the farthest territory calculation broke, since it needed the dimension data.

It has been solved since 40e4088

So I'll close it for now, hope won't reopen it while testing...


Adds support for India, Canada and much more

Area code formats now specified so that countries that share a country code can be supported, and invalid area codes (ex: 555 in North America) are recognized as invalid
Local number formats also specified. Allows for better country detection if the country differs from the default country code.
Parsing largely rewritten to better identify countries on parse.
Tests now clear global data between tests (default_country_code & default_area_code were being persisted).
German numbers are now parsed correctly.

There is already a 1.1 version, it's here: https://rubygems.org/gems/phone   but not on GitHub.  We are having our own issue with the gem as well..  I am hoping this is still maintained?

+1 for merging this.
Our testing shows that this new version handles US numbers better than the upstream.  e.g.<PHONE_NUMBER>2 != +118005551212, #parse("").to_s doesn't return "" but nil. Several other tests behaving and parsing as expected.


fix(ontology): a system admin should be able to update an ontology (DSP-1621)

resolves DSP-1621

@kilchenmann thanks for quick review :)

@SepidehAlassi Thank you for the quick fix 👍


⚠ Bofrost.hr has degraded performance

In 2eda829, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 4120 ms

Resolved: Bofrost.hr performance has improved in 8a8833b after 9 minutes.


Use the same default timestamp for converted data in convert-cisagov

🗣 Description
This pull request updates the convert-cisagov command to use the same default timestamp for any rows missing a timestamp when converting.

💭 Motivation and context
The conversion process is logically one moment in time and so all entries that need a timestamp should use the same timestamp. Though this is not particularly relevant since we are post-conversion, this mirrors how NCSC-NL data will be timestamped when that is updated and deployed.

🧪 Testing
Automated tests pass. I did a test import with an old version of the Markdown source file and it worked as expected.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 29.459%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/mdyml/convert_cisagov.py
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
103

Relevant Lines:
344

💛  - Coveralls


Some help needed

Hi Alex,
I came accross your post https://medium.com/@alexandrumarius/cloud-native-geoserver-deployment-on-kubernetes-gke-241e04df6494 and ended up in this repo.
I'm wondering if

Would consider to bumping this to 1.9.0 version of geoserver-cloud?
Would have the time to help me with a little problem I'm experiencing namely that after logging in to geoserver and navigating my SESSION cookie gets deleted. I'm stuck in a login / logout on navigation loop. The only thing that differs in my setup is that I don't use nginx.ingress.kubernetes.io/configuration-snippet in the ingress.

Anyway. If you would have some time I'd love to see if we can make this work.

Me knowing little about geoserver is probably the main problem. I believe I might have stumbled on what the problem is.
Based off
https://speakerdeck.com/simboss/scaling-geoserver-in-the-cloud-clustering-state-of-the-art-foss4g-2023
and
https://github.com/camptocamp/helm-geoserver-cloud/blob/master/examples/pgconfig-wms-hpa/Chart.yaml
"Basic example with Horizontal Pod Autoscaling applied to WMS instances in geoserver-cloud using pgconfig as catalog persistence component"
It seems that the maybe not all services should use HPA and what I was experiencing was the ingress round robin to the replicas where each replica was treating me as a new visitor. Sessions are not stored and shared in the nfs setup as everything is in memory. (Maybe they are in the PG setup)

I have bumped it to 1.9.0 and removed HPA from the config.
It's plausible that HPA is causing the session cookie to be lost. If you remove HPA do you still have the login/logout bounce?
Would say the pg backend will always be better than the datadir backend.

Hi Alex,
I’m comparing notes with https://github.com/camptocamp/helm-geoserver-cloud/tree/master/examples and an opinionized version of it https://github.com/jemacchi/gscloud-in-nutshell and from that I can see that the only thing that requires HPA is the WMS service. You’re right in that the HPA for the other services is causing the session as sessions are kept in memory and since the calls are load balanced your next call lands you in an instance that knows nothing about your SESSION thus logging you out. After turning off HPA for all services except WMS it all works fine.
I’m working on a pgconfig kustomize setup and I can ping you once it’s done. I also got recommendations that in a non data datadir profile you don’t need the discovery and config service so I removed them and things work well. I still have some cleanup to do but it’s up and running now.
As I don’t use Helm as much as I should your example helped me to get things into Kustomize.
Best,
Martin
From: AlexTzk @.>
Reply to: AlexTzk/geoserver-cloud @.>
Date: Monday, 16 December 2024 at 15:31
To: AlexTzk/geoserver-cloud @.>
Cc: Martin Malek @.>, Author @.***>
Subject: Re: [AlexTzk/geoserver-cloud] Some help needed (Issue #1)
I have bumped it to 1.9.0 and removed HPA from the config.
It's plausible that HPA is causing the session cookie to be lost. If you remove HPA do you still have the login/logout bounce?
Would say the pg backend will always be better than the datadir backend.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Martin,
Will include the HPA details in the notes and re-enable that functionality for WMS.
I am happy to hear that this repo has helped you in your geoserver-cloud quest. Feel free to send a PR if you want to contribute anything at all.


🧑💻:  GIT AND GITHUB LEARNING TRACK

Title
GIT AND GITHUB LEARNING TRACK
Enhancement Aim
WE SHOULD ALSO INCLUDE GIT AND GITHUB LEARNING TRACK SO THAT LEARNERS CAN LEARN
Changes
ADD GIT AND GITHUB LEARNING TRACK
Screenshots 📷
No response
Full Name
MEET SHAH
Participant Role
CONTRIBUTER
Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

/assign

@rishicds Can i work on this ?

assigned,follow code structure, if you run into any issues ask @pratheekv39

@rishicds Please Review PR..

@rishicds Plrease Review PR..


Asp.Net Core multiple authentication schemes

Describe the bug
Hello all, I have a question regarding the support for multiple authentication schemes in HotChocolate v11.0.6.
I am working on a multi-tenant application using Azure AD for users authentication and I configured the required services as described here: https://github.com/AzureAD/microsoft-identity-web/wiki/web-apis#using-multiple-authentication-schemes
//First tenant
services.AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
            .AddMicrosoftIdentityWebApi(Configuration, "AzureAd")
            .EnableTokenAcquisitionToCallDownstreamApi()
            .AddInMemoryTokenCaches();

//Second tenant
services.AddAuthentication()
            .AddMicrosoftIdentityWebApi(Configuration, "AzureAdB2C", "B2CScheme")
            .EnableTokenAcquisitionToCallDownstreamApi();

services.AddAuthorization(options =>
            {
                options.DefaultPolicy = new AuthorizationPolicyBuilder(JwtBearerDefaults.AuthenticationScheme, "B2CScheme")
                    .RequireAuthenticatedUser()
                    .Build();
            });

....

services.AddGraphQLServer()
            .AddAuthorization()
            .AddQueryType(d => d.Name("Query"))
            .AddTypeExtension<SomeQueries>();

Only the users from the first tenant are able to query graphql, for the users from the second tenant I get "The current user is not authorized to access this resource."
Is this scenario supported? If not, can you advise me on how to implement it?
Thanks in advance.

I had to add a "DoubleSchemeAuthenticationHandler" and register it as its own schema. In HandleAuthenticateAsync there I call this.Context.AuthenticateAsync() passing first schema name and on failure, second schema name.

I had to add a "DoubleSchemeAuthenticationHandler" and register it as its own schema. In HandleAuthenticateAsync there I call this.Context.AuthenticateAsync() passing first schema name and on failure, second schema name.

How it is related to graphql? I mean, the authentication pipeline seems to work correctly except for graphql; for example controllers decorated with the [Authorize] attribute allow the access to both users from both tenants.

Some more information.
I was able to debug the AuthorizeMiddleware in HotChocolate.AspNetCore.Authorization and it seems that the ClaimsPrincipal that it gets from IDirectiveContext.ContextData in the method TryGetAuthenticatedPrincipal  is empty when I perform a request using a token that belongs to an users from the second tenant.

So I found the culprit of the issue...
I was missing a call to RequireAuthorization() after MapGraphQL():
        app.UseEndpoints(endpoints =>
        {
            endpoints.MapControllers();
            endpoints.MapGraphQL()
                     .RequireAuthorization();
        });


Forward sigint to all processes

Currently only systemd get SIGINT from docker during restart and shutdown of container. Need to forward signal to all processes to speedup restart and dont use SIGKILL

https://github.com/xtrime-ru/antizapret-vpn-docker/commit/c96335dc0373712ffe619897f2dd8118ebc64d0f


refactor/remove-regular

Please review before merging.

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

GitHub release
v3.4.0

Your semantic-release bot :package::rocket:


Undeclared variables in examples/rasterize.js

Which version of PhantomJS are you using? Tip: run phantomjs --version.

Bug is in example js file rasterize.js

What steps will reproduce the problem?

phantomjs examples/rasterize.js  images/001.png '1200*1000px'

Please provide any additional information below.

2 of the variables aren't declared, only complains when you use the 'px' code route

Duplicate. See #13976


ews release 3.0 planning

For a possible 3.0 release I see the following features at the skyline:

Security / and current vulnerabilities (already submitted issue-tickets / more details to be outlined)
Java 8 integration and review of used 3rd party apis
Identify major gaps between ews-managed-api and ews-java-api
Fix: Token identification
Fix: project build automation

I so far threat this as an open list. Feel free to provide further comments and suggestions here.

Created to further outline this call for action


Find the enc key please

Coverage remained the same at 99.815% when pulling 431a14a8e8026e6d435ecfd10a1cdcbde1097fb8 on basvodde:master into baba5a7c616afe83a7eaabba28417925c27d7dbc on cpputest:master.


Nuxt does not polyfill when function is used only in template

Versions

nuxt: v2.14.4
node: v12.18.3

Reproduction
https://github.com/simonbrent/nuxt-polyfill-bug
Steps to reproduce

Clone the repo above
yarn install
yarn dev
Open localhost:8001/working and localhost:8001/broken in Internet Explorer 11
Click the button on each page

What is Expected?
The number on the button is incremented on both pages
What is actually happening?
The number on localhost:8001/working is incremented, but when you click the button on localhost:8001/broken, you get an error page with the message "Object doesn't support property or method 'isNaN'".
Both pages use Number.isNaN, which requires a polyfill in IE11, but this polyfill is only included when Number.isNaN is used in the page's <script>, as in pages/working.vue, and not when it is used exclusively in the <template>, as in pages/broken.vue.

This issue is still present with the latest version of nuxt-edge. I have updated the repo to use nuxt-edge@latest as a dependency.

This issue is still present with the latest version of nuxt-edge (2.14.8-26743710.2712eb3f)

This issue s still present with the latest version of nuxt-edge (2.15.0-26799875.1269773d)

Hmm ok, but I feel that this is a gotcha which should be documented somewhere...
(I note that it only applies to functions which are not used by Vue/Nuxt already, so, for example, using Object.entries exclusively in a template does not cause the error because presumably that's always being polyfilled by something internal to Vue/Nuxt)

Thanks


Update Cpp HelloWorld example for issue 1797

Pull request title
Integrate C++ Examples into Umple HelloWorld Python Files, Fixes #1797
Pull request description and content
This PR fixes #1797 by incorporating C++ code into two Umple HelloWorld examples, now renamed to HelloWorldExamples1pyAndCpp.ump and HelloWorldExamples2pyAndCpp.ump, and updating 0102HelloWorld.txt to reflect these changes. Changes were locally tested for functionality.

I see Jenkins failed on a concurrent modification exception. I will trigger it again with a tiny change in a few minutes.


Tests are broken on CI

FAIL tests/shared/components/headerIcon.test.js
● Test suite failed to run
SecurityError: localStorage is not available for opaque origins
  at Window.get localStorage [as localStorage] (node_modules/jsdom/lib/jsdom/browser/Window.js:257:15)
      at Array.forEach (<anonymous>)

See:
https://github.com/facebook/jest/issues/6766

Using testURL fixed the bug


Register nulldb adapter on foreigner

:tophat: What? Why?
Foreigner doesn't support https://github.com/nulldb/nulldb to avoid hitting the database when precompiling assets. This way it does.
:ghost: GIF

Sorry, I opened this agains the wrong branch :)


chore: update descriptions for commands

Summary

Use imperative sentences for command descriptions
Use exported functions for commands added to the root

How was it tested?

Use exported functions for commands added to the root

why is this needed?

Use exported functions for commands added to the root

why is this needed?

Not needed, because all functions are in the same package. But most of functions for commands added to the root are exported and sub-commands for these commands are unexported. So I think it's better to follow the pattern.

Oh I see. I think that may be an oversight. In general, we should bias towards not visible unless we intentionally want to export the function.
So, could we instead go the other way, and make the exported Command Functions be internal? I don't see a good reason for why they need to be exported.

Oh I see. I think that may be an oversight. In general, we should bias towards not visible unless we intentionally want to export the function.
So, could we instead go the other way, and make the exported Command Functions be internal? I don't see a good reason for why they need to be exported.

Okay, I've changed all functions to be internal.

@hezhizhen thank you! for your PR and openness to modifying it.


Arrows appear near the Agent name in Edit Agent popup in monitors of 24'

[x] Arrows appear near the Agent name in Edit Agent popup in monitors of 24'

Fixed and working properly

Fixed and working properly


generic type parameters, for generic call expressions, not reduced to canonical form, causing type inference malfuction and errors

🔎 Search Terms

type reduction fails
generic function return type inference fails

Couldn't find any issues that match this one sufficiently.
🕗 Version & Regression Information
This is the behavior in every version I tried up to 5.3.2.
⏯ Playground Link
playground
💻 Code
import { default as $ } from 'jquery';
import JQuery from 'jquery';

const t1: string | JQuery<HTMLElement> | undefined = $(); // no error

const t2: ((string | JQuery<HTMLElement>) & (string | JQuery<HTMLElement> | undefined)) | undefined = $(); // error
//    ~~
// Type 'JQuery<string | HTMLElement>' is not assignable to type '((string | JQuery<HTMLElement>) & (string | JQuery<HTMLElement> | undefined)) | undefined'.
//  Type 'JQuery<string | HTMLElement>' is not assignable to type 'JQuery<HTMLElement>'.
//    Type 'string | HTMLElement' is not assignable to type 'HTMLElement'.
//      Type 'string' is not assignable to type 'HTMLElement'.(2322)

const t11: string | JQuery<HTMLElement> = $(); // no error

const t12: (string | JQuery<HTMLElement>) & (string | JQuery<HTMLElement> | undefined) = $(); // error
//    ~~
// Type 'JQuery<string | HTMLElement>' is not assignable to type 'string | JQuery<HTMLElement> | (string & JQuery<HTMLElement>) | (JQuery<HTMLElement> & string)'.
//  Type 'JQuery<string | HTMLElement>' is not assignable to type 'JQuery<HTMLElement>'.

const JQuery_string = JQuery<string>; // error
// Type 'string' does not satisfy the constraint 'Element'.(2344)
// Type 'string' does not satisfy the constraint 'HTMLElement'.(2344)
// Type 'string' does not satisfy the constraint 'PlainObject<any>'.(2344)

🙁 Actual behavior

Even though the type declarations for t1 and t2 are the same (after reduction), only t2 shows an error.  The same relationship exists between t11 and t12.

The return type of assigned to t1 is  string | JQuery<HTMLElement> | undefined.

🙂 Expected behavior

t1 and t2 should have the same error status.  Likewise for t11 and t12.

The jQuery function $() has a generic signature <TElement=HTMLElement>() JQuery<TElement>.  So the return type assigned to t1 should be JQuery<string> | JQuery<HTMLElement> | JQuery<undefined>.

As shown at the bottom of the code,  JQuery<string> is judged to be an invalid instantiation of JQuery.  If so, the lines with t1 and t2 should both be errors, but they are not.  However, the actual declaration for the interface JQuery seems to be

interface JQuery<TElement = HTMLElement> extends Iterable<TElement>

which actually doesn't look as though it should constraint the domain of JQuery to HTMLElement, so perhaps JQuery<string> should not be an error?   Furthermore, from within a DefinitelyTyped/semantic-ui test file, JQuery<string>  does not incur an error.
Additional information about the issue
I explain where the weird declaration types (e.g.  (string | JQuery<HTMLElement>) & (string | JQuery<HTMLElement> | undefined) ) came from.
The following is quoted and condensed from filles under DefinitelyTyped/semantic-ui-search
import { default as $ } from 'jquery';

interface _Impl {
    stateContext: string | JQuery;
};

type Param =
    & (
        | Pick<_Impl, "stateContext">
    )
    & Partial<Pick<_Impl, keyof _Impl>>;

The type of Param["stateContext"] is indeed the weird type:
type StateContext = Param["stateContext"];
//   ^?  (string | JQuery<HTMLElement>) & (string | JQuery<HTMLElement> | undefined)

The error can then be incurred as follows:
const p: Param = {
    stateContext: $()
//  ~~
// Type 'JQuery<string | HTMLElement>' is not assignable
// to type '(string | JQuery<HTMLElement>) & (string | JQuery<HTMLElement> | undefined)'.(2322)
}

playground

Wow, jQuery is still a thing?

@fatcerberus -
The TypeScript CI tests includes DefinitelyTyped which includes semantic-ui which depends on jQuery`.
I submitted a fix for #56013 in pull #56373.
The bug in #56013 is improperly caching values of generic call expressions, preventing later correct reevaluation of those call expressions with different generic contexts.
I just so happens that the bug fix #56373 triggered correct reevaluations that revealed the bug in this issue (#56702).
The error reports occurred in the typescript CI tests for DefinitelyTypes/semantics-ui - you can see the list of errors here.
Just to be clear, this bug report #56702 does not use the code in pull #56373 at all, it is only using already released versions.
I've analyzed the pre-existing bug and condensed it down a simple example.

@craigphicks To be clear, my comment was a joke - jQuery is very old (2006) and it’s rare that JS middleware remains relevant (let alone actively maintained) for that long! 😄

@fatcerberus - npm jQuery still seems to have million of downloads, whereas semantic-ui is only the thousands, and it is particular usage of $() by semantic-ui in which the latent TypeScript bug is revealed as shown here.

We need a repro of this that doesn't depend on something external as large as jquery

@RyanCavanaugh  - added as an update to original post.  Note the accompanying debugger analysis provided for your enjoyment.

I am closing this because I am adding a little more debugging analysis.  When finished I will repost.


Update docker build and fix ARM64 image

Closes: https://github.com/kubernetes/node-problem-detector/issues/797

/cc @MartinForReal @vteratipally

/test pull-npd-e2e-node

/test pull-npd-e2e-node

@tzneal @vteratipally Could you take one more look? Thanks!

/test pull-npd-e2e-node

/test pull-npd-e2e-node

/pull-npd-e2e-node

Tests are now passing consistently, especially since https://github.com/kubernetes/test-infra/pull/30710 was merged.

/lgtm
/approve
Thank you for the fix.

@vteratipally k/k tried to upgrade to the last release of NPD and reverted due to this issue. Will NPD release a new version in recent days? Or what is the release plan of NPD?

We can release a new version this week to fix the issue

@vteratipally @pacoxu maybe we want https://github.com/kubernetes/node-problem-detector/pull/814 too in the release, to be on the same Go version as k/k.


Aequatus

Please merge aequatus tool, it also includes config files for aequatus datatype

Ready to be examined and merge

Hi,
maybe you can consider to port this tool into JS and write a data-provider for your visualisation. With this in place you would not need a separate tool to make the conversion.

maybe you can consider to port this tool into JS and write a data-provider for your visualisation. With this in place you would not need a separate tool to make the conversion.

@bgruening This is more complicated than a conversion, the aequatus dataset to be visualized is generated from 3 files in different formats (gene JSON, CIGAR tabular and Newick), so I'm not sure how this can be converted into a data provider.

Yes @nsoranzo is right,
Just to add, to visualise these data (gene JSON, CIGAR tabular and Newick) in Aequatus Transcript IDs among all these needs to be match, and if we give user option to load them separately like Trackster, may be possibility that user load data generated from different sets and it can break functionality of Aequatus

As we discussed, this should be ported to Python and merged with the gff3_to_json tool.

As we discussed, I changed perl scripts to python but joining this with gff-to-json will need some more thoughts

How do we want to distribute the tool, datatype and visualization on the Tool Shed? Everything together in the same repository?

@anilthanki I've added some final fixes on your branch, can you check that everything looks for you?


Devstack no longer works in pre-existing repo

% bacalhau devstack
17:15:47.197 | INF pkg/repo/fs.go:81 > Initializing repo at '/var/folders/xz/qrd28pyx1095q2nzvhd9wb_c0000gn/T/2980529133' for environment 'production'
failed to initialize repo: failed to initialize repo: failed to create user key: user key file already exists at path: /Users/simon/.bacalhau/user_id.pem

Probably solved if someone did #3296.

doesn't look like an issue anymore


Pull requests are not identified correctly when running on GitHub Actions

As a result, attempts are made to publish NuGet packages when it shouldn't:
https://github.com/cake-contrib/Cake.Json/pull/63/checks?check_run_id=1990966420#step:5:481
The IsPullRequest parameter is false, when it should be true:
https://github.com/cake-contrib/Cake.Json/pull/63/checks?check_run_id=1990966420#step:5:34

The logic for identifying a PR is:
var headRef = context.BuildSystem().GitHubActions.Environment.Workflow.HeadRef;
        var branchRef = context.BuildSystem().GitHubActions.Environment.Workflow.Ref;

        IsPullRequest = !string.IsNullOrEmpty(headRef) && !string.IsNullOrEmpty(branchRef)
            && branchRef.IndexOf("refs/pull/") >= 0 && branchRef.IndexOf("/merge") >= 0;

However, in an example PR, the following values are in play:
               CI: true
             HOME: [NULL]
       GITHUB_REF:
refs/heads/dependabot/nuget/develop/Microsoft.NET.Test.Sdk-16.9.1
       GITHUB_SHA: 861277b8e81f200042b5d9a358f5e74833c85d97
     GITHUB_ACTOR: dependabot[bot]
    GITHUB_ACTION: cake-buildcake-action
    GITHUB_RUN_ID: 604078695
   GITHUB_ACTIONS: true
  GITHUB_BASE_REF:
  GITHUB_HEAD_REF:
  GITHUB_WORKFLOW: Build
 GITHUB_WORKSPACE: D:\a\Cake.Json\Cake.Json
GITHUB_EVENT_NAME: push
GITHUB_EVENT_PATH: D:\a\_temp\_github_workflow\event.json
GITHUB_REPOSITORY: ***/Cake.Json
GITHUB_RUN_NUMBER: 15

@gep13 the build check you are linking to is not actually the pull request build, but rather a build that gets ran due to the push to a local branch.
I answered a similar (or actually the same question) here: https://github.com/cake-build/cake/discussions/3200

The failure you are seeing should be able to be resolved by a user when #642 is released.

Hmmm.... thanks for pointing this out!
While the skip duplicate would prevent this issue, I also don't want any pre-release packages being pushed when it is a PR build (whether is it the PR branch itself or a local branch from same repository).
I am assuming that this can be fixed by preventing the build to only specific named branches on local repository?

I am assuming that this can be fixed by preventing the build to only specific named branches on local repository?

Indeed, but that is probably something that should be done in a workflow and not in Cake.Recipe itself.
I think the information we have available during the build is too limited to be able to do reliably in Cake.Recipe (would love to be wrong on that though).
If done in Cake.Recipe, we would possibly skip builds that should have been ran.

Am I wrong in saying that we can use the event name, either from the environment variables, or by deserializing the event as done here: https://github.com/benfoster/o9d-guard/blob/main/build.cake and if it is a pull request event, don't do the publishing?

@gep13 not really necessary to check the event name.
There is code already available in the build provider class to check if it is a pull request or not by checking the git branch name (we don't use it when deciding to publish packages).
But there could be adding code to check the event name and fall back to the git branch parsing code already available.
It still wouldn't fix the error you saw, as it isn't considered a pull request since it is started on the repository branch itself, though.

Ok, so now I am getting somewhere...
I was looking at this PR:
https://github.com/cake-contrib/Cake.Incubator/pull/134
And the failing build took me through to this workflow:
https://github.com/cake-contrib/Cake.Incubator/pull/134/checks?check_run_id=1990873969
But this takes me to the push event build, rather than the pull request event build.  The pull request event build didn't actually run, it was skipped.
The question that this leaves is that if I update the build configuration to limit builds to only master/develop/hotfix/release branches, will I still see the pull request event build?  Going to need to play with this to confirm either way.
I think it comes down to the fact that I personally don't want pre-release packages to be published as a result of a dependabot push/pull request event.  I don't think I want to go as far as limiting publishing to the GITHUB_ACTOR which could be an option:
GITHUB_ACTOR: dependabot[bot]

The question that this leaves is that if I update the build configuration to limit builds to only master/develop/hotfix/release branches, will I still see the pull request event build?

If you remove the if statement, then yes the pull request would still build.
These are two different events, and only the push event would be limited by the branch filter (either blacklist, or whitelist).

Did some digging around in the code, and even if everything gets limited we still have something that needs to be updated in Cake.Recipe itself.
The criteria for publishing don't take into account if the current build is a pull request or not:
https://github.com/cake-contrib/Cake.Recipe/blob/develop/Cake.Recipe/Content/packages.cake#L138-L141
https://github.com/cake-contrib/Cake.Recipe/blob/develop/Cake.Recipe/Content/packages.cake#L162-L165
so this is probably something that should be updated.

Yip, sounds like a change is indeed required. Thanks for digging into this. I think there used to be a check, so this is perhaps a regression.

Yeah, either a regression or an oversight from back when we changed from only supporting AppVeyor to multiple build providers.


Get the special blocks done for all 4 colleges.

Specifically:
Nordmetall A Block
Krupp D Block
Mercator D Block
C3 4th floor

I believe we finished this?

I think so.


Check pip version

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

The package reports itself as version 0.0.0:
pip show pystache
Name: pystache
Version: 0.0.0
Summary: Mustache for Python
Home-page:


Export data to CSV or PDF

How to export the table data and selected data to CSV or pdf file.

use the "LocalDataSource" as the source for your table, then just use LocalDataSource's getAll method. From there you have to parse the data and rewrite it as in a CSV-format. If you want to get the selected row save the selected row in a private variable when you "hear" (userRowSelect), then write a method that takes care of passing that data to the parser & writter.


MINOR: Add support for RGBA encoded outline colors in SolidLineTechnique.

Previously SolidLineTechnique outline color was using ColorCache which
does not support alpha channel, this change provides RGBA support for
outline color.
Signed-off-by: Krystian Kostecki<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1277 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1277      +/-   ##
==========================================
+ Coverage   52.94%   52.95%   +<.01%
==========================================
  Files         260      260
  Lines       23390    23387       -3
  Branches     5177     5176       -1
==========================================
- Hits        12385    12384       -1
+ Misses      11005    11003       -2

Impacted Files
Coverage Δ

...e/harp-mapview/lib/geometry/TileGeometryCreator.ts
26.37% <0%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

retest this please


Fixes #44237: Add columnNumber variable

This adds a new variable that you can access from tasks and other places to be able to substitute in the currently selected column number from the active editor.

@microsoft-github-policy-service agree

@alexr00 is there a way I can advocate/lobby for community approvals to my PR?

Closing in favor of https://github.com/microsoft/vscode/pull/65264

Thank you so much for this @alexr00 !
Your work is very much appreciated.

I don't suppose there's a way I can "buy you a coffee" or whatnot? (donate to a charity of your choice?, something else?)

Nope, thank you, but it's always nice to hear that a change is genuinely useful to folks!


batch_accuracy_for_logits Errors out

I have an input of [128, 3, 224, 224] and output of [128, 59], and this is working for training/loss calculation, but when I call model.batch_accuracy_for_logits(input, output, dev, 128) with the same tensors used in training, it causes the below error.
Torch("The size of tensor a (128) must match the size of tensor b (59) at non-singleton dimension 1. Exception raised from infer_size_impl
I may be misunderstanding how to supply the resulting labels since batch needs to be specified

Would you have some minimal way to reproduce the issue? This would help tracking things down. Also note that batch_accuracy_for_logits is used in a bunch of examples, e.g. here so you may want to just print the shape of the tensors passed as argument there, the shape of the tensor that your code passes as arguments and look at how they differ.


fix: upgrade to version 3.1.11

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign hferentschik
You can assign the PR to them by writing /assign @hferentschik in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
0b306e88cd7579962ca1463248936f090d25a779
link
/test vault

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

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
0b306e88cd7579962ca1463248936f090d25a779
link
/test vault

gsm
0b306e88cd7579962ca1463248936f090d25a779
link
/test gsm

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

/retest


请求帮助need help

如何无户口从pan.baidu.com下载？
How to download from pan.baidu without an account. I was directed here because people said i could here.
Link did not work, what was saved was a html file of the download page.
edit: i managed to register a pan.baidu account. I still don't get a working link. Anyone can help?

kinh.cc

I don't think downloading from pan.baidu is related to this project in any way


Benchmark refactored implementation

Need to benchmark a number of cases:

[ ] Many concurrent site builds - at what concurrency will things bog down? Need realistic data and templates for this.
[ ] Large site builds - how long will it take to build a site with a lot of data, and what are the memory usage characteristics of such a build? Need to auto-generate a large (randomized) set of testing data.

I'm less concerned about Large site builds.
Traditionally, Webhook slows down when there are multiple pages, all of our sites will be single page builds, so I am not concerned there.
Let's focus on stress tests pertaining to concurrent builds.


Remove bundler-api (heroku app)

We want to retire the bundler-api heroku app and pull both the dependency api and the new index back into the main rubygems.org rails application.
This will help us to not duplicate code, work, and infrastructure. This will greatly simplify the Fastly config and allow us to focus our efforts on one application.
There are two areas that need work for this to happen:
Dependency API

https://github.com/rubygems/rubygems.org/pull/966
https://github.com/rubygems/rubygems.org/pull/1163

New Index

https://github.com/rubygems/rubygems.org/pull/1067

@segiddins @maclover7 @indirect @fotanus
cc: @evanphx @qrush @arthurnn

Iirc we also need to ensure that rubygems.org has the infrastructure to collect the usage stats that the bundler API currently collects

Are #966 and #1163 ready for review? I can do a read-through if so, but want to hold off if it's WIP.

@qrush Not ready yet. Those two need to be merged into one, rebased, and finished. Very WIP.

Sounds good. Happy to read through once they're a little more fully baked. :bread:

I will work on getting the dependency API, backported. We have a PR already, i will review/merge and finish any remaining work.

Newest PR for dependency API is https://github.com/rubygems/rubygems.org/pull/1225


dtype.type does not work for scalars

Not sure if this is intended but if I use the method you proposed in #1121 compilation fails, because x.dtype.type does not work for scalars. testcase:
import numpy as np
#pythran export test(complex64)
def test(x):
    return x.dtype.type(1+1j)

Ahah I knew this would pop up, but I was lazy to fix this bug :-)


fix: fix a compilation bug when inherit from another module

Description of the changes
Fix a bug introduced by the 2.7 patch
Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[x] I made my own code-review before requesting one

I included unit tests that cover my changes

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

I added/updated the documentation about my changes

[ ] 📜 README.md
[ ] 📕 docs/*.md
[ ] 📓 docs.lenra.io
[x] 🙅 no documentation needed

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


nginx plugin - no default server when SSL enabled

When enabling an endpoint that utilizes SSL, there is no default server for SSL connections.  This causes random backends to be exposed unintentionally.

I am working on identifying what a proper fix would be.  My first thought would be something that can handle modifying https://github.com/ehazlett/interlock/blob/master/plugins/nginx/template.go#L53 to change the default server to serve both http and https.  The problem is that multiple lines would need to be added; similar to:
listen {{ .Port }};
listen {{ .SSLPort }};
ssl on;
ssl_certificate {{ .SSLCert }};
ssl_certificate_key {{ .SSLCertKey }};

I don't know if this breaks how the template would work but there would have to be some changes, especially since there would have to be some way to handle a default SSL certificate as well... ick!  It just seems to get tricky quickly.

I think this is why there was no default on SSL as is.  Let me think on this a bit :)

+1 ran into a problem with multiple ssl vhosts leaking into each other
nginx doesn't check server_name unless it has to disambiguate listen, so by not having a default ssl listen, it ends up picking the first vhost ssl listen even when I'm requesting a different vhost that is no longer in the config.  A good read on it here: https://www.digitalocean.com/community/tutorials/understanding-nginx-server-and-location-block-selection-algorithms

Maybe something like: http://serverfault.com/questions/382606/nginx-serving-content-from-the-wrong-virtual-host-while-accessing-with-https
That is, use the snakeoil cert - to generate on ubuntu, install ssl-cert package, then if installing didn't create the snakeoil cert you can do so with make-ssl-cert generate-default-snakeoil --force-overwrite

This should be fixed with the new ng branch set to be released in a week or so.  Please re-open if you continue to notice the issue.  Thanks!

Has the ng branch have been merged into the master branch, as I am unable to find it anywhere?

@SergiuCip yes it has.  https://github.com/ehazlett/interlock/commit/3a36c7d0ded3a1ea21552977738f7c1aa07bcc45

@mbentley Thanks for clarifying that. You wouldn't happen to know how to bring up multiple SSL NGINX Vhosts with their own SSL certs, as I have tried using the INTERLOCK_DATA as such:
INTERLOCK_DATA='{"alias_domains": ["foo.com"], "port": 443, "ssl_only": true, "warm": true}'     ehazlett/go-demo
but still keeps port 8080 instead of replacing it with 443, and I was wondering how do you pass the certs for a specific site? Something like this INTERLOC_DATA {SSLCertPath = ""} ?
Thanks a lot.
Sergiu

It would be something like (ignore some of the values being different):
{"hostname": "hostname", "domain": "example.com", "ssl": true, "ssl_only": true, "ssl_certificate": "server.pem", "ssl_certificate_key": "server.key"}
So specifically "ssl_certificate": "server.pem", "ssl_certificate_key": "server.key"


Add spec for defined? on constant with const_missing

Make sure that defined? returns nil even if const_missing is implemented in the parent.
We noticed this while implementing defined? in Natalie (see https://github.com/natalie-lang/natalie/pull/2289, https://github.com/natalie-lang/natalie/issues/1734)

@andrykonchin Thanks for the review! Actually I've looked at the spec again and there are  already tests to check that .const_missing is not being called. I guess the issue why the spec did not fail for us is on Natalie's part then.
The current tests use a should_not_receive expectation (see for example it "does not call .const_missing if the constant is not defined"). Do you see any value in adding a test that actually implements const_missing? Otherwise I think we can close this :sweat_smile:

Yeah, I would say it makes sense to add this test case. At least it would catch the issue in your case.
There might be a tiny difference between semantic of the new test and the existing one. Actually the #should_not_receive method does the same - it defines a new method and checks that it isn't called. But its implementation could be changed (e.g. with #method_missing).

Yeah, I would say it makes sense to add this test case. At least it would catch the issue in your case.
There might be a tiny difference between semantic of the new test and the existing one. Actually the #should_not_receive method does the same - it defines a new method and checks that it isn't called. But its implementation could be changed (e.g. with #method_missing).

Ok I've implemented the changes you requested, let me know if anything is missing!

Thank you! Looks good.


fix(vite): add missed JPEG file extensions to KNOWN_ASSET_TYPES

Description
Add more JPEG file extensions to KNOWN_ASSET_TYPES. I get extensions from MDN https://developer.mozilla.org/en-US/docs/Web/Media/Formats/Image_types

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

mrmime(ext) returns undefined for these extensions.
So if these files were inlined, I suppose it will become data:undefined;base64,content.
https://github.com/vitejs/vite/blob/09648c220a67852c38da0ba742501a15837e16c2/packages/vite/src/node/plugins/asset.ts#L342
> require('mrmime').lookup('.jfif')
undefined
> require('mrmime').lookup('.pjpeg')
undefined
> require('mrmime').lookup('.pjp')
undefined

It seems there are some other extensions which have the same problem. (.ico is ok)
> require('mrmime').lookup('.ico')
undefined
> require('mrmime').lookup('.flac')
undefined
> require('mrmime').lookup('.aac')
undefined
> require('mrmime').lookup('.eot')
undefined


Added optional env vars for helper tool paths. This is to allow for usage of this tool within custom build rules. [phab:D1322]

Created by craigbarber at 2017-02-09 18:30:53:

craigbarber wrote at 2017-02-09 18:32:14:

fixed documentation header

craigbarber wrote at 2017-02-09 18:33:12:

fixed usage println

fromberger commented inline (context lost during migration) at 2017-02-09 18:35:50:
Minor nit: "to be utilized" could be dropped without loss of information, I think.

fromberger commented inline (context lost during migration) at 2017-02-09 18:35:50:
Alternatively, I think you could also say
: ${SHASUM_TOOL:=${PWD}/kythe/go/platform/tools/shasum_tool/shasum_tool}
: ${KINDEX_TOOL:=${PWD}/kythe/go/platform/tools/shasum_tool/shasum_tool}

though the ifs are fine too.

fromberger wrote at 2017-02-09 18:35:50:
A couple suggestions; otherwise LGTM.

fromberger accepted this revision at 2017-02-09 18:35:50

craigbarber wrote at 2017-02-09 19:21:54:

incorporated CL feedback

craigbarber closed via commit f461be2b4edff265c74f415c0fb3f338589723ef at 2017-02-09 20:44:19


Pipeline Submission Error: Size Exceeded

When submitting the input-process-output.ktr transformation for Dataflow processing, I get the following error. I can't find any way to see what's actually being sent in order to take any additional troubleshooting steps. My GS bucket name is only 12 characters, so I don't think that's contributing to the issue.
Caused by: com.google.api.client.googleapis.json.GoogleJsonResponseException: 400 Bad Request {   "code" : 400,   "errors" : [ {     "domain" : "global",     "message" : "(5d2feec528391b35): The workflow could not be created. Causes: (5d2feec528391c82): SDK pipeline options or staging file list exceeds size limit. Please keep their length under 256K Bytes each and 512K Bytes in total. For example, shorten pipeline options or staging file paths, or reduce the number of staging files by building a Java uber JAR.",     "reason" : "badRequest"   } ],   "message" : "(5d2feec528391b35): The workflow could not be created. Causes: (5d2feec528391c82): SDK pipeline options or staging file list exceeds size limit. Please keep their length under 256K Bytes each and 512K Bytes in total. For example, shorten pipeline options or staging file paths, or reduce the number of staging files by building a Java uber JAR.",   "status" : "INVALID_ARGUMENT" }

I don't see the same issue. Did you add extra plugins to the classpath in the Beam Job Config?
Ah, hang on... if we look at the classpath we see something like...
/home/matt/tmp/di-beam/plugins/kettle-json-plugin/kettle-json-plugin-core-<IP_ADDRESS>-519.jar, /home/matt/tmp/di-beam/plugins/kettle-json-plugin/lib/json-path-2.1.0.jar, /home/matt/tmp/di-beam/plugins/kettle-json-plugin/lib/slf4j-api-1.7.7.jar, /home/matt/tmp/di-beam/plugins/kettle-json-plugin/lib/json-smart-2.2.jar, /home/matt/tmp/di-beam/plugins/kettle-beam/kettle-beam-1.0.0-SNAPSHOT.jar, /home/matt/tmp/di-beam/plugins/kettle-beam/lib/malhar-library-3.4.0.jar, /home/matt/tmp/di-beam/plugins/kettle-beam/lib/grpc-alts-1.17.1.jar, ....
So if you install Kettle Beam in a folder that is very deep we could get in reach of that 256kB limit.  577 jar files times an extra 100 characters perhaps?
Could you verify these paths for me on your end?
I would hate to move to a fat jar system since the DataFlow system is so efficient in updating the jar files.

My path has an extra 24 characters per jar, and it was able to successfully copy all of the jars to the GS bucket. It appears the failure occurs after that, but as I mentioned before, I'm at a loss on the best way to inspect the details of what's actually causing the error. Thank you, by the way, for your efforts on this--I can see it being quite helpful for making new Dataflow jobs much more efficiently!

Just to clarify: jar file copying is supposed to happen automatically, no manual action is needed.

Correct, the process copied the files, not me. So, the first execution was "slow" while that happened, then the error occurred. Subsequent executions fail with the same error quickly since it's not copying the files.

From the error it seems it also could be another pipeline options error since the files list size isn't big enough to cause issues, not by far.  Here's what I use... you would have to change the Project ID and staging location.

Also make sure you have sufficient rights on the project and the account you're using.  For this to work I had to give every right and the kitchen sink.

Mine is very similar. I even removed all of the plugin folders to stage. It's certainly possible it's a permissions issue masquerading as this--I've been trying to take a more conservative approach to granting access to the service account. I'll start adding more rights to see if that gets me anywhere, thanks.

BTW, here's what's been granted to this point.

It was the depth of the path to PDI. I moved the installation to C:\Pentaho\PDI and it was able to submit successfully! Not totally sure I want to close the issue, however. I'll leave that up to you!

Ah, that's progress nonetheless! Thanks for the feedback.  I guess it will be easier to just move to a single fat jar on all platforms.  I'm reworking and refining a bunch of things for Beam Summit Berlin next week so I'll include this issue in the list of things to fix.

i have the same problem

You can generate a fat jar in the latest versions.  Grab a download at www.kettle.be
There is a build button in the Beam Config dialog or in the Beam menu in Spoon.


[VersionEye] Service deprecation

Addresses #1947.

Warnings

:warning:

This PR modified helper functions in lib/ but not accompanying tests. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @PyvesB!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS

Thanks to both of you for reviewing!


Guidelines for contributing in face-x

Please mention the open-source program in which you are participating while commenting on any issue.
Read this Contributing Guideline

Solve an issue in which you are comfortable and comment so I will assign that issue to you.

Create a Pull Request along with a proper readme file showing all the output and procedures.

Mention the issue which you have solved.

@shubhagupta21april

Assign this issue to me ! I'll provide the necessary solution .


corrects documentation for current step of deployment events

Current documentation does not match actual format.

@pbnsilva Thanks!


merge package template from arlac77/npm-package-template

package.json

chore(scripts): update prepare@rollup -c from template
chore(devDependencies): update<EMAIL_ADDRESS>from template

Coverage remained the same at 73.077% when pulling f916b507e7d11ef12f55273917b772dc321588ac on template-sync-1 into ebb76e6244ee445f0376a6fe17fa1046802bdd42 on master.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #705   +/-   ##
=======================================
  Coverage   68.18%   68.18%
=======================================
  Files           1        1
  Lines          22       22
=======================================
  Hits           15       15
  Misses          7        7

Continue to review full report at Codecov.

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


unclear group assignments

I read here that  "In the POSIX ACLs, every user is associated with a primary group. For example, user "alice" might belong to the "finance" group. Alice might also belong to multiple groups, but one group is always designated as her primary group.  In POSIX, when Alice creates a file, the owning group of that file is set to her primary group, which in this case is "finance." "  I also read "The owning group is copied from the owning group of the parent directory under which the new file or directory is created."   I do not see how these can both be true at the same time.   If I create a file under a folder with the owning group of 'dev' and my primary group is 'test', is the group of the file I create 'dev' or 'test'?

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

ID: 9d29fd9c-d764-3d15-bff3-2d61ec9f757f
Version Independent ID: 58503de5-d10f-eb43-34f9-6f72a0fa1f9b
Content: Overview of access control in Azure Data Lake Storage Gen2
Content Source: articles/storage/blobs/data-lake-storage-access-control.md
Service: storage
GitHub Login: @jamesbak
Microsoft Alias: jamesbak

Thanks for the feedback! We are currently investigating and will update you shortly.

@jloughlinMSFT Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thank you
From: SumanthMarigowda-MSFT<EMAIL_ADDRESS>Sent: Thursday, February 28, 2019 3:12 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: John Loughlin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] unclear group assignments (#25856)
@jloughlinMSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FjloughlinMSFT&data=02|01|joloughl%40microsoft.com|bd0f6c76b94844e3dcec08d69d547548|72f988bf86f141af91ab2d7cd011db47|1|0|636869383410526950&sdata=gqPQczk1V6IatLsZBc7Ms0YLUSNYnAoUN0nF9Zl%2FjYY%3D&reserved=0 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F25856%23issuecomment-468178204&data=02|01|joloughl%40microsoft.com|bd0f6c76b94844e3dcec08d69d547548|72f988bf86f141af91ab2d7cd011db47|1|0|636869383410526950&sdata=GyU8AOYz7AYr77ogZHR5fhO0tEi0HzOnf3ZtusYTsJ4%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAcNndz3VoyV6oTA2qxRQyxJy-mgv6t-Hks5vR49igaJpZM4bVAez&data=02|01|joloughl%40microsoft.com|bd0f6c76b94844e3dcec08d69d547548|72f988bf86f141af91ab2d7cd011db47|1|0|636869383410526950&sdata=4gG9Era%2BkIANFzOywf5oGiiXYpm3GGQe5E48a6m%2BmLs%3D&reserved=0.

Thank you for your question @ghost. Sorry it has taken so long for us to get back to you on this. Yes the owning group is always copied down from the parent. The doc does have a small section that clarifies that when the container is initially created, the owning group is set to the primary group of the user who created it.  For all instances after that (subdirectories in the root folder etc., the owning group is copied.  Here is the excerpt:
Case 1: The root directory "/". This directory is created when a Data Lake Storage Gen2 container is created. In this case, the owning group is set to the user who created the container if it was done using OAuth. If the container is created using Shared Key, an Account SAS, or a Service SAS, then the owner and owning group are set to $superuser.
Case 2 (Every other case): When a new item is created, the owning group is copied from the parent directory.
It's possible that this excerpt was added some time after you posted this question.   Thank you for posting!

#please-close


Logging polish (part 2)

Omit line number from log context when zero
Corrected outdated documentation comment

What is the current behaviour?
Logs from managed code have line number zero in the context, ex: (UnityLogListener:0|.ctor) message
What is the new behaviour?
Logs from managed code now omit the line number, ex: (UnityLogListener|.ctor) message
Note: The same functionality does not apply to FileLogHandler as it allows the user to define a log context format string, making it confusing if part of that format is silently omitted in some cases. It would be nearly impossible to cleanly omit the line number in custom format strings anyway.
What PRs does this depend on?

None

Does this introduce a breaking change?

[ ] Yes
[X] No

Related

https://github.com/decline-cookies/anvil-unity-core/pull/82

It's just the default value returned by StackFrame.GetFileLineNumber() - so if the log context fails to resolve you'd see (unknown|unknown), or more likely if the log comes from a DLL or other managed code, there is no file/line to point to, so we fall back on showing DeclaringType.Name and omit the line number, ex (UnityLogListener|.ctor).

Ahh gotcha. Ok cool thanks
On Fri, Oct 7, 2022 at 7:57 AM Tyler Vezina @.***>
wrote:

It's just the default value returned by StackFrame.GetFileLineNumber() -
so if the log context fails to resolve you'd see (unknown|unknown), or
more likely if the log comes from a DLL or other managed code, there is no
file/line to point to, so we fall back on showing DeclaringType.Name and
omit the line number, ex (UnityLogListener|.ctor).
—
Reply to this email directly, view it on GitHub
https://github.com/decline-cookies/anvil-csharp-core/pull/115#issuecomment-1271497505,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACSYQH5THV4LIMGYA774HTWCAF2HANCNFSM6AAAAAAQ7B6RIA
.
You are receiving this because you commented.Message ID:
@.***>


Pipeline failed to scan image

2023-02-13T20:13:34.363Z	FATAL	scan error: unable to initialize a scanner: unable to initialize a docker scanner: 3 errors occurred:
24
* unable to inspect the image (ghcr.io/bcgov/nr-fom:test-api): Error: No such image: ghcr.io/bcgov/nr-fom:test-api
25
* unable to initialize Podman client: no podman socket found: stat podman/podman.sock: no such file or directory
26
* GET https://ghcr.io/v2/bcgov/nr-fom/manifests/sha256:93ba7005cb3e12699a8bf68a3b50706562417730e384dddc1a489893b1773234: MANIFEST_UNKNOWN: manifest unknown

GitHub Container Registry has been showing some successful docker image builds that mysteriously can't be pulled.  Worst case we might have to upgrade to their newer package naming structure.


Add Java/Maven metadata support

I was browsing through the codebase and found this feature where manifest.mf files are scanned for data. Not all Java libraries add these metadata. However, any library packaged with Apache Maven tends to include a file named pom.properties (generally inside META-INF/maven/groupId/artifactId/pom.properties) which has the name and version info data. This is also generally more reliable than the manifest file due to, once again, typical build tooling usage. This can be very helpful when libraries are shaded together; the manifest files may get destroyed in the process, but all the pom.properties files will safely-ish combine together (other than version number conflicts, but that's also handled by shading).

@jvz thanks for filing this. We'll definitely take a look and update this ticket with progress. Anything that helps the fidelity of the detection process is much appreciated.

I'd love to help out where I can with this. I'm investigating integrating Anchore with Jenkins's Jenkins build, and if I can use the same tool for both Docker images as well as the Java libraries within, that would be fantastic. I'm not exactly sure how these data overlap with the CVE feeds, but that would be my goal at least.

Hi @jvz - that is a great observation!  There are effectively two 'sides' to the anchore process, where the first is 'gather the data' and the second is 'parse, store and use' the data for policy evaluation.  The analyzer module (32_java_packages.py) represents the first phase, where I'd suggest that we would add the pom.properties file as an additional 'metadata' element, if found.  The second part is largely implemented here in the policy engine, where candidates for the name/version are discovered from the analyzer artifacts, constructed into a set of potential CPEs, and then matched against NVD data (and used as content output when java package information is requested for an image).
I thought I'd provide a bit more context about the two 'sides' of the process when considering adding more analysis capabilities for java packages!

Great, that's exactly what I was wondering about! Are there any sample unit or integration tests to go off of? Or perhaps a desired way of doing so if there aren't any already.

hi @jvz, there aren't really any integration tests for the analyzers, but you can run time directly outside of anchore as long as you have a rootfs to point them at. The current java analyzer is: https://github.com/anchore/anchore-engine/blob/master/anchore_engine/analyzers/modules/32_java_packages.py
You can see that it is a stand-alone script that is just given some path information on where to find the rootfs etc. Any additional metadata handling for Maven builds could be added there as part of the recursive jar inspection.
We've also got a community slack channel if you want an invite so we can discuss live. Happy to send an invite.

Awesome, thanks for the pointers. The slack invite would be great, email is in my profile.
Also, this might be a separate issue, but there are some more file extensions that could be supported for Java libraries. I know that Jenkins uses .hpi for example.

@jvz Invite sent. I see the PR and we'll review it shortly. Thanks!

Come to think of it, there is a third potential source for metadata in Java libraries: OSGi metadata. This would come from the manifest.mf file and may have more detailed information. In order to properly parse these data, though, the manifest parsing logic will need to be improved to support proper manifest files (i.e., with line wrapping implemented), and some of the OSGi metadata are encoded as LDAP filters, so a syntax parser for that could be useful in the future if the data are examined more thoroughly.


Add local install and build static library tests to travis

This changes the format of the .travis.yml to implement a build matrix in order to make it slightly cleaner. It also adds a test case for building the static library and adds a local install with each build.

Fixes #120

Coverage remained the same at 92.988% when pulling da63b7948895d29e68c7a18f7794b8375c66b1f2 on kathrynfejer:travis-add-builds into<PHONE_NUMBER>0547ae596812793c6fcc067489d14b on xaptum:master.


change: Visually displays the output image provided.

Imported the matplotlib library to visualize the output images:
import matplotlib.pyplot as plt
Added a new function called show_output_image:
def show_output_image(output_image):     Displays the given output image visually.     plt.imshow(output_image)     plt.axis('off')     plt.show()
Modified the test_img2img test case to use parameterization with different sample sizes and step counts:
``
@pytest.mark.parametrize("sampler_enum", Sampler)
@pytest.mark.parametrize("num_samples", [1, 3, 5])  # New parameter: Different sample sizes
@pytest.mark.parametrize("num_steps", [5, 10, 15])  # New parameter: Different step counts
def test_img2img(self, pipeline: SamplingPipeline, sampler_enum, num_samples, num_steps):
output = pipeline.image_to_image(
params=SamplingParams(sampler=sampler_enum.value, steps=num_steps),
image=self.create_init_image(pipeline.specs.height, pipeline.specs.width),
prompt="A professional photograph of an astronaut riding a pig",
negative_prompt="",
samples=num_samples,
)
assert output is not None
# Show the output images visually
for i in range(num_samples):
    show_output_image(output[i])

``
The rest of the class and test cases remain the same as in the previous explanation.
With these changes, we have added visualization capabilities to display the output images produced by the model during the test_img2img test. The test now covers different sample sizes and step counts, allowing us to evaluate the model's performance with varying parameters.

These tests are run by the CI pipeline; it'd be super weird if they opened windows to show images.

Can be written into image files as below (in tests/inference/test_inference.py):
+        NUM_SAMPLES = 1
+        sample = sample.cpu() * 255
+        for i in range(0, NUM_SAMPLES):
+            _sample = sample[i]
+            _sample = np.transpose(_sample.numpy().astype(np.uint8), (1, 2, 0))
+            img = Image.fromarray(_sample)
+            img.save(os.path.join('output', 'output_{}_{}_{}.png'.format(sampler_enum.value, 'ii' if use_init_image else 'ti', i)))

Though all the output images are corrupted. I'm totally lost about it...  https://imgur.com/a/vXwKsnI


Use application pipeline base

Description
Hac project would like to live under application pipeline completely, without the hac prefix. This PR fixes the issue by changing the router to operate with application pipeline instead of hac.
JIRA
RHCLOUD-23566
Merge after

[x] https://github.com/RedHatInsights/chrome-service-backend/pull/73

@karelhala every now and then I see this when I refresh the page locally:

@florkbr that's strange this might be connected to local environment. I've encountered this only when restarting webpack server, after that it was just fine and any refreshes were rendering correctly. I'd say let's merge it and test in console.dev environment so we know if this issue is something with local dev or something more deeply integrated. We now have seperate builds for prod and dev environment, so we can test things in true SASS way.

/lgtm


ios profiles:list UnexpectedContentError

The "profiles:list" command is returning a "UnexpectedContentError" error when executed:
Sequoia:jenkins labrown$ ios --version
ios 0.8.1

Sequoia:jenkins labrown$ ios profiles:list
error: Cupertino::ProvisioningPortal::UnexpectedContentError. Use --trace to view backtrace
Sequoia:jenkins labrown$ ios profiles:list --trace
/Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/agent.rb:159:in `list_profiles': Cupertino::ProvisioningPortal::UnexpectedContentError (Cupertino::ProvisioningPortal::UnexpectedContentError)
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/commands/profiles.rb:8:in `block (3 levels) in <top (required)>'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/helpers.rb:50:in `try'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/commands/profiles.rb:8:in `block (2 levels) in <top (required)>'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/command.rb:180:in `call'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/command.rb:180:in `call'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/command.rb:155:in `run'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/runner.rb:402:in `run_active_command'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/runner.rb:78:in `run!'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/delegates.rb:11:in `run!'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/import.rb:10:in `block in <top (required)>'

Sequoia:jenkins labrown$ ios profiles:list distribution
error: Cupertino::ProvisioningPortal::UnexpectedContentError. Use --trace to view backtrace
Sequoia:jenkins labrown$ ios profiles:list distribution --trace
/Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/agent.rb:159:in `list_profiles': Cupertino::ProvisioningPortal::UnexpectedContentError (Cupertino::ProvisioningPortal::UnexpectedContentError)
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/commands/profiles.rb:8:in `block (3 levels) in <top (required)>'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/helpers.rb:50:in `try'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/cupertino-0.8.1/lib/cupertino/provisioning_portal/commands/profiles.rb:8:in `block (2 levels) in <top (required)>'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/command.rb:180:in `call'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/command.rb:180:in `call'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/command.rb:155:in `run'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/runner.rb:402:in `run_active_command'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/runner.rb:78:in `run!'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/delegates.rb:11:in `run!'
	from /Users/labrown/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/commander-4.1.3/lib/commander/import.rb:10:in `block in <top (required)>'

I am experiencing this issue after upgrading to el capitan.
Followed the steps here :
http://www.hacksparrow.com/os-x-el-capitan-screwed-up-ruby-gems-and-how-to-fix-it.html
reinstalled cupertino
Authenticated myself again using $ios login still facing the issue:
$ ios profiles:list --trace
/usr/local/lib/ruby/gems/2.3.0/gems/cupertino-1.3.4/lib/cupertino/provisioning_portal/agent.rb:188:in list_profiles': Cupertino::ProvisioningPortal::UnexpectedContentError (Cupertino::ProvisioningPortal::UnexpectedContentError) 	from /usr/local/lib/ruby/gems/2.3.0/gems/cupertino-1.3.4/lib/cupertino/provisioning_portal/commands/profiles.rb:9:in block (3 levels) in <top (required)>'
from /usr/local/lib/ruby/gems/2.3.0/gems/cupertino-1.3.4/lib/cupertino/provisioning_portal/helpers.rb:58:in try' 	from /usr/local/lib/ruby/gems/2.3.0/gems/cupertino-1.3.4/lib/cupertino/provisioning_portal/commands/profiles.rb:9:in block (2 levels) in <top (required)>'
from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.0/lib/commander/command.rb:178:in call' 	from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.0/lib/commander/command.rb:153:in run'
from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.0/lib/commander/runner.rb:444:in run_active_command' 	from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.0/lib/commander/runner.rb:68:in run!'
from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.0/lib/commander/delegates.rb:15:in run!' 	from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.0/lib/commander/import.rb:5:in block in <top (required)>'