alias_method_chain is deprecated

$ bundle exec rails console
DEPRECATION WARNING: alias_method_chain is deprecated. Please, use Module#prepend instead. From module, you can access the original method using super. (called from singleton class at /Users/oleg/Projects/manageiq/lib/extensions/ar_process.rb:6)
DEPRECATION WARNING: alias_method_chain is deprecated. Please, use Module#prepend instead. From module, you can access the original method using super. (called from <class:Preloader> at /Users/oleg/Projects/manageiq/lib/extensions/ar_virtual.rb:283)
DEPRECATION WARNING: alias_method_chain is deprecated. Please, use Module#prepend instead. From module, you can access the original method using super. (called from <module:FinderMethods> at /Users/oleg/Projects/manageiq/lib/extensions/ar_virtual.rb:310)
DEPRECATION WARNING: alias_method_chain is deprecated. Please, use Module#prepend instead. From module, you can access the original method using super. (called from <module:Calculations> at /Users/oleg/Projects/manageiq/lib/extensions/ar_virtual.rb:317)

Also,
$ git grep alias_method_chain
app/models/ext_management_system.rb:  alias_method_chain :clear_association_cache, :storages
config/initializers/av_number_helper.rb:      alias_method_chain :number_to_human_size, :negatives
config/initializers/av_number_helper.rb:      alias_method_chain :number_to_human_size, :significant_override
config/initializers/yaml_autoloader.rb:      alias_method_chain :resolve_class, :constantize
lib/extensions/ar_process.rb:  alias_method_chain :pid, :safe
lib/extensions/ar_virtual.rb:      alias_method_chain :preloaders_on, :virtual
lib/extensions/ar_virtual.rb:    alias_method_chain :find_with_associations, :virtual
lib/extensions/ar_virtual.rb:    alias_method_chain :calculate, :virtual
lib/resource_feeder/test/test_helper.rb:  alias_method_chain :rss_feed_for,  :ostruct
lib/resource_feeder/test/test_helper.rb:  alias_method_chain :atom_feed_for, :ostruct
lib/vmdb/productization.rb:          # a la alias_method_chain

What do we loose by making this change?
When did Module#prepend get introduced? ruby 2.2? So miq 5.5?
While it is important to leverage newer apis, it does makes back porting our code that much harder.

2.0, if that makes a difference.

@chrisarcand sweet - that does for me. thanks
change away!

Only a few places, and then this can be closed.

Following this up, the old resource_feeder lib we use is very old; after getting load paths working again I don't think these tests are at all compatible with Rails 5.
As that whole lib should be updated/replaced and these tests are so far removed, I think we can close this issue now. Those alias_method_chains are never run anyway. @matthewd ?


feat: add ssh key data source

Relates: https://github.com/1Password/terraform-provider-onepassword/issues/74
The only thing to add was support for replacing spaces with underscores and adding new fields in the data source. Now going forward it should be easier if someone wants to add more fields to the data source, as the spaces will get fixed easily.
Would be nice to get this in 😸

@volodymyrZotov could take a look please

@atammy-narmi Sure! Added this to the review list! Thanks for reminding about this!

Hi @atammy-narmi thanks for you work
I'm trying out this code and it would seem that the ssh key i'm getting is not the same as if i'm copying it manualy
How you had such issues? perhaps its pebkac

friendly ping

@atammy-narmi sorry for the delay. A migration happened in the background https://github.com/1Password/terraform-provider-onepassword/pull/167 that led to conflicts with your work. Would you have a chance to resolve them, please?

Hey @atammy-narmi!
As an addition to what @volodymyrZotov said, I've noticed that your commits are not signed. This repository requires all commits to be signed. If you work on changing this PR to make it compatible with the latest v2 version of the provider, it would be great to have your commits signed as well.
Here is some guidelines in achieving this.

Sign commits with 1Password

You can also sign commits using 1Password, which lets you sign commits with biometrics without the signing key leaving the local 1Password process.
Learn how to use 1Password to sign your commits.

Sign commits with ssh-agent

Follow the steps below to set up commit signing with ssh-agent:

Generate an SSH key and add it to ssh-agent
Add the SSH key to your GitHub account
Configure git to use your SSH key for commit signing

Sign commits with gpg

Follow the steps below to set up commit signing with gpg:

Generate a GPG key
Add the GPG key to your GitHub account
Configure git to use your GPG key for commit signing

OK, I'll work on updating this PR.

I've added a test. Is there anything blocking this from being merged?


Chatbot open (Background Scroll issue)

Description
Two things need to be fixed

if click on chatbot , background is not scrollable while chatbot is open , but after close the chatbot window is still not scrollable. it should be scrollable after closing a chatbot

While chatbot is open , background must not scroll even a single pixel.

Screenshots
No response
Additional information
No response

@vanshika2910 Assigned

@vanshika2910 Current bot will be removed and replaced with AI bot , so i am closing this issue


[CMS] CMS field focus issue in LeftAndMain.EditForm.js

When saving an item, the last field focussed is remembered. However when you then go to create a new item, that field will then be refocussed, which is a pain if you have quite a few fields above it.
I believe this may have been intended as a feature (for remembering the last open tab), but remembering the last focused field is causing grief for content editors who then have to scroll back up to the top of the form. On SS 3.3

When you say items, do you mean pages, or items in a modeladmin, or even items in a gridfield?
When you say field do you mean tabs only, or the actual form-field that was selected? (e.g. text field).

Just tried to replicate this issue on Silverstripe CMS 4, but I didn't have much details to go on. I didn't manage to find a problem.
Unfortunately, Silverstripe CMS 3 has entered limited support in June 2018. This means we'll only be fixing critical bugs and security issues for Silverstripe CMS 3 going forward.
You can read the Silverstripe CMS Roadmap for more information on our support commitments.
If you find that this is still a problem in SS4, please post additional replication steps and I'll reopen.


Sage: Integrate Pine into icons

[!CAUTION]
DO NOT MERGE

Description
Updates components and documentation to use pds-icon. Switching over the new icon system should help to eliminate the current manual process.
Screenshots
Minimal visual changes should be expected.
Testing in sage-lib

Navigate to each component for Rails & React
Verify icons render as expected.
Verify icons render in docs as expected

Testing in kajabi-products

(BREAKING) Converts icon font to pds-icons in components and documentation.

[ ] A full regression of the kajabi admin should be performed to make sure all icons still display as expected.

Related
DSS-623

Icons are coming in fine in Rails, but I can't see them on the React side. Additionally, because of them being web components, there is now a fraction of a second on page load with no cache where the icon isn't showing up. I can't imagine there will be complaints about it, but just making a note of it here, since this is being updated app-wide.

You will need to be running your local pine for the react side. Also, make sure the ports match your local instance in the packages/sage-react/.storybook/preview-head.html file.

@pixelflips - the Category item page has an alert, http://localhost:4000/pages/component/catalog_item?tab=preview, is missing the icon

Local
Live

Property is missing the icons
http://localhost:4000/pages/component/property?tab=preview

Typeahead is missing the icons in the dropdown

@monicawheeler those should all be addressed now, docs just had incorrect icon naming.
alert icons - done
property - done / forgot to push the commit for this one
typeahead - done

On local, the arrows are not aligned properly in react

On local, the arrows are not aligned properly in react

@monicawheeler before checking storybook, please apply the changes from quinton's comment above.

Could we verify the button spacing between icons is consistent with the documentation? It appears wider in the local dev environment on react compared to the public url

On local, the arrows are not aligned properly in react

@monicawheeler before checking storybook, please apply the package.json changes from quinton's comment above.

This is still an issue for me after pulling and updating the packages/sage-react/package.json file per Quinton's comments.

 Could we verify the button spacing between icons is consistent with the documentation? It appears wider in the local dev environment on react compared to the public url

This is resolved after pulling latest and following Quinton's packages/sage-react/package.json update.

Nice work getting these updated!
Can we update the Banner component as well? That icon is missing.

@anechol great catch! those should be updated now as well.


Bad formatting in "Shake it Off" ability of Gray Worm and other Worms

Like the Glacial Worm before the Formatting of the Shake it off ability ist not correct
See Pull Request #17002
Furthermore i propose to unify the description in all Worms that have this Ability, like the larvae in the Picture below

As far as my research goes the following Worms have this Ability:
Glacial Worm (Description should be adjusted), Gray Worm (Description should be adjusted), Juvenile Cave Worm (Description seems fine), Larval Cave Worm Brood  (Description seems fine)
Sry for the second issue, should have wrote it this way the first time.

@ammalagonc At the moment your PR just fixes the Glacial Worm but not the Gray Worm. Is this intentional?

I didn't read the rest of them.
Hang tight

OK im glad this was done but just wanted to be srue if i had to open another issue :) Thanks alot


No GPU backend when building with anaconda and CUDA 8

I have followed the installation instructions for anaconda.
It builds fine but won't allow me to run on the GPU (only the cpu backend is available).
I have traced this to a pycuda installation error:
$  python -m pycuda.autoinit
Traceback (most recent call last):
File "/home/tmain/anaconda2/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/home/tmain/anaconda2/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/home/tmain/anaconda2/lib/python2.7/site-packages/pycuda/autoinit.py", line 2, in
import pycuda.driver as cuda
File "/home/tmain/anaconda2/lib/python2.7/site-packages/pycuda/driver.py", line 5, in
from pycuda._driver import *  # noqa
ImportError: libcurand.so.7.5: cannot open shared object file: No such file or directory
So even if the install detect cuda 8 (which I checked by inspecting the makefile), pycuda is still built assuming cuda 7.5

Fixed this by building pycuda from source and removing the corresponding line in gpu_requirements.txt


Request for correction,  Create Application Gateway custom error pages

