Strange Error (Using Repl.it)

i keep getting this error:
[21:55:20]: LOG =-=-=-=-=-=-=- Loading command(s): 125 -=-=-=-=-=-=-=
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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: 1)
(node:84) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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: 2)
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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: 3)
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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: 4)
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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: 5)
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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: 6)
(node:84) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'includes' of undefined
at /home/runner/Discord-Bot/src/bot.js:14:34
at Array.forEach ()
at /home/runner/Discord-Bot/src/bot.js:13:13
(node:84) 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)
[21:55:20]: LOG =-=-=-=-=-=-=- Loading events(s): 27 -=-=-=-=-=-=-=
[21:55:20]: LOG Loading Event: debug
[21:55:20]: LOG Loading Event: emojiCreate
[21:55:20]: LOG Loading Event: emojiDelete
[21:55:20]: LOG Loading Event: emojiUpdate
[21:55:20]: LOG Loading Event: error
[21:55:20]: LOG Loading Event: guildBanAdd
[21:55:20]: LOG Loading Event: guildBanRemove
[21:55:20]: LOG Loading Event: guildDelete
[21:55:20]: LOG Loading Event: guildMemberAdd
[21:55:20]: LOG Loading Event: guildMemberRemove
[21:55:20]: LOG Loading Event: guildMemberUpdate
[21:55:20]: LOG Loading Event: guildUnavailable
[21:55:20]: LOG Loading Event: guildUpdate
[21:55:20]: LOG Loading Event: message
[21:55:20]: LOG Loading Event: messageDelete
[21:55:20]: LOG Loading Event: messageDeleteBulk
[21:55:20]: LOG Loading Event: messageReactionAdd
[21:55:20]: LOG Loading Event: messageReactionRemove
[21:55:20]: LOG Loading Event: messageUpdate
[21:55:20]: LOG Loading Event: ready
[21:55:20]: LOG Loading Event: roleCreate
[21:55:20]: LOG Loading Event: roleDelete
[21:55:20]: LOG Loading Event: roleUpdate
[21:55:20]: LOG Loading Event: shardDisconnect
[21:55:20]: LOG Loading Event: shardError
[21:55:20]: LOG Loading Event: voiceStateUpdate
[21:55:20]: LOG Loading Event: warn
CloseEvent {
target: WebSocket {
_events: [Object: null prototype] {},
_eventsCount: 0,
_maxListeners: undefined,
_binaryType: 'nodebuffer',
_closeCode: 4014,
_closeFrameReceived: true,
_closeFrameSent: true,
_closeMessage: 'Disallowed intent(s).',
_closeTimer: Timeout {
_idleTimeout: -1,
_idlePrev: null,
_idleNext: null,
_idleStart: 1320,
_onTimeout: null,
_timerArgs: undefined,
_repeat: null,
_destroyed: true,
[Symbol(refed)]: true,
[Symbol(kHasPrimitive)]: false,
[Symbol(asyncId)]: 165,
[Symbol(triggerId)]: 105
},
_extensions: {},
_protocol: '',
_readyState: 3,
_receiver: Receiver {
_writableState: [WritableState],
writable: false,
_events: [Object: null prototype] {},
_eventsCount: 0,
_maxListeners: undefined,
_binaryType: 'nodebuffer',
_extensions: {},
_isServer: false,
_maxPayload: 104857600,
_bufferedBytes: 0,
_buffers: [],
_compressed: false,
_payloadLength: 23,
_mask: undefined,
_fragmented: 0,
_masked: false,
_fin: true,
_opcode: 8,
_totalPayloadLength: 0,
_messageLength: 0,
_fragments: [],
_state: 0,
_loop: false,
[Symbol(kCapture)]: false,
[Symbol(websocket)]: [Circular]
},
_sender: Sender {
_extensions: {},
_socket: [TLSSocket],
_firstFragment: true,
_compress: false,
_bufferedBytes: 0,
_deflating: false,
_queue: []
},
_socket: TLSSocket {
_tlsOptions: [Object],
_secureEstablished: true,
_securePending: false,
_newSessionPending: false,
_controlReleased: true,
secureConnecting: false,
_SNICallback: null,
servername: 'gateway.discord.gg',
alpnProtocol: false,
authorized: true,
authorizationError: null,
encrypted: true,
_events: [Object: null prototype],
_eventsCount: 5,
connecting: false,
_hadError: false,
_parent: null,
_host: 'gateway.discord.gg',
_readableState: [ReadableState],
readable: false,
_maxListeners: undefined,
_writableState: [WritableState],
writable: false,
allowHalfOpen: false,
_sockname: null,
_pendingData: null,
_pendingEncoding: '',
server: undefined,
_server: null,
ssl: null,
_requestCert: true,
_rejectUnauthorized: true,
parser: null,
_httpMessage: null,
timeout: 0,
write: [Function: writeAfterFIN],
[Symbol(res)]: null,
[Symbol(verified)]: true,
[Symbol(pendingSession)]: null,
[Symbol(asyncId)]: 105,
[Symbol(kHandle)]: null,
[Symbol(kSetNoDelay)]: true,
[Symbol(lastWriteQueueSize)]: 0,
[Symbol(timeout)]: null,
[Symbol(kBuffer)]: null,
[Symbol(kBufferCb)]: null,
[Symbol(kBufferGen)]: null,
[Symbol(kCapture)]: false,
[Symbol(kBytesRead)]: 1085,
[Symbol(kBytesWritten)]: 589,
[Symbol(connect-options)]: [Object],
[Symbol(websocket)]: undefined
},
_bufferedAmount: 0,
_isServer: false,
_redirects: 0,
_url: 'wss://gateway.discord.gg/?v=6&encoding=json',
_req: null,
[Symbol(kCapture)]: false
},
type: 'close',
wasClean: true,
reason: 'Disallowed intent(s).',
code: 4014
}
0
[21:55:20]: ERROR Privileged intent provided is not enabled or whitelisted.
[21:55:20]: READY Mongoose connection successfully opened
(im also using repl.it to host the bot)

please update your config file and enable privileged intents.
Before making an issue, make sure you have the most up to date files and read the FAQ.

ok


Make use of Sentry for monitoring

[x] Remove usage of @vercel/analytics and @vercel/speed-insights
[ ] Follow https://docs.sentry.io/platforms/javascript/guides/nextjs/
[ ] Follow https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/
[ ] Install Vercel integration https://docs.sentry.io/product/integrations/deployment/vercel/
[ ] Add Sentry to the tech stack. Logo: https://sentry.io/branding/
[ ] After it's working, watch out for bundle size and ways to reduce it.
[ ] Update Privacy Policy with the help of ChatGPT.

Sentry is releasing v8 of SDK with proper support of OpenTelemetry. It is preferable to wait for v8 to have proper documentation before proceeding.


Columns on homepage - Chinese

The columns used in the About GBIF descriptor work reasonably well in English and presumably other latin-alphabet languages:

However, it doesn't work as well in Chinese:

This was confirmed by @Maofangisme who kindly provides the translations.

I've changed it to be a single column https://github.com/gbif/portal16/commit/2b828c11a970d0277335499cc1ea2db71d24f811
will be in next deploy


Tried to load 1.0.0-beta.3 with Webpack and it failed

Tried to load 1.0.0-beta.3 with Webpack and it fails.
Is this a known issue? Thanks in advance!
ui-router 0.3.1 works fine with webpack. I want to use Angular components instead of templates as explained in Hello World Solar System tutorial.
Here's the error I'm getting:
bundle.js:157Uncaught Error: [$injector:modulerr] Failed to instantiate module helloworld due to:
Error: [$injector:modulerr] Failed to instantiate module {"services":{"location":{},"locationConfig":{},"template":{}},"Category":{"0":"RESOLVE","1":"TRANSITION","2":"HOOK","3":"UIVIEW","4":"VIEWCONFIG","RESOLVE":0,"TRANSITION":1,"HOOK":2,"UIVIEW":3,"VIEWCONFIG":4},"trace":{"_enabled":{},"approximateDigests":0},"DefType":{"0":"PATH","1":"SEARCH","2":"CONFIG","PATH":0,"SEARCH":1,"CONFIG":2},"resolvePolicies":{"when":{"LAZY":"LAZY","EAGER":"EAGER"},"async":{"WAIT":"WAIT","NOWAIT":"NOWAIT","RXWAIT":"RXWAIT"}},"defaultResolvePolicy":{"when":"LAZY","async":"WAIT"},"NATIVE_INJECTOR_TOKEN":"Native Injector","RejectType":{"2":"SUPERSEDED","3":"ABORTED","4":"INVALID","5":"IGNORED","6":"ERROR","SUPERSEDED":2,"ABORTED":3,"INVALID":4,"IGNORED":5,"ERROR":6},"defaultTransOpts":{"location":true,"relative":null,"inherit":false,"notify":true,"reload":false,"custom":{},"source":"unknown"},"matcherConfig":{"_isCaseInsensitive":false,"_isStrictMode":true,"_defaultSquashPolicy":false},"default":"ui.router"} due to:
Error: [ng:areq] Argument 'module' is not a function, got Object

it is not known.
I've bundled using webpack and haven't had any problems.  Can you provide details?

Thanks @christopherthielen.
Here's my entry.js file:
var angular = require('angular'),
    angular_ui_router = require('angular-ui-router');

var myApp = angular.module('helloworld', [angular_ui_router]);

angular.module('helloworld').component('about2', {
  template:  '
Its the UI-Router
Hello Solar System app!
'
});

angular.module('helloworld').component('hello3', {
  template:  '
Ohoh
'
});

myApp.config(function($stateProvider) {
  var helloState = {
    name: 'hello',
    url: '/hello',
    //template: '
hello world!
'
    component: 'hello3'
  }

  var aboutState = {
    name: 'about',
    url: '/about',
    template: '
Its the UI-Router hello world app!
'
  }

  $stateProvider.state(helloState);
  $stateProvider.state(aboutState);
});

And here's my webpack.config.js:
var path = require('path');
var webpack = require('webpack');

module.exports = {
  entry: './entry.js',
  output: {
    path: __dirname,
    filename: 'bundle.js'
  },
  resolve: {
    modulesDirectories: ['./node_modules'],
    modules: path.join(__dirname, "node_modules")
  },
  loaders: [
  ]
}

index.html

    Hello
    About

    <ui-view></ui-view>
    <about2></about2>

Compiled web app attached.
webapp.zip
Thanks again!

https://ui-router.github.io/guide/ng1/migrate-to-1_0#commonjs-module-name

That did the trick! Thanks!

You may also use import uiRouter from 'angular-ui-router';


Card keeps refreshing/glitching

For some reason the card keeps refreshing or glitching. You could see in the gif that I could barely use the input_select entity to choose a value because the glitch will close the dropdown before I could.

Editor is not yet supported yet

I apologize, I should have been more clear. The glitching issue happens in the normal dashboard view. I only took the screen recording in the editor view so you could see the code and the issue at the same time.


Add actual date field in Handover Hanging Garden

Add actual date field as done in Mechanical Completion Hanging Garden

Nice to have.


[Bug]: Coolify healthcheck fails when using "scratch" base Docker image

Description
It appears something with coolify's healthcheck's (perhaps a dependency in the built image?) makes it so using a scratch docker base image will always fail health checks whereas using something like alpine succeeds.
Minimal Reproduction (if possible, example repository)
The following docker file pointing to a simple hello-world golang server fails health checks:
# syntax=docker/dockerfile:1
FROM golang:1.22 as build
WORKDIR /app
COPY . .
RUN go build -o /server .

FROM scratch
COPY --from=build /server /server
EXPOSE 3000
CMD ["/server"]

Changing scratch to alpine:latest causes health checks to pass consistently.
Exception or Error
Health checks fail tho the actual health check doesn't output anything useful to troubleshoot this issue. Had to figure it out with trial and error.
Version
v4.0.0-beta.258

If you use a Dockerfile (with or without git) or docker image-based deployments, the health check command is basically an exit 0 because of the lack of curl or wget commands.
But it looks like the scratch does not like that as well.
As it is hard to determine the final image source, I suggest to simply turn off health checks (on the UI) or use alpine image instead.

I ended up just moving to using Alpine 👍


CI is broken

The organization transfer may have broken Bors and/or Travis CI.

The solution was:

Uninstall and reinstall the Bors GitHub App
Uninstall and reinstall the Travis-CI.com GitHub App


[BUG]Fix Java Aggreagte Prport

java - aggregate-reports has been failing for a while. It needs to be fixed.
We should also retire the UI release pipeline and replace it with a simple yaml step like we do in the aggreagte reports pipeline of other languages.

@srnagar assigning to you as you've been looking into this already.

Resolving as we have aggregate reports running successfully.
Latest run - https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1746193&view=results


 Please do check it out. Thanks!

Hey @anubhav2102 ,
There are some issues in the preview as you can see.
Form is not taking input.


Expose a reference to jsThis within Dart components

Sometimes it's helpful, especially in testing, to get a reference to the JavaScript React component that backs the Dart component instance.
These changes store a reference to that instance on the Dart component and expose it as a getter for external use.

@hleumas @trentgrover-wf

+1

+1

@trentgrover-wf I think this is ready for merge.


Generic replica connectivity

This pull request aims to make the replica connectivity more generic. The idea is to allow other projects to re-use the connectivity while having the liberty to supply their own ordering protocol.  For example, one could want to implement another protocol that isn't MinBFT but want to use the connectivity implemented in this project.
To this end, the functions incomingMessageHander and replyCollector were replaced by interfaces ProtocolHandler and ReplyCollector, respectivly. Moreover, there are also the following new functions:
func GetReplica(id uint32, configer api.Configer, stack Stack, handler api.ProtocolHandler, logger *logging.Logger) (*Replica, error)
func GetClient(id uint32, n, f uint32, stack Stack, collector ReplyCollector) (Client, error)
func GetMinBFTHandler(id uint32, log messagelog.MessageLog, config api.Configer, stack Stack, logger *logging.Logger) api.ProtocolHandler
func GetLogger(id uint32, opts ...Option) *logging.Logger
func GetMinBFTCollector(f uint32) ReplyCollector

