Package Verifier - Installers time out in verifier but work in test environment

There are a couple of packages where choco times out the installer when running the test environment, but the package works fine when running in the test environment:

https://community.chocolatey.org/packages/QGIS-LTR/3.16.10
https://community.chocolatey.org/packages/vikunja-desktop/0.17.0
https://community.chocolatey.org/packages/tutanota/3.85.6

Temporary verifier exemptions have been added.

Found another one. https://community.chocolatey.org/packages/vikunja-desktop-unstable


OCM-5019 | ci: Updated/Created ids 76199,72526

small fix on provider

https://issues.redhat.com/browse/OCM-5019
Change type

[ ] New feature
[x] Bug fix
[ ] Build
[x] CI
[ ] Documentation
[ ] Performance
[ ] Refactor
[ ] Style
[ ] Unit tests
[ ] Subsystem tests

Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/test ?

/test rosa-hcp-advanced-optional-presubmit

/lgtm
/approve

/approve

/test rosa-hcp-private-critical-high-presubmit

/test rosa-hcp-advanced-optional-presubmit

/test rosa-hcp-advanced-optional-presubmit


Tools can only be repaired by combining eachother.

So Is it In A Anvil Or Not?

You can only repair the tools in the mod by combining them in a crafting table or anvil.

Yea That's Normal

So what is the problem?


no such package '@tflm_pip_deps_tensorflow_intel//'

Trying to run the hello_world example with bazel ("bazel build tensorflow/lite/micro/examples/hello_world:evaluate")
I get the following error:
ERROR: C:/users/userid/appdata/roaming/spb_data/_bazel_userid/zm73hpx2/external/tflm_pip_deps_tensorflow_cpu/BUILD.bazel:22:11: no such package '@tflm_pip_deps_tensorflow_intel//': The repository '@tflm_pip_deps_tensorflow_intel' could not be resolved: Repository '@tflm_pip_deps_tensorflow_intel' is not defined and referenced by '@tflm_pip_deps_tensorflow_cpu//:pkg'
Environment:
Windows 10
Bazel 6.4.0
Python 3.10.10
Windows cmd prompt

I wasn't able to reproduce this. Have you modified the python_requirements.txt file? Perhaps just a transient pypi download issue?

I just tried that on another Windows 10 machine with Bazel 6.4.0 and got the exact same error.  I have not modified the requirements. I downloaded the bazel-6.4.0-windows-x86_64.exe, renamed it to bazel and added to PATH. Installed the https://aka.ms/vs/17/release/vc_redist.x64.exe Microsoft Visual C++ Redistributable. Cloned the tflite-micro repo. Open the cmd prompt. Typed bazel build tensorflow/lite/micro/examples/hello_world:evaluate and got the exact same error.
I also tried installing the python requirements and running the evaluate.py script manually but there are other package problems there. For example the script imports matplotlib but matplotlib is not in the requirements, etc

Solved this issue by modifying the requirements:
In python_requirements.txt changed tensorflow-io-gcs-filesystem==0.32.0  to version 0.31.0 because there is no wheel for windows and version 32 at pypi.
Also added
tensorflow-intel==2.12.0
--hash=sha256:816a6b9018d1ae0defe94508aab2512228fb6a54ab51f5f4f025c857cfe5c7e5
# via tensorflow-cpu
as this is a dependency for tensorflow-cpu.
I build the hello_world:train target successfully.
But then when I try to build hello_world:evaluate target I get a new error "ERROR: An error occurred during the fetch of repository 'local_config_python':".
Correct me if I am wrong but this repo needs many modifications to work on windows.

I run into a similar problem as you, and there is no good solution yet.
I have tried to use Linux System to solve this problem, but I failed...I suspect the official documentation is out of date！！！！
If you have any problemes, Please update in here! I'm sure this will help a lot of people with similar problems.

I sloved this problem. Let me tell you how to slove it!!!
First, you must create a new virtual env by using conda:
conda create -n myenv python=3.10.0
It  is import that the version of python must be 3.10!!!!! If your python version is not 3.10, then it will definitely fail!！
secondly, you must install bazel 7.0.
Finally, don't install any packages, just run the command:
bazel build tensorflow/lite/micro/examples/micro_speech:audio_preprocessor
(you can instead this command for yours~)
Now,  everything is ok!!!

Tried in Ubuntu jellyfish and everything builds fine.

Hi,This is CannoChen.I have received your mail.And then, I will reply to you later.Thank you!

Solved this issue by modifying the requirements: In python_requirements.txt changed tensorflow-io-gcs-filesystem==0.32.0 to version 0.31.0 because there is no wheel for windows and version 32 at pypi. Also added tensorflow-intel==2.12.0 --hash=sha256:816a6b9018d1ae0defe94508aab2512228fb6a54ab51f5f4f025c857cfe5c7e5 # via tensorflow-cpu as this is a dependency for tensorflow-cpu. I build the hello_world:train target successfully. But then when I try to build hello_world:evaluate target I get a new error "ERROR: An error occurred during the fetch of repository 'local_config_python':". Correct me if I am wrong but this repo needs many modifications to work on windows.

That's really work!

Solved this issue by modifying the requirements: In python_requirements.txt changed tensorflow-io-gcs-filesystem==0.32.0 to version 0.31.0 because there is no wheel for windows and version 32 at pypi. Also added tensorflow-intel==2.12.0 --hash=sha256:816a6b9018d1ae0defe94508aab2512228fb6a54ab51f5f4f025c857cfe5c7e5 # via tensorflow-cpu as this is a dependency for tensorflow-cpu. I build the hello_world:train target successfully. But then when I try to build hello_world:evaluate target I get a new error "ERROR: An error occurred during the fetch of repository 'local_config_python':". Correct me if I am wrong but this repo needs many modifications to work on windows.

That's really work!

Well, unfortunately, I got another issue when I was trying to "bazel-bin" the .exe file. Something like this:
E:\tinyML\tflite-micro>bazel-bin\tensorflow\lite\micro\examples\hello_world\train --save_tf_model --save_dir=\tmp\model_created\ Traceback (most recent call last):   File "<frozen runpy>", line 198, in _run_module_as_main   File "<frozen runpy>", line 88, in _run_code   File "E:\tinyML\tflite-micro\bazel-bin\tensorflow\lite\micro\examples\hello_world\train.zip\__main__.py", line 559, in <module>   File "E:\tinyML\tflite-micro\bazel-bin\tensorflow\lite\micro\examples\hello_world\train.zip\__main__.py", line 454, in Main   File "E:\tinyML\tflite-micro\bazel-bin\tensorflow\lite\micro\examples\hello_world\train.zip\__main__.py", line 204, in CreateModuleSpace   File "E:\tinyML\tflite-micro\bazel-bin\tensorflow\lite\micro\examples\hello_world\train.zip\__main__.py", line 190, in ExtractZip   File "D:\Python\Python311\Lib\zipfile.py", line 1660, in extract     return self._extract_member(member, path, pwd)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "D:\Python\Python311\Lib\zipfile.py", line 1731, in _extract_member     open(targetpath, "wb") as target:     ^^^^^^^^^^^^^^^^^^^^^^ FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\MyName\\AppData\\Local\\Temp\\Bazel.runfiles_93cfifk4\\runfiles\\tflm_pip_deps_tensorflow_intel\\site-packages\\tensorflow\\include\\external\\llvm-project\\mlir\\_virtual_includes\\AffineMemoryOpInterfacesIncGen\\mlir\\Dialect\\Affine\\IR\\AffineMemoryOpInterfaces.cpp.inc'

Hi,This is CannoChen.I have received your mail.And then, I will reply to you later.Thank you!

Hi,This is CannoChen.I have received your mail.And then, I will reply to you later.Thank you!

Hi,This is CannoChen.I have received your mail.And then, I will reply to you later.Thank you!

Hi,This is CannoChen.I have received your mail.And then, I will reply to you later.Thank you!

Hi,This is CannoChen.I have received your mail.And then, I will reply to you later.Thank you!


How to get all elements of the suggestions list as suggestion

First of all, thank you very much for this library.
Just a little question.
How can i get all elements of the suggestions list as suggestion when the input text is empty (for example when first tap on the input area) ?

Same question here, did you resolve this? Does this library provide this feature?

Maybe adding a property to allow users to display all entries when the filter is empty
bool displaySuggestionsIfEmpty = false;

List<T> getSuggestions(List<T> suggestions, Comparator<T> sorter,
                        Filter<T> filter, int maxAmount, String query) {

    if (query == "") {
        if(this.displaySuggestionsIfEmpty){
            return suggestions;
        }
        return [];
    }
    suggestions.sort(sorter);
    suggestions = suggestions.where((item) => filter(item, query)).toList();
    if (suggestions.length > maxAmount) {
        suggestions = suggestions.sublist(0, maxAmount);
    }
    return suggestions;
}

This should be able to be done with the itemFilter param, something like
`
itemFilter: (suggestion, input) {
return   suggestion.name.toLowerCase().startsWith(input.toLowerCase()) || input == "";
}
`


Hashtable check - quick fix for LF end of line

Pull Request (PR) description
This PR extends the Measure-Hashtable function to cover LF end of line.
This Pull Request (PR) fixes the following issues
None
Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Documentation added/updated in README.md.
[ ] Comment-based help added/updated for all new/changed functions.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Unit tests added/updated.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

There is something up with the Reviewable link for this PR so I'm going to do this via GitHub review.

:lgtm:


Add krb5_get_renewed_creds() and krb5_get_validated_creds()

I updated the PR to include a testcase for krb5_get_validated_creds(), and now the tests on MacOS seem to be failing.
From what I can see the problem is that:

heimdal under Linux has an implementation for krb5_get_validated_creds(), but does not export it (it is not listed in https://github.com/heimdal/heimdal/blob/1b62220778c570f27e25db6eac14229b1769775a/lib/krb5/version-script.map), therefore the _creds_mit extension will not be built.
heimdal on MacOS seems to export krb5_get_validated_creds(), therefore the _creds_mit extension will be built (see e.g. the logs in https://github.com/jborean93/pykrb5/actions/runs/8269018140/job/22623359479?pr=40).
However, the heimdal implementation of krb5_get_validated_creds() seems to be broken / doing something completely different from the MIT implementation, see https://github.com/heimdal/heimdal/blob/1b62220778c570f27e25db6eac14229b1769775a/lib/krb5/verify_init.c#L217: It tries to use the creds argument, which according to https://web.mit.edu/kerberos/krb5-1.15/doc/appdev/refs/api/krb5_verify_init_creds.html is an output argument, completely ignores ccache, and then calls krb5_verify_init_creds() (which will try to do a verification of the credentials against the default keytab, which has nothing to do with asking the KDC to validate a postdated ticket). Because the creds argument is used as an input argument but is set to NULL, this triggers a segmentation fault: (I reproduced this on Linux by exporting krb5_get_validated_creds from heimdal libkrb5)

(gdb) where
#0  0x00007ffff74c174b in krb5_realm_compare (context=context@entry=0x1054ad0,
    princ1=princ1@entry=0x0, princ2=princ2@entry=0x1058960) at principal.c:1013
#1  0x00007ffff74c1782 in krb5_principal_compare (
    context=context@entry=0x1054ad0, princ1=0x0, princ2=0x1058960)
    at principal.c:991
