JavaScript runtime error: Object doesn't support property or method 'valid'

JavaScript runtime error: Object doesn't support property or method 'valid'

This starting occuring in mvc 5

could you post a jsfiddle which reproduces the issue?

https://jsfiddle.net/pe7uet7L/
From: Markus Staab<EMAIL_ADDRESS>Sent: Thursday, April 09, 2015 3:29 AM
To: jzaefferer/jquery-validation
Cc: Israel Garcia
Subject: Re: [jquery-validation] JavaScript runtime error: Object doesn't support property or method 'valid' (#1452)
could you post a jsfiddle which reproduces the issue?
—
Reply to this email directly or view it on GitHubhttps://github.com/jzaefferer/jquery-validation/issues/1452#issuecomment-91136757.

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Support for multiple regions in GridSelection and colors for each

glide-data-grid currently supports cell selection, range selection, row selection, and column selection.
// https://github.com/glideapps/glide-data-grid/blob/main/packages/core/src/data-grid/data-grid-types.ts
export interface GridSelection {
    readonly cell: readonly [number, number]; // cell selection
    readonly range: Readonly<Rectangle>; // column selection
}

export class CompactSelection {
  ...

}

// https://github.com/glideapps/glide-data-grid/blob/main/packages/core/src/data-grid/data-grid.tsx
readonly selectedRows?: CompactSelection;
readonly selectedColumns?: CompactSelection;
readonly selectedCell?: GridSelection;

However, those selections given as props are limited when representing selections that originate from interactions outside of the normal current grid and when customizing the colors and fill of individual selections.
As an example / application from google sheets, the selections come from the active input box, and can be all different types and colors.

Would it possible for this library to support that? As a quick sketch, I think it could look like this
readonly selectedRows?: CompactSelection;
readonly selectedColumns?: CompactSelection;
readonly selectedCell?: GridSelection;
readonly selections?: ((CompactSelection | GridSelection) & SelectionStyle)[];

export interface SelectionStyle {
  border?: string // css equivalent
  backgroundColor?: string // css equivalent
}

And then selections would only need to be passed as a prop into drawGrid.

It wouldn't get you the borders, but you could do the background colors yourself by setting the cell's bgCell in themeOverride
https://github.com/glideapps/glide-data-grid/blob/a29d0ba1dbf7f6f69857bb13655937251bd5b680/packages/core/src/data-grid/data-grid-types.ts#L240-L245

Thanks for the pointer! The following type would be a better and more extensive styling.
readonly selections?: ((CompactSelection | GridSelection) & Partial<Theme>)[];

https://github.com/glideapps/glide-data-grid/blob/a29d0ba1dbf7f6f69857bb13655937251bd5b680/packages/core/src/common/styles.ts#L3-L42

How would you want the keyboard + mouse interactions to work?

Keyboard + mouse interactions with selections can be left up to end user since onKeyDown and onMouseDown are already available props that could be used to update selections and override default behavior.

Turns out I think we need this for other reasons as well. To support "transparent" features like sorting and grouping without requiring the user to implement them fully or partially we will need to be able to support multiple ranges as a seleciton.
This way if a table is sorted to the user a rectangular selection in the sorted table can be broken back out to its constituent sub-regions when reporting up to the consuming application without them having to consider what the sorting was doing. It's a complicated feature set but one that will help a lot not just with your use case but with many others.

Goals
Allow the user to highlight multiple areas of the grid to indicate related cells and regions.
Implementation Details
export interface GridSelection {
    readonly cell: readonly [number, number];
    readonly range: Readonly<Rectangle> | readonly Readonly<Rectangle>[];
    readonly highlights: {
        readonly range: readonly Readonly<Rectangle>;
        readonly color?: string;
        readonly key? string;
    }[];
}

There are a variety of subtleties to this that I think are worth discussing.
Why not just use a single array of ranges?
The range in the GridSelection is basically a special highlight. It is the highlight which can be mutated via the keyboard. If you press shift+arrow keys it is the highlight which changes.  It is also the only highlight which must contain the selected cell.
How does the primary range/selection get a color?
From the theme like it does now. The others get passed colors.
What about remapped ranges?
This might be a pipe dream to make this totally transparent. At the moment my thinking is that the sorting hook can also provide getCellContent, getCellsForSelection, as well as handle all events to deal with selection. Selection then needs to be mangled/demangled based on the sort index map.
I don't immediately see why it won't work, but I didn't see why column sorting couldn't be totally transparent until I tried to implement it either.
Is this backwards compatible?
Kind of? At the very least all existing implementations will continue to work. Any library which is based on an older version of the code prior to this change may be incompatible with any code that uses the newly expanded type however. Effectively the answer is yes for applications, no for libraries.
Tags
I think it would be very useful for each highlight region to have a tag. If either the top or the bottom of the region (prefer top) is visible, we could draw a tag labeling. Problem here is I need to think very carefully about how this would impact blitting. At the bare minimum we would need to be careful when doing a damage draw that is right below one of these highlights.
Do highlight regions also need to support lists?
Technically to make these work with transparent sorting we would need to allow them to be broken up as well. However this is a feature that is only needed in very specific cases and I'm not sure we don't just disable these whenever temporary sorting/filtering is enabled. Conversely these are in fact consumption only as far as DataEditor is concerned so breaking them up simply by creating a much larger list where some of them have the same color and making sure that we merge nearby ranges with the same color by treating the color as a key would also be sufficient.
This has the unfortunately side effect of slightly leaking this behavior if a developer were to add two highlight ranges next to each other which have the same associate color. To resolve this we would need to force a key parameter as well. It could be optional and transparent transform hooks could set it if its not already set. The key would allow for a hard line between adjacent highlights with the same color. Anything without a key will be assigned a random key during normalization.
Issues

No method for providing a mechanism to support drawing dashed lines.
How does this interact with multiple overlapping highlights? Do the colors blend? What blend mode? Also ew...
If two highlight regions are overlapping how do they outlines get their colors? They can't really blend.
Can we draw this performantly? I suspect yes but I'm not 100% sure.

Version bump requirements
Bare minimum minor version bump. There is a reasonable argument to be made for major version.

It might be worth breaking this up into 2 work items

Support array of ranges
Support highlights

No method for providing a mechanism to support drawing dashed lines.

A highlight could have a drawRange callback as a prop and which passes canvas ctx as an argument. Something akin to this I've seen in another library is https://prosemirror.net/docs/ref/#view.Decoration^widget which passes current state and positioning.

How does this interact with multiple overlapping highlights? Do the colors blend? What blend mode? Also ew...

Allow that to be customized by end-user for each highlight as mentioned before via drawRange.

If two highlight regions are overlapping how do they outlines get their colors? They can't really blend.

highlights should operate as a stack here: an element at end of array is last drawn, so visually on-top of previous elements. So new highlighting ranges (typically pushed to end of array) would always take precedence.
If end-user has problem with this behavior, they could just re-arrange the array before passing it into data-grid.

I've got the basic works of this up in a branch. The colored highlight ranges aren't in yet but that will come next.

Fixed in 4.0.0


Task/recover delete studies admin

What's this PR do?
Adds the functionality to recover and delete studies for the admin. (i messed up the branch name hehe)
Where should the reviewer start?
The main thing to note is that we are reusing the views used to delete and recover studies for the capturista. Thus, we only add a bit of logic in the save method of the delete form (the logic was already implemented in the recover on the last pr).
estudios_socioeconomicos/forms.py
captura/views.py (note the decorators)
static files
How should this be manually tested?
try deleting and recovering studies with an admin. The option to delete only appears for approved studies, and the option to recover on deleted ones.
Any background context you want to provide?
This template was adapted ~stolen~ from Quickleft/Sprint.ly

Codecov Report

Merging #82 into develop will increase coverage by 0.01%.
The diff coverage is 98.03%.

@@             Coverage Diff             @@
##           develop      #82      +/-   ##
===========================================
+ Coverage    97.25%   97.27%   +0.01%
===========================================
  Files          140      140
  Lines         3937     3959      +22
===========================================
+ Hits          3829     3851      +22
  Misses         108      108

Impacted Files
Coverage Δ

estudios_socioeconomicos/forms.py
100% <100%> (ø)
:arrow_up:

administracion/test_views.py
98.92% <100%> (ø)
:arrow_up:

estudios_socioeconomicos/test_forms.py
100% <100%> (ø)
:arrow_up:

captura/views.py
86.86% <88.88%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add experimental formats for C++. [phab:D843]

Created by zarko at 2016-04-20 00:39:37:
It's expensive to store types and signatures (which contain types)
as literal strings. Formats compress types and signatures by
defining them recursively. For example, the format for a
C++ pointer is %1.*. The format for a variable foo is
%^::foo. The format for an arbitrary type application is
%0.<%1,>.
This feature is explicitly marked experimental as I expect the
definitions of format tags and the formatting algorithm to
change as we discuss and apply it.
This addresses #265.

shahms commented inline (context lost during migration) at 2016-04-20 16:01:15:
Not sure how much it matters, but I don't think these aliases qualify as "builtin".

shahms commented inline (context lost during migration) at 2016-04-20 16:01:15:
Maybe include some comments about the rationale for this; it otherwise seems questionably complicated.

zarko wrote at 2016-04-20 18:13:30:

Address comments.

zarko commented inline (context lost during migration) at 2016-04-20 18:16:00:
Clang considers them to be. If it turns out to be a problem we can desugar them to aliases.

shahms accepted this revision at 2016-04-20 18:17:14

zarko closed via commit d12944538c5160ca81483096733956b909da1037 at 2016-04-20 18:20:17


Mise à jour en cascade des dates de dernière modification

La modification ou suppression d'un document aura pour effet de mettre à jour le champ _updated des documents dont il dépend.
Fix #14 Mise à jour des dates

Codecov Report

Merging #17 into master will increase coverage by 0.16%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   94.25%   94.41%   +0.16%
==========================================
  Files          12       12
  Lines         383      394      +11
==========================================
+ Hits          361      372      +11
  Misses         22       22

Impacted Files
Coverage Δ

lib/models/numero.js
92.85% <100%> (+1.02%)
:arrow_up:

lib/util/mongo.js
80% <100%> (+1.05%)
:arrow_up:

lib/models/voie.js
91.37% <100%> (+0.47%)
:arrow_up:

