gmskdem - is it possible to compensate drift?

As in the topic. It seems that GMSK requires perfect sample timing - I'm seeing the symbols 'drifting' more and more with time. Would it be possible to somehow do active frequency drift compensation?
symsync_crcf doesn't work on FSK, right?

Slightly off-topic, but I tried using Liquid to demodulate real-world streaming GMSK with no success. The best tool I could find is the newish symbol_sync block from GNU Radio. The math is above my head, but it would be great to see someone port the synchronization schemes to Liquid.

Absolutely! One of the first DSP algorithms I wrote for liquid was a symbol-timing recovery component using polyphase filter banks. It was originally designed for linear modulation schemes (like QPSK) but it works quite well for GMSK as well. There is a basic description for how to use it here.
If you also want to incorporate automatic gain control, carrier frequency/phase correction, timing recovery, and equalization, take a look at the symtrack object which integrates all of these together while ensuring stability across the various loops.

I'm definitely going to make more tests with sym_sync and sym_track.
What I've tried so far:
resampled IQ,FSK, ~5samples/symbol -> gmskdem (k=5, m=1)
This sort of works. I can decode the data, but is slowly goes out of sync, more errors
appear and then it goes back in sync and this happens periodically throught all the data.
resampled IQ,FSK, ~5samples/symbol -> symsync(k=5) -> gmskdem (k=5, m=1)
This doesn't work at all. Maybe I'm setting 'samples per symbol' wrong?
Maybe I should use more samples/symbol on input, like 30. Then output
of symsync with k=5 would be 6, as it's a decimating block, right?
I was also thinking about running 'freqdem' on the input signal to get
something resembling a square wave and run symsync/symtrack on this and then try
to decode the bitstream.
Any thoughts/advice is appreciated.

Yes, the gmsk demodulator alone does not perform symbol timing recovery. Your second approach of applying the symsync before gmskdem is ideal and should provide the best performance. I'll look into why that's not working for you.
But to help you move forward with your problem, I implemented a solution that you suggested last: run the FM demod before symbol timing recovery. See d1fd949 which adds the gmsk_tracking_example.c program. It uses 4 samples per symbol, resamples at a rate P/Q=203/200 which is 1.5% increased rate. Running it should provide an Octave script that gives a plot like the following:

I tried adjusting k=5 in the example, but it did take a little longer to converge (needed to increase the simulation duration a bit); alternatively increasing the symync loop filter bandwidth from 0.02 to 0.05 did the trick.
Let me know if you have any questions with this, and I hope this was helpful.

What exactly is the purpose of the matched filter? I've made some test. Here is the plot of my FM demodulated data:

After applying symsync:

With the filter in between it's worse:

The matched filter is intended to remove inter-symbol interference (ISI) from the transmitted signal. This is especially prevalent with GMSK which specifically adds ISI to reduce the signal's occupied bandwidth.
I will also point out that the simulation example I added does not include noise. You should probably apply a low-pass filter before the FM demod to remove any out-of-band noise. This noise would get amplified with the FM demod, so it's a good idea to remove it first.
I can't speak to why the matched filter would make your signal worse, unless it isn't truly matched to the transmitted signal's GMSK filter. In the example, I used a bandwidth-time factor (beta) value of 0.3. This might need to be adjusted to match the transmitted signal you're working with.
And as a last point, you can always add a dynamic equalizer after the symsync object to help remove any ISI that isn't removed by the matched filter.
To summarize, try the following:

Resample to ~4 samples per symbol (keeps processing reasonably low)
Apply a low-pass filter (e.g. 3rd-order butterworth) to remove out-of-band noise
Run the FM demodulator, setting the modulation index such that the output has an RMS value of about 1.0
Run a low-pass firfilt filter (ideally a matched filter to remove ISI) with a frequency cut-off of about 0.125 or maybe a little higher
Run through the symbol synchronizer with an output rate of 2 samples/symbol
Run the result through an eqlms_rrrf object with a decision-directed output (target +1/-1 values)

I understand this seems fairly complicated, but it does have many advantages in that the maximum amount of noise is always filtered out at each stage.

I've tried 4 samples per symbol - not much difference
Applied low-pass filter, but results still worse than without it. I'm applying the filter after initial resampling to 4 samples per symbol. Maybe I should apply the filter before resampling?
Made some attempts, but no significant improvement. My original waveform after demodulation is something like this.
This is what I'm currently doing.

Will experiment more with (4) and (6) later this week. I've made a more detailed write-up of what I'm doing here. The resulta after applying symbol synchronizer are good. Maybe it's the best that can be done with a signal like this?
Thanks for all the help!

Sorry for commenting on this closed issue, but is it possible to use symtrack for GMSK?
What would be the ideal filter type and modulation scheme to pass to symtrack_cccf_create?
Since there's no LIQUID_MODEM_GMSK (I guess since it's not linear), and https://liquidsdr.org/doc/symtrack/ says "modulation scheme (can be unknown)", I tried passing LIQUID_MODEM_UNKNOWN but symtrack_cccf_create fails here:
if (_ms == LIQUID_MODEM_UNKNOWN || _ms >= LIQUID_MODEM_NUM_SCHEMES)
    return liquid_error_config("symtrack_%s_create(), invalid modulation scheme", EXTENSION_FULL);


[Feature] iphone Style Basic Calculator

Description
Calculator with basic arithmetic operation with iPhone UI .
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[x] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Provide the supporting screenshot of your issue @codewithganeshhh

hey @Rakesh9100  reply me

We already have Basic Calculator, if you want you can make it responsive @codewithganeshhh


Fix the footer to the bottom !

Please work on the issue !

It's not happening on my end

Ok , it's happening when we click over the newly made component section

assign

Ok , it's happening when we click over the newly made component section

Any solutions for it ?

Ok , it's happening when we click over the newly made component section

Any solutions for it ?

ya solved
 we have to add this in footer.html at the top

Cool. Assigning you this!

Cool. Assigning you this!

ok, do merge my PR as it may also show old commits when I generate a new PR for this issue

Cool. Assigning you this!

ok, do merge my old PR as it may also show old commits when I generate a new PR for this issue

@siddhi-244 seems Okay right?
https://user-images.githubusercontent.com/92919173/217037178-a6ac98a7-5759-493c-8b4e-980cd6561a27.mp4

@siddhi-244 seems Okay right?
Embellish.Mozilla.Firefox.2023-02-06.22-36-29.mp4

Looks awesome

@siddhi-244 seems Okay right?
Embellish.Mozilla.Firefox.2023-02-06.22-36-29.mp4

Looks awesome

ok doing

The bug is fixed now, I checked the live website

There is alot of gap here which isn't looking good. If you can open a new issue and fix it

This line of code was needed and I added that via commit

There is alot of gap here which isn't looking good. If you can open a new issue and fix it

Ok so the previous one was fixed right. for that I will open a new issue .

There is alot of gap here which isn't looking good. If you can open a new issue and fix it

Ok so the previous one was fixed right. For this bug I will open a new issue after a while .

@SurajPratap10 any updates on this?


Update active-directory-azureadjoin-windows10-devices.md

Corrected misspelling of Administrators

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@tequilaeby : Thanks for your contribution to the Azure documentation! The author, @femila, has been notified to review your proposed change.

#sign-off


security denial for opening original working directory from start-edgex.sh

I didn't notice this denial earlier, but this happens when running strictly confined either on a 16.04 classic system or on a 18.04 classic system. I haven't tested a UC16 system yet. If you run the edgexfoundry.start-edgex script as sudo from say your (non-root) normal user's $HOME directory like follows:
cd $HOME
sudo edgexfoundry.start-edgex

You will see the following denial show up in the system logs:
Aug 27 13:29:27 xenial-classic kernel: audit: type=1400 audit(1535394567.710:466): apparmor="DENIED" operation="open" profile="snap.edgexfoundry.start-edgex" name="/home/ijohnson/" pid=4060 comm="start-config-se" requested_mask="r" denied_mask="r" fsuid=0 ouid=1000

I was able to fix this issue by adding a cd $SNAP_DATA to the top of the start-edgex.sh script, though I don't understand the underlying issue as the script triggering this (comm="start-config-se" is almost certainly the start-config-seed.sh script) doesn't use cd anywhere except here: https://github.com/edgexfoundry/edgex-go/blob/california/snap/bin/start-config-seed.sh#L33
which should never trigger a cd $HOME or anything like that...

Also note that this doesn't stop the snap from running at all, so it's a low priority bug


Allow wrapping on words in Floating Window

This is my config for devdocs
require("nvim-devdocs").setup({
	dir_path = vim.fn.stdpath("data") .. "/devdocs",
	telescope = {},
	telescope_alt = {
		layout_config = {
			width = 75,
		},
	},
	float_win = {
		relative = "editor",
		height = math.floor(vim.o.lines * 0.7),
		width = math.floor(vim.o.columns * 0.8),
		border = "rounded",
	},
	wrap = true,
	ensure_installed = {},
})

I am opening DevDocs in a floating window, but some lines contain more characters than it can fit on the screen, causing the text to be hidden. By enabling the wrap = true setting, I am able to view the complete text, but it is wrapping based on the characters.

I would like the text to wrap based on words. If I run set linebreak for the floating window, then it fixes the issues.

Is there any way to set this by default for the floating window? If there isn't any, can you please add some config parameter for it?

Thanks for the report, I didn't know that wrap worked thay way, it'd make sense to add linebreak by default if wrap is set to true. I'll change this right away.

I fixed it in aa884e3

Thanks a lot


MNT bump dependencies and update, including removing hacks required by 0.19.0

Tests also seems affected by https://github.com/IntelPython/numba-dpex/issues/906
Will remain WIP until then

The pipeline can only pass after the merge on main triggers refresh of the docker image with the bumps. Merging now, will re-trigger the pipelines once the docker image is pushed.


[Feat]: Increase performance of Website (Lazy Loading)

Is your feature request related to a problem? Please describe.
Inorder to reduce the initial loading time of website, lazy loading can be implemented.
Describe the solution you'd like
Render the components only when the user requests for it.
(initial loading time will get reduced)
Also, we can add styled react loaders .
Have you read the Code of Conduct?
Yes
I wish to contribute on this issue being part of gssoc'21

Hey @WaderManasi Thanks for proposing this idea. As this issue needs to be broken down in the smaller ones so I request you to to make the issue more detailed and work on it component by component(Maybe 2 components at a time)

