1.4.0 makes my rails app unable to sign in with facebook

rails 4.1.13
devise 3.5.2
omniauth (1.2.2)
omniauth-facebook (2.0.1)
1.3.1 was fine
Error:
(facebook) Authentication failure! invalid_credentials: OAuth2::Error, :
{"error":{"message":"Error validating verification code. Please make sure your redirect_uri is identical to the one you used in the OAuth dialog request","type":"OAuthException","code":100,"fbtrace_id":"GjHr4Inn5Rq"}}

I'm getting the same error with omniauth-google-oauth2, after upgrading to 1.4.0.
(google_oauth2) Authentication failure! invalid_credentials: OAuth2::Error, redirect_uri_mismatch:
{
  "error" : "redirect_uri_mismatch"
}

:+1 here with google. getting redirect_uri_mismatch as well

I guess it has to do with this commit:
https://github.com/intridea/omniauth-oauth2/commit/26152673224aca5c3e918bcc83075dbb0659717f

I'm getting the same problem.
"message":"Error validating verification code. Please make sure your redirect_uri is identical to the one you used in the OAuth dialog request","type":"OAuthException","code":100,"

Yes it's because of https://github.com/intridea/omniauth-oauth2/commit/26152673224aca5c3e918bcc83075dbb0659717f by @sferik . The redefinition of callback_url was deleted from OmniAuth::Strategies::OAuth2 while it was used in the strategy to get redirect_uri, which should be the same redirect_uri which we sent to facebook without params (code, etc).
You can use 1.3.1 version of the gem before the bug would be fixed.

+1 to 1.3.1 temporarily fixes the issue

i struggled with this for about a week before downgrading to 1.2!
This should be fixed asap! It was an absolute nightmare!

Yeah same here!
I got it working again by changing my Gemfile to:
gem 'omniauth-oauth2', '~> 1.3.1'

it seems that version 1.4.0 works fine if you include provider_ignores_state: true like in this example:
https://github.com/joseairosa/codename-home/blob/73333bde3744d23c361cfcec6f5f1201297fef09/config/initializers/omniauth.rb

What does provider_ignores_state do?
Any doc?

unfortunately, google oauth2 doesn't work even with provider_ignores_state: true

maybe a simple fix like this would work: https://github.com/intridea/omniauth-oauth2/pull/82

I lost 2 days of work trying to track down why my custom provider was returning an invalid_grant error. This really should have been checked more before being merged.

According to Section 3.1.2 of the OAuth 2 spec:

query component…MUST be retained when adding additional query parameters

I’m sorry implementing this part of the spec has caused some OAuth providers to break. Gems for such providers should specify their omniauth-oauth2 dependency like this (until we can find a better solution):
spec.add_dependency 'omniauth-oauth2', '~> 1.3.1'

I fix using: gem 'omniauth-oauth2', '~> 1.3.1'

I test the gem 'omniauth-oauth2', '~> 1.3.1' fix, working localy not in production.
Somebody for more infos on that ?

Ok just figure out some configuration not deleted when trying to fix the problem myself.
So using 1.3.1 version is the good fix.

I'm noticing this with my custom strategy for a doorkeeper backend. I understand that it is not following the spec, but it should probably be documented how to override it since even doorkeeper breaks it.

Same issue with the Office 365 adapter too. Again forcing a downgrade to 1.3.1 in my GemFile.lock solved it.

So, what is the suggested way to fix this? Should I fix that in my omniauth-dropbox-oauth2 gem by changing the definition for callback_url or should I wait for this to be fixed?

Wouldn't it be better to have this change activated by some kind of flag? It has proved to be non backwards compatible and since there's no changelog and the project does not follow semver it is pretty hard to detect something like that might happen.
Would a contribution in that path be welcome?

Should callback_url be overridden by specific omniauth strategy?
Maybe some providers require the original definition so that's why this should be removed / moved to specific strategy gems?
Example: omniauth fix this in 3.0.0

Should callback_url be overridden by specific omniauth strategy?
Maybe some providers require the original definition so that's why this should be removed / moved to specific strategy gems?

I agree with you, but making a change which breaks backwards compatibility without any deprecation notice or changelog is not the best either and breaks a lot of stuff.

We encountered the same problem with an app that authenticates with Cloud Foundry's UAA. The UAA rejects the input because the callback path is malformed because, in effect, the query-param starting ? appears twice:
client_id=example_app&client_secret=&code=example-code&grant_type=authorization_code&redirect_uri=http://localhost/3000/auth/oauth2/callback?code=some_code&state=state_token

(The actual payload is urlencoded, I decoded it for clarity)
The change in https://github.com/intridea/omniauth-oauth2/commit/26152673224aca5c3e918bcc83075dbb0659717f means that the base strategy is used, which appends the query params. But it simply appends them, rather than trying to merge them.
While it is the case, as @sferik points out, that s3.1.2 of the RFC requires query parameters to be retained, it seems to me that 3.1.2 could be satisfied by merging the parameters, rather than physically appending them in a way which seems to break very interactions with upstream authentication servers.

Same problem with rails + omniauth + doorkeeper, fixed by downgrading to 1.3.1

Same problem here.
Ruby/Rails spit out this error whenever it tried to authorize the callback code:
"invalid_request: Missing required parameter: code {"error":"invalid_request","error_description":"Missing required parameter: code"}"
Downgrading it to 1.3.1 fixed it.

Could somebody make a big note on the "README.md" for those unaware of this issue until it is resolved? =)
Thanks

Yet another me too - just spent a few hours tracking this down with Windows Live Oauth failing - same issue, the redirect_uri fails to match if the query string params are included

@samuraraujo Thanks. That fixed my problem as well.
gem 'omniauth-oauth2', '~> 1.3.1'

1.3.1 works for me, thanks.

The 1.3.1 fix won't work for rails 5.0.0.beta1 due to differences in rack versions,
~/root/sides/herds(branch:master*) » bundle                                                                                gokul@noob
Fetching gem metadata from https://rubygems.org/..........
Fetching version metadata from https://rubygems.org/...
Fetching dependency metadata from https://rubygems.org/..
Resolving dependencies....
Bundler could not find compatible versions for gem "rack":
  In Gemfile:
    rails (< 5.1, >= 5.0.0.beta1) ruby depends on
      railties (= 5.0.0.beta1) ruby depends on
        actionpack (= 5.0.0.beta1) ruby depends on
          rack (~> 2.x) ruby

    rails (< 5.1, >= 5.0.0.beta1) ruby depends on
      railties (= 5.0.0.beta1) ruby depends on
        actionpack (= 5.0.0.beta1) ruby depends on
          rack-test (~> 0.6.3) ruby depends on
            rack (>= 1.0) ruby

    omniauth-oauth2 (~> 1.3.1) ruby depends on
      oauth2 (~> 1.0) ruby depends on
        rack (~> 1.2) ruby

Is there a plan to actually fix this given the Rails 5.0.0 incompatibility?

Seems like the decision is to have downstream gems make the required change as per https://github.com/intridea/omniauth-oauth2/pull/82.
Could have done better with deprecation, but I think we can settle this now and start opening PRs downstreams.

I commented on the closed PR #82, but just wanted to add that this was another several hours wasted here.

Just spent a good couple days trying to find the root of my error. Finally boiled it down to this.

+1 redirect_uri_mismatch
It was fixed when I downgraded to 1.1.2

Do we have a rails 5 fix?

Most omniauth strategy plugins (some of which are a few years old, but still working) do not stipulate a dependency on a specific version of omniauth-oauth2, just on omniauth itself. That's the main reason this has broken everybody's app - now each app maintainer has to stipulate that dependency themselves.
I'd consider rolling this bugfix back and making a minor release, which will "miraculously" fix all broken users of this gem (applications) upon their next deploy/bundle install invocation.
I get that it's a bug/spec fix, but those who would benefit from it are a far smaller audience (to my untrained eye) than those who would be harmed. Those who need this bugfix could benefit from a branch of this gem until a way can be found to integrate the fix into master without breaking the various omniauth strategy gems floating around (or until most release a newer version which does whatever this bugfix requires it to do).

@sferik is there not a better solution for this yet? 6 months later and I'm still running into this problem in oAuth Strategies.
How is this acceptable to just ignore?

I was able to fix the issue by restoring the callback_url method to my subclass of OAuth2
module OmniAuth
  module Strategies
    class MyStrategy < OmniAuth::Strategies::OAuth2
        ...

        def callback_url
           full_host + script_name + callback_path
        end

        ...

see  breaking change

It seems rediculous that this change occurred for ONE strategy and broke every other single strategy out there..    IMHO it would have made more sense for the one strategy that needed the query parameters to override callback_url in it's own strategy.

Is this a wontfix?

We're not planning on reverting, no. This was implemented a year and a half ago to adhere to the OAuth 2 spec.
I recommend reaching out to the specific gem providers you require for them to make the necessary updates.


Menu BG

What is your question?
Ayo! I would like to know how to change the background in the main menu. That yellow thing with BF and GF in it. If it contains code I'll take it. :>

set the custom image with the resolutions, and the characters in images/menucharacters if applicable. you can do this in the main debug menu in menu character editor and/or week editor

 set the custom image with the resolutions, and the characters in images/menucharacters if applicable. you can do this in the main debug menu in menu character editor and/or week editor

Does it have to be a certain length cause it's not working.

is there another way <:) oof...

You mean the menuBG an the variants ?
if yes, then you can just modify the images only remember that it needs to be the same resolution as the original images :D

in variants?? okay! Ill try! :)

okay ill try! :3

wait what images??? <:/

You mean the menuBG an the variants ? if yes, then you can just modify the images only remember that it needs to be the same resolution as the original images :D

I really can't find the images :( Does it go along the source code?

are you talkin about the main menu or story menu ?

are you talking about the main menu or story menu ?

its whenever the game firsts loads and you click enter then you see all the stuff like story mode, awards, freeplay, credits, that background

go to the images folder ("assets/shared/images" in 0.7.3) and look for "menuBG" and there you can edit the image and all the other variants like : menuBGBlue, menuBGMagenta, menuDesat.

Thank You!!! :D


Question : Blocks feature, functions and event handlers

