SASS Error. Failed to Compile!

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[ ] I'm reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior
I´m looking for the first render but i can´t.
Current Behavior
Failed to Compile.
./src/assets/scss/argon-dashboard-react.scss (./node_modules/css-loader/dist/cjs.js??ref--6-oneOf-5-1!./node_modules/postcss-loader/src??postcss!./node_modules/resolve-url-loader??ref--6-oneOf-5-3!./node_modules/sass-loader/dist/cjs.js??ref--6-oneOf-5-4!./src/assets/scss/argon-dashboard-react.scss)
To import Sass files, you first need to install node-sass.
Run npm install node-sass or yarn add node-sass inside your workspace.
Require stack:

C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\sass-loader\dist\getDefaultSassImplementation.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\sass-loader\dist\getSassImplementation.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\sass-loader\dist\index.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\sass-loader\dist\cjs.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\loader-runner\lib\loadLoader.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\loader-runner\lib\LoaderRunner.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\webpack\lib\NormalModule.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\webpack\lib\NormalModuleFactory.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\webpack\lib\Compiler.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\webpack\lib\webpack.js
C:\Users\dsabi\Downloads\argon-dashboard-react-master\argon-dashboard-react-master\node_modules\react-scripts\scripts\start.js

Hello @Dsabillon,
Thank you for using our products, it looks like you opened an issue on the wrong repo.
Please make sure you are opening it on the product repo.
Best,
Stefan

Hello @groovemen
I think this is the right place.
I found your products online, and I'm trying to use your Dashboard components to create one. But at the moment of starting the project, it gives me that error.

Hello again,
Please let us know if you are using Argon Dashboard React or Material Dashboard HTML.
Regards,
Stefan

Hi
Thanks for take time for reply my question.
I´m using this :
https://www.creative-tim.com/product/material-dashboard-react
I´ve trying to figure out how use the components!

Hello there, @Dsabillon ,
You have opened the issue on the wrong repository.
Please use the following form for us to understand your issue: https://www.creative-tim.com/new-issue/material-dashboard-react
Since the issue was opened wrongfully, we will close it here and we'll wait for the proper one to be opened.
Best,
Manu


Status bar labels are frequently misaligned/off-center

When looking at the progress/stage indicator of an application, the titles are frequently misaligned and not consistent.
The following are screenshots with lines drawn through the center of the stage points to better show this misalignment. Some are better than others:

Definitely not a mission-critical bug but would still be nice for appearance.

This needs to be rebuilt with simpler HTML structure  modern css, like "grid" e.g.
The current versions tries to pixel adjust things and it fails for different fonts, different screen sizes and different zoom levels.
Since we are using "system-ui" fonts it will look different on masOS vs Windows etc.


'help' and 'version' should be first class commands

Please make 'lxc help' do the same thing as 'lxc --help'. Also, 'lxc version' should do the same thing as 'lxc --version'

help should already be behaving the exact same as --help or as not passing any sub-command, that's the default behavior of the command line parser we use.
We'll re-add our own copy of the version subcommand as the parser only provides us with a --version argument and not a subcommand.

stgraber@castiana:~/data/code/lxc/lxd (stgraber/master)$ lxc version
Client version: 3.0.0
Server version: 3.0.0.beta7

Sent a pull request which addresses this issue. After it's reviewed and merged, the fix will be included in the stable-3.0 branch and be included in the candidate snap as well as in the next batch of cherry-picks for the LXD deb.


Btw dude can u post new defoult crds that used in CHAOS docker continer.

Hello. Thank u for that good tool.
U updated default crds.
That once not worked anymore.
default username: admin and password: admin
Can you provide new one plz?

Hi, sorry for delay.
The lasts docker images have an old password.
Current user and password is admin.
I updated now the docker image, you can use the tag latest or v5.0.4 to get the latest version (updated on readme).
tiagorlampert/chaos:latest
https://github.com/tiagorlampert/CHAOS#2-docker
Thanks for your feedback.


Metrics: Review mechanism for measuring message processing time (requirements and A&D)

I'm submitting a ...

[ ] bug report
[ ] feature request
[ ] support request

Do you want to request a feature or report a bug?

What is the current behavior?

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Version: 0.2.x
Environment: [kubernetes, docker-compose, standalone]
Operating system: [Ubuntu 16.04, Fedora 28, Ubuntu on VirtualBox (Windows 10), etc.]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Ticket opened after the ticket analysis meeting (June, 19th).
Related to ticket #1307


Italian translation

Okay, I've been able to create the pull request. I don't know if I've got everthing right on the programming side, but I should.
Anyway I need to discuss about an issue that occurs with my alexa whenever you can.
To be honest I don't know why Index keeps appearing as .json since in my computer it's a .js
Anyway it's to not that it's a .js, just like the original index from the master branch

You have to make some changes:
e.g. the function GetVideoIntent is used for the english language and GetVideoGermanIntent for the german version. Your new function should call something like that GetVideoItalianIntent.
Every function you insert, should relate to the Italian version. So you can pick one languages without conflict.

sure I will. Could you tell me in which subfolder can I find the base GetVideoIntent function?

You may find the function GetVideoGermanIntent into index.js 631 and the call come from Interaction-Model-German.json 47. For the Italian version you should copy the Interaction-Model-German.json  and rename it to Interaction-Model-Italian.json than replace every German word with Italian.
The next step is to copy each function related to Interaction-Model-German.json in index.js and replace with the new functions names from your Interaction-Model-Italian.json. That should be work.

Here is an useful link  to fork and create pull request  When you run git add filename.xx make sure you add only files you changed or you want to pull. Use after git add git commit -m "your message" to explain what changes you did.


Access denied for image pull using latest SAM

Invoking com.somecompany.aws.handler.GatewayHandler (java8)
Image was not found.
Removing rapid images for repo public.ecr.aws/sam/emulation-java8
Building image......
Failed to build Docker Image
NoneType: None
Error: Error building docker image: Get "https://public.ecr.aws/v2/": net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
This is using latest intellij with latest aws toolbox, with latest sam

Do you still have this issue?
I'm not able to reproduce:
% docker pull public.ecr.aws/sam/emulation-java8
Using default tag: latest
latest: Pulling from sam/emulation-java8
b844c5fb5a5a: Pull complete
e4167c993ac8: Pull complete
e3b91baeb2b6: Pull complete
Digest: sha256:7b50d8df0b2f99c3ceac8c9ebb8849c6f4be419c5301d03b19e61f052f97edd5
Status: Downloaded newer image for public.ecr.aws/sam/emulation-java8:latest
public.ecr.aws/sam/emulation-java8:latest

There has been no activity for around 6 days and we are not able to reproduce the issue. I am resolving this issue for now, please re-open if you have any questions.


Add more integration tests for ks commands

We have unit tests, however there are currently only integration tests for apply and delete. We should be supporting the other commands.

Closing as duplicate of #216


Keras load_model fails to load models with BatchNormalization layer when saved in non-eager mode

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Y
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS 10.13.6
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):
tf version = 2.0.0-dev20190622
tf git version = v1.12.1-4759-g9856697d8b
Python version: 3.6.4
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: NA
GPU model and memory: NA

Describe the current behavior
This issue is very similar to #3628, but it only happens in non-eager mode in TF 2.0.
You can save a model (having batch norm layer) in non-eager mode successfully. However, you cannot load the model in any mode.  Load model fails with:
ValueError: Node 'AssignMovingAvg/sub/x' expects to be colocated with unknown node 'bn_layer/moving_mean'

Seems like the model is not saved correctly.
Describe the expected behavior
Save model should produce a correct SavedModel in non-eager mode so that it can be loaded later.
Code to reproduce the issue

import tensorflow as tf

tf.compat.v1.disable_eager_execution()

## model architecture
input_layer = tf.keras.Input(shape=(28, 28, 1), name='image_input')
layer = tf.keras.layers.ZeroPadding2D(padding=(3, 3), name='initial_padding')(input_layer)
# add convolutional layer
layer = tf.keras.layers.Conv2D(
    filters=16,
    kernel_size=8,
    padding='same',
    name='conv_layer'
)(layer)
# batch normalization
layer = tf.keras.layers.BatchNormalization(axis=3, name='bn_layer')(layer)
# activation
layer = tf.keras.layers.Activation('relu', name='activation_layer')(layer)
# down sample
net = tf.keras.layers.MaxPooling2D(pool_size=(3, 3), strides=(2, 2))(layer)
# flatten
net = tf.keras.layers.Flatten(name='flatten_layer')(net)
# dense layer with ReLU-activation.
net = tf.keras.layers.Dense(64, activation='relu', name='dense_layer')(net)
# dropout layer
net = tf.keras.layers.Dropout(0.2, name='dropout_layer')(net)
# last fully-connected / dense layer with softmax-activation so it can be used for classification.
output_layer = tf.keras.layers.Dense(10, activation='softmax', name='predictions')(net)
# creating the model
model = tf.keras.Model(inputs=input_layer, outputs=output_layer, name='test1')

# loading data
(x_train, y_train), (x_test, y_test) = tf.keras.datasets.mnist.load_data()
# add channel to x; also divide by 255 to normalize the data
x_train = x_train.reshape(60000, 28, 28, 1)#.astype('float32') / 255
x_test = x_test.reshape(10000, 28, 28, 1)#.astype('float32') / 255

# create the training dataset
train_dataset = tf.data.Dataset.from_tensor_slices((x_train, y_train))
# shuffle, batch and prefetch for optimizing io reads
train_dataset = train_dataset.shuffle(buffer_size=1024).batch(64).prefetch(1024)
# create the validation dataset.
val_dataset = tf.data.Dataset.from_tensor_slices((x_test, y_test))
# batch and prefetch for optimizing io reads
val_dataset = val_dataset.batch(64).prefetch(1024)

# compile the model
model.compile(
    loss='sparse_categorical_crossentropy',
    optimizer='rmsprop',
    metrics=['accuracy']
)

# fit the model
history = model.fit(
    train_dataset,
    validation_data=val_dataset,
    epochs=2,
    steps_per_epoch=50
)

# the returned "history" object holds a record of the loss values and metric values during training
print('\nhistory dict:', history.history)

model.save('mnist_model')

del model
# recreate the exact same model purely from the file:
model = tf.keras.models.load_model('mnist_model')

