Fixed ExpressRoute commands

Several ExpressRoute commands were using -Circuit instead of -ExpressRouteCircuit. This PR addresses those.

Hi @jboeshart, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@ganesr - Could you review these updates in the public repo from @jboeshart?

#sign-off


application app_config must provide IHostingEnvironment

How to convert this code into Saturn compliant?
if (env.IsDevelopment()) then
    app.UseDeveloperExceptionPage() |> ignore
else
    // The default HSTS value is 30 days. You may want to change this for production scenarios, see https://aka.ms/aspnetcore-hsts.
    app.UseHsts() |> ignore

These and other questions at some point pushed me to abandon application {} and just use raw webhostbuilder.
However now that panesofglass found an overload trick for custom CE operations we could add all the missing overloads.

@NinoFloris could you explain or post an excerpt of code?

@NinoFloris could you explain or post an excerpt of code?

Seems to be referred to here: #164.
with original sources here and here.
I'm still going over the mechanism myself.

To answer the original question - this should work fine:
app_config (fun app ->
    let env = app.ApplicationServices.GetService<IWebHostEnvironment >()
    if (env.IsDevelopment()) then
        app.UseDeveloperExceptionPage() |> ignore
    else
        // The default HSTS value is 30 days. You may want to change this for production scenarios, see https://aka.ms/aspnetcore-hsts.
        app.UseHsts() |> ignore

Saturn 0.14 will include some helper to replace app.ApplicationServices.GetService<IWebHostEnvironment >() call with something more user friendly.


Verify Registration

Send the registrant a confirmation email with a unique link to verify his/her registration.

We're not going to implement this at this time.


Is there plan for more CUDA context api?

This is a dlib python api issue as well as a cuda one.
I have a python application doing face image recognition and processing. CNN face detection model of dlib is used and I have no problem compiling dlib with cuda support. The project runs well when the input image is of modest size, otherwise RuntimeError: cudaMalloc() out of memory is raised and terminates the process.
A feature we need to know is that a cuda context is created for any call of cuda api (this description may be inaccurate. I'm professional in cuda programming). Some run time error may "corrupt" the cuda context and therefore any calls afterward would fail. Some one discussed about this two years ago on StackOverflow: https://stackoverflow.com/questions/30909368/once-cudamalloc-returns-out-of-memory-every-cuda-api-call-returns-failure
A function is provided to reset the context to runnable state: cudaDeviceReset(). For the C++ programmers, this case seems to be closed.
OOM error can be caught while calling dlib python api. Naturally I want to recover the context state after  this. At first, I tried to set all reference of dlib resources to None and reload the dlib module, which didn't work. Then I wrote a cuda program, something like this:
//release all references to cuda resources before running this, otherwise you'll get segment fault.
class CudaCtx {
  public:
  void reset_ctx(int);
};
void CudaCtx::reset_ctx(int gpu_no) {
  cudaSetDevice(gpu_no);
  cudaError_t error = cudaGetLastError();
  if (error != cudaSuccess) {
    //expected branch, this would hopefully get the error code in the dlib process
    std::cout << cudaGetErrorString(error) << std::endl;
  }
 else {
    //what I actually get
    std::cout << "nothing to reset" << std::endl;
  }
  //cleanup allocated gpu memory regardless.
  cudaDeviceReset();
}
extern "C" {
  CudaCtx cc;
  void reset_ctx(int gpu_no) {
    cc.reset_ctx(gpu_no);
  }
}

I compiled it with nvcc and called it in Python project. The cudaDeviceReset() function did clean up all memory allocation by the python process. However, the error state caused by dlib in the same process is not caught (and the main python process continues to say OOM in cudaGetLastError() after calling it). This indicates that the dlib functions and the cleanup program are in different cuda context. However, the cleanup program did managed to clean up something that does not belong to it. (monitor GPU memory usage in nvidia-smi).
I couldn't find any dlib api on cuda context resetting, so there's nothing I can do to recover the ctx state other than killing the whole process. Is there any plan for such api? Any suggestions are also very much appreciated here. Thank you.

Yes, there are some cuda related context objects.  For example, https://github.com/davisking/dlib/blob/master/dlib/cuda/cublas_dlibapi.cpp#L89.  They all have that same thread_local construction. You are welcome to submit a PR that adds a method that resets these cuda context objects.  However, in general I think the right thing to do is to avoid running out of memory in the first place by doing appropriate input validation and not running images that would lead to memory exhaustion.


Feature Request: Support downloading raw files

We'd like to download raw binary and text files from the cli.  I know you can > the output but it adds the Getting files for app\nOK message to the output which is annoying and doesn't work with binary files.
I'm thinking an arg like -f would work.
cf files app app/someBinary.zip -f localBinary.zip
As a side note.  We have  a fork of the CLI.  It would be handy if there was a way to invoke net.doRequest (https://github.com/cloudfoundry/cli/blob/master/cf/net/gateway.go#L402) without net.dumpResponse dumping the response body as well.  https://github.com/cloudfoundry/cli/blob/master/cf/net/http_client.go#L59
I imagine changing that api will be necessary to fix this issue so it would be great if it was fixed in such a way that others could take advantage of the ability to execute requests that don't dump the response body aswell.
Thanks,
Mike

with SCP coming in Diego I think I'll close this.

https://github.com/ibmjstart/cf-downloadmay help for those not on Diego yet.

For those landing on this old issue, the CF ssh enables use of the scp command, whose specific syntax is detailed in https://docs.cloudfoundry.org/devguide/deploy-apps/ssh-apps.html#other-ssh-access


Getting error 'PluginTables' had unhandled error: runtime error: invalid memory address or nil pointer dereference when spreadsheet_id is empty

Describe the bug
If spreadsheet_id is not defined in config, it is failing with following error
Error: failed to start plugin 'googlesheets': failed to plugin initialise plugin 'steampipe-plugin-googlesheets': TableMapFunc 'PluginTables' had unhandled error: runtime error: invalid memory address or nil pointer dereference
Steampipe version (steampipe -v)
Example: v0.9.1
Plugin version (steampipe plugin list)
Example: v0.0.1
To reproduce
Comment/ Remove spreadsheet_id from config, and try to run steampipe query
Expected behavior
Error message should be something like spreadsheet_id must be configured.
Additional context
Add any other context about the problem here.

I'm having the same result, but not because spreadsheet_id is empty. Have yet to find a success path through the configuration maze, will try to add logging to figure it out.


Indicate in the UI when Redis is not running

What version of fleet are you using (fleet version --full)?
Any
What operating system are you using?
Any
What did you do?
Try to run a live query (with Redis not running or unable to connect).
What did you expect to see?
Query results
What did you see instead?
Spinning box animation.
Users are often confused when no results appear. The UI should make clear that no results will be coming because there is no Redis connection.

Took me a while to figure this out. This would be wonderful for those new to Fleet


Cache tags

Is there a way to use this package with cache tags so one can flush only parts of the cached requests?

I don't think so, because the only identifier is slug.

For the fastest response times, these pages are actually cached directly to disk as static files.
When serving a cached file, Laravel never runs. PHP isn't even involved at all. Cached tags don't make sense in this context.

What about tagging/grouping/prefixing the cached files? Is that doable?

Not really. The files are stored on disk directly as static files.
The only thing we could maybe add is clearing a subdirectory, but I'm not sure how useful that is.


Receiving a bit too much?

I'm on RPi 4B, and I without doing anything, I receive this within one minute:
2022-07-24 21:41:01 - [INFO] rpi-rf_receive: 12 [pulselength 836, protocol 4]
2022-07-24 21:41:13 - [INFO] rpi-rf_receive: 2 [pulselength 1335, protocol 4]
2022-07-24 21:41:15 - [INFO] rpi-rf_receive: 11264 [pulselength 1101, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1136, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1134, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1136, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1134, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1136, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1136, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1137, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:16 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:17 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:17 - [INFO] rpi-rf_receive: 512 [pulselength 1134, protocol 4]
2022-07-24 21:41:17 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:17 - [INFO] rpi-rf_receive: 512 [pulselength 1135, protocol 4]
2022-07-24 21:41:18 - [INFO] rpi-rf_receive: 1 [pulselength 1285, protocol 4]
2022-07-24 21:41:22 - [INFO] rpi-rf_receive: 24 [pulselength 381, protocol 1]
2022-07-24 21:41:31 - [INFO] rpi-rf_receive: 3 [pulselength 1176, protocol 4]
2022-07-24 21:41:31 - [INFO] rpi-rf_receive: 5 [pulselength 1019, protocol 4]
2022-07-24 21:41:40 - [INFO] rpi-rf_receive: 2 [pulselength 1213, protocol 4]
2022-07-24 21:41:44 - [INFO] rpi-rf_receive:<PHONE_NUMBER> [pulselength 1028, protocol 4]
2022-07-24 21:41:45 - [INFO] rpi-rf_receive: 49156 [pulselength 1003, protocol 4]
2022-07-24 21:41:48 - [INFO] rpi-rf_receive: 128 [pulselength 1153, protocol 4]
2022-07-24 21:41:49 - [INFO] rpi-rf_receive: 36 [pulselength 969, protocol 4]
2022-07-24 21:41:54 - [INFO] rpi-rf_receive: 4718592 [pulselength 1032, protocol 4]
2022-07-24 21:42:00 - [INFO] rpi-rf_receive: 2 [pulselength 565, protocol 2]

Is this to be expected?
When I python scripts/rpi-rf_send -p 173 -t 1 533151, the output says 2022-07-24 21:44:18 - [INFO] rpi-rf_send: 533151 [protocol: 1, pulselength: 173, length: default, repeat: 10] but on the receiver side I get the message four times:
2022-07-24 21:44:18 - [INFO] rpi-rf_receive: 533151 [pulselength 175, protocol 1]
2022-07-24 21:44:18 - [INFO] rpi-rf_receive: 533151 [pulselength 175, protocol 1]
2022-07-24 21:44:18 - [INFO] rpi-rf_receive: 533151 [pulselength 175, protocol 1]
2022-07-24 21:44:18 - [INFO] rpi-rf_receive: 533151 [pulselength 176, protocol 1]

Is this to be expected?

hi there if you use -r in your command as I put below code 3 is 1 send code 5 is 2 sends :)


character slides with camera

Describe your bug here.
As you can see in the link video at the end, the character slides with the camera. how fix
https://www.youtube.com/watch?v=jEcUvBwrl1o
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Nope.

probably a scroll factor problem

probably a scroll factor problem

I did not mess with the scroll factor of zavok, but where would I find it? Would I edit it in the stage lua or?


2021 nissan rogue support

Please add support for the 2021 nissan rogue

Car support largely comes from the community. Check out the #nissan channel on our community discord.


No back to default button

A button so the settings can be set back to default would be nice.

See: "Restore settings to Wooting recommended"

I haven't notice it because it was just bare text. So from the UX this should be changed so it's obvious that this is a button which can be pressed.


Grove sensors v3

Nodes for three analogic sensors of Galileo Grove Kit that would demand some calculation over raw readings to be useful: temperature, light and rotary. The idea is to provide a simple way to use these analogic sensors.
v3:
Exposing more parameters on nodes, like potential divider resistor resistance;
Correctly using input range mask on formulae;
Using an exponential approximation (obtained from Grove Starter Kit booklet table, using least squares) for light sensor Lux measurements;
Grove fbp examples updated to use the new nodes;
lcd-grove moved to grove module.
v2:
More meaningful name for struct member
No more relative includes.

v5 on #76


Fix permissions for home/opuser

This resolves the issues with 2.6.2.
Overview

Changes the deployment chart to be permission 600 rather than 700 due to upstream changes.

Changes

Changes permissions to resolve issue.

Checklist

[x] review the guide to contributing

Closing temporarily due to further discussion

See https://github.com/1Password/op-scim-helm/pull/108


Plugin not found

Trying to install Plugin from:
https://boopeshmahendran.github.io/AdobeXD-maps/
which then opens a new tab to:
https://xd.adobelanding.com/en/xd-plugin-download/?name=e1e8f866
which then opens XD to try to install plug in. xd plugins window opens with the message "Plug in not found"

Same here, I even downloaded de .xdx file, but when I try to generate a map, I get the following message:

+1

Go to Release page to get direct download...

+1

Having the same issue, has anyone found a workaround?

Still no fix? I'm having the same issue

Have you guys tried using the https://github.com/boopeshmahendran/AdobeXD-maps/releases/ page?

@bolak2 yes, I did, the error is a bit different but still there.

Same here, I even downloaded and installed from the .xdx file, but when I try to generate a map, I get the following message:


Make ModioC.h valid C code

Put C++'s extern "C" behind __cplusplus-guards and insert void into parameter list of functions taking no arguments (-Wstrict-prototype).
Also removes a meaningless const on a return type. (-Wignored-qualifiers)

Great contribution @morlad ! Very helpful :smiley: :+1:


No SD card slot

help my phone does not have a SD card slot

Hi,
We're working on an updated version of the app that fixes this issue. It should be ready next week, please check back then.
Kind regards

Ohh Ic.Thanks for ur guys' effort to help the community.

2.0.0 fixes this issue.


Do selective tracing for client stream errors

This is popping up quite often while bringing a cluster online and it's not very intuitive for users.
Stream errors are received asynchronous to channel building, which makes them harder to debug. Ignoring it for common case of UNAVAILABLE should be safe.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4818/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4830/
Test FAILed.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.0 please.


In the store listing tab, add the apple review notes

Don't use the same apple review notes for all apps.

[x] Under developer accounts, make it possible to create 'apple review notes' templates, next to the default one that is currently hard-coded.
[x] Under the "Store Listing" tab, add a drop down to select the correct 'apple review notes'
[x] Create custom review notes for the MobieTrain and OSA Terminal apps
[x] Modify the deployment scripts in the apps repo

Closed by 2177f8b7fcbb4c677503fa53f265c998d2d27909


Require selecting a 'project' to submit to during the upload process

Once https://github.com/2i2c-org/unnamed-thingity-thing/issues/2 is done, creating a new submission will require it be associated with a project. This task is to update the Upload form to support this.
Scope
Since we can use the django admin to create projects, we do that for testing. Eventually we will add functionality to allow users to create and manage their own projects, but since our goal here is to keep scope small (in favor of https://github.com/2i2c-org/meta/issues/1370), this task is being done before giving users ability to create their own projects.
- [ ] Create a team (using the existing UI)
- [ ] Create a few projects associated with this team (using django admin)
- [ ] Show a dropdown with all the teams and projects current user is a part of in the upload form. Can be a single dropdown, with format `{project_name} ({team_name})`.
- [ ] Validate that the user has selected a project before accepting the upload.

Completion of https://github.com/2i2c-org/meta/issues/1370 will mark the end of the epic in https://github.com/2i2c-org/meta/issues/1370

@GeorgianaElena let's hold off on this temporarily - it looks like there may be more cutting of scope and this may have to change. In the meantime, let's work on https://github.com/2i2c-org/unnamed-thingity-thing/issues/9

I've received information that we aren't going to do this, so am going to close this now. We will have more information on wednesday


Add an iree_hal_local_executable_t shim that records profiling information

It may be useful to add some lightweight statistics to iree_hal_local_executable_t - the sum of issue_call times for a given ordinal is the total time spent across all threads/tiles in a particular method and then it can be printed out on executable destroy. We could also hack this into the legacy_library_loader which already has some tracing goo in it, so long as it's behind a compile-time flag.
The advantage of doing it here is that regardless of the scheduling system (multithreaded task, TBD singlethreaded, etc) we'd get the stats. The disadvantage is that if entire dispatch times were desired they won't be visible (issue_call only sees tiles) - but those times can be gotten using things like #45 instead.
This would mainly be to replace the need to microbenchmark individual executables as the timings here would be accurate for the real workloads/threading/data (especially important with padding/etc).

#45 is the right interface for this.


Augmentation

Hi,
I've noticed you add random noise to the audio. I just wondered if it would make more sense to add random noise to the Mel spectra? Considering that this is what you might get from something like tacotron, so noisy mels.

Hi, actually I didn't intend to augment the data by adding random noise to audio. I wanted to make sure that discriminator doesn't learn that raw audios with real values are fake, and discrete values from int16 are real.

Ah, cool trick. I've seen in the official implementation they randomly vary the amplitude a bit and call it "augment", that's why I was asking (https://github.com/descriptinc/melgan-neurips/blob/master/mel2wav/dataset.py#L66-L68)
Actually I'm currently investigating that even GTA data from Tacotron seems to be quite different from actual (non-teacher-fed) data. And my first intention was to generate GTA data multiple times - assuming that because of the huge Tacotron dropout used during inference they would also differ significantly between runs, but I'll have to confirm this first.
Then I remembered this augment thing and that I saw you using random noise.

That sounds insightful. Unfortunately, I couldn't do that kind of data augmentation since the mel-spectrograms are pre-processed at first.
I'm not sure whether MelGAN is robust against the discrepancy between ground-truth & tacotron2-generated mel-spectrogram. Could you please let me know if you get insight into this?

I have the same problem with the preprocessing, especially with the GTA mels of course, that I can't do this type of augmentation.
I'm currently training with the official implementation on GTA data but it still sounds a bit raspy and has a couple artefacts in it (epoch 1300 atm) in the actual synthesis (running the vocoder on the GTA features sounds much better).
That's why I'm thinking about approaches to deal with noisier spectra than the ones it sees during training. Perhaps just training from multiple speakers would help as well.

At the moment it seems adding your noise did help, but perhaps it was just the additional night of training ;).
Although sometimes taco still generates mels that result in a raspy voice, running in a second time usually solves it.

Sounds great. Shall we close this issue?

Yeah I'll report back if something useful comes up :)


'Preview' window implementation

Goal: Create a mini preview window on the bottom/side of the screen, showing the entire workflow.

Example: http://resources.jointjs.com/demos/kitchensink

feature is completed.


Investigate fissioning stencils

It would be great to have at least some support for fissioning stencils.
We'd have to figure out how to split a stencil computation. We could translate them into generate nodes, but that's not ideal. We could potentially split them into smaller stencil computations, but we'd have to work out how exactly to do that.

I think one thing that will be interesting to see when we actually write this out is whether the "stencil" operators themselves are sufficient for doing it, or whether it needs to be broken down into finer grained constructs like Dubach & co were doing.

This is also interesting because it is a case where the operation requires a single piece in (so that it can split the input being aware of the ghost region) but can return multiple pieces. I don't think we have any other examples with this behaviour. However, the implementation of stencil certainly can't make the decision of if and how and where to split the input; that information still needs to be fed in.


docker.image(..).inside(..) is not mounting the current workspace

Reproduction workflow:
stage('test') {
    node {
        deleteDir()
        checkout scm

        docker.image('ubuntu:18.04')
              .inside {
                sh """
                    pwd
                    ls
                """
              }
    }
}

Jenkins mounts its volumes into the newly created container. Is to be seen what volumes are mounted when Jenkins runs on the host.


Store Transfer Function as an Xarray

It would be convenient if the transfer function and covariances were stored in an xarray.DataSet.  The coordinate could be period or frequency, and dims or variables could be zxx, zxy, zyy, zyx, tzx, tzy, ...
I am suggesting this because this is what I'm trying to do with MTpy and mt_metadata.  I'd like core container for any type of transfer function that can have metadata attached.  Then we can translate to/from that core container.  Added a similar issue mt_metadata issue 57

We could have a attribute _Zij(self) that returns an xarray keyed with "Zxx", "Zxy","Zyx","Zyy",
then property Zij(self) returns this xarray, creating it if it doesn't already exist.

We want to have a view of a TransferFunctionCollection that is merged on decimation levels.
For now, this will be 3D, but can be 4D later with decimation level.
We will throw warnings about duplicate periods and default to keep the estimate the had more observations

@kujaku11
I added a method to TransferFunctionCollection called merge_decimation_levels()
This returns a dict "merged_dict" with the following entries:
dict_keys(['tf', 'tf_xarray', 'cov_ss_inv', 'cov_nn'])
If you access:
merged_dict["tf_xarray"] that has all the transfer function elements keyed as you suggest.
Can you take a look and see if that works for you?  I can reformat the covariances as well if we decide on nomenclature, but as it stands they can be accessed by keys of input_channel_1, input_channel_2 (for inverse signal covariance) and output_channel_1 and output_channel_2 for noise covariance

@kkappler Nice, I think this is what I was looking for.  Probably down the road the full transfer function with covariances could be a DataSet that share the coordinate period.
Should this transfer function object live in mt_metadata?  I was thinking that could be the central location where everything get put into a common container and would avoid circular imports.  For example in aurora it would be
from mt_metadata.transfer_function import core
Instead of in mt_metadata:
from aurora import TransferFunction

@kujaku11 Agreed -- The core data structure for TF should live in mt_metadata.   I think we do not want mt_metadata (or mth5) to have to import from aurora.
Note that the covariances already exist as xarray Datasets with coordinate "period" in the merged_dict.  I was tempeted to create a class called
MergedTransferFunction

or something like that to use instead of the dictionary.    But it seems like what I actually want is
from mt_metadata.transfer_function.base import TransferFunction as MergedTransferFunction

Then an instance of mt_metadata's transfer function would get populated and returned by
TransferFunctionCollection.merge_decimation_levels()

@kkappler That sounds good.  I will try to build that code by Friday starting from what you have.

I confirm that aurora tests are passing using the main branch of mt_metadata after the fix_2 merge.


Add setting for Grass/Pot CAMC

Separates the Grass and Pot CAMC out from the rest and makes it its own setting, still affected by Stone of Agony if enabled. This enables the ability to have the usual CAMC on without altering the behavior of all grass and pots being gold until checked, or having only Grass and Pot CAMC.

Is CAMC a typo of CSMC (which seems more common?)

Is CAMC a typo of CSMC (which seems more common?)
(Apologies if this is a dumb question - I'm new here but I'm interested in contributing!)

Not a typo. Container Appearance Matches Contents. We dont separately alter the size and texture, so it makes more sense to call it more accurate to what it does.

I would recommend splitting it per container instead of coupling them. It would be inconsistent to have groups of specific containers.


add scripts for aqicn import

import doc: https://docs.google.com/document/d/1fq3MoUfzFCVM3JgEJr-bvOh8liyDzW-w8JRfI9gS1-s/edit#

Are we planning on setting this up for auto-refreshes?

Are we planning on setting this up for auto-refreshes?

thanks for the review! and yep, I think the goal is to add auto-refresh for this once it's all set up with the custom dc


game.data.version is deprecated in V9 in favor of game.version

FYI you've got a reference to game.data.version in rolladvantage-token-stamp-2-foundry.js.
This is causing a warning message in the console when your module is included and the game world is launched or relaunched with Foundry V9 build 242 (probably any V9 build)
You should be able to safely replace 'game.data.version' with 'game.version' to get the behavior you want without the warning message, though this may cause an issue for folks not using recent builds of FVTT.

Yeah there was a minor change to make it compatible with v9 so I didnt want to release two version and want to maintain compatiblity as much as possible for now. The game.data.version check is only relevant for <v9 versions, but I guess I need to add a null check if it's being removed.
Thanks for the heads up though, I only briefly tested on v9 as I'm using v8 still while my campaign finishes.


Corrected some grammatical errors

I rectified certain grammatical errors in the README file

@avinash201199 kindly review my PR #49

Thank u
On Thu, Oct 14, 2021, 1:20 PM Avinash Singh @.***>
wrote:

Merged #49 https://github.com/avinash201199/Hacktoberfest-Guide/pull/49
into main.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avinash201199/Hacktoberfest-Guide/pull/49#event-5461932930,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATNE6QCXHEIXSBRM6CQ43WTUG2DS3ANCNFSM5F667DJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Add ability to set a class on list view rows, use customScope

Set a class specified by the row item on the row’s container
(list-group-item).
Adds a customScope  parameter to allow the transcluded html to use
variables or functions defined by the applications scope.
@dtaylor113 @dgutride

This looks good to me.

LGTM


WebRTC stream does not work for some web browsers

Which version are you using?
v0.21.0
Which operating system are you using?

[x] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
When running the RTSP server with WebRTC enabled, I am not able to access the video stream from all web browsers. In my experience, Google Chrome works, but Mozilla Firefox doesn't. Logs hint at a possible difference between IPv4 and IPv6.
Describe how to replicate the issue

Start the server with default parameters (WebRTC enabled).
Start an RTSP stream from a camera (e.g. laptop webcam) using FFMPEG, in rtsp://<IP_ADDRESS>:8554/mystream.
Open http://localhost:8889/mystream in Mozilla Firefox --> stream is NOT visible
Open http://localhost:8889/mystream in Google Chrome --> stream IS visible

Did you attach the server logs?

Yes
webrtc-ipv4-ipv6.txt
Did you attach a network dump?

No

Windows OS - Microsoft Edge. Stream "sometimes" opens but sometimes doesn't, sometimes takes up to 1 minute to load. Works always immediately in Chrome.

Hello, this has been fixed by 777860f7576dae5ad700c80a7842733811c6bcad.

thank you!


[fix] (inverted index) fix query errors caused by ignore_above

Proposed changes
pick from master #37679
Issue Number: close #xxx

run buildall


Group equivalent or extension of snapshot & restore

First, thanks to all for the great work on this project.
Issue:  Could snapshot and restore be extended (or complemented by additional functions) to capture the grouping states of chosen zones and then restore them at a later time?
If you want to jump straight to my code, it's here: Sonos Doorbell
My current solution works by querying and creating a descriptor for each zone of the initial grouping configuration and then later regrouping based on those descriptors. I can come up with six different categories of zone grouping configurations to be managed. They're shown in code comments.
This is my first real programming experience since Commodore BASIC! Please fill free to tell me how it  could be made faster (especially), simpler, more elegant or more pythonic. I'm learning as I go.
Background:
The use case I have in mind--like several others on here--is to pause selected zones which may be grouped in any number of combinations, play a doorbell sound through them and then restore not only the playing state (volume, track if coordinator, etc.) but also to regroup them as they were originally--all without interrupting whatever the other players are doing at the time. Of course, TTS and other alert use cases would have similar requirements.
Compared to other implementations using snapshot and restore that I can find, the complications for my scenario are:

The doorbell should be played on selected zones only, e.g., not in rooms where children may be sleeping or on a Playbar that may be playing TV audio. For my scenario at least, the "doorbell zones" vs. "non-doorbell zones" are fixed and do not change. Party mode or playing on all zones regardless of group configuration are not options.

Prior to playing the doorbell, zones in the house initially may be grouped in any number of possible combinations, they may be grouped all together in party mode, or none may be grouped. Groups could be composed of all doorbell zones, all non-doorbell zones or mixed zones. Initial group coordinators could be either doorbell zones or non-doorbell zones.

Invisible zones like half of a stereo pair or a SUB which is slave to a Playbar should be ignored as far as group processing goes. (As is, the grouping Boolean value returned by snapshot includes invisible zones.)

Non-doorbell zones should not change state (stop playing, be ungrouped or whatever) when other zones play the doorbell sound.

Groups should be restored after the doorbell is played.

Hi, a  generic version of what you propose would be very useful for me and many others. Many are seeking to integrate Sonos into an intelligent home by using it for announcements, door bell, etc.
This is where the snapshot code came from, as a part step.
I have tried several approaches with groups and I believe it can be done but there are some challenges.

the time it takes to discover the current state of each zone, group for playing the alert can mount up. Code is very fast - even on a Raspberry Pi, its the network traffic time that mounts up depending on your network. I have currently opted not to group the zones, just to play an alert on each coordinator. Yes there can appear a bit of an echo but it works faster.

If you leave some zones playing it may be that it's coordinator is required for the announcement, hence you have to pass off what is playing to the slave. Quite doable, just another complexity and more time.

you have to group and ungroup in the right order or you can get errors.

I have tried making grouping efficient by looking for the biggest current group and then bringing other zones into that - this can speed up grouping a lot.
The other approach I have tried it to have a Raspberry Pi running permanently, and subscribing to events from all Sonos zones. This means when an alert needs to be played I already know the current groups and what is playing on each zone.
Looking forward - ideally we could get to something like:
mysonos.alert(zone_list, alert_stream, resume=No/Yes/Fade, other_zones=ignore, mute)

Just my thoughts and experiences.
Happy to help create something - lets get others thoughts and ideas.

I therefore use party mode and save each zone, save the groups, play partymode on the biggest group and then ungroup and group and  restore after playing.
i think you need to consider that some TTS messages have to be played "no matter what", even when children are sleeping.   Like the fire alarm, intruder detection,..
In my raspberry based alarm system, i configured that normal notifications are ignored after evening time and before morning time or when the home alarm is set to sleeping or is fully armed.
I also have a TTS_no_voice flag that i can set with my phone, to skip normal notifications for specific situations that may arise.
But i certainly would like to have the mysonos.alert function as depicted above that would be faster and allow to specify on what zones you would like to play instead of everywhere with party mode.
Kind Regards,
Rudy

Thanks for the comments, guys. I'm glad that I'm not the only one who would like to see this feature.
DPH > 2. If you leave some zones playing it may be that it's coordinator is required for the announcement, hence you have to pass off what is playing to the slave. Quite doable, just another complexity and more time.
The time lag is an annoyance, but I think that I'm getting around that by checking whether one of the required announcement players is currently a coordinator and, if so, doing one of two things:

If it's coordinator of  a mixed (announcement and non-announcement zone) group, note the IP of a non-announcement group member to use for later querying the current coordinator and rejoining to it. Obviously, the group coordinator is then different to what it was prior to the announcement, but I haven't see that to be a problem.
If it's coordinator of an announcement-only group, simply rejoin its slaves to it later.

Would it be possible to regroup based on the group ID rather than using this roundabout way of finding the current coordinator?
DPH > 3) you have to group and ungroup in the right order or you can get errors.
It may be blind luck, but I haven't run into this problem my the current code. I was getting lots of UPnP errors which seemed to be related to grouping in some way with some of the other methods I tried. Any pointers as to what's right or wrong re order?
DPH > The other approach I have tried it to have a Raspberry Pi running permanently, and subscribing to events from all Sonos zones. This means when an alert needs to be played I already know the current groups and what is playing on each zone.
I hadn't thought of this. I've also got a Pi running all the time for the doorbell. Do you have any code you could share or any pointers with regard to subscribing? I'd like to try it. I think that would really speed up my main annoyance, the time between someone pressing the doorbell until the alert sound plays.
DPH > Looking forward - ideally we could get to something like:
mysonos.alert(zone_list, alert_stream, resume=No/Yes/Fade, other_zones=ignore, mute)
I like this approach instead of separate group snapshot and restore commands.
rudyvan > i think you need to consider that some TTS messages have to be played "no matter what", even when children are sleeping. Like the fire alarm, intruder detection
That's not an issue for my current use case, but I can see how it would be for different types of alerts. I suppose that you could have different zone lists which apply based on different sorts/priorities of alerts.
Regards,
Ron

Hi @ronschaeffer I have some very basic code that subscribes to all sonos zones and maintains state.
It needs more work, and there may be better ways to implement this.
Ideally it would be a Cache of all zones states which could be accessed by other modules - need to debate best way to do that.
Have attached my code as is - works on Python 3.5 (may work on 2.7 but would need to change .items to .itteritems at least)
https://gist.github.com/DPH/3db3092b2b989b92c6a9ccfb66ca2865
This shows using subscribe  - also good simple examples in the events module of SoCo
Shout if any questions or ideas
Cheers David

Thanks a lot, @DPH. I'll take a look.


Retain disk name in portal conversion?

Is there a way to retain the disk name without appending a random string? This is messing with our ARM templates.

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

ID: d544e55b-bac1-0e3c-a396-90a1b4f6e1aa
Version Independent ID: 018208b2-7479-746d-2d12-a9e3c78036d7
Content: Convert a Windows virtual machine from unmanaged disks to managed disks - Azure Managed Disks
Content Source: articles/virtual-machines/windows/convert-unmanaged-to-managed-disks.md
Service: virtual-machines-windows
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the question. We are currently investigating and will update you shortly.

@liutang This is by design to append the string and there is no way to rename it. The only workaround at this point is to duplicate the disk and use a new name.


Terminal rendering breaks after waking from sleep

2 team reports today. Can reproduce by sleeping the PC on Windows and waking up:

xterm.js side fix: https://github.com/xtermjs/xterm.js/pull/4180


feat!: support public rsc pin read, using manual board

Adds support for reading public RStudio Connect pin content, using board_manual.
Cleans up how board_manual constructs paths to pins.

@isabelizimm let's look over this PR tomorrow, and see if it works for accessing public RSC pins in vetiver (we may need to tweak vetiver's from_pin method)

The failing test should be resolved by #155 -- going to merge, so we can release in a little bit


fix(deps): update terraform terraform-ibm-modules/cos/ibm to v8.2.0

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
minor
8.1.10 -> 8.2.0

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.2.0
Compare Source
Features

Surface ability to configure object locking  (#593) (4146d09)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

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

GitHub release
v0.4.5

Your semantic-release bot :package::rocket:


Ability to publish "snapshot" branches in subfolders

This is in progress. The idea is to be able to have separate branches which appear as complete "snapshots" of the website in a particular subfolder. This is to allow historical views of the project, and also to allow previews of upcoming changes before they are live.

This is mostly done. Improvements still needed:

Snapshot builds to be triggered when the snapshot branches are pushed to
Indicator import to work on the snapshot builds

I think this is working well enough already, so I'll close.


Remove deprecation for atom 0.189.0

Thanks for the pull request. I'm currently interacting with the ContextMenu API through JS only, and have deleted menus/color-picker.cson.


dc sync falls back to slow consume api

DC sync falls back to slow consume api whenever the chunked response exceeds 8MB (akka default).

Tested.


TypeError: core_1.utils.tryToMutePuppeteerWarning is not a function

In trying to upgrade from @memlab/api 1.0.23 -> 1.0.24 the following error message occurs in the CI of the eye-js project I maintain.
TypeError: core_1.utils.tryToMutePuppeteerWarning is not a function
    at Object.<anonymous> (/home/runner/work/eye-js/eye-js/node_modules/@memlab/api/dist/lib/APIUtils.js:32:22)
    at Generator.next (<anonymous>)
    at /home/runner/work/eye-js/eye-js/node_modules/@memlab/api/dist/lib/APIUtils.js:17:71
    at new Promise (<anonymous>)
    at __awaiter (/home/runner/work/eye-js/eye-js/node_modules/@memlab/api/dist/lib/APIUtils.js:13:12)
    at Object.getBrowser (/home/runner/work/eye-js/eye-js/node_modules/@memlab/api/dist/lib/APIUtils.js:29:12)
    at /home/runner/work/eye-js/eye-js/node_modules/@memlab/api/dist/API.js:236:54
    at Generator.next (<anonymous>)
    at /home/runner/work/eye-js/eye-js/node_modules/@memlab/api/dist/API.js:17:71
    at new Promise (<anonymous>)

This occurs when this script is run as triggered by this command.

@jeswr @memlab/api 1.0.24 is using a newer version of @memlab/core which includes a new API (tryToMutePuppeteerWarning). Please consider upgrading all @memlab/* packages to the latest version.

Indeed just npm upgrading everything seems to work. Thanks @JacksonGL !


Error compiling a Blazor 'client-side' project on .NET 5

Version: Electron.NET Tools 11.5.1
.NET version: 5.0.1
Target: Win 10, desktop, x64

The error is:
NETSDK1098: Applications published to a single-file are required to use the application host.
You must either set PublishSingleFile to false or set UseAppHost to true.

Full console text:
Start Electron Desktop Application...
Microsoft Windows [Version 10.0.19042.746]
(c) 2020 Microsoft Corporation. All rights reserved.
C:\...\tests\BlazorApp9>dotnet publish -r win-x64 -c "Debug" --output "C:\...\tests\BlazorApp9\obj\Host\bin" /p:PublishReadyToRun=true /p:PublishSingleFile=true --no-self-contained
Microsoft (R) Build Engine version 16.8.3+39993bd9d for .NET
Copyright (C) Microsoft Corporation. All rights reserved.
  Determining projects to restore...
  All projects are up-to-date for restore.
C:\Program Files\dotnet\sdk\5.0.102\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(145,5): error NETSDK1098: Applications published to a single-file are required to use the application host. You must either set PublishSingleFile to false or set UseAppHost to true. [C:\...\tests\BlazorApp9\BlazorApp9\Client\BlazorApp9.Client.csproj]
  BlazorApp9.Shared -> C:\...\tests\BlazorApp9\BlazorApp9\Shared\bin\Debug\net5.0\win-x64\BlazorApp9.Shared.dll
C:\Program Files\dotnet\sdk\5.0.102\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Publish.targets(96,5): error NETSDK1099: Publishing to a single-file is only supported for executable applications. [C:\...\tests\BlazorApp9\BlazorApp9\Shared\BlazorApp9.Shared.csproj]
  BlazorApp9.Shared -> C:\...\tests\BlazorApp9\BlazorApp9\Shared\bin\Debug\net5.0\BlazorApp9.Shared.dll
C:\...\tests\BlazorApp9>
Error occurred during dotnet publish: 1

I am seeing this error and couldn't find a workaround

I am seeing this error and couldn't find a workaround

you can pass in overrides on the PublishSingleFile when running electronize build.
I believe its of the form like this.
electronize build /target osx /PublishSingleFile false

you can pass in overrides on the PublishSingleFile when running electronize build.
I believe its of the form like this.
electronize build /target osx /PublishSingleFile false

you can pass in overrides on the PublishSingleFile when running electronize build.
I believe its of the form like this.
electronize build /target osx /PublishSingleFile false

electronize build /target win /PublishSingleFile false
gives me
error NETSDK1095: Optimizing assemblies for performance is not supported for the selected target platform or architecture. Please verify you are using a supported runtime identifier, or set the PublishReadyToRun property to false.
Instead,
electronize build /target win /PublishSingleFile false /PublishReadyToRun false
works.
But,
electronize start /PublishSingleFile false /PublishReadyToRun false
doesn't, because of --no-self-contained that generates this error: error NETSDK1102: Optimizing assemblies for size is not supported for the selected publish configuration. Please ensure that you are publishing a self-contained app.
So, it's usable, but for a definitive result I suggest to modify parameters in case of Blazor, for both start and build.

I've got a PR in #531 to expose all the other compile time properties /property values to the electronize command.  Does this solve your issue?

This pr does not fix the issue when dealing with Blazor wasm projects debugging via Visual Studio (that use the second project for hosting via ASP.NET Core). To recreate the issue, you can create a new Blazor webassembly project, select ASP.NET Core hosted in the options when creating the new project and in the new solution there will be 3 project (Client, Server, Shared).  Set the Server project up normally using ElectronNet.Api and electronize init/start. Running it you'll see the issue.
If I remember correctly the publishreadytorun and publishsinglefile both need to be false in this case and I'm not sure where to set that so that when the debugging is started via Visual Studio that they would get passed.  If there is a way to set that, then the pr might work.

I encountered the same issue.
If I understood correctly there is no workaround for starting application without generating executable file?

Has anyone been able to get the "start" command to work? Is there some combination of flags that can be used as a workaround?

As a workaround, I had in mind building the Blazor WebAssembly app independently and just including the published outputs with the electronized server app. Would that be possible?

Hi all, I just decided to try testing blazor in electron and see this same problem.

Hi all, I just decided to try testing blazor in electron and see this same problem. I thought made I was doing something wrong until I found this thread.

Hi Guys, in order to debug, just run from your electronized project in a CMD or powershell:
electronize start /PublishSingleFile false
Then find the project name using Visual Studio and go to DEBUG => Attach to process => and find the name of your project, then press ATTACH
Happy debugging

@cristiangiagante I still get this error when following your directions.
In PowerShell I'm executing this command. electronize start /PublishSingleFile false
error NETSDK1102: Optimizing assemblies for size is not supported for the selected publish configuration. Please ensure that you are publishing a self-contained app.

What is your issue when you try to run it normally? pls share your results

Either way I run the start command I get the same error as noted in my previous comment. I've tried running start from the command line as well as creating the debug profile as you describe. I can add a screenshot of the error in the console window if that would help.

Let start using a stable release, update your project to target Net6 instead of net6 preview

Here is the same error again using the stable version of VS and Net

5.

I'm facing the same issue, just created a blazor web assembly with asp.net backed and added the elctronNet and facing the same issue

ok, I will take a look later but seems to be you are currently using the sdk/6.0.100-preview

Oh, I see what you're saying now. I switched from using the VS2019 Preview to the most recent stable version of VS. How do I get it to use the Net5 SDK?
This line here might be part of the problem.
dotnet publish -r win-x64 -c "Debug" --output "BlazorElectronizedSln\obj\Host\bin" /p:PublishReadyToRun=false /p:PublishSingleFile=false --no-self-contained

@cristiangiagante  I am using sdk 5.0, I can create a new repo so you can try it out

I'll figure out how to make mine use SDK 5.0. Maybe that will help me.

https://github.com/mhamri/ElectronNet-blazor-bug
I have v5.0.2 sdk installed on my laptop

i just installed everything yesterday, so it should be whatever available. this is what I have available

as a workaround I created a blazor server side, this definitely increase the bundle size, but it worked

Sorry I totally forgot this, I checked your repo, I was not able to do it work but I know how to do it. You need to create your own NuGet package based on a fork of electronnet and replace the --no-self-contained in the build commands class.

but do you fix it for later version?

I don't think so this is not my project, I can't update the official nuget repository

oh, thankx for the help anyway

I'm unsure if this'll help anyone... but it seems as-if you execute your blazor app using the following command, it fires as you'd expect it to: electronize start .\source\Your.Host\Your.Host.csproj /PublishSingleFile false /PublishReadyToRun false

🎉🚀 New Electron.NET version 23.6.1 released 🚀🎉
With native Electron 23 and .NET 6 support. Your problem should be fixed here. If you continue to have the problem, please let us know. Please note the correct updating of your API & CLI. Info in the README. Have fun!

Somehow I have the same problem with a brand new .net8 Blazor app that has Client assembly. Fresh install of Electron.NET CLI and <PackageReference Include="ElectronNET.API" Version="23.6.2" />
So, in launchSettings.json

if I have

"commandLineArgs": "start",

it tries to publish with  /p:PublishReadyToRun=true /p:PublishSingleFile=true --no-self-contained and fails with Applications published to a single-file are required to use the application host. You must either set PublishSingleFile to false or set UseAppHost to true.

if I have

"commandLineArgs": "start /PublishSingleFile false"

it tries to publish with /p:PublishReadyToRun=true /p:PublishSingleFile=false --no-self-contained and fails with Optimizing assemblies for size is not supported for the selected publish configuration. Please ensure that you are publishing a self-contained app.

if I have

 "commandLineArgs": "start /PublishSingleFile false /PublishReadyToRun false",

it fails to pack the client project with error WASM0005: Unable to resolve WebAssembly runtime pack version.
What am I still missing here?


zIndex gets set to initial on modals with takeover option set to false

The zIndex only needs to get set on modals with the takeover option set to true. The zIndex should be left alone if takeover is set to false.
Currently, if you have a z-index for your modal in your CSS it'll get overwritten which isn't what we want to happen.
You can get around this by adding !important to the z-index in your CSS.

Fixed with a447fb46ae0fda2602e4f2b3b471594dd48500d6


Toolbar cleanup

Cleanup toolbars by extracting branching logic into helper methods, extracting repeated stuff and removing unneeded partials.
Finally remove defered_toolbar_render and simplify the flow of execution around toolbar selection and rendering.

Although PR this does not include new tests, it actually improves our test coverage measured by number of lines a bit :-D and if you measure test coverage in relation to number of files in the project then it's even better ;-)

@himdel, @skateman : you two are sort-of-familiar with this area, can you review, please? When done, please, ping @h-kataria or @dclarizio for merge. Thx!

@himdel : thx for the review!

LGTM! (though there is one more .send(:restful?) in there :))

Removed the .send.

@h-kataria , @dclarizio : can press the magic button?


Support reporting errors/warnings when detecting duplicate fields

Related to #570
As can be read in YAML 1.2 spec :

JSON's RFC4627 requires that mappings keys merely “SHOULD” be unique, while YAML insists they “MUST” be. Technically, YAML therefore complies with the JSON spec, choosing to treat duplicates as an error. In practice, since JSON is silent on the semantics of such duplicates, the only portable JSON files are those with unique keys, which are therefore valid YAML files.

As the recommendation of JSON specifies that fields SHOULD be unique; it would be nice to have an option to enforce this; especially for consistency between JSON and YAML parsers (e.g. both would return errors on duplicate fields consistently).
yaml.v3 errors out if there are duplicate fields. It can be very non-obvious how parsing is done if duplicate fields don't yield an error, e.g. encoding/json where duplicate fields with object values are merged, whereas all other types are replaced with the latest value (ref: https://github.com/golang/go/issues/24415#issuecomment-373513285):

The documentation doesn't specify what happens in this case; it says it matches object keys to fields but doesn't specify in what order it does so (it only matters with duplicate keys). Changing the current behaviour might break backwards compatibility too much, but I think at least it should be documented.
Also, the behaviour is inconsistent between a map and a slice field; the maps get merged, the slice field takes the last value seen, like the other types.

Here is that comment behavior in action: https://play.golang.org/p/CkCMdoGXc07
This was unexpected to me, and I believe most users, too.
I have an implementation of this fix, providing an opt-in method to enable errors on duplicate fields: https://github.com/luxas/json-iterator/tree/error_for_duplicates
But, I think it should be possible to write this as a json-iter extension as well; I don't know which one is better for the community in the long run (i.e. feature creep vs extensibility).
If json-iter supported "warnings", it'd also be possible to begin the transition to duplicate fields erroring by showing the user warnings (in the spirit of https://kubernetes.io/blog/2020/09/03/warnings/)

Go also has an open issue for this: https://github.com/golang/go/issues/48298


Manifest allowBackup issue

Hi ,
I'm using AS 3.2 RC 3 to try.After adding the library gradle that synced. But I got an error;
Attribute application@allowBackup value=(false) from AndroidManifest.xml:10:9-36
is also present at [com.ramotion.paperonboarding:paper-onboarding:1.1.1] AndroidManifest.xml:12:9-35 value=(true).
Suggestion: add 'tools:replace="android:allowBackup"' to <application> element at AndroidManifest.xml:9:5-40:19 to override.

Hello, @GorkemKarayel. Add tools:replace="android:allowBackup" to your app's Manifest element.


[risk=no] Address jackson security alert

Addresses
CVE-2019-12814
More information
moderate severity
Vulnerable versions: >= 2.0.0, < <IP_ADDRESS>
Patched version: <IP_ADDRESS>

A Polymorphic Typing issue was discovered in FasterXML jackson-databind 2.x through 2.9.9. When Default Typing is enabled (either globally or for a specific property) for an externally exposed JSON endpoint and the service has JDOM 1.x or 2.x jar in the classpath, an attacker can send a specifically crafted JSON message that allows them to read arbitrary local files on the server.

Coverage decreased (-0.5%) to 73.623% when pulling db7d21f9c68b3ce378d62ad9155899d037d28996 on jackson-upgrade into d07f7229c6400e5cc5dfb71af4a9c5aa610d7e46 on develop.


Cleanup, polish, e2e tests, dir sync, exec

Professional code barf.

[x] finish the dir sync
[x] docs

Ready for review.
=== 12 passed, 33 warnings in 22.73 seconds ===

I would prefer to merge it and continue with other work.


Troubleshooting OS tabs should be syncronized

Changing tab from Linux to macOS should alter all the tab instances.

Im not quite sure how to implement this without using some higher level state management store like redux.
Since these components can be quite nested and at very different points in the DOM im not quite sure how to cleanly sync the state of all these up. Any thoughts on how to best do this?

#4 I added some initial local OS detection but not quite sure about syncing these elements

Im not quite sure how to implement this without using some higher level state management store like redux.

React has something called Context API which can be used exactly for this. Feel free to try it or I can implement it :)


Add code to derive from ParMesh.

These are additions I needed  to derive from ParMesh for an existing parallel decomposed mesh.

Current coverage is 7.61%

Merging #50 into develop will not affect coverage as of a36ac6e

@@            develop     #50   diff @@
=======================================
  Files            95      95
  Stmts         28803   28803
  Branches          0       0
  Methods           0       0
=======================================
  Hit            2193    2193
  Partial           0       0
  Missed        26610   26610

Review entire Coverage Diff as of a36ac6e

Powered by Codecov. Updated on successful CI builds.

Hi @koning, a few questions:

Why do you need a virtual destructor for GroupTopology are you planning to inherit from it?
Shouldn't the ParMesh() constructor at least initialize some data?
Can you please replace {;} by {}? (Just commit in koning:develop to update the pull request.)

Thanks!
Tzanio
P.S. It might be more convenient for you to work with the internal LLNL repo, https://mystash.llnl.gov/projects/mfem.

Why do you need a virtual destructor for GroupTopology are you planning to inherit from it?
That is a potential,  it was forward looking change at this point.

Shouldn't the ParMesh() constructor at least initialize some data?

Yes, but since this is still new to me I thought I would elicit so comments like this one. It will call the Mesh default ctor, so which of the other objects need defaults? I added the MPI member variable defaults.

Can you please replace {;} by {}? (Just commit in koning:develop to update the pull request.)

Done.

Hi @koning, I think we should make this default constructor protected, so it can be called only from derived classes. Also, ParMesh has some public data members - have_face_nbr_data, and pncmesh that should be initialized too.

Thanks Joe, merged (also internally).


feat: separate starship installation by using fixed image

Signed-off-by: Kaiyang Chen<EMAIL_ADDRESS>

LGTM. Shall we merge the CI part to release the image or do it manually before merging the rest of this PR? @gaocegege

I think we can release a new version before the PR, then merge it to let CI release a new image. WDYT

LGTM. Shall we merge the CI part to release the image or do it manually before merging the rest of this PR? @gaocegege

I think we can release a new version before the PR, then merge it to let CI release a new image. WDYT

LGTM

https://github.com/all-contributors please add @Kaiyang-Chen for the code contribution.

@all-contributors please add @Kaiyang-Chen for the code contribution.


Create collection failed for specific directory

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
Hello,
I just wanted to create a new Brun-collection but it failed for directory
C:\Users\xxxxxxx\OneDrive - PEAC (Germany) GmbH\XxX\GitLab-Repositories\webapi_test
So I mounted the directory C:\Users\xxxxxxx\OneDrive - PEAC (Germany) GmbH\XxX as network drive U:. After that the creation of the collection for directory U:\GitLab-Repositories\webapi_test succeeded.
In my understanding Bruno has difficulties to handle:

long path names
spaces in path names

Can you pleas check it.
Kind Regards
Thomas
PS: Opening a collection at C:\Users\xxxxxxx\OneDrive - PEAC (Germany) GmbH\XxX\GitLab-Repositories\webapi_test works fine.
.bru file to reproduce the bug
No response
Screenshots/Live demo link

I have a similar problem with parenthesis in the collection's path. While creating a new collection, if the are parenthesis in the "Location" field as shown in the first screenshot, the collection cannot be created ans an error message is displayed as shown in the second screenshot.

If I remove the parenthesis, the collection is created successfully.

If I try to use parenthesis in the "Folder Name" field, I get a proper warning.


分支权限

现在有没有分支权限啊？分支权限怎么搞？

I think this is duplicated to https://github.com/gogits/gogs/issues/776


Reduce memory use through better attention?

https://pytorch.org/blog/out-of-the-box-acceleration/
Are we utilizing the "fused kernels from FlashAttention and Memory-efficient attention"?  Can we?  We may be able to have significant speedups or to save significant memory that way.

This seems like it would be pretty useful for our purposes since we're training large models. The pytorch blog says they're seeing GPU memory savings of 20%-110% during training, speedups of 10%-70% during training, and speedups of 5%-20% during inferencing. I'd like to test this in SILNLP, but it's still using torch 1.10 rather than torch 2.0, which is required for the accelerated attention mechanism in BetterTransformer. I think it might be best for me to make a new branch in SILNLP with torch 2.0 and test using BetterTransformer there. If it does provide a significant speed/memory improvement for our models, I'd imagine we'd want to upgrade the master branch of SILNLP with torch 2.0 and BetterTransformer too.

The upgrades from BetterTransformer will be incorporated into future versions of transformers and should eventually cover the M2M100 that we use.

This should be auto-done when it is put into Transformers natively.

@mshannon-sil  - this is resolved, if I am correct?  Are we using flash attention (or did some other thing get in the way of using it)?

We haven't updated to the version of HF transformers that supports SDPA yet.


tests: Use libvmi instead of 'NewRandomVMIWithEphemeralDiskAndConfigDriveUserdata

What this PR does
Use libvmi directly at the caller instead of NewRandomVMIWithEphemeralDiskAndConfigDriveUserdata.
Before this PR:
After this PR:

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/sig quality

/sig code-quality

@orelmisan @fossedihelm PTAL :slightly_smiling_face:

/retest-required

/hold
@ormergi - it looks like there are two tests in the compute lanes that are failing consistently on this PR. Placing a hold just to stop needless retests.

rebased

/close


Namespaces returns owners only, not read access

When doing a GET on https://hub.docker.com/v2/repositories/namespaces
it returns only namespaces in which members are owners, rather than read access. Thus causing this code block to raise an exception incorrectly
https://github.com/docker/migrator/blob/master/migrator.sh#L430-L436

Ah crap, you're right.  I'll have to see what API I might have access to to check what organizations a user has any access to.  In the meantime, I'll put in a PR to bypass the check as you're definitely right.

Thanks for quick response!

Apparently this should return what orgs a user belongs to: https://hub.docker.com/v2/user/orgs/

Looks like this will fix it...change https://github.com/docker/migrator/blob/master/migrator.sh#L431 to:
NAMESPACES=$(curl -sf -H "Authorization: JWT ${TOKEN}" https://hub.docker.com/v2/user/orgs/ | jq -r '.results|.[].orgname') || catch_error "curl => API failure"

@mbentley looks good, thanks for quick fix


Testing the button click code coverage

Codecov Report

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

@@           Coverage Diff           @@
##             master    #11   +/-   ##
=======================================
  Coverage          ?   5.2%
=======================================
  Files             ?      6
  Lines             ?     96
  Branches          ?      4
=======================================
  Hits              ?      5
  Misses            ?     91
  Partials          ?      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 ce25899...b64bfd1. Read the comment docs.


A server, how does the front end access the back end?

When I want the front-end to apply for the back-end API, it accesses the local 40443. How should I let it access the server?
My front-end and back-end are both on the same server

Thank you for reporting this.  The endpoint you've identified is the SignalR hub used by the REST API backend to broadcast to multiple clients.
Can you confirm that https://localhost:40443/healthz is available (or the swagger UX: https://localhost:40443/)?

Thank you for your answer. After I successfully deployed it on my computer windows system, my computer can access http://localhost:3000/, but I cannot access it using my mobile device. It will display the same content as the screenshot above.

And I found that there will be authentication explosion, I don’t know why, (this picture is a screenshot of my access to the server).

I'm very sorry for not replying to your question. Since I deployed it on the server, the URL https://localhost:40443 cannot be accessed.

Because I want to use it in a production environment, I want to know how to modify it.

This may be a bit direct, but you should already understand these concepts before deploying a complex application like this. There will be security vulnerabilities if you don't know how to properly configure and secure it.
That being said, the front end (on your phone) has been told to look for the backend at 'https://localhost:40443". This resolves to the phone, which has nothing running on 40443 (probably).
You'll need to edit REACT_APP_BACKEND_URI=https://localhost:40443/ in webapp/.env to point to your server hosting the backend.
Example:
REACT_APP_BACKEND_URI=https://chat-copilot.contoso.com:40443/ or
REACT_APP_BACKEND_URI=https://<IP_ADDRESS>:40443/

After I modify it according to your tips, the authentication fails and the backend cannot be accessed. Is it because of an SSL problem? is there any other reason. (You do not need to apply for an SSL certificate when deploying locally)

I am very happy because I have used the nginx proxy backend to enable front-end access, and I found that I need to remove "{}" when setting the key. Anyway, thank you very much for your help.


Display type of response_field

I have defined my response fields like this:
response_field :id, "ID of the logged in user", "Type" => "Integer"
response_field :email, "E-mail address", "Type" => "String"
response_field :first_name, "First name", "Type" => "String"
response_field :last_name, "Last name", "Type" => "String"
response_field :role, "Role of the user.  This can be: trainee or company_admin", "Type" => "String"
response_field :language, "Language of the user.  This can be: nl or fr", "Type" => "String"
response_field :id, "ID of the company the user belongs to", "Type" => "Integer", :scope => :company
response_field :name, "Name of the company the user belongs to", "Type" => "String", :scope => :company
response_field :id, "ID of the function of the user", "Type" => "Integer", :scope => :function
response_field :name, "Name of the function of the user", "Type" => "String", :scope => :function

In the generated documentation the types aren't being displayed.  Why is this?

Which format are you using and are you using a viewer like raddocs or apitome?

@oestrich  I generate the docs like this:
rspec spec/api/ --format RspecApiDocumentation::ApiFormatter

and view them in the browser.  config.format = [:json, :combined_text, :html]

@rept Which version of rspec_api_documentation are you using? I am using version 4.6.0 and it works as expected.


Redirect non-https requests to https

Description / What does this PR do?
Redirect insecure requests to https.
I'm not much of a node developer, but I'm pretty sure you want to use middleware to do this.
Related Issue / Story / Ticket
closes #126

another one for your review @egdelwonk

Hey @maxbeizer! Thanks for this PR. I think overall it's a good approach, but there is a small issue with how the server side rendering fetches data.
https://github.com/nashdev/jobs/blob/master/src/server/render.js#L17
const fetch = createFetch(nodeFetch, {
      baseUrl: `http://<IP_ADDRESS>:${process.env.PORT || 3000}`,
      cookie: req.headers.cookie,
    });

We create a local fetch instance that uses the local loopback address to make requests against the graphql api. This prevents it from having to go out to the open internet and then back in (reduces latency) -- However, currently, the SSL is terminated on the load balancer and as not handled in node, so, changing this request to https would break. Further, since the middleware looks for all non-https requests and redirects them, this request would be caught in that condition too, breaking the api requests.  I'm not entirely sure of what the solution would be though.
A simple hack could be to check for <IP_ADDRESS> with req.hostname in your middleware, and if it's included, and/or if it includes /grapql in req.url then prevent the redirect?
What do you think?

Interesting. So much to learn about modern JS app architecture.
I'll take a look. Thanks for the thoughtful feedback 🙌

Hey @maxbeizer was thinking it might not be a huge deal to just define a API_HOST env_var and just not query it locally. It'll add a bit of latency but will reduce the complexity a bit.
e.g.
const fetch = createFetch(nodeFetch, {
      baseUrl:
        process.env.API_HOST || `http://<IP_ADDRESS>:${process.env.PORT || 3000}`,
      cookie: req.headers.cookie,
    });

Then set it to https://jobs.nashdev.com/  in production.

Full disclosure: I attempted to use SchemaLink for a couple of hours today with no success. So i'm either missing something fundamental or the documentation isn't correct, or maybe both?

I know these feels.
I like your solution. It seems to me like the latency concern is a micro optimization. I'll remove my last commit and update the PR with your suggestion

I went ahead and just removed my pass at schema link from the history and force pushed 🙀 . Just as a heads up

Sounds good.

I like your solution. It seems to me like the latency concern is a micro optimization. I'll remove my last commit and update the PR with your suggestion

You're probably right. Although I was thinking that the latency would be something like 10-15ms versus 60-80 round trip during data fetch for SSR.

I was thinking of adding a note in the README about the new config var. Do you have a preference on that?
Also do apps like these have a pattern for erroring if required config vars are not set? Other langs/framework have a patterns where the app will blow up if config vars are not set in certain environments. Ruby example:
# production.rb
ENV.fetch('MY_CONFIG') # .fetch throws if it cannot find the key

No preference on how the readme is updated, you might want to add it to the .env.example as well. I think in this case if the env var isn't present it will go to a sane default for local dev. In production though, it'd have to be set in the environment or things would break. Your call on that one! :D


change the button for toggle between dark and light mode

This pull request addresses issue #173 "The changer icons"
Changes made:
Added moon icon for dark theme
Added sun icon for light theme
Implemented these icons in the theme changer toggle

This enhancement improves the user interface by providing visual cues for the current theme and the action of changing themes.
Closes #173

Can you tell me why it is failing a test.

Can you tell me why it is failing a test.

Hello:)
This is the error:
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/feature/the-changer-icon*:refs/remotes/origin/feature/the-changer-icon* +refs/tags/feature/the-changer-icon*:refs/tags/feature/the-changer-icon*
  The process '/usr/bin/git' failed with exit code 1
  Waiting 16 seconds before trying again
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/feature/the-changer-icon*:refs/remotes/origin/feature/the-changer-icon* +refs/tags/feature/the-changer-icon*:refs/tags/feature/the-changer-icon*
  The process '/usr/bin/git' failed with exit code 1
  Waiting 20 seconds before trying again
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/feature/the-changer-icon*:refs/remotes/origin/feature/the-changer-icon* +refs/tags/feature/the-changer-icon*:refs/tags/feature/the-changer-icon*

I don't know how to fix it, can you fix it?


Feature/todo list

Feature: Todo List
Commands with prefix ".":

todo_list: This will create or update a todo list for the user
delete_todos: This will delete todos using specified keys
show_todos: This will let the user see his or her todos

Code open for review

Command: nuke_todos maybe added later. Me tired

coooolll


New macro system with quoting and unquoting

Described in cb6a3579293e1211dfde4d834d98777dd6c586df (and foreshadowed by 1b3ea2ae217795b0928c9ade456120328579f8a8)
In progress.  This blocks #84

This also blocks #64


Crashed on startup

Microsoft PowerToys version
<IP_ADDRESS>
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
i updated from a previous version. never had the chance to restart and my pc blue screened in the meantime. Upon next boot this error was thrown when power toys loaded.
Error Message:
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.19042.0
IntPtr Length: 8
x64: True
Date: 29/06/2021 2:42:35 PM
Exception:
System.InvalidCastException: Unable to cast object of type 'System.Windows.Documents.Run' to type 'System.Windows.FrameworkElement'.
at PowerLauncher.MainWindow.SuggestionsList_PreviewMouseLeftButtonUp(Object sender, MouseButtonEventArgs e)
at System.Windows.Input.MouseButtonEventArgs.InvokeEventHandler(Delegate genericHandler, Object genericTarget)
at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
at System.Windows.UIElement.ReRaiseEventAs(DependencyObject sender, RoutedEventArgs args, RoutedEvent newEvent)
at System.Windows.UIElement.OnPreviewMouseUpThunk(Object sender, MouseButtonEventArgs e)
at System.Windows.Input.MouseButtonEventArgs.InvokeEventHandler(Delegate genericHandler, Object genericTarget)
at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
at System.Windows.ContentElement.RaiseTrustedEvent(RoutedEventArgs args)
at System.Windows.Input.InputManager.ProcessStagingArea()
at System.Windows.Input.InputManager.ProcessInput(InputEventArgs input)
at System.Windows.Input.InputProviderSite.ReportInput(InputReport inputReport)
at System.Windows.Interop.HwndMouseInputProvider.ReportInput(IntPtr hwnd, InputMode mode, Int32 timestamp, RawMouseActions actions, Int32 x, Int32 y, Int32 wheel)
at System.Windows.Interop.HwndMouseInputProvider.FilterMessage(IntPtr hwnd, WindowMessage msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.InputFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

2021-06-29.txt
PowerToysReport_2021-06-29-14-50-05.zip
✔ Expected Behavior
I was expecting powertoys to start as normal or to not start atall.
❌ Actual Behavior
Threw error when loaded into windows and program loaded (powertoys is on startup). Seemed to work fine after without restarting the application.
Other Software
No response

@mykhailopylyp this looks slightly different from the other issues related to your PR (https://github.com/microsoft/PowerToys/pull/11968).
The blue screen crash this user experienced is especially concerning. Do you think this will require further investigation?

@dedavis6797
I think the root cause for the exception on the startup is the same.
Blue screen most probably doesn't have anything to do with PowerToys. Furthermore, we don't have anything in the bug report tool that can help us investigate blue screen.

@MadCabbagee
Can you reproduce this issue consistently?
Have System.InvalidCastException happened on startup or have you clicked on a search result?

@mykhailopylyp For suggestion, this issue can be pinned. There is a lot of new issue duplicating this one.

@franky920920
And this one is a duplicate of https://github.com/microsoft/PowerToys/issues/11898

Closing as duplicate of https://github.com/microsoft/PowerToys/issues/11898


Make pip install command printed by 'logfire inspect' easy to copy

e.g. instead of printing:
To install these packages, run:

$ pip install opentelemetry-instrumentation-sqlite3 opentelemetry-instrumentation-jinja2
       opentelemetry-instrumentation-urllib opentelemetry-instrumentation-urllib3

it now prints:
To install these packages, run:

pip install opentelemetry-instrumentation-urllib opentelemetry-instrumentation-sqlite3 opentelemetry-instrumentation-jinja2 opentelemetry-instrumentation-urllib3

i.e. the whole command is on one line, and there's no $. Now triple-clicking that line (at least in my terminals) selects the whole line for easy copy-pasting.

Closes https://github.com/pydantic/logfire/issues/126


LineChart: Add a little space between checkbox and text in legend

Minor visual fix in LineChart with respect to PR #7869
Before:

After:

FYI @Gatos90


Add High Volume SMS support to Go SDK

SDK Version
Latest
Description
There are a few new APIs for RingCentral High Volume SMS. Please update the Go SDK to include these.

Duplicate


Feature request: Allow supporting custom browser/node versions

At the moment, running babel-preset-env with no targets runs all of the transformations available in es2015, es2016 and es2017, like const -> var.
However, most modern browsers support a large subset of these, making these transformations generally unnecessary (and probably worse in terms of performance too).
We could ship less code to people's browsers if we could target, say, the last two browser versions and node 7+!
I'm not suggesting to do this by default, but it would be great to have a field in next.config.js that would be passed through to babel-preset-env's targets to define some custom browser/node versions to support e.g.
//next.config.js
{
  "targets": ["node": 7, "browsers": "last two versions"]
}

There is already a PR for that #1808. It makes the babel next preset configurable.

Oops, nice!

This is still an unresolved issue and should be reopened for visibility and prioritization. The linked PR has not been merged yet; not every PR gets merged.
It's crazy we can't set our browserslist or engines for transpilation, such a low hanging fruit.


Add link to the public view of an occupation standard on admin page

This adds a link to the public view of an Occupation Standard from the admin page.

We already have this, it's just low down on the page. I might move it up. It stays on admin subdomain, but that's fine.


File extensions

File extensions are very limited. Here is my suggestion, but I'm not sure if vscode supports this. I'll give it a shot and update this issue.
Plain scripts:

.sbn
.scriban
Html
.htm.sbn
.html.sbn
.htm.scriban
.html.scriban
XYZ
.xyz.sbn
.xyz.scriban

This might help extending file extension support.

Sorry, I was not getting notifications for this repo. A reboot of your PR #6  is welcome

I'm not familiar with vscode language extensions, is there a way for it to delegate the internal format ... i.e. for mytemplate.xyz.scriban, the xyz part to the XYZ language extension, and then render the scriban syntax as an addition?


Modify dataset split script to modify super categories as well

Summary of Changes

Modified dataset split script to modify the super categories

LGTM!


2.0 Design updates

🎉
Mostly styling updates to match @skllcrn's new designs but includes some JS changes that might be worth reviewing.
Artefact: https://kap-artifacts.now.sh/design/2.0

Wow. This looks super good 👌

Clicking "Install"/"Uninstall" in the Preferences->plugins should disable the button while it's installing/uninstalling. It did that before this PR. Bonus points for a spinner inside the button indicating that it's working (I like the zoom-in effect here: http://lab.hakim.se/ladda/), but that can also be added later. We should try to get this PR landed asap.
Clicking "Open plugins folder" in the Preferences doesn't seem to work.
In the editor window, "✔ Choose an option" should rather be "Export Destinations".
In the aspect ratio window, when the first "width" textfield is focused and I press Tab, I would have expected it to focus the "height" textfield, but it takes two more Tab clicks to get there.
In the aspect ratio window, when inputting something in the "width" or "height" textfields, the ratio box should switch to "Custom".
In the aspect ratio window, selecting the "aspect width/height lock" button does nothing.
In the aspect ratio window, selecting the "switch width/height" button does not correct switch the places.

Thanks for the comment @sindresorhus. These are all definitely something I want to address but this PR has been waiting long enough and I didn't want to make it even busier. I plan to tackle them all in their own PR apart from the Open plugins folder functionality which will be fixed today and changing the text to Export destinations ✅
As far as I see it with the aspect ratio feature, it can be improved a lot by changing the way it works quite a bit.
For example, instead of having preset pixel values for each aspect ratio, we'd use the current width or height and calculate the aspect ratio based on that value and change the other value to match.
For example, the width could be 1024 and I want 16:9 aspect ratio, I should be able to select 16:9 and have Kap work out what the height should be. Should just change the height to 768. From there, if I edit a value of width or height, the other value should change accordingly.
^ This is something I'm working on since yesterday.

@CodeTheory Agreed. Get this merged asap and we can work in perfecting it later.

Woho @CodeTheory great job ❤

It's a stunner 💅


ENH: Added support for windows (convert sys calls to shelljs calls)

The postinstall command in the package.json file for electron called unix commands.   This would result
in errors being reported when run on Windows.
In this commit, those calls have been converted to use shelljs in a postinstall.js file.

@floryst - Please review and comment/merge as appropriate.   Thanks!

LGTM


new Octokit() should not fail to compile

Octokit has no required options by default. new Octokit() should work just as well as new Octokit({})

fixed via 51ee163


Object instead of a String for singleton spec

Other specs should have the responsibility of checking that two equal
strings are different objects.

Yes, that's perfectly fine, thanks @elia!

Interestingly this would also break on MRI with immutable literal strings by default.

@eregon yeah, of course @opal is the avant-garde on that front 😎


Microsoft Defender SmartScreen Warning

When running the latest UtilityBox.App.<IP_ADDRESS>.exe installer Microsoft Defender SmartScreen pops up and warns me off the app being unrecognized.
It's not a deal breaker as one can still continue with the installation by clicking "More info" and "Run anyway". Therefore it's a really low priority issue.
I don't know if it's worth your time signing/validating the app as the process seems to take a bit of time according to this thread: https://stackoverflow.com/questions/12311203/how-to-pass-the-smart-screen-on-win8-when-install-a-signed-application
But I just wanted to let you know.
I'm running Windows 10 Pro 20H2 (19042.867).
The window says:

Windows protected your PC
Microsoft Defender SmartScreen prevented an unrecognized app from starting. Running this app might put your PC at risk.
App: UtilityBox.App.<IP_ADDRESS>.exe
Publisher: Unknown publisher
More info ->
Run anyway  / Don't run

Kind regards
Eddga

Hello,
Yes I am aware of this. And consider it is a free tool, and I do not own a code signing cert I cannot resolve this, unless someone would freely pay for the cert which would be somewhere between $500-700/year. Which I am not interested of paying from my own pocket for a tool which is available to download & compile your own if there would be any concerns.
The original tool was downloaded roughly 140,000 before I decided to do this new version of it.
Br

Ah ok. In the linked thread on stackoverflow it sounded like there were simple free ways of signing the app. In that case I wouldn't spend any money on that either. 👍


linking issue when compiling on windows x64

Hello, I've attempted to compile the package on windows using this tutorial to install a terminal: http://wiki.ros.org/Installation/Windows
and changing all references to linux in the CMakelists to win (and changing uint max_devices_; to int max_devices_;)
In the end everything has compiled but when linking I get this error:
MSVCRT.lib(exe_main.obj) : error LNK2019: unresolved external symbol main referenced in function "int __cdecl __scrt_common_main_seh(void)" (?__scrt_common_main_seh@@YAHXZ)
Any ideas?
Thanks.

I've solved the problem and got it to run succesfully on windows 10. You can see my full code in this fork:
https://github.com/NU-Haptics-Lab/open_vr_ros


telegram bot refactor

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Supports multiple subscribers via /subscribe and /unsubscribe commands
Supports custom /donate text.

What is the current behavior? (You can also link to an open issue here)
Does not subscribe multiple users. Implementation was kinda buggy, because it was useless for more than 1 user.

awesome stuff, thanks!

My telegram bot working fine but in logs i see this error appearing.
Sat, 10 Mar 2018 06:13:51 GMT node-telegram-bot-api deprecated Automatic enabling of cancellation of promises is deprecated.
In the future, you will have to enable it yourself.
See https://github.com/yagop/node-telegram-bot-api/issues/319. at module.js:643:30
Node version: v8.9.4
<EMAIL_ADDRESS>Does anyone know how i get rid off this message?


Select in modal / lightbox

Hello! When triggering the select in a modal box or lightbox (bootstrap modal), the dropdown list appears under the lightbox as shown in this picture:

Any ideas how to fix it? Thank you in advance!

Late response, but just found this extension and was reading the issue report.
I think this is a z-index problem with your overlay. You have to customize the z-index of the .select-element.select-open to al value higher than your overlay z-index.


specan running continuouos

click on restart average > acquisition stops...

is this resolved?

just tried the specan acquisition again and found a somewhat messed up behavior:

i set trace_average to 12
i click "Run single"
i click "Stop" before acquisition finishes -> Run continuous (4) appears on the continuous button -> what does the (4) mean in this context?
i click on "Run continuous (4)" -> acquisition restarts
"Pause" appears (how is this different from stop)? Nothing indicates current number of averages
some more stranger things
at some point i get "Pause (12+)" with continuous updates of the data. I then click on "Restart average" and would expect a running acquisition with display ("Pause (1)") on the button, but instead the acquisition stops.

I believe a critical reading + correction of the logic behind the buttons will eliminate 99% of the confusion here..

OK, I can work on that. However, I am not able to launch the software with the current version of develop-0.9.3
On python 3, I get the error
 C:\Users\Samuel\Documents\GitHub\pyrpl\pyrpl\widgets\spinbox.py in max_num_letter(self)
    326         Maximum number of letters in line
    327         """
--> 328         return int(np.log10(np.abs(self.maximum))+1)
    329
    330     def setMaximum(self, val):  # imitates original QSpinBox API

OverflowError: cannot convert float infinity to integer

Probably due to the values at lines 369/370 of attributes.py
Have you pushed everything to the server?

this is the last bug i also had, but it should have been fixed by my commit at 11:38 this morning.


Attempting to delete files when reported out of space

Getting messages about being at 130% of space, however it only appears as if I've < 500MB in my private folder.

Hi @sean-brandt.  Out quota calculations aren't well-documented yet, and they're pretty complex since we can't actually see what you're doing with your files given the encryption.
One thing of note is that deleted data isn't immediately reflected in your quota usage -- it take at least five minutes of idleness from your client for that to happen.  (We want to make that better but it hasn't been a priority yet.)  So it might be the case that you'll delete some files, and it will take a while for that to be reflected in your usage.
  (You can check our notion of your quota usage with cat /keybase/.kbfs_status. Note that due to block padding and directory structures it will be bigger than the sum of all your file plaintext sizes.)
If you're still having issues, please do a keybase log send and we'll investigate.

Understood, and I'll let things sit a bit. But an overhead of  .. 20X does seem a bit high, doesn't it? :)

If I'm understanding your situation correctly, it's not really an overhead thing, it's just that it takes us a while to learn that the files are actually deleted, rather than just updated or moved around.  We can make this faster in the future, but there are other tradeoffs we have to consider at the same time.

AFAIK I was never near 10GB of data :) Though it's entirely possible as I've been doing a fair bit of rsync'ing in and out of that FS and there's a number of deletes involved in that process as it was being fine tuned. I'm trying to get things cleared out now for a 'clean' start of sorts, though it's taking some time ( and space usage is going up, not down )

Hmm, well it definitely shouldn't be going up if you're not actively writing into it.  However, around the time you posted this, I see that two devices were actively writing to the folder. Is it possible you forgot about one device that is writing things periodically?  Any device writing to the folder will prevent all devices from doing garbage collection and freeing quota.
Of course, there could easily be a bug in our quota tracking stuff too, just trying to rule out other stuff first.

Ok, actively working to delete content from kbfs, AFAIK I have 1 client access the system at this time and it's still reporting high usage.
➜  Logs  cat /keybase/.kbfs_status
{
"CurrentUser": "XXXXXX",
"IsConnected": true,
"UsageBytes":<PHONE_NUMBER>9,
"LimitBytes":<PHONE_NUMBER>0,
"FailingServices": {}
}
➜  Logs  du -sh /keybase/private/XXXXXX
329M	/keybase/private/XXXXXX
log id: d768e738dab216868c77081c


Remove unnecessary print statements.

Codecov Report

Merging #76 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #76      +/-   ##
=========================================
+ Coverage   22.03%   22.1%   +0.06%
=========================================
  Files          44      44
  Lines        2069    2063       -6
  Branches      225     225
=========================================
  Hits          456     456
+ Misses       1591    1585       -6
  Partials       22      22

Impacted Files
Coverage Δ

lib/python/treadmill_aws/ipa525.py
0% <ø> (ø)
:arrow_up:

lib/python/treadmill_aws/awscredential.py
0% <ø> (ø)
:arrow_up:

lib/python/treadmill_aws/ipakeytab.py
0% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update meta-resin

@agherzan @floion

Can one of the admins verify this patch?

https://github.com/resin-os/resin-raspberrypi/commit/9e0fa1be4ae0aebc1532c4a4fafbf4b95edaa0cf

Can one of the admins verify this patch?


nuget

Greetings @effyteva
I forked your repo https://github.com/olka-dev/Teva.Common.Cultures, making some changes (improvements in my opinion). And based on that fork I published it to nuget https://www.nuget.org/packages/Teva.Common.Cultures/. If you have problems with me using your project name as the name of the package - please let me know ASAP.
You are free to merge my changes back to original repo, but be aware that json serializer changed under the hood may be less desirable on .net framework consumers, as it will pull up many dependencies for system.text.json from the nuget (while having no
external dependencies at all for .net core consumer).

Thank you @olka-dev!
I'm currently on vacation, once I return I'll try to merge your changes.