Wrong Entity

I have 3 Ecosmart AL-TP-RGBW-16FT that only have a Wifi-rssi Entity but they are RGBW led light strips.  They used to work before the update.  Let me know what else I can add to help.  Thanks.
_dump_hs_raw.json
_dump_hs_devices.json
home-assistant_hubspace_2024-09-26T06-20-48.818Z.log

The traceback looks like its trying to do something with color temperature. Let me see what its doing.

Please run the following to test the fix:

Goto HACS within HA
Click Hubspace-HomeAssistant
Click the triple dots on the top-left
Click Update Information
Click the triple dots on the top-left
Click Redownload
Click Show beta versions
Select fix-color-temp-identifer  (apparently I couldnt spell this morning)
Click DOWNLOAD
Restart HA

I did that but I still don't have a Control entity. The three devices show up but the only entity they have is diagnostic. I have no way of controlling them.

Can you confirm you switched to the release fix-color-temp-identifer and restarted HA? After switching to the dev release I see the following for your devices:

Prior to the code change:

Yes, I just re-downloaded it and re-started again. This is what I see.

Can you upload the HA logs? I'll review to see what's going on
On Sat, Sep 28, 2024, 11:02 AM Bentley0227 @.***> wrote:

Yes, I just re-downloaded it and re-started again. This is what I see. A0979B18-16F0-4AA2-A195-4BE9FF5CA570.png
(view on web)
https://github.com/user-attachments/assets/92ccbc9e-693e-40c3-93f4-e91c0307a78aE4EE0685-F7F1-433A-8023-B145D86BC4D2.png
(view on web)
https://github.com/user-attachments/assets/b81a4fb9-d1d5-4025-8cfc-02bb67154c58B016262C-99C9-467F-931E-089AEAF86491.png
(view on web)
https://github.com/user-attachments/assets/ef764599-e7cc-468f-aebc-c7b9aea7c7c6
—
Reply to this email directly, view it on GitHub
https://github.com/jdeath/Hubspace-Homeassistant/issues/106#issuecomment-2380677288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEM5H2FTBQUVFQRWQTOXPDZY3AI3AVCNFSM6AAAAABO4IESJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBQGY3TOMRYHA
.
You are receiving this because you were assigned.Message ID:
@.***>

_dump_hs_raw (1).json
_dump_hs_devices (1).json
home-assistant_hubspace_2024-09-29T01-03-09.641Z.log

After reviewing the logs I believe HA was never restarted after pulling the
update as previous messages are in the log. Please restart HA and let me
know if this fixes the issue.
On Sat, Sep 28, 2024 at 9:04 PM Bentley0227 @.***>
wrote:

_dump_hs_raw (1).json
https://github.com/user-attachments/files/17177114/_dump_hs_raw.1.json
_dump_hs_devices (1).json
https://github.com/user-attachments/files/17177115/_dump_hs_devices.1.json
home-assistant_hubspace_2024-09-29T01-03-09.641Z.log
https://github.com/user-attachments/files/17177116/home-assistant_hubspace_2024-09-29T01-03-09.641Z.log
—
Reply to this email directly, view it on GitHub
https://github.com/jdeath/Hubspace-Homeassistant/issues/106#issuecomment-2381044577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEM5H3KCVHXX3UO2BK7XTLZY5GZLAVCNFSM6AAAAABO4IESJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGA2DINJXG4
.
You are receiving this because you were assigned.Message ID:
@.***>

I had restart multiple times. I even installed the custom software again. I just power cycled HA and it still looks the same. No change.

Can you provide logs for me to review?
On Sun, Sep 29, 2024, 11:17 AM Bentley0227 @.***> wrote:

I had restart multiple times. I even installed the custom software again.
I just power cycled HA and it still looks the same. No change.
—
Reply to this email directly, view it on GitHub
https://github.com/jdeath/Hubspace-Homeassistant/issues/106#issuecomment-2381395274,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEM5HYWNSIMLK2Q6OKISM3ZZAKYVAVCNFSM6AAAAABO4IESJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGM4TKMRXGQ
.
You are receiving this because you were assigned.Message ID:
@.***>

_dump_hs_raw.json
_dump_hs_devices.json
I just re-downloaded it and installed it and re-started.  Still no luck.

Can you provide the HA logs? There's probably a traceback somewhere
On Sun, Sep 29, 2024, 5:55 PM Bentley0227 @.***> wrote:

_dump_hs_raw.json
https://github.com/user-attachments/files/17181471/_dump_hs_raw.json
_dump_hs_devices.json
https://github.com/user-attachments/files/17181472/_dump_hs_devices.json
I just re-downloaded it and installed it and re-started. Still no luck.
—
Reply to this email directly, view it on GitHub
https://github.com/jdeath/Hubspace-Homeassistant/issues/106#issuecomment-2381624441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEM5H3CFBFBZYOC6AM5LFDZZBZNJAVCNFSM6AAAAABO4IESJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGYZDINBUGE
.
You are receiving this because you were assigned.Message ID:
@.***>

home-assistant.log
home-assistant1.log
These?

Yup, those! Apparently at some point I git'd wrong and lost some changes. Let me take another whack at this.

No worries!  Thanks for all the help!

Please download 3.2.0 and restart HA and let me know if the issue persists. It is now the latest release.  If it does not solve your issue please post the logs!

Winner!!! Works as expected now. Thank you!!


Finding a file, tabbing and typing 'open' freezes Quicksilver

For some time now (since the Yosemite upgrade) every time I select a file (e.g. the most recent example had 'large text' as the default right pane) then tab and type 'open', Quicksliver (latest version) freezes and I have to Quit it from Activity Monitor. Persists after clearing the cache. I found the following in Console:
27/09/2015 21:43:38.115 WindowServer[143]: CGError post_notification(const CGSNotificationType, void *const, const size_t, const bool, const CGSRealTimeDelta, const int, const CGSConnectionID *const, const pid_t): Timed out 1.000 second wait for reply from "Quicksilver" for synchronous notification type 102 (kCGSDisplayWillSleep) (CID 0x12e03, PID 362)
27/09/2015 21:18:21.506 Quicksilver[362]: AppleEvents: Send port for process has no send right, port=( port:35411/0x8a53 rcv:1,send:0,d:0 limit:5) (findOrCreate()/AEMachUtils.cp #526) QSCatalogEntry scanQueue: QSPresetiTunesAirPlayDevices
28/09/2015 20:08:43.408 Quicksilver[14422]: assertion failed: 14F27: libxpc.dylib + 62447 [5C829202-962E-3744-8B50-00D38CC88E84]: 0x89

Which interface are you using? (You can check by going to Preferences → Appearance.)

Bezel (built-in)

When you’re in the Console, click “Show Log List” (if it’s not already showing) and look under “User Diagnostic Reports” for reports from Quicksilver. That should have more detail about the crash.

Hmmm. There's nothing from Quicksilver under "User Diagnostic Reports"… searched other logs too, found nothing more for QS?

Oh, it’s just hanging and not crashing, so that’s why. In that case, you can “Sample Process” in Activity Monitor, but I have a theory.
I wonder if when you type “open”, it’s selecting the “Open With…” action. That action has to populate the third pane with items from your catalog, and can get delayed in some cases.
The most common source of problems we’ve found are symbolic links or aliases in your catalog that point to things on unavailable network (or other removable) drives.
Could that be what’s happening?

Might be fixed by #2177.


Add burn-in parameter for early stopping

Add some functionality to the EarlyStopping classes to make sure a model is trained for some minimum number of epochs before stopping training. Also some tests to make sure they're all functioning as expected.

@hmacdope this keeps getting bumped from the merge queue because of the different workers CI issue. is there a way to just push it through?

Runners very flaky atm.


Fix support for --sourcemap-output path containing spaces in Xcode projects

Summary
This change fixes the generation of source maps for Xcode projects where the output path contains spaces.
The EXTRA_ARGS environment variable, being a plain string, would be split into arguments by whitespace - so a path containing spaces was being treated as several arguments rather than one.
This change uses an array to contain the arguments instead, allowing the proper handling of arguments that may contain spaces.
Changelog
[iOS] [Fixed] - Fix support for --sourcemap-output path containing spaces
Test Plan
Tested using a sample project with the following "Bundle React Native code and images" Xcode build phase
export SOURCEMAP_FILE="$CONFIGURATION_BUILD_DIR/$UNLOCALIZED_RESOURCES_FOLDER_PATH/main.jsbundle.map"
set -e
export NODE_BINARY=node
../node_modules/react-native/scripts/react-native-xcode.sh

and a CONFIGURATION_BUILD_DIR that contains spaces - ~/Library/Xcode/Derived Data
Before
+ EXTRA_ARGS=
+ case "$PLATFORM_NAME" in
+ BUNDLE_PLATFORM=ios
+ EMIT_SOURCEMAP=
+ [[ ! -z /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map ]]
+ EMIT_SOURCEMAP=true
+ PACKAGER_SOURCEMAP_FILE=
+ [[ true == true ]]
+ [[ '' == true ]]
+ PACKAGER_SOURCEMAP_FILE='/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map'
+ EXTRA_ARGS=' --sourcemap-output /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map'
+ node /Users/nick/Desktop/RN064/node_modules/react-native/cli.js bundle --entry-file index.js --platform ios --dev false --reset-cache --bundle-output '/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/main.jsbundle' --assets-dest '/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app' --sourcemap-output /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map
                    Welcome to Metro!
              Fast - Scalable - Integrated

info Writing bundle output to:, /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/main.jsbundle
info Writing sourcemap output to:, /Users/nick/Library/Developer/Xcode/Derived

Note the incorrect sourcemap output path.
After
+ EXTRA_ARGS=()
+ case "$PLATFORM_NAME" in
+ BUNDLE_PLATFORM=ios
+ EMIT_SOURCEMAP=
+ [[ ! -z /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map ]]
+ EMIT_SOURCEMAP=true
+ PACKAGER_SOURCEMAP_FILE=
+ [[ true == true ]]
+ [[ '' == true ]]
+ PACKAGER_SOURCEMAP_FILE='/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map'
+ EXTRA_ARGS+=("--sourcemap-output")
+ EXTRA_ARGS+=("$PACKAGER_SOURCEMAP_FILE")
+ node /Users/nick/Desktop/RN064/node_modules/react-native/cli.js bundle --entry-file index.js --platform ios --dev false --reset-cache --bundle-output '/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/main.jsbundle' --assets-dest '/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app' --sourcemap-output '/Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map'
                    Welcome to Metro!
              Fast - Scalable - Integrated

info Writing bundle output to:, /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/main.jsbundle
info Writing sourcemap output to:, /Users/nick/Library/Developer/Xcode/Derived Data/RN064-cpnwckdferodycbevupbrkjydate/Build/Products/Release-iphonesimulator/RN064.app/main.jsbundle.map

sourcemap output path fixed 🎉

Why is this one closed without being merged? This is still a bug for every project that has a space in its scheme name...
Is it possible to reopen and merge?


Jroach/us108637

US108637: [UI] New web component that wraps d2l-navigation-iterator and treats the buttons as links
Thoughts on the name? d2l-navigation-link-iterator was my initial idea.
Also, since this is simply a logic wrapper, I don't think we need a designer sign off. Please let me know if you think otherwise

d2l-navigation-link-iterator sounds like a fine name. Out of curiosity, how come you didn't just add this functionality to d2l-navigation-iterator?
Although there are no visual changes, there are still "UI" changes due to the behaviour of clicking a link. For example, they might have something to say about the back and forward browser buttons. But in this case I think it's fine to bypass a designer check.

d2l-navigation-link-iterator sounds like a fine name. Out of curiosity, how come you didn't just add this functionality to d2l-navigation-iterator?

The intent of d2l-navigation-iterator is to be generic (and fire events on click) instead of redirecting - this is to support it being used on a page that might:
a) use partial reloads (and not redirect to a page)
b) show and hide info via the iterator
c) other similar use case not involving a redirect
Thus the user can wrap d2l-navigation-iterator and do what they wish with the event.