On 'Create Application Gateway custom error pages' azure-docs/articles/application-gateway/custom-error.md
Blob is normally not recommended to be configured for public access on internet. And Application gateway verifies internet connectivity to the error page files when it's configured. But the example on the doc points to Blob. Which causes confusion to the customers
"error page must be hosted in Azure blob storage"
(It doesn't have to be on blob, it can be on anywhere with) valid URL for public access
"you must point to a different blob location"
you can set 'Error page url' to 'No' and 'Save'. And try to set the error page URLs
"Application gateway doesn't periodically check the blob location"
... periodically check the error pages
"Under Error page url, select Yes, and then configure a publicly accessible blob URL"
Under Error page url, select Yes, and then configure a publicly accessible URL

One thing to add to the guide is ,.. because the docbase of errorpage is set to the Application Gateway domain/ip, docbased has to be configured explicitly in the error pages as below

  <base href="URL with prefix(http://) where the linked contents are served from" target="_blank">

Error Page 403

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

ID: 8876a502-81db-4ef1-9e22-ebff4a63cabe
Version Independent ID: c9676383-40c2-44fc-2d26-ef571466712f
Content: Create Azure Application Gateway custom error pages
Content Source: articles/application-gateway/custom-error.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@euchungmsft
Thanks for your feedback! We will investigate and update as appropriate.

@euchungmsft , We have assigned this to the content owner/author for review.
@greg-lindsay , for your review.

This is tracked as internal work item 91160 #please-close


Add example of PostBodyContentTypeParser use for JSON validation

Could you please provide an example of the following?

Use sinatra-param in combination with Rack::PostBodyContentTypeParser and Rack::NestedParams to automatically parameterize JSON POST bodies and nested parameters.

Been looking all over for an example of this, and I can't find anything.
Thanks in advance!

@lukenewman
# Gemfile

gem 'rack-contrib',    require: false
gem 'sinatra-contrib', require: false

# app.rb

require 'rack/contrib'
require 'sinatra/base'

class Api < Sinatra::Base
  use Rack::PostBodyContentTypeParser

  post "/" do
    puts "Params automatically converted to a hash if request Content-Type is application/json:"
    puts params
  end
end

# Gemfile

gem 'rack-contrib',    require: false
gem 'sinatra-contrib', require: false

# app.rb

require 'rack/contrib'
require 'sinatra/base'

class Api < Sinatra::Base
  use Rack::PostBodyContentTypeParser

  post "/" do
    puts "Params automatically converted to a hash if request Content-Type is application/json:"
    puts params
  end
end

Does anyone have suggestions about how to include the middleware into an integration test ?  This is how I have my app setup in the spec_helper (this happens to be a Sinatra app).
module RSpecMixin
  include Rack::Test::Methods
  def app()
    MyApp
  end
end

# Gemfile

gem 'rack-contrib',    require: false
gem 'sinatra-contrib', require: false

# app.rb

require 'rack/contrib'
require 'sinatra/base'

class Api < Sinatra::Base
  use Rack::PostBodyContentTypeParser

  post "/" do
    puts "Params automatically converted to a hash if request Content-Type is application/json:"
    puts params
  end
end

gem 'sinatra-contrib', require: false is not needed.
Apart from that, you can also only load the specific component you need like this: require 'rack/contrib/post_body_content_type_parser'


Make raw audio decoder more practical

Unfortunately, in audio world, there's no clear standard for defining formats like fourcc for video. While defining our own consts for audio is fine, it's impractical when we're trying to integrate frameworks like avfoundation.

LGTM
Sorry for the delay!

No worries. Thanks!


Theming Optimizations

Overview
My team is switching over from using the svg renderer from @dotlottie/react-player to this package, and we're running into some issues with theming using this package. Our use case requires the need for dynamic theming through loadThemeData.
Asset used for the examples: https://lottie.host/5edd3191-18f1-4dcb-b6b5-1c00ef8b2f5c/UQ5qeVivMe.lottie
Codepen: https://codepen.io/khuynh92/pen/RwOzKGZ?editors=0010
Slot ID implementation
Issue: There is no easy way to add a sid to a JSON file. Users have to go into a JSON file, locate the correct property, and  manually add the sid. This can become tedious if this has to be done to multiple files.
Expected Behavior: In After Effects, assets can have class names and id's injected into the JSON file simply by renaming the layer in AE. For example renaming a layer to #star-outline.lottie-icon-outline in AE will inject a class name of .lottie-icon-outline and an id of #star-outline in the exported JSON file.
It would be nice to be able to apply theme updates using the existing class name or id, that way we don't have to manually add sid to the JSON file, or if there was a script provided that added the sid automatically based off of existing class names or ids in that are in the JSON file.
Updating static/paused assets
Issue:  When an asset is paused (or has finished animating), theme updates via loadThemeData or loadTheme do not trigger a theme update. Only when the animation begins to play again will the theme update.
Expected Behavior: using the methods loadThemeData or loadTheme should update the theme, even when paused.

Memory access out of bounds error when reverting back to default theme
Issue: Using loadThemeData, I'm running into issues when trying to revert an asset's theme back to the original data that's within the JSON file. Changing the theme works properly, but when reverting the theme back using loadThemeData("") or loadTheme(""), the manipulated property changes to black and error messages are thrown.
Expected Behavior: Theme should revert back to original colors from the JSON file.

...
Motivation

What inspired this enhancement? What makes you think this should be included?
The need for dynamic theming and ease of theming implementation

...
Labels

[ ] Add the Type: Enhancement label to this issue.

Thanks @khuynh92 for reporting these issues. We will look into it and get back to you with an easier way to set the slot IDs

@khuynh92
You're right, setting the "sid" on lottie json properties manually is not easy. We're currently working on abstracting the theme creation process in Lottie Creator, but until that happens, there is a workaround that allows you to assign slot IDs to animated color properties to match the parent shape's existing class name or ID. You can use relottie to parse the Lottie JSON into a relottie AST and traverse the AST to locate the target shapes by their class name or ID, and then modify their animated color properties' slot ID to match their parent Shapes class name or ID.

It would be nice to be able to apply theme updates using the existing class name or id, that way we don't have to manually add sid to the JSON file, or if there was a script provided that added the sid automatically based off of existing class names or ids in that are in the JSON file.

I've noted down the other two issues and have been able to replicate them. I will update you as soon as I have a fix

@khuynh92
This part of the issue is fixed in<EMAIL_ADDRESS>release.

Memory access out of bounds error when reverting back to default theme
Issue: Using loadThemeData, I'm running into issues when trying to revert an asset's theme back to the original data that's within the JSON file. Changing the theme works properly, but when reverting the theme back using loadThemeData("") or loadTheme(""), the manipulated property changes to black and error messages are thrown.

Will update you once the other part of the issue is resolved, thanks

Will update you once the other part of the issue is resolved, thanks

Appreciate it! Thanks for the consistent status updates 🙂

@khuynh92 This issue is resolved in<EMAIL_ADDRESS>

Thank you @theashraf! Verified in my codepen demo that the changes do work.
As for SID implementation, I'll be using the workaround method for now, but do you have a link to where I can track the progress for easier SID implementation?

@khuynh92, you might find it interesting that we've introduced theming in Lottie Creator! Now, you can create themes to your animations and play them on dotLottie players.
Check out the details here: https://lottiefiles.com/blog/working-with-lottie-animations/how-to-add-themes-to-lottie-animations-in-lottie-creator


iOS build fails

My build is failing when I do react-native run-ios when I have

Set Preprocess Info.plist File to Yes

as per the setup instructions.
react-native-config v0.11.5
react-native 0.55.3
▸ Check Dependencies
▸ Preprocessing MyApp/Info.plist
** BUILD FAILED **

The following build commands failed:
        Preprocess build/Build/Intermediates.noindex/MyApp.build/Debug-iphonesimulator/MyApp.build/Preprocessed-Info.plist MyApp/Info.plist
(1 failure)

Nvm fixed it. Didn't do Set Info.plist Other Preprocessor Flags to -traditional correctly. Had it under the wrong setting.


Snapshot stuck @ Block 1428368

Actual behavior: The snapshot that I have taken today is stucked for more then 2 hours
Expected behavior: complete the snapshot installation at block 1428368
How to reproduce it (as minimally and precisely as possible):
 1. try to open the app

get failure message: credit failure request failed with status code 503
try to make backup- failed
make snapshot
update to 2.8.0
try to reindex the wallet- failed
use snapshot Snapshot wird ausgepackt von C:\Users\Admin\AppData\Roaming\DeFi Blockchain\snapshot\snapshot-mainnet-1428368.zip
snapshot stands still at 1428368

What are your environment parameters?

DeFiCh/app Version: 2.8.0

Operating System (OS): WIN 11
– RAM (GB):  16 GB
– CPU (Intel Core i5/i7/etc.): i7
Configs.zip

Current free storage capacity: 434 GB

Anything else we need to know?

Configs.zip

My snapshot is stuck at 1429877. The last time I had this, there was an advice to change/delete files in the defichain folder, but I can't remember what it was.

I have the same issue. Snapshot is stuck at 1429877.

Same here. I am stuck at 1429877

same here at 1429877

Same here at 1429877 on my notebook. I copied the blockcchain from my desktop pc to the notebook and it worked. Probably a snapshot with the 1429877 will work.

same here, is there somewhere a snapshot to use which is newer than monday?

mine stuck @ 1429877

same here at 1429877 ! also on the Notebook

solved!
take this Snapshot
https://defi-snapshots-europe.s3.eu-central-1.amazonaws.com/snapshot-mainnet-1430890.zip
and install it like here
https://defichain-wiki.com/wiki/Fullnode#Create_and_use_backup_for_blockchain_data

How do you install the snapshot? I don't get it

its in the wiki, scroll down

thanks @Unicornlove87 it is working with the newer snapshot

I m maked New installation wallet, with snapshot from before 3days  and still stuck on same block 1429877

dude dont be so stupid... read the text above! You have to be download an another snapshot manual

u mean this snapshot?? https://defi-snapshots-europe.s3.eu-central-1.amazonaws.com/snapshot-mainnet-1430890.zip

u mean this snapshot?? https://defi-snapshots-europe.s3.eu-central-1.amazonaws.com/snapshot-mainnet-1430890.zip

solved!
take this Snapshot
https://defi-snapshots-europe.s3.eu-central-1.amazonaws.com/snapshot-mainnet-1430890.zip
and install it like here
https://defichain-wiki.com/wiki/Fullnode#Create_and_use_backup_for_blockchain_data

i never did that, i will try later.  i m new in this,

Hi @JoergHoffmann @RainerBlessing @xXPaseoXx @MrB3x3F @Gilli1545 @Unicornlove87 @Svile30 @kmeans27 @chregg @NMF1812 , update v2.8.1 is out, this is a recommended update. Kindly backup wallet before updating.
Please read through the release notes before updating. This comment is to notify you about the update.

Hi @izzycsy unfortunately the problem still exists

Worked for me. Thanks!

after struggling several days I am @100% Sync! HURRA! Downloaded the update 2.8.1 then  take a snapshot 1430890 and now it works.
THANKS to everyone for support.


Bottom to Top Button visibility (z-index) and radius issue

Bottom to top Button is hide inside insta-post section,  so it should be top of this section,
https://github.com/user-attachments/assets/700b69fc-d4db-4bbb-8bef-2c51b18e0aad
and button should be propper round shaaped,  now it is like ovel shaped ..

hey @piyushshrma  let assign it me,,

Thanks for creating the issue in WildGuard! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Don't forget to follow @anuragverma108 – Project Admin & @PriyaGhosal -Project Collaborator  – for more updates!
We're excited to see your contribution as part of Hacktoberfest 2024! 🎉
Happy open-source contributing!


Use "golang.org/x/sys/unix" instead of "syscall" (fix #59)

According to the discussion on golang/go#18866, Termios from golang.org/x/sys/unix has better portability than syscall.Termios. So we should use it.
This PR works fine on mipsle.

Thanks for the PR!


ref: Moved /linode/volumes to /volumes

ARB-209 # done

Moved the endopoint to its own top level

Coverage remained the same at 71.27% when pulling 9e3896146d80b5c0fc171589e1d36d17e6dde777 on yoshizzle:ref/volumes into bbec7cf9087e9e9503095bd22cf9cb15a5d1d1bc on linode:pending-docs-updates.


Only use label.ont in SingleR workflow

Closes #380
This PR updates the build-celltype-ref.nf workflow to only build a model from the given params.label_name (default label.ont).  Note at one point I kept getting confused b/w two directory params, so I added some comments into config/reference_paths.config for my 🧠 .
I have not yet run this through to update files on S3, but I've tested with local output directory and it's working for me fine there!

Womp womp, noting I had accidentally set the target of this PR to main..and then I merged in main when github told me my branch was out of date 🥴 , and at this point I thought "wait, huh?" So, I promptly updated target to development, and happily there are no meaningful consequences to this silliness.

I've updated this to grab and save cell names as well. To fully ensure correct mapping between ontology IDs and cell names, I ended up adding a data frame called cell_ontology_df to the model object that stores this information. I could also have made this a named list (ontology ids as names and cell names as values), but I figure later we'll end up inner_join'ing this information with the cell annotations, so a data frame seemed better for that next step.

Noting that ontoProc was added to scpcaTools - https://github.com/AlexsLemonade/scpcaTools/pull/186


[IDE] Can't close welcome screen

Describe the bug
Welcome tab cannot be closed unlike any other tab

Progress to be tracked in:

https://github.com/eclipse/dirigible/issues/1222

Additional context

This is not a bug. It was decided a while back that the welcome screen should not be closable.
See Dirigible issue https://github.com/eclipse/dirigible/issues/900

I think of https://github.com/eclipse/dirigible/issues/900 as temporary mitigation to the problem.
If currently there are technical limitations to solve this we should discuss them and think of how to address them.
Welcome tab is not special and is not treated specially in any other development environment
I'm reopening this to have it for the 1.0 delivery


FlashMessage

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?

Fixed issues

Csrf is currently broken by https://github.com/yiisoft/session/commit/0f9dbe7ab8360ee9165001725bc312fced07a5e2

I think we should add a redirect to itself to prevent formspamming

👍


Conditional Code Annotations

Would be nice to have conditional Code annotations, e.g. explanatory words per-language.
Right now this feature is only useful if we can have text that makes sense for all language tab options. For example, the example below works because there's the same output format no matter which language option is chosen.

Get a Specific Kitten
require 'kittn'

api = Kittn::APIClient.authorize!('meowmeowmeow')
api.kittens.get(2)

import 'kittn'

api = Kittn.authorize('meowmeowmeow')
api.kittens.get(2)

curl "http://example.com/api/kittens/3"
-H "Authorization: meowmeowmeow"

The above command returns JSON structured like this:

{
 "id": 2,
 "name": "Isis",
 "breed": "unknown",
 "fluffiness": 5,
 "cuteness": 10
}

Would be nice to have code annotations specific to different code snippets as well. :)
Makes sense? Thanks!

In dev, you can now put the following code in your index file to show a language specific code annotation.

only show for ruby and shell languages

Great feature, thanks!


new module XMonad.Actions.PerLayoutKeys

Description
This adds a new module PerLayoutKeys based on XMonad.Actions.PerWorkspaceKeys, which allows conditionalizing keybindings based on the current layout. This was requested by someone in IRC.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: has to be manual

[x] I updated the CHANGES.md file

I always feel funny about merging my own changes….

Shouldn't we abstract this and have like a ConditionalKeys module where perWorkspace and perLayout are just functions?  Then we could also deprecate PerWorkspace and not have two modules with almost identical implementations

Aside from people increasingly complaining — not just about xmonad, but pretty much all over the Haskell community — about that kind of breaking change. I'm thinking it might be better to got for simplicity over Haskelly "correctness", especially since the IRC user I mentioned already seems confused about the whole thing.

I mean, I don't think a deprecation would cause a lot of users to complain; especially when the upgrade path is "verbatim replace that function with that other function"

I have my own spin on "conditional keybindings" which imo is quite powerful and user friendly, but it depends on the grab and customRegrabEvHook functions I originally exposed for modal keybindings. I was thinking of making it a PR after that one merges. Preview in gist.
Rather than layouts or workspaces, I actually use it to provide a distinct set of bindings on floats.

@LSLeary this look great to me!  Especially the use-case you described convinces me (and hopefully also @geekosaur :)

but it depends on the grab and customRegrabEvHook functions I originally exposed for modal keybindings.

This should land in #703, so that seems fine to me.


Wait for local file systems before starting prometheus

In case prometheus data is stored on mounted volume, wait on it before
starting process.
This change also makes sure that prometheus is stopped to allow safe
unmount volume during shutdown.

Thanks :+1:


Wrong property in EventType definition

In the definitions.html#EventType propertey partition_key_fields you are referring to partition_resolution_strategy which doesn't exist. I think you meant partition_strategy there.

@MALPI, looks like you didn't get a response here. @dehora, did someone make this change? Is there something more to do here, or can we close it? @rcillo @antban @v-stepanov

It's At least still in the manual.

This is fixed in #54

@MALPI looks like this issue is still there, as it is also a problem in Nakadi's API specification. We're on it.

Fixed in https://github.com/zalando/nakadi/pull/1225


root-signing-staging: Do not use the new role yet

This is a revert of half of #395.
'pulumi up' fails with this:
422 Role `write-with-bypass` is not available for the
sigstore/root-signing-staging repository.

I'm not sure why the custom role is not found. Let's revert this and I will investigate.

The visible changes in pulumi preview are from  #394 (which failed because of the issue that I'm working around in this PR)


Colorization issues in diffs

@karthiknadig I can't reproduce this trivially, do you have any more info about how you got into this state? Compare/annotate/history/merge/etc.?


is pdf parser open-sourced

I wonder whether the pdf parser is open-sourced? Thanks!

https://github.com/elifesciences/sciencebeam-parser is this used?

@harrywang try reading the dependencies, it should be pikepdf https://github.com/Weixin-Liang/LLM-scientific-feedback/blob/main/requirements.txt https://github.com/pikepdf/pikepdf


Made it possible to add and remove tags from existing entries

Made it possible to add and remove tags from existing entries.
Added tag sanitation (remove duplicate tags, trim tag whitespace, etc).
Added graceful handling of KeyboardInterrupt.
Added entry index indicator.
Moved finish_key to global namespace.

@maxwellgerber Great job! :smile: :+1:
Merging it!


Add Parasite

https://github.com/ecohealthalliance/awesome-parasite/
A curated list of awesome information and open-access databases related to parasites and host-pathogen interactions.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Requirements for your pull request

I have read and understood the contribution guidelines and the instructions for creating a list.
This pull request has a descriptive title.For example, Add Name of List, not Update readme.md or Add awesome list.
The entry in the Awesome list should:

Include a short description about the project/theme of the list. It should not describe the list itself.Example: - [Fish](…) - User-friendly shell., not - [Fish](…) - Resources for Fish..
Be added at the bottom of the appropriate category.

The list I'm submitting complies with these requirements:

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
It's the result of hard work and the best I could possibly produce.
Non-generated Markdown file in a GitHub repo.
Includes a succinct description of the project/theme at the top of the readme. (Example)
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate.
Only has awesome items. Awesome lists are curations of the best, not everything.
Includes a project logo/illustration whenever possible.

Either fullwidth or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.
Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of sub-lists, preferably none.

Has an appropriate license.

That means something like CC0, not a code licence like MIT, BSD, Apache, etc.
WTFPL and Unlicense are not acceptable licenses.
If you use a license badge, it should be SVG, not PNG.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.

Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.

Go to the top and read it again.

Hey, sorry for the lack of response, please see #1363.

Thanks for response @sindresorhus. I agree with your points in #1363. I have reviewed #1308, #1321, #1300. Also, the repo has now been open-sourced for >30 days. Let me know if there's anything else I should do.

If you use a license badge, it should be SVG, not PNG.

⬆

The heading should either be awesome-parasite or Awesome Parasite, preferably the latter.

Includes a succinct description of the project/theme at the top of the readme. (Example)

⬆

Interesting topic! :)

