How to set callback for accepted permission

Hi,
According to the readme, success callback passed to askPermission:
askPermission(Manifest.permission.READ_CONTACTS, Manifest.permission.ACCESS_FINE_LOCATION) {
   //all of your permissions have been accepted by the user
}.onDeclined { e ->
   //at least one permission have been declined by the user
}

But source shows that we are passing ResponseCallback, not AcceptedCallback. And there is not way to pass AcceptedCallback out of the box:
askPermission().onAccepted

The only way is to create new fragment extension:
fun Fragment.askPermission(vararg permissions: String, accepted: (PermissionResult) -> Unit): KotlinRuntimePermission {
    return KotlinRuntimePermission(
        RuntimePermission.askPermission(activity)
            .request(permissions.toList())
            .onAccepted(accepted))
}

btw response callback in this case is always called only if success

`private void onReceivedPermissionResult(List acceptedPermissions, List refusedPermissions, List askAgainPermissions) {
final PermissionResult permissionResult = new PermissionResult(this, acceptedPermissions, refusedPermissions, askAgainPermissions);
if (permissionResult.isAccepted()) {
for (AcceptedCallback callback : acceptedCallbacks) {
callback.onAccepted(permissionResult);
}
for (PermissionListener permissionListener : permissionListeners) {
permissionListener.onAccepted(this, permissionResult.getAccepted());
}
}
if (permissionResult.hasDenied()) {
for (DeniedCallback callback : deniedCallbacks) {
callback.onDenied(permissionResult);
}
}
if (permissionResult.hasForeverDenied()) {
for (ForeverDeniedCallback callback : foreverDeniedCallbacks) {
callback.onForeverDenied(permissionResult);
}
}
    if (permissionResult.hasForeverDenied() || permissionResult.hasDenied()) {
        for (PermissionListener permissionListener : permissionListeners) {
            permissionListener.onDenied(this, permissionResult.getDenied(), permissionResult.getForeverDenied());
        }
    }

    for (ResponseCallback responseCallback : responseCallbacks) {
        responseCallback.onResponse(permissionResult);
    }
}`

As you can see, this is definitely not the case. The response callback is always called no matter what. I wouldn't necessarily say that's counter intuitive, technically the callback is called when permissions have successfully been responded to, however, the documentation does say that it would only be called if all permissions are accepted.


Add validity check for zero approvals in a Deploy

CHANGELOG:

Added a condition during a deploy's validity check that the approvals field must be non-empty
Added a new DeployValidatorFailure variant EmptyApprovals to be raised if a deploy's approvals field is empty
Added a unit test to check that deploy's with empty approvals raise a DeployValidationFailure

Closes #1646

Ah, also one thing that I missed, due to the following lines not being a part of the PR:
https://github.com/casper-network/casper-node/blob/0369311898cc7cf05593a4f4d96be732c801786e/node/src/types/deploy.rs#L689-L692
I think the comment is now a bit rotten and should also contain a bullet about approvals not being an empty set.

bors r+


Search icon in mobile view

If search is set to false, the search icon should not be there in mobile view.

In our application we don’t have a search bar, but when you switch to mobile view we get the search button (highlighted in the image below). Can this be disabled through the template?
Gaétan Tauvette
Programmer Analyst, Service Management
Transport Canada / Government of Canada
<EMAIL_ADDRESS>Cel:<PHONE_NUMBER> / TTY: 1-888-675-6863
Programmeur analyste, Gestion des services
Transports Canada / Gouvernement du Canada
<EMAIL_ADDRESS>Cell:<PHONE_NUMBER> / ATS : 1-888-675-6863

I tested CDTS versions v4_0_27 to v4_0_32. There was a minor issue with hiding the search button in mobile view.
The issue was fixed in v4_0_31. The issue the client has on his page is not related to the template. I recommended to upgrade the template to v4.0.32, since we are supporting only three last versions. I provided recommendations how to fix the issue, with search button in mobile view. I also recommend to review the HTML code, there are some semantic issues.


vscode: add workspace to make gopls capable of handling the monorepo

gopls can't handle go.mod in subdirectories so we have to ignore them from the top level and then add them as additional folders

merged manually


ImageMagick install error on Linux Mint 19.1

I saw this error when running slipstream on a fresh install of Mint 19.1: https://github.com/Homebrew/homebrew-core/issues/36000 (C compiler error when installing imagemagick via Linuxbrew).
I ran brew link --overwrite libomp then ran slipstream again and it seemed to fix it.

Weird. I didn't get a notification email for this. Yeah, linuxbrew's a bit of a moving target right now. When packages are updated, there's a lag before bottles (pre-compiled binaries) are generated (some a few hours, some days, some weeks.)


