InvalidArgumentError:  Incompatible shapes: [200,10] vs. [200,5] [[node LogicalAnd_3 (defined at :1) ]]

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Yes, I have written custom code.

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
I am running into this problem on Google Colab, which has Tensorflow 2.2.0

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
NA

TensorFlow installed from (source or binary):
Google Colab setup, Tensorflow version 2.2.0

TensorFlow version (use command below):
TensorFlow 2.2.0

Python version: Python 3.6.9

Bazel version (if compiling from source):

GCC/Compiler version (if compiling from source):

CUDA/cuDNN version:

GPU model and memory:

Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
While the model is getting trained, it randomly fails in any of the iteration of epoch with an Incompatible shape Error for LogicalAnd node, with the shapes differing as [200, 10] and [200, 5].
The difference in axis 1 here is actually : 2*Batch_size and Batch_size. So, if I change Batch_Size, the shapes change accordingly. Also, I'm not seeing this error on my local machine if I'm running this same code, Tensorflow version 2.2.0_rc1on a CPU. But, on Google Colab, I tried it both on CPU and GPU, and I face this error.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
Deep Network containing Pretrained Bert and  ResNet50V2 networks. The final layer outputs of both networks are concatenated and connected to a final Softmax layer of 2 neurons. There are 4 Input layers for complete network : 3 for BERT and 1 for ResNet50V2.
Here's the architecture code for network :
`
def complete_model():
input_word_ids = tf.keras.layers.Input(
(MAX_SEQUENCE_LENGTH,), dtype=tf.int32, name='input_word_ids')
input_masks = tf.keras.layers.Input(
(MAX_SEQUENCE_LENGTH,), dtype=tf.int32, name='input_masks')
input_segments = tf.keras.layers.Input(
(MAX_SEQUENCE_LENGTH,), dtype=tf.int32, name='input_segments')
img_input = tf.keras.layers.Input((256, 256, 3))

bert_layer = hub.KerasLayer(BERT_PATH, trainable=True)

_, sequence_output = bert_layer([input_word_ids, input_masks, input_segments])
bert_out = tf.keras.layers.GlobalAveragePooling1D()(sequence_output)

resnet_model = tf.keras.applications.ResNet50V2(include_top=False, weights='imagenet', input_tensor=img_input, pooling='avg')
resnet_out = resnet_model.output

x = tf.keras.layers.Concatenate()([bert_out, resnet_out])
x = tf.keras.layers.Dropout(0.4)(x)
out = tf.keras.layers.Dense(2, activation='softmax')(x)

model = tf.keras.models.Model(inputs=[img_input, input_word_ids, input_masks, input_segments], outputs=out)
return model

`
The model is created and then compiled and trained with the tf.keras.fit API. I created a tf.keras.Sequence object for generating inputs in batches in format :
<batch of images for ResNet50V2 of shape (256, 256, 3)>, <batch for Input IDs for BERT of shape (512)>, <batch for Attention Masks for BERT of shape (512)>, <batch for Segment IDs for BERT of shape (512)> and output <batch of class labels (batch_size, 2)>
Below is the Error Trace which I get :
Epoch 1/5
18/1700 [..............................] - ETA: 1:31:14 - loss: 1.6233 - accuracy: 0.5889 - auc: 0.5584
InvalidArgumentError                      Traceback (most recent call last)
 in ()
----> 1 hist = model.fit(train_dataseq, epochs=5, verbose=1, steps_per_epoch=8500//5, validation_data=valid_dataseq, validation_steps=500//5)
8 frames
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
64   def _method_wrapper(self, *args, **kwargs):
65     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
---> 66       return method(self, *args, **kwargs)
67
68     # Running inside run_distribute_coordinator already.
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing)
846                 batch_size=batch_size):
847               callbacks.on_train_batch_begin(step)
--> 848               tmp_logs = train_function(iterator)
849               # Catch OutOfRangeError for Datasets of unknown size.
850               # This blocks until the batch has finished executing.
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/def_function.py in call(self, *args, **kwds)
578         xla_context.Exit()
579     else:
--> 580       result = self._call(*args, **kwds)
581
582     if tracing_count == self._get_tracing_count():
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/def_function.py in _call(self, *args, **kwds)
609       # In this case we have created variables on the first call, so we run the
610       # defunned version which is guaranteed to never create variables.
--> 611       return self._stateless_fn(*args, **kwds)  # pylint: disable=not-callable
612     elif self._stateful_fn is not None:
613       # Release the lock early so that multiple threads can perform the call
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in call(self, *args, **kwargs)
2418     with self._lock:
2419       graph_function, args, kwargs = self._maybe_define_function(args, kwargs)
-> 2420     return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access
2421
2422   @property
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in _filtered_call(self, args, kwargs)
1663          if isinstance(t, (ops.Tensor,
1664                            resource_variable_ops.BaseResourceVariable))),
-> 1665         self.captured_inputs)
1666
1667   def _call_flat(self, args, captured_inputs, cancellation_manager=None):
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
1744       # No tape is watching; skip to running the function.
1745       return self._build_call_outputs(self._inference_function.call(
-> 1746           ctx, args, cancellation_manager=cancellation_manager))
1747     forward_backward = self._select_forward_and_backward_functions(
1748         args,
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/function.py in call(self, ctx, args, cancellation_manager)
596               inputs=args,
597               attrs=attrs,
--> 598               ctx=ctx)
599         else:
600           outputs = execute.execute_with_cancellation(
/usr/local/lib/python3.6/dist-packages/tensorflow/python/eager/execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
58     ctx.ensure_initialized()
59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
---> 60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
62     if name is not None:
InvalidArgumentError: 2 root error(s) found.
(0) Invalid argument:  Incompatible shapes: [200,10] vs. [200,5]
[[node LogicalAnd (defined at :1) ]]
[[assert_less_equal/Assert/AssertGuard/pivot_f/_2738/_167]]
(1) Invalid argument:  Incompatible shapes: [200,10] vs. [200,5]
[[node LogicalAnd (defined at :1) ]]
0 successful operations.
0 derived errors ignored. [Op:__inference_train_function_64050]
Function call stack:
train_function -> train_function

@anktplwl91
Can you please share simple stand alone code to replicate the issue faced, or share a colab gist for us to analyse the error. Also include your TensorFlow version.

Hi @Saduf2019
Here's my Github Gist
Tensorflow version used in Colab is : 2.2.0 GPU and CPU both (facing issue on both)

@anktplwl91
I ran the code shared and face this issue, please share all dependencies

Hi @Saduf2019
Here's the link to my colab

An update, I ran into this same issue on my local PC. But for some reason, the issue is seen in 5th epoch. I'm using same code there also, but using smaller version of BERT from Transformers repo
TensorFlow Version : 2.2.0_rc1 CPU
Python : 3.6.5
Error Logtrace :
Epoch 1/5
W0601 21:38:38.055769 17108 optimizer_v2.py:1180] Gradients do not exist for variables ['tf_bert_model/bert/pooler/dense/kernel:0', 'tf_bert_model/bert/pooler/dense/bias:0'] when minimizing the loss.
W0601 21:38:43.361550 17108 optimizer_v2.py:1180] Gradients do not exist for variables ['tf_bert_model/bert/pooler/dense/kernel:0', 'tf_bert_model/bert/pooler/dense/bias:0'] when minimizing the loss.
531/531 [==============================] - ETA: 0s - loss: 0.7343 - auc: 0.6148 - accuracy: 0.5972
Epoch 00001: saving model to model_checkpoint
531/531 [==============================] - 10835s 20s/step - loss: 0.7343 - auc: 0.6148 - accuracy: 0.5972 - val_loss: 148.2352 - val_auc: 0.5020 - val_accuracy: 0.5020
Epoch 2/5
531/531 [==============================] - ETA: 0s - loss: 0.6840 - auc: 0.6270 - accuracy: 0.6176
Epoch 00002: saving model to model_checkpoint
531/531 [==============================] - 9856s 19s/step - loss: 0.6840 - auc: 0.6270 - accuracy: 0.6176 - val_loss: 0.8900 - val_auc: 0.4815 - val_accuracy: 0.4960
Epoch 3/5
531/531 [==============================] - ETA: 0s - loss: 0.6715 - auc: 0.6392 - accuracy: 0.6228
Epoch 00003: saving model to model_checkpoint
531/531 [==============================] - 9444s 18s/step - loss: 0.6715 - auc: 0.6392 - accuracy: 0.6228 - val_loss: 0.7030 - val_auc: 0.4856 - val_accuracy: 0.4859
Epoch 4/5
531/531 [==============================] - ETA: 0s - loss: 0.6715 - auc: 0.6364 - accuracy: 0.6289
Epoch 00004: saving model to model_checkpoint
531/531 [==============================] - 13278s 25s/step - loss: 0.6715 - auc: 0.6364 - accuracy: 0.6289 - val_loss: 45.8197 - val_auc: 0.4939 - val_accuracy: 0.4980
Epoch 5/5
376/531 [====================>.........] - ETA: 48:52 - loss: 0.6696 - auc: 0.6402 - accuracy: 0.6330

InvalidArgumentError                      Traceback (most recent call last)
 in ()
