Handbook / Flipbooks not supported

Dear AZn5ReD,
would it be possible to support these kind of handbooks / flipbooks in your downloader:
https://saplearninghub.plateau.com/icontent_e/CUSTOM_eu/sap/self-managed/Handbook/BIT603_DE_Col62/files/assets/basic-html/page1.html
Best regards!

Hi,
It seems like it is already possible to download it in pdf :

Hi,
It seems like it is already possible to download it in pdf :

Very interesting. I am getting a completely differenz view of the same link:

I am using Google Chrome 88.0.4324.104 (32-Bit) with Windows 10. (Firefox 78.7.0esr, Edge, Internet Explorer cannot display the Learning Hub content properly.)
Could you please extract the link behind "Download PDF"? Will it download the whole book or just one page?

Very interesting. I am getting a completely differenz view of the same link:

I am using Google Chrome 88.0.4324.104 (32-Bit) with Windows 10. (Firefox 78.7.0esr, Edge, Internet Explorer cannot display the Learning Hub content properly.)
Could you please extract the link behind "Download PDF"? Will it download the whole book or just one page?

Here is the link to download the PDF : https://saplearninghub.plateau.com/icontent_e/CUSTOM_eu/sap/self-managed/Handbook/BIT603_EN_Col62/files/assets/common/downloads/publication.pdf
To access it, you have to go on the learning hub, search for the course "BIT603", then select the only result (the handbook), then go to course and finally :

Here is the link to download the PDF : https://saplearninghub.plateau.com/icontent_e/CUSTOM_eu/sap/self-managed/Handbook/BIT603_EN_Col62/files/assets/common/downloads/publication.pdf
To access it, you have to go on the learning hub, search for the course "BIT603", then select the only result (the handbook), then go to course and finally :

Hi @AZn5ReD ,
first thank for the tool !
i have the same  issue with flipbooks (an timeout exception is raised by using the tool ) and no dowload button.
i ahve tried with HY400

do you know a workaround to get a pdf generated or is it posible to adapt the tool ?

Hi @AZn5ReD ,
first thank for the tool !
i have the same  issue with flipbooks (an timeout exception is raised by using the tool ) and no dowload button.
i ahve tried with HY400

do you know a workaround to get a pdf generated or is it posible to adapt the tool ?

Hi,
I've never seen this kind of handbook before. Seems like they are translated so there are 2 layers : the first layer is the image (which has all the charts and images), the second one is the svg which contains the text (which depends on the translation).
So no, the tool won't be working. It could probably be adapted to work with those kinds of handbook, but I'm not sure.

Hi,
I've never seen this kind of handbook before. Seems like they are translated so there are 2 layers : the first layer is the image (which has all the charts and images), the second one is the svg which contains the text (which depends on the translation).
So no, the tool won't be working. It could probably be adapted to work with those kinds of handbook, but I'm not sure.

Hi AZn5ReD, thank you for the English PDF link of BIT603! I wasn't aware that the English version differs substantially from the German one.
Here just some additional findings:

The German translation flipbook can not be downloaded from the guessed link: https://saplearninghub.plateau.com/icontent_e/CUSTOM_eu/sap/self-managed/Handbook/BIT603_DE_Col62/files/assets/common/downloads/publication.pdf
The German translation has no download button, also some parts of the German flipbook look kind of messed; download is only offered in the original (?) English version
The original English version asks me for Flash player (seriously, SAP?!) -> I hope, flipbooks are not the "new" standard for participants handbooks.
<img src="https://i.imgur.com/mFQwgpO.png>

Obviously, there are several kinds of handbooks.
For BIT611 there is maybe an easier to download flipbook, because it consists of single PNGs: https://saplearninghub.plateau.com/icontent_e/CUSTOM_eu/sap/self-managed/Handbook/BIT611_EN_Col62/files/mobile/
As there is nor Flash Player on my PC, anymore, it shows me the 2 alternatives:
Mobile version
SEO version
<img src"https://i.imgur.com/9e4Gse9.png">


chore: always use whatwg-url

This gives us consistency across environments when there are edge
cases (in particular, the browser URL parser handles URLs that
start with protocols very differently, e.g.
new URL("bar://foo/baz").pathname === "//foo/baz").

Can you also create a follow up issue that tracks browser bugs so we know when we can stop rely on a heavy polyfill?
Also so we don't mistakenly replace it with the built in version again. should exist a test case if it dose not already do so.


Change the documentation page.

📚 Documentation
zeel, hear. in online python documentation of this project. I find the same method issue like (#63).

Oh yes, it's been a while ever since we updated our changes will fix them up and release the changes soon

Can I work on this issue?

yeah please go ahead

This screenshot is a great start to documentation. I can see how I could going with this. Looking forward to the updates.
Great to have some telegram connection. Exciting Notifly advertisment showed up on my device. I don't have a linux vm active at the moment. The advertisement alone has convinced me to choose Notifly for some of my upcoming notification needs

Hey @soma2000-lang any updates so far ?

@rexdivakar In the readme file, I can see the send_document part has already been changed to send_file. What else changes can I do here??

thats it please make a PR once your done, apologies for the delayed response !


Add pathlib.Path support to specify CA bundle to use

A have no idea how to cover this by tests with current usage of httpbin fixtures.
httpbin_ca_bundle doesn't provide a path, it's just monkeypatching
maybe you have an idea?)

A have no idea how to cover this by tests with current usage of httpbin fixtures.
httpbin_ca_bundle

Use def test_cookie_allow_localhost_default(self, san_server): example. it provides everything you need.


EditorPlugin panel not compatible with tilemap editor panel

Godot version
3.3stable
System information
window10 pro
Issue description

When use add_control_to_container(CONTAINER_CANVAS_EDITOR_SIDE_LEFT, EditorPanel), add a custom editor panel to canvas editor, the tilemap editor panel not show on the right way, whether set it on left or right. Seems the panel is NOT on left or right to the canvas editor viewport, but below it.
Steps to reproduce
Add a custom panel to the canvas editor by editor plugin, then edit a tilemap, you will see tile selector not show on the right place
Minimal reproduction project
PluginPanelBug.zip

Same in 3.4.beta3, same for GridMap. It possibly happens for any node having such side panel, would be good to list all such nodes.


Building: dependencies aren't added back to Makefile after reconfiguration

Previous dependency (.d) that are present from before a reconfiguration aren't added back to the end of the Makefile when rebuilding.  This means that an updated header file will go undetected, and some object files may not be rebuilt as they should as a result.
This is due to a check in util/add.depends.pl, which will only allow a rebuild of the Makefile (addition of the contents of all .d files) if any .d file is newer than the existing Makefile.  This makes sense, otherwise this rebuild would be done on every single build.
However, this rebuild should also be done if any of the .d files are older than configdata.pm, since that indicates that a reconfiguration has been done recently, which clears the Makefile of all the previously added dependencies.

Marked as urgent, as we have buildbot jobs that are currently not building properly because of this issue


Provide copy&paste for developing operators tutorial

What would you like to be added:
As requested here
https://github.com/kudobuilder/www/pull/84/files#r346814591
Why is this needed:

The getting started was overhauled during docathon week and we created runbooks which has a tutorial
https://kudo.dev/docs/runbooks/admin/create-operator.html#preconditions
@alenkacz do you specifically mean that the website should have a copy button?  it doesn't have that currently but otherwise meets the needs here.   Please reopen if more is desired.


Make plots prettier

The two bar plots in the show motif page are super ugly .. Could easily redo them using plotly?
Intensity plot, maybe better to show in log scale

Probably won't do (1), but (2) shouldn't be hard.

Also see #138
The fragmentation spectra plot could be improved:

Load each spectra from server when the user clicks next/previous, rather than preloading all the data at once.
Add zooming/panning
Display neutral losses on the plot
Display feature annotations from e.g. Magma.


Update Provider.SetRecords documentation to clearly state it is for updates only

This PR aims to clarify that the SetRecords function is only
usable for updating existing records.
As the current API documentation states:

Modifies a record of a zone and returns the updated record

See: https://api.ns1.hosttech.eu/api/documentation/#/Records/put_api_user_v1_zones__zoneId__records__recordId_
Closes #11

Oh wow, I totally missed that one. I will have a look at it. Today I was focussed on integrating the client in a custom Terraform provider.
Thanks for your feedback!

I was not able to reproduce this issue. I will close the PR and eventually reopen if I manage to reproduce it.


See if keyboard is visible from controller

Is there an easy way to do this? I don't see anything in the documentation addressing this. I'm trying to see if the keyboard is visible in order to scroll the page to the selected input because as it is now, the keyboard can cover my inputs so I'm wanting to watch the keyboard and if its visible, either push the divs up or enable scrolling, haven't figured that out yet.
Note: I've tried injecting $mdKeyboard into my controllers but it is undefined when I try to call any of the methods on it or assign it to a variable in the controller. The keyboard displays correctly when I place the directive on an input though.

Figured it out, was my fault. Wasn't injecting $mdKeyboard correctly.


TextCNN with rocm-tensorflow has the same performance with tensorflow-cpu

Please make sure that this is an issue related to performance of TensorFlow.
As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:performance_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No

OS Platform and Distribution (e.g.,Linux Ubuntu 16.04):
== check os platform ===============================================
os: Linux
os kernel version: #1 SMP Wed Aug 7 18:08:02 UTC 2019
os release version: 3.10.0-1062.el7.x86_64
os platform: Linux-3.10.0-1062.el7.x86_64-x86_64-with-centos-7.7.1908-Core
linux distribution: ('CentOS Linux', '7.7.1908', 'Core')
linux os distribution: ('centos', '7.7.1908', 'Core')
mac version: ('', ('', '', ''), '')
uname: uname_result(system='Linux', node='bogon', release='3.10.0-1062.el7.x86_64', version='#1 SMP Wed Aug 7 18:08:02 UTC 2019', machine='x86_64', processor='x86_64')
architecture: ('64bit', '')
machine: x86_64

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
No

TensorFlow installed from (source or binary): binary

TensorFlow version (use command below):
== tensorflow import ============================================
tf.version.VERSION = 1.15.2
tf.version.GIT_VERSION = v1.15.0-16-g29b7532
tf.version.COMPILER_VERSION = 5.4.0 20160609
== check pips ===================================================
numpy                  1.18.1
protobuf               3.11.3
tensorflow-estimator   1.15.1
tensorflow-rocm        1.15.2

Python version: - Bazel version (if compiling from source):
== check python ===================================================
python version: 3.6.9
python branch:
python build version: ('default', 'Jul 30 2019 19:07:31')
python compiler version: GCC 7.3.0
python implementation: CPython

GCC/Compiler version (if compiling from source):
== compiler =====================================================
c++ (GCC) 7.3.1 20180303 (Red Hat 7.3.1-5)
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

CUDA/cuDNN version: - GPU model and memory:
GPU 8G rocm-3.1.0

rocminfo
ROCk module is loaded
root is member of video group
HSA System Attributes
Runtime Version:         1.1
System Timestamp Freq.:  1000.000000MHz
Sig. Max Wait Duration:<PHONE_NUMBER>3709551615 (0xFFFFFFFFFFFFFFFF) (timestamp count)
Machine Model:           LARGE
System Endianness:       LITTLE
==========
HSA Agents

Agent 1

Name:                    Intel(R) Core(TM) i5-4460  CPU @ 3.20GHz
Marketing Name:          Intel(R) Core(TM) i5-4460  CPU @ 3.20GHz
Vendor Name:             CPU
Feature:                 None specified
Profile:                 FULL_PROFILE
Float Round Mode:        NEAR
Max Queue Number:        0(0x0)
Queue Min Size:          0(0x0)
Queue Max Size:          0(0x0)
Queue Type:              MULTI
Node:                    0
Device Type:             CPU
Cache Info:
L1:                      32768(0x8000) KB
Chip ID:                 0(0x0)
Cacheline Size:          64(0x40)
Max Clock Freq. (MHz):   3400
BDFID:                   0
Internal Node ID:        0
Compute Unit:            4
SIMDs per CU:            0
Shader Engines:          0
Shader Arrs. per Eng.:   0
WatchPts on Addr. Ranges:1
Features:                None
Pool Info:
Pool 1
Segment:                 GLOBAL; FLAGS: KERNARG, FINE GRAINED
Size:                    7907180(0x78a76c) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Alignment:         4KB
Acessible by all:        TRUE
Pool 2
Segment:                 GLOBAL; FLAGS: COARSE GRAINED
Size:                    7907180(0x78a76c) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Alignment:         4KB
Acessible by all:        TRUE
ISA Info:
N/A

Agent 2

Name:                    gfx803
Marketing Name:          Polaris 20 XL [Radeon RX 580 2048SP]
Vendor Name:             AMD
Feature:                 KERNEL_DISPATCH
Profile:                 BASE_PROFILE
Float Round Mode:        NEAR
Max Queue Number:        128(0x80)
Queue Min Size:          4096(0x1000)
Queue Max Size:          131072(0x20000)
Queue Type:              MULTI
Node:                    1
Device Type:             GPU
Cache Info:
L1:                      16(0x10) KB
Chip ID:                 28639(0x6fdf)
Cacheline Size:          64(0x40)
Max Clock Freq. (MHz):   1306
BDFID:                   256
Internal Node ID:        1
Compute Unit:            32
SIMDs per CU:            4
Shader Engines:          4
Shader Arrs. per Eng.:   1
WatchPts on Addr. Ranges:4
Features:                KERNEL_DISPATCH
Fast F16 Operation:      FALSE
Wavefront Size:          64(0x40)
Workgroup Max Size:      1024(0x400)
Workgroup Max Size per Dimension:
x                        1024(0x400)
y                        1024(0x400)
z                        1024(0x400)
Max Waves Per CU:        40(0x28)
Max Work-item Per CU:    2560(0xa00)
Grid Max Size:<PHONE_NUMBER>(0xffffffff)
Grid Max Size per Dimension:
x<PHONE_NUMBER>(0xffffffff)
y<PHONE_NUMBER>(0xffffffff)
z<PHONE_NUMBER>(0xffffffff)
Max fbarriers/Workgrp:   32
Pool Info:
Pool 1
Segment:                 GLOBAL; FLAGS: COARSE GRAINED
Size:                    8388608(0x800000) KB
Allocatable:             TRUE
Alloc Granule:           4KB
Alloc Alignment:         4KB
Acessible by all:        FALSE
Pool 2
Segment:                 GROUP
Size:                    64(0x40) KB
Allocatable:             FALSE
Alloc Granule:           0KB
Alloc Alignment:         0KB
Acessible by all:        FALSE
ISA Info:
ISA 1
Name:                    amdgcn-amd-amdhsa--gfx803
Machine Models:          HSA_MACHINE_MODEL_LARGE
Profiles:                HSA_PROFILE_BASE
Default Rounding Mode:   NEAR
Default Rounding Mode:   NEAR
Fast f16:                TRUE
Workgroup Max Size:      1024(0x400)
Workgroup Max Size per Dimension:
x                        1024(0x400)
y                        1024(0x400)
z                        1024(0x400)
Grid Max Size:<PHONE_NUMBER>(0xffffffff)
Grid Max Size per Dimension:
x<PHONE_NUMBER>(0xffffffff)
y<PHONE_NUMBER>(0xffffffff)
z<PHONE_NUMBER>(0xffffffff)
FBarrier Max Size:       32
*** Done ***
You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
To run TextCNN model with AMD RX580 8G, the training and evaluating as follows:
Epoch: 1
Iter:      0, Train Loss:    6.2, Train Acc:  35.74%, Val Loss:    6.1, Val Acc:  34.82%, Time: 0:00:23 *
Iter:    100, Train Loss:    1.5, Train Acc:  95.90%, Val Loss:    1.2, Val Acc:  95.86%, Time: 0:00:54 *
This speed is the same with traing used CPU and radeontop command shows gpu run out.
Describe the expected behavior
We hope model trainned by GPU should faster then trainned by CPU
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
log_13083345 - 副本.log

Hi @hanhanyimo , can you provide the performance numbers on tf_cnn_benchmarks, comparing CPU vs GPU on your local machine?
You can find the instructions here:
https://github.com/ROCmSoftwarePlatform/tensorflow-upstream/blob/develop-upstream/rocm_docs/tensorflow-quickstart.md#tensorflows-tf_cnn_benchmarks
Besides, if you have not, please try with our pre-built docker containers below, to ensure your user-bit environment in good shape:
https://hub.docker.com/r/rocm/tensorflow

Also behaving slow, and the docker is now (once it worked) returns:
>>> print("Num GPUs Available: ", len(tf.config.experimental.list_physical_devices('GPU')))
2020-04-08 20:47:00.458792: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libhip_hcc.so
2020-04-08 20:47:00.473386: E tensorflow/stream_executor/rocm/rocm_driver.cc:975] could not retrieve ROCM device count: HIP_ERROR_NoDevice
2020-04-08 20:47:00.473411: E tensorflow/stream_executor/rocm/rocm_driver.cc:975] could not retrieve ROCM device count: HIP_ERROR_NoDevice
Num GPUs Available:  0

@atomobianco , the log implies ROCm driver stack was not properly configured on your system.
Are you able to execute /opt/rocm/bin/rocminfo?

rocminfo is working as expected, giving the card as second agent
*******
Agent 2
*******
  Name:                    gfx803
  Marketing Name:          Ellesmere [Radeon RX 470/480/570/570X/580/580X/590]
...


ionic2 rc0 chrome browser cannot find typescript .ts file

when i debug in chrome browser, i canot find .ts typescript file there only has .js file.  i cannot debug like before.

duplicated https://github.com/driftyco/ionic/issues/8298

This is a duplicate of https://github.com/driftyco/ionic/issues/8292, so let's track it there. I'm going to close this for now.
Thanks,
Dan

Whoops, that is this issue, so let's leave it open. LOL!
Thanks,
Dan

if Source map file is not available in browser for debugging code. i will cannot work with ionic RC!!!!. i hope ionic team resolve this problem

So bad design

If I run ionic serve, I have source maps for /app, /pipes, /providers,  and /pages is missing,  no typescritp files
If I run ionic serve --lab, I have no /.tmp

We are working on this. This is a high priority item for us.
Thanks,
Dan

Great to see Ionic is working on this; just a +1 here because this is an issue for me as well, but I will work around this as best as I can and try to guess when I debug :)

FYI - This also affects trying to debug in Visual Studio Code using the cordova tools extension. Used to work like a dream in both the browser using serve and directly on a device. Now all I get is the console output coming through, no breakpoints will fire.

When I run ionic serve, I see source maps to transpiled JS.  It is pretty straight forward to debug IMO.  What are you seeing, and what would you like to see?
Thanks,
Dan

I can give you some screenshots of the issue if you would like. Basically in previous versions, the chrome debugger would show all the .ts files. Now it simply shows main.js. None of my pages .ts files appear. If you are getting map files with ionic serve and are able to debug code in your pages, then that's better than what I am experiencing.

@mirata, it is the same for me.
I'm using nvm and tried two different install of ionic and two different startup 'tabs' project, the pages.ts files does not appear both with node versions 4.6.0 and 6.7.0.
In all cases this is what I get:

Actually the scripts are transpiled ,  and i can able to see the scripts in the folder
may be i am wrong
it seems the rollup remove the relation between the map file and the script file

@danbucholtz
I understand that is possible to debug the transpiled main.js file, with all the pages.ts inside it.
I also take for granted that rollup.js generates a code that is more optimized than webpack.
In any case could it be possible to have the previous ionic scripts of the webpack-gulp combination of tools so as to use them during the development?
In my opinion it is much simpler to debug with the separated .ts files, then to have the possibility to use rollup.js for a production build would be optimal and greatly appreciated.

@AntonioRossi, at this time, no.  We will get source maps working. This is a high priority item for us.
I have a hunch the vast majority of changes for RC1 will be in app-scripts, unless there is a major bug found in the framework itself.  We are iterating quickly on app-scripts as we're not satisfied with what we've produced so far.
Thanks,
Dan

@danbucholtz I'm not sure if you're saying that the source map issue won't be fixed until RC1?
If so, will a workaround be published beforehand? This is a really big problem, and I'd really rather not wait for a new RC to be built, tested and published.

I was using Visual Studio to build and debug my ionic beta project. However, since rc0, I could not even build my project using Visual Studio anymore. Please note that I'm not even talking about debugging it.
@danielabbatt (or whoever else knows the answer), you mentioned that you also use VS. Do you have to change anything in your Visual Studio to be able to build Ionic2 rc0 project?

@eddyng8 did you follow the upgrade path in the Changelog?

