Flaky with pytest breaks Ctrl+C

Simply the presence of flaky plugin will prevent Ctrl+C from stopping the test suite. Flaky will register the KeyboardInterrupt as a test failure -- as opposed to pure pytest, which will stop the test suite.
Version information is in the test log below.
with flaky:
$ pytest
============================= test session starts ==============================
platform linux -- Python 3.6.8, pytest-5.0.1, py-1.7.0, pluggy-0.12.0
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/matejcik/a/.hypothesis/examples')
Test order randomisation NOT enabled. Enable with --random-order or --random-order-bucket=<bucket_type>
rootdir: /home/matejcik/a
plugins: hypothesis-4.4.3, random-order-1.0.4, cov-2.7.1, asyncio-0.10.0, flaky-3.6.0
collected 1 item

test_pytest.py ^CF                                                         [100%]

=================================== FAILURES ===================================
___________________________________ test_foo ___________________________________

    def test_foo():
>       time.sleep(10)
E       KeyboardInterrupt

test_pytest.py:4: KeyboardInterrupt
=========================== 1 failed in 0.80 seconds ===========================

without flaky:
$ pytest -p no:flaky
============================================================ test session starts ============================================================
platform linux -- Python 3.6.8, pytest-5.0.1, py-1.7.0, pluggy-0.12.0
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/matejcik/a/.hypothesis/examples')
Test order randomisation NOT enabled. Enable with --random-order or --random-order-bucket=<bucket_type>
rootdir: /home/matejcik/a
plugins: hypothesis-4.4.3, random-order-1.0.4, cov-2.7.1, asyncio-0.10.0
collected 1 item

test_pytest.py ^C

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! KeyboardInterrupt !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
/home/matejcik/a/test_pytest.py:4: KeyboardInterrupt
(to show a full traceback on KeyboardInterrupt use --fulltrace)
======================================================= no tests ran in 1.94 seconds ========================================================

Thanks @matejcik for the report. I think this probably started happening somewhat recently, due to a pytest refactor (for versions >= 4).
https://github.com/pytest-dev/pytest/pull/4299 refactored a pytest class (CallInfo). Before, the logic was that keyboard exception would be reraised by default. Afterward, the default was to reraise nothing, and the caller needs to specify exceptions to reraise, which flaky doesn't do.
I think a fix should be fairly simple.

Fixed in #157 and released as 3.6.1.


Obisidian->Appearance->Font Size doesn't change font size (Mac, iPad, Android)

If you're reporting an UI issue, make sure you take a screenshot that shows the actual bug.

Same with Windows, changing font size in settings or using ctrl+scroll doesn't work

Agreed. Removed all font sizing and weighting. Thank you!


Disabling notification for media

Any way to disable the notification for media that pops up when watching a video or music? The notification is useful for system stuff but it can get quite annoying for media so I'm wondering if there's any way to disable it.

just comment this line


Send header to ensure UAA knows we are uri encoding basic auth

If UAA is running in the new compatibility mode, we need to send this header to ensure the UAA properly decodes basic auth credentials.
https://www.pivotaltracker.com/story/show/169157390

@jpalermo what will happen if the user provides their own http.Client using the uaa.WithClient option? I'm wondering if there may be a need to wrap the transport provided by the user.

@jpalermo what will happen if the user provides their own http.Client using the uaa.WithClient option? I'm wondering if there may be a need to wrap the transport provided by the user.

Hmm, I tried wrapping the transport later, after the ops are all applied, but it didn't seem to work when I tried that.
It looks like in Token(), it's actually ignoring the configured client and always using a fresh defaultClient(). Is what it's doing correct there?
https://github.com/cloudfoundry-community/go-uaa/blob/8cbeb1b6a435cc0b887f9d00eea7f34586289e28/api.go#L105


Add num_following and num_followers to User returned by get_user_overview

Follow-up after server-side update (https://github.com/huggingface-internal/moon-landing/pull/11107, internal).
Route /api/users/{username}/overview now returns two news attributes numFollowing and numFollowers. It would be nice to support them officially client-side. It would avoid to have to iterate over all followers to count them (currently using list_user_following/list_user_followers).
In practice, it means updating User class + add a test.

I will try doing it
Can I get assigned this issue

Hi @Amrit02102004, thanks for proposing yourself! I just assigned you the task. Let me know if you need anything :hugs:

Hey I have made the changes here is it okay or should I add something else?
Thanks

Hey @Amrit02102004, this is good yes. In addition to that, you'll need to update User's __init__ method to populate the 2 new fields + add them to the docstring. Apart from that, I think it should be good. Could you open a PR to make the proper review easier? :)

I have successfully opened a PR
https://github.com/huggingface/huggingface_hub/pull/2536


Little error in example.

Hi!
In the example page:
https://pub.dev/packages/splashscreen#-example-tab-
code say:
**backgroundGradient**: new LinearGradient(colors: [Colors.cyan, Colors.blue], begin: Alignment.topLeft, end: Alignment.bottomRight),
but must say:
**gradientBackground**: new LinearGradient(colors: [Colors.cyan, Colors.blue], begin: Alignment.topLeft, end: Alignment.bottomRight),
Greetings!

Done, thanks for contributing


Initial typescript refactor

Setup typescript and typescript-eslint configurations and converted App.tsx to typescript

PR will not be approved until certain files are reverted, to make this PR simpler

@farisdurrani build checks failing. Do we need to modify for TS?

TODO @dwu359 restore old eslint rules

This PR is not approved until major fixes are made to installing typescript


[Bug] Cannot read properties of undefined (reading 'length')

Version
5.3.1
Link to Minimal Reproduction
https://codesandbox.io/s/echarts-bug-3tyi68?file=/src/Echarts.ts
Steps to Reproduce
Turn on the gradient color in the chart. When the mouse moves into the chart, it will report errors in an endless cycle. If the gradient color is removed, it will be normal. Remove 75 lines of code in the file and everything is normal.
Current Behavior
Chart dead cycle error reporting.
Expected Behavior
You can stop reporting errors and use them normally.
Environment
- OS: macOS Monterey
- Browser: Chrome 96.0.4664.55
- Framework: React@17

Any additional comments?
No response

@D-xuanmo 请问怎么解决的呢

@kechen123 好像是因为一个 rgba 颜色引起的，我这边把颜色值换了就正常了，你可以试试


Stubbing fetch fails after upgrade to v0.12.0

This has left me stumped, but somehow, after upgrading to v0.12.0, mocking fetch on globalThis seems to fail. See failing test run – https://github.com/getoslash/chrome-webstore-cli/runs/4287269860?check_suite_focus=true
You can see that on the previous versions, the tests pass fine.
One failing test – https://github.com/getoslash/chrome-webstore-cli/blob/26e037061f286f6848240368e86acd3036d4dd62/tests/helpers.api.test.ts#L171-L309

