unable to retrieve the schemes on on UI

Hi @muralibasani/Team,
I'm trying to synchronize schemas associated with topics. From the backend logs I could see that klaw is able to sync the schems from confluent platfrom. However, when i try to see the same in UI i get below error and ui keeps loading and never retrives schema.
{"timestamp":"2024-01-02T07:28:24.175+00:00","status":500,"error":"Internal Server Error","path":"/getSchemaOfTopic"}

Attached are the debug logs. Request you to suggest on the issue.
Regards,
Karthik[
schema error debug logs.txt
](url)

Hey @karthikvijayendra , thank you for raising this issue. We will take a look and come back asap with a fix

Hey @karthikvijayendra I was able to recreate this issue,
The root cause is that the tenantModel is not configured entirely. e.g. see below where the requestTopicsEnvironmentList is null
  "tenantModel" : {
    "tenantName" : "default",
    "baseSyncEnvironment" : "DEV",
    "orderOfTopicPromotionEnvsList" : [ "DEV", "TEST" ],
    "requestTopicsEnvironmentsList" : null
  }
}

This is a correctly configured one
{
  "tenantModel" : {
    "tenantName" : "default",
    "baseSyncEnvironment" : "DEV",
    "orderOfTopicPromotionEnvsList" : [ "DEV", "TEST" ],
    "requestTopicsEnvironmentsList" : [ "DEV" ],
    "baseSyncKafkaConnectCluster" : "DEV",
    "orderOfConnectorsPromotionEnvsList" : [ "DEV" ],
    "requestConnectorsEnvironmentsList" : [ "DEV" ]
  }
}

Please see the documentation on the tenant config for more details
https://www.klaw-project.io/docs/HowTo/kafka-cluster-migration/tenant-config
You can configure the above configurations under Dashboard -> Settings with a SUPERADMIN role or a user with the UPDATE_SERVERCONFIG permission.
Thanks,
Aindriú

Thanks Aiven-Open/klaw and @Murali Basani @.***> - With
the above suggestion i was able to retrieve the schemas.
Another one that i need to check is, as a workaround to confluent ACL
application issue, we thought of capturing the CDC logs from mysql db and
to run few scripts to provide the role bindings by filtering out the
required requests. Do you think is it achievable?
Also, need to change the current metadata store from H2 to mysql - Do we
need to install mysql before making to changes to properties ?
If not, could you please suggest the better way to change the
metadata store from H2 to mysql.
Regards,
Karthik
On Tue, Jan 2, 2024 at 7:55 PM Aindriú Lavelle @.***>
wrote:

Hey @karthikvijayendra https://github.com/karthikvijayendra I was able
to recreate this issue,
The root cause is that the tenantModel is not configured entirely. e.g.
see below where the requestTopicsEnvironmentList is null
"tenantModel" : {
"tenantName" : "default",
"baseSyncEnvironment" : "DEV",
"orderOfTopicPromotionEnvsList" : [ "DEV", "TEST" ],
"requestTopicsEnvironmentsList" : null
}
}
This is a correctly configured one
{
"tenantModel" : {
"tenantName" : "default",
"baseSyncEnvironment" : "DEV",
"orderOfTopicPromotionEnvsList" : [ "DEV", "TEST" ],
"requestTopicsEnvironmentsList" : [ "DEV" ],
"baseSyncKafkaConnectCluster" : "DEV",
"orderOfConnectorsPromotionEnvsList" : [ "DEV" ],
"requestConnectorsEnvironmentsList" : [ "DEV" ]
}
}
Please see the documentation on the tenant config for more details
https://www.klaw-project.io/docs/HowTo/kafka-cluster-migration/tenant-config
You can configure the above configurations under Dashboard -> Settings
with a SUPERADMIN role or a user with the UPDATE_SERVERCONFIG permission.
Thanks,
Aindriú
—
Reply to this email directly, view it on GitHub
https://github.com/Aiven-Open/klaw/issues/2156#issuecomment-1874091346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANNFNH4X46AUUMJGCSDVPVDYMQKGLAVCNFSM6AAAAABBJV3JUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZUGA4TCMZUGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@karthikvijayendra glad it worked. Thanks for mentioning.
Regarding moving to mysql or pg, just update the config ex like below (this example already exists in the repo and also for postgres)
#spring.datasource.url=jdbc:mysql://localhost:3306/kafkametadbpro?autoReconnect=true&useUnicode=true&useJDBCCompliantTimezoneShift=true&useLegacyDatetimeCode=false&serverTimezone=UTC&cachePrepStmts=true&useServerPrepStmts=true&rewriteBatchedStatements=true&verifyServerCertificate=false&useSSL=false&requireSSL=false&allowPublicKeyRetrieval=true
#spring.datasource.username=kafkauser
#spring.datasource.password=kafkauser123
#spring.datasource.driver.class=com.mysql.cj.jdbc.Driver
#spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.MySQLDialect
#spring.datasource.platform=mysql

@karthikvijayendra just to add to what @muralibasani has mentioned there is an import export data function detailed here https://www.klaw-project.io/docs/HowTo/exportimport/
But I think it would be as quick and probably a bit easier to simply setup as you did your config for Klaw.

Thanks Aindriu and Murali. Will check and get back.
At this point, when i change the db parameters to point to mysql one,
builds are failing. Will check on this. Thanks for yor support.
Regards,
Karthik
On Wed, Jan 3, 2024 at 2:23 PM Aindriú Lavelle @.***>
wrote:

@karthikvijayendra https://github.com/karthikvijayendra just to add to
what @muralibasani https://github.com/muralibasani has mentioned there
is an import export data function detailed here
https://www.klaw-project.io/docs/HowTo/exportimport/
But I think it would be as quick and probably a bit easier to simply setup
as you did your config for Klaw.
—
Reply to this email directly, view it on GitHub
https://github.com/Aiven-Open/klaw/issues/2156#issuecomment-1875026479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANNFNHZRLXIJXI557CNCUJDYMUMAPAVCNFSM6AAAAABBJV3JUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZVGAZDMNBXHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @karthikvijayendra thats interesting, you can build without changing to mysql however if you wish and use external configuration to point to your mysql server after you deploy it, you can do this through environment variables or through specifying an external application.properties, or by creating a new application.properties and using spring profiles to use the mysql version in DEV,TEST,PRD etc.
What method for deployment are you using?
However when I get a chance I will check the mysql build as I actually tested this back on 2.5.1 and at that time it was good so it would be good to know and fix any potential issue that was added in since then

Hey @karthikvijayendra did you have any success?


Is there anyway i can put placeholder / error image [not an issue]

since we are using picasso in this slider layout? i found no method to add error image and placeholder image. i saw another question open here. the guys answered take my repo. instead i want placeholder and error image it in main repo. can anyone guide me ? thanks

never mind found it.  the error() method is for error state image and empty is for placeholder image. answered my own question because if someone was wondering it might help
textSliderView
.description(name)
.image(url_maps.get(name))
.empty(R.drawable.image2)
.error(R.drawable.image4)
.setScaleType(BaseSliderView.ScaleType.FitCenterCrop)
.setOnSliderClickListener(this);


fix pad_val not used in class Pad when pad to a fixed size

Fix #2083

Thanks for the fix.

Hi @lhcalibur ！First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤

Hi @lhcalibur ！First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


remove @show in coupling loop

This if statement in the coupling loop is meant to print the date only on the first of the month, but it prints the date every day and at each timestep.
If we change the condition to have == instead of >=, the date will only get printed on the first of the month, but it will still get printed at each timestep that falls on that day (which is better, but maybe still not ideal):
        if cs.dates.date[1] == cs.dates.date1[1]
            @show(cs.dates.date[1])
        end

Actually, this error was only coming up on my branch js/timemanager, which tests out the new ClimaUtilities.jl TimeManager module. The issue was coming from not correctly using the trigger_callback function defined there (I wasn't updating date1 with its return value)


Missing symbols for => and /=

Hello,
is there a way how to solve displaying an empty box instead characters => and /=?
Thanks

You don't seem to use Utf-8 encoding and font here.

You are right - it's because of urxvt, which doesn't show some of the characters properly. Sorry for that, not issue of hvn


On Altocms.ru can not do anything - it appears a window with VK

Hello Developers,
I am a German AltoCMS users and was about to make some more on Altocms.ru, could only me log in. If I want to download the latest CMS package, a new window appears with the login in VK and otherwise you can download anything or do something else. The window is always displayed.
Could you please check it again?
Thank you in advance.

Hi,
but where are you try to download AltoCMS? Permalink for lateset oficial version is http://altocms.ru/download/altocms/last/ And it works without any login.
Or you can download it here on github: https://github.com/altocms/altocms/releases
And I want ask you: is you site on German?

Hello,
many thanks for your quick response and I have now downloaded the very last AltoCMS version of GitHub and uploaded to the server.
Yes, the content of my website at http://wpzweinull.ch/altocms AltoCMS are in German and the backend is in English.
My regular readers understand German certainly not a Russian, so I had it in English have.
But a German translation, but there is not, or at least that?
Thank you in advance.

We have no German translation. But it's open source! So, you can translate all backend to German from Russian or from English. Just get common/templates/language/en.php (or .../ru.php) and translate it. There is no need to translate all at once, you can do it step by step. Some messages will be in German, others in English, yet you do not translate

Thank you for this information. I watch times what you can do. Yes, AltoCMS is hardly used in Germany, but I'm trying to do my part and help to bring this CMS under my regular readers.
Maybe someone is interested and live in Germany also millions of German-Russian-speaking people, so that the market would be for it exists.


Fix hessian logic

Fix #9969
Maybe a regression test should be added.

Okay. I add a regression test. (not sure it works)

Ok, holomorphic need a complex inputs, I will fix it

Done.


Hiera-eyaml broken with the new version of highline and ruby 1.8

Since the release of highline 1.7.0, hiera-eyaml breaks with Notice: hiera(): Cannot load backend eyaml: no such file to load -- io/console

This is caused by JEG2/highline#129.

@ColinHebert: Highline 1.7.0 has been yanked.  Please remove it with gem uninstall highline.  Then install a compatible version of Highline with gem install highline -v '~> 1.6.19'.  Then you can install hiera-eyaml.

@TomPoulton: We should probably add something to the documentation indicating that, if you want to install hiera-eyaml on a box that only has Ruby 1.8, you'll need to manually install a compatible version of highline first.

No, i believe the right thing to do is to make your dependency
highline ~> 1.6
Or someone could argue that highline should go up a major version ... but >= 1.6.19 would still grab a 2.0 i believe

@hurrycaine That's an option, but it would also mean that hiera-eyaml can't use the newer versions of HighLine, even when on versions of Ruby that are supported.

@elyscape Using dependencies with >= actually throws warnings when you build the gem. Its considered best practice (if your dependencies use semantic versioning) to do X ~> 1.0
For this very use case here.
Once hiera-eyaml no longer wants to support 1.8 ruby then they change it to ~> 1.7

That's fair.  I'll submit a PR soon that changes the dependency.

Thank you!

If you are using puppet to install the following snippet worked for me as a workaround for the time being. Note: this was with puppet 3.7.3.
package { 'highline':
ensure   => '1.6.21',
provider => gem,
}->
package { 'hiera-eyaml':
ensure   => installed,
provider => gem,
}

This is fixed in v2.0.7, thanks to all
The dependency was changed to ~> 1.6.19 so we'll have to remember to update it once eyaml drops support for 1.8.7. Hopefully there won't be a really cool new feature in highline that we need in the meantime :)

