"Test not run" Resharper 9.1 in VS 2015 RC

All test are discovered but when trying to run them it just says "inconclusive: Test not run".
Running the test in VS test explorer works.

Are you using the latest version of the plugin? It should be version 2.1.7.
If you are, can you run with logging enabled?

Start Visual Studio with ReSharper in internal mode: devenv /ReSharper.Internal
Go to ReSharper → Options → Unit Testing and check Enable Logging
Create the %TEMP%\JetLogs folder
Run the tests

The test runner will generate a log file in %TEMP%\JetLogs detailing what it's done. Can you post that so we can see what's going on?

I'm experiencing the same issue (except for me, even VS test explorer doesn't work...):
23:12:29.220: 1: Connecting to <IP_ADDRESS>:8078
23:12:29.225: 1: Waiting for tasks.
23:12:29.238: 3: Configuration received: <configuration><runner-paths /><runner-infos><runner-info id="xUnit" code-base="xunitcontrib.runner.resharper.runner.9.1.DLL" type-name="XunitContrib.Runner.ReSharper.RemoteRunner.XunitTaskRunner" /></runner-infos></configuration>
23:12:29.239: 3: Using JetBrains.ReSharper.TaskRunnerFramework.DummyClientController
23:12:29.268: 5:Runner thread: Processing packets
23:12:29.268: 5:Runner thread: <task-start />
23:12:29.271: 5:Runner thread: Executing 1 root tasks
23:12:29.272: 5:Runner thread: Loading TaskRunner xUnit (XunitContrib.Runner.ReSharper.RemoteRunner.XunitTaskRunner) from file:///C:/Users/Nero/AppData/Local/JetBrains/Installations/ReSharperPlatformVs14_000/xunitcontrib.runner.resharper.runner.9.1.DLL
23:12:29.275: 5:Runner thread: <task-start><task task-type="XunitContrib.Runner.ReSharper.RemoteRunner.Tasks.XunitBootstrapTask, xunitcontrib.runner.resharper.runner.9.1, Version=<IP_ADDRESS>609, Culture=neutral, PublicKeyToken=null" runnerID="xUnit" id="8a7a176a-bceb-4715-9065-ec490625704f" ProjectId="946884C9-B569-4772-AC6F-2D7244975D52" /></task-start>
23:12:29.294: 5:Runner thread: <task-exception type="System.ArgumentException"><task task-type="XunitContrib.Runner.ReSharper.RemoteRunner.Tasks.XunitBootstrapTask, xunitcontrib.runner.resharper.runner.9.1, Version=<IP_ADDRESS>609, Culture=neutral, PublicKeyToken=null" runnerID="xUnit" id="8a7a176a-bceb-4715-9065-ec490625704f" ProjectId="946884C9-B569-4772-AC6F-2D7244975D52" /><message>The path is not of a legal form.</message><stack-trace>   at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
   at System.IO.Path.GetDirectoryName(String path)
   at XunitContrib.Runner.ReSharper.RemoteRunner.TestEnvironment.GetAssemblyFolder(TaskExecutorConfiguration config, XunitTestAssemblyTask assemblyTask)
   at XunitContrib.Runner.ReSharper.RemoteRunner.TestEnvironment..ctor(XunitTestAssemblyTask assemblyTask)
   at XunitContrib.Runner.ReSharper.RemoteRunner.TestRunner.Run(XunitTestAssemblyTask assemblyTask)
   at XunitContrib.Runner.ReSharper.RemoteRunner.XunitTaskRunner.ExecuteRecursive(TaskExecutionNode node)
   at JetBrains.ReSharper.TaskRunnerFramework.StartupTaskRunnerHost.Execute(TaskExecutionNode node)</stack-trace></task-exception>
23:12:29.294: 5:Runner thread: <task-finish result="Exception"><task task-type="XunitContrib.Runner.ReSharper.RemoteRunner.Tasks.XunitBootstrapTask, xunitcontrib.runner.resharper.runner.9.1, Version=<IP_ADDRESS>609, Culture=neutral, PublicKeyToken=null" runnerID="xUnit" id="8a7a176a-bceb-4715-9065-ec490625704f" ProjectId="946884C9-B569-4772-AC6F-2D7244975D52" /><message>ArgumentException: The path is not of a legal form.</message></task-finish>
23:12:29.294: 5:Runner thread: Finished processing packets
23:12:29.295: 5:Runner thread: <task-finish result="Success" />
23:12:29.298: 3: Closing server

@Grinderofl is this an asp.net 5/DNX test? The exception indicates that the plugin has the wrong path to the assembly. If you're trying to run a DNX test, this path is likely to be invalid, as DNX does testing without compiling an actual .dll to disk.

I have the same issue. Not using DNX.

Can you get a trace for the mstest issue for me, please? I can see if it's related, and if not I can forward to the ReSharper issue tracker.

I'll see what I can do. I tracked down the main part of my issue to a workaround I had installed to get code-based test data to work in MSTest (as we know, MSTest is horrible...just need time to migrate to xUnit). I stopped using that workaround and things started working. However, in principle, the workaround should be working - so I'll try to whip up a test project that exhibits the issue and get back to you.

Hi,
I'm using xUnit 2.0.0, Visual Studio 2015 (Release version) and ReSharper 9.1.1
Test runner stopped to recognize Theory, tests became unconclusive. In logs I see
15:33:11.918: 8: Discovered: «UnitTests.Web.Site.AccountControllerTests.LogIn_ByPassword_Success - UnitTests.Web.Site.AccountControllerTests.LogIn_ByPassword_Success (d860e37fb3e4e964cdb87fbce8886e15b5a81543)»
15:33:11.920: 8:  Test case is NOT a requested method: «UnitTests.Web.Site.AccountControllerTests.LogIn_ByPassword_Success - UnitTests.Web.Site.AccountControllerTests.LogIn_ByPassword_Success (d860e37fb3e4e964cdb87fbce8886e15b5a81543)»
15:33:11.921: 8:  Test case class unknown. Cannot be a dynamic test of a requested class. UnitTests.Web.Site.AccountControllerTests - «UnitTests.Web.Site.AccountControllerTests.LogIn_ByPassword_Success - UnitTests.Web.Site.AccountControllerTests.LogIn_ByPassword_Success (d860e37fb3e4e964cdb87fbce8886e15b5a81543)»

Fixed by updating version of AutoFixture in app.config.

Glad it's working. If there are any other issues, please reopen.

This also happened to me, and found the solution that actually worked in here:
http://www.henrikbrinch.dk/Blog/2012/02/15/Making-Resharper-testrunner-work-in-64-bit
Hope this will help you

Ouch. That's a pretty nasty "fix". ReSharper already ships with both 32 and 64 bit versions of the test runner executables. It will pick one up automatically depending on the bit-ness of your test project (so it needs to be the same as your system under test, or you'll get a BadImageFormatException). But you can also force it via the options - specify which version of the .net framework to target, and which bit-ness to use. That's a much better option that copying and renaming executables. I'll add a comment to that blog post, too.

Thanks


Support the load of network stored files

This fix will support previously added require(...) of sounds from JS bundle while building project with loading assets from dev server.
Also, should support loading of sound from network, using object with uri as a first parameter of Sound:
const sound = new Sound({
    uri: 'http://some.file/from/internet.mp3',
});

NB: Works only with iOS, Android support not implemented!

@zmxv +1

Is this project still active? I might work on loading sound from the network for Android

Is it possible to load the sound file with Asyncstorage, and then play the local file with react-native-sound?

@Guichaguri Hi there. Yes, this project is active once more. If you are still interested we'd love to see an Android implementation of the above.

Tested - and works for me. Merged. Thank you

Note that this will indeed load and play audio at remote URLs - but it will do it synchronously i.e. the whole file is downloaded, blocking the current thread that React Native is using to run the native command, and then it'll play.
As a result I'd say this is really useful for working with 'require' - as it allows debug to work, but it should not be used for 'real' online audio playback.

Note: 'Real' network playback (i.e. streaming) would need us to use AVPlayer rather than AVAudioPlayer. Docs here: https://developer.apple.com/reference/avfoundation/avplayer?language=objc
It has a more limited API than AVAudioPlayer - no pan for example.

+1 I'm currently using react-native-video in order to play local and web based files. I'd love to switch to react-native-sound.

