Support for DRM_IOCTL_SYNCOBJ_TIMELINE_WAIT

The DRM_IOCTL_SYNCOBJ_TIMELINE_WAIT ioctl allows userspace to wait for work on a GPU to be completed.
There is currently no non-blocking API to accomplish this. One suggestion for how to solve this has been to attach an eventfd to the underlying syncobj that would be signaled once the work has been completed<EMAIL_ADDRESS>Personally I believe that the better solution would be to support this ioctl via io_uring. Some general new OP (e.g. IORING_OP_IOCTL) that supports a subset of all ioctls (for now only DRM_IOCTL_SYNCOBJ_TIMELINE_WAIT) could be used for this. In the future this could also be expanded to other ioctls if the need arises.
Would such a thing be appropriate for io_uring?

I was telling a couple of years ago that io_uring with ioctls sounds like a very nice base for drm. We already have an alternative to ioctls, see IORING_OP_URING_CMD

IORING_OP_URING_CMD should be perfect for wiring up a communication channel with drm, as it can be added per file type. Not much is needed to support that on the drm side, and then you get an async channel for request/responses.

That should probably work but I'm a bit worried about the IORING_OP_URING_CMD concept in general. Aside from it being blocking, the only problem with ioctl is that the ioctl numbers are not unique. This means that you cannot use ioctl easily on untrusted file descriptors: An ioctl that works fine with one file type might corrupt your memory if used with a different file type. The kernel has tried to mitigate this by only assigning unique ioctl numbers for new ioctls but now it seem that IORING_OP_URING_CMD has committed to the same design mistake. ublk just uses the following cmd_op IDs:
#define	UBLK_CMD_GET_QUEUE_AFFINITY	0x01
#define	UBLK_CMD_GET_DEV_INFO	0x02
#define	UBLK_CMD_ADD_DEV		0x04
#define	UBLK_CMD_DEL_DEV		0x05
#define	UBLK_CMD_START_DEV	0x06

In my usecase I receive file descriptors via unix domain sockets, so I have to trust that IORING_OP_URING_CMD cannot have unintended side effects.

Maybe what we could do is reserve IDs below 0x10 and tell everyone else to to use the usual ioctl numbering scheme for new cmd_ops. Unfortunately we cannot invent a new scheme because nvme has already committed to using ioctl numbers for their cmd_op IDs.

That should probably work but I'm a bit worried about the IORING_OP_URING_CMD concept in general. Aside from it being blocking,

Can you elaborate what you mean? It's not blocking, the enter syscall should never block submitting IO. Some operation may block in a worker thread, but that depends on how the handler is implemented. E.g. nvme pass through is fully asynchronous.

the only problem with ioctl is that the ioctl numbers are not unique. This means that you cannot use ioctl easily on untrusted file descriptors: An ioctl that works fine with one file type might corrupt your memory if used with a different file type. The kernel has tried to mitigate this by only assigning unique ioctl numbers for new ioctls but now it seem that
[...]
IORING_OP_URING_CMD has committed to the same design mistake. ublk just uses the following cmd_op IDs:
In my usecase I receive file descriptors via unix domain sockets, so I have to trust that IORING_OP_URING_CMD cannot have unintended side effects.

I don't think it's a mistake but rather the ioctl numbering is not seems like a mess (just like ioctl is). The user should know what file it operates on, it doesn't make much sense otherwise. What to do when you get a random file? It would be nice to have a way to discover the file type dynamically. Does statx return anything useful for that?

Can you elaborate what you mean?

I meant that the ioctl syscall is blocking.

The user should know what file it operates on, it doesn't make much sense otherwise.

With the unique-ioctl-system, the worst that can happen is that the user gets ENOTTY back.

It would be nice to have a way to discover the file type dynamically.

That would be the proper solution.

Does statx return anything useful for that?

I don't believe this is possible in general. If the kernel calls anon_inode_getfile then there is no way to identify the underlying fops.

I've implemented this here: https://github.com/mahkoh/linux/pull/1
I had to add the ability to cancel uring_cmds because syncobj waits are not guaranteed to ever complete. See the first commit. There is still at least one open issue with the current implementation. I've described this in a comment in the MR.

Looks like a good start, I added some comments to the first patch. I would suggest we move this to the mailing list though for review, if you're fine with that?

Let's complete the current round of questions in the linked MR and move to the ML for the next round.

Let's complete the current round of questions in the linked MR and move to the ML for the next round.

Yep, that sounds like a good plan.

Did you make any progress on this?

I can only work on this over the weekend. If you believe that the ability to cancel uring cmds is independently useful, the I would be more than happy for you to take this over.

Sorry, I dropped this after a bad interaction with someone involved in DRM.
For this particular usecase, Simon Ser has submitted a less powerful mechanism which has been merged<EMAIL_ADDRESS>Feel free to pick up the linked MR under any license you want. It's unlikely that I'll pursue this.


Have you found a bug? Did you feel something was missing?

It does not work on Vivaldi, too bad. I would love to use it on Vivaldi.

Thanks for the feedback. This is a duplicated issue, see #10 .


CPS-0016? | Cardano URIs

Rendered Version
This CPS proposes to be a central repository of information related to CIP-13 web+cardano: URIs and the different schemes, authorities, and solutions that exist to help expand and aid in integration.

Formally inviting: @rphair @Quantumplation @SebastienGllmt @MadOrkestra to provide their feedback and comments/review

Don't forget rename the folder

@Crypto2099 I'm happy to announce that the permalink for CIP-0107 is now available 🎉 https://github.com/cardano-foundation/CIPs/blob/master/CIP-0107/README.md

@Ryun1 I've resolved your last comment https://github.com/cardano-foundation/CIPs/pull/841#discussion_r1688318587 with https://github.com/cardano-foundation/CIPs/pull/841/commits/884b6a5aeea6797b2387b2f53168cd2d572a8251, which also fixes some things in the header: so I believe this is ready for your final review now.


[tcgc] do we want to add original name of tsp type / property before in main.tsp?

Clear and concise description of the problem
This would be the name in the tsp definition before any @clientName changes.

RLC might need this as part of onboarding to tcgc

Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For feature request in the typespec language or core libraries file it in the TypeSpec repo
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

duplicate of #1059


The pp file that auto generates causes issues with an F# web api

The nuget package wont load.  I have to load it manually.

Removed the pp files.


Convert Responsive Element component to CSS modules

Issue Description
To better align with modern CSS practices and modern CSS theming capabilities, we want to update this component to use CSS modules and if applicable, CSS custom properties.
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
Component styles are converted to use CSS modules. If applicable, convert SCSS variables to CSS custom properties.
Current Behavior
Component styles use manual name spaced classes and SCSS variables.

We can clean up a lot of the code in the responsive element by doing something like this:
breakpoints.scss
:export {
  tiny: 544;
  small: 768;
  medium: 992;
  large: 1216;
  huge: 1440;
}

ResponsiveElement.jsx
import breakpoints from './breakpoints.scss'

console.log(breakpoints.tiny);
// returns 544

I'll plan on picking this one up. The export feature looks nice.

The exports is nice but it creates an inconsistency with custom properties. The export is defined at transpile time but the custom properties can change past that. It's OK if we want use the exports, but we'll need the ability to override those defaults so that if anyone needs dynamic breakpoints (via theming) they can create a container component to wrap this one and set that dynamically.

Even more ideal would be the ability for the component to handle being passed integers OR custom properties. If handled custom properties, it would simply look up the value. This would future proof the component when we switch to 100% custom props that aren't compiled down.

Yeah I think the tricky thing with this component is CSS custom properties don't work inside of media queries.
From the spec,

The var() function can be used in place of any part of a value in any property on an element. The var() function can not be used as property names, selectors, or anything else besides property values. (Doing so usually produces invalid syntax, or else a value whose meaning has no connection to the variable.)

Essentially, you can't use it in a media query.
I haven't spent much time thinking on how to solve this but I think it will be tricky.

Closing this as no longer applicable.

Why is this no longer applicable?

Reopening, we'll look into using the export syntax used here: https://github.com/cerner/terra-core/blob/918d02e27f5e1cf10e2b7ffd9e51a3a824166f44/packages/terra-dynamic-grid/src/DynamicGrid.scss


Laravel 9 support

Any chances of Laravel 9 support?

@wast please check my fork https://github.com/dyanakiev/laravel-logger-discord-channel


Move social.lol images to a CDN

no procrastinate @newbold

https://user-images.githubusercontent.com/986721/199399914-d65932bd-ec56-4337-b594-7054e5c8a6cd.mov
Just kicked off the sync! Looks like it may take a while, but I’ll let it do its thing.

"social.lol media has been moved to a CDN" - adam on irc 2022


Improved Error types to work in browsers without v8 engine

Error.captureStackTrace is only supported in browsers with v8 engine.
With this pull request I can use request-promise in all browsers.

Awesome, thank you!
I wonder, if we should move Error.call(this) to the top to set name, message etc. afterwards. In V8 that is not necessary. But how about the other browsers? What would you prefer?

I tested this order in Firefox and IE (latest). So I would leave it as it is.

I just looked into Error.call(this) and as far as i can tell now it doesn't work as intended. See the section "Obstacle 2: a constructor that can’t be called as a function" in http://www.2ality.com/2011/12/subtyping-builtins.html
Any knowledge you can dump on me?

BTW, the test failed due to a timeout. Your PR was working fine.

