fix(type): ScrollArea type

Fix interface for ScrollAreaScrollbarImplProps
Issue: #381

const scrollAreaScrollbarImplProps = {
We need to apply this in the props as well. Not only on the TS side. If there is also an Emits status, we need to apply the emits there in the same way.
In Vue, props is the sum of emit+props.

You're great, thanks to your hands. :heart:


TEST, DO NOT MERGE

Hmm, it appears that only the "Fork PR labeler" is doing the tagging job, not the "PR Labeler"


Loop and navigation stops working when slidesPerView is greater than the number of slides

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/swiper-infinite-loop-react-forked-kzp3hc
Bug description
The navigation and the loop features stop working when the slidesPerView number is greater than the number of slides. I manage to click the right arrow twice, but not more. The left navigation button works as expected.
Expected Behavior
When the loop property is true and the slidesPerView number is greater than the actual number of slides, the right button is clickable infinite times.
Actual Behavior
Let's say I have the intention to show 4 slides per view, and load the content dynamically into the swiper carousel. I can only load 3 slides, so the last slide will be my first slide duplicated. I click the left arrow 3 times and it loads the #3, #2, #1 slides correctly, but when I click the right navigation button 3 times, the navigation stops working after the second click.
Swiper version
8.3.2
Platform/Target and Browser Versions
Chrome 103.0.5060.134
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Yes, this happens in the core version too
https://codesandbox.io/s/swiper-default-forked-btikxj?file=/index.html

'loopedSlidesLimit' option should be 'false'
https://swiperjs.com/swiper-api#param-loopedSlidesLimit

Swiper v9 comes with fully reworked and now different loop functionality. If you have similar issues in Swiper 9, open a new issue with a CodeSandbox showing the issue.


New version: Google.Chrome.Dev.EXE version 129.0.6658.0

Created by 🥟 Dumplings in workflow run #7384.

Log
Updated: 129.0.6643.2 → 129.0.6658.0
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Possible duplicate package entry.
Similar installer SHA256 hash found in manifest:

InstallerSha256: 781538b1f0497f7bfbd81a29ae5f1550f6bd60229ff7078921df73c2ad749a1e

manifests\g\Google\Chrome\Canary\129.0.6658.0

InstallerSha256: 0f6841f726bc0648d561082b1aefce37d9c178e914460047160bdd7cfa411d96

manifests\g\Google\Chrome\Canary\129.0.6658.0

InstallerSha256: 8eb37098e27d7225fe2fc25200463cc8db56571e473613612d082de65a4af67d

manifests\g\Google\Chrome\Canary\129.0.6658.0

It appears that this Canary version became promoted to Dev.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_f981pgz_xd/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_f981pgz_xd/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_f981pgz_xd/clone/recipe/meta.yaml:27: missing_pip_check: For pypi packages, pip check should be present in the test commands
INFO: /tmp/abs_f981pgz_xd/clone/recipe/meta.yaml:30: invalid_url: http://fluentd.org : URL domain redirect fluentd.org ->  www.fluentd.org
ERROR: /tmp/abs_f981pgz_xd/clone/recipe/meta.yaml:37: http_url: http://fluentd.org is not https
WARNING: /tmp/abs_f981pgz_xd/clone/recipe/meta.yaml:37: missing_description: The recipe is missing a description
Errors were found


inconsistent handling "preserve-order" on suite/test level

for configuration
    <suite 	preserve-order="true">
	  <test preserve-order="false" >

effective XmlTest.getPreserveOrder() is false
for configuration
    <suite 	preserve-order="false">
	  <test preserve-order="true" >

effective XmlTest.getPreserveOrder() is also false, but should be true

:+1:
The issue is located here: https://github.com/cbeust/testng/blob/master/src/main/java/org/testng/xml/XmlTest.java#L704-L711
Could you propose a pull request that fixes the issue and some unit tests?

some other configuration to test:
for configuration
    <suite>
	  <test preserve-order="false" >

effective XmlTest.getPreserveOrder() must be false
for configuration
    <suite 	preserve-order="false">
	  <test>

effective XmlTest.getPreserveOrder() must be false

The problem is that API is ambigous. public String getPreserveOrder() may be interpreted as configured preserve order (null, false, true), or preserver order after default is applied (false, true), effective order after suite/test resolution (false, true) or any String, like "foo/bar", "flower power" etc.

As I know, child nodes always override their parents.
And it should be public boolean getPreserveOrder() as the default value is supposed to be used in case of null. Feel free to change it too.


#325 - dataTypes in exports

Why is it so messy? If I look at file changed, it shows also non-relevant changes

Testing corrections were necessary due to previous changes. But there are in the separate commit.
Files changed in the last commit are relevant. Just the DefaultResultToAnnotatedTableAdapter contains more changes, because when I had to change methods for creating relation columns, then I made other methods for creating columns more clear and lucid and also improved work with prefixes.

@josef-janousek OK, works for alogirthm assigned datatype properties.


Highlight selected/expanded row

This function solves the problem (这个功能解决的问题)
Hi there, so far this library is awesome!
I would like to know if there is a way to simply set a class to rows when they are selected or in expanded ? (the expanded row   the newly added row)
This is a very common use case where you might want to highlight a row for any reason. of course it makes the UX better when you can highlight what you want to focus on (similar to when you use stripes mode for a table, makes it more readable)
Thanks a lot
Shani
Expected API (期望的 API)
set a class to  expanded ´expanded’.
it would be great to control it with the theme config provider. for example add a these properties to the DataTable: "selectedRowColor", "expandedRowColor", "expandedRowContentColor"

Opened a PR adding a class to expanded rows: https://github.com/tusen-ai/naive-ui/pull/4426

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Fixes #620 by adding arguments to ParameterListWrappingRule.

Reference ticket #620  for details. This now considers anything that is VALUE_ARGUMENT in addition to VALUE_PARAMETER for the ParameterListWrappingRule. This means function calls will get the same treatment as class definitions for their arguments being on separate lines (if necessary).
Example
Before, ktlint considered this valid:
fun callTest() {
    test(
        "One",
        "Two", "Three", "Four"
    )
}

This is not clear, though, as multiple arguments on one line is confusing, if they're not all on one line. Now it will auto format them to be on separate lines:
fun callTest() {
    test(
        "One",
        "Two",
        "Three",
        "Four"
    )
}

One Concern
All of the tests pass, but the annotation one is a little confusing as for me it appears it wanted to format the annotation like this:
class A {
    fun f(
        // ...
        @Annotation(
            [
            "v1",
            "v2"
        ]
        )
        // ...
}

Some back and forth with @romtsn on the ticket right now though to see if that's an issue before we merge this.

hey @AdamMc331, are you planning to finish this up or should we take over?

Hey @romtsn Sorry for falling off on this again. I still have an issue formatting this if there's a argument list inside an annotation. I will clean up my git history and write a test to demonstrate the bug, but I'd definitely appreciate some help looking into it I'm at a loss.

I made it work, but there's another corner case, which we probably should consider and I'm looking into it now:
class NoVarRuleTest : Spek({
    // this should not fail
})

:eyes: Exciting! Thanks for your help pushing this across the finish line.

I am a bit hesitant to release this in the standard ruleset, as although the change is not significant code-wise, it kind of has a big impact even on ktlint's codebase.. maybe we should move it to the experimental ruleset for the next release? Thoughts @Tapchicoma @shashachu @AdamMc331? Otherwise, it's ready to be merged.

I'm open to the experimental ruleset but that could also be the imposter syndrome talking. 😂 This is my first contribution to a ktlint ruleset and so I'm definitely not as familiar as the others tagged but I will support their opinion either way.
If we go with the experimental ruleset, what is the process like for getting it moved out of experimental in the future?

Although it may aggravate some people, I don't think it's a big deal to leave this in the standard ruleset, as its functionality is more consistent with the IndentRule, which is quite aggressive about adding newlines.

@AdamMc331 does this PR also address https://github.com/pinterest/ktlint/issues/680?

@shashachu nope, that problem is specific to indentation rule

Thanks Adam!

Thanks everyone! :)

I’m seeing lots of results for ParameterListWrapping after upgrading to detekt 1.12.0 that look either questionable or wrong. Could it be related to this PR? A couple of examples:
Questionable. Fails with Argument should be on a separate line (unless all arguments can fit a single line) and Missing newline before ")" .
foo(bar.apply {
    // stuff
})

Wrong? Fails function(arg1, arg2, arg3) with Argument should be on a separate line (unless all arguments can fit a single line).
json(
    """
    {
        "array": [
            ${function(arg1, arg2, arg3)}
        ]
    }
    """.trimIndent()
)

The latter one looks pretty much like https://github.com/pinterest/ktlint/issues/859, the former one though should not be reported (at least the arguments part). Could you create a separate issue for the former one? Thanks

@adamu The latter one looks pretty much like #859, the former one though should not be reported (at least the arguments part). Could you create a separate issue for the former one? Thanks

Done! #861


add: leo carvalho

Description
Add my profile to the list.
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

@carvalholeo Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


Added a clear button with the shortcut Command + K to clear the terminal

Description
Added a new button at the bottom of the DebugAreaTerminalView for clearing the terminal, with the clear icon. The terminal can also be cleared with the command + K keyboard shortcut. The button also has a tooltip indicating that it is used for clearing the terminal
Related Issues
Closes #891
Closes #1143

[x] I read and understood the contributing guide as well as the code of conduct
[x] The issues this PR addresses are related to each other
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] My changes are all related to the related issue above
[x] I documented my code

Screenshots
https://github.com/CodeEditApp/CodeEdit/assets/38476845/27ab4e4b-70ea-4a9e-872e-7ad5d4721c28

@CodeEditApp/maintainers Do we want ⌘K for clear terminal, or do we want something more like Xcode that ⌘K cleans the build folders?

Personally, I want to reject both PRs for the clear terminal button. You can use the clear command for that. So then the shortcut becomes available for something else.
Clearing the build folder is a bit difficult with something like this. Because the build folder can be at any location depending on the language or framework used.

@Cyanosite I am going to close this for now till we decide what exactly we would like.


GEODE-9168: Remove radish function execution

Removed CommandFunction, SingleResultRedisFunction and all
*FunctionInvoker classes.
Move necessary functionality out of CommandHelper and into
RegionProvider.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

