[DO NOT MERGE] Bumped Mesos to ci/gilbert/fuse.

High-level description
What features does this change enable? What bugs does this change fix?
Corresponding DC/OS tickets (required)

DCOS-ID JIRA title / short description.

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

Looks like this is not needed anymore for now. Closing. Please leave a comment or ping someone on Slack if this should be re-opened!


clean up MoistAir functions

The subpackage BS/Utilities/MoistAirFunctions:
https://github.com/UdK-VPT/BuildingSystems/tree/master/BuildingSystems/Utilities/MoistAirFunctions
has some duplication with Annex60/Utilities/Psychrometrics:
https://github.com/iea-annex60/modelica-annex60/tree/master/Annex60/Utilities/Psychrometrics
and possibly also with Annex60/Media:
https://github.com/iea-annex60/modelica-annex60/tree/master/Annex60/Media
We should check our package to see what functions are missing in Annex60.

related: https://github.com/iea-annex60/modelica-annex60/issues/475


Include RAlgebra version of para

I've been studying a series of blog posts on recursion schemes.  The third in the series covers para.  It shows a different formulation of para using an RAlgebra instead of a GAlgebra like matryoshka's para.  I find both versions of para are useful in different situations.  Here's one example of where the RAlgebra version from the blog post results in less boilerplate:
data ExprF a
  = Num Number
  | Var String
  | Add (Array a)
  | Sub a a
  | Mul (Array a)
  | Div a a

-- using GAlgebra para
removeAddZero :: Expr -> Maybe Expr
removeAddZero = para removeAddZeroF
  where
    removeAddZeroF :: ExprF (Tuple Expr (Maybe Expr)) -> Maybe Expr
    removeAddZeroF (Add args) = do
      let
        args' = (foldl (\acc x ->
          case snd x of
            Just y -> case unroll y of
              Num 0.0 -> acc
              _ -> acc <> [y]
            Nothing -> acc
          ) [] args)
      case args' of
        [] -> Nothing
        [x] -> Just x
        xs -> Just (roll $ Add xs)
    removeAddZeroF (Num x) = Just $ num x
    removeAddZeroF (Var x) = Just $ var x
    removeAddZeroF (Sub x y) = Just $ sub (fst x) (fst y)
    removeAddZeroF (Div x y) = Just $ div (fst x) (fst y)
    removeAddZeroF (Mul args) = Just $ mul $ map fst args

type RAlgebra f a = Mu f -> f a -> a

para' :: forall f a. Functor f => RAlgebra f a -> Mu f -> a
para' alg t = unroll t # map (para'' alg) # alg t

-- using RAlgebra para'
removeAddZero' :: Expr -> Maybe Expr
removeAddZero' = para' removeAddZeroF'
  where
    removeAddZeroF' :: RAlgebra' ExprF (Maybe Expr) -- Expr -> ExprF (Maybe Expr) -> Maybe Expr
    removeAddZeroF' _ (Add args) = do
      let
        args' = (foldl (\acc x ->
          case x of
            Just y -> case unroll y of
              Num 0.0 -> acc
              _ -> acc <> [y]
            Nothing -> acc
          ) [] args)
      case args' of
        [] -> Nothing
        [x] -> Just x
        xs -> Just (roll $ Add xs)
    removeAddZeroF' x _ = Just x

I was wondering if you'd be willing to include para' and RAlgebra as part of matryoshka?

Hello! RAlgebra is the same thing as GAlgebra
type RAlgebra f a = Mu f -> f a -> a
-uncurring->
Tuple (Mu f) (f a) -> a
-abstracting over Mu ->
Corecursive t f =>Tuple t (f a) -> a
-use type synonim ->
Corecursive t f => GAlgebra((Tuple t), f a)

BTW you can use the same "catch all case" with GAlgebra
-- using GAlgebra para
removeAddZero :: Expr -> Maybe Expr
removeAddZero = para removeAddZeroF
  where
    removeAddZeroF :: ExprF (Tuple Expr (Maybe Expr)) -> Maybe Expr
    removeAddZeroF (Add args) = do
      let
        args' = (foldl (\acc x ->
          case snd x of
            Just y -> case unroll y of
              Num 0.0 -> acc
              _ -> acc <> [y]
            Nothing -> acc
          ) [] args)
      case args' of
        [] -> Nothing
        [x] -> Just x
        xs -> Just (roll $ Add xs)
    removeAddZeroF fx = Just $ embed fx

@cryogenian thank you for sharing this.  I was struggling to figure out the "catch all case" with GAlgebra.  That's super cool that RAlgebra can be written using GAlgebra.


Jupyter variables panel appears when using notebook without a Jupyter kernel attached

Open https://insiders.vscode.dev/github.com/bamurtaugh/python-tuples/blob/614d461702fe41eb06db69165df4774dea4e1b28/Tuples.ipynb
Select Connect to a running Jupyter server for kernel
:bug: in the output panel, Jupyter option appears, but there are no variables

IMO this panel should either be hidden if the notebook hasn't been executed yet, or should show a welcome action saying "the active notebook has not been executed, run a cell to see variables" with an action button to "run all"

closing in favor of https://github.com/microsoft/vscode/issues/165445


7.16.0 does not include the default icons

All icons with the prefix "default" are missing from 7.16.0.

Hi,
thanks for filing issue, but I'm not sure what you mean that icons with prefix "default" are missing.
Can you explain it a little bit more, please ?
This repository is only chrome extension which displays vscode-icons on GH repos. So, it is possible that you wanted to create issue for vscode-icons

@dderevjanik Ahh sorry i think got the wrong repo. Sorry about that.


VS Code runs stale persisted launch configs

VS Code 1.46

start VS Code Stable
use this launch config:{
   "type": "node",
   "request": "launch",
   "name": "echo args",
   "runtimeExecutable": "echo",
   "args": [ "hello", "world" ]
},

close "launch.json" editor
run it
Observe: (everything is good)

shutdown VS Code
open same project with VS Code Insiders (or any other editor)
modify "launch.json" and include argument "1234" between "hello" and "world"
shutdown VS Code Insiders
launch VS Code Stable again
go to debug view and run "echo args" launch config
open launch.json
Observe:

VS Code runs old launch config which lacks the "1234".
Workaround: an edit change of launch.json is needed before VS Code will run the correct launch config.

We no longer store the configuration in storage. The only downside is that now when you select a dynamic configuration, reload, we do not preserve the dynamic configuratoin and you have to select it again. For that we would have to store the "origin" as you suggested, but that I will tackle as a seperate issue, thus reopening https://github.com/microsoft/vscode/issues/96293

I've verified that no stale launch configs are run.


Mobile Wallet ETA?

Are you actually working on a mobile Wallet as planned?

@Vefdec not right now, I'm waiting for information about new project direction, but it's planned and will be released as soon as other more important stuff is done.


Python: Add FT.SEARCH command

Issue link
This Pull Request is linked to issue (URL): https://github.com/valkey-io/valkey-glide/issues/2429
Checklist
Before submitting the PR make sure the following are checked:

[x] This Pull Request is related to one issue.
[ ] Commit message has a detailed description of what changed and why.
[ ] Tests are added or updated.
[ ] CHANGELOG.md and documentation files are updated.
[ ] Destination branch is correct - main or release
[ ] Commits will be squashed upon merging.

Add changelog please

Add changelog please

CHANGELOG.md updated.


Coverage

Part of #2

Codecov Report

Merging #12 (584948d) into master (8bf49c7) will increase coverage by 0.36%.
The diff coverage is 86.13%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   84.42%   84.79%   +0.36%
==========================================
  Files           3        3
  Lines         122      217      +95
==========================================
+ Hits          103      184      +81
- Misses         19       33      +14

Impacted Files
Coverage Δ

include/compare.h
100.00% <ø> (ø)

src/main.cpp
83.11% <76.00%> (-4.82%)
:arrow_down:

src/compare.cpp
85.50% <89.47%> (+4.86%)
:arrow_up:

Continue to review full report at Codecov.

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


XML Job definition issue

JQM Job definition integration doesn't trim XML tag content, leading to NoClassDefFounError on  property when indentation is moving closing XML tag on the next line.

It's actually a feature, not a bug (I always wanted to write this, sorry) at least for most fields. You are free to put whatever text, including carriage returns, in your strings, so we do not trim anything.
Now, it is true that we have applied the same logics to fields that obviously cannot have spaces or new lines. The following fields will be trimmed in next release:

name of the job definition (string)
name of the default queue
path to jar file
path type


custom ETag function isn't working for SendFile

I set the custom function according to the http://expressjs.com/en/api.html#etag.options.table
app.set('etag', function (body, encoding) {
  return generateHash(body, encoding) // consider the function is defined
})

Despite on that it's not working and the best way I could influence etag value is to call
sendFile with etag : true or false but this just turns on and off the standard etag (also not documented) , not the custom one.
https://github.com/expressjs/express/issues/2294
https://github.com/expressjs/express/issues/2129
My use case is that I want to track responses of specific files. So I wanted to make a "strong hash" that I can be 100% sure would  unambiguously represent the file.

Hi @kokushkin thank you for bringing this up. Yes, the current etag setting does not affect files, as calculating an etag for a file is noy always trivial. For example, a file can be multiple gigabytes in size and it is not practical to hold the entire contents of a file in memory in order to invoke the etag setting.
You can currently calcualte your etag and set the etag header using res.header and it will not be replaced by the framework.
We can determine what kind of API would be needed otherwise in order to assist in calculating the etag for a file. What do yoi currently do to calculate the etag of these files, may I ask?

Well, I've solved my problem by just calculated it's sha256 digest using "crypto" and writing the result into the log. But initially I tried to use "morgan" together with the standard ETag functionality of "express". I think I even tried to make it explicitly with res.header and it didn't work out (morgan was enable to access etag for some reason) but I'm not sure, I might try  it again and give you feedback). So, the initial thought was to hook into the process and say I wanna get absolutely unique digest for some types of files, let's say *.jpg files no matter where they are in the routes and I couldn't do it so I have to explore the routes.

This could be solved w/ something like:
res.sendfile(yourPath, { headers: { etag: 'your-etag' } })
Not sure how would you calculate etags in a file stream, what could the "chunk" arg be if not "data" from the filestream.


Theme reverts to default when page is reopened

Not sure if it's a feature request or issue or a config issue
Steps

Visit https://shibuya.lepture.com/
Change the theme to something that is not default (eg: if you are on light mode, change to dark mode and vice-versa)
Close the page
Reopen the page and notice that the theme is reverted (not the one you selected)

