Create About us page

Create an About page and also show the name and details of all the contributors on that page and a little bit about our project.

hey @Aayushpatil77 would you like to work on this issue?

Yes

Yes, I would like to work on this issue

Okay you can start, you are given 6 days to complete this.

@RiyaGupta89 would you like to work on this issue?

@RiyaGupta89 would you like to work on this issue?

Yes, I would like to work on this issue.

If you have some demo UI of the page, you can show me. Otherwise I'll figure it out.

If you have some demo UI of the page, you can show me. Otherwise I'll figure it out.

I do not have the UI yet. You can use your imagination.

@RiyaGupta89 I have assigned you and you are given 6 days to complete  this

@RiyaGupta89 I have assigned you and you are given 6 days to complete this

Sure, I'll complete the work in the given time.

@rockingrohit9639 Do I need to add the image of the contributors? If not then what are the details that should be present on the page about the contributors?

@rockingrohit9639 Do I need to add the image of the contributors? If not then what are the details that should be present on the page about the contributors?

You can get all the details of the contributors using GitHub API. Username and their profile image should be there on the about us page.

Hey, @RiyaGupta89 If you are having any difficulties in doing it then you can contact me I can help you out.

Yes @rockingrohit9639, I just want to know from where to get the github API. Then I will fetch and display the details on the about us page.

Yes @rockingrohit9639, I just want to know from where to get the github API. Then I will fetch and display the details on the about us page.

Make get request to this endpoint -
https://api.github.com/repos/rockingrohit9639/project-zone/contributors

https://user-images.githubusercontent.com/72425181/123535730-4ed08080-d743-11eb-8589-0c0b003aa118.mp4
Have a look, does is it look okay?

bandicam.2021-06-27.12-25-35-435_Trim_Trim.mp4
Have a look, does it look okay?

This is looking awesome.

@rockingrohit9639 Thankyou! Should I make PR then?

@rockingrohit9639 I'm not able to make the page responsive.

In the right side of the page which I marked with red color, I don't know why that extra space is left out. Its because of navbar or something or idk. I tried making the page responsive, its not that difficult too but because of that extra space present. I am not able to do it. I also don't know how to resolve it.

@rockingrohit9639 I'm not able to make the page responsive.

In the right side of the page which I marked with red color, I don't know why that extra space is left out. Its because of navbar or something or idk. I tried making the page responsive, its not that difficult too but because of that extra space present. I am not able to do it. I also don't know how to resolve it.

Okay, No problem

@rockingrohit9639 Should I make PR or not?

@rockingrohit9639 Should I make PR or not?

Make it


Can SAMed accept prompts to generate masks? If yes, how can it be done?

Can SAMed accept prompts to generate masks? If yes, how can it be done?

I don't see why it wouldn't be able to accept prompts. Simply modify the code in sam.py.
sparse_embeddings, dense_embeddings = self.prompt_encoder(
            points=None, boxes=None, masks=None
        )

You would probably have to retrain it, as the authors used default embeddings from the image during training I believe.
@hitachinsk lmk if this is correct.

Quite interested in this issue, will the SAMed be used under a prompt-able scenario?
Thanks,


Spice OSS v0.9-alpha endgame

Date: 03/14/2024
DRI: @sgrebnov
Docs DRI: @ewgenius
QA DRI: @sgrebnov
Tag v0.9-alpha; release release-v0.9-alpha

[x] Ensure all outstanding spiceai feature PRs are merged @sgrebnov
[x] Merge Docs PRs

[x] Postgres docs update - @mitchdevenport
[x] Flight SQL @phillipleblanc
[x] Parquet files on S3 - @y-f-u
[x] observability/metrics - @Jeadie

[x] Full test pass and update if necessary over README.md (please get screenshots!) @digadeesh
[x] Full test pass and update if necessary over Docs (please get screenshots!) @ewgenius
[x] ~Full test pass and update if necessary over existing and new Samples (please get screenshots/videos!)~
[x] ~Full test pass and update if necessary over existing and new Quickstarts (please get screenshots/videos!)~
[x] ~Merge Registry PRs~
[x] ~Merge Samples PRs~
[x] ~Merge Quickstarts PRs~
[x] Merge release notes @lukekim
[x] Update acknowledgements @ewgenius
[x]  Similar to https://github.com/spiceai/docs/pull/104/files
[x] ~Update data-components-contrib repo with latest tag and reference it in spiceai. See Components contrib version update~
[x] Update version using documented process at docs/RELEASE.md
[x] QA DRI sign-off @sgrebnov
[x] Docs DRI sign-off @ewgenius
[x] Create a new branch release-v[semver] and merge all relevant changes into it. @sgrebnov
[x] Release the new version by creating a draft release with the tag from the release branch. @sgrebnov
[x] Release any docs updates by creating a v[semver] tag. @ewgenius
[x] Final test pass on released binaries @sgrebnov
[x] Reddit announcement to reddit.com/r/spiceai
[x] Discord announcement to #announcements
[x] For major releases, blog.spiceai.org announcement
[x] Update version.txt to the next release version. @sgrebnov
[x] Update version to the next release version in Cargo.toml. @sgrebnov

PR reference:

Spice: https://github.com/spiceai/spiceai/releases/tag/v0.9-alpha
Docs: https://github.com/spiceai/docs/releases/tag/v0.9-alpha
Blog posts:
https://www.reddit.com/r/spiceai/comments/1bf3xbb/spice_v09alpha_is_now_available/
https://blog.spiceai.org/posts/2024/03/15/announcing-the-release-of-spice.ai-v0.9-alpha/


Remove dead code

Remove references to errorLogs and defaultLogs as they are no longer used

https://github.com/blackflux/lambda-tdd/pull/653


With Points: remove existing solution from exercise

It appears that someone saved this exercise with an existing solutions already written out.

Oops!


fix: grammar fixes, witnesses 32/50 fi

Current dependencies on/for this PR:

main

PR #74   👈

This stack of pull requests is managed by Graphite.


After despawning, new ParticleEffects don't produce particles

Environment
bevy: 0.9
bevy_hanabi: 0.5
OS: MacOS
Issue
After despawning ParticleEffect entities, newly spawned ParticleEffect entities don't produce particles
Reproducing
See the following code for a reproduction: https://github.com/theon/bevy_hanabi/commit/132ce0b17050067e14c07935a8290a1cdd6b7177
The reproduction code spawns a new ParticleEffect entity every second. Once there are MAX_EFFECTS entities it despawns the oldest entity before spawning the next.
Setting MAX_EFFECTS in that example seems to determine when the issue starts:

MAX_EFFECTS=1, 3rd spawned particle effect doesn't create particles
MAX_EFFECTS=2, 4th spawned particle effect doesn't create particles
MAX_EFFECTS=10, 12th spawned particle effect doesn't create particles

Example where MAX_EFFECTS=5 and the 7th spawned entity onwards doesn't produce particles:
https://user-images.githubusercontent.com/759170/209003169-085f376e-8465-41cd-91a8-1649c318d691.mp4
Note: There is no issue when commenting out the line with despawn_recursive()

There's 2 bugs here:

The simple one: some bind group was not re-created, so the new effect was using the bind group of the old effect. Fixed in #107.
The hard one: the indirect compute pass assumes N effects are allocated as N consecutive GPU particle buffers, but when some effects are deleted and their GPU buffer de-allocated, there may be a gap in the numbering. This was compounded by the fact we were first allocating new effect before we deallocated old ones; this part has been inverted in #107 such that despawning and spawning the same number of effects per frame doesn't produce any bug. But there's no easy fix for the case where for example more effects are despawned than are spawned in the same frame (and not in LIFO order).

Re-opening; this is only partially fixed as explained above.

Just ran into this - I'd be happy to help contribute a fix once you know what solution you'd like

Opportunistically closed by #149 as I believe this was the last bug, and everything should be fixed. Please reopen if there's still a case I missed, but RenderDoc shows no error in all cases I looked at. CC: @vultix @theon


#102

Fix for #102 issue.
Main questions:

type for show_resources: {:or, [{:list, :atom}, {:literal, :*}]}. I would much rather have it limited to not any :array, but only resources actually presented in current domain, but i have no idea how to validate it.
I have introduced &AshAdmin.Domain.show_resources/1 which based on domain config returns either all resources for domain or only allowed ones. I use this function in 6 places. It might be too excessive and in some places it might be better to leave access to all domain resources. So question is Should this task limit available resources only in top nav or globally in admin panel?
currently i have not written any tests. Should i add it to page_live_tests.exs Based on previous question scope can change, and amount of tests also would change.

This is more like initial draft, feel free to point out the good, the bad, the ugly.

That looks good to me. Use {:wrap_list, :atom} to allow :atom | [:atom] | :*. To validate this, you would add a transformer to the extension:

defmodule YourTransformer do
  use Spark.Dsl.Transformer

  def transform(dsl) do
     module =   Spark.Dsl.Transformer.get_persisted(dsl, :module)

     all_resources = Ash.Domain.Info.resources(dsl)
     resources =
       case AshAdmin.Domain.Info.show_resources(dsl) do
          :* -> all_resources
          resources ->
             resources
       end

      case Enum.find(all_resources, &(&1 not in all_resources)) do
         nil -> :ok
         bad_resource -> raise Spark.Error.DslError, module: module, path: [:admin, :show_resources], message: "#{inspect(resource)} is not a valid resource in #{inspect(module)}"
      end

      {:ok, Spark.Dsl.Transformer.set_option(dsl, [:admin], :show_resources, resources)}
  end
end

use Spark.Dsl.Extensions, transformers: [YourTransformer]

So the benefit of this is that the list will always be a proper list of resources instead of :*, and you can validate it.

#1 above kind of covers #2 I think.
For #3, the honest answer is that we just haven't really written tests for ash_admin unfortunately. If you would like to test this, that would be amazing and very appreciated, but there won't be a lot of prior art to draw from in this case. Any tests will do :)

We'll need to test that related forms over resources that are no longer shown in admin still work, but otherwise I think this is good to go. I'm not sure if we have any existing forms set up with manage_relationship but that would be the thing to check.

I added tests for AshAdmin.Domain.show_resources/1 i understand that it needs to be tested, but now im not sure how to test it programmatically.

I'll take a look when I get a chance :)
We should make sure that any pages for a given resource are not shown if it's in the show_resources option, not just filter the top dropdown.


feat(ivc): impl base case

Issue Link / Motivation
Part of #31, impl base-case, remastering of #145
Changes Overview
Code is ready, tests in progress

I'm certainly trying to make it not so scary, but there's either performance (so as not to count the same thing) or normal abstractions. I haven't managed to combine them yet.

2m37s base case for sha256 - working
@chaosma is there now an API to check that the circuit synthesis itself is successful? copy constraint on a false match with instance column now passes

