application attempts to claim a reward repeatedly

The Issue
When entering the publish section the application tries to claim the reward for "new_channel" which might eventually stress the server/api and could be avoided entirely.
Steps to reproduce

Open the application and check if you already claimed the "new_channel" reward (Channel Surfing). If that's a positive then move to the next step
open the developer console and watch the network for XHRs
click on Publish
observe the post request "new" return error 400

Expected behaviour
The application should cache already claimed rewards and never attempt to claim again
Actual behaviour
each time the publish tab is opened, a request is posted to https://api.lbry.io/reward/new with payload
app_id:2DKyAv7T9Ro8B9biH9j555Ry1wRHvsUhcg16pZnV****************
reward_type:new_channel
wallet_address:bPgwZZ1Wmb1RK1iDzJkCEtC1dAC51AQj4p
transaction_id:f3e57c42c74ede58d3d02d322ab3737682c2624dc98b805e48c2790547a7ef69

and an error 400 is  returned
{
  "success": false,
  "error": "This reward has already been awarded to this person.",
  "data": null
}

System Configuration
{
  "id": null,
  "jsonrpc": "2.0",
  "result": {
    "build": "rc",
    "ip": "*********",
    "lbrynet_version": "0.11.0rc1",
    "lbryschema_version": "0.0.6",
    "lbryum_version": "2.7.24",
    "os_release": "4.4.0-78-generic",
    "os_system": "Linux",
    "platform": "Linux-4.4.0-78-generic-x86_64-with-Ubuntu-16.04-xenial",
    "processor": "x86_64",
    "python_version": "2.7.12"
  }
}

This doesn't happen in upcoming branch (though it still does and will happen on views).


Added --extensions argument

Currently, watchgod doesn't watch files with non-python extensions. As an example, this brought an issue with not reloading when changing ".gql" files for the graphQL server in one of the projects.
With this change, --extensions argument is provided in CLI mode which with passing additional extensions, enables watch for those files too. While this could be done easily by implementing a custom watch, but this fix is handy and makes the end project cleaner and out of watching codes.

thanks so much.


When adding IDL section, if the container is an h2, don't make another one

I want to write the following:

Interface summary

But in addIDLSection, when we fetch the fill container we always add a child h2, even if the container itself is a h2.
Ideally it should even be possible to add some introductory prose between the custom heading and the IDL summary. Maybe we could drop the h2 generation altogether? Or only drop it when the fill container is not the body element?
This is needed to fix https://github.com/w3c/web-animations/issues/49

I could possibly have it check if the element is a heading and append after versus inside, but why do you need this in the first place?  Can't you just do
?  Seems like that would fix w3c/web-animations#49

Then you'd end up with a heading that says "IDL index" instead of "Interface summary".

Oh, so the thing is that you want to control the heading name for some reason? Note that the section is more than just interfaces.

I guess I don't care that much. I was thinking of interface in the sense of "Programming interface".

Can't you just do ?
I did that but unfortunately the heading it generates has no number so it messes up the document structure. I just removed the section for now and left the IDL index in its default positioning.


API: PyPDF2 deprecation

User Story - Business Need
PyPDF2 is deprecated and should be replaced with pypdf.  See the note here.  This change should be made in concert with a similar change to notification-utils, which also depends on PyPDF2.

[x] Discussed with QA

User Story(ies)
As a stakeholder in VA Notify
I want to keep all dependencies up to date
So that the application remains secure
Additional Info and Resources

PyPDF changelog regarding PyPDF2

Engineering Checklist
Required

[x] In requirements-app.txt (or Poetry), replace the PyPDF2 requirement with pypdf.

Acceptance Criteria

[x] Containers build without error using pydf
[x] All unit tests pass

QA Considerations
For QA to populate. Leave blank if QA is not applicable on this ticket.

[x] regression passes in Perf
[x] Close twistlock when this is merged to master.

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @justaskdavidb2 @k-macmillan @kalbfled @ldraney @nikolai-efimov

Updated to note that the only route using pypdf is /v2/notifications/<notification_id>/pdf.
Portal is not using that functionality.
There is a ticket #1178 to remove that route.
There will be no QA needed other than running a regression and closing the QA ticket.

Deployed to perf on master, regression passes on Perf.
132 passed, 2 skipped, 3 xfailed in 148.24s (0:02:28)


Renaming a crate

Steps to reproduce:

write an integration test that uses the main binary like this: ...Command::cargo_bin("cli-name").unwrap()
run the tests
rename the crate in Cargo.yaml (thus renaming the binary)
break some code
run the tests again

The tests pass! One easy solution is to use env!(CARGO_PKG_NAME), but I'm more concerned about it having used a stale cache without even noticing it was doing the "wrong thing".
I can see how this happens and that this crate is not doing anything wrong, but I just wanted to point this out.

Yeah, can't think of anything we should do for this but at least we now have a searchable problem / solution.  Thanks!


Fix the way formatters are selected in update_build_files.py

Add logs for testing formatter issues
More logs
Cleanup logs + change condition

Current dependencies on/for this PR:

main

PR #1   👈

This stack of pull requests is managed by Graphite.


Support merging jobs with config

2 commits.

It looks like this is merging based on an exact job match, which happens to work well for some current examples, but I don't think is actually what we want.
Consider two packages with the following mono_pkg.yaml files:
package:a
dart:
  - dev
stages:
  - unit_test:
    - test: -p chrome
      merge: true

package:b
dart:
  - dev
stages:
  - unit_test:
    - test: -p vm
      merge: true

Ideally, those two tasks would still be able to be merged together even though the package:test args are slightly different.
However, I do see how the tasks are currently set up and ran in the travis.sh script doesn't really support running different types of tasks, and that might be more difficult to support. We could potentially create an environment variable for the tasks which is a 2D array of tasks (a list of tasks per package) instead of using command line args?

If we do go down the more general path - I also think a bit more fine grained control would be nice, using possibly a "group id" or something instead of just a "merge" boolean. That would allow you to make multiple groups of tasks within each stage.

@jakemac53 see latest. much better.

@jakemac53 this needs some testing, but I think it's ready to land.

I'll let Jake and Gary take this one.


local_image_embedding

Hi @tanshuai0219 @Biao-Gong et. al.
thx for ur great work! may i ask for the reference in the main text of local_image_embedding cf. below? thx & rgds,
https://github.com/antgroup/animate-x/blob/fdc80909f911d8e487cb4e6847f2e4c6501b62af/animatex/model/unet_animate_x.py#L543

Hi, thanks for the interest in Animate-X.
local_image_embedding is not specified in the main text. But you can refer to Unified noised input in the paper "UniAnimate: Taming Unified Video Diffusion Models for Consistent Human Image Animation". Our implementation is based on UniAnimate .


fix(tokens): automated token update

Created by the update-tokens GitHub Actions workflow

:tada: This PR is included in version 2.0.0-next.10 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


inner product in the code?

Line 3 of Algorithm 3 in the paper conjugates G by QL and QR. However, the code only seems to keep track of GG^T and L (outer_product) and not G^T G and R. Where is G^T G computed in the code?

state['GG'] is a list which maintains both GG^T and G^TG.

state['GG'] is a list which maintains both GG^T and G^TG.

It seems that this is not reflected in the code


After call "compute_face_descriptor", process exits very slowly.

python：3.8.10
dlib: 19.22 with CUDA 11.0 , compiler : visural studio 2017
os: win10 x64
issue: Run "dlib-19.22\python_examples\face_recognition.py" in command line environment, if you call "compute_face_descriptor", it will takes a long time for the process to return to the console after the last line of code has been executed.   How to deal with it?

I have  this same issue, for both CUDA versions 10.2 and 11.6 and with dlib versions from 19.21 to 19.23.
Using Visual Studio 2019 on x64 Win, crashes with the error code
`Processing file: C:/Projects/dlib_test/images\image.jpg
Number of faces detected: 1
Detection 0: Left: 123 Top: 430 Right: 1503 Bottom: 1810
Process finished with exit code -1073740791 (0xC0000409)`
I added printouts and also stepped in debug to verify that it does not go past the compute_face_descriptor function.


Show full commit message when hovering over blame

Testing #235028
I think it would be useful to see the entire commit message when hovering over blame annotation. Most of the subject is already shown in the editor so showing it again in the hover feels redundant

Agreed! The git blame output only contains the git commit subject so displaying the full commit messages requires more work to be done. I will investigate how to add the full commit message to the editor decoration hover and the status bar item hover.


dsitributed on mulit-CPU

Just wondering if there is way to distribute over CPU  (single node, or multi nodes).
It would be very useful features for some sparse models.

I don't think this is supported for now. How do you use PyTorch for this use case? Accelerate is syntax sugar over PyTorch, so if the feature does not exist in PyTorch, we probably won't be able to add it.

Horovod can handle on distributed CPU:
https://github.com/huggingface/accelerate/issues/32#issuecomment-822466053

Ok, so support for Horovod is what you need. It's on the roadmap :-)

I don't think this is supported for now. How do you use PyTorch for this use case? Accelerate is syntax sugar over PyTorch, so if the feature does not exist in PyTorch, we probably won't be able to add it.

Could you please consider support for MPI backend in torch DDP? Ideally, I want to specify backend from command line argument so I can specify some custom backend. An example use for Intel oneCCL backend (using Intel torch-ccl) can be found at https://github.com/ddkalamk/transformers/blob/pcl-v4.0.0/examples/question-answering/run_squad.py#L746
Assumption is we launch the application using MPI (similar to horovod) and based on environment variable initialize distributed backend.

Will look into this one too!

Should be fixed by #63


bug: cosmoshub does not support faucet type: starship

When using the following settings with the 0.2.2 helm chart:
chains:
-  id: cosmoshub-test
    name: cosmoshub
    faucet:
      enabled: true
      type: starship

