Skip to content

Portfolio manager updates to CITYdata new version - #228

Merged
SikandarEjaz merged 3 commits into
developfrom
fix/portfolio_manager_update_2
May 19, 2026
Merged

Portfolio manager updates to CITYdata new version#228
SikandarEjaz merged 3 commits into
developfrom
fix/portfolio_manager_update_2

Conversation

@MinetteMeyo

Copy link
Copy Markdown
Collaborator

No description provided.

@SikandarEjaz

Copy link
Copy Markdown
Collaborator

The query still fails:

[{"result":"[Producer ca.concordia.encs.citydata.producers.PortfolioManagerProducer was not found. Please check whether the fully-qualified name is correct and try again., caused by: java.lang.ClassNotFoundException: ca.concordia.encs.citydata.producers.PortfolioManagerProducer]"}]

I will look into this issue tomorrow.

@SikandarEjaz

SikandarEjaz commented May 15, 2026

Copy link
Copy Markdown
Collaborator

So, in your branch PortfolioManager producer fails and also CalibrationProducer, which I fixed in my branch. Could you please update your branch with my branch fix/server_access and then fix the issue with PortfolioManager.

@MinetteMeyo

Copy link
Copy Markdown
Collaborator Author

I don't have these issues on my end, but I'll double check. Maybe we can look into it more carefully together on Tuesday?

Also, note that you could have used the example/portfolioManager branch as a base coz it's working fine, it just doesn't include the recent modifications. I created this new branch specifically to integrate those changes.

@SikandarEjaz

Copy link
Copy Markdown
Collaborator

I don't have these issues on my end, but I'll double check. Maybe we can look into it more carefully together on Tuesday?

Also, note that you could have used the example/portfolioManager branch as a base coz it's working fine, it just doesn't include the recent modifications. I created this new branch specifically to integrate those changes.

Alright, let's fix this issue coming Tuesday.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please remove the credentials from the code examples?

@SikandarEjaz SikandarEjaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the credentials hardcoded in the code examples.

@SikandarEjaz
SikandarEjaz merged commit 222b4ea into develop May 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants