Linting and formatting checks and backend tests fail in GitHub actions after project creation

Linting and formatting checks fail in GitHub actions, immediately after project creation.
To reproduce:

Create a new project with Scaf
Run make compile
git init
git add .
`git commit -m "Initial commit"
Add git remote and push

Notice how the checks fail:

Linting and formatting checks fail with:
reformatted backend/myproject/contrib/sites/migrations/0002_alter_domain_unique.py
reformatted backend/myproject/contrib/sites/migrations/0001_initial.py
reformatted backend/myproject/contrib/sites/migrations/0003_set_site_domain_and_name.py
reformatted backend/myproject/users/admin.py
reformatted backend/myproject/users/forms.py
reformatted backend/myproject/users/models.py
reformatted backend/myproject/users/tests/factories.py
reformatted backend/myproject/users/migrations/0001_initial.py
reformatted backend/myproject/users/views.py

All done! ✨ 🍰 ✨
9 files reformatted, 32 files left unchanged.

isort....................................................................Passed
ruff.....................................................................Passed
bandit...................................................................Passed
make: *** [Makefile:10: check-lint-and-formatting] Error 1
Error: Process completed with exit code 2.

Backend tests fail with:
kubectl exec -it $(kubectl get pods -l app=backend -o jsonpath="{.items[0].metadata.name}") -- bash -c "cd myproject && pytest --cov=./ --cov-report html --ds=config.settings.test"
E0413 14:15:35.839558    2282 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0413 14:15:35.840075    2282 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0413 14:15:35.841567    2282 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0413 14:15:35.842011    2282 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0413 14:15:35.843434    2282 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
The connection to the server localhost:8080 was refused - did you specify the right host or port?
error: pod, type/name or --filename must be specified
make: *** [Makefile:25: backend-test] Error 1
Error: Process completed with exit code 2.

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Frontend build failure with Next.js 8

When upgrading to Next.js 8, the build is regularly failing in our CI environment.
> next build dist

Creating an optimized production build ...

> Using external babel configuration
> Location: "/home/circleci/project/.babelrc"
internal/child_process.js:372
    throw errnoException(err, 'spawn');
    ^

Error: spawn ENOMEM
    at ChildProcess.spawn (internal/child_process.js:372:11)
    at spawn (child_process.js:559:9)
    at Object.fork (child_process.js:113:10)
    at fork (/home/circleci/project/node_modules/next/node_modules/worker-farm/lib/fork.js:12:35)
    at Farm.startChild (/home/circleci/project/node_modules/next/node_modules/worker-farm/lib/farm.js:106:16)
    at Farm.processQueue (/home/circleci/project/node_modules/next/node_modules/worker-farm/lib/farm.js:279:10)
    at Farm.<anonymous> (/home/circleci/project/node_modules/next/node_modules/worker-farm/lib/farm.js:97:21)
    at listOnTimeout (timers.js:324:15)
    at processTimers (timers.js:268:5)

Sample failure in CI: https://circleci.com/gh/opencollective/opencollective-frontend/8756
Issue in our frontend repository: https://github.com/opencollective/opencollective-frontend/pull/1442
Issue in next.js repository: https://github.com/zeit/next.js/issues/6353
We need a reliable fix that address the root cause of this issue, so we can upgrade to Next.js 8.

A $300 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $300 when the Pull Request is reviewed, accepted and merged. More info.

Looks like this was fixed upstream. @ijjk do you want to claim the bounty?

@znarf I think it would be good to transfer the bounty to another issue so someone else has a chance to get it.

@znarf Hey 👋 The Next.js team at ZEIT (which @ijjk is part of) would like to donate the bounty to https://opencollective.com/wwcode if that's possible.

@ijjk @timneutkens Thank you! We'll think about the best way to do that and will notify here when it's done.

@znarf Thanks! 💯

@ijjk @timneutkens To proceed, we would like to give you a Gift Card. You will be free to give the whole amount to https://opencollective.com/wwcode, and you will be credited for that. For Zeit to be credited (see sample screenshot), the person donating will need to be an admin (Core Contributor) of the Zeit organization on https://opencollective.com/zeit.

@znarf Sounds good to me! Is it already applied to the account or do you need details for sending the gift card? In that case you can find me on https://twitter.com/timneutkens / https://spectrum.chat/users/timneutkens

@timneutkens Great, I will PM you.


Image re-upload is not working

When we upload an image, and then try to upload the same image again, change event is not firing. Also is it possible to make it compatible with react 17?

@kakarlasaiteja I have the same issue here, did you get a proper solution ??

I am facing the same issue, any solutions?

duplicate for #59


bump IBC version

Bump to IBC v3.3.0, which has a minor security fix.

Have to correct the SDK version in the backport


More connectors to which the transport rule applies should be added.

It is difficult to see from the title of the docs that only the "office 365 outlook" connector can control the transport rules from the power platform.
It is "Email exfiltration controls for connectors" and it seems to be for all MS connectors that can send email.
I would also like to see more connectors to which the trasport rule can be applied other than the "office 365 outlook" connector.

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

ID: c7d16c2c-fbad-7ed0-ea2d-fd8faee42054
Version Independent ID: 44d87949-1656-7360-e1e5-c8f45e029604
Content: Email exfiltration controls for connectors - Power Platform
Content Source: power-platform/admin/block-forwarded-email-from-power-automate.md
Service: power-platform
Sub-service: admin
GitHub Login: @paulliew
Microsoft Alias: paulliew

Thanks for your comment, @hajimemutsuki. This issue has been assigned to the doc author for review.

@hajimemutsuki
Sorry for the delayed response. For product-related feedback, visit https://aka.ms/ppac-ideas


Stop requiring a non-existent file

Fix a bug where new account requests fail because the handler is
requiring a non-existing institutions file

Fixes #133


text-show-0.6 can't find overlap.h

The latest version refers to a system header overlap.h. Which package provides that file? I have no clue where to find that?
http://hydra.cryp.to/build/526196/nixlog/2/raw shows the build log on our CI.

Whoops! overlap.h is a CPP hack that I use to work around OverlappingInstances being deprecated on GHC 7.10. I forgot to put this in the extra-source-files section of text-show.cabal, which is why it wasn't downloaded from Hackage. I'll update the Hackage version with this change. Please reopen this bug if you continue to experience problems.


Ghost as npm module, doesn't render anything

I am trying to use Ghost in my app. My app lives on http://localhost:9000 and I want the blog url to be http://localhost:9000/blog. In my ghost.config.js I put the development configuration. I put the content path on /blog
server.js
ghost({
  config: path.join(__dirname,'/config/ghost.config.js')
}).then(function (ghostServer) {
  app.use(ghostServer.config.paths.subdir, ghostServer.rootApp);
  ghostServer.start(app);

  require('./routes')(app);
}).catch(function (err) {
  console.log("Ghost blog error", err);
});

ghost.config.js
// # Ghost Configuration
// Setup your Ghost install for various [environments](http://support.ghost.org/config/#about-environments).

// Ghost runs in `development` mode by default. Full documentation can be found at http://support.ghost.org/config/

var path = require('path'),
  config;

config = {
  // ### Production
  // When running Ghost in the wild, use the production environment.
  // Configure your URL and mail settings here
  production: {
    url: 'http://my-ghost-blog.com',
    mail: {},
    database: {
      client: 'sqlite3',
      connection: {
        filename: path.join(__dirname, '/content/data/ghost.db')
      },
      debug: false
    },

    server: {
      host: '<IP_ADDRESS>',
      port: '2368'
    }
  },

  // ### Development **(default)**
  development: {
    // The url to use when providing links to the site, E.g. in RSS and email.
    // Change this to your Ghost blog's published URL.
    url: 'http://localhost:9000/blog',

    // Example mail config
    // Visit http://support.ghost.org/mail for instructions
    // ```
    //  mail: {
    //      transport: 'SMTP',
    //      options: {
    //          service: 'Mailgun',
    //          auth: {
    //              user: '', // mailgun username
    //              pass: ''  // mailgun password
    //          }
    //      }
    //  },
    // ```

    // #### Database
    // Ghost supports sqlite3 (default), MySQL & PostgreSQL
    database: {
      client: 'sqlite3',
      connection: {
        filename: path.join(__dirname, '../blog/data/ghost-dev.db')
      },
      debug: false
    },
    // #### Server
    // Can be host & port (default), or socket
    server: {
      // Host to be passed to node's `net.Server#listen()`
      host: '<IP_ADDRESS>',
      // Port to be passed to node's `net.Server#listen()`, for iisnode set this to `process.env.PORT`
      port: '9000'
    },
    // #### Paths
    // Specify where your content directory lives
    paths: {
      contentPath: path.join(__dirname, '../blog/')
    }
  },

  // **Developers only need to edit below here**

  // ### Testing
  // Used when developing Ghost to run tests and check the health of Ghost
  // Uses a different port number
  testing: {
    url: 'http://<IP_ADDRESS>:2369',
    database: {
      client: 'sqlite3',
      connection: {
        filename: path.join(__dirname, '/content/data/ghost-test.db')
      }
    },
    server: {
      host: '<IP_ADDRESS>',
      port: '2369'
    },
    logging: false
  },

  // ### Testing MySQL
  // Used by Travis - Automated testing run through GitHub
  'testing-mysql': {
    url: 'http://<IP_ADDRESS>:2369',
    database: {
      client: 'mysql',
      connection: {
        host     : '<IP_ADDRESS>',
        user     : 'root',
        password : '',
        database : 'ghost_testing',
        charset  : 'utf8'
      }
    },
    server: {
      host: '<IP_ADDRESS>',
      port: '2369'
    },
    logging: false
  },

  // ### Testing pg
  // Used by Travis - Automated testing run through GitHub
  'testing-pg': {
    url: 'http://<IP_ADDRESS>:2369',
    database: {
      client: 'pg',
      connection: {
        host     : '<IP_ADDRESS>',
        user     : 'postgres',
        password : '',
        database : 'ghost_testing',
        charset  : 'utf8'
      }
    },
    server: {
      host: '<IP_ADDRESS>',
      port: '2369'
    },
    logging: false
  }
};

module.exports = config;