Two remarks:

The first item is not correctly formatted ("Olival et al. 2017 - Mammal-virus database with analysis.")
Create a separate section for the licence

(Pinch hitting for a very busy @arw36)
I've changed the the CC badge to SVG, rebased this branch and changed the link to the new, anchored format.


Do not update balance if  there are unconfirmed transactions

If we update balance while there are unconfirmed transactions, it wont be correct in terms of user experience even tho its correct in terms of blockchain.
e.g.
1 incoming transaction of 10 TKL.
If user wants to send 1 TKL away, all 10 TKL leave the wallet and the balance is 0 for about 30 seconds while the transaction is being confirmed.

we are just not updating the balance on sending, unless we receive confirmed txs and confirmed changes now


python yaml libraries: strictyaml, PyYAML, ruamel, ruyaml, ...

The Python/YAML ecosystem is fragmented and confusing, so I've opened this issue to help clear things up. Currently, we're using the strictyaml library to validate the input YAML via a schema. strictyaml uses the ruamel library to do the loading/dumping. strictyaml's documentation is an opinion piece rather than real documentation, and if we're going to do schema validation for both YAML and JSON input, the validation is better done at an intermediate layer rather than separately for JSON and YAML. So probably we can/should just load the YAML ourselves and drop strictyaml.