Hi @Kajol-Kumari ,
Actually we need create just single component - Loader that will render the react loader
and implement the lazy loading code snippet in App.js file.

Hi @Kajol-Kumari ,
Actually we need create just single component - Loader that will render the react loader
and implement the lazy loading code snippet in App.js file.

I see. In angular this isn't the case. If it's a single person work, please go ahead :)

@WaderManasi Please update your progress

Can I work on this issue?

hey Is anyone working on this issue if not can please assign me this issue

@himanshu1221  i want to work in this issue under JWOC

I want to work for on this issue under jwoc , please assign me

I want to work for on this issue under jwoc , please assign me


Enterprise Bot Template deployment fails

Hi, If I try to run the command below I get an error:
msbot clone services --name "SB_EnterpriseBot" --luisAuthoringKey "YOUR_AUTHORING_KEY" --folder "DeploymentScripts\msbotClone" --location "westus"
This is the error message I get.
ERROR: Unable to provision MSA id automatically. Please pass them in as parameters and try again.
I already had this in the past when I wanted to create a bot using the az bot create command is this still not fixed?

@darrenj - can you help?

Yes. Happy to.  Also feel free to raise an issue on https://github.com/Microsoft/AI/

Here you go:
msbot clone services --name "SB_EnterpriseBot" --luisAuthoringKey "eea97ba4823748f683cff09a40a8664d" --folder "DeploymentScripts\msbotClone" --location "westeurope" --verbose
[msbot] The following services will be created by this operation:
Service                                 Location            SKU                 Resource Group
Azure App Site Plan                     westeurope          S1                  SB_EnterpriseBot
Azure AppInsights Service               West Europe         F0                  SB_EnterpriseBot
Azure Blob Storage Service              westeurope          Standard_LRS        SB_EnterpriseBot
Azure Bot Service Registration          Global                                  SB_EnterpriseBot
Azure CosmosDB Service                  westeurope          1 write region      SB_EnterpriseBot
Azure LUIS Cognitive Service            westeurope          S0                  SB_EnterpriseBot
Azure LUIS Cognitive Service            westeurope          S0                  SB_EnterpriseBot
(Dispatch)
Azure QnA Maker Service                 westus              S0                  SB_EnterpriseBot
Azure Search Service                    westeurope          Standard            SB_EnterpriseBot
Azure WebApp Service (Bot)              westeurope                              SB_EnterpriseBot
Azure WebApp Service (QnA)              westeurope                              SB_EnterpriseBot
Would you like to perform this operation? [y/n]y
[msbot] Checking az botservice version
[msbot] az -v
[msbot] Fetching subscription account
[msbot] az account show
[msbot] Creating resources in subscription: SBI-MVP-AzureSubscription c84063d6-342e-432b-9d82-62fdff45ae45
[msbot] Creating Azure group [SB_EnterpriseBot]
[msbot] az group create -g SB_EnterpriseBot -l westeurope
[msbot] Creating Azure Bot Service [SB_EnterpriseBot]
[msbot] az bot create -g SB_EnterpriseBot --name SB_EnterpriseBot --kind webapp --location westeurope --insights-location "West Europe"  --version v4
[msbot] [az bot] To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code C2Y6LR2HT to authenticate.
[msbot] az bot create -g SB_EnterpriseBot --name SB_EnterpriseBot --kind webapp --location westeurope --insights-location "West Europe"  --version v4 exit code: 1
WARNING: The behavior of this command has been altered by the following extension: botservice
WARNING: To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code C2Y6LR2HT to authenticate.
ERROR: Unable to provision MSA id automatically. Please pass them in as parameters and try again.
[msbot] Usage: msbot clone services [options]
allows you to clone all of the services a bot into a new azure resource group
Options:
-n, --name                             name of new bot
-f, --folder                         path to folder containing exported resources
-l, --location                     location to create the bot service in (westus, ...)
--luisAuthoringKey         authoring key from the appropriate luisAuthoringRegion for luis resources
--luisAuthoringRegion   (OPTIONAL) [westus|westeurope|australiaeast] authoring region to put LUIS models into (default is based on location)
--luisPublishRegion              (OPTIONAL) region to publish LUIS models to (default fallback is based location || luisAuthoringRegion)
--subscriptionId             (OPTIONAL) Azure subscriptionId to clone bot to, if not passed then current az account will be used
--insightsRegion             (OPTIONAL) region to create appInsights account in (default is based on location)
--groupName                       (OPTIONAL) groupName for cloned bot, if not passed then new bot name will be used for the new group
--sdkLanguage                   (OPTIONAL) language for bot [Csharp|Node] (Default:CSharp)
--sdkVersion                     (OPTIONAL) SDK version for bot [v3|v4] (Default:v4)
-q, --quiet                                  minimize output
--verbose                                    show commands
-f, --force                                  do not prompt for confirmation
-h, --help                                   output usage information
[msbot] NOTE: You did not complete clone process.
[msbot] To delete the group and resources run:
[msbot] az group delete -g SB_EnterpriseBot --no-wait

Thanks. Tracking in this bug https://github.com/Microsoft/botbuilder-tools/issues/596

@darrenj @cloudguy-pro - closing this issue as this is being tracked in the tools repo.

I've been struggling with the same issue ... and not resolved yet:

Originally I got "ERROR: Unable to provision MSA id automatically. Please pass them in as parameters and try again."
As suggested several places, I then added arguments for appId and appSecret.. No luck here as I got a generic error
I then created a brand new application manually and retrieved appId and appSecret and made the call using msbot clone services ; I got the following error
"az bot create ...( remaining of this message ) exit code: 1
WARNING: The behavior of this command has been altered by the following extension: botservice
ERROR: At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details."
I then tried "az group deployment operation list --name Name -g Name --query "[*].properties.statusMessage" and got the below message 👍
[
{
"error": {
"code": "InvalidBotData",
"message": "Bot is not valid. Errors: Id: Id is already in use.. "
}
},
... Any ideas on what to do next?
One more, to make sure I'm using the right appId, I opened the manifest and grabbed the same ID as it appears in the file..

Perhaps the Bot name you are using already exists somewhere? Try adding something more unique to the name, based on what you’ve shown below I suspect that’s the issue (which we’ll add better tests for)
From: Nikki Ahmadi<EMAIL_ADDRESS>Sent: 29 October 2018 05:55
To: MicrosoftDocs/bot-docs<EMAIL_ADDRESS>Cc: Darren Jefford<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/bot-docs] Enterprise Bot Template deployment fails (#298)
I've been struggling with the same issue ... and not resolved yet:

Originally I got "ERROR: Unable to provision MSA id automatically. Please pass them in as parameters and try again."
As suggested several places, I then added arguments for appId and appSecret.. No luck here as I got a generic error
I then created a brand new application manually and retrieved appId and appSecret and made the call using msbot clone services ; I got the following error
"az bot create ...( remaining of this message ) exit code: 1
WARNING: The behavior of this command has been altered by the following extension: botservice
ERROR: At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details."
I then tried "az group deployment operation list --name Name -g Name --query "[*].properties.statusMessage" and got the below message 👍
[
{
"error": {
"code": "InvalidBotData",
"message": "Bot is not valid. Errors: Id: Id is already in use.. "
}
},
... Any ideas on what to do next?
One more, to make sure I'm using the right appId, I opened the manifest and grabbed the same ID as it appears in the file..

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fbot-docs%2Fissues%2F298%23issuecomment-433796079&data=02|01|darrenj%40microsoft.com|5d3acef0f06d49e02dc908d63d630b76|72f988bf86f141af91ab2d7cd011db47|1|0|636763892933407592&sdata=DclkpOeomejeBViPaLLfs0p2O5ZQ8Etqmz5cHH3oiJw%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAf93oEsNGhaZWprbBqyPDxBdxfna_bDMks5uppgrgaJpZM4W4FRT&data=02|01|darrenj%40microsoft.com|5d3acef0f06d49e02dc908d63d630b76|72f988bf86f141af91ab2d7cd011db47|1|0|636763892933417605&sdata=zpxiSzevpEEkZ9l63LFwzNKFr%2F5ksdP8f0GD1mAqosE%3D&reserved=0.


support for mask all and unmask by selector and class APP-73422

should we have our jira app numbers in these commits? or some other kind of ref?

It probably doesn't make sense to have a jira number in them, what do they tend to have in their commit messages? issue numbers or nothing?

looks like issue numbers mostly


Testing github pages with astro

Codecov Report
Base: 20.36% // Head: 20.36% // No change to project coverage :thumbsup:

Coverage data is based on head (2eae22b) compared to base (e269d44).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #37   +/-   ##
=========================================
  Coverage     20.36%   20.36%
  Complexity      199      199
=========================================
  Files            36       36
  Lines           712      712
=========================================
  Hits            145      145
  Misses          567      567

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

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


Add Service Issues with Baby Buddy 2.0.0 Upgrade

Just upgraded BB to 2.0.0 which caused an issue of duplicating all devices. I removed and reintegrated the connection between Baby_buddy_homeassistant (version 2.7.0) and BabyBuddy and am able to create timers again. I am able to log diaper changes but am unable to log food, receiving the below message.
This error originated from a custom integration.

Logger: custom_components.babybuddy.client
Source: custom_components/babybuddy/client.py:75
Integration: Baby Buddy (documentation, issues)
First occurred: 1:10:31 PM (10 occurrences)
Last logged: 1:29:06 PM

Could not create timers. error: {'start': ['Date/time can not be in the future.']}. Please upgrade to babybuddy v1.11.0. In the meantime, attempting to use 'now()'...
Could not create feedings. error: {'start': ['Date/time can not be in the future.']}. Please upgrade to babybuddy v1.11.0. In the meantime, attempting to use 'now()'...

I tried adding the 'start' offset from the below issue, but without any luck.
https://github.com/jcgoette/baby_buddy_homeassistant/issues/82#issuecomment-1416531189
I also checked the timezones, where Baby Buddy is America/New_York and HAOSS is America/New_York as well.
I created a script using both the entity and device options of the Baby Buddy integration without success.
alias: TESTING_Script
sequence:
  - service: babybuddy.add_feeding
    data:
      type: Solid food
      method: Self fed
      amount: 4
      notes: Using Choose Entity
    target:
      entity_id: switch.editedfirstname_editedlastname_timer
  - service: babybuddy.add_feeding
    data:
      type: Solid food
      method: Self fed
      amount: 8
      notes: Using Choose Device
    target:
      device_id: editedID
mode: single

So, deleting the sequence of the script and recreating did the trick. Not sure modifying the script that was used for the pre-2.0 Baby Buddy didn't work.


[configuration cache] file inputs are not recognized sometimes

Expected Behavior
File inputs for the CC are properly recognized.
Current Behavior
Some file inputs are missed.
Steps to Reproduce
Create a new basic project using init.
Add a gradle.properties where you enable the configuration cache.
Put this in the build script
file("gradle.properties").readText()
val foo by tasks.registering {
    val bar = provider {
        file("gradlew").readLines().first()
    }
    inputs.property("input", bar)
    doLast {
        println(bar.get())
    }
}

Now execute the build and have a look at the configuration cache report.
It properly recognized gradle.properties as file input,
but gradlew is not recognized as file input.
If you change gradlew CC entry is reused.
If you change gradle.properties, CC entry is not reused as inputs changed.
If you replace inputs.property("input", bar) by inputs.property("input", bar.get()), also gradlew is recognized as file input.
Gradle version
8.1.1

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team and is prioritized by them.

Thank you for the report. This issue was addressed in #23249, fix will be shipped in Gradle 8.4.


Add tag descriptions to OSS and EE admin APIs

adds tag descriptions to each set of endpoints.

Have to update something don't merge.


End point to calculate Loan Affordability for any individual

What bug or feature you wants to report.
I am a GSSOC'23 Contributor
I want to create a endpoint Loan Affordability:
A loan affordability endpoint helps users to determine how much they can afford to borrow based on their income, interest rate and expenses. This feature help individuals to take right loan.
@ighoshsubho I love to work on this, kindly assign this to me.

Sure go ahead @Shivamjha12 :)


Make EmbeddedEventLoop detect concurrent access

Motivation:
The EmbeddedEventLoop is meant to only be access by one thread at a time.
Modification:
Add concurrent access checks to the EmbeddedEventLoop, a bit like ConcurrentModificationException in the Java collections. The EmbeddedEventLoop and EmbeddedChannel are meant to be used by only one thread at any given time. By having these checks, we are more likely to identify test isolation failures involving the embedded channels.
Result:
We are now much more likely to get a meaningful exception thrown when an EmbeddedEventLoop is accessed by multiple threads.

Draft until #12870 get merged.


Write a limited fuzz test stressing combinations of index operations

Pull Request
What does this PR do?
Add a fuzz test that applies multiple operations (document import, document deletion, setting update, clear documents) to an index and verify that none of them crash.

Note: this PR will stay in a draft state for a long time probably, but I'd like to keep it alive as a reminder that fuzz-testing indexing is possible and not that complicated. We can build on it in the future.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49400

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49515

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/54414

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108510


Improve shebang for python

The shebang currently has an absolute path to python.
According to https://www.python.org/dev/peps/pep-0394/#recommendation the recommended way is:
/bin/env python3
PR incoming.

@dragetd in next version all PEP rules will handled properly. so i 'm closing the PR and issue both thanks for that anyway.

Ah, okay. Did not find any branch or info about the planned updates. Looking forward to the next version! :-)