d2l-navigation-link-iterator sounds like a fine name. Out of curiosity, how come you didn't just add this functionality to d2l-navigation-iterator?

The intent of d2l-navigation-iterator is to be generic (and fire events on click) instead of redirecting - this is to support it being used on a page that might:
a) use partial reloads (and not redirect to a page)
b) show and hide info via the iterator
c) other similar use case not involving a redirect
Thus the user can wrap d2l-navigation-iterator and do what they wish with the event.

Makes total sense, thanks.

Tests are timing out, mostly seems to be an issue with IE11 :(
Must be related to sinon - I'm digging


Autocomplete performance in large documents

v0.7-alpha.32
Occurs when entering the leading backslash of a command in a large-ish document. IDE freezes for a while (sometimes up to 10s).
jstack was taken during this time.
Video of behaviour: https://lol768.com/freeze.webm (note that I continue typing faCogs whilst it is stuck).

Thanks for reporting. We know of it, there are performance issues when working in large documents. Not just the autocomplete, also right-clicking on a file can hang a long time for example. Please advise in case you have any clue how to debug that :) (I didn't spend any time on it yet though)

I'm trying to follow the logic in UpDownAutoBracket and work out what it's doing and why:
https://github.com/Ruben-Sten/TeXiFy-IDEA/blob/9dc58e2de660a638124c2b756fe6ef2d5344671c/src/nl/rubensten/texifyidea/editor/UpDownAutoBracket.kt#L58-L63
It seems to be backtracking to find LatexNormalText. In my case, if it's looking for what I think it is, that's a good 30 lines away. Why's it need to find normal text?
Also, there's a check later on in handleNormalText that I think happens after all of this findNormalText stuff:
https://github.com/Ruben-Sten/TeXiFy-IDEA/blob/9dc58e2de660a638124c2b756fe6ef2d5344671c/src/nl/rubensten/texifyidea/editor/UpDownAutoBracket.kt#L131-L135
Where the regex is defined as:
https://github.com/Ruben-Sten/TeXiFy-IDEA/blob/9dc58e2de660a638124c2b756fe6ef2d5344671c/src/nl/rubensten/texifyidea/editor/UpDownAutoBracket.kt#L32
Wouldn't it make sense to fail fast? Can this decision be made earlier on since it seems to just rely on the inserted character?

Interesting find! I'll take a closer look at it time.
It definitely looks like a deficiency to me at first glance. I'll get back at you later!

Cheers!
And don't mention it - I'd be paying a lot more if it was an official JetBrains plugin.

I cannot possibly let our appreciation get unnotticed.
Looking at UpDownAutoBracket, it seems like it could do with a complete rewrite tbh. Things like converting a Char to a string and matching it against a regex instead of using a Set. It looks a bit cluttered overall. I think the initial rationale about looking up LatexNormalText is that it allows you to call PsiElement.inMathContext(), as the auto-brackets should only appear in math mode. Which could also be called on val element (line 46) in the beginning... So I think I just want to refactor the whole thing. That'd be better in the long run I think.

@lol768 I have installed visualvm now, but how do you find that UpDownAutoBracket class? So far I have opened the Sampler view on Idea, and in the settings when I filter on only the nl.rubensten.** package I can see what's taking time in texify (e.g. #915, so I will rename this issue to be more specific).
And when I filter on nl.rubensten.texifyidea.util.FilesKt I get all things containing that class at some point, but when I filter on nl.rubensten.texifyidea.editor.UpDownAutoBracketKt or even nl.rubensten.texifyidea.editor.** I see nothing in the CPU sampling when I do exactly as in the video.

@PHPirates IIRC I was able to reproduce it in VisualVM by sampling while typing.

@RubenSchellekens Okay thanks, not sure what I'm doing wrong then - although it's maybe not so important. But does the fix in #869 improve anything for you then?


Failed to find deployment vrZSlz7TtRjmNVeLbbvK69Vc under xsestech

I tried to copy Yandex skill and It gave me an error

☝ Closing now, in favor of this comment.


Custom Values not available in some dropdowns in Date Time Functions

Describe the Bug with repro steps
Please see internal OneNote for the examples, but we have a handful of functions in the "Equal #####" family that don't allow for a custom value in the first dropdown but do in the second dropdown.
Screenshots or Videos

Version
2.53.0
Additional context
No response
AB#24678716

@kewear we are unable to reproduce this one. I did test and it seems like 'custom input' does not show up in the dropdown until you start typing. Unsure if that is a regression or by design. Mind re-checking this one for us?


Add Meetup links and IDs to all chapters

To fetch IDs, login to Meetup, then go to https://secure.meetup.com/meetup_api/console/?path=/:urlname
Paste the Meetup URL slug (e.g., Girl-Develop-It-Ann-Arbor) in the urlname box, then select Show Response. The ID will be at the top.

Mostly adding this one to document the process; I'll knock them out in the next hour or so.

All chapters in the DB now have a Meetup ID; verified by manual check.


chore: Update to publint 0.2.0

Adds dual ESM/CJS type resolution check, so we don't need ATTW as well (which is much slower as it needs to zip up the package first)
https://github.com/bluwy/publint/releases/tag/v0.2.0

Codecov Report
Patch coverage has no change and project coverage change: -8.45% :warning:

Comparison is base (e0aad73) 97.11% compared to head (cdb89b5) 88.66%.
Report is 625 commits behind head on beta.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             beta    #5792      +/-   ##
==========================================
- Coverage   97.11%   88.66%   -8.45%
==========================================
  Files          50       74      +24
  Lines        2391     2400       +9
  Branches      706      623      -83
==========================================
- Hits         2322     2128     -194
- Misses         67      227     +160
- Partials        2       45      +43

see 124 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update Dialogflow V2

I did not use config.js and switched to .env instead. You have to rewrite that if config.js is your preferred way.
Usually, the tokens are imported with a JSON file that you download via dialogflow. The implementation is even easier than this way but I prefer to have everything in a .env file.
What needs to be in the .env file:
BOT=
PROJECT_ID=
CLIENT_EMAIL=
PRIVATE_KEY="" (necessary because the key contains blank characters)

I did not use config.js and switched to .env instead. You have to rewrite that if config.js is your preferred way.
Usually, the tokens are imported with a JSON file that you download via dialogflow. The implementation is even easier than this way but I prefer to have everything in a .env file.
What needs to be in the .env file:
BOT=
PROJECT_ID=
CLIENT_EMAIL=
PRIVATE_KEY="" (necessary because the key contains blank characters)

Apologies just saw your PR, can you also update the Readme.md to reflect the change please.

This pull request has been done API V2?

Is this gonna get merged, or...?
Seriously, this is the closest I've gotten to being able to run my bot on discord, and it's very frustrating.


Will there be an option to customize the shown columns?

Hello guys, first thanks for the great project it is really so useful =)
I would love to have a column on my feed that already exists in the xml
here per example https://colourpop.com/collections/all.atom
that is the node :
I want my feed to show only the items in which the date (without time) of  value is different than the  one
Is there anyway I can do that through commafeed?
Thanks a lot !

I'm sorry but I don't understand your question. Are there pictures missing?

Oh sorry I will try to explain
Let's take as an example the product (1) in this photo : https://i.imgur.com/T6Z9EGb.jpg
where the left side is the RSS feed and the right one is the XML
I want to add the node (3) to the placement (4) as shown in the picture if possible :)

Ah okay now I understand.
However, CommaFeed does not update existing entries when parsing a feed and just skip them if they are already known. This is by design for performance reasons.
That could be changed or be an option but I don't plan to implement this in the near future :(

Oh that would ve been a huge addition
Okay thanks a lot for your consideration :)


Broken link in readme

Describe the bug
This repo's readme contains a broken link for how to get started.
To Reproduce
Open the readme and click the getting started link at the end.
Expected behavior
Getting started documentation.
Observed behavior
HTTP 404

Done https://github.com/servicecatalog/oscm-rest-api/commit/c2fe5877247dd9540f71387218660e14db4725ec


pretrained model prediction result

Hi, I just used the pretrained model you provided to evaluate on the sample dataset, the result doesn't seem to make reasonable prediction, I also tried to train the model by myself, but hasn't good prediction either, has anyone used the pretrained model to predict good result, or have you successfully train the model to make good result?