Yes I did. I could build my project using "ionic build android" and ran it on my device using "ionic run android". So, I knew I followed everything in the Changelog correctly.
Let me back track a little bit. When I used ionic2 beta, I followed the following article to get my ionic2 project built and debugged in VS: https://blogs.msdn.microsoft.com/rido/2016/05/11/ionic2-and-visual-studio/. However, after upgraded my ionic2 to rc0 and followed the same steps again (minus the part that talked about gulp because the gulp built is removed in rc0), I got the following error when pressing F5 in VS:
MSB6006 tsc.exe exited with code 1. File: Microsoft MDA.FileMirroring.targets  line 387
Also, I made sure that VS uses the same Node and npm version as when I did everything in command line by setting the path of the external tools as described in this article: https://blogs.msdn.microsoft.com/webdev/2015/03/19/customize-external-web-tools-in-visual-studio-2015/.
I hope somebody already figured out on how to use ionic2 rc0 in Visual Studio and can help me here.

Amazing...

VS went fine with Ionic2 Beta, but after upgrading it to RC0 i get:
Build:Cannot find module './app.module.ngfactory'
however, ionic build passes and ionic serve works too

@xx0r thats in probably in the main.prod.ts, seams to me like the former Theme File linking to build directory, the file created at the build process and is available when you ionic serve under no-domain folder in the ChromeDevTools under scripts.
maybe you could comment it out and use the --dev parameter in package.json after build to force build with dev flag

Still no solution for this?

+1 Also seeing this issue. Any dirty workarounds until we see a fix?

@danbucholtz how soon can we expect to solve?

@konrin,
No timeline right now but this is high priority for us and we are moving fast with @ionic/app-scripts.
Thanks,
Dan

It seems the JS files are available for debugging when using an inline template: https://github.com/driftyco/ionic-app-scripts/issues/89

Debugging page components is impossible atm. however when I delete the js.map file I can at least set breakpoints in the main.js. otherwise I constantly get forwarded to those ngfactory files which are impossible to debug.

Are you expecting to release the patch as part of rc1, or will it be available before that via an update to @ionic/app-scripts?

This will hopefully be out soon - before RC1.
Thanks,
Dan

This is fixed now. Go ahead and update:
npm install @ionic/app-scripts@latest

Please let me know if there are still some issues.
I will re-open if needed.
Thanks,
Dan

Hey @danbucholtz ,
Thanks for the update 👍  , however it looks like the issue is not fully fixed. I'm seeing some issues when
1 - when I add a breakpoint, the program flow doesnt stop at that point
2 - doesn't show all the .ts files.
Please have a look at the example below (steps to reproduce)

create a new project ionic start demo --v2
updated the scripts (npm install @ionic/app-scripts@latest)

This is what I see in the source maps

however this is the actual directory structure

or its possible I'm missing something here too. :). I've tested this with Chrome Version 53.0.2785.143 (64-bit)
Hope this will help for your investigation
cheers
Sam

@sameera207,
Good catch. Thanks for letting us know.
Thanks,
Dan

I just upgraded to this fix and now rollup isn't capable of finding some interfaces in my Model.ts file. removing the upgrade and going back to the previous version and it works. --> causing the whole build to not happen. it's hard for me to share this though. I'll try and see if the same happens with an ionic start project

@danbucholtz after the upgrade, there was the same problem as in @pascalwhoop with finding some interfaces when building an application.

anyone has an Idea about how i can get debugging work with Visual Studio 2015? I ran the update for the ionic-app-scripts but still the breakpoint will not be triggered. Its however working with beta 11. Any Ideas? I almost gave up

@Rainson12 better give up with RC0 and keep working with Beta 11, it's way better!

@konrin, @pascalwhoop,
Can you provide a minimal example of how to recreate the situation. Can you check the @ionic/app-scripts package.json version attribute to let me know what version you're running?
Thanks,
Dan

@danbucholtz
Package version:
@ionic/app-scripts/package.json
"version": "0.0.29"
Example of error:
[18:09:30]  bundle dev failed:  Module C:/Projects/test/src/models/contact.ts does not export IContactHistory (imported by C:/Projects/test/src/components/contact-history
-item/contact-history-item.ts)
[18:09:30]  Error: Module C:/Projects/test/src/models/contact.ts does not export IContactHistory (imported by C:/Projects/test/src/components/contact-history-item/contact
-history-item.ts)
    at Module.trace (C:\Projects\test\node_modules\rollup\dist\rollup.js:7706:29)
    at ModuleScope.findDeclaration (C:\Projects\test\node_modules\rollup\dist\rollup.js:7329:22)
    at Scope.findDeclaration (C:\Projects\test\node_modules\rollup\dist\rollup.js:5349:39)
    at Identifier.bind (C:\Projects\test\node_modules\rollup\dist\rollup.js:6497:29)
    at C:\Projects\test\node_modules\rollup\dist\rollup.js:5149:50
    at UnaryExpression.eachChild (C:\Projects\test\node_modules\rollup\dist\rollup.js:5166:5)
    at UnaryExpression.bind (C:\Projects\test\node_modules\rollup\dist\rollup.js:5149:7)
    at UnaryExpression.bind (C:\Projects\test\node_modules\rollup\dist\rollup.js:6893:53)
    at C:\Projects\test\node_modules\rollup\dist\rollup.js:5149:50
    at BinaryExpression.eachChild (C:\Projects\test\node_modules\rollup\dist\rollup.js:5166:5)

Ionic info:
Cordova CLI: 6.3.1
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
OS:
Node Version: v6.7.0

In the previous version the project build was successful

@konrin,
Thanks. I will try to recreate and see what's up.
Thanks,
Dan

We published some additional fixes here.
npm install @ionic/app-scripts@latest

It work's, I can debug pages and providers. great job @danbucholtz
thanks.

@danbucholtz  sorry, but the latest version of app scripts does not resolve the interface issue mentioned by @konrin  and @pascalwhoop
I have a standalone ts file that exports a single interface, and it doesn't seem like rollup can process it on ionic serve.  Though I don't get any errors when running ionic build android... This is the error when running ionic serve
[13:45:59]  bundle dev failed:  Module edit-address/IAddress.ts does not export IAddress (imported by edit-address/edit-address.component.ts)
Even though that file does export that interface:
    streetAddress: string,
    city: string,
    state: string,
    zip: string,
    lat: number,
    lng: number;
}```

I can debug in VSCode again, thanks!

Thanks so much to the Ionic team for the hard work!
I haven't made time to check out any of the new app-script solutions that were released recently because I'm working on some UI-related and back-end related stuff at the moment, but I do appreciate the work you guys are doing in squashing the bugs that came up after RC0.

@danbucholtz thanks for the update now it works with "ionic serve", we can debug again our ts files, but still we cannot inspect or log with chrome when we do "ionic run"

Debugging still doesnt work with Visual Studio 2015. I installed VS 2015 npm task runner extension and placed the "watch" task after the build event. But still breakpoint wont be hit. When hovering over the breakpoints while debugging it even says: "The breakpoint will currently not be hit. No code has been loaded for this code location.". Any suggestions?

@joshgarwood @konrin @pascalwhoop,
Can you provide a repo of the interface thing, or tell me what I'm doing wrong?
I made a file called interface-one.ts
export interface One {
   testOne();
}

Then I made another file called interface-two.ts
import { One }  from './interface-one';
export interface Two extends One {
   testTwo();
}

Then my Page/Component looks like this:
import { Component } from '@angular/core';

import { NavController } from 'ionic-angular';

import { Two } from '../interface-two';

@Component({
  selector: 'page-home',
  templateUrl: 'home.html'
})
export class HomePage implements Two {

  constructor(public navCtrl: NavController) {
  }

  testOne() {
    console.log('test method one');
  }

  testTwo() {
    console.log('test method two!');
  }
}

I implemented the interface correctly. It is an extended interface. I can run ionic serve without issue. What am I doing wrong?
Thanks,
Dan

Hey Dan!
Thanks for the quick follow-up, and all of the work to get everything wrapped up!
For me, I'm using the interface purely as a type.  So it doesn't have any functions and nothing "implements" it.  So I have:
IAddress.ts file
    streetAddress: string,
    city: string,
    state: string
}```

Then I have an `edit-address component file`

