model_dump dict stays attached to the model's references for custom classes

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
It appears that model_dump creates a 'shallow' dump of the model: if one of the model attributes is a custom class containing mutable data, then changing the data in a dump of the model will also change it in the original model.
This feels unexpected as one would expect a 'dump' to be a record (e.g. core dump, or database dump) of the model at the time of the dump, and changing that record should not change the original model. I think that either model_dump should do 'deep' dumps by default to actually create a dictionary record (and vars(model) can be used for shallow dumps) or model_dump should have an optional deep parameter.
A current workaround for this is if you use model.model_copy(deep=True).model_dump(), then it works as expected.
Example Code
from pydantic import BaseModel

class MyClass:
    def __init__(self, data):
        self.data = data

class Model(BaseModel, arbitrary_types_allowed=True):
    var: MyClass

m = Model(var=MyClass([1, 2, 3]))
md = m.model_dump()
md['var'].data.pop()

# will give a result of [1, 2], so 3 has been popped from the model as well
print(m.var.data)

Python, Pydantic & OS Version
             pydantic version: 2.9.2
        pydantic-core version: 2.23.4
          pydantic-core build: profile=release pgo=false
                 install path: /home/rec93639/Code/RasCAL-2/.venv/lib/python3.12/site-packages/pydantic
               python version: 3.12.4 (main, Jul 25 2024, 22:42:01) [Clang 18.1.8 ]
                     platform: Linux-6.8.0-40-generic-x86_64-with-glibc2.39
             related packages: typing_extensions-4.12.2
                       commit: unknown

Hi @alexhroom,
Thanks so much for your question. This is a consequence of the fact that you're using an arbitrary type, we simply dump the instance. If you'd like different behavior here, I'd recommend using a custom serializer, like follows:
from pydantic import BaseModel, PlainSerializer
from typing import Annotated
from copy import deepcopy

class MyClass:
    def __init__(self, data):
        self.data = data

class Model(BaseModel, arbitrary_types_allowed=True):
    var: Annotated[MyClass, PlainSerializer(lambda x: deepcopy(x))]

m = Model(var=MyClass([1, 2, 3]))
md = m.model_dump()
md['var'].data.pop()

# will give a result of [1, 2, 3]
print(m.var.data)

We offer much more intuitive support for BaseModel subclasses (nested models). Let me know if you have any other questions :).


Pretrained weights

Has anyone done the transfer of learning from the original PSPNet (in Caffe) code to keras model this repository?

Don't think they are compatible


align content in "msbuild-targets" for pack with the pack spec

the content in  https://docs.microsoft.com/en-us/nuget/schema/msbuild-targets is outdated , and needs to be aligned with the updated version of spec : https://github.com/NuGet/Home/wiki/Adding-nuget-pack-as-a-msbuild-target

Rohit, can you review the changes to this topic in my branch before I do a PR? https://github.com/NuGet/docs.microsoft.com-nuget/blob/kraigb/docs/Schema/msbuild-targets.md
Note that I also updated the restore target section according to https://github.com/NuGet/Home/wiki/[Spec]-MSBuild-restore-target, and also checked https://github.com/NuGet/Home/wiki/PackageTargetFallback-(new-design-for-Imports).

@kraigb have made some comments on suggested changes.

@emgarten might want to read through the restore documentation

In the meantime, I'll do the PR to update the pack section, and can update restore as needed with a new PR.

Content should be aligned now. If you see anything, just edit the file and do a PR.


Request for access for awkward, awkward-cpp, and uproot

We (over at scikit-hep) maintain several packages that would benefit from nightlies. To begin with, I'd like to request access for the following projects:

awkward
awkward-cpp
uproot

Thanks all!

Following up on the discussions started in https://github.com/scientific-python/upload-nightly-action/issues/20#issuecomment-1594574923 I think that further approval / review requires people with org level admin powers, so tagging @tupui, @stefanv, and @jarrodmillman. (I think there may be others, but I don't know.)

I had also once asked if astropy should moved its nightlies to SP. (We already host it somewhere else.) The motivation is that downstream can pull the whole stack from the same index URL. But if not, also no big deal. I guess it depends on the goal and scope of SP hosting.

