[SUPPORT] Failed to delete key: /.hoodie/.temp/20201006182950

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
Random "Failed to delete key" error during UPSERT operation in a Spark Structured Streaming job even with "hoodie.consistency.check.enabled" set to true
To Reproduce
Steps to reproduce the behavior:
UNKOWN - Occurs randomly after running streaming job - sometimes for 15 hours and other times for about couple days
Expected behavior
UPSERT should be a simple operation and if there is an application bug then we should have faced this as soon as I launched the application but it appears intermittently. The only resolution so far is to restart the job
Environment Description

Hudi version : 0.5.2-incubating

Spark version : 2.4.4

Hive version : 2.3.6

Hadoop version : EMR 5.29.0

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : no

Additional context
This is a simple streaming application which listens to Kinesis stream with a batch interval of 15 minutes and updates the Hudi table using MERGE_ON_READ
Stacktrace
	at org.apache.hudi.table.HoodieTable.deleteMarkerDir(HoodieTable.java:333)
	at org.apache.hudi.table.HoodieTable.cleanFailedWrites(HoodieTable.java:409)
	at org.apache.hudi.table.HoodieTable.finalizeWrite(HoodieTable.java:315)
	at org.apache.hudi.table.HoodieMergeOnReadTable.finalizeWrite(HoodieMergeOnReadTable.java:317)
	at org.apache.hudi.client.AbstractHoodieWriteClient.finalizeWrite(AbstractHoodieWriteClient.java:195)
	... 66 more
Caused by: java.io.IOException: Failed to delete key: <tableName>/.hoodie/.temp/20201006182950
	at com.amazon.ws.emr.hadoop.fs.s3n.S3NativeFileSystem.delete(S3NativeFileSystem.java:767)
	at com.amazon.ws.emr.hadoop.fs.EmrFileSystem.delete(EmrFileSystem.java:337)
	at org.apache.hudi.common.io.storage.HoodieWrapperFileSystem.delete(HoodieWrapperFileSystem.java:261)
	at org.apache.hudi.table.HoodieTable.deleteMarkerDir(HoodieTable.java:330)
	... 70 more
Caused by: java.io.IOException: 1 exceptions thrown from 5 batch deletes
	at com.amazon.ws.emr.hadoop.fs.s3n.Jets3tNativeFileSystemStore.deleteAll(Jets3tNativeFileSystemStore.java:390)
	at com.amazon.ws.emr.hadoop.fs.s3n.S3NativeFileSystem.doSingleThreadedBatchDelete(S3NativeFileSystem.java:1494)
	at com.amazon.ws.emr.hadoop.fs.s3n.S3NativeFileSystem.delete(S3NativeFileSystem.java:765)
	... 73 more
Caused by: java.io.IOException: MultiObjectDeleteException thrown with 2 keys in error: <tableName>/.hoodie/.temp/20201006182950/195/2ecca5ce-ba13-4d5a-a2e3-79713261dc49-0_2061-53-37527_20201006182950.marker, <tableName>/.hoodie/.temp/20201006182950/290_$folder$
	at com.amazon.ws.emr.hadoop.fs.s3n.Jets3tNativeFileSystemStore.deleteAll(Jets3tNativeFileSystemStore.java:375)
	... 75 more
Caused by: com.amazon.ws.emr.hadoop.fs.shaded.com.amazonaws.services.s3.model.MultiObjectDeleteException: One or more objects could not be deleted (Service: null; Status Code: 200; Error Code: null; Request ID: 457C53995454141D; S3 Extended Request ID: NKQEApW06BHPRG5oQBP4RffTd6OZQXOCNl6jurU690Ee+iE3cgbRbbtNPugjqa3qyADj6x5zqBk=), S3 Extended Request ID: NKQEApW06BHPRG5oQBP4RffTd6OZQXOCNl6jurU690Ee+iE3cgbRbbtNPugjqa3qyADj6x5zqBk=
	at com.amazon.ws.emr.hadoop.fs.shaded.com.amazonaws.services.s3.AmazonS3Client.deleteObjects(AmazonS3Client.java:2267)
	at com.amazon.ws.emr.hadoop.fs.s3.lite.call.DeleteObjectsCall.perform(DeleteObjectsCall.java:24)
	at com.amazon.ws.emr.hadoop.fs.s3.lite.call.DeleteObjectsCall.perform(DeleteObjectsCall.java:10)
	at com.amazon.ws.emr.hadoop.fs.s3.lite.executor.GlobalS3Executor.execute(GlobalS3Executor.java:110)
	at com.amazon.ws.emr.hadoop.fs.s3.lite.AmazonS3LiteClient.invoke(AmazonS3LiteClient.java:189)
	at com.amazon.ws.emr.hadoop.fs.s3.lite.AmazonS3LiteClient.invoke(AmazonS3LiteClient.java:184)
	at com.amazon.ws.emr.hadoop.fs.s3.lite.AmazonS3LiteClient.deleteObjects(AmazonS3LiteClient.java:128)
	at com.amazon.ws.emr.hadoop.fs.s3n.Jets3tNativeFileSystemStore.deleteAll(Jets3tNativeFileSystemStore.java:370)```

@umehrot2 : Can you throw some light here ? when will EMR/S3 throw this error ? Is this server-side issue which will go away with retry ?

Could happen due to race condition https://stackoverflow.com/questions/38750638/spark-1-6-1-s3-multiobjectdeleteexception
I have about 100 retries on S3 failures in my application

@prashanthvg89 : With 0.6.0 release, this was no longer a fatal error. Can you try that version ?

With 0.6.0, you can set hoodie.fail.on.timeline.archiving=false to make it non-fatal

I think this is a manifestation of a bug with EmrFS that has been fixed in future EMR releases. Is it possible for you to give any release >= emr-5.30.0 a shot ?

Sure, will try emr-5.30.0 and hudi-0.6.0. The issue appears intermittently. Since the upgrade is the only option right now, may be we can close the issue. I'll reopen if I see it again even after upgrades. Thanks!

Thanks


Shellcheck should be run as a build for the shellscripts

https://github.com/koalaman/shellcheck
An automated way (instead of the current manual) shellcheck step should be added to the repo for the scripts here: https://github.com/amido/stacks-pipeline-templates/tree/feature/cycle4/scripts

is this relevant/needed/desired any more @ElvenSpellmaker ?


WIP: Adjust some qualifiers for std.experimental.allocator

Closing this as won't fix for now because I realized


Sprint Epic 26th Jan - 9th Feb

[ ] More work on the tour feature #5168 @acburdine
[ ] Adapter writeup @ErisDS

Channels

[x] Finishing up metadata refactoring #6186 @jtwebman
[x] Dynamic routing for Channels PR #5091 @ErisDS

Post Scheduling

[ ] Finish writing issues @ErisDS
[ ] Timezone issue :clock10: #6406 @AileenCGN
[ ] cron adapter #6413 @jtwebman

New Editor and Image Uploads

[ ] :boom: :fire: uploader.js and refactor our image upload components @kevinansfield
[ ] Editor: finalize CodeMirror card to highlight additional requirements for card "apps/addons" @kevinansfield
[ ] Editor: initial image card @kevinansfield
[ ] Editor: look into inline-block vs block cards (eg. images could be inline so they can be mid-paragraph, vs. a fully-featured markdown editor that has to be a completely separate section) @kevinansfield

Closing in favour of #6470


Nifi throwing an error message every 30 seconds

This does not impact running of tests.  I stumbled into these errors when debugging the Nifi flow while investigating ASCVD failures.
The error message:
InvokeScriptedProcessor[id=bd38dc04-e99c-395e-a816-0b8da5df640d] Failed to properly initialize Processor. If still scheduled to run, NiFi will attempt to initialize and run the Processor again after the 'Administrative Yield Duration' has elapsed. Failure is due to org.apache.nifi.processor.exception.ProcessException: Error creating ScriptRunner: org.apache.nifi.processor.exception.ProcessException: Error creating ScriptRunner
The process id noted in the message is for the "Break up error message if too large" processor.
David says this is a script in the "MM" flow that we base our nifi one.
("MM" refers to Mike Payne and Mike Britt)

UPDATE:   This issue is not as prevalent as I originally thought.   The day I created this issue, every ingestion deployment would have these messages. In the following days, seeing these messages in the nifi canvass was rare.


WIP: Client-NG

I checked bfd0623 and it looks reasonable.

Codecov Report
Base: 62.14% // Head: 61.74% // Decreases project coverage by -0.41% :warning:

Coverage data is based on head (783f8bf) compared to base (d76a9b4).
Patch coverage: 0.54% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1621      +/-   ##
==========================================
- Coverage   62.14%   61.74%   -0.41%
==========================================
  Files         130      132       +2
  Lines       25126    25307     +181
==========================================
+ Hits        15615    15625      +10
- Misses       9511     9682     +171

Impacted Files
Coverage Δ

fedimint-client/src/sm/executor.rs
0.00% <0.00%> (ø)

fedimint-client/src/sm/state.rs
0.00% <0.00%> (ø)

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

client/client-lib/src/lib.rs
77.95% <0.00%> (+0.10%)
:arrow_up:

modules/fedimint-mint/src/lib.rs
77.95% <0.00%> (+0.11%)
:arrow_up:

fedimint-server/src/net/connect.rs
94.64% <0.00%> (+0.24%)
:arrow_up:

fedimint-server/src/consensus/mod.rs
81.11% <0.00%> (+0.31%)
:arrow_up:

fedimint-server/src/net/peers.rs
89.42% <0.00%> (+1.42%)
:arrow_up:

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

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


Update to reflect power relations are respected with DFx.

Thanks @chcurlet-msft this should be live within half an hour


Code example in wizard for leaflet has superfluous new keyword

click https://nlmaps.nl/#wizard
click leaflet
this the the example:

Noticenew L.tileLayer, which should be either new L.TileLayer or L.tileLayer.
Some other things I noticed:

the attribution here is different from the one generated by the 'NL Maps Library'
the mapbox example uses syntax .setView([lat, lng], zoom) which I prefer over using the options object.
latlng precision is 14 decimal places, which is way more then ever needed (6 should be enough).

Als achtergrond bij laatste punt van @jieter zie Silly Geographic Precision.

This issue can be closed, I presume:

Wizard now uses NL Maps library in the wizard instead of code specific to a mapping library https://github.com/kadaster/nlmaps/commit/e24d380afc4317eb66ea8375949ce10b33d56477#diff-51bd9925ed129615606366c8417dcd4e:

Supersedes superfluous "new" in leaflet code
Aligns the attribution
No mapbox-specific implementation example

Coordinate precision reduced to 6 decimals https://github.com/kadaster/nlmaps/commit/fb5af076ac85eb26483210293c2eadcbf633ef82

@jieter can you check?

@emacgillavry first diff is a bit big to quickly grasp what it's doing. For (2) code looks good, but not deployed/updated yet? I still see center: [52.21119999999999, 5.9699], on https://nlmaps.nl/#wizard.

@jieter no worries. Just to let you know that we have made that change in the code. Indeed, the wizard has not yet been updated to reflect changes in the code.

@jieter NL Maps wizard has been deployed to production! You'll also notice the coordinates only update once the map does not move or zoom anymore.


--debug-jvm is ineffective while running a single test method

Not sure if this is a bug or if I don't know how to use gradle.
gradle test --tests path.to.a.ClassTests --debug-jvm

works (I can see Listening for transport dt_socket at address: 5005) but
gradle test --tests path.to.a.ClassTests.testMethod --debug-jvm

doesn't work (there's no Listening for transport dt_socket at address: 5005).

I'm pretty sure it works (please provide a reproducible sample). The only reason I can think of is that your gradle test --tests path.to.a.ClassTests.testMethod doesn't match any test method in the class.

doesn't match any test method in the  class

That's correct. Now I'm so ashamed.


Using gsuite email address will prevent reviews on android

Please refer to this issue: https://issuetracker.google.com/issues/167352813#comment15
Issue: If the user is logged into google play with an account using a Gsuite email address. The in-app review prompt will fail to work however "hasFlowFinishedSuccessfully" will always return true. I followed all troubleshooting steps to verify nothing else was causing this issue. Switching to a @gmail.com email address as the logged in user in google play fixed the issue. I have not tested this with any non-gmail email accounts.
Suggested fix: Until the bug is addressed by google, make note of this as a known issue in the readme file. It may save some people a lot of time and effort.

Thank you for your suggestion, appreciate that.
I believe that people have to see first closed issues, before going to invest their time in searching for their issue.
https://github.com/MinaSamir11/react-native-in-app-review/issues/25
https://github.com/MinaSamir11/react-native-in-app-review/issues/61
You have a good point and I will update README.
Thank again.

Doing further testing, this issue seems to only happen when you are testing using an internal test track. If you use internal sharing, you can use Gsuite emails in google play and you will be shown the review prompt. I have not tested this in our production build yet but will come back here to report on that soon.


doc installation for kubefedctl

For download part, currently, we have kubefed2. I don't want to make it here because it is obsolete. I just use new command name for download part for future release download.
Fixes #654

@marun

move binaries into dev guide
kubefedctl and kubectl is still kept in user guide

@marun done

Thank you!
/approve
/lgtm


crypto/x509: ParsePKIXPublicKey can not parse 1024bit RSA

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.9.2 linux/amd64
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/parazyd/go"
GORACE=""
GOROOT="/usr/lib/go"
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="x86_64-gentoo-linux-musl-gcc"
GOGCCFLAGS="-fPIC -m64 -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build264632172=/tmp/go-build -gno-record-gcc-switches"
CXX="x86_64-gentoo-linux-musl-g++"
CGO_ENABLED="0"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
https://play.golang.org/p/MPAwSqN2e7
Running with the former key from the official(?) example found at https://golang.org/pkg/crypto/x509/#ParsePKIXPublicKey works fine. However, when trying the same with a 1024-bit RSA public key, the program panics, being unable to parse it.
What did you expect to see?
pub is of type RSA:
What did you see instead?
panic: failed to parse DER encoded public key: asn1: structure error: tags don't match (16 vs {class:0 tag:2 length:129 isCompound:false}) {optional:false explicit:false application:false defaultValue:<nil> tag:<nil> stringType:0 timeType:0 set:false omitEmpty:false} AlgorithmIdentifier @3

goroutine 1 [running]:
main.main()
	/tmp/sandbox830663545/main.go:43 +0x3a0

The difference between both keys are not (only) the length of the prime of the rsas key but the format. Only the first one is a PKIX key (asn1 sequence of AlgorithmIdentifier+BitString and the real key is in the BitString), the second key is a RSA public key (without the AlgorithmIdentifier).
You can see is that one is "-----BEGIN PUBLIC KEY-----", the other is "-----BEGIN RSA PUBLIC KEY-----".

@bpressure Removing "RSA" from the identifiers does not make a difference.

Of cause not, the RSA is an indicator that both keys are serialized in a different way.
The first one was serialized with
type pkixPublicKey struct {
	Algo      pkix.AlgorithmIdentifier
	BitString asn1.BitString
}

the second one with
type PublicKey struct {
	N *big.Int // modulus
	E int      // public exponent
}

This is working as intended, ParsePKIXPublicKey properly parses 1024-bit RSA keys that have an AlgorithmIdentifier as @bpressure has explained.
This StackOverflow post also explains it in more detail.

Is there another way to parse such a key then?

@parazyd This issue tracker is not the place for questions, however I will say that you can use the encoding/asn1 package to parse keys that are encoded like that.

	const pubPEM = `-----BEGIN RSA PUBLIC KEY-----
MIGJAoGBALj23cgwEWD5R6P0SEqcOiKQkiIHaXuFKVZ3KEOb947ZRHqVPvB7It8V
0pZhit2nippBn7e7nUq/FjWVXsERyr98CYMoEWSwLv7rN7N4O0Evio8IPlK7maAG
rE0DPa61vTDm8qkJN1hYMQ0KQZYI2islND53N14NvOh8/lefIFsBAgMBAAE=
-----END RSA PUBLIC KEY-----`
	block, _ := pem.Decode([]byte(pubPEM))
	var pk rsa.PublicKey
	asn1.Unmarshal(block.Bytes, &pk)

In Go 1.10, there will be a ParsePKCS1PublicKey that can do this. (They're rare, which is why the function hasn't existed prior to this.


Point tracking always introduces merge conflicts.

The way we track points currently is that everyone has a row in an MD table, and pull requests have to update the file.
In my experience this has on just about every PR resulted in a merge conflict.
Considering how many PRs we get per day this is rather inconvenient.
I'm filing the issue to start a discussion on how we could potentially improve this.
JSONL
@KennethEnevoldsen suggested that it could be a good idea to use one JSONL file to track points after each contribution.
I'm not exactly a Git genius, but my guess is that this would still result in a bunch of merge conflicts on the last lines.
Perhaps we could have one per person, which would be great, but could potentially cause issues with reviewers.
I think the cleanest solution would be to use one JSONL file for everyone who submits a PR, but their file would not only contain their own points but also reviewers' points on their PR.
For instance this would be a JSONL file for me, that I would append with each PR I submit:
{"user": "x-tabdeveloping", "points": 6, "type": "dataset"}
{"user": "KennethEnevoldsen", "points": 4, "type": "review"}

We could potentially add even more granularity (separate lines for bonus points) so we can exactly track why people have the points.
What do you all think?

a jsonl file would still lead to merge conflict (tracked as edits with n-lines), I would just do a folder with jsonl files (seperate pr. pr).

A single folder with 1 JSONL file per PR sounds like a good option.


How to use onedrive file picker when redirect url is dynamic.

I am using onedrive file picker which works absolutely perfect when the redirect uri is static but when i try to use a dynamic uri it allows me to pick a file from drive but does not returns success and only shows the loader image. I get the following error in browser console.

I am using the following code to call onedrive file picker
$(document).on('click', '#btnOneDrive', function () {
var odOptions = {
clientId: "XXXXXX-XXXXX-XXXX-XXXX",
action: "query",
multiSelect: true,
openInNewWindow: true,
advanced: {
queryParameters<EMAIL_ADDRESS>},
success: function (files) {
/* success handler */
for (var i = 0; i < files.value.length; i++) {
<EMAIL_ADDRESS>files.value[i].name);
}
},
cancel: function () { /* cancel handler */ },
error: function (e) { /* error handler */ }
};
OneDrive.open(odOptions);
});

hi @creedrahul , would you mind sharing me more detail with your story? Are you invoking the picker in a popup? Is your host page is a dynamic url? Basically we assume the host page should be a static url due to the oauth requirement, but you can set the oauth redirect url like:
var odOptions = {
    advanced: {
        redirectUri: "<your url registered on apps.dev.microsoft.com>"
    }
}

Hi @daboxu my base url has query parameters that change with every request made to the page. I have already tried to implement the redirecturi in advance block. Is there any work around that can be used to make dynamic url work.

@creedrahul if your host app is dynamic url and you have another static url in the same subdomain for oauth redirect, then the answer is yes.
Let's say you have a host app at "www.foo.com/dynamic?random=123" page just like how you invoke the picker, and you set the redirectUri in advanced block which is used for oauth redirect to "www.foo.com/pageforoauth.htm", you should be good to go. Does this help?

@daboxu can i redirect to the url as it is. As the values in query string are important in handleing the next in coming data. The file download in my case is a part of work flow and redirecting it to any other page will break the work flow. If i am redirecting from a url like www.foo.com/dynamic?random=123 i want it to return to same url.

@creedrahul unfortunately no, the redirectUri you set in the advanced block is used in the popup window for oauth so you should be safe. As the oauth endpoint will strip out all query param there is no way we can get around of it. If my understanding is correct, the host app will not do redirect right? So basically you just need a separate static page which including the v7.0 js file as well.

@daboxu thanks for your efforts to here me out  and suggest a solution. Let's hope in future onedrive will provide some method to use redirect url with parameters just like google and dropbox.

@creedrahul thanks for your feedback! You can also let us know your voice: https://onedrive.uservoice.com/forums/262982-onedrive so we can better help other developers.

@daboxu Is there a workaround for configuring a dynamic redirect URL? We have a URL which has dynamic values embedded as part of the URI and it wouldn't be possible to separate them as query params.
e.g. http://foo.com//product//productDetails.html
In the above URL, userId and productId are part of the URI and are required in the callback


Hotfix for maximum call stack issue

fixes #39 #45

I tested the changes locally and it also worked for me. Thanks for the quick fix :pray: ! At the moment I was having to upgrade to 6.4 because of this: https://stackoverflow.com/questions/70253373/mui-v5-storybook-theme-and-font-family-do-not-work-in-storybook and this really helps.

May I know the ETA for merging? @UsulPro

@AndreasBBS thanks for the help with testing!


Archived chats appearing in main chat list after incremental sync

I believe this is due to the update() function updating the isArchived property. We can do one of the following:

Make a new DB column to house our localArchived property
Change update() so that it doesn't update the isArchived property

Now there is physically no way for this to occur cause I just automatically hide all convo tiles when they are archived and you are in the main chat list and vice versa


CI Fix pylint hook

  - repo: local
    hooks:
      - id: pylint
        name: pylint
        entry: pylint
        language: python
        types: [python]
        require_serial: true
        exclude: "docs"
        args:
          - "--rcfile=pyproject.toml"

The above configuration works without any issues in python 3.8. I verified the same using the below pre-commit.yml using with
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-python@v2
        with:
          python-version: "3.8"
      # This is required since pylint is used in pre-commit as a local hook
      - name: Install dependencies
        run: |
          python -m pip install --upgrade pip
          pip install -r requirements-dev.txt
      - uses<EMAIL_ADDRESS>
Now we are having issues because, default python version is now set to 3.9.0 since its release. Pylint on python 3.9 raises these warnings which were not raised before.
Screenshot from one of the pre-commit runs on this pull request.

My suggestion is instead of adding these disable comments now, better way would be to fix the python version to 3.8 for running the pre-commit.yml github action. Anyway tox takes care of checking if things work in python 3.9.
We can take pylint issues with python 3.9 in a separate PR.

@girip11 I just tried this a fresh Python3.8 environment, and still get the errors

************* Module tests.test_create_blank_init_files
tests/test_create_blank_init_files.py:14:4: E0401: Unable to import 'py._path.local' (import-error)
************* Module tests.test_configs_work
tests/test_configs_work.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/test_configs_work.py:12:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.test_allow_mutation
tests/test_allow_mutation.py:5:0: E0401: Unable to import 'pytest' (import-error)
************* Module tests.test_running_from_different_dir
tests/test_running_from_different_dir.py:6:0: E0401: Unable to import 'pytest' (import-error)
************* Module tests.tools.test_pylint_works
tests/tools/test_pylint_works.py:6:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_pylint_works.py:11:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.tools.test_flake8_works
tests/tools/test_flake8_works.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_flake8_works.py:12:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.test_setupcfg
tests/test_setupcfg.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/test_setupcfg.py:12:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.test_pyproject_toml
tests/test_pyproject_toml.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/test_pyproject_toml.py:12:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.test_runtime_errors
tests/test_runtime_errors.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/test_runtime_errors.py:12:4: E0401: Unable to import '_pytest.monkeypatch' (import-error)
************* Module tests.tools.test_mypy_works
tests/tools/test_mypy_works.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_mypy_works.py:12:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.tools.test_isort_works
tests/tools/test_isort_works.py:9:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_isort_works.py:14:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.tools.test_doctest
tests/tools/test_doctest.py:7:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_doctest.py:12:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.test_ipython_magics
tests/test_ipython_magics.py:8:0: E0401: Unable to import 'pytest' (import-error)
tests/test_ipython_magics.py:13:4: E0401: Unable to import '_pytest.capture' (import-error)
tests/test_ipython_magics.py:14:4: E0401: Unable to import 'py._path.local' (import-error)
************* Module tests.test_find_root
tests/test_find_root.py:5:0: E0401: Unable to import 'pytest' (import-error)
************* Module tests.conftest
tests/conftest.py:8:0: E0401: Unable to import 'pytest' (import-error)
tests/conftest.py:11:4: E0401: Unable to import 'py._path.local' (import-error)
************* Module tests.tools.test_black
tests/tools/test_black.py:9:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_black.py:15:4: E0401: Unable to import '_pytest.capture' (import-error)
************* Module tests.tools.test_pyupgrade
tests/tools/test_pyupgrade.py:9:0: E0401: Unable to import 'pytest' (import-error)
tests/tools/test_pyupgrade.py:15:4: E0401: Unable to import '_pytest.capture' (import-error)

------------------------------------------------------------------
Your code has been rated at 8.57/10 (previous run: 8.57/10, +0.00)

@neomatrix369 also got errors, I think relying on language: system is potentially pretty unreliable.
These new warnings really look like false positives to me anyway, I'd be inclined to just silence them as is done here (this also means we can upgrade to Python3.9 for our local development environments)

I changed language from system to python in precommit config and fixed python version to 3.8

And yes you are right using language:system is not reliable

Yes, and neomatrix369  got errors using the language: system hook which is on master. I don't know what version of Python he's running locally, but this kind of error can be confusing and discouraging to new contributors so IMO we should fix it asap

I completely agree with you. So I am suggesting to go the two changes I suggested (if those work) so that it will unblock @neomatrix369. Soon we can figure out other ways to fix the pylint warning raised in 3.9.
For instance NamedTuple warning raised by pylint on Python 3.9 can be fixed with dataclasses instead of suppressing.

@girip11 dataclass is python >= 3.7

YesI know but you could add it as a dependency when python 3.6 is used right. There should be some way to specify this in setup.py right.

For instance NamedTuple warning raised by pylint on Python 3.9 can be fixed with dataclasses instead of suppressing.

dataclasses are mutable though, while namedtuples aren't. If the structure was meant to be a namedtuple, IMO it's better to keep it as one, to mark the reason for suppressing the error, and then unsuppressing it when it's fixed upstream

@girip11 A yeah forgot that there was a backport 😅

Dataclasses have frozen attribute which when set to true makes it immutable after object initialization. But anyways I am fine with this PR. I was just suggesting alternate options.

I didn't know that, thanks for sharing! OK, I'll get this and #325 in so we can make a 0.3.1 release and fix #318

Somehow I always feel uncomfortable seeing  suppress comments. That's why I am very strongly inclined towards options that can avoid that.

I didn't know that, thanks for sharing! OK, I'll get this and #325 in so we can make a 0.3.1 release and fix #318

Sure. We can add a backlog issue to address pylint with python 3.9.


Create function to read a web URL

Imported from Bitbucket issue #84, created on 2015-11-13

bb-issue-84

Originally created on 2013-01-09
Make sure to handle non-200 responses.
bb-issue-comment-2801732

Originally created on 2015-12-02
Maybe use the HtmlAgilityPack to do the download? Invoke-WebRequest is too painful.
http://blogs.msdn.com/b/dotnet/archive/2015/12/01/the-week-in-net-12-1-2015.aspx
https://www.nuget.org/packages/HtmlAgilityPack/
bb-issue-comment-23757772

Read-Url.ps1.txt
Test-ReadWebUrl.ps1.txt


Fixed errors in unit tests of 2 packages

Fixed errors in unit tests of 2 packages: pkg/client/common/discovery/greylist and pkg/fab/ccpackager/lifecycle.

Fixed errors in unit tests of 2 packages: pkg/client/common/discovery/greylist and pkg/fab/ccpackager/lifecycle.
Provided signoff as instructed.

Sorry I'm new to github.
Fixed errors in unit tests of 2 packages: pkg/client/common/discovery/greylist and pkg/fab/ccpackager/lifecycle.
Provided signoff as instructed.


Fix an issue where a query may throw an error during a remote read and the precision of a duration data type is changed.

What problem does this PR solve?
Issue Number: close #8601
Problem Summary:
What is changed and how it works?
see https://github.com/pingcap/tiflash/issues/8601#issuecomment-1878393488
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix an issue where a query may throw an error during a remote read and the precision of a duration data type is changed.

Can we add a test with remote read by remote read that triggers this bug?

Can we add a test with remote read (by failpoints) that triggers this bug?

Sure, but it may not necessarily trigger the issue as the sync schema may have already been asynchronously triggered.

Can we add a test with remote read (by failpoints) that triggers this bug?

Sure, but it may not necessarily trigger the issue as the sync schema may have already been asynchronously triggered.

You can disable the schema sync background task by => DBGInvoke __enable_schema_sync_service('false')

/hold

/run-integration-test

/run-integration-test

integration-test added. /cc @JaySon-Huang @windtalker

/run-all-tests

/hold cancel

/run-unit-test

/run-unit-test


[Console-iot] - Unify all projects list results

Type of change
Unifying the results to create a list of resources that contains iotptojects and addressSpaces and can be filtered and paged through.
This is done with adding a kind field to both of the objects.

Enhancement

See README for how to use it.

@drcoolsanjeev FYI

I think the approach looks reasonable, have you tried a POC in Go just to make sure there are no gotchas? I know @lulf was looking at this too, so he might have comments.

Starting a Go POC is next in my list :)


saving current state

How can I manage to save state on rotation change ?
Like I am on Page 1 that has list and decide to go to Page2, but on rotation I am on Page 1. How can I keep position on Page 2 and prevent dissapearing of that Page ?

Paginize does not support device rotation (Activity recreated) currently, I tended to keep the implementation simple and efficient, and Page recovery/restore is not implemented. If your app needs device rotation support, Paginize might not be the way to go (at least for the moment). Or if only one or two of many Pages in your app need device rotation, you can still use Paginize with multiple Activities.
Considering device rotation is quite basic feature for a general purpose framework, it may be added in the future.

@lupajz Checkout out the latest code (a1386602), I have already added support for page recovery/restore. Now you can easily achieve what you want, please checkout the updated demo project.

Stack state is keeped only for rotation??? or even for activity destruction???

@FunnyDevs Device rotation will cause activity to be destroyed, so yes, stack state will be kept in that case.


Hcl2 and Azure support (plus some additional additions/fixes)

Update to Terraform 0.12.30 is a must (from my standpoint) because it allows to access remote state from future Terraform versions,
HCL2 support is also a must
In our case we also use Terraform AzureRM provider and therfore the az CLI seems to be mandatory to have in the image to be able to interact with Azure resources

I am interested in getting TF 13 support and better HCL parsing. Shall we get a fork maintained or try our best to rouse the maintainer ?

I am interested in getting TF 13 support and better HCL parsing. Shall we get a fork maintained or try our best to rouse the maintainer ?

I tried your PR with Terraform 0.13.6 and it didn't work. Do you have time to update it to work with newer versions of TF? Thanks a lot.


BeanInfo doesn't work when there's assymetry in getter vs setter

In Jenkins this shows up at Slave.nodeProperties whose setter accepts
List but getter returns more specific DescribableList.
@reviewbybees

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

Seems dangerous. These things do not work well in DescribableModel and it makes it harder to understand what is going on. I would rather just fix the setter to require the more specific type, or add an overload and make that the @DataBoundSetter.
In the case of Slave there is no such setter. I suppose you mean the @DataBoundConstructor for DumbSlave. So, IMO, deprecate that overload, introduce a new one requiring DescribableList, and make that one the @DataBoundConstructor.

:bee: but I generally dislike asserts in production code - especially ones that can not be hit unless you have some memory corruption :-)

@jglick I did consider that prior to settling on this but it's not that easy. DescribableList isn't really injectable because it's not a typical collection type that has a no-arg constructor.
Creating a separate overload for this property requires us to use a different property name, break compatibility, or allow @DataBoundSetter to define a property name override, which is less desirable.
I think this is the least path of resistance.

And I'd also point out that this getter setter asymmetry is already happening. Before the change in https://github.com/jenkinsci/jenkins/issues/2293 the constructor parameter type was already different from that of the getter. So this change is not making things worse in that regard.

I talked with Jesse offline and he explained to me the context and bigger issue that this touches, which I agreed is a problem that needs to solve. However, I argued that that's more than I'm willing to bite right now and he was OK to look the other way and let me put this in, in order to get other stories moving that hings on this one.


Support for flairs

Would be nice if we could support flairs from different subreddits. Would look something like

Fount this in PRAW's code overview. May be helpful:
 get_flair(subreddit, redditor)
     Return the flair for a user on the given subreddit.

     Requires the modflair oauth scope or user/password authentication as a mod of the subreddit.

     Parameters:
    •subreddit – Can be either a Subreddit object or the name of a subreddit.
    •redditor – Can be either a Redditor object or the name of a redditor.

     Returns:
     None if the user doesn’t exist, otherwise a dictionary containing the keys flair_css_class, flair_text, and user.

I`m going to take a look at doing this.

Hey @michael-lazar, I've been looking at doing this but I'm having trouble finding where the comments and submission text are actually set / drawn to the screen.  In the submissions.py module there are functions such as draw_submission and draw_comment.  In these there are variables called text which seem to then be put into the curses screen.  I've tried adding extra text to these text variables just to see what that would affect and it actually did nothing.  Am I misunderstanding what this section of the code does?

That sounds like you're doing it correctly. Keep in mind that subreddit.py controls the main page, and submission.py controls the comments page. If you're trying to change the behavior of the main page, you need to look at subreddit.draw_item(). That being said, changing the text to something like
text = '{author} test_flair'.format(**data)

should produce a visible output. If it still doesn't, my guess is that python is grabbing files from a different location than you are expecting. If so let me know and we can walk through setting everything up.

Yeah I think Python must be grabbing files from a different place.  Should I perhaps uninstall the system wide rtv that I installed via package manager? Or what would you suggest doing?

On Mar 9, 2015, at 18:22, michael-lazar<EMAIL_ADDRESS>wrote:
That sounds like you're doing it correctly. Keep in mind that subreddit.py controls the main page, and submission.py controls the comments page. If you're trying to change the behavior of the main page, you need to look at subreddit.draw_item(). That being said, changing the text to something like
text = '{author} test_flair'.format(**data)
should produce a visible output. If it still doesn't, my guess is that python is grabbing files from a different location than you are expecting. If so let me know and we can walk through setting everything up.
—
Reply to this email directly or view it on GitHub.

Here's what I've been doing lately. First make sure that your repo is up-to-date.

cd into the top directory (the one with setup.py and README.rst)
type python -m rtv

Explanation:
The -m rtv is shorthand for typing out python rtv/__main__.py. When python sees the lines
from rtv.main import main
main()

inside of __main__.py, it will search the current directory for valid packages first, and then if it doesn't find anything it will start searching through site-packages and pip installs. Since the current directory has a folder called rtv/ with an __init__.py inside of it, that is the one that python will import.

Using python -m totally fixed my problem.  I was running using python rtv/main.py before and for whatever reason it doesn't import from the correct place when running it that way.  Thanks for giving me a hand on this, I'm going to continue working away on adding this feature.

I'm having an issue where as soon as I add a get_flair() call to content.py the program starts immediately crashing with the error message "Could not reach /r/front".  My initial thought was that an exception was being thrown and getting caught by a block that does that but I haven't been able to figure out what's happening.  get_flair documentation doesn't indicate that it would be throwing any exceptions and should just be returning None if the user or subreddit was not found.  Do you have any idea why adding get_flair calls into the split_submission method in content.py would be causing it to crash with this message?  (Note that even when get_flair is called for ANY subreddit, it always says "Could not reach /r/front")

You're correct that the exception is getting caught where it shouldn't be.
content.py, line 361
content = cls(display_name, submissions, loader)
try:
    content.get(0)
except:
    # TODO: Trap specific errors
    raise SubredditNameError(display_name)

Try getting rid of this try-except block
content = cls(display_name, submissions, loader)
content.get(0)

and your error should propagate through.


Typing

i typed some stuff, i made zero logic changes so this shouldn't conflict with anything.

Thanks.


Bump swift-collections to 1.1.1

Information

Package version: 1.0.1
Swift version: 5.10

Behavior
The app upload to App Store Connect fails due to swift-collections, when not using SWIFTCI_USE_LOCAL_DEPS.
This issue has been reported in https://github.com/apple/swift-collections/issues/364.
This package needs to be updated to version 1.1.1.

It shouldn't be necessary for us to increase the minimum version here. In your application you should be able to update to the latest package versions which should pull in the 1.1.1 release if swift-collections.

In many cases, that is correct, but it can cause issues due to caches and other factors.
Since I know there are problems with swift-collections v1.1.0, it would be helpful to explicitly update the minimum version.

I understand how it can still result in build failures but I disagree on bumping the version here. This isn't going to be sustainable across the ecosystem that every single package is bumping their minimum version for a build fix from one of its dependencies.

OK, I will consider explicitly adding a dependency on swift-collections v1.1.1 in my app.
Thanks!


in migrations, class name conflicts should be resolved during registering phase

While still working on 5.1, I stumbled upon some isssue with migrations.
Step 1
I created migration with name 2015_11_28_095622_add_columns_to_inquiries_table. A migration file was generated: 2015_11_28_095622_add_columns_to_inquiries_table.php. Then i ran the migration.
Step 2
Then I decided to add new columns, so I just created another migration with EXACTLY the same command and ran the migration. The file generated had name 2015_11_28_095622_add_columns_to_inquiries_table
The class name in BOTH files were exacly the same: AddColumnsToInquiriesTable
THE OUTCOME:
On my local server on Win7 the two migrations worked fine. On production server (Linux 14.4, Digital Ocean standard droplet), however, I got this:
[Symfony\Component\Debug\Exception\FatalErrorException] Cannot redeclare class AddColumnsToInquiriesTable
this issue made reverting the last migration (the one with duplicate class) impossible.
Suggestion: in migrations, class name conflicts should be resolved during registering phase

This isn't really a bug. Just be careful when naming your migrations.


Added some spacing and a border to the blocks.

I have added a 4px top margin plus a border to make the blocks distinguishable with each other.

I was unable to the css stylesheet to load so it's also inline, not sure if that's a limitation of the backoffice?
Fixes #5

I'm a little torn on this one - in my opinion the preview should appear as close to the live site as possible but I get the need to be able to see the separation of blocks without having to hover/tab through them.

@AaronSadlerUK @rickbutterfield an alternative to this is to add some markup to the Component Views to inject a class based on the IsBlockPreview extension method - that way you can style the padding/margin/border any way you like without having to inject CSS etc.


Can "travis-worker" for openstack backend spin up VM instance using "volume"

Hi All,
In an Openstack environment , would there be a worker support in spinning up VM instance using "volume" as "booting source" besides "image".
As seen from here - https://github.com/travis-ci/worker/blob/master/backend/openstack.go. It appears to me that feature support of "booting from volume"  is not present. Seems this booting option is yet to be added from here - https://github.com/rackspace/gophercloud/tree/master/openstack/compute/v2/extensions/bootfromvolume
Please let me know if my understanding is correct and what are the plans if any to add below options for "openstack" backend.

Select Boot Source --> "volume"
Delete Volume on Instance Delete --> "yes" or "no" (options)

Hi @ghatwala
Could you elaborate on use case behind question?

Hi @michal-at-travisci  - Giving some history here - This request originated from a Travis/power build slowness for "containerd" -  https://github.com/containerd/containerd/issues/3491#issuecomment-518833118.
Analysis proved that the power jobs were failing at OSU whereas passing at PDC ( We have two openstack based data centers ( PDC and OSU ) for supporting power builds at travis-ci.org )
Further analysis of OSU env pointed to HDD being the culprit , alternative suggested was to use SSD available there to spin with VMs via travis-worker hosted there for openstack backend.
As we know worker with openstack backend currently lacks the support to spin up VMs with "booting from volume" option , as that feature support was not added from here -
https://github.com/rackspace/gophercloud/tree/master/openstack/compute/v2/extensions/bootfromvolume
So wanted to know

if anyone at travis has tried adding "booting from volume" support for worker ( openstack backend based ) ?

Is there some sort of whitelist mechanism using which a specific project's travis builds(power only) can be routed to a specific place ( AFAIK , there's just one power queue created at Travis-scheduler which caters to both PDC and OSU  (https://github.com/travis-ci/travis-scheduler/blob/master/spec/travis/queue_spec.rb#L32 ). This workaround if implemented will get the containerd's travis job - https://travis-ci.org/containerd/containerd/jobs/568051827 to PDC environment wherein it always passes.

Whatever happened to all this investigation? :D

@clnperez  - FYI , Above option was needed when Travis workers (for ppc64le) were running on a openstack backend , However post github universe , Travis officially supports LXD containers builds now( workers are at IBM cloud now). There is still the older legacy job queue active at Travis which caters to VM based jobs at "travis-ci.org" , however the plan is move everyone to new backend (LXD) as early as possible. There is a future plan to give option to user to deploy power jobs on LXD containers or VM will let @michal-at-travisci  to add more .


[Pipelines] error produced by individual pipeline steps are not part of the output

See an example at https://github.com/freme-project/basic-services/issues/77#issuecomment-240204372

This was caused by #82, not by the pipeline service. Can we close this?

I also think that sometimes the pipeline does not report errors back to the user. When I find an example then I will open a new issue for it.


Possible elm.json stale dependency

Shall this be anmolitor/protoc-gen-elm  ?

I think elm-protocol-buffers is needed for the Utils for the Int64 type:
https://github.com/anmolitor/elm-protoc-utils/blob/45d1f838513b5724ed0f5a31e9168e073868bf7d/src/Protobuf/Utils/Int64.elm
For comparison: https://package.elm-lang.org/packages/eriktim/elm-protocol-buffers/latest/Protobuf-Types-Int64

Oh... sorry, my mistake.
I thought about your fork of elm-protocol-buffers, but yes... it's a fork. Thanks for the answer!


[CELEBORN-1582] Publish metric for unreleased shuffle count when worker was decommissioned

What changes were proposed in this pull request?
Adding a worker metrics for publish unreleased shuffle count when worker was decommissioned.

Why are the changes needed?
Currently celeborn don't publish the count of unreleased shuffle key which gets lost when a worker is decommissioned. This can be useful for monitoring and configuring the forceExitTimeout.
Does this PR introduce any user-facing change?
NO
How was this patch tested?
NA

@s0nskar, how did you use unreleased shuffle count in production practice?

@SteNicholas We're currently not in production but this will help us tune the forceExitTimeout config better and see if the default value is working for us or not. As we probably won't enable replication for lot of jobs, we want shuffle data to not be lost when worker exits.

@s0nskar, please add the UnreleasedShuffleCount metric in celeborn-dashboard.json file.

@s0nskar, any update for this metric?

Sorry messed up the commit history on this one, reopening the PR again.
@SteNicholas updated the PR with dashboard changes.

@FMX @SteNicholas Getting below error in tests, how can we clean up the metrics cache.
java.lang.IllegalArgumentException: UnreleasedShuffleCount{role="Worker"} is already used for a different type of metric

@s0nskar, you should not invoke addCounter(UNRELEASED_SHUFFLE_COUNT) in WorkerSource.

@FMX ping for review.


doesn't work

Following these instructions:
"Getting started"

Install (just standard procedure of installing global dart packages):
dart pub global activate interactive
Use (just a normal binary):
interactive

Results in this error:
Run: /usr/bin/flutter/bin/cache/dart-sdk/bin/dart [--enable-vm-service=34467, file:///home/me/.pub-cache/global_packages/interactive/bin/interactive.dart-3.4.0.snapshot, --vm-service-was-enabled]
Workspace: /tmp/dart_interactive_workspace_2024-05-14T174404964295
The Dart VM service is listening on http://<IP_ADDRESS>:34467/rR3gWUo72JQ=/
The Dart DevTools debugger and profiler is available at: http://<IP_ADDRESS>:34467/rR3gWUo72JQ=/devtools?uri=ws://<IP_ADDRESS>:34467/rR3gWUo72JQ=/ws

Hi, do you see >>>?
For me I see:
(base) ➜  research_mono git:(master) interactive
Run: /Users/tom/fvm/versions/3.19.2/bin/cache/dart-sdk/bin/dart [--enable-vm-service=62128, file:///Users/tom/.pub-cache/global_packages/interactive/bin/interactive.dart-3.3.0.snapshot, --vm-service-was-enabled]
The Dart VM service is listening on http://<IP_ADDRESS>:62128/ID8JjrzMOfw=/
The Dart DevTools debugger and profiler is available at: http://<IP_ADDRESS>:62128/ID8JjrzMOfw=/devtools?uri=ws://<IP_ADDRESS>:62128/ID8JjrzMOfw=/ws
Workspace: /var/folders/j5/j6ymn7yd70564hzt31pq_0g80000gn/T/dart_interactive_workspace_2024-05-15T082218312739
>>> a=1
1

And btw, check whether you have http proxy set.

no it just dumps me back out to my normal bash prompt
I do not have any proxies.
I'm running on ubuntu 22.04 with dart installed via the instructions on their page using their official binary (not snap)

Hmm, so I guess one way is to run it by source, and add a lot of print here and there (or use your favorite debugger), to see where it gets stucked.

didn't have time to gather/provide actually valuable debug data and it felt rude to leave this open.


Issues with both sidebars, when set to view on website

Information about bug
On the right side bar, the scroll bar and alignment is not okay at all, there are large white spaces and also, the scroll bar is messed up....
on the left side bar, when a page "add to sidebar" instance is created it returns undefined as the name of the page, although i have fixed this and raised a PR...
ALL THIS OCCURS ON THE FRAPPE-WIKI APP

Module
other
Version
Frappe Framework: v14.12.0 (version-14)
ERPNext: v14.3.1 (version-14)
Wiki: v0.0.1 (master)
Installation method
manual install
Relevant log output / Stack trace / Full Error Message.
No response

This scroll bar issue seems to be fixed in commit 6d5a1e2135b18ba4e83c75242fecc109d5668e2c. Can you try installing the latest version of wiki?

Ok ,i will do that
On Mon, Dec 12, 2022 at 9:57 AM Bread Genie @.***>
wrote:

This scroll bar issue seems to be fixed in commit 6d5a1e2
https://github.com/frappe/wiki/commit/6d5a1e2135b18ba4e83c75242fecc109d5668e2c.
Can you try installing the latest version of wiki?
—
Reply to this email directly, view it on GitHub
https://github.com/frappe/wiki/issues/107#issuecomment-1346113224, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXVXPEJ3ZPHXMOT7TIPWC2TWM3SGPANCNFSM6AAAAAASX5U5VA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Closing the issue now. Feel free to reopen if the issue persists.


[QUERY] Is there a way to label (IR) active phonon modes as LO and TO?

Hello,
This is a query rather than an error resolution.
On plotting phonon dispersions, TDEP outputs a "outfile.mode_activity.csv" file which lists the identified IR and Raman active phonon frequencies at the gamma point (sourced from activity.f90).
Unfortunately, I am not very well versed with fortran to understand the calculation being done there, and therefore have the following questions:
1] Are eigenvectors being calculated (from the dynamical matrix)? (I believe the current code is already accounting for the born charges)
2] If so, can you (TDEP) use that to identify and output the respective LO (longitudinal optic) and TO (transverse optic) modes.
Alternatively, Are there any other methods to extract this information from the current version of TDEP?
Thanks in anticipation,
Sreerag.

Hi Seerag,
yes - the IR spectrum "activity" is proportional to the phonon mode dipole
moment, which is the eigenvector multiplied by the Born charges.
Eigenvectors/eigendisplacements are certainly being calculated, and if the
mode dipole is finite then they are IR active. If the displacement is along
q then they are LO, perpendicular TO (I guess you know this).
You can also deduce this from the eigenvector symmetries and the
irreducible representations they belong to (
https://www.cryst.ehu.es/rep/sam.html).
M.
On Mon, Mar 11, 2024 at 8:05 PM Sreerag @.***> wrote:

Hello,
This is a query rather than an error resolution.
On plotting phonon dispersions, TDEP outputs a "outfile.mode_activity.csv"
file which lists the identified IR and Raman active phonon frequencies at
the gamma point (sourced from activity.f90).
Unfortunately, I am not very well versed with fortran to understand the
calculation being done there, and therefore have the following questions:
1] Are eigenvectors being calculated (from the dynamical matrix)? (I
believe the current code is already accounting for the born charges)
2] If so, can you (TDEP) use that to identify and output the respective LO
(longitudinal optic) and TO (transverse optic) modes.
Alternatively, Are there any other methods to extract this information
from the current version of TDEP?
Thanks in anticipation,
Sreerag.
—
Reply to this email directly, view it on GitHub
https://github.com/tdep-developers/tdep/issues/46, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWOT32L3VH6SDIPSCAAY3TYXX57HAVCNFSM6AAAAABEQ4HJNGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TSOJZG4ZTIOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Professor Matthieu J Verstraete
Fellow, American Physical Society
Chair, Steering Committee, European Theoretical Spectroscopy Facility
www.etsf.eu
Alumnus Fellow, Young Academy of Europe  yacadeuro.org

Institute for Theoretical Physics (ITP) Department of Physics
Buys Ballot Gebouw/Building, Princetonplein 5, office
University of Utrecht,  3584 CC Utrecht
ITP Secretariat: +31 30 253 5928 E-mail<EMAIL_ADDRESS>
Group web page (Liège): http://www.nanomat.ulg.ac.be/

Nanomat lab, Q-Mat center, Université de Liège
Département de Physique, Bat. B5a, 4/50
Allée du 6 août, 19 B-4000 Sart Tilman, Liège Belgium
Phone : +32 4 366 90 17

European Theoretical Spectroscopy Facility (ETSF)

Mail<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Hi @idscum and @mjv500 ,
yes, this is implemented, but in tdeptools. There is script that computes exactly what you want: The TO and LO projected mode-resolved Born charge. If a mode only has a TO value it's pure TO, vice versa for LO, otherwise it can be mixed. I used this here in Fig. 3 to estimate the "LO activity" of the modes.
The script is in beta status, it should be functional but it is not properly documented. If you need this feature please drop me a mail and describe your use case.


Private callback does not allow me to create interfaces

Hi! I'm currently trying to swap out go-socket.io with your library (because of the v4 compatibility), but I have problems due to the private structs and private callback typings.
When I want to create an interface for a Namespace / sio.inSocketV4 that's impossible, because I cannot make an On Method definition conforming to the private eventCallback interface definition, even if it matches the signature.
So I would propose to either expose interfaces (probably eventCallback would be good enough) to public or to not return private structs which can't be even embedded in self-defined data types.
But maybe there is already a solution to this which I didn't see yet.

Can you describe your use case in a little more detail. I think I have a fix, but I want to see if it should be applied in other places as well, based in part by your use case.

I am also experiencing this issue. For example, I want to write a handler for an event named myEvent. It appears that I would just write something like this
type myItem string

func (i myItem) Callback(...interface{}) error {
  return nil
}

func main() {
  var item myItem
  server := socketio.NewServer()
  server.On("myEvent", myItem.Callback)
}

I have a feeling that this is incorrect. How would I go about creating a callback for a custom event?

https://github.com/njones/socketio/blob/main/callback/callback_test.go#L128-L139
Does this show you what you want to know? If so, there are more examples in that file.

@njones thanks for the quick reply, that looks more in line with what I expected . I will take a look and report back! Thank you!

Thank you, this seems to be working for me! I am open to creating a PR to include this in the README as subscription custom events is a common use-case of socket.io. @njones do you have a preference of how that should look?

I think something like the example above would be ok. I would put it in it's own header and code block. It would be awesome to review your Pull Request.


Item tracker

Hello,
is there any way to display items with floors on an OBS overlay?
Thank you!

I think what you’re looking for is the item tracker. This is more akin to a match history for isaac, its meant for a different functionality

Hello, is there any way to display items with floors on an OBS overlay? Thank you!

As Shaniquaniminiquani said, there is a specific item tracker app already available that every streamer use. Unless you really want one from this app (i was indeed working on a feature to open a second window with just the current run), i recommand you tu use the one mentionned by Shaniquaniminiquani, because unfortunately i'm not very active on this project for now


XML character references are not unescaped/escaped

XML character entity references, e.g. Å ("Å"), and XML numeric character references, e.g. Å ("Å"), are not unescaped/escaped by XML.unescape and XML.escape methods.

Something like the following may help (for unescaping hexadecimal numeric character references):
function unescape_unicode(s::AbstractString)
    i = firstindex(s)
    while (m = match(r"&#(x)(\w{2,4});", s, i)) !== nothing
        s = replace(s, m.match => unescape_string("\\u$(m.captures[2])"))
        i = m.offset + 1
    end
    return s
end

Hmm, these entities need to be defined in the DTD, correct?  I think we'd need (un)escape methods that take in an XML.DTDBody as well as the string.

Ah - yes - that's right - my ancient memory of XML, and in particular HTML, led me to believe that they were built-in also in XML, but I see now that XML only defines five entities - and all of the HTML-like entities are mostly/solely defined for HTML: https://en.wikipedia.org/wiki/List_of_XML_and_HTML_character_entity_references#Standard_public_entity_sets_for_characters
Perhaps one could just have some html-convenience escape methods...

Or perhaps provide something convenient for getting common DTDs like http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd


Mouse+Keyboard = err....

currently there is a bit of a bug where if you use the mouse and the keyboard at the same time weird stuff happens.
will fix this at a later date, probably.

I have no idea how to solve this. i've even tried disabling the mouse while the keyboard is being use and disabling the keyboard while the mouse is being use.
doesn't seem to help in both cases.
maybe i need to wait?????
i'll try to use the inb_w functions instead and see if that helps any.

I have no idea how to solve this. i've even tried disabling the mouse while the keyboard is being use and disabling the keyboard while the mouse is being use. doesn't seem to help in both cases.
maybe i need to wait????? i'll try to use the inb_w functions instead and see if that helps any.

that would be a no.

disabling and re-enabling interrupts doesn't seem to help either, left the code in though.
"disable interrupts?"
maybe someone can explain to me why this isn't working.
mean while i need to figure out why i have 8 DRAM controllers and 7 Display Controllers but no AHCI controller.

further bugs that might be related, it appears there is a general protection fault if the mouse is moved too early.

further bugs that might be related, it appears there is a general protection fault if the mouse is moved too early.

if you wait a second after the screen is cleared than it doesn't happen.

fix


core.BackupEntry is reconciled multiple times in a row

How to categorize this issue?

/area robustness quality
/kind bug
/priority normal
What happened:
In some gardenlet logs I observed, that a core.BackupEntry is always reconciled multiple times in a row.
time="2021-03-05T06:49:06Z" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=garden-project/some-shoot task="Deploying backup entry"
time="2021-03-05T06:49:06Z" level=info msg="Backup entry has been successfully reconciled." operation=reconcile shoot=garden-project/some-shoot
time="2021-03-05T06:49:06Z" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=garden-project/some-shoot task="Waiting until the backup entry has been reconciled"
time="2021-03-05T06:49:06Z" level=info msg=Starting backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Waiting until the backup bucket is reconciled"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Deploying backup entry secret to seed"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Deploying backup entry extension resource"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Waiting until backup entry is reconciled"
time="2021-03-05T06:49:06Z" level=info msg=Finished backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation"
time="2021-03-05T06:49:06Z" level=error msg="Failed to ensure gardener finalizer on backupentry: Operation cannot be fulfilled on backupentries.core.gardener.cloud \"shoot--it--some-shoot--backup-bucket-uid\": the object has been modified; please apply your changes to the latest version and try again" backupentry=shoot--it--some-shoot--backup-bucket-uid
time="2021-03-05T06:49:06Z" level=info msg="Error syncing backupentry garden-project/shoot--it--some-shoot--backup-bucket-uid: Operation cannot be fulfilled on backupentries.core.gardener.cloud \"shoot--it--some-shoot--backup-bucket-uid\": the object has been modified; please apply your changes to the latest version and try again"
time="2021-03-05T06:49:06Z" level=info msg=Starting backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Waiting until the backup bucket is reconciled"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Deploying backup entry secret to seed"
time="2021-03-05T06:49:06Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Deploying backup entry extension resource"
time="2021-03-05T06:49:12Z" level=info msg=Succeeded backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation" task="Waiting until backup entry is reconciled"
time="2021-03-05T06:49:12Z" level=info msg=Finished backupentry=shoot--it--some-shoot--backup-bucket-uid flow="Backup Entry Reconciliation"

What you expected to happen:
If the BackupEntry was reconciled once, it should not be reconciled another time directly afterwards.
How to reproduce it (as minimally and precisely as possible):

Create a core.BackupEntry (e.g. indirectly by creating a Shoot on a Seed that has backups enabled)
Observe the gardenlet's logs.

Anything else we need to know?:
Environment:

Gardener version:
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

Took a bit of a look into this as part of #3838, seems it's because of this code: strategy.go:84
The gardener.cloud/operation=reconcile is kept on the resource until it is completely reconciled so any updates to it (even status and annotation updates) will increase the generation and re-add it to the reconciliation queue.
I could fix it but, separately from #3838 if it is not necessary there.

Interesting @plkokanov, I'm not sure why this is done differently compared to the operation annotation on the Shoot. Probably it should be removed right away so that the controller does not have to do it itself?

Yes I think removing it right away makes sense. As I see for the shoot, the operation annotation is directly removed after the generation of the resource is increased. So I will try that out.

/assign @plkokanov
as you said you want to work on this.

Fixed in #3880
/close


Keccak sponge sha256 config

Add a config that uses the keccak sponge directly for the challenger and sha256 as the hasher along with a recursion benchmark using it.

I don't fully understand the context, but my impression is that this seems fairly specific to your use case, so I'm not sure if it'd be useful to have it in the main library.
Come to think of it KeccakGoldilocksConfig is also a bit specific to our setting, so perhaps it could be moved somewhere else.

Sure, it can totally live somewhere else.


[MM-T1282] Add e2e test coverage for default behaviour when an empty site name is configured

Summary
Adds an E2E test to cover the behaviour of the 'Site Name' customization configuration option (and default value).
Ticket Link
https://automation-test-cases.vercel.app/test/MM-T1282
Release Note
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Thanks all!


Not working with russian text

Hi.
Not working with russian text, english working normal.
If i trying find something on Russian, he return all position

Hi @Alexmg86. Can you please send me through a sample sql file to the email address on my profile? I will see what I can do.

Thank you very much for sending through the file!
Can you please update to the latest version (2.0.11), and follow the instructions here to change the Search Driver to the new FuzzySearchUnicodeDriver. This should hopefully fix your issue.
Please note: The FuzzySearchUnicodeDriver does not sanitize your query search before using it in a MySQL query so you will have to ensure it is sanitized yourself.
If you need more help getting that to work, let me know 😄 .

Nice. Big thanks!!!

Great! Thank you!


Fix issue API returning 404 on details page when id is actually valid

Can you elaborate what values failed? Your code essentially does the same thing as it did before, I fail to see the difference. Thanks!

Uhm.. on detail view page the component does a request to https://docker.localhost/api/media/find?ids=1 <-- however ids=1 is not an array with the current logic so it returns immediately a 404...

Hi! Thanks for the PR, but the issue lied in the front-end instead. Fixed it and released in version 1.0.1. Thanks for letting me know!


postgres-exporter error

When you enable postgres-exporter it will start showing this error in the logs.
ts=2022-08-22T08:36:52.303Z caller=collector.go:196 level=error msg="collector failed" name=bgwriter duration_seconds=25837.993198715 err="sql: Scan error on column index 2, name \"checkpoint_write_time\": conv │
│ erting driver.Value type float64 (\"3.23725841e+08\") to a int: invalid syntax"

Testing this locally, it would seem that it's working as expected.

Looks like it's an actual bug with postgres_exporter https://github.com/prometheus-community/postgres_exporter/pull/666
This is fixed with version 0.11.1 which was released a few days ago.  So this is already fixed in this helm-chart and in tobs.


Remove reliance on PSN

At the moment everything is geared towards PSN, with some really messy code to parse a channel for ID's.  This needs to abstracted out at least one layer so we can also use XBL gamertags.

Working on this and have a question... It seems that the reliance, at the moment, on PSN is parsing the PSN Channel - correct? Which, I will be totally honest with you, I still don't exactly understand why the project is doing that. I don't mean that in an insulting way. I am merely stating my ignorance on the matter.
After fiddling around a bit, I have managed to get some XBL/PSN dynamics working. Not ready for a pull request yet, cause I want to get all the changes implemented and test it out. You know, see what I break first. LOL.

So.... the server has a channel in which is listed everyones PSN, the games they play and their timezone (if you take a look at #psn-dev on my test server you can see typically what it looks like).  Unfortunately this was cumbersome and they started to make channels specifically for each game e.g. #ow-psn.  I wanted the destiny lookup to be able to use someones discord tag and map it to their PSN, rather than they have to browse #psn, find the person they are interested in and then use the PSN on the command.  With the #psn channel already containing this information, I just hacked together a quick parser.  This then grew into the 'players' command which allows you to lookup all those who have registered an interest in a game and removes the need for the per-game PSN channels.
Personally, I'd prefer to have this in a database (or something similar), but the server maintainer is keen to keep the #psn channel, so for now it's going to have to stay, but I am looking at abstracting where the info comes from, so you can use a DB or a channel.

This has been added to master


CQL Query support

Added initial support for cql queries.

Coverage decreased (-0.07%) to 42.21% when pulling 6b935d75c0a826fdf1c0e9a1ebca02b1e7dc91fd on charlesyu108:python-bindings into 93148af36c5c83f86017b023d14c5e782faaa500 on locationtech:python-bindings.


Cleaning up main notebook

This notebook looks great so far! I did a couple things to clean up the code and improve readability:

added loops to replace repetitive code. When there is a long chunk of similar code, usually it can be replaced with a for loop.
replaced while loops with for loops when possible.

for i in range(5):
   # do something

is preferable to
i = 0
while i < 5:
  # do something
  i += 1

descriptive variable names - it's always better to write out the full variable name rather than abbreviating or explaining in a comment. velocity_of_earth = distance / time is better than v_e = d / t. This generally enhances readability and prevents bugs.
all caps for constants. Whenever there is a constant value like a number or a string that never changes, the convention is to name the constant with all caps. For example, I would write SPEED_OF_LIGHT = 100 instead of speed_of_light = 100. This just helps to distinguish the constants from other variables.
Python styling - in Python, the convention is to use underscores like this_is_a_variable over camelcasing like thisIsAVariable. This is just a nitpick.

So many corrections in so little time, 'incredible'. I gotta admit Its a little embarrassing although I very much enjoyed seeing how you used the for loops to eliminate a lot of unnecessary code, such things as the table's 90 button's.

What you've done is definitely impressive for someone relatively new to Python! You'll develop the intuition for simplifying code pretty quickly :)


Watchcat install and update package errors

Watchcat throws errors "uci invalid argument" Anyone have a way to fix this?  Also getting package update errors when trying to update lists.  Thanks!
apk add watchcat

(1/1) Installing watchcat (1-r17)
Executing watchcat-1-r17.post-install
OK: 82 MiB in 520 packages

Errors
uci: Invalid argument
uci: Invalid argument
uci: Invalid argument

Not enough info provided


Failing to install requirements

I noticed that you have in your requirements.txt file some lines that are not packages but rather have a different format, e.g:
boltons @ file:///croot/boltons_1677628692245/work
Brotli @ file:///tmp/abs_ecyw11_7ze/croots/recipe/brotli-split_1659616059936/work

When trying to run pip install on them it fails with errors like:
ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/croot/boltons_1677628692245/work'
Any idea how to resolve this?

I think if you create the environment from enviroment.yml then you don't need to run pip install.

I think if you create the environment from enviroment.yml then you don't need to run pip install.

Even the first step seems to be not working

Hello, I also encountered the same problem. How did you solve it?

In the requirements.txt, I put "#" (comment it) in front of any thing start from "@ file:"
for example,
requests #@ file:///croot/requests_1678709721434/work


Update deps for eventing to master.

Proposed Changes

Copy old registry names used here but moved in upstream.

Release Note

NONE

/assign @mattmoor

/lgtm


ArticleActions:Follow feature does not work properly on article pages

ArticleActions.vue.
computed:

methods:

When the toggle button is clicked twice, the method works but the text of the toggle button does not change.
Following on the article page indicates that all users of other posts are followed.

When loading the article page for the first time, only User, Comments, and Article are set, but the change is the profile, so it does not bring follow results when loading.
This could be solved by loading setProfile, but I hope there's a better way to make it load longer.

Thought the Profile should be loaded like in the profile.vue
mounted() {
    this.$store.dispatch(FETCH_PROFILE, {
      username: this.article.author.username
    });
  },

and const action should instead action  be profile.following
const action = this.profile.following
        ? FETCH_PROFILE_UNFOLLOW
        : FETCH_PROFILE_FOLLOW;


Auto-generate JSON model specs

Per suggestion from @shotgunosine, it would be great to have a utility that auto-generates a best guess at an immediately executable BIDS-Model generated by scanning a BIDSLayout (or at least a skeleton that users could easily populate themselves).

Added in #119.


Added HPE volume plugin listing to docs

Added an entry in the plugin docs for HPE's volume plugin.

LGTM, thanks!


UAA asg setup script is flaky

Describe the bug
UAA won't come up because of the security group setup script.
To Reproduce
Deploy and wait for stuff to come up.
In my tests, it's been 50/50
Expected behavior
A clear and concise description of what you expected to happen.
Environment
KubeCF 1.0
GKE 1.14 or 1.15

Logs
2020-03-12T09:51:00.399444857Z Waiting for the API to be accessible...
 I
2020-03-12T09:51:01.112946456Z   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
 E
2020-03-12T09:51:01.112995153Z                                  Dload  Upload   Total   Spent    Left  Speed
 E
2020-03-12T09:51:01.113000848Z
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
 E
2020-03-12T09:51:01.113006878Z curl: (60) SSL certificate problem: unable to get local issuer certificate
 E
2020-03-12T09:51:01.113012038Z More details here: https://curl.haxx.se/docs/sslcerts.html
 E
2020-03-12T09:51:01.113017734Z
 E
2020-03-12T09:51:01.113022296Z curl failed to verify the legitimacy of the server and therefore could not
 E
2020-03-12T09:51:01.113027188Z establish a secure connection to it. To learn more about this situation and
 E
2020-03-12T09:51:01.113031986Z how to fix it, please visit the web page mentioned above.
 E
  undefined

will be solved by #480


Move desktop

Hi there, may I ask if it supports moving desktops, like in your screenshots, moving the Dev one to the first of the list?
In case it's not yet supported but you are interested, this PowerShell library supports that and you may take it as a reference: https://github.com/MScholtes/PSVirtualDesktop/blob/master/Module/VirtualDesktop.ps1#L357

Thanks for your contribution. It's been published in release v6.3.0.
🥳


Implement Debezium MongoDB connector

What changes are being made and why?

Created Debezium mongodb connector (plugin)
Implemented Capture, Trigger, RealtimeTrigger for mongodb plugin
Created unit test for tasks from above
Added CustomParameterDeserializer to deserialize String into Map and Object into Map for fields: "after" and "before" - In Envelope class
Added JsonSetter in Message class to set "collection" parameter into "table" field (As MongoDB has no tables)
Changed class ChangeConsumer to put empty map in result map instead of getting error if field "after" in Envelope class is null
Changed AbstractDebeziumTask class to put in properties hostname and port only if they are not null, as for mongodb needs connectionString instead of those fields. In providing user to define connection string instead of programmatically creating, it simplifies code. ConnectionString can have different types of connection Standart and SRV, can be sharded or replica_set

How the changes have been QAed?
triggers:
  - id: trigger
    type: io.kestra.plugin.debezium.mongodb.Trigger
    snapshotMode: INITIAL
    connectionString: mongodb://mongo_user:mongo_passwd@<IP_ADDRESS>:27017/?replicaSet=rs0

  - id: realtime
    type: io.kestra.plugin.debezium.mongodb.RealtimeTrigger
    snapshotMode: INITIAL
    connectionString: mongodb://mongo_user:mongo_passwd@<IP_ADDRESS>:27017/?replicaSet=rs0

Setup Instructions

Start mongo container in docker-compose-ci.yml file
Execute mongo.js file (data/mongo.js in project folder) in MongoDB database (Can be used through Intellij Idea database tools plugin)
Execute workflow

Not sure if it's only with MongoDB, but seems like maxRecords not quite works now


Display Boost level for all Curve vaults

Is your feature request related to a problem? Please describe.
Users would be happy to know how boosted their yield is on Curve vaults, and Yearn's comparative yield compared to other protocols.
Describe the solution you'd like
Display Boost level for all Curve vaults on vaults UI.
Describe alternatives you've considered
N/A
Additional context
N/A

This has been implemented


New version: StandardNotes.StandardNotes version 3.194.0

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-04-24 18:41:00.934 [FAIL] Installer failed security check. Url: https://github.com/standardnotes/app/releases/download/<EMAIL_ADDRESS>Result: 0x80004005

(Automated response - build 877.)

Hi @russellbanks,
Do you approve of these changes?
(Automated response - build 877.)


fatal error: concurrent map writes

Hi, thanks for sharing the code. I often get a runtime error after running sping for sometime. Please check the output in this link.
I have PR'ed #7 which I think should be a fix for this problem, treating timeSyncs as a struct with sync.Mutex as its member field. With this fix I have been running it for several hours now.

Looks like the NTP calibration is a little broken. I'll take a look
On Mon, 8 Mar 2021, 08:22 ТАМИХИРО ЛЕЕ<EMAIL_ADDRESS>wrote:

Hi, thanks for sharing the code. I often get a runtime error after running
sping for sometime. Please check the output in this link
https://github.com/benjojo/sping/files/6099816/sping.log.
I have PR'ed #7 https://github.com/benjojo/sping/pull/7 which I think
should be a fix for this problem, treating timeSyncs as a struct with
sync.Mutex as its member field. With this fix I have been running it for
several hours now.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/benjojo/sping/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALPK4TGH7F2KCCKAMRJ363TCSCLXANCNFSM4YY6DSVQ
.

Merged your PR, thanks


Fix: warnings on build due to renamed assemblies.

I had a conversation with Unity QA around a different issue and accidentally found a solution to the issue here: Unity doesn't warn about non-matching assembly names in builds when those assemblies are in an "Editor" subfolder. Turns out the "magic folder naming of old ages" still is in effect in some places. So until in some far future Unity fixes it, this is a workaround :)
Fixes #16 @StephenHodgson

Great find! I'll test and merge soon

@hybridherbst you rock man


crash after Get-KeePassEntry

I have been using the powershell script for over a year and today I noticed that something is not working. after debugging found that the script crashes after Get-KeePassEntry. So I don't understand what the root cause of this behaviour. During this year I didn't change windows, powershell, ps module versions or something else.
my code is
`  New-KeePassDatabaseConfiguration -DatabaseProfileName 'KeePass' -DatabasePath $dbpath  -UseMasterKey -Default
$keepasskey = Get-Content $global:masterpass -raw
$Securekeepasskey = ConvertTo-SecureString $keepasskey –asplaintext –force
Get-KeePassEntry -AsPlainText -DatabaseProfileName 'KeePass' -MasterKey $Securekeepasskey `
after that I get an error

I see than error is like in this issue https://github.com/PSKeePass/PoShKeePass/issues/148 but I only get entry, not update

I'm getting exactly the same issue on Get-KeePassEntry

It looks like the old KeePassLib included in the module (2.39.1) is unable to handle some recent changes to the kdbx file format.
Here is a quick and dirty workaround:

Download the newest KeePass release (2.55 as of today) and extract KeePass.exe from the zip.
If you have KeePass installed on your machine, just take C:\Program Files\KeePass Password Safe 2\KeePass.exe.
Save KeePass.exe as KeePassLib_2.39.1.dll in the bin subdirectory of the PoShKeePass module
(usually $Env:UserProfile\Documents\WindowsPowerShell\Modules\PoShKeePass\<IP_ADDRESS>\bin\KeePassLib_2.39.1.dll when the module is installed at user profile scope and the Documents folder is not redirected).

A proper fix would be to obtain/create a build of KeePassLib.dll for the current KeePass version and release a new PoShKeePass version with this dll included.


CustomMicrophone does not work on Windows

CustomMicrophone does not work due to a known issue with CMUSphinx.

Does that mean that we're un-recoverably broken on Windows.
I mean, that CustomMicrophone doesn't diverge that much from Microphone.

It's not as much a problem with our implementation as Microphone. Whenever the user says "ok idea", an IllegalStateException gets thrown. CMUSphinx is working on a patch.

I suppose the fix for this is to recompile CMUSphinx when the next version is released.

hi. is there a patch for this available @breandan ?

Hi @jskye, there is not yet a fix for Windows.

ok thanks for the reply. this only blocks the use of two LiveSpeechRecogniser's at once correct?

Yes, this issue should not affect most commands, but it will crash when you say, "okay google". I have not discovered a successful workaround.

Hi @breandan, I report same issue here. Any news on this?

We are planning to replace CMUSphinx with another ASR library, which should (in theory) fix this issue. Please follow #52 for updates.


Proxy code training results

Hi, is it possible to share the reconstruction results / visuals from only stage 1 proxy code training? (with cross-entropy loss) It would be very helpful to see how much we can get with only cross-entropy on the VQGAN codes;
Thanks!

Hi,
Thank you all for your interests in our work and patience. We have released the two-stage training code and feel free to check it. Hope this help addresses your question.


Fix icon clipping and hover color

Fixes #61

Thanks @markmals (and @elijahzsmith for creating the issue)


Proposal: Restructure command entry-point

Goals
This proposal attempts to do the following:

Provide clear separation between core business logic: Action and Command. This allows for Action to be end-to-end tested with mocked dependencies. It also allows decoupling from cobra.Command (not a big deal). This ultimately means we can execute tests that cover the entire execution flow except for CLI parsing.
Propose a replacement for IoC patterns of context.<GetComponent> with proper dependency injection. Setting up DI itself does not accomplish this goal. However, the idea is we can now start ensuring that most dependencies can be constructed prior to core business logic execution.
As a consequence of 2, we avoid dependency sprawl and are able to map out a complete object graph dependency tree for each command.

Changes outlined

The action package (used to be builder) is now to be a general-purpose interface-only library that allows decoupling from CLI frameworks. Action.Run only accepts: context, flags, and args -- which is what you'd expect for a general CLI command interface. flags is defined as a generic type that allows strong typing on the expected command flags.
With Action being cleanly defined, we can now construct an Action exactly when needed to -- when a command is fully parsed and business logic is ready to be executed.
Dependency injection is leveraged to allow building up of common components that can be reshared across commands. Validation can be done at these stages for any invariants that need to be established prior to business logic execution.

Examples included
This PR includes a small example that demonstrates the final proposed design.

init and deploy as two separate commands in the new design
up as the new composite command pattern. Notice that composite action has been removed in favor of procedural code.

Love the approach here, @weikanglim.  I think we should go forward with this.

One thing I am qurious about here is the new library we are adding google/wire.  That tool is still in beta and it might be OK to use it for now, but, once azd is going GA, are we going to set a guideline/rule for the external dependencies to be GA libraries only?

I'm not sure how to think about google/wire. We could find other replacements for it. While the version indicates beta, they consider themselves feature complete:

As of version v0.3.0, Wire is beta and is considered feature complete. It works well for the tasks it was designed to perform, and we prefer to keep it as simple as possible.

https://github.com/google/wire#project-status

One thing I am qurious about here is the new library we are adding google/wire. That tool is still in beta and it might be OK to use it for now, but, once azd is going GA, are we going to set a guideline/rule for the external dependencies to be GA libraries only?

I don't think we'll make a rule like this.  I think taking a dependency on a non-GA library is fine - we can ensure via testing on our side that the parts of the library we care about are stable.  Unlike a SDK where our choices "leak" to consumers or we have to worry about dependencies being upgraded to newer versions via a package manager, for azd we build the binary, test it, and then it doesn't change until we ship a new version.  I think we want to enforce some quality bar on dependencies, but I don't think an input to that bar is: "It has to be GA".

I'm not sure how to think about google/wire. We could find other replacements for it. While the version indicates beta, they consider themselves feature complete:

FWIW, I really love how wire works for us here.  From my reading of the diff and understanding, all the "complex wire stuff" happens when we need to generate a new file that actually builds the injectors (which should be small, since we only need to do that when adding more dependencies to our IoC "context") and the result is a very simple file that contains straight line generated code with no "wire runtime".  The only wire stuff that remains is the sets that each command exposes.  I think if we wanted, we could even we could move the code that builds the wire.Set's into wire.go which is conditionally complied and then every other file would be oblivious to wire.


Delphi Clipper2 Example1 and Example2

I had to add to the Project View Source
Clipper.RectClip in '....\Clipper2Lib\Clipper.RectClip.pas',
To get them to compile :)
Love the concept - thank you Angus.
I too live in Australia (Port Macquarie).
I have a real need to nominate two points in code in a closed complex polygon (TPoints) and splice the polygon into two separate polygons -  closing polygon one with the nominated points and staring polygon two with the nominated points.
I believe your library can help me with this.
Can you show me some Delphi example code to do this please - thanks in advance.
PS I would love to be able to email you Angus, here is my email
<EMAIL_ADDRESS>