Hi,
I think I have the same problem. Even if the input is a torch.zeros the model output has no sense !
Did someone has good results with the pretrained model ?
Thank you

I added sigmoid activation to the output during the training and evaluation and it produced reasonable results.


GitHub Theme

It'd be rad to have a theme that looks like a README.md looks here on GitHub.com.  Maybe with https://github.com/sindresorhus/github-markdown-css or something.

I just made this work locally by downloading the github-markdown.css from https://github.com/sindresorhus/github-markdown-css, renamed it github.min.css and added it to the themes folder. Then you can use theme="github" on the xmp tag. For the CSS to work you need to add a markdown-body class to the html tag.


[Bug]:  PUPPETEER_DOWNLOAD_BASE_URL authentication dropped resulting in 401

Bug expectation
When setting the PUPPETEER_DOWNLOAD_BASE_URL to an authenticated Jfrog artifactory URL, the authentication is dropped in the browsers package httpUtil script file.
I'm working behind a Corporate proxy and we must go through jfrog for any downloads. I set the PUPPETEER_DOWNLOAD_BASE_URL to go through an authenticated jfrog  URL.
Example:
export<EMAIL_ADDRESS>
I get this error:
Error: Download failed: server returned code 401. URL<EMAIL_ADDRESS>I expect the authentication to be carried forward to the browser request.
Here's the code in the httpUtil.ts that I believe needs to be updated. I added setting the auth attribute. I found in my local testing that this works with and without authenticated base URLs.
import {URL, urlToHttpOptions } from 'url';  <---- Needed to import urlToHttpOptions

export function httpRequest(
  url: URL,
  method: string,
  response: (x: http.IncomingMessage) => void,
  keepAlive = true
): http.ClientRequest {
  const options: http.RequestOptions = {
    protocol: url.protocol,
    hostname: url.hostname,
    auth: urlToHttpOptions(url).auth,,  <---- One way to set auth. Works when auth not passed as well.
    port: url.port,
    path: url.pathname + url.search,
    method,
    headers: keepAlive ? {Connection: 'keep-alive'} : undefined,
    agent: new ProxyAgent(),
  };

Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example

Error string
no error
Puppeteer configuration
No response
Puppeteer version
20.4.0
Node version
19.7.0
Package manager
npm
Package manager version
9.5.0
Operating system
Windows

Fixed in https://github.com/puppeteer/puppeteer/pull/10271


Can't authenticate if the first attempt expires

Description
If you don't activate your device within 10 minutes the authentication/authorization of the device (the MagicMirror) will expire. When it expires the module creates a token file which will be empty. This empty token file will be used in all other attempts which leads to Error: OAuth required.
This bug is probably what causes the problems in issue #17 .
Suggested solution

Handle the expire error properly without creating an empty token file.
Restart the device authorization.
Limit the number of authorization restarts so the module don’t spam trakt.tv.
Show a message in the frontend that MM has to be restarted if the limit is reached.

Logs
[20:11:17] - MMM-trakt - :  Trakt Access Code: 3EC1C65D
[20:11:47] - MMM-trakt - :  Trakt Access Code: A7D98CE3
[20:12:17] - MMM-trakt - :  Trakt Access Code: EF46560D
[20:12:47] - MMM-trakt - :  Trakt Access Code: 3C9637E6
[20:13:17] - MMM-trakt - :  Trakt Access Code: 197B5FB9
[20:13:47] - MMM-trakt - :  Trakt Access Code: 0D033DC5
[20:14:17] - MMM-trakt - :  Trakt Access Code: 3ACB08E7
[20:14:47] - MMM-trakt - :  Trakt Access Code: D7E19BA0
[20:15:17] - MMM-trakt - :  Trakt Access Code: DD60D848
[20:15:47] - MMM-trakt - :  Trakt Access Code: 3A9190CD
[20:16:17] - MMM-trakt - :  Trakt Access Code: 6540D429
[20:16:47] - MMM-trakt - :  Trakt Access Code: 8A132FDB
[20:17:17] - MMM-trakt - :  Trakt Access Code: B14461DC
[20:17:47] - MMM-trakt - :  Trakt Access Code: 5C501BE1
[20:18:17] - MMM-trakt - :  Trakt Access Code: 4716B2D7
[20:18:47] - MMM-trakt - :  Trakt Access Code: 01098E31
[20:19:17] - MMM-trakt - :  Trakt Access Code: 37C26D20
[20:19:47] - MMM-trakt - :  Trakt Access Code: A971485F
[20:20:17] - MMM-trakt - :  Trakt Access Code: 63CC0216
[20:20:47] - MMM-trakt - :  Trakt Access Code: 126339B9
[20:21:17] - MMM-trakt - :  Trakt Access Code: 3E97A4E4
[20:21:17] - ERROR MMM-trakt :  Error: Expired
    at call (/MagicMirror/modules/MMM-trakt/node_modules/trakt.tv/trakt.js:319:35)
    at Timeout.setInterval (/MagicMirror/modules/MMM-trakt/node_modules/trakt.tv/trakt.js:343:17)
    at ontimeout (timers.js:498:11)
    at tryOnTimeout (timers.js:323:5)
    at Timer.listOnTimeout (timers.js:290:5)  - [    at trakt.get_codes.then.catch.error (/MagicMirror/modules/MMM-trakt/node_helper.js:42:34)]
[20:21:17] - MMM-trakt - :  { access_token: undefined,
  expires: undefined,
  refresh_token: undefined }
[20:21:17] - DEBUG - MMM-trakt - :  Trakt {
  _authentication:
   { access_token: undefined,
     expires: undefined,
     refresh_token: undefined },
  _settings:
   { client_id: '(DELETED)',
     client_secret: '(DELETED)',
     redirect_uri: 'urn:ietf:wg:oauth:2.0:oob',
     debug: undefined,
     endpoint: 'https://api.trakt.tv',
     pagination: undefined,
     useragent: 'trakt.tv/7.1.0 (NodeJS; +git+https://github.com/vankasteelj/trakt.tv.git)' },
  calendars:
   { my:
      { shows: [Function],
        new_shows: [Function],
        premieres_shows: [Function],
        movies: [Function],
        dvd: [Function] },
     all:
      { shows: [Function],
        new_shows: [Function],
        premieres_shows: [Function],
        movies: [Function],
        dvd: [Function] } },
  checkin: { add: [Function], delete: [Function] },
  certifications: [Function],
  comments:
   { comment:
      { add: [Function],
        get: [Function],
        update: [Function],
        remove: [Function] },
     replies: { get: [Function], add: [Function] },
     item: [Function],
     likes: [Function],
     like: { add: [Function], remove: [Function] },
     trending: [Function],
     recent: [Function],
     updates: [Function] },
  lists: { trending: [Function], popular: [Function] },
  genres: [Function],
  movies:
   { trending: [Function],
     popular: [Function],
     played: [Function],
     watched: [Function],
     collected: [Function],
     anticipated: [Function],
     boxoffice: [Function],
     updates: [Function],
     summary: [Function],
     aliases: [Function],
     releases: [Function],
     translations: [Function],
     comments: [Function],
     lists: [Function],
     people: [Function],
     ratings: [Function],
     related: [Function],
     stats: [Function],
     watching: [Function] },
  networks: [Function],
  people:
   { summary: [Function],
     movies: [Function],
     shows: [Function],
     lists: [Function] },
  recommendations:
   { movies: { get: [Function], hide: [Function] },
     shows: { get: [Function], hide: [Function] } },
  scrobble: { start: [Function], pause: [Function], stop: [Function] },
  search: { text: [Function], id: [Function] },
  shows:
   { trending: [Function],
     popular: [Function],
     played: [Function],
     watched: [Function],
     collected: [Function],
     anticipated: [Function],
     updates: [Function],
     summary: [Function],
     aliases: [Function],
     translations: [Function],
     comments: [Function],
     lists: [Function],
     progress: { collection: [Function], watched: [Function] },
     people: [Function],
     ratings: [Function],
     related: [Function],
     stats: [Function],
     watching: [Function],
     next_episode: [Function],
     last_episode: [Function] },
  seasons:
   { summary: [Function],
     season: [Function],
     comments: [Function],
     lists: [Function],
     ratings: [Function],
     watching: [Function] },
  episodes:
   { summary: [Function],
     translations: [Function],
     comments: [Function],
     lists: [Function],
     ratings: [Function],
     stats: [Function],
     watching: [Function] },
  sync:
   { last_activities: [Function],
     playback: { get: [Function], remove: [Function] },
     collection: { get: [Function], add: [Function], remove: [Function] },
     watched: [Function],
     history: { get: [Function], add: [Function], remove: [Function] },
     ratings: { get: [Function], add: [Function], remove: [Function] },
     watchlist: { get: [Function], add: [Function], remove: [Function] } },
  users:
   { settings: [Function],
     requests: { get: [Function], approve: [Function], deny: [Function] },
     hidden: { get: [Function], add: [Function], remove: [Function] },
     likes: [Function],
     profile: [Function],
     collection: [Function],
     comments: [Function],
     lists: { get: [Function], create: [Function], reorder: [Function] },
     list:
      { get: [Function],
        update: [Function],
        delete: [Function],
        like: [Object],
        items: [Object],
        comments: [Function] },
     follow: { add: [Function], remove: [Function] },
     followers: [Function],
     following: [Function],
     friends: [Function],
     history: [Function],
     ratings: [Function],
     watchlist: [Function],
     watching: [Function],
     watched: [Function],
     stats: [Function] } }
(node:10458) UnhandledPromiseRejectionWarning: Error: OAuth required
    at Trakt._call (/MagicMirror/modules/MMM-trakt/node_modules/trakt.tv/trakt.js:195:122)
    at Object.tmp.(anonymous function) [as shows] (/MagicMirror/modules/MMM-trakt/node_modules/trakt.tv/trakt.js:49:33)
    at trakt.import_token.then.newTokens (/MagicMirror/modules/MMM-trakt/node_helper.js:55:32)
    at <anonymous>
(node:10458) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 22)
(node:10458) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Oh yeah, instead of just logging the error and continuing we should probably return Promise.reject(error); and restart the get_codes promise. Never actually tried to idle for so long, so I never saw this error.


Feature: vigem driver for ddrio