When I start my parent express app, going to http://localhost:9000/blog only gives me 200 logging in the terminal but doesn't render anything to the front end.
Looking at the source code, I do a console.log("respond", view, result) on node_modules/ghost/core/server/controllers/frontend/index.js on before line res.render(view, result)
  // Do final data formatting and then render
                result.posts = posts;
                result = formatResponse.channel(result);
                setResponseContext(req, res);

                console.log('respond')
                res.render(view, result);

The view is logged as index and result is logged correctly with all the standard introduction post. However it doesn't give anything back to the browser.
So I suspect the res.render doesn't render correctly.  In my parent app I am using swig as the rendering engine, is that why? I am using npm 2.
Please help

When I console.log(blogApp.get('views')) this is what I get
/Users/christiansakai/Desktop/myapp/views
Because this is what my default app views settings are.
Is there a way to override this in Ghost? maybe by passing it to the config?

Apparently the maximum string length that I can call res.send is 1023, more than that it wont work. I have no idea why

Hi @christiansakai, sorry to hear you're having trouble. As mentioned in the readme & contrib guide, if you have questions about Ghost or need help, please head on over to slack - https://ghost.org/slack/ - and post in the #help channel. There you can get help & support from the Ghost community, which we're not able to do here in the GitHub bug tracker.

You must put 404 and 505 errors route after initialization of ghost.
Example: ghost().then(function(){
app.use(ghostServer.config.paths.subdir, ghostServer.rootApp);
ghostServer.start(app);
}).then(function(){
app.use(function(req, res, next) {
var err = new Error('Not Found');
err.status = 404;
next(err);
});
...
});

@iosifnicolae2 thanks for the reply. I know what the issue was. It boiled down to my parent app has a connect-livereload in its middleware.  Once I turned that off, everything works just fine.


Strange issue with a Thorium armor set - Terrarium armor

https://github.com/487666123/ImproveGame/assets/540857/300f065a-456c-45a4-b304-aa5a0353c2ca
Terraria <IP_ADDRESS>
Thorium <IP_ADDRESS>
Quality of Life <IP_ADDRESS>
Very strange bug - the set bonus of Thorium's Terrarium armor seems to swap repeatedly between set bonus focuses.
https://thoriummod.wiki.gg/wiki/Terrarium_armor
"Its set bonus grants a "focus" based on which class the player has the most (additive and multiplicative) bonus damage in".
The bug stops when Quality of Life is disabled.
Thank you.

Pretty strange. Can you send me your player file?

Pretty strange. Can you send me your player file?

There's been an update to Thorium mod and tModloader since I made this issue, and I can't seem to duplicate it anymore. I suppose it's fixed. Thank you anyway.


Update of botbuilder messages

The aim of this pull is to give a better control on the delay of the messages :

You can change the global delay on the botBuilder (from 0 to too much)
If you set a delay on a message, it will override the global delay

⚠ WARNING ⚠
This changes the behavior of the message, instead of adding the delay, it replaces it.
If a user put a 3 sec delay on his messages, instead of 5 second (old 2 sec const + 3 sec for this message), il will be a strict 3 secs.

Hello @Tellenn,
I'll try to rephrase your suggestion to make sure we understood it right, please correct us if we misunderstood.

add a global delay in botbuilder (default 2000ms, min 200ms)
have a message specific delay that can be set in Send Message (min 200ms) or can be left to default (i-e global delay)
any existing delay in Send Message set to 0 would be treated as if the user had chosen the default value (and thus the global delay)

If we understand it correctly, this solution can still be misleading to users so we strongly advise that the UI is straightforward.
We're looking forward to reading your next proposal.
Regards
To-Thi

Hi,
I've added a commit to add instruction on how to use the delay both on the info tab and the next to the input.
As for the 3 points you rephrased, you are right.
The global delay already exist (2000 ms) but is a constant.
Regards,
Kyllian


Allow PHP 7.X usage

This allows for PHP 7.X usage by replacing the Composer tilde operator (which allows >= 5.4 < 6.0.0) by the superior or equal one.

Thanks man!


Update fs2-core, fs2-io to 2.5.8

Updates

co.fs2:fs2-core
co.fs2:fs2-io

from 2.5.6 to 2.5.8.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.5.6).
You might want to review and update them manually.
CHANGELOG.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

labels: library-update, semver-patch, old-version-remains

Superseded by #986.


Fixed Bottle Blog and Included Example Posts

Trying to use bottle blog out of the box was not working at all. I'm not sure if the problem was due to changes in the way bottle works (after all, this project is 5-6 years old). I fixed the issues I found, and created a sample post and breve for a better out of the box experience.
Now when a user downloads bottle blog, they can immediately run the app without configuring anything and visit http://localhost:8080 to see a live working blog with sample entries.
Note: I did not test disqus support yet.

Great job ! I'll merge this one and add the required (but still missing 'cause of my laziness) list of requirements !
Thanks for your fixes!

Awesome, glad I could help, and thanks for adding me as a contributor! :)


Global: colorspace parse_colorspace_from_filepath simplification

Would this function make more sense when simplified/rewritten as just finding the first relevant color space from the list of color spaces of the config? It'd make it more versatile in use instead of it internally always needing to call get_ocio_config_colorspaces.
It feels to me that parsing from filenames like this might often be frequently done close after each other - e.g. parsing multiple files. Quering the color space files rules will be relatively slow since it requires accessing the OCIO config file each time.
colorspaces = get_ocio_config_colorspaces(config_path)
for filepath in filepaths:
    colorspace = parse_colorspace_from_filepath(filepath, colorspaces=colorspaces)

This also looks to me like a much more friendly API to the user with much less arguments to pass. Plus it'd be much faster and a simpler function code body.
As a general recommendation I'd try to see if there are other areas where functions could be simplified like this. Needing to pass project name, host, project settings to all functions where they end up just parsing an OCIO config path I'd say should be rewritten to just take the config path.
Originally posted by @BigRoy in https://github.com/ynput/OpenPype/pull/4195#discussion_r1070448372
[cuID:861m8ge8z]

This is also need to address following issue from @BigRoy comment here https://github.com/ynput/OpenPype/pull/4195/files/5409c3e3b895526ad7ed9c642f60b43deba4c522#r1080980267


Restricting Pytorch to version < 2.0

Pytorch 2.0 breaks some tests.
Restrict the version until we are sure what are the main differences between Pytorch 1 and 2 that affect the current tests.

@jonbarker68 @ns-rse
Changing the config to use torch<2.0 did the trick.
The tests are not failing now.
We may need to dig a little bit to understand where the differences originate to make our recipes reproducible.
As the tests showed, the results are not the same when seeding and using Pytorch 1 vs Pytorch 2


Speed changes value after adding save dc

I believe the issue is tied the line 109 because removing it allows the value to go back to original. Not sure how to fix at this time.

Moved saveDC under rest of elements and prepended. fixed this issue.
issue fixed in 73d65864e0cde06b4b56bc1340bf9d727091823d


VM_Clone --tag does not apply tags

Versions:

Version of knife-vsphere: v2.0.1

Platform Details

Version of vSphere: v6.0

Scenario:

When using the --tag option when cloning a VM, no tag is assigned. The tags being used are preexisting and can be assigned manually.
Please paste the command and output (run with -VV). Don't forget to X out any passwords or sensitive information:
'... --tags='Prod_Backup' ...'
Expected Result:
Assign tags to the cloned VM
Actual Result:
No tags are assigned

Hi Ben, can you try it with --tags (plural), and let me know, please?

whoops, in the report I used tag. I have tried both tags and tag and it does not seem to do anything for me. When I test other options such as --annotate, that works.

Can you double check you're not accidentally calling a version that predates support for --tags?
$ knife vsphere vm clone | grep tags
FATAL: You must specify a virtual machine name OR use --random-vmname
        --tags TAGS                  Comma separated list of tags to apply to the node

Double checked. I am using the current version and received the same output as you

hm, I will have to try it out here. I created a bunch of VMs earlier this week with tags and it worked fine, though I see I'm using 2.0.0.

Oh... When you say "no tags are assigned" to the VM, are you talking about tags in vsphere or in Chef? The --tags option tags the node within Chef during the bootstrapping. This gem currently can't tag a node within vsphere, since that's not accessible through the SOAP API.
I thought we have an issue open for this but I can't find it.

I am talking about tags in vsphere. Thank you for clarifying this! That makes much more sense it is for Chef.

Yea, sorry about the confusion. I'd happily add the functionality to the gem but so far it looks like a large task.


Missing Documentation

Hi,
It seems the documentation for the Angulartics website is not available - https://github.com/luisfarzati/angulartics/blob/master/src/angulartics-segmentio.js

@Findiglay you can find the new file at https://github.com/angulartics/angulartics-segment/blob/master/lib/angulartics-segment.js


WinLeave is not triggered when Vim is invoked with -O

Describe the bug
When Vim is invoked with the flag -d, -o or -O, during the startup, none of these events seem to be triggered:

BufLeave
BufWinLeave
WinLeave

This may give unexpected results.
To Reproduce
Run this shell command:
vim -Nu NONE --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}

The status line in the right window displays active.
Expected behavior
The status line in the right window displays NOT active, because:

for the window to exist, Vim had to focus it at least temporarily
it is not focused anymore, therefore it was left
when it was left, WinLeave was triggered
when WinLeave was triggered, the autocmd au BufWinLeave,WinLeave * setl stl=NOT\ active was run
when the autocmd was run, the value of the 'statusline' option – local to the window – was set to NOT active

Screenshots

In the screenshot, initially, the status line of the second window displays active, even though the latter is not focused. Only after focusing it and coming back to the first window, does its status line display NOT active.
Environment

Vim version:  8.1 Included patches: 1-2315
OS: Ubuntu 16.04.6 LTS
Terminal: XTerm(322)

Additional context
Initially, I found this issue after starting Vim with the -d flag; the status line in an unfocused window was set as if I was focusing it. So, the issue is not specific to -o; it probably applies to all flags which create windows (-o, -O, -d).

I've run these shell commands to find out which events are triggered during startup::
vim -Nu NONE -V8/tmp/log --cmd 'exe "au "..getcompletion("*", "event")->filter({_,v -> v !~# "Cmd$"})->join(",").." * \""' +qa -O /tmp/file{1..2}
vim +'v/^executing/d_' +'sil %s/executing \(\S*\).*/\1/' +x /tmp/log
vim /tmp/log