The cosmoshub-test faucet pod fails to start:
{"level":"error","ts":1718310048.524163,"caller":"app/distributor.go:286","msg":"send token failed","cmd":"gaiad tx bank send cosmos1vhdew4wqu3tp8l2d55aqcc73aqvr0rr9yk4a0l cosmos1prdhkqcdzz2dqp3qwpmecna6e7faghkkqk2p9p<PHONE_NUMBER>0stake --chain-id=gaialocal --fees=10000000uatom --keyring-backend=test --gas=auto --gas-adjustment=1.5 --yes --node=http://localhost:26657","error":"err: exit status 1, stderr: Error: rpc error: code = Unknown desc = rpc error: code = Unknown desc = failed to execute message; message index: 0: 0stake is smaller than<PHONE_NUMBER>0stake: insufficient funds

2024-06-13 16:15:36 err: exit status 1, stderr: Error: rpc error: code = Unknown desc = rpc error: code = Unknown desc = failed to execute message; message index: 0: 0stake is smaller than<PHONE_NUMBER>0stake: insufficient funds [cosmos/cosmos-sdk@v0.45.16/x/bank/keeper/send.go:186] With gas wanted: '0' and gas used: '60680' : unknown request

The @cosmjs/faucet faucet works, but does not support sending tokens to ICA addresses. See this pull request for an upstream fix: https://github.com/cosmos/cosmjs/pull/1593

let me have a look

As for cosmjs we can wait for the upstream PR to be merged, then we can create a new version of the faucet based on the new version of cosmjs

@0xpatrickdev thanks for opening this issue. Fixed the issue in chart version 0.2.4. Please upgrade and let me know if it works.
More over, will upgrade the cosmjs version as well, once your PR is merged.


Set request_rssi Timeout

I'm writing a script that scans for a list of bluetooth addresses periodically, but I've noticed that if a device is not in range it takes a full five seconds to timeout .request_rssi(), is there a way to change how long this timeout is?

According to me, after the device has gone out of range, it takes approximately 5 seconds to connect back to the device.


Trello-131: default sort order is by created_at DESC, N/A when no results

Default sort ordered by most recent imports

Also added 'N/A' where no data available for Imported, Rejected and Imported By

The issue was after creating a new import and this is still broken

The issue was after creating a new import and this is still broken

fixed

fixed


Zstd version uses an out of date CMake

Zstd has fixed the warning produced by CMake for min versions lower than 3.5. See https://github.com/facebook/zstd/pull/3807.

Thanks for opening this, I will take a look


Having trouble uploading files

I am trying to automate uploading of local files to a pocketbase collection for storage. I am running this script with node:
const fs = require(‘fs’);
async function main() {
  const pocketbase = (await import(‘pocketbase’)).default;
  const pb = new pocketbase(‘<pocketbase_url>‘);
  const formData = new FormData();
  // Read files from local system
  const files = fs.readdirSync(‘<path/to/folder>’);
  files.forEach(file => {
    formData.append(‘multiqc_report’, fs.readFileSync(`<path/to/folder/${file}`));
  });
  // Upload and create new record
  const createdRecord = await pb.collection(‘projects’).update(‘<obj_id>’,formData);
  console.log(createdRecord);
}
main();

I get 200 response but no file is uploaded. The formData object looks like what I would expect it to:
FormData {
  [Symbol(state)]: [
    {
      name: 'multiqc_report',
      value: '<!DOCTYPE html>\n' +
        '<html lang="en">\n'...1285808 more characters
     }
  ]
}

Where 'multiqc_report' is the field which the data should be stored.
Versions:
pocketbase: 0.10.4
pocketbase js sdk: 0.14.3
node: v18.13.0
Can someone please point out what I am doing wrong?

The FormData file field value must be a Blob or File instance, aka. you could try:
formData.append("file", new Blob([fs.readFileSync(`<path/to/folder/${file}`)]), "something.png");

That worked! Thank you very much!


WIP: Replace median filter implementation for NaN handling consistency

Description
This PR proposes the replacement of the current implementation of the median filter method in phasorpy.phasor.phasor_filter, which is a convenience wrapper around scipy.ndimage.median_filter. As it was suggested in #87, this method has undefined behaviour for NaN values, giving inconsistent results.
There are two extra methods proposed at the moment:
-2d_median: based on a cython implementation of a 2D median filter. This method handles NaN values predictably, by ignoring them when creating the 2D-kernel for filtering. In the case a kernel has an even number of elements, the average value of the two middle elements is selected as the median, which is what numpy.nanmedian does.

multidim_median: is an alternative for n-dimensional filtering, which is the main advantage of Scipy's median filter. This function uses numpy.sliding_window_view and numpy.nanmedian to perform the median filtering with n-dimensional kernels.  This function also ignores NaN values when performing the median calculation for the kernel, with the same criteria as the 2d_median method.

Regarding execution times:

The fastest method is scipy_median, specially as the array size increases. Nevertheless, the 2d_medianmethod performs marginally slower than scipy_median (around 1.1 the time of scipy_median).
'multidim_median' is around 3-4 times slower than scipy_median for any dimensional arrays.

Checklist

[ ] The pull request title and description are concise.
[ ] Related issues are linked in the description.
[ ] New dependencies are explained.
[ ] The source code and documentation can be distributed under the MIT license.
[ ] The source code adheres to code standards.
[ ] New classes, functions, and features are thoroughly tested.
[ ] New, user-facing classes, functions, and features are documented.
[ ] New features are covered in tutorials.
[ ] No files other than source code, documentation, and project settings are added to the repository.

Hi @cgohlke! I am working on improving the median filter for consistency when handling NaN values. As expected, it seems it is a trade-off between consistency, performance and adaptability for multiple dimensions. Speaking with Leonel, a couple of points were raised regarding filtering phasor coordinates:

He suggests prioritizing consistency before the other aspects
Phasor coordinates are usually filtered in the 2D space, at least for microscopy images. This is related to the resolution along different axes when acquiring images.

Considering this, I testes multiple approaches to replace the median filtering from scipy, since we know is not consistent at all with NaN as I also testes in various conditions and is not precise and predictable at all. I implemented two different methods for filtering, one based on a cython function with only works with 2D arrays and performs very similar to Scipy's. I also implemented another method that uses numpy.nanmedian which is very consistent with NaN values also. The advantage of this method is that it can perform filtering in n-dimensions but the drawback is that is much slower, and filtering is already a very costly function.
I left the three proposed methods to discuss in this PR, but I believe the best approach would be the following:

Remove the scipy_median method, which would also remove Scipy as a dependency (I think).
Use only one method median that uses the cython implementation is filtering in 2D and uses the multidim_median implementation if filtering in higher dimensions.

Hi @bruno-pannunzio , I haven't tested/run this yet but left some comments.

Phasor coordinates are usually filtered in the 2D space, at least for microscopy images. This is related to the resolution along different axes when acquiring images.

Sure. As a library, PhasorPy should still be generic if reasonably possible.

Thanks @cgohlke for all the comments! I am trying to take them all into account since I think they all make a lot of sense and improve the code.

Sure. As a library, PhasorPy should still be generic if reasonably possible.

I totally agree with you! But discussing with Leonel we reached the conclusion that contemplating every possibility with this median filter can take a while and it was not one of the priorities of the project. We can discuss this next meeting, but at least giving the option to still filter n-dimensions leaves the door open to uses other than microscopy, although it can be optimized for performance.
I will try to add tests for this implementations to finish the PR.

Hi @cgohlke! I think this is ready for review considering what we discussed in this last week. Let me know if you think something can be improved.

Thanks @cgohlke for all your comments! They all helped improve the description of variables and functions and also helped to improve a little bit the performance of the 2D filter. With all the changes now the Cython implementation of the 2D median filter is approx. 10-15% faster than Scipy's.

This could be parallelized with OpenMP. Maybe later?

This is a good idea. I tried something to see if it was easy, but it seems it will take some time because we have to make sure in each parallelization that the kernel memory is allocated, at least with what I tried. So maybe we can open another issue to propose different optimizations for median filtering in 2D, and also propose some future Cython implementation for n-dimension median filtering.

Here's a parallelized version. Also added a note that "filtered_image must be a copy of image" (is that correct?), which makes it easier to understand the implementation.
def _apply_2d_median_filter(
    const float_t[:, ::1] image,
    float_t[:, ::1] filtered_image,
    const ssize_t kernel_size,
    const int num_threads=1,
):
    """Apply a 2D median filter ignoring NaN.

    filtered_image must be a copy of image.

    """
    cdef:
        ssize_t rows = image.shape[0]
        ssize_t cols = image.shape[1]
        ssize_t k = kernel_size // 2
        ssize_t i, j, di, dj, ki, kj, valid_count
        float_t element
        float_t *kernel

    if kernel_size <= 0:
        raise ValueError("kernel_size must be greater than 0")

    with nogil, parallel(num_threads=num_threads):

        kernel = <float_t *> malloc(
            kernel_size * kernel_size * sizeof(float_t)
        )
        if kernel == NULL:
            with gil:
                raise MemoryError("Unable to allocate memory for kernel")

        for i in prange(rows):
            for j in range(cols):
                if isnan(image[i, j]):
                    continue
                valid_count = 0
                for di in range(kernel_size):
                    ki = i - k + di
                    if ki < 0:
                        ki = 0
                    elif ki >= rows:
                        ki = rows - 1
                    for dj in range(kernel_size):
                        kj = j - k + dj
                        if kj < 0:
                            kj = 0
                        elif kj >= cols:
                            kj = cols - 1
                        element = image[ki, kj]
                        if not isnan(element):
                            kernel[valid_count] = element
                            valid_count = valid_count + 1
                if valid_count > 0:
                    filtered_image[i, j] = _median(kernel, valid_count)

        free(kernel)

Here's a parallelized version.

Thanks @cgohlke! I believe this works. It's very similar to what I first thought of implementing but was getting an error when trying to raise an error without GIL, but didn't think of acquiring GIL again just to raise the error. I made the corresponding modifications to phasor_filter in order to be able to pass num_threads as parameter.

Also added a note that "filtered_image must be a copy of image" (is that correct?)

At first I thought so, but I think that actually the filtered image doesn't necessary need to be a copy of image for this to work, it just has to be of same shape. The only aspect I think needed to change for this to work without needing to be a copy was the case when the image[i, j]  was NaN. Previously that needed to be the same in filtered_image because that value was omitted from replacement in filtered_image, but now I think this is covered. I replaced the copy of real and imag in phasor_filter and this works, so I think the copy is not needed.

Previously that needed to be the same in filtered_image because that value was omitted from replacement in filtered_image, but now I think this is covered

There is still the case where valid_count=0 and filtered_image is not initialized, no?

filtered image doesn't necessary need to be a copy of image

In that case, you don't need to make a copy of the image in the Python code, but use numpy.empty_like, no?

@cgohlke I made some modifications that I think improved a little bit the _median_filter_nd method.
I realized there isn't any test for checking that both median methods return the same output as the median_scipy method if there are any NaN values. I will add this test and then I think it is ready for review again.

Thanks @cgohlke for all your comments and improvements on this PR! They all helped make the median filtering more efficient, consistent and clear in documentation. I implemented the tests that were missing and removed the extra lines of code.
I will merge this but it is still open for further improvements, specially the n-dimensional filtering.


got this on error console

Hi can i know how to troubleshoot this? or this is intended. Thank you.
Line 10651: [    103640] [b2699_GTAProce]        CrBrowserMain/ Request Failed due to timeout: https://illenium-appearance/appearance_get_locales (@illenium-appearance/web/dist/assets/index.b8e72b46.js:70)
Line 10652: [    103640] [b2699_GTAProce]        CrBrowserMain/ Request Failed due to timeout: https://illenium-appearance/get_theme_configuration (@illenium-appearance/web/dist/assets/index.b8e72b46.js:70)

also

add your locale in fxmanifest

Im using the default locale that is en
fx_version "cerulean"
game "gta5"
author "snakewiz & iLLeniumStudios"
description "A flexible player customization script for FiveM servers."
repository "https://github.com/iLLeniumStudios/illenium-appearance"
version "v5.6.1"
lua54 "yes"
client_scripts {
"game/constants.lua",
"game/util.lua",
"game/customization.lua",
"game/nui.lua",
"client/outfits.lua",
"client/common.lua",
"client/zones.lua",
"client/framework/framework.lua",
"client/framework/qb/compatibility.lua",
"client/framework/qb/main.lua",
"client/framework/qb/migrate.lua",
"client/framework/esx/compatibility.lua",
"client/framework/esx/main.lua",
"client/target/target.lua",
"client/target/qb.lua",
"client/target/ox.lua",
"client/management/management.lua",
"client/management/common.lua",
"client/management/qb.lua",
"client/management/qbx.lua",
"client/management/esx.lua",
"client/radial/radial.lua",
"client/radial/qb.lua",
"client/radial/ox.lua",
"client/stats.lua",
"client/defaults.lua",
"client/blips.lua",
"client/props.lua",
"client/client.lua",
}
server_scripts {
<EMAIL_ADDRESS>"server/database/database.lua",
"server/database/jobgrades.lua",
"server/database/managementoutfits.lua",
"server/database/playeroutfitcodes.lua",
"server/database/playeroutfits.lua",
"server/database/players.lua",
"server/database/playerskins.lua",
"server/database/users.lua",
"server/framework/qb/main.lua",
"server/framework/qb/migrate.lua",
"server/framework/esx/main.lua",
"server/framework/esx/migrate.lua",
"server/framework/esx/callbacks.lua",
"server/framework/esx/management.lua",
"server/util.lua",
"server/server.lua",
"server/permissions.lua"
}
shared_scripts {
"shared/config.lua",
"shared/blacklist.lua",
"shared/peds.lua",
"shared/tattoos.lua",
"shared/theme.lua",
"shared/framework/framework.lua",
"shared/framework/esx/util.lua",
"locales/locales.lua",
"locales/ar.lua",
"locales/bg.lua",
"locales/cs.lua",
"locales/de.lua",
"locales/en.lua",
"locales/es-ES.lua",
"locales/fr.lua",
"locales/hu.lua",
"locales/it.lua",
"locales/nl.lua",
"locales/pt-BR.lua",
"locales/ro-RO.lua",
"locales/id.lua",
<EMAIL_ADDRESS>}
files {
"web/dist/index.html",
"web/dist/assets/*.js"
}
ui_page "web/dist/index.html"


trying to require 'ws' only on server side code

Hello
I'm failing in require only WebSocket for server side code:
plugin/apolloClient.js
  if (process.SERVER_BUILD) {
    const ws = require('ws')
    wsClient = new SubscriptionClient(wsUri, {
      reconnect: true
    }, ws)
  } else {
    wsClient = new SubscriptionClient(wsUri, {
      reconnect: true
    })
  }

nuxt.config.js
extend (config, {dev, isServer}) {
      if (!isServer) {
        config.node = {
          fs: 'empty',
          tls: 'empty'
        }
      }
    },

I receive following error warning / client and server side:
 WARNING  Compiled with 2 warnings                                                                                10:06:46

 warning  in ./~/bindings/bindings.js

76:22-40 Critical dependency: the request of a dependency is an expression

 warning  in ./~/bindings/bindings.js

76:43-53 Critical dependency: the request of a dependency is an expression

hello,
I am using node 6.11, Nuxt V1 RC11 and Ubuntu 16.04
I am also having the same warning messages with the "ws" module (coming from "simplewebrtc" in my case).
It seems to be due to webpack : https://github.com/webpack/webpack/issues/196
I do not know webpack at all...
To reproduce this error, just add the package "ws" to the main package.json
Matthieu

This seems to be resolved now. Closing. Reopen if still happens.


Engine Cosmos : Date Type is not supported as part of schema

With the current version, engine does not let me to define schema with date type.
type Order @model {
id : ID,
name : String,
customerId : ID,
date : Date,
customerEmail : String,
items : [Inventory]
}
the above schema throws an error

On startup, there is no more error, DateTime is a valid scalar type in graphql


CPU showing significantly less load/usage than windows task manager

I've dug around a little bit but can't seem to figure out how the numbers differ so greatly.
The widget never shows more than maybe 1% usage while windows show's 38-40%

Same problem here

me too

still having this issue


Required error

Hello,
I found an issue with the "required" option. My example is (created from your readme):
import nap

let bar = use_arg(name="bar", kind="value", required=false, help="Heaps Alloy", alt="b")

parse_args()

if bar.used:
    echo bar.value

and the result is:
./tests/test -b
'--bar' needs a value.

In your description, the "required" option allows to call this option without passing value.
I fix this issue after adding "value" parameter to given default value.
let bar = use_arg(name="bar", kind="value", required=false, value="cosmic", help="Heaps Alloy", alt="b")
A compilation error should appear in this case to prevent this problem.
Thanks.

Hi. 'required' means it's not required for the user to provide '--bar' or '-b' at all. If the user does provide it, it will check if a value was provided since it's of kind 'value', and throw an error if not. I could disable the value checking on required=false but it would be weird if the user did try to use the 'bar' argument but used it wrong.

If the user does provide it, it will check if a value was provided since it's of kind 'value', and throw an error if not.
Ok but can you explain this in the readme for others users plz.
Thanks.


browser.get is not working when only the url hash changes.

Bug report

Node Version: 7.5.0
Protractor Version: 5.3.0
Angular Version: Non-angular project
Browser(s): Google Chrome 64.0.3282.186 (Official Build) (64-bit)
Operating System and Version MacOS, Docker (ubuntu)

I recently updated to the most recent protractor: 5.3.0
I have an issue where calling browser.get when only the url hash changed is hanging.
Basically, I have the same path, but I updated the hash:
await browser.get('/foo#bar');
await browser.get('/foo#baz');
The test continues to wait until a timeout occurs:
(Session info: chrome=64.0.3282.186)
(Driver info: chromedriver=2.36.540471 (9c759b81a907e70363c6312294d30b6ccccc2752),platform=Linux 4.4.0-1049-aws x86_64)
Here is my stacktrace:
    Failed: timeout
(Session info: chrome=64.0.3282.186)
(Driver info: chromedriver=2.36.540471 (9c759b81a907e70363c6312294d30b6ccccc2752),platform=Linux 4.4.0-1049-aws x86_64)
Stack:
TimeoutError: timeout
(Session info: chrome=64.0.3282.186)
(Driver info: chromedriver=2.36.540471 (9c759b81a907e70363c6312294d30b6ccccc2752),platform=Linux 4.4.0-1049-aws x86_64)
at Object.checkLegacyResponse (/project/node_modules/selenium-webdriver/lib/error.js:546:15)
at parseHttpResponse (/project/node_modules/selenium-webdriver/lib/http.js:509:13)
at doSend.then.response (/project/node_modules/selenium-webdriver/lib/http.js:441:30)
at process._tickDomainCallback (internal/process/next_tick.js:129:7)
From: Task: WebDriver.navigate().to('https://REDACTED')
at Driver.schedule (/project/node_modules/selenium-webdriver/lib/webdriver.js:807:17)
at Navigation.to (/project/node_modules/selenium-webdriver/lib/webdriver.js:1133:25)
at Driver.get (/project/node_modules/selenium-webdriver/lib/webdriver.js:988:28)
at ProtractorBrowser.get (/project/node_modules/protractor/built/browser.js:655:32)
at UsersPage._callee2$ (/project/iam/pages/UsersPage.js:36:19)
at tryCatch (/project/node_modules/regenerator-runtime/runtime.js:62:40)
at Generator.invoke [as _invoke] (/project/node_modules/regenerator-runtime/runtime.js:296:22)
at Generator.prototype.(anonymous function) [as next] (/project/node_modules/regenerator-runtime/runtime.js:114:21)
at Generator.tryCatcher (/project/node_modules/bluebird/js/release/util.js:16:23)
at PromiseSpawn._promiseFulfilled (/project/node_modules/bluebird/js/release/generators.js:97:49)
at UsersPage. (/project/node_modules/bluebird/js/release/generators.js:201:15)
at UsersPage.get (/project/iam/pages/UsersPage.js:156:22)
at _callee2$ (/project/iam/01_usersPageSpec.js:16:16)
at tryCatch (/project/node_modules/regenerator-runtime/runtime.js:62:40)
at Generator.invoke [as invoke] (/project/node_modules/regenerator-runtime/runtime.js:296:22)
at Generator.prototype.(anonymous function) [as next] (/project/node_modules/regenerator-runtime/runtime.js:114:21)
at Generator.tryCatcher (/project/node_modules/bluebird/js/release/util.js:16:23)
at PromiseSpawn.promiseFulfilled (/project/node_modules/bluebird/js/release/generators.js:97:49)
at UserContext. (/project/node_modules/bluebird/js/release/generators.js:201:15)
at /project/node_modules/jasminewd2/index.js:112:25
at new ManagedPromise (/project/node_modules/selenium-webdriver/lib/promise.js:1077:7)
at ControlFlow.promise (/project/node_modules/selenium-webdriver/lib/promise.js:2505:12)
at schedulerExecute (/project/node_modules/jasminewd2/index.js:95:18)
at TaskQueue.execute (/project/node_modules/selenium-webdriver/lib/promise.js:3084:14)
at TaskQueue.executeNext (/project/node_modules/selenium-webdriver/lib/promise.js:3067:27)
at asyncRun (/project/node_modules/selenium-webdriver/lib/promise.js:2927:27)
at /project/node_modules/selenium-webdriver/lib/promise.js:668:7
at process._tickDomainCallback (internal/process/next_tick.js:129:7)
From: Task: Run beforeEach in control flow
at UserContext. (/project/node_modules/jasminewd2/index.js:94:19)
at runCallback (timers.js:651:20)
at tryOnImmediate (timers.js:624:5)
at processImmediate [as _immediateCallback] (timers.js:596:5)
From asynchronous test:
Error
at Suite. (/project/iam/01_usersPageSpec.js:14:3)
at Object. (/project/iam/01_usersPageSpec.js:6:1)
at Module._compile (module.js:571:32)
at loader (/project/node_modules/babel-register/lib/node.js:144:5)
at Object.require.extensions.(anonymous function) [as .js] (/project/node_modules/babel-register/lib/node.js:154:7)
at Module.load (/project/node_modules/coffee-script/lib/coffee-script/register.js:45:36)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19)
at /project/node_modules/jasmine/lib/jasmine.js:93:5
at Array.forEach (native)
at Jasmine.loadSpecs (/project/node_modules/jasmine/lib/jasmine.js:92:18)
at Jasmine.execute (/project/node_modules/jasmine/lib/jasmine.js:197:8)
at /project/node_modules/protractor/built/frameworks/jasmine.js:132:15
at Function.promise (/project/node_modules/q/q.js:682:9)
at /project/node_modules/protractor/built/frameworks/jasmine.js:104:14
at _fulfilled (/project/node_modules/q/q.js:834:54)
at self.promiseDispatch.done (/project/node_modules/q/q.js:863:30)
at Promise.promise.promiseDispatch (/project/node_modules/q/q.js:796:13)
at /project/node_modules/q/q.js:604:44
at runSingle (/project/node_modules/q/q.js:137:13)
at flush (/project/node_modules/q/q.js:125:13)
at _combinedTickCallback (internal/process/next_tick.js:67:7)
at process._tickCallback (internal/process/next_tick.js:98:9)

