text:
Config Option for Soulbound to not require souls

Hello, I was wondering if it would be possible to add a config option that would make the soulbound enchantment not require souls and instead always return the soulbound item to the player's inventory if the player dies.
The reason I would like this option is because I wanted to make most legendary items on my server soulbound so players don't have to worry about losing their hard earned legendary items without having to gather souls.
Thanks for reading!
Edit: Posted in Spigot Forums instead

This is definitely the best place to open issues - I apologize for not replying in a timely manner.
I've been considering a few different ways to get this idea to work without completely changing the behavior of the plugin and not cluttering up the config. I'll think on it some more and get back to this.

Fixed in 57f8300
Latest master available on the Jenkins if you'd like to test: https://jenkins.egg82.me/job/egg82/job/AdvancedEnchantments/job/master/


text:
Weighted QAOA

Currently, light_cone_qaoa disregards the weights in a given graph. This PR adds "weights" data to the coefficients of Paulis as well as to the calculation of ZZ expectation values to support energy calculation of weighted graphs.
For a given graph to be treated as a weighted graph, the edges must have a "weight" attribute. For the edges without this attribute, the weight is assumed to be 1.0.
note: the individual node weights are not taken into account, only edge weights.

Thanks Ayse, this looks almost good to go. Can (i) go over the suggestions and (ii) leave the notebooks (NB) out of the PR? (I.e. do not modify them.) It seems like you did not change the NBs expect from removing the output which makes it harder to refer to them over GitHub.

Thanks a lot for the review! I went over the feedback and applied changes.

source: gharchive/pull-request

license_type: permissive

license_source: github-api


text:
Tab Group Order Botched

Just tidied 30 something tabs, nothing happened so I thought TT was broken, checked releases, nothing new so hit CTRL SHIFT T a few times and tried again, same. Doing a search for one of my tabs I found a huge group of 60+ tabs (it tidied twice) but not at the top as I expect but way down in the middle.

If it helps I have about 600 tabs in my dash.

I have noticed this as well, and will need some time to dig into the issue. Thanks @iKlsR !

This only happens when you have it grouped by date, yes?

@briangonzalez Any time to look at this yet or an eta? No rush! :)  but I'm dying over here

Haha, I'll find some time today :)

I am fixing this now!!!!

@iKlsR This has been fixed, but I need to wait on Google to do transfer ownership before I can release.
For now, you can grab it here and install it manually.
https://github.com/eggplanetio/tidytab/releases/tag/3.0.0
Cheers!

@iKlsR We're live in the Google Chrome Store :)

Yay, I actually refreshed it last night. Why the long turnaround tho on G's
part
On Dec 1, 2017 11:49, "Brian Gonzalez" notifications@github.com wrote:

@iKlsR https://github.com/iklsr We're live in the Google Chrome Store :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eggplanetio/tidytab/issues/105#issuecomment-348546614,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtP8wB_cO8GK0Pt366ypHMcupMwYwbRks5s8C4ygaJpZM4QV9J8
.

I was transferring the extension from my eggplanet account to my personal and it takes 7 days  ¯_(ツ)_/¯

source: gharchive/issue

authors: briangonzalez iKlsR

repo: eggplanetio/tidytab

license: mit

license_source: bigquery


text:
Clustered TerminatedServerTest failing on windows

The following test is failing on windows with the latest ehcache on trunk..
java.lang.IllegalStateException: Active process did not appear
at org.terracotta.testing.master.SynchronousProcessControl.internalWaitForActive(SynchronousProcessControl.java:256)
at org.terracotta.testing.master.SynchronousProcessControl.waitForActive(SynchronousProcessControl.java:177)
at org.terracotta.testing.rules.BasicExternalCluster$3.waitForActive(BasicExternalCluster.java:197)
at org.ehcache.clustered.TerminatedServerTest.waitForActive(TerminatedServerTest.java:58)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:48)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.runTestClass(JUnitTestClassExecuter.java:114)
at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.execute(JUnitTestClassExecuter.java:57)
at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassProcessor.processTestClass(JUnitTestClassProcessor.java:66)
at org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:51)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
at org.gradle.internal.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32)
at org.gradle.internal.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93)
at com.sun.proxy.$Proxy2.processTestClass(Unknown Source)
at org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:109)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
at org.gradle.internal.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:377)
at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:54)
at org.gradle.internal.concurrent.StoppableExecutorImpl$1.run(StoppableExecutorImpl.java:40)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

I am no longer getting the error referenced above on my windows machine.  Now when I run "./gradlew test clean" and enable logging in the gradle.build file, I see the following in the logs:
org.ehcache.clustered.TerminatedServerTest > testTerminationBeforeCacheManagerClose STANDARD_OUT
[testTerminationBeforeCacheManagerClose(org.ehcache.clustered.TerminatedServerTest)][ConfigBuilder] Added testServer0
[testTerminationBeforeCacheManagerClose(org.ehcache.clustered.TerminatedServerTest)][ProcessControl] >>> addServerAndStart
org.ehcache.clustered.TerminatedServerTest > testTerminationBeforeCacheManagerClose STANDARD_ERROR
[testTerminationBeforeCacheManagerClose(org.ehcache.clustered.TerminatedServerTest)][testServer0] 'C:\Users\ggib\dev\ehcache3_dev\ehcache3\clustered\integration-test\build\cluster\testTerminationBeforeCacheManagerClose' is not recognized as an internal or external command,
[testTerminationBeforeCacheManagerClose(org.ehcache.clustered.TerminatedServerTest)][testServer0] operable program or batch file.
[ConnectionEstablisher-0-1382641315] WARN class.com.tc.net.protocol.transport.ClientMessageTransport - ConnectionID(4.e11e3885ee3f49f9bfef5d132808db10.8a35f41c-974a-40b2-8bbf-fa59898d2142-155c0f614c1.USER)[]:
Connection refused: no further information
[ConnectionEstablisher-0-1621991228] WARN class.com.tc.net.protocol.transport.ClientMessageTransport - ConnectionID(5.e11e3885ee3f49f9bfef5d132808db10.8a35f41c-974a-40b2-8bbf-fa59898d2142-155c0f614c1.USER)[]:
Connection refused: no further information
......
Eventually it kept printing the following log entries:
[ConnectionEstablisher-0-366477705] WARN class.com.tc.net.protocol.transport.ClientMessageTransport - ConnectionID(4.7e5340857e7c489b84de5304b45cd014.71ffec15-9db9-4784-b34a-bdabf3f904c1-155bdc7239a.USER)[]: Connection refused: no further information [lossy interval: 10000ms]
and
[ConnectionEstablisher-0-40841753] WARN class.com.tc.net.protocol.transport.ClientMessageTransport - ConnectionID(5.7e5340857e7c489b84de5304b45cd014.71ffec15-9db9-4784-b34a-bdabf3f904c1-155bdc7239a.USER)[]: Reconnect attempt 140 of unlimited reconnect tries to localhost:32732, timeout=10000
when I killed the test via ctrl c, I got the following error:
Connection refused: no further information [lossy interval: 10000ms]
Unexpected exception thrown.
org.gradle.internal.remote.internal.MessageIOException: Could not write '/127.0.0.1:50023'.
at org.gradle.internal.remote.internal.inet.SocketConnection.flush(SocketConnection.java:121)
at org.gradle.internal.remote.internal.hub.MessageHub$ConnectionDispatch.run(MessageHub.java:300)
at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:54)
at org.gradle.internal.concurrent.StoppableExecutorImpl$1.run(StoppableExecutorImpl.java:40)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.io.IOException: An existing connection was forcibly closed by the remote host
at sun.nio.ch.SocketDispatcher.write0(Native Method)
at sun.nio.ch.SocketDispatcher.write(SocketDispatcher.java:51)
at sun.nio.ch.IOUtil.writeFromNativeBuffer(IOUtil.java:93)
at sun.nio.ch.IOUtil.write(IOUtil.java:51)
at sun.nio.ch.SocketChannelImpl.write(SocketChannelImpl.java:471)
at org.gradle.internal.remote.internal.inet.SocketConnection$SocketOutputStream.writeWithNonBlockingRetry(SocketConnection.java:259)
at org.gradle.internal.remote.internal.inet.SocketConnection$SocketOutputStream.writeBufferToChannel(SocketConnection.java:247)
at org.gradle.internal.remote.internal.inet.SocketConnection$SocketOutputStream.flush(SocketConnection.java:241)
at org.gradle.internal.remote.internal.inet.SocketConnection.flush(SocketConnection.java:119)
... 6 more

Also when I run the unit test it never returns from the constructor call to BasicExternalCluster(.....)

This issue is actually a bug in Galvan.,  So I have opened the following issue: Terracotta-OSS/galvan#61

authors: gibsong rkavanap


text:
Ability to autoconfigure Ehcache

We have the requirement to be able to automatically configure a management registry in some case.
So this PR is to add the ability to provide within the classpath a configuration provider that would be discovered and add configs if they are not already there through programmatic or xml config.
This PR is not complete:

I'll send an email for more details and use case
If this solution is good, we have to decide where to put the provider interface and depending on this what we document

