osx build and linux. newbie and samples/examples?

Required Info

Camera Model
R200

Firmware Version

Operating System & Version
osx yosemite, and raspbian jessie

Kernel Version (Linux Only)
jessie

Build System
VS2013, XCode or Makefile

Hey guys. I'm excited to see the opportunity to use this R200 with my raspberry pi or my osx box but I'm at a total loss for how to

Build for my respective OSes, OSX or Ubuntu Jesse (albeit ARM for Raspberry pi)
Any sample code after it's built, or documentation for how to incorporate or use this? I'd like to in OSX app and/or Unity. Even if it's a point cloud.
I'm sorry if I missed it in the docs. I'm good with linux but not so much with C#, objective-C  and xcode direct builds. Where is the Xcode build file ? I got through to the install of on mac GLFW3.
I'm still waiting for my Raspi box though might try a qeumu Docker box to test this but I'm assuming it's just the usual make and make install as directions. I might have ARM specific issues though.

I have kinect One and 360s working fine on my mac and PC and workign well with point clouds on unity at the basics..
I can't seem to find any examples or sample code for integrating on mac or linux. I may have missed it since I'm a newbie though. I saw the ROS and Arch but don't know how to use that.
A sample openni2 or other code just to init camera, some available methods and and if there is any access to the higher level gesture etc options. I don't think those are implemented yet on librealsense as per the Readme. I'm guessing the basic point clouds et all are there.
Any help, samples and examples would be a huge help and I don't mind writing a tutorial or build documentations for OSX (best platform) and Raspi platform when I get them working.
We are hoping to use these as the basis of augmented reality apps with Unity but without windows if possible.
Thanks,
David

Hi @prussiap. Wow! So many questions! We don't really have the infrastructure to support "Getting Started" type tutorials with librealsense, yet, nor any off-the-shelf sample code for game engine integration. We do provide XCode projects for building on OSX and both Makefile and QTCreator projects for building on Linux. ARM support has been provided on other forks of this repository if you search for the list of forks or dig into closed issues on the tracker. Librealsense will not work on RPi because all of our cameras are USB 3 devices and no RPi board currently has a USB 3 host controller.
The RealSense SDK is the place to get started on Windows for augmented reality apps. I must say -- unfortunately -- that this library might be a lower-level API than you are looking for.

Hi @ddiakopoulos thanks for the responses :). Yep lots of questions but I couldn't find answers in the issues or Readme or the Intel SDK site so I thought i'd ask here and see where that got us. As for a "Getting Started" post I can write something introductory once I myself figure it out but I have zero idea where to begin now. Maybe this sucker for Xcode ? https://github.com/IntelRealSense/librealsense/tree/master/librealsense.xc/c-tutorial-1-depth.xcodeproj AFTER i make the Xcode libsense binaries are made.
This library seems to be linked from the intel site (but not supported by them) and according to Readme it builds and makes on all the OSes as long as you install the dep debs or files. I saw the Makefile I think but could I get a filename for Xcode starter ? I am Xcode/Xamarin/Depth Noobs' here. My guess is these first https://github.com/IntelRealSense/librealsense/tree/master/librealsense.xc BUT how do I import and make and what do I expect to see when done? Either using Xamarin or Xcode as necessary.
I don't mind working my way to what a tutorial would have if I can get the installs working. So then assume we build this on linux or osx and library is compiled and made (also make installed?). Then what? I realize this is Alphaish but i'm stuck then. Even if I have to just get point clouds and write my own code I need to know what methods are exposed and just some of the basics! Again sorry for the dumb and ignorant questions here but this is new to me. The RealSense R200 camera seems to be at a fantastic price point.. and if we could get past the Windows only sdk and drivers I believe this product would make waves.
We've been making "holograms" with Kinect 2 and Unity as well as some face specific stuff with Processing. The other camera I'm looking at is Structure SDK which seems to have openNi2 code and some low level libraries as well for non Windows. I do need to get this working so we can explore what is available and if I have to return the Realsense for lack of software developper community help. It's not the libs fault it seems that Intel is keen on Windows only support and even that seems roundabout with lots of circular linking :)
In any case if anybody has sample code that you've worked on that can be shared as an example? Even the super basic. init camera, load point cloud etc.. I would be GREATLY appreciative and I hope that it will be possible to build something with this library especially with the 100$ R200 price point for this depth camera and USB power which is unheard of pricing.

Hi @prussiap this repository already has everything you need!... you seem a bit confused about what this library does. librealsense will enable you to use all current generations of RealSense cameras on Windows, OSX, and Linux with nearly no dependencies outside of libusb (OSX and Linux only) and GLFW (all platforms, for showing GL windows).
On OSX, XCode will happily open this file: https://github.com/IntelRealSense/librealsense/tree/master/librealsense.xc/librealsense.xcworkspace
It contains the reusable library (librealsense proper) and a smattering of example applications. You can use a dropdown menu at the top of the XCode project to select, for example, the "cpp-pointcloud" application. Hitting build on this will automatically build + link librealsense into the app and immediately show you the camera data on launch!
The entire API is documented in a single header file here: https://github.com/IntelRealSense/librealsense/blob/master/include/librealsense/rs.hpp
After following the short installation instructions for using brew to install the libusb and GLFW dependencies, literally the only required action on your part is to hit build in the OSX project to see the camera data.

Closing issue; will still respond if you are having problems.

Hi @ddiakopoulos, I tried to run the sample project, for example, "cpp-pointcloud" as you mentioned above. I could build the project on xcode (8.2.1) but when I tried to run the project, it didn't show anything (e.g. the display of rgb image), nor any error messages.
Could you let me know how to run the project? Are there anything I can check (e.g. connection of usb, the installation of libusb and glfw3)?

Thank you. When I ran brew install homebrew/versions/glfw3, the output said the followings.
Warning: Use glfw instead of deprecated homebrew/versions/glfw3
Warning: glfw-3.2.1 already installed

So, do I need to run brew install homebrew/versions/glfw3 as well as unlink glfw3?

On my machine, after receiving the same message that you have, I cannot build the cpp-pointcloud project because it is looking for glfw3 which is not installed - I get a failure in the link step.  If you build successfully then your setup is different from mine.
Do you have an up-to-date checkout of librealsense?  If you clean and then build the cpp-pointcloud project, do you get a binary?  Do you have /usr/local/lib/libglfw3.dylib?

Thank you very much.

Do you have an up-to-date checkout of librealsense?
I git clone librealsense a few days ago.

If you clean and then build the cpp-pointcloud project, do you get a binary?
When I clean and build, build succeeds and it doesn't show any error message. However, it doesn't show pointcloud display either.
Do you have /usr/local/lib/libglfw3.dylib?
Not libglfw3.dylib, but I have libglfw.3.2.dylib, libglfw.3.dylib and libglfw.dylib under  /usr/local/lib/.


Implemented page.emulate tests

closes #44
closes #35
this will require implementing the ElementHandle class

I think this will implement just the click method in the ElementHandle class to keep it simple.
I'm debating whether to merge #123 into this since they both depend on each other

@kblok ready

Some small comments, but it's almost there


Add tags

Code for adding tags.
There is still a bug on the local client side when deleting tags. If the client views the same article right after saving changes, the deleted tags still appear as the reside in client cache. Otherwise, seems to work fine though tests are needed.
This PR is quite huge, but most of the code hooks up tags with the database. The rest is more or less a copy of how staff is handled in the ArticleController. Part of #306.

@emilgoldsmith corrections implemented and mer conflicts resolved. See my one comment on the falcor call in the falcor calls :). Thanks for the feedback, it is invaluable!

@emilgoldsmith comments have been implemented!


No NUCLEAR_MISSILE function

There is a serious flaw in Knight: the absence of a NUCLEAR_MISSILE function. There are many reasons why this is detrimental to a coherent user experience conducive to rapid and safe development.

N is not taken
The letter N is free for new built-in functions

Due to these many reasons and a plethora more, Knight should definitely add the NUCLEAR_MISSILE function.

It has come to my attention that NULL exists. I propose NULL to be replaced with this.

I strongly support this proposition.
Not to mention the very presence of NULL sounds like a flaw, while every want to be dictator would for sure adopt a language with a NUCLEAR_MISSILE function.

I don't think most users have a use for nuclear missles.
That being said, the X function does exist for a reason—maybe make an XTREME_NUCLEAR_MISSILE?

I cannot believe you fail to see how essential this facility is to a complete language. I'd say that without a nuclear missile function, Knight cannot be considered a real language.


IndyBootstrapDispatcher cannot be properly loaded in the Resin environment

Describe the bug
When starting Resin 3 using the provided startup script ($RESIN_HOME/bin/httpd.sh), the runtime is unable to load java.lang.IndyBootstrapDispatcher via the DynamicClassLoader.
After some investigation, I found that the startup script ultimately calls the com.caucho.boot.WatchdogProcess#createProcess method (located in resin.jar) to launch Resin. This method hardcodes the startup argument -Djava.system.class.loader=com.caucho.loader.SystemClassLoader, which replaces the default system class loader.
As a result, when Resin tries to load IndyBootstrapDispatcher, the DynamicClassLoader delegates the loading to SystemClassLoader, which is unable to load it. In Resin 3, the resin.conf file provides a tree-node element to configure special JAR file paths, allowing SystemClassLoader to access the target files. However, the internal implementation is hardcoded to fetch the target entry from the JAR file, like configuredJars.getJarEntry("java/lang/IndyBootstrapDispatcher.class").
The issue is that Elastic has shaded the class, and its actual resource path is represented as agent/bootstrap/java/lang/IndyBootstrapDispatcher.esclazz. This mismatch causes the class loading to fail, leading to malfunctioning agent functionality.
All the above is specific to Resin 3. Resin 4 might have the same problem, but I haven't tested it yet.
Steps to reproduce

In the resin.conf file, add the following configuration

<jvm-arg>-javaagent:/path/to/elastic-apm-agent.jar</jvm-arg>
<jvm-arg>-Delastic.apm.log_level=DEBUG</jvm-arg>
<jvm-arg>-Delastic.apm.log_file=/path/to/logs/elastic-apm-agent.log</jvm-arg>

download the vulns.jar to $RESIN_HOME/webapps
RUN $RESIN_HOME/bin/httpd.sh start
Access http://localhost:7040/vulns through your browser

