Remove problematic assertion fixes #105

Seemingly pointless annotation makes writing CI tests or enabling assertions very difficult. Have ran this in production (before this was on github) without any affects.
Example stack trace for posterity
java.lang.AssertionError: [ExchangeServiceBase.SaveHttpResponseHeaders] expect no headers in the dictionary yet.
	at microsoft.exchange.webservices.data.EwsUtilities.EwsAssert(EwsUtilities.java:350)
	at microsoft.exchange.webservices.data.ExchangeServiceBase.saveHttpResponseHeaders(ExchangeServiceBase.java:822)
	at microsoft.exchange.webservices.data.ExchangeServiceBase.processHttpResponseHeaders(ExchangeServiceBase.java:813)
	at microsoft.exchange.webservices.data.ExchangeServiceBase.internalProcessHttpErrorResponse(ExchangeServiceBase.java:319)
	at microsoft.exchange.webservices.data.ExchangeService.processHttpErrorResponse(ExchangeService.java:3696)
	at microsoft.exchange.webservices.data.ServiceRequestBase.processWebException(ServiceRequestBase.java:596)
	at microsoft.exchange.webservices.data.ServiceRequestBase.validateAndEmitRequest(ServiceRequestBase.java:696)
	at microsoft.exchange.webservices.data.SimpleServiceRequestBase.internalExecute(SimpleServiceRequestBase.java:58)
	at microsoft.exchange.webservices.data.MultiResponseServiceRequest.execute(MultiResponseServiceRequest.java:147)
	at microsoft.exchange.webservices.data.ExchangeService.findItems(ExchangeService.java:824)
	at microsoft.exchange.webservices.data.ExchangeService.findAppointments(ExchangeService.java:1106)

Hi @kentongray, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

For another report of this issue see #105

@kentongray plz have this PR updated.

:+1:

@serious6 what do I need to do?

@serious6 updated to latest!

If you squash the two commits into one this is ready to merge.

@serious6 Rebased!

@kentongray thanks for the contribution.


Are your all js files are written by type script?

How can I convert them to es6?

There's no Typescript in this repository - it's ES6 JavaScript with Flow type checking. It should be simple enough to convert from Flow types to Typescript, but as I have to experience with it I'm afraid I can't assist.


Non copy-able " in setup guide

On the setup website(http://www.rmhartog.me/octopull/#setup) the code to add to .travis.yml is not copy-paste friendly. Some of the " characters are actually “ causing travis' parser to fail.

Thanks for reporting, probably copied from the wrong editor, should be fixed now!


LoweringError when using sqrt in CUDA

Reporting a bug

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

I am using python 3.6.8, numba 0.43.1, and cudatoolkit 9.0. The GPU I'm using is NVIDIA GeForce 1050 Ti. When I run the following code I got an error.
from numba import cuda
from math import sqrt
import numpy as np

N = 1000

@cuda.jit
def foo(x, y):
    i = cuda.grid(1)
    if i < N:
        y[i] = sqrt(x[i])

a = np.arange(N)
b = np.zeros(N)
foo[32,32](a, b)

The error message is:
Traceback (most recent call last):
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/errors.py", line 627, in new_error_context
    yield
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 258, in lower_block
    self.lower_inst(inst)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 301, in lower_inst
    val = self.lower_assign(ty, inst)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 454, in lower_assign
    return self.lower_expr(ty, value)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 913, in lower_expr
    res = self.lower_call(resty, expr)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 706, in lower_call
    res = self._lower_call_normal(fnty, expr, signature)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 877, in _lower_call_normal
    impl = self.context.get_function(fnty, signature)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/targets/base.py", line 551, in get_function
    return self.get_function(fn, sig, _firstcall=False)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/targets/base.py", line 553, in get_function
    raise NotImplementedError("No definition for lowering %s%s" % (key, sig))
NotImplementedError: No definition for lowering <built-in function sqrt>(int64,) -> float64

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "test.py", line 15, in <module>
    foo[32,32](a, b)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/cuda/compiler.py", line 765, in __call__
    kernel = self.specialize(*args)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/cuda/compiler.py", line 776, in specialize
    kernel = self.compile(argtypes)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/cuda/compiler.py", line 792, in compile
    **self.targetoptions)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/cuda/compiler.py", line 62, in compile_kernel
    cres = compile_cuda(pyfunc, types.void, args, debug=debug, inline=inline)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/cuda/compiler.py", line 51, in compile_cuda
    locals={})
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 941, in compile_extra
    return pipeline.compile_extra(func)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 372, in compile_extra
    return self._compile_bytecode()
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 872, in _compile_bytecode
    return self._compile_core()
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 859, in _compile_core
    res = pm.run(self.status)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 253, in run
    raise patched_exception
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 244, in run
    stage()
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 731, in stage_nopython_backend
    self._backend(lowerfn, objectmode=False)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 681, in _backend
    lowered = lowerfn()
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 668, in backend_nopython_mode
    self.metadata)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/compiler.py", line 1063, in native_lowering_stage
    lower.lower()
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 177, in lower
    self.lower_normal_function(self.fndesc)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 218, in lower_normal_function
    entry_block_tail = self.lower_function_body()
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 243, in lower_function_body
    self.lower_block(block)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/lowering.py", line 258, in lower_block
    self.lower_inst(inst)
  File "/home/marco/miniconda3/lib/python3.6/contextlib.py", line 99, in __exit__
    self.gen.throw(type, value, traceback)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/errors.py", line 635, in new_error_context
    six.reraise(type(newerr), newerr, tb)
  File "/home/marco/miniconda3/lib/python3.6/site-packages/numba/six.py", line 659, in reraise
    raise value
numba.errors.LoweringError: Failed in nopython mode pipeline (step: nopython mode backend)
No definition for lowering <built-in function sqrt>(int64,) -> float64

File "test.py", line 11:
def foo(x, y):
    <source elided>
    if i < N:
        y[i] = sqrt(x[i])
  ^

[1] During: lowering "$18.5 = call $18.1($18.4, func=$18.1, args=[Var($18.4, test.py (11))], kws=(), vararg=None)" at test.py (11)

Just to add, this happens for many functions from the math module listed on https://numba.pydata.org/numba-doc/dev/cuda/cudapysupported.html#math

Thanks for the report. The cause of the problem you are seeing is that the CUDA math functions that are bonded to the math module (for ease) only support operating on floating point types and there is no automatic type cast from e.g. integer input to some floating point input. In the sample provided, a is defined through np.arange which causes it to be an integer type, and it is this on which the sqrt call is made. Changing the code to:
from numba import cuda
from math import sqrt
import numpy as np

N = 1000

@cuda.jit
def foo(x, y):
    i = cuda.grid(1)
    if i < N:
        y[i] = sqrt(x[i])

a = np.arange(float(N))
b = np.zeros(N)
foo[32,32](a, b)

works fine.
I am however going to mark this as a bug because this is an easily detected set of circumstances and the error message should be improved.

I see. Thanks for the reply.

This will be fixed by #6152. Closing this issue as it's duplicated by #4618.


Update the storage.md to fix typo, and some grammar/format issue

Update the storage.md:

Use the correct naming: write-ahead log, not write-ahead-log
Add missing word:
-> it is not arbitrarily scalable or durable in the face of disk or node outages and should be treated as you would treat any other kind of single node database.
Add comma for better readability
With proper storage durability and planning + "," +  storing years of data in the local storage is possible.

Thanks!


EditorOptions.toolbarItems type definition is incorrect

Describe the bug
The new nested array stated in the v3.0 docs is not reflected in the type definitions.
To Reproduce
Steps to reproduce the behavior:

use this code:
new ToastUIEditor({
el: myEl,
toolbarItems: [
["heading", "bold", "italic"],
["hr", "quote", "ul", "ol"],
["link"],
],
});

The type is incorrect

Expected behavior
Type is correct

@lucaslcode
Thanks! I will fix it.

@lucaslcode
The bug has been fixed and released in v3.0.1.


Fix deleting screenshot GitHub Action job

Idea Description
https://github.com/DroidKaigi/conference-app-2022/blob/main/.github/workflows/CommentPR.yml#L106
fatal: not a git repository (or any of the parent directories): .git
Error: Process completed with exit code 128.

Reference images and links
https://github.com/DroidKaigi/conference-app-2022/actions/runs/3117853645/jobs/5056772576

🙋

Thanks! Assigned 👍 You can check it by creating test pr.

I think #754 resolved this issue:
https://github.com/DroidKaigi/conference-app-2022/actions/runs/3122425253/jobs/5064355002

Looks great! Thanks!


docs(README.md): Add definition for alpha

Alpha didn't have a meaningful definition and assumed people already knew what it was.
Credit for definition: http://www.puzzlr.org/basics-of-d3-force-directed-graphs/

