video clicked in related content does not play

The Issue
When clicking a video from the related content below a playing video, the clicked video does not play. Instead the currently playing video briefly pauses and then resumes where it left off, and the related content view is scrolled to its top.
Steps to reproduce

Play a video in portrait mode
Click any video in the related content view below it

System Configuration

Device: pixel 4a 5G
Android version: R

I built Odyssey for Android from source, at a commit from February 18th, commenting out dependencies on GMS core and firebase.
Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of the user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

Fixed.


Example/material autocompletely

Hi @kentcdodds
This repo related to https://github.com/paypal/react-autocompletely/pull/57. Only add example link to README file

Codecov Report

Merging #69 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #69   +/-   ##
=======================================
  Coverage   17.43%   17.43%
=======================================
  Files           8        8
  Lines         218      218
  Branches       42       42
=======================================
  Hits           38       38
  Misses        142      142
  Partials       38       38

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 16421e9...a27ab4c. Read the comment docs.

I just made a pretty big update. Could you update the beta version in your codesandbox and update your code? See the README for docs 😀

haha I got it. Plan to update it :)). Notify to you when it done


Create student setup page

Update setup guide per https://github.com/jhudsl/AnVIL_Book_Instructor_Guide
Ideally would like to use cow::borrow_chapter() but that doesn't appear to be working at the moment.

github is experiencing problems right now. Closing for the time being.


Add track object to PlaybackQueueEndedEvent

What is the need and use case of this feature?
When subscribing to the PlaybackQueueEnded event, I need to use the info about the last track in the queue. The PlaybackQueueEndedEvent includes the track, which is the index, but it does not have a reference to the Track object.
Describe the ideal solution
The new event PlaybackActiveTrackChanged includes the Track objects on the event itself, which is a nice improvement to the API. Also, the semantics are more intuitive, wherein track refers to the object and index refers to the index.
A backwards compatible change would add the Track object to the existing event, but it would mean the semantics are inconsistent across event types. Alternatively, a new event name and type would be non-breaking.
Describe alternatives you've considered
It's easy enough to workaround in the code by making an async call to getTrack in my event handler. I prefer the ergonomics of the newer event.
Additional context
N/A
How I can Help
I'm happy to help test.

@dcvz @puckey what are your thoughts on this feature request?


Deprecate the global spec.high-availability field

On both KnativeServing and KnativeEventing we have:
  high-availability:
    replicas: XXX

Which does a global replica setting of all deployments.apps. This was a good start in the beginning, but I think we should deprecate this, since we can do a much better fine-grain replica setting, using the spec.deployments field

/assign @houshengbo

@matzew We can mark it as deprecated and sterilize the functions related to this field, but will not physically remove the field(keep it for a few releases). Removal of fields in the same CRD version can lead to problems.(Ran into them before, do not want to step over again)
Once there is new CRD version, like v1, ready, then we can remove this field.

high-availability is used by us to set minReplicas for HorizontalPodAutoscaler  for knative-serving
https://github.com/knative/serving/blob/release-1.6/config/core/deployments/activator-hpa.yaml#L25
https://github.com/knative/operator/blob/8738a21c9c4f5b9d7b27fef0cc9230f264e8e2d0/pkg/reconciler/common/ha.go#L68
and i don’t think we can do the same with just the deployments.replicas
https://github.com/knative/operator/blob/8738a21c9c4f5b9d7b27fef0cc9230f264e8e2d0/pkg/reconciler/common/deployments_override.go#L89


Do not enforce semicolons at the end of each line

This dispute happened on slack, so I'm copying the discussion here. It was brought by @facuspagnuolo.
@frangio: imo we have to enforce either all semicolons or no semicolons at all. either rule is quite rigorous but i think we definitely shouldn't be placing semicolons inconsistently
@elopio: yes, consistency is the objective.
@facuspagnuolo: we will always have different opinions about it but if i’m the only one against it I can live with it :slightly_smiling_face:
@elopio: facu I have no opinion, as long as it's either all or none.
I think frangio doesn't like them much
@frangio: no, i don't have strong opinions about semicolons either. aesthetically i prefer no semicolons. but i recognize that there are some corner cases where semicolon injection can cause some confusion
e.g. having an expression in parentheses at the beginning of a line will cause javascript to think it's a function of the previous line's last expression
doSomething()
(x+2).should.equal(3)

my vote is for always semicolons because of the principle of least surprise

I'm closing this. @facuspagnuolo hates semicolons, but he agreed to use them, such a nice guy! :)
If somebody else feels strongly about this, and has better arguments than @frangio, we can resume the discussion.


Added link from notebooks setup to multi-user isolation doc

Fixes https://github.com/kubeflow/website/issues/993

This change is

Preview:
https://deploy-preview-1001--competent-brattain-de2d6d.netlify.com/docs/notebooks/setup/#next-steps
/assign @joeliedtke
A very small PR to review. :)

/lgtm
/approve


[grpc] Update to 1.49.0

What does your PR fix?
Fixes https://github.com/microsoft/vcpkg/issues/26808

Which triplets are supported/not supported? Have you updated the CI baseline?
all, yes

Does your PR follow the maintainer guide?
yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
yes

I have tested the features abseil[cxx17] and grpc[codegen] in the triplets:

x64-windows
x64-windows-static

I think the arm64_windows pipeline just had a hiccup. ffmpeg should not be affected by the changes in this pull request in any way.

grpc、upb、abseil:
All features are tested successfuly in the following triplet:

x86-windows
x64-windows
x64-windows-static


Resource Youtube is redirecting to wrong website

Description
Resource Youtube is redirecting to wrong website
Screenshots
Put any screenshot(s) here.
Do you want to work on this issue?
Yes
How are you planning to resolve on this issue?
I will fix it by adding appropriate link https://www.youtube.com/
Are you raising this issue under hacktoberfest? (Remove this section if you are not a hacktoberfest participant)
Yes, I'm raising this issue under hacktoberfest.

This issue is fixed. Please merge my PR

Fixed


dubbo-nacos ,dubbo工程注册了多个服务，其中某个服务debug断点调试，导致其他服务一些会自动下线。

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: xxx
Operating System version: xxx
Java version: xxx

Steps to reproduce this issue

xxx
xxx
xxx

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
What do you expected from the above steps？
Actual Result
What actually happens?
If there is an exception, please attach the exception trace:
Just put your stack trace here!

That is when you make a debug , the background thread[like zk thread to report who is alive with session timeout] also stopped(you may make a thread breakpoint)

@qixiaobo 但是我断点释放后，下线的服务为什么没有重新上线尼？而且不是全部服务都会下线，部分服务时正常的

@qixiaobo 但是我断点释放后，下线的服务为什么没有重新上线尼？而且不是全部服务都会下线，部分服务时正常的

You can take a thread capture（jstack） to see why we cannot connect to the registry


Simplified postMessage implementation

This PR:

incorporates @KoenLav's commits from #104
merges the latest from master @ cf039a550b7206082b7dc2e17ad5d5834d3eccb2
removes all the overwriting of window.postMessage
implements and documents window.ReactNativeWebView.postMessage as described @ https://github.com/react-native-community/react-native-webview/pull/104#issuecomment-452132323
on iOS WKWebView, proxies window.ReactNativeWebView.postMessage to window.webkit.messageHandlers.ReactNativeWebview.postMessage
on iOS UIWebView, uses the namespaced private API window.ReactNativeWebview.messageReceived instead of using dispatchEvent(new MessageEvent('message:received')) globally (I imagine the related navigation mechanism exists because of some missing capability for handling communication with UIWebView?)

I haven't had a chance to test these changes yet, but I wanted to get it up for review.

 Proceeding to merge!

When will this fix make it to react native build?

I not sure I understand what you mean @mkashle. React-native will not pull back this repository. This repository is part of the lean-core effort to remove non-essential components from the core. You can read more here: https://github.com/react-native-community/discussions-and-proposals/issues/6


Investigate why sis db has studyrights without reference to student number

studyright elements and transfers also have some null studentnumbers

credits too

@vaahtokarkki said: "Toi "Investigate why sis db has studyrights without reference to student number" updater bugi? studedent numberia ei oo study rightissä suoraan ja katoin importer-db:stä et joinas personit ni kaikissa oli kyl student numberit"


cmd, cmd/celestia: Remove dev mode