Looks like v2.0.7 isn't on RubyGems.org yet.

It's up now.  Thanks for all your help and hard work!


feat: create efp list interaction

use-create-efp-list hook to handle creating an efp list (mint efp list nft) within the CreateOrUpdateEFPList component.
Also includes a use-list-registry hook to start fetching any data related to the list registry for a given user (if applicable).
TODO
Fix the build errors.

The changes were implemented in a different way with upgrading the whole app to another version.


Generated audio quality is low when making custom prompt

When I generate a prompt using a wav file, the generated audio is garbled.
I generated the prompt:
from utils.prompt_making import make_prompt

transcript = "I don't oppose war in all circumstances. And when I look over this crowd today I know there is no shortage of patriots or patriotism.
make_prompt(name="obama_1", audio_prompt_path="DATA/barackobamafederalplaza.wav", transcript=transcript)

I generated audio samples:
from utils.generation import SAMPLE_RATE, generate_audio, preload_models
from scipy.io.wavfile import write as write_wav
import torch
preload_models()
text_prompt = """
Hello, my name is Nose. And uh, and I like hamburger. Hahaha... But I also have other interests such as playing tactic toast.
"""
audio_array = generate_audio(text_prompt,'obama_1')
write_wav("DATA/obama_1_nose.wav",SAMPLE_RATE, audio_array)

text_prompt = "I don't oppose war in all circumstances. And when I look over this crowd today I know there is no shortage of patriots or patriotism.
audio_array = generate_audio(text_prompt,'obama_1')
write_wav("DATA/obama_1_oracle.wav",SAMPLE_RATE, audio_array)

Original audio:
https://www.dropbox.com/scl/fi/tylm6c2ffaw8qbqfqsblc/barackobamafederalplaza.wav?rlkey=wvvq0yi22t4dgynx4ehn6lcc8&dl=0
Generated audio:
https://www.dropbox.com/scl/fi/wqwgtfs4ao008kmc2e5np/obama_1_nose.wav?rlkey=yce5dt3gurpxt3sxxq4dadfr0&dl=0
https://www.dropbox.com/scl/fi/p05ns8jgsje3rt6l3wkuz/obama_1_oracle.wav?rlkey=x8ghpl4tcrb1fe001qqxt4j3r&dl=0

Prompt is too long, see FAQ for this issue

The wav clip the prompt is generated from is 12.66 seconds. I see in the FAQ that training is kept under 22 seconds. Moreover, the make_prompt function does not complain about the prompt duration (which I did when I tried a wav file that was too long).

The wav clip the prompt is generated from is 12.66 seconds. I see in the FAQ that training is kept under 22 seconds. Moreover, the make_prompt function does not complain about the prompt duration (which I did when I tried a wav file that was too long).

TOTAL duration should be less than 22s, not each single piece of them
Use 3s prompt usually delivers better result

I created a prompt using 3 seconds of audio. The generated audio has some problems.
https://www.dropbox.com/scl/fi/ewj55pxg9lpgtsf7e6ie7/barackobamafederalplaza_3s.wav?rlkey=py7e9vd88r3fxdl4m4nxok2ii&dl=0
https://www.dropbox.com/scl/fi/kcqn4zqmr8en5a9eqdo30/obama_2_nose.wav?rlkey=cgzcx79u89xbydbsmmofnhjrg&dl=0

C:\Users\profi\anaconda3\envs\valle\lib\site-packages\torch\nn\utils\weight_norm.py:30: UserWarning: torch.nn.utils.weight_norm is deprecated in favor of torch.nn.utils.parametrizations.weight_norm.
warnings.warn("torch.nn.utils.weight_norm is deprecated in favor of torch.nn.utils.parametrizations.weight_norm.")
Traceback (most recent call last):
File "c:\ai\VALL-E-X\launch-ui.py", line 629, in
main()
File "c:\ai\VALL-E-X\launch-ui.py", line 528, in main
upload_audio_prompt = gr.Audio(label='uploaded audio prompt', source='upload', interactive=True)
File "C:\Users\profi\anaconda3\envs\valle\lib\site-packages\gradio\component_meta.py", line 146, in wrapper
return fn(self, **kwargs)
TypeError: Audio.init() got an unexpected keyword argument 'source'
Help!

Your prompt audio has very strong background noise.
According to my very brief experience using this repo, only prompt audio of tts-level quality can produce audio with acceptable quality. But still the pronunciation is often far from "natural", I am not sure if it is related to the g2p quality or merely due to the small amount of training data.


Ex 003 "i" comment missing

//
// It seems we got a little carried away making everything "const u8"!
//
//     "const" values cannot change.
//     "u"     types are "unsigned" and cannot store negative values.
//     "8"     means the type is 8 bits in size.

we could add
//
// It seems we got a little carried away making everything "const u8"!
//
//     "const" values cannot change.
//     "u"     types are "unsigned" and cannot store negative values.
//=>=> "i"     types are "signed" and can store negative values.
//     "8"     means the type is 8 bits in size.

I think the i is not relevant for this description since the comment is just describing the dissection of const u8 and not providing an exhaustive list of types. Though another lesson showing more types may be a useful addition

@hdcos I'm going to go ahead and close this now, but please do comment/re-open/make a new one if you think there's a good place to add info about signed integers. Thanks!


[MNOE-125] Fix filtering on empty array

If a filter is an empty array, it'll be excluded when calling
params.to_query, so convert it to nil instead.
eg: where('id.in' => []) => {'id.in' => nil}

@alachaum @BrunoChauvet what do you think of this? Any unintended side effects?

Should we rather interpret this as an invalid query and raise a 400 error? I would at the very least log a warning when this occurs.

@BrunoChauvet good idea, see my last commit
@alachaum thoughts? Is there any case where we'd need .where('xxx' => []) ?

do we really need to raise an error? What's the active record behaviour when you do User.where(email: [])

Removed last commit and reverted to filtering on nil (same behavior as ActiveRecord)


Motion+Color flow algorithm

Hi!
Could you explain in detail the Motion+Color flow algorithm?
If there is a corresponding code that would be cool!

It's currently not included in this repo. The high-level idea is to steal the high-res texture from the input image, and paste it to the GAN result. The Motion+Color flow algorithm is a modified version of classic optical flow. It tries to estimate the geometric and color changes between the input GAN result, and the final GAN result. Once we estimate the changes, we can apply it to the input original image, and get a final image.  For more details, please refer to Sec 4.3 in the paper.


fix: panic on empty headers

If a header has no text,
that is it's just this:
##

It's parsed as a header without any child nodes.
That causes a panic with InsertBefore or InsertAfter.
So if a heading does not contain any children,
just append the anchor to it.

Current dependencies on/for this PR:

main

PR #2   👈

This comment was auto-generated by Graphite.


feat: ignore workloads as modules when generating the spec

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 54.781%

Files with Coverage Reduction
New Missed Lines
%

pkg/modules/generators/app_configurations_generator.go
1
65.15%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
8806

Relevant Lines:
16075

💛  - Coveralls


Code-sign artefact executables.

Mac: https://gist.github.com/txoof/0636835d3cc65245c6288b2374799c43
Win: https://learn.microsoft.com/en-us/previous-versions/windows/internet-explorer/ie-developer/platform-apis/ms537361(v=vs.85)?redirectedfrom=MSDN

Might also be neccesairy to add it to windows defender list for fast scanability instead of waiting for X amount of downloads
can be done through: https://www.microsoft.com/en-us/wdsi/filesubmission

Closing, as it turned out to be not really feasible for small projects.

I know the issue is closed, but it might be feasable through SignPath + GH Actions fully automated for free (since this is an open source project): https://about.signpath.io/product/open-source
https://github.com/SignPath/github-actions

Thanks, that's interesting.
However, I think the original motivation for this issue was to prevent or at least reduce the amount of security alerts users see, like Windows Defender blocking app execution. But it seems like the only thing that really helps with that is lots of people using an app.
So I wonder if this code signing would gain us anything.


Add Dependency Injection docs

Add some docs about DI integration, usage, example, etc including advanced customization.

Done here.


don't use mtls for bookinfo destination rules in test.sh

As mtls was disabled in PR #42

Any chance the feedback from concourse can be exposed in the status ?

That was the initial goal. @holgero , right? Currently, the pipeline runs with each new commit on master, meaning AFTER the merge when it is "too late".