EE3 version - setup installer file

Jesus, why hasn't this been merged? This is almost a year old now.


Package size optimizations

Extra:

Import react-native-elements directly so not all package is imported
Use webpack alias to make sure BN.js is imported only once

Considerations:

PR Title Convention: (Feature|Bug|Chore) Task Name
Be as descriptive as possible, assisting reviewers as much as possible.
If frontend, paste screenshots that display the UI changes.
If there is interactivity, paste a gif showing the feature.

@sirpy can you solve conflicts?

@fernandomg done

@sirpy there are more conflicts related to FR files


adding metafield type documentation for android

Updates client documentation for Android SDK.
Metafields now support Boolean, String, and Numbers

Hi @abhi-ada!
Just wondering if this would be backwards compatible. Should the new implementation be differentiated using the version?
eg
>=1.2.0 (for example)
...add new way of adding metafields

Older versions
...old way of adding metafields

thanks for confirming @abhi-ada!
if it's backwards compatible then should be good to go

@wole-ada I was looking for a second opinion on that so glad you asked. 👍


notify couch_index_processes on all shards when ddoc updated

Before when a design doc is updated/deleted, only one couch_index process
was notified - the one which shard contained a design doc.
couch_index processes from other shards still continued to exist,
and indexing activities for these processes were still be going on.
The patch notifies couch_index_processes on all shards
BugzID: 85773

Overview

Testing recommendations

JIRA issue number

Related Pull Requests

Checklist

[ ] Code is written and works correctly;
[ ] Changes are covered by tests;
[ ] Documentation reflects the changes;
[ ] I will not forget to update rebar.config.script
with the correct commit hash once this PR get merged.

Please remove BugzID: 85773. and provide JIRA reference instead.

+1 to the logic. Just a few minor style issues to clean up and then this should be good to go.


Feature/addon 34691 modify checkpoint log path

Updated the source names
Resolved bug on host assignment
Resolved bug for time assignment

:tada: This PR is included in version 1.60.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Where is map._initPathRoot()  in development version of leaflet.

I was using  Leaflet 0.7.3 and D3 to actually render some points (circle) or hexbins ( http://projects.delimited.io/experiments/hexbins/starbucks.html)  on leaflet map.  It was working fine then I starting facing problem with zooming and loading of tiles. Some tiles just does not loading when user zoom in or out.   I tried to work with the development  version  (Leaflet 0.8 ) but now all my svg elements are not displayed.  It also says that map._initPathRoot() the function I used to initialize svg pane the does not exist any more.
Here is the code I use to initialize the SVG panes
	/* Initialize the SVG layer */
map._initPathRoot()
/* We simply pick up the SVG from the map object */
var svg = d3.select("#map").select("svg"),
g = svg.append("g");

I also tried this code to get the SVG pane but still all my elements are not displayed.
var svg = d3.select(map.getPanes().overlayPane).append("svg"),
g = svg.append("g").attr("class", "leaflet-zoom-hide");

I have to go back to  Leaflet 0.7.3  to see all my points and I still have some problem with the zooming.

Try doing this instead: L.svg().addTo(map).


Sealed exception types in XamlMath.Shared\Exceptions

Also, modernized namespace references (they are now file-scoped).
P.S. I don't know what is causing the build errors. Does sealing exception classes do anything that may cause them?

P.S. I don't know what is causing the build errors. Does sealing exception classes do anything that may cause them?

By sealing the classes, you've formally broken code compatibility: before that, the user code could inherit from these classes, but now it cannot.
I sincerely hope nobody actually inherited from them, so I believe we can safely merge this without bumping the major version 😅


Is that possible to concat multiple HTML to make one PDF ?

Or concat some PDFs in juste one ?!

no. please use another tool for that.

@fouzko Yes, it's possible to print different pages in one pdf with css - depending on how your html is setup.
With a structure like this:

...

...

...anything contained within those page divs would be different pages in the PDF. I know this isn't exactly what you're asking, but the closest way to simulate "multiple" pdfs.

@scurker
Helped a lot.
Cheers.

We really should put together some best practices. Abd write down all the limitations. Phantomjs and also chromium have many limitations which aren’t that obvious.

Mine are still seeming to ignore these break rules. Would the fact I'm using A4 size make a difference to how this would behave?

Thanks It did help!

Thanks @scurker
 helped my a lot


how to get access non coding version i am already sponsoring

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
how to get access non coding version i am already sponsoring

sorry  I would set up a contact info at readme

just sent email thanks


Handle completion of Choices in Tuple Options

Completion of Choice option values was recently added, in https://github.com/pallets/click/pull/681. However, this does not cover Choice values within Tuple options. This commit adds similar handling so completion will cover these option values as well.

Completion has changed significantly since this PR was posted. Would you or @stopthatcow review this to see if it's still needed and can be rebased?

Thanks for working on this. We're rewriting the whole completion system, if this is still and issue after that we'll need to find a new solution that fits with the new system. See #1484 and #1622.


When detecting changes may need to consider the live cluster state

When detecting new changes need to be applied, now we look at the (persisted) last commit hash from the status of GitSync CR and compare with the latest commit hash from the remote repository.  There are some major issues need to watch out for, 1) what happens if the user did a force push to the repo and the last commit hash gets wiped out 2) what if the user accidentally update the status fields of GitSync CR, etc.
Therefore, extra mechanism is needed to reliably detect changes happen.

Fleshed out the algorithm for detecting changes and summarized here

Is reusing the code/algorithm from Argo CD a possibility?

We could, will weight and compare with their algorithm as well. Thanks!

Decided to switch to ArgoCD model and use gitops-engine library for the reason documented here . On top of that summarize how we can utilize the gitops-engine here

Close the issue, as we have the plan and the actual task are tracked in other issues.


feat(testing-helpers): add waitUntil helper

Async rendering is quite a hassle, this helper helps manage this declaratively in tests and demos. I have a version of this that I use a lot in my own code.

Oooh, we have a version of this in our tools that I have on a list to contribute back, BUT this version is so much nicer. The options... the message... very exciting!

suggestion from @daKmoR: add a warning/note about when to use it


[Bug] Image sampling artifacts when not rendering in physical pixels

How to reproduce:

run the supplied ui example
toggle scale with the / key

1:1 physical pixel scale:

Default DPI-aware scaling:

This issue is specifically for the aliasing seen in the middle of the image, e.g. the bird's chest. There is a different alpha bug that causes aliasing at the border of the circle that applies to any image with transparency.

Thanks for filing the issue. I'm aware of the bug but I don't really know how to fix it. I'm quite a noob in rendering, so I'd appreciate any help (either a PR or some hints on how to fix it).

I'm not an expert either. 🙂 We might need to poke some people in the bevy/egui communities. My best guess is it has something to do with how the texture is scaled/sampled to the final render target.

@mvlabat I found the issue. It appears the default SamplerDescriptor uses Nearest instead of Linear. I was able to test a fix by adding the following to the ui example in this repo:
    if let Some(sampler) = &mut assets
        .get_mut(&images.bevy_icon)
        .map(|s| &mut s.sampler_descriptor)
    {
        sampler.mag_filter = FilterMode::Linear;
        sampler.min_filter = FilterMode::Linear;
    }

Linear:

Nearest:

Not quite sure what the best solution path is. Maybe the default filtering mode in bevy - where this originates from - should be Linear?

I believe we can close this now, since the new default made it into Bevy. Thank you for looking into this and making the Bevy PR!


Export the 'mail.smtp.ssl.enable' in the batch.

Which module is your feature request related to?
Batch/.../applicationContext-batch-sender.xml
Is your feature request related to a problem?
Cannot use Gmail's SMTP easily cause by default, the sender does not enabled to use 'mail.smtp.ssl.enable' configuration.
Cause a lot of users in pinpoint might using gmail to send mail via pinpoint's batch, this feature will be useful to lots of develpers!
Describe the solution you'd like

Add the below property between line number 30 and 31 in 'pinpoint/batch/src/main/resources/applicationContext-batch-sender.xml'
<prop key="mail.smtp.ssl.enable>${alarm.mail.smtp.ssl.enable:false}</prop>
Update the configuration guide of pinpoint batch with 'Setting the value alarm.mail.smtp.ssl.enable if you needed'.

Describe alternatives you've considered
Nothing for now
Additional context
Thanks for sharing the most useful APM tool for me.
Me and my team always thanks to your team.
추가적으로 문의사항이 있으시다면<EMAIL_ADDRESS>연락주시면 감사하겠습니다!

I completely agree with your proposal.
Coud you create a PR?

@emeroad Sure! I will create a PR for this Issue.


Error: Cannot find module 'cypress-plugin-tab' from '/e2e/cypress/support'

I get this error when I try:
$ docker run -it -v $PWD:/e2e -w /e2e -e CYPRESS_baseUrl=http://XXX.444.222.222/IntegrationTest cypress/included:3.5.0
Running:  DAIv7Base/001 loginLogoutFirst.js                                              (1 of 95)
Oops...we found an error preparing this test file:
/e2e/cypress/support/index.js
The error was:
Error: Cannot find module 'cypress-plugin-tab' from '/e2e/cypress/support'
This occurred while Cypress was compiling and bundling your test code. This is usually caused by:

A missing file or dependency
A syntax error in the file or one of its dependencies

Fix the error in your code and re-run your tests.

however I think that the goal of the docker image should actually be not having to npm install anything... so any idea what I can do?

my bad... I had an index file sitting in the support folder


Add support for the Graphite exporter

https://github.com/census-ecosystem/opencensus-go-exporter-graphite

I would recommend watching "Modern Times" instead


Add Channel: M1 Hungary

Can you add the livestream for M1 Hungary to the list?
https://www.youtube.com/watch?v=5zcjLN_Nejs

Not live at the moment


fix(deps): update module gorm.io/gorm to v1.25.3

This PR contains the following updates:

Package
Type
Update
Change

gorm.io/gorm
require
patch
v1.25.1 -> v1.25.3

Release Notes

go-gorm/gorm
v1.25.3
Compare Source
v1.25.2
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: backend-go/go.sum
Command failed: docker run --rm --name=renovate_sidecar --label=renovate_child -v "/tmp/renovate/repos/github/bcgov/quickstart-openshift":"/tmp/renovate/repos/github/bcgov/quickstart-openshift" -v "/tmp/renovate/cache":"/tmp/renovate/cache" -e GOPATH -e GOSUMDB -e GOFLAGS -e CGO_ENABLED -e GIT_CONFIG_KEY_0 -e GIT_CONFIG_VALUE_0 -e GIT_CONFIG_KEY_1 -e GIT_CONFIG_VALUE_1 -e GIT_CONFIG_KEY_2 -e GIT_CONFIG_VALUE_2 -e GIT_CONFIG_COUNT -e BUILDPACK_CACHE_DIR -e CONTAINERBASE_CACHE_DIR -w "/tmp/renovate/repos/github/bcgov/quickstart-openshift/backend-go" docker.io/renovate/sidecar bash -l -c "install-tool golang 1.21.0 && go get -d -t ./..."
curl: (22) The requested URL returned error: 404
Download failed: https://dl.google.com/go/go1.21.linux-amd64.tar.gz


0174- Dungeon Game

Description of the Problem
The demons had captured the princess (P) and imprisoned her in the bottom-right corner of a dungeon. The dungeon consists of M x N rooms laid out in a 2D grid. Our valiant knight (K) was initially positioned in the top-left room and must fight his way through the dungeon to rescue the princess.
The knight has an initial health point represented by a positive integer. If at any point his health point drops to 0 or below, he dies immediately.
Some of the rooms are guarded by demons, so the knight loses health (negative integers) upon entering these rooms; other rooms are either empty (0's) or contain magic orbs that increase the knight's health (positive integers).
In order to reach the princess as quickly as possible, the knight decides to move only rightward or downward in each step.
Write a function to determine the knight's minimum initial health so that he is able to rescue the princess.
For example, given the dungeon below, the initial health of the knight must be at least 7 if he follows the optimal path RIGHT-> RIGHT -> DOWN -> DOWN.

Note:

The knight's health has no upper bound.
Any room can contain threats or power-ups, even the first room the knight enters and the bottom-right room where the princess is imprisoned.

Code
class Solution:
    def calculateMinimumHP(self, dungeon: List[List[int]]) -> int:

Link To The LeetCode Problem
LeetCode

I would like to work on this


Update datedropper.js

Case-sensitive search pattern for distinguishing 'M' and 'm' to display shortened month.

Can you close this one, more changes were needed.


Relaxed edition name restriction to allow dashes in backend

fixes #464

Codecov Report

Merging #510 (c0bb78a) into develop (0e6efb9) will not change coverage.
The diff coverage is 100.00000%.

@@              Coverage Diff              @@
##             develop        #510   +/-   ##
=============================================
  Coverage   96.43729%   96.43729%
=============================================
  Files             82          82
  Lines           2049        2049
=============================================
  Hits            1976        1976
  Misses            73          73

Impacted Files
Coverage Δ

backend/src/app/schemas/validators.py
100.00000% <100.00000%> (ø)


Can't drag Curve points

Godot version
d9daf38
System information
Windows 10 x64
Issue description

The point does single movement and cancels drag.
Steps to reproduce

Open anything that exposes Curve in the inspector (e.g. @export var curve: Curve)
Try to move any point on the curve

Minimal reproduction project
No response

Could this be a regression from #58023?


DeprecationWarnings and ResourceWarnings with Python >=3.2

(Original issue 980 created by arfrever on 2014-05-02T12:42:20.426667+00:00)
Test suite of Pygments trunk triggers 2 DeprecationWarnings and 306 ResourceWarnings with Python 3.3.
Test suite of Pygments trunk triggers 3 DeprecationWarnings and 306 ResourceWarnings with Python 3.4.
(1 additional DeprecationWarning with Python 3.4 is cause by 'import nose'.)
$ python3.4 -Wd tests/run.py

(Original comment by birkenfeld on 2014-10-07T12:11:22.952320+00:00)
Closes #980: fix DeprecationWarnings (mostly due to files closed by del) on Py3.
Also fix a bunch of other uses of open() to use the with statement.
→ <>


Styles

Hi,
I have installed the library using npm install --save-dev ember-idx-forms but it seems bootstrap styles are not included... Do I need to perform any additional actions?

have you included bootstrap as import in your Brocfile.js?

@apokinsocha No I didn't. Shouldn't the lib include it?


Running styleguide from site's local server

Hi,
I've reread this issue and @varya 's advice on using the appRoot parameter when running the styleguide from the site's local server (rather than running it on its own server, which does work for me) but localserver:3000/styleguide is loading a blank page. I'm running my server through browsersync.
The console shows that that the browser is looking for the styleguide files at the root, rather than in the styleguide folder:
GET http://localhost:3000/js/vendor.js
(index):26 GET http://localhost:3000/styleguide-app.css addCssLink @ (index):26(anonymous
function) @ (index):35(anonymous function) @ (index):36
(index):59 GET http://localhost:3000/js/app.js
(index):26 GET http://localhost:3000/styleguide_at_rules.css addCssLink @ (index):26(anonymous
function) @ (index):34(anonymous function) @ (index):36
(index):60 GET http://localhost:3000/socket.io/socket.io.js 404 (Not Found)

My (relevant) folder structure:
dist
--website
----styleguide
src
gulpfile.js
STYLEGUIDE.md

My gulp file:
var styleSourcePath = 'src/assets/scss';
var scssRoot = styleSourcePath + '/master.scss';
var styleguidePath = 'dist/website/styleguide';

gulp.task('styleguide:generate', function() {
	return gulp.src([
		styleSourcePath + '/components/*.scss',
		styleSourcePath + '/helpers/*.scss',
		styleSourcePath + '/layout/*.scss',
		styleSourcePath + '/print/*.scss',
		styleSourcePath + '/sections/*.scss',
		styleSourcePath + '/vendor/*.scss',
	])
	.pipe(styleguide.generate({
		title: 'Styleguide',
		rootPath: styleguidePath,
		appRoot: '/styleguide',
		overviewPath: 'STYLEGUIDE.md',
		customColors: 'src/assets/scss/vendor/_styleguide-overrides.css'
	}))
	.pipe(gulp.dest(styleguidePath));
});

gulp.task('styleguide:applystyles', function() {
	return gulp.src(scssRoot)
	.pipe($.sass({
		errLogToConsole: true
	}))
	.pipe(styleguide.applyStyles())
	.pipe(gulp.dest(styleguidePath));
});

gulp.task('watchstyles', ['styleguide'], function() {
	gulp.watch(['*.scss'], ['styleguide']);
});

Where have I gone wrong?

try this
appRoot: '/dist/website/styleguide',

I ended up deleting the existing styleguide folder and starting almost from scratch (though the gulp task above stayed the same), and it worked. Still not sure what the issue was.
One thing I did notice is that the appRoot needs to be /styleguide not just styleguide. It's not very clear on the README.


Chore: Bump DB-Sync to version sancho-5-0-0

Why

GovTool is currently on DB-Sync version sancho-4-3-0 with Node version 8.11.0
DBSync team have released two newer versions since with sancho-5-0-0 (node 8.11.0 compatible) and sancho-5.1.0 (node 8.12.0 compatible).
Soon (maybe late next week), SanchoNet will be respun to Node 8.12.0 which will force us to upgrade to sancho-5.1.0

To reduce the complexity of this at time of re-spin, I suggest we update GovTool to sancho-5-0-0 sooner
As the changes from 4 to 5 are significant schema changes, if we focus on the needed backend changes now, we wont be in such a rush at time of next respin.