@upthewaterspout Initially, I thought it was redundant, but have restored it and reworked it somewhat. It showed up a problem with future MOVED work that is based on this PR. So thanks for that prompt!


Rewrite ADF linked services to Synapse linked services

TBD

In the most recent version of the Hitachi Vantara synapse, I rewrote the linked service to be a key vault reference.
It makes the CD slightly easier since we don't need to override this linked service (only the KV one)
Do we want to do that for this sample too?
e.g.
{
	"name": "pipelineADLS",
	"type": "Microsoft.Synapse/workspaces/linkedservices",
	"properties": {
		"type": "AzureBlobFS",
		"annotations": [],
		"typeProperties": {
			"url": {
				"type": "AzureKeyVaultSecret",
				"store": {
					"referenceName": "keyvault",
					"type": "LinkedServiceReference"
				},
				"secretName": "Synapse-PipelineAdls-Url"
			}
		},
		"connectVia": {
			"referenceName": "AutoResolveIntegrationRuntime",
			"type": "IntegrationRuntimeReference"
		}
	}
}


Reset-SqlAdmin - Script Analyzer Test failure

We're adding Pester tests and AppVeyor to dbatools, see https://github.com/sqlcollaborative/dbatools/wiki/AppVeyor-and-Pester for more detail.
The following rules were highlighted for this function:

RuleName
Severity
Line
Message

PSAvoidUsingCmdletAliases
Warning
211
'Select' is an alias of 'Select-Object'. Alias can introduce possible problems and make scripts hard to maintain. Please consider changing alias to its full content.

PSAvoidUsingWMICmdlet
Warning
221
File 'Reset-SqlAdmin.ps1' uses WMI cmdlet. For PowerShell 3.0 and above, use CIM cmdlet which perform the same tasks as the WMI cmdlets. The CIM cmdlets comply with WS-Management (WSMan) standards and with the Common Information Model (CIM) standard, which enables the cmdlets to use the same techniques to manage Windows computers and those running other operating systems.

PSPossibleIncorrectComparisonWithNull
Warning
222
$null should be on the left side of equality comparisons.

PSUseDeclaredVarsMoreThanAssignments
Warning
260
The variable 'sid' is assigned but never used.

PSPossibleIncorrectComparisonWithNull
Warning
277
$null should be on the left side of equality comparisons.

PSPossibleIncorrectComparisonWithNull
Warning
298
$null should be on the left side of equality comparisons.

PSPossibleIncorrectComparisonWithNull
Warning
316
$null should be on the left side of equality comparisons.

PSAvoidUsingWMICmdlet
Warning
318
File 'Reset-SqlAdmin.ps1' uses WMI cmdlet. For PowerShell 3.0 and above, use CIM cmdlet which perform the same tasks as the WMI cmdlets. The CIM cmdlets comply with WS-Management (WSMan) standards and with the Common Information Model (CIM) standard, which enables the cmdlets to use the same techniques to manage Windows computers and those running other operating systems.

see https://github.com/sqlcollaborative/dbatools/issues/627 , aka "the issue that tracked them all"


PVGeo Paraview integration on Windows

Dear all,
I am trying to use Paraview for my research visualization, at the same time I am using python (Pycharm IDE) to write my code.
I have followed the guide (https://pvgeo.org/overview/getting-started.html) but encountered an issue when loading the plugins through ParaView’s Plugin Manager:

The second screenshot is in Norwegian, but it basically says that there is something wrong with the file.
Thank you very much in advance.

Hi @Geoscientist1, I haven't had the availability to maintain support for ParaView here -- I suspect the plugin infrastructure in ParaView has changed significantly and these will not work. For the past year or so, I've been telling people that I can no longer support ParaView here as its has maybe one or two people who'd actually use it. I am still supporting these tools to be used with PyVista though. PyVista is akin to ParaView, just all in Python.
https://docs.pyvista.org/version/stable/

All of the "Quick Examples" show how to use PVGeo with ParaView: https://pvgeo.org/examples/index.html

Thank you very much for the quick reply!

I have dropped support for ParaView, please use PyVista instead: docs.pyvista.org


KeyError exception when entering waypoint that isn't home

Home Assistant release (hass --version):
0.18.2
Python release (python3 --version):
Python 3.4.2
Component/platform:
device_tracker.owntracks
Description of problem:
Similar to issue #1986
KeyError on owntracks payload when firing enter zone event
Expected:
HA updates the device_tracker state rather than throw an exception in the log
Problem-relevant configuration.yaml entries and steps to reproduce:

Create waypoint in owntracks that isn't home
Trigger an enter event on that waypoint
Observe log

Traceback (if applicable):
16-05-06 11:49:21 homeassistant.core: Bus:Handling <Event mqtt_message_received[L]: topic=owntracks/me/nexus6p/event, qos=1, payload={"_type":"transition","acc":18.87,"desc":"my waypoint","event":"enter","lat":11.11,"lon":22.22,"tid":"6p","trigger":"c","tst":1462535360,"wtst":1462534036}>
16-05-06 11:49:21 homeassistant.core: BusHandler:Exception doing job
Traceback (most recent call last):
  File "/home/pi/hass182/lib/python3.4/site-packages/homeassistant/core.py", line 801, in job_handler
    func(arg)
  File "/home/pi/hass182/lib/python3.4/site-packages/homeassistant/components/mqtt/__init__.py", line 160, in mqtt_topic_subscriber
    event.data[ATTR_QOS])
  File "/home/pi/hass182/lib/python3.4/site-packages/homeassistant/components/device_tracker/owntracks.py", line 155, in owntracks_event_update
    enter_event()
  File "/home/pi/hass182/lib/python3.4/site-packages/homeassistant/components/device_tracker/owntracks.py", line 104, in enter_event
    if zone is None and data['t'] == 'b':
KeyError: 't'

Additional info:
I think L#104 of owntracks.py is the cause right there. For whatever reason my owntracks client is firing off payloads that don't have the 't' key. Instead it has a 'trigger' key. Here's an example payload from a transition event that fires enter when in the radius of an owntracks waypoint:
{"_type":"transition","acc":22.429,"desc":"test_waypoint","event":"enter","lat":11,"lon":12,"tid":"6p","trigger":"c","tst":1462543540,"wtst":1462542169}

I think this would fix the issue:
if zone is None and 't' in data.keys() and data['t'] == 'b':

Also, it might be an issue with the Android owntracks app. I filed an issue with them too.

CC @pavoni


Feat/rework sidebar

Didn't want to switch to shadcn sidebar. @Galkon's was already perfect. Just added a better scroll to the current implementation in a separate PR


Website polish

For #3396
Change List

Fix website bug when navigating documentation
New content in showcase
Hide survey link in mobile

Coverage remained the same at 80.185% when pulling 3cda9b8e86901a58452a3f02e657e3db3b153f53 on x/website-polish into 135eaea66ee4e74218bf41c1a2ab930d3670d1eb on master.


Removed unused SCVMM files from a previous implementation.

Came across these files while looking for some hardware settings the other day.  This implementation of SCVMM was designed to run from a Windows appliance which was never released.
cc @bronaghs @blomquisg @jrafanie  :scissors:

Nice deletion :scissors: :fire:

:+1:  Very nice.

Ooh, I :heart: :jack_o_lantern:  :gift:
:clap:


[Bug] Permissive Parser failing on certain type of non-standard captures

Using chess.js 1.0.0-beta.2
Noticed a small bug in the permissive parser for non-standard moves that result in a capture. Happens when the move doesn't indicate itself as a capture (no 'x'):
// with pieces:
const Chess = require('chess.js');

const chess = new Chess.Chess("r1bqk2r/p1p2pp1/2n1pn2/1p5p/2pP4/bPNB1PN1/PB1Q2PP/R3K2R w KQkq - 0 12");

console.log(chess.ascii());
console.log(chess.moves({square: 'b2'}));
console.log(chess.move('Ba3', { strict: false }));

// with pawns:
const Chess = require('chess.js');

const chess = new Chess.Chess("r1bqk2r/p1p2pp1/2n1pn2/1p5p/2pP4/bPNB1PN1/PB1Q2PP/R3K2R w KQkq - 0 12");

console.log(chess.ascii());
console.log(chess.moves({square: 'b3'}));
console.log(chess.move('bc4', { strict: false }));

Expected this to work based off the list of moves that permissive parser should interpret correctly:

Thanks to everyone who contributes to this library!

Thanks for the report @ChrisAntley1. I may have gotten overzealous writing the documentation.  I'll look into fixing this, but if it's not critical, I may defer this until after the 1.0.0 release.


Register independent vars on each install type

When jenkins install is called, both tasks are visited. And
it registers result of jenkins_install_package twice, causing
it to have a wrong value, and don't install jenkins users, causing
security to don't be configure and plugins to fail.
It's better to add different vars for each task, and trigger
the users install in case of some of the two changed.

When jenkins install is called, both tasks are visited.

What's the use case here? I'm typically deploying to one server, or many servers with the same OS (e.g. all are Red Hat, all are Debian, etc.).
Is this a case where you have one playbook building a bunch of Jenkins servers and some are running Ubuntu, some CentOS, etc.?
Because in the former case, jenkins_install_package would only ever be set once, by either the RHEL or Debian included install.