Thanks

Not sure if this is a conscious decision you made but I think the following should be stored in localStorage not sessionStorage?
https://github.com/lepture/shibuya/blob/c4e869220964de5bef857d26fd41d39783d6d066/static/js/theme.js#L12

Fixed in master code.


Ability to split into temp and switch

Please explain your feature request in a one or two sentences.

The ability to split the device into an input (useful to control other stuff based on the temp) or if on/off.

Is your feature request related to a problem? Please describe.

I can't turn the switch off (or perform other actions based on the temp)

Any particular eWeLink devices that this relates to?

Sonoff HT16

Anything else?

Hi @jacklawry
I don't believe it is possible to create HomeKit automations in the Home app based on temperature(/humidity).
This can be done via the Eve HomeKit app (not sure if you use this already) but it allows you to create more complex automations, based on temperature, based on the different services of an accessory (ie controlling the switch of a temperature sensor + switch combination accessory).
Is this what you're looking to do?

Thanks for your reply, I had tried to use Eve previously but the guides I found were quite old and didn't really apply to the current app. However, just tried again myself and I think I have got it working.

If your automations are based on a single temperature point you can use the settings to expose the device in homekit as a heater or cooler.
eg if you want to turn ON if it drops below a certain temperature or OFF otherwise, you can expose as a heater.
if you want to turn OFF when it drops below a certain temperature or ON otherwise, you can use cooler.


Improves the documentation for nuget pack -action

I was really struggling to get the nuget package versioning to work as I wanted - After a lot of trial and error, I got the process to work, and used my experiences to try and make the documentation a bit more comprehensive.
Hope it's useful!

@koskila This is great work and the contribution is greatly appreciated! There's just one small update but I'll keep an eye on it and merge it ASAP :)

@koskila : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The updates in the pull request appear to be in master, so we'll close the PR.


Documentation says "trailing slash" instead of "trailing backslash"

Here are two extracts of the page, where it says "trailing slash":

$(OutDir) | Path to the output file directory. If it's a relative path, output files go to this path appended to the project directory. This path should have a trailing slash.

$(IntDir) | Path to the directory specified for intermediate files. If it's a relative path, intermediate files go to this path appended to the project directory. This path should have a trailing slash.

Instead of "trailing slash", it should be "trailing backslash".
Reminder, as I see some confusion online: slash is / (for url), backslash is \ (for Windows paths).

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

ID: b46cf831-faea-6357-0e06-3284f114c879
Version Independent ID: 0345affe-ce33-c438-d3e4-d134f905d084
Content: Common macros for MSBuild commands and properties
Content Source: docs/build/reference/common-macros-for-build-commands-and-properties.md
Service: visual-cpp
Sub-service: tools
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

@marckruzik , good point. I'll fix that today. It'll be live by Monday afternoon.


Token erc721 bug fix

Closes #49 Closes #47

Verified the changes @techpandora
I think we can also remove the import statement for the ERC721 contract @vishaldantuluri


Can't install from Podfile

Error: Unable to find a specification for 'SwiftMQTT'

Please add to README "How-to install ..."


Update Caption Maker to use modern styling and coding standards

CaptionMaker is back! Thanks @molant!  https://dev.microsoftedge.com/demos/captionmaker/
While it still works fine, it would be great to update it to fit in with the rest of the site, and any improvements we can think of.
I can take a look, but probably need @melanierichards’ help with any design/UX changes, and @molant if we want to remove jQuery.

@ststimac FYI, I'll take care of this demo

@melanierichards sounds good!


fixed typo in upgrade doc example for volatile mode changes

Fixed typo between v1 and v2 code examples in documentation (chainer.Variable(x) -> chainer.Variable(x_data))

jenkins, test this please

LGTM!
Thank you very much!


ERROR:root:Property type Edm.Byte is not supported by mutator yet

How to resolve this?

We have to add support OData V4 to SAP/python-pyodata.

Please, ignore my comment. I have no idea why I thought we need OData V4.

I am marking this as duplicate of  #61


Discord server link in readme file is invalid

I love the idea of this project, and after messing around with it for a while I decided to join the discord... only to find that the link is invalid. Any chance you could update the link?