Reads state of ddrio and updates vigem pads. Similar in vein to viggem-sdvxio and vigem-iidxio
Useful for playing xinput related games with official hardware.

Really nice one, looking forward to giving this a try on my own cabinet.
I noticed we have some brief docs for the iidxio and sdvxio vigem clients. It would be cool to have something similar to these for the ddrio one as well and link it from the doc/vigem/README.md.

Code's looking good. Give me a ping using the "re-request review" feature and I will have a another peek and merge this asap.

Great work, thanks. Merged.


add support for explicit per-project executables

add support for explicitly specifying per-project executables
update README file with FAQ example of new functionality
new version message file
resolves situations that may occur when eslint is not a direct dependency of the project NodeLinter is not able to locate a binary for it

Please let me know if you need anymore background for this change.

Hello, can you clarify what a reason of this PR? I guess locally installed eslint inside of project is pretty enough for the most of scenarios.

You are probably correct in that locally installed works for most people. This provides support for anyone who doesn't have it installed globally or locally and for whatever reason doesn't want to or can't.
One example I have is that we use gruntify-eslint and do most of our linting via Grunt including for our build process but some of our team also use Sublime and want to be able to integrate the linting into our development real-time rather than needing to run Grunt. Installing eslint locally means that we need to have our project maintain the exact same version of eslint as gruntify-eslint. Maintaining the same version of something in two places will inevitably lead to problems. With this update we can point SublimeLinter to the version that is already installed inside of gruntify-eslint. I imagine we are not alone in this scenario. There are probably other example of how this would be useful that is just the one that is closest to me right now.
Also, other linters provide similar functionality it would be nice to have this one do the same.

@ajmichels I suppose I see some not so good practice in your example. Your gruntify-eslint should be installed inside of project and you should install eslint there explicitly, now grunt and this plugin use single instance of eslint.
Please understand that this PR almost doubles code of plugin and adds a lot of complexity and I want to be sure that it solves real problem.
Also, can you give links to plugins with similar functionality to set executable path?


Increased inline powershell script maxLength to an unlimited size for…

… more flexibility

Hi @icnocop, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@chrisrpatterson to comment, I believe the limitation is intentional and larger scripts should be checked-in.

I recommend leaving that decision to the developer for flexibility reasons.
For example, I may want to build code in a repository that I only have read-only access for.
Another example is if I want to build an older version of the repository, where the powershell script has not been checked in, or is an older version that no longer works for whatever reason.

For the scenarios you mention, in the powershell script task, you can specify any location.  It does not have to be a repo location.  For example, you can put in the location of a share.
This option was created for quick little snippets - not to be a coding environment.

I am using Visual Studio Team Services and it doesn't seem practical to have a dependency on an external file if I need 501 characters in a powershell script for example. If the first time a powershell script task works and requires less than 500 characters, great. But in a subsequent iteration, if the build step needs to do more things and exceeds 500 characters, the developer may do even worse things like make the code (ex. variable names) unreadable. If the powershell script task gets too long, maybe we can just display a warning and indicate a recommend solution. Why impose limitations on the build and on the developer? Please let the developer decide what's best for the build at the time. Let the developer decide when a good time is to move to an external file if/when needed/desired.

Because once you're beyond a block of inline and into a long file expectations change.  At that point it's a coding environment when you're coding long files
It's not arbitrary.

500 is arbitrary.
Let the developer decide what the limitations and expectations are, and not the tool.

a specific number is but the choice to limit it to something reasonable that is not optimized for in the task model and views is not.  The use case for this was I only have a few lines and I don't want to check in a file.  Once you're writing very large files you need the Monaco editor and history.
We have another feature on the backlog for additional repos outside your main repos so you can keep scripts out of source, not in a share, in the cloud and use the web editor and history
In the meantime, if you really want to do this then create your own task.  But be aware that we haven't optimized for very large task inputs throughout


License?

Hi All,
What is the license of this repository?

Thanks for the reminder about the license. We have added that.


build: coverage does not work with clang

It looks like clang coverage output is not compatible with gcov/gcovr. Need to ultimately figure out some solution here. (This is true at least as of 4.0 release).

Closing as dup of https://github.com/envoyproxy/envoy/issues/1000


Support httpS://caniuse.com

You  used to support embeding on HTTPS, but no longer do which breaks my blog post: https://www.ibuildings.nl/blog/2013/03/4-http-security-headers-you-should-always-be-using
Any chance that is coming back?

It’s even worse: HTTPS links couldn’t be opened: https://caniuse.com/#feat=css-variables

Kinda dub of #885. But yeah, with Let's Encrypt around it really should be HTTPS already…

Yeah, duplicate of #885.
The caniuse embed system is basically deprecated anyway, there's some others out there like http://caniuse.bitsofco.de/ which you can try instead.


Support for persistent HTTP connections

When transacting with an API it does three HTTP requests:

v1/chain/get_info to get the reference block
v1/chain/get_required_keys to check which key is needed
v1/chain/push_transaction to push the actual tx

We're pushing 1 transaction from 3 services every second, so we end up with 9 HTTP calls / second.
The servers we're using (confidential VMs) have strict TCP port limitations (48) and since the channels are left open (MSL has a termination of 120s) and unable to be re-used until they close we end up consuming all available ports in little time just on transaction pushing which then impacts the rest of the service as it shares the same pool.
Aside from the benefits the request originator receives, it should also lower network congestion and cpu load on nodeos itself by reducing the number of open TCP connections it needs to track in the OS.

This issue is being worked on as we speak as part of the replacement of Websocket++ with boost::beast and is expected for 2.2 RC2


