Plugins modules required to be packages

I can't really get slackbot to work with my custom modules at all (I'm on Python 3.5) but I thought I'd document one of the major hurdles I discovered while I wrangle on.
The import hook code is guaranteed to fail if a simple Python module is specified rather than a package. That's because submodule_search_locations is None in that case. If this is an intended limitation (though I can't imagine why it would be) it should be documented as such.

Actually it looks like that was the final hurdle. Once I put the custom plugins into a package I was able to get them to work, but the wart still stands.

I am stuck on the same thing (Python 3.5.1). I cannot get my own plugins to work (myplugins/myplugin.py), keep getting:
`AttributeError: module 'myplugins.myplugin' has no attribute 'path'``
Could you please walk me through, what you did to make it work?

Try just 'myplugins' ; let me know if works
On Fri, Mar 25, 2016 at 12:00 PM Jani Karhunen<EMAIL_ADDRESS>wrote:

I am stuck on the same thing (Python 3.5.1). I cannot get my own plugins
to work (myplugins/myplugin.py), keep getting:
AttributeError: module 'myplugins.myplugin' has no attribute 'path'`
Could you please walk me through, what you did to make it work?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/lins05/slackbot/issues/59#issuecomment-201240356

Thanks! I deleted my message, as I resolved this just as you proposed almost immediately after posting. Sorry I was a bit hasty.

Just for anyone who comes along later, you need to make sure you have an init.py as well in the folder you specify, in order to make sure it's a proper Python package. You also need to make sure it's in your sys.path (e.g. by proper installation into site-packages or manipulating the  PYTHONPATH environment.

This was actually fixed in #71.


Promote the Tox container image

Lumír is building a Docker image that has Fedora, Tox and all the Pythons.
We should promote that more.
Possibly turn it into a GitHub Action and write a Magazine article about it?

https://github.com/fedora-python/fedora-python-tox

When preparing my EuroPython lightning talk, I've talked about the github action and I realized it's not mentioned on fedoralovespython.org yet. We should do that.


EIP bot must error when multiple files are in EIPs asset folder but e.g. only single file is linked in specification document

Not sure if this is the right repository to file this bug report.
EIP-4973 is checking in a number of separate files into the /assets/ folder as a reference implementation: https://github.com/ethereum/EIPs/tree/master/assets/eip-4973/src
However, it seems that when wanting to request those assets as a folder, they're not available: https://eips.ethereum.org/assets/eip-4973/src/
Only individual files are: https://eips.ethereum.org/assets/eip-4973/src/ERC4973.sol

The EIP bot must check the following:
If an EIP champion checks in multiple files in the asset directory, the EIP bot must check if all of those files are listed in a table in the specification document. Elsewise if e.g. only one file is checked in, the EIP bot must check if this one file is listed in the specification document.
In case e.g. a file is checked into the assets directory but not listed in the specification document, then the EIP bot must fail/throw to indicate to the user that they have to link all documents checked-into the assets folder.

This isn't a bot issue, it has to do with the fact that neither GitHub nor Jekyll markdown renderers (I believe) render folders, they only support linking to files.  For EIPs, the current best option is to either create a manual table of contents markdown file and link to that in your EIP, or just put the table of contents into the EIP itself.
Generally speaking, we try to target the lowest common denominator for markdown rendering so the contents of this repository are maximally compatible with all markdown rendering systems.  In this case, that means no support for folders.

then IMO tests should be created that check if all files in the assets folder are linked in the EIP. I shipped a folder structure but only linked a single file, only to find later that it lead to bad experiences with people downloading the reference implementation. I fixed it now, but it would have been nice had there been a warning/check.

If you want to update the Title and description of this issue to reflect that suggestion to the bot we can re-open it.  I agree that would be a great addition.

OK, changed the title and edited the original post.

I think this would best be done in the renderer by adding an index.html table of contents in every folder and sub-folder of assets

I think this would best be done in the renderer by adding an index.html table of contents in every folder and sub-folder of assets

We have tried pretty hard in the past to ensure that EIPs repository renders in the lowest common denominator rendering system.  Any tweaks we have made to Jekyll layouts are purely visual, and not functional.  If we allowed people to link to directories and this only worked in the Jekyll rendered site, then any other rendering engine used would break and links would not work properly.

This best seems suited as an eipw issue.

Superseded by https://github.com/ethereum/EIP-Bot/issues/93


ceph-infra: open dashboard port on monitor

When there's no mgr group defined in the ansible inventory then the
mgrs are deployed implicitly on the mons nodes.
If the dashboard is enabled then we need to open the dashboard port on
the node that is running the ceph mgr process (mgr or mon).
The current code only allow to open that port on the mgr nodes when they
are present explicitly in the inventory but not implicitly.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1783520
Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>

@guits I think we should drop this feature (mgr on mon by default) on master to avoid this kind of situation in the future.

jenkins test centos-container-ooo_collocation

@guits I think we should drop this feature (mgr on mon by default) on master to avoid this kind of situation in the future.

I agree.


de-allocate what's in the queue in pqueue_free

If you do not do this, chances are your code will leak memory...

IMHO, we cannot know whether user wants us to free his/her own objects. Freeing the objects itself should be the users responsibility. Maybe we can at least document this.

Do as you want, but what is sure is that once you free the pqueue everything that's inside has to bee freed too.
I naively assumed pqueue_free did that, then I choked at the sight of the output of Valgrind :)


Set swift-format to v508.0.0

In this PR:
Solves issue: https://github.com/maticzav/swift-graphql/issues/119
My machine:
$ swift --version

swift-driver version: 1.75.2 Apple Swift version 5.8 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>.100)
Target: arm64-apple-macosx13.0

$ swift-format --version

508.0.0

Can this be pushed into the master branch. The current version builds with errors and this fixes it.


How to get access to assets file (issue opened after trouble shooting read).

Hello!
Firstly I pay your attention, that this issue about trouble shooting topic "get file from assets".
I need to access a file located in the assets folder on an iOS device. Initially, my file was located in the src/assets/react_image.png folder, but I later moved it to the root of the project at ./assets/react_image.png, which did not help.
I would like to understand how I can access files because after exploring the bundle folder, I did not find anything resembling my image.
I have tried different approaches, both with and without wrap, but nothing has helped. Below, I am attaching all the files that should help to understand the problem.

please provide the version of installed library and RN project:

a sample code snippet/repository is very helpful to spotting the problem:

import React, {useState} from 'react';
import {Button, Image, ScrollView, StyleSheet, Text} from 'react-native';
import ReactNativeBlobUtil from 'react-native-blob-util';

const styles = StyleSheet.create({
  container: {
    flex: 1,
    padding: 10,
    backgroundColor: '#e0e0e0',
  },
  text: {
    fontSize: 20,
    color: '#333333',
  },
  image: {
    width: '80%',
    height: undefined,
    aspectRatio: 3,
    marginTop: 20,
  },
});

export const BlobUtil = () => {
  // const [data, setData] = useState<string | null>(null);

  // const fs = RNFetchBlob.fs;

  // console.log(fs.asset('../../assets/react_image.png'));

  // RNFetchBlob.fs
  //   .readFile(fs.asset('../../assets/react_image.png'), 'base64')
  //   .then(data => {
  //     console.log(data);
  //   })
  //   .catch(e => console.log(e));

  const [blob, setBlob] = useState(null);

  const convertImageToBlob = async () => {
    try {
      const url = ReactNativeBlobUtil.wrap(
        ReactNativeBlobUtil.fs.asset('react_image.png'),
      );
      console.log('jjjjj', url);
      const imagePath = ReactNativeBlobUtil.fs.asset('react_image.png');

      console.log(imagePath);

      const imageBlob = await ReactNativeBlobUtil.fs.ls(
        'ReactNativeBlobUtil-file://',
      );
      console.log('imageBlob', imageBlob);
      // setBlob(imageBlob);
      // console.log('Image converted to Blob:', imageBlob);
    } catch (error) {
      console.error('error:', error);
    }
  };

  return (
    <ScrollView style={styles.container}>
      <Text style={styles.text}>root image:</Text>
      <Image
        source={require('../../../assets/react_image.png')}
        style={styles.image}
      />

      <Button title="Convert Image to Blob" onPress={convertImageToBlob} />
      <Text style={styles.text}>Blob data: {blob}</Text>
      {/* <Image source={{uri: base64Image}} style={styles.image} /> */}
    </ScrollView>
  );
};

adding my repo here: https://github.com/TimeForRelax/first-react-native-project

You have to use the correct path. You cannot access the files with a relative path.
You will have to use
ReactNativeBlobUtil.fs.dirs.bundleDir + '/path in the app' when using the library.
Not needed for the image source though.


[battery_platform_interface] Introduce package.

Description
In this PR, I have added platform interface package to the battery plugin.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Hey @ditman kindly have a look at this one.

Tagged and published battery_platform_interface: ^1.0.0

Thank You :)

In porting battery to use platform interface I have to delete the battery/test now. Can I do that ? Because the tests are no in battery_platform_interface.

@yash1200 you can leave the tests in battery/test. Why do you have to delete them? Because the method channel is not publicly visible anymore?
(Tests in battery/test should call methods in the plugin class, and in the platform_interface they should call directly methods on the method_channel class)


NCBIGene prefix fix

Related with: https://github.com/biolink/biolink-model/issues/1431
Update https://identifiers.org/ncbigene/ to http://identifiers.org/ncbigene/

Fixed with https://github.com/obophenotype/brain_data_standards_ontologies/pull/291 and https://github.com/obophenotype/provisional_cell_ontology/pull/57


Get/Set-DbaDatabaseState - Add pipe support and some extra checks

Is this a feature OR bug:
feature
Make a PR for this tomorrow -- https://github.com/sqlcollaborative/dbatools/tree/dbset-features
Will provide:

[x] Checks for connections and just skips it if -Force isnt specified and the command won't work if ppl are connected
[ ] Finish proper pipe

fixed in #608


perf: update NFs hash and Makefile webconsole

Description

Update NF's hash

Add back StaticIP UI and support verify from SMF OAM
Fix UPF panic when terminating

Perf webconsole Makefile

Decouple frontend and backend

Feat: force_kill.sh to drop free5gc optionally

Note

SMF:  https://github.com/free5gc/smf/pull/99
Webconsole: https://github.com/free5gc/webconsole/pull/85
UPF

https://github.com/free5gc/go-upf/pull/44
https://github.com/free5gc/go-upf/pull/47

fix the conflicts


Switch tabs, the scene would be reload, is it right?

