bake version info into binary for logging

we have version info in the tags we push, but we should probably also bake it into the binary for logging. we can use a golang x-def and update the build commands.
ideally this should be fully sortable git info, see https://github.com/kubernetes-sigs/kind/issues/2618, and also a starting non-fully-sortable version of embedding this info in a similar makefile in that repo as a starting point.

/sig k8s-infra
Ref: https://github.com/kubernetes/k8s.io/issues/3411
Possible to use https://tip.golang.org/doc/go1.18#debug/buildinfo ?

it might be if we upgrade to 1.18!
/help

Possible to use https://tip.golang.org/doc/go1.18#debug/buildinfo ?

TLDR: no.

That package is used to read build info from entire (on disk or embedded in memory) binaries. The main debug package can be used to get info though.
... but buildinfo for untagged versions / current development has the main module version just as (devel)
.... and even the buildinfo build settings metdata just has vcs ("git"), commit hash, and timestamp, we want something closer to git describe which is sortable.

As long as we can figure out how to get ko to pass these as well, we can just set xdef with the metadata ourselves in the build target(s), kind has an example that is close to what we want.

/remove-lifecycle stale

TODO: ban lifecycle bot from this repo.

I think this is deep on the backlog, it sounds nice to have, but we also have other ways to see what cloudrun is up to and we're only using tagged images in production.
Ordinarily for help wanted issues the author would probably help guide contributors, but I am going on vacation for a few weeks and will be relatively unavailable until sometime after July 25th.
@ameukam and @dims are repo approvers / admins and should be able to help in the meantime.
See also https://kubernetes.slack.com/archives/CCK68P2Q2/p1656629484801139?thread_ts=1656626472.238269&cid=CCK68P2Q2 in Kubernetes slack.

/retitle bake version info into binary for logging

we're not changing the version often and we know what image is deployed in cloud run config (and we're using digests so it's an exact image), so this remains low priority.

Cloud run gives us detailed info about revisions to services, I don't think this will actually help and it has the downside of making the build reproducibility dependent on git info. We can already trace back deploys to specific images and the associated commit


Add search by application number

Fixes #3289
What's on the tin.  A few notes:

I preserved the git history from our repository, but can easily squash them all if you'd prefer such a small change be in one commit
I'm not sure where in the documentation to document this feature

Process question: @frjo, do you review and then do the merge now that @theskumar has given an approval, or does @frankduncan merge, or...?
(I did look at https://github.com/HyphaApp/hypha/blob/main/CONTRIBUTING.md first; it points to https://docs.hypha.app/contributors which is a 404 right now, and I remember a discussion with the doc maintenance team about that... I'll go dig that up and see if the path to fixing that stale doc link can be revived.)

@kfogel, it will be @frjo merging the PR.
We recently moved about the docs and it now lives at https://docs.hypha.app/getting-started/contributing/code-contributions/

@kfogel, it will be @frjo merging the PR.
We recently moved about the docs and it now lives at https://docs.hypha.app/getting-started/contributing/code-contributions/

Thanks, @theskumar.  I've submitted PR #3403 to update CONTRIBUTING.md accordingly.


3dtiles Loading with holes

The 3D tiles model shows many holes when the viewing angle changes due to some tiles not being loaded

for example html
`

Title

Hi @wangcaijie, I tried loading your html and couldn't easily see the holes. Can you give the camera position for your screenshot? See the Performance Testing Guide for an explanation of how to capture the camera position.
Thanks!

Fixed in https://github.com/CesiumGS/cesium/pull/11358.


Cannot pair Homebridge bridge in Home app when homebridge-thermostat accessories are present in config.json

Hi, I've been struggling with this for hours. I wrote custom software to control the heating/cooling in our home. It was simple to implement the homebridge-thermostat HTTP API in my code, so I did that:
{"targetHeatingCoolingState":3,"targetTemperature":22.77777777777778,"currentHeatingCoolingState":0,"currentTemperature":23.059999999999995,"targetRelativeHumidity":30,"currentRelativeHumidity":30}

With Homebridge 0.x, I was able to set up homebridge-thermostat using the following settings:
 "accessories": [
        {
            "accessory": "Thermostat",
            "name": "Nursery Thermostat",
            "apiroute": "http://<IP_ADDRESS>:9269/api/nursery",
            "temperatureDisplayUnits": 1,
            "minTemp": 19.444444444,
            "maxTemp": 25.555555556
        },
        {
            "accessory": "Thermostat",
            "name": "Bedroom Thermostat",
            "apiroute": "http://<IP_ADDRESS>:9269/api/bedroom",
            "temperatureDisplayUnits": 1,
            "minTemp": 19.444444444,
            "maxTemp": 25.555555556
        },
        {
            "accessory": "Thermostat",
            "name": "Living Room Thermostat",
            "apiroute": "http://<IP_ADDRESS>:9269/api/living-room",
            "temperatureDisplayUnits": 1,
            "minTemp": 19.444444444,
            "maxTemp": 25.555555556
        },
        {
            "accessory": "Thermostat",
            "name": "Office Thermostat",
            "apiroute": "http://<IP_ADDRESS>:9269/api/office",
            "temperatureDisplayUnits": 1,
            "minTemp": 19.444444444,
            "maxTemp": 25.555555556
        }
    ],

This kinda worked. At first, all 4 accessories showed the proper temp and state, but said No response below the temperature. Sometimes, inexplicably, the No response would go away, and it would say Set to 74º or whatever. My wife, who is also a member of this HomeKit home, also saw the same behavior. We were happy, though confused about the No response.
However, a few hours later, the accessories stopped working on my phone. Then they stopped working on her phone. Then, no matter what I tried – restarting homebridge, re-pairing – it never worked again.
After poking around the Homebridge forums, I saw that Homebridge 1.0 was out, and it claimed to fix some issues with "No response" and multi-user homes. So I upgraded to Homebridge 1.1. Unfortunately this did not help. In fact, now I can't even pair Homebridge with my Home at all. I've tried everything they suggest – wiping the persist directory, etc. It turns out I can only get pairing to work if I remove the homebridge-thermostat accessories from config.json.
 This makes sense to me, as pairing seems to work, but then the Home app decides it "cannot connect" after getting the accessory list.
Any advice? Am I using the wrong settings?

Hello,
Did you tried with only on accessory ? And with the « fake api » to see if everything is working ?
Best


Carthage install issue on Xcode 7.1?

I'm trying to install Few.swift via Carthage and I'm running into some xcodebuild issues. I'm on Xcode  7.1 (7B91b).
Here's my Cartfile:
github "joshaber/Few.swift" "swift-2"

Here's the output when building iOS:
carthage update --platform iOS
*** Fetching Few.swift
*** Fetching SwiftBox
*** Checking out SwiftBox at "c316daebb98237a0c20c5fc7cb92028f44809861"
*** Checking out Few.swift at "12406946e2b11b3f0848235544ff6aaf97618f6b"
*** xcodebuild output can be found in /var/folders/s6/qx12s0856zg2zhsnx_14sqd40000gn/T/carthage-xcodebuild.ZyIMzF.log
*** Building scheme "SwiftBox-iOS" in SwiftBox.xcodeproj
*** Building scheme "Few-iOS" in Few.xcworkspace
** BUILD FAILED **

The following build commands failed:
        CompileSwift normal x86_64 /Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift
        CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
/Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift:69:234: error: type 'Direction' has no member 'Row'
A shell task failed with exit code 65:
** BUILD FAILED **

The following build commands failed:
        CompileSwift normal x86_64 /Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift
        CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)

And here's the output when I build for Mac:
→ carthage update --platform Mac
*** Fetching Few.swift
*** Fetching SwiftBox
*** Checking out SwiftBox at "c316daebb98237a0c20c5fc7cb92028f44809861"
*** Checking out Few.swift at "12406946e2b11b3f0848235544ff6aaf97618f6b"
*** xcodebuild output can be found in /var/folders/s6/qx12s0856zg2zhsnx_14sqd40000gn/T/carthage-xcodebuild.lbVR1l.log
*** Building scheme "SwiftBox-Mac" in SwiftBox.xcodeproj
*** Building scheme "Few-Mac" in Few.xcworkspace
** BUILD FAILED **

The following build commands failed:
        CompileSwift normal x86_64 /Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift
        CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
/Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift:39:21: error: type of expression is ambiguous without more context
/Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift:69:234: error: type 'Direction' has no member 'Row'
A shell task failed with exit code 65:
** BUILD FAILED **

The following build commands failed:
        CompileSwift normal x86_64 /Users/mark/projects/foo/Carthage/Checkouts/Few.swift/FewCore/Element.swift
        CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)

FWIW In Carthage/Checkouts/SwiftBox/SwiftBox/Node.swift, I see this:
public enum Direction: UInt32 {
	case Inherit = 0
	case LeftToRight = 1
	case RightToLeft = 2
}

No idea if that's the enum it's looking at, but if it is it certainly doesn't have a Row member.
Is there some mismatch between Few.swift and Swiftbox going on? Some kind of Cartfile confusion or something? Admittedly, I have never used either of these libraries, so I'm exactly sure what I'm looking at.  This library looks great - I've got a prototype build of a Redux Swift implementation and would love to hook it up with Few.swift.
Thoughts?

Looks like Few.swift should maybe be using the FlexDirection enum here instead of Direction? It looks like the FlexDirection enum was added in this commit: https://github.com/joshaber/SwiftBox/commit/d518d73ab6c5c64372f95079b67f3cfc2e31aab9
So hopefully this is just a matter of pointing Few.swift at the correct commit in Swiftbox?

Sorry for the lack of response :disappointed:

I've got a prototype build of a Redux Swift implementation and would love to hook it up with Few.swift.

That sounds awesome! :metal:
I don't have a lot of time to look into this, but I'd welcome a PR that does the needful.

Right on, I know they keep you folks busy at GitHub. I was able to pin the various Carthage stuffs to where things installed via https://github.com/mwise/Few.swift/blob/swift-2.1/Cartfile.resolved. That's likely missing a bunch of new stuff, but at least it worked. I'll see if I can get a PR based off a later commit working.
Fwiw the Redux/Swift spike I'm working on is at https://github.com/mwise/Redux.swift. It's as alpha as it gets, and would probably be private if I'd known how to dev against private repos via Carthage and if somebody hadn't gone and starred it! My Few adapter for it is still private, but I'll let you know when it's available (it'll be like two files).

Hopefully addressed by #100 :pray:


Create Lakeside School.txt

This is for Lakeside Upper School based out of Seattle,WA
www.lakesideschool.org
~Christopher Lee, Class of 2018

Please email me<EMAIL_ADDRESS>if any validation is needed...
Thanks!

Pardon..didn't mean to close pull request...

Pull request merged, thank you!


Question: GPU Performance Counters reliability on Win10 (WDDM 1.3)

On win10 PC(version > WINDOWS_10_RS3), I paly a video first;

I set EnableGpuPerformanceCounters as 1, but GPU usage column is zero;
I find GPU driver version is WDDM 1.3;
Besides, I check GPU Process Memory counter in performance monitor, but I fail to find record for my target exe (target exe pid is not find), do you know the possible reason?

Duplicate of #847


Members buying Guest WB w CC breaks

**Exception (Integrity constraint violation) 'yii\db\IntegrityException' with message 'SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'tax' cannot be null
Stack trace:
#7 [internal function]: backend\controllers\PaymentController->actionCharge()

Fixed!


Replace dep_results dataset

I use the dep_results dataset for testing the GO functions. It is based on the DEP::UbiLength dataset, which is a MaxQuant proteingroups.txt output.
I find UbiLength to not be well documented enough/biologically interpretable enough to use. I think we should just use a MaxQuant proteingroups.txt file directly from some LFQ PRIDE submission (I am looking for a suitable one now...).

Fully on board 👍


fix(request-purchase): restore changes in requestPurchase function

Restore changes with passing configuration object to requestPurchase function which was removed in https://github.com/dooboolab/react-native-iap/pull/1779
Remove old appAccountToken param

Hi @mmajk55 did you also update requestSubscription which also calls requestPurchase?

Hi @mmajk55 did you also update requestSubscription which also calls requestPurchase?

Googd point, I've updated the PR with those changes

Shouldn't have been merged.


remove process_ignore

Removing the process ignore option. With the new self-inject approach https://github.com/DataDog/auto_inject/pull/266 this option is not relevant anymore. The accessible cmdline is now straight the interpreter /.../bin/(java|python|ruby|dotnet..).
Starting with the test to pass CI, and then will remove in auto_inject

We're removing the customer configuration that this test asserts so there's no additional test to add.
The new injector will target straight the different language interpreters which shouldn't be blocked.
DD_IGNORED_PROCESSES is not publicly documented


Forked PR to Test github action

Since this was just a test PR and the functionality already merged, i close this PR.


sendDirectMessage sends a message from a page to users that have neve…

I added a sendDirectMessage function in order to send messages from a page to a user that have never been contacted. Normally one user can be contacted after the users first interaction. The threadId that this method receives is the COMMENT ID and the text of the messages. No attachments are allowed in this kind of messages. This is very helpful por fan pages that need to contact their audience. Please feel free to request any change or improvement to me. Thanks and nice to meet you.

Thanks for your contribution! I have a couple of comments/questions.
Where does the user get the comment ID to pass to this function?  It strikes me that the name you chose isn't very descriptive.  Would you say a name like startPageConversation or startPageThread captures what you're doing here?  You said this takes a comment ID, but the code refers to it as a threadID.  Does the commentID become the threadID of the resulting chat?
Is there any sensible way we can integrate this functionality into sendMessage?

@toxtli ping! :smile:

Great feedback! This weekend I send you back with the corrections and some documentation. Some users had contacted to me asking about for to use it, so I expect next commit will be ready for production :)

Hi! I add manual changes which you did in two files of this commit. Now I try to use it:
api.sendDirectMessage(msg, item.user_id, function(err, messageInfo){
      if(err){
        error(error);
      }
})

and it returnds to me error:
ERR! ERROR in sendMessage -->  { error: 'JSON.parse error. Check the detail property on this error.',
ERR! ERROR in sendMessage -->    detail: [SyntaxError: Unexpected token <],
ERR! ERROR in sendMessage -->    res: 'Content not foundFacebookNews Feed0Friend Requests0Friend RequestsMessages0MessagesNotifications2NotificationsLaiba Noor, Arun Bhukal and 5 others asked to join Programming Questions. Just nowAngga Risky Setiawan's post in Web developer needs approval: "I hope it's useful...". 25 minutes agoJule Palenuchka added 3 photos in ТК | Барахолка | Оголошення | UA | Львів Київ Харків Одеса. 2 hours agoKartik Tiwary, Andrea Bacaj and Joy Gadugdug like your Page Funny Pets. 2 hours agoIndrajeet Singh's post in Programming Questions needs approval. 2 hours agoSee all notificationsSearch0RECENT SEARCHESEditLoading...Main Menu0The page you requested cannot be displayed at the moment. It may be temporarily unavailable, the link you clicked on may be broken or expired, or you may not have permission to view this page.Back to previous pageLoading...Try AgainCancelLoading...Loading...

Hey @SashaDesigN, this PR is pretty old, so it might no longer work.  I'm going to close it so as not to confuse anyone, but if you (or anyone else) wants to clean it up and submit a new PR, please feel free :smile:

ok thanks)


Simplify PathLayer tesselation

For #3758
Background
This change simplifies the layout of the position attributes, saves a little bit of memory, and prepares PathTesselator for accepting external buffers.
In this new format, loops are padded with 2 additional vertices, and paths without loops no longer require tesselation. We may expose it later as an option for use cases where performance is more important than visual quality. @twojtasz
Change List

Use a single positions buffer in PathLayer
Unit tests

Coverage increased (+2.6%) to 82.955% when pulling 87a11ec13a0de0491c73f9d4f7b11076a721204f on x/path-tesselator into 429218a7bec9a934fe775fcfaa332c06339f6030 on master.


examples/java/Test/Test.java test fails

Running the examples/java/Test/Test.java with SWI 7.2.0 fails for me:
$ LD_LIBRARY_PATH=/lib64/swipl-7.2.0/lib/x86_64-linux CLASSPATH=.:/lib64/swipl-7.2.0/lib/jpl.jar java -Djava.library.path=/lib64/swipl-7.2.0/lib/x86_64-linux Test
test 0...passed.
test 1...passed.
test 2...passed.
test 3...passed.
test 4...passed.
test 5...passed.
test 6...passed.
test 7...passed.
test 8...passed.
test 9...bagof(X-Y, p(X,Y), XYs) failed:
        Expected: '[|]'('-'(a, a), '[|]'('-'(a, b), '[]'))
        Got:      '.'('-'(a, a), '.'('-'(a, b), '[]'))

It looks like one of the serializations still uses the old syntax.

This has been fixed (sorry I don't know exactly when) and is not an issue in current stable version (7.4.1).


superputty doesn't consider font or column settings in putty session

it seems that superputty doesn't really take into account the 'appearance' settings in a putty session, and as a result the font is huge with every putty tab i have open , unless i right click -> change settings and change it manually
this might be related to https://github.com/jimradford/superputty/issues/625
but the relevant registry keys in the registry that putty stores appearance info:
main key: Computer\HKEY_CURRENT_USER\Software\SimonTatham\PuTTY\Sessions\SESSION_NAME\
sub keys:
Font
FontCharSet
FontHeight
FontIsBold
FontQuality
FontVTMode
TermWidth
TermHeight
LockSize (when window is resized setting)
these should also probably be stored in the Sessions.XML file that superputty maintains
examples:
superputty after loading the sessions from putty:

putty loading the same session itself:

also possibly related to https://github.com/jimradford/superputty/issues/368


adding mediaquery jsx component along with tailwind css screens

Describe the feature
I'll be adding a media query component that will take input prop as screen size so that in the future if we want to build components such as a navbar for various screens using media query we can give conditions and it will display only when the screen is above or below that size.
I'll also be adding a screens extension in the tailwind.js config file  along with themes so that while writing code in tailwind CSS it will be easy to write CSS for specific screen sizes
Add ScreenShots

like in the above image we can easily give conditions on what screen we should display so in future it will be great feature for contributors
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

hyy assign this to me.......

@powerstone666 this already being handled.

but there is no mediaquery component or screens in tailwind.css

Yes as this is not our final design. You can take a look at the new design and raise the issue accordingly.

can i get that new design

so from final design we need to build it from scratch ya u have already builded components based on that


ACTUALLY uploaded whiteboard file

[x] Create a new branch in your repo called array-reverse.

[x] Your top-level readme should contain a “Table of Contents” navigation to all of your challenges and implementations so far. (Don’t forget to update it!)

[x] This assignment should be completed within the challenges subdirectory of the repository.

[x] On your branch, create…C#: a new .NET Core console project named ArrayReverse.

[x] Within your Program.cs create a new static method outside of Main() following the naming conventions below. Call your newly created method in Main() once complete.

[ ] JavaScript: a folder named arrayReverse which contains a file called array-reverse.js

[ ] Python: a folder named array_reverse which contains a file called array_reverse.py

[ ] Java: a file called ArrayReverse.java

[x] Include any language-specific configuration files required for this challenge to become an individual component, module, library, etc.

[x] Write a function called reverseArray which takes an array as an argument. Without utilizing any of the built-in methods available to your language, return an array with elements in reversed order.


⚠ GitLab Server has degraded performance

In ba6fd5c, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11151 ms

Resolved: GitLab Server performance has improved in 78118a6.


Missing Powers tab

This is a great start! The only things missing to provide basic functionality is:

a powers tab
an equipment tab
a complications tab

Otherwise, this is a fantastic start and I encourage you to continue working on this! Great work!

You also need a way to mark rED and rPD.

And, in the continuing saga of noticing stuff, Movement.


Clarify case requirement for sign-on URL

Per Parakh Jain<EMAIL_ADDRESS>reply URL (therefore effectively sign-on URL) must contain tenant name as lower-case in order to avoid error AADSTS50011: Reply address '' specified by the request is not a valid URL. Allowed schemes: 'http,https'

Sorry for not being clear - I meant indent the whole line (including the > characters). Basically align it with the next lines that has the code snippet.

#sign-off

I signed off. I'll let it merge and then can modify it


Certainty - Use reset input to advance/randomize seed?

Should be useful for unattended jams

Linus had that same suggestion. After jamming with the firmware a bit, I totally agree. I think using RST to reseed is much more useful than resetting the sequence.

But maybe leave the sequence length the same, right?

Fixed in a18e5b0178f0a5316d5966c736ec252cd5c3c050!


Linked editing does not work in this case

Issue Type: Bug

VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

Why does issue reporter create issues instead of prefilling them?

@usernamehw please file a separate issue for the issue reporter if you haven't already.

There is one #117657.

Linked editing also does not work in my react .tsx files.
VSCode 1.59.0

It's not implemented for .jsx/.tsx files #85707


Add web app

The branch implements webapp with Streamlit, and some minor changes, such as github actions, codecov and pytest.

Codecov Report

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

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   40.90%
=======================================
  Files           ?        7
  Lines           ?      396
  Branches        ?        0
=======================================
  Hits            ?      162
  Misses          ?      234
  Partials        ?        0

Continue to review full report at Codecov.

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


crypto: add intent signing struct

a rework of https://github.com/MystenLabs/sui/pull/2116/files with https://github.com/bmwill/sui/commit/f5b8026a436eeff3fa0c7f61f369ede1b8683a1f?diff=unified

Thanks @joyqvq for resurrecting this. I think that before we look at getting this merged that we should revisit some parts of the intent scheme since I believe they should actually be rolled into some of our core structures. Specifically we need to add a chain_id to all transactions and for forward-compatibility reasons we may want to augment the serialized format of our core data structures, essentially baking in a part of the "intent" (the name of the struct) into the actual serialized payload of an object.

Thanks @joyqvq for resurrecting this. I think that before we look at getting this merged that we should revisit some parts of the intent scheme since I believe they should actually be rolled into some of our core structures. Specifically we need to add a chain_id to all transactions and for forward-compatibility reasons we may want to augment the serialized format of our core data structures, essentially baking in a part of the "intent" (the name of the struct) into the actual serialized payload of an object.

add a chain_id to all transactions  by transaction you mean struct TransactionEnvelope?

Specifically to TransactionData

Specifically to TransactionData

talked to @kchalkias  offline, we decide that we do NOT want to include chain id in the transaction data bc we assume object id are guaranteed to be unique and collision resistant as they are derived from the chain id from genesis.
instead, we will have user sign over IntentMessage - this will be a breaking change that we park till after private testnet stabilizes.

We may end up needing to do something slightly different with ChainId (as this is something that we probably want to exist onchain in some capacity) but we can iterate on that later

We may end up needing to do something slightly different with ChainId (as this is something that we probably want to exist onchain in some capacity) but we can iterate on that later

Good point, I have been trying this out in a draft PR. I think this could live in AuthorityState and also expose a get_chain_id endpoint that client software can query for. User can select network/chain_id in wallet for signing.


feat: replace blank canvas with starter templates.

Description
Shows starter page templates instead of blank canvas
As part of first activation experiment, this PR implements changes for showing starter page templates and allows user to fork a starter page template when they click on any template.
PR fixes following issue(s)
Fixes #27884
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview

/build-deploy-preview

/build-deploy-preview recreate=false skip-tests=true

/build-deploy-preview recreate=false skip-tests=true

/build-deploy-preview recreate=false skip-tests=true

/build-deploy-preview recreate=false skip-tests=true

/ok-to-test


Upgrade document extraction

@mlissner
This PR is meant to improve the extraction of text from PDFs by using a few
additional simple rules to decide if text is extracted appropriately.
Those rules include

Identifying any widget/free text annotations (previously this could lead to incorrect representation of the document)
Images larger than 10% of the page.  Meant to exclude tiny images - or images of lines.
Gibberish text from weird text embedding or missing fonts
Documents with less than 10 words on average per page.

Err strings were added for each of these reasons, which should be used in checking if OCR is needed on the CL side.
Previously we would/could identify documents as needing OCR - but also returning the text none the less - so that pages could be missed and CL wouldnt be aware that it might want to OCR the document.
Additionally, an optional flag has been added skip-margins as a boolean that can be used to
crop out the 1 inch margins that are required for court opinions as well as skewed stamp text we see in some
courts.  This is meant to get the text to represent the text of the opinion.
Tests were updated for the PDF changes and different possible difficult Pdfs were included.
Finally, a change to LXML and html Cleaning was addressed by adding `lxml_html_clean'.

Cool. I made a few comments, but none that I think is too crazy. My one remaining doubt is what the output looks like compared to the old output. Can you provide some examples of normal, good, bad, ugly, etc so we can see the improvement here?
I also worry about if we move to striping margins by default that that will cause trouble down the road when we remove more than we want, like, for example, on a scanned document where the scan is off center or something. Maybe it's safer to remove the margin at the top and left and leave the bottom and right?

I have to go back thru the rest of your comments and I will provide some sample output but I wanted to address a few things.

strip_margins is set to false by default.
strip_margins only applies to good PDFs that can be extracted with OCR.  As you rightly point out we don't want to strip or crop out the margins in a scan because the margins could include actual content in an image.  And it only works for the content extracted in PDF plumber.

I'm chatting with a customer now that values doctor for its high-speed text extraction. Could we keep pdftotext in this PR, and have a v2 text extractor that has all your improvements?

I heavily simplified the code and created a NEW pr for it.  or am - so im closing this PR


Webhook URL other than /hubot-deploy?

Has any thought been put in to making the URL to receive webhooks configurable? In http.coffee, everything is hard-coded to /hubot-deploy, which can be difficult to work with depending on the particular environment that the hubot is being deployed into.
I could probably get a PR ready to do this, but I first want to check that the idea is sound.

I am slowly accepting the fact that hubot related scripts are unmaintainable and I'm moving towards installable projects like https://slash-heroku.atmos.org/

So maybe best for me to just fork this repo for my own needs? Or would a PR be accepted?

I would probably accept a PR, what's the use case where using that default path won't suffice?

I'm deploying it on an internal infrastructure which does some automatic-ish URL generation. My hubot would be hosted at, for example, foo.com/my-hubot-proj/, i.e. a shared domain with different apps separated by the postfix on the URL (my-hubot-proj in this case)
Thus I'd need to specify an endpoint like /my-hubot-proj/hubot-deploy/repos/:owner/:repo/messages in http.coffee

(And I know a better answer is "get a subdomain!" but for now I can't do that)

Oh, gotcha. I bet we could accept headers properly for proxy passing but if you sent a PR that let's us pull it from an ENV var i'll definitely accept it.

PR here https://github.com/atmos/hubot-deploy/pull/114


[chore] Persist MeshModels Policies in DB on server boot

Notes for Reviewers
Relates to https://github.com/meshery/meshery/pull/7729

This PR fixes #
Signed commits

[x] Yes, I signed my commits.

A couple of failed checks to dig into...

A couple of failed checks to dig into...

Most tests are failing because we would want the sibling PR in meshkit to be merged and released

This changes are ready for review.

Generating docs for updated APIs....

Add swagger comments to new handlers and generate docs

Good callout, @abdullah1308

🚀 Preview for commit 6ecf98929804ff8a21cbef926daaf3f5cc19e90c at: https://6474275a2837562957a2ede0--meshery-docs-preview.netlify.app

🚀 Preview for commit 9650e309cc1b78690b1cadb291c67a8103dbcc4e at: https://64742ae4ce747336f63ac4de--meshery-docs-preview.netlify.app

Add swagger comments to new handlers and generate docs

✅

A couple items to investigate - https://github.com/meshery/meshery/actions/runs/5133477244/jobs/9236107109?pr=7730#step:5:6

A couple items to investigate - https://github.com/meshery/meshery/actions/runs/5133477244/jobs/9236107109?pr=7730#step:5:6

yes, these are failing because these structs are not yet defined in the latest meshkit release. But my PR in meshkit already adds support for these. https://github.com/meshery/meshkit/pull/320/files#diff-21de2f64fdc6b6f8f96ee0e558b47f8e9e24273bafa34dccee1c90c4250a8a49R36
Once we merge that PR and make a release then all these failing lint checks would pass.

🚀 Preview for commit 73e0d9e4d320c6a9871548d088643a5e578b3503 at: https://64774caa06c6802460070b60--meshery-docs-preview.netlify.app

Let's work through the design spec on this over the weekend, if we can. Ping me to chat it through and I'll help document. // @MUzairS15


add a way to run integration tests inside openshift

TODO:

[x] document this
[x] describe what types of tests should be done like this
add stg secrets inside the job doesn't make any sense
[x] bind code inside the pod nope :/
[x] fix check_in_cont

I also added a sample test:

test fedpkg clone -a within openshift

Merge party!

I see now:
make test_image
docker build --rm -t packit-service-tests -f Dockerfile.tests .
Sending build context to Docker daemon 13.27 MB
Step 1/3 : FROM docker.io/usercont/packit-service-worker:dev
 ---> b0c53fcf16fb
Step 2/3 : RUN set -ex; mkdir -p /home/packit/.config     && ln -s $PS_PATH/files/packit-service.yaml /home/packit/.config/packit-service.yaml     && ansible-playbook -vv -c local -i localhost, ./files/recipe-tests.yaml
 ---> Running in 9c4464031a9f
+ mkdir -p /home/packit/.config
+ ln -s /src-packit-service/files/packit-service.yaml /home/packit/.config/packit-service.yaml
+ ansible-playbook -vv -c local -i localhost, ./files/recipe-tests.yaml
ansible-playbook 2.8.5
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/packit/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3.7/site-packages/ansible
  executable location = /usr/bin/ansible-playbook
  python version = 3.7.4 (default, Jul  9 2019, 16:32:37) [GCC 9.1.1 20190503 (Red Hat 9.1.1-1)]
Using /etc/ansible/ansible.cfg as config file
ERROR! the playbook: ./files/recipe-tests.yaml could not be found


email identifer possibly outdated

It seems from recent git commits that your email address has changed from gmail.com in 2011 (through synacor.com in 2015) to aneurysm9.com here in 2018.
Perhaps that calls for updating author info in POD entries of code, and extending author info in CPAN metadata to include that identifier?

Thanks for taking the time to open these issues.  I appreciate that someone is thinking about QOL issues for users.  With this one, the listed email address is still valid and I expect will continue to be indefinitely, so I will leave it as is.


feat: support learn mode

This PR adds support for joining another network using S2 and S0 and being a secondary controller
fixes: #6428
enables https://github.com/zwave-js/node-zwave-js/issues/7096
enables #1070

@zwave-js-bot automerge

Should FAQ be also updated now? https://github.com/zwave-js/node-zwave-js/blob/master/docs/getting-started/faq.md Or it is still considered "Not supported"?

It's not fully supported yet - the other half of the Inclusion Controller CC implementation is missing. But I guess it's good enough to update the FAQ.

Does this mean I can run a second instance of zwave-js-ui elsewhere in my environment, with another stick. and join it to the network as a range extender? It's not clear from the docs how to get it to join an existing network on startup.

No, a secondary controller does not have the same functionality as the primary. For example with some devices you can not have them report to more than one controller.
What you may be thinking of (multiple instances cooperating) would require some backend work. It may come one day, but no guarantees.

Right but even if I didn't want to have it report to the secondary controller, or do any management from that node, could I at least get it as far as joining the network to extend the range of the mesh? I'm confused as to what the current limited support actually allows for.

I don't think there's support for this in Z-Wave JS UI yet.

I'm confused as to what the current limited support actually allows for.

Mainly in preparation for https://github.com/zwave-js/node-zwave-js/issues/7096
/cc @robertsLando

No support in UI yet but you could still try using it with driver function

Do you have any pointers on how to do that? I'm assuming I'd need to run the driver headless and pass it the keys and such?

@brandond Never tried those new APIs so I have no clue but if you want to understand how to work with custom driver functions you can look at docs here: https://zwave-js.github.io/zwave-js-ui/#/usage/driver_function


[BUG] Server Side Rendering fails when using Gatsby

Description/Screenshot
I'm using Gatsby to create a static website but when I attempted to generate the static HTML files from the React components the Server Side Rendering (SSR) fails in AppInsights:
"window" is not available during server side rendering.
[14/1691]
See our docs page for more info on this error: https://gatsby.dev/debug-html

   ReferenceError: window is not defined

   - render-page.js:8451 new AjaxMonitor
 /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:8451:34

- render-page.js:10514 new Initialization
 /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:10514:29

   - render-page.js:50798 Module../src/AppInsights.js
/home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:50798:306

  - render-page.js:30 __webpack_require__
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:30:30

  - render-page.js:51172 Module../src/components/ProductSummary/index.js
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:51172:70

   - render-page.js:30 __webpack_require__
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:30:30

  - render-page.js:51433 Module../src/templates/ProductPage.js
  /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:51433:84

   - render-page.js:30 __webpack_require__
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:30:30

  - render-page.js:667 Object../.cache/sync-requires.js
 /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:667:364

- render-page.js:30 __webpack_require__
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:30:30

  - render-page.js:617 Module../.cache/static-entry.js
  /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:617:1526

   - render-page.js:30 __webpack_require__
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:30:30

  - render-page.js:94
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:94:18

  - render-page.js:97
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:97:10

  - render-page.js:3 webpackUniversalModuleDefinition
   /home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:3:20

   - render-page.js:10 Object.<anonymous>
/home/aaron/code/github/react-app-insights/AppInsights-Part2/public/render-page.js:10:3

Steps to Reproduce

Create a new Gatsby website (using the default starter kit, or any others)
Add AppInsights from npm + the React extension
Run npm run build to generate the static output

OS/Browser: WSL2 (but confirmed failing on Windows)
SDK Version [e.g. 22]:

    "@microsoft/applicationinsights-react-js": "^2.3.0",
    "@microsoft/applicationinsights-web": "^2.3.0",

How you initialized the SDK: Following Basic Usage from the sample README.

Expected behavior
Gatsby is able to generate a set of static files.
Additional context
You'll find a demo that the errors can be repo'ed in on my GitHub. Gatsby works fine when run using the development webserver.

Doing my own digging into the problem, the issue is with the way the AjaxMonitor dependency works, specifically the following lines:
Line 98 which fails because the line does a test of window && ..., which will fail during SSR since window isn't defined
Line 325 which fails because it expects XMLHttpRequest to be globally defined, but isn't during SSR.
Line 584 which fails because it expects window to be a global.

Workaround
The workaround for this would be to use the DefinePlugin plugin for webpack. This is doable with Gastby, by updating the gastby-node.js file like so:
exports.onCreateWebpackConfig = ({stage, plugins, actions}) => {
  if (stage === 'build-html') {
    actions.setWebpackConfig({
      plugins: [
        plugins.define({
          window: undefined,
          XMLHttpRequest: undefined
        }),
      ],
    })
  }
}

This will define window and XMLHttpRequest but define them as undefined so that the tests will pass and anywhere that's doing a typeof window === 'object' test will still fail since typeof undefined === 'undefined'.

@aaronpowell  Hi Aaron, is this issue still relevant based on the changes introduced in #1167 to support a generic getGlobal() lookup method?

@MSNev I'll have to have a test and see if I can still repo (assuming that #1167 is in a released npm package).

It's been included since v2.4.1

This appears to still be an issue in 2.5.5. I believe this is because the library is using a helper function to check whether or not something is undefined, instead of checking typeof at the call sites (example). This results in the code throwing an exception (due to referencing an undefined value) before the helper code can run.
@MSNev I have put together a sample Gatsby app demonstrating these errors: https://github.com/jasonnutter/appinsights-gatsby
For other Gatsby users encountering this bug, I was able to workaround this by using the workaround from @aaronpowell above along with putting the invocation of loadAppInsights() in my app behind a typeof window !== "undefined".

It should indeed be using typeof tp avoid the JS strict mode checking, just as is done in the dependencies @ https://github.com/microsoft/ApplicationInsights-JS/blob/master/extensions/applicationinsights-dependencies-js/src/ajax.ts#L54

@jasonnutter came across this issue. If you loadAppInsights() in gatsby-browser.js you avoid that window-problem aswell 😄


Graph iterators

Description
# Assuming that Graph building is complete,
graph.each do |node, node_neighbors|
  # do whatever is required
end

Relevant Issues : (optional)
Relevant PRs : (optional)
Type of issue :

User mode :

[ ] Installation
[x] Feature request
[ ] Bug in existing feature

Developer mode :

[ ] Code quality
[ ] Tests
[ ] Documentation

String has each_char, each_line e.t.c., but It doesn't have simple each.
Graph has each_node, each_edge e.t.c., so I think it shouldn't have simple each.
It's because he behavior of simple each is not obvious.


Command is open for antag selection from Zombie mid round event.

Description
Head of Security and presumably other members of Command (and maybe Security) can currently become ii in a mid round zombie event.
Reproduction
Mid round zombie event with Command.

this is false, people who are infected with zombie infection just get the "turn into zombie action" like an II even if they aren't an inital infected


1.5.0 patch3

Reference Issues/PRs
Add tests, bump for 1.5.0 release.

Codecov Report

Merging #191 (6a14c78) into master (8123f42) will increase coverage by 7.64%.
The diff coverage is 51.45%.

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
+ Coverage   40.27%   47.91%   +7.64%
==========================================
  Files          48       92      +44
  Lines        2811     5480    +2669
==========================================
+ Hits         1132     2626    +1494
- Misses       1679     2854    +1175

Impacted Files
Coverage Δ

graphein/grn/parse_trrust.py
37.77% <ø> (ø)

graphein/ml/diffusion.py
0.00% <0.00%> (ø)

graphein/ppi/edges.py
100.00% <ø> (ø)

graphein/ppi/graph_metadata.py
0.00% <ø> (ø)

graphein/ppi/graphs.py
54.34% <ø> (ø)

graphein/ppi/parse_biogrid.py
75.00% <ø> (ø)

graphein/ppi/visualisation.py
0.00% <0.00%> (ø)

graphein/protein/analysis.py
0.00% <0.00%> (ø)

graphein/protein/edges/intramolecular.py
22.68% <0.00%> (ø)

graphein/protein/features/sequence/utils.py
28.00% <0.00%> (+3.00%)
:arrow_up:

... and 79 more

Continue to review full report at Codecov.

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


Feat/msdk 458 get dividend distributions

:tada: This PR is included in version 5.0.0-beta.25 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 5.0.0-alpha.17 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support several reduction ops

Support reduceL1, reduceL2, reduceMax, reduceMean, reduceMin, reduceProduct and reduceSum. PTAL, thanks! @fujunwei

Let's merge it, thanks.


Make scope of jackson dependencies "compile" rather than "runtime"

Your Question
I use JEmoji in a modular project and my tests don't find the Jackson classes used by JEmoji, because their scope is "runtime" not "compile". Would it be possible to change their scope to "compile"? Thanks!
I could sort it out by manually adding the 3 jackson dependencies in my test classes but that should be unnecessary, it is also a source of error (what when the version of these depds get bumped up, if I forget to refresh the version in my tests this will lead to very murky errors).

JEmoji only uses Jackson internally and it is not required to interact with the library. Therefore, there's no reason to expose it (change it to compile).
And this is not best practices due to various reasons. Gradle has a good documentation on why this should not be done.
Can you please create a MCVE which shows your issue where it can't find the dependency at runtime?

Sure I understand, just that testing in a multi module environment, the tests were failing because of a class not found exception on these deps.
I am not a specialist but changing to "compile" (in Maven) doesn't expose them in any particular way - except making the lib fully functional to other JPMS modules.
Creating an MVCE is too cumbersome given that the project is pretty large but will try.

Your problem is more, that you try to use a library that is not at all prepared to be used with JPMS on the module path.
If you use it from the classpath, it probably works.
If this library would be made a proper JPMS module, it would also work.
But just using it as automatic module indeed can lead to these problems.
If you for example execute this build:
./build.gradle.kts
plugins {
    application
    id("org.gradlex.extra-java-module-info") version "1.4"
}

repositories {
    mavenCentral()
}

dependencies {
    implementation("net.fellbaum:jemoji:1.1.6")
}

application {
    mainModule = "foo/foo.Foo"
}

extraJavaModuleInfo {
    automaticModule("net.fellbaum:jemoji", "net.fellbaum.jemoji")
}

./settings.gradle.kts
rootProject.name = "foo"

./src/main/java/foo/Foo.java
package foo;

import net.fellbaum.jemoji.EmojiManager;

public class Foo {
    public static void main(String[] args) {
        System.out.println(EmojiManager.getAllEmojis().iterator().next());
    }
}

./src/main/java/module-info.java
module foo {
    requires net.fellbaum.jemoji;
}

you get a ClassNotFoundException: com.fasterxml.jackson.core.JsonProcessingException when executing the run task.
Now if you change
extraJavaModuleInfo {
    automaticModule("net.fellbaum:jemoji", "net.fellbaum.jemoji")
}

to
extraJavaModuleInfo {
    automaticModule("com.fasterxml.jackson.core:jackson-core", "com.fasterxml.jackson.core")
    automaticModule("com.fasterxml.jackson.databind:jackson-databind", "com.fasterxml.jackson.databind")
    module("net.fellbaum:jemoji", "net.fellbaum.jemoji") {
        exports("net.fellbaum.jemoji")
        requires("com.fasterxml.jackson.core")
        requires("com.fasterxml.jackson.databind")
    }
}

the run task works perfectly fine.
So what you actually ask for, is making this library a proper JPMS module.

changing to "compile" (in Maven) doesn't expose them in any particular way

It does. Well, not "expose", but it pollutes the downstream compile classpath in unneeded ways which leads to bad dependency declarations where you use transitive dependencies and / or slower compilation because compile classpaths are bigger than they actually need to be.

except making the lib fully functional to other JPMS modules.

Not really, maybe in your specific situation. But this library does not even have an automatic module name, so it is most probably a very bad idea to use it on the module path as is anyway.

Wow great explainer thank you!
Up to the maintainer to see whether he'd like to make the lib as a JPMS. If I may push in this direction, this would be greatly appreciated not just by me but by fellow lib maintainers who (will) depend on it and release libs designed as JPMS modules.

I only roughly tested it and don't have time for doing it properly and creating a PR.
But this "simple" change should do everything you need properly.
You just have to properly fill the module info with the necessary statements and maybe add some integration tests that run on Java 9 or newer that actually test the outcome in a JPMS environment.
diff --git a/lib/build.gradle.kts b/lib/build.gradle.kts
index 19c4403..3bbd437 100644
--- a/lib/build.gradle.kts
+++ b/lib/build.gradle.kts
@@ -34,6 +34,38 @@ plugins {
     id("me.champeau.jmh") version "0.7.1"
 }

+val java9 by sourceSets.creating
+tasks.named<JavaCompile>(java9.compileJavaTaskName) {
+    javaCompiler = javaToolchains.compilerFor {
+        languageVersion = JavaLanguageVersion.of(9)
+    }
+    options.compilerArgumentProviders.add(object : CommandLineArgumentProvider {
+        @get:InputFiles
+        @get:PathSensitive(PathSensitivity.RELATIVE)
+        val mainClassesDirs = sourceSets.main.map { it.output.classesDirs }
+
+        override fun asArguments() = mainClassesDirs
+            .get()
+            .files
+            .map { it.absolutePath }
+            .flatMap {
+                listOf(
+                    "--patch-module",
+                    "net.fellbaum.jemoji=$it"
+                )
+            }
+    })
+}
+
+tasks.jar {
+    manifest {
+        attributes("Multi-Release" to true)
+    }
+    from(java9.output) {
+        into("META-INF/versions/9/")
+    }
+}
+
 repositories {
     mavenCentral()
 }
diff --git a/lib/src/java9/java/module-info.java b/lib/src/java9/java/module-info.java
new file mode 100644
index 0000000..afe35cf
--- /dev/null
+++ b/lib/src/java9/java/module-info.java
@@ -0,0 +1,3 @@
+module net.fellbaum.jemoji {
+    exports net.fellbaum.jemoji;
+}

@Vampire thank you for the code. It works good so far and the module-info file gets bundled in the jar. But I am facing an error when adding the jackson dependency. Any chance you know straight out of the head what's missing? Seems like there is some additional configuration needed to add the dependency to the new java9 source sets
C:\Users\user\IdeaProjects\jemoji\lib\src\java9\java\module-info.java:3: error: module not found: com.fasterxml.jackson.databind
    requires transitive com.fasterxml.jackson.databind;

val java9Implementation by configurations.existing {
    extendsFrom(configurations.implementation.get())
}

But you mean requires, not requires transitive.
Former is like implementation, latter like api.
Actually, usually these should be accordingly used.

Yeah, I just forgot to change it back when testing if changing it to transitive throws a different error :)
But thanks alot! Seems to be working so far :)

@seinecle Can you confirm that the new version works in your case?

👍 will try today

works great. Thanks a lot. For anyone who wants to see the jemoji lib in action in a JPMS project with tests:
https://github.com/seinecle/umigon-tokenizer


Unable to find a specification for Kingfisher (~> 3.0)

$pod --version
1.1.1
$pod install
Analyzing dependencies
[!] Unable to find a specification for Kingfisher (~> 3.0)

You may need to do a pod repo update, or even reset your pod environment.


Parameters setting

Any suggestions for parameters setting. The performance of default setting is very poor

Sorry for the confusion, the default parameters are place holders and are
not meant to produce a working network.
I am not able to suggest parameters as optimal parameters are specific to
whichever dataset you are using.
I suggest running a parameter search algorithm. For example use optunity for a random hyperparameter
search.
Tue, Feb 21, 2017 at 3:46 AM JesseYANK<EMAIL_ADDRESS>wrote:

Any suggestions for parameters setting. The performance of default setting
is very poor
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jaredleekatzman/DeepSurv/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmmZs5En8D8mPopaNlrmx9opr9ibHQnks5reqRYgaJpZM4MHBmu
.

@jaredleekatzman  Hi, I tried your code and the network.predict_risk(X) gave output nan. Can you help explain why this happens? Thanks.

@LeZhengThu could you provide more context? Did you train the network? Is it performing well on a validation set?

@LeZhengThu
You should check your input data. If the some variables are extremely large when compared to other variables, you will get nan.
You can normalise the input data, which should work for your case.

On 10 Apr 2017, at 20:32, jaredleekatzman<EMAIL_ADDRESS>wrote:
@LeZhengThu https://github.com/LeZhengThu could you provide more context? Did you train the network? Is it performing well on a validation set?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jaredleekatzman/DeepSurv/issues/3#issuecomment-292935641, or mute the thread https://github.com/notifications/unsubscribe-auth/ALq3gruouCCLFhgaxgRUtSVqM_eGojkyks5ruiFHgaJpZM4MHBmu.

@JesseYANK @jaredleekatzman  Thank you for the reply. I'll try @JesseYANK method to normalise the data and get back to you guys.

I had the same issue as LeZhengThu. I tried JessegThu method consisted is normalizing my data :
x = ( x - mean(x) ) / 1000
It seems to works now.

@jaredleekatzman Can you please provide example of how you implemented Optunity for random hyperparameter search that is specific to deepsurv?

Hi, I am currently working on releasing the Optunity script so that people can run their own hyper-parameter search. I hope to have it done soon.

On May 24, 2017, at 4:37 PM, dareneiri<EMAIL_ADDRESS>wrote:
@jaredleekatzman https://github.com/jaredleekatzman Can you please provide example of how you implemented Optunity for random hyperparameter search that is specific to deepsurv?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/jaredleekatzman/DeepSurv/issues/3#issuecomment-303844454, or mute the thread https://github.com/notifications/unsubscribe-auth/AFmmZgdozG2Pm0K-L1zaal_cjfh8OMieks5r9JUcgaJpZM4MHBmu.