What

Bump infra configure to use DBSync sancho-5-0-0

How

Bump infra configure to use DBSync sancho-5-0-0

@Ryun1 Do you feel #1538 is a duplicate of this task?

We should now bump straight to DBSync 5.1.0, as this is compatible with Node 9.0.0 which is what SanchoNet is running on

@Ryun1 Do you feel https://github.com/IntersectMBO/govtool/issues/1538 is a duplicate of this task?

@bosko-m
No! we should probably close this ticket and just track the issue on https://github.com/IntersectMBO/govtool/issues/1538 now, as upgrading straight to 5.1.0 will be a better option

I am closing the issue then. We'll bump the db-sync straight to 5.1.0


Wallet headers dont sync and when they sync i loose my wallet coins

I will appreciate the assistance of whoever can help, please.
I upgraded from htmlcoin-<IP_ADDRESS>-win32 (beta) to htmlcoin-2.5.0-win32 and since then I have had the following issues.

The wallet will not sync with the network ... it is stuck at 91.4% for over a week now.

no connection to peers on the HTML network

I have one the following to help the situation:

I unbanned all the banned peers and there was connection to over 10 peers on the network but the headers still don't sync past 91.4% although I still had the accurate balance of my wallet.

the last block I received was 1008527 so I ran getblockhash 1008527 and the result was ee75d30ed88f67270a415a3112705088e8c4c7f00c6631d8629975a9d6ffc2c2

I took backups weekly of my wallet before the upgrade so I installed the new wallet and then removed everything in the appdata/roaming/html folder except the wallet.dat file and the peers' file, and then restarted the wallet. the wallet syncs and connects to oven 10 peers but the balance of my wallet has been at 0 since then.

I deleted the entire content of the appdata/roaming/html folder and then got the wallet to sync afresh, then I restored the wallet from File->restore wallet and the wallet balance was still 0 even though there was a connection to peers on the network and the headers synced up correctly.

not sure what else to do from here ... I am wondering if htmlcoin-2.5.0-win32 is the most stable version? or if there is something I am doing wrong.

Manually restore your wallet.dat, copy it to the HTMLCOIN data directory, then start the client. Do you know an address for any of your coins? You can check the balance of that address on the explorer to make sure it has been seen on the network. Were your coins from solo mining and staking or where some of them purchased from an exchange?
https://chainz.cryptoid.info/html/

thank you @Bushstar for reaching out.
so from the complaint I made above, I have already tried restoring the wallet.dat manually and I can't see any of my coins after doing that.
when I check the address of the last completed transaction, I see the initial amount which I staked in 2018. Everything mined since 2018 till now does not show.
and, I have made all my coins since 2018 from solo mining and staking. the amount I staked in 2018 was from a purchase made online.

There have been several hard forks since HTMLCOIN <IP_ADDRESS>, it sounds like you were on the pre-fork branch which is not part of the main chain.

hmmm .. does this mean bad news?

hmmm .. does this mean bad news?

If you were mining and staking easily, then it was too good to be true and never was. Difficulty ramped up after GPU miner, so it'd be extremely rare for an in-wallet miner via Proof of Work to get create an accepted candidate block on the main chain.

hmmm .. does this mean bad news?

Yes. HTMLCOIN 2.0 has not been on the same chain as the rest of the network for years now.

Okay, realistically, what is my best bet at this point?

Coins staked on 2.0 since the hard fork in 2018 are lost. Start staking with 2.5 and keep an eye on new releases.

that's a hard blow.
okay, when I search for my address online, I find the amount which I staked in 2018 as my current balance but I can't see it in my wallet. How do I resolve this piece?

Start the wallet with the following in htmlcoin.conf, this should be a plain text file in your data directory, create it if it does not exist. Make sure that the file extension is .conf and not .txt.
zapwallettxes=1
That will make the wallet delete all wallet transactions and then will scan the entire Blockchain for any transactions that belong to the private keys held in your wallet.dat. The splash screen should give some indication that it is rescanning the Blockchain, normally rescanning followed by a percentage complete. Remove that line from the conf file afterwards.

Start the wallet with the following in htmlcoin.conf, this should be a plain text file in your data directory,
create it if it does not exist. Make sure that the file extension is .conf and not .txt.

Will this really work? If he's already on the wrong chain, wouldn't he need to erase the chain and start over (e.g. delete everything in %appdata%\HTMLCOIN except wallet.dat and htmlcoin.conf) and first from scratch? Wouldn't zapwallettxes=1 just rescan existing blocks, but not put him on the right chain? Maybe I misunderstand how zapwallettxes=1 works on HTMLCOIN.

@sunk818 he's done that already. Deleted everything in the data directory and synced on 2.5. I'm hoping that the coins are locked in staking transactions, it's something worth trying. If there's nothing there we'll see if the wallet can dump the private key for the address they believe is in it.

@sunk818 he's done that already. Deleted everything in the data directory and synced on 2.5. I'm hoping that the coins are locked in staking transactions, it's something worth trying. If there's nothing there we'll see if the wallet can dump the private key for the address they believe is in it.

Right you are. zapwallettxes=1 has worked for me too with missing txes, so hopefully it'll work for @silasechegini

zapwallettxes=1

thanks again .... are you saying i should delete all the files in the data directory (except my wallet) before performing this process or I should just go ahead with the process leaving my data directory as is?

Leave the data directory as is.

so, I did as you had said and I was able to get back to the amount I originally staked in 2018.
thanks @Bushstar @sunk818  for all the help. Appreciated.


pyproject-toml functionality

Extend functionality to check for and work with pyproject-toml format.

Resources:

pep518
poetry pyproject.toml documentation
pip pyproject.toml documentation

Issues:

No in-built python capability for machine parsing .toml files?  Packages are available, but I don't want to add more dependencies to this.  The pep518 documentation seems to imply that it's easily parsable, but I don't see the specific implementation of that.
The dependency field seems a bit hard to manually parse.  That being said, if it were machine-parsable it would be quite useful.  Interesting that it seems to also meld a few other CiteAs-relevant formats in there as well.


python3Packages.cxxheaderparser: 1.3.4 -> 1.4.0

Description of changes
https://github.com/robotpy/cxxheaderparser/compare/1.3.4...1.4.0
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 345488 run on x86_64-linux 1

4 packages built:

python311Packages.cxxheaderparser
python311Packages.cxxheaderparser.dist
python312Packages.cxxheaderparser
python312Packages.cxxheaderparser.dist


IGAPP-809: Languages json not existing

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
This PR probably does not fix anything and is rather meant to gather additional information. This bug should not be critical as I tried and afaics all errors logged are caught and handled correctly.
The error that the languages.json does not exist always occurs after another error during fetching, most of the time after the following error: Failed to load the request for the languages endpoint. Body not allowed for GET or HEAD requests (I also found one occurrence of the same error for the cities endpoint, so it should not be the fault of the language endpoint). Nearly always it also seems like loadCityContent (or loadCities for the one occurrence) are called and run twice.
The whole thing is really weird, first because of the iniital GET/Body request thingy, second because the following languages.json does not exist error only occurs after this error (and not e.g. after regular fetch errors) and third because sagas are run twice even though we are using takeLatest which should cancel any previous sagas. I suspect that the issue is somewhere in executing loadLanguages twice though.

Finally found the issue :hear_no_evil: `merge´ also alters the first parameter so whenever a post request with body was made all following get requests had the same body set which lead to errors oc


Add actions-toolkit

Hi again! This PR adds a link to my GitHub Actions Toolkit library, a set of handy methods for building actions in Node. I'm using it over in https://github.com/JasonEtco/create-an-issue if you're curious to see it in action!
I think all of the other resources under Community are Actions - a new category (like Tooling) might be helpful - let me know what you think!

We definitely do have to start splitting them up soon. I'm not sure about tooling though because technically any action is like tooling, that might be confusing for people. I feel like once a few more entries come in the categories might emerge naturally. Open to hearing other suggestions, too. Thanks!

I'm not sure about tooling though because technically any action is like tooling, that might be confusing for people.

Totally - I'll give it some thought and follow along for any updates. Thanks 🎉 ✨


we should use the browser`s location apis to zoom in into the general position of the vehicle in the extension map

Any idea if this works when the browser is offline (e.g. on a remote deployment)?
It's probably still useful either way, but extra useful if it remembers the last valid location or something.
There may be issues with people using VPNs, but they can just deal with that by moving the map as relevant - there's nothing we can meaningfully do about that unless their VPN isn't working properly 😅

Any idea if this works when the browser is offline (e.g. on a remote deployment)? It's probably still useful either way, but extra useful if it remembers the last valid location or something.
There may be issues with people using VPNs, but they can just deal with that by moving the map as relevant - there's nothing we can meaningfully do about that unless their VPN isn't working properly 😅

a bigger issue right now is that that API requires secure context =|

… that API requires secure context =|

Is that a general BlueOS issue, or one for this extension, or both?

general BlueOS issue


Add fieldReset parameters

Fix #9 Can't upload the same file twice

@tb thanks. merging.

@jkuri thanks! Will you update npm package?
BTW What could greatly improve people contributions would be to include demo app in this repository and add task or script to publish it to gh-pages branch.

woops, forgot to publish. just did right now. I think demo app shouldn't be included in the source of this module because it will increase the install time.

Dont fixed to me :/

Hello guys, i'm trying to repeat upload with same files but it does not work for me could you help.
Thank you in advance.
it is my options.
ngOnInit() {
    let opt = {
      url: `${window.location.protocol}//${window.location.host}/api/UploadImage`,
      filterExtensions: true,
      allowedExtensions: ['jpg', 'png', 'jpeg'],
      data: { id: 0 },
      autoUpload: false,
      fieldName: this.inputName,
      fieldReset: true,
      maxUploads: 2,
      method: 'POST',
      previewUrl: true,
      withCredentials: false
    };
    this.options = new NgUploaderOptions(opt);
    this.inputUploadEvents = new EventEmitter<string>();
  }

  startUpload(id: number) {
    this.options.data.id = id;
    return this.inputUploadEvents.emit('startUpload');
  }


Falco FOSS security monitor, lab with examples

Falco is a free and open source behavioral activity monitor designed to detect anomalous activity in your containers. This lab has 6 fully reproducible Docker examples and pointers to the full documentation.