One specific question: In Firefox and IE do the errors have a stack property? (According to the blog post they won't. But the blog post may be wrong.)

Firefox does have a non-standard stack property. See https://developer.mozilla.org/de/docs/Web/JavaScript/Reference/Global_Objects/Error/stack
And IE also has a stack property. See
https://msdn.microsoft.com/library/ie/hh699850(v=vs.94).aspx/
So it would be possible to write the stack trace to the console.

Alright, I keep the if (Error.captureStackTrace) but remove the Error.call(this).
Thanks for your feedback!

I just published v0.4.1 on NPM.


X1 - DASH based VOD content playback keeps buffering

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
3.2.4
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Custom app
If custom app, can you reproduce the issue using our demo app?
No
What browser and OS are you using?
X1
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Comcast Xi6-A
What are the manifest and license server URIs?
What configuration are you using?  What is the output of player.getConfiguration()?
{
"drm": {
"retryParameters": {
"maxAttempts": 2,
"baseDelay": 2000,
"backoffFactor": 2,
"fuzzFactor": 0.5,
"timeout": 30000,
"stallTimeout": 10000,
"connectionTimeout": 20000
},
"servers": {
"com.widevine.alpha": ""
},
"clearKeys": {},
"advanced": {
"com.widevine.alpha": {
"distinctiveIdentifierRequired": false,
"persistentStateRequired": false,
"videoRobustness": "",
"audioRobustness": "",
"sessionType": "",
"serverCertificate": {},
"serverCertificateUri": "",
"individualizationServer": ""
}
},
"delayLicenseRequestUntilPlayed": false,
"logLicenseExchange": false,
"updateExpirationTime": 1,
"preferredKeySystems": []
},
"manifest": {
"retryParameters": {
"maxAttempts": 2,
"baseDelay": 2000,
"backoffFactor": 2,
"fuzzFactor": 0.5,
"timeout": 30000,
"stallTimeout": 10000,
"connectionTimeout": 20000
},
"availabilityWindowOverride": null,
"disableAudio": false,
"disableVideo": false,
"disableText": false,
"disableThumbnails": false,
"defaultPresentationDelay": 0,
"dash": {
"clockSyncUri": "",
"ignoreDrmInfo": false,
"disableXlinkProcessing": false,
"xlinkFailGracefully": false,
"ignoreMinBufferTime": false,
"autoCorrectDrift": true,
"initialSegmentLimit": 1000,
"ignoreSuggestedPresentationDelay": false,
"ignoreEmptyAdaptationSet": false,
"ignoreMaxSegmentDuration": false,
"keySystemsByURI": {
"urn:uuid:1077efec-c0b2-4d02-ace3-3c1e52e2fb4b": "org.w3.clearkey",
"urn:uuid:edef8ba9-79d6-4ace-a3c8-27dcd51d21ed": "com.widevine.alpha",
"urn:uuid:9a04f079-9840-4286-ab92-e65be0885f95": "com.microsoft.playready",
"urn:uuid:79f0049a-4098-8642-ab92-e65be0885f95": "com.microsoft.playready",
"urn:uuid:f239e769-efa3-4850-9c16-a903c6932efb": "com.adobe.primetime"
}
},
"hls": {
"ignoreTextStreamFailures": false,
"ignoreImageStreamFailures": false,
"useFullSegmentsForStartTime": false
}
},
"streaming": {
"retryParameters": {
"maxAttempts": 2,
"baseDelay": 2000,
"backoffFactor": 2,
"fuzzFactor": 0.5,
"timeout": 30000,
"stallTimeout": 10000,
"connectionTimeout": 20000
},
"rebufferingGoal": 2,
"bufferingGoal": 10,
"bufferBehind": 10,
"ignoreTextStreamFailures": false,
"alwaysStreamText": false,
"startAtSegmentBoundary": false,
"gapDetectionThreshold": 1,
"smallGapLimit": 0.5,
"jumpLargeGaps": true,
"durationBackoff": 1,
"forceTransmuxTS": false,
"safeSeekOffset": 5,
"stallEnabled": true,
"stallThreshold": 1,
"stallSkip": 1,
"useNativeHlsOnSafari": true,
"inaccurateManifestTolerance": 2,
"lowLatencyMode": false,
"autoLowLatencyMode": false,
"forceHTTPS": false,
"preferNativeHls": false,
"updateIntervalSeconds": 1
},
"offline": {
"usePersistentLicense": true
},
"abr": {
"enabled": true,
"useNetworkInformation": true,
"defaultBandwidthEstimate": 1000000,
"switchInterval": 8,
"bandwidthUpgradeTarget": 0.85,
"bandwidthDowngradeTarget": 0.95,
"restrictions": {
"minWidth": 0,
"maxWidth": null,
"minHeight": 0,
"maxHeight": null,
"minPixels": 0,
"maxPixels": null,
"minFrameRate": 0,
"maxFrameRate": null,
"minBandwidth": 0,
"maxBandwidth": null
}
},
"preferredAudioLanguage": "",
"preferredTextLanguage": "",
"preferredVariantRole": "",
"preferredTextRole": "",
"preferredAudioChannelCount": 2,
"preferredVideoCodecs": [],
"preferredAudioCodecs": [],
"preferForcedSubs": false,
"preferredDecodingAttributes": [],
"restrictions": {
"minWidth": 0,
"maxWidth": null,
"minHeight": 0,
"maxHeight": null,
"minPixels": 0,
"maxPixels": null,
"minFrameRate": 0,
"maxFrameRate": null,
"minBandwidth": 0,
"maxBandwidth": null
},
"playRangeStart": 0,
"playRangeEnd": null
}
What did you do?
Play random DASH based VOD content
Note: The playback failing we have observed on some of the replay content, which is converted as LIVE to VOD.
What did you expect to happen?
Playback should not fail. Though playback has failed player should throw the appropriate error code.
What actually happened?
When we tried to play random replay events based on DASH. some of the content does not play. the player does not throw any error, it keeps buffering. the chunk is not getting downloaded.

Can you test with v4.3.3? Thanks!


[BUG] java.lang.RuntimeException: Canvas: trying to draw too large bitmap.

Describe the bug
My phone can took 9MB+ photos, which I can't edit in this app (app crashes)
To Reproduce
Steps to reproduce the behavior:

Get large photo
Try to edit it in app

Expected behavior
App not crashes and I can edit photo
Smartphone:

Device: TECNO Camon 20 Pro 5G
OS: Android 13
Version 2.1.0 (from actions)

Additional context
java.lang.RuntimeException: Canvas: trying to draw too large(254803968bytes) bitmap.

I have the same issue; app crashes when attempting to edit a 50 MP JPG image.
Google Pixel 8 Pro with 12 GiB memory.

Will be resolved soon while also adding the support for subsampling images (loading the full resolution)

Will be resolved soon while also adding the support for subsampling images (loading the full resolution)

Great. Thanks.

Fixed in https://github.com/IacobIonut01/Gallery/commit/450a724909d1bf1ce3f00f4e93205a8350cba8c3

Fixed in 450a724

Thanks a lot.


Upgrade Kitematic.app to v0.7.5

Thank you for the contribution. It was merged directly as 6801164cc9e129496a55dd915974e548369ff347 to keep commit history cleaner. Your contribution is still credited to you.


[EventRequest] codeunit 80 “Sales-Post”, PostPayment[local]

Please add a new publisher event OnBeforePostPayment in Codeunit 80 on function PostPayment and call it as shown in the image below:

Please add the corresponding Integration Event In the bottom of the codeunit

I cannot find PostPayment in W1, which country is this?

The country is Italy (IT)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


feat(vnui): new menu icons

Description
Adds new icons and highlights the active page in the side menu.
This installs React Icons (https://react-icons.github.io/react-icons/), which gives us more icons to choose from.
Motivation and Context
Added some nicer icons and it will be easier to see which page you're on now.
How Has This Been Tested?
Manually
What process can a PR reviewer use to test or verify this change?
Run the VN as usual. The icons on the left should change.
Breaking Changes
x

[x] None
[ ] Requires data directory to be deleted
[ ] Other - Please specify

Sorry, I deleted that css reference - didn't end up using the linked css file. It should work now.


Shortcut to toggle drawing / annotation mode

Is your feature request related to a problem? Please describe.
It would be nice to be able to have a shortcut key to start the drawing mode, to be able to quickly pull it up.
Describe the solution you'd like
d has already been taken, so maybe a could be used, for annotation.
Describe alternatives you've considered
I can use the mouse/trackpad, not a huge deal, just a nice-to-have.

Add a to start the drawing mode is easy, but you need another shortcut to close it.
Because a is binded to arrow drawingMode when drawing is enabled.
https://github.com/slidevjs/slidev/blob/8d0ae82274b3528cf3bc122b0ad90110c19a0eb9/packages/client/logic/drawings.ts#L129-L131

Add a to start the drawing mode is easy
Could you explain how I would do this?  Sorry if it should be easy, I can't tell from the documentation.

The documentation is in here: https://sli.dev/custom/config-shortcuts.html#configure-shortcuts
Add below code in ./setup/shortcuts.ts:
import { defineShortcutsSetup, NavOperations } from '@slidev/types'

export default defineShortcutsSetup((nav: NavOperations) => {
  return [
    {
      key: 'a',
      fn: () => nav.toggleDrawing(),
    },
  ]
})

But as I said, this a shortcut will have a conflict with drawing's mode arrow(a) shortcut.

Thanks, it wasn't listed in https://sli.dev/guide/navigation.html#navigation-bar, so I thought it wasn't possible.  I'll make a PR to update the docs if I have a chance.
Sounds like I just need to use a key that's not conflicting with anything else in the drawing mode.  :+1:


#221 Add support for transient metastore tables

Transient tables are not persisted, so you can chain multiple transformers together without saving all of them to a storage. This allows more flexibility to separate logical structure of the pipeline while maintaining reasonable storage costs.
pramen.metastore {
  tables = [
    {
      name = "CSV_TEST_converted"
      format = "transient"
      cache.policy = "no_cache"
    },
    # ...
  ]
}

Jacoco it seems does not run integration tests, so coverage is lower. I don't know how to fix it, so let's just ignore coverage of bookkeeping for now.


ApolloError not thrown when receiving mutation result with GraphQL errors

ApolloError is not thrown when the result of a mutation contains errors. Haven't tested with queries yet so I don't know if this is only a problem for mutations.
Example:
client.mutate({
  mutation: gql`
    mutation update($input: ProfileUpdateType) {
      updateUser(input: $input) {
        _id
        email
        name
        picture
      }
    }
  `,
  variables: {
    input: {
      invalidField: 'value'
    }
  }
})
.then(({ errors }) => {
  if (errors) {
    console.log('ApolloError not thrown')
  }
})
.catch(error => console.log('ApolloError: ', error.message));

Results in ApolloError not thrown being logged to the console.
Example of result with errors:

#465 should be the fix for this.

Released in 0.4.8!


spotify-tray: 1.3.2 -> 1.3.2-unstable-2024-02-10

Description of changes
Updated to unstable to remove the patch, there hasn't been a release in 3 years.
Cleaned up
migrated to by-name

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

ah for some reason the update breaks usage like this
{
  lib,
  symlinkJoin,
  spotify,
  makeBinaryWrapper,
  spotify-tray,
}:
symlinkJoin {
  name = "spotify";
  paths = [ spotify ];
  nativeBuildInputs = [ makeBinaryWrapper ];
  postBuild = ''
    rm $out/bin/spotify
    makeWrapper ${lib.getExe spotify-tray} $out/bin/spotify \
    --prefix PATH : ${lib.makeBinPath [ spotify ]}
  '';
}

spotify-tray won't exit when spotify does


Decide if plugin functions should have a context.Context

What would you like to be added:
We should consider if context parameters should exist on plugin callbacks before formalizing the SDK.
Notes :

This is an SDK only issue because context does not pass through the wasm sandbox, nor is there any language independent way to propagate go's context (e.g. from the scheduler to wasm written in C).
This shouldn't start until we have all plugin functionality created, as otherwise we can't see holistic performance impact.

Why is this needed:
End users should comment directly what they want, so we can verify the expectation will work in TinyGo.
Completion requirements:
The completion of this issue depends one the decision:

If yes, we add context params and document how they are different than normal go context. We also have test cases and benchmarks around at least one user required application of context.
If no, we add into RATIONALE.md why we didn't, for example to reduce chance of mis-use at the cost of lack of parity between the framework and wasm SDKs.

This enhancement requires the following artifacts:

[ ] Design doc
[ ] API change
[ ] Docs update

The artifacts should be linked in subsequent comments.

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle rotten

/remove-lifecycle rotten
/kind documentation


Fix paper abstract for main.249

Volunteers: 1
https://acl2020.rocket.chat/channel/presenter-helpdesk?msg=kruC8hoeTXLon4Gnj

Hi, this is not a critical issue but the abstract for my paper on the website (https://virtual.acl2020.org/paper_main.249.html) seems to be from an earlier version of the paper (possibly the abstract we first submitted for review). The main difference in my case is that it says "(Our code will be made publicly available on publication.)" instead of "Code to reproduce our experiments is available at https://github.com/neulab/RIPPLe." (in the pdf at https://www.aclweb.org/anthology/2020.acl-main.249.pdf). This might be more of an issue for other papers though.

@hao-fang I can pick up this task


Use await/async inside of build:update-cdn-details

R: @addyosmani  @addyosmani @gauntface
Switching to the promise-based readJSON()/writeJSON() and using await/async inside the task seems to have cleared up the task completion issues I was seeing.

Hmm. Failing on AppVeyor atm.

That's fixed by https://github.com/GoogleChrome/workbox/pull/884

Approved https://github.com/GoogleChrome/workbox/pull/884. This one LGTM if we can get AV passing again :)

I'm merging this one first as I now see that AppVeyor in #884 didn't complete due to this 😄


[Bug] flags.warn_error_options.silence should be settable dbt_project.yml

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Missing silence here:
https://github.com/dbt-labs/dbt-core/blob/32a7f82772d90a4b9ab443908f5f4455d0aa2c73/core/dbt/cli/flags.py#L53-L62
Expected Behavior
def convert_config(config_name, config_value):
    """Convert the values from config and original set_from_args to the correct type."""
    ret = config_value
    if config_name.lower() == "warn_error_options" and type(config_value) == dict:
        ret = WarnErrorOptions(
            include=config_value.get("include", []),
            exclude=config_value.get("exclude", []),
            valid_error_names=ALL_EVENT_NAMES,
            silence=config_value.get("silence", []),  # this line is missing above
        )
    return ret

Steps To Reproduce

Create a project with a test in it

# dbt_project.yml
flags:
  warn_error_options:
    silence:
      - TestsConfigDeprecation

tests:  # has been renamed to data_tests
  +enabled: true

dbt parse -> still see the deprecation warning ("The tests config has been renamed to data_tests")
Apply the patch shown above -> dbt parse -> no deprecation warning

Relevant log output
No response
Environment
- Python: 3.10.11
- dbt: 1.8.0

Which database adapter are you using with dbt?
No response
Additional Context
No response

I want to add some reasoning to this issue.
We checking if we can update to DBT 1.8, and we changed our configs from tests to data_tests.
The problem is that we use several DBT packages and those packages still use tests.
And I believe that DBT package developers will not update from tests to data_tests because they want to support compatibility with previous DBT versions.
May be it even makes sense to allow warning suppression based on package name, rather than suppressing globally for the whole project.

@kokorin Appreciate that rationale! We have a separate issue for scoping that deprecation warning more narrowly:

https://github.com/dbt-labs/dbt-core/issues/9835

https://github.com/dbt-labs/dbt-core/pull/10058/files#diff-9a7ad4b74c0941bce1368c1e8e12f0745bf0152b00f92df206375702b46e3a8eR133-R140
There should be an integration test catching this.
This might be because include is being required now.(something we potentially want to change).

This might be because include is being required now.(something we potentially want to change).

For clarity, specifying include as part of warn_error_options was always required previously. Ever since the initial implementation, include was required when specifying warn_error_options. Additionally, that made sense in a world where only include/exclude (now error/warn) were the only options. Now that silence exists, that logical requirement kind of breaks down.

About my previous comment, I was wrong. The object requires include be specified, it is not optional. However, in the instantiation in core, we always default it to a list if it is not specified. We can also see that in that instantiation, silence is never passed 🙃 and that is indeed the issue. Interestingly, I'm not sure why the related are passing (TestWarnErrorOptionsFromCLI.test_can_silence, TestWarnErrorOptionsFromProject.test_can_silence)

Back when I did the work for https://github.com/dbt-labs/dbt-core/pull/10058 (specifically https://github.com/dbt-labs/dbt-core/commit/c52d6531fcc0ca51804906dccb726d68be8550ea) I thought that the warn_error_options would automatically be converted from the yaml to the WarnErrorOptions object as we were building the ProjectFlags object, which holds warn_error_options, via ProjectFlags.from_dict. And I thought this was validated by the test_can_silence test added in c52d6531fcc0ca51804906dccb726d68be8550ea. However, there were two problems:

The definition of warn_error_options on PrjectFlags is a dict, not a WarnErrorOptions object
The test_can_silence test was broken, and not testing what I thought

The quick fix (#10359) is to ensure silence is passed to WarnErrorOptions instantiation in dbt.cli.flags.convert_config. The better fix would be to make the warn_error_options of ProjectFlags a WarnErrorOptions object instead of a dict. However, to do this we first need to update dbt-common's WarnErrorOptions definition to default include to an empty list. Doing so would allow us to get rid of convert_config entirely.


Testing/operations

What's changed:

Small refactor of L.DNC.GeoExecute, pulling out logic that isn't directly related to executing an operation (ie putting results on the map) and placing that logic in L.DNC.AppController.  For the sake of testability, I think life would be easier if we work towards limiting the scope of classes and functions.  Additionally, having functions only rely on arguments as inputs (as opposed to global variables) and only using return values as output (rather than calling functions from the global namespace)

To be added:

A test for the buffer operation

:+1: from me on this one. Testing seems like it should be much easier this way. As you say "relying more on returns from functions would make it easier to treat each little chunk as a unit and to test on a more isolated level".

Hmm.... looks like some tests didn't pass.

@svmatthews there we go, forgot to use .eql for deep comparisons.

Ah, good to know!


Create 2021-06-23-thoughts-on-diversity-in-conferences.md

Added diversity blog post

checked out PR in VS Code and ran locally
Everything looks good
Thank you so much @eugman


Map: skip about when coming from reach4help.org

When the referrer is reach4help.org, go straight to viewing the map (instead of displaying the about page) to prevent people having to click "View the Map" multiple times.

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

Fixed in #511


New youtube video formats available

Just notice two new videos formats ! ID =  78 , 59
both look same though. codec info below !
Stream 0
Type: Video
Codec: H264 - MPEG-4 AVC (part 10) (avc1)
Resolution: 854x480
Frame rate: 23.976024
Decoded format: Planar 4:2:0 YUV
Stream 1
Type: Audio
Codec: MPEG AAC Audio (mp4a)
Channels: Stereo
Sample rate: 44100 Hz
Even 141 is back which seem missing from recent videos .
Here is the video - https://www.youtube.com/watch?v=a23GO0NXOtg

Thanks for the report. itags 59 and 78 will be recognized in the next version.


added audio recorder - onStop doesn't pick up

Hello!
Thank  @0x006F  you for this library and @urosran for collaborating in it 👏
I know it's not my business, but there area a couple of things I think this PR should not be merge as it is:

It adds a polyfill that, and IMO it should be the library user who decides if include it or not. Maybe I know all my devices can record, or maybe I was want to detect it and skip recording. @urosran Is it possible to implement this functionality without the polyfill?
There are some console.logs and alert calls that can be annoying for the end user

My two cents.

Besides the console.logs and alerts (we can clean up at any time), I guess @danigb has a valid point in this case. As we discussed in #19 , even though this polyfill kinda work, it doesn't support video capturing yet. So, we're losing one of our key features in here.
I sincerely apologize for this delayed replies. I'm having numerous meetings / plannings at my full-time + work. Sorry!


Refactor how the logs work

This is what I'm thinking:

[ ] - Add a new tail option to the start command.

If tail is true:

[ ] - The start command should not unref() the child, but should wait for the child to die.
[ ] - The Emulator's logs should pipe to the parent terminals stdio.
[ ] - The Emulator's logs should also continue to be written to the log file.
[ ] - Winston should use two transports: Console and File.
[ ] - If the user presses Ctrl+C the Emulator will be killed.

If tail is false:

[ ] - The start command will unref() the child and exit.
[ ] - The Emulator's logs should not pipe to the parent terminals stdio.
[ ] - The Emulator's logs should also continue to be written to the log file.
[ ] - Winston should use one transport: File.

Refactor how the logs work:

[ ] - Instead of overriding the console methods with winston methods, move logging into its own module that can be imported where its needed and components can call logger.log() instead of console.log(), etc.

Codecov Report

Merging #89 into master will increase coverage by 0.03%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
+ Coverage   76.44%   76.47%   +0.03%
==========================================
  Files          52       52
  Lines        2097     2100       +3
==========================================
+ Hits         1603     1606       +3
  Misses        494      494

Impacted Files
Coverage Δ

src/cli/controller.js
73.97% <ø> (ø)
:arrow_up:

src/cli/commands/config/index.js
100% <ø> (ø)
:arrow_up:

src/options.js
100% <ø> (ø)
:arrow_up:

src/cli/commands/start.js
73.33% <ø> (ø)
:arrow_up:

src/emulator/main.js
89.36% <75%> (-1.55%)
:arrow_down:

src/cli/commands/delete.js
88.23% <0%> (+2.94%)
:arrow_up:

Continue to review full report at Codecov.

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

Hey Jason, most of this sounds good! I'd have to ensure that if we're no longer overriding console with winston methods, console.log from the functions code will continue to be displayed in the terminal.
As well, I noticed that emulator code has a lot of console.logs in it right now. Are these meant for debugging, or are these things that you'd like to display to the user? Ideally I would like a differentiation between console.logs from the user's functions code, and console.logs from the emulator. So that the Firebase CLI only pipes the console.logs from the user's functions code, but not from the emulator. I'm not sure how to do this other than to either elevate the severity of the console.logs from user code to console.warn, or to downgrade the console.logs from the emulator to console.debug. Do you have any thoughts on this?
I also want to confirm that it is your intention to not display the error messages in the terminal when tail is set to false.

All stdout and stderr from a function should always be captured and sent to the log file and in the case of tail: true, to the terminal as well.
The console.log() calls that you're seeing are the Emulator matching what production does, e.g. Function execution started., Execution took 16ms, etc. Those should also always be captured as well.
Emulator debugging logs (console.debug()) are only logged if verbose is set to true. One would normally only set verbose to true if debugging the Emulator itself.

Sounds good. I agree with your rationale that the emulator should provide the same logs as production. I am also seeing cases such as https://github.com/GoogleCloudPlatform/cloud-functions-emulator/blob/master/src/cli/controller.js#L173, where the messages do not show up in production. Having them be written to stdout is cluttering up the terminal. Perhaps these should be changed to logger.debug instead?
Should error logs be sent to the terminal if tail is set to false?

Actually, some of the logs should conditionally change their log level based on the value of the tail option. For example, if tail is false then you're using the functions CLI and you do want to see some of those logs (to match what gcloud beta functions prints to the terminal).

Should error logs be sent to the terminal if tail is set to false?

No, because if tail is false, then the terminal has released the child process and the parent process may very well have already exited.

I've added more commits:

Put logging into its own file (utils/logger). So another file can use it by requiring it, and then calling logger.info(), or logger.debug(), etc.
Do not unref child process if tailing is true, and listen to ctrl+C to terminate parent and child processes.

Let me know what you think so far. As for your comment about conditionally changing log levels, what do you think about making use of more granular levels for logging? Winston supports all 6 of the npm logging levels. My suggestion would be:

logger.error: use for error messages
logger.info: use for messages that show up in production Stackdriver logs (e.g. Function execution started)
logger.verbose: use for messages that show up in gcloud (e.g. 'Waiting for operation to finish...')
logger.debug: use for messages for debugging the emulator itself

The Firebase CLI would use info level logs, and this CLI would can default to verbose level logs.
This would also mean adding one more config option (debug), so log level would be verbose if verbose=true, and debug if debug=true.


feat(26217): Add missing member fix should work for missing function

Fixes #26217

@typescript-bot pack this


Fix whitespace character escaping in documentation

This got me confused for a bit when reading the documentation on github, where the backlash wouldn't appear. I'm not familiar with the .rst format, this could just as well be a github frontend bug.

Thanks for the PR.
We have improved the documentation and this is no longer needed but if you find any other issues please let us know.


Exclude tests from builds

@vyasr The test_python.sh script runs the tests from the install dir. I ran into issues in #128 when I tried to change it to run the tests from the source tree.
Do you think we can switch to running tests from the source tree now, or do we have to sort out why they're failing when we do that?

I'm discussing this with Paul offline but I think we should be able to get this fixed in 128.


Add notice that this won't work for private repositories

This morning I tried to use this plugin for one private repository of my organziation. Since the plugin uses the html blame from Github it cannot be used with private repositories. See the discussion in facebook/mention-bot#8 as well.

I also added a workaround for working with private repositories. This uses the commits api from github and gets the number of commits for a given file per author and uses that count to make a guess.

Thanks! What if you log in to github before running plugin?

Hey @wojteklu,
As @claaslange, I would be interested to have your plugin working with private repositories. If this code worked, I'd be curious to know why this has not been merged?
Cheers!


Update Google Analtyics/Tag Manager/Optimize code to latest recommendations

Our implementation had fallen out of step with the latest recommendations from Google on using these services all together
This PR rectifies that.
The apparent effect of being out of step is that we are recording far fewer Optimize sessions than we should be (roughly ¼ of all traffic to a page, as measured by Analytics) is getting counted as a completed Optimize session).
This needs to be deployed as a hotfix ASAP, because we're starting an important Optimize test on Wednesday, and we'd like a little time to test and see that it resumes performing as expected with this new code.
Additions

Page hiding snippet to prevent flicker effect when alternate code is loaded by Optimize
Snippet to explicitly load Optimize (as opposed to letting Google Tag Manager load it, as before)

Testing
You can't, practically, other than confirming that the new code outputs correctly when the template is rendered.
Notes

This blog post from a group with which @kelleyholden did some training with a few months ago

Todos

Once deployed, wait a day or so for new test data to roll in and confirm that the number of Optimize sessions go back to normal.

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

Does this decrease the overall performance of the site? How does this tie into the work that @cfarm is doing?

Does this decrease the overall performance of the site? How does this tie into the work that @cfarm is doing?

I don't think so, because it's not loading any new scripts (the only code it's adding to a page load is the literal code added in this PR). But I don't think there's any real way to find out without testing it in production.
I'm certainly interested in the work @cfarm is planning to do on measuring and improving our performance, but I don't think we can wait for that to make this change. We've promised our stakeholders an A/B test beginning on Wednesday, and we need to be sure we're getting the right data.

Sounds like they recommend jQuery come before the Google code, so this should move above it.

Google Analytics/Tag Manager gets crazier every year. Still, reviewing the support page linked about, it at least makes sense to me. We should maybe figure out a date and time to regroup on this and see if it fixes things and if we can make sure it's not interfering with performance or other functions.
Other than that, 👍

@anselmbradford That should only be in cases where we need jQuery as part of an Optimize experiment. To my knowledge we've not needed that yet. Can you confirm, @kelleyholden?

I dont think we've needed that yet, but i think @virginiacc might know better than me.

@anselmbradford That should only be in cases where we need jQuery as part of an Optimize experiment. To my knowledge we've not needed that yet. Can you confirm, @kelleyholden?

That's correct, it's a recommended best practice, not a requirement. If you're in there you could move it, but doesn't need to hold up the PR.

@kelleyholden @Scotchester We have been using jQuery in the OAH email signup tests, but it's not something we're likely to do very often.

I guess I don't see the harm in doing it. It's already in the head. I'll make that change and merge.


feat(gas): implement gas refund mechanism

Feature Request
Implement gas refunds mechanisms of the EVM. The gas refunds will be reimbursed to the user after processing the transaction.
Describe the Feature Request
Describe Preferred Solution
Related Code
Additional Context
If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[ ] Yes
[ ] No

@enitrat may I work on this feature this week or did you plan to self assign?
Merry Christmas by the way :)

@TAdev0 I'll take it as it's the last step for gas mechanism and quite complex


💲[Native Checkout]  Project Pledge States UI

📲 What
The four of the five core states a project can be in:

Back this project
Manage
View your pledge
View rewards

🤔 Why
Depending on the project pledge state users can back, manage, view pledge or reward with this CTA.
🛠 How
The PledgeCTAContainerView displays the CTA depending on state. The state is determined in the PledgeCTAContainerViewViewModel where func projectStateButton(backer: User, project: Project)
returns a PledgeStateCTAType.
👀 See

Live Project 🐛
Non-Live Project 🦋

---
---

✅ Acceptance criteria
Test Non-Live/Backed Project

[x] Navigate to Profile tab
[x] Scroll and tap on a Non-Live project under the backed screen
[x] View Your Pledge CTA should show in the project page

Test Non-Live/Not Backed Project

[x] Navigate to Search tab, search for a non-live/unbacked project (ex. Cat cafe)
[x] Scroll and tap on a Non-Live/Unbacked project
[x] View Reward CTA should show in project page

Repeat steps w/ Unsuccessful projects too.

Test Live/Not Backed Project

[x] Tap on project in Explore screen
[x] Scroll and tap on a Live/Unbacked project
[x] Back this project CTA should show in project page

Test Live/Backed Project

[x] Navigate to Profile tab
[x] Scroll and tap on a Live project under the backed screen
[x] Manage CTA should show in the project page

⏰ TODO

[x] Fix snapshots

Hey @cdolm92 I know this is WIP but looking at these changes I think things may have gotten lost in your merge. I think it might be easier to roll back to before the merge and instead reapply your changes on top of master. Happy to help with that!

MultiLineButton
I wanted to verify with @ifbarrera & @dannyalright that the button is supposed to not change its height and use multiple lines for longer texts (i.e. with Dynamic Type accessibility font)? I thought this used to work previously but now I couldn't make the button expand vertically). Instead it's been trimming the text in the middle

I'm concerned about the height of this button taking up too much space if we expand it vertically as its container is fixed to the button. Thoughts? Otherwise I'm in favour of expansion.

Spacing
@dannyalright is this spacing enough? Seems like the text is really close to the button

@cdolm92 could you please add another unit (6pt) of spacing? Sorry for not specifying this in the design work.

Dynamic Type
I've noticed that scaling the font up collapses the information we're showing to the user into a very limited and unusable state (see bellow).

Wonder if we should design this button with more accessibility in mind @dannyalright maybe similar to what we do with some of the cells on the rewards pledge (see bellow).

This is an example of the same cell under normal size (items are aligned horizontally)

while for accessibility sizes we align items vertically. This gives the user enough information at all times.
Agreed. @cdolm92 could you please make the button wrap?

@dannyalright just for a reference we've used this MultiLineButton to also expand some other buttons which are part of the checkout workflow in order for the user to see the whole thing. We've made this decision while you were away.
Here's an example of how it looks (hopefully this wasn't a huge misstep)

MultiLineButton
I wanted to verify with @ifbarrera & @dannyalright that the button is supposed to not change its height and use multiple lines for longer texts (i.e. with Dynamic Type accessibility font)? I thought this used to work previously but now I couldn't make the button expand vertically). Instead it's been trimming the text in the middle

I'm concerned about the height of this button taking up too much space if we expand it vertically as its container is fixed to the button. Thoughts? Otherwise I'm in favour of expansion.

Spacing
@dannyalright is this spacing enough? Seems like the text is really close to the button

@cdolm92 could you please add another unit (6pt) of spacing? Sorry for not specifying this in the design work.

Dynamic Type
I've noticed that scaling the font up collapses the information we're showing to the user into a very limited and unusable state (see bellow).

Wonder if we should design this button with more accessibility in mind @dannyalright maybe similar to what we do with some of the cells on the rewards pledge (see bellow).

This is an example of the same cell under normal size (items are aligned horizontally)

while for accessibility sizes we align items vertically. This gives the user enough information at all times.
Agreed. @cdolm92 could you please make the button wrap?

The spacing is on iPhone SE and so I think it's fine as it is.


Fitting fails with AR emissions

import ssm
true_lds = ssm.LDS(15, 10, dynamics="none", emissions="ar")
x, y = true_lds.sample(1000)

lds = ssm.LDS(15, 10, dynamics="none", emissions="ar")
lds.fit(y, method="svi", variational_posterior="mf")

fails because of a dimension mismatch when trying to initialize the ARHMM using KMeans. In the case of an LDS (as opposed toS LDS) the number of clusters passed to Kmeans is just 1, so it's pretty much a no-op anyway.
It's line 111 (observations.py) that fails, because the dimension of data is wrong -- it's T x T x  D instead of T x D.
Seems like the cause might be here:
 xs = [self.emissions.invert(data, input, mask, tag)
              for data, input, mask, tag in zip(datas, inputs, masks, tags)]

This outputs a list where each entry is T x T x D.

Addressed in 0e397ac

• Generally, there is an issue here trying to call _invert in cases where it doesn't make sense. We should only try to call _invert if the emissions model has a single subspace and is not the NeuralNetwork emissions type.


[Snyk] Upgrade config from 3.3.11 to 3.3.12

Snyk has created this PR to upgrade config from 3.3.11 to 3.3.12.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 2 months ago.

Release notes

Package name: config

3.3.12 - 2024-06-25What's Changed

Remove usage of deprecated utils to fix warnings in Node 22 by @ KidkArolis in #764

New Contributors

@ KidkArolis made their first contribution in #764

Full Changelog: v3.3.11...v3.3.12

3.3.11 - 2024-02-01What's Changed

fix: webpack bundling compatibility by @ cbazureau in #757

New Contributors

@ cbazureau made their first contribution in #757

Full Changelog: v3.3.10...v3.3.11

    from config GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Merged into #116


Black font for all dark thems in macOS Big Sur

Describe the bug
The font color is black for all dark themes
To Reproduce
Steps to reproduce the behavior:

Select any one of the dark themes (e.g. 'Simple Dark')
The font color is black

Expected behavior
White font color for all dark themes.
Desktop (please complete the following information):

OS: macOS 11.0.1
Installation Method: homebrew
Version: Insomnia Core 2020.4.2

Hi, I tried to reproduce the bug on 'Simple Dark' and 'Hyper' but I was not able to.

Tested on:
Version: Insomnia Core 2020.4.2
Release date: 21/10/2020
OS: Darwin x64 20.1.0 // macOS BigSur 11.0.1
Electron: 9.1.1
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.69.1 OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.7 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.41.0
Can you try to inspect the console?
You can do that on View > Toggle DevTools and clicking on the Console panel.
If you find something interesting please, post it.

It looks like a font issue, I am using Source Code Pro font...

If I change it back to default, it works.

Btw, the font is registered correctly, I use it in other editors...

I experienced the same issue after upgrading to Big Sur from Catalina and found no way to resolve since. All themes rendered with dark fonts in the editor/results and text was unreadable.
I, too, am using a Source font, albeit Source Sans Pro instead of Source Code Pro.
Coming across this issue, I tried changing my fonts. No effect. Any font I selected continued to render dark even after restarting the app. Then... I selected to a non-Source font and immediately changed themes. Boom, back to the proper and expected text color. Changing back to Source Sans Pro maintains the correct colors and all is well.
I can no longer reproduce the issue to confirm the steps involved, but perhaps this may help others in a similar situation. It was some combination of changing the selected Interface Font and Text Editor Font followed by a change of theme immediately after.
Version: Insomnia Core 2020.5.2
Release date: 12/9/2020
OS: Darwin x64 20.1.0
Electron: 9.1.1
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.69.1 OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.7 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.41.0

I experienced the same issue after upgrading to Big Sur from Catalina and found no way to resolve since. All themes rendered with dark fonts in the editor/results and text was unreadable.
I, too, am using a Source font, albeit Source Sans Pro instead of Source Code Pro.
Coming across this issue, I tried changing my fonts. No effect. Any font I selected continued to render dark even after restarting the app. Then... I selected to a non-Source font and immediately changed themes. Boom, back to the proper and expected text color. Changing back to Source Sans Pro maintains the correct colors and all is well.
I can no longer reproduce the issue to confirm the steps involved, but perhaps this may help others in a similar situation. It was some combination of changing the selected Interface Font and Text Editor Font followed by a change of theme immediately after.
Version: Insomnia Core 2020.5.2
Release date: 12/9/2020
OS: Darwin x64 20.1.0
Electron: 9.1.1
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.69.1 OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.7 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.41.0

This looks like it may be resolved by #3223 - I am currently running the Alpha Build with no issue. Followed the steps above exactly as written with no issue.
Steps Taken:

Changed Interface Font
Changed Text Editor Font
Changed Theme

Outcome:
Theme rendered perfectly.


⚠ ClamAV has degraded performance

In 5b9d572, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in 966d899.


Solr core directories not being removed

Removing the solr core directories when files in the /data directory are in use results in the replacement core being nested in the old core directory e.g. corename/corename.
This might be resolved by using solr's api.

Should have been resolved by #6


[DS-3849] REST API items resource returns items in non-deterministic order (master port)

This PR updates the ItemDAOImpl class to make sure all item lists returned by this class will now be in a deterministic order.
Jira Issue: https://jira.duraspace.org/browse/DS-3849

@PhilipVis : Just an FYI, it looks like this port to master is failing a few tests in ItemTest.  Failures start here in Travis logs: https://travis-ci.org/DSpace/DSpace/builds/584044986#L3734

@tdonohue Thanks for notifying me on the failing tests. I had mistakenly edited method countArchivedByCollectionExcludingOwning instead of findArchivedByCollectionExcludingOwning. I have now fixed this, but unfortunately some tests are still failing. I'm not sure yet what the cause is now.

I have updated the failing test so that it will check the items ordered by UUID instead of title. Unfortunately this still results in the test failing in Travis, while on my local machine this did result in the test succeeding. Is there something else I need to do to?

@PhilipVis : sorry for the late reply. But, if Travis is failing, even though your local machine succeeds, then there must be some flaw in the test or the code itself.  The flaw is not immediately evident to me either, but maybe your local machine has something different that Travis does not?  Even differences in dependency version or operating system can sometimes result in different or unexpected behaviors. Travis is reporting that it is running Oracle Java 8, Maven 3.5.2 on Ubuntu Linux (those versions are reported near the top of the Travis logs).
I should also notice that (very occasionally) I've seen issues arise where the order in which Integration Tests run affects the output. This is usually encountered when one Integration Test fails to clean up after itself, and it then (unexpected) data from that test carries over into the next Integration Test. It's possible this is also the cause.
For what it's worth, I've also tried running these tests on my machine (Oracle JDK 8, Maven 3.3.9, Windows 10) using mvn clean install -Dmaven.test.skip=false -DskipITs=false and I'm seeing the same integration test failure that is reported in Travis CI.  (It is worth noting though that all unit tests succeed...it's just a failure in the findAllWithPaginationTest integration test)

it looks as though the failure is in expecting Public Item 2 but seeing Public Item 3. do the tests need some tweaking?

@PhilipVis : This appears to still need updating for our main branch. It looks like it was already applied to 6.x (in #2501)...so, it'd be a shame if this same fix doesn't appear in 7.x.


Add blackduck plugin in to the marketplace

Signed-off-by: Deepankumar Loganathan<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hello,
I created the repository after the plug-in development & test completed with the running Backstage instance. Post work done I created the PR too.


Modifying the AIX detect_os code to include the release and architecture

uname -rvp yields minor_release, major_release and architecture on AIX. Tested the code against an AIX 6.1 and 7.1 machine. The previous code displayed the machine ID number instead of the architecture (uname -m). With this fix the architecture is set to 'powerpc' on the hosts tested.

Thanks.

Released as v2.36.2.


Adds OpenTech Summit 2018 sample to the overview site

Checklist

[x] I have read the Contribution & Best practices Guide.
[x] My branch is up-to-date with the Upstream development branch.

Short description of what this resolves:
Adds OpenTech Summit 2018 sample to the overview site.

Server Link: https://open-event-webapp.herokuapp.com/
API endpoint: https://raw.githubusercontent.com/agbilotia1998/open-event/OTS18/sample/OpenTechSummit2018
API version: v1
Fixes #2061

@geekyd @ParthS007 @mayank8318 @pradeepgangwar please review, we need to merge https://github.com/fossasia/open-event/pull/290 before merging this PR.

Travis is failing. Please check.


FastAPI/Starlette integration

Hi.
Now authlib has a starlette integration, it would be great if loginpass would support it as well.

When Authlib v0.13 is released, loginpass will have a new release too.

@lepture Seeing that we are now passed v0.14 on Authlib, do you plan on releasing an update to the loginpass wrapper to support Starlette/FastAPI?

I think this is too necessary since some of the proposed configs in this library does not work with Starlette. I concretely mean Bitbucket and Azure ones. Implementing those two things is driving me crazy and I'm not sleeping too much these days because of it.
This is not just for helping me, but the entire world.
Please consider this feature

@gsemet @rubendibattista @mavaras
I'm pleased to receive PRs about Starlette integration.
Also, I'm looking for funding about Authlib and its related projects: https://docs.authlib.org/en/latest/community/funding.html

Hello, I am trying to setup a fastapi server for a react frontend. I've decided to use auth0 service as an identity provider, but it should not matter which provider is used.
Digging around made me get this error: from authlib.integrations.starlette_client import StarletteRemoteApp coming from 'loginpass._fastapi.py' while calling create_fastapi_routes function. I cannot find anything about StarletteRemoteApp. Perhaps I could look into it further if anyone knows anything about what this class is/was.
Can anyone provide any details about StarletteRemoteApp class or a wider context about Starlette + authlib + loginpass + fastapi situation?
Can anyone provide any workaround so I can make my React + FastAPI work with any identity provider by using lower level functions?
It might help us address also #83 , #91.


I want to contribute

Recently, I started making open-source contributions. I made some contributions to Flutter and Android apps. Now I want to make a contribution to some libraries. I have used Supabase personally for one of my Flutter projects, so it will be great if I can contribute to this library. Let me know if there is anything for me (bug fix, feature, improvement, or unit test).

@ishanvaghani Thanks for your interest. Feel free to play around with the library or go through the issues and find anything you can help with! Feel free to submit a PR at any time!


Fix system tests against invalid component versions

enforce the execution order + code error fixes

Current coverage is 84.20% (diff: 100%)

Merging #477 into master will increase coverage by 0.70%

@@             master       #477   diff @@
==========================================
  Files            18         18
  Lines          2079       2178    +99
  Methods         374        374
  Messages          0          0
  Branches        181        181
==========================================
+ Hits           1736       1834    +98
- Misses          343        344     +1
  Partials          0          0

Powered by Codecov. Last update 62f2940...601e81c


feature: record video demonstration for use of KubeStellar logical cluster component in place of kcp

Feature Description
I would like a demonstration of how KubeStellar can use the logical cluster component to connect with kcp, kubeflex, or other api machinery provider/client
Proposed Solution
a < 5min recorded demonstration video that shows how KubeStellar can use kcp or kubeflex as a logical cluster provider.
Want to contribute?

[ ] I would like to work on this issue.

  Or will this be ready tomorrow?

@ezrasilvera can we get an update on this?  Is this necessary?  Should it be closed?

@effi-ofer can you please add the link to the example you mentioned on the scrum call.  Thank you


skip_after_filter :intercom_rails_auto_include does not work with turbolinks

In our app, we are using turbolinks and intercom-rails. We do not want the Intercom interface to show up for views under LessonsController, so we include the line skip_after_filter :intercom_rails_auto_include in that controller.
When we visit a page in LessonsController directly, the Intercom include is not served to the client (as expected).
However, if the client starts in a view under DashboardController (which does include Intercom), then clicks a link to a view in LessonsController, the Intercom interface is still shown because turbolinks does not remove the Intercom include.

My current workaround is to add the class .hide-intercom to the body of those pages and add the following to the CSS:
.hide-intercom .intercom-container {
  display: none;
}

Hi there, sorry about the delay in responding to this. I'd recommend using our client-side API in this case to close the messenger on these pages.
e.g. the following will close the messenger:
Intercom('shutdown')

There's more information here http://docs.intercom.io/install-on-your-web-product/intercom-javascript-api.


Fix auto generated API docs

It was pointed out to me that the autogenerated API doc generation seems to have broken at some point between 0.6.5 and 0.6.7.
Needs investigation.

This is now fixed in latest. Turns out read the docs was building docs in Python 3.7, which broke because the typing_extensions package used in 3.7 builds of the new colour parsing module was not in the requirements.txt.


loadModel Error if it is called in an isolate

I have a set of images and I want to run detectObjectOnBinary parallely. but I am getting this error if I try to load the model in an isolate.
ServicesBinding.defaultBinaryMessenger was accessed before the binding was initialized.
E/flutter (10626): If you're running an application and need to access the binary messenger before `runApp()` has been called (for example, during
plugin initialization), then you need to explicitly call the `WidgetsFlutterBinding.ensureInitialized()` first.
E/flutter (10626): If you're running a test, you can call the `TestWidgetsFlutterBinding.ensureInitialized()` as the first line in your test's `main()` method to initialize the binding.
E/flutter (10626): #0      defaultBinaryMessenger.<anonymous closure> (package:flutter/src/services/binary_messenger.dart:76:7)
E/flutter (10626): #1      defaultBinaryMessenger (package:flutter/src/services/binary_messenger.dart:89:4)
E/flutter (10626): #2      MethodChannel.binaryMessenger (package:flutter/src/services/platform_channel.dart:140:62)
E/flutter (10626): #3      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:314:35)
E/flutter (10626): #4      Tflite.loadModel (package:tflite/tflite.dart:14:27)
E/flutter (10626): #5      detectCharsFromBlock (package:michelin/display.dart:236:29)
E/flutter (10626): #6      _Processor.process (package:pmap/pmap.dart:17:66)
E/flutter (10626): #7      _process (package:pmap/pmap.dart:30:17)
E/flutter (10626): <asynchronous suspension>
E/flutter (10626): #8      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:310:17)
E/flutter (10626): #9      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

The issue is in that, tflite is not registered in isolate. So you would better dig into "how isolates works" rather than blame.


Remove references to gemini as comments so they can be uncommented later

Fixes #
Changelog Entry

Description

Specific Changes

[ ] Testing Added

Coverage remained the same at 64.04% when pulling<PHONE_NUMBER>55cedb7a53a93b34faa1e98ec6847a on corinagum:embed-docs into b9fbe2e21a1857a444b991aee35755da9bcfe5f5 on microsoft:master.


Move Namespace to MII

Moves any namespace related mentions to MII.
Specifically, www.netzwerk-universitaetsmedizin.de is moved to medizininformatik-initiative.de.
This is a breaking change since tasks sent by a client need to be updated (instantiateUri, receiver etc.)

Should also take the following changes into account: https://github.com/medizininformatik-initiative/feasibility-dsf-process/pull/18


Feature request

Hi,
that plugin is really great!
During my debugging with my Android, it get's a bit annoying if that plugin is installed.
So my request is:
Is it possible to turn of the cryptonize part if the app is build with --debug ?
Or the way round: Only activate the crypto-plugin if the app is build with --release ?
Sven

That's actually a bad idea. That way we can't debug the encrypted app. Sometimes it might cause issues.
The best way is to add the plugin only when you're releasing the app, or debugging its final version.

How often you have to debug the encrypted app? If you have to remove and add the plugin everytime, it easily happens that you forget to add the plugin where you actually have to.
I think in your cases it would be better to implement an extra switch for debugging the app while encyrpted.


Works on macOS

I was able to remove the -lgcc* links and it runs on macOS.
% ./a.out
% echo $?
0

Yes - I think that it works on certain MacOS versions but has warnings on certain versions warning about the libunwind metadata (because it has a conflict with the one bundled as well).
Last time I ran it had something like errors like,
libunwind: Unsupported .eh_frame_hdr version


Org man

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks Steph! Merging this request. There's a couple header additions folks pointed out but overall i think all changes will be addressed in your prod example


Test case for large file restore issue

@bwp91 - This is a test case that recreates the ZLIB - unexpected end of file issue during restore.   I created this as we can continue to investigate the large file restore issue. https://github.com/homebridge/homebridge-config-ui-x/issues/1856
After looking into this further, my working theory is that the upload function is truncating or ??? the upload at 1048576 bytes, which is exactly 1MB.  Which feels like a setting within the UI framework around maximum file upload size.
Currently this test case fails, as uploads greater than 1mb fail with the error.

Fails
● BackupController (e2e) › POST /backup/restore large backup
expect(received).toBe(expected) // Object.is equality

Expected: 201
Received: 500

  340 |     });
  341 |