Hi everyone, every time when I tap the tab, the corresponding scene would be reload.So it cause little lag , is it need me to do some measures like cache?  Or is the use of I  is  wrong?
my StackNavigator:
const rootApp = StackNavigator(
    {
        Main: { screen: CustomTabBar },
        Home_2:{screen:Home_2},
        Contacts_2:{screen:Contacts_2},
        Find_2:{screen:Find_2},
        Me_2:{screen:Me_2}
    },
    {
        headerMode:'float',
    },
);

my custom tabBarNavigator:
const CustomTabs = createNavigationContainer(createNavigator(CustomTabRouter)(CustomTabView));

Hi @guoliang1206 thanks for the question.
I solve this problem with redux in my app. I do an initial load the first time the user navigates to a specific tab, and I save the information in my redux store. Upon each subsequent navigation to that tab, I display the information from the redux store and fire off another GET to retrieve the most recent data from my API. You could probably achieve something like this without redux.

@matthamil  thanks for your answer, I will research it

@matthamil Hi Matt, i got stuck with the same problem. Could you elaborate how do you restore previous state from redux store. In my case using Stack Navigator i go from screen A-B-C. Screen B loads a list & now when i press back on C, B shows up empty as the componentDidMount function is not called. How can i restore my components previous state on reload?

@matthamil @guoliang1206 did you solve this problems? can you give me a hint?


test: include large files in syncInputJobAttachments cancelation tests to allow more time for cancelation

What was the problem/requirement? (What/Why)
test_worker_reports_canceled_sync_input_actions_as_canceled was failing flakily because sometimes the syncInputAttachments goes too fast so we don't have time to cancel it.
What was the solution? (How)
Create bigger files in between the small files, so that  the syncInputAttachments is much slower, but it does not time out due to the perceived transfer rate being too slow.
What is the impact of this change?
More robust test_worker_reports_canceled_sync_input_actions_as_canceled test
How was this change tested?
# Linux
source .e2e_linux_infra.sh
hatch run e2e-test

# Windows
source .e2e_windows_infra.sh
hatch run e2e-test

Ran 5 times each, passed every time.
Was this change documented?
No
Is this a breaking change?
No
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

What was the problem/requirement? (What/Why)
test_worker_reports_canceled_sync_input_actions_as_canceled was failing flakily because sometimes the syncInputAttachments goes too fast so we don't have time to cancel it.
What was the solution? (How)
Create bigger files in between the small files, so that the syncInputAttachments is much slower, but it does not time out due to the perceived transfer rate being too slow.

Two things:

nit: it should be syncInputAttachments → synInputJobAttachments
These seem to be pointing to different root-causes for the test failures. One says the transfer happens so fast we don't have time to cancel it. The other says that the syncInputAttachments fails because of perceived transfer rates being low.

Which is true? Is it both?

What was the problem/requirement? (What/Why)
test_worker_reports_canceled_sync_input_actions_as_canceled was failing flakily because sometimes the syncInputAttachments goes too fast so we don't have time to cancel it.
What was the solution? (How)
Create bigger files in between the small files, so that the syncInputAttachments is much slower, but it does not time out due to the perceived transfer rate being too slow.

Two things:
1. nit: it should be `syncInputAttachments` → `synInputJobAttachments`

2. These seem to be pointing to different root-causes for the test failures. One says the transfer happens so fast we don't have time to cancel it. The other says that the syncInputAttachments fails because of perceived transfer rates being low.

Which is true? Is it both?

For 1, I will change that.
For 2, the flaky failures are because the action completes too quickly. However, we tried before with around 5000 small files but that fails the action because the transfer rate is too slow and times out the job attachments code (which the latter part refers to, pardon the lack of context). So the middle ground is to create small files with large files in between, so that it still takes a long time, but average transfer rate is brought up by the larger files.


"Another" bug with generate Heightmap

When i create heightmap with too "high" resolution size ->= 250 for me. I ve got an error and close the editor.
`Assertion failed: !AllocationFlags[Index] [File:D:\Epic Games\UE_4.16\Engine\Source\Runtime\Core\Public\Containers/SparseArray.h] [Line: 78]
KERNELBASE
UE4Editor_Core
UE4Editor_Core
UE4Editor_Core
UE4Editor_MyGame_2480!TSparseArray<TSetElement<TTuple<FVector2D,int> >,TSparseArrayAllocator<FDefaultAllocator,FDefaultBitArrayAllocator> >::AllocateIndex() [d:\epic games\ue_4.16\engine\source\runtime\core\public\containers\sparsearray.h:81]
UE4Editor_MyGame_2480!TSet<TTuple<FVector2D,int>,TDefaultMapHashableKeyFuncs<FVector2D,int,0>,FDefaultSetAllocator>::Emplace<TPairInitializer<FVector2D const & __ptr64,int const & __ptr64> >() [d:\epic games\ue_4.16\engine\source\runtime\core\public\containers\set.h:490]
UE4Editor_MyGame_2480!UPolygonMap::FindMapCenterForCoordinate() [d:\unreal projects\mygame\source\mygame\private\map\polygonmap.cpp:412]
UE4Editor_MyGame_2480!FHeightmapPointTask::MakeMapPoint() [d:\unreal projects\mygame\source\mygame\private\map\heightmap\heightmappointtask.cpp:245]
UE4Editor_MyGame_2480!FHeightmapPointTask::DoTask() [d:\unreal projects\mygame\source\mygame\private\map\heightmap\heightmappointtask.cpp:282]
UE4Editor_MyGame_2480!TGraphTask::ExecuteTask() [d:\epic games\ue_4.16\engine\source\runtime\core\public\async\taskgraphinterfaces.h:883]
UE4Editor_Core
UE4Editor_Core
UE4Editor_Core
UE4Editor_Core
UE4Editor_Core
kernel32
ntdll`
In low resolution, you could see that there is many many many many warning "LogWorldGen:Warning: Tried to fetch a pixel at 227, 123, but no pixel was found.", Don't know if it could help you. After a few execution, you could see that editor get a new error
Access violation - code c0000005 (first/second chance not available) VCRUNTIME140 UE4Editor_GameplayTags UE4Editor_GameplayTags UE4Editor_GameplayTags UE4Editor_MyGame_2480!TArray<FMapData,FDefaultAllocator>::CopyToEmpty<FMapData>() [d:\epic games\ue_4.16\engine\source\runtime\core\public\containers\array.h:2323] UE4Editor_MyGame_2480!UPolygonalMapHeightmap::CheckMapPointsDone() [d:\unreal projects\mygame\source\mygame\private\map\heightmap\polygonalmapheightmap.cpp:33] UE4Editor_CoreUObject UE4Editor_CoreUObject UE4Editor_MyGame_2480!UE4Function_Private::TFunctionRefCaller<<lambda_57bf9da02145909c0e8c545268c2f5d5>,void __cdecl(void)>::Call() [d:\epic games\ue_4.16\engine\source\runtime\core\public\templates\function.h:244] UE4Editor_Core UE4Editor_Core UE4Editor_Core UE4Editor_Engine UE4Editor UE4Editor UE4Editor UE4Editor UE4Editor kernel32 ntdll

Remake fresh install of unreal engine & plugin, do the same thing. Can't create heightmap.

I remove multi thread system, and it stop to bug. Of course it s slower than multi thread

Hi,
so so so Sorry to bother you ...
Could you tell me how to call out the delegate OnGenerationComplete showed in the ExampleProject ?

I am a freasher on UE4.
Any help is appreciated :-)

Just create a new line from on complete and add custom event. You could call it On Generation Complete

@hoaxhoax
Many thanks for the prompt reply.
And, could you provide me some information?
This plugin doesn't work on UE4.16 ?  Every time I run the ExampleProject provided, the UE4.16 editor crashes without any message just popping up the window:

I have no idea about this :-(
In my own project, create new level, draw Even Graph in Level BluePrint as follows,

However, there is nothing when I click "Play". It indeed notified the "On Generation Complete" event since the string was shown in the screen. The editor crashed frequently.
Any assistant you can provide would be greatly appreciated :-)

I never see that bug and i think that the author doesn't support it anymore. You should try to do it by yourself.

@hoaxhoax
Thank you anyway :-)

I think I've fixed the issue with commit https://github.com/Jay2645/Unreal-Polygonal-Map-Gen/commit/05f1ff523411e7c336fa33320cfae20e2da6cc5c. I'm closing this for now. Feel free to reopen it if you're still having issues!


Invoke-WebRequest / Invoke-RestMethod fails to follow HTTP redirects from HTTPS

Steps to reproduce

Install Windows Server Nano (10.0.14393)
Connect to the server using PS Remoting, then use PowerShell Core (the built-in version) to download a file, e.g. try to download .NET Core 1.1 ZIP package:

Invoke-Webrequest -Uri 'https://go.microsoft.com/fwlink/?LinkID=835028' -OutFile C:\netcore11.zip -MaximumRedirection 10
Expected behavior
File is downloaded, HTTP 302 Redirect returned by go.microsoft.com is handled properly.
Actual behavior
The following error occurs:
invoke-webrequest : Response status code does not indicate success: 302 (Moved Temporarily).
And there is no workaround, because the thrown exception is a HttpRequestException thrown by EnsureSuccessStatusCode(), which loses all header information (so, unable even to access the Location header).
Issue is still present in the master branch:
https://github.com/PowerShell/PowerShell/blob/master/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/CoreCLR/WebRequestPSCmdlet.CoreClr.cs#L352
EnsureSuccessStatusCode() should not be used, as it doesn't support redirects, and it also obfuscates the exception, making it impossible to access the returned headers/body even if the exception is caught.
Environment data
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
BuildVersion                   10.0.14393.1000
SerializationVersion           <IP_ADDRESS>
PSVersion                      5.1.14393.1000
PSEdition                      Core
WSManStackVersion              3.0
CLRVersion

After some debugging, it looks like the root cause is that .NET Core's HttpClientHandler silently refuses to follow HTTPS-to-HTTP redirects.
However, there should be some way to at least look at the underlying response headers - currently, only a generic string description of the error is provided in the caught Exception (in desktop PowerShell - it is possible to retrieve the headers).

Perhaps we should add a -AllowRedirectToHTTPfromHTTPS type switch or just output a warning message

This repros with alpha.17 on Ubuntu16 so not Nano specific

@PowerShell/powershell-committee suggests -allowInsecureRedirect

