Abuse vector: My chat and KBFS can be initiated by any user

I searched Keybase.io for a contact page, and then I raised this topic to @malgorithms who suggested I take my questions to GitHub, so I'm starting this thread.
I'm concerned about the open nature (and no option to close) access to both my chat and KBFS. There is no request and approve/deny flow.
Problem
Right now any Keybase user can create a shared folder with me, without asking first. This will send me a notification, and if I navigate to that folder on my machine, the files will automatically download.
(Update: As noted in a comment below simply lsing the directory won't download all the files in the folder. It would require an actual read of the file to do so.)
According to the documentation:

“Filesystem” means that there is no sync model -- files stream in and out on demand.

This is probably a fine process for people I know and trust. But the way Keybase is currently designed, any user can initiate this. This is a problem if a user wants to be malicious. What happens if a user wishes to fill a folder with malicious or abusive or illegal files?
Best case scenario: I resist the temptation to navigate to that directory on my machine. But there's now a folder with both my username and my access rights full of these hypothetical malicious or abusive or illegal files. I can use the UI to ignore the folder only as a reactionary action. I'm not convinced that's an adequate solution under a potential abusive situation, especially if there are multiple abusers (this is a problem Twitter has spent years trying to address, and is still working on).
Worst case scenario: I navigate to the folder and now these hypothetical malicious or abusive or illegal files are downloaded to my machine. I can then use the UI to ignore the folder, but as mentioned above, this solution seems problematic.
A similar issue arrises with chat: Anyone can initiate a chat with me. They can be abusive, spammy, or even send me illegal/offensive content. My only option is to ignore the user after the fact.
Furthermore, I'm concerned about Keybase's ability to help in an abusive situation. Since everything is encrypted end-to-end, how would Keybase step in?
Solution?
This is likely not a trivial solution to implement technically, but I think it's worth considering a "Friend" model. Keybase already has the concept of "following." As a user, I would like to be able to have a request and allow/deny flow when someone wishes to "follow" me. Or, perhaps a situation where we must be mutual followers in order to do certain types of communication.
Something like that would create an explicit social network of trust. Eg: "Since we both follow each other, it's reasonable to say we agree to receive files or message from each another."
Since my Keybase profile is public, (unlike email or other social networks) and since end-to-end encryption would prevent Keybase from implementing content filters for spam or abuse (again, unlike email or other social networks) it seems like giving users at least the option to gate access to their chats and files would be positive if not necessary.

if I navigate to that folder on my machine, the files will automatically download

This isn't true.  As you quoted above, files stream in and out on demand -- navigating to the folder isn't enough to download all the files in the folder.  For the most part, you actually have to actually instruct your machine to start reading the files to fully download them.
(There is some limited "pre-fetching" done of a folder that you ls, but it only reads at most one block of data from each entry in that directory, so it won't recurse into subdirs or download big files.  And even if so, the downloaded data will remain encrypted on the disk.)
I'm not dismissing your concerns, just clarifying a technical point.

@strib Thanks for that important clarification. I had assumed lsing would pull down the files.

KBFS issue: When it boils down to it, when mra creates a private folder with mrb without mrb's permission, this is what happens:

mra generates a key for the encryption of the folder contents.
mra encrypts the key using mrb's public key
mra uploads the key encrypted with mrb's public key
If mrb tries to look at the folder, he decrypts the key and downloads the folder directory list (and apparently one block of each file just to make sure it's not a folder etc.)
If mrb tries to read a file, it will be downloaded to disk as an encrypted file.

Where should Keybase intervene, without completely destroying the usefulness of keybase as a whole?
Someone could email you illegal files encrypted with your pgp public key, and your mail client would download the encrypted file, so you are in more trouble than with Keybase.
Summary: KBFS issue is a non-issue in my opinion.
Chat issue: Similarly to email, anyone can send you email... the way email has compromised with it is they have "spam filters" that try to guess whether mail is spam or not... This wouldn't work on keybase.
However, there is a problem with my analogies.
The only difference in the KBFS / Chat to email comparison, is that Email (SMTP) is an open protocol, so many people over the years have developed tools that stand between your inbox client and your mailserver that try to filter spam / delete attachments / restrict incoming mail to specific domains, etc.
Keybase, however, is still very new, and all the hosting is done by Keybase, so the only way to implement a "roll-your-own" solution would be if Keybase allowed self-hosting... which they don't... and they probably won't implement filters and things like that when their entire chat feature could change completely tomorrow (still very beta) requiring a full rewrite of the filter logic as well.
I'd say this is a wishlist feature, but not a necessity.
No one will arrest you because some bad guy sent a million emails with illegal files on it to your public email address.
They will arrest you if you keep them.

Where should Keybase intervene, without completely destroying the usefulness of keybase as a whole?

Someone could email you illegal files encrypted with your pgp public key, and your mail client would download the encrypted file, so you are in more trouble than with Keybase

As mentioned in my original comment, I think Keybase should implement controls on the receiving-user's side to either create a layer of approval.
You're right that anyone can email you anything, and you can't stop it. The difference here is that Gmail doesn't have a searchable directory of everyone's email address that is also tied to their Twitter/HN/Reddit/Whatever identifications. I think people would be irate if Google did that, but that's exactly what Keybase is.
I agree, for Keybase to remain useful, public proofs of identity are important. But defaulting all forms of communication to "accept" seems problematic when coupled with that public directory.
Keybase is already doing this at the team level:

If you know of a team, you can ask for access. Actually, I just made a team called keybasefriends so if you want to talk to other testers, ask for access:

 keybase team request-access keybasefriends

As an outsider, you can't tell who's on a team, so Keybase will ping the admins for you. They can then add you or ignore the request.

That ability to ignore a request seems like it'd be valuable at the individual level.

please do this before the platform becomes entirely unusable.

I have recently started getting keybase chat spam, 3 spam messages in the last few days, from accounts flyinger, kareengreen and maizeplomino.  One thing I noticed is that these 3 accounts only have associated stellar / BTC accounts, nothing else linked.

This tweet and its replies illustrate that the lack of controls is a big problem for many women users. https://twitter.com/aredridel/status/1201338814859501569?s=21. Speaking as an employer, I can’t expose women employees to this kind of harassment.

Another spam message from another account.  I've disabled notifications and emails for chat, which is a shame because I do use it with a couple people.
Perhaps the motive is to create fake accounts to receive the airdrop, and have activity that makes them harder to distinguish from real people?

It looks like there are new privacy controls available in the client:


Play plugin: script generated by dist does not accept command line arguments

It seems to me that the command, gradle dist , does not produce the same output as ./sbt dist . The archive that the gradle produces contains a shell script that does not accept any command line arguments. Where as, the same works with sbt.
Expected Behavior

the dist command produces an archive, with a shell script that should accept command line arguments like,
$ my-first-app-1.0/bin/my-first-app -Dconfig.file=/full/path/to/conf/application-prod.conf
more information can be found here (https://www.playframework.com/documentation/2.6.x/Deploying#using-the-dist-task)
Current Behavior

When the above command is executed i get the following error,
Bad root server path: /home/ec2-user/test/play-scala-starter-example/build/distributions/-Dplay.http.secret.key=abcccc
Context
I am trying to deploy an application to production, once the distribution package has been created.
Steps to Reproduce (for bugs)

git clone https://github.com/playframework/play-scala-starter-example.git
cd play-scala-starter-example/
./gradlew dist
cd buid/distributions
unzip playBinary.zip
playBinary/bin/playBinary  -Dplay.http.secret.key='abcccc'
Your Environment

Running a centos machine on aws, but the issue also persists on my local ubuntu vm.

If you lookup the startup script, there would be an environment variable like <APP_OPTS>, set it to the command line arguments that you want to send and start the application.

Any follow ups on this issue?

So I have found a workaround:
Right now the shell script that is generated from gradle dist doesn't take in arguments. Thus, any arguments must be put in $root/conf/application.conf. Something like this:
# This is the main configuration file for the application.
# https://www.playframework.com/documentation/latest/ConfigFile
play.http.secret.key="test"

Then, just proceed to run the script without any arguments:
./$root/build/distributions/playBinary/bin/playBinary (your path may be different)

Or you can write your own version of playBinary and copy it to bin folder in build.gradle:
model {
  ...
  distributions {
    playBinary {
      contents {
        from("yourScriptName") {
          into "bin"
        }
      }
    }
  }
}

Then, instead of running bin/playBinary, just run bin/yourScriptName and feed it arguments as usual.

The plugins are superseded by https://gradle.github.io/playframework/ and will be removed in one of the next major releases.


Task Retry (psq.Retry) functionality break Redis/Celery setup

The retry task functionality only works for the PSQ task manager, not for the Celery task manager.
https://github.com/google/turbinia/blob/master/turbinia/task_manager.py#L101

Fix in turbinia-ng -> https://github.com/google/turbinia/commit/7e7ae83a154fb5b7bf4fd4145ef907967a9587cc#diff-da83cfaca362f241b9f852f9177e920ff33872da6385bceec7860fb00bf3945e
Decide on merging later.

Fixed
https://github.com/google/turbinia/commit/a9b1e7d29e3448b244ebc3ad7b3e949fc759ea45


CLONE_TARGET not used for non-persistent images in shared and qcow2 TM_MADs

Description
Following the oned.conf documentation there are three modes described as follow:

clone_target : determines how the non persistent images will be cloned when a new VM is instantiated.
NONE: The image will be linked and no more storage capacity will be used
SELF: The image will be cloned in the Images datastore
SYSTEM: The image will be cloned in the System datastore

To Reproduce
There are different and more complex use cases but I think the following is the simplest one

Set CLONE_TARGET=SELF in TM_MAD_CONF for "shared"
Restart opennebula
Register an image in the IMAGE datastore as non-persistent with TM_MAD=shared and attach it to a VM with SYSTEM datastore TM_MAD=shared
Check the SYSTEM datastore placeholder for the VM - There is a copy of the file (similar behavior for qcow2 TM_MAD - the qcow snapshot is copied to the disk.X.snap/0 in the SYSTEM datastore)

Expected behavior
A symlink  to an image clone for the given datastore created in the IMAGE datastore should be created in the SYSTEM datastore.
Details

Affected Component: [Storage]
Hypervisor: [KVM]
Version: [5.6.x]

Additional context
Looks like that the behavior for the persistent images(LN_TARGET) is hard-coded too always do symlink (and always do a copy if SYSTEM TM_MAD=ssh)
The description of "Shared & Qcow2 Transfer Modes" in Open Cloud Storage Setup is also not clear when a symlink is created and when a file copy for bot persistent and non-persistent images.
I am not sure is a bug or expected behavior that need clarification.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

bump
assuming it is not processed because there are no labels yet

Hi Anton
Yes CLONE_TARGET and similar parameters are not configuration parameters. This is, they will not modify the behavior of the driver in anyway. These parameters are used to let know oned how an specific driver perform a given operation.
For example if CLONE_TARGET is self, oned knows that when that driver is used; it needs to take out space from the image datastore and not the system datastore; and so update datastore quotas and usage accordingly.
Now, we have the hybrid modes, in this case we have CLONE_TARGET_SSH for example. This means that when an image datastore is used with a system datastore of type SSH instead of using CLONE_TARGET you need to use the CLONE_TARGET_SSH, because in this case CLONE operations are going to take storage space from the system datastore and not the image datastore.
Hope is not more clear
Cheers!

Still not sure if this is a bug or feature or we can close it with the previous notes... I'll leave open (now we have a super-duper stale bot for the catch ;)

Hi Ruben,
So if the CLONE_* and LN_* variables are for accounting purposes (and for whitelisting the mixed modes) then there is some sort of inconsistency in the behaviour with the comment in oned.conf:
#   ln_target : determines how the persistent images will be cloned when
#               a new VM is instantiated.
#       NONE: The image will be linked and no more storage capacity will be used
#       SELF: The image will be cloned in the Images datastore
#       SYSTEM: The image will be cloned in the System datastore
#   clone_target : determines how the non persistent images will be
#                  cloned when a new VM is instantiated.
#       NONE: The image will be linked and no more storage capacity will be used
#       SELF: The image will be cloned in the Images datastore
# SYSTEM: The image will be cloned in the System datastore

So the question is how to define the following case?
A VM with SYSTEM datastore TM_MAD=ssh, persistent root disk on IMAGE datastore TM_MAD=shared and volatile disk. The HV has local SSD disk. So when deploying a VM I'd expect to have root disk on NFS and swap on local SSD. So if the host crashes, I'd expect to reschedule the VM on anoteher host and the VM will continue from where the initial host crashed.
I think it is not clear that when the VM is deployed the persistent root disk is not so persistent anymore because it is copied to the host's filesystem.
IMHO clone has simillar flaw - the image is again copied to the SYSTEM DS. so if we have IMAGE and SYSTEM datastores of type shared mounted on NFS it will copy the image from one datastore to the other, taking space in the SYSTEM datastore, despite the CLONE_TARGET definition in the Datastore Template...
In the same time for qcow2 and ceph TM_MADs  that are also ssh backed ones(I mean that they are on a local filesystem on the host) the shared TM_MAD behave differentlty and at least the persistent images are with symlinks because of the hard-coded test against TM_MAD=ssh.

IMHO the admin should expect the TM_MAD to behave following the description - do nothing, clone/sumlink, copy...

I don't see any inconsistency, the parameters determines how the driver perform the operations. These parameters, let say that way are for driver developers, so they can express how the driver works.  May be we can change the sentence to:
" ln_target : determines how the driver clones persistent images when  a new VM is instantiated."
May it is clearer that way.

A VM with SYSTEM datastore TM_MAD=ssh, persistent root disk on IMAGE datastore TM_MAD=shared and volatile disk.

It would be deployed in the hypervisor local storage (that's the main use case to improve I/O performance). Note that you will have the persistent semantics of SSH. A persistent disk will be copied back from system datastore to image datastore. So no matter the transfer mode, a persistent disk will be always persistent, although the specific semantics may change for different drivers (e.g. when the changes are committed to the original image).

A VM with SYSTEM datastore TM_MAD=ssh, persistent root disk on IMAGE datastore TM_MAD=shared and volatile disk.

It would be deployed in the hypervisor local storage (that's the main use case to improve I/O performance). Note that you will have the persistent semantics of SSH. A persistent disk will be copied back from system datastore to image datastore. So no matter the transfer mode, a persistent disk will be always persistent, although the specific semantics may change for different drivers (e.g. when the changes are committed to the original image).

I beg you pardon but there is no "would" in this line :)
My point is that with the current implementation there is no option that alolow the image to stay on shared storage. And it is only with SYSTEM TM_MAD=ssh.
So in the case of a Host failure the VM will lost all data when the VM is recheduled on another host. Because the image will be copied from the "shared" datastore reverting the VM at the state when the image was last saved(last undeploy). IMO the current implementation does not allow the admin to have true persistent image on the shared storage - yes, it is slower than local ssd, but the data is persistent and actual. Even on Host disk failures.
Currently the description of the ln_target in this case is "ln_target: determinaes how the space is accounted by opennebula"
I think that the code should be refactored at least to do symlink by default and do copy on admin/user consent - just like with the ceph tm-mad
does. But the trigger should be other because in both cases the type is file.
Sligthtly different but similar case is with the non-persistent images...
In general, I as an admin would like the driver to follow the CLONE_TARGET as it is described in the oned configuration. When self do a copy on the shared storage and do symlink in SYSTEM ds, when system do copy the image to SYSTEM ds. When none  in the case of shared it should rise error "unsupported configuration". Similar with LN_TARGET for the persistent images - when I set NONE - do symlink, when self do rise an error "unsupported by shared tm_mad"  and only when system  do copy the image to the system datastore.
Hope this clear my concern on the subject :)
Cheers,
Anton

I understand, but the parameter is the other way around it does not change the driver behavior but states how the driver works.
When using a shared storage combined with the ssh drivers the ssh drivers will be used. And the ssh driver works by copying the files to the remote storage. The ssh driver doesn't assume a shared storage and that must remain in the same way.
If you need other behavior then we need a different driver set a fs_ssh for example (like fs_lvm that works in a similar way as you described, i.e. assuming a shared storage).

So it is not a bug, it's a feature :)
Closing the ticket.

:)


Feature: swipe to modify category name

Is your feature request related to a problem? Please describe.
I can see that when users have some categories. They can swipe and choose to delete the category. What if user wants to modify the name of the category.
Describe the solution you'd like
I want to implement swipe action for the category, that also shows Edit Button next to existing Delete Button right now to modify the name of category.

Hi @mikaelacaron, I would like to work on this new feature. What do you think?

Hi @kiennguyenchi this exists!
Press and hold on the row and a context menu pops up for editing

Should we change it to swipe action? Because holding the row like that seems not interactive to the user.

At the moment I don't want to change it. I originally designed it after the Apple Notes app, and that's the interaction that they use for this action

Okay I see, I will propose another feature.


Remember closed state of modal.

Uses localstorage to remember modal state.
I also tried to leverage localstorage to remember the coordinates of the user's last location, but I feel this behavior is probably better executed with session storage. I was also experiencing some funny behavior when opening the app in two tabs simultaneously.
There might be some argument to home-roll the storage extensions instead of use a 3rd party library, but for now this keeps the options open for leveraging localstorage for other attributes with an easy hook. Might be handy in the future.

closes #45

Oh, additional note: I named the key ..._v1 to make it easy to 'cache-bust' the modal state should we need users to see some new information for the welcome modal.


blacked out when launching

I just loaded this player yesterday and all seemed well. I went to launch it and it's giving me the boarder but no content. Its just blacked out. I tried to uninstall/reinstall, but the issue is still going on. Any ideas?

Delete your %APPDATA%\Google Play Music Desktop Player folder and reinstall.
This is a Chromium bug that is being fixed upstream


Group Ticket

Week 5 - Group Ticket
Link to Individual Tickets

Shraddha Kadaba
Madhumita Narayan
Pranavi Inukurti
Meena Annamalai

Links to Group Work

Website Design
WIS Diagram

2.7+/3 for everyone

comprehensive wireframe about website
api is mentioned
frontend example
fibonacci completed

Team (several spoke) - Del Norte Marketplace sell belongings, sell at school.   Sell and Get tutorials services.  Brain write and wires shared.  Workflow diagram very nice as this will help balance work!!!   Very nice introduction!
Madhumita - JavaScript hacks.  Created a list object in conjunction with their project team.   Built HTML rows.
Meena - Built a search screen.  Fibo talked about all the key definitions.
2.9 for everyone

Very nice looking wireframe, workflow diagram was well organied
Team goals and task were documented well
good idea for pbl project
working off of base code, incorporated into JS/html hacks
fibonacci uses extends well, would've been nice to the extend in a spereate individual jupyter notebooks

2.7+/3.0

Great description of their idea
They also included nice wireframes of their future page
Included a search bar for their page
Fibo was extended and it works

2.8/3
Website for a student marketplace. Very detailed WIS diagram. Nice layouts of future style.
Javascript work relates to website idea, creates objects that represent listings.
Brief frontend work through google search.
Completed fibonacci with extends and abstract.

2.7+

Project idea: DN Market Place: Sell things, tutoring page (divided into subjects), UML
JavaScript code within Jupyter Notebook
Fibo with abstract class


Fix testing.BackendConfig context for repeated use

Fix testing.BackendConfig so that it can be used as a context manager multiple times.
Previously, the following code caused an error:
a = chainer.testing.BackendConfig({'use_cuda': True})
with a:
    with a:
        pass

...
  File "cupy/cuda/device.pyx", line 113, in cupy.cuda.device.Device.__exit__
IndexError: pop from empty list

Would you mind recording where this is needed?

Wrote some minor comments, but it LGTM otherwise.

Would you mind recording where this is needed?

Sorry, I forgot where I noticed the bug.

Thanks, LGTM!

jenkins, test this please.

Jenkins CI test (for commit f4ea6be61c59007413c092193cde8d77877318d4, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit f4ea6be61c59007413c092193cde8d77877318d4, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit f4ea6be61c59007413c092193cde8d77877318d4, target branch master) succeeded!

LGTM!


feat(plugin-chart-choropleth-map): smooth zooming

Describe the solution you'd like
Transition when zoom in/out

The codebase on this repo has been moved to the main Apache Superset repo, and consequently the repo is in the process of being archived. See the Superset Improvement Proposal for details: https://github.com/apache/superset/issues/13013 . While all currently open issues and PRs will be closed, we encourage you to reopen this issue on the main repo if it is still relevant.


[chip-test, hmac] chip_sw_hmac_idle

Test point name
chip_sw_hmac_idle
Host side component
Rust?
Opentitantool infrastructure implemented
Yes
Silicon Validation (SiVal)
Yes
Emulation targets

[ ] None
[ ] CW310
[ ] Hyperdebug + CW310

Contact person
Checklist
Please fill out this checklist as items are completed. Link to PRs and issues as appropriate.

[ ] Check if existing test covers most or all of this testpoint (if so, either extend said test to cover all points, or skip the next 3 checkboxes)
[ ] Device-side (C) component developed
[ ] Bazel build rules developed
[ ] Host-side component developed
[ ] Test added to dvsim nightly regression (and passing at time of checking)
[ ] For SiVal test cases, test is running relevant FPGA or silicon regression

Fixed in https://github.com/lowRISC/opentitan/pull/22043.


Not launching with NPM bin for purs executable

We had some projects that are using different purescript compiler versions, and hence I decided to add the purescript version that we use as an NPM dev-dependency so that everyone will have the same compiler version.
However, I'm having some trouble trying to launch multiple projects at the same time (in different NVIM instances of course). I looked into the configuration, and added the purescript.addNpmPath key to true in the config JSON, but still using npm-bin was coming out as false.
let config =
    \ { 'purescript.autoStartPscIde': v:true
    \ , 'purescript.pscIdePort': v:null
    \ , 'purescript.autocompleteAddImport': v:true
    \ , 'purescript.pursExe': 'purs'
    \ , 'purescript.trace.server': 'verbose'
    \ , 'purescript.addNpmPath': v:true
    \ }

This is the output I'm getting from the language server client. Can you explain what am I doing wrong here?
[LC] [Log] Starting with args: [ "/Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/node", "/Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/purescript-language-server", "--stdio", "--config", "{\"purescript.pscIdePort\": null, \"purescript.addNpmPath\": true, \"purescript.autocompleteAddImport\": true, \"purescript.trace.server\": \"verbose\", \"purescript.autoStartPscIde\": true, \"purescript.pursExe\": \"purs\"}" ]
[LC] [Log] Starting with cwd: /Users/sriharshachilakapati/Projects/*****-*******-**** and using root path: /Users/sriharshachilakapati/Projects/*****-*******-****
[LC] [Info] Resolved IDE server paths (npm-bin: false) from PATH of /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin:/Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin:/Users/sriharshachilakapati/.jenv/shims:/Users/sriharshachilakapati/bin:/usr/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin:/Users/sriharshachilakapati/.jenv/shims:/Users/sriharshachilakapati/bin:/Users/sriharshachilakapati/Library/Android/sdk/build-tools/:/Users/sriharshachilakapati/Library/Android/sdk/platform-tools/:/Users/sriharshachilakapati/Library/Android/sdk/tools/:/Users/sriharshachilakapati/Library/Android/sdk/build-tools/:/Users/sriharshachilakapati/Library/Android/sdk/platform-tools/:/Users/sriharshachilakapati/Library/Android/sdk/tools/ (1st is used):
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/purs: 0.12.3
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin/purs: 0.11.7
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin/purs: 0.11.7
[LC] [Warning] Found multiple IDE server executables; using /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/purs
[LC] [Info] Found existing port from file: 15275
[LC] [Info] Starting IDE server on port 15965 with cwd
[LC] [Log] Started IDE server (port 15965)
[LC] [Info] [Error] Version mismatch for the externs at: /Users/sriharshachilakapati/Projects/*****-*******-****/output/Control.Monad.Except.Trans/externs.js Expected: 0.12.3 Found: 0.12.2

This is because of incorrect understanding of VS-Code plugin architecture. I didn't know that VSCode will strip the section name and give only the rest.
From NVIM, I have to pass this configuration instead:
let config =
    \ { 'autoStartPscIde': v:true
    \ , 'pscIdePort': v:null
    \ , 'autocompleteAddImport': v:true
    \ , 'pursExe': 'purs'
    \ , 'addNpmPath': v:true
    \ }

I also cannot find the use of the string trace.server in the language server, so I guess it is VS Code's plugin parameter and not for the language server. The best I can do instead is to set LanguageClient's options:
let g:LanguageClient_trace="verbose"
let g:LanguageClient_loggingLevel="INFO"

Got confused by this because I have to send the prefix purescript. to commands.  Can this be added to the readme file to avoid confusion?

But even after changing the configuration to that, this is not being picked up. New traces.
[LC] Project root: /Users/sriharshachilakapati/Projects/*****-*******-****
[LC] [Info] Starting IDE server
[LC] [Log] PureScript Language Server started
[LC] [Info] {"capabilities":{"textDocument":{"colorProvider":null,"completion":{"completionItem":{"snippetSupport":false}},"signatureHelp":{"signatureInformation":{"parameterInformation":{"labelOffsetSupport":true}}}},"workspace":{"applyEdit":true,"didChangeWatchedFiles":{"dynamicRegistration":true}}},"processId":47249,"rootPath":"/Users/sriharshachilakapati/Projects/*****-*******-****","rootUri":"file:///Users/sriharshachilakapati/Projects/*****-*******-****","trace":"off"}
[LC] [Log] Starting with args: ["/Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/node","/usr/local/bin/purescript-language-server","--stdio","--config","{\"purescript.pscIdePort\": null, \"purescript.addNpmPath\": true, \"purescript.autocompleteAddImport\": true, \"purescript.trace.server\": \"verbose\", \"addNpmPath\": true, \"purescript.autoStartPscIde\": true, \"purescript.pursExe\": \"purs\"}"]
[LC] [Log] Starting with cwd: /Users/sriharshachilakapati/Projects/*****-*******-**** and using root path: /Users/sriharshachilakapati/Projects/*****-*******-****
[LC] [Info] Resolved IDE server paths (npm-bin: false) from PATH of /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin:/Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin:/Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin:/Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin:/Users/sriharshachilakapati/.jenv/shims:/Users/sriharshachilakapati/bin:/usr/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin:/Users/sriharshachilakapati/.jenv/shims:/Users/sriharshachilakapati/bin:/Users/sriharshachilakapati/Library/Android/sdk/build-tools/:/Users/sriharshachilakapati/Library/Android/sdk/platform-tools/:/Users/sriharshachilakapati/Library/Android/sdk/tools/:/Users/sriharshachilakapati/Library/Android/sdk/build-tools/:/Users/sriharshachilakapati/Library/Android/sdk/platform-tools/:/Users/sriharshachilakapati/Library/Android/sdk/tools/ (1stis used):
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/purs: 0.12.3
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin/purs: 0.11.7
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/purs: 0.12.3
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin/purs: 0.11.7
[LC] [Info] /Users/sriharshachilakapati/.nvm/versions/node/v9.5.0/bin/purs: 0.11.7
[LC] [Warning] Found multiple IDE server executables; using /Users/sriharshachilakapati/.nvm/versions/node/v11.9.0/bin/purs
[LC] [Info] Found existing port from file: 15345

Any ideas where I'm doing wrong?

@nwolverson Can you please help me out regarding this issue?
I think it's only a configuration issue, but need help in figuring out the correct JSON to pass to language server.

Bump

Finally was able to figure out after going through the atom-ide plugin's code. This is the correct config to be passed.
let config =
    \ { 'autoStartPscIde': v:true
    \ , 'pscIdePort': v:null
    \ , 'autocompleteAddImport': v:true
    \ , 'pursExe': 'purs'
    \ , 'addNpmPath': v:true
    \ }

let configWrapper =
    \ { 'purescript': config
    \ }

Sending configWrapper now to language server solved this issue. However, this information is hard to find. Can you add this to the README?


Add moduleResolution Node as default

Describe the feature you'd like to request
Having moduleResolution in base tsconfig or react-library tsconfig
Describe the solution you'd like
I want to use 3rd-party dependencies (i.e. node_modules) under my packages (e.g. packages/ui) out of the box
Describe alternatives you've considered
The current situation works ok enough, as there is a clear message from TypeScript suggesting changing moduleResolution the first time we try to import a library. However I think turborepo should just work

Sounds great. Can you make a PR for this that updates the template and starters?


Don't pollute System.Linq namespace

Currently, the Async extension methods, like CountAsync() and ToListAsync() live in the System.Linq namespace. However, most methods depend on IAsyncQueryProvider to work, which is specific to Entity Framework.
The problem here is, that if someone else writes a LINQ provider with a CountAsync() extension method with no dependency on Entity Framework, this setup cannot work anymore if you combine the two in one project. The only ways to prevent compile time problems, is one of the following awful options:

Remove the System.Linq namespace reference; but this then breaks all LINQ queries (using Where, Select, etc.), including query comprehension syntax. So that's not an option.
Rewrite all async methods over non-EF queries to to explicit Class.MethodAsync(query, ...) syntax rather than query.MethodAsync(...). (And if the other library was just as smart as EF, rewrite your EF queries too!)

Please, please, don't assume you're the only LINQ provider out there. Pick your own extension namespace (such as Microsoft.Data.Entity), or play nicer with other async IQueryProviders. Right now, there's no way to opt-out.

Our general rule is that extension methods go in the namespace of the target type... otherwise they are very difficult to find. For most methods it is extremely unlikely there will ever be a conflict because they are very EF specific (IQueryable<T>.AsNoTracking(), IServiceProvider.AddEntityFramework(), etc).
Your point is fair that it's possible another framework could define the same async methods and there would be a conflict. We've had the methods in this namespace for a number of releases now and there hasn't been an issue. If it ever becomes an issue it probably just means it's time to define a common contract for them and share it. However, it doesn't seem super likely that this will happen with a technology defining the same methods and being used in the same project as EF.

@rowanmiller, please don't just close this issue off without considering you might be wrong. Because you are very wrong.

We've had the methods in this namespace for a number of releases now and there hasn't been an issue.

The current EF does not pollute the System.Linq namespace: all queryable extension methods are defined in System.Data.Entity.QueryableExtensions. This means I can opt-out of them. So this co-opting of System.Linq is a change to the public facing side of EF. It is an entirely new issue.
And I predict: in the future, when you do realise you've broken many scenarios, the argument will probably be "we can't remove it again, because that would be a breaking change".
And, by moving QueryableExtensions to System.Linq you are actually breaking existing code, that has nothing to do with EF. At my company we actually do use like-named methods (albeit scoped to our own namespace), such as Include, and quite surprisingly to you perhaps, ToListAsync, CountAsync.
Your co-opting of the namespace System.Linq means that even the mere presence of EntityFramework.dll will require us to rewrite existing code:
var count = (from x in context.Something.Include(x => x.Relation); // <- ambiguous method call
             where x.SomethingElse == whatever
             orderby x.Orderable
             select x).CountAsync(); // <- ambiguous method call

Apart from the obfuscated identifiers, this is actual working code right now in our projects! But if EF7 would be present (directly or even indirectly), we'd have to rewrite all such code to something like
var count = OtherFrameworkExtensions.CountAsync(
             from x in OtherFrameworkExtensions.Include(context.Something, x => x.Relation)
             where x.SomethingElse == whatever
             orderby x.Orderable
             select x);

By co-opting the System.Linq namespace, EF has become viral: it infects all code that uses LINQ, whether you have a hard dependency on EF or an indirect dependency on EF through a referenced project (because the C# compiler chain knows no indirect dependencies).

If it ever becomes an issue it probably just means it's time to define a common contract for them and share it.

It already is an issue. So please don't make this decision lightly, before you unleash this breaking change.

However, it doesn't seem super likely that this will happen with a technology defining the same methods and being used in the same project as EF.

Same solution you mean. Because if any project even indirectly references EntityFramework, the NuGet and C# compiler system promote them to an actual hard depency/reference, and then boom: broken code.
And you have already broken our company's code, so the problem is indeed super likely. Unless you're writing simple pet shop or music store toy applications. Think bigger: think actual enterprise code bases, with dozens of libraries/project references.
Also, never heard of other ORMs, like NHibernate? Are they no longer allowed to support async by design? Or are they to beg the EF team to please provide extensibility points within EF just to play nice? Because, like I said: there is no longer a way to opt-out unless you completely remove EF7 from the picture, otherwise you're likely to run into this issue.
And this is a new issue with EF7. It wasn't an issue before!

@rubenprins I second that.
We also have "Include" methods defined in our code and placing the extensions in System.Linq namespace will be a breaking change for our code base as well.

Thanks for the feedback guys... and you are correct we did use System.Data.Entity namespace in past releases (which has caused some confusion). We re-discussed this today and agreed with you that these particular methods are different from the other extension methods that we put in the namespace of the "target class" because they are a more general concept and they also target a type that we do not own.
We'll move them out to Microsoft.Data.Entity

@rowanmiller, thanks for the update. Much appreciated.
It would be awesome if the EF concept of IDbAsyncQueryProvider (now IAsyncQueryProvider) were promoted to the BCL (as IAsyncQueryProvider), together with IAsyncEnumerable/Enumerator, and backed by the obvious AsyncEnumerable and AsyncQueryable classes.
That way, EF's QueryableExtensions wouldn't be needed in the first place, and no-one else would need to reinvent this framework either. (Except for EF-specific methods like Include/AsNonTracking, but these should have been scoped to Microsoft.Data.Entity anyway.)
With all the hype of Task/async/await, you'd think this would be a no-brainer.
For EF7, this would also remove the need for a dependency on Ix-Async, which does not even follow the BCL naming guideline of appending Async to actual async methods, ignores IQueryable, provides a host of RX idioms not present in System.Linq, and also nestles in System.Linq (fortunately extending only its own interface, IAsyncEnumerable, which is unfortunately placed in System.Collections.Generic).
Anyway, I very much appreciate your efforts in this area. Hopefully, the BCL team will someday too.

Fixed in 71e5bf5a7109cda2cf4399dbb565ec31ee17654d

@rubenprins I also want EF-Core to reference a common IAsyncQueryProvider.  Many of your complaints regarding Ix were resolved, and EF-Core references it anyway, so I think that's its natural home.  In fact, it's already there although EF-Core still uses its own variation.
My interest in this subject started when I encountered a situation where I need to execute a query in a way that's independent of EF-Core.  I ended up requesting a feature for Ix which I hope to eventually use with EF-Core.  That way, like you said, these async query execution methods wouldn't need to be reinvented by EF-Core or elsewhere.


Add a project board :)

[x] add the board itself
[x] make sure everyone can add issues to the board

Finished together b4 class tonight.


[BugFix] fix formatting for VideoRecorder docstring

Description
Fixes the formatting for the VideoRecorder docstring example, which is currently broken.
Types of changes

[X] Documentation (update in the documentation)

Checklist

[X] I have read the CONTRIBUTION guide (required)
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[X] I have updated the documentation accordingly.

Also, torchrl.envs.transforms.Composite is mentioned in the example docstring but it looks like this transform doesn't exist anymore, shall I delete it?
Hope to make more meaningful contributions in the future! :)

Also, torchrl.envs.transforms.Composite is mentioned in the example docstring but it looks like this transform doesn't exist anymore, shall I delete it?

I think it will work without the transforms: torchrl.envs.Composite. Sphinx links are a bit restrictive, it's not sufficient for the python import to work, you also need to declare the path explicitly.

I'm not sure I understand, I don't see any function for torchrl.envs.Composite, the closest I can find anywhere in the codebase is torchrl.data.CompositeSpec, which looks unrelated.
Either way, I tried with torchrl.envs.Composite and sphinx still doesn't pick up any link

Oh yeah lol it's Compose not Composite 😅

Haha that makes a lot more sense. Fixed, and verified!


Lumina 5 update

Builds + did a quick test in an instance (may need further testing tho)

Hi, thank you for taking interest in updating DeathRecap! I'm a little puzzled by why you decided to replace the typed Sheets with RawRow. Lumina5 still provides types for Action, Status, etc.

I may have been really tired and completely misinterpret this section LMAO
looks like you've updated it so i'm gonna go ahead and close this!


IOS App 2.2.1 on iPad does not show Domain field, no support for private server

Zulip IOS App 2.2.1 running on iPad Air 2 (Wi-Fi only) , IOS 9.1, shows only email and password fields in the connection view. There's no possibility to point the connection to a custom domain/server since the input field for domain is missing.

Same for the iPad mini first Generation. iPhone 6 app has the option to pick a custom server.


Investigate why we had to increase the memory size in qemu

Hey, I saw your comment in the paging.c file, and that you're returning huge_pages even though you shouldn't, because of an ELF loading problem. Could you check if there's any difference with my ELF changes? Maybe that's one of the big causes of memory issues, including this qemu memory size increase.

I say we investigate the whole memory management system and see if we find any inconsistencies.
I already saw that there's some functions that return 0 during a uint64_t function, which according to your references, should have been an optional value instead of 0.
Could be mistaken and use that 0 rather than considering it an error. We should create a structure for Optionals, like optional_uint64_t, for the base fields we need optionals from. Something like this:
#include <stdbool.h>

typedef struct optional_uint64 {
    bool valid;
    uint64_t value;
} optional_uint64_t;

optional_uint64_t make_optional_uint64(bool valid, uint64_t value) {
     optional_uint64_t out_value;

      out_value.valid = valid;
      out_value.value = value;

     return out_value;
}

We can also create stdbool.h from https://pubs.opengroup.org/onlinepubs/009695399/basedefs/stdbool.h.html to have bools in C.

I'm currently investigating the memory issues. I'm finding very strange behaviour, looking into the documentation to see what's wrong.

I'm currently investigating the memory issues. I'm finding very strange behaviour, looking into the documentation to see what's wrong.

Which parts specifically?
I cannot remember correctly but I could not remap the kernel last time, so I gave up with the recursive mapping. Maybe it is the problem, but maybe it's not. I guess this will be a problem eventually, thugh.

Immediately getting frames unable to allocate.
DEBUG src/mmu/frame.c:36:frame_init(): Initialized frame allocator with multiboot_start =<PHONE_NUMBER>122058 multiboot_end=0000000000122400 kernel_start=0000000000100000 kernel_end=0000000000122053
DEBUG src/mmu/paging.c:105:page_containing_address(): page_containing_address returning 262193 for<PHONE_NUMBER>040031
DEBUG src/mmu/paging.c:105:page_containing_address(): page_containing_address returning 262144 for<PHONE_NUMBER>040000
DEBUG src/mmu/alloc.c:30:alloc_init(): heap_start_page=262144 heap_end_page=262193 max_pages=50
DEBUG src/mmu/frame.c:99:frame_allocate(): allocated frame with addr=0000000000001000 free_frame=1
DEBUG src/mmu/paging.c:232:map_page_to_frame(): mapping page=262145 to frame=0000000000001000 (number=1) with flags=0x3
DEBUG src/mmu/paging.c:61:next_table_address(): addr=FFFFFFFFFFFFF000 next table=FFFFFFFFFFE00000
DEBUG src/mmu/paging.c:44:next_table_address(): entry at index=1 in table=FFFFFFFFFFE00000 is not present, returning 0
DEBUG src/mmu/paging.c:155:translate_page(): did not find p2<PHONE_NUMBER>000000), returning NULL
DEBUG src/mmu/paging.c:243:map_page_to_frame(): page=262145 does not have a valid frame number

This issue can be closed, I fixed most MMU problems and added some more tests. The memory size is now 500MB FWIW.


fix: modified pages/community/tsc.js to enable TSC social member links ope…

Description
In the TSC page of the website, the social icons present inside each TSC Member card should open their social accounts in another tab, instead in the same tab.
issue https://github.com/asyncapi/website/issues/1496

@KindEni Issue already resolved in another PR. Hence closing this one.


Background Image WordPress Header

Hello @luruke
First of all, thanks a lot for sharing this awesome plugin.
I'm testing out your plugin and it works perfectly, like it a lot 👍
I have a WP theme and would like to implement your plugin with it.
My issue is that I have a dynamic background image header, that changes.
The background image header is set by the user as a featured image on posts and pages.
When I try your plugin, everything goes well except for the image, it doesn't change and the height is cropped to half instead of being full screen.
I'm  using jQuery to adjust the width and height of this header background image on any screen size like so
jQuery(document).ready(function ($) {
    // Defining a function to set size for #masthead
    function fullscreen() {
        jQuery('#masthead').css({
            width: jQuery(window).width(),
            height: jQuery(window).height()
        });
    }

    fullscreen();

    // Run the function in case of window resize
    jQuery(window).resize(function () {
        fullscreen();
    });

});

My HTML structure is like the following code

    ...
    ... // here is the problem !

...

//#page
  ...

Wherever I put
 and
 the result is the same.
Before #page or after, before  or after, before  or after, always same thing, everything changes except the header background image.
I'm retrieving the header background image with a function
function mytheme_header_style() {
    // Declare $post global if used outside of the loop.
    $post = get_post();
    // Check if post is object otherwise we're not in singular post.
    if (!is_object($post)) {
        return;
    }
    // If Object
    $backgroundImg = wp_get_attachment_image_src(get_post_thumbnail_id($post->ID), '');
    // Add inline style to the backgroung header image.
    if (has_post_thumbnail()) {
        $custom_header_style = '
            .intro-header {
                background-image: url( ' . $backgroundImg[0] . ' );
            }
        ';
    } else {
        $custom_header_style = '
            .intro-header {
                background-image: url( ' . get_template_directory_uri() . '/components/header/images/default-hero.jpg' . ' );
            }
        ';
    }
    wp_add_inline_style( 'mytheme-main-style', $custom_header_style );
}
add_action( 'wp_enqueue_scripts', 'mytheme_header_style' );

Hope you can help me with this.
Thanks a lot for all your efforts.

If it's inside a barba-container and it's changing for each "page" you should "re-run" your javascript code.
In the specific you can use the event newPageReady (http://barbajs.org/events.html)

Hello @luruke
Yes my #masthead is inside barba-container like this
    <body <?php body_class(); ?>>

 // Here is the barba wrapper

 // Here is the barba container
                    <?php esc_html_e('Skip to content', 'mytheme'); ?>

                        <?php get_template_part('components/navigation/navigation', 'top'); ?>

                    <?php // Here is the #masthead
                    if (is_404()) {
                        get_template_part('components/header/bg', '404');
                    } elseif (is_search()) {
                        get_template_part('components/header/bg', 'search');
                    } else {
                        get_template_part('components/header/bg', 'header');
                    }

                    ?>

The bg-header.php is like so
<?php
/**
 * This file generates the background header image with the title and the subtitle.
 *
 * The background image is added by using inline style.
 * @see mytheme-functions.php | mytheme_header_style()
 */
?>

                        <?php
                        if (is_single() || is_page() || is_sticky() || is_front_page()) {
                            the_title('
', '
');
                        } else {
                            the_title('
', '
');
                        }

                        ?>

                            <?php
                            if ('post' || 'page' === get_post_type()) :
                                global $wp_query;
                                $postid = $wp_query->post->ID;
                                echo esc_html(get_post_meta($postid, '_unprefix_subtitle', true));
                                wp_reset_query();
                            endif;

                            ?>

The mytheme_header_style() function is described in the first comment.
So everything is returned as expected (title and subtitle), except that the image is always the same one (it's not retriving the expected image that is set as featured image), and it's not full screen.
I'm relatively new to JS :)
Thanks in advance for your help

Hi @LebCit, are you sure this problem is caused by something about Barba?
When you run your fullscreen() function?
Did you put it inside your barba lifecycle?

@LebCit have you solved this?
Meanwhile i'll close the issue.
If you found that the issue is relative to barba.js do not hesitate to reopen it.
Cheers


Investigate Test_Thread failure in jdk_lang tests

Once we have the openjdk_test_x86-64_linux build tied to the image builds, we should verify if the Test_Thread test still fails.
If so, compare it against the run done by the jtreg.sh script to see if there is any difference in the configuration of the tests, that cause it to fail in one place, but pass in another.

Closing, as this failure is no longer present (there are 4 other issues open #144, #141, #128, #126 relating to the jdk_lang group, but not Test_Thread).


[NeuralChat] Finetuning for LLaVA failing on image + text pairs

This PR https://github.com/intel/intel-extension-for-transformers/pull/1199 fixed training on text only examples, and I tested the fix with text only and image only examples thinking that would resolve the issue overall but there is a new error for text + image examples with an error coming from zero3 in DeepSpeed:
Traceback (most recent call last):
  File "/intel-extension-for-transformers/intel_extension_for_transformers/neural_chat/examples/finetuning/multi_modal/train.py", line 319, in <module>
    train()
  File "/intel-extension-for-transformers/intel_extension_for_transformers/neural_chat/examples/finetuning/multi_modal/train.py", line 296, in train
    trainer.train()
  File "/intel-extension-for-transformers/optimum-habana/optimum/habana/transformers/trainer.py", line 486, in train
    return inner_training_loop(
  File "/intel-extension-for-transformers/optimum-habana/optimum/habana/transformers/trainer.py", line 842, in _inner_training_loop
    tr_loss_step = self.training_step(model, inputs)
  File "/intel-extension-for-transformers/optimum-habana/optimum/habana/transformers/trainer.py", line 1367, in training_step
    self.accelerator.backward(loss)
  File "/usr/local/lib/python3.10/dist-packages/accelerate/accelerator.py", line 1983, in backward
    self.deepspeed_engine_wrapped.backward(loss, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/accelerate/utils/deepspeed.py", line 167, in backward
    self.engine.backward(loss, **kwargs)
  File "/DeepSpeed/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/DeepSpeed/deepspeed/runtime/engine.py", line 1992, in backward
    self.optimizer.backward(loss, retain_graph=retain_graph)
  File "/DeepSpeed/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/DeepSpeed/deepspeed/runtime/zero/stage3.py", line 2097, in backward
    self._get_param_coordinator(training=True).reset_step()
  File "/DeepSpeed/deepspeed/runtime/zero/partitioned_param_coordinator.py", line 203, in reset_step
    raise RuntimeError(f"still have inflight params "
RuntimeError: still have inflight params [{'id': 975, 'status': 'INFLIGHT', 'numel': 16777216, 'ds_numel': 16777216, 'shape': (4096, 4096), 'ds_shape': (4096, 4096), 'requires_grad': True, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([2097152])}, {'id': 973, 'status': 'INFLIGHT', 'numel': 4194304, 'ds_numel': 4194304, 'shape': (4096, 1024), 'ds_shape': (4096, 1024), 'requires_grad': True, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([524288])}]

I think the hack the original authors mention in this PR "# FIXME: this is a hacky fix, for deepspeed zero3 to work" was likely to avoid this issue. It might be related to this issue https://github.com/microsoft/DeepSpeed/issues/3156 as I'm guessing it happens when both image and text only examples appear in a batch, so for some examples in the batch it runs through the vision encoder and some examples it doesn't which is sort of like the dynamic forward method referenced in the DeepSpeed issue.

Looks like you can get different errors coming from the same reset_step function in partitioned_param_coordinator.py, here's another one:
Traceback (most recent call last):
  File "/intel-extension-for-transformers/intel_extension_for_transformers/neural_chat/examples/finetuning/multi_modal/train.py", line 319, in <module>
    train()
  File "/intel-extension-for-transformers/intel_extension_for_transformers/neural_chat/examples/finetuning/multi_modal/train.py", line 296, in train
    trainer.train()
  File "/intel-extension-for-transformers/optimum-habana/optimum/habana/transformers/trainer.py", line 486, in train
    return inner_training_loop(
  File "/intel-extension-for-transformers/optimum-habana/optimum/habana/transformers/trainer.py", line 842, in _inner_training_loop
    tr_loss_step = self.training_step(model, inputs)
  File "/intel-extension-for-transformers/optimum-habana/optimum/habana/transformers/trainer.py", line 1367, in training_step
    self.accelerator.backward(loss)
  File "/usr/local/lib/python3.10/dist-packages/accelerate/accelerator.py", line 1983, in backward
    self.deepspeed_engine_wrapped.backward(loss, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/accelerate/utils/deepspeed.py", line 167, in backward
    self.engine.backward(loss, **kwargs)
  File "/DeepSpeed/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/DeepSpeed/deepspeed/runtime/engine.py", line 1992, in backward
    self.optimizer.backward(loss, retain_graph=retain_graph)
  File "/DeepSpeed/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/DeepSpeed/deepspeed/runtime/zero/stage3.py", line 2097, in backward
    self._get_param_coordinator(training=True).reset_step()
  File "/DeepSpeed/deepspeed/runtime/zero/partitioned_param_coordinator.py", line 208, in reset_step
    assert_ints_same_as_other_ranks([m.id for m in self.__submodule_order])
  File "/DeepSpeed/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/DeepSpeed/deepspeed/runtime/zero/utils.py", line 97, in assert_ints_same_as_other_ranks
    raise RuntimeError(f"disagreement between rank0 and rank{dist.get_rank()}: "
RuntimeError: disagreement between rank0 and rank4: rank0: [1018575771, -1145947199,<PHONE_NUMBER>, ...

I can reproduce this with a smaller json file containing 50 text and 50 image examples llava_v1_5_mix665k_cleaned_small.json

hi, @dillonalaird
I have reproduced the problem and found that there are some dynamic shape inputs on the finetuning stage, which have impact on the training. I will rewrite the data preprocess and validate it, then push a pr as soon as possible.

the issue is fixed by PR 1212

Awesome, thanks @lkk12014402 !

hi, @dillonalaird
I have reproduced the problem and found that there are some dynamic shape inputs on the finetuning stage, which have impact on the training. I will rewrite the data preprocess and validate it, then push a pr as soon as possible.

same question. how to fix it? what is "static shape"(+1 op), thanks


关于训练

您好,感谢您的贡献,请问一下您为什么复现时损失为nan,参数均与您文章中一致.望解答,感激不尽!

This issue occurs due to the old mmdet version.
You can either resume from the pre-trained weights or adjust the coefficient lambda from 0.1 to 0.001.

This issue occurs due to the old mmdet version. You can either resume from the pre-trained weights or adjust the coefficient lambda from 0.1 to 0.001.

Sorry to bother you again. Will adjusting the lambda parameter affect the reproducibility and accuracy of the experiment? Could you provide the pre-trained weights? Thank you

Please refer to the README for pre-trained weights.

Please refer to the README for pre-trained weights.

Please refer to the README for pre-trained weights.

Please refer to the README for pre-trained weights.

Thank you for your reply. Wishing you good health, smooth work, and fruitful achievements.


Broken list text

 List text  gets broken into multiple paragraph if it contains any formatting element due to latest version changes.
Related to new list parser built for supporting nested lists.

#12 Solves it


Allow empty strings in JSON content

Coverage remained the same at 95.288% when pulling e7f4a976b696d7b615f051e76919d87752c28546 on cody-fix-pdf-removing-empty-strings into 7f5a980f4a02e2d99ee32c2b1e824a8fda777c4d on main.


Option to search for genome accessions in search bar?

As mentioned in discussion with Esraa Elsaeed on the Public Slack, it could be beneficial to search for genome accessions for taxa with validated names (and those that have been registered) to ensure that a named taxon under the SeqCode isn't named twice (i.e., a close match in NCBI to a genome with a generic taxon identifier may be in process or already validated in the registry, but the name isn't reflected in NCBI yet).

To clarify: Is this about an option to search for genome accessions or an option to do genome search using something like ANI or a similar metric?
I think a genome accession search would be very useful indeed (or even a browsing). We already offer a JSON version of it (see here), which is currently being used by both MiGA and GTDB, but I agree that a human-readable version would be great!

Yes, like a search bar where one can type in an accession and it brings up the species page if it exists for the accession. Because one can use MiGA or other genome-based searches to identify conspecific taxa, but actually searching in a human-readable version I think.


Variable Explorer generates an error while editing a DataFrame

Description of your problem
What steps will reproduce the problem?

Open spyder 3.0.0b2 (from https://anaconda.org/spyder-ide/spyder ) using PyQt5

On the ipython terminal type
import pandas as pd
df2 = pd.DataFrame([dict(a=1,b="test", c=False), dict(a=2, b="test2", c=False)])

Click on Variable Explorer, double click on df2 to open the DataFrame editor

Right click on a number (last row, column "a" has a number=2 in the example), choose To int

What is the expected output? What do you see instead?
I expect nothing to happen (it's an int already).
Instead, Spyder closes and I see Unhandled Python exception. Aborted
Please provide any additional information below
This seems to happen with other conversions.
Versions and main components

Spyder Version: 3.0.0b2
Python Version:

Python 3.5.1 |Continuum Analytics, Inc.| (default, Dec  7 2015, 11:16:01)
[GCC 4.4.7 20120313 (Red Hat 4.4.7-1)] on linux

Operating system: Linux Mint

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
IPython >=3.0      :  4.0.2 (OK)
jedi >=0.8.1;<0.9.0:  None (NOK)
matplotlib >=1.0   :  1.5.1 (OK)
numpy >=1.7        :  1.10.4 (OK)
pandas >=0.13.1    :  0.17.1 (OK)
pep8 >=0.6         :  1.6.2 (OK)
psutil >=0.3       :  3.3.0 (OK)
pyflakes >=0.6.0   :  1.0.0 (OK)
pygments >=1.6     :  2.1.1 (OK)
pylint >=0.25      :  1.5.4 (OK)
qtconsole >=4.0    :  4.1.1 (OK)
rope >=0.9.4       :  0.9.4-1 (OK)
sphinx >=0.6.6     :  1.3.1 (OK)
sympy >=0.7.3      :  None (NOK)
zmq >=13.0.0       :  15.2.0 (OK)
I haven't checked other QT APIs (PyQt4/PySide).

What PyQt5 packages are you using?

According to conda list, I am using yours (spyder-ide)
 conda list | grep spyder-ide
glib                      2.43.0                        2    spyder-ide
gst-plugins-base          1.4.5                         0    spyder-ide
gstreamer                 1.4.5                         0    spyder-ide
icu                       56.1                          0    spyder-ide
libxcb                    1.11.1                        0    spyder-ide
pyqt5                     5.5.0                    py35_0    spyder-ide
qt5                       5.5.0                         0    spyder-ide
qtawesome                 0.2.0                      py_0    spyder-ide
qtpy                      0.1.3                      py_0    spyder-ide
spyder                    3.0.0b2                  py35_0    spyder-ide

Using pyqt (having removed pyqt5 and qt5 packages) I get a message on the console instead:
Spyder Internal Console

This console is used to report application
internal errors and to inspect Spyder
internals with the following commands:
  spy.app, spy.window, dir(spy)

Please don't use it to run your code

>>> Traceback (most recent call last):
  File "/home/soller/anaconda3/envs/smartims_dev/lib/python3.5/site-packages/spyderlib/widgets/variableexplorer/dataframeeditor.py", line 433, in <lambda>
    self.change_type(func),
  File "/home/soller/anaconda3/envs/smartims_dev/lib/python3.5/site-packages/spyderlib/widgets/variableexplorer/dataframeeditor.py", line 443, in change_type
    [model.setData(i, '', change_type=func) for i in index_list]
  File "/home/soller/anaconda3/envs/smartims_dev/lib/python3.5/site-packages/spyderlib/widgets/variableexplorer/dataframeeditor.py", line 443, in <listcomp>
    [model.setData(i, '', change_type=func) for i in index_list]
  File "/home/soller/anaconda3/envs/smartims_dev/lib/python3.5/site-packages/spyderlib/widgets/variableexplorer/dataframeeditor.py", line 300, in setData
    self.df.iloc[row, column - 1] = change_type(val)
TypeError: 'bool' object is not callable

I can reproduce your error in beta2 but this seems fixed in beta3 (I tested it several times :-).
So these are great news (because I don't remember when I fixed it!).


Clarify the expectations for when static data is included and make it configurable

When static data is included in the result set it can be confusing.
The static data, when returned, might show up in row 0, but the row has no index values which can require extra user-side filtering.
Proposal

Add a new enum along the lines of  include_static to QueryExpression. The enum should have 3 values.

AUTO include a static row at row 0 only if there is static data in the dataset (the default)
ALWAYS row = 0 will be static regardless of whether there is static data
NEVER will never include an explicit row for the static data -- it will only be accessible via LatestAt.

For AUTO / ALWAYS, the choice to include the static data is independent of any value of filtered_index_range.
When using filtered_index_values or using_index_values :

The static data will only be included if the input includes TimeInt::STATIC
If ALWAYS and the input has no TimeInt::STATIC return an error
If NEVER and the input has  a TimeInt::STATIC return an error.

Note that we can implement a helper such as view(...).static_only() which is just an alias for:
include_static=ALWAYS, filtered_index_values=[TmeInt::STATIC]

For 0.19
This does not need to be done for 0.19. Let's make the behavior match AUTO as default and add the configurabillity later.

If filtered_index_values or using_index_values are set, then the choice to include static data is determined by an explicit TimeInt::STATIC in the input.
Otherwise, if the view contains any column which includes static data, then row 0 will be the Static row, and this data will be returned regardless of the range specified by filtered_index_range
We should clearly document: if there is static data in the results, it will always be in row 0 and that will be indicated by the the TimeInt for the index column being null

Thinking about this more I'm now in favor of the alternative proposal:

https://github.com/rerun-io/rerun/issues/7668


cuda out of memory

torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 1.81 GiB (GPU 1; 39.59 GiB total capacity; 34.83 GiB already allocated; 1.69 GiB free; 35.63 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 1.81 GiB (GPU 1; 39.59 GiB total capacity; 36.30 GiB already allocated; 1004.19 MiB free; 36.33 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 1.81 GiB (GPU 1; 39.59 GiB total capacity; 34.83 GiB already allocated; 1.69 GiB free; 35.63 GiB reserved in total by PyTorch)
If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

调小batch size


Merge Pull Requests

There are a few pull requests that are fully compatible and useful that I think can and should get merged in :)

Hey all,
My current employer requires project-specific approval to contribute to
open-source projects.
As a result, I'm going to step down from maintaining this project until
September (when I'll be back!).
There a handful of people with approval to merge PRs; please feel free to
do so as you see fit :)
See you all soon,
Evan
On Fri, May 26, 2017 at 13:16 Travis McCrea<EMAIL_ADDRESS>wrote:

There are a few pull requests that are fully compatible and useful that I
think can and should get merged in :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/evanbergeron/DND-5e-LaTeX-Template/issues/38, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGQvMxUOTN3COyFqTSqr4E7joxD2MS5mks5r9zMcgaJpZM4NoBMR
.

This project has been of great benefit to me, and I would like to contribute back to it. I would be happy to become a maintainer to merge in pull requests.

I'll also just tag some PRs that I feel are valid:
#28
#30
#41
#42
#44
#45

I merged in #45 and #42. I have posted comments on #44. The rest I will be taking a closer look at, but expect things to move quickly.

Thanks going through all the PRs Brian!


readme: add Python3 library dependency

libfdt (subproject) requires Python3 development headers to
compile its pylibfdt bindings.

Better solution over in https://github.com/amboar/culvert/pull/74


Also mask/disable mender client

External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

[x] Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description
I believe we also need to disable the mender client, so it doesn't auto restart on the next boot? Or maybe I'm missing something, in such case maybe we should improve the documentation.

Hi @mifi and thank you for your contribution.
It looks good to me :+1:
Some formalities required. You need to sign your commit as instructed in our guidelines. You have tried by commenting on the commit, but what you need to do is to add the signature as part of the commit body.
For that:

fork the repository
checkout it locally
modify (add the space)
git commit -s -a
push and launch PR

I am sorry, but the GitHub UI interface does not allow for adding the signature nor amending commits, so unfortunately you need to use your local Git.
If you don't want to do that I can submit the change on your name.

Hi! That's a pity as it makes it harder for people to easily contribute and improve the documentation. Anyways I signed the commit fro mthe command line now. hope it works!

Hi! That's a pity as it makes it harder for people to easily contribute and improve the documentation. Anyways I signed the commit fro mthe command line now. hope it works!

I know :weary: But the signature is important for our open source license model, so there is no other way around that asking the user to "do something manually" on their machine ;)
Thank you!

Maybe it could be worked around by a github action that auto-signs the commits 🙈

Maybe it could be worked around by a github action that auto-signs the commits see_no_evil

But that is the point, we cannot sign contributor's commits... that defeats the license agreement.


[BUG} Error when sending formatted message

Canary 106485 (02153ef) Host 0.0.131 Linux 64-Bit (5.10.82-1-Lts)

(running on powercord + bdCompat)

I think this is just something that can be fixed with a try catch or an if, since some mutations are not add mutations.
Can you check if this affects the functionality?

I think this should be fixed. Please reopen if not.


add test case

use travis-ci

using https://www.getpostman.com/docs/validating_json_collections
https://schema.getpostman.com/json/collection/v1/ use this collection json and valid json and creating markdown file in testcase.

https://schema.getpostman.com/

https://github.com/Plate-Project/postman2md/tree/feature/travis-ci


[Nova] Disable CircleCI Linux Unittests

Linux CPU and GPU Unittests have been running smoothly for several weeks, so going ahead and removing the CircleCI Linux Unittest jobs. If we see any issues or high queueing, we can bring this back.

@osalpekar I don't think this is the case for GPU: #6957. Could you look into this first?
@NicolasHug, AFAIK we want to skip the CPU tests on GPU machines, right? I think we need to update https://github.com/pytorch/vision/blob/4a310f26049371959617921d0eb9b001f4d262c6/test/conftest.py#L15-L24 to also account for GHA. Do you want to send a patch or shall I?


Time slider date labels are redundant to date-editing controls

Each of the three dates that the time slider keeps track of is represented twice in the time slider UI:

The range start date has input fields at the top and a label to the left of the slider.
The range end date has input fields at the top and a label to the right of the slider.
The current date has a label above the slider and a button to the right that opens a dialog box to edit it.

These two representations seem unnecessary, since an input field also indicates its current state. Moreover, the date range labels are presented in a compact date format, which necessitates a legend clarifying the date format. All this redundancy bloats the overall time slider control panel and makes it difficult for a new user to understand the relationship between the various controls.

Here are some suggestions for streamlining the control panel:

Move the range controls to either side of the slider, replacing the date labels and date format labels.
To maintain a compact width, shorten the month names in the month combo box.
Replace the <input type="number"> fields with <input type="text">. The increment/decrement steppers on <input type="number"> take up space, hurt accessibility, and surprisingly react to scroll wheel gestures on Windows.
Eliminate both Set buttons. Update the range immediately when the user leaves one of these combo boxes or input fields.
Move the ✏ button closer to the current date, or replace the current date label with editable controls and eliminate the ✏ button.

With these changes, it might be worth revisiting the control panel’s placement. A horizontal bar along the bottom of the map would integrate better into the overall site design. Since it wouldn’t cover up the map or any other controls, a minimized mode would no longer be necessary. There would also be room to lengthen the slider on most screens, allowing the user to navigate more granularly within the date range.

@danrademacher - these all seem like good suggestions. I cannot remember the name of the accessibility designer we worked with, but perhaps we could get her to weigh in here? I did think she had some opinions about the need for the Set buttons, but there may be others.

Frankly, I think the time slider’s accessibility is a moot point as long as the map it manipulates continues to be a raster map. #651 tracks replacing it with a vector map, which would support accessibility needs like high-contrast and large-text modes almost out of the box. With the addition of a well-used accessibility plugin, individual features would then be accessible to screen readers.

Totally agree, it's huge. My input -

Make a very long very slim slider across the full screen width at the bottom; longer means finer control when moving it (like when using a video slider, longer is better)

Just state the years at each end but don't give them change controls; make them clickable (blue underline), so you click them to get a popup to change them. The popup can either provide the one you clicked, or both, if both it should focus in the popup on the one you clicked.

Have the browser remember the last date set, a user will be fairly consistent with needs during whatever project they are working on. An option can be available on the date change popup to turn on this remembering to comply with regulations.

Month and day are redundant but if they must be there on the popup changer, make them very unobtrusive so the year stands out for changing (any prominent method is fine, e.g. give the year box a yellow background).

Current Date can be on the slider button that you drag along, or if preferred at one of the ends, I think on the button is better.

David

These are great suggestions! Recently, the Gramps Web project implemented a compact design in gramps-project/gramps-web#454 that happens to incorporate several of these ideas: extending the slider all the way across the map, only adjusting the year by default, remembering the date across page views, annotating the knob with the current date, and fitting the date range to what’s being shown on the map. Our time slider needs to include more options such as animation and precise date entry, but it suggests a way forward for this design.

Option to fit slider dates to items in the vicinity - an area maybe a bit wider than the viewing port. So you can zoom your area, click auto-fit, then get to work.

This would be nice to have in conjunction with a histogram display: #649.


Giada: not working with pipewire

Describe the bug
I think that Giada is not working with pipewire. When I start the software, I get an error You soundcard is not configured properly. Check the configuration and restart Giada. Moreover, Giada never appears in the QJackctl Graphe. I tried 3 things:

to configure it to use Jack (it uses ALSA by default) in the MIDI tab, but same issue.
to configure it to use PulseAudio in the Sound System tab (defaults to "None"), not better. Here are the logs:

[init] Giada 0.16.4
[init] Build date: Jan  1 1980
[init] Dependencies:
[init]   FLTK - 1.3.5
[init]   RtAudio - 5.1.0
[init]   RtMidi - 4.0.0
[init]   Libsamplerate
[init]   Libsndfile - libsndfile-1.0.30
[init]   JSON for modern C++ - 3.9.1
[midiMapConf::init] scanning midimaps directory '/home/leo/.giada/midimaps/'...
[midiMapConf::init] unable to scan midimaps directory!
[midiMapConf::read] midimap not specified, nothing to do
[init] MIDI map read failed!
[KA] using system 0x20
[KA] Opening device out=0, in=0, samplerate=44100
[KA] 1 device(s) found
  0) PulseAudio
[KA] rtSystem init error:
[mixer::init] buffers ready - framesInSeq=88200, framesInBuffer=1024
[KM] using system 0x3
[KM] 2 output MIDI ports found
  0) Midi-Bridge:Midi Through:(playback_0) Midi Through Port-0
  1) Carla:events-in
[KM] 2 input MIDI ports found
  0) Midi-Bridge:Midi Through:(capture_0) Midi Through Port-0
  1) Carla:events-out

to configure it to use Jack in the Sound System tab (defaults to "None")... Then it is catastrophic, it segfaults forever until I remove the ~/.giada folder:

iada
[init] Giada 0.16.4
[init] Build date: Jan  1 1980
[init] Dependencies:
[init]   FLTK - 1.3.5
[init]   RtAudio - 5.1.0
[init]   RtMidi - 4.0.0
[init]   Libsamplerate
[init]   Libsndfile - libsndfile-1.0.30
[init]   JSON for modern C++ - 3.9.1
[midiMapConf::init] scanning midimaps directory '/home/leo/.giada/midimaps/'...
[midiMapConf::init] unable to scan midimaps directory!
[midiMapConf::read] midimap not specified, nothing to do
[init] MIDI map read failed!
[KA] using system 0x1
[KA] Opening device out=0, in=-1, samplerate=44100
[KA] 8 device(s) found
  0) Audio interne Stéréo analogique
  1) Firefox
  2) Carla
  3) Audio interne Stéréo analogique
  4) Audio interne Stéréo analogique Monitor
  5) Audio interne Stéréo analogique
  6) Audio interne Stéréo analogique Monitor
  7) Carla
[KA] JACK in use, samplerate=48000
Segmentation fault (core dumped)

Steps To Reproduce
Steps to reproduce the behavior:

Install pipewire on NixOs system-wise
Run giada

Notify maintainers

@petabyteboy
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.48, NixOS, 21.11pre-git (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4pre20210601_5985b8b`
 - channels(me): `"home-manager, local-nixos-unstable-21.11pre304626.8ecc61c91a5, local-nixpkgs-unstable-21.11pre304920.dd98b100651, nixos-20-03-20.03.3328.6d1a044fc9f, nixos-unstable-21.11pre304626.8ecc61c91a5, test"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Both of these issues seems to be solved by #140628, thanks! To get input, you just need to enable it in the configuration file. I just have a strange "larsen" effect when I try to register audio, but I guess I first need to learn how to use the software ^^


Updated ALTER ROLE and REASSIGN OWNED privileges

Fixes https://github.com/cockroachdb/docs/issues/11209.
Fixes https://github.com/cockroachdb/docs/issues/11400.

This change is

@rafiss ping on re-review :)


indentationRules not applied for autocomplete text

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.74.3
OS Version: x64 10.0.19045

Steps to Reproduce:

e.g. EndIf is already text within the file. indentationRules in language-configuration.json is configured as:
"indentationRules": {      "decreaseIndentPattern": {          "pattern": "^\\s*(endif|endwhile)\\b",          "flags": "i"      }  }

When manually typing EndIf, line will outdent as expected. However if selecting autocomplete suggested text "EndIf" (if text already exists elsewhere in the file), then indent/outdent rules are not applied.

Autocomplete text ("abc" options in the popup droplist) should follow all rules as if they were manually typed.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

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


How to add audio ?

In the Existing Project Instructions, Step 6 covers adding audio to the UI.

By scene.

Open MainMenu.tscn and PauseMenu.tscn.
Select the UISoundController node.
Add audio streams to the various UI node events.
Save the scenes.

Project-wide, with extras/.

Go to Project > Project Settings… > Autoload.
Make sure UISoundControllerAutoload is listed.
Note: It does not need a global variable enabled.
Close the window.
Open UISoundControllerAutoload.tscn.
Select the UISoundController node.
Add audio streams to the various UI node events.
Save the scene.

Let me know if any of those instructions are unclear, or if you are referring to something else, like background music?

I added the following instructions:

1.  Add `Music` and `SFX` to the project's default audio busses.

    1.  Open the Audio bus editor.
    2.  Click the button "Add Bus" twice (x2).
    3.  Name the two new busses `Music` and `SFX`.
    4.  Save the project.


loader: Fix handling of surface parameters

The loader code generation scripts had a bug that
caused all the surface->ICD surface translation C
code to be placed inside a block that only ran if
the ICD didn't support the extension function,
making it effectively a no-op.  ICDs that
supported the extension were instead called with
the un-translated surface handle, which could
cause a crash if they were expecting their local
surface handle.
Also, speculatively fix up return handling in this
same class of functions by handling the void
return type.

 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.

LGTM.  @MarkY-LunarG, you'd better have a go.


400 Client Error: Bad Request for url: https://i.instagram.com/api/v1/feed/user/xxxxxxxx/story

Since early this morning I'm not able to get any stories anymore! It shows this:
2018-04-25 14:37:57,827 - instagram_scraper.app - WARNING - Retry after exception HTTPError(u'400 Client Error: Bad Request for url: https://i.instagram.com/api/v1/feed/user/xxxxxxxx/story/',) on https://i.instagram.com/api/v1/feed/user/xxxxxxxx/story/
Using latest version

So they've changed the API... again?

@MintUser34827738947 sorry I haven't tried accessing stories

After about 12 hours it is working again! It was not IP related because I tested it on two different systems with different external IP addresses. And it wasn't account related because I used two different accounts.


Could you please change slf4j to jul?

Could you please change slf4j to JUL (Java Util Logging) ?

Doing so would potentially be a breaking change for all users who already have logging configured. slf4j is fairly standard for libraries to use for logging and it provides more functionality than just plain java.util.logging.
I wouldn't expect HikariCP to change the logging framework anytime soon.

I knew JUL doesn't provide enough functionality, but if I don't want to use slf4j ?


openssl password visible

Your Windows build number:  Version 10.0.18362.720

What you're doing and what's happening:

paul@pmxps:~/tmp> openssl aes-256-cbc -a -salt -in myfile.txt -out myfile.txt.enc
enter aes-256-cbc encryption password:secret123
Verifying - enter aes-256-cbc encryption password:secret123
paul@pmxps:~/tmp> openssl aes-256-cbc -d -a -in myfile.txt.enc -out myfile.txt.new
enter aes-256-cbc decryption password:secret123

What's wrong / what should be happening instead:

password should not be echoed to screen

Strace of the failing command, if applicable: openssl.trace

The above was with openSuse 15.1 which has openssl 1.1.0i
I just tried compiling openssl 1.1.1f from source and tested that, and it does not exhibit this issue.
I guess the bug is in openssl - although I have to believe that it doesn't reproduce on native Linux.

Can't repro here, same as #2298. This run is on WSL1 19592, but that isn't going to make a difference.

although I have to believe that it doesn't reproduce on native Linux.

Impossible to verify that hypothesis absent a repro.

Appears to be SUSE specific

Given your recompile addressed, it is probably an explainable userspace thing (termcap/termio) but I can't prove that.

Sucker for the unexplained on weekends...
On Ubuntu OpenSSL 1.1.1  11 Sep 2018:
39    openat(AT_FDCWD, "myfile.txt", O_RDONLY) = 3
39    openat(AT_FDCWD, "/dev/tty", O_RDONLY) = 4
39    openat(AT_FDCWD, "/dev/tty", O_WRONLY|O_CREAT|O_TRUNC, 0666) = 5
39    ioctl(4, TCGETS, {B38400 opost isig icanon echo ...}) = 0
39    fstat(5, {st_mode=S_IFCHR|0666, st_rdev=makedev(5, 0), ...}) = 0
39    ioctl(5, TCGETS, {B38400 opost isig icanon echo ...}) = 0
39    write(5, "enter aes-256-cbc encryption pas"..., 38) = 38
[...rt_sigaction blah blah]
39    ioctl(4, TCGETS, {B38400 opost isig icanon echo ...}) = 0
39    ioctl(4, SNDCTL_TMR_START or TCSETS, {B38400 opost isig icanon -echo ...}) = 0
39    ioctl(4, TCGETS, {B38400 opost isig icanon -echo ...}) = 0
39    fstat(4, {st_mode=S_IFCHR|0666, st_rdev=makedev(5, 0), ...}) = 0
39    ioctl(4, TCGETS, {B38400 opost isig icanon -echo ...}) = 0
39    read(4, "password\n", 4096)       = 9

SUSE, my run OpenSSL 1.1.0i-fips  14 Aug 2018:
99    openat(AT_FDCWD, "myfile.txt", O_RDONLY) = 3
99    openat(AT_FDCWD, "/dev/tty", O_RDONLY) = 4
99    openat(AT_FDCWD, "/dev/tty", O_WRONLY|O_CREAT|O_TRUNC, 0666) = 5
99    ioctl(4, TCGETA, 0x7fb1d598c820)  = -1 ENOTTY (Inappropriate ioctl for device)
99    fstat(5, {st_mode=S_IFCHR|0666, st_rdev=makedev(5, 0), ...}) = 0
99    ioctl(5, TCGETS, {B38400 opost isig icanon echo ...}) = 0
99    write(5, "enter aes-256-cbc encryption pas"..., 38) = 38
[...rt_sigaction blah blah]
99    fstat(4, {st_mode=S_IFCHR|0666, st_rdev=makedev(5, 0), ...}) = 0
99    ioctl(4, TCGETS, {B38400 opost isig icanon echo ...}) = 0
99    read(4, "password\n", 4096)       = 9
99    write(5, "\n", 1)                 = 1

The latter is sending some ioctl even strace(1) doesn't grok. Someone super motivated would need to look at where that 0x7fb1d598c820 is coming from by building openssl 1.1.0i from source.
Your strace(1) log (always appreciated) for completeness. Oddly (and unexplained), your run is missing a write(2) of the string "enter aes-256-cbc encryption password" entirely.
openat(AT_FDCWD, "myfile.txt.enc", O_RDONLY) = 3
fstat(0, {st_mode=S_IFCHR|0660, st_rdev=makedev(4, 3), ...}) = 0
ioctl(0, TCGETS, {B38400 opost isig icanon echo ...}) = 0
read(0, "secret123\n", 4096)            = 10
openat(AT_FDCWD, "myfile.txt.new", O_WRONLY|O_CREAT|O_TRUNC, 0666) = 4
fstat(3, {st_mode=S_IFREG|0666, st_size=45, ...}) = 0
read(3, "U2FsdGVkX1+bChwZlGbokrhjBr6/ngLy"..., 4096) = 45
read(3, "", 4096)                       = 0
fstat(4, {st_mode=S_IFREG|0666, st_size=0, ...}) = 0
write(4, "Hello World\n", 12)           = 12

The ioctl right before the read that returns password differs: -echo (turn echo off) on the first, echo (keep echo on) on the second.


OSError: Cannot parse file b'./saved_model.pb': Error parsing message.

after download all the file and trained model/weights manually,  I run the transnetv2.py in the inference, but raise an  exception:
OSError: Cannot parse file b'/path/to/saved_model.pb': Error parsing message.
the version of tensorflow is v2.1.0.
and I also run the docker command to build an image flow the readme.md in inference, but after I built the dockerfile successfully, I run the command in the readme.md to test a video, but still raise the same exception.
how can I solve this problem? thanks

Can you provide sha256 of the file /path/to/saved_model.pb?
Do it by sha256sum /path/to/saved_model.pb.
Btw, look at issue #1.

I check the saved_model.pb carefully, and I found it's only 1.6M, smaller than 5.66M showed in the git. so I download it manually and tried again, finally it works!
thanks a lot for your help!


Fix Mellanox plugin logging

This commit fixed an issue introduced during logging refactoring [1].
[1] c155c82c0fc9a3e2db91ea8887a0d3b4ae359bc1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 25.464%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/plugins/mellanox/mellanox_plugin.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2334

Relevant Lines:
9166

💛  - Coveralls

fixed in #543


modification of http headers to avoid detection by WAF

Hello!
I had to modify the headers to bypass a WAF, by imitating part of Chrome's headers.

Hi,
I added a new option that you can use passing  --script-args=browser to the nmap script to act as a browser as you proposed. I'd like to keep the default behavior as it is by being transparent it's a scanner.
commit: https://github.com/righel/ms-exchange-version-nse/commit/e2d54b9cf5c7026a0afbc629cf86fddb8f083a2c
Hope that helps.


Create and export react component

It would be benefical to have a <MarkerClusterer map={map}> component. Need to consider if a prop of markers would be better or a child <Marker> components.

@jpoehnelt is there generally a possibility now to use this lib with react? I tried but Im not sure how to handle Marker[] type that is expected..


DNM: testing a fix for #41764

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Please test with following PRs:
apple/swift-installer-scripts#89
@swift-ci please build toolchain Windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/89
@swift-ci please build toolchain Windows platform

https://github.com/apple/swift-installer-scripts/pull/89
@swift-ci please test

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/89
@swift-ci please build toolchain Windows platform


Padrino Sprockets (before v1.0)

Just a reminder to have a gem or plugin that supports a Padrino Sprockets before our 1.0 stable release.
Reference: https://github.com/padrino/padrino-contrib/issues/2
Resources:

http://blog.envylabs.com/2011/08/using-the-asset-pipeline-outside-of-rails-serving-and-running-coffeescript-2/
https://github.com/kelredd/rack-sprockets
https://github.com/stevehodgkiss/sinatra-asset-pipeline

Ok I am going to close this, on account of https://github.com/bookworm/padrino-sprockets is a good enough starting point. Thanks @nightsailer and @bookworm


sap_vm_provision: allow Private DNS on MS Azure to use Auto Registration

Allow Private DNS on MS Azure to use Auto Registration, append new variable and logic sap_vm_provision_msazure_private_dns_auto_register that would skip the DNS Record entries created for the VM (but still append the DNS Record entries for HA Virtual IP).

NOTE:
At this time, the code will not be altered to allow no Private DNS to be specified. it remains best practice to:

use the DNS Zone record (s4hana.myorg.cloud) for the inter-Cloud communications
use the Corporate DNS Server’s Subzone record (s4hana.myorg.corp) for the internal traffic
use the Corporate DNS Server’s Subdomain Delegation record (s4hana-webdisp.myorg.com) for the public internet traffic

The sap_vm_provision attempts to keep as much homogeneity as possible for Infrastructure Platforms, and each Cloud Service Provider has a Private DNS in their designs for SAP Landscapes.

Resolved in PR #17 by using a check whether auto registration is enabled, which will skip the DNS Records creation (not the DNS Records for HA)


Feature: Implement Google Maps Integration

Added Google Maps API integration for searching and displaying places on
a map.
Implemented searchPlaces and plotPlacesOnMap functionalities.
Integrate Google Maps API

LGTM ✅ 🗺


Inconsistent verification results

There seems to be some problem in CVMFS or the network that is causing the shoal server to get inconsistent results from the same proxies. Shoal-server will occasionally fail verification on a given squid only to pass the test a few minutes later. The reoccurring failures seem to mostly happen on the Taiwan url and occasionally on the fermilab one. The request for the file times out and shoal-server subsequently blacklists whatever squid it is testing. I managed to do quite a bit of testing yesterday when it was happening frequently and nothing i did with shoal seemed to change the results even when i set an especially long timeout window.
After talking with Ron I am going to develop some code that will attempt to detect these types of failures and attempt to verify it again on a shorter interval than the one set in the verification thread (which is default 1 hour).

We should not worry about dealing with inconsistent results from the stratum one. If a server is not responding regularly then we need to take it out of the list we check, and only check the consistent servers.

All the Stratum 1 servers should be reliable. If there are intermittent problems with them, we
should report them. If it's a network problem, then it would be harder to track down.
-rt
Ryan Taylor
Grid & Cloud Computing Specialist
Data Centre Services, University Systems
University of Victoria
On 04/21/2015 10:17 AM, Ian Gable wrote:

We should not worry about dealing with inconsistent results from the stratum one. If a server is
not responding regularly then we need to take it out of the list we check, and only check the
consistent servers.
—
Reply to this email directly or view it on GitHub
https://github.com/hep-gc/shoal/issues/106#issuecomment-94877693.

I'm not sure who to report to, but these are the ones that are failing most often:
# "http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/atlas.cern.ch/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/atlas-condb.cern.ch/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/sft.cern.ch/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/grid.cern.ch/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/opt/atlas/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/opt/atlas-condb/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/opt/sft/.cvmfswhitelist",
# "http://cvmfs02.grid.sinica.edu.tw:8000/opt/grid/.cvmfswhitelist"

I did some googling and found 3 email addresses here for ASGC:
http://www.twgrid.org/en/index.php?option=com_content&task=view&id=182&Itemid=291
-rt
Ryan Taylor
Grid & Cloud Computing Specialist
Data Centre Services, University Systems
University of Victoria
On 04/21/2015 12:39 PM, Colson Driemel wrote:

I'm not sure who to report to, but these are the ones that are failing most often:
|# "http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/atlas.cern.ch/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/atlas-condb.cern.ch/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/sft.cern.ch/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/cvmfs/grid.cern.ch/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/opt/atlas/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/opt/atlas-condb/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/opt/sft/.cvmfswhitelist",
"http://cvmfs02.grid.sinica.edu.tw:8000/opt/grid/.cvmfswhitelist"
|
—
Reply to this email directly or view it on GitHub
https://github.com/hep-gc/shoal/issues/106#issuecomment-94915496.

#114 is probably a better way to handle this. I think we can close this.