import {Component, Input, OnInit} from '@angular/core';
import {NavController, NavParams, Events} from 'ionic-angular';
import {IAddress} from '../../user/directives/edit-address/IAddress';
@Component({
templateUrl: 'settings-address.html'
})
export class SettingsAddress {
address: IAddress =
{
city: '',
lat: 0,
lng: 0,
state: '',
streetAddress: '',
zip: ''
};
constructor(public nav: NavController, private navParams: NavParams, private accountService: AccountService, private events: Events, private alertService: AlertService) {

}

ngOnInit() {
  // init stuff here
}

}```

@joshgarwood, can you see my updated test.  Hmm, I am doing the same thing with the shape interface. I wonder what is different.

Hey Dan! Hmm, is it the test in your comment, or in a repo somewhere?  I can't find a shape interface in the comment above, so I'm likely missing something.
Also, I'm using ionic-app-scripts version 30, so maybe I don't have the latest?  The package.json in source control has 30, so I thiiiink I'm up to date.

@danbucholtz i prepared a test project reproducing the error. If you try to build the project through tsc, the error does not occur.
https://github.com/konrin/ion-interface-test

@joshgarwood, @konrin, and @pascalwhoop,
We think we know what the issue is and are working to resolve it.
Thanks,
Dan

Debugging works like a charm now. Thanks!

@danbucholtz thanks for the quick resolving of this issue! When you're sure what was the case, would you mind sharing? For educational purposes.

I'm trying  to implement SQLite in my project, it returns an error "Unable to execute sql", source: file:///android_asset/www/build/main.js"
Implementation inside a constructor :
this.database = new SQLite();
this.database.openDatabase({
name: "data.db",
location: "default"
}).then(() => {
this.database.executeSql("create table if not exists people(id INTEGER PRIMARY KEY AUTOINCREMENT, firstname TEXT, lastname TEXT)", [] ).then(() => {
console.log("TABLE CREATED: ");
}, (error) => {
console.error("Unable to execute sql", error);
})
    }, (error) => {
        console.error("Unable to open database", error);
    });

@danbucholtz any news about the error when building interfaces?

@konrin, working on it as we speak.
Thanks,
Dan

Thanks!

@danbucholtz are these interrelated? https://github.com/rollup/rollup-plugin-typescript/issues/65

I've ran npm install @ionic/app-scripts@latest on my project, and my breakpoints are being hit... but in the transpiled main.js, and not the source Typescript! Is anything else needed?

Before when an error occurred in the browser the console showed me that line of .ts, but now only show me the line in main.js Is right that?

@danbucholtz how to deal with error interfaces imports? I can not for a long time to build the project - for me it is a big problem. It is waiting for an update CLI.
Thank you

@konrin It seems that this happens only when interfaces are used in @Input(). I'm hoping that they fix it ASAP too.
As a workaround I'm using @Input() inputName: any | MyInterface (for some reason this works for me, and when they fix it I'll search for all occurences of any | in my project, remove them and let only the interfaces).

@konrin, we just pushed a fix and are going to start testing it out shortly. Hoping to ship soon.
Thanks,
Dan

@cocowalla : Can you give me a hint how to set up debugging from VS Code. Just new to it but would be great if i can set breakpoints there (regardless of the current prolem :))
Thanks in advance
Ralph

@ralph-fuechtenkort : I suggest you have a look at this.

@lucasbasquerotto great tip this fixes both the compiling and since its a union we get full typing support. very smart :-)

@ralph-fuechtenkort I don't want to derail this thread by going off-topic, but unfortunately GitHub doesn't have a PM facility. Anyway, you just need to install the cordova extension. More info.
Be aware though that it doesn't appear to be working with VSCode in the RC, even with @danbucholtz latest command.

Same issue with ionic run browser. No ts source map in dev tools.

Sometimes I have the problem that after a livereload the sourcemaps are gone and debugging is impossible. If no livereload happens, debugging works fine.

Seems like the latest script changes have fixed the typescript debugging issue for me. However changing html files does not update automatically if using ionic serve. I need to restart each time which is a massive pain. Anyone experiencing this?

@mirata,
What version are you on? We published two versions tonight. I think this should be fixed in 0.0.32, but I could be wrong.
Thanks,
Dan

@danbucholtz Sorry, never mind. I cleared my npm cache, reinstalled dependencies and the issue seems to be resolved.

Sorry to bump in a closed issue, but is this solved?
I'm on RC1 with app-script 0.0.39, deleted the whole node-modules directory, and run npm install again. I can't find any ts source in Chrome, just a bunch of transpiled js files under webpack://./.tmp/

Hi, I just updated app-scripts and I got the same problem again somehow...

no ts files to debug
The command I use is ionic serve with:
"build": "ionic-app-scripts build --dev",
"watch": "ionic-app-scripts watch --dev",
"serve:before": "watch",

Come on guys, whats going on here? This was fixed and now its not.

I tested it using the version 0.0.36 and it works.  But using the versions 0.0.37, 0.0.38 and 0.0.39 it fails.
Hope this helps.

When updating app-scripts or ionic, I always delete the folders  /.tmp ,
/www, and /node_modules, then run 'npm install', this avoids me a lot of
issues.
On Sat, 5 Nov 2016, 3:14 a.m. Jose Blanco<EMAIL_ADDRESS>wrote:

I tested it using the version 0.0.36 and it works. But using the versions
0.0.37, 0.0.38 and 0.0.39 it fails.
Hope this helps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/issues/8292#issuecomment-258593897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCQy7lec8ol_EOePINBErSm1TYrKCg4ks5q7B7HgaJpZM4KJ6V6
.

It looks like the plan is for souremaps to be fixed in app-scripts version 0.0.40.
You can see this (long) thread, skip to the bottom -

debugging again doesn't work with RC2 and Appscripts 0.0.42. Tested it in VS 2015

@Rainson12, use Chrome or a browser for debugging for now. We are using the default Webpack sourcemap set-up so it may or may not work with various IDEs.
Thanks,
Dan

I tried 0.0.43 in Chrome. I could see the *.ts files. But the content was a somewhat modified version of my *.ts files (it looks like WebPack modified them because it has a lot of WebPack decorators in there). I just want to know whether this is expected since WebPack is used instead of rollup? Back in 0.0.36, I saw the original content of *.ts files when debugging. I can live with this if this is the expected results. I just want to make sure that I didn't miss anything.

@eddyng8,
We are using cheaper/faster eval source maps right now by default. You can change them to be the original Typescript source map but the build will be slower. See this blog for more info.
Thanks,
Dan


mlflow.pyfunc.load_model() Unable to Load Mlflow Models from Centralized/Custom Registry

A customer created a registry and added mlflow models to that registry. They are then attempting to load the mlflow model directly from the registry through the following methods.
mlflow.pyfunc.load_model("models:/<model-name>/<version>")
When the model is not a registered asset in the workspace and only shared through a central repository/registry, this method is unable to load the model and returns an exception.
The supported format prefixed with models:/ as documented in https://mlflow.org/docs/latest/python_api/mlflow.pyfunc.html#mlflow.pyfunc.load_model
only appear to work with registered models in the AMLworkspace.
azureml:// is also not listed as a supported format.
This appears to be a current limitation but is not mentioned in the documentation. If this is indeed a limitation, the customer is requesting that this should be clearly stated in the page. As such, may you please revise the documentation to state the limitation and if possible, also include details regarding the correct usage of mlflow.pyfunc.load_model() in AML workspace? Appreciate your support. Thank you!

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

ID: 9b5b3a4d-36cb-9f2b-950d-f6723ff65ba8
Version Independent ID: 9fd1f043-61ff-e52b-d4ef-4993dd130941
Content: Manage models registries in Azure Machine Learning with MLflow - Azure Machine Learning
Content Source: articles/machine-learning/how-to-manage-models-mlflow.md
Service: machine-learning
Sub-service: core
GitHub Login: @santiagxf
Microsoft Alias: fasantia

@iamramengirl
Thanks for your feedback! We will investigate and update as appropriate.

@iamramengirl
I've delegated this to @santiagxf, a content author, to review and share their valuable insights.

Thank you for the timely updates @RamanathanChinnappan-MSFT. I have also opened an internal thread with @santiagxf. Will wait for further updates. Thank you again! :)

Hi @iamramengirl, unfortunately, organizational registries are not supported with the MLflow client. The documentation you shared speaks about workspace's model registries as stated here: Each workspace has its own independent model's registry. The following section explains multiple ways to register models in the registry using MLflow SDK. We hope we can bring support soon. If you can share some details about the use case you are trying to implement with the MLflow client and organizational registries it will help a lot. Thanks!


Constant time crypto operations

Currently the crypto implementation in both backend and frontend is not constant time, since it didn't really seem like a feasible attack model. However, for completeness and because ECDSA is so sensitive to timing attacks it is better to make it constant time.
One could imagine an attacks of someone trying to continuously querying attestation.id for certificates on emails and time the response time to try to learn information about the secret signing key.

closed per #248


Why "fantastic"?

You made a very good list, why would you not want it to be part of Awesome network?

@testman42 Oh, didn't know about that eeggs thing. Cool stuff 🤘


Return Potion error messages in exception for 400s

I'd love to see this or a similar change merged into Potion-Client so that the user of a Python client library can better debug why they are see an HTTPError. This is particularly important in the case of, e.g., Enum fields.
Thoughts @lyschoening or @joaocardoso?

Coverage decreased (-0.3%) to 87.446% when pulling 51e99483cc7f5cd43b14db2c2cfb741c8dd4357c on boydgreenfield:enhanced_status_messages into 11408c9c339595a2a084494e3d357924821b1bae on biosustain:master.

This definitely has be added in some form. My only concern is that the error message can get very long when there are repeated errors e.g. in a list of items. I guess that can be dealt with at a different time.


wrong behavior when changing the max-tokens size

In a translation task on the Europarl training data
and I noticed a very odd behavior.
By doubling the max-tokens value (from 1536 to 3172), the total number of batches per epoch does not half, as expected. Rather It decreases just a bit.
With max_tokens=1536 I have 6634 total batches per epoch
With max_tokens=3072 I have 6188 total batches per epoch
Why does this happens?
Is it a matter of pads inserting to make all sentences in batch equally long.
In this case, I see a very inefficient way to build the batches.

There's multiple possible explanations:

Do you have max-sentences set? Batches are constructed to respect both max-tokens and max-sentences.
What is the lengths of the sentences? Batches are constructed to respect the given max-tokens, but if you have very long sentences then there might be additional fragmentation.
Try setting --required-batch-size-multiple=1. The default value is 8, which is important for FP16 training, but basically rounds down batch sizes to the nearest multiple of 8. Setting it to 1 skips this behavior.

Closing due to inactivity.


Possible to buffer .264 byte to websocket?

Is it possible to buffer raw data to web socket instead of recording? So it be capture with nodejs and stream to real time screen video

It's possible with the underlaying APIs we use here, but it's not something we intend to support out of the box.
The easiest would probably be to use ffmpeg for this.


Attempting to install under Ubuntu (BASH) for Windows 10 errors

I'm running Windows 10 version 1709 (OS build 16299.64) and I'm getting the error below when attempting to install aws-sam-local with npm in Ubuntu Bash:
$ sudo npm install -g aws-sam-local
loadDep:uuid → cache add
...

<EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/aws-sam-local
go-npm install

Downloading from URL: https://github.com/awslabs/aws-sam-local/releases/download/v0.2.2/sam_0.2.2_linux_amd64.tar.gz
fs.js:681
return binding.rename(pathModule._makeLong(oldPath),
^
Error: EACCES: permission denied, rename 'bin/sam' -> '/usr/local/bin/sam'
at Error (native)
at Object.fs.renameSync (fs.js:681:18)
at /usr/local/lib/node_modules/aws-sam-local/node_modules/go-npm/bin/index.js:62:12
at /usr/local/lib/node_modules/aws-sam-local/node_modules/go-npm/bin/index.js:51:9
at ChildProcess.exithandler (child_process.js:204:7)
at emitTwo (events.js:87:13)
at ChildProcess.emit (events.js:172:7)
at maybeClose (internal/child_process.js:821:16)
at Socket. (internal/child_process.js:319:11)
at emitOne (events.js:77:13)
npm ERR! Linux 4.4.0-43-Microsoft
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "-g" "aws-sam-local"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: go-npm install
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'go-npm install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the aws-sam-local package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     go-npm install
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs aws-sam-local
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls aws-sam-local
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/mjensen/npm-debug.log

I'm also getting the same issue installing both locally and globally on RHEL.
$ node --version
v8.9.1
$ npm --version
5.5.1

You should not need to use sudo to install (and it may have caused some permission problems if you  initially tried with sudo, as it creates directories that are root-owned).
Please see the instructions on this page of the NPM documentation to fix your npm permissions:
https://docs.npmjs.com/getting-started/fixing-npm-permissions.


Accessibility support

I try to add accessibility support on your plugin.
I hope it will do the job :)
I used the link I paste in #85 ticket

Hello, thanks @Maskim pour this PR!
I'll try my best to review and merge it. A11y support is definitively something we care and want to implement in TingleJS.

Thanks again for this PR @Maskim
The tabbing navigation is working.👌
Otherwise I have a few comment:
I would prefer to keep the browser's default focus ring instead of a 1px black ring. It's not very noticeable, especially around the close button.
Maybe we could make a nicer focus ring with box-shadow, but it's a bit unsafe knowing that the focusable elements in the modal's content may have box-shadow already.
I would like to have the @robinparisi opinion on that.


kanidm: init at 1.1.0-alpha, add module

Kanidm is an identity management platform written in Rust, that I'm very excited about personally.
I only added module suport for the web application. Kanidm also comes with an optional daemon for PAM/nsswitch integration, but I didn't use it yet and therefore couldn't test it.
Motivation for this change
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Hi, it’s nice too see a pull-request for kanidm. I’m using it for quite a while but never got it clean enough to submit it.
I’m using the PAM integration and – since last week – also the oidc support to connect it to matrix-synapse and hedgedoc.
Maybe you can take some inspiration from the module: https://github.com/Flakebi/nixpkgs/commit/085df2dbebad0f3ceff470f28c30f116ea4a5e10

@Flakebi awesome to see that you came up with an alternative module. Also nice that you got the PAM integration set up. My main use case for kanidm currently is to use it for OIDC, so I didn't really care about PAM, but having it in the NixOS module would be great.
We could work on this together? You could rebase your branch ontop of my on in order to add PAM integration? Then I could pull your commits in. From a first glance you PAM integration looks pretty fine. Only it would probably make sense to also test it in the NixOS integration test.

@mweinelt, I just updated https://github.com/erictapen/nixpkgs/pull/1, which is @erictapen’s package and their module improvements on top of my module.

Please set meta.buildDocsInSandbox = false in the module to fix the manual build.

@Flakebi I'd like to remove the package option, as I did in 936819a10770e82678e271beb768438fb3cbd960.
The general consensus seems to me that a package option is only added if there is an alternative implementation available in Nixpkgs. Otherwise it's cleaner to override the package in an overlay imho.

@mweinelt I'd rather leave backup out of the scope for this initial PR, so this can be used by many people soon.
The one priority I currently have left is improving the test: I'd like to test the kanidm client interaction with the server and the PAM/NSS integration. Testing LDAP integration and maybe even OIDC would be nice too.

I only added module suport for the web application.

But we didn't package the webui AFAIU, is that correct?
 INFO     ┝━ 💬 [request.info]: Request received | src: "fd42:23:42:b864:7285:c2ff:fe67:b6c9" | http.host: "idm.lossy.network" | http.method: "GET" | path: "/pkg/bundle.js"
 WARN     ┕━ 🚧 [request.warn]: Client error --> Response sent | status: 404 - Not Found

@mweinelt thanks for the review, I put up some fixes in https://github.com/erictapen/nixpkgs/pull/2
That should fix the domain name change, the web ui and pam/nss libraries.

After thinking about it for a while I'd like to change the structure of the module. Imho kanidm should be configured as such:
{ ... }: {

  services.kanidm = {
    enable = true;
    ensureDomainName = "example.org";
    settings = { ... };                 # currently serverSettings
    pam = { # or maybe call it unix?
      enable = true;
      settings = { .. };                # currently unixSettings
    };
  };

  programs.kanidm = {
    enable = true;
    settings = { .. };                  # currently clientSettings
  };

}

That would require moving part of the logic into nixos/modules/programs/kanidm.nix.
What do you think about that? I'd start with the restructuring after https://github.com/erictapen/nixpkgs/pull/2 is merged.

Actually instead of programs.kanidm it should likely be security.kanidm, wouldn't you agree?

Actually instead of programs.kanidm it should likely be security.kanidm, wouldn't you agree?

I don't think so, as kanidm strikes me purely as a client program. It doesn't interact with the operating system at all, only with the remote service. I always interpreted security.* as modules with deep OS integration, e.g. polkit or acme.
One similar case would be openssh: programs.ssh is for client config, services.openssh for the server.

Nice fix for the ldap option!

just realised that the PAM daemon also uses the /etc/kanidm/config file. Now I get your point.

Yeah, that was my reason to leave everything under services.kanidm. I find it easier to discover if everything is under one scope (e.g. through https://search.nixos.org/options) and if one doesn’t need to touch programs.kanidm to configure the pam service in services.kanidm.
From what I can see, the security.pam options only change the pam configuration and do not enable any services. Instead, the dovecot service has an enablePAM option that also sets security.pam.services.dovecot2 (cups and doas seem to do something similar). I guess it makes sense to follow the same scheme for kanidm.
I have a working config.security.pam option locally, however, I needed some changes in the pam config that probably shouldn’t be upstreamed, so I need to write a test and trim down the diff before it’s ready. It may take a while until I find the time and motivation to do that, so I don’t want to stall this PR for it :)

The upstream did a refactor of the domain name change handling. Can we upgrade to an unstable version and update our prestart script as needed? It should hopefully get become simpler.
https://github.com/kanidm/kanidm/pull/639

@mweinelt We also have a release on april 1st if you can wait for a month or so :)

This goes into the unstable branch and our next release is at the end of may, so I think we can base our work off of an unstable version and upgrade to the next stable release before the branch-off.

Hey, how is everbody? This has been a bit quieter than I had hoped for. Do people still have the energy to finish this?

Hey, how is everbody? This has been a bit quieter than I had hoped for. Do people still have the energy to finish this?

Next release is may 1st which has the fixes here. We slid the schedule by 1 month to allow for some other things to be developed, and to prevent releasing on jan 1st each year.
@Flakebi @erictapen I'll communicate with you both when the next release is ready so this can go ahead?

I tried updating to the master branch a few weeks ago, but kanidm_domain_change still complained about the wrong origin (or domain?) and I didn’t have time to investigate more since then.

I think the new kanidm release will give me the push to work some more on this :)

Yeah, the notifications are enough, thanks!

Bumped to the 1.1.0-alpha.8 release
removed our ensureDomainName logic and replaced it with serverSettings.domain, accompanied with references to the docs
rebased on latest master, as we need the newer rustc

I tested this previously on a production host with 21.11, but due to the latest glibc 2.34 bump on unstable that doesn't work anymore. It will take me a while to migrate that setup elsewhere.

@mweinelt Interesting, I'm doing the same but the service runs fine for me? How do you compose unstable and stable? I'm just overriding kanidm in an overlay and I add the module to extraModules.

I copied both module and package into my local config tree, substituted cfg.package for package which references pkgs.callPackage ./package.nix and migrated rustPlatform to its nixos-unstable counterpart.

Does recovering the admin account work for you? It errors out for me and I'm not sure what account to login with there.
# kanidm account credential reset_credential  --name admin idm_admin
2022-05-01T14:36:05.510597Z ERROR kanidm_cli::common: No valid authentication tokens found. Please login with the 'login' subcommand.

# kanidm login --name anonymous
Login Success for anonymous

# kanidm account credential reset_credential  --name admin idm_admin
2022-05-01T14:43:34.840094Z ERROR kanidm_cli::common: No valid authentication tokens found for admin.

I think it's a basic task that we should include in the test.

@mweinelt I think you have to login as admin user first. Set an admin password like this: https://kanidm.github.io/kanidm/stable/server_configuration.html#default-admin-account
I definetly want to include that in the test, but retrieving the password from the output might get cumbersome.

From my point of view, there aren't any outstanding issues left, so to me it looks like it's good to merge.
Test coverage could be greatly improved for sure, but I'd like to keep that out of the scope of this PR. It's already better than many tests in tree, that just wait for the systemd unit.
What do you think, y'all? If anybody is willing to merge this, I'll squash the PR into two commits (one for package, one for the module).

# kanidmd recover_account -c /nix/store/nbcbk7a82yb6mn9jl7iwq6ziqh09rrrs-server.toml -n admin |& grep "password reset to" | awk '{ print $NF }'
1uffaUP9cQUKFgEpcZfqAr83EpyD7JsMvHMD0zGu6s3vhXc6

The actual question would be how to get the config path.

I already have something in the works for grabbing the generated passwort, but unfortunately I'm having trouble passing it to kanidm login. Any ideas?

I already have something in the works for grabbing the generated passwort, but unfortunately I'm having trouble passing it to kanidm login. Any ideas?

A small expect script would do this.

@mweinelt Done!

@Artturin does the 12.approvals: 2 label mean this PR needs two approving reviews? Is there some sort of documentation for ths policy? Personally I'd expect this fine to merge with one approval, as it won't break anybody's setup.

@Artturin does the 12.approvals: 2 label mean this PR needs two approving reviews? Is there some sort of documentation for ths policy? Personally I'd expect this fine to merge with one approval, as it won't break anybody's setup.

No, it is the count of how many people have approved this pr
It was added automatically by the following scripts. you can see the reasoning for creating it in the readme
https://github.com/nix-community/label-approved

@Artturin I see, thanks!
@mweinelt Would you like to have this approved by somebody else or would it be ok for you if I self-merge? I really want to get this in-tree so it can be used easily by others.

a) You wanted to squash these changes, I'm happy with merging it as si
b) I'm unclear if you wanted to look further into the password recovery and admin login flow or leave it at that.

I'm sorry, completely forgot about sqashing here.
Let's leave the password login tests for a later PR, so feel free to merge :+1:

Thanks a lot to @Flakebi @mweinelt @Firstyear for your work on this! :partying_face:

there's a ci failure..

Damn, sorry. Fixed in https://github.com/NixOS/nixpkgs/pull/171688.


OpenAI Audio: Timeout exceeded

Hey there,
I'm trying to use CreateTranscription
ctx    = context.Background()
	client = audio.NewClient(openai.NewSession(conf.openAiToken), "")
	resp, err := client.CreateTranscription(ctx, &audio.CreateTranscriptionParams{
		Language:    "en",
		Audio:       aFile,
		AudioFormat: "mp3",
	})

and I got the following error:
Failed to complete: error making a request: Post "https://api.openai.com/v1/audio/transcriptions": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
Looks like it is because the uploading of the audio file happens, and it takes some time. When the file is 1-3MB, then it works, but when I try 7+MB then, I get the timeout error.
Please advise how to fix that moment. I tried to find a solution, but the only one that I found was to change the timeout
httpClient := &http.Client{
    Timeout: time.Minute * 5, // Set a timeout of 5 minutes
}

But I couldn't figure out how then use httpClient and how to pass it to further.
Thanks in advance 🙏

You can set a custom HTTP client as below:
ctx := context.Background()

s := openai.NewSession(os.Getenv("OPENAI_API_KEY"))
s.HTTPClient = &http.Client{
	Timeout: time.Minute * 5, // Set a timeout of 5 minutes
}

client = audio.NewClient(s, "")
// Make a request with the client

Thank you @RussellLuo, it works!
I have also created a helper function that calculates the timeout based on the internet speed and the size of the file.
By the way, I noticed that there is no "prompt" support for audio. Do you have any plans to add this feature, or would it be better for me to fork and work on it myself?

cc @marwan-at-work for the question about prompting.

It looks like prompt is just a string that can be easily added to the current client according to https://platform.openai.com/docs/api-reference/audio/create
I don't see a reason for not adding it to the library's request struct, same goes for the other input parameters.

@Darwinium v1.0.7 added the prompt support, https://github.com/rakyll/openai-go/releases/tag/v1.0.7.


Reflected constructor invocation fails when using null propagation operator in .NET Native

The following code works fine in any .NET runtime environment, including Mono, Xamarin, etc. :
    public class ABC {}
    // ...
    var constr = typeof(ABC).GetTypeInfo().DeclaredConstructors.First();

    var abc = (ABC) constr?.Invoke(new object[0]);
    // abc now contains an instance of ABC

On Windows UWP with .NET Native compilation enabled, the same code throws an exception of type NotImplementedException
However, when the null propagation operator is removed, it works perfectly on .NET Native:
    public class ABC {}
    // ...
    var constr = typeof(ABC).GetTypeInfo().DeclaredConstructors.First();

    var abc1 = (ABC) constr.Invoke(new object[0]);
    // abc1 now contains an instance of ABC

    // the following line throws an exception on .NET Native!
    // but it works fine on any other .NET runtime
    var abc2 = (ABC) constr?.Invoke(new object[0]);

The line in the stack trace where the exception occurs is:
at System.Reflection.ConstructorInfo.Invoke(Object[] parameters)
in f:\dd\ndp\fxcore\CoreRT\src\System.Private.Reflection\src\System\Reflection\ConstructorInfo.cs:line 41

I asked the same question on Stack Overflow: http://stackoverflow.com/questions/43857106/net-native-code-crashes-on-constructor-invoke-null-propagation

Can you give us more details so that we can route this?

What version of Visual Studio is this? (Help -> About in VS)
What CPU architecture you're compiling for?
What version of the Microsoft.NETCore.UniversalWindowsPlatform NuGet package is your project using?

I just tried this in VS and couldn't repro this failure, so I want to make sure we have everything to repro.

It's 100% reproducible on my end when creating an empty project, enabling .NET Native compilation and adding the code I posted here.

Microsoft.NETCore.UniversalWindowsPlatform version 5.2.3
CPU Architecture x86
Visual Studio 14.0.25431.01 Update 3
"Optimize Code" turned off (tried both, same result)

Still no luck in reproing. This could be a bug in devirtualization. Code generation bugs are extremely finicky. Do you mind creating an ilcRepro file with your test project?

I will try to get you everything you need.  Can I post the link to the ilcRepro here or do I need to send it by e-mail?
BTW, this issue is occurring on 3 different machines so it doesn't seem to be environment-related.

E-mail is the proper channel for .NET Native for UWP apps, so taking it there wouldn't be a bad idea (I do get the dotnetnative at microsoft com emails and will be able to follow up).
We can close the issue here - the compiler used for .NET Native for UWP apps is different than the compiler we have in this repo. This will likely be a compiler bug that can't be fixed in this repo anyway.

Ok, I just thought that because the stacktrace clearly indicates that the Exception is thrown in the CoreRT code, that this would be the right place.

Repro file was created and I'll e-mail it to the e-mail address you mentioned.
It turns out that this issue only happens when the code is in a PCL (Profile151) referenced from the UWP project.
Reflection works so it's not caused by missing runtime directives (because the constructor was successfully found and the first line of code where the constructor is invoked without the ?. operator succeeds).

Oh, okay, with Profile151 I have a repro and it's a rather annoying breaking change in the framework.
If you want to follow along:

ConstructorInfo.Invoke(object[]) method in the System.Reflection reference assembly (C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETPortable\v4.5\Profile\Profile78\System.Reflection.dll says that the Invoke method is not virtual.
Somewhere someone decided that the method should be virtual and they changed it in the implementation. The reference assembly that the C# code compiles against was left untouched.
Normally this is not a big deal because C# pretty much always calls methods virtually (even if they're not virtual), because it needs the side effect of the virtual call (throw a NullReferenceException on null this).
Except with the null propagation operator the C# compiler knows that a NullReferenceException cannot occur and it decides to emit a normal call instruction instead of callvirt to prevent the unnecessary null check.
Doing a normal call to the ConstructorInfo.Invoke(object[]) method results in us landing in a method that should never be called.

The good news is that ConstructorInfo.Invoke(object[]) is no longer virtual as part of the NetStandard 2.0 compatibility effort (the previous link was to an old snapshot). That version of .NET Native hasn't shipped yet. The only workaround for now is not to let C# compiler optimize the callvirt to a call by avoiding the operator.
Cc @AtsushiKan


Update http4s-async-http-client, ... to 0.21.26

Updates

org.http4s:http4s-async-http-client
org.http4s:http4s-circe

from 0.21.6 to 0.21.26.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #93.


Modernize and improve bundle size

Problem
At present, the bundle is a whooping 1.4 megabytes big, or 296KB.
For most modern webapps that have to stay competitive and fast, this is a huge impediment. Users are forced to download a lot of unneeded JavaScript.

Solution

Modernize, and break out the bundle into multiple parts using tools like rollup.
Potentially migrate to TypeScript instead of Vanilla JS for improved type handling.

Alternatively

Create a "lean" version of the SDK that can cover most use cases for a frontend dApp, and a separate SDK for backend NodeJS applications.

Alternatives
No response

Hey @kantorcodes,
Thank you for your feedback.
For sure we need to work in that direction. This means a lot of breaking changes. Keep in mind that such kind of changes might be implemented and included in v3 of the SDK. I will keep you updated as well about our plans and what might be done for your suggestions.

Hey @kantorcodes, Thank you for your feedback. For sure we need to work in that direction. This means a lot of breaking changes. Keep in mind that such kind of changes might be implemented and included in v3 of the SDK. I will keep you updated as well about our plans and what might be done for your suggestions.

Thanks! Keen to hear how it goes

Are there any updates on this? The @hashgraph/hedera-wallet-connect package would benefit greatly from being able to tree shake and optimise bundles for client side browser usage.

@SimiHunjan is this still a relevant ticket?


mat-expansion-panel-header :hover style sticks after expansion

Bug, feature request, or proposal:
If you use a custom style for the header, the hover style should go away after you click to expand a panel and mouse away.
What is the expected behavior?
After expanding the original style returns
What is the current behavior?
When you mouse over the header to expand, the hover style is applied.  When you expand and mouse away it stays until you click something else or perform some other sort of action on the page.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-ojcyga
Is there anything else we should know?
Maybe this is not how you are intended to override the default style.  If so I'm not sure how that might be made more clear.

I found a simple workaround by setting the header background colors to !important
.header {
  background-color: #e4e1df !important;
}

.header:hover {
  background-color: #ffcb05 !important;
}

Closing since this is an issue with the custom styling, rather than Material.


[CI] test_shap timed out with > 20s

https://buildkite.com/organizations/xgboost/analytics/suites/xgboost-gpu-2/runs/9fa188f3-2af4-4f39-ac2b-908976e51c07

haven't seen any issue with shap test for a while now. Will close this one for the moment.


Skipped telemetry data as no segments found

I see this error in the daemon's output:
[Debug] Skipped telemetry data as no segments found
What does this error mean? Where do I start debugging?
I'm running daemon v3.2.0 and Wordpress plugin aws-xray (https://github.com/humanmade/aws-xray/) v1.2.3.

Hi sanjeevgopal,
The X-Ray Daemon sends data regarding the Daemon's performance back to the X-Ray service.
Specs are here: https://docs.aws.amazon.com/xray/latest/api/API_PutTelemetryRecords.html
Code: Telemetry Class
Log Message
This is to help identify the impact of potential issues/outtages between the Daemon and the X-Ray service to ensure stability and health of the connection.
This debug message indicates the Daemon did not receive any segments therefor is not reporting any telemetry.
I'm not familiar with that plugin, but it does look like it's hardcoded to expect the Daemon on port 2000. https://github.com/humanmade/aws-xray/blob/f447b53c86de1b28011b86de4a6ac4ce4babec27/inc/namespace.php#L193
Are you not sending segments as expected?

Closing this due to inactivity. Feel free to reopen if you still face the issue.


RHCLOUD-26015 Simplify the Kafka return logic

This PR contains preparation work for the ADR-021 implementation.

/retest


ImportError

I run it on a ubuntu headless server, when try to :
cd examples/02_episode_5/
python launcher.py
got the error:
ImportError: cannot import name 'bool_local_cluster' from 'experiment_launcher.utils' (/home/user/miniconda/lib/python3.9/site-packages/experiment_launcher/utils.py)

it looks like you're running an old version of the experiment-launcher package. What version do you run exactly? Could you try to upgrade to the latest version by running pip install --upgrade experiment-launcher?

I meet the same error, and my experiment-launcher is 2.3. And i can`t find the function 'bool_local_cluster' in the utils.py.

