Elasticsearch Client Plugin Contribution

Title: Elasticsearch Client Plugin Contribution
Link: https://github.com/bbossgroups/pinpoint/tree/elasticsearch-plugin/plugins/elasticsearch-bboss
Target: Elasticsearch Rest Client BBoss
Supported Version: pinpoint 1.9.0
Description: This agent plugin is worked with elasticsearch rest client BBoss,BBoss is a best Java Highlevel Rest client for ElasticSearch.
ServiceTypes: [9201, "ElasticsearchBBoss"],[9202, "ElasticsearchBBossExecutor"]
Annotations: [971, "es.args"],[972, "es.url"],[973, "es.dsl"],[974, "es.action"],[975, "es.responseHandle"],[976, "es.version"]
Configurations: List of configuration keys and description the plugin adds.
profiler.elasticsearchbboss.enabled=true //enable or disable this plugin, default true.
profiler.elasticsearchbboss.recordResult=false //record or disable record method return value ,default false.
profiler.elasticsearchbboss.recordArgs=true //record or disable record method args,default true.
profiler.elasticsearchbboss.recordDsl=true //record or disable record dsl,default true.
profiler.elasticsearchbboss.recordESVersion=true //record or disable record ES Version infomation,default true.
profiler.elasticsearchbboss.recordResponseHandlerClass=false //record or disable record response handle class ,default false.
profiler.elasticsearchbboss.maxDslSize=50000 //set the max length dsl script,over this size dsl will be substring.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: RoySRose:white_check_mark: binDongKim:white_check_mark: koo-taejin:white_check_mark: jaehong-kim:white_check_mark: denzelsN:white_check_mark: yin-bp:x: minwoo-jungYou have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #5613 into master will increase coverage by 0.04%.
The diff coverage is 24.84%.

@@            Coverage Diff             @@
##           master    #5613      +/-   ##
==========================================
+ Coverage   40.43%   40.47%   +0.04%
==========================================
  Files        2870     2880      +10
  Lines       86576    86799     +223
  Branches    11453    11491      +38
==========================================
+ Hits        35005    35135     +130
- Misses      48533    48608      +75
- Partials     3038     3056      +18

Impacted Files
Coverage Δ

...inpoint/collector/cluster/GrpcAgentConnection.java
80% <ø> (ø)
:arrow_up:

...orp/pinpoint/flink/receiver/TcpSourceFunction.java
0% <ø> (ø)
:arrow_up:

...point/collector/config/CollectorConfiguration.java
52.04% <ø> (ø)
:arrow_up:

.../pinpoint/collector/dao/hbase/HbaseTraceDaoV2.java
0% <0%> (ø)
:arrow_up:

.../pinpoint/flink/receiver/SourceContextManager.java
0% <0%> (ø)
:arrow_up:

...corp/pinpoint/flink/receiver/AgentStatHandler.java
0% <0%> (ø)
:arrow_up:

...point/profiler/sender/grpc/SpanGrpcDataSender.java
0% <0%> (ø)
:arrow_up:

...point/profiler/sender/grpc/StatGrpcDataSender.java
0% <0%> (ø)
:arrow_up:

...om/navercorp/pinpoint/web/view/NodeSerializer.java
0% <0%> (ø)
:arrow_up:

...n/java/com/navercorp/pinpoint/flink/Bootstrap.java
0% <0%> (ø)
:arrow_up:

... and 82 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 234099a...dd2b3d7. Read the comment docs.


[Feature] Allow User To Toggle between light/dark mode

As a user, I would like to interact with the UI depending on my light or dark preference.

I would like to try this out. Would a simple light/dark toggle button suffice?

Assign me!

I would like to try this out. Would a simple light/dark toggle button suffice?

Yes, but it also has to be functional


provide for server's hardware metircs

This is the first commit!

@matthewei

add README file
add github action job yaml. about: build、lint、docker build、docker push, etc.

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


[Feature][cluster] enhanced load balancing

Describe the feature
In the production environment, our machine configurations may not be exactly the same. Therefore, in the process of load balancing, we hope to distribute traffic according to the weight of the configured machines.

Load balancing according to work group


associated type Span, Tracer as short-hand, and *Protocol types

Motivation:
This is a revival of #84
where we try to KEEP compatibility with versions below 5.7 with a
compatibility "legacy" tracer type, but otherwise move towards requiring
5.7 for all the "nice" apis that use associated types and any   TracerProtocol and friends
Replaces https://github.com/apple/swift-distributed-tracing/pull/92
Modifications:

Tracer -> TracerProtocol
Tracer is now a namespace in order to Tracer.withSpan {} easily
Span -> SpanProtocol
Introduce LegacyTracerProtocol which does not make use of associated
type Span, and can be used in 5.6 libraries; they can deprecate and
move away form it ASAP as they start requiring 5.7

Result:
Offer the APIs we want in 5.7 but remain compatible with 5.6 until we
drop it as soon as 5.9 is released as stable - this allows us to adopt
eagerly in libraries without having to wait for 5.9 to drop.

Pinged some folks for more review -- if you spot anything off please comment and I'll address in follow ups! 👍

Resolves https://github.com/apple/swift-distributed-tracing/issues/78


does Warp-CTC support bucketing?

I try to use bucketing lstm and Warp-CTC to do OCR task,
and I find the output never update when training,
However if I only use one bucket the output works fine.
So I doubt  whether Warp-CTC could use in a bucketing way?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


OpenCollective message pushed up and hidden by dependency tree

After npm install runs, it displays a tree of installed dependencies which pushes the OpenCollective message up out of view (assuming the dependency tree includes more than a few items), so a user isn't likely to see it. :(

npm install --silent works, but that depends on users using that flag. I'm not sure if there's an obvious solution here, but just opening this issue to brainstorm some ideas.

I tried "postinstall": "sleep 5 && ./node_modules/.bin/opencollective-postinstall || exit 0" to see if that would bump the message down below the tree, but the tree prints after the postinstall script finishes, so it didn't work.


[BUG] - Event Click on screens (sm or md) does not work

NextUI Version
2.6.X
Describe the bug
Hi NextUi team,i found a bug. It is in the onClick event of any html element, that is to say when you click on any html element you don't hear the onClick event.
This only happens in sm, md screens (only in these sizes I have tested them).
Note: I read the documentation thinking that the installation method had been changed, but it seems to be ok.

In this video we can see in the case of version 2.4.6 that the onClick event works on screens (sm or md).

https://github.com/user-attachments/assets/74055105-36dd-44f8-90de-88b8f7ea2893

And in this video we can see that version 2.6.10 the onClick event on screens (sm or md) no longer works

https://github.com/user-attachments/assets/8534f6bf-0ac5-4f30-a010-f42d35532815
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
-Open the developer tools
-Adapt dimensions to a device (sm or md)
-Click on any Onclick event and it is not fired
Expected behavior
onClick events should work on sm or md screens.
Screenshots or Videos
No response
Operating System Version

OS [Linux, Windows, Android]

Browser
Chrome

you need to use onPress. See https://github.com/nextui-org/nextui/issues/4292#issuecomment-2532881764


About gpt-3.5-turbo maxContextLength

OpenAI has increased the maximum token length supported in gpt-3.5-turbo to 16k. May I ask if it's possible to update the ModelType enumeration and relevant configurations accordingly?

The new 16k gpt3.5 model is now part of the ModelType enum :slightly_smiling_face: It was just released in version 0.6.0


Tunnel not ready in Windows 1.3.1

Customer reports 1.3.1 errors out with Tunnel not ready error. 1.3.0 works fine.
Awaiting log export.

Do we handle the case where the internet resource is not sent from the portal? This will happen on all Starter accounts and Clients that don't have the policy active.


snd: 23.3 -> 23.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/snd/versions.
meta.description for snd is: Sound editor
meta.homepage for snd is: https://ccrma.stanford.edu/software/snd/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 23.4 with grep in /nix/store/sc2dpmh6i6ysa6f8ypa4ai8qqm5kp1l8-snd-23.4
found 23.4 in filename of file in /nix/store/sc2dpmh6i6ysa6f8ypa4ai8qqm5kp1l8-snd-23.4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
snd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sc2dpmh6i6ysa6f8ypa4ai8qqm5kp1l8-snd-23.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A snd https://github.com/r-ryantm/nixpkgs/archive/44ffb7247b91b2d5f936d8fa7c12fe576e9785a3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sc2dpmh6i6ysa6f8ypa4ai8qqm5kp1l8-snd-23.4
ls -la /nix/store/sc2dpmh6i6ysa6f8ypa4ai8qqm5kp1l8-snd-23.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.


Fixing the redirection links in the table of contents, #17

Description
Enhanced the README.md file to improve overall clarity and usability. Added detailed sections, structured content, and included additional information such as the tech stack, features, and contribution guidelines. Additionally, fixed the redirection links in the table of contents for smooth navigation.
Related Issues

Closes #17

Type of PR

[x] Documentation Update

Screenshots / videos (if applicable)
N/A
Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with the project's main branch before making this PR.
[x] I have performed a self-review of my work.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
This PR focuses on restructuring the README.md for better readability and user guidance, fixing the redirection links in the table of contents, and addressing the improvements outlined in issue #17.

@rahulgithub-web still not working

@rahulgithub-web please fix this other wise i have to close this pr


Add chai and mocha ES6 shims.

Allows the following:
import { describe, it } from 'mocha';
import { expect } from 'chai';

Thanks @rwjblue!


wsl.exe does not properly parse PowerShell variable for distribution

Windows Version
10.0.23570.1000
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Pengwin
Other Software
PowerShell 7.4.0-preview.6
Repro Steps
Run the following in PowerShell to set the name of the default WSL distro as $defaultDistro and echo:
$defaultDistro = wsl -l -v | Where-Object { $_ -match '(\*|DEFAULT)' } | ForEach-Object { ($_ -split ' +')[1] }; echo $defaultDistro
Run the following in PowerShell to set the username of the default user of $defaultDistro as $defaultUser and echo:
$defaultUser = wsl -d $defaultDistro whoami; echo $defaultUser
Expected Behavior
The username of the default user of $defaultDistro would be set as $defaultUser and echoed.
Actual Behavior
$defaultUser is set toArgument --distribution is missing a required parameter.
This does not correspond to a missing distribution name or incorrect distro name.