> 342 |     expect(res.statusCode).toBe(201);
      |                            ^
  343 |
  344 |     await new Promise((r) => setTimeout(r, 100));
  345 |

  at Object.<anonymous> (test/e2e/backup.e2e-spec.ts:342:28)

@NorthernMan54 if you take this test and run it from the beta branch, then for me it passes, so maybe this has been fixed in the beta version?

I was working with latest, so I just switched to base on the PR, and it showed as fixed.  I then increased the backup file size, and was able to recreate. Hopefully you are also able to recreate.

Does it fail when the file is over 25mb?
https://github.com/homebridge/homebridge-config-ui-x/blob/eea2b9c793a028fceb49fc0ffcf4b651e294d944/src/main.ts#L32

@bwp91 I think I have figured it all out.   https://github.com/fastify/fastify-multipart is used to manage the restore upload, and we defined a maximum fileSize of 25 * 1024 * 1024 ( 25 MB ).  When a file larger than this is uploaded, the data is truncated, and triggers a ZLIB - unexpected end of file.
To reduce the noise around these issues, I have added two guard rails.
1 - A backup file that is larger than 25 * 1024 * 1024 ( 25 MB ), will trigger an error to the homebridge log file Backup file exceededs maximum restore file size 25mb.
2 - When doing a restore, if the upload is truncated ( exceeds the maximum fileSize ), the message Restore file exceeds maximum size 25MB
Originally I was going to cancel the backup, but that would be leave people without anything to work with.  So I figured just logging an error was more manageable.


