Constraints are not correctly propagated across interdependencies

I have 5 .in requirement files (A, B, C, D and E) and one .in constraints file (X) with the following interdependencies:

B includes A
B is constrained by X
C includes A
D includes A
E includes B, C and D

I'm using:

the includes term above to refer to -r file.in
and the is constrained by expression to refer to -c file.in

I've tried to resolve the above with running pip-compile-multi with these flags:
pip-compile-multi --autoresolve --backtracking --strip-extras --live
pip-compile-multi --autoresolve --backtracking                --live
pip-compile-multi               --backtracking --strip-extras --live
pip-compile-multi               --backtracking                --live

In every case, the compilation outputs the .txt files for X, A, B, C and D, and when it reaches E, it fails:
Package foo was resolved to different versions in different environments: 1.35.7 and 1.35.55
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/cli_v1.py", line 26, in cli
    recompile()
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/actions.py", line 31, in recompile
    compile_topologically(env_confs, deduplicator)
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/actions.py", line 38, in compile_topologically
    if env.maybe_create_lockfile():
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/environment.py", line 51, in maybe_create_lockfile
    self.create_lockfile()
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/environment.py", line 72, in create_lockfile
    self.fix_lockfile()
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/environment.py", line 134, in fix_lockfile
    sections = [
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/environment.py", line 135, in <listcomp>
    self.fix_pin(section)
  File "/usr/local/lib/python3.10/site-packages/pipcompilemulti/environment.py", line 211, in fix_pin
    raise RuntimeError(
RuntimeError: Please add constraints for the package version listed above

Note that:

B is constrained by X, and E includes B, therefore E should inherit the constraints suffered by B

B inherits A, and B is constrained by X, thus A must also be constrained by X

These are the produced files and the versions of foo:

.txt file
foo version
observations

X
1.35.7
OK, enforces foo==1.35.7

A
1.35.55
🐛, expected 1.35.7

B
1.35.7
OK

C
-r A
🐛, expected 1.35.7

D
-r A
🐛, expected 1.35.7

E
💣
(file never produced)

Expectation
All dependencies are free to resolve to the latest compatible version, while keeping the file B constrained as per X. This implies that if A, C, D and E share any dependency with B, those shared dependencies must also be constrained as per X.

Does --autoresolve work? Or should mention creation of a temporary file with all dependencies.
What is the first line of the output?

@peterdemin sorry for the delay in the response.

Does --autoresolve work?

The --autoresolve flag did not work.

Or should mention creation of a temporary file with all dependencies.

I'm not sure what do you mean by the above.

What is the first line of the output?

Locking requirements/common.in to requirements/common.txt. References: []

where requirements/common.txt is A according to the nomenclature used in my first message.

@peterdemin I've set up this repo to reproduce the issue. Please let me know if I can clarify anything further.

In your diagram, E is the single requirements file that references (directly or indirectly) all other files. This is needed for auto-resolve to work. Please see this feature's documentation here:
https://pip-compile-multi.readthedocs.io/en/latest/features.html#autoresolve-cross-file-conflicts

I see in the repro repo, that the constraints file has an extension .in which makes it an input file for locking (which it is not).
I think if you rename mwaa-2_10_1-constraints.in to mwaa-2_10_1-constraints.txt (and the -c reference) autoresolve will see the single head of the hierarchy and resolve cross-dependencies.

Nice! You are right @peterdemin - if the constraints file has the .txt extension and the --autoresolve flag is used, it all works like charm. Thanks!


refactor: simplify scheduler code and handle key cornercase

Description:
This PR fixes potential issue with scheduler when using tasks without taskId. Such issue could occur because functions can't be used as prop keys in JS objects. Currently it seems that our codebase wasn't using scheduler in this way so no bugs ensued.

How Has This Been Tested?
Manually and unit tests

Types of changes:
Bugfix/refactor

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 22.796% when pulling 7757a996baca7561465a505bd683cee5fbd24126 on korhaliv:fix/scheduler-fn into ba51a91c48346896172159af2c1ce37360ebe314 on LN-Zap:next.


Using dirEntries and chdir() can have unwanted results

andrej.mitrovich reported this on 2011-06-09T16:13:56Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=6138
CC List

andrei
francesco.galla3
greensunny12
hsteoh
razvan.nitu1305
timothee.cour2

Description
I can't officially classify this as a bug. Jonathan said it might be a good idea to put it here just so it doesn't get lost, and then we can later decide if it's a bug or if we should just be careful what we do.

Here's some buggy code which might throw an exception (if you're lucky!):
   foreach (string entry; dirEntries(curdir, SpanMode.shallow))
   {
       if (entry.isdir)
       {
           foreach (string subentry; dirEntries(entry, SpanMode.shallow))
           {
               if (subentry.isdir)
               {
                   chdir(rel2abs(subentry));
               }
           }
       }
   }

This might throw something like this:
std.file.FileException@std\file.d(1124): .\foo\bar.d: The system cannot find the path specified.

The problem is chdir will modify curdir, and dirEntries doesn't store 'curdir' internally as an absolute address, so calling chdir() screws up the behavior of dirEntries.

A workaround is to call rel2abs as soon as possible when using 'curdir' (the first line changed here):

   foreach (string entry; dirEntries(rel2abs(curdir), SpanMode.shallow))
   {
       if (entry.isdir)
       {
           foreach (string subentry; dirEntries(entry, SpanMode.shallow))
           {
               if (subentry.isdir)
               {
                   chdir(rel2abs(subentry));
               }
           }
       }
   }

I've had a use for code like this where I was invoking a script file which was always relative to some subdirectory.

timothee.cour2 commented on 2016-03-23T05:33:36Z
sounds like a bug (and one that could cause harm);
why not have dirEntries call rel2abs ? the cost of this should be dwarfed by system calls involved in dirEntries

timothee.cour2 commented on 2018-02-10T03:25:20Z
just hit that bug again, but differently:
```
chdir(foo);
auto files=dirEntries(dirSrc, "*.d", SpanMode.depth).filter!(a=>a.isFile).map!(a=>dir.buildPath(a.name)).array;
```
crashed similarly

dlang-bugzilla commented on 2017-07-07T14:52:56Z
(In reply to RazvanN from comment #3)
> After analyzing the code I see that dirEntries does not call absolutePath on
> the path, which in my opinion is a bug. Even though I cannot reproduce the
> bug, I think a call to absolutePath should definitely be added

That might break things.

~ $ mkdir -p a/b/c
~ $ cd a/b/c
~/a/b/c $ touch a
~/a/b/c $ ls
a
~/a/b/c $ ls ~/a/b/c
a
~/a/b/c $ chmod 000 ~/a/b
~/a/b/c $ ls
a
~/a/b/c $ ls ~/a/b/c
ls: cannot access '/home/vladimir/a/b/c': Permission denied
~/a/b/c $

I think the correct solution would be to use fdopendir and openat (on POSIX).

razvan.nitu1305 commented on 2017-07-07T13:00:50Z
After analyzing the code I see that dirEntries does not call absolutePath on the path, which in my opinion is a bug. Even though I cannot reproduce the bug, I think a call to absolutePath should definitely be added

greensunny12 commented on 2018-02-04T19:43:16Z
FYI: Francesco Galla submitted a PR - https://github.com/dlang/phobos/pull/6125


Allow lookup operator [] with Java arrays

Following on from #6061 we should allow you to write intArray[i] for all the array types in java.lang.

This is now supported in the code generator.

Not sure why you assigned this to me @gavinking – the codegen part should just work – what's missing is the tc supporting this.

Because I didn't see your comment :)

Done!


layers: Make unique object IDs unique cross-device and cross-instance

Storing object ID in the layer_data map results in devices and instances with duplicate object IDs.  This was found in CTS testing.
Change-Id: Ic45e21bd3137dc0474c59f0f4cf9331f070dac20

+1

cherry-picked and pushed to master


Documentation: size limit of clipboard functions

https://www.glfw.org/docs/3.4/input_guide.html#clipboard
The doc lists no size limit, creating an expectation that usual sizes on modern systems can be handled. Yet they work on a thread local MemoryStack with a default size of 64k. Not everyone is able to deep-dive into the sources to find that "org.lwjgl.system.stackSize" limits clipboard transfers.
Suggestion: Update the Input guide to at least mention a size limitation exists, and the individual function's docs to include the default limit, the system setting controlling it, and which error you get when the limit is surpassed.

This seems to be part of the Java portion of LWGL and not part of GLFW itself. I would recommend trying one of their support resources.

No worries and thank you for the link!


Can't parse the method whose parameters contains &

static void badSource(int &data)
{
    int a=data;
}

void bad()
{
    int data=1;

    badSource(data);

}

I found that the information  of badSource in the methods in cpg is empty，which caused the problem about   callee https://github.com/joernio/joern/issues/1297#issuecomment-1109250223
joern> cpg.method.l
res21: List[Method] = List(
  ……
  Method(
    id -> 34L,
    astParentFullName -> "<global>",
    astParentType -> "NAMESPACE_BLOCK",
    code -> "<empty>",
    columnNumber -> None,
    columnNumberEnd -> None,
    filename -> "<empty>",
    fullName -> "badSource",
    hash -> None,
    isExternal -> true,
    lineNumber -> None,
    lineNumberEnd -> None,
    name -> "badSource",
    order -> 0,
    signature -> ""
  )
)

Did you tried parsing it as C++ code?

Oh,sorry.It's my fault.I didn't put it in a .cpp file.Thanks


[BUG] Tuplex fails to parse/run functions defined within shell

After defining the following function in a python3.6 shell:
def count(L):
    d = {}
    for x in L:
        if x not in d.keys():
            d[x] = 0
        d[x] += 1
    return d

running:
c = Context()
c.parallelize([[1, 1, 1], [1, 2, 1, 2]]).map(count).collect()

I get the following error:
Traceback (most recent call last):
  File "/usr/lib/python3.6/code.py", line 91, in runcode
    exec(code, self.locals)
  File "console-9", line 1, in <module>
  File "/home/rgoyal6/tuplex/tuplex/build/dist/python/tuplex/dataset.py", line 64, in map
    code = get_udf_source(ftor)
  File "/home/rgoyal6/tuplex/tuplex/build/dist/python/tuplex/utils/reflection.py", line 207, in get_source
    return get_function_code(f)
  File "/home/rgoyal6/tuplex/tuplex/build/dist/python/tuplex/utils/reflection.py", line 156, in get_function_code
    return extract_function_code(function_name, dill.source.getsource(f))
  File "/home/rgoyal6/.local/lib/python3.6/site-packages/dill-0.3.4-py3.6.egg/dill/source.py", line 354, in getsource
    lines, lnum = getsourcelines(object, enclosing=enclosing)
  File "/home/rgoyal6/.local/lib/python3.6/site-packages/dill-0.3.4-py3.6.egg/dill/source.py", line 325, in getsourcelines
    code, n = getblocks(object, lstrip=lstrip, enclosing=enclosing, locate=True)
  File "/home/rgoyal6/.local/lib/python3.6/site-packages/dill-0.3.4-py3.6.egg/dill/source.py", line 251, in getblocks
    lines, lnum = findsource(object)
  File "/home/rgoyal6/.local/lib/python3.6/site-packages/dill-0.3.4-py3.6.egg/dill/source.py", line 144, in findsource
    raise IOError('source code not available')
OSError: source code not available

thanks for filing this bug! I suspect the interactive shell does not support multi-line functions yet.


Dev

@all-contributors please add @mist8kengas for contributors
@all-contributors please add @mist8kengas for translation

@all-contributors please add @mist8kengas for code

@all-contributors add @commonly-ts for bug

@all-contributors please add @eduardozgz for translation


os/mac/xcode: add additional CLT download link.

Sadly Apple don't seem to be providing these reliably.
Fixes #6796

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

This seems to only modify the prompt information, but it does not actually solve the problem. There will still be a new version, but it is actually the latest stable version

@bpzhang This solves the problem where people do not know how to get to the latest version. I see no evidence there's a bug beyond that.


Add trigger phrases to workflow names

1 piece of feedback I've received is that its hard to know which trigger phrase to use to retrigger a job. This will make this easier. We should do this going forward as well

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @andreydevyatkin


Rename repo

I would rename repo to status or uptime since kodadot- prefix is verbose

since I moved the repo, I don't have access to the repository setting. even I'm unable to edit the description in the right side panel. maybe can anyone invite me to the repo setting?

I can rename it, no worries.
Which name do you prefer more?

I can rename it, no worries. Which name do you prefer more?

uptime seems ok 👍


operation-4xx-response rule evaluates webhooks

Describe the bug
The operation-4xx-response rule evaluates webhooks (and x-webhooks), and that doesn't make sense because of the nature of webhooks (asking clients to implement an API to receive notifications is a big enough ask, let alone to ask them to implement error handling and some error response codes and schemas). Those should be skipped.
To Reproduce
Pull rebilly/api-definitions. Run openapi lint.
Expected behavior
Don't evaluate webhooks (or x-webhooks).
Logs
n/a
OpenAPI definition
3.0
openapi-cli Version(s)
1.0.0-beta.80
Node.js Version(s)
16.10
Additional context
Trying to make the rules make the most sense out of the box.

@adamaltman we have option validateWebhooks with which we can disable the evaluation of webhooks

Thanks @IgorKarpiuk! Your comment prompted me to take a look at this and I think what we have makes sense. The default value for validateWebhooks is false, so we normally don't run this rule on those endpoints, but it does correctly check them if I set it to true. I don't think there's more to do on this so I'll close the issue.


Demotion not work ?

Good morning,
Obviously demotion to standard user is not automatic at the end of the allotted time.
Is this normal or a bug in the app?

Please see the following entries in the FAQ:
By default, is there a time limit on the admin rights granted by Privileges?

No. Admin rights are granted until some process (like running Privileges again) takes them away.

https://github.com/SAP/macOS-enterprise-privileges/wiki/Frequently-Asked-Questions#by-default-is-there-a-time-limit-on-the-admin-rights-granted-by-privileges
Can I set Privileges to give me administrator rights for a defined amount of time?

Yes. You can use the Toggle Privileges option on the dock icon to get admin rights for a set amount of time (the default amount is 20 minutes.)

https://github.com/SAP/macOS-enterprise-privileges/wiki/Frequently-Asked-Questions#can-i-set-privileges-to-give-me-administrator-rights-for-a-defined-amount-of-time


Display the JOB/INDEX in the ssh terminal prompt.

Currently, the ssh terminal prompt is relatively unhelpful:
bosh_ogre9wa5e@455a72d5-b2ab-42cb-8585-61fff1185c34:~$

Instead, perhaps show the job/index of the vm (plus the path):
cloud_controller/0:~$

Closing - will be in the next stemcell via #1316.


Fix a memory error introduced by recent changes

ExistentialLayout::getProtocols() can return an interior
pointer if the layout describes a single protocol type.
In this case we must not hold on to the result after
the layout goes out of scope.
Fix a few places where this was happening, which should
address a test failure we've been seeing.
Fixes rdar://problem/31586433.

@swift-ci Please smoke test


Resolve #379

We check if one or multi has block command, if true, then there's no need to sleep.

@rueian This test takes too much time, maybe something is wrong.
https://app.circleci.com/pipelines/github/redis/rueidis/1021/workflows/2868a79d-6e02-4b6e-aead-d65fa1c4e01b/jobs/977

@rueian This test takes too much time, maybe something is wrong. https://app.circleci.com/pipelines/github/redis/rueidis/1021/workflows/2868a79d-6e02-4b6e-aead-d65fa1c4e01b/jobs/977

It looks like the DragonflyDB crashed during the test. Now the tests are passed after rerun.

@rueian This test takes too much time, maybe something is wrong. https://app.circleci.com/pipelines/github/redis/rueidis/1021/workflows/2868a79d-6e02-4b6e-aead-d65fa1c4e01b/jobs/977

It looks like the DragonflyDB crashed during the test. Now the tests are passed after rerun.

Thanks for helping!
And BTW, I notice that the test for MaxFlushDelay is weird, it init a Waitgroup, but didn't use what.Wait() to wait for it.
What's the purpose of this test?
https://github.com/redis/rueidis/blob/f10a1ee7c7b42f00469a841ebc79afb3e83342d0/pipe_test.go#L765

@rueian This test takes too much time, maybe something is wrong. https://app.circleci.com/pipelines/github/redis/rueidis/1021/workflows/2868a79d-6e02-4b6e-aead-d65fa1c4e01b/jobs/977

It looks like the DragonflyDB crashed during the test. Now the tests are passed after rerun.

Thanks for helping! And BTW, I notice that the test for MaxFlushDelay is weird, it inits a WaitGroup, but didn't use wg.Wait() to wait for it. What's the purpose of this test?
https://github.com/redis/rueidis/blob/f10a1ee7c7b42f00469a841ebc79afb3e83342d0/pipe_test.go#L765

Yes, the wg is not used and can be removed.
The purpose of that test is just to make sure that the mock.Expect("PING").ReplyString("OK") can be passed.


Bug -

Describe the bug
Mismatched nanddump1.bin and nanddump2.bin on Corona16MB with PicoFlasher ver v3.0
To Reproduce
Steps to reproduce the behaviour:

Click on "?" ->
Checking Console...
0x00043000
Corona 16MB
Click on "Read Nand"
Reading Nand to C:\Users\xxx\Downloads\XBox360\J-Runner with Extras\output\nanddump1.bin
Read Successful!

Initializing nanddump1.bin, please wait...
Nand Initialization Finished
Reading Nand to C:\Users\xxx\Downloads\XBox360\J-Runner with Extras\output\nanddump2.bin
Read Successful!
[Mismatched]
Comparing...
0x0064
0x01C2
0x027C
Expected behaviour
nanddump1 and nanddump2 should match???
Screenshots
https://i.imgur.com/Knr11TP.png
System (please complete the following information):

OS: Windows 10
Version:

Additional context
Try several times but still same results. Files can be downloaded here https://drive.google.com/file/d/1cQZZdQE--L_KZ30BhShLpAOsW3GKwRKx/view?usp=sharing

Not a bug. If the dumps don't match, something about your setup is wrong.
Kind Regards,
Josh

thanks, I will double check

confimed, it's bad wirings and/or wire are too long for stable reading. Thanks!


Unbreak intersection() & difference() for CAGs.

Only .setColor() in intersection() & difference() for non-CAGs.
A further improvement would implement CAG.setColor(), but that's not here.
To see the problem this fixes, observe this works:
function main() { return union(circle(), circle().translate([1, 0, 0])); }
but replacing union with difference or intersection errors out with:
Error in line 353: Uncaught TypeError: a[i].setColor is not a function
This patch removes this error and results in correct-looking intersection/difference.

@fischman Thanks for the changes. I think these are fine.
However, I don't like hard coded "colors", scale, etc. I'm wondering if the "cut' color should be configurable. And as you suggested, both the "union" color and "intersection" color should be changeable. Some other people want to change the defaults for color and resolution as well. A "common" approach would be best.

@z3dev your comments about configurable colors sound fine to me, but seem unrelated to the change proposed here (and I don't grok precisely what you're proposing so I can't implement it).  Since this PR fixes a blocking bug for me would you mind applying it as-is?

Not sure why I thought this was blocking me before - I can work around the bug easily enough by using foo.subtract(bar) instead of difference(foo, bar), and so on.  Do with this patch what you like :)


Exception if the host is unreachable

If the host is unreachable, there are errors in HA logs.
2020-06-16 08:49:47 ERROR (MainThread) [homeassistant.helpers.entity] Update for climate.midea_ac_81YY000000XX fails
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 279, in async_update_ha_state
    await self.async_device_update()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 470, in async_device_update
    await self.async_update()
  File "/config/custom_components/midea_ac/climate.py", line 108, in async_update
    await self.hass.async_add_executor_job(self._device.refresh)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.7/site-packages/msmart/device.py", line 179, in refresh
    data = self._lan_service.appliance_transparent_send(data)
  File "/usr/local/lib/python3.7/site-packages/msmart/lan.py", line 76, in appliance_transparent_send
    response = bytearray(self.request(data))
  File "/usr/local/lib/python3.7/site-packages/msmart/lan.py", line 34, in request
    sock.connect(device_address)
OSError: [Errno 113] Host is unreachable

LGTM, Thanks 👍


Error- Module 1-  Exercise 1: Creating users in Azure AD

Module:1-  Exercise 1: Creating users in Azure AD## Lab/Demo: 00
Task: Task 2: Create users by using PowerShell
Step -5 ( This is correct PowerShell command )
Description of issue- Command is in correct
Correct PS command
New-MsolUser -DisplayName "Cody Godinez" -UserPrincipalName<EMAIL_ADDRESS>-Password "pass@word1" -FirstName "Cody" -LastName "Godinez" -ForceChangePassword $false -UsageLocation "US" -Title "Sales Rep" -Department "Sales"

Hello Vikasmct,
The only change to the code in the labs and the one you provided is the order that you entered the properties in and that you updated the password and UPN. Note that the @yourtenant.onmicrosoft.com is there to act as a placeholder for your tenant. Depending on your lab provider, the tenant address will be different for each learner.


Clarify skipping application step of setup

Just small clarification to adding ex_machina as application setup step

Thanks for helping us clarify this @azranel!


[fixed] hash history trigger listeners twice when push a new location

What is this supposed to do? The guard in transitionTo in createHistory already catches this. Your test case passes against master.

@taion history.createHashHistory will trigger listeners twice when path is changed
history.push('/test') // trigger listeners and modify location.hash
hashchange handler // trigger listeners

locationsAreEqual at transitionTo can not detect this problem, because of the key is different between nextLocation and currentLocation.

Which browser do you see this on?

It is chrome 47.0.2526.73

Can you push up a commit with just the test case and not your code change? I want to see the CI fail on that. The test ran successfully for me locally without the code change.

OK


feat(statsd): statsd tag support

Summary
Support including tags to the metrics for the StatsD plugin
tag doc is here https://github.com/prometheus/statsd_exporter#tagging-extensions

Checklist

[x] The Pull Request has tests
[x] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog
Issue reference
FTI-4622
FT-3596
KAG-135

tests and docs?

@hanshuebner Are you available to help review this change? Thanks in advance!

I am OOO this week, it'd be better if someone else took a look unless there
is anything in there that needs my eyes right now.

Please make sure a version compatibility PR is opened at EE side.


Prepare Chromatic POC

Prepare a working Chromatic deployment to be used as a proof of concept. The POC should include a working Chromatic deployment with screenshot comparison and pull request integration.
It would be nice to be able to demo the following steps

Introduce a change and create a PR
See a Chromatic run in the PR checks
Compare, review and approve versions on Chromatic
See approval in GitHub PR checks

This POC will serve as the base argument to get funding.

Related: #1053

Out of scope for the moment.


Wrapper should be ManyInsertable, but is only Insertable

// AlamofireCoreData
struct Wrapper: Insertable, ManyInsertable { }
struct Many<Element: ManyInsertable>: Insertable { }

struct LeaderboardPosition: Wrapper { }
Many<LeaderboardPosition> // is `Insertable`

// AppwiseCore
struct Wrapper: Insertable { }
extension Array: Insertable where Element: (Insertable & ManyInsertable) { }

struct LeaderboardPosition: Wrapper { }
Array<LeaderboardPosition> // is not `Insertable` because `Wrapper` is not `ManyInsertable`

I have created a pull request that fixes this.


refactor(deps): switch to cosmossdk.io/math from fork math

Progress towards: https://github.com/osmosis-labs/osmosis/issues/6099
What is the purpose of the change
This PR proposes creating an alias for sdkmath types in osmomath.
Primarily, its goal is to remove the need for the verbose and unreadable prefix "Legacy" from Dec.
To avoid having to directly import both sdkmath and osmomath in files where both Dec and Int/Uint types are used we also create aliases for integer types. The aliases are created in the new file sdk_math_alias.go
To avoid name clashes with BigDec and BigDec, their constrictor APIs are now broken to contain "Big" in the suffix. For example, NewDec -> NewBigDec.
Important Notes

Note: with the update, the Dec and Int do not get initialized to zero values
by default in proto marhaling/unmarshaling. Instead, they get set to nil values.
maxDecBitLen has changed by one bit so overflow panic can be triggerred sooner.
Had to update precompute.go in CL due to maxBitLen changes
For converting Int to Dec, "LegacyDec" is still present since it is impossible to define a method on Int outside of cosmossdk.io/math package
I suggest we do not backport this due to complexity of resolving all these issues and the changes to maxDecBitLen + proto marshaling

Testing and Verifying
This change is already covered by existing tests
Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[x] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[x] N/A

Just curious, what's the reason for decision behind making an alias package osmomath and then importing from osmomath directly instead of using sdkmath package?

This is to reduce verbosity from the LegacyDec API stemming from upstream

grep -r "sdk.ZeroDec("

Seems that some updates got affected by merge conflicts, thanks for catching. Will reswitch everything prior to merge


MF522 based SPI reader does not work

Trying my best to get this great software to work but have huge issues getting the MF522 reader to work.
My steps so far:

Installed the MF522 according to https://pimylifeup.com/raspberry-pi-rfid-rc522/

Works great reading tags with the example code on this page.

Taking a blank raspos.. only configure wifi + ssh
running the default install script. No Spotify, No GPIO, With the 522 hardware selected.
With lsmod | grep spi the reader shows up

pi@raspberrypi:~ $ lsmod | grep spi
spidev                 20480  2
spi_bcm2835            20480  0
-- No working reading of card in the gui

Replacing the Reader.py with Reader.py.experimental

No working reader

Running the ./setup_rc522.sh
-- No working reader

Any input would be appreciated.

Running this small test software reads cards.. Hardware seems to be working fine.
Have not managed to do it with the Reader.py.experimental file.
#!/usr/bin/env python
import RPi.GPIO as GPIO
from mfrc522 import SimpleMFRC522
reader = SimpleMFRC522()
try:
id, text = reader.read()
print(id)
print(text)
finally:
GPIO.cleanup()

Hey mippen,
did you connect the IRQ PIN from the card reader with the GPIO?
In the link to pimylifeup it isn't required, but the software of the jukebox needs this connection. It is also mentioned in the Wiki.
That was my mistake a few weeks ago. The card reader worked in a simple test, but not with the jukebox software. I just connected the IRQ PIN and it worked properly.
Best regards,
Dalada


Add more padding to give tables more space

@reviewbybees

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

:bee:


error using type union or intersection in defineProps

Vue version
3.2.47
Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-1fkknn
Steps to reproduce

Create a type union/intersection
Use the type union/intersection in defineProps

What is expected?
defineProps allows type union/intersection
What is actually happening?
defineProps doesnt allow type union/intersection
[vite] Internal server error: [@vue/compiler-sfc] type argument passed to defineProps() must be a literal type, or a reference to an interface or literal type.

/home/projects/vitejs-vite-1fkknn/src/components/HelloWorld.vue
12 |  type HelloWorldProps = BaseProps & OtherProps;
13 |
14 |  withDefaults(defineProps<HelloWorldProps>(), {
   |                           ^^^^^^^^^^^^^^^
15 |    msg: 'Hello World',
16 |  });

System Info
No response
Any additional comments?
No response

This is known and wont' be supported before 3.3 (currently in beta)


The 'hostname' in config not working when using docker

I have a influxdb container on '<IP_ADDRESS>:8086'.
Then I start a kapacitor container with kapacitor:latest:
docker run -p 9092:9092 \
      -v $PWD/kapacitor.conf:/etc/kapacitor/kapacitor.conf:ro \
      kapacitor

Here is my config
httppost = []
hostname = "<IP_ADDRESS>"
data_dir = "/var/lib/kapacitor"
skip-config-overrides = false
default-retention-policy = ""

[http]
  bind-address = ":9092"
  auth-enabled = false
  log-enabled = true
  write-tracing = false
  pprof-enabled = false
  https-enabled = false
  https-certificate = "/etc/ssl/kapacitor.pem"
  shutdown-timeout = "10s"
  shared-secret = ""

[replay]
  dir = "/var/lib/kapacitor/replay"

[storage]
  boltdb = "/var/lib/kapacitor/kapacitor.db"

[task]
  dir = "/root/.kapacitor/tasks"
  snapshot-interval = "1m0s"

[[influxdb]]
  enabled = true
  name = "default"
  default = true
  urls = ["http://<IP_ADDRESS>:8086"]
  username = "admin"
  password = "admin"
  ssl-ca = ""
  ssl-cert = ""
  ssl-key = ""
  insecure-skip-verify = false
  timeout = "0s"
  disable-subscriptions = false
  subscription-protocol = "http"
  kapacitor-hostname = ""
  http-port = 0
  udp-bind = ""
  udp-buffer = 1000
  udp-read-buffer = 0
  startup-timeout = "5m0s"
  subscriptions-sync-interval = "1m0s"
  [influxdb.excluded-subscriptions]
    _kapacitor = ["autogen"]

but in influxdb , I got this:
name: _internal
retention_policy name                                           mode destinations
---------------- ----                                           ---- ------------
monitor          kapacitor-f5d7399f-5876-417f-ad24-9a227042f67c ANY  [http://ef412dbdc801:9092]

I try to change the host value to 'kapacitor', but got another:
name: _internal
retention_policy name                                           mode destinations
---------------- ----                                           ---- ------------
monitor          kapacitor-ad4dc3b6-1f8c-46d6-b506-62241a1702bd ANY  [http://aa121f1b601d:9092]

Finally I try this, and it works:
docker run -p 9092:9092 \
    -v $PWD/kapacitor.conf:/etc/kapacitor/kapacitor.conf:ro \
    -e KAPACITOR_HOSTNAME=<IP_ADDRESS> kapacitor

The result:
name: _internal
retention_policy name                                           mode destinations
---------------- ----                                           ---- ------------
monitor          kapacitor-e0d48f68-1159-4f07-a2e3-b3c021d8bee0 ANY  [http://<IP_ADDRESS>:9092]

:smile::smile:

Same issue here, any update or fix we can try?

Just fixed it by using: https://github.com/influxdata/kapacitor/blob/master/etc/kapacitor/kapacitor.conf#L126-L130
[[influxdb]]
...
kapacitor-hostname = "kapacitor-host.tld"

,,,

With Docker/Kubernetes, I was able to get it to work by passing in the KAPACITOR_HOSTNAME as an environment variable.
The kapacitor-hostname param in kapacitor.conf resulted in a "key already defined" error for me


Automated cherry pick of #7520: fix: optimized huawei security group get and create operation

Cherry pick of #7520 on release/3.3.
#7520: fix: optimized huawei security group get and create operation

Codecov Report

Merging #7521 into release/3.3 will decrease coverage by 2.15%.
The diff coverage is 0.05%.

@@              Coverage Diff               @@
##           release/3.3   #7521      +/-   ##
==============================================
- Coverage         6.78%   4.62%   -2.16%
==============================================
  Files              676     423     -253
  Lines           118955   87467   -31488
==============================================
- Hits              8076    4049    -4027
+ Misses          110267   83144   -27123
+ Partials           612     274     -338

Flag
Coverage Δ

#aFlag
4.62% <0.05%> (-2.16%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudcommon/db/db_dispatcher.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/domain.go
3.48% <0.00%> (-0.09%)
:arrow_down:

pkg/cloudcommon/db/domainresource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/external.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/infraresource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/managed.go
0.00% <ø> (ø)

pkg/cloudcommon/db/modelbase.go
4.77% <0.00%> (-0.04%)
:arrow_down:

pkg/cloudcommon/db/opslog.go
1.52% <0.00%> (+0.03%)
:arrow_up:

pkg/cloudcommon/db/rbac.go
1.16% <0.00%> (-0.02%)
:arrow_down:

pkg/cloudcommon/db/scoperesource.go
0.00% <0.00%> (ø)

... and 340 more

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 52488ee...c216fb9. Read the comment docs.

/lgtm
/approve


Developed an alternative in nodejs

Seeing as this is no-longer being maintained, I thought I might let others know of my project written with NodeJS that is actively maintained.
https://github.com/billchurch/WebSSH2
Welcome any help or feedback there. # #

close for this is not an issue.


Some CI builds are too slow

The following five builds exceed the desired 15 minute run time for CIs:

Windows GitHub CI / shared (win32) Successful in 37m
Windows GitHub CI / shared (win64) Successful in 24m
CIFuzz / Fuzzing Successful in 20m
Windows GitHub CI / plain Successful in 17m
GitHub CI / non-caching Successful in 17m

The first two or three are well over.

On the other hand without Windows CI on PRs we are getting constant regressions. I would say we could perhaps drop some tests from the shared builds if we can identify some that are slow and unlikely to get broken only on Windows.

I would say that if you want to drop tests for speed reasons, then you need to review your priorities

But then, we might also want to review what tests should be run for every PR, and what tests only need to be done on a daily basis (or some other regular unit of time)

I would not recommend dropping any of these except of CIFuzz from the set of tests run for every PR. All these find regressions regularly and fixing them post-merge is always harder and with more overhead than when catching the issue before the PR is merged.

I don't have a problem relegating CIFuzz to one of the less often run suites.  It's not the bottleneck here, so I'd be tempted to keep it.  At least until the even slower tests are addressed.
I do agree the others find errors too often to be dropped.
Are alternate Windows tests possible?  They aren't all this slow.  We could also look into reducing the test suite coverage.

Closing this as it doesn't seem very relevant.  So CIs are slow but they aren't impacting negatively.


Update github.com/valyala/fasthttp to version >=1.34.0

There is a CVE on fasthttp: https://github.com/advisories/GHSA-fx95-883v-4q4h, github.com/valyala/fasthttp version >=1.34.0 should fix that. Please update this dependency so we can mitigate this in https://github.com/kedacore/keda/issues/2775
Thanks

<EMAIL_ADDRESS>uses<EMAIL_ADDRESS>as dependency, could you bump it too?


Update docs

Supersedes #316

This is good to go. I'll push the v0.0.4 tag whenever this gets merged

Other that file that I don't think should have been removed, feel free to self-merge. Tagging suggestion:
git commit --allow-empty -m "REL: v0.0.4rc1"
git tag v0.0.4rc1
git push upstream --tags

If everything seems to work (check __version__) then repeat, dropping the rc1.

good catch


Add apiDocsConfigRef to configure ApiDefinitionWidgets

Hey, I just made a Pull Request!
This builds off #2394 to make the definitionWidgets configurable at the app level. cc @Fox32

Added ApiDocsConfig type and ApiDocsConfigRef. This uses a selector function instead of an array widgets to provide more flexibility of the entire apiEntity rather then just the spec.type.
Added plugin api factory which defaults to using the defaultDefinitionWidgets function and logic to get the widget based on spec.type.
Removed props for definitionWidgets and rely on apiDocsConfigRef instead (which defaults to the same logic).
Fixed react warning with CardTab complaining about missing index/key

:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Not sure if this is the nicest API for this tbh, be we're also somewhat lacking sane options right now. I'd be fine to merge but re-evaluate as we're getting patterns for selecting components based on entity data in place.

@Rugvip - Can you elaborate? Are you referring to the getApiDefintionWidget(entity) => ApiDefinitionWidget | undefined?
Another option could be definitionWidgets: ApiDefinitionWidget[]. This would be simpler, but would not allow the consumer to control which widget is shown for which entity type. I suppose it could support both approaches as well, but might be confusing. Open to other ideas.

@andrewthauer yeah sorry should've been a bit more specific 😅
I mean using a Utility API altogether, as opposed to JSX-driven approach.
Something like this in the app, where widgets are selected based on type:
<ApiDefinitionCard>
  <OpenApiDefinitionWidget type='openapi' />
  <AsyncApiDefinitionWidget type='asyncapi' />
</ApiDefinitionCard>

Just looking forward though, this definitely makes sense given the existing implementation and lack of methods to produce an API like that :p


Mainnavigation: grey background-color should only appear on :hover

Mainnavigation: grey background-color should only appear on :hover, but not on :focus, as this would still appear after the user has left the navigation item with their pointer device.

I think this is already solved, can you check this @mfranzke ?

I think this is already solved, can you check this @mfranzke ?

It's a new topic that just came up.


Piano : To extend the octaves and audio updation.

🚀 Feature
I want to extend the octaves and link better audio quality notes of the keys and want to improve the styling of the site.

I am a Participant of LGM-SoC 2021.
Can I work on this issue ?

/assign


Error while getting Socket.Clients

Just trying get a particular socket with id.
   var serverSocket = scServer.client[socket.id];

The above causes: [Error] TypeError: Cannot read property 'AheT4t9bLxMGoYG9AAAA' of undefined.
Full code:
class Worker extends SCWorker {
  run() {
    console.log('   >> Worker PID:', process.pid);
    var environment = this.options.environment;

    var app = express();

    var httpServer = this.httpServer;
    var scServer = this.scServer;

    if (environment === 'dev') {
      // Log every HTTP request. See https://github.com/expressjs/morgan for other
      // available formats.
      app.use(morgan('dev'));
    }
    app.use(serveStatic(path.resolve(__dirname, 'public')));

    // Add GET /health-check express route
    healthChecker.attach(this, app);

    httpServer.on('request', app);

      socket.on('subscribe', function(channelname){
        console.log('subscription completed for ' + channelname + ', ' + socket.id);
        console.log('All Channels: ' + socket.subscriptions());

        console.log('client count : ' + scServer.clientsCount);
        var serverSocket = scServer.client[socket.id];

        if( serverSocket != null)
        {
          console.log('Server Socket Found');
        }
        else{
          console.log('server socket not found!');
        }

      })

      socket.on('disconnect', function () {
        clearInterval(interval);
      });
    });
  }
}

new Worker();

Was never able to find it. Thanks a lot!
I'm used to .net IDE & hence intellisense minded ;)


fix: Make submit button enabled when required fields are filled by default

Description

Fixes #385
Checklist

This fixes the issue where the submit button is disabled when required fields are filled by default

[ ] My code follows the contributing guidelines of this project.
[ ] I am aware that all my commits will be squashed into a single commit, using the PR title as the commit message.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code in hard-to-understand areas.
[ ] I have updated the user-facing documentation to describe any new or changed behavior.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have not reduced the existing code coverage.

Comments

Summary by Sourcery
Bug Fixes:

Ensure the submit button is enabled when required fields are filled by default by adjusting the logic that manages the 'requiredButEmpty' list.

hey @anuragxxd, please take a look at this.


Block edge 4.2.36

4.2.36 needs to be disabled:
2 clusters currently failing (17%),  3 gone (25%),and  4 successful (33%), out of 12 who attempted the update over 7d
Currently failing classification:
* API crashlooping: 1
* SDN issue: 1
* Slow etcd: 1
* UPI router degraded: 1

4.2.36 can be enabled again:
1 clusters currently failing ( 8%),  3 gone (25%),and  7 successful (58%), out of 12 who attempted the update over 7d
Currently failing classification:
* API crashlooping: 1


WIP: adding a parameter for installGit.sh instead of hard coded value

Work in progress for accepting whisk host name parameter.  @bjustin-ibm  I should probably also rename WHISK_API_HOST to WHISK_EDGE_HOST here as well?

@beemarie Yes, I would change these variable names to better reflect their values - maybe WHISK_EDGE_HOST and WHISK_API_HOST?

This is a companion PR to https://github.com/apache/incubator-openwhisk/pull/2429. As such, Travis will fail until that PR is merged.

Closes #224

closing as abandoned.

this should be subsumed by the wskdeploy manifest in #268 @pritidesai.


Fix bugs from merging PR #27

Signed-off-by: Jorge Padilla<EMAIL_ADDRESS>Fixes some bugs introduced by PR #27

Reload config after initializing dev mode.
Add CASCADE when dropping the search schema.
Add IF NOT EXIST when creating indexes

/lgtm


Create rule DDF00030: For classes with next and previous relationships, the next and previous relationships must establish an ordered list of instances of the class (valid EligibilityCriteria id values for nextId and previousId).

Create rule to check for valid nextId and previousId values for EligibilityCriteria as defined for #79

@DianeWold This one is also ready for review. It's checking that the id values used in nextId and previousId in the EligibilityCritieria class are valid for both the study design and category.

@DianeWold @ASL-rmarshall
The description of the core rule is not specific enough. It checks whether if a previous or next reference exists, then this reference is also existing in the list of eligibility criteria. Can we make it more specific?
From the actual check perspective, I think it would also be possible to include in this check that it does not reference to itself. or " previousID=id" or "nextId=id". Do you want to include that?

This rule has been updated to only report where rel_type = "definition". Associated test data have been updated and unit tests have been rerun successfully.

@DianeWold @ASL-rmarshall
This check is now performed within a category: Inclusion / Exclusion
However, we do not have to include that restriction as we can indicate the overall ordering of criteria without the limitation of category.

Saving a copy of the POC rule definition before deleting for DDF 4:
Authorities:
  - Organization: 'CDISC'
    Standards:
      - Name: USDM
        References:
          - Citations:
              - Cited Guidance: 'USDM'
                Document: 'USDM v2.6'
              - Cited Guidance: 'SDTMIG'
                Document: 'SDTMIG v3.4'
                Section: '7.2.1'
            Origin: USDM Conformance Rules
            Release Notes: ''
            Rule Identifier:
              Id: 'DDF00030'
              Version: '1'
            Validator Rule Message: ''
            Version: '1.0'
        Version: '3.0'
Check:
  all:
    - name: rel_type
      operator: equal_to
      value: 'definition'
    - any:
        - all:
            # An id value is specified for nextId
            - name: nextId
              operator: non_empty
            # And the specified nextId value is not a valid id for the StudyDesign ...
            - name: nextId
              operator: is_not_contained_by
              value: $el_crit_ids_for_study_design_cat
        # Or ...
        - all:
            # An id value is specified for previousId
            - name: previousId
              operator: non_empty
            # And the specified previousId value is not a valid id for the StudyDesign ...
            - name: previousId
              operator: is_not_contained_by
              value: $el_crit_ids_for_study_design_cat
Core:
  Id: CORE-000422
  Status: Draft
  Version: '1'
Description: 'The eligibility criteria of a StudyDesign are ordered correctly
  according to the specifed nextId and previousId values.'
Executability: Fully Executable
Operations:
  - group:
      - parent_id
      - category.code
    id: $el_crit_ids_for_study_design_cat
    name: id
    operator: distinct
Outcome:
  Message: 'The eligibility criteria of a StudyDesign are not ordered correctly
    according to the specifed nextId and previousId values: the id value
    specified for nextId or previousId is not one of the eligibility criterion
    ids for the eligibility criteria category of the StudyDesign.'
  Output Variables:
    - parent_entity
    - parent_id
    - category.code
    - category.decode
    - id
    - nextId
    - previousId
    - $el_crit_ids_for_study_design_cat
Rule Type: Record Data
Scope:
  Entities:
    Include:
      - EligibilityCriteria
Sensitivity: Record


Merge develop branch to v2 branch

Codecov Report

Merging #2436 (581bdcd) into v2 (d6bb288) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               v2    #2436   +/-   ##
=======================================
  Coverage   97.62%   97.62%
=======================================
  Files          56       56
  Lines       10685    10685
=======================================
  Hits        10431    10431
  Misses        254      254

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 d6bb288...581bdcd. Read the comment docs.


Adding metricRelabelings and relabelings options

Adding support in the Service Monitor object for relabeling.
Prometheus documentation: https://prometheus.io/docs/prometheus/latest/configuration/configuration/#relabel_config

Thanks. Please merge master to run CI.


override SessionManager

...to allow CookieManager to set the session cookie.
See https://github.com/magento/magento2/issues/27531
The actual change is very minor: https://github.com/magento/magento2/pull/27535/files
However, because of all the private properties and methods in SessionManager, we have to override a lot of it to make this change.

Sorry one small warning - due to the session_start() call (I think) in the SessionManager::start() method, when you don't have a session cookie already, this will write out two Set-Cookie headers - the second one from our CookieManager, the first from php.

@johnorourke Thanks! I didn't watch latest core implementation!


Preserve linebreaks in broker error messages

See https://github.com/openshift/origin-web-console/issues/2217

@jim-minter FYI, thanks for improving the message from TSB


Suggest argument names based on type definition of callback functions

Search Terms
callback, arguments, parameters, suggestion, auto correct

Suggestion
Let's say I have a type definition for my function like this:
function acceptsCallback(callback: (argumentNameThatIsVeryLong: string) => void) {}

When using it, that argumentNameThatIsVeryLong is a very good candidate for my argument name. I want to accept it but there is no way but typing it:

Use Cases
When users are extremely lazy this is very handy! ;) In all seriousness this can avoid bugs by suggesting the right argument name. For instance when req and res objects are misplaced in an Express middleware by mistake
Examples
https://www.typescriptlang.org/play?#code/GYVwdgxgLglg9mABAQwhApgBygZwMLIA2hARqgNYAUERpFAXIpcgE4DmIAtumFAHLJuAFQAWyKAEkcANXQsAngBkEbRjigsYYNgEpEAXgB8iAG5wYAEz0BvAL4Aoe6gzZ8tMhCqPv95ux32OkA
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

I would love this. Maybe even Auto-complete the entire callback including arguments and function body?


Random.choices() division by zero

Goost and Godot version:
4cfacd390a0e2f3c832f19677edbb195a234971f, 3.x
Steps to reproduce:
This is input from the fuzzer: https://github.com/goostengine/goost-fuzzer/runs/5261841226
/home/runner/work/goost-fuzzer/goost-fuzzer/goost/core/math/random.cpp:136:32: runtime error: division by zero

Random.choices({"roman" : 22, 22 : 25, BoxShape.new() : BoxShape.new()}, 37, PoolIntArray([]), true)

CC @sumadithya

I just didn't have checks for when is_cumulative is true
Do I have to make a new branch for this issue (to make a PR)?

Do I have to make a new branch for this issue (to make a PR)?

Yeah, feel free to submit a new PR referencing this issue (like "Closes #179", to automatically close it upon fixing).


chore: build augury, create crx file, and move it to the artifacts folder in circleci

Changes

Build the crx and zip file for Augury and place it in the Artifacts folder in CircleCI
The uploaded files are viewed within CircleCI under the Artifacts tab. https://circleci.com/gh/rangle/augury/908#artifacts (this is the build for this PR)

@ericjim LGTM :shipit:

I do have a possible solution, but it will require research. If I am successful, I'll add it as a separate PR.


Check return value of osc in rpm-packaging job

Otherwise in case the osc commands fails, the job exits with 0 and
the build looks good ut in fact it's maybe broken.
This happened i.e. when osc returned non-zero and reported:
SSL Error: (104, 'Connection reset by peer')

+1


ITM Loss Stuck at 0.63

Hi, I am trying to replicate and pretrain BLIP for distillation purposes - I am using Flickr30K + COCO and my ITM loss gets stuck at 0.63 - upon an initial look, all of the ITM predictions are 1. Is this a dataset size issue or a batch issue? I've tried changing the learning rate to a smaller LR, I've tried increasing the size of the model and more, but nothing seems to work.

It seems to predict 0s for all of the ITM labels no matter what

so what is solution for this problem? i am encountering same problem

It is a bug of the model itself


Moved lightning flash rate (LFR) calculation from MOIST to ChemEnv

This pull request is one of three that must be done together (GEOSgcm_GridComp, GMAO_Shared and GEOSchem_GridComp). The result is zero-diff for PCHEM simulations. It is non-zero-diff for simulations that use Lightning Flash Rate, such as GMI.
This set of PR's is focused on the Lightning Module, which now computes both Flash Rate and NOx Emissions. The module has been moved from GMAO_Shared to Chem_Shared, since its output is used only within Chemistry code. The lightning module has also been significantly re-worked. Different Flash Rate algorithms are now available.

@mmanyin after testing these 3 branches/PRs, it appears to be non-zero-diff. In both AMIP and replay modes, the CH3I variable has slightly different values in tr_internal_checkpoint and the FROCEAN variable was dropped from moist_import_checkpoint. Obviously, the same differences occur with CH3I in the history geosgcm_tracer. Are these changes intended as this is listed as 0-diff?

@sdrabenh indeed you are correct, this is an oversight of mine!  The changes came in the CHEM PR 137 (https://github.com/GEOS-ESM/GEOSchem_GridComp/pull/137) over 2 months ago, not the trio of PR's from last week.  And my recent zero-diff tests were against 'develop', not the previous release-- my fault.  So where does that leave us?  (1) a small number of passive tracers (TR) are non-zero-diff due to re-factoring (on the order of round-off), and one of those (CH3I) is run as part of the standard release; none of the TR tracers has any effect on any other part of the model, so this difference is extremely minor, but it does exist.  Issue (2): the FROCEAN field is no longer needed by MOIST-- admittedly I should have noted this difference, but does this "removal of deadwood" trigger a non-zero-diff flag?  Bottom line-  should we postpone the 3 PR's to make for a cleaner release today, and label them all "non-zero-diff" for the next round of updates?  Please advise.

@mmanyin thanks for the explanation on what is going on. These changes seem minor, but nevertheless, still non-zero-diff. I think the best course of action is to save these 3 PRs for the next round of non-zero-diff updates. I would be more tempted to take it now if it was only dropping FROCEAN. However, some tracer values are different. I plan to take these PRs soon since there are other non-zero-diff updates waiting in the wings regarding orbital changes and compiler upgrades. Does this sound reasonable?

@sdrabenh  yes, your plan to include these in a non-zero-diff release sounds like the right way to go.  Sorry for the confusion.


auto generated Go client

I am personally against merging generated code into the repo (but am not a maintainer of this repo). I avoid it for two reasons:

Codegen becomes stale very quickly and does not track the upstream.
Codegen bloats the repo and is trivial to build in an automated fashion.

Perhaps you could use a build system to generate this code? Or perhaps something similar to one of the existing bash scripts?
Here's some discussions on codegen in the repo:

https://github.com/dotansimha/graphql-code-generator/discussions/4253
https://stackoverflow.com/questions/893913/should-i-store-generated-code-in-source-control


Notebooks with slash in name leads to stop of sync

If you have an notebook with a slash in the name, the synchronizations stops with an error:
! (1/42) PULL: notebooks/project 2024/2025.pdf
Could not download http://<IP_ADDRESS>/download/78e130bc-75cb-4290-85ba-19b79271d01b/placeholder from reMarkable USB web interface
Make sure that Settings > Storage > USB web interface is enabled
ERROR: All renderers failed to render notebooks/project 2024/2024.pdf

I think, the main problems is the misinterpretation of the USB web interface, which is looking for a folder /project 2024/ which does not exsits, I don't know, if there is a workaround for this.
It might be better, if the pull just omits the dedicated file, and tried downloading the other ones.

Thanks for reporting! I will try to fix it when I get time.


Update CurrentCommercial.md

OPS Build status updates of commit ce69cc2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/release-notes/CurrentCommercial.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 43346c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/release-notes/CurrentCommercial.md
:white_check_mark:Succeeded

windows-iotcore/release-notes/commercial/17763.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


New Release (incompatibility with Envoy >= 1.27)

The PullRequest https://github.com/envoyproxy/envoy/pull/27865 introduced a breaking change (introducing a new RawValues field that replaces Values in the corev3 HeaderValue message), which is not yet part of any release.

This change is guarded by a runtime flag envoy_reloadable_features_send_header_value_in_bytes, if you set this to false it should fix your issue?

Hey folks!
Thx for getting back - we'll try the feature flag change but still want to migrate to the RawValue field. Is there a new release of the go-control-plane planned? (@jpeach maybe?)

no stalebot

I think we get impact by this issue too when I check the message received by server I see "3", probably due to the new value in proto and unsupported backward compatibility
pb.ProcessingRequest_RequestHeaders &{headers:{headers:{key:":authority" 3:"localhost:8000"} headers:{key:":path" 3:"/headers"} headers:{key:":method" 3:"GET"} headers:{key:":scheme" 3:"http"} headers:{key:"user-agent" 3:"curl/8.2.1"} headers:{key:"accept" 3:"/"} headers:{key:"x-forwarded-proto" 3:"http"} headers:{key:"x-request-id" 3:"dd5ef52f-249a-4191-a6b6-e3f54b8ac8cd"}}

Yes, this is exactly the same we're facing @juanmolle
We are in the process of migrating to the new go-control-plane release but it seems header modifications in the extProc are affected as well. Didn't have the time to dig deeper on it yet

Just FYI, we've resolved this in the ExternalProcessor configuration by setting allow_mode_override: true alongside using the RawValue field for getting/setting headers both in request/response context as well as header mutations.
Both fields cannot be set though and causes 500s - we solved this by a selective rollout with dedicated services for each envoy/Istio version, one using the Value and the other one using the RawValue field.
@lizan we never had a good way to set the FF so we opted to migrate this way now. Feel free to close this issue, but it definitely highlights a breaking change with the default version between configurations


I.E. 10 calls external-link-warning switching between pages

First of all, awesome plug-in!  Thanks for sharing.  Is there a way to tell IE to not call the warning when you're switching between pages.  As far as I know, this is the only browser that behaves like this.
Thanks!

@grahamunr That’s bizarre! Thanks for bringing it to my attention. I’ll have a look into it and see why IE’s doing that, and patch it when I’ve found out.

Awesome.  Thanks for the fast reply!

@grahamunr Just updated my comment above. Take a look and let me know.

I just tested it and it appears to still be calling the function when navigating to another page.


HTTP/2 client requests home page or http interface no response

Current Behavior
When the following request headers are included, the node-red server does not respond:
Connection: Upgrade, HTTP2-Settings
Upgrade: h2c
HTTP2-Settings: AAMAAABkAAQCAAAAAAIAAAAA

Expected Behavior
It should be compatible and respond to http/1.1 protocol results normally.
Steps To Reproduce
Any http client request with the above request header will not respond normally.
Example flow
const axios = require("axios")
axios.get("http://localhost:1880", {
  headers: {
    Connection: "Upgrade, HTTP2-Settings",
    Upgrade: "h2c"
  }
}).then(res => {
  console.log(res.status, res.data)
}).catch(err => {
  console.log(err)
})

Environment

Node-RED version: v2.1.4
Node.js version: v16.13.1
npm version: v8.2.0
Platform/OS: Windows Locally or Docker
Browser: None

We do not support HTTP/2 and it is unlikely that we will any time soon.
We rely on the express framework for our http route handling. Upstream issue: https://github.com/expressjs/express/issues/2761

If you REALLY need HTTP/2 support (or at least graceful fallback) you can use something like Nginx as a reverse proxy, just remember to include the proxy_http_version  1.1; option so it knows that down stream server only supports HTTP/1.1

We do not support HTTP/2 and it is unlikely that we will any time soon.
We rely on the express framework for our http route handling. Upstream issue: expressjs/express#2761

Thanks for the reply. But my problem is not here.
There is no need to support the HTTP/2 protocol, but the client's HTTP/2 negotiation request should be ignored.(express.js is like this now)
Unfortunately, the HTTP/2 negotiation request is very similar to the Websocket handshake request. As follows:
// HTTP/2
Connection: Upgrade
Upgrade: h2c

// Websocket
Connection: Upgrade
Upgrade: websocket

The following code does not judge the Upgrade request header and calls the Websocket processing function:
https://github.com/node-red/node-red/blob/91cb6ba73b521c1334b6559dc69ee165f34cb931/packages/node_modules/%40node-red/editor-api/lib/editor/comms.js#L209-L236
This problem is not serious, and it can be handled as @hardillb said

Ah right. That makes sense.
We should ensure the websocket upgrade code only gets applied to websocket upgrades. We can then properly reject any other type of upgrade.

Not fixed after 2 years in v3.1.0.

@0000sir indeed. The upstream issue is still open.

Is there any plan to support http/2?

@dhamotharan19 run Node-RED behind a proxy (e.g. nginx)


Fixing sbk-ram and sbk-gem-yal module javadoc warnings.

Issues -> #299 and #301
Fixed all sbk-gem-yal and sbk-ram module Javadoc warnings, added comments where it was missing.
@kmgowda please review my changes and please suggest any suggestions.

thank you @Amartya-Mahato


also look in just_pressed touches for position

Objective

On some devices, UI buttons are not responsive

Solution

On device with a slower frame rate, touch event can start and end in the frame rate
When looking for a touch position, also look into the just_pressed touches that are not cleared by the end event but only at the end of the frame

I think this issue will be resolved when the pointer abstraction from mod_picking is up-streamed. Touch pointer entities are inserted in First and removed in Last, so position data for each touch always lasts at-least one frame.


Update react components to enable noimplicitAny use

What

React components to enable noimplicitAny use.
One note:
The utils is still marked as any as spread operator is causing complications with casting proper typing.

Note that CI will break here because communication-react does not have the noimplicitany check enabled yet.
It is causing an error on the use of @ts-expect-error flag in the utils file.


add support for deno in test suite

[x] docgen fix for markdown-toc dependency
[x] add deno task test to CI
[ ] generate Deno import map

@tim-smart With latest deno 1.38 tests seems to work with the unstable-byonm flag enabled

Alternatively just doing deno --unstable-byonm task test does the same trick

Alternatively just doing deno --unstable-byonm task test does the same trick

Well that certainly makes it easier :)

So for deno we can just use platform-node and it will work?

that is what you're doing effectively when doing import { ... } from "npm:effect"
Ryan Lee @.***> ezt írta (időpont: 2024. márc. 23.,
Szo, 6:47):

So for deno we can just use platform-node and it will work?
—
Reply to this email directly, view it on GitHub
https://github.com/Effect-TS/effect/pull/1579#issuecomment-2016364429,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMCMZMKNMTD5GQQLMUONJLYZUJOBAVCNFSM6AAAAAA6VGT6ICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJWGM3DINBSHE
.
You are receiving this because you commented.Message ID:
@.***>


Don't let dev be the new main

Thanks for chiadog, it's replaced my own scripts. I suggest to not fall into the "dev branch trap".
We have seen at plotman what happens when the main branch is not maintained and progress only happens on dev (users have to maintain own forks, difficult to update once it is eventually merged etc.).
The chia team decided to get rid of the dev branch altogether, a good choice imo. Instead they develop features on separate branches and merge them via pull requests into main.
If you have confirmed features are working, please consider integrating them into main sooner than later.

Thanks for your concern, I'm aware of this fallacy.
So far there has not been a feature sitting dormant in dev for longer than a week so I don't see it being a problem.
Features are already being developed in separate feature branches and tested prior to merging to dev. So if you want to stay on the bleeding edge, you can run from that branch. The difference is < 7 days.
main is meant to be a stable branch that is aligned with releases and has a slower release cadence of once a week at most (unless hotfixes are necessary). It allows for features to have been tested for a few days before release and reduces the overhead that people need to go through in order to stay up-to-date. I.e. less frequent need to familiarise with new features and audit changes to the codebase.
That being said, the next release was already prepared yesterday. But last minute I've noticed that newest version of Chia removed the logs for added coins, and that presents a small setback.

Thanks! That's good to hear.


Allow Symfony 5

Well, symfony 5 is around the corner, so let's allow it (especially as the master of the skeleton targets 5.0 now)

Thank you Baptiste.


Transitioning after a save raises an error

Ember      : 2.11.0-beta.2
Ember Data : 2.11.0-beta.1
jQuery     : 3.1.1

Calling Route#transitionTo() in the save action of a form raises the following error:
Uncaught TypeError: Cannot read property 'focus' of undefined
    at form-for.js:53
    at invoke (ember.debug.js:336)
    at Queue.flush (ember.debug.js:404)
    at DeferredActionQueues.flush (ember.debug.js:528)
    at Backburner.end (ember.debug.js:598)
    at Backburner.run (ember.debug.js:721)
    at Backburner.join (ember.debug.js:743)
    at Function.run.join (ember.debug.js:21614)
    at Class.hash.success (rest.js:954)
    at fire (jquery.js:3305)

The form saves and the transition occurs, but I'd like to understand if there's a better way to do this, or if this is a bug we can fix somehow.

From what I've seen, this only occurs if the action's name is submit. Changing it to something else stopped the form from calling handleErrors inside form-for.js.

I ran into this too - how did you pass a save action into the form? It seems like it only calls submit.
I wound up overriding the component with one that didn't do the handleErrors bit, because we didn't want the form to be focused like that.

@kevlarr That was as easy as adding submit=(action 'save') to the form component arguments, as seen here https://github.com/martndemus/ember-form-for#syntax
I also wish the addon was a bit more customizable, I'll have to create some custom controls just for styling purposes, but I believe eventually it will become the go-to addon for ember form creation.

Yeah, I'm using a route-action named signup and it's raising the same error, so I don't think that's the issue, or at least not the whole issue.

I am getting this same exception, but when saving the data I am altering the template to remove the form, rather than performing a route transition, my form is defined like this:
{{#form-for (changeset model UserValidations) submit=(action "save")  as |f|}}
I am using ember 2.9.1. I'm also very new to ember so quite possibly just getting something wrong!

I have this working for me, but not entirely sure if I'm masking a potential future issue.
I noticed that the code in handleErrrors was still scheduling an 'afterRender' event even if 'errors' is an empty array. Presumably this is not intended? So I added a check for array length.
Additionally I changed the afterRender to check that this.$() is still defined before calling focus on it, as this.$() being undefined is what was causing the exception to be thrown.
https://github.com/lineupninja/ember-form-for/commit/484cf51b78972281d9d37d33376aac438e0cb59c
Hope this helps. I can submit as a PR if that's appropriate.

@Geokoumpa Ah I thought you meant passing an action to the form-for's save.. Yeah, I passed a component action to submit, but that didn't change anything for me because it still calls this.handleErrors in both conditions of the if/else after that

Also having this bug. As a workaround, I wrapped the transitionTo in the run loop.
Ember.run.schedule('afterRender', () => {
  this.transitionTo('posts.post', model);
})

After merging #147, the included tests are working without modifying form-for.js
Thanks, for the participation and we will try to be faster next time.


probable non issue data/group data missing

Query really as everything seems to be working with 5.2.2RC3
Thought maybe group.db might be created after reboot of device, but no.
Error reading the file: data/group.db - Does not exist or is not readable. continuing...
under the startup log I am seeing the above Warning.
5.22RC3 just installed
other files within
/home/pi/habridge/data are:
pi@x10:~/habridge $ cd data pi@x10:~/habridge/data $ ls device.db                         habridge.config device.db-2018-10-01-18-58-56.bk  habridge.config-2019-03-04-10-20-47.cfgbk pi@x10:~/habridge/data $ pwd /home/pi/habridge/data pi@x10:~/habridge/data $
all X-10 devices seem to work dim/bright of all lights works as expected so far.
No Echo Gen 3 devices, all gen 2
Only items I have on my system are X-10 devices.
Is this missing group.db file going to cause any issues?

Non issue, you would create the group.db manually, originally introduced here: https://github.com/bwssytems/ha-bridge/issues/713


Fixed #28791 -- Safely caught Exceptions to allow core commands to execute when DJANGO_SETTINGS_MODULE is wrongly set.

This is a fix for https://code.djangoproject.com/ticket/28791
Safely catching for ImportError and ModuleNotFoundError. Now the core commands are being executed normally. I ran the tests.
Please let me know if I should make more changes.

Also it would be best to add a relevant test

Yes, I'll write a test for it.

@adamchainz will not this test suffice for this case?
https://github.com/django/django/blob/54e5c4a00e116ff4be257accdc9aa9e068c7f4ee/tests/admin_scripts/tests.py#L232
Also, is there any way we can get by without using --settings=bad-settings in  AdminScriptTestCase.run_django_admin method. I tried setting the environment variable but that does not affect the execution of run_django_admin so modifying the environment inside the testcase does not work.
What should I do?

@geekodour you'll need to add a new test case for this. I don't have the time to check, but you shoudl be able to modify run_django_admin so the environment variable works. mock.patch.dict can be used to temporarily override an environment variable using the os.environ dict.

@adamchainz okay, I'll use it and push it very soon.

@adamchainz I pushed the changes with the tests this time. There are multiple commits towards the end on fixing whitespace. Let me know if I should send another PR with the only one commit and the changes.


Is pi/radian analysis possible?

Is it possible to know (or guess and suggest) when Angular_velocity or Frequency might be more appropriate to use?  Both on the way in and the way out of equations?

v1.15m


Native Notebooks - Shared Variable Explorer for Partners

Today in VS Code, the Jupyter, Julia, and .NET extensions are all building notebook experiences. In an effort to have consistent notebook experiences for users regardless of the language they use and to reduce duplication efforts moving forward for any other language partners, it would be great if all partners could leverage the same variable explorer.
After meeting with the Julia team (@davidanthoff and @pfitzseb) and .NET Interactive (@LadyNaggaga) below is a summary of the current state of the variable explorer experiences and what each extension enables for their users.
Jupyter:

Variable Explorer is a table rendered as a webview that is shown in the bottom panel by default. This panel can be moved to the side panel but this is not a great experience as it was designed in a "horizontal" orientation. In order for users to see the entire table they would have to enlarge the side panel a lot and then would end up covering the majority of the notebook which would defeat the purpose of opening the variable explorer and looking at their code.

Displays:

Name
Type
Size
Value

Functionality;

Sort by name
Sort by type
Entry point for data viewer

Julia:

Variable Explorer shown as tree view (copied from debugger view) in side panel
Displays:

Name
Icons (same that autocompletion uses) to represent type
Value

Functionality:

Entry point to table viewer (similar to Jupyter data viewer)
Entry point to plot navigator (similar to Jupyter plot viewer)
Can drill down into the variable and see the internal structure
Kernel Management: Since all notebooks share the same tree view, top level nodes are what help users differentiate what notebook the variables are coming from. Because of that, Julia ext allows users to kill/restart kernels from these top level nodes. (To be kept in mind when exploring the kernel management tab in the notebook panel)

Special Notes:

Thinks the variable viewer works well in the bottom panel for notebooks but tree view is crucial for their customers. Julia team brought up the table/tree view in Visual Studio and thought that could be an ideal solution UI wise that would work well for both their REPL and notebook story.
Asked about potentially running an A/B test to understand what would be best for users here
Mentioned that since size does not apply to all variables, this information could potentially be shown on hover?

.NET:

✋ Variable Explorer not functional today
Would require same functionality as Julia team
Since the .NET Interactive kernel supports multiple languages, it would be critical for users to see also have a 'Language' column.

Summary/Pending Questions:

Jupyter, Julia and .NET need the surfacing of the variable name, type, value. What is best way to do this given today's UI differences?
.NET needs ability to surface 'Language' as well.
How important is size given that it doesn't apply to all variables? Does it make sense to show on hover?
How can we support drill down and sorting capabilities?
Can we support the table/tree view similar to what Visual Studio has to have the best of both worlds?
If not, should we run an A/B test on different versions to understand whether a tree view/horizontally orientated table is best for users?
How do other requests we have gotten for variable explorer fit into this? (Pinning variables, change variable values inline, etc)
Given Julia uses this area to surface kernel management, we should see how this fits with our exploration of the kernel management panel

Since the .NET Interactive kernel supports multiple languages, it would be critical for users to see also have a 'Language' column.

The subkernels in .NET Interactive aren't necessarily just languages. For example, you can have two different SQL kernels loaded that represent different connections, and showing the connection name would be useful as well, e.g. SQL - AdventureWorks and SQL - Northwind. So a more accurate name might be "kernel" but that would also be a little ambiguous.

Didn't realize we had this issue already. We can continue in https://github.com/microsoft/vscode/issues/165445 and https://github.com/microsoft/vscode/issues/166296


Add in reloader container to manager pod with options to disable.

This fixes #953. Please test on the branch before we merge to see if it makes sense. Also, this still needs documentation.
What we're doing is adding https://github.com/stakater/Reloader as a container to the pod, and overriding the annotation names:
        - --auto-annotation
        - reloader.kudo.dev/auto
        - --configmap-annotation
        - configmap.reloader.kudo.dev/reload
        - --secret-annotation
        - secret.reloader.kudo.dev/reload

This directly corresponds to the same annotations over in https://github.com/stakater/Reloader. Mostly, you'll likely want to use reloader.kudo.dev/auto.

I tested this with reloader.kudo.dev/auto: "true" and got this working
after an update using kudo
k kudo update --instance=kafka-zntnds -p MESSAGE_MAX_BYTES=100000
time="2019-10-25T10:38:52Z" level=info msg="Changes detected in 'kafka-zntnds-serverproperties' of type 'CONFIGMAP' in namespace 'default'"
time="2019-10-25T10:38:52Z" level=info msg="Updated 'kafka-zntnds-kafka' of type 'StatefulSet' in namespace 'default'"
time="2019-10-25T10:41:48Z" level=info msg="Changes detected in 'kafka-zntnds-serverproperties' of type 'CONFIGMAP' in namespace 'default'"
time="2019-10-25T10:41:48Z" level=info msg="Updated 'kafka-zntnds-kafka' of type 'StatefulSet' in namespace 'default'"

has some issues initially caused by RBAC as the the controller I started was in default ns and kudo-manager ClusterRoleBinding was only for kudo-system ns. Once fixed that I was able to get it working.
- kind: ServiceAccount
  name: kudo-manager
  namespace: kudo-system

I guess this won't happen once merged as we won't have two controllers running, one without manager and other with reloader only

@zmalik That sounds like a regression from one of our hacktoberfest PRs, I'll check with @alenkacz since I know we were talking about what to do with our namespace being default...

not sure why linting isn't triggering, maybe it's just the check is expected but circle won't run it now since this is an older PR....anyway, I made sure make lint passes locally.

not sure why linting isn't triggering, maybe it's just the check is expected but circle won't run it now since this is an older PR....anyway, I made sure make lint passes locally.

that's because I renamed the check-formatting -> lint and made the check required. rebasing/merging master  would've helped too.

i did that at the end... weird that it didn't take

oh, I see, it kept check-formatting anyway. oh well, github weirdness.


Python dedicated documentation

Hi guys! I really like Qdrant and I'm using on daily basics, I wonder if it's possible to start building a dedicated python documentation using mkdocs for example.
I can help on that and start working on the first implementation following the Redoc and Qdrant official documentation.
What do you think? Someone already built something?
Thank you!

@JeyDi thanks for pointing this out. New documentation is coming. I will share an mvp version here.

I'll be glad to help you guys on the documentation if necessary :)

There is python dedicated documentation
It's work in progress though and will be improved later
Closing this issue for now in favour of more precise issues like #355 #354 #348


Reset site statistics

A small but nice quality of life improvement would be to able to reset a websites statistics data.
While yes you can simply delete the site and re-add it to Umami, this will generate a new website ID and require an update and redeployment of the website in question 🙂

A small but nice quality of life improvement would be to able to reset a websites statistics data.
While yes you can simply delete the site and re-add it to Umami, this will generate a new website ID and require an update and redeployment of the website in question 🙂

A simple workaround for this can be to truncate DB tables.

A simple workaround for this can be to truncate DB tables.

Unless you have multiple sites and you only want to clear data for one of them
Also, it would be great to have an option to clear old data only. For example:

Clear data for the following period:

All data
Older than X days

Another option that I would love is the ability for umami to auto-clear data after a week/month/other timeframe and only keep the recent amount of time.  I primarily use umami for its real-time/short-term analytics and have no need for anything longer than that, therefore, would like to save space/database overhead in the long run with something like this.

A simple workaround for this can be to truncate DB tables.

Unless you have multiple sites and you only want to clear data for one of them

A DELETE FROM tableWHEREwebsite_id="your-id-here" would do the job for just one domain, where you would delete from all the tables you want to clear.  You could even get really fancy and JOIN with the website table to make comparisons to domain, avoiding the extra step of finding the website ID.

Available in v1.22.0


Draft proposal to add Zcash (ZEC) to the Celo Reserve

I believe that this will promote engagement between the Zcash and Celo communities, which are ideologically very much aligned.  Down the track I hope to encourage development of automated, trustless bridges between the two protocols, which would enable automated, transparent management of the Zcash component of the reserve.  I also believe diversifying the Celo Reserve with ZEC will help to mitigate the extent to which the reserve is presently exposed to Ethereum risk.

@martinvol PR opened per your request in Discord.  Keen to get your comments. Thanks!

@alchemydc sorry it took me so long to reply!

@alchemydc from James feedback, please create a discussions link, either here on Github or on the Forum, then link to it here in the CGP.
In the preamble, you can just add:
Discussions To: <URL TO DISCUSSIONS>


Laravel 6 support

Fixes #155

Would love this to be merged!

Please merge and increment version. This is all that's needed to support Laravel 6. Surprisingly simple :)

Is this likely to get merged?

Hurra!


Add example for YOLOv2 Inference on a input image.

Tested On:
[ X ] Windows
[  ] Linux Ubuntu
[  ] Mac
[  ] Others  (State here -> xxx )
Add example for YOLOv2 Inference on a input image.

added instructions for three 3 object det. labs, image,video and webcam. @kenghooi-teoh


Secret Scope limit is at the incorrect scope

Hi
The secret scopes limit states the limit is 100 secret scopes per Account, I believe this is incorrect and the limit is 100 per workspace.

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

ID: 8cc01d52-28cb-be0b-2c38-ddaaa3cf481a
Version Independent ID: 05b74821-f013-5583-0286-48b286761ea1
Content: Limits - Azure Databricks
Content Source: databricks/resources/limits.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@mssaperla Could you please review the above feedback from customer and help update the document as needed. Thank you

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue and apologize for the delayed response. The requested updates have not yet been made. The timeline for resolution varies based on resourcing. We've created an internal work item (DOC-7158) to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close


"Internet service providers" are scored the same as "hosting providers"

It seems that the current system treats "Internet service providers" the same as "hosting providers".  But I assume that is not the intention.  For example 10 nodes in ten different cities could all be "self-hosting" using "Verizon Business" as an ISP and be penalized for using the same "provider".  Perhaps the location name should be tacked onto the end of ISP names.  For example "Verizon Business-New York City" or "Verizon Business-Scranton".

I personally think it makes sense to differentiate. Also, any "hosting provider" will hopefully be using multiple ISPs.
But given this example, what if "Verizon Business" decides to ban any crypto-related traffic? Just one hypothetical example of how ISP diversification can be crucial.

If Verizon decides to ban crypto traffic, that will likely take out half of the hosting providers in North America.
If hosting provider's Internet service providers aren't scored, then Internet service providers shouldn't be scored as a whole for "self-hosting" nodes.
Other thoughts: Even if a self-hosted validator got Dedicated Internet Access in an area with "Verizon Business" the DIA could still be though Verizon.  So the "organization" attached to node's IP would change but the underling ISP would still be Verizon.

If Verizon decides to ban crypto traffic, that will likely take out half of the nodes at hosting providers in North America.

I think that's a good reason to not consider many locations as different when they share the underlying company. I believe your proposal would remove that distinction and treat the "Verizon Business" nodes as diverse, when they share a common single point of failure.

If hosting provider's Internet service providers aren't scored, then Internet service providers shouldn't be scored as a whole for "self-hosting" nodes.

It seems more difficult to track, but if a hosting provider has redundant ISPs - any single ISP seems less relevant to overall availability. If you only have one ISP, that particular ISP is far more significant in terms of availability.

Other thoughts: Even if a self-hosted validator got Dedicated Internet Access in an area with "Verizon Business" the DIA could still be though Verizon. So the "organization" attached to node's IP would change but the underling ISP would still be Verizon.

I think you can still determine this, but I'm not sure how robust the application is that checks it.

The point I'm trying to make is the current "provider" scoring system does not take ISP into account. The purpose of this issue is to simply align "ISP names" with the current "provider" scoring system logic.
If you are suggesting ISP should be taken into account in scoring that probably is a separate issue.

I believe your suggestion will make validator nodes falsely appear more diverse/reliable/etc while sharing single points of failure/vulnerabilities. I'm simply responding to your idea and sharing my concerns.

Closing this as it's likely outdated, if it's still an issue feel free to reopen


[battery_plus] add "unknown" status (#61)

Added "unknown" battery state for batteryless systems (e.g. workstations).
Closes: #61

thanks! :slightly_smiling_face:

Curious about the fall through (ie lack of break statements) behavior on Android (and iOS) for the unknown state. @jpnurmi are those states intentionally falling through for some reason, or should we add break statements there?

@ChristianEdwardPadilla Good catch, thanks! Could you push a fix?

Will do, thanks!


Fix tf.pow(x, y) edge case with integer x and negative integer y

This fix tries to address the issue raised in #12156 and #9560 (and PR #11852) where pow(x, y) hangs with an integer x and a negative value of y.
This fix tries to throw out an error like numpy in this case:
>>> np.power([5, 5], [2, -2])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Integers to negative integer powers are not allowed.

This fix adds error to the C++ functor like safe div/mod so that and InvalidArgument error could be triggered if any one of the values of y is negative.
NOTE: Similar to safe_div/mod this fix also does not cover GPU (not working yet).
This fix fix #12156. This fix is also related to #9560.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

I'd rather we fix this upstream in Eigen, as @codrut3 commented on #9560 ?


not attaching to X Rebirth

I've put the contents of "remix-0.1.1" folder next to the game executable and can not see any splash message on top fo the screen when lounching a game
0.1.1
rtx 3060 ti
527.56
4.30 (254587) Steam

Have no idea where to get logs:

NvRemixBridge.log
d3d9.log
The game specific log will be named gamename_d3d9.log

Steps to reproduce the behavior:

Do everything according to user guide for steam version of X Rebirth
See game is working without any signs of RTX remix enabled.

Expected behavior
Game is DX9 as far as I know. And it wokrs perfectly well with Reshade 5.1.0. So I'm expecting it to work with this game.

Boost, there is (as far as i can find) 0 documentation on what to do if rtx remix doesnt work on start up. It doesnt even generate a log file.

You might have to put the files somewhere else, are there any other folders containing DLL files?

Hello! Will you please test this on the latest release and let us know if the issue persists? Thank you!


Typescript : Uncaught TypeError: Cannot read property '$isServer' of undefined

I success declare the js plugin with *.d.ts, but after I do .use(checkView), my view is blank, did this library support on Typescript?
main.ts file
import checkView from 'vue-check-view'
....
createApp(App).use(store).use(router).use(checkView).component('font-awesome-icon', FontAwesomeIcon).mount('#app')
*.d.ts file
declare module "vue-check-view"; any;

Same error here, without typescript.

Any solution for this?

Same error


Implementing the memmove() unittests

Just a question before submitting a PR for the above. Should we implement these as:

do nothing - as the memmove implementation = memcpy and memcpy has a unittest should we just remove the bullet that says that memmove needs a unit test? A comment that these tests are tested by memcpy would be useful I guess.
make the memcpy unittest public and reference it from the memmove one?
do a copy/pase of the memcpy unit test to the memmove one?

Thanks!

I would prefer this:

do a copy/pase of the memcpy unit test to the memmove one?

As the initial implementation. It is an implementation detail that memmove uses memcpy under the hood, and they have slightly different contracts
Memmove also (should) allow the source/destination memory to overlap, and there is likely the need for some tests in that area.

Also, thank you :)


Unrelated Tags in Card Selection Modal since version 1.12.0

Describe the bug
With version 1.12.0, unrelated tags are shown as well, which makes the whole appearance a bit messy. E.g. my tag for marking flashcards is "#flashcard", but sometimes there are a couple other unrelated meta-tags added to the note, like date tags and so on. Now the UI is clustered with all unrelated tags as well, not only the #flashcard tag and its sub-tags (#flashcard/sub1).
To Reproduce
Open card selection modal.
Expected behavior
Only the configured tag (in my case #flashcards), and its sub-tags should be listed.
Screenshots

Versions (please complete the following information):

OS: macOs Sonoma 14.3.1
Obsidian version: 1.5.11
Plugin version: 1.12.0

Additional context
data.json
What is a heap.md

Hi @ikelos555
Sorry about that, I've fixed the problem and added a unit test case.
Could you please use this beta version.
There is a file called main.js in this zip file
main_908A.zip
You will need to copy that to the obsidian plug-in folder, e.g.
D:\Obsidian\Obsidian\.obsidian\plugins\obsidian-spaced-repetition
Then reload the plug-in, or restart obsidian.
Any feedback appreciated.
Ronny

Thanks for the quick fix - tested it and works perfect!
Best
Marco


File Size data type to use accurate unit symbols

Description
Without abbreviations enabled a 10 kilobyte file is displayed as "10 KB" when it should be "10 kB" when using the SI definition.
With abbreviations enabled, arguably incorrectly displayed as "10kb" (i.e. kilobits).
Device
Windows

Yeah, I figured I was butchering the SI units. A person would have to be pretty pedantic to think I was measuring file size in kilobits, but I'm all for accuracy. Stay tuned.

Fixed in v2.25.1.

Pedantic? I don't know who you're taking about.
Thanks!


refactor: add validation to connected services

This change is part of the following stack:

#340

#325

#326

#341 ◀

#342

Change managed by git-spice.

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

7 of 8 (87.5%)  changed or added relevant lines in 1 file are covered.
6 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.02%) to 90.446%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

components/renku_data_services/connected_services/blueprints.py
7
8
87.5%

Files with Coverage Reduction
New Missed Lines
%

components/renku_data_services/crc/db.py
1
87.58%

components/renku_data_services/storage/blueprints.py
1
94.65%

components/renku_data_services/users/db.py
1
94.14%

components/renku_data_services/secrets/db.py
3
93.07%

Totals

Change from base Build<PHONE_NUMBER>5:
0.02%

Covered Lines:
9060

Relevant Lines:
10017

💛  - Coveralls


Details view Tag column

What's the Problem?
need to colour files or folders
Solution/Idea
in macOS the Finder has a Tag column, in the contextual menu of a selected file or folder the user can select from several colours this would in the olden days colour the background of the row for the time, now days they just put a coloured dot in the tag column, not ideal but at least we can visually colour items of importance.
Critical to a project id tag them red, amber or currently working on in green...
Alternatives
some third party tools with add an invisible folder icon file, but this aint ideal if the folder or file is on a network share, it colours the folders for everyone, kinds pissed people off init.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
v1.5
Windows Version
Windows 11 Pro 21H2 22000.194
Additional comment
No response

Tags were implemented in Files v2 💪


Add support for customizable rootFs

Initial change for adding support for customizable rootFs in containerd

i based my change on the containerd/master so it brought all the other changes with it. closing this and resending a new change based on this repo


Overwrite the Upgrade Frequency and Delay for Testnet

instant upgrade should just be an alteration in the relay chain config
which i would hope is set in the genesis spec
there's two fields which should both be set to 1 for testnets where security isn't an issue
validationUpgradeFrequency: 1,
validationUpgradeDelay: 1,

closed by: https://github.com/paritytech/polkadot-launch/commit/94f6ed769f66d72ae20c182324f75c93543298da


Update middleware.py

Debug missing import
@alecxe sorry, I miss one import

No worries, I've added it - it's in master and a part of 1.2.0. Thanks


Backup logging

Plugin version
1.2.4
NetBox version
3.2
Feature type
New functionality
Proposed functionality
I suggest adding backup logging. That is, adding a log file on the server where information about how the backup was made will be written. With different log levels (Warning, Debug, Info, etc.)
Use case
This will help a lot in debugging problems. For example, with 200 devices it will be very difficult to debug only on the basis of the web.
Database changes
No response
External dependencies
No response

There is already some logging in place, you just need to enable the logging.