There was a change to the stub function to make it possible to stub non function properties.
https://github.com/udibo/mock/pull/22
In the example below from your test, stub is now making fetch an array instead of making it a function that returns values from the array.
const fetchStub: Stub<typeof globalThis> = stub(globalThis, "fetch", [
      Promise.resolve(
        new Response(
          new Blob([
            JSON.stringify(
              {
                "access_token": "xyz",
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
      ),
      Promise.resolve(
        new Response(
          new Blob([
            JSON.stringify(
              {
                "UPLOAD_STATE": "SUCCESS",
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
      ),
      Promise.resolve(
        new Response(
          new Blob([
            JSON.stringify(
              {
                "access_token": "xyz",
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
      ),
      Promise.resolve(
        new Response(
          new Blob([
            JSON.stringify(
              {
                "status": ["OK"],
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
      ),
    ]);

There are 2 new callbacks available for getting equivalent behavior. returnsNext and resolvesNext. If you use resolvesNext, you can also get rid of the Promise.resolve wrappers, it will return a promise for getting each value from the array.
const fetchStub: Stub<typeof globalThis> = stub(globalThis, "fetch", resolvesNext([
        new Response(
          new Blob([
            JSON.stringify(
              {
                "access_token": "xyz",
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
        new Response(
          new Blob([
            JSON.stringify(
              {
                "UPLOAD_STATE": "SUCCESS",
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
        new Response(
          new Blob([
            JSON.stringify(
              {
                "access_token": "xyz",
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
        new Response(
          new Blob([
            JSON.stringify(
              {
                "status": ["OK"],
              },
              null,
              2,
            ),
          ], { type: "application/json" }),
        ),
    ]));

@KyleJune Thanks for the very detailed response, this fixes it (see https://github.com/getoslash/chrome-webstore-cli/pull/5/commits/c8ed8b4bc34e7f217f2d903e0a6ba3e790b28c89)
Although, I'm curious, is there a specific reason for using generators here? Why not use an array of promises instead?

I have returnsNext and resolvesNext use generators internally so that any iterable can be passed to it. I suggested using resolvesNext just because it will resolve your values without you having to wrap each of them. You could still wrap each value with Promise.resolve() if you wanted to. If you did that, both returnsNext and resolvesNext would behave the same for the array of promises.
Because these functions accept any iterables, you could make a stub callback return values generated when your stub function is called.
From the following MDN article, if you passed the generator to the returnsNext function, it will return a callback that will return next value on each call. Instead of having to know how many times your stub will be called and declaring an array of all the values it will return, you could make a generator for your stub to use.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Generator#an_infinite_iterator

@paambaati I just looked at your code, you shouldn't have to pass in a generator. You should be able to skip creating your responses generator and instead just pass mockResponses array directly to resolvesNext like resolvesNext(mockResponses).
I've just realized I forgot to add Iterable as an option for resolvesNext. I'll look into fixing that soon so that you won't need to create a generator just to be able to make use of resolvesNext.

@KyleJune Thanks a lot, I figured it had to do with incomplete types. Take your time!

@paambaati I've fixed the issue with resolvesNext and improved the examples in the readme.
https://github.com/udibo/mock/releases/tag/0.12.1


Add missing Box component links

BoxTitle and BoxTools were not linked in Box, I have added them.
Now you can refer to Box.Tools and Box.Title, and all is well in the world.

Closed cause I found more things to fix

It looks like these are both included in BoxHeader such that referencing would be Box.Header.Title and Box.Header.Tools.

Ohhh right right. Thanks.


/dist/bot.js:127  -- SyntaxError: Unexpected token ...

I am a node.js novice trying to get this working on Ubuntu.
After much fooling around I finally got it to compile, mostly, but it is now dying at "npm start"
Ubuntu preparation to make it go:
sudo ln -s /usr/bin/nodejs /usr/bin/node
sudo apt-get install libicu-dev
sudo apt-get install autoconf
sudo apt-get install autotools-dev
sudo apt-get install libtool
It is cranky at the end but I think I can ignore warnings?
lib/bot.js -> dist/bot.js
lib/cli.js -> dist/cli.js
lib/errors.js -> dist/errors.js
lib/formatting.js -> dist/formatting.js
lib/helpers.js -> dist/helpers.js
lib/index.js -> dist/index.js
lib/validators.js -> dist/validators.js
<EMAIL_ADDRESS>/home/somebody/discord-irc-master
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous (git://github.com/hammerandchisel/erlpack.git#ad84c69355f03770a84d4bd8a20b981dd168fcf4)
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
└──<EMAIL_ADDRESS>extraneous
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>Okay first launch time:
npm start
><EMAIL_ADDRESS>start /home/somebody/discord-irc-master
> node dist/index.js

/home/somebody/discord-irc-master/dist/bot.js:127
        this.ircClient.send(...element);
                            ^^^

SyntaxError: Unexpected token ...
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:374:25)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/home/somebody/discord-irc-master/dist/helpers.js:12:12)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:417:10)

npm ERR! Linux 4.4.0-59-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: 'node dist/index.js'
npm ERR! Exit status 1
I have no idea how to proceed from here.

That looks like it hasn't transpiled properly. Did you clone the repo then do as in the README, or did you attempt to install the git repo directly using npm? I think there are issues with the latter, but I'm not sure what.

You need node.js version 6 or higher (https://github.com/reactiflux/discord-irc#installation-and-usage). Installation instructions: https://nodejs.org/en/download/package-manager/#debian-and-ubuntu-based-linux-distributions
Maybe we should add a check for it index.js or something, since https://github.com/reactiflux/discord-irc/blob/master/package.json#L14 only really work when you install it globally.


L515 is worse than D400 series for alignment performance

Required Info

Camera Model
L515

Firmware Version
1.4.1

Operating System & Version
Ubuntu 18.04

Kernel Version (Linux Only)

Platform
PC

SDK Version
2.35.2

Language
python

Segment
others

I've got a L515 today and tested with an alignment code.(align depth_stream to color_stream)
But I could know that alignment performance is worse than other realsense camera such as D415, D435.
For instance, when I get distance between the camera and a small piece object, I first check where the small object is located in color_array as pixel index.
With that pixel index, I get depth value in depth_array(aligned), but the value is not for the object but for other object near my object.
When I used D400 series camera, they aligned color and depth almost same.
Does IntelRealSense already know this problem, if yes do you have any plan to improve the align performance of L515 later?
thank you.

For further expanation...
let me compare D415 with L515
next two images are taken with D415(color and depth)

when I get a part of chair index, color and depth, both of them have similar x,y value
but in case of L515, color and depth have different x,y value(5~10 pixels..)

can I calibrate some parameters for alignment?

@dnjfk2003 - Thanks for the notification of this issue.  We will have a look.    Or are you using some other application?  What setting are you using for the L515, resolutions for both depth and RGB, etc?  Are you using a preset (Max Range, Low Ambient, No Ambient, Short Range)?

@RealSenseSupport I used python with my code which is modified a little bit from align example code.
I set color resolution is 1280720 and origin depth's is 1024768 and depth frames are aligned to color frames.
To see depth image like rsViewer, I used pyplot.imshow with parameter vmin, vmax.
L515 align two frames well in the center of frames but outside is bad.
Hope realsense team will solve this problem soon.
Thank you.

@dnjfk2003 Hi, I wonder if it gets better when you use the same resolution, like 1024 for both.
I have D435, but I always use the same resolution when depth align to RGB. Evan with the same resolution it still have several pixels misaligned due to the different FOV. It gets worse on the border .
Does the misaligned bothers you? Like,it  increases reprojected error when you work on slam or something else. If you have any idea pls let me know.

Hi @dnjfk2003
Thanks for the information on your setup and configuration.  Can you also provide images of the IR stream with pixel location information like you have for the depth and RGB images?

@dnjfk2003 Hi, I wonder if it gets better when you use the same resolution, like 1024 for both.
I have D435, but I always use the same resolution when depth align to RGB. Evan with the same resolution it still have several pixels misaligned due to the different FOV. It gets worse on the border .
Does the misaligned bothers you? Like,it increases reprojected error when you work on slam or something else. If you have any idea pls let me know.
Thank you for the advice. but there is no option of 1024 * 768 for color streaming. L515 has only two option for color, one is HD, the other is full HD. It's very important for my application to have well aligned frames. If I know how I can make it better, I will let you know.

@RealSenseSupport
could you please tell me how to get IR stream image(or array) from python code?
I can't use L515 camera currently, but I will check later(5~6 hours later from now).
can I get ir stream with code like
"aligned_ir_frame = aligned_frames.get_ir_frame()" ?
When I got color&depth frame, i used next code.
frames = pipeline.wait_for_frames()
aligned_frames = align.process(frames)
aligned_depth_frame = aligned_frames.get_depth_frame()
color_frame = aligned_frames.get_color_frame()

Hi @dnjfk2003
It would be easier debug if you were able to move to using the RealSense Viewer for debugging.  That way it takes Python out and the potential for not being able to view all streams.

@RealSenseSupport
I can get color, depth and ir frames using realsense-viewer how can I get those frames with aligning them?
Is there any option in rsViewer?

There are quite a bit of advanced settings and things to explore within the RealSense Viewer.  You can have a look at the 3D view once you start streaming of IR, RGB and Depth, it's in the right hand corner of the viewer screen

Then you can texture map over the depth the RGB, IR or Depth

@RealSenseSupport I'm not sure I explained here what I want to know.
What I'm doing is that is a distance information between the camera and an object. For that the first thing I do is, find the object location in RGB image. Then I get the distance from DEPTH array.
Of course, I should know the pixel address in the depth array and I get this address from RGB array.
For that, I'm using "pyrealsense2.align(pyrealsense2.stream.color)" function. But D400 series cameras give good align result but L515 doesn't.(You can see the result above images I uploaded before)
Anyway.. I took a capture from realsense viewer and I just used 2D for explaining my situation better.
here it is

Let's say I want to find the distance of the points (242, 604) in color array.
To get the distance,of course, I shouldn't use depth value of (242, 604) in depth array, but (149, 350).
So that's why I'm using alignment code and hope L515 alignment performance become better..
Hope I explained well for you to understand what I'm saying.

explained
Hi, you can use : rs-enumerate-devices -c to list all Intrinsic and Extrinsic.
The SDK align filter just use these to force align one to another image. You can write your own align filter if you like.
In my opinion , if you want use the default align filter.  You should choose what you need most, align to depth provide accurate depth estimation to infrared camera, but the color always few pixels misaligned. And vice versa .

Hi @dnjfk2003
Thanks for the explanation.  Now I do understand what you're looking for.  My apologies.  @cdb0y511 has a good outline in the comment above on using the align capability of the SDK.  Couple of links to the align examples:  https://github.com/IntelRealSense/librealsense/tree/v2.36.0/examples/align and https://github.com/IntelRealSense/librealsense/tree/v2.36.0/examples/align-advanced
The alignment isn't going to be exact, as stated it will be a few pixels difference.

@cbd0y511 thank you for letting me know the command. I didn't know that I can check camera parameters from that. I have another question how I can use the matrix? I can see rotation matrix with size 3X3.
Is it something like
DEPTH_PIXEL = MATRIX_COLOR_TO_DEPTH * COLOR_PIXEL?
could you please let me know usage of those matrix?

@RealSenseSupport thank you. I've checked those examples document.
As you said, now I know there can be a few pixels difference when using alignment but L515 is worse than D415(or 435).
Can L515 be improved later? Thanks.

Hi, @dnjfk2003 . I am not an expert on this specified  image processing domain. So maybe my answer  will not be corrected.
In my opinion, the align is like Image Panorama Stitching (normally no one align two identical picture together ).
OpenCV already has a lot of  Image Panorama Stitching algorithms.
like, https://www.pyimagesearch.com/2018/12/17/image-stitching-with-opencv-and-python/.
https://docs.opencv.org/master/d9/dab/tutorial_homography.html
If you  have homography matrix , you can applied the alignment easily. Most  algorithms focus on calculate the accurate homography matrix. Since you can get the extrinsic and intrinsic from sensor, this will not be a problem.
But I still have no ideal how to deal with the different FOV. Maybe Intel have some tricks dont let us know.

hi @cdb0y511. I am learning image processing domain as well. Thank you for the good information of homography matrix. I've read some material of that and it is very difficult to understand well with just one reading. But I feel I could get some inspiration from that. If I know new things. I'll share to you.
You said, you'd like to know how to handle the different FOV... but I'm not sure we have to consider that?
(Because I just think that we can make some equations for alignment with matrix like
POINT_depth = MATRIX * POINT_color if we know their resolution... I just suppose)

Hi @dnjfk2003
Is there anything else needed on this one?
*If we don’t hear from you in 7 days, this issue will be closed.

hi @dnjfk2003 , I have met the same align problem between color and depth map. And I write a issue about it. #7541
Do you any idea about it? May I discuss to you about this question?
Thanks!

If you have homography matrix , you can apply the alignment easily.

For a general 3D scene you cannot apply homography if you want to be any accurate. Homography is about planar relationships within a common 3D perspective projection relationship. Here the scene is neither planar nor the punctual origin of the projection. So you must go 3D either using the rs.align() function (if it's accurate and the camera/lidar calibrations are as well) or doing the 3D reprojections yourself.


Golang Repo Scaffolding: implement other interfaces (uninstall/reinstall)

Description
uninstall: delete repo
reinstall: delete and create repo
ishealthy: does repo exist?

close #190


Live test VC2

Issues during live test:

[ ] ToDo: How to install vc2 in order to use the functions?
[ ] Which modules does the user need to import? Describe the individual modules maybe? User needs to know, which functions can be directly imported by virocon and which functions need to be imported by virocon.module.
[ ] Point to utils in user guide: mention that here are useful additional functions to virocon.
[ ] How to read the data? Provide one test script which uses all basic functions of virocon.
[ ] How to create dist_descriptions? Key, value pair, how to create this?
[ ] Width of Interval slicer: center default value should be half of the interval width
[x] Exponential function is hard to fit without bounds. Here we should create a hint, that the user need to include bounds in the model. Solution custom exception.
[x] Enable to print and return parameters and dependence function of a model. Enable to show parameters after fitting the model.
[ ] Docstrings of contours.py are not displayed during the live test.
[ ] Integrate Plotting.py in contours.py
[x] Isodensity plots: default isodensity in the power of 10 steps.
[x] Plot 2D_contour: if the design conditions are calculated, they should also be returned.
[ ] Cheat sheet for dictionaries in user guide of virocon.

The test is over and most things seem are done such that I would like to close the issue.
@lenastroeer  What is meant with open item "Integrate Plotting.py in contours.py" (I don't think we would to merge the functionality in plotting.py or contours.py or why would we do that?)
@kawindme You wanted to create the "Cheat sheet for dictionaries in user guide", I'll open add this in a new issue with "use rguide" as the title


Why is there a warning about encrypt-and-mac?

We tested some of our OpenSSH servers with ssh-audit and noticed a couple of "using encrypt-and-MAC mode" warnings. I am a cryptography expert and I do not understand why there is a warning? I do not know of any security weaknesses in encrypt-and-mac. E.g. "hmac-sha2-256" is a perfectly safe algorithm. Why is there a warning? One should never use mac-then-encrypt, but this is not the case here? Can you explain about that?

Give https://moxie.org/blog/the-cryptographic-doom-principle/ a read

Hello, Plloi and thanks for your answer. This is exactly what I meant: Moxie (and other cryptographers) say that  MAC-then-Encrypt is a very bad idea. However, hmac-sha2-256 is "Encrypt-then-MAC" which Moxie describes in his blog post as "The third way, 'encrypt-then-authenticate,' is optimal because it does not violate the doom principle." So, there should be no warning about an "optimal" way to handle HMACs.

I'm not an expert so I may be wrong, but i believe the ssh implementation of hmac-sha2-256 is encrypt-and-MAC, there is a separate option hmac-sha2-256-etm that is encrypt-then-MAC

Hi, Shaun and thank you for showing me where to look. I knew about "MAC-then-encrypt" and "Encrypt-then-MAC" but I did not know what "encrypt-and_MAC" meant. I could not find anything about that in the OpenSSH documentation. As I read here this means to calculate the MAC of the clear text to the encrypted text. This is wrong, of course. The only save way to go is "encrypt-then-MACW" which appends the MAC of the encrypted text. So, thanks for pointing this out and, yes, you are right, one should only use the options ending in "-etm" and now I understand the warning. You made my day :-).

No problem, glad i could help


Updated details on myblog

Updated details on myblog

Hi @m8
Can you please approve this commit? Let me know if any further inputs are required.


[Experiment] Hide filter header on scroll

iOS
Android

Pre-flight check-list
Before raising a pull request

[ ] Documentation
[ ] Unit tests

Pre-merge check-list

[ ] Tester approved

I can't seem to get the refresh control working on iOS here. It is hidden behind the header and just won't move down. The suggestion I found on the internet is to use contentInset to move the whole list down and contentOffset to move the list items back up, but it keeps messing with the initial scroll of the list.

To be fair we might not actually have a refresh control in the final app, because it might suggest to users that they need to refresh a lot. Another question for UX/D. I imagine we might change the pull to refresh action, into just refreshing every time the app opens or something like that.

Ah, that's good to know. I think I will just park this here until we know from UX if and how scrolling should hide the header. I'm not even sure we want to have this at all, although I strongly vote for it. 🙂

If you come back to this any time soon, Jun mentioned that he would be interested in having the header shrink the other way on scroll, and show only the "I'm interested in" category filter banner while the user scrolls down.

Cool. I'll have a look next weekend.

Having had a conversation with @cellyn-tan and @daralthus we think this feature would make a big difference to landscape orientation.
That said, the MVP for this will likely be a sticky header for portrait and non-stick for landscape. It also ties in to a bigger question of whether landscape is adding too much work for the user value. Should we be prioritising landscape against other stories? @andyc84

Expected behaviour for Event Listing:
'Show me' and Top Date would always appear sticky
Only on scrolling up the secondary filters would appear


Adding retrieval of data from ICOS carbon portal

Codecov Report

Merging #313 (e3aff1e) into devel (9d152b9) will increase coverage by 0.36%.
The diff coverage is 87.56%.

@@            Coverage Diff             @@
##            devel     #313      +/-   ##
==========================================
+ Coverage   79.57%   79.93%   +0.36%
==========================================
  Files         136      141       +5
  Lines        9463     9819     +356
==========================================
+ Hits         7530     7849     +319
- Misses       1933     1970      +37

Impacted Files
Coverage Δ

tests/helpers/__init__.py
100.00% <ø> (ø)

openghg/util/_download.py
18.18% <18.18%> (ø)

openghg/dataobjects/_obsdata.py
30.43% <25.00%> (-0.52%)
:arrow_down:

openghg/client/_retrieve.py
55.55% <55.55%> (ø)

openghg/store/_obssurface.py
94.23% <82.60%> (-2.01%)
:arrow_down:

openghg/retrieve/icos/_retrieve.py
88.32% <88.32%> (ø)

openghg/standardise/meta/_attributes.py
88.27% <90.90%> (-0.13%)
:arrow_down:

openghg/client/__init__.py
100.00% <100.00%> (ø)

openghg/retrieve/icos/__init__.py
100.00% <100.00%> (ø)

openghg/store/base/_base.py
92.39% <100.00%> (+0.13%)
:arrow_up:

... and 10 more

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 9d152b9...e3aff1e. Read the comment docs.

A few small comments (and a few bugs spotted) above but overall looks good!

A few small comments (and a few bugs spotted) above but overall looks good!

So I ran retrieve with a randomly selected sample of 11 sites and got correctly standardised data back for each. I did get quite a few "we haven't seen site SITE before` messages so we'll have to work on the site metadata setup.


cmd/go: remove temporary build directories when terminated by signal

Originally reported in golang/vscode-go#152.
What version of Go are you using (go version)?

$ go version
go version go1.14.4 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/jayconrod/Library/Caches/go-build"
GOENV="/Users/jayconrod/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/jayconrod/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/go/installed"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/go/installed/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/rq/x0692kqj6ml8cvrhcqh5bswc008xj1/T/go-build036956176=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Interrupt go test or any other build command using ^C or with kill -SIGINT.
It's easy to reproduce this by setting GOTMPDIR to a local directory and then running a test that sleeps for a few seconds.
What did you expect to see?
No temporary go-build directories are left behind.
What did you see instead?
Directories with names like go-build312263156 are left behind.
go test should trap the signal, delete the work directory, and exit gracefully. It should not trim the build cache or do anything else that might take a long time (which it normally does when exiting).
Fixing this would be non-trivial: we'll need to plumb contexts through to network, build, and test operations to handle the cancellation. We probably want to do that anyway though for tracing.
cc @hakim @bcmills @matloob

we'll need to plumb contexts through to network, build, and test operations to handle the cancellation

Will contexts be enough? They do provide cancellation, but there won't be a way for the main goroutine to wait for the cleanup process (deleting temporary directories) to finish.

Cleaning up build directories happens in base.Exit, which is run from the main goroutine. I think that would be mostly unchanged. We'd just get there faster by cancelling anything that takes a long time.

Duplicate of #25808


fix: fixing bug where responses don't get rewritten with pretty-printing

express-graphql uses raw response.setHeader() and response.end() to send responses when pretty-printing is enabled. express-graphql-query-rewriter currently only handles rewriting responses sent via response.json(), which gets bypassed for pretty-printing. This PR allows the query rewriter to properly rewrite responses sent via response.end() and response.setHeader() as well.

:tada: This PR is included in version 1.0.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Call a one off method when a module is enabled or disabled

As I mentioned in this issue/request #182, I really needed this functionality within the application, so thank you to @sroutier I have implemented this functionality using the changes he made within his own fork here. I hope you don't mind :).
The following changes have been made:

When creating a module a new Utils/{Module}Maintenance.php class is created. This has two static methods, enable and disable.
When a module is enabled, the static enable method on the maintenance class for that module is called.
When a module is disabled, the static disable method on the maintenance class for that module is called.

Going to hold off on this approach and instead implement events for things like this.


Why height and width are shortened?

I usually do not use shortened props in order to improve readability, and upon looking here and here I wonder why height and width only exists in shortened version.

Because on image they are real attributes. I wanted to avoid collision.

I thought so. Is there a way to extend the API myself and include those?
Also, marginX, marginY, paddingX, and paddingY are not included, is there a reason for that?


macOS High Sierra crash

Hi,
There are some of us who beta test the new macOS and don’t like to go back and forth every time.  Framer currently crashes unfortunately on it. I submitted the reports already.

Thanks for the reports, we’re tracking this internally. This repository is only for the Framer Library, so I’m closing this issue.


Cannot train keras model with 2 outputs

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_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): Windows 10 Family
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0
Python version: 3.6.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10 / 7.6.4
GPU model and memory: GeForce 940M

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
import tensorflow as tf from tensorflow import keras from tensorflow.keras.layers import Dense, Input from tensorflow.keras import Model
A tf.keras model is created with 2 output heads:
`inputs=Input(shape=(20))
x=Dense(32, activation='relu')(inputs)
output1=Dense(3, activation='softmax')(x)
output2=Dense(6, activation='softmax')(x)
model=Model(inputs=inputs, outputs=[output1, output2])
model.compile(optimizer='adam', loss='sparse_categorical_crossentropy',
metrics=['accuracy'])`
Then an arbitary sized training sample is created:
`labels=tf.constant([[0 for i in range(50)],[1 for i in range(50)]], shape=[50,2])
sample=[]
for i in range(20):
if i%2==0:
sample.append(0)
else: sample.append(1)
data=tf.constant([sample for i in range(50)], shape=[50,20])`
When the data is called for training,
model.fit(data, labels, epochs=1, batch_size=5)
an error occurs saying that the module expected 2 arrays, but when I swap out the validation dataset with a pair of 2 arrays:
model.fit(data, (labels1, labels2), epochs=1, batch_size=5)
another error pops up saying that the x and y dimensions must be the same. To put simply, keras thinks that the pair of validation outputs is an array.
Describe the expected behavior
The model.fit method runs and the model is trained for 1 epoch with 50 samples.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
copy the above code in order, with the last model.fit method not usable.
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.
Traceback (most recent call last):   File "C:\Users\admin\Desktop\working as intended\CONSTANTINOPLE\test\test duo output.py", line 25, in <module>     model.fit(data, labels, epochs=1, batch_size=8)   File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_core\python\keras\engine\training.py", line 728, in fit     use_multiprocessing=use_multiprocessing)   File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_core\python\keras\engine\training_v2.py", line 224, in fit     distribution_strategy=strategy)   File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_core\python\keras\engine\training_v2.py", line 547, in _process_training_inputs     use_multiprocessing=use_multiprocessing)   File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_core\python\keras\engine\training_v2.py", line 594, in _process_inputs     steps=steps)   File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_core\python\keras\engine\training.py", line 2519, in _standardize_user_data     exception_prefix='target')   File "C:\Users\admin\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_core\python\keras\engine\training_utils.py", line 531, in standardize_input_data     str(len(data)) + ' arrays: ' + str(data)[:200] + '...') ValueError: Error when checking model target: the list of Numpy arrays that you are passing to your model is not the size the model expected. Expected to see 2 array(s), but instead got the following list of 1 arrays: [<tf.Tensor: id=286, shape=(50, 2), dtype=int32, numpy= array([[0, 0],        [0, 0],        [0, 0],        [0, 0],        [0, 0],        [0, 0],        [0, 0],        [0, 0],        [0, 0],        [0...

in model.fit() you don't need to include the batch_size as per my opinion use the callbacks if defined, let me show you an example by attaching a .

ipynb file of code

@darsh8200 I don't think you are getting what I was asking. I wanted to ask about how to train a network with 2 output heads. And also the batch size variable is arbitary and can be removed in this case.

Hi,
To make the model training work, you need to pass labels as a list of two tensors (each associated with one of the outputs), i.e. model.fit(x, [y1, y2]) with x of shape (n, 20), y1 of shape (n,) and y2 of shape (n,) (note that you could also add a dimension for batch-size, but that is another topic).
If I go back to your initial definition of data and labels as two Tensors of respective shapes (50, 20) and (50, 2), you can run : model.fit(data, [labels[:, 0], labels[:, 1]], epochs=1, batch_size=5).
I hope this helps :)

@pandrey-fr I used your code and with some tweaking, the code ran flawlessly. I didn't know you can write grammar like labels[:, 0] before, seems like I have more stuff to learn :D Thanks for the help, apperciate it

@ongoing0217 You are welcome ! The use of slice notation is only valid in Eager execution, so you should use it with parsimony, but it can be useful in debugging contexts :)


[Event Request] Table 37 - Sales Line - OnAmountOnValidateOnAfterSetAmount

Dear ALAppExtensions team,
through this issue we would like to request adding event publisher OnAmountOnValidateOnAfterSetAmount in Table 37 Sales Line with the following signature:
[IntegrationEvent(false, false)]
local procedure OnAmountOnValidateOnAfterSetAmount(var SalesLine: Record "Sales Line")
begin
end;

which would be called as follows in the OnValidate trigger of field 29 - Amount:
trigger OnValidate()
begin
    Amount := Round(Amount, Currency."Amount Rounding Precision");
    // >>>> New >>>>>>>>>>>>>>>>
    OnAmountOnValidateOnAfterSetAmount(Rec);
    // <<<< New <<<<<<<<<<<<<<<<
    case "VAT Calculation Type" of
        "VAT Calculation Type"::"Normal VAT",
        "VAT Calculation Type"::"Reverse Charge VAT":
...

Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47021.


Allow outline to have a separate view and activity bar icon

With VSCode 1.25, the outline feature is shown in the explorer view. That view is already used to show open editors and the full folder tree. Having the outline there as well makes it a bit cluttered and each piece of information gets very little screen area.
The Code Outline extension (which is now deprecated and doesn't seem to work anymore in 1.25) used a separate view, selected by an icon on the activity bar. Please make that possible also for the new, built-in outline feature.

/duplicate of https://github.com/Microsoft/vscode/issues/49923


Dropdown menu is not working with 12.1. Works fine with 11.0

Hello,
Dropdown menu is not working with 12.1.
Works fine with 11.0
http://plnkr.co/edit/GE2R735h6XZvOCdpLFOf?p=preview
Any workaround ?
Thank you !

See release notes of 0.12 and the breaking changes:
https://github.com/angular-ui/bootstrap/blob/master/CHANGELOG.md#breaking-changes
Your fixed plunkr: http://plnkr.co/edit/BFw2bsgZEmJwtAfH7Vi1?p=preview


2.1.1 delegate is not working with arguments

Running a delegate with arguments results in unknown error
const delegate = new Binding({schema: service.originalExecutableSchema});
let response = await delegate.delegate('query', 'articles', {}, info, {context: ctx ? ctx : {}}); // works
let response = await delegate.delegate('query', 'articles', {where: {id_in: ["foo"]}}, info, {context: ctx ? ctx : {}}); // not working
// Error: Response not successful: Received status code 400
//    at throwServerError ..../node_modules/apollo-link-http-common/src/index.ts:113:17)
//    at .../node_modules/apollo-link-http-common/src/index.ts:142:11

Withot arguments — works ok.
The same query with same arguments is running at remote service directly — works ok.
Trace is absolutely useless.
On the remote server there is a graphql-yoga server. I've tried to debug incoming request:
const server = await createEntityServer( // just a wrapper function
        appName,
        entitiesDir,
        {drivers: drivers, generatedPath: prismaGeneratedDir},
        {
          locales, context: {
            requestCache: () => {console.log('foo')} // this should trigger on any incoming request
          }
        }
    );
server.start({
      options: {
        formatParams: (...a) => console.log(a) // this also trigger on any incoming request
      },
      endpoint: "/graphql"
    }, () => console.log(`Server is running on localhost:${process.env.PORT}`));

When I run delegate without args — console properly outputs:
foo
foo
{}

(dunno why foo is twice but nevertheless).
When I call delegate with args — I get only two times foo. And that's all. Yoga-server even doesn't follow to formatParams. No errors in console, no outputs — nothing.
Any ideas what can cause this and what to do?

Can you share a reproduction that can be run locally in the form of a repository? 🙂

@marktani nevermind, as usual — after posting a question one shortly finds answer — problem is not here, but in apollo's transformer in underlying layer. Sorry for bothering


Copy changes to show.yml

Amend 'base path' to 'page address path'

Are we confident that users will understand this anymore than base path?
It ends up a bit confusing as base path has quite a nuanced meaning and we lose that with "Page address path" - I guess also page address path relies on user still understanding what a path is so I wonder if the first two words are redundant?
I wondered if some of the suggestions to just remove showing this field would produce an easier experience for users?

Superseded by https://github.com/alphagov/content-publisher/pull/471.

@kevindew agreed - best solution is to remove "Base path" from the summary screen.
I believe we will need to add a caption/hint to "Page address" in the edit screen - and a different one when users create a new edition (to alert them of what happens with the existing one).


Set up FUNDING info

#3  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


Where is schema file?

Hi @medvedevden1s
Thanks for reaching out.  The schema file can be found in the AppSync console.
When in the console, select your API then select 'Schema' in the sidebar.  You will then have the option of viewing the schema in a GraphiQL-like interface or exporting it to a file.
I'm closing this issue now - but please let us know if you're question is not answered.


Remove cleanup after bounce expire and slave reactivation

@tpetr initially did this so that it /fixes #972 . This will:

check that no other task cleanups are present (i.e. if task should still be in cleaning state)
if not, find the task history updates and if the last one is TASK_CLEANING delete it
re-fire an event for the previous state the task was in (most likely running)
make sure a pending request is put in the queue to take care of any other pending tasks or launched tasks that might need to be taken care of

In addition I also did the same thing for the case where a decommissioned slave is reactivated (handled by SingularityCleaner), since right now reactivating the slave still requires all of the cleaning tasks to be properly cleaned and repalced, even though their slave is no longer deocmmissioning.
Thoughts on re-firing an event for the previous state vs creating a new event type for history updates that are deleted?

@tpetr are you cool with merging this?

Found one thing here, seems that we can accidentally reuse instance number in this case. IF a task is cleaning we will currently reuse that instance number. If it then comes back out of cleaning, we will have a duplicate instance number. Not sure if that's an issue or not, will revisit this

updated to not worry about the decommission/reactivation for now, causes more headaches than it's worth at the moment. Bounce expire was the real issue to start with anyways.

@tpetr are you good with merging this one? causes a rather annoying problem for anyone without the change when canceling a bounce

LGTM


Initial 2.2.6

This is based on the 2.2.5 release, with the following changes:

Uses 2.2.6 source instead of 2.2.5
requires meson 0.48, uses a feature option for libbsd

This should probably be rebased on top of https://github.com/mesonbuild/expat/pull/6 once it's merged? :)

mesonbuild:2.2.6 has no releases, I don't think we care, it was a pristine branch.
As for @dcbaker just ask him to push his local changes to his local repo?

I just realized that you actually pushed something to 2.2.6. This is ignored without pull hook so I am going to revert it also.


v3.2.0rc0 whats new and version string

🚀 Pull Request
Description
Updates version string to 3.2.0rc0
Moved whats new dev.rst -> 3.2.rst

Consult Iris pull request check list

A couple of quick queries here ...

is it really necessary + desirable to delete docs/whatsnew/dev.template ?
I know the dev docs say to do that, but then we have to re-introduce it post-release.  I'm not convinced it does any harm to just leave it in ?
I get that #4540 basically renamed 'latest.rst' to  'dev.rst'.  We have also changed the ref in 'whatsnew/index.rst'.
And here when cutting a new release version, we rename 'dev.rst' to '3.2.rst' in the index.
but then why do we still need 'latest.rst' at all  -- which is now just a link to 'dev.rst' ?
(I know this is really a question for #4540, but I missed the boat there)

2. but then why do we still need 'latest.rst' at all -- which is now just a link to 'dev.rst' ?

Presumably because we want to just update the symlink rather than the toctree, but I think @bjlittle added it so might have another view?

is it really necessary + desirable to delete docs/whatsnew/dev.template ?
Not sure really! I am doing it because that's what the docs say and that is what you did for the 3.1.0rc0


docs: add a document about CRI API

Ⅰ. Describe what this PR did
For the better delivery of the CRI of PouchContainer, one ducument of apis of CRI Manager is needed. And for the future,  scripts were written to automate this work which can be integrate into the project pouchrobot.
Ⅱ. Does this pull request fix one issue?

None.
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Only scripts and documents are involved
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
By the way, I update the file api.pb.go by via script bash hack/protoc/protoc.sh gen_proto

@allencloud @fuweid @YaoZengzeng PTAL. Thanks. ^_^

LGTM


Parsing recurring events doesn't account for event that has a changed DTSTART and/or DTEND listed in the file.

I have a recurring event on an ICAL file I pull from Google Apps. We recently had to modify the time of one instance of the recurring events. The modified event came up in the list at the proper new time, but the parser still generated and returned the event at the original time as well. It looks like events would be parsed for the UID of the recurring event and find out if for that RECURRENCE-ID the event differs from the RRULE setting for DTSTART and/or DTEND, and if so replace that particular event in the array generated from the RRULEs.
The events are in question are listed, with some redactions. There should be enough to replicate the issue.
Recurring event rule:
BEGIN:VEVENT
DTSTART;TZID=America/Chicago:20160406T103000
DTEND;TZID=America/Chicago:20160406T110000
RRULE:FREQ=WEEKLY;BYDAY=WE
DTSTAMP:20160426T220705Z
<EMAIL_ADDRESS>CREATED:20160406T175048Z
LAST-MODIFIED:20160406T210132Z
LOCATION:LOCATION
SEQUENCE:1
STATUS:CONFIRMED
SUMMARY:MEETING
TRANSP:OPAQUE
END:VEVENT

And a modified instance of the recurring event:
BEGIN:VEVENT
DTSTART;TZID=America/Chicago:20160427T130000
DTEND;TZID=America/Chicago:20160427T133000
DTSTAMP:20160426T220705Z
<EMAIL_ADDRESS>RECURRENCE-ID;TZID=America/Chicago:20160427T103000
CREATED:20160406T175048Z
LAST-MODIFIED:20160425T184416Z
LOCATION:LOCATION
SEQUENCE:2
STATUS:CONFIRMED
SUMMARY:MEETING
TRANSP:OPAQUE
END:VEVENT

Feel free to PR; this library is not currently in active development. Bugs will be fixed but enhancements will be added to a prioritised queue.

Is this fixed by PR #59 which is now merged?


RegisterObjectTransformation not working with ${exception:format=ToString,Properties,Data}

Hi!
I implemented logic found here https://github.com/NLog/NLog/wiki/Exception-Layout-Renderer#custom-exception-serialization May be the layout in documentation is not correct.
NLog version: 5.3.8
Platform: .NET 8.0
Current NLog config
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      autoReload="true"
      throwConfigExceptions="true"
      internalLogLevel="Debug"
      internalLogFile="./Logs/internal-nlog.txt">

    <extensions>
        <add assembly="NLog.Extensions.Logging"/>
    </extensions>

    <targets>
        <!--${exception:format=@} works as expected-->
        <!--
        <target name="file" type="File" archiveNumbering="DateAndSequence" archiveAboveSize="10000000" fileName="./Logs/nlog-${shortdate}.txt" layout="${longdate} ${level} ${message}${exception:format=@}"/>
        <target xsi:type="Console" name="lifetimeConsole" layout="${level:truncate=5:lowercase=false} ${message} ${exception:format=@}" />
        <target xsi:type="Trace" name="trace" layout="${level:truncate=5:lowercase=false} ${message} ${exception:format=@}" />
        -->
        <!--${exception:format=ToString,Properties,Data} does not work as expected!-->
        <target name="file" type="File" archiveNumbering="DateAndSequence" archiveAboveSize="10000000" fileName="./Logs/nlog-${shortdate}.txt" layout="${longdate} ${level} ${message}${exception:format=ToString,Properties,Data}"/>
        <target xsi:type="Console" name="lifetimeConsole" layout="${level:truncate=5:lowercase=false} ${message} ${exception:format=ToString,Properties,Data}" />
        <target xsi:type="Trace" name="trace" layout="${level:truncate=5:lowercase=false} ${message} ${exception:format=ToString,Properties,Data}" />
    </targets>

    <rules>
        <logger name="*" minlevel="Debug" writeTo="file"/>
        <logger name="*" minlevel="Debug" writeTo="lifetimeConsole"/>
        <logger name="*" minlevel="Debug" writeTo="trace"/>
    </rules>

</nlog>

What is the current result?
The current result is:
"Error ExceptionMessage~!@#$%^&*()_+{}| System.DivideByZeroException: Attempted to divide by zero.
at Project.Test.NLogTest.RegisterObjectTransformationLayoutTest() in C:\s\Test\NLogTest.cs:line 30"
Layout ${exception:format=ToString,Properties,Data} does not work as expected! It prints standard exception.
What is the expected result?
Expected result is:
"Error ExceptionMessage~!@#$%^&*()_+{}|
System.DivideByZeroException: Attempted to divide by zero.
at Project.Test.NLogTest.RegisterObjectTransformationLayoutTest() in C:\s\Test\NLogTest.cs:line 30"
It should print new line character after error message followed by standard exception output. The goal is to visually separate error message and exception information.
Layout ${exception:format=ToString,Properties,Data} should work as expected!
Layout ${exception:format=@} works as expected.
Did you checked the Internal log?
Yes. I didn't see anything relevant.
Please post full exception details (message, stacktrace, inner exceptions)
There is no NLog exception. User code runtime exception was "Exception thrown: 'System.DivideByZeroException' in dllFile.dll"
Are there any workarounds? yes/no
I do not know.
Is there a version in which it did work?
I do not know. May be NLog 4.7 as stated in documentation.
Can you help us by writing an unit test?
Yes.

using Microsoft.VisualStudio.TestTools.UnitTesting;
using NLog;
using System;

namespace Project.Test
{
    [TestClass]
    public class NLogTest
    {
        private static Logger logger = LogManager.GetCurrentClassLogger();

        static NLogTest()
        {
            LogManager.Setup().SetupSerialization(o => o.RegisterObjectTransformation<Exception>(ex => Transform(ex)));
        }

        private static object Transform<T>(T ex) where T : Exception
        {
            //return new { Message = Environment.NewLine + ex!.ToString() };    //Even this does not work.
            return Environment.NewLine + ex.ToString();
            //return 1;
        }

        [TestMethod]
        public void RegisterObjectTransformationLayoutTest()
        {
            try
            {
                var i = 0;
                i /= i;
            }
            catch (Exception ex)
            {
                logger.Error(ex, "ExceptionMessage~!@#$%^&*()_+{}|");
            }
        }
    }
}

Notice that RegisterObjectTransformation is meant for removing/adding properties for special/dangerous objects, by converting into a different structured-object. But it also has the ability to take a structured-object and turn it into a flat scalar value like string.
Notice that "Properties" will only output additional custom properties from the Exception-class, thus these values will not be repeated (as they are already available as Format-options):
        private static readonly HashSet<string> ExcludeDefaultProperties = new HashSet<string>(new[] {
            "Type",
            nameof(Exception.Data),
            nameof(Exception.HelpLink),
            "HResult",   // Not available on NET35 + NET40
            nameof(Exception.InnerException),
            nameof(Exception.Message),
            nameof(Exception.Source),
            nameof(Exception.StackTrace),
            "TargetSite",// Not available on NETSTANDARD1_3 OR NETSTANDARD1_5
        }, StringComparer.Ordinal);

Closing issue due to inactivity.


Updating position of a marker in rememberMarkerState doesn't work

I feel link there is a missing behavior with the rememberMarkerState() function.
Here's a code sample :
Code example
@Composable
fun Maps() {
    val cameraPositionState = rememberCameraPositionState {}

    var projectionCenter by remember {
        mutableStateOf<LatLng?>(null)
    }

    LaunchedEffect(cameraPositionState.position) {
        val projection = cameraPositionState.projection
        if (projection != null) {
            val bounds = projection.visibleRegion.latLngBounds
            projectionCenter = bounds.center
        }
    }
    GoogleMap(
        cameraPositionState = cameraPositionState,
        modifier = Modifier.fillMaxSize()
    ) {
        projectionCenter?.let {
            Marker(
                state = rememberMarkerState(key = "center", position = it),
                icon = BitmapDescriptorFactory.defaultMarker(),
            )
        }
    }
}

This code should display a marker at the center of the projection, even if you move.
It won't work, because the marker position is remembered once :  rememberMarkerState(key = "center", position = it), then it won't be updated unless you call MarkerState.position = newPosition.
With the following change, the code works :
projectionCenter?.let { latLng ->
            Marker(
                state = rememberMarkerState(key = "center", position = latLng).also {
                    position = latLng
                },
                icon = BitmapDescriptorFactory.defaultMarker(),
            )
        }

It is the expected behavior?
I'm afraid some people like me search for some times why their code is not working.
I think rememberMarkerState(position = newPosition) should updated the underlying markerState if it's already remembered, but maybe I'm wrong?
One last info : with one marker the use case is trivial, I can just do that to fix the code :
@Composable
fun Maps() {
    val cameraPositionState = rememberCameraPositionState {}

    val projectionCenter = rememberMarkerState()

    LaunchedEffect(cameraPositionState.position) {
        val projection = cameraPositionState.projection
        if (projection != null) {
            val bounds = projection.visibleRegion.latLngBounds
            projectionCenter.position = bounds.center
        }
    }
    GoogleMap(
        uiSettings = MapUiSettings(mapToolbarEnabled = false, myLocationButtonEnabled = false, zoomControlsEnabled = false),
        cameraPositionState = cameraPositionState,
        modifier = Modifier.fillMaxSize()
    ) {
        Marker(
            state = projectionCenter,
            icon = BitmapDescriptorFactory.defaultMarker(),
        )
    }
}

But I think the issue arise with a dynamic list, where it becomes hard to track the change of a specific marker.
My first test was using a mutableStateListOf()

Based on your 1st snippet, the behavior you noticed is working as expected. Calling rememberMarkerState(key = "center", position = it) multiple times won't trigger recomposition since the key stays the same. Either use rememberMarkerState and store it as a variable (as you have it in your last snippet), or construct a MarkerState(position) directly.

That's what I thought first, But using a "dynamic key" (using the latlng in it) still won't trigger recomposition:
@Composable
fun Maps() {
    val cameraPositionState = rememberCameraPositionState {}

    var projectionCenter by remember {
        mutableStateOf<LatLng?>(null)
    }

    LaunchedEffect(cameraPositionState.position) {
        val projection = cameraPositionState.projection
        if (projection != null) {
            val bounds = projection.visibleRegion.latLngBounds
            projectionCenter = bounds.center
        }
    }
    val center = projectionCenter
    GoogleMap(
        uiSettings = MapUiSettings(mapToolbarEnabled = false, myLocationButtonEnabled = false, zoomControlsEnabled = false),
        cameraPositionState = cameraPositionState,
        modifier = Modifier.fillMaxSize()
    ) {
        if (center != null) {
            Marker(
                state = rememberMarkerState(
                    key = "${center.latitude} ${center.longitude}", position = center
                ),
                icon = BitmapDescriptorFactory.defaultMarker(),
            )
        }
    }
}

result :

But I didn't know about using directly MarkerState (I wrongly read that it was internal but of course it's not !), so that might be the best solution.
(with the above snippet, using state = MarkerState(center), produce the right behavior of course)
Anyway, I think my issue with understanding is more with how rememberSaveable works and less with Compose Maps libs. I remember reading some question that were similar to mine.
I think the key changing without the MarkerState updating might still be an issue, I'll check with the Kotlin Compose Slack and will close the issue if it's linked to rememberSaveable behavior.

Actually it's a bug in Compose since 2020 -> https://issuetracker.google.com/issues/152014032

Is the question still open? Help me. the last Polyline is not updated, but markers are drawn according to the route.

I was having this same exact issue, however, I noticed that the rememberMarkerState wasn't making use of the rememberSaveable inputs at all! It is only changing the key if the key gets changed. So this would never recompose!
I believe the fix wanted would be this:
@Composable
public fun rememberMarkerState(
    key: String? = null,
    position: LatLng = LatLng(0.0, 0.0),
): MarkerState = rememberSaveable(inputs = arrayOf(position), key = key, saver = MarkerState.Saver) {
    MarkerState(position)
}


EDD Form, insufficient color contrast

Red used for "required" tag fails (foreground #ff0000, background #fff)

not sure which red is being used, but it should not be #ff0000. from the TK, the reds used have been tried and true on the a11y front...
here are all the colores in question:
http://contrast-grid.eightshapes.com/?background-colors=&foreground-colors=%23FFFFFF
%23F6D6D8
%23D0343A
%2397272C
%23FF0000
%23000
&es-color-form__tile-size=compact

This may be an old ticket that has since been resolved? The required red on https://qa-www.nypl.org/research/collections/shared-collection-catalog/hold/request/b13518955-i16541459/edd is not #ff0000.


Allow hooks to run for all parallel blocks

Address https://github.com/danielstjules/mocha.parallel/issues/36

@danielstjules let me know if I added tests in the correct place or not

@danielstjules , following up on this.

@danielstjules any updates?

Thanks! Merged the fix :)


Is it really worth using Evaluation flow?

Is it really worth using Evaluation flow?  Please give me some potential benefits for using it in my flow.

Add @cloga to help answer this question.


How to redirect an url to another ?

What version of Traefik are you using (traefik version)?
v1.1.0
What is your environment & configuration (arguments, toml...)?
[entryPoints]
   [entryPoints.http]
   address = ":80"
     [entryPoints.http.redirect]
       regex = "^http://(.*).domain.com"
       replacement = "http://$1.staging.newdomain.com"

What did you do?
I want to redirect all url *.domain.com to *.staging.newdomain.com
What did you expect to see?
url redirected
What did you see instead?
all url are now in 404
Is it possible to do that please ?

The same without regexp on v1.2.0-rc2
defaultEntryPoints = ["https"]
[entryPoints]
   [entryPoints.http]
   address = ":80"
     [entryPoints.http.redirect]
     entryPoint = "https"
   [entryPoints.https]
   address = ":443"
     [entryPoints.https.tls]
     ....

I get only
404 page not found

Thanks for your interest in Traefik !
Every questions in the issues is a feature less.
The issue tracker is for reporting bugs and feature requests only.
For end-user related support questions, refer to one of the following:

Stack Overflow using the "traefik" tag
the Traefik community Slack channel


Build Optimizer breaks Ivy Apps using platformBrowserDynamic

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [x] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions
Angular CLI: 7.2.0-rc.0
Node: 11.0.0
OS: win32 x64
Angular: 7.2.0-rc.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.12.0-rc.0
@angular-devkit/build-angular     0.12.0-rc.0
@angular-devkit/build-optimizer   0.12.0-rc.0
@angular-devkit/build-webpack     0.12.0-rc.0
@angular-devkit/core              7.2.0-rc.0
@angular-devkit/schematics        7.2.0-rc.0
@ngtools/webpack                  7.2.0-rc.0
@schematics/angular               7.2.0-rc.0
@schematics/update                0.12.0-rc.0
rxjs                              6.3.3
typescript                        3.2.2
webpack                           4.23.1

Repro steps
When using ngtsc, together with the usual platformBrowserDynamic bootstrapping code causes a runtime error.
git  clone https://github.com/alan-agius4/ivy-build-optimizer-issue.git
npm i
node_modules\.bin\ivy-ngcc
ng serve --prod

The log given by the failure
Runtime error
main.ts:12 Error: No ErrorHandler. Is platform module (BrowserModule) included?
    at core.js.pre-build-optimizer.js:17333
    at n.invoke (zone.js.pre-build-optimizer.js:388)
    at Object.onInvoke (core.js.pre-build-optimizer.js:16856)
    at n.invoke (zone.js.pre-build-optimizer.js:387)
    at t.run (zone.js.pre-build-optimizer.js:138)
    at t.run (core.js.pre-build-optimizer.js:16770)
    at t.bootstrapModuleFactory (core.js.pre-build-optimizer.js:17328)
    at core.js.pre-build-optimizer.js:17368
    at n.invoke (zone.js.pre-build-optimizer.js:388)
    at t.run (zone.js.pre-build-optimizer.js:138)

Desired functionality
No Error.
Mention any other details that might be useful
When turning off BuildOptimizer there are no runtime errors.

Blocked by https://github.com/angular/angular/issues/27841


Bump to Go 1.11.8 and 1.12.3

Sigh, there's a bug in 1.12.2 and 1.11.7 hitting programs using cgo, I don't think that any Prometheus repository is using cgo anymore but in doubt, it is safer to keep up-to-date.
Hopefully after I submitted #63, I spent a couple of hours to write a quick & dirty program that updates the repository whenever patch versions of Go are released so this PR didn't took too long...

/cc @paulfantom was also interested in doing some auto-bump work.


No reaction appeared after the training start

Hi professionals,
We tried the steps in the HugeCTR tutorial and picked DeepFM for trail and successfully started the training, but nothing happened after the 'HugeCTR training start' text (we had waited for several days).
We tried several network configs, which however focusing on the max_iter meaning that network architecture was not changed, same problem.
System: Ubuntu 18.04.4 LTS
GPU: GeForce RTX 2080 Ti
Driver Version: 440.44
CUDA Version: 10.2

Hi, can you please set "display": to 1 in solver parameters in the config file and check if anything is printed. Please ensure you enable "SM 75" when compiling HugeCTR, to work on RTX 2080 Ti.

Hi, can you please set "display": to 1 in solver parameters in the config file and check if anything is printed. Please ensure you enable "SM 75" when compiling HugeCTR, to work on RTX 2080 Ti.

still freezes on 'HugeCTR training start'

Thank you for trying, can you please build HugeCTR with debug flags and share entire log printed on screen when you start training.
cmake -DCMAKE_BUILD_TYPE=Debug -DSM=75 ..

Thank you for trying, can you please build HugeCTR with debug flags and share entire log printed on screen when you start training.
cmake -DCMAKE_BUILD_TYPE=Debug -DSM=75 ..

Nothing appeared on the screen after 'HugeCTR training start'.
More information, we skipped the 'build' folder making and directly cmake, and trimmed the size of criteo dataset with python pandas because of our limited RAM. Not sure whether they matter.

I think DeepFM sample should not consume too much memory and if memory allocation failed it will throw exception. We just release our docker container on NGC: https://ngc.nvidia.com/catalog/containers/nvidia:hugectr. Would you like to have a try? It will help you if you have problem of environments.

I installed newer HugeCTR from GitHub and the problem is solved. Thank you for your attention.

I think DeepFM sample should not consume too much memory and if memory allocation failed it will throw exception. We just release our docker container on NGC: https://ngc.nvidia.com/catalog/containers/nvidia:hugectr. Would you like to have a try? It will help you if you have problem of environments.

Could you please provide more guidance for this NGC docker container for the people who want to try it?

Thank you for your feedback! Please find the guidance on https://github.com/NVIDIA-Merlin/HugeCTR#getting-started. I would like to close this issue but please let us know if you still encounter problems.


The SPARK TROG ids are saved as 15010xx and LARK TROG ids are saved as 1703_0xx. Is the dash in LARK needed?

No, not needed.


Is it worth merging my repo with this??

https://github.com/rajzshkr/onelinejs

Looks great for me. @rajzshkr


MC 1.9.4 support

I think it complete, but it may lack something.

Has this been tested?

Since I don't have any testing methods, I couldn't.
I just tracked wiki.vg's protocol history.

Have you at least tried to log in to a server with it?

The version number still says 1.9.2

I get "Internal Network Exception" when trying to connect to a server (1.9.4 bungeecord) with this update.

Attempting to log in...
Logged in successfully
Attempting to connect to mc.ecocitycraft.com...
Connected successfully
[09:26:00] §f §r§f §r§2 §r§0 §r§4 §r§3 §r§9 §r§2 §r§0 §r§0 §r§3 §r§9 §r§2 §r§0 §r§0 §r§3 §r§6 §r§3 §r§6 §r§3 §r§6 §r§3 §r§6 §r§3 §r§6 §r§3 §r§6 §r
Disconnected: Internal network exception.
Is what I get on my chatclient trying to connect to a 1.9.4 bungeecord server. Unsure what kind of exception is actually thrown, because it gets caught before I can touch upon it.

I'll try to test it today.

I found a bug. Fixing it.

Yup fixed