Training with only Monocular Data

Hi,
In paper its mentioned, DROID SLAM trained on monocular data generalizes to stereo and as well as to RGBD.
But when I checked the training code specially with tartan data, looks we need depth as well.
Can you please confirm is this understanding correct or am i missing something?
Thanks!
Best, KP!

Hi KP, your understanding is correct, we do use RGB-D images for training. When we say is is trained on monocular data, what we mean is that the network is that the network is only given monocular data as input. This contrast to inference time when depth and stereo can also be used as input.

Thanks for confirmation!

Do you see a way, where DROID-SLAM can be retrain only w-th RGB data ?
- Something similar to D3VO  where they add addition network for depth estimation?

Best, KP

I am able to fix the issue of pre-trained (droid.pth) model loading by changing the model loading in train.py line 57:58
if args.ckpt is not None:
model.load_state_dict(torch.load(args.ckpt))
to
if args.ckpt is not None:
state_dict = torch.load(args.ckpt)
state_dict["module.update.weight.2.weight"] = state_dict["module.update.weight.2.weight"][:2]
state_dict["module.update.weight.2.bias"] = state_dict["module.update.weight.2.bias"][:2]
state_dict["module.update.delta.2.weight"] = state_dict["module.update.delta.2.weight"][:2]
state_dict["module.update.delta.2.bias"] = state_dict["module.update.delta.2.bias"][:2]
model.load_state_dict(state_dict)
But still, didn't see much difference in output, looks something is wrong with my data or how I am feeding data to the network.
Thanks!
Best, KP

@zachteed
I disable flow_loss by setting w3 = 0.
Also, in the training loop disable the build_frame_graph code which uses depth to create frame_graph.
#if np.random.rand() < 0.5:
#graph = build_frame_graph(poses, disps, intrinsics, num=args.edges)
residual error, rot error, and translation error seems to be reducing.

But the error still seems too high. and looks like some garbage output.

I use the same demo.py setting, did change anything.
Any further suggestion, what could be wrong?
Thanks!
Best,
KP!

Depth maps from monodepth2 don't have spatial and temporal consistency.
Maybe that is a problem?

@zachteed
Hi!
Thank you very much for sharing the code of DROID_SLAM.
I've trained 80,000 times on the Tartanair dataset so far, but why hasn't loss_function shown any signs of convergence so far?

@kbpachauri hi.
how you succeeded to implement monodepth2 depth images along with rgb? in my case i have got some dimension errors of the depth images. error is below;
File "/home/ubuntu/droid2/DROID-SLAM/droid_slam/geom/projective_ops.py", line 130, in induced_flow
ht, wd = disps.shape[2:]
ValueError: too many values to unpack (expected 2)
thanks

@kbpachauri
Hello, I have benefited a lot from reading the above questions. Thank you.
I also want to train a single purpose rgb dataset that can obtain corresponding depth maps and poses through airsim. Have you successfully trained? Considering that it has been a long time and it may be difficult for you to come up with some details, may I ask you some simple questions?
How to place dataset files when training with official datasets?
How to place it when using your own dataset?
3. How is the file placed by fine-tuning the model (anandonefactory, easy)? What is the command?
I really need your help, thank you very much.
Looking forward to your reply.

@zachteed I fine-tune the model on a single TartanAir dataset (abandonedfactory, easy) for 2000 iteration, and inferred results are fine with the fine-tune model, did this experiment to make sure end 2 end process is fine!
 [Backend Optimization OFF].
 [Backend Optimization ON].
Also did a new experiment to check if, without depth, model finetuning works on the TartanAir dataset (abandonedfactory, easy), and from the results, I can see I got a similar issue as my dataset, fine-tuning does not work.
 [Backend Optimization OFF].
I am stuck at this point, need your advice to move forward, what could be wrong with code, or data, or do I need to figure out the setting for fine-tuned model without depth?.
Thanks.
Best, KP!

can

@zachteed I fine-tune the model on a single TartanAir dataset (abandonedfactory, easy) for 2000 iteration, and inferred results are fine with the fine-tune model, did this experiment to make sure end 2 end process is fine!
 [Backend Optimization OFF].
 [Backend Optimization ON].
Also did a new experiment to check if, without depth, model finetuning works on the TartanAir dataset (abandonedfactory, easy), and from the results, I can see I got a similar issue as my dataset, fine-tuning does not work.
 [Backend Optimization OFF].
I am stuck at this point, need your advice to move forward, what could be wrong with code, or data, or do I need to figure out the setting for fine-tuned model without depth?.
Thanks.
Best, KP!

hello, I have the same question as you, Can I communicate with you privately? I have some questions to ask you. Thank you very much. Here is my email address<EMAIL_ADDRESS>and my WeChat ID is:<PHONE_NUMBER>9. Please feel free to contact me using whichever method is convenient for you. I would like to have a conversation with you. Thank you very much.

Thanks, @zachteed for the quick reply!
Are you using depth from mondepth2 and ground truth poses?
Yes, depth from monodepth2 and ground truth pose from code masters.
Could you provide more information about your training setup?
Below are details for my training setup.
I am trying to train the model on code masters data which gives all the pose information, which we converted to EuRoC format.
For training, I am using all default parameters
edges=24, fmax=96.0, fmin=8.0, gpus=1, iters=15, lr=0.00025, n_frames=7, name='bla', noise=False, restart_prob=0.2, scale=False, steps=250000, w1=10.0, w2=0.01, w3=0.05, world_size=1
The first error I figured out is our pose file has a extra column timestamp and is in EuRoc Format. So I changed the below lines in the tartan.py file
tx ty tz qx qy qz qw #orig poses = poses[:, [1, 2, 0, 4, 5, 3, 6]] to timestamp tx ty tz qw qx qy qz #codemasters poses = poses[:, [2, 3, 1, 6, 7, 5, 4]]
But after the above change, I am always getting the below message (which is printed by line 28 in factory.py)
Dataset tartan has 0 images.
I also make sure to delete the cache file before each run which is saved as "DROID-SLAM/droid_slam/data_readers/cache/TartanAir.pickle.
On further debugging, I found a condition in _build_dataset_index in base.py at line 60 is never triggered. Also when I checked the scene in _build_dataset in tartan.py it has all the images. so looks to be a problem with some settings.
Now I increased the max_flow in build_frame_graph from 256 to 1000, graph built and Dataset tartan has some 2600 images.
I train for 3000 iterations from scratch, and this time I didn't get any singular matrices issue but the results seem to be the same.
Looks like I am still missing some settings or information, not sure what.
I did check the tensor board and residual, rot_error, tr_error is decreasing but f_error is not decreasing.

Also, I tried to load the checkpoint for finetuning, but checkpoint loading fails. I am using just 1 GPU to train.
RuntimeError: Error(s) in loading state_dict for DistributedDataParallel: size mismatch for module.update.weight.2.weight: copying a param with shape torch.Size([3, 128, 3, 3]) from checkpoint, the shape in current model is torch.Size([2, 128, 3, 3]). size mismatch for module.update.weight.2.bias: copying a param with shape torch.Size([3]) from checkpoint, the shape in current model is torch.Size([2]). size mismatch for module.update.delta.2.weight: copying a param with shape torch.Size([3, 128, 3, 3]) from checkpoint, the shape in current model is torch.Size([2, 128, 3, 3]). size mismatch for module.update.delta.2.bias: copying a param with shape torch.Size([3]) from checkpoint, the shape in current model is torch.Size([2]).

What are bad_rot and bad_tr?
You mentioned training with only 2 other losses, but how the graph will be built if we don't have depth, you mean just use disparity to build frame graph but disable flow loss?
How DEPTH_SCALE is decided? currently, for tartan, it's set to 5. monodepth2 scale disparity to (0.1 to 100).
How to fix the pre-trained checkpoint loading?
Do I need to change something for poses ground truth or some information that I am missing?

Thanks!
Best, KP!

@kbpachauri hello, I have the same question as you, Can I communicate with you privately? I have some questions to ask you. Thank you very much. Here is my email address<EMAIL_ADDRESS>and my WeChat ID is:<PHONE_NUMBER>9. Please feel free to contact me using whichever method is convenient for you. I would like to have a conversation with you. Thank you very much. @kbpachauri

I am able to fix the issue of pre-trained (droid.pth) model loading by changing the model loading in train.py line 57:58
if args.ckpt is not None: model.load_state_dict(torch.load(args.ckpt)) to if args.ckpt is not None: state_dict = torch.load(args.ckpt) state_dict["module.update.weight.2.weight"] = state_dict["module.update.weight.2.weight"][:2] state_dict["module.update.weight.2.bias"] = state_dict["module.update.weight.2.bias"][:2] state_dict["module.update.delta.2.weight"] = state_dict["module.update.delta.2.weight"][:2] state_dict["module.update.delta.2.bias"] = state_dict["module.update.delta.2.bias"][:2] model.load_state_dict(state_dict)
But still, didn't see much difference in output, looks something is wrong with my data or how I am feeding data to the network.
Thanks!
Best, KP

Hello, I have been stuck here for a long time now, may I ask if you have solved this problem? If it is convenient, can you add a contact information for detailed communication? I would like to ask you about my wechat<PHONE_NUMBER>9, email<EMAIL_ADDRESS>

@kbpachauri Hello, I have benefited a lot from reading the above questions. Thank you. I also want to train a single purpose rgb dataset that can obtain corresponding depth maps and poses through airsim. Have you successfully trained? Considering that it has been a long time and it may be difficult for you to come up with some details, may I ask you some simple questions? How to place dataset files when training with official datasets? How to place it when using your own dataset? 3. How is the file placed by fine-tuning the model (anandonefactory, easy)? What is the command? I really need your help, thank you very much. Looking forward to your reply.

Hello, have you solved this problem and how did you arrange the data set structure?


Terminal: "after all" hook in "Terminal"

VSCode Smoke Tests (Web)
       Terminal
         "after all" hook in "Terminal":
     Error: Smoke test exit call did not terminate process after 20s, giving up
      at D:\a\1\s\test\automation\out\code.js:110:32

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=144638&view=logs&j=672276a2-8d3a-5fab-615d-090c51352f92&t=9302e8c4-de85-5274-34fa-935ce79d4e8d&l=118

@bpasero is this not just https://github.com/microsoft/vscode/issues/137725 after your refactors?

I wanted to file this issue as separate issue because it hints into a direction where we fail to terminate the server (we use tree-kill to kill hits entire hierarchy). I am not sure though there is a clear correlation...

