bug: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.util.concurrent.ExecutionException: org.eclipse.lsp...

IDE Information
GoLand 2024.2.3
Build #GO-242.23339.24, built on September 26, 2024
Licensed to Michael Henderson
Subscription is active until January 31, 2025.
Runtime version: 21.0.4+13-b509.17 aarch64 (JCEF 122.1.9)
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Toolkit: sun.lwawt.macosx.LWCToolkit
macOS 14.6.1
GC: G1 Young Generation, G1 Concurrent GC, G1 Old Generation
Memory: 16000M
Cores: 10
Metal Rendering is ON
Registry:
ide.completion.variant.limit=500
suggest.all.run.configurations.from.context=true
ide.experimental.ui=true
i18n.locale=
terminal.new.ui=true
Non-Bundled Plugins:
org.asciidoctor.intellij.asciidoc (0.43.1)
com.sourcegraph.jetbrains (7.0.13)
com.intellij.tailwindcss (242.20224.204)
Bug Description
Opened IDE
Additional context
Stacktrace:
java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.util.concurrent.ExecutionException: org.eclipse.lsp4j.jsonrpc.ResponseErrorException: codeActions/provide: document not found for file:///Users/wraith/JetBrains/tribenet/ottomap/README.md

Error: codeActions/provide: document not found for file:///Users/wraith/JetBrains/tribenet/ottomap/README.md
    at /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:133463:19
    at /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:134386:18
    at runNextTicks (node:internal/process/task_queues:60:5)
    at process.processImmediate (node:internal/timers:449:9)
    at process.topLevelDomainCallback (node:domain:160:15)
    at process.callbackTrampoline (node:internal/async_hooks:128:24)
    at async /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:132923:44
	at com.intellij.util.ExceptionUtil.rethrow(ExceptionUtil.java:136)
	at com.intellij.openapi.progress.util.ProgressIndicatorUtils.awaitWithCheckCanceled(ProgressIndicatorUtils.java:399)
	at com.sourcegraph.cody.inspections.CodyFixHighlightPass.doCollectInformation(CodyFixHighlightPass.kt:104)
	at com.intellij.codeHighlighting.TextEditorHighlightingPass.collectInformation(TextEditorHighlightingPass.java:67)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.lambda$doRun$2(PassExecutorService.java:431)
	at com.intellij.platform.diagnostic.telemetry.helpers.TraceKt.runWithSpanIgnoreThrows(trace.kt:118)
	at com.intellij.platform.diagnostic.telemetry.helpers.TraceUtil.runWithSpanThrows(TraceUtil.java:36)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.lambda$doRun$3(PassExecutorService.java:426)
	at com.intellij.openapi.application.impl.AnyThreadWriteThreadingSupport.tryRunReadAction(AnyThreadWriteThreadingSupport.kt:291)
	at com.intellij.openapi.application.impl.ApplicationImpl.tryRunReadAction(ApplicationImpl.java:965)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.lambda$doRun$4(PassExecutorService.java:417)
	at com.intellij.openapi.progress.impl.CoreProgressManager.lambda$executeProcessUnderProgress$13(CoreProgressManager.java:660)
	at com.intellij.openapi.progress.impl.CoreProgressManager.registerIndicatorAndRun(CoreProgressManager.java:735)
	at com.intellij.openapi.progress.impl.CoreProgressManager.computeUnderProgress(CoreProgressManager.java:691)
	at com.intellij.openapi.progress.impl.CoreProgressManager.executeProcessUnderProgress(CoreProgressManager.java:659)
	at com.intellij.openapi.progress.impl.ProgressManagerImpl.executeProcessUnderProgress(ProgressManagerImpl.java:79)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.doRun(PassExecutorService.java:416)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.lambda$run$0(PassExecutorService.java:392)
	at com.intellij.openapi.fileTypes.impl.FileTypeManagerImpl.cacheFileTypesInside(FileTypeManagerImpl.java:802)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.lambda$run$1(PassExecutorService.java:392)
	at com.intellij.openapi.application.impl.AnyThreadWriteThreadingSupport.executeByImpatientReader(AnyThreadWriteThreadingSupport.kt:486)
	at com.intellij.openapi.application.impl.ApplicationImpl.executeByImpatientReader(ApplicationImpl.java:178)
	at com.intellij.codeInsight.daemon.impl.PassExecutorService$ScheduledPass.run(PassExecutorService.java:390)
	at com.intellij.concurrency.JobLauncherImpl$VoidForkJoinTask$1.exec(JobLauncherImpl.java:259)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:507)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1491)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:2073)
	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:2035)
	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:187)
Caused by: java.util.concurrent.ExecutionException: java.util.concurrent.ExecutionException: org.eclipse.lsp4j.jsonrpc.ResponseErrorException: codeActions/provide: document not found for file:///Users/wraith/JetBrains/tribenet/ottomap/README.md

Error: codeActions/provide: document not found for file:///Users/wraith/JetBrains/tribenet/ottomap/README.md
    at /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:133463:19
    at /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:134386:18
    at runNextTicks (node:internal/process/task_queues:60:5)
    at process.processImmediate (node:internal/timers:449:9)
    at process.topLevelDomainCallback (node:domain:160:15)
    at process.callbackTrampoline (node:internal/async_hooks:128:24)
    at async /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:132923:44
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2096)
	at com.intellij.openapi.progress.util.ProgressIndicatorUtils.awaitWithCheckCanceled(ProgressIndicatorUtils.java:370)
	... 27 more
Caused by: java.util.concurrent.ExecutionException: org.eclipse.lsp4j.jsonrpc.ResponseErrorException: codeActions/provide: document not found for file:///Users/wraith/JetBrains/tribenet/ottomap/README.md

Error: codeActions/provide: document not found for file:///Users/wraith/JetBrains/tribenet/ottomap/README.md
    at /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:133463:19
    at /Users/wraith/Library/Application Support/JetBrains/GoLand2024.2/plugins/Sourcegraph/agent/index.js:134386:18
    at runNextTicks (node:internal/process/task_queues:60:5)
    at process.processImmediate (node:internal/timers:449:9)
    at process.topLevelDomainCallback (node:domain:160:15)
    at process.callbackTrampoline (node:intern...

Hi there 👋 sorry for the inconvenience 🙇 we know about that problem and we fixed it already!
The original issue: https://github.com/sourcegraph/jetbrains/issues/2512
The fix: https://github.com/sourcegraph/jetbrains/pull/2539
The fix will be included in the next release. The next release should be available next week.
I kindly ask you for some more patience 🙏


fix: add organization property to project schema

I ended up working around this issue by using AdditionalProperties for now to avoid churn from upstreaming patches.


Why is 401 always returned in JwtBearerEvent OnChallenge?

When adding authentication in AddIdentityInfrastructure() in ServiceExtensions.cs, the code clearly states it should return 401 when OnChallenge event is invoked. Why is that? Nothing bad has really happened?
OnChallenge = context =>
                        {
                            context.HandleResponse();
                            context.Response.StatusCode = 401;
                            context.Response.ContentType = "application/json";
                            var result = JsonConvert.SerializeObject(new Response<string>("You are not Authorized"));
                            return context.Response.WriteAsync(result);
                        }

https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.authentication.jwtbearer.jwtbearerevents.onchallenge?view=aspnetcore-3.0

@emilwkarlsson Make sure you are passing auth token with prefix like "Bearer {YOUR TOKEN HERE}". and it will resolve your issue

@emilwkarlsson"Taşıyıcı {KENDİNİZ BURADAKİ TOKEN}" gibi bir ön ek ile yetkilendirme belirtecini ilettiğinizden emin olun. ve sorununuzu çözecektir

hi, i sent expired token to api.
statuscode cannot be set because the response has already started.
what are you know this problem ?
thanks

@emilwkarlsson Make sure you are passing auth token with prefix like "Bearer {YOUR TOKEN HERE}". and it will resolve your issue

Maaaan, so many thanks to you!


Enhance NER to load multiple override files

I've generated a KB file (Phase3-Override.tsv.gz) for the proteins specified by MITRE for Phase3. It contains almost 700 entries, some of which probably conflict with our existing KBs and NER override file.
While we could add all these entries to the front of the existing NER override file (more than doubling its size), it would be better if Reach could handle multiple override files. The Grounding package can already handle this (reading the list of override files from the application.conf file) but the NER needs to be enhanced to load and process multiple override KBs (preferably from a configuration list).
This would also be extremely valuable for "one-off" processing requests from various groups who wish to use their own, custom groundings which override our default mappings.

@hickst: can you please implement this?
We need to modify the loadAll method in KBLoader:
https://github.com/clulab/processors/blob/master/corenlp/src/main/scala/org/clulab/processors/bionlp/ner/KBLoader.scala

This is implemented and passes Processor tests on my machine but runs out of memory on Travis.

Ok. Then I propose to merge and release processors.
We will have to run some experiments for CMU in the next couple of days for the next DARPA eval. So we should have this available.

I'm retesting with latest right now, just to be sure.

Tests pass again BUT let's hold off on merge and release until we discuss -- there are other issues which may need to go into a new release.

Code was merged by pull request #129


Prepare 1.1.17

Description

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fddfb54) 81.19% compared to head (4f73277) 81.19%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #566   +/-   ##
=======================================
  Coverage   81.19%   81.19%
=======================================
  Files          28       28
  Lines        2271     2271
=======================================
  Hits         1844     1844
  Misses        337      337
  Partials       90       90

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix Fortune.js link on implementations page