Adding .Trim() to the $defaultDistro assignment does not address the issue:

Diagnostic Logs
No response

Thank you for reporting this @sirredbeard.
WSL does not "parse" powershell variables. Powershell expands the variables and then calls WSL with expanded variables.
Interestingly, I can reproduce the issue you're seeing but I'm not a powershell expert so i can't explain it. I recommend opening an issue on [the powershell repository][https://github.com/PowerShell/PowerShell)

This drove me crazy so I had to look into it. The answer is because wsl.exe -l -v is UTF 16! So you can fix this by setting the UTF8 variable.
PS C:\Users\crloewen> $env:WSL_UTF8=1
PS C:\Users\crloewen> $defaultDistro = wsl -l -v | Where-Object { $_ -match '(\*|DEFAULT)' } | ForEach-Object { ($_ -split ' +')[1] }; echo $defaultDistro
Ubuntu
PS C:\Users\crloewen> wsl echo $defaultDistro
Ubuntu
PS C:\Users\crloewen>

It's always UTF.


Importing AND displaying related models at importing preview

I have this Course model (also look at the many to many fields):
class Course(models.Model):

    class Meta:
        unique_together = ('name', 'year', 'semester')

    name = models.CharField(primary_key=True, max_length=250, null=False, blank=False)
    year = models.IntegerField(null=False)
    semester = models.IntegerField(null=False)
    active = models.BooleanField(default=True)

    _students = models.ManyToManyField(to=User, blank=True)
    _teachers = models.ManyToManyField(to=User, blank=False, related_name='courses_teacher')

    def add_teacher(self, teacher: 'Teacher'):
        self._teachers.add(teacher)

    def add_student(self, user: 'User'):
        self._students.add(user)

For extracting the students from an excel sheet (which has a horrific format) I used this custom resource model:
class StudentResource(resources.ModelResource):
    username = Field(attribute='username', column_name='STUDENT_ID')
    first_name = Field(attribute='first_name', column_name='NAME')

    class Meta:
        model = User
        skip_unchanged = True
        report_skipped = False
        import_id_fields = ('username',)
        fields = ('STUDENT_ID', 'NAME')

    def import_data(self, dataset, **kwargs):
        """
        clean data from the list, only extract students
        """
        subsection = dataset[23:len(dataset) - 4]

        data = tablib.Dataset()

        # distribute data for students
        data.headers = ['NAME', 'STUDENT_ID']
        for i in subsection:
            data.append([i[1], str(i[8])])

        cleaned_data = data
        return super().import_data(cleaned_data, **kwargs)

    def after_save_instance(self, instance, row, **kwargs):
        print(instance)
        # distribute data for students
        group, _ = Student.create_group()
        group.user_set.add(instance)

As you can see, in import_data() I'm only extracting students from the sheet.
Is there a way to extract and display other data (which would be for other models) from the same excel sheet in the import preview?
In this case in particular, I would also need to extract other data from the dataset. Something like, let's say course_name= data[13][3] or teacher_name= data[17][3]. Currently I'm running out of ideas, since the overwritten import_data method returns the super method (which only receives a single dataset as a parameter).
If there's a way to do this, would it be possible to display multiple models at the preview?

This is a tricky issue since import_export is built around working with a single dataset.  Others have worked around this by using JSON in certain fields as a mechanism for supporting nested data.
You can read through this thread for some further background and ideas.

If there's a way to do this, would it be possible to display multiple models at the preview?

It's not possible with the current framework, but you could try to customize.  I think it will be tricky though.


feat: adds blocking versions of camera commands that can tween/transition

Closes https://github.com/godot-escoria/escoria-issues/issues/138

Findings :

Change button_set_pos (the current command is outside the room's limits) to

camera_set_pos_block 2 1460 1050

And all 3 camera limits for the room to 0,0,2260,2000.
After this, use the set_pos button. As the camera is resetting back to the player, click once to make them walk somewhere else. When Mark stops walking and the camera has stopped updating, click to walk somewhere else. There is an extremely minor camera jump (a handful of pixels). Maybe too little to worry about, but the perfectionist in me noticed it :-)
Happens also after camera_push_block

Camera_push_block I noticed moves to the object and not the child ESCLocation. I'm not sure if this is the default behavior or not. It does respect the camera node setting if configured though.

I'm not sure if this is fixable (I can't think of any way to fix it), but camera_set_target tweens to where the target was when the tween started. If you set the target back to the player then move the player while it's tweening, the tween doesn't update to match the player's updated position, so jumps when it gets to the destination of the tween.

It's easier to watch this bug out than me to try and describe it. Change the set_pos button action to the

camera_set_zoom 1.3 2
wait 2
camera_set_pos_block 2 1460 1050

And this looks like one of Dante's 9 circles of hell... While I expected a failure, this doesn't fail how I'd expected it to - the zoom doesn't end up where it would if you did the zoom individually (presumably as it keeps zooming until the block finishes.

camera_set_zoom 1.3 2
camera_set_pos_block 2 1460 1050

Another interesting combo. The set_target doesn't fire correctly so it doesn't reset back to the player until you move the character again which appears to be a bug.

camera_set_pos 2 1460 1050
camera_shift_block 1000 -400 2 LINEAR
camera_set_target 1 player

This is another version of that bug
camera_set_pos 2 1460 1050
wait 1
camera_set_target 1 player

Basically, as long as I don't try and mix non-blocking and blocking commands, it works fine. A non-blocking walk with blocking camera worked, but mixing camera commands leads to some unexpected situations.

Change button_set_pos (the current command is outside the room's limits) to

camera_set_pos_block 2 1460 1050

And all 3 camera limits for the room to 0,0,2260,2000. After this, use the set_pos button. As the camera is resetting back to the player, click once to make them walk somewhere else. When Mark stops walking and the camera has stopped updating, click to walk somewhere else. There is an extremely minor camera jump (a handful of pixels). Maybe too little to worry about, but the perfectionist in me noticed it :-) Happens also after camera_push_block

Yeah, this is pretty difficult to reproduce, and is, like you said, very minor. And actually, given that accept_input NONE should still be in effect when that call happens, it looks like the player being able to walk as above is a bug. (I'll raise it.)
Using camera_set_target_block fixes this anyway.

Camera_push_block I noticed moves to the object and not the child ESCLocation. I'm not sure if this is the default behavior or not. It does respect the camera node setting if configured though.

By design. That's how camera_push was originally created as far as I can tell (looking at the code in ESCCamera.)

I'm not sure if this is fixable (I can't think of any way to fix it - and I know this is in relation to a non-blocking camera command sorry :D ), but camera_set_target tweens to where the target was when the tween started. If you set the target back to the player then move the player while it's tweening, the tween doesn't update to match the player's updated position, so jumps when it gets to the destination of the tween.

3a) Should we have a blocking set_target command?

There already is camera_set_target_block ...maybe you missed it? :)

Then there's this jumpy camera where the set_pos_block unexpectedly zooms...

camera_set_zoom 1.3 2
wait 2
camera_set_pos_block 2 1460 1050

This is because the wait command is finishing too close to the set time for the zoom. Using wait 3 (or even wait 2.5) corrects this. Probably a good idea to document it, or use the blocking version. :)

And this looks like one of Dante's 9 circles of hell... While I expected a failure, this doesn't fail how I'd expected it to - the zoom doesn't end up where it would if you did the zoom individually (presumably as it keeps zooming until the block finishes.)

camera_set_zoom 1.3 2
camera_set_pos_block 2 1460 1050
camera_set_target 0 player

No. Once a blocking camera command runs, it ends any currently-running tweens, in this case, the unfinished zoom. In fact, every camera command when run will stop any already-running tweening, if any are occurring.

Another interesting combo. The set_target doesn't fire correctly so it doesn't reset back to the player until you move the character again which appears to be a bug.

camera_set_pos 2 1460 1050
camera_shift_block 1000 -400 2 LINEAR
camera_set_target 1 player

Actually this is because camer_shift_block is failing validation (going outside limits) and so camera_set_target doesn't actually fire. However, expanding the limits corrects this, although that has actually revealed another bug (at the very end, the camera transitions to the top-right of the screen for some reason). I'll fix that by delaying a frame since it looks as though the cause is the tween cancelling juuuuust before it finishes.

This is another version of that bug
camera_set_pos 2 1460 1050
wait 1
camera_set_target 1 player

Similar deal here, although the wait also isn't long enough (1s waiting for a 2s tween).

Basically, as long as I don't try and mix non-blocking and blocking commands, it works fine. A non-blocking walk with blocking camera worked, but mixing camera commands leads to some unexpected situations. Couldn't think of any other interesting commands to try and mix in with blocking cameras that would be worth testing.

Good stuff. I think we should really call out the dangers of mixing blocking with non-blocking camera commands, and that the blocking commands will work for most people in most cases. Also call out that mixing those calls can result in unexpected behaviour so use at own risk.