Other info / logs
WARNING: Logging before flag parsing goes to stderr.
W0624 13:37:50.545702<PHONE_NUMBER>85888 deprecation.py:506] From /temp/v36/lib/python3.6/site-packages/tensorflow_core/python/ops/resource_variable_ops.py:1624: calling BaseResourceVariable.__init__ (from tensorflow.python.ops.resource_variable_ops) with constraint is deprecated and will be removed in a future version.
Instructions for updating:
If using Keras pass *_constraint arguments to layers.
2019-06-24 13:37:51.383301: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-06-24 13:37:51.398677: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x7f90837c85b0 executing computations on platform Host. Devices:
2019-06-24 13:37:51.398709: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): <undefined>, <undefined>
2019-06-24 13:37:51.728750: W tensorflow/compiler/jit/mark_for_compilation_pass.cc:1541] (One-time warning): Not using XLA:CPU for cluster because envvar TF_XLA_FLAGS=--tf_xla_cpu_global_jit was not set.  If you want XLA:CPU, either set that envvar, or use experimental_jit_scope to enable XLA:CPU.  To confirm that XLA is active, pass --vmodule=xla_compilation_cache=1 (as a proper command-line flag, not via TF_XLA_FLAGS) or set the envvar XLA_FLAGS=--xla_hlo_profile.
W0624 13:37:52.842435<PHONE_NUMBER>85888 deprecation.py:323] From /temp/v36/lib/python3.6/site-packages/tensorflow_core/python/keras/optimizer_v2/optimizer_v2.py:460: BaseResourceVariable.constraint (from tensorflow.python.ops.resource_variable_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Apply a constraint manually following the optimizer update step.
Train on 50 steps, validate on 157 steps
Epoch 1/2
50/50 [==============================] - 4s 73ms/step - loss: 1.8029 - accuracy: 0.5256 - val_loss: 0.5635 - val_accuracy: 0.8345
Epoch 2/2
50/50 [==============================] - 3s 58ms/step - loss: 0.6426 - accuracy: 0.7937 - val_loss: 0.4020 - val_accuracy: 0.8787

history dict: {'loss': [1.8028993177413941, 0.6426236051321029], 'accuracy': [0.525625, 0.79375], 'val_loss': [0.563538867861602, 0.40199054775249426], 'val_accuracy': [0.8345, 0.8787]}
2019-06-24 13:38:02.261349: W tensorflow/python/util/util.cc:268] Sets are not currently considered sequences, but this may change in the future, so consider avoiding using them.
Traceback (most recent call last):
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/framework/importer.py", line 427, in import_graph_def
    graph._c_graph, serialized, options)  # pylint: disable=protected-access
tensorflow.python.framework.errors_impl.InvalidArgumentError: Node 'AssignMovingAvg/sub/x' expects to be colocated with unknown node 'bn_layer/moving_mean'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "error_showcase_graph.py", line 71, in <module>
    model = tf.keras.models.load_model('mnist_model')
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/keras/saving/save.py", line 142, in load_model
    return saved_model_load.load(filepath, compile)
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/keras/saving/saved_model/load.py", line 86, in load
    model = tf_load.load_internal(path, loader_cls=KerasObjectLoader)
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/saved_model/load.py", line 506, in load_internal
    export_dir)
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/keras/saving/saved_model/load.py", line 102, in __init__
    super(KerasObjectLoader, self).__init__(*args, **kwargs)
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/saved_model/load.py", line 111, in __init__
    meta_graph.graph_def.library))
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/saved_model/function_deserialization.py", line 301, in load_function_def_library
    copy, copy_functions=False)
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/framework/function_def_to_graph.py", line 64, in function_def_to_graph
    importer.import_graph_def(graph_def, name="")
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/util/deprecation.py", line 507, in new_func
    return func(*args, **kwargs)
  File "/temp/v36/lib/python3.6/site-packages/tensorflow_core/python/framework/importer.py", line 431, in import_graph_def
    raise ValueError(str(e))
ValueError: Node 'AssignMovingAvg/sub/x' expects to be colocated with unknown node 'bn_layer/moving_mean'

I have tried on Colab with TF version 2.0.0-dev20190622 and was able to reproduce it on only non eager mode.

Thanks for reporting this sissue! This should now be fixed with this change: https://github.com/tensorflow/tensorflow/commit/769900b011cf6a31f8e8f919e97b70900f9825d8#diff-5738cc9149537416bc9a07c7cfefae42

Thank you @k-w-w
I confirm that the issue is resolved in the tf-nightly


bash, zsh, gcloud or a sane environment to allow for shell scripts for Cloud Deplyment

Hi all. I know this was reported/requested previously ( #270 ) and I wholeheartedly agree with the policy not to allow anything non-POSIX on the firewall, let alone some shells lying around to for someone to exploit and give anyone a foothold on the network.
if there was to be a shell accessble, i agree with @fichtner that zsh would be the one but even so, he mentions compelling reasons not to have anything but plaing vanilla POSIX available. after all, this is a firewall appliance, not a full fledged networked workstation.
HOWEVER i believe there is a valid use case for a shell to be available: Cloud Centric Deployments which is my current case. Most public cloud environments require a bunch of scripts for instance spin-up, tear-down and monitoring.
Notably cloud-init on most environments, nova and suprnova for OpenStack / Rackspace, the superb gcloud from GCE and Google's stackdriver monitoring an logging agents all require bash at the very least during install time. (can't speak for azure or AWS, but i'm sure both do use cloud-init at least.
Perhaps this is a desirable feature as the use case for a VPN is pretty common, specially on GCE. Although they provide a nice cloud shell, getting a solid ssh session to your instances can be quirky, unless you fool around with their key-provisioning mechanisms and inject persistent keys on your images, defeating the purpose of their key rotation and deployment mechanism (which BTW, requires bash and the gcloud SDK installed on the instance)
So having a bastion instance to either just ssh tunnel to your gcloud VPC, or provide full a fledged VPN is a necessity and Opnsense is the perfect tool for that.
Bootstrapping and creating the VM image on Gcloud was ridiculously easy, just using the serial install live cd image straight into image import boots a working Opnsense VM, however the network is not configured on first boot, and one has to manually ifconfig the network cards and add the default route the first time around.
Google does require installing the SDK to be installed on the VM in order for it to be managed under their magic spells and it does require a shell at least for install.
So, perhaps some solutions would be:

a cloud friendly install image  with zsh available either installed or trought the pkg repo.
some sort of restricted, jailed environment, or a mechanism to provide single-use shell environments maybe taking advantage of jails lightweight containers to do so, avoiding compromising security and allowing for cloud-agnostic DIY cloud image creation
providing gcloud-sdk packages in the pkg repo ( the easy cheap solution to my specific use case ;) )

