fix regexp to extract client id from cookie

Thanks for your change to sixpack-js. I used your fork, though, and found some problems with the cookie parsing lines -- this function was returning several cookie values instead of just the one we were looking for. It works for me after changing out the cookie-parsing code.

Thanks, good catch!

Thanks, good catch!


Uniform Termination Process for Store Implementations: Ensuring Consistent Removal of Binary and .info Files

Describe the bug
Issueing the termination process the filestore removes both the binary as well as  the .info file. The gcc store only removes the binary file.
To Reproduce
The .info is being ignored explicit: https://github.com/tus/tusd/blob/main/pkg/gcsstore/gcsservice.go#L355
func (service *GCSService) FilterObjects(ctx context.Context, params GCSFilterParams) ([]string, error) {

# ...

  if strings.HasSuffix(objAttrs.Name, "info") {
    continue
  }

# ...

Expected behavior
The expected behavior is that all store implementations, including the GCS store, should uniformly handle the terminating process by removing both the binary files and their corresponding .info files. The termination process should be consistent across different store implementations to ensure a standardized and predictable behavior.

Good catch! FilterObjects has been designed to find objects in a bucket which should be composed to form the final object. The .info should not be included here and is thus explicitly removed from the list. For terminating the upload, this behavior is not wanted, as you correctly found.
A solution would be to add an object IncludeInfoObject to GCSFilterParams, which can be set by the termination code to instruct FilterObjects to not exclude .info objects. Would you be interested in opening a PR for this?


Init Bug Fix, Check for Valid AWS Creds, Rollback by Cluster

to: @chunyong-lin
cc: @airbnb/streamalert-maintainers
size: small
resolves #217
Background
The recent change to the deploy command (#520) caused the init steps to break.  While testing, I also discovered that we can add a simple check to avoid large stacktraces when AWS creds aren't loaded.
Changes

Since clusters is passed to the deploy method, we need to modify the one-off named tuple which mimics ArgParse to include a clusters attribute.
Check for valid AWS creds by looking at the os.environ dictionary

Testing

Local pylinting + unit testing
Also did test deploy and terraform commands

Coverage remained the same at 95.569% when pulling cf58efd857269b8abd28abf8e7e759ab44265376 on jacknaglieri-init-deplpy-bug-fix into 84d46e4830db310c3bfa28c715e5d5a5cff1096f on master.

Coverage remained the same at 95.569% when pulling adcb3a8261d1ca6fc7ce14b6a5f8d54aeffd16f3 on jacknaglieri-init-deplpy-bug-fix into 84d46e4830db310c3bfa28c715e5d5a5cff1096f on master.


Align NetworkStream.Read Remarks with src inline documentation.

Summary
NetworkStream.Read remarks could be misleading and should align with inline corefx documentation.
Related
Fixes https://github.com/dotnet/docs/issues/1929
corefx inline documentation: https://github.com/dotnet/corefx/blob/master/src/System.Net.Sockets/src/System/Net/Sockets/NetworkStream.cs#L262
cc: @karelz

@ViktorHofer,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@mairaw you are welcome. Yeah I thought the same. I will add it.

@mairaw do you need anything else from me? Otherwise I will go into sleep mode here ;)

No, I don't. Thanks @ViktorHofer. I need to investigate why the build is still processing though.

Builds are finally green! 🎆


Support Plan Preview for Lambda

What this PR does / why we need it:
Enabled Plan Preview for Lambda, which compares FunctionManifests.
This is almost the same as ECS's PlanPreview(#4881).
Which issue(s) this PR fixes:
Fixes #5045
Does this PR introduce a user-facing change?: no

How are users affected by this change: no
Is this breaking change: no
How to migrate (if breaking change): no

How it works:
When calling PlanPreview as GitHub Actions:

/review


[decimal](test case) porting postgres regression test cases

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Python: choose recommended approach for building packages

To build a distribution the most common approach I have seen is to use setup.py with setuptools.
Use python setup.py sdist for pure Python packages or python bdist_wheel for others.
This will generate a tarball and *.whl file respectively in the dist/ directory.
Then uploading the package to pypi with twine.

I think this is most up to date guide:
https://packaging.python.org/

What about conda packages?

Packaging your Python library as a Conda packages is needed when it depends on C-libraries.
The pure Python packages and platform wheels can be installed in a Conda environment with pip.

@sverhoeven This information should be part of building packages description.

I remember @HannoSpreeuw promised to write something about conda

Well, Daniela and me volunteered to update the paragraph about Conda in the Python chapter of the NLeSC guide.
https://nlesc.gitbooks.io/guide/content/languages/python.html
I will try to do so this month.

Thank you @HannoSpreeuw
that's vary much appreciated :)

But conda packaging seems to be covered already under
Building and packaging code ---> Build using conda

ok, maybe mentioning it's there would help

Okay, I will refer to it in the Conda section, which I am presently updating for another reason.

That's done.
Let's get back to the original issue.
According to this comparison
http://conda.pydata.org/docs/_downloads/conda-pip-virtualenv-translator.html
we should recommend conda for all of our packaging.
So conda should be our recommended build approach and not only for Python packages.

Well, I like about conda that you don't need admin rights...
Maybe we should recommend to do both conda and pip?

@jvdzwaan -- You don't need admin rights to install something via pip (at least not on a virtualenv)
@HannoSpreeuw -- the comparison shows you how to do equivalent command between conda and pip+virtualenv -- I don't think it indicates in any way that one is superior to the other.
I really liked this post -- particularly Myth 10 -- we shouldn't focus on which one is better, but rather should make them more interoperable. In my opinion, a nicely written package should be available regardless of whether your user is going to pick one or the other.

There is also an issue about adding mentioned post, because it's very informative.

Adding it to the guide you mean? I think that is a good idea. I'll try to send a PR to the python chapter.

You know, as flamewars go this one is pretty disappointing. I was all set with popcorn and everything, and now you're having a civilised discussion, with arguments and all...
My 2 cents: As a casual Pythonist, I find pip+virtualenv easier to remember than conda, being simpler. I'll happily put my stuff into both if that's what you recommend, but conda will take a bit more effort to read up on again.

@HannoSpreeuw is a reasonable man, so no need to go to war.
Packaging for both -- I will go with my favourite answer: it depends -- check what makes more sense for your package. If your package is 100% pure python, I would personally recommend to make it pip installable, and then conda users can also use it. If you have lots of external dependencies which are hard to compile, then go with conda. Use the force, and the right tool for the job.

Closing... :)


added extra menu in pause screen

Contains
Added an Extra menu in  pause menu which includes "Metrics Menu ", "Issue Reporter", "Developer Tools" along with "Back" Button #3608
How to test
Run game and press esc key.
You will be able to see a new button "Extra" which includes "Metrics Menu ", "Issue Reporter", "Developer Tools" along with "Back" Button.
Outstanding before merging

[x] Still have to adjust the text on button which is having '?' on left and right side of text

Can one of the admins please verify this patch?

@GooeyHub: ok to test

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

I resolved all the issues @Cervator .
please let me know that if something is missing or something is wrong in above changes
thanks for the help.

Can one of the admins please verify this patch?

Hooray Jenkins reported success with all tests good!

I have resolved all the issues @Cervator
please let me know if something is not correct or something is wrong in it.
thanks for help

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

@Cervator @jellysnake now its working fine '?' error is removed .
please check my PR
thanks :-)

Hooray Jenkins reported success with all tests good!

Oh, and also, if you are at all using the GitHub web-based editor to make changes and/or "blindly" updating things without testing then please don't :-) That is likely to just lead to longer review times from various quirks and unexpected consequences.
Neat that the missing comma resulted in those ? marks, huh!

Oh, and also, if you are at all using the GitHub web-based editor to make changes and/or "blindly" updating things without testing then please don't :-) That is likely to just lead to longer review times from various quirks and unexpected consequences.
Neat that the missing comma resulted in those ? marks, huh!

I'm so sorry for that and all the other mistakes that I have done.
actually this is my first time working on an open source.
I'm working on your suggestion and I'll try to use same .ui file for both purpose.
thanks @Cervator for helping me out. :-)

@Cervator
I modified my screen
please let me know it is correct or not
then i will make a PR


WME1044 for valid assembly name

Describe the bug
For assembly names that are specified with a period -- e.g. MyComponent.Foo the winmd is named MyComponent.winmd instead of MyComponent.Foo.winmd.
To Reproduce

Change the AuthoringDemo to have assembly and default namespace "AuthoringDemo.Foo"
Build
inspect winmd filename

Expected behavior

The winmd should be named "AuthoringDemo.Foo.winmd"
Initial Comments
This is either a bug in GetAssemblyName in WinRT.SourceGenerator\Generators.cs or a bug in how we verify assembly names with namespaces in WinRT.SourceGenerator\DiagnosticHelpers.cs

Ah... Windows Community Toolkit Notifications library is blocked on this, looking forward to the 1.1.2 release!

@andrewleader we just had a prerelease build go out today with authoring bug fixes including for this one if you want to use that to get unblocked.

Awesome thanks Manodasan 😊 I still haven't had a chance to try the prerelease builds out and possibly won't get to try it this week, but when I do, I'll be sure to update you and your team if I find any other issues or if it goes well! Thanks!


ec_deployment_traffic_filter_association is not reflecting in ec_deployment

Expected Behavior
After ec_deployment_traffic_filter_association creation is expecting that ec_deployment.traffic_filter will be reflected by the associations.
Current Behavior
After ec_deployment_traffic_filter_association another terraform plan/apply detects changes at ec_deployment.traffic_filter as follows:
traffic_filter         = [
          - "7e47c1313c174e0ea93a98bf80dbff80",
          - "a8c1134a5bc24fd6a06b43ad7c7eb45c",
        ]

Applying above removes associations and it's continuing endlessly.
Possible Solution
Proper handling of ec_deployment.traffic_filter presence.
Steps to Reproduce

Create any ec_deployment_traffic_filter_association
Run terraform plan/apply
Change is detecting as above.

Your Environment

Version used: 0.1.0-beta
Operating System and version: macOS

Hi @wgebis have you read the documentation for the ec_deployment_traffic_filter_association. There it states:
Note on traffic filters

If you use traffic_filter on an ec_deployment, Terraform will manage the full set of traffic rules for the deployment, and treat additional traffic filters as drift. For this reason, traffic_filter cannot be mixed with the ec_deployment_traffic_filter_association resource for a given deployment.