Docker Labs are about using Docker tools. While we do mention non-Docker tools, they're in the context of learning about Docker. Since Falco is a product of another company, we would prefer not to have labs just about using Falco. We would accept a monitoring lab that used Falco in the context of learning monitoring Docker and using other tools.


Deploy 093 - Remove frxETH Strategies

Proposal:<PHONE_NUMBER>3749395103495090988298299772264220800986246762669183539782770414789
Proposal Tx: 0xa8e5e480edfda75f1c12ef519d9fe6fa68efb5a6ef14386743a6deb5b72556ac

To do (Before execution):

Have the strategist donate a small amount of frxETH to the Vault and call depositToStrategy. Otherwise, the proposal execution will fail (since the current frxETH strategy implementation doesn't have a zero check). Done, thanks to @sparrowDom

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[ ] Code reviewed by 2 reviewers.
[ ] Copy & paste code review security checklist below this checklist.
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)


Error Messages Outputting when Changing Settings

Error is being thrown when changing settings, this is not affecting the code run at all as the program is exited at the click of a button in the GUI. Settings so far have not been affected either meaning that the error is irrational.

Run the exe file provided in new release dev version (posting soon)
Say “settings” into the microphone
When the functions window opens, if you input pronouns or click the button to change voice gender a certain error message is thrown

No errors should be thrown as nothing is happening in the program.
Windows 10, EXE file compiled in VS Code
NOTE: This error does not have any actual implication on the program, it will be investigated going into the release.

Error is a NoneType error, the error has been minimized and should not appear due to new try/except code


Add button to journal image pages

suggested in:  https://discord.com/channels/880968862240239708/880969174661353484/1255401014643654777

done in v1.8.0


Prop To Enable Selection Of A Single Date Instead Of Two

@datepicker-react/hooks or @datepicker-react/styled version
e.g<EMAIL_ADDRESS>Describe the bug
Not so much a bug as it would be a great feature.
Source code (including props configuration)
Steps to reproduce the behavior:
<DateRangeInput
  onDatesChange={data => dispatch({type: 'dateChange', payload: data})}
  onFocusChange={focusedInput => dispatch({type: 'focusChange', payload: focusedInput})}
  startDate={state.startDate} // Date or null
  endDate={state.endDate} // Date or null
  focusedInput={state.focusedInput} // START_DATE, END_DATE or null
/>

If you have custom methods that you are passing into a @datepicker-react component / hooks, please include the source for those as well.
Screenshots/Gifs
If applicable, add screenshots or gifs to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Is the issue reproducible in Storybook?
Please link to the relevant storybook example
Additional context
Add any other context about the problem here.

Made this issue before digging into the docs thoroughly, my mistake!


Provide better (runtime) feedback / errors when failing to expect

expect can error intentionally and it will automatically trace something;  though the current error message is a bit cryptic. Some improvements ideas:

[ ]  Put the original source location in the trace message for a failed expect.

[ ] Embed original source code as trace akin to the ? operator

I was just going to propose something like this.
Also, I think it would be nice if a custom message could be provided to the expect something like the following:
expect @"Something bad happened" Some(foo) = bar()

Interesting suggestion

Any plan to introduce a solution to this?

https://github.com/aiken-lang/aiken/pull/791


Screen pinning improvements

Inspired by the way ASUS implemented their screen pinning, I took it upon myself to do the
same with our ROM and move the pinning icon next to the icon for dismissing a recent app.

Matched the behavior of the recent dismiss icon
Changes icon to dark if recent app uses a light primary color

See images to see the differences
AOSP/DU - http://i.imgur.com/DYoKmNL.png
ASUS Zenfone 2 (inspiration) - http://i.imgur.com/BybyGHp.jpg
What it looks like with this commit - http://i.imgur.com/33zJK0c.png
Change-Id: Ie3e160647ba90b99fd0e130abc79fb63ed0ad3f5
Works as advertised on RR. Dont know if u guys wanna put it in or not.

Can you contact with me on hangouts?

Yep
On Jun 30, 2015 5:25 PM, "Altan KRK"<EMAIL_ADDRESS>wrote:

Can you contact with me on hangouts?
—
Reply to this email directly or view it on GitHub
https://github.com/ResurrectionRemix/android_frameworks_base/pull/17#issuecomment-117349626
.

Hey @bdogg718k @westcripp this commit is outdated and has a bug in it where the icon will stay enabled even when disabled in settings
I since fixed that and added a vector and is much nicer :-)
http://gerrit.dirtyunicorns.com/#/c/16768/

Thanks, will implement
On Jul 1, 2015 12:14 PM, "Alex Cruz"<EMAIL_ADDRESS>wrote:

Hey @bdogg718k https://github.com/bdogg718k @westcripp
https://github.com/westcripp this commit is outdated and has a bug in
it where the icon will stay enabled even when disabled in settings
I since fixed that and added a vector and is much nicer :-)
http://gerrit.dirtyunicorns.com/#/c/16768/
—
Reply to this email directly or view it on GitHub
https://github.com/ResurrectionRemix/android_frameworks_base/pull/17#issuecomment-117471212
.


Web Connection works well on emulator but fail on real devices

I have a error: connection refuse when try to connection to my server(port: 7474) with a real device, but everything is fine on AVD, anyone has a same issue or some suggestion?

We have the same problem with a wss://URI on port 443. Works on emulator, not on Galaxy tab or TomTom Bridge devices.
This is the disconnect reason we get: ApplicationError(jawampa.error.transport_closed, [], {})

What is the kernel version and Android sdk version of the device that is having problem. We issued the same problem but only on some devices. Can you try other physical devices and see if it works? I'm trying to narrow the possible cause.

Ok, so our error is caused by bug in Android build that somehow removed list of ciphers in the org.apache.harmony.xnet.provider.jsse.ClientHandshakeImpl class (and dependent ones).
The TomTom Bride got update that fixed the problem.
Also there is problem with logging in Netty client version which suppress the logging, updating the Netty client library resulted in more meaningful messages, I will make pull-request.


fixed the trig function issues

Hi @Charaf-B! Usually, when we have an issue, the goal is to fix the portion of the code the issue describes. For example, in issue #50, the problem described was for the sineh, cosh, tanh, arcsine, arccosine, and arctan functions.
Because the sine, cosine, and tangent functions worked correctly, you wouldn't need to delete or edit those parts. Instead, you would edit the portions that were described to be incorrect. You could even delete those portions and create brand new code for yourself if you feel that would be easier.
Feel free to ask any other questions! Thank you so much! Will merge your work now.


What does " (inv.)" after some currency names mean?

I'm seeing (inv.) after some of the currency names, e.g. Zambian kwacha (inv.) - what does it mean?

"(inv.)" after a currency name stands for "invariable." It means that the currency name does not change in plural form. For example, "Zambian kwacha" remains "kwacha" in both singular and plural contexts.
Arguably, it's not a great way of handling it, though. Should really be a flag or even better a singular and plural name for currencies.


Support nested objects

const data = useFakeData({
    'departure': 'location.city',
    'destination': 'location.city',
    'driver': {
        'first_name': 'person.firstName',
        'last_name': 'person.lastName',
        'price': {
            'amount': 20,
            'currency': 'MAD',
        }
    }
}, 2);

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (48198d0) to head (8e4561f).

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #6   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines           21        25    +4
  Branches         2         5    +3
=========================================
+ Hits            21        25    +4

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


Maven coordinates changed

I believe the Maven coordinates have changed from

com.microsoft.azure
elastic-db-tools
1.0.0

to

com.microsoft.azure
azure-elasticdb-tools
1.0.0

is this true?
If yes, can you please adjust the code and pages?

The versioning is all kinds of messed up.
According to the master branch, the source code is still on 1.0.1, but it's not published to Maven central.
The latest branch in Maven central is 1.1.0, but doesn't exist as a tag in the GitHub repo.
When trying to use version 1.1.0, I'm getting an error that doesn't parse connection string properties correctly, but works fine with version 1.0.1 (the unpublished current version).

I fear there is a lot more to this.
I found similar problems when passing SQL Server port to shard definitions as per samples, but it worked without a port (the default port is used instead)
Also, the shard map manager and shard database scripts cannot be dicsovered at runtime and as a result sharding structure preparation fails.
There is change between 1.0.0 and 1.1.0 in this area trying to dynamically discover the script files in executable JAR at runtime but fails to discover the scripts.
The only solution I could find is change the code and embed the scripts inside a custom clone of elastic-db-for-java JAR adjusting "Scripts" class.
I hope this helps.

I found the same thing. There's too much hand waving to get this to work. I resorted to forking just the lib (not samples/) and dealing with that directly. At least there's options so you're not stuck :/

Indeed, it needs tweaking and it has been hard to make it work but I managed to make it work for my team.
In our implementation we are getting the JDBC connection from elastic-db-tools and matching it to a JPA entity manager.


Feat: extend ThemeData in MaterialApp

Extend the theme in MaterialApp to add default styles for application. It prevent styling on some basic widgets.

@hoffmannkrzysztof please update your branch with the latest changes from alpha. Resolve the conflicts.

@hoffmannkrzysztof  Thanks for helping out. But there was a reason we were using the themeProvider haha. So you shouldn't remove all the theme providers. using the build in material theme certainly isn't a bad idea though as it gives a lot of ui items a standard color so you don't have to define it every time so you can definitely keep that. But when defining a custom color for a ui item it's better to use the custom implementation we currently have because it gives us way more freedom because it allows us to make our own ui tokens. Also the current implementation currently makes it easy to have multiple themes like dark mode, light mode, custom etc. Also i was planning to store the textThemes in the AppTheme as well so they also support multiple themes. So could you maybe undo all the places where you removed the theme provider, but keep the materialTheme you added? This will also help with the merge conflicts.

Sure no problem at all. I leave it only changes in MaterialApp

Thanks, sorry after you went trough the trouble.

Wrong branch :)

Wrong branch :)

What is the update of this?

I have already reverted unnecessary commit. So only changes to MaterialApp are included in this PR.

@hoffmannkrzysztof LGTM, thanks!
@dinurymomshad @jorre127 I will leave the merging for you guys

@dinurymomshad @jorre127 I will leave the merging for you guys

@dinurymomshad @jorre127 gentle ping


ValueError: operands could not be broadcast together with shapes (200,18,4) (100,1,1)