Agree that an explanation of alpha is welcome, but I'm not sure I would use the metaphor of energy—this is a physical quantity with a precise meaning, that might be computed from the particles speeds, positions in a field, and so on. Here, we're talking about a sort of time step multiplier specified externally.
In simulated annealing (which README evokes by talking about "reheating" the simulation), alpha might be called temperature (https://en.wikipedia.org/wiki/Simulated_annealing#Overview), but strictly speaking, the temperature of a gas is the mean kinetic energy of its particules, so if we rule out "energy" we should rule out "temperature" as well. heating factor or heat might a better name for a "hand-waving" explanation.

Agree that an explanation of alpha is welcome, but I'm not sure I would use the metaphor of energy

@Fil I've made some tweaks but generally taken the structure you've proposed:

the only major change was as the simulation "cools down" rather than implying that the alpha is the thing that cools down the simulation

I just thought that that helped the metaphor read better.

I also added a link to simulation-restart, so that they could be understood together if needed

I think saying something like “this is roughly analogous to temperature in simulated annealing” would be sufficiently imprecise to avoid confusion while being helpful.


[Testing] Enable ci automated testing

Closes https://github.com/windranger-io/windranger-governance/issues/55.

question: why are both the push and pull_request being run by this pull request.
Expected behaviour would be only the pull_request runs with a pull request, push being run on main when the PR is merged across.
e.g. Checks run in this PR https://github.com/windranger-io/windranger-treasury/pull/69

I was able to run Slither locally. It indeed shows some 2 red suggestions and 3 yellow suggestions, which needs to be updated, but it's a whole separate PR and more than 10 green suggestions.

Interesting that it shows red error in OZ code ```
TimelockController._call(bytes32,uint256,address,uint256,bytes<EMAIL_ADDRESS>sends eth to arbitrary user
Dangerous calls:
- (success) = target.call{value: value}(data<EMAIL_ADDRESS>Reference: https://github.com/crytic/slither/wiki/Detector-Documentation#functions-that-send-ether-to-arbitrary-destinations

whole separate PR

Sounds good 👍

error in OZ code

When running Slither it'll be worthwhile excluding certain paths (such as OZ) with the --filter-paths option
e.g.
slither . --filter-paths "BitDAO.sol|node_modules"


schema version required

Codecov Report

Merging #433 (9384a06) into main (d42d103) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #433   +/-   ##
=======================================
  Coverage   93.49%   93.49%
=======================================
  Files          46       46
  Lines        1798     1798
  Branches      392      392
=======================================
  Hits         1681     1681
  Misses        113      113
  Partials        4        4

Impacted Files
Coverage Δ

packages/codegen-ui/lib/validation-helper.ts
98.52% <100.00%> (ø)

Continue to review full report at Codecov.

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


Question: can change default ubuntu user password?

i want to change default ubuntu user passwd
but try to in ubuntu user enter passwd to change new pw and reboot , not work
i added persistent storage for / & home in -d data.img (50G)

If you mount the desktop iso image as vm, passwd would not persistent.
If you just want to use ssh-server for that iso image you should change this and then restart service.
change /etc/ssh/sshd_config PermitEmptyPasswords to yes

if i use server iso is can install to img? not live boot?

If you use the Ubuntu server image, please refer to issue #2 , there is a solution.

thx is work~


Upgrading to the beta version - No hint path defined for ['livewire-ui-modal']

Hello there !
Thank you for keeping it up to date and making it compatible with the V3 of Livewire !
Unfortunately we had trouble when upgrading from the previous version to the beta one
We encountered this error :
No hint path defined for [livewire-ui-modal]
Related to this line in our main blade file :
@livewire('wire-elements-modal')
We upgraded Livewire and the modal package in the same time, and followed the upgrade guide but something apparently went wrong. Any idea what might cause this?

Hi there! Which beta version are you using? I think this should be fixed in the latest release.

@PhiloNL  Same problem. Last beta (2.0.3-beta). Trying to figure out how to fix that :)

It is caused by old namespace (livewire-ui-modal) in modal/src/Modal.php. It fixed in master, but with old namespaces in 2.0.3-beta @PhiloNL @Astalyos

Tagged a new release, let me know if that solved the issue :)

2.0.4-beta fixes problem, thanks @PhiloNL !


Possiblity to use versionCode as well?

Greetings and thanks for the great lib.
Sometimes developers (me for example) publish a new build of the application without changing the versionName and only increase the versionCode.
Is that possible to specify versionCode in the XML?
Also I was wondering why you doesn't use the more efficient and common JSON instead of XML?

https://github.com/javiersantos/AppUpdater/releases/tag/2.6


render: sort not working in Internet Explorer

If this is a duplicate of something else feel free to close but I couldn't find anything that explicitly showed this and it's really rather strange.
If I define a render: { 'sort': 'bla' } using the column headers to trigger a resort ascending / descending doesn't appear to have any effect when using internet explorer.
http://jsfiddle.net/50x1m4bt/8/
Strangely I have been able to in some cases define an order option and it does appear to sort on the initial load but nothing after that, I couldn't seem to replicate this in the fiddle though (maybe since the fiddle isn't on nightly?)

You need to specify the type option as well, otherwise the _ is used for type detection: http://jsfiddle.net/50x1m4bt/10/ .


Find references of tasks inside the cluster

It may happen that you have many tasks on the cluster and you are afraid to edit/delete one of them because you don't know where they are actually used. One useful thing could be to add an action to find existing references of a task.
This can also be extended to other resources. Is X used somewhere?

Same for ClusterTasks

@lstocchi, @siamaksade, and @evidolob
I was thinking If we are deleting the task or ClusterTasks which are being used in pipeline/pipelineRun/taskRun then I will show the warning to the user that it can affect the pipeline and I will delete the taskRun also which are being referred to that Task/ClusterTask. WDYT?
Another question if we are deleting pipeline should I delete the pipelineRun and TaskRun which are being referred to that pipeline?

@sudhirverma in IJ we already have something similar. When a user tries to delete a pipeline or task, we ask if she also wants to delete its related resources. If the checkbox is checked everything is deleted (img below).

The idea to show a warning during deletion is good for me but I don't think if it fully covers everything.
If a task is used somewhere you should also show the warning if that task is being edited (during saving). Other point is, are you going to show the list of pipelines where that task is used? Because the warning is useful but if it doesn't tell me where the task is actually used it may lead to errors. Let's say i have task A used in pipeline B and C (but i'm only aware of pipeline B), when i see the warning i could  click  on OK (because i'm erroneously sure it is only used in pipeline B) and i could accidentally change the behavior of another pipeline without knowing it. To keep it simple maybe we could just create an action that show a list of pipelines  where the task is used?
Probably i think too much and it's highly likely this stuff will only be used by insane ppl like me so let's wait for @siamak's answer , maybe he can tell us how customers want this to work.


metrics: not a number fix

Skip nan values in metrics so the sum and std_div still have sense.
Also change Metric::collect to accept any kind of Collection.

+1


Commands for all scheduled cork functions

closes #97

The command structure was updated during the Aave ABI update, where the subcommand is now the cellar shortname, and a slightly different utility function for sending corks was merged. Given the conflicts, we'll need to update this PR by moving the commands to new filenames and updating them to call the same functions as the existing commands.

Just a ping @hannydevelop, let's prioritize getting this finished so we can include all the scheduled cork commands in the next point release.

@EricBolten all done!


Add VideoRAGQnA as MMRAG usecase in Example

Description
Add VideoRAGQnA as MMRAG usecase in Example
Issues
https://github.com/opea-project/GenAIExamples/issues/358
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)
[ ] Others (enhancement, documentation, validation, etc.)

Dependencies
streamlit
Tests
MultiModalRAGQnA/VideoRAGQnA/tests/test_videoragqna_on_xeon.sh

The test is WIP, and it will only work when https://github.com/opea-project/GenAIComps/pull/621 and https://github.com/opea-project/GenAIComps/pull/603 is merged. please hold
@lvliang-intel @chensuyue  could you help to tag the PR milestone for 1.0?

@BaoHuiling,
About the CI issue, you should put the 'tests' directory in the ‘MultiModalRAGQnA’.
Why do you set two level folders for this example?

@BaoHuiling, About the CI issue, you should put the 'tests' directory in the ‘MultiModalRAGQnA’. Why do you set two level folders for this example?

Hi Liang, thanks for comment, because we are contributing 2 use cases for the MMRAG, see https://github.com/opea-project/GenAIExamples/issues/358#issuecomment-2333005982
I will move the test to upper folder.

Waiting for https://github.com/opea-project/GenAIComps/pull/621/files merged

@lvliang-intel @kevinintel @XuhuiRen @Spycsh Hi all, this PR finished the file strcture update and passed all CI test, could you help to review and merge it? Thanks a lot

@yinghu5 @chensuyue Add more reviewers🙂


full screen option

Added an option to show the dashboard in full-screen mode, without Jenkins' left and top panels

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


Layer ordering is disconnected from visualization and layer selector

This behaviour does not happen always, so I was not able to replicate a clear pattern/steps. But basically sometimes changing the layers order (from a three layers map) has not effect on the layers selector and visualisation.

Already talked with @ramiroaznar offline, closing since there is no steps to reproduce. 💋  thanks mate.


Searching for 86.md

When accessing https://nips.nostr.com/86 (from the cool doc at https://khatru.nostr.technology/), I get an error message: "This nips.nostr.com page can’t be foundNo webpage was found for the web address: https://nips.nostr.com/86 HTTP ERROR 404"
Also, are there plans to add 86.md to https://github.com/nostr-protocol/nips? Thank you.

Sorry, I expected it to be there at this point. Soon.

Thanks ... good things take time ... much appreciated.

Should be fixed now that the NIP was merged.


fix(ingestion): continue of failures

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@treff7es
Can you please review this PR?


feat(postgresql): add support for psycopg3

I have not set psycopg3 as a default dbapi driver for PostgreSQL in pydapper.
But we can change that if you decide to do so.

This looks great, thank you! Once comment - I don’t think we need both psycopg and psycopg-binary as deps. Can we keep just the binary?

This looks great, thank you! One comment - I don’t think we need both psycopg and psycopg-binary as deps. Can we keep just the binary?

From my understanding psycopg-binary is an optional package from this version, when I only include the binary the tests fail because there is no psycopg to import.
https://pypi.org/project/psycopg-binary/ -> You can read here the description

ah okay - can you remove it and reinstall it like
poetry add 'psycopg[binary]' --optional

should add it to the toml how i would expect.  Didn't realize it was just an extra

poetry add 'psycopg[binary]' --optional

Should be good now.

Doesn't need to happen in this PR, but I would also like to add the psycopg3 async api as one of the async drivers.  If you wanted to PR that that'd be amazing, otherwise I'll look to add it in a couple weeks

Doesn't need to happen in this PR, but I would also like to add the psycopg3 async api as one of the async drivers. If you wanted to PR that that'd be amazing, otherwise I'll look to add it in a couple weeks

https://github.com/zschumacher/pydapper/pull/218 here it is :)


Feature Request: Check cmd state deprecation

Preface
Kindly use the check for state.cmd deprecation as an example of adding a check. #37 has to be merged first.
Description
Add the possibility to check for deprecation within the cmd state
The following options have been deprecated:

cmd.run:quiet (since version 2014.1.0)

The following options should be used:

cmd.run:output_loglevel: quiet

Example(s) of an improperly configured state:
getpip:
  cmd.run:
    - name: /usr/bin/python /usr/local/sbin/get-pip.py
    - unless: which pip
    - require:
      - pkg: python
      - file: /usr/local/sbin/get-pip.py
    - reload_modules: True
    - quiet

An example of properly configured state:
getpip:
  cmd.run:
    - name: /usr/bin/python /usr/local/sbin/get-pip.py
    - unless: which pip
    - require:
      - pkg: python
      - file: /usr/local/sbin/get-pip.py
    - reload_modules: True
    - output_loglevel: quiet

Fix requirements
A fix for this issue should (at least):

Notify the user of using a deprecated option
Notify the user since what version the option will be or has been deprecated
A suggestion on what option to replace it with
Contain unit tests for the check

As this check has been deprecated since 2014, we might want to consider not implementing this check.

Another check to add would be cmd.wait, as cmd.run should be used together with onchanges.

After a long wait, here's the regex that'll match properly:
^\s{2}cmd\.run:(?:\n.+)+\n^\s{4}- quiet$

Tested against the following states, of which only the first should fail:
getpip:
  cmd.run:
    - nas
    - name: /usr/bin/python /usr/local/sbin/get-pip.py
    - unless: which pip
    - require:
      - pkg: python
      - file: /usr/local/sbin/get-pip.py
    - qaas
    - reload_modules: True
    - test
    - quiet
    - mopper
      - quiet
    - more

getpip:
  cmd.wait:
	- name: test
    - quiet

getpip:
  cmd.run:
    - name: /usr/bin/python /usr/local/sbin/get-pip.py
    - unless: which pip
    - require:
      - pkg: python
      - file: /usr/local/sbin/get-pip.py
    - reload_modules: True
    - output_loglevel: quiet test
    - more output

See it in action here: https://regex101.com/r/D6SFFH/2

Another check to add would be cmd.wait, as cmd.run should be used together with onchanges.

A regex to match against cmd.wait:
^\s{2}cmd\.wait:(\s+)?$

See it in action here: https://regex101.com/r/qXdwDr/1


CSS styles are applied only after element is revealed

I transition an image into a headline.
class App extends Component {
constructor() {
super();
this.state = {
loading: true
};
}
componentDidMount() {
setTimeout(() => {
this.setState({loading: false});
}, 2000);
}
render() {
return (

{this.state.loading && }
{!this.state.loading && React-Overdrive Test}

);
}
}
The basis is the default create-react-app setup: https://github.com/joetm/overdrive-test
What I notice is that the style of the headline is not applied immediately.
I see the headline appear with black text left-aligned. Then it jumps to being center-aligned with white text.
Anything that can be done in this case?

While the animation takes place, the elements are clones and moved without their wrapping elements. So while it inherits the size & position, it may lose colors, text-alignment that is styled on the parent elements.
I suggest adding the colors and text-align to the element itself (via CSS) (or to the body element).

@berzniz Love the concept - this is the first library I've found that could solve the problem of how to achieve complex multi-element transitions such described for Material Design: https://material.io/guidelines/motion/material-motion.html#material-motion-what-makes-a-good-transition
However, passing styling from a parent to child, through props or CSS className is a common pattern, particularly where the child component may be provided by a 3rd party library so can't be styled any other way. (Even within [Material-UI](http://material-ui.com] we consume and in some cases restyle our own components to build up more complex components.)
This may not be a solvable problem, but just wanted to add some support to the issue.

@mbrookes - these are some good points.
Since the elements are cloned and attached to the body, they will indeed lose all CSS rules inherited from their parents. Usually frameworks (such as bootstrap) are working on the body level so styles still apply.
There could be a solution to use getComputedStyle on all child elements and capture their styles, but this could be costly (especially for deeply nested DOM trees). I'll try to play with that idea.

This could be partly solved by specifying a "root" element to nest the clones under. If the root doesn’t have position applied the absolute positioning should still work, but some CSS cascade would apply.


Fix for issue #20 caused by breaking changes to sqlalchemy

Added explicit reference to SQLAlchemy and ensure the latest versions of pip, setuptools and wheel packages are installed in the ubuntu:20.04 image before attempting to build the wheel.

Hi,
Thanks for doing this, but I see you've upgraded the Flask version to v2.1. As I haven't tested SnitchDNS with that version, I'm not very comfortable pushing this onto the main branch.
However, I've fixed the SQL issue by hardcoding the sqlalchemy version at https://github.com/ctxis/SnitchDNS/commit/52dd4c5359c71b6cd74f227726785e261ee664b6
Having said that, I'll close this pull request as that fix should be working just fine (same issue was at the crackerjack repo too).
Thanks,
Pavel


CentOS 8 no longer receiving updates

Hi,
you currently suggest to install an unsupported version of CentOS here. I am guessing you either want to switch this quickstart help to centos8 stream, rhel8 or another rhel8 clone.
Greetings
Klaas

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

ID: 73663d3d-b9a8-e93e-9feb-6105c48567a9
Version Independent ID: 11a384f4-f7fe-2887-09d3-465ca94cfab9
Content: Quickstart - WildFly on CentOS - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/redhat/wildfly-on-centos.md
Service: virtual-machines
Sub-service: redhat
GitHub Login: @Theresa-Nguyen
Microsoft Alias: bicnguy

Refs https://github.com/openlogic/AzureBuildCentOS/issues/117

Hello @Klaas,
Thank you for your feedback. We will investigate and update the thread further.

@Theresa-Nguyen,
Can you please review this and answer?

@PriyankaRudramaina , please note my username. Thanks.

@PriyankaRudramaina , please note my username. Thanks.
@klaas The previous comment was for document content author Theresa. It wasn't meant for you.

Hi @klaas,
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Add implementation for load_with_datetime in Python package.

Description
We would like to use load_with_datetime method implemented in Rust binding in our Python project. Add the implementation in Python package.

Hmm... the test failed. Let me check it later.

It's weird, the tests passed on my local machine:
In [1]: %load test.py

In [2]: # %load test.py
   ...: from deltalake import DeltaTable
   ...:
   ...:
   ...: table_path = "./rust/tests/data/simple_table"
   ...: dt = DeltaTable(table_path)
   ...: dt.load_with_datetime("2020-05-01T00:47:31-07:00")
   ...: assert dt.version() == 0
   ...: dt.load_with_datetime("2020-05-02T22:47:31-07:00")
   ...: assert dt.version() == 1
   ...: dt.load_with_datetime("2020-05-25T22:47:31-07:00")
   ...: assert dt.version() == 4
   ...:
   ...:

In [3]: dt.version()
Out[3]: 4

In [4]: dt.load_with_datetime("2020-05-04T22:47:31-07:00")

In [5]: dt.version()
Out[5]: 3

I'm not sure why it's failing in CI. Could @fvaleye @houqp take a look please?

Learned from time_travel_by_ds test case, we should set delta log mtime before running the test. Problem solved.


add doc for workload-rabalancer

What type of PR is this?
/kind documentation
/kind feature

What this PR does / why we need it:
add doc for workload-rabalancer
Which issue(s) this PR fixes:
Fixes part of https://github.com/karmada-io/karmada/issues/4840
Special notes for your reviewer:

quick click


[BUG] Build failure after installing React Markdown package | (vfile package import issues)

What happened?
I'm using Plasmo with React JS, Typescript, and src dir. I added the react markdown package and then the build started to fail.
After digging for some time, I found one package named vfile in the node modules in which some import paths are configured in the package's package.json file.
These imports are not resolved properly in the build resulting in errors.

Version
Latest
What OS are you seeing the problem on?
Linux
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

a temporary resolution method

I am getting this issue too. I saw the temp solution is to upgrade vite, but it must be upgraded by Plasmo author himself

I also met this issue when using Plasmo 0.89.3.


Bulk Blasting items can be destroyed if not picked up and user quits the game and relogs in

Steps to reproduce:

Put an item on a depot in a bulk blasting setup (for example, crushed copper ore)
Wait until the ore is cooked into an ingot
Quit the game
Reopen the game
The ingot will start cooking again and be destroyed.

Please submit that to the Create devs and not here

Because this is probably not coming from the modpack

Happy for this to be closed if it's Create mod that's the issue :)

Have you tried throwing the ingots into the blasting ? If they disappear, then that means that reloading the world makes the items think they are being smelted, instead of finished being smelted

https://github.com/cech12/WoodenBucket/issues/19

It's a create / minecraft thing when you log out it resets the entity data and anything in a create bulk blast system that has no smelt recipe will be destroyed

Fair enough. If it's the Create mod, this isn't the right place to raise. it. I'll close it.


Does this work on Windows?

Hello, just came to know about this from Reddit and I found this amazing. But I tried it on windows but facing this issue.
Redirection is not supported.

Hello, just came to know about this from Reddit and I found this amazing. But I tried it on windows but facing this issue.
Redirection is not supported.

can you guide me at what stage did you get this error, where you able to install all the dependencies?

So started the installation by running setup.py and then I installed the dependencies i.e. MPV and web-torrent-cli.
After that when I run main.py I get the above error.

pip install windows-curses
install this

Sorry, my bad, I had used git bash to clone the repo and forgot to switch to the command prompt and run the script directly in the bash terminal and bash was restricting the redirection. Sorry and thanks for the quick replies.

No issue share the project with your friends and family if you liked it ☺

Yes sure will share it. And also try to do some contributions related to GUI if possible.

GUI will consume more resources then needed still i will see if it's a popular demand

Okay, I was talking about I will try to do a contribution. :)


Add docs building dependencies

Hey @wavefrontshaping,
Thank you for your suggestions in adding build dependencies for the docs in a file!
For now, it is listed in the github action.
https://github.com/QITI/pySLM2/blob/61cbc71516c5df131d9e5ce3ac5350dea177e2fd/.github/workflows/sphinx.yml#L30
You can also find the documentation built by the CI here.

Addressed in https://github.com/QITI/pySLM2/pull/11


refactor: refactor from scratch

Codecov Report
Base: 93.53% // Head: 62.67% // Decreases project coverage by -30.85% :warning:

Coverage data is based on head (3a03983) compared to base (c11dd51).
Patch coverage: 25.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #21       +/-   ##
===========================================
- Coverage   93.53%   62.67%   -30.86%
===========================================
  Files           7        7
  Lines         371      568      +197
===========================================
+ Hits          347      356        +9
- Misses         24      212      +188

Impacted Files
Coverage Δ

src/facade.rs
48.75% <0.00%> (-51.25%)
:arrow_down:

src/lib.rs
100.00% <ø> (ø)

src/pixel_engine.rs
90.47% <0.00%> (-9.53%)
:arrow_down:

src/sub_image.rs
42.75% <0.00%> (-35.92%)
:arrow_down:

src/bindings.rs
62.38% <62.37%> (-31.27%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Consideration for Production Use

I am considering the use of this API in production to replace various applications. Seen some changes that are breaking changes and would like to know if more of such changes would be introduced and the stability of the API for production.
Thanks

Sorry for the late reply. It's stable now. We put it out in beta, and it took a while for us to get feedback from people to finally make it a stable API

I am also putting this into production, but, I do hope the api won't trip offline often as it's to be used in a critical system. Thanks


Eof token

Refactor lexer part 2

@kdy1 Can you look? Here is the breaking change, now tokens should be ended by EOF token, it is in spec and allow to improve recovery mode in some places

Seems like tests are failing due to some match expressions. @alexander-akait Can you fix itt?

@kdy1 Done

Let's close, maybe we revisit it late, now None is EOF, it still can be useful if we will need stoppable parsers, but currently we don't need it, anyway CSS and HTML parsers design to be them


added additional stakeholder types

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Looks good @mistwire, thanks for contributing!


Remove unnecessary theme config code that cause app crash before API 21

There is no theme style AppTheme.NoActionbar in style.xml file, then causing app crash on devices before API 21. Changing the activity's theme back to AppTheme is an easy and clean solution.

but this will also remove the feature changing the tab background color


Action Updater for FORCE_DURATION nonfunctional

Action updater for FORCE_DURATION should be updating the required hold length per trial/reward, but it remains static.
In contrast, updaters for Upper and Lower force threshold are working as expected.
Reproducible with this task_logic:
AindForceForagingTaskLogic_ROI_trial_test_updaters_on_reward.json

Can you attach the json file to make it easier for us to test? thanks!

AindForceForagingTaskLogic_ROI_trial_test_updaters_on_reward.json

Fixed by a604460


Transfers this repository to another organisation ?

This repository has been created in the QwantResearch organization by convenience, should we move it to another organization ?
And if we do it, should we create a Cosmogony organization or should we use an existing one ?

done !


Support for Python 3.10

I’m really excited about the potential of mountaineer and looking forward to use it in some projects. However most of my experiments are tied to Python 3.10 because of other dependencies, and upgrading to Python 3.11 isn't feasible just yet.
Would you consider adding support for Python 3.10? I understand this might add some maintenance overhead, but I've already taken a stab at it. There are a few hiccups with type subscripts and the absence of string enums is a little annoying. Yet most tests are passing.

@oguzhanmeteozturk This ended up being easier than expected. I've merged in initial support in https://github.com/piercefreeman/mountaineer/pull/105. 3.9 would be a tougher lift, mostly on the type-hinting side since it lacks language-level support for union-like primitives (ie. str | None).
Let me know if you see any issues in v0.5.0.dev2. Going to close out this thread for now.


queries that use AsExpandable and .ToListAsync() do NOT run asynchronously

I'm running a number of queries that take advantage of .invoke to re-use where clauses as well as select projections.
However, I've found that the ExpandableDBAsyncEnumerator doesn't actually run the queries asynchronously. It forces the expression to be run synchronously.
This is a pretty major issue for me. Has anyone else come across this? Does anyone have a suitable fix?
ie.
public async Task GetObjectAsync()
{
var obj = await dbSet.AsExpandable().FirstOrDefault();
return obj;
}
var t1 = GetObjectAsync();
var t2 = GetObjectAsync();
await Task.WhenAll(t1,t2);
These two calls to getobjectasync will NOT run asynchrounously - they'll only run synchronously.
Any ideas?

Beware that there is EntityFramwork limitation:
http://stackoverflow.com/questions/24702183/does-entity-framework-support-parallel-async-queries
    [Fact]
    public async Task ExpressionAsyncTest()
    {
        Expression<Func<int, bool>> condition = a => a % 2 == 0;

        var expanded = condition.Expand();

        var dbset = new[] {1, 2, 3, 4}.AsQueryable(); // from db
        var xs = dbset.AsExpandable();

        var result =
            from x in xs
            where expanded.Invoke(x)
            select x;

        var t1 = result.ToListAsync();
        var t2 = result.Skip(1).ToListAsync();

        Assert.Equal(2, (await t1).First());
        Assert.Equal(4, (await t2).First());
    }

As far as I know, LINQKit is not the one that limits you...

Running queries in parallel may not be supported, but running a single query async still has a lot of benefits since the current thread becomes free in the meantime; thus, he's available for your application or your host (web server, ui thread, ...).
The problem with the current implementation is, that the async enumerator / provider of Entity Framework isn't called, a synchronous Task.FromResult is used instead. A detection if the current enumerator / provider is in fact async and an appropriate direct call should solve this.
The pull request i've submitted will help.

Thanks for the responses.
@vinneyk Yeah, you're right I wrote the code down incorrectly - it should have read FirstOrDefaultAsync.
@Thorium You're right, I can't make 2 calls to the same dbset at the same time - I realised that after I lodged this issue. I ended up solving that by using multiple contexts.
However, as @axelheer notes, there's still an issue actually running the code (even a single call) asynchronously.
If/when I get a chance I'll look at the pull request as that would be extremely helpful. Because of this issue I basically had to remove all my linqkit code entirely. It would be nice to be able to use it again.
Thanks Axel!

I tested the code, seemed to work fine. Thanks!
I updated my open pull-request to contain also merge for this.

Thanks for all the work. I merged in the changes for this issue.


Add pagination support for first and last

Right now i am using following code:
<?= $this->Paginator->numbers([
       "size"=>"large",
       "prev"=>"i:chevron-left",
       "next"=>"i:chevron-right"
        ]);?>

and this generates pagination with prev next buttons. Please add support to append first/last buttons to pogaination as well.
Using next() and prev() causes generation of separate unordered lists.
Maybe i am using it wrong?

This is clearly a bug that I was aware of but I simply forgot to fix it, I will try to publish a patch ASAP.

Should be fixed by 167385c0e085eab3f9ebf4e4d940dafe13191c56, feel free to try it!

would be very nic3
21 lut 2016 13:25 "Mikaël Capelle"<EMAIL_ADDRESS>napisał(a):

Should be fixed by 167385c
https://github.com/Holt59/cakephp3-bootstrap-helpers/commit/167385c0e085eab3f9ebf4e4d940dafe13191c56,
feel free to try it!
—
Reply to this email directly or view it on GitHub
https://github.com/Holt59/cakephp3-bootstrap-helpers/issues/75#issuecomment-186814527
.

Ic checked out provided commit (current head i belive) and i have no navigation buttons at all (besides numers)

And using code similar to the one above:
 $this->Paginator->numbers([
                            "size" => "large",
                            "first"=>"<<",
                            "last"=>">>",
                            "prev" => "i:chevron-left",
                            "next" => "i:chevron-right",
                            "class" => "no-margin"
                        ]);

I did not test it using small datasets and it seems to not be working with small datasets (the ones that do not call _modulusNumbers. I have made a new update, that should work with any dataset, but I did not test it that much so it may still be buggy...

Are those dotted buttons (always disabled) intentional?
Works better now, last first buttons shows only if there is indeed more pages that are not listed yet (cool feature i think)

These dotted buttons are ellipsis created by Cake itself. I think these are already presents with the default Paginator helper (you can check).
It should not be that difficult to add a custom option to disable it.

You are right, but looks like there are not working as supposed to work in cake's pagination helper.
Anyway, current head is working so bug is fixed. Thank you for quick respose.

What do you mean by there are not working as supposed to work in cake's pagination helper?

I just dont follow what they supposed to to mean.
It would make sense to add them like this:
| < Last | 1 | 2 | ... | 10 | 11 | 12 | 13 | 14 | 15 | ... | 56 | 57 | >Next |
so it should either show first and/or last link, OR use ellipsis to allow first and last page to be listed in pagination so user can go to last page by clicking on it. But if there is a last,first link displayed then elipsis could be ommited.
And the docs specifies that :

The generated number links will include the 'ellipsis' template when the `first` and `last` options

 and the number of pages exceed the modulus. For example if you have 25 pages, and use the first/last
 options and a modulus of 8, ellipsis content will be inserted after the first and last link sets.

So if I understand right it would be like
| .. | <pref | 5 | 6 | 7 | 8 | >next | ... |
and that seems to be wrong/useless IMHO.

I understand what you are saying. Actually, the behavior of Cake is the following:

If last (or first) is not an int, then it is used as the text for a link to the last page, and an ellipsis (...) is added before (or after) if there are pages that are not visible in the current numbers (e.g. you have 20 pages, but the last visible page number is 12).
If last (or first) is an int, then you get link to the N last pages (where N = last) with an ellipsis before, e.g. if last is 4 you get 1 | 2 | 3 | ... | 59 | 60 | 61 | 62.

Cake's behavior for first and last options is perfectly right, the problem arises when you try to add next and prev buttons which are not available in the default Paginator helper. Since the ellipsis is part of the last and first block (these are generated by _firstNumber and _lastNumber), the prev and next are added between the ellipsis and the numbers.
Having something like this:
< Prev | 1 | 2 | ... | 10 | 11 | 12 | 13 | ... | 38 | 39 | > Next

Seems right to me, but what about the following:
< Prev | << First | 10 | 11 | 12 | 13 | >> Last | > Next

Looks strange IMO... Maybe the first case should be chosen when first / last is an int, and keep the current behavior (without ellipsis) if it is a string? What do you think?

I have implemented the mentioned behavior in the latest commit, feel free to test it and tell me what you think of the output.

< Prev | << First | is not intuitive as you would expect the opposite order.
I will have to wait with testing till we update to cake 3.2~ as this is the required dependency for helpers now.

You can check the new behaviour with the v3.0.5 release which still require CakePHP ~3.0.


The JS/TS language service immediately crashed 5 times

I am getting the following error with the Vue - Official v2.0.10 extension with the https://github.com/n8n-io/n8n repository

The JS/TS language service immediately crashed 5 times. The service will not be restarted. This may be caused by a plugin contributed by one of these extensions: Vue.volar. Please try disabling these extensions before filing an issue against VS Code.

Tried with:

Vue › Server: Hybrid Mode : false --> Still crashes
Vue › Server: Hybrid Mode : auto --> Still crashes
Vue › Server: Hybrid Mode : true --> Still crashes

VSCode version:
Version: 1.88.0
Commit: 5c3e652f63e798a5ac2f31ffd0d863669328dc4c
Date: 2024-04-03T13:28:09.495Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Darwin arm64 23.4.0

I have the same issue, all extensions disabled except Vue.
The issue happens on a fresh created vue project
Plugin version: 2.0.12
VSCode version: Version: 1.88.0 Commit: 5c3e652f63e798a5ac2f31ffd0d863669328dc4c Date: 2024-04-03T13:28:09.495Z Electron: 28.2.8 ElectronBuildId: 27744544 Chromium: 120.0.6099.291 Node.js: 18.18.2 V8: 12.0.267.19-electron.0 OS: Darwin arm64 23.4.0

switch hybrid mode to false can help you.
the setting is vue.server.hybridMode

Installing earlier version (2.0.6) of vue extension fixed it for me

The only thing that solved for me was use the 1.87v from codium. I can't find Microsoft vscode on older versions for ubuntu.

I went back to v1.8.27 to fix the issue.

I can't reproduce the error in https://github.com/n8n-io/n8n. Please try updating to v2.0.12, if it still crashes, you can configure "vue.server.hybridMode": false to completely disable the TS plugin (only applicable to >= 2.0.11).

Still got this error with the latest versions of VSCode and Vue extension

Vue
v2.0.12

VSCode
Version: 1.88.1 (Universal)
Commit: e170252f762678dec6ca2cc69aba1570769a5d39
Date: 2024-04-10T17:42:52.765Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Darwin arm64 23.4.0

Okay, I'm having this problem, but strangely, only in a React project (I use both Vue and React heavily at my job). In a Vue project, it's fine.
The JS/TS language service immediately crashed 5 times. The service will not be restarted. This may be caused by a plugin contributed by one of these extensions: Vue.volar. Please try disabling these extensions before filing an issue against VS Code.

Setting hybridMode to false fixed it for me (in React, TS doesn't crash anymore, and I still have type safety in Vue).


0.19 port

Any plans to port this library to support the newly released elm 0.19?

Hi Bidigo,
I will not  be maintaining this library anymore. Sorry for that! The main reason is that elm team will not integrate this library into their elm libraries, because elm-pouchdb contains js code.
Thanks

Hi Etienne,
thanks for the info, that's fair enough.

@eticav could you elaborate on what the issue is? Would it be worth my time to port this to pure elm? REALLY need a pouch adapter for elm. Thanks.

I see the code uses native binding quite a lot. Is that the problem? I read somewhere that starting with elm .19, no more native, except for the core team?


Handle the +/- similarly to how done in habitus

Also, I think this should normalize all Unicode characters here:
https://github.com/clulab/processors/blob/master/main/src/main/resources/org/clulab/processors/bionlp/unicode_to_ascii.tsv
This normalization should be turned on by default, but be optional in Processor.
Thanks!


Update getChannelInfo to getChannelProperty everywhere

Also implement getChannelPropertyNames()

@magland can I close this?

yes. Closing.


Spring Boot 3.3.0 support

Spring Boot 3.3.0 has now been released, can it be added to the compatibility list?
https://github.com/spring-projects/spring-boot/releases
Compatible versions:
https://github.com/Azure/azure-sdk-for-java/wiki/Spring-Versions-Mapping

This compatibility check is an anti-pattern. Can't it just be removed? If the library doesn't work, it won't work.

Yeah, I think documentation of tested combinations should be enough.

From: Derek Griffiths @.>
Sent: Saturday, May 25, 2024 4:44:16 PM
To: Azure/azure-sdk-for-java @.>
Cc: dmarsh26 @.>; Author @.>
Subject: Re: [Azure/azure-sdk-for-java] Spring Boot 3.3.0 support (Issue #40346)
This compatibility check is an anti-pattern. Can't it just be removed? If the library doesn't work, it won't work.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-java/issues/40346#issuecomment-2131312723, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB3JHP75HAMWKDM3ND3DOQ3ZECWVBAVCNFSM6AAAAABIHW7JPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZRGMYTENZSGM.
You are receiving this because you authored the thread.Message ID: @.***>

This compatibility check is an anti-pattern. Can't it just be removed? If the library doesn't work, it won't work.

Yes, I think we could remove this. The compatibility check is more like a protection for ourselves.

Hi,
Any updates on this issue? Currently, we can’t migrate our project to Spring Boot 3.3.0 due to this functionality.
Thanks a lot.

Hi,
We got problems since we are using Dependabot to manage our dependency updates - and dependabot updated to latest Spring Boot (3.3.0, and not 3.2.6) in our parent project and then we got the above error in our 25+ microservices...

Hi!
We are currently unable to migrate our projects to Spring Boot 3.3.0 due to this functionality. Any updates on this?
Thank you very much!

@LennartHaggkvist U need to control the dependency chain independently if you really do "micro-services" :) You're dependabot PRs might break, but that should be it.
@mnhock the change on azure side will probably be nothing else than bumping the spring version, as confirmed in https://github.com/Azure/azure-sdk-for-java/issues/40346#issuecomment-2134723138 and seen in previous releases on this subject - this issue is really an invented issue, it either works or it doesn't: https://github.com/Azure/azure-sdk-for-java/issues/40346#issuecomment-2131312723

We are working on this change and a new release will be out early next week if everything goes fine. And in the next version, we will disable the compatibility verifier by default.

Disabled in https://github.com/Azure/azure-sdk-for-java/pull/40407

To anyone who is currently blocked by this: You can just set spring.cloud.azure.compatibility-verifier.enabled=false to disable that compatibility check.

5.13.0 has been released with support for Spring Boot 3.3.x.

close for released


Suggestions

Hi Prof. Wu,
Here are some suggestions for this package:
Frameworks:

tensorflow
keras
pytorch
pytorch-lightning

Modules

scikit-gstat

Visualization

scikit-plot

Thanks for the suggestions. Will work on this once the package is available on conda-forge.

Pyspatialml?

@RichardScottOZ Pyspatialml is not available on conda-forge. It needs to be available on both PyPI and conda-forge before it can be added to geospatial-ml.

@davemlz All suggested packages have been added. Thanks for the suggestions.


How do I style?

How do we apply external styles?  I can override the <object> style, but not sure how to override anything in shadow dom using rules or variables.
example:
https://jsfiddle.net/easierbycode/4ps10hx2

@easierbycode use object.contentDocument to get ref on the document element. On the other case it's probably better in future to allow style modification with custom css variables... I need to investigate possible use cases for presentation styling.


Run tests from a test-integration directory

Since cookbooks have the test/integration directory already, it would be great if we could use a audit resource to trigger the inspec tests via a Chef run. The resource name could be
inspec do
  dir 'test/integration/...'
end

👍


Neither display empty gender nor birthday

If there is no gender or birthday it shouldn't be displayed at all.

I am also going to hide gender if other is selected. Let me know if you want other to be displayed.


Access evironment variables in component

Cannot able to access environment variables in components

How are you trying to access them and how is it not working?
msgflo-nodejs does not do anything with envvars, so I don't immediately understand the problem.


Smart commits are not working as expected.

Hello. We've setup Jira connector for Github between our github organization and Jira cloud.
However, we can't find github smart commits working, although every features except smart commits are working properly.
We can find our commits and PRs successfully linked with issues, but even if we have properly wrote smart commit syntaxes, it does nothing with our issues in JIRA, including transitions, comments, timestamping.
I've searched through whole documentation about enabling smart commits with github organizations without enterprise plan, but official documentation about enabling smart commits looks outdated as it describes how to configure smart commit behavior through DVCS connector, which is deprecated and not used at the moment.
I've confirmed that our Jira app for github organization has full access to all repositories as shown below:

...and our github connector app is also properly configured from Jira cloud too.

The connector app itself looks like works as expected, since we can see the linked commits from issue:

but these commits has no effect in transitioning issue status, add comment and any other behaviors supported by smart commits.

Am I missing something here? Was smart commits feature itself deprecated for the users don't have github enterprise plans enabled?

I was not able to make smart commits properly working, but I found that we can achieve what we want to do with commits using Automation rules feature of Jira project configuration.
Thus closing the issue.


Make dsl installation optional

Fixes #2

It seems that hasYarn always returns true regardless if a package-lock.json exists. This means that npm projects end up with a yarn.lockwhich is undesirable.
I've taken the liberty of refining that functionality to test using the default yarnOrNpm method and
const hasYarn = yarnOrNpm() === 'yarn'

@RonaldvdH can you review the last commit? The rest of this PR looks good to me.


Bug: if HP raidcard have no discs

We have a box with multiple raidcards and we still dont use the second, thats how we found this "bug"

Thanks, LGTM


Add pipeline config for Batch abd Send

Batch is a new transform available in the SDK, need to update the sample configuration.toml to show how to configure it.

Also need to add batch and send to the configurable pipeline


h: cache element by stack instead of creating anew

Are there cases where we need new element each time instead of modifying prev one? map?
Does that mean if mapping case was handled separately, there would be no sense to create element more-than-once per h`el` call?
. On another point - is that faster to just have spect/h as hyperscript, and for html syntax use htm?

htm static cache did exactly that - not useful.


Problem with passing data to vue component using .sync

Version
2.5.17
Reproduction link
https://jsfiddle.net/eywraw8t/364640/
Steps to reproduce
Pass json data as html attribute to a component using .sync
What is expected?
Data should be available as prop and can be used.
What is actually happening?
JSON gets distorted and console throws error.

I am trying to pass data from a laravel blade view to the vue component. I am using sync to make the prop reactive. However this does not work. Passing the json as static data does work.

Using v-bind.sync with a literal object, such as in v-bind.sync=”{ title: doc.title }”, will not work, because there are too many edge cases to consider in parsing a complex expression like this.

A simple data object like v-bind.sync="{ mail<EMAIL_ADDRESS>'}" would also be falling into that criteria I guess? If that's the case - to get my prop reactive - best practice would be to reassign my data object within a created method to a reactive data object? Like: https://jsfiddle.net/0qmh4rvg/ ?
Thanks!

Just pass an object from your data, it is reactive.
data () {
  return {
    object: 'hi'
  }
}

Then pass it with v-bind="object".

Makes sense - however I am passing the data from a laravel blade template. The vue js logic is compiled in js files which are loaded at runtime. Passing the data directly into data like:
data () {   return {     object: <?php json_encode($dataobject) ?>   } }
won't work in my case. I need to go the longer route via passing the data as attribute in the html component and it should be reactive.
Another option would be to assign my data within the blade template to the window object and then reassign it to data like:
blade template:
`

`
In my js file:
data () {   return {     object: window.data   } }
However this does also not feel like the most clean approach.

You need to put it in data

@mr-jingles In your case, I'll suggest using regular events instead of .sync(in another way update: event)

@javoski - thanks for you input. I will try that.. Till now I thought .sync is just syntactic sugar for the update event.

Till now I thought .sync is just syntactic sugar for the update event.

Exactly. So you cannot do something like @update:mail="{ ... } = $event".

Hmm ok - seems that I have no other choice than to choose from one of my suggested approaches.
Either reassign the data object within a created method or assign it globally to the window object within the blade template and then reassign it in the vue component.
@all Thanks for your input!


[BUG] Qblox rf module crashes on get_parameter

Expected behavior
In qblox non-rf modules you can ask for a parameter in a module like IF = platform.get_parameter(alias=f"drive_q{QUBIT}_bus", parameter=ql.Parameter.IF)
Actual behavior
For rf modules this doesnt work and you have to give the channel id i.e.
IF = platform.get_parameter(alias=f"drive_q{QUBIT}_bus", parameter=ql.Parameter.IF, channel_id=QUBIT)
Additional information
@paulqili raised this issue
Source code
No response
Tracebacks
No response
System Information
@paulqili raised this issue, ask him for his config
Existing GitHub issues

[X] I have searched existing GitHub issues to make sure the issue does not already exist.

This one was fixed and included in release 0.22.1


fix(metro-resolver-symlinks): don't interfere with browser redirection

Description
Metro enters a different code path than it should when resolveRequest is set and the target package uses the browser field to redirect modules. If detected, we need to unset resolveRequest and retry with Metro's resolver to avoid interference.
Resolves #1548.
Test plan

Add engine.io-client to test app:diff --git a/packages/test-app/package.json b/packages/test-app/package.json
index adf5a430..ea415f64 100644
--- a/packages/test-app/package.json
+++ b/packages/test-app/package.json
@@ -45,6 +45,7 @@
     "@rnx-kit/scripts": "*",
     "@types/react": "^17.0.2",
     "@types/react-native": "^0.64.0",
+    "engine.io-client": "3",
     "metro-react-native-babel-preset": "^0.66.2",
     "react-native-codegen": "^0.0.7",
     "react-native-test-app": "^1.0.6",
diff --git a/packages/test-app/src/App.native.tsx b/packages/test-app/src/App.native.tsx
index 2c89368b..f401b505 100644
--- a/packages/test-app/src/App.native.tsx
+++ b/packages/test-app/src/App.native.tsx
@@ -18,7 +18,10 @@ import {
 } from "react-native/Libraries/NewAppScreen";
 import manifest from "../app.json";

+require("engine.io-client");
+
 const App = (): React.ReactElement => {
   const startAcquireToken = React.useCallback(async () => {
     try {
       const authConfig = manifest["react-native-test-app-msal"];

Install and build test app:yarn
cd packages/test-app
yarn build --dependencies

Start Metro dev server:yarn start

In a different terminal, trigger the bundle:curl -v 'http://localhost:8081/index.bundle?platform=ios'

This does resolve the issue, thank you very much ! 🙏


Add ability to set TTL

Recently I needed the ability to set the Ping TTL for a project and couldn't find a package that would let me to do that so I modified this to let me. This is probably useful to other people aswell so I figure why not make a PR. Could probably do with a little bit of reformating how the TTL is sent from the Pinger to the icmpConn.

Hey, is anything gonna happen with this?

ENOTUITS. Haven't forgotten but just very busy IRL at the moment!

Hi @hamptonmoore, we've recently merged into master. Please could you rebase this?

This needs a rebase now that #177 is merged.


how to install package

Hi i am new to flutter, and have been dealing with a lot of markers.
So i wanted to try your repo.
Is it possible to add the installation process in the README.
How do i import in a dart file ? and how i include it in the pubsec ?
Thank you

  flutter_map:
    git:
      url: "https://github.com/KanarekApp/flutter_map"
      ref: canary
  flutter_map_fast_markers:
    git:
      url: "https://github.com/KanarekApp/flutter_map_fast_markers.git"
      ref: canary


CC:C Bridge animatronic incompatability (related to embeddium)

Mod Name
CC:C Bridge / Embeddium
This issue occurs when only Valkyrien Skies, addons, and the mod I have specified are installed and no other mods

[X] I have tested this issue and it occurs with only Valkyrien Skies, addons, and the mod I have specified

Minecraft Version
1.20.1
Mod Loader
Forge
Issue description
Not exactly sure if this counts as a compatability issue with vs-addition but having vs-addition and embeddium seems to cause rendering issues when a CC:C Bridge animatronic is placed on a ship despite working in older versions of vs-addition.
When placing the CC:C Bridge animatronic on a ship, only its base is rendered instead of its full body. In earlier versions of vs-addition (testing as recent as v0.0.8) the animatronic renders correctly however upon upgrading to v0.0.9 or later the rendering breaks.
At first I thought this rendering issue was simply vs-addition and CC:C bridge but it seems like the issue only happens when embeddium is also present as well as newer versions of vs-addition. I understand it may be hard to pinpoint why the newer version of this mod break something with CC:C bridge or embeddium but I figured I would make a report anyway.
Issue reproduction
Launch the game with CC:C bridge, its dependencies, embeddium, and vs-addition 0.0.9. Create a ship and place the CC:C Bridge animatronic on it. Observe the base is the only part that renders. (Next part is optional to confirm it does work in earlier versions even with embeddium or new version without embeddium) Close the game and downgrade to vs-addition v0.0.8 and repeat the previous steps, observing that the animatronic correctly renders. Then close the game, upgrade vs-addition back to v0.0.9 and uninstall embeddium.
Logs
No response

I added a config item to resolve it, you can get the latest build at Github Actions


MQTT takes too much to reconnect (IDFGH-3596)

Environment

Development Kit:     ESP32-PICO-Kit
Module or chip used:  ESP32-PICO-D4
IDF version v4.0.1
Build System:         idf.py
Compiler version: 8.2.0
Operating System:  Linux
Using an IDE: Visual studio code.
Power Supply: external

Problem Description
The MQTT client connected to a local broker is taking to much to reconnects after wifi reconnection.
// Detailed problem description goes here.
Steps to reproduce
make sure you have 2 access points with the same SSID and password, and the wifi clients reconnect to wifi after losing connection

start wifi connect to a station
connect MQTT client to the broker
get the BSSID where esp32 is connected and disconnect this router.
4 esp32 will connect to the other router - (after been connected and retrieved IP, connect the MQTT client)
it will take 1 minutes or more to connect, meanwhile, it will receive disconnects events

Debug Logs
DEBUG: MQTT_TAG: Received disconnect event
ERROR  TRANS_TCP: tcp_poll_write select error 113, errno = Software caused connection abort, fd = 57

Other items if possible

[ ] sdkconfig file (attach the sdkconfig file from your project folder)
[ ] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[ ] coredump (This provides stacks of tasks.)

Thanks for reporting, we will look into.

Hi @jonaphael
I wasn't able to reproduce your issue. I tested on v4.0.1 using our MQTT TCP example and it seems to migrate to the new AP without any issues. After connecting and getting the new IP it takes about 5-10 seconds before the MQTT client reconnects with the broker, which is as expected since network_timeout_ms defaults to 10 000.
Did you try to adjust  network_timeout_ms or are you running with the default value?

@jonaphael Any updates? Did the issue resolve itself?

@jonaphael Thanks for reporting, would you please help share if any updates for this issue? Thanks.

Closing issue due to no feedback, feel free to reopen with new information if this is still an ongoing issue.


Escape URI paths

Resolves https://github.com/phadej/github/issues/398
ping @domenkozar, does this works for your use case?

I haven't tested it, but I'm using the same function to pass the path, so it should work.


Fixing timezone string, browser location/history, person map height

adding fallbacks from when LDAP server response (location query) is missing latitude and longitude (for GeoNames API lookup), and timeZone (which might not be taking DST into account).
fixing browser location/history so back/forward buttons work
adjusting height of Person map so it doesn't create an extra scrollbar for most users on the main content

great work. glad to see the naming strategy you add in your PRs.
Just one small suggestion, almost good to go.


[Microsoft][ODBC Driver 17 for SQL Server]Client unable to establish connection (0) (SQLDriverConnect)'

Environment

Python: 3.10.1
pyodbc: 4.0.32
OS: macOS Big Sur 11.6.1
DB: SQL Server
driver: ODBC Driver 17 for SQL Server

Issue
Hi, when I try to connect to SQL Server, this error is thrown:
'08001', '[08001] [Microsoft][ODBC Driver 17 for SQL Server]Client unable to establish connection (0) (SQLDriverConnect)', i have readed this issue https://github.com/mkleehammer/pyodbc/issues/967 and move to openssl the location but still can't connect to server.
ls -l /usr/local/opt/openssl/lib :

ls -l<EMAIL_ADDRESS>:

I had the same problem and nothing in https://github.com/mkleehammer/pyodbc/issues/967 worked.
I ended with setting this environment variable when launching my script:
<EMAIL_ADDRESS>$DYLD_LIBRARY_PATH
and now it works.

I had the same problem and nothing in #967 worked. I ended with setting this environment variable when launching my script:
<EMAIL_ADDRESS>$DYLD_LIBRARY_PATH
and now it works.

how can i do it?

@altintashalitmert
Either set it using os.environ inside Python before you import pyodbc or set it for example in your IDE before running your code. If you are using PyCharm: https://stackoverflow.com/questions/42708389/how-to-set-environment-variables-in-pycharm

@v-chojas and @david-engel

via isql
which openssl

openssl version

Error:

That one does not appear to be OpenSSL-related. Timeout error is usually because you are not connecting to the right server, or it's not listening, or a firewall is in the way. In any case, it is a networking issue.

@v-chojas this works with no errors on a windows machine :(
same code, same service accts, same hostname and db

Are you able to at least ping the server from the machine you're getting the connection errors?
Does the server have a firewall that is not allowing the connection?

I can ping the server and get the following output:

@v-chojas and David-Engel
I get the correct error now (All I did is remove the '<>' from my connection string.

Thank you!

Now you can go to https://github.com/microsoft/homebrew-mssql-release/issues/59 to apply the workaround.

ln -s<EMAIL_ADDRESS>/usr/local/opt/openssl
@v-chojas this you mean? Sorry!

@v-chojas - I tried to do the above (running the 'ln -s<EMAIL_ADDRESS>/usr/local/opt/openssl'
and did 'rm /usr/local/opt/openssl'

I still get the same error:

Okay @v-chojas . sorry about the confusion, I still get the same library not loaded error :(

@v-chojas - Success (please find the screenshot below):

Solution for above:

but I get the same error using my python script:

this now!

@v-chojas - I am all set, just restarted my machine and i can connect now.
Solution: ln -s<EMAIL_ADDRESS>/usr/local/opt/openssl

@dragnblow does this if you are on linux machine we have to confirm what SSL version it's using to connect to SQL 2019 server?


System.BadImageFormatException: An attempt was made to load a program with an incorrect format.  (0x8007000B)

Copied from https://github.com/dotnet/vscode-csharp/issues/6842#issuecomment-1951014724 by @dkflRus
Using dotnet configured on PATH
Dotnet path: /usr/share/dotnet/dotnet
Activating C# standalone...
Starting server at /home/dkfl/.vscode-oss/extensions/ms-dotnettools.csharp-2.17.7/.roslyn/Microsoft.CodeAnalysis.LanguageServer.dll
Server arguments /home/dkfl/.vscode-oss/extensions/ms-dotnettools.csharp-2.17.7/.roslyn/Microsoft.CodeAnalysis.LanguageServer.dll --logLevel Trace --telemetryLevel off --extensionLogDirectory /home/dkfl/.config/Code - OSS/logs/20240218T102340/window1/exthost/ms-dotnettools.csharp
waiting for named pipe information from server...
[stderr] Unhandled exception. [stderr] System.BadImageFormatException: An attempt was made to load a program with an incorrect format.
 (0x8007000B)
[Error - 10:25:01 AM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Error: Timeout. Named pipe information not received from server.
	at q.<anonymous> (/home/dkfl/.vscode-oss/extensions/ms-dotnettools.csharp-2.17.7/dist/extension.js:2:1291052)
	at Generator.next (<anonymous>)
	at s (/home/dkfl/.vscode-oss/extensions/ms-dotnettools.csharp-2.17.7/dist/extension.js:2:1280530)

@dkflRus this is often a sign of a corrupt dotnet installation or extension installation.  Can you paste the output of dotnet --info and which dotnet?
I would also try

Completely delete and re-install the C# extension
a.  Uninstall the extension in vscode
b.  Open the extensions folder (there should be an 'open extensions folder' command in vscode)
c.  With the extension uninstalled and vscode closed, delete all the ms-dotnettools.csharp folders in the extension folder
d.  Re-open vscode and re-install.
Completely delete and re-install dotnet.  The instructions vary depending on platform and how it was installed.


Use more precise dependency declaration

Per a discussion and its subsequent off-line discussion with @bluca , we realize that package maintainers on Linux might not want to aggressively upgrade a dependency to its latest version. So, in this PR, we make it clear that the portalocker 1.6.0 dependency requirement is only needed on Windows.

Thank you!


Multiline lists are formatted wrong

Source:
main() {
  [
    "this is a very long string that forces the list to wrap",
    "this is a very long string that forces the list to wrap"
  ];
}
Output:
main() {
  ["this is a very long string that forces the list to wrap",
      "this is a very long string that forces the list to wrap"];
}
Expected: It should stay formatted like the source.

This issue has been moved to dart-lang/dart_style#307.


Pushing Favourite Product to firebase

addFavouriteProduct(data: Product): void {
	let a: Product[];

-->>>		this.favouriteProducts.push(data);
	a = JSON.parse(localStorage.getItem('avf_item')) || [];
	a.push(data);
	this.toastrService.wait('Adding Product', 'Adding Product as Favourite');
	setTimeout(() => {
		localStorage.setItem('avf_item', JSON.stringify(a));
		this.calculateLocalFavProdCounts();
	}, 1500);
}

I was trying to push the favourite items of the users to the database. But I'm getting an error at the line I represented with an '-->'.. Could you please help me with that.

@AKhileshPothuri Can you post the exact error screenshot that showed in console.


Add filters for presets

This commit adds filters to the presets index page (similar to the devices index page). Current filters include

preset name
precondition string
value of Set configurations
name of Provision configurations

This PR requires https://github.com/zaidka/genieacs/pull/277.

Hmm I'm not sure about this. Searching by precondition string can't be guaranteed to work because the same filter can be written in different ordering.
Searching by the value in a set configuration doesn't make much sense without specifying the parameter being set.

Searching by precondition was meant to be used with searchstrings like tag names or serial numbers.
Search by value in a set configuration was meant to find a preset where a specific ip address etc is set. Thus the current implementation is sufficient for our usecase.
How would you extend the implementation to be of more general use?


Timer.Context now implements AutoCloseable

Small improvement that allows Timer.Context to be used in try-with-resources

:+1: from a random watcher here -- we do this in our wrapper for Timer.Context and it's really nice.

I like it. Can you also remove Closeable, or does that cause problems?

AutoCloseable doesn't implement Closeable (no idea why though). Removing it would be a breaking change.
https://docs.oracle.com/javase/7/docs/api/java/lang/AutoCloseable.html

Yea, we can live with that. We're working on v4, so breaking changes are welcome.

Here you go.

Closeable extends AutoCloseable, not the other way around, because AutoCloseable allows throwing Exception.
The Closeable imports can be removed now, right?

I reverted the change to Timer in metrics-adapter, since that module is around for backwards compatibility with 3.x, and we shouldn't do anything that could break it.  Also I removed the Closeable import from Timer in metrics-core.


Android: Gradle 4, Android plugin 3, SDK 27

Android has released new major versions in all tools, so we should advance the sample too.
~But first the PR demonstrates immediately the current failed use of GraphHopper on Android, as described in #1157, if not commenting the mentioned code.~
EDIT: that should be fixed via #1207.

BTW: Does it work with the mmap fix now?

Yes I tested it, all seem working ok again!

Cool, thanks!

One last comment: would you mind signing the commit? (if not possible yet, no problem & no hurry, we can do this later then)

Signed and force pushed, should be ok now?

Cool, thanks!


fix: only join to pg_constraint on primary key

Hi achiku,
Thanks for the great library, I really like that you can give a custom typemap file.
Whilst testing it out I found some fields where being duplicated in the output structs, and it turned out it was because they were being used in multiple unique indexes and so being brought in multiple times, this PR drops the join on unique indexes and limits it to primary key. I don't think this affects the rest of the query as ct.contype only seems to be used if it is a primary key column, but I might have misunderstood the query.
Thanks again.

Thank you for the PR! I'll look into it this week!

@pci sorry for taking too long to look at it, but merged!


0.18.0 change the db format cause incompatibility

The .db file generated by 0.17.1 can not read from 0.18.0 vice versa.
This indicate the 0.18.0 is a release with breaking change which is not mentioned in the release note.

As stated under https://github.com/prometheus/client_python#multiprocess-mode-eg-gunicorn "This directory must be wiped between process/Gunicorn runs", we have never promised nor should it be assumed that the .db files are compatible across versions.
I would be ok calling this out in release notes as a change, but I wonder if we could also be more clear that the .db files should never be used across versions?

I did not notice this warning before.
Thank you for the information.


added extra debug info

Signed-off-by: Templeton, Randall<EMAIL_ADDRESS>

PR for debugging purpose.


docs: fix typo in command

Whelp, weird that checks are failing like this. Time to do some maintainence on this repo...

Since it fails at installing pnpm, I suggest enabling nodejs' corepack

Oh woops, I actually updated this section in #66... I migrated to a Nuxt app for docs and reviewed all the pages. I updated my suggestion from vite-plugin-web-extension to WXT: https://webext-core.aklinker1.io/get-started/introduction#bundler-setup

it's ok


Add validate_certs

As you suggest for the ansible-role-sonar, by default it is set to yes.
Source: https://docs.ansible.com/ansible/get_url_module.html

D'oh! It looks like now there's a merge conflict that snuck in; can you rebase on master? This looks ready to merge otherwise!

I beg your pardon. A missplay with PR #37 ... I restarted from a fresh rebased branch.

Any news ? I thought we were close to the merge.

@olo-dw - Sorry about the delays, can you rebase this entire PR? It looks like instead of a rebase, you did a merge from master, and now it's a bit tricky trying to review the changes (since the order is changed).

Rebased.


Transform constants to variables with Final type hint

According to PEP-0008, constants are usually declared at a module level.
In ast.py, (lines 366-368) we have 2 constants declared in if
if __name__ == '__main__':
    JAVA = sys.argv[1]
    METRICS = sys.argv[2]

The idea behind it is clear: JAVA and METRICS must not be changed.
What we can do is using Final type hint. It states that variable or attribute should not be reassigned, redefined, or overridden.
Suggestion:
if __name__ == '__main__':
    java: Final[str] = sys.argv[1]
    metrics: Final[str] = sys.argv[2]

@yegor256 can you approve this as an issue, please?

@yegor256 can you check it, please?

@ilnarkhasanov sounds like a reasonable suggestion, but let's first find out why Pylint doesn't complain about this:

According to PEP-0008, constants are usually declared at a module level.

Maybe we should use some other linter on top of pylint?

Looks like pep doesn't force us to declare constants in top level. but you can extend your pylint with your own plugins using load-plugins= option (https://pylint.pycqa.org/en/latest/development_guide/how_tos/plugins.html)
I would suggest putting this in a directory and adding new plugins to it in the future.
I could try to write this :)
@yegor256 what do you thunk?

@howcanunot please, try!)


Missing documentation

Hey,
great work and very interesting project. I try to develop a project for the same purpose as well. The only difference is that I like to read in german nutritional facts.
So I came across your project and tried to install and used it. The text results were kind of strange to me. I used it like this:
text = OcrTable(image, language='deu')
Should I also use:
                 spell_corrector: bool = False,                  show_performace: bool = False
Or should that be True?
I used text.text to get the results. So overall I think it needs a bit more of documentation.
Thanks in advance

Very nice comment @mbrissier !
i think that with you want use another language, you will need to install that language to your environment as well.
You're correct inserting the "deu" on language, but look here:
https://github.com/Lucs1590/Nkocr/blob/b5dc800c0d869df2c5bd8629c76bfd272f61bdcb/.github/workflows/python-app.yml#L30 , for example.
When I used the Github Actions (DevOps) I need to install  tesseract-ocr-por  too.
Try running:
sudo apt install tesseract-ocr libtesseract-dev tesseract-ocr-deu
And then, run the scripts again!
I'll try to improve the documentation too!!
Thanks for this issue!!

I think this problem can be closed by looking at this commit https://github.com/Lucs1590/Nkocr/commit/6d7c60bb894677f7aeec7337c97e3efc620465d6 .


Alarm via bot

Set alarm (timed note / timer) via bot, and the bot should send the user the note at specified time.

This seems to be an interesting idea so I'll leave it open. But I doubt the effect of using Telegram as an alarm app.

Not going to fix.


protonmail-bridge: 2.3.0 -> 3.0.21

I tried v2.3.0 but this does not work and seems to be outdated.
Building v3.0.21 does not work (fatal error: src/rfc5322/rfc5322_parser_capi.h: No such file or directory).
Unfortunately I'm not a Go developer and I have no idea how to fix that.
Can someone help?
Description of changes

Changelog

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

I don't understand the issue.
Similar PR was merged 10 hours ago in #221297 where the build works, etc. and it is now being propagated to unstable branch. You can either switch to unstable for this package, or if you think it's worth backporting, you can create backport PR to release-21-11 branch

I don't understand the issue. Similar PR was merged 10 hours ago in #221297 where the build works

ah nice... I'm sorry, somehow the search result for "protonmail-bridge" did not show me this :thinking: so I'm sorry for bothering you :see_no_evil:  Just ignore my PR!


Added unique binary search trees II.

Coverage remained the same at 100.0% when pulling ef0dac4f402498f3c69e7f2a661ad107eb1f7695 on unique-binary-search-trees-ii into 188790d3edde6c0611b4de2a2283bfc06cbd3fe2 on master.


(DDC Acceprtance test phase 1)

Request to review the following changes
fix for rubocop warning
enabling travis for rubocop and puppet lint
Forcing pupet 5.3.3 version
some of the acceptance tests for  DDC

LGTM


Unable to get internal redirect policy working (docs / examples lacking)

If you are reporting any crash or any potential security issue, do not
open an issue in this repo. Please report the issue via emailing
<EMAIL_ADDRESS>where the issue will be triaged appropriately.
Title: internal redirect of 307 bubbles up to browser rather than being handled
Description:
We have an Envoy router sitting in front of Consul's UI and two GraphQL-based microservices. (One of them with websockets.) The goal is to make all three services available at different paths in Envoy, e.g. /consul routes to http://localhost:8500/consul. The problem we are hitting is all three issue redirects, e.g. /graphql gets a 307 redirect to /graphql/ when you hit it. The desired behavior is that Envoy will consume this redirect, go to /graphql/ on the microservice, and then return that response to the client. The actual behavior observed no matter how I configure internal redirect policies, actions, etc. is that the 307 goes back through Envoy and then the browser tries to go to http://localhost:9900/graphql/, which does not exist.
E.g. I'd expect this would match the 307 and force it to be processed:
              internal_redirect_policy: {
                max_internal_redirects: 5,
                redirect_response_codes: [307]
              },
              internal_redirect_action: HANDLE_INTERNAL_REDIRECT

but it's the same whether that policy is set or not. Also note I struggled to find good examples of this anywhere and I saw a lot of questions out there about how to set this up for various scenarios, so some more documentation may be useful.
[optional Relevant Links:]
See attached full envoy.yaml
envoy.yaml.txt

skim reading your config its not immediately obvious to me what is/not working - im not so familiar with the internal redirect config
if you resolve your problem i would be happy to review any updates to the docs or examples that might make this clearer

cc @alyssawilk who knows about internal redirects.

did you check stats?  If you're hitting the route with internal redirects configured, failures to redirect should get tagged with one of the stats here: https://github.com/envoyproxy/envoy/blob/main/source/common/router/context_impl.h#L18
If that doesn't turn up anything if you could dump trace logs from a single request it may help show what's going on

Thanks Alyssa, I can see the stats now. We found a workaround for the GraphQL issue (a regex rewrite to avoid the trailing slash, and switching away from the embedded GraphiQL tool) and so it's just Consul that's a problem now. Give me a couple hours to play with this and if I figure it out I will add a comment and close. Thanks for the tips!

OK, so I have a bit more information. For one of those cases returning 307, I do see this stat updated:
cluster.serenity_portfolio_summary_ws.upstream_internal_redirect_failed_total: 2
and I also see this, which suggests why those two requests failed:
http.ingress_http.passthrough_internal_redirect_no_route: 2
@alyssawilk, what I would like is for a 307 sent back to Envoy to be followed and for it to fetch the new location. Is this possible in the configuration? I would have thought what I have above would do this.

Trace logs you requested for this scenario:
[2022-01-26 00:56:10.851][23989][trace][connection] [external/envoy/source/common/network/raw_buffer_socket.cc:24] [C1] read returns: 163 [2022-01-26 00:56:10.851][23989][trace][connection] [external/envoy/source/common/network/raw_buffer_socket.cc:38] [C1] read error: Resource temporarily unavailable [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:555] [C1] parsing 163 bytes [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:834] [C1] message begin [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:475] [C1] completed header: key=date value=Wed, 26 Jan 2022 00:56:10 GMT [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:475] [C1] completed header: key=server value=uvicorn [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:475] [C1] completed header: key=location value=http://localhost:9900/metrics [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:685] [C1] onHeadersCompleteBase [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:475] [C1] completed header: key=transfer-encoding value=chunked [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:1263] [C1] status_code 307 [2022-01-26 00:56:10.851][23989][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:1273] [C1] Client: onHeadersComplete size=4 [2022-01-26 00:56:10.851][23989][debug][router] [external/envoy/source/common/router/router.cc:1244] [C0][S10617589657952339435] upstream headers complete: end_stream=false [2022-01-26 00:56:10.851][23989][debug][router] [external/envoy/source/common/router/router.cc:1477] [C0][S10617589657952339435] attempting internal redirect [2022-01-26 00:56:10.851][23989][debug][router] [external/envoy/source/common/router/router.cc:1508] [C0][S10617589657952339435] Internal redirect failed

The trace logs seem to match the stats: you can see that it gets the 307 at the top and then in the last line it bails out on the redirect. So I don't think Envoy has a bug; this is a configuration problem our end causing it to not process the redirect.

so the internal redirect is supposed to save the round trip between Envoy and Client of  doing the redirect.
If you redirect from /foo to http://loopback.com/bar you still need Envoy to have a route configured for loopback.com/bar, so it knows where to send the request.  The error you send indicates you don't have a route configured for the redirect URL.

OK. I had a hunch that was the case. How is the route expressed? http://localhost:9090/foo/metrics -> http://localhost:9001/metrics/ redirects to http://localhost:9001/metrics. The latter is a cluster address. Do I need to put in a match that references localhost:9001? Or is it a match that uses a header? This is why I raised this as a documentation issue: I read the docs and examples and could not find guidance on how to configure this so it works.  Thank you very much for your patience and help.

Also, small thing: it would be nice if the trace actually logged the path that had no route. :-)

OK, I figured out a way to fix our config. It's a bit ugly, but it works! I had to add matching rules that keyed off the x-envoy-original-path header. If it matches the original inbound request path, it kicks the 307 back where it came from.
Thanks for the help, @alyssawilk, the tips helped me find this.
              - match:
                  prefix: "/"
                  headers:
                  - name: "x-envoy-original-path"
                    prefix_match: "/serenity/platform/installation_info"
                route: {
                  cluster: serenity_portfolio_summary_ws
                }
              - match:
                  prefix: "/"
                  headers:
                  - name: "x-envoy-original-path"
                    prefix_match: "/serenity/portfolio/portfolio_summary"
                route: {
                  cluster: serenity_portfolio_summary_ws
                }

Glad you're set.  You should just be able to match on the new path, but really whatever works :-P
Fair point on the logs - I'll throw together a quick PR to improve that, and close this out if that works for you

Thanks @alyssawilk, closing this issue now. Matching on new path would work too, but I had to do it this way because all our microservices sitting behind Envoy expose the same set of paths: /metrics, /graphql, etc.. So I need to disambiguate to pick the right cluster, and the x-envoy-original-path header does that nicely. :-)


[berkeley] lookup verifier index

Include changes from develop into berkeley.
See here: https://github.com/MinaProtocol/mina/pull/13899.

Blocked by https://app.zenhub.com/workspaces/core-crypto-621fcb82b7d49700156fd060/issues/gh/minaprotocol/mina/13911


Read Opacity from Texture in 1D LUT for data shader

This: https://github.com/FNNDSC/ami/blob/4e256ca023717acecc95e7dfaf95f24566cd435a/src/shaders/shaders.data.fragment.js#L96-L97 should be updated to be consistency with vr: https://github.com/FNNDSC/ami/blob/4e256ca023717acecc95e7dfaf95f24566cd435a/src/shaders/shaders.vr.fragment.js#L99-L102
repost from: https://github.com/FNNDSC/ami/commit/24a0301afeda042da7be2771320baba0954e3e77#commitcomment-25666498

Thanks for opening all the issues - much easier to keep track of things this way


We need documentation

We must generate documentation for our users.

I preferred to improve some docstrings (4ba46a8b19cd633eb4b78a31d088aba46df015bf) before creating the documentation with pydoc (5088897abd3d1361e9f9dca9095bed190a629bbc).
Closed in 5088897abd3d1361e9f9dca9095bed190a629bbc.


Poll: app.poll.answer.undefined

users: u1 and u2 (app)

s1 starts a poll with typed answer
u2 responds normally
s1 starts a poll with anon, multiple choice and type ABCD
see error on app


PLAT-139057: Update CheckboxItem

Enact-DCO-1.0-Signed-off-by: Stanca Pop<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[x] A CHANGELOG entry is included
[x] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[x] Documentation was verified or is not changed
[x] UI test was passed or is not needed
[x] Screenshot test was verified or is not needed
[ ] This is an API breaking change

Resolution
Updated CheckboxItem according to the new design requirements for Silicon skin.
Links
PLAT-139057
Comments

The horizontal position of the Checkbox is not the same as in the GUI