We no longer need dev mode in its current form. Instead, we can test real nodes against small testnets as was done in Devnet and will be automated by @jbowen93, or via an in-memory test suite as will be implemented by @Bidon15 (Ref #262).
We may later introduce a --dev flag that runs a modified version of the node type, but that will be scoped out in later iterations.
Related to #250.

Approving, though I have a strange feeling that we may miss something here.

What could it be?


Docs: fixing typos and grammar errors in docs

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
I corrected some parts in docs that include grammar errors and typos.
Closes #3840
Type of change
(Remember to title the PR according to the type of change)

[ x] Documentation update

How Has This Been Tested
(Please describe the tests that you ran to verify your changes.)

[x] sphinx-autobuild (read Developer Documentation for more details)

Checklist

[x] I added relevant documentation
[x] I followed the style guidelines of this project
[x] I did a self-review of my code
[x] I made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

@kursathalat this is still WIP right?


Apparmor on Ubuntu Xenial will not permit containers to mount devices.

The same issue is discussed here: https://github.com/moby/moby/issues/9950
This patch resolves the issue by adding the "--security-opt apparmor:unconfined" option in ceph-osd-run.sh.j2, but only for Debian systems.

@netservers looks good, but please squash your commits into one.

@leseb I've not done a commit squash before, and I don't see any option in Github to do it. Do I need write access to your repo to do that?

You have to do this in Git, see my comment here: https://github.com/ceph/ceph-docker/pull/580#issuecomment-290356545
You will have to adapt the commit ID, if you don't feel comfortable doing this then please create a new branch and push a new PR with just a single comm :).

OK. How does that look? It seems pretty squashed to me.

@netservers yes looks good now, thanks!


save client's model

hi, how do I save the model generated by the client in the Personalized Federated Learning Settings? Thanks.

I update the configuration settings in the yarm file as follow:
federate:
  mode: standalone
  save_client_model: True

but it raise an error:
 raise KeyError("Non-existent config key: {}".format(full_key))
KeyError: 'Non-existent config key: federate.save_client_model'

It doesn't work for me.


Broken Link

There is a broken link on Windows Admin Center when clicking "See how to use Windows Admin Center", it links to https://docs.microsoft.com/windows-server/manage/windows-admin-center/use/launch which results in a 404.

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

ID: 239ce76b-e4b8-77d0-8e89-f35d85580e23
Version Independent ID: 4c8f067e-2a3d-2058-6a77-8028e3889843
Content: Get started with Windows Admin Center
Content Source: WindowsServerDocs/manage/windows-admin-center/use/get-started.md
Product: windows-server-threshold
Technology: manage
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Acceptance Test for #25 Status Change Notification

User story
#25
Acceptance criteria checklist

[ ] Users are notified via email when one of their bookings is approved or denied

Accepted


from importlib import reload() for Python 3

Fixes #125

@cclauss thank you for your PR. But you have not followed the Best practices.

Link your PR to a proper issue.
Add a proper commit message
Squash your commits into one

Read through all the links mentioned above please. Especially this.

The best practices need to be updated because GitHub repos that have Allow squash merging turned on allow those with write access to the repo to do squash merges at the touch of a button.  Many contributors might use the GitHub web interface instead of command line git to submit PRs and that interface does not allow those without write access to the repo to squash their commits.

Closed in favor of #146


mutually exclusively require prop

I have a component and it can take a prop of 'src' or children, but never both.  I was unable to find a proptype supplied in this library to cover this.
@ljharb was helping me compose a prop-type for this and while it does seem to disallow the other prop if one is already provided, it doesn't also require that one exist.
here is the proposed prop-type:
static propTypes = {
src: disallowedIf(string, "children", any.isRequired),
children: disallowedIf(
oneOfType([arrayOf(node), node]),
"src",
any.isRequired
)
};
I am happy to contribute a solution for a mutulallyExclusivelyRequired or something similar if needs be.
Thanks!

I also ran into this problem, and ended up using the following solution:
const exclusivePropTypes = {
  foo: fooValidator,
  bar: barValidator,
  baz: bazValidator
}
const exclusiveProps = Object.keys(exclusivePropTypes)

Component.propTypes = {
  ...Object.fromEntries(exclusiveProps.map(exclusiveProp => [
    exclusiveProp,
    AirbnbPropTypes.and([
      exclusivePropTypes[exclusiveProp],
      (props, propName, componentName, ...rest) => {
        const propList = exclusiveProps.join(', ')
        const exclusivePropCount = Object.keys(props)
          .filter(prop => props[prop] != null)
          .reduce((count, prop) => (count + (exclusivePropTypes[prop] ? 1 : 0)), 0)
        if (exclusivePropCount > 1) {
          return new Error(`A ${componentName} cannot have more than one of these props: ${propList}`)
        }
        if (exclusivePropCount < 1) {
          return new Error(`A ${componentName} must have at least one of these props: ${propList}`)
        }
      }
    ])
  ]))
}

Ugh... we're just going to undertype things here if that's the alternative.


write unit test for NotifyVolunteerForUserUnenrolls

@tonysurma please label with "jump-in". Part of Issue #620

If nobody is looking at this, I would like to pick it up.

This looks like its intended for NotifyVolunteerForTaskUnenrollHandler (there is no NotifyVolunteerForUserUnenrolls or near equivalent) in which case its covered by tests and this could be closed /cc @stevejgordon @mgmccarthy

@recumbent Thanks, looks like this is correct so I will close this issue.


Update imports and added tests

Switched to Parcel for building
Updated the default import and class names to be more generic
Wrote some test cases to check the library

Looking very good :)
Thank you for your contribution 😎


Make port configurable via commandline

passing -p  to counts should use the argument to port

fixed in master


PropTypes and CreteClass Fix For React 16

Fixes  PropTypes and CreateClass For React 16

Hey, haven't had time doing mobile stuff but thanks! I will review when I have the time. The SwipeableRow module is a direct paste from their experimental components so we might need to update the whole thing too.

That too seems to work

thanks @hackable


Getting Zero views with premium proxies

Is there an existing issue for this?

[X] I have searched the existing issues

I'm submitting a ...

[ ] bug report
[ ] feature request
[X] support request --> Contact me over mail for support https://github.com/MShawon

Description
Hey, i'm getting zero views, i am using Royal Residential proxies from IP Royal.

These Settings with Format  - username:password@hostname:port
everything is working good except views, i am using 3 different links  .  Pls help Me
Environment
- OS : Win10
- Python :
- Script version : latest

config.json
{
    "http_api": {
        "enabled": false,
        "host": "<IP_ADDRESS>",
        "port": 5000
    },
    "database": true,
    "views": 1200,
    "minimum": 10.0,
    "maximum": 12.0,
    "proxy": {
        "category": "p",
        "proxy_type": "http",
        "filename": "GoodProxy.txt",
        "authentication": true,
        "proxy_api": false,
        "refresh": 0.0
    },
    "background": false,
    "bandwidth": true,
    "playback_speed": 1,
    "max_threads": 5,
    "min_threads": 4
}

Session Management is not done if you set rotation to Randomize IP .. while playing a single video in a chrome session every second your ip is rotating so YouTube simply remove the views...each request is going through different IP we don't want that.. we need each video playing with different IP not a single video going through different IP every second

So change it to Sticky IP and set TTL to 3 to 5 minutes ..or 50% to 70% of your average video length

YouTube needs at least 30 seconds of a watch duration to count it as a view.. so you can't just use minimum 10 maximum 12 follow best guideline or calculate the percentage required to get at least 1 minute watch duration from your average video length


FMODStudio Plugin 2.00.06 crashes when opening Unreal Editor 4.24.1 "Help" menu!

Also, the FMODStudio.uplugin file contains:
	"Version" : 20007, 	"VersionName" : "2.00.07",
Even though I clearly selected 2.00.06 from the website downloads (there isn't even an option for 07 at the time of writing), what's up with that?
Side note: It's a Windows build, by the way. But it doesn't even reach #if PLATFORM_WINDOWS in void FFMODStudioEditorModule::AddHelpMenuExtension(FMenuBuilder &MenuBuilder), so doesn't matter.

The fix for this will be out in the next release, I have also added to a branch for UE4.24, https://github.com/fmod/ue4integration/commit/db6c182c3f7aa6ec3789799717e6723b5263e5dc


Any examples of using this gem w/ new templates features of Postmark?

Hi @twanlass. Unfortunately the idea of storing templates remotely contradicts the design of ActionMailer. Because of that, if you want to use Postmark templates in a Rails app, then you’ll probably want to give up on ActionMailer. And if you decide to do so, the underlying postmark gem has examples of sending templated emails, as well as a wiki page dedicated to templates management via the API.
I’m going to leave this issue open, as we might want to bundle a re-usable abstraction layer with the rails gem, that would simplify the following aspects of integration:

Creating new instances of Postmark::ApiClient. Currently you’ll have to use a shared global instance, introduce a global factory class, or create the client instances in place when needed.
Offline development. In development and test, you’ll probably want to bypass actual sending.
Logging. Some debug output in the logs including template arguments would be very helpful. Especially, in development.
Testing. Possibly we could bundle some test helpers to match the template arguments.

Thanks for bringing this up!

@temochka thanks for the reply. Yea, I assumed this was the case though thought I would I would ask in case I could have both templated emails in ActionMailer along w/ those remotely stored. I'll move forward though w/ creating a shared global instance for now.
Cheers!

@temochka I was inspired by your comment and decided to put together a gem that is essentially a light-weight wrapper around Postmark::ApiClient#deliver_with_template that adopts the same interface as ActionMailer. All you have to do is inherit your mailer from PostmarkMailer::Base instead of ActionMailer::Base, and then update the options you pass into #mail to those you would use with the Postmark API client. Example:
class MyCoolMailer < PostmarkMailer::Base
  def some_email(user)
    mail(
      to: user.email,
      from<EMAIL_ADDRESS>
      template_id: 123456,
      template_model: { first_name: user.first_name, last_name: user.last_name }
    )
  end
end

This can be called in your app just as you would with ActionMailer:
MyCoolMailer.some_email(user).deliver_later

WARNING: I literally just put this together today, so there is minimal documentation and no tests yet (at the time of this comment), so use at your own risk. But for the most simple use cases, it's working, so hopefully it helps some other folks in this situation.
See: postmark_mailer

Hey @SeriouslyAwesome,
This is... seriously awesome! 😄  We’d like to add something similar to postmark-rails, but it needs to have a good story for test and development environments (as I’d outlined above). If you want to take a stab at a pull request, I’ll be happy to review. Otherwise, we’ll get to it eventually :)
If you’d prefer to keep it as a standalone gem, it’s ok too. We can feature it in the community libraries section of our developer docs. It’d still be nice if you could first add some tests and offline development mode.
Thank you for your work!

