Change Clowder landing page title and description

The current landing page is not informative for users that are not familiar with Clowder.
For example, the SEAD project simply calls the instance "SEAD"  and provides a description of the service.
I suggest that we consider the following:

Rename Clowder to "Sensor Data Portal" or similar
Remove references to Clowder from the landing page
Provide a brief description of the service and what first time users should do and expect (e.g., click Sign-up to register for access, etc.).

That sounds great ... In the description, there should also be a description of the tool launcher. Is there a simple way to link to the environment manager from the front page or user home page?

I'll need input from @max-zilla or @robkooper  on this one. We can certainly link to the environment manager from the description, but I'm not sure how difficult it would be to add it elsewhere in Clowder.  I'm also not sure of the steps or effort required to customize Clowder as I've suggested above.

@craig-willis made you admin, you can change some aspects right now.

If there is a writeup somewhere we can add a link to the Help menu as well (I will need to do that from the command line).

@rachelshekar Do you want to weigh in on what to call this and what should be in the description?

@craig-willis Let's change this to sensor data portal for now and revisit later.

This was done last week, closing.


refactor place where indices are sampled for DE

Fixes #1068

Thanks @talashuach
Feel free to merge @PierreBoyeau


Fix issues around the current billable metric

There is a bug where any activity triggers a groups users to become billable, rather than requiring an annotation:

[ ] https://github.com/hypothesis/lms/pull/4930

The discussion around this has highlighted some other related areas which be good to fix too

[ ] The names of the metrics in Hubspot do not cleanly map to the names in Report
[ ] https://github.com/hypothesis/lms/pull/4944

I think we are blocked here on some thumbs up for the naming scheme for the fields in Hubspot.
Currently they are all like: "LMS Teachers This Semester" which follows the pattern "LMS  ". The proposed new scheme is  "  " which might look like "Billable Teachers This Semester".
We believe we can change the names without changing the underlying field name in Hubspot, so no code changes should be required.


How will prevent empty message or spaces click

I figured out that the Chatbot component provides a validator function. Chatbot properties
That function allows you to specify what needs to be considered as a message.
So for your case, it will be like-
const validateInput = (input) =>{
 return input.length > 0 && input.trim().length !=0;
}

<Chatbot
 //other configurations
  validator={validateInput}
/>

It will prevent the message from being sent.

Worked

I figured out that the Chatbot component provides a validator function. Chatbot properties That function allows you to specify what needs to be considered as a message.
So for your case, it will be like-
const validateInput = (input) =>{
 return input.length > 0 && input.trim().length !=0;
}

<Chatbot
 //other configurations
  validator={validateInput}
/>

It will prevent the message from being sent.

Thanks @devo-id to pointing that out.


[actions] : Test-Suite action workflow

Migrating the test-suite which is currently on travis to github actions.
Additional context:
Build configuration for travis: https://travis-ci.com/github/nvm-sh/nvm/builds/220257244/config