@jcs47 First, a few general remarks. Please follow the convention to begin a commit message with a short commit title. (This reminds me of #28...) You probably wanted to submit a draft pull request.

I was also not sure how that would agree with the scope of the project.

@ynamiki @Naoya-Horiguchi What do you think?

@jcs47 FYI, to maintain a coherent git blame-able history, we do not address review comments by adding more follow-up commits. I will soon prepare a proper contribution guideline (#28). The best we have so far is the discussion in #12.

I would like to focus on the core protocol part; we have many features to be implemented for practical use in the part. And I am afraid that it is too early to generalize based on current implementation, which is under the development. Sorry for my late response.

I was also not sure how that would agree with the scope of the project.

@ynamiki @Naoya-Horiguchi What do you think?

I would like to focus on the core protocol part; we have many features to be implemented for practical use in the part. And I am afraid that it is too early to generalize based on current implementation, which is under the development.

Should we close this PR?

@Naoya-Horiguchi I would like to hear your comments.

@Naoya-Horiguchi I would like to hear your comments.

I'm OK to close this PR now because base code is updated (not because the original proposal is rejected). This feature is not in the roadmap, but that's not a good reason to reject it. So we can simply wait that new version of PR is submitted.

@jcs47 We close this PR as outdated. Please submit an updated version, if you decide to.


view on phones

need to test with phones when we have server version running, may need to change from     width: 480px under main and navigation in css.

fixed


Font Value is not accepted

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it

Why do I need to fill in a specific font type, bold light italic or regular
Environment
Zed: v0.119.11 (Zed Preview)
OS: macOS 14.2.1
Memory: 32 GiB
Architecture: aarch64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Hey, you don't need to fill in a specific font type (though I see why it is confusing);  "JetBrains Mono"/ "JetBrainsMono" should work just fine. The suggestions are mostly there to avoid typos (e.g. zed-mono instead of Zed Mono).
I will open up a PR to include family names in the completion list as well.

The current configuration of "JetBrainsMono Nerd Font Mono" is working fine, but this warning is confusing


Handle custom emoji with same name as unicode emoji better

At present, we do not effectively handle situation where custom emoji have the same name as unicode emoji (see https://github.com/zulip/zulip/issues/18135#issuecomment-1380460962).
To address this, we should:

[ ] Prevent non-administrators from creating custom emoji with the same name as unicode emoji. We can display an error similar to what's shown when a user attempts to use a name that's already assigned to a custom emoji:

Text:

Failed: There is a default emoji with this name. Only administrators can override default emoji.

[ ] When an administrator attempts to create a custom emoji with the same name as unicode emoji, pop up a confirmation modal:

Override default emoji?
There is a default emoji with this name. Do you want to override it with a custom emoji? The name :emoji name: will no longer work to access the default emoji.
[Cancel] [Confirm]

[ ] [this can be done as a follow-up] When a custom emoji name overrides a unicode emoji, show a warning icon (fa-exclamation-triangle looks to be used elsewhere) with a tooltip explaining the situation:

This custom emoji overrides a default emoji with the same name.

[ ] When a custom emoji has the same name as a unicode emoji, only the custom emoji should be shown under that name in all emoji typeaheads; see also #18135.

CZO discussion

@N-Shar-ma Are you interesting in picking this one up?

[ ]  [this can be done as a follow-up] When a custom emoji name overrides a unicode emoji, show a warning icon (fa-exclamation-triangle looks to be used elsewhere) with a tooltip explaining the situation:

This custom emoji overrides a default emoji with the same name.

@alya I'm not sure what you mean here... Do you mean we should display the warning icon in the typeahead menu alongside any options for realm emoji that override a default one?

Ah, no, I meant in the table of custom emoji, after an admin has uploaded a custom emoji that has overridden a unicode emoji.

@zulipbot claim


Wrong temperature propagation in transformer ambient

Reported by christiankral on 5 Oct 2014 17:37 UTC
In case no connectors are used, the temperature T2 of the secondary winding is not propagated to the thermal connector in Modelica.Electrical.Machines.Thermal.ThermalAmbientTransformer. Instead temperature T1 of the primary winding is propagated.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1579

Comment by christiankral on 5 Oct 2014 17:50 UTC
Bug fixed in 1b24dc9bd227d93c58f542da93876cde5cb4ac15, 31946ce3824851b13b18d5ff6a6998bf9841251a for MSL 3.2.1 (branches) and trunk.


docs: Update routing.mdx

Description (required)
Changes

Fix typo - /en-v1/info -> /en/v1/info
Fix typo - /fr-v2/info -> /fr/v2/info

Related issues & labels (optional)
N/A

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳.

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/routing.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Cloudfront batch invalidations.

I’m currently using an older version that still uses Thumbor.  I’m getting ready to switch to the newest version, but I wanted to know if the newer version will fix one of my major problems.  I have some javascript on my website that calculates the exact image dimensions needed for each image on page load and then builds the url for the image (for example:  distrobution.cloudfront.net/100x100/path/image.jpg).  It works great and is super SEO friendly.  However, the one drawback is that when image.jpg is changed you cannot invalidate all the versions of it in cloudfront without invalidating everything in the bucket (not SEO friendly at all).  Invalidating /path/image.jpg doesn’t do any good because the images being loaded are all /widthxheight/path/image.jpg and there’s no way to know what dimensions for a particular image exist.  Cloudfront doesn’t allow wildcards at the beginning of a path, so...
All that to say, my question is:
Does the newest version build the path differently, or can the way the path is built be changed easily in the newest version?  I think something like distrobution.cloudfront.net/path/image_name/100x100/image.jpg would work best, but even if I could narrow it down to only having to invalidate the folder that the image is in (distrobution.cloudfront.net/path/100x100/image.jpg) would be drastically better.  I’m open to other suggestions on possible solutions.

You can override the parseRequestType callback to handle the same format as Thumbor.
Otherwise you have to encode your the new JSON format (like the following) to base64:
{
    "key": "image.jpg"
    "edits": {
        "resize": {
            "width": 100,
            "height": 100,
        }
    }
}

The base64 encoded solution has the advantage of hiding the parameters to the end user.

Ok, but do either of those options impact my issue?  If it's the exact same format as Thumbor, the issue still exist.  If I can change it to be different than Thumbors format, then that may help.  I'm assuming the base64 encoded option doesn't offer any help at all unless invalidating those images would work differently in cloudfront?

Hey @jasonobrown: CloudFront invalidation has been something we've been looking into for some time to build into the solution as you mentioned above. Since invalidation doesn't apply to all use cases and can get expensive for customers who don't necessarily need it, we decided to keep it on the backlog for a future release.
One potential solution that we've identified for this is creating a small Lambda function that can be optionally deployed via the template that's triggered when the version of the object is changed. This would then trigger invalidations via the CloudFront API so that the newest version is delivered and cached. We'd love to hear any thoughts/feedback on this though to help deliver a better experience!

@hayesry the potential solution you described sounds great, but I have no idea how it would work.  Are the images cached in cloudfront somehow connected to the base image in s3?  Or, would the lambda function try to run an invalidation for all possible matches of that image?
For example:
bucket/folder/image.jpg exist in s3
bucket/200x200/folder/image.jpg is what exist in cloudfront
Does versioning somehow connect those 2 images?

@jasonobrown , sorry for the delayed response.
For your last question, I would say you can see "Popular objects" on Amazon CloudFront console.

@hayesry  - We are having an issue with S3 consistency when updating an image, cloudfront sometimes still uses the old version when retrieving from the bucket.
{"bucket":"xxxxxxxx","key":"images/catimg/172.jpg","edits":{"resize":{"width":320,"height":320,"fit":"inside","withoutEnlargement":true}},"version":"ff22345c3a39e0b46833b427210f67ad4fcaf14ca"}
We currently pass the "Version" as a parameter which is a hash (sha1) of the original file contents. Is it possible to change the image request to only cache it for a short time if the hash of the s3 file doesn't match the hash in the parameters? I could potentially write it but wondering if it's something you'd want in the main code or we should just update for us?

@jasonobrown we have updated our solution, and I believe your issue has been fixed. If you still see the issue with the latest version (v6.0.0), please feel free to reopen the issue.
You can refer to the recent changes here.


Add documentation for custom dimensions

This just adds documentation on how to take advantage of the new hit types added by @djw27 in their PR.
@ttmarek forked off of @djw27's project. Not sure if this is the what you were expecting. I can change the base of this fork if needed.

@bzswords @djw27 thanks for all your hard work on this guys! :100:
It's much appreciated :slightly_smiling_face:.
I'll let you know once these changes are published to npm.

@bzswords @ttmarek thanks for getting this merged!

@ttmarek any update on when this will be published?

Hey @bzswords. Sorry, some clients are in town this week so I haven't had any open evenings to work on the next release. It is so so close though. I wrote a checklist of what's left in this issue here:
https://github.com/rangle/redux-beacon/issues/180
In the meantime I would opt for simple copy-pasting the entire react-native-google-analytics target into your repo. In the next release this will be published separately under @redux-beacon/react-native-google-analytics as described here
Any help would be greatly appreciated if you can spare a moment or two. Adding typescript definitions for each event-helper would be fairly straightforward and a huge win.

@ttmarek thank you for the response, i totally understand. i may actually have time this weekend to contribute. ill comment on that checklist if so.


CWE-1075 Unconditional Control Flow Transfer Outside of Switch Block

Usage Name
Control transferred outside switch statement
Reference
https://cwe.mitre.org/data/definitions/1075
Detection Patterns

8.2.122 ASCQM Ban Control Flow Transfer

To gather data related to this pass, run the following command:
/static/driver/main.py --init --ast_pass='goto-out-of-switch'


trying to use mran snapshot

instead of fixed versions for all packages.
take II

Coverage remained the same at 86.421% when pulling 324d43b070742d950361d628d09caa2488bb8570 on mhines-usgs:master into 4c3c22915cdd9e76b750c5f0d5baecd5232001ce on USGS-R:master.


feat(dev): add getRouterName()

This PR introduces getRouterName() function in hono/dev helper and deprecates app.routerName().
I think app.routerName() is not used by many users, and if users use it, the purpose is debugging in most cases. So, we can place it in hono/dev is better.
Usage:
import { Hono } from 'hono'
import { getRouterName } from 'hono/dev'

const app = new Hono()

//...

console.log(getRouterName(app))

Author should do the followings, if applicable

[ ] Add tests
[ ] Run tests
[ ] yarn denoify to generate files for Deno

Hi @usualoma !
How do you think this PR?

I think you are right 👍

@usualoma
Thanks! I'll include this feature in the next minor version.


Remove: vim.vim version 8.2.2589

## .\Tools\YamlCreate.ps1 -PackageIdentifier vim.vim -Mode 5 -SkipPRCheck -PackageVersion 8.2.2589

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

/azp run


Allow using auth filters on creation

Following up on conversation here: https://github.com/TriPSs/nestjs-query/issues/320
When getting started with nestjs-query I found it un-intuitive that my authorizers would be executed during the creation flow, but then not evaluated in any way to prevent creation of resources that didn't match my auth criteria.
This PR allows users to opt in to having their authorization filters used when hitting creation end points.
Open Questions:

Where should the opt-in configuration be done?
-- My feeling is in the resolver options
Where should the access be evaluated?
-- The original implementation I had done implemented this in the typeorm-query-service however, given that we are evaluating the filters in JS we could move it to the resolver. My issue with that is I don't think this works very well with the assembler paradigm if you're CreateDTO is significantly different from core DTO/entity

@TriPSs I'll start on test coverage once we're happy with the overall direction.

Just now saw the open questions, sorry.
Open Questions:

Where should the opt-in configuration be done?
-- My feeling is in the resolver options

Would indeed say the resolver options (better naming is welcome):
{
  resolvers: [{
    create: {
      one: { validateWithAuthFilter: true },
      many: { validateWithAuthFilter: true },

      validateWithAuthFilter: true // Enable for both
    }
  }]
}

Where should the access be evaluated?
-- The original implementation I had done implemented this in the typeorm-query-service however, given that we are evaluating the filters in JS we could move it to the resolver. My issue with that is I don't think this works very well with the assembler paradigm if you're CreateDTO is significantly different from core DTO/entity

Agree, where it is now is indeed the best.


Have to make dynamic font unique to change size

3.2.alpha
Ubuntu 18.04
When a .ttf-file is already loaded, loading it again from another location results in not being able to change the size. Changing the size in one of those locations makes the font become invisible. Only when the original size is put into the 'size'-field inside the font-settings, will the font become visible again. Loading the font again, and making it unique solves this problem.

load a .ttf-file as a dynamic-font inside a label.
load the same .ttf-file in another label.
try changing the size of the dynamic-font

This may be related to some DynamicFont font caching issues which I've noticed but haven't gotten to writing up yet. Here's a quick brain dump...
Existing DynamicFont font cache issues
From memory, the issues are:

It's possible for a (cached?) font size to be set before the data is loaded which means the following code won't "reload" the cache because it thinks the font size is already cached:
https://github.com/godotengine/godot/blob/1fed266bf5452b30376db62495f4985f6975f2c1/scene/resources/dynamic_font.cpp#L698-L705

I seem to recall that this "manual"/direct size-setting during initialization (via https://github.com/godotengine/godot/commit/c399c41dfb065efab8d3b168ef0a1c5a5cf609d5) is what causes the issue:
https://github.com/godotengine/godot/blob/1fed266bf5452b30376db62495f4985f6975f2c1/scene/resources/dynamic_font.cpp#L1022-L1026
There was an alternate patch which used the set_size() method AFAICT correctly (and also handled https://github.com/godotengine/godot/commit/f41dddf069ab3747e38f2236db17a46ea55200b0 correctly): #21165
From memory, the initialization issue fell out of the following change to calculation/handling of CacheID (which may be should have been setting size to -1 or 0 anyway?): https://github.com/godotengine/godot/commit/8daf5491ab6bc60434e4d952830bdd258eaf0e53#diff-17a3b965179a9e0883c412a8af15cff3L59
(Which--again from memory--worked because the previous comparison operator implementation didn't meet the transitivity requirement of a comparison sort i.e. it was (incorrectly) possible for items a & b to return true for both these tests:  a < b & b < a.)

Inconsistent use of calls to: emit_changed(), _change_notify() & _reload_cache().

Possible solution
At a minimum it may be sufficient to replace these lines:
https://github.com/godotengine/godot/blob/1fed266bf5452b30376db62495f4985f6975f2c1/scene/resources/dynamic_font.cpp#L1025-L1026
with a call to set_size(16)--I seem to recall that because the DynamicFont isn't valid when first created without data the incorrect cache handling issue won't occur after this change. (Or maybe it needs to be set_size(0).)
Additional context
Hopefully this is a useful starting point for someone investigating this issue. Unfortunately my list of issues to write up "comprehensively" or "verbosely" :D (as I'm wont to do)--and ideally create a PR for--isn't getting shorter so I'm doing a brain-dump here in the hopes that imperfect & done is better than someone else having to repeat my troubleshooting efforts on the font caching side of things. :)

i can't reproduce this issue on windows 10

Can't reproduce either, please comment if you can still reproduce in Godot 3.2.3 or newer. Also include a reproduction project.


Cannot run storybook inside sandpack-react

I cloned this repository and tried to see demos inside sandpack-react directory as README.md mentioned.
I have already executed yarn install command in project root but yarn storybook command got error output as below:
ERROR in ./src/contexts/sandpackContext.tsx
Module not found: Error: Can't resolve '@codesandbox/sandpack-client' in '/Users/joshoy/examples/sandpack/sandpack-react/src/contexts'
 @ ./src/contexts/sandpackContext.tsx 76:0-83 122:17-36 187:23-37
 @ ./src/components/CodeEditor/CodeEditor.stories.tsx
 @ ./src sync ^\.(?:(?:^|\/|(?:(?:(?!(?:^|\/)\.).)*?)\/)(?!\.)(?=.)[^/]*?\.stories\.(js|jsx|ts|tsx))$
 @ ./.storybook/generated-stories-entry.js
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/storybook-init-framework-entry.js<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/preview.js-generated-config-entry.js ./.storybook/generated-stories-entry.js ../node_modules/webpack-hot-middleware/client.js?reload=true&quiet=false&noInfo=undefined

ERROR in ./src/utils/sandpackUtils.ts
Module not found: Error: Can't resolve '@codesandbox/sandpack-client' in '/Users/joshoy/examples/sandpack/sandpack-react/src/utils'
 @ ./src/utils/sandpackUtils.ts 8:0-70 63:14-36
 @ ./src/contexts/sandpackContext.tsx
 @ ./src/components/CodeEditor/CodeEditor.stories.tsx
 @ ./src sync ^\.(?:(?:^|\/|(?:(?:(?!(?:^|\/)\.).)*?)\/)(?!\.)(?=.)[^/]*?\.stories\.(js|jsx|ts|tsx))$
 @ ./.storybook/generated-stories-entry.js
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/storybook-init-framework-entry.js<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/preview.js-generated-config-entry.js ./.storybook/generated-stories-entry.js ../node_modules/webpack-hot-middleware/client.js?reload=true&quiet=false&noInfo=undefined

Any idea on what problem it is?

Yes, you have to run yarn build in the root to have the sandpack-client package compiled to be used inside the sandpack-react one. I'm updating the Readme file, as it doesn't say this.

Works for me, thanks! I'll close this issue.


Removed Fixnum reference

Since the Ruby-2.4.0 release, Fixnum and Bignum are combined into Integer class. This fix is to remove all Fixnum reference.
I opened an issue https://github.com/stympy/faker/issues/1171 for this PR.

Just took a look at the travis log and noticed that we are testing different versions of Ruby. Lemme know how we should proceed. Thanks 👍


Submit GPG key for OpenTofu Registry

Is your feature request related to a problem? Please describe.
Yes, when using OpenTofu.
Describe the solution you'd like

When using the terraform provider via tofu, you get the following warning when it's installed (see screenshot as well):

Installed tailscale/tailscale v0.13.13. Signature validation was skipped due to the registry not containing GPG keys for this provider

It would be great to see this get addressed. To do so, please do the following: "Submit new Provider Signing Key" from opentofu/registry's Issues page. Here is a recent example.

Additional context

See a related issue here and corresponding registry PR

Thank you for flagging this! Have opened https://github.com/opentofu/registry/issues/729 in the registry,

The change has been merged in the registry and it is looking like signature validation is now happening:

Going to be closing this as a result.


Create a "not contains" query modifier.

when querying a model, you can do the following.
where: {
  name: {
    'not contains': 'James'
  }
}

or
where: {
  name: {
    '!contains': 'James'
  }
}

This is a feature request by the community.

Hi, i'm trying to use this method, but it doesn't work.
name:{
not: {
contains: 'Foo'
}
}


EPSO fails on the dashboard due to unloadable imports

See http://dashboard.obofoundry.org/dashboard/index.html
Try:
robot merge -I http://purl.obolibrary.org/obo/epso.owl -o epso_merged.owl

@matentzn I'll take care of that thank you !

@matentzn The newest merged file is located here : https://github.com/phwegner/Epilepsyontology/blob/main/release/EPSO_merged.owl

@phwegner thanks! It is not important where the new merge is - the thing that matters is that the PURL works:
http://purl.obolibrary.org/obo/epso.owl
which you are responsible to configure here:
https://github.com/OBOFoundry/purl.obolibrary.org/blob/master/config/epso.yml
You can test if it works by running:
robot merge -I http://purl.obolibrary.org/obo/epso.owl -o epso_merged.owl

Okay I replaced the epos.owl with the merged one and now the purl points to the merged file which makes much more sense. Since the other one depends on the local folder structure.
I created the respective pull requests.


Migrates to CircleCI 2.0

I think I got this right, but let's see.

Okay, I think because the master branch is still using CircleCI 1.0, the status checks for PRs aren't enabled? Anyway, this is passing on Circle.
A cold build takes about 9 minutes, and a hot build (with cached gems, pods, and intermediate compiler artefacts) takes less than 3. Pretty sweet!


AttributeError: 'GatewayConnection' object has no attribute '_sent_notifications'

python 2.7.3
apns.gateway_server.send_notification_multiple(frame)
File "/home/gaoenbo/py27/local/lib/python2.7/site-packages/apns.py", line 559, in send_notification_multiple
self._sent_notifications += frame.get_notifications(self)
AttributeError: 'GatewayConnection' object has no attribute '_sent_notifications'

# Send multiple notifications in a single transmission
frame = Frame()
identifier = 1
expiry = time.time()+3600
priority = 10
frame.add_item('b5bb9d8014a0f9b1d61e21e796d78dccdf1352f23cd32812f4850b87', payload, identifier, expiry, priority)
apns.gateway_server.send_notification_multiple(frame)

+1

+1

This is due to the latest release not being available on pypi, perhaps install from github (until https://github.com/djacobs/PyAPNs/issues/163 is resolved) the issue has been resolved here https://github.com/djacobs/PyAPNs/commit/1842c40d8b9586643097c9ad142c5e447408d14f

If you want to use enhance mode, please refer to document and specify enhanced=True,
apns_enhanced = APNs(use_sandbox=True, cert_file='apns.pem', enhanced=True)


Upgrade JDK version to 17

Context
We want mantis to be on Java17. Unfortunately, this would require several gradle changes. So, the goal of this task is to make these changes and ensure the builds succeed.

I think one of the reasons it fails for higher versions of JDK is the DatatypeConverter package. As this answer on StackOverflow explains:
The module javax.xml.bind has been put outside the jdk as of Java 9.

Also, as part of the donation of Java EE to the Eclipse Foundation, its new name is Jakarta EE and accordingly the namespace was renamed to Jakarta. So you have to make the following modifications :

@Adithya2907 would you like to pick this up?

Yeah, sure I'd like to pick this up.

Hey @Adithya2907
Are you working on this?

Hi @cypherean , yes, I'm working on this. I've been trying to understand the codebase till now, and now I'll be migrating it to JDK 17. I'm seeing which all packages and libraries are failing due to the migration first


Some stuff

Back to mapping random things.

@Shnupbups Can you update to fix the merge conflicts?


Visualizer text not rendered

I am running digits in a docker container and all seem to work fine. However when I visualize a network the graph is correctly drawn but the text is displayed as squares. I assume I am missing a font. Any idea what should be installed?
Note: graphviz was installed via conda packages.

I've never run into that problem. I've used conda before but don't remember checking that particular feature.
A missing font is a good guess. Since you're in a docker container maybe it's possible graphviz expects an X server to be running or something? @flx42 is that plausible?


Explicitly free MPI shared window

In some situations, mpi4py does not free the shared window at destruction time.  We should do that explicitly in our destructor.

This was fixed some time ago.


Isolate the JS package files in a subdirectory

Overview
Isolate the pywwt NPM/JS package in a frontend subdirectory, rather than having it be intermingled somewhat confusingly with the Python package.
As part of that isolation, separate the build step, to generate a big bundled JS file with webpack, and what I'm calling a pywwt-export step, that copies the JS artifacts to their needed locations in the Python package tree. I think it would be more correct to have this "export" step be implemented fully on the Python package side, but given the existing infrastructure it's a bit simpler to implement it as an NPM script.
Checklist

Will be tested in CI
Documentation forthcoming after new build setup is worked out.

Update: ran into CI problems, initially triggered by API breakage in traitlets > 5, but also induced by trying to build on Pytho 3.8. I think that fundamentally, Tox is not a good testing tool for pywwt because it requires venvs for test environment setup, while we need to use Qt+OpenGL, which induce lots of platform-dependent gnarliness. So I am working towards using a conda-based test setup rather than Tox.

@astrofrog I'm seeing a bunch of failures in the pywwt image tests as I try to get this working (again) on Windows and Darwin. I've at least got the CI uploading the output test imagery as build artifacts, so that it's a bit easier to see what's going on. Do you have any memories of issues and/or debugging to get this stuff working? You had disabled macOS tests altogether, but Windows was working relatively recently ...

That took a while.


box tip aggregate

An alternative to #1839 for boxplot tips; this one shows the # of items in each quartile (and # of outliers), using numbers and math. symbols (as opposed to English language). We don't show empty quartiles and redundant information (i.e. when the # of values is small).
I don't know what users of box plots expect from a tip. The visual summary is one thing, but making it into “sentences” is quite challenging, given the complex definition of the elements.
For example, we can't really show percentages, as they are misleading when the data is “discrete” (few values). For instance, if you have 5 values, the proportion that are lower to the median is 2 out of 5 (40%), and the proportion that are lower or equal to the median is 3 out of 5 (60%). There is never a 50%. So, recount everything wrt the quartiles, and don't display %. But of course now it makes things much more difficult to grasp, because these statistics are coming from percentages, and it's hard to understand why the tip would show “made up” values of 4.25, 7, 10.75 as interesting edges to summarize an array of integers.

This is my gripe with the whole concept of a box plot, maybe, more than with the tip :)

I think I would find it more readable if the statistics were labeled with their names, min, max, median, q1, q3 (maybe count or n also?). I guess there is the question of whether we use the simple definitions or the more nuanced ones the box plot uses. I would look for precedence at other tools that provide one dimensional statistical summaries such as DuckDB and R.

Maybe we don't need to reinforce the difference between counts (123#) and values? Also, we could use the p25 notation, for consistency.
If we could somehow format as a table (or correctly aligned columns), I'd see something like this (made up numbers):
count
    n: 31,256 (+ 10 n/a)
quartiles
  p25:  7,530 < 23.1
  p50: 16,230 < 45
  p75: 23,530 < 56.1
outliers
  low:    156 < 12.3
 high:    347 > 89.1

(12 numbers, that's quite a lot.)
For small n we wouldn't always display all the stuff, but rather:
     n: 3
median: 42

A simpler version, doesn't show counts nor tries to explain what the 99.3% range is:

I like this general direction! I think “Min: 1.7” and “Max: 2.08” would be clearer than the [1.7 – 2.08] interval notation.
It seems OK to me if the tooltip ignores the concept of outliers… even though “how is outlier defined?” is always my first question when I see a box plot with them!!

The point is, they're not min and max — instead, they're “low” and “high” values that capture 99.7% of the (estimated) underlying normal distribution, thus defining “outliers”. If you know how a box plot is defined, the bracket notation is fine; if you don't, you’ll need a key, and I don't think that a proper definition can be conveyed in a concise tip. (Words such as “low” and “high” are not self-sufficient.)

I think low and high is better (more readable) than the bracket notation, and agree it avoids the confusion that min and max would introduce due to outliers. In either case you’d have to look up the details of what the number means, so better to give it a name at least so people know what to look for and how to refer to the presented values.

I would also prefer if the names were bolded like we do with channels, i.e., “p25 42” instead of “p25: 42”.

I can't find how to add low, high, etc as independent channels on the tip mark.
I think there's a bug somewhere since, when I do
group({low: loqr1, x: "p50"}, {x, y, z: y, channels: {low: x}, ...options}))

the low channel is generated with the correct number of values (as many as there are groups), but all its values are undefined.


Edit a How Many card/Story 2

Value Proposition
As a User
I want to save how many exercises i like to do
So that I do not have to remember how many exercises i need to do.
Description

Acceptance Criteria
Homescreen

[ ] By clicking on the How Many button the user is redirected to the form page.
[ ] The default amount of the exercises displayed is 0x.

Formpage Screen

[ ] By clicking on the back button the user is redirected to the home screen without saving the how many amount.
[ ] The default and minimum value of the how many amount is "0x", the maximum value is "∞".
[ ] By clicking on the "+" and "-" buttons the user can increase or decrease the amount of the exercise.
[ ] By clicking on the checkmark-button the how many amount is saved and the user is redirected to the home screen. The home screen then shows the how many amount the user saved.

Tasks
(for developers team)

[ ] create a button on the homescreen which brings you on a formpage
[ ] the button need to show the amount we set on the formpage
[ ] create a paragraph which describes to click the button
[ ] create a counter which you can set
[ ] create a link in form of an icon which brings you back to the homepage
[ ] create a submit button which confirms your setted amount

Complexity
L-XL

Hey @PhilipZi
Your User Story looks great!
Just small corrections:
In AC Frompage i would just write: "By clicking on the back button the user is redirected to the home screen without saving"
Why is the max value ∞ ? Maybe there should be a more reasonable max value.
I would change the "how many amount" in your texts in AC just to "amount". I'ts a bit confusing to read.

thank you vlady updated your hints

Hey Philip,
looks good to me. 👍
cheers
R.


refactor: deploy the training-operator with kubernetes resources

This commit refactors the way the training-operator is deployed, as instead of using a sidecar container that runs the workload, we are now applying the Deployment and all the Kubernetes resources required by the training-operator controller to be able to mange training resources.
We are introducing this change in preparation for the upcoming 1.8 version, as it introduces the hard dependency on a Kubernetes Secret mounted in a volume for the training-operator workload to start. For more details please refer to canonical/training-operator#159.

@orfeas-k COS integration tests were failing, this PR may need a second review.


Detect when a body has multiple parameters and when one of them is blob

This is a particular condition we don't support and should detect
Otherwise we end up with this exception

✅ No public API change.


Update schema and RLS policies in Readme

Hey @zernonia I think the schema has changed a bit and also the RLS policies. I ad a bunch of issues trying to run my fork. Could you update with a supabase db dump maybe? https://supabase.com/docs/reference/cli/supabase-db-dump

Hey @thorwebdev ! Yes for sure!

dumped latest sql here 😁
https://github.com/zernonia/madewithsupabase/commit/1d1e38d82a897adfd3b5d87c29ddbd5bebf5eda5


Diabetes Database Analysis

This section is related to all steps to analyze Pima Indians Diabetes dataset to understand what factors influence the prediction.

first step : Transfer data into SQLite databases


Vector2 math bit corruption

Godot version
3.3.2.stable.official
System information
Windows 10
Issue description
Doing maths with _Vector2_s returns slightly corrupted answers 👀, though the x and y components should be floats and not lose any precision, leading me to believe that this is not due to floating point errors.
This might be due to unnecessary maths going on in the background, perhaps deteriating the precision of the numbers.
The example in the steps to reproduce better illustrate the problem.
This bug might lead to rotated _Vector2_s having a value larger than their original length!
Steps to reproduce
func _ready() -> void:
	var anumber : float = -0.1234567
	var actual_answer : float = -123.4567
	print("multiply_then_extract_x " + var2bytes((Vector2(anumber, 0) * 1000).x).hex_encode())
	print("extract_x_then_multiply " + var2bytes(Vector2(anumber, 0).x * 1000).hex_encode())
	print("multiply_original_float " + var2bytes(anumber * 1000).hex_encode())
	print("actual_answer           " + var2bytes(actual_answer).hex_encode())

Minimal reproduction project
Run the above code in Godot.

The cause is the same as in #50022: Vector2.x, Vector2.y are float (32-bit) on the C++ side but in GDScript float is 64-bit so there are conversions happening all over in different places.
This C++ snippet shows what conversions lead to the results in your example:
#include <iostream>

void print_as_hex(float f) {
	std::cout << std::hex << *((uint32_t*)&f) << std::endl;
}
void print_as_hex(double d) {
	std::cout << std::hex << *((uint64_t*)&d) << std::endl;
}

int main() {
	double anumber = -0.1234567;
	double actual_answer = -123.4567;

	print_as_hex((float)((float)anumber * (double)1000));
	print_as_hex((double)(float)anumber * (double)1000);
	print_as_hex(anumber * (double)1000);
	print_as_hex(actual_answer);

	return 0;
}

Output:
c2f6e9d5
c05edd3a99800000
c05edd3a92a30553
c05edd3a92a30553

Is there a way to force _Vector2_s to use double precision?

Is there a way to force _Vector2_s to use double precision?

See https://github.com/godotengine/godot/issues/288.

Is there a way to force _Vector2_s to use double precision?

Not yet, see #288.

If double precision and even possible more (as per https://github.com/godotengine/godot-proposals/issues/892) were implemented it would make Godot usable for professional simulations, as many of the Vector2 problems (such as this issue, which also happens when rotating _Vector2_s, and transform not multiplying correctly) would no longer be present. It would make Godot suitable for a range of professional applications, and I believe this would be high beneficial for the project.

I believe that it is very imporant that a high-level of accuracy through precision is available to those users who need and, and having consistency by not converting between 32 and 64-bit floats, but rather using 64-bit all along. This would likely take a lot of work but this would be one of the features that would make Godot evolve into a very professional product.

@SlitheringMule I agree with you. Although I think having 32-bit floats as an option is still useful.
That said, this issue is a duplicate of https://github.com/godotengine/godot-proposals/issues/892, closing. If you want to see this implemented, feel free to give the proposal a :+1: and let people know about this problem and the proposal.


Allow forcing polyfill

I found a bug in Chrome that window.fetch does not send cookie when POSTing, even have credentials set (see Issue 490519). So we may have to polyfill chrome as well, at least provide a way to do so.
Currently github/fetch only polyfills UA whose window.fetch is undefined. So it will not polyfill the broken fetch in chrome.

That's not a big, that's per-spec.

But this polyfill sets withCredential to true, which sends cookie. Also, firefox sends cookie as well.

Your linked example doesnt set with credentials (don't forget the trailing s)

Sorry, I was on mobile and not being clear.
Your linked example does not include the { credentials: "include" } option, which (per spec) is required if you want to send cookies. So Chrome's behavior is correct; Firefox's is not, presumably because the polyfill is incorrect, and the polyfill is being used in Firefox.
In general, when two browsers disagree, it is better to check the spec, and file a bug on the browser that doesn't match the spec, instead of filing a bug on the browser whose behavior doesn't match what you wish were standard.
If anything, the bug here is that the fetch polyfill needs to strip out Cookie headers (if that's possible) when not setting { credentials: "include" }.

@domenic I am sorry I made a mistake there. This fetch is actually a wrapper I wrote, which applies credentials: "include" by default. IT IS A BUG OF CHROME!

@tjwudi Can you explain what is a bug of Chrome, i.e. how does it fail to conform to the fetch spec?
Bear in mind that something can be a bug in our implementation as well. Just because Chrome and this polyfill differ, it doesn't mean that Chrome necessarily has a bug.
Did you try using credentials: 'same-origin'?

Hi @mislav
fetch in Chrome fails to send cookie when POSTing with credentials: include set.
In the polyfill, when credentials is set to "include", it will set withCredentials to true, which is the expected behavior. See this line.
My Chrome Version 43.0.2357.65 (64-bit)

I mean, that's just not true. We have many examples, including production websites, getting credentials in Chrome. I asked you for a runnable repro case in the Chromium bug and you haven't provided. I was going to wait until you did so but at this point you're just spreading false information.

@tjwudi I just tried in Chrome 43.0.2357.65. Both same-origin and include values for credentials: key enable sending the cookie value. I'm sorry, but I can't reproduce what you're describing. And in your Chromium bug report, you tried to demonstrate the bug by including sample code, but then you said that the fetch() in the code isn't native fetch, but your own wrapper that edits the option. You'll have to provide complete code to demonstrate this issue.


Fy2019 Q2 content refresh

fixed few typos
updated Node.js steps to be specific in installing Node.js v8 LTS
updated lab & code samples to SPFx v1.7

thx @andrewconnell, appreciated.


Ability to pass validatorOptions to yup

First of all, thank you for creating this library. Would you consider implementing a way to pass validatorOptions down to yup so I can configure things like abortEarly so the ValidationError contains all errors, instead of only the first one?
Perhaps something like this?
const appConfig = yupEnv({
  schema,
  env: env.parsed,
  prefix: "APP_",
  levelSeparator: "__",
}, {
    abortEarly: false,
    strict: true
});

Good idea! Definitely a useful option. I'll implement and publish it in the next few days.

Hi again, I've just published a new release with the feature. You can pass options to yup this way:
const appConfig = yupEnv({
  schema,
  env: env.parsed,
  prefix: "APP_",
  levelSeparator: "__",
  validate: {
    abortEarly: false,
    strict: true
  }
});


Feedback from the Charcoal Test room should not go into production Metasmoke

A request by triplee. Functionality for that currently doesn't exist, so it'll have to be implemented seperately.

I'm gonna decline this. While I understand the premise that test stuff generally shouldn't be immortalized, Charcoal Test is more there just to keep inane stuff out of the way of CHQ rather than to be a true test environment. The work required to do this kinda outweighs the value we get for making it such an environment.


Implement ssh with nodeshell + nsenter alternative method

Checklist

Adresses Issue:
Quick summary of changes
Changes are GNU/BSD Compatable: [yes/no]
Changes are applicable to the wider community.
Request help with testing if needed.


Whitespace removed with variable

Describe the bug
When rendering a variable in an HTML tag, any additional whitespace is removed.
Leptos Dependencies
[dependencies]
leptos = { version = "0.5.6", features = ["csr"] }
leptos_meta = { version = "0.5.6", features = ["csr"] }
leptos_router = { version = "0.5.6", features = ["csr"] }

To Reproduce
Steps to reproduce the behavior:

Add a variable with some text
Like:

{myvariable} km/h

Let's say myvariable is 10
Output will be 10km/h instead of 10 km/h

Expected behavior
I don't expect the spaces to be removed

This is working as designed, in line with limitations of whitespace in proc macros.
The default syntax is to wrap all text nodes in double quotes like " km/h", with the nice ability to omit those double quotes in most (but not all cases).
i.e., simply use
{myvariable} " km/h"
 in situations where you need to ensure this whitespace is preserved, not trimmed.

Thank you, this explains it all


[FLINK-2903] Format numeric values in web frontend with commas

Forgot to add the component name in the commit message. I don't wanna trigger a useless build on travis, so whoever commits this should probably make that change before merging.

LGTM

Merging this PR


GUVNOR-2603 : Guided Decision Table: V&V: Adding BRL Condition column throws IndexOutOfBounds in

https://issues.jboss.org/browse/GUVNOR-2648

Fixing the conflicts.

@karreiro if you could please review.

@Rikkola +1 :-) Thanks

I left few not needed variables in, some even affected performance. Cleaned them and rebased with master.


[Bug] Wireless devices are shown as "wired"

The problem
Wireless entities are being shown as Connection Type = Wired. Worked fine in 0.24.0b3.
Your device model
ZenWiFi ET8
Firmware type
Stock
Firmware version
<IP_ADDRESS>.388_23759
Integration version with the issue
0.24.1
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2023.11.2
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
No response

@JohnyR, please, check the latest release 0.24.2 - the issue should be fixed


get smart speaker mostly working

This is generally just un-commenting what you already had, but there was a few issues I've fixed to get it working.  Would really like to see where we can go with this approach versus a Fan or Light Bulb.
Advantages of using Service.SmartSpeaker.

Shows up in Home app
Siri won't get confused about Lights or Fans
Volume can be set with automations

Disadvantages of using Service.SmartSpeaker

Cannot set volume on demand (yet)
Not exactly sure how PLAY/PAUSE/STOP (Current/TargetMediaState) should be handled (I've changed it to Stopped as this will help with Automations)

I changed smartspeaker a bit to use setCharacteristic and updateCharacteristic so get/getCharacteristic is no longer needed as Homebridge handles this for you when done this way.
I've also fixed it so they tile starts with a state of STOP on first boot, but clicking on it activates either 'PLAY' or 'PAUSE'.  I am toying with the idea of possibly translating that so 'Paused' is muted, and 'PLAY' is volume on. Would want your opinion or that of others if this would be beneficial.  Could always make the bits of it configurable I suppose.

@Martvvliet Let me know your thoughts on this as I would love to see this get merged.

How would I go about using your modified pluggin?

How would I go about using your modified pluggin?

@Martvvliet is welcome to chime in and get these changes merged. However, until Homebridge can handle volume setting on smart speakers better it was not useful enough yet in my opinion.

@Martvvliet is welcome to chime in and get these changes merged. However, until Homebridge can handle volume setting on smart speakers better it was not useful enough yet in my opinion.

@Martvvliet is welcome to chime in and get these changes merged. However, until Homebridge can handle volume setting on smart speakers better it was not useful enough yet in my opinion.

I ended up just changing each line of code myself using your commit as reference. I would have to agree not quite useful enough. I'm have issue getting this plugin to work properly anyways with my HEOS speakers. homebridge-upnp works well but it is set at light service. Smart Speaker would be much better so do accidentally turn off them when turning off actual lights in house. Would you mind helping me change the https://github.com/langovoi/homebridge-upnp plugin to speaker service

@Martvvliet is welcome to chime in and get these changes merged. However, until Homebridge can handle volume setting on smart speakers better it was not useful enough yet in my opinion.

I ended up just changing each line of code myself using your commit as reference. I would have to agree not quite useful enough. I'm have issue getting this plugin to work properly anyways with my HEOS speakers. homebridge-upnp works well but it is set at light service. Smart Speaker would be much better so do accidentally turn off them when turning off actual lights in house. Would you mind helping me change the https://github.com/langovoi/homebridge-upnp plugin to speaker service


Added more optimization in signin and signup page with improving ui

Hey, 👋👋 there this is my first pr toward this repo I optimized sign in or sign-up page in which I made them in a component so that the page will not have to load every time user wants to switch between them only a certain component will load up also the supabase function for sign in and sign up is working fine as they are. I tried to make the UI good hope you like it.

A few things:

The screenshots look good.

Can you keep the same routes as before (/signin and /signup I believe) to avoid confusion?

Have you tested all the functionalities? Is there any extra step that's needed to set this up on the Supabase side other than what's already described in README?

The sign in and sign up function is same as previous only what I did was make the UI more optimized

The routes have changed.
For example, here: https://defaang-git-fork-adityapainuli-main-ykdojo.vercel.app/
https://defaang-git-fork-adityapainuli-main-ykdojo.vercel.app/signup no longer works.
Also, did you test all the functionalities including the Google signup?

No google signup didn't work right now It just a static button I will work on that one right now and for the routes I will also create them in two different page so that both of them will work as same

Hmm it says this was merged, but it's not showing up in the commit history - not sure why.
Anyway, I requested changes. If it was merged, which I'm not sure it was - can you revert it?

let me try It is my first time trying these things out .

hey can you help me I don't know why I can't push the code after I fixed the bug can you please help me with what should I do?

Can you just make a new branch and a new PR?

Ok


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1347 ms

Resolved: Bofrost.es performance has improved in 88af509.


Fix tls-client different operating system dependencies

What does this PR do?
Fixes tls-client compiling on differents operating systems. Tested this on both linux and windows, they work. Not sure about MacOS but should work too.
Why was it initiated? Any relevant Issues?
due to issue #2253 (tls-client only compiling on windows)
PR Checklist

[x] Correct base branch selected? Should be develop branch.
[x] Enabled commit hook or executed ./bin/autoformat-nuitka-source.

Looks perfect to me, I am right now releasing 1.6, kind of locking things, but I will merge it for the hotfix soon after, 1.6.1.


Fix unit tests

Many were compromised by recent changes to BodyDelegate
Some strategy for consistency would be IDEAL. There is a lot of variability in how various tests are authored.

Tests fixed and run. Consistency isn't all there - but that'll too vague to accomplish as a one sentence task


Remove trivial snippets

There are many snippets that don't do much, such as
    "mod": {
        "prefix": "mod",
        "body": ["mod ${1:name};"],
        "description": "mod …;"

Having the snippet is not very useful in this case. Snippets should be reserved for common idioms that are somewhat annoying/long to type.
These should be removed to reduce noise and to allow more snippets in the future.
There are also some extremely niche macros that here such as include!(), column!() and line()! which I'm not sure why they are even snippets

Hi @dev-ardi, I started a PR here, to address this issue. When you get a moment, I'd appreciate your thoughts on the snippets removed.


Threadsafe?

Hey, I would like to use it in multi thread environment. Is it safe?

Ping :(

current master uses dry-events for notification bus, and it should be thread-safe because global events are stored as Concurrent::Map objects. I hope to release all of that before the end of the year.


Update README.md

add documentation link

I guess it would probably be preferable to add the link as the repo website on github too :)

This link may be not a must. But when I learn how to use this repo, I want to make a link to doc be found easier

I'll add the link as the repo website 👍


feat(hardware): add InstrumentInfoRequest to get info from Pipettes & Gripper

Overview
Since pipette and gripper are technically both robot instrument, I'm adding an InstrumentInfoRequest so both systems can respond to host when prompted.
This also means we have to update the tool detector class to read the info from both instruments correctly.

Changelog

Review requests

Risk assessment

Opened a new ticket on Jira for adding an integration test


Long-Form Speech Recognition from example doesn't work

With auto-installed dependecies:
>>> recognition_result = model.transcribe_longform("long_example.wav")

A module that was compiled using NumPy 1.x cannot be run in
NumPy 2.0.2 as it may crash. To support both 1.x and 2.x
versions of NumPy, modules must be compiled with NumPy 2.0.
Some module may need to rebuild instead e.g. with 'pybind11>=2.12'.

If you are a user of the module, the easiest solution will be to
downgrade to 'numpy<2' or try to upgrade the affected module.
We expect that some modules will need time to support NumPy 2.

After installing numpy 1.26.4 it works:
>>> recognition_result = model.transcribe_longform("long_example.wav")
INFO:pytorch_lightning.utilities.migration.utils:Lightning automatically upgraded your loaded checkpoint from v1.1.3 to v2.4.0. To apply the upgrade to your files permanently, run `python -m pytorch_lightning.utilities.upgrade_checkpoint ../../.cache/torch/pyannote/models--pyannote--segmentation/snapshots/059e96f964841d40f1a5e755bb7223f76666bba4/pytorch_model.bin`
Model was trained with pyannote.audio 0.0.1, yours is 3.3.2. Bad things might happen unless you revert pyannote.audio to 0.x.
Model was trained with torch 1.7.1, yours is 2.5.1+cu124. Bad things might happen unless you revert torch to 1.x.
>>>
>>> for utterance in recognition_result:
...     transcription = utterance["transcription"]
...     start, end = utterance["boundaries"]
...     print(f"[{gigaam.format_time(start)} - {gigaam.format_time(end)}]: {transcription}")
...
[00:00:00 - 00:16:83]: вечерня отошла давно но в кельях тихо и темно уже и сам игумен строгий свои молитвы прекратил и кости ветхие склонил перекрестясь на одр убогий кругом и сон и тишина но церкви дверь отворена
...........

I had the same issue. It seems like it is pyannote error.
pip install numpy==1.26.4 worked for me.
Also tried to run with other VAD model Silero and It worked great (However, you need to manually pass audio chunks to the model)
P.S. Thanks to the Salute development team for the new version of GigaAM!


Intersection.Aux[Int :: HNil, Int :: HNil, HNil]

There seems to be a problem with Intersection:
scala> implicitly[Intersection.Aux[Int :: HNil, Int :: HNil, Int :: HNil]]
res27: shapeless.ops.hlist.Intersection.Aux[shapeless.::[Int,shapeless.HNil],shapeless.::[Int,shapeless.HNil],shapeless.::[Int,shapeless.HNil]] = shapeless.ops.hlist$Intersection$$anon$58@342165b9

great! because { Int } ∩ { Int } = { Int }
scala> implicitly[Intersection.Aux[Int :: HNil, Int :: HNil, HNil]]
res25: shapeless.ops.hlist.Intersection.Aux[shapeless.::[Int,shapeless.HNil],shapeless.::[Int,shapeless.HNil],shapeless.HNil] = shapeless.ops.hlist$LowPriorityIntersection$$anon$56@50fc9b74

what? { Int } ∩ { Int } = ∅ also?

scala> reify { implicitly[Intersection.Aux[Int::HNil,Int::HNil,HNil]] }
res29: $r.intp.global.Expr[shapeless.ops.hlist.Intersection.Aux[shapeless.::[Int,shapeless.HNil],shapeless.::[Int,shapeless.HNil],shapeless.HNil]] = Expr[shapeless.ops.hlist.Intersection.Aux[shapeless.::[Int,shapeless.HNil],shapeless.::[Int,shapeless.HNil],shapeless.HNil]](Predef.implicitly[hlist.Intersection.Aux[::[Int, HNil], ::[Int, HNil], HNil]](hlist.this.Intersection.hlistIntersection1(hlist.this.Intersection.hnilIntersection)))

simplifies to
Intersection.hlistIntersection1(Intersection.hnilIntersection))

implicit def hnilIntersection[M <: HList]: Aux[HNil, M, HNil] =
  new Intersection[HNil, M] {
    type Out = HNil
    def apply(l: HNil): Out = HNil
  }

implicit def hlistIntersection1[H, T <: HList, M <: HList]
  (implicit i: Intersection[T, M]): Aux[H :: T, M, i.Out] =
    new Intersection[H :: T, M] {
      type Out = i.Out
      def apply(l: H :: T): Out = i(l.tail)
    }

Yeah.  hnilIntersection says ∅ ∩ {Int} = ∅, which is good.
But hlistIntersection1 says if ∅ ∩ {Int} = ∅, then ∀x, {x} ∩ {Int} = ∅, which is false when x = Int

There's an equivalent bug in Union:
import shapeless._, ops.hlist.Union
implicitly[Union.Aux[Int :: HNil, Int :: HNil, Int :: Int :: HNil]]

compiles as well.  I'm working on a patch but I'm on a very slow computer and still waiting for tests to compile :-\


[core] corrected the xutility inclusion.

https://jenkins.code.rdkcentral.com/job/github-triggered/251/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/255/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/273/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/298/ started


feat: autoinstrument: add early exit based on application name

Adds a new environment variable SPLUNK_AUTOINSTRUMENT_PACKAGE_NAMES which can be used to specify an allowlist of package names. When set, only the packages matching the list will be autoinstrumented.

Codecov Report
Base: 85.21% // Head: 85.28% // Increases project coverage by +0.07% :tada:

Coverage data is based on head (b1726ad) compared to base (f48c8a9).
Patch coverage: 82.35% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #656      +/-   ##
==========================================
+ Coverage   85.21%   85.28%   +0.07%
==========================================
  Files          27       27
  Lines        1204     1210       +6
  Branches      273      275       +2
==========================================
+ Hits         1026     1032       +6
  Misses        178      178

Impacted Files
Coverage Δ

src/instrument.ts
85.71% <82.35%> (+5.71%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Avoid using Delayed in to_csv and to_parquet

This accomplishes two things:

Lets dataframe optimizations through
Avoids a final synchronization point in the graph
which improves scheduling heuristics

Edit
I first used dask.compute more liberally in if compute: branches in order to avoid the final delayed call.  Then I remembered the handy compute_as_if_collection function which lets us avoid creating anything, and directly specify the collection type.  This is the second commit and probably the right way forward.

OK, so I'm doing something dumb with parquet, but after staring at it for 20m I'm hoping that @rjzamora can help me out.
I do generally think that this is a good direction to go in though.  It's also probably something that we should watch out for.

Then I remembered the handy compute_as_if_collection function which lets us avoid creating anything, and directly specify the collection type.

Interesting! I was actually unaware of compute_as_if_collection

Interesting! I was actually unaware of compute_as_if_collection

Yeah, I had totally forgotten about it.  Probably it was my job to tell you about it in the first place :)

I noticed is compute_as_if_collection we're converting to a low-level task graph
https://github.com/dask/dask/blob/95fb60a31a87c6b94b01ed75ab6533fa04d51f19/dask/base.py#L312
Maybe we should leave that to the optimization function instead?

Maybe we should leave that to the optimization function instead?

Probably a good idea!  This function is old.

xref https://github.com/dask/dask/pull/7969 -- let's see if tests pass : )


docs: update with-express example to use a per-request client

Types of changes

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

Description
This PR updates the with-express example to use a new client for each request. This is needed since clients are stateful, primarily due to preview support.
See the TSDoc comment in the new createClient() function for details.
Fixes #175
Checklist:

[ ] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

🐘

Codecov Report

Merging #228 (9de7eb8) into master (71182e3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #228   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          360       360
  Branches        67        67
=========================================
  Hits           360       360

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 71182e3...9de7eb8. Read the comment docs.


Create/update local, dev and prod environments

https://github.com/clevercanary/hca-atlas-tracker/issues/13
Need
Need to have configurations for the various places the application will run/be hosted. They are:

local (Localhost)
dev (https://tracker.data.humancellatlas.dev.clevercanary.com/)
prod (https://tracker.data.humancellatlas.org)

Currently, we have
Dev and prod. Def is co
Dev Steps

[ ] add a local config
[ ] update dev/local/prod with the correct browser URL
[ ] Update start/build scripts to match the environments

Merged


Run line of code (or chunk) by Ctrl + Enter and automatically include entire chunk, advancing cursor to the next line / chunk

In RStudio when you run code line-by-line the cursor automatically advances down to the next line, and if it is a chunk of code broken into several lines, it runs the entire statement across multiple lines, advancing the cursor down. So you can run the entire file by just keep pressing Ctrl + Enter.
I haven't found a way to accomplish that, and I hope it is already there, or a consideration for the future.
Thank you.

hi, there's another bug report that mentions this might not be working, I am also facing the same issue and I originally thought that this feature was not supported.
see here #1313


Naming suggestion

s/microwire/wicro/ 😄

Hahah thanks @asim
This a rough test though. We're playing with the idea of dependency injection for v5, to get rid the current service stype where all components are statically defined, to a more dynamic way where your service will only include the components you're actually using, and allow you to make any extra custom components.
We've been discussing this on discord and some hacked.io notes, if you're curious

Interested to see how it turns out. Always wanted to integrate wire but didn't have the skills to do it.

I've extended the README with the Goals.

@asim You made a great Framework here, others copied stuff from it and I like it's flexiblity a lot.

s/microwire/wicro/ smile

Sound's for me like wracked micro. :/

Looking at the examples. If I have to import this package it needs a great name. Wire itself was a good name, I don't think it would conflict with the existing package but I can see how it also might be confusing.
Just think about
Import "github.com/go-micro/xxx

service := xxx NewService(
   xxx.WithFoo(...),
)

Whatever I import I tend to read out loud in my head and I want that to make sense in a sentence. So one word relevant to whatever I'm setting up.

Thought about dicro but that sounds like dick ro
Also this project is just a POC, it will be merged into go-micro/go-micro/v5 ?

I'll leave you and @Davincible to figure out the best path forward. I'm no longer deep in the code so it makes it harder for me to be as useful in those decisions.


globalCompositeOperation values on MDN have non-standard values

Request type

[] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
I was viewing the html spec when I followed the link to MDN to see what the valid values are. Some things didn't look right so went to look at the COMPOSITE spec and the values do not agree with the MDN doc. These need updating on MDN or people use values that are not interoperable.

Looks like this was down to an issue with the rendering of the spec.


Use instead of import

Uses proper sass, as the import is going to removed eventually

Thanks, but as it has been discussed plenty of times, we are not going to switch to Dart Sass specific usage for the time being.

ok

I understand that the Bootstrap team doesn't want to switch to Dart Sass specific usage, however, me personally would like to use @use instead of @import, since the Sass team discourages using @import. For this reason, I will be (or at least try) using @use in my projects, but I'm running into some issues.
Whenever I try to use @use instead of imports, I get this error:
Error: ("primary": #0d6efd, "secondary": #6c757d, "success": #198754, "info": #0dcaf0, "warning": #ffc107, "danger": #dc3545, "light": #f8f9fa, "dark": #212529) isn't a valid CSS value.
   ╷
94 │ $theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value") !default;
   │                             ^^^^^^^^^^^^^
   ╵
  node_modules/bootstrap/scss/_variables.scss 94:29  @use
  sass/source/main.scss 10:1                         root stylesheet

Is this an issue that is "unfixable" and I have to use @import? I hope not...
@theblobscp, since you're the one who created this PR, were you able to get this working for yourself? What am I doing wrong?

I'm not sure if I can help you, I don't have the code that caused this


Favicons detection looks broken #3534

Closes #3534

Forced push after rebase master.
Also tweaked to request for a bigger icon (if available):
let url = URL(string: "https://www.google.com/s2/favicons?sz=256&domain_url=\(host)")

Original (64)
Now (256)


angular 1.6.1 - $resource: no exception in console

Do you want to request a feature or report a bug?
bug
What is the current behavior?
No error message is output in the console
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem:
No error message is output in the console:
var User = $resource('users/:id', {id: '@id'});
var user User.get({id: 123}, function (res) {
    y  = x + z; // this line not log any exception on the console
}, function () {
    console.log(arguments);
});

Error message is output in the console:
var User = $resource('users/:id', {id: '@id'});
var user User.get({id: 123}, function (res) {
    y  = x + z; // and this log the exception in the console
});

What is the expected behavior?
Exception output in the console in both cases
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
Angular: 1.6.1
Chrome: 55.0.2883.87

Argh! It's a valid issue. We are silencing Possibly Unhandled Rejection errors if an error callback is provided, but this will incorrectly silence errors in the success callback, which are never passed to the error callback.
There success/error callbacks are messy anyway and I would like to deprecate them. It is much better to use "proper" promise methods:
var user User.get({id: 123});
user.$promise.then(onSuccess, onError);
// or even
// user.$promise.then(onSuccess).catch(onError);

In the meantime, we should fix this 😞

I made a PR for this : https://github.com/angular/angular.js/pull/15628


Typesafe Global formats

Is your feature request related to a problem? Please describe.
Global formats are awesome. It's great to help achieve consistency in my application. However, if I create these global settings for the formatter, they won't be typesafe
format.dateTime(new Date('2020-11-20T10:36:01.516Z'), 'short'); // => the second parameter requested is always of type 'string'.
Describe the solution you'd like
I'd like for the formatter to be of a narrower type, based on the template literal values of what I added in my config
Describe alternatives you've considered
In my global.d.ts file:
import { formats as myFormats } from "@kdx/locales";
declare module "use-intl" {
  function createFormatter({
    _formatters: formatters,
    formats,
    locale,
    now: globalNow,
    onError,
    timeZone: globalTimeZone,
  }: Props): {
    dateTime: (
      value: Date | number,
      formatOrOptions?: keyof typeof myFormats.dateTime | DateTimeFormatOptions,
    ) => string;
  };
}

my @kdx/locales file:
import type { Formats } from "use-intl";
export const formats = {
  dateTime: {
    extensive: {
      day: "2-digit",
      month: "long",
      year: "numeric",
      hour: "numeric",
      minute: "numeric",
    },
  },
} as const satisfies Partial<Formats> | undefined;

As you can see, this ALMOST works. I am extending/rewriting the return type of createFormatter from use-intl. In my code, now the dateFormat function is typesafe.

However, there is a big problem: since I completely redeclared the return type of createFormatter, now I all of the typesafety for all other methods are gone, and only this function is declared inside it :stuck_out_tongue:

I absolutely agree, would love to have this but haven't found the time yet to look into this. There's already an issue open at https://github.com/amannn/next-intl/issues/1112, I'll close this in favor of the former one.
Let me know in case you'd be interested in helping out with this!

Sorry, I wasn't able to find this issue. I usually try to search for similar issues before posting!


Specific video is downloaded with black video content

Information

OS: Linux
Version: 0.0.9
Terminal: bash, Tilix

Describe the bug
A specific video was downloaded with black video content, even though it displays normally on Switchtube.
This both happens with -c and -v.
To Reproduce
switchtube-dl -v "YZXIaohdhH" -t $SWITCHTUBE_TOKEN
(only reproducible with a token that has access to the channels of organization 86)
Expected behavior
The video is downloaded with the correct video content.
Screenshots
N/A
Additional context
N/A

After another try, a few hours later this works perfectly.
Will close this for now, and see whether this ever comes up again.


FileTests in fml_tests are failing on Fuchsia

They are filtered out in run_tests.sh but fail when running fml_tests manually

https://github.com/flutter/engine/pull/18320

FileTest.CanTruncateAndWrite and FileTest.CreateDirectoryStructure are still filtered

@arbreng: These just bottom out to ftruncate calls. Can we investigate why this would be an issue on Fuchsia? This test is still disabled.

Filed https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=81676 so we can triage internally.  Closing this issue.


Quitnus not loading assets

I copied the code from the basic Quintus platformer from the website home page with the same assets and nothing is loading. I tried running the game from a webserver (more specifically mongoose server) which also didn't work. The assets were being loaded with the Q.load function.

Did you look at the Network tab in your DevTools?
What URL is being loaded? What error do you get?

There is nothing in the network tab at all, and on every file it says uncaught exception: Error loading <filename>

This is weird. You should see the network queries in the Network tab.
Does the Console provide more info about what's going on?

Here are screenshots of what errors I am having. I can send you the code if you would like.

Please post the code you use to load those assets and the structure of your files.

Here is a screenshot of the file structure:

And here is the code for the loading of assets:
Q.load("test.tmx, rndgrsstile.png, player01.png, enemy01.png, tower01.png",
  // The callback will be triggered when everything is loaded
  function() {
    Q.stageScene("level1");
  });

What version are you using from the CDN?
What does the Network tab show exactly?

I am using v0.2.0 as it said in the tutorial on the Quintus home page. The Network tab shows nothing in both Firefox and Google Chrome.

It's weird that no XHR request is sent.
Have you tried using an Array to specify the assets, rather than a String?

I tried using a try-catch block and printing the error to no success, as I got the same error. I used an Array which again threw the same error. It seems to act as if the files don't exist. I've tried this on two different systems to get the same error both times.

But have you used the errorCallback option (part of the third parameter for the Q.load method)?

May I ask how I would use that? I am a hobbyist programmer and Javascript is not my best language. Say if my function is set up like so:
var filestoload = ['test.tmx', 'rndgrsstile.png', 'player01.png', 'tower01.png', 'enemy01.png'];
//Q.load("test.tmx, rndgrsstile.png, player01.png, enemy01.png, tower01.png",
Q.load(filestoload,
  function() {
    Q.stageScene("level1");
  });

How would I integrate the errorCallback from here?

In the third parameter (not set here), pass an object with a errorCallback key, like so:
Q.load(files, cb, { errorCallback: (err) => { console.dir(err) } });

I got that to work after a bit of trial and error, in the end this worked out well:
var filestoload = ['test.tmx', 'rndgrsstile.png', 'player01.png', 'tower01.png', 'enemy01.png'];
var startlevel1 = function(){
  Q.stageScene('level1');
  console.log("First level loaded!");
};
var errorCallback = function(err){
  console.dir(err);
};
Q.loadTMX(filestoload,startlevel1(), errorCallback(err));

The error it throws out is this: TypeError: data is undefined for the file quintus.js

Could you give me the whole error stack (including the line number)?

Here is a screenshot of the full error:

Looks like the problem is your TMX file, not the assets loading.
And I just took a closer look at your code:
Q.loadTMX(filestoload,startlevel1(), errorCallback(err));

This does 3 things:

Ask Quintus to load a TMX file when you give a list of different files (1 TMX, 4 PNGs…), some of which are not supported
You directly call the callback (by putting parenthesis after startlevel1) instead of giving its reference, before the assets are actually loaded
Same with the errorCallback — which BTW should be in a object — which also uses an undefined err variable

That removed the error entirely, but still nothing shows up at all on the screen. I shortened my code to this to minimize errors, as all I am trying to do is get a simple level rendered:
var startlevel1 = function(){
  Q.stageScene('level1');
  console.log("First level loaded!");
};
var firstlevelfiles = ['test.tmx', 'rndgrsstile.png'];
window.addEventListener('load', function(){
  var Q = window.Q = Quintus().include("Scenes, TMX").setup({maximize:true});
  Q.scene("level1", function(stage){
    Q.stageTMX('test.tmx', stage);
    stage.add('viewport');
  });
  Q.load(firstlevelfiles, startlevel1, { errorCallback: (err) => { console.dir(err) } });
});

Here is the HTML code if you were wondering:
<!DOCTYPE HTML>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, user-scalable=0, minimum-scale=1.0, maximum-scale=1.0"/>
<script src='http://cdn.html5quintus.com/v0.2.0/quintus-all.js'></script>
<script src='game.js'></script>
<style>
  body { padding:0px; margin:0px; }
</style>
</head>
<body>
</body>
</html>

I've never used TMX files, but as long as everything is loaded, all that's left todo is add some logs to see which functions have been called.
Also check the docs to make sure you didn't miss anything on the TMX use process.

So I converted the TMX to Tiled's JSON format and it still came up blank in Firefox. I tested this again with Google Chrome, and the error came back. This time its a 404 error, but again the files are in the base directory. I am using the same code as above, here is the screenshot of the error:

The 404 is related to the favicon.ico file which is automatically loaded by Chrome if not favicon is specified in the HTML. Don't worry about it.
You do have a syntax error in your file, though, on line 16.

This error doesn't show on Firefox, only Chrome. The line that the error is occurring is Q.load(firstlevelfiles, startlevel1, { errorCallback: errorCallback } }); which, as far as I know, is syntactically correct. All of the parenthesis and brackets are there.

It's not syntactically correct as there is one closing bracket too many. Firefox should have seen it too.

Thank you, my IDE did not pick that up, its the kind of thing you wouldn't notice till someone points it out. Now Chrome is giving me a different 404 error while Firefox is not. Here is the console in both:
Chrome

Firefox

This is using the exact same code as the post from a few days ago, just with the brackets fixed.

You should check that the files are in the right folders then.
Quintus looks up files in dedicated directories by default. Make sure you follow the specs.

I got the assets to load finally, and the world loads in. I don't know where the level itself is but I think i can figure it out by tinkering with the X and Y on the character I added. Thanks for the help!


feat: Make Serde non-owning

Fix #2955
Change the Serde trait's serialize to be a non-owning method.
Some details:

Cairo corelib was updated to use .serialize as a method for cleaner code, this required some use serde::Serde
rust plugins were updated, but were kept as free functions. Otherwise, one has to use serde::Serde somewhere which seems like it would lead to trouble. In particular, use can't be used in impl.
I haven't really changed block_generator_test_data/serialization, under the assumption that this would keep the test more valid.
There is a bug in user_defined_types's TestContract following #2845 , this PR fixes it.

As a final note, I am wondering if the method should be renamed to serialize_into maybe ?

This change is

Reviewable status: 34 of 36 files reviewed, 1 unresolved discussion (waiting on @spapinistarkware)

crates/cairo-lang-plugins/src/plugins/derive.rs line 284 at r2 (raw file):
Previously, spapinistarkware (Shahar Papini) wrote…
Err no you're correct, I wrote the right thing initially and then forgot that x isn't the same fix in both cases so I merged them incorrectly. Should now work correctly.
Do you mean a cairo-test ? There is already a derive test (TwoVariantEnum) in derive

How did the test not catch the mistake?

Reviewable status: 34 of 36 files reviewed, 1 unresolved discussion (waiting on @spapinistarkware)

crates/cairo-lang-plugins/src/plugins/derive.rs line 284 at r2 (raw file):
Previously, spapinistarkware (Shahar Papini) wrote…
The derive tests only check the expansion of the code is 'as expected' - and I had manually changed it to look like what it was indeed doing, since I didn't notice the mistake in the expanded code either. As far as I can tell they don't try to compile or actually run the code, hence why I'm asking about cairo-level-tests

Gotcha, so please add a cairo level test with the issue number under bug_samples :) you can serialize and deserialize.

Had to rebase on top of #2938 as I had to explicit which casting I wanted in corelibe/serde.cairo
Added a test for a simple struct / enum serialisation. I checked that it failed with the infinite loop version (it did), and now works.

This item belongs to payment request #70D8DC on OnlyDust:

from abdelhamidbakhta to wraitii
2 items included
$500 for 1 day of work