OVJ3.1 for Mac - can't use study queue to open data

Describe the bug
After opening a study, you can't open individual spectra from the study queue.  For example, double-clicking the spectrum name or dragging it to a viewport or right-clicking and selecting "open experiment" – all these simply do nothing.  You have to use the File-Open...  menu option to open spectra in each viewport.
To Reproduce
Steps to reproduce the behavior:

Open OVJ 3.1
Use File-Open... to open a study (you can see the individual spectra populate the study queue if 'sample' is enabled)
Try to double click a spectrum or drag and drop it into a viewport or right-click on the spectrum name and choose "open experiment"
Nothing happens

Expected behavior
Normally, these actions would open the spectrum in the active viewport (or, if drag-and-drop, in the viewport that it was dragged to)
Version of OpenVnmrJ you are using.
OpenVnmrJ 3.1
downloaded from github on Nov 29th, 2021
Messages and Logs
No errors, it simply doesn't do anything
Environment (please complete the following information):

OS: OSX 10.15.7

Additional context
General processing appears to work fine aside from this.

I cannot duplicate this behaviour. Can you send the ~/vnmrsys/VnmrjMsgLog file? It may contain some clues.

Here you go.
-Scott

Dr. Scott Burt
Teaching Professor
NMR Facility Manager
Department of Chemistry and Biochemistry
Brigham Young University
C414 BNSN / C008A BNSN
Provo, Utah  84602-5700
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
chembio.byu.edu/nmr-facility https://chembio.byu.edu/nmr-facility
email: @.***

