The model and loaded state dict do not match exactly

Hello, thank you for providing great code.
After I downloaded 'detr3d_resnet101.pth' and ran the code
'bash tools/dist_train.sh projects/configs/detr3d_r101_with_seg_depth_shift.py 1 --auto-resume',
I got an error like below.
I don't know why there some mismatch between parameters.
Thanks.
> Loading annotations...
> /home/user/anaconda3/envs/shift3D2/lib/python3.8/site-packages/mmdet3d/utils/misc.py:30: UserWarning: There are no checkpoints in the path.
>   warnings.warn('There are no checkpoints in the path.')
> 2024-04-05 00:11:55,817 - mmdet - INFO - load checkpoint from local path: ckptdir/detr3d_resnet101.pth
> 2024-04-05 00:11:56,077 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.0.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,079 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.1.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,081 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.2.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,082 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.3.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,084 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.4.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,085 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.5.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,087 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.6.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,088 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.7.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,090 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.8.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,091 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.9.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,093 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.10.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,094 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.11.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,096 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.12.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,097 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.13.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,099 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.14.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,101 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.15.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,102 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.16.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,104 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.17.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,105 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.18.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,107 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.19.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,108 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.20.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,110 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.21.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,111 - root - INFO - ModulatedDeformConvPack img_backbone.layer3.22.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,113 - root - INFO - ModulatedDeformConvPack img_backbone.layer4.0.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,116 - root - INFO - ModulatedDeformConvPack img_backbone.layer4.1.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,119 - root - INFO - ModulatedDeformConvPack img_backbone.layer4.2.conv2 is upgraded to version 2.
> 2024-04-05 00:11:56,123 - mmdet - WARNING - The model and loaded state dict do not match exactly
>
> 1size mismatch for pts_bbox_head.cls_branches.0.6.weight: copying a param with shape torch.Size([10, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
> 1size mismatch for pts_bbox_head.cls_branches.0.6.bias: copying a param with shape torch.Size([10]) from checkpoint, the shape in current model is torch.Size([6]).
> 1size mismatch for pts_bbox_head.cls_branches.1.6.weight: copying a param with shape torch.Size([10, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
> 1size mismatch for pts_bbox_head.cls_branches.1.6.bias: copying a param with shape torch.Size([10]) from checkpoint, the shape in current model is torch.Size([6]).
> 1size mismatch for pts_bbox_head.cls_branches.2.6.weight: copying a param with shape torch.Size([10, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
> 1size mismatch for pts_bbox_head.cls_branches.2.6.bias: copying a param with shape torch.Size([10]) from checkpoint, the shape in current model is torch.Size([6]).
> 1size mismatch for pts_bbox_head.cls_branches.3.6.weight: copying a param with shape torch.Size([10, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
> 1size mismatch for pts_bbox_head.cls_branches.3.6.bias: copying a param with shape torch.Size([10]) from checkpoint, the shape in current model is torch.Size([6]).
> 1size mismatch for pts_bbox_head.cls_branches.4.6.weight: copying a param with shape torch.Size([10, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
> 1size mismatch for pts_bbox_head.cls_branches.4.6.bias: copying a param with shape torch.Size([10]) from checkpoint, the shape in current model is torch.Size([6]).
> 1size mismatch for pts_bbox_head.cls_branches.5.6.weight: copying a param with shape torch.Size([10, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
> 1size mismatch for pts_bbox_head.cls_branches.5.6.bias: copying a param with shape torch.Size([10]) from checkpoint, the shape in current model is torch.Size([6]).
> 1size mismatch for pts_bbox_head.query_embedding.weight: copying a param with shape torch.Size([900, 512]) from checkpoint, the shape in current model is torch.Size([200, 512]).
> missing keys in source state_dict: pts_bbox_head.mask_fcs.0.weight, pts_bbox_head.mask_fcs.1.weight, pts_bbox_head.mask_fcs.1.bias, pts_bbox_head.fc_mask.weight, pts_bbox_head.fc_mask.bias, pts_bbox_head.depth_fcs.0.weight, pts_bbox_head.depth_fcs.1.weight, pts_bbox_head.depth_fcs.1.bias, pts_bbox_head.fc_depth.weight, pts_bbox_head.fc_depth.bias, pts_bbox_head.conv_depth.weight, pts_bbox_head.conv_depth.bias

...
File "/home/user/anaconda3/envs/shift3D2/lib/python3.8/site-packages/mmcv/ops/modulated_deform_conv.py", line 101, in forward
    ext_module.modulated_deform_conv_forward(
RuntimeError: CUDA error: CUBLAS_STATUS_INVALID_VALUE when calling `cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)`
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 121525) of binary

Hi, @leesunmyung.
This looks like some of the parameters in config are set incorrectly, please check that the num_classes in the config file are all set to 6.


Control-Flow Stages

In my view, stages will not always correspond to something like an environment that needs to be updated, an Argo CD app, etc.... Consider the following deployment DAG:
                    /---> [usw1]---\   /---> [usw2]
[test] -> [staging]-----> [use1]--->[X]----> [use2]
                    \---> [euw1]---/   \---> [euw2]

The [X] Stage is what I'll call (for lack of a better word) a control-flow stage, which is basically just a stage that I want to use as an interaction point in the UI/API.
My use case is the following:

The [X] stage is subscribed to three stages
Once a state makes it past (any or all) of usw1, use1, euw1, I'd like for a way to manually simultaneously promote to usw2, use2, euw2 (using the PromoteSubscribers feature)

The way I think this could work is: we could make it possible for the promotion mechanism of this to be simply passthrough of what it is subscribed to, then immediately make that state it available in the history to subscribers. As a future improvement, we could extend this with additional user defined rules, such as requiring that the state appeared in all upstream stages history.
But the main point is not how it works, but to call out the need for a Stage as less as a thing to affect an environment but as a control flow to the overall pipeline.

Notes from discussion earlier today:

Once a state makes it past (any or all) of usw1, use1, euw1

The current focus is on the "any" and not the "all."
I will address this by removing the constraint that every stage must have at least one promotion mechanism specified.


Entity id and save

What if I want to use the same ids used in my server ?
I create a Entity instance specifying my own id, then I save it, ORMDroid throws away my own id and replaces it by last_insert_rowid.
I'm stuck there.
I think I have to create a new field, like cloudId, and use this one when manipulating Entities... Is there any other work around ?
Thanks !

Internally, the presence of a primary key is used to determine whether the object is transient, so there'd have to be another mechanism for that. Also, how would this work with database constraints? I'm open to taking a look into it though...


Fix/refactor job executor

Noticed that the usage of DependencyInjectionJobExecutor is a little bit unclear (probably only for me). It allows to pass a null during construction, but raises an appropriate exception only in ExecuteAsync(). In that case I think it would make sense to either throw an ArgumentNullExceptionin the constructor or move IDependencyResolver to ExecuteAsync() as a parameter. Deciding to go with the former, I also refactored the method and its tests a little bit.

I agree, it's a good point. What if I move it from the contructor to ExecuteAsync()?
public Task<JobResult> ExecuteAsync(JobDescription jobDescription, IDependencyResolver resolver = null)
{
    // ...
}

Doing this we can provide a client with a more clean API and remove an unnecessary class variable.

Yes, that would make more sense. Let's do that! :)


Fix version if installing from git archive

Perhaps init.py (or a version.txt) file next to it could have a tag description embedded, like here:
https://git-scm.com/docs/gitattributes
That would provide versioning even for the case where a downloaded archive is pip install'ed.
However, this does not actually change the package meta data, so is not a complete solution.

The ideas in the previous comments are valid, but will probably not be needed for this project. Closing.


feat(web): Added functions to confirm user and get user attributes

Description of changes:
Added function to confirm sign up and get user attributes.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

To fix prettier issues, run this in the root of the workspace:
npx prettier -w .

The reason is that we are using this lib server side on svelte. We wanted to use cookies and it became a bit difficult to set and get those cookies.
So we wanted to pass the token as a param instead.

Got it, understood, and thanks for the explanation.
Cool to hear you use this in Svelte on server side! Let us know any feedback you have to make life easier for that (such as more docs maybe ...)
See prettier is still failing. Make sure you have the exact same version as the lib? Should be 3.1.1

THANKS!

Published to NPM in v0.13.0


DNM: net: Fix returning wrong error code

Local variable ret "hides" another ret defined above in the scope. In
this case local error code is lost.

After the patch some tests are failing like: tests/net/tcp/net.tcp.no_recv_queue
The test is  failing following way: in the code chunk:
https://github.com/zephyrproject-rtos/zephyr/blob/774a3cd756b484825931e7e503f28a9df231c327/subsys/net/ip/net_context.c#L2120-L2141
statement https://github.com/zephyrproject-rtos/zephyr/blob/774a3cd756b484825931e7e503f28a9df231c327/subsys/net/ip/net_context.c#L2132
return -EAGAIN

After the patch some tests are failing like: tests/net/tcp/net.tcp.no_recv_queue

A few TCP tests appear to be faulty, fixing the net_context_recv() return value only revealed the problem.
The issue with this test suite is that TCP server test cases (test_server_ipv4, test_server_with_options_ipv4, test_server_ipv6) call net_context_recv() on a listening context, which is not correct, as the data should be expected on a different context, allocated when new connection is accepted.  Instead, this should be called on a communication net_context that is provided in test_tcp_accept_cb().

@rlubos @tbursztyka  @jukkar @jhedberg  trying to understand this code chunk, what it does?
https://github.com/zephyrproject-rtos/zephyr/blob/774a3cd756b484825931e7e503f28a9df231c327/subsys/net/ip/net_context.c#L2128-L2134

@finikorg net_context_* API is asynchronous in it's nature. If CONFIG_NET_CONTEXT_SYNC_RECV is enabled however, the net_context_recv() function can behave in a synchronous-like way, i. e. if a timeout other than K_NO_WAIT is provided, the function will block on this semaphore until data is avaialble (or timeout occurs). The feature is however no excessively used as net_context_recv() is typically just used to install callback.


build(deps): bump github.com/hetznercloud/hcloud-go to v2.0.0

Version 2 introduced a breaking change in the id field of all resources. They were changed from int to int64 to make sure that all future numerical IDs are supported on all architectures.
You can learn more about this here

Hope that this fulfills all requirements, its my first contribution to Prometheus :)

Thanks!


App crashes in video play

Error message:

java.lang.NoClassDefFoundError: android.media.session.MediaSession

when I'm trying to run the app in android 4.4. I think it's because MediaSession is not supported in 4.4. Am I right? What is the best practice if I want to support MediaSession when I'm running in Android 5.0 and above and without MediaSession running in Android 4.2 and above.
Thanks.

As you suggested, MediaSession was added in API 21, and that's why you are receiving this crash. We have updated the app and will soon be releasing a new version. In this version, we made the app compatible for API level 17 and above. One of the changes related to your issue is we are now using MediaSessionCompat to make it backward compatible.

Great. Thanks.


Dataset corrupt from github. You'll spend and hour cleaning this up if you use this file. https://github.com/Azure/MachineLearningNotebooks/blob/master/how-to-use-azureml/automated-machine-learning/forecasting-bike-share/bike-no.csv

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/forums/34192--general-feedback.

@residentx10 Could you please let me know the documentation URL you are working with? Once I have this I can assign the appropriate individual to work on the issue.


feat: Add support for checkout sessions to Embed tokens

Re TA-4565
This adds the ability to generate a new Embed token which will have a checkout session ID as one of it's claims.
const { token, checkoutSession } = await client.getEmbedTokenWithCheckoutSession({
  amount: 1299,
  currency: 'USD',
  buyerExternalIdentifier: 'user-1234',
  // etc
})

This function has the same signature as getEmbedToken but automatically generates a checkout session and then adds it to the JWT claims. It also returns the checkout session.
This also adds a new updateEmbedToken function that can be used to renew the Embed token with a new expiration date while keeping all scopes and checkout sessions the same. What it does not do is allow for extending the checkout session.
const newToken = await client.updateEmbedToken(token, {
  amount: 999, // new amount
  currency: 'USD',
  buyerExternalIdentifier: 'user-1234',
  // etc
})

Todos

I still need to add more details to the README and the public docs once we decide on the final usage.
I think we can update the updateEmbedToken eventually to also update the checkout session, extending the checkout session expiry date, and maybe even automatically update any data ON the checkout session (instead of putting them in the JWT).

Questions
There's a few alternative ways to implement getEmbedTokenWithCheckoutSession

getEmbedToken() with an optional flag to generate the checkout session ID
getEmbedToken() but the checkout session is always generated

@douglaseggleton What do you think of ^

@douglaseggleton What do you think of ^

I think that works - I guess the alternative is a 2 step?
const checkoutSession = client.createCheckoutSession(expiry_time=) // keep hold of this in the merchant session.

const token = client.getEmbedToken(checkoutSessionId=checkoutSession.id)

I think that works - I guess the alternative is a 2 step?

Yeah that is another way of doing it. What I like of this is that it gives us an upgrade path to moving more into the checkout session. For example, right now getEmbedTokenWithCheckoutSession would store the cartItems on the JWT, but we could update it eventually to move it to the checkout session. All the merchant would have to do is update the SDK.

I also think we should consider if we should just make getEmbedToken do the checkout session regardless, but simply not return the session. This would instantly give us an upgrade path, but does introduce a new API call for all merchants to make when they call our API.
// create a session and a token and returns them both
const { session, token } = client.getEmbedTokenWithCheckoutSession(params)

// secretly calls getEmbedTokenWithCheckoutSession but only returns the token
const token = client.getEmbedToken(params)

I think that works - I guess the alternative is a 2 step?

Yeah, that is another way of doing it. What I like about the 1 step approach is that it gives us an upgrade path to moving more into the checkout session. For example, right now getEmbedTokenWithCheckoutSession would store the cartItems on the JWT, but we could update it eventually to move it to the checkout session. All the merchant would have to do is update the SDK.
I also think we should consider if we should just make getEmbedToken do the checkout session regardless, but simply not return the session. This would instantly give us an upgrade path but does introduce a new API call for all merchants to make when they create a token.
// create a session and a token and returns them both
const { session, token } = client.getEmbedTokenWithCheckoutSession(params)

// secretly calls getEmbedTokenWithCheckoutSession but only returns the token
const token = client.getEmbedToken(params)

We should be careful on the second approach const token = client.getEmbedToken(params)
If this is happening in the background if could default the purpose of checkout sessions as they will create a new session for each token.
This is the reason I prefer two-step, but it's subjective - your original approach still solves this 👍🏻

@douglaseggleton this is now ready for review. I did not remove the 2 helper functions (getEmbedTokenWithCheckoutSession and updateEmbedToken), but maybe we should remove them till we need them.

@douglaseggleton sure. How do we do a canary release?

We don't use auto/release tooling on this repo (might be worth it https://intuit.github.io/auto/)- so I would just need push to NPM manually as something like: 0.48.0.canary+44067192f8f885961de593f17725a229945f49d7
We can then download and install it and make sure its working as expected before general release.


k9 thinks there is no network - sync disabled no network

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
should sync when wifi connected
Actual behavior
it sits idle regardless if screen is on or off, reason sync disabled no network
Steps to reproduce

install k9 on android 8.,x oneplus6

Environment
K-9 Mail version: 5.600
Android version: 8.1.0
Account type (IMAP, POP3, WebDAV/Exchange):
IMAP

To add I have obviously verified there is internet connectivity, if I simply hit the refresh button in k9 it fetches email fine.

Sorry for that. Our code currently doesn't work well with the energy saving mechanisms in newer Android versions. We know we need to work on this.
Closing this issue because it doesn't provide more information than the countless other issues.


[DNM] [utils/subgraph/router] Implement support for multiple subgraph URIs; fallback subgraph

Fixes #385
Checklist

[ ] Test manually on test-ui using remote chains.
[ ] Run load tests.
[ ] Update documentation if needed.
[ ] Update CHANGELOG.md.

I was thinking to do this simpler where we assume the subgraph is synced and only change the subgraph URL if it comes back out of sync. This should work also though, but I'm a little nervous about the complexity. This sync step happens on each loop iteration right?

We could certainly do it that way, but the problem there is that we wouldn't really have all the information to select the most in-sync subgraph URL. So we could end up with a subgraph that's lagging 8-10 blocks behind but consistently staying within what we require, while another unused resource of ours is actually totally in-sync, unbeknownst to us. By checking sync status each cycle, we ensure we're using the most insync subgraph each cycle as well.
If there's tradeoffs there, lmk


update tiflash-build-common.groovy to fix ccache process

Fix https://ci.pingcap.net/blue/organizations/jenkins/tiflash-build-common/detail/tiflash-build-common/20767/pipeline/
Verified in https://ci.pingcap.net/blue/organizations/jenkins/tiflash-build-common/detail/tiflash-build-common/20776/pipeline

/assign @purelind


Initial attempt at chaining call expressions

This seems to work pretty well, however this forces simple expressions that have a CallExpression on the left-hand to break on a newline:
foo()
  .callMethod()

The question is how can we differentiate between that and places we want to chain?

Does this convert
const a = foo.callMethod();

to
const a = (
  foo
    .callMethod()
);

Does this convert...

I don't know what the previous one did, but I just pushed a new commit that is much better, and it does not.

I just updated my heuristic for this. I'm pretty happy with it, but I've been pulling my hair out for a while trying to figure it out. It's pretty hard because of the amount of variations and subjectivity involved.
You can see a big file with all sorts of variations and the result of my printer: https://gist.github.com/jlongster/e17d35d03e565c2858e609af872db66b
My heuristic boils down to this: if there are successive CallExpression's on MemberExpression's, print them as a chain if there 2 or more functions passed in as the first arg in any of the calls. The point is that you really want it printed as a chain if you are doing this style of code:
arr
  .map(x => x + 1)
  .filter(x => x > 10)
  .some(x => x % 2)

And that's a simple example, if you have bigger functions inline it must be printed like this. However, if you didn't inline the functions, or the functions take some other kind of argument, you want it printed like this:
arr.map(foo).filter(bar).some(baz);

The reason we require 2 or more is because things like this is a pretty common pattern:
Object.keys(foo).forEach(x => {
  return x
});

That still prints the same way with my printer. Once you pass 2 functions in, though, it would bump each lookup onto a newline.
Note that when it prints a chain, it always prints a newline before the .. That's why the first map in the first example above is on a newline, which some people might not like, but it's the most consistent and not really a big deal. This does hit a weird case though if the initial expression is a function call like this:
  fetch(url, {
    method: "post",
    foo: "bar",
    credentials: "include",
    headers: {"Accept": "contsdlfkjsdfkldjs", "Accept2": "contsdlfkjsdfkldjs"}
  })
    .then(resp => resp.json())
    .then(obj => callback(obj))
    .then(obj => callback(obj))
    .then(obj => callback(obj));

I think that's OK though, and is perfectly consistent will everything else. Some people might think it looks weird, but that's subjective and we need to settle somewhere.
The last thing my commit does is make sure so split a long list of lookups/calls together if it's too long. Meaning this:
foo(foo).forEach().filter().foo().bar().forEach().filter().foo().bar().forEach().filter().foo().bar();

will be split up as this:
foo(foo)
  .forEach()
  .filter()
  .foo()
  .bar()
  .forEach()
  .filter()
  .foo()
  .bar()
  .forEach()
  .filter()
  .foo()
  .bar();

I'm done thinking about this. :) Please run it on existing code and let me know how it looks. We can tweak it later, but this is at least a good starting point.


fix: add namespace to configmap = .Release.Namespace; bump chart version to 1.9.3; update/regen docs

fix: add namespace to configmap = .Release.Namespace; bump chart version to 1.9.3; update/regen docs
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Based on proposed changes in #176 from @IronMage but with DCO fixed and more complete pre-commit hooks applied.
Run standard RH pre-commit checks........................................Passed
Helm Docs................................................................Passed
jsonschema-dereference...................................................Passed

Ref: https://issues.redhat.com/browse/RHIDP-2170

Hey @ChrisJBurns can you explain why the lint test failed here? was it something in my regenerated doc?

@nickboldt  This is because by default auth is enabled in newer versions of Backstage now. I have raised https://github.com/backstage/charts/issues/186 to resolve this. This should then allow your PR to pass testing.


fix(live-photos): issue with non-existent file path when copying live photos with ID com.apple.private.live-photo-bundle

There was an issue causing file not to be copied over to the temp directory which we have seen in Sentry for a while in out application i.e. File 'file:///var/mobile/Containers/Data/Application/A7CB430B-F16F-47FC-8EEC-D63C95D867F9/tmp/F42EBD2C-F353-4289-A5D3-4879294BFA0B.jpg' does not exist.. This was because apple was returning the following ID for some Live Photos com.apple.private.live-photo-bundle (not sure why this was the case as I was unable to detect any clear pattern) and we were not handling photos with that ID.
On other note if NSData / Image is null after running loadFileRepresentationForTypeIdentifier we should really be throwing an error, but that can a follow up PR.
This PR should catch all IDs which contain live-photo-bundle.
Closes #1905 #1836 (maybe more 🤷♂)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Validate beginner tips in Ruby

We are starting an epic task to update Elemental Selenium content.
Tips should be validated, and its content reviewed to fit the new tips headers, consdering inclusive language.
Code snipets also need to be validated.

[x] Create a new folder under updated-content named with the new tip number and title using kebab-case. You should follow the folder example: 0-template
[x] Inside of the folder, a markdown file named as the programing languaged validated should be created. Example: python.md
[x] The file should follow the tips template
[ ] Delete the equivalent folder under non-updated-tips folder
[x] Add a review tag, such as: Last review Day Month Year
[x] In the case you cannot validate/ run the code snippet, add a tag, something like: Code needs validation

Tips Code Validation

[x] Tip 1
[x] Tip 3
[x] Tip 5
[x] Tip 13
[x] Tip 18
[x] Tip 20
[x] Tip 21
[x] Tip 22
[x] Tip 26
[x] Tip 31
[x] Tip 35
[x] Tip 36
[x] Tip 45
[x] Tip 50
[x] Tip 51
[x] Tip 57
[x] Tip 58
[x] Tip 61
[x] Tip 69

Tips Content Validation

[x] Tip 1
[x] Tip 3
[x] Tip 5
[x] Tip 13
[x] Tip 18
[x] Tip 20
[x] Tip 21
[x] Tip 22
[x] Tip 26
[x] Tip 31
[x] Tip 35
[x] Tip 36
[x] Tip 45
[x] Tip 50
[x] Tip 51
[x] Tip 57
[x] Tip 58
[x] Tip 61
[x] Tip 69

[x] Tip 26 - Created an issue to escalate to Diego
[x] Tip 31 - Created an issue to escalate to Diego
[x] Tip 35 - Created an issue to escalate to Diego
[x] Tip 69 - Created an issue to defer to MacOS user


The readme.md is misleading

The readme.md is misleading since there is no releases supporting some new feature, such as sass.logError for example.

v2 is going to get pushed anytime now, it just got merged a few hours ago
@dlmanning will push it when he can

published


commenting out default servicemonitor auth to avoid checkov CKV_SECRET_6

what

Commenting out the default password and username setting for service monitor authentication to avoid running into the checkov SECRET_6 compliance check.
why

Since its disabled by default any way there should be no harm in commenting out the actual setting of the password and prevents people from deploying with a default password if they do enable the auth check for the service monitor. Commented out and left for reference of users that want to set it.
tests

Running checkov against charts with it commented out does not make the error occur.
references

Hi @rwd5213, even thought there's no actual change, it's a good practice to bump the chart. Bumping the patch version would be enough.

@GMartinez-Sisti Fixed both, thanks


 But it refers to /assets/errors.css as a stylesheet. And we don't actually serve that. I'm not sure if that's intentional (and we need to upset the 404 page not to refer to it), or if something is broken in our asset pipeline. There is an assets/stylesheets/errors.scss in the repo, but that path does not work either.
AFAICT this isn't really hurting anything, but I'd like to clean it up and I'm not sure how. Anyone with rails expertise want to weigh in?

@peffI guess issue is already solved.
check out https://git-scm.com/doesnotexist , if you visit the page you will get 404.

I think the issue is that we are referring to a css file that does not exist:

Strange ! It's working on local server. Same with 500 error screen.

@peff I want to fix this issue but don't know how to provide relative path in an .html file ?

I think the issue is that in the production site, we end up with asset files tagged with their hash, like:
~ $ ls -l public/assets/errors-*
-rw------- 1 u24297 dyno 4317 Dec  1 17:13 public/assets/errors-66c6a1ddb6367adb3d9600e2110cbb75c00453b5385854fedfd31926752e39ca.css
-rw------- 1 u24297 dyno 1504 Dec  1 17:13 public/assets/errors-66c6a1ddb6367adb3d9600e2110cbb75c00453b5385854fedfd31926752e39ca.css.gz
-rw------- 1 u24297 dyno 4114 May 19  2021 public/assets/errors-8310492f9e1ebc1fb483ac671943cd79ab2a81362e348c3bc6b5e4a6d910ec1b.css
-rw------- 1 u24297 dyno 1429 May 19  2021 public/assets/errors-8310492f9e1ebc1fb483ac671943cd79ab2a81362e348c3bc6b5e4a6d910ec1b.css.gz
-rw------- 1 u24297 dyno 4145 Nov 11 16:36 public/assets/errors-8c22f5bde2af7bffa6b370f8e5eaf35ecb60051e4ce272a3e4bf4ebb620b66c1.css
-rw------- 1 u24297 dyno 1438 Nov 11 16:36 public/assets/errors-8c22f5bde2af7bffa6b370f8e5eaf35ecb60051e4ce272a3e4bf4ebb620b66c1.css.gz

whereas in a development environment, it will process the scss on the fly as necessary from the source files. (But I'm pretty ignorant of rails asset pipeline stuff in general).
In the error view page we use some rails magic to get the right name: https://github.com/git/git-scm.com/blob/fcf5b8bcb7fad9a7e277739473925b4b26de9ec5/app/views/layouts/errors.html.erb#L15-L17
But the static 404 page just mentions the name directly: https://github.com/git/git-scm.com/blob/fcf5b8bcb7fad9a7e277739473925b4b26de9ec5/public/404.html#L8
Which kind of make sense. We aren't processing those files as templates, so they can't use the rails magic. We could make them into views, but especially for a 500, that feels like a chicken-and-egg problem (depending on what is broken, we may not be able to even render the 500 page properly). I wonder if we should just push all of the content in errors.scss into the actual static error pages themselves.
I also wonder if app/views/layouts/errors.html.erb is ever used at all. I don't see it mentioned anywhere after some grepping around, but I could be missing some place where rails implicitly loads the view.

@peff I tested the bogus page and css didn't render as expected. It works on local space.
I was checking Ruby doc and reached to this section. As mentioned this is the preferred but insecure way for error pages.
Yes, we didn't utilise public/errors.html.erb which is secure way. Therefore, we've two ways to resolve this issue.

https://u307627.ct.sendgrid.net/ls/click?upn=Q6kxRXJSTSmHQwSK6LKq5b8Uzd40led0jTnwofEvom6ZJZbn5ar-2FZ0xR39bcd-2Bz-2FYcv1IJpto7Rnk8h6L5sIMA-3D-3DOlHP_ItBfwAv5wUibnWk0uGmHL-2BRktQWb21aJCAfJhiF8i3NJPs06cs3kSo1WHa1HmfJgwcv0hM3pROXMmzfji6PDdwWYuQu-2BDzL4Vf9DoPcZaf4XGw-2FMd01H-2FwYjHfC-2BxXdaic5VVgBp00YE-2FjTfnawz10THJZf0sF-2FG0Y8m4-2FfF1fILksQigJZ8q2Bac-2F5E-2FTXSm-2BYYZYOZ7RglVni79e0COA-3D-3D

@peff This no longer seems to be an issue with issue, can we close?

Yeah, I think as of 88a8657d290ce2d4f2c2cd5dc5db74591ef76d3e we are just relying on the regular hugo styling, and the sometimes-broken css link is now gone. Thanks for cleaning up these old issues!


Update README.md

Changes casing.

this commit is destructive you removed the entire sources and include.
I guess what you want to do is remove dependency folders in your bootsrc but went ahead and remove the bootsrc folder. This commit can never be merged until it is reviewed

you will have to re-add the bootsrc folder

That was a mistake. I'll resend the PR 🙏🙏
On Thu, Apr 5, 2018, 18:52 Azeez Adewale<EMAIL_ADDRESS>wrote:

you will have to re-add the bootsrc folder
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/simple-lang/simple/pull/6#issuecomment-379021583, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Adkrj7ZGF8bJwhl0EgnlXD_7M_GBTFccks5tllnNgaJpZM4TIqTX
.

For a terrible mistake.m I'll resend PR. I'm sorry once again


BODS-6817 consumer subscription schedule creation

Closed - superseded by https://github.com/Department-for-Transport-Disruptions/bods-integrated-data/pull/388


Help with ha_mqtt_binary_sensor_ble_scanner

First I would like to thank you for this sketch, just what I been looking for.

Can you help me understand how to use "MAX_NON_ADV_PERIOD"?
(Edit.: I think I understand it but anything over 5000 (5 secs) doesn't seem to work.)
How best to implement a feature to only publish to mqtt when (RSSI() > -##)?
"##" being any number enter by the user, like (65,75, or 80). This way I can kinda limit the area it can see
the ble device.
I have no Arduino sketching experience, so I wouldn't dare mess with a sketch that was written so beautifully.  So your help would be much appreciated. Thanks in advanced

because if more than 5s the program will be slowed to read and excetuted


How to define a dependency? - Proposal and a technique to trace control flow and data flow

Current project - it works well with dependency for a file defined.
Proposal - how to define code level dependencies? for example, consider below code
var a = Object.keys(b)
Here a is dependent on Object.keys(b), hence a.dependency becomes Object.keys(b)
This will help the project to scale across and within a code as well.
With that intention, how can I define individual dependency pattern?

Being able to form dependencies at the code level would definitely be very interesting, but it falls out of the dep-tree's vision scope.

I think the problem is simple, you are right about the computational performance.
Why I think it is simple because dependency in a file is just the assignment operator. Just one case. perhaps there are more than it?
p.s - please feel free to let me know if you are solving some other problem. I opened an issue not with regards to the project and its scope, opened issue to know your thoughts and ideas to solve interesting problem. The reason I'm looking for dependency graph for code level is because once we get a graph of codebase, then, code from one language and framework can be easily regenerated to another language and another framework.

Why I think it is simple because dependency in a file is just the assignment operator.

I would argue that there are more ways of making an entity in the codebase be dependant on another entity without assignment operators, for example:
function sayHello() {
  console.log('hello!')
}

function main() {
  console.log('start program')
  sayHello()
  console.log('end program')
}

In this code sample, function main depends on function sayHello in order to work properly, but there's no assignment operator. Catching this kind of relationships between code entities would require a full parsing on the languages and an analysis on their ASTs, so it might be a big challenge.
PD: I'm going to close this as it's more a discussion rather than an issue, but feel free to write your thoughts and continue the discussion here even if closed.

Catching this kind of relationships between code entities would require a full parsing on the languages and an analysis on their ASTs,

I disagree, because knowing about AST and language is making it look like a challenging problem. Ignorance is bliss.
function sayHello() {
  console.log('hello!')
}

function main() {
  console.log('start program')
  sayHello()
  console.log('end program')
}

This code can be rewritten as

function main() {
  console.log('start program')
  (function sayHello() {
    console.log('hello!')
  })()
  console.log('end program')
}

This code can be rewritten as

function main() {
  console.log('start program')
  console.log('hello!')
  console.log('end program')
}

In other words, flattening a function calls down to its implementation in the place of definition. This will help us to see the code from jumpy thoughts to linear thoughts. It may cause code duplication, and it will make us question that, and not chase it further, ignore the thought, lets duplicate and make it look a straight line for distraction free format.
I mean, this brings a pile of dependencies of files dumped. Even if it sounds too large, its not us who will be working, let the machine run.
Like you brought up, that other than assignment operator, function call are another case, and the case may keep going on, but it will not be infinite number of cases.


Gridcoin payment add-on for Wordpress

 Issue by Erkan-Yilmaz
Friday Apr 28, 2017 at 15:56 GMT
Originally opened as https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/118

Who wants to help ?

gciapita: is anyone working on a gridcoin payment add-on for wordpress?

Erkan_Yilmaz: it exists already, !coinpayments https://wordpress.org/plugins/coinpayments-payment-gateway-for-woocommerce/

fediverse: You can buy products with Gridcoin in these shops: https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/28

gciapita: it doesnt erkan

gciapita: coinpayments is full of dodgy businesses and i want no affiliation with drug dealers and pornographers, etc.

Erkan_Yilmaz: well then you have to code one, I guess

gciapita: i know

BullShark: Most shop owners would want it to convert to FIAT on the back-end

 Comment by denravonska
Friday Apr 28, 2017 at 16:50 GMT

Doesn't the Coinpayments plugin cover this? https://www.coinpayments.net/merchant-tools-plugins

 Comment by MagixInTheAir
Monday Aug 28, 2017 at 10:06 GMT

Any update ? Is the developpement still going ?

 Comment by grctest
Sunday May 14, 2017 at 22:28 GMT

Yes, coinpayments does cover this but gciapita doesn't want to associate himself with coinpayments because allegedly dodgey websites use it.. this is the case for all businesses, not just coinpayments, so i think it should be fine just using coinpayments..

There was no development in the first place, just a suggestion to create a homebrew payment plugin - the cointopay and coinpayments plugins are sufficient & its use does not directly associate ones self with other websites which use the payment processor.


question on model zoo

Hi, Thanks a lot for your code and for running the models.
For the model Trained with vilbert-12-in-1 feature which achieves 1.158, may you please tell me the self-critical results if you ran them?

The cider score improves from 1.277 to 1.3, which is quite good. May I know is this trained with new self-critical or the normal self-critical?

It is the new self critical and also train with 40 epochs.

it's the transformer_nscl

oh i see...0.002 cider point improvement. Not significant. Thanks a lot for your help
Regards


Release v0.1.9

Checklist

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

@conda-forge-admin, please rerender

Ping @tkoyama010
Let's get the conda channel uptodate with the latest release of pytest-pyvista 🥳


nimble/mesh: We stop sending beacons after provisioning

This issue is not verified and needs further investigation.

This may be related to https://github.com/apache/mynewt-nimble/issues/118


Event Model Reorganization & Related Issue Resolution

There are two main components to this project:

EVENT MODEL REORGANIZATION
Reorganize event model to show the following page links under the event tab:
a. MARE Hosted Events (combine previously labeled "MARE Adoption Parties & Information Events" and "Fundraising Events)
b. Partner Hosted Events (combine previously labeled "Agency Information Meetings" and "Other Opportunities and Training")
C. MAPP Training (leave as is)

EVENT MODEL RELATED ISSUE RESOLUTION
Complete the following open tasks related to Events:
a. Our current capability with registration is either on/off for all user types. Update registration and modify event template to allow for turning off registration by user type (i.e. keep event posted but disallow social worker registration while still allowing family registration or vice-versa). Continue to allow event visibility even if one or both of the registration pools is turned-off.
b. FOR REGISTERED FAMILIES: When a family has an existing record and also registers for an event, prompt family to update their response to "Stages" (i.e stage in process check boxes in family record) and capture the change in the family record.
c. On the event template address information is collected. That information is used to present the address on the event summary page. We also have that info. in the description for the full event listing. When that information is presented on the event specific page it is missing the town info. and creates confusion. Resolution: Please suppress the "Details to Remember" field of the full event page (leave address info on the event preview page).
d. Modify event template to allow for choice of a specific staff persons (pick from list of staff with automated emails being generated to selected individual) for each event to automatically receive emails (remove current contact person email field from template) There should be two fields for selections: One automated email selection for families and one for social workers
e. Automatically deactivate (not delete) event on the day after the event occurs (use event date field as trigger).
f. Create field called "Event Title" ( use as event headline for website and public consumption) and keep Event Name (for internal consumption). Maintain both fields.
g. Add a field to Event Template called "Schedule Description" (non-mandatory) which allows us to add a free-flow text describing and event schedule (i.e. Meets Every Other Thursday) and suppress mandatory date selection (add checkbox for dates and if selected then make date range mandatory)
h. For Partner Hosted Event Template and MAPP Training Template add field and drop down menu to allow for selection of DCF geographic region (Boston, Northern, Southern, Central, Western) and then on the front-end organize by region for user ease of connecting to events in their own area. Basically create page headers on the front-end (event summary pages for Partner Hosted Events and MAPP Events) that sort events based upon regional selection and present by region.

RELATED:
-Story #133 in process
-Reporting project needs to include a list export of attendees/by attendees

@autoboxer Looks great in development, but has not been pushed to production.


Exported NodePath reverts to string "Null" in undefined niche case.

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0
Issue description:
Somehow, attempting to reset an edited export variable for a NodePath was setting it to the string value "Null" rather than ACTUALLY setting it to null or clearing it properly. I know this because evaluating if node_path was returning true and evaluating if node_path.is_empty() was returning false. Printing it would print "Null" to the screen, and doing get_node(node_path) would result in a "Condition '!node' is true" error. (Edit: I just tested using typeof() on it, and it returns 15 even though it's printing "Null" and getting through the conditions described above). If I specifically "Clear" the field, then the if node_path if statement properly blocks it.
This same script was attached to 4 different nodes (two nodes per instanced scene, the same scene instanced twice in a larger scene) and only 1 of them was malfunctioning in this manner (the last one in the node hierarchy - don't think that's relevant though).
Steps to reproduce:
I have not been able to reproduce this bug in smaller example scenes, so I'm not yet entirely sure what causes it. I attempted to put two child nodes with a script with an exported NodePath underneath a parent, and then instance that scene twice in another scene, altering the values for all four nodes' scripts from the default null value. None of them exhibited the abnormal behavior.
Link to minimal example project:
Here is a link to my GitHub project at the relevant commit where the bug was happening. The scene file I was testing was test.tscn. Node at /root/world/SkillUser2/skills/print_user_skill/targeters/npt2 was the erroring one.
Here is a GIF of the bug in action. The "Null" content showing up here is the state it was in when I was still getting through if statements and such. You'll notice that clicking the "revert" button doesn't actually make the "revert" icon go away like it normally would, even though the default value IS null.

@bojidar-bg

Not sure how you got to this state, but the problem is in demo/scenes/SkillUsers/test_skill_user.tscn, which has Null written out as a string (and not the value null).
Can't reproduce from clean state either, and I can't say it is a bug anymore.

@bojidar-bg Wait what? Did I seriously do that and miss it during debugging? Ugh...well then...fancy that. Sorry to have wasted your time on this. Yeah, I'll just close this.


Config precedence not implemented correctly

Is this a new bug?

[X] I believe this is a new bug
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Hi,
I have a model and in the model's .sql-file I have set {{ config(materialized='incremental') }}.
In the dbt_project.yml I set a materialization under the models key for the whole project like this:
models:
  my_fabric_project:
      +materialized: view

What I observe:
The value of model.config.materialized inside the is_incremental-macro is the one set inside the dbt_project.yml instead of the config block. The behaviour is same for views and tables as values for +materialized:.
is_incremental then returns false, therefore dbt is always running in full-refresh-mode.
Removal of setting the materialization in the dbt_project.yml fixed it and the model is correctly run in incremental mode because the value of model.config.materialized is incremental.
Expected Behavior
What I expect:
The materialization from the config-block supersedes the one from the dbt_project.yml.
Steps To Reproduce
in the model's .sql-file I have set {{ config(materialized='incremental') }}.
In the dbt_project.yml I set a materialization under the models key for the whole project like this:
models:
  my_fabric_project:
      +materialized: view

Removal of setting the materialization in the dbt_project.yml fixed it and the model is correctly run in incremental mode because the value of model.config.materialized is incremental.
Relevant log output
No response
Environment
- OS: Linux/Windows
- Python: 3.11
- dbt-adapters: dbt-fabric adapter

Additional Context
No response

Thanks for raising this issue @tkiehn (and thanks for sending it over here for further troubleshooting @prdpsvs)!
I was not able to reproduce the same issue that was reported.
I used dbt-duckdb rather than dbt-fabric, but I would expect them both to behave the same way for the issue that you reported. See below for the project files I used.
@tkiehn Could you double check that when you run the same command twice in a row that it doesn't run in incremental mode the 2nd time around?
dbt run -s your_incremental_model
dbt run -s your_incremental_model

If indeed it doesn't, could you do the following?

Try running the simplified project code below and see if it behaves the way you expect or not
Try re-running your code using the overridden definition of macros/is_incremental.sql that is provided in the project files below? It contains some debugging code that will allow you to better inspect what is going on.

Example files

dbt_project.yml
name: "my_project"
version: "1.0.0"
config-version: 2
profile: "duckdb"

models:
  my_project:
    +materialized: view

models/my_incremental.sql
{{ config(materialized="incremental") }}

{% do log("is_incremental(): " ~ is_incremental(), True) %}
{% do log("my_incremental.sql model.config.materialized: " ~ model.config.materialized, True) %}

select 1 as id

I overwrote this macro just so that we could inspect what is going on -- everything looks fine to me!
macros/is_incremental.sql
{% macro is_incremental() %}
    {#-- do not run introspective queries in parsing #}
    {% if not execute %}
        {{ return(False) }}
    {% else %}
        {% do log("is_incremental() model.config.materialized: " ~ model.config.materialized, True) %}
        {% set relation = adapter.get_relation(this.database, this.schema, this.table) %}
        {{ return(relation is not none
                  and relation.type == 'table'
                  and model.config.materialized == 'incremental'
                  and not should_full_refresh()) }}
    {% endif %}
{% endmacro %}

Output:
You should see that it does a full refresh the 1st time, and then uses an incremental update the 2nd time.
(dbt_1.8) $ dbt run -s my_incremental
22:32:37  Running with dbt=1.8.6
22:32:38  Registered adapter: duckdb=1.8.3
22:32:38  Found 5 models, 411 macros
22:32:38
22:32:38  Concurrency: 1 threads (target='dev')
22:32:38
22:32:38  1 of 1 START sql incremental model feature_456.my_incremental .................. [RUN]
22:32:38  is_incremental() model.config.materialized: incremental
22:32:38  is_incremental(): False
22:32:38  my_incremental.sql model.config.materialized: incremental
22:32:38  1 of 1 OK created sql incremental model feature_456.my_incremental ............. [OK in 0.11s]
22:32:38
22:32:38  Finished running 1 incremental model in 0 hours 0 minutes and 0.31 seconds (0.31s).
22:32:38
22:32:38  Completed successfully
22:32:38
22:32:38  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

(dbt_1.8) $ dbt run -s my_incremental
22:32:41  Running with dbt=1.8.6
22:32:42  Registered adapter: duckdb=1.8.3
22:32:42  Found 5 models, 411 macros
22:32:42
22:32:42  Concurrency: 1 threads (target='dev')
22:32:42
22:32:42  1 of 1 START sql incremental model feature_456.my_incremental .................. [RUN]
22:32:42  is_incremental() model.config.materialized: incremental
22:32:42  is_incremental(): True
22:32:42  my_incremental.sql model.config.materialized: incremental
22:32:42  1 of 1 OK created sql incremental model feature_456.my_incremental ............. [OK in 0.20s]
22:32:42
22:32:42  Finished running 1 incremental model in 0 hours 0 minutes and 0.37 seconds (0.37s).
22:32:42
22:32:42  Completed successfully
22:32:42
22:32:42  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

Hi @dbeatty10 and thanks for the detailed response!
After getting back to the issue I found that the problem comes up when using models from a package.
So my root-project has a package installed in which are several models which are configured using a config block.
For example my_incremental_model(inside the package):
{{ config(materialized = 'incremental') }}
{{ log(is_incremental(), info=True) }}
select 1 as id

dbt_project.yml of my root-project:
name: project
config-version: 2

profile: 'profile'
models:
  my_installed_package:
      +materialized: view
  my_root_project:
      +materialized: table

The dbt_project.yml of the package does not contain any model configs except tagging.
My overridden is_incremental-macro:

{% macro is_incremental() %}
    {#-- do not run introspective queries in parsing #}
    {% if not execute %}
        {{ return(False) }}
    {% else %}
        {% set relation = adapter.get_relation(this.database, this.schema, this.table) %}
        {{ log(model.name ~ ' model.config.materialized: ' ~ model.config.materialized, info=True) }}
        {{ return(relation is not none
                  and relation.type == 'table'
                  and model.config.materialized == 'incremental'
                  and not should_full_refresh()) }}
    {% endif %}
{% endmacro %}

Terminal output showing it is configured as a view:

(.venv) PS C:\Projects\project> dbt run -s my_incremental_model
08:38:30  Running with dbt=1.8.3
08:38:30  Registered adapter: fabric=1.8.7
08:38:30  Unable to do partial parsing because a project config has changed
08:38:34  False
08:38:36  Found 61 models, 205 data tests, 11 seeds, 9 sources, 1107 macros
08:38:36
08:38:38  Concurrency: 8 threads (target='dev')
08:38:38
08:38:38  1 of 1 START sql view model dbt-tkiehn-1.my_incremental_model .................. [RUN]
08:38:38  my_incremental_model model.config.materialized: view
08:38:38  False
08:38:40  1 of 1 OK created sql view model dbt-tkiehn-1.my_incremental_model ............. [OK in 1.54s]
08:38:40
08:38:40  Finished running 1 view model in 0 hours 0 minutes and 3.99 seconds (3.99s).
08:38:40
08:38:40  Completed successfully
08:38:40
08:38:40  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

When i change the model-config inside the dbt_project.yml from view to table the model.config.materialized value changes accordingly.
I would have expected the model's config block to override the config from the dbt_project.yml.
Is there a misconfiguration on my side?

After checking the docs and finding the sentence Model configurations in your root dbt project have higher precedence than configurations in installed packages. This enables you to override the configurations of installed packages, providing more control over your dbt runs. I think this can be closed.

Yep, that is precisely the relevant documentation that explains it! Closing accordingly.
Side note: this specific documentation was also relevant to https://github.com/dbt-labs/dbt-core/issues/10707.


Query parameters with '//' in them get collapsed to '/'

Google OAuth redirects to web apps with the scope included on the query string.
Example URL:
http://www.test.com/oauth-redirect?code=xxx&scope=https://www.googleapis.com/auth/gmail.modify
However, this line corrupts the query string, collapsing https:// to https:/:
https://github.com/veliovgroup/flow-router/blob/6961ad112968c8ed2c6a00e22ef9ce6581e1d4ac/client/router.js#L478

@drone1 I'm sure there was a reason for this .replace() call. Do you want to send a PR providing an option to disable this behavior?

probably to remove erroneous double-slashes from the url. which may still
be needed. but perhaps not for the query string.
On Sun 29. Jan 2023 at 19:50, dr.dimitru @.***> wrote:

@drone1 https://github.com/drone1 I'm sure there was a reason for this
.replace() call. Do you want to send a PR providing an option to disable
this behavior?
—
Reply to this email directly, view it on GitHub
https://github.com/veliovgroup/flow-router/issues/102#issuecomment-1407741311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQIEPDTHHMQQ2ID6VVGG33WU23WJANCNFSM6AAAAAAUGTRVJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@drone1 then we would need to exclude https?:\/\/ from this replacement

Sadly I don't have time to submit a PR, but hopefully this code can get someone else started:
const pathParts = path.split('?')
let fixedPath = pathParts[0].replace(/\/\/+/g, '/')
if (pathParts.length > 1) {
   fixedPath += `?${pathParts[1]}`
}
original.call(self, fixedPath, state, dispatch, push);


Exit with code 1 on errors instead of 0

Fixes issue #38

Hey, thanks for your contribution, we are trying to address this in a different way...
Please refer to this pull request and feel free to comment there.
Maybe we can close this if we agree to the approach.

To be honest, if you're going to write out the reason you exited with 1 from your program you don't really need to return anything and check for it later only to quit out anyways, but let's be real you're only wasting a couple of CPU cycles here and there on returning and checking if the return is good or not (also, you should probably fprintf to stderr when you encounter an error, otherwise those printfs are treated as stdout and if someone were to redirect only legit output without errors he'd get errors anyways because printf only prints to stdout and fprintf can print to any file stream you specify it to, whether it's a file or a file desctiptor like stderr)

Hey, @BodilessSleeper thanks for your inputs, we (me and @lucascebertin) are both very noobs with pure C, we wanted to test all the behaviors of the returns with an error, so if we want to test the behavior of exit(1) is that possible?
Thanks for your comment I didn´t knew about the differences in the printf´s.

I'm not an expert on what you guys are doing either, so I'm not really sure how you should really tackle the problem and what should you really check for so (I also got no idea how unit tests on github work, I can debug my own code, and test it too, but unit tests are just a bit of a strange concept to me)
I am gonna tell you what I do know, namely, exit(some_code) is gonna terminate the whole process immediately with the code you specify, but it is returned to the system, not a function, while return(some_code) will terminate the function and return the specified code to the place where the function was called and you can check what that returned
Basically if you can you can write tests to check system return codes and stderr output you're better off with fprintf(stderr, "error_text"); exit(1); and test for exit code 1 (meaning you errored out), and the error text for a specific function since getting 1 can mean lots of things (eg. test for "cannot open file", when testing for check_if_cant_read_file), otherwise if you can only check function return codes you should just fprintf(stderr, "error_text"); return 1; and check only a specific function for return value (no need to check the error text when you only check a specific function)

So to understand a better way to go we need to understand if our test suite can check for error codes returned by an exit(x).
I say that because the test suite is building binaries that runs and asserts the code.
But let's figure it out..

Yeah, testing functions with exit(something) is very hard, you need to change the code to be able to mock the exit function
https://locklessinc.com/articles/mocking/
O use something like the death checks like the google test suite has here, but our suite (Unity) doesn´t have something like this as I searched in the docs.
But I changed the printf as you suggested to fprintf to stderr and added you as the co-author @BodilessSleeper
Thanks a lot for your time and for sharing your knowledge.

Yeah, I've been messing around for a bit and I realised what was going on (at least on the surface level, that the system builds the binary first and then the test binaries which are based on your .h files, meaning they can all have the same functionality, so checking for exit codes is a pain)
It isn't impossible to check for exit codes tho, as I'm pretty sure you could run your binary through the test binary (using things like system(), execve()), to run the whole binary that we just built, and check for return code of said functions (which will really just be your binary's exit code, but I'm not sure if stderr is readable that way, probably is though)
But again, your way of doing it is just way easier for unit testing and it wastes very small amount of CPU cycles


funding_locked sent before channel_reestablish

This may be an old bug, I saw it while testing against the testnet.yalls.org node:
INFO  [lightning::ln::peer_handler : /home/matt/.cargo/git/checkouts/rust-lightning-61cf011b3b289b3b/7d6b98f/lightning/src/ln/peer_handler.rs, 629] Received peer Init message: data_loss_protect: supported, initial_routing_sync: not requested, upfront_shutdown_script: supported, unkown local flags: present, unknown global flags: present
...
TRACE [lightning::ln::peer_handler : /home/matt/.cargo/git/checkouts/rust-lightning-61cf011b3b289b3b/7d6b98f/lightning/src/ln/peer_handler.rs, 605] Received message of type 36 from 027455aef8453d92f4706b560b61527cc217ddf14da41770e8ed6607190a1851b8
TRACE [lightning::ln::channelmanager : /home/matt/.cargo/git/checkouts/rust-lightning-61cf011b3b289b3b/7d6b98f/lightning/src/ln/channelmanager.rs, 2225] Closing channel 51eb6695576cac1e3fb3a829b6b93e86f61d121dc59a5952da048138df33a34a due to Close-required error: Peer sent funding_locked when we needed a channel_reestablish
...
TRACE [lightning::ln::peer_handler : /home/matt/.cargo/git/checkouts/rust-lightning-61cf011b3b289b3b/7d6b98f/lightning/src/ln/peer_handler.rs, 605] Received message of type 136 from 027455aef8453d92f4706b560b61527cc217ddf14da41770e8ed6607190a1851b8

While the spec doesn't call it out explicitly, its somewhat nonsensical to send the funding_locked before the channel_reestablish:
if next_commitment_number is 1 in both the channel_reestablish it sent and received:
    MUST retransmit funding_locked.
otherwise:
    MUST NOT retransmit funding_locked.

as the first text there refers to the channel_reestablish in the past tense, and, really, why would you send a message about a channel before reestablishing the channel.

Related: https://github.com/lightningnetwork/lnd/issues/2555

just had this issue. any plans to fix it?

It seems lnd sometimes simply never sends a channel_reestablish and we could guess it based on the fact that we aren't locked yet,

So you mean upon reconnection you never see a chan reest message sent? It's the very first thing that happens once a "link" is created, if this isn't the "initial" funding event: https://github.com/lightningnetwork/lnd/blob/master/htlcswitch/link.go#L673

just had this issue. any plans to fix it?

How's the issue manifesting in your specific context?

Upon reconnection, more often than not, the first thing lnd does is send a funding_locked prior to the channel_reestablish message (which is required to come first, as lnd is required to check the values in it before deciding whether or not to send a funding_locked at all). C-lightning has special code to work around this case (by storing the funding_locked in memory until a channel_reestablish appears), but we do not currently have any such workaround, so we just force-close and kill the connection.

To complete the reference list, the c-lightning workaround was merged at https://github.com/ElementsProject/lightning/pull/2373

I think lnd doesn't send channel reestablish if the channel is pending (hasn't hit min confs yet)


Does the documentation needed an update ?

Heading says  "Use network upload to import your organization's PST files to Microsoft 365"
https://docs.microsoft.com/en-us/microsoft-365/compliance/importing-pst-files-to-office-365?view=o365-worldwide
However, the content still displays Office 365. Just checking to see if the documents need to be updated or if it is acceptable to use Office 365 instead of Microsoft 365.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 29f9f87e-6f96-9663-114a-f6bd74a7a7f4
Version Independent ID: b3ba56e3-0c78-f286-1c01-12375bf00f62
Content: Overview of importing your organization's PST files - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/importing-pst-files-to-office-365.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@yogkumgit Created PR #7325 for this issue #7323

@aravindmunna Thanks for the feedback. We will update the documentation.

Thank you


Execution failed for task ':ReactAndroid:compileReleaseJavaWithJavac' while ./gradlew :Examples:UIExplorer:android:app:installDebug

while doing ./gradlew :Examples:UIExplorer:android:app:installDebug
:ReactAndroid:compileReleaseJavaWithJavac
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:10: error: NotAnimatedItemAnimator is not abstract and does not override abstract method animateChange(ViewHolder,ViewHolder,ItemHolderInfo,ItemHolderInfo) in ItemAnimator
/*package*/ class NotAnimatedItemAnimator extends RecyclerView.ItemAnimator {
            ^
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:19: error: cannot find symbol
    dispatchRemoveStarting(holder);
    ^
  symbol:   method dispatchRemoveStarting(ViewHolder)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:20: error: cannot find symbol
    dispatchRemoveFinished(holder);
    ^
  symbol:   method dispatchRemoveFinished(ViewHolder)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:17: error: method does not override or implement a method from a supertype
  @Override
  ^
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:26: error: cannot find symbol
    dispatchAddStarting(holder);
    ^
  symbol:   method dispatchAddStarting(ViewHolder)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:27: error: cannot find symbol
    dispatchAddFinished(holder);
    ^
  symbol:   method dispatchAddFinished(ViewHolder)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:24: error: method does not override or implement a method from a supertype
  @Override
  ^
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:38: error: cannot find symbol
    dispatchMoveStarting(holder);
    ^
  symbol:   method dispatchMoveStarting(ViewHolder)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:39: error: cannot find symbol
    dispatchMoveFinished(holder);
    ^
  symbol:   method dispatchMoveFinished(ViewHolder)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:31: error: method does not override or implement a method from a supertype
  @Override
  ^
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:51: error: cannot find symbol
    dispatchChangeStarting(oldHolder, true);
    ^
  symbol:   method dispatchChangeStarting(ViewHolder,boolean)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:52: error: cannot find symbol
    dispatchChangeFinished(oldHolder, true);
    ^
  symbol:   method dispatchChangeFinished(ViewHolder,boolean)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:53: error: cannot find symbol
    dispatchChangeStarting(newHolder, false);
    ^
  symbol:   method dispatchChangeStarting(ViewHolder,boolean)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:54: error: cannot find symbol
    dispatchChangeFinished(newHolder, false);
    ^
  symbol:   method dispatchChangeFinished(ViewHolder,boolean)
  location: class NotAnimatedItemAnimator
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java:43: error: method does not override or implement a method from a supertype
  @Override
  ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
15 errors
:ReactAndroid:compileReleaseJavaWithJavac FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':ReactAndroid:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 1 mins 21.491 secs

I am using jdk1.7.0_79.jdk

Try to remove ~/.gradle/2.X.X/taskArtifacts and ~/.gradle/caches/, restart Android Studio and the click on Sync project with Gradle files button.

@alexHlebnikov still not working.
Information:Gradle tasks [:Examples:UIExplorer:android:app:clean, :Examples:UIExplorer:android:app:generateDebugSources, :Examples:UIExplorer:android:app:mockableAndroidJar, :Examples:UIExplorer:android:app:prepareDebugUnitTestDependencies, :Examples:UIExplorer:android:app:generateDebugAndroidTestSources, :Examples:UIExplorer:android:app:assembleDebug]
/Users/preetam/Documents/react_samples/react-native/ReactAndroid/src/main/java/com/facebook/react/views/recyclerview/NotAnimatedItemAnimator.java
Error:(10, 13) error: NotAnimatedItemAnimator is not abstract and does not override abstract method animateChange(ViewHolder,ViewHolder,ItemHolderInfo,ItemHolderInfo) in ItemAnimator
Error:(17, 3) error: method does not override or implement a method from a supertype
Error:(19, 5) error: cannot find symbol method dispatchRemoveStarting(ViewHolder)
Error:(20, 5) error: cannot find symbol method dispatchRemoveFinished(ViewHolder)
Error:(24, 3) error: method does not override or implement a method from a supertype
Error:(26, 5) error: cannot find symbol method dispatchAddStarting(ViewHolder)
Error:(27, 5) error: cannot find symbol method dispatchAddFinished(ViewHolder)
Error:(31, 3) error: method does not override or implement a method from a supertype
Error:(38, 5) error: cannot find symbol method dispatchMoveStarting(ViewHolder)
Error:(39, 5) error: cannot find symbol method dispatchMoveFinished(ViewHolder)
Error:(43, 3) error: method does not override or implement a method from a supertype
Error:(51, 5) error: cannot find symbol method dispatchChangeStarting(ViewHolder,boolean)
Error:(52, 5) error: cannot find symbol method dispatchChangeFinished(ViewHolder,boolean)
Error:(53, 5) error: cannot find symbol method dispatchChangeStarting(ViewHolder,boolean)
Error:(54, 5) error: cannot find symbol method dispatchChangeFinished(ViewHolder,boolean)
Error:Execution failed for task ':ReactAndroid:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.
Information:BUILD FAILED
Information:Total time: 4 mins 23.167 secs
Information:16 errors
Information:0 warnings
Information:See complete output in console

Do you using buildToolsVersion "24.0.2" ?

I get this error.
But it's fix by changing buildToolsVersion using 24.0.2 and
set compile 'com.android.support:recyclerview-v7:23.0.1' in ReactAndroid/build.gradle
Hope it helps

@facebook-github-bot label Icebox

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/execution-failed-for-task-reactandroidcompilereleasejavawithjavac-while-gradlew-examplesuiexplorerandroidappinstalldebug
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.

@facebook-github-bot close


Not changing out of loading state when using multiple dynamic asset collections

I'm using multiple dynamic asset collections to split loading up over plugins.  This works with multiple non-dynamic collections, and with a single dynamic collection, but with multiple dynamic collections it gets stuck on the loading state.  Is this a bug or am I missing some prerequisite for dynamic collections?
Example code:
main.rs
use bevy::prelude::*;
use bevy_asset_loader::prelude::*;
use iyes_loopless::prelude::*;

struct HatGuyPlugin;

impl Plugin for HatGuyPlugin {
    fn build(&self, app: &mut App) {
        app.add_loading_state(
            LoadingState::new(GameState::Load)
                .continue_to_state(GameState::Play)
                .on_failure_continue_to_state(GameState::Fail)
                .with_dynamic_collections::<StandardDynamicAssetCollection>(vec!["hat-guy.assets"])
                .with_collection::<OverlayAssets>(),
        );
    }
}

#[derive(AssetCollection, Default, Debug, Resource)]
pub struct OverlayAssets {
    #[asset(key = "hat-guy")]
    pub hat_guy: Handle<Image>,
}

struct SenseiPlugin;

impl Plugin for SenseiPlugin {
    fn build(&self, app: &mut App) {
        app.add_loading_state(
            LoadingState::new(GameState::Load)
                .continue_to_state(GameState::Play)
                .on_failure_continue_to_state(GameState::Fail)
                .with_dynamic_collections::<StandardDynamicAssetCollection>(vec!["sensei.assets"])
                .with_collection::<SenseiAssets>(),
        );
    }
}

#[derive(AssetCollection, Default, Debug, Resource)]
pub struct SenseiAssets {
    #[asset(key = "sensei")]
    pub sensei: Handle<Image>,
}

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_loopless_state(GameState::Load)
        .add_plugin(HatGuyPlugin)
        .add_plugin(SenseiPlugin)
        .add_system(debug)
        .run();
}

#[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
enum GameState {
    Load,
    Play,
    Fail,
}

fn debug(state: Res<CurrentState<GameState>>) {
    if state.is_changed() {
        println!("State changed to {:?}.", state.0);
    }
}

sensei.assets
({
    "sensei": File (
    path: "sensei.png",
    ),
})

hat-guy.assets
({
    "hat_guy": File (
    path: "hat-guy.png",
    ),
})

It's something between a bug and a missing feature. What you can change for it to work now:

only have one with_dynamic_collections call listing both files
use the same key for hat guy (hat_guy in the file vs hat-guy in the code)

You can keep the issue open until I added support for multiple calls to with_dynamic_collections.

If you use the main branch of bevy_asset_loader, point 1 from my comment above should no longer be necessary.


Touchscreen detection broken in Chrome 70 (desktop)

Chrome 70 has disabled the ontouch* APIs on desktop, see https://www.chromestatus.com/feature/4764225348042752
Leaflet currently uses the 'ontouchstart' detection method:
https://github.com/Leaflet/Leaflet/blob/b6027650ed187902f2542265a02ee680548b6252/src/core/Browser.js#L102
This check should probably be augmented/replaced with the suggestions in the first link ("site should use navigator.maxTouchPoints for touchscreen detection, and Windows.TouchEvent for TouchEvent feature detection").
Further reading:
https://bugs.chromium.org/p/chromium/issues/detail?id=897906

Sorry disregard this, 'pointer' will be true in Chrome 70 in current leaflet (I was forwarding this issue from a project using an old version of leaflet where 'pointer' ends up as false).


i had run this project and found an RuntimeError: shape '[32, 48, 16]' is invalid for input of size 22020096.

I use the pannuke dataset. But I found an error.
Traceback (most recent call last):   File "tools/train.py", line 138, in <module>     main()   File "tools/train.py", line 135, in main     SegPipline(cfg, distributed, not args.no_validate).run()   File "/home/aMI_projects/MCTrans-master/mctrans/pipline/segpipline.py", line 99, in run     self.runner.run(self.data_loaders, self.cfg.workflow, self.cfg.max_epochs)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 127, in run     epoch_runner(data_loaders[i], **kwargs)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 50, in train     self.run_iter(data_batch, train_mode=True, **kwargs)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 30, in run_iter     **kwargs)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/mmcv/parallel/data_parallel.py", line 75, in train_step     return self.module.train_step(*inputs[0], **kwargs[0])   File "/home/aMI_projects/MCTrans-master/mctrans/models/segmentors/base.py", line 152, in train_step     losses = self(**data_batch)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl     return forward_call(*input, **kwargs)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/mmcv/runner/fp16_utils.py", line 109, in new_func     return old_func(*args, **kwargs)   File "/home/aMI_projects/MCTrans-master/mctrans/models/segmentors/base.py", line 122, in forward     return self.forward_train(img, img_metas, **kwargs)   File "/home/aMI_projects/MCTrans-master/mctrans/models/segmentors/encoder_decoder.py", line 92, in forward_train     loss_aux = self._auxiliary_head_forward(x, seg_label, return_loss=True)   File "/home/aMI_projects/MCTrans-master/mctrans/models/segmentors/encoder_decoder.py", line 81, in _auxili           ary_head_forward     return self.aux_head.forward_train(x, seg_label)   File "/home/aMI_projects/MCTrans-master/mctrans/models/heads/mctrans_aux_head.py", line 51, in forward_train     outputs = self.ca(inputs_flatten)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl     return forward_call(*input, **kwargs)   File "/home/aMI_projects/MCTrans-master/mctrans/models/trans/transformer.py", line 149, in forward     query = layer(query, src)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl     return forward_call(*input, **kwargs)   File "/home/aMI_projects/MCTrans-master/mctrans/models/trans/transformer.py", line 124, in forward     tgt2 = self.cross_attn(tgt, src, src)[0]   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl     return forward_call(*input, **kwargs)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/torch/nn/modules/activation.py", line 1045, in forward     attn_mask=attn_mask, average_attn_weights=average_attn_weights)   File "/home/anaconda3/envs/mctrans/lib/python3.7/site-packages/torch/nn/functional.py", line 5301, in multi_head_attention_forward     k = k.contiguous().view(k.shape[0], bsz * num_heads, head_dim).transpose(0, 1) RuntimeError: shape '[32, 48, 16]' is invalid for input of size 22020096

I found the error from the code in the /mctrans/models/trans/transformer.py
class CALayer(nn.Module):
def forward(self, tgt, src):
self attention
q = k = tgt
tgt2 = self.self_attn(q.transpose(0, 1), k.transpose(0, 1), tgt.transpose(0, 1))[0].transpose(0, 1)
tgt = tgt + self.dropout2(tgt2)
tgt = self.norm2(tgt)
cross attention
tgt2 = self.cross_attn(tgt, src, src)[0] # error from this line.
tgt = tgt + self.dropout1(tgt2)
tgt = self.norm1(tgt)
ffn
tgt = self.forward_ffn(tgt)
return tgt

so I modified this code into ...
class CALayer(nn.Module):
def forward(self, tgt, src):
# self attention
q = k = tgt
tgt2 = self.self_attn(q.transpose(0, 1), k.transpose(0, 1), tgt.transpose(0, 1))[0].transpose(0, 1)
tgt = tgt + self.dropout2(tgt2)
tgt = self.norm2(tgt)

  # cross attention
  tgt2 = self.cross_attn(tgt.transpose(0, 1), src.transpose(0, 1), src.transpose(0, 1))[0].transpose(0, 1)
  tgt = tgt + self.dropout1(tgt2)
  tgt = self.norm1(tgt)
  # ffn
  tgt = self.forward_ffn(tgt)
  return tgt

It can run. But I'm not sure if this is right.
Also, I found something wrong with the results.

Hi, Can you provide the version information of your installed monai? Can you install monai == 0.5.3 and try again with the default code.

Thanks for your answer. The monai version of my environment is 0.8.1. After I install monai == 0.5.3, it is work. Thanks again.

Nice！

Thanks for your answer. The monai version of my environment is 0.8.1. After I install monai == 0.5.3, it is work. Thanks again.

hello!Can you tell me after the operation of ‘transpose(0,1)’, is the final average F1=68.9 ？ same to paper？ My F1 metric only 67.13.

感谢您的回答。我的环境的monai版本是0.8.1。在我安装了 monai == 0.5.3 之后，它就可以工作了。再次感谢。

想要求一下你的MCTran文件，我目前无法运行这个模型


Userlist nicknames wasn't updated

Noticed this issue when autocomplete matched the nickname as it would be at some hours of the day, instead of the one that was currently in use in the chat buffer
Reported based off 9564b561764b687f67ed48457c9aee7d1d2ee336
have since pulled, unsure if this would have any affect

Odd, it acts on users changing nicknames and all that.
I'll look into it, it might just have to do with discord sometimes crapping out....

Actually, I think I might have fixed this with the latest commits (including the one I did just now).
If after updating it still occurs let me know. I might have to build in a periodic check on some of these things.

Nah, still appears to be an issue

Okay, let me know when you have pulled those and if it still occurs. It is a bit hard for me to actively notice since I am mostly active on servers that don't have that many nickname changes.
I'll start thinking about a solution.

Noticed it several times myself now. Just have been swamped in my time available to spend on hobby projects like this.
Hopefully I can fix it this weekend.


4.2 breaks on build: #if targetEnvironment(simulator)

Installed via cocoapods v1.5.3
Xcode 9.2
deployment target: 11.0
v 4.1.0 works fine; 4.2.0 won't build (see attached)
Reachability.swift line 105 and 248:

https://github.com/ashleymills/Reachability.swift/issues/285 was meant to resolve a warning in xcode 9.3, but the implemented change breaks things in 9.2; a warning in 9.3 is far better than broken in 9.2...

You can use an old Reachability release if you're using an old version of Xcode.

True, and for now I will. At the very least this needs to be documented though.

@ashleymills works for me — appreciate your efforts developing and maintaining this!


[CORE] support for *.pt type prompt adapters

Support *.pt type prompt adapters.

This is nice! The OpenAI entry point might need to be looked at: https://github.com/vllm-project/vllm/blob/2f808e69ab16a8775cd58849b15b465d4f11b92e/vllm/entrypoints/openai/serving_engine.py#L91-L102

Closing for now, decision was made internally about monkey-patching this functionality instead


documentation bugs/questions

Intro and Data Model

[x] replace all "open connectome project" with "NeuroData" (ocp now refers only to the EM data)

[ ] (optional) it would be cool if the figures had a source url that generated that cutout

[x] replace "OCP" with "NeuroData"
[ ] is http://docs.neurodata.io/open-connectome/_images/ocp_cluster.png still correct?  maybe we should make an updated version?
[ ] "Configuration" is obviously not done.  but randal has a docker box, maybe just link to that instead?
[x] there are spelling mistakes throughout.  use a text editor with spell check to find them.  "enumerted" is one example.
[x] in this, the list is not really a list
[ ] when defining dataset, it is really important to make clear (in bold) that dataset does not contain any data in it, only metadata for a given dataset.
[ ] in scaling levels, it says ":ref:’the propagation ’ ’", which seems to be a broken link.
[ ] there are a bunch of other places with broken links like that as well.
[x] Order should be: Dataset, Project, Token, Channel
[ ] all remaining TODOs

Admin Console

[ ] make each answer an enumerated list of steps
[ ] for each, make sure to link back to the explanation of all the attributes for that component (eg, when talking about filling in the attributes for Project, link back to the project attribute definition"
(links were added, but don't work)

Ingest

[ ] make the location of the hard coded stuff easy to find, eg, tell them to look for "EDIT THESE VARIABLES" and put that in the code, or something like that.
[ ] again, paragraphs that can be enumerated lists should be.
[ ] tell them how to run the script (and what dependencies they need, eg, python 2.7, or what?)

OCP Types

[x] can't image channels have >3 dimensions? same with time? same with annotation?
[ ] for channel types,  rather than that table and the sentences above, make a bulleted list, where each "OCP type" is defined (eg, 3 bullets)

Meta API comments:

[ ] it is "APIs" not "API's".  although, since API means a "set of routines", i think we could really likely call each of these an "API" (ie, no plural).
[ ] i don't understand the examples. in particular, i don't know where to type that stuff, or whether even? i also don't know how to search for the answer.  maybe a general API section that could explain it once would be really helpful.
[ ] for each API, a 1-2 sentence summary before getting in to the details of each call?

Data APIs

[ ] maybe a sentence at the top explaining we support 3 different ways of posting cubes, and 2 different ways to post image slices?
[ ] also can say up there that the  difference between hdf5, blosc, and numpy in the url is simply whether to write hdf5, blosc, or numpy?
[ ] maybe also explain best practices here? like, do we recommend blosc? and power of 2 cutouts? etc.
[ ] please clarify what "Form Parameters" means
[ ] "The order which post data is zyx"  this seems important? is it mentioned anywhere else? if not, maybe it should be mentioned somewhere other than in the example comments?
[ ] "post_data" has both hdf5 & numpy, let's clarify before the example that it does both
[ ] "/ocp/ca/kasthuri11/image/xz/0/9000,9500/12000/50,250/ HTTP/1.1" the image is too long.  can we just use a shorter one? same with 'yz' image.

RAMON API

[ ] examples everything here please
[ ] "query object service" ?
[x] "Get an annotation to the server" to you mean get from the server?
[x] merge doesn't need "get" under it, right?
[x] under merge it says: "Post an annotation to the server".  that doesn't seem right
[ ] do we have undo now? is it documented?

Propagate

[ ] seems like we should link back to the data model or something.  in particular, how will they know what the values mean?
[ ] "propagate_value"

JSON API

[ ] autoIngest - link to tech sheet, or embed it
[ ] maybe also link to generatejason, or the docs on ingest?

Overlay API

[ ] example for GET XY Slice Cutout
[ ] seems like that should be called "GET XY Overlay Cutout"

Tile APIs

[ ] i think this is a data api, i understand why it is documented separately, but i think that will be confusing for people, and i would just put these under the Data API heading.  for example, the Viking stuff is here, but the KNOSSOS stuff is there.
[ ] getMcfcTile, i'm not sure how this is different from getSimpleTile, please clarify
[ ] getVikingTile: this is just for using with Viking? Let's say that and link to Viking

Extract Graph API

[ ] not clear how it works @Aeusman

NIFTI API

[ ] make right @gkiar @randalburns

SWC API

[ ] make right @gkiar @randalburns

Other stuff

[ ] remove "open-connectome Functions", "Gitter"
[ ] window isn't documented anywhere? @randalburns
[ ] 'oldchannel' is not listed as a type, but we use it, so we should document it (even if it is depricated), so that people knows it exists and is an option, and what it means.

I'm around starting today. Which ones do you want me to tackle @kunallillaney? I'll do at least the extract graph API this week.

hi @Aeusman welcome back! feel free to tackle these issues in your order of preference, modulo any preferences kunal has.
as you complete them, check them off, and i can confirm that it is done and clear that sub-task, cool?

Hey @Aeusman sent you a chat and an email. :)

oh, i forgot, that the figure that shows the data cluster is actually the
top priority for me, because i use it in talks and stuff all the time.
thanks guys!
On Tue, Jan 5, 2016 at 8:41 AM, Kunal Lillaney<EMAIL_ADDRESS>wrote:

Hey @Aeusman https://github.com/Aeusman sent you a chat and an email. :)
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/open-connectome/issues/174#issuecomment-169001540
.

--
the glass is all full:  half water, half air.
neurodata.io

@jovo I'm going to clean up/rewrite some of the Graphgen API stuff, but I'm not going to write an in detail explanation of what goes on under the hood yet because it will be subject to change after I re-write it to fit in the new schema. Once that's done I can go back and document in detail.

@Aeusman sounds great, glad you are back :)

@jovo @kunallillaney The generatejson.py script with irritating me because it was messy and unclear (my fault). I re-factored/wrote a new class called autoingest.py which will be called by generatejson.py to make things easier for users. I also added it to ndio after having @j6k4m8 look it over for style, since it seems like it should also reside there. I'll update the docs tonight and write some tests tomorrow.

btw if this wasn't already obvious, this is so awesome
On Mon, Jan 11, 2016, 7:15 PM Aeusman<EMAIL_ADDRESS>wrote:

@jovo https://github.com/jovo @kunallillaney
https://github.com/kunallillaney The generatejson.py script with
irritating me because it was messy and unclear (my fault). I
re-factored/wrote a new class called autoingest.py which will be called by
generatejson.py to make things easier for users. I also added it to ndio
after having @j6k4m8 https://github.com/j6k4m8 look it over for style,
since it seems like it should also reside there. I'll update the docs
tonight and write some tests tomorrow.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/open-connectome/issues/174#issuecomment-170739875
.

indeed it is!

@jovo The pull request from me has been pulled in, I think everything from me is complete as per Monday meeting

@Aeusman either the merge did not work, or you randomly checked some boxes :)
in any case, there many items that are not yet complete.  please complete or clarify them all before monday. thanks.

@jovo The site has not been updated, changes made a month ago can be seen here for example: https://github.com/neurodata/ndstore/blob/master/docs/sphinx/ingesting.rst
Not sure why this never made it into the docs online, will check with @kunallillaney

@kunallillaney has the site been updated? if not, please update asap so we can finish this.
no matter what happens this week, please update the site.
thank you.

http://docs.neurodata.io/ndstore/sphinx/ingesting.html

does this mean the site was updated?
On Tuesday, June 7, 2016, Alex Eusman<EMAIL_ADDRESS>wrote:

http://docs.neurodata.io/ndstore/sphinx/ingesting.html
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/ndstore/issues/174#issuecomment-224325853,
or mute the thread
https://github.com/notifications/unsubscribe/AACjcoM-balYg-hyLh-c0REuXncyXNEQks5qJZRMgaJpZM4Gdjv6
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York

To my knowledge it's all been updated

@kunallillaney can you clarify?

@jovo Only @Aeusman section is done. I still need to clean up and add the other sections.

ok, let's discuss after stuff in the paper is complete.
On Tuesday, June 7, 2016, Kunal Lillaney<EMAIL_ADDRESS>wrote:

@jovo https://github.com/jovo Only @Aeusman https://github.com/Aeusman
section is done. I still need to clean up and add the other sections.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/ndstore/issues/174#issuecomment-224416721,
or mute the thread
https://github.com/notifications/unsubscribe/AACjckOO4HrfUqaCzZmEmEswOW6HcyH6ks5qJd9fgaJpZM4Gdjv6
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
41e67701ee87b0e49d81621cbb803e9995f80e13
link
/test pilot-presubmit

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


Fix: corrects balance after send before and after transaction

Resolves
Issue #143

you have commits that aren't related to your PR.


make a separate getShiroBundle function in AirpalApplicationBase

Make a separate getShiroBundle() function so that the subclasses can override just that part without duplicate other bundles' code.
Also make a protected variable injector for easy access to it in subclasses as well.

@krishnap @andykram

@andykram updated the PR. Added two functions public Collection<ConfiguredBundle<T>> getConfiguredBundles() and public Collection<Bundle> getBundles() because Bundle and ConfiguredBundle interfaces are not compatible.


feat: add encode interface in proto row

What this PR does:
#102
Which issue(s) this PR fixes:

Fixes #102
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #117 (b80169d) into master (e03c2a9) will decrease coverage by 0.05%.
The diff coverage is 18.18%.

:exclamation: Current head b80169d differs from pull request most recent head 52ea9ff. Consider uploading reports for the commit 52ea9ff to get more accurate results

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   35.39%   35.34%   -0.06%
==========================================
  Files          69       69
  Lines       11002    11044      +42
==========================================
+ Hits         3894     3903       +9
- Misses       6665     6697      +32
- Partials      443      444       +1

Impacted Files
Coverage Δ

pkg/config/config.go
50.70% <ø> (ø)

pkg/executor/redirect.go
13.86% <0.00%> (+0.94%)
:arrow_up:

pkg/runtime/ast/ast.go
83.08% <0.00%> (-0.21%)
:arrow_down:

pkg/runtime/ast/model.go
41.72% <0.00%> (-2.65%)
:arrow_down:

pkg/runtime/optimize/optimizer.go
39.58% <0.00%> (-6.09%)
:arrow_down:

pkg/merge/impl/group_by/group_by_value.go
88.88% <100.00%> (ø)

pkg/mysql/rows.go
9.42% <100.00%> (+3.05%)
: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 e03c2a9...52ea9ff. Read the comment docs.