Adds backend support for connecting Airflow integration

Describe your changes and why you are making these changes

Adds the backend support for connecting an Airflow integration.
It uses the Airflow go-client to perform authentication, since this is the SDK that will be
used for the actual read operations when we need to read metadata from Airflow and load it into our Aqueduct database.

Related issue number (if any)
ENG-1276
Checklist before requesting a review

[x] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[x] I have performed a self-review of my code.
[x] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[x] If this is a new feature, I have added unit tests and integration tests.
[x] I have run the integration tests locally and they are passing.
[x] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[x] All features on the UI continue to work correctly.
[x] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

Demo: https://www.loom.com/share/aebc6367d89746e0bd592df316323ac1

@likawind Yes, that's correct


document all transactions stores

[x] add documentation for deltas and exported functions of transactions, meta-transactions and wallets-transactions stores

Comments
Issue Number
ADP-1955

bors r+


chore(main): release 0.3.0

:robot: I have created a release beep boop
0.3.0 (2024-07-17)
Bug Fixes

CLI updates for teams tier and other improvements (#286) (3d1ce0f)
Dataframe feature addition (#278) (ea439c7)
documentation update (#281) (12d81d0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Zero-True/zero-true/releases/tag/v0.3.0 :sunflower:


OS keeps rebooting on real hardware

Please help
https://user-images.githubusercontent.com/83548580/139536381-f5187e9c-6de6-4e17-bb05-c492790a79ac.mp4
Here is video of error
Also, kernel don't initialize

Maybe is my hardware problem. I am not sure

I still don't fix issue

Seems like you are having a tripple fault, it occors if even the double fault handler also throws, like if there is a page fault while printing, it will throw the page fault, which tries to print and throws a double fault which throws a triple fault which we cant handle and the CPU will just reboot

It is RAM problem but I don't know who to fix it execpt using Limine as bootloader

It is RAM problem but I don't know who to fix it execpt using Limine as bootloader

How did you know that it is a RAM problem?

I think #20 fixes it

Maby even check out #2

It That Your Selecting The Drive Name Not UEFI SHELL Whitch It The Right One

Did you Code It Right?

I Got An Imaage On It Before It Turned Off But Its Blurry Bc Of Zooming In

I get same result on real hardware and in Qemu with over of 3GB RAM

I get same result on real hardware and in Qemu with over of 3GB RAM

Memory alocation problems ig

Did you Code It Right?

@PokeyManatee4 @Andrej123456789 Turns out that the issue is caused with mouse init code. After some debugging in an VM, till 3 GM of ram, everything was fine, but when i got to 4GB, it started rebooting for no reason, so after commenting code, uncomenting, i found that issue at mouse init code. After commenting and uncommenting code, the probleam was at line outb(0x64, 0x20); after mouseWait(); and before mouseWaitInput();. I'm sure that outb is working, so something is going wrong when writing to port 0x64 when using more then 3GB of ram...

@ElectroBoy404NotFound I think this problem exist before implementing mouse

What Im Thinking Is Because The Template Is When We Had Ram Under 3 gb And Cant Handle It

@ElectroBoy404NotFound I think this problem exist before implementing mouse

But, when i remove the mouse.init();, it stopped crashing

What Im Thinking Is Because The Template Is When We Had Ram Under 3 gb And Cant Handle It

Ok

and. cant map all of the mem

Yes

Maybe Limine fix this error

and. cant map all of the mem

?

It Is A Issue Before Adding Mouse Im On EFI Mem Map And It Crashes On My
18gb Computer But not my 4gb laptop
On Fri, Mar 11, 2022 at 2:49 AM #Mouse got crash @.***>
wrote:

and. cant map all of the mem
?
—
Reply to this email directly, view it on GitHub
https://github.com/Absurdponcho/PonchoOS/issues/28#issuecomment-1064900871,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNRYPMVONMUPYGER4RQMYLU7MCLFANCNFSM5HBFAHSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yes

Even Before EFI Mem ap It Did It

so its the template

It Is A Issue Before Adding Mouse Im On EFI Mem Map And It Crashes On My 18gb Computer But not my 4gb laptop
…
On Fri, Mar 11, 2022 at 2:49 AM #Mouse got crash @.> wrote: and. cant map all of the mem ? — Reply to this email directly, view it on GitHub <#28 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/APNRYPMVONMUPYGER4RQMYLU7MCLFANCNFSM5HBFAHSQ . You are receiving this because you were mentioned.Message ID: @.>

can you please explain why it is working for me after remoing mouse.init(); ?

oh wait i get what you ment

GDT code which I use for Limine do not work for this version

Ok

Please help
Ringos.Error-1.1.mp4
Here is video of error Also, kernel don't initialize

It wasn't meant to be run on  hardware.

—
Reply to this email directly, view it on GitHub
https://github.com/Absurdponcho/PonchoOS/issues/28#issuecomment-1572344001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNRYPI7FXVBJOT6W64WFMLXJC5OBANCNFSM5HBFAHSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

he deleted account because:
a) PonchoOS is just tutorial
b) Limine is very good

c) He finally learned about google

lol

yeah ok so all of u are having ur little discussion about UEFI loaders in here but i actually REALLY want to make this work. anyone got anything?


Please add Garage Door Opener

I use another Plugin for my Garage Doors but it doesn't work very well, I need to stay at node 10.18.0 :-(
Could you please add support for Garage Door Opener?

See #76.


The stop button in the Uploader does not stop the Driffta execution

Git checked in and committed and the fix is available in latest release.


Weighted Randomness

Enhancement MoM2E;
Allow weighting randomness for Monster and Item Pool Traits.
example for a Heavy weapon to be twice as likely as a Bladed one you could do,
Item: Traits: Weapon, Pool Traits: Heavy Heavy Bladed

At this stage I am not sure that the added complexity is worthwhile. I will add it to the backlog.

I actually was trying to do this for my starting and random items and to reduce spawn location declaration.
I was attempting to make firearms possible but unlikely, and to declare one spawn point that randomly spawns creatures with easier Zombies being more frequent than say, Mi-Gos.
Looking forward to seeing this when/if it does get implemented.

For the Zombie/MiGo case you can create both as different Spawn components and have one with a condition of #rand10 == 1 for a 10% chance.


Can't access site on Safari or Chrome browser with iOS 13.5 or 13.6

This applies to both http and https. I've run gyro 1.9 and tried going to the link it lists using my iPhone 6S plus on both the Safari and the Chrome browser, and neither works. I keep getting the error that the connection timed out and the site can't be reached. I tried it when I had iOS 13.5 on my phone, then again when I updated it to 13.6. Still not working. I'm using the links provided by gyro, so I don't think a mistake in the link is the problem. Both my Google Chrome and Safari app are updated to most current versions as well. No issues with my internet either, I don't have any problems loading any other sites on my phone.

Nevermind, I figured out what was going on. For some reason my computer thought my wifi connection was a public network when it's not.


Set PrivateAssets to all for analyzer packages

Closes #916.

I think some of the analyzers come through the submodule as well although this should override them.

@shenglol do we need this change in https://github.com/Azure/bicep-types-az as well?

Yeah we do. I think OmniSHarp also pulls in some other analyzers - unless they fixed that issue on their end.

Yeah, will send a PR to update bicep-types-az.

Submitted a PR: https://github.com/Azure/bicep-types-az/pull/30.


Add the ability to turn on and off a media player and select a source.

This adds the ability to turn on and off a media player and select a source. (E.g. I am using this to remote control my NAD amplifier with an additional MQTT->RS232 tool)
It also now only shows those abilities that are actually configured as available.

Thanks for the PR!
Love this feature, and I wish my current receiver had a serial port so I could try this out 😭.
Anyways I tried out the plugin with the new changes and the existing functionality works, so closing this PR


AzureADPreview now uses different object property names

Same script but updated to new property names, and now matching security group by exact match not substring:
$GroupName = "<your group name here>"
$AllowGroupCreation = "False"

Connect-AzureAD

$settingsObjectID = (Get-AzureADDirectorySetting | Where-object -Property DisplayName -Value "Group.Unified" -EQ).Id
if(!$settingsObjectID)
{
	  $template = Get-AzureADDirectorySettingTemplate | Where-object {$_.displayname -eq "Group.Unified"}
    $settingsCopy = $template.CreateDirectorySetting()
    New-AzureADDirectorySetting -DirectorySetting $settingsCopy
    $settingsObjectID = (Get-AzureADDirectorySetting | Where-object -Property DisplayName -Value "Group.Unified" -EQ).Id
}

$settingsCopy = Get-AzureADDirectorySetting -ObjectId $settingsObjectID
$settingsCopy["EnableGroupCreation"] = $AllowGroupCreation

if($GroupName)
{
	$settingsCopy["GroupCreationAllowedGroupId"] = (Get-AzureADGroup |Where-Object -Property DisplayName -EQ $GroupName).ObjectId
}
 else {
$settingsCopy["GroupCreationAllowedGroupId"] = $GroupName
}
Set-AzureADDirectorySetting -ObjectId $settingsObjectID -DirectorySetting $settingsCopy

(Get-AzureADDirectorySetting -ObjectId $settingsObjectID).Values

@michalrok - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

https://docs.microsoft.com/en-us/microsoft-365/admin/create-groups/manage-creation-of-groups?view=o365-worldwide

Hello @michalrok Michal, Can you specify please the version of the AzureADPreview module that you are using?
I am using <IP_ADDRESS>, which appears to be the last one, and there is no ObjectId parameter in Get-AzureADDirectorySetting

@michalrok - Can you please respond to the query of get-itips?
Thank you.

@michalrok - We appreciate any feedback you can provide to make the content in the Microsoft docs better. However, we did not get any response from you so we will close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thanks!


This page only shows built in parameters, when its linked to as how to use custom parameters.

How does a user supply values to a custom parameter in a template?

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

ID: 7ffd68f8-8ba6-f868-648c-2a2b6e8899b4
Version Independent ID: 52b9100f-d71c-7e9c-303f-76d1eafcfa20
Content: Add name parameters to project and item templates - Visual Studio
Content Source: docs/ide/how-to-substitute-parameters-in-a-template.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