PyYAML has been around for many years and offers load()/dump() functionality. There's a pure python implementation, and optional bindings to the C-based libyaml. There are minor differences between the C-based and Python-based loaders/dumpers. The C-based loader is not used transparently when available (it has to be used explicitly the python code). PyYAML only supports YAML v1.1. Supports python 2 and 3.
ruamel is a fork of PyYAML that was created in response to a lack of activity in PyYAML, and the desire to fix many open issues, including the addition of support for YAML v1.2. The v1.2 spec appears to fix some odd corner cases in v1.1. ruamel has also introduced a new API, and is deprecating API compatibility with PyYAML. I couldn't find any API reference docs for the new API (just some discussion and a few examples). It's hard to tell for sure, but it seems that some of the desirable fixes are only available with the python-based loader/dumper. Supports python 2 and 3.
ruyaml is a recent fork of ruamel that was created in response to a lack of openness in ruamel. It's still very new, so it's not clear how this will develop. The API is the same as for ruamel (both PyYAML and new API available), but they've already restructured the sources and dropped python < 3.6 support. Support for the C-based libyaml loader/dumper has been (temporarily?) removed.

So, looking forward, I'd like to throw my support behind the newcomer, ruyaml. We won't use many features of YAML anyway, so if the API or ecosystem changes, we can reasonably easily swap it out. The pure-python loaders/dumpers are slow, but this only matters if you're loading/dumping thousands of files, in which case using JSON demes graphs should be preferred.

Jeez, what a mess. Your plan SGTM.


Update the dispatch screen (EXPOSUREAPP-3730)

Description
Dispatcher screen and text adjustments as provided in Figma designs.
Steps to reproduce
Go to dispatcher screen and admire the awesome new design...
Closes JIRA ticket EXPOSUREAPP-3730

Description
Dispatcher screen and text adjustments as provided in Figma designs.
Steps to reproduce
Go to dispatcher screen and admire the awesome new design...
Closes JIRA ticket EXPOSUREAPP-3730

Just an idea/suggestion for regular watchers of this repo: add some screenshots for changed UI elements in general :).


Swift pod can't depend on JSQMessagesViewController framework (v.7.2.0)

I have a new pod (written in Swift) using CocoaPods in my private repository.  It will depends on JSQMessagesViewController framework.
Unfortunately the pod validation fails since it is not possible to build JSQMessagesViewController.
#import "/var/folders/l4/wwg_21kn59l0kqsw2_7fl_140000gn/T/CocoaPods/Lint/Pods/Target Support Files/JSQMessagesViewController/JSQMessagesViewController-umbrella.h"

/var/folders/l4/wwg_21kn59l0kqsw2_7fl_140000gn/T/CocoaPods/Lint/Pods/Target Support Files/JSQMessagesViewController/JSQMessagesViewController-umbrella.h:3:9: note: in file included from /var/folders/l4/wwg_21kn59l0kqsw2_7fl_140000gn/T/CocoaPods/Lint/Pods/Target Support Files/JSQMessagesViewController/JSQMessagesViewController-umbrella.h:3:
#import "JSQSystemSoundPlayer+JSQMessages.h"

/var/folders/l4/wwg_21kn59l0kqsw2_7fl_140000gn/T/CocoaPods/Lint/Pods/JSQMessagesViewController/JSQMessagesViewController/Categories/JSQSystemSoundPlayer+JSQMessages.h:19:9: error: include of non-modular header inside framework module 'JSQMessagesViewController.JSQSystemSoundPlayer_JSQMessages'
#import <JSQSystemSoundPlayer/JSQSystemSoundPlayer.h>

To be able to build it, file JSQSystemSoundPlayer+JSQMessages.h should be changed by replacing line 19  #import <JSQSystemSoundPlayer/JSQSystemSoundPlayer.h> with this line @import JSQSystemSoundPlayer;
This is an extract of my podspec:
Pod::Spec.new do |s|
s.name         = 'MyPod'
s.version      = '0.0.20'
s.platform     = :ios, '8.0'
s.requires_arc = true
s.source_files = 'MyPod/**/*{swift}'
s.frameworks   = 'UIKit'
s.dependency "JSQMessagesViewController", "7.2.0"
end

We need to have a release or branch to reference to be able to use JSQMessagesViewController also inside a swift pod.
Important note: I can't use bridging header since it is not supported on Cocoapods. I simply import the module in my swift files in this way:
import JSQMessagesViewController

Pull request 1325 fixes it. https://github.com/jessesquires/JSQMessagesViewController/pull/1325

Closing this issue as a dupcliate in favor of #1284


[ mobile ] adjust placement of bottom bars with ads

[ ] there's 3 main notifications - gdpr, langauge selection, errors that are shown at the bottom bar.
[ ] we can either move it all the time to the top, or only when ads are running (basically leave for premium+) - but don't complicate it if that's harder.

Believe this was completed...


Remove global variables, pass context across all components and improve error handling

ISSUE TYPE

Feature Pull Request

SUMMARY

Use envconfig
Remove global variables related to Kubernetes, logger, and filters
Pass context across all components - support graceful shutdown of all BotKube components
Improve error handling (return errors in almost all places instead of ignoring/logging them)

This is just a start, as later (during new features development) we should refactor each component/package in isolation. But after this huge PR it should be possible to do it.
Fixes #220
TODO
After first review of this PR if the new filter engine approach is accepted:

[x] Update documentation (https://www.botkube.io/filters/) PR: https://github.com/infracloudio/botkube-docs/pull/81

TESTING
Tested manually with Mattermost, Discord and Slack.
You can use the following image:pkosiec/botkube:remove-global-vars-v2

Fixes: https://github.com/infracloudio/botkube/issues/568

@pkosiec is it possible to break the PR into multiple? Its becomes very difficult to review all at once :)

Hi @PrasadG193,

It doesn't actually fix #568, as HA is still not possible. That's something that need to be addressed in the future.
Thanks for comments I tried to break this PR into multiple ones, but because of the global variables there are so many interconnections across the codebase that the PRs (assuming they would be working and ready-to-merge) wouldn't be much smaller, and they would depend on each other. 😞  Fortunately, we already discussed that and @mszostok will review this PR fully 🙂 (AFAIK the review is in progress right now).
Cheers!

Okay.

It doesn't actually fix https://github.com/infracloudio/botkube/issues/568, as HA is still not possible. That's something that need to be addressed in the future.
My bad. I didn't check the description

Update: I noticed a crash when a custom Kubeconfig wasn't specified (last-minute changes, which I tested only locally). Now it is fixed on the latest commit https://github.com/infracloudio/botkube/pull/603/commits/0e2e7629f67cc2fb968716476911f8c0b9f87d45. Sorry!
Use the pkosiec/botkube:remove-global-vars-v2 image. Thanks!

Thanks @mszostok. I tested both with the following scenarios:
ElasticSearch
Install ElasticSearch:
helm repo add elastic https://helm.elastic.co
helm install elasticsearch elastic/elasticsearch --set replicas=1 --set resources.requests.cpu="100m" --set resources.requests.memory="512M" --wait

Create values.yaml:
cat > /tmp/values.yaml << ENDOFFILE
communications:
  elasticsearch:
    enabled: true
    server: http://elasticsearch-master.default:9200
    username: elastic
    appPassword: changeme
config:
  settings:
    clustername: sample
    kubectl:
      enabled: true
  enabled: true
image:
  registry: docker.io
  repository: pkosiec/botkube
  tag: remove-global-vars-v3
ENDOFFILE

Install BotKube:
helm install botkube --namespace botkube ./helm/botkube -f /tmp/values.yaml --wait

Do port forward:
kubectl port-forward svc/elasticsearch-master 9200

Imitate pod failure:
kubectl delete po -n kube-system metrics-server-<press TAB>

See ElasticSearch indices:
curl http://localhost:9200/_cat/indices

Copy the index name with botkube- prefix.
see ElasticSearch index:
INDEX_NAME="botkube-2022-06-06"
curl http://localhost:9200/$INDEX_NAME/_search\?pretty

See logged events.
Result:

Click to expand
{
  "took" : 9,
  "timed_out" : false,
  "_shards" : {
    "total" : 1,
    "successful" : 1,
    "skipped" : 0,
    "failed" : 0
  },
  "hits" : {
    "total" : {
      "value" : 12,
      "relation" : "eq"
    },
    "max_score" : 1.0,
    "hits" : [
      {
        "_index" : "botkube-2022-06-06",
        "_type" : "botkube-event",
        "_id" : "RNifOIEBzVn4jNmi6l97",
        "_score" : 1.0,
        "_source" : {
          "Code" : "",
          "Title" : "v1/pods error",
          "Kind" : "Pod",
          "Name" : "elasticsearch-master-2",
          "Namespace" : "default",
          "Messages" : [
            "0/1 nodes are available: 1 node(s) didn't match pod anti-affinity rules."
          ],
          "Type" : "error",
          "Reason" : "FailedScheduling",
          "Error" : "",
          "Level" : "error",
          "Cluster" : "sample",
          "Channel" : "",
          "TimeStamp" : "0001-01-01T00:00:00Z",
          "Count" : 0,
          "Action" : "Scheduling",
          "Resource" : "v1/pods",
          "Recommendations" : null,
          "Warnings" : null
        }
      },
      {
        "_index" : "botkube-2022-06-06",
        "_type" : "botkube-event",
        "_id" : "Q9ifOIEBzVn4jNmi6V9D",
        "_score" : 1.0,
        "_source" : {
          "Code" : "",
          "Title" : "v1/pods error",
          "Kind" : "Pod",
          "Name" : "elasticsearch-master-1",
          "Namespace" : "default",
          "Messages" : [
            "0/1 nodes are available: 1 node(s) didn't match pod anti-affinity rules."
          ],
          "Type" : "error",
          "Reason" : "FailedScheduling",
          "Error" : "",
          "Level" : "error",
          "Cluster" : "sample",
          "Channel" : "",
          "TimeStamp" : "0001-01-01T00:00:00Z",
          "Count" : 0,
          "Action" : "Scheduling",
          "Resource" : "v1/pods",
          "Recommendations" : null,
          "Warnings" : null
        }
      },
      {
        "_index" : "botkube-2022-06-06",
        "_type" : "botkube-event",
        "_id" : "RdifOIEBzVn4jNmi8F-z",
        "_score" : 1.0,
        "_source" : {
          "Code" : "",
          "Title" : "v1/pods error",
          "Kind" : "Pod",
          "Name" : "elasticsearch-master-2",
          "Namespace" : "default",
          "Messages" : [
            "0/1 nodes are available: 1 node(s) didn't match pod anti-affinity rules."
          ],
          "Type" : "error",
          "Reason" : "FailedScheduling",
          "Error" : "",
          "Level" : "error",
          "Cluster" : "sample",
          "Channel" : "",
          "TimeStamp" : "0001-01-01T00:00:00Z",
          "Count" : 0,
          "Action" : "Scheduling",
          "Resource" : "v1/pods",
          "Recommendations" : null,
          "Warnings" : null
        }
      },
      {
        "_index" : "botkube-2022-06-06",
        "_type" : "botkube-event",
        "_id" : "RtifOIEBzVn4jNmi8l9B",
        "_score" : 1.0,
        "_source" : {
          "Code" : "",
          "Title" : "v1/pods error",
          "Kind" : "Pod",
          "Name" : "elasticsearch-master-1",
          "Namespace" : "default",
          "Messages" : [
            "0/1 nodes are available: 1 node(s) didn't match pod anti-affinity rules."


Remove Redis.exists warning

This commit removes the following warning when using the gem

Redis#exists(key) will return an Integer in redis-rb 4.3. exists?
returns a boolean, you should use it instead. To opt-in to the new behavior
now you can set Redis.exists_returns_integer =  true. To disable this message
and keep the current (boolean) behaviour of 'exists' you can set
Redis.exists_returns_integer = false, but this option will be removed in 5.0.

Can you add a comment to the changelog?

@bpross Thanks! Can you ping me when a new version of this gem is released?
I would need it for https://github.com/procore/dev-portal/pull/2370

@claudioprocore will do. Once this is approved: https://github.com/procore/ruby-sdk/pull/45 we will be pushing out a new release.


Disable Mechanics 2D data display

Hi,
Is there a way to disable the display at each contact in mechanics 2d ? I mean, avoid the call to:
https://github.com/siconos/siconos/blob/01fe2e7de2b8c3803a2d76064d12f9d9ff78c897/kernel/src/modelingTools/NewtonImpactFrictionNSL.cpp#L46
Thanks.

This should be fixed in aa38ca0
This was a debug verbose that I have forgotten


Is it possible to have a custom gif/icon for loading?

I'm sure you could fork this and add that feature!  That'd be awesome.

Is this still wanted? I have a bit of time over the next few days.  Only learning RN now but simple packages like this help the learning curve so am happy to help where I can.

That would be great. I haven't had a chance to sit down on this because it's just not pressing for my project, but it would definitely be a great learning project if you want to take on it!


add notes on handling pbs only bidders

Documentation updates related to https://github.com/prebid/Prebid.js/pull/2704
Add notes about how to handle PBS only bidders for anyone using PBS.

CC @dbemiller

Thanks for the feedback @bretg, I have made the changes you recommended.


[Updater] Add a test for two overlapping groups

This is a quick unit test that verifies two things:

That we correctly parse N dependency_groups to perform two group update attempts
That if two groups overlap, we will PR the same dependencies in both by design

Ah, the test failure is a known issue, this can merge downstream of https://github.com/dependabot/dependabot-core/pull/7216 and the problem will go away.


Support Go 1.19?

A couple of brew packages fail to build on Go 1.19 in https://github.com/Homebrew/homebrew-core/pull/107165 due to
github.com/lucas-clemente/quic-go@v0.27.0/internal/qtls/go119.go:6:13: cannot use "The version of quic-go you're using can't be built on Go 1.19 yet. For more details, please see https://github.com/lucas-clemente/quic-go/wiki/quic-go-and-Go-versions." (untyped string constant "The version of quic-go you're using can't be built on Go 1.19 yet. F...) as int value in variable declaration

Would that be fixed by upgrading to 0.28 after https://github.com/lucas-clemente/quic-go/pull/3460 or is 1.19 still unsupported?

v0.28 builds with 1.19, and there were no changes regarding 1.19 support between that version and the final release.

@marten-seemann sorry for the issue-spam. It seemed best to link here in case the same question comes up again.


Add "tap tempo" to delay

This lets you configure a note which you can tap to set the delay.
Fixes #22.

it would be nice if it was a little "forgiving" so that if I stomp along with the tempo for a while it sort of "averages it out"

@helixbass Yeah, that makes sense. I'd probably want to talk that through with you before trying to tackle that. I'll file an issue.


Improve the text at the top of the home page

Received the latest home page text from the product team.

accpt


sync up make release

The manifests for 0.13.0 needs to be updated.

/cc @njhale

Thanks for contributing your time to help improve OLM @gyliu513!
We don't update the manifests for versions of OLM that have already been released. If you feel it's necessary, please open an issue with the request and we can cut a new release with the updates.
Note: As these manifests are for OCP, and OLM is installed by default in OCP > 4.1 via a different set of manifests, we also don't think many OCP users will be applying them directly anyway.
Closing this out.


add(plugins): tailwind-scrollbar-hide

Name
Link

tailwind-scrollbar-hide
_https://github.com/reslear/tailwind-scrollbar-hide_

tailwindcss plugin for visual hide scrollbar

[x] My item is in the right category
[x] My item is logically grouped below similar items
[x] My item's name and description respects the conventions of the list
[x] My item is awesome
[x] I have read and followed the contribution guidelines

Thanks!


Object inspector keyboard control behavior

From<EMAIL_ADDRESS>on 2010-08-12T10:29:49Z
Hi,
should the object inspector show the function parameters as well?
I sometimes encounter the problem, that I want a detailed description of a function. So I type in my function open the bracket and get a tool-tip with all arguments. Now I want some detailed information about the function and trigger the object inspector. The tool-tip with the function parameters disappears and I have to delete the open bracket and type it again.
A short-cut to trigger the object inspector on a function would be helpful as well. COnsider the following situation:
return_list = some_function_1() #this function returns a big list with 20 entries
some_function_2(return_list[?], return_list[?]) #this function need some entries of the previous list, but which ones? It would be very helpful to go with the cursor to the first function, trigger the object inspector and read the information about the return value, than go to second function trigger the inspector again and have a look at the input parameters. If this is a large list as I have, some switching backs are necessary.
The next question is where to set the focus when triggering the object inspector. For small documentation it would be helpful to leave the focus on the editor, for a hugh documentation it would be helpful to set the focus on the object inspector (to scroll the help). This could be done again by adding a setting to prefernecs, that the user can decide for the behaviour.
One last wish about the object explorer. It would be helpful if you could mmove the cursor in the explanation as well as there are often code snippets and it would be helpfult to copy these to the editor. I would suggest the following behavour. When hitting CTRL+ALT+I the focus is set to the combo box to scroll through the history (as current) when hitting tab the focus is set to the text field to browse and copy some documentation.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=289

From pierre.raybaut on 2010-10-26T05:10:14Z
The function parameters issue has been submitted in another issue.
Summary: Object inspector keyboard control behavior
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-Low

From pierre.raybaut on 2011-05-16T01:08:13Z
Labels: Cat-ObjectInspector

From pierre.raybaut on 2012-03-18T14:04:05Z
Labels: -Type-Enhancement Type-Enh

From ccordoba12 on 2014-08-23T13:01:05Z
This has been worked out a lot during the years.
Status: Done


Dependency analyzer stuck at "Begin database maintenance"

Hi @jeremylong ,
I am running the depedency analyzer by configuring a local Oracle DB. I modified the code to run the "initialize_oracle.sql" to create all the tables , which is getting generated correctly, but it is stuck at "Begin database maintenance" and simply does not execute further at all. The logs show no error or exception. It is there more to be done to configure a local DB?
Attaching the logs for your reference.
dc_sample.log

I replaced :
final PreparedStatement ps = getPreparedStatement(CLEANUP_ORPHANS);
in cleanupDatabase() method in CveDB with
final PreparedStatement ps = connection.prepareStatement("DELETE FROM cpeEntry WHERE NOT EXISTS (SELECT * FROM software WHERE cpeEntry.id = software.CPEEntryId)");
and it worked. Not sure how that can work.

I tried your suggestion.
I purged. I initialized the Oracle DB with initialize_oracle.sql and then i ran the dependency-check with the below command
dependency-check --updateonly [all Oracle params]
All the downloads and records creation works fine until it reaches the "Begin database maintenance."
It is stuck there for over 1 hr now.
Does the query take so long to clean up the records?
Also, there is no /opt/dependency-check/data created yet

So the explain plan report for this query shows
|   0 | DELETE STATEMENT    |		   |  8888 |  3454K|	41M  (2)|138:35: 25 |
I think the query
DELETE FROM cpeEntry WHERE NOT EXISTS (SELECT * FROM software WHERE cpeEntry.id = software.CPEEntryId)
is the right thing to do?

From a performance standpoint I've always been under the impression NOT IN was actually a faster performing query (see https://stackoverflow.com/questions/15959061/delete-records-which-do-not-have-a-match-in-another-table).
What does the explain call out for the NOT IN version of the query?
Also - were you just calling the update routine or were you also trying to analyze a directory? For testing purposes just use dependency-check.sh --updateonly.

I read that thread too. But it seems like this (NOT IN) does not work at all. I tried to run it in the sqldeveloper, even there it just keeps on running.
The Explain call that I mentioned -
| 0 | DELETE STATEMENT | | 8888 | 3454K| 41M (2)|138:35: 25 |
was for the NOT IN query that is there in the script.
But the query that I used -
DELETE FROM cpeEntry WHERE NOT EXISTS (SELECT * FROM software WHERE cpeEntry.id = software.CPEEntryId)
completes in 427 ms . It shows 0 rows deleted.
Also, I just tried the updateonly and did not perform the full scan.
Is it possible from your end to try the NOT IN query and see if the query hangs?
Thanks

I do not actually have an oracle database setup to test the integration. This code was added by a different contributor.

Is there any other way you suggest to resolve this?
I think I can use the query -
DELETE FROM cpeEntry WHERE NOT EXISTS (SELECT * FROM software WHERE cpeEntry.id = software.CPEEntryId)
to clear all the orphans which works correctly and is fast. Do you agree?

Hi @jeremylong
We are struggeling with the same problem after we migrated from mysql to postgres.
Now the database maintenance for dependency-check version 1.x (which we still need for backwards compatibility of some older projects) lasts for ten to twenty hourse.
Is there any solution for this?

@hemanthacharya I apologize that I'm just getting back to this.  Yes, the query you created looks correct. I will update the Oracle properties file.
@TorstenKruse I do not currently have a postgres DB setup.  Would the suggested query from @hemanthacharya work in postgres as well?
DELETE FROM cpeEntry WHERE NOT EXISTS (SELECT * FROM software WHERE cpeEntry.id = software.CPEEntryId)

Also, note that we are no longer supporting any of the old versions of dependency-check.  In fact, the data source used by the pre-5.0 release will be going away soon.  The NVD will no longer publish the XML data feed I believe sometime this fall.  5.0+ uses the JSON data feed.

@jeremylong yes, the other DELETE statement is much better. How can I switch to this in version 1.x?

We no longer support the 1.x version of dependency-check.  You would have to download the source, switch to the appropriate tag, update the properties file for you database, and recompile/deploy within your org.  Note that the data feed used by all version previous to 5.0.0 will be going away.  Please consider upgrading that environment.

Ok, so I'll try to upgrade the projects to a more recent version.


Collapsed/minimized sections re-opening on scroll

Issue Type: Bug
Create some cells
Define functions
Click on the little arrows next to the function definition to 'collapse' or 'minimize' them into a single line
Scroll until the cells with the collapsed functions are out of view
Scroll back so the cells are in view
The functions are no longer collapsed
Since I have functions which are long in lines of code, I collapse them so I can more quickly move back and forth in my code.
This hasn't always been the case. I can't pinpoint the update, But it happened in the last week or 2
Extension version: 2022.4.1000991945
VS Code version: Code - Insiders 1.67.0-insider (a3d53f76790d8c88f835afe85225ff6442703409, 2022-04-07T05:15:43.863Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No
Remote OS version: Linux x64 5.4.0-1059-aws

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1035G1 CPU @ 1.00GHz (8 x 1190)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
11.70GB (2.13GB free)

Process Argv
--crash-reporter-id 4285aea8-8024-49ba-97f2-9d60622a70f0

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: AWS

OS
Linux x64 5.4.0-1059-aws

CPUs
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz (8 x 3120)

Memory (System)
30.91GB (29.02GB free)

VM
0%

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30422396
pythontb:30258533
pythonptprofiler:30281269
vsdfh931:30280409
vshan820:30294714
pythondataviewer:30285072
vscod805cf:30301675
pythonvspyt200:30323110
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
wslgetstarted:30449409
pythonvspyt640:30438691
vscscmwlcmt:30465136
pynewfile477:30451556

I'm seeing that the function is open for a frame or two but then closes again, so not quite as bad as your issue, but hopefully enough to investigate and fix it. Not repro'ing in stable at all for me.

It still happens with the Jupyter extension disabled, so looks like an issue in core. Transferring.


Updated SDK compatibility with Spring boot 3x

List of changes made to the Java SDK:

Migrated necessary dependencies from Javax to Jakarta to support newer versions of Spring Boot.
Updated the pom file to include a new version 2.0.0, add a new contributor, and implement other minor changes.

This is some great work @Oluwatodimu 👏🏻👏🏻👏🏻

This is some great work @Oluwatodimu 👏🏻👏🏻👏🏻

Thanks a lot, I'm glad I could be of help. I also learned a lot from this endeavor
All that's left now is to publish the new library and update the Treblle docs.

Hey Todium,
could you update our readme and then I’ll have Kelvin pull in those chnages
in our offical docs on docs.treblle.com
We just need to know what changed readme wise .D
Thx,
Vedran
On Fri, 4 Aug 2023 at 11:01, Todimu Isewon @.***> wrote:

This is some great work @Oluwatodimu https://github.com/Oluwatodimu
👏🏻👏🏻👏🏻
Thanks a lot, I'm glad I could be of help. I also learned a lot from this
endeavor
All that's left now is to publish the new library and update the Treblle
docs.
—
Reply to this email directly, view it on GitHub
https://github.com/Treblle/treblle-spring/pull/2#issuecomment-1665277681,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEZN4FB6VF3RO4MBZOTFHTXTS27BANCNFSM6AAAAAA27WQHME
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Hey Todium, could you update our readme and then I’ll have Kelvin pull in those chnages in our offical docs on docs.treblle.com We just need to know what changed readme wise .D Thx, Vedran
…
On Fri, 4 Aug 2023 at 11:01, Todimu Isewon @.> wrote: This is some great work @Oluwatodimu https://github.com/Oluwatodimu 👏🏻👏🏻👏🏻 Thanks a lot, I'm glad I could be of help. I also learned a lot from this endeavor All that's left now is to publish the new library and update the Treblle docs. — Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEZN4FB6VF3RO4MBZOTFHTXTS27BANCNFSM6AAAAAA27WQHME . You are receiving this because you modified the open/close state.Message ID: @.>

on it

Hey Todium, could you update our readme and then I’ll have Kelvin pull in those chnages in our offical docs on docs.treblle.com We just need to know what changed readme wise .D Thx, Vedran
…
On Fri, 4 Aug 2023 at 11:01, Todimu Isewon @.> wrote: This is some great work @Oluwatodimu https://github.com/Oluwatodimu 👏🏻👏🏻👏🏻 Thanks a lot, I'm glad I could be of help. I also learned a lot from this endeavor All that's left now is to publish the new library and update the Treblle docs. — Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEZN4FB6VF3RO4MBZOTFHTXTS27BANCNFSM6AAAAAA27WQHME . You are receiving this because you modified the open/close state.Message ID: @.>

Hello Vedran,
Apologies for the late response, for some reason I didn't get a notification for your last message.
I have updated the readme file accordingly and left some helpful comments in the PR I created.
Please help review and merge.
I'm also looking at publishing the SDK, I tried to publish but the jar file is not getting created.
Currently looking into it. I look forward to hearing from you,
Thanks,
Todimu (you can call me Todi).


additional tests

Addresses #171. This brings in a bunch of tests that have tested the scalability of cbat. Note that they also introduce a dependency to run the test suite on gcc-8 for the loop unrolling pragma. This may be installed with:
sudo add-apt-repository -y ppa:ubuntu-toolchain-r/test
sudo apt-get -y update
sudo apt-get -y install gcc-8 g++-8

The tests involved include:

a hash function.
a solver for the n-queens problem. Currently set to n=4 for quick completion.
a 4 by 4 sudoku solver.
a comparison of of LSB calculations between two binaries. The first method uses debruijn sequences and the second method uses a for loop.

Okay I've removed the 128 bit code because it's too much for now. I've also added in a readme for debruijn that provides a brief summary of how it works. Finally, I've removed the gcc-8 dependency (should now work with any version of gcc!).

This is not ready to be merged. There are two outstanding issues (thanks @gltrost for pointing these out!):

nqueens has multiple solutions. All these solutions should be added as options so the test always passes.
Removing the gcc-8 dependency results, and using gcc-5 with a different set of pragmas instead results in the 8 and 16 bit debruijn loops being completely unrolled. However the 32 and 64 bit loops do not get unrolled completely.

I've unrolled the 32 and 64 bit loops manually for debruijn. The issue now is that there is an implicit loop in the 64 bit version with debruijn sequences. WP does not finish in a reasonable amount of time when manually unrolling the loop 64 times. GCC-8 does not have this problem. Since we're encountering performance problems with the 32bit version, I'm going to delete the 64bit version for now.

I'm getting SAT for Debruijin 64-bit. Am I doing something wrong here?

Oops I forgot to upstream my changes (they should be pushed now). I'm removing 64bit debruijn for the time being so we don't have a dependency on gcc-8. Note that this isn't ready to be merged; it needs to be rebased on top of #228

Okay, I've rebased onto master! Nqueens has two solutions (which are now both included).

I've added comments on the expected output to each run_wp* script!

Looks good to me! I think wp/resources/sample_binaries/hash_function/.run_wp.sh.swp was accidentally checked in, but other than that, I think it's all good! Does @gltrost have any more input?

Oops. Should be unstaged now :+1:


Addition of startIndex to allow paging

I am working with large result sets and by passing in the startIndex you can go through multiple pages using your wrapper.

Nice one - cheers! :+1:


Metric Beats and Filebeats collide with hostNetworking enabled.

Chart version: 7.10
Kubernetes version: 1.19
Kubernetes provider: E.g. GKE (Google Kubernetes Engine) Docker-Desktop
Helm Version: 3
Describe the bug:
Using metricbeat and filebeat helm charts with hostNetworking enabled collide on ports.
Steps to reproduce:

Helm install metricbeats with hostNetworking set to true
Helm install filebeat with hostNetworking set to true
Filebeat will fail to setup it's HTTP Server on port 5066.

Disabling the hostNetworking feature on either of the helm charts s
Expected behavior:
Default values for hostNetworking should not collide.
We are integrating with elastic cloud and want auto-scaling daemonsets for our kubernetes nodes for metrics and logs.
I suggest the default values for one of the ports should change.
Provide logs and/or server output (if relevant):
kubectl logs filebeat-filebeat-qtb4b
2020-12-21T21:03:23.799Z        INFO    instance/beat.go:645    Home path: [/usr/share/filebeat] Config path: [/usr/share/filebeat] Data path: [/usr/share/filebeat/data] Logs path: [/usr/share/filebeat/logs]
2020-12-21T21:03:23.799Z        INFO    instance/beat.go:653    Beat ID: 512fcc30-bdf3-4b6a-a875-8c219a5b402e
2020-12-21T21:03:23.799Z        INFO    instance/beat.go:412    filebeat stopped.
2020-12-21T21:03:23.799Z        ERROR   instance/beat.go:956    Exiting: could not start the HTTP server for the API: listen tcp <IP_ADDRESS>:5066: bind: address already in use
Exiting: could not start the HTTP server for the API: listen tcp <IP_ADDRESS>:5066: bind: address already in use

I believed that I have solved this by adding the following the filebeat values file.
filebeatConfig.http.port: "5607"
However in elastic the host.name is set to the name of the pod while kubernetes.host.name is the actual desired name.

Note: I have found a work around which is to modify the filebeatConfig.filebeat.yml to include http.port as 5067.
filebeat.inputs:     - type: container       paths:         - /var/log/containers/*.log       processors:       - add_kubernetes_metadata:           host: ${NODE_NAME}           matchers:           - logs_path:               logs_path: "/var/log/containers/"     http.port: "5067" hostNetworking: "true"

Note I have found a work around which includes changing the filebeat port.
filebeatConfig:
  filebeat.yml: |
    filebeat.inputs:
    - type: container
      paths:
        - /var/log/containers/*.log
      processors:
      - add_kubernetes_metadata:
          host: ${NODE_NAME}
          matchers:
          - logs_path:
              logs_path: "/var/log/containers/"
    http.port: "5067"
hostNetworking: "true"

Hi @kwsorensen,
The default http.port for Filebeat and Metricbeat aren't managed by these charts but by Filebeat and Metricbeat config.
The workaround you used is documented in Metricbeat doc:

By default, metrics are exposed on port 5066. If you need to monitor multiple Beats shippers running on the same server, set http.port to expose metrics for each shipper on a different port number:
http.port: 5067

However, this could also be mentioned in the charts README (see Filebeat FAQ & Metricbeat FAQ.
Would you be interested to create a PR for updating the doc?

Created a PR  to update the documentation for hostNetworking enabled on multiple beats agents.


Changes Athena to UTD Trends

Changes Athena on homepage to UTD Trends from Rebuild Home Page #28

This was handled by a previous pull request #42.


Top Collaborators - Total Time

The Calculation details for 'Total Time' in the top collaborators section is not explained.
This information is useless if I don't know what it is.

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

ID: c80bdb40-6f61-b9e4-0098-5354d399f083
Version Independent ID: 56239348-0977-4220-c8ab-40ab1e7f33a0
Content: MyAnalytics network page - Workplace Intelligence
Content Source: WorkplaceAnalytics/MyAnalytics/Use/network.md
Product: mya
GitHub Login: @madehmer
Microsoft Alias: v-pausch

Hi R-Catling,
Total time is defined in the About the metrics section on this page. Do you need more exact calculation details for this metric than this definition provides?
Thank you,
-- paul9955


Null reference exception thrown when null is returned from HandleAuthenticateAsync()

This used to work in RC
        if (string.IsNullOrEmpty(header) || !header.StartsWith("Bearer "))
        {
            return null;
        }

It causes a null reference exception in RC2. The correct code would return Task.FromResult(AuthenticateResult.Skip()); but the exception is horrid.

I reported a similar bug (https://github.com/aspnet/Security/issues/760) and AFAIK, it's fixed in RC2.
Are you sure you're not returning a null Task?

He is.
On Mon, May 23, 2016 at 11:40 AM, Kévin Chalet<EMAIL_ADDRESS>wrote:

I reported a similar bug (#760
https://github.com/aspnet/Security/issues/760) and AFAIK, it's fixed in
RC2.
Are you sure you're not returning a null Task?
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Security/issues/837#issuecomment-221051976

Then it's not a bug.
@blowdart there's a huge difference between returning a null Task and a Task that returns a null result (e.g via Task.FromResult<AuthenticateResult>(null)): trying to await a null Task will always result in an NRE.
BTW, it's absolutely not specific to this method and impacts almost every async method in the .NET Framework or in ASP.NET Core.

I was just returning null, not even a task :) Meh, fair enough, my misunderstanding :D

Meh, fair enough, my misunderstanding :D

Noob noob noob :trollface:

Just clarifying that this wasn't actually changed and tagging as 'wontfix' to ensure we're covering things right for RTM


fix: use environment file to manage outputs

Since  ::set-output is now deprecated

@bogaertg @ThibDujardin any news about the merge of this PR?


[doodstream] add new extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
Support for the DoodStream video hosting site.
An updated version of #26345. Normally I'd add all the changes there, but because of septembers removal, my repo is in this zombie-like state and I think it's better to just start over.
Closes #26345.

This looks like copyrighted content. youtube-dl does not support sites dedicated to copyright infringement. Provide an example URL that does not violate any copyrights.

I added some fixes, including the test, to their pull request (see https://github.com/sxvghd/youtube-dl-1/pull/1).

Guys, a question, if you were going to download the doodstream link in python how would you do it. I've tried with wget, with open() turning it into 'wb', with youtube_dl it doesn't work. I don't know how else I can get it. I already checked if I could use .m3u8 capture or something similar in the request and nothing worked.

Guys, a question, if you were going to download the doodstream link in python how would you do it. I've tried with wget, with open() turning it into 'wb', with youtube_dl it doesn't work. I don't know how else I can get it. I already checked if I could use .m3u8 capture or something similar in the request and nothing worked.

@R1tzy
take a look at Kodi's addon script.module.resolveurl


What is the best way to use pydata_sphinx_theme in Read the Docs

Love your theme - but I cannot deploy it on read the docs.
I have tried :-
import os
import sys
sys.path.append(os.path.abspath('../'))
import pydata_sphinx_theme
...
html_theme = "pydata_sphinx_theme"
html_theme_path = pydata_sphinx_theme.get_html_theme_path()
which works well on my windows 10 box if I put the pydata_sphinx_theme directory at the same level as my docs directory - even after uninstalling pydata_sphinx_theme from my computer.
cheers
Edgar Donk

huh that's weird, I know a number of projects that use it with readthedocs...what does RTD show for your theme?

I know - tried a few other ways but no success
Anyway rtd gave me this
Theme error:
no theme named 'pydata_sphinx_theme' found (missing theme.conf?)
which was strange - since it worked on my machine without pydata_sphinx_theme being installed as normal

Maybe if you could give me a couple of links to those that are working I could copy their methods
Thanks

ah - that error makes me think it isn't installed in readthedocs, even if it is configured properly. Can you double check the installation instructions for RTD and make sure the theme is in there

I am cloning it from Github, does that affect it?
I probably have missed out an important step - so I'll have a look at RTD from scratch

First I added a webhook.
I added readthedocs.yml and requirements.txt to the github files - not required in my computer, then got the message 👍
Running Sphinx v3.0.4
loading translations [en]... done
making output directory... done
/home/docs/checkouts/readthedocs.org/user_builds/electronic-python/checkouts/latest/docs/conf.py:54: RemovedInSphinx40Warning: The app.add_stylesheet() is deprecated. Please use app.add_css_file() instead.
app.add_stylesheet('custom.css')
Traceback (most recent call last):
File "/home/docs/checkouts/readthedocs.org/user_builds/electronic-python/envs/latest/lib/python3.7/site-packages/sphinx/theming.py", line 76, in init
inherit = self.config.get('theme', 'inherit')
File "/home/docs/.pyenv/versions/3.7.3/lib/python3.7/configparser.py", line 780, in get
d = self._unify_values(section, vars)
File "/home/docs/.pyenv/versions/3.7.3/lib/python3.7/configparser.py", line 1146, in _unify_values
raise NoSectionError(section) from None
configparser.NoSectionError: No section: 'theme'
During handling of the above exception, another exception occurred:
.....
Theme error:
theme 'pydata_sphinx_theme' doesn't have "theme" setting
This was the same error when I added files gitignore, package.json, pre-commit-config.yaml, setup.config, setup.py, webpack.common.js, webpack.dev.js, webpack.prod.js, yarn.lock
Surely some if not all of the above are not required?
I must admit that I'm none the wiser whether pydata_sphinx_theme is installed or not.

I'd check out here for how to install dependencies in RTD: https://docs.readthedocs.io/en/stable/guides/specifying-dependencies.html?highlight=environment#using-a-conda-environment-file

I decided that I was doing something fundamentally wrong by including the pydata_sphinx_theme, just
putting it and sphinx in as requirement. I included the same setup.py and setup.cfg files as in the
pydata_sphinx_theme master.
However the RTD failed with 👍
ERROR: Command errored out with exit status 1:
command: /home/docs/checkouts/readthedocs.org/user_builds/electronic-python/envs/latest/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-_usoxvrk/setup.py'"'"'; file='"'"'/tmp/pip-req-build-_usoxvrk/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-urn6qjb2
cwd: /tmp/pip-req-build-_usoxvrk/
....
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/pip-req-build-_usoxvrk/pydata_sphinx_theme/init.py'
weird!

you shouldn't need to be including the pydata sphinx theme setup.py or setup.cfg commands, it should be enough to include pydata_sphinx_theme in the requirements file for RTD

@Edgar-Donk if you remove the setup.py file, it should work (https://github.com/Edgar-Donk/Electronic-Python)
The reason is that if the setup.py/setup.cfg file is there, readthedocs thinks it is a package to install. But it cannot run the setup.py from within your doc repo.
But I think you will also need to update the readthedocs.yml file to indicate that you don't need to pip install the repo as a package (as you only have docs, AFAIU): https://github.com/Edgar-Donk/Electronic-Python/blob/master/readthedocs.yml

See here: https://docs.readthedocs.io/en/stable/config-file/v2.html#configuration-file-v2. I think you can leave out the - method: pip part in the yml file, and only have the - requirements: docs/requirements.txt in the install section

Thanks for your patience, got there in the end, there was a problem with readthedocs.yml, which is odd as I am sure that I copied it from a working one in jupyterhub, with directory change for conf.py
version: 2
sphinx:
configuration: docs/conf.py
python:
version: 3
install:
- requirements: docs/requirements.txt
As I said the result looks good, just have to check it is all there
https://electronic-python.readthedocs.io/en/latest/
cheers Edgar Donk

Good to hear it is solved!


Comment script event

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Adding a comment script event.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:

@ManuGamesDev nice, there was actually an event in place already for this purpose called "Notes" that worked pretty much the same way. I named it that way for consistency with the notes fields on Scenes/Actors/Triggers but actually "Comment" is probably more likely what people would expect it to be named.
I like the highlight colour you've added to yours too, makes it stand out much more.
What do you think, should we just remove eventsNotes.js and use this version instead?

@chrismaltby to be honest i didn't know that there was something like that already in gb studio lol people on the discord told me there was actually a notes event already, but they also said they liked my comment one better. So yes, I think we should replace it with notes

I agree, I've updated your pull request to remove the Notes event and merged that into develop now. Thanks!


When writing a test for my flux store, the store is undefined.

Hi all
I am trying to write tests for my project that uses Flux. The following is my code for a simple test using one of my stores:
jest.dontMock('../driverStore');
jest.dontMock('object-assign');

describe('driverStore', function() {

    var driverStore;

    beforeEach(function() {
        driverStore = require('../driverStore');
    });

    it('The store initializes with no data', function() {
        var all = driverStore.getList(); //This throws the error
        expect(all).toBe({});
    });
});

However the following error is thrown:
TypeError: Cannot read property 'getList' of undefined
Could you possibly tell me what I am doing wrong? For some reason I can't require the module. My jest-cli is 0.4.3. Thank you so much!

Hello. Would you mind producing a simple test case or, if that is not possible, share all your store logic so I can reproduce and investigate this issue?
If not, then I'll assume this is a problem with your code and unrelated to jest :)

Hi cpojer, it's probably  a problem with my code :) Here is the store logic:

//Imports
var AppDispatcher = require('../dispatcher/AppDispatcher');
var appConstants = require('../constants/appConstants');
var objectAssign = require('react/lib/Object.assign');
var EventEmitter = require('events').EventEmitter;
var CHANGE_EVENT = 'change';

//Store
var _store = {
    driversInBranch: [],
};

//Public Functions
var driverStore = objectAssign({}, EventEmitter.prototype, {

    addChangeListener: function(cb) {
        this.on(CHANGE_EVENT, cb);
    },
    removeChangeListener: function(cb) {
        this.removeListener(CHANGE_EVENT, cb);
    },
    getList: function() {

        return _store.driversInBranch
    },
});

module.exports = driverStore;

The issue is likely that objectAssign is mocked, therefore driverStore never receives those methods. You should unmock it by default.

That fixed it thanks!
Added:
jest.dontMock('react/lib/Object.assign');

to the top of of my test and it worked :)


Fix complex valued svd backpropagation.

The original svd_grad function doesn't work for complex valued matrices. Have fixed it according to the discussion in https://github.com/tensorflow/tensorflow/issues/13641.
I will add more test.

This PR is ready to be merged. Anyone wants to review it? @j-towns

This looks good to me. Thanks a lot!


Dioxus template crashed

When I am trying to run, cargo android run, Dioxus template crashed.
I understood that it updated a bit.
Please kindly help me.
Best Regards

Do you have any backtrace? or error logs?

Thank you for your attention @amrbashir
Yes.
Only wray template working.
If I use another template, it didn't working.

I did like this.

cargo new audiocardio_mobile_demo --lib
cd audiocardio_mobile_demo
cargo mobile init
And then, I selected Dioxus template.
cargo android open
After that, run the emulator.
cargo android run
but, it showed white page.

I changed some UI but it still showed white page.

Thank you for your attention.
Please help me..


Add support for width for Bar series graph

What problem does this feature solve?
Currently, there is no option that allows to set the width of the bar graph, whereas this is available for almost every other series type.
The aim here is to allow the user to explicitly define the width of the bar graph, in order to remove the excessive padding in the bar graph, which is just unused space.
This also removes the inconsistency existing when a datazoom component is added, since we can customise the width of the datazoom slider, but the same cannot be done with the bar graph.
What does the proposed API look like?
We need to use a width prop.

Hi @sreetamdas
Bar is used on top of grid component. And you can use width/height/left/right/top/bottom layout properties in the grid component to control the padding.

@pissang thanks a lot! Sorry but I just hadn't thoroughly gone through the documentation and missed this. This solves my use-case perfectly. Thanks a lot again!


Migrate from React.PropTypes

Hello,
I think you're doing fantastic work in this repo, thanks for that! Just a small problem I noticed as I was reading the new release notes: As of React v15.5.0, PropTypes now exist in a different package,
prop-types. React.PropTypes is deprecated.
Since using React.PropTypes gives a deprecation warning I think we should encourage using the new way.
Let me know what you think.
-- fvj

@fvj Damn that was lightning quick 🎉
Thanks for the Pr :)

You're welcome. :)


Replacement for https://www.instagram.com/{username}/?__a=1

Does anybody have a way to get the bio, media URLs and also the users who like the media?

All the necessary information is in the _sharedData variable (javascript) on the page https://www.instagram.com/{username}/ you only need to extract her (use regular expression).
This library (instagram-php-scraper) already knows how to do this (getAccount method)

Most likely this will not work

@mr-molodoy Is there any other way to get a list of likers? Even if it's not only 1 request? Is 50 the max amount?


[MINOR][TEST] Use intercept instead of try-catch to assert failures in IntervalUtilsSuite

What changes were proposed in this pull request?

In this PR, I addressed the comment from https://github.com/apache/spark/pull/27672#discussion_r383719562 to use intercept instead of try-catch block to assert  failures in the IntervalUtilsSuite
Why are the changes needed?

improve tests
Does this PR introduce any user-facing change?

no
How was this patch tested?

Nah

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23689/
Test PASSed.

Test build #118941 has started for PR 27700 at commit 65b23fd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23690/
Test PASSed.

Test build #118941 has finished for PR 27700 at commit 65b23fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118941/
Test PASSed.

I feel its better to file jira as improvements.

@maropu thanks for your suggestion, I have filed and linked the Jira.

Test build #118961 has started for PR 27700 at commit d79a2d9.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23709/
Test PASSed.

Test build #118961 has finished for PR 27700 at commit d79a2d9.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118961/
Test FAILed.

retest this please

Test build #118974 has started for PR 27700 at commit d79a2d9.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23722/
Test PASSed.

Test build #118974 has finished for PR 27700 at commit d79a2d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118974/
Test PASSed.

Thanks! Merged to master/3.0.


Automatic version update

Merging this through, as I'd like to propagate a change I've made in Maestro.


MM-4712 Add progress bar to uploads

Summary
Add progress bar to uploads as per spec
Ticket Link
(MM-4712)[https://mattermost.atlassian.net/browse/MM-4712]
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)
[x] Has UI changes
[x] Includes text changes and localization file (.../i18n/en.json) updates

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0af0af71092871498.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0af0af71092871498

@esethna Fixed 1,2,3
Re: 4. That should be because of loading pdf icon for the first time .
If we cannot detect the file we present another generic default icon so there wont be a case of having empty space there

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0506b46bb96c6c932.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0506b46bb96c6c932

@esethna Changes done.

@hmhealey @sudheerDev what's next steps on this PR?

@hmhealey @sudheerDev what's next steps on this PR?

Waiting for HH to go through the changes


MPAverageColor C99 dialect probs

Hi there, maintainer of cordova-mixpanel-plugin here.
Some users are experiencing issues with building their code which boils down to usage of C99 dialect at the following point in code:
UIImage+MPAverageColor.m:
char colorIndices[kNumberOfHexColors] = {0};

Some of them are able to change it manually from xcode itself but some have no solution as they are building using a cloud platform or are just having a dialect collision with another lib they are using.
Are you using C99 intensively?
if not, would you consider changing to something like this:
char colorIndices[kNumberOfHexColors];
memset( colorIndices, 0, kNumberOfHexColors*sizeof(char) ); //or colorIndices[0] = 0;

it should still work but would make the code more "vanilla" i.e. more compatible with other libs code.

Hi @samzilverberg. I'd be happy to make that change. Do you know if that's the only place C99 is causing issues for you? I don't know off the top of my head where we're using C99 and would like to fix the issue for you in one release.

as of a couple of months ago, this was the only place which was using C99 syntax.
some users have manually changed the code back then for their own projects and got it to compile.
nowadays i'm not sure.
you could just give it a try i guess, something like:

compile/build the lib with a xcode build flag to use a stricter dialect
see it fail because of the current code
change and compile again
success = no more usage, fail = you'll see the other C99 usages

would you mind trying that? i'm pretty sure it will succeed on the first try :)

Ah thanks. Looked like it was using GNU syntax which caused an error when compiling on C99. I wasn't able to test what happened with C89 because a lot of stuff like NSAssert couldn't compile. Anyway, I opened a PR which should fix it.

Hey @samzilverberg just merged the fix into master. If you can't wait, I'd suggest pointing to that for now. Otherwise, we'll probably release a new version of the sdk in a few weeks that will include the fix.

ty. I'll wait for the official release.

Fixed in the latest release.