Question regarding caching

Hi,
We are using MvcSiteMapProvider in MVC5 project. We have static sitemap, mix with Dynamic Nodes that implements IDynamicNodeProvider.
What is the best practice to use caching in this scenario? We noticed that when we turn the cache on the DynamicNodeProviders are not getting called at all. when we turn the cache off by setting MvcSiteMapProvider_CacheDuration=0 we can see performance impact asthe MVC provider keep scanning the dlls for assemblies when creating site maps.
We would like to have the static site map to be cached but the dynamic nodes to be always resolved.
Thanks,
Noam

Hi
I suggest using a separate sitemap with the dynamic nodes. Then you can set different cache parameters as described here: https://github.com/maartenba/MvcSiteMapProvider/wiki/Multiple-Sitemaps-in-One-Application


Task output file links not working

VSCode Version: 1.1.1
OS Version: OSX 10.11.5

Steps to Reproduce:

Run a test task with failures
Ctrl + click the filename link in the output, link never does anything

Update on this, it looks like the link did finally work if I right-clicked while holding ctrl instead of left-click

@benmillhouse how does the link look like?

Closing - please disregard.  This was an issue with how my VM was setup.


No license

Thanks for putting this out there :).
I'm putting this into my personal KiCAD library, which I plan on throwing up on github, but I would feel weird putting it up without some sort of attribution, or confirmation that that is okay with you in some way.
It's really not necessary but I do suggest putting a license file somewhere, may I suggest BSD, or the WTFPL.

@davidlenfesty you're right and I just couldn't be bothered to read the differences and choose one when I set up the repo.
It's certainly OK for you to include in your publicly hosted kicad library. Attribution would be appreciated but I don't consider it necessary as I haven't given anyone else who has built converter boards etc the impression that they ought to attribute.
The only licence clause I could think is relevant is the usual disclaimer about being fit for purpose and provided as-is with no guarantee etc. If you know one that covers that (and perhaps also attribution if you can't find one standalone) please let me know which one it is and I'll add it. But otherwise crack on!

The 2-clause BSD license is the closest and simplest thing I could find.
It does require attribution but that just means copying the license file essentially.

@davidlenfesty thanks for the research. License added!


Publish to WinGet

With every new release of ultimate-cube, use a GitHub workflow to update manifests at https://github.com/microsoft/winget-pkgs repository.
I can create a pull request if you would like.

Can you try re-running the job again, as you did earlier? I guess it should succeed now.


[libmariadb] build failure

Host Environment

OS:Windows/OSX
Compiler: MSVC/clang

To Reproduce
Steps to reproduce the behavior:
.\vcpkg.exe install libmariadb:x64-windows
Failure logs
Computing installation plan...
The following packages will be built and installed:
    libmariadb[core,openssl,zlib]:x64-windows
Detecting compiler hash for triplet x64-windows...
Starting package 1/1: libmariadb:x64-windows
Building package libmariadb[core,openssl,zlib]:x64-windows...
Could not locate cached archive: C:\Users\antao\AppData\Local\vcpkg\archives\57\57778c581dfb33181c9d98b373610b54cae3d448.zip
-- Using cached C:/Users/antao/vcpkg/downloads/MariaDB-mariadb-connector-c-a746c3af449a8754e78ad7971e59e79af7957cdb.tar.gz
-- Extracting source C:/Users/antao/vcpkg/downloads/MariaDB-mariadb-connector-c-a746c3af449a8754e78ad7971e59e79af7957cdb.tar.gz
-- Applying patch md.patch
-- Applying patch disable-test-build.patch
-- Applying patch fix-InstallPath.patch
CMake Error at scripts/cmake/vcpkg_apply_patches.cmake:54 (message):
  Applying patch failed.  Checking patch cmake/install_plugins.cmake...

  error: while searching for:

  # plugin installation

  MACRO(INSTALL_PLUGIN name binary_dir)

    INSTALL(TARGETS ${name} COMPONENT ClientPlugins DESTINATION ${INSTALL_PLUGINDIR})
    IF(WIN32)
      FILE(APPEND ${CC_BINARY_DIR}/win/packaging/plugin.conf "<File Id=\"${name}.dll\" Name=\"${name}.dll\" DiskId=\"1\" Source=\"${binary_dir}/${CMAKE_BUILD_TYPE}/${name}.dll\"/>\n")
      FILE(APPEND ${CC_BINARY_DIR}/win/packaging/plugin.conf "<File Id=\"${name}.pdb\" Name=\"${name}.pdb\" DiskId=\"1\" Source=\"${binary_dir}/${CMAKE_BUILD_TYPE}/${name}.pdb\"/>\n")

  error: patch failed: cmake/install_plugins.cmake:8

  error: cmake/install_plugins.cmake: patch does not apply

  Checking patch libmariadb/CMakeLists.txt...

  error: while searching for:

  TARGET_LINK_LIBRARIES(mariadbclient ${SYSTEM_LIBS})

  IF(UNIX)

    ADD_LIBRARY(libmariadb SHARED ${libmariadb_RC} ${MARIADB_OBJECTS} ${EMPTY_FILE})
    SET_TARGET_PROPERTIES(libmariadb PROPERTIES COMPILE_FLAGS "${CMAKE_SHARED_LIBRARY_C_FLAGS}")

  ELSE()

    ADD_LIBRARY(libmariadb SHARED ${libmariadb_RC} mariadbclient.def)
    TARGET_LINK_LIBRARIES(libmariadb  mariadbclient)
    SET_TARGET_PROPERTIES(libmariadb PROPERTIES LINKER_LANGUAGE C)

  ENDIF()

  error: patch failed: libmariadb/CMakeLists.txt:386

  error: libmariadb/CMakeLists.txt: patch does not apply

Call Stack (most recent call first):
  scripts/cmake/vcpkg_extract_source_archive_ex.cmake:141 (vcpkg_apply_patches)
  scripts/cmake/vcpkg_from_github.cmake:139 (vcpkg_extract_source_archive_ex)
  ports/libmariadb/portfile.cmake:8 (vcpkg_from_github)
  scripts/ports.cmake:79 (include)

Error: Building package libmariadb:x64-windows failed with: BUILD_FAILED

Same problem here

@an-tao, thank for reporting this issue! I have summited PR  #14059 to fix it.

@PhoebeHui Thanks for the quick fix. would you like to take a look at the issue #13848? thanks.

Thanks!
Thought I'm wondering again, how did this get merged?


Consolidating util and tool folders

Stack from ghstack (oldest at bottom):

-> #643
#592
#590

@VitalyFedyunin has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


Feature: Allow to provide a custom method to format the file path

It allows to format the showed filepaths, 2 examples:

@ThePrimeagen can I please have your review on this?


SPT - Item Commando Spawning issue

General information
Project issue is for:
Server
Expected result:
Item Spawn
Actual result:
The below error message of
"That weapon template ID could not be found. Please refine your request and try again. "
The Description of which after the command is posted, grants this error when using "handheld flare" and the flare gun.
How to reproduce

Find SPT Item ID from item finder, Specifically for this the flare.
Type following with out quotes, "spt give 62178be9d0050232da3485d9 1"
No spawn but a claim of "that weapon template id could not be found."

Log files
Server:  https://mega.nz/file/wzYwHDZb#5MWgqkiDX2PrEZ5WLJO5-p-0Xajdl01X9b8N7hVkrCw
BepinEx: https://mega.nz/file/FqpSACpa#TVKYNQ51jT-TWkUyOyYlsf3sqGnLsxTG0WG1bj-9Yvw
Client: https://mega.nz/file/xuRkQI7S#I54h9pISudi_c3-0YG6nA4R9TX0GO27BFTHfpd0gSxs

Originally written by thester

Fixed with: https://dev.sp-tarkov.com/SPT-AKI/Server/pulls/323

Originally written by chomp

closing as fixed

Originally written by CWX


Migrate from Laravel Mix to Vite

This pull request includes changes for migrating from Laravel Mix to Vite outlined in Migration Guide and automated by the Vite Converter.
Before merging, you need to:

Checkout the shift-83106 branch
Run composer update
Run npm install
Review all comments for additional changes
Thoroughly test your application (no tests?, no CI?)

Please send your feedback to<EMAIL_ADDRESS>or share the good vibes on Twitter.

:information_source: Previously, Laravel configured several npm scripts for building assets. For Vite, Laravel configures a dev script for generating and watching your assets (for development) and build script for generating your assets (for production).
Shift automated this change by replacing commands using these scripts with their new Vite script. However, you may still be referencing these scripts in your code or in deployment scripts.

:information_source: Vite requires JavaScript files containing JSX to have a .jsx file extension. Shift did not detect any JSX in files with a .js extension under the resources folder. However, if you are using JSX, be sure the files have a .jsx extension.

:information_source: Laravel renamed the environment variables used by Mix to use a VITE_ prefix, instead of MIX_. While Shift automated this change throughout your code, you should check for any additional references which may not be included in your Git repository.

:warning: Vite generates frontend assets to the public/build folder. This folder is not tracked by Git within a default Laravel project. As such, Shift added this folder to your .gitignore file.
If you wish to track the generated frontend assets within your project, such as for ease of deployment, you may undo this commit by running git revert a6270c5b.

:warning: Shift detected you are using Inertia. You may need to update your require statement for Vue components as well as configure an entry point if you are using SSR.

:information_source: The new @vite Blade directive outputs <script> tags with the type="module" attribute. This attribute implicitly defers the script. This is a difference from Laravel Mix which may change the execution order of the JavaScript on your page.
If you experience JavaScript errors relating to undefined references, you may need to reorder your scripts or add the defer attribute to any inline <script> blocks on your page.

:alembic: This Shift is still being refined. Please report any issues or suggestions to<EMAIL_ADDRESS>Your feedback is what helps improve the experience for everyone.


The website:trace.moe can't upload image

I got this, hang on...

Should be resolved, clear cache and try again?

Resolved, thx.


feat: ability to sort and filter orderbook

Adds sorting and filtering to the Orderbook table.
This PR is based on #422 and should be reviewed only if the underlying changes have been approved and found to be acceptable.
:camera_flash:

Let me know once this is ready to review. :)

Let me know once this is ready to review. :)

Ready for review! :pray:


Stop chavphobia in the server