On Dec 2, 2021, at 3:04 PM, Dan Iverson @.***> wrote:
I cannot duplicate this behaviour. Can you send the ~/vnmrsys/VnmrjMsgLog file? It may contain some clues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/OpenVnmrJ/OpenVnmrJ/issues/656#issuecomment-985038137, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGI4R6LB5ICUFTGBHOJAMV3UO7UN7ANCNFSM5JHRPM6A.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

No file was attached.

Hmmm... I could see a file, but maybe it didn't like that it had no extension.  I'll make it a tgz to see if that helps.

-Scott

Dr. Scott Burt
Teaching Professor
NMR Facility Manager
Department of Chemistry and Biochemistry
Brigham Young University
C414 BNSN / C008A BNSN
Provo, Utah  84602-5700
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
chembio.byu.edu/nmr-facility https://chembio.byu.edu/nmr-facility
email: @.***

On Dec 2, 2021, at 3:32 PM, Dan Iverson @.***> wrote:
No file was attached.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/OpenVnmrJ/OpenVnmrJ/issues/656#issuecomment-985055192, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGI4R6KRUR3CZHWDTWQH2SLUO7XXNANCNFSM5JHRPM6A.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Dan,
I tried debug tracing for this OSX study queue issue.  After debug('c3') I opened a data set and then tried double clicking several spectra followed by right-clicking and choosing open followed by dragging a protocol to the viewport.