Also, I use --direct to run the tests.

Hi all , any update on this issue . I am also having this issue with following setup :
Protractor version: 5.3.2
chromedriverVersion: '2.37.544315
Chrome : 'version' => '67.0.3396.87',
platform=Linux 3.10.0-862.3.2.el7.x86_64 x86_64
Docker : node:9-stretch

I am working around this issue by calling .get() a second time with the same values eg:
await browser.get('/foo#bar');
await browser.get('/foo#baz');
await browser.get('/foo#baz');


Can not run kaolin-dash3d

I'm getting error when I try to run dash3d, I've installed kaolin via pip
kaolin.__version__
'0.13.0'

System Ubuntu 2020, GPU RTX 2070
kaolin-dash3d --logdir=/home/log_folder/  --port=8080
Traceback (most recent call last):
  File "/home/bobi/miniconda3/envs/kaolin/bin/kaolin-dash3d", line 3, in <module>
    from kaolin.experimental.dash3d.run import run_main
ModuleNotFoundError: No module named 'kaolin.experimental.dash3d.run'

Hi @bobiblazeski , 0.13.0 wheels had some issues with dash3d, it should work fine on our latest 0.14.0. Please let us know if you have any other problem!

It works thank you very much.


Docker build broken

The docker build used for PR tests is likely broken. See this docker build which failed when testing a tiny unrelated change in the Colab notebook. I bet the current main would not build either.