I found this list:

BufEnter
BufNewFile
BufWinEnter
WinEnter

WinLeave is not included, which probably explains the unexpected status line earlier.
BufWinLeave is included, but it's not triggered when Vim is starting up; it's triggered when Vim is quitting (here via :qa).

Contrary to WinLeave, TabLeave is triggered when invoking Vim with the -p flag:
vim -Nu NONE -V8/tmp/log --cmd 'exe "au "..getcompletion("*", "event")->filter({_,v -> v !~# "Cmd$"})->join(",").." * \""' +qa -p /tmp/file{1..2}
vim +'v/^executing/d_' +'sil %s/executing \(\S*\).*/\1/' +x /tmp/log
grep -i 'tableave' /tmp/log

As a workaround, I'm experimenting with this autocmd which visits all windows after VimEnter has been triggered and Vim was invoked with -d, -o or -O:
augroup trigger_leave_events_on_startup
    au!
    au VimEnter * call timer_start(0, {-> s:trigger_leave_events()})
augroup END
fu s:trigger_leave_events() abort
    if match(v:argv, '\C^-[doO]$') != -1)
        let curwin = win_getid()
        windo "
        call win_gotoid(curwin)
    endif
endfu

It seems to work, but it also looks awkward.

In the case of the status line option, I know there are other solutions, like:
set stl=%!GetStl(winnr())
fu GetStl(nr)
    return '%{SetStlFlag('..a:nr..')} %{w:is_active ? "active" : "NOT active"}'
endfu
fu SetStlFlag(nr)
    return get(extend(w:, {'is_active': (winnr() == a:nr)}), '', '')
endfu

Or even simpler:
set stl=%!GetStl()
fu GetStl()
    return g:statusline_winid == win_getid() ? 'active' : 'NOT active'
endfu

But the current issue is really about some events which are not triggered.

I'm not sure this is a bug; maybe the events are intentionally not triggered, because the user did not create the windows from a Vim session but from the shell command-line. But if that is the reason, then why are TabEnter and TabLeave triggered when Vim is invoked with -p?

This is done intentionally, during startup windows are created and buffers loaded in no specific order and only some Enter events are triggered (ones that are expected to initialize the window and/or buffer).
You can use the VimEnter event to initialize anything else.

Ah ok, thank you very much for the explanation.

You can use the VimEnter event to initialize anything else.

Although, not always right from VimEnter. If I use the same example as before, and try to fix it with a VimEnter autocmd, it doesn't work:
vim -Nu NONE --cmd 'au VimEnter * exe "windo \"" | 1wincmd w' --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The right status line still displays active.
To make it display NOT active, the command has to be delayed further with a timer:
vim -Nu NONE --cmd 'au VimEnter * call timer_start(0, {-> execute("windo \"") + execute("1wincmd w")})' --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}
                                  ^^^^^^^^^^^^^^^^

And only with a timer; I tried to delay with a one-shot autocmd listening to SafeState, but it doesn't work:
vim -Nu NONE --cmd 'au VimEnter * au SafeState * ++once exe "windo \"" | 1wincmd w' --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}
                                  ^^^^^^^^^^^^^^^^^^^^^

Ah ok, thank you very much for the explanation.

You can use the VimEnter event to initialize anything else.

Although, not always right from VimEnter. If I use the same example as before, and try to fix it with a VimEnter autocmd, it doesn't work:
vim -Nu NONE --cmd 'au VimEnter * exe "windo \"" | 1wincmd w' --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The right status line still displays active.
To make it display NOT active, the command has to be delayed further with a timer:
vim -Nu NONE --cmd 'au VimEnter * call timer_start(0, {-> execute("windo \"") + execute("1wincmd w")})' --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}
                                  ^^^^^^^^^^^^^^^^

And only with a timer; I tried to delay with a one-shot autocmd listening to SafeState, but it doesn't work:
vim -Nu NONE --cmd 'au VimEnter * au SafeState * ++once exe "windo \"" | 1wincmd w' --cmd 'au BufWinEnter,WinEnter * setl stl=active' --cmd 'au BufWinLeave,WinLeave * setl stl=NOT\ active' -O /tmp/file{1..2}
                                  ^^^^^^^^^^^^^^^^^^^^^

On the VimEnter autocommand you will want to loop through all the
windows, or at least do enough to make the statusline update the way you
want it.
--
From "know your smileys":
:-E	Has major dental problems
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


[Dubbo-7038]Returns the same reference if no filtering has occurred, to save memory.

What is the purpose of the change
[7038]https://github.com/apache/dubbo/issues/7038

Codecov Report

Merging #7039 (a62d8cf) into master (043da68) will decrease coverage by 1.98%.
The diff coverage is 31.38%.

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

I don't think keep original reference is a goog way.


Fix sporadic spinning bug in rs-motion.cpp

If the first gyro frame is received before the first accel frame the "first" flag is cleared and the timestamp value is incorrect. This separates the flags to be method specific.

Thanks @krazycoder2k :)


chore(ONYX-326): retire AREnableDoMoreOnArtsyRail

This PR resolves ONYX-326
Description
Retire AREnableDoMoreOnArtsyRail feature flag

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

retire AREnableDoMoreOnArtsyRail feature flag -daria

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (retire AREnableDoMoreOnArtsyRail feature flag -daria)

  Generated by :no_entry_sign: dangerJS against 422aa1bf2e2cf32779cc5a349ba9d2ddb50c597d


Add footer with styling

Add  a  footer to the web app
Add styling for visual enhancement

STATUS: APPROVED :green_circle: :clap:
Hi @H-Ramalan  🙋♂,
Your project is complete!  There is nothing else to say other than... Great job 👏

Congratulations! 🎉
TO HIGHLIGHT:

Correct use of class selectors ✔
Great UI design✔
Well-structured files :heavy_check_mark:

Now you can merge this branch and move on :1st_place_medal:
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

HAPPY CODING AND GOOD LUCK WITH THE COMING ONES :crossed_fingers: :four_leaf_clover:


model.init_word_vectors fails with file names containing spaces

Traceback (most recent call last):
File "", line 1, in
File "magpie/init.py", line 188, in init_word_vectors
self.train_word2vec(train_dir, vec_dim=vec_dim)
File "magpie/init.py", line 202, in train_word2vec
self.word2vec_model = train_word2vec(train_dir, vec_dim=vec_dim)
File "magpie/base/word2vec.py", line 131, in train_word2vec
window=context,
File "/Users/rxng/.pyenv/versions/2.7.13/lib/python2.7/site-packages/gensim/models/word2vec.py", line 469, in init
self.build_vocab(sentences, trim_rule=trim_rule)
File "/Users/rxng/.pyenv/versions/2.7.13/lib/python2.7/site-packages/gensim/models/word2vec.py", line 533, in build_vocab
self.scan_vocab(sentences, progress_per=progress_per, trim_rule=trim_rule)  # initial survey
File "/Users/rxng/.pyenv/versions/2.7.13/lib/python2.7/site-packages/gensim/models/word2vec.py", line 545, in scan_vocab
for sentence_no, sentence in enumerate(sentences):
File "magpie/base/word2vec.py", line 115, in iter
d = Document(doc_id, os.path.join(self.dirname, fname + '.txt'))
File "magpie/base/document.py", line 24, in init
raise ValueError("The file " + filepath + " doesn't exist")
ValueError: The file [redacted]/notedown/snippets/python - filter out empty values list.txt doesn't exist

Thanks for the heads up!
We don't do anything with the path in Magpie, just pass it directly to the python file handler. Handling file names, spaces and special signs is not the scope of Magpie, but should be provided by python file handler - I don't think we should do anything in this. Having spaces in the filepaths is usually a bad idea and you probably want to avoid that anyway.


setup sugar not showing up

Hi, I have a Vue 3 project and I'd like to auto convert the SFC to the Setup RFC, it worked before but right not the suggestion is not showing up at all. On 0.29.5

Check the box for Volar > Code Lens: Script Setup Tools settings to enable it.

The default value used to be enable, but is now set to disable.

vscode 1.75.1 can't show up convert to setup sugar, why?

Please check CHANGELOG.

https://github.com/vuejs/language-tools/blob/master/CHANGELOG.md#110-2023216

Removed script setup convert tool

tks! @yaegassy
it means if want to use script setup convert tool, only version 1.0.24 or low？

Any reason to have removed this tool? No details are available in the changelog.

This feature was previously implemented in a hacky way, which is no longer supported by the new architecture. However, this feature will be migrated to the 1.6 Rules API.
You can temporarily downgrade to version 1.0.24 to use it.


checkPermission() & requestPermission() will return different method while they shouldn't

🐛 Bug Report

the problems is when we ask for permission using requestPermission() method and result is LocationPermission.deniedForever  and then we call checkPermission method we will get a different result which is LocationPermission.denied.
Expected behavior
checkPermission() & requestPermission() methods should return the same result in above scenario
Reproduction steps
Configuration
Version: ^2.8.0
Platform:

[ ] :iphone: iOS (did not test it on iOS)
[ x] :robot: Android

This is expected behaviour, more details in my earlier comment: https://github.com/Baseflow/flutter-geolocator/issues/994#issuecomment-1055430478


Fix typographical error(s)

@Grokzen, I've corrected a typographical error in the documentation of the redis-py-cluster project. Specifically, I've changed carefull to careful. You should be able to merge this pull request automatically. However, if this was intentional or if you enjoy living in linguistic squalor, please let me know and create an issue on my home repository.

LGTM :]


Feeds no longer updating on version 2.17

Updated Newsboat via make / sudo make install last night, and it never updated any feeds after that (neither automatically nor manually). New feeds are detected just fine, though, and no error messages show on the log file (error-log is set), so I can't tell why it's happening. Reverting back to 2.16.1 fixes it.
newsboat -v output:
newsboat 2.17.0
System: Linux 5.3.1-arch1-1-ARCH (x86_64)
Compiler: g++ 9.1.0
ncurses: ncurses 6.1.20180127 (compiled with 6.1)
libcurl: libcurl/7.66.0 OpenSSL/1.1.1d zlib/1.2.11 libidn2/2.2.0 libpsl/0.21.0 (+libidn2/2.2.0) libssh2/1.8.2 nghttp2/1.39.2 (compiled with 7.66.0)
SQLite: 3.29.0 (compiled with 3.29.0)
libxml2: compiled with 2.9.9