Custom template parameters can be added by defining CustomParameter tags in the .vsproject files.
<TemplateContent>
    <CustomParameters>
        <CustomParameter Name="$repositoryrootdir$" Value="C:\RepositoryRoot"/>
        <CustomParameter Name="$frameworkdir$" Value="C:\Program Files\Framework"/>
    </CustomParameters>
</TemplateContent>

These parameters are used in the same way the standard parameters are used. The ext_ prefix gives access to parameters defined at solution level.

However, I am closing this because the new dotnet new style templates are much friendlier to work with overall and has examples and instructions on how to customize.


Finalize method / data type names for 1.0

List of proposed changes:

[ ] IO.traverse => IO.seqEach (alternately: forEach)
[ ] IO.parTraverse => IO.parEach (alternately: forEachPar)
[ ] io.redeem => io.redeemM
[ ] io.redeemPure => io.redeem
[ ] io.asyncPure => io.asyncM
[ ] IO.sequence => io.seqAll
[ ] io.timeout => io.timeout0 (also: introduce timeout: IO[E, Option[A]])
[ ] raceBoth => raceEither (?)

Others.

- on traverse, parTraverse, sequence – IMHO it's better to retain common names for these, soft - on redeem – I personally haven't had a use for redeemPure yet, so I'd prefer to keep a shorter name for the variant I do use. + on asyncM and timeout

as @kaishh said I put traverse, parTraverse, sequence and redeem
implemented timeout
others are renamed

raceEither ⇒ race (see)
race ⇒ raceMerge
raceWith ⇒ race0
sandboxed ⇒ sandbox
sandboxWith ⇒ sandbox0
parWith ⇒ par0
seqWith ⇒ seq0
supervised ⇒ supervise
supervised ⇒ supervise0 (the more powerful version)
timed ⇒ time
timed0 ⇒ time0
uninterruptibly ⇒ atomic

In the companion:

asyncPure ⇒ asyncM
superviseWith ⇒ supervise0

I think indeed traverse and sequence are part of the functional vernacular, whereas foreach has an imperative after-taste.

my 2 bikeshedding cents:

names -- cannonical names and deprecated alisases, for example:

let forEach be the canonical name here, because it conveys better the specific meaning specific to IO
let traverse be a deprecated alias that is still easy to discover (not only in IDE) by people already familiar with the concept, but advising them to use forEach

flavors -- good as suffixes, for example:

let the sequential variant be the default, so forEach (or just each), all, ...
let the specific variants come at the end, so forEachPar (or just eachPar), allPar, ...
this is easy to discover in IDE, when exploring via pressing dot -- first you zoom on the kind of operation you want, and then you see if it also comes in a parallel variant or whatever

[ ] promise.error => promise.fail

Also I'm agreed on the desirability of aliases for sequence, traverse, etc. The aliases are for people who know FP (it's quite standard terminology) but the other ones are for the much larger non-FP market.
The "default = Seq" is interesting. We should be consistent. I also like designing for dot syntax so putting Par as a suffix always (rather than as a prefix sometimes, e.g. parAll) makes sense.

Oh, and for With, I think we should save that suffix for variants that take a "compose" function as a parameter, rather than using it for the "most powerful" variant. I admit I don't really like the foo0 convention (I'd rather foo' 😄 ), but I feel With is a poor fit for the most powerful variant, too.

@kaishh I agree it's more common to use redeem than redeemPure. Though there's an emerging pattern in ZIO to suffix M for monadic versions of functions (e.g. mapM in Schedule). I'm not sure what to do about the tension between consistency in naming convention and the higher relative importance of redeem.

@kaishh @regiskuckaertz @sideeffffect
I made some quite substantial modifications, incorporating all feedback from this thread. Summary of high-level changes:

Now par is always a suffix, rather than sometimes a suffix, and sometimes a prefix, to aid discoverability with auto-complete.
The sequential version is always the default, because parallel code has overhead and users should be explicit about that.
The 0 suffix is retained for now, for the "most powerful" version of a combinator.
The With suffix is retained in any case where a "more powerful" version accepts a function.
The "classic" names of functional programming primitives (sequence, traverse) are retained. However, aliases are introduced to make the specific meaning in the context of IO clear to those who aren't using FP libraries or don't come from an FP background.

@regiskuckaertz I didn't change uninterruptibly to atomic because I'm afraid users will think it means atomic in another sense (database, STM, etc.). But I did change it to be more consistent (uninterruptible).
I think another code audit or two are necessary to collect any outstanding deviations and incorporate them here.

Btw what do folks think of the F suffix rather than M suffix for the effectful versions? I see it's used by Cats in some places (asyncF) and has the nice benefit of starting with a sound that sounds similar to "eff" in "effect". Although M is more common in Haskell code.

Btw what do folks think of the F suffix

I like it. M and A are used in Haskell/Scalaz already to abstract over any monad/applicative. F makes more sense locally.

PureScript uses E for Eff-specific functions.

2 other bikeshedding cents:
regarding F, A, M suffixes -- shouldn't they be used for communicating if the function works with a Functor, Applicative functor or Monad respectively?
In that case, the decision should be made on case by case basis, considering each function and what functionality from the IO class it needs.
E.g. if it only uses ap or pure inside, it should use A as suffix. If it requires flatMap/bind, use the M suffix.

@sideeffffect Since ZIO is standalone, there are no dependencies on Scalaz / Cats, so no type classes, so no monad / applicative / functor.
I'd be nice to use name overloading here to bypass the issue entirely, although I think in most cases that would have no effect because after erasure, they end up the same, anyway.

@jdegoes possible typos:

[ ]  IO.collectPar: Add an alias IO.parSequence.

shouldn't it be

[ ]  IO.collectPar: Add an alias IO.sequencePar.

?
and

supervised rename to supervise
supervised rename to superviseWith

@regiskuckaertz How about Exit instead of Result? Thinking that Result is pretty generic but Exit is more specific.

@sideeffffect Fixed, thanks!

trait Fiber {
  def observe: IO[Nothing, Exit[E, A]]
  // or
  def observe: IO[Nothing, Result[E, A]]
}

The longer I look at it, the more I think Exit works.

What about:

Ref ⇒ Var
RefM ⇒ VarF
following the decision to drop the M suffix?

@regiskuckaertz 👍
If we actually do all this, we should make a ScalaFix migration to update code for the changes.

@regiskuckaertz why would you want to rename the Ref class for references to Var even though the are not variables?

@regiskuckaertz They're not variables in the PLT / math sense, but they're variables in the Scala sense (var versus val). Makes a certain amount of sense to someone working in Scala.

I'd rather we not introduce more names for the same concepts. I'm personally more in favor of keeping RefM or even RefF.

@sideeffffect I know, RefF looked odd is all. I'm not attached to it so I will go with what the consensus is 🤗
I don't think this is too far off the PLT wisdom. In languages like Scala, Java, C etc, a var x : A is just a val x : Ref[Option[A]] with syntactic sugar; see Pierce 2002.


Appstudio update test-component-pac-eplg

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-eplg-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
9 seconds
appstudio-init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
13 seconds
appstudio-configure-build

✅ Succeeded
13 seconds
sast-snyk-check

✅ Succeeded
39 seconds
build-container

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
24 seconds
sbom-json-check

✅ Succeeded
9 seconds
sanity-optional-label-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-eplg-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
13 seconds
sast-snyk-check

✅ Succeeded
13 seconds
appstudio-configure-build

✅ Succeeded
37 seconds
build-container

✅ Succeeded
21 seconds
sanity-inspect-image

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
24 seconds
sbom-json-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
12 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-eplg-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
13 seconds
appstudio-configure-build

✅ Succeeded
13 seconds
sast-snyk-check

✅ Succeeded
37 seconds
build-container

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
21 seconds
sanity-inspect-image

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
24 seconds
sbom-json-check

✅ Succeeded
12 seconds
sanity-optional-label-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
7 seconds
show-summary


Add new 'build' xtask task to build + cp executables for all arches

This is supposed to mimic the BuildAll mage target in the Go SDK
(https://github.com/grafana/grafana-plugin-sdk-go/blob/06ad448ce99f902b05c1d90be4dc0d917b05b83f/build/common.go#L181-L185)
Unfortunately cross compiling Rust is slightly harder as we need custom
linkers to be installed for each target triple. This PR includes
some updates to the .cargo/config.toml which set those linkers to
suitable values for an M1 Mac running MacOS 12.2, but I don't think they
will be portable. In the absence of anything better, this will have to
suffice!
Related: #2

Closing - we can use CI to build for other architectures.


[FR] Limit rollover to a specific heading

Amazing plugin!
Just one thing that could be really helpful: I have tasks under a heading that I have specified in the settings. I would like that only the tasks under that particular heading be rolled over to the next day and not those of the whole note.
For instance, if my template looks like this:

Big header
Something Something

[ ] Task 1
[ ] Task 2

ToDo

[ ] Task 3
[ ] Task 4

Something Else

[ ] Task 5
Blabla

Big header 2

[ ] Task 6
[ ] Task 7

Would it be possible to only rollover what is under the ToDo heading (e.g. Tasks 3 & 4) and nothing else?
Cheers
Probably related to #54

Yes, its very nice to have this plugin come back to life. Thank you very much, @lumoe !
I would also love to have this
possibility to specify a headline where todos originate from feature
as I use checkboxes for checking my daily table of contents on top of every daily note.

This should already be possible with the release-2.0 branch. Can you to test if the approach works for you?

Ok now I finally figured it out, how to use the 2.0 version :)
Would be nice if you change the version number in the package.json already?
This way the version beeing used would be shown in the Obsidian Settings Panel?
So I also get this error:
2023-02-20 is not a daily note.
files like this: vault -> !chronic/daily/2023-02-20.md
Furthermore:
I like the way I can choose one / severall Headings for where to take the TODOs from.
-> But I am missing the possibility to Rollover these TODOs to a specific Heading only,
like it used to be in the old version.
Thank you in advance, Tim !