Hi! The story behind it is rather complicated, but this server does not exist anymore :(.
Feel free to text me on Discord though, Palaiologos#5585 - I'd be glad to hear from you.


Improve the quality of annotations

Signed-off-by: yuxiaobo<EMAIL_ADDRESS>
What this PR does / why we need it:
Improve the quality of annotations.
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

/assign @bacongobbler


fix some code smell

Coverage decreased (-4.6%) to 62.097% when pulling c833681bee8065b596cb58d376b3c041022488df on RobertLara:master into 7e88b269144f5138a5820c0d89c6c261434ab99f on sachin-handiekar:master.


Tweaks

Here are some tweaks, mostly through IntelliJ IDEA's inspection capability.

Some of these changes are syntactic sugar but they are making the source code dependent on Java SE >=  version 1.8 (I believe that this has already happened elsewhere in the code base, so this comment is not 100% directed at this PR).  Is this an intentional, considered choice for this project, or has it just kinda happened?
I am not opposed to this (1.7 got EOL'ed in 2015), but it would be helpful if there were a clear statement about version requirements on the project page, a vision about version tracking and backward compatibility, and tags to identify when a dependency to a new version is introduced.  I think this would help all contributors.

@opeongo commented on Aug 15, 2018, 12:50 AM GMT+2:

Some of these changes are syntactic sugar but they are making the source code dependent on Java SE >=  version 1.8 (I believe that this has already happened elsewhere in the code base, so this comment is not 100% directed at this PR).  Is this an intentional, considered choice for this project, or has it just kinda happened?
I am not opposed to this (1.7 got EOL'ed in 2015), but it would be helpful if there were a clear statement about version requirements on the project page, a vision about version tracking and backward compatibility, and tags to identify when a dependency to a new version is introduced.  I think this would help all contributors.

It was mentioned in #81 in connection to version change that could be version 8 to reflect that requirement so I took it as granted.

making the source code dependent on Java SE >= version 1.8

@opeongo Yes minimum 1.8 as per the build script
https://github.com/beanshell/beanshell/blob/0ad109fb38beec022adc88035d931f9a71137f8f/pom.xml#L56
There should be closed issues that references this decision... it happened a long time ago.

@wilx thanks for the link to #81.  Makes sense to me now.

@wilx Just a reminder please take note from my initial review comment:

Your timing is slightly off though. =( I've been working on a fix for #444 which is going to touch a major part of the source base.
I will give you the go ahead once I'm done.

Whatever you are doing now will have to be done over so best give me chance to get this patch in before you go on. It should not take very much longer now... about the length of a piece of string =)

@nickl- commented on 16 Aug 2018, 08:47 CEST:

@wilx Just a reminder please take note from my initial review comment:

Your timing is slightly off though. =( I've been working on a fix for #444 which is going to touch a major part of the source base.
I will give you the go ahead once I'm done.

Whatever you are doing now will have to be done over so best give me chance to get this patch in before you go on. It should not take very much longer now... about the length of a piece of string =)

OK.


Toon health bar appearance not correct. Bar not showing correct health.

Description

Health number indicator is correct but the bar is not updating correctly. Sometimes it is working ok but I have to use the healbot addon to see my accurate health indicator.
I did a screen shot showing healbot (top left) and the "normal" UI in the middle.
Version Number

v1.09 enUS (WoW Classic)
Steps to Reproduce

Happens randomly. Not able to reproduce on purpose.
Expected Behavior

Display correctly
Actual Behavior

Only KkthnxUI Enabled

I will try that, if another option isnt available.
Additional Information

Error Log

Happens when I loose health. And when I heal myself, as I am a pali, it doesnt update anything but the number. The bar doesnt move.

I also noticed the target info is glitchy as well. i will damage to 50% and the bar will sometimes still show 100% health

hmmmmm, it might be that your UI is not compatible with healbot addon. I hope that I dont have to go back to my other UI. I have grown to like yours so much :-(

yep... if I disable the healbot, which I use to easily heal parties with my healers. weird that healbot doesnt mess with the UI items that are being affected, and hb's health bars are 100% correct. odd

I am trying out another healing addon, in case we cant fix the issue with healbot. So if you come up with a fix then that is great, because healbot is a great addon. If I cant find a suitable healing addon that will work with the UI, I will be forced to go back to my old UI I was using. :-(

Wrong Repo.


container/runtime plumbing

currently there is the husk of a container runner that talks to docker. the groundwork is done, but now we need the config yaml files, the structs to represent them and read them in. and supply all the data needed to start and manage containers/vm's/workloads.

I will take lead on this. let me know if anyone else wants to help out and I can chunk some of it out to them.


README Issues section has broken link

The "file an issue" url is going to defunct url:
https://github.com/ssbarnea/pytest-molecule/issues
instead of:
https://github.com/pycontribs/pytest-molecule/issues

These two pages have the broken links in their "file an issue" urls:

https://pypi.org/project/pytest-molecule/
https://github.com/pycontribs/pytest-molecule

@jacklh9 Thanks. PRs are more than welcomed!


Favicon Not Visible in Browser Tab or Search Bar..

https://github.com/user-attachments/assets/34185dc0-d8e7-425f-bd47-b590d7d23918

yess

please assign me with the task @aslams2020

i want to add ironman face there

@piyush9420707813 Add favicons in those pages, wherever it is absent.

@piyush9420707813 it is a minor issue to get you started. it's been 3days.
let me know until tomorrow or i have to close the issue due to time took to complete

Closed due to inactivity and no response


When on 5.2 LTS should we see upgrade notices for non-LTS 6.0 version

Issue Summary
5.2 was meant to be a LTS version, so I am not expecting to see an upgrade notice for 6.0, but only for 5.2.x
Steps to Reproduce

Have a Wagtail 5.2.3 LTS site
Wagtail 6.0 upgrade notice shows on dashboard

Is there a way to say in settings.py "I am sticking to the LTS for a while" and not see 6.0 upgrade notices?
Technical details

Python 3.11
Django 4.2 LTS
Wagtai 5.2.3
Chrome 121

Working on this

Anyone can contribute to this. View our contributing guidelines, add a comment to the issue once you’re ready to start.

It's not really a bug, but I am not sure if anyone has ever asked this. 5.2 was meant to be an LTS version and I currently can't move to 6.0 yet, but I don't want customers to constantly see the upgrade notice for 6.0.
When on an LTS version I would expect it to only show 5.2.x releases.

See https://docs.wagtail.org/en/stable/reference/settings.html#wagtail-enable-update-check - you should set WAGTAIL_ENABLE_UPDATE_CHECK = "lts".


Prep for npm publish.

In prepping for publishing to npm I'll be lowering the version to 0.1.0 since this hasn't really been hammered on in JS land yet.

can't publish as docdown, someone else has the name

Bikeshed time!

doc-dynasty, now your turn. go!

docaround


Hide the poster and controls on fist tap on mobile devices

This should help answer your question on https://github.com/videojs/video.js/issues/2914
This uses the parent player object to hide the poster and controls when the user first taps to play the video on mobile devices. This prevents the API from calling play on the YouTube video for the fist time and reveals the YouTube video for the user to press play on. This will prevent the video from breaking on mobile devices.
I also reversed your solution to hide the poster and play button. I feel it would be a better UX to have the poster and have the user press play twice than to not show the poster at all.

Tapping twice is not an acceptable option to me. It seems like a broken behavior to me.

Because IOS does not allow API initialization of video playback, the user must interact with the YouTube video directly to initiate the first play. Removing the poster from these instances makes for an inconsistent user experiences across different devices. As a content author or marketer, the poster image may hold more value than the video behind it. By forcefully removing it you don't give the author the option to show or hide it. Yes, tapping twice isn't desirable, but makes the visual experience consistent.

Tapping twice is going to be inconsistent with how a video should behave on mobile. By adding a poster over it, you're not fixing the problem but just hiding it behind a click. The YouTube poster will be shown, which is a better option to me than a confusing behavior that a lot of people will not understand.


Question about the input fluid model

Hi,
I meet a question how to get or generate the inputfile of the bgeo fluid model.
I had searched "bgeo" on the Internet, and found it may be the file of houdini. Then I dowmload the newest houdini and generate a bgeo file. But the SPlisHSPlasH project read nothing form it.
Cheers

Hi,
probably I should update the partio library since it seems that it cannot read the newest format.
However, we have two tools in SPlisHSPlasH that you can use. VolumeSampling generates a volumetric sampling (as bgeo) for an OBJ. This can be used as fluid. SurfaceSampling generates a bgeo for the surface of a OBJ mesh which can be used as boundary.
Hope that helps,
Jan

Thanks for the answer!


Not able to use Kivy for Python3.6.3 in Ubuntun 16.04, 64-bit

Python: 3.6.3
OS: Ubuntu 16.04
Kivy: 1.10.0
Kivy installation method: virtualenv (pip install kivy)

Description
Running my Kivy app, it happens sometimes that I get an error:
Unable to find any valuable Window provider.
The only way I found to resolve this is to restart my computer
possibly connected to https://github.com/kivy/kivy/issues/5410
Code and Logs
Simplest kivy program, see the following gist (with the errors)
https://gist.github.com/yoelk/688e0bab56e2ed52915044e5388a4866

Thanks for the comment @MrAgeo.

From your log in the gist it looks to me like it fails to initialise the SDL2 backend (probably in the call to SDL_GL_CreateContext(), not sure).
I've found a StackOverflow question which might be the same problem. But they didn't have a real solution except changing the graphics card driver.
Can't say why it would work with py2.7 and not py3.5/py3.6, though. That's probably not related.

This seems to be issue with your system rather than Kivy, so I'm closing.


[Bug]: Timed out fetching a new connection from the connection pool

Relevant scope
API response issue
Description
Often API is unavailable
"Timed out fetching a new connection from the connection pool. More info: http://pris.ly/d/connection-pool (Current connection pool timeout: 10, connection limit: 9)"

Hello,
It might be due to issue with Heroku at that time.
I added Sentry to the API to retrieve info.


No OpKernel was registered to support Op 'Dequantize'.

Hi There,
I am trying to use your code to test up-scaling of a cropped image.   i am on anaconda python 3.5 cuda 10 cudnn 7.4 with tensorflow-gpu==1.15.0.  I am getting.  can you help to identify the exact version or procedure that i have to use?
Traceback (most recent call last):
File "main_1.py", line 73, in
run.upscaleFromPb(args.image)
File "C:\Users\ASUS\Downloads\EDSR_Tensorflow-master\EDSR_Tensorflow-master\run.py", line 298, in upscaleFromPb
output = sess.run(HR_tensor, feed_dict={LR_tensor: LR_input_})
File "C:\Users\ASUS\Anaconda3\envs\tengpu\lib\site-packages\tensorflow_core\python\client\session.py", line 956, in run
run_metadata_ptr)
File "C:\Users\ASUS\Anaconda3\envs\tengpu\lib\site-packages\tensorflow_core\python\client\session.py", line 1180, in _run
feed_dict_tensor, options, run_metadata)
File "C:\Users\ASUS\Anaconda3\envs\tengpu\lib\site-packages\tensorflow_core\python\client\session.py", line 1359, in _do_run
run_metadata)
File "C:\Users\ASUS\Anaconda3\envs\tengpu\lib\site-packages\tensorflow_core\python\client\session.py", line 1384, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'Dequantize' used by node resFilter_three (defined at C:\Users\ASUS\Anaconda3\envs\tengpu\lib\site-packages\tensorflow_core\python\framework\ops.py:1748) with these attrs: [mode="MIN_FIRST", T=DT_QUINT8]
Registered devices: [CPU, GPU]
Registered kernels:

     [[resFilter_three]]

Originally posted by @seli07 in https://github.com/Saafke/EDSR_Tensorflow/issues/1#issuecomment-652944157

Hi,
I think this is because the model was created under (I believe) tensorflow-gpu=1.12.0. I also see you use windows. Apparently this combination gives your error, see the issue here.
To work around this we can either:

Retrain the model with a higher version of tensorflow. Not recommended as it takes a couple days of training on one powerful GPU.

Use the unquantized version. Unfortunately, I don't have those files any more. So this would also require retraining.

Use OpenCV for the inference part. I believe this should work with windows. You can follow this tutorial. Helpful: see this tutorial for how to install OpenCV 4.3.0 with contrib modules on windows.

Thank you. It worked with the opencv + contrib build process.

Does someone, by any chance, have non-quantizes pb files?


Draggable scrollable bounce back

Case if user wants just bottom sheet with open/closed state. (Youtube description on web for example)
No migrations needs, just optional parameter
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

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

pr in master []_[]

@Renesanse All PRs should be opened against master. Looks like changing the base branch back to master is causing issues with this PR, though. Could you please create a new PR against master with your proposed changes? In the meantime I am going to close this one to make sure we don't accidentally merge it into the dev branch.


How do I generate dynamic and static libraries from OpenSSL 1.1.1?

The wiki page: Compilation and Installation | Configure Options.
describe that's possible to use the option shared to build OpenSSL and as results both shared and static libraries being built.

Note: when building a shared object, both the static archive and shared objects are built. You do not need to do anything special to build both when shared is specified.

Windows 64 / VC 2017
perl Configure VC-WIN64A shared --prefix=C:\Build-OpenSSL-VC-64
make
make test
make install

Is it still valid?  See Compile OpenSSL 1.1.1 with VS 2017 Statically or Dynamically include removed macros
I've checked the results and makes me doubt if this option is still valid, since in the file  openssl/opensslconf.h I can see the macro OPENSSL_NO_STATIC_ENGINE.

Please see :
https://github.com/openssl/openssl/blob/3d362f190306b62a17aa2fd475b2bc8b3faa8142/NOTES.WIN#L112
Engines are optional for various purpose, but not essential to get it link or work.

Is it still valid?

Statement that VC-whatever shared build produces both shared and static libraries? No. There is one thing to recognize, The whole thing is rather Unix-centric and Windows support is kind of make-best-of-the-situation. Formally speaking it is possible produce both libraries even on in VC builds, but it would take kind of unconventional approach. On both sides. I mean user would have to link with distinctly named library in case link with static one is desired. [For example when embedding OpenSSL into another dll.]


test/components/*: de-duplicate values

timeout and retryInterval were the same for most of the components and was
duplicated.
closes: #502
Signed-off-by: knrt10<EMAIL_ADDRESS>

There are 2 different values for timeout, 1 is 7min and other is 9 min. What should be the naming convention?
TimeoutSlow and TimeoutSlower?

There are 2 different values for timeout, 1 is 7min and other is 9 min. What should be the naming convention?

7 minutes can either be left as-is (as there is no duplication) or just increased to 9 minutes.

Rebase needed.


NPM Publish Allow specifying tags or lack of for latest

Currently anything getting published to an private internal feed will get the "latest" tag creating a problem when allowing for prerelease and release versions.
My current setup has an internal feed which local repos publish to, and a second feed with an upstream to the internal feed and to npmjs, if I use the upstream internal@local then latest will usually be a prerelease, internal@release and I can't install prerelease, using both internal@prerelease and internal@release then I have to specify exact versions to avoid prerelease versions.
I think this would be solved by allowing us to set the tag/s on the task.

As a workaround, you could set the build variable/environment variable npm_config_tag to the desired value before running the publish.
Keep in mind that internal@release and internal@prerelease refer to views, not tags. Packages of any package type need to be promoted into those views, npm packages are not added to the views just by being tagged.

I revisited this issue today, and you can't set environment variables against the npm task.
I set npm_config_tag as a pipeline variable and it was ignored, I read that variables set in the pipeline can be used by subsequent tasks, so I set it in a command line task before the publish and it was still ignored.
I added publishConfig: { tag: "prerelease" } to the package.json and tried it again, I see it listed in the log under "; environment configs" with "tag: "prerelease"" but an npm install still picked up the prerelease tagged package as the latest.
I'm not sure if I'm going about it the wrong way, but it feels like the task ignores any config you provide and sticks latest on everything?

I removed the NPM task from the equation and just used the NPM authenticate task and manually set the package feed in .npmrc running the npm publish --tag=prerelease in command line afterwards and it -still- ends up being tagged as latest despite explicitly stating a different tag.
So this isn't specifically a problem with the NPM task, but a fundamental problem with NPM feeds in Azure DevOps, unless I'm doing something wrong? I would think tagging packages is a fairly commonplace thing, so I'm inclined to think I'm doing something wrong.

It looks like you're doing it right, it works for me. If this is the first time the package is getting pushed, it will be tagged with both latest and the tag, and all subsequent versions with just the tag.
c:\>npm publish --tag=prerelease
+<EMAIL_ADDRESS>c:\>npm dist-tag ls npmt7
foo: 10.8.13
latest: 10.8.12
prerelease: 10.8.14

Which version of node/npm are you using?

I was running the publish from the Hosted Ubuntu agent (https://github.com/Microsoft/azure-pipelines-image-generation/blob/master/images/linux/Ubuntu1604-README.md) so it has Nodejs v10.15.1, not sure which version of npm it has.
I tried npm dist-tag ls packagename and it only ever listed latest, even npm dist-tag add packagename prerelease didn't seem to do anything, npm dist-tag ls packagename would still only say latest.
Trying to publish from my own machine appears to work fine. This is bizarre, would you be able to check a publish via the hosted ubuntu agent with your own setup? The same command that works from my own machine doesn't appear to be tagging correctly via the release process on VSO.

Can you email me (jotaylo at microsoft dot com) with the name of your organization (dev.azure.com/organization) and the feed name/package name. I'd like to look at the logs to see if there's something unexpected going on.

@Lokiem @jotaylo was this resolved?

@alexmullans The original request for specifying tags on an npm publish is still a valid feature request.
My specific issue landed on situational issue between multiple feeds, tags between an upstream feed and a source feed are lost and everything is marked as latest. So it's somewhat of a bug / a known technical debt during implementation of feeds in Azure DevOps. There isn't a clean solution to my specific issue as discussed in this, best workaround is to publish directly to the source feed which will keep any tags intact, though hardly ideal.

@jotaylo any news on this?
I have connected the npm service under Service Connections but I'm not able to publish a npm package to npm public registry with a tag like this:
task: Npm@1
inputs:
command: 'custom'
workingDir: 'lib/dist'
verbose: true
customCommand: 'publish --tag alpha'
customRegistry: 'useNpmrc'
publishEndpoint: 'NpmJs'
It seems that this is working just if the command is 'publish' instead of 'custom', but nothing is tagged and the package is going to be published as latest.
I think you should include an option for the inputs to tag a version when the command is 'publish'.

@Lokiem i was facing the same issue. what i wound up with in my debugging was this:
- task: PowerShell@2
  displayName: 'Publish'
  inputs:
    targetType: 'inline'
    script: |
      # https://github.com/microsoft/azure-pipelines-tasks/issues/9743
      if("$(buildtag)" -ne "") {
          write-host "publishing using tag [$(buildtag)]"
          npm publish --tag "$(buildtag)"
      }else {
          write-host "publishing without a tag"
          npm publish
      }
    failOnStderr: false
    pwsh: true

this was not working which lead me to this thread. upon seeing the comment by @jotaylo, i noticed that the only real difference was the missing = in the tag assignment. i just added that and it seems to be working. going to have my front end devs validate and then i'll report back but in my quick test i was seemingly able to pull a package by the tag.
seemingly working:
- task: PowerShell@2
  displayName: 'Publish'
  inputs:
    targetType: 'inline'
    script: |
      # https://github.com/microsoft/azure-pipelines-tasks/issues/9743
      if("$(buildtag)" -ne "") {
          write-host "publishing using tag [$(buildtag)]"
          npm publish --tag="$(buildtag)"
      }else {
          write-host "publishing without a tag"
          npm publish
      }
    failOnStderr: false
    pwsh: true

My specific issue landed on situational issue between multiple feeds, tags between an upstream feed and a source feed are lost and everything is marked as latest. So it's somewhat of a bug / a known technical debt during implementation of feeds in Azure DevOps.

@Lokiem Where did you get this info? We're experiencing the same thing in our company and it's really bugging our process here... This thread is the only place where I can find something that looks alike our issue.
Currently it seems the only way to move forward is by directly pushing to our "merge feed" that merges everything we develop ourselves and the packages coming from npmjs....

we are experiencing the same issue and have also reported this at https://developercommunity.visualstudio.com/content/problem/783921/published-npm-package-in-upstream-internal-feed-lo.html

Yes, views/upstreams don't support dist-tags. The most recently added package will be tagged as 'latest', but no other dist-tags are maintained.
This is partially due to a limitation with our current implementation of views - we would need a redesign to be able to support the additional metadata required for dist-tag support at view level. We originally discussed simply passing through dist-tags to views, however that lead to all sorts of issues. A dist-tag could disappear from a view if the dist-tag was moved to a version that isn't in the view. If a feed has multiple upstream sources, how should different versions with the same dist-tag be resolved. We've had some discussions on these issues, but do not plan any changes in the short term.

@jotaylo How are we supposed to views appropriately then? No matter which view an npm package sits in, it's always the 'latest' package that gets installed, no matter which permissions are set in the views.

This issue is not fixed, it still not possible to add tag.
even after npm publish --tag=prerelease mypackage, the command npm dist-tag mypackage still return only the tag "latest".
So it's currently impossible with azure artifact do to something like npm i mypackage@beta

+1 for this feature. We also need to add tags to our packages in our feed, such as stable, dev, next, etc. This would make our publishing process much easier and consistent with other teams. Right now, we have to use workarounds to distinguish our releases. Please add this feature soon. Thanks.

+1 for this feature in 2024. I've searched a lot for this feature in Azure DevOps but still haven't found out if it has been released or not, so I assume it has not been released.

+1 for this, for any bigger public package correct handling of latest tag is must.

+1 for this feature. Same issue also opened a feature request with Azure Artifacts team.

+1 for this feature, This feature is essential.


Add Server Sent Events

Jobs should send SSE's through the API so the UI can keep up real-time.

https://www.npmjs.com/package/express-sse


expose enum variant at runtime

I have an enum for which I need to get the specific variant at runtime. I added a static field to the classes with the identifier but noted it was missing after deserialization, which from the test below I guess is the expected behaviour?
https://github.com/dao-xyz/borsh-ts/blob/a91fa5ef947d9ca71d0b8ba2d87e15f9b9ffbf30/src/__tests__/index.test.ts#L152-L169
Using the enum variant would work too, however I don't see an easy way to access it without looking into the bytes manually. Is there a workaround for this or any way I can access the variant at runtime?

Try
import { getDiscriminator } from '@dao-xyz/borsh'
it will yield all discriminator bytes before the class

static fields are not analyzed, so that is expected behaviour

that worked, thank you!


Image url not correct in HMR

Laravel Mix Version: 0.12.1 (npm list --depth=0)
Node Version (node -v): v7.10.0
NPM Version (npm -v): 5.0.0
OS: OSX

Description:
In HMR, using  in the .vue file can't load image in browser. the image url is /images/logo.png?[hash].
I think image url must be http://localhost:8080/images/logo.png?[hash] in HMR.
Steps To Reproduce:

app.js

window.Vue = require('vue');

Vue.component('example', require('./components/Example.vue'));

const app = new Vue({
    el: '#app'
});

Example.vue

<template>

</template>

Add logo.png to directory

run npm run hot

Result

It is correct. You used a relative path, so the image url will be handled by url-loader.

@ruchern Thanks for help, I know it will be handled by url-loader
but image will be processed by webpack, so image file in the webpack dev server, is it?
image didn't export to public/images directory, so browser can't get image.

@CasperLaiTW If you want the image to be in public/images you need to manually configure copy-webpack-plugin to copy the image into public/images

If I use npm run production it's all working. image will be processed to public/images/ automatic
if I remove publicPath from loader or setup resourceRoot to be //localhost:8080/ - it's working in HMR.
https://github.com/JeffreyWay/laravel-mix/blob/v0.12.1/setup/webpack.config.js#L179
so my workaround is
if (mix.config.options.hmr) {
    mix.setResourceRoot('//localhost:8080/');
}

So I am not sure it's bug or something wrong in the config.
In my point, I think I didn't change any setting or hard code in the HMR or Production mode.

In HMR mode, new images are served in webpack dev server, but the generated urls point to Laravel server. I think the solution from @CasperLaiTW should be integrated into laravel-mix or at least be documented.

How is this not documented yet? Thanks @CasperLaiTW , Anyhoo in the latest laravel-mix/webpack, i had remove the .options to get it to work. In case anyone wondering...
if (mix.config.hmr) {
    mix.setResourceRoot('//localhost:8080/');
}

I have spent far too long wondering why my images were not being copied across in HMR.
This fix worked even in the latest mix, thanks!


Increase test coverage

Right now our code coverage is somewhere around 15%. :zap:

Well, that's not good but I there are some methods in which I see no sense of covering them with unit tests. What's more, some methods are't totally clear for me because of their dependencie, so I skip them as I cn't write propert tests for them 😭

I was thinking this morning about integrating Travis-CI. What'd you say @czechboy0?

They still don't have Xcode 7 support.

Their sluggish new Xcode version support is the whole reason Buildasaur exists :laughing:

But yeah once Travis has Xcode 7 support, I'd be up for using it for testing and coverage. No idea how long that will take however.

In the meantime we have Xcode 7's code coverage metric, so run it locally and report it in each PR, so that we know what we're at.

There's no school like the old school... 😆

I've seen test coverage for XcodeServer but all of them have DEV_ prefix so they're not rally doing anything at the moment. @czechboy0 is this on purpose?

Yes, the DEV_ prefix is there on purpose.  So they need to be disabled most of the time unless you're testing live API calls.
And yes, DVR is ready. Just run carthage update --no-build and you're all set. I also added a folder just for Cassettes in the test target/folder.

Hmm, any possibility you write steps needed to use DVR? I how now idea where to put my hands on 😞

Look at test DEV_testLive_FetchAndRecordBot() in XcodeServerTests.
Create a test like that for whatever you want to test. Then you run it once and it records and crashes, printing out where it has saved the cassette. You copy that file into the test target in the Cassettes folder, add it to the project to both test targets only and then run it again. This time it will run fine, using the loaded cassette.

Great, but it's easy for GETs, so how about POSTs? Do I have to manually remove added entities?

With #48 we get 25% of test coverage for XcodeServerSDK... Slowly getting more and more 👍

Have we checked asking for CircleCI access instead of Travis maybe?

Isn't CirlceCI paid?

Yeah I tried them all yesterday. They don't yet support Xcode 7 :( Classic.

Travis today has confirmed official support from 6.4...

Link? But that doesn't help anyway. We need Xcode 7. Hopefully I'll be able to release an experimental build of Buildasaur soon locally on my Mac to get PR testing again.

We just added support for Xcode 6.4 and the iOS 8.4 SDK! Add `osx_image: xcode6.4` to your .travis.yml to begin using it.— Travis CI (@travisci) czerwiec 30, 2015

CircleCI is free if we use it only for 1 job at a time (which i think would be fine) except there's no Xcode 7 love yet.
how about Bitrise ? (just checked, they support up to 6.4)

I guess we'll have to wait for anything to support Xcode 7... Other possibility - someone has to allow us to access OS X Server 5.

We can use Ship.io; they have Xcode 7 support already https://ship.io/xcode-7-beta-is-here/
They have a free plan for individual accounts with 2 jobs or something along those lines; might be worth looking at it

👌

Just FYI, I do have an Xcode Bot running locally which tests the swift-2 branch after every single commit, so no worries there. I turned on email notifications so you'll also get them if something breaks.
Unfortunately, we need Pull Request testing, which Buildasaur has (I'm working on Xcode 7 support there) and CI services like Travis have, which don't yet support Xcode 7. It's exactly what happened last year. Ship.io doesn't give us any benefit over a local Xcode Bot. Plus, their Scheme detection is broken, I tried it yesterday.
So once Buildasaur has an Xcode 7 version ready I'll turn it on for this repo so that all PRs will get tested.

:clap: that sounds great then :tada:
TBH i didn't like Ship.io when I tried it.
If I can help with bringing Xcode 7 support to Buildasaur just let me know

Am I the only one seeing weird behavior with code coverage?
Yesterday I added a test that specifically triggers the invalid scheme path of code but this is what i'm seeing after running that tests:

Is marked as never called

That's really strange... Was it covering the failable path yesterday? Looks fine to me...
Nothing has changed in those files AFAIK.

It wasn't covered and that's why I went and wrote a test for it but didn't check. now I came to check and is still uncovered.
No matter if I run only that test or all the tests on iOS or OSX that path keeps being marked as not checked.
Perhaps the throws thing has something to do with it?

Disregard my last comment… this invalidates my theory
guard let url = NSURL(string: host) else {
            /*******************************************************************
             **   Had to be added to silence the compiler ¯\_(ツ)_/¯
             **   Radar: http://openradar.me/21514477
             **   Reply: https://twitter.com/jckarter/status/613491369311535104
             ******************************************************************/
            self.host = ""; self.user = nil; self.password = nil

            throw ConfigurationErrors.InvalidHostProvided(host)
        }

Just FYI, current state of things:

That's weird; if you check at the tests and the coverage XcodeServerConfig should be near 100% but the coverage is not working as expected

It might also have bugs. We'll see throughout the summer. We know we're still not testing most of the parsing/dictionarifying logic anyway :) I guess once we're confident about that we can tackle these weirdnesses.