[#5886] feat (gvfs-fuse): Implement an in-memory file system

What changes were proposed in this pull request?
Implement an in-memory filesystem for testing and validating the FUSE framework. You need to implement the PathFilesystem trait and support basic file and directory operations:
Why are the changes needed?
Fix: #5886
Does this PR introduce any user-facing change?
No
How was this patch tested?
IT

I think it's time to organize the code hierarchy not flatten all the code in the src directory.

I think it's time to organize the code hierarchy not flatten all the code in the src directory.

Wait until the subsequent PRs are merged before organizing. Doing the later PRs now would lead to significant conflicts.


Undo with CMD+Z does not work across multiple blocks

Precheck

[x] Do a quick search and make sure the bug has not been reported yet.
[x] Please disclose security vulnerabilities privately at<EMAIL_ADDRESS>[x] Finally, be nice and have fun!.

Description
If a user makes changes in more than one block and tries to undo those changes with CMD+Z, the plugin breaks and unexpectedly deletes a block when it should not.
Environment

Editor.js version: 2.23.2
Plugins you use with their versions: editorjs-undo (2.0.0-rc.3)
Device, Browser, OS: Macbook Pro, Chrome, MacOS

Current behavior
The plugin unexpectedly deletes content during CMD+Z operations across multiple blocks, as shown below:
https://user-images.githubusercontent.com/19937217/159501773-09e041c2-528d-4c9d-861f-13a1f240e380.mov
To recreate this bug, take the following steps:

Enter some text in a paragraph block
Press enter and enter some text in a new paragraph block
Select and delete some of the text in the first paragraph block
Select and delete some of the text in the second paragraph block
Press CMD+Z
Notice that the second paragraph block is deleted
Press CMD+Z
Notice that the following error is thrown:

Expected behavior
Undo operations with CMD+Z should work properly across multiple blocks instead of having content deleted unexpectedly, as shown below:
https://user-images.githubusercontent.com/19937217/159503247-479c187d-5e26-4714-be0f-473d19357faa.mov
Additional context
The plugin is currently incorrectly deleting a block when it should not because no entry is inserted into the stack when a user moves between blocks. This causes the blockWasSkipped function to return true when it should not. It compares the index and compIndex obtained from the stack, and since there was no stack entry for when the user switched from one block to another, this condition evaluates to true and the second block in the example I provided gets deleted.

For what it's worth, I did attempt to implement a fix for this issue by modifying the blockWasSkipped function to no longer do an index comparison and also adding a new click listener/handler to save and insert new stack entries when the user switched between blocks, but my solution did not seem stable across environments. These changes are shown below:

Unfortunately I do not have the bandwidth to continue looking into a solution for this issue, so I'm hoping someone in the community can solve it and that the information I've provided here helps with that effort.

@trevrdspcdev Hi there!
Thanks a lot for reporting the issue, it was solved in the pull request #108 and it will be available in the version v2.0 soon.
Best,
Mario


401 - Unauthorized when deploying kit

Category

[ ] Question
[X] Bug
[ ] Enhancement

Expected or Desired Behavior
No errors when running Apply-PnPProvisioningHierarchy -Path starterkit.pnp or meaningful error message.
Apply-PnPProvisioningHierarchy -Path starterkit.pnp
Observed Behavior
WARNING: Apply-PnPProvisioningHierarchy has been deprecated. Use Apply-PnPTenantTemplate instead.
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:30.3735	[OfficeDevPnP.Core]	[0]	[Information]	File starterkit.pnp retrie
ved from folder 	0ms
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:30.4225	[OfficeDevPnP.Core]	[0]	[Information]	File starterkit.pnp retrie
ved from folder 	0ms
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:30.9826	[OfficeDevPnP.Core]	[0]	[Information]	File starterkit.xml retrie
ved from folder 	0ms
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:31.2256	[Provisioning]	[17]	[Debug]	Code execution scope started	0ms	72f
e1e03-f6b5-4391-91b1-15a1afc6ed0c
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:31.2266	[Provisioning]	[17]	[Information]	ProgressDelegate registered	1m
s	72fe1e03-f6b5-4391-91b1-15a1afc6ed0c
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:31.2276	[Provisioning]	[17]	[Information]	MessagesDelegate registered	2m
s	72fe1e03-f6b5-4391-91b1-15a1afc6ed0c
PowerShell_ISE.exe Information: 0 : 2018-11-27 09:41:31.5274	[Provisioning]	[17]	[Debug]	Code execution scope ended	302ms	72f
e1e03-f6b5-4391-91b1-15a1afc6ed0c
Apply-PnPProvisioningHierarchy : The remote server returned an error: (401) Unauthorized.
At line:1 char:1

Apply-PnPProvisioningHierarchy -Path starterkit.pnp -Parameters @{"Si ...

  + CategoryInfo          : WriteError: (:) [Apply-PnPTenantTemplate], WebException
  + FullyQualifiedErrorId : EXCEPTION,SharePointPnP.PowerShell.Commands.Provisioning.Tenant.ApplyTenantTemplate

Steps to Reproduce
Follow document setup steps to create new attribute, and set provisioning account as term adminsitrator.
Open powershell IDE, run as administrator
Apply-PnPProvisioningHierarchy -Path starterkit.pnp

This may be related to https://github.com/SharePoint/sp-starter-kit/issues/179. Check to ensure you are an admin of the term store as well.

Did you ever get a resolution @cberon ?

I did not
Get Outlook for Androidhttps://aka.ms/ghei36

From: bazookadaver<EMAIL_ADDRESS>Sent: Friday, February 15, 2019 11:03:07 AM
To: SharePoint/sp-starter-kit
Cc: cberon; Mention
Subject: Re: [SharePoint/sp-starter-kit] 401 - Unauthorized when deploying kit (#208)
Did you ever get a resolution @cberonhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcberon&data=02|01||be9c4aab6d4e444ced3508d69378394f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636858541903165104&sdata=%2B6sz%2FtRnwrmIhotlhWVCswrOFc3Sbc9FgQIcD3a%2BJmM%3D&reserved=0 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-starter-kit%2Fissues%2F208%23issuecomment-464162371&data=02|01||be9c4aab6d4e444ced3508d69378394f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636858541903175109&sdata=gXK6GBaVq4xgOX3YxXmmO8xTgMls3Rq%2Byz30dylx%2FDs%3D&reserved=0, or mute the threadhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAS_PtETw7NYx5DA06KtN9MbsOF1467nks5vNwRrgaJpZM4Y2BE2&data=02|01||be9c4aab6d4e444ced3508d69378394f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636858541903185126&sdata=onogCuseQH0jvqpDlqnSNU5IXvuMabJESMALHKm7HXA%3D&reserved=0.

I just got everything working.  A few things that helped me get past a few errors.  I did a lot at once, so I apologize for no specifics on your exact error.

Made sure to disable MFA on the Global Admin (GA) you are using
Made sure to add the GA to the exception list in Azure AD for the default conditional access Policy "Baseline policy: Require MFA for admins (Preview)"
in Sharepoint Admin (modern), under Access Control, be sure to Allow Access for "Apps that don't use modern authentication" (could take up to 24 hours for this change to propagate).
Don't login using the -useweblogin parameter.    Only run Connect-PnPOnline https://contos0.sharepoint.com

I believe after everything is ran, I can then enable MFA and block access for apps that don't use modern authentication.
Hope this helps.


Logging configuration extends beyond the scope of CLISOPS

clisops version: 0.6.1
Python version: Python3.7
Operating System: Linux

Description
I think the logging configuration for CLISOPS might be a bit too aggressive. I'm noticing the logging call layout in projects that import CLISOPS but aren't specifically calling its functions. It could be that this needs behaviour to be explicitly restricted to the scope of this library to prevent it from showing up where it shouldn't.
What I Did
See below. Notice the warnings formatted using the layout specified in
clisops/clisops/etc/logging.conf:
$ pytest tests
...
tests/test_geo_utilities.py::TestFileInfoFuncs::test_single_file_check 2021-03-10 14:07:18,341 - RavenPy - ERROR - No files found. Exiting.
2021-03-10 14:07:18,341 - RavenPy - ERROR - Multi-file handling for file is not supported. Exiting.
PASSED
tests/test_geo_utilities.py::TestFileInfoFuncs::test_boundary_check 2021-03-10 14:07:18,355 - RavenPy - WARNING - Vector /home/USER/gh/raven-testdata/nasa/Mars_MGS_MOLA_DEM_georeferenced_region_compressed.tiff contains geometries in high latitudes. Verify choice of projected CRS is appropriate for analysis.
2021-03-10 14:07:18,361 - RavenPy - WARNING - Vector /home/USER/gh/raven-testdata/polygons/mars.geojson contains geometries in high latitudes. Verify choice of projected CRS is appropriate for analysis.
2021-03-10 14:07:18,368 - RavenPy - WARNING - Vector /home/USER/gh/raven-testdata/nasa/Mars_MGS_MOLA_DEM_georeferenced_region_compressed.tiff contains geometries in high latitudes. Verify choice of projected CRS is appropriate for analysis.
2021-03-10 14:07:18,370 - RavenPy - ERROR - Unable to read boundaries from unreal.zip
PASSED
...

For more context, this can also be seen in otherwise squelched warnings made visible when importing clisops before xclim:
from xclim import atmos
2021-03-11 15:58:14,790 - pint.util - WARNING - Redefining 'degree_Celsius' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,791 - pint.util - WARNING - Redefining '°C' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,792 - pint.util - WARNING - Redefining 'celsius' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,792 - pint.util - WARNING - Redefining 'degC' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,793 - pint.util - WARNING - Redefining 'degreeC' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,793 - pint.util - WARNING - Redefining 'C' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,794 - pint.util - WARNING - Redefining 'deg_C' (<class 'pint.definitions.UnitDefinition'>)
2021-03-11 15:58:14,795 - pint.util - WARNING - Redefining '[speed]' (<class 'pint.definitions.DimensionDefinition'>)

Closed with #216


Log Improvement in ManifestWork code

This is a follow up commit after #PR 494.

Hi Shyam, You are correct, no need to  pass log for (2) DRCluster. Made that change.
The changes are made in those functions when manifestwork is initiated by drpc as the signature for createOrUpdateManifestWork has an extra parameter drpc log which has drpc reconciler id.
I am not sure If I understand  clearly.
1 In  createDRPCInstance we can pass DRPCInstance.mwu.Log as log


Fixes #259: Add parseYaml function to jsonnet

Fixes issue #259
Proposed Changes

Hello :) Can you also update the README with the new function and add a test for it in tests?

For testing it, I have to create a sample yaml file. Where should I create it? Also readme doesn't contain information about other native functions, should I write about them too?

You could put the sample yaml file under a new folder in tests/test_resources/ :)

@uberspot Are any other changes needed?

@uberspot done requested changes.

:+1: Nice!


pass the sort of the cell collection variables to the java backend

@daejunpark please check if this fixes the issue with ... in <accounts> in the proof specs. I tried tests/proofs/hkg/approve-spec.k locally, and it works.

Jenkins: test this please

Jenkins: test this please

I have tested this fix, cherry-picking this on the top of the proofmodule branch, and adding the ... in the <accounts> cell of the spec. It works well! Thanks!!

looks like some kind of bug in the OCAML backend was introduced by this change to SortCells. I will investigate tomorrow.

So I found a fix for the issue in the OCAML backend and I can apply that when the rest of the diff is ready, but there is some kind of much deeper bug involved here. I don't understand exactly what is going on but this change makes the java backend's associative matching algorithm break very badly, and I was able to track it down to the fact that it changes the behavior of line 73 of AssocCommToAssoc.scala, but based on that information, I have no idea how it worked previously, and my attempts at a simple fix produced very bizarre behavior, so I have to give this up. @andreistefanescu will have to investigate this further. I am going to rerun the tests now so you can see that ktest fails.

Jenkins: test this please


Visual Glitches

Describe the bug
Theres a bug/glitch where theres just visual glitches everywhere
SEE screenshots

Expected behavior
to work
Observed/actual behavior
Visual glitches
Steps To Reproduce
Play minecraft see visual bug
Modpack version
2.1.0-beta 1.17.1
Launcher
Minecraft Launcher (vanilla)
Install method
Fresh install/new profile
Additional context
playing 1.16 with fabulously optimized works perfectly fine, i think theres either a conflicting mod in the 2.1.0 beta 1.17.1 modpack or a mod is outofdate and isnt working.

I removed all the mods and just ran sodium with sodium extras and i think sodium is having issues

I see you already reported it, that's great. https://github.com/CaffeineMC/sodium-fabric/issues/774

yeah a lot of older gpus are currently having issues with sodium 1.17.1+0.3.0
might want tot wait before proceeding with install

like wait for sodium to update?

Well, as Jelly said on her Discord, this appears on some machines and works perfectly on others, so it is just a matter of trying and seeing.
I'll continue updating to newest versions of Sodium+Iris whenever they come available and I'll keep the FO version marked as beta as long as there still are major issues.

Yeah my bug report got added to a list of same bug reports.

Jelly provided a test build for those affected: https://discord.com/channels/602796788608401408/651120262129123330/866918012896804865

Yeah unfortunately it isnt working either lots of people are still saying that the dev builds didnt work either.

Jelly provided a test build for those affected: https://discord.com/channels/602796788608401408/651120262129123330/866918012896804865

This Discord Link Is not workin for me its showing failed to load messges my internet connection is fine

@Rajbir98 It's an old build.
Go to https://jellysquid.me/discord and download the latest jar from #sodium-dev


Updating About me

I was reading poliastro docs and found a small typo ;)

Codecov Report

Merging #164 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #164   +/-   ##
=======================================
  Coverage   82.25%   82.25%
=======================================
  Files          24       24
  Lines         885      885
  Branches       66       66
=======================================
  Hits          728      728
  Misses        137      137
  Partials       20       20

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 fde9d86...a286ecf. Read the comment docs.

Haha good catch Fran! This could also be labeled as a bug :) Merging!


[Question]: What are the different reranking modes available while using multiple retrievers

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I am using QueryFusionRetriever to combine lexical+semantic search. I am using ElasticSearch vector db to store the embeddings.
I see in the  QueryFusionRetriever there is a mode parameters, i just found in the documentation around 'reciprocal_rerank'.
what other modes are supported, can we have those with explanations for reference?
Also while using reciprocal rerank the score are very low, how to read these scores? any guide would be helpful.
retriever = QueryFusionRetriever(
[vector_retriever, bm25_retriever],
similarity_top_k=3,
num_queries=1,  # set this to 1 to disable query generation
mode="reciprocal_rerank",
use_async=True,
verbose=False,
)