Additionally, on the ec_deployment resource documentation, it states the following:
Note on traffic filters

If you use traffic_filter on an ec_deployment, Terraform will manage the full set of traffic rules for the deployment, and treat additional traffic filters as drift. For this reason, traffic_filter cannot be mixed with the ec_deployment_traffic_filter_association resource for a given deployment.

Using a ec_deployment_traffic_filter_association should only be used for deployments outside of the scope of the Terraform Provider.
I hope this clarifies what you're seeing.

@wgebis Did you have a chance to take a look at my comment'

Yeah, thanks @marclop fof clarification. So it case of handling ec deployment entirely in TF matching filters shroud be done as a part of ec deployment resource 👍. Makes sense.

Great, I'm closing this issue then if there aren't any more concerns or questions.


Dead/Bad Greaseweazle? Pins 10,34 shorted to ground. Unable to read disks.

I got my new Greaseweazle v4.1 from retropassion.co.uk, and after couple of uses, it stopped working reliably.
When reading flux I'm getting what seems to just noise, and when reading a simple .img, I'm getting "Ignoring unexpected sector" for each sector.
I suspected the usb cable, the floppy cable, and the floppy drive, so I replaced all of them but to no avail.
I also have a Kryoflux which works fine in the same setup.
I noticed that when connecting the floppy ribbon cable to the Greaseweazle, the motor starts spinning, even when the Greaseweazle is disconnected from the computer.
I tested the Greaseweazle 34 pins with a multimeter, a found that pins 10 and 34 are shorted to ground. I compared it with the Kryoflux and there the pins are not shorted.
Looking at the schematics (with my limited electronics knowledge...) it seems that these pins shouldn't be shorted...
Is my Greaseweazle toast?

Are you certain that you are connecting the ribbon the correct way round? Bear in mind that it is "upside down" at the Greaseweazle end compared with a typical floppy drive. It is very easy to get this muddled and overlook this if your ribbon connector isn't keyed.
Basically, try flipping the connector at the Greaseweazle end, if you can. If this doesn't help then your Greaseweazle is sadly buggered. I assume you are in the UK and I'll be happy to send you a replacement directly in this case. Just email me (address is in my GitHub profile) with your shipping address. I supply Retropassion so perfectly happy to sort out warranty issues directly!

I'm sure that the cable is oriented correctly. It is keyed, and the colored wire is on the right, aligned with the pin 1 "arrow" mark on the board.
I assume that the motor is spinning because pin 10 is shorted on the board?
I'm from Israel... :\

Well that sounds about right. Of course this puts pin 10 on the bottom row, and all the top row are ground pins. That matches your testing? Another thing you can try is a straight ribbon (no twist across pins 10-16) and identify as --drive=b in greaseweazle tools. This would use pin 12 for select and pin 16 for motor, conveniently avoiding your suspicious pin 10.
Well, shipping to Israel is not actually a problem. I could certainly send you one by standard untracked airmail. I suppose it would take a couple of weeks?

Using the straight part of the cable did "solve" the spinning motor issue, and I'm able to seek and get rpm reading on drive "b".
But reading disks still results in an "unexpected sector" error for all tracks:
$ gw read --format ibm.360 --drive b testgw.img
Reading c=0-39:h=0-1 revs=2
Format ibm.360
T0.0: Ignoring unexpected sector C:0 H:1 R:7 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:2 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:9 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:3 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:5 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:8 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:6 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:1 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:4 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:7 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:2 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:9 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:3 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:5 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:8 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:6 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:1 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:4 N:2
T0.0: IBM MFM (0/9 sectors) from Raw Flux (85333 flux in 401.71ms)
.
.
T0.0: Ignoring unexpected sector C:0 H:1 R:1 N:2
T0.0: Ignoring unexpected sector C:0 H:1 R:4 N:2
T0.0: IBM MFM (0/9 sectors) from Raw Flux (469196 flux in 2209.53ms) (Retry #1.3)
T0.0: Giving up: 9 sectors missing
T0.1: IBM MFM (9/9 sectors) from Raw Flux (85334 flux in 401.90ms)
T1.0: Ignoring unexpected sector C:1 H:1 R:9 N:2
T1.0: Ignoring unexpected sector C:1 H:1 R:4 N:2
T1.0: Ignoring unexpected sector C:1 H:1 R:5 N:2
.
.

Please consider helping us to preserve media and continue development:
www.softpres.org/donate

$ file test.img
test.img: DOS/MBR boot sector, code offset 0x34+2, OEM-ID "TAN  3.3", sectors/cluster 2, root entries 112, sectors 720 (volumes <=32 MB), Media descriptor 0xfd, sectors/FAT 2, sectors/track 9, dos < 4.0 BootSector (0), FAT (12 bit by descriptor), followed by FAT

So I guess whatever is wrong with the Greaseweazle is more than just a "bad" pin.

Thanks, this probably indicates that the side-select pin is also pulling low. Nonetheless could you do a raw Kryoflux dump (to stream files), using your Kryoflux, of at least a few cylinders, zip them and attach them here? Then we can trivially check the sector headers on sides 0 and 1 and confirm this hypoothesis.
Another thing to check is that your drive has pull-up terminations correctly installed/selected. Greaseweazle outputs are open collector and require these terminations. Kryoflux has push-pull outputs, doesn't require terminations.  This is a bit of a long shot since (a) some pins are obviously working! (b) your greaseweazle used to work.

Indeed, pin 32 is also shorted. I guess I missed that because it is marginal (almost 10 ohm) and my too fancy for its own good multimeter takes some time to register it as short.
Here is the raw flux data from Kruoflux.
test_kf.zip
I also find it hard to believe this is a termination issue, but I tested it anyway.
I'm using a Tandon TM-100 and I took the resistor block (the blue chip) off, but It did not change much. Still getting "Ignoring unexpected sector" with the Greaseweazle.


Curve-optimizer pen / module

An upgrade of qu2cu / qu2cuPen algorithms plus cu2qu, that takes any kind of curves as input, and generates optimal, whole-contour optimized, requested curve types: all-quadratic, all-cubic, or mixed.
Ideally, make a interpolation-compatible version as well.

Congrats on hitting issue #3000 🍾


Upgrade to k3s 1.22

Details
Describe the solution you'd like:
Upgrade to k3s 1.22
Anything else you would like to add:
$ pluto detect-helm -owide
NAME                      NAMESPACE    KIND      VERSION                     REPLACEMENT            DEPRECATED   DEPRECATED IN   REMOVED   REMOVED IN
loki/loki                 monitoring   Ingress   networking.k8s.io/v1beta1   networking.k8s.io/v1   true         v1.19.0         true      v1.22.0
pushgateway/pushgateway   monitoring   Ingress   networking.k8s.io/v1beta1   networking.k8s.io/v1   true         v1.19.0         true      v1.22.0

Loki and Pushgateway ingress would be broken, waiting on these 2 PRs to be merged.
https://github.com/grafana/helm-charts/pull/700
https://github.com/prometheus-community/helm-charts/pull/1443

Forgot to close this issue, his was done a week ago.


DBZ-3897 Update UI to re-org URL

@uidoyen, any thoughts on the build failure?

@uidoyen, assuming this build failure gets fixed, can this then be previewed via the new PR preview? Does it require the PRs in the main repo to be merged first? If so, if we merge them without this change here yet, what's the implication on the published site? Essentially, I'm trying to figure how to preview/review the stuff without breaking the published site :)

Alternatively, we can also do a quick screen share session if you can show me everything from your local machine. WDYT?

Hey @uidoyen, could you clarify whether this PR still is needed? Or should we just close it?

Handled in the following PR https://github.com/debezium/debezium.github.io/pull/734 hence closing this.


Need Access to download docker image quay.io/3scale/docker:ci-2.3.1-3

I wanted to setup 3Scale API Management portal on my local machine. When I tried to build the docker image from the following source :
https://github.com/3scale/porta
I got this error:
SBK132981MAC1:porta chavdam$ docker build .
Sending build context to Docker daemon  37.22MB
Step 1/16 : FROM quay.io/3scale/docker:ci-2.3.1-3
unauthorized: access to the requested resource is not authorized

Could you please help me with this ?

Hi @chavdam ,
It's an issue in Porta. They use JIRA instead of github issues. Here's the link: https://issues.jboss.org/browse/THREESCALE-2609

👍


Watermark - Global Meeting Policy

Watermark need to be enable under "Global" meeting policy.

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Spark JavaRDD give partial documents

What kind an issue is this?
Bug report
Issue description
Using Spark and JavaRDD, documents returned are missing fields !
Posted first here: https://discuss.elastic.co/t/bug-when-reading-if-a-field-has-no-mapping-empty-array-by-ex/76393.
Steps to reproduce
Code:
Create a dummy index:
PUT /toto/events/1
{
   "toto": 1,
   "data": [],
   "name": "webapp_loaded"
}

Query it via Spark shell:
scala> sc.esRDD("toto").first.toString
res22: String = (1,Map(toto -> 1))

It looks like es4Hadoop parse mapping to figure out the structure, and if a field is not in the mapping, it skips others.
Version Info
OS:         :  MacOS / CentOS
JVM         :  1.8
Hadoop/Spark:  2.6
ES-Hadoop   :  2.4.4 / 5.0.0
ES          :  2.4.4

Gloups, specifying index/type fix this weird behaviour 🥇 yeah !!
Is it a bug or index/type is required?

@ebuildy There should really be a check to make sure there is both an index and a type available when setting the es.resource for the connector. It looks like it's pretty lax about that right now, but it's also technically a contract change for the connector.

After reviewing this a bit more over time I'm starting to think that there is more going on under the surface. We leave index parsing pretty lax to support situations like specifying "_all", but I'm starting to wonder if we should allow supporting that at all. Either way, this should probably be looked at for 6.0 where we can make a drastic enough change to require stricter index and type names to be given.

I'm pretty sure that 093e9881665c496c4354c8d474ac551919355166 fixed the root problem here. I've added explicit parsing rules for writing and requiring types, but for reads, we assume all types under the matching indices given.


Twitter location data handling

Something I ran into testing the Flask example and logging in via Twitter. If you're like me, and only had your city in the location field (i.e., Perth, instead of Perth, Australia), this would cause Authomatic to bomb out the login with the error below originating from line 659 of oauth1.py.
_city, _country = _location.split(',')
ValueError: need more than 1 value to unpack