Steps to reproduce:

Subscribe to a feed with a reliable schedule (podcasts and webcomics are good for this)
Wait until the feed is supposed to update (might take a few days, depending on their schedule)

Distro is Arch Linux, if it helps. Can't install the version on the repo because the package there is seemingly abandoned at this point.

Hmm, works fine here on Debian. Can you please do the following:

add https://hnrss.org/frontpage (Hacker News, high-traffic tech "news" site) to your urls file;
start Newsboat, update that feed, quit Newsboat;
wait about half an hour, to make sure there is a new post in the feed;
run newsboat --log-file=newsboat.log --log-level=6;
update that particular feed;
quit Newsboat.

Then attach "newsboat.log" to a new comment here. Note that the file will contain some potentially private info:

your config;
dumps of all the screens you show.

If that's not okay, use a different (one-line?) urls file and a minimal config (see --url-file and --config-file options).

Did as requested. Here's the file: newsboat.log
Just to confirm, the Hacker News feed didn't update on Newsboat despite the contents being different when I checked the link again.

Feeds are updating for me. Just bumped the Arch repo package to 2.17.

Installed the Arch version (thanks for updating, polyzen). Gonna have to wait a day to see if feeds update since the next reliable feed won't update until tonight (I should note that the Hacker News feed from above hasn't updated on my side ever since installing the Arch version), but regardless of whether or not they do this still leaves some important questions.
If the feeds don't update with the Arch version: Why don't they update on my side? Is there a new dependency I'm not aware of? Installing it from the repo didn't bring anything new with it, and version 2.16.1 still works.
If the feeds do update, however: Why don't they update for me when installed with sudo make install but they do with version 2.16.1 (compiled/installed in the same manner)? The only possible differences on my side would be the prefix (/usr/local/, which shouldn't be an issue since everything I install manually goes there) and the cpu (unless polyzen also uses a first-gen ryzen).

Intel(R) Core(TM) i5-3570K CPU

Thanks for the logs, @Aicaya! I think what's happening is Newsboat trims the feed because of max-items setting; but it does it from the wrong end, removing the newest items rather than oldest.
I was able to reproduce this with Hacker News feed (which contains 20 items) and a simple config:
max-items 10

Now I need to figure out how to reproduce this reliably, so I can find the cause and fix it. Can't wait 20 minutes between tests, in hopes HN posts something new.
This is a critical bug, so adding it to the "bugfix release" milestone.

 I'm going AFK now; I'll think tomorrow if reverting is the only option, or if we can still salvage something.
@Aicaya, temporary work-around is to comment out max-items in your config, or roll back to an earlier release.

Noted! I'll just comment it for now.

I ended up reverting the commit entirely. Below is my reasoning, mostly for future me who gets puzzled over "why did I revert this instead of doing X?"
We're sorting the feed reverse-chronologically because we want to trim oldest items if max-items is set. Ties are broken by item IDs; the larger the ID, the newer the item. This is sufficient to establish ordering even if there are a lot of items published in the same second.
But not everybody is using max-items, and I hoped I'd be able to do the sorting on the application side and only when the user really needs it. However, this is impossible, because we don't read item ID into RssItem; thus, we can't break ties when we sort RssFeed.
I guess we could append that ORDER BY clause only when max-items is set, but that doesn't sit well with me; manipulating SQL queries as string won't end well, and we're already doing it. Probably not worth the 6%, or however much that particular change saved.
So I took the easy way out, and reverted the whole patch.


Undefined index: started in PollDisplayModule.class.php

ElkArte 1.1
Current version: 1.0.4 bug fix update
MySQL version: 5.6.24
PHP: 5.6.8
Error message for Guest in looked post
Notice: Undefined index: started in C:\xampp\htdocs\elk11\sources\modules\Poll\PollDisplayModule.class.php on line 62

I cannot seem to reproduce this issue using the same environment with PHP 5.6 and Elk 1.1 (development branch). Viewing as Guest.
Please describe the steps to reproduce this bug.

The problems should be that $context['user']['started'] is defined at line 340, while the module is attached to the trigger at line 170... :blush:

Change [user][started] with (!empty[user][started])

Does my fix not cover all bases in that case?

Yep, the empty doesn't fix it, it just hides the error (and probably it makes impossible for OP to edit their polls) and neither the swapping does I feel, simple because 'started' is not defined at that point in time <= shame on me for bad testing.
The fix should be to move the block:
		// Did this user start the topic or not?
		$context['user']['started'] = $user_info['id'] == $topicinfo['id_member_started'] && !$user_info['is_guest'];
		$context['topic_starter_id'] = $topicinfo['id_member_started'];

before:
$this->_events->trigger('topicinfo', array('topicinfo' => &$topicinfo, 'includeUnapproved' => $includeUnapproved));

@wintstar no problem, reopened for you. ;)

Okay, I see. The guest check is included in that variable, so we just need to have it set before calling upon in, even as a guest. I'll see if I can make it work.

I think that should do it.

@emanuele45 Thanks for reopened. I think too. No Error message is displayed.


vscode: press ; to respect semicolons

bors r+
But we probably should setup eslint. Also, I’d personally be in favor of removing all semicolons

No! Never remove semicolons!

@matklad , are you kidding?


feat(mdc-toggle-button): New toggle button component in material-experimental

Add new component as wrapper to new mdc segmented-button component.

Closing this draft since we do not currently have MDC support for the button toggle. Hopefully soon we will be able to implement this


Black Screen When Connected (No Video)

Every time i connect i get a black screen when i try to stream. Tried to fix with multiple restarts, and Beta version 4,5,6 and version 1.2 no luck.

Turns out SD connect to my guest network for some reason. After that got it fixed.


Allow text formatting by text selection in the preview/WYSIWYG pane

Currently we can select text in the source code editor pane (on the left) and click the formatting icons on the toolbar to modify that text selection.
But the same cannot be done by selecting text in the preview/WYSIWYG pane (on the right).
It would be awesome if it could be done in both panes.
Ideally you would always be able to format selected text in both panes at all times.  i.e. Rather than there being a setting to only use one or the other.
Additionally to this, it would be amazing if we could both type and format in both panes.  But just formatting would be a great start.
All of the above, with a button to hide either of the panes would make stackedit suitable for every scenario/site/technical-ability-level imaginable:

Programmers like us can continue to use it as is
Some technical people might not feel they even need the preview pane
Non-technical people can ignore the source code editor pane, or hide it
People in the middle can swap back and forth in how they use it

I'm afraid it will be difficult to add editing capabilities without an actual editor. Though in the v5 you can now create discussions and add comments with both the editor and the preview pane.


Releasing app 0.7

What does this PR do?
releasing App 0.7
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda

@nohalon @edenlightning mind checking again :otter:

@Borda Could you include all the commits from the master? I don't see those commits: https://github.com/Lightning-AI/lightning/pull/14995 and https://github.com/Lightning-AI/lightning/pull/15187

Could you include all the commits from the master? I don't see those commits: #14995 and #15187

not really; this is specifically requested for secrets, so your components will follow in a few days, or we can make RC from master

ok, this shall be rebase & merge but never mind :otter:


Compatibility with Python 3.6

Hey, your print statements don't have parentheses around them, were you planning on providing support for python 3x?

Probably not, you can make it support if you want :)


C API: ireeCompilerSourceWrapBuffer Undefined Behavior

What happened?
I am working on a Rust binding of libIREECompiler, and have encountered an issue with ireeCompilerSourceWrapBuffer().
Sorry for not providing the C implementation; I am not really familiar with CMake, but I hope the following Rust code is easy to comprehend.
fn test() {
        let compiler = Compiler::new().unwrap();
        let session = compiler.create_session();
        let mut invocation = session.create_invocation();
        invocation.set_verify_ir(true);
        let source = Source::wrap_buffer(
            &session,
            b"module @arithmetic {}\0",
            22,
            true
        ).unwrap();
        invocation.parse_source(source).unwrap();
    }

This works as expected.
fn test() {
        let compiler = Compiler::new().unwrap();
        let session = compiler.create_session();
        let mut invocation = session.create_invocation();
        invocation.set_verify_ir(true);
        let source = Source::wrap_buffer(
            &session,
            b"module @arithmetic {}\0",
            21,
            false
        ).unwrap();
        invocation.parse_source(source).unwrap();
    }

This also works as expected.
However, when I run the following code:
fn test_noooo() {
        let compiler = Compiler::new().unwrap();
        let session = compiler.create_session();
        let mut invocation = session.create_invocation();
        invocation.set_verify_ir(true);
        let source = Source::wrap_buffer(
            &session,
            b"module @arithmetic {}",
            21,
            false
        ).unwrap();
        invocation.parse_source(source).unwrap();
    }

I get this error:
Error: buffer:1:22: custom op 'get_pluginsget_registered_hal_target_backendsget_revisioninit_invocationsession_set_get_flagssetup_global_clsource_from_buffersource_from_invalid_buffertest_compilertest_get_api_versiontest_nooootest_session' is unknown (tried 'builtin.get_pluginsget_registered_hal_target_backendsget_revisioninit_invocationsession_set_get_flagssetup_global_clsource_from_buffersource_from_invalid_buffertest_compilertest_get_api_versiontest_nooootest_session' as well)

Upon Inspection, memory region of buffer[0:100] reads to module @arithmetic {}event tests/compiler.rs:164get_pluginsget_registered_hal_target_backendsget_revisioninit_invocations, which means that the compiler is reading regions that it should not be reading.
Steps to reproduce your issue

Go to '...'
Click on '....'
Scroll down to '....'
See error

What component(s) does this issue relate to?
Compiler
Version information
2b9ade5
Additional context
No response

For clarity, does 'binary buffer' mean MLIR bytecode?

Yep!

In addition, it would be incredibly helpful if the wrapped str buffer didn't necessarily have to be null-terminated. Languages like Rust default to non-null-terminated strings, so passing strings around often requires cloning the entire string.
Thank you for your thoughtful response!

Agreed! I think this limitation comes from LLVM/MLIR but @stellaraccident may know if there's a way we could work around it. Retrofitting codebases assuming null terminated strings everywhere instead of views is tricky ;(

that's what I was afraid of ;(