Hmmm this whole test coverage works fine for me everytime but bugs are possible...

@czechboy0, @esttorhe I wonder...
Who's the first one to strat writing tests for API calls? 😜

All the tools are there. Here is an example of a working test and how easily you get a recording Server. And here is how it works.

And here is how you can reuse a recorded Cassette to add many parsing tests.

Did you just point at me? 😜

:point_left: :point_right: :point_left: :point_right: :point_down: :point_up: :point_down: :point_up:
Not pointing at anyone, just :dancer:

On thing I've noticed, you use:
let config = try! XcodeServerConfig(
            host: "https://<IP_ADDRESS>",
            user: "ICanCreateBots",
            password: "superSecr3t")

For sake of consistency, should we use default XcodeServerConfig or pass ours? I think the first option will do better as all tests will have the same XcodeServerConfig but in this case all of us will have to create ICanCreateBots user with superSecr3t password.

Depends how your XCS is setup, but I have that anyone can view bots and only logged in users can create bots. Which means that any user, including the non-existent ICanCreateBots will return all read-only requests. So you don't need to create this user at all to test most get requests.
When it comes to post requests, I haven't decided yet how we're going to test that since in post requests it's what we're sending is what we want to test, whereas with get requests it's what you're receiving that you want to test.
TL;DR - enable any user to read bots on your XCS and let's write tests for all the get requests first. I'll think of a good way to test posts later. Aight?