Can confirm that the diff above fixes the issue :)


electron invalid app

gulp build
electron index.js

get error stating that it is a invalid electron app and you should read the documentation.
I think something might have changed after v1 perhaps?

We've got a PR to update this. Thanks!


Running a vue project cross platform

Version
4.5.7
Reproduction link
https://github.com/jeremyimmanuel/vue-aca
Environment info
My ubuntu:
```System:
    OS: Linux 5.4 Ubuntu 20.04.1 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 7 2700 Eight-Core Processor
  Binaries:
    Node: 14.13.1 - ~/.nvm/versions/node/v14.13.1/bin/node
    Yarn: 1.22.5 - /usr/bin/yarn
    npm: 6.14.8 - ~/.nvm/versions/node/v14.13.1/bin/npm
  Browsers:
    Chrome: 86.0.4240.75
    Firefox: 81.0

My mac
System:
    OS: macOS 10.15.5
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
  Binaries:
    Node: 14.13.1 - /usr/local/bin/node
    Yarn: 1.22.5 - /usr/local/bin/yarn
    npm: 6.14.8 - /usr/local/bin/npm
  Browsers:
    Chrome: 85.0.4183.121
    Edge: Not Found
    Firefox: Not Found
    Safari: 13.1.1

Steps to reproduce
On ubuntu
run
vue create abc
cd abc
yarn serve

Then move folder to macos
yarn serve

This will give an error
What is expected?
I expect my project server to start on both of my machine (macos and ubuntu)
What is actually happening?
It can only run on the machine that it was created.

duplicate of #5934


Distributed training doesn't seem to be working

Hey! i tried running the training with DDP over 2 GPUs but got this error -
Error executing job with overrides: ['trainer=gpu']
Traceback (most recent call last):
  File "/home/akhil/instruct-MusicGen/src/train.py", line 125, in main
    metric_dict, _ = train(cfg)
  File "/home/akhil/instruct-MusicGen/src/utils/utils.py", line 78, in wrap
    raise ex
  File "/home/akhil/instruct-MusicGen/src/utils/utils.py", line 68, in wrap
    metric_dict, object_dict = task_func(cfg=cfg)
  File "/home/akhil/instruct-MusicGen/src/train.py", line 92, in train
    trainer.fit(model=model, datamodule=datamodule, ckpt_path=cfg.get("ckpt_path"))
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/trainer.py", line 543, in fit
    call._call_and_handle_interrupt(
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/call.py", line 43, in _call_and_handle_interrupt
    return trainer.strategy.launcher.launch(trainer_fn, *args, trainer=trainer, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/strategies/launchers/subprocess_script.py", line 105, in launch
    return function(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/trainer.py", line 579, in _fit_impl
    self._run(model, ckpt_path=ckpt_path)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/trainer.py", line 986, in _run
    results = self._run_stage()
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/trainer.py", line 1028, in _run_stage
    self._run_sanity_check()
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/trainer.py", line 1057, in _run_sanity_check
    val_loop.run()
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/loops/utilities.py", line 182, in _decorator
    return loop_run(self, *args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/loops/evaluation_loop.py", line 135, in run
    self._evaluation_step(batch, batch_idx, dataloader_idx, dataloader_iter)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/loops/evaluation_loop.py", line 396, in _evaluation_step
    output = call._call_strategy_hook(trainer, hook_name, *step_args)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/trainer/call.py", line 311, in _call_strategy_hook
    output = fn(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/strategies/strategy.py", line 410, in validation_step
    return self._forward_redirection(self.model, self.lightning_module, "validation_step", *args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/strategies/strategy.py", line 640, in __call__
    wrapper_output = wrapper_module(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1519, in forward
    else self._run_ddp_forward(*inputs, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1355, in _run_ddp_forward
    return self.module(*inputs, **kwargs)  # type: ignore[index]
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/lightning/pytorch/strategies/strategy.py", line 633, in wrapped_forward
    out = method(*_args, **_kwargs)
  File "/home/akhil/instruct-MusicGen/src/models/instructmusicgenadapter_module.py", line 157, in validation_step
    loss, preds, targets = self.model_step(batch)
  File "/home/akhil/instruct-MusicGen/src/models/instructmusicgenadapter_module.py", line 120, in model_step
    loss, preds, y = self.forward(batch)
  File "/home/akhil/instruct-MusicGen/src/models/instructmusicgenadapter_module.py", line 79, in forward
    output = self.model(
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/akhil/instruct-MusicGen/src/models/components/model.py", line 388, in forward
    embed_fn = self.cp_transformer.forward(condition_audio_code=condition_audio_code,
  File "/home/akhil/instruct-MusicGen/src/models/components/model.py", line 289, in forward
    sum_code = sum([self.emb_fn["emb"][i](condition_audio_code[:, i]) for i in range(4)])
  File "/home/akhil/instruct-MusicGen/src/models/components/model.py", line 289, in <listcomp>
    sum_code = sum([self.emb_fn["emb"][i](condition_audio_code[:, i]) for i in range(4)])
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/modules/sparse.py", line 162, in forward
    return F.embedding(
  File "/home/akhil/.local/lib/python3.9/site-packages/torch/nn/functional.py", line 2233, in embedding
    return torch.embedding(weight, input, padding_idx, scale_grad_by_freq, sparse)
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:1 and cuda:0! (when checking argument for argument index in method wrapper_CUDA__index_select)

The training does start, but this error comes up in the first 15-20 seconds. I'd imagine its probably some minor issue with file handling across multiple devices?

I utilise PyTorch-Lightning, which should be able to automatically resolve the problem. In the meantime, maybe you can check if I hard-coded cuda:0 in any python file.

@ldzhangyx No hardcoded cuda:0 calls in any python code. I believe the issue seems to be with instructmusicgenadapter_module.py -> forward function handling of tensors, not the PyTorch-Lightning side of things.


Missing mmlu as valid_dataset even though readme encourages download

(gpt5) [jfreeman@mir-81 promptbase]$ python -m promptbase mmlu
usage: main.py [-h] [--subject SUBJECT] [--list_subjects] {gsm8k,humaneval,math,drop,bigbench}
main.py: error: argument dataset: invalid choice: 'mmlu' (choose from 'gsm8k', 'humaneval', 'math', 'drop', 'bigbench')
Head of  main.py :
import argparse
from promptbase.gsm8k import gsm8k
from promptbase.humaneval import humaneval
from promptbase.math import math
from promptbase.drop import drop
from promptbase.bigbench import bigbench
from promptbase.bigbench.consts import BIGBENCH_SUBJECTS
VALID_DATASETS = ["gsm8k", "humaneval", "math", "drop", "bigbench"]
:/

I'm looking in to this as part of #23


add feature for saving extra data to session

Implemented #74

I can't ensure if this is duplicated to "post_restore". And I can't ensure if this is compatible with the vim function. Anyway, I think it is useful to allow "hook" to be a function that returns commands.

Hey, I've briefly looked into this PR, so sorry about the delay here. I'll try and look into it further this week, might even throw some improvements on here. One thing I'd like to mention is we'd want to update the readme and instructions on how to use this, or at least point to the helper files for session extra files.

hello,  I have new commit , to complete this feature

:help mks

If a file exists with the same name as the Session file, but ending in                                          "x.vim" (for eXtra), executes that as well.  You can use *x.vim files to
specify additional settings and actions associated with a given Session,                                         such as creating menu items in the GUI version.

here is sample, when you restore session, neovim will load {session_file}x.vim automatically, for this case, echo 'hello world'
require('auto-session').setup {
    save_extra_cmds = {
        function()
            return [[echo "hello world"]]
        end
    }
}

should I open a new pull request? (I am newbee)

@AbaoFromCUG hey, I checked out your commit, it looks like that should work. I think this PR just wasn't complete yet so we didn't merge it but your commit seems to check most of the boxes. You can open it up and we can discuss a few points I have in there. Thanks 🎉 and thanks to @nanozuki for putting the time a while back! (again sorry about the delay 😅)

#168 is merged, closing this, feel free to reopen if there's anything further!


[Fix][Seatunnel-api]fix timestamp type

Purpose of this pull request

This close #7125
Does this PR introduce any user-facing change?

How was this patch tested?

add UT case
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

In fact, SeaTunnel doesn't support Timestamp type at now. We should make sure flink layer not send timestamp type to seatunnel connector. Please refer https://github.com/apache/seatunnel/blob/adafd802551b9acf9c942965e4f67986a92b07a8/seatunnel-translation/seatunnel-translation-flink/seatunnel-translation-flink-common/src/main/java/org/apache/seatunnel/translation/flink/serialization/FlinkRowConverter.java#L45
cc @TyrantLucifer


Not detecting music playing on Brave (Windows x64) browser

Not sure why, disabling shields (Brave's built in adblocker that's based on uBlock Origin) doesn't fix it either.

Hey there, thanks for the report! I just noticed this on my end on Chrome as well. I believe YouTube Music updated one of their element selectors which is causing the break. Looking into getting a fix out shortly.

Just submitted a fix to the Chrome web store. Usually they review and publish it within a day or two. Will keep this ticket open until we can confirm it's fixed.

Chrome just notified me that they've published the update. I just tested it, and everything seems to be working properly 👍

Thanks for fixing so quickly!


This is too baller . . .

Thanks for your hard work putting this together.  Rejoice!  Type info on hover!

Uh, thanks!


scan_for_cameras.py not working for sony alpha 7r

Hello,
I connected my sony alpha 7r camera to the laptop via wifi and then ran python src/example/scan_for_cameras.py
But it is not able to detect the camera. Can this api work for the sony alpha 7r
Please help!

The Alpha 7r (and 7r MKII) should be supported:
https://developer.sony.com/develop/cameras/
It might be a similar problem to that I had on the HX60, in that the returned 'file' giving camera capabilities and definitions is not quite as expected.
https://github.com/Bloodevil/sony_camera_api/blob/master/src/pysony.py#L110
I'd suggest printing out the 'data' here:
https://github.com/Bloodevil/sony_camera_api/blob/master/src/pysony.py#L127
and seeing if anything is obviously different/wrong. If you post it here, I can give it a look through and compare with the cameras I have.
Simon.

BTW you did installed the app (on the camera), didn't you....?
https://www.playmemoriescameraapps.com/portal/usbspec.php?eid=IS9104-NPIA09014_00-F00002

I have a a7r as well, this is the xml file for reference.
scalarwebapi_dd.xml.txt

@mungewell
Sorry for late response. Yes I had already installed it. But still had no luck with scan_for_cameras.py. So Im not using this package anymore

Putting that 'definition.xml' through the parser outputs something that looks sensible (though I don't have my cameras with me to do a comparison).
{u'accessControl': u'http://<IP_ADDRESS>:8080', u'camera': u'http://<IP_ADDRESS>:8080', u'guide': u'http://<IP_ADDRESS>:8080'}
How did you get the 'xml', I assume a manual download/wget from the camera? In which case it might be the earlier "discover()" function which is failing.
https://github.com/Bloodevil/sony_camera_api/blob/master/src/pysony.py#L30
I'd start by printing out the data sent to "_parse_ssdp_response()" and working back from there.
https://github.com/Bloodevil/sony_camera_api/blob/master/src/pysony.py#L85
You can bypass the discovery process, you should be able to force the camera's IP by using
camera = SonyAPI('http://<IP_ADDRESS>:8080')


📃: Drum Kit Project

🔴 Title : Drum Kit Project
🔴 Tech stack : HTML, CSS, JavaScript
🔴 Objective : To create an interactive drum kit that allows users to play different drum sounds by clicking buttons.
🔴 Summary :  I will create an interactive drum kit web application featuring four buttons, each assigned to a unique drum sound. Users will be able to produce sounds by either clicking the buttons or pressing the corresponding keyboard keys. I will implement JavaScript to handle the button click events and key presses, ensuring that the correct sound is played for each action.
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name : Sneha Tiwari
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.): GSSOC, Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

@iamrahulmahato assign this to me please

The project already exists.


Thread Sniffer and Host Side Controller

We will need a Thread Sniffer for debugging and analyzing the OpenThread.
The Sniffer can also be used to debug and analyze any other thread traffic.

This functionality could be added to the NCP implementation, tracked by #31. Writing a simple sniffer host app that talks to the NCP should be fairly straightforward.

The sniffer will not implement any serial protocol. The plan is to make it being able to run on any protocols which can be NCP or any others.

I assume that to mean the sniffer would use the OpenThread API directly. That being the case, you still have to get the sniffed data off of the device, using some sort of protocol.

I agree it will be useful to include sniffer capability directly in the NCP app.

So will there be an option to enable NCP app or Sniffer app?
Is it clearer to have a standalone sniffer app?
The sniffer only needs Hdlc/Flen frame protocol, but does not need Spinel, and the interface to OT stack.

Hi Lu,
I think both are useful.  Certainly a dedicated sniffer firmware that can
stream into wireshark with a minimal Python tool is useful.
Being able to tap into a sniffer channel via a stock NCP app is useful as
well.
Martin
On May 19, 2016 7:06 AM, "Lu Wang"<EMAIL_ADDRESS>wrote:

So will there be an option to enable NCP app or Sniffer app?
Is it clearer to have a standalone sniffer app?
The sniffer only needs Hdlc/Flen frame protocol, but does not need Spinel,
and the interface to OT stack.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/openthread/openthread/issues/34#issuecomment-220318233

You need more than just HDLC or FLEN for a dedicated sniffer app. You need some way to set the channel, etc. If this capability is going to be built into the NCP (which I plan to do), might as well use that.
For example,  you can turn any spinel-based NCP into a sniffer (not considering the framing protocol) by sending it the following commands:

CMD_RESET

Bytes: 80 01

CMD_SET_PROP(PROP_PHY_ENABLED, 1)

Bytes: 80 03 20 01

CMD_SET_PROP(PROP_PHY_RAW_STREAM_ENABLED, 1)

Bytes: 80 03 27 01

CMD_SET_PROP(PROP_PHY_PROMISCUOUS_MODE_ENABLED, 1)

Bytes: 80 03 28 01

CMD_SET_PROP(PROP_PHY_CHAN, <Channel-Number>)

Bytes: 80 03 21 XX (where XX is the channel number in hex)

Then you would get raw frames on the given channel via updates to PROP_STREAM_RAW. The frames from the NCP would look like this:
80 06 71 LL LL RR ... RR MM ... MM

Where:

LL LL is the length of the raw packet (XX ... XX) in bytes, stored as a little-endian uint16.
RR ... RR is the bytes of the raw packet.
MM ... MM is optional metadata about the raw packet (Channel, RSSI, LQI, etc). Format TBD.

So, all things considered, it's really not any more complicated to talk to the NCP than it would be to talk to a dedicated sniffer app. Initialization is just sending the NCP a fixed set of frames, setting the last byte of the last frame to the channel. Then you handle any frame you receive starting with 80 06 71 as a received packet, and extract the packet bytes and metadata.
Note that this isn't actually implemented in the NCP yet, but doing so wouldn't be very difficult and is absolutely on my list of capabilities I intend to add.

RQ, thanks for your clarifications!
Could I know what is your plan for wpantund?
The host scripts would have 2 options:

capture the packets from wpan interface working with wpantund
implement part of Spinel and capture the packet from serial port directly

I was imagining a lightweight script or tool that would talk to the NCP exclusively.
Adding sniffing to wpantund could certainly be done at some point, but is currently not a feature that is planned. If there is demand for it that could change.

Yes, the sniffer is the first example of many showing we'll want the
host-side of Spinel to be easy to integrate into focused scripts such as
this sniffer python adapter for streaming into a file or into wireshark.
Making sure the code is structured such that both the client and host side
can be built as dedicated library archives will help here.  I believe
that's your plan, RQ, yes?
Martin
On Fri, May 20, 2016 at 1:29 AM, Robert Quattlebaum <
<EMAIL_ADDRESS>wrote:

I was imagining a lightweight script or tool that would talk to the NCP
exclusively.
Adding sniffing to wpantund could certainly be done at some point, but is
currently not a feature that is planned. If there is demand for it that
could change.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/openthread/openthread/issues/34#issuecomment-220476208

RQ, the host tool talk to the NCP that can be the sniffer host tool or any others is in your plan and you will finish it by your own. Yes?

@martin wrote:

Yes, the sniffer is the first example of many showing we'll want the host-side of Spinel to be easy to integrate into focused scripts such as this sniffer python adapter for streaming into a file or into wireshark.

Indeed.

Making sure the code is structured such that both the client and host side can be built as dedicated library archives will help here.  I believe that's your plan, RQ, yes?

There is a set of functions that I have written in spinel.c that are useful for parsing the Spinel data stream, but they are by no means required. The most complicated part of an implementation is likely the HDLC-lite framing.
@luwannest wrote:

RQ, the host tool talk to the NCP that can be the sniffer host tool or any others is in your plan and you will finish it by your own. Yes?

No. At least, not at the moment. I hadn't considered or planned on writing the sniffer host app myself.

I will probably end up writing a small dedicated sniffer host app which talks to the NCP as soon as the API's are in place to allow sniffing.

By the way, basic packet capture capability was added to OpenThread and wpantund. I'd still like to have a dedicated sniffer app, so I'm keeping this open for that.

@darconeous Do you mean dedicated sniffer firmware, host utility, or both?

@turon, see the previous comment from @darconeous.

@jwhui Ah yes "app" in isolation is vague as to whether it is host-side or firmware.  It sounds like the proposed solution is a dedicated host-side utility that talks to stock ncp firmware.  This is something I could take on if others don't have time. @darconeous, the initialization instructions in your May 19th comment look sufficient to get started.

Host utility.


Problem with Google Chrome Incognito mode

Issue description
If you configure recaptcha v3 on modules forms (contact form 7 or divi module form) sent mails are detected as spam in Google Chrome Incognito
Environment

OS name and version: Windows 10
PHP version: 7.4+
Web server name and version:
google/recaptcha version:
Browser name and version: Google Chrome last version Version 88.0.4324.190

Reproducing the issue
expertwp.ch in incognito mode google chrome

Repo is just for PHP client issues. This is about the reCAPTCHA service.


Build issue for v0.8.1

Getting these errors when trying to build. I had previously been on v0.8.0 with no build issues. My ESP-IDF version is v5.4-dev-3201-g46acfdce96.
build-output.txt
Cargo.txt
sdkconfig.txt

Could you also attach your .cargo/config.toml?

Sure, thanks for the reply.
config.txt

Thanks @jacksonemiller .
I fixed this issue in the fix-esp-idf-v5.2.2 branch.
Could you check it?

I am trying to build with IDF version v5.2.2 and am facing this error :
error[E0432]: unresolved imports esp_idf_sys::ble_uuid_any_t, esp_idf_sys::ble_uuid_cmp --> /home/srijan/.cargo/git/checkouts/esp32-nimble-fc9d2c54a3bfa7fe/980873d/src/server/ble_characteristic.rs:5:19   | 5 | use esp_idf_sys::{ble_uuid_any_t, ble_uuid_cmp};   |                   ^^^^^^^^^^^^^^  ^^^^^^^^^^^^ noble_uuid_cmpinsys  |                   |   |                   noble_uuid_any_tinsys
am I missing something?

Working like a charm so far, I appreciate it greatly.


[MM-33080] Set status after installation

Summary
I've checked the code base and this seems like the only place that was missing the status.
Ticket Link
https://mattermost.atlassian.net/browse/MM-33080

https://github.com/mattermost/mattermost-plugin-apps/pull/85 makes this PR redundant


Crash when Enable video is called

If you call the blueprint node "Enable Video" while any other application is gathering the camera input the editor or the build crashes with a fatal error.
It would be good to check first if the camera is available and return false if it couldnt be enabled.
Specifications

OS:  Windows
Version:

comms-sdk-unreal 1.0.0]

@ivananfruns This was fixed in a release of an underlying component recently and the fix will be available in the next plugin release (a matter of days). Thank you for the report (and the comms volume one :)).

Will be fixed in 1.0.1.

@ivananfruns Should be fixed in release 1.0.1. Please verify whether this solution is satisfactory or let us know if you would like to see some more explicit errors in this case.


Expand clickable area of DropDownButtonFormField

Description
This PR increases the clickable area of DropDownButtonFormField.
Related Issues
Resolves #53634
Tests
I added the following tests:
I did not add any tests. The existing widget tests should suffice.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Note about the failing tests: There are several dropdown tests that fail, but they fail in the master branch too.

Hi @percula, thanks for the PR.
You will need to fix the failing tests (or correct them, if they're now behaving incorrectly). Also, you will need to add a test of your own to ensure that the feature behaves as expected and does not regress in the future.

Hi @percula, since this PR has been inactive for quite a while, so I'm going to close it for now. Feel free to reopen it with updates when you feel like you have time to revisit it!

Hi @percula, since this PR has been inactive for quite a while, so I'm going to close it for now. Feel free to reopen it with updates when you feel like you have time to revisit it!

issue is still actual


REPL analyzer processes do not exit

If you open 100 REPL windows, you will have 100 analyzer processes until you exit VS.
We should have them time out and close, or close them when the window is closed.

Also, interactive analyzers should have a different comment on their command line.


Add tasks without needing to add a due date

Hello, is it possible to implement a way that doesn't require us to add a due date everytime we want to create a task?
There are stuff that are in lower priority and don't require a due date, but I am forced to always add a due date.

Sorry it took so long, but your feature request will be live on 2.2.0 update. :)

Sorry it took so long, but your feature request will be live on 2.2.0 update. :)

Awesome. Can't wait!

Sorry it took so long, but your feature request will be live on 2.2.0 update. :)

Awesome. Can't wait!


Adding alias to container configuration

Adding the possibility to autowire Client::class

Ignore this PR, gone to quickly


File browsing too slow in large git repository

Description
I am working on a directory with huge git submodules, many gigabytes worth of code. It takes 3 seconds to run git status.
It also takes 3 seconds to load the directory, and this delay disappears when git_status = false. Can it be fixed by lazy loading of git status? Fast file browsing is important, but I don't want to completely disable git integration.
Neovim version
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1703358377

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.9.5/share/nvim"

Run :checkhealth for more info

Operating system and version
macOS 14.3.1
Steps to reproduce
:Telescope file_browser
Expected behavior
File browser opens instantly.
Actual behavior
There is significant delay on opening the file browser.
Minimal config
// lazy.nvim config

return {
  'nvim-telescope/telescope.nvim',
  branch = '0.1.x',
  dependencies = {
    'nvim-lua/plenary.nvim',
    'nvim-treesitter/nvim-treesitter',
    'nvim-telescope/telescope-file-browser.nvim',
    'nvim-tree/nvim-web-devicons',
  },
  config = function()
    require('telescope').setup {
      extensions = {
	file_browser = {
	  hijack_netrw = true,
	}
      }
    }
    require('telescope').load_extension 'file_browser'
  end,
  keys = {
    { '<Leader>d', '<cmd>Telescope file_browser<cr>', mode = 'n', desc = 'Telescope file browser' },
    { '<Leader>b', '<cmd>Telescope buffers<cr>', mode = 'n', desc = 'Telescope buffers' },
    { '<Leader>ff', '<cmd>Telescope find_files<cr>', mode = 'n', desc = 'Telescope find file'},
  }
}

This is an interesting idea.
Unfortunately, I don't think telescope has a mechanism for doing such a thing at the moment.
But I'll keep this in mind for future telescope enhancements and keep this issue open.
There might be some workaround for it in the meantime I can explore.

I experience the same. I work in quite a large codebase in my company, and although telescope-file-browser is not yet unusable, it is quite slow. Using something like nvim-tree doesn't give the same issue.

Same for me, just as another reference point.  Added git_status = false, and it opens a lot quicker.


fix: remove line-clamp for mobile view

Here we need to remove the line clamp  for the mobile view

Hey Priyankarpal, can I work on this issue

I removed the line clamp for the mobile view 👇

Hey Priyankarpal, can I work on this issue

assigned to you


Feature request: FAB outside bottom bar

Amazing work on the bottom_bar_with_sheet!
Would it be possible to add a fourth option to the placement of the FAB, in order to place the FAB outside the bottom bar?
This would allow for hiding the bottom bar completely (height 0), and the bottom bar could be opened by clicking the FAB.
Inspiration:
This piece by Eike Drescher, UpLabs

Hello @eivihnd ✋
Thank you so much for your suggestion to improve the package 😃
I will try to find a way to make this feature.
You can also commit to this repository if you have an example solution.
The package is open for changes 👐

Thank you for the feedback! :)
I do not have an example solution yet, and I am fairly new to Flutter, but if I can come up with something, I will share it with you.

Hello @eivihnd ! ✋
I created a new functionality in this pull request

Now you can use this example to create UI that you offered.

Here is preview of this example

Amazing work, Frezyx!
It looks really good. I will test it out.
On the topic of non-standard way to use the button and sheet:
From a usability standpoint, do you think this can be used as the main menu?
I think it is an interesting approach, as the menu button is placed closer to your natural position of your fingers, as opposed to the (iOS) standard top left. With large screens one must stretch to reach the top, but with this it is much more easy to reach.
Some would argue that it is not good practice to hide the menu, however that is what we normally do with the hamburger menu. Furthermore, if the app is designed in a way where you normally do not need to use the menu, I think this is a very interesting approach.
Again, thank you so much!
(Btw, if I need (paid) Flutter development work done in the future, can I contact you?)

@eivihnd , I'm glad if solved your problem 😃
It will be very cool if you can give a star to this repository ⭐
This helps to promote the project 📊
Ready to help you with your questions.
You can send me an email at 📫<EMAIL_ADDRESS>


Qmaps 519 favorites interactions

Add cursor pointer on favorites items and actions
Fix junk on favorite deleted

Thanks, these improvements are very appreciated :+1:


Make "status" action of init scripts LSB compliant

Previously, the init scripts would unconditionally return 0 when called
with the "status" action, which misleads (non-human) callers into
believing that the service in question is running even though it is not.
Fix that by explicitly exiting the init script with the appropriate
return code. Also change the slightly improper usage of log_action_msg
with log_success_msg/log_failure_msg.

Looks good to me - Nice!


Pictures not shown in Word based report preview/pdf in version 21.0

On a version 21.0 container, when I use the "Preview" or "Send to... PDF" buttons on a report with a Word based layout, the pictures (both database and embedded in the Word layout) are not shown. It works when we use "Send to... Microsoft Word Document", or when we use a RDLC layout based report.
I'm wondering if this is a container issue because we're not getting this problem in the SAAS version of BC.
Scripts used to create container and cause the issue
$artifactUrl = "https://bcartifacts.azureedge.net/sandbox/21.0.46256.47234/ca"
$credential = New-Object pscredential 'xxxxxx', (ConvertTo-SecureString -String 'yyyyyy' -AsPlainText -Force)
$containerName = "sandbox210ca"
$license = 'C:\License21.flf'

New-BcContainer -accept_eula -assignPremiumPlan -containerName $containerName -artifactUrl $artifactUrl -Credential $credential -auth UserPassword -licenseFile $license -updateHosts -includeTestToolkit -includeTestLibrariesOnly -isolation hyperv

Full output of scripts
BcContainerHelper is version 1.0.16
BcContainerHelper is not running as administrator
Host is Microsoft Windows 11 Pro - Unknown/Insider build
Docker Client Version is 20.10.8
Docker Server Version is 20.10.8
Downloading application artifact /sandbox/21.0.46256.47234/ca
Downloading C:\Users\xxxx\AppData\Local\Temp\e20550ed-7f2d-43cb-9ea4-4b69557225d0.zip
Unpacking application artifact to tmp folder using 7zip
Downloading platform artifact /sandbox/21.0.46256.47234/platform
Downloading C:\Users\xxxx\AppData\Local\Temp\bb0fbc7b-13a9-4925-9a3a-5b17a908965b.zip
Unpacking platform artifact to tmp folder using 7zip
Downloading Prerequisite Components
Downloading c:\bcartifacts.cache\sandbox\21.0.46256.47234\platform\Prerequisite Components\Open XML SDK 2.5 for Microsoft Office\OpenXMLSDKv25.msi
Downloading c:\bcartifacts.cache\sandbox\21.0.46256.47234\platform\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
Downloading c:\bcartifacts.cache\sandbox\21.0.46256.47234\platform\Prerequisite Components\DotNetCore\DotNetCore.1.0.4_1.1.1-WindowsHosting.exe
Fetching all docker images
Using image mcr.microsoft.com/businesscentral:10.0.20348.1129
Creating Container sandbox210ca
Version: 21.0.46256.47234-CA
Style: sandbox
Multitenant: Yes
Platform: 21.0.46384.47190
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.20348.1129 (unknown)
Host OS Version: 10.0.22000.1098 (Unknown/Insider build)
Using hyperv isolation
Using locale en-CA
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\License21.flf
Files in C:\ProgramData\BcContainerHelper\Extensions\sandbox210ca\my:
- AdditionalOutput.ps1
- license.flf
- MainLoop.ps1
- SetupNavUsers.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container sandbox210ca from image mcr.microsoft.com/businesscentral:10.0.20348.1129
3b9c5715318e6270d80045b1f833336c6fa12def401caa7b34624d148527fa6a
Waiting for container sandbox210ca to be ready
Using artifactUrl https://bcartifacts.azureedge.net/sandbox/21.0.46256.47234/ca
Using installer from C:\Run\210-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\sandbox\21.0.46256.47234\platform\ServiceTier\Program Files
c:\dl\sandbox\21.0.46256.47234\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\21.0.46256.47234\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\21.0.46256.47234\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\21.0.46256.47234\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\sandbox\21.0.46256.47234\platform
c:\dl\sandbox\21.0.46256.47234\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\21.0.46256.47234\ca\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\21.0.46256.47234\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\21.0.46256.47234\ca\Extensions
Copying Applications
C:\dl\sandbox\21.0.46256.47234\platform\Applications
Copying Applications.CA
C:\dl\sandbox\21.0.46256.47234\ca\Applications.CA
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Restoring CRONUS Demo Database
Setting CompatibilityLevel for tenant on localhost\SQLEXPRESS
Exporting Application to CRONUS
Removing Application from tenant
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Importing license file
Copying Database on localhost\SQLEXPRESS from tenant to default
Taking database tenant offline
Copying database files
Attaching files as new Database default
Putting database tenant back online
Mounting tenant database
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Stopping Business Central Service Tier
Installation took 226 seconds
Installation complete
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is sandbox210ca
PublicDnsName is sandbox210ca
WARNING: Container starts with TimeZone = Hora estÃ¡ndar romance, which is not recognized in the list of TimeZones.
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint FBC148B9B2A80DFDADB51A05BA2542595DAEA1E3
DNS identity sandbox210ca
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using application pool name: BC
Using default container name: NavWebApplicationContainer
Copy files to WWW root C:\inetpub\wwwroot\BC
Create the application pool BC
Create website: NavWebApplicationContainer without SSL
Update configuration: navsettings.json
Done Configuring Web Client
Enabling Financials User Experience
Using license file 'c:\run\my\license.flf'
Import License
Dismounting Tenant
Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Creating http download site
Setting SA Password and enabling SA
Creating bcuser as SQL User and add to sysadmin
Creating SUPER user
WARNING: The password that you entered does not meet the minimum requirements.
It should be at least 8 characters long and contain at least one uppercase
letter, one lowercase letter, and one number.
Assign Premium plan for BCUSER
Container IP Address: <IP_ADDRESS>
Container Hostname  : sandbox210ca
Container Dns Name  : sandbox210ca
Web Client          : http://sandbox210ca/BC/?tenant=default
Dev. Server         : http://sandbox210ca
Dev. ServerInstance : BC
Dev. Server Tenant  : default
Setting sandbox210ca to <IP_ADDRESS> in host hosts file
Setting sandbox210ca-default to <IP_ADDRESS> in host hosts file

Files:
http://sandbox210ca:8080/ALLanguage.vsix

Container Total Physical Memory is 8.5Gb
Container Free Physical Memory is 5.4Gb

Initialization took 50 seconds
Ready for connections!
Reading CustomSettings.config from sandbox210ca
Creating Desktop Shortcuts for sandbox210ca
Synchronizing Any on default
App successfully synchronized
Installing Any on default
App successfully installed
Synchronizing Library Assert on default
App successfully synchronized
Installing Library Assert on default
App successfully installed
Synchronizing Permissions Mock on default
App successfully synchronized
Installing Permissions Mock on default
App successfully installed
Synchronizing Library Variable Storage on default
App successfully synchronized
Installing Library Variable Storage on default
App successfully installed
Synchronizing Test Runner on default
App successfully synchronized
Installing Test Runner on default
App successfully installed
Copy from container sandbox210ca (C:\Applications.CA\Microsoft_System Application Test Library_21.0.46256.47234.app) to C:\ProgramData\BcContainerHelper\Extensions\sandbox210ca\_Microsoft_Sys
tem Application Test Library_21.0.46256.47234.app
Publishing C:\ProgramData\BcContainerHelper\Extensions\sandbox210ca\_Microsoft_System Application Test Library_21.0.46256.47234.app
Synchronizing System Application Test Library on tenant default
Installing System Application Test Library on tenant default
App successfully published
Copy from container sandbox210ca (C:\Applications.CA\Microsoft_Tests-TestLibraries_21.0.46256.47234.app) to C:\ProgramData\BcContainerHelper\Extensions\sandbox210ca\_Microsoft_Tests-TestLibra
ries_21.0.46256.47234.app
Publishing C:\ProgramData\BcContainerHelper\Extensions\sandbox210ca\_Microsoft_Tests-TestLibraries_21.0.46256.47234.app
Synchronizing Tests-TestLibraries on tenant default
Installing Tests-TestLibraries on tenant default
App successfully published
TestToolkit successfully imported
Container sandbox210ca successfully created

Screenshots
Report 1304 "Sales - Quote", layout "./StandardSalesQuote.docx", the preview/PDF version doesn't show the company picture:

But we can see the picture when we export it to Word:

Additional context
This is the first time we get this issue, and we have tried a mix of onprem/sandbox and different countries. It has failed with the following artifacts:
https://bcartifacts.azureedge.net/onprem/21.0.46256.46853/es
https://bcartifacts.azureedge.net/onprem/21.0.46256.46853/us
https://bcartifacts.azureedge.net/sandbox/21.0.46256.48257/es
https://bcartifacts.azureedge.net/sandbox/21.0.46256.47234/ca
While it does work in the SAAS version of our customer: CA Business Central 21.0 (Platform 21.0.46384.48092 + Application 21.0.46256.46853)
And it works too if we go back to a previous version like: https://bcartifacts.azureedge.net/onprem/20.6.47024.0/es

Have you tried using -usessl (just a longshot).

Have you tried using -usessl (just a longshot).

I've just created a new container with -usessl and I'm getting the same issue.

I can definitely repro this, but I don't think this is related to BcContainerHelper or docker at all.
I can file a bug on the platform team to get this resolved - but I will not be able to follow up and respond back here on this.
You can also file the bug to support yourself and with that get direct feedback and communication with somebody.
You decide.

Alright, I just wanted to consider every possibility, because regular SAAS BC21 works properly. I'll file the bug myself, thanks a lot!

Just in case someone stumbles upon this, it was indeed a platform bug and it has been fixed in the version 21.1 on their side.


[BUG] to logical is broken for very aggregated

This: https://github.com/jfalcou/eve/blob/c8686c45ad32d527eff3743bd5cbb7bfede0e1bd/include/eve/detail/function/simd/common/to_logical.hpp#L67
Will run into problems very quickly

Philosphical question : do we want to support VERY aggregated types after all ?

Yeah. I don't know an answer to that. I'm generally not sure about aggregation, since it's not very good. Or like 64 bytes max.
Needs a decision.

wide larger than cache lien size is useless IMHO. SO we could have :

a macimum size in a static_assert
a hand unrolled implementation of aggregation faster to compile


Time out problem

i used this library to connect to a mosquitto broker running on my local server . the connection is establised after much attempts and on connect it publised a message and gets disconnected after 30 seconds or so .. this happens continously . there is a python paho mqtt cleint connecting to the broker on the same network . it issues regular PINGRES .. however the arduino is unable to do it .. i tried to change the keepalive time to 60s .. but then also it is not working ..please suggest a solution

Are you calling the loop function regularly?

#include <SPI.h>
#include <PubSubClient.h>
#include <Ethernet.h>
/*
*Light on/off green electorn
*
*
*/
byte server[]={192,168,100,203};
byte mac[] = {0x00, 0xAA, 0xBB, 0xCC, 0xDE, 0x02};
//the IP address for the shield:
// the router's gateway address:
void callback(char* topic, byte* payload, unsigned int length);
// MAC Address of Arduino Ethernet Sheild (on sticker on shield)
EthernetClient ethClient;
PubSubClient mqttClient(ethClient);
// Pin 13 is the LED output pin
// defines and variable for sensor/control mode
#define MODE_OFF    0  // not sensing light, LED off
#define MODE_ON     1  // not sensing light, LED on
int senseMode=0;
unsigned long time;
char message_buff[100];
void reconnect() {
// Loop until we're reconnected
while (!mqttClient.connected()) {
Serial.println("Attempting MQTT connection...");
delay(5000);
// Attempt to connect
if (mqttClient.connect("Plant3")) {
Serial.println("connected");
// Once connected, publish an announcement...
mqttClient.publish("sweetco/downtime", "0/0/0/0/0");
mqttClient.loop();
// ... and resubscribe
} else {
Serial.println("failed, rc=");
Serial.print(mqttClient.state());
Serial.println(" try again in 5 seconds");
// Wait 5 seconds before retrying
delay(5000);
}
}
}
void setup()
{
// initialize the digital pin as an output.  pinMode(ledPin, OUTPUT);
// init serial link for debugging
Serial.begin(9600);
mqttClient.setServer(server,1883);
mqttClient.setCallback(callback);
Ethernet.begin(mac);
delay(1500);
Serial.println("I am here");
Serial.println(Ethernet.localIP());
pinMode(2, INPUT_PULLUP);
pinMode(3, INPUT_PULLUP);
pinMode(4, INPUT_PULLUP);
pinMode(5, INPUT_PULLUP);
pinMode(6, INPUT_PULLUP);
return;
}
void loop()
{
mqttClient.loop();
unsigned int count1=0;
unsigned int count2=0;
unsigned int count3=0;
unsigned int count4=0;
unsigned int count5=0;
if (!mqttClient.connected()) {
reconnect();
}
for(int i=0;i<10;i++)
{
int schib1_status=digitalRead(2);
int schib2_status=digitalRead(3);
int schib3_status=digitalRead(4);
int schib4_status=digitalRead(5);
int schib5_status=digitalRead(6);
if(schib1_status==HIGH){
count1++;
}
if(schib2_status==HIGH){
count2++;
}
if(schib3_status==HIGH){
count3++;
}
if(schib4_status==HIGH){
count4++;
}
if(schib5_status==HIGH){
count5++;
}
delay(1000);
}
String data1 = String(count1);
String data2 = String(count2);
String data3 = String(count3);
String data4 = String(count4);
String data5 = String(count5);
char message1[10];
char message2[10];
char message3[10];
char message4[10];
char message5[10];
data1.toCharArray(message1,data1.length()+1);
data2.toCharArray(message2,data1.length()+1);
data3.toCharArray(message3,data1.length()+1);
data4.toCharArray(message4,data1.length()+1);
data5.toCharArray(message5,data1.length()+1);
sprintf(message_buff,"%s/%s/%s/%s/%s",message1,message2,message3,message4,message5);
Serial.println(message_buff);
mqttClient.publish("sweetco/data",message_buff);
mqttClient.loop();
}
// handles message arrived on subscribed topic(s)
void callback(char* topic, byte* payload, unsigned int length) {
int i = 0;
Serial.println("Message arrived:  topic: " + String(topic));
Serial.println("Length: " + String(length,DEC));
// create character buffer with ending null terminator (string)
for(i=0; i<length; i++) {
message_buff[i] = payload[i];
}
message_buff[i] = '\0';
String msgString = String(message_buff);
//Serial.println("Payload: " + msgString);
if (msgString.equals("PLEASE_OFF")) {
senseMode = MODE_OFF;
} else if (msgString.equals("PLEASE_ON")) {
senseMode = MODE_ON;
}
}
this is my code.. the paho client have options for
loop_forever and loop_start and loop_stop
can i do the loop_forever here

You could try adding a call to .loop within your for loop. That loop takes over 10 seconds to complete with no calls to loop, so the client won't be doing anything on the network during that time.


Move lower in fallback chain?

This runs as Fallback level 90.  The loosest Padatious test is at Fallback level 99, so this skill can consume those weak matches.  Since most of these responses are more for personality, I don't think it should match over other Padatious intents.
I'm proposing we move Padatious up to level 98 and move this to level 99.  Thoughts?

Closing this as outdated.


Regions and resource availability list out of date

uksouth is not listed in Regions and resource availability but it is possible to create an ACI in uksouth in a VNET.

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

ID: 8c7387c7-c5ea-584a-1463-eeb562b2cf4d
Version Independent ID: 798fcb07-735e-c79f-6ede-485c671d2364
Content: Scenarios to use a virtual network - Azure Container Instances
Content Source: articles/container-instances/container-instances-virtual-network-concepts.md
Service: container-instances
GitHub Login: @macolso
Microsoft Alias: macolso

@DavidJFowler
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.We will update you accordingly.

@macolso
Can you please check and add your comments on this doc update request as applicable

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


Adding gulp task that adds ads metadata to exe file.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30720

Relevant Lines:
68884

💛  - Coveralls


Concept to check for memory leaks

Develop a concept that allows us to check for memory leaks of the code

Best solution would be to create our own "test tool".
All other test tools are meant to execute the program on the local machine!

Requirements for Test Tool

[ ] Evaluate all calls to calloc(...) and malloc(...)
[ ] Evaluate all calls to free(...)
[ ] Keep Information in list: Memory Allocation List (MAL)
[ ] Print MAL to stdout when requested

Concept

Provide Facade Lib for memory allocation

calloc(...)
malloc(...)
free(...)
Internal handling for evaltuation, then delegation to Pico-SDK function

Library provides flag to enable test scenario

Creates linked list with allocation calls
Removes allocation from list on free(...) call to referenced address
Print list on request


Pressing one button reports same  event for both.

Describe the bug
Incorrectly triggers same event for two buttons when only one is pressed:

Expected behavior
Event should be triggered only for pressed buton.
Versions:

Home Assistant:  2023.8
Shellies Discovery GEN2: 2.17.0
Shelly device firmware: v1.0.0-beta6


Add support for Spring Boot 2.x

The current version is not compatible with Spring Boot 2.x as e.g. FailsafeAutoConfiguration is relying on org.springframework.boot.bind.RelaxedPropertyResolver which is removed in Spring Boot 2.x

https://github.com/spring-projects/spring-boot/wiki/Migrating-a-custom-Actuator-endpoint-to-Spring-Boot-2 has some additional information on how to migrate endpoint definition to Spring Boot 2.x

Thanks for raising this issue @duergner. We are still running Spring Boot 1.x. Still I see the point. I will mark this issue as Help Wanted for the beginning. Maybe someone wants to contribute.

Agree - maybe it's better to address this anyway after https://github.com/zalando-incubator/failsafe-actuator/pull/39 has been merged.

Can I look into this?

Yes @vpondala, feel free to take it. Just out of curiosity how was this issue brought to your attention?

@MALPI I've just started to contribute to open source and was looking to work on some starter type issues. As I am right now working on Spring Boot (and migrations), this looked like a perfect fit.

@MALPI I've added support for SpringBoot 2 (2.0.0.M5). I think we would want to merge these changes to a new feature branch. Could you please create a feature branch for me to submit the pull request against?

@vpondala much appreciated! I created the branch "feature/spring-boot-2" for you. Did you got attention to it by "Hacktoberfest"?

@MALPI Yes, #Hacktoberfest motivated me to do some open source :) Submitted the PR now, however I've not bumped the version yet.

Spring boot 2.0.M7 is out now and there are changes to Actuator endpoints.
Can I send a PR?
On 4 Dec 2017 5:39 AM, "Malte"<EMAIL_ADDRESS>wrote:

Closed #43
https://github.com/zalando-incubator/failsafe-actuator/issues/43.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zalando-incubator/failsafe-actuator/issues/43#event-1369452263,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYhMCnV_PU95Ig3v2Zy1JGJ8_x-obf6ks5s8ur0gaJpZM4P1AIk
.

Hey @vpondala feel free, PR is very welcome. Please also file an issue and describe what needs to be adapted then.


Runtime caching for custom url pattern does not work in Safari

Issue
Custom cached url (image from firebase storage) does not work in Safari. While Chrome and Firefox get the image from cache and therefore also work in offline mode, Safari (desktop/mobile) keeps downloading the image every time and if offline, not find anything anymore obviously.
Library Affected:
workbox-sw, workbox-build, workbox-webpack-plugin etc. (default vue quasar workbox plugin)
Browser & Platform:
Safari (desktop Version 13.0.3 (156<IP_ADDRESS>.4) / mobile iOS 13.1.3)
I'm using the workbox plugin for quasar. The following is a snippet from quasar.conf file managing the pwa and workbox options. Anyway, I don't think my issue is related to quasar or the plugin specifically but to the compatibility between workbox and safari.
  pwa: {
      workboxPluginMode: 'GenerateSW',
      workboxOptions:  {
        clientsClaim: true,
        navigateFallback: '/index.html',
        runtimeCaching: [
          {
            urlPattern: new RegExp('media&token'),
            handler: 'StaleWhileRevalidate',
            options: {
              cacheName: 'brochures-cache',
              cacheableResponse: {
                statuses: [0, 200]
              },
              fetchOptions: {
                mode: 'no-cors'
              }
            }
          },
          {
            urlPattern: '/',
            handler: 'NetworkFirst'
          }
        ],
      },
  }

Hello? Nobody there from the Workbox team?

@jeffposnick In case it was me doing something substantially wrong here, I would highly appreciate a hint where to continue debugging; I feel somewhat lost not having found a lot on this topic so far. Anyway, would be cool to have some discussion here to know at least if I'm the only one or not facing this issue. Many thanks!

Hello!
In general, the way that I'd recommend debugging this is to follow the steps at https://developers.google.com/web/tools/workbox/guides/troubleshoot-and-debug#debugging_workbox to enable the debug builds of Workbox, and check out the JS console logs in Safari. Since you're using GenerateSW, you'd want  to edit your generated service worker file after generation to add in workbox.setConfig({debug: true}) immediately after the call to importScripts() near the top of the file.
If I had to guess at what's going on, I'd say that your RegExp (/media&token/) isn't matching in Safari due to some difference in how the subresource URLs end up being constructed there. If you could pass along steps on reproducing the issue on a publicly accessible website, I could take a look myself. Otherwise, you can check out the additional debugging info and see if that helps.

@jeffposnick Thanks for the explanation and hints. Although Safari lists its files with a name instead of the actual URL, the URL is still there and includes the same pattern. For debugging I switched to /avatar/ which is included everywhere, both in the URL and in the name listed. Still, while chrome works, safari doesn't. I have reproduced it here: https://github.com/alexeigs/debugging, cd into client-a where you'll find everything setup (repo is also just to debug Webstorm using a multi-app structure).
Having the expert on board, I'm struggling also with an issue visible in register-service-worker.js inside src-pwa. I cannot make listening to events/chaining different sw's state work... Thankful for any tip, although the caching issue should be the focus here. :-)

Hello!
The following log in the JS console when running your example explains what's happening:
The regular expression '/avatar/' only partially matched against the cross-origin URL 'https://firebasestorage.googleapis.com/v0/b/sample-project-abcd.appspot.com/o/users%2F7vAA1IkAgxXdWvdM9adm4PA19br1%2Favatar.png?alt=media&token=e13cb44b-ea5d-433d-8975-bea9ff686eda'. RegExpRoute's will only handle cross-origin requests if they match the entire URL.
This behavior is explained in more detail in the Workbox documentation, e.g. at https://developers.google.com/web/tools/workbox/guides/route-requests#matching_a_route_with_a_regular_expression
(Basically, Workbox will not match third-party URLs with RegExps unless you provide a RegExp that matches the full URL. This is to avoid accidentally caching third-party traffic that you didn't anticipate.)
I'm not 100% sure why you'd see different behavior in Safari here, as Workbox behaves the same regardless of browser. I'd say that what you might be seeing is an aggressive usage of the HTTP cache in Chrome and Firefox, bypassing the service worker, but those images seem to be served with HTTP caching disabled.
What I'd suggest doing is adjusting your RegExp to something like: RegExp('^https://firebasestorage\.googleapis\.com/.*media&token') and see if that helps.

@jeffposnick I see. Unfortunately also the RegExp you suggested does not have any impact on caching. I guess it hasn't worked for you either, right? From the little experience I have so far I wouldn't be surprised if there are still some compatibility issues with Safari... Let's nail it down - any more ideas?

Following the steps that you provided with the https://github.com/alexeigs/debugging repo and swapping in RegExp('^https://firebasestorage\.googleapis\.com/.*media&token') as the RegExp in your GenerateSW config leads to something that works fine offline across all browsers, in my local testing.
I took these screenshots of Safari's DevTools (both the SW version and the main page version) while offline, and the PNG response successfully comes from the SW from the perspective of the main page:

The logging in the SW DevTools shows that the StaleWhileRevalidate strategy is being used to handle the request and it successfully generates a response from the cache (even though "revalidate" step via the network fails):

So... I'm not actually able to reproduce any issues, and I do think that switching the RegExp to ensure that it matches the start of the third-party URLs is necessary.

@jeffposnick It was on my end obviously, I forgot to manually clean Safari's cache and without it the changes were not effective. Jeff, thanks again for your help, such support is invaluable and keeps (small) dev teams like us going where otherwise we'd just feel stuck.  🙏

Happy that you got things sorted! (And apologies that the cross-origin routing behavior can be less than intuitive without logging enabled.)


[NNPA] Split a big matmul for NNPA

This patch adds a pattern to split a large MatMul into smaller ones that fit into NNPA. Given NxK * K*M, the pattern considers dimensions N and/or M to split. If N and/or M is greater than NNPA_MAXIMUM_DIMENSION_INDEX_SIZE.
For example, NxK * KxM will be rewritten into (N1xK * KxM1 ++ N1xK * KxM2) ++ (N2xK * KxM1 ++ N2xK * KxM2), where N = N1 ++ N2, M = M1 ++ M2, and ++ is concatenation along an appropriate dimension.
Tensors are splitted into chunks of the equal size of NNPA_MAXIMUM_DIMENSION_INDEX_SIZE, except the last chunk.

Do we need to pass to split all ZHigh op if their size is too large? Will this data decomposition be related to data level parallelization?

Do you mean a separate pass for splitting? I think it would be a good idea.

Will this data decomposition be related to data level parallelization?

Yes, somewhat related. However, sub-matmul ops still runs in a sequential way on NNPA.


Add grab and ungrab methods for Device

Hello,
I've read Evdev's documentation, and while I see the raw function for grabbing a device, there seems to be no safe abstraction.
Would it be possible to add grab and ungrab methods to Device struct?
It should take just few lines of code, and make grabbing significantly more ellegant.
Thanks!

hi! i no longer do software development as a hobby which has led to many of my crates such as this one falling by the wayside. i should figure out some succession, but in the meantime, if you send a PR i’ll merge it :)
On Mon, Sep 21, 2020, at 05:36, RastislavKish wrote:

Hello,
I've read Evdev's documentation, and while I see the raw function for
grabbing a device, there seems to be no safe abstraction.
Would it be possible to add grab and ungrab methods to Device struct?
It should take just few lines of code, and make grabbing significantly
more ellegant.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/emberian/evdev/issues/19, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFL56TX7Z7DEX7DFVOM62DSG4NCLANCNFSM4RUJ7WZA.

--
Ember
they/them


JBPM-5084: troubleshooting function for workitem

@mswiderski
related issue in here https://issues.jboss.org/browse/JBPM-5084.
I not sure it is reasonable, so please check it.
and i don't know how to create unit test for it , it just is draft code .

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@mswiderski
I was update it.
But, i don't know how to create unit test for it.

ok to test

+1


Added support for headers

That's not a good solution, there is a lot libraries that can get random user agent

Why does it matter if it uses random agents or not?

I'll user a randomizer to pick a user-agent, I just don't understand why it matters. Maybe you could explain

Because if everyone will download with the same user agent it will stop working eventually

@megapro17 Updated to use dynamic user-agents
@sj-dan Anything else you see needed?


Refacto statinamic

Close #471
Missing (to be done before merging):

[x] rss
[x] use of <Link> instead of <a>
[ ] images references in .md are broken
[ ] non fr posts are visible (should not for now)
[ ] 404 page is ugly
[ ] CNAME is probably missing from the build
[ ] see how to improve contributrors.json

Reminder: travis is errored with node 4.x.

cooool

Maybe needs revision but
✔  use of <Link> instead of <a>

Thanks @magsout, looks good!

@putaindecode/authors For contributors.json, if we remove it from git, everyone that will want to build the website will probably need a github token as env var (since github api without token is limited to 60 queries per hour) as soon as we will have more than 60 authors. Do you think that might be a problem?

Note: we have already more that 60 contributors so this will be a requirement :/

A token under GH_TOKEN or GITHUB_TOKEN is now required to build the app (at least the first time, if you don't have the contributors.json cache, which is now local).
This contributors.json versionned has always been awful to update anyway.

For some md code that isn't parsed (some images): I changed the engine from marked to markdown-it and it seems markdown-it stop parsing md when it encounter some html. This means that md in html tags aren't converted (that kind of make sense).
We will need to fix all post that are in this case. Youpi quoi.

We will need to fix all post that are in this case. Youpi quoi.

Seems ok, I just sent the fix (2 posts)

I just sent some improvement about NotFound page. It's like the old 404.

Do you think all the css is required?

Maybe not.

Note:for 404.html, we need this to be handled https://github.com/MoOx/statinamic/issues/47

404.html will require more time, so we might skip it atm and add it later (we will have github 404 for a while, not a big deal imo).
I will try to handle last things (hide non fr content, CNAME + adjust error for contributors.json) this evening in order to merge this asap.

I have added some new ajustements.
[10:52:06 PM] [statinamic/lib/static] ✓ Static html files: 72 files written
34s

Here it is. I am merging this and we will see what happen then.
 I am on it.

(related to travis speed)

Bug of the collection fixed by statinamic 0.4.1


task updates not working

I am updating existing tasks using PATCH httpd request, but it seems that update doesn't work properly, for example if I add new httpdOut node, it is not displayed, when fetched with browser.
Also alerts don't trigger.
I am using kapacitor 1.0.0
For now, I am just deleting tasks and recreating them again, then everything works.

@rubycut Can you provide a simple script using curl that reproduces the issue?

If task is in weird state, which means it exists in /kapacitor/v1/tasks/task_name, but it doesn't exist in /kapacitor/v1/tasks, then I can't delete it, and I can't recreate it.

So, kapacitor cli first executes GET request "GET /kapacitor/v1/tasks?dot-view=attributes&fields=link&limit=100&offset=0&pattern=ping&replay-id=&script-format=formatted" to see if task is present, so, since empty response is returned, it never executes DELETE request.

And even doing DELETE request manually with curl, I need to do it twice, it's stuck somewhere:
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|stream->stream0] 2016/09/27 17:24:53 D! closing c: 1625 e: 95
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|stats2->derivative4] 2016/09/27 17:24:53 D! closing c: 1560 e: 1560
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|derivative4->window5] 2016/09/27 17:24:53 D! closing c: 1536 e: 1536
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|window5->http_out6] 2016/09/27 17:24:53 D! closing c: 744 e: 744
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|http_out6->mean7] 2016/09/27 17:24:53 D! closing c: 744 e: 744
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|mean7->http_out8] 2016/09/27 17:24:53 D! closing c: 744 e: 744
Sep 27 17:24:53 xxx kapacitord[16602]: [edge:ping|http_out8->alert9] 2016/09/27 17:24:53 D! closing c: 744 e: 744
Sep 27 17:24:53 xxx kapacitord[16602]: [task_store] 2016/09/27 17:24:53 D! task ping finished
Sep 27 17:24:53 xxx kapacitord[16602]: [task_master:main] 2016/09/27 17:24:53 E! Stopped task: ping mean12: invalid influxql func mean with field percent_packet_lost: invalid field type: <nil>
Sep 27 17:24:53 xxx kapacitord[16602]: [task_store] 2016/09/27 17:24:53 E! task ping finished with error: mean12: invalid influxql func mean with field percent_packet_lost: invalid field type: <nil>
Sep 27 17:24:53 xxx kapacitord[16602]: [httpd] <IP_ADDRESS> - - [27/Sep/2016:17:24:53 +0000] "DELETE /kapacitor/v1/tasks/ping HTTP/1.1" 204 0 "-" "curl/7.38.0" 4d52980a-84d7-11e6-99ee-000000000000 145555

@rubycut Thanks, I have a few ideas as to what might be happening at this point.

Yes, I don't see any point in updating tasks, just drop them and recreate every time.

@rubycut kapacitor remembers the state of an alert, so there is a very good point in updating tasks. For instance, what happens if you've got an alert triggered, and you remove and recreate the task?   You will either have a new alert triggered or the old alert never being reset.  According to this, I believe the update operation should keep state of the last triggered alert.

I guess the current workaround is to disable/enable the task for this to work.

@Artemmkin , you have a point there, deleting and recreating tasks will trigger alerts again.
Also, many tasks store some data in window property, so when task is deleted, this window needs to be filled again.


add data type and time zone considerations

Adding clarifying information about time zones and data types per internal slack thread

🚀 Deployment available! Here are the direct links to the updated files:

https://docs-getdbt-com-git-mirnawong1-patch-27-dbt-labs.vercel.app/docs/build/metricflow-time-spine

thanks so much @courtneyholcomb !


Add explicit shebang for bash in go_indexer_test.bzl [phab:D1362]

Created by j3parker at 2017-02-22 14:04:47:
Bazel's sandbox mounts the users /bin and /bin/sh may not be bash
(on Ubuntu it is now dash: https://wiki.ubuntu.com/DashAsBinSh )

j3parker wrote at 2017-02-22 14:05:40:
This plus #1757 makes bazel build //... complete successfully on Ubuntu 16.10.
(There are many more hits for git grep pipefail though; I did not investigate further.)

fromberger commented inline (context lost during migration) at 2017-02-22 15:05:00:
Can you put shell here, please (i.e., as the first element of cmds) instead of in content below?

fromberger wrote at 2017-02-22 15:05:00:
Thanks. It's unfortunate that the only thing that really cares about the difference (I think) is -o pipefail. I wonder if in this case we couldn't just drop that setting and leave the shell out? This script doesn't need any pipelines and -e works in dash.

j3parker wrote at 2017-02-22 17:32:17:

! In (earlier comment), fromberger wrote:
Thanks. It's unfortunate that the only thing that really cares about the difference (I think) is -o pipefail. I wonder if in this case we couldn't just drop that setting and leave the shell out? This script doesn't need any pipelines and -e works in dash.

That sounds better to me. I can confirm that it works in dash without pipefail.

j3parker wrote at 2017-02-22 23:22:21:
Remove pipefail instead of invoking bash

fromberger accepted this revision at 2017-02-23 04:16:10

fromberger closed via commit c307d66f6857cb96051f1b481e5609d01c2403bc at 2017-02-23 04:19:23


How to determine message size (body + headers) for async_write progress callbacks

I need progress callbacks for my http client.
Implementing the read callback was quite straightforward.
But I am struggling to implement a write callback because I am not able to determine the size in bytes of my http_request.
The read callback has been implemented by seperating the reading of the header from the body.
I am then counting the tranfsered_bytes of the body (async_read_some) while using the content_length of the response to get the size of the payload.
Buuuut... How do I get the size of my complete http_request in bytes ?
Am I supposed to seperate the write as well ?
There is no need for me to explicitly write the header and then the body  and it would unnecessary complicate the write part.
Is there a possibility to avoid sending the header seperatly in order to count the transfered bytes of the body ?
Am I missing something :) ?

Calling prepare_payload on the request object will have the following effect:

This function will adjust the Content-Length and Transfer-Encoding field values based on the properties of the body.

Implementing the write in terms of http::async_write_header and http::async_write_some, as you have done with the read should then give you what you need.
That's probably the simplest option at this stage.

thanks :)
Okay cool, that's not really a problem.
I thought I was simply missing some kind of size() function for the whole message.


2 tests failed. diff:diff between utc and local (80.2) diff:diff between utc and local (80.4)

Client info
Date String   : Fri Nov 21 2014 17:09:34 GMT+0300 (RTZ 2 (зима))
Locale String : 21.11.2014, 17:09:34
Offset        : -180
User Agent    : Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.111 Safari/537.36

====
diff:diff between utc and local (80.2)
month diff
// Expected 2
// Actual   1
1 == 2

====
diff:diff between utc and local (80.4)
week diff
// Expected 3
// Actual   2
2 == 3

I think this is covered by 2089.2 and 2089.4 in #2475