-Scott

Dr. Scott Burt
Teaching Professor
NMR Facility Manager
Department of Chemistry and Biochemistry
Brigham Young University
C414 BNSN / C008A BNSN
Provo, Utah  84602-5700
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
chembio.byu.edu/nmr-facility https://chembio.byu.edu/nmr-facility
email: @.***

On Dec 2, 2021, at 3:32 PM, Dan Iverson @.***> wrote:
No file was attached.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/OpenVnmrJ/OpenVnmrJ/issues/656#issuecomment-985055192, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGI4R6KRUR3CZHWDTWQH2SLUO7XXNANCNFSM5JHRPM6A.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Can you send me the debug file at<EMAIL_ADDRESS>


Can´t see the options for tasks that refers to: Duplicate, Duplicate to another project, Move to another project and Delete

Expected behaviour
I should see the options: Duplicate, Duplicate to another project, Move to another project and Delete in the task view
Actual behaviour
I can´t see the options: Duplicate, Duplicate to another project, Move to another project and Delete in the task view, I can't execute non of the options already mentioned.
Steps to reproduce

Configuration

Kanboard version: v1.0.26
Database type and version: sqlite
PHP version:
OS: Debian
Browser: Mozilla Firefox


Lens not starting after installed with snap: ERR_FILE_NOT_FOUND