Log in https://dev.azure.com/monacotools/Monaco/_build/results?buildId=145526&view=logs&j=672276a2-8d3a-5fab-615d-090c51352f92&t=9302e8c4-de85-5274-34fa-935ce79d4e8d shows:
Stopping playwright trace took >5seconds
      1) "after all" hook in "Terminal"

Hm yeah I added that kind of logging recently. If we figure out that there is a correlation to the terminal issue here, I am fine changing the tracing to give up after 10s or so, but I wonder what that means...

Work in https://github.com/microsoft/vscode/tree/ben/pw-trace-per-test happens to:

trace per test and not suite (noticed how terminal tests seem to be one big suite, so tracing might end up very time intense)
do not store trace to disk when test succeeded

Nothing useful in the CI logs, I would have expected the playwright trace to contain console messages but it's empty.
New theory: disposing the terminals at the end of each test is triggered async on the remote side but we don't await it. I'll try listen for confirmation from remote that the process has been killed before resolving the "kill all" command.

Pushed that change 🤞

I have not seen this issue yesterday. 🤞

I think we can safely call this fixed 🎉

Let's reopen if we see it again on the main branch.


Monitors need unique names

If we monitor the same thing on multiple hosts, they need unique names. The mysql monitor just had a collision.

I think this is the wrong approach, we should be monitoring by group or by cluster in datadog, not monitor per host. Working example:
"mysql.can_connect".over("*").last(3).count_by_status()

That example only works if we enable the mysql integration and grant rights into mysql for the data dog agent. That's a lot more code than what's in this PR.
Admittedly, just checking the process was a proof of concept, so I welcome alternatives. I'll poke around at making Ansible set up the rights.

Updated the PR for the short win.

This got closed a different way.


[CleanShot X] Add a quick command to upload and share a file

Extension
https://www.raycast.com/Aayush9029/cleanshotx
Description
Currently, the fastest way to upload and share a file is:

Choose "Annotate" in Raycast.
Select the file in Finder.
The Cleanshot X bottom left window will open.
Press the upload button.
The link will be pasted to the clipboard.

I think that adding a command to directly upload and share the file from within Raycast would be beneficial for everyone.
Who will benefit from this feature?
No response
Anything else?
No response

Hello.
The Raycast Cleanshot extension heavily relies on the official URL scheme API, which does not support sharing the file right away. I can't see any workaround for this, so I guess it is not possible now.

Thanks!


matrix: Remove string tests

It was argued in #541 that the the exercise would be more interesting without the fromString function, which is not an essential part of of it.

Rewrite test suite to use [[Int]] instead of String.
Remove test that wouldn't make sense without fromString.
Remove all other references to fromString.
Fix HINTS.md to remove reference to invalid characters.

Closes #541.

I drew a different conclusion from #541 - I thought that the parsing (in particular, parsing a matrix of ints) was okay, it was simply the tests of parsing a matrix of strings (the two that say "may be tricky") that should have been removed. The matrix of chars probably could go too.
The https://github.com/exercism/x-common/blob/master/exercises/matrix/description.md does mention "given a string with embedded newlines", which does imply that the string parsing was an intended part... just probably not matrices containing non-numbers.
It is true that I wouldn't mind removing the parsing completely (there is already a lot in this exercise) and leaving it for another exercise - please indicate if that is an intended route.

Oh! When I was reading the exercise I thought that having fromString and representing all matrices as strings in the test suite was so ugly in Haskell that I almost automatically assumed that the complaint was about it.
Right now I'm a little busy, but I can rewrite the PR in a in a day or two.

Oh! When I was reading the exercise I thought that having fromString and representing all matrices as strings in the test suite was so ugly in Haskell that I almost automatically assumed that the complaint was about it.
Right now I'm a little busy, but I can rewrite the PR in a in a day or two.
If anyone want to fix it before that, fell free to open another PR and close this one.

PR Updated to just remove the two final tests.

PR Updated to just remove the three final tests.

you commented saying remove the final three, but removed only the final two.

Nice catch, @petertseng. Thanks for checking that!
I guess I forgot to push after deciding to remove the third one.

however, the commit contents are consistent with the commit message (removing tricky string). intentional?

I considered that the third one was also in the trick, so I'm keeping the description. Ok?

We forgot to increment the test version (the fourth component)


Updated version with latest version of react and bpmn-js

Just an updated version with latest version of react and bpmn-js
Closes #3 and #5

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

Closed in favor of https://github.com/bpmn-io/bpmn-js-example-react-properties-panel/pull/7.


OS9 Configuration

Hello,
I have been trying to create an image of an OS9 floppy for use on a Gotek with Flash floppy.
the Greaseweazle creates an image without errors & writes  a floppy without errors, but the diskette & the image do not work in the OS9 unit. I assume this is due to my settings. ?
The working Gotek settings:
Microware OS-9 system requires jumper at S0 and the following lines in FF.CFG. Emulation will work with 1.6MB IMG files.
interface = ibmpc
host = akai
Hence I have been setting the GW  format to Akai 1600 & 800 & others to no avail !
The manufacturers 1.6MB IMG file write parameters on their CMD line script, that writes OS9 images to floppy via a windows PC are:
diskrw A: W -t=80 -s=10 -b=1024 -f=E5 -n ... if this helps
But I cannot find a combination in GW that works.
Any help would be most appreciated, thanks for listening !

What command line are you using to generate the image using Greaseweazle? Can you zip the image file, and also a raw flux image file (eg. gw read my.scp), and attach to this ticket. Also include your FF.CFG file in the zip archive.

Hi Keirf,
I just reposted this in the discussions thinking this was the wrong place.. sorry.
I had no joy with the terminal , so I've been using the GUI which generates:
gw.exe read --format=akai.1600 --device=COM5 "C:\Users\User\Documents\GD-Service\Grease Weasel\Images\1.img"
Requested files attached:
Image-files.zip
Thanks for your help !

The scp file isn't really raw because --format akai.1600 is still on the command line. You need to get rid of that. Maybe there's a "no format" option somewhere in the GUI?

Another thing to consider: Is the Microware system and its cabling known good? Eg. Can you cross-check with a real floppy drive in the Microware system?

I believe shift + right click in the greaseweazle folder, then select "Open command window here". Then that will open a command window where you can run gw.exe.


Students dashboard page

Students dashboard page should show all previous exams they have taken. In addition to showing exams that are graded by the teacher already, the dashboard should also show exams that are not graded yet and display a message indicating that. Students should be able to click on a button that allows them to review the exam from the table.

Features were lost when I reset my computer: Student dashboard banner needs element hover effect. Add each element to a div and set the height and width to 100% and align items. Set an hover effect in the css that changes the color and sets the cursor to pointer of the element itself instead of the div. The element color should also change to red on hover.

Student main dashboard page is done. Action buttons need to have their functionality created as well but the review and take exam pages need to be created first. I am going to start working on the take exam page first.


Update issue_template.md

It feels like the template for reporting bugs is a bit redundant - we have summary, steps to reproduce, observed behaviour AND expected behaviour, which all are very similar. I think that by paring it down to the essentials, people will be less likely to skip steps.

I would argue that the expected/observed part is important, especially when receiving reports from users with low IT literacy and must describe all problems as "it does not work".

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1484   +/-   ##
======================================
  Coverage     3.2%    3.2%
======================================
  Files         132     132
  Lines        7109    7109
  Branches      686     686
======================================
  Hits          228     228
  Misses       6866    6866
  Partials       15      15

Continue to review full report at Codecov.

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

Hmm. How about if I add information about how to describe the problem under the Steps to reproduce:  section?

I also agree they are redundant, what about adding an extended description under Steps to reproduce section something like "what was expected behaviour and what did you get"

Added extended description. :)


Privacy policy

https://trello.com/c/VuAuzOzJ/216-privacy-policy-page
https://trello.com/c/86Zo8fVu/109-add-terms-of-use-page

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to dev, including commit 63297cf81097674370976cf56969d424d730c8b2

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to test, including commit 63297cf81097674370976cf56969d424d730c8b2

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to prod, including commit 63297cf81097674370976cf56969d424d730c8b2


feat: add rate limiting

The server will wait 1s if clients:

repeat the same request (same ?pos=)
repeatedly hit /sync without a ?pos=.

Both of these failure modes have been seen in the wild. Fixes #93.

(are the e2e test failures flakes?)


Support for multiple env files

Unsure why the following issue was based on voting - without too much exposure. I would voted and many other developers I know as well.
https://github.com/microsoft/vscode-python/issues/10142

Closing as a duplicate of #10142 as our thinking on that issue has not changed.

That's a bit rough, I guess this is proprietary code after all

I'm not quite sure what you mean by "proprietary" since the extension is open source, but we have to think about maintenance cost as we don't have infinite people power to keep everything running. And as part of that we have to weigh the cost of the work to the benefit of the user base, both in numbers and how much of a benefit to those that would derive benefit.
This is the same answer I give on my own personal, open source projects. Maintaining code is simply not free and so you have to decide what you're willing to "pay" for in terms of effort.

I recon that controlling my temper would have been a wiser thing to do. I apologise for my inaccurate statement above. I don't want to come across as someone who feels entitled to things in the Amazon Prime's day and age. I understand what open source and software development entail.
Maybe, secretly I wanted to hear something along the lines of "why don't you create a PR?" instead of a categorical dismissal. I believe that opensource is a collective burden that spreads beyond Microsoft's headquarters. Converting the envfile to a list within the DebugConfiguration does bring a complexity overhead and maybe someone out there would have gladly footed the "bill".

Unfortunately for us, the Venn diagram of "users of the Python extension" and "people who know enough TypeScript to contribute to the extension" is rather small. Our contribution rate on even small things is negligible at best. And I have seen too many times over my 17 years of being a core dev on Python itself to unfortunately be able to believe that no matter how well intentioned someone is, that them promising they will volunteer their spare time to keep things running will lead to them actually sticking around.
And for what you're asking for, it would require:

A spec on how the user specifies OS-specific environment files (and probably paths in general which is an even bigger discussion and spec)
Best way to communicate/explain/document this (since I'm not aware of others who support .env files as having any practice for this sort of thing)
Plumbing this through the extension (which goes into the bowels of every place we execute Python code)
Making sure it never breaks on any OS we support

IOW this isn't a minor feature request. 😉 As such, we have to seriously worry about the time it will take to implement this as well as the long-term maintenance. And as of right now we are not up to making the promise that we are ready to support this even with a PR for it ready to go. Now maybe our thoughts on this will change (we do talk about tackling the "path problem" at some point), but as of right now we just don't have the bandwidth to take this request.
If you are at all interested in reading a book that aligns pretty well with my views on open source maintenance and why it's really hard to get a community up and going where the "collective burden" actually works (especially when you have management to report to like we do about quality and support 😉 ), I would recommend reading "Working in Public" by Nadia Eghbal (disclaimer: I'm quoted in the book a few times).

Thank you for your response. I have noticed that a Venn diagram cannot represent the overlap between "what I think I know" and "what I actually know". It is better to visualise it as a drop in the ocean instead. I am not going to explore the topic of opensource as you are clearly the more experienced and I have nothing but rambling to bring forth.
I naively thought that it would not be hard to port what was already done in terms of parsing one envfile into something that can combine multiple files. One would also think that the OS is abstracted away with strong underlining libraries. I need to look deeper into the code base before I bring forth any new ideas.
Finally to ease my previous concerns and to adapt my out-of-my-control, nontraditional, multi envfile, current workspace, I have opted to pair a preLaunchTask with the current implementation of the envfile attribute. It works well for my immediate needs.
Thank you for your reading recommendation, I have yet to deepen my knowledge with Nadia's aptly named book.

And of course, thank you for your efforts and good work :)