@c0d1ngm0nk3y yes, that is the idea. We will need to explore a bit how to configure the concourse instance that we have. E.g. its build logs should become visible to others and PR authors should be enabled to re-trigger the job. Also we might need to restrict who can trigger a run (can't simply run foreign code from every pull request without some sort of check. We probably need something like that ok_to_test mechanism).


Batch processing for mediapipe pose

Is it possible to process in batch (multiple frames) for mediapipe pose estimation?
For other deep learning estimation, they can do in batch on GPU, so that a processor can interface to multiple cameras.
If I use mediapipe and in GPU desktop version, is it possible to process in batch?

mediapipe does not currently support inference batch size >1.
one idea for multiple cameras would be to run multiple graphs in parallel

@nyanmn Did you get a chance to go through above comment!

Hi @nyanmn, Assuming you are issue addressed and closing this ticket.


Stop filesets showing up in users works

Same issue as #632 but the creator is being parsed by the jbuilder file

Yeah ok, I want to get the endpoint working so Rees is unblocked but I will look at a refactor next


Jungle Logs placed, Bone Meal used, and Carrots/Potatoes broken not recognized

(Required) What is your Quests version / build number (do not say "latest")?
3.5.8
(Required) Please describe the problem and how to replicate it. Any console errors?
As the title states, I've had issues getting the plugin to recognize a few actions. I have a quest for breaking oak logs, then planting saplings, then delivering the logs to an NPC that works fine; but the exact same quest with Jungle Logs instead does not track progress when breaking the logs.
For another quest, I wanted players to plant 8 carrots and 8 potatoes, then use bone meal on them, then break the grown crops, then deliver to NPC. Planting (placing) is tracked perfectly (using crop, not item, ids), but the quest breaks at the next (Bone Meal - "Use") step; usage is not tracked at all. Upon removal of the bone meal step, the quest then breaks at the harvesting step. Harvesting (breaking) using the crop ids (Carrots, Potatoes) is not tracked for either crop.
I have several other quests that are all functioning fine, currently, on the same server. There are no errors in console during any of this.
(Optional) Can you provide an example quest from quests.yml using Pastebin.com?
Oak Log quest that works, followed by Jungle Log quest that does not - https://pastebin.com/z1aWUAaX
Had to remove errored sections of the second quest, but here it is with just plant and deliver - https://pastebin.com/X8J3J7FF

I'm going to tackle one issue at a time, starting with the Log issue.
The issue with Logs lies with durability values. Oak Logs facing upright have a value of 0, so they work just fine. However, upright Jungle Logs have a value of 3, which is why they aren't working. Side-facing Oak/Jungle logs also have different values, so neither will increase the progress count.
It seems unlikely anyone would want to specify which facing the player should break, so I'll push a commit shortly which will ignore durability for 1.13+ servers.

@DragaoNimhe Use http://ci.ac3-servers.eu/job/Quests/581/ to try the Log fix
It may also fix the crop issue. Please let me know whether that issue has been resolved, too.

The fix has solved the issues with the Jungle Log and crop harvesting, thank you!
However, bone meal still is not tracked sadly. Any potential fix for this in the future?


Make user_id the id to use for the Owner operations

Summary of Changes
For now we are passing the User Display Name to apply certain operations on the data owner.
Since we are using the same model as the User for the Data Owner, it should be the user_id for those operations.
Tests
NA
Documentation
NA
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes, including screenshots of any UI changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes all tests documented in the developer guide

Codecov Report

Merging #526 into master will decrease coverage by 5.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #526      +/-   ##
==========================================
- Coverage   81.03%   75.71%   -5.32%
==========================================
  Files         229       42     -187
  Lines        5699     1816    -3883
  Branches      676      174     -502
==========================================
- Hits         4618     1375    -3243
+ Misses       1013      391     -622
+ Partials       68       50      -18

Impacted Files
Coverage Δ

...lication/static/js/components/SearchPage/index.tsx

.../js/components/common/EditableSection/constants.ts

.../components/common/Bookmark/BookmarkIcon/index.tsx

...tic/js/components/common/EditableText/constants.ts

...eedback/FeedbackForm/RequestFeedbackForm/index.tsx

...ic/js/components/TableDetail/ColumnStats/index.tsx

...on/static/js/components/DashboardPage/constants.ts

...ic/js/components/DashboardPage/ChartList/index.tsx

...nents/TableDetail/RequestMetadataForm/constants.ts

...ic/js/components/DashboardPage/QueryList/index.tsx

... and 177 more

Continue to review full report at Codecov.

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

cc @ttannis  @Golodhros


getAllClasses() goes in a loop

Hi Nghia,
I faced with the situation when device/simulator starts consuming huge memory amount and hangs as a result. I don't know why this happens, because this is not a persistent issue and it appears from time to time (I don't see specific cause).
This is the evidence from Debug navigator:

And as per profiler looks like getAllClasses is responsible for that:

so there are more than 40 000 calls of it...
device iOS version: 14.6 (reproduced for simulators as well)
Xcode: Version 12.5.1 (12E507)
please let me know I should provide some more details
Thanks,
Andrew

Hey @andy3a thanks for your report.
Runtime.getAllClasses() should only be executed once when the app is starting. If it executes multiple times (as your screenshot), I suppose there is a bug somewhere.
I will replace it with a better version and send you a beta this week 👍

Just wondering:

Are you using any 3rd-party networking library? such as AFNetworking, or Alamofire? If yes, what version?

I would like to reproduce it in my machine 💻

yes, I'm using Alamofire 5.4.3

Hey @andy3a if you don't mind, please check out this commit and share with me if it works for you 👍
Changelog

Use a different version to get all classes that conforms NSURLConnectionDelegate in runtime

SMP

Check out this commit: 70425e4a0895ec483dfdb5b99cfba91bec6ac935

Cocoapod
pod 'atlantis-proxyman', :git => 'https://github.com/ProxymanApp/atlantis.git', :commit => '70425e4a0895ec483dfdb5b99cfba91bec6ac935'

thanks for you effort @NghiaTranUIT
successfully deployed the commit.
I'll need to use it for couple days to see if the issue can be reproduced unless you know how to reproduce the situation when the issue appeared in previous version
cheers

hi @NghiaTranUIT,
I have reproduced the issue:

It's odd that this array of AnyClass doesn't release when it exits the function 🤔
@andy3a Please try this commit 29088bc9b752498cdb74051c962855f2de77e52a, which I disable the Method Swizzling on NSURLConnection.

Hi @NghiaTranUIT,
I'm sorry but the issue has been reproduced again.

... for the latest fix provided

I could provide some details form Profiler if needed. Feel free to request the data you may need for investigation.
p.s. could it be like I do smth wrong? itbh I don't see where I could do anything wrong, but maybe you have an idea...
Andy


Unable to use Hashicorp ClusterSecretStore

Describe the bug
The ClusterSecretStore cannot be used:

Cannot read the token if put into external-secrets namespace (invalid vault credentials: configured Vault token contains non-printable characters and cannot be used)
Cannot find the token secret from an externalSecret if without namespace

To Reproduce
Steps to reproduce the behavior:
apiVersion: v1
kind: Secret
metadata:
  name: vault-token
  namespace: external-secrets
data:
  token: <base64RootToken>
---
apiVersion: external-secrets.io/v1beta1
kind: ClusterSecretStore
metadata:
  name: vault-backend
  namespace: external-secrets
spec:
  provider:
    vault:
      server: "https://vault.domain.com"
      path: "kv-secrets"
      # Version is the Vault KV secret engine version.
      # This can be either "v1" or "v2", defaults to "v2"
      version: "v2"
      auth:
        # points to a secret that contains a vault token
        # https://www.vaultproject.io/docs/auth/token
        tokenSecretRef:
          name: "vault-token"
          key: "token"

Expected behavior
The clusterSecretStore should be able to use the token

@aacater is correct. If you don't specify a namespace, you need to create one token per namespace where you have ExternalSecrets targetting your ClusterSecretStore.
This is by design 😄


Pluggable geocoder in feature_list.js

We are using Pelias as a geocoder over openstreetmap data and would like to extract the functionality in feature_list.js to be able to use a pluggable geocoder instead of directly calling nominatim.
I'm thinking that it could be a service that can be setup in the context in the same way as the nominatim service.
A simple API like the one below could be enough
function geocode(searchTerm, callback) {}

The simplest way to do introduce this would be to:

[ ] Introduce new service geocode with the api above (model the response after Nominatim)
[ ] Import service in feature_list.js
[ ] Change the eventlistener in feature_list.js to call imported service

I know that nominatim is more or less the standard but it would be nice to be pluggable in this regard and I don't think it would be intrusive to the users in any way. The default service would of course be Nominatim.

Good suggestion, I agree...

Could the current nominatim service be used if we add a function geocode?
In feature_list.js
 import { services } from '../services/index';
 ...
 function geocode() {
            services.nominatim.geocode(search.property('value'), function(resp) {
                geocodeResults = resp;
                drawList();
            });
        }

And in nominatim.js
var endpoint = 'https://nominatim.openstreetmap.org/',
   nominatimCache;

   ...
   geocode: function(value, callback) {
        var searchVal = encodeURIComponent(value);
        d3.json(endpoint + 'search/' + searchVal + '?limit=10&format=json', function(err, resp) {
            callback(resp || []);
        });
    }

Btw, what's the correct/best way of making a service pluggable for the developer without modifying the library code?

Everything in the services module is designed to be swappable at runtime.  See API.md#services
So after we fix the issues in this ticket, it should ideally be as easy as a line like this before your page initializes the iD context object.
iD.services.geocoder = replacementService;

Great! And I just want to thank you for the great work you and everyone else that has contributed to iD has done!


[Offline Login] App force closes on offline login

App version 0.1.7- preview
Phone: Grandblue G6
When trying to log in while offline - the app is giving an error in configs and then force closing after I enter the pin.
https://user-images.githubusercontent.com/58034047/207848949-81615310-6a50-4bbd-9c0a-cb0ec9501a53.mp4
Acceptance criteria

[ ] User should be able to log in while offline.
[ ] While offline and for subsequent logins - the user should be redirected to the pin login page (not the username/password page).

I tested this from the emulator and a Samsung device and I believe it's been resolved.
The app doesn't crash on offline login using a set PIN

Can you add some additional testing around this?

Assigning @allan-on to this. You can tag the open PR to this ticket to close it too

Closing this.  Since the attached PR fixes it. https://github.com/opensrp/fhircore/pull/1898


Error when running project after installing opencv4nodejs

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

Please describe your issue:
When adding opencv4nodejs to my fresh forge installation (webpack template), I am getting following error on start:
✔ Checking your system
✔ Locating Application
✔ Preparing native dependencies: 1 / 1
✔ Compiling Main Process Code
✔ Launch Dev Servers
✔ Compiling Preload Scripts
✔ Launching Application