I tried the following script. This works correctly if the set_pos_block is 2 seconds (i.e. longer than the walk takes to complete) but the set_target goes pear shaped if the walk is still going. Other than this one (and the ones that I've run that worked) I'm running out of synthetic tests for things that might break.
[ reset all room limits to 0,0,2260,2000 ]
accept_input NONE
walk player r7_lower_stairs
camera_set_pos_block 1 1260 550
camera_set_target_block 1 player
accept_input ALL

I wonder if animated gifs work with Github...

Nice! Animated gifs work with Github. I'm going to screen2gif all the bug reports now!!! :D
Here's the set_zoom_height command. It should centre on his feet (presumably) not wherever it ends up. I've never used the command before, but I can't see why a zoom shouldn't zoom on the player.

Nice!
As for where it zooms, I think it's always just on where the screen is centered
For camera_set_zoom_height, its meant to zoom in so the camera occupies the given height in pixels (according to the docstring).
For camera_set_zoom, there is no target, either...just wherever the camera happens to be centered at the time.
@StraToN will likely know more, but I don't think the intent was to zoom in on a target/character, given this is the zoom target code:
_zoom_target = Vector2(1, 1) * p_zoom_level
So it looks like this is working as desired, at least based on the original intent.

Looks good to me. Thanks!


Support environment variable alternatives from ANCM logging settings

When running in App Service with Run-From-Zip, files are read-only so user cannot tweak web.config to change things like stdoutLogEnabled. Having an environment variable alternative would solve this.
This first came up on https://github.com/Azure/app-service-announcements-discussions/issues/32#issuecomment-407813574

@davidebbo Is stdoutLogEnabled the only setting that you'd like to be able to override via env vars, or are there others?
Also, do you have comparative usage numbers of this deployment mechanism vs the others? (WebDeploy, git-based etc.)  We'd like to get a sense of priority of this for ASP.NET users on Antares.
cc @DamianEdwards @davidfowl

@muratg Not sure, as I'm not familiar with all of them. This is the one that came up.
I don't have the percentages, but note that once it's surfaced by VSTS and VS, and is suggested as the default, lots of apps will start using it by default.

This honestly also helps us with diagnostics in general. It would allow users to quickly turn on stdoutlogging instead of having to muck with the web.config.

@davidfowl were you thinking about constraining the feature to stdout logging? Or any other config key?


Enable new build package location for grype

This should enable DB publisher and builds by swapping grype installs to reference the new main package location.

@wagoodman , seems like something might still not quite be working: https://github.com/anchore/grype-db/actions/runs/5570504224/jobs/10177759882

@wagoodman , ah, I see why.  We also need to pump yardstick version under the acceptance tests since they all have their own poetry.lock.

Thanks @westonsteimel


Fix index creation

opendistro-for-elasticsearch/security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Bug fix

Github Issue # or road-map entry, if available:
https://github.com/opendistro-for-elasticsearch/security/issues/893
https://github.com/opendistro-for-elasticsearch/security-kibana-plugin/issues/598

Description of changes:
Fix requests hitting _resolve endpoint failed with 403 forbidden error due to accessing indices without proper permission. By re-configuring the logic in code, requests that were to return 403 will now success with 200 and not permitted indices will be filtered from its result.

Additional code added to unit tests and configuration ensures such behavior change does not compromise code integrity.

Why these changes are required?
Customers reported being unable to create index in Kibana due to previously mentioned 403 forbidden error. Such error is effecting customer production thus needs ti be fixed.

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)
Old behavior: _resolve endpoint is not properly handled in code, thus causing it to return 403 status code when ever indices that are not allowed access to the user are contained in the response. This error causes Kibana failed to allow user to create indices that they should otherwise be allowed.

New behavior: _resolve api is properly handled. If previously mentioned invalid access is encountered, security plugin automatically remove invalid indices accesses in the response and return with 200 ok status.

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)
Unit test regarding such feature is updated to accommodate with such change.
Manual tests on Kibana web UI is conducted to ensure creation of desired index pattern is allowed.

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)
Update multi-cluster integratoin tests as _resolve api introduced cross-cluster support for getting different index patterns

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

Could you please close the PR1100 if it's no longer needed?

Could you please close the PR1100 if it's no longer needed?

sure, closed it

The PR title is quite confusing, it suggests that there is a problem with create index requests, not with handling resolve requests.


not found file hamming.py

cannot import name 'hamming' from 'asmk'. thansk

Please follow the instructions on how to run the code https://github.com/jenicek/asmk#running-the-code


Automate - Notification for Quota Exceeded.

Modified rejected method in System/CommonMethods/QuotaStateMachine.class/methods.
Message is 'Quota Exceeded: ' and the standard quota exceeded message.
Previous Notification PR(s): 12306, 12347

@tinaafitz
Please Review

bump @tinaafitz

@billfitzgerald0120 Changes look good. Can we remove the commit WIP?

@billfitzgerald0120
Can we create a spec for this method

@mkanoor
Created spec
Please Review

See PR: https://github.com/ManageIQ/manageiq-content/pull/11


Incorrect check on optional schema parameter

Bug Report
1. Minimal reproduce step (Required)
Tried to run the example from the docs.
schema Person:
    gender: "Male" | "Female"
    name: Name

schema Name:
    first: str
    middle?: str  # Optional, but must be non-empty when specified
    last: str

    check:
        first != ""
        last != ""
        middle != "" if middle

alice = Person {
    gender: "Female"
    name.first: "Alice"
    name.last: "White"
    name.middle: ""
}

Ran with kcl person.k.
2. What did you expect to see? (Required)
Error, since middle name is empty.
3. What did you see instead (Required)
alice:
  gender: Female
  name:
    first: Alice
    last: White
    middle: ''     <-- Expected an error on this line.

4. What is your KusionStack components version? (Required)
I don't have the folder required for this step on my machine (Kusion/kclvm-go/KCLVM), since built the binaries from source (commit 82c2be097be0ae29b8a159171f0d379737bd3eba from March 8th).
Playground link.

@mihaigalos. Hello, thank you for your question. In KCL, if expressions will filter all expressions that are not true (False, empty string, empty list, etc.), just like Python and Javascript. Ref KCL FAQ: https://kcl-lang.io/docs/user_docs/support/faq-kcl#11 -how-to-judge-whether-a-variable-is-noneundefined-and-whether-a-stringdictlist-is-empty

I will transfer it to discussion


Hub product attributes not working

In abstract class AbstractProduct
this is wrong
public function getAvailableAttributesProperty()
{
    return ProductType::find(
        $this->product->product_type_id
    )->productAttributes->sortBy('position')->values();
}

has to be like this to work
 public function getAvailableAttributesProperty()
{
    return Attribute::whereAttributeType(Product::class)->orderBy('position')->get();
}

@SteliyanPStoyanov Can you explain a bit more what issue you think this is causing and why your suggestion will fix it?

That is not how attributes work with products. Product attributes are mapped via product types. Your suggestion is wrong.

in settings you have Attributes in type Product when you add new never show anyware and how have to fill this fields

You need to make sure you associate them to the product type under hub/product-types

ok but why is type Product  not ProductType like Brand is Brand


tricky column names

Need to show students more explicitly how to deal with column names with spaces etc. maybe in data cleaning

✨ backticks around name ````

This will be covered in the general issues guide #147


Ignore new sample tests

These samples have a Console.ReadKey which was hanging the live test runs.

API change check
API changes are not detected in this pull request.


What is the lru list in cache for?

The lru list is maintained, but I don't see it used anywhere.
This is not a bug, but I don't know where to post questions/comments.

It's used in the cache type's add and get methods.

You can post questions on the golang-nuts mailing list.

Thanks!
The order of list ll is maintained in add and get, but I don't see ll used anywhere.

Check out the removeOldest method.


Compatibility Issue: proc-macro server's api version (5) is newer than rust-analyzer's (2)

I encountered an error when using cargo-equip with my project. Below are the details:

My code structure:

src/main.rs
use proconio::{input};

fn main() {
    input! {
        a: u32,
    }
    println!("{}", a);
}

Cargo.toml
[package]
name = "library_checker"
version = "0.1.0"
edition = "2021"

[dependencies]
proconio = { version = "0.4.3", features = ["derive"] }

After running cargo equip, I received the following output and error message:
warning: Currently cargo-equip only supports Feature Resolver v1, and may search more crates than the real Cargo does. Please watch https://github.com/qryxip/cargo-equip/issues/94
...
error: could not bundle the code

-<EMAIL_ADDRESS>as `crate::__cargo_equip::crates::__proconio_derive_0_2_1`
-<EMAIL_ADDRESS>as `crate::__cargo_equip::crates::proconio`

Caused by:
  proc-macro server's api version (5) is newer than rust-analyzer's (2)

I expected cargo equip to bundle my project code successfully, but it fails due to a version incompatibility between proc-macro's API and rust-analyzer.

Cargo version: cargo 1.84.0-nightly (031049782 2024-11-01)
cargo-equip version: latest
Operating system: Arch Linux

Is there a recommended workaround or an update planned to resolve this compatibility issue?

Try this: --toolchain-for-proc-macro-srv nightly-2023-07-28-<arch>
To install:
rustup toolchain install nightly-2023-07-28
rustup toolchain list  # to check the one fully qualified with <arch>


Outlined text

I want to create text with for example black color outline and red fill color (see attachment). I can use code as below but in that way I can set only outline color and fill color is transparent / none. I'm now using ugly workaround with one Text red and second Text larger with black outline drawn on the same place. This is not working correctly on some display densities. So the request for enhancement is to support outlined text with only one Text and be able to set color of outline and fill color.
Why is this important? It's used for example for better readability of subtitles on different backgrounds (white text, black outline). The difference in better readability can be enormous.
Text(
    text = "Sample",
    color = "<only one color>"
    style = TextStyle.Default.copy(
        fontSize = 64.sp,
        drawStyle = Stroke(
            miter = 10f,
            width = 5f,
            join = StrokeJoin.Round
        )
    )
)

This isn't supported by the underlying text rendering engine (on Android too). Drawing the text twice is the way to go here.
As to why it looks bad on some densities, that's worth investigating. Perhaps the outline text should not be larger.

That are bad news that there is no support for this directly. But thanks for info, I hope it will be added in the future.
I have prepared minimal example of used code. I believe it should produce same good or bad (if I messed up something) result on each platform. But the results are very different even if outline width is transformed to sp to have same "scale" as font size.
Changing outline width based directly on density helps a little but it's just added "magic number" which needs to be tested.
Also sorry for different sizes and visible pixels caused by resizing to prepare all-in-one screenshot. Problem are overlaps inside the characters which are clearly visible and mostly on Samsung device.
Used device - density
Desktop PC - 2.0
Samsung Tab S9 Ultra -1.875
Pixel 6 emulator - 3.5
iPhone 15 Pro Max - 3.0

