can't update Connection entity with optimistic concurrency

I am trying to update the statecode of a connection entity with optimistic concurrency enabled but keep on getting "The version of the existing record doesn't match the RowVersion property provided" error.
the RowVersion is set to the last version.
I tried setting the RowVersion on Record1Id and Record2Id too but that did not help.
see stack trace below
"Microsoft.Crm.CrmException: The version of the existing record doesn't match the RowVersion property provided. ---> Microsoft.Crm.CrmException: The version of the existing record doesn't match the RowVersion property provided.
at Microsoft.Crm.BusinessEntities.BusinessProcessObject.ThrowIfVersionMismatch(BusinessEntity retrievedEntity, String sentVersion)
at Microsoft.Crm.BusinessEntities.BusinessProcessObject.CheckIfRowVersionMatchesConcurrencyBehavior(ExecutionContext context, BusinessEntityMoniker moniker)
at Microsoft.Crm.BusinessEntities.BusinessProcessObject.CheckForOptimisticConcurrencyForUpdate(IBusinessEntity entity, ExecutionContext context)
at Microsoft.Crm.BusinessEntities.BusinessProcessObject.<>c__DisplayClass180_0.<Update>b__0()
at Microsoft.PowerApps.CoreFramework.ActivityLoggerExtensions.Execute(ILogger logger, EventId eventId, ActivityType activityType, Action action, IEnumerable`1 additionalCustomProperties)
at Microsoft.Xrm.Telemetry.XrmTelemetryExtensions.Execute(ILogger logger, XrmTelemetryActivityType activityType, Action action)
at Microsoft.Crm.ObjectModel.ConnectionService.Update(IBusinessEntity entity, ExecutionContext context)
--- End of inner exception stack trace ---
at Microsoft.Crm.Extensibility.VersionedPluginProxyStepBase.Execute(PipelineExecutionContext context)
at Microsoft.Crm.Extensibility.PipelineInstrumentationHelper.Execute(Boolean instrumentationEnabled, String stopwatchName, ExecuteWithInstrumentation action, PipelineExecutionContext context)
at Microsoft.Crm.Extensibility.Pipeline.<>c__DisplayClass2_1.<Execute>b__0()"

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

ID: 9bd88b8b-820f-7ffd-14d0-3228dee701ea
Version Independent ID: a1389201-bd9e-717d-d171-9b91db4b39a4
Content: Reduce potential data loss using optimistic concurrency (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/developer/org-service/reduce-potential-data-loss-using-optimistic-concurrency.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

These issues are for documentation feedback, not technical support.
The purpose of the feature is to ensure that you will not update (or delete) a record which has changed on the server since you retrieved it.
If you have some feedback that the content is not correct, please call out what is not correct and should be changed. Otherwise, I recommend you contact technical support or post a question on the forums/stack overflow.

this is kind of between,
it's either a bug in dynamics or it's by design and then the documentation should mention optimistic concurrency is not enabled for the connection entity.

In my system, the IsOptimisticConcurrencyEnabled value for the connection entity is true. The content does call out that you can check this using the Metadata browser (and it includes a link to that tool). Are you saying it isn't in your environment?
Based on what you have provided, I can't call this a bug. I'm not clear about what you are expecting, and you don't provide the error code returned (you only have the stack  trace).
The content section about handling exceptions lists the expected exceptions.
If you want to report this as a bug, support is the way to do that.
If you see something missing in the docs, I'll need you to call out what is missing based on your experience. Providing detailed repro steps will help increase confidence that this will lead to actionable effort on our part.
Details repro steps would include:
Goal: What you want to do
Steps: How you want to achieve the goal, in this case a specific code snippet that includes the code you tried, any preconditions, etc
Expected: The outcome you expect based on the docs
Actual: The unexpected outcome.
I would also encourage you to download and try the sample that supports this content Sample: Use optimistic concurrency with update and delete operations
I would actually recommend you use this page in the PowerApps content: Sample: Use optimistic concurrency with update and delete operations because it is a little more up-to-date.

i wasn't sure if it's a bug or not, based on your response i understated it is, i will open one to support.
anyway here are the details in case you are interested.
IsOptimisticConcurrencyEnabled value is set to true
error code is -2147088254 (ConcurrencyVersionMismatch)
Goal: update a connection entity
Steps: get entity, change value, update
Expected: entity gets updated
Actual: -2147088254 error is thrown.
code snippets i used,
var ctx = new OrganizationServiceContext(svc);
ctx.MergeOption = MergeOption.NoTracking;
ctx.ConcurrencyBehavior = ConcurrencyBehavior.IfRowVersionMatches;

var oldConnection = ctx.CreateQuery<Connection>()
    .FirstOrDefault(c => c.Record1Id != null
        && c.Record2Id != null
        && c.StateCode == ConnectionState.Active);

var connection = new Connection
{
    ConnectionId = oldConnection.Id,
    RowVersion = oldConnection.RowVersion,
};

connection.StateCode = ConnectionState.Inactive;
connection.StatusCodeEnum = Connection_StatusCode.Inactive;

//using OrganizationServiceContext
ctx.Attach(connection);
ctx.UpdateObject(connection);
ctx.SaveChanges();

//using IOrganizationService
svc.Execute(new UpdateRequest
{
    Target = connection,
    ConcurrencyBehavior = ConcurrencyBehavior.IfRowVersionMatches
});

//using web-api
var client = new HttpClient();
client.DefaultRequestHeaders.IfMatch.Add(EntityTagHeaderValue.Parse($"\"{connection.RowVersion}\""));
var request = new HttpRequestMessage(new HttpMethod("PATCH"), $"{url}")
{
    Content = new StringContent(connection.ToJson().ToString(), Encoding.UTF8, "application/json")
};
var response = await _client.SendAsync(request);

all ways behave the same.

Are you finding this behavior only for the connection entity? Does it work for you with any other entities?
For the Web API, the expected behavior is described here: Perform conditional operations using the Web API > Apply optimistic concurrency > Apply optimistic concurrency on update

Notice that the If-Match value must be W/"<version value" which represent the ETag value returned rather than the record RowVersion  value.
The error is expected if the ETag values don't match.
For the SDK assemblies, I haven't tried using OrganizationServiceContext for this, only the IOrganizationService interface methods.

the same code works for other entities (including the web API).
i checked, the row version didn't change.
var oldConnection = ctx.CreateQuery<Connection>()
    .FirstOrDefault(c => c.Record1Id != null
        && c.Record2Id != null
        && c.StateCode == ConnectionState.Active);

oldConnection.StateCode = ConnectionState.Inactive;
oldConnection.StatusCodeEnum = Connection_StatusCode.Inactive;

try
{
   svc.Execute(new UpdateRequest
   {
        Target = oldConnection,
        ConcurrencyBehavior = ConcurrencyBehavior.IfRowVersionMatches
   });
}
catch {}

var connection2 = ctx.CreateQuery<Connection>()
                  .FirstOrDefault(c => c.ConnectionId == oldConnection.ConnectionId);

(connection2.RowVersion == oldConnection.RowVersion).Dump(); //True

Sounds like a bug to me. Please include a link to this issue in the support case you open. If there is something special about the connection entity we need to document, we can re-open this.

Hi @mottibec
Have you got any solution for this or any update from Product team? I have a similar situation with Email entity. We have a custom filed in email entity. All email attachments are moved to azure blob and ids of those azure documents are stored in this custom field as comma-separated values. Now even for received emails, we need to store attachments into the azure blob and then need to update this field.  But I observed that when there are multiple files in the received email, our workflow is getting triggered for each file and trying to update the field and ending up overwriting the same field and at last, there is only one file id in a field where as it should be a comma-separated file list.  So I used optimistic concurrency to overcome this problem but getting an error that row version is not the same.


⚠ Bofrost.be has degraded performance

In 96a5b24, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1355 ms

Resolved: Bofrost.be performance has improved in 1da9340.


add support for multicore?

Improvement Description
Is there a reason the -j option for cutadapt v3.0 isn't used which support multicore runs for cutadapt
Current Behavior
Single-thread cutadapt (default -j 1) is what current runs
Proposed Behavior
add a CLI option to specify a core / thread count to use

Hey there @hyphaltip - we already support cores in the trim-single and trim-paired methods, so I imagine you're probably referring to the demux methods? The reason we don't have the multicore exposed for the demux methods is because cutadapt simply didn't support multicore demux at the time these methods were developed. I would love to see this wired up now that cutadapt supports it - are you interested in contributing that change?

yes this is in reference cutadapt demux-paired - I didn't specify sorry.
I would be happy to try - let me see if I can figure out how to model the same cmdline params in the trim-single that would go into this plugin
Its a huge speed boost for this demux

Awesome, that would be great! Let us know if you need a hand with anything, we're always happy to chat!

I notice you don't have defaults structured in same way in _demux as you do in _trim - is that just code drift or intentional?
Is this best place to put Qs?
I'm not totally sure how to setup a test env for qiime since I'm mostly just running based on an installed system. I can see how to add all this to the plugin but I assume would also need to add something upstream in qiime core code to send in the core/threads request too?

I notice you don't have defaults structured in same way in _demux as you do in _trim - is that just code drift or intentional?

It is intentional - the demux methods don't have nearly as many parameters to orchestrate. That pattern in the filter methods is just a nice way to keep a large chunk of values consistent.

I'm not totally sure how to setup a test env for qiime

https://dev.qiime2.org/latest/

Solution on my branch - pull request coming in via #50 now.

I might back this off as while it is faster I don't know that the multicore is respecting the dual-indexing I awas trying to run - I will do some more testing on new dataset.

Fixed in #50


Color training costs based on label value

Need to map label space from 0 to 1.

Can you describe more?

Sure. The idea is to plot the loss for each class as separate lines in the loss diagram. This requires to select colors for the classes. Mapping the possible labels between 0 and 1 would allow to color those lines from a Matplotlib color map. However, now I think hard coding colors from a palette (colorbrewer2.org) is better.

I did not understand what exactly is meant by 'class' here? Is is different methods to train the NN given in gradient.py?

Haha I see. By class I mean the labels in a classification dataset.


High memory use

SDWebImageManager *manager = [SDWebImageManager sharedManager];
[manager downloadImageWithURL:post.imageURL options:0 progress:^(NSInteger receivedSize, NSInteger expectedSize)
{
    // float progress = (float)receivedSize/expectedSize;
    // [cell.imageViewFoto updateProgress:progress];
}
completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, BOOL finished, NSURL *imageURL) {
    if (image) {
        [cell.imageViewFoto setupImageView:post indexPath:indexPath isGrid:NO  image:image];
    }
}];

whats the size of the image you are trying to download?

about 500kb each...
showing in a collectionview, about 12 images at same time...
The problem occurs when i scroll the collectionview, and request more and more and more images...
Maybe i have to disable the memory cache and only store in disk.. but i have no idea how to del with it

did you try setting these two:
[[SDImageCache sharedImageCache] setShouldDecompressImages:NO];
[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:NO];

by the way, whats the highest resolution of those images?

No, i havent, i will try this code...
The API returns me a lot of images already resized, so i choose witch one to use, depending of the iPhone resolution(100x100, 200x200, 500x500, 1000x1000, 1500x1500, 2000x2000)
in this particular case, im using the 500x500 ( but the print of the collectionview with the images was using 100x100 for testing)

Btw, where should ai use this code? appdelegate or viewcontroller?

should be before you make your requests. try the appdelegate

On viewcontroller nothing changed... still increasing memory

even on appdelegate, nothing have changed too... =(

    "thumbnail": {
      "height": 100,
      "url": "https://timehi.s3-sa-east-1.amazonaws.com/post/160c6ef0a2d2db2a6502.jpg",
      "width": 100
    }

the 100x100 pixels image is about 4kb...
and increase a lot of memory

why are you using SDWebImageManager's downloadImageWithURL:... and not [cell.imageView sd_setImageWithURL:...
can you try the demo project? do you see the same increase in memory there too?

i cant show the image as soon as the dowload finish, i have to implement some logic before with the image...

if you perform extra actions on the image you could use this
did you try the demo project? does it show the same memory increase?

i will, try right now

it still increasing memory with the code bellow:
[cell.image sd_setImageWithURL:downloadImg completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, NSURL *imageURL) {
    if (image) {
        BOOL isGrid = YES;
        if (self.headerProfile.segmentedControl.selectedSegmentIndex == 1) {
            isGrid = NO;
            cell.image.delegate = self;
        }else{
            cell.image.delegate = nil;
        }
        [cell.image setupImageView:post indexPath:indexPath isGrid:isGrid image:image];
    }
}];

also have checked all strong and weak properties referencies... just to double check if the ARC is releasing all object correctly, but it's all right with it

try the demo project with your API urls but without doing anything extra to the image after download. do you still see any increase in memory?

I have to go now, but i will try in about 2 hours, tks by now

@CavalcanteLeo any updates? Any chance you are using animated (GIF) images?

by changing setShouldDecompressImages to YES, in some benchmarks, my app performance went from 600mb to about 50mb
[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:NO];
[[SDImageCache sharedImageCache] setShouldDecompressImages:NO];

and then from 50mb to about 5mb with this:
[[SDImageCache sharedImageCache] setShouldCacheImagesInMemory:NO];

also set some other properties (no impact changed):
[[SDImageCache sharedImageCache] setMaxMemoryCost:1024 * 1024 * 1];
[[SDImageCache sharedImageCache] setMaxCacheAge:3600 * 24 * 7];

and on each viecontroller e clean the cache
-(void)viewDidDisappear:(BOOL)animated{
    [super viewDidDisappear:animated];
    [self ckearImageCache];
}
-(void)didReceiveMemoryWarning{
    [super didReceiveMemoryWarning];
    [self ckearImageCache];
}
-(void)ckearImageCache{
    SDImageCache *imageCache = [SDImageCache sharedImageCache];
    [imageCache clearMemory];
}

but now, my problem is app performance, as i set ShouldDecompressImages to NO.
When i scroll my collection view to fast, my app freezes a bit

Did you try to activate the address sanitizer in your scheme? I have myself a crash because of sdwebimage when i activate it.

We have also been monitoring an issue with RAM usage being way to high. We upgraded to v3.7.3 and followed the example above. e.g.:

I am using 3.7.3 and did the code like
[[SDImageCache sharedImageCache] setShouldDecompressImages:YES];
[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:YES];
[[SDImageCache sharedImageCache] setShouldCacheImagesInMemory:NO]
it didn't work for me. How to solve the problem??

@SpaceStar2016
setShouldCacheImagesInMemory: is a method not available in v 3.7.3, so I am fairly sure you're not running v 3.7.3.
If you're using cocoapods to add dependencies to your project, change your podfile to:
pod 'SDWebImage', '3.7.3'
then run pod install
Remove the line:
[[SDImageCache sharedImageCache] setShouldCacheImagesInMemory:NO] or you will get compile errors.
Hopefully this will help.

@ynotski I am sure I am using 3.7.3 I can code [[SDImageCache sharedImageCache] setShouldCacheImagesInMemory:NO] in my project.

I think this issue is similar to #538 #586 and #1354.
#787 and 00bf467 should give you a good solution for dealing with large images. Together with the GIF handled by FLAnimatedImage, this ticket should be done now.
Closing. Let's see how this is with 4.0.0

This is closed but just want to add that I experience must higher memory usage in 4.0.0-beta2 that in mentioned here 3.7.3. In both cases with each image download memory is rising and not falling back later

Having same issue here, I am on xcode 9.4, swift 4.X, and tested on ios 11(Simulator)

After lots of R&D and applying all kind of options for SDImageCache, I have finalized below options to set to avoid crash on heavy images load with the Best performance on RAM cache load:
Hope this will help you all !!!
Add below code into your AppDelegate class:
import SDWebImage
SDImageCache.shared().config.maxCacheAge = 3600 * 24 * 7 //1 Week
        SDImageCache.shared().maxMemoryCost = 1024 * 1024 * 20 //Aprox 20 images
        //SDImageCache.shared().config.shouldCacheImagesInMemory = false //Default True => Store images in RAM cache for Fast performance
        SDImageCache.shared().config.shouldDecompressImages = false
        SDWebImageDownloader.shared().shouldDecompressImages = false
        SDImageCache.shared().config.diskCacheReadingOptions = NSData.ReadingOptions.mappedIfSafe

Thanks @smindia1988 for sharing these settings

great work.Thanks @smindia1988

Ok. I added https://github.com/SDWebImage/SDWebImage/wiki/Common-Problems#configuration-for-large-images-avoid-high-memory-usage

@smindia1988
I am using version 5.0.1 that don't has shouldDecompressImages and shouldDecompressImages.
So how to avoid crash on heavy images?

@ACode-Farmer Use .avoidDecodeImage. You can also use the OptionsProcesser for global control instead of invidual image request control. See #2736
I'll update the Wiki soon about this. Seems out of date.

Thanks for your  reply. @dreampiggy
When I load multiple pictures, the memory is still too high.Use the same website for all pictures.(imageUrl = http://qiniu.ve-link.com//img/20191007/16d4d2e4e823e7449014aff5074096508372c2e9.jpg?imageView2/3/q/100).
SDWebImage  @config:
SDImageCache.sharedImageCache.config.maxDiskAge = 3600 * 24 * 7; // 1 week
SDImageCache.sharedImageCache.config.maxMemoryCost = 1024 * 1024 * 20; //20 images
SDImageCache.sharedImageCache.config.shouldUseWeakMemoryCache = false; //Default True => Store images in RAM cache for Fast performance
SDImageCache.sharedImageCache.config.diskCacheReadingOptions = NSDataReadingMappedIfSafe;
SDWebImage Use:
[imageView sd_setImageWithURL:[NSURL URLWithString:obj] placeholderImage:nil options:SDImageCacheAvoidDecodeImage];
I tested three options setting:
1、SDWebImageScaleDownLargeImages
2、SDWebImageAvoidDecodeImage
3、SDWebImageScaleDownLargeImages | SDWebImageAvoidDecodeImage
Memory is still not reduced.

RAM usage for bitmap image usage, the fomula is really simple:
Pixel Count * 4.
For you demo image, it's a 3840px * 5120px, is 75MB.
So, you'd better choose another idea for this, such as preprocess the image with lower resolution, or using something on the Client like Wiki - Image Transformer. To scale down the resolution for images.
You can also try to use the CATiledLayer to display super large images. Which means, you don't load the image entirelly into memory, you load them by tiles. Then, track the user's scroll and load another tile, etc. This is just the technolegy used by Map apps (For example, a full map bitmap image is 10000000*10000000 pixels, this can not been hold in RAM at all).

@ACode-Farmer Update the documentation, see Wiki - Optimization for large images

Thanks @dreampiggy
Which version are these settings in?

The documentation is updated to latest SDK name. The options processor is available in 5.1.0, the other is available in 5.0.0. See wiki: https://github.com/SDWebImage/SDWebImage/wiki/Advanced-Usage#options-processor-510


[feature] Add image size adjustment function

we can freely select the picture size

can you elaborate more on which image we are talking about? you mean the background one right? what do you mean by freely select the picture size?


Fix dependency bugs

See [https://github.com/avanzu/AdminThemeBundle/pull/44](Fix dependency bugs by jibundeyare · Pull Request #44 · avanzu/AdminThemeBundle)

The pull request this issue was pointing to was deleted by the admins, so I'm closing it.


Forge missing dependency screen crashes when Chisel is installed

Minecraft:
1.12.2
Forge Version:
forge1.12.2-<IP_ADDRESS>02
Crash Report:
https://paste.dimdev.org/bumotiyobu.mccrash
debug.log
latest.log

Believe it had something to do with a Thaumcraft draco aspect issue between Thaumic Additions and Ice and Fire as everything starts up just fine without Thaumic Additions. Don't know why it triggered anything with VanillaFix though.

duplicate of #46

@DWX12 apparently updating your Forge version should help.
@Runemoro even so, shouldn't VanillaFix work with about every Forge version? Especially one that is only three versions behind?

It should, especially since there is nothing in the forge changelog which should fix this.

Thanks for the quick response. I usually only update forge if there's a mod that needs it or some other issue. Keep up the good work.

@DWX12 So updating forge worked?

@Runemoro
I didn't feel I needed to update forge since my issue was resolved by removing Thaumic Additions. I really don't know why I got that crash with VanillaFix or what the issue was with it. If, for whatever reason, I receive that specific error again, I'll see if updating forge fixes it first before I post here again.
Thank you for your time and for your great mod.

I'm not able to reproduce it. Is this still a problem with the latest forge, chisel, and thaumic additions?

I get the same crash while I don't use Thaumic Additions.

@Hypsellis You're missing Zerocore
[20:04:17] [main/ERROR]: An exception was thrown, the game will display an error screen and halt.
net.minecraftforge.fml.common.MissingModsException: Mod bigreactors (Extreme Reactors) requires [zerocore@[1.12-<IP_ADDRESS>,)]

Add that and try again.

@DWX12 You're missing HammerCore and that's why the game is throwing a fit. See if adding that fixes it on your end. There's something we need to fix on our end, but that should hopefully fix it for you.

Oh I had ZeroCore, but Twitch hadn't updated it well. I didn't know it could come from here since it's VanillaFix which crashed. Thanks for the help. ;)

Again, not sure why I got that error in the first place, but ever since removing Thaumic Additions + Hammercore (which I had added when I had gotten that error, as far as I'm aware of), I haven't had any problems.
Thanks for all the help.

@DWX12 The problem was that VanillaFix made Forge's "Missing Dependency" screen crash. If you don't have a missing dependency, then the missing dependency screen doesn't show up.

@DWX12 you didn't have HammerCore installed according to your log, that's what caused the weird error :P

In the case of Chisel it's probably because it misses Connected Textures.

@Hypsellis No, the issue only happens when Chisel is installed, no matter what the missing mod is. The VanillaFix support for Chisel is what was breaking the missing dependency screen.


Installing Keptn 0.12.4 fails (image keptn/remediation-service:0.12.4 not found)

I just tried to isntall Keptn 0.12.4 with use-case continous-delivery.
Installation didn't work, one pod does not come up:
kubectl -n keptn get pods
NAME                                                            READY   STATUS             RESTARTS   AGE
helm-service-774b9f7cf6-5bdpq                                   2/2     Running            88         61d
remediation-service-65fcd5ff69-q7t45                            2/2     Running            24         14d
keptn-mongo-56c8c56bd7-6m2f9                                    1/1     Running            12         14d
jmeter-service-58fbc89b66-mc9g6                                 2/2     Running            96         62d
argo-service-56b7d565bc-44bsx                                   2/2     Running            92         61d
keptn-nats-cluster-0                                            3/3     Running            36         14d
api-gateway-nginx-84569f8c84-nhlmw                              1/1     Running            0          2m9s
api-service-57cc5fb668-4nn9h                                    2/2     Running            0          2m9s
approval-service-59487cd554-cz8ws                               2/2     Running            0          2m8s
bridge-795bd777b9-rz22g                                         1/1     Running            0          2m9s
lighthouse-service-59689487b-nkdbj                              2/2     Running            0          2m8s
mongodb-datastore-7c8b7db5fd-7ngvr                              2/2     Running            0          2m8s
configuration-service-57c56bb7c-7v57s                           1/1     Running            0          2m5s
secret-service-7f599d8979-97bln                                 1/1     Running            0          2m4s
shipyard-controller-759cfdd9b6-nn9b8                            2/2     Running            0          104s
statistics-service-9db88d599-mn6ms                              2/2     Running            0          104s
webhook-service-5d69566cf9-hnrlx                                2/2     Running            0          95s
remediation-service-67585c55b5-q95vr                            0/2     ImagePullBackOff   2          2m9s

I already checked, looks like the image is missing:
docker pull docker.io/keptn/remediation-service:0.12.4
Error response from daemon: manifest for keptn/remediation-service:0.12.4 not found: manifest unknown: manifest unknown

This will be solved by a follow-up release we have planned for today afternoon (CET timezone)

Due to pipeline problems, we decided to remove Keptn 0.12.4 from the list of upgradable paths.
Keptn 0.12.6 is out and fixes the upgradability issue.


couldn't connect to 'https://huggingface.co' to load this file

OSError: We couldn't connect to 'https://huggingface.co' to load this file, couldn't find it in the cached files and it looks like bert-base-uncased is not the path to a directory containing a file named config.json.
Checkout your internet connection or see how to run the library in offline mode at 'https://huggingface.co/docs/transformers/installation#offline-mode'.
If I directly specify the path of bert base truncated, an error will occur and the local connection cannot be made https://huggingface.co May I ask how I can solve it

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


[Feature] Enable auto retrieving images from yaml file for configuration tests

Why are these changes needed?
Currently, for configuration tests, we need to manually assign the image version to the tests. And the assigned image version needs to be the same as the image version indicated in the yaml file. For example, an image dict is created manually as shown below. It may inconsistent with the image version indicated in the yaml file.
#https://github.com/ray-project/kuberay/blob/master/tests/test_sample_raycluster_yamls.py#L70
#https://github.com/ray-project/kuberay/blob/master/tests/test_sample_rayservice_yamls.py#L36
    image_dict = {
        CONST.RAY_IMAGE_KEY: os.getenv('RAY_IMAGE', default='rayproject/ray:2.2.0'),
        CONST.OPERATOR_IMAGE_KEY: os.getenv('OPERATOR_IMAGE', default='kuberay/operator:nightly'),
    }

With the help of retrieve_images defined in tests/framework/utils.py, images are directly retrieved from yaml file. See below as an example:
image_dict = retrieve_images(yaml_file_list)
image_dict[CONST.OPERATOR_IMAGE_KEY] = os.getenv('OPERATOR_IMAGE', default='kuberay/operator:nightly')

retrieve_images supports retrieving images for:
        #cr
        RayCluster
        RayService
        RayJob
        # Workloads:
        Pod
        Deployment
        ReplicaSet
        StatefulSet
        DaemonSet
        Job
        CronJob
        ReplicationController

Also,  unit tests for util functions defined in tests/framework/utils.py are needed. So, tests/framework/test_framework_utils.py is added to test retrieve_images and other util functions.

Related issue number
None

Checks
the below command test all yaml files under the folder ray-operator/config/samples
path: /kuberay
python3 tests/framework/test_framework_utils.py

To test more workloads and ray crs, add the test.yaml under folder ray-operator/config/samples
and add below code to line 41 in tests/framework/test_framework_utils.py:
             # Note the image version is randomly created
            'test.yaml'                         : {'nginx:1.14.2','nginx:1.14.3',
                                                   'gcr.io/google_samples/gb-frontend:v3',
                                                   'gcr.io/google_samples/gb-frontend:v4',
                                                   'registry.k8s.io/nginx-slim:0.8',
                                                   'quay.io/fluentd_elasticsearch/fluentd:v2.5.2',
                                                   'perl:5.34.0','busybox:1.28',
                                                   'busybox:1.29','nginx',
                                                   'rayproject/ray:2.2.1','rayproject/ray:2.2.2',
                                                   'rayproject/ray:2.2.3', 'busybox:1.88',
                                                   'rayproject/ray:3.3.0','rayproject/ray:3.3.1',
                                                   'rayproject/ray:3.3.2', 'busybox:1.33',
                                                   'rayproject/ray:4.0.0', 'busybox:3.87',
                                                   'rayproject/ray:4.0.1'
                                                   }

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

Related issue: https://github.com/ray-project/kuberay/issues/838#issuecomment-1544622944


Removed quartz and rztopaz config site files. (#19777)

Description
Resolves #19492
Merge from the 3.4RC to develop.
I removed the quartz386.cmake and rztopaz194.cmake config site files since those machines no longer exist. I had to adjust some other symbolic links since other cmake files linked to them. In particular, I updated the links borax2.cmake , borax5.cmake and rztrona5.cmake. I also noticed that rzgenie5.cmake was obsolete, so I removed it and replaced it with a link to rzwhippet10.cmake.
Type of change

[X] Other - Cleanup of config-site files.

How Has This Been Tested?
I ran the following on rzgenie2.
/usr/WS1/visit/visit/thirdparty_shared/3.4.1/toss4/cmake/3.24.3/linux-x86_64_gcc-10.3/bin/cmake  -DCMAKE_BUILD_TYPE:STRING="Release" -DVISIT_CONFIG_SITE="/usr/workspace/brugger/visit_dev_git/visit/src/config-site/rzgenie5.cmake" -DVISIT_INSTALL_THIRD_PARTY:BOOL="ON" -DVISIT_PARADIS:BOOL="ON" /usr/workspace/brugger/visit_dev_git/visit/src
make -j 36 package

Note that I specified the config site file as rzgenie5.cmake since I was on rzgenie2 and there wasn't a config site file for rzgenie2.
I then checked that VisIt ran in the build and in an installation from the generated tar file.
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[X] I have commented my code where applicable.~~
[X] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have confirmed new and existing unit tests pass locally with my changes.
[ ] I have added new baselines for any new tests to the repo.
[X] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

@brugger1  thanks for these changes, I am still a bit confused b/c I don't see rztopaz on develop or the rc.

I am sorry.
These are config-sites, not host-configs.


Height is not responsive on Windows

The height of the custom frame defaults to 160px on Windows.
I have tested with a clean vault and with the default them.

I can't reproduce this issue (with a clean vault and the default theme), the frame always adjusts to the size of the underlying pane.
What version of Obsidian are you on?

I tried a fresh install of the plugin and it seems to be fine now. Must have been something on my side.
simple plugin but a game changer for me. Thank you.


PopOver triggers form submit

Describe the bug
I'm using a PopOver in a InputGroup and when I click the PopOver button, it triggers the form submit.
To Reproduce
Steps to reproduce the behavior:
<form class="project-request" {{action "createRequest" on="submit"}}>
  <FormGroup @label="CPUs">
    <InputGroup @value={{this.cpus}} @onChange={{action (mut this.cpus)}}>
      <PopOver @icon="help">The number of CPUs to reserve for your project workload.</PopOver>
    </InputGroup>
  </FormGroup>

  <Button @intent="primary" @type="submit">Submit</Button>
</form>

Looks like type property should be added to Button component


(Nit) In search and bottom-up proof, Enter and ESC should work

When in the search (magnifying glass) dialogue, ESC should escape it (Cancel), and the Enter key should "do the right thing" (select "Choose Selected" if that's allowed, otherwise do "Search").
When in the bottom-up proof dialogue, ESC should escape it. The Enter key should "do the right thing" (select "Apply" if that's allowed, otherwise select "Prove").
The same is true in other dialogue boxes - ESC should cancel out of it.

This is partially implemented on dev.
It is very easy to implement when a text field is focused. So this is what implemented on dev - Esc closes a dialog, Enter starts an action, but only when some text field is focused. If, for example, a dialog is opened but no text field is focused, then Esc also should close the dialog but this is not implemented because it is not so easy (just adding onKeyDown handler to a span html element didn't work). I will keep this issue open in hope that in some time it will be implemented.


Ros2 fri 1.16

Minor changes (mostly renaming 1.15 tags to 1.16) to migrate ros2-fri-1.15 branch to ros2-fri-1.16 one.

closing this PR @pala5 . Can you please only add the LBRServer.java and keep everything else unchanged, then create a new PR. Thank you


[app/speed] fix globals shared variables

Description of change
Remove three global shared variables, so it simplify a lots some parts of the code.
Waiting for comments.

I'd prefer the testnum parameter be removed, but okay.

Out of interest is the only reason for the change to simplify the behaviour or is there expected to be follow on changes building on it? I was just wondering as the changes only seemed to very slightly simplify a few lines, while the overall lines of code was not reduced. There just didn't seem to be much of a use case for removing the global variables in a benchmarking application where the only advantage I could see would be making it threadsafe and running multiple performance benchmarks in parallel. Perhaps if there was a bigger goal to convert the multi option to use threads rather than forking or something like that? At the moment the current globals do not change during a particular test combination so there should not be issues with global values shared between fibres in the async tests.
At the point when the original code was converted to support async it was decided to allow some global variables that were deemed harmless. Is it possible to wait another day or so for @agrandi to take a look as he was involved in that original commit and probably has some comments. He is based in Ireland and it is a National Holiday there today so he won't be able to give feedback until tomorrow.


WORKSPACE new_local_repository system_linux path is wrong on Ubuntu 20.04 (Windows 10 WSL)

must be path = "/usr/lib",
instead of path = "/usr/system_libs",

@0ric1 can you please share the error you are getting because of path = "/usr/system_libs"

I called: make tests the result was:
ERROR: /mnt/c/Users/user/libcoral/WORKSPACE:86:21: fetching new_local_repository rule //external:system_linux: java.io.IOException: The repository's path is "/usr/system_libs" (absolute: "/usr/system_libs") but this directory does not exist. ERROR: /mnt/c/Users/user/libcoral/coral/dmabuf/BUILD:7:8: //coral/dmabuf:dmabuf_devboard_test depends on @system_linux//:gstallocators in repository @system_linux which failed to fetch. no such package '@system_linux//': The repository's path is "/usr/system_libs" (absolute: "/usr/system_libs") but this directory does not exist. ERROR: /mnt/c/Users/user/libcoral/coral/dmabuf/BUILD:7:8: //coral/dmabuf:dmabuf_devboard_test depends on @system_linux//:gstreamer in repository @system_linux which failed to fetch. no such package '@system_linux//': The repository's path is "/usr/system_libs" (absolute: "/usr/system_libs") but this directory does not exist. ERROR: Analysis of target '//coral/dmabuf:dmabuf_devboard_test' failed; build aborted: Analysis failed INFO: Elapsed time: 33.069s INFO: 0 processes. FAILED: Build did NOT complete successfully (28 packages loaded, 110 targets configured)

I have tried making that change execute make tests command. It is not successful. Please let us know if you are able to run the command make tests without fail.

It still fails, because of the other issue https://github.com/google-coral/libcoral/issues/12 so I think you need to fix both issues to be able to compile.

unfortunately, as of now we don't have any official support for WSL installations of libcoral without Docker image. Sorry for inconvenience.


Feature/sendinblue setup

Account has been set up.
However it seems that sendinblue is not sending anything.
The remaining mail amount is decreasing, but I haven't received any mails.
Could be that there are some additional steps that need to be made, like account activation
Or maybe we just need to upgrade the plan to paid one

I wonder how you set up the DNS settings?
Usually we have a domain/subdomain for the app, and we need to set a bunch of DNS records for that domain in order for emails to not end up in spam. In this case I don't think we have a muffi domain at all.

I'm merging this. I think it's more than good enough and we can always improve.

@jeppester Do you think that DNS records need to be setup always? In sendgrid you did not need to, for it to send mails. I understand the getting through spam reasoning but the mails simply did not send at all.
What I meant with account activation is more related to this point in the url that I commented above: "Create a ticket or contact us here to request the account activation."


Cloud Foundry Discovery Client and bits

Hi
This PR adds a Cloud Foundry Discovery Client that implements the same patterns as the Eureka, Lattice, Consul and Zookeeper implementations I've seen so far. I've reorganized the directory structure as well to be more similar to other implementations.
I didn't have a chance to do documentation and the Maven dependency situation was a little weird and could stand some scrutiny.
I also contributed tests and a sample that demonstrates the odd bits working as expected (Feign, DiscoveryCient, Ribbon/LoadBalancerClient, RestTemplate as well.) and even provides a simple sample application that could be easily deployed to Cloud Foundry to see things working.

Fixes #5


Language setup

Stand up the Natural Language service implementation. I decided to drive the development against the live service using acceptance tests, hence the lack of mocked out unit tests. Unit tests will be added in a later PR, but for now it would be good to get this implementation accepted so we can start moving on other concerns like documentation. (There is some documentation, but it is no where near final.)
The service is most similar to Vision. There is a main #annotation method that takes content arguments, and a Document class that models the content and can also call an #annotation method.

Coverage decreased (-1.5%) to 94.329% when pulling d82515d877609c3b80162fa853f92d05b63df610 on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.

Coverage decreased (-1.1%) to 94.671% when pulling 1251517eb870f771fffcd94f36a93953d2aedfb2 on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.

Coverage decreased (-0.9%) to 94.927% when pulling e5365890de59c51978d15b6d64c8c9b51b0697e2 on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.

Coverage decreased (-0.9%) to 94.927% when pulling e5365890de59c51978d15b6d64c8c9b51b0697e2 on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.

Coverage decreased (-0.9%) to 94.927% when pulling e5365890de59c51978d15b6d64c8c9b51b0697e2 on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.

Coverage decreased (-0.1%) to 95.661% when pulling 6a61d0c55e3e2280bae0a25cb599fe87bde8913a on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.

I'm really happy with this new approach to test-driving the development against a new service. The acceptance test coverage is just awesome, and the unit tests that follow are based on empirical fact, not speculation. This is the way to do it imo.

Ugh, I would love to merge this, but Job #1403 seems stalled. I'll wait a bit longer.

Coverage decreased (-0.1%) to 95.661% when pulling 6a61d0c55e3e2280bae0a25cb599fe87bde8913a on blowmage:language-setup into 598f85ffac9ab9de2b94039c70f34f75dd8e6bf3 on GoogleCloudPlatform:master.


Purpose of NGN media app name?

In the MOBILE_API.xml spec, there are 2 types of app name - "appName" and "ngnMediaScreenAppName".  I understand that NGN Media Screen App Name is supposed to be a "shorter" name for systems that don't support scrolling text and only have a limited number of characters to display.
However, the spec says both of these app names have a max length of 100 characters:
    <param name="appName" type="String" maxlength="100" mandatory="false">
      <description>Request new app name registration</description>
    </param>

    <param name="ngnMediaScreenAppName" type="String" maxlength="100" mandatory="false">
      <description>Request new app short name registration</description>
    </param>

If NGN app name is supposed to be shorter, why is the max length the same?  Why was NGN media app name added to the protocol instead of just truncating the regular app name to fit on the head-unit side?

ngnMediaScreenAppName is a legacy parameter from the v1 headunits.  It was used in lieu of truncating the appName itself to allow a more custom short name (e.g. Streaming Media Player could be "SMP" instead of "STREA").
The maxlength for most strings originally was arbitrarily set higher as a definite ceiling was unknown.

So, should the ngnMediaScreenAppName be deprecated in the libraries?

My opinion is no.   It is still required by our older head units.  It is still something that every app should/could use to avoid unwanted names on the head unit.  It still functions as a valid call.

What are the guidelines for developers in terms of setting this name since there is no real "max size"?  Should this value be 3 characters? 5? 10?  How can developers be sure their NGN name won't be truncated and, thus, look worse than if the actual app name was truncated?  If there is an actual upper limit to the value, we should modify that in the spec.
Also, does NGN stand for something in particular?  Just curious in case anyone asks me.

The implementation was a max of 5 characters on our legacy systems (older non-nav and nav headunits).  However, the shortened name is also used on our current non-nav headunit to display certain text like " Help", etc.  This case leaves the length more open / less defined.
That said, we can urge users to target 5 characters if possible.
Note:  NGN is a legacy term ("Next Gen Nav").

Thanks for the info @stefanek22 and @Heath-FoMoCo.  Closing this issue.


Style/CSS is invalid

Webpack3 build, rewrite the third-party style is invalid, webpack2 no problem, but dev when the webpack3 performance is right
<style> [v-cloak] {   visibility: visible } html, body {   height: 100%;   width: 100%;   margin: 0px;   padding: 0px; } /* overwrite iview */ .ivu-poptip-arrow {   display: none } </style>
pro
dev

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


NewtonSoft

The upgrade package does not include the newtonsoft.json.dll library which causes the upgrade to break if you do not already have v<IP_ADDRESS> installed of the lib. The main install package contains it though

Can I confirm that this is an issue with 9.2.0?

That's correct, specifically the upgrade zip
On Thu, 14 Jun 2018, 13:57 Mitchel Sellers<EMAIL_ADDRESS>wrote:

Can I confirm that this is an issue with 9.2.0?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnnsoftware/Dnn.Platform/issues/2121#issuecomment-397285666,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHGCavH8BfkBkAPgfX9idWpzj42p-NAKks5t8l3VgaJpZM4Un0Iu
.

It appears the install zip contains Newtonsoft.Json.dll v<IP_ADDRESS>18 in the /bin directory and the Newtonsoft.Json_10.00.03_Install.zip in the /Install/Module folder.
The upgrade zip only contains Newtonsoft.Json_10.00.03_Install.zip in the /Install/Module folder.
I won't be able to test this right now, as I don't have that much time at the moment. I'm guessing however that the installer/upgrader won't function on the older version of Newtonsoft.Json.dll from the previous version they are attempting to upgrade.
Also, in PR #1837 the 09.02.00.SqlDataProvider file https://github.com/dnnsoftware/Dnn.Platform/blob/482c35cf6552b8fb2090ac7c25fa9100d64fe06a/Website/Providers/DataProviders/SqlDataProvider/09.02.00.SqlDataProvider#L107-L118 the platform is updating the assembly version of Newtonsoft.Json.dll. I believe if this is updated before the /Install/Module/Newtonsoft.Json_10.00.03_Install.zip, there is a chance it'll skip copying the assembly because the DB version is the same. Unless the installer is using the repair flag on the api. Again, I wasn't able to test these theory's, but hopefully it'll help someone else from @dnnsoftware/tag dig a bit deeper to see what the root cause and proper solution might be.
It looks like we need to either back out the changes to the web.config and assembly tables and handle all of that with the extension package for newtonsoft. Or remove the extension package and just include the binary in the bin directory directly and bump the assembly in the platform. It's weird right now we have a mix of both.
Any idea why this was done this way? @ashishpd @zyhfish @tpluscode

We need to assembly to be installed via installer, rather than included in the bin/, because a difference in version can cause the site to break due to a mismatch with binding redirects.  The installer will update the binding redirect at the same time as updating the file in the bin/.
I think you're right, @ohine, that updating the Assemblies row in the database would prevent the assembly file from getting copied to the bin/.

The simple solution is to set the version to 10.0.2 via the 9.2.1 SQL script, if it's already 10.0.3, but that's going to require some good documentation to avoid confusing folks looking back on it.
That said, I don't have any other solutions that I like better.  Maybe we could run some code in Upgrade.cs which does a repair install of the package, but there are timing issues we may have to contend with there.

@ohine We're on the wrong trail here.  The change you highlight was removed during testing (1829d1810e6ca6476302669060717c3c9b1f2d86) and is not part of the 9.2.0 package
I don't see this issue doing an upgrade from a plain 9.1.1 site to 9.2.0, and looking at the errors on https://www.dnnsoftware.com/forums/forumid/200/threadid/542050/scope/posts it looks like Newtonsoft is just one part of it.  I've tracked down the issue that caused them to install it this way: DNN-10226, but I'm not sure how this was handled with earlier Newtonsoft upgrades.
I know DnnSharp includes their own version of Newtonsoft, using the <codebase> element, and I'm wondering if there's something that doesn't work if that's in the config (which would explain why it's not an issue we're hearing more about).

@bdukes Thanks for taking the time to dig into this one a bit more. It sounds like we need to do some testing with DnnSharp next and work on reproducing the issue to see what we can do. I can get this into my schedule for the 9.3 release, I highly doubt I'd have the time for 9.2.2. If anyone else from @dnnsoftware/tag has a few extra cycles, it would be helpful to try and get this resolved in 9.2.2 if at all possible.

In my case: Upgrading from 9.1.1 tot 9.2.1 with the latest and greatest from DNNSharp results in an error installing the NewtonSoft 1.0.3. module. I had to manually copy the new dll to the bin\NewtonSoft.Json-<IP_ADDRESS> directory and change the directive in the web.config.
When installing ActionForm (5.0.155, latest at this time) it will add the following to the web.config:

when upgrading DNN911 to 921 it results in a incompatibility

@bdukes @ohine
I debugged through the upgrade process of the Newtonsoft library. Here is what I found:
The DNN manifest file has two components (in this order):

Assembly component
Config component

The installer executes the Assembly component installer that copies the newtonsoft.json.dll version 10.0.3 in bin folder and then calls AddOrUpdateBindingRedirect() which updates the web.config with the binding redirect to 10.0.3.
Then, the Config installer is executed which updates the web.config with the assembly binding once more.
The issue with DnnSharp's <codebase> assembly binding (and maybe other vendors) is in that second step of the install process. The path attribute set in the Newtonsoft package to update the binding redirect is not specific enough and doesn't take into consideration that there could be other dependentAssembly elements in web.config for newtonsoft. It targets the first ab:dependentAssembly element with @name='Newtonsoft.Json'.
In a clean DNN pre 9.2.0 the <assemblyBinding> element contains only one dependentAssembly:
<dependentAssembly>
    <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
    <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
</dependentAssembly>

When installing any DnnSharp module the assemblyBinding changes to:
<dependentAssembly>
    <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
    <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
</dependentAssembly>
...
<dependentAssembly>
    <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
    <codeBase version="<IP_ADDRESS>" href="bin\NewtonSoft.Json-<IP_ADDRESS>\Newtonsoft.Json.dll" />
</dependentAssembly>

After the AssemblyInstaller is executed the web.config changes to (note the order of the elements):
<dependentAssembly>
    <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
    <codeBase version="<IP_ADDRESS>" href="bin\NewtonSoft.Json-<IP_ADDRESS>\Newtonsoft.Json.dll" />
</dependentAssembly>
...
<dependentAssembly>
    <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
    <bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" />
</dependentAssembly>

Now it is ConfigInstaller turn to update the web.config. It searches for the dependentAssembly with path<EMAIL_ADDRESS>which matches the first one (DnnSharp's one). Then it searches for the target node with /configuration/runtime/ab:assemblyBinding/ab:dependentAssembly[ab:assemblyIdentity/@name='Newtonsoft.Json']/ab:bindingRedirect which matches the bindingRedirect of the second dependentAssembly (the one added earlier by AssemblyInstaller). Next it tries to remove the bindingRedirect of the second dependentAssembly from the first dependentAssembly. This throws a System.ArgumentException with message The node to be removed is not a child of this node.
Solution
There are two solutions:

Strengthen the path's XPath like this<EMAIL_ADDRESS>and ab:bindingRedirect]. This will prevent mismatching rootNode / targetNode elements between codabase and bindingRedirect bindings. There could be other bindingRedirect bindings that could interfere with this.
Add an additional Config component before the existing one that removes all the bindingRedirects for older newtonsoft assemblies but not the codabase bindings. Then, the existing Config component will insert an bindingRedirect only if there is no bindingRedirect for version 10. There could be unforeseen issues with this approach.

I am willing to submit a pull request with what you think would be the best solution.
If you think that a better approach is to fix this in our modules, we (DnnSharp) are open to discussion if you think that we are the only vendor that does this.
Please let me know what your thoughts are on the matter.

Thanks for digging into this more @dimarobert!  I definitely think DNN should be able to handle the codebase redirect that y'all use.
Regarding your proposed fixes, would it work to just remove the Config component from the Newtonsoft package, since the platform is already adding the redirect correctly?  Is there any reason it needs to be in both places?
If there's a reason to target the redirect via XmlMerge, I think strengthening the XPath is the right solution.

@bdukes For the upgrade scenario, it should be just fine if we remove the Config component. I will have to do a clean install of DNN  9.2.1 to see if there could be issues there.
From an initial check I see that there is a newtonsoft.json.dll in bin and the bindingRedirect in the  web.config from the install package is diffrent from the one in the upgrade:
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
  <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
</dependentAssembly>

Also there is a Newtonsoft module zip in the /Install/Modules folder. If that is installed as part of the setup process then the web.config should be updated with the oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" part.

@bdukes
Some remarks:

when updating a dnn9.1.1 installation with DNNSharp AF installed, the installation of the newtonsoft installer package (in /install/modules) results in an error.
As a result the dll is not copied into the bin. However the assembly binding section in the web.config is changed to:

There now are two rebinding rules as mentioned by @dimarobert:
      <dependentAssembly>         <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />         <codeBase version="<IP_ADDRESS>" href="bin\NewtonSoft.Json-<IP_ADDRESS>\Newtonsoft.Json.dll" />       </dependentAssembly>       <!--Upgraded by DotNetNuke.Newtonsoft.Json version 10.0.3 - Date: 8/27/2018 4:56:38 PM-->       <!--<dependentAssembly xmlns="urn:schemas-microsoft-com:asm.v1"><assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" /><bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" /></dependentAssembly>-->       <dependentAssembly>         <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />         <bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" />       </dependentAssembly>
Removing the following assemblybinding:
      <dependentAssembly>         <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />         <bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" />       </dependentAssembly>
should result in the use of the first, however the site still crashes. I assume DNN requires Newtonsoft.Json version 10.
Thus we got at least two issues:

when installing a module results in an error, web.config changes are not rolled back
installing Newtonsoft.Json as a seperate/independent module is questionable because DNN relies on it

@dimarobert the thought behind the version range used is that any component which is expecting to use Newtonsoft.Json (or any other strong-named assembly) should be able to use the installed version (i.e. the version of the file in the bin folder).
The alternative would be that .NET just refuses to run the code, because Newtonsoft.Json 10 is in the bin folder but the component was compiled against version 11.

@bdukes Both the Newtonsoft upgrade step and the InstallExtensionsStep in the clean install calls the Upgrade.InstallPackage() method which does a RepairInstall of the package and the web.config ends up being the same at the end.
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
  <!--Upgraded by DotNetNuke.Newtonsoft.Json version 10.0.3 - Date: 8/27/2018 8:33:54 PM-->
  <!--<bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" xmlns="urn:schemas-microsoft-com:asm.v1" />-->
  <bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" />
</dependentAssembly>

In both cases the bindingRedirect is updated twice. So I think that removing the Config component could be a solution.
Can you think of any other scenario for the Newtonsoft.Json zip installation?

Nothing comes to mind, I think that's the right path forward.

@bdukes I've come up with another situation. If the web.config is initially in this state (in opposite order):
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
  <codeBase version="<IP_ADDRESS>" href="bin\NewtonSoft.Json-<IP_ADDRESS>\Newtonsoft.Json.dll" />
</dependentAssembly>
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
  <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
</dependentAssembly>

After the installation of the modified Newtonsoft.Json package (without the Config component) the web.config looks like this:
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
  <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
</dependentAssembly>
<!--Upgraded by DotNetNuke.Newtonsoft.Json version 10.0.3 - Date: 8/28/2018 11:55:01 PM-->
<!--<dependentAssembly xmlns="urn:schemas-microsoft-com:asm.v1"><assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" /><codeBase version="<IP_ADDRESS>" href="bin\NewtonSoft.Json-<IP_ADDRESS>\Newtonsoft.Json.dll" /></dependentAssembly>-->
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
  <bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" />
</dependentAssembly>
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
  <codeBase version="<IP_ADDRESS>" href="bin\NewtonSoft.Json-<IP_ADDRESS>\Newtonsoft.Json.dll" />
</dependentAssembly>

The upgrade process went without any issue, but do you think that this could be an issue in the future?

@dimarobert I don't think that'll cause any major issues, but we should be able to resolve the ambiguity and fix this issue for good.  I've pushed a change to your PR which I believe should do that.

Kinda lost here...I just upgraded a small site from 804 to 922. I figured this issue would be solved in 922. But unfortunately not: see error below.
Any suggestions what my next step should be or how to solve this?
[FileLoadException: Could not load file or assembly 'Newtonsoft.Json' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)]  [FileLoadException: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)]    System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks) +0    System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks) +36    System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks) +152    System.Reflection.RuntimeAssembly.InternalLoad(String assemblyString, Evidence assemblySecurity, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean forIntrospection) +77    System.Reflection.RuntimeAssembly.InternalLoad(String assemblyString, Evidence assemblySecurity, StackCrawlMark& stackMark, Boolean forIntrospection) +21    System.Reflection.Assembly.Load(String assemblyString) +28    System.Web.Configuration.CompilationSection.LoadAssemblyHelper(String assemblyName, Boolean starDirective) +38  [ConfigurationErrorsException: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)]    System.Web.Configuration.CompilationSection.LoadAssemblyHelper(String assemblyName, Boolean starDirective) +738    System.Web.Configuration.CompilationSection.LoadAllAssembliesFromAppDomainBinDirectory() +217    System.Web.Configuration.CompilationSection.LoadAssembly(AssemblyInfo ai) +130    System.Web.Compilation.BuildManager.GetReferencedAssemblies(CompilationSection compConfig) +170    System.Web.Compilation.BuildManager.GetPreStartInitMethodsFromReferencedAssemblies() +92    System.Web.Compilation.BuildManager.CallPreStartInitMethods(String preStartInitListPath, Boolean& isRefAssemblyLoaded) +290    System.Web.Compilation.BuildManager.ExecutePreAppStart() +157    System.Web.Hosting.HostingEnvironment.Initialize(ApplicationManager appManager, IApplicationHost appHost, IConfigMapPathFactory configMapPathFactory, HostingEnvironmentParameters hostingParameters, PolicyLevel policyLevel, Exception appDomainCreationException) +549  [HttpException (0x80004005): Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)]    System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +10074636    System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +95    System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +254

In the web.config the lines with newtonsoft are:
Line 376:
Line 377:
Line 377:
Line 481:
Line 482:
Line 484:
Line 491:
Line 492:
Line 492:
Line 492:
Line 494:

did you clear local browser cache?

No need, uptimerobot also sees the website is down. You can check http://www.bydaans.nl/ yourself

Your web.config should only have one dependentAssembly for Newtonsoft.Json that has a bindingRedirect inside it (rather than a codeBase), and the newVersion attribute in that should match the version in the bin/, probably it should be this:
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" />
  <bindingRedirect oldVersion="<IP_ADDRESS>-32767.32767.32767.32767" newVersion="<IP_ADDRESS>" />
</dependentAssembly>

I deleted some and kept:

This did not change the error.
As it says in the comments, DNN upgraded to 10..3, I changed the web.config to:

This created another error but that is probably related to a module, so...I think I can move on...


added reporting to Telegram messenger

Hi @denisix ,
Thanks for the contribution!
I think we should try to implement this as a configuration of reportHTTP. If not possible, we should at least create it as a class which extends reportHTTP (see reportSlack).
Additional things missing:

support for user groups;
documentation.

Ciao,
Massimo

Hi Massimo,
Thanks for your notes!
I will check regarding reportHTTP and update documentation soon.
Ciao,
Denys

Hi @denisix,
I implemented here the reportTelegram module with reportHTTP and support for user groups.
I run it for a bit and it looks working reliably. Let me know what you think (note: the configuration in config.yml is a bit different).
Thanks!
Ciao,
Massimo

Hi @massimocandela,
Thank you for updates, I will check.


[FEATURE] Warnings for unexpected behavior

Some arithmetical operations have undefinitions such as division by 0 and 0 ^ 0. The current Apalache behavior interprets undefinitions as any value of the correct type. Consider this spec:
------------------- MODULE example1 -----------------------
EXTENDS Integers

VARIABLES
 \* @type: Int;
  number

Next == number' = number * (number \div 0)

Init == number = 5

Property == number /= 35

------------------------------------------------------------
============================================================

When the undefined operation number \div 0 assumes the value 7, Property is violated:
(* Constant initialization state *)
ConstInit == TRUE

(* Initial state *)
State0 == number = 5

(* Transition 0 to State1 *)
State1 == number = 35

(* The following formula holds true in the last state and violates the invariant *)
InvariantViolation == number = 35

However, if we check a property such as number /= 42 (or any number that is not a multiple of 5), the invariant holds, since there is no possible value that the undefinition can assume.
This behavior is problematic because the user can be unaware that their spec contains undefined behavior and also possibly trust unexpected results when an invariant wouldn't hold without the undefinition.
Expected behavior
We should be able to warn the user that undefined behavior is present on the spec. As discussed here, we are now raising erros when there are undefinitions on constant operations. We are also supressing undefinitions under simplifications such as 0 * x (where 0 * undefined = 0), but we cannot warn the user since the undefinition depends on the value of a variable on a given state and this simplification is done statically.
@kukovec suggested that we can add SMT constraints saying that any division denominator should be different than zero (and similar constraints to other undefined operations), where a warning could be logged in the case they're violated. Having these as top-level constraints can result in false-positives in cases where the operation is actually safe, such as IF x == 0 THEN 0 ELSE y/x. However, adding the constraints to specific branches could be unfeasible as pointed out by @konnov.

It's definitely something we should think about. There is no simple answer to this question


Add support for ArcGIS Enterprise on Kubernetes 11.4

Add support for  ArcGIS Enterprise on Kubernetes 11.4 to AWS and Azure arcgis-enterprise-k8s templates.

Added support for ArcGIS Enterprise on Kubernetes 11.4 to AWS and Azure arcgis-enterprise-k8s templates
Upgraded ArcGIS API for Python to 2.4.0


Visual overview of the spec

I'm wondering whether a visual overview of this extension would make sense to include, to get the gist very quickly,
something like below
classDiagram
  direction BT

  class GatherMap {
    @type TermMap

    TermMap[] rml:gather 1
    [rml:Append rml:CartesianProduct] rml:strategy 0..1 rml:Append
    [rdf:Seq, rdf:Bag, rdf:Alt, rdf:List] rml:gatherAs 1
    xsd:boolean rml:allowEmptyListAndContainer 0..1 "false"
    }

We also have the diagrams.net overview diagrams which would be good to update as well.

Agree! I think @anaigmo could help a bit on this, and we can follow a similar approach as we did for RML-star https://kg-construct.github.io/rml-star-spec/, so all the diagrams are consistent between specs

Sure! I'll start with collections+containers, and then update the rest

The new diagram is available in diagrams.net file, so @frmichel @chrdebru you can add it to the spec.

Thanks! It took me a while, but I found the image in KGC CG-W3C (all)/Brainstormings/. Thanks, @anaigmo.

Hi all,
According to recent discussions, I remade the diagram for containers and collections, so that it only includes the class in which it connects to the core, but not the whole core as it is now. The original diagram is still in the same location @chrdebru mentions in the last comment.

Thx for this @anaigmo, that looks good to me.

@anaigmo Great work!
I have one remark and that is regarding rml:strategy. In #23 we decided to leave the domain of rml:Strategy open, so that it can be used more genrally. So that begs the question: What is this image depicting? Is it the ontology, or is it the shapes for this spec, or both?
I guess this is a general question :)

I would understand that the legend answers your question: rml:strategy is an object prop, therefore it relates instances, and rml:Append and rml:CartesianProduct are the objects of this property. This leaves room for creating new instances of rml:Strategy.
Maybe just a typo concern, should we name the instances rml:append and rml:cartesianProduct to stick to the rule that classe names start with a capital letter?

Thanks @pmaria :) I decided to represent only the ontology and leave all restrictions, apart from domain/range, out of the diagrams, partially because some would complicate the images, but we can change and try to represent everything. I loosely follow the diagram notation of Chowlk, so I can open the domain of rml:strategy as you suggest.
Regarding @frmichel 's comment, agree with the first part; and I'll rename the instances to stick with the rule.
Thanks both for the feedback!

I'll rename the instances to stick with the rule.

Ok I will do that too on the spec itself.
Thx,
   Franck.
Le 21/12/2022 à 14:43, Ana Iglesias a écrit :

Thanks @pmaria https://github.com/pmaria :) I decided to represent
only the ontology and leave all restrictions, apart from domain/range,
out of the diagrams, partially because some would complicate the
images, but we can change and try to represent everything. I loosely
follow the diagram notation of Chowlk
https://chowlk.linkeddata.es/notation.html, so I can open the domain
of |rml:strategy| as you suggest.
Regarding @frmichel https://github.com/frmichel 's comment, agree
with the first part; and I'll rename the instances to stick with the rule.
Thanks both for the feedback!
—
Reply to this email directly, view it on GitHub
https://github.com/kg-construct/collection-containers-spec/issues/25#issuecomment-1361328907,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI54EIVFDOHEDUVSDLXX6DWOMCQRANCNFSM6AAAAAAQS7M77E.
You are receiving this because you were mentioned.Message ID:
@.***>


 I would prefer the docs to be more in detail on how to tackle this.
So here;s my setup

I have this /viewer/scan?id=<study_id> , the study Id is used to call an API the returns a dicomjson with the urls as signed Google storage files.
When I try to open a study , I get this error

Access to XMLHttpRequest at 'https://storage.googleapis.com/64afbd53e081b578cba6fee4/663883b6ae9638024e4c468e/663883b6ae9638024e4c4693/allFiles/ead62bf9-7e1ae978-0c73aed5-a19f4848-8035ee86.dcm?X-Goog-Algorithm=GOOG4-RSA-SHA256&X-Goog-Credential=firebase-adminsdk-ru9j2%40test-server-309204.iam.gserviceaccount.com%2F20240529%2Fauto%2Fstorage%2Fgoog4_request&X-Goog-Date=20240529T074956Z&X-Goog-Expires=518400&X-Goog-SignedHeaders=host&X-Goog-Signature=a078fea0fd0c1851a05af027effd16f7cc738531cd4a47afbc9355831cb33f125d1cad28cae7691d56a6d1c76eacceaadab3f4d47b61278ab96a0f30466d5b561d45cb79b0cc365785dc20f042de4c4fbc031e49c3dabc9e3f724bea659434c4b20290de254375e4766abbc0fb76993611da852eec1c939db3e9fb1de66f904ad3293892b41a8151c9328eff9e5c26aa8f955eedb569ec06268b32110c72955fd63a6b8dedd01752dad80a6198b9d05fbd34df1ee6bd4685f56a7c3944666eb99340062fefc2732e80dbea664a10061ce7576aa5bd5cf6ade27b4f8e78daa65d59fcf51025cbdb85d3379d77f456d515bbaed3f91078ac1530e33e7c727c67de' from origin 'https://ohifv3.svasthiya.in' has been blocked by CORS policy: Request header field accept is not allowed by Access-Control-Allow-Headers in preflight response.
However, if I use the chrome extension - CORS Helper, I am able to view the images, which tells me that the GCP config is good. I just have to change the OHIF config.
Will it be possible to provide the correct suggestion on how to navigate this.
Steps to Reproduce
Create a data source that calls a backend api that fetches a dicomjson with google storage url
The current behavior
Images fail to load due to CORS issue
The expected behavior
No CORS issue and images load
OS
Windows 11
Node version
20
Browser
Latest Chrome and edge

Even after using the chrome CORS extension, a few a failing to load

You should set up your Google Healthcare server to allow your origin. See more details here:
https://docs.ohif.org/deployment/cors

Hey @sedghi , I am not using google healthcare, it's just Google Storage S3 files. Do I need to setup the cors in OHIF for this? If yes, some guidance would be helpful

@sedghi also the CORS config have been done correctly for Google Cloud Buckets, because the same urls are being fetched in OHIF v2, I guess I'm missing the areas where I'd have to update them in OHIF v3

@sedghi I have tried everything - here's the GCP Cors

Here's the webpack.pwa.js

Here's my modified DicomJSONDataSource/index.js

Been stuck on this for over a week.


"Racer returned a non-zero exit code" 1 null

I ran into the same problem as described in this issue #48. I have all pereferences configured right (*racerBinPath=*C:\Users\Zhukovskiy\.cargo\bin\racer.exe, *rustSrcPath=*C:\Users\Zhukovskiy\Documents\rustc-beta but I get "keyword autocomplete" only  from 'language-rust' plugin. I cannot figure out what's going on. I use v1.5beta right now.

My bad, i should reference rustc-beta\src not root directory.

Maybe that should be on the README. Hit that, too.


Pipeline sequence diagram

Description
As a reviewer, developer, and admin I'd like to have sequence diagram for all processes that happens in CCX data pipeline, so I will be able to understand what's going on.
Fixes https://projects.engineering.redhat.com/browse/CCXDEV-2310
Type of change

Documentation update

Testing steps
N/A

Codecov Report

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

@@           Coverage Diff           @@
##           master     #143   +/-   ##
=======================================
  Coverage   47.53%   47.53%
=======================================
  Files          12       12
  Lines         993      993
=======================================
  Hits          472      472
  Misses        462      462
  Partials       59       59

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 c19eb79...071d4cc. Read the comment docs.


iOS - SMS - Code is changed when select it in autofill

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch App
Enter SMS account
Tap on the suggestion with the code received from messages

Expected Result:
Code fill out and sms account sign in
Actual Result:
Code is changed when select it in autofill
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.75-2
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/b2d42997-8d2f-4b10-8ea0-354cdd586365

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Issue reproduced on build <IP_ADDRESS> on iPhone 14 Pro/ 17.0.2
https://github.com/Expensify/App/assets/78819774/cf960cc0-af20-4281-ae4a-fe80f854fae2

I suspect this came from https://github.com/Expensify/App/pull/24012 as it's only the recent change related to magic code input.

Hi! I'am the creator of the Magic Code PR mentioned above and tried to repro the issue and fix it but my phone number doesn't seem to work. Do you know any workaround that would help me fix it?

@wojtus7 because this is a deploy blocker, should we consider reverting the PR given this is harder to test.
I think is the staging/ production backend env you just need to have real phone number

Revert fixed this issue https://expensify.slack.com/archives/C9YU7BX5M/p1696258649574039?thread_ts=1696257638.565279&cid=C9YU7BX5M we can close it


Request icon "Bilibili", "Netease Music"

These two are common social platforms in China, please add them.
You can search for the corresponding keywords on https://www.iconfont.cn/ to find references.

"Bilibili" and "Netease Music" icon has added in v2.96


Fix golden file output

Looks like there was just an issue with the Golden File output

On my machine I was getting qualitative differences in the output too. I'm guessing the way I was running the tests wasn't threaded or something.


Adding support for 2018 GMC Acadia Denali

closed my first PR so i could clean up my fork and submit a good pull.
this has been working for a couple months now.  Everything functions as intended.  Engage and disengage without issue and never loose steering.
heres a couple drives in case its needed
hwy drive
https://community.comma.ai/cabana/?route=7cc2a8365b4dd8a9|2018-12-02--12-10-44&max=65&url=https%3A%2F%2Fchffrprivate.blob.core.windows.net%2Fchffrprivate3%2Fv2%2F7cc2a8365b4dd8a9%2F2cf78b41cf4a06693a0778e5a7afa853_2018-12-02--12-10-44
city drive
https://community.comma.ai/cabana/?route=7cc2a8365b4dd8a9|2018-12-01--15-15-36&max=27&url=https%3A%2F%2Fchffrprivate.blob.core.windows.net%2Fchffrprivate3%2Fv2%2F7cc2a8365b4dd8a9%2F9c0ec2418856d247e93c13ca0cf6d48a_2018-12-01--15-15-36

clean. Nice work


addcyclic: TypeError: invalid indexer array

lon = dataset.longitude
lat = dataset.latitude
print(dataset.shape)
(241, 480)
print(lon.shape)
(480,)
print(lat.shape)
(241,)

fig = plt.figure()
ax = fig.add_axes([0.1,0.1,0.8,0.8])
m = Basemap(projection='cyl',llcrnrlat=-90,urcrnrlat=90,\
            llcrnrlon=-180,urcrnrlon=180, resolution='c')

dataset, lona = addcyclic(dataset, lon)
TypeError: invalid indexer array, does not have integer dtype: array([slice(None, None, None), slice(0, 1, None)], dtype=object)

I can't understand why I'm getting this TypeError?

@WeatherGod
It says my version of Basemap is 1.0.8.dev0, I think I saw that there is a 1.1.0? But conda says it is already up to date. I used conda-forge to install it.
Here's the full traceback:
Traceback (most recent call last):
  File "/var/spool/torque/mom_priv/jobs/63449.rdf-xcat.SC", line 46, in <module>
    pa2, lona = addcyclic(pa, lon)
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/mpl_toolkits/basemap/__init__.py", line 5145, in addcyclic
    return list(map(_addcyclic,arr[:-1])) + [_addcyclic_lon(arr[-1])]
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/mpl_toolkits/basemap/__init__.py", line 5130, in _addcyclic
    return npsel.concatenate((a,a[slicer]),axis=axis)
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/dataarray.py", line 474, in __getitem__
    return self.isel(indexers=self._item_key_to_dict(key))
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/dataarray.py", line 819, in isel
    ds = self._to_temp_dataset().isel(drop=drop, indexers=indexers)
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/dataset.py", line 1539, in isel
    new_var = var.isel(indexers=var_indexers)
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 907, in isel
    return self[tuple(key)]
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 616, in __getitem__
    dims, indexer, new_order = self._broadcast_indexes(key)
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 487, in _broadcast_indexes
    return self._broadcast_indexes_outer(key)
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 539, in _broadcast_indexes_outer
    return dims, OuterIndexer(tuple(new_key)), None
  File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/indexing.py", line 378, in __init__
    'integer dtype: {!r}'.format(k))
TypeError: invalid indexer array, does not have integer dtype: array([slice(None, None, None), slice(0, 1, None)], dtype=object)

Ah, you are using xarray.DataArray objects. They don't behave quite like
numpy arrays. Try converting them into numpy arrays before passing them
into addcyclic. This is usually done by accessing the DataArray's .values
attribute.
When calling conda update on a package that came from conda-forge, you
should specify -c conda-forge. The basemap that is in the default channel
hasn't been updated in forever (I have no control over that).
On Tue, Oct 9, 2018 at 1:27 PM Amethyst<EMAIL_ADDRESS>wrote:

@WeatherGod https://github.com/WeatherGod
It says my version of Basemap is 1.0.8.dev0, I think I saw that there is a
1.1.0? But conda says it is already up to date. I used conda-forge to
install it.
Here's the full traceback:
Traceback (most recent call last):
File "/var/spool/torque/mom_priv/jobs/63449.rdf-xcat.SC", line 46, in
pa2, lona = addcyclic(pa, lon)
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/mpl_toolkits/basemap/init.py", line 5145, in addcyclic
return list(map(_addcyclic,arr[:-1])) + [_addcyclic_lon(arr[-1])]
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/mpl_toolkits/basemap/init.py", line 5130, in _addcyclic
return npsel.concatenate((a,a[slicer]),axis=axis)
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/dataarray.py", line 474, in getitem
return self.isel(indexers=self._item_key_to_dict(key))
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/dataarray.py", line 819, in isel
ds = self._to_temp_dataset().isel(drop=drop, indexers=indexers)
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/dataset.py", line 1539, in isel
new_var = var.isel(indexers=var_indexers)
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 907, in isel
return self[tuple(key)]
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 616, in getitem
dims, indexer, new_order = self._broadcast_indexes(key)
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 487, in _broadcast_indexes
return self._broadcast_indexes_outer(key)
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/variable.py", line 539, in _broadcast_indexes_outer
return dims, OuterIndexer(tuple(new_key)), None
File "/nerc/n02/n02/amethyst/.conda/envs/my_root/lib/python3.4/site-packages/xarray/core/indexing.py", line 378, in init
'integer dtype: {!r}'.format(k))TypeError: invalid indexer array, does not have integer dtype: array([slice(None, None, None), slice(0, 1, None)], dtype=object)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/matplotlib/basemap/issues/427#issuecomment-428279672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARy-EYPwQOg9juESxfPFPNXMUHAl1kIks5ujNyHgaJpZM4XTrjQ
.

Thanks @WeatherGod, this worked


Run all conformance tests except redirect tests

Redirect tests are failing due to a possible issue with the way upstream conformance tests have made assumptions Skipping them for now until below issues are resolved
https://github.com/envoyproxy/gateway/issues/992
https://github.com/envoyproxy/gateway/issues/993
https://github.com/envoyproxy/gateway/issues/994
Signed-off-by: Arko Dasgupta<EMAIL_ADDRESS>

Codecov Report

Merging #1014 (8739ba4) into main (b668002) will increase coverage by 0.01%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1014      +/-   ##
==========================================
+ Coverage   62.05%   62.06%   +0.01%
==========================================
  Files          55       55
  Lines        8804     8804
==========================================
+ Hits         5463     5464       +1
+ Misses       2967     2966       -1
  Partials      374      374

Impacted Files
Coverage Δ

internal/provider/kubernetes/routes.go
33.02% <0.00%> (-0.92%)
:arrow_down:

internal/provider/kubernetes/controller.go
50.33% <0.00%> (-0.23%)
:arrow_down:

internal/provider/kubernetes/helpers.go
85.13% <0.00%> (+4.05%)
:arrow_up:

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

HTTPRouteObservedGenerationBump seems to be flaky cc @Xunzhuo
2023-02-09T05:40:23.4494241Z     --- FAIL: TestGatewayAPIConformance/HTTPRouteObservedGenerationBump (0.07s)
2023-02-09T05:40:23.4495622Z         --- FAIL: TestGatewayAPIConformance/HTTPRouteObservedGenerationBump/observedGeneration_should_increment (0.04s)

@Xunzhuo please create a release note for https://github.com/envoyproxy/gateway/pull/1014#issue-1577255903.


Hubspace integration failing to discover "entities" in my Account

Hi there, trying to configure Hubspace integration via HACS on my Home Assistant VM. I've added the following snippet to my "configuration.yaml" file, I've also attached any error/warning logs related to the Hubspace integration.
I've also tried using friendly names and room names, this had no altering effects. according to the documentation it is not necessary and the lock should be discovered as a light entity automatically..
I have restarted Home Assistant and I'm currently running the latest integration v1.99.
light:
  - platform: hubspace
    username<EMAIL_ADDRESS>    password: Password1234
    debug: true

This is the only related warning I could pull up, However I think that's just something that you have to change before that feature is removed.
Logger: homeassistant.helpers.entity
Source: helpers/entity.py:1520
First occurred: 9:55:32 PM (2 occurrences)
Last logged: 9:55:32 PM
Entity None (<class 'custom_components.hacs.update.HacsRepositoryUpdateEntity'>) is using deprecated supported features values which will be removed in HA Core 2025.1. Instead it should use <UpdateEntityFeature.INSTALL|PROGRESS|RELEASE_NOTES: 21>, please create a bug report at https://github.com/hacs/integration/issues and reference https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation

I did get one brief error in my logs, but doesn't seem to be appearing anymore..
2024-01-07 21:33:56.910 ERROR (MainThread) [homeassistant.components.light] Error while setting up hubspace platform for light
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 360, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/hubspace/light.py", line 155, in setup_platform
    for childId in children:
TypeError: 'NoneType' object is not iterable

I'm specifically trying to add a Hubspace lock and plug as "light" entities to HA.
I greatly appreciate your time.

Please use the Home Assistant forum for help. This is for issues, not for help setting up the integration. You probably have a password issue, follow all the troubleshooting tips and type the friendlynames exactly. The above error looks like you specified a room name.
It is possible that your devices are not supported yet, but I would expect they would still show up.


[Hub Generated] Review request for Microsoft.AzureStackHCI to add version stable/2022-01-01

It has only one change from the last public API version. We made a readonly properly to writable.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
January 2022
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
January 2022
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[x] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@raych1 , Work item link for breaking change using the template -> https://msazure.visualstudio.com/One/_workitems/edit/13061339
This is assigned to our service PM
Justification in short -> Making one read-only field to writable.
This field is a Resource Group (RG) name which is created by the service. It was a GUID. Customer wants to pass the RG name as parameter. So it has been made to writable.

Talked with @sanray-ms offline, this PR has to be merged to generate manifest. So, ignore api readiness check.


Add card to update/create

Are this package support to add the card inside update/create page?

Unfortunately there is no possibility to add cards to the update and create forms. So we can't show the back button there.


Async support

As a concrete example of a function that should return Deferred.Or_error.t, look at reneg. I imagine others could benefit from this change as well.

Another piece of general feedback, there are a lot of places where assert false is being used, I believe, to just raise an error. See the TODO in reneg and the assert false in accept. These should all be replaced with informative error messages. If an assert false is used to indicate an impossible case, that's fine, but a comment as to why should also accompany it.

Another piece of general feedback, there are a lot of places where assert false is being used, I believe, to just raise an error. See the TODO in reneg and the assert false in accept. These should all be replaced with informative error messages. If an assert false is used to indicate an impossible case, that's fine, but a comment as to why should also accompany it.

I think, all serious feedbacks was done with the second pass (with more explicit commits). At the end, stylistic issue can be fixed and _3DES_EDE_CBC_SHA can be removed. I removed all assert false (noticed with a TODO). Most of them are replaced by an exception. On the LWT back-end, it's exactly the same - but, instead to have our own exceptions, LWT back-end raises Invalid_argument.

And if all are happy with this impl. it could be a gift for Christmas?


Sumstats not available in production for manually validated submission GCP001007 (66b7e7bce6802e0001eb6347)

The submission, which contains gene-burden studies, was manually validated. It generated study accession IDs, but studies were not found in production. The files are still found in Globus.
The files need to be copied to the production, and the file type should be non-gwas ssf.
Another submission (66b49220e6802e0001eb4602) had the same issue, and the files were manually copied to production.
Both submissions were validated on 15th Aug 2024.

Please let me know when this is done so I can email user.

For the submission 66b7e7bce6802e0001eb6347:

Created GCST ID folders in FTP staging
Copied studies from Globus to FTP staging

[ ] Should be available in public FTP in a few days (GCST90446417-GCST90446464)


Fixed invalid property value 'grid-columns'

See https://drafts.csswg.org/css-grid/#placement-shorthands

This change is also included in PR 242.


Makes ANCM an internal dependency

@natemcmaster Do I need to remove the submodule update to IISIntegration here?

Do I need to remove the submodule update to IISIntegration here?

Ideally yes since this is a "breaking" change in a way. The trick is that the commit for IISIntegration has to exist on github before you can you can update universe.

Once the build passes, on CI, update IISintegration's dev branch

cd modules/IISIntegration
git push origin dev:dev
# delete dev-gate
git push origin :dev-gate