@temochka makes perfect sense. I've been distracted by another project, but maybe I'll see if I can develop this a little more on this standalone repo until it makes sense to port into a PR for postmark-rails. Glad to hear it's of interest though!

@SeriouslyAwesome Just checking in to see if the plan is still to keep it stand-alone or merge it with this gem?
@temochka Either way, some information about using Rails and template on the site linking to this library would be super helpful!
Thank you for making this possible!

Are there are any news on that matter?
And thanks for what you do 🙇

Seems like there’s quite a bit of interest in this. Unfortunately, no major Ruby gem updates are currently on the roadmap. We have some templates improvements scheduled down the road which might present a good opportunity to build a better integration, too. Stay tuned.
That said, I’ll share a glimpse of our internal thinking. The Postmark web app is built on Rails too, and the possibility of switching from ActionMailer to Postmark templates has come up quite a few times. Usually the arguments in favor of switching bring up the ability for everyone on the team to edit copy and HTML templates and smaller HTTP payloads. Also people want to have access to built-in templates 😄 These are all good reasons, but there is also a number of other concerns:

The experience of managing several different versions of templates across multiple Postmark servers (e.g., staging, production) is far from perfect. This just got a little more manageable with support for template aliases, but some non-trivial scripting is still required.
If you run atomic deploys (Capistrano), you’d want to adopt a convention when you never change existing templates and only add new ones by duplicating the old ones. The alternative is built-in versioning support on Postmark end with ability to link to specific versions.
You also need some conventions if you have multiple developers working on the app or you keep multiple active Git branches at a time. You don’t want to face "merge conflicts" or accidentally overwrite someone’s changes.
Currently no support for partials (this will definitely be fixed).
Last but not least, you lose the ability to test email bodies with Rails (at least offline).

Don’t get me wrong, we actively use templates for smaller apps and services, but with Rails it’s very hard to compete with ActionMailer paired with Postmark adapters. I’d appreciate some feedback from the community on how you’d address the issues above and if they even present a problem to your app. Thanks!

@temochka In my experience, templates have been invaluable to work with when working in a small team or non-tech team members. It saves the pain of porting from Rails to templates or the other way around just to finalize on email templates.
I think a lot of the problems that you have mentioned are very valid for bigger teams but less so for the smaller ones. At the very least, you could have experimental support for templates using ActionMailer where in it is made clear that the workflow isn't ideal but just the first step in this process.

We're just starting to use templates so I can't provide much feedback, but we really value an ability to edit html/text for a non-developers which saves a lot of time ❤
My primary concern is testing. What would really help is a feature to forward outbound emails or at least have some kind of sandbox mode to not send emails to the real users. Forwarding emails to SMTP server would allow us to use tools like mailtrap.

Any news on that matter ? 💃

Btw I just tried the PostmarkMailer gem from @SeriouslyAwesome and it works really well !!

Just wanted to drop a quick follow-up note here that I've published postmark_mailer on Rubygems.org so you can now include the gem without referencing the Github repo: gem 'postmark_mailer'. I've only added a few new things as needed for our use of the gem at work, so no major improvements in test coverage and other normally ideal things a good gem should have, but I may find some time to polish this up a bit more in my personal free time soon.

Also going to test out the gem from @SeriouslyAwesome. I currently evaluate Sendgrid and Postmark and while Sendgrid does provide a way to use their templates, their customisation is not 100% perfect for localised templates because the preheader is not taking the Handlebar syntax into account.
I definitely think there is a reasonable space for providing templates via Postmark/Sendgrid to  ActionMailer, because sometimes its necessary to edit/version/tryout templates independently from deployments. Also if you have API-only endpoints and don't want to bother with static assets, the use of such services is handy.

In the vain of magic in Rails I expected template_alias or template_id and template_model to Just Work in the mailer, as I expect many other Rails' devs would.

Could someone connect the last dot for me ... how would I hook this in to a vanilla Rails + Devise app? That's my scenario, and I bet it's super-common: wouldn't most Rails developers be using Devise to generate their transactional mail?

Fully endorse the above.
If Devise starts getting in the way, you need to untangle it from your views, routes, controllers and mailers. Inherit from Devise base classes and write custom actions if you need to, same for custom mailers, templates. Wrap up your authentication to maintain a consistent interface and simply use that to talk to Devise.
 Devise should integrate into your application, not the other way around.

Thanks! I can see for my simple app's use case, the open source templates will be enough --- and then sending through Postmark which will inline the CSS.


Added documentation regarding controller paths and error with no swagger_root

I noticed other people having issues with something I figured out (no swagger_root) and I found a closed issue referencing how to solve the issue I was having (api versioned paths for controllers) so I added documentation to make the process simpler for future users.

No problem.  Thanks for building the gem!


The filterMode Doesn't Update from includeAll to excludeAll

Version where bug was found:
6.1.0
Describe the bug
When an experiment has no participants, the "filterMode" should be set to "excludeAll". This works as expected when an experiment is created without participants. However, once the "filterMode" is set to "includeAll" (by adding an "All" row to the Include table), it doesn't revert back to "excludeAll" after removing the "All" row from the Include table. I believe this issue should be addressed as a hotfix for v6.0 since it's crucial.
To Reproduce
Steps to reproduce the behavior:

Create an experiment with no participants
Open the "Network" tab in the browser inspector to see the response from the /api/experiments/single/{experimentId} endpoint
Observe that the "filterMode" is set to "excludeAll" (expected behavior)
Update the experiment by adding an "All" row to the Include table
View the response from the /api/experiments/single/{experimentId} endpoint again
Observe that the "filterMode" is set to "includeAll" (expected behavior)
Update the experiment by removing the "All" row from the Include table
View the response from the /api/experiments/single/{experimentId} endpoint again
Observe that the "filterMode" remains set to "includeAll" when it should have updated to "excludeAll"
Verify by making /api/v5/init and /api/v5/assign requests with any user ID to confirm that users are not being excluded

Expected behavior
The experiment's "filterMode" should update from "includeAll" to "excludeAll" when the "All" row is removed from the Include table.
Screen recording
https://github.com/user-attachments/assets/5e78f185-045e-4324-a058-8a62dc15cbe4

I will investigate if there are any other experiments like "LiveHint AI - Tutorbot for MATHstream" that have no participants but have their "filterMode" set to "includeAll" on Prod UpGrade so that we can update them if needed.

I've verified that the filterMode settings for all experiments on Production are correctly set.

QA: Passed


run backup in cron mode.

What's your idea?
There is a way to run ludusavi backup in cli without the "yes" confirmation? I like to add like a  systemd schelude service in the steam deck. Thanks

Hi! You can use the --force option for this.


Simplify setting dynamic default values

Feature Request
Please complete:

OS: Windows
Python version 3.7.4 (default, Aug  9 2019, 18:34:13) [MSC v.1915 64 bit (AMD64)]
Pydantic version: 0.30

Feature Request
Currently, if I want to have a dynamic default value, it has to be done the following way:
from datetime import datetime

from pydantic import BaseModel, validator
from uuid import UUID, uuid4

class DemoModel(BaseModel):
    ts: datetime = None
    id: UUID = None

    @validator('id', pre=True, always=True)
    def set_id(cls, v):
        return v or uuid4()

    @validator('ts', pre=True, always=True)
    def set_ts_now(cls, v):
        return v or datetime.now()

This is fine for cases where actual validation has to be provided, however, in many cases like the one portrayed below, this leads to unnecessary code, especially if there are multiple dynamically set values. For instance, a timestamp and an id.
It would be great to be able to replace this usage pattern by a pydantic field called Dynamic for instance such that one can pass a callable as a default value. This would look like this:
from datetime import datetime
from uuid import UUID, uuid4

from pydantic import BaseModel, validator, Dynamic

class DemoModel(BaseModel):
    ts: Dynamic[datetime] = datetime.now
    id: Dynamic[UUID] = uuid4

I don't expect that this would need to modify any existing functionalities. Probably just a need to create some sort of wrapper.

I guess it can be done with some kind of wrapper

I'm hesitant about pseudo types, without reading the docs it's not clear what this does.
What we could do is just is the function as the default/initial value? Then unless the type of the field was Callable it would call the function to generate a value.
This would not be completely backwards, but I think it would be more intuitive.

This would work as well!

I think this might be better suited as a keyword argument to Schema (or Field now), or similar. That’s how this functionality is handled by both attrs and dataclasses.

I think this might be better suited as a keyword argument to Schema (or Field now), or similar. That’s how this functionality is handled by both attrs and dataclasses.

agreed.

#1210 will implement Field(default_factory=...).
For use of ts: datetime = datetime.now we should wait for v2.

The name default_factory was chosen to be consistent with native dataclasses.field &  #1210.

@153957
I didn't ask why it is the way it is.
I didn't argue against having default_factory.
I suggested also having factory, because it is more ergonomic.

Ok sure, I'm still 👎🏽 on adding aliases for arguments/keywords, no good reason to add the additional complexity and documentation.