Looks like the flash-attention cannot be built in the ancient environment of main anymore. I tried:

local build of the environment - fails the same way as the docker build
local build of the environment in the pl_upgrades branch - works OK
local build of the environment with flash-attention commit changed from 5b838a8 to 9a11f44 (their current main) - fails the same way as the docker build

Maybe it's time to seriously consider the merge of pl_upgrades (which may be painful as main diverged quite far)?
The idea of fixed versions is nice but only gets us so far - we are always struggling in Colab which is like building on a moving sand and docker build is not building on a rock frozen base either.

Fixed by #437. Thank you @jnwei.
This can be closed I think.

Great, thanks for reporting the issue @vaclavhanzl , glad the fix takes care of it.
We are still aiming to upgrade to pytorch2 and merge in the changes from pl_upgrades, but we ran into some technical issues. I will share more on this later in the week.

Thanks @jnwei !


Add section on using Rust in Deno

We should add wasmbuild to the manual https://deno.com/blog/wasmbuild
I think this would go under the "Using WebAssembly" section.

Is this now covered by https://docs.deno.com/runtime/reference/wasm/ ? @dsherret

Nope, not yet. This would be to add https://github.com/denoland/wasmbuild to the docs


CopyToOutputDirectory

Steps to reproduce
Either include a project sample, attach a zipped project, or provide IDE / CLI steps to create the project and repro the behaviour. Example of a project sample:
Project file
<Project>
  <PropertyGroup>
    <Extension>cs</Extension>
  </PropertyGroup>

  <ItemGroup>
    <I Include="**/*.$(Extension)"/>
  </ItemGroup>

  <Target Name="Build">
    <Message Text="I: %(I.Identity)"/>
  </Target>
</Project>

Directory contents:
/
- a.cs
- b.cs
- dir/
     - c.cs

Command line
msbuild /bl

Expected  behavior
Actual behavior
Environment data
msbuild /version output:
OS info:
If applicable, version of the tool that invokes MSBuild (Visual Studio, dotnet CLI, etc):

Closing this issue as there is no statement as to what the issue actually is.


Add transformer simulation features

What this PR does / why we need it:

Enable users to test their standard transformer configuration without actual deployment.
The goal of this features is to shorten the development process.

Add simulate API
Add executor package as an "interface" to the transformer
Revamp deployment UI to cater simulation components
Revamp transformer tools

Does this PR introduce a user-facing change?:

* Add Simulation components on deployment page
* Modify transformer tools page

Deployment Page:

Transformation Tools (merlin/-/tools/transformer) page:

Checklist

[ ] Added unit test, integration, and/or e2e tests
[ ] Tested locally
[ ] Updated documentation
[ ] Update Swagger spec if the PR introduce API changes
[ ] Regenerated Golang and Python client if the PR introduce API changes

The API part is looking good. I will test the UI.

Generally LGTM except you tend to use very long lines which become hard to read/maintain.

For lines that are too long, you can enable word wrap in your IDE. For example https://code.visualstudio.com/docs/editor/codebasics#_how-do-i-turn-on-word-wrap
I have also formatted the code using gofumpt and husky and since they didn't modify it to the new line, I will keep it as is.


BUG: don't mask out values in AdjustedArray

This looks good to me assuming we don't need to do more work on adjustment perf.

For 1 year of the Q1500US, which has good coverage of different fields and adjustments, we get:
master: 32.664
branch: 35.976

so we get a slight slowdown in exchange for ensured correctness. I think that is okay.

Coverage decreased (-0.4%) to 86.759% when pulling<PHONE_NUMBER>b5aec85b3951f7099c8e23bdc21595 on mask-out-the-mask into 77922ddedced4458fba7d2cb4c2a0826260f82fc on master.


Odds in the Alerts/toasts need to match the odds the user has set

Odds in the toast are showing in 0-100% and odds on the Market pg are showing in decimal (which is my current setting)

looks fixed


Feature/1661_number_prime

IMPORTANT: Please do not create a Pull Request without an issue.
All Submissions:

[x] I have read the CONTRIBUTING document.
[x] My code is written in Python3 and is ending with .py
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have checked that my submission does pass the test on LeetCode.com
[x] Does your filename follow the naming Conventions?
[x] Have you linked your PR to an Issue?

Closing issues
Put closes #0713 in your comment to auto-close the issue that your PR fixes (if such).

closes #0713

closes #713

You want to put this in your first comment to really link the issue to this PR.

closes #713

You want to put this in your first comment to really link the issue to this PR.

How do this?

click on the three dots next to your first comment
choose "edit"
update the last line accordingly


Extension Issue Qlik_Sense 3.2 SR4

Hi
I have upgraded to 3.2 Qlik sense SR4, but my variable didn't work.
So i downloaded latest variable.zip. but it didn't work
Please give me guidance what to do
Thank you

This is the same issue as #21


Update README.md

Clear up some confusion in the docs. It's not optional to have more than 1 item defined because it simply wont work. You can't have more than 1 method of calling the command.
setting a config
array(
'prefix' => '!',
'nick' => false
)
would not work as expected since the config is checking for isset on the nick and not checking the other rules.

Thanks!


DOC: Consistent naming for 'Insight Toolkit (ITK)'

As discussed with 199c38ed11da9fad4608d064767beaf0fd70efc8
925eb8fe771ae5edde2d2f4341c317dc11a9b96a
use the simpler and more widely known name 'Insight Toolkit (ITK)'.

The consistency is good. But some places may need a brief description of with ITK something like:
The "Insight Toolkit" for segmentation and registration.
The "Insight Toolkit" for advanced bio-medical image analysis.
Both are lacking the full breadth of ITK, but any condensed summary will be.

Good ideas -- I'll merge and follow-up with a more detailed description in another patch.

Extended description added in https://github.com/InsightSoftwareConsortium/ITK/pull/1271


DNS is not returning correct IP

Thanks @mosajjal for this nice work.
I am running sniproxy on a VPS and it looks to be working. There are some issues with installation though. tlskey and tlscert tags changed in documentations as well as sniproxy service while the interactive installer sets wrong tags (certpah and keypath which looks from older versions) inside the configurations.
Anyway, I have changed them using sudo systemctl edit --full sniproxy
After fixing some issues finally the sniproxy starts to work, at least it looks like this.
If I use dig on the sniproxy server it returns correct IP's but, if I use the sniproxy's server IP as a DNS on my router or other dns clients all the queries returning the sniproxy's server IP. Is it as expected or I have missed something in here?
If it means to be like this, how it would work then?

Hi.
take a look at #33, might be a related issue in Iran. the midboxes sometimes rewrite the DNS response and replace it with the "real" IP which is problematic.
I've suggested some methods of 1) confirming and 2) troubleshooting the issue. if it's the same, let's follow it up in #33 or open a discussion for it, if it's different let me know so we can continue here.

Hi.
take a look at #33, might be a related issue in Iran. the midboxes sometimes rewrite the DNS response and replace it with the "real" IP which is problematic.
I've suggested some methods of 1) confirming and 2) troubleshooting the issue. if it's the same, let's follow it up in #33 or open a discussion for it, if it's different let me know so we can continue here.

It looks quiet similar to the #33 and I will continue there for better follow up and troubleshootings.
Thanks anyway


bug: bootstrap does not work in docker-from-docker setting

Are you use the envd server?

[ ] Yes, I am using the envd server.
[X] No, I am not using the envd server.

Describe the bug
When trying to call envd bootstrap on my VScode devcontainer, I get this error message (last line is the error):
$ envd bootstrap
INFO[2023-10-30T13:04:38Z] [1/5] Bootstrap SSH Key
INFO[2023-10-30T13:04:38Z] [2/5] Bootstrap registry CA keypair
INFO[2023-10-30T13:04:38Z] [3/5] Bootstrap registry json config
INFO[2023-10-30T13:04:38Z] [4/5] Bootstrap autocomplete
INFO[2023-10-30T13:04:38Z] Install bash autocompletion
WARN[2023-10-30T13:04:38Z] Warning: failed writing to /usr/share/bash-completion/completions/envd: open /usr/share/bash-completion/completions/envd: permission denied
INFO[2023-10-30T13:04:38Z] You may have to restart your shell for autocomplete to get initialized (e.g. run "exec $SHELL")
INFO[2023-10-30T13:04:38Z] [5/5] Bootstrap buildkit
error: invalid mount config for type "bind": bind source path does not exist: /home/vscode/.config/envd

I even tried to mount my local envd dir to the mentioned path and still get the same error.
To Reproduce
Devcontainer is in an docker-from-docker setting as described here:
https://github.com/microsoft/vscode-dev-containers/tree/main/containers/docker-from-docker
Expected behavior
No error
The docker info output
Client: Docker Engine - Community
Version:    24.0.6
Context:    default
Debug Mode: false
Server:
Containers: 27
Running: 1
Paused: 0
Stopped: 26
Images: 90
Server Version: 24.0.6
Storage Driver: overlay2
Backing Filesystem: extfs
Supports d_type: true
Using metacopy: false
Native Overlay Diff: true
userxattr: false
Logging Driver: json-file
Cgroup Driver: cgroupfs
Cgroup Version: 1
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: io.containerd.runc.v2 nvidia runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 61f9fd88f79f081d64d6fa3bb1a0dc71ec870523
runc version: v1.1.9-0-gccaecfc
init version: de40ad0
Security Options:
apparmor
seccomp
Profile: builtin
Kernel Version:
Operating System:
OSType: linux
Architecture: x86_64
CPUs:
Total Memory:
Name:
ID:
Docker Root Dir: /var/lib/docker
Debug Mode: false
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false
The envd version output
envd: v0.3.36
BuildDate: 2023-07-18T15:36:15Z
GitCommit: 854798c5b368505ea81eb040a5584f39eaee1d68
GitTreeState: clean
GitTag: v0.3.36
GoVersion: go1.19.10
Compiler: gc
Platform: linux/amd64
Additional context
No response

Is it related to the https://github.com/microsoft/vscode-dev-containers/blob/main/containers/docker-from-docker/README.md#using-bind-mounts-when-working-with-docker-inside-the-container?
$HOME/.config/envd is mounted to share some configurations for buildkitd. You can avoid this by using a remote buildkitd or moby (no need to bootstrap a buildkit container):

remote buildkitd

envd context create --name remote --builder tcp --builder-address <your-remote-buildkitd-addr> --use

moby (need to use the envd v1 syntax)