Improve collection of Zone Group State for large Sonos systems (partial fix for #518 / #935)

I'm a little hesitant about this approach:

Not every network can handle subscription callbacks properly.
This opens/closes a listening socket every time the ZGS is requested, which could cause other side effects.
What happens if a different event handler has already been loaded, such as the async implementation?

Thanks for the comments, @jjlawren.

Not every network can handle subscription callbacks properly.

The events-based approach is only invoked if a normal query fails. It's a last resort, so if it fails due to network limitations it's still better than not trying.

This opens/closes a listening socket every time the ZGS is requested, which could cause other side effects.

The cacheing logic limits ZGS refreshes to at most once per 5 seconds by default (using POLLING_CACHE_TIMEOUT). That seems reasonable to me.

What happens if a different event handler has already been loaded, such as the async implementation?

The logic currently only handles the standard events module. I'll try adding support for the others, although I could apply the same 'it's better than nothing' rationale as for (1).
Of course, this is all predicated on this approach actually working for large speaker populations! Reading through the issues (e.g., https://github.com/SoCo/SoCo/issues/518#issuecomment-356750673) makes me optimistic that it might. I'm looking forward to seeing some results from the use of your test script on suitably large systems.

Hi @jjlawren:
I've now added support for events_asyncio which appears to work but, due to my inexperience with asyncio development, clearly has room for improvement.
If you get a moment, perhaps you could take a look at ~ lines 181-207 and point out the right way to approach this? In particular I couldn't figure out how to retrieve the ZGS from the callback without resorting to a global.

I'd really like to have confirmation that this is the actual underlying issue first.
In my downstream implementation that uses soco, I think I'd actually prefer to know that the polling call failed (at least optionally) so I can handle the issue myself by creating my own long-lived subscriptions. Perhaps with a special exception that better describes why this is failing, like SonosNotSupported to indicate it's a Sonos-side limitation. The automatic fallback fixes here are complex and are not risk-free.

I'd really like to have confirmation that this is the actual underlying issue first.

Agree.

In my downstream implementation that uses soco, I think I'd actually prefer to know that the polling call failed (at least optionally) so I can handle the issue myself by creating my own long-lived subscriptions. Perhaps with a special exception that better describes why this is failing, like SonosNotSupported to indicate it's a Sonos-side limitation. The automatic fallback fixes here are complex and are not risk-free.

I think I'd like a default that works without plunging users into having to understand how to subscribe to and process ZGT events, etc. I take your point, though, so we should provide a config option to opt out of the fallback behaviour.

In its current state this PR is a sticking plaster that falls back to using a ZoneGroupTopology event to determine the ZoneGroupState if a direct GetZoneGroupState() lookup fails.

The fallback behaviour can be disabled by setting config.ZGT_EVENT_FALLBACK to False, in which case the SoCoUPnPException can be handled elsewhere by the caller
The fallback only works when using the standard soco.events module, not soco.events_twisted or soco.events_asyncio

It's likely that if GetZoneGroupState() fails once then it will continue to fail. Hence, an improvement after the first failure would be to avoid the expense of the initial GetZoneGroupState() call and use the event fallback to begin with. This should probably be coupled with a periodic reset to confirm that GetZoneGroupState() is still failing.

After the first fallback to using events, this now becomes the default behaviour within EVENT_FALLBACK_TIMEOUT (currently set to 120s). This saves GetZoneGroupState() calls that will probably fail.

Hi @jjlawren: From my perspective this is ready to merge now, and your review would be welcome.
There's support for falling back to using a ZGT event under the default events module and when using events_asyncio. Support for events_twisted can be added later by someone with the required expertise; the code scaffolding is there for it.
Hi @KennethNielsen FYI, in case you want to take a look.
I'd like to land this in v0.29, to be released within the next week or so.

Thanks @jjlawren and @KennethNielsen for taking the time to review this PR, and thanks also to @jjlawren for the insight that led to creating this workaround.
It still lacks support for events_twisted (I couldn't get this to work), but I suspect the Venn diagram intersection of users with very large Sonos systems and events_twisted users is rather small.
I'll merge this in the next day or two, and release SoCo v0.29.


[Storage] az storage account file-service-properties update/show: Add File Properties Support for Storage Account

History Notes:
(Fill in the following template if multiple notes are needed, otherwise PR title will be used for history note.)
[Component Name 1] (BREAKING CHANGE:) (az command:) make some customer-facing change.
[Component Name 2] (BREAKING CHANGE:) (az command:) make some customer-facing change.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S166.


Connection module: Should allow multiple subscriptions

Description
The "Connection" module, when does not allow to setup ExpressRoute connections in different subscriptions (with authorization key). According to the bicep resource it should be possible to include in the module :)
https://docs.microsoft.com/en-us/azure/templates/microsoft.network/2021-05-01/connections?tabs=bicep

I just migrated this issue over from CARML. However, the module is orphaned so it make take a moment to triage.

Hey @fabmas,
Please triage this issue when you get the chance 💪

Hey @julioas09,
Please excuse the late reply. The automatic issue assignment did not kick in as the issue was not created as a "module issue". Highly recommend going forward 😉


Add global variable table for jBallerina

Description:
We need the $title to implement global variables in debugger.
Steps to reproduce:*
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

@warunalakshitha , shall we take this up?

This will be fixed with https://github.com/ballerina-platform/ballerina-lang/issues/19451 since now instead of id and var name will be added as fields to class. Those fields can be obtained by debugger api.


Code coverage?

On top of quality checks with ktlint and detekt, it could be interesting to add coverage to your reports. I didn't find any mention of JaCoCo or any other option in your gradle files.

This is on my roadmap. Happy to prioritize it.

I was experimenting with jacoco.
There is jacoco Gradle plugin, but it looks that it needs a bit of additional configuration for Kotlin and Android. It is not always that straight forward due to the usage of Gradle Kotlin DSL.
On top of the above there are two Gradle plugins that can simplify the configuration process (for multiple modules and potentially multiple build variants):

arturdm/jacoco-android-gradle-plugin - plugin has to be applied at the subproject (module) level and it was not generating reports for me out of the box
vanniktech/gradle-android-junit-jacoco-plugin - this one looks promising because it can be applied at the root project level. It also provides some tasks to merge Jacoco reports. Unfortunately, this plugin does not support android dynamic feature modules so I have opened the [PR #158].(https://github.com/vanniktech/gradle-android-junit-jacoco-plugin/pull/158 )
(I also tried to run an updated version of this plugin locally, but it requires to provide some Android dependencies which is too much hustle for now)

Next steps:
Wait for PR to be merged.
If this will not happen soon then I may think about configuring this manually.

I've had some difficulties setting it up for my project, especially because I had my tests in a separate module and I had to concatenate source folders from the others modules for jacoco to report coverage on them. If you're interested, you can look at the /gradle/coverage.gradle file in my project httpmocker: https://github.com/speekha/httpmocker

Also Codacy and Codecov seem to interpret the reports differently. Codecov has a lot of missed lines (which are actually tested) that Codacy marks as covered. Not sure if that's because of a positive lack of accuracy on Codacy's part or a negative lack of accuracy on Codecov's part.

thx for the project, I will take a look.
This comparison between Codacy and Codecov is a nice catch - I will take a closer look at this as well. It should be quite easy to determine which one is right by analyzing Jacoco reports, running AS test coverage and comparing all results.

The differences I've noticed seemed linked to lines in kotlin that are interpreted as several different lines in Java: typically, a field declared in the kotlin constructor for instance that result in a constructor parameter, a field declaration, a getter and a setter in the byte code. So the same line can be marked as covered because the constructor parameter is executed but it can also be marked as not covered because the getter is not. Hard to decide which interpretation is the best, I guess.


Fixed starttime to now()-1h

Fixed issue that was causing multiple triggers for the same event with now()-1d instead of now()-1h
Required items, please complete
Change(s):

Fixed issue that was causing multiple triggers for the same event with now()-1d instead of now()-1h

Reason for Change(s):

Latest change was causing multiple triggers for the same event.

Version Updated:

Yes

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hello @ddamenova, Thanks for raising this PR.
This PR will be investigated and will provide you some updates before 21 November, 2023.

Hello @tduarte14 , Thanks for raising this PR. This PR will be investigated and will provide you some updates before 21 November, 2023. Until then can you please repackage the solution using V3 tool.

What is that? Can you tell me what to do please?

Here is the step by step process for repackaging the solution using V3 tool.

Here is the step by step process for repackaging the solution using V3 tool.

Hi, think you forgot the link


schema metadata propagation

This pr will focus on creating a more controlled method for propagating changes to schemas.

rerun tests

/blossom-ci


Demo EPG doesn't work - 404

@marcovandeveen I don't think this is still a problem. There was a bad deploy on October 12 as a result of us reorganizing some files for the sake of the build bundling, which I'm guessing was to blame here.  Closing the issue, but feel free to reopen if I've missed something.


Ways to Restrict Access?

Forgive me if I shouldn't be posing this question here.  I see if I use google apps to authorize the users that can log into the Simian itself and I can even stop people from the domain from seeing the page by setting ALLOW_ALL_DOMAIN_USERS_READ_ACCESS to False. But is it possible to keep my Simian install at appspot.com and use google emails to authorize and still restrict which users can even see the Simian page? I'd really would like to only have to pay for Appengine. What are my options?

It looks like if you set ALLOW_ALL_DOMAIN_USERS_READ_ACCOUNT = False, then
it will have the same effect on appspot.com; that is, it appears then no
users will be able to login unless explicitly added to one of the ACL
groups.
https://github.com/google/simian/blob/e298a94d79e242737a5022259e1b829eb1462db3/src/simian/mac/common/auth.py#L95
Please test and let me know what you find...

Line 95 in auth.py doesn't have that as an option.  The only place I saw ALLOW_ALL_DOMAIN_USERS_READ_ACCESS is in settings.py.  Is that something I need to completely change line 95? Also, I don't have a google apps account.  So I'm need to be able to authorize though normal gmail emails but still only let certain emails see the Simian page.

The link to auth.py:95 was simply evidence that I believe it will work as
your intend on appspot as well.  Set the value to False in settings.py,
then have someone with another @gmail.com account who is not an app owner
and not a member of ACL Groups give it a try.

Oh, I've done that w/ multiple emails.  Both mine and friends emails.  They can't see the Admin in the menu but they can still see the page and even open stuff about Munki packages as well.

I'm struggling to figure this out.  Are you positive you've toggled the
various to False, and redeployed to App Engine?
https://github.com/google/simian/blob/e298a94d79e242737a5022259e1b829eb1462db3/src/simian/settings.py#L39

Wow, I was editing the one in the Build folder.  Thank you so much!


Setup travis

No automated tests yet, would be a good idea to set them up.

They need to be fixed BTW:
$ go test
--- FAIL: TestHelp (0.00s)
	rpcserverhelp_test.go:16: RPC handler defined for method 'getcfilterheader' without also specifying result types
	rpcserverhelp_test.go:43: Failed to generate help for method 'getcfilterheader': no result types specified for method getcfilterheader
	rpcserverhelp_test.go:43: Failed to generate help for method 'getcfilter': getcfilter-extended
FAIL
exit status 1
FAIL	github.com/ltcsuite/ltcd	0.043s

Also, go fmt modify some files.


⚠ Samsung has degraded performance

In 0e041f4, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2887 ms

Resolved: Samsung performance has improved in cf27d77.


Fixes for test_geojson_performance with DRF >=3.5

The test_geojson_performance test was failing an assertion in DRF>=3.5   Fixed by adding fields='all' to the PerfSerializer class.

@GJdan thanks. I forgot about this because it's not in the build.


observability: integrate CPU & memory profiling with Grafana Pyroscope

Proposed by @arkbriar.

Grafana Pyroscope is an open source software project for aggregating continuous profiling data.
Grafana Pyroscope is fully integrated with Grafana allowing you to correlate with other observability signals, like metrics, logs, and traces.

Its Rust client has built-in integration with pprof-rs. We may investigate if it's possible to integrate with jeprof as well.

I'm not sure but I think it might also be possible integrate the jemalloc profiling 🤔

In addition, how about integrating await tree as a backend? It might be more helpful than CPU profiling especially for IO-intensive cases.

After some investigation, I'm afraid await-tree is not that compatible with Pyroscope:

The backends of Pyroscope should be sample-based. However, await-tree is not designed to be frequently captured as it'll acquire locks.
await-tree yields trees, instead of stacks. Aggregating the results may not be feasible.


Bug positioning the menu when using anchorPoint with an arrow.

React/React-dom version: 17.0.1
React-Menu version: 3.4.1
Describe the bug
Menu isn't positioned correctly when using an anchorPoint and arrow, but differently depending on the position, making it impossible to compensate for.
To Reproduce

Use an anchorPoint
Use an arrow.
See what happens when the anchorPoint is near the edge of the screen and when it's not.

Expected behavior
Arrow points to the anchorPoint both when near the edge of the screen and not.
Code snippets/CodeSandbox examples/Screenshots
https://codesandbox.io/s/react-menu-dark-theme-forked-elu58x?file=/src/App.js
Not near the edge of the screen.

Near the edge of the screen.

import { useState } from "react";
import { ControlledMenu, MenuItem, useMenuState } from "@szhsin/react-menu";
import<EMAIL_ADDRESS>
export default function Example() {
  const [menuProps, toggleMenu] = useMenuState();
  const [anchorPoint, setAnchorPoint] = useState({ x: 0, y: 0 });

  return (
    <div
      onMouseDown={(e) => {
        e.preventDefault();
        setAnchorPoint({ x: e.clientX, y: e.clientY });
        toggleMenu(true);
      }}
      style={{
        background: "#eee",
        position: "absolute",
        inset: "0"
      }}
    >
      Click to set anchorPoint.
      <div
        style={{
          left: anchorPoint.x,
          top: anchorPoint.y,
          height: "1px",
          width: "1px",
          position: "absolute",
          borderRadius: "50%",
          outline: "red 1px solid",
          outlineOffset: "4px"
        }}
      ></div>
      <ControlledMenu
        {...menuProps}
        arrow
        anchorPoint={anchorPoint}
        direction="top"
        onClose={() => toggleMenu(false)}
      >
        <MenuItem>Item 1</MenuItem>
        <MenuItem>Item 2</MenuItem>
        <MenuItem>Item 3</MenuItem>
      </ControlledMenu>
    </div>
  );
}

Hi @AmyAmy , can you try to set a offsetX to see if that's what you need?
<ControlledMenu
  {...menuProps}
  arrow
  anchorPoint={anchorPoint}
  direction="top"
+ offsetX={-15}
  onClose={() => toggleMenu(false)}
>

codesandbox
Similarly, if direction is 'left' or 'right', offsetY prop should be used.

That does fix it in the codesandbox, but not in my full app, I'll investigate further.

Ok, figured it out: the arrow size is expressed in rem in the default CSS, while the offset is in px. So there's some minor issues for users overriding the base font-size in their browser. But I can work around that.
Guess it's up to you whether you want to fix that in your components.

Thanks for the updates. Users should be able to figure out a proper offset value.
Like in this example, where font-size is 2rem on the root document, we can increase the offset accordingly to compensate for the arrow size.

Not exactly what I was talking about, but here is a working example, in case somebody finds this useful:
https://codesandbox.io/s/react-menu-dark-theme-forked-yle22h?file=/src/App.js
font-size is not set on the root, so the browser uses the default.
By default this is 16px in all browsers, but specific users may change this, for example for accessibility reasons. Note that this is not the same as zooming in.
In Firefox: by going to the menu, then Settings. On the General tab scroll down to Fonts and then change the font size.
In Chrome: by going to the menu, then Settings. On the Appearance tab, click Customize fonts. Use the Font size slider to change the font size.
You may need to refresh the page to see any changes.
The way around this is by just asking the browser what the current font-size when the menu opens, and keeping track of that with useState.


Forward "open" event from ReadStream for completeness.

I'm currently working on an Arduino project where I would like to be able to reliably open and close the serial port, potentially any number of times.
I thought it would be nice to have the open event from the fs.ReadStream emit to the serialport consumers, so I've added that based on the other events forwarded from the underlying ReadStream.
On a related note, while I am now getting the open event reliably, I have yet to get the close event from the serial port module.  Doing an open, close, open scenario, after the second open, the serial port on the Arduino is not reset and no data is available to read.  I've dug into the serial port module and node itself but haven't yet resolved what the issue is.  I'll keep plugging at this... but if you know any specific reason why this might not be working, that could be helpful.
I have run dtruss/strace on my test script and it appears that the serial port module actually opens and closes the serial port correctly, but the ReadStream in node is actually not closing when destroy is called.  This looks to be the stream of system events from the node I/O threads:
38512/0x229474:         8 thread_selfid(0x7FFF7A68DFB8, 0x0, 0xFFFFFFFF)		 = 2266228 0
38512/0x229474:        33 open("/dev/tty.usbserial-11IP1559\0", 0x0, 0x1B6)		 = 7 0
38512/0x229474:        40 write(0x5, "\0", 0x1)		 = 1 0
38512/0x229474:        52 psynch_cvwait(0x106DE8700, 0x100000100, 0x0)		 = 0 0
38512/0x229474:        68 read(0x7, "\0", 0xFF)		 = 10 0
38512/0x229474:        84 write(0x5, "\0", 0x1)		 = 1 0
38512/0x229474:       101 psynch_cvwait(0x106DE8700, 0x10100000200, 0x100)		 = 0 0
38512/0x22949c:        12 thread_selfid(0x7FFF7A68DFB8, 0x0, 0xFFFFFFFF)		 = 2266268 0
38512/0x2294f7:        14 thread_selfid(0x7FFF7A68DFB8, 0x0, 0xFFFFFFFF)		 = 2266359 0
38512/0x2294f7:        34 open("/dev/tty.usbserial-11IP1559\0", 0x0, 0x1B6)		 = 8 0
38512/0x2294f7:        38 write(0x5, "\0", 0x1)		 = 1 0
38512/0x2294f7:        44 psynch_cvwait(0x106DE8700, 0x20100000300, 0x200)		 = 0 0

To me, the most interesting thing above is that there is no close call and that the second open allocates a higher fd value, which doesn't happen in the serialport module, which did have the same fd allocated that was closed earlier in the process.  Thus it's like the fd for the first ReadStream is still open.
As a point of comparison, I have written a straight C++ test that can successfully open, close and re-open a serial port here: https://gist.github.com/3558773c22ad80c5028d.  So, it doesn't seem to be a device or driver issue.  The node code that I'm testing with is here: https://gist.github.com/fc4d3d7d664f348d05a9.

Update on the possible problem with serial port close.
I think this is possibly an OS bug in a few versions of OS X, regarding kqueue/EV_ONESHOT a la the following:
http://bugs.python.org/issue7401
https://svn.boost.org/trac/boost/ticket/5021
I have instrumented node.js in eio.c around the EIO_CLOSE switch item:
      case EIO_CLOSE:
printf( "%s pre EIO_CLOSE\n", __PRETTY_FUNCTION__ );
        req->result = close     (req->int1);
printf( "%s post EIO_CLOSE\n", __PRETTY_FUNCTION__ );
        break;

... and I never get the post EIO_CLOSE message for the ReadStream that is also using the serial port.
So... I might table the investigation of close problems, but I think the addition of an open event from this pull request is still a generally good thing.


Add Go 1.11 to Travis build matrix

Coverage decreased (-59.1%) to 0.0% when pulling 3b73f9366b7648e8ced1f1d12a06bc1b1408a8be on emirb:patch-1 into 0a88db3545c4089ce0f555f496bfc78c491699e5 on mattn:master.

Coverage decreased (-59.1%) to 0.0% when pulling 3b73f9366b7648e8ced1f1d12a06bc1b1408a8be on emirb:patch-1 into 0a88db3545c4089ce0f555f496bfc78c491699e5 on mattn:master.

Coverage decreased (-59.1%) to 0.0% when pulling 3b73f9366b7648e8ced1f1d12a06bc1b1408a8be on emirb:patch-1 into 0a88db3545c4089ce0f555f496bfc78c491699e5 on mattn:master.

Thank you


US369215: Test with java 17

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/CAFapi/job/CAFapi~opensuse-tomcat-jre11-image~US369215~CI


Updating the links to Offset.Earth for Treeware

We have started to try and track links to Offset.Earth for individual packages. This is the first step.

@jamesmills thanks for taking care of it! =)
Should I update all the other repos? Or is there something on your side that you will have to do?
PS: The FUNDING.md also contains a link to offset.earth

This is actually the first package I have made this PR to. We are working on trying to track trees back to the packages. I’ll go through the other packages once we have a few things confirmed. Thanks.


[BUG][SanityTest][0.5-nexus] Disable Fields Minus in grammar to avoid data correctness issue

What is the bug?
The Fields Minus Command fields - <field list> has been supported from https://github.com/opensearch-project/opensearch-spark/pull/698 which is required Spark 3.5.0+
But its grammar presented at the beginning of this repository created, which the - in fields - <field list> is ignored by default. So in branch 0.5-nexus whose spark version is 3.3.2 will cause data correctness issue (the command results equal to command fields + <field list>).

[Catch All Triage - 1, 2]


⚠ Bofrost.be has degraded performance

In ff9a3ef, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1795 ms

Resolved: Bofrost.be performance has improved in 347a895.


Generate and review validation reports: Nuxeo UCM Merced County HS (repository #239)

Step 2: Generate reports

[x] Generate validation reports (run validate_by_mapper DAG): https://7a8067cb-3b99-477e-a883-7e311175a9b4.c3.us-west-2.airflow.amazonaws.com/dags/validate_by_mapper_type/grid?dag_run_id=manual__2024-01-31T16%3A54%3A08%2B00%3A00&num_runs=50
[x] Add a comment (in this issue) to the output URLs
[x] Download the validation reports locally; prepend the collection registry number to the filename
[ ] Upload validation reports to the Rikolti Data Validation folder

Step 3: Review reports

[x] Review sample collections validation reports; leave notes in the spreadsheet
[x] Quickly (and not very deeply) scan through other validation reports (to identify if any new metadata fields surface)
[x] Synthesize notes at the mapper level
[ ] Share an update (and surface any anomalies) during a weekly check-in (Mondays at 3:00pm)

Step 4: Have data validation issues? Address & resolve validation issues (synthesized mapper-level):

[ ] Create a new issue (“Validation results: Rikolti mapper name“) and report/request fixes & investigations. Add to current sprint.
[ ] Add a comment to link to the new “validation results” issue.
[ ] Add a “blocked/paused” label to this issue. Include a status note in this issue title (e,g., HOLD for fixes; HOLD for code review; etc.)
[ ] Move this issue back to the “To Do” pile.
[ ] Reset tasks listed in steps 2-3 above; re-generate & re-review (repeat steps 2-3 above, as needed.)

Step 5: Have acceptable validation reports?

[ ] Move the valid (accepted) reports to the “VALID” folder in google drive.

[2024-01-31, 16:55:53 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/26976/vernacular_metadata_2024-01-31T16:54:23/mapped_metadata_2024-01-31T16:55:20/validation_2024-01-31T16:55:48.csv
[2024-01-31, 16:55:53 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#26976/vernacular_metadata_2024-01-31T16:54:23/mapped_metadata_2024-01-31T16:55:20/data/
[2024-01-31, 16:55:53 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27358/vernacular_metadata_2024-01-31T16:54:56/mapped_metadata_2024-01-31T16:55:28/validation_2024-01-31T16:55:53.csv
[2024-01-31, 16:55:53 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27358/vernacular_metadata_2024-01-31T16:54:56/mapped_metadata_2024-01-31T16:55:28/data/

Validation reports: https://drive.google.com/drive/u/0/folders/1J5DHpn4FH8eXMiPtLVxB8xVkAJy0EUbJ
No issues!

Accepted on 2024-02-05!


Substance as peerDependency?

We need to investigate how to use substance as a dependency correctly.
E.g. Stencila is using Texture as a library, but itself depends on substance.
If both have substance as a regular dependency, it may now happen, that Texture uses a different substance version than Stencila.
Don't know how to do this exactly. My first try would be, adding substance to the devDependencies and as a peerDependency.

@michael what is your take on this?

Sorry for the delay. I think what you proposed works well; ensures that Stencila does not end up with two different substance dependencies accidentally right? I mean at least you get warned.

No worries. This just happened to me while working on Stencila.
With the current preview versions npm does not give a warning. Hope that this would be different with regular semvers.

We need to see if npm is complaining once we have a regular version string.


Disable signup/login button until and unless user fills necessary fields

What is the feature about?
Disable login and signup buttons until and unless user fills the necessary fields (Email and Password) in case of login and (Email, Password, Confirm Password) in case of Sign-Up, and enable it back as soon as these fields are filled.
ScreenShots of the Issue

Would you like to work on this issue?
Yes

@iamareebjamal Thanks for guiding! Sorry for the mistakes. I'm new to this platform.

i want to work on this issue

@hrprt96 I am already working on this Issue.

Are you still working on this issue @shikhart98 ?

@mohak1283 yep

Are you @kshitijsuri90 still working on this?


Does Sharepoint in the Office 365 Education A1 eligible for Office 365 Extra File Storage?

As titled, can school purchase additional space for sharepoint in Office 365 A1? For example, the default storage for sharepoint is 1TB. School would like to have 16TB in total. Can school buy additional storage of 15TB?
Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 94e36d7b-9d38-d71f-e64e-938e7cdf860a
Version Independent ID: 770a6dc2-7af2-3a66-f580-98ebff5bc5c4
Content: Add storage space for your subscription
Content Source: Office365-Admin/subscriptions-and-billing/add-storage-space.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@officedocsbot assign @mypil

@CheongCS The best way to tell is to check to see if the add-on is available in the admin center on the Purchases Services page. If it's listed, then the answer is yes, they can buy additional storage.

Thanks, Cern McAtee.  My customer hasn't subscribed to Office 365 A1 yet. They are searching around any better solution to replace their current on prem file server.

@CheongCS - Thank you for submitting feedback.
From our understanding, the issue has been answered by cmcatee-MSFT. If you feel it hasn't been resolved please re-open this issue.
Thank you for engaging with the community here for the docs.

@officedocsbot close


Update version to 0.2.36, update package name

Why are these changes needed?
Updates the version number to allow us to release the pending bug fixes and features.

[!IMPORTANT]
In order to better align with a new packaging structure we have coming very soon, AutoGen is now available on PyPi as autogen-agentchat as of version 0.2.36. This is the official package for the AutoGen project.

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 29.67%. Comparing base (38cce47) to head (cf97c5e).
Report is 10 commits behind head on main.

Files with missing lines
Patch %
Lines

...gen/agentchat/contrib/retrieve_user_proxy_agent.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3592      +/-   ##
==========================================
- Coverage   30.49%   29.67%   -0.82%
==========================================
  Files         113      115       +2
  Lines       12284    12721     +437
  Branches     2602     2709     +107
==========================================
+ Hits         3746     3775      +29
- Misses       8210     8597     +387
- Partials      328      349      +21

Flag
Coverage Δ

unittests
29.67% <50.00%> (-0.81%)
:arrow_down:

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

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


an issue on the function client._getClientUrl

This function will take $_SERVER['SEVER_NAME'] as client uri rather than $_SERVER['HTTP_HOST'].
https://github.com/Jasig/phpCAS/blob/master/source/CAS/Client.php#L3537
When you config the server_name  of the web server as an expression (such as _,*.github.com), phpcas will get a wrong redirect url and then cause lots of warings.

Autodetection of the URL may fail in certain cases. You can always manually set the base URL if the autodetection does not work in your case with the function setURL(). That way you can use your own custom code to set/detect your individual URL.
The autodetection code tries to fix 95% of the use cases with autodetection. Any other special case has to be done with the manual overrides that are available for pretty much every variable inside of phpcas.

Thanks a lot.
phpCAS::setFixedServiceURL() works.


Notification Support

Hi, quick question: Does this package support notification handling?
Can you provide an example on how to receive notifications?
Thanks!

@stevenh Would you be open to a PR that adds support for notifications?
I'll have to rewrite the way that scanning is done though (an approach similar to Darfk/ts3 might make sense).

sure, PR away :)

One thing I noticed:
Why does ExecCmd and Ecec return a string array? The ServerQuery never returns more than one line as far as I know (ignoring the error message and notify messages of course).
Would you prefer...
A: The func's return an array even though it always contains only one element, or
B: The func's return a string?
The problem with B is that it would break backwards compatibility, even though it makes more sense imho.

IIRC we made it due to some command not returning any responses other than the OK / Error, that and it gave use flexibility in case there where multiple line responses in the future.
Off the top of my head having a method to enable notifications that returns a channel the caller can then read any notifications off might be a good way to go.
It may well be hard to intermixing standard commands with notifications work cleanly, not used them, so it may be easier to recommend a separate client command for notifications?

IIRC we made it due to some command not returning any responses other than the OK / Error, that and it gave use flexibility in case there where multiple line responses in the future.

That makes sense, I'll leave it that way then.

having a method to enable notifications that returns a channel the caller can then read any notifications off might be a good way to go.

I'll add a func to register events and one to set an event handler. I personally prefer a handler func over a channel but this could easily be changed if a channel makes more sense.

It may well be hard to intermixing standard commands with notifications work cleanly, not used them, so it may be easier to recommend a separate client command for notifications?

Yeah maybe but I don't really like the thought of having multiple clients for just one program and I think that I solved the problem anyway so it should also work with one client.


Case Construction

I'd like to create a case for this keyboard, with three varieties:

A bottom-plate case, to protect the electronics on the bottom of the PCB. It will still be open from the sides
A 3D-printable (and maybe eventually aluminum CNC-able) case
Just a PCB and top plate, with the ability to attach some sort of screw-in rubber feet instead of requiring sticky pads for the feet

Ideally one PCB design will be able to work for all three case styles.
Measurements

Currently the tallest component on the board is the USB-C connector, at 3.26mm, followed by the QSPI flash module at 1.95mm, and the kailh hotswap sockets (if they're used) at 1.85mm.

USB-C Connector

Flash Memory

Kailh Hotswap Socket

Constraints

The USB-C connector should still be mounted on the same side as the electronics for easier manufacturing
The reset and USB-boot buttons should ideally still be accessible in some way (side-mounted buttons? holes in the bottom plate?)
The design should require as few screws as possible, no nuts, and especially no custom 3D-printed spacers as I've used before.

Bottom Plate Design
Following on the design of other keyboards (such as the 7spro), there will be M2 spacers which go through the main PCB, with screws securing it to the top and bottom plates.

I would try to get away with 8mm spacers first if the USB connector isn't as tall as it says in the datasheet, otherwise 9mm spacers will have to do.
The screws should be as flat as possible to stay flush with the top and bottom plate. 3-4mm length should be fine.

3D Case Design
Instead of having a bottom plate, the PCB will rest on the bottom of the case, with a slot cut out for the USB connector. There should be raised edges on the top and bottom edges of the case which the PCB will sit on.
There should be slot cutouts of some sort on the PCB edge to allow it to pass through to the bottom, with equivalent "posts" on the case itself, which will prop up the top plate on its edges and keep it at the right height away from the PCB.

These can run along the top and bottom of the PCB, the sides may not be necessary.
The "posts" in the 3D case which support the top plate should have M2-sized holes in them to allow for screws to tap into
To Be Determined

[x] The drill radius needed to allow M2 spacers to pass through the PCB. (4.5mm)
[x] How to provide adjustable angles on the keyboard (#22)
[x] Are these design compatible with assembling the keyboard in a sane/easy order? For the 3D case version, the switches need to be able to be soldered to the PCB before the PCB is placed in the case.

From some testing, it seems easiest to snap in some "anchor" switches in the plate on the corners and middle, then progressively add switches from there. The combined force from the snap-in clips on the switches provides enough tension to keep the top plate separated properly from the PCB.

Cleanup tasks:

[ ] Move row 0 diodes (top row or F row) below their switches instead of above to keep them off the edge of the PCB
[ ] Move row 0 traces to be below the switch

If you are okay with an aluminum back plate, then using rivet nuts (an example) is one easy way to make custom feet attachable.

Here's a good video showing how a non-hotswap keyboard is expected to be constructed in a case similar to what I want to build:
https://youtu.be/017aGAJiNJg

Some findings from @mcginty for adding threading to the PCB:
https://www.keyelco.com/category.cfm/Spacers-Standoffs/Standoffs-Spacers-Inserts-Surface-Mount/id/1335
https://www.mouser.com/ProductDetail/Keystone-Electronics/24929?qs=Znm5pLBrcAJPd1Bwfdw1vA%3D%3D
These could be soldered directly to the PCB as a solid, no-fuss solution to adding threading points on the PCB.
Very similar to @skywhale 's suggestion of rivet nuts, but without the need of a special tool (besides a soldering iron) for attaching it.

Progress with the "PCB with threaded inserts" version of the case:

More progress!


Error when import the OPML file

I read the instruction and download the raw.opml file from the repo.
Then I use NetNewsWire to import the subscription.]
However it shows the errow below:

dac8-488e-b9f9-e3810786f2eb.jpg)
I tried to change the file name but it doesnt work.
Any problems about the origin OPML  file?

Update:
Change another RSS reader, error still exists:

Update: Issue done!

Do not download the opml file from Repo
Copy all the code in the file.
use a txt file and paste all the code.
then change .txt to.opml
import it to the RSS reader.


Spike Research for Azure Spring Initializr

Context
Getting started is a key driver in product adoption. Today to start a Spring project using Spring Cloud Azure libraries, developers need to first go to the Spring Initializr, come back to Azure portal/CLI to provision services and deploy the code. In addition, not all Spring Cloud Azure libraries are discoverable from the Spring Initializr .
So we decide to leverage the opensource projects to customize Spring Initializr for Azure.
Goal
The goal of this feature is to output:

A feature spec
A design spec
A roadmap of all features

This is the feature spec of Azure Feature-Spec-of-Azure-Spring-Initializr


Feature request: Multiple point object interaction

Right now, you can only hold an object at one point. I think it would be useful to have multiple points to grab an object. This would allow for more complex object interactions. My vision for this feature includes includes the following:

You can hold an object from any one of the points.
You can hold an object from multiple points.
Certain points can't be grabbed by the player and controlled with code.
Each point has a value to determine how much of an influence it has.

Let me know if any of this is unclear, and I can help clear it up.

There's a reset_transform_on_pickup option on packable objects which let's you hold it by wherever the hand was when it was grabbed (referred to as precise-grabbing in some frameworks).
I agree there should be options for multiple grab points, and in addition they should support overriding the hand poses. Consider holding a gun by the barrel or the grip would have different poses.

since the op was asking for multiple grabpoints and we thands to @Malcolmnixon got those now
i would suggest markin this as complete
the following sound alot game specific
*Certain points can't be grabbed by the player and controlled with code.
(what has this then to do with grab points?)
*Each point has a value to determine how much of an influence it has.

this would be more useful to be implemented as a line grab for Melee Weapons and long items.
Where u you grab point A with lets say the left hand and if u keep the trigger pressed and let go off the grip it slides down towards point B.
Once u press gripp while holding down trigger it could lock the location of hand grabpoint.

Ok, I was thinking it of enhancing it such that a single hand could hold the same pickable by different "handles". However it's challenging even thinking of a good example do perhaps it's use would be limited.
As such I think this is complete. We'll work on the two-handed grabbing as a separate item.


TEN CLI

Updated Makefile to reduce the commands to starting services
Updated Docker and DockerCompose to build for local
Created TEN cli and installer script (install_cli.sh) to simplify commands under a single API

i'm using bash + iTerm2, it seems the install_cli.sh script will return me
"/usr/local/Cellar/jenv/0.5.4/libexec/libexec/../completions/jenv.bash:15: command not found: complete"
after running the script it will turn my shell to zsh..

i'm using bash + iTerm2, it seems the install_cli.sh script will return me
"/usr/local/Cellar/jenv/0.5.4/libexec/libexec/../completions/jenv.bash:15: command not found: complete"
after running the script it will turn my shell to zsh..

I will spend some time to update the logic for supporting the different shells

i'm using bash + iTerm2, it seems the install_cli.sh script will return me
"/usr/local/Cellar/jenv/0.5.4/libexec/libexec/../completions/jenv.bash:15: command not found: complete"
after running the script it will turn my shell to zsh..

I will spend some time to update the logic for supporting the different shells

i take a deeper look and it seems it's running my .bash_profile and it's giving above errors..

i'm using bash + iTerm2, it seems the install_cli.sh script will return me
"/usr/local/Cellar/jenv/0.5.4/libexec/libexec/../completions/jenv.bash:15: command not found: complete"
after running the script it will turn my shell to zsh..

I will spend some time to update the logic for supporting the different shells

bash-3.2$ echo $(basename "$SHELL")
zsh
bash-3.2$

i think it's caused by incorrect shell detection. it's trying to use zsh run my bash_profile

i'm using bash + iTerm2, it seems the install_cli.sh script will return me
"/usr/local/Cellar/jenv/0.5.4/libexec/libexec/../completions/jenv.bash:15: command not found: complete"
after running the script it will turn my shell to zsh..

I will spend some time to update the logic for supporting the different shells

Is it really necessary to install the tool? Maybe ./cli run it is good enough, and can avoid issues if it updated.

Given the conversation in this thread, I will close this PR, make updates to the code and open a new PR when it's ready.


[SP-3573]-Backport of PDI-14913 - JSON Output Step does not pass data to Servlet when "Pass data to servlet" is enabled

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-json-plugin/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-xml-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-xml-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/step/StepMetaInterfaceExtended.java
  engine/src/org/pentaho/di/trans/steps/textfileoutput/TextFileOutputMeta.java
  engine/src/org/pentaho/di/www/RunTransServlet.java
  engine/test-src/org/pentaho/di/www/RunTransServletTest.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsonoutput/JsonOutputMeta.java
  plugins/kettle-xml-plugin/src/org/pentaho/di/trans/steps/xmloutput/XMLOutputMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -1.64%:small_red_triangle_down:
Line Change: -1.97%:small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.27%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.53%:small_red_triangle_down:
Complexity Change: -4.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.textfileoutput.TextFileOutputMeta

Complexity Change: -.36%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:
Line Change: -.13%:small_red_triangle_down:
Method Change: -.92%:small_red_triangle_down:

org.pentaho.di.www.RunTransServlet

Branch Change:  + 25.00%
Complexity Change:  + 26.09%
Instruction Change:  + 13.25%
Line Change:  + 17.20%
Method Change:  + 33.33%

org.pentaho.di.trans.steps.jsonoutput.JsonOutputMeta

Complexity Change: -.30%:small_red_triangle_down:
Instruction Change: -.02%:small_red_triangle_down:
Line Change: -.06%:small_red_triangle_down:
Method Change: -.91%:small_red_triangle_down:

org.pentaho.di.trans.steps.xmloutput.XMLOutputMeta

Complexity Change: -.74%:small_red_triangle_down:
Instruction Change: -.15%:small_red_triangle_down:
Line Change: -.28%:small_red_triangle_down:
Method Change: -1.72%:small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-json-plugin/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-xml-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-xml-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/step/StepMetaInterfaceExtended.java
  engine/src/org/pentaho/di/trans/steps/textfileoutput/TextFileOutputMeta.java
  engine/src/org/pentaho/di/www/RunTransServlet.java
  engine/test-src/org/pentaho/di/www/RunTransServletTest.java
  plugins/kettle-json-plugin/src/org/pentaho/di/trans/steps/jsonoutput/JsonOutputMeta.java
  plugins/kettle-xml-plugin/src/org/pentaho/di/trans/steps/xmloutput/XMLOutputMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 1.64%
Line Change:  + 1.97%

org.pentaho.di.repository.kdr.KettleDatabaseRepositoryCreationHelper

Branch Change:  + 1.64%
Complexity Change:  + .69%
Instruction Change:  + 2.43%
Line Change:  + 2.95%

org.pentaho.di.trans.step.StepMeta

Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.27%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.18%
Complexity Change:  + 1.35%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.33%:small_red_triangle_down:
Complexity Change: -1.85%:small_red_triangle_down:
Instruction Change: -.28%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileoutput.TextFileOutputMeta

Complexity Change: -.36%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:
Line Change: -.13%:small_red_triangle_down:
Method Change: -.92%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.www.RunTransServlet

Branch Change:  + 25.00%
Complexity Change:  + 29.17%
Instruction Change:  + 13.47%
Line Change:  + 17.20%
Method Change:  + 40.00%

org.pentaho.di.trans.steps.jsonoutput.JsonOutputMeta

Complexity Change: -.30%:small_red_triangle_down:
Instruction Change: -.02%:small_red_triangle_down:
Line Change: -.06%:small_red_triangle_down:
Method Change: -.91%:small_red_triangle_down:

org.pentaho.di.trans.steps.xmloutput.XMLOutputMeta

Complexity Change: -.74%:small_red_triangle_down:
Instruction Change: -.15%:small_red_triangle_down:
Line Change: -.28%:small_red_triangle_down:
Method Change: -1.72%:small_red_triangle_down:


how to print input data of model?

how can I print the input data as train and test of model?

You can add prin


Short solution needed: "Add row number to results" (clickhouse)

Please help us write most modern and shortest code solution for this issue:
Add row number to results (technology: clickhouse)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Add row number to results


Add in documentation that Node.js global vars may be keeped between deployments

Specially usefull information when using Webpack to drastically reduce Azure Function cold start.
I understand the global vars sharing when I tried to use babel-polyfill and got this error:

Error: only one instance of babel-polyfill is allowed

Babel polyfill is using global vars and although I was importing it only once (in my webpack.config), I constantly get this error, and I finally had to make a little wrapper around my Azure Function handle to make sure babel-polyfill is not already loaded in global vars before requiring it.
I suggest adding a global vars warning in the documentation (especially with all Node.js developers going to make a webpack bundle to reduce Azure Function cold start.
This issue is kind of related to https://github.com/Azure/azure-webjobs-sdk-script/issues/298#issuecomment-281779696 and attempts to make Azure Function work with Webpack.

@christopheranderson @yochay Shouldn't this be moved to the Functions repo, since it's about documentation?

@lindydonna - yes this should be moved

@christopheranderson Could you file a new issue in the docs repo to track this?  You have more context on the doc requirement.

This issue was moved to Azure/Azure-Functions#368


bumps version for release of 1.20.0

I don't see a version bump in the csproj?

I don't see a version bump in the csproj?

Current project version is already set to 1.20.0.

I don't see a version bump in the csproj?

The PR after the release is the one that will update the csproj.
See https://github.com/microsoft/kiota/pull/5521 and https://github.com/microsoft/kiota/pull/5523 for the previous release...


Remove fnConstruct in mouseUp

The fnConstruct function in mouseUp caused some trouble when a state was loaded, columns where moved randomly. After it was removed I didn't notice any problems.
It also resets the _ColReorder_iOrigCol which caused some problems when saving the state (the col Reorder was always sequential like [1,2,3,4 etc.]

LGTM! Thanks!


fix: DEV-2170: On project open set number of tasks defined by task per page

set default value on DataStore

/frontend build lsf master


Create KeyboardButton dynamically

Hello, i have a question. How can i create KeyboardButton dynamically, by looping datatable or list. I didn't find any example and i have no idea how to do it.

Sorry, it was programming question, i solve it by this way:
KeyboardButton[] Buttons = new KeyboardButton[DataTable.Length + 1]; int index= 0; foreach(DataRow row in  DataTable) { KeyboardButton NewButton = new KeyboardButton { Text = row[column_name].ToString() }; Buttons.SetValue(NewButton, index); index+= 1; }
You can delete issue


can tcmu be used on ubuntu systems

hi, the kernel version of the ubuntu system is higher than the centos. Can tcmu be used on ubuntu systems? thanks

Tcmu has always worked for Ubuntu, it’s a matter of features that are supported based on the different kernel levels

Tcmu has always worked for Ubuntu, it’s a matter of features that are supported based on the different kernel levels

That's good, thanks


Fix pip install --pre gcsfs

It is not possible to run pip install --pre gcsfs right now because the two latest pre-releases of aiohttp are broken (v4.0.0a0 and v4.0.0a1).
This PR proposes excluding these two versions from the requirements.
More details: https://github.com/fsspec/filesystem_spec/pull/1018

(NB: I confirmed in a local fresh environment that pip install --pre fails before this commit, and works fine after.)

Thanks for the follow up


Persist components' state across hot page reloads

When using built-in ASP.NET Core mechanism for hot reloading pages after changing source code, the whole page is being reloaded, therefore all the components loose their state. It's inconvenient when working on some places deep down the component hierarchy that resets every time.
Possible solution would be to add a custom hot-reload mechanism that will send messages via web sockets stream. As a response, browser could send a request to the back-end for the new version of current page, but additionally all the state would be sent together with that request.
To do:

[ ] Investigate
[ ] Implement

Could add a property bool IsPersistent to the HydroComponent class, so that every persistent component will save its props marked with [Parameter] attribute to cookies after every rendering, then upon a new Mount (it is triggered upon HotReload right?) will restore values from cookies.
Could be totally done from code-behind without changing hydro.js. Doesn't keep all the state though.. Well, just an idea. Can be interesting for not only HotReload, like navigating through pages and automatically having all previous parameters restored from last usage.


vtk[core] build failure

Host Environment

OS: Windows
Compiler: MSVC

To Reproduce
Steps to reproduce the behavior:
./vcpkg install vtk:x64-windows-static
Failure logs
Computing installation plan...
The following packages will be built and installed:
    vtk[core]:x64-windows-static
Starting package 1/1: vtk:x64-windows-static
Building package vtk[core]:x64-windows-static...
-- Using cached D:/vcpkg/downloads/Kitware-VTK-v8.2.0.tar.gz
-- Using source at D:/vcpkg/buildtrees/vtk/src/v8.2.0-d1dd9405dd
-- Configuring x64-windows-static
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
    Command failed: ninja -v
    Working Directory: D:/vcpkg/buildtrees/vtk/x64-windows-static-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      D:\vcpkg\buildtrees\vtk\config-x64-windows-static-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:295 (vcpkg_execute_required_process)
  ports/vtk/portfile.cmake:94 (vcpkg_configure_cmake)
  scripts/ports.cmake:90 (include)

Error: Building package vtk:x64-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: vtk:x64-windows-static
  Vcpkg version: 2020.02.04-nohash

vcpkg/buildtrees/vtk/config-x64-windows-static-out.log
vcpkg/buildtrees/vtk/x64-windows-static-rel/CMakeFiles/CMakeError.log
vcpkg/buildtrees/vtk/x64-windows-static-rel/CMakeFiles/CMakeOutput.log
vcpkg/buildtrees/vtk/x64-windows-static-dbg/CMakeFiles/CMakeError.log
vcpkg/buildtrees/vtk/x64-windows-static-dbg/CMakeFiles/CMakeOutput.log

export(TARGETS ${_vtk_compiletools_targets} failed. But I can't repro this issue now.

I have modified the vcpkg\buildtrees\vtk\src\v8.2.0-d1dd9405dd\CMakeLists.txt using the fix of the issue you mentioned and now vtk seems to install correctly.
At the end I get a note saying that some targets were omitted. I don't know if this may cause problems when I go and use it (I'll test later).
find_package(VTK CONFIG REQUIRED)
# Note: 102 target(s) were omitted.
target_link_libraries(main PRIVATE foo vtksys somelib verdict)

I have modified the vcpkg\buildtrees\vtk\src\v8.2.0-d1dd9405dd\CMakeLists.txt using the fix of the issue you mentioned and now vtk seems to install correctly.
Thanks, I'm closing the issue :)


Searching for tweets in Japanese word "-" operator does not work.

I am using Python 3.9.2
install with
pip3 install --user --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint
Command Line
twint --search "カツカレー ランチ -神戸" | less
Description of this issue
Let me give you a simple example.
I tried to exclude words containing Kobe(神戸) in this format
However, in fact, the string "kobe"(神戸) was not excluded, but rather included in the search. I tried this with other Japanese words, but the same thing happened.

Note that the exclusion operator worked fine when searching for English tweets, so it's possible that this problem occurs with 2-byte characters in general.
And to make matters worse, there is no other suitable alternative program or library f, so I'm in a bit of a hurry. Please hurry as much as you can. Please help me.
If you have any questions, I will do my best to answer them. Please do.
Environments
ArchLinux:GNOME3
Android10:Termux

Same problem in Spanish and French. In English it does not work well either (10% of tweets have one "no keyword" from my experiments this morning). Maybe because of the way the query is written.
I think adding in a config.py file line 9 something like
NoSearch: Optional[list] = None

and then in url.py just before q=q.strip():
    if config.NoSearch:
        q += f" - ({})".format(' OR '.join(config.NoSeach))

May work, but I haven't try it yet.


Add feature to stop and start AWS RDS clusters associated with the namespace

https://github.com/govirtuo/kube-ns-suspender/issues/104: Add feature to stop and start RDS clusters associated with the namespace

Hi @apapia!
Did you try this at home or in a prod-like env? Unfortunately I can't easily test this

@eze-kiel I've had it running in our dev and qa kubernetes clusters for a few months with a lot of usage and is working great.

Well, LGTM! Thanks for your contribution!

Hi @apapia, it seems that there is an issue with the image building workflow :/ I'll investigate


fix(Pagination): Improved accessibility for classic pagination

Issue: #1388 #1387
What I did

Updated aria-current from true to page
Added aria-label="pagination". This can be customized by adopters as well in case there are multiple paginations on the page.

Screenshots:

Checklist

[X] changeset has been added
[X] Pull request description is descriptive
[X] I have made corresponding changes to the documentation
[X] New and existing unit tests pass locally with my changes
[X] I have added tests that prove my fix is effective or that my feature works

How to test

Verify the use of aria-current
Verify that aria-label is visible on the nav property

-Verify the use of aria-current ✅
-Verify that aria-label is visible on the nav property ✅
All set


Fixes issue #356 : Keychain with protocolType and accessGroup

Refer issue #356 for detailed discussion.

@yangyubo Thanks! Somehow I overlooked existing behaviour. It looks good to me. I will merge soon after I test compatibility.

@kishikawakatsumi it would be great to get this rolled in if you're happy with it 👍

I've tested this now in my sample app (https://github.com/jeff-h/sharedKeychainAccessDemo) and it works great there. Would be great to see this PR rolled in.

Superseded https://github.com/kishikawakatsumi/KeychainAccess/pull/450

Superseded https://github.com/kishikawakatsumi/KeychainAccess/pull/450

Fixed by https://github.com/kishikawakatsumi/KeychainAccess/pull/450


[T4A1][T01-T3]Naren V

Ready for review

@radneran
Some comments added. Please close the PR after reading comments.


Unable to fund the multisig pool account

I have enough SATS but still unable to make the deposit to multisig
Pls suggest and thank you in advance

Are there enough confirmations on the funds in the LND wallet? It needs 6 confirmations before you can then use the funds to fund a Pool account.

The error sounds like an issue with paying the LSAT required to open an account. For that you need at least one active channel with the ability to pay for a 1000 satoshi invoice. The litd logs should have more info as well.
See https://docs.lightning.engineering/lightning-network-tools/pool/quickstart#prepare-lnd.

The error sounds like an issue with paying the LSAT required to open an account. For that you need at least one active channel with the ability to pay for a 1000 satoshi invoice. The litd logs should have more info as well. See https://docs.lightning.engineering/lightning-network-tools/pool/quickstart#prepare-lnd.

Yes that may be correct @guggero. After setting 1000 LSAT fee I got his. If you have BTC testnet balance, kindly send 0.1 BTC to tb1pyy82f5y9zqme90yxy2dw2y9rs6ppexgf4cpwql2cylld5dte2nasazw4xu
Thank you in advance.

I sent you some testnet coins to the address. Closing this issue as it sounds like you were able to resolve it. Please join our Slack server for further troubleshooting.

I sent you some testnet coins to the address. Closing this issue as it sounds like you were able to resolve it. Please join our Slack server for further troubleshooting.

Thank you @guggero  for tokens and support.


[MydealsBridge] Removed

The Mydealz.de (wrongly named mydeals) is not working anymore (it has some weird dependencies in it and the site has been reworked since the bridge was made).
The website itself provides rss feeds for basically every feature of the site (including personalized feeds), so there is no use for the out of date bridge.
See
https://www.mydealz.de/faq-benachrichtigungen#rss

I'm the author of this bridge, and yes, it has dependency on another Bridge, because it would be a nonsense to duplicate code for every Pepper Bridge.
I'm still using this bridge daily, and it still works fine.
I opened the pull request #2083 to update the group list to the actual list.
This bridge should not be removed; the integrated RSS feed does not allow to search the website, and doesn't show all the information (original price) my bridge shows.

Hi,
Woops, I'm sorry, my installation was screwed so the DealabsBridge was missing from my code. I checked the repo and found it, my bad.
closed


How to alter a column using the schema builder?

Is there an equivalent of knex.schema.table() such as table.column('foo') ? I'm trying to change the default of an existing column in a migration file.

Right now you have to do that with knex.raw. Hardly ideal but updating the schema API allowing stuff like this is going to be a big undertaking.

How exactly would you use knex.raw in such a case?
knex.schema.table('foo', function (table) {
// ...??
});

I think that:
knex.schema.raw('alter table foo modify column bar varchar(255)')

should do it (you chan chain .raw off schema and it will run in order). But yeah this is something that I plan to add better support for soon.

Okay, thanks.

Here's the syntax I ended up using for altering a column type:
knex.raw('ALTER TABLE "' + table + '" ALTER COLUMN "' + column + '" TYPE ' + newType + '')

there is nowadays .alter()which can be used to change column type. http://knexjs.org/#Schema-alter


[Fix #4321] Allow self to receive methods also defined on Kernel

Method calls with an explicit receiver of self are ok when a method with the same name is also defined on Kernel. Rubocop can't know if the explicit self is disambiguating a method call or not.
Fixes #4321
(Perhaps there should be another cop warning about defining methods that are also defined on Kernel? 🤔 )

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Apart from my small remark, I think that's a good improvement.

(Perhaps there should be another cop warning about defining methods that are also defined on Kernel? 🤔 )

Yeah, I totally agree. No idea why we haven't implemented this yet. Should be super simple.

I just realized it would have also been nice to update the docs/examples.

I figured this was an edge case that didn’t need the visibility but I didn’t think about it much.

Well, that's not that different from having local variables named the same way as a method. :-)


Content Negotiation Strategy

Added code files needed to implement content negotiation strategy to projects - spring-mv-java and spring-mvc-xml

Files to be merged for spring-mvc-java project
1.ContentManagementWebConfig.java
2.EmployeeController.java
3.Employee.java
4.pom.xml (for json dependencies)
5. Files in WEB-INF/view folder (employeeHome.jsp, employeeView.jsp, error.jsp)
Files to be merged for spring-mvc-xml project
1.EmployeeController.java
2.Employee.java
3.contentManagementWebMvcConfig.xml
4.pom.xml (for json dependencies)

Closing this pull request. Will open a new one. Thanks.


Update pre-commit hooks

Fixes #
Proposed Changes

Codecov Report

Merging #66 (ffcf582) into main (26662a6) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files


Add linebreak to toEqualImmutable message for better comparison

This pull request update the error message of toEqualImmutable, It is helpful for comparing large Immutable Objects:
//previous
Expected Map { "id": 2, "name": "Jack" } to equal Map { "id": 1, "name": "James" }

//pull-request
Expected
Map { "id": 2, "name": "Jack" }
to equal
Map { "id": 1, "name": "James" }

Jest does a colored diff of the expected and actual results, which would be ideal. But this is an improvement over what we currently have, so I'll merge it. Thank you!


Review tool only keeps one version of a review document but gives the impression that it keeps all of them.

The review tool maintains only a single image of the review document, kept on disk with no version number. "Revising" the review overwrites that single image, but tell the related Document object that a new version has been created. The views of that object then show the same bits for all versions, leading to confusion.
There's no way to reconstitute earlier versions of the reviews to date. We should either change the UI to note that the review document is mutable (as it currently is on disk) or change the storage to match every other document type and actually keep versions.

change the storage to match every other document type and actually keep versions.

is what the code was originally intended to do and is what the reviewers and consumers of the review expect.


Connect to red bull

As a sponsor only?

No , as a drink partner for the hackathon , try to find some email id/email id of the manager of Delhi/NCR and connect to him/her asap

Yes, sure


Add missing repo vars for rhel-9.x

Hi Bernd, we need to add missing repo vars for rhel-9.x

Replaced by https://github.com/sap-linuxlab/community.sap_install/pull/267.


Portuguese translation - phrasing alterations

Review and changes of phrasing to improve comprehension.

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] None of my changes are plagiarized from another source without proper attribution.
[X] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Congratulations on your first merged pull request into freeCodeCamp. 🎉🎉🎉🎉


persistent websocket not persisting with pingTimeout set

According to the documentation on the readme of this repo,

To prevent this state PWS let's you set a specific timeout after which to force a reconnection if you did not receive any messages from the server.

However, what I observe is that if I set pingTimeout, the socket connection closes after that amount of time and does not re-connect.

Oh wow. That's quite the regression. I've fixed it now, released in 4.3.0.

Thanks!


Make two distinct nightly profiles kind-sequential and kind-parallel

For kind-sequential
https://build.weblogick8s.org:8443/job/wko-tmp/41/
For kind-parallel
https://build.weblogick8s.org:8443/job/wko-tmp/40/

@anpanigr When I run that test I purposely commented out some tests to quickly get result. Let me re-run full sequential suite.

Please see the latest full sequential run: https://build.weblogick8s.org:8443/job/wko-tmp/41/
Thanks.


Data from ccfddl.github.io and ccfddl.top are not synchronized

Some conferences that have not yet closed appear on ccfddl.github.io but not on ccfddl.top in time, such as vldb 2023.

It has been fixed. Note that, for simplicity, ccfddl.top will only show the most recent one-year conference, when there are two years of conferences not yet closed.

Thank you for the prompt feedback. ccfddl.top looks very concise and convenient. However, vldb 2024 was showing but now it's gone. If possible, it would be great to show all conferences that are not yet closed in ccfddl.top like ccfddl.github.io.

Thank you for the prompt feedback. ccfddl.top looks very concise and convenient. However, vldb 2024 was showing but now it's gone. If possible, it would be great to show all conferences that are not yet closed in ccfddl.top like ccfddl.github.io.

Good suggestion. However, this involves a lot of refactoring of the code and I'm trying to figure out how to fix it, which may take some time.

Haha, if fixing it is time and energy consuming, you can ignore this suggestion. Looking forward to you making ccfddl even better in the near future.

thank you for your support. Because there are very few conferences like vldb that have two years of open deadlines at the same time, I only keep one year of each conference in the database. Maybe your suggestion will be implemented in the future, now I'm thinking about implementing a WeChat applet for ccfddl.

Haha, if fixing it is time and energy consuming, you can ignore this suggestion. Looking forward to you making ccfddl even better in the near future.

Thank you for your support. Because there are very few conferences like vldb that have two years of open deadlines at the same time, I only keep one year of each conference in the database. Maybe your suggestion will be implemented in the future, now I'm thinking about implementing a WeChat applet for ccfddl.


Type stringly-typed methods as literal types

Fixes #62
Started work on this
Still depends on

https://github.com/python/typeshed/pull/2807
mypy pulling forward its pin
Carefully mimicking behavior described in https://developers.google.com/protocol-buffers/docs/reference/python-generated#singular-fields-proto3
Additional testing

Currently this fails w/ a bunch of
test/proto/nopackage_pb2.pyi:46: error: Argument 1 of "HasField" incompatible with supertype "Message"

How did you solve the errors you are mentioning?
I'm getting plenty of those since 1.10.

Need to use the latest mypy on master!
I actually realize that I jumped the gun on releasing 1.10 since mypy hasn't cut a release since the upstream to typeshed.
If you control the mypy revision you use, and simply use the latest master for the next few weeks, that should resolve it.
Anything after
https://github.com/python/mypy/commit/d87e626e2fbde93fe7958e62b38c07b23a4e7e32
which references this
https://github.com/python/typeshed/commit/07ea6614c8c03da598903d1fd4fcfd90a35a6ad3


Manual approval required for workflow run<PHONE_NUMBER>: Approve production deployment, high risk: false

Workflow is pending manual review.
URL: https://github.com/anbl-jyskebank-dk/approval-test/actions/runs/3656231336
Required approvers: [rasmus-svith-andersen-jyskebank-dk anbl-jyskebank-dk]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

approved

All approvers have approved, continuing workflow and closing this issue.


Building the Webpack DLL...

$ npm run build:dll

<EMAIL_ADDRESS>build:dll /Users/harry/git/project name
node ./internals/scripts/dependencies.js

Building the Webpack DLL...
/private/var/folders/xw/qgz50s197dl_j_4b4ncdtqs80000gn/T/shelljs_1ce31b6ca67270e45483:3
var childProcess = child.exec("cross-env BUILDING_DLL=true webpack --display-chunks --color --config internals/webpack/webpack.dll.babel.js", {"silent":false,"cwd":"project","env":{"npm_config_save_dev":"","npm_config_legacy_bundling":"","npm_config_dry_run":"","npm_package_devDependencies_lint_staged":"3.2.1","npm_package_devDependencies_babel_preset_latest":"6.16.0","npm_package_dllPlugin_include_2":"eventsource-polyfill","npm_package_babel_presets_0_1_es2015_modules":"false","npm_package_engines_npm":">=3","npm_config_viewer":"man","npm_config_only":"","npm_config_browser":"","npm_package_gitHead":"d38c5d669fb55dd95ac415ab2c7bde3653d9b1e9","npm_package_dependencies_redux":"3.6.0","npm_package_babel_presets_2":"stage-0","npm_config_also":"","npm_package_devDependencies_react_swipe":"^5.0.8","npm_package_devDependencies_node_plop":"0.5.4","npm_package_jest_moduleDirectories_0":"node_modules","npm
SyntaxError: Invalid or unexpected token
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.runMain (module.js:604:10)
at run (bootstrap_node.js:389:7)
at startup (bootstrap_node.js:149:9)
exec: internal error

Did anyone meet the issue and how to fix it, thanks

Could you reformat your output?


Unexpected behavior

Hey all, I was just trying something out and ran into an odd issue. I am not entirely sure what the compiler is thinking.
The below code has a problem when it comes to decoding, with JSONDecoder throwing Expected to decode Array<Any> but found a dictionary instead. on files, what is interesting is if we change the type to: [String: File] it works perfectly fine. I didn't dig in to it much, but it seems like the ExpressibleByArrayLiteral conformance is being interpreted incorrectly.
Problem code:
struct Gist {
  let id: Self.ID
  let files: [File.Name: File]

  typealias ID = Tagged<Self, String>
}

extension Gist: Identifiable {}
extension Gist: Codable {}

struct File {
  let filename: Name
  let content: String

  typealias Name = Tagged<Self, String>
}

extension File: Codable {}

Hi @ericlewis! There doesn’t appear to be any decoding code in your example above that would throw. Can you provide a full sample that reproduces the issue? There are definitely some gotchas around codability, some of which we can’t work around when it comes to coding containers, but if there is a bug we can fix we’d love to know!

This is currently a Swift limitation in which only string and integer keys properly encode/decode into dictionaries. A fix has been proposed and accepted into Swift: https://github.com/apple/swift-evolution/blob/main/proposals/0320-codingkeyrepresentable.md

Ah! Seems like that is the case then @mbrandonw.

@stephencelis it just using a JSONDecoder.decode(Gist.self, from: data). Nothing fancy. Brandon's reply seems to be the case.

@ericlewis That was one of the issues around decoding containers I was alluding to, just wanted to confirm what data was beforehand :)


latest baseball databank release doesn't include 2018 stats

Just downloaded postgres_databank_backup_2018.01.tgz from the releases page, and ran the contained SQL. the latest yearID I was able to find in the batting and pitching tables is 2017. The post on the releases page says it had been updated to include 2018 stats. Any chance you can update the release to include 2018 stats?
Thanks.

Yes, I will, tonight I will run an update and push up the latest baseball databank update.  When it's up I'll update this ticket.

@rusmo the latest data has been loaded and released.  You can grab it from here https://github.com/rippinrobr/baseball-stats-db/releases/tag/2018.02  Let me know if you have any questions.

Got it, thanks!


dotnet cli Reportgenerator custom command fails

When i try to add a custom dotnet.exe task (reportgenerator), the command fails with the following message.
No executable found matching command "dotnet-reportgenerator"
I have tried all the combinations as mentioned here: https://github.com/Microsoft/azure-pipelines-tasks/issues/7194
Any help is appreciated.

I have the same issue. Running on Linux in dotnet core 2.1 project
From my Azure-Pipelines.yaml
pool:
  vmImage: 'Ubuntu-16.04'

# Run unit tests
- task: DotNetCoreCLI@2
  displayName: 'Running unit tests'
  inputs:
    command: test
    projects: '**/*Tests/*.csproj'
    arguments: '--configuration $(buildConfiguration) --logger "trx;LogFileName=testresults.trx"'

# Run code coverage
- task: DotNetCoreCLI@2
  displayName: 'Running code coverage'
  inputs:
    command: test
    projects: '**/*Tests/*.csproj'
    arguments: '--configuration $(buildConfiguration) /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura /p:CoverletOutput=$(System.DefaultWorkingDirectory)/TestResults/Coverage/'

# Create report
- task: DotNetCoreCLI@2
  displayName: "Generating code coverage Reports"
  inputs:
    projects: '**/*Tests/*.csproj'
    command: custom
    custom: reportgenerator
    arguments: "-reports:$(System.DefaultWorkingDirectory)/TestResults/Coverage/*.xml -targetdir:results -reporttypes:HTMLInline;HTMLChart"

In my .csproj for the test-project, I have:
<ItemGroup>
    <DotNetCliToolReference Include="dotnet-reportgenerator-cli" Version="4.0.15" />

Same here. Even when I try installing the tool manually:
- script: reportgenerator "-reports:$(Build.SourcesDirectory)/CodeCoverage/coverage.cobertura.xml" "-targetdir:CodeCoverage" "-reporttypes:HTMLInline;HTMLChart"
    displayName: Generate Code Coverage HTML Report

@jilpansi @ThomasBDO @RTodorov
We're investigating the issue.
To unblock yourselves, you can use dotnet-reportgenerator-globaltool as a workaround
You can install the tool by
  - task: DotNetCoreCLI@2
    displayName: Install ReportGenerator Global Tool
    inputs:
      command: custom
      custom: tool
      arguments: install dotnet-reportgenerator-globaltool -g

And then add a CommandLine script to the pipeline, to execute
reportgenerator <Arguments>

Issue with Linux:
We found a bug in Linux agent, where the PATH is not getting updating after global installation of dotnet tools. To workaround this, you need to add another CommandLine script before calling reportgenerator:
echo "##vso[task.prependpath]$HOME/.dotnet/tools"

Please note it needs to be a seperate task, before reportgenerator is used
Issue raised: https://github.com/Microsoft/azure-pipelines-image-generation/issues/798
Let me know if this doesn't work. Please be assured we'll investigate and resolve the issue with dotnet-reportgenerator-cli as soon as possible

@ThomasBDO @jilpansi @RTodorov - I assume your issues are resolved using @issacnitin's suggestions above. Let us know if you still face the issue.


Investigate Roslyn Stable

Roslyn is now 1.0.0 and officially released.

@RichiCoder1 Yes, but not the scripting parts right? I read somewhere on Twitter that they will be released "later" whatever that means :smile:

Yah, I realized that myself. I think the scripting parts were deprioritized as not critical for RTM. Hopefully they're re-prioritized for post-RTM. Can use this issue to track that.

@RichiCoder1 Sounds like a good idea!

Update: https://github.com/dotnet/roslyn/issues/4110
Aka, scripting is a WIP for 1.1 release.

Looks like xplat support as well in 1.1. Would be nice to be able to have a single scripting engine in Cake. Thanks for the update! :smile:

It seems like Microsoft.CodeAnalysis.Scripting has now a stable release, https://www.nuget.org/packages/Microsoft.CodeAnalysis.Scripting
Would it be possible to let the experimental flag use the stable version? Or may completely switch to this version?

@stffabi yep, there are newer stable versions, but these target .NET Core. So no luck until Cake is ported to .NET Core :disappointed:

@mholo65 I thought they supported .Net Framework 4.6 too?

@RichiCoder1 me too, until I downloaded every NuGet package and noticed it contained only lib\dotnet folders (except 1.1.0-beta-something...). ScriptCS seems to also use same version (1.0.0-rc2) as Cake "Experimental" does...

They need to update their wiki then ;)
https://github.com/dotnet/roslyn/wiki/Scripting-API-Samples#supported-platforms
It reads:
Scripting APIs require desktop .NET Framework 4.6 or CoreCLR 1.0.

@devlead @RichiCoder1 Might be I don't got my facts straight. Anyhow, here's the contents of Microsoft.CodeAnalysis.CSharp.Scripting
Is this a PCL compatible with both .NET Core 1.0 and .NET Framework 4.6? Easiest would probably to just test it :smile:

I just did a Install-Package Microsoft.CodeAnalysis.Scripting -Version 1.2.1 on a 4.6 console app.

So seem to work. 1.2.1 license now seem more permissive so could likely actually ship with Cake.
Assemblies in bin post nuget install:

Could be wort pursuing as a proof of concept WIP pr.

@devlead great. And porting Cake to .NET 4.6 has which consequences?

@mholo65 It would more or less break Cake on any CI system that doesn't have .NET 4.6 installed so we should probably think about how we want to proceed with this.

We could start with a WIP pr to investigate what it would mean source code wise, this would also prepare for CoreCLR.
Mono have introduced 4.6 support, so in theory that could work there too.

@devlead A WIP sounds like a good plan!

Apparently Github decided to not give me any alerts for this :|. Confirmed with the Roslyn guys that Scripting is good for .Net Framework 6 & .Net Core. Let me know if you need any help @devlead! Looking forward to getting this on a stable roslyn, then on a cross-plat core :).

@devlead @RichiCoder1 I've created a WIP for this targeting .NET 4.6. The amount of references required is ridiculous (14 MB).
Will push these changes soon to a WIP PR.

@devlead @RichiCoder1 @mholo65 Ran the integration tests and they seem to run without problem. Downside is that the nuget package size went from 1.2 MB to 4.1 MB.

Does not work on Mono, but that was not a surprise :)

@patriksvensson interesting, which version of Mono? They added 4.6 support a couple of months ago, but they used to have some patches for Roslyn wonder if that has made into the 2.0.0-beta1 or if a newer version of Mono sorts it.

@devlead The latest alpha, version 4.4.2 (I think).

Would be cool to also have a WIP for .NET Core 1.0 as RC2 is just around the corner.

@patriksvensson @mholo65 Cake.CoreCLR sorts this?

@devlead yep! But to support it in "full CLR" Cake, we need to target .NET 4.6.

@mholo65 yeah that's the natural path, how's the 4.6 support in Mono nowadays? More specific wonder if latest Roslyn is supported.

Closing since #1645 been merged to develop.


[FEATURE] finish footer + layout

make it responsive, or keep mobile footer for app regardless of screensize TBD

const mediaQueryForDesktopUp = (content) => css  @media (min-width: 1300px) {     ${content};   }


Question: about  download option

Hello, first off I would like to say thank you for your excellent job!! I have no problem, but a question. When I use the download option, it always uses the JP name of the gallery on E-H. / Ex-H even if it was a gallery of a translated version with an English name. I tried looking at configurations but couldn't find anything about this. Is there a way I could make the extension priorize an English name or Romaji name first, then JP kanji/hiragana name?

Added, There is a new option of EHentai Prefer Title in config panel. it's default value is Japanese.

Added, There is a new option of EHentai Prefer Title in config panel. it's default value is Japanese.

Worked perfectly. Thank you!!


New package: Quac v0.1.0

Registering package: Quac
Repository: https://github.com/bsc-quantic/Quac.jl
Created by: @mofeing
Version: v0.1.0
Commit: de71b85358e3720b742b1f005652f842e6daffe2
Git reference: 0.1.0
Description: Quantum Circuit representation library in Julia

[noblock]
The 5 characters long limit on the package name is not in the guidelines of the General registry. Actually the names of some of the most popular packages in juliapackages.com have also 4 letters: JuMP.jl, CUDA.jl, Flux.jl...

I've been thinking about this and I think I'll upload it to a private registry.
Could a maintainer close this PR pls?


Custom wakeword not working, spokestack word still used.

Spokestack.initialize(
clientId,
clientSecret,
{
vadMode: 'quality',
wakeword: {
filter: require('./model/filter.tflite'),
detect: require('./model/detect.tflite'),
encode: require('./model/encode.tflite'),
},
nlu: {
model: require('./nlu/nlu.tflite'),
vocab: require('./nlu/vocab.txt'),
metadata: require('./nlu/metadata.sjson'),
},
},
)

Thanks for opening an issue. Unfortunately, this isn't a lot of info to go on. That said, I'm going to test a custom wakeword model once I fix the compilation issues with RN 0.64.
In the future, avoid posting your API key on GitHub (or committing that directly to a public repo) in order to keep it secure. I went ahead and edited this issue, but to be completely safe you could rotate the key to a new one.

If you've replaced the files but they have the same name, you could try changing the file names or clearing the react native bundler cache with npx react-native start --reset-cache

Hi, tried both options but with no luck ...

I tried also with the CDN files, same result.

Oh, also try setting refreshModels to true in Spokestack.initialize(). This will clear the model cache.

That should work, and if so, I think perhaps we should set that to true in development by default.

Spokestack.initialize(
clientId,
clientSecret,
{
refreshModels: true,
vadMode: 'quality',
wakeword: {
filter: require('./model/filtertwo.tflite'),
detect: require('./model/detecttamarintwo.tflite'),
encode: require('./model/encodetamarintwo.tflite'),
},
nlu: {
model: require('./nlu/nlu.tflite'),
vocab: require('./nlu/vocab.txt'),
metadata: require('./nlu/metadata.sjson'),
},
},
)
Same result...

Are you saying that it recognizes when you say "Spokestack" or that you've verified it's using the Spokestack model files included in the example instead of yours? What happens if you uninstall and reinstall the app?

It recognizes Spokestack only, I had a look in react-native-spokestack/src/index.tsx "initialize = (" and it receives my custom models. After I run the app in the simulator with npx react-native run-ios, Spokestack only recognizes the Spokstack wake word.

Good news! I was able to reproduce your issue. I'll push a fix for you soon.

This does only seem to affect iOS. Android seems to work fine.

Yep, Android works perfectly.

It's getting late here, but we'll keep working on this tomorrow. Thanks for your patience.

We figured out the issue and will have a release of both spokestack-ios and react-native-spokestack by tomorrow.

A fix has been released! Please try version 6.0.0.

Thanks, works like a charm :D


Amadey

New

[x] windows_files_and_dirs_access_rights_modification_via_icacls.yml

updated

[x] registry_keys_used_for_persistence.yml
[x] icacls_deny_command.yml
[x] icacls_grant_command.yml

tag

[x] executables_or_script_creation_in_suspicious_path.yml
[x] scheduled_task_deleted_or_created_via_cmd.yml
[x] suspicious_process_file_path.yml
[x] windows_credentials_from_password_stores_chrome_extension_access.yml
[x] windows_credentials_from_password_stores_chrome_localstate_access.yml
[x] windows_credentials_from_password_stores_chrome_login_data_access.yml
[x] winevent_windows_task_scheduler_event_action_started.yml

Story
    new file:   stories/amadey.yml

What does this PR have in it? Screenshots are worth 1000 words 😄

Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

improved the text in the story file. LGTM!!


Reflow skipped packages

With the introduction of Db2Monitoring, we will likely discover many packages that are missing from V3.
We will need to reflow these packages. We will need to write an OPS tool to assist with this because, at least on DEV there are 200.

[ ] DEV
[ ] INT
[ ] PROD

Have a tool that will generate the list of packages that need to be reflowed from the v3-monitoring, cross checking against the env if the package has been deleted(hard/soft) or available?
Use the Admin reflow tool to reflow hard deleted ones.
Use the command line reflow tool to reflow rest of the packages.

There are two invalid packages in PROD due to https://github.com/NuGet/NuGetGallery/issues/4475. The rest of the environments are clean now.

@joelverhagen - actual bug is https://github.com/NuGet/NuGetGallery/issues/5914, not #4475

Oh yep, thanks.


feat: Add syntax highlight file for micro

This PR adds a syntax highlight file for the micro text editor.
https://micro-editor.github.io/
https://github.com/zyedidia/micro

Nice — thanks @vanillajonathan !


⚠ GitLab Server has degraded performance

In 8fd7c1d, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6633 ms

Resolved: GitLab Server performance has improved in 1df9129.