2. is there now an API to check that the circuit synthesis itself is successful? copy constraint on a false match with instance column now passes

Not yet, after we add the decider fn verifier (the non-zkp version) we can check whether the U and u are satisfied or not.

Not yet, after we add the decider fn verifier (the non-zkp version) we can check whether the U and u are satisfied or not.

Please create issue with all needed context for verification of all fold-steps. For debug - it's must have, something like feat(table): impl check_correctness, because now we don't have nothing for check this base case

Not yet, after we add the decider fn verifier (the non-zkp version) we can check whether the U and u are satisfied or not.

Please fill #160 with all needed context for verification of all fold-steps. For debug - it's must have, something like feat(table): impl check_correctness, because now we don't have nothing for check this base case

Updated #160. This is essentially to implement IVC::verify(). it depends on other parts of the IVC, e.g. IVC::new().

Updated #160. This is essentially to implement IVC::verify(). it depends on other parts of the IVC, e.g. IVC::new().
Let me know which branch to work on: ivc-base-case or ivc-non-base-case. @cyphersnake

You can work with this branch, fold step will be similar, I debugging it
I will postpone work with #154, to make possible work with rounds earlier (but without it verify will fail, because copy-constraint)


[Code Improvement] Codeunit 12 - Procedure IsOriginalEntryExist

Hello,
I propose to replace all findfirst() by isempty() in procedure IsOriginalEntryExist of codeunit 12.
This change provides a significant speed improvement when posting a journal with many lines applying.
Greetings
J

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.


libeatmydata for running tests and installing dependencies under Travis CI

Description of issue or feature request:
The integration and unit tests take a while to run under Travis CI.  Likewise, installing some of our dependencies have taken ages to complete in the past, but things have fortunately improved on this front.  @trishankatdatadog has directed me to an interesting project that can potentially address some of these concerns.
libeatmydata, as in lib-eat-my-data.
https://github.com/stewartsmith/libeatmydata
Debian uses it.  I notice some Python projects like Numpy use it, as well.
Current behavior:
Installing packages or running tests happen, slowly.
Expected behavior:
Dependencies are installed and unit tests run, quickly.

We have made improvements with the performance of the tests https://github.com/theupdateframework/tuf/pull/1096.
Still, if we want to optimize even more we can consider this.
What do you think @trishankatdatadog @joshuagl @lukpueh ?

What do you think @trishankatdatadog @joshuagl @lukpueh ?

As long as our tests can detect failures induced by libeatmydata, this should be a harmless improvement to make. But if our tests run quickly enough now, it's probably not worthwhile, and we can close this, and revisit in the future.

+1 closing


[v4] feat(theme): Content width css variable for easier customization

This moves the content width that was used throughout nextra max-w-[90rem] to a css variable. This way the changing the value in the future should be easier, but it also makes it possible to customize the max width of the content for people using nextra without needing to overload a tailwind class in a hacky way.

Why:
I have several screenshots in my docs that are a pretty consistent 1280x720, due to the max width of the content view being limited to 90rem these images are always scaled down quite a bit. I was hoping to make the max width a bit wider for this reason but didn't see an easy way to do so without overloading the _max-w-[90rem] tailwind CSS class; which didn't seem like a good idea.
What's being changed (if available, include any code snippets, screenshots, or gifs):
I've simply lifted the value passed to max width to a css variable similar to the other css variables already being managed by nextra. This way I can simply overwrite that css variable in my own styles.
Check off the following:

[x] I have reviewed my changes in staging, available via the View
deployment link in this PR's timeline (this link will be available after
opening the PR).

I don't think we can merge it, because there are people who rely on _max-w-[90rem] class
but in v4 we can do this feature
for now you can override this class in your CSS
.max-w-\[90rem\] {
    max-width: 90vw !important;
}

Do you want me to redo this patch against the v4-v2 branch? I'd be happy to.

I've rebased onto the v4-v2 branch and made the changes there as well. Let me know if you have anymore thoughts!

Didn't know those shorthands existed in tailwind, thank you!

Didn't know those shorthands existed in tailwind, thank you!

they were added in tailwind v4


[BUG] Collection was modified; enumeration operation may not execute