envd context create --name moby-test --builder moby-worker --use

Thanks that did the trick (option 2 with moby)!
Its OT, but now I face the issue that during the build its not using the proxy (it does when I use docker build). I guess I need to configure the proxy also for moby. In case you know this by hard, it would be awesome to get a little hint here, too.
Just out of curiosity, why is there no $HOME/.config/envd in my docker env after installing envd? And why does it say path does not exist, even when this exact path is mounted (from the host) and exists inside the container run?

envd up and envd down have the --proxy option that can use the HTTPS_PROXY/HTTP_PROXY environment variables.
That's because the $HOME/.config/envd is created in the "docker from docker" but it mounts the host. You can get the details from the previous link.

Thanks for help!

That helped, but again I get an proxy error:
#4 [internal] mkdir certs
#4 DONE 0.0s
#2 docker-image://docker.io/mambaorg/micromamba:1.0.0
#2 resolve docker.io/mambaorg/micromamba:1.0.0 2.4s done
#2 CACHED
#5 [internal] copy cert from mamba
#5 DONE 0.1s
#6 [internal] copy micromamba binary
#6 DONE 0.1s
#7 [internal] create envd python=3.11
#7 0.527 bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
#7 0.628
#7 0.628                                            __
#7 0.628           __  ______ ___  ____ _____ ___  / /_  ____ _
#7 0.628          / / / / __ `__ \/ __ `/ __ `__ \/ __ \/ __ `/
#7 0.628         / /_/ / / / / / / /_/ / / / / / / /_/ / /_/ /
#7 0.628        / .___/_/ /_/ /_/\__,_/_/ /_/ /_/_.___/\__,_/
#7 0.628       /_/
#7 0.628
#7 20.65 critical libmamba Multiple errors occured:
#7 20.65     Download error (28) Timeout was reached [https://repo.anaconda.com/pkgs/main/noarch/repodata.json]
#7 20.65     Could not resolve host: repo.anaconda.com
#7 20.65     Subdir pkgs/main/noarch not loaded!
#7 20.65     Subdir pkgs/r/noarch not loaded!
#7 20.65
#7 ERROR: process "bash -c /usr/local/bin/micromamba create -p /opt/conda/envs/envd -c defaults python=3.11" did not complete successfully: exit code: 1
------
> [internal] create envd python=3.11:
#7 0.628         / /_/ / / / / / / /_/ / / / / / / /_/ / /_/ /
#7 0.628        / .___/_/ /_/ /_/\__,_/_/ /_/ /_/_.___/\__,_/
#7 0.628       /_/
#7 0.628
#7 20.65 critical libmamba Multiple errors occured:
#7 20.65     Download error (28) Timeout was reached [https://repo.anaconda.com/pkgs/main/noarch/repodata.json]
#7 20.65     Could not resolve host: repo.anaconda.com
#7 20.65     Subdir pkgs/main/noarch not loaded!
#7 20.65     Subdir pkgs/r/noarch not loaded!

If I would need to guess the proxy settings and env vars are not available during build. If you have another hint for me, that would be awsome.

There are several proxy settings. The one I mentioned above is for the docker image proxy.
If you need other proxies like apt/pip/conda, you can check the envd doc.


Reads Info tag is poor representation of arrays and ints

The info tag is string only at the moment, unfortunately this makes for a rather poor representation of arrays and integers. Given that this tag appears to be intended to represent BAM/SAM/CRAM info tags it might be wise to make this a union or similar?
https://github.com/ga4gh/schemas/blob/d2b3380992d920150c6d40f884c48cda0d50f321/src/main/resources/avro/reads.avdl#L280

Not really a string.
In my reading map<array<string>> info = {}; represents a map where the values are arrays with string array elements.
I'm not sure about empty or not as option; IMHO this should read map<array<union{ null, string }>> info to allow empty arrays; but this may be covered through the  = {} declaration of a default empty map? Who could elaborate, please? This declaration is all around, and I would like to be sure of its validity for empty constructs (since info should be optional).

The current definition is intended to be a map of string -> []string (I guess the string key type is implied in Avro?), e.g. the following JSON:
info: {
  "foo": ["1", "2"],
  "bar": []
}

The string-only approach is definitely a bit inflexible. Possibly this could be improved along with or after the proposed move to proto.

My problem with jamming it into a string is that the SAM equivalent carries type info with it. Without that info you can go SAM -> GA4GH but you can't then go back the other way without knowing what type the tag is supposed to be. For unregistered or lowercase tags this may not be possible.
If we make this a union of int, float, string and array I can look at the JSON and see "" I can imply it's a string, 6 suggests an integer, 6.0 a float and [] an array. Not sure how that would translate into Proto though.

@calbach Thanks for the empty array confirmation. And yes, map keys are strings per Avro spec.

So would anyone accept the following:
  map< union{array< union{string, int, float}>, string, int, float, null }> info = {};

This doesn't really cover all the cases in http://samtools.github.io/hts-specs/SAMv1.pdf section 1.5 but it covers a lot more than we currently do.

This suggestions is in a good direction, I think.
Some kind of rich representation of attributes is important.  We
should avoid requiring strings to be parsed within an attribute
value.  This is a source of ambiguity and fragility in two dimensional
file formats.  Having values as arrays is very useful avoid parsing strings.
Sequence annotations https://github.com/ga4gh/schemas/blob/rna/src/main/resources/avro/sequenceAnnotations.avdl
has:
record Attributes {
map<array<union {string, ExternalIdentifier, OntologyTerm}>> vals = {};
}
I am unsure how useful it is to have scalar values as well as
arrays, since a scalar can be a length one array.
It would be good to converge on single attribute structure for
all of ga4gh.
Martin Pollard<EMAIL_ADDRESS>writes:

So would anyone accept the following:
map< union{array< union{string, int, float}>, string, int, float, null }> info = {};
This doesn't really cover all the cases in http://samtools.github.io/hts-specs/
SAMv1.pdf section 1.5 but it covers a lot more than we currently do.
—
Reply to this email directly or view it on GitHub.*


TGas validation errors should be the same and applicable to all TGas field

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Check TGas field validation when creating a proposal
Check TGas field validation when user vote
Check TGas field validation for Bounty

Expected behavior
TGas field validation should be the same - "Must be greater than or equal to 10"
Screenshots

Verified on dev. Fixed


Add Demographic Data

Hey! I think it'd be good to be able to generate demographic data like height, age, race, marital status etc. If anybody feels similarly, I have a potential PR that could support this feature. Feel free to provide critique on the idea!

I like this, as I'm building a census-like application that relies on this data. I'd like to take a look at that PR


Bug: add shader gameobject can only pass a img which width and heigth is power-of-2

Version

Phaser Version: 3.18.1

Operating system: MacOS

Browser:  Chrome

Description

add shader method can only pass a img which width and heigth is power-of-2
Example Test Code
var shader = this.add.shader('Tunnel', 400, 300, 800, 600, [ 'metal' ]);
For example, if "metal" img width is not power-of-2, there is an error as below, it's easy to present in the phaser3-example repository.

Additional Information

there seems to have a solution
https://stackoverflow.com/questions/47368120/render-warning-texture-bound-to-texture-unit-0-is-not-renderable-it-maybe-non?rq=1

This is because the Tunnel shader in that example requires a POT texture, rather than a bug in the Shader Game Object. Lots of shaders have this requirement. It's up to you to either provide a POT texture or code your shader in such a way that it can work with NPOT textures.

I wrote a very simple sharder that can work with NPOT textures. The shader is to turn any image to grayscale.  But still have the problem
or where can I config if I do not need a POT texture? I am debuging this for a few hours and still can not find the reason. Below is my screenshot


[Commands] remove ignoreDependencies from swift package tool

Remove --ignore-dependenices.
swift package fetch --ignore-dependenices doesn't fetch anything and does nothing.
I don't see any value from having this option and I propose to remove it

I don't know the original intent behind this option, but it does look as if one part of the code that this PR removes instantiates a Package instead of going through an SCM get operation.  From the name of the option, that sounds like the intended behavior.  Are you saying that there's no workflow in which that's useful, or that the option is fundamentally broken?
If it really does nothing, then I agree, we should remove it.

I think this functionality come from swift build --ignore-dependenices where you can build only a root package, without dependencies.
I mean that I don't see any use case when this feature can be useful. The fetch command suppose to fetch package dependencies, and --ignore-dependenices options says opposite, to don't fetch them.
The implementation of that feature is technically correct, it doesn't get any dependencies, but just instantiate a root package
from swift package --help
  fetch                                  Fetch package dependencies

@swift-ci Please test

I added this to be able to performance test parts of the tool from the command line, I'd prefer we keep it around for now.

Ok, I'm going to close this PR then, it's not critical for now :)


feat: Add ROC

Description

Add ROC

Checklist

[x] My code follows the existing style, code structure, and naming taxonomy
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have performed a self-review of my own code and included any verifying manual calculations
[x] I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage.  New and existing unit tests pass locally and in the build (below) with my changes
[x] My changes generate no new warnings and running code analysis does not produce any issues
[ ] I have added or run the performance tests that depict optimal execution times
[ ] I have made corresponding changes to the documentation

My plan is to make separate branch for docs of indicators which is not deployed yet. By doing that, we can keep updating docs without confusing. When we release new indicators, what we need to do is just merging them. How do you think?

My plan is to make separate branch for docs for indicators which are not deployed yet. By doing that, we can keep updating docs without confusing. When we release new indicators, what we need to do is just merging them. How do you think?

Makes sense.  That solved my bad timing issue -- just merge the docs once the next release is ready.  It might be helpful to get that branch started and open a Draft PR so they can be linked, reviewed, and tracked together, even if in separate branches.

Yep, let's go with it


Fixed behat3 execution by removing --ansi parameter

Allow execution for behat 3 and behat 2.5 version.

Not sure I even remember what the --ansi flag was for.
What was this doing before? I'm not a behat user.

In behat v2 it 's use to enable color output, in behat v3 the option is named --colors.
Here behat v2 manual:
--ansi               Whether or not to use ANSI color in the output.
                         Behat decides based on your platform and the output destination if not specified.

Sorry for the lack of information, i created an issue with detail.

@sgriffiths and @erran can you check this out?

I'm settling in Belfast. It could be a while before I have a chance.
Sent from my iPhone

On May 26, 2015, at 18:11, Kyle Kelley<EMAIL_ADDRESS>wrote:
@sgriffiths and @erran can you check this out?
—
Reply to this email directly or view it on GitHub.

If you get a chance in the next few days @erran, let me know. Otherwise I'll merge as is and see what issues get posted.

Let's see how this goes.

$ apm publish minor
Preparing and tagging a new version ✓
Pushing v2.25.0 tag ✓
Publishing<EMAIL_ADDRESS>✓


Automatic limiting of number of running tasks based on memory

Expected Behavior
Current Behavior
Does Gradle have any interest in offering support to limit the number of running tasks based on available memory?
Context
AndroidX discovered that lintAnalyze can take ~512M of memory and if we attempt to run ~60 of them in parallel then that might result in memory pressure and reduce the performance compared to running fewer of those tasks at a time.
Limiting the number of tasks based on available memory might also make it easier for users to notice if insufficient memory is slowing their builds down.
In the meanwhile we're looking into limiting the number of concurrent copies of lint via a shared BuildService which declares a limit on the number of concurrent usages