Fix invalid /etc/environment (fix #119)

backported from zookeeper-operator #74

Hi @verterok ! Changes look sane!
Could you just please rebase the branch also in this case? We have just merged the PR that fixes the CI in this repo as well.
Many thanks!

Could you just please rebase the branch also in this case? We have just merged the PR that fixes the CI in this repo as well.
Done!

Cheers


Garbage collection on AKS Nodes

It would be great to have an extension to manually trigger garbage collection within an aks node.
This would be useful when there is disk pressure due to insufficient space or the disk space getting filled up with the unused images.

This is super @sprab, 🎊🙏 Thank you for this! Would you be willing to please share how it is done currently, so that we can stop eat underlying work required and see how we can make it as one-click and also to iron out any possible challenges to brainstorm with you.
cc: and fyi for visibility for @peterbom , thank you

for documentation reasons:

https://eraser-dev.github.io/eraser/docs/quick-start
https://learn.microsoft.com/en-us/azure/aks/image-cleaner

❤☕🙏 Thank you @sprab ,   A quick loop back on this 🚀 I have a POC for this sceanario https://eraser-dev.github.io/eraser/docs/quick-start#automatically-cleaning-images , I guess it will be great phase 1 and at least provide users to have this available for the users, although I also see in vanilla kuberentes kubelet have the https://kubernetes.io/docs/concepts/architecture/garbage-collection/#image-maximum-age-gc
I see eraser is written by our upstream team, it seems very feasible path to add this in, will share the vsix with you. Thank you.


Schema validation & annotation

We have some of this in place at the moment, but we still need:

[x] Object
[ ] Array
[x] Enum
[x] Documentation
[x] Defaults on schema types.
[ ] Nullable

Plus a whole bunch of finessing around things like:

Keyword defaults on handler functions.
Support List in handler input types.
Coerce primitive types to schema types? (eg. bool then handles "True"/"False" string literals, str then trims whitespace by default.)
Returning aggregated validation errors for multiple parameters.
Which typing annotations we support in views.
Native formats such as datetime that have a string serialization.
If we want to constrain the typing system against GraphQL, eg. no general "Map" type.

... why not just use marshmallow ? Aren't you doing too much this way (like django with friends) ?
The same for swagger etc.

@ddorian We can't use marshmallow (or any other existing library) here because they wouldn't fulfil our design criteria of:

Actual type subclasses, that we can then validly use as type annotations.
1:1 mapping onto JSON Schema.

May I suggest PyDantic?
It supports type annotations out of the box and type subclasses.
It's pretty young but so is APIStar so it's still a good opportunity to switch.

Is there any movement on Nullable? Or is there a workaround? I'm trying to setup apistar with SQLAlchemy and I get validation errors when None is passed to a schema, even if there is a default. I can't figure out any way to make it work.

Not yet, tho certainly on the immediate roadmap yup.
Sketching out next steps, here... https://github.com/encode/apistar/projects/1

required will be a future argument?

Most likely that, or something similar yes, but needs some thinking about first.

Is there any way to define a schema property as read only? I am needing that to use in a property like 'id', to prevent user send an id in a create / post method.

Any plans yet to add a way to make a field be required? I've tried typesystem.string(min_length=1) and it seems to allow missing a field.

Aha, I found the required option for typesystem.Object.

Any progress on supporting nullable? typesystem.Object would become really useful with it.

Will it be possible to use Union as a type?


Ruff is removing used imports since 0.0.173 version

Since version 0.0.173 (up to 0.0.178 - newest when create this issue) the ruff is removing used imports
I have two examples:
https://github.com/4DNucleome/PartSeg/blob/fadc4f8ad9b552386de21bbc2a1ab3e0159f52e7/package/PartSegCore/analysis/io_utils.py#L10-L24
is converted to:

and remove the type used in type annotation:
https://github.com/4DNucleome/PartSeg/blob/fadc4f8ad9b552386de21bbc2a1ab3e0159f52e7/package/PartSegCore/mask/io_functions.py#L725
The ProjectInfoBase is a Protocol if this information is important. In the second case, such import could be hidden under if typing.TYPE_CHECKING: but I use in code other things from this module.
And even if it should be moved under the type checking clause, the autofix should not just remove it.

Thanks! Will fix. Sorry about that.

In both cases the cause is the same, which is that the import appears to be redefined-while-unused (hence the NoQA), so we then mark the import as unused.
I could suggest a variety of workarounds… e.g., you could rebind it to a variable of the same name (“Foo = Foo” before the if-statement), or add a NoQA for that case. But, of course, we shouldn’t be removing those.
I may just have to disable this behavior (of marking redefined-while-unused imports as unused) until we have more extensive branch analysis.

(I probably won’t be able to resolve this today just given my schedule but hopefully a NoQA will be ok for you for now?)

Do you mean add noqua to the import line? It may be ok.
name conditionally redefined should not be removed.

Yeah, I mean adding a noqa to the import line, to avoid marking it as unused.

I'm ok with this solution

I try to move problematic imports under if clause and i could create commit that pass pre-commit but when run pre-commit on all files then ruff creates multiple warnings:
ruff.....................................................................Failed
- hook id: ruff
- exit code: 1

warning: `U` has been remapped to `UP`
warning: `U` has been remapped to `UP`
warning: `U` has been remapped to `UP`
warning: `U` has been remapped to `UP`
warning: `U` has been remapped to `UP`
warning: `U` has been remapped to `UP`
warning: `U` has been remapped to `UP`

https://results.pre-commit.ci/run/github/166421141/1671109943.V-T5M9gkRhOblSlqIfxcUA
That hides real error (in this case E501) because the output is truncated.

@Czaki - Can you try upgrading? I made some improvements to how noqa are handled in those cases and modified the warnings to only omit once per code (you can just change U to UP in your pyproject.toml though hopefully to suppress them entirely).

I see that pre-commit hook is in version 186 and this repository is in 188. Did I correctly understand that I need to use at least 188 to test?

188 is now up. (There’s a slight delay between the two repos.)

What is the expected change in behavior? noqa in both lines preventing removal?

Yeah, if you apply this diff, Ruff won't touch that member:
diff --git a/foo.py b/foo.py
index d72d8d6e..d5d7a367 100644
--- a/foo.py
+++ b/foo.py
@@ -7,7 +7,11 @@ import numpy as np
 import packaging.version

 from PartSegCore.mask_create import MaskProperty
-from PartSegCore.project_info import AdditionalLayerDescription, HistoryElement, ProjectInfoBase
+from PartSegCore.project_info import (
+    AdditionalLayerDescription,
+    HistoryElement,
+    ProjectInfoBase,  # noqa: F401
+)
 from PartSegCore.roi_info import ROIInfo
 from PartSegCore.utils import numpy_repr
 from PartSegImage import Image

Closing for now, but let me know if you continue to run into this issue.

Thanks :heart:

You're welcome :)


updated gpcr tutorial

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
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 84.248%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
1551

Relevant Lines:
1841

💛  - Coveralls


Product card using tailwindcss

created the product-card
closes #3

@Jam-Burger this branch has some conflicts. first resolve that and then made new pull request.


Clarify usage of connect in README.md

Added a quick example of how to use connect in react-redux as there are no examples in the apollo-redux example.

Nice!

It would be really nice to complete the example with a used reducer because reducers.js is not uses in the example.

Indeed! @BafS


Stripe API compatibility: 2017-01-27

Code should be sufficient but it appears the behaviour of one of the subscription views has changed. I'm still trying to track it down, help welcome.

@kavdev Do you have any idea on the test failures?

I'll take a look later today. With API compatibility updates, it's also important to check field parity, etc. I'm assuming something's changed since we checked last March.

I'm assuming something's changed since we checked last March.

Wouldn't Stripe announce it in the changelog? I followed it to the letter.

@jleclanche Depends which changelog. Which one did you follow?

This one: https://stripe.com/docs/upgrades?since=2017-01-27#api-changelog

This depends on fixes also pushed in #463

Note that this changelog only reflects backwards-incompatible updates, so make sure you’re subscribed to our API updates mailing list to keep up with new features that don’t happen to break existing code.

I'll go through and check later this week -- there are a few flagged emails sitting in my inbox

Rebased. I think this can land.


Request for Learning Path URL for whitelisting

Category

[x] Question
[ ] Typo
[ ] Bug
[x] Additional article idea

Customer need a list of URLs that Learning Path uses for whitelisting purpose. The environment by default blocks everything and only whitelist when required.
O365 services is published  URLs here,
https://docs.microsoft.com/en-us/office365/enterprise/urls-and-ip-address-ranges
However the URLs that Learning Path's uses are different from the list published above.
Example: pnp.github.io/custom-learning-office-365, akamaized.net, ... doesn't exists as part of O365 URL list
I can run a trace to get the URL but customer is not confident that this will be the full list. Chances that their users hit an inaccessible page does exists.

Hello,
All of the source content references for the web part are pulled in and managed through the https://pnp.github.io/custom-learning-office-365 endpoints so if a wildcard exception for that endpoint can be implemented, that would be ideal (if you'd like to avoid wildcards we can look into providing you the specific endpoints themselves but those may change in the future so a wildcard is best). Otherwise, the content itself is loaded through https://support.office.com which is included in the O365 published URLs link you posted prior (the same goes for URLs like spoprod-a.akamaihd.net). Let me know if this helps or if there are any other questions. Thank you!
Regards,
Justin


Drag and drop feedback

Added feedback pane while drag-and-dropping
Perfected smart-snapping

Good working fixing all the issues!


Delete file

I saw a lot of your comments on this page:
https://chrome.google.com/webstore/detail/extended-s3-browser/ddmmmnnbkhpkgnkafpflhaoohifpdkmg
I have used both, and I must say, there are a lot of bugs in this than in the other extension.
Please release fixes for those bugs.
Some of the bugs I found:

no option to delete a file.

@chaitanyathengdi I am committed to fixing these issue. If you are interested you can help me identify these issues. I will try to split this feedback into 3 bugs and will start from there.

I have made to new issue related to buggy UI & broken search. I will consider this feedback as request to delete file

Fixed https://github.com/shishirsharma/MyS3Browser/commit/8ee3d1460a4e1a82fa9b036787289dc2c229e914


SelectTree Component Does Not Save Parent Area in Database

What happened?
SelectTree component, when used in the AreaResource, does not save the selected parent area into the database. There are no error messages displayed, making it difficult to diagnose the issue.
Component/Area:
 public static function form(Form $form): Form
    {
        return $form
            ->schema([
                TextInput::make('arabic_title')->required(),
                TextInput::make('kurdish_title')->required(),

                SelectTree::make('parentArea')
                     ->relationship('parentArea', 'arabic_title', 'parent_id')
                     ->placeholder(__('Please select a Area'))
                ->withCount()
                ->direction('buttom')
                     ->label(__('Parent Area'))->nullable(),
                TextInput::make('latitude')
                    ->required()
                    ->rules('numeric'),
                TextInput::make('longitude')
                    ->required()
                    ->rules('numeric'),
            ]);
    }

Model:
class Area extends Model
{
    use HasFactory;
    protected $fillable = [
        'arabic_title', 'kurdish_title', 'parent_id', 'latitude', 'longitude'
    ];

    public function parentArea()
    {
        return $this->belongsTo(Area::class, 'parent_id');
    }
}

migration:
Schema::create('areas', function (Blueprint $table) {
            $table->id();
            $table->string('arabic_title');
            $table->string('kurdish_title');
            $table->unsignedBigInteger('parent_id')->nullable();
            $table->foreign('parent_id')->references('id')->on('areas')->onDelete('cascade');
            $table->decimal('latitude', 13, 9);
            $table->decimal('longitude', 13, 9);
            $table->timestamps();
        });

How to reproduce the bug
Steps to Reproduce:

Go to the AreaResource form in the application.
Use the SelectTree to choose a parent area.
Save the form.
Check the database or refresh the form; the parent area is not updated in the database.

Expected Result:
The selected parent area should be saved into the database and be visible when retrieving or refreshing the Area details.
Actual Result:
The parent area selection is not saved into the database, and no errors are displayed, indicating a silent failure.
Package Version
3.1
PHP Version
8.1
Laravel Version
10.10
Which operating systems does with happen with?
Windows
Notes
No response

Since parentArea is a BelongsTo im assuming it saves a single ID to this column? Try adding parentArea to your fillable.


replacing set_key_expiration with epoch_seconds field

Background
The functions in the caching module has TTL as an optional argument, so set key expiration is no longer needed. This also comes with a performance improvement.
Changes

Replacing set_key_expiration with epoch_seconds arg

Testing

CI

looks good, are we also deprecating that function in the detection-helpers package?

No. Mostly because we don't know how customers are using it. There could be valid reasons to use that function to update a TTL without doing anything else. Hard to know


Implement support for HTTP trailers, upstream to Plug

As discussed at https://github.com/elixir-plug/plug/pull/535#issuecomment-555781624, Plug lacks suppor for HTTP trailers. Beyond their usefulness in the real world, they could also help to tie off a couple of awkward spots when dealing with chunk encoding in HTTP/1 (as discussed on the linked thread above).
At least in h2, we already implement support for reading trailers, but currently just log and discard them.

Planning issues are being moved; this is still on the books for 0.8


The 'first' modifier is not working correctly in nested (one-to-many) nodes using PostgreSQL.

Describe the bug
The 'first' modifier is not working correctly in nested (one-to-many) nodes using PostgreSQL.
To Reproduce
Steps to reproduce the behavior:

Init a prisma instance selecting the Docker local installation option with PostgreSQL as database
Deploy this datamodel User, University, Career datamodel

type User {
  id: ID! @unique
  name: String
  interestedIn: [University!]!
}

type University {
  id: ID! @unique
  name: String
  careers: [Career!]!
}

type Career {
  id: ID! @unique
  name: String
}

Load this data example-data
Try to get all the users, with all the universities they are interested in, but only including the first 10 careers for those universities

query{
  users{
    name
    interestedIn{
      name
      careers(first: 10){
        name
      }
    }
  }
}

Expected behavior
It should return the only user (username=Federico) with the three loaded universities with the first 10 careers for each university, but is only returning the first 5 careers for Harvard, the first 5 for Cambridge, and the all 3 careers for Oxford.
Screenshots

Versions (please complete the following information):

OS:  OS X High Sierra
prisma CLI:  1.16.2
Prisma Server: 1.6.2
DB: PostgreSQL 10.5

Additional context
I tried to reproduce this bug using MySQL and it works as expected.

Probably the same as https://github.com/prisma/prisma/issues/3040

We just released 1.16.4 which should fix this issue.

Thanks!
I tested and it works as expected.


New data files from Eric

Please, check the comments in the newDataStructures.

I uploaded the complete data for 2021 and ran DataChecks2021.R.  There are a few minor issues that still have to be fixed in the data and processing from field to datafiles for analysis.
Please, pull the project and run DataChecks2021.R to see the details. The main issues to be resolved are:
mus uses Lit instead of LIT for column name.
For global ID's I recommend the following column names: vpsGlobalID, cpsGlobalID, qdtGlobalID.
For vernacularNames I recommend no spaces, as in OtherCover,  Watergrass, SwampTimothy and Smartweed. VPS and CPS files still have the names with spaces.
There are duplicates records in CPS (We need to investigate this as a group?)
Except for CPS, all other files have errors in the unitName and subunitName for KRN 4A-3, KRN 4A-5 and KRN 4A-6
CPS has some 0's as subunit names. Should be blank.
Quadrat size domain has been changed relative to last years to include "cm" at the end. I suggest that units be placed in the variable (column) name, not in the values.
There are data for PIX-5 and KRN-Shrew Slough which were not surveyed. Can the spatial join be corrected to avoid this?
Thanks!

Looks like the script was still pulling from the old midseason data files. When I run that script on the data sent yesterday, several of those issues should be resolved:
-The MU's file should have LIT.
-No subunits should have a value of 0.
-There should be no data attached to PIX-5 or KRN-Shrew Slough.
For the other issues:
-global ID column changes look good. I'll incorporate that into the processing so it gets output as that. For seeds, should I just use seedGlobalID?
-I'm hesitant on the no-spaces for vernacularName. I'd like to keep things as easy as possible for anyone needing to add species in the future and think it'd make slightly more sense for someone to add the common name as is.
-Duplicate records are still there for CP's - lets talk about 'em as a group
-Quadrat size change looks good. I will incorporate the units into the column name rather than the domain.
Let me know if other issues come up!

Apologies, Eric. I still had the old zip and confused it with the new one.
-global ID column changes look good. I'll incorporate that into the processing so it gets output as that. For seeds, should I just use seedGlobalID?
Yes, thanks.
-I'm hesitant on the no-spaces for vernacularName. I'd like to keep things as easy as possible for anyone needing to add species in the future and think it'd make slightly more sense for someone to add the common name as is.
Sounds good.  I suggest that we have a master list of vernacular names as drop-down choices already linked to the species ID code. It would not be a good idea to let people add vernacular names ad libitum, lest the same species receives multiple names, each with a variety of spellings and capitalization.
I think encountered a couple of other things:
Wrong unit and subunit names for KRN in vps and qdt
vps0 %>% group_by(LIT, unitName, subunitName) %>% count() %>% print(n = Inf)
LIT   unitName         subunitName     n
..
5 KRN   4A-3             ""             10
6 KRN   4A-5             ""             11
7 KRN   4A-6             ""              9
...

Forgive me if I botched something again.

I am having trouble making sense of the GlobalID_seed. See DataChecks2021.R at the end. It looks like the ID's are not identifying seed heads. There should be not duplicate GlobalID_seed for Swamp Timothy and there should be exactly 2 reps for each Watergrass, one in each measurement type.

Resolved. unique ID's were added to files.


bug(CDK/SCROLLING): *cdkVirtualFor causes Property '' does not exist on type 'unknown' with Angular 9 strictTemplates enabled using slice pipe

Reproduction
Stackblitz can't reproduce an issue with the compiler, so I have uploaded a reproduction repo to github.
Steps to reproduce:

Download https://github.com/jahumes/slice-pipe-scrolling-strict-templates
Install packages
Run npm or yarn start
Should see the below errors

ERROR in src/app/app.component.html:13:57 - error TS2339: Property 'id' does not exist on type 'unknown'.

    13                 (click)="organizationClick(organization.id)">
                                                               ~~

      src/app/app.component.ts:9:16
        9   templateUrl: './app.component.html',
                         ~~~~~~~~~~~~~~~~~~~~~~
        Error occurs in the template of component AppComponent.
    src/app/app.component.html:14:28 - error TS2339: Property 'id' does not exist on type 'unknown'.

    14             {{organization.id}}
                                  ~~

      src/app/app.component.ts:9:16
        9   templateUrl: './app.component.html',
                         ~~~~~~~~~~~~~~~~~~~~~~
        Error occurs in the template of component AppComponent.

If you remove the pipe (uncomment line 9 from app.component.html), there are no errors in the console. Also, if you keep the slice pipe and comment out lines 1 and 18 and change the *cdkVirtualFor on line 2 to an *ngFor, then the errors go away.
The issue comes when strictTemplates is enabled in the tsconfig.json. If it is false, then the errors go a way

Expected Behavior
This should not be returning an error.
Environment

Angular: 9.1.11
CDK/Material: 9.2.4
Browser(s): In compiler so all of them
Operating System (e.g. Windows, macOS, Ubuntu): macOS 10.15.5

This seems like an issue with the typings on the slice pipe. The following simplified example fails with the same error, without the cdkVirtualFor:
<span *ngFor="let organization of user.organizations | slice: 0:5">
  <button (click)="organizationClick(organization.id)">{{organization.id}}</button>
</span>

@crisbeto So, I followed your suggestion and changed the example to the following code
<div *ngFor="let user of users;">
  <span *ngFor="let organization of user.organizations | slice: 0:5">
      <button class="btn badge pull-left" (click)="organizationClick(organization.id)">
          {{organization.id}}
      </button>
  </span>
</div>

and I don't get any errors.


Error Upon Loading

Whenever I load this alongside Jamozed's Tweaks in 0.17, I get this error: https://imgur.com/kdrVCKs

Thanks for reporting this.
0.17 replaced separate properties for each direction of belt ends with a general property belt_animation_set.
I've updated Loader Redux to use this instead.


Updated dependencies for 3.1 & removed jeweler

Hi. This version works with 3.1. I have removed jeweler. Think bundler is better way to manage gem at all. Found that rcov and flog does work with ree (default .rvmrc setting) - fails with segfault. I do not know how to fix it. May be, removing .rvmrc file is the best because in right way we should run tests in all versions (1.8.7, 1.9.2, ree and so on). Another point is that ruby 1.9.2 requires linecache19 gem instead of linecache.

Cheers!


Proposal : regex literal type

Hello,
I would suggest to add a "regex literal" type to make ensuring that a string data matches a pattern at assigment.
As the literal string feature has been added in the latest release, i found very convenient to type my variables with, e.g. for typing my react props (size : "small"| "big" )... On the same idea, it could be convenient to be able to type a variable with a pattern.
Example :
    var myString : /\d{3}.*\d{2}/;
    myString="555hhhhh44" ; //correct
    myString="55dsds5dss4"; //error

In react it could be vey useful for defining a react prop like phone numbers, emails, dates, ...
regards
Xavier

Duplicate of #6579.


Remerge epic-0.5.0 to develop

#1224 got merged to epic-0.5.0 by accident.
Upd: also,  #1167.

Codecov Report

Merging #1225 (b4c649d) into develop (be8dbf8) will increase coverage by 0.04%.
The diff coverage is 85.71%.

@@             Coverage Diff             @@
##           develop    #1225      +/-   ##
===========================================
+ Coverage    58.56%   58.60%   +0.04%
===========================================
  Files           82       82
  Lines         4117     4124       +7
===========================================
+ Hits          2411     2417       +6
- Misses        1706     1707       +1

Impacted Files
Coverage Δ

packages/squiggle-lang/src/rescript/FR/FR_List.res
86.84% <85.71%> (-0.12%)
:arrow_down:

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


[$500] Bank account - Form is auto-submitted after disabling internet and killing app

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.41-2
Reproducible in staging?: y
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Go to Workspace > Bank account.
Select Connect online with Plaid and go through the add BA flow until you reach the SSN page of Personal info.
Disable internet connection.
Enter "33".
Kill the app.
Open the app again.
Enter the address info.
Verify that the incorrect info for SSN was submitted.

Expected Result:
The form for SSN should not auto-submit and an error for incorrect data entered should appear.
Actual Result:
The form for SSN is auto-submitted with the incorrect data.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/ed83d9c6-214a-44f5-acf9-4911bb5487a0

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01126719388570e4b1
Upwork Job ID:<PHONE_NUMBER>042637824
Last Price Increase: 2024-02-14
Automatic offers:

akinwale | Contributor | 0

@shubham1206agra @MrMuzyk @barttom This one seems like something we should look at but very small edge case.

@mountiny Assign me and I'll have a look

This will happen more often than described. You could alternatively just fill in 2 digits, close the process and open it again (without disconnecting internet). All values are being saved in draft immediately after being entered. When process is opened again there is a check to see which values has been already filled in (its only checks against empty string) and opens next substep. However when you try to actually submit it (send it to BE) you do get an error explaining what's wrong.

Potential improvement would be to also validate existing draft values before navigating user to substep. I'm looking into it.

@MrMuzyk thanks for clarifications.
Frankly I think its fine since we get the error from the backend. I think changing the way validation happens is a change which should be discussed more rather than a quick fix.
In that case I would be fine with closing this

I agree, let's close it and focus on real problems.

Thanks


Adaptive Card response is throwing an error

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
Bot Builder 4.9
Describe the bug
The immediate response after an adaptive card is throwing an error in the method 'OnTurnAsync' Method
To Reproduce
Steps to reproduce the behavior:

Send an Adaptive card
Wait for a response

Expected behavior
'OnTurnAsync' should pass the response eventually onto the next step
Additional context
Error Message : Error reading JArray from JsonReader. Current JsonReader item is not an array: StartObject. Path 'previousBotResponse.$values[1].attachments.$values[0].content.body
[bug]

Thank you for the report, @Shubhankar-Nath. I would like to help, but I have a few questions.

When you say "Adaptive Card response" are you talking about the HTTP response to the request that sends the card to the channel, or are you talking about an action the user sends to the bot by clicking on the card?
What channel are you using?
What is the JSON of the card?
What is the relevant code that reproduces the issue?
What line throws the error?

When you say "Adaptive Card response" are you talking about the HTTP response to the request that sends the card to the channel, or are you talking about an action the user sends to the bot by clicking on the card?
== Response the user gives after receiving/or by the text field in a Card.
What channel are you using?
== Bot emulator and Iframe to a webpage
What is the JSON of the card?
==Attached
ProblemForm.txt
What is the relevant code that reproduces the issue?

The error is at line 48 from the screenshot. It calls the 'OnTurnAsync' of ActivityHandler class in namespace namespace Microsoft.Bot.Builder
Instead the message being received in the next waterfall step I get this above mentioned error.
What line throws the error?

@Shubhankar-Nath - I would like to help, but you still haven't given us enough information. There doesn't seem to be anything wrong with the Adaptive Card, so the problem is most likely in the bot code that handles the Adaptive Card action. You haven't given us that code, and the code you have given us is very generic and can't be the actual cause of the problem. Here are some rules to follow in general to make it easier to help you:

Never post a screenshot of code
Always post code that's formatted correctly for Markdown
Always post hyperlinks to all relevant web pages, especially documentation you have referred to

For your specific case, here are some things we need to know:

When you say your channel is "Iframe to a webpage" it sounds like you might be using Web Chat. Are you using Web Chat?
Card actions are sent to the bot as message activities. Can you provide your code that handles message activities?
Since you mentioned waterfall steps, can you provide your waterfall code?

@Shubhankar-Nath - Are you still working on this?

@v-kydela
Yes still facing the issue.Channel is WebChat
` public async Task SearchProblemInQnA(WaterfallStepContext sc, CancellationToken cancellationToken)
{
// setting the response into state2.RegisteredCarNumber
var state2 = await _accessor2.GetAsync(sc.Context, () => new DetailsModel(), cancellationToken);
state2.RegisteredCarNumber = sc.Context.Activity.Text;
var qnaMaker = new QnAMaker(new QnAMakerEndpoint
{
KnowledgeBaseId = "2946****-6cd7--8375-6a9b12b5bfab",   ///_settings["QnAKnowledgebaseId"],
EndpointKey = "259b-6d05-416c-81d6-6ac9d1d7",   ///_settings["QnAEndpointKey"],
Host = "https://hack*****istant-qnahost-5s2.azurewebsites.net/qnamaker"  ////_settings["QnAEndpointHostName"]
},
null,
httpClient);
        var options = new QnAMakerOptions { Top = 1 };
        sc.Context.Activity.Text = state2.ProblemDescription;

        // The actual call to the QnA Maker service.
        var response = await qnaMaker.GetAnswersAsync(sc.Context, options);

        //Making a card
        if (response[0].Answer.Contains('{'))
        {
            var templeteJson = File.ReadAllText(Path.Combine(new string[] { ".", @"wwwroot/Cards/QnACard.json" }));
            AdaptiveCard adaptiveCard = AdaptiveCard.FromJson(templeteJson).Card;
            var cardAttach = new Attachment()
            {
                ContentType = AdaptiveCard.ContentType,
                Content = adaptiveCard,
            };
            await sc.Context.SendActivityAsync(MessageFactory.Attachment(cardAttach), cancellationToken);
        }
        else
        {
            if (response.Length > 0)
            {
                await sc.Context.SendActivityAsync("Please find the results below: \n\n" + MessageFactory.Text(response[0].Answer).Text);
            }
            else
            {
                await sc.Context.SendActivityAsync("Sorry, we did not find any match in FAQ's\n\nPlease try again");
            }
        }

        //Send a card if it helped
        #region Yes/No Card
        List<CardAction> actions = new List<CardAction>();

        CardAction ca2 = new CardAction();
        ca2.Type = ActionTypes.ImBack;
        ca2.Title = "YES";
        ca2.Value = "YES";

        actions.Add(ca2);

        CardAction ca = new CardAction();
        ca.Type = ActionTypes.ImBack;
        ca.Title = "NO";
        ca.Value = "NO";

        actions.Add(ca);

        HeroCard card = new HeroCard();
        card.Title = "Did it help your problem ?";
        card.Buttons = actions;

        Attachment attachment = new Attachment()
        {
            ContentType = HeroCard.ContentType,
            Content = card
        };
        var suggestioncard = MessageFactory.Attachment(attachment);
        #endregion

        await sc.Context.SendActivityAsync(suggestioncard);
        var dialogresult = new DialogTurnResult(DialogTurnStatus.Waiting);
        return dialogresult;
    }

`
This is the code that is generating my Adpative card as response. Bot emulator is able to render the card correctly but Whatever is typed by the used immediately after posting an adaptive card enter the catch block below:
`public class DefaultActivityHandler : TeamsActivityHandler
where T : Dialog
{
private readonly Dialog _dialog;
private readonly BotState _conversationState;
private readonly BotState _userState;
private readonly IStatePropertyAccessor _dialogStateAccessor;
private readonly IStatePropertyAccessor _userProfileState;
private readonly IStatePropertyAccessor _detailsModel;
private readonly LocaleTemplateManager _templateManager;
    public DefaultActivityHandler(IServiceProvider serviceProvider, T dialog)
    {
        _dialog = dialog;
        _dialog.TelemetryClient = serviceProvider.GetService<IBotTelemetryClient>();
        _conversationState = serviceProvider.GetService<ConversationState>();
        _userState = serviceProvider.GetService<UserState>();
        _dialogStateAccessor = _conversationState.CreateProperty<DialogState>(nameof(DialogState));
        _userProfileState = _userState.CreateProperty<UserProfileState>(nameof(UserProfileState));
        _detailsModel = _userState.CreateProperty<DetailsModel>(nameof(DetailsModel));
        _templateManager = serviceProvider.GetService<LocaleTemplateManager>();
    }

    public override async Task OnTurnAsync(ITurnContext turnContext, CancellationToken cancellationToken = default)
    {
        try
        {
            await base.OnTurnAsync(turnContext, cancellationToken);

            // Save any state changes that might have occured during the turn.
            await _conversationState.SaveChangesAsync(turnContext, false, cancellationToken);
            await _userState.SaveChangesAsync(turnContext, false, cancellationToken);
        }catch(Exception e)
        {
            // Getting exception after adaptive card
        }
    }`

@Shubhankar-Nath - I asked for the code that handles message activities. Do you have an OnMessageActivityAsync method?
And while you did provide some of your waterfall code, you said the error is being thrown from the next step. What is the code for the next step?

@Shubhankar-Nath - Do you still need help?

I am closing this issue due to inactivity. We are still happy to provide help if you need any, though it may be a good idea to make a new post if that's the case. Please have a look at this to find out the best places to ask for help, and consider asking questions on Stack Overflow.


Avoid retry on non-recoverable errors.

Only retry when calling  seriesCache.get().

Can you include the rationale for this change?

Still work in progress - as I haven't provided rationale yet.

@jkohen ready for review. please take a look on the new change. The rationale is updated in 1st comment of the PR.


Stats could be less verbose

I really don't need to use that many variables. I should also look into case/switch statements and whether they exist in Perl.

They do exist. Some other commands, like $help, should be changed to use it as well.


Request for making some Add-ons like Patient Pofile picture

🚀 Feature Proposal
As In India there are some hospitals that ask for a profile picture of the patient while registration just for security issues.
While adding a new patient in the database
with there details there should be an option of clicking a live picture of the patient or selection a picture form system storage.
It would make decrease the duplication of the patient registration by different name and id.
Motivation
Decreases the duplication of patient and storage capability increase
Example
When Students Register for Common Entrance Exam for any University https://testservices.nic.in/ExamSys/root/applyfor.aspx

I will try my best to achieve this stone

Hi. This is surely one of the feature that we will have. However it is too early to implement it right now: we still need some backend stuff to be done in order to allow file storage. We can use this issue to refer to. Thanks for pointing this out!


TravisCI tests fail due to Github API rate limits (continuous integration)

Running tests on Travis CI randomly fail, due to Github API rate limits. This results in a 403 response.

For unauthenticated requests, the rate limit allows for up to 60 requests per hour. Unauthenticated requests are associated with the originating IP address, and not the user making requests.

To fix the issue, use authenticated requests instead.

For API requests using Basic Authentication or OAuth, you can make up to 5000 requests per hour.

In the Travis configuration you can use an encrypted environment variable for your secret key.  See documentation
I recommend creating a new (phpbrew-robot) Github user for this, that doesn't have any write privileges to these repositories.

Thank you for your advice. I have followed your suggestions, and it's now under testing. Hopefully the CI will finally go back to green after a very loooooooong time.
@c9s I've created account @phpbrew-bot for this purpose. However, this account is bind with my personal email address. so if you have any better choice, e.g. a phpbrew-related domain, I really appreciate it if you can provide a email account for this shared account. btw, could you drop me an email so I can send the password of that account to you?


MAT-5645: Basic Chip Display

MADiE PR
Jira Ticket: MAT-5645
(Optional) Related Tickets:
Summary
All Submissions

[x] This PR has the JIRA linked.
[x] Required tests are included
[x] No extemporaneous files are included (i.e Complied files or testing results)
[x] This PR is into the correct branch.
[x] All Documentation as needed for this PR is Complete (or noted in a TODO or other Ticket)
[x] Any breaking changes or failing automation are noted by placing a comment on this PR.

DevSecOps
If there is a question if this PR has a security or infrastructure impact, please contact the Security or DevOps engineer assigned to this project to discuss it further.

[ ] This PR has NO significant security impact (i.e Changing auth methods, Adding a new user type, Adding a required but vulnerable package)
[ ] All CDN/Web dependencies are hosted internally (i.e MADiE-Root Repo)

Reviewers
By Approving this PR you are attesting to the following:

Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
The tests appropriately test the new code, including edge cases
If you have any concerns they are brought up either to the developer assigned, security engineer, or leads

Addition of Code system display name for chips added.


Suggest make style file can be configured via option or path

As far as I can understand you wanted something like a loadStylesFromFile(filename) method.
Is this correct?

Something like a dot file will be great.

Something like a dot file will be great.

Where would that dot file reside?

Closed due to lack of feedback


Upgrade admin-on-rest to the 'next' version

TODO:

[ ] Update react-admin library to >= 2.9.0 (see #78).
[ ] Working blog posts example

TODO:

[x] admin-on-rest 1.x -> react-admin 2.x
[ ] Working posts example
[ ] Working login using jwt

Version v0.9.0 had a working posts example and jwt login implemented on admin-on-rest. You can use it as a reference.

https://github.com/TaitoUnited/full-stack-template/commit/81244f4551a408ebd5ab49ab6d4ff31b8c82e4d4 should do it


add dependabot

This PR adds dependabot to keep with with dependencies.

Let's skip this for now, we don't need to constantly update dependencies since this is a library, so the mix.lock only matters when working on this library itself, and won't matter for things that depend on this.


[Question] How to populate firebase auth with current user?

Hey, thank you for this fantastic library. I have ditched next-auth and migrated to this because the integration is so simple.
But I have a question; looking at the starter example, you suggested that the app should only rely on the server token and use inMemoryPersistence to fix the inconsistency issue #143 #163; is there an escape hatch when we want to have the firebase auth populated with the current user (specifically, how to construct the firebase User object)  to use the firebase API like reauthenticateWithCredential (I need this API to update re-check the current user password when updating it). I thought I could use the low-level methods from getFirebaseAuth, but apparently, those construct a pure object, not the full User object.
Thank you for the help.

Hey @trungtin!
Could you share some code where you update user password? I am interested in methods that you're using.
My guess (not sure it applies 100% to your use case) is that you should re-call /api/login endpoint with fresh idToken just after you update user password, but for that I would have to look at some code

import {
  EmailAuthProvider,
  reauthenticateWithCredential,
  updatePassword,
  getAuth,
} from "firebase/auth";

import { loginWithCredential } from "../shared/api"; // <-- exactly like the code in starter example

async function updateUserPassword({ currentPassword, newPassword }) {
  try {
    const user = getAuth().currentUser; // <-- this is what I needed for the subsequent calls
    const credential = EmailAuthProvider.credential(
      user?.email!,
      currentPassword
    );
    await reauthenticateWithCredential(user!, credential);
    // ^ reauthenticate to check for current password, also needed for refreshing the token for updatePassword call
    await updatePassword(user!, newPassword);
    const updatedCredential = await reauthenticateWithCredential( // <-- re-authenticate for firebase sdk
      user!,
      EmailAuthProvider.credential(user?.email!, newPassword)
    );
    loginWithCredential(updatedCredential); // <-- finally, re-login with next-firebase-auth-edge
  } catch (error: any) {
    console.error("Error updating password:", error);
  }
}

Here's basically what I'm using for updating user passwords. This works, but requires setting the firebase-auth persistence to browserLocalPersistence, my question is how to implement this functionality with inMemoryPersistence
Thank you.

Thanks for code samples @trungtin !
That's pretty nice use-case that I haven't yet covered.
In order not to use inMemoryPersistence with your scenario, instead of using getAuth().currentUser, you should fetch user credentials using signInWithCustomToken.
To do that, you would have to extend AuthContext to also expose user custom token, like so:
export interface User extends UserInfo {
  customToken: string; // <--- this
  emailVerified: boolean;
  customClaims: Claims;
}

export const toUser = ({token, decodedToken}: Tokens): User => {
  const {
    uid,
    email,
    picture: photoURL,
    email_verified: emailVerified,
    phone_number: phoneNumber,
    name: displayName,
    source_sign_in_provider: signInProvider
  } = decodedToken;

  const customClaims = filterStandardClaims(decodedToken);

  return {
    uid,
    email: email ?? null,
    displayName: displayName ?? null,
    photoURL: photoURL ?? null,
    phoneNumber: phoneNumber ?? null,
    emailVerified: emailVerified ?? false,
    providerId: signInProvider,
    customClaims,
    customToken: token // <--- Pass `token` from tokens object. It can be referred to as `custom token`
  };
};

Then, inside client component, you can use it like so:
  const {user} = useAuth();

  async function changePassword(currentPassword: string, newPassword: string) {
    if (!user) {
      return;
    }

    const auth = getFirebaseAuth();
    const {user: firebaseUser} = await signInWithCustomToken(auth, user.customToken);
    const credential = EmailAuthProvider.credential(
      firebaseUser.email!,
      currentPassword
    )
    await updatePassword(firebaseUser, newPassword);
    const updatedCredential = await reauthenticateWithCredential( // <-- re-authenticate for firebase sdk
      firebaseUser,
      EmailAuthProvider.credential(user?.email!, newPassword)
    );
    await loginWithCredential(updatedCredential);
  }

I haven't tested this code yet, but it seems to be what you're looking for.
Could you test it and let me know?

I will close the issue now. Feel free to reopen if you face some problems with the approach I proposed


PS should include a tool like Microsoft's handle.exe / Unix fuser

Summary of the new feature/enhancement
Particularly on Windows, users frequently need to find the process that's stopping them from writing or modifying a file or directory
Is this a popular need?
Yes. See this stackoverflow post
What do people do now?
Microsoft distributes handle.exe, a cmd based tool from Systernals. Various community hacks also exist. Unix users scrape output from fuser.
Proposed technical implementation details (optional)
Nothing specific, just to be able to target a file or directory and have a bunch of process objects as output like get-process.
It would also be good to update Microsoft's handle page to point to a PS release that includes the functionality.

Nothing in handle screams "cmd-based" to me... it doesn't look like it's based on cmd's code, or batch files, or anything else that cmd is.

@mikemaccana Users can run handle.exe / Unix fuser from PowerShell - why do we need the feature being a cmdlet?

@dhowett handle outputs text, not powershell objects. You'll find various user-made handle powershell wrappers to fix that in the Stack Overflow thread linked above.

@mikemaccana The cmdlet must be portable. If you know what is underlying APIs we could use please pull a RFC with your proposal in PowerShell-RFC repo.

@iSazonov Understood re: portability. Alas I'm not a C# (or any CLR language) developer, just a PS user (and node / python person) so I don't have the skills to implement this myself.

@dantraMSFT This is not a question and has not been answered. Can you please reopen?

@iSazonov  This is not a question and has not been answered. Can you please reopen?

@mikemaccana Your suggestion is not rejected. We have nothing to track here. If somebody want to discuss this in depth he/she should write RFC.

Your suggestion is not rejected.

OK

We have nothing to track here.

Then why do you have nothing to track?

Because it should be addressed in RFC repo.

@iSazonov As mentioned, I'm not a .net developer. You've mentioned an RFC should only be raised if I do know what the underlying APIs on all supported platforms are, so given this restriction I can't open one. What do you want me to do?

@mikemaccana To implement a new feature in the repo, we need to get a lot of votes from the community. Your request is open for voting and discussion. Also, anyone can post an RFC to move forward.
You could share this in community resources you use (forums, chats, twitter, ...) to get community support.

given this restriction I can't open one

We haven't such restriction. You could pull the RFC. But main question will be how implement this. PowerShell Core is based on .Net Core and I don't see an API which supports this. So we will have to implement workarounds in the repo that is a huge work and will be approved by PowerShell Committee  only if they will see a wide approval from community.


Feature request - Allow optionally setting duration based easings

Per our spectrum conversation:
I’m attempting to create a continuous sine wave animation. I’ve gotten pretty close with https://codesandbox.io/s/lx5nvqz6pq but noticed that the animation pauses between keyframes, likely due to friction deceleration. Would it make sense to optionally allow users to specify a duration-based easing for these kinds of zero-delay, linear, continuous animations?
(FYI - my initial approach was a Trail based animation similar to https://codesandbox.io/s/q9lozyymr9 but I needed the first element to continue along the sine wave before the last element finished animating, and even then each element’s animation would be affected by the preceding element so the amplitude of the sine wave would decrease the further they were from the first element.)
Thanks again for this delightful library!

@jimmy-guo
Added it ...

I just need to think about how to add this without adding lots of weight to the lib. Maybe as an add-on package. But the way it works now is pretty cool anyway because now it could receive any implementation, from easings, to rk4 springs, to ios oscilator spring, etc.

PS. right now this is how it looks
import { Spring, Keyframes, animated, TimingAnimation, Easing } from 'react-spring'

<Keyframes
    reset
    native
    keys={items}
    impl={TimingAnimation}
    config={{ duration: 2000, easing: Easing.linear }}
    script={async next => {
        while (true) {
            await next(Spring, {
                from: { radians: 0, color: '#247BA0' },
                to: { radians: 2 * Math.PI },
            })
        }
    }}>
    {Content}
</Keyframes>

@jimmy-guo
it's merged:
https://codesandbox.io/s/q9lozyymr9
https://github.com/drcmda/react-spring/tree/master/src/addons
For kicks i also added IOS harmonic oscillator spring (which has an additional mass attribute), implementations are fully exchangeable now.


Update he.coffee

+I'm back
+add tips translation
+add partial "clans" new feature localization (see notes in code)

Nice work :+1: It's nice to see you again :)


multiple variable search

Is it possible to search two attributes at the same time.
aka Get-ServiceNowIncident -MatchContains @{assignment_group='Corporate' -and location='texas' -and active='true'}
I would like to get specific tickets using multiple hashtables

Yes, but MatchContains expects a hash table so the format would be @{assignment_group='Corporate';location='Texas';Active='True'}.


Build: Assets and ResponsiveContext not working

I'm new to Gatsby, so maybe it's my fault but it looks like the build is not working.
Steps to reproduce:

Clone project
Run npm install
import { ResponsiveContext, Text } from 'grommet' in Layout.tsx
Add ResponsiveContext example

...
const Theme = ({ children, themeType, themeSwitch }) => (
  <Grommet theme={THEMES[themeByThemeType(themeType)]}>
    <Grommet theme={siteTheme}>
      <ResponsiveContext.Consumer>
        {(size) => {
          return (
            <>
              <GlobalStyle />
              <Box direction="column" align="center">
                <Box width="xlarge">
                  <SiteHeader />
                  <Text>{size}</Text>
                  <main>
                    <StickyContainer>
                      <Box direction="row-responsive">
                        <Box
                          basis="large"
                          flex="grow"
                          direction="row-responsive"
                        >
                          {children}
                        </Box>
                        <Box basis="medium">
                          <Sticky>
                            {({ style }) => (
                              <aside style={style}>
                                <Sidebar
                                  siteConfig={siteConfig}
                                  themeSwitch={themeSwitch}
                                  themeType={themeType}
                                />
                              </aside>
                            )}
                          </Sticky>
                        </Box>
                      </Box>
                    </StickyContainer>
                  </main>
                </Box>
              </Box>
              <SiteFooter />
            </>
          );
        }}
      </ResponsiveContext.Consumer>
    </Grommet>
  </Grommet>
);
...

Run npm run dev --> everything works like expected
Run npm run build and open index.html from public folder --> The screen size remains medium on every screen size and the assets are not loaded

After further investigation I think this is a general Gatsby-Grommet issue.


Switch to using setup.cfg for PyPI metadata

Examples:

https://github.com/pallets/flask/blob/main/setup.py
https://github.com/pallets/flask/blob/main/setup.cfg

@jeffwidman I think this is closed by #666


Feat/connect autocomplete

This connector now matches the react-instantsearch API, it exposes indices as an array and a refine function to update the query 👍
I've build a simple example with https://selectize.github.io/selectize.js/ which integrates well.

[x] documentation
[x] tests

This PR replaces #2651

travis build fails because there's a console.log in scripts/release.js

widget is here if you want to try it :)

Rebased on feat/2.8 & changed pull request base branch.

Inside the stories the select component keep the items in memory. It means that the more we search the longer is the list (see below). We can use the clearOptions() to remove all options before each a new render.

Nice catch, internally selectize keeps in memory options added and re-do a search into them.

Note that the autocomplete widget is always tied with the main search. For example if we want to only target an different index than the main one we can't. We will need to use another instance and sync them together. The use case I'm thinking of is the e-commerce demo with the suggestions.

Yes it's true, good point, as a solution we can also provide another option to do not use the "main index" and so the user has to specify at least one index in the options. What do you think?

Tests are fixed now
Added an example of multi-index search: https://deploy-preview-2841--algolia-instantsearch.netlify.com/v2/dev-novel/?widgets=jquery&selectedStory=Autocomplete.Multi index

Added an example of multi-index search: https://deploy-preview-2841--algolia-instantsearch.netlify.com/v2/dev-novel/?widgets=jquery&selectedStory=Autocomplete.Multi index

Is it expected to not have a search bar?

I don't think it will solve the issue because even with only a different index the query will still be set on the main helper. For the suggestions example it means that the "suggested" hits and the "regular" hits will always be in sync.

If I sum we have 3 cases (with my thoughts):

If it's an independant search. Having independant searches seems fine.
If if you want to sync the main results and the autocomplete results but with different indices, the idea to have an option for "not the main index" seems to solve it
If you have the main index + others and everything is synced, it's the default case

Did I miss something @iam4x @samouss ?

If it's an independant search. Having independant searches seems fine.

Yes it's fine.

If if you want to sync the main results and the autocomplete results but with different indices, the idea to have an option for "not the main index" seems to solve it

We can also go in the other directions, do not assume that the autocomplete will be on the main index. WDYT? I'm still fine with the current proposal, it's just that I didn't think about it before.

If you have the main index + others and everything is synced, it's the default case

Yes indeed, depends on your answers on the question above.


Mode 1363

...eContent() logic in ModeShapeSubsystemParser

Merged the changes into the '3.x' branch (after fixing some dependency errors).


Error when trying to get nonce

I'm trying to update a record using
./pdns-client -s https://pdns.manager/ -d _test.mydomain.com -i 10 -c "Success"
Result: ERROR: Error when trying to get nonce

Can you call the script like that and supply the output? bash -x ./pdns-client ....

Here you go:
`+++ dirname ./pdns-client
++ cd .
++ pwd

SD=/home/dns/pdns-client
SERVER=
DOMAIN=
ID=
CONTENT=
KEY=
getopts s:d:i:c:k:h opt
case $opt in
SERVER=https://pdns.manager/
getopts s:d:i:c:k:h opt
case $opt in
DOMAIN=_test.mydomain.com
getopts s:d:i:c:k:h opt
case $opt in
ID=10
getopts s:d:i:c:k:h opt
case $opt in
CONTENT=Success
getopts s:d:i:c:k:h opt
check_dependencies
openssl version
curl -V
jq --version
validate_arguments
[[ ! https://pdns.manager/ =~ ^https?://.+/$ ]]
[[ ! _test.mydomain.com =~ ^.+$ ]]
[[ ! 10 =~ ^[0-9]+$ ]]
[[ ! Success =~ ^.+$ ]]
resolve_keyfile
'[' -n '' ']'
openssl rsa -in pdns.private.pem -check -noout
KEY=pdns.private.pem
read NONCE
++ echo '{}'
++ jq -c '.domain="_test.mydomain.com"'
++ jq -c '.id="10"'
++ jq -c '.content="Success"'
++ curl -s --data-binary @- 'https://pdns.manager/api/remote.php?getNonce'
++ jq -r .nonce
'[' -z ']'
exit_error 'Error when trying to get nonce'
echo 'ERROR: Error when trying to get nonce'
ERROR: Error when trying to get nonce
exit 1
`

It seems that the response of pdns-manager is wrong. Can you insert tee /tmp/pdns-client.log |\ between line 122 and 123 try it again and post the content of /tmp/pdns-client.log ?

it says:
{"domain":"_test.mydomain.com","id":"10","content":"Success"}

Did you change anything else in the script? It seems that you inserted the line betwen 121 and 122 it should be in the line after curl?!

my fault. I was already trying to debug a bit into this. Placing it after curl, /tmp/pdns-client.log is an empty file.

Ok in this case the problem should not be in the pdns-client module. The request seems to be fine but the server does not anwer.
Have you any errors in your PHP log?
I can not reproduce the issue here. Next step in debugging would be the check where in api/remote.php the server leaves the right path. It should end in the condition in line 91 to be true. If you are familiar with PHP you can check for yourself where this is, maybe add additional debug messages, if not it would be helpful to supply information about your setup, such as versions of OS, PHP and MySQL.

I just tried to get some debug info from nginx. Looks like the whole thing does not play nice with http/2
2016/10/13 16:09:12 [info] 32008#32008: *46192 client sent stream with data before settings were acknowledged while processing HTTP/2 connection, client: <IP_ADDRESS>, server: <IP_ADDRESS>:443

I did not test the Software with Nginx nor with HTTP/2.
Maybe you can add as workaround the --http1.1 option to both curl calls in the shell script.
I currently have no time to test and improve compatibility with Nginx and HTTP/2, but if you want to make a pull request feel free to do so.

You nailed it; Getting a nonce back now.
Some research showed some bugs in Firefox and Safari and some Android HTTP showing the same behavior. Either some weirdness of Nginx H/2 implementation or some weirdness on the side of curl.

From some mailing list:

You're not the first one, so the problem was resolved by this change:
http://hg.nginx.org/nginx/rev/ce94f07d5082 (nginx 1.11.0)

Ok so it was a bug in Nginx. Then I will close this issue now.

I would make it some kind of a flag as that bug still seems to be present in the most recent version here.

A configuration file for pdns client seems to be a good option. I will reopen it and mark it as feature request.

I started to get the same error. I am using Apache 2.4.18-2ubuntu3.and my debug logs says
select protocol from , choices=http/1.1 for server powerdns.hackernet.se so


Ear Plugin: XML Schema / xmlns in Ear Descriptor for Java EE 8, Jakarta EE 9 and Jakarta EE 10 is missing / invalid

Issue type
Polishing
Problem description
I would expect the version Property for ear.deploymentDescriptor to fully support values up to "10" (Jakarta EE 10).
Currently the documentation of the version property (comment / javadoc in DeploymentDescriptor.java) states it only supports values up to "6" while in fact it currently already properly handles "7".
Higher values lead to a descriptor with invalid/missing xmlns.
A desciptor like this:
ear {
    deploymentDescriptor {
        version = "10" // <--- it currently only fully supports values up to "7"
        applicationName = "example"
        displayName = "Just an Example"
        // ... remaining properties omitted here ...
    }
}

Should generate the descriptor like this:
(With valid xmlns etc. according to the JAVA EE / Jakarta EE Schemas)
<?xml version="1.0"?>
<application xmlns="https://jakarta.ee/xml/ns/jakartaee" xsi:schemaLocation="https://jakarta.ee/xml/ns/jakartaee https://jakarta.ee/xml/ns/jakartaee/application_10.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" version="10">
  <application-name>example</application-name>
  <display-name>Just an Example</display-name>
  ... remaining elements omitted here ...
</application>

However currently the application Element looks like this:
<?xml version="1.0"?>
<application xmlns="" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" version="10">
  <application-name>example</application-name>
  <display-name>Just an Example</display-name>
  ... remaining elements omitted here ...
</application>

xmlns is an empty string for Versions 8, 9 and 10
xsi:schemaLocation is missing for Versions 8, 9 and 10

Context (optional)
The latest XSD Schemas can be found at https://jakarta.ee/xml/ns/jakartaee/
https://www.oracle.com/webfolder/technetwork/jsc/xml/ns/javaee/application_8.xsd (JAVA EE 8)
<application xmlns="http://xmlns.jcp.org/xml/ns/javaee"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      xsi:schemaLocation="http://xmlns.jcp.org/xml/ns/javaee
      	http://xmlns.jcp.org/xml/ns/javaee/application_8.xsd"
      version="8">
      ...
</application>

https://jakarta.ee/xml/ns/jakartaee/application_9.xsd (Jakarta EE 9)
<application xmlns="https://jakarta.ee/xml/ns/jakartaee"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      xsi:schemaLocation="https://jakarta.ee/xml/ns/jakartaee
      	https://jakarta.ee/xml/ns/jakartaee/application_9.xsd"
      version="9">
      ...
</application>

https://jakarta.ee/xml/ns/jakartaee/application_10.xsd (Jakarta EE 10)
<application xmlns="https://jakarta.ee/xml/ns/jakartaee"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      xsi:schemaLocation="https://jakarta.ee/xml/ns/jakartaee
      	https://jakarta.ee/xml/ns/jakartaee/application_10.xsd"
      version="10">
      ...
</application>

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritized by them.

Thank you for the pull request!

Thank you for the pull-request @p-schneider, this will be in Gradle 8.5.


BuildXL nuget repo deletion breaks building older releases

The BuildXL nuget repo was recently deleted. This broke source-build (eg https://github.com/dotnet/source-build/issues/4634) which was easily fixed.
However, this also broke online builds of older versions of .NET, which all require fixing some way or the other.
For example:

I wanted to rebuild 9.0 RC 1 with a custom fix to re-bootstrap 9.0 RC 2. But RC 1 is broken due to missing BuildXL repo and can't be fixed (due to being a point-in-time release)
Someone on my team is trying to test an older commit of .NET 8 - 8.0 RC 1 specifically - to see if it reproduces a bug. But the 8.0 build is broken because BuildXL is removed.

I know that, in general, expecting older releases to build out of the box is not realistic. Many bugs, tooling support and compatibility fixes are missing in older releases. However, this change accidentally breaks all older releases.

Transferring to msbuild as this was caused by https://github.com/dotnet/msbuild/pull/10765


problems with autocompete

We have written the needed data into your clipboard because it was too large to send. Please paste.

Please provide reproducable steps so we could do something with this issue.
Here's are notes on how to file good issues https://github.com/Microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions
Thanks


Panic when ranging over segments

https://play.golang.org/p/hmbq78PZbO
Why is the slice not the same size as the parsed segments of the playlist? How can I tell when I am about to go over the last segment and cause a panic?
Many thanks!

Ok, discovered I can check for against nil, but why make it so odd to use?

It's probably more of a legacy issue than anything else, we do auto extend now, but just by doubling of size.
To get the count of elements use https://godoc.org/github.com/grafov/m3u8#MediaPlaylist.Count
To loop them I normally check for nil and then break.


(BUG) "Invalid Device" when trying to teleport back from RFTDim to Overworld

Title says it all. I teleported into an RF Tools Dimension, created by me,  only to get stuck there because the dialing device & transmitter I setup there (I was in creative mode) told me "Invalid Device". Couldn't get back to the overworld and had to resort to slash commands.

Can you show a screenshot of that error and the setup where you got it from?

It is quite literally just a Dialing Device, Matter Transmitter, and Matter Receiver .. on both ends. Creative Capacitor supplying the power on both ends. In each setup, Matter Receiver is on the left, Dialing Device in the middle, and Matter Transmitter on the right. (No destination analyzers, nothing else like that.)
The first dimension we tried to create was an Eternal Noon void dimension. It was a void dimension, but.. not eternal day. Anyway... when trying to connect the Matter Transmitter there named "Test1" to the Matter Receiver back at spawn (called "Spawn"), we get "Invalid Device!" as an error.
That is literally it. Just a simple little popup message in the Dialing Device.

Same here! Confirmed I am using All The Mods Pack. I'll get a SS on the next dim.

So after doing some other things and using pads and Adv Charged porter for a bit I went back to the dim while recording and was going to show my viewers what happened... it worked with no issue. go figure lol. So some how it fixed its self. Not sure if this is any help or not..

yes, seems this is an intermittent problem that only happens some times. At this moment I have no clue what is causing it. I'm busy debugging this right now but I have not been able to reproduce it yet

Ok good news. I can reproduce this on a server setup. So there is something wrong when RFTools Dimensions is used on a server. I'm on it

If it helps, it will sometimes (but not always) clear up on reboot.

Nah problems that happen all the time are usually easier to debug

Ok, got something. Apparently for some reason the client thinks it is in the overworld while the player is in the rftools dimension. Debugging...

This would explain why the "Coordinates Selector" tool from Ender IO thought the coordinates I was selecting were in the Overworld (when it was in an RF tools Dimension).

Ok, I know what's causing the bug. Just need to find out how to fix it safely and properly

FOUND AND FIXED!. Preparing release now


Adding Stat Tracking

I have added a basic stat tracking package to squad JS. I find that it would be useful if squad JS had this by default hence me making this PR

First concerns is the eslint disable flags. Why is that there?


Data type translations

So I hope this isn't a repeat question or anything; I looked around but didn't see anything.
So there are a few data type translations I have some issues with, or more accurately, enhancement request for. Currently list<type>, set<type>,  and map<typeA, typeB> get translated to ArrayList, HashSet, and HashMap in Java, respectively. Why aren't those translated to List, Set, and Map java interfaces? Or is there anyway to add some options or flags so I can get those types of translations? Having to program to the actual implementation of these interfaces make things much less flexible. It seems like Objective-C doesn't really have this problem. NSArray doesn't look like it has an interface. But for Java this would make things so much more flexible and extensible, and all that jazz.
So is there a work around or way to get djinni to produce those java interfaces, or would it be a feasible and reasonable feature request?
Thanks so much for you time.
-Sterling

I'm not sure what you mean about having to program the implementation of the interfaces. Do you mean that you have some other type of List and want to pass it as an argument to a Djinni interface without copying it into an ArrayList?  I can see that being a good idea, but it's not something currently supported.  I think the reasoning was that those specific concrete types are always what Djinni creates when it marshals data into Java, and it's useful to expose that to users who can take advantage of it (e.g. if you received a list from Djinni and wanted to do an efficient binary search on it).  In the other direction, I think that's not relevant, though the marshaling code does make assumptions about the specific container types for efficiency reasons.  For instance, the marshaling of lists makes a lot of calls to get(int) which would be inefficient on a LinkedList.  That could be changed to use an Iterator (which is already used for marshaling maps).
So, I think switching to generic interfaces would be a reasonable feature, though it would have some downsides for people currently making use of the specific types, so it might need to be a new configuration option.  An alternative you could try without new features would be to define whatever type of list you want to use as an external type using a YAML file.  You'd need to write some of your own marshaling code based on the existing examples, but then you could use whatever type you want.