nexus_to_h5_dict function fails with Pint 0.20.1

Nexus writer uses nexus_to_h5_dict from silx.io.dictdump, but it fails with AttributeError:
Traceback (most recent call last):
   [...]
  File "/home/matias/miniconda3/envs/bliss_demo_env/lib/python3.9/site-packages/nexus_writer_service/io/nexus.py", line 712, in nxInit
    dicttonx(nxdict, h5group, update_mode="modify")
  File "/home/matias/miniconda3/envs/bliss_demo_env/lib/python3.9/site-packages/silx/io/dictdump.py", line 728, in dicttonx
    nxtreedict = nexus_to_h5_dict(
  File "/home/matias/miniconda3/envs/bliss_demo_env/lib/python3.9/site-packages/silx/io/dictdump.py", line 473, in nexus_to_h5_dict
    elif pint is not None and isinstance(value, pint.quantity.Quantity):
AttributeError: module 'pint' has no attribute 'quantity'

pint.quantity was removed in pint v0.20

It seems there is no test in silx, with pint - and pint is not a silx dependency (from having a brief look at setup.py).

There is some tests, but they were skipped because pint was not installed on the CI.
https://github.com/silx-kit/silx/blob/a6e3b693e1f038839b05bf9080585363a85c2ee3/src/silx/io/test/test_dictdump.py#L528
Yes, pint is not in the dependencies of silx, it's only used so far is isinstance(obj, pint.Quantity) on what is provided by the user. But for me that could be added as a strong dependency to make things simpler.
PR #3725 fixes it.

Or a test and a dependency with the tests only.

PR #3725 adds pint to requirements.txt and so to the CI test env so with this PR it will be tested.


threaded frame rate slower than speficied

I have a code example, where the processing pipe's throughput is about 23 fps, while the framerate of PiVideoStream is specified to be 40. See the following code:
from __future__ import print_function
from imutils.video.pivideostream import PiVideoStream
from imutils.video import FPS
import imutils
import time
import cv2

# function to return the difference of elementsums between consecutive frames. If it returns 0,
# then  the frames are identical with a high probability

def diff(frame1,frame2):
    return(cv2.sumElems(frame1)[0]-cv2.sumElems(frame2)[0])

vs = PiVideoStream(resolution=(720,720),framerate=40).start()
time.sleep(2.0)
fps = FPS().start()
diffs=[]
frame2 = vs.read()
while fps._numFrames < 50:
    frame1 = frame2
    time.sleep(0.025)
    frame2 = vs.read()
    diffs.append(diff(frame1,frame2))
    fps.update()

fps.stop()
print("[INFO] elasped time: {:.2f}".format(fps.elapsed()))
print("[INFO] approx. FPS: {:.2f}".format(fps.fps()))
print(diffs)

cv2.destroyAllWindows()
vs.stop()

The script is doing the following:
comparing two consecutive frames by subtracting the elementsums of each from each other. If the result is zero, then the frames must have been identical (since the scene is not static). The for loop is looping through several frames and storing consecutive differences to 'diffs'.
Remember, according to imutils.video.FPS my throughput is 23 frames per second. Therefore if the camera framerate is 40, and the PiVideoStream is actually working like it is supposed to do, then the elements of diffs should be all non-zero. But in fact up to 40% of the elements are zero. try it out
The camera itself is surely able to handel 40fps. So there must be something with the implementation of threading.

Keep in mind that the PiVideoStream and related classes does not guarantee new frames at each iteration of your loop. The thread simply runs in the background and the frame object is updated whenever a new one is polled from the camera. If your video processing pipeline is faster than your physical camera, this is totally a possibility.
I'm marking this as closed since this is the intended functionality.

The processing pipeline is not faster than the physical framerate. But the processing pipeline is faster than PiVideoStream is able to call update(). the read() method inside the procesing loop is called quicker than the threaded video stream is able to truncate the stream.

The rate, at which PiVideoStream is pooling frames is lower than the potential camera-framerate specified.

In my experience, using OpenCV + Python on the Pi it's extremely challenging it even obtain the specified camera framerate. I would suggest running more tests, this time completely removing any threading code, and then measuring the FPS and see how performance changes.
I actually think this this question would likely be better suited for the picamera Issues rather than imutils. Keep in mind that PiVideoStream simply wraps picamera, but in a threaded manner.
If .read is being called multiple times before .truncated can be called, then that is a limitation at the library level of picamera not being able to poll frames fast enough, no? In fact, your time.sleep is evidence of this. The process of clearing the buffer and reading the next frame takes longer than the time it takes to sleep and then call .read. From a PiVideoStream perspective, I don't think there is much you can do about that -- that resides at the picamera level. Again, just because you specify a framerate of 40, doesn't mean that you'll actually get a framerate of 40 out of the library.

So polling takes at least 0.1 seconds? Then there is no point in exceeding 10fps with this method. I think it should be faster to poll frames from capture_sequence. Is it possible to created a threaded frame polling based of capture_sequence?

It depends on the size of your frame. Smaller frames are read much faster. Change the frame size to 320x240 and you'll see much faster reads. As for a threaded capture from capture_sequence, you'll need to check with the picamera developers.

I found a way to create a threaded stream using capture_sequence. below an example for direct grayscale capture:

import io
import time
from picamera import PiCamera
import cv2
import numpy
from threading import Thread
import time

w=640
h=480
fw=(w+31)//32*32
fh=(h+15)//16*16
Y=None
camera=PiCamera()
camera.resolution = (w, h)
camera.framerate = 40
camera.exposure_mode='off'
camera.awb_mode='off'
camera.awb_gains=1
camera.shutter_speed=15000
stream_done=False
cv2.namedWindow("detection", cv2.WINDOW_AUTOSIZE)

def diff(frame1,frame2):
    return int(cv2.sumElems(frame1)[0]-cv2.sumElems(frame2)[0])

def outputs():
    global Y
    stream = io.BytesIO()
    while not stream_done:

        yield stream

        stream.seek(0)
        Y=numpy.fromstring(stream.getvalue(),dtype=numpy.uint8,count=fw*fh).\
           reshape((fh,fw))

        stream.seek(0)
        stream.truncate()

def capt():
    camera.capture_sequence(outputs(), 'yuv', use_video_port=True)
    pass

def startcapt():
    t = Thread(target=capt, args=())
    t.daemon = True
    t.start()

##TEST
startcapt()
time.sleep(0.5)
diffs=[]
frame2=Y
while cv2.waitKey(1) != ord("q"):
    #cv2.imshow('detection',Y)
    frame1 = frame2
    time.sleep(0.025)
    diffs.append(diff(frame1,frame2))

stream_done=True
print(diffs)
cv2.destroyAllWindows()
camera.close()

This one seems to be much faster, and and is able to run the Test without repetitive readouts.

Nice, I'll be sure to play with this and test it out.
A few questions:

Is there a reason you changed a lot of the camera parameters, such as .shutter_speed, .exposure_mode, etc. Did those help you in achieving a faster FPS?

What are fw and fh? I see how they are computed, but I'm not sure what their actual role is when calling np.fromsting.

actually I did not test whether there is an impact on FPS, if you let the awb and shutter speed on auto-mode. Concerning the shutter speed, I think there will be no impact on the framerate, because the firmware adjusts the exposure time according to the specified framerate, not the other way around. The awb algorithm should have an influence. But it is executed on the gpu, so it is quick.
For my application I had to fix these vaules to get consistent images. There is an impact on detection reliability of the hough transform.
2.These little conversions are necessary to correctly interpret the YUV byte string. If you specify a certain resolution, say 100x100, the actual pixel count will be slighty different. The horizontal resolution is rounded up to the nearest multiple of 32 pixels, while the vertical resolution is rounded up to the nearest multiple of 16 pixels. Thats how the camera chip stores the information. for 100x100 specified, the actual size of Y would be 128x112, but since you specified the resolution at camera.resolution, a few overhanging pixels will be uninitialized.
For those who want rgb, just remove the 'yuv' in the capture_sequence, and you also dont need w,h,fh,fw anymore. the resolution is speficied with camera.resolution. also you need to change
Y=numpy.fromstring(stream.getvalue(),dtype=numpy.uint8,count=fw*fh).\
           reshape((fh,fw))

to
frame=numpy.fromstring(stream.getvalue(),dtype=numpy.uint8)

I found a problem: at high resolutions there is a really anoying latency.

Thank you for the added details @Amrosik, it's much appreciated!


Notifications only from one repo

Hi,
I have configured two repositories (1 hg and 1 git) on my trac installation however im receiving notifications only from main one which is hg. Is there a way to recieve notifications from both repositories?

Frankly, we never tried to use trac-mattermost with trac repositories, I'm surprised it even works with hg, are you sure this is the plugin causing notifications? We've been using https://github.com/truveris/hg-mattermost for that. The plugin is setup as two listeners of type ITicketChangeListener and IWikiChangeListener, I really don't know how the repository changes are triggering anything.

Ok, thanks for your answer.


Fuzzy refactor pt 1

Approaches #15 - this is a tricky thing to conceptualize. I think, ideally, we would show recipes with stuff like

Bourbon whiskey (whiskey)

And would consolidate the left side. All of the core functionality needs to change a bit in pedantic mode - compare neighbors loosely, show fuzzy categories, etc.

Bourbon whiskey (whiskey)

Yeah, agreed. That also helps with:

For neighbors, I’d say that the algorithm should only count alcohol ingredients (spirits, liqueurs, fortifieds, bases) or optionally all fluids (alcohol and other fluids: syrup, juice). Swapping rum for gin is important, a lemon for an orange isn’t really that important...

Great idea!

Okay! I can still see a bunch of improvements I want to make, but this data representation seems pretty decent and works well - merging!


Admin option to cancel all games?

Should there be an "admin" feature to cancel all games? This would likely only be useful for testing purposes; relatively simple to implement, since nothing will execute publicly once a game is canceled.

I changed this feature slightly: now, if a game should be canceled, only the user that created it will be able to do so


markdown formatting fix.

this fix changes markdown formating for link to /docs/started/k8s/overview page,
that currently broken on the website.
see - https://www.kubeflow.org/docs/other-guides/virtual-dev/getting-started-minikube/

This change is

/assign
/ok-to-test

Thanks!
/lgtm
/approve


Popup and Popover behind overlay

I dont know why Popover and Popup falling behind overlay. Here is the picture..

Please i need urgent help to resolve this..
Thanks

I confirm, I had the same problem. (You can close)

Hai friend i have solution for popup overlay.
.popup-overlay{z-index: 111 !important;}


Add a completion provider for issues

Works in git SCM input and editor. In markdown, it inserts a markdown link

@RMacfarlane feel free to review as much or as little as you want. Any feedback is always welcome!


No such file or directory - git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet

Here is the error I get, do you know how to solve this?
C:\Ruby22-x64\bin\ruby.exe -e $stdout.sync=true;$stderr.sync=true;load($0=ARGV.shift) C:\Ruby22-x64\bin/bundle install
Fetching https://github.com/jejacks0n/mercury.git
Retrying git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet due to error (2/4): Errno::ENOENT No such file or directory - git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet
Retrying git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet due to error (3/4): Errno::ENOENT No such file or directory - git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet
Retrying git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet due to error (4/4): Errno::ENOENT No such file or directory - git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet
--- ERROR REPORT TEMPLATE -------------------------------------------------------

What did you do?
I ran the command C:\Ruby22-x64\bin/bundle install

What did you expect to happen?
I expected Bundler to...

What happened instead?
Instead, what actually happened was...

Error details
Errno::ENOENT: No such file or directory - git clone "https://github.com/jejacks0n/mercury.git" "C:/Ruby22-x64/lib/ruby/gems/2.2.0/cache/bundler/git/mercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a" --bare --no-hardlinks --quiet
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git/git_proxy.rb:133:in ``'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git/git_proxy.rb:133:in `block in git'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/shared_helpers.rb:71:in `call'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/shared_helpers.rb:71:in `with_clean_git_env'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git/git_proxy.rb:133:in `git'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git/git_proxy.rb:127:in `block in git_retry'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/retry.rb:38:in `call'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/retry.rb:38:in `run'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/retry.rb:28:in `attempt'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git/git_proxy.rb:126:in `git_retry'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git/git_proxy.rb:82:in `checkout'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git.rb:153:in `specs'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:541:in `block in converge_locked_specs'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:530:in `each'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:530:in `converge_locked_specs'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:192:in `resolve'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:137:in `specs'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:126:in `resolve_remotely!'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/installer.rb:194:in `resolve_if_need'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/installer.rb:69:in `run'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/installer.rb:21:in `install'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/cli/install.rb:105:in `run'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/cli.rb:172:in `install'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/cli.rb:10:in `start'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/exe/bundle:19:in `block in <top (required)>'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/friendly_errors.rb:7:in `with_friendly_errors'
  C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/exe/bundle:17:in `<top (required)>'
  C:/Ruby22-x64/bin/bundle:23:in `load'
  C:/Ruby22-x64/bin/bundle:23:in `<top (required)>'
  -e:1:in `load'
  -e:1:in `<main>'

Environment
Bundler   1.11.2
Rubygems  <IP_ADDRESS>
Ruby      2.2.4p230 (2015-12-16 revision 53155) [x64-mingw32]
GEM_HOME  C:/Ruby22-x64/lib/ruby/gems/2.2.0
GEM_PATH  C:/Ruby22-x64/lib/ruby/gems/2.2.0;C:/Users/zezima/.gem/ruby/2.2.0
Git       not installed

--- TEMPLATE END ----------------------------------------------------------------
Unfortunately, an unexpected error occurred, and Bundler cannot continue.
First, try this link to see if there are any existing issue reports for this error:
https://github.com/bundler/bundler/search?q=No+such+file+or+directory+-+git+clone+"https%3A%2F%2Fgithub.com%2Fjejacks0n%2Fmercury.git"+"C%3A%2FRuby22-x64%2Flib%2Fruby%2Fgems%2F2.2.0%2Fcache%2Fbundler%2Fgit%2Fmercury-cda6f0a977e67f14dc7f94bde1bcb92f180ba37a"+--bare+--no-hardlinks+--quiet&type=Issues
If there aren't any reports for this error yet, please create copy and paste the report template above into a new issue. Don't forget to anonymize any private data! The new issue form is located at:
https://github.com/bundler/bundler/issues/new
Process finished with exit code 1

The issue is that you don't have git available on your PATH

@segiddins Ok, but how to do that? Sorry I'm newbie here...

Well, if you want to use git dependencies in your Gemfile, you need to install Git (https://www.git-scm.com)

@segiddins I have already installed this :) Is there any tutorial how to add git to path?

@segiddins Ok, I have git im my PATH, but I still get this:
"C:\Program Files (x86)\JetBrains\RubyMine 8.0.3\bin\runnerw.exe" C:\Ruby22-x64\bin\ruby.exe -e $stdout.sync=true;$stderr.sync=true;load($0=ARGV.shift) C:/Users/zezima/Projects/backup/script/rails server -b <IP_ADDRESS> -p 3000 -e production
C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git.rb:191:in rescue in load_spec_files': git://github.com/jejacks0n/mercury.git (at master@fee3bf2) is not yet checked out. Run bundle installfirst. (Bundler::GitError) 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git.rb:188:inload_spec_files'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/path.rb:92:in local_specs' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/source/git.rb:159:in specs'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:209:in block (2 levels) in index' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:207:in each'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:207:in block in index' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/index.rb:9:in build'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:204:in index' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:198:in resolve'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:137:in specs' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:182:in specs_for'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/definition.rb:171:in requested_specs' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/environment.rb:18:in requested_specs'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/runtime.rb:13:in setup' 	from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler.rb:92:in setup'
from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/setup.rb:18:in <top (required)>' 	from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:128:in require'
from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:128:in rescue in require' 	from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:39:in require'
from C:/Users/zezima/Projects/backup/config/boot.rb:6:in <top (required)>' 	from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in require'
from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in require' 	from C:/Users/zezima/Projects/backup/script/rails:5:in <top (required)>'
from -e:1:in load' 	from -e:1:in '
Process finished with exit code 1
How to solve it?

@zezima0 Try this http://blog.countableset.ch/2012/06/07/adding-git-to-windows-7-path/
And you should close this issue if there's nothing wrong with bundler itself - as far as I could tell, no.

@zezima0 Is this still valid? If not, can you close this?


[PATCH] Linter should recognize throw as return-type

Summary
Linter should not complain about implicit return (and require an
unreachable/stale return if the last statement in a function is
a throw statement).
Test Plan
Update linter version, navigate to a markforged project that uses
our linter (i.e. Eiger), add a throw statement at the end of
a function. Run linter to make sure it doesn't complain (it did before).
Add another non-returning statement after throw (i.e. a string)
and make sure linter still complains about explicit return.

this looks fine

🐳


Host docs on Github?

This is related to https://github.com/swipesapp/swiss-react/issues/26.
I think it's a good idea to host the docs on Github so that people using it can point out bugs and suggest updates. Hosting it on Contentful just isn't scalable for an open source tool.

Similar to how React docs are hosted
Rendered docs page: https://reactjs.org/docs/getting-started.html
Source: https://github.com/reactjs/reactjs.org/blob/master/content/docs/getting-started.md

Love it! Let's host it on Github.


DevOps Services Docs Search is not very flexible

It's very hard to find documentation by the search in the DevOps services docs, as the search is only by title of the documentation. Being able to search by task name (ie, not only Publish Test Results, but also PublishTestResults) would make the documentation much faster and easier to use.

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

ID: ba91e46a-de53-c0c1-f7f7-c02e797d527c
Version Independent ID: 6f478148-461f-694a-ed82-deb4d731854e
Content: Publish Test Results task - Azure Pipelines & TFS
Content Source: docs/pipelines/tasks/test/publish-test-results.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Thank you for your feedback. I will pass this on to the website development team.


No bot on initial cla.yml PR

Hey folks, I'm trying to add this to the jank-lang org and I have the initial PR adding the cla.yml here: https://github.com/jank-lang/.github/pull/1
Unfortunately, even with the platformcontext.yml and the Policy Service installed, I'm not seeing an initial comment from the bot. Am I missing something?

Switched to https://cla-assistant.io/ which worked out of the box.


Wallet Not Syncing

Wallet sync is stuck, what can I do?

@aalex2435 try to delete everything in the Monyx folder C:\Users\YourUserName\Appdata\Roaming\Monyx EXCEPT wallet.dat file and run the app again.


Marker and route styling

Markers should be more elongated and/or pointed. The route lines should be thinner and more transparent.
In the old maps version it looked like this:

currently it looks like this:

I think we set the 'position' of the marker to the center of the bottom line of the viewBox here: https://github.com/graphhopper/graphhopper-maps/blob/799d7e2e22e73d0dfdd7f16590cc699a53a8e388/src/layers/UseQueryPointsLayer.tsx#L63
So the question becomes whether the tip of the marker path is located at the center of the bottom line of the viewBox. If it isn't the marker will be 'moving' when we zoom in or out. I tried to fix it here: 4d9f450a25d3ebb2a7855e26be209dc4db7bf68a

Btw I copied the svg to https://www.svgviewer.dev/ where there is a 'Remove Padding' button that seems to trim the view box to the size of the path.


Why not use batch_size, epoch, just iter training

I want to adapt the code to multi-GPUs training, and encountered some problems in the process of adding DDP (DistributedDataParallel).
May I ask why nerf and barf do not use batch_size, regardless of epoch training, directly use iter training?
This question is asked because DDP needs to use sampler to divide data across GPUs in each epoch. I don’t know if only iter and not disrupting data will affect performance.

This is because it randomly samples pixels from all the training images in each iteration, so there's really no notion of epochs anymore (this follows the practice of the original NeRF). You could batch images and then sample pixels from these selected images (in fact, this was how I initially developed the code); statistically all the pixels would still have equal probabilities of being seen in the long term, but it actually results in biased gradients towards these selected images, and empirically we found that it could find suboptimal results.
There are some ways I can think of to make it work with DDP:

The current code caches all training images in the GPU memory, so you could do the same for all GPUs and let each thread run its own pixel sampler. This may be the faster hack, but will be more memory wasteful.
Modify the code to train in epoch mode, which is what you're looking for (should not be difficult). If the effective batch size in DDP approaches the dataset size (i.e. number of images), this bias should be well mitigated as well.

Hope this helps!


Tag version needs to be validated with composer.json

Currently there's no validation of the tag with the version in the composer.json. This difference, besides confusing, would cause the versions compiled/packaged be different from what is shown on the pickleweb site or in phpinfo().
This interfere with the pickle issue https://github.com/FriendsOfPHP/pickle/issues/111 , so pickleweb could be using the pickle itself for the triple validation - tag, composer.json and the version in the header file (or any other way for different engines).

We should remove the version from composer json and only use the tag.
Rmtools can add it so the binaries archive has this  info.

Yeah, but we could still use pickle to check the PHP_EXT_VERSION in the source and compare with the tag, no?

Exactly, or better said we need both. Server and client side. Bad version
matching must be rejected
On May 9, 2015 7:29 PM, "Anatol Belski"<EMAIL_ADDRESS>wrote:

Yeah, but we could still use pickle to check the PHP_EXT_VERSION in the
source and compare with the tag, no?
—
Reply to this email directly or view it on GitHub
https://github.com/FriendsOfPHP/pickleweb/issues/40#issuecomment-100473781
.

Check should still be added. Just in case someone puts it in composer.json. I will do it.


HTTPError: Response code 400 (Bad Request)

I've been using this node sucessfully for a couple of months, but since a week or two it started giving this error:
HTTPError: Response code 400 (Bad Request)
I thought it had to do something with the firmware update, which I executed today. But that didn't solve it. I haven't changed anything else, it just stopped working. I tried re-entering my settings, setting it up from scratch but it keeps giving me this error.

I'm experiencing the same issue.
Started occurring out of the blue today after using it for a few weeks.

Me too. Had it before. I deleted the tokenset.json and restarted Node red. After that it seems to rune fine again.
Somehow I think the token gets out of date.
Maybe there could be some mechanism to refresh the token every 3 weeks or so (just a thought)

@bartjanisse: That did the trick, thanks ;)

Great, that did the trick.

I think the problem is that the tokenset.json is broken at some point. I had the same issue some time ago but have not logs to get into this topic more deeper. I implemented a setting in the Daikin Cloud device to not save the "tokenset.json".  This could be done by setting the "Save token" value to Disabled in your node.

@bartjanisse @MaartenProvo  @fiendie
Also check this pull request on the daikin-cloud-controller lib which is used here by this node.
Timeout and retry for GOT client
After adding the timeout and retry functionality it is now safer for me to update the token.
Since this functionality is not released into the latest version of Daikin-cloud-controller lib please use the version from  GitHub. The actual released version which is automatically installed with this npm does not support the settings for retry and timeout and ignore whatever you set in the node config. When there is a timeout during update of tokenset.json this might broke your tokenset.json.


traintest_ft.py 中缺少  calculate_stats 函数

hi there,
That is in https://github.com/YuanGongND/cav-mae/blob/master/src/utilities/stats.py.
In these two lines, the above script is imported:
https://github.com/YuanGongND/cav-mae/blob/68fe8c2a3917dc2926e41f796bfdcb331a64b42c/src/traintest_ft.py#L11-L12
-Yuan


Avatar drag error

I can drag the avatar out of the page.
https://images.zenhubusercontent.com/660762354/ee779a93-de92-49ff-91c0-a424eaf2c836/screen_recording_2023_08_07_at_17_18_40.mov

I can't reproduce it, are you able to reproduce it Miki?

Yes I tried again now and I reproudce it again. If not we can huddle and I will show you.


Update Neynar in bootstrapPeers.mainnet.ts

We had to rearrange our domains to make this work with our load balancer
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[ ] PR title adheres to the conventional commits standard
[ ] PR has a changeset
[ ] PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore)
[ ] PR includes documentation if necessary.
[ ] All commits have been signed

PR-Codex overview
This PR updates the bootstrap peers for the mainnet in the Hubble app.
Detailed summary

Updated bootstrap peers for mainnet in bootstrapPeers.mainnet.ts
Removed /dns/elen.hubs.neynar.com/tcp/2282 and /dns/tari.hubs.neynar.com/tcp/2282
Added /dns/bootstrap.neynar.com/tcp/2282 as a new bootstrap peer

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

We are still debating if we want 2 names or 1 name with 2 a records

We are still debating if we want 2 names or 1 name with 2 a records

good to proceed with 1 for now

Closed in favor of #1750


Proto buf

Related to https://github.com/frmscoe/General-Issues/issues/261

Draft: Still Supporting protobuf throughout the platform


Minimize output of git-find-ignored

If a file is ignored because the directory it's in is ignored, just print the directory name, not the untracked files within it.

I'm seeing very verbose output for make clean with pkg-php. Is that caused by this?

Yep, I have a snippet somewhere to fix this. I’ll take a look tomorrow.

For future reference:
#!/bin/bash

pathlist='/foo/
/foo/bar/
/foo/bar/file.txt
/shazam/blammo
/shazam/blammo/file1.txt
/shazam/blammo/file2.txt'

i=2

while [ $i -lt 100 ]
do
  path=`echo "$pathlist" | cut -f1-$i -d/ | uniq -d`
  if [ -z "$path" ]
  then
     echo $prev_path
     break
  else
     prev_path=$path
  fi
  i=`expr $i + 1`
done

There's possibly a better way to solve that, by sorting first and implementing a more obvious is_parent_of type function.


Clavel 2023

Publication DOI or URL
https://doi.org/10.1016/j.isci.2023.106787
Additional context
PRJEB59233

Diana did a fantastic first pass, but even myself couldn't work out which of the 400 libraries the authors made went into the final genomes - have contacted the first author.


Actions toggleable on List Create Controller.

This pr provides the ability to pass in a boolean property (showActions) to the list create controller, which then hides actions associated to list create controller.

Fix merge conflicts.


RowsPostProcessingError in the viewer of serveral datasets

We find a RowsPostProcessingError in the viewer of several datasets:
Server error while post-processing the split rows. Please report the issue.

Error code:   RowsPostProcessingError

https://huggingface.co/datasets/multi_woz_v22
https://huggingface.co/datasets/qasper

It seems like the issue is that we don't support a Sequence of dicts adequately:

A datasets.Sequence with a internal dictionary feature will be automatically converted into a dictionary of lists. This behavior is implemented to have a compatilbity layer with the TensorFlow Datasets library but may be un-wanted in some cases. If you don’t want this behavior, you can use a python list instead of the datasets.Sequence.

https://huggingface.co/docs/datasets/v2.5.2/en/package_reference/main_classes#datasets.Features

#603 only fixed part of the datasets, ie https://huggingface.co/datasets/qasper

But not https://huggingface.co/datasets/multi_woz_v22

Working on it in #605

OK, #605 fixed https://huggingface.co/datasets/multi_woz_v22 as well

Still 29 datasets failing with RowsPostProcessingError -> it seems to be due to another reason: missing fields (columns), or None values, in some rows. https://github.com/huggingface/datasets-server/pull/606 will fix it


Adds additional javadocs and removes broken link.

Description

Fixes javadoc warnings by adding javadocs to the default constructor in Event Hubs and Service Bus. (Causing partial failures on our builds.)

"WARNING] /mnt/vss/_work/1/s/sdk/eventhubs/azure-messaging-eventhubs/src/main/java/com/azure/messaging/eventhubs/models/SendOptions.java:14: warning: use of default constructor, which does not provide a comment
[WARNING] public class SendOptions {
[WARNING] ^
[WARNING] 6 warnings
"

Removes broken ReadmeSamples.java link.

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-messaging-servicebus


Fix Android Nougat datetime picker mode="spinner"

Summary
I got it from https://gist.github.com/jeffdgr8/6bc5f990bf0c13a7334ce385d482af9f and
did some adjustments in order to work with DatePicker. Workaround for this bug: https://code.google.com/p/android/issues/detail?id=222208. In Android 7.0 Nougat, spinner mode for the DatePicker in DatePickerDialog is incorrectly displayed as calendar.
Test Plan
Run Example with Android Emulator with Android Nougat (7.0), and modify example setting mode="spinner"
What are the steps to reproduce (after prerequisites)?
Run Example in current master branch with Android Emulator with Android Nougat (7.0), and modify example setting mode="spinner" the emulator shows calendar
Checklist

[X] I have tested this on a device and a simulator

This should be merged assap :)

Sorry this is mistype? display="spinner" or mode="spinner"

@FDiskas display="spinner" is correct :)


Update DG

Update Acknowledgements section
Fix Design section
Removed Implementation section
Update Appendix: Requirements section with 3 "Requirements yet to be implemented"
Update Appendix: Planned Enhancements section

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:


HIVE-24224: Fix skipping header/footer for Hive on Tez on compressed files

Change-Id: I918a2eff0197e7d92db1f1858f3402b874d3b10a
What changes were proposed in this pull request?
Fix header/footer skipping for compressed files -- bug discovered for Hive on Tez
Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@abstractdog  @mustafaiman can you please take a look?

+1 LGTM


Server: Apache/2.4.23 (Ubuntu)
Engine-Mode: "ENABLED"

--7a086438-Z--

right, I found it!
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#noauditlog
SecRule REQUEST_FILENAME '^/asdf/health-test-sql-injection(/.*)?$' \
  "phase:2,id:11111,nolog,noauditlog,ctl:auditEngine=Off"

@jurgenweber Yeah, that's the same thing I always do to completely skip logging :)


Replaced return statements in DataAccess classes to return Task.FromR…

Bit of refactoring of the data access classes to return Task.FromResult(0) instead of just null when the return type is Task, as per #613

@YannickMeeus that was fast!
I'm not too sure if this also applies to methods returning async Task or async Task<T>. I'll let you know if that's the case and you can pick the rest of that stuff up.
Also, if you're looking for something to do next, there are still controllers out there with no unit tests. A simple one to pick up would be OrganizationApiController. More involved; ActivityController ad TaskController. VERY involved: ManageController.


pkgs(proj): 9.1.1 -> 9.2.0

Also contains Nixpkgs PR: https://github.com/NixOS/nixpkgs/pull/218719

Fails with:
cmake flags: -DCMAKE_FIND_USE_SYSTEM_PACKAGE_REGISTRY=OFF -DCMAKE_FIND_USE_PACKAGE_REGISTRY=OFF -DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_LOCALEDIR=/nix/store/15mx0ds90b50g162fnnmf08yyjz1p6cw-proj-9.2>
-- The C compiler identification is GNU 11.3.0
-- The CXX compiler identification is GNU 11.3.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /nix/store/dkw46jgi8i0bq64cag95v4ywz6g9bnga-gcc-wrapper-11.3.0/bin/gcc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /nix/store/dkw46jgi8i0bq64cag95v4ywz6g9bnga-gcc-wrapper-11.3.0/bin/g++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Requiring C++11
-- Requiring C++11 - done
-- Requiring C99
-- Requiring C99 - done
-- Configuring PROJ:
-- PROJ_VERSION                   = 9.2.0
-- Looking for localeconv
-- Looking for localeconv - found
-- Looking for strerror
-- Looking for strerror - found
-- Looking for dladdr in dl
-- Looking for dladdr in dl - found
-- Looking for exp in m
-- Looking for exp in m - found
-- Found nlohmann_json: /nix/store/2hh33wn2h6ckmwb4vjvvnxxnbw4255cd-nlohmann_json-3.11.2/share/cmake/nlohmann_json/nlohmann_jsonConfig.cmake (found version "3.11.2")
-- found nlohmann/json version 3.11.2
-- nlohmann/json: external
-- Found Sqlite3: /nix/store/329w43vambhv2pd4kbnqqlyksgyn3jd9-sqlite-3.39.4/lib/libsqlite3.so
-- Sqlite3 version: 3.39.4
-- Found TIFF: /nix/store/0kbsq5gd29l64xqm9qalcjcrg8rdld8f-libtiff-4.4.0/lib/libtiff.so (found version "4.4.0")
-- Found CURL: /nix/store/9g6z13kf6zq6vsbzz6r44mqf43i1spkj-curl-7.86.0/lib/libcurl.so (found version "7.86.0")
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- Configuring proj library:
-- ENABLE_IPO                     = OFF
-- PROJ_CORE_TARGET_OUTPUT_NAME   = proj
-- BUILD_SHARED_LIBS              = ON
-- PROJ_LIBRARIES                 = proj
-- Could NOT find GTest (missing: GTEST_LIBRARY GTEST_INCLUDE_DIR GTEST_MAIN_LIBRARY) (Required is at least version "1.8.1")
CMake Error at test/unit/CMakeLists.txt:13 (message):
  External GTest >= 1.8.1 not found

-- Configuring incomplete, errors occurred!


in Json tripped me up

I've been testing vamp-router, and was copying and pasting examples from the readme.
There are lots of hashes in the json examples so anyone that just copies and pastes, that son will not parse. I also would only get the bad request error back and nothing helpful was dumped into the log so I had to dump c.Errors to figure out it was the parsing error. I'm happy to fix this if you guys are ok with me removing the hashes and submitting a PR.

Hi Sean, thanks for notifying! We'd be happy to update the examples. After internal discussion i'd like to propose that we provide better explanation so people directly see what can be pasted and what needs to be removed as it is a comment to explain. Is that something you could live with?

Sounds good Olaf.

i am not able to post a single request despite rest client APIs are up. It always says bad request


Test generated zod schemas

Maybe make https://github.com/ome/ngff a submodule and reuse the examples from there?
Thoughts @joshmoore, @will-moore?

see #3

Nice, @manzt! Another option that was discussed was packaging the content of ome/ngff for re-use. Let us know what would work best for you.

Thanks, josh! If the test cases were all transitioned to the tests/<schema_name>_suite.json used in v04/latest, then it would be simple enough to fetch these fixtures in a structured manner and run the tests. The entire "test" suite for "suite" files fits in <10 LOC:
import { describe, expect, test } from "vitest";
import * as schemas from "../src/0.4";
import { gather_test_cases } from "./utils";

let cases = await gather_test_cases("0.4", schemas);
describe.each(cases)("$description - $schema.id", ({ Schema, tests }) => {
  test.each(tests)("$formerly", ({ data, valid }) => {
    expect(Schema.safeParse(data).success).toBe(valid);
  });
});

For now, the v01/v02/v03 require special handling of paths and I would worry about keeping things in sync.

:+1: for backporting the suites and happy to help. I won't be able to prioritize until mid April or so. Are you ok to file an issue on ome/ngff and see if we get any buy-in?

Also won't be able to look in deeper until April. I'll try to file an issue ASAP


Wampire lord Crash.

Describe the bug
I accepted to become vampire after bringig Serana back to the castle (Start of the Dawnguard) after transition to tutorial area I activate change to Vampire Lord and game crashes. I tried to restart game and pc few times but it always crashes right as change happenes. Right when model changes from human to VampLord.
To Reproduce
Steps to reproduce the behavior:
Accept to become Vamprie from Volkihar, then in tutorial are try to use Vampire Lord power. It always crashes for me.

Go to '...'
Do '...'

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem. I highly recommend opening the console and clicking on the object that has a bug. If your bug is location based, take a screenshot of You are Here MCM Option from DynDOLOD as well as a screenshot of the world map.
Modlist version
Do not say latest
2.1.1 (Balanced)(No adational modifications) (Installed from wabbajack and played)
Did you add any additional mods?
No

Problem is with " Scion - A Vampire Overhaul " when i disable that mod game no longer crashes during Vampire Lord transformation.

Can you please upload the crash-log files.
They are found in Screenshots ! Racemenu Presets ! Game Generated
and in there in the NetScriptFramework/Crash folder.
Cause finding the actual issue is important to me.

Crash_2021_12_27_22-9-1.txt

Crash_2022_1_12_16-15-3.txt
This is latest report.

Crash_2022_1_12_15-46-34.txt
Crash_2022_1_12_16-15-3.txt
Crash_2022_1_12_15-55-50.txt
Crash_2022_1_12_15-54-8.txt
Crash_2022_1_12_15-52-1.txt
Crash_2022_1_12_15-50-17.txt
Crash_2022_1_12_15-48-37.txt

loadorder.txt.zip
could you put the 3 files in that zip into the ModOrganizer 2\profiles\Tales from the Northern Lands (Balanced) Folder and then hit this button:

on the plugins tab (right side)

I am pretty sure I messed up a mod order there. That backup should reorder it.

There are no backup to restore.

how does the folder look did you extract the zip

First like that, but then i deleted numbers after .txt and replaced ones that were in Tales from the Northern Lands (Balanced)

that screenshot is inside the archive and not extracted
drop them like they are here in the folder where the originals ones are and hit the restore button then launch the game an see if the transformation works

your rename might work as well but it might not work if modorganizer 2 is open while replacing the file.

This is folder after i click rewrite the files during transfer.

FNIS missing and still crashes.
Backup worked second time i unzipped files.

Crash_2022_1_12_16-52-55.txt

FNIS missing is on purpose

my guess scion and this mod have an incompatibility causing the crash. can you try to redo the transformation with that mod disabled but scion enabled. and send the log of that.

Ok, so old save os in infinite loading. But new game runs and I tried add VampLord power and it didnt crash during transformation.
Where do I find log if it is not a crash?

Sadly there is no way for a log with that info. Guess I will have to remove the mon in an update that will require a new save soon.

Sorry :)
But new save works fine, thank you so much for help!

thx for reporting it and trying the stuff for me 👍


Command "python setup.py egg_info" failed with error code 1

When I try to install torch-scatter, it always comes with this problem.
I tried 'pip install --upgrade setuptools', but it doesn't work.
Anyone can help me fix this problem???  Thanks a lot!

Hello Lee,
PyTorch Geometric (and the old PyTorch FFI interface) currently do not support Windows. We are actually calling a bash script to compile the CUDA kernel in the installation script and there may be some path issues. This should be fixed if all dependencies are updated to the new extension API (e.g. torch-unique should install fine).
I do not own a Windows machine by myself, so currently I can't help you, I'm really sorry!

Got it and thank you for your reply! Looking forward to your better PyTorch.


AudioDestinationNode is not a subtype of type 'void' error when connecting

Code snippet on dartpad that executes with no errors:
https://dartpad.dartlang.org/c368a7743d437802887a8887a1c45153
The same code when running in an application (Firefox/Chrome) gives a runtime error on the connectNode call, "AudioDestinationNode is not a subtype of type 'void'".  No duplicate class names in the application code.
Using dart sdk version 1.17.1

The example from dartpad:
import 'dart:web_audio';
import 'dart:typed_data';
void main()
{
	Float32List tempList = new Float32List(100);
	AudioContext _audioCtx = new AudioContext();
	AudioBufferSourceNode source = _audioCtx.createBufferSource();
	AudioBuffer audioBuffer = _audioCtx.createBuffer(1, tempList.length , 44100);
	audioBuffer.getChannelData(0).setAll(0, tempList);
	source.buffer = audioBuffer;
	source.connectNode(_audioCtx.destination, 0, 0);
	source.start(1000);
}

The error is because connectNode is a void-function but returns a node.
The fix is easy (don't use "=>" but use "{}" instead):
(Note: I copied the changes from the template, since I don't have a Dartium checkout on my machine).
https://codereview.chromium.org/2118633002
@alan-knight : could you please add a test and then land this. Thanks.

That was it! Good find.

I landed a CL that fixes the problem, but reopening, since there is still a test missing.


Export using previously selected files and settings does not pick up new files within folders

Describe the bug
When I configure an export, and check off a folder, I am expressing an intent to "export all notes within this folder". Intuitively, I would expect that if I add a note to a folder that I checked off during my last export, the new note would automatically get included with the next export, since in my mind I had selected the entire folder, not each individual note therein at the time of export. Unfortunately, this is not how it works. If I create new notes since my last export, I must go through the full export wizard (re-selecting hundreds of notes and folders) in order to restore my previous configuration + the new notes. This is a suboptimal user experience that causes me to spend a lot more time exporting my notes than I would like.

This is already implemented in the 1.7.4 branch and will be released in the next update :)

This should be fixed in 1.8.0. Let me know if it still does not work as expected and reopen the issue. If you find any new issues please submit a new issue :)
Thanks!


Lyrics skipping

OS: Windows 7
Issue Descriptions: Lyric scrolling does not track correctly when songs change. The time progress does not always reach the end.
Steps to Reproduce:

Play Song A all the way through.
Open up the lyrics and leave them open.
At the end of Song A but the progress bar has not reset (left at anywhere from 80-100% typically).
Song B starts, lyrics still left open.

The progress bar continues from that 80% point for Song B, despite being at the start of the song, and as a result most of the lyrics scroll by fast to match the progress bar.
It could likely be solved by ensuring that the progress tracking is reset to 0 at the start of each song.

This is the time fetching issue that was fixed in master


Bugs 2 and 4 also apply to Chrome

I had the same issue on:

Nexus 4, Android 5.0.1, Chrome 41.0.2272.96
Linux Ubuntu 14.04, Chrome 41.0.2272.101 (64-bit)
Linux Ubuntu 14.04, Chromium 41.0.2272.76 (64-bit)

Should I create a PR to add them?

I can't reproduce this. Are you looking at the Codepen examples listed in the README? Can you upload a screenshot?

I had it in my code, this exact situation. Didn't try the codepen as I found your repository after searching for the issue and finding the fix. I'll try to reproduce and upload screenshots.

I'm going to close this for now, can you please upload them via the comment if you get around to providing the screenshots?


Make the application more resilient to endpoint that are not in the catalogue

If a connected endpoint does not have a corresponding entity in the entity catalogue the application fails to login.
There are various places that throw exceptions in this case, we should fail as gracefully as possible.

Some work done on this recently - check other outstanding issues.


Run on-prem?

How much work would it be for this service to be run on-prem against an on-prem AD? I would love to use this service however we're not allowed (currently) to use Azure (or any cloud) services for our code / binaries.

Technically the service would work fine, but you'd need a way to authenticate the accounts that have access to call the service.

Would it be possible to link that to an on-prem AD. Would on-prem AD be much different from Azure AD?

If you're running on-prem, then I suppose the best way would be to secure access to the service via Kerberos/NTLM group membership. There'd probably be code changes needed to support this though.

Ok when I get to that part of our infrastructure changes I will have a look at what it takes to make that work. Would you be willing to take the changes if I put in a pull request? If not that's cool then I'll just keep a fork around somewhere. Obviously all assuming we actually get to that point :)

Closing this for now; the service itself will run on-prem. Using a different auth mechanism could be an option but not one I plan on writing. I'd take a PR that would make it configurable though.


[PE-D][Tester C] Allow 2 same person to be in the same meeting

Might be better to flag out that 2 same person have been added to the same meeting.

Labels: severity.Low type.FunctionalityBug
original: Tanziyanghenry/ped#3

Will fix this bug so that future add-schedule commands will not be able to have duplicate indices.


No documentation for 'bring your own access token' with NuGet client libraries

@idg10 commented on Mon Jan 21 2019
I am trying to use the NuGet client SDK to access a protected feed (hosted on Azure DevOps). I need to perform some operations that aren't directly supported by existing NuGet apps or ADO build pipeline tasks, which is why I'm programming directly against the SDK. The relevant code will be running as part of a build pipeline. It's not entirely clear how NuGet clients are supposed to authenticate in this scenario, but having asked some ALM MVPs, they've pointed me at the system.accesstoken build variable.
If I program directly against the NuGet REST API with HTTP requests, this works fine—I'm able to get the token from that build variable, and it works just fine for accessing the protected feed. I just need to put the token into the HTTP request's Authorization header, prefixed with Bearer , and the ADO package feed is happy to talk to me.
But I don't want to work directly at the HTTP level. I was hoping to use the NuGet client SDK, because it has a whole load of well thought out and extensively tested support for working with NuGet feeds.
Unfortunately, the NuGet Client SDK docs appear to consist entirely of "Go read these three blog entries, and then because those were written in 2016 and there have been important changes since then, meaning that what you've just read is out of date, now read this fourth blog entry."
Here's my starting point:
SourceRepository source = Repository.Factory.GetCoreV3("https://api.nuget.org/v3/index.json");
ListResource listResource = source.GetResource<ListResource>();
IEnumerableAsync<IPackageSearchMetadata> x = await listResource.ListAsync(
    "Endjin.Retry",
    prerelease: false,
    allVersions: true,
    includeDelisted: false,
    log: new NullLogger(),
    token: CancellationToken.None);

IEnumeratorAsync<IPackageSearchMetadata> e = x.GetEnumeratorAsync();
while (await e.MoveNextAsync())
{
    IPackageSearchMetadata c = e.Current;
    Console.WriteLine(c.Identity);
}

This works well enough against the public NuGet.org feed (although given the state of the documentation I've no idea whether this is the best way to do this). It searches for the publicly listed Endjin.Retry package. It finds it. This is good.
However, if I replace the URL there with the one for my private feed: "https://pkgs.dev.azure.com/IanGriffiths/_packaging/Ian.LibraryVersioningExperiment/nuget/v3/index.json" I get this error:
System.AggregateException: One or more errors occurred. (Unable to load the service index for source https://pkgs.dev.azure.com/IanGriffiths/_packaging/Ian.LibraryVersioningExperiment/nuget/v3/index.json.) ---> NuGet.Protocol.Core.Types.FatalProtocolException: Unable to load the service index for source https://pkgs.dev.azure.com/IanGriffiths/_packaging/Ian.LibraryVersioningExperiment/nuget/v3/index.json. ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 401 (Unauthorized).
This seems to occur inside the call to source.GetResource<ListResource>().
This is unsurprising of course—this is a protected feed, and my code has not yet made any attempt to supply credentials, so you'd expect it to fail.
But the question is: how am I supposed to supply a token?
It looks like HttpHandlerResourceV3Provider.cs is where it sets up the HttpClientHandler and HttpMessageHandler. In particular, I found at https://github.com/NuGet/NuGet.Client/blob/dev/src/NuGet.Core/NuGet.Protocol/HttpSource/HttpHandlerResourceV3Provider.cs#L62 this code:
messageHandler = new StsAuthenticationHandler(packageSource, TokenStore.Instance)
{
    InnerHandler = messageHandler
};

This made me wonder if I needed to supply the token store with my token. So I tried adding this line before doing anything else with the client libraries:
TokenStore.Instance.AddToken(new Uri(FeedUrl), token);

But it made no difference. And although I can't claim to have understood how 'STS Authentication' fits into this code, it appears it sets the X-NuGet-STS-Token header. But the token I've got is for use as an HTTP Authorization Bearer header. In case it's relevant, here's the decoded body of the JWT that I've got:
{
  "nameid": "fb6c9195-1108-490b-a5a2-1d36f69972af",
  "scp": "app_token",
  "aui": "1e7bc101-05e3-4396-8ae2-481fd28eed86",
  "sid": "f3614a90-e4df-4cf5-ad12-f2f3f0c14fa8",
  "iss": "app.vstoken.visualstudio.com",
  "aud": "app.vstoken.visualstudio.com|vso:49ebc6d2-c78c-4aa6-96af-1a4273e6143b",
  "nbf":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>
}

To reiterate, this works just fine against the package feed when I hit it manually from Postman. (Obviously I need to get a new token every now and then because they expire, but I've manually verified that the token I'm trying to use with the NuGet Client SDK is still valid even after seeing 401 failures in my app by using it again directly against the HTTP API.)
I've cloned the NuGet/NuGet.Client repo and searched it for use of the Authorization header, and the only place it shows up directly is in unit tests, all of which appear to be looking for basic auth. So it appears that the client doesn't set the Authorization header directly. As far as I can tell, instead it sets the HttpClientHandler.Credentials property with an HttpSourceCredentials. This seems to end up providing a NetworkCredential, and it's not clear whether it's even possible to set the Authorization header to a Bearer token with one of those. Pretty much every example out there showing how to use bearer tokens in .NET does it by setting the Headers.Authorization property on the HttpRequestMessage.
So the 'obvious' mechanisms for that would be either to use the HttpClient.DefaultRequestHeaders, or to insert an HttpMessageHandler, but it looks like the creation of the client and handler chain is all locked up inside HttpHandlerResourceV3Provider and is not open to extension.
I took a look at https://github.com/Microsoft/artifacts-credprovider too, and from that I get the impression that it doesn't actually set a Bearer token either.
At this point I dredged up a dim memory that with ADO APIs, you can also use basic auth, with anything you like as the username and a PAT as the password, so I tried this:
private class Creds : ICredentialProvider
{
    private readonly string token;

    public Creds(string token)
    {
        this.token = token;
    }

    public string Id { get; }

    public Task<CredentialResponse> GetAsync(Uri uri, IWebProxy proxy, CredentialRequestType type, string message, bool isRetry, bool nonInteractive, CancellationToken cancellationToken)
    {
        return Task.FromResult(new CredentialResponse(new NetworkCredential("x", this.token)));
    }
}

and also this:
var ps = new AsyncLazy<IEnumerable<ICredentialProvider>>(() => Task.FromResult<IEnumerable<ICredentialProvider>>(new ICredentialProvider[] { new Creds(token) }));
HttpHandlerResourceV3.CredentialService = new Lazy<ICredentialService>(() => new CredentialService(ps, true, true));

This has in fact enabled me to move forwards. But I'm left with a feeling of "Surely this isn't what we're actually expected to do?"
Details about Problem
NuGet product used: NuGet Client SDK (NuGet.Credentials, NuGet.Protocol)
Package versions: 4.9.2
OS version: Windows 10 v1809 (17763.253)
Tagging @rrelyea since he offered to help with this on Twitter

@cpyfferoen commented on Tue Jun 04 2019
@idg10 with your excellent issue post i was able to move forward on my own project. I posted my API (unpolished, but functional) as a gist
https://gist.github.com/cpyfferoen/74092a74b165e85aed5ca1d51973b9d2
THANK YOU for providing the hints I needed to get me over this hill of ridiculously sparse documentation/examples

Until the docs are written up, if you have the PAT, you can use PackageSource and PackageSourceCredentials to seed the SourceRepository instead of just the string.


implement a way that redis can establish a new connection via service discovery

At the moment Redis will attempt to reconnect to the old address when it receives a "no such host" error but we should find a way to fetch a new address via service address because it might have changed when the redis actually died. From what I can see it will require initializing a new redis client.

will be done in the other redis connection ticket


Add Icon component

Description

Implements the Icon component
Icon paths provided by Feather
Icon path file added to theme, means we can extend/adjust icons via the theme
Script to regenerate default icon paths is repeatable with yarn gen-icons

Todo

[x] In hindsight, take the theme icon's viewBox into account as 3rd party icons may have different view boxes 🤦♂

Related Issues
n/a

Decided to place icons in their own @spicy-ui/icons package rather than @spicy-ui/core


feat: ignore client error in metric query_failed

Description
Problem:

We have a high error rate, some error is type of server error (500) or client error (400).
But they are returned as Http Status 200 on Client so making affect the error rate metric on Observability dashboard in Grafana.

Solution
Ignore these kind of errors out of apollo_query_failed metric
BAD_USER_INPUT
UNAUTHENTICATED
GRAPHQL_PARSE_FAILED
GRAPHQL_VALIDATION_FAILED
BAD_REQUEST
Related Issues
Ticket: #OD-411
How Has This Been Tested?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Screenshots (if appropriate):

How about we separate to 2 kind of error metrics
One for client error and another for server error. It more makes sense

I don't think we should ignore the errors. The clients should not be making UNAUTHENTICATED or BAD_USER_INPUT, something wrong with the front end code.
Or someone is trying to attack using token scraping.

Agree


Install-OSServerPreReqs should have a -Sources parameter for offline installations

This cmdLet will be removed. We will install rabbit and erlang in the Install-OSServerPreReqs and the configuration of Rabbit will be done by the configuration tool.

Delivered on version <IP_ADDRESS>
Install-OSServerPreReqs -SourcePath ....


Created MergeSort.cpp

Issue:
Changes propsosed:
Errors:
Mentions:

Issue: ~
Changes proposed: Added Merge Sort Algorithm
Errors: None
Mentions: ~


FAQ link does not work

In the section "Oh no my stored data is not cleared after reinstalling my application. How do I resolve this 'issue'?
" in the FAQ: https://github.com/TheCocoaProject/cordova-plugin-nativestorage#faq
you are referencing a stackoverflow question: https://stackoverflow.com/questions/42538421/ionic-nativestorage-not-clear-when-reinstal-application-release
but the link resolves in a 404

Thank you for the feedback.
It seems that the question on SO has been deleted.
I will update the readme.
But short:

on Android: https://stackoverflow.com/questions/41808957/sharedpreferences-not-reset-on-reinstall


Normal text and Disabled text colors are interchanged

Gotcha, there was a css class stating color of td,th to white

Hi, is this solved? If it's, and it's what it looks like, please close this issue.


[EASI-2735] Help Center Operational Solution Review Updates

EASI-2735
Changes and Description

Address all the points listed in Natasha's review of the solution help center

How to test this change
Go through Help Center and Operational Solutions to check the points listed in Natasha's review is addressed.

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

@patrickseguraoddball i was not able to reproduce the CPI Vetting indent issue you mentioned:


Fixing typo in Finnish translation

Changing "kopio linkki" to "kopioi linkki"

please pull from staging instead of new-design


Support skipping Figma nodes

Figma UI nodes that use "!skip" as the first part of the name will
not be included in generated code. Any child nodes are also ignored.
Figma UI nodes that start with "//" will not be included in generated node, nor will any of its child nodes.
Figma UI nodes that start with "//" will not be rendered by the FigmaSharpApp.

Fixes #342

Is this ready to merge? :)

Yeah it can be merged if @netonjm is happy with it.


Issue with k8s.io/docs/tasks/configure-pod-container/configure-liveness-readiness-probes/

This is a...

[X] Feature Request
[ ] Bug Report

Problem:
In the chapter define-readiness-probes, the given example is misleading : people could understand that the readiness is only used when a container start and not for the rest of its life cycle (see issue 11018 for instance, happened to me as well).
Proposed Solution:
It would be nice to improve the explanation by stating more clearly that the readiness probe is effectively used during all the container life cycle.
For instance, something like: "Readiness probe are run on the container during all its life cycle to determine if the parent pod is ready to receive traffic or not."
Before : "A pod with containers reporting that they are not ready does not receive traffic through Kubernetes Services"
Maybe add some more exemple, like an app in a container depending on an external service (DB, Storage, ...) to illustrate the need of the readiness probe in a different scenario than startup.
For example, if the DB goes wild (network temporary issue/latency for instance), the readiness probe could detect that and allow the service to prevent traffic from hitting the related pod until things go back to normal.
Page to Update:
https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-probes/#define-readiness-probes

Created PR regarding this issue : PR 11613


Changing from duplicate to extend displays or vice versa breaks fullscreen on windows 10

Doesn't matter the initial screen mode. Also happens on framework. Input on keyboard still works when focused.
https://user-images.githubusercontent.com/35318437/108006088-56bb2f80-6faf-11eb-954b-db6638b01260.mp4
Ref: https://osu.ppy.sh/comments/1461677

Does the window appear on your second display, or is it completely non-existent everywhere?

The window is non-existent everywhere.

Can't seem to reproduce anymore.


Include successful dependencies only on failed request

Describe your environment.

SDK version: 2.12.0
.NET runtime version (.NET or .NET Core, TargetFramework in the .csproj file): .NET Framework, v4.8
Hosting Info (IIS/Azure WebApps/etc): ASP.NET MVC, IIS Express
Platform and OS version: Windows 10 1909

What are you trying to achieve?
85%+ of my submitted events and ingested data are SQL dependencies, and I haven't even enabled SQL command logging. While I can disable all non-failing SQL dependencies using an ITelemetryProcessor, this means I lose the context of multiple SQL calls during a single failing request.
I think this can be summarised as:

A failed dependency should submit
A success dependency should submit only if the request as a whole failed (e.g. a later dependency failed)
The operation as a whole should follow normal sampling logic.

Documentation explains how to filter at an individual item, and how to influence sampling, however it does not explain how to filter in the context of a request.
I understand this will skew my results, however I cannot afford to pay for 10x data ingestion just to have context for rare events!
Additional context/attempted solutions
I can suppress successful dependencies using https://stackoverflow.com/a/41807317/3175307
altered to dependency?.Type == "SQL" && dependency?.Success == true.
However, as far as I can tell I can't get the context of whether the parent request succeeded or not.
I experimented with ITelemetryInitializer,SamplingPercentage, ExcludeTypes and IncludeTypes, however this is at the wrong scale as I don't want to affect sampling.
The closest solution I found was this article https://blog.ramondeklein.nl/2017/05/05/filtering-application-insights/ where the author manually tracks requests to provide this. However, I can't find any documentation on how the ASP.NET library orders events, so I'm not sure I can safely assume RequestTelemetry will come before DependencyTelemetry.

@tboby I understand your issue. We don't have a feature inbuilt in application insights SDK to track the complete flow when one of the dependencies fail. Implementing Telemetry processor is a good way to customize for this scenario.
Normal ASP.NET project, receives request first and then makes backend call. If your application model is based on it, RequestTelemetry gets initiated before DependencyTelemetry and ends after DependencyTelemetry to track the complete request duration.

I think I have the same issue. Basically every time there is a failed request I want to disable sampling for that failed request and include all dependency, trace, and exception information for that specific request.
As far as I understand, this doesn't seem to be possible at the moment, correct?

I think I have the same issue. Basically every time there is a failed request I want to disable sampling for that failed request and include all dependency, trace, and exception information for that specific request.
As far as I understand, this doesn't seem to be possible at the moment, correct?

Yes its not possible.
The flow is something like this:
Request starts (RequestTelemetry instance created, success/failure = unknown)
make dependency call - (DependencyTelemetry created)
dependency succeeds or fails. (dependencyTelemetry.Sucess is set accordingly)
dependencytelemetry is "Tracked" - sampler is applied to dependencyTelemetry. It doesn't know if the Parent Request is success/failure, so no decision can be made on that. The current sampler makes decision purely based on operationid.
(other telemetry like trace, etc are tracked and has same fate as dependencytelemetry)
Request ends (RequestTelemetry.success/failure is set)
requesttelemetrty is "Tracked" - sampler is applied - at best sampler can decide to retain the RequestTelemetry, but the "child" telemetry's decision was already made and retaining parent RequestTelemetry alone will lead to broken traces.
In short, not possible with any builtin sampler.

Closing this. Let us know if there are more question.


Errors with VM and VLAN already configured

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Errors just after the setup, because my vm has a vlan affected.
VM is blocked.
It's like a saturation.
Cilium Version
1.15.3
Kernel Version
5.14.21
Kubernetes Version
1.28.8
Regression
No response
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[X] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I'm sorry, but to make progress here you'd need to actually provide any kind of information / data that describes the error.
Closing for now.


Wrong matching with [AntPathMatcher] for path exclude

We have a project with the following path exclude
paths:
    - pattern: "{gradlew,gradlew.bat}"
      reason: "BUILD_TOOL_OF"
      comment: "Gradle build files"

However it seems this pattern excludes the build.gradle in the project.
After debugging, I came to evaluate this bit of code in ORT Reporter:
AntPathMatcher().apply { setCaseSensitive(true) }.match("{gradlew,gradlew.bat}", "build.gradle")

and to my surprise it evaluates to  true.
As anyone an idea what happens here ?
It does not seem to conform to what's in Spring Javadoc 1.
As a sidenote, should ORT not use 2 instead for glob matching ?

Thinking about it maybe this is not a bug. Maybe this expression is just not supported.
Please advise.

As a sidenote, should ORT not use 2 instead for glob matching ?

Unfortunately, that standard implementation behaves oddly in some edge cases, i.e. it does not match files that we expect to be matched for certain patterns.

As anyone an idea what happens here ?

Probably @fviernau can advise.

@nnobelis @sschuberth , I wonder if the AntPathMatcher syntax is actually the best choice.
When I have some time I'd make some research what other implementations are available, so that we could switch
to something else. Do you agree at least rearching it makes sense?

Do you agree at least searching it makes sense?

Definitively. Spring also as a replacement for AntPathMatcher if I am not mistaken:
https://www.baeldung.com/spring-5-mvc-url-matching

When I have some time I'd make some research what other implementations are available, so that we could switch
to something else. Do you agree at least rearching it makes sense?

Yes, doing some research does not hurt, and I did that myself some time ago as I was considering to write a Kotlin-only (and multi-platform) glob matcher because the Kotlin stdlib unfortunately does not provide one (nor could I find a decent one, except for this outdated library). An article I found helpful was https://research.swtch.com/glob as it gives some nice overview of the approach used in different programming languages and their performance.

JGit also has an (internal) class to convert .gitignore-style globs to Java Regexes that we maybe could use.


Development: Update about us page

Checklist
General

[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

The features mentioned on Artemis are outdated and do not include all available features. We have integrated the functions of Artemis (from Github) into the "About us" page.
Description

Added features
Translated into German
Made it reusable and expandable

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to the About us page (/about)
Check the features -> Github
Check if something is missing (or mistranslated)

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Screenshots

German

English

@b-fein Thank you for your suggestions!


feat: packagemanifest api

would unblock the package-loader and packagemanifest controller.
Signed-off-by: Yashvardhan Kukreja<EMAIL_ADDRESS>

Please also include a /apis/manifests/manifest.go file to aggregate all versions (Check /apis/core/core.go as example) and add this new API group to /apis/apis.go

Regarding this Nico, I had to explicitly remove it because, at least to what I noticed in the core/core.go file, the apis.go file was just aggregating the AddToScheme of all the "Groups" (core, manifests, etc.).
But in the case of manifests.package-operator.run, we are not exporting its scheme. I did that because we don't intend to hook the PackageManifest type as a GVK/resource to the Kubernetes API at anytime, it's just a Kubenetes-compliant format to represent a Package's info. Therefore, due to the absence of Scheme/SchemeBuilder/AddToScheme methods, there was nothing to put in the apis/apis.go and manifests/manifest.go, therefore, I had to get rid of them.
Would love to hear your points :)


Netapp Cluster View Dashboard for NABox 3

Looking to replicate the Netapp Cluster view dashboard from the old version before I migrate.  Same view as the first screenshot on https://nabox.org/.  This helps me keep an eye on all our Netapp clusters.

hi @outrebor1st have you checked out the current NetApp Detail: Cluster dashboard, (pasted below)?
It should have the same information as the Harvest 1.6 screenshot you pasted. The top section is configured a bit different to show more customer-requested data about model, serial, health, etc, but the graphs and throughput, latency, IOPS tables are all there.
If there's something missing or you'd like to suggest a change, the more specific, the better.

I've been working with this view for a few days but it doesn't provide the same results that I had previously.  This gives cluster overviews - but I need node level breakdowns as I have a cluster with 10 nodes with various levels of disks (SAS, FSAS, and SSD) and I'd like to monitor, as well as multiple locations/datacenters that I am looking for issues in.

hi @outrebor1st
which version of Harvest are you using?
Does the node level breakdown shown in the [NetApp Detail: Node - Details] dashboard have what you need? Both dashboards have variables at the top that help filter the contents and in 22.05 we introduced an import switch to grafana import that modifies all dashboards as they are imported to add multi-select drop-downs for each variable.


Symlink lts/X.yaml

Minor bumps are typically very safe and don't require application changes to accommodate. If we provided a symlink at (e.g.) lts/17.yaml to lts/17/15.yaml and kept it pointed to the latest minor version of that major, applications could use that and receive updates implicitly. We could fully automate our biweekly bump, as long as the major version is not changing. Major version changes, which typically require application changes, would still go through the manual process, and establish a new lts/(X + 1).yaml symlink.
The downside is of course when a particular minor version bump is not totally smooth. Users may get a compilation error seemingly out of nowhere whenever a particular environment decides to pull the new file.
This would also impact checksums, but I'm not sure it's in a meaningfully positive or negative way. Presently, we let Stack write the checksum to stack.yaml.lock for us, whatever it is. This means we'll see in the diff when it changes. If we went forward with this proposal, users could be surprised to see that random diff noise. We could also decide some day to use an explicit checksum. In that case, this proposal would cause the build to error and the new checksum need to be accepted when the symlink contents shift.

You've convinced me that symlinks break atomicity of change, so I'm against this plan. No spooky action please :pray:

Love to argue against myself :+1:


Merge PEP 660 implementation into main

The idea of this PR is just to mark the implementation as "complete enough" / "tested enough" that it can be merged into main.
The main concepts of the implementation were presented in https://github.com/pypa/setuptools/pull/3265.
The development of this feature was split into several PRs previously submitted.
Anyone interested in checking these smaller PRs can have a look on the following searches:

https://github.com/pypa/setuptools/pulls?q=is%3Apr+660+author%3Aabravalheri
https://github.com/abravalheri/setuptools/pulls?q=is%3Apr+editable

Several discussions and feedback were collected in:

https://discuss.python.org/t/help-testing-pep-660-support-in-setuptools/16904/39
https://github.com/pypa/setuptools/pull/3265
https://discuss.python.org/t/pep-660-and-setuptools/14855
https://discuss.python.org/t/how-to-implement-namespace-packages-as-in-pep-420-via-import-hooks/15192

Hopefully the development was slow enough and the community engagement was broad enough that there were enough chances of involvement for feedback, design changes, etc... 🤞
Instructions for testing the feature before the PR is merged can be found in https://discuss.python.org/t/help-testing-pep-660-support-in-setuptools/16904.
Summary of changes

Added editable hooks according with PEP 660
Added a "strict" editable installation mode
Added a "compat" editable installation mode (this mode is transitional and will be removed in future versions of setuptools - my personal suggestion is to remove this mode after the end of the year).
Added mechanisms for custom build sub-commands to tap into the editable install mode and do the necessary adaptions.
Updated userguide/development_mode docs.
Changed build_py and build_ext to add a mechanism of getting a list of files being packages and to which original files they correspond to.
Improved handling of --build-option and --global-option in the config_settings argument of the build_meta hooks (to match the historic pip behaviour)

this is, at least in theory, backwards compatible and only warn the user when a given --global-option should actually be a --build-option, but accept it anyway.

Closes #2816
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d/.
(See documentation for details)

While I was hoping to see this release an year ago, even later is better than never. Thanks for taking the time to do it! I eagerly waiting for the moment I would be able to make use of it in production.

🎉  A million thanks @abravalheri!


libcec_platform: 2.0.1 -> <IP_ADDRESS>

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found <IP_ADDRESS> with grep in /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>
directory tree listing: https://gist.github.com/be00d099a69bdff53bd6e138cf5d6877

cc @titanous for review

No attempt on x86_64-darwin
The following builds were skipped because they don't evaluate on x86_64-darwin: libcec_platform
No partial log is available.

Success on aarch64-linux (full log)
Attempted: libcec_platform
Partial log (click to expand)
-- Installing: /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>/include/p8-platform/util/util.h
-- Installing: /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>/lib/pkgconfig/p8-platform.pc
-- Installing: /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>/lib/p8-platform/p8-platform-config.cmake
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>/lib
patching script interpreter paths in /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>
checking for references to /build in /nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>...
/nix/store/kpb5k4dzycnxm8w0b1r9valmvjpkbr9g-p8-platform-<IP_ADDRESS>

Success on x86_64-linux (full log)
Attempted: libcec_platform
Partial log (click to expand)
-- Installing: /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>/include/p8-platform/util/util.h
-- Installing: /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>/lib/pkgconfig/p8-platform.pc
-- Installing: /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>/lib/p8-platform/p8-platform-config.cmake
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>/lib
patching script interpreter paths in /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>
checking for references to /build in /nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>...
/nix/store/klb1bvsf1az785f2jzvsg5g0sxs8g4l3-p8-platform-<IP_ADDRESS>


accessibility Help

is there any docs/example how to UI test my swipe actions? Thank you

There is no docs describing UI Testing. You can programmatically initiate the swipe, and the buttons have all the appropriate default accessibility labels/api in place to also invoke those actions programmatically as well.


New algorithm for generating Clifford circuits for single qubit.

Summary
We generate all 24 transpiled Cliffords once (in the basis {rz, sx}. Then, for every rb_circuit, select Cliffords at random and compose them to a circuit
Details and comments
Current version is faster than the original version. The main change I made since the previous version, is adding the parameter inplace to the QuantumCircuit.compose() operation. This uses the existing circuit, rather than copying to a new circuit.
For benchmarking, I ran the test test.randomized_benchmarking.test_randomized_benchmarking.TestStandardRB.test_single_qubit from the test suite. Here are a few measurements (average of 5 runs):
With num_samples = 1,   existing rb_experiment: time = 1.2374 sec
this branch:                   time = 0.7710 sec
With num_samples = 10 existing rb_experiment: time = 8.3614 sec
this branch:                   time = 1.2056 sec

There is a difference between the EPC computed by this branch as compared to the existing rb_experiment. This must be investigated.

@dekel.meirom ran a comparison between the RB version in this PR and QC code on Bangkok. The experiment consists of  circuits with lengths = [1, 2, 3, 5, 9, 17, 31, 56, 99, 177, 316, 562, 999, 1778, 3162].  QC code takes 83 sec while this version takes 100 sec


Dataset points mutability

Hello. I am not pretty sure that it is good idea to use MutableList in Dataset. My point is that we could use private val points: List<DataPoint> instead. The reason of my suggestion in fact that it is not good when everybody knows that the list inside of Dataset is mutable.

Thanks for the suggestion. Let me think about it for a bit and I'll come back to you.
I can see how the library right now does not gain anything by having a mutable list,. However in the future, for example a new feature could be added where the LiveChart is notified to redraw when there is a small change in its list of DataPoints. similar to a List view.


Fix the styling of footer component.

Hey @DhairyaBahl, the text input field along with the color combination in the footer looks really dull. Please, assign me to fix these issues.

@ankita-04 If you have better color combination than these then I will definitely merge them.
All the best 🙂🔥


docs(figma notes): add figma and design notes

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: # (issue, if applicable)
Summary:
This PR adds the blueprint reference and release note docs to the repo.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


wich version of xtermjs ?

I've still have this problem :
https://github.com/xtermjs/xterm.js/issues/992
This bug is correct in xtermjs now, but actually webssh use an old xterm version ?
wich version webssh use ?

4.2.0

ok i've ugrade to 4.9.0, still the same problem, in fact it was the font,i change it and it's working.


[stable/jenkins] Jenkins jobs specified in master.jobs are not created

Describe the bug
Starting with chart version 1.5.4, Jenkins jobs as specified in master.jobs are not being created.
Version of Helm and Kubernetes:
Helm: 2.14.1
Kubernetes: v1.12.8-gke.10
Which chart:
stable/jenkins
What happened:
Prior versions of this chart allowed job XML to be specified as master.jobs in values.yaml, but starting with version 1.5.4 jobs are no longer being created.
master.enableXmlConfig was added as configuration value, but even when set to true jobs are not being created.
What you expected to happen:
I expected jobs to be created.
How to reproduce it (as minimally and precisely as possible):
Install chart version 1.5.4 (or higher?) with master.jobs specified in values.yaml

By default, the Jobs Map is only used to set the initial jobs the first time
that the chart is installed.  Setting overwriteJobs to true will overwrite
the jenkins jobs configuration with the contents of Jobs every time the pod starts.
overwriteJobs: false

Have you tried setting master.overwriteJobs to true? Just pay attention that this is what you want as it overwrites Jenkins job configurations.

Thanks for your reply. The behavior on chart version 1.5.3 is that jobs are created regardless of whether overwriteJobs is set to true or not. Even if overwriteJobs is required from 1.5.4 and onward, it seems like a defect to change default behavior without mentioning it in the documentation.
I just confirmed that jobs are not being created on 1.5.4 even when overwriteJobs is set to true.

However, I've deployed 1.7.2 and confirmed that jobs are being created properly, so I assume the defect was resolved at some point between 1.5.4 and 1.7.2. Since it has been fixed in the latest release, it's probably not worth figuring out why it broke in the first place.
Thanks.


Profile Page - Fix Select Image

Problem
The Edit profile image option is not working.. Ie. it doesn't save the users image and display it on the profile page.
Solution
Please allow a user to select a photo, save it and make sure that it is displayed directly on the profile page. This should be connected to firebase as well, so the image must be uploaded to firebase storage and the users' document should link that url under the 'image' field.
Additional context
Please get this done by Tuesday
Code of Conduct

[X] I agree to follow this project's Code of Conduct and Contributing Guidelines

Decided to remove the image from the entire project, so no need to do this.


[$2000] Distance - Missing "Finish" button and "Add Stop" button does not response

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to staging.new.expensify.com
Go to 'Request Money' > 'Distance'
Tap "Start"" button and add address
Tap "Add Stop" button

Expected Result:
When user navigate to Distance tap there should be "Start" and "Finish" button.
When tap on "Add Stop" button the new routes waypoint editor should be open
Actual Result:
"Finish" button is missing and "Add Stop" button does not response when tap on it
This issue is reproduced on iPhone 6S: 750 x 1334
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.57-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/77accb33-868c-4660-8ea3-601768ae0e63
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0162c51e788145a4b3
Upwork Job ID:<PHONE_NUMBER>166036992
Last Price Increase: 2023-09-04

Heh - why was @kevinksullivan assigned here?

increased to $2000

yes that's correct let me grab the issue that fixed this
but yes we can close this out

this issue. PR was merged and the issue should be fixed now


Job event woes

Shuffled job-socket around so we're no longer inheriting from EventEmitter
Fixed bug with rerunning a View that would cause points from separate jobs to get merged.

@go-oleg @davidvgalbraith

Why don't we want JobSocket to inherit from EventEmitter?

@go-oleg fixed the race

@davidvgalbraith see comments from oleg and demmer: https://github.com/juttle/outrigger/pull/57


Reason for using ref object?

I was wondering, why did you make the Url a ref object, instead of a normal stack allocated object like nim's Uri? Also, do you plan on updating puppy with the latest urlly? Nimble seems to complain if i try using the latest urlly with puppy, even though it's specifying >= 0.2.0, not sure why.

Because it's faster. It seems like stack objects that have strings or seqs are slower because they end up coping the strings and sequences too much. By making Url a ref object it saves a ton of work. My soft rule is that if object contains a seq or a string it should be ref object for speed. But again profiling beats all rules, your use case might be different.
What kind of error do you get? Thats news to me. I use #head version of both?

Gotcha. But declaring it as a normal object would give people the choice to use either, no? I could still declare ref Url as my type. Never understood why nim lets you bake the "refness" into the type definition itself.
Regarding the error, i get this if i add urlly >= 1.0.0 to my .nimble file:
 Installing urlly@>= 1.0.0
Downloading https://github.com/treeform/urlly using git
  Verifying dependencies for<EMAIL_ADDRESS> Installing<EMAIL_ADDRESS>   Success: urlly installed successfully.
       Tip: 39 messages have been suppressed, use --verbose to show them.
     Error: Cannot satisfy the dependency on urlly 1.0.0 and urlly 0.2.0

But all of the internal code would have to use object. It would make no sense to give you nonref Url but accept ref Url every where in the API. And I don't want to use nonref Url in the API because it's probably slower. I think Nim does ref handing correctly. I like the use of ref objects, sorry.
Your second bug sounds like a nimble issue. If you re-order your imports in the nimble file and maybe uninstall your currently installed Urlly it should work. Its strange: https://discord.com/channels/371759389889003530/753721959308853319/792549349590302730

Fair enough i'll look into that, thanks.


Use official mongo driver + origin quering

This is WIP and not ready for merge.

WOW you're the best @kwando !!


Add Addresses in Bulk

Hi, it would be nice to be able to add thousands of addresses separately per line.
and of course avoid duplicates.
It would also be good to use the hot wallet that is already configured in bitcoin, to generate the USDT TRC20 addresses.

Hi,
Thanks for the feedback.

The bulk capacity was added.

For the address generation based on the configured hot wallet, I don't think it is a good solution for product at scale, for security and compatibility (the btc xpriv derivation path is different). I've just opened the discussion with the BtcPayServer core team. Keep you posted.


how to get response_type of token call back url parsed?

Hi guys,
I've noticed the way to get access token, which is to set response_type to token, will get back a url like /callback#access_token=6874dd3766b35536abcb76a9e3a57867&expires_in=86400,
Could you please share some ideas to parse this url with anchor? According to this article http://www.mikeduncan.com/named-anchors-are-not-sent/ , I think server side will never receive params after anchor "#". Any ideas? Thanks

Sorry, just realize it's not the proper way to do this, I am closing it.


Credits

Hi, I'm the owner of Free Domains and an administrator of Open Domains, I see you have copied our CLI and register repo code.
We would appreciate if you would provide credit in the README of both the CLI and register repositories.

Hi, I'm the owner of Free Domains and an administrator of Open Domains, I see you have copied our CLI and register repo code.
We would appreciate if you would provide credit in the README of both the CLI and register repositories.

Done you can check.

Hi, I'm the owner of Free Domains and an administrator of Open Domains, I see you have copied our CLI and register repo code.
We would appreciate if you would provide credit in the README of both the CLI and register repositories.

Done you can check.

Thank you for your quick cooperation, have a great day :)


Update types for arguments in docs

LGTM, ty :+1:
Any reason for draft?

Yeah actually, I thought that maybe I will find more places where the docs do not match. If not, I will mark it as ready for review.

Ahhh okay, nice


[Bug]: RichTextEditor no value when copy patse

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When I copy any content to RichTextEditor then I read content of RichTextEditor by RichTextEditor.text but no value
Steps To Reproduce

Copy content
Paste content to RichTextEditor
Read content of RichTextEditor but no value

Environment
Release
Version
Cloud

@NguyenMinhDung this seems to be working for me. Can you please share a video?

Here video to reproduce the bug
https://user-images.githubusercontent.com/2491855/149336661-2a6cff5e-cb94-4add-afbc-eee957cfe16a.mp4
Below video is another bug, when I clear content of RichTextEditor but the content is still there
https://user-images.githubusercontent.com/2491855/149336731-4e3d2c72-0f0e-4381-8563-2fb1ea1c49b1.mp4

@NguyenMinhDung thank you for reporting this. It seems to happen inside a modal and in some other cases


Fix Compilation Error Structure

Changes in Check.java File

There are some conflicts that must be resolved before merging your pull request

Welcome to the AutoCompiler community.