Sound mixer settings keeps resetting

Describe the bug
I used MicroWin to install a debloated version of Windows 11. With this version, my sound mixer does not save the changes when I adjust the volume (e.g., system sounds, Discord, etc.). I've seen others with the same issue using MicroWin. Applying the fix mentioned in issue #2747 helps save the volume adjustments after closing the sound window or restarting the PC. However, if I receive a notification from any application, the sound settings reset.
To Reproduce
Steps to reproduce the behavior:

Debloat Windows 11 with MicroWin.
Install the debloated Windows 11.
Let Windows download and install all updates.
Restart your PC as many times as needed for the updates.
Adjust the system sound in the sound mixer.
Close the window.
Open the sound mixer window again (at this point, the settings should already be reset because the fix hasn't been applied yet).
Apply the fix mentioned (this should work after closing the sound window and restarting the PC).
Trigger any notification from Windows (the sound settings should reset again).
Open the sound mixer and check that all changes have been reset.

Expected behavior
Sound adjustments should been saved.
Screenshots

Additional context

I did some more research on this bug. It only occurs with notifications from third-party applications. Turning off the notification sound "fixes" it, but this is not a proper solution. It’s important to note that this issue is not specific to MicroWin, as I tested it on a fresh, untouched Windows installation and encountered the same problem.

I also have this after installing Windows 11 with MicroWin... Really annyoing

Fixed with #2853. /close


Target Node.js 18, fix adding dependencies when no path given

Fixes #22

Targets Node.js 18

Removes Node.js 16 from the test matrix (should Node.js 21 be added?)

Updates dependencies
Adds tests for omitting the optional filePath parameter
Fixes addPackageDependency when not providing a filePath

I wasn't able to reproduce #23.

I don't see the need to make this a breaking change. I generally try to only do major releases when there are actually breaking changes or it's important to update a dependency that requires it. So let's leave out targeting Node.js 18 for now.

should Node.js 21 be added?

👍


composition of dockerfiles

It would be good to have composition of dockerfiles, something like an include keyword that would act like a sh "source", importing an image's files and it's configuration, collapsed to 1 layer
e.g
#jboss.dockerfile
FROM scratch
ADD jboss /opt/
ADD secret /secret
VOLUME /logs
ONBUILD RUN git clone<EMAIL_ADDRESS>/src/
ONBUILD RUN something /secret # build it commands
ONBUILD RUN rm -rf /src/
ONBUILD RUN rm -rf /secret

# fronendFoo.dockerfile
FROM centos
INCLUDE include:jboss
INCLUDE include:memcached
ADD file /files
RUN ...
CMD ...

cd $jboss.dir && docker build -t include:jboss -f jboss.dockerfile .
cd $frontendFoo && docker build -f fronendFoo.dockerfile .

The onbuild is run in the "outside" image (e.g. fronendFoo.dockerfile)
something like this would
a) reduce the complexity of ignore files and project layout, in the project{a,b,c} + common pattern, each included image is created in it's own context
b) enable some secret(s) and temp file erasing, they are not present in the final image
c) keep the history of how the final image was created, by having the included image's name and ID is in the history
d) layer reducing, everything in the included image is reduced to one layer
e) cache friendly and will download the sub images from a registry
It's probably been discussed before but i can't find the issue, anyway here is poc PR to kick the proverbial tyres

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

https://github.com/docker/docker/pull/12749
-Doug
Sent from my iPhone

On Jun 23, 2015, at 8:05 AM, Sam Abed<EMAIL_ADDRESS>wrote:
It would be good to have composition of dockerfiles, something like an include keyword that would act like a sh "source", importing an image's files and it's configuration, collapsed to 1 layer
e.g
#jboss.dockerfile
FROM scratch
ADD jboss /opt/
ADD secret /secret
VOLUME /logs
ONBUILD RUN git clone<EMAIL_ADDRESS>/src/
ONBUILD RUN something /secret # build it commands
ONBUILD RUN rm -rf /src/
ONBUILD RUN rm -rf /secret
fronendFoo.dockerfile
FROM centos
INCLUDE include:jboss
INCLUDE include:memcached
ADD file /files
RUN ...
CMD ...
cd $jboss.dir && docker build -t include:jboss -f jboss.dockerfile .
cd $frontendFoo && docker build -f fronendFoo.dockerfile .
The onbuild is run in the "outside" image (e.g. fronendFoo.dockerfile)
something like this would
a) reduce the complexity of ignore files and project layout, in the project{a,b,c} + common pattern, each included image is created in it's own context
b) enable some secret(s) and temp file erasing, they are not present in the final image
c) keep the history of how the final image was created, by having the included image's name and ID is in the history
d) layer reducing, everything in the included image is reduced to one layer
e) cache friendly and will download the sub images from a registry
It's probably been discussed before but i can't find the issue, anyway here is poc PR to kick the proverbial tyres
—
Reply to this email directly or view it on GitHub.

Closing for it is a duplicate of #735


BUG: CAGRA multi-cta illegal access with bad queries

CAGRA search kernel errors with cudaErrorIllegalAddress in some conditions, specified in the new test case.
According to compute-sanitizer, the illegal access happens in compute_distance_to_child_nodes(...) function accessing the graph.
The parent_id variable in that function sometimes appears to be out-of-bounds (larger than the graph size / number of records); it's invalid and seems to be the same reported by multiple threads, yet it's not an invalid_index, neither index_msb_1_mask, neither any derivative of the two.
Further observations:

I've checked the graph just before calling the search kernel; it does not contain any invalid indices.
One should disable any fancy pool memory resources to make it easier to reproduce the error (so that parent_id does not hit other user allocations in the pool)
It seems important that the query yields infinite distance to the dataset.
Running the search with a newly created raft::resources seems to increase the chance to hit the error
Even with all conditions satisfied, the error does not reproduce every time...

Reproducer
./build.sh -n tests --limit-tests=NEIGHBORS_ANN_CAGRA_TEST && ./cpp/build/gtests/NEIGHBORS_ANN_CAGRA_TEST --gtest_filter=AnnCagraBugMultiCTACrash*

CC @enp1s0