Hey I try the uri , new Sound({uri:'http://streamthesound.com/blablabla'} , but look like doesn't work ? I getting this error playback failed due to audio decoding errors

Same here i am also getting error with this code.
 var whoosh = new Sound({uri: 'https://www.sample-videos.com/audio/mp3/crowd-cheering.mp3'}, (error) => {       if (error) {         console.log('failed to load the sound', error)         return       }     // loaded successfully       console.log('duration in seconds: ' + whoosh.getDuration() + 'number of channels: ' + whoosh.getNumberOfChannels())     })

@appify-waheed  Thank you


Issue with Wrds

I'm having issues with wrds, specifically version 3.1.6 which is required by FinRL. However, version 3.1.5 seems to work fine and I had to downgrade before reinstalling FinRL without the depedencies

I'm having issues with wrds, specifically version 3.1.6 which is required by FinRL. However, version 3.1.5 seems to work fine and I had to downgrade before reinstalling FinRL without the depedencies

Can you explain more and the code position?  WRDS is not necessary…I forget how wrds is requirement?

did you use the data processor wrds, or encounter issues when installing wrds?

Can you explain more and the code position? WRDS is not necessary…I forget how wrds is requirement?

So, library wrds>=3.1.6 is installed from here when I install from git

It then is imported here

Which in turn is used by data_processor

I'm trying to use DataProcessor in my code to get hourly data, since I don't think YahooDownloader has an option for interval. I think I saw this in one of the example notebooks.

The error I mentioned earlier, I think is related to wrds and not FinRL itself.

did you use the data processor wrds, or encounter issues when installing wrds?

No, I didn't. Installation went well but I get this error when trying to import DataProcessor (which in turn imports wrds)

WRDS is a requirement. See #939.

WRDS is a requirement. See #939.

Why is this version 3.1.6 needed exactly? I found the offending code which is

sql.py of wrds 3.1.6

This is the difference between f"{err=}" and f"{err}"

Python 3.7 should be supported by FinRL and so is wrds, but this behavior is not supported in 3.7.  Remove the = (equal sign) from wrds' code and it'll function properly.

I don't know why that version is required. I just discovered last weekend, while trying to reproduce, that (1) the package is required and (2) the dependency was not specified in pyproject.toml.
This syntax error in wrds was introduced in wharton/wrds#39. I would suggest opening a PR against wrds that just removes that "=".

I have created a new pull request on https://github.com/wharton/wrds/pull/40.


ZIO Schema doesn't differentiate Either[L, R] from fallback semantics

ZIO Schema has both support for a member of type Either[L, R] via Schema.either and for something that appears to be intended for fallback via Schema.orElseEither, but the latter just encodes to the same thing as the former.  For example, you cannot use it to decode JSON "[42, "43"]" to something that can either decode from Long or String.
This is demonstrated in this Scastie: https://scastie.scala-lang.org/gnp/078lqgCNRjuNBwvnOPyWFg/10

/bounty $350
This addition to ZIO Schema would be a true fallback, which will encode properly, and attempt decode left first, followed by right if that fails. Because decoding either side could succeed, this implies that use of such a fallback operator will violate laws for union data that is encoded ambiguously.
Bounty for addition of the feature, updating existing codecs, and test suite.


Track files read with/written by built-in libraries

Built-in libraries like codecs and json.
Robin's comment on Slack:
I've worked out what the problem with codecs was. You were right: it is because of the way that it is provided as a 'built-in library' (note: this is different to a 'built-in function' like open). Some (most? all?) of the libraries that are actually provided with Python (such as sys, os...and codecs) are actually semi-loaded by default when you open the Python interpreter. That sounds weird, but basically loading a module in Python consists of two steps:

The module name is 'searched for', and eventually the right .py file containing the module is found, and the module is loaded and put into sys.modules (which is just a dict with keys of module names, and values of the Module object itself). This is the bit that recipy hooks into, and it alters the module before it gets into the sys.modules dict.

The module is added to the 'global scope' so that you can access it by using its name.

When you run something like import numpy, then both of these steps are run, _but_ when you run import codecs then it finds that codecs already exists in the sys.modules dict, and so it doesn't do step 1 (it is already 'found' and 'loaded'), it just has to make it available in the current scope (basically it does codecs = sys.modules['codecs']).
The result of all of this is that it doesn't work automatically with recipy at the moment, because recipy has to catch it _before_ it goes into sys.modules - but it is in there as soon as you start the Python interpreter! The way to make it work is to add these two lines at the beginning of the script:

del sys.modules['codecs']```

which removes it from `sys.modules`, so that when `import codecs` is called, it goes through both steps above and recipy has a chance to patch it.

I've pushed a commit to the `patchCodecs` branch which adds those two lines and shows that it works.

The key question is what we do about this going forward....I'm not sure what is best. We could automatically run the `del sys.modules[MODULENAME]` for any 'built-in module' that we patch, but there could be some nasty side-effects from this (though I guess we never know til we try...!).

Hope this all made sense - any thoughts?

http://www.dabeaz.com/modulepackage/ModulePackage.pdf This presentation covers all of the crazy nuts and bolts of importing in Python - and may well help with our `codecs` problem

If you delete codecs from the sys.modules and then import pandas, pandas starts complaining:
(recipy)jvdzwaan@esciencelt018:~/code/recipy-demo$ python example_script.py
recipy run inserted, with ID 8ee5c13c-6b0c-412d-9a56-5d06702b72d0
Patching numpy
Patching input function: genfromtxt
Patching input function: loadtxt
Patching input function: fromfile
Patching output function: save
Patching output function: savez
Patching output function: savez_compressed
Patching output function: savetxt
Patching codecs
Patching input function: open
Logging exception incompatible codecs in module "encodings.unicode_escape" (/home/jvdzwaan/.virtualenvs/recipy/lib/python2.7/encodings/unicode_escape.py)
Traceback (most recent call last):
  File "example_script.py", line 2, in <module>
    import pandas as pd
  File "/home/jvdzwaan/code/recipy/recipy/PatchImporter.py", line 43, in load_module
    mod = imp.load_module(name, file_obj, pathname, desc)
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/pandas/__init__.py", line 37, in <module>
    import pandas.core.config_init
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/pandas/core/config_init.py", line 17, in <module>
    from pandas.core.format import detect_console_encoding
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/pandas/core/format.py", line 20, in <module>
    from pandas.tseries.index import DatetimeIndex
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/pandas/tseries/index.py", line 135, in <module>
    Int64Index):
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/pandas/tseries/index.py", line 811, in DatetimeIndex
    def _format_native_types(self, na_rep=u('NaT'),
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/pandas/compat/__init__.py", line 248, in u
    return unicode(s, "unicode_escape")
  File "/home/jvdzwaan/.virtualenvs/recipy/lib/python2.7/encodings/__init__.py", line 134, in search_function
    (mod.__name__, mod.__file__)
encodings.CodecRegistryError: incompatible codecs in module "encodings.unicode_escape" (/home/jvdzwaan/.virtualenvs/recipy/lib/python2.7/encodings/unicode_escape.py)

This has something to do with the order in which libraries are imported, so if you import pandas before deleting codecs from sys.modules, it works.

But then matplotlib.pyplot.savefig starts complaining with a different error message:
(recipy)jvdzwaan@esciencelt018:~/code/recipy-demo$ python example_script.py
Patching numpy
Patching input function: genfromtxt
Patching input function: loadtxt
Patching input function: fromfile
Patching output function: save
Patching output function: savez
Patching output function: savez_compressed
Patching output function: savetxt
Patching pandas
Patching input function: read_csv
Patching input function: read_table
Patching input function: read_excel
Patching input function: read_hdf
Patching input function: read_pickle
Patching input function: read_stata
Patching input function: read_msgpack
Patching output function: DataFrame.to_csv
Patching output function: DataFrame.to_excel
Patching output function: DataFrame.to_hdf
Patching output function: DataFrame.to_msgpack
Patching output function: DataFrame.to_stata
Patching output function: DataFrame.to_pickle
Patching output function: Panel.to_excel
Patching output function: Panel.to_hdf
Patching output function: Panel.to_msgpack
Patching output function: Panel.to_pickle
Patching output function: Series.to_csv
Patching output function: Series.to_hdf
Patching output function: Series.to_msgpack
Patching output function: Series.to_pickle
recipy run inserted, with ID 61c46967-968a-4cbd-a94f-9d7b26b75876
Patching matplotlib.pyplot
Patching output function: savefig
Input from /home/jvdzwaan/code/recipy-demo/data.csv using pandas
Output to /home/jvdzwaan/code/recipy-demo/newplot.pdf using matplotlib
Patching codecs
Patching input function: open
Logging exception incompatible codecs in module "encodings.mac_roman" (/home/jvdzwaan/.virtualenvs/recipy/lib/python2.7/encodings/mac_roman.py)
Traceback (most recent call last):
  File "example_script.py", line 10, in <module>
    savefig('newplot.pdf')
  File "/home/jvdzwaan/code/recipy/recipyCommon/utils.py", line 49, in f
    return wrapped(*args, **kwargs)
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/pyplot.py", line 688, in savefig
    res = fig.savefig(*args, **kwargs)
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/figure.py", line 1565, in savefig
    self.canvas.print_figure(*args, **kwargs)
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/backend_bases.py", line 2232, in print_figure
    **kwargs)
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/backends/backend_pdf.py", line 2542, in print_pdf
    file.close()
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/backends/backend_pdf.py", line 547, in close
    self.writeFonts()
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/backends/backend_pdf.py", line 650, in writeFonts
    fonts[Fx] = self.embedTTF(realpath, chars[1])
  File "/home/jvdzwaan/.virtualenvs/recipy/local/lib/python2.7/site-packages/matplotlib/backends/backend_pdf.py", line 1071, in embedTTF
    ps_name = sfnt[(1, 0, 0, 6)].decode('macroman')  # Macintosh scheme
  File "/home/jvdzwaan/.virtualenvs/recipy/lib/python2.7/encodings/__init__.py", line 134, in search_function
    (mod.__name__, mod.__file__)
encodings.CodecRegistryError: incompatible codecs in module "encodings.mac_roman" (/home/jvdzwaan/.virtualenvs/recipy/lib/python2.7/encodings/mac_roman.py)

Which you cant't fix by importing matplotlib before deleting codecs.
The reason I use (and want to patch) codecs is that it fixes reading utf-8 encoded files in Python 2.7. Perhaps it is better to try and find another solution. I'm thinking again about making a recipy.open (see also #44).
What do you think? Any other ideas?

Once you've run del sys.modules['codecs'] are then importing codecs again manually before importing pandas etc? (ie. running import codecs) If not, then try that - that theoretically should look no different to things like pandas and matplotlib!

Apparently it does look different to pandas and matplotlib! It only works if you import pandas before deleting and then importing codecs. And then it crashes on the savefig. It doesn't really make sense. I'll try it on my Mac tomorrow, maybe that makes a difference

That's really weird... Out of interest, is this Python 2 or 3? Might be worth trying on both.

It's Python2. I was having troubles installing python3 (in a virtual environment on Ubuntu).
But if I understand correctly, in python3 you can read utf-8 encoded files with standard open, so there is less need to patch codecs there.

Okay, I added example scripts and a Dockerfile for testing in a clean environment, and the errors still occur. (Also on my Mac by the way.)
In python 3 there are no errors, of course...

Do these problems still exist?

Probably. My solution was to make a workaround (recipy.open; see also #44).

Thought so - I'll close this, and if there are any further problems reported then we can open other issues.


hyprctl animations -j missing a terminating square bracket

Steps to reproduce:
hyprland animations -j | jq
Expected outcome: valid json
Noted outcome:
parse error: Unfinished JSON term at EOF at line 129, column 0
This appears to come from a missing square bracket ending the json output.

thanks! fixed in 287f31329e7a16b42c3f20b9b7ecf3ec6adbc51d


By passing header with url data is not giving by event source.

Please help.

Hi @rahulkumar7, can you provide some example of the problem you are experiencing, please?

let eventSource =   EventSource(url: url, headers: ["Authorization":str])
eventSource.onOpen {
print("open")
}
eventSource.onMessage { (id, event, data) in
print(data as Any)
//  self.updateLabels(id, event: event, data: data)
}
here cam not getting data while printing

Sample given by you also not working while passing with header is any problem with sdk?

Do you get anything through eventSource.onError ?

Error Domain=com.inaka.eventSource.error Code=-1 "(null)" UserInfo={message=HTTP Status Code: 406}
This the error I got.

Alright, that means your server is responding 406 to your request.
406 means Not Acceptable. Your server is probably incapable of serving event streams 🤷♂.
Have you tried using curl or other more basic http request tool to hit the same server?

I got response when I give it in postman.So issue with Api?? I think streaming api will not work in postman.

Exactly. Postman doesn't request streaming APIs… if it works in postman, that means it will likely fail in EventSource.

Is it some public server? Can you let me know the url you're trying to hit so I can test it?

Thanks,it worked for me.


Create systemd unit file & logging directory

Also partially fixes #168.
Pull Request Checklist

[x] Pull request includes a sign off

I'm reluctant to include anything at this stage which is specific to any particular user's deployment, because there are endless combinations which can viably be included which then pollute the core of the installation instructions. This includes adjustments to GOPROXY.
As a general rule, this encompasses things like:

reverse proxies (e.g nginx/apache/caddy/haproxy)
specific installation in certain OSes/environments/architectures (e.g raspi/cloud providers/plan9/solaris/windows)
service management (e.g k8s/ansible/systemd)
extraneous supporting services (e.g pgbouncer)

Aside from it adding noise to the installation instructions, it also becomes a maintenance burden for myself because these instructions end up getting out-of-date quickly, and are typically not tested. I'd rather not add that burden at all.


TypeError: Method Map.prototype.set called on incompatible receiver # at Map.set ()

My issue seems to be related to https://github.com/mikro-orm/mikro-orm/discussions/3795 but I was not entirely sure if it was a good idea to continue the discussion or to open a new issue (as it seems to have been fixed with a PR)

Describe the bug
Well, I've made a GitHub workflow to runs tests using GitHub actions before merging stuff on my main branch, those tests works fine on my computers (both my mac & windows) but when I do push my changes and when the workflow runs, it fails with the following error :
Stack trace
    TypeError: Method Map.prototype.set called on incompatible receiver #<Map>
        at Map.set (<anonymous>)

      22 | // So this runs before all tests of the suite
      23 | beforeAll(async () => {
    > 24 |      const moduleFixture: TestingModule = await Test.createTestingModule({
         |                                           ^
      25 |              imports: [AppModule],
      26 |      }).compile();
      27 |

To Reproduce
Steps to reproduce the behaviour:

Clone that repository (do not forget submodules)
Switch to the setup-mikrorm branch (yes there is a typo in the branch name my bad 😅)
Run pnpm i
Execute tests with pnpm test
See the error (If the error does not show at first, try to play with the detectOpenHandles option inside the jest.config.ts file)

Expected behavior
The workflow should run tests without crashing
Additional context

The GitHub Action that fails
The workflow file

The error seems to fix itself when doing pnpm jest --clearCache before running tests, but it does nothing on the workflow :/

Versions

Dependency
Version

node
20.4.0

typescript
5.1.6

mikro-orm
5.7.12

your-driver
PostgreSQL

jest
29.6.1

ts-jest
29.1.1

@nestjs/testing
10.0.5

@mikro-orm/nestjs
5.2.0

I saw you've set up the global setup feature of jest, try to remove that config line to see if it works again. Your db setup can be moved to before* hooks temporarily.


Fix: Dashboard UI tests

Failed tests from job/cypress/job/dashboard-tests/33:

ODS-1813: Verify Users Can Start, Stop, Launch And Delete A Workbench
ODS-1818: Verify Users Can Start, Stop And Launch A Workbench From DS Projects Home Page
ODS-2276: Verify RHODS Admins Can Import A Custom Serving Runtime Template By Uploading A YAML file

AFTER THE FIX:

I can see that the job that is referenced was run against the RHOAI 2.14rc1, thus is applicable for our current master branch. As such, I'm gonna merge this one rightaway to see the results from the nearest nightly job triggered.


Improve details summary polyfill

The current polyfill wasn't correctly detecting the “open” attribute and was firing twice in Chrome,
when the summary was clicked.
For now, it has two checks - first for native support, which uses the
boolean ‘open’ attribute.  Then for non-native support, which checks
for an ‘open’ attribute with an empty value, otherwise ‘open’ is
undefined (not present) and the details content should be closed.
For non-native support, an open or closed arrow (pointing right or down) is added, depending on the initial ‘open’ state. This arrow is hidden from assistive technology using aria-hidden.
A detailed explanation from Léonie:

The implicit role for  should be "group". Since most browsers haven't implemented support for the element (let alone accessibility support), we need to apply this role manually – so that assistive technologies know that it's a container for a group of related content.
…
[1] role="button *
The implicit role for  should be "button", but as with  we need to provide the accessibility support explicitly using the role attribute.
[2] aria-controls="details-content-0"

Note: aria-controls is not consistently supported by assistive technologies.
Those that do support it usually provide a keyboard shortcut that lets the user move focus to the start of the element being controlled. It is not expected behaviour for a screen reader to automatically start to speak the content of the disclosure widget when it opens.
It's also worth noting that VoiceOver/Safari doesn't support aria-controls in any case.

[3] tabindex="0" *
Without basic browser support for  it can't be focused on with the keyboard. Using tabindex="0" places the element into the natural tab order of the page, so as someone using a keyboard tabs through the page they'll be able to focus on it in order to activate it.
[4] aria-expanded="false" *
The aria-expanded attribute informs assistive technologies that the  element triggers the expansion/collapse of something. When focus first moves to the  element, its state is announced as "collapsed", and once the button is activated it changes to "expanded".

Really interesting summary. Thanks.

This is a great improvement and the summary of why the changes are needed is brilliant. :+1:

I've re-read the spec and my original presumption was incorrect so it's worth explaining why, mainly because it also explains why using aria-hidden is actually the advised approach.
In our use-case, the markup we tried for the arrow is:
▼

The ▼ is a text node which is a child of the  tag. The spec has this to say about how to handle descendants of elements given the role of 'presentation':
"...the user agent MUST expose content and descendant elements that do not have an explicit or inherited role of presentation."
The ▼ text node has a role of "text" so it will be exposed. This explains why the arrow was still being spoken when the  tag was given the 'presentation' role.


[bazel] Use the new rules for building ROMs

Rewrite sw/device/silicon_creator/rom/BUILD to use the new binary rules.
Add an output_groups rule that can select several output groups from an OutputGroupInfo provider and emit a DefaultInfo provider with all of the requested files.
Rework the packaging of the hello_world and ROM binaries to use the output_groups rule.

Fixes #19922.

Unfortunately, a few tests in rom e2e directly refers to rom_with_fake_keys_fpga_cw310.elf which is now inaccessible. You might to change those to rom_with_fake_keys_fpga_cw310_elf as well?


Update core/ chrono dependency version to 0.4.22, specify patch version

Currently core compile fails with chrono v0.4.19.
So, update the chrono dep version and specify the patch number.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 98.292%

Files with Coverage Reduction
New Missed Lines
%

test-suite/src/tester/mod.rs
1
89.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
35861

Relevant Lines:
36484

💛  - Coveralls


Add dynamic metatags to be read by seo4ajax bot

Step towards resolving #909
Likely will implement in same PR with #1337
Notes:

We don't have post images yet so we will use the same one as the main site.
Seems like 300 characters is good for description - https://stackoverflow.com/a/9552296 - we can take the first part of post body for now if longer

So for example https://staging.fightpandemics.work/post/5f04b73849aca117671ae4cd would have:
<!-- DIFFERENT THAN INDEX (dynamic except og:type)  -->
<meta property="og:type" content="article" />

<meta name="title" content="COVID-19 Open Innovation Grants from Protocol Labs">
<meta property="og:title" content="COVID-19 Open Innovation Grants from Protocol Labs" />
<meta property="twitter:title" content="COVID-19 Open Innovation Grants from Protocol Labs" />

<meta name="description" content="The Protocol Labs Request for Proposals (RFP) increases our impact by supporting community pursuit of solutions to important problems. It works on the basis of discrete calls directed to specific open problems, for which we solicit short applications. Applications are welcome from anyone, regardl…" />
<meta property="og:description" content="The Protocol Labs Request for Proposals (RFP) increases our impact by supporting community pursuit of solutions to important problems. It works on the basis of discrete calls directed to specific open problems, for which we solicit short applications. Applications are welcome from anyone, regardl…" />
<meta property="twitter:description" content="The Protocol Labs Request for Proposals (RFP) increases our impact by supporting community pursuit of solutions to important problems. It works on the basis of discrete calls directed to specific open problems, for which we solicit short applications. Applications are welcome from anyone, regardl…" />

<meta property="og:url" content="https://staging.fightpandemics.work/post/5f04b73849aca117671ae4cd" />
<meta property="twitter:url" content="https://staging.fightpandemics.work/post/5f04b73849aca117671ae4cd" />

<!-- SAME AS INDEX FOR NOW -->
<meta name="image" content="https://staging.fightpandemics.work/Offer-Help-Request-Help.png" />
<meta property="og:image" content="https://staging.fightpandemics.work/Offer-Help-Request-Help.png" />
<meta property="twitter:image" content="https://staging.fightpandemics.work/Offer-Help-Request-Help.png" />
<meta property="twitter:card" content="summary_large_image" />

FYI  @manualzuru does the above seem correct?
Not sure about the non og/twitter items like name="title"/name="description", not sure if this will be used by anything if the others exist but I guess it might not hurt?

Resolved by #1378


Decimals for units other than seconds

I'd like to have output like "12.4 days" or "6.5 weeks", but I can't figure out how to get this.
I tried { largest: 1, round: false } but it doesn't work. Is this possible?

Hmm, that seems like it should work.
I don't have lots of time to maintain this module, unfortunately—any chance you could attempt a pull request?

Running into the same issue. Happy to do a PR, but I want clarity into the problem and the potential solution.
const ms = 1000 * 60 * 60 * 1.5 // 1.5 hours

humanizeDuration(ms, {
    units: ['h', 'm'],
    largest: 1,
})
// Current output: "1 hour"
// Expected output?: "1.5 hours"

humanizeDuration(ms, {
    units: ['h'],
    largest: 1,
})
// Current output: "1.5 hours"
// Expected output: "1.5 hours"

Does the first call output a buggy result in your eyes?
Should these 2 calls return the same output?

That change seems reasonable to me.
This feels like a bugfix and not a breaking change, but are there any cases where it could arguably be a breaking change? If so, I'm more hesitant to make this change.

are there any cases where it could arguably be a breaking change?

If there are, then the existing tests are not comprehensive enough to expose breaking changes. I was surprised that all tests passed after my modifications. I'm happy to add tests for the proposed change, but I can't guarantee that there are no potential breaking changes.
The decision is yours.

Understandable. The fact that the existing tests stilled passed after my modifications is worrisome. I don't have the time to discover breaking changes, so it may be better to leave this alone.

I admit that this is the kind of thing that makes me not want to maintain this library any more—there is a large matrix of possible options that are hard to test exhaustively. (I'm sure it could be done by a more competent maintainer.) For example, what happens if you throw round into the mix here?
I'm inclined to keep things as is. However, if someone wants to fork this library to fix this bug and make some additional changes (like the one in #141), I'd link to that fork.

There hasn't been movement on this issue in several months so I'm going to close.
Feel free to let me know if you'd like me to reopen—I don't want to stop important discussion, I just want to clean up old issues.


[Feature] Support ImVoxelNet on KITTI

Reimplementation of ImVoxelNet for kitti-3d-car

In addition, maybe we can inherit the kitti base dataset config in the imvoxelnet config?
I think there are too much point-cloud related transforms and we need only images.

Also, do I need to add a unit test?

In addition, maybe we can inherit the kitti base dataset config in the imvoxelnet config?

I think there are too much point-cloud related transforms and we need only images.
Also, do I need to add a unit test?

OK, then we can keep the current format for configs.
Unit tests are needed, we'd better also add them in this PR.
I observe that the linting failed. You can refer to this guidance for useful pre-commit tools to avoid this problem.

I will also run training on 8 GPUs one more time to be sure that refactoring Anchor3DGenerator and ConvModule not much influenced anything.

I reverted anchor ranges, metrics are fine now. Can share model and log if needed.


Include support for multiple rules policy/vocabulary files

Completely overhaul the Deployment Framework's support for the BizTalk Rules Engine.

New features include:
Support for any combination of policy and vocabulary files; files with single ruleset/vocabulary and files with multiple rulesets/vocabularies
Support for adding policies to a BizTalk application's resources so that they show up in BizTalk Admin under Policies and integrate with BizTalk's app start/stop logic
Support for policies without vocabularies
Support for standard policy and vocabulary file naming and directory structure (e.g. $(ProjectName).BRE folder, similar to existing BAM support) OR fully customized file locations
Support for undeploying and unpublishing policies and vocabularies by name and version
Greatly improved informational and error messages in the script output
- Visual Studio Tools menu items for deploying and undeploying policies and vocabularies
Start menu shortcuts on server install for redeploying policies and vocabularies
Automatic packaging of all policy and vocabulary files into the server install MSI

Improved DeployBTRules.exe utility
This work item was migrated from CodePlex
CodePlex work item ID: '6084'
Assigned to: 'tfabraham'
Vote count: '1'

[UnknownUser@3/9/2010]
Resolved with changeset 37979.

[UnknownUser@3/10/2010]

[UnknownUser@3/10/2010]
Resolved with changeset 38079.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


Custom icons and file colours based on file name

Hi, In my Meteor.js project I have files that have different suffixes "_collection", "_view", "_container". It would be great if I could change the icon based on the file name. Also, do you have any idea if the colour of the filename can be changed as well ?
Something similar to this:

Great suggestion!
:+1:
It would be really great, so I can make Interfaces have another color, to have a bit of contrast!

@tomitrescak custom file associations has been already shipped. Custom colors will be also considered.

@robertohuertasm thanks for this wonderful update! ;) I already put it to the good use ;)
Just an idea here. What if you wouldn't use images to display icons but actual icons from fontawesome or others. This way, users would be free to modify the behaviour to their liking and use myriads of possible icons. Something in this ballpark: https://atom.io/packages/file-icons
Yet, since you have to modify actual vscode files to make this work, I guess it would not be easy :/
Thanks again!

@tomitrescak I've also explored this idea too but it's not the easiest path to follow at the moment. Microsoft is exploring similar ideas and willing to integrate the extension to their new coming API #10294

I'm closing this issue as it's partially completed. I've created #203 to explore file coloring.

I'm really confused. Does this functionality exist? I followed the link to #203 and was left even more confused.

@ciel check my comment in #946

Thank you so much. You're absolutely phenomenal. I am loving this extension SO much! It's made my work space so much cleaner and prettier.

Just to clarify, this issue was where the extension didn't used the official way, but we hacked inside to provide these icons, even then when vscode didn't supported them. This way we had the possibility to make anything we can imagine.(e.g. file coloring) And now we hope that vscode supports the things we had(considered to) implement(ed).


Allowlist polygon STARD-TUSD-wUSDR

Description
Replaces #3783
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 not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[x] Other

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Visual context
Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have requested at least 1 review (If the PR is significant enough, use best judgement here)
[ ] I have commented my code where relevant, particularly in hard-to-understand areas
[ ] If package-lock.json has changes, it was intentional.
[ ] The base of this PR is master if hotfix, develop if not

@dakotajakiser no initial liquidity has been provided, closing for now. If you do decide to add liquidity, feel free to re-open this PR.


Rest joystick option

Very good ! Thanks for that !
Can you run the build and bump the version as well please?

There we go. 😃

Thank you, I'll test it later today, and I'll merge it !

I've merged your two first commits, I'm prepping another version !
Thank you so much for your help!


Rethink UX of search results

Underline on title when hovering over the top-half portion of the card (DOI, title, summary)
Underline on "Details, services ..." when hovering over that particular button
Focus outlines on title link and "Details, services ..." button

Expanded result:

I think the provider was a bit difficult to see in the bottom corner. I've swapped the release date and provider and made the provider bold. Let me know what you think:

Thanks! :blush:


how to get juice wrld in fortnite

Issue Title: how to get juice wrld in fortnite Babylon.js
Issue Description:
I'm exploring ways to create a Juice WRLD-themed interactive experience or event in Fortnite-style using Babylon.js. The goal is to replicate vibrant, immersive visuals and music-reactive elements that align with Juice WRLD's aesthetic. Could Babylon.js support integrating interactive animations, dynamic lighting, and physics-based character movements for such an experience? Any guidance on implementing 3D character models, custom environments, or syncing audio with visuals would be helpful. Additionally, are there existing Babylon.js plugins or tools that could simplify this process? Looking forward to suggestions and documentation references to kickstart this project. Thank you!

Like the template mentions, please use our forum for questions but it should be all doable.


Readme seems to have small error

In the example code, command.add('M',doMotor,'motor'); should probably be command.add('M',doMotor,"motor");
I was getting compilation errors until I made this change.

Not sure why you are havving problems ,,,,, in Python  ' ' and  " "  can be used and are exactly the same


[SPARK-45485][CONNECT] User agent improvements: Use SPARK_CONNECT_USER_AGENT env variable and include environment specific attributes

What changes were proposed in this pull request?
With this PR similar to the Python client the Scala client's user agent now:

Uses the SPARK_CONNECT_USER_AGENT environment variable if set
Includes the OS, JVM version, Scala version, and Spark version

Why are the changes needed?
Feature parity with the Python client. Better observability of Scala Spark Connect clients.
Does this PR introduce any user-facing change?
By default, the user agent string now contains more useful information.
Before:
_SPARK_CONNECT_SCALA
After:
_SPARK_CONNECT_SCALA spark/4.0.0-SNAPSHOT scala/2.13.12 jvm/<IP_ADDRESS> os/darwin
How was this patch tested?
Tests added & adjusted.
Was this patch authored or co-authored using generative AI tooling?
No.

My bad, there was a PR merged with a wrong PR ticket. I switched the ticket each other.

FYI @grundprinzip

Lemme merge this as it seems pretty consistent with the exiting codebase.

Merged to master.


Change boot and rpc version to release

change rpc version to 5.4.6
Motivation:
Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #93   +/-   ##
=========================================
  Coverage     55.04%   55.04%
  Complexity      393      393
=========================================
  Files            53       53
  Lines          1733     1733
  Branches        308      308
=========================================
  Hits            954      954
  Misses          600      600
  Partials        179      179

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 1ad2fe3...0f993cb. Read the comment docs.


Add a property to control assets root directory

Adds the root property to Assets service, that defaults to current working directory
Adds method root() to  Assets to get value of the property
Adds method set_root()  to  Assets to set value of the property
Adds method import_from  to  Assets to keep possibility to import file from anywhere

For iOS applications some startup system should be considered, for example:
fn startup(mut assets: Mut<Assets>) {
  // before loading any asset from FS
  assets.set_root(
    std::env::current_exe()
      .expect("Executable path to be available")
      .parent()
      .unwrap()
  );
}

Based on the PR #145 from @russellwmy


Avoid sending avatar URLs of long term idle users to clients

This adds a new property exclude_old_users_avatars which is sent from clients to the server.
The purpose is to avoid sending avatar URLs of the long term idle users to the clients regardless of the client_gravatar value. This will reduce the user payload size even if email visibility is limited to admins.
This approach will be more efficient especially for realms with 10,000+ users like chat.zulip.org.
Fixes #15287.
Marked as WIP because this PR requires one more commit to enable this feature on the web client.

The name exclude_old_users_avatars isn't great; this is a "client capability", so it should describe what the client can do.  "user_avatar_url_field_optional` seems more appropriate -- i.e. the client has support for the the avatar_url field of user objects being optional.
We should document this in the client_capabilities section of zerver/openapi/zulip.yaml, and bump API_FEATURE_LEVEL (etc.) to document the change.

Other than that, and some style cleanup (do a careful audit on the Python code), I think we can merge the backend.

@timabbott please have a look. It looks like @zulipbot is currently down, so the "has conflict" tag is not being removed.

This is great!  I fixed the following:

Added some longer block comments explaining the feature.
Added an entry in templates/zerver/api/changelog.md
Reordered arguments to be more readable (having target_user in between the client options felt really weird)

And merged this as 5200598a31a795a245ad2e79de07c72718b59b2e.  I used the following logic to test this on a browser frontend:
diff --git a/static/js/people.js b/static/js/people.js
index 0d7255b0ba..07e5d0015f 100644
--- a/static/js/people.js
+++ b/static/js/people.js
@@ -604,7 +604,10 @@ exports.small_avatar_url_for_person = function (person) {
     if (person.avatar_url) {
         return exports.format_small_avatar_url(person.avatar_url);
     }
-    return gravatar_url_for_email(person.email);
+    if (person.avatar_url === null) {
+        return gravatar_url_for_email(person.email);
+    }
+    return exports.format_small_avatar_url('/avatar/' + person.user_id);
 };

 exports.sender_info_with_small_avatar_urls_for_sender_ids = function (sender_ids) {
@@ -647,17 +650,23 @@ exports.small_avatar_url = function (message) {
         return exports.format_small_avatar_url(message.avatar_url);
     }

-    // For computing the user's email, we first trust the person
-    // object since that is updated via our real-time sync system, but
-    // if unavailable, we use the sender email.
-    let email;
-    if (person) {
-        email = person.email;
-    } else {
-        email = message.sender_email;
+    if (person.avatar_url === null || message.avatar_url === null) {
+        // For computing the user's email, we first trust the person
+        // object since that is updated via our real-time sync system, but
+        // if unavailable, we use the sender email.
+        let email;
+        if (person) {
+            email = person.email;
+        } else {
+            email = message.sender_email;
+        }
+
+        return gravatar_url_for_email(email);
     }

-    return gravatar_url_for_email(email);
+    // If we don't have an avatar_url at all, this is what the
+    // client_capability is for.
+    return exports.format_small_avatar_url('/avatar/' + person.user_id);
 };

 exports.is_valid_email_for_compose = function (email) {
diff --git a/zerver/lib/users.py b/zerver/lib/users.py
index 181b635dd2..727319eb4c 100644
--- a/zerver/lib/users.py
+++ b/zerver/lib/users.py
@@ -347,7 +347,7 @@ def format_user_row(realm: Realm, acting_user: UserProfile, row: Dict[str, Any],
     # bandwidth).  At present, the server looks at `long_term_idle` to
     # decide which users to include avatars for, piggy-backing on a
     # different optimization for organizations with 10,000s of users.
-    include_avatar_url = not user_avatar_url_field_optional or not row['long_term_idle']
+    include_avatar_url = not user_avatar_url_field_optional # or not row['long_term_idle']
     if include_avatar_url:
         result['avatar_url'] = get_avatar_field(user_id=row['id'],
                                                 realm_id=realm.id,
diff --git a/zerver/views/home.py b/zerver/views/home.py
index 3976b15073..84e66e06b7 100644
--- a/zerver/views/home.py
+++ b/zerver/views/home.py
@@ -192,6 +192,7 @@ def home_real(request: HttpRequest) -> HttpResponse:
     client_capabilities = {
         'notification_settings_null': True,
         'bulk_message_deletion': True,
+        'user_avatar_url_field_optional': True,
     }

     register_ret = do_events_register(user_profile, request.client,

We want to use this feature in the Zulip; doing so will save a lot of time in loading chat.zulip.org from slower networks.  Can you prepare a cleaned up version of that patch with proper tests?
Because messages contain avatar_url data for the senders, the easiest way to see this change in action is in the compose box typeahead using avatar URLs there.
Thanks @hashirsarwar!


write: support xz compression

It's more CPU-intensive, but results in much smaller ACIs.

The reason I didn't add it when initially adding the compression support was because it'd add the dynamic library dependency. I don't like the compression handling that shells out to xz, but the dynamic linking then makes it required on the host.
One thing that may be worth trying though would be to make the pre-compiled binaries with each release be statically compiled...

I'd be in favor of exploring statically compiled releases. It'd also help prevent other issues like when I accidentally build releases on NixOS: https://github.com/appc/acbuild/issues/133. There does appear to be a pure golang xz package, but it isn't stable so it's probably not worth it: https://github.com/ulikunitz/xz.

IIRC statically linking is probably not feasible because of resolver issues.
But in any case I don't understand how that relates exactly to the xz case? What does shelling out have to do with dynamic linking? unless you're suggesting baking a copy of xz into the releases or something..


Staff View: Dissolution Filing SP/GP#11411 | comments

Issue #: /bcgov/entity#11411
Description of changes:

Added staff comments
Redirection for SP/GP filing from App.vue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Also, you need to fix the 2 failing checks before this is merged,


Release Version 2.0.21

Features

Allow users to override _total=estimate (#2091)
Debian build script (#2089)
Fixes #2074: Add Bulk Export to SDK (#2071)
bulk import command for Medplum CLI  (#2097)
Create new parser for StructureDefinitions into schema representation (#2119)
CLI Export from External FHIR Server (#2102)
Added CDK cacheNodeType setting (#2132)
Fixes #2136 - Add options to executeBatch (#2137)
Fixes #2122 - Add CloudFront logging to CDK config (#2135)
Fixes #2133 - Added Task-due-date search parameter (#2141)
Fixes #2108 - support large file bulk import in CLI (#2142)

Changes

Copy commit message to example repos (#2078)
Fixes #2100 - support reference strings in _compartment search (#2101)
Add Outlet components to ResourcePage in the examples/medplum-hello-world (#2096)
Enabled eslint 'curly' rule (#2120)
Fixes #2123 - enable S3 bucket versioning by default (#2129)

Bug fixes

Fixes #2084 - handle malformed value in CodeableConceptInput (#2086)
Suppress debug console output in tests (#2085)
Clear header search (#2094)
Include login param for SignInForm when used with external auth (#2095)
Fixes #1413 - Clear ErrorBoundary on url changes (#2087), thank you @Aquilaafuadajo
Fixes #2109 - Handle invalid FHIRPath expression in csv export (#2113)
Fixes #2114 - prevent reset password enum attack (#2115)
Add foomedical to examples (#2099)
Fix double status bug in SignInForm (#2134)
Fixes #2130 - use Communication.sent in resource timeline (#2144)

Docs

Update install-on-ubuntu with postgres 12 notes (#2090)
Adding g10 blog content (#2103)
Fixes #2121 - Added NIST cloud security guidelines to docs (#2128)

Coverage: 94.324%. Remained the same when pulling 87271fa16eb5593a962a8aaa7a6ef43e9baac36d on cody-version-2.0.21 into 3216a34765419af00da03d63873655c31222f149 on main.


Dimensions need to use parseInt

Invalid prop: type check failed for prop "width". Expected Number with value 100, got String with value "100".
  at <LineChart width="100" height="100" chartData= ...

Sometimes the props are set directly in which case the chart needs to handle string as number on its own.

No need for parseInt. You can pass number in props just by doing <LineChart :width='100'/>

i know i can, it does not mean i should be forced to use it when standard props do not require programmatic parameters.


Unfortunately, Android Keyboard (AOSP) has stopped

Describe the bug
I reflashed the Comma 2. It was successful, however, now I can't use the android: This pops up "Unfortunately, Android Keyboard (AOSP) has stopped."
No matter how many times I reflash from the github.
What hardware does this issue affect?
comma two
Provide a route where the issue occurs
I am reflashing the Comma 2, following the instructions on the github, and have it reflashed, and upon turning on, it says "Unfortunately, Android Keyboard (AOSP) has stopped."
openpilot version
0.8.10
Additional info
No response

have you tried restarting device ?

Have you tried reflashing the device with https://github.com/commaai/eon-neos?

Closing this since it's not an openpilot bug. If you still have the issue after reflashing NEOS, check out the community discord for help.


Prep release 3.1.7

Codecov Report

Merging #5083 (20285a7) into master (4cacf26) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5083      +/-   ##
==========================================
- Coverage   95.84%   95.83%   -0.02%
==========================================
  Files         667      667
  Lines       66466    66466
==========================================
- Hits        63704    63695       -9
- Misses       2762     2771       +9

Flag
Coverage Δ

servertests
36.62% <ø> (ø)

unittests
95.77% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/cognitoidp/models.py
94.91% <ø> (ø)

moto/resourcegroupstaggingapi/models.py
83.18% <0.00%> (-0.59%)
:arrow_down:

moto/route53/responses.py
96.69% <0.00%> (-0.42%)
:arrow_down:

moto/sns/models.py
96.83% <0.00%> (-0.40%)
:arrow_down:

moto/efs/models.py
91.20% <0.00%> (-0.30%)
:arrow_down:

moto/sqs/models.py
96.76% <0.00%> (-0.18%)
:arrow_down:

moto/config/models.py
99.22% <0.00%> (-0.13%)
:arrow_down:

moto/ssm/models.py
94.42% <0.00%> (-0.11%)
:arrow_down:

moto/elbv2/models.py
92.58% <0.00%> (ø)

moto/elbv2/responses.py
99.43% <0.00%> (ø)

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 4cacf26...20285a7. Read the comment docs.


Add myself to the contributors

It's an honor to help you folks with this awesome repository 💙

💖


When the client's clean_session is set to false, after a normal disconnection, but the start_keep_alive_timer method is already running？

Sounds like a long exists bug, will check this out.


Add missing (implied) .git suffix in the "git clone" command

(a detail in the README.md template)

This is fine to add for precision, but doesn't it work without the .git as well on nearly any reasonable Git version?

Maybe I'm wrong but I believe the fact it "works" with both URLs is orthogonal to Git (I didn't see any mention of .git autocompletion in man git clone, actually .git is the standard naming convention for bare repos); so I guess this is implemented by GitHub as a redirection of the truncated URL (normally used for the web application GUI) to the full Git URL.

I see. Well, since we may well use different platforms in the future, I guess this is reasonable to have.


last_vacuum/analyze : raise an error if last_... & last_auto... are both NULL

The service does not return an error when some tables have last_${type}  last_auto${type} both at NULL.
I have a database where the autovacuum analyze may be stuck for days (there is a bug in the function of a functional index) and it leads to null values in these fields. But any other table with valid values hide this and the database-wide last_analyze is okay. This is the only way I can think of to detect this problem through check_pgactivity.
I don't see why last_${type} and last_auto${type} may both be NULL, except after a crash or table creation, and this should not last long.

Probably solved by #249 / #256

Hi,
I'm not sure to understand the original problem. To be honest, I did not invest a lot of time on it yet. But, as it seems you are able to reproduce the problem on your side...may I ask you to check if #249 / #256 actually fixed your issue?
Many thanks,

I can't reproduce, this seems solved. Case closed.


Element not visible when running test using testcafe docker image but visible in local (Google login page)

What is your Test Scenario?
I'm trying to login into https://accounts.google.com/signin/v2 through a pipeline using the testcafe docker image in Drone.io
What is the Current behavior?
Locally, I can login without any issue, but when running it in a pipeline (Drone.io) and clicking the Google's next button after entering the email, nothing seems to happen (also tried with pressKey('enter')), since the password field never becomes visible.
What is the Expected behavior?
After entering the email and clicking on the next button, the password field should be displayed, so that a successful login happens.
What is your web application and your TestCafe test code?

https://accounts.google.com/signin/v2

Your complete test code (or attach your test files):

LOGIN PAGE:
 import { t, Selector } from "testcafe";

class LoginPage {
  constructor() {
    this.usernameGoogleText = Selector("input[type='email']");
    this.passwordGoogleText = Selector("input[type='password']");
    this.loginWithGoogleButton = Selector("div")
      .withText(/log in with google/i)
      .parent("a[type='button']");
    this.continueGoogleButton = Selector("span")
      .withText(/siguiente|next/i)
      .parent("button");
    this.twoStepVerificationLabel =
      Selector("span").withText(/2-step verification/i);
  }

  async loginWithGoogle(username, password) {
    await t
      .click(this.loginWithGoogleButton)
      .typeText(this.usernameGoogleText, username)
      .click(this.continueGoogleButton)
      .typeText(this.passwordGoogleText, password)
      .click(this.continueGoogleButton)
      .expect(this.twoStepVerificationLabel.exists)
      .notOk({ timeout: 60000 });
  }
}

export default new LoginPage();

TEST
import { BASE_URL, CREDENTIALS } from "../data/constants";

import { LoginPage } from "../pages";

fixture("Login feature").page`${BASE_URL}`;

test("Log in with a regular Google user", async (t) => {
  await LoginPage.loginWithGoogle(
    CREDENTIALS.REGULAR_USER_NAME,
    CREDENTIALS.REGULAR_USER_PASSWORD
  );
});

PIPELINE CONFIG FILE
  - name: e2e-tests
    image: testcafe/testcafe
    failure: ignore
    environment:
      TESTCAFE_SLACK_WEBHOOK:
        from_secret: testcafe_slack_webhook
      AUTOMATION_USER_NAME:
        from_secret: automation_user_name
      AUTOMATION_USER_PASSWORD:
        from_secret: automation_user_password
    commands:
      - /opt/testcafe/docker/testcafe-docker.sh chromium packages/e2e/src/tests/*
  when:
    branch:
      - develop
      - feature/*
      - hotfix/*
      - support/*
      - bugfix/*
      - feature_*
      - hotfix_*
      - test/*

Any google credentials can be used

Your complete test report:
```
+ /opt/testcafe/docker/testcafe-docker.sh 'chromium --no-sandbox --disable-dev-shm-usage' packages/e2e/src/tests/*
--
2 | Running tests in:
3 | - Chrome 90.0.4430.93 / Linux 0.0
4 |
5 | Login feature.
6 |
7 | ✖ Log in with a regular Google user
8 |
9 | 1) - Error in Role initializer -
10 | The element that matches the specified selector is not visible.
11 |
12 | Browser: Chrome 90.0.4430.93 / Linux 0.0
13 |
14 |
15 | 19 \| .click(this.loginWithGoogleButton)
16 | 20 \| .typeText(this.usernameGoogleText, username)
17 | 21 \| .wait(3000)
18 | 22 \| .click(this.continueGoogleButton)
19 | 23 \| .wait(3000)
20 | >>> 24 \| .typeText(this.passwordGoogleText, password)
21 | 25 \| .wait(3000)
22 | 26 \| .click(this.continueGoogleButton);
23 | 27 \| await t
24 | 28 \| .expect(this.twoStepVerificationLabel.exists)
25 | 29 \| .notOk({ timeout: 60000 });
26 |
27 | at <anonymous>
28 | (/drone/src/packages/e2e/src/pages/Login.page.js:24:8)
29 | at asyncGeneratorStep
30 | (/drone/src/packages/e2e/src/pages/Login.page.js:1:251)
31 | at _next (/drone/src/packages/e2e/src/pages/Login.page.js:1:589)
32 | at <anonymous>
33 | (/drone/src/packages/e2e/src/pages/Login.page.js:1:754)
34 | at <anonymous>
35 | (/drone/src/packages/e2e/src/pages/Login.page.js:1:495)
36 | at LoginPage.loginWithGoogle
37 | (/drone/src/packages/e2e/src/pages/Login.page.js:17:45)
38 | at <anonymous>
39 | (/drone/src/packages/e2e/src/helpers/roles.js:6:19)
40 | at asyncGeneratorStep
41 | (/drone/src/packages/e2e/src/helpers/roles.js:3:134)
42 | at _next (/drone/src/packages/e2e/src/helpers/roles.js:3:472)
43 | at <anonymous>
44 | (/drone/src/packages/e2e/src/helpers/roles.js:3:637)
45 |
46 |
47 |
48 | 1/1 failed (39s)
```

Screenshots:

### Steps to Reproduce:

Executing through a pipeline:

Go https://accounts.google.com/signin/v2
Enter google email
Click on the next button

(Basically just executing the test pasted above)
Your Environment details:

testcafe version: testcafe image 1.14.2
node.js version: testcafe image 1.14.2
command-line arguments:  /opt/testcafe/docker/testcafe-docker.sh chromium:headless packages/e2e/src/tests/*
browser name and version: Chrome 90.0.4430.93
platform and version:          Linux 0.0
other:  everything was run inside a docker container with the testcafe image v1.14.2 through Drone.io

@edgarmfarias
Hello,
There is a platform independent issue related to the Google login. You received a security message after clicking the continue button. I'll close this issue as a duplicate. You can track the original thread.


Cleaned up browser errors.

Kenan,
This is a small pull request to get rid of the errors regarding undefined modules/require in browsers; no sense trying to require stuff unless require exists.
If you don't mind pushing these changes to the npm modules as well, I would much appreciate it.
Thanks,
Alex

If you want to use this in the browser you should use browserify, webpack, etc., no? Otherwise PRs like this would be necessary for thousands of other modules.

I agree with that assertion. However, I was operating under of the premise of this being akin to a polyfill for cryptographic random across browsers and node, especially since the dependencies are purely for node.js and it is relatively short. In my use case, it makes less sense to require browserify or webpack to include get-random-values in my project, as my other dependencies export to browsers nicely already without browserify.
However, I understand if you don't wish to merge this as I can throw in browserify as easily as you can merge.

@cipherboy Yeah, I think I would be more willing to accept a PR that added a prepublish script that browserifies index.js to something like browser.js which exports to window. Seems a bit silly that I would rather add a new file to the published tarball rather than add a few lines but in my opinion it keeps the source code cleaner.


Redesign dependencies page

Summary
Mainly I do the following two things in this PR.

Detach Dependencies from DiscoverPage.
Refactor Dependencies code.

Details
Detach Dependencies from DiscoverPage
Dependencies are currently embedded in the Discover Page as follows:

At first glance, this design seems to share GlobalSearch between traces searching and dependencies searching.
Actually, however, Dependencies search uses only Lookback search criteria.

I thought this would be very confusing to users.
And, I think that the topology view considered here(https://github.com/openzipkin/zipkin/issues/2701) should be placed where Dependencies is now.
Therefore I want to detach dependencies from DiscoverPage.
Refactor Dependencies code
Currently, the code of Dependencies is the oldest and worst code in Lens.
I want to refactor it.
Screenshots

Current State:

@tacigar so let's add a description. say what you intend to do, why you are doing this, and why you love it!

@openzipkin/ui another epic awesome on the way

@adriancole Sorry, I added description!

Thanks for the summary. The only thing I might suggest is that #2537 needs to fit somewhere.. would the filtering by service be a part of a separate screen? If not, wouldn't we want to share the context here?

In a disconnected view like this, it would seem useful to me to have a way to link off to the traces view from a selected service, retaining the used from/to time parameters. Maybe inside the sidebar that opens when selecting a node?

Thanks for the summary. The only thing I might suggest is that #2537 needs to fit somewhere.. would the filtering by service be a part of a separate screen? If not, wouldn't we want to share the context here?

+1, these changes are great but doesn't solve the issues when there are a huge cluster of services in one graph

@anuraaga Thank you for your kind reviews!

In a disconnected view like this, it would seem useful to me to have a way to link off to the traces view from a selected service, retaining the used from/to time parameters. Maybe inside the sidebar that opens when selecting a node?

@bulicekj Thank you for your feedback!
I tried to implement that as follows.

And I tried to do the same hack as https://github.com/openzipkin/zipkin/pull/2537 in the new dependencies page.

@bulicekj Thank you so much!

PTAL 🙇

@adriancole Thank you so much!
You are my teacher! :)

PTAL 🙇

using yelp.json I cannot read the labels.. they look obfuscated (firefox)

same in safari. so it seems zooming in doesn't change the size of the labels it only makes the text blurry. I wonder if the font is not scaling for some reason. Possibly the font size for the dependency graph should be a function of how many nodes exist?

Thank you for your report.
I don't know the reason, but I think the root cause is in third party libraries (Vizceral? Three.js?)
Anyway, I don't know the solution at all, so I want to solve it later.
If the browser's zoomup is between 80% and 120%, we can read the label correctly.
So I don't think this is basically a big problem...

if the label size was larger, I'd be ok. right now I can barely read them

I can also run the previous just I don't recall the label text being so small before.. maybe it was..

@adriancole Please tell me your browser's zoom percentage?

The animated gif started with no zoom level. I will try to post a higher resolution screenshot of zero zoom level.

reason I try zoom was the labels were small and I couldn't read :P

I see. thank you.
I investigated it, but I could not found way to adjust the font size.
My best guess is the font size is fixed....
https://github.com/Netflix/vizceral/blob/45c8c49c7210e883d357ffb28b03de46132da6f3/src/base/nodeNameView.js#L86

I see. anyway I can read as long as I don't attempt zoom as it makes them worse.

I haven't tracked the comments from @anuraaga, but if they are considered, I'm fine to merge.

@anuraaga thank you for your review!

great team work!


Infrastructure for unit tests

Is your feature request related to a problem? Please describe.
While working on a PR to improve server side rendering (#1091) of the index page, we discovered there's no infrastructure/skaffolding for unit tests.
Describe the solution you'd like
Basic infrastructure and CI integration for unittests, something like pytest, so we can rapidly iterate on backend features without breaking compatibility, fostering better collaboration and API stability.
Describe alternatives you've considered
Using Cypress E2E testing to test backend features. Integration tests are great but it overcomplicates things when it comes to clear backend features.
Additional context
https://circleci.com/blog/unit-testing-vs-integration-testing/

Ref: https://docs.pytest.org/en/latest/explanation/goodpractices.html


regression function

Here are the regression functions I wrote.
Need to use Ben's new event2neural function and merge_cond function.
These works on my computer, but somebody should check if things make sense. Otherwise, I'll look more carefully at these codes over the weekend.

Ok, should we see if #34 can be merged first? b/c of eventstoneural_extended
Also:
Question: Do you think we can figure out how to plot multiple plots in on figure? This way we dont need to output/save so many files, such as the the massive for loops we have in plotting meanSig, fd, dvars; will probably try to implement this sometime tomorrow if people agree

As for figures, I guess we can add a boolean on whether save of not (like I think you already did), cuz figures are just helpful for visualization. We don't necessarily need the figures to identify outliers.
But by plotting them together, do you mean if we can have 3 subplots side by side in 1 figure? I think it's do-able, we just need to figure out how. It's probably something like, fig = plt.figure(); ax = fig.add_subplot(131); fig.add_subplot(132); fig.add_subplot(133)... we would need to google around for that.

The reason I dont really like saving things inside functions is because it is hard to write test function for them. But yeah boolean would be good, if false we can just return the values instead.
Yes, 3 subplots side by side works, or can do even more and create a lattice of subplots, yeah have to look into how to do that but I think I might be able to steal some ideas off of other groups, lol

Awesome work! Just out of curiosity, is there an advantage to doing it this way and not using something like this?
http://statsmodels.sourceforge.net/devel/regression.html

Updated the outliers functions, so now they should be stored in a txt file like a python dictionary.
Updated the graph functions to label outliers.
To Brian's suggestion: I was doing the regression like the hw, so I just didn't search around for packages. But I guess we could use already existed packages for regressions. Let me google around and try it.

If both methods work, we can use one way as a test function for the other way

i can write the test functions for regression scripts if you havent done so already

That would be great, cuz I haven't done it yet.

Minor error, the regression_script.py calls in behavtask_tr.py, but it's in utils and so there's a path error

moved all the codes to the utils folder and changed some paths in the codes accordingly. Hopefully, all the path are correct now.

Here are some more minor errors I found:
REGRESSION_FUNCTIONS.PY
-regression_functions.py needs to import events2neural_extend and merge_cond at the top
-in getGainLoss function, hrf_at_times is undefined. Did you mean to set hrf_at_times = hrf(tr_times) in the script? I see you define hrf_at_times in the regression script, but it won't recognize since hrf_times in getGainLoss is local and not global. I recommend we define hrf_times within the function itself or something similar
-in getGainLoss all_tr_times is defined but never used. all_tr_times is again defined in a different scope in the script. Let's pick one and stick with that
-in calcBeta and calcMRSS, data is again undefined. We should define data within the function, because right now we're defining data outside of that scope and doing it in the script, but i don't think it will run correctly because of that.
In general perhaps it would be better practice if we made data an argument to the functions and passed it in through there. Same with all_tr_times
REGRESSION_SCRIPT.PY
behavtask_tr.py should just be behavtask_tr i believe
I can clean this up and make a new pull request if you're pressed for time, but I wanted to check in and see if there was a reason for defining data and all_tr_times outside of the functions before making changes. Also I think Ben's thoughts on changing the parameter from the file to actual arrays is good

Sorry about those, let me take a look and fix these.

Actually, do you have the problem that if you don't specifically import numpy as np inside a function, the function won't find np?

hmmm no i don't have that problem. what's the exact wording of the error? import numpy as np is at the top of everything so it should be fine right

I fixed these functions again. Can someone check these functions?

Let me know when you get a chance to update these, I have some test functions written that we can check.

Updated the function with X design being the task condition data.
Also, switched the events2neural and merge_cond function into the script rather than in another function.
I checked a couple diagnostic stats, still pretty bad. Jiang is working on the inferences testing. We'll get some result tmr or Friday.

Also, here are some code if we try to use sklearn or statsmodels.api package.
Didn't want to pull request on this.
from sklearn import datasets, linear_model
X, Y are constructed like the way in calcBeta(data, gain, loss).
X = np.ones((len(gain), 3))
X[:, 0] = gain
X[:, 1] = loss
designp = npl.pinv(design)
T = data.shape[-1]
Y = np.reshape(data, (-1, T)).T
Create linear regression object
regr = linear_model.LinearRegression()
Train the model using the training sets
regr.fit(X, Y)
regr.coef_
regr.intercept_
import numpy as np
import statsmodels.api as sm
Single Y
YY = Y[:,40000]
mod = sm.OLS(YY, X)
res = mod.fit()
print res.summary()

ok I will use this to write test functions for regression

思瑶：
sorry，我不是很会传github,明天一定好好学学。我写好了那个check的function,同时regression full
也有小点点修改。我把两个文件先发给你。
On Thu, Nov 19, 2015 at 12:46 AM, changsiyao<EMAIL_ADDRESS>wrote:

Also, here are some code if we try to use sklearn or statsmodels.api
package.
Didn't want to pull request on this.
from sklearn import datasets, linear_model
X, Y are constructed like the way in calcBeta(data, gain, loss).
X = np.ones((len(gain), 3))
X[:, 0] = gain
X[:, 1] = loss
designp = npl.pinv(design)
T = data.shape[-1]
Y = np.reshape(data, (-1, T)).T
Create linear regression object
regr = linear_model.LinearRegression()
Train the model using the training sets
regr.fit(X, Y)
regr.coef_
regr.intercept_
import numpy as np
import statsmodels.api as sm
Single Y
YY = Y[:,40000]
mod = sm.OLS(YY, X)
res = mod.fit()
print res.summary()
—
Reply to this email directly or view it on GitHub
https://github.com/berkeley-stat159/project-theta/pull/50#issuecomment-157991143
.


How to add home and about page

I was roaming around the web in the search of a simple yet attractive blogging site with netlify cms . this site is all i want . I configured it according to me and it looks awesome.but i am facing one issue i want to add a home and about page to my site. How can i do so ? Thank you in advance!

Hi @thekamalkashyap, there is already a home page: https://github.com/ixartz/Eleventy-Starter-Boilerplate/blob/master/src/index.ejs
You can just add a new file inside this directory: https://github.com/ixartz/Eleventy-Starter-Boilerplate/tree/master/src


Error "Symbol '__start' not found" with my asm code

I use nasm to compile "demo.asm" to "demo.o"
then I try "smlrl -tiny demo.o -o demo.com"
and I get the Error "Symbol '__start' not found"

Duplicate of https://github.com/alexfru/SmallerC/issues/37 or https://github.com/alexfru/SmallerC/issues/17?

ok
I fixed my code
section .text 	global	__start __start: 	ret
It's working,thank


Script Error - Automation-First-Runbook-Textual.md

In the following article, Step 7.1, there is an error on the script

It should be: Start-AzureVm -Name $VMName -ServiceName $VMServiceName
https://github.com/Azure/azure-content/blob/master/articles/automation/automation-first-runbook-textual.md

You example is from step 6, but yes, 7.1 should use the workflow parameters.

Sorry @BernieWhite , I copied it wrong!


Add clarification about using F method 4 with detailed F values

Resolves issue #266 and linked to ss3-source-code PR #644.

@Rick-Methot-NOAA sadly that addition of the multicolumn numbers into the pdf is a lualatex issue that I don't think should be solved in this PR. I will have to work with Kelli on it sometime later this week or when I get back from leave the first week of December because it might take some time to sort out what is going on.

pdf looks good.  ready to merge when other checks are passing


AbstractMavenPublisher results in lost System.out

Expected Behavior
During the build lines printed to System.out reach gradle stdout.
Current Behavior
Observed only during our longest and terriblest builds, System.out will sometimes "flicker" (alternating between making it to gradle stdout and not) and ultimately stop coming through.
Context
Nope.
Steps to Reproduce (for bugs)
I tried to reproduce this with simpler cases but I just could not get the timing.  The data I can give based on debugging and print instrumenting our long and terrible builds that reproduce this:
*) Multiple instances of StartableLoggingSystem exist, all pointed at the same DefaultStdOutLoggingSystem.
*) Each start() saves the previous log level and sets it to some specified value, and each stop() resets it to the saved value.
*) start/stop are called on these in a manner that is "consistent" per StartableLoggingSystem instance, but not for the shared DefaultStdOutLoggingSystem as a whole (e.g.  you can have start1, start2, stop1, stop2).
*) Almost all of these set log level QUIET so it sort of doesn't matter, all saved values are QUIET.
*) But!  AbstractMavenPublisher.execute produces one at INFO.
*) An interleave happens like:  start1 INFO, start2 QUIET (saves INFO), stop1 INFO (restores original), stop2 QUIET (restores INFO!)
*) System.out events from then on get level INFO which means they're dropped later (some later piece of logging infrastructure has threshold LIFECYCLE which passes QUIET and drops INFO).
Your Environment
This is all as of gradle 4.10.2.  It's unfortunately a long way off to upgrade our long and terrible builds to newer gradle and as mentioned, I failed to engineer a free-standing repro so I can't really test newer gradle.

trace.txt
This is the logging I got from StartableLoggingSystem during the relevant part with commentary added on the important lines.

@AmlingPalantir could you try the latest nightly? I think this may be fixed.
You can upgrade with gradlew wrapper --gradle-version=5.1-20181126000019+0000

Unfortunately our build is on the complex side and is unlikely to just work with a newer version of gradle.  I'll look into it, but in the mean time, was there a specific change you thought might have fixed it?

Oh, obviously 7bc44d5b1ec0.  Nm.  Upgrade still ... in progress.

There was some logger juggling that was starting/stopping the logger system while publishing that looked unnecessary.

Well, that was an adventure.  I'm not sure we'll be able to upgrade our actual build right away, but I was able to get enough of it running and use the same process to both reproduce the issue on 5.0-20181121000205+0000 and not reproduce the issue on 5.1-20181126000019+0000.

Based on the comment above, I'm going to close this as fixed.  If you feel like there's still something that needs to be addressed here, we can always re-open it.


Change partion from Fat32 to Fat16

Fix #2
Creates a Fat16 partition instead a Fat32, so both Amiga and MacOS can mount the first partition

Not sure this is needed.  Fat32 can already be ready by Amigas and MacOS

MacOS does not recognize the type 0x0C, after changing the type to 0x0E I could access to the partition with no changes in the OS... It was some time ago when I did this change, but I think I had some similar problem with Amiga OS 3.1.

MacOS does not recognize the type 0x0C, after changing the type to 0x0E I could access to the partition with no changes in the OS... It was some time ago when I did this change, but I think I had some similar problem with Amiga OS 3.1.

We have been resting the fat32 partition on our Amigas since early PiStorm days. So it's not needed for that.
I feel this should be added as an optional thing, not for all.

This needs to be tested on real amiga with pistorm... noone reported the fat32 partion created with this tool not being able to be mounted on the amiga...

@TomCat666 I did not create the issue #2 , so at least we are 2 people with the same problem.
I understand you do not want to use this as a general solution, so I can try to add some changes to do it optional as @LemaruX  suggested.

MacOS does not recognize the type 0x0C, after changing the type to 0x0E I could access to the partition with no changes in the OS... It was some time ago when I did this change, but I think I had some similar problem with Amiga OS 3.1.

The 0x0E is overwritten later in the code when Fat32 partition is being formatted back to 0x0C ... so this change is basically ignored. Creating the IMG file with this change produces identical file.
If you want FAT16 image you need to change fmt_opt = {FM_FAT32 |    to fmt_opt = {FM_FAT |

@TomCat666 I did not create the issue #2 , so at least we are 2 people with the same problem.
I understand you do not want to use this as a general solution, so I can try to add some changes to do it optional as @LemaruX suggested.

The change of 0x0C to 0x0E will do nothing, the resulting image still has 0x0C in the MBR since it is re-written by the FatFS system when it formats and builds the filesystem (fat32).
Try making an IMG file with 0x0C and 0x0E and compare the resulting images.

You are right @TomCat666. Just cleaned a SD card and tried both options and the result is the same, and this time worked with no problem in Mac.
I created the PR back in March, and I don't remember if there is any other reason for this change... so I am going to close this PR.
Thank you.


Avoid string allocations when generating object name

Description
Hi, I am trying to optimize YARD a bit to speed up documentation runs on a large codebase.
The codebase is private, so I am using https://github.com/watir/watir/ as a smaller stand-in to report benchmarking numbers from https://github.com/SamSaffron/memory_profiler.
Test set-up

Ruby 2.7.6 (installed via rvm on Linux)
Watir version: 293bed2b57c
MemoryProfiler version: 1.0.0
YARD baseline: 0a550939f9b
Test command:

ruby-memory-profiler --no-color --retained-strings=500 --allocated-strings=500 --max=300 -o prof.log ./run-yard.rb stats

run-yard.rb is a smaller version of /bin/yard (MemoryProfiler was not able to run /bin/yard)
I am using the stats command to avoid memory exhaustion

Performance data

Before

Total allocated: 187901758 bytes (1987572 objects)
Total retained:  4052113 bytes (53463 objects)

After

Total allocated: 187459865 bytes (1976523 objects)
Total retained:  4052153 bytes (53464 objects)

Savings

Total allocated: 441893 bytes (11049 objects)
Total retained:  -40 bytes (-1 object)

Completed Tasks

[x] I have read the Contributing Guide.
[x] The pull request is complete (implemented / written).
[x] Git commits have been cleaned up (squash WIP / revert commits).
[x] I wrote tests and ran bundle exec rake locally (if code is attached to PR).

Thanks for this improvement!


Earn: add option for more privacy

Currently Jam Earn uses only the feature of "yield-generator-basic.py" from Join Market.
It would be a bonus to have the option that uses the feature available in "yg-privacyenhanced.py" of Join Market where the upper and lower amounts and the percentage are randomly created from the variance variables "cjfee_factor" and "size_factor".  This feature would make Jam Earn more private.

@0xbytes This would be a great feature.
Currently, it does not seem possible due to a hardcoded dependency on the "basic" approach. Would you be able to open a ticket on the backend repo regarding this feature request? That'd be awesome :pray:
Please note that this must also be configurable via an API call in order for Jam to make use of it.


🐛 StopRecording doesn't emit any event (finish or error), on specific android devices

What were you trying to do?
I try to record video. Some specific android devices do not fire any callback (neither onRecordingError nor onRecordingFinished) after calling camera.stopRecording(). I don;t even get any callback through the Camera component in the onError prop.
This is happening no matter the size of the video (even on 2-3 sec video)
We have disabled frame processors, mentioning just in case.
The only device I have reproduced at the moment (100% reproduce, happening all the time, using either front or back camera) is:
Brand: Samsung
Model: Galaxy Tab A8
Version: Android 13
RAM free: 778.43 MB
Disk free: 39.77 GB

Reproduceable Code
The way i choose camera is:

const devices = useCameraDevices();
let device;
if (props.useFrontCamera) {
    device = devices.front;
} else {
    device = devices.back;
}

### What happened instead?

I get no callback. The app continues to operate as normal, no crash or something, just no video is recorded and i don't get any event.

### Relevant log output

_No response_

### Device

Samsung Galaxy Tab A8 (android)

### VisionCamera Version

2.13.3

### Additional information

- [ ] I am using Expo
- [X] I have read the [Troubleshooting Guide](https://react-native-vision-camera.com/docs/guides/troubleshooting)
- [X] I agree to follow this project's [Code of Conduct](https://github.com/mrousavy/react-native-vision-camera/blob/main/CODE_OF_CONDUCT.md)
- [X] I searched for [similar issues in this repository](https://github.com/mrousavy/react-native-vision-camera/issues) and found none.

I have the same issue.

@angelos3lex @rosendosalazarbta
Can you check if you are taking microphone and camera permission before starting recording.
I faced this same issue
Just added permission for microphone and camera and it starting working.

@pranav-lio not the case for me, adding&asking microphone permisions don't fix it. also I don't use audio.

Happens for me as well on Android 9. Android 10 works fine, haven't checked others yet

@angelos3lex have you found a workaround?

Unfortunately no, still an issue

Hey! I've rewritten the entire Android codebase of VisionCamera from CameraX to Camera2 in the efforts of ✨ VisionCamera V3.
I just now completed the Camera2 rewrite and I believe the core structure is running, but there might be some edge cases to iron out. Can you try and test the PR #1674 for me to see if you can still reproduce this issue here?
Here's an instruction on how you can test that: https://github.com/mrousavy/react-native-vision-camera/pull/1674#issuecomment-1684104217
If the issue cannot be reproduced with that version/PR anymore, then hoorayy, I fixed it! 🎉
Otherwise please let me know and I'll keep this issue open to keep track of it.
Thank you!

I can verify this is fixed in V3.3.1 that we checked! Nice job @mrousavy!
We have some other crash on some cases but ill open a new issue once we find out more about how to reproduce this.
Till then, i just leave here the trace in case you have any idea.
Fatal Exception: java.lang.IllegalArgumentException: Surface was abandoned
       at android.hardware.camera2.utils.SurfaceUtils.getSurfaceSize(SurfaceUtils.java:135)
       at android.hardware.camera2.params.OutputConfiguration.<init>(OutputConfiguration.java:615)
       at android.hardware.camera2.params.OutputConfiguration.<init>(OutputConfiguration.java:325)
       at com.mrousavy.camera.core.outputs.SurfaceOutput.toOutputConfiguration(SurfaceOutput.kt:38)
       at com.mrousavy.camera.extensions.CameraDevice_createCaptureSessionKt.createCaptureSession(CameraDevice+createCaptureSession.kt:57)
       at com.mrousavy.camera.core.CameraSession.getCaptureSession(CameraSession.kt:438)
       at com.mrousavy.camera.core.CameraSession.startRunning(CameraSession.kt:532)
       at com.mrousavy.camera.core.CameraSession.access$startRunning(CameraSession.kt:49)
       at com.mrousavy.camera.core.CameraSession$startRunning$1.invokeSuspend(:14)

Awesome, thanks!
Yea this error might be the Preview that does something wrong, I'll need a bug report, logs & repro :)


[UPDATE] SWAP Flow

To make the Swap flow more clear I updated figma, while at it I implemented some feedback and improvements:

[ ] Change fee labels to: Network fee and Swap fee

[ ] Updated pairing screen to have more negative space. Also implement automatic start of keysign again

Please go over the whole flow and verify that all views match figma

This one is duplicated of #1359


Add Travis for CI

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New pod config option: 'signature'

See https://github.com/perl6/doc/issues/794#issuecomment-237981574
Firstly, cast @gfldex, since he is the main inventor of our doc-compiling project.
Since we need to compile signatures(errors occur there too) we need some method/routine/sub body to compile it without error. The thing is, we don't want reader to see this ugly {} empty method body.
Because of that, using @coke advice, I've implemented this option. If you write it like this:
=begin code :signature
    method some(Bar)
=end code

or even like that
=begin code :signature
    method some1(Bar)
    method some2(Bar)
=end code

empty brackets will be added automatically(in second case, for every string respectively).
My English is less than awesome(I guess, it really is), so I'll be really grateful if someone will fix up my bad-written description at the CONTRIBUTION.md page.
It can be merged if @gfldex approves.

It sure feels a bit wordy, but solution with "Let's check it with regexes and will hope it didn't break somewhere" or something similar is not cool too.

What about changing this:
$content.subst("\n", "\{\} \n", :g)

to something like this (untested):
$content.subst(/ [ ‘)’ | returns \s+ \S+ ] <( $$ /, " \{\}", :g)

This way it should support multi-line signatures as well. Like the one on Str#split

@AlexDaniel my code was tested on all(updated) pages that start with "A" character. I'll try your variant, but it seems a bit more obscure, than mine(or not). If it passes, I'll likely update my pr.
But I'm not completely sure about what to do with multi-line signatures. Maybe this behavior will be changed in the future.

On Friday, August 5, 2016, Altai-man<EMAIL_ADDRESS>wrote:
...

Because of that, using @coke https://github.com/coke advice, I've
implemented this option. If you write it like this:
=begin code :signature
I think you need to document the new config option in S26.  And how about
the Perl 6 grammar and actions in rakudo?

@tbrowder, I believe that Rakudo and Perl 6 grammar itself have nothing to do with how exactly we "test" our documentation, it's a separate thing. It is a custom option that is useful for the documentation testing, but it's definitely not mandatory. S26 only defines a presentation. It doesn't say anything about testing the examples. Please, correct me if I'm wrong here.
This test can be changed anytime if needed, hence I don't think we should make it more official then we already did.
For example, someone can implement an another way to test the documentation and just ignore this config option.
If you'll argue that we need to replace explicit signature marking with implicit signature guessing(like 'If there a word method|routine|sub|etc after four spaces and then symbols, but not the {} ones', you can be right. Then we need to revert this and rewrite our examples-extracting code a bit.
I'm not a S26 specialist, but by previously added :skip-test option I assumed, that custom config options are not something scary. Am I wrong?

Also, after your comment I thought a lot about implicit detection and now it looks for me as a really better solution than explicit marking.
If we revert this PR and write a method detection, it'll automatically(i.e. without any changes) make all our signatures correct, without adding of two lines for every signature(and we have a lot of them).

On Sunday, August 7, 2016, Altai-man<EMAIL_ADDRESS>wrote:

Also, after your comment I thought a lot about implicit detection and now
it looks for me as a really better solution than explicit marking.

I'm certainly not an S26 expert either, but I have been dabbling in the
guts of rakudo pod handling and wanted to warn about possible side effects
from defining a pod attribute outside rakudo.

If we revert this PR and write a method detection, it'll
automatically(i.e. without any changes) make all our signatures correct,
without adding of two lines for every signature(and we have a lot of them).

I don't pretend to understand completely, but that sounds better for the
moment. However, I think the more we customize the pod cleanup for the html
output we shouldn't forget that, eventually, we need to convert pod to
other output formats.

@tbrowder in fact, the pod testing doesn't affect the html or any other output format. But yes, you're totally correct.
I'm going to revert this and implement signature detection.


[Suggestion] A few suggestions for a few Categories

Hi,
Just went through this amazing list. Found a few missing but amazing node libraries / frameworks / thingys that have made my node life a lot more amazing over the past years.
Please find the suggestions along with their probable categories below. I’m open to discuss and open a PR for each of them.
Real-time

aedes (mcollina/aedes): From the creator of MQTT.js, it's a "Barebone MQTT broker that can run on any stream server, the node way".
Personally, I have used it on one of my production apps which is running for over 2 years.

Weird

xkcd (grant/xkcd): For people who love xkcd. This is an easy CLI / application tool to get your favourites.
I use this to get the latest one and send it over email to me. Start the day with a little xkcd and coffee. :)

Process Management, or maybe Miscellaneous

headrush (gavindmello/headrush): A minimal solution to defer application start after resolving your network & processing dependancies.
I personally use this in one of our production apps to wait for node to connect to all required DBs and queues before starting the HTTP server and start accepting incoming requests.

Audio (New section??), or maybe Miscellaneous

MIDI.js (mudcube/MIDI.js): Making life easy to create a MIDI-app on the web. Includes a library to program synesthesia into your app for memory recognition or for creating trippy effects.
Never really used it that much (want to soon enough). But there are some wonderful usage examples on their readme, my favourite is 3d-Piano

Real-time, or maybe Miscellaneous

monit-mqtt (debdutto/monit-mqtt): Made by yours truly. It's a small command line utility to monitor your mqtt broker's stats. A quick way to test if your broker is up, receiving messages and other stats.
I used it to monitor my raspberry pi broker's stats which I used to collect motion sensor data for one of my pet projects among others.

aedes (mcollina/aedes)

👍 PR welcome.

xkcd (grant/xkcd)

👎 Would be a better fit for https://github.com/aharris88/awesome-cli-apps

headrush (gavindmello/headrush)

👎 I don't really see the point of this module. You can easily do this with existing generic flow control libraries, or even just async/await:
(async() => {
	await Promise.all([
		initialProcessing(),
		connectToMongo(),
		connectToRedis()
	]);

	console.log('ready');
})();

MIDI.js (mudcube/MIDI.js):

👎 It's mainly for the web and not Node.js. Only the generator is Node.js and honestly too niche.

monit-mqtt (debdutto/monit-mqtt)

👎 Looks too unfinished and unmaintained. The readme could use a lot of work and the version is 0.0.6. Also bit too niche for this list.


NOT FULLY OFFLINE ~

Maybe I'm just the one but, I thought that this docs are 100% offline and I figured it out when I transfer the files to my other computer. Can you download the online files, create a folder copy then paste it there and put it offline.? Anyway the docs, are so helpful for me. Thanks anyway ^_^

If you download our whole repo just open the index.html file locally and that's our docs


[WelcomeUX] Consider going to Stack Overflow vscode sorted by votes

Testing #18091
It would probably be more useful to show the most popular questions over the newest (which may or may not have answers):
http://stackoverflow.com/questions/tagged/vscode?sort=votes&pageSize=50

done


Add traslation for lists-and-keys.md

Transation for List-and-keys.md

@dulmandakh Please review this