Hi, as far as i can understand, all functions (forexample event handlers) are defined at the Page level. I understand this makes it easy for components to interact with each other and data sources.
Imagine a scenario where i have 5 different components that interact with each other. And then i want to provide this group of 5 elements as a block. None of the functions / interaction will be saved so it will be less than optimal.
Is there a workaround to this? There is a "ConfigProvider" component as i can see which is very similar to a Page. Maybe a way to define functions at the nearest configprovider level and saving the configprovider as a block?
Best regards,

actually there is a Component Type, works just like a react component, unfortunately our demo for this is not ready yet. and it`s not easy to describe how to implement that without a demo, we will provide that in about 2 - 3 months

Thank you @JackLian for the answer. Is there a roadmap of the features you're looking to implement in the future?

well, not really a roadmap, but we do have an article about what we are about to do in the next few months , but it`s in chinese, maybe you can try google translate it https://xie.infoq.cn/article/d91d4a66fc647a783b33efe39
if google tranlate is not working, here are some of the planning feature

application-level editor
lowcode component (component implemented in lowcode)
flow editing, ( with x6 an g6 embeded)
i18n
theme support
playground for lowcode engine
a free lowcode-platform ( more like a more complete demo with backend services, implemented with lowcode engine)
...


Accordion content blinking

Facing blink issue while open accordion content from header.
https://github.com/oblador/react-native-collapsible/assets/140066955/45284153-c656-41a5-91f1-405122f4fc13
Facing this with RN new architecture.

reproducable

same

@meghvars
try this
renderChildrenCollapsed={section.length < 1}

Fixed in 1.6.2


Milestone One Assignments

Gabe: Team Contract
Gloria: Code of Conduct
Paniz:   Meeting issue, Dataset selection issue
Luming: Contributing, Project Structure
All: Forks and Pull Requests!
(Reference: https://www.tomasbeuzen.com/post/git-fork-branch-pull/)
To be decided:
Formalizing project proposal
Script for download
Literate Code Document (EDA)

Converted each of these to individual issues for better tracking!


Refactor to use type instead of name internally

It's somewhat confusing that token types are called names until they are emitted, so we should probably refer to them everywhere as types.
Tim has a bunch of cool PRs in the works right now that touch that code, though, so let's wait until those are merged.

We should do this right after #25.

FWIW, the reason I didn't do this is it seemed nicer to call a rule/group by its "name" than by its "type.

That is a good point. I suppose you could call it tokenType?


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.2.29 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


export design token file: Cannot read properties of null (reading 'r')

Description
Figma Version: 114.6.1

select "export design token file"
Press "Save & Export" button
See: modal doesn't close and an error code is printed into the console.

If you just go to "Design Tokens > Settings" that is the screen.

Here is the settings screen:

I don't think I can share the color styles publicly, though. the tokens are named like "Pink/pink-100" though

Hey @kaitlynhova any updates here? Does the problem persist? Can you create an example file with the issue so that I can test it myself?

Thank you for checking in! The problem has persisted but we found a work around by using a different Figma file that works for some reason 🙏

@kaitlynhova that is weird. And you have all colors in this new file as well? No color that was only in the old file that could produce this error?


tweak windows build scripts for GUI directory changes

@chrisnojima Assuming those symlinks are going away

i actually updated my PR again to remove the symlinks wanna test windows again

It built, thanks. But if you wouldn't mind this change, it will make the Jenkins production build work with these dir changes.


Adds feature to support global deprecation of exercises

Currently when exercises are no longer supported in any language tracks, each track that has implemented it should add it to the deprecated list in the config.json file so that it doesn't get delivered automatically when people exercism fetch exercises.
The exercise should not show up in the "to be implemented" list for the other language tracks
In order to do this,  an empty file named .deprecated will be added in the exercise directory of x-common.
For example:
It will look like this for a deprecated exercise:
$ tree -a exercises/point-mutations/
exercises/point-mutations/
├── .deprecated
├── description.md
└── metadata.yml
An exercise that is not deprecated will not have the .deprecated file. X-api  will need to make it so that if the .deprecated file is there, then the problem will not show up in the "todo" lists.
Changes are made on lib/xapi/problems.rb to filter out deprecated problems.

Thank you @ramyaravindranath, I will review this today.

Thank you @ramyaravindranath! Congrats on completing RGSoC, and good luck with your new job :heart: :heart: :heart:


fix: Change log level of observer and use dedicated function

Codecov Report
Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Project coverage is 48.84%. Comparing base (e93b66d) to head (03143e5).

Files
Patch %
Lines

cmd/principal/main.go
0.00%
11 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
+ Coverage   48.75%   48.84%   +0.08%
==========================================
  Files          52       52
  Lines        3366     3368       +2
==========================================
+ Hits         1641     1645       +4
+ Misses       1581     1578       -3
- Partials      144      145       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Is this repo abandoned?

Hi maintainers,
PRs to upgrade to FluentValidation 10 are still pending after weeks. Is there a problem with those or is this repo abandoned? I'm not asking to be rude/mean, I prefer this lib over the others for FluentValidation in Blazor, but the potential future lack of support might be a problem if I use this lib in my projects.
Thank you!

Looks like it...

Check out this fork that is keeping the project alive: https://github.com/pablopioli/FluentValidation.Blazor


Don't lookup config on every container-creation.

On profiling the invoker I saw, that the lookup of the config blocks the invoker-threads on the beginning of a burst.
The reason is, that the config is read on every container-creation. It is enough to read this configuration once.
I also verified, that the threads are not blocked after the change.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Good find @cbickel ! So with future changes we must be careful that we do not perform config lookups on critical paths


Private channels not appearing for vendors for the desktop Mac OS client

Private channels not appearing for vendors for the desktop Mac OS client. Only appears on the web application.

@AustinTByrd Could you please share a screenshot of the issue you are facing.

Please reopen with screenshots if you are still facing issues.


[Closed] Java program to find the sum of all numbers in an array is added

Why:

Closes #2261
What's being changed:
Java program to find the sum of all numbers in an array is added.

Okay sir, I will do that now


Added Total Ones

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added Total Ones
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#6414

@HarshCasper Sir Please review this pr and plz let me know if further changes are required.


The text input field moves up every time a key is pressed

Only on Firefox. When typing in the text input the text field grows upwards by one pixel. This is a regression, because I tested older versions (like this from a month ago) and it did not have this problem.

Only happens when the page is zoomed between 50% and 100% exclusively, therefore this is quite a low priority problem.


Implement pipeline unit tests

Device an implement tests for each of the main modules of the pipeline in order to easily test future changes.

@miguelglezb this is the other big chunck of work that is currently pending. For now, we can survive with the current tutorial-based implementation, but after the major refactoring of #475 we can come back to this.


added AWS c5.* m5.* instance types to all appropriate regions

Adds c5.* and m5.* for all AWS regions in which the instances are available.
Description
Since the original c5.* and m5.* were added to libcloud, they have become available in more regions.
There were some tests that asserted the length of various lists of instance types. In general, I've found these kinds of tests are not helpful because they simply test that the code says what the code says (and hence are quite brittle). I'm not sure there is a good test (beyond successful compilation) that a static data structure is correctly written.
(I'm also not clear on why the library maintains these static lists instead of querying AWS for the valid values, but I assume there's a good reason for that).
Status

done, ready for review

Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

@ZuluPro can you please review this change?

Hello @pquentin @yanowitz
You've seen I'm on the case ;)
To avoid to update listing manually, I created this package https://github.com/ZuluPro/python-ec2-sizes.
It parses the AWS pricing JSON (>200MB) and stores it as pickles, dicts and JSONs.
Do you think we could integrate it to libcloud (as third package or in contrib) ?

@ZuluPro It would make sense to do this before each release, and allow users to do it manually if they want to, yes. Is this what you were thinking about?

Hey @pquentin
Yep good idea !

@pquentin @alex Any news ?

@yanowitz We appreciate the pull request! But we're going to merge #1211 instead of this so that you never have to do this kind of pull requests again. :) Thanks.

That's a much better approach!


TypeError: Cannot read property 'image_compress' of null

Hello!
I have a project in React Native Expo managed.
I installed react-native-compressor as indicated in the manual, except for
expo prebuild
and
eas build
because I'm currently using Expo Go.
However, my application still doesn't recognize the Image component and throws the error
'Possible Unhandled Promise Rejection (id: 1): TypeError: Cannot read property 'image_compress' of null.'
What am I doing wrong?

any solutions?

🙏 sorry for a long ago I was not maintaining due to the huge office workload, hopefully, I am going to start fixing bugs on a weekend basis,
I will fix at least 2 issues on the weekend, for priority bugs you can join the discord channel https://discord.gg/dtYzk8sp where we can discuss which bug is most important

can you try 1.8.5


fix class name in toString method

Sorry about that typo

No problem. I missed that too. Merged. Thank you!


which version of pytorch is used?

Hello Hao,
Would you tell me what's the version of pytorch you are using please?
Will appreciate if u can add a requirements list of this project.
Cheers,
Chao

@xhuanlc pytorch0.4.0 and python2.7 work for me on ubuntu14.04

@xhuanlc I use pytorch0.4.0 and python2.7. Python3 is unavailable.

See README


Browser becomes unresponsive after downloading file