Continue to review full report at Codecov.

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


IPsec encrypt status JSON output.

Cilium agent encrypt status command refactored and JSON output support implemented.
The changes were requested in Cilium CLI PR: https://github.com/cilium/cilium-cli/pull/2212 to improve encryption status parsing.
api/cli: Encryption status now includes rendering IPsec status in JSON.

Overall this looks good, but it needs two things:

A release-note block in your initial PR comment.
An addition to to the upgrade guide outlining the change in the API.

Done.

/test

/test

/test

/test

@nathanjsweet @asauber @learnitall @rgo3 do you think it will possible to backport these changes in the 1.12, 1.13, 1.14?
I'm asking because of the conversation: https://github.com/cilium/cilium-cli/pull/2212#discussion_r1446119226

@viktor-kurchenko If you think this needs to be backported can you please add backport labels? Thanks!


Fix a few more dartdoc errors

Turns out that []: means something in markdown, so you have to
escape the : with a backslash.


[BRAND REQUEST] Dreamwidth

Please enter the brand name.
Dreamwidth
Please enter the brand website.
https://www.dreamwidth.org/
Please enter the brand SVG. (If Applicable)

or

I will bulk do them so if there is more, feel free to make more requests!

Thanks a lot! I can go take a look around. Please let me know if I can make things easier for you!


TF doesn't fuse Conv2D since v2.7

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.8, tf 2.9, tf 2.10
Custom Code
No
OS Platform and Distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
11.6/8.1
GPU model and memory
No response
Current Behaviour?
The TF no longer fuse the convolution+bias+activation patterns from 2.8 and later.

Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np

use_nhwc = True

N, H, W, C = 1, 2, 3, 3
k, r, s, c = 3, 2, 2, C

if use_nhwc:
  x_format = 'NHWC'
  x_format_keras = 'channels_last'
  bias_format = 'N...C'
  x_shape = (N, H, W, C)
  channel_axis = -1
else:
  x_format = 'NCHW'
  x_format_keras = 'channels_first'
  bias_format = 'NC...'
  x_shape = (N, C, H, W)
  channel_axis = 1
f_np = np.random.random([r, s, c, k]).astype(np.float32)
f = tf.Variable(f_np)
b_np = np.random.random([k]).astype(np.float32)
b = tf.Variable(b_np)

@tf.function
def fused_conv_bias_relu(x):
  y = tf.nn.conv2d(x, f, strides=(1,1), padding='SAME',
                   data_format=x_format, name='xxx_cno1')
  y = tf.nn.bias_add(y, b, data_format=bias_format)
  y = tf.nn.relu(y)
  return y

inputs = tf.random.normal(x_shape)
outputs = fused_conv_bias_relu(inputs)
print(outputs)

Relevant log output
$ TF_CPP_VMODULE=remapper=2 python demo.py
In 2.7.1 container:
...
2022-04-28 20:35:13.516124: I tensorflow/core/grappler/optimizers/remapper.cc:1345] Fuse Conv2D with BiasAdd and Relu: activation=Relu bias_add=BiasAdd contraction=xxx_cno1

In nightly-gpu container:
...
<No fusion info>

Initial investigation shows that here: the returned input_props of the conv2d node is an empty vector and further here: the member input_properties_ is also an empty map.
vis. @nluehr @wenscarl

This following script might be more clear. On 2.7.0 container, it will output True but on 2.8.0 it will output False.
from absl.testing import parameterized

from tensorflow.core.protobuf import config_pb2
from tensorflow.core.protobuf import rewriter_config_pb2
from tensorflow.python.client import session
from tensorflow.python.framework import constant_op
from tensorflow.python.framework import dtypes
from tensorflow.python.framework import ops
from tensorflow.python.framework import test_util
from tensorflow.python.ops import init_ops
from tensorflow.python.ops import math_ops
from tensorflow.python.ops import nn
from tensorflow.python.ops import nn_ops
from tensorflow.python.ops import random_ops
from tensorflow.python.ops import variables
from tensorflow.python.platform import test
from tensorflow.python.util import _pywrap_utils
from tensorflow.python.eager import context

def _input(shape):
  """Generates an input of a given shape."""
  return variables.Variable(random_ops.truncated_normal(shape, seed=0))

def _weight(shape):
  """Generates a weight of a given shape."""
  # Note that the lambda is needed to allow construction inside loops.
  return variables.Variable(lambda: init_ops.glorot_uniform_initializer(seed=0)
                            (shape))

def _bias(shape):
  """Generates a bias of a given shape."""
  return constant_op.constant(0.1, shape=shape)

def _get_config(remapping_on=False):
  """Returns a CongfigProto with remapper optimizer on/off."""
  rewrite_config = rewriter_config_pb2.RewriterConfig(
      remapping=rewriter_config_pb2.RewriterConfig
      .ON if remapping_on else rewriter_config_pb2.RewriterConfig.OFF)
  rewrite_config.min_graph_nodes = -1
  graph_options = config_pb2.GraphOptions(rewrite_options=rewrite_config)
  config = config_pb2.ConfigProto(graph_options=graph_options)
  return config

def test_conv2d_biasadd_relu_fusion():
  """Test Conv2D+BiasAdd+Relu fusion."""
  run_options = config_pb2.RunOptions(output_partition_graphs=True)
  metadata = config_pb2.RunMetadata()

  n, h, w, c = (5, 3, 3, 4)

  ops.reset_default_graph()
  x = _input([n, c, h, w])
  w = _weight([2, 2, c, c])
  b = _bias([c])
  y = nn_ops.conv2d(x, w, strides=(1, 1), padding='SAME', data_format='NCHW')
  z = nn.bias_add(y, b, data_format='NC..')
  out = nn.relu(z)

  # Compute reference value.
  config = _get_config(remapping_on=False)
  with session.Session(config=config) as sess:
    sess.run(variables.global_variables_initializer())
    output_val_ref = sess.run(
        out, options=run_options, run_metadata=metadata)
  # Compute output with fusion.
  config = _get_config(remapping_on=True)
  with session.Session(config=config) as sess:
    sess.run(variables.global_variables_initializer())
    output_val = sess.run(out, options=run_options, run_metadata=metadata)
    graph = metadata.partition_graphs[0]

  # Graph should contain fused op.
  found_fused_op = False
  for node in graph.node:
    if node.op == '_FusedConv2D':
      found_fused_op = True
      break
  print(found_fused_op)

with context.graph_mode():
  test_conv2d_biasadd_relu_fusion()

I think I found the root cause and I filed a PR to fix it: https://github.com/tensorflow/tensorflow/pull/55923

@kaixih We see that, the raised PR is merged.
Please feel free to move this issue to closed status if it is resolved. Thank you!

Yes, I am closing this.


sdk-wallet: fix names, move behavior (functions)

I think it might be worth to add an IWallet specification, since besides the generate_mnemonic method they share the same methods and signatures. Either in this PR or in another one, whatever you may fancy.

Indeed, added IWalletProvider and IMnemonicComputer.


REMUX MKV RIP Playback - DoVi Profile 7 Issues - Multiple Devices / Android Versions

Describe the bug
I have multiple Android TV boxes and/or Smart TVs in our home. I am using Just Player 1.51 as the primary player across all my devices. DoVi Profile 7 failback only works on my devices that support the full range of DoVi encoded profiles.
With my Sony Bravia Android TV we can natively play DoVi Profile 7 videos. On some rips the videos default to HDR anyways without an error and there is no way, that I am aware of, to run a debug to find out why this is happening. This issue exists regardless of the switch turned in the advanced settings is toggled to play it unchanged or play it in HDR mode. Using the prefer decoder settings toggles doesn't impact the failover functionality. Tunneling also does not impact or change this functionality. No combination of Tunneling on/off, combined with decoder preferences, or DoVi failback. I went through a rigorous testing process to verify each combination.
With my RockTek G2 Android TV I cannot run DoVi Profile 7 videos in Just Player. I get the error code saying that the video file exceeds the limit of the supported format. This issue exists regardless of the switch turned in the advanced settings is toggled to play it unchanged or play it in HDR mode. Using the prefer decoder settings toggles doesn't impact the failover functionality. Tunneling also does not impact or change this functionality. No combination of Tunneling on/off, combined with decoder preferences, or DoVi failback. I went through a rigorous testing process to verify each combination. Oddly enough on this device DoVi failback and/or DoVi pass through of Profile 7 work in Nova Player on this same machine. I have Nova Player setup to use the Android Media Library, basically the same as Just. Just Player can play and passthrough other DoVi profiles to the my Sony Bravia, and I can verify this by checking the HDR/Picture settings the picture works fantastic. Dolby Vision Bright and Dolby Vision Dark display perfectly. With this box there is also no issue playing any Dolby Audio formats. The DolbyVision rips that work are MKV UHD 2160P x/h264 or HEVC. Web-DL, WEB, and x/h264 HEVC DoVi haven't been an issue.
Expected behavior
DoVi Profile 7 movies failback to HDR 10 when opening Hybrid REMUXs that include DoVi Profile 7 and HDR10 / 10+.
(Nova Player's implementation works better)
Device (please complete the following information):
Sony Bravia Android TV

Model: Sony X950H (XBR-65950H)
Processor: MediaTek Mt8993
OS: Android 9
Version: Bravia_UR3_UC-user 9 PTT1.190515.001.S104 650661

RokTek G2

Model: RT-G2 / SEI804RT (YYJ)
Processor: Amlogic S905X4
Android 11
Version: RTT0.211009.001.53100

*** WILLING TO PAY TO GET THIS BUG RESOLVED ***
$150-200 CAD

I'm afraid I won't be able to help here. As fallback of hybrid DV/HDR10(+) files work fine on my devices and it's fully in control of ExoPlayer.


An unrelated warning when transpiling an empty set of circuits

Environment

Qiskit Terra version: a60374091
Python version: 3.9.5
Operating system: Linux

What is happening?
Transpilation of an empty set of circuits issues a warning about schedule transpilation.
How can we reproduce the issue?
from qiskit import transpile, IBMQ
backend = IBMQ.load_account().backend.ibmq_belem  # can be any backend
transpile([], backend)

Output:
qiskit/compiler/transpiler.py:246: UserWarning: Transpiling schedules is not supported yet.
  warnings.warn("Transpiling schedules is not supported yet.", UserWarning)

What should happen?
No warning
Any suggestions?
Just write
if len(circuits) > 0:

Write before the line
warnings.warn("Transpiling schedules is not supported yet.", UserWarning)

Sounds sensible - we only enter that block as a side-effect of all([]) being defined as True.  Fix in #7288.


Feat: implement decrease_position()

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ]  Documentation content changes
[ ] Testing
[ ] Other (please describe):