@Composable
fun OutlinedText() {
    Box(
        modifier = Modifier.background(Color.White)
            .padding(8.dp)
    )
    {
        val textSize = 20.sp
        val textOutlineWidth: Float
        with(LocalDensity.current) {
            textOutlineWidth = 2f.toSp().value // 2f without conversion - same result
        }

        val liningStyle = TextStyle.Default.copy(
            drawStyle = Stroke(
                width = textOutlineWidth,
            ),
            fontSize = textSize,
        )

        val fillStyle = liningStyle.copy(drawStyle = Fill)
        val fillColor = Color.Red
        val liningColor = Color.Black

        Text(
            text = "Sample",
            color = fillColor,
            style = fillStyle,
        )

        Text(
            text = "Sample",
            color = liningColor,
            style = liningStyle,
        )
    }
}

I need stroke effect too, but it is impossible:
https://github.com/paulocoutinhox/dataxow/blob/main/src/main/kotlin/com/dataxow/ui/components/AutoSizeText.kt

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Updated rule about colon associativity to be clearer about other use cases.

@paulmiard

@paulmiard

@ThibaultKlein

:+1:

:+1:


[Feature] Bug explanation

I delevelled my Hisuian Decidueye from 37 to 27 and used candies to get it to 30 so it would learn bulk up but instead it learned aura sphere.
To Reproduce
Steps to reproduce the behavior:

Get Hisuian Decidueye
Delevel it below 30
Use rare candies to get level 30
Decidueye will learn aura sphere and not bulk up

Expected behavior
Hisuian Decidueye is meant to learn bulk up at level 30, not aura sphere. You can find this info on both the pixelmon wiki and the game itself.
Screenshots
Do not have
Username:
TheAzureKnight
Server name (all applicable):
Titan - NebulaMC
Logs:
Where needed, please find your logs with the method down below. Make sure to include the file, or better yet, use a site like https://pastebin.com/ to add it to the ticket without copy pasting all the contents of the file.
For the Minecraft Launcher : C:\Users\USER\AppData\Roaming.minecraft\logs
For the Technic Launcher: C:\Users\USER\AppData\Roaming.technic\modpacks\pixelmon-reforged\logs
For the ATLauncher: C:\Users\USER\AppData\Roaming\ATLauncher\Instances\PixelmonCraft\logs
For the Twitch Launcher: C:\Users\USER\Documents\Curse\Minecraft\Instances\PACK\logs
Additional context
I have been unable to teach Decidueye bulk up via tms, via levelling, and there is no move tutor and it's really annoying considering in solo worlds it learns bulk up just fine but specifically on this server it seems unable to learn the move. I think this may be a bug with bulk up in general but Im just some 16 year old so Im prolly wrong.

The base issue was fixed, and the de-leveler refunded!  Sorry for all the confusion, it was a very odd issue, but everything should be good now!  Thank you for your patience!
Closing!


VESPA-16236: endpoint as a dimension to search requests

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@arnej27959 is this the correct way to do it?

@tokle fyi


Passing a Payload to _setState

Is possible to pass a payload of an object to _setState function?
Use case
Passing the error to _setState("Error").
Error state patches the colours of the UI and the error message in a box.
Something like:
this.webSocket.onerror = error => this._setState('Error', error);

Hi, this is currently possible, but i see that it's not been documented properly (we will add it).
Let's follow up on your example. The first parameter in the  $enter hook of a state holds an object with information that identifies the statechange:
class SomeState extends this{
  $enter({ prevState, newState, sharedState}){ ... }
}

we're using Destructing assignment

You can provide a second argument to the setState function: https://github.com/WebPlatformForEmbedded/Lightning/blob/d46ae917c68bfd5547787a92c1a6b28d8d56ab34/src/application/StateMachine.mjs#L97 and this needs to be an array (that will be spread as paramaters to the $enter hook in your application:
this._setState("SomeState", [{a:1, b:2, message:"hi"}, "some string", 12]);

and inside your $enter hook:
$enter(args, ...payload){
  // payload is an array with the following elements:
  //  [{a:1, b:2, message:"hi"}, "some string", 12]
}


"aframe new --template default" seems to require yarn

I can't see any reference to yarn in the repo but when I try to use aframe new --template default, I get an error message about yarn not being installed... Sounds like you should mention this somewhere.
Also couldn't you use npm for this?
I wasn't actually intending to use the command above, but the instructions from here https://hacks.mozilla.org/2017/07/picasso-tower-360o-tour-with-a-frame/ also fail so I can't install the template for the 360-tour either. I guess you really mean it when you say it's not ready for use (although shame the blog doesn't give any hint of that... :cry: )
Anyway, lots of potential, I'll have to check back in when it's a bit more mature :smile:

We'll be removing yarn (dependency of lenra). PR #68 handles all that. Going to be getting that merged, and everything will be a lot more predictable from a developer's ergonomical POV. Thanks for your patience and being a guinea pig 💯 . I really appreciate it. 👍


nose wants to test vendored libraries

My app is based on appengine-flask-skeleton , which uses vendoring to place third-party libraries in lib. NoseGAE seems to search all those libraries for tests. They aren’t designed for nose-style test matching. What can I do to exclude them?
One example of this seems to be Flask’s make_test_environ_builder which seems to show up in the following run:
appengine-flask-skeleton nop$ nosetests --with-gae
EEE............EEEEEEEEEEEEEEEE................EE
======================================================================
ERROR: Creates a new test builder with some application defaults thrown in.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
TypeError: make_test_environ_builder() takes at least 1 argument (0 given)

I’ve attached the full log.
nosegae-on-skeleton.txt

That's an unfortunate side effect of the way app engine does vendoring now. You can use nose -I option to ignore via regex or install a plugin like nose-exclude to ignore lib completely. The latter is my personal preference.

Oh, nose-exclude works great. Thank you! At first reading I thought  nose -I should have gotten a bite at the whole path.
Now I just need to get JetBrains to fix nose plugins so I can stop monkeypatching a file inside PyCharm.app :–)

You and me both. I have to test a lot of my gae projects outside of pycharm now if they use buildout with a custom interpreter.


MySql: Building queries using limit clause

Hi,
How can I construct my queries with the statement builder to use the LIMIT clause?
E.g.
connection.Find<Entity>( st => st.Limit( 100 ) ); // select * from entity limit 100

I've been through the code and wiki, but I can't find a solution. What am I missing here?
Thanks.

Isn't there a Top method on the statement?

There is, but it cannot be used without OrderBy, which is not the case in MySql.

It is the case with any SQL engine out there, including MySql, even if it doesn't blow up on execution. The order of the queried records is undefined without an ORDER BY. You MUST order by something.

I'm indifferent about the order of the returned records in this case.
Using just the LIMIT clause is syntactically correct as far as the manual goes and IMO a query builder should follow that.

That's a fair point. I'll look into turning the validation off.


There is a phenomenon where the content of notion gets cut off when it gets longer.

There is a phenomenon where the content of notion gets cut off when it gets longer.

I also experienced this, I think it's the same issue described in #46 and #41.
notion-client of NotionX seems to have fixed that issue. https://github.com/NotionX/react-notion-x/blob/master/packages/notion-client/src/notion-api.ts#L71

This just got published!


Problems with special German letters

Hi,
I have some problems with reading letters like Ä, Ü, Ö,... from a JSON file.
In my JSON file there is (besides a lot of other data):
{     "Szene_3": [         {             [...]             "Header": {                 [...]                 "Text": "Bücher"             }         }     ] }
When I read this (jsonFile being of type json and i being an int)
std::ifstream inStream("config.json");
nlohmann::json jsonFile;
inStream >> jsonFile;
[...]
for (int i = 0; i < size; i++) {
    std::string header = jsonFile["Szene_3"][i]["Header"]["Text"];
    [...]
}

I get "BÃ¼cher" in the string. When I write this exact string back into my json File it becomes "Bücher" again, yet I need to display the correct string in my application.
Can please somebody help me with this?

In order to asses your issue, we need the following information:

What is the issue you have?

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

What is the expected behavior?

And what is the actual behavior instead?

Which compiler and operating system are you using? Is it a supported compiler?

Did you use a released version of the library or the version from the develop branch?

If you experience a compilation error: can you compile and run the unit tests?

(Just a note: the library only supports UTF-8.)


[bug]: dependabot isn't updating pyproject file

Steps to reproduce
The PRs being opened by dependabot only change the poetry lock file, not the pyproject file.
Expected result
Both the lock and pyproject files should be updated.
Actual result (the bug)
But only the lock file is updated.
Optional: Relevant log output
No response
Severity/Priority
2: Medium

Seems this is an issue with dependabot: https://github.com/dependabot/dependabot-core/issues/8603 So will keep this open for now but we can't do anything about it yet.


Fix Modrinth badges

v1 of Modrinth's API has been deprecated since January 2022 and is scheduled to begin breaking in February, with a complete breakage being done by March. This pull request has been created as a courtesy to update it to use the dedicated shields.io badge rather than manually constructing the API request.
After merging this PR, please also update any usages on other pages, for example a CurseForge and/or Modrinth project description, and also cherry-pick/copy these changes to any other branches you might have.

@ENDERZOMBI102 Please make sure you merge this before the March 14th deadline.


updated models with password, brcrypt and not saving whole credit car…

…d numbers
UPdated both seeds and models.  You will have to npm run seed to update the database

also added the value of type so that when the search is looking for a category they can search for the type from the entries in the database.  Types are: Keyboards, Keycaps, Switches


[eventgrid] Update msrest to honor NULL sentinel

update ms rest after https://github.com/Azure/msrest-for-python/issues/236 is done
Unskip any tests impacted

As EG don't ship models anymore, not needed anymore


Erro on link libStrand.so