Thanks for bringing this up again!
I guess the issue is actually the opposite, so you're running a too-new version of experiment_launcher. Could you try downgrading to 1.3?

I meet the same error, have u solved?


[BUG] No live payment success confirmation with certain funding sources

Describe the bug
Observed with funding source OpenNodeWallet, BlinkWallet - reported to be the case as well with ZBD funding source by @bitkarrot
To Reproduce
Steps to reproduce the behavior:

Go to Create invoice in LNBits Wallet and pay invoice
Payment settles but does not get picked up live (no payment confirmation checkmark and balance doesn't update)

Expected behavior
When payment settles, invoice should indicate that it was paid and balance should update without refresh
version 0.12.2

Noticed same occationally on 0.12.4 with lnd as funding source.
Sure it was on Android Chrome as UI. Can't remember where payment came from.

Payment notification works fine with BlinkWallet now.
However it is not implemented for OpenNodeWallet

@motorina0  also not implemented for ZBDWallet, also it appears zbd doesn't have a wss endpoint

Closing the issue. There is nothing we can do if the 3rd party does not offer the API.
The LNbits user must refresh the page in order for the payment to be checked again.

Closing the issue. There is nothing we can do if the 3rd party does not offer the API.

We can still do polling, no?
Question is whether we want to poll ...

Yes, we can do polling in two ways:

On the funding source level. But this would make a request for each pending invoice on the server. A server with many users could potentially have thousands of pending invoices. As a result the funding source might get IP blocked.
On the UI level.  If the wallet page is open then the browser can make requests at regular time intervals to update that particular wallet. This means we need a flag on the funding sources to know if they support paid_invoices_stream and only then poll.

Option 2 could be implemented if there is demand.


doc: add ZenHub in readme

Pull Request Test Coverage Report for Build 650

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 649:
0.0%

Covered Lines:
41

Relevant Lines:
41

💛  - Coveralls


HTML, minor performance?

@FreHu can you try this, to see if the performance is a few ms better?

not much, seems to be about 20ms improvement


Handle href for Linear Gradient

For Issue https://github.com/react-native-community/react-native-svg/issues/919
Cache the children incase an href is used.
This wont handle id's which are loaded after a href

The resolution probably needs to be done on the native side, and per svg root. Such that id's only get resolved from within a single svg document rather than globally, and such that the gradients can be defined in any order, as I suspect some people/tools might author such content.

So — 2 years later — any news?


input file at Annotation step?

Hi, you can annotate the input BED file directly, and lift it over afterwards.


evaluation per epoch

Hello
Although I know this error is not related to the implementation, but after spending too much time I appreciate if help me if you know.
I get this error after 3000 iteration for evaluation per epoch:
Fast COCO eval is not built. Falling back to official COCO eval.
INFO:croptrain.engine.inference:Start inference on 274 batches
Traceback (most recent call last):
File "train_net.py", line 94, in
launch(
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/launch.py", line 69, in launch
mp.start_processes(
File "/home/moghaddami/.local/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
while not context.join():
File "/home/moghaddami/.local/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 150, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 1 terminated with the following error:
Traceback (most recent call last):
File "/home/moghaddami/.local/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/launch.py", line 123, in _distributed_worker
main_func(*args)
File "/media/2TB_1/CZDet/DroneDetectron2/train_net.py", line 85, in main
return trainer.train()
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/defaults.py", line 484, in train
super().train(self.start_iter, self.max_iter)
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/train_loop.py", line 156, in train
self.after_step()
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/train_loop.py", line 190, in after_step
h.after_step()
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/hooks.py", line 556, in after_step
self._do_eval()
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/detectron2/engine/hooks.py", line 529, in _do_eval
results = self._func()
File "/media/2TB_1/CZDet/DroneDetectron2/croptrain/engine/trainer.py", line 233, in test_and_save_results
self._last_eval_results = self.test_crop(self.cfg, self.model, self.iter)
File "/media/2TB_1/CZDet/DroneDetectron2/croptrain/engine/trainer.py", line 277, in test_crop
results_i = inference_with_crops(model, data_loader, evaluator, cfg, iter)
File "/media/2TB_1/CZDet/DroneDetectron2/croptrain/engine/inference.py", line 67, in inference_with_crops
all_outputs = model(inputs, infer_on_crops=True, cfg=cfg)
File "/home/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 885, in forward
inputs, kwargs = self.to_kwargs(inputs, kwargs, self.device_ids[0])
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 993, in to_kwargs
kwargs = self._recursive_to(kwargs, device_id) if kwargs else []
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 986, in _recursive_to
res = to_map(inputs)
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 981, in to_map
return [type(obj)(i) for i in zip(*map(to_map, obj.items()))]
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 977, in to_map
return list(zip(*map(to_map, obj)))
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 981, in to_map
return [type(obj)(i) for i in zip(*map(to_map, obj.items()))]
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 977, in to_map
return list(zip(*map(to_map, obj)))
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 981, in to_map
return [type(obj)(i) for i in zip(*map(to_map, obj.items()))]
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 977, in to_map
return list(zip(*map(to_map, obj)))
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 981, in to_map
return [type(obj)(i) for i in zip(*map(to_map, obj.items()))]
File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 981, in
return [type(obj)(i) for i in zip(*map(to_map, obj.items()))]
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/yacs/config.py", line 86, in init
init_dict = self._create_config_tree_from_dict(init_dict, key_list)
File "/media/2TB_1/Miniconda/conda/envs/Detectron2/lib/python3.8/site-packages/yacs/config.py", line 123, in _create_config_tree_from_dict
for k, v in dic.items():
AttributeError: 'tuple' object has no attribute 'items'

Hmm, it looks like the error is from the cfg object creation. It is coming at this stage when you call eval method due to the lazy executionn of the built-in function ._create_config_tree_from_dict.
I don't have an immediate answer to fix it, because it is not clear to me why it happens.  But I could give certain things I would try to probe it further. First, make sure that the cfg object is created successfully and that it is reading the command line arguments properly in its place. As you already ran the model until a checkpoint is saved (3000 iterations in the code by default), you can use that weights and run the evaluation only for debugging. Check the command line arguments are passed without any mistakes (I suspect this part has the trouble). In eval mode, you can just run without any cmd arguments by putting it all in the YAML file.
Check in eval mode whether the model is receiving the correct config here:
File "/media/2TB_1/CZDet/DroneDetectron2/croptrain/engine/inference.py", line 67, in inference_with_crops
all_outputs = model(inputs, infer_on_crops=True, cfg=cfg)

Thank you for your answer. This problem may have occurred after I ran the pycharm software directly connected to the server. After I directly output the running command on the server, it did not report an error.


V4 .has-error not in repo

There is .has-danger, but .has-error does not exist in the code repo.
It shows fine on demo, but missing in code!
http://v4-alpha.getbootstrap.com/components/forms/#validation

@iBasit Please review #17153 which merged this change after the release of alpha 1.


I'm sure I'm missing something but...

... where exactly do you put your DeepL API key?
Also, I cannot see any reference to the endpoint you're using.
It seems to me that you're just basically emulating a request coming from the iOS app, directly to DeepL's own servers, therefore bypassing all of their public API. Is that so?
Because otherwise there should be some obvious place to put one's API credentials. Unless these are supposed to be some environment keys (but the code doesn't seem to need/want any).

As you said, this project simulates the requests of the DeepL iOS client, so you don't need any API Key.

Thank you! I thought as much. Nice overall work, BTW :)


Begin linting with CookStyle

/cc @chef-cookbooks/engineering-services

👍

👍 if Travis is happy.

👍  pending travis being happy

I am cherry-picking these changes into #37 and I was able to reproduce the travis failure locally. I believe have identified and fixed the issue which was causing travis to fail in this PR, so let's hope travis agrees with my local test results. :smile:


Towards v0.2.0

ADDS:

individual SE criterion per measurement ("mixed" criterion)

possibility to upper bound rho, helper util

non gaussian measurement creation

DISABLES:

SDP tests because unstable

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e801453). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   74.28%
=========================================
  Files             ?       15
  Lines             ?     1050
  Branches          ?        0