Describe the bug
lens does not start after install with snap.
message:
(node:109547) electron: Failed to load URL: file:///snap/kontena-lens/185/resources/static/splash.html with error: ERR_FILE_NOT_FOUND
To Reproduce
Steps to reproduce the behavior:

installed lens on UBUNTU with snap install kontena-lens --classic
started on command line with kontena-lens
no window opens, log appears on screen containing ERR_FILE_NOT_FOUND

Expected behavior
lens starting, window coming up
Screenshots
Environment (please complete the following information):

current snap version 185
Ubuntu 22.04
snap --classic

Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
$ kontena-lens
(node:109547) [DEP0111] DeprecationWarning: Access to process.binding('http_parser') is deprecated.
(Use `kontena-lens --trace-deprecation ...` to show where the warning was created)
info:    ▪ [CLUSTER-STORE]: LOADING from  ... +0ms
info:    ▪ [CLUSTER-STORE]: LOADED from /home/gaylord/snap/kontena-lens/185/.config/Lens/lens-cluster-store.json +4ms
info:    ▪ 🔌 Starting LensProxy +2ms
info:    ▪ [LENS-PROXY]: Proxy server has started at <IP_ADDRESS>:39557 +3ms
info:    ▪ 🔎 Testing LensProxy connection ... +0ms
info:    ▪ [USER-STORE]: LOADING from  ... +3ms
info:    ▪ [USER-STORE]: LOADED from /home/gaylord/snap/kontena-lens/185/.config/Lens/lens-user-store.json +9ms
libva error: vaGetDriverNameByIndex() failed with unknown libva error, driver_name = (null)
info:    ▪ ⚡ LensProxy connection OK +16ms
warn:    ▪ No suitable icon found for task bar. +7ms
info:    ▪ [CREATE-ELECTRON-WINDOW]: Loading content for window "splash" from file: /snap/kontena-lens/185/resources/static/splash.html... +0ms
error:   ┏ [CREATE-ELECTRON-WINDOW]: Failed to load window "splash" +45ms
error:   ┗ [1] { code: -6, desc: 'ERR_FILE_NOT_FOUND' }
(node:109547) electron: Failed to load URL: file:///snap/kontena-lens/185/resources/static/splash.html with error: ERR_FILE_NOT_FOUND
This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
Error: ERR_FILE_NOT_FOUND (-6) loading 'file:///snap/kontena-lens/185/resources/static/splash.html'
    at rejectAndCleanup (node:electron/js2c/browser_init:161:7647)
    at EventEmitter.failListener (node:electron/js2c/browser_init:161:7860)
    at EventEmitter.emit (node:events:538:35)
    at EventEmitter.emit (node:domain:475:12)
    at emitter.emit (/snap/kontena-lens/185/resources/app.asar/node_modules/@sentry/electron/main/integrations/electron-breadcrumbs.js:119:20)
(node:109547) UnhandledPromiseRejectionWarning: Error: ERR_FILE_NOT_FOUND (-6) loading 'file:///snap/kontena-lens/185/resources/static/splash.html'
    at rejectAndCleanup (node:electron/js2c/browser_init:161:7647)
    at EventEmitter.failListener (node:electron/js2c/browser_init:161:7860)
    at EventEmitter.emit (node:events:538:35)
    at EventEmitter.emit (node:domain:475:12)
    at emitter.emit (/snap/kontena-lens/185/resources/app.asar/node_modules/@sentry/electron/main/integrations/electron-breadcrumbs.js:119:20)
(node:109547) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 7)
info:    ▪ [CREATE-ELECTRON-WINDOW]: Window "splash" loaded +96ms
^Cinfo:    ▪ [EXTENSIONS-STORE]: LOADING from  ... +4m
info:    ▪ [EXTENSIONS-STORE]: LOADED from /home/gaylord/snap/kontena-lens/185/.config/Lens/lens-extensions.json +1ms
info:    ▪ [FILESYSTEM-PROVISIONER-STORE]: LOADING from  ... +0ms
info:    ▪ [FILESYSTEM-PROVISIONER-STORE]: LOADED from /home/gaylord/snap/kontena-lens/185/.config/Lens/lens-filesystem-provisioner-store.json +1ms
^C

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
kubeconfig worked with older version of lens, also i do not think it is related since the problem is quite obvious.

Additional context
Add any other context about the problem here.
$ ls -l /snap/kontena-lens/185/resources/static/splash.html
ls: cannot access '/snap/kontena-lens/185/resources/static/splash.html': No such file or directory
gaylord@gaylord-XPS-13-9310:~$ ls -l /snap/kontena-lens/185/resources/static/
ls: cannot access '/snap/kontena-lens/185/resources/static/': No such file or directory
gaylord@gaylord-XPS-13-9310:~$ ls -l /snap/kontena-lens/185/resources/
total 983838
-rw-r--r-- 1 root root<PHONE_NUMBER> Dez  1 15:35 app.asar

I have the same issue - apparently many directories are missing in resource folder.


Breaking story fix

Who is this for?
Users of story search
What is it doing for them?