Noticed the link to fortune.js was out of date on the Implementations page - this PR updates that link.
(Let me know if the commit message format is alright! Didn't see anything about it in CONTRIBUTING.md, so I just went with something standard.)

Thanks!


NIH PROD: Configure SSM

All AWS-related resources for SSM need to be configured. This mostly involved IAM roles, profiles and permission policies. Take example from sim4ife or aws tip. This is needed for Autoscaling EBS

Will be done by https://git.speag.com/oSparc/osparc-ops-deployment-configuration/-/merge_requests/851


Adding any list to scan adds the first list name to the "Already scanned" section.

Using the new/current version on https://fedifinder.glitch.me/, I scanned my Twitter followers (annoyingly it limited it to 1,000 of the 1,194 accounts I follow), and then went to scan the lists. As I was adding each of the 21 lists, I noticed they all showed (at the top of the "Scan your network" section) the same list name (that of the first list) - although the user counts were correct and the select list correctly removed the selected item.
Browser: Brave [Version 1.45.131 Chromium: 107.0.5304.110 (Official Build) (64-bit)
OS: Windows 10 Pro

Thanks for bringing this to my attention. The list names should be fixed now. Let me know if you still have that issue.
Regarding the number of scanned accounts: They are scanned in batches of 1000. And each batch is displayed in the list of scanned accounts. You should see "1000 followers, 194 followers". Each batch takes a few seconds.

Thanks - for some reason my brain didn't register the "1,000 followings, 195 followings" as two batches of the same scan (I was expecting "1,195" to appear somewhere).
Thank you again for a wonderful service!


Vanilla Hand Items & Dynamic Hands

Hi, thank you for Good Mod!
I suggest adding what I think are some pretty playable Features. This is mostly done for Dynamic Hands.
I have tried to maintain compatibility with previous features.
I did not add support for autoVanillaHandItems and autoDisableItems in Vanilla Hands Item. I will leave it up to your discretion.

Vanilla Hands Item
Shows the vanilla first person hands if have item
https://github.com/tr7zw/FirstPersonModel/assets/70120313/7cc138aa-6efc-4122-bbbe-a4c73a5d7ef7

Dynamic Hands
Dynamically changes the hands <vanilla <-> first person> when the viewing angle changes (need Vanilla Hands Item)
https://github.com/tr7zw/FirstPersonModel/assets/70120313/1f782219-c653-4969-9c57-9f9b165738e2

Ok I'll be honest, the Dynamic Hands idea is kinda neat. I'm not 100% sure what the difference between Vanilla Hands Item and Vanilla Hands/autoVanillaHandItems is? Might also be because I accidentally had this exact effect in the past due to a bug in the Vanilla Hands logic because it is such a mess of code. Might be worth to refactor, to get more options like this, and untangle the code.

Ok I'll be honest, the Dynamic Hands idea is kinda neat. I'm not 100% sure what the difference between Vanilla Hands Item and Vanilla Hands/autoVanillaHandItems is? Might also be because I accidentally had this exact effect in the past due to a bug in the Vanilla Hands logic because it is such a mess of code. Might be worth to refactor, to get more options like this, and untangle the code.

The Vanilla Hands Item idea is an alternative to Vanilla Hands so that you don't have to enter all of the required exception items.
About refactoring, honestly I don't have enough understanding so I can still refine it.

About refactoring, I honestly don't know what I would change.

I mean more my code than anything. Recent bugs like https://github.com/tr7zw/FirstPersonModel/commit/c1b71f375b0a1499676c80eddec9ce25121099a5 showed that there was too much overlap going on.

About refactoring, I honestly don't know what I would change.

I mean more my code than anything. Recent bugs like c1b71f3 showed that there was too much overlap going on.

Understood. Then look forward to solving this task......

https://github.com/tr7zw/FirstPersonModel/commit/559d1a7a42a705e506c783ce2645cc1daca1304a took quiet some time, but I think all the modes (auto vanilla hands, dynamic mode, vanilla mode for all items, vanilla mode even without items) now seem to work together,


Trials failed when no space left on device

Trials failed and the experiment stopped but no information on WEBUI. The relevant trial log file does not exist. Only use nnictl view can see the cause OSError: [Errno 28] No space left on device. It is better to show such errors in WEBUI.

Thanks @colorjam for reporting the issue, this sounds a good suggestion.
@Lijiaoa @chicm-ms - may you suggest about the dev effort needed here?

There is Error Status on the Overview page.


minor typo in readme

There is a minor typo in the readme: "1" instead of "i".
And thanks for the EuroCC workshop at VSC last week, that was very helpful.

Thanks for the fix!
Let us know if you have any further questions about the project!


Inconsistent Behaviour - Auto Generated

For eample, For two routes (not custom, but basic sails definition), I get two weirdly different results.
The routes in question are defined as such:
'DELETE /api/v1/projects/:project_id/assignments/:assignment_id': 'AssignmentController.destroy',
'DELETE /api/v1/projects/:project_id/members/:user_id': 'ProjectController.deleteMember',

The Swagger UI generated for the the first is (correct) :

And for the second (wrong):

Any idea why that happens and how to fix it?

Oh I think the default DELETE for assignment model is overriding your definition

No. Once I remove the route for assignment, nothing is generated for it in the Swagger UI (just a header 'Membership').
Automatic routes for actions are disabled in my project.

I guess the .destroy is taking it as default blueprint action


how to make virtual list responsive?

When we change the size of screen we need recalculate all the list.

This issue has been automatically closed due to inactivity.

If this issue was opened as a "help" ticket, please consider asking your question in the TanStack Discord.
If this issue represented an unfulfilled feature request, please start a new discussion instead of an issue.
If this issue needs more attention despite its activity, please comment to reopen, or open a new issue with updated information. Thanks!


Feature update

Update for https://github.com/CUNY-CL/yoyodyne/issues/47. Big update so quite likely introduces breaking changes. May be worth to push to another branch. LMK if that's preferred.
Follows up on conversation in above issue. Primary changes:

Encoder behavior managed by a source_encoder submodule instantiated in BaseEncoderDecoder with all logics found in modules sub module. Primary encoder types are: LSTMEncoder, TransformerEncoder, FeatureInvariantTransformerEncoder LinearEncoder.

To reduce boilerplate, the decoder portions of each model class are also instantiated in module class.  They inherit from various encoders. However, their behavior is different. Each model has a get_decoder method that instantiates the decoder class. Meanwhile, encoders are called directly by their various classes in BaseEncoderDecoder. It's a slight distinction, but is enforces that each model has a specific decoder while encoders can be swapped around.

For example: Calling a transformer model will ALWAYS create a TransformerDecoder through the TransformerEncoderDecoder get_decoder method. However, different encoders may be passed with the command line flag source_encoder_arch. If transformer is passed, then you have a standard transformer encoder-decoder. However, you may also pass a feature-invariant-transformer as a SOURCE architecture string and now you have a  feature invariant transformer. This drastically reduces boilerplate. (All models are just three encoder variants, four decoder types.)

Now supports variation in encoder behavior allowing for further work in https://github.com/CUNY-CL/yoyodyne/issues/2 and https://github.com/CUNY-CL/yoyodyne/issues/3 . (FWIW i did some custom experiments and you can see some decent performance this way. Just mind output sizes.)

For pointer-generators and transducers, now allows choice of feature encoders through a feature-encoder-arch command line parameter. Works the same as source_encoder but for features. Note that for pointer generator, choice of a transformer vs. lstm architecture will change how hidden weights are instantiated. (I just swap in init_hiddens from LSTMEncoderDecoder.)

Fixed issue with transducer found in https://github.com/CUNY-CL/yoyodyne/pull/71. This just adds an unk_symbol into the transducer edits. I saw a slight loss in performance (~.02) percent but should stop oov errors.

Also made some namespace cleanup decisions and code reduction. Please review if I broke something.

N.B. This involves https://github.com/CUNY-CL/yoyodyne/pull/63 issue so may be best to resolve that before this.

@bonham79 this is great. I want to say I give broad approval to the design changes here.
I see you mention #2 and #3 and can you confirm my understanding that this doesn't, for instance, let you do a transformer pointer-generator because if you tried there'd be a mismatch error raised here. Is that right?
If you want to finish #63 first fine with me. I think you had mine and Adam's reviews already, but I don't see responses yet so I'll leave that to you once you're ready.
I will now do a quick once-over for style stuff here.

I see you mention #2 and #3 and can you confirm my understanding that this doesn't, for instance, let you do a transformer pointer-generator because if you tried there'd be a mismatch error raised here. Is that right?

nope, you can. just pass --source_encoder_arch transformer instead of lstm. Just keep in mind that it now has to create new init variables for the decoder since it can't use the feature outputs. There's probably a few issues that need to be tidied up but I'm leaving that for when there's need for more in-depth experiments with that type of architecture. (Can only do so much over the weekend...)
Mismatch only really happens with the TransformerEncoderDecoder but that's because you need to double check the hidden values so they line up with the target embedding size. (I think the fix is you need to make the encoder hidden size half the target embedding.)

If you want to finish #63 first fine with me. I think you had mine and Adam's reviews already, but I don't see responses yet so I'll leave that to you once you're ready.

Did my responses not go through? I see them on my side. I'll double check this evening though.

No, I don’t see any of your responses to my notes (and the threads you have with Adam require your response too)…take your time tho.
K

On Jun 19, 2023, at 11:30 AM, tbartley94 @.***> wrote:
I see you mention #2 https://github.com/CUNY-CL/yoyodyne/issues/2 and #3 https://github.com/CUNY-CL/yoyodyne/issues/3 and can you confirm my understanding that this doesn't, for instance, let you do a transformer pointer-generator because if you tried there'd be a mismatch error raised here https://github.com/bonham79/yoyodyne/blob/7faebc0765807819b9f5a01e1540dd1b58d007cd/yoyodyne/models/modules/__init__.py#L15. Is that right?
nope, you can. just pass --source_encoder_arch transformer instead of lstm. Just keep in mind that it now has to create new init variables for the decoder since it can't use the feature outputs. There's probably a few issues that need to be tidied up but I'm leaving that for when there's need for more in-depth experiments with that type of architecture. (Can only do so much over the weekend...)
Mismatch only really happens with the TransformerEncoderDecoder but that's because you need to double check the hidden values so they line up with the target embedding size. (I think the fix is you need to make the encoder hidden size half the target embedding.)
If you want to finish #63 https://github.com/CUNY-CL/yoyodyne/pull/63 first fine with me. I think you had mine and Adam's reviews already, but I don't see responses yet so I'll leave that to you once you're ready.
Did my responses not go through? I see them on my side. I'll double check this evening though.
—
Reply to this email directly, view it on GitHub https://github.com/CUNY-CL/yoyodyne/pull/72#issuecomment-1597387659, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABG4OP7UREUXF7Y5XSO553XMBWCJANCNFSM6AAAAAAZLHPMWQ.
You are receiving this because you commented.

Some style notes. Looks very good though. It is perhaps not an accident that our overall architectural design mimicks FairSeq (the good parts...).

I consider it fairseq but with better commenting habits ;^)

I have to imagine this will make it easier to play with ByT5 and pre-trained character transformer models later...which is probably gonna give us state of the art performance on a lot of things...

yep! excited to give this some dissertation mileage with BERT style pretraining.

Oh, dictionary is much better idea. Just wasn't unsure about introducing a new pattern into forward passes.
From: Kyle Gorman @.>
Sent: Tuesday, June 20, 2023 9:57 AM
To: CUNY-CL/yoyodyne @.>
Cc: Travis Bartley @.>; Comment @.>
Subject: Re: [CUNY-CL/yoyodyne] Feature update (PR #72)
@kylebgorman commented on this pull request.

In yoyodyne/models/pointer_generator.pyhttps://github.com/CUNY-CL/yoyodyne/pull/72#discussion_r1235311690:

   assert batch.has_features

   source_encoded, (h_source, c_source) = self.encode(

       batch.source, self.encoder

   )

   features_encoded, (h_features, c_features) = self.encode(

       batch.features, self.feature_encoder

   )

   h_0 = self.linear_h(torch.cat([h_source, h_features], dim=2))

   c_0 = self.linear_c(torch.cat([c_source, c_features], dim=2))

   if self.beam_width is not None and self.beam_width > 1:

       # predictions = self.beam_decode(

       #     batch_size, x_mask, encoder_out, beam_width=self.beam_width

       # )

       raise NotImplementedError

   encoder_output = self.source_encoder(batch.source)

   if isinstance(encoder_output, tuple):

Can you think of any other way to do this? I really don't think this kind of pattern is okay; it's not obvious, undocumented, flaky to further changes. My suggestion would be to either return a dictionary or better yet some kind of tuple/namedtuple/dataclass with nullable values; the latter was the reason I moved us to the batches classes and I think it was a big improvement.

Reply to this email directly, view it on GitHubhttps://github.com/CUNY-CL/yoyodyne/pull/72#discussion_r1235311690, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVR4EMTYVWPHJQSBVWE55FLXMGT2TANCNFSM6AAAAAAZLHPMWQ.
You are receiving this because you commented.Message ID: @.***>

Oh, dictionary is much better idea. Just wasn't unsure about introducing a new pattern into forward passes.

It's relevant to the PR here so go for it. (I think a class is better than a dictionary though. We do use a dictionary as a return for some code that interfaces with Lightning because that's what it wants but that's not relevant here.)

It's relevant to the PR here so go for it. (I think a class is better than a dictionary though. We do use a dictionary as a return for some code that interfaces with Lightning because that's what it wants but that's not relevant here.)

iirc, isn't there a way to use dictionary syntax with dataclasses?

iirc, isn't there a way to use dictionary syntax with dataclasses?

At construction you trivially do something like:
mydc = MyDataClass(x=foo, y=bar)

but I'm not sure if that's what you have in mind. Dataclass objects also inherit a method astuple which does what you'd expect.

iirc, isn't there a way to use dictionary syntax with dataclasses?

At construction you trivially do something like:
mydc = MyDataClass(x=foo, y=bar)

but I'm not sure if that's what you have in mind. Dataclass objects also inherit a method astuple which does what you'd expect.

Alternatively, there is asdict

but I'm not sure if that's what you have in mind. Dataclass objects also inherit a method astuple which does what you'd expect.

Alternatively, there is asdict

nevermind, i was thinking about omegaconf behavior. We'll leave that to another fairseqification pr

Sorry I have not left any comments yet, will try to get to that soon.
Just wanted to comment that this is a massive PR (I wish I could pull it and explore it in a code editor), and I find it a bit scary. One thing I wanted to discuss is related to this "fairseqification". I often find that with these research libraries, I have an issue with the large amounts of abstraction entailed by conforming to OOP principles. Since as researchers, I think we would commonly like to tweak the inner-workings of these architectures, often in simple ways -- it would be great if we can ensure that such tweaks actually are simple. I guess what I mean is, it would be great to prioritize ease of development over minimization of code duplication, or other principles.
I have not used fairseq much for anything that is not out-of-the-box (largely due to so many breaking changes between minor versions, and inconsistent documentation) -- so it's hard for me to say if my critique is true, but more so I want it to be food for thought, or a prompt for discussion with people who may have a more nuanced perspective on these types of principles when developing libraries than I do.
As an example, though I love the general HuggingFace ecosystem, I despise the library. They take care of an amazing amount of complexity for training/finetuning LLMs, but imo introduce a huge amount of complexity for making even small changes to training/inference workflows. This is something i'd like to avoid.
So in summary, I think this adds the possibility of a lot of modeling variation in a really clean way, but am wary because I would advocate for us introducing abstraction, and thus, I think, more implicitly called methods, with extreme caution.

I basically agree with you @Adamits but think Travis has introduced only the minimal additional abstraction to do #2 and #3 in a non-crazy-making way, plus open the path to #73.

@Adamits I concur with the thoughts (and dislike of Huggingface code :^)). My preference for modulation is that it makes customization for research purposes easier. My goal with these types of PRs is to make model customization as easy as simply writing a new module that overloads prior behavior and then swapping it into whatever architecture works best. The idea here is to make the code for individual models focused only on what sets them apart from other models. (e.g. Feature-invariant transformer really is just a transformer with different encoding schema, Attentive-LSTM is just a different decoder).

Sounds good both :). @bonham79 what you've described makes sense -- I will try to give this a closer look.