#2  0x00007ffff74d1ca9 in krb5_get_validated_creds (context=0x1054ad0,
    creds=creds@entry=0x7ffff761dd98, client=<optimized out>,
    ccache=<optimized out>, service=service@entry=0x0) at verify_init.c:228
#3  0x00007ffff6adc9af in __pyx_pf_4krb5_10_creds_mit_get_validated_creds (
    __pyx_self=<optimized out>, __pyx_v_in_tkt_service=...,
    __pyx_v_ccache=0x7ffff75bcbe0, __pyx_v_client=0x7ffff75c4200,
    __pyx_v_context=0x7ffff773ae10) at src/krb5/_creds_mit.c:2670

So I'll change the PR to never include krb5_get_validated_creds with heimdal, even if it is available.

So I'll change the PR to never include krb5_get_validated_creds with heimdal, even if it is available.

Makes sense, I probably prefer to favour the MIT implementation anyway.


rewrite threading system so there is a max number of goroutines

This will help against goroutines building up and all running at the same time, killing the server.

this still needs testing on servers with 400+ domains, as well s thy --delay functionality still needs to be added from here: https://github.com/Liamraystanley/marill/issues/22

kicked up about 250 wordpress installs (and 20 other misc. erronous/working sites) on a 1gb, single core vps, and with the new setting, load wasn't too bad, esp. compared to before. though, a default delay of 1 second may be worth while still


PatternFly UI fixes to workbench

+1

Exact duplicate of https://github.com/droolsjbpm/optaplanner-wb/pull/23 - closing this one so the other one can live :)

@ge0ffrey this is a PR for different branch (master), the other one is for 6.4.x. I believe we need both (assuming the change should land on 6.4.x as well).

Reopened
@psiroky Thanks Petr!
@cristianonicolai Ignore my comment above.

Merged, thanks Cristiano


I'm stuck. Whats wrongs

Challenge Create a Bulleted Unordered List has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

<u1>

milk

cheese

scratch post

</u1>

Thanks for reporting. But your code is incorrect.
Please consult Help Chat Room for assistance.
Happy coding!


Access to path is denied for writing feature.*

Hey everyone, I'm on my work laptop and trying to execute the initial setup for pickles.  However, I'm sure there is some security policy preventing the command to execute:
Something went wrong during generation: System.UnauthorizedAccessException: Access to the path 'C:\pathtofeatures\features.docx' is denied.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize)
at System.IO.Abstractions.FileWrapper.Create(String path)
at PicklesDoc.Pickles.DocumentationBuilders.Word.WordDocumentationBuilder.Build(Tree features)
at PicklesDoc.Pickles.Runner.Run(IContainer container) Access to the path 'C:\GeneratedDocs\features.docx' is denied.
Pickles did not complete successfully Access to the path 'C:\pathtofeatures\features.docx' is denied.
Is there any other way to execute to generate these files?

Hi,
One question first: do you perhaps have that file open in Word? Word blocks access to the files it's working on.
If not: you could try generating the file to a different path - for example a location where you can save Word files using Word.

Thanks for the quick reply, unfortunately, no word isn't open.  I've even gone as far as restarting the machine and not having anything open.  I've also tried to change the file path and launch with the most elevated privileges we get.  I got with our desktop support and had them run as admin, and same error.  I know this is likely environmental here, but just didn't know if there were any alternatives.  We're using SpecFlow with Gherkin and NUnit.  Do you think if I used the Pickles.MSBuild it'd help?  Or is it just executing the same command under the hood?

All 4 runners basically run the same command under the hood ... you could try one of the other runners (like the MSBuild runner) but I'm not optimistic :-(

Thanks again for the quick replies, I'm about to close this out.  After experimenting with some PowerShell scripting, I was able to get it to work.  Cheers

I'm glad you got it to work!

Thanks!  I was getting a little worried too, as this looks promising.

No, this is the only forum for questions.
I’m afraid I will be off-line for the next week and a half, so I won’t be able to answer other questions.
But by all means, give Pickles a good spin, and if you notice anything that can be improved, feel free to create a ticket - or even better a pull request ;-)

Sounds great!  I just mainly didn't want to clutter your stuff.


Revert "Expose Variant operator= to GDNative"

Reverts godotengine/godot#54508
See the PR for rationale. The PR can be redone (it's a good change) with the proper structure and cherry-picked again for 3.4.

Cherry-picked for 3.4.1.


can't show events list in macOS Sierra

Hi, I just downloaded it yesterday. Adding events via itsycal is cool! However, I can't see the events added on ltsycal bar. So, I also can't delete an events. I don't know if it is caused by the new macOS Sierra I use. BTW, I have chosen the "Events list show" in preferences. Here is the situation, having events in calendar app but not in itsycal bar

@Angryrou,
I am running Sierra so I don't think that is the issue. Some questions:

When you first launched Itsycal, did you give it permission to access your calendars?
Do you have the calendars you want to see checked off in Itsycal's preferences?
Thanks.

@sfsam ,
I forgot to check on the categories, seems it is off by default. It works now, THANKS!


Staging

Q
A

content fix?
yes

What's in this Pull Request?
Updated SharePoint Framework Toolchain doc with more information around build tasks and fixed formatting issues.

Hi @chakkaradeep, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Chakkaradeep (Chaks) Chinnakonda Chandran). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Schedule API stopped working after runtime 1.14 rc.2

Scheduling a job stopped working on 1.14-rc.3.
I've tried the two curl commands below. they both return the same error:
  http://localhost:6280/v1.0-alpha1/jobs/r2-d2 \
  -H "Content-Type: application/json" \
  -d '{
  "job": {
    "data": {
      "@type": "type.googleapis.com/google.protobuf.StringValue",
      "value": "R2-D2:Oil Change"
    },
    "dueTime": "2s",
    "schedule": "@every 5s"
  }
}

  http://localhost:6280/v1.0-alpha1/jobs/r2-d2 \
  -H "Content-Type: application/json" \
  -d '{
  "job": {
    "data": {
      "@type": "type.googleapis.com/google.protobuf.StringValue",
      "value": "R2-D2:Oil Change"
    },
    "dueTime": "2s",
  }
}

In what area(s)?

/area scheduler

/area runtime

/area operator

/area placement

/area docs

/area test-and-release

What version of Dapr?

1.14-rc.2

1.1.x
1.0.x
edge: output of git describe --dirty

Expected Behavior

The job should be scheduled successfully.
Actual Behavior

Error:
{"errorCode":"SCHEDULER_SCHEDULE_EMPTY","message":"Schedule is empty","details":[{"@type":"type.googleapis.com/google.rpc.ErrorInfo","domain":"dapr.io","metadata":{"appID":"job-service","namespace":"default"},"reason":"SCHEDULER_SCHEDULE_EMPTY"}]}

Steps to Reproduce the Problem

On:
CLI version: 1.14.0-rc.6
Runtime version: 1.14.0-rc.3
Create a simple Dapr http server and schedule a job using the example in the docs:
$ curl -X POST \
  http://localhost:3500/v1.0-alpha1/jobs/jobforjabba \
  -H "Content-Type: application/json"
  -d '{
        "job": {
            "data": {
                "HanSolo": "Running spice"
            },
            "dueTime": "30s"
        }
    }'

Release Note

RELEASE NOTE:

This is intentional. It is because we made the HTTP API flat. Put all the attributes from job structure directly into the root structure.


Allow deprecating input fields and arguments

This is PR for the issue #110

@TaProhm thanks so much for this 🙏🏻

@spawnia Thanks for your review, I've made corrections according to your suggestions and fixed the failing test.


Install/Setup content

@jdeoliveira , should this install page contain the link https://login.salesforce.com/packaging/installPackage.apexp?p0=04t4P000002ErbpQAC, or say "the package update installation link provided by your MuleSoft representative"?

PTAL at the 'Communities' and 'Users' sections - I think the headings should be nested (they appear to be at the same level as the overall section heading), but I'll defer to you. Thanks!

'Communities' and 'Users' should be at level 2, under 'Configure API Community Manager', then? In that case the document is 4 levels deep, and lines 49-101 have the first two sections, while lines 103 through 404 are part of the third section. This is a deeper and more uneven division of a file than we usually have, but it is long and complex material, and the content may dictate that organization pattern. As long as you're sure, it's good with me.

Added the additional nesting to avoid confusion.

@wwarner-salesforce - PTAL - thank you!

Looks great! I built locally and am making changes now to fix formatting errors.

I think line 272 is a bit redundant - do we need an intro sentence (I think the title is pretty clear, but perhaps our style guidelines dictate this)?

I do always try to have some text in each section before the subsections start, but I don't think the style guide dictates it, so I'm ok with removing 272.

I just did a local build to check, and the link "Enable Communities Inside Your Organization" causes the new tab to pop open when you click that item in the TOC, so I'm going to modify that slightly.

Jesus confirmed today that we should refer customers to their Salesforce rep and not link to the installation package.
I'm removing line 272.
Currently we have "Set Up Your API Community" with no content and two sub-pages: "Installation Pre-requisites" and "Installation".  So "Installation" already has almost all the info on these three pages, and I'm fixing that confusing structure by replacing the three with the one "Installation" page. I'm happy to modify that later if needed.

I'm going to go ahead and merge this PR with the manual line numbers on certain section headers, but I look forward to moving to the better :sectnums: and :sectnums!: system once Fernando approves.


In array fix

Hi Sam
Here is a pr for the problem with has key instead of in_array

Resolved in tag 4.11.0


spark_read_json prohibitively slow with many json files

@kevinushey @kevinykuo I have about 40 very large json files (each is about 100GB) and I try to load them using
spark_read_json(mypath/files_json*, memory = FALSE)
However, no matter what my cluster allocation is (number of executors, number of cores, allocated RAM), the reading spawns 500k tasks and takes forever. I have the impression only one executor is trying to map all the data while this is something that can easily be parallelized.
Some files can fit into the RAM of one cluster, but not all of them (some of them are much larger than the RAM available on a single node)
Any idea what read_spark_json is doing here?

@randomgambit 40 files isn't very many so I suspect this is due to schema inference; the reader will iterate through your data even if you're not caching it in memory. We currently don't have a very friendly way to pass a schema to spark_read_json(), though it can be done. Would you be able to provide a sanitized sample of your json with the relevant structure (e.g. nested or not)? Looking at some use cases will help us shape the features.

@kevinykuo do you mean there is a workaround right now? I would be really interested to know it. I tried to convert my json to parquet, but this is a pain.
The json are really simple., There is only one field that is nested (it contains the longitude-latitude). There are about 15 fields including strings and numbers.