=========================================
  Hits              ?      780
  Misses            ?      270
  Partials          ?        0

Continue to review full report at Codecov.

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


Vector Similarity Search

I've been experimenting with the Vector similarity feature and have some uses for it (with both text and images) so plan on adding support when I have chance.
Early thoughts are to have a new field type of vector that would accept one of two similarity objects containing the parameters for each type (FLAT vs NHSW)
The actual JavaScript object datatype would be an Uint8Array or Float32Array and the lib would handle encoding for Redis.
It's likely that some additional search options would also be required.

I think this would be popular internally. As VSS is fairly new, I could see changes and improvements to it happening down the road that might change how we'd want to abstract it from Redis OM. So, we'll want to make sure we document it with appropriate caveats.

CC @leibale We talked about it...

So, as I understand there is no VSS support in Redis OM yet?


Use radix sort to sort postings when index sorting is enabled.

This switches to LSBRadixSorter instead of TimSorter to sort postings whose index options are DOCS. On a synthetic benchmark this yielded barely any difference in the case when the index order is the same as the sort order, or reverse, but almost a 3x speedup for writing postings in the case when the index order is mostly random.

Here is the synthetic benchmark that I used if someone is interested in reproducing:
  enum Order {
    RANDOM,
    ASC,
    DESC;
  }

  public static void main(String[] args) throws IOException {
    Order order = Order.RANDOM;
    Directory dir = FSDirectory.open(Paths.get("/tmp/a"));
    IndexWriterConfig cfg = new IndexWriterConfig(null);
    cfg.setInfoStream(new PrintStreamInfoStream(System.out));
    cfg.setMaxBufferedDocs(100_000);
    cfg.setRAMBufferSizeMB(IndexWriterConfig.DISABLE_AUTO_FLUSH);
    cfg.setIndexSort(new Sort(LongField.newSortField("sort_field", false, SortedNumericSelector.Type.MIN)));
    IndexWriter w = new IndexWriter(dir, cfg);
    Document doc = new Document();
    LongField sortField = new LongField("sort_field", 0);
    doc.add(sortField);
    StringField stringField1 = new StringField("string_field", "", Store.NO);
    doc.add(stringField1);
    StringField stringField2 = new StringField("string_field", "", Store.NO);
    doc.add(stringField2);
    StringField stringField3 = new StringField("string_field", "", Store.NO);
    doc.add(stringField3);
    for (int i = 0; i < 5_000_000; ++i) {
      long sortValue = switch (order) {
      case RANDOM -> i % 15;
      case ASC -> i;
      case DESC -> -i;
      };
      sortField.setLongValue(sortValue);
      stringField1.setStringValue(Integer.toBinaryString(i % 10));
      stringField2.setStringValue(Integer.toBinaryString(i % 100));
      stringField3.setStringValue(Integer.toBinaryString(i % 1000));
      w.addDocument(doc);
    }
  }