Possible workaround.
Works in PowerShell Core 6.0.1 and PowerShell 5.
function Get-RedirectedUrl() {
  param(
    [Parameter(Mandatory = $true, Position = 0)]
    [uri] $url,
    [Parameter(Position = 1)]
    [Microsoft.PowerShell.Commands.WebRequestSession] $session = $null
  )

  $request_url = $url
  $retry = $false

  do {
    try {
      $response = Invoke-WebRequest -Method Head -WebSession $session -Uri $request_url

      if($response.BaseResponse.ResponseUri -ne $null)
      {
        # PowerShell 5
        $result = $response.BaseResponse.ResponseUri.AbsoluteUri
      } elseif ($response.BaseResponse.RequestMessage.RequestUri -ne $null) {
        # PowerShell Core
        $result = $response.BaseResponse.RequestMessage.RequestUri.AbsoluteUri
      }

      $retry = $false
    } catch {
      if(($_.Exception.GetType() -match "HttpResponseException") -and
        ($_.Exception -match "302"))
      {
        $request_url = $_.Exception.Response.Headers.Location.AbsoluteUri
        $retry = $true
      } else {
        throw $_
      }
    }
  } while($retry)

  return $result
}

$url = Get-RedirectedUrl "https://go.microsoft.com/fwlink/?LinkID=835028"

Invoke-Webrequest -Uri $url -OutFile C:\netcore11.zip

Note: Exception type checking works better in this way. Prevent errors in PowerShell 5 (non-existent type).

This is more than HTTP to HTTPS. This repros as well.
Get-Command -Name New-Item | Select-Object HelpUri

HelpUri
-------
https://go.microsoft.com/fwlink/?LinkID=113353

Invoke-WebRequest -Uri (Get-Command New-Item).HelpUri

@thezim It's the same problem..
Invoke-WebRequest -Uri (Get-Command New-Item).HelpUri
$error[0].Exception.Response.Headers.Location.AbsoluteUri

result: http://technet.microsoft.com/library/hh849795.aspx
the problem is when an HTTPS uri redirects to an HTTP (no-S) uri.

@fcabralpacheco The code snippet looks great but it doesn't seem to work in Windows Nano Container.

I stumbled across this issue when I discovered my script using System.Net.WebClient worked on pwsh but not on Windows. On Windows, or in Windows Powershell, you'll get a 401 Unauthorized because WebClient follows redirects without adding Authorization headers. But this was not the case with pwsh 6.0.0. It did not fail at all.
To fix the issue, I switched to Invoke-RestMethod:
Invoke-RestMethod -Uri $url -MaximumRedirection 0 -Method $method -Body $json -Headers $headers

Now the script worked fine on Windows Powershell but pwsh 6.0.0 now failed with the very same 401 Unauthorized response. That means, it followed the redirect even though I used -MaximumRedirection 0.
Upgraded to pwsh 6.0.4, same issue in that version.
Installed pwsh-preview (6.1.0-preview.4) and now got the same exception but for the redirect response.
Microsoft.PowerShell.Commands.HttpResponseException: Response status code does not indicate success: 303 (See Other).
   at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)

Conclusion:

Windows Powershell
pwsh 6.0.0-6.0.4
pwsh 6.1.0 preview

WebClient.UploadString
Follows redirect without auth headers
Follows redirect with auth, no error
Follows redirect with auth, no error

Invoke-RestMethod -MaximumRedirection 0
No follow, no error
Follows without auth, throws 401
No follow, throws 303

Gah, these differences makes it so hard to write something that works/can be tested on all our platforms.

@anderssonjohan that appears to be a different issue. The issue here is when an HTTPS site redirects to an HTTP site, regardless of the presence or lack of auth headers. Would you be so kind as to open a new issue for that, please?

@markekraus Thanks! Sure, I'll do that. It seemed related to me and I didn't want to add duplicate issues.

This issue is present in ps7 but not the ps bundled with windows.

@CarlGustavAlbertDwarfsteinYung WinPS uses a different HTTP client from .NET Framework that could be argued is doing the insecure thing.  .NET Core has a new HTTP client that rejects this and we'd have to code around that.

@SteveL-MSFT  Hi thank you for your reply. As far as i can undestand from what I read the issue is that PS expects a standardized HTTP implementation but in reality there is no standard. This is why Firefox recommends doing 302 only when method changes ie GET to POST or vice versa but not in all other redirect cases. Seems like PS gives a 302 Found for every redirect. The standard added 303 and 307 that is expected in response to 302. Mozilla recommends using 302 sparingly because every browser uses their own system for redirects. Of course I am not an expert in this project so I may be completely wrong.=

In the mean time:
C:\Windows\System32\curl -L -o outfile "https://example.com/somefile"

In the mean time:
C:\Windows\System32\curl -L -o outfile "https://example.com/somefile"

This issue was a major obstacle for me when I migrated from PS5 to PS7.
I hope it will be addressed as soon as possible.


Porting Music to Iphone bug

[x] Using latest version as provided on the master branch
[x] Searched for similar issues including closed ones

What is the purpose of your issue?

[x] Bug
[ ] Feature Request
[ ] Question
[ ] Other

Description

Music downloads fine, can play it on PC without any problems, however when i port the music from my PC to IPhone via itunes, it goes to my music at it should, but when i play it it merges two songs into one, it may say for example, "Aer - Island" and it'll play that song, but it says it is ~5mins long when it should be ~2mins, once it finishes the song, Island, at the ~2min mark, it starts another song, which seems to always be the one underneath, or up next, but the song timer goes to ~5mins, and keeps the original song label, and name "Aer - Island" but will play a different song. I originally copy +_pasted to songs onto phone in one go, this bug happened, so i removed them from phone restarted it, then added then one by one, shuffled them, restarted phone again, and the bug still is here. If you play a song, and it gets to the quote on quote "second song" whilst still being under the original song, delete the song its playing from library, which will be the one labelled as up-next, it'll stop playing that one and start playing the new song labelled as "up-next", guessing this is a problem w/ my phone but was hoping someone knew how to fix it.
Log

No log, music download works fine, can play it on my PC without any problems or loops

Hi, firdt of all. In what format are you saving the songs?

.m4a

Well, i'm not an iPhone expert but try porting them to MP3. Probably the quality will go down or maybe not. Give it a try. In my iPod, all my songs are in MP3 and they're working just fine.

Not sure what's going on here but my guess is that your iPhone is having a hard time reading the track duration. I don't have an iPhone anywhere near, so I can't test this out.

I think this is related to #334.

Well, I faced the same issue with type 'm4a'. I couldnt play it in Itunes.
In my case, I kept the same format just removed video encoding.
ffmpeg -i "mysong.m4a" -vn -acodec copy "1_mysong.m4a". Hope it helps.

@Amit-L I see. Thanks for reaching out, this is certainly very helpful. I want to check this one last thing so I can understand what's going on.
Could you try downloading the m4a stream using youtube-dl with this command:
$ youtube-dl -f "bestaudio[ext=m4a]" "https://www.youtube.com/watch?v=E7q_wbbwGyU" -o "test_audio_youtubedl.m4a

and using python (with pafy):
>>> import pafy
>>> pafy.g.opener.addheaders.append(("Range", "bytes=0-"))

>>> video = pafy.new("https://www.youtube.com/watch?v=E7q_wbbwGyU")
>>> audio = video.getbestaudio(preftype="m4a")
>>> audio.download("test_audio_pafy.m4a")

Then try playing these downloading audio (test_audio_youtubedl.m4a and test_audio_pafy.m4a) on iTunes. If I'm right, the one downloaded via youtube-dl should play properly and the python downloaded should be broken. Is this the case?

I also want to know whether this problem happens with every .m4a downloaded with spotdl?

You are correct. The one with youtube-dl worked fine in itunes and pafy failed.
Yes it happens with every .m4a file. If you remove video encoding it works fine.
We need to check how to not encode video meta.


connect to SimpleFSList

This is a bare minimum that loads folder list from RPC. I haven't made the waiting etc. yet, but wanted to make sure what I have looks good before moving on.
@chrisnojima I couldn't make takeEveryPure to work, so I went with the takeEvery approach like in chat.
And here's a screenshot that got myself pretty excited:

Hmm it looks like the breadcrumbs doesn't work for subfolders. It just doesn't show at all for e.g. /keybase/public/songgao/me. I'll try to figure out why tomorrow.

Please take another look @chrisnojima @jzila, now with pending progress tracking!

I don't understand the argument for this path function from node. The data comes from us and is clean right so why isn't it just a simple join with /

I'd prefer to make those constants than pull in this code
On Sun, Feb 11, 2018 at 6:21 PM Song Gao<EMAIL_ADDRESS>wrote:

@songgao commented on this pull request.
In shared/actions/fs.js
https://github.com/keybase/client/pull/10483#discussion_r167454582:

@@ -0,0 +1,54 @@
+// @flow
+import * as Constants from '../constants/fs'
+import * as FsGen from './fs-gen'
+import * as I from 'immutable'
+import * as RPCTypes from '../constants/types/rpc-gen'
+import * as Saga from '../util/saga'
+import * as Types from '../constants/types/fs'
+import * as FsPath from '../constants/fs-path'