I can't agree with you... While testing POST requests it's necessary to assert request but response is also important!

@czechboy0 strange thing happened... I've written test for getRepository() which looks like:
func testGetRepositories() {
        let expectation = self.expectationWithDescription("Get Repositories")
        let server = self.getRecordingXcodeServer("get_repositories")

        server.getRepositories() { (repositories, error) in
            XCTAssertNil(error, "Error should be nil")
            XCTAssertNotNil(repositories, "Repositories shouldn't be nil")

            if let repos = repositories {
                XCTAssertEqual(repos.count, 2, "There should be two repositories available")

                for (index, repo) in repos.enumerate() {
                    XCTAssertEqual(repo.name, "Test\(index + 1)")
                }
            }

            expectation.fulfill()
        }

        self.waitForExpectationsWithTimeout(10) { error in
            if let error = error {
                print("Timeout error: \(error.localizedDescription)")
            }
        }
    }

So at first I had to move get_repositories.json and re-run tests - fine, works and even test is passing but when I look at code coverage I'm like WTF, bro!?:

I can agree that the fragment where completion with nil is passed may not be executed but [Repository] parsing?

@esttorhe seems like I've been having similiar kind of issue with guard statements...

Yeah; I saw your tweet to Joe; seems like radar time!! :dancers:

New follower...

Things just got serious! 😁

:fireworks:

Getting around 50% now. Not great but getting much better, especially thanks to @cojoj!

This is nice, we had fun with it, but it's not really actionable. Closing.


FR: Provide option for collapsing the HTML display in notebooks

Issue description
The overly long output of the text repr of xarray always bugged so I was very happy that the recently implemented html repr collapsed the data part, and equally sad to see that 0.16.0 reverted that, IMHO, correct design implementation back, presumably to align it with the text repr.
Suggested solution
As the opinions will vary on what a good repr should do, similar to existing xarray.set_options I would like to have an option that let's me control if the data part (and maybe other parts?) appear in a collapsed fashion for the html repr.
Additional questions

Is it worth considering this as well for the text repr? Or is that harder to implement?

Any guidance on

which files need to change
potential pitfalls

would be welcome. I'm happy to work on this, as I seem to be the only one not liking the current implementation.

Related: #4182


Add 1.1.1 issue template and input manifests

Description
Add 1.1.1 issue template and input manifests.
This is an improvement of the now closed #1100.
Issues Resolved
Related to #870
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1275 (450c964) into main (f83f53c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1275   +/-   ##
=========================================
  Coverage     94.09%   94.09%
  Complexity       11       11
=========================================
  Files           127      127
  Lines          2863     2863
  Branches         10       10
=========================================
  Hits           2694     2694
  Misses          159      159
  Partials         10       10

Continue to review full report at Codecov.

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


MG & MIS Review

Hi Yiding,
Please take a look at my Module Guide and Module Interface Specification.
List the files:

MG
MIS

Okay head-up its actually ready for review now

Sorry for the delay - Your document was so well written  that I struggle to find problems in it. Not an excuse though, being late is being late :)


Localize the shell extension's menu item

This commit introduces localization for the "Open in Windows Terminal"
menu item and differentiates it based on compile-time branding (rather
than runtime detection!).
@leonMSFT's tray icon pull request had the excellent idea to use the
TerminalApp's resource compartment for auxiliary resources for projects
that can't otherwise be localized the same way. Doing localization in
the shell extension (or WindowsTerminal.exe) would require us to use
MUIRCT and split the build process up to support mui files. That's a
huge amount of work... but this is not a huge amount of work.
Fixes #6112

@msftbot merge this in 1 minute


httping: 2.5 -> 2.9

Description of changes
https://github.com/folkertvanheusden/HTTPing/releases/tag/v2.9

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 199557 run on x86_64-linux 1

1 package built:

httping


Got error ，when i try to create datasets via kaggle api

First , I run this " kaggle datasets init -p D:\kaggle\PredictFutureSales\data"
then I run this  "kaggle datasets create -p D:\kaggle\PredictFutureSales\data"
and I got this error :
'file' object has no attribute 'readable'
Upload unsuccessful: test.csv
Dataset creation error: Please specify at least one file.
btw , I have give the test.csv all permission

What version of the API are you using and what does your metadata file look like?

I use the Kaggle API <IP_ADDRESS> ，and  matadata file is :
{
"title": "My Awesome Dataset",
"subtitle": "My awesomer subtitle",
"description": "My awesomest description",
"id": "changtaoyin/my-awesome-dataset",
"licenses": [{"name": "CC0-1.0"}],
"resources": [
{
"path": "test.csv",
"description": "This is my awesome data!"
}
],
"keywords": [
"beginner",
"tutorial"
]
}

Someone help me？

It's possible something is going wrong with relative paths (although I thought I had fixed all of those issues).  Try cd D:\kaggle\PredictFutureSales\data and running the command from there.  Make sure that test.csv is in the same directory as dataset-metadata.json.

Are you sure that the file/data is present in the path that you have provided as the meaning of the error is, It required at least one data set to process.

I have the same problem, my Kaggle version is also Kaggle API <IP_ADDRESS>

I already 'cd' to the folder where both metadata and data file are in


Reimplement LivenessCheck and ReadinessCheck as plugins

Description
I noticed, while working on a plugin that required to extend the ReadinessCheck, that this check is implemented not through plugins but hardcoded as an RPC method during initialization.
It would be much cleaner if these checks can be implemented as plugins, as in the future if someone needs to do something similar, it would be easier and faster to do it.

LivenessCheck is a trivial probe to check if process is UP or DOWN
ReadinessCheck requires 2 checks:

Make sure that the sync is within a certain distance from the head of the chain. For this, a numMaxBlocksBehind parameter is used to check for distance. This check could be done in the plugin version by registering an BesuEvent for addSyncStatusListener(SyncStatusListener).
Using a parameter minPeers check that the number of peers connected is higher than that value. In the plugin version, I can't see a way to have access to the number of peers but this could easily be exposed by P2PService.

Acceptance Criteria
Readinees and Liveness checks are re-implemented without side effects. Both probes will be functioning the same as before but now implemented as plugins.

Should this be implemented inside the plugins directory, with a new sub-directory called health-checks, or can they go inside the existing rocksdb directory?

IMO, we should go with a sub-directory inside plugins. health should be enough as a name.


LL: Make it possible to show more than one version of a Labware

As a Labware Library user, I should be able to view more than 1 version of a labware.
Acceptance criteria

[ ] If more than one version of a labware exists, it should be possible to tell each version apart in the Labware Library

Design

TBD

Open Questions

Should we show each labware version as a separate card or add a version dropdown to the existing card?

Stale


bufferRangeForScopeAtCursor counts tabs as spaces