I would love an example indeed. For ehcache.xml maybe. For programmatic, I don't see why a ServiceCreationConfiguration can't be given directly.

@henri-tremblay : see my email and use case in email ;-) this is for ee

LGTM ; elegant way to let ehcache configure services via META-INF/services ; will be useful for Management and monitoring

My question is: Why do we need a provider? Why not getting a DefaultManagementRegistryConfiguration directly in the services? Using a factory method to get your defaults.

test this please

test this please

test this please

Note: had 2 times a random failure not related to my changes:


text:
Made it stupid easy to create a single agent, single action UI

Have a look at chatbot agent in the UI.  You just need a manifest setup properly to create a single agent, single action UI.

Luke -- I also added a title test and fixed a bug in it.

beautiful, just approved

authors: dbrewster harivmasoor

repo: eidolon-ai/eidolon


text:
Page contact - UX

Quelques retours :

redirection page candidature EIG

le premier bloc sur la candidature est un peu confusant, je changerais l'angle en termes de rédaction en écrivant à quoi sert ce formulaire de contact, pour ensuite dire, si vous souhaitez devenir EIG, vous pouvez vous rendre sur (....page de candidature)
en corollaire enlever le remplissage vert

Une fois le formulaire rempli, on arrive sur le truc formspree, mais ce serait pas ensuite de revenir sur la page du site ?

La redirection vers le site en retour n'est possible que pour les plans payants sur Formspree malheureusement : https://help.formspree.io/hc/en-us/articles/360012378333--Thank-You-redirect

authors: AntoineAugusti MathildeBras

license: MIT


text:
WIP: view the source element

Fixes #358.
We have to extract more information to locate the image, including

tag name
property name
property value (before url transform)
tab id
frame id

or is it possible to ref the image without leaking memory?

Assign a unique id to picked elements?


text:
Ricci Tensor and problems with summation of two fractions

🐞 Problem
As you can see here in the image below I have an issue with the Ricci tensor.

All the components should be zero as I'm using the Schwarzshild metric. It's easy to see that the output in the image is correct but the two components are not simplifyed.
It's the same with the evaluation of the Ricci Scalar. It is zero but the output is not simplifyed.

It could be solved by typing:
Ric.simplify() instead of Ric.tensor().
My question is: is it possible to set the sum as default in .tensor() or is it necessary to write the .simplify()?
Thanks a lot.

Hi @manzonis98 , thanks for the issue.
If you see the docs here , on calling R.simplify() before calling R.tensor(), the tensor contained in the class is automatically simplified and R.tensor() returns the simplified version.
However, the current design does not look good to me, and an option of auto_simplify=True would be great I think.

I opened #494 to address this :)

repo: einsteinpy/einsteinpy


text:
Update ShEx for Disease History and Diagnosis

Main changes including:

remove prov:informedBy triple
add "so:has-input" triple
make choices within the "process"

Apart from Rajaram's comment, no more changes from my side, thank you Shuxin

repo: ejp-rd-vp/CDE-semantic-model


text:
release 4.1.0

Add a pretty print option to toString(false) pass true pretty print #230
Add an xmlparser that will output a config that would generate that same file
cli: use --parser to output the complete config --line-separated to print out line separated config compatible with the --json input option for cli
lib: import parseSitemap and pass it a stream

@ekalinin Can you do another release? Thanks in advance!

This is actually now 4.1.1 as I forgot to include a dependency in the original.

Can you do another release? Thanks in advance!

@derduher done:

https://www.npmjs.com/package/sitemap/v/4.1.1

Thank you so much!

Hi, I saw a mention of inputting a stream of urls and getting an output stream of the xmls? I have a lot of urls to be parted in sitemap files by 5k/10k. Does this library support this kind of usage? Without streaming there are memory limitations. Thank you

@okaris the cli supports that now. The library has it planned but for now you can get it working by
stitching some of the elements together like so.
import { SitemapItem, Sitemap } from 'sitemap'
normalizedURL = Sitemap.normalizeURL(url/config)
stream.write(SitemapItem.justItem(normalizedURL))

if you are asking about the CLI npx sitemap < listofurls.txt > sitemap.xml running that will stream.

@okaris I put together an alpha streaming interface in #243. It runs so much better I'm thinking of pushing most users to it. Feel free to comment. I still have some work to do before I merge and release it.
I would love to know from you how you would like to stream in your URLs? Are you pulling it from a file? pulling from a database? I want to try and cover your use-case in the documentation.

Hi @derduher thank you for all the information. I have a big list of products and their urls in a database. If I fetch them all at once the process uses a lot of memory. Both for the initial list and then probably for the object that holds the sitemap data. I was planning to use Mongo's streaming option to pipe the urls to the sitemap module.

authors: derduher ekalinin okaris


text:
ankurjain00 | BubbleSort.py | Add

Add bubble sort for python
Issue #41

Thanks for the contribution! Merging with master

repo: ekdnam/Algorithms


text:
Merge pull request #3 from el-hoshino/feature/Issue#2-MR

Feature/issue#2 mr

Wrong Merge

repo: el-hoshino/NotAutoLayout


text:
Blockstate issues with iron bars (cage spell)

Iron bars don't have the right blockstates when spawning cage structure through Cage spell.

That probably will not get fixed, TBH.

At some point I'm hoping to implement support for Sponge schematics.

Support for Sponge schematics was added, and builtin schematics updated.

authors: NathanWolf YanisBft

repo: elBukkit/MagicPlugin


text:
APM server error "requires at least one of the fields 'value;values'"

Log message

Dec 27 09:12:13 server_name apm-server[32711]: {"log.level":"error","@timestamp":"2023-12-27T09:12:13.677-0500","log.logger":"request","log.origin":{"file.name":"middleware/log_middleware.go","file.line":58},"message":"data validation error","service.name":"apm-server","url.original":"/intake/v2/events","http.request.method":"POST","user_agent.original":"apm-agent-java/1.38.0 (sat_email_manager)","source.address":"172.18.8.65","http.request.id":"d76fb865-ac01-48ce-9a27-f4bbbb441c40","event.duration":10014499490,"http.response.status_code":400,"error.message":"validation error: metricset: samples: requires at least one of the fields 'value;values', validation error: metricset: samples: requires at least one of the fields 'value;values', validation error: metricset: samples: requires at least one of the fields 'value;values', validation error: metricset: samples: requires at least one of the fields 'value;values', validation error: metricset: samples: requires at least one of the fields 'value;values'","ecs.version":"1.6.0"}