+function* folderList(action: FsGen.FolderListLoadPayload): Saga.SagaGenerator<any, any> {

if (action.payload.path === '/keybase') {

As mentioned above, this is because the RPC as of now doesn't support
listing /keybase. I'll make the changes in SimpleFS to support that, and
will remove this along with the private, public, team constants
afterwards. Sounds good?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/pull/10483#discussion_r167454582, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfhU4KTZXvJogQgWBc9YPw4AFY1g9xuks5tT3XwgaJpZM4R_RMI
.

@chrisnojima We'll be supporting pasting in paths in M2 or something right? I was thinking that path will need to be cleaned.

I'd prefer to make those constants than pull in this code

Hmm sorry I don't fully understand. When navigating to /keybase, it'd still trigger the RPC right?
I guess alternatively I could also make the SimpleFS change first before merging this PR so this code never goes in master.

Let's not add complexity for things that aren't in m1. This is literally the first pr to actually hook up the data. Let's not focus on future features like pasting paths

OK sounds good; will fix!

@chrisnojima please take another look; thanks!!
Note that the FS tab will appear broken until https://github.com/keybase/kbfs/pull/1480 is in.

Thanks! Will hold off merging until KBFS side is in.


[plot3d] Add a general purpose widget for 3D rendering

Target is a replacement candidate for PyMca Object3D widget that can display:

3D scalar field on regular grid (as ScalarFieldView)
3D scatter plot
2D image on regular grid
2D scatter plot

and the associated widget to set the parameters of the scene.

Status:
Scene items:

[x] 3D scalar field on regular grid with iso-surfaces and cut plane (one for now)
[x] 3D scatter plot as 3D points
[ ] 2D image on regular grid:

[x] as 2D image
[ ] as surface plot

[x] 2D scatter plot as: points, lines or surface (through Delaunay triangulation)
[x] Clip plane (one for now)
[ ] 3D mesh

Features:

[ ] Per item transform:

[x] translate, rotate, scale, 3x3 matrix
[ ] configurable center of rotation

[ ] Per item bounding box
[ ] Per item lighting enable/disable
[ ] Picking of item and position in items
[ ] Planes interaction
[ ] Parameter tree widget to update item properties/scene content

Extra:

[ ] Multiple cut planes
[ ] Multiple clip planes

Misc:

[ ] Documentation
[ ] Sample code
[ ] Tests

Closing as most features are available.
Using one issue per required new feature is now a better way to track this.


[Tabular] Fix ImagePredictor

Issue #, if available:
ImagePredictor was not properly trained in multimodal preset after the transition to MultiModalPredictor.
This issue wasn't caught by our unit test because it doesn't crash AG training. An example for the failure could be found in our documentation site
Description of changes:

Filtered the dataframe to contain only image features prior to call _fit

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

Updated the PR to fix the original reason for the bug.
Also added specific feature type passing from tabular to AutoMM instead of having AutoMM infer feature types.


Fix runtime error

@mjudeikis Wasn't this the same bug @thekad found in closed PR #734?  I thought Angus wanted this in?

@troy0820 yes, but too early. Will raise again :) too much multitasking happening :D

This got fixed by @jim-minter in #825 (or at least partially)

actions do not have f there is panic somewhere hidden in here... I got it once, but didn't tracked "/


Applications/156253778

Pivotal Tracker Story ID: [#156253778]
Description of Changes:

go to home page
hover and click the nav link
now it works

@sexta13
I know what you mean. Since #154707979 has been closed, I will create a new pull request for this revision on ProjectDetail. Will be done soon.
Thanks

Ok ;) Thanks.
By the way, and I don't know exactly if related with this story(ies) I noticed that I can directly access pages that should be only available to logged in users. Can you double check that please? Thanks

@Mongolialady , can I close this PR? Or are you still doing something for it?

@sexta13
This branch is not used any more. I created a new branch bug/#156253778, which is going to merge to master.
This one(application/#156253778) can be closed or deleted.
Thanks,
Haiyan


PHP 7.3 support in 3.1

When I try and run orocommerce 3.1 on PHP 7.3 I see these errors when I visit the admin dashboard
2019/06/28 14:01:47 [error] 29657#29657: *1880 FastCGI sent in stderr: "PHP message:
PHP Warning:  preg_match(): Compilation failed: invalid range in character class at
offset 72 in /var/www/html/orocommerce-application/var/cache/prod/srcProdProjectContainerUrlMatcher.php
on line 1549PHP message: PHP Warning:  preg_match(): Compilation failed:
invalid range in character class at offset 72 in /var/www/html/orocommerce-application/var/cache/prod/srcProdProjectContainerUrlMatcher.php
on line 1549" while reading response header from upstream, client: redacted, server:
redacted, request: "GET /admin/dashboard/view/1?change_dashboard=1 HTTP/2.0", upstream:
"fastcgi://unix:/run/php/php7.3-fpm.sock:", host: "redacted"

Is there any plan to support PHP 7.3?

Hi @jasonblewis,
PHP 7.3 compatibility is in our backlog, at least it must be supported in OroCommerce 4.1 LTS. Currently, I'm not sure will it be ported to 3.1 as there is a lot of changes.
Please downgrade PHP to 7.2 for now.

The minimum required version for OroCommerce 4.2 LTS is 7.4, so I'm closing the issue as it's not relevant anymore.


Removal of interfaces should not require many params

Removing trunk or vlan interfaces requires some amount of params that the define would expect to receive when creating.  This should not be required during removal of an interface.

I believe this has been resolved by unifying the configuration validation puppetx library.


Update metadata to API v61, add Apex Docs to code, improve help text and descriptions.

I have implemented Cache Manager in my organization therefore it went through code review and was aligned to the current API and I have added as many descriptions as I could. I want to contribute with that changes to your project so others can easily understand how it works.
Changes:
Add Apex Docs to CacheManager.cls based on similar Nebula logger class and own understanding
Renamed PlatformCacheType to PlatformCachePartitionType in CacheManager.cls to align it with Nebula Logger class.
Add debug messages in case cache partition is not found to help in case of debugging.
Add description to cache CacheManagerPartition.cachePartition-meta.xml
Changed package version to 1.0.3
Updated Readme.md by required and optional configuration after installation and how to diagnose cache.
Add help text and descriptions based on Salesforce documentation lecture.

@jongpie Please review my changes.


No artifacts to source

CLOSES #249

Would it be possible to create a flag to make it global and without touching the pipeline code with decorators?
Something like gitlab-ci-local --no-artifacts-to-source


Requires 'laravel/framework' really necessary?

I would quite like to implement this library into my Lumen based project, however, when I install it, it removes all of the individual `illuminate/*' libraries I have added, and installs the whole of Laravel, which, in turn, breaks my project.
Is it possible to amend the composer.json to not require Laravel? Maybe just require the individual illuminate libraries it requires? Or, alternatively, create a form specifically for Lumen? I am happy to do this but would rather it was done by the author of course :-)

Could you please checkout the lumen branch and test thoroughly? Just to see if there are other issues/errors in Lumen after substituting the laravel/framework dependency.

I have loaded that dev-lumen branch and for the minimal functionality I require the library for, it works perfectly. To test it thoroughly will take me some time but as far as I can see, it works brilliantly!
Thank you very much!!

I've tagged a new release (1.2.2).


Get started with Flux using Helm instructions not lead to working example

Followed those steps https://github.com/weaveworks/flux/blob/master/site/helm-get-started.md
No pods/deployments in test namespace, see log file attached with
kubectl -n flux logs deployment/flux -f
output
flux.log

@kopachevsky the Flux Helm Operator that you've installed is no longer compatible with the CRD used in the flux-helm-test repo. Sorry for this, I've updated the docs in #1527

@stefanprodan nice to see such quick fix, can you clarify what documentation file should I use for quickstart? same helm-get-started.md?

You can use this https://github.com/weaveworks/flux/blob/60dc37429a5f61762851ad4e286a7a18c6cdd8dc/site/helm-get-started.md

Closing this as the docs have been updated in master.


Add safety rail for register_children method

This gives a beginner-friendly error in cases where Masonry detects that register_children() was improperly implemented.

When would someone want the implementation of register_children to do anything other than register each of the things in children_ids?

Never, hence why we have a safety rail which checks that they're the same.
I'm still thinking about how to document this, but basically children_ids acts as the source of truth, and safety rails will give error messages that assume that the other method that doesn't match children_ids is wrong. (In my experience, children_ids methods are very easy to write and people never get them wrong. On the other hand, it's easy to forget to iterate over children in layout or register_children.)

Curious why there are essentially two sources of truth here.

Basically the problem is that children_ids returns, well, ids, but when we want to add new widgets to the tree (the purpose of register_children), we need access to WidgetPods. In principle we could have a children() method that returns Vec<WidgetPod<dyn Widget>> or something, but that adds a lot of complications.

If there's no rush, might as well see about the comment improvements before approving rather than have to remember that this needs further work again.

Oh, I thought you were talking about the formatting thing.

I've updated the documentation to make it a bit clearer.

"Panics in dev mode", then?

Maybe "trigger debug assertions"


go: 1.15beta1 -> 1.15

Actions taken:
ssl patch removed ARM section, file no longer exists
bump source version and hash
Motivation for this change
https://golang.org/doc/go1.15
Things done
Modified patch to remove failure. File no longer exists. Needs additional testing on other architectures.

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

[x] NixOS - x86 only
[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

ssl patch removed ARM section, file no longer exists

It was renamed to root_darwin_ios.go.

Tested and works for me!

@GrahamcOfBorg build direnv

... build direnv

buildGo{Module,Package} is still on go_1_14.

 The distinction seemed to be sandbox vs no-sandbox
armv7l builds and seems to function.
Should we keep/remove the procps dependency? Removing it on linux doesn't seem to break tests and removes lots of dependencies from the build closure.

@ofborg build go_1_15

@zowoq I'm wondering what the state of this is. Which builds do not work right now?
Can we still move forward without setting go1.15 as the default? go1.14 had a regression and projects like Kubernetes now require 1.15…

Can we still move forward without setting go1.15 as the default?

Yes, we can merge this and switch the default in another PR.

@ofborg build go_1_15


Adding support for Binary expression via Acorn Parser

Hey @raj-kumar-s-dev thank you for the PR.
Could you check pipeline please?
There is a build issue


Create Decapode from equation

The user will likely create and edit Decapodes at the equation level. Therefore, having a way for a user to declare equations that are then compiled to work with Decapodes would be a big value add.

Is this just writing the decapodes code itself? E.g.
Advection = @decapode begin
  C::Form0
  ϕ::Form1
  V::Form1
  ϕ == ∧₀₁(C,V)
end

So I think if the notebook let's you write Julia and the sysimage has the libraries loaded this should work out of the box, right?
If the user were to "load in" another decapode (e.g. #21) then they could compose this Advection decapode they craft in the notebook with, say, a Diffusion one they load in from Terarium.
Does that make sense?

@brandomr the idea here is that the DSL wouldn't be used and plain unicode/latex/mathml/something else could be used to declare the equation. The conversion to a Decapode should happen under the hood


gamification brainstorming

This includes the “progress bar” in the profile and the mayor’s words of encouragement above the checklist.
Spoke Heidi about this, and she recommended that we talk to John. Added to Tuesday 9/13's agenda.


Problem building in IOS

When I tried to build for test I got this message:
framework not found IQKeyboardManager

Hi @vesvello,
I have just tested sample-Groceries from release branch on iOS and it seems to work as expected.
Could you provide some more info about the problem and the steps you are following while building the app? It would also help if you could provide some info about your environment(CLI, node versions).

I tried several times...
download de project.

Open the project with Xcode

Goto general option to manage signing

Build the project on device and got this error

ld: warning: directory not found for option '-F/Users/rav/Library/Developer/Xcode/DerivedData/Aditivos-aqcxrszaeigssqdingycmzrmtidk/Build/Products/Debug-iphonesimulator/IQKeyboardManager'
ld: framework not found IQKeyboardManager
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Probably my mistake is open xcode right?

Hi @vesvello,
You should use NativeScript CLI and our run command instead of opening the project via Xcode.
To build and deploy the app on Simulator or device, you could follow the below-attached steps.

git clone https://github.com/NativeScript/sample-Groceries
cd sample-Groceries
tns run ios

If you would like to deploy it only on emulator you could run tns run ios --emulator instead of tns run ios

Hi @vesvello,
Do you still have an issue with building sample-Groceries for iOS?


Feat:Adding the Nav Bar ,adding the suggested songs

Describe the feature
ADDING the nav bar and adding the suggested songs @Satyam1923 @
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Please allocate this task to me. I'm capable of tackling it as I possess all the requisite skills to address the matter at hand.

@Venkatknight2295 if you want to work on it then you have to prepare different react components for it.

@Satyam1923 sure what components you want me to build

@Venkatknight2295 check figma design

ok i will do the signin and create an account page does it work??

Thank you

Hii @Satyam1923
I am interested to do this project can you please assign this task to me.


Kusama support: Implement Metadata v7

Need to create /structs/metadataV7.h that implements metadata v7 according to https://github.com/polkadot-js/api/tree/master/packages/types/src/Metadata/v7
Also, need to decode metadata in class MetadataFactory in files metadatafactory.h and metadatafactory.cpp.

@chernant the team let me know the PR looked good, feel free to submit here.


Routing form target port

In routing form target port are accepting string, It should accept only numbers
use custom validation in getFieldDecorator

Done. coming in v0.16.0


Create Spark Import Pair[Key,Value]RDD to accumulo Operation

This operation will take an RDD of accumulo keys & values and bulk load the contents on that RDD into accumulo.
This will live in the spark-accumulo library

Merged into develop.


Add redirect field to Python operations

What problem is your feature request trying to solve?
In https://github.com/MobilityData/mobility-database-catalogs/pull/309, redirect was added to the schema and CSV, but not the Python operations. That means right now, if I want to add redirect and still generate the right bounding box I need to
Add a feed via the add_gtfs_schedule_source or add_gtfs_realtime_source operations
Push PR
Modify JSON to include redirect directly
Update PR
rather than add all fields through the operations.
Describe the solution you'd like
Include redirect in Python operations, to add and update gtfs schedule and gtfs realtime feeds.
Include redirect object in schema in README

@cka-y Looks like the test is validating for a string value, when redirect is a number: https://github.com/MobilityData/mobility-database-catalogs/commit/4b71837339c26728d51161676ae6609907d683b5#diff-92300112b36301d0886c9786601b64e36393c1ae3dd25102b2ce3af4cfb72795.

@cka-y Looks like this is breaking Fred's script:

Traceback (most recent call last):
46  File "", line 2, in
47  File "/home/runner/work/mobility-database-catalogs/mobility-database-catalogs/tools/operations.py", line 142, in add_gtfs_schedule_source
48    catalog.add(**data)
49  File "/home/runner/work/mobility-database-catalogs/mobility-database-catalogs/tools/representations.py", line 168, in add
50    if len(redirects) > 0:
51       ^^^^^^^^^^^^^^
52 TypeError: object of type 'NoneType' has no len()

Are the operations not set up to take the redirect as an optional field? The script run we did last night had 0 redirect fields populated.


spanner: commit stats example

LGTM.

@jiren I've done a release of the Spanner client for Ruby so you should be able to get the tests working. Please merge once the tests pass.

@jiren I've done a release of the Spanner client for Ruby so you should be able to get the tests working. Please merge once the tests pass.

Ok.


External links to GitHub and/or NPM?

I'm not sure if I'm just not seeing it but are there any external links to GitHub and/or NPM? By the way, awesome project!

You can use markdown in your docs so you should be able to use normal [link](url) syntax.


Range of publications in publication page

Is there any way to set the section size in _pages/publications.md? I understand that it involves the archive-single.html but i'm having difficulties figuring out how to modify that.

This looks like something that should be addressed at the same time as #48

Looking at this item a second time, what do you mean by the section size? The formatting of the page or the number of items included?

Closing since this issue looks to be stale.


Created a provider to inject the session object into Elogram's container

Thanks, I'll take a look in this as soon as I've time to spare.

Please merge this

This repo is just useless until this is merged, isn't it?

I'll take a look at this soon. No worries.

Is this fixed and merged yet? Need this library.

Green = not merged
Purple = merged
Red = closed

That change I pointed out still needs fixing.

Closing this in favour of https://github.com/larabros/elogram/issues/40


prevent the game from crashing when it is open in two tabs

same idea as in #139
=> before joining a game check the local storage before clearing it. if the token is valid join the game and skip the rest. if not continue as usual.

closed by https://github.com/sopra-fs23-group-13/meme-it-client/commit/14c55c6645135b829cd46e22ca5b03b86b7048e2


SplitDetail model and additional fields

Summary

this wraps split details in another model so we can validate these fields only when they're being edited in this form
this PR also includes two new fields, location and platform

/domain @Betterment/test_track_core

nanda?

bump @noveng05

<< platformlgtm
I would say my review was more of a platform review. I'd feel more comfortable if someone who's more close to the test track domain can take one more quick look at it! @Betterment/test_track_core

<< domain lgtm platform lgtm


Cleanup 6/6: Refactor ./blockchain/validator

There's still an issue with the block height. I saw what I think is the same issue on develop, so I don't think it's caused by this refactor.
It looks like the BPT is not always written to disk. If I print out the block height and mdRoot in ./internal/abci.Accumulator.Commit, I see something like this:
I[2021-09-27|01:38:51.761] executed block    module=state height=1 num_valid_txs=0 num_invalid_txs=0
I[2021-09-27|01:38:51.777] committed state   module=state height=1 num_txs=0 app_hash=0000000000000000000000000000000000000000000000000000000000000000

I[2021-09-27|01:38:53.268] executed block    module=state height=2 num_valid_txs=0 num_invalid_txs=0
I[2021-09-27|01:38:53.276] committed state   module=state height=2 num_txs=0 app_hash=0000000000000000000000000000000000000000000000000000000000000000

I[2021-09-27|01:38:59.148] executed block    module=state height=3 num_valid_txs=1 num_invalid_txs=0
Committed height=3 hash=0AF24917BF784EFD9C2B9EDE7E6256BB5D75F139F6248EE3DFC0C50C36FD7FAD
I[2021-09-27|01:38:59.543] committed state   module=state height=3 num_txs=1 app_hash=0AF24917BF784EFD9C2B9EDE7E6256BB5D75F139F6248EE3DFC0C50C36FD7FAD

I[2021-09-27|01:39:00.603] executed block    module=state height=4 num_valid_txs=2 num_invalid_txs=0
Committed height=4 hash=E3D3D5DB76646EF96DC6C97B4D7B677BAE1A85D5D4D0D1D425B4701B2ABCE505
I[2021-09-27|01:39:00.803] committed state   module=state height=4 num_txs=2 app_hash=E3D3D5DB76646EF96DC6C97B4D7B677BAE1A85D5D4D0D1D425B4701B2ABCE505

I[2021-09-27|01:39:02.111] executed block    module=state height=5 num_valid_txs=1 num_invalid_txs=0
Committed height=5 hash=9CF3CAAEDBB778EF7F7E1046BC9EC7EB2D986C09EA0BB2FCB39172280CA95051
I[2021-09-27|01:39:02.453] committed state   module=state height=5 num_txs=1 app_hash=9CF3CAAEDBB778EF7F7E1046BC9EC7EB2D986C09EA0BB2FCB39172280CA95051

I[2021-09-27|01:39:03.570] executed block    module=state height=6 num_valid_txs=0 num_invalid_txs=0
I[2021-09-27|01:39:03.585] committed state   module=state height=6 num_txs=0 app_hash=9CF3CAAEDBB778EF7F7E1046BC9EC7EB2D986C09EA0BB2FCB39172280CA95051

However, if I shut down the node and reload it, sometimes I get the correct result, and sometimes the BPT root hash appears to be one version back:
Good:
Loaded height=5 hash=9CF3CAAEDBB778EF7F7E1046BC9EC7EB2D986C09EA0BB2FCB39172280CA95051

Bad:
Loaded height=5 hash=E3D3D5DB76646EF96DC6C97B4D7B677BAE1A85D5D4D0D1D425B4701B2ABCE505
panic: state.AppHash does not match AppHash after replay. Got
E3D3D5DB76646EF96DC6C97B4D7B677BAE1A85D5D4D0D1D425B4701B2ABCE505, expected 9CF3CAAEDBB778EF7F7E1046BC9EC7EB2D986C09EA0BB2FCB39172280CA95051.

In the bad case, it appears as though block 5 needs to be replayed, so the actual height is 4. I tried playing with the height, but if I tell Tendermint that the latest block height is 4, it replays block 5, and MerkleManager throws a fit because it saves the actual latest block index (5) and throws a fit if it sees block index 5 again.

the problem is because we don't have code in the StateDB to support replayed blocks yet.


Upgrade PHP to 5.6.21 & 7.0.6

We should think about a way to store away the precompiled binaries in a trusted way somehow and only compile them when needed.

Once again the PGP Servers are down or deny traffic from travis or whatever. -.-
Feel free to restart the build in a while.

Thanks @cedricziel !


add notebook explaining the effect of jittering

Closes #180

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 91.54%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
5984

Relevant Lines:
6537

💛  - Coveralls


Disable console.log() on content change

I'm using the component for Vue.
When text is edited, the content is shown in console.log(). There is no prop to disable it. Could you remove it?
This behavior can be seen at: https://highlightable-input.vercel.app/

Fixed in v0.2.3.


BZ 59203 - interrupt tomcat threads instead of stopping

interrupt tomcat threads instead of stopping. It seems existing test cases are already checks whether the ThreadPoolExecutor is terminated.

Thanks for the PR.
There are multiple different changes in this commit. It needs to be broken down into one commit for the changed described in the PR summary and one commit for each type of clean-up.
The configuration option that controls this behaviour is called clearReferencesStopThreads. That makes no sense with this PR applied. However, the behaviour of stopping threads needs to be retained.
Note there are multiple places where stop() is called. This PR addresses only one of them.
It would be better to align the behaviour for Executor and non-Executor threads. i.e.:

interrupt
pause
stop if still running

There was some debate whether there should be multiple iterations if interrupt/pause but I'm leaning towards not. We are dealing with broken web applications here.

I will remove the cleanup changes.
So apparently we should call interrupt, check if thread is still active, if yes then call stop. Correct me if I'm wrong here.

Thanks for the updates. Exactly the sort of thing I had in mind.
Looking at the updated code, I'm wondering whether a slightly wider refactoring makes sense. Something like:

for each thread where the web application class loader is the context class loader (this code is already present)

if using executor, stop the executor (which should interrupt the thread) else interrupt the thread
add the thread to a list of threads that should be stopping

wait until all the threads that should be stopping have stopped or a set amount of time passes (currently we use 2 seconds)
call stop on any threads still running

i.e. interrupt all threads, wait for all threads, stop all remaining threads rather than interrupt, wait, stop one thread at a time.

Looks like the first two steps are already covered in the code. Small refactoring would be to ensure interrupt is not called on executor thread.
if (!usingExecutor && !t.isInterrupted()) {
       t.interrupt();
}

correct me if I am not on a same page.

@markt-asf could you please suggest if any further changes are required here.

Thanks. I used a variation of your PR to fix this.


/rooms/{roomId}/initialSync support

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions

Describe your problem
Despite being deprecated, /rooms/{roomId}/initialSync is needed to support peeking into rooms for guest accounts.
Would you be interested in adding support for it?
Describe your ideal solution
In order to add support, one need to create

a new endpoint function in the MatrixApi.
two new models to parse the API response (InitialSyncUpdate and PaginationChunk

I created both of them in an extension
This enables adding support for room peeking.
I created a peeking extension, which could be added in matrix_dart_sdk.
The membership is set to Membership.leave to prevent storing the peeked events into the database.
Version
No response
Security requirements
No response
Additional Context
No response

We can just add it manually to this class: https://github.com/famedly/dart_matrix_api_lite/blob/main/lib/src/matrix_api.dart should not produce any problems I think. We should consider marking it as Deprecated() in dart until the mentioned issue is clarifyied. However currently there is no priority for us to do this. Would you implement this?


Delete delta_sigma from repo

As pointed out by @Christopher-Bradshaw, there should really not be more than one way to compute the same quantity. Even though the mean_delta_sigma function is ~30-50% slower than delta_sigma, this is outweighed by the improved accuracy. In any case, this performance hit will be rectified in the v1.0 release when all the functions in the mock_observables sub-package are replaced with versions that use mpi4py to parallelize across GPU-accelerated nodes, so the slow performance of mock_observables is a temporary problem anyway. So the old delta_sigma function should be deleted outright, along with all references to it in the docs and tutorials.

This was done in #980.


update workflows to not reference janeway

We're not using the janeway service account anymore, so this changes out the references to be more generic.

Thank you 🫡


Test-loss

Hi, Jusper, i added noise output based on your model, and now the output of the model are speaker one , speaker two and noise. But how can i change the test loss, let the loss won't calculate the noise si-snr-i into total si-snr-i? looking forward for your reply.

.

Thank you for reaching out with your question. If you want to modify the test loss so that it doesn't include the SI-SNR-I for noise, you can simply remove the noise dimension from the input. By doing this, the model will not consider the noise component when calculating the total SI-SNR-I, thus effectively excluding it from the loss calculation. Hope this helps, and if you have any more questions or need further clarification, feel free to ask.


Debug rare spurious shutdown of the game map

During the burn-in test, at 45 days in, the game map cancellation token was signalled unexpectedly, log.
Since the entity system remained up, it spun and logspammed.
Nobody was online at the time.
However, given that the machine is old and crappy, I can't rule out hardware issues (e.g. bitflips, bad RAM, etc). However, I'd still like to:

[ ] audit the possible ways that the game map cancellation token could be signalled, and if any of them are likely to be hit spuriously.
[ ] verify that circuits (or other game content) can't activate that cancellation spuriously
[ ] recover more gracefully from the map being unexpectedly shut down

Weird observations:

Only 10 instances of perovskite_server::game_state::game_map: Map writeback exiting. I expected 16
However, all 16 cache cleanup threads shut down

Because this took 45 days to manifest, I think low priority. If it happens again more frequently, I'll prioritize it higher.

Commit bc48ba843da2dd5efdf9889d659399ca788976de (on audio, not yet rebased onto main but could be cherrypicked) avoids spinning and logspamming at high speed

When I started the server earlier today, it crashed with too-many-open-files. Adjusting ulimit fixed it.
However, the log didn't show a rocksdb panic (why not?)

I'm more convinced of open files + bad error propagation in the game map. I think some tests with error injection are in order.

Working on an initial commit to handle map errors from the writeback/flush threads. However, there are a lot of surfaces where we need to handle DB errors and initiate crashes properly. Partial list:

[ ] auth (reads db directly)
[ ] Chunk error state: This essentially poisons the chunk, crashing the coroutines of any player that goes near
[ ] Inventory?


Result using original option doesn't return cropped region

so exactly if you set option size to original instead of viewport it returns the image but cropped from 0,0, sizeY,sizeX of the image dimensions instead of the actual cropped part, is this how its supposed to work because my understanding is it should return cropped image but with the size relative to the image size not the viewport size

I'm having the same issue

I have this issue when calling croppie.result() before the user does any zooming.

I think I have found the issue, its a result of prematurely calling self.elements.preview.getBoundingClientRect(), which returns incorrect values.
    function _updatePropertiesFromImage() {
        var self = this,
            ...
            img = self.elements.preview,
            imgData = self.elements.preview.getBoundingClientRect(),
            ...

        ...

        self._originalImageWidth = imgData.width;
        self._originalImageHeight = imgData.height;
       ...
   }
...

imgData.width === 344 but img.width === 480 (as it should be).  Is there any reason img.width is not used directly here?

Oh this appears to be caused by #398 which isn't released yet - how frustrating!  Can we have a 2.5.2 please?

Yeah - I plan on releasing 2.5.2 soon, sorry.

Going to go ahead and close this one since it's fixed in the next release.


How to get container.json from running container?

Not a bug but a question...
Given following snippet:
require 'docker'
container = Docker::Container.create('Cmd' => ['ls'], 'Image' => 'ubuntu:latest')
puts container.json
container.kill
container.delete

I'd like to get the output from container.json for a container that's already created outside docker-api.
As far as I can tell container.json does not take parameters.
What's the recommended way to get this information?

Are you just looking to get a reference to the container?
require 'docker'
container = Docker::Container.all.select { |c| c.id == "" }
container.json

This'll grab all the images that are shown in docker ps.

I was more interested in this kind of stuff:
{
  "Id": "74aa5632b33a400",
  "Created": "2015-07-23T10:20:28.023310481Z",
  "Path": "tail",
  "Args": [
    "-f",
    "/etc/hosts"
  ],
  "State": {
    "Running": true,
    "Paused": false,
    "Restarting": false,
    "OOMKilled": false,
    "Dead": false,
    "Pid": 1740,
    "ExitCode": 0,
    "Error": "",
    "StartedAt": "2015-07-23T10:20:28.392565651Z",
    "FinishedAt": "0001-01-01T00:00:00Z"
  },
  ...

... which is available at container.json.
Docker::Container.all is limited to:
"Docker::Container {
  :id => 74aa5632b33a40070dd,
  :connection => Docker::Connection {
    :url => tcp://<IP_ADDRESS>:2376,
    :options => {
      :client_cert=>\"~/.boot2docker/certs/boot2docker-vm/cert.pem\",                      :client_key=>\"~/.boot2docker/certs/boot2docker-vm/key.pem\",
      :ssl_ca_file=>\"~/.boot2docker/certs/boot2docker-vm/ca.pem\",
      :scheme=>\"https\"
    }
  }
}",
...

@arioch but given each container object, you can call #json to get what you're looking for.
Docker::Container.all.map(&:json)

That did the trick. :+1:
Thanks!


Generated Prisma client TS typings incompatible with graphqlgen typings for optional fields

When using the Prisma client together with graphqlgen in TS scrict mode, the generated typings for optional fields from the Prisma datamodel and the graphqlgen resolver arguments don't match. Example:
datamodel.prisma
type User {
  id: ID! @unique
  email: String! @unique
  name: String
  posts: [Post!]!
}

type Post {
  id: ID! @unique
  createdAt: DateTime!
  updatedAt: DateTime!
  published: Boolean! @default(value: "false")
  title: String!
  content: String
  author: User!
}

For example, content on Post is optional here. Now consider this GraphQL schema:
scalar DateTime

type Query {
  feed: [Post!]!
  filterPosts(searchString: String): [Post!]!
  post(id: ID!): Post
}

type Mutation {
  signupUser(email: String!, name: String): User!
  createDraft(title: String!, content: String, authorEmail: String!): Post!
  deletePost(id: ID!): Post
  publish(id: ID!): Post
}

type Post {
  id: ID!
  createdAt: DateTime!
  updatedAt: DateTime!
  published: Boolean!
  title: String!
  content: String
  author: User!
}

type User {
  id: ID!
  email: String!
  name: String
  posts: [Post!]!
}

The content field is optional in the createDraft mutation. Now, the generated resolver args from graphqlgen for the input to createDraft look like this:
export interface ArgsCreateDraft {
  title: string
  content: string | null
}

On the other hand, the generated PostCreateInput type for the createPost operation in the Prisma client API looks as follows:
export interface PostCreateInput {
  published?: Boolean;
  title: String;
  content?: String;
  author: UserCreateOneWithoutPostsInput;
}

content here is declared as an optional instead of string | null. As fas as I understand, the optional is interpreted as string | undefined. Now, when implementing the createDraft resolver like so:
createDraft: (parent, { title, content, authorEmail }, ctx) => {
  return ctx.prisma.createPost({
    title,
    content,
    author: {
      connect: {
        email: authorEmail,
      },
    },
  })
},

TypeScript throws this error in strict mode:
src/resolvers/Mutation.ts:15:7 - error TS2322: Type 'string | null' is not assignable to type 'string | undefined'.
  Type 'null' is not assignable to type 'string | undefined'.

15       content,
         ~~~~~~~

  src/generated/prisma-client/index.ts:282:3
    282   content?: String;
          ~~~~~~~
    The expected type comes from property 'content' which is declared here on type 'PostCreateInput'

src/resolvers/Query.ts:14:29 - error TS2345: Argument of type '{ where: { OR: ({ title_contains: string | null; } | { content_contains: string | null; })[]; }; }' is not assignable to parameter of type '{ where?: PostWhereInput | undefined; orderBy?: "id_ASC" | "id_DESC" | "createdAt_ASC" | "createdAt_DESC" |"updatedAt_ASC" | "updatedAt_DESC" | "published_ASC" | "published_DESC" | ... 4 more ... | undefined; ... 4 more ...; last?: number | undefined; }'.
  Types of property 'where' are incompatible.
    Type '{ OR: ({ title_contains: string | null; } | { content_contains: string | null; })[]; }' is not assignable to type 'PostWhereInput'.
      Types of property 'OR' are incompatible.
        Type '({ title_contains: string | null; } | { content_contains: string | null; })[]' is not assignable to type 'PostWhereInput | PostWhereInput[] | undefined'.
          Type '({ title_contains: string | null; } | { content_contains: string | null; })[]' is not assignable to type 'PostWhereInput[]'.
            Type '{ title_contains: string | null; } | { content_contains: string | null; }' is not assignable to type 'PostWhereInput'.
              Type '{ title_contains: string | null; }' is not assignable to type 'PostWhereInput'.
                Types of property 'title_contains' are incompatible.
                  Type 'string | null' is not assignable to type 'string | undefined'.
                    Type 'null' is not assignable to type 'string | undefined'.

 14     return ctx.prisma.posts({
                                ~
 15       where: {
    ~~~~~~~~~~~~~~
...
 24       },
    ~~~~~~~~
 25     })
    ~~~~~

As suggested by @Weakky here, we need to In order to fix this, we need to both fix prisma-client and graphqlgen to type all nullable input fields as field?: type | null.

I just found out that this problem also applies to the Flow client. This is the Flow error that's thrown:
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/resolvers/Mutation.js:12:34

Cannot call ctx.prisma.createPost with object literal bound to data because:
 • null [1] is incompatible with string [2] in property content.
 • object literal [3] is incompatible with undefined [4] in property author.connect.

     src/resolvers/Mutation.js
       9│     })
      10│   },
      11│   createDraft: (parent, { title, content, authorEmail }, ctx, info) => {
      12│     return ctx.prisma.createPost({
      13│       title,
      14│       content,
      15│       author: {
 [3]  16│         connect: { email: authorEmail },
      17│       },
      18│     })
      19│   },
      20│   deletePost: (parent, { id }, ctx, info) => {
      21│     return ctx.prisma.deletePost({

     src/generated/graphqlgen.js
 [1] 249│   content: string | null;

     src/generated/prisma-client/index.js
 [2] 276│   content?: String;
        :
 [4] 287│   connect?: UserWhereUniqueInput;

As fas as I understand, the optional is interpreted as string | undefined

Yep

Effect of ? on object keys

As fas as I understand, the optional is interpreted as string | undefined

There is a difference between e.g. a: undefined | 1 and a?: 1
$ ts-node -v

ts-node v7.0.1
node v10.11.0
typescript v3.2.2

$ ts-node -O '{"strict":true}'

type Args = { a: undefined | 1, b?: 2 }

let args: Args

args = {}
// error TS2741: Property 'a' is missing in type '{}' but required in type 'Args'

args = { a: undefined }
// ok

let { a, b } = args
console.log(a, b)
// undefined undefined

As you can see only the a: undefined | 1 variant guarantees that a key is present. But from a consumer's point of view, it does not matter.
null vs undefined in graphql
Consider this schema:
  type Message {
    content: String!
  }

  type Query {
    messages(foo: Boolean, bar: Boolean = null): [Message!]!
  }

In the following queries, observe the resolver args values:
query implicit {
  messages {
    content
  }
}

{ bar: null }

query explicit {
  messages(foo: null) {
    content
  }
}

{ foo: null, bar: null }

To summarize, we have the following graphql-to-typescript mapping:
GQL                  TS
a: String            a?: null | string
                      ^  ^
                      |  | 2. user may pass explicit null
                      | 1. user may pass nothing

a: String = null     a: null | string
                        ^
                        | 1. user may pass explicit null
                        | 2. user may pass nothing, thus gql null default

Reconciling graphqlgen and prism
One thing I would suggest tweaking about the proposed fix is that graphql gen should be able to output either case stated above: a?: null | string or a: null | string. As long as prisma client args have type a?: null | string the types will work.

Excellent work. Hope this gets fixed soon.

I have exactly the same problem.
Graphql-code-generator has an avoidOptionals setting for this.
https://graphql-code-generator.com/docs/plugins/typescript-common
Maybe something similar could be the solution here?

@breytex there is already a clean solution identified though please let us know if you identified any flaws!

@jasonkuhrt do you mean this PR? https://github.com/prisma/graphqlgen/pull/329
I just found it.
Unfortunately I cant test if this solves my problem, because npm install prisma/graphqlgen#1f8b347 gives me "Missing package name". prisma/graphqlgen doesnt have a name specified in the package.json. Any other way I can test this without monkey-patching?
Thanks :)

Also looking forward for this fix. Any updates?

@nikolasburk Should Este switch to Nexus / Yoga2 to allow nullable fields? Thank you.

I am going to close this in favour of #3774 as it has a more recent discussion about this and it covers more generic use case.
@steida Yes, you can use those. I am personally disabling strictNullChecks in my tsconfig while the typings get fixed


[FLINK-18119][table-runtime-blink] Retract old records in time range …

…bounded preceding functions
What is the purpose of the change
This fixes a bug in time range bounded preceding functions that the old records that is no longer required are retracted only if a new record with the same key comes in. This prevents unlimitedly growing state especially when the keyspace mutates over time.
Brief change log

Register retract timer when new record comes in
Retract all records when the timer fires and no more record has came in

Verifying this change
This change is already covered by existing tests, such as OverWindowITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): yes
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Thanks for your contribution, could you add a unit test to this issue? This can help ensuring we don't introduce this bug again in the future.

@KurtYoung Added unit tests and verified the tests fail without fix.

@wuchong Considering the cases you've mentioned it totally makes sense to me to remove the state ttl from time range bounded over aggregations. Thank you for opinion and I'd be glad to work on that.

@wuchong Applied ranged timer timestamp. For the timer namespace it seems to be a big refactoring as you said so left it as TODO.

It seems that the e2e test has failed by timeout. Is there anything I can do?

You can rebase to master and push force to trigger the build again.

Rebased and force pushed to branch... Hope it didn't break anything you were doing?

Passed in my branch: https://dev.azure.com/imjark/Flink/_build/results?buildId=177&view=results

Will merge it.


Cuda 11.7 Issue

When I try to switch to Cuda 11.7 using cuda: "11.7" I'm getting this error:
E: Unable to locate package python-openssl
No idea how they are connected but my yaml works perfectly fine with 11.6.2 but throws that error with 11.7.
Here is the repo and branch (a modified version of the official cog for Stable Diffusion): https://github.com/yekta/stable-diffusion-cog/tree/v2

python-openssl might be a dependency for CUDA 11.7+. Unsure on this - just a theory.
You can install the python-openssl package using the apt package manager by running the following command in the terminal: sudo apt-get install python-openssl

Thanks @krrishdholakia but I'm not sure that's quite the problem here.
@yekta What version of cog are you using? (cog --version will tell you.) Please upgrade to the latest version of cog and let us know if you still see this problem as I believe it was fixed in 31872fd0a4131eef14b046fd654b969c2d4d0657 which was released in v0.5.0.


Camera raw metadata open fail : Unsupported file format or not RAW file

Hi，
I've got some raw data dumped from camera（no header ,just raw meta data），10bit  4160x3120 gbgr. I updated to the latest release version of rawpy， and rawpy always open fail
  File "evalution.py", line 44, in main
    with rawpy.imread(rawfile) as raw:
  File "/home/butter/.local/lib/python2.7/site-packages/rawpy/__init__.py", line 20, in imread
    d.open_file(pathOrFile)
  File "rawpy/_rawpy.pyx", line 266, in rawpy._rawpy.RawPy.open_file
  File "rawpy/_rawpy.pyx", line 662, in rawpy._rawpy.RawPy.handle_error
rawpy._rawpy.LibRawNonFatalError: Unsupported file format or not RAW file

Tried open with Irfanview 4.51 , it could get preview， but could not do well postprocess . raw open param is like below, do I need extra param with rawpy or any hack could be done to achieve that?

Can you upload your image here?
Did you tried to read it using numpy.load() directly?
RawPy (or libraw) may not suitable to this kind of data (without headers).

@Iamanorange  Thanks for reaching out！
The camera data is on a remote lab machine， so I could only get screen shot(remote access) or text from that machine.
tried load like this, but not sure about the unpacked process，
filename="/home/butter/pic/S5K3L6JKM_LITEON.raw"
ImageSize = 4160 * 3120 * 2 # height * width * 2   raw10 is uint16
with open(filename, 'rb') as f:
    data = f.read(ImageSize)
    packed = np.frombuffer(data, dtype='uint8').astype('uint16')

also tried this  from github， could not see a proper image but could see a blurred contour
scene_infile = open(filename,'rb')
scene_image_array = np.fromfile(scene_infile,dtype=np.uint16,count=4160*3120)
scene_image = Image.frombuffer("I",[4160,3120],
                                 scene_image_array.astype('I'),
                                 'raw','I',0,1)
plt.imshow(scene_image)
plt.show()

rawpy may need header data to process， is it possible to pass the npdata and other data in ? I just want a 16 bit linear output jpg for enhance testset by already processed dng with rawpy

import numpy as np
from PIL import Image
import rawpy

# For Adobe DNG image
input_file = 'path/to/rawimage.dng'
rawimg = rawpy.imread(input_file)
npimg = rawimg.raw_image

# For raw image (without any headers)
input_file = 'path/to/rawimage.raw'
npimg = fromfile(input_file, dtype=np.uint16)
imageSize = (3648, 2736)
npimg = npimg.reshape(imageSize)

# Save the image from array to file.
# TIFF is more suitable for 4 channel 10bit image
# comparing to JPEG
output_file = 'out.tiff'
Image.fromarray(npimg/1023.0).save(output_file)

You can use scp to download the image file from remote machine.

import numpy as np
from PIL import Image
import rawpy

# For Adobe DNG image
input_file = 'path/to/rawimage.dng'
rawimg = rawpy.imread(input_file)
npimg = rawimg.raw_image

# For raw image (without any headers)
input_file = 'path/to/rawimage.raw'
npimg = fromfile(input_file, dtype=np.uint16)
imageSize = (3648, 2736)
npimg = npimg.reshape(imageSize)

# Save the image from array to file.
# TIFF is more suitable for 4 channel 10bit image
# comparing to JPEG
output_file = 'out.tiff'
Image.fromarray(npimg/1023.0).save(output_file)

You can use scp to download the image file from remote machine.
hi, I want to know:
how can use rawpy postprocessing for numpy array?

how can use rawpy postprocessing for numpy array?

You can't.


Background service not available

I'm submitting a ...

[*] bug report
[ ] feature request
[ ] support request

General information
Latest Kodi, latest beta Netflix: 0.14.0~beta23. Execution timin enabled
Prerequisites
[*] Are you running the latest version?
Description
At startup Netflix Addon error:
"The requested action timed out. The background service may not yet be available if you just started Kodi/the addon. In this case try again in a moment"
Execution timing is enabled. And after several minutes a general Netflix error occures (Netflix Addon Error: see log file).
Expected behavior: [What you expected to happen]
No errors, off course ;-)
Context (Environment)
Installation
[*] I installed the plugin via zip from the Releases page
[ ] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)
Operating System
[*] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[ ] Linux (ARM)
[ ] Android
Additional informatin on the environment
It is a LibreElec installation
Debug log
http://ix.io/1EAI

Add timeout parameter to the openurl call, on line 76 on ipc.py
You got this error
File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/common/ipc.py", line 76, in make_http_call
it it this....
urllib2.urlopen(url=url, data=json.dumps(data)).read(),
change it to:
urllib2.urlopen(url=url, data=json.dumps(data), timeout=60).read(),
:)

I have added your suggestion, but no difference. Still the same error. I have uninstalled the addon, reset the computer, installed that latest addon version from here: https://github.com/CastagnaIT/plugin.video.netflix
Now I cannot login, get the error at every turn you start netflix. Background services cannot start, and I understand because I cannot login. Basically these latest updates/beta's have rendered the plugin unusable.
Log file: http://ix.io/1EYP

have you tried using defaults?
Why did you enable execution timing?

Yep tried both, did not help. Have found a workaround and it works now:
Uninstalled the plugin and installed version 0.13.22 (https://github.com/asciidisco/plugin.video.netflix)
Then reset the computer, and it asked for credentials
Updated to version 0.14.0~beta23 (https://github.com/asciidisco/plugin.video.netflix)
Reset the computer and it asked again for credentials.
It all seems to work now.
This is the log file: http://ix.io/1EZ5

Previously I tried execution timing, because it gave this error that the background services were not started. I read somewhere that execution timing might help with that. However with or without execution timing, it did not matter. But as you can see the previous post, it is working now  ... somehow.


[camera_web] Make plugin publishable for the first time.

This PR gets the camera_web package ready to publish into pub.dev, as v0.6.0
(Partially addresses https://github.com/flutter/flutter/issues/45297)

Clarify in the README that the plugin is not endorsed yet.
Tweak the list of not-supported operations
Call this v0.6.0, @mariamhas agrees

Thanks @bselwe!!
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

(Heads up @mvanbeusekom, @BeMacized; web support is finally coming!)

ERROR: * line 16, column 1 of lib/src/camera_web.dart: This package does not have stream_transform in the `dependencies` section of `pubspec.yaml`.

Nice check! I'll fix it right away! (I wish it was part of the analyzer checks, and not something on pre-publish)

Please update the PR description to s/0.6.0/0.2.0/ before landing though.

@stuartmorgan I think I scrubbed all the references to 0.6.0, thanks for the reminder!


[3.2] Fix issues with Linux clipboard

Backport for X11 Display Server fixes on 3.2 branch.
1. Implement SAVE_TARGETS mechanism
Master PR: #42652
Allows sending the clipboard content to the clipboard manager on exit to keep the content when using a clipboard manager that doesn't automatically makes a backup when copying.
MULTIPLE selection mechanism also had to be implemented, because in this case, the clipboard manager might request multiple selection targets at once.
Known use case: Ubuntu with XFCE4
Note: On the 3.2 version, sending the clipboard data to the clipboard manager is done in OS_X11::delete_main_loop because the internal clipboard content is cleared before the call to OS_X11::finalize.
2. Implement INCR mechanism
Master PR: #42676
Allows pasting from x11 clipboard to receive data incrementally, which is required when handling data size > 256KB.

Thanks!


Fix link to Kubeflow community in Serving overview site

In https://www.kubeflow.org/docs/components/serving/overview/
under "Further Information" > KFServing: 3rd bullet:

[Community]/docs/about/community/)

Use this PR to fix this (to point to the community url)

This change is

/assign @jinchihe

/lgtm
/approve

Thanks AGAIN @jinchihe !


pooling size > stride

Hey,
Last I checked, theano did not support max-pooling op with size > stride. For example:
MaxPooling2D(poolsize=(3, 3), poolstride=(2,2))

Does keras support it using the cudnn backend?
btw, great work guys!

theano.sandbox.cuda.dnn.dnn_pool supports it without problems (I've been using it for a couple of months). I think it was recently implemented for Theano's default pooling op as well, but I'm not sure how functional this implementation is at the moment.

cool! thanks benanne. Does that mean that there's no such equivalent operation on CPU?

No idea about that. I think they were working on that also, but I wouldn't know since I always run my code on the GPU anyway :)


[CLOSED] Limit sidekiq workers to 2

 Issue by benilovj
Monday 10 November 2014 at 18:27 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/320

If not overridden, sidekiq starts with 25 workers by default. This can
cause problems if too many workers try to connect to the database at the same time,
etc.
This change reduces this number to a more reasonable number (2).

benilovj included the following code: https://github.com/alphagov/specialist-publisher/pull/320/commits

 Comment by benilovj
Monday 10 November 2014 at 21:52 GMT

I've been bitten by this in another app...

 Comment by bradwright
Monday 10 November 2014 at 20:31 GMT

Have we seen actual problems with this?

 Comment by james
Tuesday 11 November 2014 at 13:12 GMT

25 does seem like a lot of workers for our needs. I would have thought 2 is quite low, but currently it seems we are only using sidekiq to publish manuals, which I can't imagine much of a queue building up on. I'm happy to merge unless objections from @bradleywright

 Comment by bradwright
Tuesday 11 November 2014 at 13:15 GMT

It's fine to merge as-is, I'd just rather we had a bit more science than conjecture in PRs changing configuration, so I was making a general point.
Whitehall's concurrency is set to 2 also, so there's prior art.


Suggestions do not receive onBlur event on touch devices

open this code pen on an ipad or iphone
http://codepen.io/anon/pen/ENVGYR?editors=0010

Focus on the second input field, type C
scroll down the suggestion list
tap on the first input field, type c

Observed behaviour: Suggestions on the second input field stay open and doesn't trigger the onBlur event
Expected behaviour: Suggestions should be closed and trigger the onBlur event

This is probably related to #170 (which is outdated).

I am also experiencing this. I thought it was something I had messed up, heh.
I can look into it and see if I can come up with a solution.

@devgeeks Thanks, that would be awesome!

The same issue is here, I close all autosuggestions via saving links and this.autosuggest.setState({ isCollapsed: true }) when user clicks somewhere outside autosuggestion component

I think the problem is when you onTouchStart bind onSuggestionMouseDown method. And in this method you set justSelectedSuggestion variable on true.
Later you have condition to do blur only if justSelectedSuggestion is false. So I think that is the problem.
It is necessary to have this condition in onBlur method? And why do you bind same method on onTouchStart as on onMouseDown?
@moroshko

Any new update on this? I am still facing this issue where I can not hide the auto-suggestion when I click outside of the view with iOS devices

Just ran into this same issue. I'd leave a reaction, but it looks like this issue could use a bump. We use this module on zeit.co/docs and have trouble when iOS Safari users can't close the suggestions list.


 An error occurred! :(

==============

Traceback (most recent call last):
  File "/home/ovrlrdq/.pyenv/versions/3.6.4/envs/venv364zappa/lib/python3.6/site-packages/zappa/cli.py", line 2610, in handle
    sys.exit(cli.handle())
  File "/home/ovrlrdq/.pyenv/versions/3.6.4/envs/venv364zappa/lib/python3.6/site-packages/zappa/cli.py", line 505, in handle
    self.dispatch_command(self.command, stage)
  File "/home/ovrlrdq/.pyenv/versions/3.6.4/envs/venv364zappa/lib/python3.6/site-packages/zappa/cli.py", line 520, in dispatch_command
    if not self.vargs['json']:
KeyError: 'json'

==============

Need help? Found a bug? Let us know! :D
File bug reports on GitHub here: https://github.com/Miserlou/Zappa
And join our Slack channel here: https://slack.zappa.io
Love!,
 ~ Team Zappa!

Your Environment

Zappa version used: 0.45.1
Operating System and Python version: Debian, Python 3.6.4
The output of pip freeze: https://hastebin.com/igepoveked.makefile

Python 2 vs 3 problem (doesn't happen in 2.7). @dfurtado's PR fixes it.

Fixed via PR.


Nick: Watch All[\W_]+conference[\W_]+alert(?!.com(?<=allconferencealert.com))

Nick requests the watch of the watch_keyword All[\W_]*+conference[\W_]*+alert(?!\.com(?<=allconferencealert\.com)). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

All[\W_]*+conference[\W_]*+alert(?!\.com(?<=allconferencealert\.com)) has been seen in 4 true positives, 1 false positive, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Remove button

refs #535

Specifically:

If there is more than one instance of a resource property, a remove button will be available for all instances of the resource property.
If a resource property is repeatable, there will be an add button for just the first instance.


ShellComboBox refereshing ever few seconds, flickering, selected item jumps

Here is the recording on how it behaves (Win 8.1):
http://recordit.co/UF0Q9NJuxJ

Is this happening on the unchanged FileDialog project?
I am also running Win 8.1 and I'm not seeing that behaviour. Could it be caused by a shell extension you have installed?

Good point, I did change some code while debugging, but it trivial.
Nevertheless I will try with clean one.
Well I have some extensions like:
Git, Java, LibreOffice, Notepad++, WinMerge, AVG, although most of them are context menu extensions (don't know if it matters)
I will try it on my laptop, clean win 8.1 intallation, and postback.

I tried it on my laptop, as I said, clean installation of Win 8.1,
only 3 apps are installed: BitTorrent, 7Zip, Chrome and with cleanly built solution.
Same thing happens.

Strange, it really doesn't seem to happen here - I wonder if it could be related to region? I am using a UK version of Win 8.1. What are you using?

That was my first thought,
I tried changing Formats, Location, Keyboard layout, System locale (restarting after) to UK, I had Croatia (Croatian), but still same thing happens.
These two windows installations didn't came from same iso.
One of them was downloaded from MSDNAA, I'm not sure I understand what you mean by "UK version of Win 8.1", language?

Believe I'm seeing something similar on Windows 7 and Windows 10 systems.  It seems to be triggered by Refreshes tripped by SHCNE.UPDATEDIR notifications that are caused by Windows updating things like the Quick Access list with Desktop as parent  or Recent lists with parent shell///AppData/Microsoft/Windows.
Trying to figure out how to suppress ones that can't affect the folders currently open or ?  Wondering if there's some sort of nesting going on because sometimes my app goes Not Responding for minutes while I watch Shell.ShellTreeView.SetNodeImage be called for every folder down from the Desktop I believe multiple times.
Checking now to see if I'm running the latest version of Gong Shell...


bug(Table): Table on delete button click in error message

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

Expected Behavior
delete button click error message
Interactive render mode
Interactive Server (Interactive server-side rendering (interactive SSR) using Blazor Server)
Steps To Reproduce
auto
Exceptions (if any)
No response
.NET Version
auto
Anything else?
No response


Refactor logic of checking predefined cids

Issue summary

A couple of issues with the current implementation

Vec::contains is suboptimal, should use HashSet instead
Cid parsing and Vec/HashSet construction should only be called once

https://github.com/ChainSafe/forest/blob/main/vm/actor_interface/src/builtin/account/mod.rs#L24
Other information and links

@lemmih I made a fix proposal in https://github.com/ChainSafe/forest/pull/2620  let me know if it looks OK, if so I will go ahead and complete the implementation