Yeah I don't love aliases either, but aliases are a good solution for backwards-compatibly migrating to a better name, and this is right in that middle ground of very minor but permeating improvement that is rarely worth doing on a case-by-case basis in a project, but would gradually ripple a slight benefit across many projects if done upstream.

I do totally sympathize with the resistance. I do the same thing when people bring suggested changes to any of my repos - I don't like messing up and growing my public interfaces with changes unless they seem like clear strict improvements, especially if it doesn't feel like an extremely clear win, especially if it feels like it can be trivially layered on by the user.
But I do think factory is strictly better than default_factory in this context, and I think a backwards compatible alias is better than a breaking name change for a project like Pydantic.

Let me know if I am in the wrong place, I came across this thread posing this question and have a question about v2
https://stackoverflow.com/questions/70026994/is-it-a-good-idea-to-create-derived-non-setable-fields-in-pydantic
Is there a way to allow a default_factory to access other values so the field can be derived from other values? Or is this the sort of thing that a process working with a pydantic class is supposed to handle (eg, we dont want to support derived data in the interest of keeping the class less complex)
Wwhat I have now works as intended, but I am wondering if there is or should be a more idiomatic/explicit way to do this?
In my mind it would look something like

class DynamicValueModel(BaseModel):
    field_1: str
    assigned_id: str

   @validator('assigned_id', derived=True)
   def hash(val, values):
       return hash(values['field_1'])

DynamicValueModel(assigned_id='oops_lol')
# raises ValidationException or ValueError
# 'Cannot assign value to derived field!

Is the default_factory expected to be triggered for None, empty("") values of string?

Is there any way to provide an argument to default_factory callable object?
class Foo(BaseModel):     a: int     b: int = (default_factory=bar(Foo.a))

use partial.

I was going to suggest partial as well, but I think, the real question from @personage-hub lies in his example: "Is there a way to provide model field as an argument to default_factory callable object?".
Quick answer: no.
You can achieve the effect you're looking for via @root_validator, though.

field-with-dynamic-default-value
Hi, I wonder if default_factory is still in beta in v1 latest version, or is it possible to use it in fastapi, Because I encountered a problem about this field in using fastapi.
relevant: default_factory of pydantic's Field not used when included in response_model

is it correctly conceived that the value in default_factory generates once when importing the model? Then how is this different from just default=func()?

is it correctly conceived that the value in default_factory generates once when importing the model? Then how is this different from just default=func()?

Not quite. default_factory is called to generate new value whenever model is instantiated.
You can quickly test this by using functions that return different value:
from datetime import datetime

from pydantic import BaseModel
from pydantic import Field

_counter = 0

def factory():
    global _counter
    try:
        return _counter
    finally:
        _counter += 1

class Model(BaseModel):
    counter: int = Field(default_factory=factory)
    timestamp: datetime = Field(default_factory=datetime.utcnow)

for i in range(3):
    print(Model())


Updated small grammatical errors

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

So I'd love to be able to merge your changes but it looks like you have some merge conflicts. ⚠
Once you resolve these conflicts, I will be able to review your PR and merge it. 😊

If you're not familiar with the merge conflict process, feel free to look over GitHub's guide on "Resolving a merge conflict". 🔍
Also, it's good practice on GitHub to write a brief description of your changes when creating a PR. 📝


ADB/iDevice path not found in IntelliJ plugin

I didn't find a place to report problems with the IntelliJ plugin so i'll post here for now.
When I try to connect, the IntelliJ dialog displays ADB/iDevice path not found with a yellow warning sign.
I am not sure what to do, this worked previously at some point.
Any ideas?

Can you go the the IntelliJ settings -> Other settings -> Moor inspector. There should be a button to test/adapt your configuration.

Can you go the the IntelliJ settings -> Other settings -> Moor inspector. There should be a button to test/adapt your configuration.

Didn't see this before. Same thing there. My colleague has the same result in the settings but he can connect to a device.
I will try to manually configure.

Didn't see this before. Same thing there. My colleague has the same result in the settings but he can connect to a device.
I will try to manually configure.

Does it say anything specific in the test configuration results? Can you include it here?

Does it say anything specific in the test configuration results? Can you include it here?

No but now I configured the correct ADB path and installed libimobiledevice via brew. My colleague hasn't done this and can connect. For me it isn't even listing the device.
I am testing with an attached iPad.
Testing ADB
=======================================
ADB defined at path: /Users/foo/Library/Android/sdk/platform-tools/adb
ADB path seems ok
Checking adb command
Starting adb server
Listing devices
ADB devices returns: 0 device(s)

Testing iDevice
=======================================
iMobileDevice folder defined at path: /usr/local/bin
ideviceinfo seems ok
iproxy seems ok
idevice_id seems ok

No but now I configured the correct ADB path and installed libimobiledevice via brew. My colleague hasn't done this and can connect. For me it isn't even listing the device.
I am testing with an attached iPad.
Testing ADB
=======================================
ADB defined at path: /Users/foo/Library/Android/sdk/platform-tools/adb
ADB path seems ok
Checking adb command
Starting adb server
Listing devices
ADB devices returns: 0 device(s)

Testing iDevice
=======================================
iMobileDevice folder defined at path: /usr/local/bin
ideviceinfo seems ok
iproxy seems ok
idevice_id seems ok

Curious, perhaps he had it installed as a transitive dependency from somewhere else.
As I don't have a readily available iPhone or iPad lying around I can't really check right now. When working with the inspector I am usually running on an ios simulator anyway.
I will have to find some time/devices to see if I can reproduce it

Curious, perhaps he had it installed as a transitive dependency from somewhere else.
As I don't have a readily available iPhone or iPad lying around I can't really check right now. When working with the inspector I am usually running on an ios simulator anyway.
I will have to find some time/devices to see if I can reproduce it

@kuhnroyal can you provide me the output of: idevice_id -l

@kuhnroyal can you provide me the output of: idevice_id -l

I have restarted everything, disabled MacOs firewall but no success.
$ idevice_id -l                                                                                 1 ↵  10151  16:08:40 
34c3e41ae86f22a8063d05026e056b391d61068f

I have 2 iOS simulators running and one attached device. This is the device that is returned.
iPad Pro (12.9-inch) (2nd gen) (mobile)                    • 34c3e41ae86f22a8063d05026e056b391d61068f • ios         • iOS 14.0.1
iPhone 11 Pro Max (mobile)                                 • 8FDA16EA-3C61-4DE3-BB5B-C9F0CD5A13F6     • ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
iPad Pro (12.9-inch) (4th generation) (mobile)             • 664ACE11-3C4C-4947-BBE4-BBE2C5DF0E7C     • ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)

I have restarted everything, disabled MacOs firewall but no success.
$ idevice_id -l                                                                                 1 ↵  10151  16:08:40 
34c3e41ae86f22a8063d05026e056b391d61068f

I have 2 iOS simulators running and one attached device. This is the device that is returned.
iPad Pro (12.9-inch) (2nd gen) (mobile)                    • 34c3e41ae86f22a8063d05026e056b391d61068f • ios         • iOS 14.0.1
iPhone 11 Pro Max (mobile)                                 • 8FDA16EA-3C61-4DE3-BB5B-C9F0CD5A13F6     • ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
iPad Pro (12.9-inch) (4th generation) (mobile)             • 664ACE11-3C4C-4947-BBE4-BBE2C5DF0E7C     • ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)

It also is not showing any Android emulator or devices.

It also is not showing any Android emulator or devices.

Ok, so it is finding the connected device (idevice_id never finds simulators, they work differently).
What does adb devices return as output?
Are you running any apps at that time that use the moor_inspector dart library? This is required for them to show up in the device explorer.

Ok, so it is finding the connected device (idevice_id never finds simulators, they work differently).
What does adb devices return as output?
Are you running any apps at that time that use the moor_inspector dart library? This is required for them to show up in the device explorer.

Yes the app is running, I also verified that the "server started with port number" appears in the logs.
$ adb devices                                                                                   10153  16:10:40 
List of devices attached
emulator-5554	device

Yes the app is running, I also verified that the "server started with port number" appears in the logs.
$ adb devices                                                                                   10153  16:10:40 
List of devices attached
emulator-5554	device

If you try to connect directly to the ipad by entering the port number you see in the logs and it's IP address, does it work then?
The idevice and adb commands are used internally to setup a reverse TCP tunnel to facilitate this for you but manual mode 'should' work. If this works (or does not), we are zeroing in on the issue. Note that this does not work for simulators/emulators.

If you try to connect directly to the ipad by entering the port number you see in the logs and it's IP address, does it work then?
The idevice and adb commands are used internally to setup a reverse TCP tunnel to facilitate this for you but manual mode 'should' work. If this works (or does not), we are zeroing in on the issue. Note that this does not work for simulators/emulators.

If you try to connect directly to the ipad by entering the port number you see in the logs and it's IP address, does it work then?
No, I have tried this but I will try again with the disabled firewall.

If you try to connect directly to the ipad by entering the port number you see in the logs and it's IP address, does it work then?
No, I have tried this but I will try again with the disabled firewall.

I don't know about your network setup, but on LAN the firewall should not kick in.

I don't know about your network setup, but on LAN the firewall should not kick in.

Yea it should not and it also didn't change anything.
Copied the port number directly from the output.

I don't know... since it doesn't work with simulator either, it is probably something weird.

Yea it should not and it also didn't change anything.
Copied the port number directly from the output.

I don't know... since it doesn't work with simulator either, it is probably something weird.