What is the current behavior?

Resolves: #330
What is the new behavior?

Added decrease_position() logic
Added support for Default::default() on i128
Added mocked missing functions

Does this introduce a breaking change?

Other information

Test can't be handled due to logic function missing.

Can you please resolve conficts and it should be good


community: add mlrun LLM

Added an option to integrate llms served on the mlrun framework with langchain.
Added an initial Mlrun(LLM) with basic capalbilities, an example notebook and some tests.
dependecies: mlrun

Hey @efriis , i can't see why the vercel is failing, it's like i have premissions, is there a way i can see what's going on? also, how do i know if this is related to the fact that the pr isn't being reviewed?

Hey there - if you click "inspect" now you should be able to see vercel logs!
Hey! This adds a net-new community integration or feature, which has been replaced by dedicated integration packages. I would recommend reopening with just docs updates, as well as registering your package in libs/packages.yml! We'll be able to review simple PRs that only modify these two things much faster :)
Here's the guide, and if you have questions, feel free to leave them in the comments on those pages so others can see them! https://python.langchain.com/docs/contributing/how_to/integrations/

Closing with the recommendation of implementing this in a separate package (e.g., langchain-mlrun). We've documented instructions for doing this in the link @efriis shared. Please let us know if you have feedback on this.


Sparkle24

@philipperolet
This is introducing a hopefully better version Page!

:pray: Did you publish an alpha version? I'll test it on my branch with it
Note: Currently I solved it by wrapping all the page content in a flex col with gap 6 div

🙏 Did you publish an alpha version? I'll test it on my branch with it