noticed in java agent versions 1.38-1.42 (We have not updated to newer versions of the java agent

I believe that's fixed in 1.43.0 in #3290 and #3304, please try that version or higher

Thank you

repo: elastic/apm-agent-java


text:
Create proxy transasction if not reported by agent

Motivation
Create proxy transactions if agent is not able to report the transaction due to unexpected errors like OutOfMemory.
Related issues
Part of #315
Testing
(Currently, the PoC for agents is implemented only in golang so the lambda function must use go1.x runtime)

Create a lambda function (in golang) that causes an timeout (or OOM) and run it a few times.
Make sure to import apm-agent-go with go-agent PoC pull. Example function for reference.
Observe in elastic to assert that a transaction for the function invocation is reported as event.outcome: failure.

The above steps can also be executed using the testing/benchmarking module by editing the test function (same as this example).

:broken_heart: Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-10-28T03:37:43.074+0000

Duration: 4 min 16 sec

Steps errors
Expand to view the steps failures

notice

Took 0 min 6 sec . View more details here
Description:                 make NOTICE.txt                git diff --exit-code --quiet && exit 0 || echo "regenerate NOTICE.txt" && exit 1

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Tested extension on main branch (commit 3032883ee8622076fc3bbc557d44ad5542d48dd2), with ESS 8.6.0 and a PoC that takes a param to decide whether it times out. Not all timed out calls are recorded in ES. Will have to investigate.

Tested extension on main branch with ESS 8.6.0 and PoC with go agent on branch https://github.com/lahsivjar/apm-agent-go/tree/1323-lambdatxn
This PR is working well. Timed out requests will report event.outcome: failure, with the exception that if the first invocation is a failure in an execution environment, the data will be dropped due to missing metadata. The discovery is shared with @lahsivjar @felixbarny @axw

authors: apmmachine carsonip lahsivjar

repo: elastic/apm-aws-lambda


text:
test: disable cgo in go test function

Disable cgo in go test function to ensure there are no glibc/ABI compatibility issues between the build host and the target Lambda execution environment.
Similar to https://github.com/elastic/apm-aws-lambda/pull/292
Discovered while testing https://github.com/elastic/apm-aws-lambda/pull/347
/var/task/main: /lib64/libc.so.6: version `GLIBC_2.32' not found (required by /var/task/main)
/var/task/main: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by /var/task/main)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-12-05T04:32:10.287+0000

Duration: 8 min 16 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
204

Skipped
2

Total
206

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

authors: apmmachine kruskall


text:
[FEATURE] Include sequence id under event details

ECS integration/library project(s) (e.g. Elastic.CommonSchema.Serilog):
Elastic.CommonSchema.Nlog
Is your feature request related to a problem? Please describe.
Enable clearly sequencing of logs
Describe the solution you'd like
I would like the sequenceId which is available as part of the logevent to be recorded under my event as defined in the spec
Describe alternatives you've considered
Rely on timestamps
Additional context
Definition as per: https://www.elastic.co/guide/en/ecs/current/ecs-event.html#field-event-sequence

Opened https://github.com/elastic/ecs-dotnet/pull/392 to address this thanks @thompson-tomo

thanks @Mpdreamz i noticed in my testing/debugging when i instpected the logevent it was being incremented on each run through but if it is planned to be removed that is also ok.

authors: Mpdreamz thompson-tomo

repo: elastic/ecs-dotnet


text:
Version bumps in the tests workflow for the 8.15 release

Version bumps in the tests workflow for the 8.15 release

@joemcelroy I think we can merge this now.

repo: elastic/elasticsearch-labs


text:
Elasticsearch::Persistence::Repository.search bug

I believe Line 49 should be this
response = client.search( { index: index_name, type: type, body: query_or_definition }.merge(options) )

instead of
 response = client.search( { index: index_name, type: type, q: query_or_definition }.merge(options) )

That q key should be body instead when a query string is passed to the method as also mentioned in the example here

Any feedback on this?

@alolis Sorry for not getting to you sooner, I was too busy with the elasticsearch-dsl gem. I am a bit confused by your suggestion -- so what I do is to pass the query as a body when it's a Hash, and as a URL parameter when it's a String, which makese sense, no?
I'll be only partially online in the next two weeks, so please forgive if I'm sketchy with replies.

@alolis Any news here? I still don't see the issue.

@karmi , sorry, i never saw your reply! On my initial test, my query was a string and passing it as q: was causing a search query exception. I then changed it to body: and it worked normally. Maybe I should just make a quick test on the irb to help you reproduce this behavior.

@alolis I understand your description of the problem, but again: the method sends Hashes as body and Strings as q. In other words, when you pass your search definition as a String, it is intentionally sent as a q parameter.
Can you explain a bit better what concrete problem you have with the above in mind?

If you do the following with the current code, ElasticSearch throws SearchPhaseExecutionException
qs = '{"query": {"match_all": {}}}'
MyRepositoryClass.search(qs)

if you change the line i mentioned in my first post, then it works as expected.

...but that is expected -- anything sent in as a String is sent as q parameter to Elasticsearch, to use the "simple search", with Lucene syntax and what not. In other words, this is by design, and I'd be reluctant to change that, since I believe many people rely on that. If you want to use Query DSL, just send the search definition as a Hash. Is that an option for you? Or is there something which makes it impractical or incovenient for you?

I am already sending as a Hash and everything works perfectly. I wasn't sure if this is by design or if it was a bug that's why I posted it :) I am sorry if i caused any confusion, it is perfect as it is (covers both simple and more advanced cases)

Great! It is really by design.

May I suggest for a comment somewhere (string simple search vs hash query dsl) in order to avoid any future confusion

I tried to make it clear in the documentation here: https://github.com/elastic/elasticsearch-rails/blob/master/elasticsearch-persistence/lib/elasticsearch/persistence/repository/search.rb#L13-L19, would you have a suggestion for better phrasing?

Even though I did read it, my brain was absent at that moment I guess. It's probably just fine as it is if nobody else has complained :)

Great! Let's close this one then! :)

authors: alolis karmi

repo: elastic/elasticsearch-rails


text:
Show total size of payload in track info when listing tracks

I used to use the tiny track to do quick/simple rally testing because I knew it was a small track to both download and store.
With the other tracks, you have no idea of the size required on disk or the network impact (size and time) without going into the track file and adding up all the payload file sizes or by running it once to see the resulting payload files on disk after they're retrieved.
It would be neat to show a total size indicator when listing tracks with rally.
This might even just be easy to add to the end of the description, but would be nicer if it was calculated from the track file where the sizes of payloads are specified.

I used to use the tiny track to do quick/simple rally testing because I knew it was a small track to both download and store [...] with the other tracks, you have no idea of the size required on disk or the network impact (size and time)

Are you aware of --test-mode? It download only a tiny portion of the benchmark corpus (1.000 docs) and you can use it for all our tracks in the default repository. E.g.:
esrally --track=geopoint --test-mode
esrally --track=logging --test-mode

Does this help or do you still think we need to show the corpus size in the list command?

Yeah test-mode is probably fine for "just getting some data in there", but it still needs to download the full payload first?

No, test mode does not download the full data set. The track author needs to prepare a stripped down file containing just the first 1.000 docs with the suffix -1k for that (I admit the docs are light on that one). E.g. if your data are contained in documents.json.bz2 you need to provide documents-1k.json.bz2 as track author (and all our default tracks do just that).

So I guess if that is sufficient for you I'd close the ticket as "Won't fix".

authors: danielmitterdorfer geekpete

repo: elastic/rally


text:
"found" field added after MyDocType.get(): expected behaviour?

I've been playing around with the persistance example.
I just noticed that after I use the get method, there is a "found" field added to my document.
Saving, then adds that field to the document, and obviously to the mapping.
Is this the expected behaviour?

This is not an expected behavior, thanks for the report, I will fix it.

Happy to help out!

Honza, I have been working with the latest stable version (0.0.3). That is where I saw the bug.
After you assigned yourself the bug, I went and downloaded the latest dev version (0.0.4).
I just tested it now, but the issue does not seem to occur.
So this seems to be a non existing bug in the dev version, although present in the current stable release.

Ah cool. Yeah - we changed the behavior of that part so I wondered if it would still be an issue. Thanks for checking and coming back with the results.
I am closing this ticket as it has been resolved on master. I will release a new version soon to distribute the fix.

authors: HonzaKral njoannin

repo: elasticsearch/elasticsearch-dsl-py


text:
Optional types have to implement default

If I do this:
struct TestType;

#[builder]
fn test(test_type: Option<TestType>) {}

fn run() {
    test().call();
}

I get this error:
the method `call` exists for struct `TestBuilder`, but its trait bounds were not satisfied
the following trait bounds were not satisfied:
`TestType: std::default::Default`
which is required by `bon::private::Optional<viewer::kmp::checkpoints::TestType>: bon::private::IntoSet<std::option::Option<viewer::kmp::checkpoints::TestType>>`

Am I missing something? Why do optional types need to implement Default?

Interestingly, if you change it to this:
struct TestType;

#[builder]
fn test(test_type: Option<TestType>) {}

fn run() {
    test().maybe_test_type(None::<TestType>).call();
}

the error goes away. If you just do None and not None::<TestType> there is an error which is strange!

Thank you for the bug report! It's a really small bug in bon, I'll fix it shortly and make a patch release, thank you!

The fix has been released in a new patch version 1.0.2.
Feel free to reopen the issue if the problem persists.

If you just do None and not None::<TestType> there is an error which is strange!

This is a bit different problem that is inherent to the setter method signature:
fn maybe_test_type(self, value: Option<impl Into<TestType>>) -> NextBuilderState

Because it accepts an Option<impl Into<TestType>> the type of None needs to explicitly specified unless inferred by the compiler from other context.

authors: ThomasAlban Veetaha

repo: elastio/bon


text:
HACS and Google Drive Backup

Hi,
I have two issues:

After restart HA or refresh page HACS goes below in order (like order no is 10 but it is actually 2). After I click edit sidebar and click "done" button immediately (without order sidebar manually), it is fixed

I couldn't give order number to Google Drive Backup Addon. What is its item name? (on browser address bar: http:192.168.1.1:8123/cebe7a76_hassio_google_drive_backup/ingress).

id: sidebar
order:

item: overview
order: 0
item: studio code
order: 1
item: HACS
order: 2
new_item: true
item: ADD-ONS
href: "/hassio/dashboard"
icon: mdi:arrow-decision
order: 4
new_item: true
item: INTEGRATIONS
href: "/config/integrations"
icon: mdi:transit-connection-variant
order: 5
new_item: true
item: ENTITIES
href: "/config/entities"
icon: mdi:devices
order: 6
new_item: true
item: AUTOMATIONS
href: "/config/automation"
icon: mdi:robot
order: 7
new_item: true
item: SCRIPTS
href: "/config/script"
icon: mdi:script-text-play
order: 8
new_item: true
item: CHECK & RESTART
href: "/developer-tools"
icon: mdi:restart-alert
order: 9
item: settings
bottom: true
item: history
hide: true
item: snapshots
hide: true
item: map
hide: true
item: energy
hide: true
item: media
hide: true

Hi @cataseven,
For the first issue, could you check your browser dev console and see if there is a warning coming from the plugin saying that it cannot find the item HACS?
Also, in your browser dev console, could you go to Application > Local storage and check if you have an entry called sidebarPanelOrder.
About the second issue, I don't understand your question, could you explain better what is the issue?
Regards

Thx for quick response.
For the second issue; Home Assistant Google Drive Backup addon is installed. I see its icon on the sidebar and I want to give an order number but I do not know the item name. For example for Studio Code Server  it is - item: studio code.
What should I do to give order number for Google Drive Backup addon

For the second issue; Home Assistant Google Drive Backup addon is installed. I see its icon on the sidebar and I want to give an order number but I do not know the item name. For example for Studio Code Server it is - item: studio code.

You say that you see the icon in the sidebar but at the same time you say that you don't know its name, that is weird. So does it appear in the sidebar without a name? Post screenshots about it, in your screenshots I cannot se HACS neither the Google Drive backup.

Ohh ok I understand now. My Hacs is named as Store.

When I leave it blank and save it retuns Store as default name.
So below is working for hacs now. Thanks

item: store
order: 2

Ah super, everything is solved then.
Just remember that there are other matchers apart from name. You can also match by data-panel or by href.
Regards

authors: cataseven elchininet

repo: elchininet/custom-sidebar


text:
🛑 BIP is down

In 7bb2311, BIP (https://www.bancoprovincia.bancainternet.com.ar/eBanking/login/inicio.htm) was down:

HTTP code: 0
Response time: 0 ms

Resolved: BIP is back up in fae548d.

repo: eldikra/monitoreo


text:
Diffing, error if field is missing

If you try missing.required: true and it's not in the data, it doesn't fail?

Possible solution:
add required: true automatically to all fields in test function

repo: eldoy/flekk


text:
QSPI data is incorrect when using the APP_TYPE = BOOT_SRAM

QSPI data is being corrupted when using APP_TYPE = BOOT_SRAM.
Repro:

Use the QSPI DaisyExample
Setup the Daisy Seed to run the Daisy Bootloader for APP_TYPE = BOOT_SRAM
Make the needed modifications to the launch.json files to support the SRAM launch and debugging.
Set a breakpoint on line 59

    hw.qspi.Erase(address, address + size);

Set a breakpoint on line 60

    hw.qspi.Write(address, size, (uint8_t*)wavform_ram);

run through the program for the first time
Run past the second break point (post the Write operation)
Notice the QSPI memory starts with 0.00000 and starts to smoothly build in a positive direction.
Restart the debugging of the program
Go to the first break point, so that the erase does not happen, but the system is initialized.
Look at the memory location for the QSPI data
Notice: The data is corrupt and not the same 0.000 data ramping up in the positive direction.

Expect: the QSPI data remains the same over restarts.
Notes:

After every restart the QSPI data is consistently incorrect in the same way:

Example array items 7-10 are always zero.

This does not happen when the Diasy Bootloader is not in use.

Note: as an experiment I changed the QSPI MEMRY segment in the _SRAM.LDS file to this:
  QSPIFLASH (RX): ORIGIN = 0x90100000, LENGTH = 6M

This fixes the issue.
I was wondering if there was an alignment issue happening due to the 0x90040000.  I'm not sure if that was really this issue, but once I changed the to the 0x90100000 the QSPI flash seems to be returning correct data for me now across restarts.
Also, please note: the 6M in the above may not be accurate, I placed something there to give the starting memory location change a try and I wanted to share this update.
Hope this helps.

I had two thoughts this morning, I will try one of the two later tonight my time. 😊

I am wondering if the data structure that I put in QSPI memory should have the attribute packed?”
I was also wondering if in the linker script could the ALIGHN(4) be incorrect for the Flash?

I was thinking about both of these because when I looked into the memory before and after the Flash, then restart, then read the memory was consistently "corrupted" in the same way.  Example from above: array items 7-10 were always zero.  This type of pattern was common no matter the type of data I was using.
I wanted to share that idea in case it helped.

Hi Brett,
Thank you for writing a detailed report. We will have a closer look at this as soon as we can.

The main issue you're probably facing is that your original code was using the same address space in QSPI for the program itself and anything you later wrote to it. The Daisy bootloader uses the region beginning at 0x90040000 for the program, which can be anywhere from a few kB to around 500 kB. It's probably a good idea for us to adjust that BSS region for the linkers that use the bootloader in any case.
I wouldn't mess with the packed attribute or the alignment in the linker. If you're storing arrays, the data is already "packed." If you're storing a lot of your own structs, just place data types of similar size together if you're really worried about the density. Using the packed attribute or changing the alignment might result in read or write issues if the QSPI peripheral doesn't support unaligned operations. I haven't specifically checked the manual on whether it's an error, but it's generally not a robust approach.

authors: CorvusPrudens Electro-Jam takumi-ogata

repo: electro-smith/libDaisy


text:
Snapcraft builds are always auto rejected

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Forge Version:

6.0.0-beta.51

Electron Version:

8.2.3

Operating System:

Ubuntu 18.04 x64

Last Known Working Electron Forge version::

N/A

Expected Behavior
The created snapcraft files should be able to be uploaded and pass review.
Actual Behavior
The snap builds always fail the automatic review and are rejected with this:
checksums do not match. Please ensure the snap is created with either 'snapcraft pack <DIR>' (using snapcraft >= 2.38) or 'mksquashfs <dir> <snap> -noappend -comp xz -all-root -no-xattrs -no-fragments'.
If using electron-builder, please upgrade to latest stable (>= 20.14.7). See https://forum.snapcraft.io/t/automated-reviews-and-snapcraft-2-38/4982/17 for details.
security-snap-v2_squashfs_repack_checksum What does this mean?
found errors in file output: unusual mode 'rwsr-xr-x' for entry './swach/chrome-sandbox' security-snap-v2_squashfs_files
human review required due to 'deny-connection' constraint (interface attributes). If using a chromium webview, you can disable the internal sandbox (eg, use --no-sandbox) and remove the 'allow-sandbox' attribute instead.
For QtWebEngine webviews, export QTWEBENGINE_DISABLE_SANDBOX=1 to disable its internal sandbox.
declaration-snap-v2_plugs_connection (browser-sandbox, browser-support)

To Reproduce

Additional Information
I have read in a bunch of places to use --no-sandbox which I am doing with this:
if (process.platform === 'linux') {
  app.commandLine.appendSwitch('--no-sandbox');
}

I have also tried setting browserSandbox: false in the forge config. My current config:
    {
      name: '@electron-forge/maker-snap',
      platforms: ['linux'],
      config: {
        confinement: 'strict',
        description: 'A robust color management tool for the modern age.',
        features: {
          audio: true,
          browserSandbox: false,
          webgl: true
        },
        grade: 'stable',
        name: 'swach',
        summary: 'A robust color management tool for the modern age.'
      }
    },

Is there a way to get this to work?

Closing in favor of https://github.com/electron-userland/electron-installer-snap/issues/74

@malept wherever the issue should live, the problem exists in both places and will have to be documented. Do you have any feedback here on how to fix this?

wherever the issue should live, the problem exists in both places and will have to be documented.

It's still a duplicate issue and it's linked so I don't see a problem with marking this as duplicate and closing it.

Do you have any feedback here on how to fix this?

I would need to make some time to investigate this, although I maintain this in my very limited free time, so I don't know when that will be.
For the record, the reasons for enabling browser sandbox for Electron >= 5.0.0 is documented here: https://github.com/electron-userland/electron-installer-snap/pull/22

@malept I see in that PR you linked part of the changes added a console.warn saying 'The browser-sandbox feature will trigger a manual review in the Snap store.'. So does that mean there is no possible way to build an electron app that can pass auto review and it will always be rejected and require manual review?
I see that the PR says this was implemented to support https://github.com/electron/electron/pull/17269, but is sandbox required for Electron apps to run at all on Linux? I am still confused why this is the default with no options to disable it.

Reopening this to review/make some snap improvements

@VerteDinde I'm using https://github.com/davidwinter/electron-forge-maker-snap to get snap to work, since the base lib didn't support it

This was fixed upstream in electron-installer-snap, closing this as Fixed.

authors: VerteDinde malept rwwagner90

repo: electron-userland/electron-forge


text:
new releases are not being shown in releases/

https://www.electronjs.org/releases/alpha is missing v17.0.0-alpha.6 (released 2 days agoi) and v17.0.0-alpha.5 (release three weeks ag).
CC @electron/wg-ecosystem @electron/wg-releases

I believe this is just a manual merging problem. Should be close to updated with #5822 but might need another PR for the latest ones since I think Dependabot only checks once a week.

@erickzhao Releases pages have stopped updating for the past few months.

authors: ckerr elincoln erickzhao


text:
feat(template): fix vite's main process template

Packages like axios have a browser field in package.json. The main process needs to resolve the node module, not the browser module.

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project follows, as appropriate.
[ ] The changes are appropriately documented (if applicable).
[ ] The changes have sufficient test coverage (if applicable).
[ ] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
Packages like axios have a browser field in package.json.
The main process needs to resolve the node module,
not the browser module.

Request code review
/cc @caoxiemeihao

@jclab-joseph CI Lint failed. You can try execute:
npx prettier --write .

@caoxiemeihao Do you have anything to add?

@caoxiemeihao Do you have anything to add?

Nothing :)

Just keep the browserField, and in the future as well :)

This code will break in the future if you only use browserField. It is okay?
Wouldn't it be better if it worked even into the future when only mainFields are left?

Just keep the browserField, and in the future as well :)
export default defineConfig({
  resolve: {
    // Some libs that can run in both Web and Node.js, such as `axios`, we need to tell Vite to build them in Node.js.
    browserField: false,
  },
});

This code will break in the future if you only use browserField. It is okay?
Wouldn't it be better if it worked even into the future when only mainFields are left?

This code will break in the future if you only use browserField. It is okay? Wouldn't it be better if it worked even into the future when only mainFields are left?

browserField interferes with mainFields parsing because browserField has higher priority.

browserField in L294
mainFields in L1041

mainFields will always exist, and it contains browserField which is equivalent to ['browser', 'module', 'jsnext:main', 'jsnext'], so browserField will be removed because Vite doesn't just build Web.

What I'm concerned about is when Vite completely removes browserField functionality.
In "the future", this code will no longer work without mainFields.

Vite v5.0 will remove browserField, but it has nothing to do with mainFields. There is no effect on forge either :)

@jclab-joseph Sorry, I misread! …（⊙＿⊙；）…
@BlackHole1 LGTM (￣.￣)

LGTM. But I don't have merge permission.
Request CR/Merge @erickzhao @VerteDinde

Thank you for the contribution! Sorry for the edit history, I tried to make the PR title clearer and more concise.

authors: BlackHole1 caoxiemeihao erickzhao jclab-joseph

repo: electron/forge


text:
fix(plugin-vite): right time to start

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[ ] The changes are appropriately documented (if applicable).
[ ] The changes have sufficient test coverage (if applicable).
[x] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
Fix: #3208

Awesome to see this fixed! I just migrated to this plugin but sadly as my app is pretty big it is failing now. Could this be soon released as a hotfix?
I will be very thankful ❤

authors: IhsenBouallegue caoxiemeihao


text:
Don't treat static archives as PLAIN

Some native packages like sqlite3 and cap leave various .a static archives around and universal was treating them like PLAIN files.

Instead of adding special handling for various "stray" file types I think we should keep the error that occurs with these files.
In this case the "correct" thing to do is to delete / not copy these .a files when packaging.  Having this tool implicitly leave one of them around when they are different is semantically incorrect behavior.

Via package.json -> build -> files ["!...../foo.a"] ?
I'm not familiar with all the exclusionary techniques available.  Or does sqlite3 need to make changes?
It's worth noting that a non universal electron app will hapily let you include all the stray files you can come up with, and there are a LOT of dead files in a typical Electron app IME.
But I take your point.  It does feel tedious to try and learn what all the non-binary types are.  I just don't know a better way to dance around my builds not working.

It's worth noting that a non universal electron app will hapily let you include all the stray files you can come up with, and there are a LOT of dead files in a typical Electron app IME.

Sure, the whole point is that a universal build of an app needs to somehow reconcile every single file in both architectures and somehow merge them.  We can't teach universal about every possible file type so instead we insist that the developers handle these stray files themselves.

Via package.json -> build -> files ["!...../foo.a"] ?

We neither support nor recommend electron-builder so I have no idea how to do this over there 🤷 Officially supported Electron tooling consists of electron-packager / electron-forge, et. al

authors: MarshallOfSound mayfield

repo: electron/universal


text:
Send report to a github page report

Is your feature request related to a problem? Please describe.
Sending reports through S3 is great but by default the s3 website will be publicly accessible. They are way to make it private but they seems pretty complex for a single page website.
Describe the solution you'd like
We could host the report on a github page repository, that would ease access control.
The idea would be to push the html file to a github repository.

Hi @courentin! Thanks for the great suggestion.
I'd like to share a couple of things if that's okay.
First off, you can also send your report to Slack or to a GCS bucket if those fit you better.
Regarding the S3, I agree that setting up a website could be tedious,
however, you can also upload the report to the bucket with a fixed name like report.html and grant access to that file.
Regarding your request, it seems like it should already be possible to do that.
Basically what you'd need to do is create a GitHub workflow in your GitHub Pages repository that does the following steps:

Run Elementary's action.
Run the add-and-commit action to commit the report to the repository.

Afterwards, you should see the report being updated in your website.
How does that sound to you?
Care to give it a shot and share some insights?
Thanks a lot!
Feel free to let me know if you have any questions.

Hello @elongl thanks for your answer!
Yes I can definitely do use an action for this, that would work perfectly (and it's actually what's I'm doing for the dbt doc).
That was more an open question about "would there be a benefit of including it directly in the send-report command".
I have not strong opinions on it as above's solution works perfectly fine for me!

@elongl Maybe we should add this use case to our docs?

@Maayan-s Sounds like a great idea!
@courentin Would you perhaps be interested in adding a section to Elementary's docs regarding how you integrated Elementary's report with your GitHub Pages? I think it can be an interesting use-case for a lot of users and it seems like you really nailed this mechanism. Let me know what you think 😄
You can do it by clicking the Edit this page on the Share observability report page.

I recently noted that there's a very popular action that deploys to GitHub Pages. Maybe worth looking into it and understand the difference.

authors: Maayan-s courentin elongl

repo: elementary-data/elementary


text:
🛑 UNER - Redmine is down

In 8605585, UNER - Redmine (https://redmine.uner.edu.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: UNER - Redmine is back up in cb019ee.

repo: elfoche/monitoreo


text:
🛑 UNER - Turnero is down

In c4b2b01, UNER - Turnero (https://turnosfirmadigital.uner.edu.ar/nuevo-turno) was down:

HTTP code: 0
Response time: 0 ms

Resolved: UNER - Turnero is back up in 55616ac after 2 hours, 4 minutes.


text:
🛑 UCU - SIG is down

In 4815954, UCU - SIG (http://administracion.ucu.edu.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: UCU - SIG is back up in e3b51a6 after 3 minutes.


text:
🛑 UNER - Turnero is down

In 52c15c1, UNER - Turnero (https://turnosfirmadigital.uner.edu.ar/nuevo-turno) was down:

HTTP code: 0
Response time: 0 ms

Resolved: UNER - Turnero is back up in 59a1b3e.


text:
Burst with Libreelec

Wondering if anyone else has this issue?
I have 2 installations of Elementum,  one on a Sony Bravia TV, works fine, the other an installation of Libreelec on a raspberry pi 4 , on this one I only get results from one provider, MagnetDL, pretty sure both setups have the same settings, including the same half a dozen or so providers?

if they in the same network - then they should behave the same. make 100% sure that they have the same settings - you can copy paste appropriate xml files. see https://kodi.wiki/view/Userdata#Location and https://kodi.wiki/view/Userdata#addon_data .
if they in different networks - then try to use VPN, for example.

Yeah, I use a VPN with same settings for both, I'll give your copy/paste file a go and see if that helps, thanks.

make sure to copy settings.xml for both elementum and burst.

you can also enable debug logging and see if there any errors in log file - most likely it will help you to see the issue with your configuration or network.
see https://elementum.surge.sh/contribute/log/

@antonsoroko

if they in the same network - then they should behave the same.

Yes and no.
https://www.zenrows.com/blog/what-is-tls-fingerprint
There is for some time TLS fingerprinting. By default Python is using OpenSSL with static TLS configuration. Same for default Golang http library.
And Cloudflare can detect that is it not a browser.
So same code and same scripts can run differently on different platforms, because Python is built differently per each OS in Kodi.
There are ways to pretend from Python/Golang to be a real browser, but that is a different story. And I have tried to make a quick change in built-in Elementum's proxy to pretend a browser, but it could not operate at all at that time.
I think because the change enables HTTP2, which is supported by modern browsers, but is not quite supported by default requests Python module, so that Python is expecting to send HTTP1 request through Elementum proxy but receives HTTP2 response.
It would be just easier to add an endpoint in Elementum that would make requests, but not as a usual http proxy, but as a service instead.
And again that won't help a lot.

So copying settings files proved to be a bit awkward on my TV, so gave up, just looked through them all, then tried to remember when looking at kodi on my libreelec set up.
Anyhow, success! Didn't know what I was doing, but on looking through proxy settings in elementum, and remembering elgatito mentioned http , I noticed the 'use built in http' (or similar) was off, so tried putting it on, and bingo!
Thanks for your help guys!
Don't know why it was off (maybe a libreelec thing?), or even what it does to be honest, but it is on on my TV setup.

@RobJM2

mentioned http , I noticed the 'use built in http' (or similar) was off, so tried putting it on, and bingo!

When you enable Elementum proxy - then all your requests are going through Elementum, that has different TLS signature, so it can be treated by Cloudflare as a different client. Sometimes it helps - sometimes not.

@elgatito should this issue kept open so people can easily find it if they have similar issue?
seems to be a common issue.

@RobJM2

mentioned http , I noticed the 'use built in http' (or similar) was off, so tried putting it on, and bingo!

When you enable Elementum proxy - then all your requests are going through Elementum, that has different TLS signature, so it can be treated by Cloudflare as a different client. Sometimes it helps - sometimes not.

Cheers, whatever all that means 😊
Much appreciate your work on this app , it's the only one I use.

authors: RobJM2 antonsoroko elgatito


text:
Categories as Lightswitches

Any interest in developing a custom plugin that presents categories from a category group as lightswitches?

I would certainly be open to discussing the commission of such a thing. I probably won't get around to developing it "for fun", but I love custom plugin development projects. Why don't you email me at eli@elivz.com and we can discuss!

authors: elivz tinchalamo

repo: elivz/craft-single-cat


text:
feat: create tes task as k8s job

PS: This is WIP, consider this PR for POC, this maybe/will be further broken down into smaller PR once a concensus on code is reached and everthing starts to work.

Summary by Sourcery
This pull request introduces the ability to create TES tasks as Kubernetes jobs. It includes a new Kubernetes client wrapper, a converter for TES tasks to Kubernetes jobs, and updates to the deployment configuration to support these new features.

New Features:

Introduced a Kubernetes client wrapper to abstract interactions with the Kubernetes API.
Added functionality to convert TES tasks to Kubernetes jobs using a new converter module.
Implemented a new module for creating TES tasks, including the creation of Kubernetes jobs and config maps.

Enhancements:

Updated the deployment configuration to include a taskmaster template for Kubernetes jobs.
Enhanced the CreateTask function to convert TES tasks to Kubernetes jobs and print the resulting job object.

Deployment:

Added a new taskmaster template and environment properties to the deployment configuration.

The CI will break, those are easy fixes, please ignore them and review the rest of the code and logic :)

repo: elixir-cloud-aai/TESK


text:
Deprecate interpolation in join

The interpolation of subqueries is not necessary since there is a subquery construct and the interpolation of queries can be easily handled today with constructs such as dynamic on.

We can't because we actually need this for associations to work. :)

repo: elixir-ecto/ecto


text:
Documentation for Ecto.Schema read_after_writes incorrect?

The documentation on https://hexdocs.pm/ecto/Ecto.Schema.html states:
:read_after_writes - When true, the field only sent on insert if not nil and always read back from the repository during inserts and updates.
From our testing, it appears the first part of this explanation is not correct. It still sends the value nil along to the database when inserting. The second part (reading it back) works as described.
Thanks

The docs are outdated. It means it will read the fields every time there is an insert or update. However, Ecto will always send all fields by default. The work around so far is to provide a before_insert/before_update callback that will call delete_change on the field you don't want to be sent. I will fix the docs on master, thank you! :)

@josevalim What is the proposed solution to this considering callbacks are being phased out?

We will definitely address this issue, but the solution is not yet fully realized.

+1, need a way to be able to insert models where default values are provided by the database but not by Ecto.Schema. before_insert macro is currently working for me. :read_after_writes and :virtual don't match my use case. maybe a :omit_insert keyword is needed?

AFAIK Ecto 2.0 won't send default nil values to the database on insert - this should solve the issue.

@michalmuskala thanks for the clarification

authors: brightbits divmgl josevalim michalmuskala sorentwo

repo: elixir-lang/ecto


text:
" ... "mix compile" failed" in coc workspace output

I installed both coc.nvim and coc-elixir via vim-plug.
Go-to-definition only works for functions in the same module/file. For 'remote' functions, I see [coc.nvim] definition not found in the message bar.
When I run :CocCommand workspace.showOutput elixir I see the compilation errors.
Now here's the tricky part – the compilation error is a known issue (for my project)! One of my project's Mix dependencies has to be compiled first (when performing a full/clean mix compile). Is there some way to configure coc-elixir to do that?
In a shell in my project's root directory, I was able to run MIX_ENV=text mix compile and it was successful (because the relevant dependency was already compiled) so I'm guessing coc-elixir is not running mix compile exactly the same way as I am from my shell.

I'm guessing this might be a 'problem' (for me at least) with elixir-ls itself. I'm going to look into that now.

I wasn't able to reproduce this with another Elixir project. I'm going to create an issue in the elixir-ls project for this.

repo: elixir-lsp/coc-elixir


text:
Error loading onnx model

Hi, I'm trying to load a onnx model and I'm getting this error:
(RuntimeError) Failed to create ONNX Runtime session: Load model from /home/sonic182/work/myproyect/models/e5-small-v2-onnx/model.onnx failed:This is an invalid model. In Node, ("MaskReduceSum_0", ReduceSum, "", -1) : ("attention_mask_int32": tensor(int32),) -> ("mask_index_0",) , Error Unrecognized attribute: axes for operator ReduceSum
(ortex 0.1.6) lib/ortex/model.ex:28: Ortex.Model.load/3
/home/sonic182/sandbox/livebooks/embeddings.livemd#cell:s35av5wcbejnldtu5ekss352elzjd74y:3: (file)
The model was generated with this two code lines in python 3.10.12 (using colab, moving results to my drive and downloading it)
!pip install transformers "onnxruntime<1.15" optimum[onnxruntime,exporters] torch
!optimum-cli export onnx -m intfloat/e5-small-v2 --optimize O2 e5-small-v2-onnx --opset 18

using huggingface optimum cli, the model is e5-small-v2
This kind of model operations are not supported?

Seems that the format generated by optimum-cli is different than using torch.onnx, now it works ok using this code (based on your stablelm example)
!mkdir output

import torch.onnx
from transformers import AutoTokenizer, AutoModel

model_id = "intfloat/e5-small-v2"

tokenizer = AutoTokenizer.from_pretrained(model_id)
model = AutoModel.from_pretrained(model_id)

prompt = "this is arbitrary text"
inputs = tokenizer(prompt, return_tensors="pt")

torch.onnx.export(
    model,
    (inputs["input_ids"].cpu(), inputs["attention_mask"].cpu()),
    "output/e5-small-v2.onnx",
    input_names=["input_ids", "attention_mask"],
    dynamic_axes={
        "input_ids": {0: "batch_size", 1: "sequence_length"},
        "attention_mask": {0: "batch_size", 1: "sequence_length"},
    },
)

I left the issue open in case you want to support the optimum-cli generated models

A quick question, in my case, I'm trying to get embeddings using model intfloat/e5-small-v2, this is the correct way of doing it?

This is the code that I think it gives me the embeddings, the result embeddings are a bit different from using the model with bumblebee + axon/XLA directly, but I think it has to be with the format of onnx

Based on that error, my best guess is that optimum is using a custom opset that isn't part of the default ONNX spec (specifically ReduceSum with an axes attribute), however I don't know much about how optimum works at a deeper level. You may be able to reduce the opset in the CLI output to force optimum to not do this, however I'm not sure.
As far as slightly different values of Ortex vs XLA output, that's expected. Since they're using different Nx backends they'll be slightly different for floats.
I'm going to close this for now since supporting optimum specific exports isn't planned, but will investigate why this failed and may open a related issue if it turns out to be due to Ortex/onnxruntime.

Forget this, It was because I wasn't having all onnx model files from the folder created when generating the onnx model.
I mean, from colab when generating the model, I only downloaded the .onnx file, and I needed to download all other files too (the whole folder)

I really didn't finish that test, because I started to use torch.onnx export, whatever, it splits in multiple files whenever the model size is more than 2GB I think, so is the same stuff that it splits in various files

repo: elixir-nx/ortex


text:
Add Previous and Next lesson links.

The lesson is already set up to contain the next and previous lessons, we just need to add the UI. Additionally I parsed all the lessons and pulled the first paragraph (above the TOC) out and into the excerpt attribute. This allows us to display this along with these links.
I'll work on tackling this next.

This and #26 are quasi-related, that will need to be fixed for this to function correctly.


text:
panic: cannot find first child from: &ast.MemberExpr

Hi all! Using c2go v0.21.13 Zinc 2018-03-11 (git master), The following code produces
panic: cannot find first child from:  &ast.MemberExpr{
Addr:0x7f9a818c43c0,
Pos:ast.Position{File:"/Users/.../c2go-issue1.h", Line:11, LineEnd:0, Column:43, ColumnEnd:46, StringValue:""},
Type:"char [64]",
Type2:"",
Name:"id",
IsLvalue:true,
IsBitfield:false,
Address2:"0x7f9a818c3c00",
IsPointer:true,
ChildNodes:[]ast.Node{(*ast.ImplicitCastExpr)(0xc42040c000)}}

#include <stdlib.h>
#include <string.h>

typedef struct ss {
	char id[64];
} ss;

int main() {
	ss* p = (ss*)malloc(sizeof(ss));
	memset(p->id, 0, sizeof p->id);
	return 0;
}

EDIT: further reduced test case

@Konstantin8105 do you know what is causing this?

@elliotchance , after my PR #641, that is not solved.
○ → c2go transpile 11.c
panic: cannot find first child from: &ast.MemberExpr{Addr:0x249dd98, Pos:ast.Position{File:"/home/lepricon/go/src/templorary/11.c", Line:10, LineEnd:0, Column:26, ColumnEnd:29, StringValue:""}, Type:"char [64]", Type2:"", Name:"id", IsLvalue:true, IsBitfield:false, Address2:"0x2484308", IsPointer:true, ChildNodes:[]ast.Node{(*ast.ImplicitCastExpr)(0xc42012e120)}}

goroutine 1 [running]:
github.com/elliotchance/c2go/transpiler.transpileUnaryExprOrTypeTraitExpr(0xc42029a3c0, 0xc4200b07e0, 0xc420542428, 0xc420542440, 0x0, 0xc4205423d8, 0x58e365, 0xc42029adc0, 0x69b2e5, 0xc4205423d8, ...)
	/home/lepricon/go/src/github.com/elliotchance/c2go/transpiler/unary.go:595 +0x525
github.com/elliotchance/c2go/transpiler.transpileToExpr(0x7dbf20, 0xc42029a3c0, 0xc4200b07e0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x812ac8, 0x0, ...)

More compact C test example:
typedef struct ss {
        char id;
} ss;

int main()
{
        ss v;
        ss *p = &v;
        return (sizeof p->id);
}

AST tree with panic:
`-FunctionDecl 0x387f100 <line:5:1, line:10:1> line:5:5 main 'int ()'
  `-CompoundStmt 0x387f4a0 <line:6:1, line:10:1>
    |-DeclStmt 0x387f270 <line:7:2, col:6>
    | `-VarDecl 0x387f210 <col:2, col:5> col:5 used v 'ss':'struct ss'
    |-DeclStmt 0x387f3a0 <line:8:2, col:12>
    | `-VarDecl 0x387f2f8 <col:2, col:11> col:6 referenced p 'ss *' cinit
    |   `-UnaryOperator 0x387f380 <col:10, col:11> 'ss *' prefix '&'
    |     `-DeclRefExpr 0x387f358 <col:11> 'ss':'struct ss' lvalue Var 0x387f210 'v' 'ss':'struct ss'
    `-ReturnStmt 0x387f488 <line:9:2, col:22>
      `-ImplicitCastExpr 0x387f470 <col:9, col:22> 'int' <IntegralCast>
        `-ParenExpr 0x387f450 <col:9, col:22> 'unsigned long'
          `-UnaryExprOrTypeTraitExpr 0x387f430 <col:10, col:20> 'unsigned long' sizeof      /// <--- PANIC HERE !!!
            `-MemberExpr 0x387f3f8 <col:17, col:20> 'char' lvalue ->id 0x387efa0
              `-ImplicitCastExpr 0x387f3e0 <col:17> 'ss *' <LValueToRValue>
                `-DeclRefExpr 0x387f3b8 <col:17> 'ss *' lvalue Var 0x387f2f8 'p' 'ss *'


text:
Show warning comments near the location of the code. Fixes #374

close #374

This change is

Reviewed 3 of 5 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 4 of 5 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


text:
Fix unsubscribing from ports

Hey, so I found a weeeee bug in the unsubscription of a port. It would only pop the last added subscription instead of the one you are actually unsubscribing. Let me know if there is anything else that needs to be done to get this merged.

I think this still behaves badly in the case where it's given a handler that isn't subscribed, since array.splice(-1, 1) removes the last element of array.

@dfreeman nice catch. Fixed in db7efc54963afcffa4b127de8fbe4b6314779947.

Thanks! It works this way in the upcoming version of things.

authors: RGBboy dfreeman evancz

repo: elm-lang/core


text:
Add Tuple.mapBoth function

Helper function when you need to apply a function to both elements in a Tuple. Like a combination of mapFirst and mapSecond:
mapBoth ((+) 1) (6, 8) == (7, 9)

Ummm... I think I'm missing something. Build is failing with an elm-make: elm-package.json: openBinaryFile: does not exist (No such file or directory) error, which I'm getting locally as well. I'm looking around for some technical documentation on contributing to core but haven't been able to find it or anything that is helpful with dealing with this error. Could someone point me in the right direction?

Maybe the most important information to you, wanting to contribute to core, is this document: https://github.com/elm-lang/core/blob/master/.github/CONTRIBUTING.md
Note in particular what it says in the "Adding New Functions" section.

@jvoigtlaender thanks for sending me that. What's documented there totally makes sense. If this seems like it belongs in an *-extra package instead, then I can move it.

Contributing it to https://github.com/elm-community/basics-extra would make sense. That package already has a section of tuple functions (http://package.elm-lang.org/packages/elm-community/basics-extra/2.1.2/Basics-Extra#=>).

This is already in the dev branch I believe!

authors: evancz jvoigtlaender minond


text:
Fix wrong Unix Epoch date

Unix time began on 1/1/1970, not 1990.  Given the way all of the Date library examples are using dates in 1990, this seems like a clear typo.

This is an already deprecated version of the docs, I see. My bad.


text:
Remove superfluous git pull

That git pull leads to surprising "error messages", like here: https://github.com/elm-lang/elm-platform/issues/62#issuecomment-135454405

You are not currently on a branch. Please specify which
branch you want to merge with. See git-pull(1) for details.
git pull <remote> <branch>

The commit that introduced that call says it was done to make sure to get the latest stuff if it's already checked out. But actually, when that call happens, we have just a moment before done a git clone to get the repository in the first place. So we do have the latest stuff for sure, don't we?

I think the idea was that things work nicer if you have the structure already and want the latest of all the master branches.
I sort of think we should break this into more than one script for the different scenarios.

authors: evancz jvoigtlaender

repo: elm-lang/elm-platform


text:
Events.onWithOptions doesn't work

In trying to create non-propagating events, I've discovered the aforementioned function doesn't work at all, not even with default options. Here I attempt to create an event that should be equal to onClick, yet it does nothing at all. The regular onClick (as commented out) works fine.
import Html.App
import Html
import Html.Attributes as Att
import Html.Events as Events
import String
import List
import Json.Decode

-- MODEL

type alias Model = Bool

init = False

-- UPDATE

type Msg = Click

update msg model =
  not model

-- VIEW

newEvent eventName msg =
  Events.onWithOptions
    eventName
    Events.defaultOptions
    (Json.Decode.succeed msg)

onClick = newEvent "onclick"

(=>) = (,)

view model =
  Html.div
    [ Att.style
      [ "position" => "absolute"
      , "width" => "100px"
      , "height" => "100px"
      , if model then "background-color" => "blue" else "background-color" => "red"
      ]
    , onClick Click
    --, Events.onClick Click
    ]
    []

-- APP

main: Program Never
main =
  Html.App.beginnerProgram
    { model = init
    , update = update
    , view = view
    }

Ahhh, I see. Maybe that should be made more explicit in the documentation. Although it can be concluded from the example given. I might publish a PR adding more details. Anyway, thank you.

repo: elm-lang/html


text:
Symstore add doesn't seems to care about previous transactions

I would expect that when I'm doing another add, symstore would know that files with the same checksums were already added in previous transactions and only add the ones that are actually changed. It seems all files are always added to the transaction file even though the update was not really necessary as the file was previously known in the transaction files (I'm not necessarily talking about OS file system level which should not matter).