Should the DotNet.ReproducibleBuilds package be an SDK instead?

The DotNet.ReproducibleBuilds.Isolated package is an SDK, DotNet.ReproducibleBuilds is not. Should they both be?
The specific issue that prompts this issue is that I see devs using properties set by RB like $(ContinuousIntegrationBuild) in Directory.Build.props, but the D.B.props file is imported first per https://github.com/dotnet/msbuild/pull/994#issuecomment-245946950.
Since RB is essentially an extension to the SDK, this feels like a defensible change to me. Open question about how much of a breaking change this would be. We likely could leave a stub .props file behind that emits a warning to explain the upgrade path.

I think if we did fix this we'd need to do a staged migration - keeping the Sdk.props and Sdk.targets but delegating those to import props and targets from the 'normal' build/DotNet.ReproducibleBuilds.[props|targets] locations so that users can reference it as an SDK or a PackageReference.
Then we could break after enough time for folks to migrate.

Yes, I think SDK instead is better.


New tool suggestion

Hi there,
I would like to suggest adding Touchlead to the list. Touchlead is an all-in-one marketing solution designed to help businesses acquire more customers and grow their business. It provides a streamlined way to manage leads, automate workflows, and schedule social posts, making it a valuable tool for any type of business.
You can find more information about Touchlead at https://touchlead.app/.
Thank you for considering this suggestion!
Best regards,

Hi, send a PR


Update entra app permission add to support adding permissions to existing resources

We should consider updating entra app permission add to allow adding permissions to existing resources. Right now, if the app already has some permissions for the same resource, running the command with another permission fails with an error saying that you're trying to add a duplicate record. This is caused by the fact, that permissions are stored in records where the key is composed of the app and resource ID. All permissions for the combination are listed as a space-separated string.

Seems logical to me indeed 👍

Thanks for reporting this @waldekmastykarz! I've added it to the epic.

Anyway, we'll need to retrieve existing permissions and patch them. I find it strange that we missed this in the first place 😅

awesome suggestion. lets do that

Anyway, we'll need to retrieve existing permissions and patch them. I find it strange that we missed this in the first place 😅

It's not super obvious until you need to update an app reg. Anyways, it'll be great to have this solved 🚀

To clarify, adding the permission itself works just fine. It's the combination of adding the permission and granting admin consent that fails


Unable to fetch list of bridges due to non-existing bridges showing up in list

I'm unable to get a list of bridges using the API. This is because my https://discovery.meethue.com/ is returning three bridges even though I only have one. This is an issue by itself, but I should at least be able to get a list.
The reason it's not working is because the discovery API us not only returning a list, but trying to get its description:
https://github.com/peter-murray/node-hue-api/blob/80a6e4490c4955da9e8d934981a073f5efb65f76/lib/api/discovery/index.js#L37-L43
The getBridgeConfig method resolves each bridge to get some additional info. Because my bridge list returns 3 even though I only own one, the other IP adresses doesn't response and Promise.all is rejected.
Bridge list:
[
   {
      "id":"001788fffeb02653",
      "internalipaddress":"<IP_ADDRESS>"
   },
   {
      "id":"001788fffe6b9a6a",
      "internalipaddress":"<IP_ADDRESS>"
   },
   {
      "id":"001788fffe2b03d8",
      "internalipaddress":"<IP_ADDRESS>"
   }
]

Those values returned are all bridges registered against your use account at, https://account.meethue.com/bridge
If you sign into the above url, then you can remove the invalid bridges.
Currently it expects these bridges to all be valid, as these are all bridges that have been bound to your portal account at some point. The code needs to validate these to ensure it is a hue bridge before allowing you to proceed.
I will look into some potential error handling here, but I need to think about what to do with errors on any bridges, as it will not be valid to return them.

Is it necessary to get additional info when listing bridges? If yes, maybe it's a posibility to give the bridge 1-2 seconds to respond and simply return the list of bridges with a state as not responding?

It is, as the info retrieved is used to validate the self signed ssl certificate as well as proving that the reported values are real bridges in the client network.
I might consider returning invalid/offline bridges, but I need to properly think about how I want to do that, if at all. This discovery service is not heavily document by signify, but I need to only return real and valid bridges to the user.

In your case though, if you have only one bridge, then you do need to clear out the invalid bridges from your account, and this will all work as intended.

Yes, I see. Of course. The problem is I'm thinking about shipping node-hue-api in an Electron app. So I don't really have control over other users bridges. My only option for now is to do the discovery myself, maybe use node-hue-api to validate the certificate? Don't know if these mehods are public.
Maybe even ask the user to fill in the IP manually is better at this point :-)
Side note: how is it possible to delete old bridges? I don't see an option for it. Only this:

2 bridges were recently seen on your network. If you'd like to link a different bridge to your account, unlink your bridge at the bottom of this page and follow the instructions when adding a bridge.

And an option to unlink my current (and working) bridge.

You don’t have to run discovery, the api will support you specifying the IP address of a valid bridge, it will perform the necessary validations of the self signed ssl cert upon connection.
The discovery code perform validations on the proposed bridges, so that any results it returns are in fact real bridges.
I have two bridges myself, one for my network, the other for testing the api, but these are always resolvable and valid. In your case, the discovery portal is reporting three bridges for your network/account, I have never seen, or had reported previously, invalid bridges being reported by users.
I am looking at various options currently, but depending upon the solution, which I cannot fully test as I cannot manifest your use case, this may result in a major release as it will likely be in a change of behaviour.

I have released a test candidate for you into npm registry, 5.0.0-alpha.1 or next tag for you to try, it has error handling around the resolving of the bridge results.
Can you please let me know if that works, as I cannot manifest your failure condition easily so this has not been tested properly yet.

Thanks, will test it today!

It does seem to work. Even though the request takes about 10 sec per non-responding bridge:
[
   {
      "name":"Philips hue",
      "ipaddress":"<IP_ADDRESS>",
      "modelid":"BSB002",
      "swversion":"1937045000"
   },
   {
      "error":{
         "message":"Problem connecting to bridge '<IP_ADDRESS>': connect ETIMEDOUT <IP_ADDRESS>:80",
         "description":"Failed to connect and load configuration from the bridge at ip address undefined",
         "ipaddress":"<IP_ADDRESS>",
         "id":"001788fffe2b0335"
      }
   },
   {
      "error":{
         "message":"Problem connecting to bridge '<IP_ADDRESS>': connect ETIMEDOUT <IP_ADDRESS>:80",
         "description":"Failed to connect and load configuration from the bridge at ip address undefined",
         "ipaddress":"<IP_ADDRESS>",
         "id":"001788fffeb02635"
      }
   }
]

It will be using the default timeout of 10 seconds on the request, I can tweak that down, but 5 seconds would be the minimum I would do, as the bridges can take some time to respond on this end point when under load.
In your use case though it would be extremely valuable to me to find out how you have these other bridges in your account if you have only ever had one. I would recommend raising a support ticket with signify to find out what is going on here. I have never had anything like this reported before, and I have two bridges that come and go from the portal as expected. There could be a bug, or some old data issues on the signify side.

It seems that my issue is a cornercase. But at the same time it should be handled somehow. I think the current solution is a good compromise. We can leave this issue open and see if other people have any suggestions maybe?

If you really only have one bridge as you stated, then I need to understand how you ended up with three, all with unique Mac addresses registered against your account. That is the only real way to finalise a solution to this problem. I have never seen this, and it is either a bug, or there is something else going on here that to support I would need to know about.
Please, if you really only have one bridge, then raise a support ticket with signify on this so that it can be properly diagnosed, resolved and then the details fed back to this ticket so it can be properly resolved for all users in the future.
In the meantime, I will add a timeout to the nupnp function so it can shortened and then prepare this change for an upcoming 5.x release, as I have some more API changes that I need to make before I can cut that release.

I came across this discussion thread today https://developers.meethue.com/forum/t/n-upnp-shows-me-more-bridges-than-my-own/6073
It seems to think that the n-UPnP discover operates off of an external IP Address that the hue systems see. So for example if you are in situation where multiple homes are behind a common IP Address the service can report extra bridges that you do not own.

Hello,
I seem to be running into the same issue as @Kordonme. I too only have a single bridge on my network, but discovery.meethue.com reports 3. I too don't have an option to remove non-existing bridges. The few apps I have installed all report a single bridge, but I'd imagine they filter non-reachable bridges.
I'm getting the following error @ 4.0.9:
ApiError: Problem connecting to bridge '<IP_ADDRESS>': connect EHOSTUNREACH <IP_ADDRESS>:80
Upon updating the repository to next I too get the following response:
[   {     "name": "Philips hue",     "ipaddress": "<IP_ADDRESS>",     "modelid": "BSB002",     "swversion": "1940094000"   },   {     "error": {       "message": "Problem connecting to bridge '<IP_ADDRESS>': connect EHOSTUNREACH <IP_ADDRESS>:80",       "description": "Failed to connect and load configuration from the bridge at ip address undefined",       "ipaddress": "<IP_ADDRESS>",       "id": "001788fffe28e505"     }   },   {     "error": {       "message": "Problem connecting to bridge '<IP_ADDRESS>': connect EHOSTUNREACH <IP_ADDRESS>:80",       "description": "Failed to connect and load configuration from the bridge at ip address undefined",       "ipaddress": "<IP_ADDRESS>",       "id": "001788fffe7fa6ee"     }   } ]
My requests seem to be quite a bit faster though, with responses between 500ms - 3.5s

So <IP_ADDRESS> is the functional bridge there in that response, the other two are not reachable.
I have been working on some other changes that will be incorporated in to the next release as due to this being a mjaor change I can modify some of the APIs.
To fix this problem, I am going to have to bump the major, as the current 4.x releases are not set up to cater for the multiple return result with the errors.
I am also on the fence here are to how to present this data via the API. Your feedback here would be appreciated. Are you only interested in the successful bridges, or do you potentially want to see all and the details as to why a possible match "failed". I can think of scenarios where by a real bridge is returned, but does not respond in time (due to load or other networking related issues) so it is useful to see that it was detected but had a failure.


Metadata system lazor

Codecov Report

Merging #68 (935a6bf) into main (9fe71ce) will increase coverage by 18.03%.
The diff coverage is 74.10%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main      #68       +/-   ##
===========================================
+ Coverage   51.18%   69.22%   +18.03%
===========================================
  Files          20       23        +3
  Lines         715      809       +94