Page was breaking when an article didn't have a promo image. We thought it always had one, but turns out it doesn't (see examples here https://wellcomecollection.org/series/WcpOjygAAJsLEXTc). I've taken the same treatment as the example page; no promo image, no replacement. This might be something we'd like to review down the line but for now this is a bug fix.

Note: Sometimes they do have an image, but no richCrops or simpleCrops information and therefore get ignored. We do want all our images to be of the same size so this avoids unequal grids.

Thinking out loud: if we have image data but no human-specified crops, could we drop in the image with object-fit: cover to look slightly better than no image at all?

could we drop in the image with object-fit: cover to look slightly better than no image at all?

That's what I was doing before with a lot of complex max-widths, etc. getCrop now allows us to not worry about all of it and just allow images to be as is, and simplify the code so well... I'd like to dig deeper into Prismic content and how it works to understand it all, but am also conscious we'll be moving a lot to ES and am less sure of what that would change for images?
I can still investigate a different solution/compromise as part of the follow-up ticket, but it requires a bit more time that I'd think would be best done separately.


Expose transform

See https://github.com/tensorflow/addons/pull/2293

Thanks for the PR! This is intended as a private utility. We should not expose it in the public API so as to keep our API surface small. Implementer-facing private APIs like this one should typically be forked in your own repo, or otherwise reimplemented.

Thanks for the PR! This is intended as a private utility. We should not expose it in the public API so as to keep our API surface small. Implementer-facing private APIs like this one should typically be forked in your own repo, or otherwise reimplemented.

I understand the API surface point but this is really going to create an overhead for "friends" repositories in the SIG ecosystem. E.g. what Is the Keras CV approach? Is going to fork this or use the private API?
Probably It Is better to handle private API changes and use private API that sync forks in our ecosystem. What do you think?


animation on reset

Is it possible to use animation on reset action?

I'd like to ask the same question?

+1

+1

+1

+1

+1

+1

Any updates on this please...

+1

+1 please!

1

1

+1

+1

+1

+1

+1

+1

no news on this yet? it seems a recurrent issue


Fix issue due date always being UTC

Fixes: https://github.com/go-gitea/gitea/issues/11686
Creates a local 00:00 date and toISOString will convert it to UTC. So essentially the exact due time is now always 00:00 in the user's current timezone instead of previously 00:00 UTC.

Unsure if this is correct approach. Due date does not present time picker so no time should be saved in database.
Most likely we should replicate behavior from milestone, which sets due date to 23:59 in server's local timezone.

Sounds good from a consitency perspective to have timezone-independent dates on the server, but the display issue on the client (which converts from UTC to local) will remain in that case.


Request: support updating collection field on on-chain metadata

I ran into an issue trying to add more NFTs to a collection that had already been minted. With Metaplex's new collection standard, when using Candy Machine to mint the new items, this resulted in a separate collection token for the new items, this resulted in, for instance: Phantom wallet shows them as two collections even though they share identical data on the arweave metadata.
In any case, it would be useful to be able to set the collection field, presumably with the set command.

Talk to a Solana Support on the Live support page so that your issue can be looked into.

Talk to a Solana Support on the Live support page so that your issue can be looked into.

Thanks for your reply! Would you mind expanding on how they would be able to help me? I posted here because metaboss is a useful tool that I've used successfully multiple times for when I need to administer the tokens for the various projects I work on, so I'd like to understand how feasible or unfeasible it is to add it to the tool. Thank you!

Looks like #126 might do what I need! Thanks @samuelvanderwaal!


Completed the Shell assignment.

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

@dtxe
@juliagallucci
@michaeladrouillard

Checklist

[ ] I can confirm that my changes are working as intended

Hi Julia, thank you for reviewing my assignment. I have two questions and appreciate if you could please help. For question #4 it is asking for the fifth word of the fifth fictional Space Wars series: "Space Wars: Future Legends and Past Legacies" - Isn't the fifth word here "and"? In official answer it says "Past".
For question #6 it is asking for the fourth word to the fourth Hunger Games movie: "Hanger Games & the Stars of Darkness" - Isn't the fourth word "Stars"? In the official answer it says "Star" instead.

Hi Julia,
Thank you for reviewing my assignment. I have two questions and would
appreciate it if you could please help. For question # 4 it is asking for
the fifth word of the fifth fictional Space Wars series: "Space Wars:
Future Legends and Past Legacies" - Isn't the fifth word here "and"? In
official answer it says "Past".
For question # 6 it is asking for the fourth word to the fourth Hunger
Games movie: "Hanger Games & the Stars of Darkness" - Isn't the fourth word
"Stars"? In the official answer it says "Star" instead.
Thank you,
Zarrin
On Tue, Apr 30, 2024 at 5:50 PM juliagallucci @.***>
wrote:

@.**** approved this pull request.
Official answer: Paper Rings 10 Meets Past Lucky Star
—
Reply to this email directly, view it on GitHub
https://github.com/zrasi/shell/pull/2#pullrequestreview-2032586034, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BH3W3SD66AX62VYCYO4AH33ZAAG3JAVCNFSM6AAAAABG3KQPS2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMZSGU4DMMBTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe "and" was not counted as a word, so it would be considered Past
Yes, you are correct, it is Star!

Thank you Julia for your response.
I only have this one different from the official answer, I am not sure if
it is considered pass or not, given that there wasn't any instruction on
not to count "and". I hope it is considered Pass! 🙂
On Wed, May 1, 2024 at 11:30 AM juliagallucci @.***>
wrote:

I believe "and" was not counted as a word, so it would be considered Past
Yes, you are correct, it is Star!
—
Reply to this email directly, view it on GitHub
https://github.com/zrasi/shell/pull/2#issuecomment-2088637307, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BH3W3SDW7ANDYTZDMRMWTOLZAEDB7AVCNFSM6AAAAABG3KQPS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBYGYZTOMZQG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, it's a pass!

Perfect! Thank you!
On Wed., May 1, 2024, 11:50 a.m. juliagallucci, @.***>
wrote:

Yes, it's a pass!
—
Reply to this email directly, view it on GitHub
https://github.com/zrasi/shell/pull/2#issuecomment-2088664726, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BH3W3SAAYC4MV6AEKYVFOSDZAEFLFAVCNFSM6AAAAABG3KQPS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBYGY3DINZSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


build output content has something like ["__file","/Users/xxxxx/workspace/www/src/pages/Home.vue"]

Describe the bug
the file path is in production build output ,is ok?
Reproduction
always show
System Info
no

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required

@airene this was fixed in #356. I'm thankful you reported it because it motivated me to try and find the source of the bug 🙏🏽

great work! Thanks you too.


Launch children in a new process group.

This has the effect that, when einhorn is run interactively, signals
sent by the terminal (e.g. in response to ^C) only go to the einhorn
master, and not to any of its children.
r? @zenazn

LGTM


Not show closed indices by default

By default closed indexes checkbox always plugged on. I plug this off, go to another partition of UI.  I again plug this off.
Why this behaviour? I close indexes, because they are not needed now! But UI force show it me again and again. Very bad UX :-1:
I try to solve this via https://github.com/lmenezes/cerebro/pull/159, but maybe I was wrong of my code.

hey @darky
This is very bad UX for you, for me this works perfectly.
I did close #159 because you just created a fix for an issue that you have, without asking if this was something that I thought was interesting, which I don't(at least in this form).
Also, the code is not really doing what you think it is, and it doesn't really follow at all the contributing guide https://github.com/lmenezes/cerebro/blob/master/CONTRIBUTING.md

Ok, compromise, maybe would be good, if UX will save state in Local Storage?

@darky this could be, but could also be annoying that you keep this across sessions. I anyway don't really have much time atm for working on cerebro, so I won't be looking into this any time soon. If this is such a big deal for you, you can always just build from source and change this on the HTML(its jus ta checkbox, so setting it to false would do the trick).


AST: Add a no-type-parameters early return to findProtocolSelfReferences

I was going to add this to ProtocolDecl::findProtocolSelfReferences as well, but apparently, GenericFunctionType is purposely ignorant of its type parameters

@swift-ci please smoke test

@swift-ci smoke test

ping @slavapestov

Let's just take this
⛵


⚠ Bofrost.es has degraded performance

In e8bb6d0, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1633 ms

Resolved: Bofrost.es performance has improved in 600b6a3 after 6 minutes.


Can import a proof multiple times, creating duplicate asset records

Using latest lnd and latest on taro
To reproduce:

Export a proof via REST
Import this proof via REST

You'll now see duplicate asset records created

Also, I always assumed the "amount" field was supposed to be only my current balance of the asset, correct?
I would expect it to be 0 if I import a proof of an asset I do not own

So what we want to do here is fix the unique constraint on the assets to resolve this. Also if we get a proof import and we can't actually sign with the script key, then we can detect that and not factor it into the balance calls.

Closing as duplicate of: https://github.com/lightninglabs/taproot-assets/issues/287
Also the import call rn is considered to be a dev call.


Support for Beaglebone Black wireless?

Hi!
Any change there will be support for the Beaglebone Black wireless?
Thanks,
Fokko

thanks @FRVisser
@shaunmulligan so shall we order the exact same revision?

@floion , yeah sounds like a good idea

@FRVisser can you give an update? Last time we talked this started working. did you reflash to check again?

@floion ai, it got on my todo list 😔. Give me an hour or 2, then it will be on my done list 👍

So, I reflashed it and it connected to the dashboard immediately. I even didn't edit the uENv.txt file and only added the am335x-boneblack-wireless.dtb to resin-flashA/boot/ and after it was flashed to the /resin-rootA/balena/.../_data/ folder.
Capemgr is missing, but it could be because of the dtb file I used. Just pulled it of the internet form Debian I believe. Link:am335x-boneblack-wireless.dtb

Thanks. Why did you copy it to the data folder after flashing? that should not be needed IMO

The am335x-boneblack-wireless.dtb doesn't copy over. Only the uEnv.txt does.

Is it possible to just add /boot/am335x-boneblack-wireless.dtb to the image file?

Yeah, that's what I plan to do when I have some time to look at this. Had other things in the queue and hope to get this done this week sometime.

Hi, on what OS version were you able to have the device online in the dashboard?

Also, were you connecting to a hidden network?

Should be fixed by https://github.com/resin-os/resin-beaglebone/pull/142

2.12.5+rev1 with the above PR merged, has been deployed to our staging environment. Can you try it on your side and let us know if it works for you?
Thanks

@floion I currently have it on Resin OS 2.9.6+rev1 (dev). It showed up in the dashboard without problems. I didn't test a hidden network.
I'll test 2.12.5+rev1 today. Saw it passing by in the release notes.
Thanks

Hi @FRVisser did you have a chance to check this? Thanks

@floion Yes it works flawlessly, including the cape-manager.
Thanks!

Good. Closing this item


Added return statements to create-methods.

I think you forgot something. Without these modifications I could not get it to work. Now it works.

Ow, my bad. Left some 7.x syntax in.


marshal pointer to primitive typedefs

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


[image_picker 0.6.3]  crash in flutter 1.12h5

[image_picker 0.6.3]  crash on flutter 1.12h5 after build apk,but debug mode.  or version 0.6.2 work great,

Hi @saviourdog
can you please provide your flutter doctor -v
and your flutter run --verbose?
Also, to better address the issue, would be helpful
if you could post a reproducible minimal code sample
or the steps to reproduce the problem.
Thank you

[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57,
locale zh-Hans-CN)
• Flutter version 1.12.13+hotfix.5 at /Users/cxm/environment/flutter
• Framework revision 27321ebbad (4 weeks ago), 2019-12-10 18:15:01 -0800
• Engine revision 2994f7e1e6
• Dart version 2.7.0
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/cxm/environment/androidsdk
• Android NDK location not configured (optional; useful for native profiling
support)
• Platform android-29, build-tools 29.0.2
• ANDROID_HOME = /Users/cxm/environment/androidsdk
• Java binary at: /Applications/Android
Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3, Build version 11C29
• CocoaPods version 1.8.4
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 42.1.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build
1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.41.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.7.1
[✓] Connected device (1 available)
• MI 6 • 88eaff4c • android-arm64 • Android 9 (API 28)
• No issues found!
flutter run --verbose .  is  too long
my code is so sample
and I mean  app crash . when  launch,not in  run  image_picker dart code

Hi @saviourdog
if the output of flutter run --verbose is to long for a post
please attach it to the post in a .txt file
(voiding .docx and other less safe formats)
thank you

https://github.com/flutter/flutter/issues/25862

#25862

thanks a lot