Well, it does not help that I was trying to parse the IP address as the port number 🤦 I will fix this in a new release but this takes time as it needs to be reviewed by JetBrains.
If you still happen to have an ios simulator running could you run the following command:
echo -e '\x01' | nc localhost 6395
This will trigger all local running processes that have moor_inspector on to report their connection information.
Eg: echo -e '\x01' | nc localhost 6395 [{"packageName":"com.test","port":57213,"pid":-1,"protocol":1,"extensions":[{"name":"tag","data":"NTM4OTJi"}]}]

Well, it does not help that I was trying to parse the IP address as the port number 🤦 I will fix this in a new release but this takes time as it needs to be reviewed by JetBrains.
If you still happen to have an ios simulator running could you run the following command:
echo -e '\x01' | nc localhost 6395
This will trigger all local running processes that have moor_inspector on to report their connection information.
Eg: echo -e '\x01' | nc localhost 6395 [{"packageName":"com.test","port":57213,"pid":-1,"protocol":1,"extensions":[{"name":"tag","data":"NTM4OTJi"}]}]

Hmm that worked for the simulator, also seems to work every time I reinstall.

Hmm that worked for the simulator, also seems to work every time I reinstall.

Ok, so it seems the library is working, that's something. So now we need to figure out why it is not showing up on your device explorer...
In IntelliJ -> Help -> Diagnostic Tools -> Debug Log Settings  (that the path for the most recent IntellIJ version at least). You can add categories to log. Add: com.chimerapps. Then try to connect using the plugin and see if there is something related in the IntelliJ logs (or append them here if they contain no sensitive information)

Ok, so it seems the library is working, that's something. So now we need to figure out why it is not showing up on your device explorer...
In IntelliJ -> Help -> Diagnostic Tools -> Debug Log Settings  (that the path for the most recent IntellIJ version at least). You can add categories to log. Add: com.chimerapps. Then try to connect using the plugin and see if there is something related in the IntelliJ logs (or append them here if they contain no sensitive information)

Need to clarify my last statement. After I executed the command, the device shows up in the plugin. And it works with this simulator afterwards without the command.
Gonna test more tomorrow.

Need to clarify my last statement. After I executed the command, the device shows up in the plugin. And it works with this simulator afterwards without the command.
Gonna test more tomorrow.

That's great to hear, even though it makes zero sense to me why that would have worked 🤔

That's great to hear, even though it makes zero sense to me why that would have worked 🤔

Tested again with real devices and 0.1.3 of the plugin but no success.
The device is not showing up, so I can't connect automatically.
Manual connect attempt just closes the window and nothing happens.
There is nothing in the logs after enabling logging, just some Git/Sonar plugins.
2021-01-20 15:06:59,639 [4215094]   INFO - lij.diagnostic.DebugLogManager - Set TRACE for the following categories: com.chimerapps

Tested again with real devices and 0.1.3 of the plugin but no success.
The device is not showing up, so I can't connect automatically.
Manual connect attempt just closes the window and nothing happens.
There is nothing in the logs after enabling logging, just some Git/Sonar plugins.
2021-01-20 15:06:59,639 [4215094]   INFO - lij.diagnostic.DebugLogManager - Set TRACE for the following categories: com.chimerapps

If your devices are on the same network, can you execute echo -e '\x01' | nc <deviceip> 6395 and see if that returns anything?

If your devices are on the same network, can you execute echo -e '\x01' | nc <deviceip> 6395 and see if that returns anything?

Yes it does:
$ echo -e '\x01' | nc <IP_ADDRESS> 6395                                                                                 0|SIGINT(2) ↵  10216  16:15:24 
[{"packageName":"com.example","port":52932,"pid":-1,"protocol":1,"extensions":[{"name":"icon","data":"Zmx1dHRlcg=="},{"name":"tag","data":"MzhkMDMx"}]}]%

But nothing in the plugin window and no manual connection with ip/port possible.

Yes it does:
$ echo -e '\x01' | nc <IP_ADDRESS> 6395                                                                                 0|SIGINT(2) ↵  10216  16:15:24 
[{"packageName":"com.example","port":52932,"pid":-1,"protocol":1,"extensions":[{"name":"icon","data":"Zmx1dHRlcg=="},{"name":"tag","data":"MzhkMDMx"}]}]%

But nothing in the plugin window and no manual connection with ip/port possible.

Do you have a proxy set in your IntelliJ by any chance? That's the only thing I can imagine right now why it would not work on your setup: sockets created from inside the IDE cannot access your localhost (that does not make any sense to me though)

Do you have a proxy set in your IntelliJ by any chance? That's the only thing I can imagine right now why it would not work on your setup: sockets created from inside the IDE cannot access your localhost (that does not make any sense to me though)

Nope, does the plugin start the announcement server and the devices register with it or how does this work?

Nope, does the plugin start the announcement server and the devices register with it or how does this work?

The devices themselves run a single announcement server per device (using automatic master/slave primary/secondary architecture). The plugin loops over all found devices (adb devices, idevices, localhost) and asks the announcement server about all the running clients on that device (this echo -e '\x01' | nc ...). The way it connects to the announcement server is by creating a TCP proxy to the target device (using iproxy, adb forward or without proxy for localhost) to the announcement port (6395). This proxy forwards localhost connections to that port so plugin and device can communicate directly.
That means that if for some reason, the plugin cannot connect a TCP socket on the address returned by the JVM's InetAddress.getLoopbackAddress(), the discovery can never work. The fact that it works on terminal (without the reverse proxy, true) indicates that the system can connect, just the plugin can't.

The devices themselves run a single announcement server per device (using automatic master/slave primary/secondary architecture). The plugin loops over all found devices (adb devices, idevices, localhost) and asks the announcement server about all the running clients on that device (this echo -e '\x01' | nc ...). The way it connects to the announcement server is by creating a TCP proxy to the target device (using iproxy, adb forward or without proxy for localhost) to the announcement port (6395). This proxy forwards localhost connections to that port so plugin and device can communicate directly.
That means that if for some reason, the plugin cannot connect a TCP socket on the address returned by the JVM's InetAddress.getLoopbackAddress(), the discovery can never work. The fact that it works on terminal (without the reverse proxy, true) indicates that the system can connect, just the plugin can't.

Thanks for the explanation. I will try tu run some tests with real android devices tomorrow.
iOS simulators now consistently show up.

Thanks for the explanation. I will try tu run some tests with real android devices tomorrow.
iOS simulators now consistently show up.

I have just released another update to the plugin, I am afraid it does not address the issues you have been having as I am still not sure how/why this is happening.
Any luck during testing?

I have just released another update to the plugin, I am afraid it does not address the issues you have been having as I am still not sure how/why this is happening.
Any luck during testing?

Hi, does this plugin work on Windows? I am using Android Studio 4.2 Beta4 and use only for Android app development. How should I configure the 'select a device to connect to' options. I was able to get the emulated devices IP from Phone status but not sure what port I should use.

@rajumb502 please do not highjack an unrelated ticket to ask for help, create a new one instead.

I tried this again today with 0.2.0 but no luck.
It seems the server port on the iPad is not open or not listening. I did a port scan on the device IP.
flutter: Server running on 50877