@dareneiri Sorry for the delay, but I've uploaded the script I used to run a random hyper-parameter search under the directory hyperparam_search/.
If you still need to use it, let me know how it goes.


Xamarin forms: System.PlatformNotSupportedException: 'Shared mode is not supported in platforms that do not implement named mutex.'

I get this error when I try to make the database connection as shared:
public static string DevicePath = Environment.GetFolderPath(Environment.SpecialFolder.Personal);
public const string LiteDBName = "Object.db";
public static string DBPath = Path.Combine("Filename="+DevicePath, LiteDBName + ";connection=shared");
    public List<EventModelDB> GetEvents()
    {
        try
        {
            using (var db = new LiteDatabase(Constants.DBPath))
            {
                var col = db.GetCollection<ModelDB>();

                return col.FindAll().ToList();
            }
        }
        catch (Exception e)
        {
            throw e;
        }
    }

@Nitharshan1993 The LiteDB shared mode uses a system-wide named mutex to synchronize file access between multiple processes. If the platform does not support named mutex, then you won't be able to use shared mode with it.
Anyway, you should only use shared mode if you need multi-process access to the same datafile. In all other cases, you should use direct mode.


Cleanup workaround

It seems that the workaround was exclusively used in the DescriptionExtractor spec.
Am I missing something?
Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[-] Updated documentation.
[-] Added an entry to the changelog if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).
[x] ADDRESS FIXME notes

It seems that the workaround was exclusively used in the DescriptionExtractor spec.

The WorkaroundCop was introduced in #234, and the DescriptionExtractor changes were added later on.
The comments say it’s a “Clone of the the normal RuboCop::Cop::Cop class so we can rewrite the inherited method without breaking functionality”.
That said, I’m also not completely sure whether we need the workaround :-)

There's a tiny change TBD here, RuboCop::Cop::RSpec::Cop gets documented despite being abstract. I'll clamp this, there's a FIXME flag where this should be done.

rewrite the inherited method without breaking functionality

I have no idea what was the exact problem there.
@registry = Registry.new
class << self
  attr_reader :registry
end

def self.inherited(base)
  registry.enlist(base)
end

fails out of the box, because @registry is per-class and is nil in our Cop, so we have to relay enlist to the original.
Semi-related: https://github.com/rubocop-hq/rubocop/pull/8023

@bquorning @Darhazer Green!

Maybe we should drop RuboCop::Cop::RSpec::Cop altogether? The other extensions don't seem to have their own base cops.

Ping

FYI I found https://github.com/rubocop-hq/rubocop/pull/3837 which changed the inherited hook, which was mentioned in #234 as the reason for WorkaroundCop.

Thank you @pirj . It truly does look like this Cop superclass was not necessary anymore.


Allow default dfk #50

This PR adds a singleton class for loading the DFK which allows for more natural importing of apps. These changes were necessary in order to allow the testing framework to swap DFKs in and out such that we can factorize the testing code and the testing configurations, and cycle through such that each test is run on each configuration. Note it is possible to load a DFK either from a file containing a configuration or from a local DFK instance; I have only documented the latter way in the docstring, trying to keep things simple for now. Here is a preview of the new documentation:

@benclifford sorry for the excessive rebasing, as we were discussing on Slack, I'm trying to keep "doing X", "undoing X, doing Y"-type commits out of the history where possible. Next time I'll put the unrelated commits in a separate PR, or at least before the relevant commits so that only the actually-changed commits are updated. I've implemented your requests, you can see the relevant diff here: 1ec7d04...3f0fbac. A few notes:

To avoid using globals I wanted to retain the singleton class, but I've concealed it from the user by importing DataFlowKernelLoader.load in parsl.__init__.py such that the user can just call parsl.load. The motivation for doing that over writing a wrapper function was so that the method documentation only needs to appear once (note it is accessible via >>>help(parsl.load)).
Because init might be confused with __init__, I went with parsl.load instead of parsl.init. Let me know if you have a strong preference.

Thanks for your feedback! Here's a preview of the docs:

looks good


microk8s start successful, but there is no any pod

I am in centos 7.6, I install the snap and microk8s. my microk8s version is v1.14.0
then I start microk8s, but there isn't any pod in kubectl's output:
why?
BTW, I'm try this solution in microk8s (https://github.com/AliyunContainerService/gpushare-scheduler-extender), can I do this? Or Do I have to try this solution in native k8s cluster?
[root@localhost gpushare]# microk8s.start
Started.
[root@localhost gpushare]# microk8s.kubectl get po --all-namespaces
No resources found.
[root@localhost gpushare]# microk8s.inspect
Inspecting services
  Service snap.microk8s.daemon-containerd is running
  Service snap.microk8s.daemon-apiserver is running
  Service snap.microk8s.daemon-proxy is running
  Service snap.microk8s.daemon-kubelet is running
  Service snap.microk8s.daemon-scheduler is running
  Service snap.microk8s.daemon-controller-manager is running
  Service snap.microk8s.daemon-etcd is running
  Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system info
  Copy network configuration to the final report tarball
  Copy processes list to the final report tarball
  Copy snap list to the final report tarball
  Inspect kubernetes cluster

/snap/microk8s/492/inspect.sh: line 100: ufw: command not found
Building the report tarball
  Report tarball is at /var/snap/microk8s/492/inspection-report-20190401_100339.tar.gz

thanks for your time.
inspection-report-20190401_100339.tar.gz

Hi @15050050972
Right after you snap install MicroK8s you get an empty Kubernetes cluster with no pods. You have to then enable the addons you want eg microk8s enable dns dashboard gpu. And deploy whatever workload you wish. I am not faminiral with the solution you want to deploy but I see it wants GPU support, so I would strongly recommend you use Ubuntu and deploy there the nvidia GPU drivers before you microk8s.enable gpu.

I think we should close this issue.


Code doesn't work

I'm running .NET MAUI on VSC using the .NET MAUI extension. Neither of those code samples - either in .cs or .xaml - worked, both generated errors.

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

ID: 039cf416-fa7c-dca3-ef72-f9029c613ecd
Version Independent ID: 039cf416-fa7c-dca3-ef72-f9029c613ecd
Content: TabbedPage toolbar placement on Android - .NET MAUI
Content Source: docs/android/platform-specifics/tabbedpage-toolbar-placement.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @rogersm-uwosh
I've just tried the code in VS (see https://github.com/davidbritch/dotnet-maui-samples/tree/main/PlatformIntegration/PlatformSpecifics) and there are no errors. So it's either a VSCode setup problem for you, or a problem with the MAUI extension for VSCode.


[stable/concourse] Add support for encryption key

Add support for the new concourse encryption feature described here:
https://concourse.ci/encryption.html

/ok-to-test

@boazy Could you also bump the patch version of the chart.
Also note that the PR cannot be merged unless you sign the CLA https://github.com/kubernetes/charts/pull/1545#issuecomment-317608983

Chart version bumped, CLA signed.

/ok-to-test


Update hex.pm

Hi! Please update hex.pm to latest master. Thank you!

@dziaineka Updated. The latest tag is 1.0.1


fix: allow marked v11

Allow marked v11 as a peer dependency

Could you add this to test case in workflows also?

Thanks!


Running acceptance tests with codeception

PHPCI is running and building fine now, however I still don't know how to do acceptance testing with Codeception. The codeception plugin says it "Enables full acceptance, unit, and functional testing.". However, I don't know how to set up the codeception.yml / acceptance.suite.yml.
For acceptance testing (and API testing) I need a web-path for the project. Is there anything I can do with PHPCI at the moment? Has anyone already successfully set up acceptance testing?
Regards
Alex

