Need a way to get Intermediate Layer Inputs/Activations for tf.keras Models

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using):
2.0
Are you willing to contribute it (Yes/No):
Yes, if there is a consensus on how it should be designed.

Describe the feature and the current behavior/state.
In eager mode, there is no way to access a tf.keras model's layer inputs/outputs during training (as far as I can tell, please correct me if I'm wrong). In TF 1.x (graph mode), this was not a problem, since you could use layer.input or layer.output layer.inbound_nodes or layer.outbound_nodes to get these tensors and use those values, but this is no longer possible in eager mode.
PyTorch solves this issue by allowing users to register hooks on layers, which is essentially a function that is called before/after the forward/backward pass on a layer. Here is the code for register_forward_hook in PyTorch.
Alternatively, the input/output properties of a layer could store a reference to the tensors used in the most recent forward pass.
Will this change the current api? How?
Yes, depending on how this is implemented. If a hooks approach is used, this public method would have to be added to tf.keras.layers.Layer. If the input/output property approach is used, these properties would have new behavior in eager mode.
Who will benefit with this feature?
Being able to access, record, manipulate, or otherwise use layer inputs and outputs for models during training/inference is generally very useful.
A specific example is the K-FAC optimization algorithm, which uses each layer's inputs and pre-activation gradients to approximate the Fisher information matrix. The current implementation does not support eager. PyTorch implementations (e.g. this one) of this algorithm use hooks to do this.
Another use case is visualizing intermediate activations of CNNs. This example uses layer.outputs in TF 1.x + Keras to grab the right tensors then creating an augmented model. This process would be greatly simplified by allowing access to intermediate activations without augmenting the model.
Any Other info.
Here is a related issue about getting intermediate activations.
EDIT (2019-10-20): I learned that layer.inbound_nodes and layer.outbound_nodes used to have this behavior, not layer.input and layer.output. layer.input and layer.output track the tensors that are created when model.build(input_shape) is called. When you use the model as a callable on your own input (i.e. predictions = model(inputs)), new tensors are created (reusing the model's weights/architecture). In TF 1.x, inputs would be added to the inbound_nodes list and predictions would be added to the outbound_nodes list during this call. Now, since in eager mode the model is called on a new EagerTensor for every training step, it is not reasonable to add that many new tensors to these lists, so the property was deprecated. Since this feature used to exist, I think it's important for a reasonable replacement to exist in TF2 (such as hooks or a layer property tracking the most recent inputs/outputs).

@n2cholas If you are looking for tf.keras Models, model.get_layer(layer_name).outputs or model.layer[layer_index].output will provide the details you are looking. This was already answered in the link you provided.
Please let us know what you think. Thanks!

Thanks @jvishnuvardhan for your response. I don't think model.layer[layer_index].output gives me the outputs I need.  Here's a minimal example of trying to compute the output gradients from a Dense layer using this property (runs in colab):
%tensorflow_version 2.x
import tensorflow as tf

(x_train, y_train), _ = tf.keras.datasets.mnist.load_data()
train_ds = tf.data.Dataset.from_tensor_slices((x_train, y_train)).batch(32)

model = tf.keras.models.Sequential([
  tf.keras.layers.Flatten(input_shape=(28, 28)),
  tf.keras.layers.Dense(128),
  tf.keras.layers.ReLU(),
  tf.keras.layers.Dense(10)
])

loss_object = tf.keras.losses.SparseCategoricalCrossentropy(from_logits=True)

images, labels = next(iter(train_ds))

with tf.GradientTape() as tape:
  predictions = model(images)
  loss = loss_object(labels, predictions)

print(tape.gradient(loss, model.layers[1].output))

This prints None instead of the gradient. Manually tracking the outputs at each layer then computing the gradient gives me the output gradient of the Dense layer, which is what I need:
outputs = [images]
with tf.GradientTape() as tape:
  for layer in model.layers:
    outputs.append(layer(outputs[-1]))
  loss = loss_object(labels, outputs[-1])

print(tape.gradient(loss, outputs[2]))

In this sequential case, manually doing the forward pass (as I did with the loop) is trivial, but it can become difficult with other architectures, such as ResNets. With hooks, this would not be necessary.
Am I doing something wrong? Is there a better way for me to track the intermediate activations?

If you change the model code to not include an input_shape, the model is never built and the input and output properties don't exist, even after the forward pass. If you change the model code from my previous example to the following:
model = tf.keras.models.Sequential([
  tf.keras.layers.Flatten(),
  tf.keras.layers.Dense(128),
  tf.keras.layers.ReLU(),
  tf.keras.layers.Dense(10)
])

The first bit of code in my previous reply will result in AttributeError: Layer dense has no inbound nodes.
The input and output properties simply track the tensors created when the model built, not the tensors created during the predictions = model(images) call. The tensors I'm interested in are the ones created during that call.

@jvishnuvardhan @tanzhenyu Any update on this? I would be happy to implement hooks and submit a PR if hooks seem like the most suitable way to move forward.

@n2cholas I will happily test your PR :D

Same problem here, great problem description @n2cholas!

My current application also needs to access the intermediate activations and backward gradients. Since a solution to this has not been implemented yet, does it make sense to just save these tensors as attributes of my class which is inherited from keras.models.Model?

Sorry about the delay, I won't have the bandwidth to work on my implementation until the end of December. I'd be happy to test someone else's implementation if they do it sooner!

My current application also needs to access the intermediate activations and backward gradients. Since a solution to this has not been implemented yet, does it make sense to just save these tensors as attributes of my class which is inherited from tf.keras.models.Model?

Yes, this is what I've been doing. I subclassed all the layer types I needed, then saved the input/output tensors as attributes during my forward pass.

Should I create an RFC for this, since it's an API addition? Or just a PR? Also, how should this interact with autograph and model saving? The base_layer.py code is pretty intimidating.  I'd love some guidance from a TensorFlower, if possible.
Here's a temporary layer wrapper I've been using to get this functionality. This is pretty similar to what I'd do in the tf.keras.layers.Layer code (in the __call__ function):
%tensorflow_version 2.x
import tensorflow as tf
from typing import List, Callable, Optional

class LayerWithHooks(tf.keras.layers.Layer):
  def __init__(
      self,
      layer: tf.keras.layers.Layer,
      hooks: List[Callable[[tf.Tensor, tf.Tensor], Optional[tf.Tensor]]] = None):
    super().__init__()
    self._layer = layer
    self._hooks = hooks or []

  def call(self, input: tf.Tensor) -> tf.Tensor:
    output = self._layer(input)
    for hook in self._hooks:
      hook_result = hook(input, output)
      if hook_result is not None:
        output = hook_result
    return output

  def register_hook(
      self,
      hook: Callable[[tf.Tensor, tf.Tensor], Optional[tf.Tensor]]) -> None:
    self._hooks.append(hook)

model = tf.keras.Sequential([
  LayerWithHooks(tf.keras.layers.Dense(50, input_shape=(1,))),
  LayerWithHooks(tf.keras.layers.Dense(1)),
])

If you have an existing model, the code below seems to work, though I'm always hesitant to use monkey patching:
model = tf.keras.Sequential([
  tf.keras.layers.Dense(50, input_shape=(1,)),
  tf.keras.layers.Dense(1),
])

def get_call_fn(layer: tf.keras.layers.Layer) -> Callable[[tf.Tensor], tf.Tensor]:
  old_call_fn = layer.call
  def call(input: tf.Tensor) -> tf.Tensor:
    output = old_call_fn(input)
    for hook in layer._hooks:
        hook_result = hook(input, output)
        if hook_result is not None:
          output = hook_result
    return output
  return call

for layer in model.layers:
  layer._hooks = []
  layer.call = get_call_fn(layer)
  layer.register_hook = lambda hook: layer._hooks.append(hook)

Here's an example of saving inputs/outputs with this:
class InputOutputSaver:
  def __call__(self, input: tf.Tensor, output: tf.Tensor) -> None:
    self.input = input
    self.output = output

savers = {}
for layer in model.layers:
  saver = InputOutputSaver()
  layer.register_hook(saver)
  savers[layer] = saver

inps = tf.convert_to_tensor([[1.], [2.], [3.],])
print(model(inps))
print(savers[model.layers[0]].output)

This temporary solution workaround was not thoroughly tested, please let me know if you have any suggestions.

Did you know that model.outputs is a mutable Pyhon list? You can just change it to return all outputs.
# assuming model is a keras Sequence and x is a valid input
output_names = [l.name for l in model.layers]
model.outputs = [l.output for l in model.layers]
model.build(input_shape=x.shape)
output_values = model(x)
layer_name_to_output_value = dict(zip(output_names, output_values))

Has some improvement been done to solve this issue? Or do we still need to save manually intermediate output as an attribute of the model? It is very impractical to have this problem when using model subclassing in eager mode

hey guys
If i want to get the output of a intermediate layer in my NN, What should i do? which function can give me the output of the specific layer? and can i define a new loss function at that layer? for example for parallel learning : parent classification and subclass classification.

Sorry for the delay!
I'm not sure if layer.input and layer.output cannot achieve what you're looking for.
First of all, layer.inbound_nodes and layer.outbound_nodes are deprecated in 2.0.
Second of all, layer.input and layer.output are still symbolic tensors, just as you described, they're created under the first layer.build(input_shape)
So to make your first use case work, i.e., the activation visualization work, it should be exactly the same code as before, i.e.,:
original_model = tf.keras.Model(inputs, outputs)
activation_outputs = [layer.output for layer in original_model.layers]
activation_model = tf.keras.Model(inputs, activation_outputs)

To make your second use case work, i.e., getting the right gradient, remember input and output are symbolic tensors, what you really need is the gradient of an EagerTensor imperatively, so:
seq_1 = tf.keras.models.Sequential([
  tf.keras.layers.Flatten(input_shape=(28, 28)),
  tf.keras.layers.Dense(128),
])
seq_2 = tf.keras.models.Sequential([
  tf.keras.layers.ReLU(),
  tf.keras.layers.Dense(10)
])
loss_object = tf.keras.losses.SparseCategoricalCrossentropy(from_logits=True)
images, labels = next(iter(train_ds))
with tf.GradientTape() as tape:
  out = seq_1(images)
  predictions = seq_2(out)
  loss = loss_object(labels, predictions)

print(tape.gradient(loss, out))

Or you can create a subclassed model that returns the final output as well as the intermediate output.
If this is too much of a hassle, maybe we could consider layer.eager_input and layer.symbolic_input.

BTW, you can still call a layer repeatedly and add inputs to the nodes.
input_1 = tf.keras.Input((5,), name='input_1')
dense = tf.keras.layers.Dense(units=3)
output_1 = dense(input_1)
print(dense.inbound_nodes[0].input_tensors)
input_2 = tf.keras.Input((5,), name='input_2')
output_2 = dense(input_2)
print(dense.inbound_nodes[1].input_tensors)

Also check this as suggestion:
https://github.com/tensorflow/tensorflow/issues/33129#issuecomment-539876444

Closing it for now. Feel free to re-open if you still have concerns :-)