Describe the bug
FO is throwing exceptions when reporting health.
The below is a stack trace found in the log files at: C:\observer_logs\ObserverManager
2021-11-08 00:24:40.3433--WARN--Handled AggregateException from AppObserver:
System.AggregateException: One or more errors occurred. (One or more errors occurred. (Collection was modified; enumeration operation may not execute.))
 ---> System.AggregateException: One or more errors occurred. (Collection was modified; enumeration operation may not execute.)
 ---> System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
   at System.Collections.Generic.List`1.Enumerator.MoveNextRare()
   at System.Linq.Enumerable.Average(IEnumerable`1 source)
   at FabricObserver.Observers.Utilities.FabricResourceUsageData`1.get_AverageDataValue() in D:\a\1\s\FabricObserver.Extensibility\Utilities\FabricResourceUsageData.cs:line 138
   at FabricObserver.Observers.ObserverBase.ProcessResourceDataReportHealth[T](FabricResourceUsageData`1 data, T thresholdError, T thresholdWarning, TimeSpan healthReportTtl, HealthReportType healthReportType, ReplicaOrInstanceMonitoringInfo replicaOrInstance, Boolean dumpOnError) in D:\a\1\s\FabricObserver.Extensibility\ObserverBase.cs:line 1180
   at FabricObserver.Observers.AppObserver.<>c__DisplayClass43_0.<ReportAsync>b__0(ReplicaOrInstanceMonitoringInfo repOrInst, ParallelLoopState state) in D:\a\1\s\FabricObserver\Observers\AppObserver.cs:line 369
   at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`1.<ForWorker>b__1(RangeWorker& currentWorker, Int32 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`1.<ForWorker>b__1(RangeWorker& currentWorker, Int32 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
   at System.Threading.Tasks.TaskReplicator.Replica.Execute()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.TaskReplicator.Run[TState](ReplicatableUserAction`1 action, ParallelOptions options, Boolean stopOnFirstFailure)
   at System.Threading.Tasks.Parallel.ForWorker[TLocal](Int32 fromInclusive, Int32 toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Parallel.ThrowSingleCancellationExceptionOrOtherException(ICollection exceptions, CancellationToken cancelToken, Exception otherException)
   at System.Threading.Tasks.Parallel.ForWorker[TLocal](Int32 fromInclusive, Int32 toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEachWorker[TSource,TLocal](IEnumerable`1 source, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Action`3 bodyWithStateAndIndex, Func`4 bodyWithStateAndLocal, Func`5 bodyWithEverything, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEach[TSource](IEnumerable`1 source, ParallelOptions parallelOptions, Action`2 body)
   at FabricObserver.Observers.AppObserver.ReportAsync(CancellationToken token) in D:\a\1\s\FabricObserver\Observers\AppObserver.cs:line 142
   at FabricObserver.Observers.AppObserver.ObserveAsync(CancellationToken token) in D:\a\1\s\FabricObserver\Observers\AppObserver.cs:line 133
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at System.Threading.Tasks.Task.Wait(TimeSpan timeout)
   at FabricObserver.Observers.ObserverManager.RunObserversAsync()

To Reproduce
Deploy the latest Service Fabric Observer using default config to a fresh 1-node 'OneBox' Development Cluster using the latest version of Service Fabric v8.2.1235.9590)
Expected behavior
Exceptions to not be thrown.
Desktop (please complete the following information):

OS: Windows 10 PRO
Version: 21H1 (OS Build 19043.1288)

Additional context
Add any other context about the problem here.

Thanks for reporting this. Will look into it and fix.

I installed the latest FO (3.1.20) on a one node cluster (dev machine) with a fresh install of SF Runtime 8.2.1235.9590. I am unable to repro this. The OS is Windows 11, but I don't think the OS is the problem. That exception means that a collection was changed while an iteration over it was taking place (typically, this would be a foreach loop). Looking at the stack, my hypothesis is that the collection (Data property of FRUD) was changed while the Average was being computed.
Can you consistently reproduce this behavior?

I have been unable to reproduce this all day. If you have a consistent repro, perhaps we can get on a Teams call and you can show me, we can debug on your machine.

Thanks for the follow-up. I too would doubt that the issue lies with Windows 11.
This is still ongoing:

We grabbed Microsoft.ServiceFabricApps.FabricObserver.Windows.FrameworkDependent.3.1.20.sfpkg from the GitHub releases and deployed via PowerShell command line.
A difference that might be relevant is that we configure our SF Cluster with the following values:

Hosting.EnableActivateNoWindow = true
ReconfigurationAgent.SendLoadReportInterval = 60

Let me know if I can help further. Perhaps a memory dump?
Thanks

Can you zip up your log files, including AppObserver.log and send to me or provide with a link to the zip file on a cloud drive?
ctorre at microsoft.com
Thanks

I updated node configuration with the same two settings you mention above (I don't see how these are related to this issue). I also deployed FO 3.1.20 FrameworkDependent from the SFPKG on the repo. Still no repro.
What version of .NET Core is installed on your one-node cluster? Since you are using FrameworkDependent, this means you know that the same version of .NET Core is installed on the target machine.
Can you try with SelfContained and see if this repros?

Sure, I've emailed you a ZIP archive of the logs.
With “framework dependent” we are running .NET Core 3.1.414.
Will try “self contained” version and let you know the outcome of that later this evening.
Thank you

Thanks for all the logs. You are not doing anything wrong nor do you have any misconfigurations or incorrect runtime version...
This is a concurrency bug.
Until I fix it, please just disable AppObserverEnableConcurrentMonitoring like this:
    <!-- Concurrency/Parallelism Support -->
    <Parameter Name="AppObserverEnableConcurrentMonitoring" DefaultValue="false" />

and change MonitorDuration to 1 like this:
    <Parameter Name="AppObserverMonitorDuration" DefaultValue="00:00:01" />

in ApplicationManifest.xml.
Thank you for reporting this. Since the mitigation is to simply disable concurrent monitoring in AppObserver, you should be set to move forward. Please do remember to keep the MonitorDuration at 1 as it applies to each service process. You can check back on this Issue for updates.

Awesome, that works.
Thank you for looking into this.

Update: There is a fix for this transient concurrency issue in the develop branch. This will ship in 3.1.21. Please clone this repo, change branch to develop, build, deploy to your test machine and see if this happens again. Thank you!

Update 3: FO 3.1.21 RC4 is now in develop branch. More performance improvements.

Fixed in 3.1.21


Remove the Base64 format check

We changed the logic to base64 encoding the continuation token to avoid some encoding issues. To maintain backwards compatibility, we added the logic to check and see if the incoming token is base 64 encoded or not and processes it differently. We should remove this logic after a few releases.

Additional context: Added the encoding in PR #584, the check is in SearchOptionsFactory.cs

@LTA-Thinking is this still an issue?

Yes, we never came back to this. This section needs to be edited so that we only support base 64 encoded continuation tokens.

AB#76206

In order to prevent possible interruptions in export jobs running during a system upgrade a temporary fix had to be added to #1278 so that it checks for base64 encoded CTs. This should be removed soon.

@LTA-Thinking Is this one okay to close?


documentation typo

heigth – The height of the output video. Must be greater than 0.
should be
height – The height of the output video. Must be greater than 0.

This got fixed in #596. I think this can be closed


False positive for keyword "fn"

It was fixed in https://github.com/zargony/atom-language-rust/pull/99
@zargony,
I suppose you should publish a new version.

Okay, sorry, couldn't find it.


These functions neglect to perform correct error checking on memory allocation

This is basically an invitation to get rooted if you use this code in the kernel...

i await your pull request with eager anticipation


Pushables collisions

Pushable objects have a strange behaviour within flipmaps.

Pushable 1 has furthermore this odd behaviour where it can't be pushed forward if it isn't pushed back once.
Pushables need to be moved to trigger an animating on the room above that unlocks a switch, which triggers the flipmap. When the flipmap is triggered, all the three pushable have a collision block next to them that can't be moved.
From here you can download project, tga, wad2, scripts and compiled .trc
Enhancement suggestion:
As future enhancement, it would be interesting prevent pushable pushing/pulling if there is a static mesh on the block where it is not supposed to be moved by Lara.

In the pushables remake, I edited several issues that generated an error like this (specifically, a problem with how the pushables were populating the "Stoppable" flag in the tiles around them).
This situation described in this issue shuldn't happen anymore in the remake pushables.

Fixed applied in version TEN 1.2


I have a new entry for one of the algorithms, can I publish a pull request please?

Is there a way to ask individual contributors directly? how does it work? I'm fairly new to github.
I tried creating a local branch and pushing it to the repo; it says I don't have permission to do that.

Plz refer to https://docs.github.com/en/get-started/quickstart/fork-a-repo

Thanks!
https://github.com/hanabi1224/Programming-Language-Benchmarks/pull/213


Alternate web API to get user posts

There's a new endpoint I started seeing today through the web app. It's accessible in a tab labeled as 'Creator videos' next to the comments tab when viewing a video accessed through the user page.
It currently doesn't require any signatures or valid tokens (like the verifyFp) to return data
Minimum Params
Latest: https://www.tiktok.com/api/creator/item_list/?aid=1988&type=1&count=15&cursor=1689909282235&secUid=MS4wLjABAAAAys5jIEEWJXF-bCt_11wTP363Xvnma4SAex4cTxY-EDrUe2CPXUr1rh-tZtqa5Src&verifyFp=verify_
Next Page: https://www.tiktok.com/api/creator/item_list/?aid=1988&type=1&count=15&cursor=1677186594000&secUid=MS4wLjABAAAAys5jIEEWJXF-bCt_11wTP363Xvnma4SAex4cTxY-EDrUe2CPXUr1rh-tZtqa5Src&verifyFp=verify_

secUid
aid

1988

type

1 returns sorted item lists on all requests
0 returns item lists with dupes unless its the most recent 15 posts

count

Max 15

cursor

Setting to current time in milis will return the most recent posts
The 'next' cursor used on frontend is the timestamp of the last post in the item list
Setting to 0 will return the oldest posts

verifyFp

Any string is valid as long as it starts with verify_

Res

No next/previous cursor values
hasMoreLatest & hasMorePrevious = true/false

Normal params
https://www.tiktok.com/api/creator/item_list/?aid=1988&app_language=en&app_name=tiktok_web&browser_language=en-US&browser_name=Mozilla&browser_online=true&browser_platform=Win32&browser_version=<>&channel=tiktok_web&cookie_enabled=true&count=15&cursor=<>&device_id=<>&device_platform=web_pc&focus_state=true&from_page=video&history_len=4&is_fullscreen=false&is_page_visible=true&language=en&os=windows&priority_region=US&referer=&region=US&screen_height=1080&screen_width=1920&secUid=<>&type=0&tz_name=<>&verifyFp=<>&webcast_language=en

Nice find. I've written a version of the user extractor that uses the new endpoint and ditches the playwright dependency; take a look / try it out:
https://github.com/bashonly/yt-dlp-TTUser/tree/creator
It has a problem, though:
The first API request (with the cursor as current timestamp in milliseconds) works fine; it returns the 15 most recent videos posted by the user.
But when the next page is requested with the cursor set to the timestamp of the final video from the current page, that request yields only 5 new videos and 10 duplicates from the previous page. This happens for every subsequent page request.
Not sure what do to about this. I don't like the idea of assuming that the first 10 videos of each subsequent page are dupes and skipping them, since that could result in non-dupes being skipped.

Looks like its just the type param that I missed on subsequent calls. Setting it to 1 on all requests will return properly ordered items. Setting it to 1 on the first request also gives the same result as 0
All that changes by setting to 1 is the hasMoreLatest value it removed, but its not needed since we're only going down the post list

Right on, pushed working implementation to master

Awesome, I can confirm it's working perfectly in multiple regions.
Now we wait and see if/when the tiktok ninjas find this and neuter it.

Closed by 9919edfd34f2de32ac83126a1883574fa977fb67


Attempt at deobfuscating unknown instruction approval

That is how it looks like.

I think each account should be enriched to display isWritable and isSigner for each account. I will investigate how to do that.
for isWritable it seems we can get it from https://solana-labs.github.io/solana-web3.js/class/src/message.js~Message.html#instance-method-isAccountWritable
I also need to check how it looks when there are multiple unknown instructions
#91

That is where it is at, an example with 2 unknown instruction


feat(form-field): support for different spec variants

Add support for variants and create 3 to start with: legacy (same as current master), standard (same as current master + a couple tweaks to be more in line with the spec), box (the box style from the spec).
Demos:
https://mmalerba-demo1.firebaseapp.com - current master
https://mmalerba-demo2.firebaseapp.com - default variant = legacy (should be identical to master)
https://mmalerba-demo3.firebaseapp.com - default variant = standard
https://mmalerba-demo4.firebaseapp.com - default variant = box
Couple notes:

legacy is the actual default I used, so nothing should change for anyone unless they explicitly set a variant.
I didn't add the background ripple shown in the box spec yet.
Non-floating labels really don't work well with the box variant because of how the label is aligned (see the "first name" input at https://mmalerba-demo4.firebaseapp.com/input). Should we allow it to look broken? Ignore the floatLabel="never" option?
Prefixes & suffixes look funky in the box variant. The reason is that we're using prefix/suffix for 2 different purposes:

Icons/buttons such as a password visibility toggle (e.g. https://storage.googleapis.com/material-design/publish/material_v_12/assets/0B5ZSepuCX1xOdEc4cHZWV3ZzTzg/password1.png)
Things like putting a '$' symbol in front of the user-inputted value. (e.g. https://storage.googleapis.com/material-design/publish/material_v_12/assets/0B5ZSepuCX1xOdVBQcERBbTJpSm8/prefix1.png)

I think (ii) is actually a misuse of the prefix/suffix. It should behave more like a placeholder that only appears once the label floats up. The spec provides more examples of this kind of thing here: https://material.io/guidelines/components/text-fields.html#text-fields-input-types. I think we should probably add support for formats to MatInput to handle this case rather than relying on the form-field's prefix/suffix. If you guys agree with this I'll go ahead and make the prefix/suffix behave like the icon case in the spec for standard and box and leave the legacy behavior as is.

I don't get it. Why so drastic change in the behavior of the non floating label? I liked that feature and I'm using it in production. Can't we not place that mat-label and have a placeholder like thing, but with the required asterisk, you can put a colored asterisk in the placeholder attribute normally. With that drastic change I'll be forced to stick to 5.2.1. So is it possible to make 1 more component mat-placeholder to act like a placeholder, but have a required mark and be possible to not use mat-label for the purpose of a placeholder

@talamaska I've upped the priority on #7645 for adding text mask support to the input. I'm not 100% sure our text masking solution will be able to cover this use case, but I'll keep it in mind while designing.
The never-float placeholder stuff was really just an artifact of the fact that the spec didn't used to distinguish between label and placeholder and we therefore needed some way to emulate standard placeholder behavior with the floating label. The spec is now much more clear about the fact that the label and placeholder are two different things and the required * really belongs with the label

Having a required mark in the placeholder doen't have anything to do with the text masking. It is a totaly separated feature.


It ain’t working in my browser..

I cloned the repo, opened chrome and navigated to the the file, chose index.html and it was blank. I tried commenting the css link line and it worked without css!! Please help!! I got her birthday on 27th 😬😓

I am really just speculating, but I think that the problem may be CORS request, so I would recommend hosting or running it on a local server and trying again. Because just opening the file directly with chrome will most likely end up in CORS request errors.

@Creativty thank you so much!! It worked in localhost!! 😅

Sorry, didn't notice it sooner! 😅
@Creativty is correct, it doesn't link the javascript file correctly due to CORS when you open the HTML directly. But that's because I was loading the script through babel (for transpiling). Reading this issue, I reconsidered if I really need that anymore. It's almost 2021 and the code used in the project is supported by all modern browser out of the box now, I believe.
So I dropped it. 🙂
Loading the HTML file directly will work now. The JSON file doesn't work that way though. You'll still need to run it on a (local)hots to get that working.
Hope you got to make it work in time @abood1703? 😃

@faahim I understand now!! Yeh It worked on time 😅
Thank you so much for this project. It helped me a lot 🖤

Always fills my heart with joy hearing someone (and their friend/'her' 😉 ) liked/enjoyed it.
Thank you for using the project! 😊
Gonna close the issue. 🙂


Feature/normalise file upload label

Whilst browsing Storybook I noticed that the FileUpload component rendered bold label text when there was an error, unlike any of the other form fields.
This change resolves this by removing the error prop passed to the LabelText component in FileUpload bringing it inline with the other form components and stops the label from rendering bold when there is an error.

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged

I'm a bit confused as to what the intentional behaviour is, the following screenshot is from  https://govuk-elements.herokuapp.com/form-elements/example-form-elements/

It shows a drop down with a bold label and regular hint.
The file upload has a regular label and no hint.
I'm not sure on:

why the inconsistency in label font weight?
is it possible to have a file upload with both a hint and a label?
what does govuk-frontend do?

Ah, thanks for that link, I wasn't aware of that.

why the inconsistency in label font weight?

I am not sure why this is the case, however I think we should support both and can add this in so that it is an option for whoever is implementing it. Would you agree with this?

is it possible to have a file upload with both a hint and a label?

By looking at the component code and both the govuk-elements this appears to be supported. We currently do not have an example of the file upload component without a hint in storybook.

what does govuk-frontend do?

They do not support the bold formatting: https://github.com/alphagov/govuk-frontend/blob/master/src/label/_label.scss

I think the confusion was because some example fields had bold labels and some didn't. It was like, some had bold when there was an error and  then some didn't

That makes sense - it looks like this was never locked down previously and was left as an option for people implementing a form label to decide whether they wanted it regular or bold.
From the highest priority link you provided Alasdair, their form label implementation allows an optional class: https://github.com/alphagov/govuk-frontend/blob/master/src/label/_label.scss
So I guess I should confirm whether we want to carry this through on this project, and, if everyone else is happy for me to make the change on this PR?


Decimal amount not exact

In quiffen version 1.2.1 from PyPi, this line in QIF file:
T-143.66
is translated to:
amount=Decimal('-143.659999999999996589394868351519107818603515625')
Perhaps issue #5 fix got dropped somewhere???

I have not understood the code yet, but this code in accounts.py looks problematic:
elif line_code == 'L':
kwargs['credit_limit'] = Decimal(round(float(field_info.replace(',', '')), 2))
elif line_code == '$' or line_code == '£':
kwargs['balance'] = Decimal(round(float(field_info.replace(',', '')), 2))
field_info is a string, float() converts it to a float, which loses the exactness that we want from Decimal.

This is a known issue and it's frustrating me to no end. That was an attempt to sort it, but it doesn't seem to have worked.

I think I have most of it. All uses of float are evil.
Let me prepare a patch for you to review...

I sent you a Pull Request.
It seems to fix the problem in Transactions, at least on my laptop.
Let me know what to do next, because I am a n00b actually working with Git.
However, Loan info is still kept in Float, and I think that should probably
change, too.
On Wed, Feb 2, 2022 at 6:07 AM Isaac Harris-Holt @.***>
wrote:

This is a known issue and it's frustrating me to no end. That was an
attempt to sort it, but it doesn't seem to have worked.
—
Reply to this email directly, view it on GitHub
https://github.com/isaacharrisholt/quiffen/issues/18#issuecomment-1027873357,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3AQGKRUN6UEMVZHM4LB43UZEM6PANCNFSM5NKEHL3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Fixed as of Quiffen v1.2.2


Fix in case 'type' allows more than two types

there was an error in my implementation, leading to false negatives.
on that occasion:

small code-cleanups
output the errors in case an assertSuccess fails

thanks for the spot, it was even missing a test-case.

pushed the fix.

thanks!


What is the point of this compared to HTML ?

What is the point of Nota compared to plain, standard, HTML ? Even more when it's completely replicating the HTML anchors and structure ?
I don't know if the goal is to be a bit higher level than markdown (than can be edited and read perfectly out of the browser) and like, but I find it too close to HTML, so that it's not possible to read it out of the browser (and even worse: without JS).
The good point I see is being able to specify variable, a bit like HTML's <cite> tag and perform simple computations (which aren't computed without JS).

@ekzhang is correct, but just to add: Nota generates plain HTML. It uses HTML anchors and structure in the standard way. Nota does not replace HTML, but rather augment HTML. But HTML has a very limited set of components. Only with Javascript can you do things like toggle between natural language and math.

Can it generate something other than HTML though? For example LaTeX? Or directly PDFs?
Otherwise the value-add compared to HTML or HTML-JSX seems low...

To be clear, "Nota" consists of three things:

A syntax for writing React Javascript programs, i.e. a JSX alternative.
A set of React components for structuring a document.
A code editor that specifically supports Nota syntax and components.

Each of these provides a different kind of value. The syntax makes it easier to specify structured, interactive documents without the syntactic noise of HTML or JSX. The components enable authors to describe certain structures in their document (e.g. a definition and a reference to that definition), which in turn enable readers to get a better document-reading interface (e.g. clicking on a reference and seeing a tooltip containing the definition). And the editor makes it easier for authors to create these documents.


Can't download some video on bilibili.com

youtube-dl.exe -v http://www.bilibili.com/video/av6759147
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', 'http://www.bilibili.com/video/av6759147']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: none
[debug] Proxy map: {}
[BiliBili] 6759147: Downloading webpage
ERROR: Unable to extract player parameters; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmphn9u2rzv\build\youtube_dl\YoutubeDL.py", line 694, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmphn9u2rzv\build\youtube_dl\extractor\common.py", line 356, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmphn9u2rzv\build\youtube_dl\extractor\bilibili.py", line 48, in _real_extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmphn9u2rzv\build\youtube_dl\extractor\common.py", line 659, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract player parameters; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
But some one none copyright videos can download fine.

Proxy can be obtained here: https://vnet.one/?rc=2879
You can register unlimited free accounts which temp emails for unlimited 256 MB data, which is what I am using currently...
Use Google Chrome for translation if you don't understand the web page
@yan12125

This video has a new URL now, which will be supported in the next version:
$ youtube-dl -v "http://bangumi.bilibili.com/anime/5532/play#96395" --geo-verification-proxy https://secure.uku.im:993/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://bangumi.bilibili.com/anime/5532/play#96395', '--geo-verification-proxy', 'https://secure.uku.im:993/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.02.01
[debug] Git HEAD: bd8f48c78
[debug] Python version 3.6.0 - Linux-4.9.5-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 3.2.2, ffprobe 3.2.2
[debug] Proxy map: {}
[BiliBili] 96395: Downloading webpage
[BiliBili] Downloading episode 96395. To download all videos in anime 5532, re-run youtube-dl with http://bangumi.bilibili.com/anime/5532
[BiliBili] 96395: Downloading JSON metadata
[BiliBili] 96395: Downloading video info page
[debug] Invoking downloader on 'http://ws.acgvideo.com/1/39/11006760-1hd.mp4?wsTime=1486058583&wsSecret2=04d779e8673aa22b623eb32bfbddba6f&oi=2344923069&rate=190'
[download] Destination: 少女编号-96395.mp4
[download]  19.0% of 182.86MiB at 190.24KiB/s ETA 13:17^CException ignored in: <_io.FileIO name='少女编号-96395.mp4.part' mode='wb' closefd=True>
ResourceWarning: unclosed file <_io.BufferedWriter name='少女编号-96395.mp4.part'>

ERROR: Interrupted by user


docs: improve docs content

What kind of change does this pull request introduce?
Improving the docs, fixing typos, and making sure the documentation is clear.
What is the new behavior?
The docs are crystal clear.
Checklist

[ ] Documentation
[ ] Ready to be merged

@sannek added some improvements from my side. For the rest, for me, it's logically clear. On the other hand, I can't assess in many cases it's clear enough to use it.

Looks good to me! I suppose I should also use this branch to do my copy editing/nitpicking, because otherwise i'm looking at dated stuff?
I'll fix the merge conflict and go over it as well.

Yes please! Also I'm still planning to go through on this once again. So it's a good collaborative experience in CSB projects! Thanks!

Actually - i realised that my interactive examples aren't on this branch - so we really should merge this one asap and get on a new branch.
I'm still finishing up on the custom-content guide, but I'm a bit stuck on it so figured I'd take a little break to edit the rest first.


SSLError (SSLCertVerificationError) when using gdown from some machines

When using gdown to try to download a file with file id _gdrive_long_file_id_  being a real file id:
$ gdown --id _gdrive_long_file_id_

I get the next error when doing so from some servers (it works from other machines):
Traceback (most recent call last):
  File "/.../py3.8.2/bin/gdown", line 11, in <module>
    load_entry_point('gdown', 'console_scripts', 'gdown')()
  File "/.../gdown/gdown/cli.py", line 104, in main
    download(
  File "/.../gdown/gdown/download.py", line 115, in download
    res = sess.get(url, stream=True, verify=verify)
  File "/.../python3.8/site-packages/requests/sessions.py", line 546, in get
    return self.request('GET', url, **kwargs)
  File "/.../python3.8/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/.../python3.8/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/.../python3.8/site-packages/requests/adapters.py", line 514, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='drive.google.com', port=443):
Max retries exceeded with url: /uc?id=_gdrive_long_file_id_ (Caused by
SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED]
certificate verify failed: self signed certificate in certificate chain (_ssl.c:1108)')))

This seems a known problem (e.g., see this issue) with the verification by Requests of the server's TLS certificate. A solution to this issue is to provide the users of gdown with a way to control Requests' verify parameter (it defaults to True, but see details in the API docs). I will link a PR for this solution to be considered and merged. Thanks.

With the changes in the PR, now:
$ gdown --id _gdrive_long_file_id_ --no-verify

is finally working (with some previous relevant warnings by InsecureRequestWarning):
  warnings.warn(
/.../python3.8/site-packages/urllib3/connectionpool.py:997: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
Downloading...
From: https://drive.google.com/uc?id=_gdrive_long_file_id_
To: /.../_gdrive_long_file_
673MB [27:38, 406kB/s]

With the changes in the PR, now:
$ gdown --id _gdrive_long_file_id_ --no-verify

is finally working (with some previous relevant warnings by InsecureRequestWarning):
  warnings.warn(
/.../python3.8/site-packages/urllib3/connectionpool.py:997: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
Downloading...
From: https://drive.google.com/uc?id=_gdrive_long_file_id_
To: /.../_gdrive_long_file_
673MB [27:38, 406kB/s]


Undefined variable $categoryKey

Hi,
We are having some trouble lately with a Contact3 form on an up-to-date Bludit website. When I try to submit the form I get the following error: "Undefined variable $categoryKey".

Bludit v3.13.1
Build 20200728
PHP v8.0.10
OS Alpine Linux v3.14

Thank you in advance for your help.

Hi,
in Contact3 the variable $categoryKey isn't used. Could it be an other plugin?
Is it possible to get the whole php error log?

Hi @david-novafacile, thanks for your answer!

in Contact3 the variable $categoryKey isn't used. Could it be an other plugin?
Is it possible to get the whole php error log?

@myrdn could you take a look please?

@david-novafacile I double-checked the problem and realised (thanks to my colleague) that the plugin wasn't up-to-date. I upgraded it to the last known version published on GitHub (v2.1.0) and now the error is different:

Oops ! Une erreur est survenue pendant l’envoi de votre message, merci de réessayer ultérieurement.

I went through the online documentation you provide at https://bludit-plugins.novafacile.com/#contact3 and wonder now if we aren't missing some PHP dependancies. Is there some requirements that can be missing from our setup?
Also, there is nothing really useful on our webserver logs, no error is logged when I send the request.
00.ZZZ.YYY.XXX - - [05/Oct/2021:16:26:27 +0000] "POST /contact HTTP/1.1" 200 11989 "https://REDACTED/contact"

Thanks again for your time.

Thank you for your help.
We use our own email infra, so there is no limitation on this end.
I activated the debug logs and here is the lines printed on the page:
Deprecated: Required parameter $filename follows optional parameter $path in /var/www/bludit/bl-kernel/helpers/filesystem.class.php on line 220

Deprecated: uasort(): Returning bool from comparison function is deprecated, return an integer less than, equal to, or greater than zero in /var/www/bludit/bl-kernel/pages.class.php on line 419

Warning: ini_set(): Session ini settings cannot be changed when a session is active in /var/www/bludit/bl-kernel/helpers/session.class.php on line 11

Warning: session_set_cookie_params(): Session cookie parameters cannot be changed when a session is active in /var/www/bludit/bl-kernel/helpers/session.class.php on line 27

Warning: session_name(): Session name cannot be changed when a session is active in /var/www/bludit/bl-kernel/helpers/session.class.php on line 31

Notice: session_start(): Ignoring session_start() because a session is already active in /var/www/bludit/bl-kernel/helpers/session.class.php on line 34

So it seems there isn't any error at all... I'm a little lost with this problem. :(


[rush] [engsys] Disable build cache

The azsdkjsrush storage account has seemingly been deleted, so this will always fail and waste effort during CI.
Tracking issue for a full fix: #28865

/azp run js - core - ci


Translation setting

Translation setting seemingly not using official api??
I read your code looks directly modify the dom???

Yes. This package will modify the dom in settings view.
Do you know another approach for translation with official api?

no...


Feature/build description

@chetanddesai added build description

[Current coverage][cc-pull] is 27.04%

Merging [#38][cc-pull] into [master][cc-base-branch] will decrease coverage by 4.50%

@@             master        #38   diff @@
==========================================
  Files             6          6
  Lines           168        196    +28
  Methods           0          0
  Messages          0          0
  Branches         22         24     +2
==========================================
  Hits             53         53
- Misses          112        140    +28
  Partials          3          3

Powered by Codecov. Last updated by [8ec6f47...17f2cb1][cc-compare]
[cc-base-branch]: https://codecov.io/gh/SxMShaDoW/embeddable-badges-plugin/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/SxMShaDoW/embeddable-badges-plugin/compare/8ec6f47dd49ea96a8f1dada1e01b7858ad508e21...17f2cb1119ef24829915c17191c16c1ea2cb4f59
[cc-pull]: https://codecov.io/gh/SxMShaDoW/embeddable-badges-plugin/pull/38?src=pr

Resolves #34


Update world scripting

https://github.com/FamilySearch/pewpew/pull/166
updated root lock file and fixed clippy warnings

Acceptance:

Coverage:

All of the checks passed remotely but locally I am still seeing some issues ( may be Mac related )


Add/3rd party libs paths to config

Allows to specify the path of third party libs in the config file. Handy if you need to install such tools in a folder that is not included in PATH.
As per https://discourse.gohugo.io/t/specify-the-path-of-pygmentize/8242/6
This is just for pygmentize.
I'd like to know if you like this approach/style before going on with rst2html and asciidoc.
Cheers.

 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.

Nothing wrong with your approach, but I don't like sending that cfg around (it may also be a performance issue; probably not in this case, but doing Viper lookups in hot paths are not performant). Since this is about consolidating I suggest you follow the example of PathSpec etc. and create a new ExtHelperSpec which can take that cfg and do all the path logic on creation time -- and will then own the HasPygments() etc. Don't hesitate to ask.

@bep I admit I am a bit lost :)
Do you have an example of the pattern you illustrated?
Even a high level overview is fine.

See helpers.PathSpec.

Updated discussion here: https://discourse.gohugo.io/t/specify-the-path-of-pygmentize/8242
I think this modification doesn't make a lot of sense now.
If you agree feel free to close this PR.


Searchable on multiple properties and where's

2 feature requests around indexing.
1) Ability to set multiple Searchable properties.
Scenario is, I have a bunch of photos, there are 3 queries I will run often.
session.Query<Photo>().Where(x => x.MemberId = 123);
session.Query<Photo>().Where(x => x.MemberId = 123 && x.IsPrivate == true);
session.Query<Photo>().Where(x => x.IsPrivate == false);

I would like to be able specify an index which spans both MemberId and IsPrivate
2) Ability to add conditions to an index
_.Schema.For<Login>()
  .Searchable(x => x.Email, where: x => x.Status == Login.Status.Active);

This would generate a table with Email indexed with a where clause to pre-filter where the status is Active.
Like:
CREATE INDEX mt_doc_login_idx_email ON public.mt_doc_login (email) WHERE(data->>'Status') == 'Active';

@phillip-haydon Man, I'm sorry, but I'm declaring this one a little too much work prior to 1.0. Maybe in the next round. For now, I think we just need to say do this kind of index by hand.

No worries. I want to fix the date issue before I even look into this one.

Closing as conditional indexes is already implemented. Raising new ticket for multi-property indexing.


object-fit on IE breaks saving images

Hey there :)
On images with the object-fit applied to them on IE browsers cannot be saved to desktop anymore (it attempts to save an untitled.png instead)

Yeah that's the downside of using OFI. If it's required then you'll have to use a different solution.

No worries :) Thank you for responding.


fix(clerk-js): Fix border issue for invitation previews in OrganizationList

Description
Before:

After:

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!preview

Hey @anagstef, your preview is currently deploying and should be available shortly.

Status
Preview
Updated (UTC)

:cook: Deploying...
Inspect
Feb 13, 2024 09:56 AM


Add "Test if all/any items match predicate in parallel" example

Showcase a any and all with rayon crate.
Please discuss the idea here before starting implementation.
cc https://github.com/rust-lang-nursery/rust-cookbook/issues/272

I'm interested in working on this. Should this be one or two examples? Do we want something more complicated than "check whether all the elements in this array are even"? These functions are so basic that it feels like adding additional complexity might just obscure their use without adding value.

I would go with the simplest approach possible (also mentioning the sequential versions from std in the description). We can showcase both all and any in the same snippet.


install: Fix templates against xt_socket changes

Recent changes neglected to run make -C install/kubernetes so some files were out of date.
These changes are being backported, so mark this PR for backport.

This change is

test-me-please

Coverage increased (+0.02%) to 45.434% when pulling ae9cc00136a04135e9e414ac6d27ba0b48289397 on joestringer:submit/fix-helm-xt-socket-fallback into 7a24487c0613e5e356105c3e5fa5c9bb21596f18 on cilium:master.

I was under the belief that because true is the default value, the change was omitted on purpose. No harm in backporting this though.

Superseded by #10347.


Add demo

This repo is for APP to read out sensors' data on Cannon board.

No pass.Do not have a unified directory structure.

it is a very good reason to close the PR.


Channel comments

❓ How can I comment a channel post?
Example:
         `public function execute(): ServerResponse {
            $userTL = $this->getMessage()->getFrom();
            $data['chat_id'] = '-10********';
            $data['parse_mode'] = 'markdown';
            $data['caption'] = 'markdown';
            $data['text'] = 'Example *channel post*';

            $post= Request::sendMessage($data);

             // I want to comment this post
            // and save the discussion link
            $postDirectLink = 't.me/'.$data['chat_id'].'/'.$post->result->message_id  ;

        return $this->replyToChat(
                'Ciao!' . PHP_EOL . PHP_EOL .
                $postDirectLink . PHP_EOL . PHP_EOL .
                print_r($post, true)
            );
        }`

In this command I want to post a message in my channel and then tag some user in the 1st comment of the channel discussion. How can I achieve this?

Hi @Shyghar
Unfortunately I don't understand what you'd like to achieve.
Could you please explain in a bit more detail?
Thanks!

As far as i could read in the Bot API Docs https://core.telegram.org/bots/api there is no way for the Bot to make Comments in a channel, exept it may be handled as an reply

Ok i figured out how you could do it. You need a Group linked to the Channel. If you Post a message in the Channel it will appear as a Message in the Group. If you then reply on the Message of the Group, it will appear as a Comment in the Channel Post

@Shyghar Have you managed to figure out what you were trying to achieve?
Look at the comments that @Hitmare posted, that should help!

Closed due to lack of additional feedback.


State of LDC 3

Please report bugs in this repo or tell them in a commend on this issue if it is connected to one of the listed points. Thanks!

MUST FIX

[x] Reimplement using vue.js
[ ] Results displayed too early (caused 0 %)
[ ] Share with link: Share Important-Flag
[x] OGP + Twitter Cards
[ ] Matrix update:

[ ] Answers
[ ] Distributions
[ ] Decision multi/ single answer per question

[ ] Minimize JavaScript & CSS
[ ] Fix typos
[ ] deploy to distrochooser.de
[ ] redirect beta.distrochooser.de to distrochooser.de

OPTIONAL

[ ] CDN <> Local stored 3rd party switch
[ ] New Distributions (suggest in comments)
[ ] Allow comment display in "other user results"
[ ] Image question pictures
[ ] Blacklist some referrers
[ ] subdomains

[ ] htaccess (anybody know to redirect /?l=x to y.distrochooser.de?)
[ ] de.distrochooser.de (distrochooser.de)
[ ] en.distrochooser.de

[ ] Update distrochooser-translations

[ ] Further translations (en, fr, ru) -> help needed

[ ] Performance Updates

What about using vue-router to present the results in a separate page? The modal overlay isn't very mobile friendly and could look strange to persons who click on an results link to display them.

The only modal is currently the 'about'-Button..

@squarerootfury The predecessor implementation was different. That's why I was mention it.


Changes to the setup

During setup do the following

Gather config file details and generate a config file (Logins, passwords, checks ect)
Create tasks or ask for task ids to allow user to specify task ids manually
Initialize the cache

2 and 3 are done.
1 is easy enough with the example config i don't think it's necessary.


Suggestion: Week mode starting on Today.

Please add ability to start the week mode week starting "Today" --
I figure this could be triggered easiest by setting firstDayOfWeek to -1 and testing for it.

??? mode: day is for that?

Okay, I see that now.... "Day" to me is a single day.
Thanks!


[FLINK-5799] Let RpcService.scheduleRunnable return a ScheduledFuture

This PR is based #3310.
The returned ScheduledFuture instance allows to cancel a scheduled runnable and obtain
the delay until the runnable will be executed. Furthermore, it allows to wait on the
completion of the runnable.

Cool change, good to merge from my side.
As an optional thought: We could probably expose a lightweight Scheduler interface rather than the ScheduledExecutorService, given that quite some methods are not available. That Scheduler interface might be easier to wrap around akka's scheduler as well.

Thanks for the review @StephanEwen.
You mean something like the Executor interface with the additional methods for the scheduled execution. Differently said, removing the Service methods like shutdown, shutdownNow and awaitTermination from the interface?
That is a good idea. If we should stumble across the problem that a component needs an actual ScheduledExecutorService implementation, then we can still have a wrapper which throws an UnsupportedOperationException when calling the respective methods. I will adapt the PR in that way.

I've introduce the ScheduleExecutor interface which extends the Executor interface with the schedule methods but not the service control methods of the ScheduledExecutorService interface.

Thanks for the review @StephanEwen. Merging this PR.


Multi distro travis

replaces #620
includes #621
no kinetic_dev needed

The job exceeded the maximum time limit for jobs, and has been terminated.

...oh boy :man_facepalming:

I am preparing a patch for industrial_ci that should speed up the build by filtering unrelated packges.

You can use another docker image to speed thinks up:
DOCKER_IMAGE='ros:kinetic-robot  ADDITIONAL_DEBS="build-essential python-catkin-tools python-pip"

Beware, this prevents detecting missing dependency tags.

I have just noticed the build was finished just in time.
For now you can just set NOT_TEST_INSTALL=true. This prevents rebuilding everything from scratch for install space.

rebased on top of #621

as discussed, we should not use option 2 (docker image) or option 3 (NOT_TEST_INSTALL=true)...
if @ipa-mdl's filter for industrial_ci works...fine...
still, releasing an update to indigo/kinetic will be the best solution...as it would clear the .rosinstall files.
other options, we might want to critically review dependencies and try to find stuff that is not necessarily needed...
other than that, @ipa-fmw, don't we pay for travis? shouldn't that grant us some advantages, i.e. infinite build time?

wait for my industial_ci PR which handles dependencies better
downstrip external dependencies

see https://docs.travis-ci.com/user/customizing-the-build/#Build-Timeouts for Travis build timeouts

wait for my industial_ci PR which handles dependencies better

here it is: https://github.com/ros-industrial/industrial_ci/pull/137, it would be great if you could test it

Tested @ipa-mdl's enhancement from https://github.com/ros-industrial/industrial_ci/pull/137 locally on my laptop in cob_robots (not the meta package :stuck_out_tongue:)  by running:
rosrun industrial_ci run_ci ROS_DISTRO="kinetic" ROS_REPOSITORY_PATH=http://packages.ros.org/ros/ubuntu UPSTREAM_WORKSPACE=file

(Well, I again had to add python explicitly to the call of deps.py)
ici_fold:end:setup_workspace<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
Function setup_workspace returned with code '0' after 2 min 54 sec

Upstream packages: canopen_402 canopen_chain_node canopen_master canopen_motor_node cob_android_msgs cob_android_script_server cob_base_drive_chain cob_base_velocity_smoother cob_bms_driver cob_calibration_data cob_cam3d_throttle cob_canopen_motor cob_collision_monitor cob_collision_velocity_filter cob_command_gui cob_control_mode_adapter cob_control_msgs cob_dashboard cob_default_env_config cob_description cob_docker_control cob_footprint_observer cob_frame_tracker cob_generic_can cob_hand_bridge cob_helper_tools cob_image_flip cob_light cob_linear_nav cob_mimic cob_monitoring cob_msgs cob_obstacle_distance cob_omni_drive_controller cob_perception_msgs cob_phidget_em_state cob_phidget_power_state cob_phidgets cob_reflector_referencing cob_relayboard cob_safety_controller cob_scan_unifier cob_script_server cob_sick_lms1xx cob_sick_s300 cob_sound cob_srvs cob_supported_robots cob_teleop cob_trajectory_controller cob_twist_controller cob_undercarriage_ctrl cob_utilities cob_voltage_control libntcan libpcan libphidgets raw_description schunk_description schunk_libm5api schunk_powercube_chain sick_visionary_t_driver socketcan_interface ur_description ur_driver ur_msgs usb_cam

Target packages: cob_bringup cob_default_robot_behavior cob_default_robot_config cob_hardware_config cob_moveit_config cob_robots
Target tests: cob_bringup cob_default_robot_behavior cob_default_robot_config cob_hardware_config cob_moveit_config cob_robots

Downstream packages:
Downstream tests:

Ignored packages: can_msgs cob_3d_mapping_msgs cob_android cob_android_resource_server cob_android_settings cob_camera_sensors cob_cartesian_controller cob_command_tools cob_common cob_control cob_driver cob_elmo_homing cob_environments cob_extern cob_grasp_generation cob_hand cob_head_axis cob_interactive_teleop cob_lookat_action cob_manipulation cob_map_accessibility_analysis cob_mapping_slam cob_model_identifier cob_moveit_bringup cob_moveit_interface cob_navigation cob_navigation_config cob_navigation_global cob_navigation_local cob_navigation_slam cob_object_detection_msgs cob_object_detection_visualizer cob_obstacle_distance_moveit cob_perception_common cob_pick_place_action cob_substitute cob_undercarriage_ctrl_node cob_vision_utils libconcorde_tsp_solver libdlib libqsopt opengm ros_canopen schunk_modular_robotics schunk_sdh schunk_sdhx schunk_simulated_tactile_sensors service_tools socketcan_bridge universal_robot ur10_moveit_config ur3_moveit_config ur5_moveit_config ur_bringup ur_gazebo ur_kinematics

The current implementation assumes that we link all test dependencies.
However, cob_bringup does not reference it's dependencies for the roslaunch checks.

restarting travis after https://github.com/ipa320/cob_control/pull/147 was merged...

missed .travis.rosinstall file

.travis.rosinstall (for indigo) is already availablie in indigo_dev (nothings needs to be changed)

cob_robots should be always tested with the current indigo_dev branches, at least for cob_calibration_data and cob_supported_robots

this is what @ipa-mdl commented this morning, i.e. it's a "downstream" test...
I will create a new issue for it https://github.com/ipa320/care-o-bot/issues/39

Travis for indigo succeeded
Travis for kinetic failed due to timeout...will merge anyway...as this will be fixed after first cob_* packages have been released for kinetic and thus, we can remove some entries from the .rosinstall.kinetic file


moveXY

Peasant Protection, in the Blackwoods Forest:
I'm having a little problem with getting the mission complete.
I'have seen post on how to do it but all involves the moveXY code...
And i'm unable to use that command, It says that, that book is restricted
Can anybody shed some light on this problem? Please...
Thanks

the moveXY code is not given by a book, it is given by shoes.  By peasant protection, it is almost essential to have moveXY.  Thus, you should go buy the book.
If you don't have enough gems, you can go get more by simulating a few games.


Feature Request : Support a Callback for SLA Miss Reporting

Currently, my company uses VictorOps to dispatch alerts to the on-call engineer(s). VictorOps is like PagerDuty. Airflow supports email notifications for various event types : email on error, email on retry, email on SLA miss, etc... Unfortunately, there is only one place in a DAG to specify an Email DL for such notifications. It is possible to have Airflow support a separate DL for each cause of email notification, but this is will quickly add more configuration than is necessary. At my company, we don't want to page someone because of a task failure or a task retry, since failures are typically transient and resolved on retries. If our DAGs do not meet SLAs, then we need to page the on-call. In order to do this, we would like a python callback that we can leverage to call the VictorOps API and possibly also the Slack API (to post the issue to a Slack channel).
Here's an example DAG that specifies email_on_failure and email_on_retry as well as specifies an SLA for a task in the DAG
default_args = {
    'owner': 'sanand',
    'depends_on_past': False,
    'pool': 'ep_data_pipeline',
    'start_date': START_DATE,
    'email': [import_ep_pipeline_alert_email_dl],
    'email_on_failure': import_airflow_enable_notifications,
    'email_on_retry': import_airflow_enable_notifications,
    'retries': 1,
    'retry_delay': timedelta(seconds=120),
    'priority_weight': import_airflow_priority_weight
}
dag = DAG(DAG_NAME, schedule_interval='@hourly', default_args=default_args)

send_email_notification_flow_successful = PythonOperator(
    task_id='send_email_notification_flow_successful',
    execution_timeout=timedelta(minutes=15),
    pool='ep_data_pipeline_metrics_gathering',
    provide_context=True,
    sla=timedelta(hours=2),
    python_callable=send_email_notification_flow_successful,
    dag=dag)
send_email_notification_flow_successful.set_upstream(aggregate_db_message_job)

I'm proposing the following PR : https://github.com/airbnb/airflow/pull/869

+1 this would be extremely useful.

@mistercrunch : ^

This was just merged into Master. Closing.

I'm providing an example of how we use this callback to send VictorOps/PagerDuty notifications whenever our DAG misses its hourly latency SLA -- i.e. it takes longer than an hour to complete and hence runs into the next hour!
default_args = {
    'owner': 'sanand',
    'depends_on_past': False,
    'pool': 'ep_data_pipeline',
    'start_date': START_DATE,
    'email': [import_ep_pipeline_alert_email_dl],
    'email_on_failure': import_airflow_enable_notifications,
    'email_on_retry': import_airflow_enable_notifications,
    'retries': 1,
    'retry_delay': timedelta(seconds=120),
    'priority_weight': import_airflow_priority_weight
}
dag = DAG(DAG_NAME, schedule_interval='@hourly', default_args=default_args, sla_miss_callback=sla_alert_func)

# FUNCTIONS
def sla_alert_func(dag, task_list, blocking_task_list, slas, blocking_tis):
    logger.info('Executing SLA miss callback')

    message_type = 'info'
    if ENV == 'EP_PROD':
        message_type = 'critical'

    # Execute VictorOps alerting
    message_name = "{env} - {dag_name} SLA Miss for task `{task_list}`".format(**{
        'dag_name': dag.dag_id,
        'task_list': task_list,
        'env': ENV,
    })

    userdata ={"message_type": message_type,
               "state_message": message_name,
               "monitoring_tool": "airflow",
               "entity_id":"airflow/telemetry/SLA_miss"}
    requests.post('https://alert.victorops.com/integrations/generic/0000000/alert/{}/ep'.format(victorops_key), data=json.dumps(userdata))

    # Execute Slack alerting
    report_issues_on_slack_util(message_name)


[PHPStanRules] Add negated negated method warning

Ref https://github.com/rectorphp/the-power-of-automated-refactoring/pull/36#discussion_r553223758
// here is double negation
-if (! $this->isAllowed($value)) {
+if ($this->isSkipped($value)) {
  // ...
}

-private function isAllowed($value)
+private function isSkipped($value)
{
-    if (some_condition) {
-       return false;
-	}
-
-    return true;
+    return some_condition;
}

cc @matthiasnoback

the meaning changes, so there has to be some kind of OppositeInflector - ref thesaurus.com/browse/include?s=t

Close as not priority ATM


the translation was "the UI at every moment" but it actually meant "at a particular moment" 思考 UI 在某一時間應該如何呈現

conflict


Reduce duplication of generator params

The generator params are repeated many places: the magma generator fn, the fn model generator fn, and multiple times in the wrapper file. Let's try to consolidate these, without changing the interfaces.

Addressed by #54


Tip: A Quick Reference Guide

Thank you for this excellent plugin. It's great that it doesn't need to make any network connections to work.
I needed a quick reference so I generated one with some jq filtering.
Here's the script and the output if anyone is interested
Code
(echo '```' > quickref.md &&  curl -s https://raw.githubusercontent.com/DailyBotHQ/universal-emoji-parser/main/src/lib/emoji-lib.json | jq 'to_entries | .[] | "\(.key) -> \(.value.keywords  | join(", "))"' -r >> quickref.md; echo '```' >> quickref.md)

Example Output
😀 -> grinning_face, face, :D, grin
😃 -> grinning_face_with_big_eyes, happy, haha, :)
😄 -> grinning_face_with_smiling_eyes, joy, funny
😁 -> beaming_face_with_smiling_eyes, kawaii
😆 -> grinning_squinting_face, lol, satisfied, glad, XD
😅 -> grinning_face_with_sweat, hot, laugh, relief
🤣 -> rolling_on_the_floor_laughing, rolling, floor, laughing, rofl

Gist
https://gist.github.com/chriscz/6ccd217796b0e088f9d63bdad60df83f
Now move it into your LogSeq directory or paste it into a page
PS: I tried importing it into LogSeq, without using the code backticks and it bombed out!

I added it to the README


[HOLD] Adds ability to request transfers.

closes #2936
Why was this change made?
Closing the loop?
How was this change tested?
Unit
Which documentation and/or configurations were updated?


jfrog-cli: assuming go.mod in non go mod package

we are facing issues with resolving private artifactory golang mdoules:
--Azure Devops ArtifactoryGo@1 task is used which just wraps jfrog-cli
--pass an artifactory service connection which contains the username+token
The issue is jfrog is assuming the external module is go mod compliant but in some cases the modules is pre mod
and thus jfrog fails with 404 error:
go get<EMAIL_ADDRESS>reading https://<EMAIL_ADDRESS>404 Not Found
[Error] v46.0.0+incompatible:github.com/Azure/azure-sdk-for-go
this is speciifcally for the azure go sdk:
https://github.com/Azure/azure-sdk-for-go
To Reproduce
azdo:

task: ArtifactoryGo@1
inputs:
artifactoryService: 'ARTIFACTORY SERVICE CONNECTION'
resolutionRepo: repo-name/
command: get
goArguments: -d<EMAIL_ADDRESS>env:
GONOSUMDB: '*'

this under the hood does this:
jfrog rt go get -d<EMAIL_ADDRESS>Expected behavior
jfrog should not assume all modules are go mod compliant and in this case the package should be downloaded
Versions

JFrog CLI version: 1.47.1
JFrog CLI operating system: ubuntu 18.04LTS (Azure)
Artifactory Version: 7.19.9 (SaaS)

incorrect filled bug and have fixed the issue, a different bug has been found but not in jfrog-cli but artifactory


[AutoRelease] t2-edgezones-2024-05-15-88038(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5073
Live test success

issue link:https://github.com/Azure/sdk-release-request/issues/5073

/azp run prepare-pipelines

/check-enforcer override


[Bug]: E2E Treats All Emitted Events as Anonymous (Ignores Event Names)

What happened?
The problem was first noticed here:
https://github.com/OpenZeppelin/rust-contracts-stylus/pull/352#discussion_r1834604590
Explanation: we are trying to assert that the correct event was emitted when Ownable2Step::transfer_ownership  was called, which should be OwnershipTransferStarted, but the wrong event was asserted to had been emitted (OwnerhipTransferred). This assertion should have failed the test run, but it somehow passed!
This is when we realized there might be a problem with our e2e tool.
After further investigation, the reason for this became evident - currently used version of alloy-sol-types (0.7.6) does not check event signature when decoding logs, in effect treating all events as anonymous.
Pasting below the main part of the investigation report:

I think I have found the reason for Ownable2Step e2e bug.
Long story short, I managed to confirm that e2e treats all events with the same parameter signature as interchangeable.
I confirmed this by swapping places for OwnershipTransferred and OwnershipTransferStarted in various event assertions, and even creating an additional throwaway event to assert for instead, modifying their signatures on each test run and verifying that the tests pass only if the expected event parameter signature matches the emitted event's parameter signature, despite the name difference.
After digging a bit deeper into e2e, I soon realized the problem lay with alloy-sol-types - the version we're using (0.7.6) does not check event signatures when decoding logs, it only checks indexed parameters (topics[1..]) and data, and that the problem was fixed with version 0.8.4.
Unfortunately, we're stuck with the older version stylus sdk uses...

In short, to solve the issue we will have to wait for Stylus SDK to start supporting alloy ^0.8.4 for this to be addressed.
platform

[X] linux
[ ] windows
[ ] macos

Expected behavior
Event signatures should be taken into account when asserting that certain events were emitted.
Contribution Guidelines

[X] I agree to follow this project's Contribution Guidelines

We're tracking new releases of https://github.com/OffchainLabs/stylus-sdk-rs that we depend on to address this


REQUEST: Support for Custom System Builder (V11)

REQUEST: Support in Custom System Builder?
There's a new rpg that uses heart containers that this would be GREAT for, but it doesn't yet have its own home system.

What is the property path to HP (or its equivalent) in Custom System Builder, and what other related properties are there? Such as temporary hit points.


Add Leif Denby person info

Excited to join this community and hopefully contribute something useful!

Welcome @leifdenby!
Would you mind also adding your info here: https://github.com/pangeo-data/pangeo/blob/master/docs/data/people.yml
We're close to transitioning to this new website and at the moment, we have some duplication going on.

All done, thanks!

@leifdenby - I'll merge this now but I don't see your PR on the https://github.com/pangeo-data/pangeo repo...


Drag-n-Drop feature

Description
Hey! I'm experiencing a very annoying bag on Windows 11 where I can't just drag-n-drop something out of Files App to the other app while using Grid view. It's either not letting me do that (not showing the small hovering preview of files that I drag) and freeze for a couple of seconds OR completely freezing after showing those previews and I have to fully restart it OR it works fine rarely
Steps To Reproduce
Use "Grid" view -> hold LMB on a file -> drag it outside of the app window
Requirements
Fix crash that occurs when I try to drag-n-drop file from Files App to any other app
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3296
Log File
debug.log

Any other app. It’s not related. It won’t even let me drag the file out of
Files App window. For example: acted Effects, Figma, Desktop, etc
*George PalkinUX/UI DesignerPortfolio: *click
https://behance.net/itsgeorge
On Fri, 12 Apr 2024 at 3:58 PM, Josh65-2201 @.***>
wrote:

Thanks for the report, What is the other app you are dragging too?
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/15163#issuecomment-2051713614,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIDOVG5YE3KOFWZJLPTVIGDY47K6HAVCNFSM6AAAAABGD7TLP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJRG4YTGNRRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm unable to reproduce this and your log doesn't show any crash error. Do you have any grouping options enabled?

It’s not crashing. It just freeze.
Not sure about what those grouping options are tbh. If u can explain me,
then I’ll be able to answer that. But I’ve noticed that it happens in grid
layout only. Haven’t noticed in any other layouts
*George PalkinUX/UI DesignerPortfolio: *click
https://behance.net/itsgeorge
On Fri, 12 Apr 2024 at 4:08 PM, Josh65-2201 @.***>
wrote:

I'm unable to reproduce this and your log doesn't show any crash error. Do
you have any grouping options enabled?
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/15163#issuecomment-2051731020,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIDOVG46B72IOGZO6EFL6W3Y47MGPAVCNFSM6AAAAABGD7TLP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJRG4ZTCMBSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

They are in this menu

Here
[image: image.png]
On Fri, Apr 12, 2024 at 4:14 PM Josh65-2201 @.***>
wrote:

They are in this menu
image.png (view on web)
https://github.com/files-community/Files/assets/26977075/7054f885-c8a6-405a-b291-cc6a190e4ebe
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/15163#issuecomment-2051740414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIDOVG6GZLOZRP36WC6EVYTY47M27AVCNFSM6AAAAABGD7TLP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJRG42DANBRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
*George PalkinUX/UI DesignerPortfolio: *click
https://behance.net/itsgeorge

Email attachments don't work. You will have to click the View on GitHub link at the bottom of the email > Copy and paste or click the add attachment button to select the file > Wait until you don't see uploading in the text > Click the green comment button.

Now, I've noticed a thing where you simply can't drag a file outside of Files App window in "Details" layout. It just locks on the edge and won't let me move it out of the boundaries.
This whole bug is the only thing that annoys me, and the reason why I consider switching back to regular explorer. And I really don't want to switch. But I have to drag-n-drop files very often...

Hey! Any updates on the topic?

I still can't reproduce this. Can you see if a issue in preview https://files.community/download

I am able to reproduce as well nearly every time I try.
In my instance, I am dragging an Excel file from the Downloads folder onto Firefox in the "Drag here to upload" area within Azure DevOps. Firefox is on my main monitor, Files is on my secondary. As soon as I start dragging the file, a "🚫" looking icon was displayed on my mouse, and when I moved it across monitors the file name preview that shows to indicate you're dragging freezes for a second on the other monitor before stuttering over to the new one and unfreezing. If I drag it back to the other monitor, it does the same thing. Attempting to let go of the file (Anywhere, doesn't matter if it's on top of an app, on your desktop, within Files, etc.) causes Files to freeze entirely, and it will stay frozen until either Windows asks if you want to close it or you force the process to end in Task Manager. Event Viewer only shows "MoAppHang" events with the very unhelpful description of "Files stopped interacting with Windows and was closed". Logs do not show any crashes either.

However, the most recent time I tried I attempted to record it with OBS. This time, of course, it was fine. If I can reproduce it myself I will also post a video here.


Update NLog.Extensions.AzureBlobStorage.csproj

The existing version of this project refers (indirectly) to System.Text.Encodings.Web version 4.7.2, which contains critical security issues. Including this direct reference to the newest version will ensure that the newest and more secure version i used instead, thus removing the security issue.

Thank you for the pull-request. Can you provide the details of the security-tool that complains?
Have tried running this command:
dotnet list ./src package --vulnerable --include-transitive

And it only shows this alert:
System.Text.RegularExpressions      4.3.0      Moderate   https://github.com/advisories/GHSA-cmhx-cq75-c4mj

Hi! Thanks for the quick reply, and sorry for not including more details in the original PR.
For the context I was working in, the issue was reported by Snyk (https://snyk.io/), which states the following as the source of the problem:

Introduced through: project@* ›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>
There is some information about it from Snyk, and also a warning in the listing under the NuGet Gallery:

https://security.snyk.io/vuln/SNYK-DOTNET-SYSTEMTEXTENCODINGSWEB-1253267
https://www.nuget.org/packages/System.Text.Encodings.Web/4.7.1

Note that I got the version number wrong originally: It is 4.7.1 - 4.7.2 is a newer version, in which the issue is fixed.
That also means that you are correct in not needing to change the major version: It should be sufficient to upgrade to 4.7.2.

Would it make sense to convince Microsoft to fix the issue closer to the source:
https://github.com/Azure/azure-sdk-for-net/
Creating pull-request for the above repository to fix the nuget-package Azure.Storage.Blobs ?

Btw. think there is a bug in the Snyk-tooling since you are right about this dependency-chain:

<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>
But there is also this dependency-chain:

<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>><EMAIL_ADDRESS>
And dotnet will then resolve<EMAIL_ADDRESS>

You may be right. I'm not sure about the Snyk question, or whether Snyk really can be trusted here, but in any case, the issue should probably be solved somewhere else. Thanks for your responses, I hope you didn't spend too much time on this.


HDP 3.1 NoSuchMethodError with HDFS Jar Storage

Startup on an HDP 3.1.0 and HDF 3.4.0 Cluster fails with following error. Looks to be a dependency issues.
Exception in thread "main" java.lang.NoSuchMethodError: org.apache.hadoop.io.retry.RetryUtils.getDefaultRetryPolicy(Lorg/apache/hadoop/conf/Configuration;Ljava/lang/String;ZLjava/lang/String;Ljava/lang/String;Ljava/lang/Class;)Lorg/apache/hadoop/io/retry/RetryPolicy;         at org.apache.hadoop.hdfs.NameNodeProxies.createNNProxyWithClientProtocol(NameNodeProxies.java:318)         at org.apache.hadoop.hdfs.NameNodeProxies.createNonHAProxy(NameNodeProxies.java:235)         at org.apache.hadoop.hdfs.NameNodeProxies.createProxy(NameNodeProxies.java:139)         at org.apache.hadoop.hdfs.DFSClient.<init>(DFSClient.java:510)         at org.apache.hadoop.hdfs.DFSClient.<init>(DFSClient.java:453)         at org.apache.hadoop.hdfs.DistributedFileSystem.initialize(DistributedFileSystem.java:136)         at org.apache.hadoop.fs.FileSystem.createFileSystem(FileSystem.java:3303)         at org.apache.hadoop.fs.FileSystem.access$200(FileSystem.java:124)         at org.apache.hadoop.fs.FileSystem$Cache.getInternal(FileSystem.java:3352)         at org.apache.hadoop.fs.FileSystem$Cache.get(FileSystem.java:3320)         at org.apache.hadoop.fs.FileSystem.get(FileSystem.java:479)         at com.hortonworks.registries.common.util.HdfsFileStorage.init(HdfsFileStorage.java:63)         at com.hortonworks.streamline.webservice.StreamlineApplication.getJarStorage(StreamlineApplication.java:222)         at com.hortonworks.streamline.webservice.StreamlineApplication.registerResources(StreamlineApplication.java:245)         at com.hortonworks.streamline.webservice.StreamlineApplication.run(StreamlineApplication.java:102)         at com.hortonworks.streamline.webservice.StreamlineApplication.run(StreamlineApplication.java:76)         at io.dropwizard.cli.EnvironmentCommand.run(EnvironmentCommand.java:43)         at io.dropwizard.cli.ConfiguredCommand.run(ConfiguredCommand.java:85)         at io.dropwizard.cli.Cli.run(Cli.java:75)         at io.dropwizard.Application.run(Application.java:79)         at com.hortonworks.streamline.webservice.StreamlineApplication.main(StreamlineApplication.java:80)

which database are you using for ambari ?

PostgreSQL 9.6
Sent from my iPhone
On Apr 3, 2019, at 1:29 AM, aditya777hp <notifications<EMAIL_ADDRESS>wrote:
which database are you using for ambari ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hortonworks/streamline/issues/1345#issuecomment-479357648, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJUPRZFlKKw1krBgsIftkTdlfmoRnEL7ks5vdEo8gaJpZM4cSXZ9.

Something similar happened with me as well, try changing the driver.Maybe using an older version of the driver might help !!!

I'm confused at what Postgres has to do with this. It's saying that the method org.apache.hadoop.io.retry.RetryUtils.getDefaultRetryPolicy(Lorg/apache/hadoop/conf/Configuration;Ljava/lang/String;ZLjava/lang/String;Ljava/lang/String;Ljava/lang/Class;) doesn't exist. I'm assuming It's because sam and HDP were built different versions of the Hadoop Libraries but I don't know. Seems like an odd problem to have.

Ok, It might be so...


doRequestWithRetries doesn't work correctly when req.Body is not nil

I still think not retrying PUT requests is the right thing to do, but this was responsible for the strange errors seen while retrying the PUT update of a monitor seen in #91.
HttpClient.Do was returning errors on retries when the request contained a Body. When the http request contains a body it seems it cannot be retried because the req.Body is closed after the first try. After the first try the request always errors out because of the empty body not matching the ContentLength header. The issue seems to be the same as this:
https://github.com/mitchellh/goamz/issues/63
#91 may lessen the impact of #76 . As far as I know the only requests that would be sent to datadog which would contain a non nil body would be PUT or POST, which are now not retried. #91 definitely doesn't solve #76 but makes it less likely to be exposed.
This fix is probably not necessary now that PUT isn't retried. I don't think any DELETE requests contain bodies, but it could start a conversation.
I'm not sure what the best approach to sanitizing the error message containing the full request URL would be? I'm going to check with datadog to see if they have an undocumented way to pass credentials via a header.

Thanks!
Very much appreciated if you can check with Datadog and/or convince to stop requiring to put credentials in query params.

When reviewing I #94 for which I've raised #95. Apologies if this causes a merge conflict for you. The PR is against v2 but it needs to land in master too.

@ojongerius I rebased on top of v2. The changes in #95 are great! Could we backport those to v1 to get them into the terraform provider, or are there plans to update that to use v2 soon? I ran into a case where that will help. If you GET a deleted monitor, that will respond legitimately with a 404, which will be retried until the timeout before the monitor will be re-created by terraform.
I removed the comments about api credential leakage. It's probably easier to deal with that separately. I did contact datadog. They've had requests for authentication via HTTP headers in the past but it doesn't seem like it's an active project at this point. I let them know we'd really like to see that happen and will keep after it.
Thanks for this library! This in conjunction with the terraform datadog provider has provided huge benefits for us.

@bshelton229, appreciated, it's great to hear people are enjoying the work.  Let's discuss the API credential leakage in #76, thanks for the investigation!
The v2 branch has made it into Terraform via https://github.com/hashicorp/terraform/pull/12098.


Hunt returns 404 - Old API

Describe the bug
When logged into Viewer with Google SSO, if I perform a shift refresh in Chrome I get a 404 and an empty page with the string Old API
However, if I go back to /sessions and then just click the Hunt tab it works fine.
Elasticsearch version:
8.9.1
Arkime/Moloch version:
4f6fb35f8443d561988a78c1edd9459e141d5b78
OS name and version:
Ubuntu 22.04.3 LTS
How was Arkime/Moloch built/installed: (rpm, deb, easybutton, ...)
easybutton
Provide logs, stack traces and steps to reproduce:
Mon, 04 Dec 2023 17:07:09 GMT<PHONE_NUMBER>61898851742 GET /hunt 404 - bytes 5.447 ms

fixed in #2533


Resolve release pipeline issues (1.20.5 post-mortem)

description
we have been hit by an issue on our various release branches of running out of disk space when building the docker images - this then prevents the release from publishing
what failed ?
we were again hit by docker disk space issues on the release branch
could it have been prevented ?
yes, but the issue was complicated by the fact that the PR to the release branch passed, but when the PR landed it failed
if we hadn't published the branch when the PR landed and failed we could have resolved the issue
i think the best way to prevent this happening in the future is to automate the tagging/release after a succesful CI run - that way the tagging happens iff the release branch can pass
part of the issue seems to be that what is tested in a PR to what is not exactly the same as what is tested in actual releases - but having said that this did fail postsubmit before/unrelated to the failing tag build
do we need to worry about disk space as these seem to be an endemic release issue ?
afaict, not any longer in terms of current main - the binaries/images/build are all much smaller so the problem seems to be resolved
that said our current azp ci image uses up most of its space with dotnet and other stuff we are unlikely to ever use - and even if we did we would most likely do so with bazel. It would be great if we could use an underlying image that just had a bit more disk space and didnt have lots of stuff we wont use.
we will need to resolve the diskspace issue for this release (or 1.20.6 at least) and as a precaution ill add the aggressive rm -rf patch when i add the patch stack etc for 1.19.x
related links

release PR
failed CI on patch landing
failed CI on release

cc @mattklein123 @lizan

I agree it would be really nice to fully automate the tagging/release so that it only gets tagged if it actually passes. I'm not sure how to do this though. I think we would probably have to have a manually triggered GH action that somehow does this?

i think we could have azp CI trigger a branch tag when it gets far enough through - not sure exactly - but it seems likely - @lizan ?


weekly_bashbeautify.yaml: Only run on canonical

Another example for https://github.com/alexbelgium/hassio-addons/issues/1482

Thanks!


Find file is way slower in 1.7 then it was in 1.6

After upgrade to 1.7 Ctrl + P got way slower on large project. It now takes 2-3 seconds (just to open dialog) on large project where with 1.6 was instant.
Other people complained on Twitter too https://twitter.com/tommy/status/720354578680537089, but I find this issue very important to I am posting it here.
If it can be debugged somehow, I will be happy to do it.

This is a duplicate of atom/fuzzy-finder#188 - feel free to subscribe there for updates.


Update Manager should offer (maybe link to) details of an update before the user chooses to apply that update

Enhancement request.
With SysAdm as is, selecting an update reveals no additional information; there's just the name/identifier.
This area of the GUI might be a good place to link to (or represent relevant content from) https://raw.githubusercontent.com/trueos/trueos-changelog/master/CHANGELOG

Additionally: if an update will completely remove any item, it will be good to make that fact unmistakably clear. (Minimise the risk of unwanted post-update surprises.)
Long-term, I guess that some improvement could be made to order of output from pc-updatemanager. In the meantime, maybe a grep for the word removed could allow the SysAdm UI to forewarn that a removal should be expected.

Housecleaning time.  Many changes have occurred so I'm closing old tickets. If the issue still remains we can reopen.