Looks like dosu bot isn't giving the right answers 😓
There are just 2 modes right now:
class FUSION_MODES(str, Enum):
    """Enum for different fusion modes."""

    RECIPROCAL_RANK = "reciprocal_rerank"  # apply reciprocal rank fusion
    SIMPLE = "simple"  # simple re-ordering of results based on original scores

but you can add on a number of different node postprocessors for re-ranking. Read more here: https://docs.llamaindex.ai/en/stable/module_guides/querying/node_postprocessors/root.html

Looks like dosu bot isn't giving the right answers 😓
There are just 2 modes right now:
class FUSION_MODES(str, Enum):
    """Enum for different fusion modes."""

    RECIPROCAL_RANK = "reciprocal_rerank"  # apply reciprocal rank fusion
    SIMPLE = "simple"  # simple re-ordering of results based on original scores

but you can add on a number of different node postprocessors for re-ranking. Read more here: https://docs.llamaindex.ai/en/stable/module_guides/querying/node_postprocessors/root.html

@Disiok  Thanks for your reply.


Fixes #1441 use raw string when given no arguments

Check List:

Fixes #1441
Unit tests : NO
Javadoc with a code example (API only) : NA

Thanks @asolntsev, can you add a test for the overridden isEqualTo that checks it delegates to its parent method, it should be possible to do so with mockito with callRealMethod or spy

@joel-costigliola As I know, it's not possible to verify that super.something() was called.
And even if it was possible, would be such test really useful?
A much better alternative is to add a test in Kotlin which would actually fail with current AssertJ implementation. But do you want to introduce kotlin to AssertJ project?..

just want to make sure that the method is called in tests (which probably is but let's verify that).
Although I like Kotlin, I don't want to have a dependency on it.

Integrated thanks @asolntsev!


Calibration of sensors not allowing negative offset in HA

HA version 2022.12.0
ThermIQ HACS - 2.05
I've noticed that I'm not able in HA to calibrate my sensors with a negative value. The input number does not go negative if I press the - (minus) in the Lovelace Card, positive works and a MQTT is sent. I can manually send a MQTT with a negative value and in ThermIQ 2 web I see that it has taken effect and can confirm that the sensor value is 1 degree lower.

Pictures from HA and ThermIQ web

I've noticed the same and it is fixed in next release

Fixed


Allow validating body content using MockMvc's validation or XMLUnit

Right now it's not possible to validate XML body using contracts.
Existing MockMvc code cannot be rewritten into contract:
   .andExpect(content().xml(readFileToString("/responses/response.xml"), true))

That's a pity because generated test is using ResponseOptions but API returns MockMvcResponse which has MockMvc instances to access, technically it would only require a simple call to validate body content. I had tried to write custom execute method but there is not way to pass file content there.
Underneath MockMvc is using XMLUnit maybe new matchers API can support that library or MockMvc validation.

I don't follow. We don't generate MockMvc tests. There's another issue related to XML and we're planning on adding any sort of support for XML. We can try to generate stuff in the similar way we do it with JSON via https://github.com/marcingrzejszczak/xmlassert .
Until now, what you can do is to write MockMvc tests manually and assert the XML stuff manually.

Yes, tests doesn't use MockMvc's validation functionality, technically it can be used out of box.
I think XMLUnit support would be enough.

Yes, tests doesn't use MockMvc's validation functionality, technically it can be used out of box.

Yeah, if you write the full support for MockMvc for generated tests which we are not planning on doing. So there's nothing that comes out of the box for now. But maybe I don't follow what you're saying.

I have tried to implement custom method for server side validation but it looks like there is something wrong:
body(execute("equalsXml(\$it, \"${StringEscapeUtils.escapeJava(file('responses/ok.xml'))}\")".toString()))

but it generates:
assertThat(responseBody).isEqualTo("EXECUTION>>equalsXml($it, ");

Would you be for using XmlUnit for xml server side validation?

I don't see any added value. If Wiremock is already using it for matching why not to use it for validation?

I would prefer just simple XMLUnit  in the beginning

So you can create your own way of generating tests - spring cloud contract is fully pluggable.

I was thinking to add default XML comparing by XMLUnit... but you don't want it?

I explained it to you in my previous comments.


Fixed the 1.20.1 respawn bug with capabilities issue. Fixes #3747

This request fixes the issues I was having and seeing extensively and causing problems with my players.  We've had to spend more time than I care to admit to debug and restore inventories and player data to resolve this issue.
As a result of the headache, we've gone ahead and found what was causing it and issued a local fix ourselves, then forked the repo and submitted the patch to you.  Hopefully, this prevents others from having this issue and our headache as well.

thank you :) we we had the same issue on our server

thank you :) we we had the same issue on our server

You're welcome. I'm hopeful that it will help many people and that LuckPerms will accept the PR due to the nature of the bug.

The reason I haven't merged this yet is because I'm not sure whether this is really the correct fix. I think we need to be careful about "revivingCaps" when the player genuinely is in an inactive state. It feels to me like Forge need to fix the capabilities system at their end if it is causing problems. As far as I can tell, LuckPerms' usage of the system is correct.
Maybe we should just switch to a centralised Map<UUID, User> instead of using the capabilities if it is going to continue to be painful

I agree Forge is to blame. We sadly have to work around them sometimes due to the nature of that project. Most will likely move to NeoForge, and I have higher hopes for them. For now, while it's not the ideal fix, I didn't want to re-work your entire plugin for any platform this touches.
Changing it to a map might be better; however, if we go that route vs this patch, you'd have to backport it to 1.20. and 1.20.1 vs. forward porting. The choice is ultimately up to you.

Hello, i was be testing this problem on my main servers and trying fix error in this way:
UserCapabilityListener
private final ThreadLocal<Player> previousPlayerData = new ThreadLocal<>();

@SubscribeEvent
public void onPlayerClone(PlayerEvent.Clone event) {
    Player previousPlayer = event.getOriginal();
    Player currentPlayer = event.getEntity();

    previousPlayer.reviveCaps();
    try {
        UserCapabilityImpl previous = UserCapabilityImpl.get(previousPlayer);
        UserCapabilityImpl current = UserCapabilityImpl.get(currentPlayer);

        current.initialise(previous);
        current.getQueryOptionsCache().invalidate();
    } catch (IllegalStateException e) {
        // continue on if we cannot copy original data
    }
    previousPlayerData.set(previousPlayer);
}

@SubscribeEvent
public void onPlayerRespawn(PlayerEvent.PlayerRespawnEvent event) {
    if (previousPlayerData.get() != null) {
        previousPlayerData.get().invalidateCaps();
    }
}

UserCapabilityImpl:
Remove the getCapability(Player): LazyOptional method and make this:
public static @NotNull UserCapabilityImpl get(@NotNull Player player) {
    return (UserCapabilityImpl) player.getCapability(CAPABILITY).orElseThrow(() -> new IllegalStateException("Capability missing for " + player.getUUID()));
}

public static @Nullable UserCapabilityImpl getNullable(@NotNull Player player) {
    return (UserCapabilityImpl) player.getCapability(CAPABILITY).resolve().orElse(null);
}

Now let me explain why I did it that way. First, when the player respawns, Capabilities are attached, after which the player is cloned. At the end of all this action, permissions are sent to the client. I noticed that with previous fixes, you tried to get data from the player's Capability using reviveCaps and then invalidateCaps.  Thus, the Capability of your player (the expected one) becomes invalid already at the stage of its creation (in our case respawn). I removed all the extra code and divided Clone into some 2 phases: Clone - copying data and restoring old ones, and PlayerRespawn, when the player is already fully ready to appear in the world and the need for old data is not needed.
Perhaps ThreadLocal will be superfluous in my version, or maybe not, I did not check it thoroughly. But in the current variation, there is no error at entry or after death.
If many people's mistake is corrected due to my extraordinary decision and @lucko himself or someone from the LuckPerms team gives a comment or consent to the implementation, then I will be ready to provide a PR for this code

A similar fix was added in https://github.com/LuckPerms/LuckPerms/commit/5c1ea5633ed5986c02d3fa1ccf187638d143c091 which should fix the problem :)


Added IOWOW key/value storage engine

Please add description of IOWOW key/value storage engine. http://iowow.io
Thank you!

Good luck!


Better Vault Value checker interface

Right now the vault vault checker, used to verify that no MEV has happened strategist allocations, the checks the upper and lower bound of allowed totalSupply and vaultValue. This is safe, and safer that what we had before, but has two disadvantages.
The metastrategy creates or removes large amounts of OUSD during moves in and out. The amount of these changes can be affected by other users in the curve pools, and can dwarf the actual profit and loss.  Because of this, an unrelated trade in the OUSD/3pool can drastically change the amounts, and cause a previously valid strategist allocation transaction to fail the checks.
Secondly, the parameters are a little bit of a pain to check. For example, the last allocation looked like this:
(
-1378302004686863985848902,  // Low value change
-1376802004686863985848902,  // High value change
-1378399395878459980498569,  // Low supply change
-1376899395878459980498569,  // High supply change
)

This does show that we are thinking about roughly a 1.3 million reduction in supply from moving funds out of the metastrategy, but makes it extremely difficult to see the far more important expected profit/loss of the move, and is very sensitive to metastrategy changes.
An alternate API for the vault value checker might be:
(
-200, // Expect to lose 200 dollars on these allocations
100, // We are okay with a +- 100 dollar difference between expected and actual PL.
-1,378,000, // Vault value is expected to go down by 1.3 million
100,000 // We are okay with a +- 100,000  on the expected combined vault value / total supply change.
)

This is much more readable. 👍

This is live for OETH. Be good to go live with an OUSD version and switch to it.


How about using the 'shelf_cors' package

How about using the 'shelf_cors' package instead of a custom version? I am open to any suggestions to make it more suitable for dart-services.

Looks good; we weren't aware of the package. It looks like shelf_cors supports OPTIONS and we can (optionally) use our own custom CORS headers if we want. Adding @lukechurch to see if he has opinions, but generally it looks like it would work just fine. If you want to send a PR, patches welcome :)

OK, I will send you a PR tomorrow.

Done - https://github.com/dart-lang/dart-services/pull/61.


Flex-Wrap Breaks when adding margin to child elements

All, it seems there is an issue with flex-wrap when adding margins. Here is how to repro.

