Parsing is slow when alternating between projects

As discussed here https://github.com/fsharp/FSharp.Compiler.Service/issues/488. I've tried <IP_ADDRESS> and the same problem is there.
I recorded a screencast highlighting the issue. You can see that when I switch projects to a previously parsed project that the parse step is slower than expected. I think it could related to the project options too, but not sure exactly what. We cache the project options, so that GetProjectOptionsFromProjectFile is only called once per project.
The project options are here https://gist.github.com/nosami/c15cc73f41bde56b1240 These are the project options for the Xamarin Studio F# plugin and test project.

Any chance you could redo the screencast so that you switch back and forth between the same files in the two projects (in the screencast you switch back to a different file - one further down the list of files). thanks
BTW this table needs updating in the xamarin addin: https://github.com/fsharp/xamarin-monodevelop-fsharp-addin/blob/bfdf8253776fd299b6d878970417a1d9334c1fa3/MonoDevelop.FSharpBinding/FSharpProject.fs#L24

Also it might be good to double check that InvalidateConfiguration is not being called: https://github.com/fsharp/xamarin-monodevelop-fsharp-addin/blob/bfdf8253776fd299b6d878970417a1d9334c1fa3/MonoDevelop.FSharpBinding/FSharpProject.fs#L35, though it doesn't look like it would be.

Hmmm... It looks like the "ProjectFileName" field is inconsistently populated in that data, see

I checked that InvalidateConfiguration isn't being called - it isn't. Here is a screencast with me alternating between the same 2 files each time.
It's not as obvious because the files are already highlighted, but you can see the parse time by looking at the log.
The ProjectFileName does look wrong. I'll take a look at that.

It was the wrong ProjectFileName that was causing the slow parses. I'm a little annoyed with myself for not spotting that earlier :)
Thanks!

Great to know!

Just to double-check, that isn't a problem with the project cracking I need to fix?

@rneatherway No, it was a bug that I introduced in Xamarin Studio. You might want to normalise the paths everywhere though.

At the moment we are using our own cracking, so that we can have a quick turn around on issues.


[sx] sx does not load gui when in a notebook

When sx is imported in a notebook it does not load gui stuff.
Do we want to change that?
In this case, ideally it would need to check if it is run locally or remotely...

My opinion is that it would be a good feature to have. The notebook was really helpful for the training exercises, for the trainees who were lucky enough to have it installed on their laptops.
Having "sx.plot" available would allow us to skip the "%pylab inline" and "%gui qt" explanations.

Problem is that it would load qt even when logged remotely to the notebook.
It should print a message that it is not loading guy


bug: Could not copy config file [/var/lib/ziti/config.json]