I encountered an error while running the lane detection script.
2023-07-13 16:38:19.233089: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.10.2 Traceback (most recent call last):   File "/home/nvidia/Downloads/TfLite-Ultra-Fast-Lane-Detection-Inference/videoLaneDetection.py", line 32, in <module>     output_img = lane_detector.detect_lanes(frame)   File "/home/nvidia/Downloads/TfLite-Ultra-Fast-Lane-Detection-Inference/ultrafastLaneDetector/ultrafastLaneDetector.py", line 79, in detect_lanes     self.lanes_points, self.lanes_detected = self.process_output(output, self.cfg)   File "/home/nvidia/Downloads/TfLite-Ultra-Fast-Lane-Detection-Inference/ultrafastLaneDetector/ultrafastLaneDetector.py", line 135, in process_output     loc = np.sum(prob * idx, axis=0) ValueError: operands could not be broadcast together with shapes (200,18,4) (100,1,1)
It would be really helpful if you could look into this issue and recommend a solution.

I solved it. I downloaded the wrong model(tusimple-> culane).


langid+wtl

Otherwise looks good! Could you please also increment the minor version in https://github.com/ZurichNLP/compare-mt/blob/master/compare_mt/init.py#L12?


fix microseconds parsing in iso8601TimestampToDateValueTransformer

fix for https://github.com/RestKit/RKValueTransformers/issues/26

Could you add a test for this, please?

Done.
Notice that I used [formatter setDateFormat:@"yyyy-MM-dd'T'HH:mm:ss.SSS"]; with only three S's at the end. For some reason NSDateFormatter will ignore additional S's and produce zeroes instead

:+1: ace, thanks!


Custom Table in Block Content Example?

Sanity.io is an amazing project - and we're all in - but sheesh - sometimes its tough.
This is a good example.
The README for this plugin explains exactly how to use the @sanity/table plugin as a field. Excellent.
But we'd really like to give our editors a table in the block content editor - aka - portable text.
This is as far as I've gotten (definded in a blockContent definition file as an array with 'block' and 'types')...
 {
      title: 'Table',
      name: 'inlineTable',
      type: 'object',
      components: {
        preview: TablePreview, // Add custom preview component
      },
      fields: [
        {
          name: 'table',
          title: 'Table',
          type: 'table', // Specify 'table' type
        },
        {
          name: 'caption',
          type: 'string',
          title: 'Caption',
        },
      ],
    },

Which appears in the editor as this...

I have a very simple preview component TablePreview - which simply wraps the object in a div and gives it a border for now...
export function TablePreview(props: any) {
  return
{props.renderDefault(props)}

}

I'm totally okay with the  concepts of portable text, and custom render components - but after reading the docs for a couple of hours now, I have absolutely no idea how to take this further and create a custom preview component in Studio, and custom render component (formerly serializers) in our published document.
Any suggestions, pointers, example code etc., greatly appreciated

Note for reference, I've looked at the default TablePreview component included with the module, as well as tried several variations of preview components..
Here's our complete content schema defintion...
import { TableWithCaptionPreview, ExperimentalTablePreview } from "../components/TablePreview"

/**
 * This is the schema definition for the rich text fields used for
 * for this studio. When you import it in schemas.js it can be
 * reused in other parts of the studio with:
 *  {
 *    name: 'someName',
 *    title: 'Some title',
 *    type: 'blockContent'
 *  }
 */
export default {
  title: 'Block Content',
  name: 'blockContent',
  type: 'array',
  of: [
    {
      title: 'Block',
      type: 'block',
      // Styles let you set what your user can mark up blocks with. These
      // correspond with HTML tags, but you can set any title or value
      // you want and decide how you want to deal with it where you want to
      // use your content.
      styles: [
        { title: 'Normal', value: 'normal' },
        { title: 'H1', value: 'h1' },
        { title: 'H2', value: 'h2' },
        { title: 'H3', value: 'h3' },
        { title: 'H4', value: 'h4' },
        { title: 'Quote', value: 'blockquote' },
      ],
      lists: [
        { title: 'Bullet', value: 'bullet' },
        { title: 'Number', value: 'number' }
      ],
      // Marks let you mark up inline text in the block editor.
      marks: {
        // Decorators usually describe a single property – e.g. a typographic
        // preference or highlighting by editors.
        decorators: [
          { title: 'Strong', value: 'strong' },
          { title: 'Emphasis', value: 'em' },
          { title: 'Code', value: 'code' }
        ],
        // Annotations can be any object structure – e.g. a link or a footnote.
        annotations: [
          {
            name: 'link',
            type: 'object',
            title: 'Link',
            fields: [
              {
                name: 'href',
                type: 'url',
                title: 'URL',
              },
              {
                name: 'external',
                title: 'External',
                description: 'Is this an external link? i.e. a link to another site?',
                type: 'boolean',
              },
              {
                name: 'blank',
                title: 'Open in new tab',
                description: 'This option will force the link to open in a new tab.',
                type: 'boolean',
              },
            ],
          },
          {
            name: 'internalLink',
            type: 'object',
            title: 'Internal link',
            fields: [
              {
                name: 'reference',
                type: 'reference',
                title: 'Reference',
                to: [
                  { type: 'guide' },
                  // other types you may want to link to
                ],
                weak: false,
              },
            ],
            icon: () => 'Ref'
          },
        ],
      },
    },
    // You can add additional types here. Note that you can't use
    // primitive types such as 'string' and 'number' in the same array
    // as a block type.
    {
      title: 'Table',
      name: 'inlineTable',
      type: 'object',
      components: {
        preview: TableWithCaptionPreview, // Add custom preview component
      },
      fields: [
        {
          name: 'table',
          title: 'Table',
          type: 'table', // Specify 'table' type
          components: {
            preview: ExperimentalTablePreview,
          }
        },
        {
          name: 'caption',
          type: 'string',
          title: 'Caption',
        },
      ],
    },
    {
      title: 'Image',
      name: 'image',
      type: 'image',
      options: {
        hotspot: true,
        metadata: [
          'blurhash',   // Default: included
          'lqip',       // Default: included
          'palette',    // Default: included
          'exif',       // Default: not included
          'location',   // Default: not included
        ],
      },
      fields: [
        {
          name: 'caption',
          type: 'string',
          title: 'Caption',
        },
        {
          // Editing this field will be hidden behind an "Edit"-button
          name: 'alt',
          type: 'string',
          title: 'Alt text',
        },
      ],
    },
  ],
}

Lastly, a simple table component without any fields defined as ...
{
    title: 'Table',
    name: 'inlineTable',
    type: 'table',
},

.. works fine  but I'm not able to add my custom caption field as defined above. The above definitely works fine when editing, including the table and caption - but the preview does not.
If I try to add a field to the 'table' type as show in this comment, I receive the following error:
"Error: Cannot override fields of subtypes of "object"
Hope some of this helps.

Okay - success - albeit hacky for now...
Here's the composite table, caption type defined as an additional type in the 'blockContent' schema...
{
    title: 'Table',
    name: 'inlineTable',
    type: 'object',
    components: {
      preview: TableWithCaptionPreview, // Add custom preview component
    },
    fields: [
      {
        name: 'table',
        title: 'Table',
        type: 'table', // Specify 'table' type
      },
      {
        name: 'caption',
        type: 'string',
        title: 'Caption',
      },
    ],
    preview: {
      select: {
        table: 'table',
        caption: 'caption'
      }
    }
  },

What was missing was the preview select.
And here's the TableWithCaptionPreview which 'borrows' the table plugin's own TablePreview component and expects rows and title to be at the top level of the props object...
import {TablePreview} from '@sanity/table'

export function TableWithCaptionPreview(props: any) {
  const {caption, table} = props
  console.log('TableWithCaptionPreview props')
  console.log(props)

  const tablePreviewProps = {...props, rows: table.rows, title: props.title}

  return (
    <>
      <TablePreview {...tablePreviewProps} />

{caption}

    </>
  )
}

Hope this helps.

could u share the complete example

If you use the table type directly and later want to add a custom setting is there any way to get existing table cell content into the new structure???

@58bits I was so excited to use this preview technique, but sadly it seems to do nothing at all. Here's my take...
import { FaTable } from "react-icons/fa";
import { TablePreview } from "@sanity/table";

export default {
  title: "Table Block (NEW)",
  name: "tableBlock",
  type: "object",
  icon: FaTable,
  fields: [
    {
      name: "layout",
      type: "string",
      title: "Layout",
      components: {
        preview: (props) => {
          const { table, layout, title, ...rest } = props;
          const tablePreviewProps = { ...rest, rows: table?.rows || [] };

          return (
            <>
              <TablePreview {...tablePreviewProps} />

{layout}

            </>
          );
        },
      },
      options: {
        list: [
          { title: "Default (first row headers)", value: "" },
          { title: "First Row Headers", value: "firstRowHeaders" },
          {
            title: "Both Headers (first row and column)",
            value: "bothHeaders",
          },
          { title: "No Headers", value: "noHeaders" },
        ],
      },
    },
    {
      name: "table",
      title: "Table Content",
      type: "table",
    },
  ],
  preview: {
    select: {
      table: "table",
      layout: "layout",
    },
  },
};

Anyone looking for a complete example, check out my Sanity PortableText Table Widget gist


Add Rust clippy to build process

clippy is a Rust linting tool. Whereas rustfmt is a strictly a
formatter, clippy offers opinions about how code should be written.
Some of these suggestions can be considered "style improvements", like
replacing a single-pattern match with if let and getting rid of
unnecessary return statements.
Other suggestions can actually impact performance, like getting rid of
unnecessary type / ownership converions (.iter(), .clone(),
.into(), etc). One of the more sweeping changes it proposes is to
use pass-by-reference instead of pass-by-value where possible. This
supposedly results in fewer allocations (and hence better performance)
and also makes figuring out ownership simpler in some cases.
Overall, clippy seems like a very useful tool, and it exposes a lot
of language features that we aren't currently using. Certainly I've
learned a lot about Rust from looking at it.
Downsides

clippy is actually implemented as a set of compiler plugins.
These plugins rely on non-stable compiler internals, and
consequently it can only be run on the nightly toolchain. This does
not mean that we have to switch to nightly; the stable and nightly
toolchains can be used side by side, and apparently it is not an
uncommon practice to install nightly for the sole purpose of using
clippy. The drawback here is that sometimes the nightly compiler
breaks, and so one day our builds could fail without warning. I
don't know if that's a common occurrence or a rare one.