I can provide some documentation on how to get Opnsense running smoothly on gcloud now that i'm in the middle of it.
Please consider this issue as a suggestion for future releases.
i was able to work around by getting a statically compiled bash on my cloud instance, but we have to assume most people will be careless by leaving such binaries around on the system, (i'd consider it compromised at that point) so its not a bad idea to provide a controlled and secure alternative for this increasingly common use case.
Thx n.n

Hi @hlecuanda,
I can see the need, though adding a shell won't have it end up in the default install. If it were a default, we'd have to reconsider the stance for POSIX shell because people would start writing bash and zsh and that's not so good as stated earlier.
I like the gcloud-sdk thing. I'll take a look :)
Cheers,
Franco

Whoa, almost a year to the date exactly! And my OpenSense FW has been churning at it since then, not a hiccup, not an issue, hell I hardly remember it was there!
Good software just works, but Great  siftware is noticed only when absent. Thank you and everyone invokced:grin: @fitchner


Any plan on embedding the nrf52840 directly on the board?

It seems like JLCPCB now has NRF52840-QIAA-R in stock. If we embed the nrf52840 directly on the board, the whole board can be assembled using JPCPCB SMT soldering service, with no hand-soldering required.
It's also probably cheaper to do it this way since we don't have to purchase the components separately

https://github.com/joric/nrfmicro/wiki/PCBA


Added sample deployment script.

Test:
./launch.sh --marathon marathon.service.consul --config hello-world.json
{"id":"/hello-world","cmd":null,"args":null,"user":null,"env":{},"instances":2,"cpus":0.1,"mem":128.0,"disk":0.0,"executor":"","constraints":[],"uris":[],"storeUrls":[],"ports":[],"requirePorts":false,"backoffSeconds":1,"backoffFactor":1.15,"container":{"type":"DOCKER","volumes":[],"docker":{"image":"keithchambers/docker-hello-world","network":"BRIDGE","portMappings":[{"containerPort":80,"hostPort":0,"servicePort":0,"protocol":"tcp"}],"privileged":false,"parameters":{}}},"healthChecks":[],"dependencies":[],"upgradeStrategy":{"minimumHealthCapacity":1.0},"version":"2015-03-06T00:40:51.914Z"}

Looks good.


refactor: remove unnecessary replica selectors

What this PR does
This patch removes unnecessary replica selectors - internal/admin.(randomReplicaSelector) and
internal/admin.(victimSelector).
Which issue(s) this PR resolves
Resolves #122
Anything else

Current dependencies on/for this PR:

main

PR #14   👈

PR #15

This comment was auto-generated by Graphite.

Codecov Report

Merging #14 (c068b22) into main (2a4768e) will increase coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   62.84%   62.99%   +0.15%
==========================================
  Files         125      125
  Lines       16951    16874      -77
==========================================
- Hits        10653    10630      -23
+ Misses       5748     5696      -52
+ Partials      550      548       -2

Impacted Files
Coverage Δ

internal/admin/replica_selector.go
84.52% <ø> (-5.54%)
:arrow_down:

pkg/util/runner/stopwaiter/stopwaiter.go
85.71% <0.00%> (-14.29%)
:arrow_down:

pkg/mrc/mrconnector/mr_connector.go
73.28% <0.00%> (-2.68%)
:arrow_down:

...metarepos/dummy_storagenode_client_factory_impl.go
62.50% <0.00%> (-1.39%)
:arrow_down:

internal/storagenode/storagenode.go
61.85% <0.00%> (+0.37%)
:arrow_up:

internal/storagenode/replication_server.go
80.00% <0.00%> (+1.00%)
:arrow_up:

internal/storagenode/logstream/committer.go
82.91% <0.00%> (+1.06%)
:arrow_up:

internal/metarepos/storage.go
80.57% <0.00%> (+1.09%)
:arrow_up:

pkg/util/netutil/netutil.go
50.94% <0.00%> (+1.88%)
:arrow_up:

internal/metarepos/report_collector.go
84.61% <0.00%> (+5.93%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix: missing information for cs0759

Summary
Missing information for cs0759
Fixes #37518

Internal previews

📄 File
🔗 Preview link

docs/csharp/misc/cs0759.md
Compiler Error CS0759

@luizhlelis
I'm closing this as very stale. The modified article has been combined with other related errors in a different PR.


Importing Quickstrom dependency for purescript projects

It would be great to have a published library for Quickstrom so it can be imported to have completions etc.
My current hack is to clone Quickstrom and change package.dhall of my project so that an addition points to the qucickstrom repo:
let additions = {=} with quickstrom = ../quickstrom/dsl/spago.dhall as Location

Yeah, I've also hacked around this for now. In the Nix shell there's a wrapped purs ide server with similar paths configured:
https://github.com/quickstrom/quickstrom/blob/main/shell.nix#L12-L19
Also, I think that the purescript-language-server has support for setting source globs explicitly using an environment variable:
https://github.com/nwolverson/purescript-language-server/pull/76
For a more nicely packed up solution for Quickstrom, would it make sense to include a language server for specifications in the CLI? You'd need to reconfigure your editor to use quickstrom for those files. I really don't want to require people to install the PureScript tools just to write specs (with good editor support).

Also worth noting, I think publishing a Quickstrom package in the normal PureScript sense would be confusing. It has no value and can't work on its own. It's almost worth thinking about the specification language as not being PureScript, and more like a new language that happens to use PureScript as an implementation detail. The evaluation semantics is and will most likely stay different from PureScript, also.

Maybe a good end result could be a section in the doc talking about lsp support for quickstrom docs? One possible solution is spago and a published package of the DSL but maybe there's something better?

Yeah. I'm going to investigate bundling an LSP thing with quickstrom.

In the longer run the PureScript-based specs are likely going away, so I've decided to not spend time on this right now. If anyone has suggestions on a low-effort improvement on the current situation, please open a new issue.


View.propTypes.style throwing errors in Expo

View.propTypes.style is causing errors in new versions of expo,  propTypes should be taken out altogether.

@Liroo would you mind reviewing this, please?
Thank you.

Obviously not using the same version of react-native, actually i'm just getting a warning but this is no changing anything, merged.


fix(components): Properly support kinds = false as documented

If config.user.kinds is set to false, the expression
config.user.kinds == false and nil or get_kind_icon(nesting.kind)

still evaluates get_kind_icon(nesting.kind), which errors out at config.user.kinds[kind_to_type[kind]].
To fix this, just make get_kind_icon check if config.user.kinds is false.

Thanks for contributing! Just format with stylua and I'll merge.


Improves Text component

What:

Adds new secondary props and deprecates the weak
Migrates Text components to color tokens
Improves the documentation

Closes #103

I'm sorry it took that much time. All the comments are addressed, please, take a look

@nlopin I left some comments

@rafael-sepeda take a look


Use connect on HOC compoennts

Is there any way one could use connect on HOCs and fetch a data from store?
import React from 'react';
import {connect} from 'react-redux';

function withCurrentUser(Component) {
  return function(...props) {
    return <Component currentUser={/* Pass current user fetched from store here */} {...props} />
  }
}

function mapStateToProps(state) {
  return {
    currentUser: state.user
  }
}

export default connect(mapStateToProps)(withCurrentUser);

Not written that way, since connect expects an actual component type as the second function's parameter.  However, this should work:
import {compose} from "redux";
import {connect} from "react-redux";

const composedHoc = compose(
    connect(mapStateToProps),
    withCurrentUser
);

export default composedHoc;

@markerikson That helped a lot! thanks 👍

You can certainly flip the order of composition so that the connect wrapper is inside the withCurrentUser wrapper.  Also, you may want to try binding specific action creators so that they become known props, instead of skipping the mapDispatch argument.  See http://blog.isquaredsoftware.com/2016/10/idiomatic-redux-why-use-action-creators/ for some related thoughts.

Random question, stumbled on this thread.
I'm getting cannot call a class as a function when I copy paste out your code here. Any idea why?

It's a bit old, but you could achieve this with a little modification on your code:
import React from 'react';
import {connect} from 'react-redux';

function withCurrentUser(Component) {
  function yourFunction(...props) {
    return <Component currentUser={/* Pass current user fetched from store here */} {...props} />
  }
  return connect(mapStateToProps)(yourFunction);
}

function mapStateToProps(state) {
  return {
    currentUser: state.user
  }
}

export default withCurrentUser;

Also a bit late but here is my solution:
`import React from 'react'
import { connect } from 'react-redux'
const Container = Comp => {
const Wrapper = props => (
class extends React.Component {
render () {
return <Comp {...this.props} />
}
}
)
return connect(mapStateToProps)(Wrapper());
}
function mapStateToProps ({ uuid }) {
return { uuid }
}
export default Container
`

Also a bit late but here is my solution:
`import React from 'react'
import { connect } from 'react-redux'
const Container = Comp => {
const Wrapper = props => (
class extends React.Component {
render () {
return <Comp {...this.props} />
}
}
)
return connect(mapStateToProps)(Wrapper());
}
function mapStateToProps ({ myValue }) {
return { myValue }
}
export default Container`

Also a bit late but here is my solution:
import React from 'react'
import { connect } from 'react-redux'

const Container = Comp => {
  const Wrapper = props => (
    class extends React.Component {
      render () {
        return <Comp {...this.props} />
      }
    }
  )

  return connect(mapStateToProps)(Wrapper());
}

function mapStateToProps ({ myValue }) {
  return { myValue }
}

export default Container

@BradyEdgar94  @markerikson  All redux actions are passing into wrapped component when connect is used in hoc. May this behaviour be prevented?

All suggested solutions didn't work for us. Check the error below. We figured out that the problem was in version 7.0.3. Roll back to 5.0.1 fixed everything.

@yuzhakovvv : if you're seeing that error, please make sure that you have exactly matching versions of React and ReactDOM in your app, and that you're not pulling in any other versions of React anywhere.


not compatible with androidx

this library not compatible with androidx is there any new version make it support with android x libs
and if there is kotlin version will be very helpful and ktx version @mikepenz

as long as the android x libraries are not in a final version there won't be an update yet.

@mikepenz :( that disappoint me

it will be updated as soon as it is stable. until then you can always create a PR or build it for yourself to make it compatible

@mikepenz thanks a lot your library help me a lot most of my developers asking me about my secret of the the adapter i always say it's @mikepenz 👍 🥇

Thank you very much.

Closing this for the time being. Please open a new ticket as soon as androidx is final ;)


boards: nrf9160_pca10090: Clean up default configurations

This PR contains two patches that remove a couple of entries from default configurations for the nrf9160_pca10090 board. These entries do not make much sense for this board, as the corresponding features are not present in the nRF9160 chip.
drivers: entropy: nrf5: Fix dependency of the enabling Kconfig option
This driver makes use of the nRF RNG peripheral, so it can be used only
for SoCs that are equipped with one, and not all nRF SoCs are.
The option enabling the driver should then depend on HAS_HW_NRF_RNG,
which indicates the presence of this peripheral in a given SoC.
This patch removes also entries disabling this driver in default
configurations for nRF9160 SoC, as these were needed only because
of the invalid dependency of this option.
soc: nrf9160: Remove unsupported Kconfig option GPIO_AS_PINRESET
In nRF9160 the reset pin is a dedicated one, it cannot be configured
as a regular GPIO pin, so this option should not be presented to users
building for this SoC, to not generate confusion.

@aescolar could you take a quick look at this trivial patch?

@anangl I remove the "Meta" tag as that was supposed to be used for "issues" which are used to collect other issues

Only looked at it quickly, so tell me if I'm misunderstanding.
if a symbol needs to be a seen by a large part of the tree, even its for a kinda "hackish" reason, then I think it's best to put a single definition somewhere high up, like @aescolar suggested.
That's also a good spot to add a comment that explains why it needs to be there.

Only looked at it quickly, so tell me if I'm misunderstanding.
If a symbol needs to be a seen by a large part of the tree, even its for a kinda "hackish" reason, then I think it's best to put a single definition somewhere high up, like @aescolar suggested.
That's also a good spot to add a comment that explains why it needs to be there.

In this particular case, I don't think we have many options. HAS_HW_NRF_RNG cannot be (naturally) defined elsewhere than in soc/arm/nordic_nrf
And if the nrf52_bsim board needs it, we need to make that visible - or re-define it
What do you think @ulfalizer ?

@aescolar @ulfalizer @ioannisg
What about modifying the PR this way, i.e. adding the read of soc/arm/nordic_nrf/Kconfig.peripherals to boards/posix/nrf52_bsim/Kconfig, so that HAS_HW_NRF_RNG could be simply selected in boards/posix/nrf52_bsim/Kconfig.board? I think it is less ugly than duplicating the definition of HAS_HW_NRF_RNG, but still all the HAS_HW_NRF* symbols don't need to be parsed for all architectures (what I see as a downside of what @aescolar suggested), only when the nrf52_bsim board is actually selected.

@aescolar @ulfalizer @ioannisg
What about modifying the PR this way, i.e. adding the read of soc/arm/nordic_nrf/Kconfig.peripherals to boards/posix/nrf52_bsim/Kconfig, so that HAS_HW_NRF_RNG could be simply selected in boards/posix/nrf52_bsim/Kconfig.board? I think it is less ugly than duplicating the definition of HAS_HW_NRF_RNG, and still all the HAS_HW_NRF* symbols don't need to be parsed for all architectures (what I see as a downside of what @aescolar suggested), only when the nrf52_bsim board is actually selected.

I am OK with that. @aescolar pls, comment :)

@ioannisg @anangl I think that if the discussion drags for to long, you can merge this as it is if you are in a hurry to get it for 2.0

What about modifying the PR this way..

Ok, it is less ugly (I would add the source line at the top of the file instead of the bottom though).
But I would certainly want @ulfalizer 's blessing on this.

Ok, it is less ugly (I would add the source line at the top of the file instead of the bottom though).
But I would certainly want @ulfalizer 's blessing on this.

Alright, let's see what @ulfalizer has to say about that.

I updated the PR as proposed above, with applied @aescolar's suggestion regarding moving the source line.


encoding/xml: failure to handle conflicting tags in different namespaces

What does 'go version' print?

go1.3.1

What steps reproduce the problem?

Try to decode document with the same tag in multiple namespaces
<example>
    <title>Example</title>
    <link>http://example.com/default<;/link>
    <ns:link xmlns:ns="http://www.w3.org/2005/Atom";>http://example.com/ns<;/ns:link>
</example>`

http://play.golang.org/p/FT56UXolGO

What happened?
main.Example field "Link" with tag "link" conflicts with field
"AtomLink" with tag "http://www.w3.org/2005/Atom link"

What should have happened instead?

It should be possible to decode both tags in this document.

Also, if the name-spaced tag isn't provided, the second "link" element shadows
the first though it's in another namespace.

I just ran into this issue working with a legacy SOAP API that had a lot of namespaces in the fields.

I am also having this issue (funnily enough with RSS readers as well). Is there a workaround, yet or do we have to wait for Go 10 (ie. #14407)

I found a workaround by creating a custom string-type with a custom unmarshal function. If anyone is interested, here it is:
type NonNamespaceString string

func (s *NonNamespaceString) UnmarshalXML(d *xml.Decoder, start xml.StartElement) error {
	if start.Name.Space != "" {
		// We do not want a namespace, so we need to consume it
		d.Skip()
		return nil
	} else {
		str := ""
		d.DecodeElement(&str, &start)
		*s = NonNamespaceString(str)
	}

	return nil
}

I also create a gist with a unit test, if you are interested.
It seems to work and can almost be used as a drop-in replacement for any string. You just need to cast it back to a string if you want to use it.

Comparing namespaces of fields was missing and detecting false conflicts. A fix is submitted.


Typecheck frontend in dev: add concurrently and tsc

Fixes #308 by using https://www.npmjs.com/package/concurrently which can run both vite and tsc
Does not show colors from tsc (as seen in the picture), but i think that is not a super big deal.
And better than no type-checking, which is the current state.
Seems to be solved in next turbo version : https://github.com/vercel/turbo/issues/223#issuecomment-1895099649
Pic:

Solved by #313


Kubeflow with OIDC possible scopes error

Hi, for context, I work for a DS team within large organization. Myself and a DevOps team member are attempting to deploy Kubeflow which must conform to security requirements. We were able to provision the cluster and node groups correctly following the documents here and also employ a config map to maintain instance security policies.
The issue we seemed to have run across applies to the auth configuration. The config file we are using is here: https://raw.githubusercontent.com/kubeflow/manifests/v1.0-branch/kfdef/kfctl_aws_cognito.v1.0.2.yaml
Auth is configured to use OIDC using the instructions [here] (https://www.kubeflow.org/docs/aws/authentication-oidc/) with github providing user federation as we cannot use Cognito due to corporate policy. We get errors in the authzadapter seemingly related to scopes:
kubectl logs authzadaptor-89c9d49bc-dbnr9 -n istio-system
time="2020-06-01T21:04:21Z" level=error msg="Email doesn't exist in user's claim map[exp:1.591045581e+09 iss:https://vlt-kubeflow.auth0.com/ name:Mark Persiko nickname:mcpersiko picture:https://avatars3.githubusercontent.com/u/32829230?v=4 sub:github|32829230 updated_at:2020-06-01T21:04:20.541Z]. This is not supported"
time="2020-06-01T21:04:21Z" level=error msg="Email doesn't exist in user's claim map[exp:1.591045581e+09 iss:https://vlt-kubeflow.auth0.com/ name:Mark Persiko nickname:mcpersiko picture:https://avatars3.githubusercontent.com/u/32829230?v=4 sub:github|32829230 updated_at:2020-06-01T21:04:20.541Z]. This is not supported"

We tried modifying the oidc-scope's to 'openid profile'. We also had to add at least two public subnets as an annotation (Istio can't seem to find these on its own.):
C02Z72G8LVDR:oidc mark$ cat ingress.yaml
apiVersion: extensions/v1beta1 # networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: istio-ingress
  annotations:
    alb.ingress.kubernetes.io/auth-type: oidc
    alb.ingress.kubernetes.io/auth-idp-oidc: '{"Issuer":"$(oidcIssuer)","AuthorizationEndpoint":"$(oidcAuthorizationEndpoint)","TokenEndpoint":"$(oidcTokenEndpoint)","UserInfoEndpoint":"$(oidcUserInfoEndpoint)","SecretName":"$(oidcSecretName)"}'
    alb.ingress.kubernetes.io/certificate-arn: $(certArn)
    alb.ingress.kubernetes.io/listen-ports: '[{"HTTPS":443}]'
    alb.ingress.kubernetes.io/auth-scope: 'openid profile'
    alb.ingress.kubernetes.io/subnets: 'subnet-###############, subnet-###############'

You can see where we had to manually add the subnets for the ALB.
The manifests file we use is https://github.com/kubeflow/manifests/archive/v1.0-branch.tar.gz. To modify we download and unzip the manifest: tar -xvxf <manifest-file>
cd to <manifest-dir>/aws/istio-ingress/overlays/oidc
and edited the file ingress.yaml in a text editor. We package the manifest file back up with tar -cvzf <manifest-file> <manifest-dir> and edit the line in the kfctl config to use this file.
Removing auth, we can access the kubeflow dashboard and ingress is working.

cc @Jeffwan
/platform aws

/assign @Jeffwan

alb.ingress.kubernetes.io/auth-scope: 'openid profile'
See the manifest, we assume to use email here, if you don't have email. that would be a problem. @GdMacmillan
https://github.com/kubeflow/manifests/blob/316c3f16698fce5de8349daf1a59756aeaeec18a/aws/istio-ingress/overlays/oidc/ingress.yaml#L10

Thanks @Jeffwan we're taking a look at it...


[uamqp] update keep alive thread to be a daemon thread

fixing issue: https://github.com/Azure/azure-uamqp-python/issues/197

Any status on this? :blush:


1.x: Race condition at boot can cause instance to miss ENI attachments

Amazon Linux 2 instances run cloud-init at launch with a default configuration that installs pending security updates. In the case where systemd is one of the packages being updated, there is a short period of time in which systemd-udevd is not listening for kernel uevents. If an ENI is attached during that time window, the ec2-net-utils udev rule is never triggered and the interface is not configured.
The following log was taken from an instance that experienced impact from this scenario.  A potential solution is to run the ec2net-scan service with a systemd timer.
Nov  3 23:08:42 ip-172-16-58-243 yum[2437]: Updated: systemd-libs-219-78.amzn2.0.20.x86_64
Nov  3 23:08:42 ip-172-16-58-243 systemd: Closed udev Control Socket.
Nov  3 23:08:42 ip-172-16-58-243 systemd: Closed udev Kernel Socket.
Nov  3 23:08:42 ip-172-16-58-243 systemd: Stopping udev Kernel Device Manager...
Nov  3 23:08:42 ip-172-16-58-243 systemd: Stopped udev Kernel Device Manager.
Nov  3 23:08:42 ip-172-16-58-243 dbus[1778]: [system] Reloaded configuration
Nov  3 23:08:43 ip-172-16-58-243 kernel: pci 0000:00:06.0: [1d0f:ec20] type 00 class 0x020000
Nov  3 23:08:43 ip-172-16-58-243 kernel: pci 0000:00:06.0: reg 0x10: [mem 0x00000000-0x00003fff]
Nov  3 23:08:43 ip-172-16-58-243 kernel: pci 0000:00:06.0: BAR 0: assigned [mem 0x80000000-0x80003fff]
Nov  3 23:08:43 ip-172-16-58-243 kernel: ena 0000:00:06.0: enabling device (0000 -> 0002)
Nov  3 23:08:43 ip-172-16-58-243 kernel: ena 0000:00:06.0: ENA device version: 0.10
Nov  3 23:08:43 ip-172-16-58-243 kernel: ena 0000:00:06.0: ENA controller version: 0.0.1 implementation version 1
Nov  3 23:08:43 ip-172-16-58-243 kernel: ena 0000:00:06.0: LLQ is not supported Fallback to host mode policy.
Nov  3 23:08:43 ip-172-16-58-243 kernel: ena 0000:00:06.0: Elastic Network Adapter (ENA) found at mem 80000000, mac addr 06:38:13:48:56:f3
Nov  3 23:08:43 ip-172-16-58-243 dbus[1778]: [system] Reloaded configuration
Nov  3 23:08:43 ip-172-16-58-243 dbus[1778]: [system] Reloaded configuration
Nov  3 23:08:43 ip-172-16-58-243 cloud-init: Updating   : systemd-libs-219-78.amzn2.0.20.x86_64                       2/12
Nov  3 23:08:43 ip-172-16-58-243 systemd: Reexecuting.
Nov  3 23:08:43 ip-172-16-58-243 systemd: systemd 219 running in system mode. (+PAM +AUDIT +SELINUX +IMA -APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ +LZ4 -SECCOMP +BLKID +ELFUTILS +KMOD +IDN)
Nov  3 23:08:43 ip-172-16-58-243 systemd: Detected virtualization amazon.
Nov  3 23:08:43 ip-172-16-58-243 systemd: Detected architecture x86-64.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Unset automount Arbitrary Executable File Formats File System Automount Point.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Stopping Arbitrary Executable File Formats File System Automount Point.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Set up automount Arbitrary Executable File Formats File System Automount Point.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Listening on udev Control Socket.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Listening on udev Kernel Socket.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Starting udev Kernel Device Manager...
Nov  3 23:08:44 ip-172-16-58-243 systemd-udevd: starting version 219
Nov  3 23:08:44 ip-172-16-58-243 systemd-udevd: Network interface NamePolicy= disabled on kernel command line, ignoring.
Nov  3 23:08:44 ip-172-16-58-243 systemd: Started udev Kernel Device Manager.

If anyone run into this, you could run sudo bash -c 'echo 1 > /sys/bus/pci/rescan' to have the devices re-scanned.


Slash encoding

I am still getting the problem when trying to pass dates.
ui-sref="jobs-list({quote_due_from:'18/12/2014',quote_due_to:'18/12/2014'})"

The following URL is generated:
http://{mydomain}/#/jobs?quote_due_from=18%252F12%252F2014&quote_due_to=18%252F12%252F2014

This is my state provider code:
module.config(function($stateProvider) {
    $stateProvider
        .state('jobs-list', {
            url: '/jobs?quote_due_from&quote_due_to&quote_followup_from&quote_followup_to&start_date_from&start_date_to&quote_status_id',
            controller: 'JobsListCtrl',
            templateUrl: 'jobs/jobs.list.html'
        });
});

I am using the latest version of ui.router
"name": "angular-ui-router",
"version": "0.2.13",

I got around this issue by explicitly configuring the parameter types to 'any'.
module.config(function($stateProvider) {
    $stateProvider
        .state('jobs-list', {
            url: '/jobs?{quote_due_from:any}&{quote_due_to:any}&{quote_followup_from:any}&{quote_followup_to:any}&{start_date_from:any}&{start_date_to:any}&quote_status_id',
            controller: 'JobsListCtrl',
            templateUrl: 'jobs/jobs.list.html'
        });
});

Hi,
I'm still experiencing double encoding when the string I'm sending in is part of the URL, like so:
example.com/id
I'm sending in this:
<a ui-sref="search.doc({ id: 'CBVPPYMD10.1017/S003329171400227X' })" ...
and on click I'm seeing on my browser
url.com/editor/CBVPPYMD10.1017%252FS003329171400227X
I'm also seeing this same doubly encoded string as the href attribute generated by ui-router on Inspect Element.
My real doc id is CBVPPYMD10.1017/S003329171400227X
I just updated to @version v0.2.15 and I have latest angular (1.4)
Thanks.

Oh, I see that this is being addressed here?
https://github.com/angular-ui/ui-router/pull/2071
Is this supposed to be fixed?

@tweinerpq #2071 won’t change the “double encoding” of the slash in the URL, however it will fix the decoding, so the encoded slash from the URL (%252F) is correctly decoded to the slash character (/) in the state params, as soon as the PR is merged.

:+1:
The problem comes from UrlMatcherFactory.js:360. The value is encoded on line 338, and then that encoded value is passed to encodeURIComponent on line 360:
/* 338 */ var encoded = param.type.encode(value);
// …
/* 360 */ encoded = map(encoded, encodeURIComponent).join('&' + name + '=');

I have made some research on this.
Actual problem is in valToString function urlMatcherFactory.js:572
  function valToString(val) { return val != null ? val.toString().replace(/\//g, "%2F") : val; }
  function valFromString(val) { return val != null ? val.toString().replace(/%2F/g, "/") : val; }

  var $types = {}, enqueue = true, typeQueue = [], injector, defaultTypes = {
    string: {
      encode: valToString,
      decode: valFromString,

This behavior was introduced in 3045e41577a8b8b8afc6039f42adddf5f3c061ec and 0c983a08e2947f999683571477debd73038e95cf fixing #1119
Seems that it fixes path params, but breaks search params
Maybe we should encode/decode path params and search params differently

Solution here:
https://github.com/angular-ui/ui-router/issues/1119#issuecomment-134603427

Not in Chrome, @zivc
See: https://github.com/angular-ui/ui-router/issues/1119#issuecomment-134686629

Is there any reason why one would ever want to encode the slash (via valToString) and then encodeURIComponent on top of that, either in parameters or in a path component?

What I don't like is that the href is shown with an encoded slash when passing a url with a slash as a parameter to ui-sref: site.com/page%2Fhome. I feel like this is not good when search engines are crawling the website for links.

Any update on this bug? It's causing me a few problems. It's not only that the slashes are double-encoded, but it make decoding them wrong, because by manually replacing those slashes you take them out of step with the rest of encoding/decoding.
For example let's say this is URL: http://example.com/q=/--+
The proper way to encode it would be: http://example.com/q=%2f--%2b
But because of the manual slash encoding by uiRouter and then usage of encodeURIComponent the resulting URL is this: http://example.com/q=%252f--%2b
Now when you try to decode the q parameter you get %2f--+ and if you try to do it again, you get /--  (the last character is a space).
This might be a simple example, but in my case this is causing me a lot of problems with email confirmation tokens.

We are experiencing a similar problem: One of the parameter in our routes is a URL, e.g. /something/:url meaning that it needs to be properly encoded for it to work. We spent some time investigating that and we have I think a good grasp of it so I'll dump our analysis here in the hope it will be helpful in finding a solution.
Pre-requisite reading: http://blog.lunatech.com/2009/02/03/what-every-web-developer-must-know-about-url-encoding
I think the core of the problem is that we need to consider that we're inside the fragment part of the browser URL, e.g. http://example.com/angular-app/#/this-is-the-fragment-part. Values in the fragment part are encoded in a specific way where the following characters are permitted unencoded: /?:@-._~!$&'()*+,;=
On a regular (non AngularJS) link, it means you can have a valid fragment part like http://example.com/page.html#http://www.example.com/?key=value .
But within an AngularJS app you can't have that, because / is used to separate route parts http://www.example.com/angular-app/#/route/child-route/grand-child-route. If the / were left unencoded you could have http://www.example.com/angular-app/#/route/http://example.com/something.html which would result in the route parts:

route
http:
[empty]
example.com
something.html

What you really want is: http://www.example.com/angular-app/#/route/http:%2F%2Fexample.com%2Fsomething.html to get only 2 route parts route and http://example.com/something.html.
So I suspect the solution is to encode/decode the values following the rules for URL fragment parts but to make an exception just for slashes. Not respecting the fragment rules may cause problems with other special chars ``/?:@-._~!$&'()*+,;=` . I think only the slash is special in our case.
To add to the confusion, encodeURI() and encoderURIComponent() cannot be used as-is to do this encoding because they are too aggressive and not suited to encode fragment parts. It seems to be a known problem by the AngularJS developers who come up with their own functions to do so:

https://github.com/angular/angular.js/blob/e57cf13d5da204a3324e8b083718c0dcfbb58207/src/Angular.js#L1328
https://github.com/angular/angular.js/blob/e57cf13d5da204a3324e8b083718c0dcfbb58207/src/Angular.js#L1347

Hope this helps!

Same issue here. Slashes in query string parameters are getting double encoded when going through $state.go for example.
C/L becomes C%252FL where it should be C%2FL

This should be fixed in release 0.2.16+ We now encode slashes specially, using a custom escape character (tilde)
see  02e9866

I have a problem since upgrading ui-router in an app which is using ui-router's HTML5 mode. The path part of the URL is getting encoded with ~2F. Is this because the code is incorrectly encoding URLs parsed in HTML5 mode?
From the discussion above it appears to me that only the fragment part of the URL should be encoded with this change. That is clearly not happening in my case.
Since upgrading ui-router the URL after a state.go() is rewritten in the browser as this:
https://localhost:8443/~2Fdocp~2Fapp~2Fwelcome~2Findex.html/register?access_token=4c1b26d8-bd1b-41b4-b7a6-3d7c748b3235
(Note that it only encodes some of the forward slashes.)
It should be:
https://localhost:8443/docp/app/welcome/index.html/register?access_token=4c1b26d8-bd1b-41b4-b7a6-3d7c748b3235
I have tested other apps after upgrading ui-router not using HTML5 mode and the URLs are written (as expected) without the ~2F encoding.
Should I raise a new issue for this?

Seems like this just makes an error less likely to happen by doing your own encoding with ~, while introducing another issue. What happens if ~2F is in a parameter? Then it will get decoded to a /. How come you guys aren't using encodeURIComponent?

How come you guys aren't using encodeURIComponent?

@fallXone because angular $location has special handling for splitting on '/' which causes encoded parameter values with slashes like /foo/param%2Fval/bar to be forcibly decoded to /foo/param/val/bar.
See my analysis ticket: https://github.com/angular-ui/ui-router/issues/2598

url: 'products/{slugPath:.*}'
with
url: 'products/{slugPath:any}'

Angular 1.6.7 introduced a change of how forward slashes are handled.
https://github.com/angular/angular.js/commit/e06ebfdbb558544602fe9da4d7d98045a965f468
https://github.com/angular/angular.js/issues/16312
This broke our current angular app which is using url: '{aliasUrl:any}'.  Has this change broken any other apps?  Temporarily we've reverted back to 1.6.6 to resolve the issue.

Broke our app too, URLs get double-encoded (%20 becomes %2520).

Does anyone have any updates on this issue? It is breaking our site as well.

Follow ther angular repository for updates on this.
Related angularjs 1.6.7 issues and prs:

https://github.com/angular/angular.js/issues/16312
https://github.com/angular/angular.js/pull/16316
https://github.com/angular/angular.js/pull/16350
https://github.com/angular/angular.js/pull/16354

So, has this been fixed in 1.6.8?

The uirouter/angularjs test suite passes using angular 1.6.8 (it does not pass with 1.6.7)


Fixing squid: ModifiersOrderCheck  should be declared in the correct order

This pull request is focused on resolving occurrences of Sonar rule
squid:ModifiersOrderCheck - “Modifiers should be declared in the correct order”.
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:ModifiersOrderCheck
Please let me know if you have any questions.
Fevzi Ozgul

Coverage remained the same at 39.925% when pulling 997733899ed23dd02cf9e297afe1fd0be7804f71 on DevFactory:release/Modifiers_should_be_declared_in_the_correct_order into 38449cef286c4c772ec13081ae1100542cca3527 on alexvoronov:master.


Bazel-based codegen #2 (remove package_yaml2 command line arg)

Make package_yaml2 command line argument redundant for GAPIC_CODE artifact type (Java only for now). This "transitively" makes  artman_.yaml config redundant for GAPIC_CODE artifact type generation.
Add google_java_format support for formatting repo sources (verification rules will be committed later).

Do Not Merge: gapic.yaml configuration files must be updated in googleapis (to move release_level configuration from artman yaml to gapic yaml) before merging this.

Codecov Report

Merging #2236 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2236      +/-   ##
============================================
- Coverage     86.93%   86.93%   -0.01%
  Complexity     5125     5125
============================================
  Files           452      452
  Lines         20310    20303       -7
  Branches       2173     2173
============================================
- Hits          17657    17650       -7
  Misses         1884     1884
  Partials        769      769

Impacted Files
Coverage Δ
Complexity Δ

...sformer/java/JavaDiscoGapicSurfaceTransformer.java
100% <ø> (ø)
7 <0> (ø)
:arrow_down:

...degen/transformer/java/JavaSurfaceTransformer.java
98.44% <100%> (-0.02%)
60 <0> (ø)

...oogle/api/codegen/gapic/GapicGeneratorFactory.java
89.83% <100%> (-0.04%)
11 <0> (ø)

.../transformer/java/JavaGapicSurfaceTransformer.java
100% <100%> (ø)
6 <1> (ø)
: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 1f65247...77e6c65. Read the comment docs.


pectra-system-contracts-audit.md is missing links to appropriate PRs.

I'm not sure if this is the right place to mention it, but—the file open-rfps/pectra-system-contracts-audit.md says there are pull requests linked below, but they never are
In particular, it says:

Audit Requirements

The audit should focus exclusively on the smart contract bytecode, referenced in the pull requests below.

The only repositories the document references are:

fjil/geas
lightclient/sys-asm

sys-asm indeed has a 2935 pull request, but I was not able to find 7002 and 7251

The RFP links to the rendered EIPs on https://eips.ethereum.org/, which contain the bytecode, rather than the PRs.

Fixed, thanks for spotting @hacker-DOM :+1:


How to run pylyzer ?

What are the arguments to run in all .py files in my project, sounds simple, but I didn't find it in the docs

If you specify the entry point file (e.g. main.py), all dependent files will be checked. The option to check all files, including those without dependencies, will be implemented in the future.

The option to check all files, including those without dependencies, will be implemented in the future.
Thanks - will need to wait for that


Artifact links for ppc64le GPU builds.

whl files for tensorflow_gpu..._ppc6le are now hosted on the OSU
Jenkins build server. Nightly builds and Stable Release builds
are provided. I didn't include the version number so we won't need
to update the readme file for every new release

Links can be tested by looking at the README.md file here: https://github.com/wdirons/tensorflow/blob/update_readme_pp64le_gpu_artifacts/README.md

Nagging Assignee @caisq: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hi @gunan Both Nightly build and release build artifacts have been generated. Could you merge?

Thank you @gunan , we appreciate all the reviews you have been doing for the power specific changes.


Please invite me to the GitHub Community Organization

Name
TechTitan0624
Discord Username (if applicable)
No response
Additional Context
I want to join

I joined

I joined


Configure file associations not inserting value when settings is dirty and value got removed before

VSCode Version: alpha
OS Version: Linux

Steps to Reproduce:

auto save off
open JSON file
Configure File Association
Remove inserted association but don't save
redo step 3

Observe: nothing is inserted into the settings file. I know why, but from a user presepctove not expected.

Actually that doesn't even work if 3 and 4 are left out.

... when the settings.json file is dirty.

We removed the feature that 'Configure File Association' adds a settings string due to the unresolved issues with the configuration open in a dirty editor.


feat: add cwd option to instrument command

This adds support for the instrument --cwd command, and some supporting tests.  There are few changes in here so I've got some more detailed notes for the commit, and a list of assumptions.  BTW, I'm not going to widen the scope of this PR, this is it!

Notes:
Add instrument command support for the 'cwd' option
Refactor nyc-integration instrument tests so tests with output folder specified aren't in the no output folder suite
Refactor instrument.js and instrumentAllFiles command to use modern javascript
Remove call to deprecated 'process.exit()' in instrument command
Removed unused transform metadata param 'relFile'
Use file extension as an index to find the correct transform, rather than trying to match a transform name to the rightmost characters of a filename

Assumptions:
If you're instrumenting a single file cwd/a/b/d.js, and specifying an output directory cwd/e, the result will be cwd/e/d.js
That we want to use file extensions for find transforms, the existing implementation would use the '.js' instrumenter for both '.js' and '.mjs' files, these changes assume that was wrong.
I've set metadata.filename to be a relative path here, as that's what the existing tests expect
When instrumentAllFiles fails to find the correct transform, it returns the uninstrumented source,
It looks like the relFile property isn't used anymore, so I'm assuming it can be safely removed

Coverage decreased (-0.04%) to 96.601% when pulling 65d727fc7f8b5752d1a395a9a6221231f007f796 on AndrewFinlay:instrument-cwd-option into 7b5dad5231b4f5040a344377f5fad6691fb147e9 on istanbuljs:master.

This is being split up into a es6+ PR and a instrument --cwd PR, so I'm closing this one.


Hide headings for delegating tasks at normal verbosity

If we consider the following script, that has one task that does something (Default) and two delegating tasks that doesn't have any actions:
Task("Foo");

Task("Bar")
    .IsDependentOn("Foo");

Task("Default")
    .IsDependentOn("Bar")
    .Does(() =>
{
    Information("Hello Cake!");
});

RunTarget("Default");

If I run this with normal verbosity, I get the following output.
========================================
Foo
========================================

========================================
Bar
========================================

========================================
Default
========================================
Hello Cake!

Task                          Duration
--------------------------------------------------
Default                       00:00:00.0012661
--------------------------------------------------
Total:                        00:00:00.0126071

I've actually never thought about this, but why are we showing the headers of delegating tasks when we're explicitly only show delegating tasks with verbose verbosity?
Proposed change:
I think that we shouldn't show the heading (which does not convey any meaning since they're only delegating) on normal verbosity. The more I think about it, I don't see a good reason to show the headings at all, except perhaps on diagnostic verbosity.
We should however consider displaying the delegating tasks in the summary view on normal verbosity and hide the headings. The delegating tasks in the summary view would be gray like on verbose verbosity.
========================================
Default
========================================
Executing task: Default
Hello Cake!
Finished executing task: Default