I tried to call editor.bufferRangeForScopeAtCursor(".string") to get range of currently selected string. Then I tried to get that string by calling editor.getTextInBufferRange(range). It works as expected and I get entire string if line is indented with spaces.
If line is indented with tabs, returned string starts several characters to the right, it is the right length (or cropped if there are no more characters). bufferRangeForScopeAtCursor seems to be counting single tab as multiple characters (based on its width), while getTextInBufferRange counts tab as only one character.
For example, if line starts with 2 tabs (4 chars wide) immediately followed by string, start column of range returned from bufferRangeForScopeAtCursor is 8 (or undefined, see below). Since getTextInBufferRange counts tab as one character, it returns string starting with eighth character which is already 6 letters inside of string I was trying to get.
Let's assume this code:
range = editor.bufferRangeForScopeAtCursor(".string")
editor.getTextInBufferRange(range)

and this input:
\t\t"my string", some other text

If cursor is placed before "r", bufferRangeForScopeAtCursor returns undefined. That is because editor.getCursorBufferPosition (used internally by bufferRangeForScopeAtCursor) counts tab as one character and therefore it returns column < 8, which as far as bufferRangeForScopeAtPosition (also used internally by bufferRangeForScopeAtCursor) is concerned is not within any string.
If cursor is placed after "r", it returns "ring", some".
When you look at tokenizedLines in TokenizedBuffer for that line, you see that tags start [4,4,...] (4 may be replaced by actual tab width) and as far as I can currently tell this seems to be the problem. I assume that either tokenizer should not count tabs as multiple characters or bufferRangeForScopeAtPosition should not take tags as buffer column numbers.

Thanks for the detailed report, @zboro!
@nathansobo @maxbrunsfeld Do you perhaps want to take a look at this one? I was able to reproduce this

Yeah, the issue here is the difference between 'screen' coordinates and 'buffer' coordinates. The buffer coordinates pretty much map directly to the characters in the underlying file. In particular, tabs are just characters; they are not handled specially. In screen coordinates, however (e.g. getTextInScreenRange), tabs are sized according to the editor.tabLength setting.

Ok, I completely misunderstood your comment at first @zboro. I see that bug that you're talking about. bufferRangeForScopeAtCursor returns an incorrect range. Its columns are screen columns, not buffer columns.

I also just realized that this is a duplicate of https://github.com/atom/atom/issues/8685. @zboro Please subscribe to that issue for updates. Thanks again for the report!


Scan locally built and tagged image w/o pulling from ECR

Hello,
Qq, is it possible to scan a local image that was built, tagged, and pushed without the need to pull it back down from ECR? This is my config for the trivy scan:
    - name: Run Trivy vulnerability scanner
      uses: aquasecurity/trivy-action@master
      with:
        image-ref: "***.dkr.ecr.us-****-*.amazonaws.com/xp-trivy:trivy-testing-${{ env.GIT_SHORT_SHA }}"
        format: table
        vuln-type: 'os,library'
        severity: 'CRITICAL,HIGH'
        timeout: 30m

I've been running into this error that can be solved by pulling the image back down before running the trivy scan but just wondering if this is really necessary since the image-ref exists locally on the runner:
2023-11-03T16:05:54.090Z	FATAL	image scan error: scan error: unable to initialize a scanner: unable to initialize a docker scanner: 4 errors occurred:
	* unable to inspect the image (***.dkr.ecr.us-****-*.amazonaws.com/xp-trivy:trivy-testing-f32****): Error response from daemon: no such image: ***.dkr.ecr.us-****-*.amazonaws.com/xp-trivy:trivy-testing-f32****: No such image: ***.dkr.ecr.us-****-*.amazonaws.com/xp-trivy:trivy-testing-f32****
	* containerd socket not found: /run/containerd/containerd.sock
	* unable to initialize Podman client: no podman socket found: stat podman/podman.sock: no such file or directory
	* GET https://***.dkr.ecr.us-****-*.amazonaws.com/v2/xp-trivy/manifests/trivy-testing-f32****: unexpected status code 401 Unauthorized: Not Authorized

Figured out what I was doing wrong.

What was the issue & fix?  I'm having the same issue with images built by GitHub actions using podman tooling & not docker.

i'm also interested in solution - ran into same issue

Okay, found it.
In your build step:
      - name: Build image
        id: build
        uses: docker/build-push-action@v5
        with:
          file: Dockerfile
          load: true
          platforms: linux/amd64
          push: false

Notice id: build, we will use it in next step:
      - name: Run Trivy vulnerability scanner
        uses: aquasecurity/trivy-action@master
        with:
          image-ref: ${{ steps.build.outputs.imageid }}
          format: table
          exit-code: 1
          ignore-unfixed: true
          vuln-type: os,library
          severity: CRITICAL,HIGH

So after building we save imageid, and use it in image-ref of trivy-action to load our image, refering via steps.build.outputs.

Hey! Sorry just saw this. Yeah, the fix for me was discovering the load argument in the docker/build-push-action.
@hoopty If the trivy gh action isn't working for you, you can also download & install Trivy straight up and scan a tarball:
      shell: bash
      run: |
        echo "Downloading Latest Trivy Version"
        curl -sfL https://raw.githubusercontent.com/aquasecurity/trivy/main/contrib/install.sh | sh -s -- -b . latest
        echo "Scanning ${{ inputs.serviceName }} service for vulnerabilities"
        tar=$(find . -name 'jib-image.tar')
        for path in ${tar[@]}
        do
        ./trivy image  --timeout 15m --format table --ignore-unfixed --vuln-type  os,library --severity  CRITICAL \
        --input "$path"
        done


Enhance: Navbar buttons

Description
Hello,
I would like to add CSS to nav bar and hover the buttons.
I would like to work on this issue under GSSOC'23.
Please assign me this issue.
Screenshots
No response
Additional information
No response

I have assigned you the task. All the best

Thank you for assigning me this issue. I will create a pr according to the requirements. Please give me some time till next week I will create the pr as I have my exams going on.

The issue is solved in pr #158.


metaphor to requirements.txt

deploying to azure app services, during setup it uses requirements.txt to load up the virtual environment.
believe that metaphor-python needs to be in there.
Or am I missing something ;^)

Metaphor is not a mandatory requirement. If you want to use it you can install it and add the keys in streamlit secrets.toml file

Metaphor is not a mandatory requirement. If you want to use it you can install it and add the keys in streamlit secrets.toml file

I am submitting a pr that removes all dependence on st.secrets and the sectrets.toml
But this issue is about the fact that if you ask rags to add the web tool, it will fail because metaphor-python is not installed in your environment and rags does not auto install it for you


Add CRF module

Conditional Random Field (CRF) is a basic model of natural language processing, which is widely used in word segmentation, named entity recognition, part-of-speech tagging and other tagging scenarios. For example, BiLSTM-CRF model is widely used in NER. Could Flax add CRF module? That will useful! Thanks

Flax currently tries to keep the framework minimal, adding only essential layers that can serve as building blocks for more complex layers and layers that are very commonly used. For everything else, we encourage users to fork our repo and create their own custom layers.


Add a few missing Window APIs back in.

What does the pull request do?
#11062 made WindowBase.PlatformImpl internal, but a couple of APIs from there weren't exposed in our public API surface, so this PR adds alternatives back in:

Added WindowBase.TryGetPlatformHandle. : WindowBase.PlatformImpl is now internal so we need to expose a different want to get the window handle
Added WindowBase.Resized event.: exposes the resize reason and new client size. Required renaming PlatformResizeReason to WindowResizeReason. Made TopLevel.HandleResized method internal.

SizeChanged events were added to other controls to follow WPF. Having a separately termed Resized event for conceptually the same thing, but with a few more details, may not be the best API. I'll point that out but expect you won't change it.

Hi @robloo, we needed to add that to support an existing enterprise customer.
While looking to WPF for inspiration is helpful, we’re not looking to copy that API 1:1 and can make changes when necessary.

You can test this PR using the following package version. 11.0.999-cibuild0033618-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

As stated, I wasn't expecting you to change it but wanted to point it out for consideration. API review and consistency is useful for everyone. And unless there is a good reason to deviate from WPF/WinUI usually its best to follow established convention -- which is what this project has done so far. Microsoft is far more strict about such things though and we can find a middle road.
Being open source by nature solicits feedback and different viewpoints to consider. I'll just assume there were reasons I am unaware of for why you went with a different name. Nothing wrong with pointing that out though.

I see your point but to me these are two different events at two different levels: a Resized event on a window causes the control's size to change, raising SizeChanged.
We could make WindowResizedEventArgs inherit from SizeChangedEventArgs and overload the SizeChanged CLR event in WindowBase with a new EventHandler<WindowResizedEventArgs> SizeChanged event, but then the we'd be introducing an inconsistency in that the handler delegate type would be different between Control and Window, unless you cast Window to Control. More confusing IMO.

Although... this did make me check the order that the events are raised, and it's wrong: SizeChanged is raised before Resized as we do an explicit layout pass when we get a resize message from the platform. This should be fixed.

are two different events at two different levels: a Resized event on a window causes the control's layout size to change, raising SizeChanged after layout has run.

In my mind Resized would be an event coming from the OS and handled internally to Avalonia (not exposed for apps). Then once it is processed it raises the normal SizeChanged event for apps (which is already done).
The sequence of these events will be confusing to app developers and I almost guarantee they will just be using them interchangeably... except for the one company you are adding this for. I'm also not sure apps need to know WindowResizeReason which is the only property that necessitates new event args I think. Again, it seems like that would just be internal to Avalonia.
I also think the comments for the Resized event need to be a lot more clear about what it's doing in the context of this discussion.


[6412] Placement snagging II validation fixes

Context
Validation
Changes proposed in this pull request
Fixed the correctivenes of validation
Added ability to open collapsed section due to validation
Removed unneeded validation
Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Snagging II: Validation fixes

I've taken a look and all looks correct to me. Good job @defong


How to use monai do extract high level features of 2D Image

Hi, I'm a beginner of pytorch and monai，I have a question about how to extract the high level features of 2D image (eg：after multil layers convolutions), for example , if I use resnet to classification, but I only need  the data of the  layer befor FC（Full connect）, should  manual edit the resnet or have other method to resolve my problems. thanks for answer my question.

I think you can try the feature_extraction from torchvision
https://pytorch.org/vision/stable/feature_extraction.html


[BUG]  Failed fetching audio key! gid

