Internal Python Language Server error

Description
What steps will reproduce the problem?

After starting a new Spyder session, this error pops up!
Traceback
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls_jsonrpc/endpoint.py", line 113, in consume
    self._handle_request(message['id'], message['method'], message.get('params'))
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls_jsonrpc/endpoint.py", line 182, in _handle_request
    handler_result = handler(params)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls_jsonrpc/dispatchers.py", line 23, in handler
    return method(**(params or {}))
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls/python_ls.py", line 275, in m_text_document__hover
    return self.hover(textDocument['uri'], position)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls/python_ls.py", line 216, in hover
    return self._hook('pyls_hover', doc_uri, position=position) or {'contents': ''}
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls/python_ls.py", line 122, in _hook
    return hook_handlers(config=self.config, workspace=self.workspace, document=doc, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/hooks.py", line 289, in __call__
    return self._hookexec(self, self.get_hookimpls(), kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/manager.py", line 68, in _hookexec
    return self._inner_hookexec(hook, methods, kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/_tracing.py", line 80, in __call__
    return outcome.get_result()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/callers.py", line 80, in get_result
    raise ex[1].with_traceback(ex[2])
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/callers.py", line 52, in from_call
    result = func()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/_tracing.py", line 78, in <lambda>
    outcome = _Result.from_call(lambda: self.oldcall(hook, hook_impls, kwargs))
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/manager.py", line 62, in <lambda>
    firstresult=hook.spec.opts.get("firstresult") if hook.spec else False,
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/callers.py", line 208, in _multicall
    return outcome.get_result()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/callers.py", line 80, in get_result
    raise ex[1].with_traceback(ex[2])
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pluggy/callers.py", line 187, in _multicall
    res = hook_impl.function(*args)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/pyls/plugins/hover.py", line 10, in pyls_hover
    definitions = document.jedi_script(position).goto_definitions()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/api/__init__.py", line 220, in goto_definitions
    definitions = helpers.evaluate_goto_definition(self._evaluator, context, leaf)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/api/helpers.py", line 143, in evaluate_goto_definition
    return evaluator.goto_definitions(context, leaf)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 230, in goto_definitions
    return eval_expr_stmt(context, def_, name)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 48, in wrapper
    return func(context, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 284, in eval_expr_stmt
    return _eval_expr_stmt(context, stmt, seek_name)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 300, in _eval_expr_stmt
    context_set = context.eval_node(rhs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 145, in eval_node
    return self.evaluator.eval_element(self, node)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 200, in eval_element
    return self._eval_element_if_evaluated(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 212, in _eval_element_if_evaluated
    return self._eval_element_cached(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/cache.py", line 40, in wrapper
    rv = function(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 216, in _eval_element_cached
    return eval_node(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 48, in wrapper
    return func(context, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 97, in eval_node
    context_set = eval_trailer(context, context_set, trailer)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 183, in eval_trailer
    return base_contexts.execute(args)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 65, in mapper
    for context in self._set
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 34, in from_sets
    sets = list(sets)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 65, in <genexpr>
    for context in self._set
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 62, in execute
    context_set = func(arguments)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/context/function.py", line 73, in py__call__
    return self.infer_function_execution(function_execution)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/context/function.py", line 95, in infer_function_execution
    return function_execution.get_return_values()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/cache.py", line 40, in wrapper
    rv = function(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/recursion.py", line 87, in wrapper
    result = func(self, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/context/function.py", line 176, in get_return_values
    context_set |= self.eval_node(children[1])
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 145, in eval_node
    return self.evaluator.eval_element(self, node)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 200, in eval_element
    return self._eval_element_if_evaluated(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 212, in _eval_element_if_evaluated
    return self._eval_element_cached(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/cache.py", line 40, in wrapper
    rv = function(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 216, in _eval_element_cached
    return eval_node(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 48, in wrapper
    return func(context, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 85, in eval_node
    context_set = eval_atom(context, first_child)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 202, in eval_atom
    search_global=True
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/_compatibility.py", line 274, in <lambda>
    return lambda *args, **kwargs: self.func(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 162, in py__getattribute__
    return f.find(filters, attribute_lookup=not search_global)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/finder.py", line 67, in find
    types = self._names_to_types(names, attribute_lookup)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/finder.py", line 178, in _names_to_types
    contexts = ContextSet.from_sets(name.infer() for name in names)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 34, in from_sets
    sets = list(sets)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/finder.py", line 178, in <genexpr>
    contexts = ContextSet.from_sets(name.infer() for name in names)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/filters.py", line 102, in infer
    return tree_name_to_contexts(self.parent_context.evaluator, self.parent_context, self.tree_name)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 555, in tree_name_to_contexts
    types = _remove_statements(evaluator, context, node, tree_name)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 497, in _remove_statements
    return eval_expr_stmt(context, stmt, seek_name=name)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 48, in wrapper
    return func(context, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 284, in eval_expr_stmt
    return _eval_expr_stmt(context, stmt, seek_name)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 300, in _eval_expr_stmt
    context_set = context.eval_node(rhs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 145, in eval_node
    return self.evaluator.eval_element(self, node)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 200, in eval_element
    return self._eval_element_if_evaluated(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 212, in _eval_element_if_evaluated
    return self._eval_element_cached(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/cache.py", line 40, in wrapper
    rv = function(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/__init__.py", line 216, in _eval_element_cached
    return eval_node(context, element)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 48, in wrapper
    return func(context, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 97, in eval_node
    context_set = eval_trailer(context, context_set, trailer)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/syntax_tree.py", line 183, in eval_trailer
    return base_contexts.execute(args)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 65, in mapper
    for context in self._set
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 34, in from_sets
    sets = list(sets)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 65, in <genexpr>
    for context in self._set
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/debug.py", line 81, in wrapper
    return func(*args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 62, in execute
    context_set = func(arguments)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/context/function.py", line 73, in py__call__
    return self.infer_function_execution(function_execution)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/context/function.py", line 95, in infer_function_execution
    return function_execution.get_return_values()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/cache.py", line 40, in wrapper
    rv = function(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/recursion.py", line 87, in wrapper
    result = func(self, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/context/function.py", line 156, in get_return_values
    context_set = docstrings.infer_return_types(self.function_context)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/cache.py", line 40, in wrapper
    rv = function(obj, *args, **kwargs)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/base_context.py", line 277, in wrapper
    return ContextSet.from_iterable(func(*args, **kwargs))
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/common/context.py", line 20, in from_iterable
    return cls.from_set(set(iterable))
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/docstrings.py", line 305, in infer_return_types
    for type_str in search_return_in_docstr(function_context.py__doc__()):
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/docstrings.py", line 302, in search_return_in_docstr
    for type_ in _search_return_in_numpydocstr(code):
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/jedi/evaluate/docstrings.py", line 83, in _search_return_in_numpydocstr
    doc = _get_numpy_doc_string_cls()(docstr)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/numpydoc/docscrape.py", line 153, in __init__
    self._parse()
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/numpydoc/docscrape.py", line 408, in _parse
    self['See Also'] = self._parse_see_also(content)
  File "/anaconda3/envs/pmeal/lib/python3.7/site-packages/numpydoc/docscrape.py", line 330, in _parse_see_also
    raise ParseError("%s is not a item name" % line)

numpydoc.docscrape.ParseError: functions. See the 'hybr' `method` in particular. is not a item name in "Find the roots of a function.\n\nReturn the roots of the (non-linear) equations defined by\n``func(x) = 0`` given a starting estimate.\n\nParameters\n----------\nfunc : callable ``f(x, *args)``\n    A function that takes at least one (possibly vector) argument,\n    and returns a value of the same length.\nx0 : ndarray\n    The starting estimate for the roots of ``func(x) = 0``.\nargs : tuple, optional\n    Any extra arguments to `func`.\nfprime : callable ``f(x, *args)``, optional\n    A function to compute the Jacobian of `func` with derivatives\n    across the rows. By default, the Jacobian will be estimated.\nfull_output : bool, optional\n    If True, return optional outputs.\ncol_deriv : bool, optional\n    Specify whether the Jacobian function computes derivatives down\n    the columns (faster, because there is no transpose operation).\nxtol : float, optional\n    The calculation will terminate if the relative error between two\n    consecutive iterates is at most `xtol`.\nmaxfev : int, optional\n    The maximum number of calls to the function. If zero, then\n    ``100*(N+1)`` is the maximum where N is the number of elements\n    in `x0`.\nband : tuple, optional\n    If set to a two-sequence containing the number of sub- and\n    super-diagonals within the band of the Jacobi matrix, the\n    Jacobi matrix is considered banded (only for ``fprime=None``).\nepsfcn : float, optional\n    A suitable step length for the forward-difference\n    approximation of the Jacobian (for ``fprime=None``). If\n    `epsfcn` is less than the machine precision, it is assumed\n    that the relative errors in the functions are of the order of\n    the machine precision.\nfactor : float, optional\n    A parameter determining the initial step bound\n    (``factor * || diag * x||``).  Should be in the interval\n    ``(0.1, 100)``.\ndiag : sequence, optional\n    N positive entries that serve as a scale factors for the\n    variables.\n\nReturns\n-------\nx : ndarray\n    The solution (or the result of the last iteration for\n    an unsuccessful call).\ninfodict : dict\n    A dictionary of optional outputs with the keys:\n\n    ``nfev``\n        number of function calls\n    ``njev``\n        number of Jacobian calls\n    ``fvec``\n        function evaluated at the output\n    ``fjac``\n        the orthogonal matrix, q, produced by the QR\n        factorization of the final approximate Jacobian\n        matrix, stored column wise\n    ``r``\n        upper triangular matrix produced by QR factorization\n        of the same matrix\n    ``qtf``\n        the vector ``(transpose(q) * fvec)``\n\nier : int\n    An integer flag.  Set to 1 if a solution was found, otherwise refer\n    to `mesg` for more information.\nmesg : str\n    If no solution is found, `mesg` details the cause of failure.\n\nSee Also\n--------\nroot : Interface to root finding algorithms for multivariate\nfunctions. See the 'hybr' `method` in particular.\n\nNotes\n-----\n``fsolve`` is a wrapper around MINPACK's hybrd and hybrj algorithms.

I doubt that's the case. This error occurs when you're hovering (i.e. putting your mouse on top of) a particular piece of code in the editor.
Please post a the code that generates this error so we can fix it in our side. Thanks!

So far, I haven't managed to reproduce the bug. I'll keep you posted though.

Great, thanks for the follow up!


'large' size markers should not be scale: 1.5

Icons should never be scaled above 1.0. Nobody wants a pixelated icon.

hit in https://github.com/mapbox/simplespec-to-gl-style/pull/23


IDFix tool _Offline installation

HI,
Is there any option to run the tool in offline mode with out connecting first to the internet?
Thanks.

Hi,
The setup is only used to check the prerequisites.
It is possible to pass parameters to it (do "setup -h" to display them).
I downloaded the files to my computer, then I copied them to a server that did not have internet access. By setting the various parameters to point them to the directory in which I had deposited the files, the prerequisite test is passed.
On the other hand afterwards it tries to run "IdFix.application" to make the effective installation and that I did not manage to make it work.
But by directly executing "IdFix.exe", it works.
Regards,
Fred

dont use the single launch setup.exe..  download the full zip package run direct from ..\idfix-master\publish\Application Files\IdFix_2_3_0_0\idfx.exe

An MSI is available for install to those that cannot run the ClickOnce application:  https://github.com/microsoft/idfix/tree/master/MSIs


I cannot kill this process on centos7, sending SIGKILL and process comes back up.
How do I termina this process gracefully?

Don't terminate ethminer process. Watchdog raise the process again.
Need to terminate the watchdog.

Which one is Watchdog process? I cannot find it :)

This is it right?

Under normal circumstances, you should only see one ETHminerWatchDogDmW process.


Matter*PluginServerInitCallback are NOT guaranteed to be called only once

Problem
The Matter*PluginServerInitCallback functions are implemented as if they are called once per process lifetime.  But that's not correct.  In particular, any time DeviceController::Init is called (which can happen during process lifetime due to shutting down an existing DeviceController and starting a new one), it will call InitDataModelHandler, which will call emberAfInit (if USE_ZAP_CONFIG is defined along with use of DeviceController), which does MATTER_PLUGINS_INIT.  Which will call these callback functions try and register their attribute access callbacks.
Unfortunately, what that will lead to is loops in the linked list, and presumably infinite loops during attr reads after that.
Proposed Solution
Sort out sane startup/shutdown semantics.  Presumably InitDataModelHandler should have a shutdown parallel that cleans up things as needed....

@vivien-apple

Or we should guard these "will only be called once" callbacks on something that actually ensures that in emberAfInit

Should this be targeted for TE7.5 @bzbarsky-apple ?

I think this is a bugfix we can do after 7.5.


Key Pinning

Allow users to pin keys to a specific domain or domain+path

we should probably have an option to mangle the current url into a wildcard or regex, and allow the user to specify their own wildcard and regex patterns

Yep, totally agree. (This may be a good task for you to start on if you want - fairly straightforward, but would let you touch quite a bit of code.)

shit, did not intend to close the issue; I will let you know if I think I can get this done in a timely manner, for now do not let me hold up https://github.com/winhowes/grd.me/milestones/v4

cool, no worries. Milestone v4 is far off, still working on 3 for the next few weeks

This issue was moved to GrdMe/GrdMe#28

This issue was moved to GrdMe/GrdMe#28


Logged-in facebook capturing works on webrecorder.io, but not on the local pywb-webrecorder

Locally, a 'Please Log In' alert box is shown constantly when recording locally.
This might be because facebook checks for https protocol?

Fixed from pywb >= 0.8.3, issue was with secure cookies, which would be dropped in http replay.
For now, dropping the secure on replay/proxy


Force positive sign

I just can't find a way to force the positive sign prefix. Optometrists use negative and positive numbers in their prescriptions and I want to force the positive sign when numbers are positive.

Number formatter doesn't provide such option, but we can accept such enhancement if you're willing to implement it. In such case, I'd recommend discussing the API (option name and high level functionality) either in this issue (or in #437), then go for PR.
Thanks!
Cross links:

https://www.unicode.org/reports/tr35/tr35-numbers.html#Explicit_Plus
https://github.com/globalizejs/globalize/issues/437#issuecomment-160496314

Dup of #437


WP POT Support

I love gulp, use it for my plugin builds, and just found this repo.
In Popup Maker and its extensions we use gulp-wp-pot to automatically scan and update a translation.pot file.
I believe I got a few tips from your first guide for our routines, but mine have a few more things I have added for extra awesomeness.
A few things you could merge here would be
POT Langpack creation - Watcher on **/*.php
Automated Build Cycle (Ours uses the package.json data to copy build assets into a named directory and then zips with the current version number like popup-maker_v1.4.7.zip)
Also from testing, csso works better than uglifycss and why not include gulp-load-plugins and skip the declarations alltogether?
You can see our POT & build tasks here https://github.com/PopupMaker/Popup-Maker/blob/master/gulpfile.js#L72

@danieliser Thanks for dropping by.
I have been thinking to update this repo, I have a lot more routines in my personal workflow as compared to this one. POT files is a definite feature. I plan to add it here. You are more than welcomed to submit a PR.
Build routines are not added because everyone deals with it in a different manner. I am planning to add a simple zip building routine in there. But trying to keep everything as generic and portable as I can.
Can you explain how and why CSSO is better than UglifyCSS?
Again, to keep things portable I have not used gulp-load-plugins module. It helps when you are trying to debug and want to ignore any single module.

@danieliser POT file generation support is here.

Nice


Clarify gt_plt_summary

Question from Giang :: Mcsimulation table: I do not understand what is Missing 0.0 %

From the summary section https://htmlpreview.github.io/?https://github.com/CWWhitney/nifam_codas_school_garden/blob/main/Index.html

Addressed with #26


quotient filter builder bucket suggestion change

The Quotient filter number of slots suggestion function currently hardcodes the load factor for the lgNumSlots suggestion.  When we decide whether to pass the LOAD_FACTOR constant as a parameter to the filter, we also need to appropriately update the suggestion functions.
Alternatively, we could add a further parameter to suggestLgNumSlots(long maxDistinctItems) to change it to suggestLgNumSlots(long maxDistinctItems, double maxLoad) along with the change
byte result = (byte) Math.ceil(Math.log(maxDistinctItems / maxLoad) / Math.log(2));

Resolved in #584


Upgraded webpack dependencies.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Upgrades the webpack dependencies to latest, now that webpack 2.2 is released:
https://medium.com/webpack/webpack-2-2-the-final-release-76c3d43bf144
webpack
webpack-dev-server
Also updated these two dependencies to latest. Not sure if there is any breaking changes here:
webpack-dev-middleware
webpack-dll-bundles-plugin

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

Thanks @jamesanglin


Unable to show table records for Athena driver when not select on 'default' schema

Describe the bug
By using Athena driver, seems that the extension only fix for use 'default' schema to preview data. Anyway, when browse to another schema and try to "Show Table Records" it returns error since the 'default' schema name has been fixed to the extensions' generated query.
[1678932086202] ERROR (Error):
Error: SYNTAX_ERROR: line 2:6: Table awsdatacatalog.default.my_table does not exist
at AthenaDriver.rawQuery (c:\Users\User.vscode\extensions\brunoventura.sqltools-athena-driver-0.3.0\out\ls\driver.js:38:23)
at async AthenaDriver.query (c:\Users\User.vscode\extensions\brunoventura.sqltools-athena-driver-0.3.0\out\ls\driver.js:58:29)
at async Promise.all (index 0)
[1678932086204] ERROR (ext): ERROR: Execute query error SYNTAX_ERROR: line 2:6: Table awsdatacatalog.default.my_table does does not exist, 'SYNTAX_ERROR: line 2:6: Table awsdatacatalog.default.my_table does does not exist'
ns: "error-handler"
[1678932086204] ERROR (ext): ERROR: Error while showing table records SYNTAX_ERROR: line 2:6: Table awsdatacatalog.default.my_table does does not exist, {"code":-1,"data":{"driver":"driver.athena","driverOptions":{}}}
ns: "error-handler"
[1678932087537] INFO  (ext): EXECUTING COMMAND => sqltools.showOutputChannel
To Reproduce
Steps to reproduce the behavior:

Create a new Glue database on data catalog name 'my_db'
Create a new table e.g. 'my_table' (by a crawler or Glue job)
Configure the VSCode profile and SQLTools extension connection to this data source
When connected successfully, browse to node -Catalogs -> choose 'AwsDataCatalog' -> Databases -> choose 'my_db'->Tables
Right-click on 'my_table' and select 'Show Table Records'
See the error above

Expected behavior
The auto-generated query should be applied with a selected schema's (database) name instead of 'default'
Screenshots
n/a
Desktop (please complete the following information):

SQLTools Version: 0.27.1
VS Code Version: 1.76.2
OS: Windows 10 Pro
Driver:

Athena

Database version: AWS Glue data catalog

Additional context

Please report this to the author of the Athena driver extension. VS Code"s Report Issue option from the Help menu can probably assist you to do this.


Make error messages for add edit delete commands more specific

Fixes #244

Add a check for invalid prefix
Add a check for extraneous inputs by the user

Note: Please help to test extensively as there are many unintended ways that a user can input the commands.

Codecov Report

Merging #246 (d7cf769) into master (a09bba5) will increase coverage by 0.31%.
The diff coverage is 87.50%.

@@             Coverage Diff              @@
##             master     #246      +/-   ##
============================================
+ Coverage     69.74%   70.06%   +0.31%
- Complexity      638      643       +5
============================================
  Files            94       94
  Lines          1980     2001      +21
  Branches        269      273       +4
============================================
+ Hits           1381     1402      +21
  Misses          496      496
  Partials        103      103

Impacted Files
Coverage Δ

.../java/seedu/masslinkers/commons/core/Messages.java
0.00% <ø> (ø)

...linkers/logic/parser/AddInterestCommandParser.java
0.00% <0.00%> (ø)

...kers/logic/parser/DeleteInterestCommandParser.java
0.00% <0.00%> (ø)

...inkers/logic/parser/FindInterestCommandParser.java
0.00% <0.00%> (ø)

...ava/seedu/masslinkers/model/interest/Interest.java
80.00% <ø> (ø)

.../masslinkers/logic/parser/DeleteCommandParser.java
72.72% <50.00%> (ø)

...edu/masslinkers/logic/parser/AddCommandParser.java
86.95% <100.00%> (+1.95%)
:arrow_up:

...du/masslinkers/logic/parser/EditCommandParser.java
94.44% <100.00%> (ø)

...du/masslinkers/logic/parser/FindCommandParser.java
100.00% <100.00%> (ø)

...ava/seedu/masslinkers/logic/parser/ParserUtil.java
97.26% <100.00%> (+0.89%)
:arrow_up:

... and 20 more

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

Tested abit so far, will continue

okay thanks! I have made the changes

Summary of all the changes.

Fix error messages when the user inputs an invalid input such as edit 1 sdnn.

Originally, error message was "The student index provided is invalid."
Now, "Input with missing parameter prefixes found! Invalid Input: sdnn"

Fix error messages when the user inputs an two indexes such as edit 1 2.

Originally, error message was "The student index provided is invalid."
Now, "There is more than one valid index."

Fix error messages when the user inputs an invalid input after command word such as  edit poop 1

Originally, error message was "The student index provided is invalid."
Now, "Input with missing parameter prefixes found! Invalid Input: poop"

Fix error messages when user inputs an invalid prefix such as add o/sdknn n/jodj t/scdnc

Originally, error message was "Missing arguments for the given command! "
Now, "The prefix o/ is not recognised by Mass Linkers."

Fix error messages when user inputs edit n/ksnknd 1

Originally, error message was "The student index provided is invalid."
Now, "Index is missing, not a non-zero unsigned integer or is not specified before valid prefixes."

Fix error messages when user inputs add n/sdinef t/adkn kdnekf (is this a bug??)

Originally, error message was "Telegram handle is invalid!"
Now, "Telegram handle should not have any spaces."

Fix bug where mod mark all skdnn is rejected and mod mark cs2103t is accepted.

Now, anything after mod mark all will be ignored (both examples will be accepted)

Fix bug in isValidModName.

An error occurs when user inputs mod add c2100
Now user needs to input at least 2 alphabets before numbers for module name.

Removed some changes that were originally committed due to feature freeze

Checks if user input m/ prefix in mod commands.
Add command usage after every error

okay thanks! i've fixed it in the recent commit


Snyk/Dependabot/Sonar Review - March 2024

Review the Snyk/Dependabot/Sonar reports, and resolve any High and Critical items that can be addressed.

Snyk review and update completed in May:https://github.com/CDCgov/prime-simplereport/pull/7702
Dependabot has had multiple reviews and updates since this ticket was created
Sonar is being reviewed and updated with this PR:https://github.com/CDCgov/prime-simplereport/pull/7849


default.xml: change to use setup-environment-internal

Apply changes to 2.1 version

Like with #47, can you reformat the commit message to be the others? Also, don't make default.xml executable.

Done

As with #47: You only changed the title of the pull request, not the commit message. The 0755 change is also still in there.

This one leads to:
OSError: [Errno 2] No such file or directory: '/build/v2016.06/layers/meta-angstrom/setup-environment-internal'


Reduce size of repository

The repository is currently over 200MB. How can we reduce the size?

@mariobehling

we can remove the libraries which are not of utmost importance right now. Like ButterKnife etc.
we can remove duplicate items, unused imports, unused drawables in the project.

@mariobehling

We can reduce the size of screenshots in the docs folder. The images size is abnormally large (some are even 1.8MB large). We can look into that.

Can I PR for this.

The repo size is large due to binary files like images and apk. For these resources, git cannot diff and only store incremental changes. It has to the store whole file.
As each commit builds a new apk, the apk branch contributes most to the size of repo and  will continue to grow. There are options for contributors like :

Fetch/Pull only required branch. For eg: Pull master only and then create local development branch and pull it from origin. This will only cost you ~35 MB for master and then development branch will be a few hundred KBs as it is ahead of master by a few commits.
Use git-filter branch or BFG to remove the stale apk files from apk branch. But this requires rewriting history. Meaning, all contributors have to first pull from upstream and then commit. But as far as I know, no one commits directly to apk branch, so it would be the most viable option to actually reduce the size of repo.

@iamareebjamal
i don't think apk branch is contributing significantly towards the total repo size. The total size of apk branch is not more than 8MB.  Also the old apks are deleted from the apk branch when new ones are pushed.

@free4murad git stores the deleted files too. That's the whole point of version control, in case you want to go back in history. The actual size of the folder at a state is just the size consumed by the files currently in the repo. The old ones exist as well.
Here, I just pulled only apk branch to show its size. It's 172 MB, which is 97% of the total repo size of 177 MB
Screenshot attached

@iamareebjamal
thanks for sharing this.

@mariobehling I can apply a similar workaround here like I did on susi-android which reduced repo size from 1GB ~ 20mb, should I proceed?

@betterclever Sounds like a plan! 👍

@betterclever Please let us know the changes that you plan to make.

@mananwason Since already latest apk is required for testing purposes, we need not maintain all previous APKs that were generated, so a orphan branch can be made with just one commit, that maintains latest APK and current branch can be overwritten by that.
You may refer the implementation here :smile:
https://github.com/fossasia/susi_android/blob/master/exec/apk-gen.sh


Wrong Profile Image (different metadata)

on Steemit.com and Busy.org sometimes it picks wrong metadata and show different profile image

for example @the-ego-is-you
on busy.org

two different pictures

This issue seem to only affect accounts starting with 'the' with a dash or a dot in username.
@the.masses
@the-ego-is-you
@the-future
@the-alien
@the-explorer
@the-reginald
@the.intercept
Note: I have seen the same issue on steemit.com with the.masses's account so the issue may not be related to busy's code but steemit.com's code.

oh, I am so sad that it is so :sob:

This issue has been solved by @nil1511


Handle repeated argument types (Type.Arg.Repeated)

sure, didn't noticed there are syntactic tests

LGTM

Thank you very much for your contribution!


only add as slack command ?

The readme have the instructions of how add as slack command, this way only you can see the output.
maybe is useful: add as outgoing webhooks, this way every in the channel can see the output.

A lot of people I talked to said that they didn't want to clutter everyone's view with their searches. Hence the slash command response.


Module 2 Instructions on VNET Creation doesn't match new Interface Changes

Module: 02
Lab/Demo: 01  Exercise 1
Task: 02
Step: 02 and 03
On step 2, you need to click on Security button first, then the IP Addresses button.

On step 3, when you click on the default subnet, there is no longer an option to edit the service endpoints at creation time.  You need to go back and edit the created VNET and add the Service Endpoint of microsoft.sql at that time

.
Repro steps:

Thanks for your feedback, this issue has been addressed.


Delete 02-chapter_of_course.Rmd

The only thing I noticed is we had two Rmds with the prefix of 02-

02-chapter_of_course.Rmd
02-DataCleaning.Rmd

Let's see if deleting the first boilerplate Rmd fixes https://github.com/fhdsl/bench_to_bytes/actions/runs/9303023514/job/25604848313

Commit https://github.com/fhdsl/bench_to_bytes/pull/6/commits/02c275a791970a74de4eac9972b080180aee938b fixes https://github.com/fhdsl/bench_to_bytes/actions/runs/9307494595/job/25618898150?pr=6.


I can create a columns block without having to hack the container primitive UI

Following from #3180
The current way selection in volto is done and the UI for the container primative gets in the way of creating some basic layout blocks such as a columns block.
@sneridagh
Consider the following layout. Multiple vertical containers that can hold any of the basic block types. Ie a Grid > Column > Block.

My clients need to create things like this. EEA do too, that’s why they created eea volto-columns-block.
It’s a fairly basic requirement I would guess.
One thing you haven’t answered is how the container primitive would work with this common columns scenario. How the UI would work without block developers have to remove the primitive UI and replace it with their own inconsistent versions?
Out of the box the container primitive would look like this as it is now.

I can only see three options

Prevent any 2 level containment. Ie grid block is allowed but columns is not
Give up on WYSIWYG and allow extra spacing/chrome around containers all at the same time. This prevents two different containers toolbars from being on top of each other.
Go to a proper selection model where you only show overlays about the currently selected block and not it’s parents.

I remember the original discussions about containment in tokyo. I know albert thinks containment is an antipattern. You seem to be trying to stick to that idea of not dealing with containers yet wanting to create a universal container primitive at the same time. You can’t have it both ways. If you have a container then people will use it to make containers inside containers. This is unavoidable. Not dealing with this issue now will just create more problems later. You would be releasing a half baked solution and causing churn.
The current version on the container primitive means volto-columns-block would have to disable your container toolbar due to the overlapping toolbar issues I mentioned. Out of the box the container primitive would give them this.
Then they need to add in another way to insert and add in another way to navigate up the the parent container, like they had to do once already

And lhen later, once you get around to trying to solve this, which you will have to do eventually, they will then have to upgrade their block again with container primitive 2?
We are going to have to do the same. And others too. You are creating a lot of work for other developers and an inconsistent user experience for end users.
So I think for this reason trying to restrict containers to one level is also a bad idea.
I’d guess giving up on WYSIWYG is also out of the question. I believe it’s a core volto principle. Putting spacing/chrome/buttons around containers that make the items not appear how they would in the final version I believe is not the way to go.
So that only leaves showing chrome and overlays only on the current selected block (like every other WYSIWYG container based component editor I’ve found so far). Gutenburg may have iterated but that’s a good thing. We can take advantage of all their testing and not make the same mistakes.
However you don’t have to get this perfect now. You just have to provide a way that works regardless of how the containers are used so that block authors don’t have to remove your UI and replace it with their own.
What I’m suggesting is very close to the ideas in quanta (depending how you interpret it because it doesn’t deal with containers) and doesn’t change the current UI much.
The grid block toolbar is converted to a quanta toolbar “lite” that follows the block that is selected
The key point is that it’s very clear what block is selected and the toolbar is contextual to the selected block. You aren’t showing a toolbar for the grid while you are working on a sub-block.
The toolbar doesn’t need slate merged in with yet or a context menu. Those can be done later. It doesn’t need the settings button as selecting the block will automatically do that. It just needs an add and delete and the  “add” will always “add after” the current selection so the user experience is consistent.

“add” will add a block below this grid block

“add" will insert a new block after this text block. This is the most common thing you will want to do.
I’ve left the outer containers having a border (but visually different) as you seem to prefer that (but I‘m personally not sure it adds much).

“add” will insert a new column in the middle. (but you could also do this in the settings of the grid)

Because this is WYSIWYG and because containers might not have much padding to allow them to be clicked on superate from their contents there needs to be at least one other way of selecting a parent other than clicking. Gutenburg has 4 or 5 ways. We only need one to begin with. Again it doesn’t have to be perfect now.
You didn’t like having breadcrumbs in the side bar even though technically there is no reason it can’t be there but how about this instead?

but the gutenberg solution at the bottom of the screen is also good and could be added later (so can it’s outline view)

If you think the method to get to the parent is still too subtle another idea is the quanta toolbar could include a button that moves the selection to the parent. A kind of “back to parent” button. I’ve just noticed volto-columns-block has exactly this.
So for grid scenarios you get something like this (not sure on the icon yet).

The add inserts a new column in the middle and “back” will select the parent grid block so you can adjust its settings.
You will probably say the add button doesn’t make it clear where the insert will take place even if it is consistent and I agree with you. But there is no reason another overlay can’t be used to show where it will be added if desired.

And clicking “back” we get to the grid block and can then add a new block to the page.

As you may have realised, the page is now a container so there is no inconsistency for how blocks work within the page vs how they work inside containers.
Maybe @avoinea and @tiberiuichim could add their thoughts?

@djay My interpretation of quanta toolbar is like this:

At any single moment, exactly only one block toolbar shows up on screen
This block toolbar is for the currently selected block or sub-block (if it's contained in another block)
The block toolbar provides navigation to the container parent. It could also provide a limited set of controls for the parent block.

@tiberiuichim great. so what I proposed is what you consider to be the quanta toolbar (except maybe the toolbar containing some controls on the parent block). And that's what you think we need in volto and it will replace the current container block UI and @davisagli top level only add button?

@sneridagh my summary of the meeting is

Elementor takes a different approach by using dnd to create new blocks and right click context menu for everything else. That means no on screen toolbar is needed. This is actually similar to mosaic except that  elementor still has explicit container blocks. It does seem to have a small toolbar for the parent container, can't remember why.
pretty much in agreement that quanta toolbar was intended to work in the way outlined and gutenburg way is most similar.
discussed that text blocks shouldn't have the same boxed selection line that container or other blocks might need
The problem is that the current architecture of selections makes it hard to achieve a quanta toolbar appearing next to the current block. All parent containers think they are selected and the individual block is responsible for rendering itself as selected rather than being a generic framework thing. What would be needed instead is for the framework to be adjusted so containers are responsible for rendering the toolbar and selection and they know if the block is directly selected or one if it's decedents is. Need to investigate if this can be achieved without breaking the block apis.
but It would be possible to at least hide the container toolbar if whats selected is another container so at least they don't overlap.
the idea of getting some kind of quanta toolbar lite into 17 was not considered and the whole thing is seen a big problem that needs to be solved at some future undermined point.

In terms of UI, I looked for the quanta toolbar mockups. They don't deal with containment so leave some questions open.
https://github.com/plone/volto/issues/4206

the image there shows a add in the quanta toolbar but the mockups don't have this. they only have a global add at the bottom of the page and context menu insert before and after. They don't include what happens in a container or how to navigate container hierarchies.
There is a blue selection which is a line for text blocks and a box for other blocks

I personally think that either an add button in the quanta toolbar or add at the edge of the current block is fine (probably the later) but not sure about the idea of "add" being always visible at the bottom of the page.
If you keep to the ideas of quanta you might get something like this?

however

delete and add being too close together is not great. you might hit delete by mistake
still don't like the delete icon being the same as the cancel icon. You might think it will just undo your changes not remove the whole block. It should be a trash can IMO. Also when put next to the add you can see they look similar.
I've taken out the parent container dotted lines as we agreed it doesn't add much but can experiment.

Another question that remains open is where containment breadcrumbs might go.
The mockups show you can no longer get to the page settings from the side bar but it's not clear how you get to the page settings actually. Does the toolbar settings icon toggle the settings of the currently selected item? if so that means you need a way to select the document itself to get to the document settings since there doesn't seem to be another way provided?

However this shows that you can still get to the page settings without unselecting the current block. It's also won't be obvious this is how you get to page setting. So there has to be another way to get to the page settings?

Based on this it seems the gutenburg idea of putting container breadcrumbs at the bottom of the screen is the better one.

@djay In Volto the sidebar tab automatically switches to content settings when you select a title or text block (it's configurable in each block config, the sidebar index). So the text block can show the page settings and any other block can show its own settings.

@tiberiuichim text block has settings in the verison I use. But this was more a question about how its supposed to work in the quanta design which doesn't have tabs in the sidebar.

Here in out company we use Plone in several projects and we are doing some tests with Plone 6 and Volto. One major need we have is to add multiple blocks inside the same column of a grid, which is currently not possible.
I've tried to create a Group block using the Container primitive, setting its direction prop to "vertical", but it didn't work. Is this implemented yet?
Is there any example of a vertical container?

@rotavio
While they're not based on the code in Volto (they predate it and they were the "source" for extracting the BlocksForm component from Volto, check https://github.com/eea/volto-group-block
There's more of them https://github.com/collective/awesome-volto#layout-addons

Thank you!
We've tried some of their addons, tabs, columns, group and accordion, and are currently using them in our proof of concept. They are great, and work well, but we found their UI were not consistent between them. Also, they have this bug where you only get the "+" add button on text blocks. So, to add new blocks, you would have to select the last one, hit Enter to add a new text, click the add to open the Block chooser and then delete the text block. Also, some of them do not accept adding a new block outside and below it, there's no add button neither Enter works.
Also, we would rather use a native official solution, to avoid future upgrade problems and content migrations.
Are there any plans to add more features to the grid ro container blocks?
The vertical prop for the container does not seem to work. Is this by design, a bug, or something that I've missed?
We have two big projects due to the next months, but the limited layout builder in Volto is currently keeping us on older Plone versions.

The current sticking point is that the internal architecture on how selection works is centred on the block itself recognising when it's selected. Whereas the kind UI discussed above needs containers to be in control of selection.
The thinking is we need to rehaul the internal APIs to make this possible. We do have an idea how we can achieve this within the current architecture and will be doing a POC but have been a little busy.
But even if that POC works, it wouldn't get into 17 so still might require some time to be useful.
We do however use eea.columns in production. And grids. It is inconsistent but it works.


Seek for Help

Hello！I wonder where to get the wgfmu.lib,it seems not work.   :(

Hello,
There is a list of libraries/drivers to install in order to able the connection with the B1500.
-NI 488 drivers https://www.ni.com/fr-ca/support/downloads/drivers/download.ni-488-2.html#467646
-IO library suites from Keysigh https://www.keysight.com/ca/en/lib/software-detail/computer-software/io-libraries-suite-downloads-2175637.html
-B1500 drivers https://www.keysight.com/ca/en/lib/software-detail/driver/b1530a-wgfmu-instrument-library--sample-programs-2117445.html.
With all of that installed, you should be able to connect and communicate with the B1500. Make sure you are using a GPIB-USB-HS interface controller from NI as it did not work for other type of GPIB controller.
I'll update the readme file to add these required libraries.

Thank you for your help!


⚠ Jenkins Plugins Artifactory has degraded performance

In d5fce28, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1292 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 5e98329 after 15 minutes.


typo: Balace => Balance

Is it a typo?
https://github.com/GSabadini/go-bank-transfer/blob/e91fe0591900af47bdaa796d2e5a9abce4154dfa/usecase/find_account_balance.go#L16-L17

Yes, I fixed it, thank you! 😄


Add DataColumn API for breaks in Legend/Axis

A common case:
legends {
   breaks(column.distinct())
}

Please provide detailed instructions on how to complete this task

Add breaks(DataColumn) function with the same logic.


Handle badly encoded URLs gracefully (square brackets!)

See https://github.com/comp500/packwiz-installer/issues/2#issuecomment-813106598 for the current workaround. Ideally I'd like to move to OkHttp, which has a URL class with better handling of unencoded brackets/spaces.

Is this going to be fixed anytime soon? It appears to be possibly the biggest bug currently around

https://github.com/packwiz/packwiz-installer/commit/685e4c71d512f936d4c8c0a10b9010f7b2fb78c7 should fix this once it's integrated with the rest of packwiz-installer, and the targetrewrite branch is released :)


[move] IR module definitions require an address - (97)

Motivation
In Move source language, module definitions must either have leading addresses specified (module 0x42::M { ... }), or be defined within an address definition (address 0x42 { module M { ... } }). In Move IR, however, modules cannot be defined with an address, and instead command-line tools that process the IR specify the address under which to publish the module.
Remove the ability to pass in addresses from the command-line with Move IR, and instead require modules to be defined with a leading address.
What's in this pull request?
This is a large pull request, so I'll explain myself a bit. At its core this does one thing: modify the move_ir_types::ModuleDefinition so that instead of a name: ModuleName, it has a identifier: QualifiedModuleIdent field, and modify the parser to parse a leading address for that change. This requires module definitions to have leading addresses, so most .mvir tests, and some tests that define MVIR programs as strings (such as in the e2e-testsuite) need to be modified in order to pass (this is why this PR touches 300+ files).
Everything beyond that is "optional," but I decided to bundle these changes up together rather than touching hundreds of files in multiple pull requests. These changes include:
Since it's now unused, the compiler executable no longer needs to take an --address argument, so I deleted it. A module's address is the one it's defined with; the one passed in on the command-line is now meaningless and would be confusing to keep around.
Specifying an address is also now obsolete when invoking the IR compiler programmatically. So I removed the address: AccountAddress arguments from functions related to parsing Move IR, or compiling Move IR modules into bytecode modules.
The Compiler trait used by functional tests defined a compile method took an address argument. This argument was unused by the Move source language compiler, and now it's unused by the IR compiler as well (the only two objects that implement this trait). So I removed the argument from the trait method.
The transactional test runner's "publish" command required an --address argument be specified only when compiling Move IR. That's no longer necessary in the Move IR case, so I removed the argument altogether. This also means all transactional tests need to be modified to no longer pass that argument, which would be another large pull request -- that's part of my rationale for bundling all these changes together in one pull request.
What about address 0x42 { module M { ... } } syntax for Move IR?
I dunno, do we need it? Move IR exists almost entirely for writing tests for the Move VM, and most of those tests would not define multiple modules within a single address block. Supporting that syntax would make things more complex.
None of the changes in this pull request prevent that syntax from being supported, though. Even if Move IR were to be modified to support that syntax, the vast majority of the 300+ test files modified in this PR would not be modified. So IMHO it's better to land this PR first and potentially make that change later.
Test Plan

cargo test --package bytecode-verifier-transactional-tests
cargo test --package compiler --lib -- unit_tests::expression_tests::compile_script_expr_addition --exact --nocapture

/canary

:sunny: Canary successful

/land


MsgSwapExactAmountIn does not modify tick's quantity in case of a partial fill

Background
I am using poolmanager's MsgSwapExactAmountIn against an orderbook to swap tokens (against orders created by myself, for testing purposes). After I do that and I query the initially created order, I notice that the quantity field of an order is unchanged. Hence, on osmosis's UI, my order is partially filled, but if I query a state of the tick via CLI, it does not reflect that.
This issue affect queries, I have not noticed any critical vulnerabilities.
Suggested Design
swaps made through poolmanager should update partially filled ticks' states
Acceptance Criteria
mentioned query returns an updated quantity

cc: @crnbarr93 @AlpinYukseloglu

looked into this repository better, now I see the reason for my confusion


feat(eos_cli_config_gen): Implement l3dot1q support for interfaces

Change Summary

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[ ] Other (please describe):

Related Issue(s)
Fixes #546
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes

Implement new interface type for both ethernet_interfaces and port_channels:

ethernet_interfaces:
  Ethernet1:
    description: to WAN-ISP1-01 Ethernet2
    type: routed
  Ethernet1.101:
    description: to WAN-ISP-01 Ethernet2.101 - VRF-C1
    type: l3dot1q
    encapsulation:
      dot1q:
        vlan: 101

port_channel_interfaces:
  Port-Channel8:
    description: to Dev02 Port-channel 8
    type: routed
  Port-Channel8.101:
    description: to Dev02 Port-Channel8.101 - VRF-C1
    type: l3dot1q
    encapsulation:
      dot1q:
        vlan: 101

How to test
Molecule for arista.avd.eos_cli_config_gen
Checklist:

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have updated molecule CI testing accordingly
[x] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

README updated to report data model new entries


Redmine API: Errors on loading an issue are swallowed on JSON parsing error

redmine_api.rb, in get(path, ...):99 the issue_id variable is undeclared and will thus raise a wrong error.

Thanks @marudor for the report.


Add FDG module for reading data from files

Adds the ability to pull json/yaml data from a file, including the ability to pull data from a subkey of a data structure. Also includes the ability to have nested subkeys with a custom separator. Finally, includes the ability to pull somewhat arbitrary data from "config-style" files.

This is tested and ready for review.


[config] scrape：Add "scrape.reload.time" to the startup parameter

[config] scrape：Add "scrape.reload.time" to the startup parameter

This is totally internal and should not be changed or changeable. What is your use case?

This is totally internal and should not be changed or changeable. What is your use case?

   We collect more than 10000+ machines, and the acquisition target will constantly change. We attach great importance to the stability of the scraper.
   We do not want the scraper to be reloaded all the time, which will lead to the blank in grafana, or make the alarm false alarm, so we hope that the frequency of the scraper will be lower.
    A 10 minute update would make sense for us. The monitoring system developed by our company is also updated every 10 minutes
   However, changing the default reload frequency of Prometheus obviously does not meet the scenario of other customers, so we hope reload frequency is an option

Your use case is I believe using file_sd, if you only have updates every 10m then don't update the file more often than that.
This 5s is a sanity throttling, in case some discovery started spewing large amounts of target updates. It shouldn't be configurable.

Your use case is I believe using file_sd, if you only have updates every 10m then don't update the file more often than that.
This 5s is a sanity throttling, in case some discovery started spewing large amounts of target updates. It shouldn't be configurable.

OK, I got it. I'll  close this PR first.
This is a difference I found when comparing the internal monitoring system with Prometheus.
The main interior is 10 minutes, Prometheus is 5 seconds.

Your use case is I believe using file_sd, if you only have updates every 10m then don't update the file more often than that.
This 5s is a sanity throttling, in case some discovery started spewing large amounts of target updates. It shouldn't be configurable.

OK, I got it. I'll close this PR first.
This is a difference I found when comparing the internal monitoring system with Prometheus.
The main interior is 10 minutes, Prometheus is 5 seconds.

It is not like that. Various SD's have configurable sync intervals. The 5s is not for this use case.


Code Beautification

Currently the code a lot of html files is pretty unorganized, so it has become necessary the code for anyone viewing it.

Begin work today

Finished


Iterating command router

@dmdashenkov FYI: there's overlap in what Aggregate, ProcessManager, and CommandHandler do with generating events. We'll need to address it in a separate PR.

Commits that are not visible here:

4e0abb8 — Simplify test
42da12e — Simplify return type covariance in command routers

@dmdashenkov, PTAL


plot_centroids figure return

CrossSection.plot_centroids() now returns the plot figure and axes object.
Example scripts also updated to reflect this.
This PR should resolve Issue #27, allowing users more flexibility with the cross-section figure results. For example, adding points/arrows/circles to the figure, or saving the figure without relying on plt.savefig(name), where the "current figure" isn't always correct.

@robbievanleeuwen , since the dxf update seems to be taking a while, could I request a v1.0.8 packaged release with this change implemented? It would help me out to be able to pip install the library directly, rather than building the master branch myself, and I see no harm in including this small patch as really the only feature in 1.0.8.
Either way, it's no big deal! 😄

Hi @Czarified ,
Apologies for the lack of response, I was recently in an accident and have had a bit of a slow recovery. I will try to have a look at this in the coming days :)
Robbie


DOC: AdaptiveHistogramEqualizationImageFilter default radius

Closes #727

@gdevenyi please take a look

Looks good :)

Test failures should be unrelated.


[Helm integration] Remove handler implementation from Operator controller and reuse from Rukpak

Reuse the handler in Rukpak to covert registryV1 to chart: https://github.com/operator-framework/rukpak/blob/19e1839a4d6bb44a15bfcd8f876b5780ecdbc445/internal/provisioner/registry/registry.go#L22
The only value from BD that it needs is spec.InstallName, which we can set from CE while creating a mock for BD in the controller: https://github.com/operator-framework/operator-controller/blob/570b89f8409148fb0b7c2939aa723d6e5ea72be5/internal/controllers/clusterextension_controller.go#L256.
This requires us to make handler code external.

/assign

This issue has been addressed as a part of #928. Hence closing it now.


Fixes #56 - update supabase_flutter to 2.0.1

What kind of change does this PR introduce?
(https://github.com/supabase-community/flutter-auth-ui/issues/56)
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Thanks for this! It seems like with the latest version of Flutter, the lint rules have changed, causing the format test to fail, but we should be able to fix that in a separate PR! Once those are sorted out, we can publish a new version of supabase Flutter auth UI!

np @dshukertjr yeah I wasn't sure what happened with the changes to the quotes.  at one point I reverted the doubles to singles with vi but I missed one.

I think there is a problem with this change.  there is some issue with access to the 'Provider' enum.  in my local I needed to change all references to 'Provider' in this file lib/src/components/supa_socials_auth.dart to 'OAuthProvider' which I believe is from the gotrue package.  I was able to run the example app locally after this change but I'm really not confident that I tested it properly.


Latest gitea container causes PostgreSQL database error, no longer launches

Gitea version (or commit ref): don't know, gitea doesn't output this in the log when launching in docker (or only does so after the error occurs)
Git version: docker image gitea/gitea:latest
Operating system: docker / ubuntu 17.10 host
Database (use [x]):

[X] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[X] No
[ ] Not relevant

Log gist:

Description
After I updated the docker container to the latest version (keeping all config data), it no longer launches. It seems to be stuck in a restart loop without ever successfully running, showing this in the log:
...
gitea_1  | 2018/03/08 18:38:00 [T] Custom path: /data/gitea
gitea_1  | 2018/03/08 18:38:00 [T] Log path: /data/gitea/log
gitea_1  | 2018/03/08 18:38:01 [T] AppPath: /app/gitea/gitea
gitea_1  | 2018/03/08 18:38:01 [T] AppWorkPath: /app/gitea
gitea_1  | 2018/03/08 18:38:01 [T] Custom path: /data/gitea
gitea_1  | 2018/03/08 18:38:01 [T] Log path: /data/gitea/log
gitea_1  | 2018/03/08 18:38:02 [T] AppPath: /app/gitea/gitea
gitea_1  | 2018/03/08 18:38:02 [T] AppWorkPath: /app/gitea
gitea_1  | 2018/03/08 18:38:02 [T] Custom path: /data/gitea
gitea_1  | 2018/03/08 18:38:02 [T] Log path: /data/gitea/log
gitea_1  | 2018/03/08 18:38:03 [T] AppPath: /app/gitea/gitea
gitea_1  | 2018/03/08 18:38:03 [T] AppWorkPath: /app/gitea
gitea_1  | 2018/03/08 18:38:03 [T] Custom path: /data/gitea
gitea_1  | 2018/03/08 18:38:03 [T] Log path: /data/gitea/log
...

The database log shows this:
db_1     | LOG:  database system was shut down at 2018-03-08 06:22:36 UTC
db_1     | LOG:  MultiXact member wraparound protections are now enabled
db_1     | LOG:  database system is ready to accept connections
db_1     | LOG:  autovacuum launcher started
db_1     | ERROR:  syntax error at or near "num_teams" at character 34
db_1     | STATEMENT:  ALTER TABLE "org_user" [is_owner num_teams]
db_1     | ERROR:  syntax error at or near "num_teams" at character 34
db_1     | STATEMENT:  ALTER TABLE "org_user" [is_owner num_teams]
db_1     | ERROR:  syntax error at or near "num_teams" at character 34
db_1     | STATEMENT:  ALTER TABLE "org_user" [is_owner num_teams]
db_1     | ERROR:  syntax error at or near "num_teams" at character 34
...

Screenshots

Damn, looks like I broke it :(

Well unless it's my fault somehow? I wouldn't know that it is, but who knows. I haven't looked at that gitea in a while, but a month ago it worked.
Can I provide you with anything to help figure out what is going on? I can't dump the entire database here (has some confidential projects) but I could figure out a way to give you access personally if that was of any help.

@JonasT I have submitted PR to fix this. Somehow I had broken my written code after I did test it. Sorry for that

@lafris so when will the docker container be updated? I just checked gitea/gitea:latest and launching it still results in the same error...

@JonasT this fix should be already be in latest

Ah sorry, I forgot to cleanly recreate the container. Works now :smile:

Thanks Lunny,
Centos 7 will not be happy with anything beyond 1.8.
But turns out my real problem was the firewall. It is back up now.


BP Monitor Reading using DL

Pull Request for DL-Simplified 💡
Issue Title : BP Monitor Reading using DL

Info about the related issue (Aim of the project) : Create a DL model, which will monitor the reading given in the images and identify them.
Name: Pallavi Patil
GitHub ID: github.com/itspallavi20
Email ID<EMAIL_ADDRESS>Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant) I am contributing in GSSOC(GirlScript Summer of Code)2024

Closes: The issue number assigned was #245
Describe the add-ons or changes you've made 📃
I created deep learning model with 3 algorithms implemented- 1.Custom CNN, 2.Resnet50 (Transfer learning) and 3.RNN with CNN features comparing the best algorithm in this case which provides the highest accuracy.
Type of change ☑
What sort of change have you made:

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ x] This change requires a documentation update

How Has This Been Tested? ⚙
The created model has been tested and the comparisons for the accuracies and loss are displayed very clearly in the form of graphs and visualizations. The algorithm with the highest accuracy is considered to perform the best on the medical images and annotations dataset.
Checklist: ☑

[x ] My code follows the guidelines of this project.
[ x] I have performed a self-review of my own code.
[ x] I have commented my code, particularly wherever it was hard to understand.
[ x] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ x] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Hi @itspallavi20 create a project folder first and then push all the subfolders inside the dedicated project folder.


Clarify details in boundary layer

There are several issues with the documentation of the boundaries layer:

admin_level property is defined as numeric. I believe this should be clarified as unsigned integer (see also #31)
The feature table allows only admin_level 2 or 4. But in the description of the disputed property it says "unset or between 2 and 4". Is it on purpose that level 3 is allowed here? That unset is allowed here?
The definition for the "disputed" property is hard to parse and with several ands and ors without braces, it is ambigous.
We should probably explain more clearly that processing for the boundaries layer has to look at both the tags of the ways and the relations and how they interact, especially for the disputed case.

I'm not sure it's possible to control the precise output type on all generation systems and someone may not have a choice between int, uint, and sint. If it comes to it I can just produce non-compliant tiles as any use-case won't depend on what the message type is.
Aside: The most efficient way to represent integers that could be negative is probably to pick the type based on the individual value, using uint for positive and sint for negative.

I think there are two separate issues with the numbers:

What is the "natural data type" for this property, ie. the narrowest "usual" datatype that can store all values of this property. In this case this is "unsigned integer", but the definition is somewhat fuzzy, we could also say "a 4 bit unsigned  integer is enough for this". But this is what we have to decide first.
The next question is what detailed encoding to use specifically for vector tiles. This is informed by (1) and also the tools that exist etc.

I think for 1 this is really a case of we should be defining the allowable values. So admin_level is an integer between 2 and 8, layer is an integer between -5 and 5, etc.


Text overlap in spin deploy when asking What would you like to name your database?

Describe the bug
The text description for the prompt What would you like to name your database? is overlapping the text input when using the spin deploy command.
Frequency of the bug
Please check one:

[x] Consistent repro
[ ] Intermittent repro
[ ] No repro

Set-up information
Please share the version of Spin used when the bug occurred. You can find the version of Spin by running spin -V.
spin -V
spin 2.6.0-pre0 (VERGEN_IDEMPOTENT_OUTPUT VERGEN_IDEMPOTENT_OUTPUT)

To Reproduce
Steps to reproduce the behavior:

spin deploy

Expected behavior
The ]: should be at the end of the text rather than in the middle of the description.
Screenshots
If applicable, add screenshots to help explain your problem.
(Remember to remove any personally-identifiable information.)
This is the initial problem before typing anything.

This is when I ran it again and typed:  TESTKEYVALUESTORENAMEHERE

Additional context
I have had it use some of this text in the name of the keystore, so it is pulling some of these characters into the prompt value.
If I use the backspace the text is removed.

Understood, sorry for posting in the wrong repo.


Valea Mare-Bratia village from Balilesti is not found in Arges county

Precondition
SOLR: http://zimbor.go.ro/solr/#/romania/query
browser: Chrome
Steps to Reproduce:

Step
Action
Expected
Status

1
open SOLR in browser
SOLR UI is loaded without any errors
Pass

2
type one by one in the q field all the list of the components of every village in Arges county found in test dataTest data.xlsx
you are able to type one by one in the q field  all the list of the components of every village in Arges county found in test data
Pass

3
type judet: Arges in the fq field
you are able to type judet:Arges in the fq field
Pass

4
click on Execute Query button
Verify that all the list of the components of every village in judet:Arges form the test data are displayedTest data.xlsx
Fail

Actual Results:
Valea Mare-Bratia village from Balilesti is not found in Arges county

https://orase.testquality.com defect D30.

Este comuna Balilesti cu satul Valea Mare-Bratia.


Pin knative.dev/{pkg,test-infra} to release-0.16

Fixes #

Proposed Changes

Release Note

NONE

Docs

/retest

/lgtm
/Approve


Guidance on deployment or migration to isolated worker model

Emails were sent for Functions use in-process model instead of isolated workers. What needs to be done to update this solution once deployed or deploy using the other model?
There appear to be a number of changes per https://learn.microsoft.com/en-us/azure/azure-functions/migrate-dotnet-to-isolated-model?tabs=netframework48 required in the code.
Looking for guidance or timeframe expectation when we will have guidance on migration or new deployment option.

I guess we should throw in the earlier deadline to move from .NET 6 to 8 as well. https://learn.microsoft.com/en-us/azure/azure-functions/update-language-versions?pivots=programming-language-csharp

Hi, I received and Alert directly from Azure Health, requesting me to upgrade my own developed Functions to .Net 8 and isolated  process model. So I expect that Azure Team is going to upgrade this repo or the Auto-Update feature of Start Stop solution.
So, I'm adding a +1 to this issue.

I am also looking for information on performing this upgrade, or whether the Azure team will be taking care of it.

We will imminently release a new version of StartStop V2 that moves to .NET 8 and uses the isolated worker model. We thank everyone for their patience.


Exiting early doesn't save latest training state

If you exit early it saves the latest model (net_g_latest and net_d_latest) but not the latest training state, so when you resume training it restarts from the latest regular checkpoint.

Hi, sorry for the late reply. This is a known issue that, unfornatelly, is more complicated to solve than it seems. Hopefully I can take a look at it in the future. Feel free to send PRs regarding the issue.


Added fbjs eslint config, fixed lint errors and warnings

I decided to remove eslint:recommended config and just use fbjs to be consistent with the React project... Let me know if this is better!

I decided to remove eslint:recommended config and just use fbjs to be consistent with the React project...

I think this is much better! 😄

Sounds great, @bvaughn! Will make those changes.

@bvaughn - Seems like the button produces a blue outline when clicked (:focus). Removing it with outline: 'none' will also encounter accessibility issues... What do you reckon should we do? Leave it, or remove the outline completely?

Yup. The outline is important for keyboard users until/unless we provide a different focus style (cc @joecritch). I think for now we should leave it in.

Awesome!


try-finally without catch clause is formatted strangely.

try
{
  //something
}

finally
{
  // do something
}

(Note the blank line between the closing bracket of try and beginning of finally)

This fix will be out in the next release, thanks for reporting it!


WIP Add a CFTimeIndex-enabled xr.cftime_range function

[x] Closes #2142
[x] Tests added (for all bug fixes or enhancements)
[x] Tests passed (for all non-documentation changes)
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

I took the approach first discussed here by @shoyer and followed pandas by creating simplified offset classes for use with cftime objects to implement a CFTimeIndex-enabled cftime_range function.  I still may clean things up a bit and add a few more tests, but I wanted to post this in its current state to show some progress, as I think it is more or less working.  I will try to ping folks when it is ready for a more detailed review.
Here are a few examples:
In [1]: import xarray as xr

In [2]: xr.cftime_range('2000-02-01', '2002-05-05', freq='3M')
Out[2]:
DatetimeIndex(['2000-02-29', '2000-05-31', '2000-08-31', '2000-11-30',
               '2001-02-28', '2001-05-31', '2001-08-31', '2001-11-30',
               '2002-02-28'],
              dtype='datetime64[ns]', freq='3M')

In [3]: xr.cftime_range('2000-02-01', '2002-05-05', freq='3M', calendar='noleap')
Out[3]:
CFTimeIndex([2000-02-28 00:00:00, 2000-05-31 00:00:00, 2000-08-31 00:00:00,
             2000-11-30 00:00:00, 2001-02-28 00:00:00, 2001-05-31 00:00:00,
             2001-08-31 00:00:00, 2001-11-30 00:00:00, 2002-02-28 00:00:00],
            dtype='object', calendar='noleap')
In [4]: xr.cftime_range('2000-02-01', periods=4, freq='3A-JUN')
Out[4]: DatetimeIndex(['2000-06-30', '2003-06-30', '2006-06-30', '2009-06-30'], dtype='datetime64[ns]', freq='3A-JUN')

In [5]: xr.cftime_range('2000-02-01', periods=4, freq='3A-JUN', calendar='noleap')
Out[5]:
CFTimeIndex([2000-06-30 00:00:00, 2003-06-30 00:00:00, 2006-06-30 00:00:00,
             2009-06-30 00:00:00],
            dtype='object', calendar='noleap')

In [6]: xr.cftime_range(end='2000-02-01', periods=4, freq='3A-JUN')
Out[6]: DatetimeIndex(['1990-06-30', '1993-06-30', '1996-06-30', '1999-06-30'], dtype='datetime64[ns]', freq='3A-JUN')

In [7]: xr.cftime_range(end='2000-02-01', periods=4, freq='3A-JUN', calendar='noleap')
Out[7]:
CFTimeIndex([1990-06-30 00:00:00, 1993-06-30 00:00:00, 1996-06-30 00:00:00,
             1999-06-30 00:00:00],
            dtype='object', calendar='noleap')

Hopefully the offset classes defined here would also be useful for implementing things like resample for CFTimeIndex objects (#2191) and CFTimeIndex.shift (#2244).

I haven't had the chance to look in detail at this yet, but one small point I would suggest is renaming it from xarray.date_time to xarray.cftime_range. This makes it more obvious that the function is really for users of cftime.

@spencerkclark - sync this branch with master and we'll get this all wrapped up.

Thanks @jhamman!  This should be synced up and ready for review.  The test failure under the dask-dev build doesn't appear to be related.

Awesome, thanks @shoyer.

Many thanks for the initial review @shoyer.  I think I got to everything so far.  I agree restricting cftime_range to only return CFTimeIndexes makes things much simpler to reason about.


New version

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: K0R0L:white_check_mark: AlexeyMatveev686:white_check_mark: KhromovNikita:white_check_mark: VikulovDmitry:x: MashaNemtYou have signed the CLA already but the status is still pending? Let us recheck it.


Map valid math functions into the IR as NativeFuncCalls

Description
Implementation of math functions as NativeFuncCalls.
Co-authors: @eddie-c-davis @twicki
To do:

[x] Add deeper parsing test(s)
[x] Add code generation / integration test(s)
[x] Add @twicki to AUTHORS.rst

I'm fine with the changes for the frontend/IR modules I had reviewed. The only change still needed is to undo my previous suggestion of using the string values of the NativeFunction node as DSL names. After discussion with @twicki, we agree that it is better to go back to use explicit mappings from DSL symbols to the NativeFunction IR enum, in order to achieve a better decoupling of the DSL and the IR.
Note that a proper definition of the GTScript math symbols without introducing circular dependencies between the ir, gtscript and gtscript_frontend modules, is trickier that it looks like (look at the root __init__.py to see the right import order). I suggest to define the function names in the gtscript module first, for example:
_MATH_BUILTINS = {"min", "max", "sin", ... }

# GTScript builtins
builtins = {
    "I",
    "J",
    "K",
    "IJ",
    "IK",
    "JK",
    "IJK",
    "FORWARD",
    "BACKWARD",
    "PARALLEL",
    "Field",
    "Sequence",
    "externals",
    "computation",
    "interval",
    "__gtscript__",
    "__externals__",
    "__INLINED",
    *_MATH_BUILTINS
}

The mapping translating DSL names to IR symbols could be later defined either in the gtscript_frontend module (probably the right approach if this mapping is only needed for parsing) or as a NativeFunction member in the ir module (not ideal, but probably a sensible approach if the mapping is needed from different places and also follows the same style of other symbols in the current version).

I will finish tomorrow the review of the missing part of the PR.


test(cli-import): added improvements to test cases

This PR fixes CS-34654.

@vkalta As we discussed over the call make the environment variable changes in the code wherever it's required. If any queries reach out to @abhinav-from-contentstack or me.
Thanks


Create preprocessing.py

Change the location of the NDRobustScaler to utils.

Thanks for the review @qbarthelemy :)


Kafka topic - decrease number of partition

What happened?
I created kafka topic's modification request, which decreased partitions number. I tried to approve that request, but it was unsuccessful (Unable to process request. Please contact with administrator). As the result topic was removed from kafka cluster.
Klaw 2.9.0 configured with Aiven (SASL_SSL_SCRAM_MECHANISM_256) - kafka cluster in Azure

Hey @maciejmaciejko-gid thank you for this, were their any logs in the cluster-api that might indicate why the operation failed.
I think when decreasing partitions it has to delete the topic and recreate with less partitions as kafka doesnt support downsizing partitions on topics. so i think it looks like the re-creation with less partitions was the issue.

@maciejmaciejko-gid Like Aindriu mentioned, topic will be deleted and has to be recreated. If re-creation failed, can you share us any logs ?


Use strict mode of snap

As per discussion here: https://forum.snapcraft.io/t/classic-confinement-request-dunner/11528
Need help with testing this on linux. It seems complex to setup snap in docker.

let's merge this to develop first


expand align-deps to include some vetted custom profiles

When using dep-check currently, it has only one profile that it brings along: ours, the Microsoft one.
But as we have been seeing and hearing from the community, it might be not comprehensive enough or up-to-date (ex. https://github.com/microsoft/rnx-kit/issues/1549).
While we are firm believers in the idea that every repo will probably need its own custom profiles locally, in order to make it easier for folks to hop in dep-check, we came up with the idea to have dep-check "bring along" a set of vetted custom profiles that the user can pick from as the starting base.
This idea has basically 2 parts:

we need to give an easy way for folks to contribute (in a standardized naming convention) their profiles to this repo (most likely within a dedicated folder, akin to dep-check/custom-profiles/<standardized-name>)
we need an easy way for folks to use these profiles as baseline, and still be able to add their own custom local profile on top

closing off, with align-deps things have been streamlined


Multiple Clients Solution

Hi,
I wanted to know how a multiple client solution for erpc would be implemented and what changes would I need to do. Also, if its not possible, how can I do it using TCP only?
Thanks,

Hi @yahya010 , please take a look to https://github.com/EmbeddedRPC/erpc/issues/174

Hi @Hadatko, I‘m so looking forward to the feature, will it be released to develop branch this month?

Hi @vonku , Although supporting multiple client is almost ready, i need to appologize that i didn't focus on it last months. I distracted myself with CI which is also very important to properly test every our change we do. Now CI support build and test erpcgen under Mac, Linux, Windows. This can help us to do less mistakes and better releases.
Right now i don't see more important task than this (multiple client support.)

Hi @vonku , Although supporting multiple client is almost ready, i need to appologize that i didn't focus on it last months. I distracted myself with CI which is also very important to properly test every our change we do. Now CI support build and test erpcgen under Mac, Linux, Windows. This can help us to do less mistakes and better releases. Right now i don't see more important task than this (multiple client support.)

Hi @Hadatko , I fetched your pr and tried it out lastday. I found it different that the IDL annotation @types_header(string) did not work, with which I wanted to declare all the typedefs and structs to a specific header file to avoid redefinitions. Now they were generated to  xxx_common_erpc.h and xxx_common_standard.h. Is this change on purpose?

Hi @vonku , Although supporting multiple client is almost ready, i need to appologize that i didn't focus on it last months. I distracted myself with CI which is also very important to properly test every our change we do. Now CI support build and test erpcgen under Mac, Linux, Windows. This can help us to do less mistakes and better releases. Right now i don't see more important task than this (multiple client support.)

Hi @Hadatko , I fetched your pr and tried it out lastday. I found it different that the IDL annotation @types_header(string) did not work, with which I wanted to declare all the typedefs and structs to a specific header file to avoid redefinitions. Now they were generated to xxx_common_erpc.h and xxx_common_standard.h. Is this change on purpose?

As shown below, I hope that the structs defined in bt_common.erpc would be generated to bt_erpc_common.h, but no bt_erpc_common.h file was generated.
@output_dir("bt_source")
@types_header("bt_erpc_common.h")
program oppo_bt_erpc

import "bt_common.erpc"

Hi @vonku, i think i removed it intentionally as the type definitions are now generated into their own files by default. So i can add two bullets to my work:
-> Type definition should be generated only into one file. They may not depend on interface (like in IDL they are not depending on interface).
-> Keep option for renaming the file (Do you think it is necessary?).

Another issue, If I define the ERPC_ALLOCATION_POLICY  as below, the shim code would compile failed.
#define ERPC_ALLOCATION_POLICY (ERPC_ALLOCATION_POLICY_STATIC)
Because the erpc_port.h file would not be included in xxx_client.cpp file but it still uses erpc_malloc() and erpc_free()
#if ERPC_ALLOCATION_POLICY == ERPC_ALLOCATION_POLICY_DYNAMIC
#include "erpc_port.h"
#endif

Sounds like the malloc issue @diagano had at #349

Hi @Hadatko, is there any update plan for this feature?

Hi @vonku , same as last time. In eRPC is this my main issue right now (finish this task). But eRPC is not my only project. So i couldn't work on it last months.


initial sponsors page

Addresses issue https://github.com/OSFeels/OSFeels.com/issues/5

++++ To veggie use!
Also, COURGETTES!  SQUEE!

Small columns of justified text can end up looking weird. Mind if I make this a two column layout with right-aligned text…

Or justified text…

???

@strand Either sounds good to me! :+1:

To complete get descriptions for:

[ ] GitHub
[ ] HoneyBadger
[ ] npm
[ ] NIRD
[ ] StickerMule
[ ] Thomas Street

Get logos for:

[ ] NIRD
[ ] StickerMule

Nice to have:

[ ] Extract the data to YML or JSON and iterate through the HTML in an each loop.

lgtm :boat:

Can we ship this and add other sponsors as details come in?


block not invoked when using :include option for deeply nested object

I am trying to clone my Meeting object.
It all works except for copying the the PaperClip image attachment in one of the polymorphic Item objects.
When I added a block to process these model objects separately it seems the block is never evaluated.
I then added a debugger halt and it never got called either.
new_meeting = self.deep_clone include: { topics: { topic_items: :item } } do |original, kopy|
  byebug
  kopy.image = original.image if kopy.respond_to?(:image)
end

I've duplicated this scenario in the tests for deep_cloneable ... and it works in the tests!
So obviously there is a difference I haven't discovered yet.

Found the issue, not surprised it was my fault.


Add Delta table support for filesystem destination

Description
This PR enables writing datasets to Delta tables in the filesystem destination.
A user can specify delta as table_format in a resource definition:
@dlt.resource(table_name="a_delta_table", table_format="delta")
def a_resource():
    ...

Related Issues
Contributes to #978

@rudolfix Can you review once more?
I addressed some of your feedback. Biggest changes since last review:

Introduction of the "directory job" so multiple files can be loaded at once. Thus far, the concept of a job was tightly coupled with a single file (if I understand correctly). But if there are multiple Parquet files, we want to be able to load them into the Delta table in a single commit. That is now possible. Code is still rogue/minimal, but I want some feedback before polishing it up.
Dedicated job for Delta table loads: DeltaLoadFilesystemJob.
Turned Delta-related methods into static utility methods.

@rudolfix I addressed your feedback. Can you review?
I changed the credentials handling for deltalake a bit since last time. It's a more generic setup now.


Inference widget consumes wrong mask token

Hello 👋
I hope this is the right repository to open issue to. 🤞
This model takes [MASK] token (as seen in special_token_names.json) but mask token is taken as < mask > which makes widget fail.

Hi there :wave: The mask token does not come from special_token_names.json afaik.
They can be:

Specified in the metadata of the repo
Automatically retrieved as seen in

export const MAPPING_MASK_TOKENS = new Map([
	["[MASK]", ["distilbert", "albert", "bert", "electra"]],
	["<mask>", ["camembert", "xlm-roberta", "bart", "longformer", "roberta", "xlnet", "amazon/bort"]],
	["<special1>", ["xlm", "flaubert"]],
]);

Internal code https://github.com/huggingface/moon-landing/blob/93d758a5d5a23f97c7f0029520db347dd3ce0641/server/repo_types/ModelInfo.ts
cc @mishig25

very true sir!

Closing as no follow-up is needed here


claim.lens.xyz not working with VPN

When using PIA VPN, I would get the following errors. By turning off the VPN, the claiming was completed quickly.

(couldn't find a more relevant repo to open the issue)

Hey @odyslam - I'm forwarding this internally to the team maintaining the site, please bear with me until I came back with updates.

And welcome to Lens :)

@donosonaumczuk would you be able to share the root cause? Just curious for my own understanding

@odyslam  The claiming site's firewall was configuration was too restrictive.

Cool, thanks for circling back 🫡
Le 5 oct. 2022 à 6:28 PM +0300, Alan @.***>, a écrit :

@odyslam The claiming site's firewall was configuration was too restrictive.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Convert lever  issue in getorderHistory

It gives error when I want to access to orderHistoty
  var ordrtHistory = api.Trade.GetOrderHistoryAsync(OkxInstrumentType.Swap, position.Instrument)
{: [2] Deserialize JsonReaderException: Could not convert string to integer: 70.0. Path 'data[0].lever', line 1, position 391. Path: data[0].lever, LineNumber: 1, LinePosition: 391

Apparently they changed leverage data type from integer to float. I've fixed it and will release new verison in one hour.
Thanks for feedback

you can check it https://github.com/burakoner/OKX.Api/commit/523cbc108a903ae556c107602e8996808798f0a9 and https://www.nuget.org/packages/OKX.Api/1.0.5


 It didn't break in PR CI (https://dev.azure.com/lowrisc/opentitan/_build/results?buildId=21647&view=logs&j=1454b898-4e5e-5cfa-4c06-965e95c83d2e&t=7e6b977f-9c6a-5377-ff05-a6efdb19c231), so something on master likely changed between the PR opening and the merging.

@rswarbrick

This PR has a fix for it: https://github.com/lowRISC/opentitan/pull/3468

can spin that out as a separate PR if we want the fix now without awaiting review for the rest of it

Breakage was introduced by this change: https://github.com/lowRISC/opentitan/commit/e4665c1fd2861ecd274d3d86c967422cf5a5d717 though the bug was in the OTBN verilator simulation top. It failed to specify the dmem size as 256 bit which didn't matter until that change came in.
OTBN smoke only runs on OTBN changes which is how this slipped through.

No need to split it out, I'll have a look at #3468 now. Thanks Greg!

Breakage was introduced by this change: e4665c1 though the bug was in the OTBN verilator simulation top. It failed to specify the dmem size as 256 bit which didn't matter until that change came in.

Sorry about that. The PR (#3459) didn't run the OTBN smoke test as part of CI because the change didn't touch any OTBN-specific code. Oddly, things seemed to work locally, but maybe I didn't run exactly what I thought I'd run.


Fix endpoint creating on routing screen

At the moment creating bi-directional connections aren't possible, when the user drags from the endpoint it picks up the state instead.

@SteveBarnett ready for review

:+1:


No Reference to ImageResizer.Plugins.EPiFocalPoint is added

This is due to the fact that we target net461, and you have net46 as targetFramework in you project.
(note: we had that discussion separately)

Fixed in release 1.1.1


Imenu support

I reimplemented imenu support into a new package js2-imenu-mocha, taking inspiration from your package. I think the code is simpler and also contains more features (e.g., support for fit() and fdescribe()). I would like to have this code included into js2-mode itself.

Wouldn't it benefit everybody if this were part of mocha.el instead?

I think more people would benefit if it was included in js2-mode directly but I'm fine migrating the code to mocha.el if they don't want it. Let's wait a bit for their answer before deciding anything.

The work got merged into js2-mode.


Uplift repository (node, libs, build and release)

I have uplifted this repo with a number of updates and minor improvements.
Note that there are no changes to the javascript code itself.

Update Action to a supported node version (16)
Update libraries to their current, stable versions
Remove the need for committing node_modules to the git repo by using ncc at build time

Removed the node_modules directory (hence Github thinking there are 1682 files changed).

Add .nvmrc with supported node version
Add .gitignore to ignore build artifacts
Update the main build/deploy workflow

Use setup-node for the build
Run on master and main branches as well as develop
On master/main only:

Automatically tag the action
Automatically release the action

Fix markdown formatting (markdown lint)

If @Bardavon-Health is no longer using this Action, I would be happy to take over ownership and maintain it.

cc/ @rhenry-bardavon & @nathanshire as I can't request you as reviewers 😄

And just after submitting this I noticed your fork @dennybiasiolli 🥲

@sammcj they merged my fork in the main branch and released a v1.3 🥳
You can switch back to this official version with
- uses<EMAIL_ADDRESS>

Closing this PR since it's been a while and there are now a bunch of merge conflicts and certain things are no longer applicable. Feel free to re-submit a new PR if desired.