@achirkin The out-of-range index seems to slip into the internal topk list in the random node pickup step. Can you change the following line and run the test again?
https://github.com/achirkin/cuvs/blob/e7496886792f7f409ce54d6015417c39604fa9f0/cpp/src/neighbors/detail/cagra/device_common.hpp#L148
- if (hashmap::insert(visited_hash_ptr, hash_bitlen, best_index_team_local)) {
+ if (best_index_team_local < dataset_desc.size && hashmap::insert(visited_hash_ptr, hash_bitlen, best_index_team_local)) {

The problem is that result_distances_ptr[i] is set to an uninitialized variable best_norm2_team_local if the norm is inf or nan. An alternative solution is to add std::isfinite(best_norm2_team_local) instead of best_index_team_local < dataset_desc.size.

Many thanks, @enp1s0, for finding the root cause of the problem!
The solution indeed works, though I slightly modified it:

Initialize the  best_index_team_local with the invalid index value on the declaration site <- this should already be enough
Check for the index value <- just to avoid costly hashmap check.

I think it's better to check the index than the distance, because the distance can be infinite on valid inputs. Also it's better not to touch dataset_desc without need, because it's in shared memory (not in registers) in the current implementation.

/merge


Missing projectId after upgrading to 0.16.0

Environment details

OS: Linux (running on GCE instance)
Node.js version: 8.9.4
npm version: 5.6.0
@google-cloud/common version: 0.16.0

Steps to reproduce
I updated the version of google-cloud/common to 0.16.0 in google-cloud/profiler (repo here). Then when running the profiler on a GCE instance without specifying the projectId, I got this error message:
Error: Sorry, we cannot connect to Cloud Services without a project ID. You may specify one with an environment variable named "GCLOUD_PROJECT". See https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/authentication for a detailed guide on creating an authenticated connection.
I switched the version back to 0.15.1 of google-cloud/common and no longer saw this error.
I'm not sure if this is an intended change with the latest release, but it felt surprising to me.

/cc @JustinBeckwith

Did the 0.16.0 release have a version bump of the google-auth-library?  From version x? to y?

It looks like google-auto-auth stayed on google-auth-library 0.12.0 through this release, so I'm not sure if this was a change down there that started causing it.

From 0.15.1 to 0.16, we went from using<EMAIL_ADDRESS>to ^0.9. In that span, google-auto-auth only had one commit: https://github.com/stephenplusplus/google-auto-auth/commit/423cf05cd8f26033b577468abc41c3dd44059e14
It looks like this is what is eventually given to google-auto-auth: https://github.com/GoogleCloudPlatform/cloud-profiler-nodejs/blob/4417ff5ba7e33db788aafb2cb6906b0af6855718/ts/src/index.ts#L59-- what value is that on GCE? If it's a project ID, g-a-a should use that. If it's nothing, g-a-a tries to get the project ID from google-auth-library in the same way it did in 0.8. So, I'm not able to see yet the trouble that linked commit causes.
And just to confirm, 0.15.1 works? Could your local install have pulled from cache and got 0.15.0? Because there was a much bigger change between 0.15.0 and 0.15.1, when we went from<EMAIL_ADDRESS>to 0.8. In 0.8, we started using a new method from google-auth-library, "getCredentials()". We have another issue where that method might be the problem: https://github.com/googleapis/nodejs-logging/issues/53#issuecomment-360172285:

@ofrobots I would guess that this problem relates to google-auto-auth and/or google-auth-library. In @google-cloud/logging 1.1.1, we used<EMAIL_ADDRESS>In 1.1.2 (when the problem originates), we use<EMAIL_ADDRESS>In that release span, g-a-a started using google-auth-library's "getCredentials" method. Is it possible that method doesn't support GKE environments?

Just confirmed -- 0.15.1 works, and 0.16.0 does not.

This is the diff from my package-lock.json when upgrading from 0.15.1 to 0.16.0:
 "dependencies": {
     "@google-cloud/common": {
-      "version": "0.15.1",
-      "resolved"<EMAIL_ADDRESS>-      "integrity": "sha512-cnVtHLvyiSQvb1RzXWDp7PA1sA8Jmc47+wp/xwHwdGOlQZfKog5iluZ0C/LB8iklFXpcTwlNMorqLu
Z/qH0DDA==",
+      "version": "0.16.0",
+      "resolved"<EMAIL_ADDRESS>+      "integrity": "sha512-ApY0Qcmjt48i1SyScskKzU/VGYVtBzWD98ylQvmw/GjRwPu/fB/3Z2WX5MCdgV9/EjgjNBN3hVDraQ
bzRe0rHg==",
       "requires": {
         "array-uniq": "1.0.3",
         "arrify": "1.0.1",
@@ -16,7 +16,7 @@
         "duplexify": "3.5.3",
         "ent": "2.2.0",
         "extend": "3.0.1",
-        "google-auto-auth": "0.8.2",
+        "google-auto-auth": "0.9.3",
         "is": "3.2.1",
         "log-driver": "1.2.5",
         "methmeth": "1.1.0",
@@ -1121,25 +1121,14 @@
       }
     },
     "google-auto-auth": {
-      "version": "0.8.2",
-      "resolved": "https://registry.npmjs.org/google-auto-auth/-/google-auto-auth-0.8.2.tgz",
-      "integrity": "sha512-W91J1paFbyG45gpDWdTu9tKDxbiTDWYkOAxytNVF4oHVVgTCBV/8+lWdjj/6ldjN3eb+sEd9PKJBjm0kmCxvcw==",
+      "version": "0.9.3",
+      "resolved": "https://registry.npmjs.org/google-auto-auth/-/google-auto-auth-0.9.3.tgz",
+      "integrity": "sha512-TbOZZs0WJOolrRmdQLK5qmWdOJQFG1oPnxcIBbAwL7XCWcv3XgZ9gHJ6W4byrdEZT8TahNFgMfkHd73mqxM9dw==",
       "requires": {
         "async": "2.6.0",
-        "gcp-metadata": "0.3.1",
+        "gcp-metadata": "0.4.1",
         "google-auth-library": "0.12.0",
         "request": "2.83.0"
-      },
-      "dependencies": {
-        "gcp-metadata": {
-          "version": "0.3.1",
-          "resolved": "https://registry.npmjs.org/gcp-metadata/-/gcp-metadata-0.3.1.tgz",
-          "integrity": "sha512-5kJPX/RXuqoLmHiOOgkSDk/LI0QaXpEvZ3pvQP4ifjGGDKZKVSOjL/GcDjXA5kLxppFCOjmmsu0Uoop9d1upaQ==",
-          "requires": {
-            "extend": "3.0.1",
-            "retry-request": "3.3.1"
-          }
-        }
       }
     },

Thank you for that! Is it possible for you to show a small script I can run on GCE to see the error? I'm not very familiar with the profiling library.

This is the script I'm running:
const profiler = require(@google-cloud/profiler).start({
  serviceContext: {
    service: 'service',
  }
}

function busy() {
   setTimeout(busy, 1000)
}

busy()

Reproduced and found a fix. PR coming shortly.

@stephenplusplus is this in google-auto-auth, or google-auth-library?

PR sent #32


Uncaught exception by map looku

Consider the following definition:
module TEST
  syntax Exp ::= Id | Int
  syntax Stmt ::= Exp ";" [strict]

  configuration
    <k> $PGM:Stmt </k>
    <state> $ST:Map </state>

  rule <k> X:Id => St[X] ...</k>
       <state> St </state>
endmodule

and the file ex.test including x;. If we run the above definition with the empty map assigned to ST then we get an  Uncaught exception error message:
$ krun ex.test -cST=".Map"
[Error] Internal: Uncaught exception thrown of type AssertionError.
Please file a bug report at https://github.com/kframework/k/issues

This error is due to the lookup operation over maps. This can be easily observed if we run the example with a valid map:
$ krun ex.test -cST="x |-> 2"
<k>
    2 ~> (HOLE ;)
</k>
<state>
    x |-> 2
</state>

The same is true for the update operation over maps.

This seems to be intricately tied with the way lookups happen in KItoBackendJavaKILTransformer, which is likely to change in the new implementation. So I am going to make sure this works in the new KORE, but not fix it for the current version.

This seems to work as expected in the new pipeline.  But there's no assertion error and if you add syntax KResult ::= Int then it will heat and cool correctly. I am going to close this now.

Actually never mind I just retested and we are getting a different error. Namely, it is throwing an AssertionError because the MapLookup is returning Bottom and the rewriter doesn't know how to handle this. I am going to assign this to @andreistefanescu . Andrei, obviously there should be some kind of friendly error message here.


How do i install battery for it?

https://github.com/Fr4nkFletcher/ESP32-Marauder-Cheap-Yellow-Display/issues?q=is%3Aissue+is%3Aclosed+battery


Remove the "Reviewable" Webhook from Openstack-Helm

Kubernetes Version (output of kubectl version): N/A
Helm Client and Tiller Versions (output of helm version): N/A
Development or Deployment Environment?: N/A
Release Tag or Master: N/A
Expected Behavior: N/A
What Actually Happened: N/A
How to Reproduce the Issue (as minimally as possible): N/A
Any Additional Comments:
Disabling the "Reviewable" webhook on this repo to leverage the GitHub review process. We may attempt to leverage something like Gerrit for GitHub at some point to bridge the Openstack review process...but this is undecided and we want this to be a community decision. We will put this on the agenda for the next community meeting.

4 people affirm that Reviewable is out!

done.


[PerfStress] Implement the random stream utility

Implement a class or function that returns a buffer that contains random values.
Here's Mike Harder's implementation: https://github.com/mikeharder/azure-sdk-perf/blob/master/net/Azure.Test.PerfStress/RandomStream.cs

Closing this for this same reason: https://github.com/Azure/azure-sdk-for-js/issues/8057#issuecomment-617978932


DE32879: Remove the hover class from a tile when animation completes.…

… Fixes the z-location of animated tiles, which were holding onto hover unnecessarily.

Although unpin was removed when animation was complete in a previous commit, we also need to remove hover to restore the correct z index to the animated tiles.

I may have messed up my BSI build, but I'm still seeing this as an issue.  Let me try to rebuild

I'm seeing the hover effect be removed from the new tile sliding into place, but if I hover back over it and move away, it stays and never turns off.

I'm seeing the hover effect be removed from the new tile sliding into place, but if I hover back over it and move away, it stays and never turns off.

Well that's charming. I'll look into it.


Fixed issue 1414

Fix issue #1414 in db/remote/db.go

Closing due to inactivity, feel free to reopen whenever


build: add automatic summary to PRs

Summary of changes
Acceptance Checklist
What types of changes does your code introduce?
Put an x in the boxes that apply
Dependencies

[ ] Any dependent changes have been merged and published in downstream modules
[ ] Any teams which manage the dependencies have been notified of the breaking changes

Bug (non-breaking change which fixes an issue):

[ ] Have you written tests to protect against future occurrences of the bug?

Feature (non-breaking change which adds functionality)

[ ] Have you seen it working in a development environment?
[ ] Have you written tests for happy and unhappy paths?
[ ] Have you implemented this feature as per the issue acceptance criteria?

Substrate
If this change involves substrate, have you remembered:

[ ] Storage Migration?
[ ] RPC methods?
[ ] Chainspec, both JSON specs & the module?
[ ] Runtime versioning?
[ ] Benchmarks?

Breaking change (a feature that would cause existing functionality not to work as expected)

[ ] Is the breaking change documented?
[ ] Are your commits fully representative of the change?
[ ] Has any previous functionality been deprecated and versioned?

CI/CD

[ ] If introducing new actions, have you looked at the action code for anything spurious?
[ ] Have you written custom scripts for things that could be actions already on the marketplace?
[ ] If introducing new actions, have you pegged a static version?

Documentation Update

[ ] Have you checked other documentation, such as the docs repository?
[ ] If updating script documentation, have you tested it on both environments?

Further comments
Feel free to explain in further detail your choices if this is a significant change.
Screenshots (Please demonstrate this working in PolkadotJS)

Summary by OpenAI
Release Notes:

Chore: Added a new GitHub Actions workflow.

"A new workflow has arrived,
No code changes, no need to strive.
A chore it may be,
But it brings us glee."

@openai why didnt you comment lgtm on this pr?


Finish Pool implementation

[ ] Basic Pool implementation
[ ] Documentation
[ ] Make it work for any callable not only Closures (see https://gist.github.com/rintaun/2773168 and https://wiki.php.net/rfc/closurefromcallable)
[ ] release new version

Based on https://gist.github.com/beberlei/4145442
Another option would be to get rid of the class alltogether and force the user to implement the recursion himself, see https://github.com/lstrojny/functional-php/pull/109

As you promote fp, I think it would be cool if you implement tail rec without classes :)

Classes can be used like functions without any hassle. Anonymous functions are even instances of the Closure class in the engine. Do you know of any downside using a class with an `__invoke`` method compared to a more "traditional" anonymous function ?
PHP not being really good at functional programming, I think it is better to make do with what is available instead of dogmatically refuse to use classes.
I saw you implementation on the linked issue, the reason I used a class is because I think using $this for the continuation like I am doing is cleaner than have to pass the function by reference using use. But this is more of a personal preference.
Your implementation has the advantage to work with any callable and not only Closure instances as is the case of mine for now. This is why I am considering using only a function. I need to think this through.
Thanks for your feedback !

Yes I understand you. But I requently use $this inside callables as handle to current object. So that is because I reworked that solution to have no side effects outside wrapper and do not change behavior of wrapped callables. I just shared with you my opition. :)


De-duplicate / combine overlapping records

Need to think carefully about how to do this, but how do we combine data that are available in multiple indices? For instance: GEO record from NCBI GEO itself, or Omics DI, or Mendeley.  Ideally, this would be a single record, but need to figure out how to merge info, resolve conflicts, etc.

For Mendeley:

[ ] Zenodo
[ ] Figshare
[ ] NCBI GEO

Mendeley doesn't expose the data they harvest from other sources, so no need to de-duplicate!


ADDED feature like Object tracking and human detection

@hasmithagunda123 you can review the following changes I have committed.

@hasmithagunda123  could you please add levels and gssoc'24 label in this PR,
Also, assign it to me.

@hasmithagunda123 could you please add the labels

@hasmithagunda123 The score of this PR is not visible in the leaderboard, could you add labels to this pr


Feature: unusualActivityFailPercent

Description

< unusualActivityFailPercent ></ unusualActivityFailPercent > tag for both SAX and simpleXml parsers in datasets.xml
Any integer value from 0 to 100 to tweak the threshold percentage, Default = -1 for sending no mails

Have not really tested this. I think should work fine but let me know if otherwise


Trying to run nvidia gpu example failed to find libdevice.10.bc

codon run gpuEx.py
libdevice.10.bc: error: Could not open input file: No such file or directory
where gpuEx.py is:

import gpu

@gpu.kernel
def hello(a, b, c):
    i = gpu.thread.x
    c[i] = a[i] + b[i]

a = [i for i in range(16)]
b = [2*i for i in range(16)]
c = [0 for _ in range(16)]

hello(a, b, c, grid=1, block=16)
print(c)

OS; Ubuntu 22.04
where libdevice.10.bc is in

ls /usr/lib/cuda/nvvm/libdevice
.  ..  libdevice.10.bc
I also tried adding /usr/lib/cuda/nvvm/libdevice to /etc/ld.so.conf and ran sudo ldconf  and verified directory was in ldconf path
using:
ldconfig -v | g nvvm
...
/usr/lib/cuda/nvvm/libdevice: (from /etc/ld.so.conf:2)

I also tried putting a symbolic link in /usr/lib/

ll /usr/lib/libdevice.10.bc
lrwxrwxrwx 1 root root 44 Jun  6 21:14 /usr/lib/libdevice.10.bc -> /usr/lib/cuda/nvvm/libdevice/libdevice.10.bc

None of the above was successful. Any ideas?

Hi @kwmartin -- can you try passing the path to libdevice.10.bc through the -libdevice flag (i.e. -libdevice /path/to/libdevice.10.bc -- https://docs.exaloop.io/codon/advanced/gpu#libdevice)? Let me know if that works.

Unfortunately no:
codon run -libdevice=/usr/lib/cuda/nvvm/libdevice/libdevice.10.bc tst.codon
JIT session error: Symbols not found: [ seq_nvptx_memcpy_d2h, seq_nvptx_memcpy_h2d, seq_nvptx_device_free, seq_nvptx_function, seq_nvptx_invoke, seq_nvptx_load_module, seq_nvptx_device_alloc ]
Failure value returned from cantFail wrapped call
Failed to materialize symbols: { (main, { std.internal.types.range.range:std.internal.types.range.range.__iter__:1[std.internal.types.range.range].183.resume, std.internal.types.error.LookupError:std.internal.types.error.LookupError.__init__:3[std.internal.types.error.LookupError,str,str].513, ._lambda_136:0[int].271, std.gpu.Memory[byte]:std.gpu.Memory._free:0[std.gpu.Memory[byte]].631, int:int.__mul__:1[int,int].14, std.internal.types.error.IndexError:std.internal.types.err
...

tst.codon:
import gpu

MAX    = 1000  # maximum Mandelbrot iterations
N      = 4096  # width and height of image
pixels = [0 for _ in range(N * N)]

def scale(x, a, b):
    return a + (x/N)*(b - a)

@gpu.kernel
def mandelbrot(pixels):
    idx = (gpu.block.x * gpu.block.dim.x) + gpu.thread.x
    i, j = divmod(idx, N)
    c = complex(scale(j, -2.00, 0.47), scale(i, -1.12, 1.12))
    z = 0j
    iteration = 0

    while abs(z) <= 2 and iteration < MAX:
        z = z**2 + c
        iteration += 1

    pixels[idx] = int(255 * iteration/MAX)

mandelbrot(pixels, grid=(N*N)//1024, block=1024)

Duplicate of #193—will keep the progress updates there.


Lumber detected as TSQL even if SQL is ignored in a branch

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[x] verified an issue has not already been logged for your issue (linguist issues).

Problem Description
The repository https://github.com/ForestAdmin/lumber/ is considered as TSQL. It's because there is a dump in assets/database-examples/movies.
I override the rules here: https://github.com/ForestAdmin/lumber/blob/fix/github-linguist/.gitattributes following the recommandations about overriding. I tried two ways :
assets/* linguist-generated=true

And:
*.sql linguist-detectable=false

But it seems to have no impact. When I go to my branch : https://github.com/ForestAdmin/lumber/tree/fix/github-linguist, it's still indicated as TSQL.
URL of the affected repository: https://github.com/ForestAdmin/lumber/
Last modified on:
2019-10-04
Expected language:
Javascript
Detected language:
TSQL
I hope I respected the guidelines for contributions. Feel free to ask for more info!

I override the rules here: https://github.com/ForestAdmin/lumber/blob/fix/github-linguist/.gitattributes following the recommandations about overriding.

You need to commit your .gitattributes file to your default branch for it to have any effect.

You can see this in action in the fork I made of your repo at https://github.com/lildude/lumber

Ok thanks @lildude 🙏! Still I can't see it in action on your fork (it still displays TSQL):

Did I missed something?

🤔 not sure. I see this:

Looks to be a caching issue. It'll probably take a few mins to propagate out to all the front-end servers. Wait and refresh and you'll eventually see it.

OK. I'm waiting for it! Thanks!
I will close the issue as soon as I don't see it anymore :)

OK, I see it now! Thank you! 👍


Logstash chokes on all-lowercase month in syslog_timestamp

I tried to use the module to log to Logstash and got the dreaded "_grokparsefailure". I noticed that the default grokpatterns in Logstash assumes that the month is capitalized and fails otherwise.
(https://github.com/hpcugent/logstash-patterns/blob/master/files/grok-patterns)
I don't know what the RFCs has to say about it.
Anyway, I patched my local installation and thought I'd inform you of the problem and the solution.

Hi Fredrik,
Thanks for the pull request.
What language/locale settings are you using on the Windows Machine sending the SYSLOG message from? When I run both the before and after code, I am getting the same output.

C:\Users\kiera> $timestamp = Get-Date
C:\Users\kiera> $timestamp

Sunday, 27 March 2016 11:23:59 PM

C:\Users\kiera> (Get-Culture).TextInfo.ToTitleCase($Timestamp.ToString('MMM dd HH:mm:ss'))
Mar 27 23:23:59
C:\Users\kiera> $Timestamp.ToString('MMM dd HH:mm:ss')
Mar 27 23:23:59
C:\Users\kiera>

Reading RFC3164 again, I see:
     The TIMESTAMP field is the local time and is in the format of "Mmm dd hh:mm:ss" (without the quote marks) where:
     Mmm is the English language abbreviation for the month of the
     year with the first character in uppercase and the other two
     characters in lowercase.  The following are the only acceptable
     values:
     Jan, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, Nov, Dec

I will merge your pull request, and release a new version out to the gallery.
Thank you so much for your work on this one, it is very much appeaciated.
Kieran Jacobsen

Hi Kieran.
That’s interesting. My Windows installation is english language/swedish locale. I tried from a couple of different Win 2012 R2 installations with similar results, the month was always all lower case. You got me intrigued, now I have to try this with a different locale! ;)
Thanks,
Fredrik

27 mars 2016 kl. 14:35 skrev Kieran Jacobsen<EMAIL_ADDRESS>Hi Fredrik,
Thanks for the pull request.
What language/locale settings are you using on the Windows Machine sending the SYSLOG message from? When I run both the before and after code, I am getting the same output.
C:\Users\kiera> $timestamp = Get-Date
C:\Users\kiera> $timestamp
Sunday, 27 March 2016 11:23:59 PM
C:\Users\kiera> (Get-Culture).TextInfo.ToTitleCase($Timestamp.ToString('MMM dd HH:mm:ss'))
Mar 27 23:23:59
C:\Users\kiera> $Timestamp.ToString('MMM dd HH:mm:ss')
Mar 27 23:23:59
C:\Users\kiera>
Reading RFC3164 again, I see:
 The TIMESTAMP field is the local time and is in the format of "Mmm dd hh:mm:ss" (without the quote marks) where:
 Mmm is the English language abbreviation for the month of the
 year with the first character in uppercase and the other two
 characters in lowercase.  The following are the only acceptable
 values:
 Jan, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, Nov, Dec

I will merge your pull request, and release a new version out to the gallery.
Thank you so much for your work on this one, it is very much appeaciated.
Kieran Jacobsen
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/poshsecurity/Posh-SYSLOG/pull/2#issuecomment-202054878


Cake whit Goose Egs

It could be a good feature if you added the Goose Egg cake, if we can make whit the duck eggs, why can't we make whit the Goose eggs?

Good point! Probably just an oversight on my part.


Added /join command

Also starts a decent framework for more commands.  Almost closes https://github.com/moose-team/friends/issues/94
Still need to handle # removal

Seems to work actually without any # processing.

Do we support channels with spaces?

I'm fine with part, but /wc is standard IRC too no?

oh /wc is from irssi looks like (/window close), i was always BitchX or mIRC :)
feel free to add both

I gotta run to bed now :[  If this is the general direction you want to take feel free to merge and keep working on it.  I dont want to hold the train up.
See you all tomorrow!

While we're at it, might as well also add an alias for /leave to do the same as /wc.

Right now, channel names can have spaces or, for that matter, it appears any character. Is that a bug or a feature? It does seem that if you are going to type a channel name, white space would be expected as a delimiter.

While we're at it, might as well also add an alias for /leave to do the same as /wc.

yea, and I think we first do the /alias api and signature, then these things would be easy.

It does seem that if you are going to type a channel name, white space would be expected as a delimiter.

atm, yes, but its easy for change

@pokemonmegaman Cool!  self.data.activeChannel.name  sounds promising.  I'll get that fixed up now.

@tunnckoCore  I'm a little on your first coupe comments in the thread.  Mind adding line numbers?

Part/leave/wc works great now.  @pokemonmegaman++ for pointout out that object structure to me.  I should have just looked a few lines above XD

@ngoldman unrecognized commands now just send!  They also log the unrecognized command.

:+1:

looks great, good job! :+1: :)


Update to pharo70

Espell version updated.
This version runs in Pharo70

It seems these commits were already there... This means we should probably generate an image and put it in the releases in this repo. Should we?

Yes we should, unless we decide to move it to a repo under a pharo-project account.
Who has access to that account? After we generate the image we must remember to update the reference in https://github.com/pharo-project/pharo/pull/4558#pullrequestreview-294828411

Yes we should, unless we decide to move it to a repo under a pharo-project account.

It does not really matter. We can transfer the repository in its entirety, with releases, issues and PRs.

Who has access to that account?

I do

After we generate the image we must remember to update the reference in pharo-project/pharo#4558 (review)

Let's do a todo list.
1- The first point you've already done it in your repo, don't you. I can just upload that file into my this repo.
2- I can do the transfer
3- Once I'm done with that, could you make that pull request?

Yes we should, unless we decide to move it to a repo under a pharo-project account.

It does not really matter. We can transfer the repository in its entirety, with releases, issues and PRs.

Who has access to that account?

I do

After we generate the image we must remember to update the reference in pharo-project/pharo#4558 (review)

Let's do a todo list.
1- The first point you've already done it in your repo, don't you. I can just upload that file into my this repo.
Yes, the file is here so you can upload it
2- I can do the transfer
3- Once I'm done with that, could you make that pull request?
Of course, I will be waiting then


[Roadmap] UI design/mockups

Need to start to decide on UI design and possibly get some mockups.

This should be accomplished after styleguide during the development phase as the UI is not necessarily needed right away as tooling is created.

The way I see this occuring is to have a base design paid for once simple things such as colour schemes and logo and stuff sorted out and then tweak it from there based on feedback on github


Ionic react displays white space after installation and debuts Typescript error reports

Ionic react  is not compatible with the android browser kernel webview  and displays white space when accessed.
Android Version：7.0
Google Webview Version：51.0.2704.91
@ionic/react Version：^5.0.7

Thanks for the issue. Stencil, the web component compiler Ionic is built with, only supports Chrome 60+. You will need to bundle certain polyfills in order to get the application to run. There are 2 things we recommend:

Modify the browserslist portion of your package.json to include support for older versions of Chrome. This is useful for indicating which polyfills to bundle with your application. For example, I was able to add >0.2% to the development array in browserslist, and additional polyfills were bundled during a dev build of the app. See https://github.com/browserslist/browserslist for more examples.

There may be polyfills you need even after modifying browserslist. If that is the case, we recommend bundling those polyfills yourself. You can usually find a polyfill for a certain API with a quick search. For example, here is a doc which contains a polyfill for Array.from: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/from.

Hope this helps!


Anko for RecyclerView

Can't we use Anko for the RecyclerView here?:
https://github.com/antoniolg/Kotlin-for-Android-Developers/blob/master/app/src/main/java/com/antonioleiva/weatherapp/activities/MainActivity.kt#L28

Using the Kotlin Android Extension would be even better,

I missed this issue, sorry... Yeah, I didn't use it at that moment because it was a future part of the book and I hadn't explained it yet. This App is now using Kotlin Android Extensions and Anko wherever it helps writing an easier code.

Thanks! :+1:


Install fails if latest commit short hash can be parsed as an int

Capturing this an installation error in an issue as it was a bit tricky to debug.
Error: accumulating resources: accumulation err='accumulating resources from '../manager': '/home/runner/work/modelmesh-serving/modelmesh-serving/config/manager' must resolve to a file': couldn't make target for path '/home/runner/work/modelmesh-serving/modelmesh-serving/config/manager': invalid Kustomization: json: cannot unmarshal number into Go struct field Image.images.newTag of type string
error: no objects passed to apply
Error: Process completed with exit code 1.

Core of the error message:
cannot unmarshal number into Go struct field Image.images.newTag of type string
How to reproduce it?
Assume the latest git commit short hash is composed of numeric characters exclusively, i.e.
    IMAGE_TAG: 7284872

Run:
  export NAMESPACE_SCOPE_MODE=false
  export NAMESPACE_SCOPE_MODE=false
  kubectl create ns modelmesh-serving
  ./scripts/install.sh --namespace modelmesh-serving --fvt --dev-mode-logging

Output:
namespace/modelmesh-serving created
Setting kube context to use namespace: modelmesh-serving
Context "minikube" modified.
Getting ModelMesh Serving configs
Using config directory at root of project.
~/work/modelmesh-serving/modelmesh-serving/config/default ~/work/modelmesh-serving/modelmesh-serving/config
~/work/modelmesh-serving/modelmesh-serving/config
~/work/modelmesh-serving/modelmesh-serving/config/rbac/namespace-scope ~/work/modelmesh-serving/modelmesh-serving/config
~/work/modelmesh-serving/modelmesh-serving/config
~/work/modelmesh-serving/modelmesh-serving/config/rbac/cluster-scope ~/work/modelmesh-serving/modelmesh-serving/config
~/work/modelmesh-serving/modelmesh-serving/config
Deploying fvt resources for etcd and minio
service/etcd created
deployment.apps/etcd created
secret/model-serving-etcd created
service/minio created
deployment.apps/minio created
secret/storage-config created
persistentvolumeclaim/models-pvc-1 created
persistentvolumeclaim/models-pvc-2 created
persistentvolumeclaim/models-pvc-3 created
job.batch/pvc-init created
pod/pvc-reader created
Waiting for dependent pods to be up ...
Pods found with selector '-l app=etcd' are not ready yet. Waiting 10 secs ...
All -l app=etcd pods are running and ready.
Pods found with selector '-l app=minio' are not ready yet. Waiting 10 secs ...
All -l app=minio pods are running and ready.
model-serving-etcd secret found
Creating storage-config secret if it does not exist
NAME             TYPE     DATA   AGE
storage-config   Opaque   1      20s
Installing ModelMesh Serving RBACs (namespace_scope_mode=false)
serviceaccount/modelmesh created
serviceaccount/modelmesh-controller created
role.rbac.authorization.k8s.io/modelmesh-controller-leader-election-role created
role.rbac.authorization.k8s.io/modelmesh-controller-restricted-scc-role created
clusterrole.rbac.authorization.k8s.io/modelmesh-controller-role created
rolebinding.rbac.authorization.k8s.io/modelmesh-controller-leader-election-rolebinding created
rolebinding.rbac.authorization.k8s.io/modelmesh-controller-restricted-scc-rolebinding created
clusterrolebinding.rbac.authorization.k8s.io/modelmesh-controller-rolebinding created
networkpolicy.networking.k8s.io/modelmesh-controller created
networkpolicy.networking.k8s.io/modelmesh-runtimes created
networkpolicy.networking.k8s.io/modelmesh-webhook created
Installing ModelMesh Serving CRDs and controller
Enabled Self Signed CA: Update manifest
secret/modelmesh-webhook-server-cert created

kustomize build default

Error: accumulating resources: accumulation err='accumulating resources from '../manager': '/home/runner/work/modelmesh-serving/modelmesh-serving/config/manager' must resolve to a file': couldn't make target for path '/home/runner/work/modelmesh-serving/modelmesh-serving/config/manager': invalid Kustomization: json: cannot unmarshal number into Go struct field Image.images.newTag of type string
error: no objects passed to apply
Error: Process completed with exit code 1.

The root cause of the error was in the sed command (see the swapped quotes in the before/after in the code block below).
sed -i.bak 's/newTag:.*$/newTag: '"$GIT_COMMIT_SHORT"'/' config/manager/kustomization.yaml. # before
sed -i.bak 's/newTag:.*$/newTag: "'${GIT_COMMIT_SHORT}'"/' config/manager/kustomization.yaml # after

That sed command (or variations of it) can be found in various places. There is a fix in PR #473, but in case I missed one, or, if a similar one may get added in the future, I created this issue to save future developers some time.
Originally posted by @ckadner in https://github.com/kserve/modelmesh-serving/pull/464#discussion_r1442361903

When the short hash of the most recent commit has no non-numeric characters (e.g. "7284872") and gets used as the newTag for building and installing the latest modelmesh-controller image, then Kubernetes interprets the value of the newTag field as an int, rather than a string.
See this sed command in one of the install scripts
sed -i.bak 's/newTag:.*$/newTag: '"$GIT_COMMIT_SHORT"'/' config/manager/kustomization.yaml

This will cause this error during the installation (i.e. when running FVT on IKS)
Error: accumulating resources from '../manager':
  '/config/manager' must resolve to a file':
    couldn't make target for path '/config/manager':
      invalid Kustomization:
        json: cannot unmarshal number into Go struct field
          Image.images.newTag of type string
error: no objects passed to apply

Check the latest git commit:
git log -1 --format=%h --abbrev=7
    7284872

A git commit short hash with exclusively numeric characters happened 7 times in the past (as of 2023/12/02):
git log --format=%h --abbrev=7 | grep -E "^[0-9]+$"
    7284872
    4808804
    4746079
    0281170
    1631706
    5293579
    4207065


org.springframework.cloud.client.discovery.DiscoveryClient can't get update from eureka

I have 3 services 'A', 'B' and 'C' using same eureka config except application name.
Service 'A' use org.springframework.cloud.client.discovery.DiscoveryClient to find service 'B'.
If 'B' start and register to eureka before 'A', 'A' can find 'B'.
If 'B' start and register to eureka after 'A' , 'A' can't find 'B'.
I can see both of them on eureka:8761 page but com.netflix.discovery.shared.Applications.shuffleVirtualHostNameMap never update.
'C' is a gateway using zull, it can find service 'B' whether start before or after it.
spring-cloud Angel.SR4

Not enough information. How long did you wait before you tried again? It can take between 30-120 seconds before a new instance shows up. What is your configuration?

I wait long enough, more than five minutes

bootstrap.xml like this
eureka:
client:
serviceUrl:
defaultZone: http://${ENV:dev}-home1-eureka:8761/eureka/
...
application.xml like this
eureka:
instance:
leaseRenewalIntervalInSeconds: 10
statusPageUrlPath: /admin/info
healthCheckUrlPath: /admin/health
nonSecurePort: 80
metadataMap:
instanceId: ${ENV:dev}-${spring.application.name}:${server.port}
client:
registryFetchIntervalSeconds: 5
serviceUrl:
defaultZone: http://${ENV:dev}-eureka:8761/eureka/
...

We don't support xml files for configuration. They need to be .yml extensions. Does the host dev-eureka resolve?

sorry it is yml actually, my mistake.

yes it resolve. DNS an network connection is ok.

Do you have sample project that recreates the problem?

My project is complicated. I need some time to create a sample project.

I just switch back to Angel.SR1 and the problem solved.
I use spring-cloud-netflix from very early version.
This issue is never appear until I upgrade form Angel.SR1 to Angel.SR4.

Angel.SR1 to Angel.SR3 is ok, but Service 'A' can't see any service start after it, even itself.
I use spring-io-platform 1.1.4.RELEASE and spring-boot 1.2.6.RELEASE.
root project build.gradle:
...
imports {
mavenBom "io.spring.platform:platform-bom:${springIoPlatformVersion}"
mavenBom "org.springframework.cloud:spring-cloud-starter-parent:${springCloudVersion}"
}
...
I override some old dependencies from platform-bom by using spring-boot-gradle-plugin and import spring-cloud-starter-parent after platform-bom.
May be it is dependency problem.

Works for me. I used a client like this:
@SpringBootApplication
@EnableDiscoveryClient
@RestController
public class TwoApplication {

	@Autowired
	DiscoveryClient client;

	@RequestMapping("/")
	public List<String> home() {
		return client.getServices();
	}

	public static void main(String[] args) {
		SpringApplication.run(TwoApplication.class, args);
	}
}

with config (application.properties) containing only server.port and spring.application.name. Launched 2 instances with different config with a pause in between while the first one registered with Eureka and I could see it in its own catalog ("/"). The first one to launch didn't list the other in its catalog (or in "/health") as quickly as the second did, but it showed up after a couple of minutes.

Closing for lack of feedback.


Add devcontainer, update  test_requirements.txt

Motivation: makes easier to contribute.
Currently to test things locally I need to delete aws config, and manually change requirements for tests.
I can confirm that after spinning up the container, I can run pytest s3fs and it works:
========================================== 313 passed, 1 skipped, 3 xfailed, 12 warnings in 150.68s (0:02:30) ==========================================

How is this executed? Is this docker? I would say that requiring docker is actually more onerous. Simpler would be a pytest fixture that masks any environment variables that were creating trouble for you.

docker or podman-as-docker if you run it on your machine.
For me that's this button:

For local development, pycharm and vs code support devcontainers now and start/manage container for you.

For me that's this button:

you need devcontainer.json first to make button work correctly.

So you still need docker, and also one of the particular IDEs that supports this mechanism. I am mildly against the idea.

up to you of course.
I'd spend 10 minutes setting this up rather ~3 hours figuring out why tests didn't work on mac or on linux, but worked in CI


Opt Out Email text fixes

Updates the greeting to use @alias instead of the blank given name. Adds instructions on how to opt out of the monthly email.

Coverage remained the same at 57.398% when pulling 8b1fe51052cabd24d6121770fe2dfd37ce6a1c29 on i#1520-opt-out-fix into 71ca6b850374a98ccc5824936a1f7a86712f0dec on develop.


Update k8s-views-global.json

Updated to show "real" memory usage by instance (node).  These values now add up the Real Global RAM Usage amount.
Pull Request
Required Fields
:mag_right: What kind of change is it?

fix

:dart: What has been changed and why do we need it?

Memory usage by node (instance) updated to match Real Global RAM usage amount.

Now Matches:

Thank you @reefland  :slightly_smiling_face:


Could not find a paired wear os device

Hello , thanks for such nice application. The thing is I can get to install the app on my watch. It is a LG G Watch with Android wear 1.5.0, and the app shows this message; Could not find a paired wear os device
Maybe it is because the watch is old?
Thanks
Bruno

Hi @elblogbruno, yes, unfortunately Android Wear 1.x is not supported, as it doesn't support standalone apps. Google has recently made even more efforts to deprecate the old model, so I won't be able to support this.

Thanks for answering!
Great app I use it everyday!


v3 extensions not copied from csr to crt

Heyho, thanks for the sweet role! One minor issue I encountered was that the extensions (in this case mostly subjectAltNames) were not copied to the certificate in my openssl version, which defeats much of the purpose of the feature, since CN's are considered outdated for server/host verification:
 $ openssl version
OpenSSL 1.1.1m  14 Dec 2021

Hence I would suggest the following changes (if they don't break existing older versions, didn't test)
diff -ad -uNrp ansible-snakeoil-master/library/snakeoil_openssl.py bodsch.snakeoil-cert/library/snakeoil_openssl.py
--- ansible-snakeoil-master/library/snakeoil_openssl.py 2022-01-05 09:52:29.000000000 +0100
+++ bodsch.snakeoil-cert/library/snakeoil_openssl.py    2022-03-27 17:32:50.547204392 +0200
@@ -99,6 +99,10 @@ class SnakeoilOpenssl(object):
             _ssl_args.append(crt_file)
             _ssl_args.append("-signkey")
             _ssl_args.append(key_file)
+            _ssl_args.append("-extfile")
+            _ssl_args.append(self.openssl_config)
+            _ssl_args.append("-extensions")
+            _ssl_args.append("req_ext")
             _ssl_args.append("-days")
             _ssl_args.append(str(self.cert_life_time))

and
diff -ad -uNrp ansible-snakeoil-master/tasks/main.yml bodsch.snakeoil-cert/tasks/main.yml
--- ansible-snakeoil-master/tasks/main.yml      2022-01-05 09:52:29.000000000 +0100
+++ bodsch.snakeoil-cert/tasks/main.yml 2022-03-27 17:32:15.873889666 +0200
@@ -107,6 +107,7 @@
         state: crt
         directory: "{{ snakeoil_local_tmp_directory }}"
         domain: "{{ snakeoil_domain }}"
+        openssl_config: "{{ snakeoil_domain }}.conf"
         cert_life_time: "{{ snakeoil_life_time | int }}"

to add the extensions also into the certificate. I could possible mess with my own openssl conf or create an own one especially for this role, but I'd argue reusing the existing csr conf is a better solution. Keep up the good work and thanks again!
Cheers,
Robert

Thanx for your Issue!
I will have a look at it as soon as I have time again!

I have opened a corresponding PR.
It would be nice if you could give me an example configuration so that I could integrate it into the tests!

Thanks a lot for the fix! In terms of configuration I just double checked with your test configuration and to share mine:
snakeoil_extract_to: '/etc/ssl'
snakeoil_domain: registry.lan
snakeoil_email: "cert@{{ snakeoil_domain }}"
snakeoil_life_time: 720
snakeoil_alt_names:
  - dns:
    - registry.lan
snakeoil_dhparam: 1024
snakeoil_force: false
snakeoil_dn:
  country: DE
  state: Berlin
  location: Berlin
  organisation: borgkube Inc.
snakeoil_extract_owner: docker-registry
snakeoil_extract_group: root

I did too many test runs with involved servers and clients, but I think the root cause was the missing extensions in the resulting certificate (a problem that I also had with own openssl scripts). So this has changed in the openssl defaults from automatically including the v3 extensions to the certificate to later not doing this anymore. So I would just suggest to add a test that runs
openssl x509 -noout -text < result.crt | grep -q 'DNS:\|IP:'

or some other more programmatic approach. This should have definitely failed before your fix on newer Linux distros. For reference my involved servers were:
ansible host: Arch Linux /Manjaro 21.2.6 (Qonos) (OpenSSL 1.1.1m  14 Dec 2021)
target host: Ubuntu 20.04 (OpenSSL 1.1.1f  31 Mar 2020)

Cheers and thanks again!

It was... work, but I wrote a test that should cover your scenario.
You can find it under molecule/borgkube.
If you want to work with make, this might help you:

make converge -e TOX_SCENARIO=borgkube
make verify -e TOX_SCENARIO=borgkube
make destroy -e TOX_SCENARIO=borgkube


add RBAC tests for policy Creation and Removal

I can see the following lint failures
/home/ksrot/devel/github/RHACM/grc-ui/controllers/index.js
  16:26  error  Unable to resolve path to module '@open-cluster-management/security-middleware'  import/no-unresolved

/home/ksrot/devel/github/RHACM/grc-ui/controllers/ui/index.js
  17:25  error  Unable to resolve path to module '@open-cluster-management/security-middleware'  import/no-unresolved

/home/ksrot/devel/github/RHACM/grc-ui/src-web/containers/App.js
  30:37  error  Unable to resolve path to module '@open-cluster-management/ui-components'  import/no-unresolved


Set required checks

Set required checks on the main branch. Wait for this https://github.com/torolocos/react-rtc/pull/85 to be merged.

Required checks are set up.


systemd service: when use the -f in the service file,cannto mount.

although find that the s3/fuse log show same log as the command line mount's log, i find not mounted when ls the mount dir.
The log print that we have "main.INFO File system has been successfully mounted."
i am not using the backgroud/damon mode to mount goofys.
i use the foreground mode and use the systemd to daemon it .
The segment of the service file is :
[Service]
Type=notify
WorkingDirectory=/root
Environment="AWS_ACCESS_KEY_ID=XX" "AWS_SECRET_ACCESS_KEY=XX"
ExecStart=/goofys --endpoint http://<IP_ADDRESS>:6081  -f --debug_fuse --debug_s3  root /mnt/goofys
ExecStop=/bin/kill -s INT $MAINPID
PrivateTmp=true
Restart=on-failure
[Install]
WantedBy=multi-user.target

found the cause:
PrivateTmp=true.


MessageRoot of MMRLeaf should be supported for multi-chain

MMR Leaf Spec {

    message_root // multichain_
}

message_root

ethereum_message_root bsc_message_root

ethereum_messages     bsc_messages

process(all_channel_message) {

    // group by channel (according to channel address/id)

    forward(channel_a);

    forward(channel_b);
}

MMR Leaf Spec {
    version,
    message_root // multichain_
}

-- messages

-- outbounds                                                                      -- inbounds

-- outbound (0, 1)                                                                -- inbound(0, 1)

message_root

filter(chain)

ethereum_message_root                       bsc_message_root    heco_message_root

chainID: 0                                  1                   2

channel_a_root  channel_b_root              channel_c_root      channel_d_root

channel_a_messages  channels_b_messages
channelID: 0        1 *

filter(origin)
app0 app1           app2 app3

A more appropriate way to represent this hierarchy is to use a double Merkle tree, so that new channels can be added without affecting existing channels.


Sort by shortest match first

I've got the case that I'm typing proc (exact match) and I've got 20+ entries proc_xxx which 'hide' the proc completion. It seems that the entries are sorted by source first [tag] then [ultisnips] then [buffer].
Can this behavior be configured or does it require a change?

I can't understand... sorry.
Please more clarification.
Currently, source priority can be configured by following settings.
let g:compe = {}
let g:compe.source = {}
let g:compe.source.buffer = { 'priority': 1000 }
let g:compe.source.tags = { 'priority': 500 }

In the attached image, you can see that I'm typing pro. I'm wondering, why proc is not at first place, as it is currently the 'best' match.
How can I find out the default priority you are describing?

I kinda have the exact same issue where it always bothered me.

Although the issue has been closed, the problem for me remains. Could you elaborate why it is closed, or what has to be configured to achieve the desired effect?
Thanks for your hard work, I really appreciate it!

Hm... nvim-compe will sort items as follows.

priority sort.
score sort
alphabetical sort

I'm guessing you don't want to sort with priority. If so, you should set priority as same value for each all sources.

It would be great to be able to set the priority of specific ItemKinds to be higher. For example, if I want all the fields of an object to appear first, then its methods, and then extra stuff that the language puts there (like JS's prototype, etc)

Thank you, that solved my issue!
For others here the lua equivalent config:
source = {
sort = { priority = 500; };
tags = { priority = 500; };
spell = { priority = 500; };
path = { priority = 500; };
buffer = { priority = 500; };
calc = { priority = 500; };
nvim_lsp = { priority = 500; };
nvim_lua = { priority = 500; };
ultisnips = { priority = 500; };
nvim_treesitter = { priority = 500; };
};

I'm kind of missing the feature I've once had in emacs, where tab would complete until the next character where you have to make a distinctive selection. It is the same as e.g. on the terminal when changing directories, where it will expand until to the character for which all possible options are equal, then you can type a further letter and expand further. But maybe that is off topic, or topic for a different issue...

@dstadelm I was just thinking of that feature from the JetBrains IDEs yesterday! It would be a great feature

I'm having a similar, but distinct problem.  I have defined a custom source here.  Despite having messed around some with the sort_text property of the items, I cannot get suggestions to appear in a reasonable order.
For example, if I type \chi, you would expect the first completion in the list to be \chi, after all, it's exactly the same string, but instead the first suggestion is \Chi and \chi appears way down the list.  This basically renders it unusable.  I think I could fix some special cases by putting in sort_text properties, but so far I can't find a general way of ensuring it returns a reasonable list without writing my own elaborate filtering and sorting methods.

@ExpandingMan You can specify sort = false for your custom sources' metadata.

The sort=false means compe will keep an order that item arrays returned from source.

I have a similar issue where the exact match is not at the top.
sort=false or sort=true doesn't help.


Wrong text merged in Briding-Header.h

Hi,
By using your cordova plugin, I was able to create my own plugin using Swift 3. The plugin I created requires the CommonCrypto framework which needs to be added to the Bridging-Header file.
In order to do so, in my plugin.xml file I added:
<header-file src="MyPlugin-Bridging-Header.h"/>

Pointing to a file containing:
#import <CommonCrypto/CommonCrypto.h>

As dependency in the plugin.xml file I added:
<dependency id="cordova-plugin-add-swift-support"/>

After adding my plugin to my project, the Bridging-Header.h file in "Other Resources" has been changed, but instead of the file contents (# import <CommonCrypto/CommonCrypto.h>), the name of the file is merged. So the contents of the Bridging-Heder.h file is:
#import <Cordova/CDV.h>
#import "MyPlugin-Bridging-Header.h"

Am I doing anything wrong? Or is this a bug?

Hello !
To me your Bridging-Header.h file looks good like that. Maybe you need to add CommonCrypto/CommonCrypto.h as an header file in your plugin.xml ?
What's your actual error ?

Ah, now I see. Sorry, my mistake. I expected the Bridging-Header.h file to look like:
#import <Cordova/CDV.h>
#import CommonCrypto/CommonCrypto.h

But I understand now that the Bridging-Header.h file is importing my header file which on its own contains the CommonCrypto library. So it looks like:
#import <Cordova/CDV.h>
#import "MyPlugin-Bridging-Heder.h"

Clear. It is working correctly. I am able to build my application.
Thanks!
Ivo


Long term goals

[ ] multi file generation, one per interface, with includes
[ ] bug-free host support for resources
[X] namespace prefix to link everything into a single binary
[ ] (Rust host support)

[ ] remove unreachable() from Rust generated code
[ ] perhaps move resource also to separate include file, this way dependencies are handled (topo-sort) - for now just move the definition file
[ ] mesh: vector leak() doesn't work
[ ] mesh: get_handle() should be get_rep()


Update Grant-CsExternalAccessPolicy.md

fix intro to describe what ExternalAccessPolicy is

Docs Build status updates of commit a72699e:
:clock10: Full build: the average full build time is 18 min(s) 1 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit a72699e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Grant-CsExternalAccessPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@get-itips - I am going to close this one since the applicable won't work with "Teams" as an option. Would you please add a note in the description that this cmdlet also applies to Teams? I will go ahead and close this one and merge the new one @bjwhalen.   Thanks.


multi-dimensional arrays - PostgeSQL

Problem
PostgreSQL supports multi-dimensional arrays: https://www.postgresql.org/docs/current/arrays.html
Prisma however does not. I would not mind using raw queries for querying the data, but as far as I know, Prisma does not provide an easy way to support custom types or partially bypass the schema.prisma.
Suggested solution
Add support for multi-dimensional arrays
Alternatives
Make it possible to partially bypass the schema.prisma or support custom types (but I would assume this is more involved considering the code generation happening)

related https://github.com/prisma/prisma/issues/11582

Is that "related" issue correct?
I think this is actually a duplicate of https://github.com/prisma/prisma/issues/11120 - do you agree @bram209? Then please close this issue. Thanks.

Is that "related" issue correct?

It is related considering if you could partially bypass the schema, you could at least still use Prisma in combination with it's raw queries as a workaround for the lack of multi dimensional array support in Prisma itself

I think this is actually a duplicate of #11120 - do you agree @bram209? Then please close this issue. Thanks.
This looks indeed like a duplicate, I searched through the issues but apparently not good enough, sorry for that 😅


error_notify: Add some idea of the deployed code version.

This is not particularly tested!  Sending to prevent us
forgetting about it.

OK, tested with the help of our handy /emails/ display, and it works. Also added git describe, which is ideal when applicable.

Looks great.  Merged, after fixing the missing mypy annotation.


Screenshot is no longer accurate with V2 pricing module now released

Per MS blog article on Azure Stream Analytics has Launched a New Competitive Pricing Model!

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

ID: f3ddce15-4eca-cc10-f90c-f901990e09da
Version Independent ID: 3fcebed3-3575-6d9c-f9f7-9bde05e8ec30
Content: Create an Azure Stream Analytics Cluster quickstart - Azure Stream Analytics
Content Source: articles/stream-analytics/create-cluster.md
Service: stream-analytics
GitHub Login: @xujxu
Microsoft Alias: xujiang1

@wsmelton
Thanks for your feedback! We will investigate and update as appropriate.

@wsmelton
Thank you for bringing this to our attention.
I've delegated this to content author @xujxu, who will review it and offer their insightful opinions.
@xujxu
Could you please review add comments on this, update as appropriate.

Updated the screenshot. The update will go live in 3 hours or so.
#please-close


Turbo battery indicator

Fixes #108

@JakeRoggenbuck

Very nice! sorry I missed it a few days ago

Tested


Ring Axis property is always visible in ParticleMaterial

Godot version
3.4 stable
System information
Windows 10
Issue description
In the Emission Shape settings of a ParticleMaterial, the option for Ring Axis parameter is always visible, no matter which Emission Shape was selected.
The correct behaviour should probably be to only show the property when a Ring Emitter is selected.

Steps to reproduce

Create a Particles2D node
Add a ParticleMaterial resource
Expand the EmissionShape parameter dropdown.

Minimal reproduction project
No response

Fixed by #55591.


KAFKA-2579; prevent unauthorized clients from joining groups

Build failure looks like recurrence of KAFKA-2455.

LGTM

Good catch.


importing an old wallet seems to not regenerate the active_key

https://bitsharestalk.org/index.php?topic=14607.0;all
Are active keys derived from the owner key?
Is there a call to force the client regenerate the active key?

Hi Vikram. is there a way to manually add the active key back?

Another: https://bitsharestalk.org/index.php?topic=14603.msg189813#msg189813


Version Conflict for Xamarin.GooglePlayServices.Basement for apps that use GooglePlayServices.Ads

NU1107: Version conflict detected for Xamarin.GooglePlayServices.Basement. Install/reference Xamarin.GooglePlayServices.Basement 71.1620.0 directly to project MyProject.Droid to resolve this issue.
MyProject.Droid -> Xamarin.GooglePlayServices.Ads.Lite 71.1712.1 -> Xamarin.GooglePlayServices.Basement (>= 71.1620.0)
MyProject.Droid -> modernhttpclient-updated 3.4.1 -> Xamarin.GooglePlayServices.SafetyNet 60.1142.1 -> Xamarin.GooglePlayServices.Basement (= 60.1142.1).

I'm experiencing difficulties with that too.
I don't use any of the Google Services on my project, so I don't know if I want to install the new dependencies of ModernHttpClient (I show them from latest commit).
Anyway as it is now I cannot even install them because of the version conflict.
I also have another problem and I think it is related to this.
After latest version there is a call to Google Play services (ProviderInstaller).
I have an android 4.4 emulator without Google Play Services installed and I cannot work on that emulator any more.
I think all of these changes made to support TLS 1.2 on old android versions.
Maybe it's a better idea to call the ProviderInstaller after a failure of TLS handshake, otherwise on every new connection (or instance of moderHttpClient) a call to ProviderInstaller is made, and if the device is already updated that call does nothing.
IMO the application is responsible for this and not the communication library. The application can call it once when it starts or after a TLS failure for example.
Anyway thanks to alexrainman for his work to this library.

I am going to remove that dependency and provide it as a workaround instead. I wasn’t so convinced of it anyway.

Version 3.4.2 is out.


Migrations are not applied on sentry upgrade

I installed the plugin via pip, ran the upgrade, but the migrations are not being applied. Executing sentry plugin list I get sentry-hipchat-ac 1.0.0.
In consequence every time I click on the integration button in the sentry interface I get an exception about some models that don't exists.
Am I doing something wrong?

I had to execute SENTRY_CONF=/etc/sentry sentry django migrate sentry_hipchat_ac in order to install the migrations.

There are also int(11) vs bigint(20) issues that cause foreign keys to break on MySQL.

I was going to fix this but its a bit of a pita. We need to change the ManyToMany relationships to just be intermediary models, and to use FlexibleForeignKey.

I am running into this issue still now when trying to use the integration on my On Premise instance.
Here is the full error that Sentry Internal is showing. This happens when I try to configure the plugin, which shows up in Project Settings.
ProgrammingError: relation "sentry_hipchat_ac_tenant" does not exist
LINE 1: ...", "sentry_hipchat_ac_tenant"."auth_user_id" FROM "sentry_hi...
                                                             ^

SQL: SELECT "sentry_hipchat_ac_tenant"."id", "sentry_hipchat_ac_tenant"."room_id", "sentry_hipchat_ac_tenant"."room_name", "sentry_hipchat_ac_tenant"."room_owner_id", "sentry_hipchat_ac_tenant"."room_owner_name", "sentry_hipchat_ac_tenant"."secret", "sentry_hipchat_ac_tenant"."homepage", "sentry_hipchat_ac_tenant"."token_url", "sentry_hipchat_ac_tenant"."capabilities_url", "sentry_hipchat_ac_tenant"."api_base_url", "sentry_hipchat_ac_tenant"."installed_from", "sentry_hipchat_ac_tenant"."auth_user_id" FROM "sentry_hipchat_ac_tenant" INNER JOIN "sentry_hipchat_ac_tenant_projects" ON ( "sentry_hipchat_ac_tenant"."id" = "sentry_hipchat_ac_tenant_projects"."tenant_id" ) WHERE "sentry_hipchat_ac_tenant_projects"."project_id" = %s

At one point we'll fix this, but I'm going to point that we don't support MySQL these days, so we're absolutely not prioritizing this (and its non-trivial to fix).

Ah, gotcha - didn't think about how this was written before the Postgres migration. Thanks @dcramer for the quick response & update :)

@dcramer probably just need to update the readme as it is currently misleading :(

After upgrading to Sentry 8.7.0 I've run sentry django migrate sentry_hipchat_ac as @alejandrodnm suggested and ended up with some version conflict:
Failed to load plugin 'hipchat_ac':
Traceback (most recent call last):
  File "/srv/sentry/local/lib/python2.7/site-packages/sentry/runner/initializer.py", line 42, in register_plugins
    plugin = ep.load()
  File "/srv/sentry/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2248, in load
    self.require(*args, **kwargs)
  File "/srv/sentry/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2265, in require
    items = working_set.resolve(reqs, env, installer)
  File "/srv/sentry/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 859, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
ContextualVersionConflict: (requests 2.11.1 (/srv/sentry/lib/python2.7/site-packages), Requirement.parse('requests[security]<2.11.0,>=2.9.1'), set(['sentry']))

As a result the HipChat integration isn't available in the project settings anymore.

It sounds like something you ran installed an incompatible version of requests. I would confirm that Sentry's requirements are still correct, as sentry-hipchat-ac does not change the requirements for 'requests' (it simply says its required), and thus you should have a version that satisfies sentry.


buffers in parenthesis

All of the slack buffers are currently printed in parenthesis for me, which is how non-connected/joined IRC buffers are printed, so it's rather distracting. I can’t find anything in the wee-slack script about it though. Any ideas?

are you using a plugin to show buffers? these are "python." buffers, so may appear differently.

Oh right, it's buffers.pl. Didn't think about that. Wonder how it checks..

seems to be the setting mark_inactive. when i turn i do /set buffers.look.mark_inactive off it goes away.

i'll take a pull request to change this behavior, but not really interested in fixing it myself..


Lethal Progression

The mod Lethal Progression's values are being reset when using any version of this mod after 1.4.0.  Reverting to 1.4.0 works fine, and none of the changes in 1.6.0 resolved this, so it seems something between 1.4.0 and 1.5.0+ broke compatibility with most other mods that modify saves, which is certainly problematic.

It also sometimes keep LethalProgression's level from another previous save and, no matter if you delete save, create new thunderstorm profile or uninstall and reinstall both lethal company and thunderstore, the data isn't cleared and you recover your level from a random save.

bump... Does this need new info or something?  Any indication that this has been noticed would be appreciated.

Apologies, I kept meaning to come back to this one. I don’t have any experience with Lethal Progression. Can you confirm that this is still an issue as of v47 + LCBetterSaves 1.7.0? If so, I will look at adding support as with LateGameUpgrades

Hi @Khunkurisu, can you confirm the issue for me? I'll close the issue if it's been resolved sometime over the last several mod updates.

Sorry for the wait, winter weather has caused issues.  I’ll be testing things out with just these two mods installed on their latest versions this evening and reporting back.

No worries, thanks for the help

https://github.com/paul-merkamp/LCBetterSaves/assets/17495210/29e4ba3e-76c9-46bc-81e3-2ee8266904e2
In the linked video you can see that I have a single save file, with only a few levels in Lethal Progression.  Exiting back to the main menu and creating a new save, I suddenly have many more levels.  It appears it's loading an existing save for Lethal Progression that was once tied to this slot.  As noted in the original post, this doesn't happen when using v1.4.0 of LCBetterSaves.  On v1.4.0, the save files are properly interacting (e.g. it doesn't load an old, existing save just because the slot # is the same).
If I had to guess without going through the commit differences from 1.4.0's release to when it breaks, I'd say it's a combination of not deleting all of the attached files properly when removing/resorting a save slot and also not having a common indicator between the mods to indicate what file belongs to which save slot.

Attached image for example of save directory for Lethal Progression.

Player.log
Player.log included in case needed.

If I might suggest something, since this has happened with two relatively popular mods now: You could create a generic method of handling saves from other mods, provide info in the readme about how to make a mod compatible with your save setup, and then it'll be easier to track bugs as the issues go to the mod dev to add compatibility, rather than you having to figure out why every mod with its own savefile doesn't work and adding compatibility afterwards.

Good point. I think I want to encourage other mod authors to embed their save data in the file itself with ES3, it makes things even simpler. I'll work on putting together a doc with some resources for that process.

I've added instructions to the readme, so we can point mod authors to that.
I'll add a note on the relevant issue on LethalProgression.

Closing this issue since we'll just be awaiting support on their end now - here is the relevant issue on LethalProgression
https://github.com/stoneman2/LethalProgression/issues/44


Type constraints on typealiases are evaluated eagerly

Consider the following (the required typealias redirection is another issue):
// foo.pkl
typealias FooMapping = Mapping<String, String>
foo: FooMapping(toMap().every((k, v) -> bar.containsKey(k))

bar: Mapping<String, Boolean>

// bar.pkl
amends "./foo.pkl"

bar {
  "a" = true
  "b" = false
}

foo {
  "a" = "aaa"
}

$ pkl eval bar.pkl
foo {
  ["a"] = "aaa"
}
bar {
  ["a"] = true
  ["b"] = false
}

As expected. However, if we change foo's definition to:
typealias FooMapping1 = Mapping<String, String>
typealias FooMapping = FooMapping1(toMap().every((k, v) -> bar.containsKey(k))
foo: FooMapping

Then I get:
$ pkl eval bar.pkl
–– Pkl Error ––
Type constraint `toMap().every((k, v) -> bar.containsKey(k))` violated.
Value: new Mapping { ["a"] = "aaa" }

2 | typealias FooMapping = FooMapping1(toMap().every((k, v) -> bar.containsKey(k)))
                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
at foo#foo (file:///.../foo.pkl, line 2)

8 | foo {
    ^^^^^
at bar#foo (file:///.../bar.pkl, line 8)

Placing a trace() on bar reveals that bar is a Mapping {} during the evaluation of the constraint, which is unexpected (to me at least). Is this intended behavior?

My understanding is that accessing bar from within a type constraint isn’t allowed because it isn’t const. If you don’t get an error for that it’s probably a bug.

I can't reproduce this (but also; the snippets you included are not syntactically valid Pkl, so I think something got lost in reduction of the example).
There is a known bug that typealiases resolved names in their definition scope, instead of their use scope. This has been addressed in #373 and will be part of 0.26 (hopefully June).
By the way, toMap() already forces the entire Mapping (keys and values). If you want to keep things lazy, consider keys.intersect(bar.keys) == keys instead.

I think @sin-ack is using 0.26.0-dev, which reproduces for me.
We currently have an issue in the latest dev version where typealias bodies are not late-bound (changed a fix for a scoping bug in https://github.com/apple/pkl/pull/144). But, at the same time, they aren't required to be const.
We're thinking through whether typealias bodies should be late-bound, or if this is actually the correct behavior. If this is the correct behavior, then we will add an additional rule that references must be const. If not, then we need to adjust the behavior here so that those references are late-bound.


SyntaxWarning: "is not" with a literal. Did you mean "!="?

upon running the command, I am getting below errors
python3 cloudfail.py --target seo.com
/home/kali/CloudFail/cloudfail.py:180: SyntaxWarning: "is not" with a literal. Did you mean "!="?
while choice is not 'y' and choice is not 'n':
/home/kali/CloudFail/cloudfail.py:180: SyntaxWarning: "is not" with a literal. Did you mean "!="?
while choice is not 'y' and choice is not 'n':
/home/kali/CloudFail/cloudfail.py:180: SyntaxWarning: "is not" with a literal. Did you mean "!="?
while choice is not 'y' and choice is not 'n':
/home/kali/CloudFail/cloudfail.py:180: SyntaxWarning: "is not" with a literal. Did you mean "!="?
while choice is not 'y' and choice is not 'n':
/home/kali/CloudFail/cloudfail.py:182: SyntaxWarning: "is" with a literal. Did you mean "=="?
if choice is 'y':
Traceback (most recent call last):
File "/home/kali/CloudFail/cloudfail.py", line 9, in
import socks
File "/home/kali/CloudFail/socks.py", line 62, in
from collections import Callable
ImportError: cannot import name 'Callable' from 'collections' (/usr/lib/python3.10/collections/init.py)
~}}

Same, on arch, python3, requirements installed fine

what python version are you using ?

Having the same error, did anyone find a fix?

Hello, if you are using python3 then you can fix this by with edit socks.py file.
You need to change the line 62
Replace this "collections import Callable" with this "from collections.abc import Callable"
But please, remember this is temporary solution, Zamanry has an open PR about this, so when this pr will merge you can just pull this changes

Hello, if you are using python3 then you can fix this by with edit socks.py file. You need to change the line 62
Replace this "from collections import Callable" with this "from collections.abc import Callable"

@Fojx Thank you for your comment. After some time tinkering around, it now works, even with GitHub actions.


any idea for decoding kafka payload by myself

Is there any way to decode kafka payload by myself If I got a []byte request or response payload.
For example, when producing a Kafka message from a HTTP request body

You have to look at https://github.com/Shopify/sarama/blob/master/record.go#L39 and related things.


Add Night Vision Filter to Filter list

The actual RGB night vision values can be tweaked if needed, but I think it looks pretty good as is.

how to make people face beautiful?
can you add some beauty face filter algorithms?

Bilateral filter is pretty useful for that. The result: Smooth faces in a gentle way.


Workflow: add trace and rightpanel

Describe what this PR does / why we need it
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: conghuhu:x: congguoqingcongguoqing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Empty preconstructed pandas results in error in Variable Explorer

If I construct a DF like this:
iv_bd = pd.DataFrame(columns=['Jan','Feb','Mar'])
It causes an internal error instead of showing something like e.g. "empty Dataframe"
Spyder Internal Console

This console is used to report application
internal errors and to inspect Spyder
internals with the following commands:
  spy.app, spy.window, dir(spy)

Please don't use it to run your code

>>> /home/george/anaconda/lib/python2.7/site-packages/nbformat/current.py:19: UserWarning: nbformat.current is deprecated.

- use nbformat for read/write/validate public API
- use nbformat.vX directly to composing notebooks of a particular version

  """)
Traceback (most recent call last):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dicteditor.py", line 503, in createEditor
    if not editor.setup_and_check(value, title=key):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 494, in setup_and_check
    self.dataModel = DataFrameModel(data, parent=self)
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 90, in __init__
    self.max_min_col_update()
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 124, in max_min_col_update
    mask = float_intran & (~ self.complex_intran)
  File "/home/george/anaconda/lib/python2.7/site-packages/pandas/core/generic.py", line 639, in __invert__
    arr = operator.inv(_values_from_object(self))
TypeError: ufunc 'invert' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
Traceback (most recent call last):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dicteditor.py", line 503, in createEditor
    if not editor.setup_and_check(value, title=key):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 494, in setup_and_check
    self.dataModel = DataFrameModel(data, parent=self)
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 90, in __init__
    self.max_min_col_update()
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 124, in max_min_col_update
    mask = float_intran & (~ self.complex_intran)
  File "/home/george/anaconda/lib/python2.7/site-packages/pandas/core/generic.py", line 639, in __invert__
    arr = operator.inv(_values_from_object(self))
TypeError: ufunc 'invert' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
Traceback (most recent call last):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dicteditor.py", line 503, in createEditor
    if not editor.setup_and_check(value, title=key):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 494, in setup_and_check
    self.dataModel = DataFrameModel(data, parent=self)
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 90, in __init__
    self.max_min_col_update()
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 124, in max_min_col_update
    mask = float_intran & (~ self.complex_intran)
  File "/home/george/anaconda/lib/python2.7/site-packages/pandas/core/generic.py", line 639, in __invert__
    arr = operator.inv(_values_from_object(self))
TypeError: ufunc 'invert' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
Traceback (most recent call last):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dicteditor.py", line 503, in createEditor
    if not editor.setup_and_check(value, title=key):
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 494, in setup_and_check
    self.dataModel = DataFrameModel(data, parent=self)
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 90, in __init__
    self.max_min_col_update()
  File "/home/george/anaconda/lib/python2.7/site-packages/spyderlib/widgets/dataframeeditor.py", line 124, in max_min_col_update
    mask = float_intran & (~ self.complex_intran)
  File "/home/george/anaconda/lib/python2.7/site-packages/pandas/core/generic.py", line 639, in __invert__
    arr = operator.inv(_values_from_object(self))
TypeError: ufunc 'invert' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Spyder 2.3.7
Python 2.7.10 64bits, Qt 4.8.7, PyQt4 (API v2) 4.11.4 on Linux
Ubuntu 14.04 LTS

Thanks for reporting. I'll take a look at this one as soon as I can :-)

I'll fix this problem in version 2.3.8. Thanks for your patience.

Thank you for your work, Sir.


Fix edge detection

This was outdated and the new format for Chromium-based Edge wasn't supported.

Sorry, I intended to open this pull request against our our fork (basecamp/useragent) 😅


[Account Abstraction] Move registered authenticators to the app so that it works with muliple chains in testing

This may actually have been fixed by https://github.com/osmosis-labs/osmosis/pull/6298/commits/7af618f31e92ee03b7ad8239ddeb01a52f594fa6.


[SPARK-15038][SQL] Add option to lazily broadcast at SQL execution time

What changes were proposed in this pull request?
This allows the user to specify whether to do the broadcasting of a small DataFrame asynchronously at query preparation time or during execution time.
How was this patch tested?
Manual testing

Can one of the admins verify this patch?

@pwoody any reason why you did for 1.6?

Yeah - I was hoping for this to be a part of a 1.6.2 release. Will happily do this against master as well - I'm not sure of the protocol when there is divergence in the diff.

This is an improvement not a bug fix; we only backport those in rare cases. It could be added to Spark 2.1 since we are already in QA for Spark 2.0.
In Spark 2.0 we did some work in this department and we have added a BroadcastExchangeExec node which will take care of any broadcasting. I think it will be quite easy to add this feature there.

hm maybe instead of this, we just need to set the size of the thread pool for broadcasting? if it is set to 1, then it effectively turns it into a sequential execution.

I could be wrong, but I believe the main difference between this patch and limiting the threadpool is that the broadcasts will still be retained in the memory manager by the reference in the node (which should stick around until execution is completed). The change here should get GC'd and properly evicted from the memory manager as each node gets executed.
Re: bugfix v.s. improvement - this issue prevents certain large queries (100s of broadcasts) from reasonably being completed with DataFrames. This change might be an improvement, but I think there should be a backport of something to 1.6.x to mitigate this issue.

Can you explain more about the memory differences?

Hey apologies for the late reply!
There are effectively two failure modes w.r.t the memory manager:

If you retain the SparkPlan then the broadcast references for the eager/async broadcasts get retained as well and aren't cleaned by the ContextCleaner.
With async, you will be pushing many broadcasts onto disk storage before they get used. In normal execution they will simply hit memory and be issued out to the nodes (almost) immediately.


Fix JDK8 install error on Travis

Most workarounds suggested downgrading to trusty (14.04), but one mentioned that the problem is specific to xenial (16.04), so hopefully 18.04 will work.

Thanks @vrevelas and sorry for the delay!

Awesome, thanks! No prob!


Suggestion: each source file with its header?

The code has general header files (i.e. funcs.h for functions).
These headers are big and slow the compilation if included in every source file.
Moreover, many source files have extern definitions because some functions are not defined in the header files.
It is more common in general projects that each source file has its corresponding header file.
Then, source files can benefit from shorter and more specific header files.

The basic structure of the EPANET code hasn't changed much since it was first written almost 30 years ago, before encapsulation and object oriented designs became popular. Some six years ago a new OWA project, epanet-dev (not to be confused with the dev branch of this project) was begun to modernize the EPANET code base using object oriented C++. Rather than spending time on re-engineering the EPANET 2 code base I believe it would be more beneficial to direct such efforts to completing and improving epanet-dev (so it can become EPANET 3).

I am a good C programmer and I can contribute to improve and modernize C codes.
Nevertheless, in C++ I am a very mediocre programmer. It will be much more difficult for me to contribute.


What is the principle of computing major/minor/least method?

Why the principal component analysis can give a 3D axis length? Actually, when rotate the object, the result of least axis length is not correct. Does anyone known that？

@JoostJM

@manguoxia, the PCA analysis does not give the 3D axis length. Rather, it is related to the axes of the enclosing ellipsoid.
I'm afraid I do not fully understand your question about rotation. What do you mean by "least axis length is not correct", what do you define as "correct"?

I'm afraid I do not fully understand your question about rotation. What do you mean by "least axis length is not correct", what do you define as "correct"?

Thank you for your reply !
I generated a standard ellipsoid with three axis lengths of 60, 40 and 20, respectively.
I calculated the three axis length, the results are: major 53.63 /  minor 35.76 / and least 17.78
Then it is rotated at an angle(45° along X axis) in space.
After rotated the ellipsoid, I calculated the three axis length again, but I got a different result:
major 65.25 /  minor 52.41 / and least 43.49
I'm confused about that.I think the results should be consistent.

The unit of the ellipsoid  is pixels. It's isotropic volume. The size of the image is 150150150 .
The ellipsoid has three axis lengths of 60, 40 and 20, respectively. I didn't consider the spacing of the image when generating the ellipsoid, just in pixel.
The images are saved with raw binary format.
data.zip

@mangguoxia, Can you share the code you used to arrive at your results? I attempted to replicate your inconsistencies, but was unable to.
Here's what I did:
I first loaded the raw data file as a numpy array, and then stored them as NRRD images (PyRadiomics does not accept Numpy Arrays as input, as it requires geometric information.
When I then extract the shape features with PyRadiomics, I obtain very similar results for both the original and the rotated one (some small differences exist, but this is most likely due to rounding and machine precision errors).
Here is the code I used to load your data:
import numpy as np
import SimpleITK as sitk

arr = np.fromfile('ellipsoid.raw',  'int16')
arr.reshape((150, 150, 150))
im = sitk.GetImageFromArray(arr)  # results in pixel spacing (1.0, 1.0, 1.0)

sitk.WriteImage('ellipsoid.nrrd', arr, True)

arr_rot = np.fromfile('ellipsoidRotated.raw',  'int16')
arr_rot.reshape((150, 150, 150))
im_rot = sitk.GetImageFromArray(arr_rot)  # results in pixel spacing (1.0, 1.0, 1.0)

sitk.WriteImage('ellipsoidRotated.nrrd', arr_rot, True)

I then used PyRadiomics command line to extract features with parameters:
featureClass:
  shape:
    - Maximum3DDiameter
    - MajorAxisLength
    - MinorAxisLength
    - LeastAxisLength

imageType:
  Original: {}

pyradiomics ellipsoidRotated.nrrd ellipsoidRotated.nrrd -p params.yml
pyradiomics ellipsoid.nrrd ellipsoid.nrrd -p params.yml
Alternatively, you can do it interactively:
from radiomics.featureextractor import RadiomicsFeatureExtractor

extractor = RadiomicsFeatureExtractor('params.yml')
featurevector = extractor.execute(im, im)  # you can use the data both as image and mask, as only mask is used for calculation

featurevector_rot = extractor.execute(im_rot, im_rot)

This yields the following results:

feature
ellipsoid
ellipsoid_rotated

Maximum3DDiameter
61.0
60.828

MajorAxisLength
53.638
53.750

MinorAxisLength
35.768
35.746

LeastAxisLength
17.788
17.772

@mangguoxia, as to the reason why the maximum 3D diameter is about 61, instead of 60: This is due to how PyRadiomics sees the volume. Using a marching cubes algorithm, it constructs a triangel mesh representing the hull of the shape. The points of the triangles are always halfway between a voxel that is part of the mask and one that is not. Therefore, the edges is .5 x spacing higher and lower that the outer most voxels along your axis of 60 pixels. Given the spacing of 1.0, this results in 60 x 1.0 + 2 x 0.5 x 1.0 = 61. The reason we use this triangle mesh is that we can use it for approximations of maximum diameters, volume and surface area. We opted to use one approximation to calculate all these values, instead of e.g. voxel centers for diameter, but mesh for surface and then voxels volume for volume calculation. (N.B. PyRadiomics does implement the volume as no. of voxels times volume of a single voxel, but this is for comparison to volume calculation in current general practice. Shape features are based on mesh volume where applicable.

@JoostJM  Thank you for your reply!
I am very sorry that my mistake led to the wrong result.
But i find a new method to get a more accurate result of Major/Minor/Least length.
In my image, the results are 60.103 39.609 20.0 .
Here is the code https://github.com/joe-of-all-trades/regionprops3/blob/master/regionprops3.m
The difference is the usage of eigen vectors.
I think you can try this method to get a more accurate method.
And here is the similar information of this method.(The most voted answer)
https://math.stackexchange.com/questions/207685/how-to-find-the-minimal-axis-parallel-ellipse-enclosing-a-set-of-points
Thank you so much！


deployments(ticdc): fix broken links

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/8150
What is changed and how it works?
fix broken links. Using https://www.shorturl.at to replace git.io.
Check List
Tests

Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
No
Do you need to update user documentation, design documentation or monitoring documentation?
No
Release note
None

/merge


How to attach turtlebot_arm to turtlebot package?

Linking urdf , i dont know how to do that. Can someone help me out?

Sorry @hammadasifalvi, but github issues is not the right place to ask questions on how to use the code, but to report problems or propose improvements. I would recommend you to post the question in http://answers.ros.org/, where I'll happily answer.


Weird error when identifying some titles

I've found this error at least in two titles. They keep unindentified after several days because the parser seems to choke in the results.
The error I get if I search the title and double click on it to select an issue is this:
Oops.An unexpected error occured:
File "comictaggerlib/issueselectionwindow.py",line
35,in__It_
TypeError: '<' not supported between instances of
'NoneType' and 'float'

When auto-tag is used, the issue is not found.
I imagine this can be replicated by naming any cbz with the same titles:
We Only Find Them When They're Dead 011 (2022) (digital) (Son of Ultron-Empire).cbz
Neverender 001 (2022) (Digital) (Mephisto-Empire).cbz
Also, if I change the numbering, auto-identify works OK, but same error crops up if I try to navigate the issues.

Have you tried on the latest beta? 1.4.4-beta.1

Welp, the beta came out right as I logged the ticket. It seems indeed to have fixed the issue :D