IndyBootstrapDispatcher  is in fact not supposed to be loaded by the system-classloader. It is injected by the agent into the bootstrap-classlaoder. We specifically shade it to agent/bootstrap/java/lang/IndyBootstrapDispatcher.esclazz so that system-classloaders don't find it and then normally should delegate to the bootstrap classloader.
I tried reproducing with your instructions, unfortunately I couldn't get resin to work quickly (likely due to MacOs / some env requirements). It would be best if you could provide a github repo with a dockerfile as a self contained and portable way to reproduce.
Alternatively we could start out by you provding us the logs here, maybe this already helps me spot the issue.

IndyBootstrapDispatcher  is in fact not supposed to be loaded by the system-classloader.

Yes, but when I mentioned the system-classloader, I was referring to Resin's specific implementation. It doesn't seem to follow the parent delegation principle. As a result, when it attempts to load IndyBootstrapDispatcher, it doesn't delegate to the bootstrap-classloader, which ultimately causes the class loading to fail.
I’ve created a Dockerfile to reproduce this issue, and you can download it directly for testing.
dockerfile.zip
When the /opt/resin3/resin-3.1.16/log/jvm-default.log file appears inside the container and contains the message Resin started in xxms, it indicates that Resin has started successfully. At this point, you can use tail -f to continuously monitor the log file's output. Then, visit localhost:8080/vulns in your browser to observe the file's output. Eventually, you will see the error message:
java.lang.BootstrapMethodError: java.lang.NoClassDefFoundError: java/lang/IndyBootstrapDispatcher.
Here’s the log output I observed in my environment (in case it helps):
java.lang.BootstrapMethodError: java.lang.NoClassDefFoundError: java/lang/IndyBootstrapDispatcher
	at _jsp._index__jsp.init(_index__jsp.java:80)
	at com.caucho.jsp.JspManager.loadPage(JspManager.java:404)
	at com.caucho.jsp.JspManager.compile(JspManager.java:247)
	at com.caucho.jsp.JspManager.createPage(JspManager.java:171)
	at com.caucho.jsp.JspManager.createPage(JspManager.java:150)
	at com.caucho.jsp.PageManager.getPage(PageManager.java:248)
	at com.caucho.jsp.PageManager.getPage(PageManager.java:178)
	at com.caucho.jsp.PageManager.getPage(PageManager.java:161)
	at com.caucho.jsp.QServlet.getSubPage(QServlet.java:295)
	at com.caucho.jsp.QServlet.getPage(QServlet.java:210)
	at com.caucho.server.dispatch.PageFilterChain.compilePage(PageFilterChain.java:238)
	at com.caucho.server.dispatch.PageFilterChain.doFilter(PageFilterChain.java:145)
	at com.caucho.server.webapp.WebAppFilterChain.doFilter(WebAppFilterChain.java:187)
	at com.caucho.server.dispatch.ServletInvocation.service(ServletInvocation.java:265)
	at com.caucho.server.http.HttpRequest.handleRequest(HttpRequest.java:273)
	at com.caucho.server.port.TcpConnection.run(TcpConnection.java:682)
	at com.caucho.util.ThreadPool$Item.runTasks(ThreadPool.java:743)
	at com.caucho.util.ThreadPool$Item.run(ThreadPool.java:662)
	at java.lang.Thread.run(Thread.java:750)
Caused by: java.lang.NoClassDefFoundError: java/lang/IndyBootstrapDispatcher
	... 19 more
Caused by: java.lang.ClassNotFoundException: java.lang.IndyBootstrapDispatcher in DynamicClassLoader[SimpleLoader[/opt/resin3/resin-3.1.16/webapps/vulns/WEB-INF/work]]
	at com.caucho.loader.DynamicClassLoader.loadClass(DynamicClassLoader.java:1224)
	at com.caucho.loader.DynamicClassLoader.loadClass(DynamicClassLoader.java:1203)
	... 19 more

In Resin version 3.1.16, has the same problem ,but 4.0.67 is ok
[11:26:45.222] {http--8080-6$57877721}  java.lang.NoClassDefFoundError: java/bridge/Bridge
[11:26:45.222] {http--8080-6$57877721} 	at org.springframework.expression.common.TemplateAwareExpressionParser.parseExpression(TemplateAwareExpressionParser.java:52)
[11:26:45.222] {http--8080-6$57877721} 	at org.springframework.expression.common.TemplateAwareExpressionParser.parseExpression(TemplateAwareExpressionParser.java:43)
[11:26:45.222] {http--8080-6$57877721} 	at _jsp._index__jsp._jspService(_index__jsp.java:47)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.jsp.JavaPage.service(JavaPage.java:61)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.jsp.Page.pageservice(Page.java:578)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.server.dispatch.PageFilterChain.doFilter(PageFilterChain.java:195)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.server.webapp.WebAppFilterChain.doFilter(WebAppFilterChain.java:187)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.server.dispatch.ServletInvocation.service(ServletInvocation.java:265)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.server.http.HttpRequest.handleRequest(HttpRequest.java:273)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.server.port.TcpConnection.run(TcpConnection.java:682)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.util.ThreadPool$Item.runTasks(ThreadPool.java:743)
[11:26:45.222] {http--8080-6$57877721} 	at com.caucho.util.ThreadPool$Item.run(ThreadPool.java:662)
[11:26:45.222] {http--8080-6$57877721} 	at java.lang.Thread.run(Thread.java:748)


[Bug]: : Error: spawn ENAMETOOLONG

Describe the bug
When I start Obsidian, git doesn't commit anymore.
Instead I get an error.
Relevant errors (if available) from notifications or console (CTRL+SHIFT+I)
git: Everything is up-to-date
plugin:obsidian-git:45237 git obsidian error: Error: spawn ENAMETOOLONG
plugin:obsidian-git:26944 Uncaught (in promise) Error: Error: spawn ENAMETOOLONG
    at GitExecutorChain.onFatalException (plugin:obsidian-git:26944:79)
    at GitExecutorChain.eval (plugin:obsidian-git:26936:24)
    at Generator.throw (<anonymous>)
    at rejected (plugin:obsidian-git:25805:29)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Steps to reproduce
start Obsidian
Expected Behavior
create a git commit
then a git push
Addition context
No response
Operating system
Windows
Installation Method
None
Plugin version
2.24.1

i manually commited my files, let's see if this error comes back.
it's gone for now.

seems to commit fine now

Great it's solved. Same as #238
A solution is probably to set the commit message not via command line argument, but per an extra input file.

it's solved for me, which is great. 🥳
what I'm concerned about: I rely on Obsidian Git to auto backup my notes, which the plugin failed to do.
I can close this issue, since my problem is solved, but the problem wont be prevented in the future for anyone else.
Recently I've been manually checking git weekly, since a merge issue lead to my notes not being backed up for 4 months 😱
I wouldn't have noticed this bug otherwise.
Git obsidian has otherwise been working fine for auto back up for the last 2+ years.

Encountered the same issue with the Iconize plugin. Deleting the pre-defined icon packs and adding my local icon pack caused many file changes. Fixed it by adding the icon folder to .gitignore.
If it's applicable and you have bulk of redundant files, try doing this.

I limit now the amount of files to list in the commit message. This should solve this issue.
a0416edf5f3ac5d9adc9fc37cf9a9c932583ced4


Protobuf reference page refers to non existant types.

For example, https://developers.google.com/protocol-buffers/docs/reference/google.protobuf#google.protobuf.Enum
This doesn't seem to exist anywhere in the code base.

Found that it was in the "type" file. Perhaps adding an annotation to the docs would be helpful?

Are you referring to this repo's README.md page or the Protobuf developers guide?
If former, I don't see any mention of it in README.md.  If latter, you can try to send request to<EMAIL_ADDRESS>

Sure, will send to group.


Invalid PKG-INFO when using reStructuredText README

[x] I have searched the issue tracker and believe that this is not a duplicate.

Steps to reproduce
I have a project with a README.rst file. When building the package using pdm build -v, I get this output:
Preparing isolated env for PEP 517 build...
Building sdist...
Collecting pdm-pep517
  Using cached pdm_pep517-0.5.8-py3-none-any.whl (146 kB)
Installing collected packages: pdm-pep517
Successfully installed pdm-pep517-0.5.8
 - Adding LICENSE.rst
 - Adding README.rst
 - Adding assertion/__init__.py
 - Adding assertion/assertion.py
 - Adding assertion/deco.py
 - Adding pyproject.toml
 - Adding PKG-INFO
Built sdist at /home/tom/proj/assertion/dist/assertion-0.1.6.tar.gz
Building wheel...
Collecting pdm-pep517
  Using cached pdm_pep517-0.5.8-py3-none-any.whl (146 kB)
Installing collected packages: pdm-pep517
Successfully installed pdm-pep517-0.5.8
 - Adding assertion/__init__.py
 - Adding assertion/assertion.py
 - Adding assertion/deco.py
 - Adding assertion-0.1.6.dist-info/WHEEL
 - Adding assertion-0.1.6.dist-info/METADATA
 - Adding assertion-0.1.6.dist-info/LICENSE.rst
 - Adding assertion-0.1.6.dist-info/RECORD
Built wheel at /home/tom/proj/assertion/dist/assertion-0.1.6-py3-none-any.whl