Add this code from the grid example (slightly modified to make width 1/2 on LG):

<template>

</template>

Then modify it to this:
<template>

</template>

You'll notice in the first example you'll get 3 rows of 2. In the second example after adding margins you will get 6 rows of 1 that are half width.
I'm using VueJS and testing this with Sizzy, confirmed same issue in Chrome.

Margins are added to the outside of an element, so when you have two elements with w-1/2 and margin on each of them, the total width is greater than 100% causing things to wrap.
Use padding instead of margin:
https://tailwindcss.com/components/grids/#column-spacing

Thanks Adam! I figured it was me, which is why I created this in the Discuss section.


Subscription api proposals

Overview

Changes direction of dependency between Plan & Subscription
Defines batch edit based on RFC6902 JSONPatch
Add expansion for subscriptions

Notes for Reviewer

3 endpoints are tagged as Proposal. This is to denote that at this point its unclear if they'll make it to the final api. (They're not needed, there's equivalent functionality elsewhere, but they'd make sense to have in a REST API).

I am trying to fully understand the patch language.

I think you mentioned there can be no rate card with the same feature (a key restriction). I assume the same is true about featureless rate cards with the same price? Is this an issue? Probably not but note to myself: we need to enforce this on the UI.
Are the above (key, feature, price) restrictions per subscription or per phase? I assume per phrase and the "edit id" for the card also includes the phase id.

There are 2 things here:

You can only have access to, and pay for, a given feature once.
RateCards & SubscriptionItems should have a Key based on this proposal which is locally unique. For that I suggest we use the featureKey if present, if not then a key has to be provided/generated.

I am not sure I understand the flat patch language for the nested structure of rate cards and phases. How do I edit a phase? For example if I want to extend a phase I send the whole phase with all rate cards and only the duration field changed? Networking is still a none issue imo but storing patches like this may defeat the purpose.

The patch language is only used for the RateCards (there are 3 extra EPs only for the phase management). We could extend it for the phases as well, my issue is that if you want to add phases, it's really not intuitive to reference them as json document members. It would be much more intuitive to reference them array-like, so you could insert phases, remove phases etc... but then (if we use the same path) the SubscriptionItem references wouldn't be stable.

What if I delete a phase? Do I send a phase delete patch or do I also send all enclosed rate cards delete patches too to make it work?

see above, there's a dedicated ep defined

Same goes about adding a phase: is it a single big patch or do I need to flatten it out? If so, how?

same

What about other interdependencies in the subscription structure. For example if I have a discount applying to a rate card and I delete the rate card do I simply send the rate card delete and let the api make it work or do I also have to send the discount patch without the rate card id?

Discounts are not yet defined in the API, but a fair question. I suppose we could think of a Discount's appliesTo field as a filter and not as a reference, it shouldn't be an issue to delete the SubscriptionItem


Problem with Home Assistant 2023.

System Health details
System updated to 2024.2
Checklist

[X] I have given details of my install including what type of system airsource/ watersource or groundsource
[X] I have provided logs showing any errors.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
Starting Home Assistant 2024.2.0 failes due to some mastertherm dependency for Python.
Reproduction steps
N/A
Logs
Starting Home Assistant 2024.2.0 failes due to some mastertherm dependency for Python.

2024-02-07 20:17:52.642 ERROR (SyncWorker_4) [homeassistant.util.package] Unable to install package masterthermconnect==2.2.10: ERROR: Ignored the following versions that require a different python version: 1.1.0 Requires-Python <3.12,>=3.9; 1.1.0rc0 Requires-Python <3.11,>=3.9; 1.1.0rc1 Requires-Python <3.11,>=3.9; 1.1.0rc10 Requires-Python <3.12,>=3.9; 1.1.0rc11 Requires-Python <3.12,>=3.9; 1.1.0rc2 Requires-Python <3.11,>=3.9; 1.1.0rc3 Requires-Python <3.11,>=3.9; 1.1.0rc4 Requires-Python <3.11,>=3.9; 1.1.0rc5 Requires-Python <3.11,>=3.9; 1.1.0rc6 Requires-Python <3.11,>=3.9; 1.1.0rc7 Requires-Python <3.11,>=3.9; 1.1.0rc8 Requires-Python <3.11,>=3.9; 1.1.0rc9 Requires-Python <3.12,>=3.9; 1.1.1 Requires-Python <3.12,>=3.9; 1.2.0 Requires-Python <3.12,>=3.9; 1.2.0b0 Requires-Python <3.12,>=3.9; 1.2.0b1 Requires-Python <3.12,>=3.9; 2.0.0 Requires-Python <3.12,>=3.9; 2.0.0a0 Requires-Python <3.12,>=3.9; 2.0.0a1 Requires-Python <3.12,>=3.9; 2.0.0a2 Requires-Python <3.12,>=3.9; 2.0.0a3 Requires-Python <3.12,>=3.9; 2.0.0a4 Requires-Python <3.12,>=3.9; 2.0.0a5 Requires-Python <3.12,>=3.9; 2.0.0b1 Requires-Python <3.12,>=3.9; 2.0.0b2 Requires-Python <3.12,>=3.9; 2.0.0b3 Requires-Python <3.12,>=3.9; 2.0.0b4 Requires-Python <3.12,>=3.9; 2.1.0 Requires-Python <3.12,>=3.9; 2.1.0b0 Requires-Python <3.12,>=3.9; 2.1.0b1 Requires-Python <3.12,>=3.9; 2.1.0b2 Requires-Python <3.12,>=3.9; 2.1.0b3 Requires-Python <3.12,>=3.9; 2.1.0b4 Requires-Python <3.12,>=3.9; 2.1.0b5 Requires-Python <3.12,>=3.9; 2.2.0 Requires-Python <3.12,>=3.9; 2.2.1 Requires-Python <3.12,>=3.9; 2.2.10 Requires-Python <=3.12,>=3.9; 2.2.2 Requires-Python <3.12,>=3.9; 2.2.3 Requires-Python <3.12,>=3.9; 2.2.4 Requires-Python <3.12,>=3.9; 2.2.5 Requires-Python <3.12,>=3.9; 2.2.5b0 Requires-Python <3.12,>=3.9; 2.2.5b1 Requires-Python <3.12,>=3.9; 2.2.6 Requires-Python <3.12,>=3.9; 2.2.6b0 Requires-Python <3.12,>=3.9; 2.2.6b1 Requires-Python <3.12,>=3.9; 2.2.6b2 Requires-Python <3.12,>=3.9; 2.2.6b3 Requires-Python <3.12,>=3.9; 2.2.6b6 Requires-Python <3.12,>=3.9; 2.2.6b7 Requires-Python <3.12,>=3.9; 2.2.7b0 Requires-Python <3.12,>=3.9; 2.2.9 Requires-Python <=3.12,>=3.9; 2.2.9b1 Requires-Python <=3.12,>=3.9
ERROR: Could not find a version that satisfies the requirement masterthermconnect==2.2.10 (from versions: 0.1.0)

Diagnostics dump, if available
N/A

Currently don't upgrade to 2024.2.0, if upgraded downgade to the previous backup

I just upgraded to 2024.2.0 and running into this same issue. Not so much of a techy, but did some search on what is the best (read: safe) way to downgrade this. Saw some info on running a command through the Terminal. Would you be so kind to confirm the steps for downgrading this the right way?
Would be much appreciated.
Kind regards,
Gerrit

Ok, I figured it out. Just ran a partial restore of only the OS to 2024.1.6. I can confirm Mastertherm integration is back up and running now...

Fixed and releasing next version, issue was that python released 3.12.1 and the connect module only went up to 3.12.  Even though Home Assistant is still on 3.11 this cased a dependancy issue.


Fix splunk user home attribute being ignored

Even if set in the attributes file, $HOME ended up being /home/splunk otherwise

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@mikkoc

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Thanks @mikkoc


Make this a keyword

Currently this pollutes the function scope.

3e89e7ab0fda9adc39d4f8d698f87dfd9db9074e
Needs to be added in the reference

3e89e7ab0fda9adc39d4f8d698f87dfd9db9074e


feat(#262): fix the bug with java 17 support - text blocks

Add Java 17 support for javaparser library
Closes: #262

PR-Codex overview
Detailed summary

Added a new Java test class with Java 17 features.
Modified the parse method in JavaParserClass to set the language level to Java 17 when parsing Java files.
Added license headers to the test resource files.

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

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)

@rultor release, tag is 1.1.1, title is 1.1.1.

@rultor release, tag is 1.1.1, title is 1.1.1.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 1.1.1, title is 1.1.1.

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)


Validate routing container port

Whenever a routing path references an unexposed container port, we will now mark the pod as not routable instead of creating a route and failing at runtime.

lgtm


Create train.py

Makes it simpler and faster to quick-start training:
python train.py --config="tacotron2_config.json"

@zachwe , tested and tried new train.sh with no problems!

bash train.sh "tacotron2_config.json"


Add support for assembly element sets

Assembly element sets allow more than 1 part instance to be analysed. This is needed when looking at the knee, where we want to see both the femur, tibia and both implants on the same virtual xray

Done in v2.0


BuddyBuildSDK Integration

@SidBala, here's the BuddyBuildSDK. You'll get an email that lets you install and try out the build

Feedback from<EMAIL_ADDRESS>: Test test test


DRILL-4091: Support for additional gis operations in gis contrib module

This based off of PR https://github.com/apache/drill/pull/258. I have attempted to address the comments from the PR left by @cgivre
Some additional notes:

in STUnionAggregate.java, there was a comment mentioning null handling, but I'm unable to find any null handling support when defining interfaces for aggregate functions.
I have removed one of the failing tests from TestGeometryFunctions.java. The failing tests were attempting to test non-point geometry on functions that require it. While returning Double.NaN seems appropriate for the situation, I found two issues with testing this:

DefaultSequelHandler.transform() casts the results of the Float8Holder to BigInteger which does not have a representation for NaN. As a result, this is returning a UserException giving a system error, which would be the expected behaviour. However, I can't seem to test for this exception given the test builder that is used.
No other tests in the file are testing for invalid geometry. This is perhaps a larger issue.

@cgivre added comments