If you mean the transaction list files in '000Admin', the 000Admin/0000000XXX file, then it's true. Symstore does not take into account any previous transactions, it will list all file specified with the 'add' command.
This is the expected behavior. I checked what the miscrosoft's symstore.exe does, and it behaves in the same way.
Does that cause any problems for you?

No, not necessarily. I just thought it's the reason it existed. Thanks for checking this out and thanks for the package.

authors: elmirjagudin mihaipopescu

repo: elmirjagudin/symstore


text:
Unsupported UI Hint

The image below from when using the docker image elsaworkflows/elsa-server-and-studio-v3
Something should be off, correct?

I was actually using this: https://hub.docker.com/r/elsaworkflows/elsa-v3 This seems broken, I would suggest removing it if possibly.

repo: elsa-workflows/elsa-studio


text:
New tmp form doesn't work on multiple commands (tmp may only be used inside function)

Running the following command is not allowed with the new tmp syntax:
> echo "A" ; tmp TXT = "B" echo $TXT

compilation error: 11-34 in [tty]: tmp may only be used inside a function
compilation error: tmp may only be used inside a function
[tty 11], line 1: echo "A" ; tmp TXT = "B" echo $TXT

The opposite order also doesn't work:
> tmp TXT = "B" echo "A" ; echo $TXT