Everything correct, right?
Twine, however, is not happy:
pdm run twine check dist/*
Checking dist/assertion-0.1.6-py3-none-any.whl: PASSED
Checking dist/assertion-0.1.6.tar.gz: FAILED
  `long_description` has syntax errors in markup and would not be rendered on PyPI.
    line 13: Severe: Unexpected section title or transition.

When I extract the PKG-INFO file, it looks like this:
...
Project-URL: Repository, https://gitlab.com/coherentminds/assertion/
Description-Content-Type: text/x-rst
Description: .. image:: https://img.shields.io/pypi/v/assertion.svg
            :target: https://pypi.org/project/assertion/
            :alt: pypi version
...

Maybe the indentation of the long description breaks the reStructuredText markup.
Environment Information
PDM version:        1.4.2
Python Interpreter: /usr/bin/python3 (3.8.8)
Project Root:       /home/tom/proj/assertion
{
  "implementation_name": "cpython",
  "implementation_version": "3.8.8",
  "os_name": "posix",
  "platform_machine": "x86_64",
  "platform_release": "5.11.6-1-default",
  "platform_system": "Linux",
  "platform_version": "#1 SMP Thu Mar 11 16:11:36 UTC 2021 (7358b30)",
  "python_full_version": "3.8.8",
  "platform_python_implementaiton": "CPython",
  "python_version": "3.8",
  "sys_platform": "linux"
}

Let's track it in pdm-pep517's issue tracker, close this one. The pdm project will also transfer to pdm-project workspace to allow the tranfer.


Fixing Typo - removing static in build command

Description

Linked issue

Type of change

[ ] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🛠 New feature (non-breaking change which adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[ ] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[ ] I have read the style guide document, and my code follows the code style of this project.
[ ] If my code deals with exceptions, it follows the guidelines.
[ ] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

QA Steps

Screenshots (if available)

@jmitchell
this build command seems stale - at least from my quick skim over release.nix

@akegalj : can  I close this PR?

@jcmincke

can I close this PR?

yes.


Refactoring of DataSource component for easier, future, extensibility

Hi! I'm proposing a refactoring of folktables's DataSource component (refer to this ZIP to see the changes already implemented).
Proposed changes
In this section I describe the changes I'm proposing that will affect the structure of the DataSource component.

Creating utilities that help to (down)load different datasets: many of the functionalities already coded in the current version of folktables can be reused to (down)load different datasets (e.g., making the HTTP request to download a dataset from the U.S. Census Bureau's website, saving the content of the HTTP request into local memory, ...). Hence, I'm proposing creating utilities for these common tasks as it will allow us to reuse the code in the case that more datasets will be supported.
Added the ability to download files asynchronously.
Changing the DataSource from an ABC to a Protocol: as I see it, the role of DataSource is to define an interface for classes that help with the (down)loading of different datasets. By having DataSource be a Protocol we'll be taking advantage of Python's duck typing and will make it more straight forward that this is the interface that should be followed (plus, we have the added benefit that we do not have to directly inherit from DataSource whenever defining this component for a different dataset).
Introducing DownloadResource, LoadResource, and FilesResource: these are dataclasses that contain information pertaining to file paths and URLs for the different datasets the user wants to (down)load. These objects (note that FilesResource just contains instances of DownloadResource and LoadResource) makes it easier to keep track of the resources needed to (down)load files.
Refactored load_acs.py and the ACSDataSource class in acs.py into one file: in this refactoring I'm following the re-defined DataSource interface, I'm using the utility functions I created or refactored, and I'm taking advantage of the *Resource objects defined. One additional thing I did was to turn the hard coded URL into a "constant" variable.
Introducing custom exceptions.

Proposed file structure
folktables
|     __init__.py
|     folktables.py
|     exceptions.py
|     acs.py
|_____data_source
|     |     __init__.py
|     |     acs_data_source.py
|     |     data_source.py
|_____utils
|     |     __init__.py
|     |     download_utils.py
|     |     files_resource.py
|     |     load_utils.py

Hi Ricardo, this is a hugely valuable effort. Thanks so much!
I just had a look at the zip file that you link. This generally all looks good. There are a few higher level points to discuss and then some suggestions and minor comments.
Higher-level points:
There's a balance to strike between the generality of the code and the needs of the typical machine learning researcher. When changing the API, we should keep this in mind. A typical ML grad student should be able to create their own data sources quickly without too much of a learning curve. Otherwise instead of using folktables, researchers will go and write their own ad-hoc code.
Concrete suggestions:

I probably would've skipped the extra layer of the FilesResource abstraction, collapsing download_utils.py and load_utils.py into one.
It's nice to have asynchronous downloads. I wasn't able to tell though if this is optional from the perspective of the data source creator.  Does this mean that someone who wants to instantiate DataSource(Protocol) will have to know what the async keyword in  async def _download_data entails? If so, this might provide a bit of an extra hurdle to creating new data sources.
Is it possible to break this up into a few different pull requests so that it's easier to review each individual component?

Minor comments:

files_resources.py has documentation that mention SIPP. I assume these should be general. Likewise acs_data_source.py has a few methods were SIPP shows up in the doc string.

Paging @francesding @millerjohnp @ludwigschmidt since this touches on some major design choices. Please let us know if you have any comments or concerns.

Hi Ricardo,
Thanks for integrating SIPP, it will be a great addition to folktables!
A few points below:

As Moritz said, it would be good to make this a real pull request (as opposed to linking to a zip file). That makes it easier to see the code changes in the Github UI, comment on individual lines, etc.

Sharing code between the ACS download routines and SIPP is great. What I don't fully understand right now is how a future contributor should use these building blocks. For instance, what is the purpose of _load_data and _download_data in the DataSource interface? If these functions can be called from code external to DataSource, why do the names start with an underscore? If the functions are only for use internal to DataSource implementations, why are they part of the interface?

Asynchronous downloads are great to increase download speed if all the resulting complexity is handled in the DataSource API. I think users of DataSource should not have to know about asynchronous Python since it can be a bit complicated.

The utils folder should contain only code that is independent of a specific data source. The documentation string in files_resources.py says "Stores information needed to (down)load the SIPP dataset." Should this be in one of the SIPP files?

Re-using code is good, but there is also a cost to creating abstractions. For instance, I'm not sure if we need the load_json function in load_utils.py. The amount of code we save is only one line, and the two lines in the function for loading a JSON are relatively easy to understand. Introducing a new function makes the code harder to read for others who don't know exactly what the function is doing but would understand the two common Python lines for loading a JSON.

Hi, thanks for the proposal and initial changes! Beyond what others have mentioned, just want to understand if this is targeting a new dataset or collection of datasets? Or is there a particular part of the current interface you find confusing or annoying?
I think it'd be easier to reason about the costs/benefits of interface changes if there's actually a concrete target.  Additionally, I like the lightweight interface, and I think it only really makes sense to add a lot of additional complexity and spend dev time implementing and maintaining something it if we're actually getting a new resource or improving quality of life for users.

Hi all,
Thanks for all the feedback and comments. I'll address all of them here.
Feedback
Purpose of the proposed changes
The main idea behind the changes I've made is to keep the user facing API intact while changing the way in which the implementation is done. This is in order to make it easier to integrate other datasets in the future. Hence, the example you all list in the README would still be valid:
data_source = ACSDataSource(survey_year='2018', horizon='1-Year', survey='person')
acs_data = data_source.get_data(status=["CA"], download=True)

The way to use the SIPPDataSource is very similar, the only difference being the parameters  you have to pass to the constructor and to get_data.
I started to work on these changes when I started implementing the functionality to (down)load the SIPP dataset. I found a lot of the existing code to be useful but couldn't access it for my purposes as it was tailored for the ACS dataset. Hence, I went ahead and refactored the existing code to make it more modular and reusable.
FilesResource
You all are right; SIPP shouldn't be mentioned at all in files_resource.py as FilesResource is agnostic of any dataset/datasource. Rather, it's just supposed to contain information that till be useful to (down)load any dataset. I'll go ahead and remove any mention of the SIPP dataset from this file.
I originally thought of having the DownloadResource and LoadResource abstractions as they help separate the behavior of the core functionalities the DataSource provides. However, I don’t mind getting rid of them and just assigning all the fields to FilesResource.
Asynchronous downloads
Yes, the idea is that all asynchronous downloads are handled internally and that the implementation details are hidden from the user. You can use the ACSDataSource without having to worry about what async and await entails. This comment made me rethink the design and I think it would probably be best to keep the DataSource objects free from any async code and push it all (if we end up deciding to do so) to utils.download_utils.py.
There is, however, a caveat with asyncio’s to_thread coroutine: it was introduced in Python's version 3.9, which means that it might limit some backwards compatibility.
DataSource’s _load_data and _download_data
My idea of including _load_data and _download_data in DataSource was to make it explicit that these are the two core functionalities that DataSource objects provide (and it also made it easier for me to divide the code into these functions). However, we can certainly remove them from this protocol class as they don’t really add much.
load_json
This is a good point, Ludwig. I can go ahead and remove this function and directly use the JSON loading code in the rest of the code.
Moving forward
Following what Moritz and Ludwig proposed, I’ll go ahead and break this up into a couple of pull requests:

I’ll start by making a pull request for the utils directory as I think this is the easiest to reason about, since it doesn’t depend on any other code in folktables and it’s the building block for future pull requests (it also won’t affect any of the code you all currently have).
Once you’ve all decided on the utils directory, I can go ahead and push the code for DataSource and the refactored ACSDataSource.
Lastly, I can push the code for the SIPPDataSource.

Please let me know what you all think about this plan and if you all have any other questions!

SGTM - Thanks so much!

Sounds great to me! Re: asynchronous downloads, it might be nice to maintain some flexibility with python versions. concurrent.futures might be one alternative to asyncio

Regarding Python versions: apart from the async part, the switch from ABC to Protocol also requires at least Python 3.8 because Protocol was introduced then. I'm not sure what our policy on Python versions should be. According to our unit tests, we currently support 3.7 onwards. Python 3.8 was released in October 2019, so it may well be that it's widely used by now and doesn't cause trouble for users of folktables. Any thoughts?

@ludwigschmidt You are right, my idea for the _load_data and _download_data methods is for them to only be called internally by DataSource objects. I'll go ahead and remove them from the externally visible interface.
I'll also go ahead and revert back to using ABC instead of protocol in order to still be able to support Python 3.7, and will switch from using asyncio to concurrent.futures.
I'll go ahead and send the first pull request shortly.

Thanks Ricardo for this work! The revised changes sound good to me.
Thanks,
Frances
On Mon, Aug 1, 2022 at 2:17 PM Ricardo Sandoval @.***>
wrote:

@ludwigschmidt https://github.com/ludwigschmidt You are right, my idea
for the _load_data and _download_data methods is for them to only be
called internally by DataSource objects. I'll go ahead and remove them
from the externally visible interface.
I'll also go ahead and revert back to using ABC instead of protocol in
order to still be able to support Python 3.7, and will switch from using
asyncio to concurrent.futures.
I'll go ahead and send the first pull request shortly.
—
Reply to this email directly, view it on GitHub
https://github.com/zykls/folktables/issues/26#issuecomment-1201731556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLSMAFFVDYDHONZZKHJEE3VXA5HVANCNFSM543MMQ3Q
.
You are receiving this because you were assigned.Message ID:
@.***>


downgrade surefire plugin to 3.0.0-M3

@flbruto do you remember this issue from our upgrade attempt long ago?

FWIW - M4 works fine, it's just M5 that introduces the regression. I think it could be related to this, I tried with both Legacy and Surefire ForkNodeFactory implementations but neither seemed to make a difference: https://maven.apache.org/surefire/maven-surefire-plugin/examples/process-communication.html


Crashes before shutting down, sometimes randomly

The app crashes/doesn't respond every time I'm trying to shut down my MacBook Pro 2019, thus preventing the system from shutting down. I have to force quit the app to be able to turn off my computer. Sometimes also crashes randomly, resulting in not having any sound (and a loading cursor when I drag the mouse over te eqMac menu bar icon).

Hi, this seems like a duplicate of #657. I am closing this bug as duplicate. If this is still the case for you with the latest version of eqMac, please reopen this issue since the other issue mentioned is closed.


Registering a new project type using VSPS in a VS.Extensibility extension

We have an extension created with VSSDK and VSPS (VisualStudio Project System). VSPS is used for registering a new project type. We want to migrate the project registration type utility to a new extension built with the VS.Extensibility framework.
Throughout this text, I will be referring to the contents of this document: https://learn.microsoft.com/en-us/visualstudio/extensibility/visualstudio.extensibility/get-started/in-proc-extensions?view=vs-2022#add-a-visualstudioextensibility-extension-to-an-existing-vssdk-extension-project
By following the docs, I've set up a project for the hybrid VSSDK and VS.Extensibility extension, but soon I realized I didn't know how to proceed next.
From my understanding, there are two models the hybrid extension can be built with:
a) Having VSSDK as the entry point - in this case, there is a package class used as the entry point
b) Having VS.Extensibility as the entry point - in this case, the entry point is the class with the VisualStudioContributesAttribute derived from the Extension class.
We want to build our extension primarily with the VS.Extensibility framework, so the entrypoint will be created according to model "b" mentioned above.
In the docs, there are two ways mentioned for consuming VSSDK: a) MefInjection; b) AsyncServiceProviderInjection. However, I'm not sure that either of these is applicable in our case, because for project type registration, the only thing you have to do is define a class decorated with the ProjectTypeRegistrationAttribute.
For better understanding, here is the code we use for project type registration:
using System.ComponentModel.Composition;

using Microsoft.VisualStudio.ProjectSystem;
using Microsoft.VisualStudio.ProjectSystem.VS;

[Export]
[AppliesTo(ProjectCapability)]
[ProjectTypeRegistration(
    projectTypeGuid: VSSDKPackage.ProjectTypeGuid,
    displayName: "SystemTest Package",
    displayProjectFileExtensions: "SysTestPack",
    defaultProjectExtension: ProjectExtension,
    language: ProjectLanguage,
    PossibleProjectExtensions = ProjectExtension,
    DisplayProjectTypeVsTemplate = "SystemTestPackage",
    ProjectTemplatesDir = "SystemTestPackageTemplate")]
internal sealed class PackageProjectUnconfigured
{
    public const string ProjectExtension = "systestpack";

    public const string ProjectLanguage = "SystemTest";

    public const string ProjectCapability = "NewSystemTestPackage";
}

My questions are:
a) How do I use the project type registration utility in the VS.Extensibility extension?
b) Also, I don't understand the "container" project mentioned in the docs. Do I have to create a special project that references the hybrid VSSDK and VS.Extensibility extension project?
c) In the future, are you planning to integrate the capabilities of VSPS (e.g. project type registration) into the VS.Extensibility framework?

@tinaschrepfer
Thank you for your reply. The docs looks clear now, and I'm glad that I can now have both ExtensionEntrypoint (VS.Extensibility) and PackageClass (VSSDK) in the same project and the final extension will be constructed from both models at the same time. I have to say: "Good job, the VS.Extensibility team!" :)
I'll try creating my extension following the steps from the docs, and hope it will work (if any issues pop up, I'll update this thread).

Great to hear your issue is resolve!


Upgrade to qiskit-sphinx-theme 2.0.0

This upgrades to qiskit-sphinx-theme 2.0.0. It includes the Qiskit logo because the logo is no longer included in the theme.
qiskit-sphinx-theme 2.0 supports dark mode as a new feature, so both light and dark versions of the logo are included.
This implements the same change as the PR at
https://github.com/Qiskit/qiskit-serverless/pull/1481.

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

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

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
2408

Relevant Lines:
2408

💛  - Coveralls


No proper description in the Contact us section.

I would like to add some proper text in the contact us section  in the place of the lorem ipsum

@SaiSantosh2209 Please assign this to me and add the hactktoberfest labels

@SaiSantosh2209 Go ahead
Star the repo 🌟 to show your love

@acenasir Thank you for our interest
Plz create new issues or look for other issues.
Star the repo 🌟 to show your support.

Hey @Arun9739 please merge my PR

@Arun9739  please review my PR


cups service: fix printing for chrome

Motivation for this change
Closes https://github.com/NixOS/nixpkgs/issues/34963
Also I'm not sure about original Listen /var/run/cups/cups.sock. Who creates /var/run/cups folder?
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

That's strange; at least on Ubuntu socket is placed there too. Not sure about "who creates it" part. Either way, why would our Chromium use /var/run/cups.sock instead?

@abbradar

That's strange; at least on Ubuntu socket is placed there too

Right.

Not sure about "who creates it" part

I mean /var/run/cups folder. I've once got a situation where CUPS wouldn't create /var/run/cups/cups.sock socket because /var/run/cups folder was not exist.

Either way, why would our Chromium use /var/run/cups.sock instead?

Not only Chromium. I've straced Firefox too, and by some reason it only looks for /var/run/cups.sock too: https://github.com/NixOS/nixpkgs/issues/34963 Maybe something is broken only on my system?

Yep, found it -- we indeed have a broken configuration in CUPS:
#define CUPS_STATEDIR "/run"
#define CUPS_DEFAULT_DOMAINSOCKET "/run/cups.sock"

I'd say rather than fixing NixOS we should fix this in CUPS to give it a separate directory in /run. It can be created using systemd's RuntimeDirectories.

@abbradar Cool, thanks for finding the root of issue!


add passing build stamp in README

use GitHub actions for continuous integration (with a passing build stamp in the README)

'use GitHub actions for continuous integration (with a passing build stamp in the README)'


Task23

Data manipulation

@bhavi98
Task -23
Code verified.
Task -24
Code verified.
Task -25
Code verified.
Task -26
Code verified.


Revert "Default Keyboard ScrollActions with PrimaryScrollController"

Reverts flutter/flutter#69795
device lab microbenchmarks failing consistently after this change.

Landing revert on red to green the tree.


chore(package): update babel-preset-env to version 1.5.1

Closes #141

Codecov Report

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

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   27.48%   27.48%
=======================================
  Files          11       11
  Lines         131      131
  Branches       22       22
=======================================
  Hits           36       36
  Misses         93       93
  Partials        2        2

Continue to review full report at Codecov.

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


deconz: Bump deCONZ to 2.05.77

Bump deCONZ to 2.05.77
Release notes: https://github.com/dresden-elektronik/deconz-rest-plugin/releases/tag/V2_05_77

Please don't merge yet.
Binary still not available in the deconz repo (http://deconz.dresden-elektronik.de/).

Converted PR to draft, until binaries are available and you can test the PR.

@frenck Binary available and tested.
Please proceed with the review.
Thanks in advance.


explanation of the role of activeDeadlineSeconds in initContainers be…

explanation of the role of activeDeadlineSeconds in initContainers
Refs: https://github.com/kubernetes/website/issues/29324#issuecomment-927525917
fixes https://github.com/kubernetes/website/issues/29324

/assign  @s-kawamura-w664

/lgtm

/cc @s-kawamura-w664

/lgtm

Thanks for the reviews!! looking for approve.

/assign @kbhawkey

The additional update is look good to me.
Please look for approve.
/lgtm

looking for approve.

Thank you for your contribution, @RA489!
/approve


Tests?

I don't see any unit / integration tests. Do you expect to add some?
Regards
Stephan

Hi @stephanos, sorry for late response we're working on it, actually we have a Pull Request we new changes and tests.
Thanks

sounds good :)


remove projects folder from incuna-sass

To be moved to a closed source project library
@incuna/frontend please review

This should be fine when https://github.com/incuna/incuna-epatient-web/pull/284 is finalised :)

@incuna/frontend can this be merged? It would be good to do it before #51

I'm not deleting the branch as for now - just in case if someone decides we should revert this commit:)

@maciejpi GitHub allows easy reverting of merged pull-requests even after the branch is deleted.


Games: Add a play again button

Add it to both Hangman and Guess Addon, on all endgame screens

Sure Lemme do it

thanks!


gobject-introspection: 1.60.0 -> 1.60.1

Things done
Updated the shlib patches:

Upstream now forces basename in sanitize_shlib_path(lib) if the system isn't
darwin. We patch the function to always prefer absolute paths
Due to the above we no longer need the macos patch
sanitize_shlib_path is applied after resolve_from_ldd_output so we need to
apply fallback_libpath afterwards too (else we get stuff like
<EMAIL_ADDRESS>on darwin)
Note that the fallback_path no longer have any unit tests. To fix this it might be better making sanitize_shlib_path take  fallback_path as an argument and return a sanitizer which applies it, though not sure if it's a big deal.

https://gitlab.gnome.org/GNOME/gobject-introspection/blob/1.60.1/NEWS

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build gobject-introspection

Maybe macOS now has the dependencies in cache?

IIRC the builds finished successfully when targeting master.

I did a rebuild of my system using this and the glib update, haven't seen any issues. Pretty confident this is good to go.


Added Secondary Champion Option

I just want to note that this will not save your pick after you reopen the app but otherwise 👍


Improve from skill and knowledge elements across GUI

closes : https://github.com/instructlab/ui/issues/149

I messed up the PR, I will open a new one


[Relay][Bug] Type unification bug (when unifying quantified functions)

PR #3559 added the ability to unify functions with type arguments as part of allowing for quantified local functions (needed for continuation-passing style, if I understand correctly). However, unification with quantified functions appears to fail in some cases.
This case (see test_type_solver.py) results in a unification exception:
    solver = make_solver()
    a, b, c = relay.TypeVar('a'), relay.TypeVar('b'), relay.TypeVar('c')

    ft1 = relay.FuncType([a, relay.TupleType([b, c])], relay.TupleType([a, b, c]), [a, b, c])
    ft2 = relay.FuncType([a, relay.TupleType([a, c])], relay.TupleType([a, a, c]), [a, c])
    unified = solver.Unify(ft1, ft2) # crashes attempting to unify a and b
    assert unified == ft2

A case like this should work, since if we have a function f with type fun[a, b, c](a, (b, c)) -> (a, b, c) and we give it arguments of types a, a, and c, f would produce an output of type (a, a, c). However, the unification fails because (I believe) the unification rule for func types makes the assumption that the type variables of the two function types should correspond to each other by their order, which does not hold in this case (ft2 drops the middle type variable).
It is not clear how to solve this in our present approach to unification -- perhaps instantiate func types with fresh incomplete types in the func type unification case and then bind the result again over the substituted type vars? Should we be generalizing types elsewhere?
Thoughts, @jroesch @MarisaKirisame or others?

https://github.com/apache/incubator-tvm/pull/3947


Changed to in-class member initializers and initializing all members

[approve ci]

[approve ci autest]

[approve ci autest]

[approve ci autest]

[approve ci autest]


dashboard scroll/edit behaviour reflecting in remote users browser session

Hi,
I noticed a behaviour, when scrolling screen in a dashboard, reflects in scrolling behaviour in remote user browser session. Same with even create dashboard, screen.
Any clue on this ? Please suggest to resolve.

Duplicate #871


Models regeneration for Autorest v3

Work on https://github.com/Azure/autorest.python/tree/autorestv3
Short version:
Make the template structure to generate the models folder ONLY (no operation yet at this point).
Dependency on https://github.com/Azure/autorest.python/issues/114 first to regenerate easily

We did enough progress to close this one which is almost an "Epic", and create smaller focused one


Multiple mapping

Hello,
Thank you developing this tool. I have a question related to the following point mentioned in the readme.
The abundance is the number of bases mapped to a contig, divided by the length of the contig. Reads mapping to n different locations are weighted 1/n.
In my test run (even with -N option set to >1), I only see secondary alignments which need not be equally well mapped to reference like the primary alignment. How is multiple mapping defined here?
Also, is it possible to get MD flag field (like Bowtie2) to help compute sequence identity for the alignment?
Thank you.

Hi @yazhinia,

The abundance is the number of bases mapped to a contig, divided by the length of the contig. Reads mapping to n different locations are weighted 1/n.

The text above refers only when using the command-line option --aemb which outputs coverage estimates only. How are you running strobealign?

Also, is it possible to get MD flag field (like Bowtie2) to help compute sequence identity for the alignment?

Strobealign does not support an MD field, but with option --eqx it outputs CIGAR strings with =/X for matching and mismatching bases. Such CIGAR strings allow identity computation between the reads and the reference.

Thank you for the prompt reply. I wanted to understand how the aligner chooses n alignments for a read, if exist, in the --aemb mode? When you say Reads mapping to n different locations, does it mean mapping to multiple locations are equally perfect alignment? If I wanted to get n alignments that are perfect for every read, how to get it in output?
Is -N option in strobealign same as -k option in bowtie2?
Great, --eqx option is very useful.

I wanted to understand how the aligner chooses n alignments for a read, if exist, in the --aemb mode?

In --aemb mode it chooses based on seed hits and their scores, specifically the NAM score described in the strobealign paper. Aemb uses only the mapping location(s) with the best NAM score. If n identical best scoring locations it uses those n locations.  No extension alignment is performed.

Is -N option in strobealign same as -k option in bowtie2?

Yes, it reports N alignment in decreasing order of score. The best score is the primary alignment.

If I wanted to get n alignments that are perfect for every read, how to get it in output?

You can set -N n --eqx (n your integer of choice) and parse cigar strings that only look like e.g. 150= if the read is of length 150. This will give you all "perfect" alignments, i.e., no difference to reference.

Thank you very much for the explanation.


Loop doesn't work with images on Android.

I will upload GIF:

I tried to initialize these images with other styles and props.
UPDATE Only last image isn't fine.

I made misstake, i didn't refresh when images are loaded up.

What do you mean by 'refresh'?

Ignore that. Images were loaded more times than once , method shouldComponentUpdate wasn't setup.
After setup all images are displayed! Thanks for plugin.


Rate limit possibly does not work

just try from two diffferent devices and check whether it gives 100 limit to both and not combined

fixed by adding x-real-ip header to proxy in nginx configuration on server


aalert - specify config file explicitly

We start aalert like this: (at least in branch 2.2)
docker run -d $DOCKER_PARAM -i -p $ALERTMANAGER_PORT:9093 \
    -v $PWD/prometheus/rule_config.yml:/etc/alertmanager/config.yml:Z \
     --name $ALERTMANAGER_NAME prom/alertmanager:$ALERT_MANAGER_VERSION > /dev/null

However when trying to start the container, I don't see the config being applied, and instead I see:
[glauber@glauber scylla-grafana-monitoring]$ sudo docker logs aalert
level=info ts=2019-04-10T00:44:49.569025991Z caller=main.go:177 msg="Starting Alertmanager" version="(version=0.16.0, branch=HEAD, revision=73bdd966e0055f3b828340ade3ef1f3a38169cdc)"
level=info ts=2019-04-10T00:44:49.569489336Z caller=main.go:178 build_context="(go=go1.11.4, user=root@fbea1f55d584, date=20190121-12:58:06)"
level=info ts=2019-04-10T00:44:49.575743943Z caller=cluster.go:161 component=cluster msg="setting advertise address explicitly" addr=<IP_ADDRESS> port=9094
level=info ts=2019-04-10T00:44:49.578261655Z caller=cluster.go:632 component=cluster msg="Waiting for gossip to settle..." interval=2s
level=info ts=2019-04-10T00:44:49.605760847Z caller=main.go:334 msg="Loading configuration file" file=/etc/alertmanager/alertmanager.yml <==== WRONG

Patching our start-alertmanager.sh makes it work.
The reason seems to be that according to docker inspect:
        "Created": "2019-04-10T00:44:49.132412174Z",
        "Path": "/bin/alertmanager",
        "Args": [
            "--config.file=/etc/alertmanager/alertmanager.yml",
            "--storage.path=/alertmanager"
        ],

So maybe some versions of prometheus specify this path in their args. This sounds a bit fragile the way it is right now

Fix by #571


isEqual performance optimization

Optimized isEqual for speed on primitive types. Now it's multiple times faster than lodash in all browsers.
Performance of isEqual is critical for Backbone.Model.set performance.
http://jsperf.com/underscore-isequal/5
Integer comparison:
Chrome: 70x faster
Firefox: 50x faster
Safari: 140x faster

I think this is getting more complicated than is needed. @megawac's original code is able to get the same performance with common usage without another function or duplicating all the checks already in eq(). https://github.com/jashkenas/underscore/compare/master...jridgewell:isEqual

@jridgewell, in fact, it's quite oppoiste. This is an only line of additional logic this change adds, all other are optimizations of existing code for modern browser's JIT.
if (typeof a != 'object' && typeof b != 'object') return a != a && b != b;

And it's way faster in case if you compare objects too, cause it's eq function optimization, not a hack on a top of it, as "original  change" is.

Hey. Looks like lodash guys are already testing the fix, improving performance to match this function. :)
http://jsperf.com/underscore-isequal/6
That's enough. I'm switching to lodash. Bye.

Ok, that was my last commit. Now you have one of the fastest recursive deepEq implementations currently available. Unfortunately, I have no time to deal with that issue any more.

Should this be closed then?

Yap. Micro-opt is micro-opt.


Move to SCS for sessions

Seems not terrible, don't need to roll as much from scratch.
This is just using the in-memory store, which is good enough for now. Can probably add the SQLite backend tho.
Still have a gorilla shim for oidc middleware, would be nice to fix that at some point.

This needs to be tested out.

not going to do this.


Rustup nightly install failed for the 2019-05-11 update

Problem

I was trying to update my rustup to nigthly version with rustup update nightly to the latest, which happens to be 1.36.0-nightly (d595b1135 2019-05-10), but it said this instead:
error: failed to install component: 'rust-docs-x86_64-pc-windows-msvc', detected
 conflict: '"share/doc/rust/html"'

More in-depth console output:
info: syncing channel updates for 'nightly-x86_64-pc-windows-msvc'
info: latest update on 2019-05-11, rust version 1.36.0-nightly (d595b1135 2019-0
5-10)
info: downloading component 'rustc'
info: downloading component 'rust-std'
info: downloading component 'cargo'
info: downloading component 'rust-docs'
info: installing component 'rustc'
 60.5 MiB /  60.5 MiB (100 %)   7.8 MiB/s ETA:   0 s
info: installing component 'rust-std'
 55.2 MiB /  55.2 MiB (100 %)   9.2 MiB/s ETA:   0 s
info: installing component 'cargo'
  2.9 MiB /   2.9 MiB (100 %)   1.5 MiB/s ETA:   0 s
info: installing component 'rust-docs'
  2.1 MiB /  11.0 MiB ( 19 %)  16.0 KiB/s ETA: 9.513163248697916 min 30.78979492
  2.8 MiB /  11.0 MiB ( 26 %)  22.4 KiB/s ETA: 6.24749755859375 min 14.849853515
  2.8 MiB /  11.0 MiB ( 26 %)  19.2 KiB/s ETA: 7.281802707248264 min 16.90816243
  2.9 MiB /  11.0 MiB ( 26 %)  14.4 KiB/s ETA: 9.68129249855324 min 40.877549913

Irrelevant part is cut as it repeats the same installing progress, you can see in comment history.
  8.2 MiB /  11.0 MiB ( 75 %)   9.6 KiB/s ETA: 4.924716525607639 min 55.48299153
  8.3 MiB /  11.0 MiB ( 75 %)   8.0 KiB/s ETA: 5.809659830729166 min 48.57958984
  8.4 MiB /  11.0 MiB ( 76 %)  16.0 KiB/s ETA: 2.813163248697917 min 48.78979492
 11.0 MiB /  11.0 MiB (100 %) 260.8 KiB/s ETA:   0 s
info: rolling back changes
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\k_
oncg7yt_1u10dc_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\etc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\97
mi688spn10mdxc_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\etc/bash_completion.d'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\0u
owi_fmlyydq6sf_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\share/doc/cargo'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\fk
nq1e6suhpyhwzq_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\share/man'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\vi
fnxj91uuz9aa4z_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\share/man/man1'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\gz
iqtal1bvw0e990_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\share/zsh'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\q0
6wwmwfywgd180i_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\share/zsh/site-functions'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\zd
w0y3d47xyfx48r_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\ii
_0ndjz6ltychen_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\_1
eeszp07vykuqn__dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/lib'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\us
occ4ohyy34tllx_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\lib/rustlib/etc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\3z
z06qy6sngp_wx9_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\jw
004mq5wbtq52aa_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_6
4-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/codegen-backends'
error: failed to install component: 'rust-docs-x86_64-pc-windows-msvc', detected
 conflict: '"share/doc/rust/html"'

I am using Windows 8.0 and using CMD terminal. I tried it twice, in the VSCode terminal and directly in CMD (using administrator privilege for the latter), both yields the same result.
Steps

I set the directory of the terminal to my project folder, previously made using cargo init.
Simply typed rustup update nightly

Possible Solution(s)

-
Notes
Output of rustup --version: rustup 1.17.0 (069c88ed6 2019-03-05)
Output of rustup show:
Default host: x86_64-pc-windows-msvc

installed toolchains
--------------------

stable-x86_64-pc-windows-msvc
nightly-x86_64-pc-windows-msvc

active toolchain
----------------

stable-x86_64-pc-windows-msvc (default)
rustc 1.33.0 (2aa4c46cf 2019-02-28)

I am using Windows 8.0.
This problem is encountered when following the instructions to set up the IntermezzOS in its book
Command Prompt is run both with and without administrator privilege.

Hi,
Nightly can sometimes have problems like this.  Could you confirm if this remains the case in a few days?
If it does, then we might have something to investigate.
Thanks,
Daniel.

Hi
So i have tried it again today, same problem still presist, here's the console output:
info: syncing channel updates for 'nightly-x86_64-pc-windows-msvc'
info: latest update on 2019-05-15, rust version 1.36.0-nightly (372be4f36 2019-05-14)
info: downloading component 'rustc'
 60.6 MiB /  60.6 MiB (100 %)   1.1 MiB/s ETA:   0 s                n 1.8309497833251953 s
info: downloading component 'rust-std'
 55.3 MiB /  55.3 MiB (100 %)   1.2 MiB/s ETA:   0 s
info: downloading component 'cargo'
  2.9 MiB /   2.9 MiB (100 %) 721.7 KiB/s ETA:   0 s
info: downloading component 'rust-docs'
 11.0 MiB /  11.0 MiB (100 %) 883.0 KiB/s ETA:   0 s
info: installing component 'rustc'
 60.6 MiB /  60.6 MiB (100 %)   2.4 MiB/s ETA:   0 s                n 53.821121717282864 s
info: installing component 'rust-std'
 55.3 MiB /  55.3 MiB (100 %)   7.6 MiB/s ETA:   0 s
info: installing component 'cargo'
  2.9 MiB /   2.9 MiB (100 %)   1.5 MiB/s ETA:   0 s
info: installing component 'rust-docs'
 11.0 MiB /  11.0 MiB (100 %) 240.0 KiB/s ETA:   0 s                n 50.54638671875 s
info: rolling back changes
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\d3u_nbuskqgbfxfh_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/zsh'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\nitsuf2puwxx1rdh_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/zsh/site-functions'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\5ds4m_rr3yc5xyur_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/man'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\rot4574qpm6g5bqs_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/man/man1'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\s08vpr9ca5uv8089_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/doc/cargo'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\yuon_z996rcxks___dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\etc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\y2zlizxlju8fh84c_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\etc/bash_completion.d'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\6zj9fieb1lu0f8ze_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\sfkfb_ougoh0gcb2_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\s89yu3rlw20n3iz3_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/lib'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\hai5lwntkare5a6w_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\78uxkxsuefv48hjb_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/etc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\2t1y_0hu4ubpsdbe_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/codegen-backends'
error: failed to install component: 'rust-docs-x86_64-pc-windows-msvc', detected conflict: '"share/doc/rust/html"'

Okay, thank you @parampaa2 for trying again.
So now we know there's something we need to diagnose, let's step another step along the path.
Based on the log you're showing, I think you're running a slightly old copy of rustup, so let's eliminate that as a problem by first running rustup self update before trying rustup update once more.
Also, can you confirm how much free space you have on the C: partition, and whether you are running any anti-virus beyond Windows Defender?

Alright, so i have recently updated the rustup using rustup self update, and it was a success with this output:
info: checking for self-updates
info: downloading self-update
info: rustup updated successfully to 1.18.2

However, it still does not relieve the problem, using rustup update nightly yields the same result:
info: syncing channel updates for 'nightly-x86_64-pc-windows-msvc'
381.3 KiB / 381.3 KiB (100 %)  44.6 KiB/s ETA:   0 s
info: latest update on 2019-05-16, rust version 1.36.0-nightly (7158ed9cb 2019-05-15)
info: downloading component 'rustc'
 60.6 MiB /  60.6 MiB (100 %) 337.6 KiB/s ETA:   0 s
info: downloading component 'rust-std'
 55.3 MiB /  55.3 MiB (100 %) 347.1 KiB/s ETA:   0 s
info: downloading component 'cargo'
  2.9 MiB /   2.9 MiB (100 %) 411.7 KiB/s ETA:   0 s
info: downloading component 'rust-docs'
 11.0 MiB /  11.0 MiB (100 %) 669.4 KiB/s ETA:   0 s
info: installing component 'rustc'
 60.6 MiB /  60.6 MiB (100 %)   1.8 MiB/s ETA:   0 s
info: installing component 'rust-std'
 55.3 MiB /  55.3 MiB (100 %)   4.5 MiB/s ETA:   0 s
info: installing component 'cargo'
  2.9 MiB /   2.9 MiB (100 %)   1.5 MiB/s ETA:   0 s
info: installing component 'rust-docs'
 11.0 MiB /  11.0 MiB (100 %) 254.4 KiB/s ETA:   0 s
info: rolling back changes
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\0lrta43etg8q_uvc_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/zsh'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\7a0u8cad801j4f6b_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/zsh/site-functions'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\60ug_pulb8orob_5_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\etc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\99lny0j1fqwve0sl_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\etc/bash_completion.d'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\iwarmgwoys44ede__dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/doc/cargo'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\4rn6a1yphvu3utkp_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/man'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\t0t7yv20omyfj6vu_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/man/man1'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\mlb_o0mtq_27p7mp_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\7d0n7j_v69dknj4k_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\cjm91p2z3py4ewin_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/lib'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\gpobkv2kkypodmve_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/codegen-backends'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\b5_hqzgla2akwah5_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\3qia6ol22m5jzew6_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/etc'
error: failed to install component: 'rust-docs-x86_64-pc-windows-msvc', detected conflict: '"share/doc/rust/html"'

I recently backed up my personal data from my C: partition, which is 41 GB right now. Previously, on the first encounter of the problem, i have 33 GB.
Also as an additional information, 2 (out of 36) of my friends claimed they successfully executing rustup update nightly, I will ask their computer details. And also, i once tried this on one of the computers in my faculty computer lab (Using Win10, First rust installation is about one or two month ago, and has VS 2017 with C++ package installed) and the rustup update nightly went succeeded.
An addition to my current Win8.0 PC, i used VS 2015. I am running AVG antivirus, disabling it yields the same result:
info: syncing channel updates for 'nightly-x86_64-pc-windows-msvc'
381.3 KiB / 381.3 KiB (100 %)  41.2 KiB/s ETA:   0 s
info: latest update on 2019-05-16, rust version 1.36.0-nightly (7158ed9cb 2019-05-15)
info: downloading component 'rustc'
info: downloading component 'rust-std'
info: downloading component 'cargo'
info: downloading component 'rust-docs'
info: installing component 'rustc'
 60.6 MiB /  60.6 MiB (100 %)   3.1 MiB/s ETA:   0 s
info: installing component 'rust-std'
 55.3 MiB /  55.3 MiB (100 %)   1.9 MiB/s ETA:   0 s
info: installing component 'cargo'
info: installing component 'rust-docs'
 11.0 MiB /  11.0 MiB (100 %) 420.8 KiB/s ETA:   0 s
info: rolling back changes
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\8q7yfn29ohs0gstb_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/man'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\sgbeo2yfot3hzz0p_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/man/man1'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\zzfgil6joj5kg5ub_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\3iic0q19v3cps6hz_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/doc/cargo'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\c0lyrb4yzvf3sgwf_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\etc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\bjeiq65lhzvb73wm_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\etc/bash_completion.d'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\cb85tf1xe6icittf_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/zsh'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\x1zl466u2y1faxv9_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\share/zsh/site-functions'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\lrkwf0btnpat58pu_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\x9tzffk_se6hjj8d_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/lib'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\j5x_fgvqh2ba3e83_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/codegen-backends'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\il8dy6vkd900k0vo_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/x86_64-pc-windows-msvc/bin'
error: could not rename component file from 'C:\Users\Aufi Rayesa\.rustup\tmp\f_i62v905oiyo_pk_dir\bk' to 'C:\Users\Aufi Rayesa\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/etc'
error: failed to install component: 'rust-docs-x86_64-pc-windows-msvc', detected conflict: '"share/doc/rust/html"'

Thanks for your assistance, i hope these helped.

Hi,
Thanks for that, it has eliminated some paths of investigation, for sure.
So what we're left with is a question of whether it could be a Windows-specific issue, perhaps related to path lengths, or a rustup issue with the space in your username, or something else.
@rbtcollins Do you happen to have any ideas on this issue?  Would it help if we had a zip of this user's .rustup to look at?

The errors being logged are all during the rollback - the bk directory is for rollbacks. We're not seeing the actual error other than 'detected conflict'.
Thats a ComponentConflict, which only gets emitted from dist::component::transaction::ChangedItem.dest_abs_path.
add file, copy file, copy dir, move file, move dir call that.
dest_abs_path stats the destination and if it exists, triggers the error.
        if utils::path_exists(&abs_path) {
            Err(ErrorKind::ComponentConflict {
                name: component.to_owned(),
                path: relpath.clone(),
            }
            .into())

I believe the logic is this:

move the existing files to backup
unpack and move into place
delete backups

So this is failing at 2 because something is causing "share/doc/rust/html" to still exist in the target directory.
I can think of a few possilbilities;

perhaps rust-docs was removed from the metadata (so its not being backed up) but it still on disk (so we get a conflict). Or some variation.
perhaps the file system is returning from the move call in 1 but stil reporting the files presence in the stat during 2. possibilities including network file system caching layers, or the weird thing I've read about but not dug into where delete() can return but the file still exists for a bit - perhaps that happens on move too, but we just haven't actually observed it previously.

I'd start by looking at the rustup metadata to see if it thinks rustc-docs is installed in that toolchain presently.

I should mention, I haven't looked closely at the local data store details,
so modify my suggested investigation path based on your better knowledge.
Tldr though is that we should make sure rust knows there is content at the
conflicting path before the update begins, and that it is moving it out of
the way, before suspecting platform issues.
Perhaps running with -v will help.

My working hypothesis here, since I've now seen it elsewhere, is that at least one of the manifest files in lib/rustlib/ under the toolchain has been lost.

Hi @parampaa2
Could you tell me if you're still having this problem, or if you resolved it?  If you're still having it, could you tell me if any of the files called manifest-... in the toolchain directory under lib/rustlib exist but are empty (zero size) ?
If you run rustup show it'll tell you where Rustup's home is.  Look in the toolchains directory in there.

This is another instance of missing/corrupt manifest data. Without knowledge of the installed file, rustup cannot remove it prior to installing the updated version, and then hits a conflict due to the file already existing, and you get the error.
We've got a bunch of these bugs scattered all over the place, I'm closing them all to bring order. See #2417.


[PokemonTabletopAdventures_v3] Allows Collapsing of Origin and Class Features

Changes / Comments

fix: Small CSS fix for the ball modifier section

improves appearance at the standard width of character sheets
ensures transparency is correctly set for text background colours

feature: Collapsing origin feature

Added the capability to collapse the Origin Feature to save scroll time
Added the capability to send the Origin Feature to the chat log in a neat and colour-coded format

feature: Makes class features collapsible

Updates text area and number input backgrounds to use the type-variable text background colour
Updates number inputs to use bold font-weight for consistency across the character sheet
Adds collapsing to class features
Corrects the assignment of relative position for the origin feature div

chore: Adding more to-do entries

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

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
Yes.
[x] Is this a bug fix?
Not primarily, although there are some small fixes included.
[x] Does this add functional enhancements (new features or extending existing features)?
Technically yes, although it's a secondary purpose.
[x] Does this add or change functional aesthetics (such as layout or color scheme)?
Yes - the ability to collapse features

Character Sheet Info Roll20 Internal Use only.


Remove Resources section in footer

Remove the Resources section in the footer of the site:
Resources
Cloud Services
Community
Documentation
Knowledge Base
Learning
Partner Connect
Support
Tech Zone

updated with https://github.com/euc-dev/euc-dev.github.io/commit/7c0d449cedd6ee4da42eb2372c8c4ff7d6b8960e


net451 support

net451 support would be nice to have, because many companies are still limited to this framework.

It would be even better to support net45 which has almost no difference with net451.

Gotcha, net45 and net451 support are slated for 0.0.2 and should be up this weekend.

@fubar-coder @sandrock The only blocker for it should be adding framework assembly references to the project.json file

@fubar-coder  @sandrock Version 0.0.3 is up on nugget, it supports net4.5 and netstandard 1.3


i2c-sam0 sleeps waiting for interrupt

Describe the bug
This is likely more a question than a bug report.  However, at this point I have not been able to determine what the correct course of action is to address this observation.
When CONFIG_SYS_POWER_MANAGEMENT and CONFIG_SYS_POWER_SLEEP_STATES are enabled, the i2c-sam0 driver can suffer corruption when the device enters a sleep state.  This happens because the i2c-sam0 driver waits for a semaphore.  During the wait, a sleep state can be entered where the i2c clock is turned off.
Here is where this issue manifests itself in the i2c-sam0 driver: https://github.com/zephyrproject-rtos/zephyr/blob/42c5b0a7fafa3360b8d835d4b76d7b1bebbe9237/drivers/i2c/i2c_sam0.c#L459
I believe this issue is related to issue #10524.  In that issue @nashif indicates:

i2c and other drivers might require multithreading. this is documented now in the single thread configuration option.

and subsequently closed the issue.  From the comment it is not clear what is referred to.  I reviewed the CONFIG_MULTITHREADING documentation but there is no mention of how to address the pm issue with i2c.
To Reproduce
I have been testing this with my own SAMD21 board with an i2c peripheral.  I believe any SAMD21 board (Adafruit M0 proto board) will reproduce the problem if the config options are set as described above and the following sleep states are defined:
void sys_set_power_state(enum power_states state)
{
	LOG_DBG("SoC entering power state %d", state);

	/* FIXME: When this function is entered the Kernel has disabled
	 * interrupts using BASEPRI register. This is incorrect as it prevents
	 * waking up from any interrupt which priority is not 0. Work around the
	 * issue and disable interrupts using PRIMASK register as recommended
	 * by ARM.
	 */

	/* Set PRIMASK */
	__disable_irq();
	/* Set BASEPRI to 0 */
	irq_unlock(0);

	switch (state) {
#ifdef CONFIG_SYS_POWER_SLEEP_STATES
#ifdef CONFIG_HAS_SYS_POWER_STATE_SLEEP_1
	case SYS_POWER_STATE_SLEEP_1:
        SCB->SCR &= ~SCB_SCR_SLEEPDEEP_Msk;
        PM->SLEEP.reg = 2;
        __DSB();
        __WFI();
		break;
#endif /* CONFIG_HAS_SYS_POWER_STATE_SLEEP_1 */
#ifdef CONFIG_HAS_SYS_POWER_STATE_SLEEP_2
	case SYS_POWER_STATE_SLEEP_2:
        // Disable systick interrupt:  See https://www.avrfreaks.net/forum/samd21-samd21e16b-sporadically-locks-and-does-not-wake-standby-sleep-mode
        // If SysTick is not enabled then no need to mask the interrupt
#ifdef CONFIG_CORTEX_M_SYSTICK
        SysTick->CTRL &= ~SysTick_CTRL_TICKINT_Msk;
#endif
        SCB->SCR |= SCB_SCR_SLEEPDEEP_Msk;
        __DSB();
        __WFI();
        // Re-enable systick interrupt if SysTick is enabled
#ifdef CONFIG_CORTEX_M_SYSTICK
        SysTick->CTRL |= SysTick_CTRL_TICKINT_Msk;
#endif
		break;
#endif /* CONFIG_HAS_SYS_POWER_STATE_SLEEP_2 */
#ifdef CONFIG_HAS_SYS_POWER_STATE_SLEEP_3
	case SYS_POWER_STATE_SLEEP_3:
		break;

Expected behavior
System power sleep states that affect the i2c peripheral clock  (sleep state 2 in my example above) should be avoided when an i2c master is waiting for an i2c bus event or transaction to complete.
Note, if a driver is configured to act as an i2c slave, it is possible to wake the cpu on an i2c address match without an internal clock gated on to the peripheral.  So, in this instance, the peripheral clock could be gated off while waiting for the device to be addressed.
Impact
I2C is non functional during sleep.

I have another observation regarding the sam0 i2c driver.  It appears that the i2c bus can be left in a non idle state.  This is a problem because the SCL and/or SDA lines can be held low by the sam0 during and after sleep resulting in significant current (>500uA) due to the pull up resistors on these lines.
I have not isolated what is causing this issue but I have confirmed using an oscilloscope that the SCL and/or SDA lines remain low after an i2c transaction.
It is possible this issue is not related to the topic of this issue and should be a recorded as a separate issue.

@sslupsky is this still an issue? Otherwise please close it.

@carlescufi I submitted a PR quite a while ago that addresses this.  The PR hasn't been accepted yet so I left this open.
I'll close it.


Fix/package name

With version 1.x native-packager uses the packageName setting to define the generated package name.

Ah ha. Thanks @muuki88!
The issue we have is the DebianPlugin needing a changelog and then "conflicting source packages" (see #125). Any ideas there?

Unfortunately not :(
It's been a long time since I built a debian package. I'll take a look tomorrow and see if I can find anything suspicious.


pip installation shows errors

Hi,
I just found that the installation via pip throws a few errors. Doesn't look like they cause trouble as it only relates to the wheel and clean targets:
pip install 'https://github.com/genestack/python-client/archive/v0.24.0.tar.gz#egg=genestack-client-0.24.0'

Collecting genestack-client-0.24.0 from https://github.com/genestack/python-client/archive/v0.24.0.tar.gz#egg=genestack-client-0.24.0
  Downloading https://github.com/genestack/python-client/archive/v0.24.0.tar.gz (106kB)
    100% |████████████████████████████████| 112kB 414kB/s
  Running setup.py (path:/tmp/pip-install-L_p2VH/genestack-client-0.24.0/setup.py) egg_info for package genestack-client-0.24.0 produced metadata for project name genestack-
client. Fix your #egg=genestack-client-0.24.0 fragments.
Requirement already satisfied (use --upgrade to upgrade): genestack-client from https://github.com/genestack/python-client/archive/v0.24.0.tar.gz#egg=genestack-client-0.24.0
 in ./lib/python2.7/site-packages
Requirement already satisfied: keyring in ./lib/python2.7/site-packages (from genestack-client) (12.2.1)
Requirement already satisfied: requests in ./lib/python2.7/site-packages (from genestack-client) (2.18.4)
Requirement already satisfied: pyOpenSSL in ./lib/python2.7/site-packages (from genestack-client) (18.0.0)
Requirement already satisfied: entrypoints in ./lib/python2.7/site-packages (from keyring->genestack-client) (0.2.3)
Requirement already satisfied: secretstorage<3; (sys_platform == "linux2" or sys_platform == "linux") and python_version < "3.5" in ./lib/python2.7/site-packages (from keyri
ng->genestack-client) (2.3.1)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in ./lib/python2.7/site-packages (from requests->genestack-client) (3.0.4)
Requirement already satisfied: certifi>=2017.4.17 in ./lib/python2.7/site-packages (from requests->genestack-client) (2018.4.16)
Requirement already satisfied: urllib3<1.23,>=1.21.1 in ./lib/python2.7/site-packages (from requests->genestack-client) (1.22)
Requirement already satisfied: idna<2.7,>=2.5 in ./lib/python2.7/site-packages (from requests->genestack-client) (2.6)
Requirement already satisfied: cryptography>=2.2.1 in ./lib/python2.7/site-packages (from pyOpenSSL->genestack-client) (2.2.2)
Requirement already satisfied: six>=1.5.2 in ./lib/python2.7/site-packages (from pyOpenSSL->genestack-client) (1.11.0)
Requirement already satisfied: configparser>=3.5; python_version == "2.7" in ./lib/python2.7/site-packages (from entrypoints->keyring->genestack-client) (3.5.0)
Requirement already satisfied: cffi>=1.7; platform_python_implementation != "PyPy" in ./lib/python2.7/site-packages (from cryptography>=2.2.1->pyOpenSSL->genestack-client) (
1.11.5)
Requirement already satisfied: enum34; python_version < "3" in ./lib/python2.7/site-packages (from cryptography>=2.2.1->pyOpenSSL->genestack-client) (1.1.6)
Requirement already satisfied: ipaddress; python_version < "3" in ./lib/python2.7/site-packages (from cryptography>=2.2.1->pyOpenSSL->genestack-client) (1.0.22)
Requirement already satisfied: asn1crypto>=0.21.0 in ./lib/python2.7/site-packages (from cryptography>=2.2.1->pyOpenSSL->genestack-client) (0.24.0)
Requirement already satisfied: pycparser in ./lib/python2.7/site-packages (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography>=2.2.1->pyOpenSSL->genestac
k-client) (2.18)
Building wheels for collected packages: genestack-client, genestack-client
  Running setup.py bdist_wheel for genestack-client ... done
  Stored in directory: /tmp/pip-ephem-wheel-cache-5tmtTd/wheels/b9/d3/34/9e29e503a5414ac3059ef14cb7b8296677b5f7a49602a99d7d
  Running setup.py bdist_wheel for genestack-client ... error
  Complete output from command /home/marcherm/git/stool/bin/python2 -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-L_p2VH/genestack-client/setup.py';f=getattr
(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" bdist_wheel -d /tmp/pip-wheel-wxM_hC --python-tag cp2
7:
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
  IOError: [Errno 2] No such file or directory: '/tmp/pip-install-L_p2VH/genestack-client/setup.py'

  ----------------------------------------
  Failed building wheel for genestack-client
  Running setup.py clean for genestack-client
  Complete output from command /home/marcherm/git/stool/bin/python2 -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-L_p2VH/genestack-client/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" clean --all:
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
  IOError: [Errno 2] No such file or directory: '/tmp/pip-install-L_p2VH/genestack-client/setup.py'

  ----------------------------------------
  Failed cleaning build dir for genestack-client
Successfully built genestack-client
Failed to build genestack-client

Looks like a pip bug to me: pypa/pip/issues/5271


brotli is implemented in zlib

Why?
https://github.com/bobanetwork/boba/issues/213
Brotli was affecting how react app gateway was starting up.
How?
Use native zlib implementation of brotli.
https://nodejs.org/api/zlib.html#zlibbrotlicompresssyncbuffer-options
https://nodejs.org/api/zlib.html#zlibbrotlidecompresssyncbuffer-options

Looks good & reasonable!

Great!


allowTextSelectionOnPress does not work with mouse selection

Expected Behaviour
Pressing the left mouse and dragging on a text should select the text when usePress({ allowTextSelectionOnPress: true}).
Going further, in my opinion,  usePress({ isDisabled: true}) default behavior should allow text selection.
Actual Behaviour
The text is un-selectable with mouse, however allowTextSelectionOnPress does make selections with touch interaction.
Platform and Version
Chrome, FireFox, React Aria version is 3.14.0
Sample Code that illustrates the problem
https://codesandbox.io/s/react-aria-text-selection-issue-xc4bhw?file=/src/App.js

Sure, really appreciate your reply!
This might be because I'm not correctly understanding the API, but I was trying to use usePress hook for my cards. Since cards aren't usually pressable, but we could imagine some scenario where pressing them does something, I thought I'd have  usePress hook  configured with isDisabled: true by default, and that would make them just like regular containers. So obviously I would not want my unpressable cards to disable text selection.
Now I understand that isDisabled in usePress acts like a disabled option for a button, but initially I thought it was a means to opt-out from usePress without messing with "Rules of Hooks". In my opinion, disabling text, preventing defaults and so on should be handled in useButton for buttons and button-like elements and usePress should just provide keyboard and press listeners for the element, because not everything pressable is a button. This is just my opinion and I absolutely trust your judgements.
allowTextSelectionOnPress is weird to me because it actually changes things and works with touch, but it does not with mouse, and I don't see how it's intentional.

That makes a lot of sense, and yeah, that default behavior seems hard to change. Thanks for the suggestion, it definitely solves my use case!

Closing for now, I haven't been able to come up with a solid real world use case yet that wouldn't follow the same omit pressProps approach that I described above.

Commenting here because I just spent the last while trying to get allowTextSelectionOnPress to work on my clickable item. If its not going to work across mobile and mouse, could we please update the docs to reflect that?

Sure, that sounds reasonable, mind opening a separate issue for the docs update? I figure we can do a callout in the docs of this and maybe a type description update

Found this issue while trying to find out why selection doesn't work on pressable element :)
@LFDanLu Hi, is it intentional that press and text selection are mutually exclusive? Is it possible to make them both work? My use case is a clickable card with title that users may select and copy

There are currently some discussions around updates to usePress that this may fall under, but at the moment it isn't possible to enable text selection in a pressable card.


Add a slim version

This will jut have the start() method.

Will probably update it to make it more slim and better


DSS domain redirect

Add a stack to create DSS Route53 records
Points [www.]digitalservicesstore.service.gov.uk to the nginx ELB
using Route53 alias records.
There's a stage check for hosted zone domain value that will make
stack creation fail on non-production environments. This is making
sure that records don't get accidentally changed to staging ELB.
Add nginx config to redirect DSS domain to digitalmarketplace
Update nginx AMI with new digitalservicesstore config

Left a comment but looks good as far as I can tell... :+1:


CI needs to be compiling and building the performance framework

Yep, can you please file an issue. Let's get a plan together to stay green and avoid having a dev spend the time, like yourself, always having to re-build locally to verify. For most (if not all) checked-in code, verifying it is building is good to incorporate in CI.
Originally posted by @ahsonkhan in https://github.com/Azure/azure-sdk-for-cpp/pull/2371#discussion_r643400432

cc @vhvb1989, @danieljurek


Add stl and images

Pull requests are fun and all but can you perhaps give me write access?
Cheers

Done! Sorry about that!

From: conquillacarl<EMAIL_ADDRESS>Sent: Friday, November 13, 2020 6:07:01 PM
To: Space-at-your-Service/Spacesuits<EMAIL_ADDRESS>Cc: O'Hara, C.A<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Space-at-your-Service/Spacesuits] Add stl and images (#1)
@Ohara124c41https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOhara124c41&data=04|01||3645280f33f449a8d8e108d887f68a76|cc7df24760ce4a0f9d75704cf60efc64|1|0|637408840238801465|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7PtVGHayrVDmpRiEGFIImV2iBI8zXX2VLT13FaV6Axc%3D&reserved=0 Pull requests are fun and all but can you perhaps give me write access?
Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSpace-at-your-Service%2FSpacesuits%2Fpull%2F1%23issuecomment-726866402&data=04|01||3645280f33f449a8d8e108d887f68a76|cc7df24760ce4a0f9d75704cf60efc64|1|0|637408840238811412|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PJrFs563sreOKIW%2BtiKj6Y2WD1UP0fdwY2%2Br1d%2FIZPE%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGXGFJDDAOIZC3WD6HQDLM3SPVRTLANCNFSM4TUZGD2Q&data=04|01||3645280f33f449a8d8e108d887f68a76|cc7df24760ce4a0f9d75704cf60efc64|1|0|637408840238811412|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=zbKuVnPA42D13r2%2BSnoaipiYvYVvqvr4WAUrobkMtQw%3D&reserved=0.


Add supported encoding formats to RawImage and CompressedImage

Public-Facing Changes

Add supported encoding formats to RawImage and CompressedImage

Resolves #32

ready to merge?

Closing this PR, and addressing in https://github.com/foxglove/schemas/pull/60.


[Android Gradle] needs to download CMake, but got no chance to agree to license agreement, failed builds

Hi there
I am a Microsoft employee working on an cross-platform & cross-language project based on Djinni. It includes an Android project that will also compile C++ code. I chose the "Hosted" agent pool, and used the API level 25 that is supported by the Hosted agent pool. My project needs to download CMake and some other libraries, but there's no chance to accept the license agreement during the build process. Please check the pasted log. If this could be supported in soon, I'd appreciate it very much.
VSO
Using Hosted agent, agent queue name: Hosted
team: Microsoft Auth Telemetry System
build definition: MATS Android (branch - master) for debug purpose
Link : https://authtelemetry.visualstudio.com/Microsoft Auth Telemetry System/_build/index?context=mine&path=\&definitionId=12&_a=completed
Error logs
log_vso_fail_download_cmake.txt

Have you tried the Hosted VS2017 pool?

yes, same thing

Closing this since it is an old issue. We believe this has been addressed.


⚠ Bofrost.lu has degraded performance

In 1776d40, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2778 ms

Resolved: Bofrost.lu performance has improved in 3d089bd after 6 minutes.


Update about-packages-and-modules.mdx

Abnormal rendering of changed part in browser

References

This was fixed in #460


Copying doesn't work

Whenever I select some text to copy (both the command and the output) and hit Ctrl, the selection disappears, therefore it's not possible to copy text from terminal.
There's also no right-click-dropdown menu to copy text without the keyboard, that would also be nice.
EDIT: the Copy button from the top menu works, however
Seen on Ubuntu 16.04 64-bit

Should work in the latest master. Please reopen otherwise.