Thank you for your interest in Gradle! Sorry for the late reply.
This issue will be closed as a duplicate of

https://github.com/gradle/gradle/issues/20262

Please add your use case and 👍 to that issue.

The other has a slightly better description and covers more cases.


Website gives 405 error

This page works: https://www.machinerytrader.com
This page gives 405: https://www.machinerytrader.com/listings/for-sale/skid-steers/1055
Is there any fix for this?
The code is just this...
Document webPage = Jsoup.connect(getUrl()).userAgent(
"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.3 Safari/605.1.15").get();
I've tried varying the userAgents to no avail.
java.lang.IllegalStateException: Failed to execute ApplicationRunner
at org.springframework.boot.SpringApplication.callRunner(SpringApplication.java:763) ~[spring-boot-3.0.5.jar:3.0.5]
at org.springframework.boot.SpringApplication.callRunners(SpringApplication.java:750) ~[spring-boot-3.0.5.jar:3.0.5]
at org.springframework.boot.SpringApplication.run(SpringApplication.java:317) ~[spring-boot-3.0.5.jar:3.0.5]
at org.springframework.boot.SpringApplication.run(SpringApplication.java:1304) ~[spring-boot-3.0.5.jar:3.0.5]
at org.springframework.boot.SpringApplication.run(SpringApplication.java:1293) ~[spring-boot-3.0.5.jar:3.0.5]
at com.ironmarket.crawler.CrawlerApplication.main(CrawlerApplication.java:25) ~[classes/:na]
Caused by: org.jsoup.HttpStatusException: HTTP error fetching URL. Status=405, URL=[https://www.machinerytrader.com/listings/for-sale/skid-steers/1055]
at org.jsoup.helper.HttpConnection$Response.execute(HttpConnection.java:889) ~[jsoup-1.15.4.jar:1.15.4]
at org.jsoup.helper.HttpConnection$Response.execute(HttpConnection.java:828) ~[jsoup-1.15.4.jar:1.15.4]
at org.jsoup.helper.HttpConnection.execute(HttpConnection.java:365) ~[jsoup-1.15.4.jar:1.15.4]
at org.jsoup.helper.HttpConnection.get(HttpConnection.java:352) ~[jsoup-1.15.4.jar:1.15.4]
at com.xxxx.crawler.machinerytrader.MachineryTraderCrawler.crawl(MachineryTraderCrawler.java:36) ~[classes/:na]
at com.xxxx.crawler.service.CrawlerFactory.initCrawler(CrawlerFactory.java:30) ~[classes/:na]
at com.xxxx.crawler.CrawlerApplication.run(CrawlerApplication.java:41) ~[classes/:na]
at org.springframework.boot.SpringApplication.callRunner(SpringApplication.java:760) ~[spring-boot-3.0.5.jar:3.0.5]

I expect the server (or CloudFlare in-front) is running some anti-bot code fingerprinting, and there are some specific headers or other trust score mechanism that you will need to work out.
Unfortunately without visibility into the specifics of the check, it's hard to guess what the required incantation will be. I had a quick try by adding the various "sec-ch-ua" etc headers to no immediate avail.
I won't treat this as a jsoup bug because AFAICT there's nothing wrong with the request per se. If you find a way to successfully make the request, it would be helpful to add notes here on how.

I was able to coax a 200 out of it, but it's not particularly helpful. These headers:
        Connection session = Jsoup.newSession();

        session.header("Referer", "https://www.machinerytrader.com/listings/for-sale/skid-steers/1055");
        session.header("Accept-Language", "en-US,en;q=0.9");
        session.header("Accept", "text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7");
        Connection con = session.newRequest().url("https://www.machinerytrader.com/listings/for-sale/skid-steers/1055");

But CloudFlare is running a captcha:

Pardon Our Interruption

As you were browsing something about your browser made us think you were a bot. There are a few reasons this might happen:

You've disabled JavaScript in your web browser.

You're a power user moving through this website with super-human speed.

You've disabled cookies in your web browser.

A third-party browser plugin, such as Ghostery or NoScript, is preventing JavaScript from running. Additional information is available in this support article.

To regain access, please make sure that cookies and JavaScript are enabled before reloading the page.

After completing the CAPTCHA below, you will immediately regain access to the site again.

As the author of the website doesn't want user-agents such as jsoup accessing the site, I won't look for further ways to access it.

Thanks for the insight on this.   I'll look for ways to enable javascript and cookies with jsoup and anything else to get around the CloudFlare block.

Note that jsoup does support cookies, but it does not execute javascript.


Refactor save_extract_face.py to more readable please 😓

Anyone please can refactor and make save_extract_face.py more readable ?
https://github.com/mozarik/bangkit-capstone-CAP0166/blob/develop/ml-project/save_extract_face.py#L30-L57

Im solving this issue.


How to using jquery confirm to make a loading wait data from server

How to using jquery confirm to make a loading wait data from server

Please use content function, it returns ajax so you can do everything you want. Read docs https://craftpip.github.io/jquery-confirm/#ajaxloading


no-home option error

HI,
I ran the pipeline in my server (nextflow run genepi/mtdna-server-2 -r v2.1.9 -profile test,singularity) using your test dataset and I get this error: "ERROR: Unknown option: --no-home"
Can you help with this?
nextflow run genepi/mtdna-server-2 -r v2.1.9 -profile test,singularity
Nextflow 24.04.1 is available - Please consider updating your version to it
N E X T F L O W  ~  version 23.10.1
Launching https://github.com/genepi/mtdna-server-2 [irreverent_jepsen] DSL2 - revision: 1140d5c7fd [v2.1.9]
Welcome to mtDNA-Server 2 v2.1.9
(c) Sebastian Schönherr, Hansi Weissensteiner, Lukas Forer
WARN: Access to undefined parameter output_reports -- Initialise it to a default value eg. params.output_reports = some_value
executor >  local (2)
[be/5d16b1] process > MTDNA_SERVER_2:INDEX                     [100%] 1 of 1, failed: 1 ✘
[20/03d34c] process > MTDNA_SERVER_2:CALCULATE_STATISTICS (1)  [100%] 1 of 1, failed: 1 ✘
[-        ] process > MTDNA_SERVER_2:INPUT_VALIDATION          -
[-        ] process > MTDNA_SERVER_2:QUALITY_CONTROL           -
[-        ] process > MTDNA_SERVER_2:MUTSERVE                  -
[-        ] process > MTDNA_SERVER_2:MUTECT2                   -
[-        ] process > MTDNA_SERVER_2:FILTER_VARIANTS           -
[-        ] process > MTDNA_SERVER_2:MERGING_VARIANTS          -
[-        ] process > MTDNA_SERVER_2:VCF_MERGE                 -
[-        ] process > MTDNA_SERVER_2:HAPLOGROUPS_CONTAMINATION -
[-        ] process > MTDNA_SERVER_2:COVERAGE_ESTIMATION       -
[-        ] process > MTDNA_SERVER_2:ANNOTATE                  -
[-        ] process > MTDNA_SERVER_2:REPORT                    -
[-        ] process > MTDNA_SERVER_2:SAMPLE_REPORT             -
ERROR ~ Error executing process > 'MTDNA_SERVER_2:INDEX'
Caused by:
Process MTDNA_SERVER_2:INDEX terminated with an error exit status (1)
Command executed:
samtools faidx mt_contigs.fasta
samtools dict mt_contigs.fasta      -o mt_contigs.dict
Command exit status:
1
Command output:
Command error:
ERROR: Unknown option: --no-home

Thanks. I was able to execute it on singularity without errors. Can you copy/paste the error message from the work directory (work/be/5d16b1/.command.log or .command.err). I believe it could be a singularity problem (--no-home is an option we're not setting in the code). My server runs singularity version 3.8.3. You could also try to run it with Docker if possible.

interesting but again I think this is a singularity problem not a pipeline profile. You could also try with the latest version v2.1.13 but I think this is not related to the pipeline


Missing options to add

Hi
I like glidejs but I think some nice features are missing:
enable or remove prev and next with a simple option with true or false
the same for the dots, like happen with other carousels
set the items number without paddings for each classic default breakpoint like owl carousel orv slidesToShow for slick for responsive devices.
https://owlcarousel2.github.io/OwlCarousel2/demos/responsive.html
I've seen other requests about these points I tried to summarize them in a single post
I prefer glidejs but at this moment I cannot use it
thanks
bye

Hello. There is no direct API method for this. However, it can be done with little of javascript and .refresh() method.
There is an example in examples/destroy.html file.
https://github.com/jedrzejchalubek/glidejs/blob/master/examples/destroy.html#L83-L90


Fix link in "Evidence of colocalisation” column

In the "Gene prioritisation using locus-to-gene pipeline" table on the gene prioritisation page (e.g. LDL cholesterol + locus around 19_44908822_C_T), there is a column called "Evidence of colocalisation".
Within the column, if the value is "Yes" (based on the hasCololc response from the API), it links the user to the "GWAS Study Colocalisation" table.

However, upon review by @Jeremy37, this is a bug and instead, the "Yes" link to should direct users to the "Gene prioritisation using colocalisation analysis" table on the same page.

closing this one as work is done.


[WIP] The Mysterious Case of Disappearing Artist Bios

@gnilekaw and I have been troubleshooting an issue where certain Artist Bios are disappearing from the Artist detail page. It's tricky to reproduce this issue, here's what I've had success with:
zero

one

two

three

four

I set one artist to have that bio and another artist to have one fewer lines. The former will have their partner-artist-blurb be empty and the latter works as expected. Upgrading the library fixed this bug, but there's a regression. Even though we're providing ellipsis: '' in our config object, an ellipsis is showing up.
So, what we're wondering, @mzikherman, is if you have any context here that we're missing and the reason for the forking of the library in the first place.

          1 Warning

:warning:
Please assign someone to your PR

  Generated by :no_entry_sign: danger

See https://github.com/artsy/force/issues/673 for the original ticket.

Very tricksy :(
Too bad we even need to use a library/JS for such a 'minor-ish' feature.
From what I remember, I wanted to use the library and they hadn't yet published it to NPM at that time, so I just copied it and pushed it to npm so we could use it. These days I'd probably try to make that change upstream! (tho I see it's already on npm so all good there).

@mzikherman cool, so you're good with us hoping off your forked version?
Any thoughts on the fact that the ellipsis is showing up even though we have configured it to empty string?

@mzikherman cool, so you're good with us hoping off your forked version?

Definitely 👍

Any thoughts on the fact that the ellipsis is showing up even though we have configured it to empty string?

Not really, it's been a long while and don't really recall anything specific.
Do you mean the dotdotdot library stuff from https://github.com/artsy/force/blob/9257affb41a86ea341a5ea4c8fa8bf0ac1aaf942/components/blurb/view.coffee#L20-L22 ?

Ugh, things are worse than I realized. jquery.dotdotdot is on npm now, so that's good, that should mean we can hop off of @mzikherman's fork. But when you go look at it's page you see this:

Note: Because its performance can not be improved, this plugin is no longer actively maintained. Feel free to use it and submit pull requests though.

And it's GitHub link is to here:
https://github.com/BeSite/jQuery.dotdotdot
Which redirects to here:
https://github.com/FrDH/jQuery.dotdotdot
With the same message. On GitHub the project is up to version 1.8.3, but the last release to make it to npm is 1.7.4.
So we have an abandoned dependency on our hands. 😕
My suggestion: let's live with the regression here and get a fix into production for these disappearing artist bios. Said another way, I think it's worse to have missing author bios than it is to have a bonus ellipsis. Agreed?
Then, that would allow us to take a step back and think about what we want to do here. My preference would be to find another library but maybe it would also make sense to just write some code that does this for us.

Totally down for this, IMO the ellipsis isn't a big deal at all :)
This also seems an easy enough front-end widget that we could just support/rewrite it as needed.
Merging.


#6320 (fix for #6305) is flawed, because it may expose incorrect environment-variable values via Get-Item

A follow-up from https://github.com/PowerShell/PowerShell/issues/6305#issuecomment-375025005, as requested by @TravisEz13:
#6320 attempts to fix #6305, but only does so superficially:

while it fixes the immediate problem by preventing an error from surfacing when you use Get-item env: while case-variant environment variables are unexpectedly defined,

it can end up exposing an effectively hidden value that neither $env:<name> nor [environment]::GetEnvironmentVariable('<name>') see intermittently; in other words: Get-Item env:foo may intermittently report an incorrect value for $env:foo

Steps to reproduce
Note: Run this in PowerShell Core on Windows,  with Node.js installed and #6320 applied (without it, Get-Item will fail, complaining about a duplicate key).
$env:FOO = 'A'
$tries = 0
do {
  Write-Host '.' -NoNewline
  ++$tries
  $res = node -pe @'
    env = {}
    env.fOo = 'B' // redefine with a case variant name and different value
    env.FOO = process.env.FOO // also include the original case variant with its original value.
    // Contrast $env:foo with Get-Item env:foo
    require('child_process').execSync(\"pwsh -nop -command $env:foo; (Get-Item env:foo).Value\", { env: env }).toString()
'@
} while ($res[0] -eq $res[1])

"
Values differed after $tries tries: '$($res[0])' vs. '$($res[1])'"

Expected behavior
The loop should never exit, because $env:foo and  Get-Item env:foo should always report the same value.
Actual behavior
Because Get-Item env:foo intermittently reports B rather than A, the loop exits after a few tries with, e.g.:
Values differed after 3 tries: 'A' vs. 'B'

Fix
As proposed in https://github.com/PowerShell/PowerShell/issues/6305#issuecomment-371721480, in https://github.com/PowerShell/PowerShell/pull/6320/files#diff-1bee4b279bb56aca25fb479994641529, replace:
  // Windows only: duplicate key (variable name that differs only in case)
  // NOTE: Even though this shouldn't happen, it can, e.g. when npm
  //       creates duplicate environment variables that differ only in case -
  //       see https://github.com/PowerShell/PowerShell/issues/6305.
  //       However, because retrieval *by name* later is invariably
  //       case-Insensitive, in effect only a *single* variable exists.
  //       We simply ask Environment.GetEnvironmentVariable() which value is
  //       the effective one, and use that.
providerTable.TryAdd((string)entry.Key, entry);

(Note: the comments, taken from my original proposal, which still stands below, don't match the code - the comments explain what the code should do, but doesn't - see below.)
with:
try {
    providerTable.Add((string)entry.Key, entry);
} catch (System.ArgumentException) { // Windows only: duplicate key (variable name that differs only in case)
    // NOTE: Even though this shouldn't happen, it can, e.g. when npm
    //       creates duplicate environment variables that differ only in case -
    //       see https://github.com/PowerShell/PowerShell/issues/6305.
    //       However, because retrieval *by name* later is invariably
    //       case-INsensitive, in effect only a *single* variable exists.
    //       We simply ask Environment.GetEnvironmentVariable() for the effective value
    //       and use that.
    providerTable[(string)entry.Key] = new DictionaryEntry((string)entry.Key, Environment.GetEnvironmentVariable((string)entry.Key));
}

Environment data
PowerShell Core post-v6.0.2 with #6320  applied.

I see.  Thank for the detail repro and explanation.  Do you want to submit a PR?

@TravisEz13: Sure; please see #6489. Note that I've tweaked the fix slightly to recreate the entry when(ever) the effective value is encountered, so that the name casing reflects the duplicate that took effect; however, if duplicates define the very same value, it is unspecified which casing is reflected, but in practice that shouldn't matter.

Fixed by #6489.


Allow the use of OpenAI's rough token count

Context
Noticed that the tiktoken_ruby dependency isn't thread-safe. For us, this resulted in the Sidekiq process crashing consistently. There is also an open issue regarding this problem
Proposal
Allow the use of OpenAI's rough token count instead of using tiktoken ruby. The token count selection will be made when initializing the LLM:
    Langchain::LLM::OpenAI.new(
      api_key: openai_access_token,
      llm_options: {},
      default_options: { token_counter: :openai }
    )

tiktoken_ruby will be left as the default option.

@Sergio-e Do we still need this? I got rid of the automatic max_tokens validation via tiktoken. I think that feature was more annoying than helpful.

@Sergio-e Do we still need this? I got rid of the automatic max_tokens validation via tiktoken. I think that feature was more annoying than helpful.

It seems not. The issue is now fixed on the latest Tiktoken ruby release


Add a notice in README.md that IsTerminal is now in std

As of Rust 1.70, IsTerminal is now available in std, so users that are able to use Rust 1.70 should use that version instead of this crate.

Fixes #27.


We dont know how to use it

Can you make a bot of clash of clans for us please with more command

It's odd, I recall there being a website with some examples of all the methods used, though I cannot find it anymore.

Can you make a bot of clash of clans for us please with more command
I went around this by viewing the Client.js in the source file of clashofclans.js in node_modules. It's not the best way to find the methods on how to use, but it's typical markup so it's easy to find what you're looking for. After that it's just basic js fetches.

Thanks for the package btw. Works well, but I agree documentation could be better considering it's hard to find the methods you can use and their exact names.

The documentation site still exists. https://clashofclans.js.org/api/

Oh I see. I don't think that sidebar exists on mobile.

I've listed all available methods in readme.
https://github.com/clashperk/clashofclans.js#methods


Updating dependency to csbiginteger 3.0.0

Snowy, now project was moved to NeoResearch page for future and more stable maintainance. I'm using "bn.js" library to handle internals now, so it's very stable. The idea is to manage it to be compatible with C# representation, but at least now the code is much shorter and easier to maintain. I believe this major can last a long time now, since it's very tested on many cases.

Ok, no worry, I thought that was being used in the base already. Anyway, it's quite similar to bn, with few exceptions... for you it may not matter much, but since I want to build a neovm simulator, I need it exactly the same, thats the goal of the project. Tske your time.

Obsolete as neon-core no longer relies on this package (not for 2.x and not for 3.x)


Yifang image

merge for ImageDataCollector to not mix with ImageFile

@YifangZhang I see you have some broken code, need any help?


Removing aaf since aafd daemon is disabled

Tracked-On: OAM-108679

Merged internally, /absp/builders/celadon-mergerequest/builds/3307


[stable/elasticsearch] disable all xpack features in es5 when cluster.xpackEnable: false

What this PR does / why we need it:
Fixes issue where xpack features default to enabled for es5 when cluster.xpackEnable: false.
Which issue this PR fixes

fixes #9343
fixes #8996

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@simonswine @icereval @rendhalver @desaintmartin

I can confirm having xpack related issues, when trying to provision ES version 5.6.14

@cameronattard, thanks for this, with your fix I was able to setup ES 5.6.14 using this chart, without it master healthcheck was failing..
Please update it, by merging master into it, and bumping chart version.. otherwise, it's got merge conflict.

@anapsix No problem. It's up to date now.

@fabriziofortino looks like this affects the latest release as well that went out a couple hours ago.
The master and data nodes can't start.

@sc-chad Are you sure of that? I have upgraded 2 clusters from 6.5.0 to 6.5.3 using the new chart without any issue.

This is specifically affecting ES versions 5.x @fabriziofortino

I think I had a different root cause.  My default resources were to low to spin up all 7 members.  So they would stall.
After increasing the memory and coy the cluster started.

any objections to merging this from any of maintainers? ping

I confirm this problem for elasticsearch 5 non-oss version without licence.

Could this be simplified by removing the if/else conditions? Since all values are false by default.

That was only to ensure cross-version compatibility with cluster.xpackEnable.


Allow duplicate values for slsa_provenance_v0.2 schema

When processing multi-arch images, it is possible for the image index and the index manifests to have the same name. This change enables that. It is also in compliance with
https://slsa.dev/spec/v0.2/provenance#schema

/ok-to-test

As an example, Chains created a SLSA Provenance with this for subject:
[
  {
    "name": "quay.io/redhat-user-workloads-stage/ynanavat-tenant/bootc-image-builder/bootc-image-builder",
    "digest": {
      "sha256": "df93e3976eb26199f3eaaa06cc4cd83f9d6663491aabaf73f67018cbf094dbc8"
    }
  },
  {
    "name": "quay.io/redhat-user-workloads-stage/ynanavat-tenant/bootc-image-builder/bootc-image-builder",
    "digest": {
      "sha256": "91a66493a17b922553468c24ce4d719fa22db3cabac3064c91f8ad9774ff0e91"
    }
  },
  {
    "name": "quay.io/redhat-user-workloads-stage/ynanavat-tenant/bootc-image-builder/bootc-image-builder",
    "digest": {
      "sha256": "929e3d5f989c811078e38d38ccd0708cdc6c18d06064d2a23bddb61b861e3ec2"
    }
  }
]

There's also nothing in the spec that mentions the name should be unique. In fact, it states this:

// Output file; name is "" to indicate "not important".
"subject": [{"name": "", "digest": {"sha256": "5678..."}}],

That implies the name could be repeated, e.g. _, if there are multiple subjects where the name is not important.

/ok-to-test

/retest

/retest

/retest

/retest

This tells me that whatever is generating the SLSA provenance is not doing it in conformance with the specification:

subject[*].name string, required
Identifier to distinguish this artifact from others within the subject.

The semantics are up to the producer and consumer. Because consumers evaluate the name against a policy, the name SHOULD be stable between attestations. If the name is not meaningful, use "_". For example, a [SLSA Provenance](https://slsa.dev/provenance) attestation might use the name to specify output filename, expecting the consumer to only considers entries with a particular name. Alternatively, a vulnerability scan attestation might use the name "_" because the results apply regardless of what the artifact is named.

MUST be non-empty and unique within subject.

This tells me that whatever is generating the SLSA provenance is not doing it in conformance with the specification:

subject[*].name string, required
Identifier to distinguish this artifact from others within the subject.

The semantics are up to the producer and consumer. Because consumers evaluate the name against a policy, the name SHOULD be stable between attestations. If the name is not meaningful, use "_". For example, a [SLSA Provenance](https://slsa.dev/provenance) attestation might use the name to specify output filename, expecting the consumer to only considers entries with a particular name. Alternatively, a vulnerability scan attestation might use the name "_" because the results apply regardless of what the artifact is named.

MUST be non-empty and unique within subject.

Ah!  I didn't check the in-toto spec which actually better defines that field. Well the culprit here is Chains. I also think that a unique name is somewhat shortsighted when it comes to images. Filed an issue on the spec: https://github.com/in-toto/attestation/issues/348
Regardless of the outcome there, I think we should merge this. We don't get any value from enforcing this restricting. Arguably, the uniqueness check also doesn't capture the special meaning of _.

/retest

Filed EC-558 for the long-term solution to this problem.


Various refactorings

Some refactoring to make adding redirects from legacy advanced finders
easier.

adds FactoryBot and some taxonomy related factories
refactors taxonomy_spec_helper so that only two methods are used to set up taxonomy tests.
removes memoization of Service object which can cause problems with tests

Lets get this merged, as I think it'll help get https://github.com/alphagov/finder-frontend/pull/1204 done


Theme2.1

Created a lab for adding a custom theme to WAb. Sorry for all the commits I was trying to format with each commits. Thanks to John G for showing me the mark up editor. Please squash this pull request to reduce the commits.

we need to get rid of the add_custom_theme/Corel Auto-Preserve/ folder

Formatting is screwed up need to fix it the editor didn't catch it

67 commits later, LGTM.


LibriSpeech RNN-T Workload for ASR: JAX

Workload
Task
Automatic Speech Recognition
Dataset
LibriSpeech (clean 100 hour)
Model
Deep Speech model (CNN LSTM)
Reference Implementation
Leda provided us with a reference implementation in PyTorch.
A PyTorch implementation is currently in progress, see the PR #27 and the associated issue #30.
There is also an existing MLCommons Training implementation for PyTorch.

Hi @fsschneider,
I would like to work on JAX implementation.

Wonderful!
@znado do you two want to meet to discuss tackling this workload?
@kamalkraj Let me know if there is anything you need that I can help you with. I linked our current PR for the PyTorch version. It is in a working state and we are currently checking whether we can boost the performance of our trained model further. But this should be a good reference implementation.

Now that we have deepspeech jax workload working, we can close this issue


Angular tutorial project never loads

Describe the bug
Angular tutorial project never loads but instead hangs at installing dependencies step. I waited 10 minutes once, 30 minutes a second time. Guess I'll go do a different tutorial.
Link to the blitz that caused the error
https://stackblitz.com/run?file=package.json
Steps to reproduce

Go to https://stackblitz.com/run?file=src/app/app.component.ts

Expected behavior
I can use StackBlitz
Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 96.0.4664.45
Major version = 96
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.45 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 62557170,
  "usedJSHeapSize": 59430142,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 16495809c277ab8697fd8fae8005569533001817

Additional context
No response

@brian-cato-lab49 hey, sorry for the troubles. Do you see any errors in the browser console? Could it perhaps be related to https://github.com/stackblitz/webcontainer-core/issues/405?

:wave: It seems this issue has been sitting for a while without new info. I'm closing this, but please feel free to open a new one if you still encounter this problem.


Add nightly runs for zstd codec

Description
Added nightly runs with zstd codec index setting for 2.9 and 2.10 OpenSearch versions.
Issues Resolved
Related to https://github.com/opensearch-project/OpenSearch/issues/9312
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

submitted adhoc run with the zstd index setting and everything seems to be working as expected.
https://build.ci.opensearch.org/blue/organizations/jenkins/benchmark-test/detail/benchmark-test/4779/pipeline


Roll Engine from fae92ed57427 to 43166df232d2 (4 revisions)

https://github.com/flutter/engine/compare/fae92ed57427...43166df232d2
2020-11-16<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from E3briMHHv... to 2R7OWHAQq... (flutter/engine#22537)
2020-11-16<EMAIL_ADDRESS>Roll Dart SDK from 92e087bf82e2 to d67a5c245285 (1 revision) (flutter/engine#22536)
2020-11-16<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 6LXPsNi-P... to LOnq8wpIx... (flutter/engine#22534)
2020-11-16<EMAIL_ADDRESS>Roll Skia from 031a76756e24 to cce84d1fd893 (4 revisions) (flutter/engine#22533)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from E3briMHHvl6d to 2R7OWHAQqLwf
fuchsia/sdk/core/mac-amd64 from 6LXPsNi-PFxz to LOnq8wpIxWpg
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


[FLINK-4628] provide user class loader during input split assignment

In analogy to the configure() method, this also sets a context class loader during input split assignment.

Looks good, +1 to merge

Merging this...


add support for safety checker (NSFW filter)

Now you can activate the Hugging Face diffusers library safety check for NSFW and other potentially disturbing imagery.
To turn on the safety check, pass --safety_checker to invoke.py on the command line. For developers, the flag is safety_checker=True passed to ldm.generate.Generate(). Once the safety checker is turned on, it cannot be turned off unless you reinitialize a new Generate object. This PR does not add a safety checker toggle to the WebGUI.
When the safety checker is active, suspect images will be blurred and a warning icon is added. There is also a warning message printed in the CLI, but it can be a little hard to see because of its positioning in the output stream.
Example blurred image:

There is a slight but noticeable delay when the safety checker runs.
Note that invisible watermarking is not currently implemented. The watermark code distributed by the CompViz distribution uses a library that does not seem to be able to retrieve the watermarks it creates, and it does not appear that Hugging Face diffusers or other SD distributions are doing any watermarking.

@lstein; What is the 'diffusers' dependency being used for? I don't see it imported anywhere.

@lstein; What is the 'diffusers' dependency being used for? I don't see it imported anywhere.

It is imported in the safety_checker() method of generator/base.py. It takes a while to load, and will not be used by everyone, so I thought it better to defer loading until needed.

Merged from command line.

Is it possible to have the safety checker return a nsfw flag via the api instead of replacing the image? For my use case as a discord bot via api it would be ideal to still get the nsfw render and simply hide the imag behind a spoiler tag that requires clicking to see it (or in a seperate nsfw channel etc).
The idea of throwing away "expensive" renders based on results from a inaccurate nsfw check (in the SD beta test version it regularly flagged SFW and ignored NSFW) feels wrong. Sorry to bring it up after you've already merged, I only just saw the thread... My $0.02


'Question': is there any way to have collapsable sections in the middle column of the document?

Ah, we don't support this ourselves, but you could probably add something that does it using jQuery in the all_nosearch.js!


Disable the btrfs collector for node-exporter by default

We have seen in profiles that node-exporter is spending 6% of its CPU time in the btrfs collector.
Disabling the btrfs collector for most cluster is probably a good idea by default.
--no-collector.btrfs

Nope, it was running on a GKE node that has no btrfs enabled nor were we able to find any metrics generated from that collector.

If anyone wants to contribute to kube-prometheus, here's an rather easy one:
https://github.com/prometheus-operator/kube-prometheus/blob/117ce2f8b51621ccb480e8bdd712ae21e92b744e/jsonnet/kube-prometheus/components/node-exporter.libsonnet#L211-L212
We need to add
--no-collector.btrfs


Thank You

Thank you so very much for porting both of this and PlayerEx to 1.20

No problem! 💝
I'll close this, but do feel free to report issues on our repository whenever you see them!


Implement logging for hie-bios

Adds optional dependency for hslogger which is turned off by default.
The logging should not touch stuff for ghcide, cc @ndmitchell.
@mpickering Is this how you would imagine logging?

Maybe this branch should rather be a PR against multi-cradle?

Needs to be targeted against the right branch.


Lexicon v2

I read through, but didn't have any major questions. I'll probably just tweak a little formatting and merge.

Cool! If there's any tweaks to the API let me know so I can update my usage.
On Tue, Nov 10, 2015 at 8:31 AM, John DeNero<EMAIL_ADDRESS>wrote:

I read through, but didn't have any major questions. I'll probably just
tweak a little formatting and merge.
—
Reply to this email directly or view it on GitHub
https://github.com/oscii-lab/lex/pull/11#issuecomment-155477398.

That change should be fine. PhrasalRules are created in DecoderClientCorpus from RuleQueryResponses, and I think it is not possible that one of them is null and the other isn't. But better be on the safe side.

Aside from the comments above, looks good to me! Merge at your leisure.


complete

Reflect and summarize on the in-class process:
Single-line Summary
Today, Jason completed the lab. It took about 3 hours
Reflect and summarize on your process for each TODO item :

First, added the html elements
Next, we used css to create mobile UX.
Next, we used css to create desktop UX.

Checklist (before submitting, fill in each set of square brackets with an 'x')

[x] We have titled the Pull Request similar to our branch name (ex: 'brian-rick').
[x] This PR includes commits from both myself and my partner; e.g. We followed good pair programming practices by switching driver/navigator roles.
[x] There is no extraneous, unrelated code included in this PR.
[x] We have summarized our TODO: process above.


Fix for arm64 build failure when bluetooth enabled on linux platform

Description of Problem: In the platform/Linux code for bluetooth, we are using "typeof" in ConnectDevice function. which throws an error ("typeof not found") for the arm64 build.
Description of Fix:
Changed the "typeof" to "decltype" which is a valid C++ Standard keyword.
Testing Done:
Locally tested, arm64 build successful

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

Reopening, we should just make this change.

I've added the same fix for Tizen platform.


VM: Implement Stack Bitwise Opcodes

TODO

Opcode
status
PR

INVERT
open

AND
open

OR
open

EQUAL
open

PR #266 merged so closing this one


[Mastodon] Error on login

Extension
https://www.raycast.com/SevicheCC/mastodon
Raycast Version
1.78.1
macOS Version
14.5
Description
I'm unable to login to hachyderm.io using the extension. I have no idea if it's hachyderm.io or the extension. The website throws an error.

[logs in]

I would copy the url error here, but I don't know if there's sensitive information in the url string.
I've tried it in Firefox and Arc(chrome) both throw errors.
Steps To Reproduce

Install from raycast store
launch extension
sign in to hachyderm.io
see error.

Current Behaviour
Hachyderm throws error.
Expected Behaviour
Hachyderm sends appropriate credentials to raycast extension.

Same here :((

Add my MODIFIED link here:
https://mas.to/oauth/authorize?client_id=lH5gNdX0SMoSJV2rQs&redirect_uri=https%3A%2F%2Fraycast.com%2Fredirect%3FpackageName%3DExtension&response_type=code&scope=read%3Astatuses+read%3Abookmarks+read%3Aaccounts+read%3Afavourites+read%3Anotifications+write%3Afavourites+write%3Amedia+write%3Abookmarks+write%3Astatuses+write%3Anotifications&code_challenge=5UXqHZgoLfu3ykjhbQ5NGBp8bh71P8Xp3M&code_challenge_method=S256&state=eyJpZCI6ZpZGVyTmFtZSI6Ik1ljkhxhdm9yIjoicmVsZWFzZSJ9

but with modified getting this -.-

Would be great, to have a way to login with generated token or username/password as backup :///
any ideas?

@dvogt23 Be careful. Don't post your private credentials to the public.

I cannot reproduce the problem with https://mastodon.social. Do you guys have the mastodon.social account? Please let me know if you can reproduce the problem with the mastodon.social instance.

Thanks @LitoMore with mastodon.social it works :// what could be wrong with the mas.to link?

It might be related to:

https://github.com/mastodon/mastodon/issues/12915

It looks more like a bug from the instance.

@raycastbot rename this issue to "[Mastodon] Error on login"

Great tips @LitoMore did contact admin from mas.to and he adjust config like here and it works 🥳

Great. @raycastbot close this issue.