You need to run a server (what phpci doesn't do for your test)
I have explain two ways in #743


Copying cells containing double quote adds more double quotes

When copying Text cell whose data includes a double quote, text copied gets escaped with two more double quotes.
Note that this happens only when copying into TextBuffer and when copying cell, not text in the textarea in edit mode
What is the rationale for escaping here? Can customize this behavior to respect raw value? Explicitly providing copyData doesn't seem to help
https://glideapps.github.io/glide-data-grid/?path=/story/glide-data-grid-dataeditor-demos--copy-support
screencast-newtab-2024.02.08-12_51_47.webm

The general rationale here is to make the copy/paste fully compatible (as much as possible) with excel, excel online, google sheets, and apple numbers. The copy buffer contents are both simultaneously well standardized between the different spreadsheets and a total shitshow.
Apple Numbers puts in useless extra newlines, Google Sheets generally is sane, and Excel... oh god...
That said... yeah I could probably escape less aggressively. I think escaping due to the quotes is technically only needed when the cell contains a newline or a tab character.

How do we turn this off?! No company in my industry copies-and-pastes between Excel and Google Sheet, and any casual user would absolutely intend for copy and paste to actually copy and paste the contents of the cells without modifying them.
This has completely broken every single Google Sheet in my company, we are in dire need of a complete reversion or fix.


Can't find CreateEnList.Sex enum type!

HI.
Can't find CreateEnList.Sex enum type! If you have recently defined this enum type in server side code, make sure your project builds successfully and transform T4 templates. Also make sure that enum is under your project root namespace, and your namespace parts starts with capital letters, e.g. MyProject.Pascal.Cased namespace
...
[RadioButtonEditor(typeof(MemberSex)), Required(true), Placeholder("Sex")]
...

How Can I fix it? Thanks EveryOne

这个框架好难啊，封装的太厉害了。从后台到前端全封装，太难改了

dotnet sergen  你知道是做什么用的吗?

    **[ScriptInclude]**
    public enum MemberSex
    {
    }

try to write [ScriptInclude] attribute, and then transform T4 templates

$(DotNetSergen) ?


[SMALLFIX]doc fix for docs/cn/File-System-API.md

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14093/
Test PASSed.


Enhancement - possibility to show only dropdown menu for reason

Possibility for reason window to only show dropdown menu without text (or textbox only appearing if reason « other » chosen?)

it is possible to have only text box but not only dropdown for now.

As we cannot change the size of the alert while it is shown, we could only hide the text field and this looks a bit weird. So this is unfortunately not possible.


[Telemetry] Send preset value even there is early error during crc start

Looking at amplitude matrix around preset, there is lot of number around none preset. Then I looked at the segment which have raw data around our telemetry and found out in case there is early error we are not sending the preset value to segment and hence this none happen.
client.Track(&analytics.Track{
  UserId: "xxxxxxxxxxxxxxxxxxxx",
  Event: "crc start",
  Properties: map[string]interface{}{
    "duration": 3558,
    "error": "Hyper-V Virtual Machine Management service not running",
    "error-type": "os.CodeExitError",
    "remote": false,
    "source": "cli",
    "success": false,
    "tty": true,
    "version": "2.20.0",
  },
})

or
client.Track(&analytics.Track{
  UserId: "xxxxxxxxxxxxxxxxx",
  Event: "crc start",
  Properties: map[string]interface{}{
    "duration": 2926,
    "error": "crc should be ran in a shell without administrator rights",
    "error-type": "os.CodeExitError",
    "remote": false,
    "source": "cli",
    "success": false,
    "tty": true,
    "version": "2.20.0",
  },

we are not sending the preset value to segment and hence this none happen.

Why is that actually not happening? As it seems, there is an 'error'. Any idea what the error is that prevents the preset to be sent? Or is this very generic?

We are now sending preset details for each event/interaction. hopefully with next release we will see less none around it.


check_cincludes can't find backtrace.h on macosx

Describe the bug
I have install libbacktrace on macosx, backtrace.h was in /usr/local/include, and the library was in /usr/local/lib. I use check_cincludes to check whether libbacktrace is installed, and xmake didn't find it.
Expected behavior
Xmake should find libbacktrace if it is installed.
Related Environment
Please provide compiling and running environment information:

xmake version: v2.5.7
os: macos 10.13.6
target platform: macos x64

Additional context

xmake.lua

includes("check_cincludes.lua")
includes("check_cxxincludes.lua")
includes("check_links.lua")
check_cincludes("HAS_BACKTRACE_H", "backtrace.h")
check_cxxincludes("HAS_CXXABI_H", "cxxabi.h")
check_links("HAS_LIBBACKTRACE", "backtrace")

这是因为 macOS 下，默认走 xcode 编译，会设置 isysroot 到 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk
这会导致默认检测的头文件都走到 xcode sdk里面去，不再默认从 /usr/local/include 中找。
一种解决办法，更新到 dev 。。我改进了 check_cincludes 你可以追加 includedirs 进去检测
check_cincludes("HAS_BACKTRACE_H", "backtrace.h", {includedirs = "/usr/local/include"})

第二种办法，走包模式，可以把 libbacktrace 包提交到仓库走 add_requires。。也可以仅仅走 on_fetch ，不装包，仅仅查找系统包，这种走的 pkg-config ，不会默认走 xcode sdk 路径
例如：https://github.com/xmake-io/xmake-repo/blob/8c25b8bb5a9f81dd8fa3df781e52bea2487795bd/packages/g/gtk%2B3/xmake.lua
package("xxx")
        on_fetch("linux", function (package, opt)
            if opt.system then
                return package:find_package("pkgconfig::xxx")
            end
        end)
package_end()

add_requires("xxx", {optional = true})

if has_package("xxx") then
    add_defines("HAS_BACKTRACE_H")
end

ok, thanks for the reply!


Remove expired sponsors from sponsors list

The Issue
We've been showing long-since expired github sponsors and featured sponsors, and that's not fair to the folks that are currently active.
How This PR Solves The Issue

Remove inactive featured sponsors
Remove inactive github sponsors

Manual Testing Instructions
Review the results on the rendered home page
Rendered

main page
support ddev page

It won't show up on https://github.com/ddev/ddev yet of course, but that should be the same.

I'm not clear about what part of the arrangement you're talking about, thanks for looking.

I think it looks OK,

Everything I've been doing here has been just prompting AI, since I don't understand most of what goes on :(

Okay, thanks, I just checked how it would look at https://github.com/ddev/ddev


SPLoaderError.loadComponentError on Hub Site home page components

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
I am running the SPOL Provisoning Service using the SharePoint Starter Kit solution template.  All the pre-requisites are checked OK and the three sites are created with no error messages.
Observed Behavior
If I go to the Hub Site home page several apps/web parts are displaying this error message:
Something went wrong
If the problem persists, contact the site administrator and give them the information in Technical Details.
TECHNICAL DETAILS
[SPLoaderError.loadComponentError]:
***Failed to load component "26cb4af3-7f48-4737-b82a-4e24167c2d07" (pnpTiles).
Original error: ***Failed to load URL 'https://publiccdn.sharepointonline.com/imgroupltd.sharepoint.com/sites/appcatalog/ClientSideAssets/3624777a-337d-4b34-8384-5ab774c069aa/solution-propertycontrolstrings_en-us_24df0ae17fe08d19f6a630a1cbd23af4.js' for resource 'PropertyControlStrings' in component '26cb4af3-7f48-4737-b82a-4e24167c2d07' (pnpTiles). There was a network problem.
This may be a problem with a HTTPS certificate. Make sure you have the right certificate.
***INNERERROR:
***Failed to load URL 'https://publiccdn.sharepointonline.com/imgroupltd.sharepoint.com/sites/appcatalog/ClientSideAssets/3624777a-337d-4b34-8384-5ab774c069aa/solution-propertycontrolstrings_en-us_24df0ae17fe08d19f6a630a1cbd23af4.js' for resource 'PropertyControlStrings' in component '26cb4af3-7f48-4737-b82a-4e24167c2d07' (pnpTiles). There was a network problem.
This may be a problem with a HTTPS certificate. Make sure you have the right certificate.
***CALLSTACK:
Error
at t [as constructor] (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-03.013/sp-pages-assembly_en-us_5493085219cddd015fe284db23f333b4.js:915:16084)
at new t (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-03.013/sp-pages-assembly_en-us_5493085219cddd015fe284db23f333b4.js:1538:21156)
at Function.e.buildErrorWithVerboseLog (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-03.013/sp-pages-assembly_en-us_5493085219cddd015fe284db23f333b4.js:1538:16215)
at Function.e.buildLoadComponentError (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-03.013/sp-pages-assembly_en-us_5493085219cddd015fe284db23f333b4.js:1538:12246)
at https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-03.013/sp-pages-assembly_en-us_5493085219cddd015fe284db23f333b4.js:1538:61575
Steps to Reproduce
This bug was present on a site created Thursday as well as a new one today.  It is reproduced by provisioning the SharePoint Starter Kit solution template.
Thanks for your contribution! Sharing is caring.

Just double-checking.  The tenant that you are accessing this from is imgroupltd.sharepoint.com ?

We experience the same error on one of our clients tenant. We have 1 DEV environment where the error never appears, but on the TST environment it does. Is there a temporary workaround available to resolve this issue?
Error :
[SPLoaderError.loadComponentError]:
Failed to load component "24bddb3e-9e7d-43da-8126-5bb7970a21f6" (GoDossieraanvraagFormWebPart).
Original error: Failed to load URL 'https://publiccdn.sharepointonline.com/.sharepoint.com/sites/appcatalog/ClientSideAssets/38d70d14-7a41-451d-bd9a-b5dec7940e2f/go-dossierbeheerwebparts-bundle_6eb8f529380c189c56e3a58195bf60ac.js' for resource 'go-dossierbeheerwebparts-bundle' in component '24bddb3e-9e7d-43da-8126-5bb7970a21f6' (GoDossieraanvraagFormWebPart). There was a network problem.
This may be a problem with a HTTPS certificate. Make sure you have the right certificate.
INNERERROR:
Failed to load URL 'https://publiccdn.sharepointonline.com/.sharepoint.com/sites/appcatalog/ClientSideAssets/38d70d14-7a41-451d-bd9a-b5dec7940e2f/go-dossierbeheerwebparts-bundle_6eb8f529380c189c56e3a58195bf60ac.js' for resource 'go-dossierbeheerwebparts-bundle' in component '24bddb3e-9e7d-43da-8126-5bb7970a21f6' (GoDossieraanvraagFormWebPart). There was a network problem.
This may be a problem with a HTTPS certificate. Make sure you have the right certificate.
CALLSTACK:
Error
at t [as constructor] (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.005/sp-pages-assembly_nl-nl_5acaa6a406dd536235a0f0ae17aff897.js:862:16092)
at new t (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.005/sp-pages-assembly_nl-nl_5acaa6a406dd536235a0f0ae17aff897.js:1485:21156)
at Function.e.buildErrorWithVerboseLog (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.005/sp-pages-assembly_nl-nl_5acaa6a406dd536235a0f0ae17aff897.js:1485:16215)
at Function.e.buildLoadComponentError (https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.005/sp-pages-assembly_nl-nl_5acaa6a406dd536235a0f0ae17aff897.js:1485:12246)
at https://spoprod-a.akamaihd.net/files/sp-client-prod_2019-05-24.005/sp-pages-assembly_nl-nl_5acaa6a406dd536235a0f0ae17aff897.js:1485:61659

just double checking two things:

Is the error still occurring?
if you disable the Public CDN feature, does the error go away?
Thanks

Hi @lucabandMSFT ,
The error is still occuring sometimes. I will disable the public CDN again and and then wait some time before I enable it again.
Keep you posted! ;)


Develop output cs qy

Adds the ability to output cross sections, quantum yields, and the radiation field from TUV-x stand-alone runs. Also updates the build scripts for building on Casper (GNU/Intel) and modifies some code to avoid Intel ICEs

Codecov Report
Patch coverage: 88.33% and project coverage change: -0.12 :warning:

Comparison is base (e604cac) 77.70% compared to head (9636033) 77.59%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #259      +/-   ##
==========================================
- Coverage   77.70%   77.59%   -0.12%
==========================================
  Files         106      107       +1
  Lines        8398     8557     +159
==========================================
+ Hits         6526     6640     +114
- Misses       1872     1917      +45

Impacted Files
Coverage Δ

src/cross_sections/hno3-oh_no2.F90
88.75% <ø> (-0.14%)
:arrow_down:

src/cross_sections/no2_tint.F90
78.57% <ø> (-0.17%)
:arrow_down:

src/cross_sections/o3_tint.F90
88.23% <ø> (-0.08%)
:arrow_down:

src/cross_sections/rono2.F90
88.29% <ø> (-0.13%)
:arrow_down:

src/cross_sections/temperature_based.F90
83.66% <ø> (-0.09%)
:arrow_down:

src/cross_sections/tint.F90
87.20% <ø> (-0.11%)
:arrow_down:

src/quantum_yields/acetone-ch3co_ch3.F90
98.24% <ø> (-0.04%)
:arrow_down:

src/quantum_yields/c2h5cho.F90
100.00% <ø> (ø)

src/quantum_yields/ch2chcho.F90
96.55% <ø> (-0.12%)
:arrow_down:

src/quantum_yields/ch2o.F90
100.00% <ø> (ø)

... and 22 more

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


How to structure Stimulus within Rails apps - Examples

Hi,
this is not really an issue (although somewhat it is) but i am look for example projects which go beyond introductions using just 1 controller, to learn how to structure Stimulus properly within mid-sized Rails applications.
Can you provide information on this or point to such applications? (Maybe examples from Basecamp)
Thanks

Our community forum is a great place for questions like this: https://discourse.stimulusjs.org/t/work-with-rails/43


Epic: otdfctl e2e tests

Background
#234 recently added e2e test infrastructure to run a suite in CI with bats against a running platform instance.
There should be e2e tests (tracked under independent issues linked to this epic) that cover the majority of the CLI commands in otdfctl.

[ ] #326
[ ] #327
[ ] #328
[ ] #329
[ ] #330
[X] kas-registry CRUD
[X] kas grants assign/unassign
[ ] #332
[ ] #331
[ ] #334 (blocked by #333)

I've filed https://github.com/opentdf/otdfctl/issues/347 which should provide some of this functionality


Passing token from iOS app returns 401

After several days of debugging and even nuking my repo and starting over, I still can't seem to get any authorization with a JWT token I'm passing from an iOS app authenticated with Auth0 Lock. After updating to Knock 1.5, I've narrowed it down to self.from_token_payload not creating a new user. Any assistance would be greatly appreciated!
Here's my repo: https://github.com/brandonmanson/happening-api

It would be helpful if you could provide a failing request spec for this issue.
Also, could you link to the documentation explaining which claims are passed through the payload in your case?
It seems like you expect the payload to contain a field :email. Are you sure this is valid? When reading this, it seems like the user id is passed through the subject claim (i.e. payload["sub"]).

Auth0 allows you to specify claims in this documentation. Here's what my payload looks like:
"payload<EMAIL_ADDRESS>"email_verified"=>true, "iss"=>"https://brandonmanson.auth0.com/", "sub"=>"auth0|57843c212f0e1ca74ee67b60", "aud"=>"PGQxxDeLQBkABM5Pg6v1lOZgz2tLcTlU", "exp"=>1469081229, "iat"=>1469045229}"
I was trying to use find_or_create_by with something other than payload["sub"] since I hadn't gotten that to work either.
I don't have a failing request spec yet. I am passing a valid token in an HTTP request from XCode Simulator. I'll come up with a way to mock that.

I'll close this issue as it does not seem related to Knock directly but you can keep posting here and I'll be happy to help once we have a failing spec :)

👍 thank you!


The Keyword "Application Intent" is incorrect

The Keyword "Application Intent" is incorrect. The correct Keyword is "ApplicationIntent" with NO space.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f7197d3b-5829-e8f5-7245-e51869cc0836
Version Independent ID: 40f0e4c5-95eb-24e0-6f0f-2776c8f89bb0
Content: Using Connection String Keywords - SQL Server Native Client
Content Source: docs/relational-databases/native-client/applications/using-connection-string-keywords-with-sql-server-native-client.md
Product: sql
Technology: native-client
GitHub Login: @markingmyname
Microsoft Alias: maghan

@genifycom -- thank you for catching this.  I am looking into the article.

@markingmyname -- Mark, please review private PR 15631.


Sample Audio File not working

See question on Dev Works: https://developer.ibm.com/answers/questions/205005/speech-to-text-demo-play-sample-audio-not-working.html
Steps Taken:

Click 'Deploy to Bluemix' Button in the README file
Click through to deploy application in my own bluemix space
Click route (http://speech-to-text-deploybutton.mybluemix.net/) then click any of the options (record audio, upload file, play sample 1, play sample 2) and there is no response.

There is most likely a step I'm forgetting here, but I appreciate any help you can provide.

I just tested and everything seems to be working fine


no-unresolved caseSensitive weird edge case

I'm using atom-xo-linter which relies on this package. If I open atom with atom ~/code/medusa/ atom-xo-linter shows the casing does not match the underlying filesystem warning yet if I open it using atom ~/code/Medusa/ there's no warning.
I'm on macOS Sierra 10.12.3 (16D32).

This may possibly be related to how Atom opens file.
Ref: https://github.com/atom/atom/issues/8221

atom-xo-linter, like all other linter providers there, uses TextEditor::getpath() here to get the path sent to XO (and thus here in eslint-plugin-import), so I would agree with you that this is a problem that needs to be solved on Atom's part.

Happy to reopen if it ends up being an issue here.


chore: update with 1.4 branch stuff

Not to be landed until after the first 1.4.x release

Doh! Sorry to duplicate effort; didn't realize you'd already worked on it.

@jeffbcross - I had forgotten that I had done that!!


added modules into a try: statement

added modules into a try: statement and prompts auto-install && closed loop

ofc bro thanks fr fr

@ixEcIipse check out the latest vers

(did this manually btw)


Add tickets for all PM-ish follow up tasks (check Notion page)

Notion Page: NYC Offsite

closed in favour of this ticket #9086


Feature/add byebug to gitignore

Squash it when merging so the noise about reverting the commit goes away


error with npm start

Looks pretty interesting (as was cam2pc :) )
Got some errors just trying to run it (node 22 installed):
npm start

<EMAIL_ADDRESS>start
cross-env DEBUG=1 electron-forge start

✔ Checking your system
❯ Locating application
✔ Locating application
❯ Loading configuration
The CJS build of Vite's Node API is deprecated. See https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.
✔ Loading configuration
❯ Preparing native dependencies
✔ Preparing native dependencies
❯ Running generateAssets hook
✔ Running generateAssets hook
❯ [plugin-vite] Launching dev servers for renderer process code
failed to load config from witsy/vite.renderer.config.ts
✖ [plugin-vite] Launching dev servers for renderer process code [FAILED: No "exports" main defined in witsy/node_modules/estree-walker/package.json]
An unhandled rejection has occurred inside Forge:
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main defined in witsy/node_modules/estree-walker/package.json

found the issue, I had to clean the npm cache and try again:
npm cache clean
rm -rf node_modules
rm package-lock.json
npm install
npm start

Glad you were able to fix it! How do you know cam2pc? How do you know it's me? 😀


vcpkg install cairo:x86-windows-static FAILED

f:\3rdparty\vcpkg>vcpkg install cairo:x86-windows-static
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
The following packages will be built and installed:
cairo[core]:x86-windows-static

dirent[core]:x86-windows-static
expat[core]:x86-windows-static
fontconfig[core]:x86-windows-static
glib[core]:x86-windows-static
Additional packages (*) will be modified to complete this operation.
Starting package 1/5: glib:x86-windows-static
Building package glib[core]:x86-windows-static...
Glib relies on DllMain and therefore cannot be built statically
CMake Error at ports/glib/portfile.cmake:13 (message):
Glib only supports dynamic library and crt linkage
Call Stack (most recent call first):
scripts/ports.cmake:71 (include)

Error: Building package glib:x86-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: glib:x86-windows-static
Vcpkg version: 2018.10.20-nohash
Additionally, attach any relevant sections from the log files above.

glib did not support static build?

Glib relies on DllMain and therefore cannot be built statically
CMake Error at ports/glib/portfile.cmake:13 (message):
Glib only supports dynamic library and crt linkage
Glib only supports dynamic library and crt linkage


Utilities for cuda streams + disk IO

handling disk io for checkpointing with cuda streams is a nontrivial task. If you're  not careful, you can easily get broken code (need to wait for data to be on the CPU before you can start writing the buffer to the disk), or synchronous behaviour (because memory is not page-locked, so async copies are not possible)
Therefore, this PR introduces two new utility functions to do the disk <-> device data transfer.
In addition to being less error prone and reducing code duplication, this also gives us a single point at which we can implement double buffering to actually get an overlap between device transfer and disk writes; with an added bonus that we no longer need to allocate giant CPU-side arrays (think: 8 A100s on the same node each wanting to write 40 GB model state; we'd attempt to allocate 320 GB of host memory there; maybe the boxes are big enough even under that scenario, but do we really want to do that?)
I've also added a unit test for these functions; its very rough, I am not going to touch the makefile so for now you need to compile yourself, and it leaves behind its temp file. But it already payed off, because my first implementation had a wrong offset somewhere, and the test caught this :)
There is also the problem notices first with #522 that we currently miss master weights in the saved state. This is hacked in here very quickly, but its not really a good solution; should be combined at least with #522's addition of a flag in the file that indicates whether to expect master weights or not.
Because we now do file IO with cuda, we get that cuda_common.h includes utils.h, and requires us to mark all functions there inline. I also had to add a checked write function, for which I've just copied the error handling; not 100% sure if that makes sense.
For the double-buffered transfers, I've put a size of 32MiB, but that is not based on any actual data. I just wanted to put something there that is neither super tiny nor super big :)

Left some comments - lgtm!


[CONTENT CREATION] Implement pages for service offerings

Databanks and data governance
Data infrastructure and development
Data science

PR for first pass: https://github.com/QUT-Digital-Observatory/australian_digital_observatory_site/pull/89


Fix the photo handling on the split-answer answers

The photo's not appearing

Fixed as of 140e1501a379dc6b7c404fdab05abbcbc91bed3c


Enforce minimum and maximum players

https://github.com/tannerkrewson/rocketcrab/blob/253899d57b251a6a360d25c1e85b514ef314a6c9/config/games/_template.ts#L62-L67
This is blocking #81

done in fcf02bb617618a684e19b3da051e93c2693407e4


wombat implementation

essentially someone needs a wombat repository.  Probably anna. Then people can just git clone it into the project directory

do not merge this without making a seperate repository for wombat.


NXP: KE1xF: Add SoC power management support

This PR adds SoC power management for the NXP Kinetis KE1xF SoC and the TWR-KE18F development board through a number of smaller changes:

Determine CONFIG_SYS_CLOCK_HW_CYCLES_PER_SEC default from devicetree.
Introduce a new OS timer driver for the NXP Kinetis Low Power Timer (LPTMR) IP (the Arm SysTick cannot wake up the KE1xF from deep sleep/stop).
Add support for KE1xF SoC power management (idle, stop, partial stop 1, and partial stop 2).

I will follow-up with an OS timer driver for the NXP Kinetis Real Time Clock (RTC) IP block, which should be able to provide tickless operation.

Are they not in the datasheet (not the reference manual)?

Ah. Yes, they are in the data sheet. I will update the devicetree accordingly and retest.

Ah. Yes, they are in the data sheet. I will update the devicetree accordingly and retest.

Updated.

@nashif, @MaureenHelm - are your comments resolved?

Needs rebase

Needs rebase

Done.


Logarithmic scale for axis

First of all, I would like to say thank you for such an amazing library! I really like your approach. Hope it will evolve to be a de-facto standard for Rust graphics.
The question is: how am I supposed to configure plotter to draw in an logarithmic scale? I am particularly interested in a logarithmic histograms.

I'm glad you finding its useful. Previously the logarithmic coordinate is planned but not yet support.
Now I make the change, so that it can support log coordinate. Now you can pass LogRange wrapped range to specify the axis is log scale !
 let mut chart = ChartBuilder::on(&root)
        .x_label_area_size(25)
        .y_label_area_size(40)
        .caption("Histogram Test", &font)
        .build_ranged(0u32..10u32, LogRange(0u32..10u32));

Also, please note the library is in very early stage and I believe I am currently the only user.
But I am more than happy to get more and more people involved !
Hope this is helpful.

Now you can pass LogRange wrapped range to specify the axis is log scale !

Awesome, thaks! I will give it a try.

Also, please note the library is in very early stage and I believe I am currently the only user.

Sure, I understand that. Nevertheless, even for a young library plotter demonstrates an impressive amount of features. Looking forward for more to come!


Question: problems with integrating ng-packagr generated package with Angular CLI app: Module not found

Here's a repo that reproduces it: https://github.com/dragonflypl/ng-packagr-issue

I've create a simple package logging.  npm run build generates the library + does npm pack
Then I generated fooGui with Angular CLI and installed library it via npm run consume that installs tgz
Running npm run build from fooGui throws:

ERROR in ./src/app/app.component.ngfactory.js
Module not found: Error: Can't resolve 'logging/src/modules/logging/logger/index' in 'C:\XXX\dev\ng-packagr-issue\fooGui\src\app'
resolve 'logging/src/modules/logging/logger/index' in 'C:\XXX\dev\ng-packagr-issue\fooGui\src\app'
  Parsed request is a module
  using description file: C:\XXX\dev\ng-packagr-issue\fooGui\package.json (relative path: ./src/app)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: C:\XXX\dev\ng-packagr-issue\fooGui\package.json (relative path: ./src/app)
    resolve as module
      C:\XXX\dev\ng-packagr-issue\fooGui\src\app\node_modules doesn't exist or is not a directory
      C:\XXX\dev\ng-packagr-issue\fooGui\src\node_modules doesn't exist or is not a directory
      C:\XXX\dev\ng-packagr-issue\node_modules doesn't exist or is not a directory
      C:\XXX\dev\node_modules doesn't exist or is not a directory
      C:\XXX\node_modules doesn't exist or is not a directory
      C:\node_modules doesn't exist or is not a directory
      looking for modules in C:\XXX\dev\ng-packagr-issue\fooGui\node_modules
        using description file: C:\XXX\dev\ng-packagr-issue\fooGui\package.json (relative path: ./node_modules)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: C:\XXX\dev\ng-packagr-issue\fooGui\package.json (relative path: ./node_modules)
          using description file: C:\XXX\dev\ng-packagr-issue\fooGui\node_modules\logging\package.json (relative path: ./src/modules/logging/logger/index)
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              C:\XXX\dev\ng-packagr-issue\fooGui\node_modules\logging\src\modules\logging\logger\index doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              C:\XXX\dev\ng-packagr-issue\fooGui\node_modules\logging\src\modules\logging\logger\index.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              C:\XXX\dev\ng-packagr-issue\fooGui\node_modules\logging\src\modules\logging\logger\index.js doesn't exist
            as directory
              C:\XXX\dev\ng-packagr-issue\fooGui\node_modules\logging\src\modules\logging\logger\index doesn't exist

Any idea what I did wrong or it is a bug somewhere?

Thanks for reporting this issue. This issue is now obsolete due to changes in the recent releases. Please update to the most recent Angular CLI version.
If the problem persists after upgrading, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.