I had to add to the Project View Source
Clipper.RectClip in '....\Clipper2Lib\Clipper.RectClip.pas',
To get them to compile :)

Hi Andy.
RectClip isn't used in either of these examples (and I have no problems compiling).


It doesn't work in combination with scroll-snap-type mandatory

When using vue-scroll-to, it doesn't scroll smoothly when also having the css property 'scroll-snap-type: y mandatory;'

Try toggling off scroll-snap-type before scrolling, and then toggle it back on in the completion callback.

Thanks, I tried that, and it works!


Add ingress class name to example-ingress.yaml

The reason for this pull request is described in the issue: https://github.com/kubernetes/website/issues/46228
The missing ingress class name property in the example-ingress.yaml file present in 6 different locales causes the created ingress not to work properly. Adding ingressClassName: nginx fixes the issue.

@24online24 Thanks you for your contribution.
Could you please sign the CLA before the PR can be reviewed.
You can follow the steps documented here: CLA Steps

Explicit hold until this PR only covers one language
(Ok to unhold once that's done)
/hold

@dipesh-rawat Thanks a lot for the guidance and warm welcome!
I have now changed the commit to only modify the English version of the file.

Great catch. I think there may be some IngressControllers that do not use this, but it should be the preferred way to set things up.
  ingressClassName	<string>
    ingressClassName is the name of an IngressClass cluster resource. Ingress
    controller implementations use this field to know whether they should be
    serving this Ingress resource, by a transitive connection (controller ->
    IngressClass -> Ingress resource). Although the
    `kubernetes.io/ingress.class` annotation (simple constant name) was never
    formally defined, it was widely supported by Ingress controllers to create a
    direct binding between Ingress controller and Ingress resources. Newly
    created Ingress resources should prefer using the field. However, even
    though the annotation is officially deprecated, for backwards compatibility
    reasons, ingress controllers should still honor that annotation if present.

So I think it's a good call to add it to our examples.
/lgtm

/retitle Add ingress class name to example-ingress.yaml
We don't mention issue references in PR titles (as these can / will end up as commit messages, and that causes problems for forks of this project)

/unhold

/remove-language es
/remove-language ja
/remove-language ko
/remove-language pt
/remove-language zh

I don't understand the issue we are solving with this PR. Do you mean the manifest doesn't work if the ingressClassName is not specified? That is contradictory to what the API spec says. Maybe the implication is that the field must be specified? Then we need to revise the API spec so no one will step into this again.

I don't understand the issue we are solving with this PR. Do you mean the manifest doesn't work if the ingressClassName is not specified? That is contradictory to what the API spec says. Maybe the implication is that the field must be specified? Then we need to revise the API spec so no one will step into this again.

Ingress without an ingressClassName works, but not defining one is discouraged if not actively deprecated.
This avoids implying it's OK to omit the name of the IngressClass.
/approve


Add a var for ANSIBLE_MERGE_BEHAVIOUR=merge

We would like to have this be available as a variable to add to group_vars so it doesn't require doing this:
ANSIBLE_HASH_BEHAVIOUR=merge ansible-playbook -i inventories/site.yml confluent.platform.all --tags kafka_broker -l site-kafkatest-01 --skip-tags validate_hash_merge

cp-ansible/roles/common/tasks/main.yml requires this to be set in ansible.cfg or it fails.
- name: Confirm Hash Merging Enabled
  assert:
    that: lookup('config', 'DEFAULT_HASH_BEHAVIOUR') == 'merge'
    fail_msg: "Hash Merging must be enabled in ansible.cfg"
  tags:
    - validate
    - validate_hash_merge

We use a central repo and we cannot have hash behaviour = merge set globally in ansible.cfg, or it will cause problems. It is difficult to only set this for one of our collections or repos. I know other issues have been reported on this in the past.

Also, I believe DEFAULT_HASH_BEHAVIOUR is deprecated as of 2.13...

This issue is a duplicate of https://github.com/confluentinc/cp-ansible/issues/474, and I think changing this would be a huge task in cp-ansible. That's why I marked the other issue as an "epic" back then.
But I think there is a new team at Confluent maintaining this project now. Maybe @mansisinha or @rrbadiani could take a look and/or forward the request to a product owner in Confluent?

@prabhamanepalli please have a look. We can analyse this more and include in our 2024 FF roadmap.


build: update to lnd v0.17.0 and latest version of Pool

We point to the rc for 0.17.0 for now. Also we had to update Pool as well, due to a replace directive inserted into lnd.

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

Replaced by https://github.com/lightninglabs/lightning-terminal/pull/645.


Set up project board

Wasn't it already there?
https://github.com/users/kkarakas/projects/1
Did I do something wrong?

The link leads to "Page not found" - your project is set to private!

Opened it! Thank you for your notice.


Add gpu dev container support

Add a new docker image to handle gpu dev container
Cleaned up some commas and warning messag

@guzman-raphael Not sure why it not letting me add you as reviewer so pinging you here.

I also remove vscode user since there was permissions issue with the files and my rootless docker system which maps file permission of my user to root inside the container.