And flush times for postings:

Main
Patch

Index sort matches indexing order
6
7

Index sort is reverse indexing order
7
8

Random sort
27
10

I purposedly introduced a bug to see what would fail, and only high-level tests that check early query termination or dynamic pruning failed, so I introduced lower-level tests that make sure that postings get reordered correctly.


timer.h issue

in timer.h this_thread is not declared im not saying compiler said it when i try to run main.cpp
Thread not declared
thread is not a member of std
thread in namespace std does not name a type

do you have c++11 standards enabled in code blocks?
To check go to Setting -> Compiler
and you should see a box that says
[ ]Have g++ follow the C++11 ISO C++  language standard [-std=c++11]
if it unmarked than you have to toggle it on. Than try compiling main.cpp again and see if it works

yes but in timer.h error again in std:: thread Thread this line i don't know why?
recompile your timer.h if it works in my system doesn't work. Error detected.
.

is it the same error?

no it's not code is wrong or either something missing.

I only get errors when i don't have the c++ 11 standards box marked. Other wise I am able to compile with no problem

can you send me the errors

See errors i get in timer.h

Please i am new in this programming and here in this site..i saw over 5 codes for the email keylogger and i downloaded power.docx and sendmail.h and i read d script..is that all i need or do i still need to add the source code in my SendMail.h code..?