(I have never reported a bug so I am sorry if I do something wrong😞)
Describe the bug
Sometimes after I save a file, all buttons become unresponsive. I cannot switch tabs or close them and I am forced to close the entire window via the panel (I use XFCE).
To Reproduce
All you should have to do to encounter this is save a file.
Expected behavior
The file is saved and the download bar at the bottom of the screen pops up, showing the filename and the menu that when opened allows you to open the file location as well as other things. After that browsing can be continued as usual.
Environment
This has happened on both Debian stable and Manjaro. On Debian I was using the "latest" contributor binary and on Manjaro (which I am currently using) it is also a binary, version "85.0.4183.102 (Official Build) Arch Linux (64-bit)" (Revision: ffe848af6a5df4fa127e2929331116b7f9f1cb30-refs/branch-heads/4183@{#1770}).
Additional context
It only happens some of the time when I download something. Other times it works perfectly fine.
And in case it helps, in "About Version" the line "Command Line" is
--password-store=basic --flag-switches-begin --load-media-router-component-extension=0 --disable-features=GlobalMediaControlsForCast,GlobalMediaControlsPictureInPicture,GlobalMediaControlsSeamlessTransfer,TabHoverCardImages,TabHoverCards,UseMediaHistoryStore --flag-switches-end
I don't have this information about the Debian binary I was using because I switched to Manjaro thinking it was a problem with Debian.
Thank you🙂

I have Arch Linux and have downloaded a lot of files and haven't experienced this issue. Does it happen to any file? or a specific one? Also, latest version is 85.0.4183.121, you might want to update.

Does it happen to any file? or a specific one?

It happens with any file. (It happened again earlier, and interestingly, a video that was playing in another tab continued to play even though I could no longer click any buttons). I tried disabling the "use GTK+ theme" option and saved a few files to test. They worked fine, so it may actually just be a GTK issue, considering that the download bar looks different...? However, since it isn't something that happens every time I save something, I can't be 100% sure it's fixed yet.

Also, latest version is 85.0.4183.121, you might want to update.

Oh! Thank you for telling me 😮

I'm using XFCE and ungoogled-chromium in Gentoo, never had such issues with downloads.

It happened again 😥

I'm using XFCE and ungoogled-chromium in Gentoo, never had such issues with downloads.

If you run ungoogled-chromium via the terminal, do you get the error InitializeSandbox() called with multiple threads in process gpu-process.? Since I was getting that, I have taken the steps to fix it just now by using --disable-gpu and --disable-software-rasterizer, and I will see if that does anything.

InitializeSandbox() called with multiple threads in process gpu-process

That is a unrelated and harmless message, but if you ask, I do also have it in logs.

Alright. I did some more testing today and it always happens the first time I download something via ungoogled-chromium when the save destination is the flash drive I have plugged in. I was using a different flash drive when it was happening on Debian, so I don't think it's because of the flash drive itself. [ERROR:browser_main_loop.cc(269)] Gtk: gtk_widget_set_visible: assertion 'GTK_IS_WIDGET (widget)' failed appears in my terminal right when it happens. (The error [ERROR:ev_root_ca_metadata.cc(841)] Failed to register OID: 0 also appears right when I start ungoogled-chromium, but since that happens every time I do, I do not know if it is related or not.)
I would like to avoid installing actual vanilla Chromium on my computer, but if no one else can reproduce this bug, then I guess I will have to. 😞 Again, I am very sorry if I have made a mistake somewhere.

Those two other errors are common and unrelated to your issue.

when the save destination is the flash drive I have plugged in

This is likely the culprit.  I'll bet the reason it's unresponsive for the first download is because the drive hasn't been mounted yet!
See if setting your download directory to an internal drive solves the problem.  If you want to keep the flash drive as the download location you might want to see if there are any system settings to automount the flash drive before first use.


Website extremely slow on Firefox

Please give the docs site a try on Firefox. Click around a few times. It's really slow, if functional at all

Is it specific pages that just load longer? Or do all pages start to be slow after some while?

I'm closing this issue as we don't have sufficient info to reproduce it and have made several updates to the docs site since the creation of the issue, but please feel free to open other issues in the future if you notice anything that can be improved in the Prisma docs 🙏


pythonPath in settings.json is ignored

Environment data

VS Code version: 1.52.1
Extension version (available under the Extensions sidebar): 2021.1.502429796
OS and version: XXX
Python version (& distribution if applicable, e.g. Anaconda): v3.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: .
Relevant/affected Python-related VS Code extensions and their versions: .
Value of the python.languageServer setting: Pylance

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
When there's a pythonPath settings the extension should use it
Actual behaviour
is uses a previous settings and don't read anymore the settings file
Steps to reproduce:
Unfortunately I don't know
Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

Error: spawn  ENOENT

Since it was super urgent to be able to work I did

quit vscode
remove Library/Application Support/Code
restar vscode

In the python logs see if you are in the pythonPath deprecated experiment.

Closing this since there has been no response in a long time. Please create a new bug with requested details.


How to set overlapoffset for events on same time period

HI,
I want to Set overlapOffset dynamically if events are in the same time period. I attached a screenshot for you to look over.

Currently the prop is a fixed value and cannot be set dynamically. If you create a pull request for this, I'm happy to merge it! Thanks


Consider using isolation_level="IMMEDIATE" for write connections

We currently use the default isolation level for both read and write connections.

Here's the change:
diff --git a/datasette/database.py b/datasette/database.py
index ffe94ea7..e56dcf03 100644
--- a/datasette/database.py
+++ b/datasette/database.py
@@ -85,12 +85,17 @@ class Database:
             return "db"

     def connect(self, write=False):
+        kwargs = {
+            "uri": True,
+            "check_same_thread": False,
+        }
+        if write:
+            kwargs["isolation_level"] = "IMMEDIATE"
         if self.memory_name:
             uri = "file:{}?mode=memory&cache=shared".format(self.memory_name)
             conn = sqlite3.connect(
                 uri,
-                uri=True,
-                check_same_thread=False,
+                **kwargs,
             )
             if not write:
                 conn.execute("PRAGMA query_only=1")
@@ -110,9 +115,7 @@ class Database:
             qs = ""
         if self.mode is not None:
             qs = f"?mode={self.mode}"
-        conn = sqlite3.connect(
-            f"file:{self.path}{qs}", uri=True, check_same_thread=False
-        )
+        conn = sqlite3.connect(f"file:{self.path}{qs}", **kwargs)
         self._all_file_connections.append(conn)
         return conn

I don't want to commit this until I can run some kind of load test that demonstrates a problem with the default connection mode that is solved by switching to IMMEDIATE.

I tried running this locustfile.py using python -m locust -f locustfile.py and then simulating 100 users:
from locust import HttpUser, task, between
import random
import json
import os

class APIUser(HttpUser):
    host = "http://localhost:8001"
    # wait_time = between(0.05, 0.1) # in seconds

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self.id_counter = 0
        self.api_key = os.environ.get('DS_API_KEY')
        if not self.api_key:
            raise ValueError("DS_API_KEY environment variable is not set")
        self.headers = {
            'Content-Type': 'application/json',
            'Authorization': f'Bearer {self.api_key}'
        }

    @task(3)
    def get_data(self):
        self.client.get("/data", headers=self.headers)

    @task(3)
    def get_mytable(self):
        self.client.get("/data/mytable?_sort_desc=id", headers=self.headers)

    @task(2)
    def get_mytable_with_id(self):
        self.client.get("/data/mytable?id=5", headers=self.headers)

    @task(4)
    def post_create_data(self):
        self.id_counter += 1
        payload = {
            "table": "mytable",
            "rows": [
                {
                    "id": self.id_counter,
                    "name": random.choice(["Tarantula", "Black Widow", "Huntsman", "Wolf Spider", "Jumping Spider"])
                }
            ],
            "pk": "id",
            "replace": True
        }
        self.client.post("/data/-/create", data=json.dumps(payload), headers=self.headers)

On both my Mac and GitHub Codespaces I was unable to get it to trigger any database lock errors. Mac results:

I tried on Codespaces as well because I thought maybe my M2 performed too well and didn't trigger errors.

The currently open challenge here is this: create a locustfile.py which reliably triggers database locked errors when exercising a Datasette instance calling the JSON write API (and reading other endpoints at the same time).

Here's Codespaces Lotust run with WAL enabled (the macOS one above didn't have WAL enabled):

I used https://github.com/datasette/studio to run this

I'm going to push this code anyway, and exercise it in an alpha for a bit.

Rails 8 made a similar change:

https://github.com/rails/rails/pull/50371


cmd/cli(ticdc): Add more information about ignoring tables without valid indexes

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/8837
What is changed and how it works?
Add more information about ignoring tables without valid indexes.
Check List
Tests

Manual test

Questions
Will it cause performance regression or break compatibility?
No
Do you need to update user documentation, design documentation or monitoring documentation?
No
Release note
Add more information about ignoring tables without valid indexes

Duplicate of https://github.com/pingcap/tiflow/pull/8840


Array with hole support

Array with last element as a hole will generate wrong output.
eval(tosource(Array(3))).length // should be 3, but get 2

How would you expect this to serialize?

Array(3) should be serialized to [, , ,].

Specification: http://www.ecma-international.org/ecma-262/7.0/index.html#sec-array-initializer

ES2016 is a bit newer than I'm supporting… what's the browser support like?

Actually, this behavior had been defined in ES5 (I'm not sure what ES3 does say). IE 10+ and all other browsers support it. (IE <=9 is buggy)


FAQ: English page links to German terms of use

Where to find the issue
https://www.coronawarn.app/en/ Section Transparency

Describe the issue
https://www.coronawarn.app/en/ (English) links to German terms of use https://www.coronawarn.app/assets/documents/cwa-eula-de.pdf
Suggested change
Change the link on https://www.coronawarn.app/en/ (English) to the English terms of use https://www.coronawarn.app/assets/documents/cwa-eula-en.pdf

The link seems to have been this way for months. I just wanted to get feedback from the Open Source Team before submitting a PR, since this is a legal document. I can't see any reason though not to link to the English Terms of Use on the English page.
The app itself is displaying English Terms of Use when the UI is set to English.

@MikeMcC399
Sorry, I definitely did not want to steal you a PR (if you like to I can close mine and you can open a new one) 🙂
It was a quick fix so I just thought let's do it. If they, for what reason so ever, don't want to change the link, that's not a big problem, was a quick one 😄

@Ein-Tim
No, don't close your PR. It's fine!
Just in future check with me before opening a PR on one of my issues please, as I will sometimes open an issue first if I think it needs a discussion before submitting a PR. Before I set up a PR I like to have the feeling that the Open Source Team is likely to approve it! When I started here (now months ago!) I did not have the experience with GitHub to know how to do things right. In the meantime I've learned a bit, so I can now do most things myself. 😀

@MikeMcC399

Just in future check with me before opening a PR on one of my issues please

For sure, will do so! 👍

EXPOSUREAPP-4985

@Ein-Tim  @MikeMcC399 Thanks !


Print base64 files received by user

Good afternoon, I'm using your print-to-pdf library for Windows, and I want to know if there is any way to send a base64 instead of a path, the following example doesn't work, I can only print if I use the relative path from my computer (C://...)
Can you help me with this or can you point me to existing solutions, please?
Don't works:

Works:

hi @davimateus1, pdf-to-printer is using SumatraPDF to print files. So, check if SumatraPDF allows this. If yes, then we could think of a solution.

closing the issue for now. feel free to re-open it if you have more details or questions


[GLUTEN-4989][CH] Support function timestamp_add

What changes were proposed in this pull request?
(Fixes: #4989)
How was this patch tested?
Test by ut

LGTM

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_master_03_20_2024_time.csv
log/native_master_03_19_2024_e977d79e3_time.csv
difference
percentage

q1
38.53
36.21
-2.311
94.00%

q2
24.00
24.11
0.109
100.46%

q3
37.50
36.64
-0.854
97.72%

q4
38.61
38.02
-0.590
98.47%

q5
71.63
69.07
-2.563
96.42%

q6
7.37
5.91
-1.459
80.19%

q7
83.96
82.31
-1.651
98.03%

q8
84.46
85.26
0.803
100.95%

q9
125.15
121.93
-3.219
97.43%

q10
45.58
45.61
0.027
100.06%

q11
20.52
20.57
0.048
100.23%

q12
28.80
28.32
-0.482
98.33%

q13
48.70
47.17
-1.533
96.85%

q14
22.25
16.86
-5.393
75.76%

q15
31.14
31.19
0.052
100.17%

q16
15.42
16.26
0.846
105.48%

q17
102.47
99.14
-3.327
96.75%

q18
143.60
142.77
-0.829
99.42%

q19
13.64
14.68
1.044
107.65%

q20
27.02
28.29
1.265
104.68%

q21
225.52
227.96
2.443
101.08%

q22
13.96
14.00
0.041
100.29%

total
1249.81
1232.28
-17.534
98.60%


blog/mindshare - pagination

Can we make pagination sticky to the top of the "latest articles" section so that when you click from page 1 to 2, etc., you end up at the top of that section instead of the bottom (having to scroll back up is annoying).

@randy I made a fix for this but it looks like you may have as well let me know ?


OSGi bundle

Thank you for your work.
Make the library even better.
Discussing version 3.10.1.
JAR not ready for the OSGi environment.

For OSGi 6.0 (on the example Karaf 4.2.8)
When deploying JAR as a Bundle we get an error:

org.apache.karaf.features.internal.util.MultiException: Error:
Unable to create resource for bundle mvn:com.auth0/java-jwt/3.10.1
at org.apache.karaf.features.internal.download.impl.MavenDownloadManager$MavenDownloader.(MavenDownloadManager.java:91)
at org.apache.karaf.features.internal.download.impl.MavenDownloadManager.createDownloader(MavenDownloadManager.java:72)
at org.apache.karaf.features.internal.region.Subsystem.downloadBundles(Subsystem.java:457)
at org.apache.karaf.features.internal.region.Subsystem.downloadBundles(Subsystem.java:452)
at org.apache.karaf.features.internal.region.SubsystemResolver.resolve(SubsystemResolver.java:224)
at org.apache.karaf.features.internal.service.Deployer.deploy(Deployer.java:393)
at org.apache.karaf.features.internal.service.FeaturesServiceImpl.doProvision(FeaturesServiceImpl.java:1062)
at org.apache.karaf.features.internal.service.FeaturesServiceImpl.lambda$doProvisionInThread$13(FeaturesServiceImpl.java:998)
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
at java.base/java.lang.Thread.run(Thread.java:834)
Suppressed: java.lang.Exception: Unable to create resource for bundle mvn:com.auth0/java-jwt/3.10.1
...
Caused by: org.osgi.framework.BundleException: Unable to build resource for mvn:com.auth0/java-jwt/3.10.1: Unsupported 'Bundle-ManifestVersion' value: 1
...
Caused by: org.osgi.framework.BundleException: Unsupported 'Bundle-ManifestVersion' value: 1
...
Error executing command: Error:
Unable to create resource for bundle mvn:com.auth0/java-jwt/3.10.1

Need to update META-INF/MANIFEST.MF

When building a JAR, consider the OSGi requirements.

Temporary solution.

When building your project as a bundle, the maven-bundle-plugin is used
(by org.apache.felix).
Instructions:
<Embed-Dependency>java-jwt;inline=true</Embed-Dependency>
Allows you to include the library inside the bundle and work with it.

:wave: Thanks for the report! Currently the workaround you supplied of using the org.apache.felix:maven-bundle-plugin and Embed-Dependency seems to be the best path forward when deploying to an OSGI container.

I'm curious why a workaround is the best path forward? Are there technical limitations to OSGi support? (it can be quite trivial to include the bndtool to generate appropriate manifest headers)

OSGi manifest headers don't do any harm for anyone. So just providing those with the library in the first place is a big advantage.

Team, is there any other workaround to make it work? I'm trying to  in my maven for an Adobe AEM project and the dependencies are not resolved

Hey @jimmyjames. Will you accept a PR for this?


OSError: libopenblas.so.0: cannot open shared object file: No such file or directory in mxnet 1.8.0

Description
mxnet 1.8.0 emits the following error when running import mxnet:
OSError: libopenblas.so.0: cannot open shared object file: No such file or directory

Error Message
(generated from the dockerfile attached in the to-reproduce section)
+ Step 1/3 : FROM python:3.7
 ---> 7fefbebd95b5
+ Step 2/3 : RUN pip install mxnet
 ---> Running in fc634966f9aa
Collecting mxnet
  Downloading mxnet-1.8.0-py2.py3-none-manylinux2014_x86_64.whl (38.7 MB)
Collecting graphviz<0.9.0,>=0.8.1
  Downloading graphviz-0.8.4-py2.py3-none-any.whl (16 kB)
Collecting numpy<2.0.0,>1.16.0
  Downloading numpy-1.20.1-cp37-cp37m-manylinux2010_x86_64.whl (15.3 MB)
Collecting requests<3,>=2.20.0
  Downloading requests-2.25.1-py2.py3-none-any.whl (61 kB)
Collecting certifi>=2017.4.17
  Downloading certifi-2020.12.5-py2.py3-none-any.whl (147 kB)
Collecting chardet<5,>=3.0.2
  Downloading chardet-4.0.0-py2.py3-none-any.whl (178 kB)
Collecting idna<3,>=2.5
  Downloading idna-2.10-py2.py3-none-any.whl (58 kB)
Collecting urllib3<1.27,>=1.21.1
  Downloading urllib3-1.26.4-py2.py3-none-any.whl (153 kB)
Installing collected packages: urllib3, idna, chardet, certifi, requests, numpy, graphviz, mxnet
+ Successfully installed certifi-2020.12.5 chardet-4.0.0 graphviz-0.8.4 idna-2.10 mxnet-1.8.0 numpy-1.20.1 requests-2.25.1 urllib3-1.26.4
WARNING: You are using pip version 20.3.1; however, version 21.0.1 is available.
You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.
Removing intermediate container fc634966f9aa
 ---> b1c12d9f4376
+ Step 3/3 : RUN python -c "import mxnet"
 ---> Running in 8362fc58b280
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.7/site-packages/mxnet/__init__.py", line 23, in <module>
    from .context import Context, current_context, cpu, gpu, cpu_pinned
  File "/usr/local/lib/python3.7/site-packages/mxnet/context.py", line 23, in <module>
    from .base import classproperty, with_metaclass, _MXClassPropertyMetaClass
  File "/usr/local/lib/python3.7/site-packages/mxnet/base.py", line 351, in <module>
    _LIB = _load_lib()
  File "/usr/local/lib/python3.7/site-packages/mxnet/base.py", line 342, in _load_lib
    lib = ctypes.CDLL(lib_path[0], ctypes.RTLD_LOCAL)
  File "/usr/local/lib/python3.7/ctypes/__init__.py", line 364, in __init__
    self._handle = _dlopen(self._name, mode)
+ OSError: libopenblas.so.0: cannot open shared object file: No such file or directory
The command '/bin/sh -c python -c "import mxnet"' returned a non-zero code: 1

(key lines are colored green)
To Reproduce
Steps to reproduce

Prepare the following dockerfile:

FROM python:3.7

RUN pip install mxnet
RUN python -c "import mxnet"

Then, run docker build .

What have you tried to solve it?

Environment
We recommend using our script for collecting the diagnostic information with the following command
curl --retry 10 -s https://raw.githubusercontent.com/apache/incubator-mxnet/master/tools/diagnose.py | python3

Environment Information
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
Address sizes:       39 bits physical, 48 bits virtual
CPU(s):              8
On-line CPU(s) list: 0-7
Thread(s) per core:  1
Core(s) per socket:  1
Socket(s):           8
Vendor ID:           GenuineIntel
CPU family:          6
Model:               158
Model name:          Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
Stepping:            13
CPU MHz:             2400.000
BogoMIPS:            4800.00
L1d cache:           32K
L1i cache:           32K
L2 cache:            256K
L3 cache:            16384K
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht pbe syscall nx pdpe1gb lm constant_tsc rep_good nopl xtopology nonstop_tsc cpuid tsc_known_freq pni pclmulqdq dtes64 ds_cpl ssse3 sdbg fma cx16 xtpr pcid sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch pti fsgsbase bmi1 avx2 bmi2 erms xsaveopt arat
----------Python Info----------
Version      : 3.7.9
Compiler     : GCC 8.3.0
Build        : ('default', 'Nov 18 2020 14:10:47')
Arch         : ('64bit', 'ELF')
------------Pip Info-----------
Version      : 20.3.1
Directory    : /usr/local/lib/python3.7/site-packages/pip
----------MXNet Info-----------
No MXNet installed.
----------System Info----------
Platform     : Linux-4.19.76-linuxkit-x86_64-with-debian-10.6
system       : Linux
node         : 9bcb86c4d6cf
release      : 4.19.76-linuxkit
version      : #1 SMP Tue May 26 11:42:35 UTC 2020
----------Hardware Info----------
machine      : x86_64
processor    :
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0155 sec, LOAD: 0.9034 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.1894 sec, LOAD: 0.2668 sec.
Error open Gluon Tutorial(cn): https://zh.gluon.ai, <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate has expired (_ssl.c:1091)>, DNS finished in 0.3720698356628418 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0193 sec, LOAD: 0.5068 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0474 sec, LOAD: 0.5751 sec.
Error open Conda: https://repo.continuum.io/pkgs/free/, HTTP Error 403: Forbidden, DNS finished in 0.01907634735107422 sec.
----------Environment----------
Removing intermediate container 9bcb86c4d6cf

I think this is due to a change in CD that openblas is no longer statically linked in libmxnet. For now you can install openblas separately.
cc @mseth10 @leezu

@harupy it's unclear what you are doing. You need to provide more details.

We observe the same issue when trying to use pip-installed MXNet 1.8 (ubuntu, CPU): https://github.com/awslabs/sockeye/runs/2161521442?check_suite_focus=true

@mseth10 @access2rohit please take a look why the CD didn't package the libopenblas.so
tools/pip/setup.py includes instructions for copying libopenblas.so in v1.8.x, v1.x, and master. But apparently that didn't work in v1.8.x, potentially due to some missing library file names in the jenkins files?
v1.8.x https://github.com/apache/incubator-mxnet/blob/a0535ddfb0246f53f7b851baf861fc06d3ff48c3/tools/pip/setup.py#L170-L172
v1.x https://github.com/apache/incubator-mxnet/blob/cfa1c890a7ecb8b5e29ff4e90d6784141f09c4cd/tools/pip/setup.py#L164-L166
master https://github.com/apache/incubator-mxnet/blob/4d706e8c19b3354878eda9467b149c0ce1fd6d47/tools/pip/setup.py#L165-L167
However, I noted that v1.8.x also attempts to copy libquadmath, which MUST NOT happen due to license of libquadmath. That should be fixed. Fortunately that code didn't run due to the current bug.

The problem is that https://github.com/apache/incubator-mxnet/pull/19514 is missing on v1.8.x

My container suddenly started failing to build and upon looking, this was the error. I started using previous version which is 1.7.0.post2 and works perfectly

@praneethkv we are working on patching the wheels to fix the problem


Add BipedRobot in ci

@blegat I don't really understand the error in the test as this should be solved by https://github.com/JuliaRobotics/RigidBodyDynamics.jl/commit/0c096fa3cf9e87aee79e88c81fe18b76cebf61d5

We should probably do all the type piracy needed in the BipedRobot instead of in each test/example.

We should also open issues in RigidBodyDynamics and Quaternions for ambiguity

Yes, I think these could be fixed upstream with package extensions


Remove redundant is:private terms

See commit messages.

I think this isn't quite right in the case of is:private<EMAIL_ADDRESS>-- we'll incorrectly drop the is:private there.

For the negated case Filter.term returns not-pm-with, so we should be fine here.

I added a test to verify the negated case.

Great, sorry I'd missed that detail of how filter.term works.  Merged!


Fixed deduplication key calculation for multi-topic rules

cc @wikimedia/services

Coverage increased (+0.05%) to 88.356% when pulling dee19f2d80b2d83a1e4731d3cb1e525f2dc3a4b0 on Pchelolo:dedupe_key into b05fb596449661df76db35afb4004e02cbdcc8f0 on wikimedia:master.


update CommandBox.java

Coverage increased (+0.3%) to 85.51% when pulling 8c4bc0126a880c3c67967985ba8d405448c6dc7e on zhangshuoyang:zhangshuoyang into f53a6e13ed5c3fabd78650c9d5628ebdca0262c8 on CS2103AUG2017-W11-B1:master.


[Event Change Request] codeunit 408 DimensionManagement - OnBeforeCheckDimComb

Hi,
could you please add the parameter TempDimBuf from procedure CheckDimComb as a parameter to the event OnBeforeCheckDimComb?
The resulting signature would be:
local procedure OnBeforeCheckDimComb(
    var DimensionCombination: Record "Dimension Combination";
    var TempDimBuf: Record "Dimension Buffer" temporary)

Thanks.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 48695.


Specify the name of a single changed signal in EventedModel

My problem, I am using attrs to make a dataclass with a name attribute.
I want to modify the name before setting it, but I cannot use attrs.converters because they are
not bound to the instance. So I create a private _name field and use a name property to validate the name
before setting it.
Now I would like to emit a signal when name is changed. This is my working example using PR #260.
Now the signal is emitted at self.events._name_changed but I would like it to be self.events.name_changed or self.events.name, i.e. without the "_" prefix because then I will add other signals like self.events.color and I want it to be consistent, so all without the "_" prefix.
Any idea how I can do that without another PR :) ?
from typing import ClassVar
from attrs import define, field
from psygnal import SignalGroupDescriptor, EmissionInfo

@define
class Model:
    events: ClassVar[SignalGroupDescriptor] = SignalGroupDescriptor(signal_suffix="_changed")

    _controller: str = field()
    _name: str = field(kw_only=True)

    @property
    def name(self) -> str:
        return self._name

    @name.setter
    def name(self, value: str) -> None:
        # Generate a unique name among siblings
        siblings = [self.controller]
        value = f"{value}_1" if value in siblings else value
        self._name = value

    @property
    def controller(self) -> str:
        return self._controller

m = Model("ctt", name="c")
@m.events.connect
def on_any_change(info: EmissionInfo):
    print(f"field {info.signal.name!r} changed to {info.args}")

m.name = "ctt"; m.name
# >> field '_name_changed' changed to ('ctt_1', 'c')
# I would like it to be 'name_changed'

I managed to make it work with a custom signal_group_class argument. I have to see how it behaves with sub-classing though.
from attrs import define, field
from psygnal import SignalGroupDescriptor, Signal, SignalGroup, EmissionInfo

ModelSignalGroup = type("ModelSignalGroup", (SignalGroup,), {"name": Signal(str, str)})

@define
class Model:
    events: ModelSignalGroup = SignalGroupDescriptor(
        signal_group_class= ModelSignalGroup,
    )

    _controller: str = field()
    _name: str = field(kw_only=True)

    @property
    def name(self) -> str:
        return self._name

    @name.setter
    def name(self, value: str) -> None:
        # Generate a unique name among siblings
        siblings = [self.controller]
        value = f"{value}_1" if value in siblings else value
        self._name = value

    @property
    def controller(self) -> str:
        return self._controller

m = Model("ctt", name="c")
@m.events.connect
def on_any_change(info: EmissionInfo):
    print(f"field {info.signal.name!r} changed to {info.args}")

m.name = "ctt"; m.name
# >>> field 'name' changed to ('ctt_1', 'c')

It works with sub-classing, although I have to redefine the events class attribute for the subclasses.

see #262  and https://github.com/pyapp-kit/psygnal/pull/260#issuecomment-1924217669 for another option.

This can be closed, with #299 merged, it can be done in two ways (that are not exactly equivalent).
With aliases only:
from typing import ClassVar
from attrs import define, field
from psygnal import SignalGroupDescriptor, EmissionInfo

aliases = {"_name": "name", "name": None, "_controller": None}

@define
class Model:
    events: ClassVar[SignalGroupDescriptor] = SignalGroupDescriptor(signal_aliases=aliases)

    _controller: str = field()
    _name: str = field(kw_only=True)

    @property
    def name(self) -> str:
        return self._name

    @name.setter
    def name(self, value: str) -> None:
        # Generate a unique name among siblings
        siblings = [self.controller]
        value = f"{value}_1" if value in siblings else value
        self._name = value

    @property
    def controller(self) -> str:
        return self._controller

m = Model("ctt", name="c")
@m.events.connect
def on_any_change(info: EmissionInfo):
    print(f"field {info.signal.name!r} changed to {info.args}")

m.name = "ctt"; m.name
# >> field 'name' changed to ('ctt_1', 'c')

With aliases and a SignalGroup subclass (thanks to #291, it will work on new fields if subclassing Model without the need to redefine events in the subclass):
from typing import ClassVar
from attrs import define, field
from psygnal import SignalGroupDescriptor, Signal, SignalGroup, EmissionInfo

ModelSignalGroup = type("ModelSignalGroup", (SignalGroup,), {"name": Signal(str, str)})

@define
class Model:
    events: ClassVar[ModelSignalGroup] = SignalGroupDescriptor(
        signal_group_class= ModelSignalGroup,
        signal_aliases={"_name": None, "_controller": None}
    )

    _controller: str = field()
    _name: str = field(kw_only=True)

    @property
    def name(self) -> str:
        return self._name

    @name.setter
    def name(self, value: str) -> None:
        # Generate a unique name among siblings
        siblings = [self.controller]
        value = f"{value}_1" if value in siblings else value
        self._name = value

    @property
    def controller(self) -> str:
        return self._controller

m = Model("ctt", name="c")
@m.events.connect
def on_any_change(info: EmissionInfo):
    print(f"field {info.signal.name!r} changed to {info.args}")

m.name = "ctt"; m.name
# >>> field 'name' changed to ('ctt_1', 'c')


Crashing all over the place

Running this on windows 11.

Tried to use the portable button.

Could not create a group either.

I'm using it in portable mode. But you may have an overriding issue. First, see if there is a Taskbar Groups background app running in the system tray and then kill it. Next, see if there is something similar in task manager and also kill it.
Finally, log out and back in and see if it will start up without crashing. If it crashes again, see if your .NET runtimes need updating.

Ok so I redownloaded and then checked the portable mode. That worked this time. Now when I create shortcuts and save it crashes. It is the same message as before.

Yeah, I think I had a similar problem once - I think either a reboot solved it, or something else. Let me look for the thread I reported it in.

https://github.com/PikeNote/taskbar-groups-pike-beta/issues/55
You have it installed versus using the .ZIP extracted (portable) package?
Try this:

Close all instances of TBG (use Task Manager).

In Windows Explorer, or using the Run dialog, open %localappdata% and see if there is a folder named Jack Schierbeck

If so, rename it

Try opening TBG again.

This may solve it - but may not. The link I posted was actually to delete all the other files inside of the portable (extracted) folder, because the TBG executable tries to create other files based upon the current version, but if it finds the files already there, it may get confused and not create the right versions, and thus the DLLs get wonky. I think, anyway. But you said installed, so that should have fixed it, especially if you uninstalled it and then downloaded it again and installed it. That's why I think renaming the Jack folder might help.
If it does work, delete the old folder that you renamed.

Nope. Every time I open TaskbarGroup.exe it recreates the folder. Every time I try to save a change it crashes.

Let me go back and look, I definitely had that issue. ISTR that Pike put out a private build that fixed it?
BRB

OK, I know what the issue might be. I suspect you have version 2.2.0. from the releases, but that has been superseded a couple of times with ad-hoc patched versions. I'm actually using that last ad-hoc build, <IP_ADDRESS>:

I used to have the exact post bookmarked, but I must have deleted it. Let me find it.

Here it is:
https://github.com/PikeNote/taskbar-groups-pike-beta/issues/31#issuecomment-1429146371
I just re-downloaded it, extracted it, and compared the hash to the one I have. It is a match.
I realize now that there is no real installation - my recommendation is to close all instances, deleted all existing files wherever you have them, and delete that Jack folder in %localappdata%. Then create a new location to store this new executable and run it form there, after all those program closings and file deletions.
This version will not play nicely with any pre-existing files like:

Taskbar Groups Backgroundf.exe
Taskbar Groups.exe (though, technically, if you were to extract the zip to the existing folder, it should overwrite it).

And, if you delete the old folder and start with a brand new folder, there should be nothing in that folder when you start, and then copying this ad-hoc build should let you start fresh.
Sorry it took me this long to remember that I'm not running the last listed release. It's been a while since I went through all this myself.

Having the exact same crashing issues on a fresh install using the <IP_ADDRESS> ad-hoc build.
I cannot toggle portable mode, nor can I save a group without having errors.


Pending Circuit Support

This is a tracking issue for pending circuit support issues. Our current workflow is to first add non-circuit support for new features, then update the circuit accordingly (in a separate PR). Whenever such new functionality is added, we should create a corresponding circuit issue and add a link to the list below. When the issue is closed, the box in the checkbox-list below should be checked.
Let's keep this issue open as a master list to be done and historically completed.

[x] #161
[x] #163
[x] #167
[x] #178
[x] #184
[x] #186
[x] #204
[x] #219
[x] #421

These boxes should now automatically get checked off when an issue closes (epics github workflow).

There's nothing left!


Adding RunRepository unit tests

Resolves
N/A
Description
I went ahead and added a unit tests to ensure that the RegisteredDate is actually properly set.
This is a fun PR as it showcases the use of:

InMemoryDataBase for EF Core
Named tuples in C# 8

@Brett-Parker  This is ready for  review. I just went ahead and added the unit test we spoke about during the stream.


feature request: extend agent to write/read to/from filesystem instead of in-memory with retention

Background of the problem
Hi.
We are working on managing edge devices from a central management application. Management application executes in k8s.
At the moment, we collect/scrape metrics ourselves in the device. Metrics are saved in TSDB format.
We need to send these metrics to the k8s cluster.
In the cluster, metrics should be available for querying. Either by user or Grafana.
We also need to be able to forward the metrics from the cluster to a multi-cluster management application like Stolostron.
We were made aware of Prometheus Agent Mode and thought of using that as the collector and forwarder in the devices (replace our implementation). It will write to a Prometheus/Thanos server in the cluster.
Request/Proposal
The agent is missing a few features:

write from TSDB files and not from memory
disable in-memory metrics (default 2 hours) for lowering the memory usage
delete/forget whatever was sent from TSDB files
apply retention rules. delete metrics older than the configured threshold.

Existing features that might need verification or enhancements:

Agent should retry endlessly to send the data. Our use case is 10000s of devices and not all can write at once.

Links
feature request: Add agent support for beyond WAL persistent storage and metric export

@bwplotka here's a new issue following our discussion. Thanks for all your help. appreciate it.

Prometheus agent is read/writing metrics from filesystem already, I am not sure what you mean with this issue. The agent is using a Write-Ahead-Log on disk.

cc @rfratto

It sounds like the proposal is for Prometheus Agent to write scraped metrics directly to the WAL and hold absolutely nothing in memory long-term (including from remote_write which also holds series in memory). Am I following correctly?

@rfratto @roidelapluie
Perhaps I am not aware of how agent works. I thought that it saves up to 2 hours of metrics. All of it as part of WAL and may be persisted to filesystem.
What I need is to save more than 2 hours. Days. Write the metrics to remote server. Delete sent metrics.

Hm let's clarify this issue @ydayagi

write from TSDB files and not from memory

Do you mean remote write from TSBD blocks? I believe this ask is covered by https://github.com/prometheus/prometheus/issues/9607

disable in-memory metrics (default 2 hours) for lowering the memory usage

I guess this is the core of this issue and it means what @rfratto tried to rephrase?

It sounds like the proposal is for Prometheus Agent to write scraped metrics directly to the WAL and hold absolutely nothing in memory long-term (including from remote_write which also holds series in memory). Am I following correctly?

Is this what we are discussing here?

Perhaps I am not aware of how agent works. I thought that it saves up to 2 hours of metrics. All of it as part of WAL and may be persisted to filesystem.
What I need is to save more than 2 hours. Days.

I don't think that is physically possible because some code has to operate that WAL and that needs memory. Plus, I don't think we can have days of data in WAL format as it much less consistent

How heavy is the Prometheus server? If configuration includes only a few targets and no alerts are required , can it be considered a small server?

Yes, it can be considered "small". But there are things we can optimize further for your case:

Removal of any memory structures required for efficient querying capabilities, since you probably don't want that
Removal of data right after a remote write was successful for such batch of data.

Yet, there is nothing wrong in using it first and trying how it goes. Still, I don't think it matches your other point: the ability to remote write older data like "days", since we can only remote write data from first 2h. And even if we could remote write older data in the event of this binary being out of network for days, most of the backends/remote write receivers does not accept writes older than 2h.
We choose to instead of trimming Prometheus from those features, we will implement an agent mode that starts from the above points. Then we can slowly add things like persistent storage with blocks and ability to forward that data (and remove after forward). This is what https://github.com/prometheus/prometheus/issues/9607 asks for.

delete/forget whatever was sent from TSDB files

I think this is also covered by https://github.com/prometheus/prometheus/issues/9607, please join discussion there.

Closing as duplicate of #9607


Broken image in concat

Documentation Related To Component:

concat
Please check those that apply

[ ] typo
[ ] documentation doesn't exist
[x] documentation needs clarification
[x] error(s) in example
[ ] needs example

Description Of The Issue

There's a broken image in the “Description” section.

Thanks for pointing. Unfortunately it's not just related to concat. For each operator the image is missing. We are working on it. I'll add new informations to this issue

Please close this issue, as it is a duplicate of #4366.

Yeah, good idea to get rid of duplicate issues and have just one place to track things.


Fix empty array check

empty array [] in ngIf is truthy
Loading...  line will never shows, check length instead

We took another fix for the PR


[Development] Add eslint-plugin-import

eslint-plugin-import is a great eslint plugin that speeds up development by detecting any unresolvable imports. We should add it to the project to help catch these issues as early as possible.

Ran into an issue with this PR.
Tracking the fix in eslint-plugin-import https://github.com/benmosher/eslint-plugin-import/issues/416

Can you PR this?

I will do it sometime this week.


Size from network weights

This PR add the --size_from_weights flag to override --dim and blockdims and directly loads the previously trained LoRA no matter the shape.
This allows finetuning a LoRA that has been resized with resize_lora.py (see https://github.com/kohya-ss/sd-scripts/pull/243) or with an unknown blockdims shape.
My Current Workflow:

Train LoRA with dim=128
Use sd-webui-lora-block-weight and sd-webui-supermerger to tweak block weights
Use resize_lora.py with --dynamic_method="sv_ratio"  --dynamic_param=(between 10 and 20) to greatly reduce size and untrained noise.
Use this PR with lr cut in half to retrain the smaller LoRA, this seems to do a great job in cleaning up small details like eyes.

There may be a better way to handle the interpretation of block lrs. since there isn't a separate function in lora.py to call.

Agreed on the parsing function.
Cheers!

You are always welcome to make any and all changes to my PRs. I find things that are useful to me and share them, but I am in no means a great programmer.

Hi, can you give some examples or detailed description on how to use sd-webui-lora-block-weight and sd-webui-supermerger to tweak block weights? Thanks very much!


Extension release 2018.9.5

Reviewer: @jdorweiler

Description:
New extension version 2018.9.5
Automated tests:

[ ] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

🌵


Broken localization

Forgot add a folder with the localizations in the release?
Nothing in /usr/local/lib/python3.5/dist-packages/Electron_Cash-3.1.6-py3.5.egg/electroncash
folder from the Electrum only partially.

Sorry, it`s for electron cash

This issue has gone undetected from the linux build for something like 5 months (f8b3c6e804183f6bb8f19adbb603dedd1de884c7), but has been fixed in 2cc6026bdecc7a76c8b64be760be1cd84d06d409, which will be included in the next release.


Add Button to Main File List

In Outlook, is there a way to add a button to the main email view page, rather than only on the opening on a message? I'm not seeing a way to do that clearly here.
Looking for a way to add a button where a user can report suspected Phishing by clicking on a button and having it be sent as an attachment to an email address. I don't want the user to have to open the message in order to do this.
I know I can do this as a COM add-in, but I'd prefer the new plugin format, if possible.

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

ID: 06e092f1-aa6a-45b5-50ce-a1afca772776
Version Independent ID: fa9c31b7-b63a-9867-f298-4d579b899c30
Content: ExtensionPoint element in the manifest file - Office Add-ins
Content Source: docs/reference/manifest/extensionpoint.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@exextoc: Can you help with this? thanks.

Currently the feature you requested, is not a part of the product. We track  feature requests on our user-voice page. Please add your request there. Feature requests on user-voice are considered when we go through our planning process.
https://officespdev.uservoice.com/forums/224641-general/category/131778-outlook-add-ins

@exextoc: Thanks for your response.
@securityvoid: Please add your feature request to UserVoice as exextoc recommends. I'll go ahead and close this issue.
Thanks.


Add missing cpu platform field to GCE driver

Add missing cpu platform field to GCE driver
Description
Added to 'extra' dict missing fields - cpuPlatform and minCpuPlatform.
Those fields already coming from the "list_nodes()" methods and was just ignored.
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[ ] Documentation
[x] Tests
[ ] ICLA (required for bigger changes)

Codecov Report

Merging #1343 into trunk will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            trunk    #1343      +/-   ##
==========================================
- Coverage   86.34%   86.34%   -0.01%
==========================================
  Files         372      372
  Lines       75825    75827       +2
  Branches     6928     6928
==========================================
  Hits        65473    65473
- Misses       7569     7570       +1
- Partials     2783     2784       +1

Impacted Files
Coverage Δ

libcloud/compute/drivers/gce.py
76.84% <100%> (+0.01%)
:arrow_up:

libcloud/test/compute/test_upcloud.py
90.06% <0%> (-1.33%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks for the contribution.
I added a test case for it (39f2841a33d34a4460b269d02fd4dbff0c5ca5a7) and merged it into trunk.

@Kami Great! Happy to help a littie bit to this great project!
Is there a date to the next release?

It's been a while since the last release so v2.6.0 will likely be out some time in the next couple of weeks.

@Kami Great to hear, looking forward to it!
Thanks for the help.


extract origin dual fisheye and imu measurements

Thanks so much for your job! i'm a newbie about gopro or ffmeg. I wonder  if could i get the origin dual fisheye images and imu measurements from gopro max .360 file as i want to use it to do vio job. Thanks for your help!

Thanks @fushi219 .
Check out this post, it might help: https://www.trekview.org/blog/2022/gopro-telemetry-exporter-getting-started/
If not, please raise any other general questions like this one on our Discord: https://discord.gg/ZVk7h9hCfw


Before trying to load SO results (activation)

check that there are valid results before trying to visualize them
Fixes issue #1061 partly
Brief summary of changes
Before loading activations, make sure the storage containing them is not null
Testing I've completed
Run through issue steps and now only get first error due to missing/failed-to-locate force-data file.
CHANGELOG.md (choose one)

no need to update because bugfix

Verified locally that I do not get a red circle error after not finding the ExternalLoads.


Addition of elements in fp12

Hello,
CiFEr configures amcl with BN254. The BN254 internally uses fp12 to represent points.
For some implementation we need addition in fp12 which is unfortunately not implemented
in the amcl library. Is there a reason why this is not in general possible.
Aside
Do you know of any pairing friendly curve where addition of curve elements is possible ?

We resolved the issue. We didn't need addition after all.


Add files via upload 2

Have you de-embedded the MZIs used in the filter in separate test structures?


Rework Ceph RBD migration documentation

This patch represents a rework of the current RBD documentation to move it from being a POC to a procedure that we can follow and test in CI.
In particular:

we split the RBD procedure between Ceph Mgr and Ceph Mons migration
we rework both MGR and Mon docs to be more similar to the procedures that the user should follow

@klgill still WIP, but let's start looking at the procedure and fix the style (or build a todo list for a follow up)

Adding /hold to check and review.
We need to have an intro for the Ceph procedure, and the cardinality section should be a separated one (it's not tied to a particular daemon migration).

LGTM, we should wait for @klgill to come back before merging. Render links:
Upstream https://logserver.rdoproject.org/61/461/97458fee23c1268f0bbe562617da31251b0ebdf6/github-check/adoption-docs-preview/0dd7329/docs_build/docs_build/adoption-user/index-upstream.html#ceph-migration_migrate-object-storage-service
Downstream preview https://logserver.rdoproject.org/61/461/97458fee23c1268f0bbe562617da31251b0ebdf6/github-check/adoption-docs-preview/0dd7329/docs_build/docs_build/adoption-user/index-downstream.html#ceph-migration_migrate-object-storage-service

LGTM, we should wait for @klgill to come back before merging. Render links:
Upstream https://logserver.rdoproject.org/61/461/97458fee23c1268f0bbe562617da31251b0ebdf6/github-check/adoption-docs-preview/0dd7329/docs_build/docs_build/adoption-user/index-upstream.html#ceph-migration_migrate-object-storage-service
Downstream preview https://logserver.rdoproject.org/61/461/97458fee23c1268f0bbe562617da31251b0ebdf6/github-check/adoption-docs-preview/0dd7329/docs_build/docs_build/adoption-user/index-downstream.html#ceph-migration_migrate-object-storage-service

Hi @jistr ,
thanks for the review, I agree we should wait for @klgill's feedback, in the meantime I'll rebase the patch!

@fmount FYI I'm going to try to finish my review by EOD Tuesday (US EDT).

Hi @klgill np at all and thank you for the help with this! Take all the time you need and I'll follow your suggestions while you review the ceph doc!

I left off at line 196 in docs_user/modules/proc_migrating-mon-from-controller-nodes.adoc. I'll pick this up again on Wednesday.

I left off at line 196 in docs_user/modules/proc_migrating-mon-from-controller-nodes.adoc. I'll pick this up again on Wednesday.

Np and thank you for all the work you've done so far (not an easy review).
I'm addressing all your comments and we can resume this work from L196 of the mon doc!

@fmount I'm done with my review. After you apply my comments, I'd like to look at the output before we merge.
Overall, I agree with the new structure. I did a more thorough review because that will help us prepare for GA. We can keep iterating on this chapter over the coming weeks.

@fmount I'm done with my review. After you apply my comments, I'd like to look at the output before we merge. Overall, I agree with the new structure. I did a more thorough review because that will help us prepare for GA. We can keep iterating on this chapter over the coming weeks.

Thanks a lot for the multiple reviews and iterations over this procedure. I applied everything you suggested, and it should be almost ready to go (we should look at the preview).
I'll rebase and wait for your feedback in case we want to refine something else.

@fmount I added some minor changes. After those changes are committed, this iteration can be merged.
Note that in the preview, the numbering in the "Migrating Ceph Monitor daemons to Red Hat Ceph Storage nodes" procedure is out of sequence. However, I think we need to discuss breaking up that procedure in another PR because it is very long.
Thanks for your work on this chapter!

@fmount I added some minor changes. After those changes are committed, this iteration can be merged. Note that in the preview, the numbering in the "Migrating Ceph Monitor daemons to Red Hat Ceph Storage nodes" procedure is out of sequence. However, I think we need to discuss breaking up that procedure in another PR because it is very long. Thanks for your work on this chapter!

Thank you for the multiple iterations over this text. I agree to have dedicated patches where we review specific components, rbd is big enough and not so easy to review.
Hopefully now we have a better structure and procedure compared to the previous version!

@fmount This LGTM except for the numbering in the "Migrating Ceph Monitor daemons to Red Hat Ceph Storage nodes" procedure. Currently, that procedure is more than 30 steps total.  It should be split into probably 3 procedures, something like the following:

Migrating Ceph Monitor daemons to Red Hat Ceph Storage nodes

Decommissioning the source node
Configuring the target node to host the redeployed daemon
Applying the ceph mon spec to new nodes ??

Do you want to discuss this further and split this chapter in a separate PR?

@fmount This LGTM except for the numbering in the "Migrating Ceph Monitor daemons to Red Hat Ceph Storage nodes" procedure. Currently, that procedure is more than 30 steps total. It should be split into probably 3 procedures, something like the following:
* Migrating Ceph Monitor daemons to Red Hat Ceph Storage nodes

  * Decommissioning the source node
  * Configuring the target node to host the redeployed daemon
  * Applying the ceph mon spec to new nodes ??

  Do you want to discuss this further and split this chapter in a separate PR?

Hi @klgill yes, I fixed the monitoring stack numbering, so this patch should be good to go now.
I think we can follow up on the other parts with different patches, and discuss how we can split them for better maintenance!


compilation error with observer

good morning ,
I pull caffe2 this morning  and try to recompile it but I got following issue  relative to perf_observer.cc, I might missing something , can you help ?
thanks in advance
here is the error:
###############################################
[ 89%] Built target proto_utils_test
make[3]: Entering directory '/home/b2bot/Project/caffe2/caffe2/build'
Scanning dependencies of target Caffe2_CPU_OBSERVER
make[3]: Leaving directory '/home/b2bot/Project/caffe2/caffe2/build'
make[3]: Entering directory '/home/b2bot/Project/caffe2/caffe2/build'
[ 89%] Building CXX object caffe2/share/contrib/CMakeFiles/Caffe2_CPU_OBSERVER.dir/observers/net_observer_reporter_print.cc.o
[ 89%] Building CXX object caffe2/share/contrib/CMakeFiles/Caffe2_CPU_OBSERVER.dir/observers/observer_config.cc.o
[ 89%] Building CXX object caffe2/share/contrib/CMakeFiles/Caffe2_CPU_OBSERVER.dir/observers/perf_observer.cc.o
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc: In member function ‘virtual bool caffe2::PerfNetObserver::Start()’:
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:54:15: error: ‘class caffe2::OperatorBase’ has no member named ‘AddObserver’
op->AddObserver(caffe2::make_unique(op, this));
^
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc: In member function ‘virtual bool caffe2::PerfNetObserver::Stop()’:
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:77:58: error: no matching function for call to ‘caffe2::OperatorBase::GetObserver(caffe2::ObserverBasecaffe2::NetBase&) const’
op->GetObserver(observerMap_[op]))
^
In file included from /home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:7:0:
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:237:31: note: candidate: caffe2::ObserverBasecaffe2::OperatorBase caffe2::OperatorBase::GetObserver()
ObserverBase* GetObserver() {
^
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:237:31: note:   candidate expects 0 arguments, 1 provided
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:241:37: note: candidate: const caffe2::ObserverBasecaffe2::OperatorBase* caffe2::OperatorBase::GetObserver() const
const ObserverBase* GetObserver() const {
^
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:241:37: note:   candidate expects 0 arguments, 1 provided
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:87:42: error: no matching function for call to ‘caffe2::OperatorBase::RemoveObserver(caffe2::ObserverBasecaffe2::NetBase&)’
op->RemoveObserver(observerMap_[op]);
^
In file included from /home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:7:0:
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:192:8: note: candidate: void caffe2::OperatorBase::RemoveObserver()
void RemoveObserver() {
^
89%] Built target proto_utils_test
make[3]: Entering directory '/home/b2bot/Project/caffe2/caffe2/build'
Scanning dependencies of target Caffe2_CPU_OBSERVER
make[3]: Leaving directory '/home/b2bot/Project/caffe2/caffe2/build'
make[3]: Entering directory '/home/b2bot/Project/caffe2/caffe2/build'
[ 89%] Building CXX object caffe2/share/contrib/CMakeFiles/Caffe2_CPU_OBSERVER.dir/observers/net_observer_reporter_print.cc.o
[ 89%] Building CXX object caffe2/share/contrib/CMakeFiles/Caffe2_CPU_OBSERVER.dir/observers/observer_config.cc.o
[ 89%] Building CXX object caffe2/share/contrib/CMakeFiles/Caffe2_CPU_OBSERVER.dir/observers/perf_observer.cc.o
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc: In member function ‘virtual bool caffe2::PerfNetObserver::Start()’:
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:54:15: error: ‘class caffe2::OperatorBase’ has no member named ‘AddObserver’
op->AddObserver(caffe2::make_unique(op, this));
^
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc: In member function ‘virtual bool caffe2::PerfNetObserver::Stop()’:
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:77:58: error: no matching function for call to ‘caffe2::OperatorBase::GetObserver(caffe2::ObserverBasecaffe2::NetBase&) const’
op->GetObserver(observerMap_[op]))
^
In file included from /home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:7:0:
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:237:31: note: candidate: caffe2::ObserverBasecaffe2::OperatorBase* caffe2::OperatorBase::GetObserver()
ObserverBase* GetObserver() {
^
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:237:31: note:   candidate expects 0 arguments, 1 provided
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:241:37: note: candidate: const caffe2::ObserverBasecaffe2::OperatorBase* caffe2::OperatorBase::GetObserver() const
const ObserverBase* GetObserver() const {
^
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:241:37: note:   candidate expects 0 arguments, 1 provided
/home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:87:42: error: no matching function for call to ‘caffe2::OperatorBase::RemoveObserver(caffe2::ObserverBasecaffe2::NetBase*&)’
op->RemoveObserver(observerMap_[op]);
^
In file included from /home/b2bot/Project/caffe2/caffe2/caffe2/share/contrib/observers/perf_observer.cc:7:0:
/home/b2bot/Project/caffe2/caffe2/caffe2/core/operator.h:192:8: note: candidate: void caffe2::OperatorBase::RemoveObserver()
void RemoveObserver() {

Hi!
I'm experiencing exactly the same error.
Thank you in advance for your help!

This was fixed by this revert commit 3h ago: 4479d781cfb88ebf83a0b89ccf5588116b426843

thanks Pietern for your prompt support, I pull again the repository this morning 10/13/17 and build then  python loading of "core"  is OK


Add logging to long migrations

Also fix v136 to not use models
Closes #11645
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Although an enhancement I think this should be backported because some of these migrations take time and people have complained about poor logging in these already.

So this is weird I'm getting a log timeout for test ...

It will may pass if #11646 was merged?

@6543 the issue was that I had made it so that the migrations never actually finished! Fixed now!

Make lg-tm work


Update nl_NL i18n translations

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.99%. Comparing base (7669cef) to head (033ece2).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             kivy       #7   +/-   ##
=======================================
  Coverage   89.99%   89.99%
=======================================
  Files          61       61
  Lines        3027     3027
  Branches      481      481
=======================================
  Hits         2724     2724
  Misses        217      217
  Partials       86       86

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support iPhoneX screen size

Please ensure you have given all the following requested information in your report.
Issue details
The game screen size is smaller than device screen size. I believe that we have to modify final boolean didFinishLaunching (UIApplication uiApp, UIApplicationLaunchOptions options) {..} method inIOSApplication.java class to fix this error.
Reproduction steps/code
Even if you think your issue is trivial to reproduce, please supply a SSCCE that demonstrates your issue.  This saves time on our end, and makes it much more likely that your issue will be fixed.
You can find barebones templates here
Version of LibGDX and/or relevant dependencies
Please provide the version(s) affected.
1.9.7
ROBOVM 2.3.4 SNAPSHOT
Stacktrace
//Please provide the stacktrace if applicable

Please select the affected platforms

[ ] Android
[ x] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

MOE has the same, I made a ticket on their forums:
https://discuss.multi-os-engine.org/t/warning-when-running-libgdx-with-moe-1-4-0-on-iphone-x-simulator/1247

Yup, we just need to add the correct launch image to both. Will do so later today.

Alright, I'll help testing the MOE side in my app when you have added it.


Idea: Replace "Files" with "Publishers" on front page

Currently the front page stats summary has 3 software focused stats (projects, releases, files) and 2 people focused stats (projects, users). (I consider projects to be collections of people interested in a particular piece of software, hence the appearance on both lists)
What do you think about changing the front page statistics to be: Projects, Releases, Publishers, Users?
Where "Publishers" would be a count of the distinct accounts with admin access to one or more PyPI projects.
My goal with such a change would be to highlight a couple of things:

that PyPI is, first and foremost, a software publishing platform, and publication doesn't happen without publishers
getting a rough sense of the project/publisher ratio (i.e. if we have more projects than publishers, that means the typical bus factor is necessarily less than 1, with each publisher maintaining multiple projects on average)

It would also be interesting to get a sense of the publisher/user ratio, but we know that will be way off, since there isn't much reason to register if you're not a software publisher yourself.

It would also be interesting to get a sense of the publisher/user ratio, but we know that will be way off, since there isn't much reason to register if you're not a software publisher yourself.

I've had a similar assumption, that nobody would register for PyPI unless they were publishing a project, but just for kicks I just checked, we have a total of 159,246 users but only 34,832 user's show up in our roles table (which is where we store the mapping of Project -> User for permissions). This means that we're averaging 2.5 projects per "contributing" user.
As it stands, if we add "Publishers", I'm not sure that it makes much sense to keep "Users", but my immediate reaction is that "Publishers" probably fits the spirit of those statistics we show better than Users does anyway. The only reason it's really "Users" at all is because it was the most obvious thing at the time (the other 3 are just COUNT(*) queries on their tables, but "Publishers" is a COUNT(DISTINCT user).

My rationale behind keeping both would be "Every PyPI user is a potential future publisher". We don't really know what a healthy ratio looks like, but it's a safe bet that there is a healthy range, even if we don't currently know what it is.

@ewdurbin and I chatted about this briefly today, and concluded:

Files is a useful metric, let's keep that
Users is the one that likely needs some change, as pointed out by @dstufft

If someone wanted to pick this up, here's some details to be aware of:

the stats are driven from database triggers that keep the stats (RowCount) up to date
we now have multiple role tables - Role, TeamRole, OrganizationRole - so those need to be taken into consideration
we have owner, maintainer Roles, as well as Org manager, member, Team member - all of these should be counted as publishers

So if someone want ed to pick this up, hopefully there's enough context to get started.


Gender Classification using DL

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Gender Classification using DL
:red_circle: Aim : To identify whether the given face is masculine or feminine.
:red_circle: Dataset https://www.kaggle.com/datasets/cashutosh/gender-classification-dataset
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name : Om Achrekar
GitHub Profile Link : achrekarom12
Email ID<EMAIL_ADDRESS>Participant ID (if applicable): -
Approach for this Project : I will use VGG16, ImageNet and other good architectures to classify faces.
What is your participant role? (Mention the Open Source program) SSoC '23 Contributor

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Issue assigned to you @achrekarom12


Updated README.md

closes #699
updated readme.md file

@Niketkumardheeryan have a look on this pr . Thanks

@Niketkumardheeryan have a look on this pr

@Niketkumardheeryan have a look on this pr

@Niketkumardheeryan can you please merge this pr Pending from more than 40 days . Thanks

@Niketkumardheeryan ????

@Niketkumardheeryan @invigorzz313 have a look on this pr pending since june 3rd

@Niketkumardheeryan

@Niketkumardheeryan @invigorzz313 have a look on this pr pending since june 3rd

please review this pr . Thanks

@Niketkumardheeryan @invigorzz313 have a look on this pr pending since june 3rd

please review this pr . Thanks

@Niketkumardheeryan @invigorzz313

@Meetjain1 we are already working on our readme.md file and your changes are minors ,can't with these changes only


Dependency and build matrix tweaks

Removes rails 4.2 from the matrix as it is covered by the standard Gemfile deps
Removes rubies 2.0 and 2.1 as they're covered by the 2.2.2 ruby
Switches logstash-event to the canonical elastic\logstash repo and v1.5.0 tag

Removing the redundant entries (rubies and rails) will speed up our builds that
were becoming somewhat long in the tooth.

:heart_eyes_cat:


rund: init at version 1.0.0

Motivation for this change
Adding a D Programming Language Compiler Wrapper tool that runs and caches D programs.
This is an alternative to the existing rdmd tool.  It's a rewrite that leverages a new compiler feature that allows it to run the compiler once instead of having to run it twice like rdmd does.  This is a big change to how the cache logic works so I decided to fork the tool rather than trying to update rdmd to support both the old and new logic in the same tool.
It also introduces new features like "Source Compiler Directives" which allow D Language source files to contain compiler configuration. It addresses the same problem described here:
http://chriswarbo.net/projects/nixos/nix_shell_shebangs.html

By using /usr/bin/env, we lose the ability to pass arguments to the nix-shell command. To work around this, nix-shell will look at that second line to get its options.

D programs can use this tool by adding the shebang line #!/usr/bin/env rund to the top of their main source file allowing them to be executed with ./myprogram.d.
i.e.
#!/usr/bin/env rund
//!debugSymbols
//     ^ this is an example of a source compiler directive
import std.stdio;void main() { writeln("Hello World!"); }

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[N/A] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[N/A] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@tadeokondrak thanks for the help, still learning all this nix stuff.

Ping

Wow, I just realized that there have been 3,158 pull requests opened since this one....and it's only been 8 weeks!  That's over 50 pull requests a day!

yes, a lot going on in this repo. sadly sometimes PRs get stalled for this reason.
anyway: can you make a separate commit for the maintainers file?

Ok I've added myself as a maintainer in this PR: https://github.com/NixOS/nixpkgs/pull/70060
Will rebase this change once that gets in.

Rebased