@pllim to avoid crossover of discussions can I ask that you make a request in a separate Issue like Issue #20 and Issue #29?
https://github.com/scientific-python/upload-nightly-action/blob/a2ba178f650e68bd6f9d8e2b7d59d16ffd257c29/README.md?plain=1#L39-L45
(We don't have this super formalized at the moment, but c.f. https://github.com/scientific-python/upload-nightly-action/issues/20#issuecomment-1594574923 for how we might make it so in the future.)

@agoose77 as I know that awkward uses hatchling, if your request is approved just a heads up that you might run into https://github.com/Anaconda-Platform/anaconda-client/issues/676. I will tag Ofek there to see if he has any suggestions for the anaconda-client team.

I am +1 for adding

awkward
awkward-cpp
uproot
astropy nightlies

We probably want to add some more guidance about what we can include, but we can sort things out as we go. I am busy this weekend, but can look into this next week.

We probably want to add some more guidance about what we can include, but we can sort things out as we go.

I really would prefer if the guidelines are figured out first than starting to add random packages (==non core, and core dependencies), then ending up to need to say no for others later when we run into technical limitations, e.g. not enough space, etc.
That guideline could be very generic, that e.g. specific things from the domain stack could use the same space, etc, but I definitely see the value of figuring it out first rather than when a handful of  packages are already there.

To mitigate this I've made a scikit-hep-nightly-wheels Anaconda Cloud organization which with https://github.com/scikit-hep/awkward/pull/3012 is now taking care of nightly wheels for awkward and awkward-cpp (contingent on https://github.com/Anaconda-Platform/anaconda-client/issues/702 being resolved).
I'll let the Awkward dev team decide if they want to keep this Issue open for future discussion, of if they are content with this solution for the long term, in which case they can close it. :+1:

Here's a somewhat radical proposal: include everyone who will find this service useful. Once we outgrow our space, we'll have to tell new projects "no" until we figure out a solution. The solution may be as easy as hosting our own conda server, for which we already have server capacity.

include everyone who will find this service useful

I don't find it that radical. As far as I understood this initiative when we discussed it during the summit in Seattle last year, this was the intention. I wasn't involved very much in subsequent discussions but if there's a way of providing this service to the whole ecosystem, rather than a selection of packages based on unclear criteria and essentially a feeling of a package importance, let's try to do that.

@stefanv @martinfleis I am +1 on this. I would be very happy to accept the requests that are already open in this Issue and Issues #45, #50, and #51, but to have this be a decision that is easily understood in the future can the following happen (in the very near future)?

Have the @scientific-python/spec-steering-committee agree on moving forward with this proposal in the very near future
Have the @scientific-python/spec-steering-committee advise on what a future policy would be (this is https://github.com/scientific-python/upload-nightly-action/issues/30#issuecomment-1934573445) so that when space limits are hit there is a clear transition plan.

@stefanv as you are a member of the @scientific-python/spec-steering-committee can you comment on how soon you think a discussion could take place? I'm not sure how often async discussions happen, but I'd be keen to move on this.

The solution may be as easy as hosting our own conda server, for which we already have server capacity.

@stefanv Can you comment (over on https://github.com/scientific-python/upload-nightly-action/issues/30) on how much server space Scientific Python realistically has? If more than 30 GB could be given to storage of wheels then I would be interested in helping to get this setup even if this is just as a test instance. I'm in general +1 on controlling infrastructure, but if Scientific Python's grant funds can be spent on things other than server costs I'm also +1 on taking advantage of Anaconda's generosity to open source.

@agoose77 @jpivarski Good news, we have the approval (https://github.com/scientific-python/upload-nightly-action/issues/30#issuecomment-1960037619) to add awkward-cpp, awkward, and uproot to the nightly wheels Anaconda Cloud org https://anaconda.org/scientific-python-nightly-wheels/ !
To get started on this can @agoose77 and @jpivarski please do the following:
Steps to gain upload access to https://anaconda.org/scientific-python-nightly-wheels/
(@jpivarski already did these steps in https://github.com/scikit-hep/awkward/pull/3012 for https://anaconda.org/scikit-hep-nightly-wheels/ so he doesn't need to do them again, and the token he made is still good here. :+1: Though @agoose77 if you want upload permissions/responsibility then please make an Aanconda Cloud account too.)

[x] ~Create an account on Anaconda Cloud~
[x] ~Reply here with the link to your account~
[x] ~Make a new Anaconda Cloud API token with permissions described in https://scientific-python.org/specs/spec-0004/#process-for-adding-new-projects~

[x] ~Save the API token as a repo GitHub Actions secret named ANACONDA_ORG_UPLOAD_TOKEN~
[ ] Verify that I have added you to the Awkward and Uproot team "Group" on https://anaconda.org/scikit-hep-nightly-wheels

If you'd also like to use the scientific/upload-nightly-action GitHub Action that this repository provides please check out this repo's README. If you need to collect wheels from different CI jobs, consider creating a new GitHub Actions workflow with logic similar to matplotlib's or Awkward's.
If you have any questions please just ask here!


XIVY-6494 Improve inline edit label ui and quick action positioning

Can easily be merged without the changes from the core


Xaml support was removed from the crate

[Enter feedback here]
The Documentation is outdated the Xaml support was removed see commit b8d83c2

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

ID: 1cf3f708-e517-41c8-0033-5e737e58cbd6
Version Independent ID: a143550c-f85f-5b35-2fa4-0bcc243cac1d
Content: Rust for Windows, and the windows crate
Content Source: hub/dev-environment/rust/rust-for-windows.md
Product: dev-environment
Technology: windows-rust
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Thanks for the heads up, that's been edited now. :)
-Steve


Immutable object should be used in place of the EnumMap for providing metrics to the UI

EnumMap is mutable; look for a better way to provide this state

Somewhat addressed by #10


Using MPFR for BigFloat

The MPFR library addresses several deficiencies in the mpf_* functions provided by GMP:

It has proper IEEE 754 semantics, e.g. infinity and accurate rounding;
It provides definitions for almost all of the mathematical functions we define in Math, like sin and the floating-point manipulation functions;
MPFR floats support arbitrary-precision string conversions (see https://github.com/crystal-lang/crystal/pull/10632#issuecomment-885418088).

So I think we should use that on top of GMP (we cannot replace it because GMP as a whole also provides BigInt and BigRational). MPFR provides functions to convert to and from GMP's mpf_t in case any Crystal code depends on LibGMP::MPF directly.

I was thinking about how to bridge the differences between our rounding modes and MPFR's. MPFR defines seven:

MPFR_RNDN, equivalent to RoundingMode::TIES_EVEN
MPFR_RNDZ, equivalent to RoundingMode::TO_ZERO
MPFR_RNDU, equivalent to RoundingMode::TO_POSITIVE
MPFR_RNDD, equivalent to RoundingMode::TO_NEGATIVE
MPFR_RNDA, round away from zero, has no Crystal equivalent
MPFR_RNDF, faithful rounding, only supported for a subset of operations, has no Crystal equivalent
MPFR_RNDNA, internal use only, equivalent to RoundingMode::TIES_EVEN

the public macro mpfr_round_nearest_away emulates this, and fortunately can be ported to Crystal

Those rounding modes are available in almost every operation, where rounding occurs in the least significant bits in the mantissa, not just mpfr_round (round to integer). There is also a default rounding mode configurable by mpfr_set_default_rounding_mode, but it doesn't appear to be used by MPFR itself, and is only for the GMP MPF compatibility mode shown here. Thus it would make no sense to bind that function here, because adding MPFR to the standard library implies not using LibGMP::MPF at all.
I don't think the standard library would enjoy supplying a rounding mode parameter to every method on BigFloat. So outside of the few rounding-related methods, MPFR_RNDN should be passed to MPFR. #11097 might mean that we could roll our own default mode.

One particular benefit is it provides mpfr_frexp which avoids an overflow in our current implementation of Math.frexp(BigFloat):
LibGMP.mpf_get_d_2exp(out exp, 2.to_big_f ** (UInt64::MAX // 2))
Errno.value # => Errno::EINVAL
exp         # => -9223372036854775808


fix(TCOMP-586): Reuse classloaders for HDFS components.

This reverts the changes made to fix TCOMP-422, which should reappear, but fixes many problems with memory leaks in the sandboxed runtime.
What is the current behavior? (You can also link to an open issue here)
ClassLoaders are not reused for HDFS components, relying on the sandboxing to isolate these runtimes.  This causes many memory leaks with HDFS and Hadoop.
What is the new behavior?
ClassLoaders are reused again for HDFS.
I expect TCOMP-422 to reappear as a regression.  The fix for this bug was not stable.
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:

Pull request test result: success :star:


 Details: Could not find file 'T:\m3xw\toc.json'.

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\m3xw\bread\toc.json'.

building-blocks/extending-templates/toc.json

[Warning] Error happen when converting building-blocks/extending-templates/toc.json to Pdf. Details: Could not find a part of the path 'T:\m3xw\building-blocks\extending-templates\toc.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

This file already contains fasttrack in the custom metadata.  Thanks!


Feature Request Non 90 degree rotation

I noticed the rotation method only takes multiples of 90 degree angles. Normally that would cover all normal rotation needs, but for a project I am working on, I need to be able to rotate an image on any angle with resizing.
Again, thanks for all of your hard work on this library!!

Hello, this is covered by feature request #998.


Handling the "month change" event

I've been trying to call a function when I change the month in my calendar using the left and right arrows. Is there a way to do this?

@danielrvt Have you looked at viewRender ? According to the documentation it also gets called when a new date-range is rendered.
http://fullcalendar.io/docs/display/viewRender/
Hope it helps ^.^

Where is viewRender function ?
viewRender : function (view, element) {     console.log('new date range start : ', view.start, 'new date range end : ',view.end); }
it is giving syntax error ,and the link http://fullcalendar.io/docs/display/viewRender/ is not working


Prometheus metrics for default-http-backend

Should export qps for this thing: https://github.com/kubernetes/contrib/tree/master/404-server in prometheus format

I'll take a stab at this

@cmluciano are going to fix this? I can see some commit and no PR.

Weird that it did not get linked. I have the PR here https://github.com/kubernetes/contrib/pull/2376. Would be great if someone could help debug what Godeps wants.

Should be closed by kubernetes/contrib#2376

@bowei can you close this


Integrate with or differentiate from resin-analytics

Now that this has matured a little, we should come up with a plan for this & resin-analytics. At the moment they both duplicate mixpanel, and are evolving independently and duplicating lots of effort, which seems clearly wrong.
What would be a good end result for us, in terms of analytics libraries?
How do we get there?
It would be good to do this sooner, rather than later, as I've seen people talking independently about integrating both resin-analytics and corvus with various other projects, and the more they spread across our components the harder it'll be to bring the two back together.

Now being discussed and planned in HQ, so this can go. See https://github.com/resin-io/hq/issues/880


Doesn't like folder paths with spaces / plus signs

Tried out your container, worked great.
BOTO apparently doesn't like folder with spaces or plus signs.

I'd be happy to accept any pull requests, especially if it is related to hardening the existing code.
Are you sure the issue is with boto or is it possibly not being properly interpreted by the docker/bash when passed as a parameter.


autosave feature

Hello
Do you think it should be possible to add an "auto-save" function. For example the file I am working on is automatically saved  every 10 minutes ?

Hi
That is a good idea, will add it to the plan. Should be easy to implement.
Thanks a lot!
Cheers


pull request for e2e test at 12/20/2016 23:45:20

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Load custom services from config folder

Load modules found in configdir/contrib on startup. If modules implement a BaseService they should add it to pyethapp.app.services in order to get initialized on startup.
This was started before, non working code can be found here:
https://github.com/ethereum/pyethapp/blob/develop/pyethapp/utils.py#L14

This one seems easy enough to start with. Can someone please give me a hint which module/service I can use to verify the change?

already solved. missing thing is documentation in the wiki and an example in the examples/ folder.
@vbuterin had a write up, how to use it. but i can't find it.


minor bug fix

This project uses semver, please check the scope of this pr:

[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Fixed issue when get(this, listConfig.items) return nested path.

_listItems = null/underfined case is covered inside component

Changes Unknown when pulling 9a1d4f436fdff7628d83c47ef00d739507ca6986 on quincyle:develop into ** on ciena-frost:master**.

:+1:


Don't apply bootstrap styles and submit invalid form?

I'm trying to add ngx-formly (current project) in aspnetcore-angular2-universal project of Mark Pieszak.
I added the packages and add the proper modules like the description in readme.md. It looks like that start to work but bootstrap doesn't apply the validation styles over the form. Form example when the field is required it have to go in red state but it didn't. My changes are in fork of your repository:
.
Can you check why the styles are not applied when the form is invalid and required fields are empty? And why when the form is invalid it process submit handler and doesn't stop it. I'm trying to migrate one old project and rewrite the code to use the same logic. In old version submit button didn't continue in case when the form is not completely valid. Can you check in the project last commit . What is wrong to have no validations style as well. I added the bootstrap module which are described in the documentation.
Thank you

make sure you have included bootstrap style bootstrap.min.css in your index.html
and define validationMessages in the FormlyModule
FormlyModule.forRoot({
  validationMessages: [
    { name: 'required', message: 'this field is required' },
  ]
}),

why you have defined seveval formly-form on your form, I think only one is enough and if you want to use bootstrap grid system you just have to pass className in field declaration see the example https://stackblitz.com/edit/angular-5ioeet

We had added multiple formly-form to control the design more flexible. That was the reason but It hadn't be a problem in old version for angular 1.x. May be is not problem and here. Actually I forked your example and changed one field to be required. In the example the bootstrap error styles aren't applied as well:
https://stackblitz.com/edit/angular-ye9nqh
If I change FormlyModule.forRoot parameters to
 FormlyModule.forRoot({   validationMessages: [     { name: 'required', message: 'this field is required' },   ] }),
The message ''this field is required' appears to the box but is not in red as well.How can be set up to use bootstrap error styles? Or we have to add our own styles to show the error fields?
Thank you,
GetTaxSolutions Team

@GetTaxSolutions because you used an old version of bootstrap, you should include the beta version
https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0-beta.2/css/bootstrap.min.css

Is there version which is compatible with bootstrap 3.3.7?

I think we can support both of them, I'll provide a PR tomorrow

@GetTaxSolutions the support for bootstrap 3.3.7 has been added in 2.0.0-beta.7

I tried to see it in example and modify your with 3.3.7 and 2.0.0-beta.7 - https://stackblitz.com/edit/angular-hvklvh
It looks that styles are correct applied but it check validation on text change, not on blur. Can I setup to check when the input control lost the focus?

it's a regression, I'm on it.

fixed in 2.0.0-beta.8

Now looks good in example. Thank you


adding zalando end user members

Signed-off-by: Diego Roccia<EMAIL_ADDRESS>

FYI @andrewhowdencom @raza51 @szuecs @truemped


fix: string interpolation

where did it cause some issues ?

No, It has not caused any issue for me.
I just found these wrong string interpolations while working on https://github.com/Mozilla-Actions/sccache-action/pull/123


[QUERY] Correctly configure spring-cloud-azure-starter-keyvault-secrets for client ID/secret authentication

Query/Question
I am having an issue configuring the spring-cloud-azure-starter-keyvault-secrets to use client ID/secret authentication to the key vault. Following the instructions at https://learn.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-azure-key-vault#add-key-vault-configuration-to-the-app, I am setting the following properties in my application.yml file (with placeholders for this submission):
spring:
   cloud:
      azure:
         keyvault:
            secret:
               property-sources:
                  - endpoint: ${keyvaultUri}
                    profile:
                       tenant-id: ${tenantId}
                    credential:
                      client-id: ${clientId}
                      client-secret: ${clientSecret}

I am getting the following error:
com.azure.core.exception.ClientAuthenticationException: The current credential is not configured to acquire tokens for tenant ${tenantId}. To enable acquiring tokens for this tenant add it to the AdditionallyAllowedTenants on the credential options, or add "*" to AdditionallyAllowedTenants to allow acquiring tokens for any tenant. See the troubleshooting guide for more information. https://aka.ms/azsdk/java/identity/multitenant/troubleshoot
However, the tenant ID matches the one that the service principal was created under. Additionally, I'm not building the DefaultAzureCredentials and letting the Spring Boot autoconfiguration do it. Looking for guidance on either how to manually wire the Azure Credentials so I can add that AdditionallyAllowedTenants block, or what I should be doing differently in the plugin configuration in order to make this work.
Why is this not a Bug or a feature Request?
I assume that I am doing something wrong or just missing something, so I do not believe this is a bug in the code.
Setup (please complete the following information if applicable):

OS: macOS 13 (also appears on AKS in an openjdk:11-jre-slim container)
IDE: IntelliJ IDEA
Library/Libraries: com.azure.spring:spring-cloud-azure-starter-keyvault-secrets:4.4.1, org.springframework.boot:spring-boot-starter-parent:2.7.5

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Query Added
[X] Setup information Added

Hi @zpdeering thank you for reaching out to us via this github issue.  @chenrujun could you please follow up?

/cc @vcolin7 as fyi

Hi @zpdeering thank you for reaching out to us via this github issue. @chenrujun could you please follow up?

Hi @joshfree, for any issue related to azure spring and keyvault, please assign to me while doing issues triage.
Hi @zpdeering, I will follow up your question.

Hi, @zpdeering
I have some questions about your app:

Is the ${clientId} a multi-tenant app?
Is the ${clientId} belong to the ${tenantId}? I think the answer is yes as you said tenant ID matches the one that the service principal was created under.
Is the ${keyvaultUri} belong to the ${tenantId}?

Hi @chenrujun. The AAD app ${clientId} is not a multitenant app, and it does belong to the tenant ${tenantId}. The Key Vault is under a child tenant of the parent tenant with ${tenantId}.

@zpdeering
Thank you for your quick response.
Here is my current understanding: There are 2 tenants: teannt-1 and tenant-2:

tenant-1 is the parent of tenant-2.
The client-id used in your app belong to tenant-1. Let's name it client-1.
The key vault used in your app belong to tenant-2. Let's name it key-vault-2.

I'll try to reproduce the problem after you confirmed above items.

@zpdeering
How did you run this step:

Configure the Key Vault to allow get and list operations from that managed identity.

In my side, when the client-id and the keyvault belong to different tenant, this step failed with error like this:
$ az keyvault set-policy   --name ${keyvault-name}   --spn ${client-1} --secret-permissions get list
Unable to find user with spn '${client-1}'
Unable to get object id from principal name.

@chenrujun
So truthfully, I did it through the Portal UI because we had already set these up before trying to get this Spring Boot dependency to work. I did not have that issue though - our subscription "tenant-2" can browse the full AAD from "tenant-1" allowing us to use those to set policy.
And for the record, this all works fine with the older (pre-Azure Identity auth changes mentioned at https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/identity/azure-identity/BREAKING_CHANGES.md) dependencies, so it's not like it's never worked.

@chenrujun so I actually managed to get this to work this morning - I had confused my tenant IDs and had been using the tenant-2 ID and not the tenant-1 ID in the Spring Boot parameters.

@zpdeering
Thank you for your feedback.

I actually managed to get this to work this morning - I had confused my tenant IDs

I have some questions:

What's current status? Did you still use client-1 (belong to tenant-1) to access key-vault-2 (belong to tenant-2) ?
our subscription "tenant-2" can browse the full AAD from "tenant-1" allowing us to use those to set policy. Is it right for now?

@chenrujun
Yes, sorry - I had misrepresented what I was putting into my application.yml because I had tenant-1's ID and tenant-2's ID confused. So what I had originally was actually:
spring:
   cloud:
      azure:
         keyvault:
            secret:
               property-sources:
                  - endpoint: key-vault-2-uri
                    profile:
                       tenant-id: tenant-2-id
                    credential:
                      client-id: client-1-id
                      client-secret: client-1-secret

And once I updated tenant-id to be tenant-1-id, it worked.

So here is the conclusion:
Context:

There are 2 tenants: teannt-1 and tenant-2. tenant-1 is the parent of tenant-2.
client-1 belong to tenant-1.
key-vault-2 belong to tenant-2.

Conclusion

client-1 can access key-vault-2 even though they belong to different tenant.
The value of property spring.cloud.azure.keyvault.secret.property-sources[0].profile.tenant-id and spring.cloud.azure.keyvault.secret.property-sources[0].credential.client-id should belong to same tenant.
The The value of property spring.cloud.azure.keyvault.secret.property-sources[0].endpoint and spring.cloud.azure.keyvault.secret.property-sources[0].credential.client-id not necessary belong to same tenant.

@zpdeering

Tell me if I have some misunderstanding.
Please close this issue if you don't have further questions.

@chenrujun you have it correct. I will close out the issue, thank you for the assistance!


Added Hungarian Translation

Thanks for your contribution @belidzs! I'll be happy to merge this and release a minor version including it after you look into my comments.

@krizalys thanks, I've pushed a new commit

Thanks @belidzs, your translation will be added to the 1.2.0 version, to be released later today.

@belidzs 1.2.0 is out and uploaded on JED.
Project pages have been updated to give you the due credit. Thanks for your contribution :)

Updated and works fine. Thanks!


feat: Switch to coveralls.

Coverage decreased (-81.6%) to 0.0% when pulling f7101cbd19299c13741535e7b9c16007061565a5 on dc/coveralls into c49359eb71ec644571fec17059a208e6471cafa7 on master.


Support QuadraticModel in ConstrainedQuadraticModel

Codecov Report

Merging #878 (f0c3fae) into main (06effd9) will decrease coverage by 0.00%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #878      +/-   ##
==========================================
- Coverage   85.65%   85.65%   -0.01%
==========================================
  Files          75       75
  Lines        6644     6664      +20
==========================================
+ Hits         5691     5708      +17
- Misses        953      956       +3

Impacted Files
Coverage Δ

dimod/quadratic/quadratic_model.py
90.39% <50.00%> (-1.67%)
:arrow_down:

dimod/constrained.py
89.65% <85.29%> (+1.55%)
:arrow_up:

dimod/serialization/fileview.py
90.54% <0.00%> (+1.35%)
:arrow_up:

Continue to review full report at Codecov.

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


Jobs get stuck into processing state until the server is restarted

I wanted to reporting this issue here for visibility. I first experienced this issue with the Postgres driver of which I opened this PR.
After a week or so, I started experiencing the same symptoms. I decided to try the Redis Queue driver it too had the same issue. I now convinced the issue is in this repository.
The issue is as follows:
1 - Push a message to the Job queue, sometimes the messages just get stuck in processing state and never processed
2 - Once the server is restated, all the messages in queue get processed
3 - The process repeats.
Environments
swift-driver version: 1.113 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.6 clang-16<IP_ADDRESS>) Target: arm64-apple-macosx15.0
Swift 5.10.1 Docker

Do you have the logs for a failed job?

Do you have the logs for a failed job?

The jobs don't fail, they get stuck with status processing. The jobs stay in the queue until the server restarts. After restarting the server, all jobs get processed without failure.

Ok the logs for the job that is stuck in the processing state. If search your logs for the UUID it should give us an idea of what happened.
By the way I have written a soak test for the job queue and run it with both Postgres and Redis and cannot get it to fail. Perhaps you can expand on the test to get it to fail. Here is the redis version
import Jobs
import JobsRedis
import Logging
import NIOCore
import NIOPosix
import RediStack
import ServiceLifecycle

var logger = Logger(label: "Soak")
logger.logLevel = .debug
let redis = try RedisConnectionPool(
    configuration: .init(
        initialServerConnectionAddresses: [.makeAddressResolvingHost("localhost", port: 6379)],
        maximumConnectionCount: .maximumActiveConnections(2),
        connectionFactoryConfiguration: .init(
            connectionDefaultLogger: logger,
            tcpClient: nil
        ),
        minimumConnectionCount: 0,
        connectionBackoffFactor: 2,
        initialConnectionBackoffDelay: .milliseconds(100)
    ),
    boundEventLoop: MultiThreadedEventLoopGroup.singleton.any()
)

let jobQueue = JobQueue(
    .redis(
        redis,
        configuration: .init(
            pendingJobInitialization: .remove,
            processingJobsInitialization: .remove,
            failedJobsInitialization: .remove
        )
    ),
    numWorkers: 4,
    logger: logger
)

struct MyJob: JobParameters {
    static var jobName = "Test"

    let sleep: Int
}

struct MyError: Error {}
jobQueue.registerJob(parameters: MyJob.self, maxRetryCount: 4) { parameters, _ in
    try await Task.sleep(for: .milliseconds(parameters.sleep))
    if Int.random(in: 0..<100) < 3 {
        throw MyError()
    }
}

try await withThrowingTaskGroup(of: Void.self) { group in
    let serviceGroup = ServiceGroup(
        configuration: .init(
            services: [jobQueue],
            gracefulShutdownSignals: [.sigterm, .sigint],
            logger: logger
        )
    )
    group.addTask {
        try await serviceGroup.run()
    }
    group.addTask {
        for _ in 0..<100_000 {
            try await jobQueue.push(MyJob(sleep: Int.random(in: 1..<20)))
            try await Task.sleep(for: .milliseconds(Int.random(in: 1..<10)))
        }
    }
    group.addTask {
        for _ in 0..<100_000 {
            try await jobQueue.push(MyJob(sleep: Int.random(in: 1..<20)))
            try await Task.sleep(for: .milliseconds(Int.random(in: 1..<10)))
        }
    }
    try await group.next()
    try await group.next()
    try await Task.sleep(for: .seconds(1))
    await serviceGroup.triggerGracefulShutdown()
}

let promise = redis.eventLoop.makePromise(of: Void.self)
redis.close(promise: promise)
try await promise.futureResult.get()

I created and POC https://github.com/thoven87/swift-job-stuck/tree/main which should help with the issue I am facing. https://github.com/thoven87/swift-job-stuck/tree/main
To run the job, you'll need SMTP credentials. I can forward credentials in DM on discord if needed.

Closing this issue as it seems to be an with SMTPKitten


Fix the distutils hacking behavior with the environment variable SETU…

Summary of changes
The change is to ensure the distutils hatching behavior works by default, except the environment variable SETUPTOOLS_USE_DISTUTILS is specified as "stdlib".
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d. See documentation for details

I specifically created this variable to be a non-boolean variable so that the value signals the purpose (rather than the presence/absence of the variable). I suspect you're coming to this variable expecting it to be a boolean value when it is not. The two supported values are currently "stdlib" and "local". It may make sense to raise a warning if any other values are specified, but I've been treating it as anything other than "stdlib" or "local" is the same as not setting the variable.
This PR is making some assumptions, fixing something that I don't think needs fixing. If you feel strongly about it, please file an issue describing the issue that this PR attempts to fix and we can explore possible fixes in the issue.
Thanks for your patience and for taking the time to prepare this PR.

When I upgraded to setuptools 50.0, my understanding of the intention was to hatch the distutils module by default, unless "stdlib" is specified (yes similar to a boolean value). It was bizzare to see the behavior falling back to the "stdlib" one even a typo / empty string was specified. So I thought it's a minor bug and set the PR without creating a ticket.
Now I saw the change in 50.1 to fall back the 49.6 behavior so I think you're right to close the PR as it's no longer valid anymore. Not sure if I understand what you meant of "I've been treating it as anything other than "stdlib" or "local" is the same as not setting the variable". I guess it's equivalent to falling to the default behavior if any other values are specified, isn't it?

I guess it's equivalent to falling to the default behavior if any other values are specified, isn't it?

Yes, that's right.


pre-commit: add pyupgrade, autoflake8, mypy, upgrade codespell

pre-commit:

add:

pyupgrade,
autoflake8,
mypy,

upgrade codespell, move options to setup.cfg

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4939

Relevant Lines:
6093

💛  - Coveralls

Thanks!


New version broke Hops!

Vanilla Tweaks Version:  1.2.10

MinecraftForge Version:  <IP_ADDRESS>54

Minecraft Version: 1.12

(Optional) Just Enough Items (JEI) Version:

Client or Server?:

[ X] Client
[ X] Server

Crash Report (via. Pastebin):

What happens / happened?:  Hops III does not work anymore on my enchanted boots

How can I reproduce it?:

(Optional) Mod List:

Hmm thanks for reporting it should be an easy fix but i have real life work but i will release a fix as soon as possible. Thanks for reporting

Looking at this new version broke all version

Pushed an new build to curse. Now to wait for approving. Can you make sure this works @Undesolate

Hey Strikerrocker - I'm logging in now using the build you created yesterday (4-15-18) not the one you uploaded today (4-16-18). When I tried to run the one you uploaded today I got this error and the game crashed via client side. Hops does seem to work though! I'll let you know if I encounter anymore errors.
Caused by: java.lang.NullPointerException
at io.github.strikerrocker.vt.capabilities.SelfPlantingProvider.(SelfPlantingProvider.java:12)
at io.github.strikerrocker.vt.events.VTEventHandler.addItemCaps(VTEventHandler.java:112)
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_142_VTEventHandler_addItemCaps_AttachCapabilitiesEvent.invoke(.dynamic)
at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90)
at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:179)
at net.minecraftforge.event.ForgeEventFactory.gatherCapabilities(ForgeEventFactory.java:657)
at net.minecraftforge.event.ForgeEventFactory.gatherCapabilities(ForgeEventFactory.java:627)
at net.minecraft.entity.Entity.(Entity.java:219)
at net.minecraft.entity.item.EntityItem.(EntityItem.java:73)
at cpw.mods.ironchest.client.renderer.chest.TileEntityIronChestRenderer.(TileEntityIronChestRenderer.java:43)

The new build fixed it. It was due to some refractor i was doing in the build before that.

I guess I'm confused.. This is what I see on twitch

This one dated 4-16-18 is the one that errors out. The one I currently have installed works perfectly.


WIP a middleware identity provider for zend authentication service + tests

proposal

Hey @basz ! @Orkin wants to use ZfcRbac in an Expressive app and want to take the lead on the net steps. Maybe you could try to join forces and make some progress on the port to Expressive?

Ah, now I see this comment. :)
That great news Orkin! Please grab my PR as you see fit. I am sort of planning to use this in an expressive app (a prooph app https://github.com/prooph/service-bus-zfc-rbac-bridge ) too after my holidays so I definitely will be able to do some testing. Unfortunately I need to point my focus on the actual application itself and will be using what works at the moment)

Tests still not passing :)

Changes Unknown when pulling 7961fbb500ab94dba21221e8a1bde6b84491c5a4 on basz:WIP-middleware-zend-auth into ** on ZF-Commons:develop**.

Are you taking this PR further @Orkin ?

Yes why not but I don't know where you want to go with :D

Same boat! I have no idea what the desired direction is for this repo. Removing state is one of the stated goals I believe. But removing assertions isn't part of the plan.
I have no direct involvement and only started this PR to gain some insights in how the middleware idea worked. Then things stalled :)
But I presume you must have some idea as you created a PR specifically against my PR?
Additionally I have no time to continue this ATM.
Bas

Hmm now i need to make some time as i need it in an expressive app too. :) I review Monday and do a proposal. I think most work is done already.

IMO this PR should be closed in favour of 316 which I now have working in an application. If we want to provide convenience tooling such a Middleware we're create a new PR from the most up to date dev state.


compilation times

A typical problem of c++ libraries is compilation time.
We're already running into this with the compilation time of the tester. Some solutions:

Explicit template instantiation to get some precompilation
Cool it a little with the templates, since more templates = extra compilation times (yes, i'm still against the side and op types ;) )
Instead of including <tlapack.h> only include the necessary headers in the tests and examples.

All good ideas @thijssteel.
Explicit template instantiation to get some precompilation => A little against. Main reason: this can wait. We might need to do this at some point though, I agree. Cool it a little with the templates => Yes, maybe. Same thing, maybe for now this can wait. But I see your point. This would help. Instead of compiling 2^4=16 versions of TRSM we would compile 1. This is mostly for our tests though, our tests test all 16 of them. I would assume a user code would only use a handful of these. Although, well, it goes fast. Indeed, someone easily uses quite a few of them. Instead of including <tlapack.h> only include the necessary headers in the tests and examples => Yes. We can try. Good idea for sure. This is the one idea I am the most in favor of.
Also to add to the list, one issue is ./tester.exe in itself, and it has all our unit tests, and we do not have many. This is not scalable. If we double the number of unit tests (which hopefully we will soon), we double the compile time. And, since this is a build that is done repeatedly during debug mode, development mode, it's really a pain for developer. We are small and it is already biting us. Can we break ./tester.exe in one executable per unit test?

A typical problem of c++ libraries is compilation time.

Yes, thanks for raising the discussion!

* Explicit template instantiation to get some precompilation

I don't think this would help, but we can time it.

* Cool it a little with the templates, since more templates = extra compilation times (yes, i'm still against the side and op types ;) )

This could help a bit in terms of compilation time. But I think we should think more on usage than compilation time, when taking decisions on this thread. I have in mind:

Every vector and matrix in the interface of a function can belong to a different class.
Parameters like Op and Uplo could be templated with trans_t and uplo_t to allow the benefits of compile-time conditionals. We can improve this design, sure. Let's talk more about it! :)
Integers, sizes and constants passed through the interface of a function don't need to have its own template type. We can certainly cool things here.

* Instead of including <tlapack.h> only include the necessary headers in the tests and examples.

This is the most relevant I think.

Solution: We provide separate testers, and add an option in the building system to enable/disable the compilation of the big tester.exe.

I like this idea.

Actually, when looking at the compilation, most of the time is spent building the .o files for the tests. Linking them together into the tester isn't that much work. If we make sure that only the relevant files are included, the building should be fast.

Actually, when looking at the compilation, most of the time is spent building the .o files for the tests. Linking them together into the tester isn't that much work. If we make sure that only the relevant files are included, the building should be fast.

I tried this solution here: https://github.com/tlapack/tlapack/pull/143, but the compilation times didn't change that much.
I also isolated the installation of Catch2. On Windows and MacOS, this may take close to 1min.

Also to add to the list, one issue is ./tester.exe in itself, and it has all our unit tests, and we do not have many. This is not scalable. If we double the number of unit tests (which hopefully we will soon), we double the compile time. And, since this is a build that is done repeatedly during debug mode, development mode, it's really a pain for developer. We are small and it is already biting us. Can we break ./tester.exe in one executable per unit test?

I forgot to comment on that. Tests were broken at some point, I forgot the number of the PR. And, with #143, the tests start to depend only on the files they use. Good news for us, developers!


[BUG] DefinedName and localSheetId lost when workBook.xlsx.load

🐛 Bug Report!
Except for print_title, DefinedName and localSheetId are lost.
Lib version<EMAIL_ADDRESS>Steps To Reproduce

make EXCEL-book by MS-EXCEL that's Named ranges with the same name on different pages.
Load the XLSX-Book-file in exceljs and write it.
　　(Nothing changes XLSX-object, as shown in the javascript-code at the bellow)
Compare the contents of the /xl/workbook.xml in the excel-book zip directory  to before load and after write.
Please see the attached figure.
Except for print_title, DefinedName and localSheetId are lost.

var workBook1 = new ExcelJS.Workbook();
workBook1.xlsx.load(fileData1)
 .then(workBook2 => {
    console.log("at loaded definedName id lost:", workBook2)
    return  workBook2.xlsx.writeBuffer()
                 .then(wbout1 => {
                      saveAs(new Blob([wbout1], { type: "application/octet-stream" }), newBookName);
                  });
                });
           })

Compared the contents:

The expected behaviour:

Expect the DefinedName and localSheetId not to be lost.
Similar to MS-excel, which allows exceljs-API to define named ranges with the same name on different pages.
I also hope that the following issues will be resolved.
https://github.com/exceljs/exceljs/issues/1497
https://github.com/exceljs/exceljs/issues/393

Possible solution (optional, but very helpful):
None, for long times.

+1


How would this work with components?

Do you know how this could work with Angular 1.5 components?
I am using angular-ui-router 1.0.0-beta.3:
https://ui-router.github.io/guide/ng1/route-to-component

Unfortunately not; I've not played around with Angular 1.5 Components as of yet!


KTOR-6745 Add a new tutorial 'Full Stack development with Kotlin Multiplatform'

KTOR-6745
Preview on staging

proofread and added new content from Garth's article 'Full Stack development with Kotlin Multiplatform'
added screenshots and gifs for light and dark theme
added and referenced code example
added link and meta descriptions

to do:

[x] replace the GIFs in "create a server" and "run the service" sections
[x] display procedures in side navigation
[x] add a TLDR element to the top with the code example and libraries used
[x] include links in the "Next steps" section

It has preview feature for the compose multiplatform and allows to run and debug both platforms in a single tool


Add recordtype by record, get djsonb fix

Adds a filter for RecordType to get the record type that corresponds to a given record, to enable things like record detail views to get the right type and schema given a record ID.
Also increments the djsonb version to get the fix in https://github.com/azavea/djsonb/pull/10.

Travis be grumpy about this one until that djsonb PR is merged.

+1


[Chore] Add codecov setup

What happened 👀
Trying out codecov service to visualize the code coverage of the repo
Insight 📝-

https://app.codecov.io/
https://docs.codecov.com/docs/supported-ci-providers

Proof Of Work 📹
Pending for the result

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@6c2629d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             develop    #108   +/-   ##
=========================================
  Coverage           ?   4.85%
=========================================
  Files              ?      19
  Lines              ?    1318
  Branches           ?       0
=========================================
  Hits               ?      64
  Misses             ?    1254
  Partials           ?       0

Flag
Coverage Δ

unittests
4.85% <0.00%> (?)

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

Continue to review full report at Codecov.

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


GPL License Violation in the kserve python package

The version of the kserve package that is currently on PyPI (version 0.10) violates the GPL license because it depends on table-logger, distributed under GPLv2 (you'll see that the library is now MIT, the author updated the license just a few days ago, but hasn't released a new version with the new license yet). No GPLv2 packages should be imported given that kserve has an Apache 2 license.
This was recently fixed by this PR https://github.com/kserve/kserve/pull/2673, which accidentally resolved the issue by replacing table-logger with tabulate (MIT License)
cc @yuzisun @cliveseldon @jinchihe @ellistarn
Is it possible to quickly release a patch release 0.10.1 to include the above patch and make sure kserve is compliant with the Apache license? As it stands, any distribution and vendor using kserve is liable for a license violation.

@StefanoFioravanzo i do not think this is released in 0.10.
@andyi2it can you confirm and see if we can remove table-logger ?

@yuzisun Indeed, the PR I mentioned was merged 3 days ago, so it's not part of 0.10. If we were to release a patch version (0.10.1) including that PR (https://github.com/kserve/kserve/pull/2673) we would solve this.

@yuzisun Indeed, the PR I mentioned was merged 3 days ago, so it's not part of 0.10. If we were to release a patch version (0.10.1) including that PR (#2673) we would solve this.

@StefanoFioravanzo thanks for the catch? Curious how you found out this and want to make sure we can prevent this next time.

@yuzisun We did an internal license audit of all the software we bundle in Arritko EKF, and this came up.  We will bring it up to the community if we ever detect anything out of the ordinary.

@yuzisun We did an internal license audit of all the software we bundle in Arritko EKF, and this came up. We will bring it up to the community if we ever detect anything out of the ordinary.

How did this come up as it is not released yet?

Hi @yuzisun , I see that you merged the cherry-pick to branch release-0.10, that's great! (Thanks @andyi2it ).
When you do you think you'll manage to publish a new version? To you have a timeline?

Hi @yuzisun , I see that you merged the cherry-pick to branch release-0.10, that's great! (Thanks @andyi2it ).
When do you think you'll manage to publish a new version? Do you have a timeline?
We have a few other bug fixes which needs to go in, the current ETA is to release 0.10.1 by end of month.

Closing this since 0.10.1 has been released. @yuzisun Thanks for the quick fix!


How do I compile this?

Hello,
How do I compile or run this?
Thanks

Hi, it's a library not an app, you include it in your project then use  its functions and classes


Docker swarm mesh networking not working

The only relevant log I could find on docker was Jan 17 20:37:33 scw-f118f1 dockerd[3495]: time=“2018-01-17T20:37:33.047047682Z” level=error msg=“Failed to deserialize netlink ndmsg: Link not found"
Steps to reproduce:

init a swarm
join on scaleway node
deploy something like docker service create
--name my-web
--publish published=8080,target=80
--replicas 1
nginx
try to curl from a scaleway host which doesn’t have a copy of the service running
curl will hang there, then say it has timed out

what kernel do you use?  I head no problem running swarm make sure you have the right kernel you can google for kubernetes/scaleway. Same kernel recommendations apply to swarm.

@Vad1mo I tried most of the available kernels, same results every time.

What's actually not working? What is the error message?
I had Swarm running without any problems a few months ago.

https://docs.docker.com/engine/swarm/ingress/#publish-a-port-for-a-service
Suppose the scaleway node is in the same situation as node3, it should forward the requests to a node running the published service. What happens is some error in the communication between nodes and the client connection times out.

when setting up swarm mode you should use the private IP address not the public ones.

You do need to use the public IP when your cluster is located on more than one datacenter. In my case, I set up a swarm between amazon, azure and scaleway instances. Communication between amazon and azure nodes went just fine.

yes because scaleway public IPs are NAT to private instance. This is a different to AWS/Azure. This might be the problem.

They all use NAT. And I disabled all the three firewalls.
When running nmap, I found something interesting: while amazon and azure nodes would always keep the communication port open, the scaleway node would inconsistently open and close said port. Actually, you might be able to get some traffic a few moments after restarting, but the port will shut closed soon enough.

do you see why the connection is closed? Any message/status code? HTTP or TCP level?

I might have overlooked it, but the only message I saw on the docker daemon which could be relevant was the single line on the op. Now I regret not dumping the full log, as I already teared down this setup (but would bring up again if needed).

+1
I am not even in any cloud - We have hosted private cluster and nodes has been joined over private network  (not any NAT)

Apparently the issue is the kernel conf of the default images, there are missing modules that are needed for the overlay network. The “Failed to deserialize netlink ndmsg: Link not found" is unrelated
See:

https://github.com/moby/moby/issues/28168#issuecomment-259202309
https://adminswerk.de/docker-swarm-scaleway/
https://github.com/scaleway/image-ubuntu/issues/78

having exactly the same problem service ports are not published.
Using the private IP also isn't ideal since if you reboot your server, you kill the swarm because the IP address get's changed.

BTW, to fix the network issue I had to change the bootscript to the 4.15.* one. The with "Docker" in the name didn't work.

Tested all bootscripts available for the paris start1-xs instances. None will allow me to use swarm. It connects to the swarm, but none of the containers are reachable from any other node. I don't have this issue with any other of my swarm nodes.
Will this be fixed anytime? Because without swarm, the server is sadly useless to me.

I have the same issue with Docker installed as one click app and previously with Docker installed on Debian. The version of Docker is 18.09.
I have one manager and 3 workers.
My manager and two of the workers are in par1 zone. The last worker is an OVH server.
I created the swarm using this command :
docker swarm init --advertise-addr public_ip.
Then all the workers joined the swarm with this command:
docker swarm join --token SWMTKN-1-TOKEN public_ip:2377
It worked and all my nodes are detected.
My services are also distributed to the nodes. No problem.
If the services are on the same node, they can communicate.
But if they are on different nodes, they can't. When I connect into a container, I see that, they can ping each other (using their service name or IP) but, as an example, mongo client cannot access a mongo database (connection time out). It's not just a problem with mongo as redis have the same problem.
I have tested all bootscripts but not one makes it work.
However, it's worth to note that I have this error logs on the node that have the mongo/redis database :
time="2019-02-03T18:03:08.231472040Z" level=info msg="API listen on /var/run/docker.sock"
time="2019-02-03T18:03:11.121914063Z" level=info msg="Node 8cebd88d9847/<IP_ADDRESS>, joined gossip cluster"
time="2019-02-03T18:03:11.122878954Z" level=info msg="Node 8cebd88d9847/<IP_ADDRESS>, added to nodes list"
time="2019-02-03T18:03:11.123329059Z" level=info msg="Node a37309267714/<IP_ADDRESS>, joined gossip cluster"
time="2019-02-03T18:03:11.124267840Z" level=info msg="Node a37309267714/<IP_ADDRESS>, added to nodes list"
time="2019-02-03T18:03:23.121109458Z" level=warning msg="memberlist: Refuting a suspect message (from: 8cebd88d9847)"
time="2019-02-03T18:05:03.150819926Z" level=warning msg="memberlist: Refuting a suspect message (from: 8cebd88d9847)"
time="2019-02-03T18:05:19.120830135Z" level=warning msg="memberlist: Refuting a suspect message (from: 8cebd88d9847)"
time="2019-02-03T18:06:38.186586864Z" level=error msg="Bulk sync to node 8cebd88d9847 timed out"

I googled about it but found no answer.
Moreover, I can't exactly understand what is from:8cebd88d9847 in the previous command.
Does anyone have a fix for this ? Or an idea ?
I said earlier, without swarm the server is useless for me. I also already searched for this error for more than 20 hours.
Here is the result of Docker info :
Containers: 4
 Running: 4
 Paused: 0
 Stopped: 0
Images: 2
Server Version: 18.09.0
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: active
 NodeID: 8shuebl5wzitghr8244dpgo5y
 Is Manager: true
 ClusterID: s6rear2esk0swp9f1acbgg6vt
 Managers: 1
 Nodes: 4
 Default Address Pool: <IP_ADDRESS>/8
 SubnetSize: 24
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 10
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
  Force Rotate: 1
 Autolock Managers: false
 Root Rotation In Progress: false
 Node Address: public_ip.136
 Manager Addresses:
  public_ip:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: c4446665cb9c30056f4998ed953e6d4ff22c7c39
runc version: 4fc53a81fb7c994640722ac585fa9ca548971871
init version: fec3683
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.14.33-mainline-rev1
Operating System: Ubuntu 16.04.5 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.95GiB
Name: server-name
ID: XJ6Y:DQDH:NOQ5:IRTF:BFYM:JVNB:3764:WDWX:GIUN:WYKN:USZ5:ZXTI
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false
Product License: Community Engine

@lauevrar77 never find any fix for this issue, and i think actually scaleway don't care about docker swarm, they try to promote kubernetes instead.

Same here @lauevrar77. I moved to Hetzner cloud, everything working flawlessly.

same here. Suddenly scalway feels quite useless... :'(

Recently I've faced some issues creating swarm cluster (using Caprover project) and after days of struggling I've found this issue, so I'll bump it for others to know its still relevant and some info what I was facing. Reproducing the same flow on digital-ocean was successful.
I’ve got a cluster of 2 nodes on scaleway (tested with Elastic Metal, Compute and combinations) connected in private network. And I can’t access my srv-captain--postgres service on manager from worker inside of the container. However everything seem to be configured correctly.
What I can do:
On Manager node (<IP_ADDRESS> this is the IP assigned by VPC to Manager Node)
manager-node> $ docker exec -ti $() psql -h localhost (from postgres container)
manager-node> $ docker exec -ti $() psql -h srv-captain--postgres (from another container)
manager-node> $ psql -h localhost from host, port is exposed
manager-node> $ docker exec -ti $() ping srv-captain--postgres (resolves to <IP_ADDRESS>)
manager-node> $ docker exec -ti $() psql -h <IP_ADDRESS>
On Worker node:
worker-node> $ psql -h <IP_ADDRESS>
worker-node> $ docker exec -ti $() psql -h <IP_ADDRESS>
worker-node> $ docker exec -ti $() ping srv-captain--postgres  (resolves to <IP_ADDRESS>)
However, what I can not do on worker is this: worker-node> $ docker exec -ti $() psql -h <IP_ADDRESS> — it leads to timeout saying:
2023-08-14T15:15:16.171836731Z PG::ConnectionBad: connection to server at "<IP_ADDRESS>", port 5432 failed: Operation timed out
2023-08-14T15:15:16.171841340Z Is the server running on that host and accepting TCP/IP connections?

But:
postgres definitely works on this port and IP
pg_hba.conf has host    all             all             <IP_ADDRESS>/0            md5
ufw is disabled on both manager and worker
postgres port 5432 is exposed


Crash after many minutes not answer for question

Hey,
I am not sure this is a bot problem or maybe this is API issue but he do not answer for question and leave IRC with "timeout"
14:29 <@TaKeN> Zygmunt: Wydaje mi się, że premiera 14th generacji intela będzie chyba w 2023 roku?
14:31 <@TaKeN> Zygmunt: jesteś tam?
14:32 <@TaKeN> Zygmunt: która jest godzina?
14:34 -!- Zygmunt<EMAIL_ADDRESS>has quit [Ping timeout]
and after another few minutes he crash:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/openai/api_requestor.py", line 672, in _interpret_response_line
    data = json.loads(rbody)
  File "/usr/lib/python3.9/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.9/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.9/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/root/chatgpt/./chatgpt.py", line 31, in <module>
    response = openai.Completion.create(
  File "/usr/local/lib/python3.9/dist-packages/openai/api_resources/completion.py", line 25, in create
    return super().create(*args, **kwargs)
  File "/usr/local/lib/python3.9/dist-packages/openai/api_resources/abstract/engine_api_resource.py", line 153, in create
    response, _, api_key = requestor.request(
  File "/usr/local/lib/python3.9/dist-packages/openai/api_requestor.py", line 226, in request
    resp, got_stream = self._interpret_response(result, stream)
  File "/usr/local/lib/python3.9/dist-packages/openai/api_requestor.py", line 619, in _interpret_response
    self._interpret_response_line(
  File "/usr/local/lib/python3.9/dist-packages/openai/api_requestor.py", line 674, in _interpret_response_line
    raise error.APIError(
openai.error.APIError: HTTP code 502 from API (
<title>502 Bad Gateway</title>

502 Bad Gateway

nginx

)

So maybe add support for 502/500 errors? and ofc handling timeouts.

I will take care of it this week, thanks for reporting all the issues!


simpler directory structure for python/resources

trying to find a structure with fewer directories and the ability to package resources into released wheels. this is what i'm currently finding pleasant along with some naming conventions for certain types of python packages.

src/rats/… for standard rats python import packages belonging to the rats.* namespace package.  this is what we have now but without the additional subfolder src/python.
src/rats_resources/ as a new namespace package that is meant to only hold non-python files. i follow the same naming conventions as src/rats where there is a sub-package per domain; but files inside each domain follow the naming conventions of standard resources which uses dashes instead of underscores.
src/rats_e2e also follows the structure of src/rats and contains the end-to-end or integration tests of the component. these are shipped with the wheel unlike the things in test/ specifically because they are meant to be runnable without testing framework. these should run on final artifacts before publishing instead of being run in a development environment.
test/* matches the new directory structure above without the _e2e package. unit tests and lightweight integration tests meant to be run on a development environment install that includes tools like pytest.

this doesn't stop us from using any other rats_* prefix to packages but it's starting to show a convention with rats_e2e (new), rats_resources (new), and rats_test (existing) where we group things at the top level before giving people short and descriptive import names below the top level.
Example:

src/rats/apps has libraries for creating applications
src/rats_e2e/apps has some example apps and other runnable code to ensure rats.apps is well integrated
src/rats_resources/apps/some-asset.txt has some information used by the rats.apps libraries
test/rats_test/apps has the unit tests for the public facing api of rats.apps

How about moving resources and e2e folders to the same level as src and test?
src/rats/apps
resources/rats
e2e/rats
test/rats

Just thinking e2e and resources are not exactly source code...

a few notes after our chat this morning:

rats_e2e is part of src/ because we want to ship it and make it visible to users of the libraries.
resources/rats should be possible without custom build-system configs, but both test/ and src/ have their own resources, so i'm not sure thinking about resources at the component level is possible. i need to be able to define resource files used by tests without shipping them with my wheel. i was thinking of trying resources/rats/… next to resources/rats_test/… at the root, but it would take custom build system configs as well and leave us without being able to support all of them.

i think today the main difficulty here is the non-standards around configuring the layout of your project. PEP 621 helped make the [project] section consistent, but it does not specify any standards around the build system. so my intuition is to use completely standard structures which would only allow for src/ and test/ to exist as the two folders where one is for the wheel (src/) and one is for local development (test/).

I don't like src/rats_resources because I don't know if that is a python package or actual resources and you have to look inside to know.

i think this has been my struggle the entire time. i think of resources as "not a python package" but the python ecosystem very much does not. so after this restructure, the answer to "is this a package?" is always yes; and some packages contain resources.
the python community puts .py files next to any related resource. the proposed structure is still fighting against that convention, but accepting the other one. what i'm doing here is saying that any package ending in _resources is the sibling to the non-suffixed one that contains the relevant resources. so we'll have a single __init__.py file in rats_resources/ in order to make it a valid python package, and everything else can be assumed to be the resource for the matching package in rats/. this is the most separation i've been able to achieve so far without relying on undocumented behavior in any given packaging tool. everything else i've tried either works in the development environment, or in the wheel installation, but never both.

Ok, I don't usually require resources, but if they are part of packages and the community puts them alongside python modules, I am fine with that. We can follow the convention you propose in rats, but I am not too keen on extending this convention to other projects, if there is no consensus in the community.
But thanks for explaining all of this, and documenting the inconsistencies between setting up the dev environment and packaging...


  Whether config master is set to local or local[*], DataFusion Comet does not significantly improve performance, and there are even many queries that result in negative gains. Could you please help check if my configuration is incorrect? I tested with 100GB and 1TB TPC-DS datasets, and the performance improvement with DataFusion Comet is very low, with the total query duration improving by only about 6%. My container specifications are 128 cores and 1TB of memory.🙏🙏🙏

Hi @shaileneF Are you testing with the 0.3.0 release or the latest from the main branch? I am going to be running benchmarks today and tomorrow in preparation for the 0.4.0 release so will share my results with you.

Hi @shaileneF Are you testing with the 0.3.0 release or the latest from the main branch? I am going to be running benchmarks today and tomorrow in preparation for the 0.4.0 release so will share my results with you.

Yes，0.3.0，I download the release jar from https://datafusion.apache.org/comet/user-guide/installation.html.
thank you for running the benchmark. I want to know my spark-submit config is right or not.

One more question @shaileneF ... is your data set partitioned by date?

One more question @shaileneF ... is your data set partitioned by date?
the dataset was partitioned during generation, but it was not partitioned by date.

One more question @shaileneF ... is your data set partitioned by date?

Here is the dataset generation shell.
https://github.com/apache/incubator-gluten/tree/main/tools/workload/tpcds/gen_data/parquet_dataset

We likely need to resolve https://github.com/apache/datafusion-comet/issues/1123 to get better performance results


typo

The note at the top of the page shows "Microsoft Entra Verified ID is now Microsoft Entra Verified ID" which is redundant, it should say something like "Azure AD Verified ID is now".....

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

ID: d71e007d-1acb-61f7-89ca-3dd907f53015
Version Independent ID: 7a27bfc9-587a-d302-78e5-7513bb97bbbe
Content: Tutorial - Quick setup of your tenant for Microsoft Entra Verified ID - Microsoft Entra
Content Source: articles/active-directory/verifiable-credentials/verifiable-credentials-configure-tenant-quick.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

@DeanGross
Thank you for bringing this to our attention.
I've delegated this to content author @barclayn, who will review it and offer their insightful opinions.

@DeanGross thank you for pointing this out. #please-close


Sugar for recipe2plan: making use of namespace to shorten particle class path

After this PR, if a particle path starts with ., we will prepend the namespace path. This will help developers write less verbose manifests.

Will merge after TAP is fixed and we can pass the presubmit.


Construct the topology map instead of just adding to the map

We were just adding to the existing topology map any new things that were there but never removing the non-existent node labels. This resulted in a situation where the topology map will still show a node's labels even after the node was deleted. So fixing it by constructing the topology map from scratch in each reconcile so that it will always be in sync with the current state of the cluster.
Solves: https://bugzilla.redhat.com/show_bug.cgi?id=2102304

solves : https://bugzilla.redhat.com/show_bug.cgi?id=2102304

/cherry-pick release-4.13

/hold for testing

This is now tested,
/hold cancel

/cherry-pick release-4.12


[Request] Ability to replace/redefine custom element

I'm interested in the ability to replace a custom element with a new one while preserving the property values of each instance of that element.  This would have very positive implications for live component reloading.  I played a little bit with re-assigning and re-stamping templates, but my impression is that this feature would rely at least somewhat on Polymer providing some sort of low-level API.

I took a first shot at this using private Polymer APIs.  Here's a very small proof-of-concept to inject one element's template into another then re-render it,
const a = document.querySelector('el-a');
const b = document.querySelector('el-b');
a._template = b._template;
a._initFeatures();

I'm sure there are problems with this.  For one, I get a deprecation warning from Chrome,
Calling Element.createShadowRoot() for an element which already hosts a shadow root is deprecated. See https://www.chromestatus.com/features/4668884095336448 for more details.

So I'm not sure what the current way to replace a shadow root is.  According to @ebidel's article from 2013, it's to call createShadowRoot() multiple times, which is now deprecated.

custom-element definitions must be unique as described by the specification. Therefore redefining a custom element is disabled by the spec.


issues using cpp_utils -

I am trying to use the cpp_utils library in my platformio environment (VSCode), but having some problems.
Probably user error, but figured I would ask:
https://community.platformio.org/t/vscode-cannot-include-private-library-intellisense-not-working-and-build-fails/4997
Now, I have to add cpp_utils to every include statement in every file in the cpp_utils folder ....  seems like there is anotther way but right now, still working through that.

Im not working with platformio but i am using VSCode with esp-idf and most of files are not recognised by intellisense. Maybe its my fault and i dont know how to configure it, but i am using msys cli to build binary file so its not big issue to me (i dont have to include any cpp_utils files except those i am using).

Ah... Intellisense does work for me...and when I added the cpp_utils
library, it totally finds those files too.
Just wondering if I am doing the includes wrong.
On Fri, Sep 7, 2018, 10:10 AM chegewara<EMAIL_ADDRESS>wrote:

Im not working with platformio but i am using VSCode with esp-idf and most
of files are not recognised by intellisense. Maybe its my fault and i dont
know how to configure it, but i am using msys cli to build binary file so
its not big issue to me (i dont have to include any cpp_utils files except
those i am using).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nkolban/esp32-snippets/issues/642#issuecomment-419488657,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmH_Vte0zW3nutUl9PgKm7_sb8SuQgtks5uYppbgaJpZM4Wesa1
.

There was few questions about platformio in here, maybe some answer will help you.
https://github.com/nkolban/esp32-snippets/issues?utf8=✓&q=is%3Aissue+is%3Aopen+platformio

alright, getting it to recognize the cpp_utils but had to add the files I am using to another folder
cpp_utils->WiFi (just a name, doesn't have to be this) ->src -> files.
They show up in intellisense.   there is a squiggly line under the include file, but it will navigate to it and compiles just fine.
lib_extra_dirs = lib/cpp_utils
and it recognizes any subfolder in there, but not the individual files.


[UI] secondary storage - Display text and change the badge color of the Read-only column

Description
Fixes https://github.com/apache/cloudstack/issues/5169

Types of changes

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

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

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

Screenshots (if appropriate):

How Has This Been Tested?

@blueorangutan ui

@davidjumani a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5176 (SL-JID-303)

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5176 (SL-JID-310)


Discussion about Support for Multiple Programming Languages

@magaton @Major-wagh @sandeshchand @alikinir Adapted from issue #60 for continued discussion in this thread.
Background
Currently, the repository primarily supports Python by leveraging jedi and custom-built modules to construct the Repository Map. Developers with the necessary skills are encouraged to understand, modify, and adapt this functionality for their specific needs. Due to the substantial workload required, this project will not consider supporting additional languages in the short term. The functionality constructs the Repository Map and saves it to .project_doc_record, which serves as the basis for subsequent features in the repoagent project:
https://github.com/OpenBMB/RepoAgent/blob/825d988127d7bfd757237d9c4e8678d9104030f0/repo_agent/doc_meta_info.py#L296-L391
Community Interest
Multiple users have expressed interest in extending this capability to additional programming languages and have provided valuable feedback. We sincerely appreciate their contributions and ideas. Below are some of my thoughts on this topic.
First, the aider project utilizes tree-sitter to implement their Repository Map feature. However, it has limitations. Since different programming languages have unique features, supporting multiple languages simultaneously requires developers to possess strong cross-language expertise.
I have personally attempted to use tree-sitter to mimic jedi for constructing reference relationships here.
However, this effort has been temporarily shelved.
Additionally, RepoGraph has made progress in this area and published a related paper.
The work introduces an effective plug-in, repo-level module that provides the desired context and significantly enhances LLM-based AI software engineering capabilities.
Another approach is the Language Server Protocol (LSP). With implementations for many languages, LSP can assist in static analysis.
Proposal
Based on the above, my personal suggestion is to adopt different implementations for different programming languages to assist in analysis. The results can then be stored in a unified format to integrate seamlessly with repoagent.
This is a preliminary idea intended to spark further discussion.

Hello, you are referring to RepoGraph. How is that different / better from Aider's repomap?
This is what Aider brings:

support for all the languages from py-treesitter-languages
max-token as parameter
construct networkx graph based on the call references
use pagerank to detect most important nodes


Shell add dictionary command

Added dictionary commands concept.

Added dictionary commands concept.

can you add some more details about this concept? How about describing this in the documentation as well?

Added dictionary commands concept.

can you add some more details about this concept? How about describing this in the documentation as well?

I've updated the documentation and I've added some examples.


fix: add the ability to upgrade terraform via version stream

so we can lock down the terraform module repository versions in the version
stream

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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


[North Star Applicant UI] Add domain info to footer

Replace "domain.gov" with the CE instance's domain.  For example "seattle.gov".
Replace the "cityorcounty.gov" link with the value of this environment variable:  COMMON_INTAKE_MORE_RESOURCES_LINK_HREF.   If the variable value is blank, don't show the "Looking for more information..." in the footer.

So for now our footer shows this

This bug tracks adding the necessary info. It depends exactly what content we decide to add how much plumbing we'll need to add

This could be a P0 or P1

The URL for the civic entity can be found by calling this method:
public Optional<String> getCommonIntakeMoreResourcesLinkHref(RequestHeader request)
This part of the HTML code controls the footer: https://github.com/civiform/civiform/blob/d29bac8dc3692c4cb3e770bace5996bbc1176a16/server/app/views/applicant/NavigationFragment.html#L263

This issue is done when ALL of the following are true:

[ ] The footer shows the URL of the civic entity's website
[ ] The footer shows "Looking for more information or services? Visit cityorcounty.gov" (replace "cityorcounty.gov" with the actual URL)
[ ] Text is styled appropirately using CSS
[ ] Hyperlink(s) are styled appropriately using CSS

@erikang20 Here's another good starter issue


route53: update managed by DNS record comment

I got confused when I saw that we had records "Managed by Lego", when we're not running Lego. Updated to say "Managed by cert-manager" instead which I think makes more sense.

/release-note-none
/ok-to-test
/approve
/lgtm
Let's see how many of these commands work for me :)

Thanks for the fix by the way, good catch.
I'll leave the remaining commands for @munnerz

/lgtm
/approve
/release-note-none


Provided R code in the sample does not work

The provided R code has bugs.  The missing data is not provided by using MICE, and the new column is not added to the output as the column name is misspelled in the script.

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

ID: a4253d0e-7a9f-a0a7-edee-4cbd387ec98d
Version Independent ID: 14a7818e-43ac-86ca-a4a1-d4da38441e42
Content: Use R in Power Query Editor - Power BI
Content Source: powerbi-docs/connect-data/desktop-r-in-query-editor.md
Service: powerbi
Sub-service: powerbi-ai
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @Rich-at-Chester-Business-Analytics. This issue has been assigned to the doc author for review.

Thanks for your feedback. We've fixed the code (and added a couple screen shots) to make it all work. Thanks!


TypeScript definitions are outdated

Describe the bug
It seems to me TypeScript definitions are outdated for pinecone-route v4.3.0.
You can check it here: https://codesandbox.io/p/sandbox/pinecone-router-dvlkv3?file=%2Fsrc%2Fmain.ts
To Reproduce

Start new project.
npm i alpinejs pinecone-router
Use window.PineconeRouter.settings.templateTargetId or window.PineconeRouter.context in code.
npm run tsc

You will see TypeScript errors:
error TS2339: Property 'context' does not exist on type '{ version: string; name: string; viewCache: {}; settings: Settings; notfound: Route; routes: Route[]; currentContext: Context; add(path: string, options?: {} | undefined): void; remove(path: string): void; loadStart: Event; loadEnd: Event; }'.

error TS2339: Property 'templateTargetId' does not exist on type 'Settings'.

Also VSCode IntelliSense suggests currentContext and viewSelector fields which was changed in 4.0.0

Thanks for the report! I forgot to use build:types script. to avoid this I merged the command into the build cript

v4.3.1 should have the updated types.
Feel free to reopen if the issue isn't fixed

Thanks a lot!
Everything is fine now.


⚠ GitLab Server has degraded performance

In 5e5d7d3, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6763 ms

Resolved: GitLab Server performance has improved in 2718226.


Trying to define functions in REPL doesnt work

It seems to work but isn't callable. Trying to get just the value causes segfault

fixed in v0.3.7


[wip] 111, 100 - generic enum return type and enum errors

fixes #110 #111

Current coverage is 93.77% (diff: 67.08%)

Merging #114 into master will decrease coverage by 3.33%

@@             master       #114   diff @@
==========================================
  Files            27         29     +2
  Lines          1039       1092    +53
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1009       1024    +15
- Misses           30         68    +38
  Partials          0          0

Powered by Codecov. Last update 6f00bbc...31ccdb2


Support a maxBondLength parameter in MolDrawOptions

Really small molecules, like ethane, are drawn to fill the available space:

This can end up looking pretty silly, particularly in a setting where other images have larger molecules drawn.
We could resolve this by adding an option to MolDrawOptions which allows the maximum bond length (in pixels?) to be set.

@DavidACosgrove : seem like a reasonable idea to you?

I don’t have a computer handy at the moment so can’t check but I think you can fix a bond length in pixels. If it makes the scale such that the drawing would go outside the image, the scale is reduced so it is effectively a maximum.

I don’t have a computer handy at the moment so can’t check but I think you can fix a bond length in pixels. If it makes the scale such that the drawing would go outside the image, the scale is reduced so it is effectively a maximum.

yeah, sorry, I got confused by the name of the fixedBondLength option.


feat/DCV-1047 Various generate properties changes requested by Noel

Changed --models flag for different dbt ls selectors: --select/-s, --exclude, --selector
Added messages to both generate properties and sources: on .sql and .yml file creation/recreation/update
Fixed yaml indentation when updating an existing file: implemented new Dumper with standard yml indents
Updated README.md

Codecov Report
Base: 33.81% // Head: 32.42% // Decreases project coverage by -1.39% :warning:

Coverage data is based on head (c69579f) compared to base (31b403a).
Patch coverage: 18.10% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #249      +/-   ##
==========================================
- Coverage   33.81%   32.42%   -1.40%
==========================================
  Files          29       25       -4
  Lines        2055     1949     -106
==========================================
- Hits          695      632      -63
+ Misses       1360     1317      -43

Flag
Coverage Δ

unittests
32.42% <18.10%> (-1.40%)
:arrow_down:

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

Impacted Files
Coverage Δ

dbt_coves/tasks/setup/all.py
71.42% <ø> (+3.78%)
:arrow_up:

dbt_coves/tasks/setup/main.py
100.00% <ø> (ø)

dbt_coves/utils/flags.py
16.52% <0.00%> (ø)

dbt_coves/tasks/extract/airbyte.py
23.65% <7.69%> (ø)

dbt_coves/tasks/generate/sources.py
21.85% <8.69%> (+3.55%)
:arrow_up:

dbt_coves/tasks/generate/base.py
18.40% <15.57%> (-6.60%)
:arrow_down:

dbt_coves/tasks/generate/properties.py
34.42% <22.64%> (-4.66%)
:arrow_down:

dbt_coves/utils/jinja.py
25.00% <42.85%> (-2.28%)
:arrow_down:

dbt_coves/config/config.py
60.16% <87.50%> (-0.64%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


CityCamp NC logo'd cups, etc.

for lunch

I have been told we have plenty of stickers

We have 2 of these cards open for cups, one if to do and one in progress... not sure which is correct

Ran out of time to assure delievery

I thought we decided 2 mtgs ago we didn't need logo cups that we were getting regular store bought?


Coverity automation

Created on behalf of @pgiffuni
It would be really nice to get the Coverity automation working on github. Just to give ourselves something else to co ;).

I can't find coverity are you referring to synopsys?
Wasted quite a bit of time trying to get scrutinizer configured... got to love it when things just don't work =( Basically I wanted test coverage reports and finally found a solution with codecov which required zero configuration... booyah!!! See README for banner.
Please point me in the right direction and I will gladly give it a shot.

Ok so if it was synopsis (Coverity Scan) then I've tried.
I registered with GitHub account went to DashBoard to add project, everything but beanshell appears as my GitHub projects. The tip says to see if my member status is public, chased that wild goose too.
Proceeded to manually add the project... hit a What The?!?!? it expects a reference URL which from what I can tell is a link to a commit in my name, supposedly to prove my involvement or something... what's the point of this? I can have commits in a repository if a PR of mine was committed. So what is this trying to prove exactly.
Just before I submitted it caught my eye that there will be a 48 hour approval delay... now who has time for that? Screw this...
Is it too much to ask to expect a public and live system to work? There really isn't much more to it.

Do you think I am likely to return or suggest this service to anyone else?

Sorry for the delay ...
Indeed I meant coverity-scan which was bought by Synopsis
It is supposed to be relatively easy to setup from Travis, which we already have.
I tried once to add the project logging with my github account but failed miserably. What you mention sounds familiar, so I guess we need someone with patience :(. The service is rather nice once you get it working.

@pgiffuni Sweet... at least we are talking about dim same apples.
If you also struggled getting the project configured then indeed, it appears to be for everyone and not just for us. This probably means the next approach should be manually adding a project and waiting the 48 hrs for moderation.
Kinda lost the warm and fuzzies after the initial struggle which now unfortunately feels like a chore. If I do find myself needing a challenge I will certainly give it another whirl. Watch this space for more...

Not quite sure what to report back, it seems to be doing something.
Had to remove "application access restrictions" under the project settings, "Third-party application access policy" for coverity to list our project. It appears that we only have coverity application access on a personal level and this restriction appears to prevent personal applications from accessing project data. The status now reads:

All applications authorized by organization members have access to beanshell’s data.

And adds a less comforting red exclamation mark icon on the page which leaves much to be desired.
Only took 3 attempts to configure travis before coverity accepted the token and allowed the magic to happen... a build was uploaded to the project page and after a little patience it was analysed.
The report states we have 39 defects which is very surprising I expected much worse. However it appears they are preventing me from accessing the details. (Hurry up and wait...) From what I can tell this is pending a 1 to 2 business day lock for someone to review my association as well as our projects open source-ability or some such.
I invited @pgiffuni and @stain, since I can confirm you are active but should anyone else require access please don't hesitate to shout.
Watch this space for more feedback... in or around 1 or 2 business days give or take =)

Ok something just happened which hasn't happened before, your user was added to the project member list and I sent the invite again, using the apache.org email address that I have of yours.
The project status however still remains pending their review so we wait some more. Let me know what you find...

I don't use my apache address on github. Try giffunip_at_ yahoo.com. Apparently I could try to register the project too but lets wait.

Invitations created successfully.
1 users added to project.
1 invitation email sent successfully.

I added a badge to the readme!
Great job, really!

👍

Small note: I activated the Findbugs(TM) reports, so we may see a sudden surge in bugs, including false positives, for the next sweep.

Oh great yeah! =)
@pgiffuni I am reopening this thread, something tells me there is still some to discuss.
We already have 2 false positives but I was not able to create the appropriate "modelling file" for this. Their documentation shows the C versions of what they require but for java they don't have too much to say and what they do say is how to increase detection not how to avoid already detected false positives.
The option of adding annotations makes zero sense. If they did this the same way as with C by identifying simple comments I would agree. Instead they implemented this with java annotations which require us to have a coverity dependency in order to describe the annotation objects. Unless you disagree with me, I don't believe the costs involved in adding additional dependencies is justified for code analysis.
You obviously noticed I managed to clear the majority of the defects but the most alarming ones they don't seem to identify. The CWE-22 which they claim they found 2 occurrences of and has it flagged as "High impact security" but they fail to give us any indication of how or where we are supposed to find this.
I completely understand what CWE-22 is referring to, it basically needs us to prevent file traversal access outside of a predefined sand box. ie. if current working directory is /home/username then you should not be able to supply a path ../../etc and gain access to the system configuration.
This is all fine and well but without telling me where this is happening patching against their panic is going to be very difficult. There are several places in our code where this could be raised some of them will not be a concern others a necessity ie. class path needs to be able to go anywhere you please but unless you are pointing to java classes (jar files, modules, etc) it will not be of any use to you.
Before applying any restrictions we first need to identify if it is a risk and whether we need to fix it and how we want to restrict traversal (ie what should be the sandbox). Without knowing what exactly they have flagged I don't know how we are going to address this. Your thoughts?

Closed resolved we have coverity coverage automation


Include class constructor types in dependency graph

Now class Foo(a: Bar, b: Baz) extends Qux would produce the following dependency graph:
       Bar
       /
Foo ---- Qux
       \
       Baz

Still includes primitives and library types such aš Int or String, and could also be enabled by terminal flag.

Thanks for this! Looks good so I've now merged it in.