Task                          Duration
--------------------------------------------------
Foo                           00:00:00.0152253
Bar                           00:00:00.0142460
Default                       00:00:00.0087549
--------------------------------------------------
Total:                        00:00:00.0382262

If no one have any objections I will add a PR for this tonight.
Ping @cake-build/cake-team

OK for me

Ok for me too.


Input interface should be more user friendly

It took me a while to understand that the interrogation mark it's actually the submit button. It really looks like an help button. Also, it's missing the cursor pointer.
First time I tried I input parent first, child second, seems more natural to a coder. I was confused with the result.
The arrow to change the parent/child relation is very confusing, giving no indication what is parent what is child.
I would suggest having a code like approach. Where child is inside (bellow/after) parent.
Also, clicking the arrows would in fact change the inputs values.
No need to keep arrows active, since we can swap as many times as we want.
I'm not a designer, but something like this.

Hi, @jaapaurelio !
Thank you for your feedback.

I agree about the button! Right to make it clearer, I really need to replace the question mark with text.

Regarding the strange behavior of the swap button of tag names:
The thing is, I am not using client-side JS at all. For this reason, everything is done using CSS and HTML (switching tabs in the mobile view, activating the swap button also without JS).
In the future, I think to use client-side JS in order to make UI more comfortable to use.