----> 1 hist = transformer_model.fit(train_dataseq, epochs=5, verbose=1, steps_per_epoch=8500//16, validation_data=valid_dataseq, validation_steps=500//16, callbacks=[tf.keras.callbacks.ModelCheckpoint(filepath='model_checkpoint', save_weights_only=True, verbose=1, save_freq='epoch')])
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\keras\engine\training.py in _method_wrapper(self, *args, **kwargs)
63   def _method_wrapper(self, *args, **kwargs):
64     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
---> 65       return method(self, *args, **kwargs)
66
67     # Running inside run_distribute_coordinator already.
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\keras\engine\training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing, **kwargs)
781                 batch_size=batch_size):
782               callbacks.on_train_batch_begin(step)
--> 783               tmp_logs = train_function(iterator)
784               # Catch OutOfRangeError for Datasets of unknown size.
785               # This blocks until the batch has finished executing.
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\def_function.py in call(self, *args, **kwds)
578         xla_context.Exit()
579     else:
--> 580       result = self._call(*args, **kwds)
581
582     if tracing_count == self._get_tracing_count():
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\def_function.py in _call(self, *args, **kwds)
609       # In this case we have created variables on the first call, so we run the
610       # defunned version which is guaranteed to never create variables.
--> 611       return self._stateless_fn(*args, **kwds)  # pylint: disable=not-callable
612     elif self._stateful_fn is not None:
613       # Release the lock early so that multiple threads can perform the call
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\function.py in call(self, *args, **kwargs)
2418     with self._lock:
2419       graph_function, args, kwargs = self._maybe_define_function(args, kwargs)
-> 2420     return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access
2421
2422   @property
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\function.py in _filtered_call(self, args, kwargs)
1663          if isinstance(t, (ops.Tensor,
1664                            resource_variable_ops.BaseResourceVariable))),
-> 1665         self.captured_inputs)
1666
1667   def _call_flat(self, args, captured_inputs, cancellation_manager=None):
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
1744       # No tape is watching; skip to running the function.
1745       return self._build_call_outputs(self._inference_function.call(
-> 1746           ctx, args, cancellation_manager=cancellation_manager))
1747     forward_backward = self._select_forward_and_backward_functions(
1748         args,
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\function.py in call(self, ctx, args, cancellation_manager)
596               inputs=args,
597               attrs=attrs,
--> 598               ctx=ctx)
599         else:
600           outputs = execute.execute_with_cancellation(
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow\python\eager\execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
58     ctx.ensure_initialized()
59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
---> 60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
62     if name is not None:
InvalidArgumentError:  Incompatible shapes: [200,32] vs. [200,16]
[[node LogicalAnd (defined at :1) ]] [Op:__inference_train_function_24857]
Function call stack:
train_function

@anktplwl91
I ran the code shared by you, every cell has error in it, can you please share code that is complete with all dependencies so we could replicate it.

Hi @Saduf2019
Can you please download following files from here

img, bert-base-from-tfhub directories

train.jsonl, dev.jsonl, tfv2_tokenization.py and fb_meme_compeition.py files

Also, make sure you have tqdm and tensorflow_hub installed

This should run the code without any errors. Please let me know if you face anymore issues.
Thanks

Hi @Saduf2019 @jvishnuvardhan
I tried to run the model by removing metric tf.keras.metrics.AUC from model.compile, and have run the model for quite sometime now, around 10 epochs. I haven't seen this bug pop until now. Maybe the issue is coming from somehwere in AUC metrics' code

The code sample is fairly complex, and it would need to be simplified before we could reliably debug. Since it looks like you resolved the issue in any case, I will close this. If you find you can reproduce the problem with a straightforward snippet, please file a new issue or reopen.

I also faced the same issue and found that there was issue with my pooling layer. It was giving out incorrect output. Changing maxpool2d to globalaevaragepool solved the issue. If you want to stick to maxpool, then change it's output shape.


fix(MessageProxy): channel conflict

What does this PR do?
Every class that is registered to the MessageProxy system must have a static CHANNEL string property, which will be used to prefix every channel.
StudioAPI declared a function ping before the name of the method was used, which could lead to conflict with other registered method (InstructLabAPI), therefore we uses the CHANNEL as prefix, the channel become StudioAPI-ping.
Allowing scaling, and will avoid problem while working on instructlab api.
Screenshot / video of UI
N/A
What issues does this PR fix or reference?
Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/1837
How to test this PR?

[x] unit tests has been provided

Manually
You can ensure that nothing change by playing with this version of AI Lab

Can't we use the class name as channel ?

Sadly no, the problem is the following
The frontend do not have access to the StudioApiImpl class (file on backend), it only has access to the StudioAPI abstract class (file on shared). Therefore on the frontend, we don't know the string StudioApiImpl.

IMHO I think there should be some time where brainstorming should occur between between when issues are opened and PR being opened

Can't we use the class name as channel ?
@jeffmaury sadly no, the problem is the following
can we use the interface name instead ?

usually how it works on proxy is that you ask stuff for a given interface
client =
myProxy = Proxy(Interface)
server =
registerProxy(Interface, implementation)
so registering also two times the same interface name would conflict
I think the channel is an implementation details and should not be exposed

IMHO I think there should be some time where brainstorming

Movind to draft

I think the channel is an implementation details and should not be exposed

Fixed

and signature would be something like getProxy(proxyClass: {name: string})

Fixed

Moving to draft: abstract class do not seems to expose their abstract method through the .prototype property

After reading a bit about abstract class in Typescript, I found There is no code generated for an abstract method^1
Therefore, in the MessageProxy we are unabled at runtime to list the methods and create the corresponding channels.
https://github.com/containers/podman-desktop-extension-ai-lab/blob/5d6a3428ed534da6ddc7d2949dc921240984df19/packages/shared/src/messages/MessageProxy.ts#L108-L111
We are getting an empty array when providing StudioAPI, only a non-abstract class would allow us to get a non-empty array. This is why we need StudioApiImpl
We cannot transform the StudioAPI to an interface for technical reasons, interfaces do not exists at runtime in JS

This is the reasons why abstract class was used when implementing the channels system.
Hardcoded CHANNEL property
I still believe my solution with the const CHANNEL is the right one.

CHANNEL is in the abstract class, not per method and the .name property is on the abstract class as well
so I don't see the difference there as you need to provide the Abstract class both from client and server side

Sorry if I was not clear in the previous message.
As I show above

We can clearly see at runtime, you cannot access to the list of methods from an abstract class compiled to javascript.
Now let's dig into how frontend and backends are communicating.
Backend
The backends, register a class object, and using reflection, read the prototype of the object provided to list all the methods available.
Today (before this PR) we iterate over all methods and the backend register channels. Example ping channel is created from the ping method declared in StudioApiImpl.
https://github.com/containers/podman-desktop-extension-ai-lab/blob/ea02986c994f9eb60e587bcf7a35a309cde6c315/packages/shared/src/messages/MessageProxy.ts#L113-L116
Then when we received an event from the webview, we can directly call the right method.
https://github.com/containers/podman-desktop-extension-ai-lab/blob/ea02986c994f9eb60e587bcf7a35a309cde6c315/packages/shared/src/messages/MessageProxy.ts#L79-L85
Frontend
The frontend, do not create channels, it create a proxy object, and when you call anything on the proxy, it will transform the method you are calling into string, and be used as channel.
If I do studioClient.ping() the proxy handler will received a property call, named ping. With this information the proxy handler will call the channel ping that the backend will receive and makes it corresponds.
https://github.com/containers/podman-desktop-extension-ai-lab/blob/ea02986c994f9eb60e587bcf7a35a309cde6c315/packages/shared/src/messages/MessageProxy.ts#L178-L181
The name problem
The issue with the name property is that StudioApiImpl.name === 'StudioApiImpl' and StudioApi.name === 'StudioApi'.
Yes they both have a name property, but not the same.
Let's look at our options
registerInstance<T extends Record<keyof T, UnaryRPC>>(classType: new (...args: any[]) => T, instance: T): void

The signature of registerInstance (backend) needs the following arguments

classType is used to list the methods (need to be a real class not abstract, so we cannot use StudioApi).
instance: kept to call when needed

We could simplified the first argument by not using it to generate the channel, and use the instance. By doing some hacky stuff we can get the prototype (methods name) from the instance Object.getOwnPropertyNames(Object.getPrototypeOf(instance)), and then we would provide the interface at first argument.

New problem on the frontend side, we are minifiying the code, which result in class names not being consistent

The only possibility I see is:  Disabling the minify.

let's defer the discussion about how proxy should like (no abstract class but interfaces, etc) out of this PR
and merge the current solution that fixes an issue

merge the current solution that fixes an issue (with constant)

I revert the code to use the CHANNEL property


[DP] Update corpora-api.yml to include /dp/*

In order to support /dp/* and /cellxgene/* for api.cellxgene.cziscience.com we need:
cellxgene:

BrianM’s BE/FE base URL separation  — @bmccandless is preparing a PR

DP:
Update backend/config/corpora-api.yml  to prefix all routes with /dp/ also update API test cases accordingly. E.g., /v1/project-> /dp/v1/project
Single Cell Infra:
To address the concern Trent raised above, regarding API routes knowledge being spread across three repos (cellxgene, data-portal, and infra), Trent suggests that we at least include cellxgene and DP’s action item PRs in single-cell-infra’s PR that updates the API domain
What:
Implement DP#1

This is merged in main now!

PR: https://github.com/chanzuckerberg/corpora-data-portal/pull/594


Integration Test - Release - 2019/05/29 19:52:21

Auto create for integration test. IssueCommandFlow

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


Reorder README sections, minor README changes

Move "Apache Maven with a settings path" section to other Maven sections
Add link to "See below for using the settings-path"
Make empty lines in example workflow snippets consistent

Thank you for submitting these changes.
We have recently released actions/setup-java@v2 where reworked structure of documentation. If you think that your changes still make sense - could you please recreate PR and pull latest changes?


write module supporting modular arithmetic

The field operations (recip and (/.)) are now built-in and apply to Z n for prime n, so I think we can consider this closed.


#780 Implement Condition.Range

PR for #780

Job #801 is now in scope, role is REV

Codecov Report

Merging #801 into master will increase coverage by 0.04%.
The diff coverage is 92.3%.

@@             Coverage Diff              @@
##             master     #801      +/-   ##
============================================
+ Coverage     76.62%   76.66%   +0.04%
  Complexity      953      953
============================================
  Files           215      215
  Lines          4517     4530      +13
  Branches        359      361       +2
============================================
+ Hits           3461     3473      +12
  Misses          904      904
- Partials        152      153       +1

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/takes/misc/Condition.java
87.5% <92.3%> (+3.28%)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

This pull request #801 is assigned to @g4s8/z, here is why. The budget is 15 minutes, see §4. Please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket. If you decide that this PR should not be accepted ever, also inform the architect.

@t-izbassar method fits() has a side-effect, it's not clear and may produce unpredictable behavior, because each call to this method will change internal state of an object.
Also I don't think that it's a correct design to implement Range as a Condition type (because range is not a condition logically) and use it inside Select:
new Select<Integer>(
  Arrays.asList(3, 5, 7, 9, 11, 13, 15),
  new Condition.Range<Integer>(2, 4)
); // -> [7, 9, 11]

In my opinion Range should be Iterable and used like this:
new Range<Integer>(
  Arrays.asList(3, 5, 7, 9, 11, 13, 15),
  2,
  4
); // -> [7, 9, 11]

@yegor256 what do you think? Maybe we can use https://github.com/yegor256/cactoos here?

@g4s8 I agree about side-effect, but that's the only possible way to do this task. Range can be a condition because the condition is that the element fits given range. I don't see semantic misconception in that case. Also, this is what was requested by original issue:  #281.
Your suggestion is valid. However, it cannot be used with Select and cannot be combined with other conditions. Example:
new Select<Integer>(
  Arrays.asList(3, 5, 7, 9, 11, 13, 15),
  new Condition.Not<Integer>(
    new Condition.Range<Integer>(
      2,
      4
    )
  )
)

@g4s8 and can you clarify, what exactly an unpredictable behavior is? When Range object is created, it's leave with the state, and state changes. That's seems natural to an object. It's still immutable. Also, the usage of conditions are limited to Select, which checks the boundaries within Iterable.
Of course, if very same object will be used in two different selects, behavior will be questionable. However, what is your take on this issue? I don't see how I can possibly address that. Maybe, just mentioning in javadoc of the class will be useful?

@t-izbassar

can you clarify, what exactly an unpredictable behavior is?

I mean that Range.fits expected to be used only in Select because Select knows that each call to fits() will change internal counter. If Range is used in other classes, it will cause some maintenance issues, because each class which use Range must know how it implemented internally.

requested by original issue: #281.

This Range implementation was mentioned in a comment https://github.com/yegor256/takes/issues/281#issuecomment-101216901 (not in a task description), this is why I'm aksing @yegor256  to look at this changes. If this comment is fully valid, I'd accept this PR.

@g4s8 original task requested to have Condition.Skip. And skip is easy operation when you have Range. So this could be treated as being relevant part to what was requested originally. Also implementing Skip without Range will be harder. And it will have all the issues that you've already mentioned. It will have side-effect.
The whole purpose of having Condition is to use it in Select. The all test cases even placed in SelectTest. So I would not reject that as being coupled to Select usage - that's just design decision. If you think it is wrong, then you can submit issue or propose something else. But, with regards to implemented design, I believe this solution is valid.

@yegor256 ping

@g4s8 looks like we are reinventing what Cactoos already has, you are right... Maybe we should just let Takes have one dependency on Cactoos?

@yegor256 @g4s8 I can add this dependency in this PR and left puzzles to delete this Select and Condition staff. Will it be okay?

@t-izbassar I think you can keep this code, just add a @todo to replace Select and Condition with Cactoos analogs

@g4s8 done

@yegor256 ping

@yegor256 ping

@yegor256 can you merge this one?

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@t-izbassar @yegor256 Oops, I failed. You can see the full log here (spent 5min)
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 2964: running'
waiting for AppVeyor build 2964: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/2964
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 2964: running'
waiting for AppVeyor build 2964: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/2964
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 2964: running'
waiting for AppVeyor build 2964: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/2964
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 2964: running'
waiting for AppVeyor build 2964: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/2964
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 2964: running'
waiting for AppVeyor build 2964: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/2964
++ jq -r .build.status
+ status=failed
+ '[' failed == null ']'
+ '[' failed == success ']'
+ '[' failed == failed ']'
+ echo 'see https://ci.appveyor.com/project/yegor256/takes/build/2964'
see https://ci.appveyor.com/project/yegor256/takes/build/2964
+ exit 1
container 4fa33287c2712a2129cd0b2b6f54259864ea8b70acb7ee040db0e02a81c45f57 is dead
Mon Feb 19 20:25:51 CET 2018

@g4s8/z this job was assigned to you 5 days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@yegor256 hi, can you re-trigger this merge as well? Thanks.

@yegor256 ping for the merge

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 16min)

@elenavolokhova/z please review this job, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@elenavolokhova confirmed.

@t-izbassar Please, add some explanation of the solution proposed to the description section.

@g4s8/z this job was assigned to you 8 days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@elenavolokhova sure, I confirm

@0crat quality acceptable

@0crat quality acceptable (here)

@elenavolokhova There is an unrecoverable failure on my side. Please, submit it here:
com.zerocracy.farm.reactive.StkGroovy[125] java.lang.IllegalStateException: java.lang.reflect.InvocationTargetException in com/zerocracy/stk/pm/cost/make_payment.groovy
sun.reflect.NativeMethodAccessorImpl[-2] java.lang.reflect.InvocationTargetException: null
com.jcabi.xml.ListWrapper[162] com.jcabi.xml.ListWrapper$NodeNotFoundException: XPath '//*[@id='45.share']/text()' not found in '<?xml version="1.0" encoding="UTF-8"?><html xmlns=..35780..</script>\uA      </body>\uA   \uA</html>': Index (0) is out of bounds (size=0)

1.0-SNAPSHOT /cc @yegor256

QA review completed: +8 points just awarded to @elenavolokhova/z, total is +83

Order was finished, quality was "acceptable": +15 points just awarded to @g4s8/z, total is +4615

@0crat quality acceptable (here)

@elenavolokhova You can't do that, unless you have one of these roles: ARC, PO. Your current roles are: QA.

Payment to ARC for a closed pull request, as in §28: +10 points just awarded to @yegor256/z, total is +10824

This pull request #801 is assigned to @g4s8/z, here is why. The budget is 15 minutes, see §4. Please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket. If you decide that this PR should not be accepted ever, also inform the architect.

@0crat status

@0crat status (here)

@g4s8 This is what I know about this job, as in §32:

The job #801 is in scope for 16days
The role is REV
The job is assigned to @g4s8/z for 5days
There is a monetary reward attached
@g4s8/z will get 25% of their hourly rate for this job
The job doesn't have any impediments
The budget is 15 minutes/points
These users are banned and won't be assigned:

@t-izbassar/z: This user reported the ticket

Job footprint (restricted area)

@yegor256 could you please remove this job from WBS?

@yegor256 ping

Are you speaking to me or about me here. You must always start your message with my name if you want to address it to me, see §1.

The user @g4s8/z resigned from #801, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -15 points just awarded to @g4s8/z

@0crat out

@0crat out (here)

@yegor256 The job #801 is now out of scope


Update Hero.astro

#123

Hmm looks hard to align.
If it is possible, make the 2 pages like the Patrons page:

and same with creators image center in mobile

I think creators is perfect aligned now.

Yes in mobile @krishguptadev
this is how it looks in desktop

Hmm looks hard to align.
If it is possible, make the 2 pages like the Patrons page:
and same with creators image center in mobile

So I think it is best to copy the Patrons alignment except it's mobile view

Looks good for this issue #124.
Now the current issue is the navbar of Creators and Patrons.
The navbar of Developers page is good

as stated the navbar is evenly aligned based on vw. Can't do anything to it.

Hmm this still needs to be fix, we can't let it stay like that. I'll look into it later. U can merge this PR

Hmm this still needs to be fix, we can't let it stay like that. I'll look into it later. U can merge this PR

we can change the nav code to fix this...


Cannot use Chinese

check your font in D2RMH.ini, be sure to use a font that supports chinese characters

thanks


explain how to run the development/testing server

This adds the src/main/scripts/yumrepos command to the readme, explaining how to run a quick development or testing instance of yumrepos.

Coverage remained the same at 83.878% when pulling 7365ddeb567dd8e38586ac77b5bf485a21c8acd4 on luto:document-dev-server into 19ccf03b2108a259b909e82a924cc920c8cf2f7c on arnehilmann:master.


Can't established connection between two Android's devices

Hello!
I cloned the react-native-bluetooth-classic project and built BluetoothClassicExample for Android. The app was successfully installed on my Android device and I tried to connect it to my second Android device, but the connection failed with the message "Сonnection failed" and the code "EUNSPECIFIED".
If I connect to an iOS device, the connection is successfully established.
Before attempting to connect the devices, they were successfully paired together in the bluetooth menu in both cases.
Below I fully present the error I caught when trying to connect one Android device to another Android device.
{
  "framesToPop": 1,
  "nativeStackAndroid": [
    {
      "methodName": "onConnectionFailed",
      "lineNumber": 740,
      "file": "RNBluetoothClassicModule.java",
      "class": "kjd.reactnative.bluetooth.RNBluetoothClassicModule"
    },
    {
      "methodName": "connectionFailed",
      "lineNumber": 230,
      "file": "RNBluetoothClassicService. java",
      "class": "kjd.reactnative.bluetooth.RNBluetoothClassicService"
    },
    {
      "methodName": "access$300",
      "lineNumber": 28,
      "file": "RNBluetoothClassicService.java",
      "class": "kjd.reactnative.bluetooth.RNBluetoothCMassicService"
    },
    {
      "methodName": "run",
      "lineNumber": 320,
      "file": "RNBluetoothClassicService.java",
      "class": "kjd.reactnative.bluetooth.RNBluetoothClassicServiceSConnectThread"
    }
  ],
  "userInfo": {
    "class": 524,
    "id": "9C:28:40:9F:90:08",
    "address": "9C:28:40:9F:90:08",
    "name": "SecondAndroidDevice"
  },
  "message": "Connection unsuccessful",
  "code": "EUNSPECIFIED",
  "line": 18,
  "column": 1111,
  "sourceURL": "index.android.bundle"
}

I looked carefully at the library code and found that the error is thrown because of a failed execution of this line. But I'm not good at Java and I don't understand why this is happening.
What am I doing wrong? How do I connect one Android device to another Android device correctly?

Version
React Native
Android

0.10.5
0.61.5
8.1.0

Did you have the other Android device setup and waiting for a connection?  The line you posted
mmSocket.connect();
is the BluetoothSocket attempting to connect.  This requires that the other Android device has an active BluetoothServerSocket that is listening.  In order to test this you're going to need:

Two devices running BluetoothClassicExample
a) one device waiting for connection (click the button at the bottom)
b) one device attempting to connect to said connection

Can you confirm:

That you had the other device waiting for the bluetooth connection?
Which app were you using to open the bluetooth connection on the other device?

Without the other device waiting for a connection, you're going to get that error message.   The reason you don't get this on IOS is that IOS bluetooth is always accepting connections.  You don't need to place the app into a "listen" mode, as it's done by the system.   With that said, if you're actively sending data from IOS to Android, I'd love to see how your'e doing it - as it's not possible using this library (unless you magically have the Android MFi protocols for the device you're using).
Disclaimer - since this isn't the primary use case I can't guarantee when I'll get around to testing it.  This was provided by @tpettrov - I tested quickly using two BluetoothClassicExample apps running on separate devices and it seemed good.

@yeswell You can use Serial Bluetooth Terminal from play store to test connections.

Hey @yeswell, any update on your testing?


PLINK killing at mapping step when using --skip_prune, --gwas, and --p flags together

Please make sure that this is a bug.
System information:

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Catalina
GenoML Installed from (source or binary): Source
GenoML Version: v1-b6
Python Version: 3.7+

Describe the current behavior:
Currently, when using both the --skip_prune yes coupled with the --gwas and --p flags, PLINK kills the command at the "mapping stage". This is likely due to improper files being made
Describe the expected behavior:
Should keep running!
Code to reproduce the issue:
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
When munging, if you provide a GWAS file and a p-threshold while also choosing to skip the built-in pruning, you will see this error too
Other Information / Logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

This bug was caused by an incomplete PLINK command, resulting in the commands being killed during mapping. Now if all these options are chosen, proper temporary binary files are generated to keep moving forward


uninitialized constant Faker::Address::STATE

Got error message "uninitialized constant Faker::Address::STATE" when using Faker::Education.school in my rake task using ffaker version 1.12.1.

Thanks, fixes on 1.13.0


additional check to prevent error while checking ansible_selinux.status

Got error on Ubuntu 16.10:

FAILED! => {
"failed": true,
"msg": "The conditional check 'consul_is_ui and consul_install_nginx_config and ansible_selinux.status == "enabled"' failed. The error was: error while evaluating conditional (consul_is_ui and consul_install_nginx_config and ansible_selinux.status == "enabled"): 'bool object' has no attribute 'status'\n\nThe error appears to have been in '/home/suvitruf/ansible/roles/consul/tasks/install-ui.yml': line 79, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: allow nginx to connect to consul (selinux)\n  ^ here\n"

Aditional check ansible_selinux is defined prevents it.

Tested this locally too and works as intended.


Support all options for the Agent check command

Motivation
Feature parity with https://github.com/DataDog/datadog-agent/blob/master/cmd/agent/app/check.go

Note: the check command seems broken and it's not caught by CI. It'd be nice if it was reported somehow.


Added support for Azure China and Azure US Gov

Added support for Azure china and Azure US Gov

This will close https://github.com/Azure/aml-workspace/issues/18#issuecomment-650318246

confirmed that aml-workspace works for AzureUSGovernment -Thanks


Wait a configurable amount of time for benchmark channels to connect

Needed for an internal benchmark in which the server takes a while to become ready because of the time it takes for service discovery mechanisms to come up. This fixes the situation on that end.

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,976,356      Total (=)      1,976,356

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,770,407      Total (=)     10,770,407

 No significant differences in binary sizes

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,976,356      Total (=)      1,976,356

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,770,413      Total (>)     10,770,408

 No significant differences in binary sizes

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

How long does it usually take to become connected? If it is under a minute I think we should just increase the timeout, rather than add yet another configuration environment variable

At least from what I've seen so far, it's commonly around 2 minutes. What do you think?

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

Good point to document it. I've added a comment, but I'm having a thought about adding it to doc/environment-variables.md: it looks like that md file only currently contains environment variables that affect the C-Core production library - test code, C++, etc..... so I've added documentation to tools/run_tests/performance/README.md instead.
That look ok?

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,976,356      Total (=)      1,976,356

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,770,403      Total (<)     10,770,409

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,976,356      Total (=)      1,976,356

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,770,409      Total (<)     10,770,412

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,976,356      Total (=)      1,976,356

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,770,412      Total (>)     10,770,410

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,976,356      Total (=)      1,976,356

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,770,404      Total (<)     10,770,409

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

tests failures: TSAN C++ https://github.com/grpc/grpc/issues/16124
objc dbg and opt: https://github.com/grpc/grpc/issues/16156


fix: Use commit sha when there is no RC or tag available

hello team!
This is a small PR, it contains a change to print the commit sha when we are not running an RC or a version, This might be useful when we want to run specific commits, branches, etc... Otherwise the value will be empty.
This comes from after testing the new metric version, I can see this when we are not running tags:
cometbft_consensus_step_duration_seconds_bucket{chain_id="<whatever>"",step="Commit",version="",le="0.2682695795279726"} 153

Please team, if you approve it, merge it after, I don't have permission to do it.
Thanks in advance! 🚀
Jose Ramon Mañes

Codecov Report

Merging #2190 (c48a82a) into main (6c72c6d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2190   +/-   ##
=======================================
  Coverage   24.93%   24.93%
=======================================
  Files         127      127
  Lines       14309    14309
=======================================
  Hits         3568     3568
  Misses      10374    10374
  Partials      367      367


REMOVE default_attributes FROM TypographyP

BEFORE

AFTER

THE CODE
Tooltip(placement: "bottom") do
  TooltipTrigger do
    plain(helpers.lucide_icon("message-circle", class: "w-4 h-4 text-primary flex-none"))
  end

  TooltipContent(class: "flex flex-col gap-2 w-80 p-2") do
    TypographyP(weight: :medium) { "Comentário" }
    TypographyP { @supplier.approval_justification }

    div(class: "flex flex-row items-center gap-2") do
      plain(helpers.lucide_icon("user", size: 20, class: "text-slate-700"))
      TypographyP { @supplier.approval_user_email }
    end
  end
end

@pierry01 ,
This is exactly the same code shadcn has:
export function TypographyP() {
  return (

      The king, seeing how much happier his subjects were, realized the error of
      his ways and repealed the joke tax.

  )
}

https://ui.shadcn.com/docs/components/typography#p

@pierry01 ,
This is exactly the same code shadcn has:
export function TypographyP() {
  return (

      The king, seeing how much happier his subjects were, realized the error of
      his ways and repealed the joke tax.

  )
}

https://ui.shadcn.com/docs/components/typography#p

I have opened an issue in shadcn-ui to FIX that
https://github.com/shadcn-ui/ui/pull/4889


minecraft: Launcher is stuck at "Preparing..."

Describe the bug
Upon pressing the "Play" button in the Minecraft launcher, the button is then stuck in the "Preparing..." phase.
To Reproduce
Steps to reproduce the behavior:

Launch the launcher
Log in if you haven't logged in yet
Press Play

Additional context
This is what is shown in the log:
[03:14:30 FATAL]: Unhandled exception in thread Thread[pool-2-thread-2,5,main]
com.google.gson.JsonSyntaxException: duplicate key: null
	at com.google.gson.internal.bind.MapTypeAdapterFactory$Adapter.read(MapTypeAdapterFactory.java:191) ~[minecraft.jar:?]
	at com.google.gson.internal.bind.MapTypeAdapterFactory$Adapter.read(MapTypeAdapterFactory.java:146) ~[minecraft.jar:?]
	at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$1.read(ReflectiveTypeAdapterFactory.java:93) ~[minecraft.jar:?]
	at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.read(ReflectiveTypeAdapterFactory.java:172) ~[minecraft.jar:?]
	at com.google.gson.Gson.fromJson(Gson.java:795) ~[minecraft.jar:?]
	at com.google.gson.Gson.fromJson(Gson.java:761) ~[minecraft.jar:?]
	at com.google.gson.Gson.fromJson(Gson.java:710) ~[minecraft.jar:?]
	at com.google.gson.Gson.fromJson(Gson.java:682) ~[minecraft.jar:?]
	at net.minecraft.launcher.updater.RemoteVersionList.getCompleteVersion(RemoteVersionList.java:32) ~[launcher.jar:1.6.89-j]
	at net.minecraft.launcher.updater.MinecraftVersionManager.getLatestCompleteVersion(MinecraftVersionManager.java:213) ~[launcher.jar:1.6.89-j]
	at net.minecraft.launcher.updater.MinecraftVersionManager.getLatestCompleteVersion(MinecraftVersionManager.java:37) ~[launcher.jar:1.6.89-j]
	at com.mojang.launcher.game.runner.AbstractGameRunner.playGame(AbstractGameRunner.java:67) ~[launcher.jar:1.6.89-j]
	at net.minecraft.launcher.game.GameLaunchDispatcher$1.run(GameLaunchDispatcher.java:111) ~[launcher.jar:1.6.89-j]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) ~[?:1.8.0_212]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) ~[?:1.8.0_212]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_212]

From what I have seen on the Internet, this happens when the launcher is outdated, so there is that. I will tag @Infinisil, since he was the last one who fixed an issue related to the launcher.
EDIT: might be a good time to mention that I am nearly sure this isn't an issue with my computer. Just a few hours ago, I was playing Minecraft just fine. Now it's giving me this error, without generation changes nor reboots nor whatever. I even tried deleting the entire Minecraft folder and rebooted to get a fresh install, but it's still happening.

Yeah I can't launch either, and it doesn't seem to be a NixOS-specific problem, there's many more people having the same problem: https://www.minecraftforum.net/forums/support/java-edition-support/2985294-minecraft-launcher-stuck-on-preparing, also a bug report at https://bugs.mojang.com/browse/MCL-11990 (which was closed with Invalid??)
Anyways from the thread there it seems to indicate we finally need to update our launcher. The one we use in NixOS is like at least 4 years old, even though a new one has been release for a while.

The provided .deb file includes the icon but I can't find a URL that would point to a fixed version of it (which is available for the general package, but that does not include the icons) - we can't add copyrighted material to the repo or have the build fail because a new version is released and the hash doesn't match anymore, and I would like to not have to pull the AUR repo.
cc @peterix

I previously sent unpolished derivations for both the new Minecraft launcher and a launcher for running Minecraft-PE to @Infinisil .
Here's a link to an archive for the derivations: https://0x0.st/zLAg.gz (Note that it's is a .tar.gz despite the filename)
If you are curious about the Minecraft-PE launcher it essentially uses the x86 version of Minecraft-PE so that you can run it. This is useful if you want to use the Bedrock version of the game. If you are unaware there are two main ecosystems for Minecraft, Java Edition (the original) and Bedrock Edition (multiplatform W10, Android, iOS, switch, xbox one, etc).

A bit off-topic - MultiMC launcher seems to work fine (and I consider it a better launcher :D).

Right. This is interesting.
If you are making binary packages and putting them on a repo, you are technically breaking copyright law. So, that flatpak thing? Illegal. Avoid it. I just don't have the heart to forward it to enforcement.
We could totally put the icon somewhere so you can grab it and use it.

@peterix Perhaps you meant to comment in https://github.com/NixOS/nixpkgs/pull/68175/? I'm just noticing that while nixpkgs doesn't use the Flatpak thing, we still had the intention of providing a prebuilt package for the launcher. Because of a problem however, this never actually worked. I'll make sure that in above PR we switch from unfreeRedistributable as a license to unfree such that no altered version will ever be provided by the build servers, so users will continue to have to let their machine patch it locally.
Having the icon at a fixed url would be great yeah (preferably a versioned url in case it ever changes), or even included in https://launcher.mojang.com/download/linux/x86_64/minecraft-launcher_${version}.tar.gz (probably won't work well because of its structure though).
Nice to have someone from the Mojang team here :D

Having the .deb fetchable at a fixed url would be great also.

@peterix 🏓


Inline script in index.html

Any particular reason it's not in an external file? I'm assuming it's related to performance? Or habit? :wink:

It's for both. I try to keep external files limited to 5 (very old habit) and I find that browser caching is less of an issue when I write and test code on the main page before moving it off the landing page (testing performance). From what I have seen, there are no problems keeping a minimal set of JavaScript on the main page that points elsewhere, and do prefer the pattern.

Limiting the files is definitely a good idea, my main criticism with inline (anything really) is that it's hard to debug during the development, although I've found caching a frustration it is outweigh by easy debugging.
I think I'm still thinking in terms of when it comes to put it live bundling and minifying whatever I can which will improve performance. Source maps help to debug when using those files when it is live.
I'll leave it where it is.


Symbols don't originally appear in the chemistry editor on Safari

The symbols in the chemistry equation editor don't appear when you first open it. It is only once you have clicked on all of the tabs, that the symbols actually appear. The numbers are there from the start. This only happens the very first time you load the chemistry equation editor.
chem editor.pdf

This appears to be a z-index issue, that somehow resolves itself when the 4th tab is opened. Weird that it should only affect Safari!


Tooltip Accessibility

Based on https://www.w3.org/TR/wai-aria-practices/#tooltip it should

never receives focus. Focus stays on the textbox.
be shown via keyboard focus or by the onmouse event.
be hidden by removing focus from the text box or by moving the mouse off the textbox.
be hidden by pressing the Escape key (bootstrap currently doesn't do this)

Also,

The element that serves as the tooltip container has role tooltip.
The element that triggers the tooltip references the tooltip element with aria-describedby. (not sure if this is realistic or possible, we probably depend on the developer using the component to add this, but it would be nice if it happened automatically while keeping the current API for tooltips which allows them to be disjoint from their trigger)

Not 100% sure if Popover is more like a tooltip of a dialog/modal of sorts. We definitely don't want mouse events other than click opening/closing it...


Copying file to storage bucket always times out/HTTP/HTTPS proxy setting

I am trying to copy a file to storage bucket, but the operation always times out.
I tried increasing various time out parameters but its not helping.
File size is less than 150kb.
Please refer the below code which times out.
storage.list() method times out.
`private void uploadFile(String bucketName, String content) {
    HttpTransportOptions transportOptions = StorageOptions.getDefaultHttpTransportOptions();
    transportOptions = transportOptions.toBuilder().setConnectTimeout(60000).setReadTimeout(60000)
            .build();
    StorageOptions storageOptions = StorageOptions.newBuilder()
            .setRetrySettings(retrySettings())
            .setTransportOptions(transportOptions)
            .build();
    Storage storage = storageOptions.getService();

    Page<Bucket> buckets = storage.list(BucketListOption.pageSize(10), BucketListOption.prefix(bucketName));
    for (Bucket bucket : buckets.iterateAll()) {
        InputStream inputStream = new ByteArrayInputStream(content.getBytes());
        bucket.create("file_name", inputStream, "text/plain");
        break;
    }
}
private RetrySettings retrySettings() {
    return RetrySettings.newBuilder().setMaxAttempts(10)
            .setMaxRetryDelay(Duration.ofMillis(30000L))
            .setTotalTimeout(Duration.ofMillis(120000L))
            .setInitialRetryDelay(Duration.ofMillis(250L))
            .setRetryDelayMultiplier(1.0)
            .setInitialRpcTimeout(Duration.ofMillis(120000L))
            .setRpcTimeoutMultiplier(1.0)
            .setMaxRpcTimeout(Duration.ofMillis(120000L))
            .build();
}`

Error stack:
12:12:27.410 [pool-3-thread-1] ERROR org.springframework.scheduling.support.TaskUtils$LoggingErrorHandler [] - Unexpected error occurred in scheduled task.
com.google.cloud.storage.StorageException: connect timed out
at com.google.cloud.storage.spi.v1.HttpStorageRpc.translate(HttpStorageRpc.java:219) ~[google-cloud-storage-1.19.0.jar!/:1.19.0]
at com.google.cloud.storage.spi.v1.HttpStorageRpc.list(HttpStorageRpc.java:314) ~[google-cloud-storage-1.19.0.jar!/:1.19.0]
at com.google.cloud.storage.StorageImpl$6.call(StorageImpl.java:272) ~[google-cloud-storage-1.19.0.jar!/:1.19.0]
at com.google.cloud.storage.StorageImpl$6.call(StorageImpl.java:269) ~[google-cloud-storage-1.19.0.jar!/:1.19.0]
at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:89) ~[gax-1.19.0.jar!/:1.19.0]
at com.google.cloud.RetryHelper.run(RetryHelper.java:74) ~[google-cloud-core-1.19.0.jar!/:1.19.0]
at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:51) ~[google-cloud-core-1.19.0.jar!/:1.19.0]
at com.google.cloud.storage.StorageImpl.listBuckets(StorageImpl.java:268) ~[google-cloud-storage-1.19.0.jar!/:1.19.0]
at com.google.cloud.storage.StorageImpl.list(StorageImpl.java:257) ~[google-cloud-storage-1.19.0.jar!/:1.19.0]

After setting up the http/https proxy in the application, it works out.
At the start of java application, setting below system properties solves this problem.
    `System.setProperty("http.proxyHost", "proxy-host-name");
    System.setProperty("http.proxyPort", "proxy-host-port");
    System.setProperty("https.proxyHost", "proxy-host-name");
    System.setProperty("https.proxyPort", "proxy-host-port");`

Hi @pgbhagat thanks for the letting us know. I wasn't able to repro this issue, so may I ask about your environment settings? (Are you running on any google cloud machines or are you running locally?)

I was running my program behind the firewall, and setting those http proxy settings in application solves the problem. Closing this issue now.


Implementation of Discretization B

Dear Gu, I checked your code scan_ref for the implementation of S6. In line 126 I saw you simply use deltaB for the discretization, instead of the ZOH method in your paper. I wonder if there is anything I miss in this code? Or this way (deltaB ) is more useful?

It makes the implementation slightly simpler without affecting empirical performance. One can think of it as a mix of ZOH for A and Euler discretization for B.

Thank you  for the answer!


host_filter task unit tests (auto-generated)

Using this script which runs the workflow to completion on given test inputs, then records all the intermediate inputs & outputs and generates a pytest case for each task, which runs the task individually and then compares its actual & expected outputs. A bunch of pytest fixtures minimize the amount of boilerplate required in each test case. The script functionality is pretty interesting and might eventually make its way into a top-tier miniwdl tool (needs further generalization).
The current host_filter cases are derived from one of the synthetic inputs from the benchmarking paper ("bench3"). More can be slotted in later. Current warts:

6/9 cases are succeeding in GitHub Actions currently, the remaining 3 pending on public access to idseq-database S3 assets.
The lack of world-readability for the GitHub Packages docker registry (even for public repos) will remain an annoying roadbump for use elsewhere.
A woeful hack in RunTrimmomatic was needed to get it to use an adapter_fasta not sourced from S3 (in addition to https://github.com/chanzuckerberg/idseq-dag/pull/308 and motivating https://github.com/chanzuckerberg/idseq-dag/pull/314)

To see what's going on it's probably easier to browse the tree than the PR diff: https://github.com/chanzuckerberg/idseq-workflows/tree/mlin-generate-task-tests/tests/host_filter/tasks

Now with README: https://github.com/chanzuckerberg/idseq-workflows/tree/mlin-generate-task-tests/tests
Marked 3 cases as expected fail (xfail) while we work through how to adapt the S3 interactions.

@kislyuk @katrinakalantar @morsecodist any feedback on whether this seems like a feasible skeleton for adding more test cases+assertions? (Start from the readme and task cases directory as the diff view isn't navigable)

@mlin the test structure and fixtures/helpers look good. Do you have any guidance for finishing test coverage for all the steps?

I agree that the structure looks good!


[DE] Add german definition for canary deployment

Signed-off-by: Baris Cubukcuoglu<EMAIL_ADDRESS>Describe your changes
This PR adds the german definition for Canary Deployment according to Canary Deployment
Related issue number or link
Resolves #512
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO.

@iamNoah1, deine Anmerkungen habe ich nun eingearbeitet. Die Commits habe ge-squashed. Es wäre Ready zum Mergen.


less rounded corners?

less rounded corners pls
or an option to do so

There is an option to do so. In the app's context menu


Update versions of lint libraries

What this PR does / why we need it?

update package.json.

remove node-sass, add sass
update versions of typescript and eslint

modify eslint setting
need to discuss with type error (Wip)

Any background context you want to provide?
I've uploaded issue with type on #196
Do I roll back typescript to previous version for the backward compatibility or just add type file for navigator?
What are the relevant tickets?
Working on #196
Checklist

[ ] Added relevant tests or not required
[ ] Didn't break anything

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

I've pulled updated code and downgraded version of typescript.

CI build fails because sass is not included in the dependencies. I think this problem will be solved when #202 is merged into master.

@blurfx I've rolled back the code. Sorry for the late response.

@gollumnima I think I missed something. Why did you commented these codes before? to remove or just temporary?

@blurfx Because when I updated the version of typescript, the commented code made type error. So I just rolled back to the previous version. At the time, I was considering whether updating the version of typescript and making d.ts file to fix type error or just removing this code. I thought I had left a note for this somewhere, but I cannot find any note🥲

@blurfx Ah I found this link. Then can I delete the commented code here?

@gollumnima Sorry for late response. I think we can drop old browser support through remove these commented codes.

@blurfx In the local environment, the build succeeded but ci/build failed for this lint issue. Can I just add dependencies in useEffect logic? I'm afraid that my work might affect the result of the whole code.

@gollumnima I think some empty dependencies are intended. How about changing the eslint react-hooks/exhaustive-deps rule to warn?

@blurfx I think our ci rule is too strict for the lint issue 🥲.

The warning was treated as an error when the action runner set process.env.CI to true.
Now we have many ways to solve this problem.

Turn off the react-hooks/exhaustive-deps rule
Add eslint-ignore comments to all code where errors occur
Modify hook dependency
…

@blurfx Finally...!

Great work 😄 Thank you for your contribution!


electron thermal conductivity during 2T state

Hi,
After femtosecond laser excitation of metals is it assumed that the electronic thermal conductivity (k_el) varies in the way:
k_el = k_0*(Te/Ti),
where k_0 - is the initial electronic thermal conductivity, Te - electronic temperature, Ti - lattice temperature.
I tried to modify this parameter in the code in the way:
k_el_Au     = lambda Te, Ti: (Te/Ti)317u.W/(u.m*u.K);    #Heat conductivity
But an error occurs, am I doing somethin wrong or the code does not support such modification?
Only one parameter dependence works:
k_el_Au = lambda Te: (Te/300)317u.W/(u.mu.K);  #Heat conductivity
But in this case the Ti always equals only to 300 K (initial temperature of lattice), however the lattice normally heats up during the first several ps.

Hi Serj, I did not know these type of model was used.
In the current version of the code, the conductivity of a given system only depends on the temperature of the same system. I am afraid you can not simulate the model you need.
Luckyly, I am currently working on an update of the code and I would like to upload it in the second half of June. The update will surely include this feature.
Please, let me know more about the model you need to simulate:

Is this "multiple dependence" used also in 3 temperature model? Does the conductivity depends also on the spin temperature?
Is this used also for the specific heat? May it depend on other systems temperatures too?

Please, notify us any issue or success of the code in reproducing experimental data, as this is fundamental for implementation of future updates.
Thank you for using our code :)

Hello,
For example, in the attached file in the APPENDIX B: THERMAL CONDUCTIVITY OF Ni AND Au UNDER CONDITIONS OF ELECTRON-LATTICE NONEQUILIBRIUM you will find several dependences of electron conductivity during 2T state on the temperatures of electrons and lattice for Au. I think, the equation (B1) is the most frequently used one for Au. You can see that electron conductivity always depends both on Te and Ti, but this dependence significantly changes with increasing Te. So for Au, in the temperature range of electrons 330 - 15000K one should use the dependence (B1). The shown graphs of course are valid only for Ti = 300 K, with increasing Ti the electron conductivity depndence will change, so all this is discribed by the B1 equation.
Therefore, the possibility to model the electron conductivity dependence on Te and Ti (B1-equation in the paper) in your code would be very very helpful.
PRB03_meltNiAu.pdf


[TF2] Stats not changing

I just tried this and it didn't work.
I had 617 kills before and now i have the same kills (I got the succesfull message so yeha, it should have worked)

Show your config (Censor account username and password first)

Show your config (Censor account username and password first)

{
"boostingAccount": {
"username": "MainAccount",
"password": "MainPassword"
},
"botAccount": {
"username": "BotAccount",
"password": "BotPassword"
},
"itemID": "10032097344",
"appID": 440,
"eventType": 0,
"incrementValue": 500,
"repeat": 1
}

Try repeat 500 and incrementValue 1. The eventType you are using 0 (Kills) does not support incrementt.

I had to use null and not 1 in incrementValue in conjunction with repeat: "desired number". It kinda makes sense because kills already count as 1, so there's no need to tell the game to add 1 kill again.

I have this same issue, I tried setting it to "null" but it still doesn't work for me


showBrowserColorPicker not working on safari and edge

Jodit Version:  3.2.xxxxx
Browser:
OS:
Code
// A *self-contained* demonstration of the problem follows...

Expected behavior:
Actual behavior:
i am setting in the code with pram showbrowsercolorpicker = true, but  color picker not show in safari and edge, please help me to check it. thank you so much. in internet exploere sometime working sometime not ... :((

Enable showBrowserColorPicker - will show native browser color picker, but unfortunately, some browser does not support it. I will try to create a plugin for this but now you can attach some another color picker plugin.
For example something like this plugin https://narsenico.github.io/a-color-picker/
Added example for you https://xdan.github.io/jodit/examples/color-picker.html

thank you so much 😘
Vào Th 7, 20 thg 6, 2020 vào lúc 05:49 Valeriy Chupurnov <
<EMAIL_ADDRESS>đã viết:

Closed #368 https://github.com/xdan/jodit/issues/368.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xdan/jodit/issues/368#event-3464075511, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOZET23NYRY5WIE7D7HFRFTRXPTRHANCNFSM4LE46GVA
.

but here i can not distinguish "color" or "background" tabs
Vào Th 5, 9 thg 7, 2020 vào lúc 16:29 Đoàn Hữu Trung <
<EMAIL_ADDRESS>đã viết:

thank you so much 😘
Vào Th 7, 20 thg 6, 2020 vào lúc 05:49 Valeriy Chupurnov <
<EMAIL_ADDRESS>đã viết:

Closed #368 https://github.com/xdan/jodit/issues/368.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xdan/jodit/issues/368#event-3464075511, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOZET23NYRY5WIE7D7HFRFTRXPTRHANCNFSM4LE46GVA
.


Make the lib compatible with last PHPSpec version

Which requires to:

use PHP 7.1
in order to use typed function returns in Maintainer class
fixes #1

Please publish this fix! I need this! :D
Thanks!


[Discuss] Enable user access to detailed routing information

What version of V2Ray are you using?
4.27.0
What's your scenario of using V2Ray?
Build a custom geosite/geoip file.
What did you see?
The demand is brought up in following issues (and more):
v2ray/discussion/issues/770

减少dns查询的话，我的方法是每次启动前都会读取上一次运行的acce.log，其中有这样
2020/07/04 04:18:17 tcp:<IP_ADDRESS>:41298 accepted tcp:api.vc.bilibili.com:443 [drOut]
2020/07/04 04:18:43 tcp:<IP_ADDRESS>:41313 accepted tcp:google.com:443 [giaOut]

通过自动提取giaOut行中的域名字段，合并到之前的域名文件中，重新生成dat文件，再启动v2ray。
//结合一下这个路由
{
    "type": "field",
    "outboundTag": "giaOut",
    "domain": [
        "ext:proxy.dat:giaOut"
    ]
}

这样之前代理过的网站就不会再查询dns了呢。相当个人定制个域名库

v2ray/discussion/issues/592

"log": {
"loglevel": "debug",
"access": "access.log"
},
cat access.log |grep proxy | awk '{print $5}'
sort uniq....

v2ray/discussion/issues/804

希望通过v2ray透明代理（v2ray在网关上）来看美国hbo/netfilx等美剧（有AirPlay/> chromcast之类的所以只能做全局代理）
v2ray在网关上配合iptables，使用了
"geosite:hbo",
"ext:h2y.dat:hbo",
等的数据库来决定走不走代理，但是感觉不够全，经常会出现错误，而如果在手机上用SR全部数据走代理的时候则不会出错。
请问有没有额外的数据库，或者简易点的爬相关视频网站ip的工具？
谢谢！

In a word, there are demands and practices to build a custom geosite/geoip file according to the routing information of V2Ray.
In referenced issues, outbound tag is used to classify domestic and oversea sites and to augment a custom geosite file when not included in standard ones.With routing context, there's much more that could also be done:

With trasparent proxy, we may track the source IP of devices like Nitendo Switch, so as to retrieve actual domains it uses and IPs of game servers.
With tools like proxychains, we can detour network of a specific application to a specific V2Ray inbound, then by tracking the inbound tag we are able to retrieve all the related domains and ips.

... and many of these information could be finally contributed to v2fly/domain-list-community.
What's your expectation?
Currently, usage of routing information goes around access.log, which provides following information:

Source IP
Srouce Port
Network
Target IP / Target Domain
Target Port
Outbound Tag

But there are some problems with access log:

Fields like inbound tag are absent.
Balancers not tracked, only the final outbound is reported.
Sniffed domains won't be used in log. Only proxy requests that directly send domain to inbound (e.g. http and socks) can get it shown in log. So transparent proxy with dokodemo-door inbound will result in an access log without any domain:2020/08/19 00:20:17 <IP_ADDRESS>:55724 accepted tcp:<IP_ADDRESS>:443 [direct]
2020/08/19 00:20:17 <IP_ADDRESS>:55423 accepted udp:<IP_ADDRESS>:53 [dns-out]
2020/08/19 00:20:18 <IP_ADDRESS>:55386 accepted tcp:<IP_ADDRESS>:443 [proxy]

Usage of access log usually involves reading of on-disk log file (in stdout access and error log often get messed up), which poses potential privacy problem.

Considering above arguments, I expect to provide users a proper way of access to complete routing context, as well as keeping such access to restricted usage.

@kslr @Robot-DaneelOlivaw
Currently, two APIs related to routing is proposed:

Subscribe routing stats
Test routing stats

Regarding how to expose these services to user, two styles of config are proposed:
Reuse StatsService
JSON config:
"stats": {
  "routing": {
    "enabled": true
  }
}
"api": {
  "services": ["StatsService"]
},

All the service code is implemented in app/stats/command.
Use new RoutingService
JSON config:
···
"api": {
"services": ["RoutingService"] // Or "RouterService"?
},
···
All the service code is implemented in a new package app/router/command.
Which one do you prefer?

I think it's better to use it as a new Service.

I'm not sure if I have a say in this since I've never used statistics information nor understood potential influence between different implementations. Currently, there's no parameter in StatsObject. Reusing StatsService might be easier to understand semantically without leaving "stats": empty.
But I'll agree with anyone who has a different thought. And as always, appreciate your work.

Considering reusing StatsService, there're some perspectives to be discussed.
Code Organization
For now, there are two routing APIs to be put in StatsService:

Subscribe to Routing Stats
Manual Route Test

If we were to support new APIs for DNS, there are also two I could come up:

Subscribe to DNS Lookup Records
Manual DNS Lookup Order Test.

Grouping all these APIs altogether results in StatsService's inflation. Apart from inflation, Manual Route Test and Manual DNS Lookup Order Test cannot be really regarded as service about statistics.
If one could accept Manual Route Test as to get routing stats manually so it is a stats API, then imagine providing APIs for an outbound health check app, we may have:

Subscribe to Outbound Health Check Result
Manually feed a URL to health check an outbound

The latter API then cannot be recognized as stats API from my perspective.

Code Implementation
It should be noted that "stats" and "services": ["StatsService"] correspond to different modules in the core:

"stats": app/stats/stats.go: Manager
"StatsService": app/stats/command/command: statsServer

And from command.statsServer it cannot retrieve the config of stats.Manager. By setting
"stats": {
  "routing": {
    "enabled": true
  }
},

, stats.Manager is controlling the registration of routing stats channel to indirectly control whether statsService supports subscribing routing stats.
This does not apply to Manual Route Test. There is no way for stats.Manger to prevent statsService from requiring Router feature. So once "StatsService" is written in JSON config, along with other stats API, Manual Route Test will also be open to user by default and cannot be closed.
By adopting "RoutingService", API division is more fine-grained, we can use this service to control whether to open API for manual route test.

@Loyalsoldier @Robot-DaneelOlivaw
After writing above discussion, I come up with a new proposal to utilize both of your ideas.
To achieve finer granularity, we may use the new "RoutingService", but also keep the semantic of stats config:
"stats": {
  "routing": {
    "enabled": true
  }
},
"api": {
  "services": ["RoutingService"]
},

Here, two routing APIs belongs to "RoutingService", but in order to subscribe to routing stats, user need to register the channel in "stats" config, so v2ray-core will allocate resources to listening to routing statistics. Other services like DNS could also follow this pattern, by grouping them in "stats" config user can make sure v2ray-core will not take extra overhead on recording these statistics.


Can't get ParseFacebookUtilsV4 working with cocoapods and Xcode 7

I have the following in my Podfile:
pod 'ParseFacebookUtilsV4'
pod 'Parse'
pod 'ParseUI'
pod 'ParseCrashReporting'
After pod update and build the project (xcode 7, swift 2) I get the following errors:
/Pods/FBSDKLoginKit/FBSDKLoginKit/FBSDKLoginKit/FBSDKLoginButton.h:21:9: Include of non-modular header inside framework module 'FBSDKLoginKit.FBSDKLoginButton'
/Pods/FBSDKLoginKit/FBSDKLoginKit/FBSDKLoginKit/FBSDKLoginConstants.h:21:9: Include of non-modular header inside framework module 'FBSDKLoginKit.FBSDKLoginConstants'
/Pods/ParseFacebookUtilsV4/ParseFacebookUtils/Internal/PFFacebookAuthenticationProvider.h:12:9: Could not build module 'FBSDKLoginKit'
If I add this  to the Podfile:
post_install do |installer|
installer.pods_project.build_configuration_list.build_configurations.each do |configuration|
configuration.build_settings['CLANG_ALLOW_NON_MODULAR_INCLUDES_IN_FRAMEWORK_MODULES'] = 'YES'
end
end
I get:
/Pods/Headers/Private/FBSDKCoreKit/FBSDKGraphRequestConnection.h:325:1: Unknown type name 'FBSDK_EXTERN'
/Development/Fungry/App/iOS/I am Fungry/Pods/Headers/Private/FBSDKCoreKit/FBSDKGraphRequestConnection.h:325:1: Unknown type name 'FBSDK_EXTERN'
error: unknown type name 'FBSDK_EXTERN'
FBSDK_EXTERN NSString *const FBSDKLoggingBehaviorCacheErrors;
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
Any ideas?

Hey @avdwerff, thanks for the report.
This is a known issue with FBSDKCoreKit+FBSDKLoginKit pod that impacts all pods that depend on those.
The workaround for it is to removeuse_frameworks! from your Podfile.
We are tracking this issue in the appropriate repo, so I am going to close this one out.
https://github.com/ParsePlatform/ParseFacebookUtils-iOS/issues/2


[Bug]: Figure in metpy tutorial is not being shown

What happened?
After merging #89, it seems that the figure in the metpy tutorial is not being rendered. However, in the PR CI, everything worked fine eventually. Maybe we should just retrigger it.
Minimal Complete Verifiable Example
No response
Relevant log output
No response
Environment
-

Anything else we need to know?
No response

Fixed!


HTTP Crawler Memory Leak?

Which package is this bug report for? If unsure which one to select, leave blank
@crawlee/http (HttpCrawler)
Issue description
I'm not completely comfortable with Crawlee and dealing with memory leaks in NodeJS with Express, but I suspect there might be a memory leak issue with Crawlee. I have been running it on my server for the past few days, handling tens of thousands of requests each day. However, I keep encountering memory issues. Each time I check the memory usage, it is really high and continues to grow.
Below is some simple code using the HTTP crawler. I might not be properly disposing of the Crawlee instance, which could be causing the memory to increase with every run. The attached picture shows the growth of the process—each snapshot represents another 100 individual runs. The increase is minimal each time, but it adds up significantly over time.
Snapshot 3 - 0 runs, just initiating the server
Snapshot 4 - 100 runs, since the start
Snapshot 5 - 200 runs, since the start
Snapshot 6 - 300 runs, since the start
I let the crawler sit for a few minutes after finishing each 100 runs to let the memory settle.

This is the code I ran to test the express server:
for _ in range(100):
  req = requests.post("http://localhost:3002", json={
      "requests": [
          {"url": "https://httpbin.org/get"},
      ]}
  )

Please let me know if I am doing something wrong or this is how it's supposed to be! Again, I am not too comfortable with all of this so not really sure, but all I know is that my server keeps running out of memory which is 4GB.
Also I did a lot of searching for previous memory leak issues with crawlee and came across a few threads. I believe I implemented what was suggested, sorry if I missed anything!
Code sample
import express from "express";
import type { RequestOptions } from "@crawlee/core";
import { HttpCrawler, Configuration } from "crawlee";

const app = express();
const port = 3002;

app.use(express.json());

type Request = {
  apiKey: string;
  requests: RequestOptions[];
};

type Response = {
  url: string;
  method: string;
  payload?: string;
  headers?: Record<string, string>;
  content: string;
};

function updateRequests(requests: Partial<Request>) {
  requests.requests?.forEach((request) => {
    if (request.method === "POST") {
      request.useExtendedUniqueKey = true;
    }
  });
  return requests.requests;
}

app.post("/", async (req, res) => {
  try {
    const request = req.body as Partial<Request>;

    const responses: Response[] = [];

    const crawler = new HttpCrawler(
      {
        maxRequestRetries: 50,
        useSessionPool: false,
        persistCookiesPerSession: false,
        async requestHandler({ request, body }) {
          const response = {
            url: request.url,
            method: request.method,
            payload: request.payload,
            headers: request.headers,
            content: body.toString(),
          };

          responses.push(response);
        },
      },
      new Configuration({
        persistStorage: false,
      })
    );

    await crawler.run(updateRequests(request));

    await crawler.teardown();

    return res.send(responses);
  } catch (error: any) {
    console.log(error);
    return res.status(500).send(error.message);
  }
});

app.listen(port, () => {
  console.log(`Server is running on port ${port}`);
});

Package version
3.10
Node.js version
20.11.1
Operating system
No response
Apify platform

[ ] Tick me if you encountered this issue on the Apify platform

I have tested this on the next release
No response
Other context
No response

Hello and thank you for your interest in Crawlee!
Unfortunately, with your scenario, I wasn't able to reproduce the error. The increase from ~30 to ~40 MB in the snapshots you've made is completely normal and cannot be considered indicative of any problem. The mark-sweep garbage collection is an expensive operation, so Node.JS always tries to defer it - if you have 4GB of RAM available, Node won't run the garbage collection because of 10 extra megabytes.
You can try using node-oom-heapdump which takes a heap snapshot right when Node encounters the OOM error. This might give you a better idea about what's going on in your process. If your process doesn't fail with an OOM error (and the only issue is that the process is eating more memory than you would expect), then it's probably just Node utilizing all the memory you give it. You can change this using the --max-old-space-size CLI option for Node.


Increase request execution timeout duration for "Try it" feature in ApiDemoPanel

Describe the bug
For the "Try It" feature, if the request takes more than 5 seconds to complete execution, the ApiDemoPanel will show a "Connection Failed" error.
Expected behavior
The ApiDemoPanel shows the response even if the request takes more than 5 seconds to complete
Current behavior
The ApiDemoPanel shows a "Connection Failed" error. The request actually completes its execution and returns a response (verified from the network tab)
Possible solution
The timeout is hardcoded in makeRequest.ts (line 15). If there is no specific reason for choosing that value, it can be increased to a higher timeout.
Another option is to add a new plugin config variable that let's the user choose a timeout value themselves.
Steps to reproduce
Can be reproduced by sending an API requesting in the ApiDemoPanel that takes more than 5 seconds to return a response.
Context
I am building the documentation for a text-to-speech API that sometimes takes a fair bit of time to generate a response. It would be awesome if the "Try it" feature worked for slower APIs too.
Your Environment

Version used: 1.2.1

Hi @devgeetech, thanks for reporting this issue. I think a new plugin config option that allows users to configure the timeout makes sense. We'll add it to our backlog of feature requests.

@devgeetech, we recently improved support for swizzling OpenAPI docs theme components, including the makeRequest.ts module that handles generating the request. That said, it should be possible to swizzle in order to override the timeout.

Closing since this option is now supported with swizzling. Will revisit possibility of adding a timeout plugin option in the future.


How combine MFCC and CPCfeatures

Thank you for sharing your code, I have meet some problem.
When we use CPC, it is [128,256] but mfcc is [frame,39]，
as you result， I wonder how to  combine it in [frame, 39 + 256] dims.
Thanks again

hi @cyl250
It is common to combine features by simply concatenate them (along the feature dimension).
The CPC feature is [num_framess, 256] and MFCC is [num_frames, 39]. Concatenating them would give [num_frames, 256+39].

sorry, I have some trouble to understant
After model.predict() the CPC features  is in [128,256] dims.
Do I need change the numbers of node of network to fix mode.prdict() return a [num_frames,256] vectors?

128 is the number of frames during TRAINING. In the CPC training, random chunks from the raw waveform are selected and input to the encoder. For example, a random chunk of 20480 data points corresponds to 1.28 seconds, or 128 frames (16k Hz audio).
During inference, you should input the entire utterance instead of the chunks. This will give you the correct number of frames instead of 128.

Thank you very much, I got it.
It helps a lot.
Thank you again.
------------------ 原始邮件 ------------------
发件人: "Cheng-I Jeff<EMAIL_ADDRESS>发送时间: 2019年7月22日(星期一) 晚上8:30
收件人<EMAIL_ADDRESS>抄送: "陈雨龙"936229102<EMAIL_ADDRESS>主题: Re: [jefflai108/Contrastive-Predictive-Coding-PyTorch] How combineMFCC and CPCfeatures (#3)
128 is the number of frames during TRAINING. In the CPC training, random chunks from the raw waveform are selected and input to the encoder. For example, a random chunk of 20480 data points corresponds to 1.28 seconds, or 128 frames (16k Hz audio).
During inference, you should input the entire utterance instead of the chunks. This will give you the correct number of frames instead of 128.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


SSL certificate expired

curl returns the following error
curl: (60) SSL certificate problem: certificate has expired
More details here: https://curl.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

when I try to download a matrix.
Our Julia package SuiteSparseMatrixCollection.jl is not working anymore due to this issue.

Try it now.  I had an issue that led to a delay in getting the new certificate installed.  It is updated now and should work fine.

It works, thanks @DrTimothyAldenDavis !


Sketch 69.2, 107504: Window fails to open

Started first experiencing this with the named version, but may have been occurring before. Video displays invoking the window, it not opening, and then you can see me open another plugin to show that there are multiple instances of the selected symbol.
It appears to be intermittent. The window wouldn't appear yesterday, but worked again last night. Will happily donate to prioritize a fix; Symbol Instance Locator is an indispensable part of my daily workflow.
Video

Same issue on my end, would also love a fix, integral to my workflow.

Same issue on my end, would also love a fix, integral to my workflow.

@WilNichols sorry for any troubles with the plugin. As you can probably tell, my development on the plugin has slowed due to competing priorities. It's not for lack of interest though, rest assured. I have just published a fix that should improve responsiveness, result accuracy, and better darkmode support. Also includes a fix for Sketch 72. Do you mind giving it a shot?

@sonburn Absolutely no worries, and I appreciate your followup. I'll get a look!

So far, so good, and the dark theme changes are further appreciated. Would you prefer I make a new ticket if there are issues with this release?

@WilNichols Glad to hear it. I am going to close this issue, if you run into anything else, please do create a new one.


closed tags and removed redundant closing tags

I have removed redundant closing tags and added closing tags for unclosed html tags. Review this and please suggest improvements.

Current coverage is 42.58% (diff: 100%)

Merging #812 into development will not change coverage

@@           development       #812   diff @@
=============================================
  Files                6          6
  Lines              735        735
  Methods             74         74
  Messages             0          0
  Branches           118        118
=============================================
  Hits               313        313
  Misses             422        422
  Partials             0          0

Powered by Codecov. Last update b413eb7...1f87806

Now we have underline against current page not just under home.