compilation error: 0-23 in [tty]: tmp may only be used inside a function
compilation error: tmp may only be used inside a function
[tty 12], line 1: tmp TXT = "B" echo "A" ; echo $TXT

See https://elv.sh/ref/language.html#tmp. Your first example should be written something like the following. Note I've introduced a TXT var for the first echo as I think that more clearly shows the expected behavior:
0.000234 elvish> var TXT = a
0.000261 elvish> echo $TXT
a
0.000258 elvish> { tmp TXT = b; echo $TXT }
b
0.000279 elvish> echo $TXT
a

Maybe I'm misunderstanding something - my intent was actually to override some environment variable, so it looked a bit like this:
yarn build ; E:A=B E:C=D yarn deploy

What is the new recommended syntax?

What is the new recommended syntax?

It's the same as my previous example:
yarn build
{ tmp E:A = B; tmp E:C = D; yarn deploy }

I came across this while searching issues for this exact error message. I came across this because the 0.18 docs say to use tmp as a replacement for the old/deprecated behavior. From https://elv.sh/blog/0.18.0-release-notes.html#deprecated-features

The legacy temporary assignment syntax (e.g. a=foo echo $a) is deprecated. Use the new tmp command instead (e.g. tmp a = foo; echo $a).

How are we to to do temporary variable assignment in 0.19? Having to do it in a function seems very odd for something that I'd think happens pretty regularly.
Happy to open a separate issue, but thought to chime in here first.

