Fusion workfile template builder

Changelog Description
This PR starts the work to port Workfile Template Builder to Fusion!
I have gotten the creation of the placeholder nodes down with its data creation and so on.
Currently it can load footage but there are a lot more to fix and test until its ready for prime time.
My focus is to get everything up and running correctly and after clean up the code and the menu UI.
 They will be converted to functional Fusion code while I'm working on this PR.
Additional info
For the placeholder node I have created an "OPEmpty" fuse. It simply pushed the image along and have 0 controllers on it.
The placeholder-creator populates the fuse with the needed info on creation.
I took this path as it's waaaay easier and faster to search for a specific type of node in Fusion than taking an existing node, strip it of everything and then populate it with the needed info.
If there are any backlash I guess it would be done but this way is just easier for everyone.
Testing notes:

Follow this tutorial to set everything up: https://www.youtube.com/watch?v=kfMJn3SXUGM
Try it out!

You can create templates as expected that includes loaders and creators!
workfile_template_builder.py contains a bunch of TODO's still. Biggest TODO is to convert the siblings code (line 264-295 and 545-576 inc 2 functions each).
I don't fully understand what siblings does or when those parts would get executed so... Any help on those parts is highly appreciated!

Now it's time to start checking out the code if you want @BigRoy :)
There are a lot of duplicate codes between the classes FusionPlaceholderLoadPlugin and FusionPlaceholderCreatePlugin. Idk if it's possible to share it somehow? Or maybe it's just best to have them split if there are any need to modify per class later on.

@iLLiCiTiT I saw that you made most of the code for the Nuke implementation of workfile_template_builder.py
I would love to know what the siblings-code does in it to know if it's a needed part for the Fusion implementation and how I should code it up. :)

@friquette it seems like it was you who made the Nuke version.
I would love to know what the siblings-code does in it to know if it's a needed part for the Fusion implementation and how I should code it up. :)

Hey, thanks for your work, man!
Just in case, if you'd want to continue working on this, I updated this branch with the latest develop branch, resolved the conflicts and cleaned up all black formatting changes, so the diffs would be more clear. We can reformat the code later.
Feel free to merge on top of this PR.
https://github.com/ynput/OpenPype/compare/develop...movalex:OpenPype:feature/fusion-template-builder

Hey, thanks for your work, man! Just in case, if you'd want to continue working on this, I updated this branch with the latest develop branch, resolved the conflicts and cleaned up all black formatting changes, so the diffs would be more clear. We can reformat the code later. Feel free to merge on top of this PR. develop...movalex:OpenPype:feature/fusion-template-builder

Thanks @movalex !
I currently don't have much time to work on this integration so if you wish to take over the development of it I'm more than happy to do so!


New version?

It's been two years since the previous version (see #180), and some useful changes have happened since then. Could I ask you to create a new version of the current stable state, please?

Of course! You got it man:-)
Here you go: https://github.com/andsens/homeshick/releases/tag/v2.0.0


[Bug]: SIM card not detected though it is inserted

Description
The airtime purchase is not completed because the app is unable to detect the SIM card
Steps to reproduce

Open Stax app
On homescreen, tap the purchase airtime button
Select destination network
Enter the recipient number
Enter the amount
Tap "continue"
Enter M-PESA pin

Expected result
A confirmation from Safaricom showing a successful transaction and a similar amount reflecting on my network SIM-card
Actual result
I receive a "sim card error" message even though sim card is well inserted.
Link to screenshots/screen recordings
https://drive.google.com/file/d/13DWB8dxR5VdN0yANI4dPBB45WKMUF_3c/view?usp=sharing
Device name
Alcatel 9003X
Android OS
6.1
Is your phone dual SIM or single SIM?
Single SIM
Which telco are you using for this test?
Safaricom
Stax device ID or email address
<EMAIL_ADDRESS>

My pleasure

Are you still experiencing this issue @ashioyajotham

No sir.

Closing the issue then.


Review for accessibility examples

Michael and Jesse -- would love your review on the following examples from #912
accessibility-listeners
accessibility-updating-pdom
accessibility-slider
accessibility-animation
Thanks in advance!

General:

These entries aren't in the index page: scenery/examples/index.html

Accessibility Listeners:

Instead of using switchShape, I thin kit makes more sense to add the square as a child to the scene, and then swap out the hex/circle as children to the square. You can overwrite the children like square.children = [...] even if there is just one item in it. That may be a nicer pattern. Also note that I think it is a bug that the startingShape is disposed when swapped.
It is fragile to access a Node like scene.children[1].fill.

Updating PDOM:

// creates the scenery node to which both hexagon and buttons will be added.
const node = new scenery.Node();

Rename node to something nicer, like perhaps "container"

clearScene is kinda in no-mans land, splitting up to section resonsible for displaying the PDOM. Can we bring it up to be with the rest of the example?

clearScene could just overwrite children like scene.children = [].