I'm just running the playbook locally, on localhost. I know the task is run only once, but the var is stored in both cases. Look at some debug i added to try to detect the problem:
TASK [geerlingguy.jenkins : Ensure Jenkins is installed.] **********************
changed: [jenkins_master] => {"changed": true, "msg": "", "rc": 0, "results": ["Loaded plugins: fastestmirror\nLoading mirror speeds from cached hostfile\n * base: mirrors.evowise.com\n * epel: download-i2.fedoraproject.org\n * extras: mirrors.seas.harvard.edu\n * updates: mirror.metrocast.net\nResolving Dependencies\n--> Running transaction check\n---> Package jenkins.noarch 0:2.7.1-1.1 will be installed\n--> Finished Dependency
Resolution\n\nDependencies Resolved\n\n================================================================================\n Package           Arch             Version             Repository         Size\n================================================================================\nInstalling:\n jenkins           noarch           2.7.1-1.1           jenkins            65 M\n\nTransaction Summary\n=============================================
===================================\nInstall  1 Package\n\nTotal download size: 65 M\nInstalled size: 66 M\nDownloading packages:\nRunning transaction check\nRunning transaction test\nTransaction test succeeded\nRunning transaction\n  Installing : jenkins-2.7.1-1.1.noarch                                     1/1 \n  Verifying  : jenkins-2.7.1-1.1.noarch                                     1/1 \n\nInstalled:\n  jenkins.noarch 0:2.7.1-1.1
\n\nComplete!\n"]}
TASK [geerlingguy.jenkins : debug] *********************************************
ok: [jenkins_master] => {
"jenkins_install_package": {
"changed": true,
"msg": "",
"rc": 0,
"results": [
"Loaded plugins: fastestmirror\nLoading mirror speeds from cached hostfile\n * base: mirrors.evowise.com\n * epel: download-i2.fedoraproject.org\n * extras: mirrors.seas.harvard.edu\n * updates: mirror.metrocast.net\nResolving Dependencies\n--> Running transaction check\n---> Package jenkins.noarch 0:2.7.1-1.1 will be installed\n--> Finished Dependency Resolution\n\nDependencies Resolved\n\n========================================
========================================\n Package           Arch             Version             Repository         Size\n================================================================================\nInstalling:\n jenkins           noarch           2.7.1-1.1           jenkins            65 M\n\nTransaction Summary\n================================================================================\nInstall  1 Package\n\nTotal download
size: 65 M\nInstalled size: 66 M\nDownloading packages:\nRunning transaction check\nRunning transaction test\nTransaction test succeeded\nRunning transaction\n  Installing : jenkins-2.7.1-1.1.noarch                                     1/1 \n  Verifying  : jenkins-2.7.1-1.1.noarch                                     1/1 \n\nInstalled:\n  jenkins.noarch 0:2.7.1-1.1                                                    \n\nComplete!\n"
]
}
}
TASK [geerlingguy.jenkins : Ensure dependencies are installed.] ****************
skipping: [jenkins_master] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}
TASK [geerlingguy.jenkins : Add Jenkins apt repository key.] *******************
skipping: [jenkins_master] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}
TASK [geerlingguy.jenkins : Add Jenkins apt repository.] ***********************
skipping: [jenkins_master] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}
TASK [geerlingguy.jenkins : Ensure Jenkins is installed.] **********************
skipping: [jenkins_master] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}
TASK [geerlingguy.jenkins : debug] *********************************************
ok: [jenkins_master] => {
"jenkins_install_package": {
"changed": false,
"skip_reason": "Conditional check failed",
"skipped": true
}
}
Look at the debug of jenkins_install_package. See that the first time, runs for RHEL, and it collects the result from the package install. But then runs for debian, and it just skips the task but stores the result anyway, with a changed false, and causing the admin user to don't be installed, security config skipped and plugins failing due to invalid parameter --username

Hmm... that seems like a bug with Ansible—if the included file is skipped, it shouldn't register any variable value.
What version of Ansible are you running currently? I'll do a little digging on my end too.

Or... hmm... https://github.com/ansible/ansible/issues/15710
Apparently my understanding of skipped tasks and register is completely incorrect :P

i install from pip
i have ansible <IP_ADDRESS>
2016-08-01 19:45 GMT+02:00 Jeff Geerling<EMAIL_ADDRESS>
Hmm... that seems like a bug with Ansible—if the included file is skipped,
it shouldn't register any variable value.
What version of Ansible are you running currently? I'll do a little
digging on my end too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/geerlingguy/ansible-role-jenkins/pull/72#issuecomment-236653338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACaeLXP0PhPwgen9iOtSfVH01C7Z8DJ_ks5qbjCjgaJpZM4JZkCq
.

--
Yolanda Robla
email<EMAIL_ADDRESS>teléfono: 605641639


Simplify ParticleContainer

This PR simplifies the type parameters of ParticleContainer, and removes the conditional field which by definition was always nothing.
Moreover, instead of appending newly allocated arrays, push! now resizes the existing arrays and sets the new values, which is faster:
using BenchmarkTools

function f(x, n)
    y = zeros(eltype(x), n)
    append!(x, y)
    nothing
end

function g(x, n)
    m = length(x)
    mpn = m + n
    resize!(x, mpn)
    @inbounds for i in (m + 1):mpn
        x[i] = zero(eltype(x))
    end
    nothing
end

a = rand(1_000)

@btime f($(copy(a)), 100) # 339.092 ns (1 allocation: 896 bytes)
@btime g($(copy(a)), 100) # 263.981 ns (0 allocations: 0 bytes)

The empty! function now defines an array of particles whose element type is based on the previously used array instead of always using the abstract element type Particle.
The copy! function of ParticleContainer creates a set of copied particles by list comprehension instead of pushing them individually to an initially empty array.
Moreover, the definition of Trace is simplified by removing the completely uninitialized inner constructor and reordering the fields (making it easier to create new instances without specifying a task).

I'm really glad to see some of this stuff updated. The particle stuff has always been a little rickety.
Anyone know why tests aren't running for this PR?

I think there is a problem with tests on all our PRs.

@devmotion The CI is now fixed on the master branch. Can you try to rebase master into this PR? Once all tests pass, I'm happy to merge it.

I rebased on master, but as observed above tests aren't running. Skimming through other PRs it seems that tests don't run for PRs from forks?

Yep; seems that's true. I don't know how to activate CI on forks.

Sure, I can do that. I just noticed that apparently the actions are run on my fork but not on the main repo: https://github.com/devmotion/Turing.jl/actions

Sure, I can do that. I just noticed that apparently the actions are run on my fork but not on the main repo: https://github.com/devmotion/Turing.jl/actions

That is equivalent - happy to merge once the actions based CI passes.

It's not completely clear to me from the documentation, but maybe the issue for PRs could be solved by replacing lines such as
https://github.com/TuringLang/Turing.jl/blob/87a3db946a1e63b46f57fdee460acaa7309c4db6/.github/workflows/DynamicHMC.yml#L3
with on: [push, pull_request]. To run the Travis CIs for all PRs, one should probably enable Build pushed pull requests in the settings of Travis.

Thanks, Travis CI is now turned on. I’m not sure about the actions based CI though, cause turning both push and pull request on causes duplicate builds in the past.

I guess we'll face the same issue with Travis CI and end up with duplicate tests for forks from the main repo. However, I think that can be avoided by adding
branches:
  only:
    - master