The package 2.4 is already published. You can just use it.
This PR is fixing a small thing with a 2.5 (which you don't need) and bringing the front to 2.4 (which you need).


fix(Videos): Fix videos embed

What this PR does 📖
1. Fix videos embed

Which issue(s) this PR fixes 🔨

Resolve #1408

Special notes for reviewers 🗒
Additional comments 🎤

btw youtube works but youtube shorts don't display the video 🔨

we can fix the youtube shorts in another PR 🔨


Support multiple highlights in MudHighlighter

Feature request type
Enhance component
Component name
MudHighlighter
Is your feature request related to a problem?
Currently, it's not possible to highlight multiple substrings in a MudHighlighter.
Describe the solution you'd like
Add a new parameter IEnumerable<string> HighlightedTexts to MudHighlighter which accepts an enumeration of substrings to highlight in the text.
I've already implemented a prototype of that here. It's quite straightfoward.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
A more flexible (though more complex) alternative would be regex support instead, such as requested in Regex based highlighting for MudHighlighter #2952.though
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

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

Dup/similar #3129

@Enderlook Why not create a PR for this, since you already have it working?

@Enderlook Why not create a PR for this, since you already have it working?

I thought I had to ask for permission before making a PR.
I modified my commits to apply the PR guidelines and just make the RP.


Retry Logic for Status Code 429 Too Many Calls

Is there plans to put in retry logic when making calls to the Luis.ai api if a status code of 429 is returned?
The free plan for luis.ai is 5 calls per second. Even the paid plan is only 10 calls pre second. That isn't very much if we intend to release a bot to production. I would be happy to try and implement a solution but didn't want to start working on it if there was already something in the works.

@msft-shahins : As we have the limit of 10 calls/sec even for paid version do we have any retry logic in built?


make install no longer respects lib_binary_dir_path

So I have a gem with a tree structure looking like this:
ext/digest/xxhash/ext.c
lib/digest/xxhash/version.rb

And before https://github.com/rake-compiler/rake-compiler/pull/191 was merged, I was able to configure rake-compiler like this:
Rake::ExtensionTask.new('digest/xxhash', Bundler::GemHelper.gemspec)

This is because binary_path is generated from name which contains the partial directory path digest, and the old install command, install tmp_binary_path, "#{lib_path}/#{binary_path}" refers to it.
But now I have to manually specify directories:
Rake::ExtensionTask.new('xxhash', Bundler::GemHelper.gemspec) do |task|
  task.ext_dir = "ext/digest/xxhash"
  task.lib_dir = "lib/digest"
end

I'm not sure but maybe it has to do with the line siteconf.puts "dest_path = mkintpath(#{File.expand_path(lib_path).dump})".  Instead of lib_path, it should specify lib_binary_dir_path instead.
Maybe specifying name with a subdirectory is illegal but if that's the case, what's the point of calculating and using lib_binary_dir_path as an argument to directory, and as a requirement argument instead of just using lib_path.

Also I did modify rake-compiler to use lib_binary_dir_path, installed it, and tested it against my gem with the Rake::ExtensionTask.new line reverted to the old one and it worked.

Is the your gem https://github.com/konsolebox/digest-xxhash-ruby ?
How to reproduce your problem?

One way to reproduce this is by cloning the repo to a temporary directory, checkout the v0.2.1 tag, install rake-compiler 1.1.3 or 1.1.4, then run rake test.
git clone https://github.com/konsolebox/digest-xxhash-ruby.git
cd digest-xxhash-ruby
git checkout v0.2.1
bundle
rake test

The significant output lines should be something like this:
/usr/bin/gmake install target_prefix=
/usr/bin/install -c -m 0755 xxhash.so /var/tmp/t/digest-xxhash-ruby/lib
cp tmp/x86_64-linux/digest/xxhash/3.0.2/digest/xxhash.so tmp/x86_64-linux/stage/lib/digest/xxhash.so
/usr/bin/ruby30 -w -I"lib" /usr/lib64/ruby/gems/3.0.0/gems/rake-13.0.6/lib/rake/rake_test_loader.rb "test/test.rb"
<internal:/usr/lib64/ruby/site_ruby/3.0.0/rubygems/core_ext/kernel_require.rb>:85:in `require': cannot load such file -- /var/tmp/t/digest-xxhash-ruby/test/../lib/digest/xxhash (LoadError)

You can see that xxhash.so is copied to /lib instead of /lib/digest.
Here's also a failing log in AppVeyor: https://ci.appveyor.com/project/konsolebox/digest-xxhash-ruby/builds/41851529/job/9st82d9etf11w5q9

Could you try 1.1.5?

I tested it locally and also in AppVeyor.  It looks fixed: https://ci.appveyor.com/project/konsolebox/digest-xxhash-ruby/builds/41857053

Thanks a lot!


validation error for French mobile number +337XXXXXXXX

Country/region affected : FR
Example number(s) affected : + 33 7XX XX XX XX
The phone number range(s) to which the issue applies : + 33 7XX XX XX XX
The type of the number(s): mobile
Phone number starting by 07 (national) , + 33 7 (international) are not valid mobile phone number.
According to http://www.arcep.fr/index.php?id=8146, number starting by "073" to "079" are mobile phone valid numbers.
Completes specifications are available here : http://www.arcep.fr/uploads/tx_gsavis/12-0855.pdf (page 17)

Actually, please see #438 for discussion over arcep and what it is really saying. Do you have a particular phone number  that is diallable that we mark as invalid? The doc you reference is from 2012, no? If something has changed recently we're happy to update but we need real evidence that our conclusion from last time is wrong.


⚠ Samsung has degraded performance

In 5df15e9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2509 ms

Resolved: Samsung performance has improved in 7466464.


OpenAL gives wrong include and libs path with pkg-config

Describe the bug
The pkg-config file for OpenAL is unusable.
Steps To Reproduce
Steps to reproduce the behavior:

open a devshell with pkg-config and openal (works both with the stable and unstable release of nixpkgs)
pkg-config --cflags
notice that the flags are not correct: « -I/nix/store/iccwkh7yacv40zyqnj1mah73616mid5s-openal-soft-1.21.1//nix/store/iccwkh7yacv40zyqnj1mah73616mid5s-openal-soft-1.21.1/include -I/nix/store/iccwkh7yacv40zyqnj1mah73616mid5s-openal-soft-1.21.1//nix/store/iccwkh7yacv40zyqnj1mah73616mid5s-openal-soft-1.21.1/include/AL »

in a single command, nix-shell -E 'with (import <nixpkgs> {}); stdenv.mkDerivation{name="foo"; buildInputs=[pkg-config openal];}' --run 'pkg-config --cflags openal'
here is the result on unstable: -I/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1//nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/include -I/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1//nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/include/AL
Expected behavior
Include path is

/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/include
/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/include/AL
instead of
/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1//nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/include
/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1//nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/include/AL

Likewise for --libs, -L/nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1//nix/store/dvb8l321pvzdyb2c2ljpc9wvrfzs335w-openal-soft-1.21.1/lib should not contain the path twice.
Notify maintainers
@ftrvxmtrx
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.106, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4`
 - channels(root): `"nixos-21.11.336340.2ebb6c1e5ae, nixpkgs-git"`
 - channels(afontain): `"nixos-20.09-20.09.4407.1c1f5649bb9, nixpkgs-unstable-22.05pre342949.28d58b97925, nixpkgs-20.09-20.09.4407.1c1f5649bb9"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

See https://github.com/jtojnar/cmake-snips on how to fix this.

cc @fufexan @ilya-fedin

@jtojnar I'd happily work on this but I have no experience with CMake. Perhaps with some guidance I could try to make it work.
Would it need a patch on our side, or maybe a fix upstream?

Ideally, it would be fixed upstream. These are the offending lines (variables are absolute paths):
https://github.com/kcat/openal-soft/blob/22e6c0df6017aa86e2cd83d8073e4d490c55dd11/CMakeLists.txt#L1256-L1258

https://github.com/kcat/openal-soft/pull/696

https://github.com/NixOS/nixpkgs/pull/181875 merged

Why not to update it to 1.22.2 rather than patching? :thinking:

Why not to update it to 1.22.2 rather than patching? thinking

looks like it was already done. i added a commit removing the patch in https://github.com/NixOS/nixpkgs/pull/172347

Does search-out.patch actually apply? The code in 1.22.2 is different, so it's very surprising if it applies cleanly. It's possible to set -DALSOFT_SEARCH_INSTALL_DATADIR=ON instead of the patch.


SPARKC-542: SPARKC-503 Backport

Foward merge of 542, we can skip it

Build against Scala 2.10 finished with failure

Build against Scala 2.11 finished with success


Wifimanager don't open dialog window automaticly when connect to ESP-32 AP

Normaly a popup window comes when PC or smartphone connect to WIFIMANAGER ESP-32 in access-point mode. At my devices this feature not work. If you connect on the IP-address with a webbrowser you are able to configure WIFI, but not automaticly. After analyse packets between my Macbook and ESP32 I found there is no valid DNS-Server entry in DHCP packet. The value is alway "<IP_ADDRESS>". After hours of work I found a solution to set the entry manualy. In WiFiAP.cpp at the end around line 140 in function WiFiAPClass::softAP paste the following code:
tcpip_adapter_dns_info_t dns_info;
dns_info.ip.u_addr.ip4.addr = IPAddress(192,168,4,1);
tcpip_adapter_set_dns_info(TCPIP_ADAPTER_IF_AP, TCPIP_ADAPTER_DNS_MAIN, &dns_info);
and if you set your IP-address manualy in WiFiAPClass::softAPConfig  paste around line 180:
tcpip_adapter_dns_info_t dns_info;
dns_info.ip.u_addr.ip4.addr = info.ip.addr;
tcpip_adapter_set_dns_info(TCPIP_ADAPTER_IF_AP, TCPIP_ADAPTER_DNS_MAIN, &dns_info);
So the DNS-setting in DHCP-protocol is correct and the configuration window pops up when connecting to WiFIMANAGER

@Radeckf Please could you advise me, there is no file called WiFiAP.cpp in this repository (as far as I can see), can you confirm where this file can be found? Thanks

This file is part of the ESP-32 WiFi library, not of the Wifimanager. You will find this file in the core library.
Regards,
Radeckf

Am 13.06.2018 um 18:20 schrieb anwarbashir<EMAIL_ADDRESS>@Radeckf Please could you advise me, there is no file called WiFiAP.cpp in this repository (as far as I can see), can you confirm where this file can be found? Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Radeckf, Hi Thanks, I found the file here https://github.com/espressif/arduino-esp32/blob/master/libraries/WiFi/src/WiFiAP.cpp, can you confirm is this correct? Thanks :)

Hi, yes that’s correct
Regards
radeckf
Am 17.06.2018 um 17:07 schrieb anwarbashir <notifications<EMAIL_ADDRESS>@Radeckfhttps://github.com/Radeckf, Hi Thanks, I found the file here https://github.com/espressif/arduino-esp32/blob/master/libraries/WiFi/src/WiFiAP.cpp, can you confirm is this correct? Thanks :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/zhouhan0126/WIFIMANAGER-ESP32/issues/2#issuecomment-397885035, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABPMlJtdEojCjChZ-5kuBgmYkTY4lwULks5t9nCQgaJpZM4Q-6tB.

@Radeckf. Thanks for confirming the location of the library. At present I am using Platform.IO and I have WiFIManager working (subject to further testing) but without the use of this library. I am not sure how to implement this library into Platform.IO and this prevents me from implementing your code, at the moment.
Presently, on my andriod phone the browser dialog does not appear after the ip connects, so I will try and implement your code to resolve this. Any guidance would be gratefully received.

i tested in Windows and Android and not worked


Fixed running TransactionsPerRequestTests in isolation.

If it is instead preferable to add django.contrib.auth to available_apps, I can make that change instead. Let me know.

@jdufresne it would seem wrong to me to reference 'django.contrib.auth' into a test for a core feature.

Did you run into this in any particular way? 11856ea44e4dd1ff2358c559c0d04ac848832c23 is the same issue -- I wonder if it's going to pop up again and again. It started happening after cb7bbf97a74fa7800865e3615f196ad65dc4f281. I wonder if moving backend to inner imports in django/contrib/auth/backends.py would be a better solution.

Did you run into this in any particular way?

I was investigating a different issue. My changes had created a test failure in this class. So to iterate quickly, I started testing against this class only. That is when I noticed it didn't pass on the master branch in isolation. It fails because the auth middleware is loaded, but not the auth app & models. Seemed like a straight forward fixed, so I created the PR.

If there is a deeper, more fundamental fix in the works, I will withdraw this PR in preference to that.

I don't have anything in mind besides reverting the change that caused the problem:
diff --git a/django/contrib/auth/backends.py b/django/contrib/auth/backends.py
index 64f9af3..0b51502 100644
--- a/django/contrib/auth/backends.py
+++ b/django/contrib/auth/backends.py
@@ -1,8 +1,6 @@
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import Permission

-UserModel = get_user_model()
-

 class ModelBackend:
     """
@@ -10,6 +8,7 @@ class ModelBackend:
     """

     def authenticate(self, request, username=None, password=None, **kwargs):
+        UserModel = get_user_model()
         if username is None:
             username = kwargs.get(UserModel.USERNAME_FIELD)
         try:
@@ -96,6 +95,7 @@ class ModelBackend:
         return False

     def get_user(self, user_id):
+        UserModel = get_user_model()
         try:
             user = UserModel._default_manager.get(pk=user_id)
         except UserModel.DoesNotExist:
@@ -137,6 +137,8 @@ class RemoteUserBackend(ModelBackend):
         user = None
         username = self.clean_username(remote_user)

+        UserModel = get_user_model()
+
         # Note that this could be accomplished in one try-except clause, but
         # instead we use get_or_create when creating unknown users since it has
         # built-in safeguards for multiple threads.

Not that it might not break again in the future for some reason. The fundamental problem is ALWAYS_MIDDLEWARE in runtests.py includes contrib middleware. I'm not sure if it's practical to change that though.

I can confirm that your patch does allow running the test in isolation. I don't mind including it in the fix. Do you think it still makes sense to override the middleware setting to [] as a signal of intent? Or would you consider that unwanted noise?

I think if we accept the need to put MIDDLEWARE=[] in tests, then there's no point in my patch considering that we won't be able to detect the need for MIDDLEWARE=[] if it's there. We might audit all available_apps usage and INSTALLED_APPS overriding and see if this fix is needed elsewhere. This could be quite tedious.

I wrote the following script to run all Django tests in isolation. As expected, this found TransactionsPerRequestTests, but it found no other tests that fail in isolation. So, AFAICT, this is the only test that needs modification to run in isolation. The script only ran against SQLite due to the time consuming nature of setup & destroy of the database for every test with other backends.
import ast
import os
import subprocess

pwd = os.getcwd() + os.sep

def parse_run(path):
    rel_path = path[len(pwd):]
    python_path = '.'.join(rel_path[:-3].split('/'))

    class ClassVisitor(ast.NodeVisitor):
        def visit_ClassDef(self, node):
            if any(isinstance(n, ast.Name) and n.id.endswith('TestCase') for n in node.bases):
                path = '%s.%s' % (python_path, node.name)
                print('***** %s' % path)
                subprocess.check_call(['python', 'runtests.py', '--noinput', path])

    visitor = ClassVisitor()

    with open(path) as fp:
        tree = ast.parse(fp.read(), path)
    visitor.visit(tree)

for dirpath, dirnames, filenames in os.walk(pwd):
    for fn in filenames:
        if fn.startswith('test') and fn.endswith('.py'):
            parse_run(os.path.join(dirpath, fn))

Thanks for investigating. As a separate task, it could be useful to incorporate an option in runtests.py to run tests in isolation.


Minecraft Crashes in my Modpack when I try to start it with exit code 0

I am working on a Minecraft Modpack for My future Server and I am having trouble with the Chiseled me Mod, I tried Various versions of the Mod and Forge, I also deleted ironchests in the mods folder but it keeps crashing
I hope there's a way to fix it
thanks in advance
newest debug.log:
debug.log

Uh, it's another one of these.
So this exact crash is known, but I have absolutely zero idea why it happens:
Forge complains that a thing that I register is not registered while, I mean, I do register it, in code, properly, what else to tell it :shrug:
Try to update Forge, but usually this is caused by some other random mod somehow (at one point I remember it was Forestry, but then the next forestry version fixed it) so you can try to find it, or maybe it's your unique set of mods, who knows.
But yeah, this crash is known and once I know wth happens I will fix that.
P.S. process exit codes mean absolutely nothing in the context of minecraft or modding it, in fact exit code 0 means that the program finished successfully, not a single clue why mojang decided to include the exit code in the crash popup but I am tired of people telling me their game 'crashed with exit code 0'.
You are not like them, you at least gave the debug.log which is all I am asking for, keep it up, you are great.

Hey, thanks for your answer, And I figured out, why it was crashing the whole time, it seems like, that it isn't compatible with LagGoggles and/or tickCentral
(I have no Idea what those mods do, so I deleted them, and now it works just fine ^^)
maybe this information will help you to fix it some day :)

Yeah this would help, ty


Revive the legacy debug form in DocService

Motivation:
Some users find it inconvenient to use the new debug form especially when they have to refer to request specifications while writing requests.
As the debug page has enough space on the bottom side, it would be better to provide the two debug form styles.
Modifications:

Add DebugInputs component to reuse components used for rendering a debug form.
Render both an inline style and a modal style.

Thanks to React state, the values of the two forms are fully synchronized.

Result:
You can now use either an inline debug form or a debug form modal when using DocService.

@Dogacel Would you mind reviewing this PR if available?

This slipped under my radar... The "Copy as cURL" logic still works on both legacy and modal forms? I'm don't completely follow whether the debug-form ID is duplicated or not and if the browser can find the correct element.

After addressing @jrhee17's comment d4d4a66 (#5072), "Copy as cURL" worked on legacy and modal forms.
Did you find a case where "Copy as cURL" didn't work with the latest version?

https://github.com/line/armeria/assets/1866157/38ea2b8e-8cae-4f8c-b549-c02b6a6792a3

Ah, I haven't actually checked. I just wanted to confirm, since that code is very finicky 😄


Develop/Make Architectural Diagram

@stefanv assigning to you...please add a slide to the google doc on architectural diagram

post to README/readthedocs


chore(deps): :arrow_up: update dependency jest-preset-angular to v13.1.4

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

jest-preset-angular (source)
13.1.3 -> 13.1.4

Release Notes

thymikee/jest-preset-angular (jest-preset-angular)
v13.1.4
Compare Source
Bug Fixes

transformers: handle single string styles or styleUrl property (#2186) (0c99aae)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm WARN cli npm v10.2.3 does not support Node.js v20.2.0. This version of npm supports the following node versions: `^18.17.0 || >=20.5.0`. You can find the latest version at https://nodejs.org/.
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@angular/animations
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-11-14T11_39_35_765Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-11-14T11_39_35_765Z-debug-0.log


the meaning of "restval"

hi, in file prepro_ngrams.py , img['split'] == 'restval' ,what's the meaning of "restval" ? restval not in "train val test".

we use restval as training set.

if so ,why "(params['split'] == 'train' and img['split'] == 'restval')" ,not just"(params['split'] == 'train' )".du you have 'restval' flag in your dataset ?

yes, there are restval in the dataset.


Android Permissions

Hi,
does this SDK automatically add new permissions to the app? If yes, why do you need them?
E.g.:

android.permission.RECORD_AUDIO
android.permission.READ_PHONE_STATE
android.permission.GET_ACCOUNTS

Hi,
READ_PHONE_STATE and RECORD_AUDIO are used for the readers still relying on the audio cable.
GET_ACCOUNTS was used in the old way of login but seems not to be used anymore and therefore can probably be removed in a future version of the sdk.
Thanks for the input. We will take action internally

Hi,
did you consider removing those permissions yet?
I don't really need them for my scenario and Google is giving me a hard time because of it as they consider those permissions "Dangerous".

sumsub android.permission.RECORD_AUDIO causes google app submission to ask for video to explain how you going to use it. is it still neccessary?

Your app uses the FOREGROUND_SERVICE_MICROPHONE permission. You can only use this permission if your app performs tasks noticeable to the user when they're not directly interacting with your app.

Microphone
What tasks require your app to use the FOREGROUND_SERVICE_MICROPHONE permission?


Pretty print schema, minor typo fix

Passing an indent should allow the schema from introspection to be pretty-printed. I also fixed a minor typo that I found in the gradle plugin.
I've not tested this change yet because I couldn't find an existing test in ServiceTests, but I will add one as a follow-up.

Thanks !


Need help for Data Model

Hello,
I need some help for creating the Data Model based on this JSON :
{
"posts": [
    {
      "id": "",
      "title": "",
      "author": {
        "id": "",
        "name": ""
      },
      "related": {
        "id": "",
        "name": ""
      },
      "original_post": {
              "id": "",
              "title": "",
              "author": {
                "id": "",
                "name": ""
              },
              "related": { ... },
              "original_post": { ... }
       }
    }]
}

Thanks a lot !

So a Post can have an Original Post inside and this Original Post can have an Original Post inside, right?
You can have Post ==(to many)==> Original Post ==(recursive to itself)==> Original Post

Here's an example of a recursive relationship. You can find it in the Tests project:
https://github.com/hyperoslo/Sync/tree/master/Tests

I hope this was helpful :v:

Thanks @3lvis, I will try this one :)


Add Nestify / hosting

website:
https://nestify.io/
examples:
https://www.1hindi.com/
https://simplypure.com/
https://www.sexkontakte.com/

Support:
use Cloudflare nameserver and all are different according to sites.
Not confirmed by support :
"headers": {
"x-nestify-cache": ""
},
The accounts team is not on the desk right now, we will inform you shortly.


feat: #935 - now we display localized (downloaded) data during onboarding

Deleted files:

sample_eco_knowledge_panels.json: replaced by new file sample_knowledge_panels.json
sample_health_knowledge_panels.json: replaced by new file sample_knowledge_panels.json

New files:

abstract_onboarding_data.dart: Abstraction of data we download, store and reuse at onboarding.
dao_string.dart: Where we store strings.
onboarding_data_knowledge_panels.dart: Helper around knowledge panels we download, store and reuse at onboarding.
onboarding_data_product.dart: Helper around a product we download, store and reuse at onboarding.
onboarding_loader.dart: Helper around data we download, store and reuse at onboarding.
sample_knowledge_panels.json: json data downloaded today for en_US, cf. onboarding_data_knowledge_panels.dart
tmp.dart: Fixes to off-dart.

Impacted files:

knowledge_panel_page_template.dart: added parameters to 1. use the database (and downloaded data) and 2. limit the display to a given panel
knowlegde_panel_builder.dart: added an optional parameter to limit the display to a list of panels
local_database.dart: added new dao DaoString
next_button.dart: added an optional call to async methods at "next" time - e.g. data download
onboarding_flow_navigator.dart: now using the database to get more relevant (previoulsy downloaded) data
preferences_page.dart: now using previously downloaded data instead of assets
sample_eco_card_page.dart: now using previously downloaded data instead of assets, and displaying only the environment card
sample_health_card_page.dart: now using previously downloaded data instead of assets, and displaying only the health card
sample_product_json.dart: json data downloaded today for en_US, cf. onboarding_data_product.dart

This is what we display now between first 2 onboarding pages:

Thank you @jasmeet0817 for your code review and the questions it raised!
I cannot say that I totally disagree with you, but my opinion is that off-dart is supposed to give tools around off. Among the tools, some that download data into strings, some that convert those strings into objects.
To be continued... ;)


Add consul storage

Implements the storage interface for Consul key-value storage. Basicly a copy of the Redis storage interface and changed to support the consul go API.

I implemented a key prefix so that we could run multiple go-ipam against same key-value. Seeing https://github.com/metal-stack/go-ipam/pull/43 now I assume that should be removed in favor of an official support like that.

Hi @tobias-urdin
Thanks for the contribution ! Some thoughts from my side:

I would also suggest to wait for #43, i am still not very happy with the current solution howto calculate the namespace key for the different storage backends, but this will be sorted out soon
you must add the appropriate integration test infrastructure in testing_test.go, then all tests and benchmarks are executed against a real storage instance.
i have seen no optimistic lock protection in the UpdatePrefix func, i dont know what ACID guarantees consul gives, but i am quite sure this is required there as well. Look in redis,go for reference.
Last but not least some words in README.md should be added as well to the newly supported backend

Thanks for your feedback I will improve it based on that after the namespace support is done.
Best regards

Ops, wrong button :)

Will follow this up in another PR.


Bump files with dotnet-file sync

devlooped/oss

Add fallback GITHUB_TOKEN to bot defaults https://github.com/devlooped/oss/commit/5406d90
Add rate limiting fix https://github.com/devlooped/oss/commit/ef47d78
Use bot account for sync commit author https://github.com/devlooped/oss/commit/1d3a2f4
Use BOT_ defaults consistently https://github.com/devlooped/oss/commit/98919f7
Ensure checkout before defaults https://github.com/devlooped/oss/commit/721ee85
Rework to pass secrets explicitly https://github.com/devlooped/oss/commit/15e9486
Ensure both author and committer match https://github.com/devlooped/oss/commit/d94ddb1
Update to public bot defaults action https://github.com/devlooped/oss/commit/b9671b9
Resolve includes after file sync https://github.com/devlooped/oss/commit/8f45cf2
Remove the source link in the package description https://github.com/devlooped/oss/commit/b9fb0a7
Revert "Add default package source mapping for central package versions" https://github.com/devlooped/oss/commit/1537aa9
Ignore xUnit1013 so records can be used as test classes https://github.com/devlooped/oss/commit/369cd2b
Use simple bash variable expansion https://github.com/devlooped/oss/commit/acedd1d

devlooped/.github

Replace existing script with new reusable marketplace action https://github.com/devlooped/.github/commit/4108ce6

 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.


No files logged in S3 for lambda, but no errors thrown

In our lambdas we can't get S3 logging to work, tried all possible options. The code works when run locally with dotnet-lambda-test-tool-6.0, as well as from ECS service, but not from lambda.
Deploying directly from Visual Studio 2022 using AWS Toolkit,, Publish to AWS Lambda option
Attaching dummy lambda with sample code (.NET 6).
DummyLambdaUncommon.zip

specifying Time Zone on the lambda did it:
"TZ": {
"Type": "String",
"Default": "America/Chicago"
}

specifying Time Zone on the lambda did it:
"TZ": { "Type": "String", "Default": "America/Chicago" }

Sorry for the late reply. Great that you found it.


request - internetradio - display actual track name

is it possible to add the display of track name while playing internet radio?
would be a great feature.

Currently not feasible.


Fix models with compressed textures

The bug and fix were reported on the forum:
https://groups.google.com/forum/#!topic/cesium-dev/VcBGxhL5yzk

Thanks!


How to config smtp?

hi i dont know how to config smtp :|

Do you know how to configure SMTP on microkube?

No need, Figured out https://github.com/rubykube/microkube/blob/master/config/app.yml#L34


double eth1data params

Ensure the cycle of eth1data voting (follow distance + voting period) is more than a sleep cycle (~8 hours) to more easily allow for intervention/recovery in the event of an issue (attacker or bug).
Rational on the UX degradation: The time it takes to go from eth1 deposit to being inducted as a validator is already perceptibly long to the user and must be due to the security requirements. Doubling this time does not alter the UX much in the grand scheme of things, but provides a much better security profile.
Also, it is worth noting that the expected time to be inducted as a validator is still very small in comparison to the expected validation time for Phase 0 validators (1+ year).
cc: @vbuterin

Ok


Truncate HTML case table

@grecine is it possible to truncate the HTML case tables you generate such that the last row of the table is 3/19/2020, the first reported case in town?

fixed in 99a7d54c2737af2cfe428cd9d0cea53d15d5a227
New feature: this cutoff date can be set via an env var called FILE_START_DATE, in string format YYYYMMDD. 20200319 is the default
https://github.com/rutherford-nj/data-committee.covid-data/blob/28ba9079d6d16df858c1e7abc5a780f0375c2fcf/lib/defaults.py#L14


Bug: Custom CloudWatch metrics not always appearing for split route(s)

Expected Behaviour
When uploading a custom metric using the aws_lambda_powertools.metrics package, the metrics should appear in CloudWatch Metrics.
Current Behaviour
Metrics appear intermittently and appear to be tied to whether a lambda was cold started (and also had a cold start metric uploaded).
Code snippet
Split route handler:

metrics: Metrics = Metrics()

@router.delete("/v1/asset", cors=True)
@tracer.capture_method
def delete_asset() -> dict:
    metrics.add_metric(name="AssetDeleted", unit=MetricUnit.Count, value=1)
    return {"message": "Asset deletion successful"}

Lambda handler:

@metrics.log_metrics(capture_cold_start_metric=True)
@logger.inject_lambda_context(correlation_id_path=correlation_paths.API_GATEWAY_REST, log_event=True)
@tracer.capture_lambda_handler
@auth.enforce_allow_list
@auth.restrict_api_key
def lambda_handler(event: Dict, context: LambdaContext):
    return app.resolve(event, context)

Possible Solution
When examining the cloudwatch logs, the unsuccessful metrics appear in the same log line as the tracer log, for example:
{
    "resource": "/v1/asset",
    "path": "/v1/asset",
    "httpMethod": "DELETE",
...more data...
}{
    "_aws": {
        "Timestamp":<PHONE_NUMBER>055,
        "CloudWatchMetrics": [
            {
                "Namespace": "MyNamespace",
                "Dimensions": [
                    [
                        "service"
                    ]
                ],
                "Metrics": [
                    {
                        "Name": "AssetDeleted",
                        "Unit": "Count"
                    }
                ]
            }
        ]
    },
    "service": "MyLambda",
    "AssetDeleted": [
        1
    ]
}

When successful metrics appear (such as when they are uploaded as part of a cold start), they appear in their own log line in CloudWatch and are presumably parsed.
Steps to Reproduce

Create a split route lambda handler, log cold starts
In the split route(s) attempt to create/upload custom metrics
Observe not all metrics appear in CloudWatch metrics as expected

AWS Lambda Powertools for Python version
latest
AWS Lambda function runtime
3.6
Packaging format used
PyPi
Debugging logs
No response

Thanks for looking into this! I have noticed that I have lambda context data appearing twice, once as a log item as you do, and then a second time clobbered with the metrics data, so I think there might be a dependency interfering. I'll try to get to the bottom of this and confirm.

Thanks to reply.
Let me know if I can help in any way to detect if this is indeed a bug in Powertools. If so, we are fully interested in fixing it.

Ok, I believe I found the issue. The extra JSON appearing in the logs was because I had "debug" enabled on the APIGatewayResolver object, causing this line to be executed:
       if self._debug:             print(self._json_dump(event), end="")
This gets combined with the metrics output and then ignored by CloudWatch metrics. A member on our team was able to confirm setting debug mode to FALSE results in metrics appearing as expected.
I'm not sure why this specific print behaves this way, maybe because a newline is being omitted? It's a small change to test, so I'll try it out. Regardless I think it's an unexpected interaction.


Refactor(eos_designs): Wildcard dict to list l3_edge

"Wildcard Keyed Dict" to "List of Dicts" Data model migration

Data model key
l3_edge
Checklist
Contributor Checklist

[x] Update README_v4.0.md with new data model
[x] Update templates/eos/< template >.j2 and templates/documentation/< template >.j2:

[x] Add arista.avd.convert_dicts('<primary key>') filter for loops previously using wildcard keys
[x] Update arista.avd.natural_sort('<primary key>') to sort on the primary key (if applicable)

[x] Verify no changes to generated configs/docs

Reviewer Checklist

Reviewer 1: (Claus)

[x] Verify data model changes
[x] Verify no changes to configs/docs on any molecule scenario
[x] Verify that CI pass

Reviewer 2: (Tony)

[x] Verify data model changes
[x] Verify no changes to configs/docs on any molecule scenario
[x] Verify that CI pass

We need to add molecule scenario for this as well.

We will add CI coverage in a separate PR. Issue #1792.


Add scripts used for RPi energy analysis

Add scripts used for Raspberry Pi energy analysis.
The R Markdown script was done jointly with @w268wang.
@w268wang @daemon please review?

LGTM

Looks good.


FPS issues.

So I downloaded the new osiris update and compiled it 2 days ago, and my CPU usage is going for like 70%-80%, my specs are:
8 GB RAM
IGD Core i5-3330 @ 3.00 GHz
64 bit OS.

@NexSqaud  can u helpme find the avx option in build/project settings.
i cant seem to find it.


Merging Develop into Master

Codecov Report

Merging #13 into master will increase coverage by 5.77%.
The diff coverage is 53.34%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   66.75%   72.51%   +5.77%
==========================================
  Files           1        1
  Lines         454      451       -3
==========================================
+ Hits          303      327      +24
+ Misses        151      124      -27

Impacted Files
Coverage Δ

Travis/travis_uptimerobot_monitor_utility.sh
72.51% <53.34%> (+5.77%)
:arrow_up:

Continue to review full report at Codecov.

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


loadgen: Make server test duration depend on scheduled time.

Prior to this patch, it depended on the issue time which doesn't
match how the QPS metric is calcualted.

Genererates exactly the number of queries to issue.
Attempt to issue all queries generated.
Update MinDurationMet to depend on scheduled time.

This patch also fixes a bug where a "ran out of queries"
error would spuriously be reported in the corner case
that all queries were used but the minimum durations
happened to be met.

Ptal. This passes the existing unit tests.
It should also help with the effort @tejasprabhune is working on to make the scheduled QPS exactly equal the requested QPS.

Friendly ping. Just rebased this patch so there are not conflicts.


[MM-25757] - Add latest UI requirements to search hint popover

Summary
Latest acceptance criteria from UX regarding the hint popover. Issues seem to be fixed for me On desktop (latest version) and webapp (chrome and firefox)
Ticket Link
https://mattermost.atlassian.net/browse/MM-25757

Hey @nevyangelova does this resove this issue?

If I click the @ mentions icon it opens up the RHS with the title Recent Mentions. If I then clear the search field the content of the RHS remains the same (which is fine), but the title of the RHS changes to 'Search Results'. I think it's fine to leave the mentions content and title there until you actually enter a new search term. We shouldn't change the title to 'Search Results' until you actually press enter on a new search term

@matthewbirtch I will submit a different PR for that point since the Empty search results states didn't get into v5.24 and was pushed to v5.26 instead. You can check also in https://community-release.mattermost.com/ how the popover will look with the old search states.

Thank you @nevyangelova
One suggestion
Now that hint appears for the 3rd modifier, we should probably expand 1. point from the linked issue to include it
"When I try to add second or third modifier in the search input, the hint shouldn't go away until I fully type from: (including the colon) or any other modifier"

@jgilliam17 thank you for the observation, will do :)

Good call @jgilliam17. @nevyangelova I am still seeing that weird bug where sometimes I can't click the modifier in the search hint. NOTE: I clicked the from modifier here 3 times, but this is not the only case I'm experiencing this issue.

@matthewbirtch this happens on desktop app latest version? I tried on both MacOS and Windows and just can't reproduce this. I suspect it might have to do with one module 'electron-context-menu' that handles click events. There is a PR that upgrades that package https://github.com/mattermost/desktop/pull/1313 and should handle this better.

I am not able to reproduce modifier needing more than one click. Not reproducing on the latest desktop v4.5-rc1 for both macOS and Windows.

this happens on desktop app latest version?

Apologies for the lack of detail, I experienced this in Chrome in the Webapp actually - not the desktop app. I'll try restarting my machine or something and see if this gets resolved, but I seem to be reproducing consistently.

Thank you @nevyangelova
I'll wait for others to review before adding my approval, just in case I have to retest any changes.
Looks great so far!

Second and third modifiers can be added fully, including the colon, by typing & without hints going away too early.
Not able to repro needing multi click to select modifier on Chrome, Firefox, Microsoft Edge and desktop app (mac&win)
 Similar issue found on Safari has been fixed on Catalina, but on Mojave it still occurs intermittently, per discussion with Nevy - due to Safari having known problems with click handlers
Search hints can now come up and be added as a 3rd modifier
When RHS is closed, hint popover appears after search field is done expanding
When clearing search text input, field no longer tries to collapse and expand, x clears the field without trying to collapse it.

@nevyangelova
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#5647-upstream-release-5.24-1591613575
Branch 'automated-cherry-pick-of-#5647-upstream-release-5.24-1591613575' set up to track remote branch 'release-5.24' from 'upstream'.
+++ Downloading patch to /tmp/5647.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/5647.patch

Applying: - Add latest UI requirements to search hint popover
Applying: Fix tests
Applying: Fix click behaviour
Using index info to reconstruct a base tree...
M	components/search_bar/search_bar.jsx
Falling back to patching base and 3-way merge...
Auto-merging components/search_bar/search_bar.jsx
CONFLICT (content): Merge conflict in components/search_bar/search_bar.jsx
Patch failed at 0003 Fix click behaviour
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU components/search_bar/search_bar.jsx

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.


AKS: Internal Load Balancer setup?

Describe the bug
How to set up AKS internal load balancer to the api PODs?
To Reproduce
Steps to reproduce the behavior:
Set up the LB like this:

YAML file

apiVersion: v1
kind: Service
metadata:
  name: internal-app
  annotations:
    service.beta.kubernetes.io/azure-load-balancer-internal: "true"
spec:
  type: LoadBalancer
  selector:
    app: presidio-demo-presidio-api
  ports:
    - protocol: TCP
      port: 80
      targetPort: 8080

Applied the YAML file:

$ kubectl apply -f internal-lb.yml
service/internal-app created

Service is up:

$ kubectl get service internal-app
NAME           TYPE           CLUSTER-IP     EXTERNAL-IP   PORT(S)        AGE
internal-app   LoadBalancer   <IP_ADDRESS>   <IP_ADDRESS>    80:30953/TCP   81s

$ kubectl describe service internal-app
Name:                     internal-app
Namespace:                default
Labels:                   <none>
Annotations:              service.beta.kubernetes.io/azure-load-balancer-internal: true
Selector:                 app=presidio-demo-presidio-api
Type:                     LoadBalancer
IP:                       <IP_ADDRESS>
LoadBalancer Ingress:     <IP_ADDRESS>
Port:                     <unset>  80/TCP
TargetPort:               8080/TCP
NodePort:                 <unset>  30953/TCP
Endpoints:                <none>
Session Affinity:         None
External Traffic Policy:  Cluster
Events:
  Type    Reason                Age               From                Message
  ----    ------                ----              ----                -------
  Normal  EnsuringLoadBalancer  7s (x2 over 45m)  service-controller  Ensuring load balancer
  Normal  EnsuredLoadBalancer   7s (x2 over 44m)  service-controller  Ensured load balancer

Cannot curl from Azure cloud console to the external IP:

$ curl http://<IP_ADDRESS>
curl: (7) Failed to connect to <IP_ADDRESS> port 80: No route to host

If the set up the proxy through kubectl, I'm able to curl to the API:
$ kubectl port-forward -n presidio $(kubectl get pod -n presidio -l app=presidio-demo-presidio-api -o jsonpath='{.items[0].metadata.name}') 8080:8080
Forwarding from [::1]:8080 -> 8080
Handling connection for 8080

$ curl localhost:8080
404 page not found

Expected behavior
curl http://<IP_ADDRESS> should work from the cloud shell.

Hi @fd98279, we recently released a new version of presidio (aka V2) https://github.com/microsoft/presidio/discussions/584, also includes an update to the k8s deployment sample. Could u check if this is still an issue for you?

@fd98279  We recently released the new version on presidio V2, whcih includes updates to the deployment samples https://github.com/microsoft/presidio/tree/main/docs/samples/deployments
This issue will be closed. Feel free to re-open in case this is not resolved.


#14790: Extracting animation out of FadeInImage

Description
This is an alternate approach to #14790 (other PR is #57223). This is raised as an alternate PR to keep the ideas separate and facilitate discussion. I'm happy to close off either, as appropriate.
FadeInImage now has the animation strategy provided in a constructor parameter. This allows consumers of FadeInImage to decide how to fade between the placeholder and target image without requiring a class for each in the framework (which is effectively what #57223 is doing).
Related Issues

Issue #14790
PR (also raised by me) #57223 .

Tests
I added the following tests:

FadeInImage tests pass without modification, confirming backwards compatibility.
None, yet, for AnimatedFadeOutFadeIn. This will be completed after discussion.

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

I likes the idea of extracting out the fade in animation builder over the other one. This give us the option to expend different type of image fading

The analyzer is also not happy with this change.

I'm also happy to keep this as an external package. Raised here first to get an opinion on best home. The analyzer is complaining as not all the documentation/testing has been completed. I'll resolve that when we're in agreement about the approach :)

I am not really too happy with the API published by FadeInImageAnimationBuilder. Previously, this was all private, so it was fine. But if we make it a public API we should design something more useable.
From the top of my head, I don't have a great idea of how this should look like. So maybe keep this in an external package for now if you need it?

Hey @goderbauer, sounds good. I'll pull this out into a package and will update the related ticket (might allow you to close that off as well?)
Thanks for the input!


test team filter

test team filter in bot

@esmvaltooltestbot is this your name?

@esmvalbot is this your name?

 Would it be possible for it to give a nicer error message like 'got request from user outside of 'trigger_team'?

test

@esmvalbot hi

back in the team :sunglasses:

@esmvalbot hello again

@esmvalbot still there

@esmvalbot again I'm not a member

@esmvalbot did stefan fix you

@esmvalbot why are you not responding

@esmvalbot are you ignoring me

@esmvalbot playing hard to get

@esmvalbot this will make you talk to me

@esmvalbot recipe_test.yml

@esmvalbot recipe_try_again.yml

@esmvalbot recipe_example.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot invalid_recipe.yml

@esmvalbot invalid_recipe.yml

@esmvalbot example/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml examples/recipe_python2.yml examples/recipe_python3.yml

@esmvalbot examples/recipe_python.yml

@esmvalbOT exAMples/recIpe_python.yml

@esmvalbot exAMples/recIpe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml

@esmvalbot examples/recipe_python.yml


WIP: [PXN-2217][bugfix] fix the infinite alloc/repeat of screen content each viewWillAppear()

What have changed?
Added a new recursive method for correctly clear all UIView tree hierarchy of a single UIview. Also added two calls for it in setupScrollView() and renderContentView() in order to correctly remove all old instances before replace them with the new updated ones.
Kind of pr:

[x] BugFix
[ ] Feature
[ ] Improvement
[ ] Refactor

How to test:
Every time viewWillAppear() is called, a new set of UX objects is instanced over the previous ones. The change on setupScrollView() with a correct recursive removal of the view tree hierarchy solves the wrong stacking of buildFooterView(), and the second change on 'renderContentView()' solves the "super-stack" behavior, where several groups of repeated content were available for scrolling in the UIScrollView.
Prints for layout changes:
buildFooterView() stack and super-stack of content evidences attached on issue:
https://mercadolibre.atlassian.net/browse/PXN-2217
Extras:

Good morning everyone. @pedro-nadolny @pesilsa @ericertlMELI this PR is available for code-review. Could you please, if it doesn't take too much the time of you all, perform this code-review?
thanks! :)


docs: add contract information

add additional info on system and precompile contracts based on the resource currently available

hold

hold

unhold

https://github.com/axonweb3/axon-docs/blob/408b45a5a70783ffdcc532f87cf33221188bf172/docs/contract/system_contacts.md#L32
From the Axon code, I think the system contract address should be prefix with 0xFfFfFFFfFFfFFFFfFFFFffFfFFfffFFFfffffF.


Cannot install with Pypi

Hi,
thank you for your work!
When I want to install your pkg via pip install, I get following error:
Collecting cmask2polygons==1.1.1

Could not find a version that satisfies the requirement cmask2polygons==1.1.1 (from versions: )
No matching distribution found for cmask2polygons==1.1.1

Presumably it's because I am using Python v. 3.6.9. Why do you require >=3.7.5? Can you loosen the requirements?

Thanks for your feedback.
According to your request, I have changed the python_requires>=3.7.5 to python_requires>=3.6.5.
If you have any other problems, Feel free to ask.


Refactored functions into descriptive named files

Preparing for adding more tests. Refactored functions into a file with a more detailed/descriptive name.
Also added a few tests for functions in the helper file.

Thanks for that.
You must have in background the following things :

the code actually covers both chats and it is not clear that your new function files are only covering the longChat pane
the shortChat pane needs to be updated to include not many but some of the improvements made to the messages (this as to be discussed but I'm thinking of edit and delete, or reply but not threads or the message signature)
the chat-threads branch is the next longChat version. This branch is containing the changes needed for Threads and Matrix compatibility. with your refactoring the branches are incompatible

To increase clarity, I suggest retitling this PR to refer to descriptively — rather than  descriptive — named files

@bourgeoa thank you for pointing these out.  I indeed did not look at the chat-threads branch.  I will close this out for now until chat-threads is ready. We can discuss tomorrow in case there is something else I can do to add tests perhaps to the chat-threads? Otherwise I can work on some of the other issues assigned to me.


m4: sync autoconf-archive (ax_*) macros with upstream

Update the various ax_* macros (which originate from autoconf-archive)
Switch from acx_pthread.m4 -> ax_pthread.m4

Fixes: https://github.com/mm2/Little-CMS/issues/339
Signed-off-by: Sam James<EMAIL_ADDRESS>

I don't think there's any harm in the additional stuff this readds given that it only affects tests you actually check the #defines for anyway.


Run utPLSQL tests option disabled in SQL Developer

utPLSQL version: v<IP_ADDRESS>2
utPLSQL-SQLDeveloper plugin Version: 0.2.1
Database Version: Oracle Database 12c Enterprise Edition Release <IP_ADDRESS>.0 - 64bit
Problem:
“Run utPLSQL test” option went disabled
Troubleshooting:

Uninstall, and install the utplsql plugin, and restart SQL Developer tool.
Uninstall, and install the SQL Developer tool & utplsql plugin, and restart SQL Developer tool.

But the problem still persists. Can you help us with necessary steps to fix the issue?

Does you connection name contain spaces? If yes, then you are hiitting bug 16. This is bug is fixed in the current snapshot version. I suggest to use this version.

It works now after using latest version 0.3.0 snapshot
Thanks

Thank you for the confirmation. I close this issue.


godep fails on trying to find std lib package

 → godep save ./...
godep: cannot find package "strings" in any of:
	/usr/local/Cellar/go/1.5.1/libexec/src/strings (from $GOROOT)
	/Users/pdeuchler/Dropbox/workspace/gocode/src/strings (from $GOPATH)

godep v34 (darwin/amd64/go1.5.1)
go version go1.5.3 darwin/amd64

Please upgrade to v53 and let me know if this is still an issue.
Thanks!

Fixed! Thanks!


[7.0] Remove * as a plugin option

Not a good idea right now because spec changes and don't want to break stuff. We'll want to figure out something different. This is useful for babel-eslint/astexplorer, etc. cc @fkling
If you look at the plugin options: https://github.com/babel/babylon#plugins
You can add them to an array like
{
  plugins: ["jsx", "flow", "dynamicImport"
}

We thought it might be convenient for certain tools (not apps) to have an option of always getting the latest spec plugins (astexplorer.net). However I think we probably want a better way of doing this.

[ ] grep for * in the codebase. Remove hasPlugin('*') type code + associated tests.

Hi, I'd like to work on this issue, if that's ok

@micopc Perfect! It's all yours.


[Feature Request] Add ability to hide text for the use of storing extra information

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
Sometimes it would be useful to store extra information not presented to the user in the fzf window.
For example, I have a script that uses fzf to list the windows in my x session and to focus a window upon selection.
However this script is reliant on the window title to operate and it is plausible for two windows to have the exact same title.
A solution would be to include the window id with the title, but this is not information that the user really needs and is only useful programmatically.
My proposed solution is to include a --hidden-text cli option allowing the user to give a delimiter that would hide all text before said delimiter.
For example: echo "this text is hidden ; this text is displayed" | fzf --hidden-text=';'
Once selected the full text would the printed to stdout for the user to parse in whichever way he wished.

Have you tried using --with-nth?

This is exactly what I need.
How did I  miss that?
Thank you


chore(deps): upgrade dependencies and runtime to Node.js 20

also add .gitignore file for node modules, Intellij IDEA and VScode
fixes #28

@varunsridharan would this work? If so, could you release it?

@varunsridharan ping... any chance to get this released?

Pinging again...

Hoping this ping helps a little bit. Perhaps it's time to swap to a fork?

Hello @varunsridharan any intentions to do this?

Now the main branch is broken, see the error below:
/home/runner/work/_actions/varunsridharan/action-github-workflow-sync/main/node_modules/@octokit/plugin-throttling/dist-node/index.js:172
    throw new Error(`octokit/plugin-throttling error:
^
Error: octokit/plugin-throttling error:
        You must pass the onSecondaryRateLimit and onRateLimit error handlers.
        See https://octokit.github.io/rest.js/#throttling

        const octokit = new Octokit({
          throttle: {
            onSecondaryRateLimit: (retryAfter, options) => {/* ... */},
            onRateLimit: (retryAfter, options) => {/* ... */}
          }
        })

Hello everyone,
Due to personal reasons and health issues, I am currently unable to work on my open-source projects. I truly appreciate all the support and contributions from this amazing community; your involvement has been invaluable in driving these projects forward.
If anyone is interested in collaborating or even taking over one of these projects, please reach out to me directly. I’d be happy to discuss adding someone who can continue building and supporting these tools. Thank you all again for your understanding and support during this time.
Best regards,
Varun Sridharan
E-Mail<EMAIL_ADDRESS>@steveteuber @patrickc91 @mgerlach

@steveteuber @smlgbl @varunsridharan see #31


KLEE: treat linking errors as tool errors not SUT errors

Tool errors are reported as UNKNOWN
while SUT errors are true errors reported as ERROR

LGTM


[Bug]: Console metadata: Renaming a metadata key does not delete the old key

Preflight Checklist

[x] I could not find a solution in the documentation, the existing issues or discussions
[x] I have joined the ZITADEL chat

Environment
ZITADEL Cloud
Version
No response
Database
None
Database Version
No response
Describe the problem caused by this bug
When editing the metadata for a resource like a user in the console, if the key for an entry is changed and then saved, instead of modifying the entry, the console saves a new copy of the entry under the new key while keeping the old entry, leaving duplicate data.
To reproduce

Edit the metadata for a resource like a user.
Create a metadata entry with key "Foo" and value "1" and save the new entry.
Change the key "Foo" to "Bar" and save your changes.
Observe that the edit popup only shows "Bar" with "1" and not "Foo"
Close the edit popup and observe that "Foo" -> "1" and "Bar" -> "1" are both present.

Screenshots

Expected behavior
Renaming a key will delete the old key.
Operating System
No response
Relevant Configuration
No response
Additional Context
No response

I think that's because the API only allows setting a value by it's key. Changing the key means a new entry.
@peintnermax we should not allow changing the key in the UI. Just the value is changeable. If a user needs to change the key, they should delete the old entry and create a new one.

makes sense, I've estimated the issue at .5 days and moved it to bugs


Api Rest

Hello,
This application can work as an AP REST....?
Thanks

If you meant "REST API", Sure, it can.

Excellent.....
Thanks...


[release-1.4] fix bool unmarshal issue.

This is an automated cherry-pick of #438

@googlebot I consent.


--attach-acr is inconsistent

Depending on using the --attach-acr parameter with az aks create or az aks update we're supposed to use registry ID or only the name. For both the parameter description points to using the registry ID but the example of az aks update says to use the name of acr.

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

ID: d75c3a8d-1187-bea1-a7d4-7ebba71c1790
Version Independent ID: 75b0c352-762e-16ae-5cc6-d807ffc1dc3f
Content: az aks
Content Source: src/azure-cli/azure/cli/command_modules/acs/_help.py
Service: container-service
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@jacekmackiewicz , thanks for reporting, we are looking at it.

in both the instance the description is : Grant the 'acrpull' role assignment to the ACR
specified by name or resource ID.
Can you point to the specific example? Also what version of CLI are you on?

Close issue.


Query params are not set on relationship when reloading page

I noticed in the ModelMixin that the relationship is not reloaded if it is not yet loaded.
    //if the hasMany content is already loaded, we must reload it
    if (value.get('content').get('isLoaded')) {
      value = value.reload();
    }

In my use case I have a route that is loading the model but on the very first load of the model the route wants to pass query params for when the hasMany relationship is fetched (the hasMany relationship looks like it is being called automatically when I display the relationship data in my template).
I was trying to do something like this in the tag-entity-pair model hook of my router:
    return this.store.findRecord('tag-entity-pair', params.tag_entity_pair_id, {adapterOptions: {query: params}}).then(function(tagEntityPair){
      // force the first fetch of the relationship to set the sort value of the relationship being fetched
      return tagEntityPair.query('comments', {sort: 'comment.last-edited'}).then(function(comments){
        return tagEntityPair;
      });
    });

However, because the comments have not been loaded at the time I run tagEntityPair.query the values are not reloaded and no query params are used.
Is there a way to use the query params on the very first load of the relationship if it's being loaded from the model hook of the router?
Apologies if this question makes no sense as I'm still trying to wrap my head around Ember Data (I have a feeling I might be setting up something wrong here).
Thank you for your effort on this plugin!

The reason for the value.reload() is, if the hasMany relationship data is already loaded, Ember Data will not reload from the backend again. If the relationship has not yet been loaded, the query method will automatically request data from the server, and therefore reload is not needed.
I suspect something else is going on. What does your tag-entity-pair model look like?
You could try returning the tag-entity-pair and the comments separately in the model hook (and change the template accordingly):
    return this.store.findRecord('tag-entity-pair', params.tag_entity_pair_id, {adapterOptions: {query: params}}).then(function(tagEntityPair){
      // force the first fetch of the relationship to set the sort value of the relationship being fetched
      return tagEntityPair.query('comments', {sort: 'comment.last-edited'}).then(function(comments){
        return {
          tagEntityPair: tagEntityPair,
          comments: comments
        };
      });
    });

@mdehoog Thank you for the information on value.reload().  That makes a lot of sense.  Here are the relevant parts of my two models:
// models/tag-entity-pair.js
export default DS.Model.extend(HasManyQueryModelMixin, {
  comments: DS.hasMany('comment')
});

// models/comment.js
export default DS.Model.extend(HasManyQueryModelMixin, {
  tagEntityPair: DS.belongsTo('tag-entity-pair'),
});

I added some output statements your two mixins and it seems like the following events are happening:
MODEL MIXIN: Enter
model.js:29 MODEL MIXIN: Set _commentsQueryParams to:
model.js:30 Object {tag_entity_pair_id: "6", sort: "-comment.last-edited", page: Object}
model.js:42 MODEL MIXIN: Setting _commentsQueryParams to undefined

Refresh the page
Model Mixin gets called
The model mixin is retrieving the query params correctly

{
   sort: comment.last-edited
}

The model mixin sets _commentsQueryParams properly
The content is not already loaded so we don't reload it
value.finally returns and _commentsQueryParams is set to undefined
The Adapter mixin runs
The Adapter retrieves _commentsQueryParam and it is undefined
The Adapter calls this.ajax with no query params since it was undefined

Based on the steps above, the ModelMixin appears to be setting the value of the comments query params to undefined before the AdapterMixing has a chance to run.

Thanks for the excellent analysis. Which version of Ember Data are you using? Does it help if you explicitly set async: true in your DS.hasMany options hash?

Here are the versions I'm running:
Ember             : 2.1.0
Ember Data        : 2.1.0
jQuery            : 1.11.3

Setting {async:true} explicitly doesn't seem to change anything.  The Adapter calls are still executed after the model mixin has set the query params back to undefined

Just updated to latest and still having the issue.  Will continue to play around with it and see if I can get any further with the debugging.
Ember             : 2.2.0
Ember Data        : 2.2.1
jQuery            : 1.11.3

I'm having trouble reproducing this issue. Any chance of a cut down example that demonstrates the issue?
The model mixin query function calls get for the passed hasMany attribute, which should trigger a the adapter to fetch the data, appending the query params to the URL. Here is my call stack:

You can see that the query function (currently selected) calls this.get(propertyName);, which ends up in this addon's rest-adapter mixin's findHasMany function. This adds the query parameters to the URL and calls ajax before the value.finally clears the query params. Perhaps you could set a breakpoint in the rest-adapter mixin findHasMany function and include your call stack here?

Thank you for investigating the issue.  I set breakpoints in both model.js where the _queryParamProperty gets set to null
//model.js
self.set(_queryParamPropertyName, undefined);

and then in the rest-adapter where the queryPramPropertyName is read back
//rest-adapter.js
var queryParams = snapshot.record.get(queryParamPropertyName(relationship.key));

and for whatever reason in my case the queryParam is set to undefined before the rest-adapter grabs the value.
My call stack looks identical to yours.  I'm thinking it must be the way I'm trying to load the relationship in the model:
    return this.store.findRecord('tag-entity-pair', params.tag_entity_pair_id).then(function(tagEntityPair){
      return tagEntityPair.query('comments', params).then(function(comments) { //model.js query gets called here and the promise returns and sets the parameter to undefined
        return tagEntityPair; // this gets returned before any rest-adapter calls are made
      });
    });

I'll see if I can recreate it in a jsbin.

I think I've run into this same problem. Has anyone found a solution?

Sorry I'm still having the issue.  We've just lived with it for now until we have some cycles to look into it again:
We're on the following versions now:
Ember             : 2.3.1
Ember Data        : 2.3.3
jQuery            : 2.1.4

@sarus An update to the latest version of this addon and to ember-data >= 2.5 may help, I have switched the reloading logic to use the new ds-references implementation.
Note that this addon only works with an async HasMany relationship that is included in the parent model server response in the links attribute, e.g.
{
    tag-entity-pair: {
        ...
        links: {comments: "comments"}
    }
}

Thank you for the info.  I'll get our app updated to 2.5 and report back.

I found a bug which is similar to this issue in that, when making two queries in quick succession, the first query is aborted and then query parameters are cleared, which causes them to be cleared for the second query too. I've fixed this issue in v0.1.5 of this addon; it may help with this issue.


Provide Hashable instances for Vectors

Instances for boxed, unboxed, storable and primitive vectors as well as pure streams.

Oops, I just noticed there was a no-dependency rule (#13). Please feel free to reject this, then I'll release a separate package containing orphan instances.

I should have probably stated that rule more clearly somewhere. I hope you didn't spend too much time on this.
Here's what I want to do: I want text and bytestring, text, and vector to depend on hashable and for hashable to have no dependencies but base. That's a bit tricky though as bytestring and text are in the platform but hashable isn't. I'm not sure how to cut this Gordian knot.