An unhandled rejection has occurred inside Forge:
TypeError: Cannot set property 'native_modules/opencv/build/lib/libopencv_plot.3.4.dylib' of undefined
    at Promise.all.files.map (/Users/steph/Documents/workspace/electron-opencv-test/node_modules/@marshallofsound/webpack-asset-relocator-loader/dist/index.js:14689:98)

Electron Forge was terminated. Location:
{}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `electron-forge start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

What command line arguments are you passing?
npm start
What does your config.forge data in package.json look like?
"config": {
    "forge": {
      "packagerConfig": {},
      "makers": [
        {
          "name": "@electron-forge/maker-squirrel",
          "config": {
            "name": "electron_opencv_test"
          }
        },
        {
          "name": "@electron-forge/maker-zip",
          "platforms": [
            "darwin"
          ]
        },
        {
          "name": "@electron-forge/maker-deb",
          "config": {}
        },
        {
          "name": "@electron-forge/maker-rpm",
          "config": {}
        }
      ],
      "plugins": [
        [
          "@electron-forge/plugin-webpack",
          {
            "mainConfig": "./webpack.main.config.js",
            "renderer": {
              "config": "./webpack.renderer.config.js",
              "entryPoints": [
                {
                  "html": "./src/index.html",
                  "js": "./src/renderer.js",
                  "name": "main_window"
                }
              ]
            }
          }
        ]
      ]
    }
  },

Here is the repo:
https://github.com/stoefln/electron-opencv-test

We took a look at the code in opencv4nodejs and it seems that there's some weird code in there trying to detect Electron + Webpack:
https://github.com/justadudewhohacks/opencv4nodejs/blob/5d02be9b541c29b7e78061e8e5d82772f2f422cf/lib/opencv4nodejs.js#L4-L11
If that special case gets removed, it'll probably "just work".
Closing since it's very unlikely to be specifically an Electron Forge bug.

@malept I have created a small sample app showing the problem.
https://github.com/gallagherrchris/opencv4nodejs-electron-test
I added in some code to force the logic you highlighted above (should return true and force the first code path).  I am still having the issue with the module not being found and it hasn't "just work"™


Support multiple resource directories in aapt

This is a fix for Issue #45: Support multiple resource directories in aapt
Rather than use the pluginConvention.resDir field, it allows you to set multiple resource directories into pluginConvention.resDirs, all of which are then passed to the aapt ant task.
I haven't used Groovy or Gradle much before, so if there's a preferred way of doing any of this, please let me know and I'll fix.

Thanks for this.  Let me find time to review and then merge if possible.

Hi @shawnlauzon, would it be possible to add an integration test for this specific scenario?

Yes, I'll add an integration test, but won't have time for a few days.

I'm having a problem running the tests. Here's the command I'm trying to run:
groovy -cp .:$GRADLE_HOME/lib/gradle-core-1.0-milestone-8a.jar:$GRADLE_HOME/lib/gradle-native-1.0-milestone-8a.jar com/jvoegele/gradle/android/HelloProjectTest.groovy

and the result is:
org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
/Users/Mac/Codez/git/gradle-android-plugin/src/integTest/groovy/com/jvoegele/gradle/android/support/TestProject.groovy: 21: unable to resolve class org.gradle.os.OperatingSystem
 @ line 21, column 1.
 import org.gradle.os.OperatingSystem
 ^

1 error

@shawnlauzon this was fixed in #50 - you should be able to merge master into this branch to get the fix.


Install script tries to use root if install destination does not yet exist even in standalone mode.

https://github.com/cdr/code-server/blob/d8c3ba6a179d185aa42ceb6761dd8a2be339d68f/install.sh#L378
The '-w' test fails if  "$STANDALONE_INSTALL_PREFIX" doesn't exist. Since this variable defaults to $HOME/.local the user may not have created that hidden directory before attempting to install.

Ah, nice catch! Thanks for pointing this out and raising an issue.


Fix instructions to override ApplicationInsights endpoints

Fix ASP.NET Core and add Azure Functions instructions.

@lmolkova : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@lmolkova : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@lmolkova : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Exclude some common directories

Does not yet contain a CLI argument to extend/override exclusions.
See #27.

Does not yet contain a CLI argument to extend/override exclusions.

Should it be a CLI argument or an environment variable?

As this is meant to extract code from a WordPress plugin or theme, I wonder whether it wouldn't be preferable to use a WordPress filter to make the exclusions customizable from within a plugin/theme...

Here's some prior art: https://github.com/wp-cli/find-command#using
[--include_ignored_paths=<paths>]
	Include additional ignored paths as CSV (e.g. '/sys-backup/,/temp/').

I'm sticking with an extra argument for now.
Wondering though if I should rename exclude to ignore 🤷♂

Merged now. Good work, @swissspidy!


Uable to generate client for github public graphql schema

Encountered two issues while trying to generate client
https://docs.github.com/public/ghes-2.22/schema.docs-enterprise.graphql

The schema uses keywords like package, public which caused it to error -
Resoled it by renaming them to resolve it like packageName, publicName
Encountered Out of memory -
Resolved it using org.gradle.jvmargs=-Xmx2048m -XX:MaxPermSize=512m -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8
After resolving the above issues, the generateJava tasks runs for ever with no output or errors, it was struck for more than 5 hrs with no output, had to terminate the process.
schema.docs-enterprise.graphql.zip

Please try limiting the queries you want to generate code for and also setting the maxProjectionDepth setting. This should prevent OOM issues when generating classes.
https://netflix.github.io/dgs/generating-code-from-schema/#limiting-generated-code-for-client-api


Question only: docker-splitted enviroments attackable?

Just a technical question, our servers are updated :)
We use a splitted enviroment where nginx runs in one container, php-fpm in another. I tried to reproduce the issue with your exploit, but it seems to hang at

2019/11/05 21:30:58 The target is probably vulnerable. Possible QSLs: [1735 1740 1745]

while the project runs localy (8mins now). As i inspect your docker-file, you seem to have your enviroment combined in the same container. If the split we did protected us from the attack anyway, this would be a nice side-info :)
Also, if this is really the case, it would be may a idea to add it to the requirements?
Thanks for developing this poc!

No, splitting nginx and php-fpm in different containers doesn't protect you in any way.
It is strange that QSLs are found even though you have updated your servers. I suggest you to double check that.

It is strange that QSLs are found even though you have updated your servers. I suggest you to double check that.

i am sorry as i was unclear. i retest the attack by downgrading the docker-images to our previous versions and run them only local.
as i tested it on our productive servers, it was not able to do any part of the attack.

No, splitting nginx and php-fpm in different containers doesn't protect you in any way.

ok, interessting then, it still hangs there. i guess your exploit would be able to handle that "remote-connection" (to the other container).


Suggestion (Missions&Research)

If you could make the bot be able to do delete software missions or ddos missions that would be amazing and also if it could have like if you was on the fbi's most wanted and you was over a certain amount of $$ for bounty it could then change it's IP through the ISP?
and also a complete all missions button for when I want to go ahead and make a lot of money on a alt account and then transfer it over to my main.
If you could also have a research bot that can automatically research a software that you want and you could set it to use a certain amount of $$ max and if it runs out of money in wallet it would either do missions or sell the amount of bitcoin that you need to research that software

Yeah. I've been being very busy lately, but I'll try to make the delete software missions sequence.

Any update?


python312Packages.netifaces2: init at 0.0.22

Closes #353740
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4819

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354736

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.netifaces2
python311Packages.netifaces2.dist
python312Packages.netifaces2
python312Packages.netifaces2.dist

aarch64-linux

:white_check_mark: 4 packages built:

python311Packages.netifaces2
python311Packages.netifaces2.dist
python312Packages.netifaces2
python312Packages.netifaces2.dist

aarch64-darwin

:white_check_mark: 4 packages built:

python311Packages.netifaces2
python311Packages.netifaces2.dist
python312Packages.netifaces2
python312Packages.netifaces2.dist


NTR: polybrominated diphenyl ether

https://en.wikipedia.org/wiki/Polybrominated_diphenyl_ethers
syn/abbrev: PBDE
is_a: CHEBI:37141 ! organobromine compound

For role, consider environmental contaminant

Hi Chris,
This term has been added to CHEBI with ID#134094.
Kind regards,
Steve


Refactor rendering in MiniStats

removed support for multi draw calls, which is no longer used
rendering using a mesh instead of setting up everything on the device manually

Is that something that worked with the previous version of MiniStats? I have not touched anything related to this.

Nope, I don't think it ever worked. Maybe some pre-AppBase did. Not sure.


decorators.py: Improve yield_once speed

Changes:

yield_once uses a set instead of a list to lookup already seen items
change the docstring to clarify that it does not wrap iterators but methods that return iterators
a doctest verifies that it works.

In order to run the doctests, type
python -m doctest coala_decorators/decorators.py

I wasn't sure whether there is a CI unit test suite yet, so I just created a simple doctest. This might be related to issue #9. Please let me know if anything can be improved.

As you have three changes, I think you can make three commits out of them :)
nice btw that you figured out a manual way for doing doctests until we set up our CI^^

OK, I see how changing the docstring warrants its own commit, but surely, adding a doctest still belongs to this commit, right?

I think what @Makman2 meant by the 3 changes was:

Simplify if-else by removing redundant pass
Use set instead of list for better performance
Improve documentation of yield_once (doctests are also a method to improve documentation :))

These are all separate and need not be in the same commit

Ah! Gotcha.

actually I meant

doctest
documentation text changes
code changes

but @AbdealiJK 's version is also okay, this makes sense too :+1:

@justuswilhelm small changes needed, removing pending review label

I hope I've captured all your requirements :) Thanks for waiting, I had a long week.
As always, run
python -m doctest coala_decorators/decorators.py

to perform tests.

@Adrianzatreanu can you review?

Do we want to move this repo to GitLab and see how CI setup is going on GitLab? I think it'd make things easier

Sounds like a good idea. If it turns out to be a lot easier, would it make sense to start moving other repos over to GitLab as well?

ack https://github.com/coala-analyzer/coala-decorators/pull/15/commits/0ecb35eb9ff276aeb127cff5a6d39c6970f52171 https://github.com/coala-analyzer/coala-decorators/pull/15/commits/19b6ecaf07756eb22de6d22d3795ba80b992f00e https://github.com/coala-analyzer/coala-decorators/pull/15/commits/eeaee97fefb990841c3b18c5a1e6f8232160a115 https://github.com/coala-analyzer/coala-decorators/pull/15/commits/b53e93f660c8cb752eed2bd27553e87de0598ac7 https://github.com/coala-analyzer/coala-decorators/pull/15/commits/e914ec5de7193fb01067f25dc0f4ba6e2ffec3cb

@sils1297 Thanks!

@justuswilhelm nice work :)

@justuswilhelm rebase?

Rebase? Is the branch not up to date anymore?

it's not, adrian merged a commit adding the VERSION file to package_data

@sils1297 done. I wish I was able to see that information on GH as well. Only thing I get to see is that

You're not authorized to merge this pull request

@justuswilhelm now do a git log --oneline master.. and paste it together with a reack into the github comment. reack must be at start or end of the comment

reack d81754b decorators.py: improve yield_once wrapped method
12ffb0b decorators.py: Remove else branch in yield_once
ba356c5 decorators.py: Use set in yield_once
2e23c40 decorators.py: Fix documentation
a08a8fd decorators.py: Add yield_once doctest

@rultor merge

great work, @justuswilhelm

😅 oh the wonders of Git

@rultor merge

@adrianzatreanu OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@adrianzatreanu Done! FYI, the full log is here (took me 1min)


Implement the line break system

I want the most classical system like:
1 -> 3 -> 5 -> 7.......
       2 -> 4 -> 6 -> 8.......
The number means the line number of your lyrics,
Lyrics will take turn to switch on first position and second position.
It's different from most English speaker familiar with, which is lines of lyrics just simply scrolling down.

Finish on f4e4fa4b8a3bd5e9ca01c84e496e2dd1048f4e7a


haskell: add impure cabal-plan translator based on hackage translator

The translator proposed here is basically an "impure wrapper" for the cabal-plan translator. It's a more general version of the hackage translator--all that's changed is that the source is pulled from the source option, instead of being downloaded from hackage using name and version.
The translator copies the source into a temporary directory, runs cabal freeze, runs the cabal-plan translator, and then patches in the original source (just like the hackage translator).

I'm not familiar with how the tests work, but I've added an example identical to the cabal-plan example, with only the translator changed.

I'd edited the wrong projects.toml.

(I'm sorry for the small mistakes.. I'm not able to run the tests locally, since the examples assume an x86_64-linux system)

Ideally I think the example should be a project that doesn't keep lockfiles in its repo.

I opened an issue for the tests not being cross platform. Should be an easy fix: https://github.com/nix-community/dream2nix/issues/448

I poked around a bit and the problem was that the name declared in the project.toml was different from the detected package name. Therefore the generate dream-lock.json was ignored. fixed via https://github.com/nix-community/dream2nix/pull/443/commits/073357486cc374df60cfb8123634a92b93c70a21


I am requesting the corresponding tutorial for the code. I know it will take some time, but please provide some help to so many people who are interested in your research. Thank you

Hello, I am a student. I have been researching the code you provided on Github for a long time, but in the end, I couldn't find a way to run it on my HoloLens2, which has caused my current research to stagnate. At the same time, I also see that others are interested in your research but have not found a way to use code. Please provide some help or guidance, and I would be very grateful!!

Hello @tuboshu12, thanks for a suggestion, I've added an instruction how to run a project in a readme, also made some cleanup in code. In the following days I'll also commit another version of a project, which uses a Sentis library. It makes possible to use a YOLOv5nu, YOLOv6 and YOLOX models, which, according to our tests, perform faster.
Could you try to follow tutorial to run a project and give a feedback?

Of course, it's my honor to work with you. Please provide the versions of software such as Unity and VS. Could you please take some time to provide detailed project operation instructions as much as possible. Because I am a beginner in Unity, thank you very much for your help! I will conduct testing on HoloLens2 glasses based on the project operation instructions you provided and provide timely feedback. In the future, I also want to continue to follow up and make my own improvement plans for the YOLO algorithm mentioned above, and provide them to this project. Thank you again for your reply!

Hello @tuboshu12, thanks for a suggestion, I've added an instruction how to run a project in a readme, also made some cleanup in code. In the following days I'll also commit another version of a project, which uses a Sentis library. It makes possible to use a YOLOv5nu, YOLOv6 and YOLOX models, which, according to our tests, perform faster. Could you try to follow tutorial to run a project and give a feedback?

Hello, I had some problems when running the project. I sent you an email. Could you please help me? Thanks!

Hello @tuboshu12, thanks for a suggestion, I've added an instruction how to run a project in a readme, also made some cleanup in code. In the following days I'll also commit another version of a project, which uses a Sentis library. It makes possible to use a YOLOv5nu, YOLOv6 and YOLOX models, which, according to our tests, perform faster. Could you try to follow tutorial to run a project and give a feedback?

Hello Professor, I have raised some questions about the types and confidence levels of displayed objects. Could you please reply to my email and guide me? Thank you! I've been looking at the code for a long time and still don't understand how to display these two important values.


Xbox one controller not detected

Neither my Xbox one wired controller or wireless controller are being detected by the mod but they are detected by steam. I had it working just fine the other day but I updated forge and it will no longer work. I tried going back into a previous forge version but it still didn’t work

This is basically what makes the Controllable mod a SCAM.

I've written a bunch of guides on different methods of connecting your controller and troubleshooting issues. I believe it will help resolve your issue since connection issue are based on a lot of factors. While Xbox One could work fine on Windows, it may not work the same on macOS or Linux and may require extra steps.
https://mrcrayfish.gitbook.io/controllable-documentation/guides/how-to-connect-a-gamepad


[bug] "npm run build" or "npm run serve" didn't work as expected

By following the contribution guide, after drafting the PR for Issue https://github.com/open-telemetry/opentelemetry.io/issues/3488, I was trying to build the site for the final review but the process was broken.
npm version
{
  npm: '10.1.0',
  node: '20.8.1',
...
}

npm install
npm run build

I got this:
> prebuild
> npm run _prebuild

> _prebuild
> npm run seq -- get:submodule cp:spec

> seq
> bash -c 'for cmd in "$@"; do npm run $cmd || exit 1; done' - get:submodule cp:spec

> get:submodule
> npm run _get:${GET:-submodule}

> _get:submodule
> set -x && git submodule update --init ${DEPTH:- --depth 1}

+ git submodule update --init --depth 1
remote: Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
fatal: transport 'file' not allowed
fatal: Fetched in submodule path 'content-modules/community', but it did not contain 0cb6dcadb53f1f7ffe5ef819ceef02ac9e6886c8. Direct fetching of that commit failed.

So I manually updated the submodules with:
git submodule update --recursive --init

This didn't work:
fatal: transport 'file' not allowed
fatal: Fetched in submodule path 'content-modules/community', but it did not contain 0cb6dcadb53f1f7ffe5ef819ceef02ac9e6886c8. Direct fetching of that commit failed.

So tried something else:
git submodule update --force --recursive --init --remote

And all submodules had been updated, expected 'themes/docsy':
Submodule path 'content-modules/community': checked out '8d480e00ab3e07d011f35b10fc3b76eb1a3d86ff'
Submodule path 'content-modules/opamp-spec': checked out 'b2037c3456caf173f2c1c6cd1d9500fe9c1c545d'
Submodule path 'content-modules/opentelemetry-proto': checked out 'f674adf15f7e779c6b83ef0dce240fa286b860d8'
Submodule path 'content-modules/opentelemetry-specification': checked out 'd855249b210ecd832889fc4ce91a4444f82e252a'
Submodule path 'content-modules/semantic-conventions': checked out '65a3cc743f71ee202242ca4e984c5f8bb56e187d'
fatal: Unable to find refs/remotes/origin/opentelemetry.io revision in submodule path 'themes/docsy'

Obviously, the submodule of 'themes/docsy' had issues.
And then tried again with _get:submodule ignored:
npm run serve --ignore-scripts "_get:submodule"

I got this:
> serve
> npm run serve:hugo _get:submodule

> serve:hugo
> npm run _serve:hugo _get:submodule

> _serve:hugo
> hugo server -DFE --minify _get:submodule

Watching for changes in /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/{archetypes,assets,content-modules,content,data,iconography,layouts,package.json,static}
Watching for config changes in /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/hugo.yaml
Start building sites …
hugo v0.120.3-a4892a07b41b7b3f1f143140ee4ec0a9a5cf3970+extended darwin/arm64 BuildDate=2023-11-01T17:57:00Z VendorInfo=gohugoio

Built in 166 ms
Error: error building site: assemble: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/content/en/_index.md:14:1": failed to extract shortcode: template for shortcode "blocks/cover" not found

So is this a known issue or did I do something wrongly here?

So the focus will be the submodule themes/docsy and more experiments were conducted:
$ git submodule update --init --recursive --force themes/docsy
Submodule path 'themes/docsy': checked out 'fd669b752b7f83fb2eb57cacd2c8f334f86ca7d2'

$ ls themes/docsy
CHANGELOG.md        LICENSE             dependencies        go.sum              images              package.json        theme.toml
CONTRIBUTING.md     README.md           docker-compose.yaml hugo.yaml           layouts             postcss.config.js   userguide
Dockerfile          assets              go.mod              i18n                netlify.toml        static

$ npm run serve --ignore-scripts "_get:submodule"

Then I was seeing more errors:
> serve
> npm run serve:hugo _get:submodule

> serve:hugo
> npm run _serve:hugo _get:submodule

> _serve:hugo
> hugo server -DFE --minify _get:submodule

Watching for changes in /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/{archetypes,assets,content-modules,content,data,iconography,layouts,package.json,static,themes}
Watching for config changes in /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/hugo.yaml, /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/themes/docsy/hugo.yaml
Start building sites …
hugo v0.120.4-f11bca5fec2ebb3a02727fb2a5cfb08da96fd9df+extended darwin/arm64 BuildDate=2023-11-08T11:18:07Z VendorInfo=gohugoio

ERROR spec_status: Can't find page at '/docs/specs/otel/trace/api'.
ERROR spec_status shortcode error: page '/docs/specs/otel/trace/api' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/trace/api': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/trace/sdk'.
ERROR spec_status shortcode error: page '/docs/specs/otel/trace/sdk' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/trace/sdk': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otlp'.
ERROR spec_status shortcode error: page '/docs/specs/otlp' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otlp': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/metrics/api'.
ERROR spec_status shortcode error: page '/docs/specs/otel/metrics/api' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/metrics/api': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/metrics/sdk'.
ERROR spec_status shortcode error: page '/docs/specs/otel/metrics/sdk' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/metrics/sdk': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/baggage/api'.
ERROR spec_status shortcode error: page '/docs/specs/otel/baggage/api' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/baggage/api': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/logs/bridge-api'.
ERROR spec_status shortcode error: page '/docs/specs/otel/logs/bridge-api' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/logs/bridge-api': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/logs/sdk'.
ERROR spec_status shortcode error: page '/docs/specs/otel/logs/sdk' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/logs/sdk': matched line is ''
ERROR spec_status: Can't find page at '/docs/specs/otel/logs/event-api'.
ERROR spec_status shortcode error: page '/docs/specs/otel/logs/event-api' does not contain a line matching 'Status'.
ERROR spec_status shortcode error: 'Status' value did not match one of '(?i:experimental|stable|deprecated|removed|mixed|feature.freeze|alpha|beta)' in file '/docs/specs/otel/logs/event-api': matched line is ''
ERROR render of "page" failed: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/layouts/_default/baseof.html:19:7": execute of template failed: template: _default/search.html:19:7: executing "_default/search.html" at <partialCached "scripts.html" .>: error calling partialCached: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
ERROR render of "page" failed: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/themes/docsy/layouts/blog/baseof.html:33:7": execute of template failed: template: blog/single.html:33:7: executing "blog/single.html" at <partial "scripts.html" .>: error calling partial: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
ERROR render: failed to render pages: render of "page" failed: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/layouts/_default/baseof.html:19:7": execute of template failed: template: _default/single.html:19:7: executing "_default/single.html" at <partialCached "scripts.html" .>: error calling partialCached: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
Built in 1106 ms
Error: error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/themes/docsy/assets/scss/main.scss:1:1": File to import not found or unreadable: ../vendor/bootstrap/scss/functions.

Hi. I just did a clean build and it works fine. Let's see if a clean build works for you too. Try the following from a new terminal:

cd ~/tmp or any other temporary dir that you have access to
git clone https://github.com/open-telemetry/opentelemetry.io.git
cd opentelemetry.io
nvm install
npm install -- capture the output and share it here.

Interestingly, it worked in a clean clone.
I didn't use nvm so I thought that might be needed but it still didn't work in my existing clone.
$ git remote -vv
origin	git@github.com:brightzheng100/opentelemetry.io.git (fetch)
origin	git@github.com:brightzheng100/opentelemetry.io.git (push)
upstream	https://github.com/open-telemetry/opentelemetry.io.git (fetch)
upstream	https://github.com/open-telemetry/opentelemetry.io.git (push)

$ git fetch upstream

$ git merge upstream/main
Already up to date.

$ nvm install
Found '/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io/.nvmrc' with version <lts/*>
v20.9.0 is already installed.
Now using node v20.9.0 (npm v10.1.0)

$ npm install

> prepare
> npm run seq -- get:submodule _prepare:docsy

> seq
> bash -c 'for cmd in "$@"; do npm run $cmd || exit 1; done' - get:submodule _prepare:docsy

> get:submodule
> npm run _get:${GET:-submodule}

> _get:submodule
> set -x && git submodule update --init ${DEPTH:- --depth 1}

+ git submodule update --init --depth 1
remote: Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
fatal: transport 'file' not allowed
fatal: Fetched in submodule path 'content-modules/community', but it did not contain 0cb6dcadb53f1f7ffe5ef819ceef02ac9e6886c8.  A complete log of this run can be found in: /Users/brightzheng/.npm/_logs/2023-11-09T08_48_08_191Z-debug-0.log

I tried deinit the submodules but it didn't help:
$ git submodule deinit -f --all
Cleared directory 'content-modules/community'
Submodule 'content-modules/community' (https://github.com/open-telemetry/community) unregistered for path 'content-modules/community'
Cleared directory 'content-modules/opamp-spec'
Submodule 'content-modules/opamp-spec' (https://github.com/open-telemetry/opamp-spec) unregistered for path 'content-modules/opamp-spec'
Cleared directory 'content-modules/opentelemetry-proto'
Submodule 'content-modules/opentelemetry-proto' (https://github.com/open-telemetry/opentelemetry-proto) unregistered for path 'content-modules/opentelemetry-proto'
Cleared directory 'content-modules/opentelemetry-specification'
Submodule 'content-modules/opentelemetry-specification' (https://github.com/open-telemetry/opentelemetry-specification.git) unregistered for path 'content-modules/opentelemetry-specification'
Cleared directory 'content-modules/semantic-conventions'
Submodule 'content-modules/semantic-conventions' (https://github.com/open-telemetry/semantic-conventions) unregistered for path 'content-modules/semantic-conventions'
Cleared directory 'themes/docsy'
Submodule 'themes/docsy' (https://github.com/cncf/docsy.git) unregistered for path 'themes/docsy'

$ git status
On branch update-building-a-trace-receiver
Your branch is ahead of 'upstream/main' by 2 commits.
  (use "git push" to publish your local commits)

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   content/en/docs/collector/trace-receiver.md

$ npm run serve

> serve
> npm run serve:hugo

> preserve:hugo
> npm run _prebuild

> _prebuild
> npm run seq -- get:submodule cp:spec

> seq
> bash -c 'for cmd in "$@"; do npm run $cmd || exit 1; done' - get:submodule cp:spec

> get:submodule
> npm run _get:${GET:-submodule}

> _get:submodule
> set -x && git submodule update --init ${DEPTH:- --depth 1}

+ git submodule update --init --depth 1
Submodule 'content-modules/community' (https://github.com/open-telemetry/community) registered for path 'content-modules/community'
Submodule 'content-modules/opamp-spec' (https://github.com/open-telemetry/opamp-spec) registered for path 'content-modules/opamp-spec'
Submodule 'content-modules/opentelemetry-proto' (https://github.com/open-telemetry/opentelemetry-proto) registered for path 'content-modules/opentelemetry-proto'
Submodule 'content-modules/opentelemetry-specification' (https://github.com/open-telemetry/opentelemetry-specification.git) registered for path 'content-modules/opentelemetry-specification'
Submodule 'content-modules/semantic-conventions' (https://github.com/open-telemetry/semantic-conventions) registered for path 'content-modules/semantic-conventions'
Submodule 'themes/docsy' (https://github.com/cncf/docsy.git) registered for path 'themes/docsy'
remote: Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
fatal: transport 'file' not allowed
fatal: Fetched in submodule path 'content-modules/community', but it did not contain 0cb6dcadb53f1f7ffe5ef819ceef02ac9e6886c8. Direct fetching of that commit failed.

I'm a bit confused now.
It can be different issues but this is reproducible on my side so please advise what is wrong:
$ git clone git@github.com:brightzheng100/opentelemetry.io.git opentelemetry.io-mine
$ cd opentelemetry.io-mine

$ git remote add upstream https://github.com/open-telemetry/opentelemetry.io.git
$ git fetch upstream
$ git merge upstream/main

$ npm run serve

Output:
> serve
> npm run serve:hugo

> preserve:hugo
> npm run _prebuild

> _prebuild
> npm run seq -- get:submodule cp:spec

> seq
> bash -c 'for cmd in "$@"; do npm run $cmd || exit 1; done' - get:submodule cp:spec

> get:submodule
> npm run _get:${GET:-submodule}

> _get:submodule
> set -x && git submodule update --init ${DEPTH:- --depth 1}

+ git submodule update --init --depth 1
Submodule path 'content-modules/opentelemetry-specification': checked out 'd855249b210ecd832889fc4ce91a4444f82e252a'
remote: Enumerating objects: 220, done.
remote: Counting objects: 100% (220/220), done.
remote: Compressing objects: 100% (100/100), done.
remote: Total 113 (delta 85), reused 19 (delta 9), pack-reused 0
Receiving objects: 100% (113/113), 43.69 KiB | 6.24 MiB/s, done.
Resolving deltas: 100% (85/85), completed with 79 local objects.
From https://github.com/open-telemetry/semantic-conventions
 * [new tag]         v1.22.0    -> v1.22.0
remote: Enumerating objects: 31, done.
remote: Counting objects: 100% (31/31), done.
remote: Compressing objects: 100% (9/9), done.
remote: Total 11 (delta 10), reused 3 (delta 2), pack-reused 0
Unpacking objects: 100% (11/11), 1.32 KiB | 51.00 KiB/s, done.
From https://github.com/open-telemetry/semantic-conventions
 * branch            da1dbb567f0f0c866dde79a7564a58ab8019b1cf -> FETCH_HEAD
Submodule path 'content-modules/semantic-conventions': checked out 'da1dbb567f0f0c866dde79a7564a58ab8019b1cf'
remote: Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
remote: Enumerating objects: 7, done.
remote: Counting objects: 100% (7/7), done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 4 (delta 3), reused 1 (delta 0), pack-reused 0
Unpacking objects: 100% (4/4), 977 bytes | 488.00 KiB/s, done.
From https://github.com/cncf/docsy
 * branch            fd669b752b7f83fb2eb57cacd2c8f334f86ca7d2 -> FETCH_HEAD
Submodule path 'themes/docsy': checked out 'fd669b752b7f83fb2eb57cacd2c8f334f86ca7d2'

> postget:submodule
> git submodule

 0cb6dcadb53f1f7ffe5ef819ceef02ac9e6886c8 content-modules/community (0cb6dca)
 7718250cb5d3982a366c06fb151d67b57048c9c7 content-modules/opamp-spec (7718250)
 c4dfbc51f3cd4089778555a2ac5d9bc093ed2956 content-modules/opentelemetry-proto (c4dfbc5)
 d855249b210ecd832889fc4ce91a4444f82e252a content-modules/opentelemetry-specification (v1.27.0)
 da1dbb567f0f0c866dde79a7564a58ab8019b1cf content-modules/semantic-conventions (da1dbb5)
 fd669b752b7f83fb2eb57cacd2c8f334f86ca7d2 themes/docsy (fd669b7)

> cp:spec
> ./scripts/content-modules/cp-pages.sh

OTEL SPEC pages: copied and processed
OTLP SPEC pages: copied and processed
OTLP SPEC protos copied and processed
COMMUNITY pages: copied and processed
SEM CONV  pages: copied and processed
OpAMP SPEC page: copied and processed

> serve:hugo
> npm run _serve:hugo

> _serve:hugo
> hugo server -DFE --minify

Watching for changes in /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/{archetypes,assets,content-modules,content,data,iconography,layouts,package.json,static,themes,tmp}
Watching for config changes in /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/hugo.yaml, /Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/themes/docsy/hugo.yaml
Start building sites …
hugo v0.120.4-f11bca5fec2ebb3a02727fb2a5cfb08da96fd9df+extended darwin/arm64 BuildDate=2023-11-08T11:18:07Z VendorInfo=brew

ERROR render of "page" failed: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/layouts/_default/baseof.html:19:7": execute of template failed: template: _default/single.html:19:7: executing "_default/single.html" at <partialCached "scripts.html" .>: error calling partialCached: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
ERROR render of "page" failed: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/themes/docsy/layouts/blog/baseof.html:33:7": execute of template failed: template: blog/single.html:33:7: executing "blog/single.html" at <partial "scripts.html" .>: error calling partial: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
ERROR render: failed to render pages: render of "page" failed: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/layouts/_default/baseof.html:19:7": execute of template failed: template: _default/search.html:19:7: executing "_default/search.html" at <partialCached "scripts.html" .>: error calling partialCached: "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
Built in 2278 ms
Error: error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): "/Users/brightzheng/workspaces/OSS/OpenTelemetry/opentelemetry.io-mine/themes/docsy/assets/scss/main.scss:1:1":

Interestingly, it worked in a clean clone.

Thanks for confirming that a clean clone builds just fine for you too. I'll remove the "bug" label in that case, and we can continue attempting to debug your setup and/or workflow.

I just tried more and you're right that my flow may not be correct previously, so I updated my previous comment with the workable flow I used.
BTW, is the nvm a must?

I'm glad it's working now!

BTW, is the nvm a must?

No, use of nvm is optional. What is required is to use the active LTS version of Node -- see step 3 of the Local setup instructions.


Update sbt-wartremover, wartremover to 2.4.15

Updates

org.wartremover:sbt-wartremover
org.wartremover:wartremover

from 2.4.2 to 2.4.15.
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.wartremover" } ]

labels: library-update, semver-patch

Superseded by #158.


Can you upload the code? plz!

+1

FYI, we have uploaded the codes of mmTransformer. And this issue will be closed


make idl enum fields which are snake case have proper typing to camel case

solves #2374
fixes:
ts: idl enum field types properly convert from snake case to camel case

Looks great, would you mind to

[ ] Add a change log
[ ] Add a small test, just add some enums with to tests/misc and try to load it in a typescript test it() to verify that the naming is correct.

for why the struct ones are fine I'm not 100% sure but I think it's just because the idl json generated for structs converts the fields to camelcase whereas the enum one does not. so they were parsing the idl basically the same  (see https://github.com/coral-xyz/anchor/blob/master/ts/packages/anchor/src/coder/borsh/idl.ts#L121) but the idl itself was just different.
I'll try to do the test i dont see the misc test but will figure something i guess.

Here's the program for misc, you could try adding some custom types maybe? I don't think you need to go all the way to actually add them to an account or load data into them since this is just to test the IDL.
program: https://github.com/coral-xyz/anchor/tree/master/tests/misc/programs/misc/src
ts test: https://github.com/coral-xyz/anchor/blob/master/tests/misc/tests/misc/misc.ts

i think the test uses an idl that gets build (it's not committed) im fixing it so i can add.

We want a test though in which the IDL gets built? Please don't commit the IDL itself.

idk the issue i was having was that the program had a bunch of UncheckedAccount or similar without /// Check: so i couldn't build the idl for misc. i just added the check and added my enum account and it worked. I wasn't trying to commit the idl

Ah right, you need to run with anchor test --skip-lint

ah well, i guess not anymore

Great, looks good to me. Thanks for the PR!

closes: #2374

oh cool thanks.
ohh i put my changelog in the comment not in the file lmao didn't know about that. Issue was just for the types not the actual object but i guess it's close enough.

So what would be a more accurate changelog?

ts: Fields inside enum variants weren't being converted to camelCase correctly


[SPARKR-183] Fix the issue that parallelize collect tests are slow.

Construct the whole request message to send it once, avoiding write-write-read pattern in case of Nagle's algorithm.

For reference, this post by Nagle recommends avoiding write-write-read at application-level instead of disabling the algorithm.
I didn't read closely, but this post suggests that Windows not suffering from this issue is because of pure luck. So it sounds like doing an OS type check is not a sufficient workaround?

@sun-rui - This is a great find and this fix seems like a nice way to avoid the TCP delays. I tested this in a linux VM and it did seem to fix the issue.  So I'd vote for going with this change over #133 or #135 for now and we can revisit this if we find any further issues.
@concretevitamin - We don't need OS type checks with this change ? Also that post seems pretty old and I think MTU sizes have changed over the years (especially for the loopback interface we use in SpakR).

@shivaram I was referring to OS type check used in the other patch, so I'm also +1 on this fix too

fixed comments.

@sun-rui Can you rebase?

rebased.

Thanks for the patch! Merging in.


Pro Image Viewer

Hey, love the package and really appreciate the work you have put in.
Question, Is there a way to build a viewer version with primary purpose being to import changes JSON and display them.
I want to use this package to allow presenter to make changes and for those to, on a realtime basis, reflect on participants screens. Exporting an image in such a case is not feasible.
Would love to know your thoughts.

Thanks, it's nice to hear that you like my package and appreciate my work, that means a lot to me :)
I never tested it in a real-time case, but I think it should be possible. Below is the example with two widgets that should help you realize this project.
Btw, I will move your question to the Discussions tab now because I prefer that tab for questions.
// Flutter imports:
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';

// Package imports:
import 'package:pro_image_editor/pro_image_editor.dart';

// ignore: constant_identifier_names
const DEMO_IMAGE = 'https://picsum.photos/id/230/2000';

class RealtimeViewerExample extends StatefulWidget {
  const RealtimeViewerExample({super.key});

  @override
  State<RealtimeViewerExample> createState() => _RealtimeViewerExampleState();
}

class _RealtimeViewerExampleState extends State<RealtimeViewerExample> {
  final _editorKey = GlobalKey<ProImageEditorState>();

  @override
  void initState() {
    _connectToRealtimeEvents();
    super.initState();
  }

  @override
  void dispose() {
    // TODO: Don't forget to close your realtime connection
    super.dispose();
  }

  void _connectToRealtimeEvents() {
    String json = '{}';

    /// TODO: Listen to realtime changes and import it like below.

    if (kDebugMode) debugPrint('Import changes');
    _editorKey.currentState?.importStateHistory(
      ImportStateHistory.fromJson(
        json,
        // Optinal configs
        configs: const ImportEditorConfigs(
          recalculateSizeAndPosition: true,
          mergeMode: ImportEditorMergeMode.replace,
        ),
      ),
    );
  }

  void _openViewerEditor() async {
    await precacheImage(const NetworkImage(DEMO_IMAGE), context);

    if (!mounted) return;

    Navigator.push(
      context,
      MaterialPageRoute(
        builder: (context) => _buildEditor(),
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return ListTile(
      onTap: _openViewerEditor,
      leading: const Icon(Icons.smart_display_outlined),
      title: const Text('Realtime-Viewer'),
      trailing: const Icon(Icons.chevron_right),
    );
  }

  Widget _buildEditor() {
    /// There can be problems if the aspect ratio is not the same as in the
    /// presenter view, so I recommend that you make sure the editor has the
    /// same aspect ratio as the presenter.
    return ProImageEditor.network(
      DEMO_IMAGE,
      key: _editorKey,
      callbacks: const ProImageEditorCallbacks(),
      configs: ProImageEditorConfigs(
          designMode: platformDesignMode,
          customWidgets: ImageEditorCustomWidgets(
            mainEditor: CustomWidgetsMainEditor(
              appBar: (editor, rebuildStream) => ReactiveCustomAppbar(
                stream: rebuildStream,
                builder: (_) => AppBar(
                  title: const Text('Realtime-Viewer'),
                ),
              ),

              /// I hide the bottom bar in the viewer because in the viewer it's
              /// not needed for editing.
              bottomBar: (editor, rebuildStream, key) => ReactiveCustomWidget(
                stream: rebuildStream,
                builder: (_) => const SizedBox.shrink(),
              ),

              /// Wrap the full body content inside `IgnorePointer` that the user
              /// can't move layers. Optionally is it maybe also usefull when your
              /// users can zoom so you can also wrap it with `InteractiveViewer`.
              wrapBody: (editor, rebuildStream, content) {
                return IgnorePointer(
                  child: content,
                );
              },

              /// Optionally add elements to the body. For example, show a
              /// progress-spinner each time changes are made.
              /// bodyItems: (editor, rebuildStream) => [
              ///   ReactiveCustomWidget(
              ///     builder: (_) => const CircularProgressIndicator(),
              ///     stream: rebuildStream,
              ///   ),
              /// ],
            ),
          ),

          /// Optionally change the background color like below
          imageEditorTheme: ImageEditorTheme(
            background: Colors.blue.shade100,
          )

          /// Optionally open the editor directly with a state history from the
          /// presenter like below
          /// stateHistoryConfigs: StateHistoryConfigs(
          ///   initStateHistory: ImportStateHistory.fromJson(json),
          /// ),
          ),
    );
  }
}

class RealtimePresenterExample extends StatefulWidget {
  const RealtimePresenterExample({super.key});

  @override
  State<RealtimePresenterExample> createState() =>
      _RealtimePresenterExampleState();
}

class _RealtimePresenterExampleState extends State<RealtimePresenterExample> {
  final _editorKey = GlobalKey<ProImageEditorState>();

  void _sendRealtimeEvents() async {
    if (kDebugMode) debugPrint('Start export');
    var export = await _editorKey.currentState?.exportStateHistory(
      configs: const ExportEditorConfigs(
        /// Send only the current view
        historySpan: ExportHistorySpan.current,

        /// Optionally disable some things from being sent. Keep in mind that
        /// sending stickers will slow down the export progress because the
        /// editor has to convert them to png images first.
        exportSticker: true,
      ),
    );

    var sendJson = await export?.toJson();

    /// TODO: Send your JSON `sendJson` over your real-time connection to the viewers.
    if (kDebugMode) debugPrint('Done export');
  }

  void _openPresenterEditor() async {
    await precacheImage(const NetworkImage(DEMO_IMAGE), context);

    if (!mounted) return;

    Navigator.push(
      context,
      MaterialPageRoute(
        builder: (context) => _buildEditor(),
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return ListTile(
      onTap: _openPresenterEditor,
      leading: const Icon(Icons.present_to_all_outlined),
      title: const Text('Realtime-Presenter'),
      trailing: const Icon(Icons.chevron_right),
    );
  }

  Widget _buildEditor() {
    /// There can be problems if the aspect ratio is not the same as in the
    /// presenter view, so I recommend that you make sure the editor has the
    /// same aspect ratio as the presenter.
    return ProImageEditor.network(
      DEMO_IMAGE,
      key: _editorKey,
      callbacks: ProImageEditorCallbacks(
        onImageEditingComplete: (bytes) async {
          /// Handle the final image => Maybe upload it to your server.
        },

        /// Optionally send changes directly when changes occur. Below in
        /// `customWidgets` you can see the example of how to add a button to
        /// send manually for more control for the presenter.
        /// mainEditorCallbacks: MainEditorCallbacks(
        ///   onAddLayer: (value) => _sendRealtimeEvents(),
        ///   onUpdateLayer: (value) => _sendRealtimeEvents(),
        ///   onUndo: _sendRealtimeEvents,
        ///   onRedo: _sendRealtimeEvents,
        /// ),
        /// paintEditorCallbacks: PaintEditorCallbacks(
        ///   onDrawingDone: _sendRealtimeEvents,
        ///   onUndo: _sendRealtimeEvents,
        ///   onRedo: _sendRealtimeEvents,
        /// ),
        /// More editor callbacks...
      ),
      configs: ProImageEditorConfigs(
        designMode: platformDesignMode,

        /// Update the appbar with a button that the presenter can send it to the
        /// viewers
        customWidgets: ImageEditorCustomWidgets(
          mainEditor: CustomWidgetsMainEditor(
            appBar: (editor, rebuildStream) => editor.selectedLayerIndex < 0
                ? ReactiveCustomAppbar(
                    stream: rebuildStream,
                    builder: (_) => _buildCustomAppBar(editor),
                  )
                : null,
          ),
        ),
      ),
    );
  }

  AppBar _buildCustomAppBar(ProImageEditorState editor) {
    return AppBar(
      automaticallyImplyLeading: false,
      foregroundColor: Colors.white,
      backgroundColor: Colors.black,
      actions: [
        IconButton(
          tooltip: 'Cancel',
          padding: const EdgeInsets.symmetric(horizontal: 8),
          icon: const Icon(Icons.close),
          onPressed: editor.closeEditor,
        ),
        const Spacer(),

        /// Create your custom button to send realtime events to the viewers
        IconButton(
          tooltip: 'Send to viewers',
          padding: const EdgeInsets.symmetric(horizontal: 8),
          icon: const Icon(Icons.send),
          onPressed: _sendRealtimeEvents,
        ),
        const Spacer(),
        IconButton(
          tooltip: 'Undo',
          padding: const EdgeInsets.symmetric(horizontal: 8),
          icon: Icon(
            Icons.undo,
            color: editor.canUndo == true
                ? Colors.white
                : Colors.white.withAlpha(80),
          ),
          onPressed: editor.undoAction,
        ),
        IconButton(
          tooltip: 'Redo',
          padding: const EdgeInsets.symmetric(horizontal: 8),
          icon: Icon(
            Icons.redo,
            color: editor.canRedo == true
                ? Colors.white
                : Colors.white.withAlpha(80),
          ),
          onPressed: editor.redoAction,
        ),
        IconButton(
          tooltip: 'Done',
          padding: const EdgeInsets.symmetric(horizontal: 8),
          icon: const Icon(Icons.done),
          iconSize: 28,
          onPressed: editor.doneEditing,
        ),
      ],
    );
  }
}


Debug a Googletest testcase from the Testing interface gives error

If I try to debug a single test case through the Testing panel by clicking the "Debug Test" icon, test starts but nothing happens. Output of the test result windows stays empty, an error message is shown and it will be stuck like this forever:

I tried to configure a debug launch configuration as explained here, but it doesn't work. If I understood that page correctly, it works only with the test executable currently selected as "launch target".
It happens on any CMake project using googletest I have on my machine. Running and debugging test executables separately works perfectly fine.
Information:
OS: Windows 10
Visual Studio Code 1.77.3
CMake Tools extension 1.14.30

@NoMore201 Could you test this again in the most recent CMake Tools version? I believe it should be fixed.

Based on last comments, this should be fixed. If not, @NoMore201 Could you please create a new issue with more context and a repro project? Thanks.


User role should have READ-ONLY access to all Guardium records

Comments

x_ibmrt_gdpva.user role should have READ-ONLY access to all Guardium records
x_ibmrt_gdpva.user role should not have WRITE access to any Guardium record

Fixed in version 1.3.26


v1.0.12 and v1.0.13 do not load properly

Thanks for this extension!
I tried v1.0.13 and the extension is stuck in "loading state". The exact output is "Loading extension state...". See screenshot below. After trying older versions, I found out that v1.0.10 works fine, but v1.0.12 and v1.0.13 do not load at all on my Windows 10 computer.

Here is the result when using v1.0.10

VS Code information
Version: 1.86.2 (user setup)
Commit: 903b1e9d8990623e3d7da1df3d33db3e42d80eda
Date: 2024-02-13T19:40:56.878Z
Electron: 27.2.3
ElectronBuildId: 26908389
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

Hi, could you please try to set the api key using the "Deepl: Configure" command?

I forgot to specify that the "DeepL: Configure" command does not work either on my computer.

There was a problem with initialising the extension state, if no api key was configured.
The problem was fixed with the release of version 1.0.14.


Bug in perfdata formatter

This commit here introduces a bug in the perfdata formatting:
https://github.com/NETWAYS/check-sar-perf/commit/ad8166d53bf2c3f897707222d138986c4c8c7f31#diff-f8328edc0a35a97a06cef6e296c43ed22512ea7d28355779fbd2b225b1660a64L105
This new code uses strip which only works if the unwanted characters are at the start/end. However, this misses  rkB/s=0.00
columns[idx] = ''.join(j for j in element if j not in badchars)
string = "%s=%s" %(element.strip('%/'), data[idx].strip())

Output should be:
sar OK | tps=0.00 rkBs=0.00 wkBs=0.00 areq-sz=0.00 aqu-sz=0.00 await=0.00 svctm=0.00 util=0.00
Output is:
sar OK | tps=0.00 rkB/s=0.00 wkB/s=0.00 areq-sz=0.00 aqu-sz=0.00 await=0.00 svctm=0.00 util=0.00

Plus, the columns[idx] assignment is now unused and could be dropped.


init + finish runners

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

GitHub release
v1.4.0

Your semantic-release bot :package::rocket:


ITInstanceAdminTest #update / #updateViaEntity fail when two tests are running simultaneously

The update and updateViaEntity tests update the current instance display name to a random one. If there are two tests running simultaneously the assertions might fail, because the updated name can match the wrong display name.
We need to fix these tests so that they are self-contained.

cc @rajatbhatta  to take a look at the related issue

thanks for looking into this @rajatbhatta! Any update on this?


[Storage] Use embedme to inject code snippets in README

READMEs are the first point of interaction for users looking to use our SDKs. It contains several useful code snippets for users to get started quickly. It is important to keep these code samples correct at all times. However, when code is modified README gets outdated and hard to figure out which README is referencing the modified piece of code.
Use https://github.com/zakhenry/embedme to embed code snippets from compilable Java files.
Code snippets in README
Samples in all README.md files should use compilable code to ensure they are updated when source code is updated. To do so, we use a utility tool to embed source code from Java files into README markdown files. Steps to add code snippets in README using the tool:

Download and install NodeJS
Create ReadmeSamples.java file in src/samples/java/ for which README.md should be updated
Add all the code snippets in the above Java file that should be included in README.md
Now, in README.md, add the embedme tag as shown below (the sample shows inserting code from line number 10 to 20)

<!-- embedme path/to/ReadmeSamples.java#L10-L20 -->
```java
```

To embed code using the tool, run the following command

npx embedme path/to/README.md

Verify the README.md file to confirm the samples are inserted as expected

please assign me this issue

@mclay Thank you for taking this up! I have assigned it to you.

Hi *,
in this pull request #17864 I made a change on the README of Azure Storage File Share.
Consequently, I rewrote some embedme references.
In order to be sure that it was all fine, I checked the other README of Azure Storage and all snippets have an embedme reference.
I don't want to take control of this issue. I want just to do feedback.
Anyway, it is better to wait for the checking of @mclay in order to close the issue.

Closing as it appears that Storage is using codesnippet-maven-plugin (the new way to inject README code snippets) to inject code snippets into README files.


Feedback from Joel

Major:

[x] Main view is map and point color is price, so that locations can be compared
[x] Make map interval selection that filters the data further after the widget selections are applied.

Minor:

[x] Capitalize names in tbale
[ ] Color code maps based on neighborhood
[ ] Color based on neighbood
[x] Zoom map / fix margins
[x] Make numbers in cards a big bigger

[x] Consider showing median?

A note on the color code of the neighborhoods:
While we were planning on coloring by neighborhood in our discussion with Joel, we ultimately decided that coloring by price was a better choice, as it added more visual information to our map.