Address concerns w/compute_mit_user_fullname

The function compute_mit_user_fullname is not broken, per se, but it has some concerns that we should address:

It is a trap for test writers who might be unaware that it hits DNS.  (See #1472 for more context.)
We are trying to remove/isolate specific references to MIT in our code. (See #1334).
We might want to alter the production code to be more robust to DNS failures.

In my opinion the easiest initial task is to fix the tests that hit DNS to mock out either the Athena lookup or the entirety of compute_mit_user_fullname.  We already have some tests that directly test compute_mit_user_fullname, and they are already mocking DNS correctly.  Given that we have decent direct coverage on compute_mit_user_fullname, I think our strategy for all other tests that need computer_mit_user_fullname to mock the entirety of the function, rather than the fiddly DNS stuff inside of it.

I think now that we block network access in the unit tests, this has been fine for a few years, so I'm closing.


Poor default for api-path in check-graylog2-alive.rb

I discovered in  https://github.com/sensu-plugins/sensu-plugins-graylog/issues/8#issuecomment-366415650 that basically to make the check sucessful you had to change a default.
The option is defined here: https://github.com/sensu-plugins/sensu-plugins-graylog/blob/1.2.0/bin/check-graylog2-alive.rb#L65-L69
According to these docs: http://docs.graylog.org/en/2.4/pages/configuration/load_balancers.html#loadbalancer-integration
It should be:
GET /api/system/lbstatus

I can only find docs for versions 2.0 I think we should change this default in a major version.

I realized that it is not a breaking change as even if they already set it their result would take precedence

fixed via #11


Converting PyTorch Models to TensorFlow/TFLite with Training Support for Edge Devices

Description of the bug:
I am currently working on a project that involves converting a trained PyTorch model to LiteRT and deploying to edge devices. I need the LiteRT model to have training capabilities (for all trainable layers) with the addition of appropriate signatures.
I need some guidance regarding the process of converting a PyTorch model into a TensorFlow or LiteRT model using pytorch_converter, while retaining training capabilities for edge-based fine-tuning.
Does the converter support the creation of a trainable graph of tensorflow or LiteRT during conversion from a pytorch model?
Actual vs expected behavior:
No response
Any other information you'd like to share?
I already tried setting the pytorch model to .train() mode and converting it. But, even though few additional layers were added when compared to the one converted in .eval mode, the model generated is not actually trainable (frozen graph).

Hi @AdwaithAnand, I don't see documentation specific for this repo. We do have documentation using an older workflow: https://ai.google.dev/edge/litert/models/ondevice_training. I will log this as a feature request for now.


How i get the image name

How i get the image name after selecting the image. please help. Thanks in advance.

@riazurcse No you can't get the name for now.


DSEGOG-213 Adding @yarnpkg/sdks dependency for automatic SDKs installation

Description
I was getting annoyed at having to run an extra command to get set up with the required SDKs for developing OperationsGateway while using Yarn 3 with PnP. We've got this documented in the repo wiki, which requires the user to run something like
yarn dlx @yarnpkg/sdks vscode

if they want to be able to resolve external dependencies in VSCode, for example.
There is a solution to make this happen automatically and this PR provides it.
First, we can add @yarnpkg/sdks to our devDependencies. What this does is it means we now only need to run the command yarn sdks and yarn will automatically install the correct SDKs for our environment. Another advantage of this is that renovate can now keep our SDKs up-to-date (in theory).
Second, now that we have this more generic command, we can add this to the yarn postinstall command to have it automatically run whenever the user runs yarn install. More users are aware of and should know to run yarn install before running yarn start so this improves the process overall.
Testing instructions
The only way to properly test this is to clone a fresh copy of the repo in a brand new place on your computer. This eliminates the chance of any up-to-date lingering SDKs or Typescript libraries being resolved to the new workspace. Now open the workspace in your IDE of choice. If you're like me, the workspace will resolve the global version of Typescript I have installed on my computer (4.6.4 when we want the latest 4.9.3) and you won't be able to resolve external dependencies in the IDE.
Now run yarn install. This should set up the workspace with its normal dependencies plus its additional SDK files. Following this command's execution, your IDE should now be able to resolve its external dependencies. If you're running VSCode, you should get a popup asking you to switch to the workspace version of Typescript (the newly installed 4.9.3), otherwise I believe this also happens automatically.

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage

Agile board tracking
Closes DSEGOG-213

I am still getting a slight side-effect of running the yarn sdks command. The newly generated files are marked as modified by git even though it sees no content changes. Minor annoyance but I'd like to resolve it.

Googled the issue and many people say this is my Windows environment changing the line endings on those newly generated files to CRLF. To avoid this, I've enforced LF line endings in a new .gitattributes file but this doesn't seem to have worked.
@louise-davies can you check you get a similar thing on your machine (YMMV since you're running WSL). If not, it's probably just a local config thing I have set and it's not an issue.

@sam-glendenning I can confirm that the overall process worked (i.e. cloned a new repo with this branch, opened VSCode in it, SDKs weren't working, ran yarn install and they work & VSCode prompted to use workspace VSCode version etc.). If I run yarn sdks I get no git changes (as expected in WSL). I'll test it in windows to see if your fix works on my machine

@sam-glendenning does adding a .editorconfig file with the following contents fix it for you?
[*]
end_of_line = LF
charset = utf-8

It fixed things for me - initially I ran yarn sdks on windows and git had the files as changed. When I added that file it fixed things - to the point even if I remove the file I get no changes upon running yarn sdks. I supposed VSCode changed its settings when it was there...

No. But since these problems appear to be local to me, I'm not too worried about it affecting others. Are you happy for this to go in?


3 less textures for ray caster

This PR eliminates the use of 3 textures in the ray caster.

Not needed with the new DVR implementation.


⚠ ClamAV has degraded performance

In 7237ff2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 817 ms

Resolved: ClamAV performance has improved in aeb2fff.


How is the colorbar to be interpreted?

It is not clear to me what exactly the values on the colorbar indicate. It seems obvious that the higher values indicate more salient regions and vice versa. In the heatmap below:

It is not clear how the value of 0.0016 is to be interpreted for example

We did not analyze the utility of the absolute values in the saliency maps. Technically, they reflect the average class probability when the region is preserved. I'm not sure if they are normalized in the implementation (normalization does not affect the relative saliency values of different regions.)

thanks for answering @eclique! I figured as much, I just wanted to get your confirmation. Will close the issue.


Rails/ActiveSupport 6 allowed

Coverage increased (+1.08%) to 99.055% when pulling 2293f2101ef6a7899e54882640fc206ab83b9b27 on szTheory:master into 3e3c75116891f17bc33da7c9ad7684cfeee34ca2 on adhearsion:master.

Thanks for the contribution @szTheory. Have you done any testing with Rails 6? We locked it to < 5.0.0 because that was the latest version we tested with. Can you comment on what testing you've done to ensure compatibility?

Ran the specs locally with bundle exec rake and they all passed. I'm also running it in an app and CI specs around TTS are still passing.

CI looks ok - failure is due to bundler no longer support Ruby < 2.3.0. I'm going to merge this. Thanks for your contribution!

@szTheory @bklang Head's up, this unfortunately got merged to master instead of the develop branch.  We may want to either remove from master, or merge master -> develop.

Thanks for catching that @sfgeorge !   I've cherry-picked the relevant commit back to develop, which I think will be the least obtrusive change.


Bugs (Ethereum): teth_master branch bugs

A few problems found so far (in testnet mode):

[x] 1. GET /ob/profile => where it should say "ETH" in the currencies array, it says ""

[ ] 2. POST /wallet/spend => attempting to spend ETH returns a 400 error

[ ] 3. GET /wallet/currencies => returns a 403 error

[ ] 4. GET /wallet/currencies/:currencyCode returns a 403 error

Split of into individual issues, so I'm closing.


Compatibility issues with optifine

Cannot load properly with optifine. The screen doesn't shows the OpenGL info correctly. Also, the game after 2 minutes just crashes.

The crashlog would be useful.
The OpenGL information being null is known and, as far as I know, there isn't much I can do about it because, from what I understand, OptiFine pulls that information earlier, leaving Macula with null.

Okay i already figured it out that the problem is STATUS_STACK_BUFFER_OVERRUN, and I did resolve it.


Update method is not updating submit button disabled state

Currently I have a set of radio buttons that trigger bootstraps collapse method for a div which contains a few text inputs.
The collapse is triggered with jQuery and the update is triggered on Bootstraps hidden.bs.collapse event:
$("#rentingInfo").on('hidden.bs.collapse shown.bs.collapse', function(){ 	$("#whereForm").validator('update'); });
The text inputs that are shown are required if they are visible. I have edited the validator selector to:
Validator.INPUT_SELECTOR = ':input:visible:not([type="hidden"], [type="submit"], [type="reset"], button)'
So what should happen, is the form must be valid if the text inputs are hidden (which they are by default)  and invalid if they are visible and empty.
As far as I can tell, the form does validate, however it does not toggle the submit buttons disabled state. IE: The submit button is enabled when the form is loaded as the text inputs are hidden by default. When I toggle the radio button, the text inputs are shown, but the button remains enabled, however, when I click on the submit button, it becomes disabled and highlights the fields as invalid. Now with the button in the disabled state, I toggle the radio button which hides the text fields, which makes the form valid again, but the submit button still remains disabled, however, when I click the disabled submit button, it enables it and submits the form sucessfully.

I hadn't considered this. I'll make a change to run toggleSubmit() when you call .validator('update').


Feature/better style props

This PR includes a few breaking changes to the API. This is the beginning of changes that will be published as v2.0

<BaseComponent/> API has been updated.

Removed the following props: panel, top, left, bottom
Added the following props:

baseType - choices are none (default), panel, button
sx - set of styles that are passed to the base container (this existed but is now passed consistently)
buttonOptions - set of options passed to button-style components

New <ButtonComponent> (baseType = 'button') provides and out-of-the-box UI option for button triggers

@tannerjt please still review this but I need to publish these changes to npm to move forward on a few projects. Just going to publish from this branch, not merge yet.

latest changes here remove any direct emotion-theming dependencies in favor of 100% theme-ui. Looks like we were using emotion-theming mainly to support the Storybook site. I found a storybook addon for theme-ui - only downside is it renders in the same panel as the map stuff rather than being registered to storybook's addons panel, but overall it should be a better move for the library.


Update fuuid, fuuid-doobie to 0.5.0

Updates

io.chrisdavenport:fuuid
io.chrisdavenport:fuuid-doobie

from 0.2.0 to 0.5.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.2.0).
You might want to review and update them manually.
build.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.chrisdavenport" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #139.


4.3.0

Install this RP:
npm i Garlic-Team/GCommands#dev

Okay, I go make final test

EmbedBuilder

why'd they remove structures doe

https://github.com/discordjs/discord.js/issues/6039#issuecomment-873621116


Changing Send signature: Adding Context return param

Send now returns a context, which each transport can populate with specific information. In the case of the HTTP transport, we are setting the StatusCode and Header of the responses for now.
Updated UTs.

ping

ping


Update workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (dae5646) to head (9a96a6d).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #89   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          368       368
  Branches        44        44
=========================================
  Hits           368       368

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update to fix minor directory/screenshot inconsistency

I think I will create a pull req from my fork since I changed a screenshot file. I don't really know how git hub works yet :-)


phILR transformation

We should consider adding phILR transformation to mia
This is phylogeny-aware transformation that additionally requires a tree. It has been successfully used in some recent studies.

Opened an issue in the philr package

Now philr supports TreeSE.
This example shows how to run it. Note that the implementation in the philr package returns a matrix of balances.
Suggestion: create a mia function that integrates philr function as one of the transformations in transformSamples (and transformCounts). This works differently from the other transformations, however, because the dimension of the returned feature matrix is d-1. Hence it cannot be integrated as a new assay (which always needs the same dimensionality than the input data). Should the returned phILR balance matrix then be integrated as altExp in TreeSE? Main downside of that is that it is potentially confusing if some transformations are available as assay and others as altExp.
p.s. Moving the discussion on ALR and ILR into a separate issue #128
## Prepare example data
library(mia)
library(tidyr)
data(GlobalPatterns, package="mia")

## Select prevalent taxa
tse <-  GlobalPatterns %>% subsetByPrevalentTaxa(
                                    detection = 3,
                                    prevalence = 20/100,
                                    as_relative = FALSE)

## Pick taxa that have notable abundance variation across samples
variable.taxa <- apply(assay(tse, "counts"), 1, function(x) sd(x)/mean(x) > 3.0)
tse <- tse[variable.taxa,]

# Collapse the tree
tree <- ape::keep.tip(phy = rowTree(tse), tip = rowLinks(tse)$nodeNum)
rowTree(tse) <- tree

## Add a new assay with a pseudocount
assays(tse)$counts.shifted <- assay(tse, "counts") + 1

## Run philr for TreeSummarizedExperiment object
## using the pseudocount data. This returns a matrix of balances:
res.tse <- philr(tse, part.weights='enorm.x.gm.counts',
                    ilr.weights='blw.sqrt', return.all=FALSE,
                    abund_values="counts.shifted")

Now philr (1.19.1 in Bioc devel) supports TreeSE.

This means it accepts TSE as inputs, but it doesn't return a modified/enhanced object, does it?

Note that the implementation in the philr package returns a matrix of balances, with d-1 features and hence it cannot be added as assay

Which feature is dropped? Always the first? always the last? A selected reference?
If this is non random, than I would suggest adding an NA row or a row with zeros (depends on the meaning) would be the most straightforward answer.
I think it would be good to discuss any "underlying" issue, e.g. meaning of the returned data and how it is used further, before jumping to conclusions where to put it.

Yes, TreeSE is accepted as input but not provided as output in philr.
We cannot really say that a specific feature would be dropped in phILR transformation. In fact the returned features (balances) correspond to the tree nodes i.e. there are as many features as there are nodes. My initial statement of d-1 features was misleading, we just seem to have this case in our demo data sets but in general I think the number of balances could be different.
According to the philr vignette:

For a given set of samples consisting of measurements of taxa, we transform data into a new space of samples and orthonormal coordinates termed ‘balances’. Each balance is associated with a single internal node of a phylogenetic tree with the taxa as leaves.

The phILR transformation can be used for subsequent statistical analyses and visualizations as done for instance in the links above. Some of these are generic, for instance PCA / PCoA could be done in the phILR space; some are more specific, the phILR can be complemented with some dedicated statistical tests and visualizations that have been designed to deal with balances.
This method can be useful and in fact it exploits the capabilities of TreeSE container more than the already implemented transformations that do not utilize phylogenetic tree info. It would seem logical that we like to support relevant tree-based approaches in the TreeSE ecosystem.
It seems that the usual assayapproach does not work here. Because this is, however, a transformation of original assay, it would seem more logical to place it in altExp than in MAE.

I read up on this on the philr repo. I guess this can be closed. Please reopen, if needed

I read up on this on the philr repo. I guess this can be closed. Please reopen, if needed

I would like to suggest to create a wrapper in mia.
But that would introduce a new dependency (philr), and the use would be limited to a single transformation.
Hence, perhaps a better solution is to add an example to transformSamples on how to add transformations that are not readily provided by mia. I will prepare a PR.

I would like to suggest to create a wrapper in mia.
But that would introduce a new dependency (philr), and the use would be limited to a single transformation.
Hence, perhaps a better solution is to add an example to transformSamples on how to add transformations that are not readily provided by mia. I will prepare a PR.

PR #197

PR #197


Break up searchRows into more reasonable parts

Before this change, searchRows was doing far too much - so I broke it up into 3 reasonable parts.

getUnfilteredRows - selects the next page from cassandra
getRowsForDocument - selects a page of rows for a particular document
getCandidateRows - selects rows for the "candidacy filtering" algorithm we use for evaluating search filters with AND

The common parts of these 3 functions were also refactored (most of them have to do with converting data into C* predicates)

Hi @EricBorczuk, I would like to suggest some changes that could be part of this PR.

Rename the addRowsToMap method to groupRowsByKey or groupByKey;

The following lines in updateExistenceForMap can be changed to ``groupByKey(documentChunks, rows)`:

    for (int i = 0; i < rows.size(); i++) {
      Row row = rows.get(i);
      String key = row.getString("key");
      List<Row> chunk = documentChunks.getOrDefault(key, new ArrayList<>());
      chunk.add(row);
      documentChunks.put(key, chunk);
    }

I'm actually working on a parallel refactoring under #828 that touches a lot of the code refactored here... it might be worth syncing up on this offline @EricBorczuk :)

#937 removed searchRows, closing this


Material UI v5.0.0-alpha.14 documentation page

is there a way I can check updated documentation based on git tag like https://v5.0.0-alpha.14.material-ui.com/ if not can you please publish it so that we can go through the documentation and see the changes.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

https://next.material-ui.com/


Editor crashes when generating ArrayMesh

Godot version:

Godot 3.1 custom build with hash 9eb4b6d
OS/device including version:
Windows 10 (64 bits) AMD Radeon R5 M330
Issue description:
I have a scene with a script that works with arraymesh when it ends generating the array I get this crash dump:
ERROR: CowData::get: FATAL: Index p_index=0 out of size (size()=0)
At: C:\Users\Home\Documents\TRABAJO-LEO\Godot Game Engine\godot\core/cowdata.h:149
CrashHandlerException: Program crashed
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[0] CowData::get (c:\users\home\documents\trabajo-leo\godot game engine\godot\core\cowdata.h:149)
[1] Vector::operator[] (c:\users\home\documents\trabajo-leo\godot game engine\godot\core\vector.h:89)
[2] RasterizerSceneGLES3::_fill_render_list (c:\users\home\documents\trabajo-leo\godot game engine\godot\drivers\gles3\rasterizer_scene_gles3.cpp:3155)
[3] RasterizerSceneGLES3::render_scene (c:\users\home\documents\trabajo-leo\godot game engine\godot\drivers\gles3\rasterizer_scene_gles3.cpp:4131)
[4] VisualServerScene::_render_scene (c:\users\home\documents\trabajo-leo\godot game engine\godot\servers\visual\visual_server_scene.cpp:2140)
[5] VisualServerScene::render_camera (c:\users\home\documents\trabajo-leo\godot game engine\godot\servers\visual\visual_server_scene.cpp:1714)
[6] VisualServerViewport::_draw_viewport (c:\users\home\documents\trabajo-leo\godot game engine\godot\servers\visual\visual_server_viewport.cpp:70)
[7] VisualServerViewport::draw_viewports (c:\users\home\documents\trabajo-leo\godot game engine\godot\servers\visual\visual_server_viewport.cpp:308)
[8] VisualServerRaster::draw (c:\users\home\documents\trabajo-leo\godot game engine\godot\servers\visual\visual_server_raster.cpp:107)
[9] VisualServerWrapMT::draw (c:\users\home\documents\trabajo-leo\godot game engine\godot\servers\visual\visual_server_wrap_mt.cpp:104)
[10] Main::iteration (c:\users\home\documents\trabajo-leo\godot game engine\godot\main\main.cpp:1867)
[11] OS_Windows::run (c:\users\home\documents\trabajo-leo\godot game engine\godot\platform\windows\os_windows.cpp:2736)
[12] widechar_main (c:\users\home\documents\trabajo-leo\godot game engine\godot\platform\windows\godot_win.cpp:150)
[13] _main (c:\users\home\documents\trabajo-leo\godot game engine\godot\platform\windows\godot_win.cpp:172)
[14] main (c:\users\home\documents\trabajo-leo\godot game engine\godot\platform\windows\godot_win.cpp:184)
[15] __scrt_common_main_seh (f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl:288)
[16] BaseThreadInitThunk
-- END OF BACKTRACE --
Steps to reproduce:

Open the only scene existing
add the custom node I created "TerrainPrimitive"
select noise texture
select simplex noise
you should see the editor crash with the same dump

Minimal reproduction project:
GodotTerrainPrimitive.zip
I have looked for similar issues, and found the following (stack trace is similar):
#21892
I might be wrong but, to me it seems like the problem is here:
drivers\gles3\rasterizer_scene_gles3.cpp:3155
This part of the code:
int ssize = mesh->surfaces.size();

				for (int i = 0; i < ssize; i++) {

					int mat_idx = inst->materials[i].is_valid() ? i : -1;
					RasterizerStorageGLES3::Surface *s = mesh->surfaces[i];
					_add_geometry(s, inst, NULL, mat_idx, p_depth_pass, p_shadow_pass);
				}

It cycles through surfaces but it uses the index of the surface as the material index (but, is there always a material set into a mesh?), and as I place a material override then I managed to avoid the error.

Dang, I can't reproduce anymore the error..

Hey,
I did receive this email about your patch, but I can’t see it on the thread, do you know what happened?
From: bithunter32
Sent: viernes, 23 de noviembre de 2018 18:25
To: godotengine/godot
Cc: Toshio Araki; Author
Subject: Re: [godotengine/godot] Editor crashes when generating ArrayMesh(#23790)
Can confirm this on a fresh linux build. Surface without material leads to out of bounds index. Attached patch.
0002-Fix-out-of-bounds-access-for-surfaces-without-materi.patch.txt
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Questa e-mail è stata controllata per individuare virus con Avast antivirus.
https://www.avast.com/antivirus

I found this thread while looking up a similar crash. It does not happen during the creation of an ArrayMesh, but the error message seems very similar:
E 0:01:01:0661   FATAL: Index p_index=2 out of size (((Vector<T> *)(this))->_cowdata.size()=0)
  <C Source>     ./core/vector.h:49 @ operator[]()

I'm still trying to trace back where exactly this is happening, but it seems to be related to setting shader parameters in threads.

@kb173 it crashes because the array of materials on the meshinstance lags behind the number of surfaces you add. When you add a surface to the arraymesh it needs to be mirrored in the number of material overrides you have on the meshinstance. That is handled by signals but for some reason the signal is processed too later. When the meshinstance gets rendered without the material overrides being in place it crashes.
This only happens when the mesh instance and array mesh are created first, and the surface is added later. If everything is in place before the scene gets rendered it all works ok.
I've been trying to get @reduz to try out the example scene I added a few posts up as I'm not sure how to solve the delayed signal processing.

@BastiaanOlij Thank you for the clarification! Interestingly, the p_index the Error spits out is different depending on where the call to set_shader_param is placed: In my own (GDScript) thread, it's 2, whereas if it happens in the main thread (in _process) right after my thread is done, it's 3.
I hope this is helpful information, I haven't dived too much into the engine code.

@reduz nice one!

Unfortunately, I'm still getting the error sometimes (FATAL: Index p_index=0 out of size (((Vector<T> *)(this))->_cowdata.size()=0)... I will try to provide more detail once I found where exactly it's coming from.

@kb173 you should tey with beta 9 and the project Bastiaan has created, above:
https://github.com/godotengine/godot/issues/23790#issuecomment-457566379

@Toshiwoz That project works fine for me with beta 9, even with multithreading. However, my example project in #24471 still causes crashes, so it might be related to that? I only get the error when using multiple threads.


Java 8 Compatibility

Hi,
This driver does not appear to work with Java 8 with the standard configuration.  If the driver requires additional steps to fully work in Java 8, please include them in the ReadMe.

You'll have to be more clear.  It has worked fine for me in J8.

Thanks, it may be my team's JVM configuration causing the issue then.

I assume whatever J8 issue you have has been resolved.


Update LICENSE copyright date

Updating the copyright date from 2022 to 2023.
Github sees a change in the last paragraph, I dont know why.

There is a lot of license files with "2022" instead of "2023".
@Trafterian @walteratwork
Minor issue but just informing.
Good luck with the development guys, we are all waiting patiently.


Again problems with CSOM SendEmail

Category

[x] Question

Question
We are again seeing problems when using
Microsoft.SharePoint.Client.Utilities.Utility.SendEmail(context, properties);
through CSOM.
Most of the time mails sent this way get delivered, sometimes they don't. We are always sending mails to recipients in the same tenant.
In the past this method threw errors (see #4929) but since several weeks no more errors are thrown. Sending the mails just fails silently so that our retry logic doesn't get triggered.
Yesterday I sent 5 mails with nearly the same content in a 2 minute timeframe to the same recipient. 3 got delivered, 2 are still missing.
Most likely this is not a problem in the SendEmail method but again some type of misconfiguration in the underlying mail server infrastructure. But how could we report that to Microsoft? Support just tells us: This is coming from code so it's a developer issue.
Environment details (if applicable)

Your Developer Environment: Windows 10
Target Environment: SharePoint Online
Framework: .NET Framework v4.7
Browser(s): All
Tooling: Visual Studio 2019

@cwdata Did you ever resolve this issue? I have encountered it myself recently and having issues in finding a way to resolve

@Siglid,
no, unfortunately there's nothing we can do from our end to resolve this. We switched to Graph for sending mails, which incurs a lot of additional configuration. But beware, Graph is also not a guarantee that your mails get delivered. We encountered the exact same issue there too, but only for a couple of days in several months. I think, if more and more people will go that route we'll see things getting worse on the Graph side too.


fix: resolve issues with the staking UI implementation

Improve naming as part of handling some of the comments I (@crypto-engineer) left from https://github.com/interlay/interbtc-ui/pull/184.
Correct the validation message.
Before:

After:

Introduce TRANSACTION_FEE_AMOUNT as a constant.
Improve availableBalance calculation.

@nud3l
Done! Could you please review it again?

Looks like the validation is not working as intended. It's actually not substracting the amount correctly.

From the validation perspective, I think it's working as expected, but the substracted transaction fee was too small because we had not added the third argument.


Help option for commands

Sorry if this is already covered somewhere, I couldn’t find it if so.  Would it be feasible to add a —help or similar option to each individual command?  There is some really great documentation for each command, it would be nice to be able to access these (or similar) descriptions while using the tool so I don’t have to keep the website open for searching.
Example of what I’d like to see:
$ dephell deps convert —help

Convert dependencies between formats.
Options:
—from : Format or path to convert from
—to: Format or path to convert to
—from-format: Format to convert from
..etc

Available formats:
- pip
- poetry
- pipenv

Obviously that’s a lot to add for every command, so maybe the goal should be to generate these from the docs?  Or generate part of the docs from the descriptions?  Just to make it easier to maintain.  An easier solution would just be to display the docstring which has the link to the full docs.
I think the way to do this with ArgParser is to use parent parsers so each sub command gets its own -h/—help option.

An easier solution would just be to display the docstring which has the link to the full docs.

This is exactly how it works now :) Just docstrings contain only short description and link on the docs.
❯deps convert --help
usage: dephell deps convert [-h] [-c CONFIG] [-e ENV] [--from FROM]
...

Convert dependencies between formats.
https://dephell.readthedocs.io/en/latest/cmd-deps-convert.html

optional arguments:
  -h, --help            show this help message and exit
...

However, I understand your point. It would be nice to provide some quick examples for each command here.

Huh, I could have sworn I tried that and didn’t see all the helpful help content.  Maybe I’d typed something wrong because of course now I see it just fine :P.  Well it seems yet again you’ve thought of everything!

Maybe, we have to simplify "usage" string :thinking:

Maybe, there is a lot there.  Could try to decide on what the most basic case is for each command and just show that in usage.  In the case of convert that'd be something like dephell deps convert --from FROM --to TO right?
Another thing that I think would help is to modify the message which prints out when you type something wrong.  Right now it shows all the possible commands in an unformatted blob, I think it's just the default behavior of argparse when it sees an unknown command.  Would be better to show the default help message (dephell --help), though I don't know how to override the argparse behavior.  Maybe using parse_known_args instead of parse_args in cli.py?
I haven't done much experimenting, just spit balling while reading through pieces of the code :).

I've split this conversation by 3 different issues to make the context more clear.


Disable Pin.InfoWindow completly

SUMMARY
Some apps have their own "InfoWindows" and show them in a separate control nearby the map.
It would be nice to have a way to disable InfoWindow altogether. So it will not appear when the marker is clicked.
Map.DisableInfoWindow = true;
// or
Pin..DisableInfoWindow = true;

PLATFORMS

[x] Android
[x] iOS
[ ] UWP

Looks like there is a workaround. You need to set Pin.Label to string.Empty and InfoWindow will be disabled.


add the usual github workflows

I guess you haven't used PkgTemplates, which I'd use for every new Julia package, to setup CI etc correctly:
using PkgTemplates
t = Template(;
    dir=joinpath(@__DIR__, "dev"),
    user="SimonDanisch",
    authors=["Simon Danisch"],
    plugins=[
        Git(; manifest=true, ssh=true),
        Codecov(),
        GitHubActions(),
        Documenter{GitHubActions}(),
    ],
)
# Creates the package MathTexEngine at
# ./dev/MathTeXEngine
t("MathTeXEngine")

I kind of was lazy on that front (and a bit afraid to see the test coverage ^^'). Thanks a lot!


20230110 Generalize the 'crystal' tab in the 'laser' app (aka 'silas')

The requested time scale is 4 to 6 weeks.
This is a follow-on to issue #91
We want users to be able to switch from linear simulations (what is presently implemented in the Sirepo app) to nonlinear simulations.
The workflow for nonlinear simulations is captured in this notebook,
https://github.com/radiasoft/rslaser/blob/master/examples/thermal/NLThermalSaturation.ipynb

After discussion with @moellep and @k-wolfinger, we decided that this tab is good enough as is and should be further improved in the future (if funding is available) in response to user feedback.


⚠ Hartman Foundation has degraded performance

In 8422e8b, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 2643 ms

Resolved: Hartman Foundation performance has improved in 846aa2b.


Fixes Devouring Already Nested Hosts

About the PR
Made is so you can't devour currently nested marines.
Why / Balance
Fixes a bug that allowed you to devour already nested marines, which would keep them in a nested state.
Media
https://github.com/user-attachments/assets/e829da66-91d0-4bb6-90ff-8eb4106d482e
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

[x] By submitting this code and/or assets, I confirm that I either own them or have provided the correct necessary licenses to use and distribute them. I agree to be fully responsible for any legal claims or issues arising from the use of these materials.

Changelog
:cl:

fix: You can no longer devour currently nested hosts.

guh, forgot to make a new branch


suffix with json not work, with add_swagger_documentation default option.

in project grape-swagger-0.10.1/example folder:
$ ls
api.rb      config.ru   splines.png

$ rackup config.ru
/Users/tangmonk/.rvm/gems/ruby-2.1.5/gems/grape-swagger-0.10.1/example/api.rb:4: warning: class variable access from toplevel
Thin web server (v1.6.3 codename Protein Powder)
Maximum connections set to 1024
Listening on localhost:9292, CTRL+C to stop

$ curl http://localhost:9292/swagger_doc/swagger_doc.json
Not Found

$ curl http://localhost:9292/swagger_doc/swagger_doc

{"apiVersion":"0.1","swaggerVersion":"1.2","resourcePath":"/swagger_doc","produces":["application/json"],"apis":[{"path":"/swagger_doc","operations":[{"notes":"","summary":"Swagger compatible API description","nickname":"GET-swagger_doc","method":"GET","parameters":[],"type":"void"}]},{"path":"/swagger_doc/{name}","operations":[{"notes":"","summary":"Swagger compatible API description for specific API","nickname":"GET-swagger_doc--name","method":"GET","parameters":[{"paramType":"path","name":"name","description":"Resource name of mounted API","type":"string","required":true,"allowMultiple":false}],"type":"void"}]}],"basePath":"http://localhost:9292"}%

If I add theformat: :json to add_swagger_documentation it will works with swagger-ui.

This is by design, but has been causing a lot of issues, see https://github.com/intridea/grape/issues/914. Can you please PR the fix for this repo for the demo?

solved in the actual version


[WIP] feat(server): support ssr running in vm

Summary

🤖 Generated by Copilot at 1d351d4
Refactored and improved server-side rendering logic for prod-server. Extracted rendering code to runServerRender function in libs/render/ssr/utils.ts and added VM mode support for better isolation and security. Moved utils.ts to ssr folder for better organization.
Details

🤖 Generated by Copilot at 1d351d4

Simplify and enable VM mode for server-side rendering logic (link, link, link, link)
Remove unused SERVER_RENDER_FUNCTION_NAME import from ssr.ts and add runServerRender import from utils.ts (link)
Replace bundleJS require and serverRender call with runServerRender function in render function in ssr.ts (link)
Move utils.ts file from libs/render to libs/render/ssr and add or remove imports accordingly (link)
Add runServerRender function to utils.ts to handle the server-side rendering logic with or without VM mode (link)

Related Issue

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Do you mean a natural raw JavaScript Context without no API support by "VM"?

Closed as the PR is outdated, feel free to reopen it if it is still needed.


Uncaught Error: Must be used within Provider when using an external provider such as wagmi

@AngelofPc can you share how you're passing the external provider? here's a working example in our portal: https://portal.thirdweb.com/sdk/set-up-the-sdk/frontend#from-a-signer--provider

Here @joaquim-verges

@AngelofPc looking at your stack trace, you're using the ConnectWallet component. That's only usable with the ThirdwebProvider which manages the wallet connection for you.
If you want to pass your own wagmi signer/provider using the ThirdwebSDKProvider, then you need to implement the wallet connection logic some other way. Here's an example using rainbowkit https://blog.avneesh.tech/how-to-use-rainbowkit-with-thirdweb

Tried it with Connectkit and Rainbow Wallet before switching things up, But will try this and let you know. @joaquim-verges

Not working, same as before.
I think it's the now deprecated "useNftCollection" causing it.
I see that we are supposed to now use useContract() but there is no documentation for it or how to do a signature mint with it? @joaquim-verges

@AngelofPc but are you still using the ConnectWallet component from @thirdweb-dev/react in your code?
Btw curious why you need to pass the provider externally? Wonder if you can just use the normal thirdweb react setup to achieve what you want.

No, i'm no longer using the ConnectWallet component.
Want to pass the use either ConnectKit or RainbowKit because of their UI especially.
Worked better before @thirdweb-dev/react 3.0.0 though.
@joaquim-verges

Got it. I think i see what the problem is. We'll get that fixed this week.
Thanks for reporting this 🙏
cc @jnsdls

Thanks @joaquim-verges, Really appreciate.
Could you please let me know when it's done?
I'm present in the discord as well if it's easier.

@AngelofPc can you try this build and let me know if everything works for you?
npm install<EMAIL_ADDRESS><EMAIL_ADDRESS>

Works so far, thanks @joaquim-verges.
However, I noticed DevTools getting disconnected every time since i installed the update

Also, can you help with your email or twitter handle, I'd like your answer/feedback on something.
Thanks

I still have the error Must be within Provider, I don't understand why my _app is pretty simple :
<WagmiConfig client={wagmiClient}>
        <SessionProvider refetchInterval={0} session={pageProps.session}>
          <RainbowKitSiweNextAuthProviderCustom>
            <RainbowKitProvider chains={chains}>
              <ThirdwebProvider wagmiClient={wagmiClient}>
                <AnimatePresence exitBeforeEnter>
                  {getLayout(<Component {...pageProps} />)}
                </AnimatePresence>
              </ThirdwebProvider>
            </RainbowKitProvider>
          </RainbowKitSiweNextAuthProviderCustom>
        </SessionProvider>
      </WagmiConfig>

Here is the trace. @joaquim-verges any idea?

@Sense thanks for reporting this.
What's inside your Claim component? Looks like the error is coming from there.
In general the thing to keep in mind when using an external wallet connector is that you can't use the thirdweb hooks related to wallet connection (like useAddress, useChainId etc) - those require the standard ThirdwebProvider since that's what would normally handle the wallet connection internally.
Instead for these wallet related hooks, you should use the ones from the library you're using (wagmi in your case).
All the contract related hooks should be usable though, that's what the dev build fixes (will ship it to stable today)

Thanks @joaquim-verges my Claim page looks like :
const Claim = dynamic(() => import('../../components/Claim'), {
  ssr: false
})

const NftClaimPage = () => {
  return (
    <div className="flex flex-col">
      <Claim contractAddress={process.env.editionDropAddress} />
    </div>
  )
}

The thing that Thirdweb hooks are working great using the v2 of @thirdweb-dev/react and @thirdweb-dev/sdk so it's probably related to the v3 ? There is an opened thread on it on [this Discord thread].
(https://discord.com/channels/834227967404146718/1020660790296072242)
I made it work with the magicLinkWalletConnector as an external connector, but still with the v2.

@Sense and it still happens with the latest releases? (v3.0.4)
we might have missed another hook that uses the wallet connection hooks internally. The stacktrace shows that something inside that /components/Claim is causing the issue. Likely one of our hooks. If you can help me pin point which one I can get a fix out very quickly.

@Sense also - unrelated, but if you want to ping me on discord about that Disable SSR as thirdweb SDK only working on browser - i'm curious why you think that. We use our SDK in SSR on our dashboard so it's definitely possible!

Let's continue the discussion on Discord then @joaquim-verges, thank you.


How to sign out?

I can only sign out by clearing the browsing data. Is there any other way?

The same question, is there any way to do it elegantly?


refactor: Remove options used solely by the connector

Options not recognized recognized by the base Camel component cannot be specified on the endpoint URI, this removes those options that are used only by the connector.

@zregvart  I'm assuming they are removed /after/ being used to potentially use them to contract another parameter like the connection url:
https://github.com/syndesisio/connectors/blob/master/connectors/sql-stored-connector/src/main/java/io/syndesis/connector/springboot/SqlStoredConnectorConnectorAutoConfiguration.java#L105

Yeah, so this is a Camel only thing, the *AutoConfiguration thing is Spring's and one does not affect the other.
And mind you, this is just a quick & dirty fix, I think proper fix might be to use those properties in the Connector and create the DataSource there, and leave the *AutoConfiguration classes to Camel Maven plugin to manage.

Yeah, and the tests passed as we don't set those options on the endpoint parameters (https://github.com/syndesisio/connectors/blob/master/connectors/sql-stored-connector/src/test/java/io/syndesis/connector/SqlStoredConnectorComponentTest.java#L78), only the template.

The connector generates the following meta-data about the endpoint options (sql-stored-connector.json) file
  "properties": {
    "procedureName": { "kind": "path", "displayName": "Template", "group": "producer", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Sets the StoredProcedure name to perform" },
    "template": { "kind": "path", "displayName": "Template", "group": "producer", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Sets the StoredProcedure template to perform" },
    "batch": { "kind": "parameter", "displayName": "Batch", "group": "producer", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "description": "Enables or disables batch mode" },
    "noop": { "kind": "parameter", "displayName": "Noop", "group": "producer", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "description": "If set will ignore the results of the template and use the existing IN message as the OUT message for the continuation of processing" }
  }

The properties are the endpoint options (they are named properties when it was first created and hence we are stuck with that name, instead of endpointProperties or endpointOptions. As you can see those options you attempt to remove are not listed there.
However in the same file, you have the component options listed as
  "componentProperties": {
    "url": { "kind": "property", "displayName": "Connection URL", "group": "producer", "label": "common", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "defaultValue": "jdbc:postgresql:database", "description": "JDBC URL of the database" },
    "user": { "kind": "property", "displayName": "Username", "group": "security", "label": "common,security", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "defaultValue": "", "description": "Username for the database connection" },
    "password": { "kind": "property", "displayName": "Password", "group": "security", "label": "common,security", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": true, "description": "Password for the database connection" },
    "schema": { "kind": "property", "displayName": "Schema", "group": "producer", "label": "common", "required": false, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Database schema" },
    "catalog": { "kind": "property", "displayName": "Catalog", "group": "producer", "label": "common", "required": false, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Database catalog" }
  },

@davsclaus what are you trying to say? Did we mislabel the properties?

Okay I look at the source code, and there is a problem that the component class does not have getter/setter for the 4 options it has defined in the camel-connector.json
   "componentOptions" : [ "user", "password", "url", "schema", "catalog" ],

But instead a hack has been added to the SqlStoredConnectorConnectorAutoConfiguration class where you read those configurations and create the datasource in some post constructor code.
So you added code to a file that says Generated by camel-connector-maven-plugin - do not edit this file!
Down the road I dont think this is advised.

@KurtStam @zregvart look in the target folder what the JAR includes.
Look at

target/classes/camel-connector.json
target/classes/camel-connector-scham.json
target/classes/io/syndesis/connector/sql-stored-connector.json

I assume camel-connector-schema.json is what Sydesis UI is using to know what a connector can do
{
  "component": {
    "kind": "component",
    "baseScheme": "sql-stored",
    "scheme": "sql-stored-connector",
    "syntax": "sql-stored-connector:template",
    "title": "SqlStoredConnector",
    "description": "SQL Stored Procedure Connector to invoke a SQL Stored Procedure",
    "label": "sql-stored",
    "deprecated": false,
    "async": false,
    "producerOnly": true,
    "lenientProperties": false,
    "javaType": "io.syndesis.connector.SqlStoredConnectorComponent",
    "groupId": "io.syndesis",
    "artifactId": "sql-stored-connector",
    "version": "0.4-SNAPSHOT"
  },
  "componentProperties": {
    "url": { "kind": "property", "displayName": "Connection URL", "group": "producer", "label": "common", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "defaultValue": "jdbc:postgresql:database", "description": "JDBC URL of the database" },
    "user": { "kind": "property", "displayName": "Username", "group": "security", "label": "common,security", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "defaultValue": "", "description": "Username for the database connection" },
    "password": { "kind": "property", "displayName": "Password", "group": "security", "label": "common,security", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": true, "description": "Password for the database connection" },
    "schema": { "kind": "property", "displayName": "Schema", "group": "producer", "label": "common", "required": false, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Database schema" },
    "catalog": { "kind": "property", "displayName": "Catalog", "group": "producer", "label": "common", "required": false, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Database catalog" }
  },
  "properties": {
    "procedureName": { "kind": "path", "displayName": "Template", "group": "producer", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Sets the StoredProcedure name to perform" },
    "template": { "kind": "path", "displayName": "Template", "group": "producer", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Sets the StoredProcedure template to perform" },
    "batch": { "kind": "parameter", "displayName": "Batch", "group": "producer", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "description": "Enables or disables batch mode" },
    "noop": { "kind": "parameter", "displayName": "Noop", "group": "producer", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "description": "If set will ignore the results of the template and use the existing IN message as the OUT message for the continuation of processing" }
  }
}

componentProperties = options you can configure on the component level, or via spring boot auto configuration, eg application.properties
properties = options you can configure in endpoint uri (yeah it should have been named endpointProperties, but we are stuck with this name).

Sometimes properties can be configured on both component and endpoint level and hence why they may appear in both of them. Other times you have exclusive properties in either of them.
For example this connector is such a case, where properties are either only on the component level or the endpoint level.

I a Camel connector you can specify which properties you want included, to filter out unwanted properties from the base Camel component (eg sql-component).
You do this in the camel-connector.json where you have done
  "componentOptions" : [ "user", "password", "url", "schema", "catalog" ],
  "endpointOptions" : [ "procedureName", "template", "batch", "noop" ],

PS: I do not know what catalog is used for, it does not look like its in use.

This PR does not make sense as the options are NOT on the endpoint level at all from the beginning.
If they are in use somehow, then its a bug in Syndesis where it provides those invalid options to Camel. The only endpoint options allowed are the ones in the properties section in the camel-connector-schema.json file, which are these:
 "properties": {
    "procedureName": { "kind": "path", "displayName": "Template", "group": "producer", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Sets the StoredProcedure name to perform" },
    "template": { "kind": "path", "displayName": "Template", "group": "producer", "required": true, "type": "string", "javaType": "java.lang.String", "deprecated": false, "secret": false, "description": "Sets the StoredProcedure template to perform" },
    "batch": { "kind": "parameter", "displayName": "Batch", "group": "producer", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "description": "Enables or disables batch mode" },
    "noop": { "kind": "parameter", "displayName": "Noop", "group": "producer", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "description": "If set will ignore the results of the template and use the existing IN message as the OUT message for the continuation of processing" }
  }

So I dont understand the need for this PR in the first place, unless its to workaround a bug in Syndesis, such as if it provides invalid endpoint options

So I dont understand the need for this PR in the first place, unless its to workaround a bug in Syndesis, such as if it provides invalid endpoint options

Yeah, we workaround to get the UX we need, we can't expect the user to provide a DataSource reference or the template parameter, which is something that the base SQL stored procedure component expects.
I think we might be able to work this out by adding additional logic to the connector component implementation by creating a DataSource and template parameters there.
The issue this PR is trying to address is that those properties that we use to synthesize DataSource and template parameters get added by the runtime to endpoint URI as parameters, and rightfully Camel complains about that.
Perhaps we can merge this and log an issue to see if we could add the properties needed for the creation of DataSource and template parameters in the connector component?

@zregvart yeah sure the PR does no harm.
And +1 to log a new ticket about adding those options to create the DataSource/template stuff more correctly.

@lburgazzoli has been working on a feature so we can add 'custom' properties to a syndesis connector. This work was done before this feature was ready. We maybe able to upgrade so we are using it. In the meantime the maven plugins in the pom where commented out.


Alt-W while editing a path freezes

Normally Alt-W hides the features to let you see the satellite imagery. But if you happen to be editing a path when you press Alt-W, then boom, the page freezes and you have to reload and hope your changes were cached locally.

@ts-web This looks bad. Thanks for reporting.


[RELAY][AST] Add virtual device as a first class field to Relay expressions

This is a proposal to add virtual devices as a first class field to Relay expressions.
cc @mbs-octoml @mikepapadim @jroesch

cc @comaniac @zhiics @ZihengJiang

Here's my proposed implementation: https://github.com/apache/tvm/pull/9641/files

Thanks Lily, LGTM.
Let me apologize for getting us into the current on_device + function attributes mess your proposal gets us out of. It seemed like a workable approach at the time which would not require both an AST change and careful treatment of AST node copies. But in retrospect the on_device + function attributes approach has caused at least four bugs due to the fragility of maintaining its annotation invariant.
I'm hoping the experience we gain from adding virtual_device_ will inform improvements to checked_type_. In particular getting us out of the need to rerun InferType after every xform (and sometimes in strict succession). But one step at a time.

@comaniac Thanks for reviewing! Looking forward to it going in :D

Thanks @electriclilies and all reviewers. This RFC is merged.

Thanks cody!


Referrer and Referrer Policy

What is the purpose of this pull request?

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:

What changes did you make? (provide an overview)
This adds support for the referrer and referrerPolicy options.
Which issue (if any) does this pull request address?
#1036 #839
Is there anything you'd like reviewers to know?
Specifications are referenced in-line.

@tinovyatkin You are quite right.  I've updated the TS types

@tinovyatkin You are quite right.  I've updated the TS types

Hi, is it just lack of reviews that stops this from getting merged?

@jimmywarting thanks!
@bitinn looks like this only awaits your review now, would be really nice to merge this.
As a side note: in Chrome dev tools, when one selects "copy as Node.js fetch", the request is copied with referrer and referrerPolicy options which doesn't work "as is" but will work once this gets merged.

@jimmywarting I had to resolve some merge conflicts and fix some errors, but this is ready to merge now.  To follow good form, would you mind giving it one more once-over and merge if it looks good?

looks good to me

Thanks for the amazing work!

Great! In which version can we expect the update?

Should be version 3.1.0, I don't think that there have been any talks about when to release it, but the sooner the better in my opinion 👍

Don't think we have any valuable new features to push out, it's mostly docs, deprecations and other minor things.
would wish for #1314 to be merged, after that i would like to release 3.1.0
it will come when it comes


add property for getting mailboxes quotas

In order to display used resource and set limitations to users, some server-managed quotas restrictions info have to be provided as part of mailboxes list.

resolved by https://github.com/linagora/jmap-client/pull/76


Not able to add custom dimensions to spanner metrics

Hello Team,
I'm creating java-spanner client and using the PrometheusHttpServer as the metrics collector.
        PROMETHEUS_SERVER = PrometheusHttpServer.builder().setPort(portNumber).build();

I want to add custom dimensions to SDK provided metrics and for that I'm using the Resource like below,
    SpannerOptions.enableOpenTelemetryMetrics();

    Resource resource = Resource.getDefault().merge(Resource.create(Attributes.of(
        SERVICE_NAME, MY_SERVICE_NAME,
        SERVICE_VERSION, "1.0.0"
    )));

    SdkMeterProvider sdkMeterProvider =
        SdkMeterProvider.builder()
       .setResource(resource)
       .registerMetricReader(PROMETHEUS_SERVER)
       .build();

    OpenTelemetry openTelemetry =
        OpenTelemetrySdk.builder().setMeterProvider(sdkMeterProvider).build();

    optionsBuilder.setOpenTelemetry(openTelemetry);

However, when I start my application, I'm not able to see any custom dimensions attached to the SDK metrics.
# HELP spanner_get_session_timeouts_ratio_total The number of get sessions timeouts due to pool exhaustion
# TYPE spanner_get_session_timeouts_ratio_total counter
spanner_get_session_timeouts_ratio_total{client_id="client-1",database="test-db",instance_id="test-projects",otel_scope_name="cloud.google.com/java"} 0.0
# HELP spanner_gfe_header_missing_count_ratio_total Number of RPC responses received without the server-timing header, most likely means that the RPC never reached Google's network
# TYPE spanner_gfe_header_missing_count_ratio_total counter
spanner_gfe_header_missing_count_ratio_total{database="test-db",instance_id="instance-id",method="google.spanner.v1.Spanner/BatchCreateSessions",otel_scope_name="cloud.google.com/java",project_id="test-id"} 0.0
# HELP spanner_max_allowed_sessions_ratio The maximum number of sessions allowed. Configurable by the user.
# TYPE spanner_max_allowed_sessions_ratio gauge
spanner_max_allowed_sessions_ratio{{client_id="client-1",database="test-db",instance_id="test-projects",otel_scope_name="cloud.google.com/java"} 401.0
# HELP spanner_max_in_use_sessions_ratio The maximum number of sessions in use during the last 10 minute interval.
# TYPE spanner_max_in_use_sessions_ratio gauge
spanner_max_in_use_sessions_ratio{{client_id="client-1",database="test-db",instance_id="test-projects",otel_scope_name="cloud.google.com/java"} 0.0

Only I see 1 data which is not the SDK metric
# TYPE target_info gauge
target_info{service_name="my-service-name",service_version="1.0.0",telemetry_sdk_language="java",telemetry_sdk_name="opentelemetry",telemetry_sdk_version="1.41.0"} 1

This is working as expected, since all the metrics exported will have your defined resource attributes (https://opentelemetry.io/docs/concepts/resources/#introduction).
Can you tell me what was your expectations? Were you expecting the attributes to be added in all the metrics?


finalizer may never be called

Is your feature request related to a problem? Please describe.
In Julia, it looks like finalizers are always called when exiting.
E.g., the following code always prints "hola".
mutable struct S
    a
end
function S()
    return finalizer(_->Core.println("hola"), S(3))
end
S()

However, if we wrap it with juliacall, the finalizer is never executed.
import juliacall
juliacall.Main.include("kk.jl")

Describe the solution you'd like
I'd like finalizers to always be executed
Describe alternatives you've considered
My current workaround is finishing with
juliacall.Main.GC.gc()

Ok yeah looks like I should call jl_atexit_hook to stop the Julia runtime when Python exits. https://github.com/JuliaLang/julia/blob/f3298ee5e2b08f633a6a313d094c732ff1ce7e85/src/init.c#L236

thanks, @cjdoris what about #516 ?
I works with this
Main.seval(
            """ mutable struct S
                    a
                end;
                function S()
                    return finalizer(_->Core.println("works!!!"), S(3));
                end;
                S();
        """


Tons of errors when opening in Visual studio 2019

i get hundreds of errors when i open in visual studio 2019 but only in the Plants vs zombies project not anything else

you may need game resource
---Original---
From: @.>
Date: Thu, Aug 19, 2021 02:10 AM
To: @.>;
Cc: @.***>;
Subject: [ErLinErYi/PlantsVsZombies] Tons of errors when opening in Visual studio 2019 (#14)
i get hundreds of errors when i open in visual studio 2019 but only in the Plants vs zombies project not anything else
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
 		  			从QQ邮箱发来的超大附件

	 			 资源文件_0619.rar (47.01M, 无限期) 			 			 				进入下载页面：https://mail.qq.com/cgi-bin/ftnExs_download?k=9a9d5e61db4b5b4ba8ee1661666663351341c86364666335404d5e5856565307510415515003521851510104495e0002064f5d020050020d5d500b51000776358dd7bc87def685a3e28683d73b5655045c4c4a0016725acd42574b27423574caa5b58eaa7a1244226953&t=exs_ftn_download&code=eb8adfc5

Ill try but it was saying in visual studio that you are not allowed to use these character on many differant cpp and h files

I see. Because some variables in the code use Chinese characters and your computer language is English, an error has occurred. You may need to change the Chinese characters in the code to English.
---Original---
From: @.>
Date: Thu, Aug 19, 2021 19:01 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [ErLinErYi/PlantsVsZombies] Tons of errors when opening in Visual studio 2019 (#14)
Ill try but it was saying in visual studio that you are not allowed to use these character on many differant cpp and h files
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Okay i will try also amazing project im trying to get better at c++ with this project

It gave error for this line in the code
ÏÔÊ¾ÐÅÏ¢ = 1,
È«ÆÁ,
¸ßÖ¡ÂÊ,
Êó±êÒþ²Ø,
ÀÉìÏÔÊ¾,
»ºÈë¶¯»,
´¹Ö±Í¬²½

not sure why its like that

some of the comments in the code are like that too

maybe glitched code

would be nice if you can send english translated version or something

Because Chinese characters cannot be parsed normally, they are displayed as garbled codes. Those garbled codes are enumeration types. In fact, you can replace them with English at will. If you can't finish it yourself, wait for me to update it.
---Original---
From: @.>
Date: Thu, Aug 19, 2021 23:02 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [ErLinErYi/PlantsVsZombies] Tons of errors when opening in Visual studio 2019 (#14)
It gave error for this line in the code
ÏÔÊ¾ÐÅÏ¢ = 1,
È«ÆÁ,
¸ßÖ¡ÂÊ,
Êó±êÒþ²Ø,
ÀÉìÏÔÊ¾,
»ºÈë¶¯»,
´¹Ö±Í¬²½
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

yes but in your code is it chinese because if it is i can translate it to english

actually in github the code is english so i just type the english version in the code but one line called Display information = 1 is giving an error on the word information

It also has issues importing other files such as .h


Text cell editor doesn't respect the 'Tab Moves Focus' setting

Steps to Reproduce:

Open a notebook and create a text cell
Run the 'Toggle Tab key Moves Focus ' command

Set the focus in the text editor and then press tab

Expected:
Just like in other editors, I should be able to move the keyboard focus out from the text cell
Actual:
focus stays in the text cell

@alanrenmsft which edit mode are you in? WYSIWYG, or split/markdown view for the text cell?

I tried all 3 modes, behaves the same

Whoa that's wild. Code and text cells use the same editor control. Good catch.


fix(docs): add correct key for commit sha value

Fixes #471

Hi @technosophos! Is there anything else that would be required in this PR?

Hey, @OrthoDex!
First of all, thanks for your contribution!
Going over all other references for the event commit, I came across the following:

the document referring to the BrigadeEvent class - https://github.com/Azure/brigade/blob/master/docs/topics/javascript.md#the-brigadeevent-class - as we can see in the BrigadeEvent TypeScript class, it does not contain the commit property anymore, but instead expses it through a revision property - the doc should reflect that.

a portion of documentation referring to an Event object - https://github.com/Azure/brigade/blob/master/docs/topics/javascript.md#the-event-object
@technosophos - is the Event object mentioned in the docs still used?

@radu-matei yeah, the Event object is still used in a few places, though it might be constructed as a bare object in most cases.

@radu-matei Thanks for the feedback! I'll update those references in the PR as well.

Hey @radu-matei, anything else to be added to this PR?

Sorry for the delay and thank you again for these changes @OrthoDex ; merging this in!


Example: Fix Bazel Build.

The native git_repository rule is deprecated. load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository") for a replacement.

See https://travis-ci.org/census-instrumentation/opencensus-java/jobs/462666269#L567.

Codecov Report

Merging #1626 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1626      +/-   ##
============================================
- Coverage     83.51%   83.46%   -0.05%
  Complexity     1813     1813
============================================
  Files           261      261
  Lines          8339     8339
  Branches        803      803
============================================
- Hits           6964     6960       -4
- Misses         1121     1124       +3
- Partials        254      255       +1

Impacted Files
Coverage Δ
Complexity Δ

...e/ocagent/OcAgentTraceServiceConfigRpcHandler.java
75.86% <0%> (-3.45%)
12% <0%> (ø)

...census/implcore/trace/export/SpanExporterImpl.java
94.52% <0%> (-2.74%)
9% <0%> (ø)

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 8a9b674...e4ca4dc. Read the comment docs.


Update commandunit function

Update commandunit function.
In the commandunit-example project, there is an improved version of commandunit function and it should be imported to the main project side.

The function doesn't exist anymore and was replaced with a wrapper script.
The script now automatically grabs the most recent version of the script from the commandunit repo.


Add Links to Examples in API Docs.

Hi all,
I think it might be useful to add links in the API docs (i.e. the docstrings) to examples in the tutorials. I am finding that the doc strings themselves are often not enough to go by and a few simple examples would go a long way to clarifying how to use the various classes. If we did not want to set up links we could just add examples to the doc strings.
Regards
Jordan

Yeah I do want to try and expands the docs in general with minimal examples, I figured this would be a part of the process of making the beginner ∂Lux overview tutorials.

Done with PR #218


Having problem importing material properties from xml file and heat flux from csv file

Hi there,
I am trying to automate a series of steady state thermal simulations with a same geometry and different heat flux boundary conditions imported from a CSV file. I have created a PyMechanical script attached below. My problem is that I couldn't find out how to import the material .xml file and the heat flux data from CSV file. In workbench I was using the Engineering Data and External Data to import these data, but I don't know how to do it in PyAnsys. The CSV file includes the X,Y,Z coordinates and the heat flux value for each coordinate. Can you please help me add the necessary commands to import these data appropriately.
I have an open Service Request<PHONE_NUMBER>4 on Ansys Support, but I was directed to GitHub Issues for PyMechanical.
The script is attached below and as a txt file.
Thanks a lot,
Amir
# Steady State Thermal analysis

import os

from ansys.mechanical.core import launch_mechanical

geometry_path = "C:\\ANSYS\\Geometry.scdoc"
material_path = "C:\\ANSYS\\Material.xml"     # Don't know how to use it
heat_flux_path = "C:\\ANSYS\\power_map.csv"   # Don't know how to use it

###############################################################################
# Launch Mechanical
# ~~~~~~~~~~~~~~~~~
# Launch a new Mechanical session in batch, setting ``cleanup_on_exit`` to
# ``False``. To close this Mechanical session when finished, this example
# must call  the ``mechanical.exit()`` method.

mechanical = launch_mechanical(batch=True, cleanup_on_exit=False)
print(mechanical)

###############################################################################
# Initialize variable for workflow
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# Set the ``part_file_path`` variable on the server for later use.
# Make this variable compatible for Windows, Linux, and Docker containers.

project_directory = mechanical.project_directory
print(f"project directory = {project_directory}")

# Upload the file to the project directory.
mechanical.upload(file_name=geometry_path, file_location_destination=project_directory)

# Build the path relative to project directory.
base_name = os.path.basename(geometry_path)
combined_path = os.path.join(project_directory, base_name)
part_file_path = combined_path.replace("\\", "\\\\")
mechanical.run_python_script(f"part_file_path='{part_file_path}'")

# Verify the path
result = mechanical.run_python_script("part_file_path")
print(f"part_file_path on server: {result}")

###################################################################################
# Execute the script
# ~~~~~~~~~~~~~~~~~~
# Run the Mechanical script to attach the geometry and set up and solve the
# analysis.

output = mechanical.run_python_script(
    """
import json

# Import Geometry

geometry_import_group_1 = Model.GeometryImportGroup
geometry_import_1 = geometry_import_group_1.AddGeometryImport()

geometry_import_1_format = Ansys.Mechanical.DataModel.Enums.GeometryImportPreference.Format.Automatic
geometry_import_1_preferences = Ansys.ACT.Mechanical.Utilities.GeometryImportPreferences()
geometry_import_1.Import( part_file_path, geometry_import_1_format, geometry_import_1_preferences)

# Import material properties and assign them

      ## How to import material properties from an XML file?
with Transaction(True):
    body_1 = DataModel.GetObjectsByName('SYS\Part1')[0]
    body_1.Material = "Indium"

with Transaction(True):
    body_2 = DataModel.GetObjectsByName('SYS\Part2')[0]
    body_2.Material = "Silicon"

# Define Steady state thermal analysis

Model.AddSteadyStateThermalAnalysis()
ST_STA = Model.Analyses[0]
CS_GRP = Model.CoordinateSystems
ANALYSIS_SETTINGS = ST_STA.Children[0]
SOLN= ST_STA.Solution

ExtAPI.Application.ActiveUnitSystem = MechanicalUnitSystem.StandardMKS
ExtAPI.Application.ActiveAngleUnit = AngleUnitType.Radian

# Model Named Selections

Part2_bot = Model.NamedSelections.Children[0]
Part1_bot = Model.NamedSelections.Children[1]

# Model connections
connections_1 = Model.Connections
connection_group_1 = connections_1.AddConnectionGroup()
connection_group_1.ToleranceType = ContactToleranceType.Value
connection_group_1.ToleranceValue = Quantity(0.001, "mm")

MSH = Model.Mesh
MSH.ElementSize =Quantity("1.0 [mm]")
MSH.GenerateMesh()

# Import heat flux from external data .csv file
    ##### How to import the heat flux data from csv file?

imported_load_1 = ST_STA.AddImportedLoadExternalData()
imported_load_1.GetExternalDataFiles

heat_flux_1 = imported_load_1.AddImportedHeatFlux()
heat_flux_1.Location = Part1_bot
heat_flux_1.ImportLoad()

# Add heat transfer
convection_1 = ST_STA.AddConvection()
convection_1.Location = Part2_bot
convection_1.FilmCoefficient.Output.SetDiscreteValue(0, Quantity(0.001, "W mm^-1 mm^-1 K^-1"))
convection_1.AmbientTemperature.Output.SetDiscreteValue(0, Quantity(35, "C"))

Temp_1 = ST_STA.Solution().AddTemperature()
ST_STA.Solution.Solve(True)

temperature_details = {
"Minimum": str(Temp_1.Minimum),
"Maximum": str(Temp_1.Maximum),
"Average": str(Temp_1.Average),
}

json.dumps(dir_deformation_details)
"""
)
print(output)

###############################################################################
# Download output file from solve and print contents
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# Download the ``solve.out`` file from the server to the current working
# directory and print the contents. Remove the ``solve.out`` file.
def get_solve_out_path(mechanical):
    solve_out_path = ""
    for file_path in mechanical.list_files():
        if file_path.find("solve.out") != -1:
            solve_out_path = file_path
            break

    return solve_out_path

def write_file_contents_to_console(path):
    with open(path, "rt") as file:
        for line in file:
            print(line, end="")

solve_out_path = get_solve_out_path(mechanical)

if solve_out_path != "":
    current_working_directory = os.getcwd()

    local_file_path_list = mechanical.download(solve_out_path, target_dir=current_working_directory)
    solve_out_local_path = local_file_path_list[0]
    print(f"Local solve.out path : {solve_out_local_path}")

    write_file_contents_to_console(solve_out_local_path)

    os.remove(solve_out_local_path)

###########################################################
# Close Mechanical
# ~~~~~~~~~~~~~~~~
# Close the Mechanical instance.

mechanical.exit()

example_01_steady_state_thermal_solve.txt

I found the command for importing the material data from Example_06 which  are:
MAT = ExtAPI.DataModel.Project.Model.Materials
MAT.Import(mat_Copper_file_path)
MAT.Import(mat_Steel_file_path)

However, when I run this example I get the following error:
_MultiThreadedRendezvous: <_MultiThreadedRendezvous of RPC that terminated with:
	status = StatusCode.UNKNOWN
	details = "'Materials' object has no attribute 'Import'"
	debug_error_string = "UNKNOWN:Error received from peer  {created_time:"2023-07-07T15:44:16.840381658+00:00", grpc_status:2, grpc_message:"\'Materials\' object has no attribute \'Import\'"}"

Materials.Import is only supported from Ansys 2023R2 onwards. Prior to that version, we have a workaround. See the file src/ansys/mechanical/core/embedding/shims.py for more information.

@koubaa Hi Mohamed. Thanks, I used the jscript as a workaround to import the material until the R23R2 is released. I have another issue that is mentioned in the original post for importing external data. I use the following script and I get an error:
Model.AddSteadyStateThermalAnalysis()
ST_STA = Model.Analyses[0]
imported_load_group_104 = ST_STA.AddImportedLoadExternalData()
external_data_files = Ansys.Mechanical.ExternalData.ExternalDataFileCollection()
external_data_files.SaveFilesWithProject = False
external_data_file_1 = Ansys.Mechanical.ExternalData.ExternalDataFile()
external_data_files.Add(external_data_file_1)
external_data_file_1.Identifier = "File1"
external_data_file_1.Description = ""
external_data_file_1.IsMainFile = True
external_data_file_1.FilePath = r"C:\\Ansys\\CCD_power_map.csv"
external_data_file_1.ImportSettings = Ansys.Mechanical.ExternalData.ImportSettingsFactory.GetSettingsForFormat(MechanicalEnums.ExternalData.ImportFormat.Delimited)
import_settings = external_data_file_1.ImportSettings
import_settings.SkipRows = 0
import_settings.SkipFooter = 0
import_settings.Delimiter = ","
import_settings.AverageCornerNodesToMidsideNodes = "True"
import_settings.UseColumn(1, MechanicalEnums.ExternalData.VariableType.XCoordinate, "mm", "X Coordinate@B")
import_settings.UseColumn(2, MechanicalEnums.ExternalData.VariableType.YCoordinate, "mm", "Y Coordinate@C")
import_settings.UseColumn(3, MechanicalEnums.ExternalData.VariableType.ZCoordinate, "mm", "Z Coordinate@D")
import_settings.UseColumn(4, MechanicalEnums.ExternalData.VariableType.HeatFlux, "W m^-2", "Heat Flux@E")
imported_load_group_104.ImportExternalDataFiles(external_data_files)
imported_heatflux_111 = imported_load_group_104.AddImportedHeatFlux()
imported_heatflux_111.Location = CCDs_NS
imported_heatflux_111.Import()

The error screenshot is attached. Do you know what might be wrong that causes this error? The error is for imported_load_group_104.ImportExternalDataFiles(external_data_files) which is AttributeError:'NoneType' object has no attribute 'ImportExternalDataFile'

Found the issue through Ansys Support. The problem was I was not using the Stand alone version of Mechanical using the following command:
'C:\Program Files\ANSYS Inc\v231\aisol\Bin\winx64\AnsysWBU.exe' -DSApplet -AppModeMech


The install script fails with:

Changing access permissions for binary script :: weasel
ln: /Users/jfinlays/.oh-my-zsh/themes: No such file or directory
/Users/jfinlays/.zshrc: line 30: ${(k)dirs_to_prepend[@]}: bad substitution
/Users/jfinlays/.zshrc: line 43: /Users/jfinlays/z/z.sh: No such file or directory

Hi, anyone has any idea on this? I'm on Sierra.
After removing this error (by deleting (k)) I get:
/Users/amit/.shell_aliases: line 193: ${EDITOR} ${$(cd -P $(dirname $0) && cd ..; pwd -P)}: bad substitution
/Users/amit/.shell_config: line 6: setopt: command not found
/Users/amit/.shell_config: line 7: setopt: command not found
/Users/amit/.shell_config: line 8: unsetopt: command not found
/Users/amit/.shell_config: line 11: setopt: command not found
/Users/amit/.oh-my-zsh/oh-my-zsh.sh: line 12: autoload: command not found
/Users/amit/.oh-my-zsh/oh-my-zsh.sh: line 31: syntax error near unexpected token `('
/Users/amit/.oh-my-zsh/oh-my-zsh.sh: line 31: `for config_file ($ZSH/lib/*.zsh); do'
/Users/amit/.zshrc: line 105: fortune: command not found
/Users/amit/.zshrc: line 105: ponysay: command not found

getting this also:
.zshrc: line 30: ${(k)dirs_to_prepend[@]}: bad substitution .zshrc: line 43: z/z.sh: No such file or directory .shell_aliases: line 193: ${EDITOR} ${$(cd -P $(dirname $0) && cd ..; pwd -P)}: bad substitution .shell_config: line 6: setopt: command not found .shell_config: line 7: setopt: command not found .shell_config: line 8: unsetopt: command not found .shell_config: line 11: setopt: command not found .oh-my-zsh/oh-my-zsh.sh: line 12: autoload: command not found .oh-my-zsh/oh-my-zsh.sh: line 31: syntax error near unexpected token('
.oh-my-zsh/oh-my-zsh.sh: line 31: for config_file ($ZSH/lib/*.zsh); do' .zshrc: line 103: fortune: command not found .zshrc: line 103: ponysay: command not found

I am having the same issue:
/Users/zenahirsch/.zshrc: line 30: ${(k)dirs_to_prepend[@]}: bad substitution
/Users/zenahirsch/.zshrc: line 43: /Users/zenahirsch/z/z.sh: No such file or directory
/Users/zenahirsch/.shell_aliases: line 193: ${EDITOR} ${$(cd -P $(dirname $0) && cd ..; pwd -P)}: bad substitution
/Users/zenahirsch/.shell_config: line 6: setopt: command not found
/Users/zenahirsch/.shell_config: line 7: setopt: command not found
/Users/zenahirsch/.shell_config: line 8: unsetopt: command not found
/Users/zenahirsch/.shell_config: line 11: setopt: command not found
/Users/zenahirsch/.oh-my-zsh/oh-my-zsh.sh: line 12: autoload: command not found
/Users/zenahirsch/.oh-my-zsh/oh-my-zsh.sh: line 31: syntax error near unexpected token `('
/Users/zenahirsch/.oh-my-zsh/oh-my-zsh.sh: line 31: `for config_file ($ZSH/lib/*.zsh); do'
/Users/zenahirsch/.zshrc: line 103: fortune: command not found
/Users/zenahirsch/.zshrc: line 103: ponysay: command not found

I am also having the same issue
/Users/josiah/.zshrc: line 30: ${(k)dirs_to_prepend[@]}: bad substitution
/Users/josiah/.zshrc: line 43: /Users/josiah/z/z.sh: No such file or directory
/Users/josiah/.shell_aliases: line 193: ${EDITOR} ${$(cd -P $(dirname $0) && cd ..; pwd -P)}: bad substitution
/Users/josiah/.shell_config: line 6: setopt: command not found
/Users/josiah/.shell_config: line 7: setopt: command not found
/Users/josiah/.shell_config: line 8: unsetopt: command not found
/Users/josiah/.shell_config: line 11: setopt: command not found
/Users/josiah/.oh-my-zsh/oh-my-zsh.sh: line 12: autoload: command not found
/Users/josiah/.oh-my-zsh/oh-my-zsh.sh: line 31: syntax error near unexpected token `('
/Users/josiah/.oh-my-zsh/oh-my-zsh.sh: line 31: `for config_file ($ZSH/lib/*.zsh); do'
/Users/josiah/.zshrc: line 103: fortune: command not found
/Users/josiah/.zshrc: line 103: ponysay: command not found

Had the same issue - now stuck with a lot of messed up stuff. :)

Had the same issue... Any progress?

same issue here :(

Has anyone resolved this issue? I'm getting this in 2020.


Issue #1 : Display says funny quote text when showing the goodbye screen, source link

Says haha look ${name} I'm funny when it has run out of funny quotes

Also shows the source link

Changed display text at end


GP dialog, mappin enhancements

See issues #40  and #42. Hope I committed correctly now. :o

Absolutely fantastic, I'm testing the changes now

Some outstanding issues with the current changes:
Wonder district icon is missing some string data, which makes the tooltip a bit ugly

The great prophet panel is stashed to the right when prophets are still available, but once they're all claimed it moves back to the center and doesn't hide

Huh, I don't even know what kind of district that is, probably wrong data. Filtered out now.
Can you give me a save file for the GP one? Worked fine on my side.

Here's the save: https://docs.google.com/uc?id=0BzRHuYM0ypGoeXRRYWV6Mm1HNEU
If you'd like to be sure you're on exactly the same version I am, follow these instructions

Hm, you started a new game for this, right? Weird, values that are set for every other item in your save, and every item in my save, is completely missing for the great prophet in your save. DOn't know why yet.

This is a save I test lots of things on, but I I never save the progress after finishing my tests, so it's effectively a clean slate

Any progress?

Pins are fixed, didn't find a solution to the GP problem yet. Can't explain why your data is null.

Is there a way to remove files from a pull request? Otherwise I'd close this one and just create a new one with the fixed pins.

@jacks0nX You should just be able to add the new changes to your existing fork and they'll be part of the PR too. For the GP problem, I think it'll just be best if we nixx hiding the panel for now if we can't iron out this behavior.

I feel the GreatProphet should NOT get hidden by any means.. only pushed to the far right is enough.
BUT -- i do have a recommendation for that panel. Haven't yet done the necessary MockUP to visualize what i mean, yet i can explain what should solve most issues -- verbally here...

Discard all instances of "RECRUIT PROGRESS" & "or Recruit with Patronage" except for the GreatProphet slot as a unique indirect reference to context. To make room for...

A re-calibrated panel that prevents the Progress blocks from sharing their side bars. It's quite simple.. you'd gain at least 16 pixels by compressing the two Patronage cost boxes together while leaving only 4 pixels between them. Then, you also reduce all the growth meterbars & the Biography button by the same amount (16).
Secondly, i we get to keep (as much as i am still hoping for & demonstrated with an image in a post on CivFanatics) the Vanilla Framework panels (Modal Backgrounds).. it also can crop its own 16 pixels off the middle part to match any total width(s).

Long names like "Ludwig von Beethoven" might overlap the Panel side lines a little.. unless we keep (again) the Vanilla structure that puts a dividing line for the ERA reference.

I'll return with direct visual hints later.

I'm not quite sure what you mean, but I'll gladly wait for a mockup! :p

Jesus, I messed up that whole commit/push/ stuff again.. I'll just close this and create a new one.

Copy/Paste from #40 for your convenience!


Q: Is there any way to export the data from the Database or access the container from outside?

Hi @mpous,
I have been using MING for a while in a project, and recently I have been asked if there is a possibility to extract the data from the InfluxDB database.
Do you know if there is any way, is it possible to access the container from outside Balena Cloud?
As options, to have the data more at hand, it occurs to me to use InfluxDB Cloud, but I would already have to change the current project and start from scratch... and I am interested in extracting the data I already have.
Please let me know if there are any options.
Thank you so much in advance!

@alexcorvis84 first of all apologizes for the delay answering this message! i didn't realize about your issue!

Do you know if there is any way, is it possible to access the container from outside Balena Cloud?

What do you mean? What try to access do you need? SSH? REST API? I'm sure Influx provides multiple ways to query the measurements.

if there is a possibility to extract the data from the InfluxDB database.

How do you need the data? did you try to expose a REST API to query the Influx database from NodeRED?


nixos/acme: Disable bash tracing

Motivation for this change
This is horrible if you want to debug failures that happened during
system switches but your 30-ish acme clients spam the log with the same
messages over and over again.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Honestly I'm not too keen on disabling this. When it was introduced there was quite some discussion about it (see here for some of that). The main reason it is there is that ACME failures tend to be really hard to reproduce, and having the script's logs has always been really helpful in debugging.
With the module working reliably for quite some time now, I can understand that the value of having it enabled is probably being outweighed by the log spam. I'm still undecided though, I'd be keen to hear what other people think (@NixOS/acme & co).

It's probably easier to opt in than to opt out because one can just do:
script = lib.mkBefore ''
  set -x
'';

This is not possible the other way around

Ok well for a middle ground, how do you feel about adding security.acme.enableDebugLogs (or similar), and defaulting it to true? That way users who run into issues on their first few runs will have clear logs of what went wrong, and those with working systems with lots of logs can disable it.

I hope this is close to what you had in mind

@m1cr0man Do you want me to remove the backport label or are you fine with backporting this change?

Actually.. I hate to be indecisive like this but maybe this is better as an option under security.acme.certs? In your PR description you point out the difficulty debugging if theres many acme clients. If its under the submodule options, that can be solved here too :)

@m1cr0man Do you want me to remove the backport label or are you fine with backporting this change?

Keep it for now. I'm not entirely sure if #147784 will be backportable yet.

Looks good now!

@GrahamcOfBorg test acme


Issue 180

Coverage remained the same at 95.435% when pulling 0bff4bdbdd1ba5688291f53a988e1e12c642ae9c on issue-180 into 2caf7d61cc94213cbd158740b7e9798fd94c2404 on master.

SonarCloud claims there's no test coverage on two lines you changed here. Also, it seems to still be using @chutch 's personal SonarCloud account.


Add adminuser option

Would be nice to be able to set the admin username. Most Citadel apps have the default username "citadel" and it's confusing for users to have different usernames for each app.

Thanks for the PR, I will include this in the next release.


Updates based on signing scheme update in notation-go-core

Updates to accommodate changes as part of https://github.com/notaryproject/notation-core-go/pull/17

Closes https://github.com/notaryproject/roadmap/issues/28

Codecov Report

Merging #85 (660a881) into main (55fd020) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
+ Coverage   72.12%   72.14%   +0.01%
==========================================
  Files          36       36
  Lines        2264     2265       +1
==========================================
+ Hits         1633     1634       +1
  Misses        506      506
  Partials      125      125

Impacted Files
Coverage Δ

signature/plugin.go
68.50% <100.00%> (+0.15%)
:arrow_up:

signature/signer.go
51.80% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Getting old version of React in the package when create Sharepoint webpart

Category

[x] Bug

I have created the Sharepoint webpart by running yo @microsoft/sharepoint and I got all the old versions of react and related packages in package.json.

To resolve this issue,
I have removed existing node package and installed the latest version 12.16.1
- when I run npm -v, I get v.12.16.1
ran npm i npm@latest -g
- now npm version 6.15.2
ran npm install @microsoft/generator-sharepoint@latest -g
- now i have<EMAIL_ADDRESS>but after I upgraded the node/npm and created the new webpart, I still get the old react versions in package.json.
I would need at least react 16.8 or higher version in order to use some of the react libraries.
Any ideas?

Hi,
What's your package version if you run npm outdated -g, I just did upgrade my local environment from 1.8 to 1.10 successfully.
Update SharePoint Framework packages
Lee
Microsoft SharePoint Community Support

Hi,
Try to use node version 10.18.0 for better SPFx compatibility. I dont think so v12.x.x is support for SPFx.

The latest supported  nodejs version 10.19.0.
The current supported LTS version of NodeJS for the SharePoint Framework is Node.js v8.x and Node.js v10.x.
Notice that 9.x, 11.x or 12.x versions are currently not supported with SharePoint Framework development.
https://docs.microsoft.com/en-us/sharepoint/dev/spfx/set-up-your-development-environment#install-developer-tools
Try to use the latest node version

@PrasadKasireddy said:

Try to use the latest node version

No, that's not correct. Don't use the latest... use what's listed as supported and recommended on the page you linked to.
@hh747hh need more context... what environment are you targetting? I suspect you are pointing to SharePoint 2016 / SharePoint 2019. Those only have React v15... they don't support React v16 as React v16 was added to a later version of SPFx than what's available on-prem.

@PrasadKasireddy said:

Try to use the latest node version

No, that's not correct. Don't use the latest... use what's listed as supported and recommended on the page you linked to.
@hh747hh need more context... what environment are you targetting? I suspect you are pointing to SharePoint 2016 / SharePoint 2019. Those only have React v15... they don't support React v16 as React v16 was added to a later version of SPFx than what's available on-prem.

it's my mistake, it's a type of error -  Try to use the latest node version (10.19) instead of 10.18.


Exceptions being thrown from DeviceMonitor Dispose()

When Dispose is called on DeviceMonitor after being Started, AdbExceptions are thrown from  this.monitorTask.Wait();. Throwing exceptions from dispose is a bad practice.
Repro:
            var socket = new AdbSocket(new IPEndPoint(IPAddress.Loopback, AdbClient.AdbServerPort));
            using (DeviceMonitor monitor = new DeviceMonitor(Socket))
            {
                monitor.Start();
                Thread.Sleep(1000);
                monitor.Dispose(); //exception thrown
            }

I'm seeing the same after the recent refactoring work (which I love btw!). In my case it's an AdbException that has an inner exception of a SocketException. For now I'm using this:
try
{
    monitor.Dispose();
}
catch (AggregateException exception) when (
    exception.Flatten().InnerExceptions is ReadOnlyCollection<Exception> innerExceptions
    && innerExceptions.Count == 1 && innerExceptions[0] is AdbException)
{
    // Ignored.
}


Bump dependencies and CI

include latest GHCs in CI
latest minor version is determined by haskell-actions/setup
freeze file is a bad cache key as it contains a time stamp,
use the build plan instead (the correct choice)

Successful CI run at: https://github.com/andreasabel/Haxl/actions/runs/6725224644

@simonmar : PR ready

Hi @andreasabel ! Thanks for the patch! Unfortunately it looks like it's broken for 9.4.

Updated this PR.  Not much remains, since test.yml was deleted on main.
Now this just updates the Haskell CI to the latest GHCs.

GHC 9.10 comes with filepath-1.5, so I relaxed the upper bound.

CI passes now.


Clean up incomplete files if sync is interrupted

It is possible for syncMetadata process to be interrupted due to user cancellation or IOExceptions.
We need to recover from incomplete files if we happen to have created the file but not completed the file.

Codecov Report

Merging #12828 (ff58644) into master (33b564b) will decrease coverage by 34.02%.
The diff coverage is 0.00%.

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

Offline discussion lead to a cleaner alternative approach, will open a separate PR