Hi @qrkourier @scareything,
I am starting to upgrade the ziti-edge-tunnel image to the latest version via helm (0.22.23), and it seems that the new livenessProbe implemented here via command ziti-edge-tunnel tunnel_status is logging some errors and warnings every minute!
See,
(7)[2024-03-07T08:18:22.276Z]    INFO ziti-edge-tunnel:ziti-edge-tunnel.c:644 on_cmd() received cmd <{"Command":"Status"}
>
(7)[2024-03-07T08:18:22.276Z]    INFO ziti-edge-tunnel:ziti-edge-tunnel.c:227 on_command_resp() resp[1,len=9422] = {"Success":true,"Data":{"Active":true,"Duration":298949,"StartTime":"2024-03-07T08:13:23.327855Z","Identities":[{"Name":"name","Identifier":"/ziti-edge-tunnel/ziti-edge-tunnel-identity.json","FingerPrint":"/ziti-edge-tunnel/ziti-edge-tunnel-identity","Active":true,"Loaded":true,"Config":{"ztAPI":"URL"},"ControllerVersion":"v0.31.0","IdFileStatus":true,"MfaEnabled":false,"MfaNeeded":false,"Services":
...
["tcp"],"Addresses":[{"IsHost":true,"HostName":"hostname","Prefix":0}],"AllowedSour
(7)[2024-03-07T08:18:22.276Z]   ERROR ziti-edge-tunnel:instance-config.c:136 save_tunnel_status_to_file() Could not copy config file [/var/lib/ziti/config.json] to backup config file, the config might not exists at the moment
(7)[2024-03-07T08:18:22.276Z]   ERROR ziti-edge-tunnel:instance-config.c:142 save_tunnel_status_to_file() Could not open config file /var/lib/ziti/config.json to store the tunnel status data
(7)[2024-03-07T08:18:22.276Z]    WARN ziti-edge-tunnel:ziti-edge-tunnel.c:629 on_cmd() received from client - EOF. Closing connection.
(7)[2024-03-07T08:18:22.276Z]    WARN ziti-edge-tunnel:ziti-edge-tunnel.c:641 on_cmd() IPC client connection closed, count: 0

IMHO, the ERROR should be handled by the tunnel gracefully, so that it does not get triggered at all.
And the resulting warning messages at the end could also be saved if possible.
I look forward to hearing your thoughts. Thank you!

relevant issues about unexpected log level precedence, relating to config.json file:

https://github.com/openziti/ziti-tunnel-sdk-c/issues/736
https://github.com/openziti/ziti-tunnel-sdk-c/issues/531

Shawn noted that the config.json file is only important for Windows, and that we should ignore it on Linux. I'd be OK with that since the only time I need to think about it on Linux is when I'm deleting it work around the log level precedence issue.
Shawn wrote in https://github.com/openziti/ziti-tunnel-sdk-c/issues/461#issuecomment-1198620116

As of now this notion of a "config file" should not be processed by ziti-edge-tunnel on Linux.

IMHO, log level works as expected. The issue here is more about the error log which should not be triggered running in a container. If this is a windows thing, maybe it helps handling this case for != windows.

@scareything any news on this? Due to the livenessProbe this issue logs two errors every minute.

@scareything this is still active with 0.22.32, and produce a lot of unnecessary logs.


Fix flags in README example

The example invocation of reddsaver uses flags that don't seem to exist according to the reddsaver --help command. This PR changes those to match the real flags.
Arguments changed:
--e → -e
-f → -d

Thanks for fixing the typo!

Thanks for fixing the typo!


Fix readthedocs warnings

When built locally, there are warnings like
Cirq/docs/api.rst:72:<autosummary>:1: WARNING: Inline substitution_reference start-string without end-string.
Cirq/docs/api.rst:72:<autosummary>:1: WARNING: Inline substitution_reference start-string without end-string.
Cirq/cirq/circuits/circuit.py:docstring of cirq.Circuit.apply_unitary_effect_to_state:8: WARNING: Unexpected section title or transition.

--------------
Cirq/cirq/circuits/circuit.py:docstring of cirq.Circuit.to_unitary_matrix:7: WARNING: Unexpected section title or transition.

--------------
Cirq/cirq/sim/density_matrix_simulator.py:docstring of cirq.DensityMatrixStepResult.density_matrix:23: WARNING: Field list ends without a blank line; unexpected unindent.
Cirq/cirq/sim/density_matrix_simulator.py:docstring of cirq.DensityMatrixTrialResult:23: WARNING: Field list ends without a blank line; unexpected unindent.
Cirq/cirq/__init__.py:docstring of cirq.ISWAP:1: WARNING: Inline substitution_reference start-string without end-string.
Cirq/cirq/ops/common_gates.py:docstring of cirq.ISwapPowGate:1: WARNING: Inline substitution_reference start-string without end-string.
Cirq/cirq/sim/sparse_simulator.py:docstring of cirq.SparseSimulatorStep.qubit_map:1: WARNING: duplicate object description of cirq.SparseSimulatorStep.qubit_map, other instance in Cirq/docs/generated/cirq.SparseSimulatorStep.rst, use :noindex: for one of them
Cirq/cirq/sim/sparse_simulator.py:docstring of cirq.SparseSimulatorStep.state_vector:19: WARNING: Field list ends without a blank line; unexpected unindent.
Cirq/cirq/sim/wave_function.py:docstring of cirq.StateVectorMixin.qubit_map:1: WARNING: duplicate object description of cirq.StateVectorMixin.qubit_map, other instance in Cirq/docs/generated/cirq.StateVectorMixin.rst, use :noindex: for one of them
Cirq/cirq/sim/wave_function.py:docstring of cirq.StateVectorMixin.state_vector:20: WARNING: Field list ends without a blank line; unexpected unindent.
Cirq/cirq/study/trial_result.py:docstring of cirq.TrialResult.measurements:1: WARNING: duplicate object description of cirq.TrialResult.measurements, other instance in Cirq/docs/generated/cirq.TrialResult.rst, use :noindex: for one of them
Cirq/cirq/sim/wave_function_simulator.py:docstring of cirq.WaveFunctionTrialResult.state_vector:19: WARNING: Field list ends without a blank line; unexpected unindent.
Cirq/cirq/sim/wave_function.py:docstring of cirq.density_matrix_from_state_vector:12: WARNING: Literal block expected; none found.
Cirq/cirq/google/sim/xmon_simulator.py:docstring of cirq.google.XmonStepResult.qubit_map:1: WARNING: duplicate object description of cirq.google.XmonStepResult.qubit_map, other instance in Cirq/docs/generated/cirq.google.XmonStepResult.rst, use :noindex: for one of them
Cirq/cirq/google/sim/xmon_simulator.py:docstring of cirq.google.XmonStepResult.state_vector:19: WARNING: Field list ends without a blank line; unexpected unindent.

I already started working on this so you can assign to me if you want,


OpenEBS backend

I wonder if instead of NFS you can use freeNAS as the  openEBS iSCSI target?

non-issue, closing


[docs-infra] Simplify CSS classes extraction in API docs generator

Currently we have mostly duplicated logic for CSS classes extraction in API docs builder and two similar fields in the resulting API description JSON files: styles and classes. This PR unifies the processing of CSS classes and gets rid of the styles field.
The format of the classes field was also changed to make it easier to use in the docs: the classes and globalClasses fields inside the classes object were changed to an object with several properties:
old:
"classes": {
  "classes": ["active", "disabled", "focusVisible"],
  "globalClasses": {
    "active": "Mui-active",
    "disabled": "Mui-disabled",
    "focusVisible": "Mui-focusVisible"
  }
}

new:
"classes": [
  {
    "key": "active",
    "className": "Mui-active",
    "description": "State class applied to the root `button` element if `active={true}`.",
    "isGlobal": true
  },
  {
    "key": "disabled",
    "className": "Mui-disabled",
    "description": "State class applied to the root `button` element if `disabled={true}`.",
    "isGlobal": true
  },
  {
    "key": "focusVisible",
    "className": "Mui-focusVisible",
    "description": "State class applied to the root `button` element if `focusVisible={true}`.",
    "isGlobal": true
  }
]

(while the description field isn't strictly necessary here, it was added to match props definitions)
The docs were adapted to use this new format.
Another change was moving the responsibility of figuring out the actual class name to the API docs builder. Previously, we had logic for this in many different places - in some cases the .Mui- prefix was added in the docs, in other in the docs builder. With this change the docs will display the class name exactly as it was defined in the JSON file, without adding any prefixes. This will allow to configure how classes are generated in the API docs builder. This was my primary motivation for this PR, as I need to customize the prefix of Base UI CSS classes.
This PR does not introduce changes visible to our users (except for a couple of corrected strings).
I recommend reviewing it by commit.

Nice work. A bit related to this, I've created this issue: https://github.com/mui/material-ui/issues/39867 we should not show the rule name for the state classes as you can't override them without bumping the specificity.

@mnajdova fixed in https://github.com/mui/material-ui/pull/39808/commits/a394217e76569355db54a612e835c2a9fc3e556d


Add support for omitting args when using enqueuing methods

This is useful if your job does not take any parameters.
Before it used to work so that omitting arguments caused the arguments to be [undefined], which was a problem if your perform signature was "function(callback)". In that scenario callback was assigned undefined value when enqueuing with "enqueue('queue', 'job')"
Also fixed allDelayed to return empty hash if timestamps is empty

Thanks!


problem in applying sc.pp.highly_variable_genes(adata, flavor="seurat_v3", n_top_genes=3000)

Any people meet same question?  When I running sc.pp.highly_variable_genes(adata, flavor="seurat_v3", n_top_genes=3000),error occurs.

what's amazing is I runned it successfully yesterday, but today, when I rerun it, the error occurs. I don't do any modification!!!

there are some related issue in github in https://github.com/scverse/scanpy/issues/2073

In a windows environment, I solved this error by install the "scikit-misc" package by https://www.lfd.uci.edu/~gohlke/pythonlibs/#scikit-misc.


Add eventing kafka broke reviewers role

Proposed Changes

Add eventing-kafka-broker-reviewers role as reviewer to repository

/cc @pierDipi @matzew

The alias doesn't exist yet https://github.com/knative-extensions/eventing-kafka-broker/blob/a60586958fa6f8028624e7cd7e99731f00d738a3/OWNERS_ALIASES#L48-L51
should we create it first?

The alias doesn't exist yet
https://github.com/knative-extensions/eventing-kafka-broker/blob/a60586958fa6f8028624e7cd7e99731f00d738a3/OWNERS_ALIASES#L48-L51
should we create it first?

@pierDipi I think I tried to create it in https://github.com/knative/community/pull/1428. Will that work or do I need to do something else as well?

/retest

/verify-owners

@pierDipi the owners file is valid now :)

/retest-required


chore(chart): remove faux @superset-ui/core TS declaration

🏠 Internal
This PR removes the faux @superset-ui/core TS declaration in @superset-ui/chart (introduced before core was re-written in TS). All of the changes are to clean up resulting TS errors from that removal. Some notes on specifics that were done:

Refined and consolidated some types that were duplicated across files like ChartType, (Pre/Post)TransformProps, etc.. Turns out the duplication wasn't always entirely consistent though at surface level it might not be obvious.
Added the needed generics to all of the chart plugin registrys
Updated the logic for supporting Value | { default: Value } generally across loaders. This is now confined to ChartPlugin in order to simplify types elsewhere, while still supporting dynamic loading of ES modules: () => import('foo').default

(EDIT: The following was refactored so as not to be a breaking change)
~💔 Breaking Changes~

~Removed support for { default: Type } in Registry loaders, which was previously allowed due to a Chart type using module.exports instead of export defaulting. This simplifies typing when using these components which I think is a win 🏆. I think worst case if this is problematic in incubator Superset we could update () => import('./module.xxx') calls to () => import('./module.xxx').default~

TODO

[x] 💯 % coverage
[x] fix .default issue .. may not be a breaking change

@kristw @conglei @xtinec

all right, made this compatible with the default syntax, should be good to go 🚀


Implementation of fire_event

Hi, @pravic.
When are you gonna implement  fire_event function for Element?

Not only fire_event, actually, need to add much more functions.

ok, I see.
Do you have an estimation?

Do you want to make a pull request for it?

I'll try but can't guarantee that I'll be able to manage it since I'm newbie in Rust.

@pravic I've started implementing and have questions about the next signatures:
pub SciterSendEvent: extern "system" fn (he: HELEMENT, appEventCode: UINT, heSource: HELEMENT, reason: UINT_PTR, /*out*/ handled: * mut BOOL) -> SCDOM_RESULT,
pub SciterFireEvent: extern "system" fn (evt: * const BEHAVIOR_EVENT_PARAMS, post: BOOL, handled: * mut BOOL) -> SCDOM_RESULT,
Why do we need raw mutable pointers for variable "handled"?

Check this doc if I understand your question correctly.

@pravic  I'm talking about type of variable handled. Why is it *mut? it might be just bool type I guess.
Also I can't create mut pointer for boolean type is Rust.

Because of /*out*/ BOOL* handled - it is out argument.

Also I can't create mut pointer for boolean type is Rust.

let mut handled = false as BOOL;
let params = BEHAVIOR_EVENT_PARAMS::default();
let ok = SciterFireEvent(&params, false as BOOL, &mut handled);
if ok == SCDOM_RESULT::OK && handled {
  // no error and behavior event was handled by element
}

Done in this and this commits.
However I found that send event feature mixes source and target elements, asked about it here: http://sciter.com/forums/topic/confusing-scitersendevent-k/.

But in general, why do you need this functionality in native code? Its easier (and better) to work with DOM via script, which can ask the native app about data and notify it about events.

A goal was studying Rust but finally I use script;)

You can study it providing high-level logic and data in app, leaving UI-related operations to UI level.

yeap, that's why I migrate to tiscript

Enjoy then :)


Add language event handling

The user can set the language.  When a room is created, the room
language is set and only users with the same language set can join.
Signed-off-by: Anderson Toshiyuki Sasaki<EMAIL_ADDRESS>

Tested with https://github.com/yuumasato/have_you_heard/pull/65


Passing an empty string to getAccount returns a list of automation accounts.

Package Name: @azure/ms-rest-nodeauth, @azure/arm-automation
Operating system: windows 10
[ ] nodejs

version: 3.7.4

[ ] typescript

version:3.7.4

Describe the bug
After authentication If you call AutomationClinet.getAccount(resourceGroup, "") like this you will get a list of all automation accounts in that resource group.
To Reproduce
Steps to reproduce the behavior:

Create an instance of AutomationClient using ApplicationTokenCredentials
Call getAccount method with a legitimate resource group but an empty string for account name
Receive a list of all automation accounts for that resource group

Expected behavior
Expected to get a 404 Error.

JobOperations.get() method likewise lists all jobs per account if an empty string is passed in for jobName. Is this intended behavior?

Add service team attention here.
A constraint [ minLength>0 for "AutomationAccountNameParameter" ] should be defined in swagger to prevent this happen.

This is triaged and currently in the backlog, will consider it if resourcing allows.

Unfortunately because of other priorities we were not able to pick this up, please expect an update in Q4CY20.

[Update] : We got the issue triaged & the analysis is in case of API call , any empty filed will be treated as not passed hence API gets called at the user context evel , which will return all the accounts use has access to . This is an expected behaviour.

Currently, team is busy with other high priority items and this would be picked up in CY2021 Q1.

Currently, team is busy with other high priority items and this would be picked up in CY2021 Q1.

This is an expected behaviour as mentioned in my previous comment , hence closing it.

This is an expected behaviour as mentioned in my previous comment , hence closing it.


Not able to Increase the number of samples in generated synthetic data

I'm using the below command to generate the synthetic data with sdg-scale-factor as 100:
ilab data generate --endpoint-url http://localhost:8080/v1 --chunk-word-count 1000 --model mistralai/mistral-large --pipeline full --sdg-scale-factor 100
The number of samples in synthetic data generated always remains the same that is 30 and it's not getting affected by --sdg-scale-factor value.

cc. @bbrowning @jaideepr97  could you PTAL?

Because the number of samples generated is always stuck at 30, this user is generating skills and is hitting #420 - our behavior today is confusing because we do use --sdg-scale-factor to scale some of our generated data, but we only ever mix in 30 samples from any taxonomy leaf nodes into the final output.


Add support for PayHub Payment Gateway. Support transation purchase, voi...

@j-mutter  @shopify-admins @shopify-deployer @shopify-hooks
Have added support for Pay Hub Payment gateway. http://www.payhub.com/
Currently it only supports purchase, void and refund transactions. Please take a look and let me know if I missed anything.
Thanks

One nitpick, everything else looked :cool:, but I'm still getting the holiday cobwebs out of my head - @ntalbott care to give this another quick once-over?

@j-mutter I'll give it a once over in a bit and take care of merging - thanks for the review!

@j-mutter @ntalbott Have removed the comments. Hope to see it gets merge soon now.

@wvanbergen  @ntalbott @j-mutter Hope you guys are doing really great!!
Just wanted to checkin if you are waiting for anything from my end.

@ntalbott @j-mutter Hi, would you please let us know the status of this pull request?  It looks like we're just waiting on a final review.  Thanks in advance for your help on this.

Couple minor nitpicks in the tests, otherwise LGTM. @lonsun can you post the output of the remote tests?

Going to get this merged.

@ntalbott, @j-mutter, et al
Thank you for your help on this!
If you still want me to send the output of the remote tests let me know.

@lonsun no, I was able to run them myself no problem. Thanks!


Suggestions: What if you remove the semicolon

We are in the 21st century, semicolon seems like an old thing, I suggest experimenting with a Flutter, with no semicolons, and see what

There is an open issue in the Dart SDK discussing the possibility of making semi-colons optional here:
https://github.com/dart-lang/sdk/issues/30347
It's best to keep all discussion there in the same place.

Thank you.


Remove/some colors

And add nice way to handle odd section

Summary by CodeRabbit

New Features

Introduced a new pagination component to enhance user navigation.
Added new hover effects and color variables for improved UI feedback.

Bug Fixes

Updated background colors for various components to ensure visual consistency and improved accessibility.

Style

Refined color schemes across multiple components for a more cohesive design.
Enhanced hover and active states with new gradient backgrounds for better user interaction.
Streamlined styling for inverted sections to improve visual consistency.

Documentation

Added a new color variable file to streamline color management across the application.
Modularized color definitions for better organization and maintainability.

Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (1)

packages/styles/framework/src/elements/_button.scss (1)> 66-67: Approve the change in the visual representation of disabled buttons.

The change in the background color and text color for disabled buttons using the getFadeColorValue function is approved as it likely aims for a different aesthetic or improved accessibility.
Consider removing the !important flag.
The use of !important is generally discouraged as it can make the code harder to maintain and override in the future. Consider removing the !important flag and using a more specific selector if you need to override other styles.
Apply this diff to remove the !important flag:
-background-color: getFadeColorValue('main') !important;
-color: getFadeColorValue('disabled') !important
+background-color: getFadeColorValue('main');
+color: getFadeColorValue('disabled');

Review details

!important is important here because we need to override the default rule body [disabled].


Go To Type Definition opens .class instead of .java

In https://github.com/enola-dev/enola/, when I Go To Type Definition e.g. in some **/src/test/java/**/*.java editor on a "symbol" that is something in the src/main (instead of /test) of the same module/project/whatever, then instead of .java it opens some read-only .class (read-only, but not decompiled; it's the correct source text).
I'm not sure how to describe this more accurately - let me know if some... screenshots (?), or something, would help to clarify.
@guw

Hm, this strangely only seems to happen in some cases, and I haven't found out the correlation why, yet:
For example, when common/common/src/test/java/dev/enola/common/io/resource/ClasspathUrlResourceTest.java is open and I F12 on that UrlResource in line 44, then it correctly opens common/common/src/main/java/dev/enola/common/io/resource/UrlResource.java.
However when I'm in core/impl/src/test/java/dev/enola/core/meta/TypeRepositoryTest.java (which is not yet on the main branch as of writing this, but will eventually be merged in, from https://github.com/enola-dev/enola/pull/451) and Ctrl-Click on the TypeRepositoryBuilder, then instead of core/impl/src/main/java/dev/enola/core/type/TypeRepositoryBuilder.java, it opens a TypeRepositoryBuilder.class from somewhere... check out this screenshot:

so apparently, in some cases, it doesn't set up the Inter-Project Dependency in the underlying JDT correctly?
Looks like for the this ^^^ case it references this libimpl-ijar.jar, whereas in the case where it works it "does the right thing", and jumps into the project, instead of opening a class from (in that case) libcommon-ijar.jar.
The BUILD for those 2 modules look very similar though... it's a [java_test( with a "simple" deps = [":common", respectively deps = [ ":impl",.
Strange.

What makes this more annoying than simply Navigating is that when one makes changes (like adds new methods, or whatever) to code under test in main/java code, they are not "visible" (red) in test/java, until a full rebuild - that's quite a PITA.

@vorburger I tried to reproduce but I ran into an issue:
ERROR: .../enola/tools/hello/BUILD:29:13: Compiling Java headers tools/hello/libhello_lib-hjar.jar (1 source file) failed: (Exit 1): java failed: error executing Turbine command (from target //tools/hello:hello_lib) external/local_jdk/bin/java '--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED' '--add-exports=jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED' ... (remaining 47 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
tools/hello/src/main/java/dev/enola/common/hello/Library.java:27: error: unexpected input: {
        return STR."hello, \{planet}";
                            ^

@vorburger I tried to reproduce but I ran into an issue:

See https://github.com/enola-dev/enola/pull/452, but...
... even if you are on a commit from before that fix, you should never see this.
Could this be due to some Bazel/ Bazelisk version discrepancy between us? E.g. related to #88?
Having said that... I (now) can no longer reproduce this problem myself - huh?!
Unless you are interested in digging more, let's close this? I'll comment when (if) I see it again, hopefully then with a clear(er) reproducer.

Going to the .class usually happens when the .java source is not properly set up as project. It can also happen if there is a bug in classpath resolution and Bazel Eclipse fails to resolve a jar dependency to an Eclipse project. Please re-open when you have such a case.


refactor(all): improved setup

major changes to the code base.

[x] Rewrote Components
[x] Easy Nuxt/Vue setup

[x] Build both in the same time
[x] export all components
[x] ability to change prefix from E to whatever the user wants ( removed cus of an issue in rendering template )

[x] Improved Demo

[x] Ability to reload to view changes
[x] Mobile/Desktop view
[x] Ability to send emails
[x] Toasts
[x] updated design/feel
[x] can be deployed
[x] add extra templates ( now only 6 will be added, will add more after merge )
[x] ability to view vue code

[x] Github Workflow

[x] PR check, run tests

[x] Added Tests
[x] Easy Publish Setup ( to be discussed )
[x] Update docs with new changes

i resolved some of the comment, but for stuff like remove lang='ts' from the script tag, i left, as i get an error when building,
[@vue/compiler-sfc] <script> and <script setup> must have the same language type. and i dont see an issue if its even there, as its not something that affects the end user, as everything gets compiled


Nice error page in debug mode

It would be nice if Bailador would catch any exceptions during its processing and instead of "Internal Server Error" it would give a nice error page. (e.g. like Flask does)

what do you think of the idea that the error page could be a template as well. all we need to do is ship a "hardcoded"  500.debug template.
there would be 4 different cases

debug mode  == 1 and no $!location/views/500.debug available
debug mode == 1 and a 500.debug is avilable  (so somone could still create his own default error page
debug mode == 0 and no 500.xx => still the Inernal server error
debug mode == 0 and a 500.xx evailable => template should be used

I am working on a template for the 500 page. I am not sure what would we want to show in debug mode if there is a user-created 500 template. Actually I am not even sure we want to stay with a simple template for 500 errors. Flask for example allows you to create a "virtual route" which is called in case there is a 500 error. That way the developer can do more than just show a plain page.

Wouldn't it be better if this was an external file, instead of hard-coding it? Better user configuration, too.

I agree with you, what is hardcoded could/should probably be in the resources. This is the last resort, if there are error templates in your application, I think they're named 500.xx, you would use them. So from the configuration/usability perspective I'd say it is okay. no?


[4pt]  - Create feature: add new comment - Student B

Implement the "add new comment" form on the Comments pop up according to the wireframe:

When the user clicks on the "Comment" button, the data is recorded in the Involvement API and the screen is updated.

https://github.com/kamranahmad123/Movie-Site/pull/33


MM-23542 new permission for system console access

Summary

This PR adds a new permission ACCESS_SYSTEM_CONSOLE which determines if the System Console menu item should be displayed for the user in webapp.

use permission for accessing system console in main menu
use permission for displaying "Please configure your Site URL in System Console"
update snapshots

Ticket Link

https://mattermost.atlassian.net/browse/MM-23542
Related Pull Requests

Has server changes https://github.com/mattermost/mattermost-server/pull/14190
Has redux changes https://github.com/mattermost/mattermost-redux/pull/1110

I have swapped myself for others on my team as I feel they may have a better understanding and scope of this ticket.

Closing as new permission for system console access is not desired.


DRAFT: Post Mount Logic Check

The bits in nnf_node_storage_controller.go is the main part.

From @matthew-richerson:

I just realized while reading through your PR that this isn't going to work
the nnf-node-manager where that controller runs has a client cache that's limited to the namespace of whatever the Rabbit hostname is
those NnfNodeStorage resources could be spread across lots of different Rabbits, so lots of different namespaces
I think the NnfStorage controller is going to have to be the one that coordinates this
an easy way would be to have the NnfStorage controller create the NnfNodeStorage for OST0 only after all the other NnfNodeStorages are Ready
then the NnfNodeStorage controller doesn't have to do any extra checking, it just runs PostMount() if it's OST0
we'd lose a little bit of speed setting up Lustre file systems, but most parts would still happen in parallel


Enable leak tracking for release mode.

Currently leak tracking is disabled in release mode, because of availability of the API reachabilityBarrier (package dart:developer).
If you are interested in leak tracking in release mode, please, upvote this issue and justify  your case.

I believe this would be very helpful for release builds as well, so people can upload their leaks to error reporting tools such as sentry.io and fix them.
Similar to https://square.github.io/leakcanary/uploading/

Could be useful for edge cases with special releases on beta channel. Would be good to add an enableInRelease flag.

@a-siva and @bkonyi , thoughts?

@a-siva and @bkonyi , thoughts

This is for flutter release builds I presume or is this request for command line Dart ?

I assume this is for PRODUCT mode builds, which includes Flutter release. Given that this is basically a counter that we increment (even in PRODUCT mode, although it's not accessible) I think it's low-risk to expose this in PRODUCT mode. However, if this feature requires any sort of VM service connection to work properly then this won't be something we can do.

Thanks. Yes, it is just counter. VM service connection is not needed as the diagnostics will be saved and analyzed offline. Can we plan implementing this? Should I transfer the issue to VM repo?

I vote for it, if the performance drop is minimal in release mode. More sanity checks never do any harm (as long as they do not affect performance)!

https://dart-review.googlesource.com/c/sdk/+/315981

I vote for it, if the performance drop is minimal in release mode. More sanity checks never do any harm (as long as they do not affect performance)!

I agree with this comment.

The feature is unblocked, but not implemented yet, and not part of leak_tracker MVP.
Reopening.

The feature is unblocked, but not implemented yet, and not part of leak_tracker MVP. Reopening.

We have turned  the reachability barrier on in product mode, why do you say it is not implemented yet?

@itsjustkevin, I am reopening the issue: while reachability barrier is now available in release mode, leak tracker is still off for release mode.

I do not think we should make this an option unless we have significant profiling data to support it.
In particular, we should have profiling data from a mid-tier Android device that shows this has little to no impact on frame/raster times. The profiling data we do have so far from macOS desktop applications suggests that it would.

I do not think we should make this an option unless we have significant profiling data to support it.
In particular, we should have profiling data from a mid-tier Android device that shows this has little to no impact on frame/raster times. The profiling data we do have so far from macOS desktop applications suggests that it would.

I agree. Most likely in release mode leak analysis and reporting should happen not on timer, but on request from application, when application is known to be in stale state.


[DOCUMENTATION] Update & Deleting give wrong example command

https://beanie-odm.dev/tutorial/updating-%26-deleting/
bar = await Product.find_all(Product.price > .5).inc({Product.price: 1})

find_all can not be used with a query, this command will update all documents.

True, this is a typo. THank you for the cacth!


change background color; change guesture for better UE

Drag directly
Long press to copy text to paste board
Double click to show/hide the log view.
Change background color to black with alpha 0.7 to be more professional

Hi,

I deliberately made on long press instead of directly pan, in order to prevent accidental dragging (especially if you are using it on an app full of swipe gestures).
Sounds good. But actually I plan to implement a context menu where you can copy or clear logs.
Again I deliberately chose 3-finger swipe, as it is a rarely used gesture and less likely to have conflicts with normal app usage.
Black looks more professional? :) I just chose macOS Terminal Red Sands theme, as I believe it is very visible on a wide range of UI colors.

So, thank you again for PR, but I am sorry to say that our tastes do not match and closing it.


Improved BTBuilder

Hi all,
This PR contains a new algorithm for building a BT from the plan. It is simpler than the previous one, and fix some issues like https://github.com/IntelligentRoboticsLabs/ros2_planning_system_examples/pull/12 https://github.com/IntelligentRoboticsLabs/ros2_planning_system/issues/67 and others...
Best

Now, the Behavior tree for the last plan is also  stored in /tmp/{namespace}/bt.xml

Now, the Behavior tree for the last plan is also  stored in /tmp/{namespace}/bt.xml


Back-off pulling image rancher/mirrored-prometheus-alertmanager

In emberUI enable monitoring, set up slack notifier and add alert.
Test alert can be sent when setting up notifier.
After adding alert rule (tested pod not running) and removing pod alert rule disapears but notification is not sent.
ErrImagePull: rpc error: code = Unknown desc = Error response from daemon: manifest for rancher/mirrored-prometheus-alertmanager:latest not found: manifest unknown: manifest unknown

Chart: dev-v2.6 (commit: 2452663cae8dbd9083e1374782ef445440491ae6)
Cluster1: Custom v1.20.7 (Oracle 8.4)
Cluster2: Infra provider DO (Ubuntu 20.04)
Rancher: single node master-head (c7f3cce)

Hi @kravciak  I assume you are using the monitoring v1 chart version v0.2.3, right?
It is weird to see it tries to pull the latest tag because we always avoid the latest tag in all our charts.
The abovementioned image is used is at https://github.com/rancher/system-charts/blob/cd9dbb6b031a58b7b7d2b1f1edc2ccd99ea6a0d8/charts/rancher-monitoring/v0.2.3/values.yaml#L210-L211
    repository: rancher/mirrored-prometheus-alertmanager
    tag: v0.22.0

Can you try it one more time to see if it works now?

Yes, monitoring v1 is v0.2.3. I was not able to reproduce again, everything seem ok in master-head (ac7cef3)

is there any different between the rancher prometheus docker image and the upstream one?


select all in the results pane should only select the content of the results pane

I am on a Mac. I also tested this on Windows. If you hit ctrl-A or cmd-A, every piece of content from every field, box, and pane is selected. The correct behavior should be only to select the content from within the selected pane, in this case the results pane.
Great product. This one is a tweak, but an important one for some DevOps workflows.

I agree its annoying :smile: I wish HTML made this easier to do. Alas, I'll get it added. Thank you for the suggestion, and glad you're enjoying it!

Fixed in linked PR. Will go out with next release.


Update docs for multiple listeners and non-partitioned topic support

Motivation
https://github.com/streamnative/kop/pull/690 supports non-partitioned topics and https://github.com/streamnative/kop/pull/742 supports multiple listeners and marks some configs as deprecated, so the current documents are outdated.
Modification
Even if KoP supports non-partitioned topics now, it's still better to configure allowAutoTopicCreationType with partitioned by default. This PR modified the explanation for the reason.
This PR also adds docs for how to enable multiple listeners and updates the related configuration docs.
In addition, https://github.com/streamnative/kop/pull/742 marks kafkaListenerName as deprecated, but it also makes this config not work anymore. So this PR removes the config and related tests.

@Anonymitaet Please take a look at the doc part. If you have any question, please ping me.

@Anonymitaet comments are addressed, PTAL again.

I've addressed the comments, PTAL @Jennifer88huang


Multiple REST requests

I need data from multiple tables to display on a page so am calling multiple REST requests but only some requests are giving me the data and am getting empty array from the remaining requests.How to call multiple REST requests in restangular so that i get data from all the requests.. please help me..
thanks in advance

Were you able to resolve this issue?

No am not able resolve that instead i removed some calls from onLoad to onClick
.

Okay, I'm going to close this for now. Feel free to re-open if there's still an issue.


intercept sparse

See #19

Codecov Report

Merging #20 into master will decrease coverage by 0.94%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
- Coverage   97.05%   96.11%   -0.95%
==========================================
  Files           4        4
  Lines         170      180      +10
  Branches       12       14       +2
==========================================
+ Hits          165      173       +8
- Misses          3        4       +1
- Partials        2        3       +1

Impacted Files
Coverage Δ

celer/dropin_sklearn.py
100% <ø> (ø)
:arrow_up:

celer/wrapper.py
100% <ø> (ø)
:arrow_up:

celer/tests/test_homotopy.py
97.05% <100%> (+0.08%)
:arrow_up:

celer/homotopy.py
91.52% <77.77%> (-2.6%)
:arrow_down:

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 b0b0558...9c74156. Read the comment docs.

@agramfort ready for review if you fancy. Travis is green


Some changes

Codecov Report
Patch coverage: 34.88% and project coverage change: -0.51% :warning:

Comparison is base (8a16e7b) 53.53% compared to head (d9631f8) 53.03%.

:exclamation: Current head d9631f8 differs from pull request most recent head 3c4d93c. Consider uploading reports for the commit 3c4d93c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   53.53%   53.03%   -0.51%
==========================================
  Files          63       63
  Lines        2376     2391      +15
  Branches      760      769       +9
==========================================
- Hits         1272     1268       -4
- Misses        584      597      +13
- Partials      520      526       +6

Files Changed
Coverage Δ

...ollaborative_learning/ModelManagerReceiverNode.hpp
100.00% <ø> (ø)

.../collaborative_learning/ModelManagerSenderNode.hpp
100.00% <ø> (ø)

tools/agent_tool/src/cpp/main.cpp
0.00% <0.00%> (ø)

...l/src/cpp/user_interface/arguments_configuration.h
0.00% <ø> (ø)

.../collaborative_learning/ModelManagerSenderNode.cpp
45.16% <8.33%> (-13.86%)
:arrow_down:

...pp/src/cpp/types/model/ModelStatisticsDataType.cpp
50.00% <44.44%> (-0.68%)
:arrow_down:

...ollaborative_learning/ModelManagerReceiverNode.cpp
50.00% <66.66%> (-1.52%)
:arrow_down:

... and 1 file with indirect coverage changes

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


A PHP Error was encountered

l followed your installation direction but when l run in the browser, the following is what displays and below is the homepage though it doesn't display correctly. Can you please help.
Severity: 8192
Message: __autoload() is deprecated, use spl_autoload_register() instead
Filename: PHPMailer/PHPMailerAutoload.php
Line Number: 46
Backtrace:
File: C:\xampp\htdocs\ecommerce_kiril\application\libraries\SendMail.php
Line: 4
Function: _error_handler
File: C:\xampp\htdocs\ecommerce_kiril\application\libraries\SendMail.php
Line: 4
Function: require
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Loader.php
Line: 173
Function: _ci_load_library
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Loader.php
Line: 190
Function: library
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Loader.php
Line: 153
Function: libraries
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Loader.php
Line: 65
Function: initialize
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Base.php
Line: 55
Function: __construct
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Base.php
Line: 60
Function: __construct
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Controller.php
Line: 4
Function: require
File: C:\xampp\htdocs\ecommerce_kiril\application\third_party\MX\Modules.php
Line: 123
Function: include_once
File: C:\xampp\htdocs\ecommerce_kiril\application\core\MY_Controller.php
Line: 3
Function: spl_autoload_call
File: C:\xampp\htdocs\ecommerce_kiril\index.php
Line: 292
Function: require_once

l have got it resolved.
Note: If encountering problem when display in browser with php error, do the following;

remove from .htaccess file
"RewriteBase /"
if getting php error do the following:
open application/libraries/PHPMailer/PHPMailerAutoload.php
and change at line 46  __autoload to spl_autoload_register and it will work better

This is because autoload is deprecated. Hope that works for you.

@lampatt i know about this issue. Resolved - https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/commit/11d89fd0ccac266ea725b66d6eaa61dd9e0bd921


Reinstate test-softlayer-win2012r2-x64-1 in jenkins

I guess this was removed as it wasn't in jenkins. We should probably see if we can get into it and run some tests on it to increase our testing capacity (maybe after searching issues to see if there was a reason it was shut down and/or repurposed)
Originally posted by @sxa in https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1342#issuecomment-635336985

So.... This machine hasn't had the playbook ran on it. Interesting ...

Running the playbook on it, it failed halfway through the Cygwin role. When trying to restart the ansible connection, either through ansible all or ansible-playbook, I was met with the following error message:
localhost:ansible williamparker$ ansible all -i playbooks/AdoptOpenJDK_Windows_Playbook/hosts -u Administrator -m win_ping

[DEPRECATION WARNING]: DEFAULT_SQUASH_ACTIONS option, Loop squashing is deprecated and this configuration will no
longer be used , use a list directly with the module argument instead. This feature will be removed in version 2.11.
Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.

[WARNING]: ERROR DURING WINRM SEND INPUT - attempting to recover: WinRMError The pipe has been ended.  (extended fault
data: {'transport_message': 'Bad HTTP response returned from server. Code 500', 'http_status_code': 500,
'wsmanfault_code': '109', 'fault_code': 's:Receiver', 'fault_subcode': 'w:InternalError'})
<IP_ADDRESS> | FAILED! => {
    "msg": "winrm send_input failed; \nstdout: \nstderr PowerShell : The term 'PowerShell' is not recognized as the name of a cmdlet, \r\nfunction, script file, or operable program. Check the spelling of the name, or \r\nif a path was included, verify that the path is correct and try again.\r\nAt line:1 char:1\r\n+ PowerShell -NoProfile -NonInteractive -ExecutionPolicy Unrestricted - ...\r\n+ ~~~~~~~~~~\r\n    + CategoryInfo          : ObjectNotFound: (PowerShell:String) [], CommandN \r\n   otFoundException\r\n    + FullyQualifiedErrorId : CommandNotFoundException\r\n "
}

This was due to (I believe) part of the Cygwin role changing the Windows Path, adding a speech mark to the end of the path environment variable. This caused the last item at the end of the path, which happened to be the path the C:/Windows/System32/WindowsPowerShell/v1.0 to become invalid and therefore Ansible couldn't find powershell. Removing the added speech mark reinstated powershell into the path and allowed ansible to connect.

To do:

[ ] Connect to machine
[ ] Run playbook on machine
[ ] Create node in Jenkins
[ ] Connect node to Jenkins
[ ] Run test Grinders

Note: The machine has an unallocated 25GB virtual disk. The C:/ drive is at 100GB max capacity, with 22.1GB free after the playbook has ran through.

Running sanity.openjdk Grinder on it: https://ci.adoptopenjdk.net/job/Grinder/3249/console

Grinder ran through okay, closing the issue


assign in forRoot modelOptions

is there a way in forRoot, to assign modelOptions the value of updateOn: 'blur' by default ?

you can use a custom extension instead

ok, thanks


Selenium and Kotlintest

Hi.
I'm messing around with Selenium and Kotlintest. Quite easy to get this starting to work, but I'm running into an issue. I want to be able to start up the browser, do some stuff on a page, and then run a bunch of tests on the page. E.g., (using WordSpec, my preferred format):
RunSearchTest : WordSpec() {
  init {
  "When I run search the page" should {
    // ***
    // start up chrome driver, initialize page, do the search
    // ***
    "contain exactly 1 result" {
      // Test
    }
   "contain a search again button" {
      // Test
    }
  }
}

This works, but not exactly the way I want it to, of course - it initiates 3? webpages, and runs the test on the last two. I only want to initiate the selenium webdriver and navigate the pages once for each set of tests, of course, since it's so expensive.
Interceptors do not seem to be the answers to this problem. Wondering if there is an obvious simple solution to this that I've overlooked.

(Sorry for the very late reply, I'm looking at older issues now)
Due to how the isolation works, for every test the whole spec is executed. So "When I run search the page" will be executed every time before the test scenarios.
If this isn't the behavior you want, could you explain a little more what you're looking for?

For instance, the following behavior works:
class Foo(var int: Int = 0)

    init {
        val foo = Foo()

        "Foo" should {
            foo.int++

            "Should be executed only once" {
                foo.int shouldBe 1
            }

            "Not twice" {
                foo.int shouldBe 1
            }

            "Not thrice" {
                foo.int shouldBe 1
            }
        }
    }

foo.int++ is executed only once, and not a lot of times. So your selenium browser should open only once! Tell me if I didn't understand this correctly

Thanks for the response, though we've moved on from this as kotlintest (at the time at least - more than a year ago), didn't meet all our requirements.
The way the tests ran, it certainly seemed like foo.int++ was executed multiple times - or at least Selenium/ChromeDriver was, in the way it interacted with kotlintest. Might be some weirdness in the latter too, though - we had some issues with the version we were using even when working with plain junit before we found a good solution.
The test code involved here is lost, though (it was just a prototype), so I can't go back to check it out.

It as an issue with the way multiple tests were created. In fact, it was working as intended, but 3.1 onwards introduced new modes that do things in a way that makes more sense for your use case.


Update Puppet Providers - Gitter

Updates Puppet Providers -  Gitter section in references.
References issue #890
ScreenShots :

@Soumi7  make the CLI green

@Soumi7

@shraddhavp

Have updated. Once the CLI turns green, its ready to merge

@Rohitesh-Kumar-Jain

@iamrajiv as you requested changes, we will need your approval to merge this PR. Can you please approve this?


[ Feature request ] Connecting code blocks through pipes.

Hello good people of maid,
It would be useful if code blocks could communicate somehow.
Stdin - stdout
In the following pattern the code blocks are connected through the stdin-stdout:

## build

Builds important stuff

Snippet 1

```js
process.stdout.write("hi there from snippet 1");
```

Snippet 2

```js
process.stdin.on('data', console.log); // hi there from snippet 1
```

Next
Another way of allowing communication between code blocks is by injecting a method inside each code block in a similar way of how express.js does it with their middlwares.

## build

Builds important stuff

Code block 1

```js
next('hi there from snippet 1');
```

Code block 2

```js
console.log(maid.result); // hi there from snippet 1
```

This pattern can also be used to control the execution flow:

## build

Builds important stuff

Code block 1

```js
next('hi there from snippet 1');
```

Code block 2

```js
if (true) {
	console.log(maid.result); // hi there from snippet 1
}else {
	next("hi there from snippet 2");
}
```

Code block 3

```bash
 // This will never get executed

```

@AlvaroBernalG for the first snippet why couldn't you just have both of those in the same block of code?

@mfix22 Because the purpose of the example was to show how two code blocks could communicate.

hmm can you help me understand the use case?

@egoist yea, i'm curious to hear too.
If we decide to do it, it looks like that we should add some global variables like maid and next to the content when we execute the js code snippet. Which not sounds like a good thing.

@egoist

Some task are easier to do in some languages than others

There are some task that are easier to do in certain languages. Take for example downloading a config file from a website:
Node.js:
const http = require(‘http”)
http.get({
  hostname: 'mywebsite',
  port: 80,
  path: '/',
  agent: false  // create a new agent just for this one request
}, (res) => {
  // Do stuff with response
});

Now compare that solution with the simplicity of bash:
curl https://mywebsite.com > config.js

It will be easier to follow UNIX philosphy of doing one thing but doing it good, by allowing the creation of mini task rather than having one bloated code block.

// take a css file and strip out the comments.

THEN  =>
// take the result of stripping out comments and minimize it.

THEN =>
// Take the result of strippign out comments and minimized and save it in my S3 bucket .

@AlvaroBernalG
My opinion is that this is out of scope of what maid should and can do.
I do think your idea is actually very good, but it's going to add layers of complexity which maid shouldn't have to deal with directly.
To give some background, I have already developed a similar system to what you have described ( connecting code snippets through STDIN / STDOUT or as HTTP middlewares ). Mostly we used HTTP, but it's all essentially just generic streaming interfaces.
If you are interested in how we did this, here are some examples including chaining services of multiple languages which may inspire you:
https://github.com/Stackvana/microcule/tree/master/examples
https://github.com/Stackvana/microcule/blob/master/examples/express-chain-services.js
I'm glad to offer any input I can. I've been working with this kind of development pattern for a few years and am eager to see more developers thinking about the problem space.

@Marak @egoist I do agree this wouldn't be trivial to add, so I understand if you don't want to push this forward.
@Marak. Awesome stuff, I will definitely have a look later.


Refactoring labels and additional labels support

Support recommanded labels (Helm and Kubernetes) : https://helm.sh/docs/chart_best_practices/labels/ and https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/
Feature to add custom labels to all resources.

/reopen

@dmitryax i make some new commits


Rethink UnexpectedStatus model

The UnexpectedStatus wraps the request method and URL which is handy, but it's a pain to construct. I proprose:

it should instead wrap a RequestPrelude, so as to expose more information about the request
It should have a convenience constructor that takes a Request

Should it also include the ResponsePrelude, to capture any response headers along with the status?

Sounds good to me 👍


Louisville Metro Government Notification

If Louisville Metro Government has a notification push to IFTTT trigger.

There is an inkling of a plan to build a centralized api to accept requests for notifications and then distribute those notifications through various channels, email, text, ifft, Alexa, Google home, push, etc. as new technologies arise the service can be extended without need to reimplement the initial request.

This issue was moved to office-of-civic-innovation/IFTTT-Smart-Louisville#1


⚠ GitHub has degraded performance

In f868f7f, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 402 ms

Resolved: GitHub performance has improved in 0a3d24a.


[Azure] Allow users to specify VPN settings

They should be able to specify specifics of VPN configurations if they'd like through a config file.

@seankimkdy any chance this could be "good first issue", "help wanted", or "easy"?

@divega Idk if it's any of those three?

good first issue/easy: requires a lot of changes from cli, orchestrator, and all clouds and would be the first instance in Paraglider where we allow the user to customize network settings. I think this one might be one that might benefit from more discussion in terms of how much customizability we want to permit users.
help wanted: not really a top priority because it's just a nice to have feature that a customer may want in the future.


Revert "Merge pull request #3601 from codesections-forks/signatures"

This reverts #3601, which was merged due to a miscommunication in #3578.  To be discussed further in that issue.

Thanks, @codesections


Fixes step navigation when there are multiple wizards on a page

Changed the step navigation events to directly dispatch to the step's wizard component. This way navigating between steps still works properly when there are multiple wizards on a page.

The tests seem to fail for this. Could you have a look?

Thanks!


installing wheel for pyahocorasick==1.4.1 in python3.6.8

Hello,
I was installing pyahocorasick version 1.4.1 using pip, but it fails to build the wheel. I don't get where this error comes from. Please help me solve this issue.

You are missing MSVC installation on your computer. The error message gives you the URL for download the compiler.

Thank You. I'll try with this


Feat/demoeng 1370 create environment deploy

Summary
Creates github action for creating a new Github environment from given credentials

Some of the files in this PR are tied to this other PR
The main change of this PR is environment-deploy.yaml
You'll need to generate a Github Personal Access Token and add to the repo as GH_API_PAT due to pipeline permissions
Checklist

[ ] Tested changes end to end
[ ] Updated documentation
[ ] Requested one or more reviewers

Hey @lvaladares
Now that we have taken the approach (at least for the immediate short term) to leverage the cli deployment of services - could you revise this PR to leverage those changes instead of the TF application of the services package?
this will be the least invasive right now as we perform further changes to enhance the TF management.  Then we can look at introducing serverless as TF deployment at a later date (for the benefit of sid reference in TF config, unless we come up with another approach to inject the SIDs but remove the complexity of all the scripting)

😢