Left a few comments. Wondering if, as you suggested, this can go into a development branch, and we can pull and train/predict with some models before merging?

I'm fine with a dev branch. Would probably be way safer.

How long are the files? 500 lines is a little long, 1,000 is very long.
On Thu, Jun 29, 2023 at 10:56 PM Travis Bartley @.***>
wrote:

@.**** commented on this pull request.
In yoyodyne/models/pointer_generator.py
https://github.com/CUNY-CL/yoyodyne/pull/72#discussion_r1247358350:

class Error(Exception):
pass
-class PointerGeneratorLSTMEncoderDecoderNoFeatures(lstm.LSTMEncoderDecoder):

"""Pointer-generator model with an LSTM backend and no features.
+class GenerationProbability(nn.Module):

I'm of the opinion that since they're small enough and only relevant to
those respective modules that they might as well be included. (I may also
just have an irrational dislike of modules imports...)
—
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/yoyodyne/pull/72#discussion_r1247358350, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4ONZS3A2XWGTMUIRTGTXNY56VANCNFSM6AAAAAAZLHPMWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

How long are the files? 500 lines is a little long, 1,000 is very long.

genertion_probabilities is just 50 lines.  pointer_generator is 369.
positional_encoding is just 60 lines. transformer is 307.
So the addition of the formers to the latters is trivial.

genertion_probabilities is just 50 lines. pointer_generator is 369.
positional_encoding is just 60 lines. transformer is 307.
So the addition of the formers to the latters is trivial.

Okay I'm fine with what you did, then.

Left a few comments. Wondering if, as you suggested, this can go into a development branch, and we can pull and train/predict with some models before merging?

I'm fine with a dev branch. Would probably be way safer.

If you want to do this this should have an informative branch name. Work on #73 should go into it too and then be merged into the master branch before very long (drop dead date August 1st?). I don't want to get into the situation where the master branch and this dev branch both have good, new features and you have to pick one or the other.

LGTM once you merge and tests pass.
I would lightly encourage you to consider doing this right into the master branch since merging when files are being moved around is an enormous pain, and a change of this scale it might take an hour or more. Plus we have a relatively stable release right before this on PyPI.

can you explain hat you mean? i'm not following

I’m just saying that it’d be easier to commit this to master than to fork
into a dev and a master branch at this point. I think Adam was also okay
with it.
On Wed, Jul 5, 2023 at 9:12 PM Travis Bartley @.***>
wrote:

LGTM once you merge and tests pass.
I would lightly encourage you to consider doing this right into the master
branch since merging when files are being moved around is an enormous pain,
and a change of this scale it might take an hour or more. Plus we have a
relatively stable release right before this on PyPI.
can you explain hat you mean? i'm not following
—
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/yoyodyne/pull/72#issuecomment-1622768150, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4OKBJMUZ7BT7A2FUFFDXOYGHFANCNFSM6AAAAAAZLHPMWQ
.
 I think Adam was also okay with it.
…
On Wed, Jul 5, 2023 at 9:12 PM Travis Bartley @.> wrote: LGTM once you merge and tests pass. I would lightly encourage you to consider doing this right into the master branch since merging when files are being moved around is an enormous pain, and a change of this scale it might take an hour or more. Plus we have a relatively stable release right before this on PyPI. can you explain hat you mean? i'm not following — Reply to this email directly, view it on GitHub <#72 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABG4OKBJMUZ7BT7A2FUFFDXOYGHFANCNFSM6AAAAAAZLHPMWQ . You are receiving this because you were mentioned.Message ID: @.>

ah gotcha, well we good to go in a second. there's some typing issues that only pop up on circle itnegration.


in python3.x not have StringIO

I think, should such an amendment.

Thanks! :sparkles:

AFAIK (I haven't checked), all of our documentation is written with 2.x in mind. If this holds true, I believe this is a slightly user-hostile change, as the majority of user userbase are not using Python 3.
If we want to be subtly suggestive that they should be using 3, we need to make sure it's happening everywhere, not just in this one spot.
Again, I haven't checked.

@kennethreitz Python 2.7 also has the io module: this change keeps the code working as expected on Python 2.

Ah yes, I forgot a StringIO class was also available under io in 2.7. Obviously, it's fine to not pander to 2.6 users :)
Either way, my above comment is generally relevant. We may want to start making our examples 3.x soon, if we want to "move the needle". I don't think there's any rush to do so, though.

Either way, my above comment is generally relevant. We may want to start making our examples 3.x soon, if we want to "move the needle". I don't think there's any rush to do so, though.

@kennethreitz have you looked at the downloads for requests recently? I'm not sure which version is "winning" but regardless, I think we can write the examples to work across all supported versions without having to have an opinion on whether they're all Py2 or Py3 or moving any needle.

I have, yes.
That would be my preferred method too. I'm speaking more if we'd like to make a more political move within our documentation, akin to what the Django project has done.
I don't think we need to, but am brining it up as a possibility.


require object keys to be in alphabetical order

Keeping things in alphabetical order just makes things easier to find.
{a: 1, b: 2} // good
{b: 2, a: 1} // bad

Possible configuration: minimum number of keys before the rule is enforced

Seems like a style issue. Maybe more appropriate for jscs? See what others think though

@caitp I agree. I can't think of any hazard resulting from ad-hoc key ordering. For lengthy object initializers, it might increase the chances of duplicate keys (which is definitely hazardous), but JSHint already warns about that.

And it's already a feature there :) http://jscs.info/rule/validateOrderInObjectKeys

Cool. Thanks for the feedback


could i reset shared count ?

During development, couple of sharing will be made.
Could I reset the count before putting the pages on live server ?
I found that pages putting on live server are still have the shared count made in development

No you cannot :(
Sharecounts are not 'remembered' by this script or the website you are using the script on, sharecounts are provided by the platform you shared on.
So when you share urlX on facebook, the count (1) is provided by Facebook. If you would like to reset this to 0 then you should ask facebook :)

@Ruud68, thanks for the detailed answer!


Don't import data if already present in db.

This implementation is quite naive, but works well for this "simple" example.

@jahow can you check if it work better ?
To check I'll do:
docker-compose down -v -t1
docker-compose up
#ctrl-C when import is over
docker-compose down -t1
docker-compose up
# it should not import.


Align.java

Add a private constructor to reinforce the utility class.
Rename filed in toString method StringBuilder buffer -> StringBuilder builder
the name buffer might be ambiguous, cause there is StringBuffer in java.

Spotless java check fails.
I'll recreate a new pull request.


Ideas to improve Scoop settings

I have a few ideas on how to improve Scoop options in the settings:

[x] Unify "Remove bucket" and "Show installed" like in https://github.com/martinet101/WingetUI/issues/284:

[ ] this option should be greyed out if Scoop is disabled or not installed:

[ ] All options under Scoop preferences shoud be disabled if it isn't detected:

3.1 However, Install Scoop should be disabled only if it is detected
3.2 Disable Scoop should be forced to be checked if it isn't detected

Maybe unify "Remove bucket" and "Show installed" like in https://github.com/martinet101/WingetUI/issues/284

I've got one more, unless I'm missing something. I already have scoop, and dozens of apps. How about "browse to scoop directory" or similar so it simply picks up current scoop settings (and installed apps).

I've updated this issue considering the new menu of version 1.5.1

That seems as if it's per-app setting, if I'm not being confused by screenshot? I'd rather have something more global, as eg. my Scoop has close to 100 packages, going through them one by one would be... Unwelcome. It would literally be less time to delete scoop and just start with WingetUI from scratch.
I have to note, I've since made my own workflows that help me manage existing apps with very little effort (couple scripts), so I've put WinGetUI on hold indefinitely. So if it's just me, and there's no other non-trivial way to "import" all existing Scoop packages, you can ignore my idea, I won't mind.
On the other hand, if you think it's welcome feature to add, then maybe look at Scoop and its own list command and see how/where it gets list of apps. I couldn't figure out if it keeps list of installed apps somewhere centrally or it goes through install directory every time (I tried, I'm just not that good and got lost ;D ).
Anyway, thanks for considering it!

I don't know how to achieve what you want (I believe it is to backup settings for all Scoop apps). Maybe @martinet101 knows better. WingetUI has a function to import and export packages, but their settings aren't stored. I know you can use scoop list in PShell to see all installed scoop packages as well. There's also scoop import and scoop export, but I don't know how to use them.
I also would like to know if there's a way to backup Scoop package settings like you propose. You could copy the folder C:\Users\ YourUsername \scoop\apps and the list of packages, but this would result in an unnecessarily big file. A more efficient solution would be to save only the persistant data from C:\Users\ YourUsername \scoop\apps\ AppName. That would certainly be useful IMO, but unfortunately I don't have the necessary skill...

Hi @luxzg,
I don't really understand what you are requesting.
I understand an option to open where scoop installs its apps?
Regarding the scoop app exporting, you can already export lists of installed packages (including scoop ones) from the installed section, but the individual applicaton data can't be exported or copied to a backup file without moving the whole scoop folder (as @ppvnf said).
Is this what you meant?

@luxzg, there's also this file C:\Users\your_user\.config\scoop\config.json in case you want to backup specifically the scoop settings itself (replace your_user)

Sorry all, I was away a little.
Just to clarify, what I'd like is for this UI to simply continue using my existing Scoop (together with settings and apps already installed).
To me this is logical, many UI apps, let's say fir example audio/video encoders, have such options. If you install UI it will A) ask where encoder exe is if you have it already, or B) install it's own copy of exe.
I don't see this app doing either. I installed UI using Scoop, opened UI, and it listed no Scoop apps. I tried to setup path to Scoop for it, it doesn't have the option.
Just seems like a missed opportunity. Someone installs Scoop, eventually discovers UI, installs it with Scoop, just to get basically told you can't use it without changing your setup.
IDK, am I missing here something obvious?
But as I said, I'm ok without UI. Just putting suggestion here as IMHO it would help this project and its potential users.

You might need to enable scoop from the settings window.
When installing from scoop, you won't be asked to install scoop, since the installation is done silently; so the default option will be taken: disabling it.
That's not a WingetUI's installer faullt, since this is what happens when using package managers: installers get ran silently.
The best thing you can do here is enable scoop from the settings window.

Well, I' like to think I'm not that stupid :)
OK, to cover everything:

installed Scoop
discoevered Winget UI through scoop search
installed WinGetUI package
run WinGetUI
went to "Installed packaged"
don't see packages from scoop (both searching by names of packages, scrolling, or sorting by source, no scoop)
go to menu, WinGetUI Settings
scroll to "Scoop preferences" section, open it
enable scoop, restart app, refresh, do whatever
back to "Installed packages" - still no Scoop apps
"Scoop preferences" offers "Install" button but I don't want to install (another) Scoop
added random bucket (php) to scoop via WingetUI - somehow it works
restarting and refreshing, still no scoop packages in "Installed packages"
I do have some weird stuff in package lists marked as "+" and trying to get info seems to just be "Loading..." forever (waited a minute)
these weird packages are:
Package names: "+", "+", "At", "For", "+", with same respective Package ID, and Installed versions "Category info", "FullyQualifiedErrorId", "line:1", "more", "scoop"

So I'd say it gets confused by my existing installation (default install, nothing weird here).
Perhaps having a Browse button and text field to enter path would fix the issue.

Thus far WingetUI for me is unusable. But hey, I'm OK with that.

What would happen if you tried uninstalling wingetui and installed it manually?
It is because some other people also faced issues with scoop when installing WingetUI from scoop.

Uninstalled through scoop. Downloaded and installed manually, left the recommended option to not install Scoop. But - nope, still the same issue. On a side note, "scoop list" in Terminal works, both as Admin and unprivileged. But WingetUI has same errors even after reinstall, tried both as unprivileged and with "Run as Admin".
Anyway, sorry, I really shouldn't spam this thread then, in case what you say is true (and you ARE the dev) then my "Browse" idea thing isn't needed, it's different issue altogether, and I should move this to separate bug thread instead. Might be best to delete my comments from here :-/

If you could open a new thread it would be great.
P.D. Try uninstalling wingetui, deleting a filder called .wingetui from your home user directory and installing again, installing scoop right from the installer. This will not overwrite your scoop installations or apps, but it will prepare wingetui to suport scoop from the installer, since it seems that WingetUI can't detect scoop for itself.

The feature for managing scoop is implemented and will come with the next version

The items listed here have been implemented, and they will come with the next update

3.1 However, Install Scoop should be disabled only if it is detected
3.2 Disable Scoop should be forced to be checked if it isn't detected

Thanks. I don't think it's really important, but would you mind explaining the reason why these suggestions were not implemented?
It's still possible to enable Scoop if it isn't detected:

It's still possible to install Scoop if it is detected:

3.1 However, Install Scoop should be disabled only if it is detected

Install scoop will also install git, a needed dependency that sometimes might lead to issues, so running the install script again may help in cases where scoop is installed but git is not

3.2 Disable Scoop should be forced to be checked if it isn't detected

In the way scoop is detected, if scoop -v returns a non-zero value, it will report scoop not being installed (i have faced this issue).
I am working on this so I can properly locate scoop, and then implement some kind of dialog to warn the user that scoop is enabled but not installed or something.


Git tracking local files related to ganache deployment

After setting up the local test environment (with ganache) using the "contributing" docs: https://docs.beamerbridge.com/contributing.html
i noticed that some of the test files are tracked by git.
To be more specific:

deployments/ganache-local folder
test-tokens.json
0x*.json

This files should be ignored (inside .gitignore) since they are related to the local setup.
@istankovic @fredo what are your thoughts on this?

* `deployments/ganache-local` folder

* test-tokens.json

* 0x*.json

This files should be ignored (inside .gitignore) since they are related to the local setup. @istankovic @fredo what are your thoughts on this?

I don't think these should be added to .gitignore because in the docs they are merely examples of filenames, which could change anytime. Whoever is reading the docs is free to choose filenames but we had to choose some filenames in order to make the examples work and easy to follow.
Additionally, if we change the filenames in the docs, should we then update .gitignore? Should we also update .gitignore whenever we add another filename (example) to the docs? That does not feel right.

I agree with @istankovic on this topic. gitignore should contain files which have consistent naming, are generated or used during development and is something what we really want to make sure, it should not be on remote.

Okay, thanks for the feedback from both sides!


[BugFix] Fix ConcurrentModification when olap table deepCopy

Signed-off-by: kangkaisen<EMAIL_ADDRESS>What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：
for https://github.com/StarRocks/StarRocksBenchmark/issues/325
The directly cause of this bug is in RangePartitionInfo's gsonPreProcess method, serializedIdToRange will be modified.
when I resolve this issue, I also find a potential bug，when analyze the olap table, we must hold the db lock.
Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 15 / 15 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/RangePartitionInfo.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/statistic/StatisticExecutor.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/sql/StatementPlanner.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/qe/SetExecutor.java
1
1
100.00%
[]

run starrocks_admit_test


 What should I do ?
Thanks :)

Morning @moreaupascal56 , is the source freshness exclusive from DBT cloud? if so, to connect to DBT cloud use the official DBT provider included with airflow


MODE sends to OSWRCH, OSBYTE 0 returns L=ZX Spectrum, INKEY(neg) retu…

MODE sends to OSWRCH, OSBYTE 0 returns L=ZX Spectrum, INKEY(neg) returns, INKEY-256 returns machine

Got carry wrong way around in INKEY


Overhaul soft reset

Now soft reset should work

Thanks for your work on soft reset. I will test that ASAP. :)


MidPointCircularSlider jumps when it circles through the max.

Is there anyway to make it so that it doesn't jump but goes through smoothly?

Hi @tommyyoon, the mid point circular slider can't take a value between the max and min, it's for that it jumps

Hey @tommyyoon, I will close the ticket for now. If you get the chance to look into this issue further, you can always respond to reopen the ticket


Fix colors disabling in non-interactive runs

What does this PR change?
The NoColor has to be set before assigning the consoleWriter.
Test coverage

No tests: add explanation

No tests: already covered

Unit tests were added

[X] DONE

Changelogs
Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository
If you don't need a changelog check, please mark this checkbox:

[ ] No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)
Before you merge
Check How to branch and merge properly!

Thank you for debugging and fixing this, Cedric!


How to get in touch regarding a security concern

Hey there!
I belong to an open source security research community, and a member (@ktg9) has found an issue, but doesn’t know the best way to disclose it.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

@JamieSlome Thanks for the advice. I will consider adding a SECURITY.md file. Now you can email me. My email address is<EMAIL_ADDRESS>


fix bug: CUDA 12 classifiers [skip ci]

Got following error when uploading 24.10 candidate to Test.PyPI
400 'Environment :: GPU :: NVIDIA CUDA :: 12.0' is not a valid classifier.
Corrected the CUDA 12 classifiers in pyproject.toml refer to https://pypi.org/classifiers/

build

build


multik in andriod14 ，the transpose funtion is too

in andriod13，my app with multik run a ndarray only use about 100ms to transpose a ndarray，but in andriod 14，this process needs 2000+ms

Oh, this is very strange. Matrix transposition is in multik-core and isn't tied to platforms or android versions.
Does the code differ from the version?
What is the size of the ndarrays?
Transposition, as far as I remember, is a view/lazy operation. We don't directly copy the data itself, we only change its representation. Copying occurs if data consistency is required for a subsequent operation.


Setup Kerberos principal/keytab for CDAP Master

Kerberos framework. Copied from HBASE.

few minor comments, not blocking.  👍 lgtm


Raw data viewer

Possible starting point. Based on Vispy.
Features

Simple paging system.
Load the entire page into GPU memory, no dynamic undersampling (first approach).
Load and show the previous and next pages.
Pan & zoom.
Change channel scaling uniformly.
Optional automatic page scrolling with a timer.

Inputs

NumPy array (or memmap array) of size (nchannels, nsamples)
h5py dataset
[Optional] spike trains (spike times, neuron indices, masks) to show the spikes within the traces

Options

Color of the channels
Page size

API

visual.traces: (n_samples, n_channels) array (may be a NumPy array, a memmapped array, or an HDF5 dataset: the slicing interface is the same)
visual.channel_order: the list of channel ids to display, from top to bottom. One can display a subset of the channels
visual.channel_colors: the list of colors to use for the channels (same length as channel_order)
visual.spike_samples: the full array of spike samples
visual.spike_clusters: the full array of spike clusters
visual.cluster_ids: the list of cluster ids to display
visual.cluster_colors: the list of cluster colors (same length as cluster_ids)
visual.channel_positions: the positions of the channels on the y-axis (linear uniform by default)
visual.channel_scale: a scalar with the scale factor for all channels on the y-axis
visual.interval: the (start, end) section to display
visual.sample_rate
visual.n_samples_per_spike


About SURVIVOR merge parameter

Hello,
I have a doubt about the parameters ./SURVIVOR merge sample_files 1000 2 1 1 0 30 sample_merged.vcf, I don't know what does the 0 before 30 stands for? What is the difference between the results of 1 and 0 here?
Best wishes ~

I think this parameter is disabled, as said in the instruction. So it doesn't matter what value you use just keep a position for it will be fine.

yes that parameter is currently disabled. I didnt want to change the interface for people that have implemented a script around it or so..
Thanks
Fritz


Rule @@||example.org^$important,document doesn't display as whitelisted if not located in the "User rules" filter

Hello,

Here is a second one. :relaxed:

:warning: This is only aesthetic and informational because the whitelisting takes place in fact (in AdGuard).

Case n°1
:large_blue_diamond: Correct result produced when the rules are added in the "User rules" filter.

:one: Add these two rules in the "User rules" filter:
example.org##h1
@@||example.org^$important,document

(The first one only serves as a means to verify the actual behavior.)

:two: Observe the correct behavior:

Case n°2
:large_orange_diamond: Incorrect result produced when the same rules are added elsewhere (in this case, in a new filter added via the "Add custom filter" button).

:one: First, remove the added rules from the "User rules" filter.
:two: And now, add these rules in a new filter (added via the "Add custom filter" button) instead.
:three: Observe the incorrect behavior:

... when in fact, the whitelisting also did take place — i.e. protection is disabled.
Otherwise the h1 element would have disappeared visually!

:memo: Same problem if added from an external source using the "Import filter" button for information. (Local test web server used in my case, using a well functional filter — loaded from <IP_ADDRESS> — by the way, because deactivating the whitelisting rule produces the expected result, i.e. h1 element hidden, for verification.)

:information_source: AdGuard Browser Assistant version: 1.1.16 (used on Firefox 93.0)
:information_source: AdGuard version: 7.7
:information_source: OS information: Windows 10 (21H1)

Thanks in advance. :thumbsup:

Both #1545 and #1546 should be fixed in current assistant. In this cases, switch is not available.


New version: demumble_jll v1.3.0+0

Autogenerated JLL package registration

Registering JLL package demumble_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/demumble_jll.jl
Version: v1.3.0+0
Commit: df8a158d786f3ad691d678923b7b0dba460ca938
Revision on Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/commit/15d2c21d66bd52d8c23de10cac47712be80fbf72
Created by: @maleadt

Manual intervention needed here.


Implemented GetUpperShape(), GetLowerShape() and GetTrailingEdgeShape() using face traits

This PR changes existing behavior. GetUpperShape(), GetLowerShape() and GetTrailingEdgeShape() now ignore the TiglShapeModifier (for now), but return shapes taking guide curves into account (which they previously didn't).

@bernhardmgruber I quickly scanned the code. Very good to simply query the faces with "Top" Traits etc. What, if - in case of guide curves - there are multiple Top faces?

@bernhardmgruber To make it more general, you could combine all faces that have a top/bootom face trait. Then, this code will still work, if e.g. a top guide curve is inserted.

Good idea. Just on a side node though: We still need to adapt the face traits of the wing for the case when there are more guide curves. Currently it is implicitly assumed that a wing segment has two or three faces, not more. I opened a new issue: #478

Now all faces having the same name (face trait) are used.

I forgot some applying the transformation and fixed the component segment loft generation. I think it should work now (at least our tests run now).

Can be merged once the CI has finished.


modular basins: Santa Barbara Channel

basin data:
CVMHB-Santa-Barbara-Channel-Basin.zip

points:
CVMHB-Santa-Barbara-Channel-Basin-points.zip

https://github.com/SCECcode/cvmhbn


Vector of types with inner types

Serializing a vector of types with inner types produces incorrect JSON:
The following code, for example :

class F {
public:
	F() : element(4) {

	}

	void serialize(JSON::Adapter& adapter) {
		JSON::Class root(adapter,"F");
		JSON_T(adapter, element);
	}
protected:
	int element;
};

class E {
public:

	E() : element1(54) {

	}

	void serialize(JSON::Adapter& adapter) {
		JSON::Class root(adapter,"E");
		JSON_E(adapter, element1);
		JSON_T(adapter, element2);
	}
protected:
	int element1;
	F element2;
};

class D {
public:

	void serialize(JSON::Adapter& adapter) {
		JSON::Class root(adapter,"D");
		JSON_T(adapter, data);
	}

	std::vector data;
};

Produces

{"D":{"data":[{"E":{"element1":54,{"F":{"element":4}}}}]}}

Which has an extra { } around F":{"element":4}

Hello Raphael,
Thanks for the report. Did you use the latest code when producing this error?

I believe so - are you having trouble reproducing it?

Not yet tried it! Thought this problem got fixed a day or so ago. Let me pull your PR.

It is a related problem but I don't think it is the same. The previous bug concerned the _vector flag being cleared when it shouldn't have been (with a vector of vectors), whereas this concerns it being set when it shouldn't (inner type of type within vector).
My PR removes the problematic _vector and handles the brackets for vectors in a different way, fixing this problem.

Sounds  ideal. I'm just going to pop your test case into the framework and see what gives.

The std::vector data; is supposed to be a vector of E? i.e.
std::vector<E> data;

Oh yes. Not sure how that got lost.

Excellent. Thanks for the PR.


Urgent help needed: FolioReader crashing in iOS

Hello,
Thank you so much for the great library, but recently after the release of iOS 11.1.2, the Folio is crashing in iOS when I try to open two or three .epub file..the problem is coming when I try to open scroll the book content, and opening another book, the app crashes.
Please see the code implemented in Xcode and the "null" error coming in Xcode.

@hebertialmeida can you pls help?

@zohdirizvi i am getting the same error. did you manage to fix it?

@hebertialmeida yes it is still happening for version 1.2.0.
If user press on cancel button while the screen was scrolling then that crash happens.

@hebertialmeida please respond asap.

Yesterday I have deployed a new version 1.3.0 this should be fixed. If it is not fixed I will need the books that are causing the crash.


Some fixes to watchr.rb and one improvement

watchr.rb wasn't working for me at all, so I applied a few fixes.
Specifically:

Removed the initial cd .. (I was getting an error message)
Changed the ">" to ">>" in the logging statements so log will be appended
Removed the "&" from the test execution logging statement, since its presence was causing nothing to get written to the log file for me.

I also updated the usage instructions to reflect the changes above.
Finally, I added support for desktop alerts on systems with libnotify-bin (notify-send command).  This means the developer can leave watchr.rb running and forget about it.  If they break a test, they'll get a desktop alert instead of having to tail the log file and keep an eye on it.

thanks. I'll review all this stuff once I'm upgrading the tutorial to the 1.0 apis in a few weeks


Not working after hosting on server

Hello,
when i'm running my code in local machine, it is working fine, this is the output -
[   '[youtube] Extracting URL: https://youtube.com/watch?v=zgJPTTiCK2E\n' +     '[youtube] zgJPTTiCK2E: Downloading webpage\n' +     '[youtube] zgJPTTiCK2E: Downloading ios player API JSON\n' +     '[youtube] zgJPTTiCK2E: Downloading android player API JSON\n' +     '[youtube] zgJPTTiCK2E: Downloading m3u8 information\n' +     '[info] zgJPTTiCK2E: Downloading 1 format(s): 140\n' +     '[download] Destination: 8306.mp3\n' +     '\r[download]   0.0% of    5.28MiB at  Unknown B/s ETA Unknown\r[download]   0.1% of    5.28MiB at  Unknown B/s ETA Unknown\r[download]   0.1% of    5.28Mi B at  Unknown B/s ETA Unknown\r[download]   0.3% of    5.28MiB at    7.62MiB/s ETA 00:00  \r[download]   0.6% of    5.28MiB at    6.18MiB/s ETA 00:00\r[downloa d]   1.2% of    5.28MiB at    8.89MiB/s ETA 00:00\r[download]   2.3% of    5.28MiB at   12.50MiB/s ETA 00:00\r[download]   4.7% of    5.28MiB at   19.31MiB/s E TA 00:00\r[download]   9.4% of    5.28MiB at   29.53MiB/s ETA 00:00\r[download]  18.9% of    5.28MiB at   43.63MiB/s ETA 00:00\r[download]  37.9% of    5.28MiB  at   51.39MiB/s ETA 00:00\r[download]  75.7% of    5.28MiB at   38.09MiB/s ETA 00:00\r[download] 100.0% of    5.28MiB at   35.93MiB/s ETA 00:00\r[download] 10 0% of    5.28MiB in 00:00:00 at 26.12MiB/s  ' ]
But when i depoly my app on heroku this is not working like it search the youtube link but i dont know why it is not downloading the files
[ 2023-12-18T14:36:22.725136+00:00 app[web.1]:   '[youtube] Extracting URL: https://youtube.com/watch?v=zgJPTTiCK2E\n' + 2023-12-18T14:36:22.725137+00:00 app[web.1]:     '[youtube] WzlO79d3S8c: Downloading webpage\n' + 2023-12-18T14:36:22.725137+00:00 app[web.1]:     '[youtube] WzlO79d3S8c: Downloading ios player API JSON\n' + 2023-12-18T14:36:22.725138+00:00 app[web.1]:     '[youtube] WzlO79d3S8c: Downloading android player API JSON\n' + 2023-12-18T14:36:22.725138+00:00 app[web.1]:     '[youtube] WzlO79d3S8c: Downloading m3u8 information\n' + 2023-12-18T14:36:22.725138+00:00 app[web.1]:     '[info] WzlO79d3S8c: Downloading 1 format(s): 140' 2023-12-18T14:36:22.725139+00:00 app[web.1]:  ]
My code
const stream = youtubedl(URL, { format: 'm4a', output: fileDown, maxFilesize: "104857600", preferFreeFormats: true, });
        await Promise.all([stream]).then(async (r) => {
            console.log(r);
            if (r?.includes("max-filesize")) {
                return sendMessageWTyping(from,
                    { text: "File size exceeds more then 100MB." },
                    { quoted: msg }
                )
            } else {
                let sock_data;
                if (command == 'song') {
                    sock_data = {
                        document: fs.readFileSync(fileDown),
                        mimetype: "audio/mpeg",
                        fileName: title + ".mp3",
                        ppt: true,
                    }
                } else {
                    sock_data = {
                        audio: fs.readFileSync(fileDown),
                        mimetype: "audio/mpeg",
                        fileName: fileDown,
                    }
                }
                await sock.sendMessage(from, sock_data, { quoted: msg });
                fs.unlinkSync(fileDown);
                console.log("Sent");
            }
        }).catch(err => {
            console.log(err);
            sendMessageWTyping(from, { text: err.toString() }, { quoted: msg });
        })

can u help?

Hello, when I try to get that video it's returning the following error:
ERROR: [youtube] zgJPTTiCK2E: Video unavailable. This video is not available

So there is nothing I can do there ☹

oki


eslint extension keep crashing in a project created by neutrinojs cli v8.3.0

eslint keep crashing in this project created by neutrinojs and yarn lint is okay.
step to reproduce
download this repository, just trigger a save action in any js file while "eslint.autoFixOnSave": true. I had set "eslint.runtime": "node" from this issue #634 , but no use.

extensionHost.ts:216 dyld: lazy symbol binding failed: Symbol not found: __ZN2v816FunctionTemplate3NewEPNS_7IsolateEPFvRKNS_20FunctionCallbackInfoINS_5ValueEEEENS_5LocalIS4_EENSA_INS_9SignatureEEEiNS_19ConstructorBehaviorENS_14SideEffectTypeE
  Referenced from: /Users/crown/Desktop/code/Trying/tt/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node
  Expected in: flat namespace

dyld: Symbol not found: __ZN2v816FunctionTemplate3NewEPNS_7IsolateEPFvRKNS_20FunctionCallbackInfoINS_5ValueEEEENS_5LocalIS4_EENSA_INS_9SignatureEEEiNS_19ConstructorBehaviorENS_14SideEffectTypeE
  Referenced from: /Users/crown/Desktop/code/Trying/tt/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node
  Expected in: flat namespace

versions
vscode-eslint: 1.8.2
vscode: 1.33.1
eslint: 5.16.0
node: v10.15.0

This is very likely cause by the fact that we use Electron as a node runtime and the native modules compiled are not working with the Electron node version. Can you try to use the following setting eslint.runtime and point it to a node version (the executable) for which the libraries got compiled. You can take the node version which is used when executing eslint in a terminal if that works successful.

Did you mean that ?

But it still broken ...

Yes, but this is now very strange since it crashes the extension host. Can you provide me with a GitHub repository I can clone that demos what you see (might require some additional info on how to reproduce)

@dbaeumer I had listed the address at the beginning, download this repository

Sorry, missed that.

Can't reproduce this. It works for me even without setting eslint.runtime

Any additional info?

I have open a new project based on neutrino@next, but it's no problem in there. This may be my own personal problem.
Turn off the issue first. If find the other reason, I will reopen this.


[BEAM-6703] Adds a job running Direct runner's ValidatesRunner with Java 11

Added a Jenkins job to run ValidatesRunner test suite on Direct Runner built using Java 8 with Java 11 runtime.

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

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

Run seed job

Run seed job


InfluxDB should better optimize memory usage of large queries

Feature Request
Proposal: [Description of the feature]
InfluxDB should better optimize large queries to use less memory.
Current behavior: [What currently happens]
Version: influxdb-1.0.2
Currently running a search over a 7 day time span will use much more memory than running the same search 7 times over 1 day spans.
This behavior can cause InfluxDB to OOM itself.
For example:

This is a graph of the _internal.runtime.HeapAlloc metric. The gaps are where I restarted InfluxDB to make sure it had a clean start.
The first bump at 17:14:00 is when I ran the following:
for i in {0..6}; do
  influx -database telegraf -execute "SELECT count(active) AS active FROM telegraf.autogen.postgresql_activity WHERE host = 'fll2gdbs10stg' AND time > now() - $((7-i))d AND time < now() - $((6-i))d AND active = true GROUP BY time(5m) fill(0)"
done

The second bump at 17:16:45 is when I ran:
influx -database telegraf -execute "SELECT count(active) AS active FROM telegraf.autogen.postgresql_activity WHERE host = 'fll2gdbs10stg' AND time > now() - 7d AND active = true GROUP BY time(5m) fill(0)"

Both queries will return the same data, but the first query uses far less memory.
Also by splitting the executions up into multiple queries, the memory usage plateaus, no matter how far back you go. However with the one big query, memory usage increases the larger the time range.
Desired behavior: [What you would like to happen]
InfluxDB should better utilize memory so it doesn't OOM itself.
Use case: [Why is this important (helps with prioritizing requests)]
Being able to run large queries without triggering OOM, and without requiring the user to have to manually break their query up provides a much better experience.

@phemmer Would you be able to test with the current nightlies by chance?  There are many changes that reduce memory usage for queries and writes.

@jwilder where are the nightlies kept these days? I used to be able to find them on the main download page (https://www.influxdata.com/downloads/), but they are no longer there.

@phemmer Sorry, we're in the process of getting the nightlies back on the Downloads page. Here are the links:
https://dl.influxdata.com/influxdb/nightlies/influxdb_nightly_amd64.deb
https://dl.influxdata.com/influxdb/nightlies/influxdb-nightly.x86_64.rpm
https://dl.influxdata.com/influxdb/nightlies/influxdb-nightly_linux_amd64.tar.gz

Ok, tried with the nighly, and wow. The difference is insane. The heap usage barely even moves. And it completes over 5 times faster.
Going ahead and closing this since it seems to be resolved.
Thanks

Can you tell when the new version was installed? :-)


[Windows] antrea-agent panics when disable AntreaProxy feature

Describe the bug
antrea-agent will panic if disable AntreaProxy.
To Reproduce
Disable AntreaProxy and start antrea-agent. antrea-agent will fail to start with panic messages.
featureGates:
# Enable antrea proxy which provides ServiceLB for in-cluster services in antrea agent.
# It should be enabled on Windows, otherwise NetworkPolicy will not take effect on
# Service traffic.
  AntreaProxy: false

The root case is that uplink table is not initialized in generatePipeline() when disable AntreaProxy feature.
Which will cause antrea-agent crashed when tring to access the table(nil pointer).


SSP API Interface

Changed Transfer() function from type uint8_t to uint16_t.
Changed "clock_divider" to "divider".

Codecov Report

Merging #163 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #163   +/-   ##
=======================================
  Coverage   73.23%   73.23%
=======================================
  Files          18       18
  Lines         710      710
=======================================
  Hits          520      520
  Misses        190      190

Continue to review full report at Codecov.

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


Dyes! And new textures that auri doesnt like!!

What is this.

Ah uh, oops. Left that in by accident!

Also why is the default signed scroll using a different coloured ribbon?

Because it looks nice :3

It's the only one...

Correct, because its the only neutral one.

But. Black would look good on it too.

The quill isn't different on the neutral scroll and quill, why would the written one be any different with the ribbon?

I mean, I thought it looked nicer I guess

Hm...

Alright, fine. Good to merge.

:D


Allowing mixing links and inventory variables

We can use links to specify some default values for group_vars and host_vars. This PR gives user ability to override default values within molecule.yml file. This align with Ansible variable precedence convention.
Inspired by https://github.com/ansible/molecule/pull/3501 and fixes https://github.com/ansible/molecule/issues/2759
A few people need this feature including myself but original ticket was closed because issue opener found another solution. I am trying to pickup the work.

What's the status of this?
Will this result in the following variable precedence? (Ordered from least to greatest)

provisioner['inventory']['links']['group_vars']['all']
provisioner['inventory']['links']['group_vars']['groupname']
provisioner['inventory']['group_vars']['all']
provisioner['inventory']['group_vars']['groupname']
provisioner['inventory']['links']['host_vars']['ansible_host']
provisioner['inventory']['host_vars']['ansible_host']
...followed by normal ansible role/playbook precedence


Find usages API not completely working

Describe the bug
Find label usages is not properly working on UI, the API requests seems good but the UI is not properly filled in:

The text should be something like  Report config <b>{loc.title}</b> in {loc.where} {loc.name ? loc.name : ""}
As you can see loc.where and loc.name  (and also loc.configId)are empty.
After some investigation, the API is properly returning those fields properly filled in.
IIUC the problem seems in the OpenAPI definition, as it was constructed assuming some sort of inheritance which does not actually exists in OpenAPI specs:

In the horreum-api we have an abstract LabelLocation schema which is extended by several locations, e.g., LabelInReport
The autogenerated client code (for the models) does not reflect that inheritance and you'll have two distinct classes, the LabelLocation and LabelInReport (the latter having all fields coming from LabelLocation and their owns)
The UI calls the client method findUsages which is returning LabelLocation[] the most generic one, thus it will simply discard all other fields

To Reproduce

Simply look for a label (in a specific schema having some usages on reports) and click find usages
You should see the table is not properly filled in
If you click the "Go to", it will redirect to an undefined report config id, this is because also loc.configId is undefined.

Version
What is the version of Horreum ?
Discovered on 0.13 but it was affecting also previous versions.

I think the proper way to handle this use case is using the Discriminator.


[FEATURE] HypEx in Kaggle

Please ask Ivan to do a comparison on big data and add a table with the comparison here


Centering and Font size using SpeedView

Thank you for this excellent project.  Really nicely done, and very helpful.
Short description
Issue: Having issues with Unit Text not centering, nor being able to change font size.  I set the setBackgroundColor to 255 on the SpeedView object, to remove the white that was there and wonder if that is part of what I am doing incorrectly?
Tool: Using Android Studio, using SpeedView which is in my Appbarlayout (centered_horizontal)
Longer description:
I am not sure where I am going wrong with centering the Text, but after adding in the following and reading through the threads and making changes, I still cannot seem to resolve.  I use Bottom_Center tag, but it shows the text on the bottom_right on my device/emulator.  I tried all the other tags, and they seem to be in reverse order in some capacity, but none put the text directly under the SpeedView.  I tried to use Speedometer and my project has issues with loading the main Context View.  I also tried changing the font size via the XML and object.  I have set the speed to transparent as it is not really needed for this project.
Please find the XML info I am using for reference:
<com.github.anastr.speedviewlib.SpeedView
android:id="@+id/speedView"
android:layout_gravity="center_horizontal"
app:sv_indicatorLightColor="#BB0000"
app:sv_unitTextSize="15dp"
sv_speedometerMode="TOP"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
app:sv_highSpeedColor="#4cff00"
app:sv_mediumSpeedColor="#FFFF00"
app:sv_lowSpeedColor="#ec0918"
app:sv_indicator="NeedleIndicator"
app:sv_lowSpeedPercent="33"
app:sv_mediumSpeedPercent="66"
app:sv_tickNumber="0"
app:sv_speedTextColor="@android:color/transparent"
app:sv_speedTextPosition="BOTTOM_CENTER"
app:sv_unitUnderSpeedText="false"
app:sv_markColor="@android:color/transparent"
app:sv_textColor="@android:color/transparent"
/>
Any suggestions would be helpful
Thank you.

For color issue, did you try to make sv_backgroundCircleColor transparent ?
May you add picture of your speedometer view?

I have that in my XML, and the colors look good.  It is the centering of the text itself that I am having an issues with.  The text is being pushed to the right hand size, and very tiny (not readable), when the request is to center in the middle/under the image (or any request for the location).  I tried it in a regular "Tab" window, and the text is indeed under the image, as shown.  This speedometer is in the Toolbar, and I wonder if the size of it is causing an issue with locating the text correctly?
Any suggestions?
Thank you again, this really is a fantastic item to have.

i tried your view, and i think if you change sv_unitUnderSpeedText from false to true it may solve your issue.


Most Popular and Highest Rated show VS 2019 extensions instead of VS 2022

@prashantvc this seems like a quick fix that may be impactful for VS 2022 extension discovery 🙂

Yeah, I agree. We need to change the query to surface VS 2022 extensions.
VS '19 and '20 being two different entries; '19 entries do come out on top

The fix for this issue has been pushed. I will update once the fix has been deployed to production. Thanks


Update sbt-scoverage to 1.8.2

Updates org.scoverage:sbt-scoverage from 1.8.1 to 1.8.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #230 (271529d) into main (4589378) will decrease coverage by 0.10%.
The diff coverage is n/a.

:exclamation: Current head 271529d differs from pull request most recent head f9e5c79. Consider uploading reports for the commit f9e5c79 to get more accurate results

@@            Coverage Diff             @@
##             main     #230      +/-   ##
==========================================
- Coverage   96.48%   96.37%   -0.11%
==========================================
  Files           7        7
  Lines         966      966
  Branches       93       93
==========================================
- Hits          932      931       -1
- Misses         34       35       +1

Impacted Files
Coverage Δ

core/shared/src/main/scala/cats/parse/Parser.scala
96.10% <0.00%> (-0.13%)
:arrow_down:

Continue to review full report at Codecov.

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


Map Wraps Around on Widescreen

@yuzhang0302  pointed out that the map wraps around if the screen is wide:

This seems to be the intended behavior of Mapbox and I don't see a way to work around it after reading the mapbox forums. I believe a decent workaround is to set a reasonable zoom level from the outset that just shows the available sensors.

I've updated the map to automatically calculate the center position and zoom levels based on the sensors available. You can see that the change sets this position and zoom automatically now:


Add snippets for JavaScript and Python

Description

Type of Change

[x] ✨ New snippet
[ ] 🛠 Improvement to an existing snippet
[ ] 🐞 Bug fix
[ ] 📖 Documentation update
[x] 🔧 Other (please describe): Adds the Java programming language.

Checklist

[x] I have tested my code and verified it works as expected.
[x] My code follows the style and contribution guidelines of this project.
[x] Comments are added where necessary for clarity.
[x] Documentation has been updated (if applicable).
[x] There are no new warnings or errors from my changes.

Related Issues

Closes #
Additional Context

Screenshots (Optional)

Click to view screenshots

wait i forgot something

should be good now


mariadb_configuration template uses current cookbook as template source; documentation issue

When defining a mariadb_configuration resource, the template resource in configuration.rb references new_resource.cookbook, which uses the name of the calling cookbook, which typically(?) isn't maraidb, and by default, doesn't have the templates in use defined.
================================================================================
Error executing action `create` on resource 'template[/etc/mysql/conf.d/some_conf.cnf]'
================================================================================

Chef::Exceptions::FileNotFound
------------------------------
Cookbook 'some-cookbook' (2.0.0) does not contain a file at any of these locations:
        templates/debian-7.8/conf.d.generic.erb
        templates/debian/conf.d.generic.erb
        templates/default/conf.d.generic.erb
        templates/conf.d.generic.erb

Changing the value to:
cookbook 'mariadb'

resolves this issue. Additionally, the documentation referencing the option hash as:
  option {foo: 'bar'}

fails with a syntax error, while something like
option :foo => bar,
  :baz => quux,

works. If these are both considered legitimate bugs, I'd be happy to put together a pull request to resolve them.

Please submit a PR, seems you're right :)

I just corrected the mariadb_configuration problem. I let you submit a PR concerning the documentation :)

@sinfomicien Sure, I can put together a pull request for the documentation. Thanks :)

As the PR is merged, i close the issue. Thx for your help!


tox: test upgrades to luminous only on stable-3.0

stable-3.0 must not attempt to run an upgrade from Luminous to Mimic, so
if running on stable-3.0 we should only do:

Jewel to Jewel minor
Jewel to Luminous
Luminous to Luminous minor

Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

jenkins test luminous-update_docker_cluster

Running the pipeline is not necessary here.


Rust-analyzer underlining missing

Rust-analyzer gives useful information about which variables are mutable and which methods mutate through underlining. This extension used to respect that, but now there is no underlining anywhere.

I'm not sure when it stopped exactly, but I remember it did work around July 2023. It's too bad because I love this feature, but I also love the true blacks!

For anybody else who has the same issue, adding this to my settings.json did the trick
"editor.semanticTokenColorCustomizations": {
  "enabled": true,
  "rules": {
    "*.mutable": {
      "underline": true
    }
  }
},

Thank you! Sorry for my long hiatus on GitHub/my open source contributions at large. I will get this updated when I get the chance to get my local dev environment going.
If you have properly tested I don't mind accepting a PR (didn't check if you opened one), but to get the extension updated I'll have to fight the monster that is figuring out how to break into my azure account.


Missing the file discourse.csv in the DomainIndependentModel

Hi,
When I run the the java code of DomainIndependentModel, I find the repo does not provide me with the discourse.csv, could you please add it in the repo or provide me the link to download it. Thanks.

Hi,
The discourse.csv is not required. As the file which I was referring to is unavailable. You can comment the code and proceed.


getDomRangeFromRangeCfi() does not accept some CFI generated from addSelectionHighlight() function

This issue is a Bug
Related issue(s) and/or pull request(s)
None
Expected Behaviour
CFI generated by reader.plugins.highlights.addSelectionHighlight() should pass the isRangeCfi() checking.
Observed behaviour
A sample CFI generated by the plugin function addSelectionHighlight() looks like /4,/64/1:433,/66/1:10 when the selection spanning across paragraphs. When the same CFI is fed into Readium's reader.getDomRangeFromRangeCfi() function, it failed the validation at cfi_interpreter.js's isRangeCfi() function.
Test file(s)
The epub file with no image downloaded from http://www.gutenberg.org/ebooks/271
Product
Readium JS and Readium JS highlight plugin.
On Linux.

I assigned @JCCR just as a heads-up, just in case there is already a fix via another issue or pull request.
Related: https://github.com/readium/readium-shared-js/issues/353

Related: https://github.com/readium/readium-shared-js/issues/345


Nightly rust warning: cross-crate traits with a default impl, like Sync, should not be specialized

Using the macro result in a warning with rust nightly:
For example, running cargo test in this crate result in many warning. First one:
error: cross-crate traits with a default impl, like `Sync`, should not be specialized
   --> src/lib.rs:233:17
    |
233 |                 unsafe impl Sync for $name<'static> {}
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
...
731 |         scoped_thread_local!(static BAR: for<'a> Foo<'a>);
    |         ------------------------------------------------- in this macro invocation
    |
note: the lint level is defined here
   --> src/lib.rs:136:23
    |
136 | #![deny(missing_docs, warnings)]
    |                       ^^^^^^^^
    = note: `#[deny(suspicious_auto_trait_impls)]` implied by `#[deny(warnings)]`
    = warning: this will change its meaning in a future release!
    = note: for more information, see issue #93367 <https://github.com/rust-lang/rust/issues/93367>
note: try using the same sequence of generic parameters as the struct definition
   --> src/lib.rs:215:9
    |
215 | /         $vis struct $name<$lt> where ::std::cell::Cell<::std::option::Option<$ty>>: 'static {
216 | |             inner: &$lt ::std::thread::LocalKey<::std::cell::Cell<::std::option::Option<$ty>>>,
217 | |         }
    | |_________^
...
731 |           scoped_thread_local!(static BAR: for<'a> Foo<'a>);
    |           ------------------------------------------------- in this macro invocation
    = note: `'static` is not a generic parameter
    = note: this error originates in the macro `scoped_thread_local` (in Nightly builds, run with -Z macro-backtrace for more info)

This warning was a false positive that should be fixed in rustc now.


Draft status cannot be set on child pages

When I set a parent page for a post in "Advanced -> Parent" I am no longer able to set the status on a page to "draft". I can set the switch to draft but when I hit "save" it always resets back to published.
Is there a reason for this behavior or is it a bug?
Using Bludit Pro 3.9.2 on PHP 7.2.8 (local server).

I confirm that this problem exist.
Just wondered why my new post is getting published after setting a parent.

This bug was fixed for the next version. thank you


Improve experience for low vision readers

From https://mailarchive.ietf.org/arch/msg/tools-discuss/0P6JR4SoKb-bbPM7-UEAD810zt0

I often read web pages referenced in e-mails with w3m, a text browser.  This gives me a view of the page which is approximately that of a speech browser.  I saw
IETF Logo Datatracker Dev Mode
• Groups
□ By area/parent
□ Apps & Realtime
□ General
□ Internet
□ Ops & Management
□ Routing
□ Security
□ Transport
□ IAB
□ IRTF
□ IETF LLC
□ Other
☆ Active AGs  etc, etc, etc,...
and some 50 odd lines before I got to the subject.  On behalf of "low vision" readers, could you put a link at the very top of the page to the subject matter.
It would save a lot of time, and doesn't get in the way of users of "high vision" browsers.
The page https://www.w3.org/WAI/people-use-web/tools-techniques/
gives an example of how to do this.
Thanks, Roger

Is this a regression with bs5? The old page structure was basically identical.

No, it's not a regression. I can remove the label, but I do think that any work here should happen in the bs5 framing and not the bs3.

Assign it to me? I'll add a skipnav link during codesprint.

See #3645.


Remove monit

Do we still need monit? There are no known bugs causing Mopidy to hang or crash, this extra complexity and load doesn't seem necessary.

I've got a bunch of Mopidy restart messages in /var/log/monit.log, but this could just be because I was hacking around (although I always try to stop monit first).
If you are not seeing any restarts being performed by monit then I agree, and we should probably take it out.

Hmm, I'm in favour of removing complexity, but I would argue that any decent consumer device would have a watchdog or similar to ensure that the device remained functional in spite of unexpected issues (and let's face it, software always has unexpected issues).
If monit really is causing extra load or complexity, are there alternatives? I've used daemontools in the past, which can automatically restart processes when they exit.

I'm still not sure what we need monit to do but we don't need it to sit there polling the http interface every 5 seconds (or whatever it's currently doing). I would really like to know some usage stats on how often it's stepped in and restarted Mopidy, excluding those reported time it's done so unnecessary and caused people issues.

I've included an extract of my monit.log file below. I keep Mopidy running 24/7 on a Pi Musicbox - it looks like monit does a restart about once a week (with a lot of manual restarts in between).
Not sure if it is worth the additional complexity or hassle, but it does seem to be doing its thing every now and then:
[SAST Aug  6 05:43:42] info     : Starting monit daemon
[SAST Aug  6 05:43:42] info     : Monit start delay set -- pause for 300s
[SAST Aug  6 05:49:00] info     : 'system_MusicBox' Monit started
[SAST Aug  6 19:35:17] error    : 'mopidy' failed protocol test [HTTP] at INET[localhost:6680] via TCP -- HTTP: Error receiving data -- Resource temporarily unavailable
[SAST Aug  6 19:17:20] info     : Starting monit daemon
[SAST Aug  6 19:17:20] info     : Monit start delay set -- pause for 300s
[SAST Aug  6 21:59:03] info     : 'system_MusicBox' Monit started
[SAST Aug  7 05:49:10] info     : monit daemon with pid [2700] killed
[SAST Aug  7 05:49:10] info     : 'system_MusicBox' Monit stopped
[SAST Aug  7 05:49:24] info     : Starting monit daemon
[SAST Aug  7 05:49:24] info     : Monit start delay set -- pause for 300s
[SAST Aug  7 05:54:43] info     : 'system_MusicBox' Monit started
[SAST Aug  7 06:26:44] error    : 'mopidy' process PID changed from 2744 to 3141
[SAST Aug  7 06:26:44] error    : 'mopidy' failed, cannot open a connection to INET[localhost:6680] via TCP
[SAST Aug  7 06:28:44] info     : 'mopidy' process PID has not changed since last cycle
[SAST Aug  7 08:32:53] error    : 'mopidy' failed protocol test [HTTP] at INET[localhost:6680] via TCP -- HTTP: Error receiving data -- Resource temporarily unavailable
[SAST Aug  7 08:34:58] error    : 'mopidy' failed protocol test [HTTP] at INET[localhost:6680] via TCP -- HTTP: Error receiving data -- Resource temporarily unavailable
[SAST Aug  7 08:34:58] info     : 'mopidy' trying to restart
[SAST Aug  7 08:34:58] info     : 'mopidy' stop: /etc/init.d/mopidy
[SAST Aug  7 08:35:29] info     : 'mopidy' start: /etc/init.d/mopidy
[SAST Aug  7 08:37:29] info     : 'mopidy' connection succeeded to INET[localhost:6680] via TCP
[SAST Aug 10 13:28:47] error    : 'mopidy' failed protocol test [HTTP] at INET[localhost:6680] via TCP -- HTTP: Error receiving data -- Resource temporarily unavailable
[SAST Aug 10 13:28:47] info     : 'mopidy' trying to restart
[SAST Aug 10 13:28:47] info     : 'mopidy' stop: /etc/init.d/mopidy
[SAST Aug 10 13:29:17] info     : 'mopidy' start: /etc/init.d/mopidy
[SAST Aug 10 13:31:17] info     : 'mopidy' connection succeeded to INET[localhost:6680] via TCP
[SAST Aug 10 14:28:39] info     : monit daemon with pid [2688] killed
[SAST Aug 10 14:28:39] info     : 'system_MusicBox' Monit stopped
[SAST Aug 10 14:32:13] info     : Starting monit daemon
[SAST Aug 10 14:32:13] info     : Monit start delay set -- pause for 300s
[SAST Aug 10 14:37:13] info     : 'system_MusicBox' Monit started
[SAST Aug 11 19:22:59] info     : monit daemon with pid [5417] killed
[SAST Aug 11 19:22:59] info     : 'system_MusicBox' Monit stopped
[SAST Aug 11 19:23:12] info     : Starting monit daemon
[SAST Aug 11 19:23:12] info     : Monit start delay set -- pause for 300s
[SAST Aug 11 19:28:29] info     : 'system_MusicBox' Monit started
[SAST Aug 13 15:33:17] error    : 'mopidy' failed protocol test [HTTP] at INET[localhost:6680] via TCP -- HTTP: Error receiving data -- Resource temporarily unavailable
[SAST Aug 14 14:17:37] error    : 'mopidy' process PID changed from 2708 to 6515
[SAST Aug 14 14:19:37] info     : 'mopidy' process PID has not changed since last cycle
[SAST Aug 14 15:55:39] error    : 'mopidy' process PID changed from 6515 to 12634
[SAST Aug 14 15:57:39] info     : 'mopidy' process PID has not changed since last cycle
[SAST Aug 14 16:20:18] info     : monit daemon with pid [2685] killed
[SAST Aug 14 16:20:18] info     : 'system_MusicBox' Monit stopped
[SAST Aug 14 16:20:38] info     : Starting monit daemon
[SAST Aug 14 16:20:38] info     : Monit start delay set -- pause for 300s
[SAST Aug 14 16:25:38] info     : 'system_MusicBox' Monit started
[SAST Aug 14 16:27:12] info     : monit daemon with pid [12858] killed
[SAST Aug 14 16:27:12] info     : 'system_MusicBox' Monit stopped
[SAST Aug 14 16:27:30] info     : Starting monit daemon
[SAST Aug 14 16:27:30] info     : Monit start delay set -- pause for 300s
[SAST Aug 14 16:32:30] info     : 'system_MusicBox' Monit started
[SAST Aug 14 17:03:19] info     : monit daemon with pid [13010] killed
[SAST Aug 14 17:03:19] info     : 'system_MusicBox' Monit stopped
[SAST Aug 14 17:03:38] info     : Starting monit daemon
[SAST Aug 14 17:03:38] info     : Monit start delay set -- pause for 300s
[SAST Aug 14 17:08:38] info     : 'system_MusicBox' Monit started
[SAST Aug 14 17:25:53] info     : monit daemon with pid [13199] killed
[SAST Aug 14 17:25:53] info     : 'system_MusicBox' Monit stopped
[SAST Aug 14 17:26:09] info     : Starting monit daemon
[SAST Aug 14 17:26:09] info     : Monit start delay set -- pause for 300s
[SAST Aug 14 17:33:08] info     : 'system_MusicBox' Monit started


Unable to perform transfer transaction

Describe the bug
Unable to perform transfer transaction
To reproduce the bug
Actual:

Version test: https://6d61e6e3.avail-space.pages.dev/settings

[ ] Do not update transaction status on History detail


Can't control the Plug Mini. Serial log: Failed to connect, deleted client

Hi,
I followed all the instructions and used PlatformIO to flash to ESP32. I got the Plug Mini to appear in HA. But I couldn't control (turn on/off) the plug. My ESP chip is is ESP-WROOM-32.

I set the printSerialOutputForDebugging to true to debug. When I send a command from HA (eg, turn off the plug). These logs get printed out constantly
Device:
big_monitor
Device value:
OFF
Try to connect. Try a reconnect first...
New client created
lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
Failed to connect, deleted client
Try to connect. Try a reconnect first...
New client created
Failed to connect, deleted client
Try to connect. Try a reconnect first...
New client created
Failed to connect, deleted client
Try to connect. Try a reconnect first...
New client created
Failed to connect, deleted client
Try to connect. Try a reconnect first...
New client created
lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
Failed to connect, deleted client
Try to connect. Try a reconnect first...
New client created
lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
Failed to connect, deleted client
Try to connect. Try a reconnect first...

So I'm guessing this is a problem with connecting to the plug via bluetooth? The plug firmware is v1.5

do u have the first version of the plug ( no homekit support) or the newer one that has homekit support?
mine is the 1st version and seems to work fine


How to add pitch?

I'm attempting to add pitch, but it causes the process to hang.
I've tried adding pitch to mapMosaicStream and mapImageStream functions.
map.jumpTo({     center: viewport.center,     zoom: viewport.zoom,     pitch: 50   })
Any ideas?

Hi @bbbford, this script works by splitting the requested area into tiles and exporting one at a time, then stitching those tiles together. The logic for tiling a map with a pitch is going to be quite complicated and hard to implement. I would be open to a PR on this if somebody got this working, but we’re not going to be working on it ourselves.


Windows issue responsibilities

on dev branch added
## Installation

pip install DeepForest
pip install git+git://github.com/bw4sz/keras-retinanet.git

DeepForest can alternatively be installed from source using the github repository. The python package dependencies are managed by conda. For help installing conda see: [conda quickstart](https://docs.conda.io/projects/conda/en/latest/user-guide/install/). DeepForest depends on a [fork](https://github.com/bw4sz/keras-retinanet.git) of the keras-retinanet to perform object detection. *For windows users, DeepForest requires functioning versions of keras and tensorflow before installation.*