Sunnepazy123. Unless you want to change how the sendmail function work the only thing you need to do is add what it says in the readme

So all i need to use in the source is the sendmail_h source code by adding
the powershell.docx code if the sendmail_h code have issue..
On May 28, 2018 11:37 PM, "Siegfried5"<EMAIL_ADDRESS>wrote:

Sunnepazy123. Unless you want to change how the sendmail function work the
only thing you need to do is add what it says in the readme
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392612587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVbQqCDspBtLmgDZozFMj5ZPyPWZrks5t3HxGgaJpZM4TPW-L
.

I really appreciate your quick response because i wasn't expecting it this
soon.thank you Siegfried5
On May 29, 2018 2:08 AM, wrote:

So all i need to use in the source is the sendmail_h source code by adding
the powershell.docx code if the sendmail_h code have issue..
On May 28, 2018 11:37 PM, "Siegfried5"<EMAIL_ADDRESS>wrote:

Sunnepazy123. Unless you want to change how the sendmail function work
the only thing you need to do is add what it says in the readme
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392612587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVbQqCDspBtLmgDZozFMj5ZPyPWZrks5t3HxGgaJpZM4TPW-L
.

Yes. Read the readme. It's will tell you what to add to make the key logger work

Okay..Thank you so much..i will keep u posted..
On May 29, 2018 3:07 AM, "Siegfried5"<EMAIL_ADDRESS>wrote:

Yes. Read the readme. It's will tell you what to add to make the key
logger work
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392632861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVTdt1B2koH7X_NonTfJUjKvBB7XPks5t3K1rgaJpZM4TPW-L
.

I don't know what u mean here..and how to fix it..pls kindly tell me
added a new flag with the following Setting -> Compiler
name: windowsAPI
Compiler flags: -mwindows
linker flags: -mwindows
On May 29, 2018 3:09 AM, "Sunny Jay"<EMAIL_ADDRESS>wrote:

Okay..Thank you so much..i will keep u posted..
On May 29, 2018 3:07 AM, "Siegfried5"<EMAIL_ADDRESS>wrote:

Yes. Read the readme. It's will tell you what to add to make the key
logger work
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392632861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVTdt1B2koH7X_NonTfJUjKvBB7XPks5t3K1rgaJpZM4TPW-L
.

are you using codeblocks?

Yes i am
On May 29, 2018 4:53 AM, "Siegfried5"<EMAIL_ADDRESS>wrote:
are you using codeblocks?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392645586,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVXRIZ6chlTdHqBVtGeLvGV3Ec5rNks5t3MZFgaJpZM4TPW-L
.

go to codeblocks setting, than in the setting go to compiler. Their is were you need to create and the compiler flag. Also make sure you turn on the C++ 11 standards( its located in the same place)

I have turn on the c++ 11 standard..i will send you a snapshot of what i
did..
On May 29, 2018 7:07 AM, "Siegfried5"<EMAIL_ADDRESS>wrote:
go to codeblocks setting, than in the setting go to compiler. Their is were
you need to create and the compiler flag. Also make sure you turn on the
C++ 11 standards( its located in the same place)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392662527,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVQUjvvsglayly9bj9RskusWoJ2IJks5t3OWWgaJpZM4TPW-L
.

Did you create a new C++ Console App. Also did you just drag and drop the files into code block. Can you send Me a screen shot of codeblocks. It will be really helpful.

I downloaded the SENDme_h file and the powerscript and i did exactly what d
Readme said..it show all green in my codeblock but refuse to build it
saying -mwindows no such file or directory in the code block build messages
log...i didnt download the rest of those other source code..i hope that is
not the problem..
On May 29, 2018 5:58 PM, "Sunny Jay"<EMAIL_ADDRESS>wrote:

I have done as you said in the compiler settings..kindly check my
screenshot
On May 28, 2018 11:37 PM, "Siegfried5"<EMAIL_ADDRESS>wrote:

Sunnepazy123. Unless you want to change how the sendmail function work
the only thing you need to do is add what it says in the readme
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Siegfried5/Keylogger/issues/4#issuecomment-392612587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al3oVbQqCDspBtLmgDZozFMj5ZPyPWZrks5t3HxGgaJpZM4TPW-L
.

Again I do not see a screen shot on here.

and AV detect this ?

do you have c++11 standards enabled in code blocks?
To check go to Setting -> Compiler
and you should see a box that says
[ ]Have g++ follow the C++11 ISO C++ language standard [-std=c++11]
if it unmarked than you have to toggle it on. Than try compiling main.cpp again and see if it works

And also make sure CodeBlocks is version 16.01. I completely uninstalled CodeBlocks 17.12 and Installed
CodeBlocks 16.01 as in the Udemy course. and it Worked with those flags. Newer version of CodeBlocks not compiling even with compiler flags.


Refactor Parquet Array/Map/Struct ColumReaders

Description
Currently these XXXColumnReaders ( MapColumnReader, ListColumnReader, and StructColumnReader) call global functions that uses the pointer to the XXXColumnReader itself as a parameter, and does type check switches in side. Given the fact that they are already typed, we could remove the type check and remove the global functions. Here's what I think is better:

Add prepareRead() override to these XXXColumnReader class, which would replace the global function ensureRepDefs()
In each XXXColumnReader::prepareRead(), go recursively to the leaf level's ParquetData object and PageReader, then call PageReader::decodeRepDefs() to read the definitionLevels_ and repetitionLevels_, then return the pointer to these arrays in the XXXColumnReader::prepareRead()
To avoid https://github.com/facebookincubator/velox/issues/3614 we can remember the scanned pages PageReader::decodeRepDefs() reads. Note that we don't need to read all pages in the RowGroup like the current PageReader::decodeRepDefs() does, but just read enough pages for this batch.
Then in XXXColumnReader, we can override readNulls() to parse the rep/def levels to read nulls, lengths and offsets. The structural decoder will be a class member of the XXXColumnReader and the API to decode the rep/defs will be called directly there (DefLevelsToBitmap(), DefRepLevelsToList(), or our native implementation NestedStructureDecoder::readOffsetsAndNulls)

In this way the code would be better aligned with the common reader interfaces, has better encapsulation, and easier to read.

CC: @Yuhta @oerling

@yingsu00 What is the latest update on the progress now?
Does the issue mentioned in #3614 only occur on the V1 pages? Is it that only the first page is being decompressed repeatedly?


Load file needs to be updated with correct initial version

"release-number" of initial version should be 1.0.0 instead of 2.0.1. config file needs to be corrected.

Closing this issue as the values are already there as expected.


DevHome Hyper-V Create: Very confusing

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
The Creation flow is confusing. After clicking create environment, you are presented with a static page, and there are 2 disconnected buttons.

Done
Environments Page

i did not know what to do. I was scared about clicking done because I did not want it to cancel.
Expected result
I should see the creation of the image and status like we show on the enviroments page.
Actual result
I am lost and do not know anything is happening or what to do next.
Included System Information
No response
Included Extensions Information
No response

This is by design until we have notifications support in Dev Home.


Humanitarian filter for Budgets?

Related to #1299

Is your feature request related to a problem? Please describe.
When trying to analyse humanitarian spend data, it is useful to compare spend, commitments, and budgets. As far as I can tell, the only way to get specifically humanitarian budgets at the moment is to get all humanitarian activities in JSON, and include  {query}... fl= ... ,budget, ...
So something like:

https://iati.cloud/search/activity?q=(transaction_date_iso_date:[2019-01-01T00:00:00Z%20TO%20NOW])%20AND%20reporting_org_ref:GB-GOV-1%20AND%20hierarchy:2%20AND%20humanitarian:1&fl=iati_identifier,title_narrative_text,description_narrative_text,humanitarian,reporting_org_ref,activity_date_start_planned,activity_date_start_actual,activity_date_end_planned,activity_date_end_actual,activity_status_code,transaction,budget&wt=json&rows=5

This requires unpacking of a string JSON object, and is more technical than is practical for most users, I believe, as the returned JSON looks like this:
    // request metadata omitted
    "docs": [
      {
        "description_narrative_text": [
          "This activity (Food and Agriculture Organisation (FAO) funding for the Central African Republic Humanitarian and Recovery Programme (CHURP) 2016-19) is a component of CHURP Central African Republic Humanitarian Recovery Programme 2016 to 2019 reported by DFID, with a funding type of 'Humanitarian assistance' and a budget of £6,000,000. This component benefits Central African Republic, and works in the following sector(s): Material relief assistance and services . , with the following implementing partners: Food and Agriculture Organization of the United Nations (FAO). The start date is 01-04-2016 and the end date is 31-03-2019."
        ],
        "activity_status_code": "4",
        "title_narrative_text": [
          "Food and Agriculture Organisation (FAO) funding for the Central African Republic Humanitarian and Recovery Programme (CHURP) 2016-19"
        ],
        "budget": [
          "{\"type\":{\"code\":\"2\",\"name\":\"Revised\"},\"status\":{\"code\":\"1\",\"name\":\"Indicative\"},\"period_start\":\"2019-01-01\",\"period_end\":\"2019-03-31\",\"value\":{\"currency\":{\"code\":\"GBP\",\"name\":\"Pound Sterling\"},\"date\":\"2016-04-01\",\"value\":164066.0},\"xdr_value\":\"0.0000000\",\"usd_value\":\"0.0000000\",\"eur_value\":\"0.0000000\",\"gbp_value\":\"0.0000000\",\"jpy_value\":\"0.0000000\",\"cad_value\":\"0.0000000\",\"activity_id\":\"GB-1-205286-101\"}",
          "more budgets here..."
        ],
     // further elements omitted...
     }]

Describe the solution you'd like
Addition of a humanitarian='1' or humanitarian='2' filter on the /budget endpoint, which filters budgets based on their parent's humanitarian flag.
Describe alternatives you've considered
The alternative is the workaround above, or to get all budgets regardless of humanitarian and filter them post hoc - both of which are quite time consuming and require some technical ability.
Additional context
This could be very useful for attempts to use IATI data in the context of FTS data preparation.

Could I ask for some explanation here - is there a plan to implement this, or is it being closed because it won't be implemented?

This ticket was implemented and merged into develop a few weeks ago. It was closed once the testing was finished.