Latest Contributions by NathanCastle1

When a graphic is added to a graphics overlay that is already in a mapview or sceneview, the graphic will be shown automatically, assuming there is a renderer or the graphic is configured with a symbol. Based on the code you shared, it looks like you're following the pattern set in the template projects and the developer tutorials. If that is the case, you will see something like the following in your MainWindow.xaml file: <Window.Resources> <local:MapViewModel x:Key="MapViewModel" /> </Window.Resources> That code will create an instance of your viewmodel when InitializeComponent is called. The MapView's Map and GraphicsOverlays properties are set via binding: <esri:MapView x:Name="MainMapView" Map="{Binding Map, Source={StaticResource MapViewModel}}" GraphicsOverlays="{Binding GraphicsOverlays, Source={StaticResource MapViewModel}}" /> The view is bound to the specific instance of the viewmodel that is added to the `Window.Resources` dictionary. Your code is creating another instance of viewmodel, but that instance is never connected to the view in any way, so the view won't update to reflect changes in viewmodel state. You could fix this by re-setting up bindings when you create the viewmodel, or by using the existing viewmodel instance created in XAML. I recommend updating your code to reference the copy of the ViewModel created in XAML: You could use something like the following to get a reference to the viewmodel: mvm = FindResource("MapViewModel") as MapViewModel;   ... View more

Hi, It looks like there are a few problems in the code. If you are using the default basemaps, the map's spatial reference will be WebMercator. Currently, the code is creating a new point with WebMercator coordinates while having a WGS84 spatial reference.   var mapPoint = new MapPoint(point.X, point.Y, SpatialReferences.Wgs84);   should change to    var mapPoint = new MapPoint(point.X, point.Y, point.SpatialReference);   But note that this step shouldn't be necessary at all. You could reference `point` directly without creating `mapPoint`. I also recommend a few other changes: Consider using `GeoViewTapped` instead of `MouseLeftButtonUp`, as that will save you from needing to call `ScreenToLocation` https://developers.arcgis.com/net/wpf/api-reference/html/E_Esri_ArcGISRuntime_UI_Controls_GeoView_GeoViewTapped.htm GeoViewInputEventArgs has a Location property. You should be able to add an overlay to the overlay collection rather than re-creating the collection each time. You can also add multiple graphics to a single overlay. If you just want to be able to draw graphics on the map, you should only need to set up the overlay once at startup. The ViewModel is being re-created every time the user clicks, and it isn't clear that the view is ever being re-configured to reference that ViewModel. The ViewModel would typically be created for the view only once. As written, I don't think you would ever see points, because a new ViewModel is created without the view being updated. Creating the symbol for the graphics each time is perfectly valid code, but there is a simpler way. You can define a renderer on the overlay and any graphics you add will be drawn with the defined symbol. There is an example here: https://github.com/Esri/arcgis-runtime-samples-dotnet/tree/main/src/WPF/ArcGISRuntime.WPF.Viewer/Samples/GraphicsOverlay/AddGraphicsRenderer  I hope that helps. Let me know if you have any other questions. ... View more