===========================================
+ Hits          366      560      +194
+ Misses        349      249      -100

Impacted Files
Coverage Δ

incipyt/commands.py
56.75% <0.00%> (-2.71%)
:arrow_down:

incipyt/tools/git.py
34.88% <0.00%> (+34.88%)
:arrow_up:

incipyt/tools/pep517/flit.py
45.45% <0.00%> (+45.45%)
:arrow_up:

incipyt/tools/pep517/hatch.py
45.45% <0.00%> (+45.45%)
:arrow_up:

incipyt/tools/pep517/pdm.py
45.45% <0.00%> (+45.45%)
:arrow_up:

incipyt/_internal/utils.py
80.00% <33.33%> (-13.03%)
:arrow_down:

incipyt/tools/poetry.py
27.50% <33.33%> (+27.50%)
:arrow_up:

incipyt/tools/venv.py
54.54% <33.33%> (+54.54%)
:arrow_up:

incipyt/tools/pep517/base.py
32.35% <40.00%> (+32.35%)
:arrow_up:

incipyt/__main__.py
44.73% <53.19%> (+44.73%)
:arrow_up:

... and 5 more

... and 7 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


"Cite Us" link leads to 404

As per title. I've tried to find where the new page has gone, but no luck.

Thanks @marco-brandizi. This changed in the most recent website update. Instead of having a designated page, we now show a banner which is copyable on all the Cases/News/Publication pages. The link can be changed to: https://knetminer.com/category/cases/

@Arnedeklerk better remove the link from 5.7 header?

@marco-brandizi why do you think so? I've updated it to work now - although there isn't a designated page anymore it does still clearly show the citation there and since we've seen users cite the old paper on occasion, I think it's good to keep it findable...

@Arnedeklerk got it wrong, sorry. Changing it WP side is better than on KnetMiner code. I'm closing this again.


couldn't understand kern.osversion `15.2.0'

Building native extensions.  This could take a while...
ERROR:  Error installing cityhash:
    ERROR: Failed to build gem native extension.

    /Users/row/.rvm/rubies/ruby-2.2.3/bin/ruby -r ./siteconf20151215-5759-1xlapqo.rb extconf.rb
creating Makefile

make "DESTDIR=" clean

make "DESTDIR="
compiling city.cc
couldn't understand kern.osversion `15.2.0'
city.cc:1: error: bad value (native) for -march= switch
city.cc:1: error: bad value (native) for -mtune= switch
make: *** [city.o] Error 1

make failed, exit code 2

Gem files will remain installed in<EMAIL_ADDRESS>for inspection.
Results logged to<EMAIL_ADDRESS>

@RobWu hey, thanks for reporting it! Yeah, it's been 2 years, sorry. I just released 0.9.0 version that should compile without CRC methods. Please give it a try!


Web - Profile picture is not updated on public room even after refreshing the page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging dot on web chrome and login with User A
Create a new room in Public mode
Send a message in room - very Imp step
Click on room header and copy the share room URL
Open an incognito window and paste the room URL as an anonymous user
Now Go back to User A and change the profile picture of User A
On incognito window, notice the profile picture is not updated. Refresh the page and notice that even after refreshing the page, profile picture is not updated.

Expected Result:
Profile picture should be updated on public room
Actual Result:
Profile picture is not updated on public room even after refreshing the page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:  1.3.35-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/aeb958f0-ce48-4303-a2f5-1cee76fbb40c
https://github.com/Expensify/App/assets/93399543/44a5587a-753b-4d29-a708-1a737bab0143
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688289180818999
View all open jobs on GitHub

This is ultimately the same issue as https://github.com/Expensify/App/issues/21488.
Going to go ahead and close as a dupe.


_validate_get_device() could be nice to be called also in model.sample() and model.predict()

Hi!
I have noticed that _validate_get_device() is called only on model.fit() but it could be nice if is called also in model.sample() and model.predict() so that there is no need to pass device argument in the methods when no cuda is available to use CPU. model.fit() already calls it and there is no need to pass device, it automatically detects what to use.

@echatzikyriakidis, thanks for noting this! I pushed a patch to use _validate_get_device in the other interfaces. 😀

Thank you!


Feat  add filter by pr status

add filter by PR status, please review :dagger:

LGTM!
thank you for your first contributions 🎉


i7-1355U support (Thinkpad T14s Gen4)

Is it possible to add support for the following CPU model (Thinkpad T14s Gen4)?
model name      : 13th Gen Intel(R) Core(TM) i7-1355U
cpu family      : 6
model           : 186
stepping        : 3

In the meantime, testing this...
❯ git diff
diff --git a/throttled.py b/throttled.py
index b9f4c41..0e2b5f7 100755
--- a/throttled.py
+++ b/throttled.py
@@ -170,6 +170,7 @@ supported_cpus = {
     (6, 165, 5): 'Comet Lake-S',
     (6, 166, 0): 'CometLake',
     (6, 167, 1): 'RocketLake',
+    (6, 186, 3): 'RaptorLake-U',
 }

 TESTMSR = False

There are >1 people curious about this specific CPU :) Any update?

I did a test myself. With this package the upper frequency went up from 1.8GHz to 2.2GHz. A welcome improvement, but not done.

Output of throttled.py --debug:
debug.txt

Unfortunately, that didn't work. It went down to 1.7GHz after a couple of days of uptime and stayed at that.

I didn't spent much time on this, but for me is working as expected combined with tuning of the platform profile (i.e. /sys/firmware/acpi/platform_profile) and the CPU scaling governor (i.e. /sys/devices/system/cpu/cpu*/cpufreq/scaling_governor). Sharing just in case it helps.

@carlosabalde those didn't help (both were set to "performance").
HOWEVER removing tlp fixed the cpufreq issues on my machine! I had tlp enabled for all my laptops since forever for battery management ("stop charging at 80%"), not any more!

Hello folks, I have this CPU and the https://github.com/erpalma/throttled/issues/346#issuecomment-1911006183 made throttled happy to load and work with it. The system seems to perform well.


question: do I need to provide defaults when overriding exclude list?

I'm doing the following, is this necessary or can I just add the one filetype and it'll get appended to the defaults?
Thanks!
    use { "zakharykaplan/nvim-retrail",
      config = function()
        require("retrail").setup({
          filetype = {
            exclude = {
              "markdown",
              -- following are the defaults I've copied over in case they get overwritten
              "",
              "alpha",
              "checkhealth",
              "diff",
              "help",
              "lspinfo",
              "man",
              "mason",
              "TelescopePrompt",
              "Trouble",
              "WhichKey",
            }
          }
        })
      end
    }

Great question!
When designing the configuration I basically had to make this exact design choice. I ended up opting for the user's config to override the defaults rather than append, my reasoning being that otherwise there'd be no (obvious) way to unset defaults. (Although, as an aside excludes have priority over includes, so that could have be used to partially override defaults.)
TL;DR is that how you currently have it set up is correct.

Awesome, thanks for the detailed answer 👍🏻
I feel like filetype.append would be nice to have because I generally trust a plugin author has done the hard work of configuring sensible defaults and in cases like this plugin, I only need to 'tweak' one thing. So I'm a big 👍🏻 on supporting this.
Thanks again!

For anyone else wanting to extend the include/excludes (ie: not overwrite the existing defaults), I was able to use this w/ lazy.nvim:
{
  -- Highlights trailing spaces + deletes on write
  "kaplanz/retrail.nvim",
  opts = function(_, opts)
    local opts = require("retrail.config").parse(opts)

    opts.enabled.filetype.dashboard = false -- `include` and `exclude` are resolved into `enabled`
    vim.list_extend(opts.filetype.exclude, { "dashboard" }) -- not used after retrail.config.parse(...), but being cautious
    return opts
  end,
}

Definitely not "simpler", but at least any additions to the excludes upstream will be added when you updated the plugin. 😅

Great comment.
As an aside after working through this, do you have any opinions on how a non-breaking config change to update (without overriding) the default include and exclude lists should look?
I never got around to adding filetype.append since I didn't feel I could get the design right while staying simple and discoverable. IIRC I played around with the idea of having a separate option override which defaults to true and when false cases both lists to append rather than override, but was worried that wouldn't be so obvious how it's used.


Automated cherry pick of #11205: feat(baremetal): support management of none BMC host

Cherry pick of #11205 on release/3.7.
#11205: [WIP] feat(baremetal): support management of none BMC host

/approve


AM I the only one

Am I the only one that can not get this to work. I have followed all of the steps outlined several times and simply can not get the project to work for me. I have deleted everything and started over several times and have even tried at my house in case there was something happening with my firewall at work but nothing. All that I get when I build the project is this:

I really only want to make some simple changes to the visuals and yet I am failing.
Please someone tell me what I am doing wrong.

try  to use the google chrome or firefox while you are running your project, i think it will solve your problem

Thanks for the reply, I was using Chrome and checked Firefox just to be sure. Same result.

It looks like it hasn't pulled in the css file. Do you see any errors in the browser devtools console?

Hi, Justin. By all signs you have forgotten to install gulp.
When in root folder of PowerBI Try to run this command first : npm install -g gulp
Then npm install
It should solve your problem.

Thanks guys, turns out it was a network issue. It is working now.


support config model path in extra_model_paths.yaml

use model path of extra_model_paths.yaml first,
use MODELS_DIR + model, if no model path config in extra_model_paths.yaml
Issue
https://github.com/WASasquatch/was-node-suite-comfyui/issues/345

I'll test this as soon as I can. Thank you.

BLIP has undergone a complete refactor https://github.com/WASasquatch/was-node-suite-comfyui/commit/6e01ba1a8bc260af6a47f45d25a5c568bebe4ccb


step05 has two jl files

steps 01-04 each have just one stepNN.jl file but step05 has two.  This inconsistency necessitate what seems like avoidable confusion in the condor submit file and shell scripts.   Are both these necessary?  Is there another solution?

I propose that we just move the smooth quadratic model into a new step (step06).

For the next release...

Since this is a super simple change, I went ahead and moved the quadratic model into step06.   Hope this is ok.   I think it makes the project easier to understand and it follows the approach of  later steps building upon earlier steps.