#17537 Switch to JSCH fork

@ShadelessFox Could you also link the tickets associated with JSCH fork migration? I think we have several and we possibly want to test them too.

I'd eliminate workaround used here https://github.com/dbeaver/dbeaver/pull/18590/files as we don't need it anymore.

It may solve this issue #14749

Verified


Add ESGF ini file or equivalent information

ESGF ini file for CMIP6:
https://github.com/ESGF/config/blob/master/publisher-configs/ini/esg.cmip6.ini
Much of this can be derived from the mip-cmor-tables and the fields to be added to this repo

@sashakames @wolfiex it would be great to get both the ESGF/COG ini and metagrid configs sorted ASAP - is this week pushing out luck Sasha?

esg.cmip6.ini is used by esgmapfile,  still used by legacy esg-publisher installations, but I will not support those, they need to be upgraded.
I had added CMIP6Plus to Metagrid, perhaps there is a PR somewhere...
For COG, this is a frontend change, why don't we add a link to Metagrid to get people away from the site we plan to decommission soon.

@sashakames so I found these COG related files:
https://github.com/ESGF/config/blob/master/search-configs/search_cmip6.cfg
https://github.com/ESGF/config/blob/master/publisher-configs/ini/esg.cmip6.ini
Did we need other configs for metagrid, or does it read the same format?
It make sense to me that moving forward these things would be json, if that format makes sense to the dev-side