Description of the issue
There is lots of discrimination against people who identify a "chavs". They are often spoken about like they are less in our society when they are not. Often people say on the server how most of them are crimals, but this stereotype is not true. (You wouldn't say people in a certain race or sexuality are crimals just because a few people who identify in those categories are) This stereotyping is awful. Since the server mods quickly remove rasist and homophobic content it is only fair they do the same for chavphobic content. You cannot discriminate against types of discrimination.
Proposed solution
Make the mods aware of the issue and make them actively recognise when this discrimination takes place. Make it clear that ANY type of discrimination is unacceptable in the server, especially discrimination against people from different backgrounds/cultures

chavs have no rights


Add Identity Access Management (IAM) to the Storage API (#1812)

Adds support for bucket-level IAM (currently in limited alpha). More information
about IAM in Google Cloud Storage can be found at
https://cloud.google.com/storage/docs/access-control/iam

cc @rybosome

LGTM

Coverage decreased (-0.01%) to 80.939% when pulling 04a7df39913b898312f8343b9aa54c4ccd856b23 on garrettjonesgoogle:storage-iam into 56d0a30a6847cae19c8dc74324b9b4935159ba2c on GoogleCloudPlatform:master.

Coverage decreased (-0.01%) to 80.939% when pulling 04a7df39913b898312f8343b9aa54c4ccd856b23 on garrettjonesgoogle:storage-iam into 56d0a30a6847cae19c8dc74324b9b4935159ba2c on GoogleCloudPlatform:master.


feat/better-null-management

Please review before merging.

:tada: This PR is included in version 1.57.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Safer defaults for usage by making usage of $replicas() opt-in by default

Currently, the default behavior of this extension is sending every read query to the replica(s) and every write to the primary. While you can opt-in to using the primary with $primary() , it's default is to always use the replicas for any read. In my opinion, this is a dangerous and unintuitive default, as replication lag could cause unexpected behavior for users if they immediately try to access a newly inserted row after a write.
To mitigate this, a safer default would be to send all reads and writes to primaries by default. This way, there is a guarantee that the reads after writes will always be available. When the user wants to explicitly use replicas, they should call the $replicas() method and chain their queries to it, so this way they are knowingly opting into relaxed consistency and accepting the replication lag that comes with it.
For reference, this is how a lot of other frameworks/ORMs do it. See Django and Rails for example.

+1, came to note the same thing. This is about six months old now. Do we any updates as to whether this will be addressed? We want to issue queries to the read replica but the only other solution is to have a second Prisma client.


Error message set position slice number ROI manager

I am trying to run SPIRO Assays for Root Growth, and I get "germination data" all right from it - but when it comes to actually tracing the root growth - I run into the following error:

Here is the Debug window screenshot

Hi! Thanks for submitting an issue. May I know what version of ImageJ you are running? You can find this at Help > About ImageJ.
The unrecognized command could be due to an old ImageJ version since RoiManager.setPosition was only introduced in 1.53c. Could you please try running the macro using a clean install of FIJI ImageJ v1.53f with only the required plugins TurboReg and MultiStackReg). At the time of writing ImageJ v1.53f has been confirmed to run smoothly i.e. recognizing all commands used in the SPIRO macros. You can get this version from the FIJI archive

Thanks. I have downloaded the correct version - now the software is stuck here:

Hi! Was everything fine during the step where a dialog box pops up with "Please delete any ROIs that should not be included into analysis, e.g. objects wrongly recognized as seeds. Unrecognized seeds can also be added as ROIs."? Did the ROI Manager display properly and you could add/change ROIs?
If that went fine, then ROI Manager did not save the file. Can you verify this for me by going to your folder containing plate 1 Col-0 and checking the .zip file does not exist?
I noticed also in your screenshot that you have seedlings already in your first time point. The automatic detection of seeds does not work then and you would have to manually select the "root starts" i,e the top of the root and add them as ROIs. I have attached an image to show what I mean.

Thanks again - the zip file indeed does not exist - see the screenshot below:

Moreover - this message is popping up before I had a chance to select any ROI. Once I did select my ROI - I dont have any message box to click on to proceed to the next step:

Regarding roots on the 1st image - we are interested in root growth - not germination. Since we are transplanting/treating the plants at 4 days old stage, we do not have the images containing the seed at the beginning of the experiment.

@mmjulkowska Just a heads up that we're currently using the germination data for root growth normalization downstream, so the provided R scripts will not work using your current setup. The data from the macro can still be used for analyses though, but you may need to tweak the scripts or do your own custom analysis.
In our experience, we needed to normalize root growth to germination occurrence, since the variation within the group would otherwise be too large when some seeds would get a "head start". Therefore we set the zero timepoint per-seedling as the timepoint where germination is detected.
This may not be needed if you're only interested in root growth rates though, but beware that the scripts will not work for your use case out of the box.

Thanks @jonasoh for the heads-up - we can do/adjust the R analysis downstream no problem. Once we have the R script that is working for us - we would be happy to share it with y'all :)
RE variation: Since we transfer the seedling to our treatment plates - we are already reducing the germination variation by transferring only the seedlings of the approximately same size.

Sounds good! We'd very much like to see others building their own pipelines based on the SPIRO hardware :)


[WIP] ZeRO-3 refactoring (sharding)

A huge refactoring of Sharded:

Sharded is now a sub-module of TensorParallel, not a wrapper. Sharded is not supposed to be interfaced with by end user.
TensorParallel now keeps track of which parameters were modified to pass this list to Sharded. This removes the need to find replicated parameters by hand since all those that were not modified are replicated.
Sharded is marked for deprecation, but a backwards compatibility placeholder is implemented.

Tests modifications:

All the basic tests now also test with Zero3.
All the transformers tests now also test with Zero3.
Better dispatch tests to test for specific use-cases.

Questionable choices:

Zero3 now depends on get_parameter_name_mapping which utilizes config regex.  Regex is always unreliable. Might need to find a better solution.

The idea was that users confused Sharding with tensor parallelism itself since we call tensor parallel parts "shards". I wanted to pick a different name. But since Sharded is no longer public I guess it's okay.
I've reverted all the names back to what they were.


Automatically generate unit tests

What do you want to change?
It would be great to automate creation of unit tests for the generated functions. So for each xxx.go generated file there would be accompanying xxx_test.go.
What database engines need to be changed?
None
What programming language backends need to be changed?
Go

This is a great example of a feature request that could work as a codegen plugin. Different folks might have very different ideas on the types of tests to write.

Hey! It's a good suggestion! I'm a fresh user of the project so I haven't thought about this approach.


Deprecation warnings in Atom 1.13.0

Starting from Atom v1.13.0, the contents of atom-text-editor elements are no longer encapsulated within a shadow DOM boundary. This means you should stop using :host and ::shadow pseudo-selectors, and prepend all your syntax selectors with syntax--. To prevent breakage with existing style sheets, Atom will automatically upgrade the following selectors:
:host atom-text-editor .highlight.highlight-selected .region, atom-text-editor atom-text-editor .highlight.highlight-selected .region, atom-text-editor::shadow atom-text-editor .highlight.highlight-selected .region => :host atom-text-editor .highlight.highlight-selected .region, atom-text-editor atom-text-editor .highlight.highlight-selected .region, atom-text-editor.editor atom-text-editor .highlight.highlight-selected .region
:host .highlights .highlight-selected .region, atom-text-editor .highlights .highlight-selected .region, atom-text-editor::shadow .highlights .highlight-selected .region => :host .highlights .highlight-selected .region, atom-text-editor .highlights .highlight-selected .region, atom-text-editor.editor .highlights .highlight-selected .region
:host .highlights .highlight-selected .region, atom-text-editor .highlights .highlight-selected .region, atom-text-editor::shadow .highlights .highlight-selected .region => :host .highlights .highlight-selected .region, atom-text-editor .highlights .highlight-selected .region, atom-text-editor.editor .highlights .highlight-selected .region
:host .highlights .highlight-selected.background .region, atom-text-editor .highlights .highlight-selected.background .region, atom-text-editor::shadow .highlights .highlight-selected.background .region => :host .highlights .highlight-selected.background .region, atom-text-editor .highlights .highlight-selected.background .region, atom-text-editor.editor .highlights .highlight-selected.syntax--background .region
:host .highlights .highlight-selected.light-theme .region, atom-text-editor .highlights .highlight-selected.light-theme .region, atom-text-editor::shadow .highlights .highlight-selected.light-theme .region => :host .highlights .highlight-selected.light-theme .region, atom-text-editor .highlights .highlight-selected.light-theme .region, atom-text-editor.editor .highlights .highlight-selected.light-theme .region
:host .highlights .highlight-selected.light-theme.background .region, atom-text-editor .highlights .highlight-selected.light-theme.background .region, atom-text-editor::shadow .highlights .highlight-selected.light-theme.background .region => :host .highlights .highlight-selected.light-theme.background .region, atom-text-editor .highlights .highlight-selected.light-theme.background .region, atom-text-editor.editor .highlights .highlight-selected.light-theme.syntax--background .region

Automatic translation of selectors will be removed in a few release cycles to minimize startup time. Please, make sure to upgrade the above selectors as soon as possible.

Dupe of https://github.com/richrace/highlight-selected/issues/131

It seems that update for Atom 1.13.0 makes old version like 1.12.* cannot work normally.

Well following advice from (here)[http://flight-manual.atom.io/shadow-dom/sections/removing-shadow-dom-styles/#when-should-i-migrate-my-themepackage]:

"If you already want to test the migration on master or Beta channel, make sure to change your package.json file to "engines": { "atom": ">=1.13.0 <2.0.0" }. This will prevent Atom from updating your theme or package before the user also updates Atom to version 1.13."

Difference between the latest version and the old version:
https://github.com/richrace/highlight-selected/blob/v0.11.2/package.json#L12
https://github.com/richrace/highlight-selected/blob/v0.12.0/package.json#L12
I would say that it's Atom.

I know, that is strange. Atom alerts me that packages are deprecated. And after that I know Atom has a newer version, it doesn't tell me.


Works on ionide?

I trying to run https://github.com/panesofglass/grpc-fsharp-sample/ and create a simple using the tutorial, but I have
The value, namespace, type or module 'Greeter' is not defined. F# Compiler(39) on ionide

Upon further investigation, I have found no way to instruct Ionide to scan the generated sources. I've opened an issue.

I've pinged the Ionide people and ionide/ionide-vscode-fsharp#1535 is open again. Closing since this isn't an FSharp.GrpcCodeGenerator issue.


When condition for Source Control View for better keybindings

Currently there are no working 'when' conditions to bind specific keyboard shortcuts to the Git Source Control view window. There are however for other views such as the file explorer.
Having one would open to door to vim like key bindings. 's' for staging for example, or 'u' for upstaging and 'd' for discard changes, 'p' for push etc.
This would enhance the tool into something more powerful like vim-fugitive.
As developers can often put together multiple commits a day, having the ability to define keyboard shortcuts that only use one letter would save developers (who hate the mouse) a lot of reduced keystrokes!

Does this work for you:
{
   "key": "alt+i",
   "command": "git.stageAll",
   "when": "sideBarFocus && activeViewlet == 'workbench.view.scm'"
 },


Increasing resolution parameter leads to weird kmap binning

Increasing the resolution parameter in the KSMap configuration file leads to images that seem to have the wrong bin size. The attached image was made using the parameters below.
# Parameters
resolution: 1                    # Pixel resolution of the map in arcminutes
width: 110                       # Width of the map in pixels

For a 23 x 17 arcminute image, a resolution parameter of 1 should yield a 23 x 17 pixel image, but that's clearly not what gets produced!

A mistake was made with the 'width' parameter, where it wasn't adjusted accordingly when 'resolution' was changed. The width of the image in pixels is equal to the field size (arcmin) / resolution (arcmin), so we update the code to calculate automatically.


Override dependencies that dont return nothing

Hi.. I started this discussion on discord but seems like a more tricky thing.
I'd like to know if dependency_overrides works when the dependency function dont return anything.. in my case my dependency function only pass if not wrong happens or throw an httpexception if something wrong happens..
The dependency override is not working because in fastapi.dependencies.utils, when the code will try override the dependencies, the reference of functions are different and then the override dont occurs..
My dependency:
async def database_check():
    try:
        _database_is_alive()
        if not database.is_connected:
            await database.connect()

    except ConnectionDatabaseError:
        if database.is_connected:
            await database.disconnect()

        raise HTTPException(status_code=503)

My 3 attempts functions to override dependency:
from ..api.dependencies import database_check

async def database_check_pass():
    return True

async def database_check_pass():
    pass

async def database_check_pass():
    def handle():
      return True
    return handle

My override dependency code:
app.dependency_overrides[database_check] = database_check_pass
Something I notice, inside fastapi code, the function reference inside dependency_overrides_provider is different than the original function in app/api/dependencies.py
fastapi.dependencies.utils
call = getattr(
    dependency_overrides_provider, "dependency_overrides", {}
).get(original_call, original_call)

My app structure:
app/
  api/
    dependencies.py
    views.py
  tests/
    my_test.py
  main.py

What am I missing here?

I can't see any issues with this code - here's a full example that works fine:
from fastapi import FastAPI, Depends
from fastapi.testclient import TestClient

app = FastAPI()

async def database_check():
    try:
        _database_is_alive()
        if not database.is_connected:
            await database.connect()

    except ConnectionDatabaseError:
        if database.is_connected:
            await database.disconnect()

        raise HTTPException(status_code=503)

async def database_check_pass():
    return "the value were expecting"

app.dependency_overrides[database_check] = database_check_pass

@app.get("/")
def route(database_check=Depends(database_check)):
    return database_check

tc = TestClient(app)
assert tc.get("/").json() == "the value were expecting"

@Mause thank you a lot for trying to help me.
But you didnt replicate my case. You made a fresh new one. Without packages and imports.
Try to reply the same you did but within this structure:
app/
  api/
    dependencies.py
    views.py
  tests/
    my_test.py
  main.py

:)

Why don't you provide that structure within a repository, so we can help you easily?

@Kludex
Right. I'll do that.
When ready I'll leave the link here.
Thank you.

Copying only dependencies files and routes to a fresh project works properly.  :|
I'll move file by file to this new fresh project until get the error.
Is there some cache to clean or anything related?

I closed because probably is something related to my project.

Found! In tests file I was importing ..main and ..api.dependencies
Changed to app.main and app.api.dependencies.
Thank you guys!


Pacbio analysis gets stuck "forever" at close to 100%

I've had an issue with an event when running in batch mode, where it gets stuck at close to 100% for >1 hour and still hasn't finished.  Any ideas?
Here's the command line:
svviz -t batch --max-deletion-size 30000 --max-reads 30000 --skip-cigar --summary /Volumes/UHTS1/AJPacBio/PracticeSVs_vcfForsvviz_svvizsummary_pacbio_b.tsv --export /Volumes/UHTS1/AJPacBio/PracticeSVs_vcfForsvviz_svvizfigs -b /Volumes/UHTS1/AJPacBio/HG003/CSHL_bwamem_bam_GRCh37/BWA-MEM_HG003_merged_11_12.sort.bam -b /Volumes/UHTS1/AJPacBio/HG004/CSHL_bwamem_bam_GRCh37/BWA-MEM_HG004_merged_11_12.sort.bam -b /Volumes/UHTS1/AJPacBio/HG002/CSHL_bwamem_bam_GRCh37/BWA-MEM_HG002_merged_11_12.sort.bam --aln-quality 0.2 /Volumes/SSD960/references/human_g1k_v37.fasta /Volumes/UHTS1/AJPacBio/PracticeSVs_vcfForsvviz_test.vcf
Here's the event:
16	7733013	7733013	.	TGTATATATACACACACACACACATATATACGTATATATATGTGTGTGTGTATATATACACACACACATATATACGTATATATGTGTGTGTGTGTATATATATATATATATATATA	.	.	 I've looked into workarounds but unfortunately it's not simple. Forgot about this, as very few people are using svviz on pacbio (that I know of). You can try emailing mengyao as she had suggested she was hoping to work on this at some point.

Is there any possibility of just skipping an SV if it runs more than a
certain amount of time?  It looks like this is happening again - in 2 out
of the first 6 SVs I ran in batch mode.  If it's this common, it will make
running a lot of SVs really tedious.
On Mon, Dec 14, 2015 at 5:21 PM Noah Spies<EMAIL_ADDRESS>wrote:

This is likely caused by this issue:
mengyao/Complete-Striped-Smith-Waterman-Library#22
https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library/issues/22.
I've looked into workarounds but unfortunately it's not simple. Forgot
about this, as very few people are using svviz on pacbio (that I know of).
You can try emailing mengyao as she had suggested she was hoping to work on
this at some point.
—
Reply to this email directly or view it on GitHub
https://github.com/svviz/svviz/issues/35#issuecomment-164578595.

Gah, sorry it’s having so much trouble
Unfortunately, this causes python to crash — it’s running in multiprocessing mode, so one of the sub-processes actually died (at least, assuming you’re hitting this bug and not some other issue). I wasn’t able to figure out how to detect a subprocess that’s died from python. If we were able to do that, it’s possible that switching which sequence is getting aligned against which would be a workaround for the problem (ie which one is reference and which one is query). No way to know beforehand which should be which, though.
And I haven’t found another python-compatible smith-waterman alignment implementation that’s within 2 orders of magnitude of the speed with the ssw library. Otherwise I’d be happy to switch to something better maintained.
The bwa smith-waterman alignment implementation should be fast, but I don’t have the know-how to wrap it in python (this was I believe why ssw was published).
On Dec 14, 2015, at 2:39 PM, jzook <notifications<EMAIL_ADDRESS>wrote:
Is there any possibility of just skipping an SV if it runs more than a
certain amount of time? It looks like this is happening again - in 2 out
of the first 6 SVs I ran in batch mode. If it's this common, it will make
running a lot of SVs really tedious.
On Mon, Dec 14, 2015 at 5:21 PM Noah Spies <notifications<EMAIL_ADDRESS>wrote:

This is likely caused by this issue:
mengyao/Complete-Striped-Smith-Waterman-Library#22
https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library/issues/22.
I've looked into workarounds but unfortunately it's not simple. Forgot
about this, as very few people are using svviz on pacbio (that I know of).
You can try emailing mengyao as she had suggested she was hoping to work on
this at some point.
—
Reply to this email directly or view it on GitHub
https://github.com/svviz/svviz/issues/35#issuecomment-164578595.

—
Reply to this email directly or view it on GitHubhttps://github.com/svviz/svviz/issues/35#issuecomment-164582711.

That is unfortunate - I'll email the author also.  Would the problem be
alleviated if I ran in single-threaded mode?  Since I have a lot of events
to run, I could just run a bunch of batch processes at the same time in
single-threaded mode, and it probably would be just as efficient.
On Mon, Dec 14, 2015 at 5:49 PM Noah Spies<EMAIL_ADDRESS>wrote:

Gah, sorry it’s having so much trouble
Unfortunately, this causes python to crash — it’s running in
multiprocessing mode, so one of the sub-processes actually died (at least,
assuming you’re hitting this bug and not some other issue). I wasn’t able
to figure out how to detect a subprocess that’s died from python. If we
were able to do that, it’s possible that switching which sequence is
getting aligned against which would be a workaround for the problem (ie
which one is reference and which one is query). No way to know beforehand
which should be which, though.
And I haven’t found another python-compatible smith-waterman alignment
implementation that’s within 2 orders of magnitude of the speed with the
ssw library. Otherwise I’d be happy to switch to something better
maintained.
The bwa smith-waterman alignment implementation should be fast, but I
don’t have the know-how to wrap it in python (this was I believe why ssw
was published).
On Dec 14, 2015, at 2:39 PM, jzook<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Is there any possibility of just skipping an SV if it runs more than a
certain amount of time? It looks like this is happening again - in 2 out
of the first 6 SVs I ran in batch mode. If it's this common, it will make
running a lot of SVs really tedious.
On Mon, Dec 14, 2015 at 5:21 PM Noah Spies<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

This is likely caused by this issue:
mengyao/Complete-Striped-Smith-Waterman-Library#22
<
https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library/issues/22>.

I've looked into workarounds but unfortunately it's not simple. Forgot
about this, as very few people are using svviz on pacbio (that I know
of).
You can try emailing mengyao as she had suggested she was hoping to work
on
this at some point.
—
Reply to this email directly or view it on GitHub
https://github.com/svviz/svviz/issues/35#issuecomment-164578595.

—
Reply to this email directly or view it on GitHub<
https://github.com/svviz/svviz/issues/35#issuecomment-164582711>.
—
Reply to this email directly or view it on GitHub
https://github.com/svviz/svviz/issues/35#issuecomment-164584838.

I think this would at least make it crash, rather than running forever
On Dec 14, 2015, at 2:55 PM, jzook <notifications<EMAIL_ADDRESS>wrote:
That is unfortunate - I'll email the author also. Would the problem be
alleviated if I ran in single-threaded mode? Since I have a lot of events
to run, I could just run a bunch of batch processes at the same time in
single-threaded mode, and it probably would be just as efficient.
On Mon, Dec 14, 2015 at 5:49 PM Noah Spies <notifications<EMAIL_ADDRESS>wrote:

Gah, sorry it’s having so much trouble
Unfortunately, this causes python to crash — it’s running in
multiprocessing mode, so one of the sub-processes actually died (at least,
assuming you’re hitting this bug and not some other issue). I wasn’t able
to figure out how to detect a subprocess that’s died from python. If we
were able to do that, it’s possible that switching which sequence is
getting aligned against which would be a workaround for the problem (ie
which one is reference and which one is query). No way to know beforehand
which should be which, though.
And I haven’t found another python-compatible smith-waterman alignment
implementation that’s within 2 orders of magnitude of the speed with the
ssw library. Otherwise I’d be happy to switch to something better
maintained.
The bwa smith-waterman alignment implementation should be fast, but I
don’t have the know-how to wrap it in python (this was I believe why ssw
was published).
On Dec 14, 2015, at 2:39 PM, jzook <notifications<EMAIL_ADDRESS>notifications<EMAIL_ADDRESS>wrote:
Is there any possibility of just skipping an SV if it runs more than a
certain amount of time? It looks like this is happening again - in 2 out
of the first 6 SVs I ran in batch mode. If it's this common, it will make
running a lot of SVs really tedious.
On Mon, Dec 14, 2015 at 5:21 PM Noah Spies <notifications<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

This is likely caused by this issue:
mengyao/Complete-Striped-Smith-Waterman-Library#22
<
https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library/issues/22>.

I've looked into workarounds but unfortunately it's not simple. Forgot
about this, as very few people are using svviz on pacbio (that I know
of).
You can try emailing mengyao as she had suggested she was hoping to work
on
this at some point.
—
Reply to this email directly or view it on GitHub
https://github.com/svviz/svviz/issues/35#issuecomment-164578595.

—
Reply to this email directly or view it on GitHub<
https://github.com/svviz/svviz/issues/35#issuecomment-164582711>.
—
Reply to this email directly or view it on GitHub
https://github.com/svviz/svviz/issues/35#issuecomment-164584838.

—
Reply to this email directly or view it on GitHubhttps://github.com/svviz/svviz/issues/35#issuecomment-164586415.

I've implemented a workaround which uses subprocesses to wrap the ssw Smith-Waterman alignment library, allowing realignment to skip reads which crash ssw. To use this mode, specify --processes -1. See a1732c5beafd84e47a523fcb4eb42cb5f9509a34

Note that this mode is several fold slower (depending on the length of the reads)

closed by 756655b3af4d346235663a8f52a72ca2740e51dc


refactor model progression api in model and preprocessor

With new additions to the model to run for time rather than time_steps (#78) and have checkpointing ability (#74), we need routines to manipulate the model in new ways. Currently, the model still operates via update() only.
My proposed solution is BMI-ish, we should have update_until(time) and update_for(elapsed_time) and update_iterations(iterations), each of which wrap the update() routine in various while or for loop configurations. I think that these function should be in the model.
Then, we need to update the preprocessor to take arguments relating to these new configurations. This means that the preprocessor will probably have identically named methods that simply wrap the model methods. The trick will be implementing the command line interface options for resuming jobs and optionally taking either time_steps or until_time or for_time etc as inputs to the CLI and python high-level api.
I believe that some of this can be streamlined by using @property within the preprocessor to convert both CLI args and py-signature args into the properties. setter logic will allow for handling mutual exclusivity of these options as well. This will no longer use args as an object which holds useful information then, because it will be processed into properties of the preprocessor.
In the process of this refactoring, I will keep in mind another issue that requires refactoring of the preprocessor, to see if I can get both done at once. This issue relates to parallel jobs (#56), whereby these jobs need to be initialized on the thread that they will be executed on. In the current implementation the jobs are all constructed, then the pool is created and the jobs are assigned out, but this fails because the open netCDF file cannot be passed across the parallel threads. So one option would be to only open the netCDF file after passing jobs to the threads, but this would require (extensive?) refactoring inside of model. The other way is to just create the pool up front, and create jobs on the different threads. This may cause problems if the number of jobs exceeds the number of threads, because jobs may finish asynchronously, and threads will not be available for the next job until the first is finished. For this reason, I think it may be cleaner to connect and disconnect to the netCDF file as part of the update routine. Something at the top of update() like if not self._netcdf_is_connected: self.netcdf_connect() with a similar self.netcdf_disconnect called during finalize() and wherever else needed. This will add some overhead, but I think it will be small. I will verify this.
relates to #56, #78, #74

I no longer think this is a high priority or even necessary, so I'm taking it off 2.0

Revisiting this, I now think this isn't even a good idea any more.

we have BMI support for those who want it via BMI_pyDeltaRCM
we have the ability to specify run time or timesteps via the Preprocessor already.
if you are already scripting at the low level, it's trivial to just write a while or for loop to move to the new state.

If a use case arises where someone really needs update_until at the low-level model api, we can consider it again. Closing this for now.


Make sure our touch/point conversion works in middleware (C#-hosted, …

…etc.) scenarios.  Controls that are placed within arbitrary Xaml scenes don't have a superview chain up to a valid UIWindow, so touch point conversion is failing.  The fix is to - in middleware scenarios - use the app's main UIWindow for any UIView that doesn't have a superview.
Fixes #1818.

This change is

:shipit:


Unable to create geo_shape index

Elassandra version: <IP_ADDRESS>-14
jvm: openjdk version "1.8.0_212"
OS: Linux 631697ed6903 4.9.0-8-amd64 #1 SMP Debian 4.9.144-3.1 (2019-02-19) x86_64 GNU/Linux
docker image: strapdata/elassandra:latest
Hello, i'm trying to create geo_shape index for field with following request:
PUT http://localhost:9200/weather
{     "settings": {     	"keyspace":"weather"	     },     "mappings": { 	    "weather_data" : { 	    	"_source": { 	    		"enabled": true 	    	}, 	        "properties": { 	        	"geom": { 	        		"type":"geo_shape" 	        	} 	        } 	    }     } }
got the error:
{     "error": {         "root_cause": [             {                 "type": "mapper_parsing_exception",                 "reason": "Failed to execute query:null : Existing column [geom] type [text] mismatch with inferred type [list<text>]"             }         ],         "type": "mapper_parsing_exception",         "reason": "Failed to execute query:null : Existing column [geom] type [text] mismatch with inferred type [list<text>]",         "caused_by": {             "type": "configuration_exception",             "reason": "Existing column [geom] type [text] mismatch with inferred type [list<text>]"         }     },     "status": 400 }
The weather table:
CREATE TABLE weather.weather_data (     id text,     updated_at timestamp,     alt int,     created_at timestamp,     es_options text,     es_query text,     geom text,     predict tinyint,     radius int,     time timestamp,     type tinyint,     version bigint,     weather text,     x int,     y int,     PRIMARY KEY (id, updated_at) ) WITH CLUSTERING ORDER BY (updated_at DESC)     AND bloom_filter_fp_chance = 0.01     AND caching = {'keys': 'ALL', 'rows_per_partition': 'NONE'}     AND comment = ''     AND compaction = {'class': 'org.apache.cassandra.db.compaction.SizeTieredCompactionStrategy', 'max_threshold': '32', 'min_threshold': '4'}     AND compression = {'chunk_length_in_kb': '64', 'class': 'org.apache.cassandra.io.compress.LZ4Compressor'}     AND crc_check_chance = 1.0     AND dclocal_read_repair_chance = 0.1     AND default_time_to_live = 0     AND gc_grace_seconds = 864000     AND max_index_interval = 2048     AND memtable_flush_period_in_ms = 0     AND min_index_interval = 128     AND read_repair_chance = 0.0     AND speculative_retry = '99PERCENTILE';
Why geo_shape index needs list type?
And if add 'cql_collection' parameter to index request:
{     "settings": {     	"keyspace":"weather"	     },     "mappings": { 	    "weather_data" : { 	    	"_source": { 	    		"enabled": true 	    	}, 	        "properties": { 	        	"geom": { 	        		"type":"geo_shape", 	        		"cql_collection":"singleton" 	        	} 	        } 	    }     } }
I got the following error:
{     "error": {         "root_cause": [             {                 "type": "mapper_parsing_exception",                 "reason": "Mapping definition for [geom] has unsupported parameters:  [cql_collection : singleton]"             }         ],         "type": "mapper_parsing_exception",         "reason": "Failed to parse mapping [weather_data]: Mapping definition for [geom] has unsupported parameters:  [cql_collection : singleton]",         "caused_by": {             "type": "mapper_parsing_exception",             "reason": "Mapping definition for [geom] has unsupported parameters:  [cql_collection : singleton]"         }     },     "status": 400 }
How can I create geo_shape index with field type 'text'?

By default, elasticsearch field are multi-valued, so field are backed by a Cassandra list.
cql_collection : singleton allow to back fields by X (where X is text, int, long etc....), but the elasticsearch geo_shape type parser does not inherits from all usual properties including the cql_collection, so it fails.
As a workaround, you could use geom list.
We have fixed that, and the next release will accept cql_collection: singleton for geo_shape.
Thanks.

Thanks, when i change geom filed type to 'list', i get the error in cassandra logs:
ElasticSecondaryIndex.java:2198 buildContext error com.fasterxml.jackson.core.JsonParseException: Illegal character ((CTRL-CHAR, code 0)): only regular white space (\r, \n, \t) is allowed between tokens  at [Source: $POLYGON ((0 0, 0 5000, 5000 0, 0 0));

There is bad characters in your json polygon.

Stale issue


Add support for the Bitbucket Build API

https://blog.bitbucket.org/2015/11/18/introducing-the-build-status-api-for-bitbucket-cloud/

Awesome! I wanted to do this but didn't get a chance to :+1:


Deleting folders when invisible files are left

When a folder contains only .acl files, the folder listing will show up as empty. This is by design, as we do not wish to show .acl files.
However, when trying to delete the container, it will fail because the container is non-empty; yet we have no way of emptying it because we cannot see the .acl files that are there.

P: The root container's ACL can't be deleted.
P: Hierarchical ACL inheritance algorithm.
P: Resources come to existence prior to server advertising an associated ACL.
P: The lifecycle of a resource and associated ACL (if exists) is coupled. Deleting a resource must remove associated ACL resource (not including inherited ACL). Deleting an ACL doesn't remove all resources that use that particular ACL.
C: A container can't contain any dangling ("invisible") ACLs of the resources under its containment.
Note: The view of a resource, however that is treated eg. whether acl and meta resources are to be "listed" in containment triples, or not, is both orthogonal and overruled by their own lifecycle.
Common resource delete rules apply.


Support Request

Support Request
A default label of Support will be added to the issue. Also add a second label for service associated to support request.  Then close issue.
valid service label options are:

Pingdom
PagerDuty
LastPass
Sentry
OS Data Hub APIs
SSL Certificate Management
Domains Management
MoJ GitHub Organisation
Docker
Other Support Request

Collaborator request


Support windows

I know you have windows branch. But this change is bits smaller that it.

Thanks for your work. Merged to master branch.


Spark Launcher communication with Spark API and kill submissions correctly

Coverage increased (+1.6%) to 53.012% when pulling 25afbfb93d13511503e904d1a00ba2ce9cbecf5b on compae:feature/kill-submissions-Spark-API into 1ffa459e27bfa0d8d35c75461e31fb62825ca07e on Stratio:master.

Coverage increased (+3.9%) to 55.298% when pulling 57dbe5c18243957564aedef33e0132b5f397a028 on compae:feature/kill-submissions-Spark-API into 091be0098c4537649585eca15066a8317738d241 on Stratio:master.

Coverage increased (+3.9%) to 55.272% when pulling 57dbe5c18243957564aedef33e0132b5f397a028 on compae:feature/kill-submissions-Spark-API into 091be0098c4537649585eca15066a8317738d241 on Stratio:master.

Coverage increased (+1.6%) to 52.978% when pulling 2d7b036355d395efbd782249d6c1c354afd1cc8b on compae:feature/kill-submissions-Spark-API into 091be0098c4537649585eca15066a8317738d241 on Stratio:master.


ROCm enable cwise op zeta

#159 disabled the zeta op for the rocm path.  This PR re-enables zeta to see if CI passes all tests.

jenkins: retest rocm-xla please

I have updated https://github.com/tensorflow/tensorflow/pull/29372 to include zeta.


Eliminate unnecessary casts.

Original pull request by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

This looks a lot better. Sorry for not spotting this earlier.
Bastiaan.

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

Roy, could you please make a pass through the list of PRs and pull those that look allright and comment on those that don't? Thanks a lot!
Bastiaan.


feat(CategoryTheory): if LR is abstractly isomorphic to the identity functor, then the unit is an isomorphism.

To do this, we give a general way to transport (co)monad structures on functors along isomorphisms of functors.

bors d+

Thanks for the reviews!
bors merge


GTK clean up and a few new features

You should read this PR commit by commit as the style update touches almost every line. The style update adds and removes spaces, and it removes empty and commented methods. It does not touch the documentation, there is still a lot of work to do there.
Please read the commit description of "lib/gtk: ignore deprecated warnings" for the rationale of the change.
GTK will be used to implement the abstract UI on GNU/Linux.

+1

Aside from my remarks, :shipit: !


Need editing/deleting capabilities

Need editing functionality. I would like to have the ability to mark out words and replace them with colored text. I think this functionality is available through another package, so it's really just a matter of finding the right one.
It would work like \add{blahblahblah} or \delete{blahblahblah}
Mostly this works as a to-do list for me.

I think soul supports this. It offers strikethrough and coloring, in exactly the same way you're using it for highlighting in the notes.
See this thread for where I was tipped off:
https://tex.stackexchange.com/questions/139463/how-to-make-hl-highlighting-to-automatically-place-incompatible-commands-in
Also looks like there may be some useful info there on how to make the notes not break in certain situations (citations, references, etc). This seems like important functionality, but I have not tested anything.


flutter.io home page suggestions

Take all of these suggestions with a grain of salt.  They are mostly just initial impressions and I might not have the context for why things are stated the way they are:

The first heading says "What is flutter?"  But that was answered above that question.  This heading would probably be better as "Why use flutter?" ((I see that's a later heading, maybe remove that one and put it here?  That section starts with "What are some advantages of Flutter?" so maybe that would be a better header for that second section.))
"The goal is to deliver apps that feel natural on different platforms, and we embrace differences in scrolling behaviors, typography, icons, and more." Might be better as:
"The goal is to deliver apps that feel natural on different platforms: we embrace differences in scrolling behaviors, typography, icons, and more." or just as two separate sentences.  And maybe "The goal" could be "Our goal"?
The two screen shots of the Shrine app feel odd to me - why are there two?  One is iOS one is Android?  that's not called out as to which is which.  Also the fact they look very different doesn't necessarily make me feel good about using flutter.  As an app developer I may want them to look identical, not different.  As a developer its nice to know it'll work on both but calling out that they will look different is not really a plus.  just my two cents.
"even non-programmers have made Flutter apps!" is begging for a link to something a non-programmer has written.  Do we have any public use cases we could link to?
"Do more with less code, even on a single OS, ..."  I'm not sure we really need to sat even on a single OS.  It's confusing as OS could refer to MacOS vs Windows vs Linux but I had to reread it to understand that you're referrign to the point above it.
"Realize custom, beautiful, brand-driven designs, without the limitations of OEM widget sets"  Not sure I understand what "without the limitiations of OEM widget sets" means.  Is it necessary?
"Unlike other frameworks that separate views, controllers, layouts, and other properties,"  this implies this is a bad thing but I think a lot of developers see Model-View-Controller design patterns as being a Good Thing.  This sentence feels like its saying this is better because it doesn't use that pattern...  Is that what we're really saying?
"There are widgets for padding, alignment, row, columns, and grids." => "There are widgets for padding, alignment, rows, columns, and grids."  Maybe we should also link to them all?
"These layout widgets do not show themselves in an app’s UI." => "These layout widgets do not have a visual representation in and of themselves"?
"For the complete set of libraries, see Flutter’s API documentation."  should this refer to layers instead?  The previous sentences all said "layers" but then we switched terminology to libraries.
"Choose a customized approach to unlock the full expressive power of the framework, or use building blocks from the widgets layer, or mix and match. You can compose the ready-made widgets Flutter provides, or create your own custom widgets using the same tools and techniques that the Flutter team used to build the framework."  I feel like these two sentences are saying the same thing?  I prefer the latter to the former so maybe drop the first one?
"You define the unique characteristics of a widget by implementing a build function that returns a tree of widgets."  This is the first time we mentioned tree.  Previously we were calling it a hierarchy of widgets, should we be consistent and call it a tree everywhere or a hierarchy everywhere?  My personal preference is tree as that's how I visualize it.
"which framework then stitches together into a tree." => "which are then stitched together into a tree."
"A widget’s build function should be free of side effects. Whenever it is asked to build, the widget should return a new tree of widgets regardless of what the widget previously returned."  This isn't in the documentation of the build function but it seems like a perfect thing to be in there.
"This automated comparison is surprisingly effective"  I'm not sure we should say we're surprised by it being effective...  It implies it wasn't deliberately made effective by us.
"to create a new instances of the child" => "to create new instances of the child"
"Install the Flutter SDK and set up IntelliJ (optional)."  We should split this into  two steps and label the second optional?  Otherwise it could be construed the entire step is optional.
The layer cake has low contrast between the font color and background color - could we switch to a lighter text color for the lower levels of the cake?

$18 is related to https://github.com/flutter/flutter/issues/8546

Thank you for the feedback. Implemented 1-3, 12, 16, 18.
9 -10 are TBD. If you'd like to understand why changes were/were not applied, I'm happy to meet!

Only 14. (which is for docs.flutter.io) is open at this point.

Not sure if we're still taking action here? If not (which is fine), we can close this?

Yes, the front page has been replaced. Closing.


Topic: Big data

The definition of big data is data that contains greater variety, arriving in increasing volumes and with more velocity. This is also known as the three Vs.
Put simply, big data is larger, more complex data sets, especially from new data sources. These data sets are so voluminous that traditional data processing software just can’t manage them. But these massive volumes of data can be used to address business problems you wouldn’t have been able to tackle before.

Fault tolerance in Spark -  Spark automatically recovers and recompute lost data or tasks using lineage information(which it creates for lazy job) which tracks the sequence of transformations applied to resilient distributed datasets (RDDs) which guarantees successful job completion even after node failures
contact -<EMAIL_ADDRESS>

Can the maintainers review


Fix support for overloading dispatcher with non-compatible first-class functions

Make it clear that Dispatcher.get_compile_result is raising NumbaError.
Fix casting from Dispatcher to FunctionType to handle the above error correctly.

(Fixes #9071)

Ah, look good to me!
If the caller has multiple functiontype sigs, before this patch, numba only tries the first sig, and throws an error if dismatch. After this patch, will try all sigs. I think so.

I am not familiar with Dispatcher code, and I didn't see side effects with this patch. Hope merge this ASAP!

can this patch go into 0.58 milestone? @esc


Fix/bindings get account assets

Description of the Change
Binding tests fixes for GetAccountAssets query that was missed in #1402
Benefits
Tests passes
Possible Drawbacks
None
Running Tests
cmake .. -DSWIG_JAVA=ON -DSWIG_PYTHON=ON -DSWIG_NODE=ON
make
ctest -R (python|java)

Well, GetAccountAssets tests work in docker develop image after some actions, but other tests fail.
  ..................................................x............................................................................terminate called after throwing an instance of 'std:
:out_of_range'
  what():  bitset::set: __position (which is<PHONE_NUMBER>2523542336) >= _Nb (which is 43)
Aborted (core dumped)

  ---
    operator: throws
    expected: '/Hash set should contain at least one hash/'
    actual:   undefined
    at: Test.<anonymous> (/opt/iroha/iroha/shared_model/packages/javascript/tests/queryBuilder.js:92:5)
    stack: |-
  ...

  24 tests
  126 passed
  1 failed

javascripts are indeed failing, but that behavior was before the pr and requires shared model permissions (and maybe smth else)


8505.1 DEV Write a migration to alter operationResourceName type to varchar(255).

https://github.com/Shippable/pm/issues/8505

Completed


Update Frame Selection by time to use an Epsilon

Use Case
The simularium-website experienced crashes and slider playback bugs following commit: 7f0fb26d95a5543c11e1e9c8bbd207288156f949 Fix/floating point time errors (update viewer to v2.9.2)
which updated frame requesting to use a floating point epsilon for comparison.
The bugs seem to happen when an unrounded time is sent back to the backend. Currently the back-end server returns the nearest frame with time strictly less than the request time.
Solution
Update the back-end server to return the closest frame to the requested time.
e.g., with frame A at time 1 and frame B at time 2, currently a request for 1.999 would return frame A; it is now expected to return frame B. Similarly, a request for frame 1.50001 will now return frame B instead of frame A.
We expect this update to resolve the observed bugs in the simularium-website.
Alternatives

@EricJIsaac Is this change deployed? Let me know when I can try removing the rounding in the frontend :)

@EricJIsaac Is this change deployed? Let me know when I can try removing the rounding in the frontend :)

The Jenkins build (auto-deploy) doesn't seem to be working. Will look into this today.


mantineFilterSelectProps data attribute

mantine-react-table version
1.0.0-beta.5
react & react-dom versions
18.0.28/18.0.11
Describe the bug and the steps to reproduce it
mantineFilterSelectProps has the following typings Partial(SelectProps) & HTMLPropsRef<HTMLInputElement> where both SelectProps and HTMLInputElement has data attribute.
There is a filterSelectOptions prop which partially solves the issue. If you want to pass other Select props, mantineFilterSelectProps will override options.
Maybe there should be an alias like htmlData for HTMLProps. Omitting data from SelectProps may be confusing and not reusable.
Thanks.
Minimal, Reproducible Example - (Optional, but Recommended)
// Type 'string[]' is not assignable to type 'readonly (string | SelectItem)[] & string'.
// Type 'string[]' is not assignable to type 'string'

const columns: MRT_ColumnDef<Reptilian>[] = [
  { filterVariant: 'select', mantineFilterProps: [{ data: ['hey'] }],
]

// works fine
const columns: MRT_ColumnDef<Reptilian>[] = [
  { filterVariant: 'select', filterSelectOptions: ['hey'],
]

// runtime error 'cannot read properties of undefined (reading 'map')'
const columns: MRT_ColumnDef<Reptilian>[] = [
  { filterVariant: 'select', filterSelectOptions: ['hey'], mantineFilterSelectProps: { sx: { width: 500 } }
]

// same error as in example 1
// when data is manually omitted in html, filterSelectOptions is overridden by data from mantineFilterSelectProps
const columns: MRT_ColumnDef<Reptilian>[] = [
  { filterVariant: 'select', filterSelectOptions: ['hey'], mantineFilterSelectProps: { data: ['hey hey hey'], sx: { width: 500 } }
]

Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
Yes, I think I know how to fix it and will discuss it in the comments of this issue
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

looks like I was wrong and filterSelectOptions does not exist ;(
just tested it like in the example: https://github.com/KevinVandy/mantine-react-table/blob/main/apps/mantine-react-table-docs/examples/customize-filter-variants/sandbox/src/TS.tsx

filterSelectOptions exists in the MUI version of MRT, but was removed in this Mantine version as the data is passed in mantineFilterSelectProps easy enough. There are some weird TypeScript types coming directly from Mantine though that seem to always result in an error. Will have to override that.

This type as been fixed in beta.19


shading seems broken between 1.0.0-RC2 and RC3

I upgraded sbt-ensime to RC3 (you should be able to easily reproduce that) and got this in the scripted tests, despite not shading jsoup
[info] java.lang.NoClassDefFoundError: Could not initialize class ensime.shaded.coursier.shaded.org.jsoup.nodes.Entities$EscapeMode [info] at ensime.shaded.coursier.shaded.org.jsoup.nodes.Document$OutputSettings.<init>(Document.java:372)

I minimised it to an upgrade from RC2 to RC3, still borken in RC5

I think it originates from jsoup being shaded in coursier since 1.0.0-RC3...  Its resources themselves are moved, but the hard-coded paths towards them are not adjusted accordingly. This can be fixed by suggesting the shading plugin to shade whole namespaces, like done here for example.
Supplying whole namespaces this way makes the shading itself faster, which (good news!) allows to shade scalaz along coursier too, if the scalaz namespace is provided (done in the example above).

You shouldn't need to shade the org.jsoup namespace in your case, as jsoup is now already shaded in coursier. You would need to shade coursier at least, and optionally scalaz.

thanks! So I just don't depend on jsoup anymore?

hmm, that still causes the same problem. Also if I have a list of shaded namespaces (just coursier), I still get the error. I guess that's because it's shading your shaded jsoup.
Would it maybe be best if you just don't shade anything and let that be a downstream decision? If you were going to shade anything, I'd want you to shade scalaz, but then that forces me to shade it and that's a lot of time to shade.

Huh, weird, do you an open PR with it somewhere? It works me in my modified Ammonite (and was working in https://github.com/lihaoyi/Ammonite/pull/604).

I'll push a branch for you to try https://github.com/fommil/ensime-sbt/tree/coursier-bug
Try the fastest test, which is:
scripted sbt-ensime/simple-build-sbt

shadeNamespaces seems to be overridden (reset) by ShadingPlugin.projectSettings... Putting the shadeNamespaces  ++= ... after it seems to work (I now get a ensime expect / diff error when running the scripted test, but that seems unrelated to shading proper).

Doh! Ok I'll try that. What setting did you use?

I only used shadeNamespaces += "coursier". You could shade scalaz too now, by adding "scalaz" there and not depending explicitly on it. It's fast enough thanks to shadeNamespaces now.

(Closing the issue...)

nice this seems to be working, thanks for the help!

do I still need to transitively depend on quasiquotes?

Not necessarily... you can try to shade it. (I don't think I tried to shade this one afaic, I only shaded coursier in 2.11.)


mwpw-152920 built facade for get app for mobile widget

resolves: MWPW-152920
Test links:
https://mwpw-152920--dc--adobecom.hlx.page/acrobat/online/test/pdf-editor-mobile-widget
https://mwpw-152920--dc--adobecom.hlx.page/acrobat/online/test/request-signature-mobile-widget
https://mwpw-152920--dc--adobecom.hlx.page/acrobat/online/test/sign-pdf-mobile-widget
https://mwpw-152920--dc--adobecom.hlx.page/acrobat/online/test/chat-pdf-mobile-widget
**It is important to test it on mobile devices (or developer tools -> mobile devices) so the layout gets displayed

Codecov Report
Attention: Patch coverage is 8.88889% with 82 lines in your changes missing coverage. Please review.

Please upload report for BASE (stage@a98a903). Learn more about missing BASE report.

Files
Patch %
Lines

acrobat/blocks/mobile-widget/mobile-widget.js
0.00%
81 Missing :warning:

acrobat/scripts/scripts.js
88.88%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##             stage     #690   +/-   ##
========================================
  Coverage         ?   88.85%
========================================
  Files            ?       37
  Lines            ?     4611
  Branches         ?        0
========================================
  Hits             ?     4097
  Misses           ?      514
  Partials         ?        0

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


⚠ Mail queue doesn't survive reboots ⚠

Context

Using the recommended volume configuration won't save your mail queue from restarts:
https://github.com/tomav/docker-mailserver/blob/f225e14a219824e4ebb34e63b144e6d7c1314bcb/docker-compose.yml.dist#L13-L17
What is affected by this bug?
Deferred mails in queue will not be resent after a reboot or container rebuild.
When does this occur?
When some mail gets deferred and queued and, later, you reconfigure your container (quite common for fixing some kind of problem, like a relay password that has changed), the mail queue is lost.
How do we replicate the issue?

Configure with SMTP_ONLY=1 and some DEFAULT_RELAY_HOST that makes the sent mails fail. The failure might be a wrong password or (in the case below) a wrong TLS setting.

Send a mail.

Execute postqueue -p in the container. You'll see an output similar to:
$ docker-compose exec smtp postqueue -p
-Queue ID-  --Size-- ----Arrival Time---- -Sender/Recipient-------
2F68C223A6C     3701 Tue Aug  4 11:30:32<EMAIL_ADDRESS>
            (TLS is required, but was not offered by host <IP_ADDRESS>[<IP_ADDRESS>])
<EMAIL_ADDRESS>4.

Fix the problem.

Rebuild with docker-compose up -d --force-recreate

Execute postqueue -p in the container.

Actual Behavior

The new mail queue will be empty.
Running this command seems to explain why:
$ postconf -p queue_directory
queue_directory = /var/spool/postfix

That's the directory where mail queue is stored, and it's not in any volume.
Expected behavior (i.e. solution)

The queue should be stored under /var/mail-state/spool-postfix. Or /var/spool/postfix should be a recommended mount point. Or maybe even /var/spool? Because this command shows that it's used also somewhere else:
$ postconf -p | grep /spool
queue_directory = /var/spool/postfix
smtpd_sasl_path = /var/spool/postfix/private/auth

Your Environment

Amount of RAM available: 8GiB
Mailserver version used: If the org.label-schema.vcs-ref label is right, it points to f225e14. In any case, the exact image is sha256:701355354a94a936006df1e7635eec5fff2cedf166c93fb6cffc0663e9ff0e34 downloaded from Docker Hub.
Docker version used: Docker version 18.06.1-ce, build e68fc7a
Environment settings relevant to the config: Ubuntu 18.04.1 LTS

@Tecnativa TT25079

I just noticed that /var/spool/postfix is a symlink to /var/mail-state/spool-postfix so there might be no config problems there:
root@1bdeba46f720:/# ls -l /var/spool/
total 8
drwxr-xr-x 1 root root 4096 Jul 24 09:28 cron
lrwxrwxrwx 1 root root    7 Jul 20 00:00 mail -> ../mail
lrwxrwxrwx 1 root root   29 Aug  4 12:01 postfix -> /var/mail-state/spool-postfix
drwx------ 2 root root 4096 Feb 26  2019 rsyslog

However, it's true that mail queue doesn't survive. Just check this out after restarting the container:
root@1bdeba46f720:/# tree /var/spool/
/var/spool/
|-- cron
|   `-- crontabs
|       `-- root
|-- mail -> ../mail
|-- postfix -> /var/mail-state/spool-postfix
`-- rsyslog

5 directories, 1 file

root@1bdeba46f720:/# tree /var/mail-state/spool-postfix/defer/
/var/mail-state/spool-postfix/defer/
`-- 2
    |-- 2C77D223A6D
    `-- 2F68C223A6C

1 directory, 2 files

root@1bdeba46f720:/# postqueue -p
Mail queue is empty

:see_no_evil: I think I'm becoming crazy... I just repeated all things and it worked fine.
Sorry for the noise, closing.


[10.1.x] ISPN-11816 Elytron 1.10.6

https://issues.redhat.com/browse/ISPN-11816

Closing as no longer required.


Problem with K3 not finding DI2 in KI2 APK file

Dear,
Can somebody help me with the problem that after intalling ki2 on my new hammer head karoo 3 device di2 is not found.
with my former models the karoo 1 and 2 there was no problem.
I Hope somebody can help me or tell this will be solved in a new version of Ki2?
I am looking forward to the response.
Best Regards,
MArc

Hi MArc,
it seems like Hammerhead still has to enable the SDK for the karoo 3 like they had for 1 and 2 before (see this thread).
You can go to this page and write Hammerhead a friendly message that the SDK availability (and thereby ability to use di2 with ki2) is a decisive aspect in your purchase decision.
Cheers

Oh dear... that doesn't sound good...

I sent an email to Hammerhead Support about this same matter and got a similar (although less specific) response:
"As of now, we have not implemented SDK capabilities on new Karoo and due to this, you wouldn't be able to install any third-party apps on new Karoo.
Though we do not have a timeline to share with you, we hope to see this SDK capability in the future.
Your request will be forwarded to the product team."
I have informed them I will be taking advantage of the 30-day return policy (and simply continue to use my K2) is SDK is not activated in the next few weeks.

Same here, I think we need more people to write them that the SDK for Ki2 is crucial:

As of now, we have not implemented SDK capabilities on new Karoo and due to this, you wouldn't be able to install any third-party apps on new Karoo.
Though we do not have a timeline to share with you, we hope to see this SDK capability in the future.
Your request will be forwarded to the product team.
Let us know if you have any further queries.

There now is a thread in the Hammerhead Support forum for this as well, feel free to express your support!

Thanks for the input and discussion here, looking forward to have Ki2 on the Karoo 3. I will close this issue and let's keep the discussion and tracking this on the #220.


Add support for Dropwizard 2.0.x

This change set adds support for Dropwizard 2.0.x while retaining support for Dropwizard 1.3.x in the dropwizard1 sub-project.
Refs #2554

Thanks! That was really quick, @joschi. Did you have a chance to try this suggestion, though? - https://github.com/line/armeria/issues/2554#issuecomment-595217934

@trustin Thanks for the pointer! I missed that comment and will update this PR accordingly. 👍

BTW, please ignore the build failure against Java 11. We have a flaky test.

@trustin I've removed the unused buildscript from "dropwizard" and "dropwizard1" and rebased the branch on current master.

Let me take care of the build failure later.

Merged. :clap::clap::clap: to @joschi!

@trustin @ikhoon @minwoox Thanks everyone for the smooth experience contributing to Armeria.
Keep up the good work!  👍

@joschi @trustin
Should this page be updated?
https://line.github.io/armeria/advanced-dropwizard-integration.html

Oh, yes, indeed. :sweat_smile:

@OneCricketeer The site has been fully updated: https://line.github.io/armeria/docs/advanced-dropwizard-integration/

looks great!


Concurrency Control Bug: Handling of PENDING State in Invocation Runners

Title: Enhancement: Include PENDING State in Concurrency Control Checks
Description:
Summary
There's a need to enhance our concurrency control mechanism by including checks for invocations in the PENDING state, in addition to those already in the RUNNING state. This adjustment aims to address a concurrency control bug where invocations in the PENDING state are overlooked, potentially leading to more invocations being processed than intended.
Issue Detail

The current implementation of is_authorize_to_run_by_concurrency_control only considers invocations in the RUNNING state when determining if a new invocation can start. This approach misses invocations that have been marked to start (PENDING) but haven't transitioned to RUNNING yet.
This oversight can lead to a scenario where multiple invocations that should be serialized based on the task's concurrency settings are allowed to run concurrently, violating defined concurrency constraints.

Proposed Change

Update the is_authorize_to_run_by_concurrency_control method to include checks for both PENDING and RUNNING states when fetching existing invocations. This ensures that all invocations about to run or already running are considered in concurrency control decisions.
Check again before task execution and abort the execution if another tasks is RUNNING (ignore PENDING)

Current Implementation Snippet
# Snippet from the current implementation
if invocation.task.conf.running_concurrency == ConcurrencyControlType.DISABLED:
    return True
running_invocation = next(
    self.get_existing_invocations(
        task=invocation.call.task,
        key_serialized_arguments=invocation.call.serialized_args_for_concurrency_check,
        status=InvocationStatus.RUNNING,
    ),
    None,
)
if not running_invocation:
    return True

Proposed Enhancement
# Enhanced implementation to include PENDING state
if invocation.task.conf.running_concurrency == ConcurrencyControlType.DISABLED:
    return True
pending_or_running_invocation = next(
    self.get_existing_invocations(
        task=invocation.call.task,
        key_serialized_arguments=invocation.call.serialized_args_for_concurrency_check,
        status=[InvocationStatus.PENDING, InvocationStatus.RUNNING],  # Check for both PENDING and RUNNING
    ),
    None,
)
if not pending_or_running_invocation:
    return True

Fixed in the wrong branch


Discriminator support for many values to shared types

I am trying to achieve a TPH approach with discriminators where there are multiple values that share the same type. I have a class structure as follows:
public class BaseLineItem
{
    public int Id { get; set; }

    public string LineItemType { get; set; } // Non-null discriminator

    // Other shared properties
}

public class LineItemType1 : BaseLineItem
{
    // Specific logic
}

public class LineItemType2 : BaseLineItem
{
    // Specific logic
}

And my configuration looks like this:
// BaseLineItem configuration
buidler.HasDiscriminator(e => e.LineItemType)
  .HasValue("DEFAULT")
  .HasValue<LineItemType1>("TYPE1")
  .HasValue<LineItemType2>("TYPE2")
  .HasValue<LineItemType1>("OTHER_TYPE_SIMILAR_TO_1")
  .HasValue<LineItemType2>("OTHER_TYPE_SIMILAR_TO_2");

The problem I am facing is that previous values specified with the discriminator gets wiped away and the last specified value types will be used instead. I expect that a line item with type == "TYPE1" OR type == "OTHER_TYPE_SIMILAR_TO_1" should both map to LineItemType1. Instead, I am restricted to LineItemType1 == "OTHER_TYPE_SIMILAR_TO_1" and LineItemType2 == "OTHER_TYPE_SIMILAR_TO_2". This can be seen in the query generated by EF Core:
SELECT *
FROM [LINE_ITEMS] AS [li]
WHERE [li].[lineitem_type] IN (N'DEFAULT', N'OTHER_TYPE_SIMILAR_TO_1', N'OTHER_TYPE_SIMILAR_TO_2')

Further technical details
EF Core version: 2.2.3
Database Provider: SqlServer (2.2.3)
Operating system: Windows 10 (1809)
IDE: Jetbrains Rider (2019.1 EAP 6)

Note for triage. Related to #10140 and #4722.

From #10140:

Would an explicit list of discriminator values, instead of just one, work for you instead of the wildcard? In other words, you could say that values "A", "B", and "C" will all result in creation of the same type "X".

Perhaps something like the following similar to this:
buidler.HasDiscriminator(e => e.LineItemType)
  .HasValue("DEFAULT")
  .HasValue<LineItemType1>("TYPE1", "OTHER_TYPE_SIMILAR_TO_2")
  .HasValue<LineItemType2>("TYPE2", "OTHER_TYPE_SIMILAR_TO_2");

In such a case, when encountering type "X", which discriminator value would you expect EF to choose when none is set?

buidler.HasDiscriminator(e => e.LineItemType)
  .HasDefaultValue<BaseLineItem>("DEFAULT");

Or use the base type (given it is not abstract). Or throw an exception during configuration if a default value type is not specified.

Closing as duplicate of #10140


write an executable equivalent to dragonfly serve

That logic can be found here: https://github.com/project8/dragonfly/blob/d70d20a42c4f8375dbb69b313b6e78d617a8980b/dragonfly/subcommands/open_spimescape_portal.py#L28
This should make use of scarab's CLI feature set

There's an initial version of this included in PR #32


Optimize sorting for aggregations over sorted inputs

When multiple aggregations share sorting keys and orders, sort inputs once.

@mbasmanova : This is a nice change. Thanks !


fix/ perp curie errors

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
https://github.com/hummingbot/hummingbot/issues/5693
This rp fixes some issues with perp connector
Tests performed by the developer:

Did end-to-end testing between client and qa and ensure I don't have error.
Ensured that unit tests pass.

Tips for QA testing:

Basically do end-to-end testing and confirm issue doesn't persist.

@vic-en Could you please resolve branch conflicts?

Tests performed:
Cloned and installed fix branch
gateway generate certs and gateway create - confirmed working
Client testing: blocked with issue on spot-perpetual https://github.com/hummingbot/hummingbot/issues/5890
test endpoints
ping gateway endpoint - confirmed working
/amm/perp/market-prices - confirmed working
/amm/perp/market-status - confirmed working
/amm/perp/pairs - confirmed working
/amm/perp/position - confirmed working
/amm/perp/open - confirmed working
/amm/perp/close - confirmed working
/evm/approve - confirmed working

@nikspz Resolved.

PR update:

Setup long term bots for test monitoring
We have observed that there are no Unhandled error in background task: unsupported operand type(s) for -: 'NoneType' and 'decimal.Decimal' for the longest time
Noticed issues on order execution which is not related to this PR
There were instances of 'ServerDisconnectedError' on both AWS and local tests

While this happened we review the docker log files (attached) and we'd like to confirm if this is possibly something to do with the default nodeURL rpc.ankr.com/optimism since its return an error of https://links.ethers.org/v5-errors-CALL_EXCEPTION?

2022-11-16 10:44:59,004 - 75262 - hummingbot.core.gateway.gateway_http_client - WARNING - Call to https://localhost:31207/amm/perp/position failed. See logs for more details.
2022-11-16 10:44:59,007 - 75262 - GatewayEVMPerpetual - ERROR - 'ServerDisconnectedError' object has no attribute 'get'
Traceback (most recent call last):
  File "/home/ubuntu/5885/hummingbot/connector/gateway/amm/gateway_evm_perpetual.py", line 456, in _status_polling_loop
    await safe_gather(
  File "/home/ubuntu/5885/hummingbot/core/utils/async_utils.py", line 22, in safe_gather
    return await asyncio.gather(*args, **kwargs)
  File "/home/ubuntu/5885/hummingbot/connector/gateway/amm/gateway_evm_perpetual.py", line 555, in update_positions
    trading_pair = f"{position.get('base')}-{position.get('quote')}"
AttributeError: 'ServerDisconnectedError' object has no attribute 'get'

We also observed the error 'ValueError' object has no attribute 'get' and call....perp/position failed after getting the ServerDisconnectedError
2022-11-18 18:11:38,107 - 3311 - hummingbot.core.gateway.gateway_http_client - WARNING - Call to https://localhost:26762/amm/perp/position failed. See logs for more details.
2022-11-18 18:11:38,107 - 3311 - GatewayEVMPerpetual - ERROR - 'ValueError' object has no attribute 'get'
Traceback (most recent call last):
  File "/home/rapcmia/github/5885/hummingbot/connector/gateway/amm/gateway_evm_perpetual.py", line 456, in _status_polling_loop
    await safe_gather(
  File "/home/rapcmia/github/5885/hummingbot/core/utils/async_utils.py", line 22, in safe_gather
    return await asyncio.gather(*args, **kwargs)
  File "/home/rapcmia/github/5885/hummingbot/connector/gateway/amm/gateway_evm_perpetual.py", line 555, in update_positions
    trading_pair = f"{position.get('base')}-{position.get('quote')}"
AttributeError: 'ValueError' object has no attribute 'get'

Pending:

Running long term bot using optimism infura nodeURL https://optimismmainnet.infura.io/v3/... to compare test results

5885.zip

@vic-en can you please resolve conflicts in this PR? Thanks


FEAT :  Improve compare page

Title
UI/UX Enhancements for Compare Page
Enhancement Aim
Improves Dialog Boxes, Graph Visibility, About Text Styling and overall UI.
Changes
The compare page as of now can be improved using the enhancements in the following areas :

Improving crypto1 & crypto2 dialog box in dark mode and its alignment
Improving the graph size and visibility across themes
Improving the crypto's about text below the graph using better fonts and styles.

Screenshots 📷

Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

Full Name
Irtesaam Atfi
Participant Role
GSSoC
WoB
Hacktoberfest

Hey @Irtesaam  tell me about WoB this is also GSSoc type

Hey @Irtesaam tell me about WoB this is also GSSoc type

It's merged with gssoc

Hey @Irtesaam tell me about WoB this is also GSSoc type

Yes. Both are organized by GirlScript foundation and have kind of similar projects. However, WoB focuses more on blockchain related projects.
You can have a look at WoB projects over here Projects


fedora: Port downstream PR-49

https://src.fedoraproject.org/rpms/python-scikit-build-core/pull-request/49

Oh, should have made it a draft. packit is having trouble updating, it didn't get to run these. Anyway I can still check back on main to see the status later


onTouchTap doesn't allow stopping of event bubbling

I've got an <IconButton /> inside of a table row. When I click the button, the row selects.
<IconButton onTouchTap={e => e.stopPropagation()}><Delete /></IconButton>

However, when I change the event handler to this:
<IconButton onClick={e => e.stopPropagation()}><Delete /></IconButton>

Then it works, and clicking the button does not select the row. Is there some way we can make onTouchTap able to stop event bubbling?

have you tried e.preventDefault()?

Yes, that doesn't work either. (though preventDefault is not about event bubbling)

@ffxsam does this look related? https://github.com/zilverline/react-tap-event-plugin/issues/54

I don't know. It might be. As a workaround, I'll just have to use onClick on any buttons contained within a table row. I suppose this is more of a bug in the tap event plugin, not MUI.

I am not sure if I understand this completely but I did work on a similar issue w.r.t event bubbling within the ListItem component on a dropdown icon click.
The issue was in the TouchRipple component and putting a event.stopPropagation() line in the handleTouchTap(event) function of TouchRipple worked.
https://github.com/callemall/material-ui/pull/3593

@ffxsam Looks like the solution has been found. I am closing the issue. However, if you still have questions regarding it, you can comment below and we will look further into it


Speeding up adding factors + computing wiring

Main goal of this PR is to speed up the process of adding factors to a FactorGraph and to compile wiring. In particular

we do not instantiate factors in a FactorGraph by default. Instead all the computations, including wiring, happen at the factor groups level.
all the factors-related operations are computed on demand, when the user requires them
we create a subfolder pgmax/groups/ to clarify the structure

Before:

creating FG for the RBM exp takes 25s, building run_bp takes 45s
creating FG for the convor exp takes 20s, building run_bp takes 25s

After:

creating FG for the RBM exp takes 2s, building run_bp takes 4s
creating FG for the convor exp takes <.1s, building run_bp takes 10s

Codecov Report

Merging #129 (b5697fb) into master (81c4f20) will decrease coverage by 4.17%.
The diff coverage is 94.93%.

@@             Coverage Diff             @@
##            master     #129      +/-   ##
===========================================
- Coverage   100.00%   95.82%   -4.18%
===========================================
  Files           10       13       +3
  Lines          854      911      +57
===========================================
+ Hits           854      873      +19
- Misses           0       38      +38

Impacted Files
Coverage Δ

pgmax/factors/enumeration.py
92.18% <ø> (-7.82%)
:arrow_down:

pgmax/factors/logical.py
100.00% <ø> (ø)

pgmax/fg/groups.py
96.49% <86.95%> (-3.51%)
:arrow_down:

pgmax/fg/graph.py
92.95% <90.00%> (-7.05%)
:arrow_down:

pgmax/groups/logical.py
91.83% <91.83%> (ø)

pgmax/groups/enumeration.py
97.27% <97.27%> (ø)

pgmax/fg/nodes.py
100.00% <100.00%> (ø)

pgmax/groups/variables.py
100.00% <100.00%> (ø)

... and 4 more

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 81c4f20...b5697fb. Read the comment docs.

Thanks for your thorough review @StannisZhou
There are a few points we should discuss related to mypy behaviour
I totally agree that the next PR should be focused on speeding the factor group wiring using numba!


Fix 500 when repo has invalid .editorconfig

Creating a notice instead
Fixes gogits/gogs#3643

Current coverage is 2.18% (diff: 100%)

Merging #59 into master will not change coverage

@@            master       #59   diff @@
========================================
  Files           31        31
  Lines         7508      7508
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           164       164
  Misses        7327      7327
  Partials        17        17

Powered by Codecov. Last update 137dcbf...a53a62d

Any chance you could add a test for this ?
I've tried this locally and confirm the bug and the fix, but I hadn't found where the notice about broken .editorconfig was sent. Where should I look ?
Other than that, LGTM

@strk
See the original issue https://github.com/gogits/gogs/issues/3643
To add a test, we should first have a mock of the git interface, or a real git repo, since this touch git. It's something for the future.
Also, how and why did you add two commits to this PR?

I've added two commits by clicking "Update branch" button in github interface. I belive another button will let me rebase/squash before merge. Will try right now, as this LGTM (ok for postponing test)

Uhm, it takes a second LGTM, mine was already counted

Needs a rebase.

Rebase can be done via github button, as long as LGTM passes

Rebase can be done via github button, as long as LGTM passes

Nope, there is no button to update, so maybe there are merge conflicts.

Yup, looks like being the case, @andreynering can you manually rebase and resolve conflicts (then I guess it'll need another LGTM round ?)

Merging  manually shows no conflicts, should I try pushing the merge via commandline ?

The branch is out of sync, that's why this can only be done by admins, but we should avoid that.

I remember having seen an "Update branch" button in other PRs, when they were out of sync, did any configuration change about that ? (or maybe the branch owner had the branch protected..)


feat: implement dht values validators

Overview
Closes https://github.com/celestiaorg/orchestrator-relayer/issues/72 and https://github.com/celestiaorg/orchestrator-relayer/issues/73

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

check that the evm address is one that is expected

There is no way to know that the EVM address is one that is expected from the P2P network perspective

we're we're not querying for keys unless they are from the validator set

anyone can post faulty signatures (if they tweak the orchestrator-relayer software), but a relayer will always query the correct thing that will be 100% accepted by the QGB contract. Because the same checks done on the QGB contract are done in these checks (including implementing this https://github.com/celestiaorg/orchestrator-relayer/issues/224)


Clean up dlink and some bug fix

Description:
Clean up dlink and some bug fix

current_power_watt -> current_power_w

Add available property

When the connection to the device fail. Make sure we wait before retrying. Reducing the chance for this issue https://github.com/LinuxChristian/pyW215/issues/9

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

cc: @LinuxChristian

I think we are ready to merge?


[VS Code] Joining workspace: The host has blocked you from joining the collaboration session.

<!--
For Visual Studio problems/feedback, please use the Report

Ban not revelant

@Risuro This message usually indicates that the host removed you from the session. Do you know if they accidentally did that? Was the host in VS? The way to work around this is to simply re-start the session, since you're only blocked for the duration of the session, not permanently.
In the meantime, I'm going to close this issue as a duplicate of #1199. Thanks!


Fixed an issue that the property file was not closed

Fixes https://github.com/googleapis/gax-java/issues/337

Codecov Report

Merging #338 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #338      +/-   ##
===========================================
+ Coverage     72.77%   72.8%   +0.02%
  Complexity      409     409
===========================================
  Files            81      81
  Lines          1987    1989       +2
  Branches        161     161
===========================================
+ Hits           1446    1448       +2
  Misses          473     473
  Partials         68      68

Impacted Files
Coverage Δ
Complexity Δ

...va/com/google/api/gax/core/PropertiesProvider.java
37.03% <100%> (+5.03%)
3 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

@garrettjonesgoogle PTAL

LGTM


National Bank of Poland - PR offer

Hello,
I just implemented National Bank of Poland Service in my application using Swap. Would you be interested of me porting it to this package?

@fmasa Sure!


Add Catalyst installation example

Depends on https://github.com/realm/realm-cocoa/issues/6259.

Is there any summary of how to install RealmSwift for catalyst short of this issue being completed?

does this mean catalyst is not supported for realm-swift?
the realm react native appears to since mid 2020.
I enabled catalyst and ran my working app on the Mac to an immediate crash in realm:
2022-04-19 15:27:13.838957-0500 ctx[31206:7589368] *** Terminating app due to uncaught exception 'RLMException', reason: 'Keychain returned unexpected status code: -34018'
*** First throw call stack:
(
0   CoreFoundation                      0x00000001809eeb08 __exceptionPreprocess + 240
1   libobjc.A.dylib                     0x0000000180739e14 objc_exception_throw + 60
2   ctx                                 0x000000010494021c _ZL17RLMTranslateErrorIZ49-[RLMApp initWithId:configuration:rootDirectory:]E3$0EDaOT + 128
3   ctx                                 0x0000000104940034 -[RLMApp initWithId:configuration:rootDirectory:] + 436
4   ctx                                 0x0000000104940550 +[RLMApp appWithId:configuration:rootDirectory:] + 296
5   ctx                                 0x00000001049406ac +[RLMApp appWithId:configuration:] + 96
6   ctx                                 0x000000010494076c +[RLMApp appWithId:] + 72
7   ctx                                 0x0000000104847094 $sSo6RLMAppC2idABSS_tcfCTO + 56
8   ctx                                 0x00000001048f432c $s3ctx0A3AppVACycfC + 148
9   ctx                                 0x00000001048f4558 $s3ctx0A3AppV7SwiftUI0B0AadEPxycfCTW + 12
10  SwiftUI                             0x00000001bda09550 $s7SwiftUI3AppPAAE4mainyyFZ + 112
11  ctx                                 0x00000001048f4224 $s3ctx0A3AppV5$mainyyFZ + 40
12  ctx                                 0x00000001048f456c main + 12
13  dyld                                0x0000000108a19088 start + 516
)
libc++abi: terminating with uncaught exception of type NSException
*** Terminating app due to uncaught exception 'RLMException', reason: 'Keychain returned unexpected status code: -34018'
terminating with uncaught exception of type NSException

Incidentally did this as part of adding visionOS support in #8309.


directoryDomain - Formats / Examples

Not exactly an issue, Trying to setup my creds for SAML. But not sure what is the exact format.
FQDN?
or
just the corp-directory-name.com?
Is it possible for you to provide some default examples for say:Okta, Azure AD etc.,

Hi @miztiik,
Sorry for the late answer. I build this tool to simplify my day-by-day login to AWS's SSO Solution, so didn't have a chance to test in Azure on something like this.
Generally, it should look like "directoryDomain": "https://directory.mycorp.com" but you fail to login, please provide me the error log and I will happily investigate it and try to help you!

Thanks for response, Lets close this, At this moment, this tool may not able able to satisfy my needs and have removed my setup and unable to provide any more logs.
Appreciate your efforts though.

@miztiik thank you too!
BTW, if you have some ideas how to improve this tool - please open an issue with enhancement tag, we will try to handle in next releases.


5.1 sound

I still don't know support it youtube or not, I've read some saying they can play 5.1 sound in chrome, but youtbe-dl downloads all in stereo. If there are no any 5.1 enabled video in youtube, why there are many tests like this:
https://www.youtube.com/watch?v=PqVCPE8_ntE&hd=1

How to enable 5.1 audio in Chrome? For this video I got itag 251 in Chromium, which is stereo.

http://superuser.com/questions/820845/how-to-get-5-1-audio-in-google-chrome/1055177#1055177

In chrome://flags/, there's no option #try-supported-channel-layouts. I've also tried flags from https://www.youtube.com/watch?v=dcUZXRYroZM. Maybe youtube won't use 5.1 audios if I don't have a 5.1 output?

Did you try add params to the Chrome shortcut or cmd? And maybe it depends on Adobe flash player, But i've read that youtube's 5.1 is not native, something with html5.

Yes I've tried --disable-audio-output-resampler, --try-supported-channel-layouts and --force-wave-audio.

maybe it depends on Adobe flash player

I can't find a way to force Flash on YouTube. And it does not make sense - Google is urging on killing Flash.
Can you right-click on the video and choose "Stats for Nerds"? What's the value of item "DASH"?

160/250
I have an old Chrome and Win XP, try to add params to Chrome, but maybe it's too old to support it, Can't get make sounds from center, rear speakers and woofer.

Audio 250 is also stereo.

Can't get make sounds from center, rear speakers and woofer.

Actually there are also rumors that YouTube does not support 5.1 audios at all. See https://productforums.google.com/d/msg/youtube/jMoMzEyFJak/OrJWAeMCAQAJ
It's impossible to have youtube-dl support more than browsers, so I'm closing this. If you find a way to play 5.1 audios from YouTube, feel free to leave comments and I'll reopen this.

I have used Youtube apps on TV hooked to a receiver and I am able get 5.1 audio.
even youtube allows to 5.1 +stereo
Advanced encoding settings
Audio codec: AAC-LC
•Channels: Stereo or Stereo + 5.1
•Sample rate 96 khz or 48 kHz
and why youtube-dl -F dose not show this available

Most likely youtube-dl mimics normal browsers, so YouTube returns stereo audios only. It may help if you specify the User-Agent string of that App via --user-agent.

I have found few samples which youtube-dl can download with 5.1 quality:
https://www.youtube.com/watch?v=FgulvA9sc_s
https://www.youtube.com/watch?v=DRjbuXjNwrk
https://www.youtube.com/watch?v=Ec1pwCHrm-8
https://www.youtube.com/watch?v=thedigitaltheater

Hopefully, this will help you to investigate what's wrong with other videos. I can see two possibilities: they either were uploaded before youtube started to support 5.1 or it is an youtube-dl issue.
P.S. Once downloaded, you can check media info with the following:
ffprobe <file>

Once again: Youtube does not support 5.1. None of these videos are actually 5.1.