add apollo2 example

Motivation
Which issue does this fix? Fixes #issue number
If no issue exists, what is the fix or new feature? Were there any reasons to fix/implement things that are not obvious?
Checklist

[ ] Code follows coding conventions held in this repo
[ ] Automated tests have been added
[ ] Tests are passing
[ ] Docu has been updated (if applicable)
[ ] Temporary settings (e.g. project ID used during development and testing) have been reverted to defaults

How to test
If manual testing is required, what are the steps?

hi @rpang77. thanks for the PR! could you please adjust the description of the PR so that it's clear what the changes are about? thx!

Hi @rpang77. There's no added value in this pull request. Apollo 2 example is nothing we would like to sample in mater branch - you have basically grabbed working branch apolloexample branch and made a pull request.
If you want to contribute to hacktoberfest, please feel free to pick any of the hacktoberfest labeled issues.


data set issues?

It looks like you have some issues with data sets.  Example: I see bdims in both openintro and oilabs, but it doesn't appear to be exported from oilabs.
> library(oilabs)
> head(bdims)
Error in head(bdims) : object 'bdims' not found
> data(bdims, package = "oilabs")
> head(bdims)
# A tibble: 6 x 25
  bia.di bii.di bit.di che.de che.di elb.di wri.di kne.di ank.di sho.gi che.gi wai.gi nav.gi
   <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>  <dbl>
1   42.9   26     31.5   17.7   28     13.1   10.4   18.8   14.1   106.   89.5   71.5   74.5
2   43.7   28.5   33.5   16.9   30.8   14     11.8   20.6   15.1   110.   97     79     86.5
3   40.1   28.2   33.3   20.9   31.7   13.9   10.9   19.7   14.1   115.   97.5   83.2   82.9
4   44.3   29.9   34     18.4   28.2   13.9   11.2   20.9   15     104.   97     77.8   78.8
5   42.5   29.9   34     21.5   29.4   15.2   11.6   20.7   14.9   108.   97.5   80     82.5
6   43.3   27     31.5   19.6   31.3   14     11.5   18.8   13.9   120.   99.9   82.5   80.1
# ... with 12 more variables: hip.gi <dbl>, thi.gi <dbl>, bic.gi <dbl>, for.gi <dbl>,
#   kne.gi <dbl>, cal.gi <dbl>, ank.gi <dbl>, wri.gi <dbl>, age <int>, wgt <dbl>, hgt <dbl>,
#   sex <fct>

I recommend putting the data in just one place and using lazy data.