If Metagrid does not have a test suite ready soon after we are ready, and sites are still balking on upgrading without that (though its been running fine at LLNL for a long time), sure I could help with the CoG config.  However, I think we are very close now, and then will be reasonable to require sites to upgrade for CMIP6Plus support.   Does CEDA have a timeframe to start publishing yet?
ESGF already officially deprecated the old software stack so we are under so obligation to support COG.
Metagrid doesn't have individual files, neither does the new publisher, as its based on CMIP6 as a "stock" project.

https://esgf-dev1.llnl.gov/search?project=CMIP6Plus  Unless I wipe the db, this should stay persistent for demo purposes.  We can reach out to Carlos for a date to update the main beta server and Nimbus will follow.

Having a look at the esg.cmip6.ini, the following fields would need a tweak, whereas other fields could likely be directly used from the CMIP6 template"
[project:cmip6]

activity_drs_options = AerChemMIP, C4MIP, CDRMIP,

categories =
    mip_era            | enum   | true  | true  | 0
    activity_drs       | enum   | true  | true  | 1
    institution_id     | enum   | true  | true  | 2
    ...

category_defaults = project | CMIP6

experiment_id_options = 1pctCO2, 1pctCO2-4xext, ...

experiment_title_map = map(experiment_id : experiment_title)
    1pctCO2                     | 1 percent per year increase in CO2
    1pctCO2-4xext               | extension from year 140 of 1pctCO2 with 4xCO2
    1pctCO2-bgc                 | biogeochemically-coupled version of 1 percent per year increasing CO2 experiment
    ...

institution_id_options = AER, AS-RCEC, AWI, ...

min_cmor_version = 3.2.4

min_cf_version = 1.6

min_data_specs_version = 01.00.13

mip_era_options = CMIP6

model_cohort_map = map(source_id : model_cohort)
    4AOP-v1-5                 | Published
    ACCESS-CM2                | Published
    ACCESS-ESM1-5             | Published
    ...

project_options = CMIP6

source_id_options = 4AOP-v1-5, ACCESS-CM2, ACCESS-ESM1-5, ...

sub_experiment_id_options = none, s1910, s1920, ...

AFAIK there has not been an update to esgprep/esgf-prepare the module that uses the .ini file.  So I'd say we can pass on updating that for now.

@sashakames this is required for the COG interfaces at DKRZ and CEDA, to configure the CMIP6Plus project, no?

No, the file is not used for CoG.  There is an optional input file for CoG project definitions and facets but its fairly trivial to generate.  They can copy the CMIP6 and drop any facets we plan to omit, eg. product .
But more broadly speaking, why support CoG at all when Metagrid is ready, otherwise they will waste time with a deprecated platform.  I know CEDA is close now to having a working Metagrid, DKRZ is up to Katharina who is indeed time constrained.

@sashakames great, we had queries about how close CEDA is the Metagrid deployment, if there are close, excellent. If DKRZ is a while off, I wonder how we help that process along?

Its a resource issue, of course we can try to simply things as much as possible, but sounds like she has to do 3 jobs or something...    She has been maintaining CoG and could probably make the call which suits her.  Other CoG sites are IPSL, LIU and NCI, we can check on their plans.  Next CDNOT meeting not till 14 Feb, lets see where we are at if earlier engagement is needed.


[YOLOV8] Update version

Merge on-hold pending deepsparse support
Summary

Update sparseml to use ultralytics == 8.0.124.
Also update some of the workflows which did not work currently (such as exporting segmentation models with samples)

Testing

Training, Validation, and Export workflows were tested for a dense network and while applying a recipe during training
Recipe applied during training:

version: 1.1.0