Starting Nmap 7.91 ( https://nmap.org ) at 2021-04-26 13:33 CEST
Nmap scan report for <IP_ADDRESS>
Host is up (0.0061s latency).
Not shown: 998 closed ports
PORT      STATE SERVICE
49152/tcp open  unknown
62078/tcp open  iphone-sync

Nmap done: 1 IP address (1 host up) scanned in 14.47 seconds


fix: 'owner' should be 'AccountOwner' in associated-token-account.md

The old wording seemed incorrect. Solana uses the term 'owner' to mean program owner. I believe this sentence is saying the AccountOwner on the new ATA will be the associated wallet, can someone confirm that's correct?

Nice I think this is more clear.
micro-nit: can you wrap this at 80 columns and use ticks on AccountOwner?

So, technically the field that represents the controlling authority of a Token Account is called owner.
https://github.com/solana-labs/solana-program-library/blob/23916b24700cd222ad2397e9de8162ad6980e2e8/token/program/src/state.rs#L93
However, the SetAuthority command used to change it is called AccountOwner.
https://github.com/solana-labs/solana-program-library/blob/23916b24700cd222ad2397e9de8162ad6980e2e8/token/program/src/processor.rs#L441-L462
As you can see in that snippet, it will update owner based on that set_authority call.
So, although I think it's a bit confusing, owner is actually probably more accurate here.
Maybe we can reword this a bit and say:
Regardless of the creator, the new associated token account's [`owner`](https://github.com/solana-labs/solana-program-library/blob/23916b24700cd222ad2397e9de8162ad6980e2e8/token/program/src/state.rs#L92-L93) field will be the wallet, thus it will be controlled by the wallet, as if the wallet itself had created the associated token account.

micro-nit: can you make sure to wrap the columns at 80?

Got it, it sounds like the real issue is the code itself has two things called owner - account owner and program owner. It might be better to switch to more distinct terms in order to avoid confusion.

micro-nit: can you make sure to wrap the columns at 80?

Sure! Existing code is a mix of hard and soft wrapped and I didn't want to reformat unrelated code.

Got it! Your suggestion is definitely clearer. It sounds like the real issue is the code itself has two things called owner - account owner and program owner. It might be even better to switch to more distinct terms (rather than reuse 'owner') in order to avoid confusion. I know that sounds like significant change but if there's going to be ten times as many Solana developers in the future, it may be a good one to make sooner rather than later.

Agreed, and to make matters worse we also call it authority in a lot of places, too. 💀


oao bootstrap fails under node 6 and yarn 1.0.0+ with "Killed" error during yarn link

STR:

Download this testcase: https://github.com/edmorley/neutrino-dev/archive/testcase-oao-yarn-link-killed.zip
Push to a new test repo that has Travis enabled
Whilst waiting for the jobs to complete, edit .travis.yml, replacing the yarn install step --version 1.0.0 with --version 0.28.4
Push to the repo again

Expected:
All four Travis jobs pass (each push triggers jobs on 2 node versions).
Actual:

Using yarn 1.0.0:

node 6: fails (example log)
node 8: passes (example log)

Using yarn 0.28.4:

node 6: passes (example log)
node 8: passes (example log)

Worth noting:

This is using oao 0.10.3 (see testcase's package.json / yarn.lock).
Yarn 1.0.0 is the first release to break, 0.28.4 the last working. This still reproduces using latest yarn too (1.2.0).
Travis caching has been disabled, so there are no node_modules present, and the yarn cache is empty.
In the .travis.yml I'm running oao directly rather than via a yarn run command, to rule out any leftover YARN_SILENT issues, however there is still no useful stdout/stderr from the failed yarn link command (I'll file a new issue for adding more debugging info for cases like these).
There were sadly no yarn-error.log files created that might have shed some light on the error.
I haven't attempted to repro locally, since switching node versions is a pain on Windows.

The yarn 1.0.0 release notes are here:
https://github.com/yarnpkg/yarn/releases/tag/v1.0.0

How's this different from #50?

Oops - I forgot there was a link to a Travis run in the OP of that issue (the 0.25.4 version in the title and the other issues discovered there which are fixed in oao 0.10.3 threw me); the error there looks the same as here.
Though given #50 is about a mixture of issues, perhaps it's worth moving the discussion over here? Happy either way :-)

After the OOM theory in #52, I tried switching from the Travis container infra (4GB RAM) to their fully-virtualised infra (7.5GB RAM), and the node 6 job is now passing:
https://travis-ci.org/edmorley/neutrino-dev/jobs/286159286
So it appears that:

yarn 0.28.4 -> yarn 1.0.0 increases memory usage during yarn link
(unsurprisingly) node 7/8 are more memory efficient than node 6 (or perhaps just more aggressive at GC)

This also explains why we couldn't reproduce locally.
What will be interesting is to see locally how much the yarn link memory usage increased between yarn versions. Either we were always very close to the limit, or else perhaps yarn has had a major regression.  I'll try to dig into this more next week.
On oao's side, it seems that the only actionable items are:

displaying the exit code (since 137 would have pointed at OOM straight away)
fixing the buffering issue (since if we'd have seen some output, it might have been clearer what was going on)

...however since these are #52 I think we can close this now.

Great, thanks for the insightful reporting!


Callback not being sent to specified callbackUrl

Hi there.
The callback event is not being sent to the callbackUrl specified when you call initNetverify or initAuthentication. Instead, it is being sent only to the callbackUrl configured at the Jumio Portal. I tried using the example project but it still doesn't works.
URL configured at the Jumio Portal: http://gatekeeper.uat.quickfin.com.br/biometry/callback
URL set in Flutter: http://gatekeeper.uat.quickfin.com.br/biometry/callback/2
Both URLs are working. I saw my backend application logs and no request was sent to http://gatekeeper.uat.quickfin.com.br/biometry/callback/2, only to http://gatekeeper.uat.quickfin.com.br/biometry/callback.
Jumio scanReference: c02dfbec-1584-4f68-9e27-5f78dc9d11fc.

Hi @brunovsiqueira there is already a ticket for this issue and it's being handled.
Just to update here as well, we're currently getting (targetUrl=https://gatekeeper.uat.quickfin.com.br/biometry/callback]) - org.springframework.web.client.HttpServerErrorException$InternalServerError: 500 Internal Server Error: [{"msg":"unknown error","status":"500"}]) on our end, which means the SDK is getting a server error when trying to reach the callbackUrl that was specified.

Hi @bratapfelmus. The problem is not the fact you guys trying to reach http://gatekeeper.uat.quickfin.com.br/biometry/callback and getting an error. The problem is you guys not trying to reach the callbackUrl I specified at the frontend (http://gatekeeper.uat.quickfin.com.br/biometry/callback/2), only the one specified at the Jumio Portal (http://gatekeeper.uat.quickfin.com.br/biometry/callback). Se the "/2" difference between both.

@brunovsiqueira this issue seems to have been resolved with using https callbackURL instead of http URL.
Will close this for now.


docker-trusted-registry url points to older DTR version

Short url https://docs.docker.com/docker-trusted-registry/ could probably use updated to point to version 2.3.  It currently points to DTR 2.0.
File: index.md, CC @johndmulhausen

Looks like we need to move the redirects from /datacenter/dtr/2.0/index.md to /datacenter/dtr/2.3/guides/index.md.
redirect_from:
- /docker-hub-enterprise/
- /docker-trusted-registry/overview/
- /docker-trusted-registry/


Unescaped translated string in JS breaking the JS

validation_msgs['target'] = '{% trans "Please complete all required fields in the Targets tab." %}'; in indicator_form_common_js.html needs escapejs filter.

This can be seen when loading a program page in French.

@sanjuroj I changed my language to French and brought up a number of program pages where all the targets are not complete, as well as created a test indicator and deliberately didn't fill out the targets tabs to see if validation worked. Everything seems fine, but I'm not sure I'm testing exactly what I'm supposed to?

@hannalcamp Sorry, I should provided better testing instructions.  I found this when I navigated to a program page and tried to expand the indicators to show the results table.  Unfortunately, I didn't write down which program I was trying (it might have been in Jordan?)
The message that was modified was "Please complete all required fields in the Targets tab", so if you can find an create an indicator that would generate that message, switch to French and try to open the results table, that should be a good test.

OK, I'm not getting the errors so I think that's fixed. Two things to note, though:

In the indicator tables, "Missing targets" is not translated. (See TolaData Test for examples)
It looks like "Please complete all required fields in the Performance tab" has not yet been translated either. This screenshot shows what happens when I try to save an indicator with only a name (no level or targets.)

@hannalcamp Thanks for letting me know about the missed translations.  I'll make sure they get translated before we release.


[QUESTION] Server Timeout Configuration Not Working

Problem Description:
I'm facing difficulties in properly configuring the server timeout in the Hyperf framework. I've tried following the documentation and online answers, but so far, I haven't been able to get the timeout to work as desired.
Steps to Reproduce:

Added the timeout configuration in the config/autoload/server.php file as per the provided instructions.
Restarted the Hyperf server.
Made a request to the server with a runtime longer than the configured timeout.
The request was not terminated within the defined timeout.

Additional Information:

Hyperf Version: [insert Hyperf version]
Swoole Version: [insert Swoole version]
Timeout Configuration in the config/autoload/server.php file:'settings' => [
    \Swoole\Constant::OPTION_TIMEOUT => 15,
    \Swoole\Constant::OPTION_ENABLE_COROUTINE => true,
    // Other configurations...
],

I kindly request the assistance of the community in finding the correct solution and getting the timeout to work as desired. Thank you in advance for any assistance provided.

[x] I've been search all existed issues
[x] I've been read all documentation

I use the Hyperf of version 3.0.18，if I set \Swoole\Constant::OPTION_TIMEOUT to the settings，when I start the server，I got an error，Uncaught ErrorException: unsupported option [timeout] in @swoole/library/core/Server/Helper.php:212
I take a look of the source，maybe these setting can be used for you，OPTION_SEND_TIMEOUT，OPTION_SOCKET_SEND_TIMEOUT，OPTION_SOCKET_WRITE_TIMEOUT，etc.

The only configuration that had an effect was OPTION_MAX_IDLE_TIME, but not as expected. Apparently, it closes the process in which the request is being executed. This results in an internal error being thrown, rather than a timeout.


Add location to signup

Currently it is only a text field "city", which is ignored when the user sets their location from jail

What's needed from backend for this?
I'm not sure what the current design is like, but would it be okay to have two API calls: basic details, and then the other stuff? I'm thinking we'd want to put the higher effort stuff on a separate screen so if people think it's too much, we still get their details and can spam them to finish signing up 😈

Don't know if we need two calls. Just need to add lat/lng and TOS agreement to the CompleteSignup call

OK, I can do that


BLD: Add lapack_lite_module back + support win32noblas

Some of the tests still required the lapack_lite module, so this sub-PR adds it back. There was also a failure on win32noblas where xerbla was missing. I think this will fix that too.

Thanks! Incorporated with a tweak.


I know what to do when I have a linux machine

@aslakhellesoy Would it be possible to merge this?

I'm not maintaining this anymore. Try https://rtomayko.github.io/bcat/


ARROW-9766: [C++][Parquet] Add EngineVersion for toggling between new and old logic

Is there a reason to introduce this and burden the API with a new setting? If the "new" engine supports a superset of what the current engine supports, then it seems it should just replace the current engine.

@pitrou that is the plan, but since there will be a decent amount of new code, I think it is worth-while to provide the option.  Once the new engine is implemented that will become the default.
FWIW, I took a similar approach with supporting nested writes as well.

actually, change might be invasive enough that this probably won't mitigate risk, we can hold off until I get a little further.

Going to close this as its been demonstrated existing implementations probably have bugs so, I'm going to focus on a replacing instead of adding code.


Removed: icons/litecoin.svg, renderer/components/Icon/Litecoin.js

Removing Litecoin
Deleting these files is a step towards removing Litecoin completely from Zap.

Hi @SvenSchnieders . Thanks for the PR. Can you make sure that you'r commit message follows out standard format please? See https://github.com/LN-Zap/zap-desktop/blob/master/docs/CONTRIBUTING.md for details. Thanks!


Create A New Welcome Card

Tutorials: https://developers.sap.com/tutorials/cp-mobile-cards-welcome.html
Hi ,
The Welcome Card Template - Single Instance is not downloading automatically it shows downloading but never downloads.
Until we do unsubscribe and subscribe the card from application.
Kindly check on this .
Labels to be added:
prio: Medium
Sprint Label: SCPTest-2106A
Scenario Label: SCPTest-mcard
Landscape Labels: trial1,trial2,trial3
Thanks and Regards,
Vanita

Hi Vanita,
I was not able to reproduce the issue. Thus, can we please sync internally to see what's happening on your landscape?

Hi Vanita,
Since the issue is the same as the one you raised today, I am closing this ticket.
https://github.com/SAPDocuments/Issues/issues/493
Warm Regards
Sandeep T D S


add freenove

git refuses to take my push - so here's the needed file and change:-
cat creations/freenove.md
# Freenove

Community Allocated Creation IDs for [Freenove](http://www.freenove.com/)

Creator ID: 0x1C33_0000

## `0x00D0_xxxx` - WROVER-E (ESP32-D0WDQ6) dev boards

*  `0x00D0-0001` [ESP32-WROVER-DEV-CAM](https://store.freenove.com/products/fnk0060)

$ grep -C5 Freenove  README.md
* `0x1996_0000` [Silabs](./creations/silabs.md)
* `0x1998_1000` [Maker Go](./creations/makergo.md)
* `0x1999_1000` [Sunton](./creations/sunton.md)
* `0x1A00_0000` [Makerfabs](./creations/makerfabs.md)
* `0x1BBB_0000` [Waveshare](./creations/waveshare.md)
* `0x1C33_0000` [Freenove](./creations/freenove.md)

## `0x4xxx_xxxx`

* `0x4496_E3F4` [SQFMI](./creations/sqfmi.md)
* `0x4D69_0000` [Oak Dev Tech](creations/oakdevtech.md)

I see it here just fine: https://github.com/creationid/creators/pull/66/commits/f8630d616676eac4cb776c1251dbe8ef5734922f


Add 'info' folder to .git folder if missing (fixes #403)

Before writing the exclude file for the imported git repos, mbed-cli now checks if the info folder is there. If it isn't it now creates it before writing the file.
This should fix #403.

Works for me, thanks!

For your consideration @screamerbg :)


oneshot: init at 1.0.1

Motivation for this change
I use the package and want to make it easily available to other nixpkgs users.
Things done

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

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

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

Thanks @danieldk for the review. I committed your non-optional changes as requested, so should be ready to merge now.
Also I noticed that there is an update to 1.1.0 already available and updated accordingly. I hope that's okay. If you prefer me to submit a second PR for that update, let me know and I will split it.

Also I noticed that there is an update to 1.1.0 already available and updated accordingly. I hope that's okay.

That's fine! Could you squash all commits into one and use the following commit message?
oneshot: init at 1.1.0

done


refactor(migrate): convert callback to async

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
507

Relevant Lines:
507

💛  - Coveralls


Bundle Idea: Tip/Donation Goal

It should work with any currency and be able to trigger an alert from simple bundles once the goal has been reached.
It should have a rest API.

Done https://github.com/tuxfoo/simple-goals


Can't pass color of type images.ColorHex to images.Padding

Found while testing https://github.com/gohugoio/hugo/pull/12374.
This may also affect images.Dither, where we should be able to pass it a slice of  images.ColorHex.

#12374 is not merged ...

I was assuming it would be because it works great.

Will re-open if necessary.

I have added note about this in the PR ... I don't mind bugs, but I'm not a big fan of bugs on future code ...


Explicitily set window focus

Description
Explicitly set window focus to true so that the application begins rendering on start.
Currently the first frame renders but rendering is then halted until the window is perceived as focused again. For GLFW this takes multiple resizes or min/maximizing the window. Setting focus to true to begin with resolves this issue for me
Fixes #399
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x] My changes do not add any regressions
[ ] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme

@SaschaWillems can you check if this resolves the initial issue?
If it works it may be a better fix to force GLFW to focus the window on creation

This does indeed fix my issue. I can no longer reproduce the bug mentioned in #399 :)

Unfortunately this will not solve situations where the application will lose focus explicitly on launch. For example, when running these samples using the Radeon Developer Panel, the samples cannot be captured unless the sample app regains focus as the panel will focus itself to the top level when a client application (Vulkan app or DX12 app) connects, stealing the focus from the samples.
Could there be a build option or runtime option implemented that removes entirely the check for application focus?

@MattGuerrette I agree, the current implementation doesn't work well with what the samples is trying to achieve. Especially when considering the developer tools use case.
If I recall correctly, stopping rendering when loosing focus was implemented so that in demos phones has a chance to cool down in between long runs without having to close the app. Closing the app would mean we need to wait for assets to re load etc.
That said, it makes sense to allow the user to decide what "optimizations" should be used. I prefer runtime options over compile time options especially when it is something as small as this. Let me see if I can add a quick plugin to toggle this

Interesting, I also needed to bump Volk and Vulkan headers to 1.2.198 to build on windows on my intel laptop... This has not been caught in the CI


[1119][Improvement] Explicitly throw BUFFER_LIMIT_OF_HUGE_PARTITION instead of NO_BUFFER

What changes were proposed in this pull request?
Explicitly throw BUFFER_LIMIT_OF_HUGE_PARTITION instead of NO_BUFFER
Why are the changes needed?
Fix: # https://github.com/apache/incubator-uniffle/issues/1119
Does this PR introduce any user-facing change?
No.
How was this patch tested?
test on prd

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (ecbf2e7) 53.22% compared to head (0d68f5c) 53.45%.
Report is 12 commits behind head on master.

Files
Patch %
Lines

...niffle/client/impl/grpc/CoordinatorGrpcClient.java
0.00%
6 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1393      +/-   ##
============================================
+ Coverage     53.22%   53.45%   +0.22%
+ Complexity     2715     2190     -525
============================================
  Files           418      342      -76
  Lines         23932    15423    -8509
  Branches       2043     1427     -616
============================================
- Hits          12738     8244    -4494
+ Misses        10408     6715    -3693
+ Partials        786      464     -322

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

Could you add some tests for huge partition no buffer exception ?

Ok, I added sendDataAndRequireBufferTest() test case in ShuffleServerGrpcTest. Mainly:

Could you reply all the comments from zuston?

**zuston ** reviewed  Dec 28, 2023
 View reviewed changes

Could you reply all the comments from zuston?

@jerqi  I checked and find one place miss modify, and I corrected it.
I thin all comments is reply and corrected.

**zuston ** reviewed  Dec 28, 2023
 View reviewed changes

Could you reply all the comments from zuston?

@jerqi I checked and find one place miss modify, and I corrected it. I thin all comments is reply and corrected.

You should reply the comment instead of marking the comment resolved. The reply is the your solution. Maybe it will be very simple like fixed. Let reviewer mark the comment resolved.

#1425 Merged. And close this.


[TA] (417) A user can view information about a property

Context
Trello #417
Changes in this PR

Adds a bare-bones premises information page, that can be reached from the premises listing, or by creating a new property

Screenshots of UI changes

Closing as overtaken by pass at bringing this in line with wireframe designs


fix typos in some consensus rules

Motivation
This is small typo fix which can be important for people searching for the consensus rule by a part of its text (a.k.a. me some hours ago)
Specifications
Original text of the rule for reference:

Complex Code or Requirements

Solution

Review
This is super low priority, and comment-only, so we may want to just admin-merge it

Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

Follow Up Work

A few jobs failing downloading protoc.
I am retrying them, maybe resource was not temporally available.


remove ml-commons from 3.0 due to build issue

Signed-off-by: Ashish Agrawal<EMAIL_ADDRESS>Description
Remopve ml-commons due to build failure
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Close this PR as we can use component section to skip ml-commons.


error downlading video

I tried downlading a file with a japanese title
https://www.youtube.com/watch?v=QDeYr-Qhfg0
but failed, I already performed the update with:
youtube-dl -U
but still get the following:

C:\Users\User\Apps\youtube-dl>youtube-dl --verbose https://www.youtube.com/watch?v=QDeYr-Qhfg0
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=QDeYr-Qhfg0']
[debug] Encodings: locale cp950, fs mbcs, out cp950, pref cp950
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg N-91693-g3aacb0d196, ffprobe N-91693-g3aacb0d196
[debug] Proxy map: {}
[youtube] QDeYr-Qhfg0: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 1012, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

#30839 !


Training with coco and evaluation metric

I created the data files for train and val for coco using the prepare_dataset script. I edited it to get a subset of the dataset.
Anyway when i train a network, everything is fine, but when it comes to the evaluation step at the end of an epoch, i'll get the following error:
[07:54:47] /home/travis/build/dmlc/mxnet-distro/mxnet-build/dmlc-core/include/dmlc/logging.h:308: [07:54:47] src/operator/nn/./../tensor/../elemwise_op_common.h:122: Check failed: assign(&dattr, (*vec)[i]) Incompatible attr in node  at 0-th output: expected (32,43,6), got (32,54,6)

Stack trace returned 10 entries:
[bt] (0) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x26ae5c) [0x7fd8eba57e5c]
[bt] (1) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x376c03) [0x7fd8ebb63c03]
[bt] (2) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x377c5b) [0x7fd8ebb64c5b]
[bt] (3) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x214dbd2) [0x7fd8ed93abd2]
[bt] (4) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x213a149) [0x7fd8ed927149]
[bt] (5) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2090721) [0x7fd8ed87d721]
[bt] (6) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(MXImperativeInvokeEx+0x63) [0x7fd8ed87dac3]
[bt] (7) /home/GR_lookup/anaconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call_unix64+0x4c) [0x7fd91315631c]
[bt] (8) /home/GR_lookup/anaconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call+0x1f5) [0x7fd913155a75]
[bt] (9) /home/GR_lookup/anaconda2/lib/python2.7/lib-dynload/_ctypes.so(_ctypes_callproc+0x3e6) [0x7fd91314d126]