[Bug]:  Polyfill:"Usage",but the packaged product es5 syntax has a scope problem, causing my variables to be contaminated.

System Info
System:
OS: macOS 14.5
CPU: (8) x64 Apple M1
Memory: 30.48 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 14.17.0 - ~/.nvm/versions/node/v14.17.0/bin/node
Yarn: 1.22.22 - /opt/homebrew/bin/yarn
npm: 6.14.13 - ~/.nvm/versions/node/v14.17.0/bin/npm
Browsers:
Chrome: 131.0.6778.87
Safari: 17.5
Details
I used polyfill:'usage' to configure compatibility with my browser, but the packaged product es5 syntax has a scope problem, causing my variables to be contaminated. In my code, show_type in 3 should be used instead of show_type in 1（figure 1 is the compiled code ）. How do I fix this, Or is this a bug in rsbuild?

Reproduce link
https://codesandbox.io/p/github/rspack-contrib/rsbuild-codesandbox-example/csb-zktgdv/draft/great-flower?file=%2Fsrc%2FApp.jsx%3A11%2C54
Reproduce Steps
npm run build
checkout index.[hash].js , search "show_type" the result will be appear！！

Seems like I have the same problem

The codesandbox link is invalid, can you have a look at it?

The codesandbox link is invalid, can you have a look at it?

I checked. This connection is accessible. Would you please check if there is anything wrong with your network?

The codesandbox link is invalid, can you have a look at it?

Or you can convert the following code with polyfill:usage
const composeIndicator = (keynode, logList) => {   if (!keynode || keynode.length < 1) return [];   for (let i in keynode) {     const { key_list, relation, name, show_type } = keynode[i] || {};     if (!key_list || key_list.length < 1) continue;     const pvArr = [];     for (let j in key_list) {       const { abbr, show_type, id, key_id } = key_list[j] || {};       const { pv, uv } =         logList.find((i) => {           if (id && i.id) {             return i.id === id;           } else if (key_id && i.key_id) {             return i.key_id === key_id;           }         }) || {};       pvArr.push({         pv,         uv,         abbr,         show_type       });     }     const dataList = [];     for (let l in pvArr) {       const { pv, abbr, show_type, uv } = pvArr[l] || {};       if (Array.isArray(pv)) {         for (let k in pv) {           if (!dataList[k]) {             if (relation.indexOf(abbr) > -1) {               let copyRelation = relation;               const data = show_type === 1 ? pv[k] : uv[k];                const regx = new RegExp(${abbr}, 'g');               copyRelation = copyRelation.replace(regx, data);               dataList.push(copyRelation);             }           } else {             if (dataList[k].indexOf(abbr) > -1) {               let copyRelation = dataList[k];               const data = show_type === 1 ? pv[k] : uv[k];                const regx = new RegExp(${abbr}, 'g');               copyRelation = copyRelation.replace(regx, data);               dataList[k] = copyRelation;             }           }         }       }     }     keynode[i].pv = dataList.map((i) => {       const calculate = eval(i);       const val =         show_type === 1           ? calculate === Infinity             ? 0             : Math.round(calculate)           : calculate === Infinity           ? 0           : Number((calculate * 100).toFixed(2));       return isNaN(val) ? 0 : val;     });      keynode[i].clicktag = name;   }   return keynode; };

You can provide a public GitHub repo instead.

Here, this is Github Repo: https://github.com/PropersonCyber/rebuild-demo.git
run yarn build and checkout the index.[hash].js  you will be able to replicate my problem.

Can you try adding output.minify: false to see if this is an issue of SWC minimizer?

Can you try adding output.minify: false to see if this is an issue of SWC minimizer?

I've tried it before, and it does have problems, how can I avoid them when compatible with lower versions

I think the current compilation result is correct:

Which one do you think was not as expected?

I think the current compilation result is correct:

Which one do you think was not as expected?

Yes, the relationship of variable correspondence you provided is correct. However, it appears there is an issue with variable contamination. The show_type variable indicated by the red arrow seems to be affected by the show_type variable pointed to by the green arrow. Could this be related to variable hoisting with var? I had assumed that instead of directly converting const and let to var, considerations would be made for scope. Interestingly, this issue doesn't occur when I use polyfill: 'none', as I'm utilizing let, which is restricted by block-level scope. So I think is there something wrong with the strategy.

@GiveMe-A-Name Can you help to confirm if this is a bug in the SWC transformation? ❤

I sure it's related to variable hoisting with var.
See a simple example:
var keynode = {
  show_type: 1,
  key_list: [
    { show_type: 'a', },
    { show_type: 'b' },
    { show_type: "c" },
  ]
}

var { show_type, key_list } = keynode;

console.log('show_type in loop output:', show_type);

for (var key in key_list) {
  var { show_type } = key_list[key];

  console.log("show_type in loop1:", show_type);

}

[1, 2, 3].forEach((i) => {
  console.log("show_type in loop2:", show_type);
})

// show_type in loop output: 1
// show_type in loop1: a
// show_type in loop1: b
// show_type in loop1: c
// show_type in loop2: c
// show_type in loop2: c
// show_type in loop2: c

It seems swc do not care about variable hoisting in this case.
But after babel transform, the code would be like:
"use strict";

var keynode = {
  show_type: 1,
  key_list: [{
    show_type: 'a'
  }, {
    show_type: 'b'
  }, {
    show_type: "c"
  }]
};
var show_type = keynode.show_type,
  key_list = keynode.key_list;
console.log('show_type in loop output:', show_type);
for (var key in key_list) {
  var _show_type = key_list[key].show_type;
  console.log("show_type in loop1:", _show_type);
}
[1, 2, 3].forEach(function (i) {
  console.log("show_type in loop2:", show_type);
});

It's correctly.
In short, it is a bug in swc.

I sure it's related to variable hoisting with var. See a simple example:
var keynode = {
show_type: 1,
key_list: [
{ show_type: 'a', },
{ show_type: 'b' },
{ show_type: "c" },
]
}
var { show_type, key_list } = keynode;
console.log('show_type in loop output:', show_type);
for (var key in key_list) {
var { show_type } = key_list[key];
console.log("show_type in loop1:", show_type);
}
[1, 2, 3].forEach((i) => {
console.log("show_type in loop2:", show_type);
})
// show_type in loop output: 1
// show_type in loop1: a
// show_type in loop1: b
// show_type in loop1: c
// show_type in loop2: c
// show_type in loop2: c
// show_type in loop2: c
It seems swc do not care about variable hoisting in this case.
The code would be transformed by babel, like bellow:
var composeIndicator = function composeIndicator(keynode, logList) {
/// ...
var dataList = [];
for (var l in pvArr) {
var _ref2 = pvArr[l] || {},
pv = _ref2.pv,
abbr = _ref2.abbr,
// using a other name to avoid variable hoisting
_show_type = _ref2.show_type,
uv = _ref2.uv;
}
keynode[i].pv = dataList.map((i) => {
var val = show_type === 1 ? calculate === Number.POSITIVE_INFINITY ? 0 : Math.round(calculate) : calculate === Number.POSITIVE_INFINITY ? 0 : Number((calculate * 100).toFixed(2));

});
/// ...
};
In short, it is a bug in swc

So this is related to the SWC used by Rspack, will it be fixed in the future?

So this is related to the SWC used by Rspack, will it be fixed in the future?

It's report to swc. We will help swc to fix it.

So this is related to the SWC used by Rspack, will it be fixed in the future?

It's report to swc. We will help swc to fix it.

Thank you for the clarification. I'm looking forward to the fix in future updates.


Update scala to 2.12.10

This requires #677

after we merge #677, merge in master and maybe CI will go green.

thanks!

looks like the java-api stuff breaks in 2.12.10... man... what happened with 2.12.10? It seems like it is breaking more that you would expect for a stable minor version.

cc @smarter

looks like the java-api stuff breaks in 2.12.10... man... what happened with 2.12.10? It seems like it is breaking more that you would expect for a stable minor version.

This happened: https://github.com/scala/scala/commit/e48cfd26f66c8f128f966ecd100d23171d9429cf which backported this: https://github.com/scala/scala/commit/3aea776ca1aa82c9de44cc6806dcdb242f3b40f8. Basically chill was taking advantage of a compiler bug, these classes were never supposed to be visible from Scala (they can't really be hidden from Java, so they're still visible there)

So it seems that the culprit here is ScalaUnidoc/doc. For 2.12 javaapi is already being discarded, so I applied the same thing to the ScalaUnidoc scope.
https://github.com/twitter/algebird/blob/4bb8ffb1e10614e99bd19f585e49eacb79da0f1c/build.sbt#L209-L210
The supposed issued java api and 2.12.10 led me to also to #679

Codecov Report

Merging #678 into develop will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #678      +/-   ##
===========================================
- Coverage    89.49%   89.43%   -0.06%
===========================================
  Files          113      113
  Lines         8944     8944
  Branches       314      314
===========================================
- Hits          8004     7999       -5
- Misses         940      945       +5

Impacted Files
Coverage Δ

...src/main/scala/com/twitter/algebird/Interval.scala
77.39% <0%> (-7.83%)
:arrow_down:

...c/main/scala/com/twitter/algebird/MapAlgebra.scala
74.77% <0%> (-0.91%)
:arrow_down:

.../main/scala/com/twitter/algebird/BloomFilter.scala
94.69% <0%> (+1.32%)
:arrow_up:

...in/scala/com/twitter/algebird/scalacheck/Gen.scala
100% <0%> (+9.09%)
:arrow_up:

Continue to review full report at Codecov.

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


build(deps): update cryptoxide requirement from 0.3.6 to 0.4.1

replaces https://github.com/txpipe/pallas/pull/34
cc @NicolasDP

looking good !

looking good !

Welcome back! =)


[notification hubs] Updating to README snippets

Packages impacted by this PR

@azure/notification-hubs

Issues associated with this PR
Describe the problem that is addressed by this PR
This updates Azure Notification Hubs to use README and JSDoc snippets to extract the snippets from the unit tests.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Allow multiple languages to be cached.

This allows setting or showing values using $model->translate('en') for multiple languages (e.g. in admin interface).
E.g:
$group = new Group;
$en = $group->translate('en');
$nl = $group->translate('nl');
$en->name = 'Name EN';
$nl->name = 'Name NL';
$group->save();