Adding clippy to the build process seems to add a lot of time
(look at the build times for this PR). I don't know if there's a
way to speed it up.

I don't think we have to commit to using it forever. We could try it
for a week or two and then get rid of it if it turns out to be too
much of a hassle.

clippy / Rust CI discussions:

https://dev.to/cad97/great-rust-ci-1fk6
http://lorenzo.villani.me/2017/04/25/rust-and-clippy/
https://news.ycombinator.com/item?id=13323796

Also, the commits are structured right now to make review easier, but they should probably be squashed somehow before going in.

Commits that get rid of unnecessary stuff:

lifetimes: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/d51e502af03b13391c5a1a8429525d950338d02a, https://github.com/hyperledger/sawtooth-core/pull/1569/commits/706f569639cf715035d79f5dd29d8faa9a74ff75
lets: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/586422b02f5f1c7adc141aa50d05f8b97e6cdedf
.iter(): https://github.com/hyperledger/sawtooth-core/pull/1569/commits/47c867868d377fdc9a4a0e7f9e6a4f42caa3117d
.into(): https://github.com/hyperledger/sawtooth-core/pull/1569/commits/c848c78e202e6644e654742089f45e267afeb490
.clone(): https://github.com/hyperledger/sawtooth-core/pull/1569/commits/0ee54628f908ec81fa90ea6279b2ea1bd6460751
to_string(): https://github.com/hyperledger/sawtooth-core/pull/1569/commits/c463076fa0fca26b113a8b0c426cabfcfd7648de
return: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/7ce5768a0a959526bb8a501d30c7404ee8b6bc41

Commits that affect correctness (maybe):

Replace lossy casts with lossless casts: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/5e674188462c49480d20e9dc455f087b855861e9
Remove a stray semicolon (I'm actually not sure how this was compiling before): https://github.com/hyperledger/sawtooth-core/pull/1569/commits/fabdcf4695b4babac22362e6442d7f9f127c4394

Commits that affect performance (maybe):

Cut redundant closures: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/e316ce812b41690f282f8e2c9d735d2799635558

-        val.ok().map(|v| Vec::from(v))
+        val.ok().map(Vec::from)

Replace eager-evaluating ok_or with lazy-evaluating ok_or_else: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/09b482dbf0516373ac1da00275ae4a5ed88251c8

-            .ok_or(DatabaseError::NotFoundError(format!(
-                "Block not found: {}",
-                block_id
-            )))?;
+            .ok_or_else(|| DatabaseError::NotFoundError(format!("Block not found: {}", block_id)))?;

Replace single-character string split with single character split: https://github.com/hyperledger/sawtooth-core/pull/1569/commits/2ceb81f7ff05d880df500d3923ba5e7fcd359456

-            let split: Split<&str> = s.split(",");
+            let split: Split<char> = s.split(',');

The biggest change is replacing pass-by-value with pass-by-reference (https://github.com/hyperledger/sawtooth-core/pull/1569/commits/c74736c3245343e5e7f944a6784e4c7904f7dd9b, https://github.com/hyperledger/sawtooth-core/pull/1569/commits/a0eb427e5c772084589da74f1ebd524632776c41). This supposedly requires fewer allocations, and also allows for doing away with some conversion functions:
-    pub fn mark_submit_success(&mut self, batch_id: String) {
-        self.batches_by_id.remove(batch_id.as_str());
+    pub fn mark_submit_success(&mut self, batch_id: &str) {
+        self.batches_by_id.remove(batch_id);

-        timed_batch_iterator.mark_submit_success(batch_id1.clone());
-        timed_batch_iterator.mark_submit_success(batch_id3.clone());
+        timed_batch_iterator.mark_submit_success(&batch_id1);
+        timed_batch_iterator.mark_submit_success(&batch_id3);

-fn load_yaml_array(yaml_str: Cow<str>) -> Result<Cow<Vec<Yaml>>, PlaylistError> {
-    let mut yaml =
-        try!(YamlLoader::load_from_str(yaml_str.as_ref()).map_err(PlaylistError::YamlInputError));
+fn load_yaml_array(yaml_str: &str) -> Result<Cow<Vec<Yaml>>, PlaylistError> {
+    let mut yaml = try!(YamlLoader::load_from_str(yaml_str).map_err(PlaylistError::YamlInputError));

New changes:

Ran rustfmt on Smallbank Rust, since I forgot to add it to the last PR.
Took out the breaking API change in the Rust SDK and disabled that warning locally.
Per request of @peterschwarz, separated the Rust lint Jenkins stage from the Python and Go linting.
Added a missing docstring comment slash in the Rust SDK.


Add terraform 1.1.x to the docker image

I'd like to have terraform 1.1 available in the latest docker image in order to integrate infracost into our CI workflow using the CircleCI Orb.  Without it, I get failures related to the new moved syntax.  Happy to put up a PR if there aren't reasons not to add this!
Running infracost breakdown using:
  $ infracost breakdown --no-color --format json --path /workspace/.terraform/output-dev/tfplan
time="2022-02-17T02:22:03Z" level=info msg="Detected Terraform plan file at /workspace/.terraform/output-dev/tfplan"
time="2022-02-17T02:22:04Z" level=info msg="Starting: Running terraform show"

Error: terraform show failed: exit status 1

  Terraform command failed with:
    Error: Unsupported block type
      on foo.tf line 25:
      (source code not available)
    Blocks of type "moved" are not expected here.
    Error: Unsupported block type
      on foo.tf line 37:
      (source code not available)
    Blocks of type "moved" are not expected here.

Hi @oeuftete - we're in the process of moving away from our old approach (that relied on the TF CLI being baked into the Infracost docker image and running the diff.sh script). We haven't had a chance to update our CircleCI Orb yet, but the other CI/CD integrations have been updated to use this approach - can you possibly try this and let us know if it works ok?

Use our new CI Docker images
Generate a Terraform plan JSON file using the official Terraform image (this enables you to use any version of TF)
Use the Terraform plan JSON file as follows:

infracost breakdown --path terraform_plan.json --format json --out-file infracost.json
infracost comment github --path infracost.json # or infracost comment bitbucket

Our GitLab CI integration does this, here are the docs for the infracost comment command - do you use github or bitbucket?

https://www.infracost.io/docs/features/cli_commands/#circleci-with-github
https://www.infracost.io/docs/features/cli_commands/#circleci-with-bitbucket

@alikhajeh1 Thanks for the prompt response, I appreciate it.  I jumped the gun a bit, and noticed I wasn't converting the plan from binary to json using the old approach, so I do have it working now.  But I'll try the new approach (we're using github).

@oeuftete awesome! When you get the chance to try it, I'd appreciate it if you let me know if the new approach worked for you too.
Ok if I update this issue's title/desc a bit so it reminds us to update the CircleCI Orb readme to describe the new method?

@oeuftete awesome! When you get the chance to try it, I'd appreciate it if you let me know if the new approach worked for you too.

New approach worked great.

Ok if I update this issue's title/desc a bit so it reminds us to update the CircleCI Orb readme to describe the new method?

For sure.
Thanks again for the assistance, @alikhajeh1 .

@oeuftete that's great! I updated this docs section with the above details.

@oeuftete can you please take a quick look at https://github.com/infracost/infracost-circleci and let us know what you think? @aliscott decided to JEDI this GH issue instead of linking to the docs and letting users figure-out the rest (JEDI = Just Effing Do It 😄)


interpolated response

improve custom response with liquid JSONPath

This looks like JSONPath, not Liquid.  Can you not get that you need with Liquid?

@cantino Excuse me, I thought of one thing and write another.
The idea is to be JSONPath, maintaining consistency with the payload_path.

@thiagotalma What is your use case here? I don't understand why JSONPath is required, as I see it Liquid should sufficient.
We generally use JSONPath whenever an Agent needs to work with a hash or an array (like for the payload in this Agent) and Liquid when the output can be a string.

In fact, I'm a problem that will end up coming to many people who try to use webhook with the Facebook API.
See how the request is made:
http://requestb.in/pf1twqpf?hub.mode=subscribe&hub.challenge=1275892619&hub.verify_token=s
The answer should be the value of hub.challenge.
The dot in the parameter name ruined my weekend!  :laughing:
I swear I spent my weekend trying to work around the problem and found no solution.

@thiagotalma I see, that is indeed a bit tricky, does {{["hub.challenge"]}} work?

@dsander But what is the context of interpolation?
It seems that the liquid does not take the parameter values.
Example:
{
  "secret": "3b9661ffaecb41",
  "expected_receive_period_in_days": 1,
  "payload_path": ".",
  "response": "{{ key }}",
  "verbs": "post,get",
  "code": "200"
}

Request: https://myuginn.herokuapp.com/users/1/web_requests/115/3b9661ffaecb41?key=val
Response: EMPTY

See if {{["hub.challenge"]}} works if you delete response_message and change [response_message, code] to:
[interpolated(params)['response'] || 'Event Created', code]

If so, I think that'd be a reasonable change with a spec.

[OFFTOPIC]
@cantino @dsander  how do I run the tests using the docker-compose in multiple-proccess?
I have done tests on Travis, commiting code here :laughing:
I think the project needs a specific wiki to teach contributors the process of setting the environment and run the tests before opening a PR. I'm not the RoR world (i'm PHP) and take a whipping to get code something to help the project. A specific docker-compose for this would be perfect!!
[/OFFTOPIC]

Right, I thought the interpolation context was already set.
@thiagotalma The docker containers are running in the production environment, it's not possible to run the specs there. The Wiki has a few links on how to set up a local development environment: https://gist.github.com/mjhea0/b6b58eefc38985380ff9 and https://github.com/cantino/huginn/wiki/Novice-setup-guide

thank you guys!
code fixed and tests ok!
(@dsander How about a docker-compose ready for development and testing? :relaxed:  )

Interesting. I wonder if we really want the ability to return an empty response body?  I guess you should remove presence for now to make it stay the same. Sorry :)

done!

Thanks @thiagotalma!


JSX code handling

It should handle JSX files just like how Atom handles them.
Currently, we should add an extra setting as:
"files.associations": {
    "*.js": "javascriptreact"
}

Why do you feel you need to add that setting?


build: update deps

This also fixes formatting command as it was broken, also it seems that the Pre-commit Action was broken as well, so fixed that as well.

Note: There are some merge conflicts