How do we the same thing when running non-eager mode, i.e tf.function?

BTW, you can still call a layer repeatedly and add inputs to the nodes.
input_1 = tf.keras.Input((5,), name='input_1')
dense = tf.keras.layers.Dense(units=3)
output_1 = dense(input_1)
print(dense.inbound_nodes[0].input_tensors)  #<tf.Tensor 'input_1:0' shape=(None, 5) dtype=float32>
input_2 = tf.keras.Input((5,), name='input_2')
output_2 = dense(input_2)
print(dense.inbound_nodes[1].input_tensors)  #<tf.Tensor 'input_2:0' shape=(None, 5) dtype=float32>

@tanzhenyu
I want to get intermediate tensor,not layer's output.for example:
model = train_model()
func = tf.function(model)
tensor_specs1 = tf.TensorSpec.from_tensor(model.input)
call = func.get_concrete_function(tensor_specs1)
graph = call.graph
outputs = graph.get_tensor_by_name('model_1/word_encoder_time/word_attention/Softmax:0')
pred_model = tf.keras.models.Model(model.input,outputs)
results = pred_model(tensor_specs1)

but raise an exception:
raise ValueError("Tensor %s is not an element of this graph." % obj)
ValueError: Tensor Tensor("model_1/word_encoder_time/model/word_attention/BiasAdd:0", shape=(?, 10), dtype=float32) is not an element of this graph

outputs = [model.get_layer(name=output).get_output_at(0) for output in output_layers]
pred_model = tf.keras.models.Model(model.input,outputs)

It's working ,but it's not what I want.

@tanzhenyu any updates ?

Any updates?

Using monkey patch to break in and hook, that is not a elegant way, but it works
def proxy_call(input:tf.Tensor, obj:tf.keras.layers.Layer)->tf.Tensor:
    if obj._before_call is not None:
        obj._before_call(obj,input)
    output = obj._old_call(input)
    if obj._after_call is not None:
        hook_result = obj._after_call(obj, input, output)
        if hook_result is not None:
            output = hook_result
    return output

def hook_layer_call(layers: List[tf.keras.layers.Layer],
                    before_call: Callable[[tf.keras.layers.Layer, tf.Tensor],None]=None,
                    after_call: Callable[[tf.keras.layers.Layer, tf.Tensor, tf.Tensor],Optional[tf.Tensor]]=None):
    for layer in layers:
        layer._before_call = before_call
        layer._after_call = after_call
        layer._old_call = layer.call
        layer.call = functools.partial(proxy_call, obj=layer)

def print_input_output(layer:tf.keras.layers.Layer, input:tf.Tensor, output:tf.Tensor):
    print(input, output)

# suppose you have a model(such as a tf.keras.Sequential instance)
hook_layer_call(model.layers, after_call=print_input_output)

I'm a bit late, but I created a utility tensorflow-hooks (installable via PyPI) that provides forward pre-hooks and forward hooks to TensorFlow Keras layers: https://github.com/rachthree/tensorflow-hooks. I hope it'll be useful for others. Thank you @k-lam for the inspiration!


add docs for ignore_upstream_failure

Add documentation for the newly implemented ignore_upstream_failure method for pipeline tasks. If this feature is called the pipeline task will run when any specified upstream tasks complete, even if unsuccessful. This method effectively turns the caller task into an exit task if the caller task has upstream dependencies.
Sample:
@dsl.pipeline()
def my_pipeline(text: str = 'message'):
    task = fail_op(message=text)
    clean_up_task = print_op(
        message=task.output).ignore_upstream_failure()

/assign @sen-sam


discuss initial version

[x] run lint again  (done)

range plot float16ext

Looks stable, close for now.


has the granting of your recent patent with Adobe effectively killed open source image matting?

https://patents.google.com/patent/US11004208B2/en
patent seems to cover the idea of user interaction to create binary matte -> trimap -> final matte and every variation of that.

interactive matting hmmm,  nany webservice are using interactive matting such  https://clippingmagic.com/

Hi thank you for your interest in my work.
I am not a lawyer, and no longer an employee of Adobe so unfortunately I cannot give you a legal answer.
Please contact representatives at Adobe for a more satisfying definitive answer.

sorry - didn't mean to come off as so accusatory. Amazing work so far. Really looking forward to what you come up with next.


Debugger Banner does not work with Live Extension enabled

Upstream issue https://github.com/MicrosoftDocs/live-share/issues/575

Wrong repo


[BUG]  "./MysqlDatasource" has no exported member named 'MysqlDatasource'

Information

Version: v4.6.4
Packages:

Steps to reproduce

Create new project using this following options

tsed init .
 Choose the target platform: Express.js
? Choose the architecture for your project: Ts.ED
? Choose the convention file styling: Ts.ED
? Check the features needed for your project Database, Swagger, Testing, Linter
? Choose a ORM manager TypeORM
? Which TypeORM you want to install? MySQL
? Choose unit framework Jest
? Choose linter tools framework EsLint
? Choose extra linter tools Prettier, Lint on commit
? Choose the package manager: Yarn Berry
↓ Write RC files [SKIPPED]
✔ Initialize package.json
✔ Install plugins
↓ Write RC files [SKIPPED]
✔ Initialize package.json
✔ Install plugins
✔ Load plugins
✔ Install plugins dependencies
✔ Generate project files
✔ Generate files for eslint
✔ Add dependencies
✔ Generate files for jest
✔ Generate typeorm:datasource spec file to 'datasources/MysqlDatasource.spec.ts'
✔ Generate TypeORM datasource file to 'datasources/MysqlDatasource.ts'
✔ Generate docker-compose configuration
✔ Install dependencies
✔ Add husky prepare task
✔ Generate barrels files

run build

yarn build

You should see

src/datasources/MysqlDatasource.spec.ts:2:10 - error TS2724: '"./MysqlDatasource"' has no exported member named 'MysqlDatasource'. Did you mean 'MysqlDataSource'?

Possible Solution
Replace export const MysqlDataSource  with export const MysqlDatasource in here  ? (not sure if this is the correct file)

:tada: This issue has been resolved in version 4.6.6 :tada:
The release is available on:

v4.6.6
GitHub release

Your semantic-release bot :package::rocket:


add uma abis

Signed-off-by: John Shutt<EMAIL_ADDRESS>Issue #2814 [partial]
Changes in this PR:
Adds UMA ABIs to constants file, as a first step to integrating UMA execution in the current user interface. Eventually UMA execution will be included in the big refactor, but separating development of this feature into its own track will allow us to launch before the refactor is done.
How to test and review this PR?
Compare the new ABIs with the actual ABIs for UMA's optimistic governor, optimistic oracle, and voting contracts. (Note: The ABIs are the same as those in the big refactor branch, where UMA execution appears to be working.)

Closing in favor of https://github.com/snapshot-labs/snapshot/pull/3337


Add Button to UI to Do Something

Based on an alarm, a user may want to take action (what good is an alarm if you cannot resolve the issue?).  This request is to allow a user to add a button.  The action the button takes is up to the user.  It can be a script that fires for example to switch a device from a primary input to a secondary input.

Initial thoughts on this one.
Flow:

Something needs to be selected on the diagram surface (Channel, Bucket, etc.)
Depending on the type of node selected, one or more action buttons are shown (somewhere) that are associated with that node type
The button sends a POST to an endpoint
Response from the endpoint is shown as-is in a pop-up dialog

We need to be able to reference data from the node to build the URL path, parameters, or POST body. Consider using JSONPath notation to make these references.
Selected node data:

Example URL with referenced ARN from the selected node:
https://actionendpoint.something.else/{$.Arn}
Example POST body with referenced node data:
{
"Command": "SwitchTo",
"Input": "{$.InputAttachments[1]. InputId}"
{
The {$.Arn} reference above is replaced with the arn of the resource.
The {$.InputAttachments[1]. InputId} reference is replaced with the input ID of the second input in the list.
Security mechanism of the endpoint called by MSAM is outside of this scope.
Button/action/substitution specifications are kept in the Settings table in the cloud
Does the invocation of the endpoint happen at the browser, or is it queued into the AWS account and initiated cloud-side from the AWS network?


MS Access 97 pyodbc Pandas dataframe to_sql throws Invalid Precision Value

Environment
Python 3.8.5 32-bit on a Windows 10 Workstation.
MS Access 97
datacompy==0.7.1
et-xmlfile==1.0.1
jdcal==1.4.1
numpy==1.19.2
openpyxl==3.0.5
pandas==1.1.2
pyodbc==4.0.30
python-dateutil==2.8.1
pytz==2020.1
pywin32==228
six==1.15.0
SQLAlchemy==1.3.19
sqlalchemy-access==1.0.7
Issue
I'm using the pandas dataframe's to_sql function to save records to MS Access 97 MDB. But I'm getting the error below:
pyodbc.Error: ('HY104', '[HY104] [Microsoft][ODBC Microsoft Access Driver]Invalid
precision value  (98) (SQLBindParameter)').

Answered on Stack Overflow here.

@gordthompson thanks for great answer! However, we can't just upgrade the MS Access from 97 to newer one because of a legacy software. The above suggested code didn't work for me. However, overriding the encoding/decoding to UTF-8 worked.
import pyodbc conn = branch_db_conn.raw_connection()
conn.setdecoding(pyodbc.SQL_WCHAR, encoding='utf-8')
conn.setencoding(encoding='utf-8')
cursor = conn.cursor()

@mkleehammer - IMO this issue can be closed.

@mkleehammer - IMO this issue can be closed.


Add Clover Finance SS58 Address

This is a trivial PR for reserving 128 SS58 address for Clover Finance.

@pangwa closing this for now.
As stated in the email, you are free to reach out and start the process.

@pangwa closing this for now.
As stated in the email, you are free to reach out and start the process.

Hi @BenWhiteJam , I haven't received any email on this, which email did you send the email to? Thanks!


Adopt VS Code's 'asWebviewUri' API

Hi, I maintain VS Code's Webview API
Issue
Our telemetry suggests that your extension uses webviews and may be loading resources in these webviews using hardcoded vscode-resource: URIs. These URIs have some important limitations and don't work properly when VS Code is run in a browser. We are also making changes in the desktop version of VS Code that may cause these URIs to not work properly in future versions of VS Code.
While we are making our best effort to continue support existing webview extensions that use vscode-resource: URIs on desktop versions of VS Code, we will not able to fully support all uses cases.
Fix
To ensure that your extension continues to work in as many environments as possible—including on web and remotely—please:

Switch to use the Webview.asWebviewUri function for loading resources.

Switch to use the Webview.cspSource property in content security policies.

These APIs shipped around 2 years ago so they should be available in all modern versions of VS Code. You can find additional information about the issue here: https://github.com/microsoft/vscode/issues/97962
Let me know if you have any questions about this change

@mjbvz is this just a general ask or did you have a specific reason to ask us to do this? We should already be using asWebviewUri everywhere.
Perhaps there's somewhere not so obvious when looking at the code but you have some logging to indicate otherwise?

The only place vscode-resource is mentioned in the code should be in our cspSource declaration.
See here:
https://github.com/microsoft/vscode-python/blob/5d13fa2bd948fd0949d5a12c571045bf698f7253/src/client/common/application/webPanels/webPanel.ts#L166

Our telemetry showed the python extension was webview-resource somewhere. You'll need to update the csp as describe to use Webview.cspSource

@rchiodo I still see a vscode-resource: in two csps. But it looks like it's in webPanel.ts and webPanelServer.ts and in webPanel.ts it's only in the path for generateServerReactHTML. That path was just for the web server idea that we abandoned? Maybe we should pull that code out at this point. I can open an issue if needed for that.

Yeah it's not in any code the runs for a user. WebPanelServer can't use cspSource because it doesn't create a webview. But the server react code can (or should be removed).

Created #12140  for removing the old code. And validating this one.


With no JavaScript, animating anyway the elements or showing a message

With JS disabled, nothing appears visually. Maybe show a message or animate the elements anyway, not based on scrolling ?

u should do server side rendering

It look like it’s because objects show or hide depending on animation, who depends from scrolling, but can only be triggerd by JS

oh then i think it will be our own job to fix it and its not hard too :)


ROX-17499: Create quay registry once

Description
Currently the quay.io registry integration is created and deleted on a test spec basis. If tests ever run in parallel, this could be problematic for tests that require it. This PR changes it to create and delete the integration on a test run basis.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
all-qa-tests.

/retest

/test all


deezer and chat.google

Is is possible make it work for deezer.com and chat.google.com?

See the warning at the bottom of the extension popup. If some website that plays audio doesn't work, it won't work. i.e. if you get silence on deezer, that means deezer is using cross-domain URLs in <audio>, which Firefox doesn't allow touching.
WebRTC calls are a special case, that's definitely not <audio>/<video> tags. I haven't looked into it and have no plans to do so. (It's unlikely to be possible, all that stuff is controlled by the site's JavaScript.)

See the warning at the bottom of the extension popup. If some website that plays audio doesn't work, it won't work. i.e. if you get silence on deezer, that means deezer is using cross-domain URLs in <audio>, which Firefox doesn't allow touching.
WebRTC calls are a special case, that's definitely not <audio>/<video> tags. I haven't looked into it and have no plans to do so. (It's unlikely to be possible, all that stuff is controlled by the site's JavaScript.)


[wip] script tests for self_supervised

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes # (issue).
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@Borda  @ananyahjha93 what happened to this?

@ananyan is adding tests which we spoke about some time ago...

@williamFalcon this got pushed a bit due to a few things like a few blocking factors for simclr and swav. Still on my list right after those.

@ananyahjha93 is it still actual? if so, mid update according to the laters master...

@ananyahjha93 is it still actual?

@ananyahjha93 is it still actual?


Metrics not discussed in the paper

Hello,
The WEFE paper presented at IJCAI discuss WEAT, RND, and RNSB, but it does not discuss MAC and ECT.
I wondered whether there were an updated version that discussed those?
Without opening a new issue, the reference for MAC should actually be

Black is to Criminal as Caucasian is to Police: Detecting and Removing Multiclass Bias in Word Embeddings

Instead in the documentation is written as:

Black is to criminals caucasian is to police: Detecting and removing multiclass bias in word embeddings

(criminals is plural and "as" is missing: it0s a minor thing but the reference manager doesn't find it written in this way :))

Hello @raffaem,
In the IJCAI paper we only limit ourselves to the metrics you mention (WEAT, RND, and RNSB).
ECT was not included in our paper mainly because we did not know it until then.
On the other hand, we were not able to implement MAC well at the time, so we did not consider it.
In a few days I will release a new version of WEFE with MAC implemented (plus some debias methods).
I also expect this version to include a script that allows to evaluate bias using all of these metrics and then update the results obtained in IJCAI.
Thanks for your indication of the error in the paper Black is to Criminal as Caucasian is to Police: Detecting and Removing Multiclass Bias in Word Embeddings. I will keep it in mind to correct it!
In case you find it useful, you can find the MAC implementation in the multiclass_evaluation function of https://github.com/TManzini/DebiasMulticlassWordEmbedding/blob/master/Debiasing/evalBias.py.
On the other hand, ECT is proposed in the paper: Attenuating Bias in Word Vectors
You can find the original implementation here:
https://github.com/sunipa/Attenuating-Bias-in-Word-Vec as well as at https://github.com/dccuchile/wefe/blob/master/wefe/metrics/ECT.py
Best regards,
Pablo.


Pls update this patch, i have something wrong with the skin changer everytime when im going to practice tool everything is good but when im playing ranked it doesnt work. Sometimes the skinchanger works cuz im reinstalling everytime and its annoying.

opening more and more issues like this is pointless.


Boot Diagnostics/Serial Console vs Custom Storage Account Firewall

This page mentions that "Serial Console is now compatible with managed boot diagnostics storage accounts and custom storage account firewalls". However, this page  mentions that "User managed storage accounts associated with boot diagnostics require the storage account and the associated virtual machines reside in the same region and subscription and accessible from all networks."
As user managed account = custom storage account, those two articles contradict each other.

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

ID: 3a4dfc75-25a4-ae83-8d6a-00f267bf85b9
Version Independent ID: faa41c37-7b66-671b-09ca-3fb076f701e3
Content: Azure Serial Console - Virtual Machines
Content Source: support/azure/virtual-machines/serial-console-overview.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@ems75, Serial console supports the firewall in the custom storage account. To use the firewall, we must add your client/browser IP and Serial console services IPsto the firewall allow list.
I will update the article to reflect the latest updates.

https://docs.microsoft.com/troubleshoot/azure/virtual-machines/serial-console-linux#use-serial-console-with-custom-boot-diagnostics-storage-account-firewall-enabled
This link doesnt contain IP's for Africa Region,

@jumaqsoo Thanks for the comment. We are still working on boot diagnostic firewall support for the SouthAfrica geography, including storage accounts in South Africa North. When this compatibility is available, we will update the documentation.


feat: ✨ getTransactionReceipt

getTransactionReceipt Implemented

Hi @redDwarf03 Requesting you to check & merge this. Also we need a better layout for Example app - its becoming messier as we keep adding new feats - Any suggestions?

Hi. Ok i will check it and merge
Agree about example app.....
NB: i open a discussions feed on the repo with issue to discuss
NB: Sorry, i insist but i want to let my analysis_options.yaml

Also requesting you to first check and merge PR's every-time BEFORE you directly commit and push to this repo.
Unnecessary conflicts happen to resolve in the downstreams.

My bad... Sorry.
I will use PR now instead of commit directly
I tried to rebase your PR but i can't push -f on your PR...

I will copy your PR content in main branch and after that, i will improve process.. sorry again

You want us to resolve conflicts here?

no no. i will take your code, push in main branch directly. i won't merge this PR.
And after that, we will work only with PR. Let me few minutes please

included in main branch via https://github.com/archethic-foundation/wagmi_flutter_web/commit/73e9cf02e59949a3e9064ac5264d39c84583201a commit
I close this PR


Add hosting options

adding hosting options section
including Algorithmia

any update on merging this in? thx!

any reason for rejecting this?


Move Fungus examples to separate project on 2nd repo

Definitely need to remove the Sherlock stuff.
Unsure about the rest of the demos? Do we have license to redistribute the art and such?
See 1.2.g:
https://assetstore.unity.com/publishing/submission-guidelines#1. Content Restrictions
The Demos are a source of much of the trouble I suspect... most of the audio and possibly trademarked materials is in them.
I want to remove them all, and make them available as a separate package on the 2nd repo.
This would make Fungus-lite the default edition, which has the added advantage of reducing the core package size to under 4mb (>70mb with demos).

See #20


INKVConnInternal::do_io_*: handle null buffer

It's common for users of VC's to cancel io via a 0 byte, nullptr read or write on the VC. INKVConnInternal::do_io_read and
INKVConnInternal::do_io_write were not prepared to handle such cancellations. This updates them to handle this gracefully rather than crashing on a nullptr dereference. This change was found to be needed for the multiplexer plugin for handling HttpTunnel aborts.
For reference, see, for example, UnixNetVConnection::do_io_read which handles a nullptr MIOBuffer. This basically copies that logic into INKVConnInternal so it handles cancellation gracefully.

For Review
Hiding whitespace when viewing the diff makes this patch more understandable.

This didn't backport cleanly to 10.0.x, so I created a separate PR for it:
https://github.com/apache/trafficserver/pull/11802


check_exe_basename broken for python.

check_exe_basename does not work for python.
https://github.com/pwncollege/chio/blob/047fe35671e59f4327d768bded55e6eb42985ae7/chio.py#L75
On production, we get Executable must be 'python'. Yours is: python3.11. This also impacts program-interaction.

We should:

change check_exe_basename to accept a regular expression instead of just a string (re.match(basename_regex, os.path.basename(process.exe())))
have check_python and check_ipython do a regex like r"python(\d(\.\d+)?)?$"

I don't have the connectivity to test this right now, but if someone wanted to do that, that'd be awesome

Looks like @robwaz fixed this, right?

Yes, this can be closed.


feat: derpmap field in config

[x] read the CONTRIBUTING guidelines
[ ] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[ ] updated CHANGELOG.md

Currently DERP map can be set only from URLs or local yaml config paths. This PR adds a simple and optional DERPMap config field which allows setting initial DERP map which is useful when using headscale programatically.

Summary by CodeRabbit

New Features

Enhanced configuration capabilities with the addition of a DERPMap field in the DERP configuration.
Improved logic in the DERP map retrieval process to prevent runtime errors by excluding nil values.

Hi sorry, I've missed this, you can already load DERP config via file, so another method seems excessive, I don't think we will include this.

@kradalby I understand it can be loaded from a file, but that's a bit awkward when using headscale programmatically (as a lib). It would require the application to first write a file and then load it. Additionally, it provides more overhead if the configuration has to be updated after it was already written.
From the looks of it, the PR wouldn't undermine loading the map from file, it simply allows users to add an initial derp map as an argument of the config.

When you say lib, do you mean as part of a go program? because in that case you can just import it and override yourself.
I still dont think we will include this, sorry.

When you say lib, do you mean as part of a go program? because in that case you can just import it and override yourself.

This is how we set the initial config in Daytona and the use it to initialize the server here. It would be great if we could just add DERPMap to the initial cfg as well.

Ok fair, sorry, in my head I thought it would expose it to users via the config, as long as we dont let users set it via config.yaml, it is ok.

Ok fair, sorry, in my head I thought it would expose it to users via the config, as long as we dont let users set it via config.yaml, it is ok.

Thanks!


Enable AOM (AV1) on centos

Seems like there is no --enable-libaom on centos for some reason.

For anyone trying to build aom on centos, you can try this:
## aom
RUN \
	DIR=/tmp/aom && \
        git clone --branch ${AOM_VERSION} --depth 1 https://aomedia.googlesource.com/aom ${DIR} && \
        rm -rf CMakeCache.txt CMakeFiles && \
        cd ${DIR} && \
        mkdir -p ./aom_build && \
        cd ./aom_build && \
        cmake -DCMAKE_INSTALL_LIBDIR="lib" -DCMAKE_INSTALL_PREFIX="${PREFIX}" -DBUILD_SHARED_LIBS=1 .. && \
        make -j$(nproc) && \
        make install

Note DCMAKE_INSTALL_LIBDIR="lib" which is required to install to /opt/ffmpeg/lib instead of /opt/ffmpeg/lib64.
This option can fix the following error:
ERROR: aom >= 1.0.0 not found using pkg-config

interesting !
yeah centos never had aom in our image : https://github.com/jrottenberg/ffmpeg/commit/5fd29bda1b3703243993407458ee94f5a1ee9d6b

remove aom for centos7 since it's doesn't have the build tool required

but your workaround looks promising

Also the Centos version is bumped to 8, so probably the build tool is already here (but I'm not sure what tool exactly).
AFAIK libaom is still in Dockerfile, but just not linked:
https://github.com/jrottenberg/ffmpeg/blob/23c16671466d777ae4d27a5c5b41b288902fd88c/docker-images/4.2/centos/Dockerfile#L322-L332

@ernado if you want to try the latest centos8 (or even the centos7)  containers libaom should be available.

Thank you!


KAFKA-14563: RemoveClient-Side AddPartitionsToTxn Requests

https://issues.apache.org/jira/browse/KAFKA-14563

@artemlivshits Can you also help take a look?

@artemlivshits @jolshan @rreddy-22
Summary of the updates:

Address Artem's comments for the EndTxn server side handling
Make it a fatal error for clients when receiving produce response with InvalidPidMapping

@artemlivshits @jolshan I added a table for the state transition. Can you help double check if it is what we expected?

Green build https://github.com/apache/kafka/actions/runs/11979198275/job/33405415151


python310Packages.pyisy: 3.1.13 -> 3.1.14

Description of changes
Diff: https://github.com/automicus/PyISY/compare/refs/tags/v3.1.13...v3.1.14
Changelog: https://github.com/automicus/PyISY/releases/tag/v3.1.14

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 218381 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.isy994
python310Packages.pyisy
python311Packages.pyisy


Installation of 1.3.0, it still reports versions as 1.2.0

Hi,
Installing 1.3.0 , it reports everything still at 1.2..0 level:
              ## ## ##       ==
           ## ## ## ##      ===
       /""""""""""""""""\___/ ===
  ~~~ {~~ ~~~~ ~~~ ~~~~ ~~ ~ /  ===- ~~~
       \______ o          __/
         \    \        __/
          \____\______/

| |__   ___   ___ | ||__ \ | | ___   ___| | ___ _ __
| '_ \ / _ \ / _ | | __) / _` |/ _ \ / __| |/ / _ \ '|
| |) | () | () | | / / (| | () | (|   <  / |
|_./ / _/ __|,|/ _||_|_|
boot2docker: 1.2.0
3.16.1-config-file : e75396e - Fri Aug 22 06:45:30 UTC 2014
docker@boot2docker:~$ docker version
Client version: 1.2.0
Client API version: 1.14
Go version (client): go1.3.1
Git commit (client): fa7b24f
OS/Arch (client): linux/amd64
Server version: 1.2.0
Server API version: 1.14
Go version (server): go1.3.1
Git commit (server): fa7b24f
docker@boot2docker:~$
I did an install of 1.3.0 over 1.2.0 with this result, then completely uninstalled Boot2Docker and did a clean installation, and the same result
Ideas?

Sorry, this repository is long-since deprecated in favor of Docker Toolbox (whose usage is now also discouraged in favor of Docker for Windows and Docker for Mac).


Fix COMPRESSION_ERROR on valid HPACK input

This fixes a bug that causes a valid http2 header value to be rejected with a COMPRESSION_ERROR:
With the test input I added, a leaf node is found exactly on a byte boundary. Because the leaf node check happens before the shift check, byte_boundary_crossed is set to 0, but then immediately incremented again. And because finding the next leaf node crosses three byte boundaries, we end up with 4 total and return an error. Reordering the checks fixes the issue.
Note: I wasn't able to run the whole test suite so I'm not sure if this breaks anything else.

@ezelkow1 Can you kick off builds here?

[approve ci]

[approve ci]

Thank you for the fix. I ran test_HPACK with a couple of external test data and I was realized that current HPACK decoder seems to be broken and it's still broken even with your fix.
What I did are:
$ cd trafficserver/proxy/http2
$ git clone https://github.com/http2jp/hpack-test-case.git
$ ./test_HPACK -i hpack-test-case/go-hpack/

test_HPACK(16235,0x10da86600) malloc: nano zone abandoned due to inability to preallocate reserved vm space.
32 test stories
REGRESSION_TEST initialization begun
REGRESSION TEST HPACK_Encoding started
    REGRESSION_RESULT HPACK_Encoding:                           PASSED
REGRESSION TEST HPACK_Decoding started
BX=c99r6jp89a7nou0026b=3u0026s=q4; localization=en-us%3Bus%3Bus <-> BX=c99r6jp89a7no&b=3&s=q4; localization=en-us%3Bus%3Bus
RPRINT HPACK_Decoding: Story 8 sequence 1 failed.
    REGRESSION_RESULT HPACK_Decoding:                           FAILED
REGRESSION TEST Regression started
RPRINT Regression: regression test
RPERF Regression.speed 100.000000
    REGRESSION_RESULT Regression:                               PASSED

It might be an error in the test data, but I think I checked ATS can decode all the encoded data from different implementations when we were actively working on HTTP/2 and HPACK. Let me check the error above first.
I haven't looked into your change closely, but I'd suggest to move the test case you added into test_HuffmanCode since it's purely for Huffman decoding.

Seems like the errors I found are due to incompatibility between the test data and test program. We can ignore the errors for now.
And good news is that your change passed all the test we can try. I'll look into your change closely later.

@maskit sorry for the delay: I had to update my machine to get the autest suite to run. I pushed a fix for the failing EOS test. That fix itself made one of the Huffman tests fail, because it was trying to decode an EOS symbol, which is invalid, so I changed it to check for an error.
I also moved the tests I added to test_Huffmancode.cc

Cherry-picked to v9.2.x


Bug: list creation feature in beta seems to have degraded?

Describe the bug
Not sure what's going here but the list creation feature has degraded (on Firefox and Chrome, including private windows)

The search bar is gone so I'm not able to filter on contributors
There are no contributors loaded. It seems that the v1/contributors call is returning good data but it's not populating the list of people at all.

There's nothing obvious in the console or the network tab so not sure what's going on. @OgDev-01 any ideas?
Steps to reproduce

Go https://beta.insights.opensauced.pizza
Log in
Go to https://beta.insights.opensauced.pizza/hub/lists/new

Browsers
No response
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

This is also a followup to https://github.com/open-sauced/app/pull/1698 which last week was working in the deploy preview but seems to have regressed.


Add generated docs to Matrix, Vector and Point types

Sorry for deleting the old pr, but my git-fu is not that good and I ended up deleting everything while rebasing.
Previous commit/discussion is here

Sorry! Looks like bors isn't hooked up, and this slipped past me.
Would you mind rebasing/reviving this, please?


refactor: move update submission API call to store

🔧 What changed
This change moves the API logic from the component to the store following a discussion we had in a huddle with the team.

nice improvement!

I would +1 adding a pinia test now that it's properly in the store, this is a good method to have a test on.
your description is missing the issue #. I would add one or just remove that line

I think I made an issue for this. Might have assigned myself @ermish @theyokohamalife

nice improvement!

I would +1 adding a pinia test now that it's properly in the store, this is a good method to have a test on.
your description is missing the issue #. I would add one or just remove that line

I think I made an issue for this. Might have assigned myself @ermish @theyokohamalife

I had. It has a requirement for a test. But that can be done in a future PR if you would like. But I assigned it to you @theyokohamalife . It is #801

@ermish @theyokohamalife @Anissa3005. The testing of this due to it calling the API requires a lot of setup with a mock server request and change of the vitest.config.ts to be changed to use nuxt instead of the way it is set up now so it can correctly resolve imports. @gminetoma And I looked and read up on this a lot today. So maybe the tests for this and #819 should be completed after tickets to reconfigure the vitest.config are made and PR is merged as well as a mock server is setup properly


Openjdk10 release updates

Release pipelines are failing on a section that's different between the nightly and release sections of the openjdk10 pipeline. This unifies the code between both of them

@sxa555 conflicts?

@sxa555 merge conflicts

I'm going to cancel this because arm32/JDK10 is pointless and superceded by JDK11


Review possible architecture of chrome extension with Firebase backend

Description:
There are scenarios where a connection between the cookie analysis extension and a backend service will be needed. For example, for Enterprise Chrome users, the Admin of a given platform may want to control certain aspects of the extensions for users logged as belonging to the organization. The purpose of this task is to determine the architectural details of such a configuration.

E.g. To use Firebase backend with Chrome extension, we need to:

Create a Firebase project.
Enable the Realtime Database and Cloud Firestore
Install the Firebase SDK for your Chrome extension.
Initialize the Firebase SDK in your extension's background page.
Use the Firebase SDK to access the Realtime Database and Cloud Firestore from the extension's code.

Reference: https://github.com/firebase/quickstart-js/tree/master/auth/chromextension


[Feature request]: Add timeout for keyboard backlight

Please describe your feature request
First of all a big thanks for wluma! It is really a program to handle my backlight automation needs!
Nonetheless it would be nice to have a timeout for the keyboard backlight.
Although there are some tools that provide a keyboard backlight  timeout after a certain period of time, where the keyboard is not interacted with, to turn of the backlight, I assume that these tools would interfere with wluma's learning thing.
My assumption is, that if i had a script running that would turn of the backlight, wluma would think, that my backlight under this lighting conditions should be turned off.
I think it would make more sense for wluma to handle it by itself.

Hello!
Sorry for the long delay in my response!
First of all, I want to say that the use-case is definitely valid, and you are right that a separate script changing the backlight would indeed make wluma learn the new behavior.
Could you tell me more about your experience, in what cases do you want the keyboard backlight turned off? What if you are just using a mouse / touchpad for several minutes, is that the reason to turn the keyboard backlight off? If you are watching a movie, would you rather want to turn off entire wluma off for this period, as opposed to just keyboard?
What I'm trying to understand is whether it makes sense to turn off entire wluma on inactivity (and possibly save some energy on not trying to scan screen contents and various sensors), or there are some cases when only keyboard backlight needs to be turned off. For the former, a cheap solution could be to make swayidle start and stop wluma process.

Thanks for the reply!
I don't know if my use case is common. I am using my Laptop as a desktop replacement. It's hooked up to a second monitor and the internal screen is used as a secondary display.
Apart from the keyboard backlight everything works as expected and it's really nice to have both the internal and the external screen controlled by the ambient light sensor.
What annoys me is the keyboard backlight not turning off since I am not using the internal keyboard it is more of a distraction.
I think this case can be handled in 2 ways:
-Completely turn off keyboard backlight when connected to an external display

a simple timeout that is only affected by the interaction with the touchpad/internal keyboard

This way it should just stay off during "Desktop mode".
What do you think of this approach?
I think a timeout would be also superior during a movie session or similar cases because the brightness of the screen should stay in auto mode. If you would just stop wluma it would also stop the brightness control or not? 🤔

Thanks for the details, it really helps to understand the actual use-case!
My immediate thoughts is that detecting whether interaction occurs via an internal vs external keyboard would probably be quite difficult to do, and perhaps more importantly I don't think it's up to wluma to impose such kind of rules on its users.
It sounds like, if you had the ability to tell wluma to stop managing a specific device (keyboard in this case), it would solve your use-case. We don't have such feature built-in, but we do have a way to simulate this, to see if it actually leaves you happy with the workflow. Would you like to try this?
If yes, here's how I would do it:

Copy your current config.toml into config-laptop.toml and config-desktop.toml, and remove the keyboard part from the config-desktop.toml one (so they differ only in that one of them has keyboard, and another does not).
Remove config.toml and instead create a symlink config.toml, pointing to one of the actual files.
In order to switch profile (this part you can automate via a script, if you want):

Stop wluma
Point symlink to another config file
Turn off keyboard backlight
Start wluma

Let me know if this works at all, and if yes, whether it actually gives you the experience you desire?
If this works well, then we can think of how to build this in, whether by having "profiles" that you can choose between, or an ability to toggle specific devices on and off.

Hey, did you manage to experiment with this, I'm wondering if it's still relevant for you?


Fetch subsequent paginated data on list_* functions

This PR addresses #80.

I now implemented it for the other list_* functions too


Matching patterns in subdirectories not greyed out

For example if the .gitignore contains:
build/

Directory structure:
project
\---build/
\---sub
    \---build/
\---.gitignore

project/build/ will be greyed out, but project/sub/build/ will not, even though project/sub/build/ is correctly detected as ignored.

@benedictleejh  Thank you for reporting. Does this issue appear in the IntelliJ 15 ?

Yes, I'm using idea-gitignore with IntelliJ 15.

This issue has been already discussed in #174
I've suggested a walkaround in the third post which you can apply.
Anyway the issue is still open and waits for the fix.
Thanks again for your time and have a happy coding !


"Settings" page buttons do not return to main display

When going to the settings page ("File" -> "Settings"), pressing either the "Save" button or "Cancel" button do not return to the main display. User must quit the application (at the traybar) and relaunch it.
Note: The functionality of the buttons works correctly (AFAIK), meaning "Save" will save changes, and "Cancel" disregard changes. It's a GUI thing.

Please tell me about your environment? (see also CONTRIBUTING.md)

Application version
Operating system

Have the same issue on Windows 7 x64 Enterprise and Window 8.1 Professional. Can't remember if I tried on Windows 10.
My solution was to autoprovision config.json to all users with hideMenuBar set to true, and hope people don't create more teams until it is fixed ;)

I encounter the same problem. I'm using version 1.1.0 and Windows 10 x64.
The button do save/cancel when you click'em. However they do not "close" the settings screen.
The only way back is CTRL+Q

I also experience the same issue on Windows 8.1 and Mattermost Client 1.1.0.
The issue seems to only occur if there is a blank in the path where the client is installed. When starting the client from e.g. C:\Mattermost\Mattermost.exe, Cancel and Save close the Settings page, but when the path contains a blank (e.g. C:\Matter Most\Mattermost.exe) the buttons do not work.

Reproduced. Thanks.


rewording suggestion for step 26 learn introductory javascript by building a pyramid generator

Describe the Issue
first discussed on the forum here: https://forum.freecodecamp.org/t/ambiguous-wording-in-learn-introductory-javascript-by-building-a-pyramid-generator-step-26/689842
I had a concern about the starting sentence of step 26 of the js pyramid project here: https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-introductory-javascript-by-building-a-pyramid-generator/step-26
It starts out with: "The let keyword allows a variable to be reassigned." which I think can be misinterpreted to mean that we can use the let keyword to reassign a variable. I believe a reword to "Declaring a variable with the let keyword allows it to be reassigned." will be not as easy to misinterpret.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-introductory-javascript-by-building-a-pyramid-generator/step-26
Your code
N/A
Expected behavior
slight reword to instructions needed as per above
Screenshots
No response
System
N/A
Additional context
No response

Seems like a good change.

This looks like something that can be fixed by "first-time" code contributors to this repository. Here are the files that you should be looking at to work on a fix:
File that needs to be changed:
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/15-javascript-algorithms-and-data-structures-22/learn-introductory-javascript-by-building-a-pyramid-generator/660f165270622fd4ec0da3f7.md
Change needed:
From:
The `let` keyword allows a variable to be reassigned.

To:
Declaring a variable with the `let` keyword allows it to be reassigned.

Please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or our forum if you need help contributing; our moderators will guide you through this.
Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing.


Steam Deck: Heroic Games Launch in Background

Hi, after pulling my hair out trying to use Heroic Bash Launcher, I finally gave up and tried Boilr.  It looks promising and I'm looking forward to using it for other launchers.  Right now, I'm focused on Heroic.
Setup of the shortcuts seems to work just fine.  Games show up in Steam with artwork and all looks good.  Problem is, when I start a game (I've tried Kena and Outer Wilds), the game loading screen shows (Steam UI) and the game loads in the background.  In both cases, I can hear the game audio, but I cannot figure out how to bring the game to the front (since, as far as I know, Game Mode does not have ALT-TAB).
I cannot figure out why this isn't working!  Happy to test things if helpful.  FYI, I'm on SteamOS Beta, Steam UI Stable, latest version of Heroic Games Launcher and Boilr (both via Flatpak).
THANK YOU!
EDIT:  It occurred to me that this is probably a Heroic Games Launcher issue, not a Boilr issue.  As far as I can tell, the shortcut is set up properly and should work (as evidenced by this working just find in Desktop Mode on the Steam Deck).

If you launch the game from heroic directly, does it then work?
Have you tried to go into boilr settings and change between "launch directly" and "launch through heroic"? (if you are launching directly, make sure that you have a proton version set on the steam shortcut, if it is through heroic then make sure proton is off)

I am going to close this issue here on BoilR, but if you find that BoilR needs to do something different to enable Heroic to launch the games better let me know and I will reopen this.

Sounds good. FYI, launching directly from Heroic has always worked for me. Oh and the Boilr shortcuts also work in desktop mode. So this doesn't appear to be a Boilr issue at all.

Just to provide a further update, I went with the nuclear option - uninstalled all games, game shortcuts, heroic launcher AND Boilr (including user data in home/deck/.var).  Reinstalled heroic, got 1 game working through that, then installed Boilr.  First tried the shortcut from desktop mode, and after confirmed it worked, went into Game Mode and tried it and... voila!  It worked!
No idea what was happening the first time.

Thanks for the update! Glad it worked out.
On Mon, May 30, 2022, 19:23 krelltunez @.***> wrote:

Just to provide a further update, I went with the nuclear option -
uninstalled all games, game shortcuts, heroic launcher AND Boilr (including
user data in home/deck/.var). Reinstalled heroic, got 1 game working
through that, then installed Boilr. First tried the shortcut from desktop
mode, and after confirmed it worked, went into Game Mode and tried it
and... voila! It worked!
No idea what was happening the first time.
—
Reply to this email directly, view it on GitHub
https://github.com/PhilipK/BoilR/issues/162#issuecomment-1141364907, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEE7DES22KG3BKGLG5KLELVMT2QHANCNFSM5XI2EVFA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I am having issues with GOG games installed through heroic as these shortcuts don't seem to run through heroic but to the exes directly. In Boilr settings only Epic games are showing under Heroic as well, so I would guess these are related.

@oscarsiles can you open a new issue for this? I think it is something else, just so I can track it.

@oscarsiles I found the bug, gog games where not shown in the list of "launch through heroic". The newest version 1.3.18 has the fix.


crash caused by forgero

modpack im using is DreamCraft, i added this mod in because i like it.
crash-2022-11-03_17.59.06-client.txt

same thing

Is this issue present with the newest 0.10.1 version?

Closing this as it is using an old version of Forgero


Broken Documentation

Much of the example code is outside of many of the <pre> elements at this link ; this makes the learning curve even worse.
I'm using a temporary email so don't bother.

this issue is reported in wrong repository - should be in https://github.com/godotengine/godot-docs
Anyway i pushed fix there https://github.com/godotengine/godot-docs/pull/307


adds shell command imp

Types of changes

[ ] Enhancement (project structure, spelling, grammar, formatting)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

A description of the changes proposed in the Pull Request

implements the shell command IMP requested in #83
adds a new file to src/lib containing a single function shellCommand() that does the following:

spawns a child process that calls a local executable
handles the data from the local executable
passes an error to stderr if the call fails

the local executable is expected to expose a cli that accepts two arguments: --calculate and --impl
the returned data is displayed in the console via stdout and saved locally as a yaml file

Note
This is in draft pending:

collab wit @gnanakeethan on IEF integration
collab with folks working on #84 (pimpl.py), including refining the error handling

Also you can rename file name as shell-imp.ts to be in the same convention with the other repo parts. Also, I'm out of the context, but the module name and the function naming seem too general. F.ex. shellCommand can be renamed to something that will describe better what the function does.


Access to local images

Can Kitematic access and manage local images (the ones created with docker build and stored into the local machine?)?

Not yet - but there's a PR to do just that: https://github.com/kitematic/kitematic/pull/882
Closing for now, but feel free to re-open if needed


Write the package config location to the test bootstrap.

Work towards. This is the Flutter companion to https://github.com/dart-lang/test/pull/2245.

@christopherfujino @natebosch @jakemac53 can you advise on the best way to get the package config URI here?

Is Isolate.packageConfig null? I think flutter tool might be precompiled which could cause that.....
Unfortunately, package:package_config does not expose something to just do the package config lookup for you, see https://github.com/dart-lang/package_config/issues/126 for the feature request which you might want to chime in on.

This code is in flutter_tools - does that package have any existing code related to finding or using the package config that we can reuse?

@christopherfujino @natebosch @jakemac53 can you advise on the best way to get the package config URI here?

Is Isolate.packageConfig null? I think flutter tool might be precompiled which could cause that..... although there is likely some way to get one still (in pub we have one even though package:test is also compiled to kernel).

We want the package_config.json for the test app, not the tool, or am I misunderstanding the use case here?

We want the package_config.json for the test app, not the tool, or am I misunderstanding the use case here?

We want the one for the app, not the tool. Good point that in the flutter case, these are not the same (in the dart test case, they are the same).

@sigurdm do you have any insights on the best way to get a test app's package-config.json, since you're doing some similar work in the tool?

flutterProject.packageConfigFile.uri will probably work.
But I think it would be better to thread a BuildInfo object from:
https://github.com/flutter/flutter/blob/8e4f704abc4576206c3d56e1a430f66457b706f3/packages/flutter_tools/lib/src/commands/test.dart#L358
in to testRunner.runTests() https://github.com/flutter/flutter/blob/8e4f704abc4576206c3d56e1a430f66457b706f3/packages/flutter_tools/lib/src/commands/test.dart#L591
in to loader.installHook()
https://github.com/flutter/flutter/blob/0287c225646b59fa4a08cae699bb16fa3ccf5af8/packages/flutter_tools/lib/src/test/runner.dart#L242
to FlutterPlatform()
https://github.com/flutter/flutter/blob/4e70bfae2bff2c272af7fb7a870695edd9f8cfb4/packages/flutter_tools/lib/src/test/flutter_platform.dart#L86
And then use globals.fs.path.toUri(buildInfo.packagesPath). (will be globals.fs.path.toUri(buildInfo.packageConfigPath) after https://github.com/flutter/flutter/pull/150559)

Thanks for the suggestion @sigurdm. I have applied the changes you suggested.

@christopherfujino This is ready for review now.

I suspect google testing will pass once https://github.com/flutter/flutter/pull/150559 makes it into g3.


Houdini Plugin Conflicting with latest SvelteKit

Describe the bug
Trying to build application with latest sveltekit gives
Multiple universal layout module files found in src/routes/ : +layout.js and +layout.js
Apparently, there is some problem with the houdini plugin and latest @sveltejs/kit (worked fine in 1.15.0, but fails in 1.15.5+)
Removing houdini() from plugins in vite.config lets the build succeed.
Using the houdini plugin is apparently creating a new +layout., which is conflicting with the existing +layout. of the user
It seems related to this PR (https://github.com/sveltejs/kit/pull/9590)
Severity
blocking an upgrade
Steps to Reproduce the Bug

Create new Svelte kit project
Add Houdini to it
Run the project

Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-jjttza

Thanks for reporting this @maxzeroedge! This has been fixed in 1.2 which should be released very soon. If you have time to test the release candidate, that would be super helpful. You can grab it with the @next tag.

1.2.0 is out! Thanks for the patience 👍


iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiihb!$

Output of docker version:
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):

sorry, my cat was sitting on the keyboard ...

haha, no worries :cat:


example: template

take a look at whether you want this to be the template

we can append a link to it in the OneNote afterwards maybe

Looks good to me.


CHANGELOG: fix typo

Happy new year, and thanks for all the fish !
Cheers

Thanks:). Even though I don’t get the fish reference 😅.

Thanks:). Even though I don’t get the fish reference 😅.

See https://en.wikipedia.org/wiki/So_Long,_and_Thanks_for_All_the_Fish ;-)

See https://en.wikipedia.org/wiki/So_Long,_and_Thanks_for_All_the_Fish ;-)


refactor(beatree): remove hard dependency on BranchesTracker in updater

Same logic as the preceeding leaves PR, but for branches.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#531  👈
#530
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Nov 1, 3:11 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Feature/automated testing - closes #744

Make sure to pull in the recent changes from develop. There are concole errors about the skin helper when branding is empty.

Also, the assertion fails on my end. Did the assertion "should contain search form at "/catalogue" page" pass in your local branch?

Alter adding $(function(){ ... }); all (two) tests passed successfully. Which one fails for you`?

Resolved conflicts.
The test case which is testing /catalogue page still fails at first. It only passes when the meteor server reloads itself automatically after changes made to the code. This happens even being wrapped to $(document).ready(function(){ ... }).
@brylie Any ideas?

Please review my commits. The code is now simpler, and slightly easier to read. Both tests pass on my local development environment.

@brylie First test still does not pass on my local env.

@ashakunt Do you have any suggestions on what might be the problem here? Please see the discussion above.

Hey @frenchbread check out jasmine-jquery plugin. It might make our DOM checks a bit more semantic, without requiring the Array.length checks, etc. The jQuery dependency is alright, since jQuery has a clean API and is a central part of our project.

Here is a video demonstrating Intern.js setup, testing, etc:

The previous video recommends using a library called Leadfoot for simulating user interaction. Leadfoot has methods relating to DOM Elements, such as findById. The DOM Element may be compatible with assertion libraries, e.g. expect(element).toExist().

@frenchbread here is the fixed video link:

Any status update on this PR?

@brylie Working in feature/theintern-automated-testing branch.

@brylie Can you check what to do with the issue and the PR?

Closing, since this task is not currently in progress. We can leave the branch open.


[breaking] move packaging functionality into its own package

How to migrate

npm install -D @sveltejs/package (or pnpm, or yarn)
If you have configured some package options, move them to the top level of svelte.config.js:

const config = {
	kit: {
- 		package: {..}
	},
+	package: {..}
};

Replace svelte-kit package with svelte-package in your package.json script

PR description
Draft PR to move the package functionality out of SvelteKit. This makes the following decisions (all up for discussion):

New package is called @sveltejs/package
Package has its own CLI, svelte-package, svelte-kit package will be removed
Minimal SvelteKit interop; kit.files.lib is used to set source if source is unset
We don't call svelte-kit sync in this package because you can use it standalone, but the template will call it before calling the package command

TODOS:

SvelteKit and this new package share some utility code around filesystem stuff and options parsing. How do we best share this, if at all? Should we create a private package in the monorepo and enhance the rollup config to inline it, so no visible changes to the outside? Sounds like the best solution to me
How to deploy this as a new package? Changeset masterminds please help. Is it enough to create a changeset and the rest is taken care of?
Adjust SvelteKit code: remove packaging, remove the config (error and tell to switch to this new package)

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Tests for create-svelte are failing because the package isn't available yet I think

Did as much of the grunt work of updating this post-#5748 as possible. Since kit is no longer bundled, the @internal/shared package isn't really an option unless we were to actually publish it to npm. I think it's preferable to just copy the few utility functions kit and package have in common, unless we find ourselves using them in more places.
I think we should get rid of the exports and files options in favour of having people include the "exports" field in their package.json files, pointing at the src code, so that the generated pkg.exports has the same shape but points to the package directory. This would allow people to use packages in a monorepo without needing to constantly build them. The exports would be treated as entry points, and we'd follow all the imports from those entry points to figure out which files need to be included in the package, making files unnecessary.
I don't have a strong opinion about whether that should happen in this PR or a follow-up.

I think we should merge this as-is and do the other stuff in follow-up PRs.


Add copying of stall and review details from user interface

Resolve #148

Codecov Report

Merging #169 (59f87b7) into master (f162a52) will decrease coverage by 1.95%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #169      +/-   ##
============================================
- Coverage     69.81%   67.85%   -1.96%
  Complexity      508      508
============================================
  Files            84       84
  Lines          1630     1677      +47
  Branches        179      179
============================================
  Hits           1138     1138
- Misses          435      482      +47
  Partials         57       57

Impacted Files
Coverage Δ

src/main/java/foodwhere/ui/ReviewListPanel.java
0.00% <0.00%> (ø)

src/main/java/foodwhere/ui/StallListPanel.java
0.00% <0.00%> (ø)

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


Community Partner Active/Inactive Flag

Community Partner Active/Inactive Flag must be implemented in the system. A Community Partner must be made active when they are tagged to a project and inactive when they are not.

@kalpanajha  assigned as QA

Adding scheduler to execute query during Sprint 4.

@kalpanajha as QA tester

Community partner active/inactive flag
https://drive.google.com/open?id=1rnWKtNhnu94gLRyKt1JVRYoFq5H5sCM5


robust entity value extraction problem

Rasa NLU version: 0.14.3
Operating system: Windoes 10
Content of model configuration file:
language: "de_core_news_sm"
pipeline:

name: "nlp_spacy"
name: "tokenizer_spacy"
name: "ner_crf"
name: "intent_featurizer_spacy"
name: "intent_classifier_sklearn"
name: "ner_synonyms"

Issue:
I try to extract the value of a entity 'domain' from a sentence.
In my training data there is an example 'Who is responsible for the [network](domain:network)'
I wrote a simple helper function in python to evaluate the model performance. For example the sentence "Who is responsible for the servers".
The model chooses the right intent but can not extract an entity. The sentence is almost the same like in the training data, only the entity value is different.
How I make the model more robust against such errors?
I do not have many training examples but shouldn´t the model be able to extract the correct entity if the sentence is nearly the same like one in the training data?

Have you tried  Lookup table ?
It might enhance the ner_crf performance
https://medium.com/rasa-blog/entity-extraction-with-the-new-lookup-table-feature-in-rasa-nlu-94c6c30876a3

I´ve heart about the lookup table but didn´t tried it yet. Is it the only option to gain a higher model performance?

Please ask these kind of questions in our forum


Updated to use presets

Presets are required to get compiling in babel now.

Don't like hardcoded presets. Why you don't want to specify them in .babelrc?

I don't like hardcoded presets either, but I also don't like that I have to create .babelrc just for that (I build with Browserify and I'm sending the presets on the the transformer). Isn't there a middle ground?

I think a good middle ground will be adding support in Jest to add params to the transformer.  I will look into writing this this weekend and see if I can get that approved by their team.

@mikevalstar that would be great, something like a scriptPreprocessorSettings, and it wouldn't be helpful just for this, but we could fix things like #51 without the need of a new release (or a fork).

Looks like its already built in but not utilized.  I'll have to leave #51 to you (not quire sure what you need to do for that ;), but I'm fixing it up now.

Ok, so assuming the Jest team is good with my other pull request, you will be able to send in any options to the babel transformer that you would be able to send into the regular babel-core

@mikevalstar awesome :smile: ! And regarding #51, the istanbul ignore next comment was removed, so the coverage report includes the babel code, but it shouldn't.

This can be solved by using a .babelrc file. If you don't like using a babelrc file, please use your own preprocessor. See https://github.com/facebook/fbjs/blob/master/scripts/jest/preprocessor.js for an example.


Does canvas resizing support HiDPI?

Hi Andrew Smith, I'm planning to use Springroll 2.0 with Phaser3 for one of my games. We had HiDPI support in Springroll 1.0, but in Springroll2, I couldn't find it. I have checked SafeScaleManager. Will you help me with this?
Thank you.

Hi @navinpambi, we don't currently have any extra support included for HiDPI displays. My understanding is that the SafeScale should work fine as it just works based on pixel values. Is there something you are aware of that is missing or not working?

No activity for over a year, closing.


Empty List “Request a New Project Set” button in Public Cloud leads to creation of Private Cloud Product

Describe the bug/ Actual behaviour
Empty List “Request a New Project Set” button in Public Cloud leads to creation of Private Cloud Product. - Urgent. Priority: H. Size: S. Assignee: Oamar
Definition of done:
Bug is fixed and software behaves as expected.

Now empty List “Request a New Project Set” button in Public Cloud leads to creation of Public Cloud Product:
[Uploading... image.png]


chore: add support of worker-types v3 on cf adapter

Cloudflare released an update for @cloudflare/workers-types last month.
It is now automatically generated from the source as announced here.
This is really helpful for development as it ensures typing to be always up-to-dated with the latest runtime.
However, the current CF adapter has a peerDependency of v2 only. It would be nice if we could add support of v3 as well.

Closing this in favour of #645 as it should be branching off from dev based on the CLA


feat(CX-2293): explanatory Banner Appears in My Collection after Artwork Submission

The type of this PR is: Feature

This PR resolves CX-2293
Description

Show explanatory banner when successfully uploading an Artwork to MyCollection with SWA flow.
The banner is displayed once per session and can be closed
Based on LocalStorage

https://user-images.githubusercontent.com/36167539/156234482-31ab12f5-aeb9-467b-ae92-d58b9606809a.mp4
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Show explanatory banner when successfully uploading an Artwork to MyCollection with SWA flow. The banner is displaued once per session and can be closed -daria

iOS user-facing changes

Android user-facing changes

Dev changes

This PR contains the following changes:

Cross-platform user-facing changes (Show explanatory banner when successfully uploading an Artwork to MyCollection with SWA flow. The banner is displaued once per session and can be closed -daria)

  Generated by :no_entry_sign: dangerJS against f80dcbe290a109abaa30daecf2a0997003617a4d

Thank you for the suggestion @MounirDhahri ! Sorry it took that long
I added changes, I hope this is what you meant :)

one test is failing after merge with mail, i'll check main for updates later


Not compatible with AOT

Please could you package up the library in a way that's compatible with AOT (https://angular.io/docs/ts/latest/cookbook/aot-compiler.html). Thanks.

You can use https://github.com/doorgets/ng-truncate for AOT ;)

@mounirrquiba That package has a lot of code for something relatively simple like this.


Update configure-role-based-administration.md

updated to reflect radio button options in SCCM dialogue

@NonSecwitter : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.

Thanks @NonSecwitter for the contribution, good catch! By my count, there are actually 11 instances of that string on the page. Would you mind changing them all to be consistent? (If you're using the GitHub.com editor, CTRL-F opens a search box in the browser editor.)
Also, do you want to fix the string on the other option in that dialog as well? :) Instead of "All securable objects that are relevant to their associated security roles", it's now "All instances of the objects that are related to the assigned security roles."

Hi @NonSecwitter, are you able to make these other changes? If not, please let me know and I'll include the changes myself (but still accepting your contribution). If I don't hear from you by Tuesday, 14 August, I'll just do it.

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

Huh...well there's a new one. Sorry, I didn't realize that my commit will trigger the license agreement requirement for you. Well, a good thing to have in place if you ever feel the need for a larger contribution!
@NonSecwitter please sign the Contributor License Agreement so that we can continue with this.

I haven't heard back from @NonSecwitter. Since his changes were minor (and did not originally trigger a CLA check), I'm just going to merge this pull request with my added commits.


Download files and fix populate

Download files from the backend. And fix some populates issues

Story Points: 5
See the discussion in Backend S2 W8
Powered by Parabol


fix: Rename trigger rav to Rav trigger

Change hardcoded trigger value of 1000 to a metric tap_rav_request_trigger_value
fixes #313

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 68.282%

Totals

Change from base Build<PHONE_NUMBER>5:
0.02%

Covered Lines:
4213

Relevant Lines:
6170

💛  - Coveralls


[Docs] Refinement of the description of relationships and models

Signed-off-by: Lee Calcote<EMAIL_ADDRESS>Notes for Reviewers
This PR augments existing docs pages with additional material.
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit a289a44079c728d799edb28a9f01a92ba1853039 at: https://666f52407b5d7f45e92557b2--meshery-docs-preview.netlify.app

🚀 Preview for commit 835bd612da17c933fa586c754fb57c58af96cc8f at: https://666f5e2656f07c493e1b0354--meshery-docs-preview.netlify.app


SBT-based project can't resolve 'json-schema-codegen-sbt' dependency

Hi,
As the default (unspecified) SBT resolvers don't appear to resolve your artifact in my project, is it at all possible that a, correctly configured, 'Resolver.bintrayRepo("owner", "repo")' declaration necessary/might work (See 'SBT-Resolvers')? Something like the folowing (doesn't work :-( ):
resolvers += Resolver.bintrayRepo("voxsupplychain" "ivy-public")

I followed your 2 concise SBT-build additions from the README but, when I try to run my new project (even simply invoking the 'sbt' console command (SBT 0.13.9) within my project's root directory), the project is unable to resolve your artifact(s) at one of its known repos (I list SBT repo URL output, below).
Just for the record, could you confirm that the following URL is where you'd expect to resolve version 0.3.0 of the 'json-schema-codegen-sbt' code dependency?
http://dl.bintray.com/voxsupplychain/ivy-public/com.voxsupplychain/json-schema-codegen-sbt/scala_2.10/sbt_0.13/0.3.0

After adding only your code's dependency requirements to my project (it's a fresh Intellij IDEA SBT project), the sbt console is attempting to resolve to the following URLs:
<SBT CLi output fragment>
[warn] 	module not found: com.voxsupplychain#json-schema-codegen-sbt;0.3.0
[warn] ==== typesafe-ivy-releases: tried
[warn]   https://repo.typesafe.com/typesafe/ivy-releases/com.voxsupplychain/json-schema-codegen-sbt/scala_2.10/sbt_0.13/0.3.0/ivys/ivy.xml
[warn] ==== sbt-plugin-releases: tried
[warn]   https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.voxsupplychain/json-schema-codegen-sbt/scala_2.10/sbt_0.13/0.3.0/ivys/ivy.xml
[warn] ==== local: tried
[warn]   /Users/myLocalUserHome/.ivy2/local/com.voxsupplychain/json-schema-codegen-sbt/scala_2.10/sbt_0.13/0.3.0/ivys/ivy.xml
[warn] ==== jcenter: tried
[warn]   https://jcenter.bintray.com/com/voxsupplychain/json-schema-codegen-sbt_2.10_0.13/0.3.0/json-schema-codegen-sbt-0.3.0.pom
[warn] ==== public: tried
[warn]   https://repo1.maven.org/maven2/com/voxsupplychain/json-schema-codegen-sbt_2.10_0.13/0.3.0/json-schema-codegen-sbt-0.3.0.pom
[warn] ==== bintray-voxsupplychain-ivy-public: tried
[warn]   https://dl.bintray.com/voxsupplychain/ivy-public/com/voxsupplychain/json-schema-codegen-sbt_2.10_0.13/0.3.0/json-schema-codegen-sbt-0.3.0.pom
</SBT CLi output fragment>
Only the last URL is similar to the one I believe we're expecting (i.e. beginning with "https://dl.bintray.com...").
Could you have a look into this and give me a little advice, it would be great to make use of your library.
Thanks
Rich

Rich,
I have the following in my plugins.sbt:
resolvers ++= Seq(
Resolver.url("vox-public-ivy", url("http://dl.bintray.com/content/voxsupplychain/ivy-public"))(Resolver.ivyStylePatterns)
)
addSbtPlugin("com.voxsupplychain" %% "json-schema-codegen-sbt" % "0.3.0")
and it resolves. It seems plugin resolvers are different than normal ones.
Todor


Create test-file_arun.rst

This is a test file for learning.Please ignore
What changed, and why it matters

Thank you @supparun for participating in the session! We're looking forward to your contributions to the dev portal materials. Please let us know if you have any questions, feedback or need any help working with the articles.
I'm closing this test PR.


Support running the analysis with the SBOM without having to provide the repo path

Description
This ticket is created to track the enhancement for running the analysis given an SBOM.
At the moment, Macaron needs to have the git URL of the main repo or the config file as an entry point to the analysis. In some cases, we don't have the git URL to begin with or we cannot access the main target repository because it's behind authentication wall, etc.
In those cases, it's very important that we:

Could extract the necessary information about the main target from the given SBOM without having to provide the git URL beforehand.
If the main target cannot be accessed we reflect it in the reports and keep on analyzing the dependencies.

Blocking

[x] #95
[x] #157
[x] #162

We should break this into two PRs.

One for handling the case where the main target repo is not provided via -rp or -c, in which case we should still properly analyze the dependencies and the main target results (not available) should be reflected in the reports.
A separate PR can take care of the case where we need to resolve the main target repository through the SBOM itself.

Sounds good to me. I will create another 2 GH issues and link it to this main issue.

We can remove https://github.com/oracle/macaron/issues/158 from tasks in this issue and track that separately.

Thanks @behnazh-w I close this issue.


feat(guide): added java implementation to quicksort

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

@AhmedKhaledAK Can you make the indentation of the code 2 spaces?
Thank you.


Typo "@@SPID"→"@@SPID"

https://docs.microsoft.com/en-us/sql/t-sql/functions/spid-transact-sql?view=sql-server-ver15
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Merging based on related recent PR approvals; thanks @hyoshioka0128.


Fixed scope of translation file

Devise introduced a new method to find the scope of the local file.
The old behaviour was "devise.#{controller_name}" so it worked fine,
but they changed it to use whatever the parent define. In this case,
the scope changes from 'en.devise.checkga.user.error' to
'en.devise.sessions.user.error'.
See : https://github.com/plataformatec/devise/commit/c2fb80d4d9482cc9e923d2434556f2aadc4cbd3c

@aouji

Nice find


Pagination not working with search

Hi @bsbodden ,
I am trying to paginate data using EntityStream.getSlice() method, but it is not working with search. No query is printed in profiler.
It returns empty list in output.
Below is sample code:
entityStream.of(entityClass)
.filter(predicate)
.getSlice(PageRequest.of(pageOffset, pageLimit)
.collect(Collectors.toList()); // returns empty list
Is there any other way to paginate data ?

Example of pagination are:

https://github.com/redis/redis-om-spring/blob/main/redis-om-spring/src/test/java/com/redis/om/spring/search/stream/EntityStreamsAggregationsDocsPagingTest.java
https://github.com/redis/redis-om-spring/blob/main/redis-om-spring/src/test/java/com/redis/om/spring/search/stream/EntityStreamsAggregationsHashPagingTest.java

Hi @bsbodden,
I have used a simple predicate and model class.
I am trying to use getSlice method to provide pagination inputs for search(offset and limit). But after going through that method internally, it works only with cursor and not for ft search.
I am creating ft.search predicate using searchstream and want to paginate the data simply by providing Pageable as input. I am not able to find how to provide offset value using searchstream.
Also, the links you have provided uses ft.cursor read with aggregate queries, but is it possible to use offset directly for ft.search queries ?


Example usage-1.Train ahent

hello ,
When  I tried to
python rl_agent/scripts/train_scripts/train_ppo.py
It had error here:
Traceback (most recent call last):
File "rl_agent/scripts/train_scripts/train_ppo.py", line 13, in
from rl_agent.env_wrapper.ros_env_cont_img import RosEnvContImg
File "/home/hantewin/test/src/drl_local_planner_ros_stable_baselines-master/rl_agent/src/rl_agent/env_wrapper/ros_env_cont_img.py", line 19, in
from rl_agent.env_wrapper.ros_env_img import RosEnvImg
File "/home/hantewin/test/src/drl_local_planner_ros_stable_baselines-master/rl_agent/src/rl_agent/env_wrapper/ros_env_img.py", line 15, in
from rl_agent.env_wrapper.ros_env import RosEnvAbs
File "/home/hantewin/test/src/drl_local_planner_ros_stable_baselines-master/rl_agent/src/rl_agent/env_wrapper/ros_env.py", line 32, in
from rl_agent.env_utils.task_generator import TaskGenerator
File "/home/hantewin/test/src/drl_local_planner_ros_stable_baselines-master/rl_agent/src/rl_agent/env_utils/task_generator.py", line 29, in
from pedsim_srvs.srv import SpawnPeds
ImportError: cannot import name 'SpawnPeds'
Can you help me?  please,I don't know how to deal with it ,  I had chmod 777 als

@RGring

Seems that your pedsim is not setup correctly. Did you load the third-party repositories? https://github.com/RGring/drl_local_planner_ros_stable_baselines/blob/master/.rosinstall

yes,I downloaded it according to the version
but when I catkin_make the "flatland",I need to comment some code：
//Bkarso-----------------------------------
//if (target != m_targetA)         //comment here,I think that it doesn't matter;
{
m_bodyB->SetAwake(true);
m_targetA = target;
}
//Bkarso-----------------------------------
And when I run it : python rl_agent/scripts/train_scripts/train_ppo.py ;
It had some error :
(venv_p3) hantewin@hantewin-System-Product-Name:~/test/src/drl_local_planner_ros_stable_baselines-master$ python rl_agent/scripts/train_scripts/train_ppo.py
/home/hantewin/path_to_venv/venv_p3/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:526: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/hantewin/path_to_venv/venv_p3/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:527: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint8 = np.dtype([("quint8", np.uint8, 1)])
/home/hantewin/path_to_venv/venv_p3/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:528: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint16 = np.dtype([("qint16", np.int16, 1)])
/home/hantewin/path_to_venv/venv_p3/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:529: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint16 = np.dtype([("quint16", np.uint16, 1)])
/home/hantewin/path_to_venv/venv_p3/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:530: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint32 = np.dtype([("qint32", np.int32, 1)])
/home/hantewin/path_to_venv/venv_p3/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:535: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
np_resource = np.dtype([("resource", np.ubyte, 1)])
Traceback (most recent call last):
File "rl_agent/scripts/train_scripts/train_ppo.py", line 22, in
from stable_baselines.common.vec_env import VecNormalize, SubprocVecEnv, VecFrameStack
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/init.py", line 1, in
from stable_baselines.a2c import A2C
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/a2c/init.py", line 1, in
from stable_baselines.a2c.a2c import A2C
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/a2c/a2c.py", line 8, in
from stable_baselines import logger
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/logger.py", line 17, in
from stable_baselines.common.misc_util import mpi_rank_or_zero
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/common/init.py", line 9, in
from stable_baselines.common.cmd_util import make_vec_env
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/common/cmd_util.py", line 13, in
from stable_baselines.common.atari_wrappers import make_atari, wrap_deepmind
File "/home/hantewin/test/src/drl_local_planner_forks/stable-baselines/stable_baselines/common/atari_wrappers.py", line 6, in
import cv2  # pytype:disable=import-error
ImportError: /opt/ros/kinetic/lib/python2.7/dist-packages/cv2.so: undefined symbol: PyCObject_Type

help me please.thanks.
@RGring

I also have another download spencer_messages and spencer_tracking_rviz_plugin in pedsim_ros

how do you solve your problem?


Added option to avoid removing all tiles in GridLayer.redraw()

Enable external calls to GridLayer.redraw to trigger the tile update mechanism without first removing the tiles. Used for tile layers that can change over a small temporal period.

It looks like this commit changes the default redraw behavior.  Previously, it removed all tiles by default.  Now it only removes all tiles if you pass in true.  Was that the intention?

See #3642 - there might be similar use cases where redrawing all the tiles too fast might be problematic.

@colllin The intention was to leave the default behaviour the same. If removeTiles is undefined then it is set to true to keep the default behaviour. The new behaviour should only take effect when you change a call from removeTiles() to removeTiles(false).

@TolonUK OK, but what problem does this solve?

@IvanSanchez It's useful for tile layers that change over a short period of time and need to be frequently refreshed.
Without this change all the tiles in the layer are removed when the new tiles are requested; to the user this looks like the entire layer disappears for a fraction of a second until the new tiles are received.
With the change the old tiles stay in place until the new ones are actually received.

@TolonUK If that's the case, then your rapidly-changing custom layer can call GridLayer._update() instead. IMHO it's way cleaner than changing the default methods.

@IvanSanchez Ah, now I see. It seems a little bit obvious now looking at the implementation of redraw()... :) I'll try calling _update() instead of redraw() and see if I get the desired behaviour. Thanks.

It seems there's no need for this after all :-)

Yep, calling _update() does the trick for me. Thanks again!