@mmlb, What makes you think temporary assignments happen "pretty regularly"? My experience has been they are used rarely and almost always to temporarily change an env var, not a regular var. Note too that you don't have to define a function. A simple lambda to create a new scope is all that is needed. That's four extra characters (not counting the need for tmp): {  }.

Hey @krader1961 you're right about the regular-var vs env-var, I was definitely thinking about temporary envvar overrides. I don't think the distinction matters for the issue at hand, as env vars are also affected, for example I regularly run E:TERM=xterm-256color vagrant ssh -t provisioner and similar uses of temporarily override an envvar. tmp still doesn't work here, with out some sort of function (or lambda).
I get it that its only a few extra characters, but it just feels weird. How often do you type a command, have it fail in some form, you then go back and add/tweak an envvar to the line? Now we're going to also wrap in { } too? It feels cumbersome to me.
sidenote: this feels like a shift from favoring interactive use over to favoring scripted instead. Maybe its just me but elvish is still primarily an interactive shell to me, I've only got a couple of elvish scripts.

@mmlb, You wrote
For example I regularly run E:TERM=xterm-256color vagrant ssh -t provisioner

Why? Are you running vagrant from inside a Tmux session and don't want the Tmux TERM value (e.g., tmux-256colors) to be used by the vagrant command? If that is the case under what conditions would you want the vagrant command to use the Tmux TERM value? Why not simply define a vagrant function that always temporarily sets TERM to your desired value then invokes e:vagrant?