Traceback (most recent call last):
  File "train.py", line 134, in <module>
    voc07_metric=args.use_voc07_metric)
  File "/home/GR_lookup/apexAI/mxnet-ssd/train/train_net.py", line 268, in train_net
    monitor=monitor)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/module/base_module.py", line 528, in fit
    batch_end_callback=eval_batch_end_callback, epoch=epoch)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/module/base_module.py", line 244, in score
    self.forward(eval_batch, is_train=False)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/module/module.py", line 596, in forward
    self._exec_group.forward(data_batch, is_train)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/module/executor_group.py", line 407, in forward
    _load_label(data_batch, self.label_arrays, self.label_layouts)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/module/executor_group.py", line 72, in _load_label
    _load_general(batch.label, targets, major_axis)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/module/executor_group.py", line 59, in _load_general
    d_src.copyto(d_dst)
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 1635, in copyto
    return _internal._copyto(self, out=other)
  File "<string>", line 25, in _copyto
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/_ctypes/ndarray.py", line 92, in _imperative_invoke
    ctypes.byref(out_stypes)))
  File "/home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/base.py", line 146, in check_call
    raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: [07:54:47] src/operator/nn/./../tensor/../elemwise_op_common.h:122: Check failed: assign(&dattr, (*vec)[i]) Incompatible attr in node  at 0-th output: expected (32,43,6), got (32,54,6)