The dispose method is only for when you never want to use that Node again, and you are trying to prevent a memory leak (where the Node can't be garbage collected because there is still a reference to it). This makes sense for the buttons, but not for the hexagon. Why not just have an easily cleared container just for the buttons. Then you don't have to worry about re-adding the hexagon each time anyways.

addPressListener sounds like a function. Perhaps rename to something else.

colorsToGenerate.splice( 3,

While this works, it is fragile. I would come up with a better way to handle this. Perhaps keep a list like initialButtons that you can reset back to.

In createButtons, I see you use forEach with an outside var. This is totally fine, but to me forEach takes an independent callback. I think that a traditional for loop would be a bit nicer here.

I think that this is a really good example. I like that you use PressListener, and I think that its complexity adds to the quality of example. It may be helpful to add a code comment about why use the PressListener, what types of events the PressListener handles (especially a11y events). At the very least explain the conversion of press to input events.

in generateButton, rectangle should add a PressListener, it is more symmetrical to the rest of the example. (don't forget about disposing!)

I think there could be some improvement on organization, for example putting colorsToGenerate before all of its usages.

Accessible Slider:

In this example, the updatePDOMDisplay function isn't quite working for us. A little background: this is just a hacked out function @jessegreenberg and I worked on one day, it could definitely be improved on, perhaps we could add more line breaks so that we don't have that nested scroll bar for that container. For example, I quickly added this line of code to get a few more line breaks, but I'm sure it can be improved upon:
   var extraFormattedPDOM = formattedPDOM.replace( /[\w-]+="/g, '\n$&' ); // add breaks in the string

Noted the rainbow gradient TODO, now could be the time to do that.

rename ticks to be numberOfTicks

// checks to see if thumb moves off the track (and doesn't allow). This comment is to a line of boolean logic that could be greatly improved with factored out var names or doc or something. It is also more than 120 chars.

The final "else" in the display's inputListener seems like a great chunk for its own function.

Accessibility Animation:

same thoughts about dispose as above examples
There is large duplication for the initial hexagon and when creating each subsequent one. I would recommend factoring that our, perhaps to a class.
 scene.children[ 0 ] is a fragile way to access the initial hexagon, furthermore this is also duplicated and could go into that Path subtype class
I'm confused by the a11yclick event on the display. This is atypical because normally you only want to add a11yclick on HTMLElements that produce the click event. I guess the display is a div in the PDOM, and can be clicked with the spacebar. Even so I would recommend listening to keydown in this case.
initialScene as a name gives no understanding about what it returns.
In updateOnRequestAnimationFrame, updatePDOMDisplay only needs to be called once per requestAnimationFrame, not for each item in the scene
child.tagName should not be set on each frame, just on creation
Is is sloppy to access the zeroith child to get the fillValue. You can set/reference this easily if a class is created, described above.
child is not the most descriptive var name for the loop in updateOnRequestAnimationFrame.

Thank you for these review comments, @zepumph! I'm starting to take a look at them this week.

@zepumph thanks again for these comments, and for your patience with me getting to them! I've been adding examples to the index page as they're updated to address your comments. I re-did the css, since I do think we need more space to see the pdom than half a page will allow. I also hacked together some regex to format the pdom more nicely. I still think these could be improved in the following ways:

[ ] I think it's confusing to have a slider made from scratch when AccessibleSlider is a type. I wonder how this example can be updated to either use AccessibleSlider (tougher since it lives in Sun), or at least could contain documentation noting that the type exists.

[ ] The regex that formats the PDOM could be much nicer and more standard. Right now, it needs to be customized for each example to respond to the level of nesting contained in the example, and the specific contained tags.

[ ] I'm still working on the review comments for accessibility-slider, but the rest should be implemented.

@zepumph, okay, I believe I've implemented all of your suggestions. Let me know if you'd like to review again! Assigning over to you for next steps.


Set output encoding to UTF8

Fixes #635
cc @mlorbetske

Rebased from rel/2.0.1 and removed the change for test projects as that's already made in rel/2.0.1.
Will merge this if the CI build is green.


Error status 0x0181 sent back in BLE write call-back.  What does it mean?

My app is getting back the error status 0x0181 (385 decimal) in the BLE write callback when it tries to write to the FTMS's Fitness Machine Control Point characteristic 0x2AD9.  Browsing the NimBLE code, I can't figure out what this status code means.  The BLE device rejecting this write request is a ZwiftHub indoor bike trainer.  No idea what BLE stack this device uses.  Does anyone know?

Hi,
In Nimble error codes have ranges so that one can figure out from which layer error originated from.
0x181 is 0x100 + 0x81 which means it is ATT (0x100) error code with value 0x81.
0x81 is from ATT application error code range which means that its meaning is defined in FTMS specification.
More info on errors can be found in documentation: https://mynewt.apache.org/v1_12_0/network/ble_hs/ble_hs_return_codes.html

Thanks for your clear explanation @sjanc !
For NimBLE neophytes (such as me) it would be helpful to have a comment with your clear explanation in the definition of the 'struct ble_gatt_error' inside ble_gatt.h.


OOM tracking: backport #1348, #1346 and #1340 (release-1.17)

Backports #1348, #1346 and #1340.
/sig scalability
/cc jkaniuk
/cc wojtek-t

/lgtm
/approve


deep steganography issue

Traceback (most recent call last):
File "Steganography.ipynb", line 401, in
"execution_count": null,
NameError: name 'null' is not defined

I got the same issueTT

I got the same issueTT

you could change :
null =>  ' '
However, it couldn't run either


If the time is not reach starttime countdown should appear, but there is only MINT.

not worked countdown function.
isActive useState is not work!!
const [isActive, setIsActive] = useState(false); // true when countdown completes

#50
Try this

why not worked countdown function..?

#50
Try this

i need isActive function :) automate countdown -> mintbutton..

<Countdown
date={StartDate}
onMount={({ completed }) => completed && setIsActive(true)}
onComplete={() => setIsActive(true)}
renderer={renderCounter}
/>
is my isActive const change true at started...!

(new Date(props.startDate));

i tryed to remove setStartDate(goLiveDate); but not work haha just connect wallet. isActive value change to true

(new Date(props.startDate));

i tryed to remove setStartDate(goLiveDate); but not work haha just connect wallet. isActive value change to true

Did you do the thing mentioned in #50 and added "000" to end of your unix timestamp?

(new Date(props.startDate));

i tryed to remove setStartDate(goLiveDate); but not work haha just connect wallet. isActive value change to true

Did you do the thing mentioned in #50 and added "000" to end of your unix timestamp?

I did Unix timestamp * 1000
Can u show me ur modified src?

connect wallet -> onMount={({ completed }) => completed && setIsActive(true)} tick up true !!

try adding a hardcoded date to the Countdown date props like a date = {Date.now() + 10000} and see if the coundown turns up.

send the code. lets see


prometheus: 2.13.1 -> 2.14.0

Motivation for this change
Changelog: https://github.com/prometheus/prometheus/releases/tag/v2.14.0
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build prometheus

The tests are failing for me. An extra dependency is apparently needed.
running tests
compilation error :go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp: lookup proxy.golang.org on [::1]:53: read udp [::1]:56691->[::1]:53: read: connection refused

FAIL    github.com/prometheus/prometheus/cmd/prometheus 0.096s
FAIL
builder for '/nix/store/f6lmxbn3fmjflh4x43dzlzj7m7r27fbk-prometheus-2.14.0.drv' failed with exit code 1
error: build of '/nix/store/f6lmxbn3fmjflh4x43dzlzj7m7r27fbk-prometheus-2.14.0.drv' failed

Why do they want to send data to opencensus.io anyway? :scream:

Why do they want to send data to opencensus.io anyway?

I don't think that's what they're doing; opencensus is the tracing and metrics library that prometheus is using.  It's just another OSS project.

the error happens because in buildGoPackage the go command tries to build the module in the checkPhase, and fails with invalid certificate. Fixed in https://github.com/NixOS/nixpkgs/pull/73380

I've added a fix until #73380 lands in master


Interface is very small on high DPI displays

On high resolution displays the UI is tiny.

It would be nice if there were an option to increase UI scale, or otherwise detect DPI and scale everything accordingly. Not sure if it's doable with GML but a UI scale slider might work.

A scaling option could be useful not only for high DPI monitors, but for regular ones as well, when your song contains too many layers. Zooming out in a song would allow for more layers to be displayed on the screen at once, making navigation a lot easier (for example, when you have to select a big portion of a song).

This would take a while to implement but I do think it is useful
The menu/option bars should retain the same size, but the size of the note blocks being small like this should be an option.
For example, this song still requires you to scroll down to see everything, even with the high DPI.

I'll see if I can implement this, but Note Block Studio's engine is rather rigid.

Any update on this? I'm having the same problem.

@vberlier Hello! Sadly, changing your scaling settings on Windows remains the only way to remedy this. As version 4.0 will be delayed a bit, I'm currently looking for a "hacky" solution in the current version, which would be to make everything half the size, and then scale it to the whole window. Since the program is a bit weird on how it draws things, I'm not sure if that would work at all, but I'll make an update here as soon as I give it a go. :)

Update: I finally got it to work! I'll try to ship an update as soon as possible with this patch. 😄
(please don't mind my regular screen size haha)
https://user-images.githubusercontent.com/29354120/114827552-35a07080-9d9f-11eb-8ff8-76dc19948d9e.mp4


Improvement to readme.md

Added some instructions to make it work.
If for some reason the local build fails, faas-cli deploy downloads the image from DockeHub, which is broken currently. The code in the image is intended to read the GITHUB_TOKEN from a secret, but is incomplete.
Signed-off-by: Patricio Diaz<EMAIL_ADDRESS>

Thank you @padiazg ! Yes, as you've mentioned, I need to update the code as well. The most up to date version is in my forked version.
https://github.com/kenfdev/faas-github-stats
Need to get these merged.


writes script to batch exclude reportbacks

What's this PR do?
Provides a new custom drush command to batch exclude reportbacks to clear campaign inboxes.
Command can be used for any campaign and campaign run by taking the following parameters:
-nid - campaign nid
-run_nid - campaign run nid
-quantity_limit - threshold for exclusion - reportback quantity will be less than this number
How should this be manually tested?
run drush batch-exclude-reportbacks command with above parameters
check campaign reportbacks and make sure only reportbacks that are under the quantity threshold remain in the reportback inbox
e.g. Run drush batch-exclude-reportbacks 1321 1837 200. On your local, visit http://dev.dosomething.org:8888/us/node/1321/rb and make sure only reportbacks with quantities over 200 remain in the inbox.
What are the relevant tickets?
Fixes #6392

👍


fix: enable namespacePrefixWarning in subscription

Checklist

[ ] npm test passes
[ ] tests are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Description of change

any feature?
close https://github.com/dvajs/dva/ISSUE_URL

Coverage increased (+0.06%) to 86.557% when pulling 0bf7fda08c924f170a238415596043af645b9ca0 on zhanba:master into a5e4ca58b8d40685545cc5bff6a92736afc94d24 on dvajs:master.


Fine-tune a LLaVa model for Visual question answering task on some custom set of images.

Question
I want to fine-tune a LLaVa model for Visual question answering task on some custom set of images. I wanted to know the Dataset format required for training and then fine-tuning. I found this github folder: https://github.com/haotian-liu/LLaVA/tree/main/scripts/v1_5 . And I found this is an example dataset: https://huggingface.co/datasets/liuhaotian/LLaVA-Instruct-150K/blob/main/detail_23k.json . But since I am new in these tasks still I didn't get a good idea about how to do it correctly. Can someone direct me?

I am not sure what you don't know. You can finetune with the finetune script with the llava format dataset.

I am not sure what you don't know. You can finetune with the finetune script with the llava format dataset.

Thank you so much! I am sorry, and I am confused. I know how to train CNN models. We need to access the image folders along with the labels. But here in those Github folders, I cannot see any image data folders. Can anyone tell me where we need to keep the image folder along with that json file for fine-tuning the LLaVA?

@anjanakg  oh I see.
U can setup your dataset along with this script https://github.com/SkunkworksAI/BakLLaVA/blob/main/setup_finetune.sh .
The script is from another repo. You need to download the datasets from different websites and it's annoying. Using this script makes it easier

@anjanakg oh I see. U can setup your dataset along with this script https://github.com/SkunkworksAI/BakLLaVA/blob/main/setup_finetune.sh . The script is from another repo. You need to download the datasets from different websites and it's annoying. Using this script makes it easier

Oh...! Understood! Thank you so much! I will try in that way.
So, I need to create a json data file(with required format) for my images and keep the image folder somewhere on my local computer or cloud. And, then give the link to the folder and the json file as in that fine-tuning script. Am I correct? If not could you please correct me?

@anjanakg Hi, hope you are doing well. I have certain doubts on training this model. if you don't mind can we connect regarding this. gmail<EMAIL_ADDRESS>

Hi @anjanakg , @gyupro ,
Could you please help me out how to fine tuning the model using custom dataset. it would be better if we connect on it.
email<EMAIL_ADDRESS>Thank you


Make pass source follow symlinks

Since my hosts share some secrets, I use symlinks inside my password-store. Currently, krops does not copy the secrets that are linked, since find -type f ignores them.
This PR adds the -follow option to find to also include symlinked secrets.

tested it, worked. Actually a cool idea. thanks!


HTML/CSS files added, route created

Changes in this PR

Created sign-out.html file and built using GDS components
Create route for sign-out page
Create sign-out.css file and linked to the base.html
Added minimal custom css class to GDS button

JIRA ticket
https://national-archives.atlassian.net/browse/AYR-442
Screenshots of UI changes
Before
After

[ ] Requires env variable(s) to be updated

Have deleted the SCSS footer file. @colinbowen @anthonyhashemi


README.md references nonexistent command

ko auth login my.registry.io -u username --password-stdin`

from README.md produces
Error: unknown command "auth" for "ko"

This should probably be crane auth

Yup! Dupe of https://github.com/google/ko/issues/155
Going to close this in favor of that issue, but it's currently in master and not in any release, so we need to cut a release (and fix the help text).
Thanks 😄

FWIW we cut v0.7.0 which should include this command (which I've moved to ko login).


Add Light & Dark mode in Contributor page & Rate Us page

I want to add Light & Dark mode Toggle system at Contributor & Rate Us pages.
Please Assign me under JWOC 2024.

@Jagroop2001
Please assign me under JWOC

Hello @Jagroop2001, I would like to express my interest in working on this issue as part of WOC'24. Please consider assigning it to me. Thank you!

@rahulkhandait-sde , go for it.


Small amount of input lag when linting

I noticed that there is a small amount of input lag when linting a file (i.e. the cursor does not move, then all actions buffered during the lag are unloaded when the lag ends). This lag is not present in validator.vim, a similar plugin with a bit less filetype support. Is there a way to improve this plugin so there is no input lag?
I tried both the flake8 and coffeelint linters, both with the same noticeable lag.

Never mind, this mysteriously went away on its own.

Actually, it seems to have returned - not sure what caused it.

Could you run some profiling, per the advice in this Stack Overflow answer? http://stackoverflow.com/a/12216578/1836507
That's the best way to find out what's causing the problem.

Here's my profile.log, with the following .vimrc, run on a coffee file:
call plug#begin('~/.vim/plugins')
Plug 'sheerun/vim-polyglot'
Plug 'w0rp/ale'
call plug#end()

 8              0.985705         let l:job = job_start(l:command, l:job_options)

Hmm, there's where the problem is coming from. That's interesting. Almost a full second spent on calling job_start, which is what starts the jobs in the background. That's quite unusual. That means you're seeing a high cost just for starting a job on your machine.
What kind of machine are you testing on? I wonder if there's something which would make forking a new process take a long time.

I'm running a 2015 Macbook Pro:

I should also note that this issue has never happened with validator.vim, a similar plugin.

Could you try editing the plugin with this patch?
diff --git a/autoload/ale/engine.vim b/autoload/ale/engine.vim
index 25be045..879c371 100644
--- a/autoload/ale/engine.vim
+++ b/autoload/ale/engine.vim
@@ -269,11 +269,6 @@ function! s:RunJob(command, generic_job_options) abort
         else
             " Execute the command with the shell, to fix escaping issues.
             let l:command = split(&shell) + split(&shellcmdflag) + [l:command]
-
-            " On Unix machines, we can send the Vim buffer directly.
-            " This is faster than reading the lines ourselves.
-            let l:job_options.in_io = 'buffer'
-            let l:job_options.in_buf = l:buffer
         endif

         " Vim 8 will read the stdin from the file's buffer.
@@ -299,7 +294,7 @@ function! s:RunJob(command, generic_job_options) abort

             call jobsend(l:job, l:input)
             call jobclose(l:job, 'stdin')
-        elseif has('win32')
+        else
             " On some Vim versions, we have to send the buffer data ourselves.
             let l:input = join(getbufline(l:buffer, 1, '$'), "\n") . "\n"
             let l:channel = job_getchannel(l:job)

I wonder if sending the buffer lines into the command manually might be faster in some versions of Vim. If this is the case, then I'll write an option for this, so you can use that instead.

Seems to work! I'll let you know if the issue resurfaces.

You can now add the following to your vimrc file to use ch_sendraw() for sending buffer text to commands instead.
let g:ale_use_ch_sendraw = 1

The option has been added to the documentation, and there's a test for it.

Unfortunately, this issue seems to have resurfaced again, even with the new option set.
Here is a profile.log, with the .vimrc I used and the file on which it occurred.

12              1.645295                     call ch_sendraw(l:channel, l:input)

The cost is definitely coming from sending the buffer data to the command. The expensive processing happens inside of Vim. I'm not entirely sure how to help there, other than suggesting a different version of Vim.

One thing I can suggest is increasing the delay before linters are called. You can set this with. let g:ale_lint_delay = some_milliseconds. Or, you can disable linting while you type entirely, with let g:ale_lint_on_text_changed = 0. You could turn on linting on save, with let g:ale_lint_on_save = 1, if you want that.

I just realised that this advice is generally pretty good for tweaking performance. I'll make a note to create an FAQ entry with this advice. The default settings work for a lot of people, but default settings never work for everyone.

I'm using the latest version of Vim on brew. In terms of performance, I'm testing input lag by making a change in the text file, then hammering j in normal mode to observe any input lag. This should be independent of the g:ale_lint_delay setting, as that only controls how long until the linter is run.
It's unusual that validator.vim does not have this problem whatsoever. Here is the profile.log, with this vimrc.

I don't know why you're seeing such bad performance here. The performance of how fast Vim sends data to a program is out of my control. I can only suggest increasing the delay or disabling linting while typing.

Could an option be added to reset the g:ale_lint_delay timer on cursor move, to mitigate the probability of seeing input lag?

That's the current behaviour.

@metakirby5 Try out ALE again with the latest commit. I have switched to using neither in_io nor ch_sendraw. Now the buffer gets written to a temporary file which is automatically deleted, and the piped in via the commandline to whatever program is being used. That should hopefully fix your issues.

Seems to work like a charm! Thanks for investigating :)

Hate to dredge this up again so quickly, but it seems this temporary file fix causes a bug with set bt=nofile. You can replicate this via:
vim -c 'let g:startify_disable_at_vimenter = 1 | set bt=nofile | set ft=python'

Then, type in a few characters.
The error looks like this (pictured is my plugin codi.vim, which is how I discovered this bug):

Okay, try it again. The bug came from the code trying to get the basename for a buffer which has no filename. Now when the buffer has no name, ALE will use a dummy filename with a guess for the file extension. The file extension guessing code is auto-loaded so it shouldn't increase load times in most circumstances.
I couldn't write a Vader test for empty buffers themselves, but I could write Vader tests for the file extension guessing.

Awesome - looks like it's fixed :)

:+1:


feat(forms-web-app): add appeal site address

Ticket Number

UCD-1161
Description of change

Add appeal site address
Checklist

[ ] Requires infrastructure changes
[ ] I have updated the documentation accordingly
[x] I have merged commits to avoid fixing things in this PR
[x] New features have tests
[ ] Breaking change (team conversation required)

Important
Please do not merge from master (please only rebase). This keeps the history linear and easier to debug.

ty


Can't open reaper now that I created reasonus with my fadrport 16

Hello, I was trying on my latest veroin of Windows 10 reaper to use reasonus .  I added the osc controll, i think that is waht it is called, or it is CSI (CONTROL SERFICE INTIGRATOR) in reaper.  But when it asked me in the reaper preferences to add my faderport or the csi control of my faderport 16, reaper crashed.  Now I cant get back to the preferences in reaper to delete the csi that I added.  How do i get back into reaper without reaper crashing?
Lou

RESET REAPER TO FACTORY RESETS AND IM GOOD NOW


read Signal without cloning

Recently Signal::sample_with() was removed. I was unable to find an alternative to get a reference to the signal's value. This is important to me because I store Strings and other big types in my text editor project.
Is there really no way to get a reference or should I go RTFM?

The ability to get a reference to the value was removed because it could cause a RefCell borrow violation (in the old version) under some special condition, and that code was really complex and
hard to debug. The new version of Signal doesn't even have that, it's just a function:
pub struct Signal<T>(Arc<dyn Fn() -> T + Send + Sync>);

It's much simpler now, but AFAIK you can't return a reference to closure data. Maybe it can be done with some hack I yet have to research. For now you have to wrap your types on Arc or similar to prevent the cloning (or make them cheap to clone). If you're using strings you may want to look into string interning libraries.


Environment vars that are multiline text are getting newlines removed

If you specify an environment variable like the following in the rancher-compose file:
  environment:
    GITLAB_OMNIBUS_CONFIG: |
      foo = bar
      port = 1234

And then try to do an upgrade. The environment variable is displayed in a text field. This should be a text field or text area instead. But this isn't just a display issue, the real issue is that the newlines are removed and so that it's impossible to accurately modify during an upgrade.

It doesn't look like multiline environment variables are supported in docker-compose?
https://docs.docker.com/compose/compose-file/#environment

Nothing in that documentation indicates to me that a value of the environment dictionary can't be a multiline string. I'm working off of the Gitlab Omnibus docker-compose file seen here: https://gitlab.com/gitlab-org/omnibus-gitlab/blob/master/docker/docker-compose.yml
It works fine in docker and docker compose.

This is just a UI issue, <input type=text> can't store newlines.

Yes, sorry I didn't make that clearer.

Version - v1.0.1-rc3
Verified fixed


FIX: Add tool support to open ai compatible dialect and vllm

Automatic tools are in progress in vllm see: https://github.com/vllm-project/vllm/pull/5649
Even when they are supported, initial support will be uneven, only some models have native tool support
notably mistral which has some special tokens for tool support.
After the above PR lands in vllm we will still need to swap to
XML based tools on models without native tool support.

Still need to add a test to validate this stuff cause the existing test is not properly addressing this.


Remove dependency on github.com/blang/semver

Since #374 we have dependency on both github.com/blang/semver and github.com/Masterminds/semver. We introduced github.com/Masterminds/semver so that we can support certain version range syntax (#379).
I believe we should be able to get rid of dependency on github.com/blang/semver. As far as I see, the only place where we use it is here:
https://github.com/operator-framework/operator-controller/blob/05ffd850dc099f215b09b3427f49692cdd9a7a0a/internal/catalogmetadata/filter/bundle_predicates.go#L34-L42
This piece of code was migrated from old entity source code.
And we only use InBlangSemverRange here.
I believe we can replace InBlangSemverRange with InMastermindsSemverRange in this ocassion.

@tmshort could you please take a look at this issue? I think we can remove github.com/blang/semver, but I have a feeling that I'm missing sometihng. I would really appreciate if you can confirm this or provide some background into why we did not remove InBlangSemverRange in #374.

Ah! I see this thread now. We need blang to support olm.package.required :(
Closing this one.

We can't get rid of blang/semver since it's used to parse bundle versions. The syntax different between blang and Masterminds is subtle, and we don't want to break bundle version checks.


T1 Mask not at the same resolution of DWI

Hi!
I noticed that the resolution of the T1w mask is not the same as the DWI. While I do agree that they are not supposed to be the same, it might be very useful to have the T1w mask resampled to the DWI resolution.
This is generally required for across-session/ longitudinal studies or test-retest types of analyses.
A simple mri_vol2vol or dipy_reslice should be very fast and simple to implement!

@ShreyasFadnavis could you take a look here and let me know if this clears up how the DWI masking is done for reconstruction workflows?


GOSS benchmarking allowed? (NDA question)

@guolinke NDA question: is it allowed to provide public benchmark results of GOSS before the paper is out?
If yes, which PR do you recommend to use? I am currently using 1bf7bbd for benchmarking stuff on v2.
(just wondering, because the v2 pull request comment is empty now)

@Laurae2
better to keep these benchmark private for now since they may be not the final solution.  Thanks !


Why doesn't it disable draggable?

Hello team formkit,
I'm trying to disable boardFirst so that board Second doesn't receive more elements but it's not working... I don't know what I'm doing wrong.
Thank you for reading
Guide
Code

Reproduction
The problem is file src/routes/sum.tsx
https://stackblitz.com/~/github.com/labs-company/mathematics

@castrogarciajs Firstly, very cool app! Do you want to give v0.0.29 a shot?

For your example, you may need to apply user-drag: none to avoid dragging of the underlying images.

Thank You @sashamilenkovic, It is an application for children to learn arimetic with tens... Of course I will try version v0.29.0

@sashamilenkovic It worked, that's great!!!... You just saved my life, I thank you very much.

@castrogarciajs Thank you for reporting the issue! Let me know if you have any other issues.


Allow adding titles to wiring diagrams

For visualizing a dynamic process (each timestep represented by a wiring diagram), I've found it's helpful to have a title that provides information during that timestep.  E.g.

To do this, I added a Title as a new kind of Graphviz.Statement and added a title::Union{Nothing,String} keyword argument to to_graphviz.

Looks good to me. I like the kwag defaults to nothing so that we can identify explicit empty string vs no string passed.

Thanks Kris, looks good!


Change prop validator

styled-components will be using this - https://github.com/styled-components/styled-components/pull/1937/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R65

Nice! 🎉
Could you please import it directly on this file and remove validAttr.ts?
https://github.com/reakit/reakit/blob/51d8852fb1bdb017efdae88c597b5c473f975c82/packages/reakit/src/_utils/pickHTMLProps.ts#L1-L14

done

Codecov Report

Merging #200 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #200   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          30     29    -1
  Lines         309    304    -5
  Branches       74     73    -1
=====================================
- Hits          309    304    -5

Impacted Files
Coverage Δ

packages/reakit/src/_utils/pickHTMLProps.ts
100% <100%> (ø)
:arrow_up:

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 51d8852...5574b12. Read the comment docs.


File Type column in the File Explorer on Windows shows "Visual Studio Code" rather than the actual file type

VSCode Version: 1.1.1
OS Version: Windows 10 Pro

Steps to Reproduce:

Try to open a file (any extension: xml, html, js, cpp, etc.). Windows prompts for default way to open the file. Choose Visual Studio Code
The file explorer now shows the file type being Visual Studio Code in the Type column.

The problem with this is that it's impossible to know what file is what since the extension of the file is hidden. The only way to know is to open the file or run ls in the terminal.

When registering the files to use Sublime Text by default, the file types are shown in the Types column in the file explorer as shown in the following picture.

This is terrible! And I finally got what the fuss was all about, thanks @CGreenburg!

#3419


player lift feature

Hi Josh,
This is not so much an issue as it is a request. Is it possible to make the player levitate (or lift) so that a better appreciation of the brick cube (or any other floating structure) is possible?

sent a pull request with the above request passed. -Andrew


Missing install dependencies

We are missing some dependencies for our optional components. These can typically be installed manually, but we should include them.
This list will grow as I discover more:

Microsoft.Build

This one is fixed, and since we haven't found any more, let's close this.


Merge pull request #1 from crosswalk-project/master

update from oirgin

this pull request also does not make sense. if you need help with your git workflow, please send an email to the crosswalk-dev mailing list with your questions.


handle missing table in SDSSIO.read_photoobj

Addresses #175.

It may not suffice to just skip the photoobj files. For fields that have timed out, I think some of the other files are missing too. We probably want to use window_flist file to decide what fields to skip.

Do you have some specific reason to believe that such fields would have other missing files, or is this just being cautious?

I tried just skipping the bad photoobj files earlier, but then problems
with other files for those fields appeared. I think we just want to skip
the fields with score=0 in the window_flist file.
On Mon, Mar 28, 2016 at 3:46 PM Kyle Barbary<EMAIL_ADDRESS>wrote:

Do you have some specific reason to believe that such fields would have
other missing files, or is this just being cautious?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/181#issuecomment-202613410

Fair enough. Instead of reading score from the window_flist file, I added the requirement score!=0 to the original query used to get all the field info (in the /project/projectdirs/dasrepo/celeste-sc16/field_extents.fits file). This eliminated a few percent of the fields. I'm not completely positive that the SkyServer table has the same info as window_flist in this respect, but it seems to. The query used to get the field extents file is
select run, camcol, field, ramin, ramax, decmin, decmax
into mydb.field_extents
from dr12.field
where rerun=301 and score != 0.0

which is documented in /project/projectdirs/dasrepo/celeste-sc16/README.
This PR shouldn't strictly be necessary but should probably be merged. The possibility of a missing table seems to be part of the SDSS data model (at least in practice), and this PR allows read_photoobj() to handle that case gracefully.

Sounds good, I'll try this out on regions of the sky that have crashed Celeste before.

If there are still problems, I'll investigate whether the window_flist file contains score=0 entries that are score!=0 in the database.


MB-5006 initial react table filter setup

Description
Creates the initial setup for filtering on the react-table.
Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make client_test

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
this article explains more about the approach used.

Gonna approve the Happo changes since the filter design will change.


[SPARK-41447][CORE] Reduce the number of doMergeApplicationListing invocations

What changes were proposed in this pull request?
When restarting the history server, In history server log,
we can see many INFO FsHIstoryProvider: Finished parsing application_xxx followed by
INFO FsHIstoryProvider: Deleting expired event log for application_xxx.
This is caused by the expired and unlisted log files also be parsed, and then execute checkAndCleanLog to delete parsed info, this means that the parsing is unnecessary.
If there are a large number of expired log files in the log directory, it will affect the speed of replay.
In order to avoid this, we can add a logic to clean up these expired log files before calling doMergeApplicationListing.
Why are the changes needed?
Reduce the number of doMergeApplicationListing invocations when execute checkForLogs, avoid parse unnecessary log files.

Does this PR introduce any user-facing change?
NO

How was this patch tested?
UT

Can one of the admins verify this patch?

cc @HeartSaVioR , too

Could you retrigger the failed tests?

Could you retrigger the failed tests?

Retriggered

@shuyouZZ . I added you to the Apache Spark contributor group and assigned SPARK-41447 to you.
Welcome to the Apache Spark community!

@shuyouZZ . I added you to the Apache Spark contributor group and assigned SPARK-41447 to you. Welcome to the Apache Spark community!

Thanks for your review. @dongjoon-hyun @mridulm @thejdeep


Update config-set() and config-reset() to respect $config-manger-variables

LOL, you deleted my branch, but our app depended on it, so our build broke. :)
We updated to depend on sass-projects/sass-config-manager#develop while waiting for a release.

Ah im so sorry!!!, in still need a better idea for this. The test failing because of this change...

I'll try to look into tests tomorrow.

Uhm, did you revert all the global-local stuff?

I changed all var to global. I make now a new reales. It should work for you and all others.

Uhm... Do you mean that the change back to global is temporary?

No, it now remains so.

But then it won't work for me.

the problem is, that if we change set and reset, no to use global var, it wont work..

But this PR we're commenting on now -- it has set and reset use global conditionally and it does work?

If you check the master or develop branch all test are green, now you need to test if it works for you.

My goal is to not have anything in the global scope -- just like in JS strict mode.

Ah.. but the set and reset wont work without the global hack...

Let me ask again: why did this PR not work for you? E. g.:
      @if ($config-manger-variables == 'global') {
        $config-default: map-remove($config-default, $setting) !global;
      } @else {
        $config-default: map-remove($config-default, $setting);
      }

if i run the test with this.
$config-default: map-remove($config-default, $setting);

im only get errors. Because all vars are empty.

Uhm... Isn't it obvious that the local scenario needs a different test setup than the global one?
I didn't look into tests because you said you got them done, but I can.

Please, do so.

Will you revert to the $config-manger-variables approach if I manage to fix the tests?
Which commit should I work from? (the last commit with $config-manger-variables)

Yes i do :)
From develop.

Looks like ba70cddf15 is the last commit that has $config-manger-variables.
But there are not broken tests. What should I look for?

They only brock if you make changes on set and rest.
@if ($config-manger-variables == 'global') {
        $config-default: map-remove($config-default, $setting) !global;
} @else {
        $config-default: map-remove($config-default, $setting);
}
````


Fix pickling of Artifact.from_type(...) classes

Description
Pickling of instances from Artifact.from_type(...) classes fails because the class name is not actually available in arti.artifacts. This fixes that by using a _DynamicArtifact intermediate class with a custom __reduce__ implementation that triggers the dynamic class to be regenerated and then an object instantiated.
While we could simply add the generated types to globals() after generation, that wouldn't handle:

loading a pickled representation from a separate session if that specific dynamic class hadn't already been generated in this session
classes with duplicate names (they're currently named based on the Type.__name__, but there might be multiple, different instance of the same type)

I think there are some larger changes we can make to Artifact/Producer that will remove the need for these dynamic classes (and thus these hacks), but this will suffice for now.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Tests have been added for simple pickle.{dumps,loads} in the same session, but I tested reloading a pickle file across sessions by hand (this is harder to reliably test for).
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in upstream modules

Codecov Report

Merging #259 (e8c9679) into golden (b4f1c81) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            golden      #259   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           41        41
  Lines         2534      2544   +10
  Branches       523       497   -26
=========================================
+ Hits          2534      2544   +10

Impacted Files
Coverage Δ

src/arti/artifacts/__init__.py
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Superseded by #263.


Adds a ParsingOptions OptionSetType

ParsingOptions is used when parsing JSON that may have null values or missing keys.  ParsingOptions provides .NullBecomesNil and .MissingKeyBecomesNil options to treat this JSON as nil to prevent the throwing of errors.
This PR also adds tests for what Freddy considers JSONSubscripting.
Fixes #128

We should get this in soon. @jgallagher pointed out the other day that this, a breaking API change, means a x.0.0 bump based on strict adherence to semver.

Do we consider adding backwards compatible method definitions, which would instead be a 0.x.0 bump?
As Swift resilience isn't, well, a thing yet, do we not strictly to adhere to semver this early on in Freddy's life?

I would rather adhere to semver correctly. Adding deprecated backwards compatible methods sounds best to me.

Great! Swift gives us both forwarding/@available(*, deprecated) versions and @available(*, unavailable) versions. Either choice we make, I feel that, as long as we are careful with the copypasta, the backwards versions don't need the full docs and testing treatment.
I'll add this to the Milestone; "Next" will tentatively be 2.1.

So, shall we merge?

We need to add the ifNull-parameter versions back for compat, but that can be done separately. I'll file an issue in my morning. :sleeping: otherwise we're good

:zzz: well!


fix: ESM only import issue chalk v5, use v4

fix #224
we ran into this issue before, my bad for forgetting to test a CJS next config

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

Project coverage is 71.70%. Comparing base (377ea12) to head (ff6c9df).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #225   +/-   ##
=======================================
  Coverage   71.70%   71.70%
=======================================
  Files          30       30
  Lines        2410     2410
  Branches      158      158
=======================================
  Hits         1728     1728
  Misses        678      678
  Partials        4        4

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


Support Custom Authorizer

It does not look like current SDK support custom authorizer

https://docs.aws.amazon.com/iot/latest/developerguide/custom-authorizer.html

We need to be able to set custom headers in the websocket connection and not use certificates or AWS credentials. paho mqqt client has added support for this.

We also have this issue. I don't think the paho version has been released yet, it says released in version 1.3.0 in this issue here, but I don't see that version out yet.  https://github.com/eclipse/paho.mqtt.java/issues/502

Hi, has anyone looking into this? Is there some way of "hacking" the headers or otherwise have some sort of support for this in the SDK?
Cheers,
Mario


Adding NAPTR support

Addition of support for the NAPTR RR Type
Clone of SRV support behaviour, adjusting as needed.
Room to improve validation, beyond my immediate abilities as a non-scala developer.

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

Thanks @tpretz! Did a quick glance so far and this is pretty sweet! Leaving a reminder for someone to make an issue to add NAPTR Batch Change Support when this is merged

Thanks @tpretz for the PR.  This is really good stuff :) .

A few comments:

Should add a functional test (add a python test under modules/api/functional_test in the create_recordset_test.py.  I asked @nimaeskandary to help out with that
We need to add cards to our backlog to update the different client libraries for support.  Those should all be pretty small

@tpretz do you use the client libraries at all, or only the web portal?

My initial usecase, that drove this addition, is just via the portal.
(essentially self-service for a mobile team to drive their telco infra zones which use NAPTR heavily)
So, i don't have any other pull requests pending right now.
(although, the GUI modals are a bit cramped with the amount of fields NAPTRs require, didn't want to adjust that in this PR tho as its a bit out of scope for this RR addition)
My only other deviation is to disable the dotted hosts check (i know its generally good practice to avoid these, but with some of these infrastructure zones its not really practical to delegate at every ".")
I can see a proper solution is already raised for this in #521
Being able to opt a zone out of these checks is going to be essential to on-board telco zones, or legacy zones prior to a cleanup.
Anyway, thanks for making this project opensource, it's been very useful and i hope to expand my deployment to other usecases soon.

@tpretz Would you have time to meet up so we can discuss your use case / deployment?  Would love to get feedback to help us drive our backlog

Also, feel free to add yourself to https://github.com/vinyldns/vinyldns/blob/master/AUTHORS.md in this PR if you'd like!

@tpretz made a pr against yours with a couple func tests https://github.com/tpretz/vinyldns/pull/1

made a couple records in the portal as well


Mutations do not perform reference integrity checks

An ID to an entry that does not exist can be set
An ID to an entry of the wrong type can be set
An entry that is referenced by another entry can be deleted

Fixed in 0.10.0


Bring pack the pack docs

Chatting with people in the larger world, a couple of them really wish the website still had the pack info. Even though it's just a display of the raw data from github, folks derived value.
This was https://web.archive.org/web/20151231160821/https://osquery.io/docs/packs/#osx-attacks
We should probably look at adding this back

I am opposed to this because it's a further endorsement that packs are some "official, well maintained source of intel".
Instead I'd propose a page called "Examples" which has a subset of exemplary queries that are useful and showcase osquery.
At this point in time there's also a number of external resources (@zwass has a few great talks) about how to make osquery useful. Linking them would be useful.
We can(and should) improve the educational resources around using osquery. The amount of knowledge required is too much. But further endorsing the existing packs as some blessed resource is not helpful to users in the long run.

it's a further endorsement that packs are some "official, well maintained source of intel".

Might need to wait till we have some kind of resolution on https://github.com/osque then. But I'm pretty sympathetic to needing to document it if we're shipping it

Is the code that created the old docs site (and packs documentation) still around somewhere? I looked and couldn't find it.

We've mostly concluded the packs are so stale as to be 🤷


Add newline after each json report object

Fixes #314
Signed-off-by: Melvin Hillsman<EMAIL_ADDRESS>

/assign @mrhillsman

/approve


Set device in dataloaders

Fixes #651
Goals :soccer:
Fix multi-gpu training notebook.
Implementation Details :construction:

Depends on https://github.com/NVIDIA-Merlin/dataloader/pull/113.
device is set identical to local_rank.
Without dropping the last batch (dataloader_drop_last=True), recsys_trainer.evaluate hangs. Probably need to investigate this because this didn't happen before the list column refactoring in merlin-dataloader (see ticket).
torch.distributed.launch  is replaced with torchrun because the former has been deprecated.

Testing Details :mag:
Manually tested in nvcr.io/nvidia/merlin/merlin-pytorch:23.02 by installing the main branch of all Merlin libraries via pip install . --no-deps. Run 01 notebook first and then run 03 notebook.

@edknv thanks for the quick fix. I have just one comment. In this doc is says if use torchrun  Change your training script to read from the LOCAL_RANK environment variable as demonstrated by the following code snippet:
import os
local_rank = int(os.environ["LOCAL_RANK"])

what do you think? does it make a big difference or not?

@edknv thanks for the quick fix. I have just one comment. In this doc is says if use torchrun Change your training script to read from the LOCAL_RANK environment variable as demonstrated by the following code snippet:
import os
local_rank = int(os.environ["LOCAL_RANK"])

what do you think? does it make a big difference or not?

In our case, it doesn't look like it makes a difference. From what I can tell, torchrun seems to make use of the local rank automatically. I think the doc is saying, if you need to use the local_rank variable in your script, use local_rank = int(os.environ["LOCAL_RANK"]). Our script does not make use of this variable, so I didn't include it.


[BD-46] Improve tree-shaking detectability

According to Bundlephobia Paragon does not support tree-shaking, even though local tests say otherwise. One possible reason is that package.json has no module entry, as can be seen from these two issues (first, second).
By adding module entry we might be able to help services such as Bundlephobia to detect tree-shakability more easily.
JIRA: PAR-746

Thanks for the pull request, @viktorrusakov! I've created BLENDED-1152 to keep track of it in Jira.   More details are on the BD-46 project page.
When this pull request is ready, tag your edX technical lead.

@viktorrusakov 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

:tada: This PR is included in version 19.10.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@adamstankiewicz Great, glad it worked! Also forgot to mention that I've tested the build with frontend-app-learning and adding module to package.json doesn't seem to break anything


Any plans for the iOS and WIN Support?

I think you should use cordova-plugin-geolocation.
This plugin exists only to add support for Google Play Services Location APIs which only available on Android.


Distance Conversions

Currently the internal code converts the distance, given in parameters.in in [Mpc], using a simple geometric angular conversion
conversion_factor = distance*1.0e6_dp*tan(pi_d/(648d3))*parsec_km

Compared to a proper cosmological conversion, the discrepancy is >100pc for z > 0.1
Any solution would necessarily need to either assume a cosmology, or allow for user-specified cosmological parameters. I'm guessing the latter would be preferred, but would entail modifications.
Happy for discussion on the best way forward.

I came across this article "Distance measures in cosmology" that may help in implementing a proper cosmological conversion.
As a note: the conversion happens in multiple places in the code (VisibleComponent.get_M_stars_tot(), Plotter.orbit_plot(), Decomposition.create_orbital_component_files_giu()), perhaps a system parameter would be in order...
dist_ref.pdf


Providing an interface to nixpkgs.config

Your issue may already be reported!
Please search on the issue tracker before creating one.
Would your feature fix an existing issue?

I recently needed to set nixpkgs.config.firefox.enablePlasmaBrowserIntegration = true; and the only way I think that can be done (correct me if I'm wrong) is modifying lib/default.nix, specifically the pkgImport function.
This is also where allowUnfree = true is set (I'm all for sane defaults, but this is very implicit).
Describe the solution you'd like

Maybe collect nixpkgs.config from within the nix` folder? I'm not very familiar with the structure of devos so I can't say many things (again, this might be a non-existent issue and I don't know how it's done)
Describe alternatives you've considered

Additional context

No. I don't think so.
See https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/applications/networking/browsers/firefox/wrapper.nix#L56
for example. There are many useful (and often necessary) options you might want to set.
I for example needed to activation KDE framework integration in firefox, something that nixos modules can't do (a different build is needed).

This is also where allowUnfree = true is set (I'm all for sane defaults, but this is very implicit).

This is a huge issue, indeed. It is being worked on in the context of #232

This is going to be solved in api-next branch which exposes hostDefaults.config or host.config.
In an effort to clean up issues, I'm going to close this


[Listen] Repeated question gets no answers

Describe the bug
In Listen, a question is spoken and answers are displayed. If the user selects Settings or Keyboard and returns to listen, repeating the same question will not display any answers. A different question must be asked.
To Reproduce

Ask a question yes or no question.

the question appears in the output label.
possible answers appear on the screen.

You may or may not answer the question.
Tap the Settings or keyboard button.
Tap the close button.

user is returned to the "Listen" category
the last question still appears in the output label. I think this should be reset.
the answers are no longer displayed.
the "Listening..." message appears.

Ask the same question again.
Expected: The question reappears in the output label. Possible answers appear.
Actual: The question reappears in the output label. No answers appear.

Work Around
Change categories or ask a different question.
Screenshots
Ask me and I'll demo the behavior for you.
Device Information

Device: iPhone 12 Pro Max
OS: 15.0.2
TestFlight build 2781

I reproduced this issue on an iPad.
Device Information

Device: iPad Pro
iOS: 15.3.1
TestFlight build 2781 and develop branch


The property 'Arguments' cannot be found on this object

I had weird spurious errors today on <IP_ADDRESS>.
Exception calling "EndInvoke" with "1" argument(s): "The property 'Arguments' cannot be found on this object. Verify
that the property exists."
At line:11 char:21

                $Data = $job.InnerJob.EndInvoke($job.Handle)

                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : CmdletInvocationException

I run with Set-StrictMode -Version Latest and $ErrorActionPreference = "Stop".
But I had a look through this line of code and I couldn't see how this part would even touch an Arguments property unless it's buried in some other section of code...
I had the issue for a while, and then it disappeared as quickly as it came. I'm raising this issue in case others see it or to track any further occurrences. I haven't been able to force it to happen hence the vagueness.

I can pretty much reproduce this with the following code;
1|Start-RSJob {
    $ErrorActionPreference='stop'
    $_/0
}|Wait-RSJob|Receive-RSJob

Error:
WriteStream : Exception calling "EndInvoke" with "1" argument(s): "Attempted to
divide by zero."
At C:\Users\proxb\Documents\WindowsPowerShell\Modules\PoshRSJob\Public\Receive-RSJob
.ps1:105 char:18
+             $_ | WriteStream
+                  ~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,WriteStream

Looks like something in WriteStream might be causing this when ErrorActionPreference='stop' is used within the scriptblock.

It seems like the error stream from within in the runspace is creating a false positive that an error is happening when EndInvoke() is called.
$RS = [runspaceFactory]::CreateRunspace()
$PS = [powershell]::Create()
$RS.Open()
$PS.Runspace = $RS
$Null = $PS.AddScript({
    $ErrorActionPreference = 'Stop'
    1/0
})
$Handle = $PS.BeginInvoke()
$Results = $PS.EndInvoke($Handle)
$PS.dispose()

Exception calling "EndInvoke" with "1" argument(s): "Attempted to divide by zero."
At line:1 char:1
+ $PS.EndInvoke($Handle)
+ ~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : RuntimeException

I'm not sure of a good way to get around this just yet and more research will be needed.

I don't think that's the same error.
I use a modified version of PoshRSJob with this code in WriteStream:

            Switch ($Stream) {
                "Verbose" { $IndividualJob | Select -ExpandProperty Verbose | Where { $_.Message } | ForEach { Write-Verbose $_ } }
                "Warning" { $IndividualJob | Select -ExpandProperty Warning | Where { $_.Message } | ForEach { Write-Warning $_ } }
                "Error"   { $IndividualJob | Select -ExpandProperty Error | ForEach { $_ } }
                "Output"  { $IndividualJob | Select -ExpandProperty Output }
            }

Here's my output when I run your test:

Exception calling "EndInvoke" with "1" argument(s): "Attempted to divide by zero."
At line:11 char:21

                $Data = $job.InnerJob.EndInvoke($job.Handle)

                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : RuntimeException

That's an Output notification not an Error one. That's what I'd raised on a separate issue though I'm not sure if others would like it as it's not a great idea to pollute the Output stream.
The error recurred again today. I've found it happens if I have a large number of jobs queued (on a 4 core machine with 10 jobs running and a few hundred queued across two PowerShell sessions). Now that I know this I can reliably / randomly generate it plus catch stack traces.

ErrorRecord                 : The property 'Arguments' cannot be found on this object. Verify that the property exists.
StackTrace                  :    at CallSite.Target(Closure , CallSite , Object )
at System.Dynamic.UpdateDelegates.UpdateAndExecute1[T0,TRet](CallSite site, T0 arg0)
at System.Management.Automation.Interpreter.DynamicInstruction`2.Run(InterpretedFrame
frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(Interp
retedFrame frame)
WasThrownFromThrowStatement : False
Message                     : The property 'Arguments' cannot be found on this object. Verify that the property exists.
Data                        : {System.Management.Automation.Interpreter.InterpretedFrameInfo}
InnerException              :
TargetSite                  : Void CheckActionPreference(System.Management.Automation.Language.FunctionContext,
System.Exception)
HelpLink                    :
Source                      : System.Management.Automation
HResult                     : -2146233087

ErrorRecord                 : The property 'Arguments' cannot be found on this object. Verify that the property exists.
StackTrace                  :    at System.Management.Automation.Internal.PipelineProcessor.SynchronousExecuteEnumerate
(Object input, Hashtable errorResults, Boolean enumerate)
at System.Management.Automation.PipelineOps.InvokePipeline(Object input, Boolean
ignoreInput, CommandParameterInternal[][] pipeElements, CommandBaseAst[]
pipeElementAsts, CommandRedirection[][] commandRedirections, FunctionContext funcContext)
at Test-DbReportServer(Closure , FunctionContext )
WasThrownFromThrowStatement : False
Message                     : The property 'Arguments' cannot be found on this object. Verify that the property exists.
Data                        : {System.Management.Automation.Interpreter.InterpretedFrameInfo}
InnerException              : System.Management.Automation.PropertyNotFoundException: The property 'Arguments' cannot
be found on this object. Verify that the property exists.
at
System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext
funcContext, Exception exception)
at
System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame                                frame)                                  at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(Interp                               retedFrame frame)                                  at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(Interp                               retedFrame frame)                                  at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)                                  at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)                                  at System.Management.Automation.PSScriptCmdlet.RunClause(Action1 clause, Object
dollarUnderbar, Object inputToProcess)
at System.Management.Automation.PSScriptCmdlet.DoEndProcessing()
at System.Management.Automation.CommandProcessorBase.Complete()
TargetSite                  : Void EndInvoke()
HelpLink                    :
Source                      : System.Management.Automation
HResult                     : -2146233087

ErrorRecord                 : Exception calling "EndInvoke" with "1" argument(s): "The property 'Arguments' cannot be
found on this object. Verify that the property exists."
StackTrace                  :    at System.Management.Automation.ExceptionHandlingOps.ConvertToMethodInvocationExceptio
n(Exception exception, Type typeToThrow, String methodName, Int32 numArgs, MemberInfo
memberInfo)
at CallSite.Target(Closure , CallSite , Object , Object )
at lambda_method(Closure , Object[] , StrongBox1[] , InterpretedFrame ) WasThrownFromThrowStatement : False Message                     : Exception calling "EndInvoke" with "1" argument(s): "The property 'Arguments' cannot be                                found on this object. Verify that the property exists." Data                        : {} InnerException              : System.Management.Automation.CmdletInvocationException: The property 'Arguments' cannot                                be found on this object. Verify that the property exists. --->                                System.Management.Automation.PropertyNotFoundException: The property 'Arguments' cannot                                be found on this object. Verify that the property exists.                                  at                                System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext                                funcContext, Exception exception)                                  at                                System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame
frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(Interp
retedFrame frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(Interp
retedFrame frame)
at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)
at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)
at System.Management.Automation.PSScriptCmdlet.RunClause(Action`1 clause, Object
dollarUnderbar, Object inputToProcess)
at System.Management.Automation.PSScriptCmdlet.DoEndProcessing()
at System.Management.Automation.CommandProcessorBase.Complete()
--- End of inner exception stack trace ---
at System.Management.Automation.Runspaces.AsyncResult.EndInvoke()
at System.Management.Automation.PowerShell.EndInvoke(IAsyncResult asyncResult)
at CallSite.Target(Closure , CallSite , Object , Object )
TargetSite                  : Void CheckActionPreference(System.Management.Automation.Language.FunctionContext,
System.Exception)
HelpLink                    :
Source                      : System.Management.Automation
HResult                     : -2146233087

MyCommand             :
BoundParameters       : {}
UnboundArguments      : {}
ScriptLineNumber      : 11
OffsetInLine          : 21
HistoryId             : -1
ScriptName            :
Line                  :                     $Data = $job.InnerJob.EndInvoke($job.Handle)
PositionMessage       : At line:11 char:21
+                     $Data = $job.InnerJob.EndInvoke($job.Handle)
+                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
PSScriptRoot          :
PSCommandPath         :
InvocationName        :
PipelineLength        : 0
PipelinePosition      : 0
ExpectingInput        : False
CommandOrigin         : Internal
DisplayScriptPosition :

PSMessageDetails      :
Exception             : System.Management.Automation.MethodInvocationException: Exception calling "EndInvoke" with "1"
argument(s): "The property 'Arguments' cannot be found on this object. Verify that the
property exists." ---> System.Management.Automation.CmdletInvocationException: The property
'Arguments' cannot be found on this object. Verify that the property exists. --->
System.Management.Automation.PropertyNotFoundException: The property 'Arguments' cannot be
found on this object. Verify that the property exists.
at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext
funcContext, Exception exception)
at System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame                          frame)                            at                          System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame                          frame)                            at                          System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame                          frame)                            at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)                            at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)                            at System.Management.Automation.PSScriptCmdlet.RunClause(Action1 clause, Object
dollarUnderbar, Object inputToProcess)
at System.Management.Automation.PSScriptCmdlet.DoEndProcessing()
at System.Management.Automation.CommandProcessorBase.Complete()
--- End of inner exception stack trace ---
at System.Management.Automation.Runspaces.AsyncResult.EndInvoke()
at System.Management.Automation.PowerShell.EndInvoke(IAsyncResult asyncResult)
at CallSite.Target(Closure , CallSite , Object , Object )
--- End of inner exception stack trace ---
at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext
funcContext, Exception exception)
at lambda_method(Closure , Object[] , StrongBox`1[] , InterpretedFrame )
TargetObject          :
CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : CmdletInvocationException
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at , : line 11
PipelineIterationInfo : {}

You know what give me a little bit - I think I found the problem in my own code :-(

Confirmed, this is a non-issue. It turns out what I wrote to write stack traces was occasionally throwing the error. Nevermind.


Do you support the openPower architecture

I want to use the Openshfit Container Storage platform. I found the ocs-operator can easily deploy a storage cluster for me. Do you support the openPower architecture?

'support' is a big statement. The OCS-operator can certainly be compiled and run on Power.

We have no upstream testing on Power at this time. Official support statements will come through official Red Hat channels, in the form of release notes and official Red Hat blogs.


Broken submission

Summary
Unbug the submission
Details and comments

[x] Allow return None in function get_by_url
[x] Update some if
[x] Tests run --> (new repo SSVQE, actual repo)

Closes #518

Thanks! I think you need to change the add_repo_*_results methods to do something like:
repo = self.get_by_url(repo_url)
if repo is None:
    return None

Yes :) for the tests, I'm doing it. Your review too fast I didn't finished yet ;)

This Repository | None:
Doesn't works with 'NoneType' apparently :
  File "/home/runner/work/ecosystem/ecosystem/ecosystem/daos/dao.py", line 120, in DAO
    def get_by_url(self, url: str) -> Repository | None:
TypeError: unsupported operand type(s) for |: 'ABCMeta' and 'NoneType'

@frankharkins ready for your review :)

Thanks! This looks like a good fix.
Q: Should add_repo_style_result and add_repo_coverage_result also return None?

Thanks :)
Yes I forget about them. I fixed them and applied your feedbacks.
By the way, what does IMO means ?


Reveals correct answers after the user submits the post test

For anyone trying to make alerts accessible to users of screen readers, here’s a thing I learned today. When an element with role='alert' appears on screen, most readers will immediately alert the user (without changing their position on the page). Now, there are many ways to add a DOM element to the page, but not all trigger that alert announcement, even if the correct role is set. One notorious example: React's optimized tree modification, which saves work by changing attributes on a node a when a node b is replacing it. Normal use of react: no alert announcement!
In order to force it, I had to render it as display: none and then immediately change it to display: initial
export class AccessibleAlert extends Component {
  state = { style: { display: 'none' }}

  componentDidMount () {
    setInterval(() => this.setState({ style: { display: 'initial' }}), 1)
  }

  render () {
    return (
      <span role="alert" style={this.state.style}>
        {this.props.children}
      </span>
    )
  }


Shouldn't URL of the dataset also be included?

Dear All, Please answer my noob question:
I'm looking for a solution that is between native integrations and a documented API: an easy-to-setup mapping of fields (stored in any accessible database) for our API. I saw gr8 potential in SSSOM but then I realized that it might just contextualize fields but I don't see where the URL of the dataset is specified.
Is this because it isn't ideal for such use case or the recommended usage is storing those info beneath the SSSOM.tsv file? (credentials, OpenAPI GET endpoint or do you know about such a schema?)
Thanks for clarifying!🙏🏽

@entin-hun thank you for reaching out! Before I can help you answering all your questions I need to ask you some things.

I realized that it might just contextualize fields but I don't see where the URL of the dataset is specified.

The URL of the dataset is usually specified with subject_source and object_source. This field is essentially saying: "the subject of the mapping lives in X". Is this what you mean here? I would be great if you could can provide a full example of your use case, ideally in some kind of table format.

Is this because it isn't ideal for such use case or the recommended usage is storing those info beneath the SSSOM.tsv file? (credentials, OpenAPI GET endpoint or do you know about such a schema?)

I need to know a concrete example here - You can always share your API credentials either using a extension, or as part of the "comment" field if needed.
In any case, I think I am misunderstanding the question so lets continue with a concrete example :)

Thank you @matentzn for replying quickly and mindfully! Context of our use-case:
SSSOM or LinkML seems to be an appropriate schema for data providers to guide GET requests which fields to fetch. When someone wants to utilize already existing data (but needs only a subset or 5 fields from the 35), there are still cases when the data provider agrees but can't/doesn't want to send POST requests to the former (e.g. SQL or if it's a manually-updated Excel).
In these cases the utilizers could make the GET requests, based on data providers' field schema. What do you think?

@entin-hun I am very sorry, I think you caught me in a scenario very far outside my comfort zone!

SSSOM or LinkML seems to be an appropriate schema for data providers to guide GET requests which fields to fetch.

This sentence alone is difficult for me to understand as I do not know how you generally "guide" someone to the right fields to fetch.
Unfortunately I dont understand your use case at all. Did you have better luck asking the LinkML folks? Else, maybe I need a more fleshed out concrete example which explains what role the mappings play. Sorry about that! Please keep asking until we nail this down.


parse a list of ServerAddress

Hi,
just an helper to have multiple ServerAddress from a simple String.

Hi, apologies for not coming back to this ticket sooner.  As you may be aware much has changed in the codebase with the new and improved 3.x series.  This ticket can no longer be merged, so its being closed.
Currently the standardized way of dealing with multiple hosts in a string is to use a connection string.  Adding an alternative way is not currently planned, as it simple enough to do with a quick loop.


impl Ranged for TextRange

This adds the missing implementation of Ranged for TextRange itself
impl Ranged for TextRange {
    fn range(&self) -> TextRange {
        *self
    }
}

It also adds .venv to the .gitignore

Current dependencies on/for this PR:

main

PR #20   👈

PR #21

PR #22

This comment was auto-generated by Graphite.

Can you extend the summary with a use case where the implementation is necessary.

The original motivation was from extracting comments with differing nodes. Technically AnyNodeRef would also work there, but i think a function that requires an &impl Ranged should also TextRange in general


Turn groundings and word2vec into classes and move out of EidosSystem

There are now AdjectiveGrounding and OntologyGrounding which are nearly self sufficient, leaving EidosSystem with its more important things to do than manage them.

This passed the tests and also ran through the DocX.txt test documents with the exact same output as master, so I'll go ahead and check it in.  Some of these changes are getting in the way of the next updates.


Websocket ping - (lib/Web.js)

Hi, It's all right?
I think that there are a compatibility problem between botkit 0.6.14 and ws 4.0.0 and newer versions.
Until the version 3.3.3 of ws the signature of ping method was ping (data, mask, failSilently), on the version 4.0.0 the signature was modified to ping (data, mask, cb).
How the botkit version 0.6.14 on the lib/Web.js set a setInterval function that use ws.ping('', false, true) , with the package.json dependency setup as "ws": "^5.1.1", happens a error when this code run because the new signature is ping (data, mask, cb) incompatible with the signature used in setinterval function.
Bellow the related code.
botkit/lib/Web.js 0.6.14
`
botkit/lib/Web.js
    var interval = setInterval(function ping() {
        wss.clients.forEach(function each(ws) {
            if (ws.isAlive === false) {
                return ws.terminate();
            }
            //  if (ws.isAlive === false) return ws.terminate()
            ws.isAlive = false;
            ws.ping('', false, true);
        });
    }, 30000);

`
** ws 3.3.3**
`
ws/lib/WebSocket.js
/**

Send a ping message.

@param {*} data The message to send
@param {Boolean} mask Indicates whether or not to mask data
@param {Boolean} failSilently Indicates whether or not to throw if readyState isn't OPEN
@public
*/
ping (data, mask, failSilently) {
if (this.readyState !== WebSocket.OPEN) {
if (failSilently) return;
throw new Error('not opened');
}

if (typeof data === 'number') data = data.toString();
if (mask === undefined) mask = !this._isServer;
this._sender.ping(data || constants.EMPTY_BUFFER, mask);

}
`
a206e98  on 5 Jan ws 4.0.0
`
ws/lib/WebSocket.js
/**

Send a ping.

@param {*} data The data to send
@param {Boolean} mask Indicates whether or not to mask data
@param {Function} cb Callback which is executed when the ping is sent
@public
*/
ping (data, mask, cb) {
if (typeof data === 'function') {
cb = data;
data = mask = undefined;
} else if (typeof mask === 'function') {
cb = mask;
mask = undefined;
}

if (this.readyState !== WebSocket.OPEN) {
  const err = new Error(
    `WebSocket is not open: readyState ${this.readyState} ` +
      `(${readyStates[this.readyState]})`
  );

  if (cb) return cb(err);
  throw err;
}

if (typeof data === 'number') data = data.toString();
if (mask === undefined) mask = !this._isServer;
this._sender.ping(data || constants.EMPTY_BUFFER, mask, cb);

}
`
Best regards,
Rodrigo Fernandes de Paula

Are you still seeing these troubles?

Hi Peter,
Because of this issue i fixed the botkit version at 0.6.13, the last one that has ws ^3.x.x as dependency, then i don't have problems any more, but i can't update my botkit package.
I checked the ping ws.ping method call on Web.js at the last version (0.6.16) and the method signature is the same ws.ping('', false, true) as so the ws dependency (^5.x.x). I did not do a new test, but i belive that the problem persists.
Best regards,
Rodrigo Fernandes de Paula


Unable to rename collection, throws error

Repro steps:

run db.[existing_Collection_Name].renameCollection("[new_collection_name]");

Action: cosmosDB.executeMongoCommand
Error: exception: connect failed
The output window may contain additional information.
Error running process: The process exited with code 1.
Version: 0.11.0
OS: win32
Product: Visual Studio Code
Product Version: 1.38.1
Language: en

Call Stack
Object.module.exports.t.wrapError extension.bundle.js:537:160963
m extension.bundle.js:537:160421
module.exports.executeScript.o.push._process.onStdErr.e extension.bundle.js:537:159927
u.fire extensionHostProcess.js:50:207
Socket.module.exports.startCore.n._childProc.stderr.on.e extension.bundle.js:622:1043927
Socket.emit events.js:182:13
addChunk _stream_readable.js:283:12
readableAddChunk _stream_readable.js:264:11
Socket.Readable.push _stream_readable.js:219:10
Pipe.onStreamRead [as onread] stream_base_commons.js:94:17

Hi @pspkshah
Sorry, but renaming collections is not supported at this time, but it's possible it could come in the future!

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Added CustomEvent to bom.js

After using a CustomEvent on a project, we found out it was missing from Flow and we had to declare it ourselves every time. I think it could be a good addition to bom.js.
doc : https://developer.mozilla.org/en/docs/Web/API/CustomEvent
Is there any test for bom.js ?
@mroch

thanks @Emile-Filteau
there's not a test specifically for bom.js

@facebook-github-bot import

Thanks for importing. If you are an FB employee go to Phabricator to review.

hey, sorry to do this but I noticed that https://github.com/facebook/flow/pull/1873 added this class first but we neglected to merge it for a while.
I'd also like to add the CustomEvent constructor since that's apparently the non-deprecated way to do things now. so, we'll merge some combination of this PR and the other one plus some related fixes. thanks!


Avoid providing a warning if the requested number of threads is the current number of threads

It seems that this thread number parameter is always defined.
When starting multiple contexts, it is seem hard to not specify this.
Therefore, you end up with many warnings.
Thank you for considering

@hmaarrfk This thread initialization should be done only once, and from the code, it seems OK if the return code is NULL (meaning thread pool initialization failed). So In what case would this happen?

I can see a few reasons:

In live applications (i.e. where you can't spawn a new process and start a totally new global context), you may want to tune the number of threads as the desired resolution changes. This can be when a user is changing the resolution of the output video due to encoding a totally different stream, or when changing the resolution of the broadcasting resolution in a streaming sense.
In applications where you are writing multiple videos to disk, you may also want to tune the number of threads to a particular need. Do you have many other steps in your pipeline happening in parallel? Choose "1" thread? If you have fewer expected background processing, you can increase this to "4".

I think FFMPEG will always set this:
https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/libopenh264enc.c#L176
So even if you serially encode, multiple different 1080 p streams in a single application (using libavcodec directly in your program instead of the ffmpeg executable), you will start to see this popup many times.

I was somewhat surprised that the threadpool is global to all different encoding contexts, instead of being local to each encoding context.
I would like for example, to start 5 encoding contexts, and limit each to 2 threads. I couldn't find a good way to do this.

Thank you@!


DFN (DAE form) sensitivities IC convergence

PyBaMM version - 21.12:
Python version - 3.8.10:

Describe the bug
Perhaps already known. Convergence issues with the DFN (DAE form) when sensitivities are included. Resolved by switching to differential surface form so likely related to initial conditions. Can also be resolved by passing "calc_ic" = false to IDAs so perhaps this could be the default behavior when sensitivities are included. In the example below, I choose to have the negative electrode diffusivity as an input parameter as the initial conditions should not depend upon this parameter so in principle should not interfere with initialization but may be used in the initilization by IDAs under the hood.
Error is:
The linesearch algorithm failed with too small a step
(which is probably not helpful...)
To Reproduce
import pybamm
import numpy as np

chemistry = pybamm.parameter_sets.Marquis2019
parameter_values = pybamm.ParameterValues(chemistry=chemistry)
parameter_values["Negative electrode diffusivity [m2.s-1]"] = "[input]"

options = {}
# options = {"surface form": "differential"}  # this works
model = pybamm.lithium_ion.DFN(options=options)

solver = pybamm.CasadiSolver()
# solver = pybamm.CasadiSolver(mode="safe", extra_options_setup={"calc_ic": False}) # this works
sim = pybamm.Simulation(model, parameter_values=parameter_values, solver=solver)

inputs = {
    "Negative electrode diffusivity [m2.s-1]": 1e-12,
}

t_eval = np.linspace(0, 30, 1000)
# sim.solve(t_eval=t_eval, inputs=inputs)  # this works for DAE case
sim.solve(t_eval=t_eval, inputs=inputs, calculate_sensitivities=True)

Expected behaviour
Default config should allow for IDAs solver to run for the DFN model with sensitivities.

@Scottmar93 This is a few versions old, can we close this?


Meteor gridfs

Hello, awesome project !!
I am using apollo with meteor and I have successfully integrated your package with apollo and meteor.
Can you please give an example on how to use the internal meteor gridfs to save the received file?
Thank you

Hi 👋 and thanks 😊
I don't use Meteor myself, but @joncursi might be able to share some pointers. Specific examples for the new streaming API might be more tricky to come by, since not everyone has upgraded yet.

Ah, unfortunately I am not using GridFS in my Meteor implementation so unfortunately I am unable to comment. Personally, I am using the AWS SDK to upload the files to S3.

Thank you anyway guys, I will try to figure out.

Closing this question due to inactivity. A good place to ask general usage questions is the Apollo Slack #uploads channel 🙂


README points to incorrect hub.getdbt page

Description & motivation
The README points to the Segment hub page currently
Checklist

[x] I have verified that these changes work locally
[x] I have updated the README.md (if applicable)
[x] I have added tests & descriptions to my models (and macros if applicable)

Thanks for this!


Error message: Google Play Music is currently available in select territories.

OS: Win 10
Issue Descriptions: When opening the app, but all I get is a blank page with the message: "Google Play Music is currently available in select territories." I also see that I'm properly logged on at top-right. It worked fine for a few days, but now, nothing. I'm not on a VPN, and I live in the USA.
Steps to Reproduce: Open the app.

Did you tried to use GPM in your web browser?

Whatever this is it will be an issue on Google's side not on ours.
You can try wiping the cache and starting again by deleting the %APPDATA%/Google Play Music Desktop Player folder

It works fine in the browser. I'll try wiping the cache. thanks.

Wiping the cache did nothing. Looks like I sent you my $10 prematurely. Live and learn. And thanks for closing the thread on me. Very helpful.


Documentation for datasette.client

I'm going to document this in a separate issue.

Originally posted by @simonw in https://github.com/simonw/datasette/pull/1000#issuecomment-706269271

This can become a section on https://docs.datasette.io/en/stable/internals.html

https://docs.datasette.io/en/latest/internals.html#client


[FEATURE]:Documentation update for nfsv4 mount option

Adding the Comment for nfsv4 mount option, in Isilon cluster NFSv4 should be set.
To mount a volume with NFSv4, specify mount option vers=4. Make sure NFSv4 is enabled on the Isilon Cluster

@Tharun-vr is this addressed now ? you need any support here ?

updated doc in dell/csm-docs#112


BOM resolution failure when the plugins block is used for a locally published plugin

It appears using enforcedPlatform (or platform) for a Gradle plugin which is available only in the local repo breaks the dependency resolution. Here is a minimal reproducer and instructions
https://github.com/aloubyansky/playground/tree/gradle-plugin-bom
Thanks!

Thanks for the report and the reproducer.

The issue was caused by the metadata being loaded without the Java derivation strategy being applied and the in-memory processed metadata cache not taking into account the derivation strategy in its cache key
The first part has been fixed as part of this issue, the second part will be fixed in #9415 as its occurrence in practice is near impossible.


VIM segment fault

vim thrown a segment fault !!!

vim version is
VIM - Vi IMproved 7.2 (2008 Aug 9, compiled Apr  5 2012 10:17:30)
The core file you could find in
core.17392.zip
I don't know why it's going wrong and it never happened before.
May this could help community.

I see that you're using vim-7.2 from 2008.
Why do you use such an old version?
I recommend downloading or compiling a more recent version of Vim.
Many bugs were fixed over the last few years.
The latest version in git is vim-8.0.95.
Attaching a core file without its corresponding executable is also useless, but there is no need to add it, as I don't think that anybody will debug such an old version of vim. The bug was likely already fixed. If it still happens with a more recent version of vim, you need to describe the steps to reproduce it.

closing. If you can verify with a more recent version, feel free to reopen

@dpelle The reason I use such old vim is the only choice in working environment. Thanks for your kind explanation anyway.


added Etc2 tool

This makes possible to use KTXProcessor for generating Etc2 textures with a help of etc2comp

Does this work with Mac OS X?

Nope, somebody needs to build a binary

Where does the binary come from? What's the license?
On Mar 18, 2017 7:47 PM, "John"<EMAIL_ADDRESS>wrote:

Nope, somebody needs to build a binary
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/libgdx/libgdx/pull/4657#issuecomment-287566364, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfYBAXPbOpum0JhKZABkAUAh6Z2UdL-ks5rnCaogaJpZM4MhKfm
.

https://github.com/google/etc2comp   Apache License 2.0

Updated with Mac OS binary 9f7624d, should now work probably

macOS executables don't have extension .app, in fact, .app is something completely different, which makes this naming very confusing. macOS executables don't have any extension, I suggest renaming it to "etctool-mac" or something similar.
Same could be done with Linux.

Done 004e348.

not a good idea to include binaries in the repo, users can build them and stay up to date.


"replace deno?" EOF or read error

I just tried updating from 0.35 to 0.38 using the script. This is what happens in the terminal:
$ which deno
/home/daniel/.local/bin/deno

$ deno --version
deno 0.35.0
v8 8.1.310
typescript 3.8.2

$ curl -fsSL https://deno.land/x/install/install.sh | sh
######################################################################## 100.0%
Archive:  /home/daniel/.local/bin/deno.zip
replace deno? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
(EOF or read error, treating as "[N]one" ...)

$ uname -a
Linux hostname 4.4.0-176-generic #206-Ubuntu SMP Fri Feb 28 05:02:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

That EOF or read error happens immediately; it doesn't seem like there's a chance for me to do anything.
Solved by removing the existing deno binary first.

Got that too

Thanks for the notice. This must be from changes made in b635b52.
I think that using "-o" with unzip will fix this https://github.com/denoland/deno_install/pull/101

cc @bartlomieju I think #101 fixed this so this issue can be closed.