How often do you type a command, have it fail in some form, you then go back and add/tweak an envvar to the line?

Extremely rarely. Obviously, for you, it is more frequently. But, really, how often do you do this? Can you provide some real examples? There may be another way to address your use case(s).

Going back through history I'm at just shy of 1% of my commands match grep '^E:'. Things like:
E:DHCP_LEASE_TIME=1d ./tinkerbell
E:DHCP_LEASE_TIME=240h ./tinkerbell
E:DHCP_LEASE_TIME=30m ./tinkerbell

@mmlb, You wrote
For example I regularly run E:TERM=xterm-256color vagrant ssh -t provisioner

Why? Are you running vagrant from inside a Tmux session and don't want the Tmux TERM value (e.g., tmux-256colors) to be used by the vagrant command? If that is the case under what conditions would you want the vagrant command to use the Tmux TERM value? Why not simply define a vagrant function that always temporarily sets TERM to your desired value then rune:vagrant?

note this is just one example, the most recent and such fresh in my brain. I override TERM because I'm using kitty and the remote vagrant vm doesn't know about TERM=xterm-kitty. I find that overriding envvars is usually for immediate experimentation and wrapping in a script or updating rc file just doesn't really make sense.

I also run a variety of DevOps-related commands, and I often have to specify things for tools via env vars - AWS_PROFILE, AWS_REGION, sometimes our ENV_NAME, TERM, etc.
For some tools, it's a common form of parameter-passing, but I understand this may not be the primary goal of a more structured shell like Elvish.