Stack trace returned 10 entries:
[bt] (0) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x26ae5c) [0x7fd8eba57e5c]
[bt] (1) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x376c03) [0x7fd8ebb63c03]
[bt] (2) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x377c5b) [0x7fd8ebb64c5b]
[bt] (3) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x214dbd2) [0x7fd8ed93abd2]
[bt] (4) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x213a149) [0x7fd8ed927149]
[bt] (5) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2090721) [0x7fd8ed87d721]
[bt] (6) /home/GR_lookup/anaconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(MXImperativeInvokeEx+0x63) [0x7fd8ed87dac3]
[bt] (7) /home/GR_lookup/anaconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call_unix64+0x4c) [0x7fd91315631c]
[bt] (8) /home/GR_lookup/anaconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call+0x1f5) [0x7fd913155a75]
[bt] (9) /home/GR_lookup/anaconda2/lib/python2.7/lib-dynload/_ctypes.so(_ctypes_callproc+0x3e6) [0x7fd91314d126]

I get the error with both the provided mxnet and the preinstalled mxnet.
I tried voc metric and the other option too.

Okay i found out the problem, i have been setting the label-width to -1 which ended up padding training and val images with different widths causing this problem.

Hi, when I use the default label_width, I can not train. Then I set it to 404. It is ok to train.But when I test it, I met the same kind of problem of you. Can you tell me how to solve it?Thank you very much! @cvasfi


[release-24.05] haproxy: 2.9.7 -> 2.9.10

https://www.haproxy.org/download/2.9/src/CHANGELOG
Fixes: CVE-2024-45506
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 339840 run on aarch64-darwin 1

1 package built:

haproxy


Setting multi-select fails

When attempting to modify the contents of a multi-select field in a table row, an error is thrown.
This happens if you attempt to set with either a list of strings, or a \n separated string.
KeyError                                  Traceback (most recent call last)
<ipython-input-45-3522a46b5071> in <module>
      3 row.name = paper["title"]
      4 row.bibtex = paper["bibtex"]
----> 5 row.authors = "\n".join(paper["publisher"])

~/opt/miniconda3/envs/datascience/lib/python3.7/site-packages/notion/collection.py in __setattr__(self, attname, value)
    361             super().__setattr__(attname, value)
    362         elif attname in self._get_property_slugs():
--> 363             self.set_property(attname, value)
    364         elif slugify(attname) in self._get_property_slugs():
    365             self.set_property(slugify(attname), value)

~/opt/miniconda3/envs/datascience/lib/python3.7/site-packages/notion/collection.py in set_property(self, identifier, val)
    493             )
    494
--> 495         path, val = self._convert_python_to_notion(val, prop, identifier=identifier)
    496
    497         self.set(path, val)

~/opt/miniconda3/envs/datascience/lib/python3.7/site-packages/notion/collection.py in _convert_python_to_notion(self, val, prop, identifier)
    532             if not val:
    533                 val = []
--> 534             valid_options = [p["value"].lower() for p in prop["options"]]
    535             if not isinstance(val, list):
    536                 val = [val]

KeyError: 'options'

I also encountered this problem

Any updates on this problem? I also have the same problem and it seems that my multi-select column can only take one tag instead of multiple ones


Improve rgb input handling

For rgb inputs, these are first converted to ycbcr inputs and stored internally. During gainmap generation, these are again converted back to rgb. These redundant color space conversions are removed.
fixed few compilation warnings
Test: ./ultrahdr_unit_test

Thank you for making this happen! A side question I assume we'll have a follow-up CL in the example app?

Thank you for making this happen! A side question I assume we'll have a follow-up CL in the example app?

Sample app has changes that allow for rgb inputs. so no additional change is required.


ROX-17754: Verify Splunk E2E tests work

Verification PR - DO NOT MERGE
Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

@ivan-degtiarenko As you've self-assigned to this PR, let me give you the TL;DR what happened:
Newer versions since splunk-ta 1.2.0 had been failing the gke-qa-e2e tests, and I investigated why.
Turns out that the eventtypes.conf was missing in newer versions of the TA. This lead to alerts not being generated.
As our tests assert on alerts being created, this led to tests failing (which was correct). So we don't need to fix the tests, but instead need to fix the TA itself :smile:


Todos scheme crashes

I'm not able to run the todos scheme:
CoreData: error: Property 'incompleteCount' is a scalar type on class 'Todos.List' that does not match its Entity's property's scalar type.  Dynamically generated accessors do not support implicit type coercion.  Cannot generate a getter method for it.
2015-06-01 16:15:07.462 Todos[10611:799384] -[Todos.List incompleteCount]: unrecognized selector sent to instance 0x7a1754d0
2015-06-01 16:15:07.468 Todos[10611:799384] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[Todos.List incompleteCount]: unrecognized selector sent to instance 0x7a1754d0'

In List.swift, I changed 'int' to Int64, and now it works


[feature request] Install Zoo as a package

It would be great to be able to install Zoo as a package/module (even better would be one registered to PyPI) so that I can access things like utils from other projects.
Until then I think my options are:

Use sys.path.append(ZOO_REPOSITORY_ROOT).
Add my own setup.py.
Copy/link the code to site-packages.

None of these are great options when I'd like my project to view Zoo as a dependency.

Hello,
I understand your issue. However, the rl zoo is not really meant to be a package.
My main concern is that the current repo is quite big because of the trained agents, which is not the case anymore in the newest version of the zoo: https://github.com/DLR-RM/rl-baselines3-zoo.

None of these are great options when I'd like my project to view Zoo as a dependency.

If you have a custom version of the zoo that is online (with a setup.py, you can always write it as a pip dependency:
pip install git+https://github.com/DLR-RM/rl-baselines3-zoo

would work ;) (with a proper setup.py)

@araffin Thanks for the reply. Yes, I was afraid that was going to be your answer :P Though it's understandable given that there is data in the repo. I was just hoping to be able to use it as a dependency without maintaining a fork with setup.py.
When you say "with a proper setup.py", do you mean I'd have to write one inside (a fork of) zoo? Or I could mangle my own project's setup.py to install zoo as a package (which doesn't have setup.py).
Also, I was not aware of https://github.com/DLR-RM/rl-baselines3-zoo. Or even stable-baselines3. Thanks!

i meant a setup.py in your fork.

Also, I was not aware of https://github.com/DLR-RM/rl-baselines3-zoo. Or even stable-baselines3.

oh...we should advertise it more, we are close to a 1.0 (the library is already usable, we mostly need to polish several aspects)

Good news for you, I'll be migrating most of the utils (wrappers and callbacks) to the sb3 contrib package ;)

@araffin Awesome! Thanks so much for letting me know!!

now done in now done in https://github.com/DLR-RM/rl-baselines3-zoo/releases