On build exemple show the erro:
Linking .build/debug/libStrand.so
/usr/bin/ld: /tmp/vapor-example/.build/debug/Strand.build/Strand.swift.o: relocation R_X86_64_PC32 against protected symbol `TWPO6Strand11StrandErrors13ErrorProtocolS' can not be used when making a shared object
/usr/bin/ld: final link failed: Bad value
clang-3.7: error: linker command failed with exit code 1 (use -v to see invocation)
:0: error: link command failed with exit code 1 (use -v to see invocation)
:0: error: build had 1 command failures
error: exit(1): /usr/bin/swift-build-tool -f /tmp/vapor-example/.build/debug.yaml default

What version of Swift are you using? If you are using Swift 3.0, which snapshot?

wift version 3.0-dev (LLVM 699a786c15, Clang 77080f2c03, Swift d22638766e)
Target: x86_64-unknown-linux-gnu
DEVELOPMENT.SNAPSHOT.2016.03.16.a-2

It is relevant: https://bugs.swift.org/browse/SR-1023 ???

Vapor is only compatible with 03-24 snapshot. I'd recommend you use https://github.com/kylef/swiftenv to manage Swift installations since that can be a source of problems.

I update and the erro continue
/tmp/vapor-example>$ swift build
Compiling Swift Module 'InterchangeData' (1 sources)
Linking .build/debug/libStrand.so
Linking .build/debug/libHummingbird.so
/usr/bin/ld: /tmp/vapor-example/.build/debug/Strand.build/Strand.swift.o: relocation R_X86_64_PC32 against protected symbol _TWPO6Strand11StrandErrors13ErrorProtocolS_' can not be used when making a shared object /usr/bin/ld: final link failed: Bad value clang-3.7: error: linker command failed with exit code 1 (use -v to see invocation) <unknown>:0: error: link command failed with exit code 1 (use -v to see invocation) /usr/bin/ld: /tmp/vapor-example/.build/debug/Hummingbird.build/Socket.swift.o: relocation R_X86_64_PC32 against protected symbol _Tv11Hummingbird10sockStreamVs5Int32' can not be used when making a shared object
/usr/bin/ld: final link failed: Bad value
clang-3.7: error: linker command failed with exit code 1 (use -v to see invocation)
:0: error: link command failed with exit code 1 (use -v to see invocation)
:0: error: build had 2 command failures
error: exit(1): /usr/bin/swift-build-tool -f /tmp/vapor-example/.build/debug.yaml default
$ swift -version
Swift version 3.0-dev (LLVM b010debd0e, Clang 3e4d01d89b, Swift 7182c58cb2)
Target: x86_64-unknown-linux-gnu
using: swift-DEVELOPMENT-SNAPSHOT-2016-03-24-a-ubuntu15.10.tar.gz
in arch linux 64 bits

My guess there is that Arch Linux is not yet supported by swift build. Apple only officially supports OS X, Ubuntu 14.04, and Ubuntu 15.10.

Closing this for inactivity. Let me know if you need more help with this issue.


Added missing attributes to generic class based views

This lets us pass them as arguments to .as_view()

Merged in df26c8de807ad4780732df065a4ba61cd2a321a3


Auto-save

Also, make it really obvious when you haven't saved yet.

Actually, we think you should just be prompted for a project name when you click New, so it's impossible to have an unsaved project.  That plus auto-save would solve the problem completely.

(Following up) Actually, the idea of never having a project that's unsaved is impossible.  You don't need to even be logged in to use CodeWorld.  You can also open other people's CodeWorld projects by sharing a URL with a hash, and whether or not you are logged in by cookies, we don't want to add an interstitial phase to prompt for a project name, nor automatically add the shared code to your account.
So here's the new proposal.

If you have an existing project open, it auto-saves.
If the code isn't an existing project, and the buffer is dirty, add an obvious warning sign.

Whether you're prompted to save a newly created project when you're logged in is an open question.  But you should be able to say "no".  I actually routinely open CodeWorld to hack at some quick idea that I don't want to save.  I've done it hundreds of times, and I don't want a hundred one-off test projects in my menu every time I log in.

I guess we can store something like buffers in local storage. We have some empty space down from project buttons.

Or better to call it drafts. This way we can have current buffer, which will save on switching to another buffer. It will silently update per time. It will not contain directories.
If one buffer going to be saved as full project with name - it will be deleted from drafts.
Maybe it is over complicated for users, because it yet another project type, one more UI thing and one more place for mess. Also not logged in users can use it as light replace of full projects tool.

I agree that this is too complicated for users.


RKE loading wrong ingress template during cluster provisioning

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

In Rancher UI, navigate to settings
Scroll down to the rke-metadata-config section
Modify the config to use the following:

{
  "refresh-interval-minutes": "1",
  "url": "https://raw.githubusercontent.com/ryansann/kontainer-driver-metadata/fa58036b8a5c7a2496444d4ac93e1983d86a0d91/data/data.json"
}

Add a cluster with RKE (I am using Digital Ocean though this isn't specific to a cloud provider)
Choose your node template
In Kubernetes Options, for the Kubernetes Version, choose: v1.16.8-rancher1-4
Create Cluster

Result:
The ingress daemon set is provisioned using the wrong template
Other details that may be helpful:
I have added a new ingress template for an updated ingress container, (see data.json). The new template is expected to be used based on the configuration in the rke-metadata-config, but a different template is being used.
The way you can distinguish that the correct template is being used is by running the following command: kubectl get configmap rke-ingress-controller -n kube-system -o yaml | grep runAsUser against the cluster that RKE provisioned. The output should show a value of 101 for the runAsUser field. If the value is 33, the wrong template is being used.
Per @kinarashah "it's a regression in 2.4 when kdm logic went through a refactor"
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.4.2
Installation option (single install/HA): single install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Infrastructure Provider
Machine type (cloud/VM/metal) and specifications (CPU/memory): Cloud (Digital Ocean Droplet) - 2 vCPUs / 4GB RAM
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:16:51Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.8", GitCommit:"ec6eb119b81be488b030e849b9e64fda4caaf33c", GitTreeState:"clean", BuildDate:"2020-03-12T20:52:22Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        afacb8b7f0
 Built:             Wed Mar 11 01:25:46 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       afacb8b7f0
  Built:            Wed Mar 11 01:24:19 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Related to https://github.com/rancher/rancher/issues/27350

Tested using a custom rancher image where it was not vendored in

When pointing to the correct KDM branch (dev-v2.4 in this case), all new kubernetes versions are available and come up successfully (v1.17.6-rancher2-1, v1.16.10-rancher2-1, 1.15.12-rancher2-2)
nginx-ingress-controller comes up successfully using image rancher/nginx-ingress-controller:nginx-0.32.0-rancher1
All other system images come up successfully and the cluster is operational


Collab Note Book ?

anyone know how to convert this using colab, i need Japanese tts voices

Here is the colab I used to convert MeloTTS English+Chinese model to sherpa-onnx
https://colab.research.google.com/drive/1XsKyAXti1e6_qYiJ3Fiyt8E7d1lPch75?usp=sharing


#3948 Log the current deliverable when publishing DITA-OT project

Description
I added an info log with the current deliverable description.
The log message has the following format: 'Publishing deliverable: "deliverable.id/deliverable.name" (x of y)'
Motivation and Context
This is a fix for the following issue:  #3948

Our original intention was to prefer to display the deliverable ID in the console output but I think it makes sense to display also the deliverable human readable name if present.
How about something like?
Publishing deliverable 1 of 3: Product A Online Help [del-olh-A-map]

so present the deliverable name if present and present also the ID in brackets.
For the set of errors which occur when a certain deliverable is published, we plan to present in Oxygen in the Problems view the error prefixed with the deliverable ID, something like:
 [del-olh-A-map][DOTZZZ] Class value for conref is ....

Thank you for your feedback.
The initial implementation prefers to present the deliverable ID in that message. The deliverable name was presented only when the ID is not present. This I wanted to represent with the / character in my first message.
I agree that is better to present both of them in the message.
I close this pull request and I will create another one that will show the message like this one:
Publishing deliverable 1 of 3: Product A Online Help [del-olh-A-map]

Thanks. It was easier for me to create a new pull request: https://github.com/dita-ot/dita-ot/pull/4100


Make HTTP2StreamChannel non-generic

Motivation:
As part of the work done for #214 we made HTTP2StreamChannel generic
over the type of data it reads and writes. Unfortunately this cost us a
non-trivial amount of performance.
Modifications:

Remove the genericism from 'HTTP2StreamChannel' by adding a mode for the
stream data type the channel is operating on (frames or frame
payloads), wrapping and unwrapping the data where necessary.
Simplify the implementation of 'MultiplexerAbstractChannel'
Remove 'HTTP2FrameConvertible' since it is no longer required

Result:
A perf win, hopefully.

@swift-nio-bot test this please


[BUG] Clone doesn't seems to be work as an action

Describe the bug
<CloneButton {...uiDefaults.buttonProps} recordItemId={props.id} />
Added a clonebutton but clicking it returns 404
Steps To Reproduce
Added a clonebutton but clicking it returns 404
Expected behavior
Should clone like edit but creates a new item when save
Packages

@refinedev+react-router-v6 version 4.5.5

Additional Context
I think the issue is in here.
https://github.com/refinedev/refine/blob/master/packages/react-router-v6/src/create-resource-routes.tsx#L59
Add a "clone" to this array....
(["list", "show", "edit", "create"] as const).forEach((action) => {
And removing this......
if (action === "create") {
actions.push({
action: "clone",
element: element,
path,
});
}

Hey @yeochinyi did you add clone path to your resources and correct route?

I didn't specifed any custom routing and opted for resource routing for all 'show,create,edit,delete, clone'.
i.e.
const resources = [
{
name: "coaches",
list: CoachList,
create: CoachCreate,
edit: CoachEdit,
clone: CoachClone,
show: CoachShow,
meta: {
canDelete: true,
icon: ,
},
},
The default routing routes to this URL
"https:///coaches/clone/1" which cannot find the React "CoachClone" fn.
All routing works as expected except "clone"
Any docs on how to resolve the issue ? the clone docs is a bit lacking..

Hello @yeochinyi, are you using Refine@v3? If you provide us npm run refine whoami it would be awesome.

   pnpm run refine whoami

<EMAIL_ADDRESS>refine /Users/christopheryeo/workspace/run/procoach/procoach-refine
refine "whoami"

System:

OS: macOS 14.2.1
CPU: (10) arm64 Apple M2 Pro

Binaries:

Node: 16.20.2 - ~/.nvm/versions/node/v16.20.2/bin/node
Yarn: Not Found
npm: 8.19.4 - ~/.nvm/versions/node/v16.20.2/bin/npm

Browsers:

Chrome: 121.0.6167.184
Safari: 17.2.1

Refine Packages:

@refinedev/antd: 5.37.4
@refinedev/cli: 2.16.24
@refinedev/core: 4.47.1
@refinedev/devtools-internal: 1.1.5
@refinedev/devtools-server: 1.1.24
@refinedev/devtools-shared: 1.1.3
@refinedev/devtools-ui: 1.1.17
@refinedev/devtools: 1.1.32
@refinedev/inferencer: 4.5.20
@refinedev/kbar: 1.3.6
@refinedev/nestjs-query: 1.1.1
@refinedev/react-router-v6: 4.5.5
@refinedev/supabase: 5.7.6
@refinedev/ui-types: 1.22.4

@yeochinyi I couldn't reproduce the issue, you can find an example here
Did you add your "clone" page to your routes? If you provide reproducible example, I will happy to debug


Enable providing custom SSL checkpoint

Models that use pre-trained SSL currently rely on hardocded path in run_ids.py, but these runs are only available in one Workspace. The user should be able to provide their own SSL checkpoint

Closed in #642 and #562


Add Law service points to tsv

Add Law testing service points to tsv file: Law - Book Cabinet” and Law - Tech Services


Rails 5 and Authem

I installed Authem on a Rails 5 beta application.
It installed version 1.5. bundle update didn't upgrade it to 2.0 either. Is this due to Authem's dependencies? Am I retarded? Please advise.

When I explicitly require Authem 2.0.1:
Bundler could not find compatible versions for gem "railties":
  In Gemfile:
    authem (~> 2.0.1) was resolved to 2.0.1, which depends on
      railties (~> 4.0)

    coffee-rails (~> 4.1.0) was resolved to 4.1.1, which depends on
      railties (< 5.1.x, >= 4.0.0)

    jquery-rails was resolved to 4.1.0, which depends on
      railties (>= 4.2.0)

    rails (< 5.1, >= 5.0.0.beta3) was resolved to 5.0.0.beta3, which depends on
      railties (= 5.0.0.beta3)

    rspec-rails was resolved to 3.1.0, which depends on
      railties (>= 3.0)

    sass-rails (~> 5.0) was resolved to 5.0.4, which depends on
      railties (< 5.0, >= 4.0.0)

    web-console (~> 3.0) was resolved to 3.1.1, which depends on
      railties (>= 4.2)

Point at master and try it now. Should work for you.

Thanks Paul!
On Wednesday, March 9, 2016, Paul Elliott<EMAIL_ADDRESS>wrote:

Point at master and try it now. Should work for you.
—
Reply to this email directly or view it on GitHub
https://github.com/paulelliott/authem/issues/28#issuecomment-194604285.

If there are no other rails 5 issues let me know and I'll cut a release.

I installed Authem successfully on a Rails 5.0.0.beta3 application @ https://github.com/plicjo/rails_5_authem
I created a user and reset their password successfully in the console.

Awesome. I cut version 2.1.0 with these changes. Thanks!


feat: Bepu, Allow user to set thread count

PR Details
See Title
Types of changes

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

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] I have built and run the editor to try this change out.

Thanks !


When using blank? no need to check empty or ==''

Excerpt from ruby docs: An object is blank if it's false, empty, or a whitespace string.
If it is blank, the second part of condition won't be evaluated. It it is not blank, the second part will be falsey.
@miq-bot add_label technical debt, ui, euwe/no
@miq-bot assign @mzazrivec

@isimluk The changes look good, just fix the rubocop comments please.

@isimluk looks good, fix the space cop complaint and we can merge  👍


Update README

Our repository README has practically no information at all and needs urgent work.

@EdgSgroi recommended a great repo as an example to follow.


[qt5-base] Move 'gui' and 'icu' out of 'core' feature

What does your PR fix?
This PR creates separate gui and icu features, reducing the number of dependencies and the build time for the core feature.

Adds an icu opt-in feature, reducing build time and deployment size of the default configuration.
(It might make sense to leave it as a default feature to avoid a change for currents users.)
Adds a gui opt-out feature to qt5-base and qt5-tools, meant to benefit cross builds which have a host dependency on qt5-base[core] or qt5-tools[core].
(This PR is separated out of work to fix building qt5 for mingw, but android and other triplets might benefit as well.)
Adds a quick opt-out feature to qt5-declarative, meant to benefit users which only need Qt Widgets, not Qt Quick.

In addition, there is some minor cleanup.

Which triplets are supported/not supported? Have you updated the CI baseline?
all, no

Does your PR follow the maintainer guide?
yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
yes

This should be beneficial for cross builds which have a host dependency on qt5-base[core].

... if vcpkg wouldn't install default features for host dependencies: https://github.com/microsoft/vcpkg-tool/pull/177.

cc @Neumann-A

It is basically good enough to review, although downstream ports will need changes to fully leverage qt5-base feature control.
In any case, this will be the base for the mingw (or more general: cross-build) changes which are another PR.

I think you should add a widgets feature to qt5-base to control the remaining downstream ports. If you want to then control the tools in qt5-tools do that in a separate PR only controlling the tools and nothing more.
So what I would like to see is only changes to qt5-base here.

I could revise the focus of the PR to qt5-base (widgets, icu) and add more of the pkgconfig and maintainability changes.
Then I would follow with a PR fixing mingw builds at least for qt5-base, qt5-imageformats.
And then other PRs could gradually improve the remaining qt5 ecosystem.

I think you should add a widgets feature to qt5-base to control the remaining downstream ports.

I just have to avoid an installation order problem. Some port have only transitive dependencies on qt5-base now. In the future, they would need a direct dependency on the gate, qt5-base[core,widgets].

I just have to avoid an installation order problem. Some port have only transitive dependencies on qt5-base now. In the future, they would need a direct dependency on the gate, qt5-base[core,widgets].

So long you make it the default-feature until downstream qt5 ports do it correctly it doesn't matter. Fixing that still can be done later.

Please temporary close this PR if you don't have spare time to finish it.

Touching this PR again:
It is not enough to control gui or widgets only in qt5-base: This doesn't cut port dependencies of other qt5 modules.

To avoid transitively requiring these dependencies for the host, qt5-tools really needs the gui feature.
For users who need Qt Widgets also for Qt Tools, but don't need Qt Quick, the quick feature of qt5-declarative is a necessary and sufficient control. However, this should be a default feature.

Requires review, not author-response.

If I gate everything only by the qt5-base[gui] feature, it means that for example vcpkg install qt5-svg will be empty and not install Qt5 SVG if only qt5-base[core] is installed. Okay?

If I gate everything only by the qt5-base[gui] feature, it means that for example vcpkg install qt5-svg will be empty and not install Qt5 SVG if only qt5-base[core] is installed. Okay?

I would like to keep the feature design as proposed in this PR, with qt5-base[gui], qt5-declarative[quick] and qt5-tools[gui].
Gating only on qt5-base[gui] doesn't allow to cut dependencies from depending ports (e.g. qt5-imageformats).
If there are concerns about how the feature selection is passed into each port in absence of direct options, I could still add patches for these options.

I think you should make agreement with @Neumann-A first.

I think you should make agreement with @Neumann-A first, sorry for forgot this PR.

Waiting for signal from @Neumann-A.

Waiting for signal from @Neumann-A.

I still don't get why you would want to separate qt5-base gui from qt5-tools gui. I mean for qml/quick stuff that makes sense because it drags in the dependency of qt5-declarative but if qt5-base is already build with gui why do you require qt5-tools to be not build wit gui? For me this is just a step into feature hell.

If I gate everything only by the qt5-base[gui] feature, it means that for example vcpkg install qt5-svg will be empty and not install Qt5 SVG if only qt5-base[core] is installed. Okay?

This means qt5-svg is dependent on the gui feature and need to add it as a dependency.

Gating only on qt5-base[gui] doesn't allow to cut dependencies from depending ports (e.g. qt5-imageformats).

What dependencies are you trying to cut? (i don't think it is qt5-imageformats which probably requires the gui feature?)

Any conclusion here?

Any conclusion here?

I feel that some requested changes to the current state are not really acceptable or sustainable.

Controlling features in qt5-tools by presence of features in qt5-base.
Controlling features in qt5-declarative by properties which correlate but semantically mean a different thing.

I'd appreciate confirmation in one or another direction.

I'd appreciate confirmation in one or another direction.

Ping.

I think we might need to confirm these points with @Neumann-A .

Any conclusion here?

I feel that some requested changes to the current state are not really acceptable or sustainable.
* Controlling features in `qt5-tools` by presence of features in `qt5-base`.

* Controlling features in `qt5-declarative` by properties which correlate but semantically mean a different thing.

I'd appreciate confirmation in one or another direction.

Ping.

Controlling features in qt5-tools by presence of features in qt5-base.

 I don't see a reason to build qt5-base with gui and qt5-tools without it.
    vcpkg_list(APPEND OPTIONS
        "QT.quick.name="
        "config.qttools.features.assistant.disable=true"
        "config.qttools.features.designer.disable=true"
        "config.qttools.features.distancefieldgenerator.disable=true"
        "config.qttools.features.pixeltool.disable=true"
        "config.qttools.features.qtdiag.disable=true"

Ah maybe I am just annoyed by the name of the feature because it actually doesn't use "QT.gui.name=" (but that is similar to what i said in https://github.com/microsoft/vcpkg/pull/23668#discussion_r856060692; maybe just go with the qtdeclarative approach and spin out the tools with gui as gui-tools instead and the cmd list tools just as single feature).
Or just follow https://code.qt.io/cgit/qt/qttools.git/tree/configure.json?h=v5.15.7-lts-lgpl and we don't have to discuss at all. (even if it adds 16 features where probably 3-5 are platform dependent and doesn't need to be features explicitly.)

Controlling features in qt5-tools by presence of features in qt5-base.

I take the approval of #29153 as an indication that we don't want that.

There: control qttools dbus by qtbase dbus presence.
Here: control qt5-tools gui by qt5-base gui presence.

I admit that gui is a cross-cutting concern, and dbusviewer isn't.

I take the approval of https://github.com/microsoft/vcpkg/pull/29153 as an indication that we don't want that.

@dg0yt

Or just follow https://code.qt.io/cgit/qt/qttools.git/tree/configure.json?h=v5.15.7-lts-lgpl and we don't have to discuss at all. (even if it adds 16 features where probably 3-5 are platform dependent and doesn't need to be features explicitly.)

https://github.com/microsoft/vcpkg/pull/29153 is exactly that. upstream feature == vcpkg feature

Closing this PR since it seems that no progress is being made. Please ping us to reopen if work is still being done.


Change Parcel Id to Parcel ID

we should have the info panel display Parcel ID and not Parcel Id

We call it parcel number in upper search. Should it be number instead or should we change number also?

Good question. Some counties have requested that a specific field of theirs be used to populate Parcel ID. So the values stored in each county for the field Parcel ID are actually a mish mash of account numbers, serial numbers, Parcel IDs, etc. I wonder how many queries are done for the parcel numbers in the app as the unique identifier for a parcel could be different depending on the users information. Here is an example:

I'm thinking to be consistent and generic let's have both be Parcel Number.

I wonder how many queries are done for the parcel numbers in the app

There have been ~3k parcel number searchers in the last month

we'll be famous


Bundle headers and pkgconfig file

This PR fixes the fact that the headers and the pkgconfig file weren't being distributed.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[CT-1014] [CT-800] [Feature] Apache Spark configurations ++options

Is this your first time opening an issue?

[X] I have read the expectations for open source contributors

Describe the Feature
I want to be able to do the following in a dbt sources.yml:

version: 2
sources:
- name: sources_csv
  schema: csv
  quoting:
     identifier: true
  tables:
     - name: dims_csv
       identifier: "/mnt/landing/dims_csv"
       options:
             sep: ','
             header: 'true'
             inferSchema: 'true'
             multiLine: 'true'
             escape: """

Describe alternatives you've considered
Tried the following options:

dbt_external_tables

https://hub.getdbt.com/dbt-labs/dbt_external_tables/0.3.2/
https://github.com/dbt-labs/dbt-external-tables/blob/main/sample_sources/spark.yml

This didn't work, but even if I get it to work, it is just making unnecessary complications to what can be achieved easily by the above suggestion.

Error:

$ dbt run-operation stage_external_sources --vars "ext_full_refresh: true"
16:07:52  Running with dbt=1.1.0
16:07:52  Unable to do partial parsing because config vars, config profile, or config target have changed
16:07:53  [WARNING]: Configuration paths exist in your dbt_project.yml file which do not apply to any resources.
There are 1 unused configuration paths:
- models.dbt_project.extractions

16:07:53  No external sources selected

sql_header

https://docs.getdbt.com/reference/resource-configs/sql_header

This didn't work, But even it if it worked it is a not needed work around.

Code:

-- Supply a SQL header:
{% call set_sql_header(config) %}
    create table if not exists dims_csv
    using csv
    options ('header' = 'true')
    location "/mnt/landing/dims_csv";
{%- endcall %}

-- Supply your model code:

SELECT
           *
FROM dims_csv

Error:

dbt run -s csv_to_parquet_cp
16:13:28  Running with dbt=1.1.0
16:13:29  Unable to do partial parsing because config vars, config profile, or config target have changed
16:13:29  [WARNING]: Configuration paths exist in your dbt_project.yml file which do not apply to any resources.
There are 1 unused configuration paths:
- models.dbt_project.extractions

16:13:30  Found 4 models, 3 tests, 0 snapshots, 0 analyses, 258 macros, 0 operations, 0 seed files, 7 sources, 0 exposures, 0 metrics
16:13:30
16:13:39  Concurrency: 2 threads (target='dev')
16:13:39
16:13:39  1 of 1 START table model dbt_schema_madelre_01.csv_to_parquet_cp ............... [RUN]
16:13:44  1 of 1 ERROR creating table model dbt_schema_madelre_01.csv_to_parquet_cp ...... [ERROR in 4.66s]
16:13:46
16:13:46  Finished running 1 table model in 16.61s.
16:13:46
16:13:46  Completed with 1 error and 0 warnings:
16:13:46
16:13:46  Runtime Error in model csv_to_parquet_cp (models/transformations/csv_to_parquet_cp.sql)
16:13:46    org.apache.hive.service.cli.HiveSQLException: Error running query: org.apache.spark.sql.AnalysisException: Table or view not found: dims_csv; line 23 pos 5;
16:13:46    'CreateTable `dbt_schema_madelre_01`.`csv_to_parquet_cp`, ErrorIfExists
16:13:46    +- 'Project [*]
16:13:46       +- 'UnresolvedRelation [dims_csv], [], false

Extra sql CT to rename csv columns and remove 1st row containing headers from data set

This worked, but it won't fix other options than header, e.g. sep, multiLine, escapeQuote
Who will this benefit?
Everyone using dbt to create data pipelines that require csv sources for transformation job.
Are you interested in contributing this feature?
yes
Anything else?
No response

We intend to re-open the conversation around supporting external tables at the source level, which would also allow us to revisit the extensibility of sources.
I'm thinking 1.6 for that, which is a couple of quarters away. So I'm going to mark this issue as help_wanted for now, if there's a short win to be had locally in the meantime.


fix branch for docs

Description
fix some docs branch name to main
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run python - communication - ci


ClusterInfra Assesment Incorrectly Reports Findings

What steps did you take and what happened:
We have Trivy Operator running in our EKS cluster and when it runs the clusterInfraAssessment report it comes back with checks that aren't valid.
It returns KCV0080, KCV0081, KCV0090 even though they are setup correctly on the cluster.
What did you expect to happen:
Those checks should not return as unsuccessful because they are implemented correctly
Anything else you would like to add:
I have attached the report as well as the config file from a worker node.
Environment:

Trivy-Operator version (use trivy-operator version): 0.16
Kubernetes version (use kubectl version): 1.24
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): running on the cluster
config.txt
clusterInfraAssessmentReport.txt

@schuman0 thanks for this input , can you please check please run node-collector on the cluster and let me know the output

attaching output from node-collector
output.txt

@schuman0 its look like something is preventing from node-collector to properly run as I see that out put is empty.
do you see any error in operator log ?

I can't find anything in the logs but at one point I did see that the node-collector jobs were listed as out of memory. Is there a parameter I can use to up the memory limits for node-collector? I don't see anything in the helm chart

I'll add support and cut an Rc release for it

I can't find anything in the logs but at one point I did see that the node-collector jobs were listed as out of memory. Is there a parameter I can use to up the memory limits for node-collector? I don't see anything in the helm chart

I'll add support and cut an Rc release for it

after trivy-operator v0.16.2 is releases you can use the same resources config for node collector

Okay great, I will try that

@chen-keinan I updated to 0.16.2 and added the resources lines to the node-collector config but when I look at the job on the cluster I still see the old resource config.

@schuman0 you'll need to use the same resource config as for trivy scan job

Okay, but I have that set much higher than the job on the node-collector config shows.

Okay, but I have that set much higher than the job on the node-collector config shows.

@schuman0 byt the image you put above I see that you set it on wrong helm params, trivy operator do not read the param from where you put it above it put it from the same param as [scan-job resources](resource config](https://github.com/aquasecurity/trivy-operator/blob/main/deploy/helm/values.yaml#L364) please set it up correctly and update me with results

Yes I had already done that:

@schuman0 its the node collector job still have OOM issue ?

I have not seen those recently but I still can't get the reports or node-collector to work no matter what I try.

are you talking on the same initial issue ClusterInfra Assesment  are still generated wrongly or not generated at all, or you experience other issue

that's correct. The reports are still generating the same. I also still can't get node collector, when done as above, to generate anything but a blank respose.


Support static per vhost

Right now, static files/folders (using app.static or bp.static) are served for all vhost.
I'd like Sanic to support static assets per vhost.

thanks for reporting it, feel free to open another issue if you see any problems

ah, darn i missed bp.static 🤕

Actually the previous PR works for blueprints too. Feel free to reopen this if you see any issues.


[CS2103T-W16-3] Fine$$e

Fine$$e allows tertiary students to better manage their finances, cultivating good financial habits such as saving. It is optimised for CLI users so that expenses and income can be tracked efficiently by typing in commands.

Codecov Report

Merging #83 into master will decrease coverage by 0.78%.
The diff coverage is 86.11%.

@@             Coverage Diff              @@
##             master      #83      +/-   ##
============================================
- Coverage     72.31%   71.52%   -0.79%
+ Complexity      400      387      -13
============================================
  Files            70       69       -1
  Lines          1228     1194      -34
  Branches        124      117       -7
============================================
- Hits            888      854      -34
- Misses          308      309       +1
+ Partials         32       31       -1

Impacted Files
Coverage Δ
Complexity Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

.../seedu/address/logic/parser/FindCommandParser.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

...ain/java/seedu/address/model/transaction/Name.java
80.00% <ø> (ø)
6.00 <0.00> (?)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...rc/main/java/seedu/address/ui/TransactionCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...in/java/seedu/address/ui/TransactionListPanel.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <10.00%> (ø)
1.00 <0.00> (ø)

...in/java/seedu/address/model/category/Category.java
80.00% <80.00%> (ø)
5.00 <5.00> (?)

...n/java/seedu/address/model/transaction/Amount.java
80.00% <80.00%> (ø)
6.00 <6.00> (?)

src/main/java/seedu/address/model/AddressBook.java
91.66% <92.30%> (ø)
12.00 <8.00> (ø)

... and 31 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 6389d85...1c5decb. Read the comment docs.

Codecov Report

Merging #83 into master will decrease coverage by 2.93%.
The diff coverage is 75.56%.

@@             Coverage Diff              @@
##             master      #83      +/-   ##
============================================
- Coverage     72.31%   69.37%   -2.94%
- Complexity      400      454      +54
============================================
  Files            70       82      +12
  Lines          1228     1525     +297
  Branches        124      143      +19
============================================
+ Hits            888     1058     +170
- Misses          308      428     +120
- Partials         32       39       +7

Impacted Files
Coverage Δ
Complexity Δ

...a/ay2021s1_cs2103_w16_3/finesse/AppParameters.java
75.00% <ø> (ø)
9.00 <0.00> (?)

.../main/java/ay2021s1_cs2103_w16_3/finesse/Main.java
0.00% <ø> (ø)
0.00 <0.00> (?)

...in/java/ay2021s1_cs2103_w16_3/finesse/MainApp.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...21s1_cs2103_w16_3/finesse/commons/core/Config.java
68.18% <ø> (ø)
7.00 <0.00> (?)

...cs2103_w16_3/finesse/commons/core/GuiSettings.java
69.23% <ø> (ø)
5.00 <0.00> (?)

..._cs2103_w16_3/finesse/commons/core/LogsCenter.java
78.37% <ø> (ø)
10.00 <0.00> (?)

...s1_cs2103_w16_3/finesse/commons/core/Messages.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...1s1_cs2103_w16_3/finesse/commons/core/Version.java
82.50% <ø> (ø)
19.00 <0.00> (?)

...cs2103_w16_3/finesse/commons/core/index/Index.java
100.00% <ø> (ø)
10.00 <0.00> (?)

...se/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)
1.00 <0.00> (?)

... and 115 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 6389d85...6a4047a. Read the comment docs.


Errors in definitions in VS2013

Hi,
I'm not sure if I'm being completely stupid or if I've done something wrong.
Whenever I add the angular js definitely typed definitions to my project via nuget, I get over 100 errors in the error prompt.
These generally consist of 'Identifier expected', 'Default arguments are only allowed in implementation',  'Build: = expected' etc. Do I have to update my TSLint (which is what I presume is causing the problems) or is there anything else I can do?
Thanks

TSLint hasn't been updated to support TypeScript 1.4 yet so union types and any other new language features will most likely cause errors. I had the same problem and had to turn off TSLint for now until a new release comes out.

how do you turn off TSLint? i thought it was part of web essentials. no?

@bigerock  TSLint is a nodejs based utility but is included in Web Essentials.
The settings to turn this on or off are under Tools > Options > Web Essentials > TypeScript > Linter

thanks! i turned off 'run on save' so now it's not running on save or build.
i still get the errors mentioned though. any thoughts?

i see - i updated to the latest angularjs definitelytyped and this is what broke it. i downgraded to a few versions previous and now it's not throwing those errors. hope this helps someone else.

Any fix?

Hi All,
I fixed my issue by installing this latest TypeScript update: https://visualstudiogallery.msdn.microsoft.com/2d42d8dc-e085-45eb-a30b-3f7d50d55304.
Fixed it for me and I no longer get any errors, so I'm going to close this off.


Godot 4 RC3 and RC4 Several builds and attempts to install when trying to remote debug in Android in Linux system

Godot version
Godot 4 RC3 and RC4
System information
Pop OS
Issue description
When selecting "Remote Debug" to Android device, in my case, to PICO 4 VR glasses, Godot builds and tries to install a lot of times. Finally the game installs and runs in the device. If you stop the several attemps with the "Esc" key in Godot window it runs as well.
In my system until Godot 4 RC2, included, and including all the betas or in any Godot 4 version in Windows, the same projects work fine, build, install and run in VR glasses at first attempt.
Steps to reproduce
Select "Remote debug" with any project in POP!_OS 22.04 LTS linux system.
Here a video showing the issue:
https://drive.google.com/file/d/1v3bTlYi3eg4VF18Ssc5wD-1e9cqqPbo7
I attach a image with the final message showed if you let it do all the attemps (the game works in the device after this message window).

Minimal reproduction project
N/A

Same issue as https://github.com/godotengine/godot/issues/73842, should be fixed by https://github.com/godotengine/godot/pull/73885.


Win 7?

Does this work with Win 7?
Thanks

It may not work if you install it through the installer, but it definitely works if you install it with cargo.
Let me know if you have more doubts, otherwise you can close this issue.

@harryray33 I suggest you to try downloading the installer from here.
I cannot guarantee it'll work on Windows 7 since I built it on Windows 10.
If it doesn't work, you can install Rust and then download the app with cargo install sniffnet.
In any case, don't forget to download the needed dependencies.
I'm closing the issue since I think you have now enough info. Feel free to reopen it if you have problems.

it works fine with .msi installer. I installed it via scoop...

@bt4ibwem8
Which OS did you have success in Installing SniffNet?

it works fine with .msi installer. I installed it via scoop...

@bt4ibwem8
Which OS did you have success in Installing SniffNet?

@kenrmayfield
I am running it on 64bit SP1 system. Maybe you can try to install https://github.com/doitsujin/dxvk and also try to install Sniffnet via scoop.

@bt4ibwem8
I did Install Scoop with Success.  I then Installed NPCAP via .MSI File with Success.  However upon Installing SniffNet, Scoop stated that NPCAP was not Installed.  I did try to Run SniffNet and it still Complains about the Same DLL's.
By the way when you Install Scoop,  which should you use:
irm get.scoop.sh | iex
iwr -useb get.scoop.sh | iex
Also what is the Differnece between irm and iwr in the Syntax?

I don't know how scoop performs the installation, but the fact that it says Npcap is missing is weird and can be a signal that the problem is there...

Thats just half the Problem......it still Complains about the DLL's.

@kenrmayfield
i installed it with irm get.scoop.sh | iex


Moved lineups to joined request

Description
The requests to get the players for a mach was split into separate requests, but could be improved by joining then in the match request.
fixes #108
Type of Change

[x] Delete splitted requests
[x] Added join in request query
[x] Small refactor

How has this been tested?

[x] Logged the response data
[x] Checked the frontend of the statsreader page if the players still got displayed correctly

Checklist:

[x] My code follows the styling guidelines of this project
[x] My changes don't create conflicts with the main branch

ziet er goed en netjes uit


MAP estimate

This PR implements the Maximum-a-posteriori (MAP) estimate for SNPE, SNLE, SNRE. It is obtained by gradient ascent from some starting locations (usually sampled from the posterior).
See #306 and #276 and #392
Naming
Some ideas for the function names:

posterior.mode()
posterior.map()
posterior.estimate_map()
posterior.map_estimate()

Of course, the last two also work with mode instead of map.
Why not use cma?

relatively high interrun-variability.
black-box algorithm, I have no idea what is happening there.
one more dependency
why not use gradients if we have them?

We could still revert to cma though, I will not squash the commit.
Why not implement the Maximum-likelihood-estimate?

for SNPE: we would obtain the likelihood by dividing the posterior by the prior. However, if the prior drops off faster than the posterior, the likelihood will explode, leading to highly unstable results.
for SNRE and SNLE: the likelihood estimate will be highly inaccurate in regions where there is not much data for theta. In these regions, the trained NN can do whatever it wants and it might just assign very high probabilities to some absurd regions (for sampling the posterior, this does not matter since the prior makes the posterior log-prob very small)

We could still revert to using it though, I will not squash the commit.
Dealing with bounded priors
If the prior is BoxUniform, we transform it into unbounded space and optimize there.

Codecov Report

Merging #412 (3cb122a) into main (ab42a3c) will increase coverage by 0.18%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   82.77%   82.96%   +0.18%
==========================================
  Files          50       50
  Lines        3187     3234      +47
==========================================
+ Hits         2638     2683      +45
- Misses        549      551       +2

Flag
Coverage Δ

unittests
82.96% <96.29%> (+0.18%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sbi/utils/__init__.py
100.00% <ø> (ø)

sbi/inference/posteriors/base_posterior.py
85.21% <94.44%> (+1.35%)
:arrow_up:

sbi/inference/posteriors/direct_posterior.py
96.84% <100.00%> (+0.06%)
:arrow_up:

...inference/posteriors/likelihood_based_posterior.py
98.03% <100.00%> (+0.08%)
:arrow_up:

sbi/inference/posteriors/ratio_based_posterior.py
97.10% <100.00%> (+0.08%)
:arrow_up:

sbi/utils/sbiutils.py
87.50% <100.00%> (+0.83%)
:arrow_up:

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 ab42a3c...3cb122a. Read the comment docs.

Codecov Report

Merging #412 (3cb122a) into main (ab42a3c) will increase coverage by 0.18%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   82.77%   82.96%   +0.18%
==========================================
  Files          50       50
  Lines        3187     3234      +47
==========================================
+ Hits         2638     2683      +45
- Misses        549      551       +2

Flag
Coverage Δ

unittests
82.96% <96.29%> (+0.18%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sbi/utils/__init__.py
100.00% <ø> (ø)

sbi/inference/posteriors/base_posterior.py
85.21% <94.44%> (+1.35%)
:arrow_up:

sbi/inference/posteriors/direct_posterior.py
96.84% <100.00%> (+0.06%)
:arrow_up:

...inference/posteriors/likelihood_based_posterior.py
98.03% <100.00%> (+0.08%)
:arrow_up:

sbi/inference/posteriors/ratio_based_posterior.py
97.10% <100.00%> (+0.08%)
:arrow_up:

sbi/utils/sbiutils.py
87.50% <100.00%> (+0.83%)
:arrow_up:

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 ab42a3c...3cb122a. Read the comment docs.


chore(Makefile): remove duplicate line

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Nice catch. This means the qq.Identify module is defined twice in the built code. Even though this is a small module and size will only decrease a small amount, I think that merits a hotfix release. Thanks!
Going forward, changes should be initially based off of develop. This is why I have an issue template, which you deleted. But I do thank you again for your help here.

Apologize for not following guidelines, I just saw that duplicate line while I was reading the code and went to submit a pull request using Github's UI. Will submit changes from develop next time 😸

This is going to go out today (likely) as part of 5.11.10. Thanks for your help!


Simplify configurable optimizer API

Types of changes

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

Motivation and Context / Related issue
The simplified process means:

writting an optimizer with extra parameters (more than just parametrization, budget, num_workers)
specifying a ConfiguredOptimizer class, with the appropriate init (the extra parameters) and the right base algorithm + nice docstrings since that will be visible in the doc
that's it, but if you want you can pre-create many variants using the configured optimizer class

How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[x] All tests passed, and additional code has been covered with new tests.

Agreed by @teytaud , merging and following with other PRs to propagage