training_modifiers:
  - !EpochRangeModifier
    start_epoch: 0
    end_epoch: 15

pruning_modifiers:
  - !GMPruningModifier
    init_sparsity: 0.05
    final_sparsity: 0.10
    start_epoch: 5.0
    end_epoch: 10.0
    update_frequency: 1.0
    params: ["model.0.conv.weight", "model.1.conv.weight"]

quantization_modifiers:
  - !QuantizationModifier
    start_epoch: 11.0
    freeze_bn_stats_epoch: 12.0
    disable_quantization_observer_epoch: 13.0
    ignore: ["Upsample", "Concat", "SiLU"]

Testing Example Commands:
GPU + Recipe
Train: sparseml.ultralytics.train --recipe ~/sparseml/recipe.yaml (
Export: sparseml.ultralytics.export_onnx --model runs/detect/train/weights/last.pt --export-samples 20
Val: sparseml.ultralytics.val --model runs/detect/train/weights/[best.pt](http://best.pt/)

GPU + Dense
Train: sparseml.ultralytics.train  --epochs 50
Export: sparseml.ultralytics.export_onnx --model runs/detect/train/weights/last.pt --export-samples 20
Val: parseml.ultralytics.val --model runs/detect/train/weights/best.pt

GPU + Recipe - Segmentation
Train: sparseml.ultralytics.train --recipe ~/sparseml/recipe.yaml --task segment --model yolov8n-seg.yaml --data coco128-seg.yaml
Export: sparseml.ultralytics.export_onnx --model runs/segment/train/weights/last.pt --export-samples 20
Val: sparseml.ultralytics.val --task segment --data coco128-seg.yaml --model runs/segment/train/weights/last.pt

GPU + Dense - Segmentation
Train: sparseml.ultralytics.train --epochs 50 --task segment --model yolov8n-seg.yaml --data coco128-seg.yaml
Export: sparseml.ultralytics.export_onnx --model runs/segment/train/weights/last.pt --export-samples 20
Val: sparseml.ultralytics.val --task segment --data coco128-seg.yaml --model runs/segment/train/weights/last.pt

Trained + exported models were visually examined using netron to make sure the recipes were applied properly during training
Validation loss was compared for quantized models before and after applying the changes to make sure they were similar

Ran validation on the following model:
sparseml.ultralytics.val \
  --model zoo:cv/detection/yolov8-s/pytorch/ultralytics/coco/pruned65_quant-none \
  --data coco.yaml

The<EMAIL_ADDRESS>and mAP@0.5:0.95 match the values listed on sparsezoo and the predictions plots also make sense

merging already approved PR now that blocking issue (deepsparse support) is resolved


addToDictionary command not working

Hey! Maybe I'm missing something but it looks like the Add to dictionary command does not work.
I see this in the RPC messages when looking in the logs:
// Send:
{"jsonrpc":"2.0","id":386,"method":"workspace/executeCommand","params":{"command":"_ltex.addToDictionary","arguments":[{"uri":"file://__REDACTED__.md","words":{"en-US":["__REDACTED__"]}}]}}
// Receive:
{"jsonrpc":"2.0","id":386,"result":{"success":false,"errorMessage":"Unknown command \u0027_ltex.addToDictionary\u0027, ignoring"}}

I guess I could add my custom words in the Zed config, but this code action was given so I would've hoped it worked :innocent:
I don't know if this is an issue within Zed extension or an issue within ltex-ls itself. If you need some more info let me know :smile:

Yeah, I also encountered that. I think this an issue with ltex-plus itself since it provides code actions as commands but it does not handle them:
{"jsonrpc":"2.0","id":13,"method":"workspace/executeCommand","params":{"command":"_ltex.addToDictionary","arguments":[{"uri":"file:///Users/vslobodin/Development/ruby-lsp/README.md","words":{"en-US":["userrs"]}}]}}
// Receive:
{"jsonrpc":"2.0","id":13,"result":{"success":false,"errorMessage":"Unknown command \u0027_ltex.addToDictionary\u0027, ignoring"}}


Add the number of link for double-link

counting the number of link created by code

The index has been the number of links. Just to get it.


Exclude nio from zos

In order to avoid running into encoding issues that the generated class files of NIO load test pose (see openj9-openjdk-jdk11-zos/issues/342 for details), while on z/OS we will run a mixed load test that does not include Nio load tests.
This PR creates a new inventory file to be used by MixedLoadTest while on z/OS which essentially uses the same loads as before minus NIO tests.
Signed-off-by: Mesbah-Alam<EMAIL_ADDRESS>

Test on z/OS :  Grinder/23049 (Passes with the new load on z/OS).
TEst on Linux: Grinder/23048.

lgtm


Newer satellizer has a different flow for oauth1

This implementation and https://github.com/cosmojs/loopback-satellizer are written for v0.8.8 of satellizer, which had a different flow for oauth1. Do you happen to have an example that works with the latest satellizer (v0.11.2)?

Nevermind, I realized one exists in satellizer's example server code:
https://github.com/sahat/satellizer/blob/master/examples/server/node/server.js#L595-L684


[BENCHMARK DATASET REQUEST] schibsted-text-tasks

Dataset name
Schibsted text tasks
Dataset link
https://huggingface.co/collections/Schibsted/schibsted-text-tasks-66655bce94d0f40432519347
Dataset languages

[ ] Danish
[X] Swedish
[X] Norwegian (Bokmål or Nynorsk)
[ ] Icelandic
[ ] Faroese
[ ] German
[ ] Dutch
[ ] English

Describe the dataset
we released an evaluation dataset inspired by our real world finetuning efforts at schibsted: front title, seo title and news summarizations. hope this can help the community better evaluate llms.

Relates to https://github.com/ScandEval/ScandEval/issues/540 which was fixed in https://github.com/ScandEval/ScandEval/pull/558

The remaining datasets are two title tasks: SEO- and front titles. Both of them are considered a fairly hard task within media, so thats why we shared them.


Nested tabs don't have the same methods and properties of the parent tab

I've created a tab with ngResource so its dynamically loaded. The data has nested objects. When I assign the data as tabs in my template, the data in the parent tab automatically gains additional properties such as active, selected, etc. However the same does not happen with the data of the nested tabs. Is this expected?

Closing due to inactivity.


[minor: revert typo fix] partial hydration page

This is a minor, typo PR! Do not disturb Translation status issue!

LGTM!


Add 'enableCrushUpdates' flag to custom block pool

https://bugzilla.redhat.com/show_bug.cgi?id=2306496
Corresponding Rook + OCS changes:
https://github.com/rook/rook/pull/14322
https://github.com/red-hat-storage/ocs-operator/pull/2751

/test odf-console-e2e-aws

/cherry-pick release-4.17

/cherry-pick release-4.17-compatibility


Clean raw data view page

Take a look at http://www.martin-thoma.de/write-math/view/?raw_data_id=324096

This is related to #31

The page could eventually use tabs on the grey border of the h1-header 'View'. The default tab could be 'recording and answers' (but should be phrased shorter), the second one could be 'data' and the third one could be 'analyze'
Or an extra page 'data' which is linked below the canvas.


Use Int and compat Dict, fix 0.4 deprecations.

Yo @hayd does your editor add a blank line at the end of each file automatically?

I guess it does! Will see if I can correct that if it's an issue. :/
Does this need to be @compat Int(...) for 0.3?

Not sure, seems like it?

julia> using Compat

julia> Int(0x11)
ERROR: type cannot be constructed

julia> @compat Int(0x11)
17

fixed.

should this be tagged at some point?


Create rules in a specific order so that has:userlabels works as a filter condition

A few of my rules rely on ordering, which I understand it not guaranteed, however it seems to work.  The specific requirement is that they test for the presence (or absence) or labels from other rules.  This allows me to avoid giant nested structures.  I can do this with 'has:userlabels' or 'has:nouserlabels'
Is there any potential for supporting this as a condition?

I had no idea that was possible; neat!
Sorry to say that I have not looked into this. I think it would require some work for the RuleSet class itself, the XML generator, and the uploader. I can see how it would work for generated XML easily enough, but for the uploader you have to take into account any filters that already exist on the server (or always nuke-and-replace, which is slower).
I'd welcome pull requests that implement this, but it will require enough changes to the internals and the command line that we'll definitely need more tests than we have today.

The only catch with sending the rules in order is there needs to be a ~1.5 sec cooldown in between rules sent or it's possible that the API will process and add rule 3 before it is finished processing 2 giving you a order of 1324


"Quickfix" not showing inside collection expressions

When I am pressing the alt + enter ("quickfix" shortcut on Intellij) it shows no options if the widget is inside conditions or for loop inside collections, like this:

I am on Windows 10 with flutter 1.9-hotfix5. lso my dart and flutter plugins for Intellij are the latest version.

I think someone else reported a very similar bug, or maybe it was re: completion.
@scheglov may have fixed this last week...

https://github.com/dart-lang/sdk/issues/38947


XMLHTTPrequest failed to update

Have you read Troubleshooting.md? If No, please do so before filing an issue.
Yes
Have you tried Googling the problem?
Yes
Which python version are you using to run the project? In the terminal, type which <python-path-you-have-in-clientServer.php> and enter the output here
Python version -3.7
Describe the bug
A clear and concise description of what the bug is.
I have followed all the steps as mentioned, using Windows laptop. The extension shows errors such as
Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
Error handling response: Error: Failed to execute 'send' on 'XMLHttpRequest': Failed to load 'file:///D:/codes/Malicious-Web-Content-Detection-Using-Machine-Learning-master/clientServer.php'. when i tried putting the file location directly
and
Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.	in line- xhr.open("POST","http://localhost/Malicious-Web-Content-Detection-Using-Machine-Learning-master/clientServer.php",false);
I am not sure where i am missing what and am confused as the extension doesnt show any output if website is phished or not.
To Reproduce
Steps to reproduce the behavior:

Go to 'extensions and load unpacked extension'
Click on 'new tab and pressed website"google.com and click the extension, safe or not button pops up but no output, go to extensions tab and click errors on the extension pack'
Scroll down to 'see the warnings and errors with it'
See error

Expected behavior
I expected the code to run the extension and extract and see if the url is phished or not, even from the local browser or the real web browser
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 OS
Browser chrome
Version (optional) i dont know

Smartphone (please complete the following information): N/A

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version (optional) [e.g. 22]

Additional context
please help to solve the issue as i also want to learn how phishing works and check on my computer.. as a learning experience i need to understand how it is done. i am new to the machine learning and cyber security.

hello , have you somehow fixed this bug ?

Hey, did you fix the bug? Even I am facing the same issue

Hey BRO.. Is the Error fixed. Please let me know


stopping VM instances from the host side

Consider an infinite loop.
Maybe it would stem from a malicious "joke" mod the user downloaded, maybe from the user himself when trying to create some macro script or whatever.
In that case you'd probably be able to shut down your application/game gracefully, which by extension would mean stopping the blocking VM instance. By gracefully, I mean really anything from not blocking execution at all to stopping a level, showing an error message and going back to the main menu to closing the application without crashing the window manager in the process.
Apparently, some ways to implement this (but probably not limited to) are:

Not at all. CLua for example doesn't seem to have something like this officially. Inofficially you can somehow abuse some debugging hooks and some error flag, with a couple of drawbacks.
Having an official stop flag/method. AngelScript for example lets you call an abort method from a watchdog thread.
Having a loop limit. This seems like a tacked-on solution in most cases, but I believe UnrealScript did this.
Outsourcing the interpreter loop and letting the host application execute a couple of instructions each update cycle incrementally. Jynx and Papyrus VM (from Skyrim/Fallout 4) do this. I'd guess this allows for the easiest implementation of latent functions.

Some of those possibilities have probably drawbacks like reduced instruction throughput and some probably can't  be realized from the main thread, which probably rules out stopping the VM manually by GUI.
That being said, I didn't find anything like it in the documentation. What is Wrens take on this?

AFAIK, wren VM is single threaded and don't support instrumentation from an external thread.
And there is no simple solution to this halting/suspend problem, since most of the solutions can lead to memory leaks.
Maybe the more interesting solution is a stop flag/count. That way, we should be able to perform debugging and stepping, that are also crucial for creating debugging tools.

Keep in mind that we've already implemented debugging with stepping and will be upstreaming it soon. You wouldn't typically want debugging support enabled for something like user facing mods though because it will cost performance, but also because it will also be able to introspect state, view module sources (if string based) and such.
You could probably put an external mutex around vm->fiber->error and set it that way which will stop the vm loop from what I see.

Nice I wait to see stepping in action, and try to do a debugger yummy.
It is not really possible to do so, because if the user decide it is ok to infinite loop and we might need to be able to resume. So I really like to see how is your implementation of stepping, and its impact on speed.

I put a link to the video in the above comment but since thread is a different topic best to discuss the debugger in it's own thread when we post soon.

Hi,  I want a Hook interface like lua script  "lua_sethook"

...
lua_sethook(luaVm, Lua_YeildHook, LUA_MASKLINE, 0);
....

void Lua_YeildHook(lua_State* _L, lua_Debug* ar)
{
	if (bStop) {
		lua_yield(_L, 0);
	}
}


Added links to versions preview rows that re route to clicked version

This PR is in response to this comment.

Clicking a row on the versions preview re directs to that version
Added url parameter for the specific version

Looks good buddy


Update animals

Updated animals table
Added transactions
Added new queries

1st Transaction

2nd Transaction

3rd Transaction

1st query

2nd query

3rd query

4th query

5th query

6th query

Changed the datatype of species from integer to string(varchar(20)).

Adjusted data.sql

I have corrected my mistakes on my pgAdmin file (as shown on the screenshot) but forget to update the schema.sql and data.sql. Sorry for the inconvenience.


Can README specify supported/tested browsers, Ember versions, etc.?

I'm glad to see this project reawaken, but I don't know if it will work for us. Are IE11 and old Edge supported by the reawakened addon? What about Ember versions? How far back? Maybe this could go up near the installation section, where people are deciding whether to install it or not.

If it helps I'm using Ember 2.8.3 and it didn't work for me because the layout helpers are using the new modular imports:
import { helper } from '@ember/component/helper';
import PercentageColumns from 'ember-collection/layouts/percentage-columns';

export default helper(function (params) {
  return new PercentageColumns(params[0], params[1], params[2]);
});

I had to create my own helper with the old format:
import PercentageColumns from 'ember-collection/layouts/percentage-columns';

export default Ember.Helper.helper(function(params) {
  return new PercentageColumns(params[0], params[1], params[2]);
});

I'm also using the alpha 8 version. Hope that helps.


Fix editor compile error when the VisualScript module is disabled

Caused by #57720.

Good catch. I would prefer to revert those back to a string name check, that's less convoluted than this.

Thanks!


.json schema doesn't allow for semantic versioning prerelease naming conventions

Your .json schema doesn't allow for semantic versioning prerelease naming conventions (i.e. 1.0.0-alpha) [http://semver.org].

This issue is valid as of 2015/02/16. I'm cleaning my inbox and translating all my notes into issues.


Escape invalid characters in tag name/value and field name/value

According to https://influxdb.com/docs/v0.9/write_protocols/write_syntax.html, any space, comma, or equals sign needs to be properly backslashed in tag names/values and field names/values. This allows you to write data like:
cpu-utilization,cpu_name=CPU\ 1 value=0i
cpu-utilization,cpu_name=CPU\ 2 value=0i
cpu-utilization,cpu_name=CPU\ 3 value=0i

Thanks!


优化文档，突出特色功能 #72

#72

Codecov Report

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

@@           Coverage Diff           @@
##           master      #73   +/-   ##
=======================================
  Coverage   66.14%   66.14%
=======================================
  Files          20       20
  Lines         765      765
=======================================
  Hits          506      506
  Misses        221      221
  Partials       38       38

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 36ba802...abf005e. Read the comment docs.


tour: finded typo

Context: https://go-tour-ru-ru.appspot.com/moretypes/15
Finded typo in ru version tour

This is fixed in the last version of the ru tour, AFAICT https://go-tour-ru-ru.appspot.com/moretypes/15, so I'm closing this issue.


Props works wrong when component is written in index.tsx

Bug Report
Describe the bug
Props works wrong when component is written in index.tsx.
To Reproduce

Follow the steps in https://github.com/doczjs/docz/tree/master/examples/typescript#download-manually, use docz typescript example.

npm run dev. Everything is normal.

Rename src\components\Button.tsx to src\components\index.tsx

Edit src\components\Button.mdx:

import { Button } from './index.tsx';

<Props of={Button} />

npm run dev. Props works wrong.

Undo rename and edit.

npm run dev. Everything is normal again.

Expected behavior
Props works fine when component is written in index.tsx.
Environment

docz version: 2.0.0-rc.68
OS: Windows 10
Node/npm version: Node 12/npm 6

Hey @Tarocch1
Thanks for reporting this issue ! It will hopefully be fixed in the next docz release candidate.
Meanwhile, you can fix it temporarily from your end.
In your doczrc.js file add a filterComponents methods :
// doczrc.js
export default {
  typescript: true,
  filterComponents: files =>
    files.filter(
      filepath =>
        /\/[A-Z]\w*(\/index)?\.(js|jsx|ts|tsx)$/.test(filepath) ||
        filepath.includes("index")
    )
};

and it should work as expected.
I'll update this issue once the issue is resolved from docz's side 👍

Fixed in 2.0.0-rc.69. Please feel free to re-open if it still happens.


Projects cannot be deleted

I was able to see this for the templates space but not jg-tests.

OK, it turned out to be a route conflict. There was a route that used 'templates' and so the 'templates' space hit the wrong route when trying to delete a project. Fixed in the dev branch.

Merged into main. Closing.


XAPK Error

Using WindyApp.xapk

java.lang.RuntimeException: com.abdurazaaqmohammed.AntiSplit.main.MismatchedSplitsException: Error: Key (the app will not run without it) split (config.x86.apk) has a mismatched version code.
com.reandroid.apkeditor.merge.Merger.extractAndLoad(SourceFile:394)
com.reandroid.apkeditor.merge.Merger.run(SourceFile:56)
com.abdurazaaqmohammed.AntiSplit.main.MainActivity$ProcessTask.doInBackground(SourceFile:41)
com.abdurazaaqmohammed.AntiSplit.main.MainActivity$ProcessTask.doInBackground(SourceFile:1)
android.os.AsyncTask$3.call(AsyncTask.java:394)
java.util.concurrent.FutureTask.run(FutureTask.java:264)
android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:305)
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:644)
java.lang.Thread.run(Thread.java:1012)
Output directory: /storage/emulated/0/Download/aaa/pair/Windy.app_50.1.0_apkcombo.com_antisplit.apk
Searching apk files…
/document/primary:Download/aaa/pair/Windy.app_50.1.0_apkcombo.com.xapk
Extracted co.windyapp.android.apk
Extracted config.x86.apk
Extracted config.ja.apk
Extracted config.xxhdpi.apk
Extracted config.it.apk
Extracted config.x86_64.apk
Extracted config.th.apk
Extracted config.nl.apk
Extracted config.mdpi.apk
Extracted config.de.apk
Extracted config.hdpi.apk
Extracted config.en.apk
Extracted config.zh.apk
Extracted config.ldpi.apk
Extracted config.ru.apk
Extracted config.tvdpi.apk
Extracted config.pt.apk
Extracted config.pl.apk
Extracted config.es.apk
Extracted config.ms.apk
Extracted config.fi.apk
Extracted config.ar.apk
Extracted config.hi.apk
Extracted config.xxxhdpi.apk
Extracted config.vi.apk
Extracted config.in.apk
Extracted config.fr.apk
Extracted config.arm64_v8a.apk
Extracted config.xhdpi.apk
Extracted config.armeabi_v7a.apk
Skipping manifest.json: Not an APK file
Skipping APKComboInstaller.url: Not an APK file
Skipping icon.png: Not an APK file
Found apk files: 30

As it says, x86 and some other splits have mismatched version code, I notice this happens with some apps on APKCombo where some of the splits are outdated sometimes. I don't know how they make this error, it is a big problem because the splits won't install (Even in APKCombo installer) and if you force merge it, it installs but any split with mismatched version code won't work.
I recommend downloading from APKMirror when available or APKPure instead of APKCombo, this app is available on APKMirror - https://www.apkmirror.com/apk/windy-weather-world-inc/windy-wind-weather-forecast/windy-wind-weather-forecast-50-1-0-release
By the way, I know the app can run without x86 (or whatever split is mismatched) library on any devices that don't require that split, but I show the error for all architectures because if merging universal APK all architecture should be working. (If you enable "Select splits for your device specifications", it should not have the error, atleast not for x86)


Update callshader-function.md

Invalid link syntax

Hi, it looks like the CLA hasn't been signed yet (please see comment in the thread above). If you could sign that, then the button will appear on this end for me to complete the PR. Thanks! :)

@microsoft-github-policy-service agree [company="default"]

@microsoft-github-policy-service agree