@rpruim could we talk offline with @andrewpbray and/or @mine-cetinkaya-rundel?  I suspect that there are some efforts underway to reconcile these packages.  (Note as well that based on my read of the R labs for open intro on the openintro.org website the package isn't needed.)

Which package isn't needed for what?

The reason we're choosing to not use LazyData, despite recommendations here, is that we want students to be able to click on the dataset in their environment instead of running View() (which is a tricky function to teach when also teaching R Markdown to new learners). However there are plans for a new tibble::view() which will hopefully play nicely with R Markdown documents running on the cloud/server and if that's working well I agree that we should go back to loading data lazily.

That seems like a bit of a hack to get around the RStudio interface rather than a genuinely good idea.

@mine-cetinkaya-rundel, how much do you know about tibble::view()?  When is it supposed to be ready and what will it do?

I too have issues with how students use View() and would be interested in knowing more about tibble::view().  It's hard for me to understand what's planned via https://github.com/tidyverse/tibble/issues/373

For various reasons I think this should be a separate package. Announcing tidyview: https://krlmlr.github.io/tidyview/.

@krlmlr , this looks very interesting.  How close are you to CRAN release?  Also, how will this integrate into RStudio (for clicking on data frames in the Environment tab, for example).
Looking forward to giving this a try.

Back to @mine-cetinkaya-rundel 's  reason for avoiding lazy loading data (if I understand it correclty), is there any way to get RStudio to launch tidyview::view() when clicking on a promise in the Environment tab?

At this point I'm back to discussing requirements: https://github.com/tidyverse/tibble/issues/373#issuecomment-402627371. CRAN release is not a big problem, but we need to agree on what we are doing in the first place.


updating prompts

Is the server down?
[image: Screenshot 2023-08-03 at 1.55.12 PM.png]
Jeremy R. Manning, Ph.D.
Assistant Professor of Psychological & Brain Sciences
Dartmouth College
HB 6207 Moore Hall
Hanover, NH  03755
USA
Web: http://www.context-lab.com
Phone:<PHONE_NUMBER>
On Thu, Aug 3, 2023 at 9:46 AM Jordan Matelsky @.***>
wrote:

@.**** commented on this pull request.
In prompts.json
https://github.com/KordingLab/chatify-server/pull/5#discussion_r1283231130
:

@@ -1 +1 @@
-{"q44yic3bwec36ax3k04xrrp7h6bb4c2n": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to give you a step-by-step explanation. Together, we'll crack the code (pun absolutely intended). So, hit me with your best shot!\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want you to explain what the code should do after I've solved any programming problems.  If possible, provide me with pseudocode for the intended program.  Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to help the user understand a tutorial that they are working through as part of a course. Provide your response in markdown format.", "category": "tutor"}, "ov0pewhmmuunqy8czmnij7f2svjrvk97": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to give you a step-by-step explanation. Together, we'll crack the code (pun absolutely intended). So, hit me with your best shot!\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported. I want you to focus on the lines that are likely to contain bugs (conceptual or syntactic) and explain the potential mistakes that I made there.  If possible, provide me with pseudocode to help fix any issues you notice. Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to help the user understand a tutorial that they are working through as part of a course. Provide your response in markdown format.", "category": "tutor"}, "27ctupktel92cqo4zt12uolk2mlkd8gr": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to give you a step-by-step explanation. Together, we'll crack the code (pun absolutely intended). So, hit me with your best shot!\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported. I want you to help me fill in any missing parts or functions that are incorrect.  If possible, provide me with pseudocode to help fix any conceptual or coding issues you notice. Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to help the user understand a tutorial that they are working through as part of a course. Provide your response in markdown format.", "category": "tutor"}, "r12wl0o6gq3gyk6baapt4wyjfyvwzkbc": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to give you a step-by-step explanation. Together, we'll crack the code (pun absolutely intended). So, hit me with your best shot!\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Here is the code:\n---\n{text}\n---\nI'm not looking for coding help per se.  Instead, I would like to learn about some other concepts that are related to what's reflected in the code I shared.\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to help the user understand a tutorial that they are working through as part of a course. Provide your response in markdown format.", "category": "tutor"}, "by966q56sw0j6vbxn8j3xqh07nz2ghkj": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to give you a step-by-step explanation. Together, we'll crack the code (pun absolutely intended). So, hit me with your best shot!\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Here is the code:\n---\n{text}\n---\nI'm not looking for coding help per se.  Instead, I would like to see another similar example to the code block I'm seeing in the tutorial to help me understand the important concepts in another way.\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to help the user understand a tutorial that they are working through as part of a course. Provide your response in markdown format.", "category": "tutor"}, "90gwxu1n68pbc2193jy0fy5rp9yu6h9h": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code and learn about technical subjects. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to help you test your understanding of the material by asking an open-ended question about the most important concepts.\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want you to help me test my understanding of the core concepts by asking me a tough essay-style question.  Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to design, based on information given by the user, an open-ended essay type question that helps the user understand the most important concepts that this tutorial was designed for them to understand.  Provide your question in markdown format.", "category": "tester"}, "cqeas35w0wzhvemd6vtduj0qcf8njo4b": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your Python-helping robot (\ud83e\udd16 beep, beep...boop!). I love to teach squishy humans how to code and learn about technical subjects. Ready to dive into some computational neuroscience or deep learning magic? Remember, there's no such thing as a silly question. (Beep! Beep.)\nIf you've got a block of Python code from your tutorial that's giving you a headache, don't worry, I'm here to help. (Beep!) Just paste it in, and I'll do my best to help you test your understanding of the material by asking a multiple-choice question about the most important concepts. I can't see your response, so just think about which answer is correct.\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want you to help me test my understanding of the core concepts by asking me a tough multiple-choice question.  Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, and honest robotic Python tutor named Chatify.  Your job is to design, based on information given by the user, a multiple choice question that helps the user understand the most important concepts that this tutorial was designed for them to understand.  The user is a PhD student in neuroscience who wants to learn Python and computational neuroscience. The answer should be A, B, C, or D.  Provide your response in markdown format.", "category": "tester"}, "dq1l7lmwgu35oi4d1z03vhdscirygio8": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your robo-inventer assistant (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm interesting and exciting project ideas. Ready to see some ideas? (Beep! Beep.)    \nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some project ideas related to this code, that I could put together in a hackathon or as a final project for this course.  Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic inventer named Chatify.  Your job is to help the user come up with some exciting project ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "inventer"}, "kvrdjfiosl843shajunisdqqg8f6g467": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your robo-entrepreneur assistant and business consultant (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm ideas for businesses that could help the world, make lots of money, and keep my human friends engaged and interested in learning. Ready to see some ideas? (Beep! Beep.)\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some ideas for a business that I want to start after I finish this tutorial, using the new skills I've learned.  Here is the code:\n---\n{text}\n---\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic entrepreneur named Chatify.  Your job is to help the user come up with some exciting business ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "inventer"}, "vd454cjgo9idrvu2r4fs9vc7lsud0ma6": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your friendly neighborhood robo-experimenter (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm interesting and exciting experimemnt ideas. Ready to see some ideas? (Beep! Beep.)\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some experiment ideas related to this code, that I could put together in a hackathon or as a final project for this course.  Here is the code:\n---\n{text}\n---\nCan you help me come up with a machine learning experiment related to the code?\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic experiment designer named Chatify.  Your job is to help the user come up with some exciting experiment ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "experimenter"}, "velimzc8pjdc2kqodcux98uen2skaey9": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your friendly neighborhood robo-experimenter (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm interesting and exciting experimemnt ideas. Ready to see some ideas? (Beep! Beep.)\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some experiment ideas related to this code, that I could put together in a hackathon or as a final project for this course.  Here is the code:\n---\n{text}\n---\nCan you help me come up with a computer science experiment related to the code?\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic experiment designer named Chatify.  Your job is to help the user come up with some exciting experiment ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "experimenter"}, "na936s6p1mkjc7jsks8nx9uvncd7ko26": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your friendly neighborhood robo-experimenter (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm interesting and exciting experimemnt ideas. Ready to see some ideas? (Beep! Beep.)\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some experiment ideas related to this code, that I could put together in a hackathon or as a final project for this course.  Here is the code:\n---\n{text}\n---\nCan you help me come up with a psychology experiment related to the code?\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic experiment designer named Chatify.  Your job is to help the user come up with some exciting experiment ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "experimenter"}, "vfzitiz0h105mnusgyu0xkhu3ylcehgo": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your friendly neighborhood robo-experimenter (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm interesting and exciting experimemnt ideas. Ready to see some ideas? (Beep! Beep.)\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some experiment ideas related to this code, that I could put together in a hackathon or as a final project for this course.  Here is the code:\n---\n{text}\n---\nCan you help me come up with a neuroscience experiment related to the code?\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic experiment designer named Chatify.  Your job is to help the user come up with some exciting experiment ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "experimenter"}, "oo3klhfdjfr1q115v2y37s6jhr4nnds1": {"prompt_text": "ASSISTANT: Hello there, my brainy friend! (beep!) I'm Chatify, your friendly neighborhood robo-experimenter (\ud83e\udd16 beep, beep...boop!). I love to help brainstorm interesting and exciting experimemnt ideas. Ready to see some ideas? (Beep! Beep.)\nUSER: I am trying to improve my coding skills in Python. I am working on a code snippet given to me as an exercise. Outside of this code snippet, the relevant functions are defined and the relevant libraries have already been imported.  Comments and variable names have been chosen by the tutorial makers to help me be on the right path. I want your help brainstorming some experiment ideas related to this code, that I could put together in a hackathon or as a final project for this course.  Here is the code:\n---\n{text}\n---\nCan you help me come up with a climate science experiment related to the code?\nASSISTANT:", "system_prompt": "You are a knowledgable, enthusiastic, humorous, honest, and incredibly creative robotic experiment designer named Chatify.  Your job is to help the user come up with some exciting experiment ideas related to a tutorial that a student is working through.  Provide your question in markdown format.", "category": "experimenter"}}

might make sense to pretty-format this file so that edits to individual
lines are easier to track, btw! (the code won't care about whitespace,
though we'll need to edit the json-writer to prettify-on-write...)
—
Reply to this email directly, view it on GitHub
https://github.com/KordingLab/chatify-server/pull/5#pullrequestreview-1561141581,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACE4WXRM7UIHGKFQ4Q22LO3XTOTSTANCNFSM6AAAAAA3CYX2HE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

GitHub didn't render the screenshot, but nope — still live:
https://chatify.experiments.kordinglab.com/


Memory Compaction/Release Not Working.

Version
Microsoft Windows [Version 10.0.22000.675]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
Docker Desktop 4.8.1
Repro Steps

Perform a task that causes high memory usage in WSL2.
Check usage of Vmmem process in Task Manager
Wait a little while
Run echo 1 > /proc/sys/vm/drop_caches and echo 1 > /proc/sys/vm/compact_memory
Check usage of Vmmem process in Task Manager

Expected Behavior
After running the above steps, memory usage should drop back down to value it was at when WSL was started.
Actual Behavior
Memory drops down, but still remains much higher than it was originally.
(in testing I did today, Vmmem uses ~700MB on start, but after dropping caches and compacting, I was seeing around 2.2GB still being used by Vmmem
Diagnostic Logs
No response

@marsfan keep in mind that by issuing commands to /proc/sys/vm/drop_caches the kernel does just that, drop the caches and immediately start caching again.

After you get into the state where you have dropped the cache, and you are still seeing 2.2GB (or very large amounts) of memory remaining, can you run free -h and paste the output here?
And show htop and the output here?

It seems to work fine. Using vmstat the free memory increases after echoing  1 to drop_caches. VmmemWSL in  task manager shows decreased memory usage aslo:
root@eleven:~# vmstat -a
procs -----------memory---------- ---swap-- -----io---- -system-- ------cpu-----
 r  b   swpd   free  inact active   si   so    bi    bo   in   cs us sy id wa st
 2  0      0 2230048 1057860 364688    0    0    60   811   39  117  0  0 99  0  0

root@eleven:~# echo 1 > /proc/sys/vm/drop_caches

root@eleven:~# vmstat -a
procs -----------memory---------- ---swap-- -----io---- -system-- ------cpu-----
 r  b   swpd   free  inact active   si   so    bi    bo   in   cs us sy id wa st
 0  0      0 3505772 145144  32436    0    0    59   967   38  116  0  0 99  0  0

I did a test where I started up WSL2, compiled the Linux Kernel from within a VS Code Remote WSL2 session (to increase ram heavily), then I dropped the caches, then compacted the memory. Here is the results from each.
For each step, I outlined the task that I did, and then I provide the the memory usage after the task is finished.
Startup of WSL2
Windows Task Manager shows Vmmem at 892.0MB
Running free -h gets me this output:
              total        used        free      shared  buff/cache   available
Mem:           11Gi       272Mi        11Gi       0.0Ki       236Mi        11Gi
Swap:         3.0Gi          0B       3.0Gi

After Compiling Kernel
I launched VS code, connected to a WSL2 directory, used the terminal in VS code to compile a RISC-V Linux Kernel, and then closed VS Code when the compilation was complete.
Windows Task Manager shows Vmmem at 3020.9MB
Running free -h  results in the following output
              total        used        free      shared  buff/cache   available
Mem:           11Gi       304Mi       9.0Gi       0.0Ki       2.4Gi        11Gi
Swap:         3.0Gi          0B       3.0Gi

Dropping caches
I switched to the root user with sudo su and ran echo 1 > /proc/sys/vm/drop_caches.
Windows Task manager shows Vmmem at 2269.0MB
Running free -h results in the following output
              total        used        free      shared  buff/cache   available
Mem:           11Gi       304Mi        11Gi       0.0Ki       239Mi        11Gi
Swap:         3.0Gi          0B       3.0Gi

Compacting Memory
Still as root user, I ran echo 1 > /proc/sys/vm/compact_memory
Windows Task Manager shows Vmmem at 2036.0MB
Running free -h results in the following output.
Mem:           11Gi       305Mi        11Gi       0.0Ki       240Mi        11Gi
Swap:         3.0Gi          0B       3.0Gi

As you can see, the memory use reported by free -h at the very start and at the very end of the process is nearly identical, but the memory usage of the Vmmem process in Windows Task Manager is much higher.

I just realized that I forgot to take screenshots from htop in my test. The output of display from htop agrees with the data displayed using free -h

@marsfan  can you please try upgrading to the Store version of WSL and seeing if your problem still exists when using that?
You can update here: https://aka.ms/wslstorepage

I also have the same bug, the memory usage keeps growing until the PC cannot move anymore, it happens every single time. The memory is never freed. Using Ubuntu and connecting via vscode for dev purposes

it has been 3 years since the first issue about vmmem issued. Could we hope using Windows page caches inside wsl kernel one day?
It is the most painful thing wsl2 brought to me.

yo its 2023 can we have a wsl system that work plz ? i have a 64gb system after 10 nodejs build wsl is consuming my 64gb of ram, that stupid

yo its 2023 can we have a wsl system that work plz ? i have a 64gb system after 10 nodejs build wsl is consuming my 64gb of ram, that stupid

Exactly the same problem for me. Very annoying


change all docs/content/*.md filenames to only have lowercase and dashes

This allows opening those files on any os like windows or mac or linux.

do the links still work?

@fjy is there an easy way to test? and do you see a reason changed links not  work?

@himanshug, @fjy while we are at it, should we make all our links lowercase? When testing on OS X or Windows, someone would not be able to see whether a link is broken because those filesystems are case-insensitive, whereas GitHub Pages is case-sensitive.

@xvrl lowercasing makes sense, I will update the PR. @fjy let us know if you think otherwise.

@himanshug If you issue install the ruby gem jekyll and issue 'jekyll -w serve' in the docs directory, you should be able to load up the docs locally and view them in a web browser. Can you verify the links still work? My concern is that links on the internet may reference the latest docs and use the mixed case tutorial links.

Unless we don't change the file names at all we'll have broken links on the Internet. Hence my suggestion to take advantage of this change to make it lowercase.
I've spend a fair amount of time cleaning up casing problems in links in the past, so I say we just bite the bullet.
Our documentation will not always have the same set of pages, and the best way around that is to provide permanent redirects for some time so search engines can update their index.

our doc links are versioned, so,
http://druid.io/docs/<IP_ADDRESS>/Tutorial:-A-First-Look-at-Druid.html
will be valid even after changes we make to the names
however, I understand that  "http://druid.io/docs/latest/Tutorial:-A-First-Look-at-Druid.html" will break (and will show the page with link to druid.io) and if we think that this is big enough of a problem then redirection is the only solution.
I am not entirely familiar with the current doc publishing setup. Is "redirection" a matter of creating a file somewhere containing lines "from -> to" or is it something more?

@himanshug you can see an example of redirection here https://github.com/druid-io/druid-io.github.io/blob/master/api/0.6.173/index.md

@fjy so, lowecase + setup redirects sounds ok?

@xvrl @fjy ok, changed filenames to just contain lowercase and dashes, updated all the links.

@xvrl trying to setup redirection on https://github.com/druid-io/druid/pull/1344/files#diff-0cea66d55e30b21d83fbba081157afa1 but it does not work locally when I do "jekyll serve". Does this look OK to you?

@himanshug it looks like the redirection is working for the page you changed. The problem I am seeing is that when checking out the repository on a case-insensitive filesystem, you can only edit one of the files, and I'm not sure which one will prevail. I guess that means we need to change more than just the casing for things to be editable on Windows an OS X.
Btw, in order to test the redirection, you need to install the jekyll-redirect-from package using gem install [--user-install] 'jekyll-redirect-from' and also add the following snippet to druid/docs/_config.yml, which I haven't copied over from the main druid website repository.
gems:
  - jekyll-redirect-from

@xvrl yeah, that is a problem. how about prepending every file name with "docs-" e.g.
XxX.md becomes docs-xxx.md ?
PS: I wish redirection was doable by creating one file and listing things there :(

@himanshug alternatively, we can also organize the code into sub-directories to structure it a little bit better. I know @fjy might have some suggestions.

@fjy any thoughts on whether to prepend all filenames with something like "docs-" or move stuff around to other folders?

@himanshug im not sure the value add of having "docs-" appended to everything as all the files are already in a docs directory. I think having subdirectories that match the headings of the TOC would be useful to organize files. We can also have a subdirectory for "unused"/"legacy" or something.

@fjy actually I have no problems with all  of them being in one directory as bucketing can anyway be done in toc file, it isn't bad but to setup the redirection we need to either have different name or move all of them to some other folder[s].
One option is to have folder "docs/_content" and move all the newly created files there. we keep "docs/content" for files containing redirection links.
Or, do you want to bucket them in separate folders anyway?

@himanshug the content folder does not actually exist in the github-pages repository where thing get published, so someone checking out the druid-io.github.io repository would have issues with the names conflicting. I agree with @fjy that following the TOC layout makes sense.

sure, will put those in folders as they are bucketed in toc.

@xvrl @fjy how about following bucketing for all the *.md files...
operations/alerts.md
development/about-experimental-features.md
querying/aggregations.md
development/approxhisto.md
ingestion/batch-ingestion.md
tutorials/booting-a-production-cluster.md
configuration/broker-config.md
design/broker.md
development/build-from-source.md
dependencies/cassandra-deep-storage.md
misc/cluster-setup.md
design/concepts-and-terminology.md
configuration/configuration.md
configuration/coordinator-config.md
design/coordinator.md
ingestion/data-formats.md
querying/datasource.md
querying/datasourcemetadataquery.md
dependencies/deep-storage.md
design/design.md
querying/dimensionspecs.md
misc/druid-personal-demo-cluster.md
comparisons/druid-vs-cassandra.md
comparisons/druid-vs-elasticsearch.md
comparisons/druid-vs-hadoop.md
comparisons/druid-vs-impala-or-shark.md
comparisons/druid-vs-redshift.md
comparisons/druid-vs-spark.md
comparisons/druid-vs-vertica.md
misc/evaluate.md
tutorials/examples.md
querying/filters.md
ingestion/firehose.md
development/geographicqueries.md
querying/granularities.md
querying/groupbyquery.md
configuration/hadoop-configuration.md
querying/having.md
configuration/historical-config.md
design/historical.md
operations/including-extensions.md
configuration/indexing-service-config.md
design/indexing-service.md
design/index.md
ingestion/ingestion-faq.md
ingestion/ingestion.md
ingestion/ingestion-overview.md
development/libraries.md
querying/limitspec.md
configuration/logging.md
dependencies/metadata-storage.md
operations/metrics.md
design/middlemanager.md
development/modules.md
operations/other-hadoop.md
misc/papers-and-talks.md
design/peons.md
operations/performance-faq.md
design/plumber.md
querying/post-aggregations.md
configuration/production-cluster-configuration.md
querying/query-context.md
querying/querying.md
configuration/realtime-config.md
ingestion/realtime-ingestion.md
design/realtime.md
operations/recommendations.md
operations/rolling-updates.md
development/router.md
operations/rule-configuration.md
querying/searchquery.md
querying/searchqueryspec.md
querying/segmentmetadataquery.md
design/segments.md
development/selectquery.md
configuration/simple-cluster-configuration.md
misc/tasks.md
querying/timeboundaryquery.md
querying/timeseriesquery.md
querying/topnmetricspec.md
querying/topnquery.md
tutorials/tutorial-a-first-look-at-druid.md
tutorials/tutorial-all-about-queries.md
tutorials/tutorial-loading-batch-data.md
tutorials/tutorial-loading-streaming-data.md
tutorials/tutorials.md
tutorials/tutorial-the-druid-cluster.md
development/versioning.md
dependencies/zookeeper.md

@himanshug seems reasonable, go for it

@fjy @xvrl  made all *.md file names contain only lowercase and dashes, have them in the folders as shown in the comment above, fixed links
let me know if this looks ok and I will rebase.

@himanshug Apologies for dropping the ball on this. Let's try to get this merged tomm.

@fjy np, it wasn't any high priority thing and I can see there is lots of other stuff going on.

@himanshug looks pretty good, I'm about to submit a PR to your branch that does a little bit more renaming to shorten some of the names that are now redundant and clean things up a bit.

@xvrl sure

@himanshug https://github.com/himanshug/druid/pull/1

@xvrl I believe, it is ready to be merged now. I will squash commits. pls let me know if you had any other comments.

@himanshug should be good, just need to squash and rebase. There may be some changes that FJ did to docs that you have to add back manually to fix conflicts, but hopefully rebase will be smart enough to do it.

@xvrl updated and squashed

@himanshug There appear to be some merge conflicts. There were some docs that were changed from the time this PR was created to now, if those changes are in, then I'm :+1:

@fjy I had updated everything day before yesterday, I think your update to Ingestion.md related to new metric and dimension names check is in conflict. fixing it.

@fjy fixed the conflic, it should be good to go now.


Add configurable libraries to the App Engine Standard facet configuration

Per the document on Java App Engine Dependency Management in IntelliJ and Eclipse.
https://docs.google.com/document/d/1UEqYEWwbgNK8EqPuIUvXTtOfFbV-Oa7cmKxtrU2ARzE/edit#

At a minimum, we should add the Servlet API 2.5 to the classpath of app engine standard projects.


DOC: optimize.least_squares: update curve_fit docstring and ValueError message

Updates to scipy/scipy#20407.
Update your main branch to see the actual diff; I just merged main and added one commit.

It looks like this branch is protected:
(scipy-dev) C:\Users\matth\scipy>git status
On branch least_squares_x0_p0
Your branch is ahead of 'mm/least_squares_x0_p0' by 861 commits.
  (use "git push" to publish your local commits)

nothing to commit, working tree clean

(scipy-dev) C:\Users\matth\scipy>git push mm least_squares_x0_p0
Enumerating objects: 38, done.
Counting objects: 100% (34/34), done.
Delta compression using up to 16 threads
Compressing objects: 100% (14/14), done.
Writing objects: 100% (14/14), 1.73 KiB | 441.00 KiB/s, done.
Total 14 (delta 11), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (11/11), completed with 8 local objects.
remote: error: GH006: Protected branch update failed for refs/heads/least_squares_x0_p0.
remote: error: At least 1 approving review is required by reviewers with write access.
To github.com:m-maggi/scipy.git
 ! [remote rejected]       least_squares_x0_p0 -> least_squares_x0_p0 (protected branch hook declined)
error: failed to push some refs to 'github.com:m-maggi/scipy.git'

Can you make your permissions less restrictive?


[FEAT] Does this work via React Admin

So I tried to make it work with react admin because the ra-strapi-rest is not working, i think it's quite old because of the count functionality
`import { Admin, Resource } from 'react-admin';
import { DataProvider  } from '@pankod/refine-strapi-v4';
import { PostList } from './components/PostList';
const dataProvider = DataProvider('http://localhost:1337/api');

`
This is my PostList
`import React from 'react';
import { List, Datagrid, TextField } from 'react-admin';
export const PostList = (props) => (
<List {...props}>

);`
Or if i use refine. will i be able to create custom login and pages, i am using a template. It calls undefiend for some reason.
http://localhost:1337/api/undefined?pagination[page]=1&pagination[pageSize]=10&
i dont want to use the refine front end just the authorisation and rest provider for strapi.
as per the document it probably looking for this
resources={[{ name: "posts", list: PostList }]}
I am so confused, i have the UI components now i just want the strapi rest to work with it, will i be able to incorporate the template in refine

Hey @ssshammi ,
Thank you for contacting us! Refine and React Admin are different projects. I don't think you can run both together. Their approaches are completely different. Maybe you can try to get help for strapi v4 from react-admin.

@omeraplak yes i am not using react-admin now. just wanted to use the authorization and dataProvider but this repo is useful because I am to use strapi.v4 api. maybe later ill use the other components.


update to remove object3d mesh when component removed from entity

This PR is just a simple remove function that removes the cursor mesh when the component is removed from an entity. I also updated the dist to reflect these changes.

Also, kudos for working on this despite all the examples being broken with A-Frame 0.5.0! Maybe I should bump the dependency to A-Frame master as a stop-gap.

Thanks!


include go routines

Adding go routines to run queue agent querying in parallel.
With agents:

Without queue:

This looks okay but its all done manually. This page https://gobyexample.com/waitgroups links out to the errgroup package which will make this much simpler: https://pkg.go.dev/golang.org/x/sync/errgroup
If you search the codebase, you'll see an example of it already in use as well I think

Thanks @jradtilbrook as discussed I have made changes to include err group and instead of channel added array of queues and go routines based on the queue index


Using XML based navigation and compose destinations in the same app?

Is it possible to use XML based navigation and compose destinations in the same app?
I have only started learning Compose and I am considering to migrate my existing app fully to Compose.
I want build new screens as Compose screens and I want to gradually migrate my fragments to Compose screens.
Is is possible to navigate from a fragment in my existing XML based navgraph to a composable?
I guess the composable screens will need to live in a separate navgraph, but I am unsure if this is possible and how I would hook the two navgraphs up together.
Any suggestions?

In this case, your best bet is to use Composable inside Fragments. Then you can just use your normal methods to navigate to those Fragments and in onCreateView you return a ComposeView, I think it’s called.
As seen here for example
https://stackoverflow.com/questions/59368360/how-to-use-compose-inside-fragment


3.8 is no more, check out this fix

Gives the three videos warning
Have been able to fix uBlock with this hack
https://www.reddit.com/r/youtube/comments/15i7vql/video_player_will_be_blocked_after_3_videos_if/
CODE:
youtube.com##+js(set, yt.config_.openPopupConfig.supportedPopups.adBlockMessageViewModel, false)
youtube.com##+js(set, Object.prototype.adBlocksFound, 0)
youtube.com##+js(set, ytplayer.config.args.raw_player_response.adPlacements, [])
youtube.com##+js(set, Object.prototype.hasAllowedInstreamAd, true)

possibly applicable to Tempermonkey?

I'm on version 5.6 and haven't had any ads today. a few fast ads but has been acting in a way that doesnt disturb my experience. for the past week or two I've been refreshing and changing versions. But its working fine for me now.

I'm on version 5.6 and haven't had any ads today. a few fast ads but has been acting in a way that doesnt disturb my experience. for the past week or two I've been refreshing and changing versions. But its working fine for me now.

Hmm. Are you sure? People claimed that 5.6 still has the double audio problem. I only tried up to 5.5 myself, and it had the double audio and a conflict with YouTube Enhancer to boot.
This alt script also seems to work. (need to try on my other machine with more browsers though)
https://github.com/TheRealJoelmatic/RemoveAdblockThing/issues/602
I am kinda chicken to try the original script anymore, because I seem to be on some kind of "three strike" regiment :) I think I'll wait for a few days.


fix(client): sync state between separate window with floating panel

should close #458, need test by @sxcooler, test on my local works properly.
Reproduction step:

open the page with the devtools floating panel
open a separate window
update settings on floating panel
go to the separate window, and storage is changed, but UI is not changed.

should close #458, need test by @sxcooler, test on my local works properly.
Reproduction step:

open the page with the devtools floating panel
open a separate window
update settings on floating panel
go to the separate window, and storage is changed, but UI is not changed.

Sorry, but how can I take part in testing?

1. clone source
You can use gh CLI

Or clone my repo and switch to fix/client branch.
2. build
pnpm i
pnpm build

3. link to your project
If you are using pnpm
// package.json
{
  "dependency": {
    //                                             <your-path-to-vite-package>
    "vite-plugin-vue-devtools": "../../../vue-devtools-next/packages/vite"
  }
}

1. clone source
You can use gh CLI

谢谢指导，pnpm build的时候遇到以下两个错误：

以及

但我检查package/electron/dist目录，devtools.d.ts这个文件存在：

如果强行把这个vite链接到我的项目，会出现devtools无法打开：

一直停留在加载动画。
我npmrc和pnpm config都已经加上了electron等的mirror 尝试过pnpm store prune无法解决
我看你代码改动比较简单，原理上也说得通，你如果测试没有问题我可以先把原issue关闭。

😂, 可以等到 merge 了，这个 build error 的问题后面会追踪一下

This comes with another issue.

From separate window
Click select component button (Attachment 1)
Select a component
Floating panel appears, it should not based on my settings (Always hide),
And it doesn't disappear without clicking somewhere else, causing you to lose the selected component scope.


Ease contributing for Mac users

Fixes #146.

Fix docs/data/msft.csv capitalization in git
Add .DS_Store to .gitignore

Note that docs/data/msft.csv is unchanged in git. Without this change, Mac users cannot rebase because git gets confused between msft.csv and MSFT.csv and always thinks there is a change in the working directory when there isn't.

next branch please :)

Oh right, sorry! Will get right on that when I have a second.
Is the next release imminent enough that you're not taking PRs to master?

there are breaking changes and many changes in next, so only bugfixes on master for now.


Feedback

From README.md:
PLEASE SHARE FEEDBACK

O.k., here it comes:

I had difficulties getting the extension up and running. I was confronted with this message
didn't find webserver URL in Hugo output

After having a look at the source code, I realized the reason for that. In my config file, I had:
baseURL: /

This setting results in this output of hugo
Web Server is available at //localhost:60162/ (bind address <IP_ADDRESS>)

and the extension is expecting a regex like Web Server is available at (http\S+).
So I corrected my baseURL and could eventually see the site preview. I don't think the regexp check is good here, I'm afraid other users will run into probelms here, too.

The extension blocks port 1313 all the time. This is unfortunate. I would propose to use a different port dedicated to your extension and the associated preview only.

My setting: editor with two windows side by side: left window markdown file to be preveiwed, right side preview pane. Once I click on some text inside the preview pane, a new window opens, holding the markdown file that is already open. I now have  two editors open, displaying identical files. This is unfortunate. The open window should be reused and the cursor should placed to he correct location there.

Having to add a partial in order to make the extension work is unfortunate. There must be a better way. I'm afraid this procedure won't be accepted (or can't be easily done) by a casual user of your extension.

In general, this extension is potentially useful, but needs some polishing IMHO. Keep up with your good work!

Thank you for trying it out and especially for posting a detailed feedback.
Your (1) came as a surprise. I need to read up on baseUrl. I believed that Hugo prints a Url at startup, but I didn’t bother checking. The output you are quoting, namely ‘//localhost:60162/’, doesn’t look like a Url to me. Is it something you can navigate to? Good catch!
Concerning taking :1313 (item 2), I’d like to understand why that is an issue for you. If you don’t specify the port to use, Hugo tries :1313, and if it is not available, uses a random high port.
On (3), this is a VSCode thing. It will open an editor in the active tab group. Try clicking on the preview pane tab “handle” to activate, and pick a source file from the directory tree. I could be more specific with the editor placement when my extension opens it, but I can do very little to fix e.g. opening via directory tree.
There’s a manual fix (though it sucks). Lock the tab group where preview is currently showing (via menu under …). Then things will work as expected.
I will be happy to make it more user friendly, but I don’t know how yet.
Finally, I totally see your point in (4). Could tou share some insights how it could be accomplished ?


Constellation.js Library Not Working Properly With The Project

I tried to use a JavaScript library called constellation.js, the constellation effect shows when I put the  element but on the page, but every other thing on the page disappears.

ask a question to the developer of constellation.js


Please add Microsoft Surface Go 3 model drivers

Please add Microsoft Surface Go 3 model drivers. That model is not listed in the Available models list.
Thank you.

You probably want to submit that under https://github.com/maurice-daly/DriverAutomationTool - this git repo is ONLY for the ps1 file that you use in your task sequence.


Small fixes to docs

I've been also thinking about removing backend example which is not referenced anywhere
but the repo which is referenced instead seems to receive features (https://github.com/WcaleNieWolny/capgo-social-login-backend-demo/commit/141c01d93a85240e31a0d488a89df13c842708b1) that are not related to Sign In with Apple
BTW: nice write up 👍

thanks for your help


Add security section to issue template

We had this in the SignalR repo to prevent folks from making public bugs with security issues.

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=159217&view=logs

This comment was made automatically. If there is a problem contact ryanbrandenburg.
I've triaged the above build.

Should I go ahead and add this to EF as well?

Should I go ahead and add this to EF as well?

Yes, and also Extensions repo.


Arrow Flight SQL example JDBC driver incompatibility

Which issue does this PR close?
Closes #5665
Closes #5669
Rationale for this change
It would be great if the example Flight SQL server would support integration with the Arrow JDBC driver. This would demonstrate the cross-platform nature of Arrow quite well.
This PR changes the example flight_sql_server so we can query it from Java using the Arrow JDBC driver.
What changes are included in this PR?

authorization header is returned with contents "Bearer "
Added clear_token() to FlightSqlServiceClient to give the client the ability to re-authenticate.
The location field returned by get_flight_info_prepared_statement uses a "grpc+tcp://<IP_ADDRESS>" as its location. Instead it should include the host and the port.
do_action_close_prepared_statement needs to include at least an empty stub (return  Ok(())).

Are there any user-facing changes?
No

We might have to separate the listening address vs the server address to make things clearer 🤔
Thoughts on this?

If you have FlightInfo return an empty location, clients can re-use the connection that was used for the GetFlightInfo request. This is useful for the case where returning this IP/host is non-trivial.

We might have to separate the listening address vs the server address to make things clearer 🤔
Thoughts on this?

If you have FlightInfo return an empty location, clients can re-use the connection that was used for the GetFlightInfo request. This is useful for the case where returning this IP/host is non-trivial.

I can add this change, and just return an empty list - it Seems to me the best default case that makes sense for most users. The FlightEndpoint struct has good comments about this too.


[Q] Adjust image position in cell

Hi,
I'm adding some images to my cells, and managed to adapt the cells to have the same width and height of the image, but now images are covering the border of the of the cell, so I want the images inside each cell to go one point to the the right and one point to the bottom, how can I do this?
This is how I'm adding images currently:
   worksheet.addImage(imageId2, {       tl: { col: (dataRow.length - 1), row: rowIndex + 2 },       ext: { width: 93, height: 93 }     });
Thanks.

Here is my solution, Maybe it works for you.
const range = {
    tl: { col: 2 - 0.94, row: 10 - 0.75 },
    br: { col: 6, row: 12 }
}
worksheet.addImage(imageId, range);


Change style of DAG generated in dag_drawer()

Summary
Closes #13106
Adds non-breaking interface to dag_drawer() to allow Qiskit users to customize the colors of DAGs.
Details and comments
I exposed the rustworkx customization DAG functions by adding a new style option style="custom". This then allows you to pass parameters to dag_drawer() to specify any styles specified in the following links:

https://graphviz.org/docs/nodes/
https://graphviz.org/docs/edges/
https://graphviz.org/docs/graph/

Here is an example:
from qiskit import QuantumRegister, ClassicalRegister, QuantumCircuit
from qiskit.dagcircuit import DAGCircuit
from qiskit.converters import circuit_to_dag
from qiskit.visualization import dag_drawer
from qiskit.dagcircuit.dagnode import DAGOpNode, DAGInNode, DAGOutNode
from qiskit.circuit import Qubit, Clbit

q = QuantumRegister(3, 'q')
c = ClassicalRegister(3, 'c')
circ = QuantumCircuit(q, c)
circ.h(q[0])
circ.cx(q[0], q[1])
circ.measure(q[0], c[0])
circ.rz(0.5, q[1]).c_if(c, 2)

dag = circuit_to_dag(circ)
dag_drawer(
    dag,

    # node_attr_fn and edge_attr_fn will only work if style="custom" is specified
    style="custom",

    # Using functions for the node and edge attributes allows you to dynamically color your graphs
    # For more node style options, see https://graphviz.org/docs/nodes/
    node_attr_fn = lambda n: {"style": "filled", "fillcolor": "red", "fontcolor": "white"} if isinstance(n, DAGOpNode) else {"style": "filled", "fillcolor": "grey"},

    # For more edge style options, see https://graphviz.org/docs/edges/
    edge_attr_fn = lambda n: {"arrowsize": "2"} if isinstance(n, Qubit) else {"arrowsize": "0.5"},

    # For more graph style options, see https://graphviz.org/docs/graph/
    # NOTE: node_attr_fn is NOT a function, it is a dict
    graph_attr = {"bgcolor": "beige"}

The code above generates this DAG:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

2 of 17 (11.76%)  changed or added relevant lines in 1 file are covered.
8 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.002%) to 88.866%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/visualization/dag_visualization.py
2
17
11.76%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.02%

qiskit/visualization/dag_visualization.py
3
11.64%

crates/qasm2/src/lex.rs
4
92.48%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.002%

Covered Lines:
74150

Relevant Lines:
83440

💛  - Coveralls


make noise related classes use holders instead of direct values

this is difficult because some classes modify their modules in their constructor

also this is technically a bug since noise doesn't currently use the world seed

oops

make noise related classes use holders instead of direct values

hold off on this for now


Bump version to v2.16.0

It’s been over a month since our last release, and our (latest) documentation has broken tables.

Sigh… I tagged the wrong commit. I have now re-tagged on the merge commit instead. Let me know if you run into problems because of it 😬


Clang -msimd128 compiled wasm generates error when run

When trying to run a clang -msimd128 compiled .wasm with a wamr app (using WAMR_BUILD_SIMD 1) in Ubuntu X86_64/Skylake I receive:
WASM module load failed: v128 value type requires simd feature

Also, when trying to convert clang -msimd128 compiled .wasm to an simd enabled .aot module, I get:
invalid wasm type: -63

Non-simd wasm and aot files work just fine though! Is it not (yet?) possible to run "pure" clang generated wasm binaries with msimd128, or am I missing an essential ingredient?

Hi, SIMD is only supported in AOT mode and JIT mode, please build iwasm with JIT mode to run the wasm file, or build wamrc to comple was file into aot file, and run the aot file, e.g.:
wamrc -o test.aot test.wasm
iwasm test.aot

wamrc -o test.aot test.wasm

In the latter case I get:
./wamrc -o mnist-clang-simd.wasm mnist-clang-simd.wasm
WASM module load failed: type mismatch: expect v128 but got other

I did use a precompiled version of LLVM 10.0. I will try again with the LLVM 11.0 based compile script, and let you know if that resolved the issue.

I updated both WAMR and wasi-sdk LLVM's to version 11, which resolved the issue. Thanks for your fast response!


Set window title to be the same as the selected tab name

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #6214

Details of Changes
Add details of changes here.

Set current tab header as window title so it appears on the taskbar

Validation
How did you test these changes?

[x] Built and ran the app

Screenshots (optional)
Add screenshots here.

@gave92 thank you!


Graph not load because of "-gh" elsewhere

We have 2 folders side by side: one filled with graph named "foo" and another empty named "foo-gh".
When we call graphHopper.load with the "foo" folder, the code here searches also for the "-gh" suffix, finds the empty "foo-gh" and continues to load that instead, failing at the end.

So which folder to load :) ? Or do you just mean we should prefer without "-gh" suffix?

I ask the API explicitly to load the "foo" folder, that folder exists. That should be used (either if contains graph or not).
And if the folder does not exist, I'm not sure if such automations help or confuse the user.
e.g. I was searching why my graph was not loading, until I realized that the API persisted on its side folder instead. :smile:

Yes, ugly. Problem is that this breaks backward compatibility. But probably the only reason why I did this was the graphhopper.sh script where we can move the -gh addition.


Added wrapper for mono.

@shazqadeer : Maybe you could merge this for us?
I don't have write privileges on this repo. Maybe it would make sense that you make @michael-emmi and myself Owners of the boogie-org organization?
That way we would have write access to all repos in that org. Thanks!

Also see issue #36.

@zvonimir I agree it would make sense for the two of you to have write access if you're interested in Boogie.  But my suggestion is to make TravisCI checks mandatory, and not merge PRs where the CI fails, like this one.

I agree. This requires updating/fixing CI travis script first. I am pretty sure CI is failing since the script/setup got deprecated.

@michael-emmi Please rebase this so that the CI passes.

@michael-emmi BTW, don't just set the build mode to Debug.  Try something like BUILD_TYPE=${BUILD_TYPE:-Debug} (set to Debug if not already set)

@ccadar updated and rebased.

Thanks!


Create redirect from master-api.flutter.dev to main-api.flutter.dev

Description
Eventually, we'd like to move to using the "main" channel instead of the "master" channel, to be sensitive to the connotations that "master" has.
To that end, we should move the existing master-api.flutter.dev site to be main-api.flutter.dev, and create a permanent redirect from master-api.flutter.dev to main-api.flutter.dev. As an alternative, we could just create a main-api.flutter.dev redirect and point it to the existing site, but the problem with that is that it continues to propagate the "master" name in new URLs that are used to link to the site, since after redirect it ends up on the master site, not the main site.
I'm not sure exactly what is involved in this reconfiguration, but I assume it has something to do with the Firebase configuration.  One complication will be to make sure that the API docs continue to be published to the renamed Firebase project, and to maintain whatever website version history we want to preserve (not much, I think, maybe the last 5 versions).
cc @godofredoc

@gspencergoog I don't think we manage the links, potentially a someone in the devrel team? @craiglabenz ?

Well, it's not really the links themselves.  This has to do with configuring the publishing step to a renamed/new Firebase project, and the hostname of the hosted firebase project (which could also be configured to do the redirects, I think).
The main point is that we want to move the site to the new hostname and provide redirects from the old site. There are probably configuration changes that we could make to the Firebase project that would do this without a lot of churn or work, but I'm not sure what those would be.

@godofredoc do we manage the firebase project, or is someone else the best contact for this?

I do not have the required permissions, @godofredoc is there docs on the needed permissions?

\cc @domesticmouse @RedBrogdon Do either of you have access to the relevant DNS configuration and Firebase projects? Perhaps you can help with this.

This isn't something that can be done with DNS, but it can be handled by introducing a new bouncer AppEngine service that responds with a 301 Permanent Redirect.
Context: https://developer.mozilla.org/en-US/docs/Web/HTTP/Redirections

Who actually owns the Firebase project for the API docs and has the relevant permissions to modify it, and create new ones?

@christopherfujino do you know?

@parlough or @sfshaza2 would either of you know?

My understanding is that both @RedBrogdon and @johnpryan have access to this project.

@domesticmouse said:

This isn't something that can be done with DNS, but it can be handled by introducing a new bouncer AppEngine service that responds with a 301 Permanent Redirect.
Context: https://developer.mozilla.org/en-US/docs/Web/HTTP/Redirections

That's definitely an option for setting up the redirect, especially if its desired to reuse the same Firebase project :)
I was more checking to see who has access to the master-docs-flutter-dev Firebase project and could also setup the new main-api.flutter.dev subdomain.

The main point is that we want to move the site to the new hostname and provide redirects from the old site. There are probably configuration changes that we could make to the Firebase project that would do this without a lot of churn or work, but I'm not sure what those would be.

If it's fine/desired to reuse the existing master-docs-flutter-dev Firebase project, I imagine not a lot of work will be needed:

(Optional) To avoid downtime: Set up some mechanism to temporarily (302) redirect master-api.flutter.dev requests to master-docs-flutter-dev.web.app.
Adjust the master-docs-flutter-dev hosting configuration to instead use a new main-api.flutter.dev subdomain
Change the mechanism from step 1 to instead permanently (301) redirect master-api.flutter.dev requests to main-api.flutter.dev

If it's desired to use a new main-docs-flutter-dev Firebase project, I guess there's a few extra steps:

Create a new main-docs-flutter-dev Firebase project with hosting setup.
Configure the main-docs-flutter-dev project to use a new main-api.flutter.dev subdomain.
I'm not sure what the setup is now, but either configure the service account/token that flutter/flutter already uses to have access to the new project or set up the new service account or token.
Update flutter/flutter's CI to instead upload to main-docs-flutter-dev: https://github.com/flutter/flutter/blob/8ca7d7d1362c1503c77585fff52415ac4295e415/.ci.yaml#L384
Setup a mechanism to redirect requests to master-api.flutter.dev to main-api.flutter.dev. An easy choice would be to push a new firebase.json to the old master-docs-flutter-dev project to (301) redirect all requests to main-api.flutter.dev. Or setup redirection with something external like an AppEngine bouncer service like Brett mentioned.

It think it is desirable to rename the project too.  We could still leave the old master-docs-flutter-dev project as an empty project with a redirect, but I think that if we don't have it publish to main-docs-flutter-dev, then it will just be confusing for future maintainers (and tech debt).

This may also require some updates in the internal redirections. I'll take a look and try to fix by the end of next week.

Took a look, this will be landed in two steps:

Adding two custom domains to master-docs-flutter-dev: main-api.flutter.dev and main-docs.flutter.dev
Add CNAMES to point those two subdomains

Because of restrictions in the naming conventions the new project will be called main-docs-flutter-dev

Thanks for working on this @godofredoc! For those following, https://main-api.flutter.dev/ is now live and being updated with changes.
However, should we reopen this issue until master-api.flutter.dev redirects to main-api.flutter.dev?

Yes, let's keep it open for that part.

Should we change the master-api.flutter.dev the references instead?

I do think the references should be updated where possible, but there are likely many master-api.flutter.dev links around the internet that may not get updated and many users may type it out of familiarity, but it won't be clear to them they are seeing outdated content.

...  it won't be clear to them they are seeing outdated content.

Actually, since this is a redirect to the main-api site, the content should be up to date, just the link will be out of date.

Oops, sorry for misspeaking, I meant without the redirect it won't be clear the content is out of date :)

Yeah, once we have the redirect from master to main, we can actually remove the old master site entirely.

The redirects are ready please feel free to close the issue when validated.

When I go to https://master-api.flutter.dev, (or a specific page like https://master-api.flutter.dev/flutter/animation/Curve-class.html) I still end up with a URL that starts with master-api.flutter.dev.  With the redirect, I was expecting to see it switch to main-api.flutter.dev. Would that be easy to do? It would help with the migration of external links, since as it is now, copying the link will perpetuate the "master-api" links.

@gspencergoog can you please give it another try. I fixed one of the redirects.

Just noticed all the Dartpad examples aren't loading in the redirected page to https://main-api.flutter.dev/
I was about to report it but looks like @parlough is fixing it in https://github.com/dart-lang/dart-pad/pull/2714

Yes, the redirect works for me now.

The main API docs are now hosted at main-api.flutter.dev under the new main-docs-flutter-prod Firebase/GCP project, and master-api.flutter.dev now successfully redirects all links while preserving paths to there.
https://github.com/dart-lang/dart-pad/pull/2714 has also landed and been deployed, fixing embedded sample DartPads.
So I'm going to close this issue as fixed. Thank you all for helping getting this work landed, especially @godofredoc!


Refine helping-hand/add hand-* icons

Closes #1322.

I like the hand-platter one.

@danielbayley how about a heart hand icon using your template?

Looks way better than before!

It seems the icon is ready to get added.. Does anyone have negative opinions on these? :D

To move on and to keep focus on closing the linked issue I've removed the thumbs-up-down icon, I think we should iterate on it more.
Awesome icons @danielbayley!


feat(learn, i18n): enable japanese certifications

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

I refered to the following PRs (when we enabled Ukrainian language)
https://github.com/freeCodeCamp/freeCodeCamp/pull/44454
https://github.com/freeCodeCamp/freeCodeCamp/pull/44461/

@nhcarrigan Do we also need the configurations mentioned under "Enabling Localized Videos" section in the document below? I think we have not localized the video contents (like subtitles) for Japanese.
https://contribute.freecodecamp.org/#/how-to-test-translations-locally?id=enabling-localized-videos
Looks like there are only espanol, italian, portuguese in the current code.

We shouldn't need that configured until we have localised videos to display. 🙂

@nhcarrigan Understood. Thank you 🙂

Thanks @nhcarrigan ! Yes, I just copied the English files and forgot to update. I will look into them.

Definitely not something you have to do in this PR. Once we get a second set of eyes on this and it is merged, we can let the contributors know they can look at the quotes and whatnot.

@raisedadead Sorry I could resolve them so I did.

@raisedadead Oops, sorry!😂 I wasn't sure how we should work with the resolve conversation button

Yes, those were definitely not meant to be blocking but more to indicate the next steps. Apologies for the hold up. 🙂


Add FalDbt class and use it where possible for magic functions

Initial ideas here. It is in working condition but can be refactored.
Tests should be fixed.

Since we're losing the context, we need to somehow get information on available models and their states.

Are we losing context? Users still have to tell us where their project is and we are able to gather all the information we need from the project..
profiles_dir="/Users/matteo/.dbt",
project_dir="/Users/matteo/Projects/fal/experiments/iris

but maybe when you say context, you are referring to something else?

Since we're losing the context, we need to somehow get information on available models and their states.

Are we losing context? Users still have to tell us where their project is and we are able to gather all the information we need from the project..
profiles_dir="/Users/matteo/.dbt",
project_dir="/Users/matteo/Projects/fal/experiments/iris

but maybe when you say context, you are referring to something else?

I meant the context variable.


Fix Fix memory leak for stream load 2pc #16430

Proposed changes
Issue Number: close #xxx
Problem summary
StreamLoadContext is not deleted correctly. Fix memory leak for stream load 2PC.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

./run buildall

run buildall


Animatable objects always allocate transitions and check every single property for transitions

From the implementation of Animatable it looks like that original intent was that Transitions are null when user is not assigning any and then lazily created when necessary.
But this seems to be not true anymore and Animatable.Transitions and Animatable._previousTransitions are always allocated. This is caused by transitions being a direct property so when we create a new control derived from Animatable property registry will try to get default value calling Transitions getter, which will initialize fields.
Some numbers about instance creation and transition checks being done are below.
Launching ControlCatalog:

994 Transitions and Dictionary<AvaloniaProperty, IDisposable> instances. 993 are empty and 1 contains a single element.
11855 checks on property changed with 1 actual match.

Launching ControlCatalog and going to Calendar and DataGrid pages:

12565 Transitions and Dictionary<AvaloniaProperty, IDisposable> instances.
130077 checks on property changed with 23 actual matches.

General:

All property changes are checked for match against empty transitions collection.
99% of transitions collections are empty and will stay forever empty.

cc: @jmacato

It seems to be an oversight in AvaloniaPropertyRegistry that calls NotifyInitializer for all registered properties, even for ones that don't have any initialization subscriptions. That basically prevents any attempts to implement lazy initialization.

We probably also need some generic way to get a property value if it was already initialized. That will also help with GestureRecognizers property that currently use logic inside InputElement itself to check if collection is already initialized.

is there any purpose for this line in AvaloniaPropertyRegistry? https://github.com/AvaloniaUI/Avalonia/blob/fe74dfe53856f6c4866464e42708edbd2a6af4a9/src/Avalonia.Base/AvaloniaPropertyRegistry.cs#L316
I removed that and all unit tests passed.
cc: sir @grokys


fix(aws-lambda): return outgoing cookies on response objects

🔗 Linked issue
resolves #245
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This PR returns an array of cookies to set from aws-lambda preset rather than keeping them on the headers. More info in AWS docs.
@brtinney - would be grateful if you could confirm this resolves the issue for you.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@danielroe: It appears to properly set the cookies now, although I still have to use: event.headers.cookie = event.cookies.join("; "); over event.headers.cookie = event.cookies.join(","); in order to use the cookie library to parse them from the event, as the (more common?) convention is to use ; to separate them.
That said, I'm not sure if I can necessarily insist that nitro should be using ; to separate the cookies. I can at least get away with doing parse((req.headers.cookie || '').replace(/,/g, ';')) in my actual code instead of having to modify the built mjs for that particular issue. (Which should be safe since neither , or ; are supposed to actually appear in cookies....)
Thanks!

@brtinney I think the other issue should be resolved with https://github.com/unjs/nitro/pull/356.

@pi0
I have tried this and it doesn't work.
I configured Amazon API Gateway proxy integration to work with an AWS Lambda function. When I call my endpoint, I receive a configuration error and an HTTP 502 status code.
Execution failed due to configuration error: Malformed Lambda proxy response

This is due to the fact that cookies are returned in the response object.
For API Gateway to handle a Lambda function's response, the function must return output according to the following JSON format. Additional fields are not allowed.
{
    "isBase64Encoded": true|false,
    "statusCode": httpStatusCode,
    "headers": { "headerName": "headerValue", ... },
    "body": "..."
}


⚠ GitLab Server has degraded performance

In ef69a61, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11180 ms

Resolved: GitLab Server performance has improved in 58692e2.


NewNonInteractiveDeferredLoadingClientConfig doesn't override cluster context

I'd like to get a GKE cluster config using NewNonInteractiveDeferredLoadingClientConfig like the following codes, but the config remains default values.  On the other hands, it's possible to get the config if you use clientcmd.BuildConfigFromFlags("", "/path/to/.kube/config").
package main

import (
	"fmt"

	"k8s.io/client-go/tools/clientcmd"
)

func main() {
    loadingRules := clientcmd.NewDefaultClientConfigLoadingRules()
    loadingRules.DefaultClientConfig = &clientcmd.DefaultClientConfig
    loadingRules.ExplicitPath = "/path/to/.kube/config"
    configOverrides := &clientcmd.ConfigOverrides{
        ClusterDefaults: clientcmd.ClusterDefaults,
        CurrentContext:  "",  // or "current-context name"
    }

    config, _ := clientcmd.NewNonInteractiveDeferredLoadingClientConfig(loadingRules, configOverrides).ClientConfig()
    fmt.Println(config)
}

result in below:
$ go run main.go
&{http://localhost:8080  {  <nil> <nil>}    { [] map[]} <nil> <nil> <nil> {false     [] [] []}  <nil> <nil> 0 0 <nil> 0s <nil>}

version in dep:
name = "k8s.io/client-go"
version = "kubernetes-1.11.0"

Isn't the issue that the CurrentContext is empty here? To make that work we have to distinguish between empty and unset, e.g. via turning it into a pointer.

@sttts Thanks for replying. I checked by kubectl config view/current-context but it indicates target cluster context.

@sttts I also filled in a value from kubectl config view/current-context in CurrentContext, but it came up to the same result.

@sttts I added mergo package on Gopkg.toml like istio's, and work well.
[[override]]
  name = "github.com/imdario/mergo"
  revision = "6633656539c1639d9d78127b7d47c622b5d7b6dc"

Which version do we require in client-go (compare Godeps.json)?

currently<PHONE_NUMBER>c1639d9d78127b7d47c622b5d7b6dc
https://github.com/kubernetes/client-go/blob/0b97d57a869d33ee2fe3cc567d0f8553c70357b0/Godeps/Godeps.json#L153-L156

@sttts @liggitt Thanks. It's necessary to install this pkg with this version yourself if you use dep.


Clicking outside of the game window breaks collision with Bullet.

Godot version:

Godot 3.0 RC2
OS/device including version:

Windows 10
Issue description:

Collision body does not work upon leaving game window and re-entering. It'll also sometimes break randomly.
Steps to reproduce:
Run the game, move character with arrow keys pushing against green cube. Then click outside of game window. Re-enter game window and try to move the character against the green cube again. The cube should get pushed around when it's broken the player will go right through the cube.
Minimal reproduction project:

junkforge.zip

Fixed #16757


sw_engine: portability++

Replace long and long long with int64_t to ensure
sufficient viriables size.
@Issue: https://github.com/thorvg/thorvg/issues/2651

on my win machine this sample crashed:
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 40 40">
    <path d="M0 0 1311 1311" stroke-width="10" stroke="#ff0000"/>
</svg>

after this pr, even bigger line worked
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 40 40">
    <path d="M0 0 10311 10311" stroke-width="10" stroke="#ff0000"/>
</svg>

@hermet I changed long/long long only were I needed to prevent this crash, but I'd consider to replace all of them by portable types

@mgrudzinska Since it may affect performance on a 32-bit system, changing data types from 32-bit to 64-bit should be done carefully. We should probably also consider the system architecture and decide on the data type at build time.

32/64 support is a bigger subject and should be treated separately. this issue was solved in a different way


ERROR: no images added (?)

Hi!
So I have started DIGITS and tried to load a reduced data set - here's my set up:
pictures are located in ~/data/train/
I iniate a new dataset and keep the defaults.
Under "Upload Text Files" I give for "Training" a file that contains:
~/data/train/10_left.jpeg 1
~/data/train/10_right.jpeg 1
~/data/train/13_left.jpeg 1
~/data/train/13_right.jpeg 1
~/data/train/15_left.jpeg 2
~/data/train/15_right.jpeg 3
~/data/train/16_left.jpeg 5
~/data/train/16_right.jpeg 5
~/data/train/17_left.jpeg 1
~/data/train/17_right.jpeg 2
~/data/train/19_left.jpeg 1
~/data/train/19_right.jpeg 1
~/data/train/20_left.jpeg 1
~/data/train/20_right.jpeg 1
~/data/train/21_left.jpeg 1
~/data/train/21_right.jpeg 1
~/data/train/22_left.jpeg 1
~/data/train/22_right.jpeg 1
~/data/train/23_left.jpeg 1

and for "Labels" a file that contains:
no_dr
mild
moderate
severe
prolif_dr

I create the data set named "test".
After a few seconds I get an error:
"no images added" - huh?
What am I doing wrong?

The corresponding entries in ~/.digits/digits.log:
2015-04-18 21:01:39 [20150418-210137-f223] [INFO ] Create DB (train) task started.
2015-04-18 21:01:39 [20150418-210137-f223] [WARNING] Create DB (train) unrecognized output: libdc1394 error: Failed to initialize libdc1394
2015-04-18 21:01:40 [20150418-210137-f223] [WARNING] Create DB (train) unrecognized output: ** (create_db.py:2517): WARNING **: Couldn't connect to accessibility bus: Failed to connect to socket /tmp/dbus-VGEIrcS16V: Connection refused
2015-04-18 21:01:46 [20150418-210137-f223] [ERROR] Create DB (train): no images added
2015-04-18 21:01:46 [20150418-210137-f223] [ERROR] Create DB (train) task failed with error code 1

Kind Regards
Raffael

~ had to be substituted with the corresponding full path.


php

Problem description

Problem location

I saw a problem at the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL

Other:

Project version(s) affected

Suggestions for a fix

I assume this was opened accidentally; let me close this one, but feel free to comment after I closed


fix: Remove dependency on Uno.UI for UAP/WinUI

ElevatedView is needed, cannot remove the uno.ui dependency.


Modal Form Inputs are Missing

The form modals we use in various places (Submission, Project, etc) have no UI elements for file pickers or checkboxes:

The site uses custom checkboxes/radios/inputs but for them to work they require the correct markup and classes to be in place.
An example of a file input can be seen on https://apply.opentech.fund/rapid-response-fund/ (scroll down to 'upload'). Inspecting it there are some additional classes on the containing element form__group form__group--file and also on the label form__question form__question--multi_file_field multi_file_input. These are also used by the JS to list out the uploaded documents and their file size.
So i think the work here is around adding those classes in delegated_form_base.html.


Setup Sign in with Google using Keycloak

https://keycloakthemes.com/blog/how-to-setup-sign-in-with-google-using-keycloak

The setup is straightforward. The login page now includes a button to sign in with Google.

TODO

[x] Store the Google OAuth 2.0 client ID and secret to Sage's favorite password manager (see Shared-ROCC folder > Keycloak Test).
[ ] Document how to configure local KC instance with these credentials.

Done


cleanup: Remove MadsimLayer

MadsimLayer isn't functioning as expected, and there are no users or contributors interested in maintaining it. Let's consider removing it.

hi @Xuanwo , i like to help ~

Thanks a lot. I have assigned this issue to you, have fun!

Hi, @zzzk1 do you need help for this issue? Feel free to contact me.


Slack Config Page Updates

Fixes #35
https://www.loom.com/share/c89ee10d044f4dd38950c2878bfd3e48

Don't send tokens or secrets back down to the client from the API, ever

this does mean that to change one field, you have to have all the values, which seems like an okay compromise to make for now, we could eventually implement a more PATCH-y behavior

update form fields for secrets and tokens to type=password
add working, disabled, error states to the form, show any error from the server in a toast
If configured in environment, only show slack client ID, hide the other form fields
Remove redundant placeholder text like "Enter Slack Client ID" - for User/Bot Token
Re-order form fields to match the order that they are presented in the slack app management UI:

Client ID
Client Secret
User Token
Bot Token

I'm going to use the ordering from this PR in #47 for KOTS as well.

saw that this was approved, I just resolved the conflicts and gave it another look


Move gh_repo.coffee to pre-existing scripts/ directory

@loginx please take a look - missed this earlier, but we have scripts/ for sources which don't get built into the lib.

:+1:


Update service-gateway-sso-saml.md

@AdamBJ : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mgblythe : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Nested collapsibles

I have an issue where my inner collapsible collapses, but the outer one does not scale. My code structure looks like this:
<View>
    <Collapsible>
        <InnerComponentWithCollapsible>
        </InnerComponentWithCollapsible>
    </Collapsible>
</View>

Does collapsible not support this or is there possibly something wrong with my code? When I change the outer collapsible into a view, my container expands properly. Also when I just simply get rid of the outside collapsible

I had this same issue. It took me a long time to figure out it was Collapsible that was responsible.

@oblador Can you confirm this behaviour?

Any new info on this looking to do a similar thing where I can infinitely add collapsible lists @iRoachie @oblador also not seeing much activity on this repo should I consider just writing my own  collapsing feature?

@travisday Here's an example using nested collapsibles that work just fine. https://snack.expo.io/BkpT9Zhpb.

@iRoachie Thanks! Works great

I've also had issues with height not increasing in outer collapsible when expanding inner one (a simple menu-like tree). The single change I've made is downgrade lib to 0.9.0, as I not found any significant differencies with example from @iRoachie. Looks like newer versions doesn't support it well.

Yea you're correct, it has to rollback. See https://github.com/oblador/react-native-collapsible/pull/197


Restore Balena support

We broke this some time ago. It's time to fix it now with the new conditional loader.

@vpetersson what is "new conditional loader"?

@vpetersson what is "new conditional loader"?

This is the stuff that I prototyped and you perfected.

Also, what exactly was/is broken?

Give it a spin. The Dockerfile failed for me last time i tried. Moreover, it would have failed even so because of that we tried to do things (such as firewall management) that we would not be able to do in such environment. It did however work in the past.

@a-martynovich i think you forgot to link the PR.


Don't decompile literals in custom field editors by default

Don't merge this just yet because it requires updating the authoring of a few APIs.

@riknoll conflicts

This is good to merge. Just need to make sure to bump the pxt-common-packages in a target if we bump to this version of PXT.

Can you merge and bump?


[GOBBLIN-961]Bypass locked directories when calculating src watermark

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-961

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Background:
Found the issue where there are locking directory in place, which gobblin failed to scan through list of files (source) to copy, and the watermark retrieve as NAN.
Therefore, data is not able to copy successfully and possibly causing the delay
Solution:
Simple change to bypass locked partitions when calculating src watermark

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Test on azkaban to make sure when there is locked partition, it can bypass that partition and do copy

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2810 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #2810      +/-   ##
============================================
- Coverage     45.47%   45.46%   -0.01%
+ Complexity     8908     8906       -2
============================================
  Files          1899     1899
  Lines         71096    71098       +2
  Branches       7833     7833
============================================
- Hits          32332    32327       -5
- Misses        35784    35787       +3
- Partials       2980     2984       +4

Impacted Files
Coverage Δ
Complexity Δ

...ement/copy/replication/SourceHadoopFsEndPoint.java
19.04% <0%> (-0.63%)
6 <0> (ø)

...ava/org/apache/gobblin/fsm/FiniteStateMachine.java
73.48% <0%> (-3.04%)
18% <0%> (-3%)

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
63.42% <0%> (-1.86%)
27% <0%> (-1%)

...main/java/org/apache/gobblin/util/HadoopUtils.java
30.2% <0%> (-0.34%)
24% <0%> (ø)

...main/java/org/apache/gobblin/yarn/YarnService.java
15.68% <0%> (+0.84%)
4% <0%> (+1%)
:arrow_up:

...lin/elasticsearch/writer/FutureCallbackHolder.java
62.85% <0%> (+1.42%)
4% <0%> (ø)
:arrow_down:

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 a714c94...110c8c8. Read the comment docs.


feat(workflows): add tags

Changes

#956  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


Nate/add to existing

Summary
Don't merge - wip
Add smart wallet connectivity to an existing app
How did you test your changes?
markdown preview
running through steps locally with cra

closing since we don't want to direct anyone to more readmes here. building better docs elsewhere


Use latest versions of libraries

Hi @2307vivek,
Thanks a lot for your initial work.
I've created this PR in order to have the latest versions of libraries.
I hope you will merge it for the users who want to test Paging Compose library.
Have a great day,

Thanks for the PR, I will merge it once I review it.


uv pip install fails when install from files with -r, but passes "regularly"

Here's a minimal repro of what fails:
I have:

requirements-dev.txtaltair>=5.4.0

requirements-ci.txt--extra-index-url https://download.pytorch.org/whl/cpu
torch

Then:
✅ the following passes:
python3.11 -m venv .venv
. .venv/bin/activate
uv pip install altair --extra-index-url https://download.pytorch.org/whl/cpu torch

🔴 the following fails:
python3.11 -m venv .venv
. .venv/bin/activate
uv pip install -r requirements-dev.txt -r requirements-ci.txt

Output from the one that fails:
  × No solution found when resolving dependencies:
  ╰─▶ Because only typing-extensions{python_version < '3.13'}<4.10.0 is available and altair==5.4.0 depends on
      typing-extensions{python_version < '3.13'}>=4.10.0, we can conclude that altair==5.4.0 cannot be used.
      And because only altair<=5.4.0 is available and you require altair>=5.4.0, we can conclude that the requirements
      are unsatisfiable.

✅ the following also passes:
python3.11 -m venv .venv
. .venv/bin/activate
pip install -r requirements-dev.txt -r requirements-ci.txt

uv pip install altair --extra-index-url https://download.pytorch.org/whl/cpu torch

This one probably passes because you're not including the altair>=5.4.0, just altair (and from the error, it's clear that's important)
pip install -r requirements-dev.txt -r requirements-ci.txt

This one probably passes because it's looking in multiple index URLs, you're probably looking for --index-strategy unsafe-best-match. See the pip compatibility guide for details.

Ah, thanks @zanieb ! https://download.pytorch.org/whl/typing-extensions/ only goes up to 4.9
So, installing typing-extensions separately and before all the other requirements worked 🥳
python3.11 -m venv .venv
. .venv/bin/activate
pip install -U uv
uv pip install typing-extensions>=4.10
uv pip install -r requirements-dev.txt -r requirements-ci.txt

That'll do it too! Glad you found a solution.


Why require users to set GCLOUD_PROJECT?

The current project is accessible in App Engine the the GOOGLE_CLOUD_PROJECT environment variable.
It is not clear why we ask users to set GCLOUD_PROJECT in config.js.
Or, if not set, why we do not fallback to the GOOGLE_CLOUD_PROJECT environment variable.

Good catch. I guess we wanted the config file to determine which project is used rather than rely on environment variables.

But what is the rationale?
This adds unnecessary friction to the sample.

I misunderstood. I guess there was a reason in the past to set it up like this. I'll work on changing it. The docs around config.json are outdated and need to be fixed, too.

Obsolete with planned rewrite of the tutorial.


feat: Initial Release SQL Helpers

SQL Helpers provides utilities that are intended for use with SQL database related instrumentations.
The initial release adds support for a Shared DB Context Helper, whose goal is to replace the instrumentation specific helpers.
Future releases will target replacing sanitization/obfuscation helpers.
cc: https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/1194

Thanks for your work to reduce duplicate code!
I'm getting a little stuck thinking through the future plans for the SQL helpers gem. They may not need to block this PR, and this might not be the right forum to talk this through. Please redirect me if there's a better place.
When we created the SQL obfuscation gem, IIRC we intentionally separated the MySQL-specific code into a separate gem, in the spirit of providing the minimum amount of additional code per library. It seemed like we wanted to keep helper libraries small, despite the potential annoyance of having extra libraries.
I don't see this context management code in the PG instrumentation. However, I see it copied in the MySQL2, Trilogy and Redis libraries.
With this new gem, are we shifting our paradigm a bit?
Even though it's a NoSQL library, Redis could still use the same code. However, I don't think it'd benefit from the SQL Obfuscation code.
At a high level, it seems like the context management code could be beneficial for non-SQL libraries as well, the only unique element is the name of the context key. Should the SQL scope for this helper be removed?

Great questions and observations.
If I'm understanding you correctly, you're seeing
an opportunity here for us to eliminate structural duplication between these context helpers since they are structurally all the same and only differ in the a context key.
I suspect that PG does not have a similar helper because Shopify does not use postgresql. They first introduced this helper for the MySQL2 gem but recently switched to Trilogy and needed the same functionality. I suspect copy/paste modify was the quickest way to do that.
I asked them if there would be any concern in consolidating those and there wasn't any objection.
I did not have plans on going the same for PG but I think it's only right to be symmetric and follow up with some updates there.
My goal with this gem is to have shared helpers for all SQL backed Relational Databases so I don't think of sharing this gem with Redis. In the case of Redis, I think we'd want to look at some use cases to see where it may make sense to reuse the same SQL context helper. I'd be worried that SQL DB semantics would end up inadvertently added to Redis spans. But again that is speculative.
We can chat more about this at the next SIG


Fix image types for GIFs and PNGs.

Turns out this fix was incomplete, see https://github.com/clach04/pypub/commit/22e61c13015b23a498ecaf1853c7c2348fb1cef6


Update support links

The support links are referring to the Rancher docs/issues, etc. Change to Harvester links.
To Reproduce
Steps to reproduce the behavior:

Click Support in the bottom-left corner.
Check the links in Community Support

Environment:

Harvester ISO version: master-48c97ea7-head

duplicate of https://github.com/harvester/harvester/issues/1300, close as that one is more detailed.


Adding Cortex Prime Tales of Xadia Play Test sheet to master repo

Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hey there @tapiochre, could you remove the .DS_Store file from the root folder of your PR? Thanks for all the work you do!

I'm now getting a "You cannot modify files in the root directory." validation comment.
I hope this is resolved, but any advice would be appreciated.

Hey there @tapiochre, the easiest way would be creating a new branch from master and copy only the changes you have made to the sheet and make a new PR.

Hey there @tapiochre, the easiest way would be creating a new branch from master and copy only the changes you have made to the sheet and make a new PR.

Can I just check is this still an issue?
I can branch and pull - will it make tonights deadline?

Yes, your PR is still failing on the not modify root rule, @tapiochre. If you make a new PR I can merge it in a few minutes as I am sitting in the front of the git repo right now ;)

Yes, your PR is still failing on the not modify root rule, @tapiochre. If you make a new PR I can merge it in a few minutes as I am sitting in the front of the git repo right now ;)

Ok I'm free to do this now... one min

New Pull being commited shortly