@ChrisSandison This isn't a review comment, but @k255 was working on a format plugin for ESRI shape files.  Would you want to take a look and see if we could get that in as well?  I could see that being VERY useful once we get all this GIS functionality in.

@cgivre do you mean including that in this PR?

If you want to include it on this PR that would be great, but I’m fine with including it in a separate PR as well.  I want to get this functionality into Drill asap, so it probably would be quicker to do a separate PR.

On Apr 10, 2018, at 10:32, Chris Sandison<EMAIL_ADDRESS>wrote:
@cgivre https://github.com/cgivre do you mean including that in this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/apache/drill/pull/1201#issuecomment-380121592, or mute the thread https://github.com/notifications/unsubscribe-auth/AFQfvmYpl5h9CH5cLHPn7aweR0KUhbrTks5tnMJkgaJpZM4TFMOI.

@cgivre I agree. Would you be able to open a JIRA ticket with any details of the work that @k255 was doing for it?

@ChrisSandison Done.  Here's a link to the JIRA https://issues.apache.org/jira/browse/DRILL-6319

@cgivre I'm also needing to use ST_DWithin (https://postgis.net/docs/ST_DWithin.html). I'm going to add it to this PR.

LGTM +1
One thing, which I suspect may be a problem with my environment, not this PR, but when I try to build the module with the tests, I get the following errors:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:2.6:single (source-release-assembly) on project drill-gis: Error reading assemblies: Error locating assembly descriptor: src/main/resources/assemblies/source-assembly.xml
[ERROR]
[ERROR] [1] [INFO] Searching for file location: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml
[ERROR]
[ERROR] [2] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.
[ERROR]
[ERROR] [3] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:2.6:single (source-release-assembly) on project drill-gis: Error reading assemblies: Error locating assembly descriptor: src/main/resources/assemblies/source-assembly.xml

[1] [INFO] Searching for file location: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml

[2] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.

[3] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:213)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:154)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:146)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:956)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:290)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:194)
    at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:497)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:289)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:229)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:415)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:356)
Caused by: org.apache.maven.plugin.MojoExecutionException: Error reading assemblies: Error locating assembly descriptor: src/main/resources/assemblies/source-assembly.xml

[1] [INFO] Searching for file location: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml

[2] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.

[3] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.
    at org.apache.maven.plugin.assembly.mojos.AbstractAssemblyMojo.execute (AbstractAssemblyMojo.java:453)
    at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:208)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:154)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:146)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:956)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:290)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:194)
    at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:497)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:289)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:229)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:415)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:356)
Caused by: org.apache.maven.plugin.assembly.io.AssemblyReadException: Error locating assembly descriptor: src/main/resources/assemblies/source-assembly.xml

[1] [INFO] Searching for file location: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml

[2] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.

[3] [INFO] File: /Users/cgivre/github/drill-dev/drill-gis/drill/contrib/gis/src/main/resources/assemblies/source-assembly.xml does not exist.
    at org.apache.maven.plugin.assembly.io.DefaultAssemblyReader.addAssemblyFromDescriptor (DefaultAssemblyReader.java:293)
    at org.apache.maven.plugin.assembly.io.DefaultAssemblyReader.readAssemblies (DefaultAssemblyReader.java:109)
    at org.apache.maven.plugin.assembly.mojos.AbstractAssemblyMojo.execute (AbstractAssemblyMojo.java:449)
    at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:208)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:154)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:146)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:956)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:290)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:194)
    at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:497)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:289)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:229)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:415)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:356)
[ERROR]
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
Charles-MacBook-Pro:gis cgivre$

@cgivre I had some issues between this and a previous branch I was working on, but I fixed it by upgrade the base image that I was building it on. I ended up building it on maven:3.5.3-jdk-8

In that case,  LGTM +1
Thank you for your work on this!.  Are you going to take a look at the ESRI format plugin?

@arina-ielchiieva Should I do the commit or would you like to do that.

Thanks @arina-ielchiieva
@ChrisSandison Can you please:

Squash commits into one
Apply the Drill coding styles
Fix the license headers

And we'll get it committed!  Thanks!

@cgivre updated and squashed

@ChrisSandison,
The PR is failing the TravisCI build due to unused import statements:
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STYFunc.java:29:8: Unused import - com.esri.core.geometry.Geometry.Type.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STYFunc.java:30:8: Unused import - com.esri.core.geometry.ogc.OGCPoint.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STXFunc.java:20:8: Unused import - java.sql.Types.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STXFunc.java:31:8: Unused import - com.esri.core.geometry.Geometry.Type.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STXFunc.java:32:8: Unused import - com.esri.core.geometry.ogc.OGCPoint.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STUnionAggregate.java:33:8: Unused import - com.esri.core.geometry.SpatialReference.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STGeomFromText.java:22:8: Unused import - com.esri.core.geometry.ogc.OGCGeometry.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STGeomFromText.java:32:8: Unused import - java.nio.ByteBuffer.
/home/travis/build/apache/drill/contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STTransform.java:28:8: Unused import - org.apache.drill.exec.expr.holders.NullableVarCharHolder.

Could you please remove these, squash the commits and I think we're done.

+1

@ChrisSandison
Thank you for your contribution to Drill!

@parthchandra @cgivre why has this been closed?

It's committed into master.
On Sat, Jun 2, 2018 at 4:46 AM, Chris Sandison<EMAIL_ADDRESS>wrote:

@parthchandra https://github.com/parthchandra @cgivre
https://github.com/cgivre why has this been closed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/drill/pull/1201#issuecomment-394081247, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGOgcDFWkgQ38FXHvrsm5UbVtd1gLaNmks5t4nslgaJpZM4TFMOI
.


Update license to make it easier for github to recognize the license

Use the template from https://opensource.org/licenses/BSD-3-Clause This time it should work.

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #12   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         138    138
=====================================
  Hits          138    138

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 5a7beab...102e845. Read the comment docs.

I just noticed: you need UT-Battelle, LLC instead of Oak Ridge National Laboratory.

I changed it ORNL -> The Cool Kids


requires Windows agent

page should state a Windows agent is required, both source and destination agents

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

ID: 5e0ef2cf-ed52-6ded-bf14-4820d73b4ed0
Version Independent ID: 7781e0f0-43aa-9008-abda-c03a37e2d7b4
Content: Windows Machine File Copy task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/windows-machine-file-copy.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

We believe the task name is quite descriptive.

in the context of Azure the requirements to run this task are not clear. For example, as I understand it, Powershell may be run on Windows or Linux, and bash scripts may be run on WIndows or Linux. Isn't it a simple matter to clearly state the requirements?

I just ran into this same issue. Based on my reading of the docs I thought only the destination machine needed to be windows. Especially after reading AzureFileCopy docs which clearly note the shortcoming for linux agents.


Improve performance by ~20-25% and fix syntax error in draw.py

This commit aim is to solve performance issues #17 and fix #18
The reason why the performance improvement is working is that calling subprocess.run or Popen in every drawing cycle is very expensive, because subprocess will call out to the shell which will execute xdotool or xprop after which we need to parse the output. Instead of these slow calls we should use Wnck.Window methods to get the same properties.
This part of the refactoring caused a 5-10% performance improvement,
In the other hand I've removed the self.input_shape_combine_region(cairo.Region()) from _on_draw which caused another 15-20% improvement on CPU usage.
I'm not sure why that line was there, so I've left it commented out with a TODO note for future removal.

The self.input_shape_combine_region(cairo.Region()) line was there for the desktop stress toy. I'm probably gonna remove that feature as it is not needed and is tanking the performance. Also thank you for letting me know about Wnck I didn't know it existed. Thanks for the PR!

There were a few problems like not checking if Wnck.get_active_window() returned None but no worries I've fixed them.

Thank you for merging and fixing the return None problem, I've only tested xborders with open windows, its my bad :/


About the batchsize and epoch

Hello, thank you for your work.Your code only has places for iteration, but not for batchsize and epoch.So how are these two parameters set?
Thank you very much.

For each iteration, we fed one image to the model, along with 15 augmented GT pairs and at most 60 Neg pairs (if there were enough).
We didn't use epoch to measure the training process.

But in the code, the shape of fc7_H is (16, 1024)，and the sp is (46, 1024).So I want to ask what does 16 and 46 mean?

The shapes of fc7_H and some other features varies during the whole training process, depending on the number of pairs fed along with the image. In this example, 16 and 46 means this image is fed to the model along with 16 GT pairs and 30 Neg pairs (16 + 30 = 46).

Also, the shapes of fc7_H and sp mentioned ((16, 1024) and(46, 1024)) were probably wrong, since fc7_H should have a size of (?, 2048) and sp should have a size of (?, 5408).

Thank you for your answer.I made a mistake because I changed the dimensions of both to 1024.But (?, 2048), (?,5408 )is the shape of the tensor that I'm going to put out directly, and then I'm going to put it into numpy form, and the outputs are (16,2048) and (46,5408).

GT pair is given directly by the label, right? So how does Neg pair come about?

Yep. And if you output the shape in another iteration, you will find their shapes vary.

GT pair is given directly by the label, right? So how does Neg pair come about?

You can refer to #25 and #30 for this question.

Thank you very much. Your answer helped me a lot


It's nicer to show year and month opposite in month header

Hi, I love this package cuz its conviniet and we can customize well.
One thing.
I have issue and want to suggest that month header will be able to show month and year opposite style.
defalut
=> 2月 2023年

hope
=> 2023年 2月

Defalut expression is not fine in Japanese. So if we can use Widget like this , it will be great.
ScrollableCleanCalendar(
  monthHeader : MonthHeader.monthAndYear
  monthHeader : MonthHeader.yearAndMonth
)

I know there is dayBuilder, but it will be more usefull.
please consider it. thank you.
dai10512

Just this way is more flexible and use easy.
monthBuilder: (context, String month, String year) {

monthBuilder will give you a string of MMMM yyyy for now.
I use the DateFormat format MMMM yyyy to yyyy 年 MM 月.
I think this way is better.
monthBuilder: (context, String locale, DateTime date)

@tatsuyuki25
Thank you. That is better and I hope so!

@rodrigodobbin @dai10512 Can try to modify it?

Same issue #42