And:
echo $group->translate('en')->name;
echo $group->translate('nl')->name;

Also fixes not being able to set translated attributes directly (using $model->name) and only allowing mass assignment before having saved a translation at least once.

Thanks for your contribution. This looks very good! I like your approach and that this isn't necessarily a breaking change. I'll try this out myself first and then merge.

What does your second commit do?

Yes, sorry... I didn't intend to add a second commit.
It's completely stand alone and fixes issue https://github.com/vinkla/translator/issues/1
(or what I experienced anyways; return Model::all() includes translated properties now)

No problem. Could you split them up into two pull requests?

See https://github.com/vinkla/translator/pull/27

Thanks for splitting them up!


Handle orphans with hashtable instead of list

Loading from disk speedup is around 400%
define on compiler USE_BASIC_CONFIG if you can't run the configure of secp256k1 (like Windows OS) to create the configuration of secp256k1

Finish

Finish

define on compiler USE_BASIC_CONFIG if you can't run the configure of secp256k1 (like Windows OS) to create the configuration of secp256k1

@sgaragagghu yes, all global variables have to have prefix "g_"

@jonano614 i added g_ to global variables

Thank you for your efforts!


Access raw http request/response for debugging/logging

From what I can tell with a JestResult there doesn't seem to be anything other than status code available to figure out why requests are failing. Is there any way to get at the raw response body or headers? Inspecting the raw request would help too..

anything other than status code

getErrorMessage()

get at the raw response body

getJsonString()

When I was debugging agains an AWS ES stack trying to authenticate with signed requests both of those  returned empty stings anytime AWS gave a 403 response.

When dealing with a large cluster, having access to the HttpUriRequest object would be nice. Currently it's impossible to know which of my nodes Jest queried which makes isolating problematic nodes near impossible.


Fix the change exercise

Implement the changes required by the linter #399
No changes are required in the change.spec.js file.  Only the example.js file is change for this exercise.

I closed this PR and re-opened another.  I was trying to reuse a branch from the previous PR and the comment history is all messed up.  Sorry.

Thanks for creating a clean PR, that's very helpful.
I understand you. Sometimes, git gets in the way. You're not alone, everybody mess something up from time to time 😜


Death UI

When the player dies, he will receive two death screens

Bug reports like these likely affect the core Geyser module and not just Geyser-Fabric. I would appreciate if this issue were re-opened in the Geyser repository following the issue template. There's not much we can do with this amount of detail, unfortunately (I've died many times and haven't seem this behavior).
Thank you for reporting anyway!


[1.3 Alpha Testing] Add Shortcut Keys for Data, Signals and Logs

Thoughts:

Data: Ctrl + Alt + D
Signals Ctrl + Alt + S
Logs: Ctrl + Alt + L

Through trial and error, these will be:

Data: Ctrl + Alt + 7
Signals Ctrl + Alt + 8
Logs: Ctrl + Alt + 9


Alarmo card not recognized after HA update to core-2021.6.1

core-2021.6.1

Same issue on core-2021.6.0

I am here on 2021.6.1 as well:

But no problems for alarmo-card:

I don't really know how to help.

I returned back to v5.5 and run a direct upgrade to 6.1 and now the card is working again.FYI.

Are there errors in the logs? Some more details could help clarify things better

Is not something to do with this?
https://developers.home-assistant.io/blog/2021/05/19/lit-2.0
Reported in the breaking changes?

I am eager to upgrade. Don't want any trouble just yet lol. Rebuilding my HA environment. Dont want to be doing rollbacks etc.
@nielsfaber what do you think?

Same issue on core-2021.6.0

I just upgraded my HA to 6.2 and it's working fine. The frontend also was updated. Try to reset the browser cache and see if that works.

I am not aware of any problems with Alarmo-card.
I think clearing browser cache may solve the problem (this has been the cause many times in the past)

After upgrading to core-2021.6.x get error message Wrong code provided. Code has not been changed and i reentered the code, now result.
Logger: custom_components.alarmo.alarm_control_panel
Source: custom_components/alarmo/alarm_control_panel.py:379
Integration: Alarmo (documentation, issues)
First occurred: 11:33:25 AM (1 occurrences)
Last logged: 11:33:25 AM
Wrong code provided.

I also get the 'Custom element doesn't exist: alarmo-card.' in the Android HA companion app, shows fine in Chrome. No log entries shown.

Closing this issue since the problems are due to browser caching and not a bug in the implementation.
Caching is behaviour of HA itself, I cannot help it.


URL Issue

Hello! First of all, thank you so much for creating this. I am having a problem with the URL. I followed everything properly. But when I go to the URL, http://localhost:3000/?backend=ws://localhost8999, and open my client and start a tournament game mode, nothing appears on the URL.

Also, I forgot to mention, my League client is in a Garena server.

wrong address
http://localhost:3000?backend=ws://localhost:8999
that is the right address


explitictly unset prioritize on deprioritizing

closes #525
Make sure that the prioritize flag is unset on de-prioritizing.

@KiOui any idea why this is not unsetting the flag currently? I am stuck

@KiOui any idea why this is not unsetting the flag currently? I am stuck

The OrderSerialize has the prioritize field as read-only. It should be altered in such a way that deprioritize overrules prioritize. I think we have three options here:

The best option is to just create a new field "priority" with three options: "deprioritize", "normal" or "prioritize". This option does however require a database migration.
We could also maybe adjust the save method to automatically set prioritize to false whenever deprioritize is true.
Or we could change the OrderSerializer to also accept the prioritize field and then adjust the OrderRetrieveUpdateDestroyAPIView such that it only accepts changes to the prioritize field if the user has permission.

For me, this is a frontend thingy. There's nothing wrong with both prioritize and deprioritize flags set in principle. It's just not the behavior people expect if they hit the button. So keep it in the API / serializer

I don't agree with you on that point. It's weird to have an order that both has priority and does not have priority. So I think migrating this to just one field that can either be prioritized, normal or deprioritized is the best option.

This has been fixed in #534


Flexible Dependencies for Development

In addition to pinning dependencies in master, it might make sense to keep the develop branch nice and flexible, with the expectation that it might break.
I think it makes sense to retain the capability to build from "unpublished" source when necessary.
As a rule-of-thumb, maybe we also agree that the minimum version accepted by develop be the version we pin in master.

What does this add to the release process of a new PyPi version? Do the version numbers have to change each time? Will they stay constantly out of sync between develop and master?

@konklone, I think develop and master always being out of sync is a good point.
Better would be not to use the current version for each package, but instead figure out the minimum version that will work.


How can I split terminal (horizontal or vertical) and change default terminal to powershell?

I didn't see any description about split and change terminals.
Have you supported it yet?
If not, how can I custom by myself?
Thank you

Hi @baroxFPTU
for open the terminal in a split, you can add a mapping like nnoremap <something> :split | ToggleTerminal but you'll have to keep track of the split and close it if you want that to also toggle. you can have a look at nvim_toggle_terminal.vim to understand how the terminals are handled.
I keep this plugging minimal, because I only use it to toggle the terminal buffer in the same window :)
Regarding you second question, I assume you are using windows right?
I haven't tested this plugin on windows, and I am not familiar with it, but since the shell that you run isn't managed by neovim itself, I guess in windows should be the same, so you'll probably have to configure it outside neovim.


Allow deleting suspended objects

Reorders the object suspended check in all the reconcilers to allow
deletion of objects when they are suspended. Objects used to get stuck
on delete because the finalizers were not getting removed due to the
suspended state.

Add setters and getters for spec.suspend and status.artifact in internal/object package.
This is needed for writing generic tests for any source kind.

Adds a generic test for all the reconcilers to check if a suspended
source object can be delete.

Some observations related to the metrics:
When a suspended object is created, finalizer is set and status.observedGeneration remains in -1.
apiVersion: source.toolkit.fluxcd.io/v1beta2
kind: GitRepository
metadata:
  ...
  finalizers:
  - finalizers.fluxcd.io
  generation: 1
  name: podinfo3
  namespace: default
  ...
status:
  observedGeneration: -1

With prometheus query gotk_reconcile_condition{kind=~"GitRepository",name="podinfo3"}==1, following are the observed metrics:

status=Unknown when suspended on creation

gotk_reconcile_condition{container="manager", endpoint="http-prom", exported_namespace="default", instance="<IP_ADDRESS>:8080", job="monitoring/flux-system", kind="GitRepository", name="podinfo3", namespace="flux-system", pod="source-controller-8bb478cb9-knmg4", status="Unknown", type="Ready"}

status=Ready on resuming

gotk_reconcile_condition{container="manager", endpoint="http-prom", exported_namespace="default", instance="<IP_ADDRESS>:8080", job="monitoring/flux-system", kind="GitRepository", name="podinfo3", namespace="flux-system", pod="source-controller-8bb478cb9-knmg4", status="True", type="Ready"}

status=Ready when the object is suspended again, same as above.

status=Deleted when the suspended object is deleted

gotk_reconcile_condition{container="manager", endpoint="http-prom", exported_namespace="default", instance="<IP_ADDRESS>:8080", job="monitoring/flux-system", kind="GitRepository", name="podinfo3", namespace="flux-system", pod="source-controller-8bb478cb9-knmg4", status="Deleted", type="Ready"}

Fixes #934

Looks like main brach is broken as well

failed to checkout and determine revision: recovered from git2go panic: runtime error: invalid memory address or nil pointer dereference

Seems to be because of git2go panic, which is resulting in a different status condition. Also, only panicking on macOS 11.

Ok we can ignore the panic then, it's not something we can fix in this PR.


Add an option to change the title

When you have a lot of sorbet.run tabs open, it's hard to figure out which is which. This allows you change the tab title to a custom name, so you can distinguish them.

For example, here's a bookmarklet you can save to do this on any page
javascript:(function() {
    var newTitle = prompt("Enter a new title for this page:", document.title);
    if (newTitle !== null) {
        document.title = newTitle;
    }
})();

Right-click on your bookmarks bar and choose "Add page" (or "Add Bookmark").
In the "Name" field, you can enter something like "Change Page Title".
In the "URL" field, paste the JavaScript code provided above.

You could even tweak the thing above to e.g. set the favicon to a random emoji or something like that, so you can just click your button and have it randomize the favicon.