Traceback (most recent call last):
File "/root/Spotify-downloa-bot/mbot/plugins/Spotify.py", line 166, in spotify_dl
await create_task(episode_down(item , item_name, randomdir , Mbot , message, get_prem))
File "/root/Spotify-downloa-bot/mbot/utils/SpotDL.py", line 97, in episode_down
randomfile = await create_task(download_file(session, episode_id, randomdir, get_prem))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/Spotify-downloa-bot/mbot/utils/SpotDL.py", line 58, in download_file
stream = session.content_feeder().load(episode_id, VorbisOnlyAudioQuality(AudioQuality.HIGH), False, None)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/librespot/audio/init.py", line 742, in load
return self.load_episode(playable_id, audio_quality_picker,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/librespot/audio/init.py", line 780, in load_episode
return self.load_stream(file, None, episode, preload, halt_listener)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/librespot/audio/init.py", line 756, in load_stream
return CdnFeedHelper.load_episode(self.__session, episode, file,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/librespot/audio/init.py", line 391, in load_episode
key = session.audio_key().get_audio_key(episode.gid, file.file_id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/librespot/audio/init.py", line 277, in get_audio_key
return self.get_audio_key(gid, file_id, False)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/librespot/audio/init.py", line 278, in get_audio_key
raise RuntimeError(
RuntimeError: Failed fetching audio key! gid: ef41ab6ee4495ebb9a4a8e9593699e6c, fileId: 051faa3f4b7b38a1687bd7f3ba2a6f7d0236bf28
Item_,type = Episode

Generally, you are trying to load 320 kbps music on a non-premium account or region-locked.

I got this error with Audio key error, code: 2
Account premium, if retry, it maybe work again. So it's not region-locked.

I got this error with Audio key error, code: 2 Account premium, if retry, it may work again. So it's not region-locked. first try:
[DEBUG    11:52:07] https://guc3-spclient.spotify.com:443 "GET /metadata/4/track/17bd212d25dd45d088136391057e274f HTTP/1.1" 200 702
[DEBUG    11:52:08] https://guc3-spclient.spotify.com:443 "GET /storage-resolve/files/audio/interactive/3e65c9ae6648bef681f0b66508693f4cfd78bd4d HTTP/1.1" 200 347
[CRITICAL 11:52:08] Audio key error, code: 2
[CRITICAL 11:52:08] Audio key error, code: 2
Traceback (most recent call last):
  File "spot.py", line 108, in <module>
    stream = session.content_feeder().load(trackId, VorbisOnlyAudioQuality(AudioQuality.VERY_HIGH), False, None)
  File "/usr/local/lib/python3.8/dist-packages/librespot/audio/__init__.py", line 739, in load
    return self.load_track(playable_id, audio_quality_picker, preload,
  File "/usr/local/lib/python3.8/dist-packages/librespot/audio/__init__.py", line 800, in load_track
    return self.load_stream(file, track, None, preload, halt_listener)
  File "/usr/local/lib/python3.8/dist-packages/librespot/audio/__init__.py", line 754, in load_stream
    return CdnFeedHelper.load_track(self.__session, track, file,
  File "/usr/local/lib/python3.8/dist-packages/librespot/audio/__init__.py", line 339, in load_track
    key = session.audio_key().get_audio_key(track.gid, file.file_id)
  File "/usr/local/lib/python3.8/dist-packages/librespot/audio/__init__.py", line 277, in get_audio_key
    return self.get_audio_key(gid, file_id, False)
  File "/usr/local/lib/python3.8/dist-packages/librespot/audio/__init__.py", line 278, in get_audio_key
    raise RuntimeError(
RuntimeError: Failed fetching audio key! gid: 17bd212d25dd45d088136391057e274f, fileId: 3e65c9ae6648bef681f0b66508693f4cfd78bd4d

success in 2nd try:
[DEBUG    11:53:14] https://guc3-spclient.spotify.com:443 "GET /metadata/4/track/17bd212d25dd45d088136391057e274f HTTP/1.1" 200 702
[DEBUG    11:53:14] https://guc3-spclient.spotify.com:443 "GET /storage-resolve/files/audio/interactive/3e65c9ae6648bef681f0b66508693f4cfd78bd4d HTTP/1.1" 200 347
[DEBUG    11:53:14] Starting new HTTPS connection (1): audio4-fa.scdn.co:443
[DEBUG    11:53:14] https://audio4-fa.scdn.co:443 "GET /audio/3e65c9ae6648bef681f0b66508693f4cfd78bd4d?1725105194_hUqm75ZOf5qGSCKdiQ1YRTqpRWmc5SHiEKzh6jRBSKU= HTTP/1.1" 206 131072
[DEBUG    11:53:14] Chunk 1/57 completed, cached: False, stream: file_id: 3e65c9ae6648bef681f0b66508693f4cfd78bd4d
[DEBUG    11:53:14] Loaded normalization data, track_gain: -9.830001831054688, track_peak: 1.033494472503662, album_gain: -9.959999084472656, album_peak: 1.0541013479232788

Yes buddy i too facing same issues:)

seems problem is about rate limit, checkout https://github.com/justin025/onthespot/issues/3

seems problem is about rate limit, checkout here I switched to fetch audio key from seektables & mp4 format

Fetching from seemtables also has the rate limit issue right?

Fetching from seektables also has the rate limit issue right?
Not yet. Still in testing

Fetching from seektables also has the rate limit issue right?

Not yet. Still in testing

Could you please share the repo which support seektables

@Masterolic I'm using this

@Masterolic I'm using this

Thank you
https://github.com/KagChi/librespot-python
Support seektables seems like that also has limits after certain time

seems problem is about rate limit, checkout here I switched to fetch audio key from seektables & mp4 format

Any idea about the limit? For me the error appears at random, even with sequential track processing.

@Masterolic I'm using this

Thank you
https://github.com/KagChi/librespot-python
Support seektables seems like that also has limits after certain time

Hey, did you got it working with the repo? 🤔

Hey, did you got it working with the repo? 🤔

No buddy it wont as 404 raise from seektables seems like its a cdn so generate when requested but dont know how to request
Purchasing family package and creating 6 account and sharing the premium and using multiple account to generate link works fine

Hey, did you got it working with the repo? 🤔

No buddy it wont as 404 raise from seektables seems like its a cdn so generate when requested but dont know how to request
Purchasing family package and creating 6 account and sharing the premium and using multiple account to generate link works fine

seektables only works aac format which is need wvd files as far i know


Don’t start Parallels Tools on xhyve

Relating to https://github.com/ailispaw/boot2docker-xhyve/issues/6

I don't know how to check whether in Parallels or not and then just commented out for now.

Hi @legal90,
How are you?
I was @yungsang.
It's been a long time. Do you remember me?
I'm playing with xhyve for a while, but I have a problem with the original boot2docker v1.7.1.
https://github.com/ailispaw/boot2docker-xhyve/issues/6
It could not only cause a problem on xhyve, but also on other hypervisors.
Could you advise me? How can I check whether a VM is in Parallels or not?

Oops.

Moved to #2

Hi, @ailispaw! Glad to meet you again :)

It could not only cause a problem on xhyve, but also on other hypervisors.
Could you advise me? How can I check whether a VM is in Parallels or not?

Hm, it looks weird... In fact prltoolsd does some checks on the start, so it should exit silently on bare metal and all hypervisors except Parallels. I personally verified this behavior on VirtualBox.
What happens if you run sudo /etc/rc.d/prltoolsd in the VM manually? Has /var/log/parallels.log been created?

HI @legal90,
Thank you for the reply.
When I execute sudo /etc/rc.d/prltoolsd, immediately I got the same error message at https://github.com/ailispaw/boot2docker-xhyve/issues/6#issue-95300897 and it freezes.

You can try by yourself.
https://github.com/ailispaw/boot2docker-xhyve

@ailispaw Thanks. I've forwarded the problem to the people in Parallels responsible on this.
Will let you know about any updates.

@legal90 Thank you so much for your help. :)

@legal90 I'm still waiting for a response from Parallels. Any update?


Changed TableViewCells add subviews to contentView instead of self

Hey, sorry I wasn't getting notified of PRs for some reason. lmk when you're ready to get this reviewed

Also this should be a pr into dev


changing code so it can work outside of its normal environment

Coverage decreased (-0.8%) to 70.88% when pulling 5fb2b4effe20e114658e6e6cebd0276acec702fb on jpm_bqsr into 6f588faeeb240b70efd275fc8b67a07ef4aa8c54 on master.

@akiezun @jean-philippe-martin hm can't seem to reassign this PR.

@dionloy do you want to reasign? I thought i'd take the first pass at this and reassign back to @jean-philippe-martin. Do you have something else in mind?

@akiezun wanted to reassign to you, so all is good now.

@jean-philippe-martin can you explain what you're trying to accomplish in this pull req? Is this a playground are or is this aimed to be a replacement for the current BaseRecalibrator? I think it's the former, in which can I suggest you move it to a special package to mark that fact (and add a special group something like DevToolsGroup). Back to @jean-philippe-martin I need more context to review this code.

Removed the inheritance. Moved the file, though I can move it again if someone feels strongly about where it should be.

Coverage decreased (-0.76%) to 70.92% when pulling 2b7a59992359cc12d58da58c6f4f4babbe21a7f4 on jpm_bqsr into 6f588faeeb240b70efd275fc8b67a07ef4aa8c54 on master.

LGTM
(really, it doesn't because it's duplicated code without tests but let's merge it into dev and keep moving for now)

back to @jean-philippe-martin

or i'll just merge it


How to prevent superposition of objects

Is there any built in possibility to prevent superposition of objects?!

Any news ?

@robinalexandre sorry, but no


The environment configuration is incorrect

windows system
error：
ERROR: Failed building wheel for egl_probe
Running setup.py clean for egl_probe
Failed to build egl_probe
ERROR: Could not build wheels for egl_probe, which is required to install pyproject.toml-based projects

Hi, is it possible to test with a ubuntu system? We didn't try it on the windows system.


Dragging connecting line between gates should drag the whole thing

When a gate is controlled, dragging the control or the gate should break the connection but dragging the connection itself should move the whole thing as a unit.

I don't think this is a good idea anymore.


Build fails on windows when one of the bundled data files has non-ascii characters

This error only happens on windows, and only happens when I try to include a file that has some korean characters in its path string. Perhaps the fix to this could be to force utf-8 usage? Apologies in advance if this is not actually a meson-python issue, I'm just assuming it is from the traceback
Traceback (most recent call last):
 File "C:\Users\runneradmin\AppData\Local\Temp\cibw-run-a3rpdcsp\cp310-win_amd64\build\venv\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
   main()
 File "C:\Users\runneradmin\AppData\Local\Temp\cibw-run-a3rpdcsp\cp310-win_amd64\build\venv\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 335, in main
   json_out['return_val'] = hook(**hook_input['kwargs'])
 File "C:\Users\runneradmin\AppData\Local\Temp\cibw-run-a3rpdcsp\cp310-win_amd64\build\venv\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 152, in prepare_metadata_for_build_wheel
   whl_basename = backend.build_wheel(metadata_directory, config_settings)
 File "C:\Users\runneradmin\AppData\Local\Temp\pip-build-env-1c4y2iz0\overlay\Lib\site-packages\mesonpy\__init__.py", line 985, in wrapper
   return func(*args, **kwargs)
 File "C:\Users\runneradmin\AppData\Local\Temp\pip-build-env-1c4y2iz0\overlay\Lib\site-packages\mesonpy\__init__.py", line 1039, in build_wheel
   return project.wheel(out).name
 File "C:\Users\runneradmin\AppData\Local\Temp\pip-build-env-1c4y2iz0\overlay\Lib\site-packages\mesonpy\__init__.py", line 890, in wheel
   return builder.build(directory)
 File "C:\Users\runneradmin\AppData\Local\Temp\pip-build-env-1c4y2iz0\overlay\Lib\site-packages\mesonpy\__init__.py", line 445, in build
   counter.update(src)
 File "C:\Users\runneradmin\AppData\Local\Temp\pip-build-env-1c4y2iz0\overlay\Lib\site-packages\mesonpy\_util.py", line 69, in update
   print(line)
 File "C:\Users\runneradmin\AppData\Local\pypa\cibuildwheel\Cache\nuget-cpython\python.3.10.11\tools\lib\encodings\cp1252.py", line 19, in encode
   return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 72-75: character maps to <undefined>

What raises an exception is trying to log to the relative path of the file added to the wheel archive to the standard output. Python thinks that standard output should use the cp1252 encoding but this encoding cannot represent the path of the file being added, as expected: the cp1252 encoding cannot represent Korean script. What surprises me a bit is that you can have files with names in Korean script on a system that apparently uses cp1252 encoding.
I thought that system using the cp1252 encoding were extinct. On which system are you getting this error?

This is a standard github actions windows VM so I guess it's not something exotic or ancient. I can also reproduce the same fail locally while running a fresh windows 11 instance under virtualbox.

Proposed fix in #536


[GasController] Check for undefined basefeePerGas

PR Title
GasController: Don't fall back to eth_gasPrice if node returns empty fee history
Description
go-ethereum based chains never return empty baseFeePerGas because this is omitted in the serializer:
type feeHistoryResult struct {
	OldestBlock  *hexutil.Big     `json:"oldestBlock"`
	Reward       [][]*hexutil.Big `json:"reward,omitempty"`
	BaseFee      []*hexutil.Big   `json:"baseFeePerGas,omitempty"` <---
	GasUsedRatio []float64        `json:"gasUsedRatio"`
}

Current code checks for baseFeePerGas.length > 0 without checking for the occurence of this element.
This leads to an exception and in later flow to fallback to eth_gasPrice mode

FIXED:
no feeMarket mode available in case one of past requests return no results

Checklist

[X] Tests are included if applicable
[X] Any added code is fully documented

Issue

Good catch! I tried to read the geth source code when implementing this file but I obviously didn't see this. Makes total sense though. Would you be willing to add tests for this?

done

Looks like this is out of date, would you mind rebasing? Thanks.

@peak3d Apologies — do you mind rebasing this again? I will make sure to get this in today.

Apologies — do you mind rebasing this again? I will make sure to get this in today

NP, but I guess I'll have to rebase one of my two commits because they affect the same file.....
I'm CET timezone, so would be great if you merge at least one so I can care about the other....


How do we successfully add CSS and Javascript file from 3rd Party CSS Framework into this Project?

I am relatively new to webpack, so this question may seem newbie..
According to the earlier issue (https://github.com/erikras/react-redux-universal-hot-example/issues/171), and also as I have checked, I can add the css from other front framework ( for example, materialize.css (http://materializecss.com/about.html), or bootstrap.css ) and make it work. I do:
import 'materialize/css/materialize.css';    (in the Client.js)

in the Client.js,  with a line " { test: /.css$/, loader: "style-loader!css-loader" }, " added in the loader configuration, and it works out good.
However, when I do as above to include the jquery.js, and materialize.js / bootstrap.js as:
import 'materialize/js/jquery-2.1.1.min.js';    (in the Client.js)
import 'materialize/js/materialize.js';     (in the Client.js)

I get a ton of errors like:
Cannot resolve module 'jquery' ...
Cannot resolve module 'hammerjs' ...
So anyone could tell me what should I do if I want to use 3rd party framework like Materialize ?
Thank you a lot!

@likeqi, are your Materialize module installed from NPM?

Yes..actually I hand-copies the css and js file into node_modules folder.
And now I import from materialize-css, but still with errors.
Have you successfully used the bootstrap js?

@lukeqi  could you let me see your webpack config, and your Client.js ?

I have a notion I do not know if it is right,  that we cannot "import / require"   materialize.js, jquery.js in the client.js (webpack entry file), because they are not CommonJS modules? But I wonder what is the good way to put external js into this app.

It is like:
import makeRouteHooksSafe from './helpers/makeRouteHooksSafe';
import 'materialize-css/bin/materialize.css';
import 'materialize-css/js/jquery.js';  ( I put it there myself )
import 'materialize-css/bin/materialize.js';

const client = new ApiClient();

You need to require() these files, and let webpack know how to deal with .css files in dev.config.js.

@lukeqi  http://stackoverflow.com/questions/30133526/using-materializecss-with-webpack-cannot-resolve-module-hammerjs

https://github.com/Dogfalo/materialize/issues/1823

Yes, yes thank you ! @hank7444  @trueter
I did two things:

I do as  https://webpack.github.io/docs/library-and-externals.html does for including external jquery;
I npm installed hammerjs, just like you @hank7444  suggested
And now it compiles all good.

@lukeqi  You are welcome, otherwise, could you provide your solution - partial code of client.js & webpack config to help other people who have the same problem also?

@lukeqi  http://stackoverflow.com/questions/23305599/webpack-provideplugin-vs-externals
That has two ways to inject 3-party plugin, I prefer webpack.ProvidePlugin if the modules from my npm module:)

Currently, I am doing this:
1),  In  dev.config.js (webpack config):
add loader:
{ test: /.css$/, loader: "style-loader!css-loader" },
2), In client.js:
require('materialize-css/dist/css/materialize.css');
window.jQuery = require('jquery');
window.$ = require('jquery');
require('materialize-css/dist/js/materialize.js');
require('materialize-css/js/init.js');
It is done.

As @hank7444 said, you should avoid this:
window.jQuery = require('jquery');
window.$ = require('jquery');

With:
 plugins: [
        new webpack.ProvidePlugin({
            $: "jquery",
            jQuery: "jquery"
        })
    ]

In your webpack.config.js

Got it, great @gabrielmadruga

Is there a way to serve up this third party css or js to only a specific route or component? It seems like requiring in client.js means they will be loaded on all the pages.

Check out code splitting as in the huge-app example of react-router and use a component scoped require to load jQuery. Or, even better, find a solution that works without jQuery ;)

this is probably inefficient in terms of number of requests that have to exchange between client and server, but I was also able to get this to work by downloading the files I wanted to include, serving as static files with express then including a script tag in the component jsx. Thanks for the tip though, that seems much more production ready.

Thanks for the approach on index.js @Lucas-Qi !!!


train the original SAM

Hello, may I ask if we can train the original SAM in your code, or the coding only have lightning-sam? Because I haven't found the guide for Sam's original training, please give me some guidance？ Thank you very mach

lightning-sam uses the original SAM pretrained models to finetune.


Add an option to prevent IP saving on trackable

We like to use trackable to have information about if and when a user has logged in before. However, there are some privacy concerns regarding the automatic saving of the user's IP address. Right now trackable does not provide any mechanism to prevent recording the IP address.
For now we override the method in our user class:
  def update_tracked_fields(request)
    old_current, new_current = self.current_sign_in_at, Time.now.utc
    self.last_sign_in_at     = old_current || new_current
    self.current_sign_in_at  = new_current

    self.last_sign_in_ip     = nil
    self.current_sign_in_ip  = nil

    self.sign_in_count ||= 0
    self.sign_in_count += 1
  end

However, it would be nice to have a configuration option to disable this feature properly.

@josevalim  @lucasmazza do you think the configuration mentioned above is a good feature to add on

@ethirajsrinivasan I don't think a config is worth for this case - developers can override the update_tracked_fields on their models and remove the assignment of the last_sign_in_ip/current_sign_in_ip attributes on their own.


ERROR DiscoveryClient - ........ was unable to send heartbeat!

Hi,
First, I upgraded spring cloud from Hoxton. SR12 to 2021.0.2, and spring cloud netflix from 2.2.10. RELEASE to 3.1.2. Now, The discovery client are occasionally unable to send heartbeat!, It will recover automatically after 5 seconds. This phenomenon happens about 10 times a day. Could you help on this:

My code for Eureka Server yml file is below:
eureka:
instance:
hostname: ${HOSTNAME:localhost}
lease-renewal-interval-in-seconds: 10
lease-expiration-duration-in-seconds: 20
secure-port-enabled: true
secure-port: ${server.port}
non-secure-port-enabled: false
client:
registerWithEureka: false
fetchRegistry: false
serviceUrl:
defaultZone: https://${eureka.instance.hostname}:${server.port}/eureka/
server:
waitTimeInMsWhenSyncEmpty: 0
enableSelfPreservation: false
dashboard:
enabled: true
The discovery client application.yml file is as below:
eureka:
client:
register-with-eureka: true
service-url:
defaultZone: ${EUREKA_SERVER_URI:@spring.eureka.address@/eureka/}
instance:
hostname: ${HOSTNAME:localhost}

If you'd like us to spend some time investigating, please take the time to provide a complete, minimal, verifiable sample (something that we can unzip attached to this issue or git clone, build, and deploy) that reproduces the problem.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


session not available in templates

I think there might be a small piece missing from the README.
Somewhere, it should probably point out that the {{#if session.isAuthenticated}} bit won't work unless you run the following first:
ember g controller application

And put the following into it:
import Ember from 'ember';

const {inject} = Ember;

export default Ember.Controller.extend({
  session: inject.service()
});

At least this is the case for Ember 2.x
Cheers!

Thanks @artburkart, good point. Would you be willing to take a crack at this?

Sure, np

Thanks for the PR!


All Products

how to list all products using the module ?

I guess you can list all the products of the home category as all the products in prestashop belong to that