So I also get this error:
2023-02-20 is not a daily note.
files like this: vault -> !chronic/daily/2023-02-20.md

can you provide a screenshot of the settings for your daily-notes plugin and rollover-daily-todos, as well as the full path of 2023-02-20.md inside your vault? This would help me with debugging :)
As for the second part: The old behavior of collecting all todos under one specific heading isn't planned to further exist. It conflicts with the multi-heading approach. Maybe it will become a configuration-option if enough users request this feature to further exist.

full path is this:
vault folder name: _interconnect -> _interconnect/!chronic/daily/2023-02-20.md

I would like them rollovered checkboxes to come under
rest erledigt ? (rest done?)
I really prefer this way, instead of polluting my new todos :)
Nice would be to have the possibility to have them wrapped inside a callout as well...
Now I see this callout stuff might have killed your approach...

No stress, brother :D

Your issue lead to me finding not one, but two bugs! :D Can you check if it works for you now?

So, the rollover is working !
I get still get this error...

It works but:
Everything (not the specified heading only) gets rollovered
to the end of the destination note.
Extra information for you:
When I look into the Plugin settings, i see that the source headings are not up to date.
I changed some in the meantime (not the source heading) but it won't reflect in the settings at any point.
My Case (just wishing, might not be needed):
It would be nice to take my callout headings and my ## rest erledigt inside my main # Todos : with me. Maybe even a blank line seperating my different sections would do the job, too.
My conclusion for me:
I might change my Todos section accordingly. Insert checkbox headings for the different sections.
But in the end, for my approach, it should be possible to choose a / severall source headings
and it would be nice to get them spilled out under one spicific heading (in my case: ## rest erledigt).
I see this conflicts with your approach. But as described above I cannot make changes to check out your approach.
Thank you and well done for now!

Thanks for checking it! Yes, refreshing the template-headings is not done at the moment. We basically assume, that your daily-notes-template is static and unchanged. I might have to come up with a cache-invalidation-strategy at some point, but I consider this to be out of scope for now.
To fix it for you: Uninstall and reinstall the plugin and activate the headings to your liking.
I'll leave this issue open though! It's a request that's not totally open from my POV. Thanks again for checking it and helping me find those two bugs!

Wow, allright!
I deleted all that callout stuff & put everything on lowest level.
Now, stuff gets copied / moved from & to the appropriate headline.
Everything working as aspected.
Thanks a lot !! ✨🥳

how easy can you integrate a button to refresh the template headings manually?
my old template seems to stay cached even after reinstalling the release-2 branch

@kxc0re that’s a good idea. I have no idea how Obsidian caches settings, but I noticed this problem as well.

Can someone briefly explain how I can set up Release 2.0 to only carry over tasks to the next day under a specific heading?

See https://github.com/lumoe/obsidian-rollover-daily-todos/issues/80 for related discussion


Performance seems inadequate

Hi
When this container is installed and being used in a Synapse environment, pulling data from a DB2 database, transfer times are really slow! 700k records which is around 3GB of data takes 9 1/2 hrs to transfer.
If we take docker out of the equation and install the SHIR directly on the windows server the performance issues are gone.
I am told Docker is setup to use Default process isolation mode, which should allow docker to consume as much of the hosts resources?
I also note that the stats are blank/static within the Integration Runtime monitor when using Docker, again when direct on Server they work. Memory is always 0MB, CPU stuck at 50%, Network always 0

As far as i am aware there is no other way to install multiple SHIRs on a VM for Synapse so i really need to get this working!

Hi @Alexdc-12
the monitor for SHIR in container is a known issue now due to docker limitation, and thus the data on the portal is not accurate.
could you please provide the ActivityID of the slow copy for us to help troubleshoot? thanks!

Hi @Alexdc-12 the monitor for SHIR in container is a known issue now due to docker limitation, and thus the data on the portal is not accurate. could you please provide the ActivityID of the slow copy for us to help troubleshoot? thanks!

Here is an ActivityID of one of the slow copies however, 6854ee8d-13c4-47be-9279-18d86c00c6fa

Hi @Alexdc-12 the monitor for SHIR in container is a known issue now due to docker limitation, and thus the data on the portal is not accurate. could you please provide the ActivityID of the slow copy for us to help troubleshoot? thanks!

Here is an ActivityID of one of the slow copies however, 6854ee8d-13c4-47be-9279-18d86c00c6fa

glad to hear it finally works well. With the ActivityID we double-checked that there are no running errors on IR and thus the problem is transient. If the abnormal performance reduction occurs frequently, please kindly to raise an Azure Support Ticket for more helps.
We are working on root problem verification of the monitor issue, and will update it on Microsoft Learn page Limitations.
Thanks!


Added optional argument "constraints" to RTCPeerConnection constructor

Currently the user cannot pass the needed "constraints" value to RTCPeerConnection when it is initializing.
This PR allows us to do that.
This PR does not break BC.

Constraints have been dropped from the standard.


cargo update

net -13 crates

bors r+

bors r+

Resolves https://github.com/notify-rs/notify/issues/267 which might have impacted us (I remember a discussion a few years ago).

Resolves https://github.com/notify-rs/notify/issues/267 which might have impacted us (I remember a discussion a few years ago).


Support for vscode-insiders

Will you support vscode-insiders?

Hi @tomy8910,
Thanks for bringing up that issue, since we're not distributing the extension through the store yet I'll make sure to see that with the team in charge of plugins install.

Closed due to age. The Kite extension is now available in the VS Code extension marketplace.


[WIP] Typescript Migration

PR for Migrating to Typescript (#7).
Features:

Type definitions available for consumable functions
Support es module imports

Lackings:

support commonjs module requires
documentation on how to install, build.

I will take over from here

There are a few places where we are using ? (optional) for required parameters. Please update those.


Do not use workers when untarring bundles in the preload phase

Haven't yet determined the performance impact but it should be faster.
I confirm I have signed the Contributor License Agreement.

I think we should establish if this is better before we merge

Hey @vkalpias, this PR was opened over a year ago. What would you like to do with it?

I will close this for now and we can bring it back if it becomes relevant again.


AttributeError: type object 'Tensor' has no attribute 'as_subclass'

i am having trouble using fastaiv2 , the problem is in line 240 of torch_core.py file
_
_
if not hasattr(torch,'as_subclass'):
setattr(torch, 'as_subclass', torch.Tensor.as_subclass)

Do you have PyTorch 1.6 installed? torch.Tensor.as_subclass is introdudced in 1.6 (https://github.com/pytorch/pytorch/releases).   The latest fastai2 requires Pytroch requires 1.6 see forum post.

Do you have PyTorch 1.6 installed? torch.Tensor.as_subclass is introduced in 1.6 (https://github.com/pytorch/pytorch/releases). The latest fastai2 requires Pytroch requires 1.6 see forum post.

Oh thanks worked ! didnot know that !


DEBUG: PID 14371 (ippfind _ipp._tcp) exited with no errors.

Implemented ippfind_based_uri_converter() .It is about 0.02 sec faster than the cupsBackendDeviceURI()
function, hence,used it for both printer and fax URIs.
This command successfully generates the ppd.
driverless-fax cat driverless-fax:ipps://HP%20OfficeJet%20Pro%206960%20%5BC3B82D%5D._ipps._tcp.local/

Some issues:

IPP URIs do not always end with /ipp/print and fax URIs not always with /ipp/faxout. Especially Printer Applications can support more than one physical device or high-end devices could have more than one print engine. See here for examples. Therefore you always have to insert the actual content of the "rp" and "rfo" TXT record fields. You can read out these fields with ippfind, with the same command line as you are already using for reading out the service name and the port.
Tou have replaced the use of the function cupsBackendDeviceURI() by your new ippfind_based_uri_converter() function, but put the new function into utils/driverless.c and not into cupsfilters/ipp.c. You also removed the call of the old function from the get-printer-attributes functions, which probably breaks cups-browsed which also uses this function. You should put ippfind_based_uri_converter() into cupsfilters/ipp.c and perhaps allow some switch between the two. get-printer-attributes needs a URI converter function call for cups-browsed to work.
Also how did you find out that your new function is 0.02 seconds faster? And what is the speed advantage in percent?

You should put ippfind_based_uri_converter() into cupsfilters/ipp.c and perhaps allow some switch between the two. get-printer-attributes needs a URI converter function call for cups-browsed to work.

I have to change the defintion of get-printer-attributes function to add one more argument to check if its getting called by cups-browsed or driverless(as only these two calls the get-printer-attributes)and therfore could switch between ippfind_based_uri_converter() and cupsBackendDeviceURI()  accordingly. Would it be fine?

Also how did you find out that your new function is 0.02 seconds faster?

I have used clock() function available in time.h

To add a parameter to a function, changing its call scheme, do not change it straight away, as someone could use libcupsfilters for something which we do not know about. Therefore one has to keep the API. You see already that we have get_printer_attributes(), get_printer_attributes2(), and get_printer_attributes3(), with slightly different call schemes. During the time since we introduced the function we needed some call scheme alternatives.
In (the original) driverless.c I see a line
response = get_printer_attributes(uri, NULL, 0, NULL, 0, 1);

It is a call of the get_printer_attributes() function which takes 6 arguments. Assume you want to change this one to add an is_fax argument:
response = get_printer_attributes4(uri, NULL, 0, NULL, 0, 1, is_fax);

You see that we introduce a new name, to not change the old name's call scheme. Now we need to modify ipp.c (and ipp.h) to have the get_printer_attributes4() which takes the extra, boolean argument for whether the fax channel arguments of the device specified by uri should be polled.
What you have to do now is to have a look into ipp.c which function does the actual "dirty" work. It is get_printer_attributes3(). Copy the

ipp_t *
get_printer_attributes3(http_t http_printer,
const char raw_uri,
const char* const pattrs[],
int pattrs_size,
const char* const req_attrs[],
int req_attrs_size,
int debug,
int* driverless_info)
{
To right over the comment above it. Then, at the original, add an extra `, int is_fax` at the end and change the name from `get_printer_attributes3` to `get_printer_attributes5`. To the copy above you add

get_printer_attributes5(http_printer, raw_uri, pattrs[], pattrs_size, req_attrs[], req_attrs_size, debug, driverless_info, 0);
}
to complete the function definition. So the new get_printer_attributes3() calls get_printer_attributes5() with `is_fax` set to `0`, so that it behaves as the old function. The new function get_printer_attributes5() which now does all the dirty work you need to modify, so that it resolves the URI with your new function (which you have to move into ipp.c). Your new function simply takes the `is_fax` argument and so does the right thing.
Now add a get_printer_attributes4() with the same call scheme as get_printer_attributes() but with `is_fax` added. This one calls get_printer_attributes5() then.

Also make sure that cups-browsed stays working.

Thanks, @nidhijainpnp, now you only need to check whether all allocated memory (which is not a return value) gets freed before exiting the function, and if allocated memory is a return value, that it can be freed by the caller (the pointer returned has to point at the beginning of the allocated memory).

Till,
I have checked ,all allocated memory gets freed . Allocated memory by ippfind_based_uri_converter()in cupsfilters/ipp.c is returned to generate_ppd() in utils/driverless.c as ipp_t response which is freed by ippDelete(response)

Great, thanks.
Now we need the second part. The PPD file generator needs fax support. The PPD file generator is in the file cupsfilters/ppdgenerator.c. For a fax queue the PPD has to get two items added:
First, the PPD must tell CUPS that it is for fax. For this, it needs
*cupsIPPFaxOut: True

This makes the function cupsBackendDeviceURI() of libcups (in cups/backend.c of CUPS source code) identify a queue as fax queue and so resolves the DNS-SD-based URI to an IPP-based fax URI.
second, we have to tell the print dialog (at least it should work with the GTK one) that we want to have an option named "phone" which takes a string. This is the phone number to be dialed by the printer where the fax should go to.
For this we want to add something like
*% Base "Phone" option
*OpenUI Phone/Phone Number: PickOne
*OrderDependency: 10 AnySetup *Phone
*DefaultPhone: None
*Phone None: ""
*CloseUI: *Phone
*% Custom "Phone" option
*CustomPhone True: ""
*ParamCustomPhone Text: 1 string 0 64

to the PPD file. This adds an option named "Phone" and using CUPS' PPD extensions the print dialog is told to show a string input field for a maximum of 64 characters. The user can then enter the phone number here or copy and paste it from his contact list.
The option entry is always the same, regardless of printer. The "Phone" is a standard IPP option understood by all IPP printers with fax support.
Could you add your changes in the PPD generator to this Pull Request? Thanks.

I have read this from the standard IPP Fax Out Standard Abstract of June 18, 2014:
ipp-features-supported (1setOf type2 keyword)
This specification defines the REQUIRED keyword 'faxout' for the "ipp-features-supported" Printer attribute.

It is given in the table of  REQUIRED Printer Description attributes for a Printer. So,I have checked the fax queue with ipp-features-supported attribute as key and faxout as value.

I have read this from the standard IPP Fax Out Standard Abstract of June 18, 2014:
ipp-features-supported (1setOf type2 keyword)
This specification defines the REQUIRED keyword 'faxout' for the "ipp-features-supported" Printer attribute.

It is given in the table of REQUIRED Printer Description attributes for a Printer. So,I have checked the fax queue with ipp-features-supported attribute as key and faxout as value.

OK, great, then we go this way, as you did.

You can always test with CUPS' cupstestppd utility.

I have checked with both phone(lowercase) and Phone(uppercase). Both ways were working. But ,I have used lowercase one as you have suggested.

print a fax via lp -d fax-printer -o phone=12345678 file.pdf, ...

I have also checked this ,it is successfully printing fax.
lp -d HP_OfficeJet_Pro_6960_C3B82D_ -o phone=12345678 /home/nidhi/Desktop/finally.pdf

Thank you very much. I have merged it now and also will pass it over to 1.x branch.


Should not set Content-Transfer-Encoding to 7bit for 8bit files.

Mail::Message#add_file sets Content-Transfer-Encoding to 7bit even if the file is 8bit.
For example,
m = Mail.new
m.add_file("utf8.txt")
print m

The above code produces the following output:
Date: Fri, 28 Apr 2017 10:50:22 +0900
Message-ID<EMAIL_ADDRESS>Mime-Version: 1.0
Content-Type: multipart/mixed;
 boundary="--==_mimepart_59029fdec19b9_16722af16027d69478448";
 charset=UTF-8
Content-Transfer-Encoding: 7bit

----==_mimepart_59029fdec19b9_16722af16027d69478448
Content-Type: text/plain;
 charset=UTF-8;
 filename=utf8.txt
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename=utf8.txt
Content-ID<EMAIL_ADDRESS>
テストです。

----==_mimepart_59029fdec19b9_16722af16027d69478448--

7bit is the content transfer encoding, not the encoding of the content itself. The transfer encoding ensures that the message is ASCII-safe and does not contain arbitrary binary data.

@jeremy

7bit is the content transfer encoding, not the encoding of the content itself. The transfer encoding ensures that the message is ASCII-safe and does not contain arbitrary binary data.

I don't catch your point.
m.add_file("utf8.txt") generates a part with Content-Transfer-Encoding: 7bit and
8bit characters, while it's not allowed in Section 6.2 of RFC2045 as follows:

The proper Content-Transfer-Encoding label must always be used.
Labelling unencoded data containing 8bit characters as "7bit" is not
allowed, nor is labelling unencoded non-line-oriented data as
anything other than "binary" allowed.

Is it intended behavior?


thanos store sharding memory issue

thanos, version 0.8.1 (branch: HEAD, revision: bd8278859b2321aaaa7514edde764816cc039d34)
  build user:       root@2227d9a2fdb1
  build date:       20191014-12:03:55
  go version:       go1.13.1

Object Storage Provider: minio
What happened:
I have multiple prometheus instances in single s3-bucket with different external labels service.
When I tried use --selector.relabel-config for sharding, but all my shards use some memory (as previously all-in-one store instance). So, because all shards have all-in-one memory, sharding not working now.
My configs:
3 prometheus witth external labels:
service: one
service: two
service: another
2 shards with configs:
 - source_labels: [service]
      separator: ;
      regex: one
      replacement: $1
      action: keep

 - source_labels: [service]
      separator: ;
      regex: not_exists
      replacement: $1
      action: keep

I specifically use a nonexistent label for the experiment. I this setup all my shards use around 10Gb Ram (not_exists too).
In thanos query ui i see:

one: true labels and time range
not_exists: no labels and time range beetween 292278994-08-17T07:12:55Z

In s3 logs i see that thanos get for blacklisted block all: index, index.cache.json, meta.json for all shards.
What you expected to happen:
Sharding with --selector.relabel-config requere memory as need for only shard blocks

@R4scal Is your first store memory usage in expectation?

@jojohappy yes

In addition, attach CPU and memory on a non-existent shard

This is definitely wrong, we are looking in this, thanks for reporting :+1:

Found the problem, fixing now

This should be fixed, can you verify? E.g by using master image: master-2019-10-22-64af185e from https://quay.io/repository/thanos/thanos?tag=latest&tab=tags

Yep, I will try and reply today

Awesome!
I think we internally agreed to not release v0.8.2 unless necessary as sharding is experimental. We want to release this in v0.9.0 around 20thNov.
Is that fine?
cc @jojohappy

I think we internally agreed to not release v0.8.2 unless necessary as sharding is experimental. We want to release this in v0.9.0 around 20thNov.

Agree.

Ok, thanks


Add brazilian portuguese support

Hi, with this pull request i added support for the most common keyboard layout here in Brazil:
Option "XkbLayout" "br"
Option "XkbVariant" "abnt2"
I've been using this for some time with good results.
I would greatly appreciate if you merge this,so everybody here in brazil would benefit from it.
Thanks

As a suggestion, i would write only one function to deal with the layouts.
Instead of using in layouts.py:
def english_usa(keys)
def spanish(keys)
def pt_br(keys)
I would replace it with only one function with the layout as other argument, ex.:
def deal_with_layout(keys, layout=english_layout)
This would make it easier to add support for many other languages and keyboard layouts, but as it involves many more files to change, it's subject for other pull request.

Hi phoemur, thanks for the PR, is now merged.
Greetings!


Dispose HTTPClient after disposing buffer sink first.

This correct the dispose order issue   #4.

thank you, @SilentBlueD666, for the contribution and sorry for the delay


Replace moment to luxon

Use luxon as datetime library.

How about dayjs?
It requires only 2KB footprint (more smaller than luxon). Also it's a drop-in replacement of moment.
See also: https://www.npmtrends.com/dayjs-vs-luxon-vs-moment-vs-date-fns

Ah I didn’t notice no tz database (olson or Intl API) support found for dayjs.
If required, we need to write plugin for it.
https://github.com/iamkun/dayjs/issues/46

#535

I decide to use date-fns instead of luxon. The main purpose of this PR is for #535. date-fns is more small, used library.
https://blog.date-fns.org/post/we-cut-date-fns-v2-minimal-build-size-down-to-300-bytes-and-now-its-the-smallest-date-library-18f2nvh2z0yal
https://www.npmtrends.com/date-fns-vs-luxon-vs-moment


Synthesis failed for nodejs-bigquery

Hello! Autosynth couldn't regenerate nodejs-bigquery. :broken_heart:
Please investigate and fix this issue within 5 business days.  While it remains broken,
this library cannot be updated with changes to the nodejs-bigquery API, and the library grows
stale.
See https://github.com/googleapis/synthtool/blob/master/autosynth/TroubleShooting.md
for trouble shooting tips.
Here's the output from running synth.py:
 git checkout -b <new-branch-name>

HEAD is now at 16d2a50 chore(deps): update dependency sinon to v10 (#935)
2021-04-09 01:02:19,233 autosynth [DEBUG] > Running: git checkout fbc7b766ef43c75293b26159ecaf66cb3fb23770
Note: checking out 'fbc7b766ef43c75293b26159ecaf66cb3fb23770'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at fbc7b76 feat: add excludes and patch callback arguments to owlbot_main (#1032)
2021-04-09 01:02:19,342 autosynth [DEBUG] > Running: git branch -f autosynth-76
2021-04-09 01:02:19,347 autosynth [DEBUG] > Running: git checkout autosynth-76
Switched to branch 'autosynth-76'
2021-04-09 01:02:19,354 autosynth [INFO] > Running synthtool
2021-04-09 01:02:19,354 autosynth [INFO] > ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2021-04-09 01:02:19,354 autosynth [DEBUG] > log_file_path: /tmpfs/src/logs/nodejs-bigquery/76/sponge_log.log
2021-04-09 01:02:19,357 autosynth [DEBUG] > Running: /tmpfs/src/github/synthtool/env/bin/python3 -m synthtool --metadata synth.metadata synth.py --
2021-04-09 01:02:19,621 synthtool [DEBUG] > Executing /home/kbuilder/.cache/synthtool/nodejs-bigquery/synth.py.
On branch autosynth-76
nothing to commit, working tree clean
2021-04-09 01:02:19,786 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
DEBUG:synthtool:Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.eslintignore
.eslintrc.json
.gitattributes
.github/CODEOWNERS
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/workflows/ci.yaml
.kokoro/.gitattributes
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/lint.cfg
.kokoro/continuous/node12/samples-test.cfg
.kokoro/continuous/node12/system-test.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/populate-secrets.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/samples-test.cfg
.kokoro/presubmit/node12/system-test.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/publish.sh
.kokoro/release/docs-devsite.cfg
.kokoro/release/docs-devsite.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.kokoro/trampoline_v2.sh
.mocharc.js
.nycrc
.prettierignore
.prettierrc.js
.trampolinerc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
api-extractor.json
renovate.json
samples/README.md
2021-04-09 01:02:19,942 synthtool [DEBUG] > Installing dependencies...
DEBUG:synthtool:Installing dependencies...
npm WARN deprecated<EMAIL_ADDRESS>Breaking change found in this patch version

><EMAIL_ADDRESS>prepare /home/kbuilder/.cache/synthtool/nodejs-bigquery
> npm run compile

><EMAIL_ADDRESS>precompile /home/kbuilder/.cache/synthtool/nodejs-bigquery
> gts clean

version: 14
Removing build ...

><EMAIL_ADDRESS>compile /home/kbuilder/.cache/synthtool/nodejs-bigquery
> tsc -p . && cp src/types.d.ts build/src/

node_modules/@types/sinon/index.d.ts:778:36 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'TimerId'.

778     type SinonTimerId = FakeTimers.TimerId;
                                       ~~~~~~~

node_modules/@types/sinon/index.d.ts:780:39 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'InstalledMethods'.

780     type SinonFakeTimers = FakeTimers.InstalledMethods &
                                          ~~~~~~~~~~~~~~~~

node_modules/@types/sinon/index.d.ts:781:20 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'NodeClock'.

781         FakeTimers.NodeClock &
                       ~~~~~~~~~

node_modules/@types/sinon/index.d.ts:782:20 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'BrowserClock'.

782         FakeTimers.BrowserClock & {
                       ~~~~~~~~~~~~

Found 4 errors.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>compile: `tsc -p . && cp src/types.d.ts build/src/`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>compile script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepare script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2021-04-09T08_02_45_851Z-debug.log
2021-04-09 01:02:45,885 synthtool [ERROR] > Failed executing npm install:

None
ERROR:synthtool:Failed executing npm install:

None
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/nodejs-bigquery/synth.py", line 14, in <module>
    node.install()
  File "/tmpfs/src/github/synthtool/synthtool/languages/node.py", line 171, in install
    shell.run(["npm", "install"], hide_output=hide_output)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['npm', 'install']' returned non-zero exit status 1.
2021-04-09 01:02:45,954 autosynth [ERROR] > Synthesis failed
2021-04-09 01:02:45,955 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at 16d2a50 chore(deps): update dependency sinon to v10 (#935)
2021-04-09 01:02:45,965 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2021-04-09 01:02:45,972 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Removing node_modules/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 356, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 191, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 336, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 68, in synthesize_loop
    has_changes = toolbox.synthesize_version_in_new_branch(synthesizer, youngest)
  File "/tmpfs/src/github/synthtool/autosynth/synth_toolbox.py", line 259, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth is still having trouble generating nodejs-bigquery. :sob:
Please investigate and fix this issue within 5 business days.  While it remains broken,
this library cannot be updated with changes to the nodejs-bigquery API, and the library grows
stale.
See https://github.com/googleapis/synthtool/blob/master/autosynth/TroubleShooting.md
for trouble shooting tips.
Here's the output from running synth.py:
xample:

  git checkout -b <new-branch-name>

HEAD is now at 16d2a50 chore(deps): update dependency sinon to v10 (#935)
2021-04-10 01:02:02,818 autosynth [DEBUG] > Running: git checkout 0a071b3460344886297a304253bf924aa68ddb7e
Note: checking out '0a071b3460344886297a304253bf924aa68ddb7e'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 0a071b3 chore: add constraints file check for python samples (#1012)
2021-04-10 01:02:02,931 autosynth [DEBUG] > Running: git branch -f autosynth-81
2021-04-10 01:02:02,934 autosynth [DEBUG] > Running: git checkout autosynth-81
Switched to branch 'autosynth-81'
2021-04-10 01:02:02,941 autosynth [INFO] > Running synthtool
2021-04-10 01:02:02,941 autosynth [INFO] > ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2021-04-10 01:02:02,941 autosynth [DEBUG] > log_file_path: /tmpfs/src/logs/nodejs-bigquery/81/sponge_log.log
2021-04-10 01:02:02,943 autosynth [DEBUG] > Running: /tmpfs/src/github/synthtool/env/bin/python3 -m synthtool --metadata synth.metadata synth.py --
2021-04-10 01:02:03,202 synthtool [DEBUG] > Executing /home/kbuilder/.cache/synthtool/nodejs-bigquery/synth.py.
On branch autosynth-81
nothing to commit, working tree clean
2021-04-10 01:02:03,362 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
DEBUG:synthtool:Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.eslintignore
.eslintrc.json
.gitattributes
.github/CODEOWNERS
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/workflows/ci.yaml
.kokoro/.gitattributes
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/lint.cfg
.kokoro/continuous/node12/samples-test.cfg
.kokoro/continuous/node12/system-test.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/populate-secrets.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/samples-test.cfg
.kokoro/presubmit/node12/system-test.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/publish.sh
.kokoro/release/docs-devsite.cfg
.kokoro/release/docs-devsite.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.kokoro/trampoline_v2.sh
.mocharc.js
.nycrc
.prettierignore
.prettierrc.js
.trampolinerc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
api-extractor.json
renovate.json
samples/README.md
2021-04-10 01:02:03,530 synthtool [DEBUG] > Installing dependencies...
DEBUG:synthtool:Installing dependencies...
npm WARN deprecated<EMAIL_ADDRESS>Breaking change found in this patch version

><EMAIL_ADDRESS>prepare /home/kbuilder/.cache/synthtool/nodejs-bigquery
> npm run compile

><EMAIL_ADDRESS>precompile /home/kbuilder/.cache/synthtool/nodejs-bigquery
> gts clean

version: 14
Removing build ...

><EMAIL_ADDRESS>compile /home/kbuilder/.cache/synthtool/nodejs-bigquery
> tsc -p . && cp src/types.d.ts build/src/

node_modules/@types/sinon/index.d.ts:778:36 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'TimerId'.

778     type SinonTimerId = FakeTimers.TimerId;
                                       ~~~~~~~

node_modules/@types/sinon/index.d.ts:780:39 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'InstalledMethods'.

780     type SinonFakeTimers = FakeTimers.InstalledMethods &
                                          ~~~~~~~~~~~~~~~~

node_modules/@types/sinon/index.d.ts:781:20 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'NodeClock'.

781         FakeTimers.NodeClock &
                       ~~~~~~~~~

node_modules/@types/sinon/index.d.ts:782:20 - error TS2694: Namespace '"/home/kbuilder/.cache/synthtool/nodejs-bigquery/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'BrowserClock'.

782         FakeTimers.BrowserClock & {
                       ~~~~~~~~~~~~

Found 4 errors.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>compile: `tsc -p . && cp src/types.d.ts build/src/`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>compile script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepare script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2021-04-10T08_02_26_766Z-debug.log
2021-04-10 01:02:26,797 synthtool [ERROR] > Failed executing npm install:

None
ERROR:synthtool:Failed executing npm install:

None
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/nodejs-bigquery/synth.py", line 14, in <module>
    node.install()
  File "/tmpfs/src/github/synthtool/synthtool/languages/node.py", line 171, in install
    shell.run(["npm", "install"], hide_output=hide_output)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['npm', 'install']' returned non-zero exit status 1.
2021-04-10 01:02:26,851 autosynth [ERROR] > Synthesis failed
2021-04-10 01:02:26,852 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at 16d2a50 chore(deps): update dependency sinon to v10 (#935)
2021-04-10 01:02:26,860 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2021-04-10 01:02:26,866 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Removing node_modules/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 356, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 191, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 336, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 68, in synthesize_loop
    has_changes = toolbox.synthesize_version_in_new_branch(synthesizer, youngest)
  File "/tmpfs/src/github/synthtool/autosynth/synth_toolbox.py", line 259, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


Take 2: Use 70% less memory for TokenizedLines

#6757 contained a couple of regressions that required me to back it out. This should fix both of them. I'm going to merge this after we release today.

Trying again. Hold onto your :tophat:s. Please ping me if you see anything :fish:y.


Adding boolean account status 'closed'

Hi,
Created a pull request to add the 'closed' boolean object as a property in the Account endpoint. I noticed this was not being pulled from the API response - this would help when iterating over a list of accounts to only attempt to pull transactions from accounts that are not closed. For example, I have the old 'mondo' prepaid account still listed which looks just like my current account (begins 'user...'), but is now closed.
Please let me know if anything is missing or does not look right.
Thanks,
Elias

Hi @eliasbenaddou thank you very much for the merge request. Everything looks good.
I am currently away on holiday. I will get this merged once I return.
Thanks again.


fix ring of pain stuff and make destiny draw better

I took a quick look at this as my last thing before signing off for the night, just to take a bit of the PR burden off Gk. Looks good to me.


Find a way to load from axios a components library, mount to window, and use components as if from externals

Merge into #385


Remove usage of method placeholder functions

This PR introduces a new minor client version 0.67.x that does not create method placeholder functions for modal classes.
I've tested all forward lookup scenarios (.lookup() of a 0.67 class from a pre 0.67 client) as well as backwards lookup scenarios (.lookup() of a pre 0.67 class from a 0.67 client) and all work, except for a 0.62 client looking up a 0.67 class, which will be explicitly not supported (this maintains our current restriction of not being able to lookup a 0.63+ class from a 0.62 client).

lovely! did we decide to not backfill method metadata on existing classes in order to remove existing placeholders? (completely understandable, seems messy).
Another related question: Will "forward" lookups work after this? I.e. can a 0.63 client .lookup() a 0.67 class? The way I understand the ClassGet backend implementation the answer is yes, since it "mocks out" placeholders with the function id set to the class service function?

@freider  yes, forward lookups will work, excluding lookups from a 0.62 client, which is currently not supported either.
re: your first question, yea it feels a bit messy. i'm guessing i would have to have the server modify the class service function definitions coming from older clients to include the method definitions mapping and that will likely introduce more confusion

@prbot approve


Parameter name tableClasses on table component is not consistent with other components that just use classes

If table macro is removed (#2352), this is not required


Display multiple type definitions for overloaded methods

Currently if a method is overloaded, only one definition is displayed. For example the AngularJS service method is defined as:

method) angular.IModule.service(name: string, serviceConstructor: Function): angular.IModule (+2 overloads)

With all the extra space in the tooltip box, this description could be expanded to show all definitions (in this case the other 2 overloads).

Hmm, this functionality is mostly dictated by what is provided by the TypeScript language service. If support for displaying more info is in there and you'd like to work on this, please feel free to submit a pull request.


Implement main functions for custom dialect tools and drop alwayslink from dialect

Adds a function to explicitly register the custom dialect.
Adds own main functions for custom-translate and custom-opt, in which the registration function is called.
Drops alwayslink from the custom dialect.

Closes #1636

The layering check is upset in a big way on this one. It wants direct deps for everything, not going through iree_opt_library

I added those deps providing the headers used in the source code with is compiled into the cc_binary targets. Hopefully got them all :)

Messed up during rebasing and squashing, but should be fine now.

Thanks! Normally I would've just fixed the deps on import, but it seemed like it materially altered the intent here :-)
So it's a little unclear now what the purpose of iree_opt_lib is. It's just bundling some deps, but some of them don't seem like the sort of thing that need to be bundled. If we used it to encapsulate most of the work of the main function, so that the custom main didn't need so much copypasta I think that would make sense, but as it stands now it's just bundling deps, which is usually discouraged in bazel for the same reasons I outlined above. I think it makes sense for deps that are alwayslink because there you end up with runtime errors if you forget them, but for "normal" deps it's usually better to just include them directly.
Options here:

Inline all the deps of iree_opt_lib
Change iree_opt_lib -> iree_dependent_dialect_registration, or somehttps://github.com/tensorflow/tensorflow/commit/9ca9a4f8e9b4d60408a092db4320805711367042thing like that, and only leave in the XLA dialect registrations and similar type deps. Inline the rest of the deps.
Leave it as-is, with a TODO to add a source file to iree_opt_lib to pull some of the main functionality in there. We can do this to reduce the churn here if it's the next step anyway. I don't like leaving this sort of library around very much though.

So it's a little unclear now what the purpose of iree_opt_lib is. It's just bundling some deps, but some of them don't seem like the sort of thing that need to be bundled.

You're right, the intention somehow got lost during the iteration.

If we used it to encapsulate most of the work of the main function, so that the custom main didn't need so much copypasta I think that would make sense, but as it stands now it's just bundling deps, which is usually discouraged in bazel for the same reasons I outlined above.

Agreed. However, I think there isn't much it is worth to encapsulate.

I think it makes sense for deps that are alwayslink because there you end up with runtime errors if you forget them, but for "normal" deps it's usually better to just include them directly.

Well, I think we'll get rid of alwayslink (maybe expect for TF). So no real need to bundle deps in a separate library. The ones that have to be linked as whole archive are documented in the out-of-tree example anyway :)

Options here:
1. Inline all the deps of `iree_opt_lib`

2. Change `iree_opt_lib` -> `iree_dependent_dialect_registration`, or something like that, and only leave in the XLA dialect registrations and similar type deps. Inline the rest of the deps.

3. Leave it as-is, with a TODO to add a source file to `iree_opt_lib` to pull some of the main functionality in there. We can do this to reduce the churn here if it's the next step anyway. I don't like leaving this sort of library around very much though.

I think the second option would just introduce another not really necessary target. Thus I went for the first option.

Now we're getting failures from the tf side because it's still depending on iree_opt_main/iree_translate_main for its main file and such. I think you can revert your changes to iree/tools/BUILD and iree/tools/CMakeLists.txt and everything should be fine form for the custom opt, right?

I've confirmed that reverting the changes in tools/BUILD is fine for our internal builds. I can do that as part of import and fixup the description to match, if that's easier for you than pushing new commits here

Missed to grep through integrations and that the main entry functions are linked into iree-tf-opt and iree-tf-translate :/ Sorry. Reverted the changes.

Missed to grep through integrations and that the main entry functions are linked into iree-tf-opt and iree-tf-translate :/ Sorry. Reverted the changes.

No worries. Sorry for all the back and forth here. If we had OSS presubmits for the TF code this would be much easier :-/ Imported and all the tests pass. Waiting on another Googler to approve.
Meta question: it's easy for me to make arbitrary changes when merging, but I've avoided doing that because I want to make sure to get your input and not rewrite your descriptions. What is your preference on this? Is the current situation OK, or should I be more free with changes during merge?


Skeleton component don't work as expected

The skeleton component only reacts correctly to the className property. The style prop will the passed into the outer container of the skeleton. Not on the skeleton itself.

The correct div for the skeleton is the div with the class "flow--skeleton".

The issue you reported is a regression introduced by fa4ea9c0d75e110df458130a158ef6667a768aef. So a revert should be ok.
But we still need a Skeleton component, that behaves like text with a given line-height and font-size. Compare the loading view of the List items, to see the difference:

after your change: https://mittwald.github.io/flow-previews/storybook/fix/skeleton/?path=/story/structure-list--default
and before: https://mittwald.github.io/flow-previews/storybook-main/?path=/story/structure-list--default

I would suggest the following:

copy over the current Skeleton to SkeletonText, fix the className issue and remove the height prop
revert the commit fa4ea9c0d75e110df458130a158ef6667a768aef


Remove all instances of  'ZERO WIDTH NON-JOINER' (U+200C) present in mappings.

Please no invisible unicode characters.
For reference, other non-ASCII characters in Yarn are:

One “§” U+00A7 Section Sign
https://github.com/FabricMC/yarn/blob/fd85633178de1fb3c2b82654f868f6f1370f0277/mappings/net/minecraft/client/font/TextVisitFactory.mapping#L38
Four “×” U+00D7 Multiplication Sign
https://github.com/FabricMC/yarn/blob/fd85633178de1fb3c2b82654f868f6f1370f0277/mappings/net/minecraft/world/biome/source/BiomeCoords.mapping#L8-L9

Idk how this even happens. Removing the other two might not be a bad idea, but this change is good 👍

We could include this in Javadoc linting as well.

Afaik, the javadoc tool's output html include such no-break whitespaces. Maybe we just happened to copy the contents straight from the javadoc html and inadvertently included such spaces as a result.

other non-ASCII characters

Just curious, what's wrong with other non-ASCII? Yarn already enforces utf-8 for compilation and production.
If we do need to remove those characters, we can use § for § and × for × (which I am already doing).

Just curious, what's wrong with other non-ASCII?

Nothing I'm aware of. I just had already compiled this information and decided to put it here in case anyone else wanted to know if there were any other odd unicode characters present (or in other words, as a justification for this PRs completeness).


[$250] Add support for viewing full screen Group Chat custom avatars

Problem
When we added support for custom Group Chat avatars there are some places left where we cannot yet view the full screen avatar.
Solution
Navigate to the correct page like we do here when user taps on the avatar: https://github.com/Expensify/App/blob/ddb83f2493c3d08b3b3febcfd9dc93729f38e3c2/src/components/RoomHeaderAvatars.tsx#L19-L28
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c10e84b6d429f37d
Upwork Job ID:<PHONE_NUMBER>325901824
Last Price Increase: 2024-04-29
Automatic offers:

s77rt | Reviewer | 0

Issue OwnerCurrent Issue Owner: @

No progress yet. Focused on QBO

I think ideally we'd follow the same pattern that we use for custom workspace avatars.

cc @Expensify/design - I actually think the order of these should be Upload, View, Remove. But that's not a super pressing concern.

Hey @s77rt, let us know if you have started working on this. I can make it external, but would love if you could be the C+ still.

I agree with the proposed solution from Danny.

@marcaaron Making this External sounds good to me. I'd love to be the C+ still

@nexarvo Thanks for the proposal. Your RCA is correct. The solution looks overall good to me.
🎀 👀 🎀 C+ reviewed
Link to proposal

Thanks everyone 🚀

@nexarvo The PR has been reverted. Can you raise a PR with fixes for the found regressions?

@nexarvo Are you still able to work on this?

@marcaaron Let's reopen this for proposals

@tsa321 I haven't read the code fully but let's not use base64 uri. Dealing with long strings can be slow. If you want to pick up this issue please use the proposal template addressing the regressions from the previous PR

@marcaaron Can you readd Help Wanted label

summarizing the status of this one... seems like @nexarvo has bailed on this issue and needs to be unassigned here?
@tsa321 do you have some proposal for us?

@tsa321 Looking forward for the proposal. (code-diffs, branches, PRs, should be avoided)

@tsa321 Can you continue from https://github.com/Expensify/App/pull/41586 and just state the changes needed to fix each regression independently

@tsa321 Your proposal only covered adding support for viewing the group chat avatar while in the creation flow and didn't include a solution for doing so from the report screen (report details page).

@tsa321

I believe we don't really need to store the avatarFilename in optimistic data when creating a group chat

This is still needed, otherwise if you create a group chat offline and try to view the avatar in the report details page it won't work and we shouldn't use a fake file name either as we can't predict the extension

@tsa321

the navigation is smooth because we return a null element instead of the AttachmentModal

Let's try another approach, let's keep the modal displayed but display the default avatar instead (same as in the confirmation page)

I am just moving the condition into isPolicyRelatedReport to determine whether it is a policy-related report.

isPolicyRelatedReport = policyName !== unavailableTranslation looks like a hack. Can we find a better solution e.g. use report.policyID?

@tsa321 Can you update your proposal based on the two above points?

Just a heads up, I am going OOO until August 30th. I won't reapply the bug label as we're far from payout, but please can you keep this issue updated daily with your progress @tsa321 @s77rt
Thanks!

@tsa321

We could displays default avatar after user refresh the report avatar page

Can you explain how will you do that?

@tsa321

they will see a different default avatar for the group

Why would they see a different default avatar? They should get the same default avatar that was shown in the modal. They should see the avatar as highlighted in the video below
https://github.com/user-attachments/assets/27983a08-eece-4ce2-baa2-240b184e8aa6

@tsa321 Please do not post code diffs, PRs, etc. I think we should discuss the details in the PR.
🎀 👀 🎀 C+ reviewed
Link to proposal

Not overdue

Waiting on @marcaaron to review and assign @tsa321 for his proposal based on https://github.com/Expensify/App/issues/39850#issuecomment-2308761598

Just assigned Melv.

PR was deployed to production 2 weeks ago. This is ready for payment cc @zanyrenney

payment summary
paid @s77rt $250 via upwork
paid @tsa321 $250 via upwork.
Thanks 👋🏼

I have refunded 50% for the regressions (there were 2 but both found at same period and one of them was due to a "feature" that we just removed)


Fix buffer_response=true (#3145)

We need to call StartFun as it might add headers, etc.

+1, I will cut an RC2 for this.


QuickLaunch crashes on quit with new allocator

Debug information for team /boot/system/apps/QuickLaunch/QuickLaunch (2308):
CPU(s): 2x Intel Core™ i7-3635QM
Memory: 2.00 GiB total, 405.51 MiB used
Haiku revision: hrev53137 May 16 2019 19:57:09 (x86_64)

Active Threads:
	thread 2318: team 2308 debug task
	thread 2308: QuickLaunch (main)
		state: Call (thread 2308 tried accessing address 0x57ae40 which is not allocated (base: 0x57ae40, size: 440, alignment: 16, allocated by thread: 2308, freed by thread: 2317))

		Frame		IP			Function Name
		-----------------------------------------------
		00000000	0x1645e68db7f	_kern_debugger + 0x7
			Disassembly:
				_kern_debugger:
				0x000001645e68db78:   48c7c0e1000000  mov $0xe1, %rax
				0x000001645e68db7f:             0f05  syscall  <--

		0x7f7719d868d0	0x1645e736130	panic(char const*, ...) + 0xc0
		0x7f7719d86930	0x1645e7365ac	guarded_heap_segfault_handler(int, __siginfo_t*, void*) + 0x1ac
		0x7f7719d86930	0x7ffaa33c123b	commpage_signal_handler + 0x2b
		0x7f7719d86d20	0x1c075b0b0e1	QLSettings::~QLSettings() + 0x21
		0x7f7719d86d90	0x1c075b0b827	QLApp::~QLApp() + 0x97
		0x7f7719d86fb0	0x1c075b054b8	main + 0x28
		0x7f7719d86fe0	0x1c075b055fc	_start + 0x4c
		0x7f7719d87010	0x4e524f96c0	runtime_loader + 0x100
		00000000	0x7ffaa33c1260	commpage_thread_exit + 0

I had a quick look, and moved saving the search term from the app's d'tor to the MainWindow d'tor (c829080e496ee7481422322be63d5a9a1df651b6). That didn't solve the issue though... :}
Interesting: there's no issue on 32bit...

Seems to be fixed after commenting out https://github.com/humdingerb/quicklaunch/blob/master/QLSettings.cpp#L97
Not sure exactly why...

The issue always appears with the guarded heap, which isn't new. The new default allocator just has memory usage patterns enough like the guarded heap to trigger this.
Not sure why it's not an issue on 32-bit, though.

Thanks for the hints, waddlesplash!
@diversys : Should be fixed now. Can you retest? I'll push a bugfix release if it's OK later...


Added to show the number of instances in the topology list UI.

Added to show the number of instances in the topology list UI.
It would be nice to have this on the topology list. It is necessary to determine the status of which topology and how many instances are being used.

I might have missed something, but got the following error when attempting a local run.
Installed locally:
bazel run -- scripts/packages:heron-install.sh --user
I ran heron-tracker and heron-ui.
Ran a local topology: heron submit local ~/.heron/examples/heron-api-examples.jar org.apache.heron.examples.api.AckingTopology acking
Saw this error in the heron-tracker.
[2022-05-14 22:03:39 -0400] [INFO]: Adding new topology: acking, state_manager: local
Exception in thread Thread-1:
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 973, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 910, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/nnezis/.pex/unzipped_pexes/bca9046b34d3961d3cc495fabd3817f4243df382/heron/statemgrs/src/python/filestatemanager.py", line 133, in monitor
    trigger_watches_based_on_files(
  File "/Users/nnezis/.pex/unzipped_pexes/bca9046b34d3961d3cc495fabd3817f4243df382/heron/statemgrs/src/python/filestatemanager.py", line 112, in trigger_watches_based_on_files
    callback(proto_object)
  File "/Users/nnezis/.pex/unzipped_pexes/bca9046b34d3961d3cc495fabd3817f4243df382/heron/tools/tracker/src/python/topology.py", line 626, in set_execution_state
    self._update(execution_state=execution_state)
  File "/Users/nnezis/.pex/unzipped_pexes/bca9046b34d3961d3cc495fabd3817f4243df382/heron/tools/tracker/src/python/topology.py", line 609, in _update
    info = self._rebuild_info(t_state)
  File "/Users/nnezis/.pex/unzipped_pexes/bca9046b34d3961d3cc495fabd3817f4243df382/heron/tools/tracker/src/python/topology.py", line 278, in _rebuild_info
    metadata=self._build_metadata(topology, physical_plan, execution_state, tracker_config),
  File "/Users/nnezis/.pex/unzipped_pexes/bca9046b34d3961d3cc495fabd3817f4243df382/heron/tools/tracker/src/python/topology.py", line 395, in _build_metadata
    "instances": len(physical_plan.instances),
AttributeError: 'NoneType' object has no attribute 'instances'

Is it possible that I'm not running the right binary of heron-tracker? I'll try to verify what I did, but wanted to post just in case before this got merged in.

I pulled the latest from the branch and it runs better. I see the instance count on the main heron page, but I don't see it when viewing the topology details.

@nicknezis  Thanks
There seems to be a case where the object temporarily does not exist at the time of update.
So I added it to check whether the object exists.
And I executed the component as below.
Tracker
bazel build heron/tools/tracker/src/python:heron-tracker && bazel-bin/heron/tools/tracker/src/python/heron-tracker --config-file ~/.heron/conf/heron_tracker.yaml --verbose

HeronUI
ln -sf ~/.heron/release.yaml /private/var/tmp/_bazel_thinker0/1db3221e319a1f6b3420bd0c6f4d8d08/execroot/org_apache_heron/bazel-out/darwin-fastbuild/bin/heron/tools/ui/src/release.yaml

bazel build heron/tools/ui/src/python:heron-ui --verbose_failures && ./bazel-bin/heron/tools/ui/src/python/heron-ui --tracker-url http://localhost:8888

When you click on one of those links to view the details of a specific job, do you see the instance count beneath the logical and physical plan diagrams?

yes.

yes.

The value is blank in your image. I see the instance count at the top, but not in the middle of the image.

The value is blank in your image. I see the instance count at the top, but not in the middle of the image.

It is because the tracker cannot calculate all metrics.
This is a problem with topologies with many instances.

200 or so instances works fine.

It may be pending because heron-tracker takes a long time to get the results of the stats of the metrics-manager of all instances and calculate it.

Interesting. I wonder if we should update the page to use the variable that is used to present the instance count at the top of the page. I tested locally with the AckingTopology and it was empty. But I did not wait long enough for the metrics gathering to complete. I'll take a look at the html to better understand the difference of the top instance count and the middle instance count.

This change fixed the missing instance count.


[branch-2.11] [fix] [ml] Change the modifier of some variables in ManagedLedgerImpl to default to resolve compilation errors

Motivation
The modifier of variables currentLedgerEntries, currentLedgerSize and currentLedger in ManagedLedegrImpl changed to protected in PR #18265, this PR is for new features, so there is no cherry picking to branch 2.11.
PR #19404 fixes the problem of the managed ledger, and is already cherry-picked into branch 2.11, but this PR uses the above change( The modifier changes of some variables ), so the compilation of branch 2.11 errors now.
Modifications

change modifier of variables currentLedgerEntries, currentLedgerSize and currentLedger to default
Fixed license in some files

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Hi @Technoboy- , I think this PR cannot pass the CI because its target is branch-2.11, not master.


[move-lang] Remove old error reporting

Remove old error reporting as a part of error migration starting in #8588
@tzakian it might no longer make sense to use move-langs errors in the unit testing reporting :/

Motivation

More cleanup from #8588 migration
Next step will do some cleanup from some other crates

Test Plan

Ran em

/land


Revisit Map Apply UDF Retry

test pipeline
apiVersion: numaflow.numaproj.io/v1alpha1
kind: Pipeline
metadata:
  name: test-retry
spec:
  vertices:
  - name: in
    source:
      http: {}
  - name: forward-message
    scale:
      min: 1
    udf:
      container:
        image: quay.io/numaio/numaflow-go/map-forward-message-test:latest
  - name: log-output
    sink:
      log: {}
  edges:
  - from: in
    to: forward-message
  - from: forward-message
    to: log-output

test UDF in golang: the handler will exit and restart the UDF container
package main

import (
	"context"
	"log"
	"os"

	functionsdk "github.com/numaproj/numaflow-go/pkg/function"
	"github.com/numaproj/numaflow-go/pkg/function/server"
)

func mapHandle(_ context.Context, keys []string, d functionsdk.Datum) functionsdk.Messages {
	val := d.Value()
	log.Print(d.Metadata().NumDelivered(), string(val))
	os.Exit(1)
	return nil
}

func main() {
	server.New().RegisterMapper(functionsdk.MapFunc(mapHandle)).Start(context.Background())
}

Use port-forward and send curl -kq -X POST -d "101" https://localhost:8444/vertices/in
Expect to see d.Metadata().NumDelivered() gets increased after UDF restarts.
Actual results: (I forgot to add a space between numDelivered and value, it should be 1 101)

Potential root cause
Need to fix and clean up the retry logic https://github.com/numaproj/numaflow/blob/main/pkg/forward/forward.go#L707-L718 because we've added the "retry 5 times" in https://github.com/numaproj/numaflow/pull/744/files#diff-0c876de13c4addba8242fe3b40e5fd1eb0f3daab316cf0cbb51eb6c3cd1b8ba0R108-R114 for retryable errors.
May need to do the same for MapTransformer

Follow up: I think we may not be able to rely on JetStream numDelivered
Explanation:
Tested using the same pipeline and UDF. I updated the https://github.com/numaproj/numaflow/blob/main/pkg/forward/forward.go#L707-L718 to simply return nil, err which will later trigger the function call isdf.fromBufferPartition.NoAck(ctx, readOffsets)
I think the reader still keeps reading during UDF restarts and thus increases the numDelivered. While I believe the customer wants to see how many times the UDF has processed the same message.

example UDF in python will be
import os

from typing import List

from pynumaflow.function import Messages, Message, Datum, Server

def my_handler(keys: List[str], datum: Datum) -> Messages:
    val = datum.value
    _ = datum.event_time
    _ = datum.watermark
    print("delivered", datum.metadata.num_delivered)
    os._exit(1)
    return Messages(Message(keys=keys, value=val))

if __name__ == "__main__":
    grpc_server = Server(map_handler=my_handler)
    grpc_server.start()

https://github.com/numaproj/numaflow-python/issues/87

let's remove the metadata altogether. we will revisit it later if this is required.


Allow str to be passed as directory name in demo dataset loader

https://github.com/deepinv/deepinv/blob/2e2f27167ef95f44154b379bb4f844170b384123/deepinv/utils/demo.py#L54-L57
Change to from pathlib import Path; dataset_dir = Path(data_dir) / dataset_name

Thanks ! It will be changed with this PR #103


PNA server installation

@kaapstorm this is important. we need to be able to change ssh port otherwise UFW will lock us out.

Any reason not to merge this? It doesn't look like it would affect anything in prod


Core table functionality not working as expected?

So I have got 1.2.0 and have enabled tables option and used the example markup

| *foo* | **bar** | ~~baz~~ |
|:------|:-------:|--------:|
| 100   | [a][1]  | ![b][2] |

and it generates

Now I was expecting that the headers would have their * and ~~ parsed into styling, however it just translates them directly, same with the output content. I am assuming it should be taking the styling in the headers into account?

Actually, this is by design since it follows md tables spec1. However spec 2 does allow span elements in the titles. see https://michelf.ca/projects/php-markdown/dingus/
I will add a milestone and implement it in a future release.

no worries, it just seemed odd that the example on the website included this if it is not supported.

Ah, good catch

Fixed it in the README.md

give it a shot
https://github.com/showdownjs/showdown/pull/180

Just got the latest showdown.js from the dist folder on master and it still seems to be acting the same, the commit says its against master, could it just be that the dist was not updated?

you need to use the tivie-patch-1 branch

ah kk, will give it another go shortly, thanks again for the quick turn around.


Allow installing vault enterprise

Bumps default vault, consul and consul-modul versions at the packer template
Refactors install-vault script. It already allowed passing a url, but it was not explicit that it would supercede downloading a binary by the version from the releases page. So I updated to now have more parity with the consul-install module,
Allows passing consul and vault download urls to packer template
Updates docs

@infosecgithub

Does this change also allow the "Pro" version and License for Vault?
On Wed, Aug 29, 2018 at 2:44 PM Yevgeniy Brikman<EMAIL_ADDRESS>wrote:

@brikis98 approved this pull request.
LGTM 👍
Only tweak to consider is to specify both params in the Packer template to
simplify the code a bit. Other than that, ship it!
In examples/vault-consul-ami/vault-consul.json
https://github.com/hashicorp/terraform-aws-vault/pull/87#discussion_r213844254
:

@@ -57,7 +59,11 @@
},{
"type": "shell",
"inline": [

 "/tmp/terraform-aws-vault/modules/install-vault/install-vault --version {{user `vault_version`}}"

 "if [[ -n '{{user `vault_download_url`}}' ]]; then",

 " /tmp/terraform-aws-vault/modules/install-vault/install-vault --download-url {{user `vault_download_url`}};",

 "else",

 " /tmp/terraform-aws-vault/modules/install-vault/install-vault --version {{user `vault_version`}};",

Instead of this if-statement, another option to consider is that you can
set both --download-url and --version, except one will be set to empty
string and ignored.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hashicorp/terraform-aws-vault/pull/87#pullrequestreview-150766853,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnALxvpfeEmXCflAFFdjhSNBhgQXsxovks5uVwsqgaJpZM4WFmzX
.

--
Thanks and best regards,
David Donahue
Cloud Security Engineer, CBS Interactive InfoSec
<EMAIL_ADDRESS>(415) 601-7894

@infosecgithub This change allows installing any version by giving the build a specific url hosting the package. This change does not yet activate licenses. This will be done on a future PR


Added header for driver version and other info

See related issue for more information regarding requirements.
Added driver info header in HttpApiTransport.ApplyHeaders()

@DiscoPYF , this is done and ready. It'd be great if you can do a final check of approval before I merge.


Modify end of line comments, SimpleDateFormat >> LocalDateTime

修改行尾注释，去除不必要的异常抛出，修改SimpleDateFormat为LocalDateTime。

 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.cheney seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi, could you please sign the CLA?

Hi, could you please sign the CLA?
Already signed

It seems your email is not bound with Github.

It seems your email is not bound with Github.

seems good now

license/cla Pending — Contributor License Agreement is not signed yet.

please sign the CLA , thanks.

license/cla Pending — Contributor License Agreement is not signed yet.

please sign the CLA , thanks.

It seems i have signed

hey, you commit with two Github ids and only one of them has been signed.

嗨，您提交了两个Github ID，但其中只有一个已签名。

id 'cheney' was my mistake, and it has been fixed later


Pool overlaps with other one on this address space

I try to get 40 agents working on a beefier machine on EC2. My setup calls for 4 services running on docker compose. When a full run is running, there are maybe 30-35 concurrent steps. When it starts, I get this error for each step:
$ docker-compose -f docker-compose.yml -p buildkite5b8fb2692e7846369dfe5f795769144c -f docker-compose.buildkite-197-override.yml up -d --scale backend=0 backend
--
  | Creating network "buildkite5b8fb2692e7846369dfe5f795769144c_qms" with driver "bridge"
  | ERROR: Pool overlaps with other one on this address space

I tried creating and assigning a custom network to the services like so:
networks:
  qms:
    driver: bridge
    ipam:
      driver: default
      config:
        - subnet: <IP_ADDRESS>/16

But without success. Am I missing something in my configuration or is there something missing in this plugin to make this work?

Hi @juggy! The error would point to an issue with the networks themselves not being cleaned up for some reason. Unfortunately debugging that can not be done in isolation but as a history of the whole agent and instance itself. Maybe some other job not using this plugin (that runs a docker-compose down as a pre-exit hook to clean up after itself). It may be interesting to see the status of all the docker networks at that time to see more information on the matter as well. I could even be a limitation in the OS with the amount of virtual interfaces being run. Options are endless :(


[API] Contract more than two tensors

Ke told me that the current Contract function only work for two tensors. We should extend it to do more tensors.

This is supported in 16e0f25 with new function
Contracts(T1,T2,T3, .... )


[main < ] Update C query modules dev experience

TODOs

[ ] Consider how to restructure the custom query module page initial page
[ ] C page should include all info about basics, configs, implication and options
[ ] Example c is the getting started with C query module

Description
Please briefly explain the changes you made.
Pull request type
Please check what kind of PR this is:

[ ] Fix or improvement of an existing page
[ ] New documentation page, release related

Related PRs and issues
PR this doc page is related to:
(especially necessary if the PR is related to a release)
Closes:
(paste the link to the issue it closes)
Checklist:

[ ] Check all content with Grammarly
[ ] Perform a self-review of my code
[ ] Make corresponding changes to the rest of the documentation (consult with the DX team)
[ ] The build passes locally
[ ] My changes generate no new warnings or errors
[ ] Add a corresponding label
[ ] If release-related, add a product and version label
[ ] If release-related, add release note on product PR

We improved the C++ query module experience, which was a bigger priority than C.  For the time being, this can be closed.