@kevinykuo is the workaround similar to the one in http://spark.rstudio.com/example-s3.html
like
flights <- spark_read_csv(sc, "flights_spark",
                          path =  "s3a://flights-data/full",
                          memory = TRUE,
                          columns = list(
                            Year = "character",
                            Month = "character",
                            DayofMonth = "character"

Do you think that will seriously cut the number of tasks generated?

@randomgambit yeah but it's gonna be a bit more involved. The csv case is more well supported (in fact you can just pass infer_schema = FALSE). For json right now I think you'd have to get hacky and do something like
string_type <- invoke_new(sc, "org.apache.spark.sql.types.StringType")
integer_type <- invoke_new(sc, "org.apache.spark.sql.types.IntegerType")
metadata <- invoke_new(sc, "org.apache.spark.sql.types.MetadataBuilder") %>%
  invoke("build")
struct_field_1 <- invoke_new(
  sc,
  "org.apache.spark.sql.types.StructField",
  "my_string_column",
  string_type,
  ensure_scalar_boolean(TRUE),
  metadata
)

struct_field_2 <- invoke_new(
  sc,
  "org.apache.spark.sql.types.StructField",
  "my_integer_column",
  integer_type,
  ensure_scalar_boolean(TRUE),
  metadata
)

my_struct_type <- invoke_new(sc, "org.apache.spark.sql.types.StructType") %>%
  invoke("add", struct_field_1) %>%
  invoke("add", struct_field_2)
my_struct_type
# <jobj[681]>
#   class org.apache.spark.sql.types.StructType
# StructType(StructField(my_string_column,org.apache.spark.sql.types.StringType@37671448,true), StructField(my_integer_column,org.apache.spark.sql.types.IntegerType@26cce37a,true))

then pass my_struct_type to spark_read_json() via options.
I think we'd want an interface closer to spark_read_csv(), but we need to think about how it's gonna work with nested stuff.
@javierluraschi is this a fair assessment of the current state or am I missing something?

@kevinykuo @javierluraschi woooooot this is not what I call a workaround!! :D

I'll take a look, we added infer_schema = FALSE for sparklyr 0.6 so setting the columns parameter should be enough...

@javierluraschi I just wonder how you can specify the variable type for nested variables?

@randomgambit could you update sparklyr with devtools::instsall_github("rstudio/sparklyr") followed by using the new columns parameter? Something like:
spark_read_json(
  "mypath/files_json*",
  memory = FALSE,
  columns = list(col1 = "character", col2 = "character")
)

setting them all as "character" is fine here since we are interested in troubleshooting the performance issue you are reporting more than getting the right types.


More thumbnail work - images already in project.

If we have an image already in a project, we need to load it a second time to use it as a thumbnail (see water-use-15).

The thumbnail would be the same resolution as the image in the viz?

Yeah, facebook and twitter actually allow for pretty big images. It's mainly our "landing" page that needs a very specific sized one.


New Changes

Added my Github Profile

@Frinlanz  thnx for contributing, try this year, and make sure you follow me and star this repo for updates about special swags I will be giving.
Contribute in October for Hacktoberfest 2020


Bootstrap 3.3.4 tootlip-html-unsave deprecated

After an CSS-Update to bootstrap 3.3.4. I got a warning, that the use of tooltip-html-unsafe is deprecated and that I should use tooltip-html instead.
Then I tried to use tootlip-html but nothing happens.
I use Bootstrap CSS 3.3.4 and angular-ui/bootstrap v0.13.0

tooltip-html takes an Angular expression as it's value.
And upgrading your Twitter Bootstrap CSS dependency couldn't have produced that deprecated warning.  Even though this seems to be a support request, provide a plunkr and I'll take a look.

The warning is new as of UI Bootstrap 0.13.0 - this is because its usage is now deprecated, and it is recommended to use the tooltip-html or tooltip-template directives instead.

Closing as a support issue, feel free to provide more information if you'd like to have help seeing any problems & the fixes for them.


[Feature] SDK Python testcases needed to be commissioned again.

Feature description
As per the latest image : Version: 2.10-beta2+spring2024  SDK Yaml testcases and SDK python testcases need to commissioned seperately  for the successful execution.
Here the Example if the both are executed in a single test , the python tet cases is fails with the timeout error

Failed to establish secure session to device: src/controller/python/ChipDeviceController-ScriptPairingDeviceDiscoveryDelegate.h:59: CHIP Error 0x00000032: Timeout
Failed to commission: src/controller/python/ChipDeviceController-ScriptPairingDeviceDiscoveryDelegate.h:59: CHIP Error 0x00000032: Timeout
[MatterTest] 01-12 11:27:07.710 INFO [Test] test_run_commissioning FAIL

Reference log:
yaml&py.log
Use Cases
By implementing this feature the whole test suite canbe run in the single commissioning process and does not require the interaction for advertising the dut.
Note: if this feature is not vaild please feel free toclose this issue

Hi @Saravana-kr22,
I’m not 100% sure, but I believe that you are suggesting to keep the commissioning between YAML and Python tests.
If that’s the case, then I can say that it’s something that we have considered but, since the YAML and Python tests use different controllers, that’s not possible. The YAML tests use the chip-tool and chip-app binaries in interactive mode and the Python test scripts are executed directly, not using this binaries. The ways that the binaries and the Pyhton scripts store the commissioning information are not the same, so the pairing are not shared between them.
Maybe we can suppress the need for the user prompt by opening a commissioning window at the end of a suite execution and then performing the new commissioning automatically, but we’re not 100% sure if or how that would work, we’ll have to look into that.
If the suggestion was related to running multiple Python tests one after the other without having to re-commission the device, then that’s what we aimed to do by diving those tests in separate test suites. However, some Python tests expects the DUT to be commissioned and others expect it to not be commissioned and TH needs to know which are which. In order for that to happen, TH expects the scripts to follow a certain template that indicates whether the DUT should be commissioned or not. So unfortunately, until all Python tests are updated to follow this template, we must ask the user which test needs commissioning and which one doesn’t.

Also, regarding the timeout error: that can be avoided by reseting the DUT one you see the "Make sure the DUT is in Commissioning Mode" pop-up. Only after it has been reset do you select DONE and submit the answer.

This issue is resolved in latest TH image.


Add Steps component to v1.mdx

Add Steps component to v1.mdx
Adds the Step component where needed.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/upgrade-to/v1.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

!coauthor


Bad example

The naming used in the following example from this page makes it confusing to understand.
public class Generic
{
public T Field;
}
The topic is generics, but naming the sample class "Generic" is confusing.  Some may interpret it to mean that all generic classes must be named "Generic".
Instead, much clearer is to use arbitrary names such as:
public class foo
{
public T bar;
}
or something like:
public class myFileReader
{
public T readLine;
}
Sincerely
Phil
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0fc82239-30f3-14d2-ed23-5c0b66bd370e
Version Independent ID: 26fccd46-c489-1a05-2da2-dda3a8e5fac8
Content: Generics in .NET
Content Source: docs/standard/generics/index.md
Product: dotnet-fundamentals
GitHub Login: @adegeo
Microsoft Alias: adegeo

@pshisbey I agree Generic is unnecessarily ambiguous. foo and bar are specifically forbidden. myFileReader is too specific. I think  the following would be better:
public class MyGeneric<T>
{
    public T MyField;
}

@gewarren I would like to work on this - please assign me so I have it on my list.


Make informational prints go to stderr

Currently, Dataset prints out
Processing...
Done!

when processing a dataset. This is useful feedback for the user (usually), but goes to stdout, corrupting the output of scripts that depend on it. (I am trying to build a UNIX-style script where messages go to stderr and only the machine readable output goes to stdout for easy piping into a file or other tool.)
This PR switches those print statements to print to stderr instead. Two other options are:

Use Python's built-in logging to give the user full control of these messages
Do not emit these updates at all

Searching, this seems to be the only place in the code that uses print.

Codecov Report

Merging #2950 (c887518) into master (33d03c4) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2950      +/-   ##
==========================================
- Coverage   84.32%   84.27%   -0.05%
==========================================
  Files         210      210
  Lines        9593     9566      -27
==========================================
- Hits         8089     8062      -27
  Misses       1504     1504

Impacted Files
Coverage Δ

torch_geometric/data/dataset.py
95.12% <100.00%> (+0.03%)
:arrow_up:

torch_geometric/transforms/base_transform.py
75.00% <0.00%> (-5.00%)
:arrow_down:

torch_geometric/data/temporal.py
28.57% <0.00%> (-2.68%)
:arrow_down:

torch_geometric/utils/homophily.py
85.71% <0.00%> (-1.25%)
:arrow_down:

torch_geometric/data/data.py
91.45% <0.00%> (-0.37%)
:arrow_down:

torch_geometric/transforms/gdc.py
78.35% <0.00%> (-0.23%)
:arrow_down:

torch_geometric/data/graph_saint.py
97.97% <0.00%> (-0.05%)
:arrow_down:

torch_geometric/nn/models/gnn_explainer.py
95.75% <0.00%> (-0.03%)
:arrow_down:

torch_geometric/nn/conv/rgcn_conv.py
98.51% <0.00%> (-0.03%)
:arrow_down:

torch_geometric/nn/models/re_net.py
100.00% <0.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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

Thanks! I think I'm in favor of doing this via logging, as outputing to stderr may also have undesired side-effects.

@rusty1s updated the PR, agree that logging is the most proper way to do it

While we are at it, feel free to also make adjustments to data/extract.py and data/download.py.

^

Thanks. I actually wasn't aware that logger.info does not print out anything by default (only with logging.basicConfig(level=logging.DEBUG)). Is it possible to print out logging information by default, and only silence logging in case the user wants to (instead of the other way round?)

I don't think so, since my understanding is that the whole point of the architecture is to prevent libraries from circumventing the users settings on the root logger, giving the host application full control...
Obviously, changing the root logger's settings from a library would be very bad practice.
You could of course circumvent logging entirely and have your own torch_geometric.logging_on option of some kind, but that seems like a lot of code infrastructure for a couple of mostly unimportant informational messages.

Yes, I understand. Changing root logger settings is definitely something we do not want. As I think the print information is quite useful for the average user, it might be a bit weird to disable them by default. If you do not mind and it fits your use-case, I would very much appreciate it if you can change the behavior to print(..., file=sys.stderr) once again. Sorry for being so unexperienced with the logging module and for stealing your time.

@rusty1s no worries, and as always we appreciate your responsiveness when little issues come up!
I reverted (a bit messily, couldn't figure out the git commands) to the print statements and changed all others I found in the codebase to go to stderr.
While I do still think it would be best not to put any unexpected output anywhere by using logging, I agree with your assessment that this might be surprising for the majority of users of the code. Most people probably use torch_geometric in a "Jupyter notebooks and hacked together data science scripts" context, which wants responsive updates on what's happening, and not "tools with well defined output" like we are. At least by using stderr we avoid corrupting stdout data outputs :+1:


income basis calculation

Hi
this is great, I'm looking for a solution to calculate tax on a income basis rather than capital gains,
would be great to have an option for that!
thanks

The project does not calculate the tax owed. It just use the trade matching rules to calculate taxable income/loss.
Income basis not sure if it is the same. Please check the tax law.


Add custom value and link for GITHUB_RUN_ID

Also renamed the existing workflow search link to be consistent with other CI systems. So:

GITHUB_WORKFLOW => CI Workflow build scans
GITHUB_RUN_ID => CI Run build scans

@bigdaz Why not done for Maven, too? And why no review by someone else?

Both of those are very good points. This got mixed up with my other fix for the build scan links, and I thought it was innocent enough to "just merge" and get it out of the way.
Clearly review from both myself and someone else is important :). Apologies.

LGTM


Invalid node version in .nvmrc: vXX

Build/Submit details page URL
No response
Summary
As of<EMAIL_ADDRESS>the following command has started failing in our environment:
eas build -p ios --profile test --non-interactive --no-wait

With the following error:
Invalid node version in .nvmrc: v16

Contents of our .nvmrc file:
v16

Managed or bare?
Managed
Environment
expo-env-info 1.0.5 environment info:
    System:
      OS: macOS 13.4.1
      Shell: 5.9 - /bin/zsh
    Binaries:
      Node: 16.20.0 - ~/.nvm/versions/node/v16.20.0/bin/node
      Yarn: 1.22.19 - ~/.nvm/versions/node/v16.20.0/bin/yarn
      npm: 8.19.4 - ~/.nvm/versions/node/v16.20.0/bin/npm
      Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
    Managers:
      CocoaPods: 1.12.1 - /opt/homebrew/bin/pod
    SDKs:
      iOS SDK:
        Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
    IDEs:
      Android Studio: 2022.2 AI-222.4459.24.2221.10121639
      Xcode: 14.3.1/14E300c - /usr/bin/xcodebuild
    npmPackages:
      expo: ^48.0.4 => 48.0.4
      react: 18.2.0 => 18.2.0
      react-dom: 18.2.0 => 18.2.0
      react-native: 0.71.3 => 0.71.3
      react-native-web: ~0.18.9 => 0.18.10

Error output
Invalid node version in .nvmrc: v16
Reproducible demo or steps to reproduce from a blank project

Add a .nvmrc file to your project with the following contents:

v16

Make sure eas.config does not contain a node version number

Run EAS build:

eas build -p ios

It seems like this is the offending line: https://github.com/expo/eas-cli/blob/47968a1b8b2cf80e19bd9fb695a05e851d7d6326/packages/eas-cli/src/utils/profiles.ts#L86
semver is used to validate the version, however .nvmrc has a more liberal format and the value should probably be coerced using semver.coerce first.

Thank you very much for reporting this!
I will revert https://github.com/expo/eas-cli/pull/1954 and publish a new version to make sure that no more builds are broken.
The semver.coerce will do the job, although I tested it and we need to make additional changes on our backend to support versions like just v16 to make sure that everything works properly, so for now, I will revert it.
We will re-add this functionality once everything works properly.

Just published version 3.18.2 which reverted this change

@szdziedzic
Issue seems to be back on our side
Since yesterday our pipeline now failing due to eas version 5.0.0
eas build -p android --profile staging --non-interactive
Error
Invalid node version in .nvmrc: lts/hydrogen
    Error: build command failed.

This PR should have fixed this issue ⬆


Implement ssz proof merging

Builds on #145
What was wrong?
When we request content in the Alexandria network, we get it in pieces.  We need a way to merge these pieces into a single unit.
How was it fixed?
Quick and dirty implementation of Proof.merge which combines the two proofs into a single new proof, dropping any elements from the proof that are duplicated or no longer necessary (such as intermediate tree nodes for which we have all of the underlying leaves.
Cute Animal Picture

This has been rebased.  @marcgarreau and @lithp do either of you have anything additional you'd like me to cleanup?


How to generate Test report ?

Hi Team,
How to generate Test report ?
I want to see my all test cases in a html file report like total test cases run, failed or how many pass etc.
Please help me how to generate such html file with example.

I did a blog post which shows how to set up Extent Reports with an EasyRepro project.

Hi, Can you give me access to read your blog. This is my email<EMAIL_ADDRESS>Thanks

I did a blog post which shows how to set up Extent Reports with an EasyRepro project.

Please give access to read you blog -<EMAIL_ADDRESS>

I did a blog post which shows how to set up Extent Reports with an EasyRepro project.

Can you please provide me access to read our blog for mail ID-<EMAIL_ADDRESS>

@jal looks like your blog is private.

I got this working, the details are here


Allow to delete last bucket

Environment info

NooBaa Version: VERSION
Platform: Kubernetes 1.14.1 | minikube 1.1.1 | OpenShift 4.1 | other: specify

Actual behavior

See discussion in #860
Deleting the last bucket from the operator/CLI fails with an error:

Code=LAST_BUCKET
Message=Cannot delete bucket

Expected behavior

Deleting last bucket should be allowed.

Steps to reproduce

Use the noobaa cli:

. devenv.sh
nb install
nb bucket delete first.bucket

More information - Screenshots / Logs / Other output
$ nb bucket delete first.bucket
INFO[0000] ✅ Exists: NooBaa "noobaa"
INFO[0000] ✅ Exists: StatefulSet "noobaa-core"
INFO[0000] ✅ Exists: Service "noobaa-mgmt"
INFO[0000] ✅ Exists: Service "s3"
INFO[0000] ✅ Exists: Secret "noobaa-server"
INFO[0000] ✅ Exists: Secret "noobaa-operator"
INFO[0000] ✅ Exists: Secret "noobaa-admin"
INFO[0000] ✈  RPC: bucket.delete_bucket() Request: nb.DeleteBucketParams{Name:"first.bucket"}
ERRO[0000] ⚠ RPC: bucket.delete_bucket() Response Error: Code=LAST_BUCKET Message=Cannot delete bucket
PANI[0000] ☠  Panic Attack: [] Cannot delete bucket

Main thing would be to verify we don't have flows which assume there is at least 1 bucket (same in the FE)


PF-577 Modified readme and comments in config.ini

@ChuckSpringer-AI ,
I've made changes in Readme and config.ini files based on our discussion this morning.
One thing I want to correct is that ArchiveWhenFileNumberIsLargerThan is actually required.
Can you take a look?
Thanks,
Bill

@DougSchmidt-AI , can you merge this? The project team has reviewed this and believed this is good to go. Thanks!


Browse All Page - Reconstruction Method Filter Typo

Reconstruction Method filter:

Expected behavior: 3 dropdown options "Weighted backprojection", "SART", "SIRT"

Current behavior: "FOURIER SPACE", "SIRT" "Weighted backprojection", "Weighted backprojection;SIRT"

Screenshot:

@uermel can you confirm if "FOURIER SPACE" is supposed to be a dropdown option? i see one dataset that lists it as the reconstruction method in the metadata.
Also, can there be multiple reconstruction methods e.g., "Weighted backprojection; SIRT"?

@uermel
i'm not sure if we're keeping this metadata schema table up to date but if we are, would you be able to add in fourier space?
It looks like "Weighted backprojection; SIRT" was listed as a tomogram reconstruction method hence it showing up as a filter dropdown option. Is this something that needs to be corrected in the metadata then?

Yes, I can add that to the metadata table. This particular case does look like a metadata issue in case of the combined methods. @anchi2c Is there a reason you added two different reconstruction methods for these tomograms?

With the new json being generated by Anchi, we will use that to regenerate the tomogram config. The weighted back tomogram should be used as the Canonical Tomogram.

I pushed https://github.com/czimaginginstitute/czii-data-portal-processing/commit/c95da43abfc0150128c0551f20f8ef07fd1e7b53 to fix item 2. @manasaV3 this is ready for you.

The bug has been fixed.


meta/tkv: fix clean stale session

Codecov Report

Merging #1041 (30729e0) into main (f593dbf) will increase coverage by 0.05%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##               main    #1041      +/-   ##
============================================
+ Coverage     45.42%   45.47%   +0.05%
  Complexity      149      149
============================================
  Files           146      146
  Lines         23776    23780       +4
  Branches        422      422
============================================
+ Hits          10800    10815      +15
+ Misses        11639    11627      -12
- Partials       1337     1338       +1

Impacted Files
Coverage Δ

pkg/meta/tkv.go
75.65% <25.00%> (-0.26%)
:arrow_down:

pkg/vfs/writer.go
78.64% <0.00%> (-1.02%)
:arrow_down:

pkg/meta/sql.go
69.48% <0.00%> (+0.09%)
:arrow_up:

pkg/vfs/handle.go
86.55% <0.00%> (+0.84%)
:arrow_up:

pkg/vfs/reader.go
79.56% <0.00%> (+3.10%)
:arrow_up:

Continue to review full report at Codecov.

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


CUBICSPLINE rotation export: invalid values

Describe the bug
I have a simple blend file with some bezier curves animation. Upon exporting it to gltf and opening in a viewer (like babylonjs sandbox), the animation sometimes jumps.
To Reproduce
Steps to reproduce the behavior:

Open the blend file
Export it to gltf or glb with "Always sample animations" turned off
See erron in a gltf viewer
Curiously, importing it back to blender, the animation plays back correctly

Expected behavior
The animation is exported correctly.
Screenshots
Here's the output opened in babylonjs (I can't upload a webm, sorry for the bad quality):

Here's the curve I'm exporting:

An interesting info is that I ended up with this curve by repeatedly exporting to gltf and importing back to blender this curve:

You can see that w became y, x became z and got inverted.
.blend file/ .gltf
example.zip
Version

OS: linux
Blender Version: blender-2.83-50ef801a79b5

Additional context
The problem manifest itself first between keyframe 1 and 2 (at frame 3.2 and 4.8). If I look at the exported gltf data, the target values for these two keyframes are:
[ [ -0.0024046599864959717, -0.014678483828902245, -0.0007305145263671875, 0.004064374603331089 ], [ 0.10037517547607422, 0.027345599606633186, -0.9945357441902161, 0.008700160309672356 ], [ -0.0024046599864959717, -0.014678480103611946, -0.0007305145263671875, 0.004064374603331089 ] ]
[ [ 0.0009030848741531372, 0.008994728326797485, 0.0008977651596069336, -0.004734877496957779 ], [ -0.10758800804615021, -0.07530241459608078, 0.9912505149841309, 0.013295253738760948 ], [ 0.0009030848741531372, 0.008994728326797485, 0.0008977651596069336, -0.004734877496957779 ] ]

The tangent values are pretty small so they shouldn't matter much, but the z values for the two vertices are -0.9945357441902161 and 0.9912505149841309 for keyframe 1 and 2 respectively. According to my f-curves both of them should be positive, around 0.99.

Disregard what I wrote in the last paragraph, keyframe 1 is a 179deg rotation around almost -z axis, while keyframe 2 is 178deg rotation around about the z axis, so they are pretty near, but it looks like none of the gltf viewers I checked (babylon.js and https://gltf-viewer.donmccurdy.com/) use slerp with CUBICSPLINE (Checking the gltf spec only mentions slerp at the LINEAR interpolation, in appendix C it only mentions that the resulting quaternion must be normalized and exporters should take care not to produce 0 length quaternions.).

There's more discussion of this in #960.


Add "White" as a cable color

Environment

Python version:  3.6
NetBox version:  2.5.1

Proposed Functionality
Add color white to the cable colors.
Optionally add:

slate(Dark Grey works, almost identical color)
rose
violet (Fuschia works, almost identical color)
aqua

Use Case
These fiber strand colors are missing
Database Changes
None
External Dependencies
None

I'm fine with adding white. Any additional colors will need to be compared to the existing list to ensure they are easily distinguished from all existing colors.
We'll also need to ensure that the proper foreground color is used. There's a utility function named foreground_color() which outputs the ideal foreground color (black or white) given an arbitrary background color. We can expose this as a template filter.


Use a cargo workspace.

Implements #233.
This doesn't seem like a huge change but makes building the project and using all the cargo commands a little simpler.

The workspaces section of the cargo book was a useful reference
The workspaces section of the rust book was less useful, it made it sound like only one option, what the cargo book calls the virtual manifest, was possible.

This is not building on CI for some reason and I'm out of time to investigate for now.

CI seems busted for now, I'll keep #291 open to investigate / fix.

This is passing on CI now so I've marked it as ready for review.
@penelopezone I think everything is still working how it was and this seems like an improvement. WDYT?


Struggling to understand the distinction between "Load GraphQL Schemas" and "Load GraphQL Documents"

Is your feature request related to a problem? Please describe.
I'm using hasura as my graphql engine and have used graphql-codegen init to set the project up. I was not sure about how to answer "Where are your operations and fragments?:" and accepted the default " src/**/*.graphql". My graphql understanding was that everything would be contained in the schema
Describe the solution you'd like

An explanation or example of what these src/**/*.graphql files ought to look like would be immensely helpful. I am relatively new to the graphql ecosystem so perhaps this is trivial for other users, I'm not sure.
Describe alternatives you've considered
Additional context
The error is:
$ graphql-codegen --config codegen.yml
  ✔ Parse configuration
  ❯ Generate outputs
    ❯ Generate src/generated/graphql.ts
      ✔ Load GraphQL schemas
      ✖ Load GraphQL documents
        → Unable to find any GraphQL type definitions for the fo
…
        Generate
    ❯ Generate ./graphql.schema.json
      ✔ Load GraphQL schemas
      ✖ Load GraphQL documents
        → Unable to find any GraphQL type definitions for the fo
…
         I have a similar issue, my use-case:
schema.graphql:
schema {
  query: Query
}

type Greeting {
  name: String!

type Query {
  sayHello(name: String): Greeting
}

Now I would like to generate the 'document that holds your operations', something looking like:
export const sayHello = `query sayHello($name: ID!) {
  sayHello(name: $name) {
    name
  }
}
`;

However, codegen.yml:
overwrite: true
schema: "./schema.graphql"
generates:
  ./generated/graphql.d.ts:
    plugins:
      - typescript-document-nodes

yields:
./src/generated/graphql.d.ts
    Error: Unable to find any GraphQL type definitions for the following pointers: ./generated/graphql.d.ts

How can I get to that file of GQL strings (or DocumentNodes) using graphql-codegen?


Position not showing

Describe the bug
I'm building a geolocation app that needs to track people walking in the street and draw their path as they walk. I'm using react-native-background-geolocation and a kalman filter. The output of which is saved in my state as my coordinates. I'm having issues with my user position, it is not showing. My zoom level is also not working, I don't know what I'm doing wrong ...
To Reproduce
I call set Center at every new coordinate, it works, then nothing happens
setCenter(location:Location) {
if (!this.refs.map) { return; }
this.refs.map.flyTo([location.coords.longitude, location.coords.latitude], 1000)
}
<MapboxGL.MapView
ref="map"
zoomLevel={16}
logoEnabled={false}
style={styles.flex}
styleURL={MapboxGL.StyleURL.Street}
showUserLocation={true}
scrollEnabled={true}
userTrackingMode={MapboxGL.UserTrackingModes.Follow}
centerCoordinate = {this.state.coordinates} >
</MapboxGL.MapView>
Expected behavior
I expect to see my GPS point going from one point to another.
Screenshots
If applicable, add screenshots to help explain your problem.

Versions (please complete the following information):

Platfrom: iOS
Device: Simulator
OS: iOS12.2
SDK Version [e.g. 22]
React Native Version: 0.59

Additional context
Add any other context about the problem here.

@nitaliano I have read everything I could find, you're my last hope ! Thank you for your help !

@GigiRewind Adding user location is slightly different than on https://github.com/nitaliano/react-native-mapbox-gl. You need to add <MapBoxGL.UserLocation /> tag inside your MapBoxGL.MapView.
Same thing happens with the zoomLevel. You need to change it to MapBoxGL.Camera and add it inside MapView.
 <MapBoxGL.Camera             ref={(ref) => (this.camera = ref)}             zoomLevel={10}             centerCoordinate={[Lng, Lat]}             animationDuration={2000}           />

please check out CHANGELOG

Thank you both for your answers, I did miss the changes in the changelog !

The fact that you need to add <MapBoxGL.UserLocation /> is missing from the changelog. This had me confused for quite some time... Thanks @DejanBesic for the tip!

Can you tell me how you applied the Kalman filter? @GigiRewind


'-i -' option infinitely loops

Is it time to read from a file instead of stdin?

The problem was the input file, and not an issue with the option.


HDDS-4741. Only test upgrade from 1.0.0

What changes were proposed in this pull request?
Make upgrade acceptance test verify upgrade from Ozone 1.0.0 instead of 0.5.0.  Upgrade from 0.5.0 to 1.0.0 is already verified by previous runs of the test.
https://issues.apache.org/jira/browse/HDDS-4741
How was this patch tested?
https://github.com/adoroszlai/hadoop-ozone/runs/1762340366

@adoroszlai Thanks for adding this test. From 1.0 Ozone GA should acceptance test, test out all upgrade paths?

@adoroszlai Thanks for adding this test. From 1.0 Ozone GA should acceptance test, test out all upgrade paths?

Closing this for now.


Application is Not Run

I have Imported this Application in Eclipse IDE Suppose I am running this but i am getting only Error:0 Exception. please help

It sounds like you’re trying to run the reference Java version of the code—this library is for Go and only has the java code as a reference in a non-runnable state.
If you’re trying to run the Go code you’ll need to write your own package main and import the library to use it.

hi dlclark,
Thank you for your quick response, its really appreciated.
I will try the same , if i am facing any issue i will ask you.
Is this applicable for training dataset?
Suppose we are trying to test the dataset in currently your algorithm we are not getting any path:  we are getting only these line for line by line reading line   (but Attrib,find.go,metaphone3.go,metaphone3,metaphone3_utils_test.go,metaphone3_words_test.go ) we are unable to understand.
Can you help me sir.
metaphone3.txt


add a flag to skip rewriting source file, only prettier output stream

This loader is perfect if i use HTML file as input.
It fails when it is used after an HTML preprocessor (for example Nunjucks or EJS).
As the loader rewrite source file if source code is different from prettier code, it rewrites root template with the processed code (which is the result of the preprocessed root file). In this use case, it would be great to have a way to disable rewriting file.
Here is an example with nunjuncks :
{
	test: /\.njk$/,
	use: [
		{
			loader: "html-loader"
		},
		{
			loader: "prettier-loader",
			options: {
				filepath: "foobar.html",
				skipRewritingSource: true
			}
		},
		{
			loader: "nunjucks-html-loader",
			options: {
				searchPaths: [...glob.sync("./src/njk/")]
			}
		}
	]
};

The idea is the same with other preprocessor like EJS.

Hi, @astik , thanks for PR,
Can you add docs and tests on this?

I have added a more precise documentation (with use case) and add unit test.
I have tried not to change much code for the test. So that, i'm not testing the full feature ='(
I'm only testing that entry stream is left unchanged. I'd have liked to also test that processed file is "prettier checked" but it contains code from webpack.
Is the current test enough for your taste ? Should we go further in the test ? in that case, any idea how to retrieve processed content ? an additional loader ?

I've just published new version with dropped support of node@4, it did not support rest/spread syntax, so you've got failed check. But now it has conflicts. I can add some code to finalize this PR, if you want it.
So yes, I think additional loader will be fine.

I can work on this today, i'll notice you when it'll be ok =)

I've added unit test for output content (through an additional loader in the loader chain and jest mock).
It should be ok now. Feel free to tell me if it's not =)

@astik good one! Just 2 small suggestions left 👍

3.1.0


Source code project support Apple Silicon natively

Issue Description
I am reading source code on Apple M1 device, but I get some faults. The error message indicates that Gradle is not support for M1 chips.
Gradle has support for Apple Silicon since version 7.0. Now we are using Gradle version 4.6 in the project, so I got the previous faults.
Expected Behavior
Open azkaban gradle project without error
Things todo

Upgrade Gradle version to 7.0+
Change old version Gradle grammar to new ones, like compile to implementation
Support plugin C natively in module az-exec-util

Current progress
I have done step 1 and 2, and get stuck in step 3, with error message Invalid NativePlatform: osx_arm-v8

me too

@blackberrier  we are internally using Gradle 5 and have no immediate requirement to move to Gradle 7+. However, if you are willing to upgrade to gradle 7 and contribute, I will help review the code and merge in the open source repo.

This is legit the only place in the entire[^1] Github — as of today[^1] — where "osx_arm-v8" is mentioned.
I think it's a Gradle problem. But I am too new to Gradle to know where the issue is; let alone propose a fix
[^1]: The search link may have more when you check

This is legit the only place in the entire Github — as of today1 — where "osx_arm-v8" is mentioned.
I think it's a Gradle problem. But I am too new to Gradle to know where the issue is; let alone propose a fix
Footnotes

The search link may have more when you check ↩

I am so confused. Why is this the only result? That search was how I found this link, and I'm not even using this project, just a barebones JNI project.

I have a working branch for Apple Silicon / ARM and relevant instructions that I just pushed to this branch and opened a PR
(Note: it wasn't clear what value the "build scan" Gradle plugin was providing - and it was the only thing I couldn't get working on ARM).  The builds, distribution and tests all work.
https://github.com/azkaban/azkaban/pull/3115
https://github.com/jeffsteinmetz/azkaban/tree/jeffs/apple-silicon-arm

@blackberrier we are internally using Gradle 5 and have no immediate requirement to move to Gradle 7+. However, if you are willing to upgrade to gradle 7 and contribute, I will help review the code and merge in the open source repo.

See PR ^^


public, static, and private keywords appear as errors if class extends two lines

Notice the open curly brace for the `class` is now on line `108`

The reason this is a problem is not because I want a curly brace on it's own line. It is because sometimes the class definition is too long for one line

Notice how all highlighting on line 108 is also gone.

Just going off of the commit message, perhaps this was attempted to be fixed in: https://github.com/HerringtonDarkholme/yats.vim/commit/d1bfa9621f54ddef42b0225492ce18c0bb78dec3?

Thanks for reporting! I'm trying to fix it.

Fixed in https://github.com/HerringtonDarkholme/yats.vim/commit/6cc20675649e411bc9adb414341176c9ee845ce5


Update HybridCompatibilityMixin to handle RESTCapableHybridTopologyFactory and add repex self-consistency tests

Description

Updated HybridCompatilityMixin to automatically detect which type of factory (HybridTopologyFactory or RESTCapableHybridTopologyFactory) was inputted and create a sampler depending on the factory type.
Added self-consistency tests for dipeptide systems (running repex with the new RESTCapableHybridTopologyFactory):

Neutral mutation: ALA->THR vs THR->ALA
Charge changing: ARG->ALA->ARG vs LYS->ALA->LYS (see docstring of test for explanation on why these particular transformations are necessary)

TODO:

I put the tests in test_samplers.py -- not sure if this is the right place -- may want to move it somewhere else and/or make it an example?
Not sure if the tests in test_samplers.py are even working -- should we delete everything else in here?
n_iterations should be bumped up to 1000 and it should be run on a gpu (@mikemhenry : Can you help with this?)

Motivation and context

Resolves #984
How has this been tested?

Change log

Could we run a few iterations on the CPU but skip the validation checks, and run the 1000 iterations on the GPU? An advantage of just doing a few iterations on the CPU is it will help find API/other run time errors faster

Yes that sounds good. Sorry I wasn’t clear, that’s actually what I meant.
On May 4, 2022, at 03:51, Mike Henry @.***> wrote:

Could we run a few iterations on the CPU but skip the validation checks, and run the 1000 iterations on the GPU? An advantage of just doing a few iterations on the CPU is it will help find API/other run time errors faster
—
Reply to this email directly, view it on GitHubhttps://github.com/choderalab/perses/pull/992#issuecomment-1117022332, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIPGJCSQ5JGHJHIY6KPYFA3VIIT57ANCNFSM5VALN3DQ.
You are receiving this because you authored the thread.Message ID: @.***>

Can this make it into the 0.10.0 release, or will it have to wait until 0.11.0?

@jchodera : This can probably make it into 0.10.0, we just need to decide whether we want to keep the tests I wrote as tests or convert them to examples. Was thinking we could discuss this tomorrow at the dev sync.
After we make a decision, we might need @mikemhenry to help specify which tests should be run on cpu vs gpu.

I should also note that the new tests I added in this PR are the reason why the GHA workflow has failures. This is because the tests are only running 2 cycles of repex, which is surely not enough to get the free energies to be equal and opposite. We'll need to move this test to the gpu to get the free energies to match.

Notes from dev sync:

We need to determine the smallest number of replicas and iterations necessary to get the repex examples to converge.
Make this an example (not a test) -- which will be marked as gpu only and will run this once a week on aws (or lilac??) to catch potential regressions.

I keep getting the following error traceback when running the counterion mutation test using 4 or more iterations (2 work):
Traceback (most recent call last):
  File "/lila/data/chodera/pulidoi/sandbox/perses-992/test_REST_counterion_mutation.py", line 79, in <module>
    f_ij, df_ij = analyzer.get_free_energy()
  File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/openmmtools/multistate/multistateanalyzer.py", line 1932, in get_free_energy
    self._compute_free_energy()
  File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/openmmtools/multistate/multistateanalyzer.py", line 1892, in _compute_free_energy
    (Deltaf_ij, dDeltaf_ij) = self.mbar.getFreeEnergyDifferences()
  File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/pymbar/mbar.py", line 541, in getFreeEnergyDifferences
    Theta_ij = self._computeAsymptoticCovarianceMatrix(
  File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/pymbar/mbar.py", line 1679, in _computeAsymptoticCovarianceMatrix
    check_w_normalized(W, N_k)
  File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/pymbar/utils.py", line 358, in check_w_normalized
    raise ParameterError(
pymbar.utils.ParameterError: Warning: Should have \sum_n W_nk = 1.  Actual column sum for state 0 was 0.635215. 12 other columns have similar problems

Ah yes that’s an issue with your version of pymbar. Make sure you install from master.
On May 24, 2022, at 20:04, Iván Pulido @.***> wrote:

I keep getting the following error traceback when running the counterion mutation test using 4 or more iterations (2 work):
Traceback (most recent call last):
File "/lila/data/chodera/pulidoi/sandbox/perses-992/test_REST_counterion_mutation.py", line 79, in
f_ij, df_ij = analyzer.get_free_energy()
File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/openmmtools/multistate/multistateanalyzer.py", line 1932, in get_free_energy
self._compute_free_energy()
File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/openmmtools/multistate/multistateanalyzer.py", line 1892, in _compute_free_energy
(Deltaf_ij, dDeltaf_ij) = self.mbar.getFreeEnergyDifferences()
File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/pymbar/mbar.py", line 541, in getFreeEnergyDifferences
Theta_ij = self._computeAsymptoticCovarianceMatrix(
File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/pymbar/mbar.py", line 1679, in _computeAsymptoticCovarianceMatrix
check_w_normalized(W, N_k)
File "/home/pulidoi/miniconda3/envs/perses-dev/lib/python3.9/site-packages/pymbar/utils.py", line 358, in check_w_normalized
raise ParameterError(
pymbar.utils.ParameterError: Warning: Should have \sum_n W_nk = 1.  Actual column sum for state 0 was 0.635215. 12 other columns have similar problems
—
Reply to this email directly, view it on GitHubhttps://github.com/choderalab/perses/pull/992#issuecomment-1136547382, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIPGJCVM65SVZ3DD5UM4GNDVLVVCDANCNFSM5VALN3DQ.
You are receiving this because you authored the thread.Message ID: @.***>

@ijpulidos : This is the pymbar issue related to your error: https://github.com/choderalab/pymbar/issues/419

@ijpulidos : Actually, which version are you using? 3.0.6+ should be fine, 3.0.5 is broken.

@zhang-ivy yes, I was using 3.0.5, I can confirm 3.0.6+ works fine. Thanks!
I checked the behavior of the free energy with the number of iterations. I don't see any tendency or convergence. The results are as follows
# neutral mutation
n_iterations, energy_diff
2,      4.18895
4,      0.990343
8,      -0.00456955
16,     -5.50846
32,     -0.54498
64,      -1.93432
128,     0.827766
256,     0.176885
512,     0.0994963  # took ~3 hours

# counterion mutation
n_iterations, energy_diff
2,      4.79188
4,      6.45721
8,      0.626254
16,     1.82293
32,     -1.89355
64,     1.7545
128,    -2.88888   # took ~1.5 hours

None of them converged (they all resulted in an AssertionError as per the test). The walltime for the longest ones are specified.

@ijpulidos : I think the n_iterations is too small,  can you try n_iterations=250, 500, 1000, 1500, 2000? You basically already did the first 2 for the neutral mutation, so need to repeat those.

@zhang-ivy Sure I can try, I'm just wondering if we are okay with tests running for more than 6 hours in the GPU CI. I guess it's not much of an issue if these are running overnight or so.

Yah lets first figure out what the minimal amount of time we can spend is, then we can figure out what we want to do with that. It may be something we check before a release or once a month or something. A p2.xl is something like 90 cents an hour, so even it it takes 10 hours, that isn't munch per month or week if we decide we need it.

The most recent results are as follows (I'm included previous results as well just in case we spot some pattern or something).
# neutral mutation
n_iterations, energy_diff
2,      4.18895
4,      0.990343
8,      -0.00456955
16,     -5.50846
32,     -0.54498
64,      -1.93432
128,     0.827766
256,     0.176885
512,     0.0994963  # took ~3 hours
1024,   -0.199971
1512,   -0.026376  # took ~13 hours in A40

# counterion mutation
n_iterations, energy_diff
2,      4.79188
4,      6.45721
8,      0.626254
16,     1.82293
32,     -1.89355
64,     1.7545
128,    -2.88888   # took ~1.5 hours
512,    1.87339
1024,   -1.04253
1512,   -1.37579
2048,   0.446562
4096,   -0.247799
5000,   -0.436782  # took ~17 hours in A100

These are still not converged in terms of np.isclose which, as far as I can see, defaults to checking around 5 significant figures. I wonder if we are expecting too much and whether we should change our convergence criteria. As far as I can see, it also depends on the system, the counterion one is one order of magnitude further apart compared to the neutral system. Thoughts?

As far as I can see, it also depends on the system, the counterion one is one order of magnitude further apart compared to the neutral system. Thoughts?

Actually, nvm this part, if we check the significant figures in the actual values (not the difference) it is similar for both systems. For example:
# neutral system
ALA-THR is -51.84165478283243. THR-ALA is -51.81527874133261
# counterion system
ARG-ALA-ARG is -277.7522884618811. LYS-ALA-LYS is -277.31550615053516

I wonder if we are expecting too much and whether we should change our convergence criteria.

Yes, I think the convergence criteria is too stringent. Let's just check whether the difference in the forward and reverse free energies is < 1 kT, or perhaps 0.5 kT if we want to be a bit more stringent.

Ok, so I think we leave these as tests (now I think they are actually tests not examples, sorry for my confusion before). I'll be pushing the changes soon.
@mikemhenry I'm wondering if we use a new pytest mark for these kind of tests that could potentially take hours/days maybe having yet another CI workflow that run these specially marked tests every once in a while or so? Ideas on how could we accomplish this are appreciated.

@ijpulidos How about we merge this in and see how long it takes on AWS? How long does it take if we only care about <1kT?

@ijpulidos How about we merge this in and see how long it takes on AWS? How long does it take if we only care about <1kT?

So there are two tests, the first one might run in a reasonable time, the second one I have the suspicion that it is going to take more than 24 hours to run on the current AWS instance we are using. I guess we can just try and see, I'm going to make the changes for this to be run (it's now explicitly skipped with pytest). What's a good limit for this? My assumption is that anything below 24 hours should be okay since we are running the GPU CI every 24 hours as well, but I don't know if there are other limits (github/aws limiting it).

On a self hosted runner I think it's like a 30 day timeout 🤣 let's do it an see what happens

@ijpulidos : I just reviewed your changes and I don't think we should call the file test_topology_factories.py -- we aren't actually testing the factory here -- we do that in test_relative.py -- I think we should leave it as test_samplers.py, since we are checking to make sure the sampler HybridCompatabilityMixin is working properly.

@zhang-ivy Yes, I agree. I think it makes sense to have a test_repex.py module or similar with all the repex tests (these ones at least, maybe we'll migrate/create others with time). Similarly with others like sams and neq-switching, etc. I'll make the changes.


Allow spaces when tokenizing option-value

Previous behavior split the incoming token long-name value using the equal sign as the separator.
This would cause an option's value containing spaces to get truncated.
New behavior does regex. match to make sure format is correct and gets the full value.

To achieve same behavior contained within PR, I need to pass arguments like:
./app --option "first name=Bill;last name=Boga"

Hi @billboga based on the code you replaced I think the problem was actually the = rather than spaces. In any case you've fixed the issue, so thanks for contributing!

Yeah, I had two different thoughts in my head when I was typing. What you wrote is what I meant :smile:.


Fixed TSLint warnings in 'msbot-connect-*.ts' files

Proposed Changes
Fix the following TSlint warnings:

typedef
align
curly

Testing
Travis will no longer report these warnings

Pull Request Test Coverage Report for Build 270

15 of 16 (93.75%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.006%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/MSBot/src/msbot-connect-file.ts
5
6
83.33%

Totals

Change from base Build 267:
0.0%

Covered Lines:
3183

Relevant Lines:
4740

💛  - Coveralls

Replaced with https://github.com/Microsoft/botbuilder-tools/pull/425


Consider renaming repo+npm to @foxglove/schemas?

Not a big deal, but it seems simpler.

@jtbandes thoughts on this? If it's too much effort just close the ticket. I think it would be aesthetically nice but not critical.


Add a test for fd_readdir

There are still some unclarities to be resolved before merging:

[ ] readdir(2) shows that the filename (d_name) should be null-terminated but the current implementation doesn't null-terminate the filename. Is this intended? This looks like something that may cause problems for programs written in C
[ ] what are the semantics of dircookie_t and why is it a separate type? readdir(2) is just using a long. What are the semantics of the cookie input parameter? The docs say that: The location within the directory to start reading., which is unclear to me. In particular, the test should test the behavior with cookie != 0
[ ] why is . a regular file and not a directory?

There are still some unclarities to be resolved before merging:
* [ ]  `readdir(2)` shows that the filename (`d_name`) should be null-terminated but the current implementation doesn't null-terminate the filename. Is this intended? This looks like something that may cause problems for programs written in C

For C programs calling readdir, WASI libc takes care of handling all the low-level details, including NUL-terminating the strings.
The lack of NUL termination in fd_readdir comes from CloudABI, so I don't know the original motivation, but looking at WASI, we have users other than C which don't need NUL-termination and in general we'd like to avoid baking it in when we can.
That said, argv/environ (which aren't from CloudABI) do have NUL-terminators, so things aren't entirely consistent right now. I'm hoping that we can clean up some of these details once we can move to interface types.

* [ ]  what are the semantics of `dircookie_t` and why is it a separate type? `readdir(2)` is just using a `long`. What are the semantics of the `cookie` input parameter? The docs say that: _The location within the directory to start reading._, which is unclear to me. In particular, the test should test the behavior with `cookie != 0`

__wasi_dirent_t contains a dircookie_t value which you can use to restart the readdir at that point. A typedef is used primarily for documentation purposes, to clarify that the __wasi_dirent_t struct field is related to the cookie argument to fd_readdir.

* [ ]  why is `.` a regular file and not a directory?

That's a good question. What WASI implementation is this, and what host OS is it on?

For C programs calling readdir, WASI libc takes care of handling all the low-level details, including NUL-terminating the strings.

IOW, if we're compiling from C, and the C implementation of readdir expect a null-terminated string, the encode/decode step will take care of adding those nulls? This sounds like O(n) complexity to move the output around, and we actually do the copy twice on Linux - once for the underlying syscall and once to comply with the readdir API.

* [ ]  what are the semantics of `dircookie_t` and why is it a separate type? `readdir(2)` is just using a `long`. What are the semantics of the `cookie` input parameter? The docs say that: _The location within the directory to start reading._, which is unclear to me. In particular, the test should test the behavior with `cookie != 0`

__wasi_dirent_t contains a dircookie_t value which you can use to restart the readdir at that point. A typedef is used primarily for documentation purposes, to clarify that the __wasi_dirent_t struct field is related to the cookie argument to fd_readdir.

In other words, does the cookie tell fd_readdir to skip writing first n bytes to the output buffer?
Assume that for the first entry d_next equals 25. What if the caller re-executes fd_readdir with cookie = 8?
Such an argument is missing from readdir(2) and readdir(3) on my system.

* [ ]  why is `.` a regular file and not a directory?

That's a good question. What WASI implementation is this, and what host OS is it on?

I'm running on Arch Linux. This is not a bug in WASI - the behavior is consistent between the host and WASI, i.e. on the host I'm also getting the type of . to be a regular file. I'll upload the code tomorrow.

That's a good question. What WASI implementation is this, and what host OS is it on?

@sunfishcode, so @marmistrz and myself had a little scan through the codebase, and discovered it was a problem in mapping host's dirent to __wasi_dirent_t. This is now fixed in cranestation/wasi-common#78, plus @marmistrz suggested rewriting fd_readdir using (safe) nix anyhow, which I think is a great idea :-)

I'm not sure if this test is correct at all: from readdir(3p):

   If  a file is removed from or added to the directory after the most recent call to opendir() or rewinddir(), whether a subsequent call to readdir() returns an entry for that file
  is unspecified.

but we don't close the fd's, so we won't also do closedir.


Navigation

#Navigation
In this project:

Added a nav bar with a logo
Added extra content that displays when links are clicked

Good Day,
my partner did most of the work for the second review, hence I thought it
was only fair.
However, changes will be made.
Regards
Daron
On Thu, Sep 22, 2022 at 10:49 AM Teddy-Livingstone Ememandu <
@.***> wrote:

@.**** commented on this pull request.
STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @Daron976 https://github.com/Daron976,
While you made a great effort in this project, unfortunately, I cannot
proceed to review your code.
Invalid Code Review Request This is a collaborative project. Kindly
ensure your coding partner contributes to this project
Your Code Review Request will be marked as invalid in your Dashboard, so
please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear. And please, remember to tag me in your question so I can
receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.
—
Reply to this email directly, view it on GitHub
https://github.com/Daron976/Awesome-Books/pull/6#pullrequestreview-1116640682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQQJYFXLDOHY4WYVK7DT5E3V7QMQDANCNFSM6AAAAAAQS2DNMI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Make a v5 Bootstrap

Please can you update your code to the v5 of Bootstrap ?

forked and modified with new versions includng v5.0.2
check here:
https://github.com/zakk616/Isolated-Bootstrap


sci-colorado/sub-5740_T2w.nii.gz - weird intensity range

The sci-colorado/sub-5740/anat/sub-5740_T2w.nii.gz image has a weird intensity range:
$ fslstats sub-5740_T2w.nii.gz -R
-32767.000000 32516.000000

This results in a "gray nothing" when opening the image in FSLeyes:

When manually adjusting the min and max to 0-1000 interval, the image looks good:

The weird intensity range results in the failure of sct_deepseg_sc:
$ sct_deepseg_sc -i sub-5740_T2w.nii.gz -c t2 -kernel 2d

--
Spinal Cord Toolbox (git-jv/sct_analyze_lesion_improvements-16742e4eada8e9853993e61fe4ce559767d4f42a*)

sct_deepseg_sc -i sub-5740_T2w.nii.gz -c t2 -kernel 2d
--

Config deepseg_sc:
  Centerline algorithm: svm
  Brain in image: True
  Kernel dimension: 2d
  Contrast: t2
  Threshold: 0.7
Creating temporary folder (/var/folders/w6/bnzr2qls11l69w0_8q95ghtr0000gn/T/sct_2023-09-11_10-43-36_deepseg-sc_sk1d9x5y)
Reorient the image to RPI, if necessary...
Finding the spinal cord centerline...
Creating temporary folder (/var/folders/w6/bnzr2qls11l69w0_8q95ghtr0000gn/T/sct_2023-09-11_10-43-37_optic-detect-centerline_8wgfjh8r)
/Users/valosek/code/sct_latest/spinalcordtoolbox/centerline/optic.py:102: RuntimeWarning: overflow encountered in short_scalars
  data_rescaled = img_data.astype('float') * (max_out - min_out) / (max_in - min_in)
To avoid intensity overflow due to convertion to +uint16+, intensity will be rescaled to the maximum quantization scale
Remove temporary files...
rm -rf /var/folders/w6/bnzr2qls11l69w0_8q95ghtr0000gn/T/sct_2023-09-11_10-43-37_optic-detect-centerline_8wgfjh8r
Cropping the image around the spinal cord...
Normalizing the intensity...
Segmenting the spinal cord using deep learning on 2D patches...
Reassembling the image...
Resampling the segmentation to the native image resolution using linear interpolation...
Spinal cord could not be detected for /Users/valosek/data/data.neuro.polymtl.ca/sci-colorado/sub-5740/anat/sub-5740_T2w.nii.gz
    EmptyArrayError: 'Spinal cord not detected. Please make sure that there is sufficient contrast between the spinal cord and CSF to ensure good results.'

Very interesting bug! Thank you for reporting this, Jan

Note that this intensity issue results in the failure of the baselines/comparison_with_other_methods_sc.sh script for
seed710 and seed123 train-test splits. --> we have no SC predictions for this subject for these two seeds.

I had a quick look at the data for this image (sub-5740_T2w.nii.gz), and all the values which are outside the range [0, 10000] are concentrated in a small region of the image:

z-coordinate = 33 (the last slice)
y-coordinate in [0, 12] (out of 512)
all 512 x-coordinates.

So maybe it would make sense to zero out this part of the image.


Pass cookiebot ID as param/option

I think it would more optimal for the Cookiebot ID to be passed as a param, .e.g.
import CookieBotLoader from "@danielgietmann/react-cookiebot";

function App() {
  return (
      <html lang="en">
            <body>
                <CookieBotLoader cookieBotId={process.env.COOKIEBOT_ID} />
                {children}
            </body>
        </html>

  )
}

Some people might not have their env variables set exactly as COOKIEBOT_ID.
Perhaps set cookieBotUrl as an optional param.
This is behaviour that I think most people would expect. Cheers! 🍻

Thanks for the suggestion! I will be adding a feature that allows setting the domain via a .env file for more flexibility. However, there are no current plans to implement a function within the component's props for this.
Appreciate your input!


Get filezise after save() return the original size

Hi
I read an image:
$img = Image::make(public_path() . '/ft_' . $id . '_' . $images['path']);
The image already exists, then read with no problem, after that, I read his size and give me 378.56:
$size = $img->filesize() / 1000;
Then I want to reduce the quality until the file size is lower from 100 kbytes, I tried this but don't work:
`
while ($size > 100) {
$img->save(public_path() . '/mr-photos/mr_' . $id . '_' . $images['path'], $quality)->encode('jpg');
$img->filesize();
dump('size: ' . $size);
if ($size <= 100) {
exit;
} else {
dump('quality: ' . $quality);
$quality--;
}
                                if ($quality < 60) {
                                    $size = 100;
                                    exit;
                                }
                            };

`
The problem is that the $size never change, in the folder the image reduce it's size until have 26kb, but the $size variable never reduce.
I tried too:
$img->save(public_path() . '/mr-photos/mr_' . $id . '_' . $images['path'], $quality); $img = Image::make(public_path() . '/mr-photos/mr_' . $id . '_' . $images['path']); $size = $img->filesize() / 1000;
but is wrong, because now the size still the original size but the image is damaged. Then I tried:
                                     $img = Image::make(public_path() . '/mr-photos/mr_' . $id . '_' . $images['path']); $img->save(public_path() . '/mr-photos/mr_' . $id . '_' . $images['path'], $quality); $size = $img->filesize() / 1000; $img->destroy();
Same error, I can't update the size variable and the image is damaged.
I hope you can help me...
Regards
Bruno

I have the same problem, getting the file size after save is returning the original one, even if I opened the file under another variable.

Have you tried to clearstatcache after file save ?

I tried almost everything, open with different variable, and everything I try obtain the same error.

I can see some things that might not working.
1- clearstatcache() normally helps since it clears filemtime or filesize operations
2- Have you tried to download the image and validate it for corruption?
3- Have you tried to use php's native filesize functions?
4 - Another variable seems a bad idea if intervention is somehow caching or cloning the file reference ( from what I am reading and it's purely theoretical.)

I can see some things that might not working.
1- clearstatcache() normally helps since it clears filemtime or filesize operations
2- Have you tried to download the image and validate it for corruption?
3- Have you tried to use php's native filesize functions?
4 - Another variable seems a bad idea if intervention is somehow caching or cloning the file reference ( from what I am reading and it's purely theoretical.)

clearstatcache() tell me Command (clearstatcache) is not available for driver (Gd)
Yes if I download the image is not corrupted, is fine.
Yes I try it and return me the same filesize, the original filesize not the new.
Nothing works...

Thanks

Same issue, fixed by calling clearstatcache(); after the save()
PHP doc : https://secure.php.net/manual/en/function.clearstatcache.php
$im = $manager->make($file_path);
dump([$im->filesize(), filesize($file_path)]);
$size = 200;
$im->resize($size, $size)->save();
clearstatcache();
dump([$im->filesize(), filesize($file_path)]);

result :
array:2 [
  0 => 104516
  1 => 104516
]
array:2 [
  0 => 15131
  1 => 15131
]


Added interactive sql practice

What does this PR do?
Added a really good free course on Stepik to help you gain new practical skills in SQL
Description
In the course, most of the steps are hands-on exercises for creating SQL queries. Each step includes minimal theoretical aspects of a database or SQL language, examples of similar queries, and an explanation of the implementation.
Why is this valuable (or not)?
This course is aimed at teaching the student to create databases and implement queries to them in the SQL language for various subject areas.
How do we know it's really free?
This is a free course in Stepik
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

all the stepik courses are in the courses list. is this one different?

My bad, i saw interactive in his name and add in this category. I will create new pull request in courses list 🥺


How to solve the problem that the input image can't be decoded

Have you ever met the decoded input image?
And the error is displayed as below
"mxnet.base.MXNetError: [14:35:46] src/io/image_io.cc:147: Check failed: !res.empty(): Decoding failed. Invalid image file."
If you have ever met, how did you solve the problem?

could you please send me the model<EMAIL_ADDRESS>


[release-0.41] Fix [test_id:6978] Should detect a failed migration

This is a manual cherry-pick of #6643
/assign enp0s3
Signed-off-by: Igor Bezukh<EMAIL_ADDRESS>Release note:
NONE

/cc @xpivarc

/test pull-kubevirt-unit-test

/test pull-kubevirt-unit-test-0.41


DJ process stop running alone

I've been a problem using delayed_job to send email in the background. Without any reason, the process appears frequently as no running. I've one server to run my application and other only to run the jobs.
That's my configuration of production deploy:
config/deploy/production.rb
# APP
server '<IP_ADDRESS>', port: 22, roles: [:web, :app, :db], primary: true

# JOBS
role :jobs,                   '<IP_ADDRESS>'
set :delayed_job_server_role, :jobs
set :rails_env,               'production'
set :branch,                  :master
set :stage,                   :production

I'm using this gems:
Gemfile
gem 'delayed_job_active_record', '~> 4.1', '>= 4.1.1'
gem 'daemons', '~> 1.2', '>= 1.2.4'
gem 'delayed_job_web', '~> 1.2', '>= 1.2.10'
gem 'whenever', '~> 0.9.7', require: false
gem 'capistrano3-delayed-job', '~> 1.7', '>= 1.7.2'

Also, I'm not sure if I need to have my application running on both servers...

What do the logs tell you?

@albus522 unfortunately, I couldn't find anything that helps to identify when and why it's been stopped. No errors, no warning.

I've been seeing similar behavior too. I've checked system logs, rails logs, and even setup an independent logger for delayed job ..not a thing is being outputted prior to the process failing. My next test will be to run it in the foreground and redirect output to a file handle to see if I can come up to something. Will post an update if I do, otherwise, time for a process watchdog.

Process is still failing without anything logged after several days --- in my case about a week. I gave up trying to get something out of it and threw in a "-m" flag to the delayed_job start command. This flag runs an extra monitoring process through the daemon gem that watches and restarts the process when it fails. The exact command for usage for me is:
RAILS_ENV=production bin/delayed_job -m start

Closing stale


Error in error handling hides the real error message

https://github.com/erocarrera/pydot/blob/d6ac9e9244d1a882103422ac2b35ceef96f5dfe3/dot_parser.py#L554
probably want str(err), not just err? Currently I am getting
pydot raised an error: must be str, not ParseException

When I would expect to be getting?
pydot raised an error: Expected "}" (at char 688), (line:19, col:29)

Thanks for reporting. I submitted a PR to fix this: https://github.com/pydot/pydot/pull/219.


comment changes only to test travis build which seems to be failing

So it looks like Travis CI builds for all PRs are failing (this PR as you can see contains no meaningful changes from master). I don't know where to look from here but am happy to contribute time to help figure it out. Any pointers to fix the CI build process?

Gem::InstallError: rack requires Ruby version >= 2.2.2.
It looks like the travis.yml needs to restrict rubies to 2.2.2 and above.

Sorry, I don't get notified when additional commits are made.


Fix Gazebo link

Gazebo seems to be having some DNS issues. http://www.gazebosim.org doesn't work, but http://gazebosim.org does. This changes that link.

@pedrovhb Thanks!


Optional mandatory requirements [sic]

fcs_tlsc_ext.1.1 has the following note:
      If <i>implement TLS 1.2 (RFC 5246)</i> is selected, then
      <linkref linkend="FCS_CKM.2.1"/>,    <linkref linkend="FCS_COP.1.1(1)"/>,
      <linkref linkend="FCS_COP.1.1(2)"/>, <linkref linkend="FCS_COP.1.1(3)"/>, and
      <linkref linkend="FCS_COP.1.1(4)"/> are required.

which kinda implies that those requirements are selection-based, but they're not. What is this trying to say?

Rewrite to mandate that TLS 1.2 is supported?
Rewrite suggestion:
FCS_CKM.2.1, FCS_COP.1.1(1)...etc... are mandatory requirments to ensure TLS 1.2 operates in a conformant manner?
Or...
Ensure that all the tests listed aren't marked as optional?

This is a vestige of copy-paste from the App PP.  After copy-pasting that document, it should have been scrubbed.

Scrubbed


Feature Absorber SkyFactory 3

I'm attempting to get huge orbs dimlet... and idk how to get the feature absorber to absorb a sphere :l
Thanks.

bump.. same issue, when absorbing it comes up as "unknown feature", when crafting it becomes a volcanoes feature dimlet.

have you guys made a random dim yet to get more dims? or make a enderman farm and get dim packages

Note that the feature selected is random. It doesn't matter that you place it on an orb. It will just absorb a random feature from that dimension. I'll fix the bug that it always says unknown even at 100% though

At 100% waila, JEI, and TOP should now show the real feature


DLRS - can't deactivate; can't save

I'm wondering if I should uninstall DLRS in Salesforce.
It seems like I really screwed up all of my lookups.
I can't deactivate, I can't save, I can't edit my lookup fields on my contact field.
Can anybody tell me if the following message justifies my thinking?
`Apex script unhandled trigger exception by user/organization: 005f2000008y2gu/00Df2000001Kv2H
dlrs_ContactTrigger: execution of AfterUpdate
caused by: System.QueryException: value of filter criterion for field 'Current_Grade__c' must be of type string and should be enclosed in quotes
Class.dlrs.LREngine.QueryExecutor.query: line 662, column 1
Class.dlrs.LREngine.WithoutSharingQueryExecutor.query: line 680, column 1
Class.dlrs.LREngine.rollUp: line 238, column 1
Class.dlrs.LREngine.rollUp: line 117, column 1
Class.dlrs.RollupService.updateMasterRollupsTrigger: line 1035, column 1
Class.dlrs.RollupService.handleRollups: line 892, column 1
Class.dlrs.RollupService.triggerHandler: line 311, column 1
Trigger.dlrs_ContactTrigger: line 7, column 1`

@R0cket could you add a screenshot of your rollup configuration? It sounds like the "Current_Grade__c" field in your relationship query field isn't assembled correctly. If it is the target of your rollup it might be a few other things.

All fixed thank you aheber!


create a docker image for token chaincode (tcc)

Hello,
we're using zkatcc as an external chaincode on HLF. As such we're building by ourself the zkat chaincode docker image but I think this project should be the proper location for such build.
For such task, we would need a docker image repository. As Fabric Token SDK is now open source,  I suggest to use the official docker hub. Then we should think on the delivery pipeline for such chaincode delivery :

how do we manage latest image update ?
how do we manage tagged image publication ?
...

Also, on our current zkat chaincode, we embedd the tokengen tool that way we just need the zkat chaincode image to use that tool and we do not need to build it on each of our developpers env. This choice may be discussed and may be we would prefer other way to deliver such tool :

a dedicated docker image for fts toolings ?
a binary delivery for any platform (linux, osx ...) ?
... ?

will address when time comes


[testfairy] Add 'testers_groups' option to 'testfairy' action

Change testers_group type to array
Add optional parameters test
Add short option value

Issue: #4212
Signed-off-by: Joe Greubel<EMAIL_ADDRESS>

This is a replacement/fix pull request to https://github.com/fastlane/fastlane/pull/4210.
please close other pull request.

Looks good from my side, @mfurtak ready to merge whenever you say it's okay 👍

Thanks folks, this is great! 👍

Woohoo!

Awesome !
On Mon, May 2, 2016 at 7:56 AM Joe Greubel<EMAIL_ADDRESS>wrote:

Woohoo!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/fastlane/fastlane/pull/4425#issuecomment-216257746


Port the frontend to TypeScript

Related to #30

I added a basic model IVideo which you can share in the future with your backend
I ported some class components to function components
I think you can remove propTypes since you're using TS now

Backend can also be ported but requires some extra coordination with the tooling

For some reason travis bailed, I think it'll work if you restart the build.