To add more detail here:
nvm requires being able to compile node 0.4 through 19 (and later, as new versions are released), so it needs to be able to get multiple versions of python and gcc, and figuring out the exact OS requirements, and CPU architecture requirements, will be part of the work here.
You can see here: https://github.com/nvm-sh/nvm/blob/946da2e743a5a9040f18af667594ae609a11ac5f/.travis.yml that we run tests in all of 5 shells (bash, sh, dash, zsh, and ksh, although ksh is disabled for now), and for the installation ones, both with curl available and without curl (but with wget).
We also then run the matrix of test suites - fast, sourcing, slow, installation_iojs, installation_node (and we may need to shard these differently in the future).
Caching of nvm's cache dir, or node_modules, is not necessarily required, but should be explored to keep test runtimes manageable.
In the future (after we're migrated off of travis), we'll want to run the same test suite on Mac and Windows, so the action matrix should be written with that in mind.


travis CI fix + test + architecture asset

Motivation:

The construction of the image was being executed in all the branches. Now I'm just forcing the master.

Additional:

more test
misspell fix 77% -> 100%
more information on README to ENVs
adding architecture:

@thiago-scherrer can you insert the architecture image on EN-US?

@thiago-scherrer can you insert the architecture image on EN-US?

Done! => PR-4


Use huge L3 page table entries to offset map the entire physical memory?

Here is my config:
pub static BOOTLOADER_CONFIG: BootloaderConfig = {
    let mut config = BootloaderConfig::new_default();
    config.mappings.physical_memory = Some(Mapping::Dynamic);
    config
};

When I looped through every page table entry (L4, L3, L2, L1) I noticed that there were only huge pages for L2 entries (so only chunks of 2MiB), and no huge pages for L3 entries. Even with I run qemu with 4G of memory, there are still only many huge L2 entries and no huge L3 entries.  Wouldn't it be better if huge L3 entries were used instead of huge L2 entries?

Sure. We'd have to make sure that the CPU supports it (CPUID leaf 8000_0001, EDX, Page1GB). Feel free to open a PR.


Running 'gulp default' resets source css file?

Hey, every time I modify css styling and run gulp default, it resets it. Am I doing something wrong or suppose to modify the gulp script in some way?

 If anyone is reading this, the source files of css isn't the .scss files, its the .less files.


[unexpected behavior] Restrict Storybook to be opened inside Storybook

Issue details
I was crating a Link component and gave href as / for the anchor tag.  This opened Storybook inside the rendering area of the main Storybook.
Screenshot is attached below -

Please specify which version of Storybook and optionally any affected addons that you're running

@kadira/react-storybook-addon-info : ^3.4.0
@kadira/storybook : ^2.35.3
@kadira/storybook-deployer : ^1.2.0

Affected platforms

Mac OS High Sierra ( 10.13.1 )

Looks like you are using a very old version of Storybook. Can you please try the @storybook/react v3.2.x

@igor-dv The new version does not have this issue. 👍🏻
But when I change href to '#', it opens up the iframe in for the component, with the url as -
http://localhost:9009/iframe.html?selectedKind=Link&selectedStory=with%20text

Is this an intended behavior?

What would you expect instead in that case?


Softtype deserializer should return node type

It now returns as part of the principal properties:
{
    "id": 10966,
    "label": "x1",
    "sessionIndex": 3,
    "type": "RoME.Pose2",
    "properties": {
        "label": "x1",
...

Will add to SDK.

#88

thanks!


Support Modular Arithmetic for Quantities

Support the new Quantity overloads of div and mod:
https://cql.hl7.org/2020May/09-b-cqlreference.html#modulo
https://cql.hl7.org/2020May/09-b-cqlreference.html#truncated-divide

https://github.com/DBCG/cql_engine/pull/495

Closing the issue based on PR https://github.com/DBCG/cql_engine/pull/495 is merged.


Added support for PHP 7.3.

The pull request added support php 7.3 for those still using it and not yet ready to upgrade to version 8

PHP 7.3 only has support for another 18 days as of today. If anything, i'd consider supporting 7.4 but I wasn't planning on supporting 7 at all.


Spelling + grammar fixes

🛠 Description
Minor tweaks to fix spelling and grammar in the support escalation doc.
📋 Checklist

🙋♀ Do any of the following that apply?

[ ] 🚨 This is a breaking change.
[x] 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

[ ] ✅ Yes (required for dev PRs)
[ ] ➡ Will cover in a future PR (feature branch PRs only)
[x] ❎ Not needed (small/internal change)

📖 Did you update documentation?

[ ] ✅ Public docs in docs (required for dev)
[ ] ✅ Internal dev docs in src (required for dev)
[ ] ➡ Will cover in a future PR (feature branch PRs only)
[x] ❎ Not needed (small/internal change)

#Sign-off


Put common parts in a module

Move the common code from analyse.py scripts to a common module.

Repo with minimal example of local module in poetry:
https://github.com/manulera/example_local_python_module_poetry


[Question]: navigation through commits and creating branches

Hi,
I understand it is possible to reset a branch to a commit hash, or checking out branches previously created.
But is it possible to create branches on commit hashes created earlier on the current branch and checking them out?
Thanks!

Please clarify your question? Thanks!

Nice!
Erm I started trying dotmesh out exactly 55 mins ago, so I'm probably going to finish the tutorials before starting hacking everything ^^

I'd be glad to assist if you fancy giving it a try :-)
In the meantime, I'm going to close the issue as the question is answered!


Stormblessed: Watch danielle6751(?:\W*(?:at\W*)?outlook\W*(?:dot\W*)?com)?

Stormblessed requests the watch of the watch_keyword danielle6751(?:\W*(?:at\W*)?outlook\W*(?:dot\W*)?com)?. See the MS search here and the Stack Exchange search here.

danielle6751(?:\W*(?:at\W*)?outlook\W*(?:dot\W*)?com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


No handler could be found error in 03_read_telemetry.py

I've been following your youtube tutorial and I'm having problem in running the script. It said "No handlers could be found for logger 'dronekit'" and the script terminated. Any suggestion to avoid this error please?

did you install dronekit from source?


Wrong example code in README

In the README inside the API usage it is written that the frequency can be changed using mrf.frequency. However, it is freq that must be used.

Thanks for pointing @gvannoye. Made a pull request to correct this. https://github.com/tai/morfeus/pull/3/commits/b54142412da0e863e3dfaaad4e790a8b1ffccfcc

Thanks for the fix!


homebridge?

Did you ever get homebridge working in your cluster? This is one of the few things left on my old infra that I'd like to get moved over but I haven't read a lot of positive things about running it in k8s. Found the homebridge helmrelease in the archive folder and tried to deploy it, but I can't add the bridge to my HomeKit.

Well nevermind. I was able to get this working by setting the networking mode to host.


#176:add box plot

Hey @timmolter
I add box plot(#176 )
This is the effect picture, the example is xchart-demo/src/main/java/org/knowm/xchart/demo/charts/boxPlot/BoxPlot01.java

This is the main logic of my box plot
Each box plot is calculated from all y values corresponding to the same x value. The calculation formula is shown below:
Example, all y values are 1, 2, 3
First arrange all corresponding y values from small to large
First quartile position = q1 = (n +1) / 4
Second quartile position = q2 = (n + 1) / 2
Third quartile position = q3 = 3 * (n + 1) / 4
If the calculation result is an integer, the corresponding position is taken
Q1 = 1, Q2 = 2, Q3 = 3
Then the first quartile = Q1 = 1, the second quartile = Q2 = 2, and the third quartile = Q3 = 3
Otherwise calculated with the following formula
For example, 1, 2, 3, 4
q1 = (4 +1) /4=1.25
q2 = (4 + 1) /2=2.5
q3 = 3 * (4 + 1) /4=3.75
then
Q1 = 1+ （2-1） * 0.25 = 1.25
Q2 = 2 + （3-2） * 0.5 = 2.5
Q3 = 3 + （4-3） * 0.75 = 3.75
Upper limit = Q3 + 1.5 * IQR = Q3 + 1.5 * (Q3-Q1)
Lower limit = Q1-1.5 * IQR = Q1-1.5 * (Q3-Q1)
Please review the code for me. If you are not satisfied, please tell me in time. I will give you a satisfactory answer.
Hava a nice day!

hi,tim @timmolter
may take your time to view this PR ?

Looks great (mostly)!
Can you change the way the series are added? Instead of :
    BoxPlotSeries[] boxSeries = new BoxPlotSeries[3];
    boxSeries[0] = chart.addSeries("aaa", Arrays.asList("Blue", "Red", "Green", "Yellow", "Orange"),
        Arrays.asList(40, -30, 20, 60, 50));
    boxSeries[1] = chart.addSeries("bbb", Arrays.asList("Blue", "Red", "Green", "Yellow", "Orange"),
        Arrays.asList(10, -10, 30, 50, 60));
    boxSeries[2] = chart.addSeries("ccc", Arrays.asList("Blue", "Red", "Green", "Yellow", "Orange"),
        Arrays.asList(50, -20, 10, 40, 50));

make it like this:
    boxSeries[0] = chart.addSeries("first", Arrays.asList(40, -30, 20, 60, 50));
    boxSeries[1] = chart.addSeries("second", Arrays.asList(10, -10, 30, 50, 60));
    boxSeries[2] = chart.addSeries("third", Arrays.asList(50, -20, 10, 40, 50));

No legend is needed.
I think that's a more intuitive way to interact with a box plot. See the MATLQB API description:
https://www.mathworks.com/help/stats/boxplot.html

Input data, specified as a numeric vector or numeric matrix. If x is a vector, boxplot plots one box. If x is a matrix, boxplot plots one box for each column of x.

On each box, the central mark indicates the median, and the bottom and top edges of the box indicate the 25th and 75th percentiles, respectively. The whiskers extend to the most extreme data points not considered outliers, and the outliers are plotted individually using the '+' symbol.

Thanks, i will modify the code


Fixing a couple issues with the spark chart and adding an external LB…

I need the cassandra cluster exposed externally so I can run cassandra cql queries within our zeppelin notebooks.

works for me.  @timfpark can you merge

Thanks for taking a look @jmspring


Fix #10

Thanks!

You are welcome :-)

Can this be published to npm? Or is it better to wait until we fix the thing with the files extensions?

Sure, sorry! v2.2.3

Thqank you! :-D


ryujinx: 1.1.100 -> 1.1.178

Description of changes
See the changelog for more information.
cc @06kellyjac
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and “core” functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Since #173116 (23rd May) r-ryantm hasn't done a single auto update
The last one being https://github.com/NixOS/nixpkgs/commit/38a9ef814bcbcc9e6265c99fe1e47cb67b06ac7d 9th April
Can you put nixpkgs=./. like before please.
It wont work if you're in the dir when you run it but it works perfectly from the root and more importantly via the passthru when using the maintainer script or via the r-ryantm bot
Previous discussion here:
https://github.com/NixOS/nixpkgs/pull/165756#discussion_r836504978

Other than my comment the change looks good

Yeah it was a different maintainer that removed it
👍 LGTM


If manually move an event back into that month, it is not picked up (bug in http request?)

If manually move an event back into that month, it is not picked up (bug in http request?)

closing - seems to work OK


Teamnado 5960: Add manifest creation gate via chrome.auth.user.entitlements

This adds a gate for manifest creation based on whether or not the user
has a satellite subscription. This is checked via whether or not the
user is entitled to smart_management.
When the user is entitled, the application functions the same as it has
previously. In the case that the user is not entitled, yet has
manifests, they should still be able to manage preexisting, but not
create new, manifests. In the case the user is not entitled and has
no manifests they should receive a friendly empty state that directs
them to support, and their subscriptions.

Codecov Report
All modified lines are covered by tests :white_check_mark:

Files
Coverage Δ

...tButtonWithModal/CreateManifestButtonWithModal.tsx
100.00% <100.00%> (ø)

src/components/NoSatelliteSubs/NoSatelliteSubs.tsx
100.00% <100.00%> (ø)

src/hooks/useUser.ts
87.50% <ø> (ø)

...es/SatelliteManifestPage/SatelliteManifestPage.tsx
100.00% <100.00%> (ø)

src/utilities/consts.ts
100.00% <100.00%> (ø)

src/utilities/factories/user.ts
100.00% <ø> (ø)

:loudspeaker: Thoughts on this report? Let us know!.

/retest

/retest


Replace References to Log Entity with New BuildStep Entity

This PR integrates a new Entity that represents the step-by-step information for a given build.
This change comes as a result of a decision to no longer source the build logs from the top-level "logs" field, but instead from the "log" field attached to each child of the steps field.
Justifications for change

The steps field provides several other pieces of supporting meta-data on top of the logs.
The logs field doesn't give enough detail to the purpose or meaning of the logs.
There is not a one-to-one relationship with steps and logs. The steps field helps us identify which logs belong to which step.

Points of contention:

What should the name of the Entity via the @Entity annotation?

What should the name of the Entity via the @entity annotation?

I'd say "buildstep"


Colors differ from terminal style guide

Hi there! 👋
Thanks for the port!
I noticed that the background color seems off - I think Mantle is supposed to be Base, e.g. #181825 -> #1e1e2e for Mocha. The style guide for terminals has just been updated today, in the catppuccin/catppuccin dev branch.

Just updated the background colours to those on the style guide for terminals :)


No application type "Other" when creating an OAuth 2.0 client ID

It is now called as "Desktop App"

@Rango305


PR: Improve path manager

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @goanpeca

Hello @goanpeca! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/app/mainwindow.py:

Line 2983:1: W293 blank line contains whitespace
Line 2998:25: E222 multiple spaces after operator

@impact27 I am testing this PR with https://github.com/spyder-ide/spyder-kernels/pull/181
I am getting FAILED on test_runcell_pdb. Any ideas 🙃 ?

@impact27 I am testing this PR with spyder-ide/spyder-kernels#181
I am getting FAILED on test_runcell_pdb. Any ideas 🙃 ?

I think you are using the latest master for spyder but not for spyder-kernels.

@ccordoba12 tests finally passed! So, could we merge the spyder-kernels PR ? so this can be updated to use master?

Test now depend on https://github.com/spyder-ide/spyder-kernels/pull/181 being merged @ccordoba12

What happened with the "synchronized" functionality, was this hacked in this PR? I thought this was a very useful feature and I used it very often...

Ok I just saw that the sync functionality is hidden temporarily while the code for it is being updated.
@goanpeca is this planned to add it back soon?


Clarify Environment Variable syntax

I'm only seeing a large textbox labeled "Environment variables". This is annoying and frustrating. I have no idea how it expects the variables to be formatted.

Additionally, it seems like the environment variables aren't even persisted into the project files. No matter what I enter, it is not saved.
However, when I add an Environment node into my .njsproj file, the values in it appear in the UI.

I agree on the Environment Variables formatting. I'll update that for the next update.
The saving issue is fixed in VS 2017 Update 4

I'm going to add this to the docs, I spend too much time trying to add this to the property pages with very little success..


Scenario 26 - Needs to recheck all the scripts, fix, improve the scripts and verify against IS550

Committed the test script.
Tested and verified in cluster.


Test Failure Related To Arbitrum Network

Currently the monorepo tests are failing with the following error, seemingly related to Arbitrum, and possibly the Nitro upgrade:
@umbra/umbra-js:   1 failing
@umbra/umbra-js:   1) Utilities
@umbra/umbra-js:        Recipient identifier lookups
@umbra/umbra-js:          looks up transaction history on arbitrum one:
@umbra/umbra-js:      AssertionError: Target cannot be null or undefined.
@umbra/umbra-js:       at Context.<anonymous> (test/utils.test.ts:190:30)
@umbra/umbra-js:       at runMicrotasks (<anonymous>)
@umbra/umbra-js:       at processTicksAndRejections (node:internal/process/task_queues:96:5)
@umbra/umbra-js: error Command failed with exit code 1.
@umbra/umbra-js: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
@umbra/umbra-js: error Command failed with exit code 1.
@umbra/umbra-js: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run test exited 1 in '@umbra/umbra-js'
error Command failed with exit code 1.

The issue is that the arbType and arbSubType fields are deprecated, so instead of returning 3 and 4 like we expect, they return 0 and null, so we never find a signed tx. It seems like just checking for the presence of a nonzero r/s/v in the tx should work as the fix, but waiting to get confirmation from arbitrum
https://github.com/ScopeLift/umbra-protocol/blob/efa104c21a0658cfa64ede7bae470fe060927c7f/umbra-js/src/utils/utils.ts#L137-L154


[DG] More useful content for new Developers

[x] More diagrams (Object, Class, Sequence, Activity)
[x] User stories
[ ] Check if all use cases are included
[x] Glossary
[ ] Manul Testing
[x] Effort ❗

#505

Manual testing:


Write out a chunk per thread plus one for the shared event definitions.

This also eliminates the shared string table. Thread specific string tables are currently not pruned on clear. This is tricky and will be done in a followup.

Awesome - nice to not need the runtime anymore


Azure.Configuration should be renamed to Azure.ApplicationModel.Configuration

The library has been renamed but there are still a number of project files with the older name:
Example:
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/appconfiguration/Azure.ApplicationModel.Configuration/src/Azure.Configuration.csproj
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/appconfiguration/Azure.ApplicationModel.Configuration/tests/Azure.Configuration.Tests.csproj
Those should be renamed to match the assembly/package name.

Pr merged


Update main.tf to use preferred object assignment syntax

New Terraform cli throws
Error: Invalid Argument Name in resource "aws_api_gateway_integration" "cors_integration": "application/json" = <<EOF Argument names must not be quoted.
This fix brings it in line with the new preferred syntax.

Closing this instantly in favour of https://github.com/mewa/terraform-aws-apigateway-cors/pull/1 which fully updates the files to meet 0.12.0 changes.


This patch allows users explicitly trigger the linting.

This patch allows users to pass an option to the lint addon to disable automatically triggering linting on change, and export (via defineExtension) a method to explicitly trigger linting from user code.

I've merged this as 536bba1 and modified it in 3686816 to:

Rename the option to lintOnChange (to avoid double negatives) -- must be false to disable.
Make the extension method use its this binding, rather than also expecting the editor to be passed as an argument.

Thanks a lot!


Issue #81 : Use default maven source plugin configuration

to allow the install and deploy mojos to attach sources jars

no backport


Supporting the repository

Supporting the repository

Is this just spam? I am going to close this PR because it looks likes spam, if it is not, you are welcome to reopen and tell me.


Add ComfyUI support

Hope to add support for ComfyUI

support

The project is expected to be completed in 1-2 days：https://github.com/AIGODLIKE/ComfyUI-ToonCrafter


Vertex group index error

Describe the bug
While trying to export a model from Blender I get the following error: IndexError: bpy_prop_collection[index]: index 16 out of range, size 15
vertex_group_name = blender_vertex_groups[vertex_group_index].name
I have previously succesfuly exported this model as a GLB file, but after messing around with removing certain bones from vertex groupes I get this error and can't export the model anymore.
If this issue is not solvable without supplying my model, I will be able to send it in private to someone who would want to help me, but I would rather not put it on github.
Thank you in advance!
Version

OS: Ubuntu 19.10
Blender Version 2.81.16

Full error log:
Traceback (most recent call last):
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/init.py", line 420, in execute
return gltf2_blender_export.save(context, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_export.py", line 40, in save
json, buffer = __export(export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_export.py", line 54, in __export
__gather_gltf(exporter, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_export.py", line 69, in __gather_gltf
active_scene_idx, scenes, animations = gltf2_blender_gather.gather_gltf2(export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather.py", line 36, in gather_gltf2
scenes.append(__gather_scene(blender_scene, export_settings))
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_cache.py", line 65, in wrapper_cached
result = func(*args)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather.py", line 55, in __gather_scene
node = gltf2_blender_gather_nodes.gather_node(blender_object, blender_scene, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py", line 45, in gather_node
node = __gather_node(blender_object, blender_scene, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_cache.py", line 65, in wrapper_cached
result = func(*args)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py", line 59, in __gather_node
children=__gather_children(blender_object, blender_scene, export_settings),
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py", line 125, in __gather_children
node = gather_node(child_object, blender_scene, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py", line 45, in gather_node
node = __gather_node(blender_object, blender_scene, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_cache.py", line 65, in wrapper_cached
result = func(*args)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py", line 63, in __gather_node
mesh=__gather_mesh(blender_object, export_settings),
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py", line 297, in __gather_mesh
export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_cache.py", line 65, in wrapper_cached
result = func(*args)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_mesh.py", line 41, in gather_mesh
primitives=__gather_primitives(blender_mesh, blender_object, vertex_groups, modifiers, material_names, export_settings),
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_mesh.py", line 117, in __gather_primitives
export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_cache.py", line 65, in wrapper_cached
result = func(*args)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_primitives.py", line 50, in gather_primitives
vertex_groups, modifiers, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_cache.py", line 65, in wrapper_cached
result = func(*args)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_primitives.py", line 94, in __gather_cache_primitives
None, blender_mesh, blender_object, vertex_groups, modifiers, export_settings)
File "/snap/blender/34/2.81/scripts/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py", line 714, in extract_primitives
vertex_group_name = blender_vertex_groups[vertex_group_index].name
IndexError: bpy_prop_collection[index]: index 16 out of range, size 15
location: :-1

Hello,
Can you please send me the model in private? This will not be possible to find the issue without it.
You can use email (contact AT julienduroure.com), or twitter DM if you prefer

For info: I received the model in private.
This is quite a big model, so investigation will not be really easy

(Not sure if this is helpful, or even the same issue or not :/)
I can get a backtrace to the same line (but I don't think this is the same way you did it).

Instructions to trigger

Create a rigged mesh. I just imported RiggedSimple.
Select the mesh in Object mode and press Alt+D to create a linked duplicate. Move the duplicate out of the way.
Select the duplicate and add a new Vertex Group to it. Now the original has N vertex groups and the duplicate has N+1 vertex groups.
Go into Weight Paint mode on the duplicate, select the new Vertex Group, and paint some vert. That vert now has a .groups[i].group index that is out-of-bounds for the Vertex Group list on the original object (but in bounds for the duplicate).
Export.

Explanation:
This code assumes that D.meshes['mymesh'].vertex.groups[i].group is a valid index into blender_vertex_groups:
https://github.com/KhronosGroup/glTF-Blender-IO/blob/28e0ccf3fc153d38c286e435467e5fba79bc9be1/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py#L733-L734
The blender_vertex_groups variable is initialized here:
https://github.com/KhronosGroup/glTF-Blender-IO/blob/28e0ccf3fc153d38c286e435467e5fba79bc9be1/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_nodes.py#L260
vertex_group_index does not have to be a valid index into this (sort of like how a material_idx does not have to be a valid index into blender_object.material_slots).
So maybe the extract code should continue when vertex_group_index is not in bounds?

Thanks for this analysis.
So, linked to #900 too (maybe not directly linked, but we have to have it in mind when searching a solution for these 2 tickets)

Just want to give you guys an update, I went ahead and restarted my project with a fresh model and used the things I learned in Blender to continue working on the project.
If there is anyone else who would like to look into it, you can contact me, and I hope this can help fix this bug.
Thanks guys!


cannot install via curl

$ curl -L git.io/hosty
$ curl -L git.io/hosty | cat
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
$ curl -L git.io/hosty
$

I think git.io/hosty on installation isn't returning any text.

Currently, it works for me, I don't know if it may be a problem with your device or network, or if git.io went temporally down, either way you can always directly use https://raw.githubusercontent.com/astrolince/hosty/master/install.sh


feat: TxOptions addition and SubmitTx removal for v0.15.0 prep

Prep for celestia-node v0.15.0
Will need to be tested with the node release

applied the complete removal of with golang version. let's see how it goes


Avoid unnecessary query for edges when only fetching totalCount in connection

Feature Request Type

[ ] Core functionality
[x] Alteration (enhancement/optimization) of existing feature(s)
[x] New behavior

Description
When I was analyzing the SQL queries using the debug toolbar, I found that Django creates two queries for such a connection query. However, one of the queries is unnecessary.
# Query for a connection
query {
  books {
    totalCount
  }
}

# Query for edges (unnecessary)
SELECT books_book.id, books_book.title, books_book.author, books_book.price
FROM books_book
LIMIT 101;

# Query for totalCount
SELECT COUNT(*) AS "__count"
FROM "books_book";

The first SQL query is created when the strawberry connection resolver (not strawberry_django) iterates over the queryset to create edges. The creation of edges will always happen even if edges are not requested in the query. Because a Connection, which includes edges and page_info, is always created first and then used as a source to construct its sub_fields such as edges, totalCount, and __typename.
# strawberry/relay/types.py
@classmethod
def resolve_connection(...)
...
  edges = [
      edge_class.resolve_edge(
          cls.resolve_node(v, info=info, **kwargs),
          cursor=start + i,
      )
      for i, v in enumerate(iterator) # SQL query is triggered here
  ]
...

I saw there is a connection resolver for ListConnectionWithTotalCount. However, it is just a wrapper around the connection resolver in strawberry. Should we create a customized connection resolver for Django, which checks the request fields before doing operations that trigger SQL queries? For example, we can check if the edges is in the query before creating edges in the resolver.
selections = {s.name for s in info.selected_fields[0].selections}

edges = []
if "edges" in selections:
  edges = [
      edge_class.resolve_edge(
          cls.resolve_node(v, info=info, **kwargs),
          cursor=start + i,
      )
      for i, v in enumerate(iterator)
  ]

If this is a valid idea. I can create a PR for this optimization. Thanks for your time and consideration.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Hey @Carlisle345748,
Hrm, that's interesting. It is actually the first time I see someone only wanting to query the totalCount and not the edges =P
I do understand your use case, and I do welcome you to create a PR to optimize this :)
Feel free to ping me on discord if you need any help with the development

is a patch available somewhere for this one ?

This was fixed by https://github.com/strawberry-graphql/strawberry/pull/3480


fixed navigation bar so that it takes full width in big screens

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #59 on Mike's repo
Test URLs:

Before: https://main--impala--jzachari.hlx.page/blog/
After: https://issue-59--impala--jzachari.hlx.page/blog/

Looks good!


Platform netatmo does not generate unique IDs

Home Assistant release with the issue:
Home Assistant 0.89.1
Last working Home Assistant release (if known):
N/A
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
Netatmo
Description of problem:
I have netatmo setup but when I start HA, I have the following error:
2018-12-20 10:37:44 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/app/homeassistant/helpers/entity_platform.py", line 344, in _async_add_entity
    raise HomeAssistantError(msg)
homeassistant.exceptions.HomeAssistantError: Entity id already exists: sensor.netatmo_outdoor_temperature. Platform netatmo does not generate unique IDs

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- platform: netatmo
  station: HOME
  modules:
    Indoor:
      - temperature
      - max_temp
      - min_temp
      - humidity
      - noise
      - pressure
      - co2
    Outdoor:
      - temperature
      - max_temp
      - min_temp
      - humidity

Traceback (if applicable):

Additional information:

Potential duplicates #18113 #20375 #20383 #21031

testing fix here : https://community.home-assistant.io/t/netatmo-homeassistant-exceptions-homeassistanterror-entity-id-already-exists-error/86050/10?u=maheus

Still have the issue with 0.93.2.

Fixed with 0.94 and by adding: discovery: false in the config files.


Add get_readiness tool to retrieve status of all registered components

Add a new tool get_readiness to retrieve the status of all registered datasets, models, embeddings, views, and other primitives.

New Tool Implementation

Add get_readiness tool in crates/runtime/src/tools/builtin/get_readiness.rs to retrieve the status of all registered components from RuntimeStatus.
Implement the get_readiness tool to use the RuntimeStatus struct to get the status of all components and return it.

Module Changes

Add get_readiness module in crates/runtime/src/tools/builtin/mod.rs and include it in the list of available tools.

Runtime Status Changes

Add a new method get_all_statuses to the RuntimeStatus struct in crates/runtime/src/status.rs that returns the status of all registered components.

For more details, open the Copilot Workspace session.

Is the description made available to the users in any way? And are views always ready? Or do they propagate the underlying source's readyness


Number.EPSILON support

Node.js currently doesn't support Number.EPSILON. Mozilla reports that it isn't available in several browsers and its value is about 2.2204460492503130808472633361816E-16.
I ran the previous version that included the vsmall calculation and found its value to be 1.4272476927059598e-15. Since javascript is numerically stable across platforms it is likely that this value will work in multiple environments.  I obtained the same value using Safari browser on JSFiddle. I tested Number.EPSILON and 1.43e-15 as initial values to the vsmall calculation with surprising results. Assuming the vsmall calculation is related to the calculations used in the algorithm, then Number.EPSILON may be too small. When I used the Number.EPSILON constant in my test problems I obtained the same results.
Can you change line 99 from:
vsmall = Number.EPSILON;
to:
vsmall = Number.EPSILON || 2.2204460492503130808472633361816E-16;
Since vsmall is only used to test values near zero, it seems like Number.EPSILON is usable. In reviewing the code, I noticed that one comparison uses '<=' while the others use '<'.

I think I revert the change, using the original code calculating  vsmall.
The comparisons are different also in the original FORTRAN code.
You may give a look at the code solve.QP.f on CRAN.
Tonight I create a new release: 1.3.0.
Thanks for the feedback! :)

Published 1.3.0.

Thank you! Your solution will operate slightly faster since vsmall is calculated only once. If you are concerned about performance, have you consider declaring the work array to be:
new Float64Array(2 * n + (r * (r + 5)) / 2 + 2 * q + 1)
Since the work array is only used internally this might improve performance without changing the interface. I think that all browsers and javascript environments support typed arrays now.

Thanks, Typed Arrays are in my radar. Maybe in 2.0.


Distance, containment, etc. implementations

@phaza, I'd like to implement these for you. However, what structure would you like these to have, and how would like me to lay out tests for those things?
For distance, Euclidean distance fails on the surface of the earth, so you'd probably need something like the Haversine formula or better yet, Vincenty's which more accurate, but also more computationally expensive. Would the distance method live on the Point class and take another Point though?
For polygon containment, there's plenty of algorithms there, and ray-casting is the fastest I know of (Github folks, correct me). I would think both Polygon and MultiPolygon would have an implementation that takes a single Point.
What are you thoughts?

Better yet, could we simply integrate with https://github.com/phayes/geoPHP?

That's great news!
Geometry::distanceTo(Geometry) would be the abstract idea, with Point to Point being the easiest. Eventually we could include finding the closest points between two geometries(Like Polygon and Linestring), and then calculate the distance between them. But any start is a good start, so just starting with Point to Point would be great. IMO precision is important, so Vincenty's sounds good, but everything is relative. If it makes a request->response cycle too slow/unfeasible, we can't use it.
For containment Polygon and MultiPolygon should check containment against all other Geometries. A Polygon/MultiPolygon can potentially contain another polygon. But again, any start is a good start, so just Polygon::contains(Point) would be awesome.
As for including geoPHP I'm a bit vary. I find the code a bit dirty, and I find code like this a bit weird to say the least: Collection.class.php#L282-292
It's also not namespaced, nor using autoloading, and generally it looks more academic than PHP if you get what I mean.

I can certainly agree with geoPHP opinion. One last idea: Since this module is relying on PostGIS support, why not use the built-in database functions to perform this, since it already performs most of the operations you'd want:
    DB::select('select ST_Distance(Point(0, 0), Point(1, 1))');

This isn't the prettiest, but I'm pretty sure it'd be faster than any PHP implementation. Perhaps there's a cleaner way to leverage the database for this?

Well, of course being able to create distance queries via the DB makes sense, and it needs a prettier interface than today, but if you have two objects, hitting the db for those is a huge penalty. In good optimized apps, hitting the db is always the bottle neck. Now imagine you have 10 objects, and you want to calculate the distance between them all, that's 1000 db calls..

I'm 95% sure it will be faster to do the calculation in PHP, but feel free to benchmark it to be sure.
Do 100 iterations with geoPHP, and 100 iterations where you do DB calls like the one you showed, and we'll see.

I was able to get Distance and nearby queries working on a geography database.

SELECT id,name,
ST_Distance(ST_GeographyFromText('SRID=4326;POINT($srid_point[1] $srid_point[0])'), location) as distance
FROM places
WHERE ST_DWithin(ST_GeographyFromText('SRID=4326;POINT($srid_point[1] $srid_point[0])'), location, 10000)

where $srid_point[1] $srid_point[0] is longitude and latitude values respectively.

My database is on a whole other server (around 1,000km away from my PHP server!) so for my instance I suspect going out to the server might be slow-ish ... but then again, if someone else is offering to implement it, then I will politely say thankyou and accept it.  :-)

@BrendanTWhite if you're reasonably "mathy enough", the inverse vincenty algorithm will give you distances all via PHP/Code: https://en.wikipedia.org/wiki/Vincenty's_formulae#Inverse_problem
Bear in mind that these do not take into account terrain, and will assume WGS84 (if you use those parameters).

@mstaack by closing this issue, am I to understand that this feature is unwanted?


Can't access folders/files with characters like "ç, ã, é"

If I create a folder like "test" and add a file "test.pdf", the file is indexed and that's the normal behavior.
But if I create "test/testç.pdf", the file isn't indexed nor it appear on any cache file at .config/dmenu-extended/cache.
Also if I create "testç" and put the files "test.pdf" and "exampleç.pdf", even if the folder is recorded at dmenuExtended_folders.txt, the files are not.
Anyway I hope I'm being clear because english is not my mother language.
Thanks in advance!
PS: I use utf-8 scheme.

Hi there,
I think I understand what you're saying, I am also using UTF8.
I've had struggles in the past with non-english characters, debugging these problems gets hard because I don't really know how to handle them correctly.
I'm finding that if I launch dmenu_extended_run from a keybind (i3 exec statement) then opening the files and folders you've given fail.
However, If I run dmenu_extended_run from within a terminal everything works as expected.
Maybe it is something to do with environment variables not being loaded?
Are you able to try launching dmenu_extended_run from a terminal, rebuilding your cache from that terminal launched menu, then relaunching dmenu_extended_run from the terminal and opening those files for me and let me know if that also works for you?

Hey Mark,
I tried running from terminal the dmenu_extended_run and updating the cache from the menu and also tried dmenu_extended_cache_build but the problem continued.
I'm no programmer, but I tinkered a little today and set the debug flag on the dmenu_extended.py and this is what I got:
PS: the stdout was just the names of the files and folders.
vinieich@archeve> dmenu_extended_cache_build > /dev/null                                                                     ~/descomplica
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/dmenu_extended.py", line 666, in cache_save
f.write(item+"\n")
File "/usr/lib/python3.6/codecs.py", line 718, in write
return self.writer.write(data)
File "/usr/lib/python3.6/codecs.py", line 376, in write
data, consumed = self.encode(object, self.errors)
UnicodeEncodeError: 'utf-8' codec can't encode character '\udce3' in position 88: surrogates not allowed
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/bin/dmenu_extended_cache_build", line 6, in
dmenu_extended.dmenu().cache_build()
File "/usr/lib/python3.6/site-packages/dmenu_extended.py", line 1076, in cache_build
self.cache_save(filenames, file_cache_files)
File "/usr/lib/python3.6/site-packages/dmenu_extended.py", line 683, in cache_save
print('Culprit: ' + item)
UnicodeEncodeError: 'utf-8' codec can't encode character '\udce3' in position 97: surrogates not allowed
Then searching about this error got me to this page:
https://stackoverflow.com/questions/27366479/python-3-os-walk-file-paths-unicodeencodeerror-utf-8-codec-cant-encode-s
I made sure that my files and folders already has the filename as UTF-8.
Like I said, I don't know python programming (or any other by the way) but that link may help.
Thanks for trying to solve this. I tried but luck can only get me to a certain point lol

Thanks for your response.
Have you tried the suggestion of trqx on this page: https://github.com/markjones112358/dmenu-extended/issues/38
That might help, it has for others.

Sorry, I thought that I'd mentioned that.
Yes, that's why I was certain that I already has UTF-8 filename encoding.
I'll paste the output here:
convmv -f windows-1252 -t utf8 -r testcaseçç                                                                     ~/Documents
Starting a dry run without changes...
Skipping, already UTF-8: testcaseçç/thesecóndtestcase
Skipping, already UTF-8: ./testcaseçç
No changes to your files done. Would have converted 0 files in 0 seconds.
Use --notest to finally rename the files.
And this was the same for the production folders.
That link that I posted didn't give any light into it?
I ask because it was almost like alien language to me, but I'll try to understand that when I have some time.
Thanks in advance.

Ok, found the problem.
There was a file that wasn't encoded on utf-8 hidden in my folders.
The problem is, when dmenu-extended find such files, it automagically delete all other files from cache that contain foreign character even if it is already in UTF-8.
You can reproduce this just by converting any UTF-8 folder to windows-1252.
After that, all files/folders with foreign characters (á, ç, õ") will vanish from cache
At least for me this is how it is happening.

I'm closing this issue as the workaround works.
But probably the automagically delete thing still happens.


feat: Update copy on auth pages GRO-731

Sometimes you just need a quick win - here are some pretty pictures:

before

after

before

after

https://artsyproduct.atlassian.net/browse/GRO-731
/cc @artsy/grow-devs

Looks like this just needs a quick change to the tests: src/v2/Apps/Authentication/__tests__/authenticationRoutes.jest.tsx is what failed!

Good call on Integrity! I poked around a bit and didn't bump into anything obvious. I checked out a file like this one:
https://github.com/artsy/integrity/blob/main/cypress/utils/signUpUser.ts
And I don't see anything that's tied to the text of the headline or title of the page so maybe we're ok?


Vim9: crash when suppressing "Pattern not found error"

Describe the bug
In Vim9 script, suppressing a "Pattern not found" error might make Vim crash.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9

    def Func(b: bool)
        Eval(b)
        Crash()
    enddef

    def Eval(b: bool)
        eval !b
    enddef

    def Crash()
        sil! :/not found/d _
        sil! :/not found/put _
    enddef

    Func(true)
EOF
)

Vim crashes.
Expected behavior
Vim doesn't crash.
Environment

Vim version: 8.2 Included patches: 1-2380
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(363)

Additional context

backtrace
valgrind log
asan log

Using valgrind to check for the error, it also happens without the Eval() function.

Using valgrind to check for the error, it also happens without the Eval() function.


Chat persistence

We currently only store the chat transcript in-memory and it gets lost when we restart the IDE. The goal of this issue is to persist the transcript in project settings so that you can 1) manage multiple transcripts and 2) the chat message is available after you restart IntelliJ

Merged in https://github.com/sourcegraph/jetbrains/pull/438


Enable compression on webpack-dev-server (#966)

This enables compression of generated assets over webpack-dev-server. Especially useful if you're developing on a remote machine with subpar connection speed.
I couldn't think of any test cases but I'm open to suggestions.

Looks great. Thanks!


"No tests found"

I'm getting a "No tests found" error. Tests run fine using vim-test, cargo test and cargo nextest run. See below for the example lib.rs that fails to run:
use std::env;
use std::error::Error;
use std::fs;

pub struct Config {
    pub filename: String,
    pub query: String,
    pub ignore_case: bool,
}
impl Config {
    pub fn new(args: &[String]) -> Result<Config, &'static str> {
        if args.len() < 3 {
            return Err("Not enough arguments");
        }
        let query = args[1].clone();
        let filename = args[2].clone();
        let ignore_case = env::var("IGNORE_CASE").is_ok();
        Ok(Config {
            filename,
            query,
            ignore_case,
        })
    }
}

pub fn run(config: Config) -> Result<(), Box<dyn Error>> {
    let contents = fs::read_to_string(config.filename)?;
    for line in search(&config.query, &contents, config.ignore_case) {
        println!("{}", line);
    }
    Ok(())
}

pub fn search<'a>(query: &str, contents: &'a str, case_insensitive: bool) -> Vec<&'a str> {
    let query = if case_insensitive {
        query.to_lowercase()
    } else {
        query.to_string()
    };
    let mut results = vec![];
    if case_insensitive {
        for line in contents.lines() {
            if line.to_lowercase().contains(&query) {
                results.push(line);
            }
        }
    } else {
        for line in contents.lines() {
            if line.contains(&query) {
                results.push(line);
            }
        }
    }
    results
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn case_sensitive_test() {
        let query = "duct";
        let contents = "\
Rust:
safe, fast, productive.
Duct tape.
Pick three.";
        assert_eq!(
            vec!["safe, fast, productive.",],
            search(query, contents, false)
        );
    }
    #[test]
    fn case_insensitive_test() {
        let query = "dUCt";
        let contents = "\
Rust:
safe, fast, productive.
Duct tape.
Pick three.";
        assert_eq!(
            vec!["safe, fast, productive.", "Duct tape."],
            search(query, contents, true)
        );
    }
}

Hmm, this file works for me. Do you have nvim-treesitter installed and the Rust grammar installed for that?

I'm a numpty. This was exactly the problem...
I've just ported my nvim.init over to lua. That's my excuse, anyway...

Thanks for your help.

We should feed this info back to the neotest repo maintainer. It would be useful to have the information in the 'installation' section that the relevant treesitter languages must be installed...


Temporary tasks to get lost scholarsphere oals

@banukutlu I'm just going to deploy this branch temporarily to production (after I test it on qa), run the task, and then redeploy the last release once its done.  Then, I'll just delete this branch since it is only meant to be temporary.
@anaelizabethenriquez There were only 81 Scholarsphere Open Access Locations with an associated Scholarsphere Work Deposit that had no DOI.  I was only able to identify 2 of the three Scholarsphere Work Deposits with no DOI and an associated publication that has no Scholarsphere OAL.  The Scholarsphere Work Deposit IDs for these two are 1138 and 42.  Not sure what the issue is with these.  I'm also not sure why the third one isn't popping up in my queries.  I must be forgetting something 🤔 .
Edit: The third one has a Scholarsphere Work Deposit ID of 201.  It wasn't showing up in my queries because this publication doesn't have any OALs.

@ajkiessl I investigated the three ScholarSphere work deposits for which the associated publication didn't have a ScholarSphere link. The articles all turned out to have DOIs, so I added those in ScholarSphere and (if necessary) in RMD. I'll check on them after the next ScholarSphere import to make sure RMD picks up the ScholarSphere links.


Test passing though the code doesn't check all test cases.

Describe your problem and - if possible - how to reproduce it
I am using this code and it is passing though I am not checking it for all users.
while(users.hasOwnProperty("Alan")){
return true;
}
return false;
Add a Link to the page with the problem
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/basic-data-structures/check-if-an-object-has-a-property
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: Version 67.0.3396.99 (Official Build) (64-bit)
Operating System: Windows 10

If possible, add a screenshot here

it also passes with the following:
if ('Alan' in users) {
return true;
} else {
return false;
}
Since all tests contains Alan it always passes if all you check for is Alan.

This has been fixed in #19390 , @RandellDawson since you had made the PR, I request you to also close the issue :)

This is actually not resolved yet.  There is a PR #19477 which seems to be a possible fix.  I will keep this open until it has been tested, approved and merged.


Class Inheritance

Related to issue #33
Detect super classes for some simple inheritance methods.
I added this into the class transform. It seems better to have as an option that can be enabled but I am not sure how best to add it as an option.
Reacts to the following inheritance methods:
function Class1() {}
function Class2() {}
Class1.prototype = Object.create(Class2.prototype);
Class1.prototype.constructor = Class1;

and...
function Class1() {}
function Class2() {}
Class1.prototype = new Class2();
Class1.prototype.constructor = Class1;

and...
var util = require('util');
function Class1() {}
function Class2() {}
util.inherits(Class1, Class2);

See test/transform/classInheritance.js for coverage.

Thanks, this looks absolutely splendid!
I will merge this in, though I have a bunch of comments about the implementation. I appreciate if you tackle them, but if not I'll do it by myself after merging.
I think there really is no need for an option to turn the inheritance detection off. If you run the class transform, I think you'd expect it to also cover inheritance. Anyway, we'll see after the release if somebody starts to complain.
prototypal inheritance detection
It seems overly strict that the following code alone does not constitute inheriting from Class2:
function Class1() {}
Class1.prototype = new Class2();

only when it's also followed by constructor assignment:
function Class1() {}
Class1.prototype = new Class2();
Class1.prototype.constructor = Class2;

I've personally written lots of inheritance code where I did not do that additional constructor assignment. Unless you do rely a lot on instanceof, you won't notice the difference. It's more likely that doing new Class2() will bite you first when the constructor contains some code that has side effects.
imports detection
I really like the detection that util.inherits comes from require('util'). I wonder whether we should also detect import util from 'util'. Currently one cannot run commonjs transform before class transform as then the inheritance detection would not work. As I'd generally like to avoid such dependencies, then I think we should also support the import syntax. It is a nuisance though to write this code - feels like duplication, but I don't know what a good alternative would be.
doc-comments
I also like that you have organized the code to nice classes. You have even taken care to document the methods. I should point out that the code-base is following a convention that all public methods have doc-comment (/** ... */) while private ones do not (they can use // comments) - this helps one to distinguish at glance which methods are only called from within the class itself and which ones are also called from outside.
Use of matchesAst()
Calling matchesAst({...})(node) looks ugly. I think I should create a separate version of this function that would perform the matching immediately instead of returning a matcher function.
That's more of a note for myself.

Thanks for reviewing! I will work on addressing your comments soon.

I merged it in and addressed the remaining smaller issues by myself.

I've documented the remaining issues in #183


Workbox doesn't cache in iOS standalone

Library Affected:
workbox-sw
workbox-webpack-plugin
Browser & Platform:
iOS 11.4
Issue or Feature Request Description:
I trying to build a PWA that have the offline capability. I'm using workbox-webpack-plugin with InjectManifest class. The app runs well on Safari/Chrome (MacOS and iOS) under both online and offline condition. When I added to the app to home screen from Safari (iOS) and turn off internet connection, I got No connection page from Safari. I tried to debug from Safari but there are no error messages.
App URL: https://nordot-pwa.herokuapp.com/
Webpack config:
new InjectManifest({
      swSrc: path.resolve(paths.appSrc, "sw.js"),
      swDest: "service-worker.js",
      // Don't precache sourcemaps (they're large) and build asset manifest:
      exclude: [/\.map$/, /^(?:asset-)manifest.*\.js(?:on)?$/]
})

Sw.js:
console.log("Service worker is running!");

workbox.skipWaiting();
workbox.clientsClaim();

workbox.routing.registerNavigationRoute("/index.html", {
  whitelist: [/^(?!\/__).*/]
});

// Background sync for tracking post api
const bgSyncPlugin = new workbox.backgroundSync.Plugin("trackingQueue", {
  maxRetentionTime: 24 * 60 // Retry for max of 24 Hours
});
workbox.routing.registerRoute(
  /\/api\/.*\/*.json/,
  workbox.strategies.networkOnly({ plugins: [bgSyncPlugin] }),
  "POST"
);

workbox.precaching.precacheAndRoute(self.__precacheManifest);

When reporting bugs, please include relevant JavaScript Console logs and links to public URLs at which the issue could be reproduced.

Hey @jeffposnick I put the workbox script inside the service worker but it still doesn't seem to work.
Is there any way to make this work?

I have the same problem using GenerateSW. Everything works nice in Safari itself but not after adding the page to the home screen. Ok it works partially - that is when I open the app while having the internet and then disable it - I can go through the app and assets are loading properly from cache - but if I go to home screen and then back to the app - it shouts there's no internet - just like the service-worker had been deleted and it can't load index.html from cache anymore.

Interestingly when I move precache manifest's content from separate file the service-worker.js itself it works as expected 🤔 Do you have any idea why it might behave like this? @jeffposnick

In the end I decided to use sw-precache-webpack-plugin and now it works like a charm.

Here's what seems to be the relevant, underlying bug in Safari: https://bugs.webkit.org/show_bug.cgi?id=190269
I'm going to leave this tracking bug against Workbox open for the time being, and we should loop back around once that WebKit bug fix makes it out to iOS Safari and confirm that it's fixed.

This appears to be working now in iOS12.1?

does it work for you like that?
because I have the same configuration but it doesn't work for me!!😔
I mean  for InjectManifet()

I also, lost my cache once I close Safari with iOS12.1.
It seems to still not be working.

If folks on the latest iOS release still can reproduce issues, let us know and we can reopen.

@jeffposnick Hey Jeff, I have tested on iOS 14.6, app works offline on safari, but the app that is saved on home screen, doesn't work offline.

I believe that the existing service worker registration and cache isn't shared when an app goes from being loaded in Safari to standalone on the iOS homescreen, but after you initially load the iOS homescreen version (while online), a new service worker registration should go into effect.
So are you testing this after loading the iOS homescreen version at least once while online, or are you testing after immediately going offline?
If you're testing while first opening it while online, and you never see the service worker caching take place, that sounds like a bug. Could you share the URL of the web app you're testing with?

@jeffposnick
Thank you for the prompt reply.
I did a test with going inline first with the standalone app then switched offline and tried.
I will not be able to share the app I am working on at the moment, but I can duplicate the setup and host somewhere.
I Will message here when that is up.
To confirm though, do you have a iOS standalone app working offline?

https://www.tiktok.com/ uses Workbox and has a custom offline experience, and when I just tried adding it to my homescreen using the most current iPad version of Safari, I was able to see that custom offline experience after switching to Airplane mode.

I'm seeing the same issue on iPadOS 15.3. My Workbox-based PWA is cached correctly inside Safari - I can move to other pages while offline, even pages I hadn't visited before, and also close the tab and open again - but a version of it installed to the homescreen does not cache anything at all.

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


Feature: add  inline field from selection

New context menu command: add  inline field from selection.
This command opens pop up selection similar to "Add field at cursor" but filtered only for input-type fields.

do you mean that it should search for a matching field in the fileclass definition, or create a new field with the selected text as name?

do you mean that it should search for a matching field in the fileclass definition, or create a new field with the selected text as name?

I'm not sure if I thought this feature thoroughly if fileclasses are used. Because fileclasses help to construct yaml fields efficiently. I'm not familiar with using fileclasses, so you should think yourself if this feature is useful in case of the user utilize fileclasses.
Maybe there are successful workflows that don't utilize fileclasses (I'm not sure about it). In case of that, my feature enhances making inline fields quickly.
Currently my Metadata Menu is not working properly (Obsidian 1.3.5, mac ventura 13.0, Apple M2).I cannot see command to Add field at cursor. I see only Metadata Menu: field options and Metadata Menu: fileClass attributes options.
What was my original workflow:
Write sentences in language learning and then mark them somehow for later reference. This could be achieved several ways, like separating sentences by newline and then writing tag to the line. Or one could use some QuickAdd script (or Templater) to append the selection to some file (not very easy thing to do). Obviously the most elegant way is to make an inline field of the selection so that the selection itself is used as a value. Then there should be options to select previously used fields as field name (or to create new field name).
One could store content for later reference by using links. This can be done by [[my reference | my content]]. This involves manual typing and I'm not sure how to query that content to my reference. Also there is possibility to use note blocks in the note you are editing, and then in your reference note make links to these note blocks. I think that is also very tedious work.
One more solution: separate sentences by newline and then add separate link to the source. There could be several links. Then later query these lines that include certain link. The only bad thing here is that sentences are separated by new lines, making the text fragmented. But maybe one could use regular expressions to query content between {}. Then one could query text of the form {my text} ... links ... [[my link]].
Conclusion: I think data query fields offer easy way to find well defined objects. But in my case one sentence can belong to many categories. What is the most elegant way to classify text in a note that belongs to multiple categories. Could there be some new operator that assigns a value to multiple fields simultaneously or something completely different?

I could start with a new « set as field value » command where the selected string "xyz" would be wrapped in a (fieldName:: xyz) or [fieldname::xyz]
A "full line" would be prefixed :
fieldName:: full line

I'm not sure if inline fields are as effective as I thought previously. But if you believe in them then go for making them better.
Alternate workflow is to just capture selections to new notes and then use note-specific metadata for organization. When I think about that, it makes me wonder do you need inline metadata for anything.


Crash while presenting the credential after re login

Complaining didmethod is empty need to investigate

fixed as present credential of the sdk no longer need didMethodType so this issue no longer exits


Allow specifying environment variables when generating a pod template spec

Many applications use environment variables for runtime configuration and without the ability to specify environment variables when generating the PTS, generating a PTS is not very useful.

Environment variables will be specified by client at image build time and baked into the Dockerfile, pre-build, thus they are built into the image pulled by the PTS.
For documentation:

these env variables should not contain sensitive information (i.e. secrets, tokens, etc) as they are insecure.
PTS ("deployment-time") specified env variables will override duplicates specified in the Dockerfile ("build-time")

The other option was using the PTS URL as "storage" and encoding these env variables into the query string.


[Warning timer error]

Traceback (most recent call last):   File "C:\Users\cuzto\AppData\Roaming\Python\Python39\site-packages\discord\client.py", line 333, in _run_event     await coro(*args, **kwargs)   File "C:\Users\cuzto\Desktop\agb local\Cogs\mod.py", line 696, in on_message     end = datetime.strptime(user["warn_time"], "%m/%d/%Y") KeyError: 'warn_time'

If you just updated this from the old warning system; the json file has a new layout and so all users will need to be reset, I'm sure there is a way of doing this without resetting the file, but the simplest way is to reset the file.

If you just updated this from the old warning system; the json file has a new layout and so all users will need to be reset, I'm sure there is a way of doing this without resetting the file, but the simplest way is to reset the file.


Ticker Class not implemented?

loc = ticker.MultipleLocator(15.0)
plt.gcf().gca().xaxis.set_major_locator(loc)

This should produce a graph where the x-axis contains all integers that are multiples of 15, as described here: http://matplotlib.org/api/ticker_api.html#matplotlib.ticker.MultipleLocator. It works fine on the matplotlib side as you can see with this image:

However, when I convert to a mpld3 interactive graphic it changes the base to 10 and the step is wrong as you can see here:

If mpld3 doesn't support this, is there a workaround?

Thanks for identifying.  I will add this to the list of known issues.  As a workaround, you can use a FixedFormatter, e.g. plt.xticks(range(-90, 91, 15)). That will be funky when you zoom out, though.
Making this work correctly would be cool, and might proceed analogously to adding the FixedFormatter, which is captured in this commit: https://github.com/jakevdp/mpld3/commit/07abb8052af719d8c60bfc830936ad80a28d9a32

@aflaxman Great. The problem with the FixedFormatter is that it removes the interactivity. An interactive y-axis with a fixed x-axis doesn't look very good.

Agree, and /cc @etgalloway who just made the Date ticks do interactivity really well, in case he has an inspiration or suggestions for this.

@avipersin, as a variation on the workaround suggested by @aflaxman, try something like:
ticklocs = range(-20*15, 20*15+1, 15)
axislim = -90, 90
plt.xticks(ticklocs)
plt.xlim(*axislim)

This extends the ticks beyond the initial axis limits.  Ticks that would otherwise appear in the figure margins are filtered out (thanks to PR #345, recently merged into master).


simplify archive regex

https://github.com/cs3org/reva/pull/2235#pullrequestreview-801039778

In #2235 (review) @ishank011 mentioned that the current regex is
/eos/user/[a-z]/[a-z]./\w.
Is there a place where the format of accepted folders is described in the docs?
I also see another problem about this expression.
A case that this expression would accept is
/eos/user/a/a;/3?
which is not allowed.


reset to first page

Hi,
First of all very nice and useful plugin, congrat!
Anyway I've a problem cause I don't know how can I reset the plugin to first page. I need this because I use pagination together with some filters on the pages, and after one filter is applied I need to move paginator to first page. Physically I've reset the dom elements to show the good page but the issue is that after filter is applied I display first page on the current paginator page.
Basic example: I'm on 3rd page on paginator, I apply one filter which hide/show some  in list, I do the backend job to display the right content for the first page but paginator stays at 3rd page.
Currently as a workaround I've done:

$('ul.bootpag>li').each(function(){
if($(this).hasClass('disabled')){
$(this).removeClass('disabled');
return false;
}
});
$('ul.bootpag>li').each(function(){
if($(this).attr('data-lp') == 1){
$(this).addClass('disabled');
return false;
}
});

This doesn't work if you're on page 11. It just goes back to page 11.
Is there any way to do this through event triggering or through the API?


field statistics

What is the fill ratio (i.e. how often is used) each prop of each class?

https://github.com/Accord-Project/bsdd/blob/field-stat/bsdd-Key-fields-statistics.md

Statistics section added to README, it takes everything fro bsdd-key-field-statistics.md and added more..


[bug] visualViewport API fails to account for mobile keyboard height

Describe the bug
The visualViewport height in Tauri does not adjust correctly when the keyboard is shown or hidden, which affects the layout of websites. This issue is significant because many websites use visualViewport to adjust their layout dynamically, particularly to align tools and content above the on-screen keyboard.
Reproduction

Use the code snippet below to set up a listener for viewport resizing:

if (window.visualViewport) {
    // bottomOffset = '30px'
    viewportResizeUnsubscriber = createUnsubscribableListener(window, 'resize', () => {
        positionBottom = `${window.innerHeight - window.visualViewport!.height}px`;
    });
}

Open the application on an Android device or emulator with a Tauri environment.
Trigger the on-screen keyboard by focusing on an input field.
Observe that the visualViewport height does not change correctly, resulting in incorrect positioning of elements.

Expected behavior
When the on-screen keyboard is shown or hidden, window.visualViewport.height should accurately reflect the visible portion of the viewport, allowing for proper adjustment of elements positioned above the keyboard.
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.22631 X64
    ✔ WebView2: 127.0.2651.98
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.80.0 (051478957 2024-07-21)
    ✔ cargo: 1.80.0 (376290515 2024-07-16)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.11.1
    - pnpm: 8.15.5
    - npm: 10.2.4
    - bun: 1.1.8

[-] Packages
    - tauri [RUST]: 2.0.0-beta.25
    - tauri-build [RUST]: 2.0.0-beta.19
    - wry [RUST]: 0.41.0
    - tao [RUST]: 0.28.1
    - @tauri-apps/api [NPM]: 2.0.0-beta.16
    - @tauri-apps/cli [NPM]: 2.0.0-beta.23

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../build
    - devUrl: http://localhost:1420/
    - framework: Svelte
    - bundler: Vite

Stack trace
No response
Additional context
This problem is related to Tauri issue #9907.

Observation for debugging: On Android, the issue can be resolved by navigating to the home screen without closing the app and then returning to the app. It is unclear if this workaround also applies to iOS.


hasDirtyRelations not working on multiple belongsTo

I have a model with two belongsTo attribute. When a change the first belongsTo relation, hasDirtyRelations is triggered perfectly but on changing the second one, I get false for hasDirtyRelations.
qualification.js
export default DS.Model.extend(Validations, {
  profile: DS.belongsTo('profile'),
  degree: DS.belongsTo('degree')
});

profile.js
export default DS.Model.extend({
  name: DS.attr('string'),
  country: DS.attr('string'),
  qualifications: DS.hasMany('qualification')
});

degree.js
export default DS.Model.extend({
  name: DS.attr('string'),
  type: DS.attr('string'),
  country: DS.attr('string')
});

Serializer
export default DS.RESTSerializer.extend(DS.EmbeddedRecordsMixin, {
  attrs: {
   profile: { embedded: 'always' },
   degree: { embedded: 'always' }
  }
});

The relationships are established perfectly in both the cases for profile and degree and I could able to retrieve values from both the models. But if any value gets changed in degree model, qualification.get('hasDirtyRelations') returns false while the same returns true if there is any change in profile data. Is there anything that am missing out? The configs are enabled in the environment

That is really odd @arparthasarathi , because change tracker does not support nested dirty tracking.
So, the hasDirtyRelations should be false both times ( when you change properties in a belongs to relation )
hasDirtyRelation should only be true in qualification if you replace the belongsTo with another degree or profile  or you remove the profile or degree ( make them null )
Does that make sense?
There have been others expressing interest in making in possible to do nested dirtyness ( but not sure you need that )
I think you are just saying .. hey .. qualifications should not  'hasDirtyRelations' if you change a property on the belongsTo
if that is what you are saying .. then it would help me if you could make a test ( for the repo ) and show in a test that this is happening .. so i can see the issue ( you might have a particular setup that triggers it ) and then i can fix it

Thanks for pointing out @danielspaniel that the nested properties won't affect the output of hasDIrtyRelations.
The mistake was on my part, that I reassigned the whole profile object in the belongsTo that made hasDirtyRelations to return true. But was playing with the properties of degree.
Thank you for your prompt response. Let me give it a try. Closing this issue for now and shall reopen it if any issues persist.


Add support for functional testing

We should probably use cypress for this.
See @aarongill1's work on this.
Once we have this we could even do things like detect broken links.

There were some complications with CI/github actions and the monorepo structure, which I don't think were completely resolved and team priorities evolved, so still some work to do, however it was working locally and IIRC, wasn't far from working.


a variety of small notes and fixes after my first run of the year

most of our tiles are working well and things are good!
here are a few things that came up while i was in AoSOL; some are AoSOL related and some aren't.

there is no wand in aosol. we should, ergo, not recommend users get a wand. almost tricked me like 3 separate times lol
there's a weird "empty data" state for the 8bit tile that i noticed; i had to visit the realm for mafia prefs to update and start out with black. what i should add is a catch for when the mafia pref is not populated that says it's likely vanya's but to reload 8bit and check bc the pref is weird. (one additional note -- some parts of the tile update faster than others, in a way that made no sense to me. i bet this is because it's a very pref-based tile and some of the prefs update at different cadences but i did not notice this when i was developing the tile and am weirded out that this happened in-run. notably, the "turns left in this zone" and "turns left overall" were almost always 1 off unless i refreshed tourguide. weird!)
if there is value in goose-duping the boss rewards, we should put a gray bullet or a goose emoji next to the boss item in the AoSOL tile. specifically, there is a ton of value in goose duping the wishbone (+100% item drop for two accessories? yes please) and the bat paw (-50 ML for bridge & +50 ML for tavern)
we need to add a catch for standard for the melodramedary added bullet in the desert tile; camel is out but it still shows up as a recommendation in the guide. (while doing this, we probably should add a "use these milestones" if you have milestones in your inventory.)
after spending a run with the rufus/phone tiles, i think they work really well and i'm not super motivated to make changes immediately. however, i do think there is some potential value in a tile that tells you if there is a zone with the monster that drops your targeted item + shadow slabs; it's a useful piece of info and i was pretty constantly checking the wiki to see if the artifact quest was meaningfully good/easy to snag by picking the right rift to jump into.
the CMC tile has this "# of pushes to double your pill" thing and it is nice but when it turns negative it should just say "your pill is doubled" because the - here is a little confusing

in addition to these things, i think we should do a more full-scale refactor of our everything-looks-yellow/blue/red tiles now that there are more things that interact with them. this is a bigger deal and i will probably finish out everything else in this issue, close it, and make a new issue for the ELY/B/R stuff before i do it... but i think that would be a valuable thing to do. sort of like the clover rework, just a good place for us to figure out how ezan's old scaffolding works and either remake it or use it more properly.

re: cmc pushes, you are right. it seems to have major issues with lockets, which are zoneless. probably worth a mafia forum thread bc i bet that’s a simple change; if i have time today i’ll add one

Finished up 1/3/4/5 in #144. Noting for posterity that we still need to figure out 2/6, including whether some underlying mafia changes can be made to help.

closing this because i have resolved everything in #147


Promoting a tab named the same as default tabs causes it to be unselectable via UI

Description
When promoting extended data to a tab, the tab cannot be selected via the UI.

Investigation
Digging into the source it looks like Angular is targeting the tab by its name with ng-click="vm.activateTab(tab.title)", which there is two of. The tab is not selectable because the extended data tab title is named the same as one of the default Exceptionless tabs.
Workaround
Find the HTML for demoting the tab that is non-selectable, assign it a CSS ID so that it is selectable by JavaScript:
Demote Tab

Then, in the web console, execute the command to mimic a click to demote the tab.
console.dir(document.getElementById("demote").click());

Then, don't name extended data objects the same as the default ones. Would be better to have tabs selectable by id rather than title?
Hopefully this is useful to any who encounter this. Thanks.

Thanks for reporting this and sorry about taking so long to get this fixed. The latest nightly has a fix for this issue.


versions: update supported docker version

update from v17.12 to v18.03.
18.03 is the version installed in the CI VMs
and when using v17.12 some hotplug cpu tests
fail. then we need to make v18.03 our official
supported version.
Fixes: #418.
Signed-off-by: Salvador Fuentes<EMAIL_ADDRESS>

lgtm

lgtm

lgtm.
Related: https://github.com/kata-containers/ci/issues/41.


support log panel in explore

addresses issue #46 to return results in a way that works with the log view.
mostly this required making sure whatever field we want to use for the message is returned as the first string field in the array. beyond that it's some minor things to make the time and level fields configurable (if there is more than one time, choose the time; if there is no level field, pick one and rename it level).
does not use labels since those are on the field rather than the row so doesn't appear to be a good way to use them here.

force push after rebase to clear out changes to yarn.lock since they were accidentally committed and it was fixed in master


Check if simd is supported

A function that check if simd is supported. Based on wasm-check.
Can be used here instead of try/catch.
private simdSupported(): boolean {
  const u32 = (...bytes: number[]) => Uint32Array.of(0x6D736100, 1, ...bytes);
  const u32a = (...bytes: number[]) => u32(1610679297, 33751040, ...bytes, 40239360, 259);

  let wasm = u32a(84344833, 6357249, 17369600, 4259847, 186257917,<PHONE_NUMBER>);
  const buffer = wasm.buffer;
  if (WebAssembly.validate(buffer)) {
    try {
      new WebAssembly.Instance(new WebAssembly.Module(buffer)).exports['0']();
      return true;
    }
    catch (_a) {
      return false;
    }
  }
  return false;
}

Thanks for pointing this out. There is also the one from GoogleChromeLabs which looks great.
TF.js is using a copy of it:
WebAssembly.validate(new Uint8Array([
  0, 97, 115, 109, 1, 0, 0, 0, 1,  4, 1,   96, 0,  0, 3,
  2, 1,  0,   10,  9, 1, 7, 0, 65, 0, 253, 15, 26, 11
]))

This looks cleaner. I'll update my code. Thanks.


Exclude native asset selection from swap joins/exits

Description
See title.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?

[ ] Test that you can't join or exit with native asset in single asset flow for boosted pools.

Visual context
n/a
Checklist:

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

Fixed by https://github.com/balancer-labs/frontend-v2/pull/2335/commits/0ab23353ffed9c00cef6960bcfaa87f5788a1a22


Hover state on gallery card

It seems that we introduce somewhere a border-radius: 0!important on the nft-card class but this has not been done on the hover state

i will fix it.

🙈


Fix serialization bug with JSON slices

Fixes a bug involving JSON path expressions with slices not serializing properly (PXT-259).

This change is


HDDS-6098. Add a metric to track amount of data committed.

What changes were proposed in this pull request?
This is a simple fix to add a small metric to track amount of data committed into OM. tracking this will help in tracking the amount of write activity in the cluster
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6098
How was this patch tested?
Added a new test

Thanks for the contribution @mukul1987 and the reviews @swagle, @bharatviswa504.


[SPARK-27240][PYTHON] Use pandas DataFrame for struct type argument i…

…n Scalar Pandas UDF.
Now that we support returning pandas DataFrame for struct type in Scalar Pandas UDF.
If we chain another Pandas UDF after the Scalar Pandas UDF returning pandas DataFrame, the argument of the chained UDF will be pandas DataFrame, but currently we don't support pandas DataFrame as an argument of Scalar Pandas UDF. That means there is an inconsistency between the chained UDF and the single UDF.
We should support taking pandas DataFrame for struct type argument in Scalar Pandas UDF to be consistent.
Currently pyarrow >=0.11 is supported.
Modified and added some tests.
Closes #24177 from ueshin/issues/SPARK-27240/structtype_argument.
Authored-by: Takuya UESHIN<EMAIL_ADDRESS>Signed-off-by: Bryan Cutler<EMAIL_ADDRESS>Upstream SPARK-XXXXX ticket and PR link (if not applicable, explain)
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.
Original PR: https://github.com/apache/spark/commit/594be7a911584a05f875f217ac39af9a5eeff049

@jboreiko @gatesn Could we get this merged before the breaking changes.

@jboreiko @gatesn Could we get this merged/released?

included in https://github.com/palantir/spark/pull/582


nix-optimise.service: nix.optimise.dates should be list

Motivation for this change
building Nix...
building the system configuration...
error: The option value `nix.optimise.dates' in `/nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs/nixos/modules/services/misc/nix-optimise.nix' is not a list of strings.

Setting the time in /etc/nixos/configuration.nix with surrounding square brackets fixed the error.
nix = {
  optimise = {
    automatic = true;
    dates = ["3:45"];
  };
};

Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Shoot, I changed the type on a suggestion over in #18378, but forgot to change the default value to match!  :(  This should definitely be merged.  Thanks for catching it!  cc @joachifm

thanks.


feat: add volar-service-markdown

This integrates vscode-markdown-languageservice into Volar.
This was extracted from https://github.com/mdx-js/mdx-analyzer/pull/340.

Again, thanks for your work!


Job not found

response: Job not found, when using result analyze API

something's off, can't parse anything too

Same - is there an outage?

I can't parse any page all day using the Llamaindex client.

Seems to be an outage....

Still nothing. Eagerly waiting for that product to be reliable.

You now need to choose the language:  If you don't select something (either on the web or API) it just hangs.

I just get job not found for the job ids that are active in the llamaindex parser history.


Sort-Object doesn't respect - as a symbol when sorting strings

Filed on UserVoice, repros on both 5.1 and the latest 6.0-alpha.18.
Steps to reproduce
"somefile1","somefile2","s-abc","s-little","s-foo","s-poo","s-wtf" | Sort-Object

Expected behavior
s-abc
s-foo
s-little
s-poo
s-wtf
somefile1
somefile2

Note: this is actually the behavior already when using other symbols like _:

"somefile1","somefile2","s_abc","s_little","s_foo","s_poo","s_wtf" | Sort-Object
s_abc
s_foo
s_little
s_poo
s_wtf
somefile1
somefile2

Actual behavior
s-abc
s-foo
s-little
somefile1
somefile2
s-poo
s-wtf

Environment data

Name                           Value
----                           -----
SerializationVersion           <IP_ADDRESS>
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSVersion                      6.0.0-alpha
BuildVersion                   <IP_ADDRESS>
PSRemotingProtocolVersion      2.3
PSEdition                      Core
WSManStackVersion              3.0
CLRVersion
GitCommitId                    v6.0.0-alpha.18

@joeyaiello This s dup #3425 and it is External bug 😕

Wow, you're good. Leave a comment next time in the UV item and I'll make sure not to re-file? 😉

Clear. It is good workflow.


Sample data is not fully provided

Dear,
I would like to test the code with l1000 data set.
But when I got data from the link url where you provided in README, I could't find the file below in the zip archive.
'robust_normalized_tumors.npz'
Could you please add the file to the zip archive?
Thanks.

Hi, @iwatobipen!
Did you download the training data from step 4 of Installation instructions?

Thanks for your prompt reply. Yes, I downloaded from the link.
But the zip file has following two files only.

experiments_filtered.csv
robust_nomalized_controls.npz
Thanks,

Did you use this link? https://drive.google.com/drive/folders/1rMQmnAVMPPQxQTw0pjVvrGE-Toxys-w7

Yes I used the link and downloaed lincs as a zip archive. But the file contains two files.
1,experiments_filtered.csv
2,robust_normalized_controls.npz

@iwatobipen, this file is there, on Google drive (see screenshot). Just in case you have a problem with downloading it, try using this link


Ansible Operator playbook / role getting executed twice after initial operator installation, without any modification in resources

Bug Report

What did you do?
Installed sample memcached operator as explained in operator-sdk documentation.   the tasks are idempotent, just creating one sample deployment, secret and configmap.
What did you expect to see?
Playbook should be executed only once if its successful.
What did you see instead? Under which circumstances?
After operator installation,  the Playbook/role inside operator was executed twice even though playbook execution was successful in first attempt.  Also there were no changes in any of resources to trigger any kind of reconciliation.
Below are status of play book execution after first time installation. Full log is attached.
kubectl logs deploy/memcached-operator-controller-manager -n memcached-operator-system -c manager -f | grep rescued
localhost : ok=3 changed=3 unreachable=0 failed=0 skipped=0 rescued=0 ignored=0
localhost : ok=3 changed=0 unreachable=0 failed=0 skipped=0 rescued=0 ignored=0
memcached.log
Environment
Operator type:

/language ansible

Kubernetes cluster type:
Native K8s cluster.
operator-sdk version: "v1.25.0", commit: "3d4eb4b2de4b68519c8828f2289c2014979ccf2a", kubernetes version: "1.25.0", go version: "go1.19.2", GOOS: "linux", GOARCH: "amd64"

$ go version (if language is Go)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.3", GitCommit:"1e11e4a2108024935ecfcb2912226cedeafd99df", GitTreeState:"clean", BuildDate:"2020-10-14T12:50:19Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.3", GitCommit:"1e11e4a2108024935ecfcb2912226cedeafd99df", GitTreeState:"clean", BuildDate:"2020-10-14T12:41:49Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}
Possible Solution
Check the playbook execution status and if its successful dont execute the playbook again.
Additional context

Closing this issue as it is a duplicate of #6250 . Please feel free to reopen this issue if you feel it has been closed incorrectly.


Add a method for reseting a form's child element validations for form reuse

I have a bs-form embedded in a bs-modal.  If any of the form's child elements have been focused on during a previous iteration of the modal, the child element retains the showOwnValidation=true and immediately display their validation when the modal is opened.  The request is some way to reset the showOwnValidation back to false between iterations of the modal.
Work around for now is to surround the bs-modal or embedded bs-form with an {{#if...}} conditioned on the modal's display.
Example:
{{#if showModal}}
  {{#bs-modal open=showModal ...}}
    {{#bs-form}}
    ...
    {{/bs-form}}
  {{/bs-modal}}
{{/if}}

Output from ember bootstrap:info (available since 1.0.0-alpha.9):
 $ ember bootstrap:info
Npm packages:
ember-bootstrap: 1.0.0-alpha.12 -> 1.0.0-alpha.12
ember-cli: 2.13.2 -> 2.13.2
bootstrap: n/a
bootstrap-sass: ^3.3.7 -> 3.3.7
ember-cli-sass: 6.1.3 -> 6.1.3
ember-cli-less: n/a
Bower packages:
bootstrap: n/a
bootstrap-sass: n/a
ember-bootstrap configuration:
importBootstrapCSS: false

@simonihmig how can I help push this forward?

@rwwagner90 Great to see that you are willing to work on this one. I think discussing public API would be the first step.
The title of this issue assumes that it should be done per form element. But the use case requires per form. We should get an agreement of what to support.
Some of the components provided by ember bootstrap use an argument to reset internal state. Setting the property bound to it, resets the internal state. E.g. <BsButton> has a @reset argument, which resets the fulfilled / rejected state.
To be honest I'm not a big fan of that pattern. But I'm also not aware of an alternative pattern. And consistency of the APIs is also an important point.
For this specific case I'm wondering if it should be integrated with @model argument. Maybe validation state should be reset when object used as model changes? 🤔

@jelhan

For this specific case I'm wondering if it should be integrated with @model argument. Maybe validation state should be reset when object used as model changes?

This sounds reasonable. I think the model for our forms changes each time. The problem is the form elements themselves do not always change, so if one form has the same field as another and we switch forms, we end up with stale validation state.

@jelhan

For this specific case I'm wondering if it should be integrated with @model argument. Maybe validation state should be reset when object used as model changes?

This sounds reasonable. I think the model for our forms changes each time. The problem is the form elements themselves do not always change, so if one form has the same field as another and we switch forms, we end up with stale validation state.

Not sure if I can follow. I wasn't describing a feature existing today but a potential public API to serve the use case.
To be more precise I think we would need to add an additional argument @resetValidationStateOnModelChange to <BsForm>. If true  the validation state of all form elements of the form should be reset when the object bound to @model argument changes.
@resetValidationStateOnModelChange needs to be false by default for backward compatibility. We might want to change default value in next major release.

@jelhan

Not sure if I can follow. I wasn't describing a feature existing today but a potential public API to serve the use case.

Yes, I did not say it existed, I said it sounds reasonable and I agree.
I think @resetValidationStateOnModelChange sounds like a good API. Shouldn't it be true always though? If the whole model changes, I am not sure why we would want to keep old validations around.

I think @resetValidationStateOnModelChange sounds like a good API. Shouldn't it be true always though? If the whole model changes, I am not sure why we would want to keep old validations around.

We can't introduce breaking changes. Therefore it must be false at least until the next major.

@dspigarelli @rstudner perhaps this is something someone in Expel could take on internally? Could be a good way to contribute back 😃

@dspigarelli @rstudner perhaps this is something someone in Expel could take on internally? Could be a good way to contribute back 😃

Is this still an issue in recent versions? I would expect that <BsModal> and a <BsForm> components are teardown when the modal is hidden. But I could be wrong. At least I cannot recall having noticed this bug.