to .travis.yml (at least that's what Google told me :smile:).

Apparently, in the same way Github Actions can be filtered: https://help.github.com/en/actions/automating-your-workflow-with-github-actions/configuring-a-workflow#filtering-for-specific-branches-tags-and-paths

That looks quite promising. Would you like to give it a try in this PR? It seems I can't push to this PR easily.


docs: FAQ/doc on LB overview

The current LB arch overview guide for load balancing is too detailed and complicated for most new users to make sense of. We need an even high level "getting started with Envoy load balancing" that introduces high level concepts and then links to the more detailed arch documentation. For example, it might flow like:

What is load balancing
Distributed vs. centralized load balancing
Etc.

Basically, break https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/load_balancing into sub-pages with a high level overview first that helps people understand how to even start thinking about which options to choose.

I can take this. One question: when you say distributed vs centralized, are you then referring to active health checking being distributed and the EDS weights being centralized?
/assign @snowp

For distributed versus centralized I mean the general concept of doing distributed LB in which each LB operates independently (like in general what Envoy does) vs. having a global load balancer take part in the process (like what more advanced control planes do. A concrete example of this is doing zone aware load balancing either directly in Envoy (option 1 in the docs) vs. in the control plane (option 2 in the docs). Also, of course, most advanced people end up doing a combination of distributed and centralized. I think a general treatment of this concept would be very helpful as the existing docs are way too large and intimidating for the new user.

Also, thank you very much @snowp for offering to do this!!!

Calling this done with the work @snowp has done.


Broken links on author's bio pages

On the author's bio pages the link to the author's site is broken. I've tested it on Chrome, Firefox and Safari all on OSX 10.9.5.
Checkout http://alistapart.com/author/bradfrost. The href on the link to Brad's site is http://http//bradfrostweb.com/. See attached for more info.

Thanks! I'l fix this up now.

@michellekondou It looks like the Parse URL plugin has stopped working altogether.


Roadmap?

This plugin seems great for websites with markdown content! Is there anything I can do to help push it forward?

I'm actually working on it as we speak, hope to get a real release out in the next few weeks 👍

That's amazing! I'm looking forward to it!

This will be a rather great complement to TailwindUI 💯

Is the light variant being considered?

Is light/dark variants being considered?

This would be pretty useful, however there are many ways in which developers could implement Dark mode... It may turn somehow difficult, unless there's a enable/disable property for the plugin, or a way for the plugin to rely on JS for Dark/Light mode.

Released 👍


Ordering in OpenAPI source files is not maintained.

Since the OpenAPI spec is read as JSON using the standard Go JSON reader and represented with Go maps, the ordering of values in the source files is lost.

I noticed that this Go YAML library can read YAML files into this structure which preserves key order.

This commit completes a conversion from the JSON reader to the Go YAML library noted above. YAML is read into MapSlice structures, which preserve order. The order-preserving reader is used both to read OpenAPI specifications and the OpenAPI schema (in jsonschema/jsonschema.go).

Confirmed with a unit test in this commit.


Bug: \Config\Home::index not found

PHP Version
7.4, 8.0, 8.1
CodeIgniter4 Version
4.3.5
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
Maria DB 10.4.22
What happened?
Showing there is no route if i use the CodeIgniter 4 syntax which is
$routes->get('/', [Home::class, 'index']); it doesn't work instead have to use the older syntax that is
$routes->get('/', 'Home::index');

Steps to Reproduce
Instead of CI3 syntax in route if you use CI4 syntax there is error
Expected Output
As i am using CI4 it only wise to use the CI4 syntax instead of the older CI3 syntax
Anything else?
No response

It is just a misuse, not a bug.
See the screenshot. It shows not found: \Config\Home::index.
The controller should be App\Controllers\Home.
If you use ::class keyword, you must specify the correct classname.
You need to learn PHP namespaces.

Yes I am new to this stuff but could you help in understanding why in some route it works and the in others it doesn't and i have to use the old technique/syntax. This is my routes.php
setDefaultNamespace('App\Controllers');
$routes->setDefaultController('Home');
$routes->setDefaultMethod('index');
$routes->setTranslateURIDashes(false);
$routes->set404Override();

// $routes->get('/', [Home::class, 'index']); ///this doesnt work idk why
$routes->get('/', 'Home::index'); // instead this works
$routes->get('/login', 'LoginController::index');
$routes->get('/register', [RegisterController::class, 'index']); // but in this case it works
$routes->get('/tasks', [TasksController::class, 'index']);  //also this works
$routes->get('/tasks/show/(:num)', [TasksController::class, 'show']);   //also this works

$routes->post('/register', [RegisterController::class, 'do_register']); //also this works
$routes->post('/login', 'LoginController::do_login');   //the ::class doesnt work here as well

if (is_file(APPPATH . 'Config/' . ENVIRONMENT . '/Routes.php')) {
    require APPPATH . 'Config/' . ENVIRONMENT . '/Routes.php';
}
//------------------------------------------------------------------------------------------------------------------------------

> It is just a misuse, not a bug. See the screenshot. It shows `not found: \Config\Home::index`. The controller should be `App\Controllers\Home`. If you use `::class` keyword, you must specify the correct classname. You need to learn PHP namespaces.

Because you don't understand PHP namespaces.
It is not CI4 thing, but PHP thing.
CI4 and modern PHP uses namespaces everywhere.
So I recommend you learn namespacing completely.
Read:

https://www.w3schools.com/php/php_namespaces.asp
https://www.php.net/manual/en/language.namespaces.rationale.php

P.S. If you show your code, do not use screenshots, but use copy and paste, please.

Thank you for the knowledge shared by you.

And Yes i have got it I realized now why you kept saying namespaces even though I used it but only on the RegisterController and I should have done the same for LoginController and Home as well.
And yes I have not used Markdown so much.

![Thank You Logo]
(https://www.google.com/url?sa=i&url=https%3A%2F%2Ftenor.com%2Fsearch%2Fthanks-you-gifs&psig=AOvVaw1yf8lT6ViH7HUaQf2IyHA_&ust=1686287660332000&source=images&cd=vfe&ved=0CBEQjRxqFwoTCJil7Zn1sv8CFQAAAAAdAAAAABAJ)


feat: enhanced snap highlight

This pull request enhances the functionality of snap hud for defrag. This pull request also lays the groundwork for the CGAZ 2.0 overhaul.
Changes:

When using highlight mode 2, only the portion of the highlighted snap zone that can provide acceleration is colored
Zones (snaps, CGAZ, mirror) are refactored as objects containing a reference to some panel, angles representing the left and right edges of that panel, and pixel values for the left and right edge of the panel on screen.
Cleaned up member variable names to align with style convention (camelCase)
Fix js error when scaling/interpolating across a range with 0 distance.

Checks

[x] I have thoroughly tested all of the code I have modified/added/removed to ensure something else did not break
[x] I have followed semantic commit messages e.g. feat: Add foo, chore: Update bar, etc...
[x] My branch has a clear history of changes that can be easy to follow when being reviewed commit-by-commit
[x] My branch is functionally complete; the only changes to be done will be those potentially requested in code review
[x] All changes requested in review have been fixuped into my original commits.
[x] Fully tokenized all my strings (no hardcoded English strings!!) and supplied bulk JSON strings below

Weird commit message in #681c33c @PeenScreeker

Weird commit message in #681c33c @PeenScreeker

ssh didn't want to comply, but I finally got this fixed.


Fix #111. An empty target dir is ok

Thanks for the PR, had to get a release out so didn't wait on the changes.


Not working on iOS Safari

When I enable ng-idle on my page and access it from Safari angular is broken.  Comment out Idle.watch() and the page works fine.  I also can't run the demo on this site from an iPad.

Please provide more information.

What model device and iOS version are you using?
Can you provide a demo site or plunkr (or similar) that replicates the issue?
Are there any errors logged in the debug console (you'll need to either enable this feature in Settings for Safari, or try debugging during iOS emulation in Safari)?
What are you expecting it to do vs what it is actually doing?
What version of Angular and ng-idle are you using?

There are a number of open issues resulting from multi-tab support that I need to resolve. However, I ran the demo on my phone and my iPad (a 5s and original Mini, both using latest iOS) as well as emulated the most recent devices and I had no problems with the demo.

+1
Sometimes works, in private browsing never works
i got this error
QUOTA_EXCEEDED_ERR: DOM Exception 22: An attempt was made to add something to storage that exceeded the quota.

@ruimserrano The private browsing issue was addressed in 1.1.1 which went out last night.

great..i've updated the lib and it's working now.
Thanks
Keep up the good work

Yes, this turned out to be a private browsing issue.  Thanks for the fix.


DNS resolution (local & remote) + DNS suffix

First, sorry if it isn't clear and if the text isn't formatted, it dosen't work and i dunno why.
Hi there, thanks everyone for the support and that wonderful script. Thats the only way my openvpn works.
Here's my issue:
1st case: Pivpn installed on a RPI3B+. Works extremly well. I can ping, access remote network, no problem at all except the DNS resolution
remote network:
Pivpn : <IP_ADDRESS>
Gateway/dns/router/dhcp : <IP_ADDRESS>
I'm at work, we've a domain, let's call it "work.com"
This is my client.
My remote network (where the vpn/rpi is), has no domain at all (no suffix).
When i use my VPN, i need to use a "dot" after my hostnames.
For exemple " Laptop. " to connect by RDP.
It's a bit annoying as network shares etc.. don't work this way.
But the worst downside is when the vpn is active on my client, i have to use the domain after the hostnames for local ressources.
server.work.com for exemple. (so outlook dosen't work aswell this way)
ipconfig /all on my client gives for the tap:
Suffixe DNS propre à la connexion. . .   : Oui
Configuration automatique activée. . .   : Activé
I can't send my nic work info, but with the vpn active, it stays the same as with the vpn off.
So on ipconfig the vpn dosen't change my real nic (except the vpn one)
The vpn pushes the right infos (right dns for exemple)
So what could it be? (i hope i'm pretty clear, sorry i'm a poor frenchie)
2nd case:
On a vm, at my parent's home, i installed openvpn access server (with an appliance)
DNS works the same, i have to put a dot for remote BUT local resources work. I don't have to add the "work.com"
Why?
I'm unable to find the server.conf file on the openvpnas, it could help me to compare as local dns work this that one.
Last info: my dns/dhcp one both remotes are ISP routers and i can't change the domain pushed by the DHCP...
I'd like to try with a .local and push the .local on my RPI server.conf but i can't. I also tried to push a "dot"  domain on my RPi conf but i gives me of course " .." so laptop.. (and it doesn't resolves)
Here are some output from my Raspberry
Any "-------" are just private infos
Thanks for any tips..

::: Copy saved to /tmp/debug.txt
:::

Console Output of sudo iptables -t nat -S
  -P PREROUTING ACCEPT
-P INPUT ACCEPT
-P OUTPUT ACCEPT
-P POSTROUTING ACCEPT
-A POSTROUTING -s <IP_ADDRESS>/24 -o eth0 -j MASQUERADE

Console Output of sudo iptables -S

-P INPUT ACCEPT
-P FORWARD ACCEPT
-P OUTPUT ACCEPT

output of sudo netstat -uanp | grep openvpn
  It outputs nothing (normal?)

No one? :(

sorry @ftt-prod for late response pivpn has been quite stalled for a while .. but or I am missing something, or to me, all seems to be working as it should.
However, this doesn't even seem to be related to PiVPN.
if you want to further discuss this or some help, please take the issue to our subreddit, as from now on GitHub is strictly for issues related with PiVPN itself.
Thank you for using PiVPN.


I wrote an absolutely minimal typescript ambient module declaration file. You should have it.

declare class Template {}

declare class CodeInput {
  templates: { custom(highlight: (pre: HTMLElement) => HTMLElement, usePre: boolean, setClass: boolean, giveCodeInput: boolean, plugins: []): Template };
  registerTemplate(name: "syntax-highlighted", template: Template): void; }

declare const codeInput: CodeInput;

Thank you very much for this! I will add it ASAP.

Please could you explain how you could use this with TypeScript, as I have not used TypeScript before and there do not seem to be many resources on front-end TypeScript (especially on how to recognise classes from other files, without node-style imports).

Yes. Your typescript build is configured using tsconfig.json, which should
have an array entry for "files". When code-input.d.ts is added to the
"files" array, typescript treats the declarations in code-input.d.ts as
globally visible.
"files": ["code-input.d.ts", "repl.ts"],
I don't use a node-style import. I use the <script src= tags to include
code-input.js before any script where its declarations should be visible,
and i use code-input.d.ts in any script where it will be visible.

Typescript now knows the type of the global variable codeInput in
repl.ts.
On Sat, Jun 3, 2023 at 2:37 AM WebCoder49 @.***> wrote:

Please could you explain how you could use this with TypeScript, as I have
not used TypeScript before and there do not seem to be many resources on
front-end TypeScript (especially on how to recognise classes from other
files, without node-style imports).
—
Reply to this email directly, view it on GitHub
https://github.com/WebCoder49/code-input/issues/45#issuecomment-1574715604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDEXTEH6AOD6YZINBJNCDTXJLLRZANCNFSM6AAAAAAYURWPHY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
-Theodoric


Update libc & libcxx support documents

Address consistency issues between LibcxxSupported.md and LibcSupported.md

Scrub LibcSupported.md and LibcxxSupported.md for consistency with current test coverage.

Format LibcxxSupported.md by functional areas.

Fixes #1544.

@anakrish Feel free to file a separate issue to track culling the published headers. The Intel SDK doesn't seem to do it beyond dropping the support folder and there's work to ensure that there are no header dependencies for what is dropped. Also keep in mind that we still have #28 in the backlog, which was the original plan to make unsupported headers more obvious at build time.

@anakrish Thanks for the review; I've been reluctant to explicitly state that we support a given C++ standard though. The closest level of confidence I have is the C++11 feature set, and that is largely for parity with Intel SDK. Even there we have a few exceptions.
We do test at C++14 level, but we also omit specific features associated with that standard (e.g. shared_mutex). There's an ongoing conversation about how we want to frame the C++ support moving forwards as we look at the v8.0.0 LLVM update.

bors r+

bors retry


Can you provide those svg files should be placed in folder assets/svg_images?

That svg files have copyright, then no.


Client build is done twice in chain and crashes.

Also, two build notifications are passed to the frontend.

This happens only when building the experiment clients.


Pass the parameters to after_inactive_sign_up_path_for

How can I pass the params such as email, username and name of newly created user to the route specified in after_inactive_sign_up_path_for? I need to display them right after registration...

after_inactive_sign_up_path_for receives the resource as an argument (e.g. an instance of User or whatever scope you're using), so you should be able to extract the attributes from there and build your route (or store into the session).
Please use the mailing list or stack overflow for questions, we reserve the issues tracker for issues. Thanks!


[WIP]: GitHub Source controller

Fixes knative/eventing#449
This is heavily based on both the ContainerSource and KubernetesEventSource.
Proposed Changes

Create a GitHubSource CRD in sources.eventing.knative.dev
Add a skeleton controller implementation

Release Note

New source GitHubSource: Registers a webhook with GitHub and sinks incoming GitHub events to a Sinkable.

This is still very much in-progress, but opening this PR gives others visibility into the work so far.
/hold

With the latest changes this is starting to incorporate knative/eventing#448 as well. It just seemed easier to start working on both in tandem now that this is far enough along.

As of the latest commit the controller happy path works in e2e testing. The receive adapter still needs manual testing to confirm incoming GH events get shuffled along properly. And more automated tests. I'm removing the duplicate manual hold, since the wip hold will still apply.
/hold cancel

This works for creating GitHubSources. However, when updating or deleting a GitHubSource, there isn't any logic in place yet to update/delete the actual GitHub webhook.

This now handles deleting the webhook on GitHub when a GitHubSource is deleted. Updates aren't done yet but I removed the WIP label in case we don't want to hold up the release to get updating in here.
Updating will be more tricky because changing the secrets will requiring deleting and recreating the webhook on GitHub and the ksvc. Other updates may just require doing an update to the GitHub webhook.
I've manually tested far enough to verify that the GitHub webhooks get created/deleted in response to the GitHubSource being created/deleted and that sending the proper JSON payload to the receive adapter (via curl) works from there. I haven't done an entire end-to-end manual test yet - that will require a cluster with a public hostname that GitHub can reach.

I've verified this works with manual end-to-end testing and opened knative/docs#521 with a sample of how to use GitHubSource.

It definitely gives a better user experience to surface exactly what's missing with secrets and/or sinks to the user via the conditions. Thanks for catching that!

/lgtm
/approve
/hold for @n3wscott to remove hold since he's also written a few of these.


NPM: fix npmrc generation for v3 package-locks

Fixes #6661
Using a v3 package-lock, dependency-urls was returning [] because there is no top-level dependencies property.  [].all? turns out is always true, so it was always sticking the line registry=<my-configured-reigstry> into the .npmrc even for those hosted on GHPR which isn't a reverse-proxy style registry. Thus it is not working.
I switched to a regex based approach to extract all of the "resolved" lines. This is probably more correct than the current implementation even for v2 lockfiles as this checks all dependencies, not just the ones at the top level. In the event package-lock v4 removes or renames the resolved lines, I've added a check that dependency-urls isn't empty.
Also once I fixed this, some tests started doubling up on scoped registries, so I had to add a uniq.

@yeikel I could be missing something but this code path doesn't appear to inject registries from the resolved lines. It's only using it to identify possible scopes for registries defined as credentials. It's also using it to decide if there's a global registry. In both cases it uses the registry defined in the credentials to write to the npmrc, so if there were a path there it would preserve it.
It looks like https://github.com/dependabot/dependabot-core/pull/7030 will have the same issue with v3 lockfiles though so I could add some tests around that when we get to that PR to make this same change.


Question: Convert to Typescript definition files possible?

Can this project also generate typescript definition files (.d.ts) from flow files?

It would be pretty straightforward to write a generator for that (see generator.ts in this repo). What’s the use case?

Whoops, when I wrote that comment I thought I was in a different repo 😅.
Two things:

This package is not yet ready to use (see https://github.com/bcherny/flow-to-typescript/issues/1)
When it is (or if you want to contribute!), I’d run the generated .ts file through TSC with the -d flag to generate .d.ts


Xcode not reflecting filesystem: Missing directories post-repo cloning

name: Bug report
about: Xcode project not reflecting filesystem structure
title: 'Xcode does not show directories despite presence in filesystem'
labels: 'bug, xcode, filesystem'
assignees: 'Jaime-A-Perez'

Describe the bug
After cloning the repository and switching to the develop branch, certain directories (AuthModels and AuthViewModels) that are present in the filesystem are not being displayed in the Xcode project navigator.
To Reproduce
Steps to reproduce the behavior:

Clone the repo git clone https://github.com/Jaime-A-Perez/To-Do-List-Apple-Ecosystem
Switch to the develop branch git checkout develop
Open the project in Xcode open ToDoListAppPro.xcodeproj
Observe that AuthModels and AuthViewModels directories are not displayed in the project navigator

Expected behavior
All directories and files that are present in the filesystem after cloning the repository should be visible and properly referenced in Xcode's project navigator.
Desktop (please complete the following information):

OS: macOS
Xcode version: 15.1

Additional context
The missing directories were added manually to the project as folder references, and they appear as blue folders in Xcode. Running git status reveals changes in the project.pbxproj, which seem to be related to the manual addition of these directories.

Hello Diana,
Thank you for bringing this issue to our attention! I have reviewed the problem you described and can confirm that the AuthModels and AuthViewModels directories are not showing up in Xcode's project navigator after following the steps you provided. This behavior is indeed unexpected, and I appreciate you pointing it out.
It appears that the issue is related to how these folders were added to the Xcode project. Manually adding directories as folder references can cause such discrepancies, especially if the changes are not properly reflected in the project.pbxproj file.
Thank you again for your contribution to improving the quality of the project.

With the successful merge of Pull Request #19 , which addressed the missing folder references in the Xcode project, I am closing Issue #18. All directories should now be visible in the project navigator as expected.
Thank you all for your contributions and support in resolving this issue.


Copy-FromToMaster fails on certain directories

Describe the bug
Copy-FromToMaster fails on certain directories. This failure is coming from the fact that scp.exe is not working correctly when using -r flag.
To Reproduce

Copy folder controller to a place on your disk (like c:\x)
Execute the internal command:
scp.exe -o StrictHostKeyChecking=no -r -v -i "C:\Users\YOURUSER\.ssh\kubemaster\id_rsa" c:\x\controller remote@<IP_ADDRESS>:tmp/
You will observe that scp did not copy all of the subfolders in the remote VM

Expected behavior
All subfolders are copied to target directory in VM
Additional context
Adding this code part:
       # check if the source is a directory
        if (Test-Path -Path $Source -PathType Container) {
            # get last item in path from source
            $DirectoryAtTheEnd = Split-Path -Leaf $Source
            # get directory path from target after last :
            $TargetDirectory = $Target -replace '.*:', ''
            # make directory in master VM
            ExecCmdMaster "mkdir -p $TargetDirectory/$DirectoryAtTheEnd"
            # extend source with all files
            $Source = $Source + '\*'
            # extend target with directory
            $Target = $Target + '/' + $DirectoryAtTheEnd
            Write-Log "copy directory: $Source to: $Target"
        }
        scp.exe -o StrictHostKeyChecking=no -r -i $global:LinuxVMKey "$Source" "$Target" 2>&1 | ForEach-Object { "$_" }

in method Copy-FromToMaster solves the problem in image build use case, but a more general solution (recursive path creation) needs to be implemented.

https://github.com/Siemens-Healthineers/K2s/blob/6ee9580c1ed6e7f5fb625d8612a98987ac6ca1f3/addons/gpu-node/Enable.ps1#L141C1-L144C91

scp fix for Windows worker VM missing


WinSCP

Including WinSCP would pair well with the existing PuTTY install.

WinSCP is available to install as a Chocolaty package
https://community.chocolatey.org/packages/winscp


Running dailyops on Perlmutter

Has anyone successfully run dailyops on Perlmutter? I'm getting this error message when loading the desisurveyops environment:
rongpu@login18:~> source /global/common/software/desi/desi_environment.sh master
rongpu@login18:~> module load desisurveyops/main
Lmod has detected the following error:  The following module(s) are unknown: "desisurveyops/main"

Please check the spelling or version number. Also try "module spider ..."
It is also possible your cache file is out-of-date; it may help to try:
  $ module --ignore-cache load "desisurveyops/main"

Also make sure that all modulefiles written in TCL start with the string #%Module

I stopped there and will stick to Cori for next week's dailyops, but it might be good to start the discussion on transitioning to Perlmutter.

thanks Ronpgu!
with looking 2min:
it looks like on cori module load desisurveyops/main is related to this file: /global/common/software/desi/cori/desiconda/20211217-2.0.0/modulefiles/desisurveyops/main
and such a desisurveyops/main file does not exist in the equivalent perlmutter location:
raichoor@perlmutter:login27:~> ls /global/common/software/desi/perlmutter/desiconda/20220119-2.0.1/modulefiles/
desiconda  desimodel	     desispec	 desitransfer  dust	    gpu_specter  QuasarNP	    simqso    specsim
desietc    desisim	     desisurvey  desitree      fastspecfit  nightwatch	 redrock	    specex    specter
desimeter  desisim-testdata  desitarget  desiutil      fiberassign  prospect	 redrock-templates  speclite  surveysim

I just installed desisurveyops/main on perlmutter.  Please try again, also using "desi_environment.sh main" instead of "master" (and please correct any docs that mentioned "desi_environment.sh master")

@sbailey Thanks. I can load desisurveyops now. Do you (or anyone) foresee other issues with running dailyops on Perlmutter? If not, I can try doing it next Monday.

Please go ahead and try.  Honestly, I thought I had already tried this!

Another problem: on Perlmutter I cannot run the collabsu command (needed for the fiberassign reproducibility check): https://docs.nersc.gov/systems/perlmutter/#connecting-to-perlmutter-with-a-collaboration-account

The replacement for collabsu is on your laptop run sshproxy.sh -c desi and then ssh<EMAIL_ADDRESS> If your copy of sshproxy.sh doesn't recgonize -c, grab the latest version.  Please update any surveyops wiki instructions that still mention collabsu.

Yeah, this works, but you have to do it via the way the describe at your link.

just checking, in case it s relevant: these lines connect as desiproc?

FWIW, I just updated the DailyOps page to have a collaboration accounts section.

@araichoor I believe you'd use sshproxy.sh -c desiproc in that case. The command is specific to the collaboration account you're trying to access.

One more detail:
ssh -Y -i $HOME/.ssh/desiproc<EMAIL_ADDRESS>
i.e. you have to tell ssh which key to use

Thanks, that works. I have added these commands to the DailyOps page.

@rongpu: Can we close this issue?

I didn't end up running the full dailyops procedures on Perlmutter during my shift. But if you or anyone have successfully run the procedures on Perlmutter, then we should close it.

Ah, makes sense, thanks for the note. Maybe we should wait until we're back in a more normal mode of survey operations, where we run through all the steps one-by-one. Everything's a tad asynchronous at the moment!

I've now tried all of the daily operations steps on Perlmutter, including:

Fiberassign reproducibility.
Tile QA.
Archiving tiles.
MTL.

and I confirm that everything works. So, I'm closing this issue.


GitHub Metadata : build_revision is always NIL

TL;DR :
I have currently an issue with themes for github pages where the variable site.github.build_revision is always NIL.
I think this it's an issue when the github page is build, the variable git_commit_sha cannot be retrieve.
Informations
To be sure it's global I did some basic reproduction steps:

Creating a new github repository
Added a README.md
Configure a Github page in the Github repository settings
Selected a Theme (for exemple Cayman)
Waited the end of the build in the Actions tab
Locked in the page source of the github.io page for the stylesheet link where the version need to be concatenated

The link is <link rel="stylesheet" href="/test_101/assets/css/style.css?v=">, there is nothing after the v=.
I checked every GitHub Metadata by adding these lines in the html :

and the build_revision (a.k.a git_commit_sha) is the only empty value.
More details

At the end of the "Build page with Jekyll" part of the build process we can found that:

   GitHub Metadata: Calling @client.repository("BenSouchet/test_101", {:accept=>"application/vnd.github.drax-preview+json"})
   GitHub Metadata: Calling @client.pages("BenSouchet/test_101", {})
fatal: not a git repository (or any of the parent directories): .git
           Writing: /github/workspace/docs/_site/assets/css/style.css
           Writing: /github/workspace/docs/_site/index.html
                    done in 0.624 seconds.
 Auto-regeneration: disabled. Use --watch to enable.

So there is an ISSUE , maybe related to this.

I tried to add lot of settings in the _config.yml my file ended up a bit like that:

theme: jekyll-theme-cayman
title: Test 101
description: Bug testing
repository: BenSouchet/test_101
encoding: UTF-8
plugins:
- jekyll-remote-theme
- jekyll-github-metadata

@benbalter I think you are the man of the situation, you will I think find the bug in no time 🥇

Good news the issue will be fixed soon ( https://github.com/jekyll/github-metadata/issues/220#issuecomment-1006765976 ), I close this issue, new comments will be on the issue I linked.


Dictionary is always empty when defined as a member variable.

Godot version
v4.0.beta4.mono.official [e6751549c]
System information
Fedora Linux 37 (Workstation Edition)(beta) 64 bit
Issue description
Expected behaviour
The variable dict contains a dictionary containing the specified keys and values.
Actual behaviour
The variable dict is an empty dictionary.
print(dict) prints "{ }"
Note: This is not an issue of print(), dict doesn't contain anything when accessing a key.
This is fixed when adding the @onready annotation to the definition like so @onready var dict = {"hello": 1}
Steps to reproduce

Create a script which extends Control

Define a member variable and store a Dictionary containing some values
var dict = {"hello": 1}

Check the value of the variable
print(dict)

Minimal reproduction project
No response

@SassyJammy Can you reproduce this in any of the previous 4.0 betas to determine when the regression started?

@Calinou just tested all non-mono versions. Looks like the issue started in beta 4.

Hmm that's weird. I've rewritten my code just the way it was and now it suddenly does work.

If it's fixed then close it. Can always be reopened if you encounter it again (hopefully with more details).


Juniper/Rocket/MySQL integration

I'm having a difficult time getting Rust, Juniper/GraphQL, and MySQL to all work together.  This problem is compounded by the fact that each piece is working just fine, it's the integration that is a problem.  So who shall I beg insight from?
As Rocketeers know, Rocket has a method for connecting to MySQL. It involves creating an object, in my example I'll call it MyRocketSQLConn, and feeding said object to any route handlers that need a connection to MySQL.  This part works just fine.
As Junipeters know, Juniper defines a query, schema, and context that are also fed to any Rocket route handlers that need them.  The route handler will invoke query.execute(schema, context) to do its thing. This part also works just fine.
I can successfully deliver MyRocketSQLConn, Query, Schema, and Context into a route handler, but get stuck shortly thereafter.
In the examples that I've seen Context has been used to hold a toy database but it looks like an obvious place to inject MyRocketSQLConn into the Juniper works. Unfortunately this is trouble.  The basic reason for this is that the context is immutable for reasons of thread safety and I would need to use inner mutability to inject MyRocketSQLConn into this.
I have come tantalizingly close to a solution by adding a field to the Juniper Context object, which I call JunDatabase:
pub struct JunDatabase {
    pub conn: Arc<Mutex<Option<MyRocketSQLConn>>>,
}

The Arc and Mutex deal with the thread safety issues.  I need Option because I must create the Juniper Context before I have instantiated MyRocketSQLConn.
So the route handler that I have is:
#[rocket::post("/graphql", data = "<request>")]
fn post_graphql_handler(
    context: State<M::JunDatabase>,
    request: juniper_rocket::GraphQLRequest,
    schema: State<Schema>,
    mut conn: MyRocketSQLConn
) -> juniper_rocket::GraphQLResponse {

    {
        let m = &*context.conn;
        let mut m1 = m.lock().unwrap();
        *m1 = Some(conn);
    }

    request.execute(&schema, &context)

}

This part works just fine and execution will eventually get to:
fn helloJuniper(database: &JunDatabase) -> i32 {
    let m1 = &*database.conn;
    let m2 = m1.lock().unwrap().as_ref().unwrap();

    ...
}

At this point the code compiles and upon execution m2 is a &MyRocketSQLConn.  This is where I'm stumped.
The basic issue is that, depending upon which of the 99,000 subsequent variations I try, the error messages generally complain about moves, borrows, mutability, and lifetimes.  I have a general understanding of these issues individually, but undoubtedly have more to learn about these topics.
Given one particular guess yields:
let m3 = (*m2).prep_exec(...);
^^^^^ `m2` is a `&` reference, so the data it refers to cannot be borrowed as mutable

I'll spare you the loooong list of guesses that I've made about this as I thrash hopelessly hoping that the infinite monkey battalion can succeed in the absence of divine insight.  If I'm missing a magic * somewhere, I'd be happy to have my nose rubbed in that fact.  But I'd really like to hear some ideas for a general purpose approach to dealing with this issue.
Thank you.

Is your WIP code open source? Happy to take a look.

Howdy @LegNeato ,
Thanks for the offer.  The code is indeed open source and you can find it here
The route handler named post_graphql_handler can be found starting at line 257 in src/bin/server.rs
The function that I'm trying to feed the MySQL connection into is named currencies and can be found starting at line 18 in src/schema.rs.  This is a method of the Query type that Juniper user and Juniper is responsible for invoking this function.
The MySQL connection object is of type MyRocketSQLConn, defined in src/db.rs
The code as-is will compile and it will deliver a MyRocketSQLConn into the currencies function, but subsequent usage is hampered by whatever mysterious issues are presently stumping me.
My first choice is to figure out how to get a mut MyRocketSQLConn into said currencies function.
My secondary choice is to figure out how to retrieve a suitable connection object from MyRocketSQLConn using a function call to it.  I have an impl block for MyRocketSQLConn with a commented out attempt at doing so.
Hope you can find the "doh!" issue that I'm missing.  Thanks again for the help.

P.S. When the server is running...
curl -X POST -H "Content-Type: application/json" -d '{"query": "{ currencies }"}' http://localhost:3003/graphql

https://github.com/bostontrader/bookwerx-core-rust/pull/1 allows it to compile but don't know if it is the right thing to do for rocket. Let me know if it is still an issue or if that solves it!


Check the correct sql state and error code when unwrapping the except…

…ion chain.

Coverage increased (+0.06%) to 76.43% when pulling 803dcf6ddad8ce99fa779e0372a886039816cfa5 on johnou:correct_checks into 135fb0bc5d5d66c075cd0c18ccda30726b2f233f on brettwooldridge:dev.

@brettwooldridge apologies for not catching this sooner.

@brettwooldridge any estimates on when we might see fixes for https://github.com/brettwooldridge/HikariCP/pull/723 and https://github.com/brettwooldridge/HikariCP/pull/724 in a release?

@johnou How about now?  I just published 2.5.1.

@brettwooldridge you rock!

@doom369 heads up.

@johnou @brettwooldridge everything works fine for 1 week in production already. Thanks!

Great.


Question: Grid default filteration

Hi,
In version 5 i used to filter the grid by default by creation date and i was using the following code :
 var qrys = new System.Collections.Specialized.NameValueCollection();
  qrys.Add("Grid-CreatedOn-GreaterThanOrEqual", DateTime.Now.ToString("dd/MM/yyyy"));
grid.Query = qrys;

But in version 6 it is not working so is there any way to filter the grid by default ?
Thank you

If you are using latest sources, then it would be similar to creation-date-later-than-or-equal.

It is not working i have used the following code
q1 = "created-on-equals=2019%2F05%2F21%20&created-on-op=&created-on-equals=";//Copied as is after doing manual filter
                grid.Query = new QueryCollection(QueryHelpers.ParseQuery(q1));

I added this to the index page of roles and it seems to works. I'm not sure where are you initializing all this, but maybe something is overriding your values?
@{
    var html = Html
        .Grid(Model)
        .Build(columns =>
        {
            columns.AddAction("Details", "fa fa-info");
            columns.AddAction("Edit", "fa fa-pencil-alt");
            columns.AddAction("Delete", "fa fa-times");

            columns.AddProperty(model => model.Title);
            columns.AddProperty(model => model.CreationDate);
        })
        .ApplyDefaults();

    html.Grid.Query = new QueryCollection(QueryHelpers.ParseQuery("creation-date-later-than-or-equal=18%2F10%2F2018"));
}

@(html)

The diiferent thins is I am using server side code

   IGrid<RoleView> grid = new Grid<RoleView>(Service.GetViews());
  grid.Columns.Add(model => model.Id).Titled("#");
   grid.Columns.Add(model => model.CreationDate).Titled("CreationDate");
var qrys = new System.Collections.Specialized.NameValueCollection();
  qrys.Add(""creation-date-later-than-or-equal", DateTime.Now.ToString("dd/MM/yyyy"));
grid.Query = qrys;

 foreach (IGridColumn column in grid.Columns)
            {
                column.Filter.IsEnabled = true;
                column.Sort.IsEnabled = true;
            }

            return grid;

I have moved the filter code to the markup and now it is working.
But i don't know if am missing something in the above code?

Found out that html grid constructor overrides your query with request one.
I will swap initialization so it uses already set grid's query instead, as it should be.
For now you can force filter initialization with column.Filter.Method = column.Filter.Method; after enabling the filtering in the for loop.


Is it possible to execute 'docker-compose run ...' while building a stack in portainer?

Question:
There are a few stacks (as for example ruby on rails + postgresql; https://docs.docker.com/compose/rails/) which require the execution of several steps using 'docker-compose run' commands in each of those steps. Is there a way to do a similar procedure in portainer? It seems the stacks option only allows to link to a single docker-compose.yaml file. Am I understanding it correctly?
Thank you
Goncalo

Hi @GoncaloBorges, sorry for the late answer.
It is not possible to execute the equivalent of docker-compose run ... inside Portainer. Whilst this is technically possible (as supported by libcompose: https://godoc.org/github.com/docker/libcompose/project#EmptyService.Run) this would require us to think about a way to to make this easy to use inside the app UX/UI wise.
At the moment, you can simply execute the equivalent of docker-compose up and docker-compose down via Portainer for Compose 2 stacks.
Tagging as an enhancement of the stack feature.


ALPR bug

[     script:sna-alpr] SCRIPT ERROR in timer: Error: Execution of native 000000002f7a49e6 in script host failed: Argument at index 1 was null.
[     script:sna-alpr] >  (@sna-alpr/server/server.js:8164)
[     script:sna-alpr]

Fixed in 0.1.0


[BUG?] setTextAlign a nil value

The function documented here, "setTextAlign"  is nil.
How to replicate:
local basalt = dofile("lib/basalt")
local mainFrame = basalt.createFrame("mainFrame"):show()
local label = mainFrame:addLabel("label"):setSize(45,1):setValue("This is a label"):setTextAlign("center","center"):show()
basalt.autoUpdate()

This errors: test.lua:3: attempt to call method 'setTextAlign' (a nil value)
It seems it's on Button, not Object?

Hey!
Thanks you. Sadly i have to say the setTextAlign is wrong placed in the objects page. Because obviously not all of the objects do have text which can get centered. I will have to change this in the wiki page's. Labels don't have a setTextAlign function yet.
BUT the idea is actually pretty nice. So i decided to implement it quick. The last version should have setTextAlign(horizontal, vertical) for labels. If something doesn't work correct just tell me!


Minor fixes

This PR addresses two crashes:

Line 1088 crashes AFLNet if the user wants to exit AFLNet and the iteration is still in the beginning with response_bytes uninitialized.
When response_buf is printed (e.g. for manual debugging purposes) extra memory is read.

Thanks, @andronat!!


Question: API to add items to menu

There are some user actions that don't make sense to have in the primary UI, but should be accessible via a native menu. Are there plans to add an API to support doing this?

There are no plans to add a Menu Bar API directly to react-native-macos.   But it would be a great component that could be a separate project and published as a Cococapod.


chore: Add v1 APIs

Fixes #N/A
Description

This change adds the v1 APIs for the EC2NodeClass. This PR is intended to copy the v1beta1 APIs, defaults and validation. This PR does not change the EC2NodeClass  used by controllers or includes conversion between the v1beta1 and v1 APIs
Conversion webhooks and updating all the controllers to use the v1 APIs will be followed-up in separate PRs.

How was this change tested?

All the tests have been copied over from v1beta1
make presubmit
Does this change impact docs?
[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.4%) to 79.064%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.4%

Covered Lines:
5725

Relevant Lines:
7241

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.077%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5726

Relevant Lines:
7241

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.077%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5726

Relevant Lines:
7241

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.127%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5781

Relevant Lines:
7306

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.14%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5782

Relevant Lines:
7306

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.14%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5782

Relevant Lines:
7306

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.14%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5782

Relevant Lines:
7306

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

252 of 601 (41.93%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.3%) to 79.095%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
90.56%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5785

Relevant Lines:
7314

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

253 of 602 (42.03%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.3%) to 79.139%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/v1/ec2nodeclass_defaults.go
0
1
0.0%

pkg/apis/v1/ec2nodeclass_status.go
0
9
0.0%

pkg/apis/v1/ec2nodeclass.go
7
20
35.0%

pkg/apis/v1/ec2nodeclass_validation.go
145
207
70.05%

pkg/apis/v1/zz_generated.deepcopy.go
67
331
20.24%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
5789

Relevant Lines:
7315

💛  - Coveralls


Dungeon Travelers 2 [PCSE00693]

Game summary

Game name: Dungeon Travelers 2 [USA]
Game serial: PCSE00693
Game version: 01.00

Vita3K summary

Version: v0.1
Build number: 1374
Commit hash: https://github.com/vita3k/vita3k/commit/c61efe6b

Test environment summary

Tested by: dj.tuBIG/MaliceX
OS: Windows 10 Build 18363
CPU: Intel i7-3770
GPU: NVIDIA GTX1070
RAM: 32GB (DDR3-1866)

Issues

On attempt to launch game, colours appear but stops at yellow/orange screen.
Logs indicate a number of IO errors, several unimplemented functions and appears to have crashed/hanged on sceKernelTryLockLwMutex returning SCE_KERNEL_ERROR_LW_MUTEX_FAILED_TO_OWN (0x80028185)

Screenshots

Log
DT2_vita3k.log
Recommended labels

Nothing
Crash

Game summary

Game name: Dungeon Travelers 2
Game serial: PCSE00693
Game version: 01.00

Vita3K summary

Version: v0.1.3
Build number: 2260
Commit hash: https://github.com/vita3k/vita3k/commit/c88796dd

Test environment summary

Tested by: dj.tuBIG/MaliceX
OS: Windows 10 20H2 Build 19042.1165
CPU: AMD Ryzen 3900X
GPU: NVIDIA RTX2070
RAM: 128GB (DDR4-3600)

Issues

On attempt to launch game, seems to hang on boot.
Logs indicate unimplemented functions and an error for sceKErnelTryLockLwMutex

Screenshots

Log
vita3k.log
Recommended labels

Nothing
Crash

Game summary

Game name: Dungeon Travelers 2: The Royal Library & the Monster Seal
Game serial: PCSE00693
Game version: 01.00

Vita3K summary

Version: v0.1.3
Build number: 2321
Commit hash: https://github.com/vita3k/vita3k/commit/5fad7f08

Test environment summary

Tested by: dj.tuBIG/MaliceX
OS: Windows 10 20H2 Build 19042.1288
CPU: AMD Ryzen 3900X
GPU: NVIDIA RTX2070
RAM: 128GB (DDR4-3600)

Issues

On attempt to launch game, appears to now boot into the loading screen but quickly crashes.
Logs indicate an issue with various unimplemented sceGxm API functions and eventually an INVALID_POINTER returned due to unimplemented stubs. (eg: sceGxmSetWBufferEnable)
Can at least see the first frame of the loading screen appear!
According to @Zangetsu38 crash hits draw,cpp:169 relating to   fragment_program_gxp.is_native_color() function call.

Screenshots

Log
vita3k.log
Recommended labels

Boot
Crash

Game summary

Game name: Dungeon Travelers 2: The Royal Library & the Monster Seal
Game serial: PCSE00693
Game version: 01.00

Vita3K summary

Version: v0.1.4
Build number: 2484
Commit hash: https://github.com/vita3k/vita3k/commit/7e18eac2

Test environment summary

Tested by: dj.tuBIG/MaliceX
OS: Windows 10 20H2 Build 19044.1645
CPU: AMD Ryzen 3900X
GPU: NVIDIA RTX2070
RAM: 128GB (DDR4-3600)

Issues

Similar to previous report, but different log errors involving unimplemented SceGxm functions..
New log pasted here.

Screenshots

See previous report

Log
vita3k.log
Recommended labels

Crash
Intro

Game summary

Game name: Dungeon Travelers 2: The Royal Library & the Monster Seal
Game serial: PCSE00693
Game version: 01.00

Vita3K summary

Version: v0.1.5
Build number: 2658
Commit hash: https://github.com/vita3k/vita3k/commit/48f3e7ce

Test environment summary

Tested by: dj.tuBIG/MaliceX
OS: Windows 11 21H2 Build 22000.675
CPU: AMD Ryzen 3900X
GPU: NVIDIA RTX2070
RAM: 128GB (DDR4-3600)

Issues

Game now starting. Audio stutter / slowness identical report to https://github.com/Vita3K/compatibility/issues/624#issuecomment-1149826430, however faster than the sequel 2-2.
Similar to above report, crash during ingame first dialogue sections. (Related: [handle_access_violation]: Unhandled write protected region was valid. Address=0x4C)

Screenshots

Log
vita3k.log
Recommended labels

Crash
Ingame-

Vita3K summary

Version: v0.1.6
Build number: 2841
Commit hash: https://github.com/vita3k/vita3k/commit/0c75bbb5
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with no issues
Screenshots:

Log
vita3k.log
Recommended labels

Playable

Came to report the same as back in June 2022
Not playable and crashes near the start. To be exact, it is “after” the first 2 girls join the party by a few minutes. It is not during the long opening scene.
Same exact log as last poster, redundant to repost.

App summary

App name: Dungeon Travelers 2: The Royal Library & the Monster Seal
App serial: PCSE00693
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 3091
Commit hash: https://github.com/vita3k/vita3k/commit/e3956248
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: jinniper
OS: Windows 11
CPU: Intel(R) Core(TM) i5-12400F   2.50 GHz
GPU: NVIDIA GeForce GTX 3060
RAM: 32 GB

Issues
Title appears to have stable 30fps in most areas / situations I've tested besides menu overlays, large entity number combat encounters, etc. It seems to be tied only to the number / length of the list of things it needs to detail. i.e. the Equipment menu where there can be dozens of characters listed, in shops that have a plethora of list entries, so on and so forth.
Screenshots

(^ Plentiful items listed vs. when I clear the inventory out to a degree)

Log
Nothing to note
Recommended labels

slow
Ingame +

Came to report the same as back in June 2022
Not playable and crashes near the start. To be exact, it is “after” the first 2 girls join the party by a few minutes. It is not during the long opening scene.
Same exact log as last poster, redundant to repost.

Still unplayable (using it on android) from this day (Oct.13), everytime i save the game it crashes. I tested it while not saving and it still crashes from time to time.

Played on steam deck OLED via EmuDeck.
Slow with crackling sound and occasionally crashes after the intro video when starting a new game, always crashes when trying to save.

App summary

App name: Dungeon Travelers 2: The Royal Library & the Monster Seal
App serial: PCSE00693
App version: 1.00

Vita3K summary

Version: v0.2.0
Build number: 3691
Commit hash: https://github.com/vita3k/vita3k/commit/31826697
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Benjamin Toledo
OS: Windows 11
CPU: AMD Ryzen 5 4600G
GPU: AMD Radeon(TM) Graphics
RAM: 15 GB

Issues

No issues

Screenshot

Log
vita3k.log
Recommended labels

Playable


Added Navbar Hamburger design for smaller devices

I have changed the hamburger design of Navbar from

to

Resolves : #620

Kindly check whether it is okay


Login logic flow

In reviewing the Auth0 cleanup code it became clear to me that the flow for logging in has been a bit muddled and isn't very clear - more to the point, doesn't make sense where it's happening and doesn't seem like it will work correctly. Currently the login steps happen in the Navbar component, which doesn't make a lot of sense to me. In my mind, this should be logic that happens in the very beginning of loading the app, before components have started loading (not during the loading of a sub-component). There's also the fact that there is dummy-data being used in the back-end when loading a user profile that should be updated now that Auth0 in the FE should be working.
My proposal:

FE: Move the login logic from Navbar.js into the App component in index.js.

Get the necessary info from useAuth0() to send the profileId to the BE (there's a user object that is made available after the user has authenticated. Make use of isAuthenticated, also provided by Auth0).

BE: Update the logic in the authRequired.js middleware to actually use the profile, based on the profileId that is passed from the FE

This requires receiving the Auth0 profileId or email and retrieving the corresponding profile from the postgres DB, where we have the role recorded, which should be passed back to the FE and saved to state (used for many components)

Helpful reference to what can be destructured from useAuth0(): https://auth0.github.io/auth0-react/modules.html

Thanks @mcfrr3 we ended up capturing this work and I forgot to close out the issue.


add statistics

This adds a statistics command. Pretty simple: just pull out the number of datasets in which a concept occurs.
Here's the datasets occuring in 50+ datasets:
 3407   239  FLOWER                                                    50
 3408   730  SNAKE                                                     50
 3409   906  TREE                                                      50
 3410  1360  DUCK                                                      50
 3411  1502  GOAT                                                      50
 3412  1622  SHIRT                                                     50
 3413   587  ANT                                                       51
 3414   772  BELT                                                      51
 3415  1208  CAT                                                       51
 3416  1313  MOON                                                      51
 3417  1381  SHOE                                                      51
 3418  1775  TABLE                                                     51
 3419   978  WAVE                                                      52
 3420  1223  HEART                                                     52
 3421  1297  LEG                                                       52
 3422  1352  KNIFE                                                     52
 3423  1394  BONE                                                      52
 3424  1430  STAR                                                      52
 3425   227  FISH                                                      53
 3426  1476  CHAIR                                                     53
 3427  2009  DOG                                                       53
 3428   937  BIRD                                                      54
 3429  1343  SUN                                                       54
 3430  1663  BED                                                       54
 3431   615  HORSE                                                     55
 3432  1248  EYE                                                       55
 3433  1489  CLOUD                                                     55

It also shows (left entry) how many concepts are actually linked, etc.
I think, @AnnikaTjuka, this should be enough for you to create a histogram or do other investigations with the data. E.g., simple statistics can already be derived if you e.g., manually carve out the most frequent 100 concepts and check with pyconcepticon, how many of those are also in the Swadesh list of 100 concepts.
You can output the data in various formats using
norare stats --format tsv > stats.tsv

There is whitespace in this output (not my favorite) but this can be deleted with a regular expression, I assume.
Let me know if there are problems with this, but for the paper, I assume, mentioning the most frequent concepts in a table of 10 top, and maybe vs. this flop 10, or similar, may even be enough, comparing to Swadesh or similar is maybe even not important. And I think one can make an easy histogram out of this, and this would probably better be plotted in R anyway, right? for consistency?

Codecov Report

Merging #18 into master will decrease coverage by 2.61%.
The diff coverage is 42.10%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   98.19%   95.57%   -2.62%
==========================================
  Files          20       21       +1
  Lines         388      407      +19
==========================================
+ Hits          381      389       +8
- Misses          7       18      +11

Impacted Files
Coverage Δ

src/pynorare/commands/stats.py
42.10% <42.10%> (ø)

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 8adfa9a...6e0d84c. Read the comment docs.

Cool, thanks! I think that's all I need for the NoRaRe stats section and the histogram. I'll let you know when I updated the Overleaf document.

Oh, and we want tests! :)

Yep, tests, I have a question: how DO I test here? I could not find any test of the commands when I just looked for it...).

Here's one https://github.com/concepticon/pynorare/blob/8adfa9a505a8c028e9e98924ed80ead47cbddf0e/tests/test_cli.py#L15-L21

perfect, didn't search well enough then obviously, will add this in a
minute.

@LinguList test_cli doesn't follow the usual "find tests for module xyz in tests/test_xyz" semantics - but I didn't want to call the module test___main__ :)

Okay, we have a meeting with Russell now, so I'll have to do this in
about an hour, but I hope it'll all work nicely (I addressed the other
two issues already).

Okay, I added tests now.

We wanted to make a concepticon release soon, anyway, right @chrzyki ? Also releasing norare-data - say v0.1 - would be ok with me.

As to the concepticon 2.4 release: this may take a bit longer, I am afraid, if we aim to have all lexibank datasets covered by then.

Okay. I just thought: if we fiddle a lot with things and don't want to
update the website again, we could have pure technical releases, right?
This would make it easier, as we will always have to decide for authors,
how to quote, texts, etc.

Hm. I think the releases should be properly citable - but I agree that not every release needs to be online at concepticon.clld.org (in particular if this would only be the case for a couple of weeks)

We wanted to make a concepticon release soon, anyway, right @chrzyki ? Also releasing norare-data - say v0.1 - would be ok with me.

Yes, I also think that a release sooner than later is for the better. Do you have any preferences with respect to the exact time frame? We can prioritise issues in concepticon-data and push them back for 2.5 in autumn.

We need the release in this week.

And regarding citability, I'd propagate the following policy: we allow for smaller releases, to avoid that we have to make checkouts, in these releases, however, the citation on Zenodo stays the same, only different version number, for 2.3.1. etc., so we don't have to discuss a lot here. And with the .zenodo.json on github, this is just as much as writing a new release on github, right?

the alternative: for the submission of norare, we just use a specific checkout point, and make a label or a pre-release?

Hm. I would find a patch release 2.3.1 which adds new data - rather than
just corrects errors - unintuitive.
Do you think the bookkeeping for a proper release would be that much work?
Johann-Mattis List<EMAIL_ADDRESS>schrieb am Mo., 20. Juli
2020, 18:06:

the alternative: for the submission of norare, we just use a specific
checkout point, and make a label or a pre-release?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/concepticon/pynorare/pull/18#issuecomment-661134486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGUOKEEADEK2LHPMDOY5DTR4RTQXANCNFSM4PCFOQQQ
.

I expect it to be half a day of panic, when everybody wants to make sure
the best things go into that release, and I cannot really live with this
panic before end of August. I also want to have time to look at things
myself, and I won't have it now. But I'd say: we just want to submit a
paper, everybody can understand for the review phase, that we just say:
git-clone the concepticon now, please. Right? And we can even include
the checkout branch that they should use for now, if we wanted...

Ok. Then I'd opt for pre-release, I.e 2.4.b1 or similar.
Johann-Mattis List<EMAIL_ADDRESS>schrieb am Mo., 20. Juli
2020, 18:25:

I expect it to be half a day of panic, when everybody wants to make sure
the best things go into that release, and I cannot really live with this
panic before end of August. I also want to have time to look at things
myself, and I won't have it now. But I'd say: we just want to submit a
paper, everybody can understand for the review phase, that we just say:
git-clone the concepticon now, please. Right? And we can even include
the checkout branch that they should use for now, if we wanted...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/concepticon/pynorare/pull/18#issuecomment-661153123,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGUOKGYWOG4QXYIJKZMW5TR4RVWPANCNFSM4PCFOQQQ
.

yes, maybe the best.


Issues for introducing static factory functions named from

Description
Based on #242, we would like to create issues for introducing an experimental static factory function named from for types encapsulating validations in the kotools.types.experimental package.

Here's the list of issues created during the resolution of this one:

#342
#347
#349
#350
#351
#341
#352
#353
#354
#355


deal with spurious wakeups of Object.wait()

As in the one argument version, interrupts and spurious wakeups are possible, and this method should always be used in a loop. [1]

[1] http://docs.oracle.com/javase/7/docs/api/java/lang/Object.html#wait(long)

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test


Reward (Objective function) calculation is missing

Dear Authors,
I would like to know that where did you calculate the reward function.
I am unable to find the sum spectral efficiency.
In main. py file, Line 200 utilises a_EE,a_SE = env._compute_EE_(), but compute_EE is not explicitly defined.
Regards.

I am sorry that the repo is SOLELY UTILIZED FOR DEMONSTRATION PURPOSES because our work is still being extended. The full version of the code will be reuploaded once the extended work and paper is published. Many thanks for your interest in our work :)