@mmlb, For you tinkerbell examples I would write a very simple function along these lines:
fn tinkerbell {|lease-time|
    tmp E:DHCP_LEASE_TIME = $lease-time
    ./tinkerbell
}

For your vagrant example I would probably use an abbreviation but a function would work too.
@hexsel and @mmlb, For those cases where there isn't enough regularity to the overrides to justify an abbreviation or function I would use the env command:
env A=B C=D yarn deploy

Also, while env should be available on every UNIX platform Elvish runs on it is trivial to implement as an Elvish function. Which could allow specifying env vars with or without the E: namespace prefix in order to avoid the dissonance of being required to type env A=B yarn deploy rather than env E:A=B yarn deploy.

@mmlb, you replied

Going back through history I'm at just shy of 1% of my commands match grep '^E:'.

I would conclude you use that mechanism "rarely" but perhaps not "extremely rarely". I'm willing to bet that most of those temporary env var changes can be handled by my prior comment(s) to use a function or command abbreviation. And the remainder can be handled by the env command. So I am still not convinced the legacy temp var syntax should be reinstated.

authors: hexsel krader1961 mmlb

repo: elves/elvish


text:
Warn user when using deprecated builtin features

(Edit: see https://github.com/elves/elvish/issues/898#issuecomment-602282707 for the current thinking on how to implement.)
Implement a method on *Frame that shows a deprecation message:
func (*Frame) Deprecate(reason string)

However, during the entire lifecycle of an Evaler, the same deprecation message should only be shown for the same source location once. For example, if you have a function that uses a deprecated feature, the same message should only be shown the first time the function is called.
Until this feature is implemented, deprecated feature will have a TODO(#898): Formally deprecate feature X attached to them in the Go code.

I don't see how this proposal proactively tells users that a given feature is deprecated. They have to actually use the feature to see the deprecation message. Presumably you still intend to publish the deprecation, independent of this feature implementation, in a CHANGELOG file or similar mechanism.
What happens if the feature is used in a non-interactive context? If it occurs in a non-interactive context how does the deprecation message interact with the consumer of the output of the deprecated feature?

@krader1961 you are right, deprecating a feature also requires documenting it properly. I changed the title of this issue to reflect it.
Let me elaborate a bit on why I closed #939 citing this issue; I consider this functionality to be a prerequisite to deprecating any feature. If we document that something is deprecated, people can proactively change their code to avoid deprecated features, but (1) it's easy for people to miss part of the release notes (2) humans are error-prone and may not find all uses. Instead, it's much better for a machine (in this case, the Elvish interpreter) to discover uses of deprecated features and warn the user.
Re. where the warning goes in a non-interactive context: the warning goes to stderr.

On the technical side, a better way to implement this mechanism is during compilation instead of evaluation. This has a few advantages:

The deprecation warning is shown even if the code that uses the deprecated function or variable is not executed

Deprecation warnings are clustered - all deprecation warnings for the same source are shown at the same time, making it easier to fix in batch.

There is no need to remember that a deprecation message has already been shown for a certain source location, since each source is compiled only once.

Re. where the warning goes in a non-interactive context: the warning goes to stderr.

The risk is that in a non-interactive context the deprecation warning, even when written to stderr rather than stdout, can break things. Other languages, such as Python, behave per your proposal; modulo Python's configurability with regard to this behavior. I've actually been bitten by this so am "gun shy" about introducing it in a new language. Yet I don't have a better proposal.
Also, PR #939 did not say that the builtins would be removed after a specific release. It said that the features could be removed any time after that release. Including long in the future after this issue is resolved. It served to tell users that they should start future-proofing their programs. Not to mention hinting why there were two pairs of seemingly identical commands.

Re writing warning to stderr can break things: care to elaborate how you were bitten?
Re when to remove features: you are right that we can document the deprecation first, and implement the warning mechanism after that. But I'd like to send a more consistent message to users on what "deprecation" means - which should be all of the following at the same time:

Code that uses such feature will see warning when run
Deprecation is documented
Feature deprecated in release N will be removed in N+1


text:
Runtime fix for #402

This only fixes #402 at runtime level , meaning that in order to restart the daemon you have to start another instance of an elvish client .

Improved error checking and handling has been implemented in 14ca5756d1bef16ffc5a2b35696c7a398cd352c5, so I am closing this. Still, thank you!

authors: adracea xiaq


text:
Apollo Server GraphQL Subscription Documentation Missing

there is no documentation on how to implement apollo server graphql subscription using the bun websockets or any websockets

there is no documentation on how to implement apollo server graphql subscription using the bun websockets or any websockets

I just checked https://github.com/elysiajs/awesome-elysia but didn't see anything there for this. Seems like a non-common implementation so you might not have too may docs out here on this!

authors: normanaranez print-VarunSharma

repo: elysiajs/elysia


text:
Not a bug, but a feature request.

Could you add an option to make the background colour transparent?
This would be handy for various desktop themes without changing the existing colour scheme.
Thank you...

It may be. Thanks.

repo: emartisoft/AmiWeatherForecasts


text:
Handle case where pin value could be 0

In the case where GPIO mapping could look like this:
PA5:
SPI1_SCK: 0
The pin would not get any generated impl because the if expression would
evaluate to false.
Fix this for all cases in gen.py by comparing against None

Doh!
LGTM

Very tricky, nice catch!

authors: Dirbaio bobmcwhirter lulf

repo: embassy-rs/embassy


text:
stm32/can: docs, cleanup interrupt handling.y

stm32/can: cleanup interrupt traits.
stm32/can: docs.

bender run


text:
format request: audio URL (podcast / music / mp3)

I'm currently manually loading, chunking, and writing buffers for mp3 files retrieved via RSS urls. Would be great to use the abstractions from embedchain to just call .add("audio_url","https://mypodcasthost.com/mypodcast/myepisode.mp3")

How do you want to convert to chunkable text?

Hi @heaversm, we have the capability to process audio files that can accept both a file path and an audio URL. Check out the documentation here: https://docs.embedchain.ai/components/data-sources/audio.

authors: Dev-Khant cachho heaversm

repo: embedchain/embedchain


text:
Create gpt4free.py

Add gpt4free

@Yusufibin thanks for the PR. We don't plan to add gf4 support in the near future. We will let you know if we plan to do so in the near future.

authors: Yusufibin deshraj