GenerateJsonSchema2Pojo Does Not Allow DependsOn or Timing

I have a series of subprojects in a multi-project setup.  What I need is to have the generate occur when we would get to step 3 of 7.  Only it always runs before step 1.  This means that the files are not in place (because they need minor modification before generating).  Here is my build.gradle for the generate subproject.
plugins {
    id "com.github.eirnym.js2p" version "1.0"
    id "java-gradle-plugin"
    id "idea"
    id "eclipse"
    id "jacoco"
    id "maven-publish"
    id "java-library"
    id "groovy"
    id "application"
    id "java"
}

repositories {
    maven { url "https://mavenrepo.local.com/nexus/content/groups/public" }
    maven { url "https://mavenrepo.local.com/nexus/content/repositories/releases/" }
    mavenCentral()
}

ext {
    sourceCompatibility = JavaVersion.VERSION_1_8
    targetCompatibility = JavaVersion.VERSION_1_8
}

dependencies {
    compile "javax.validation:validation-api:1.1.0.CR2"
    compile "com.fasterxml.jackson.core:jackson-databind:2.9.7"
    compile "joda-time:joda-time:2.2"
    compile "me.itzg:jsonschema2pojo-rules-bettermaps:1.0.2"
    compile project(":step1-clean")
    compile project(":step2-prepare-files")
}

bootJar {
    enabled = false
}
jar {
    enabled = false
}

project.evaluationDependsOn(":step2-prepare-files")
project.tasks.compileJava.mustRunAfter clean,
        ":step1-clean:myClean",
        ":step2-prepare-files:copyAllV1FilesToTemp",
        ":step2-prepare-files:copyAllV2FilesToTemp",
        ":step2-prepare-files:editV1Files",
        ":step2-prepare-files:editV2Files"

// Config for task generateJsonSchema2Pojo
jsonSchema2Pojo {

    // The target version for generated source files.
    targetVersion = "1.8"

    // Location of the JSON Schema file(s). This may refer to a single file or a directory of files.
    source = files("$projectDir/../step2-prepare-files/TEMP/")

    // Package name used for generated Java classes (for types where a fully qualified name has not
    // been supplied in the schema using the "javaType" property).
    targetPackage = "com.local.consolidated.models.json"

    // Target directory for generated Java source files. The plugin will add this directory to the
    // java source set so the compiler will find and compile the newly generated source files.
    targetDirectory = file("$projectDir/../step3-generate-pojo-from-jsons/TEMPFILES/src/main/java/")

    // The type of input documents that will be read. Supported values:
    //  - jsonschema (schema documents, containing formal rules that describe the structure of JSON data)
    //  - json (documents that represent an example of the kind of JSON data that the generated Java types
    //          will be mapped to)
    //  - yamlschema (JSON schema documents, represented as YAML)
    //  - yaml (documents that represent an example of the kind of YAML (or JSON) data that the generated Java types
    //          will be mapped to)
    sourceType = "jsonschema"

    // Whether to allow "additional" properties to be supported in classes by adding a map to
    // hold these. This is true by default, meaning that the schema rule "additionalProperties"
    // controls whether the map is added. Set this to false to globabally disable additional properties.
    includeAdditionalProperties = true

    // Whether to use primitives (long, double, boolean) instead of wrapper types where possible
    // when generating bean properties (has the side-effect of making those properties non-null).
    usePrimitives = false

    // The characters that should be considered as word delimiters when creating Java Bean property
    // names from JSON property names. If blank or not set, JSON properties will be considered to
    // contain a single word when creating Java Bean property names.
    propertyWordDelimiters = ["_"] as char[]

    // Whether to use the java type long (or Long) instead of int (or Integer) when representing the
    // JSON Schema type "integer".
    useLongIntegers = false

    // Whether to use the java type BigInteger when representing the JSON Schema type "integer". Note
    // that this configuration overrides useLongIntegers
    useBigIntegers = false

    // Whether to use the java type double (or Double) instead of float (or Float) when representing
    // the JSON Schema type "number".
    useDoubleNumbers = true

    // Whether to use the java type BigDecimal when representing the JSON Schema type "number". Note
    // that this configuration overrides useDoubleNumbers
    useBigDecimals = false

    // Whether to include hashCode and equals methods in generated Java types.
    includeHashcodeAndEquals = false

    // Whether to include a toString method in generated Java types.
    includeToString = false

    // The fields to be excluded from toString generation
    toStringExcludes = [] as String[]

    // The style of annotations to use in the generated Java types. Supported values:
    //  - jackson (alias of jackson2)
    //  - jackson2 (apply annotations from the Jackson 2.x library)
    //  - jackson1 (apply annotations from the Jackson 1.x library)
    //  - gson (apply annotations from the Gson library)
    //  - moshi1 (apply annotations from the Moshi 1.x library)
    //  - none (apply no annotations at all)
    annotationStyle = "jackson2"

    // The Level of inclusion to set in the generated Java types for Jackson1
    // and Jackson2 serializers.
    //
    // Supported values
    // - ALWAYS
    // - NON_ABSENT
    // - NON_DEFAULT
    // - NON_EMPTY
    // - NON_NULL
    // - USE_DEFAULTS
    inclusionLevel = "NON_NULL"

    // A fully qualified class name, referring to a custom annotator class that implements
    // org.jsonschema2pojo.Annotator and will be used in addition to the one chosen
    // by annotationStyle. If you want to use the custom annotator alone, set annotationStyle to none.
    customAnnotator = "org.jsonschema2pojo.NoopAnnotator"

    // A fully qualified class name, referring to an class that extends
    // "org.jsonschema2pojo.rules.RuleFactory" and will be used to
    // customRuleFactory = "me.itzg.jsonschema2pojo.bettermaps.BetterMapsRuleFactory"
    customRuleFactory = "org.jsonschema2pojo.rules.RuleFactory"

    // Whether to include JSR-303/349 annotations (for schema rules like minimum, maximum, etc) in
    // generated Java types. Schema rules and the annotation they produce:
    //  - maximum = @DecimalMax
    //  - minimum = @DecimalMin
    //  - minItems,maxItems = @Size
    //  - minLength,maxLength = @Size
    //  - pattern = @Pattern
    //  - required = @NotNull
    // Any Java fields which are an object or array of objects will be annotated with @Valid to
    // support validation of an entire document tree.
    includeJsr303Annotations = true

    // Whether to include JSR-305 (http://jcp.org/en/jsr/detail?id=305) annotations
    // (for schema rules like Nullable, NonNull, etc) in generated Java types.
    includeJsr305Annotations = false

    // Whether to empty the target directory before generation occurs, to clear out all source files
    // that have been generated previously. Be warned, when activated this option
    // will cause jsonschema2pojo to indiscriminately delete the entire contents of the target
    // directory (all files and folders) before it begins generating sources.
    removeOldOutput = true

    // The character encoding that should be used when writing the generated Java source files
    outputEncoding = "UTF-8"

    // Whether to use {@link org.joda.time.DateTime} instead of {@link java.util.Date} when adding
    // date type fields to generated Java types.
    useJodaDates = false

    // Whether to use {@link org.joda.time.LocalDate} instead of string when
    // adding string type fields of format date (not date-time) to generated
    // Java types.
    useJodaLocalDates = false
    useJodaLocalTimes = false

    // Whether to add JsonFormat annotations when using Jackson 2 that cause format "date", "time", and "date-time"
    // fields to be formatted as yyyy-MM-dd, HH:mm:ss.SSS and yyyy-MM-dd"T"HH:mm:ss.SSSZ respectively. To customize these
    // patterns, use customDatePattern, customTimePattern, and customDateTimePattern config options or add these inside a
    // schema to affect an individual field
    formatDateTimes = true
    formatDates = true
    formatTimes = true

    // Whether to initialize Set and List fields as empty collections, or leave them as null.
    initializeCollections = true

    // Whether to add a prefix to generated classes.
    classNamePrefix = ""

    // Whether to add a suffix to generated classes.
    classNameSuffix = ""

    // An array of strings that should be considered as file extensions and therefore not included in class names.
    fileExtensions = ["v1.json", "v2.json"] as String[]

    // Whether to generate constructors or not.
    includeConstructors = false

    // Whether generated constructors should have parameters for all properties,
    // or only required ones.
    constructorsRequiredPropertiesOnly = false

    // Whether to make the generated types Serializable
    serializable = true

    // Whether to include getters or to omit these accessor methods and create public fields instead.
    includeGetters = false

    // Whether to include setters or to omit these accessor methods and create public fields instead.
    includeSetters = false

    // Whether to generate builder-style methods of the form withXxx(value) (that return this),
    // alongside the standard, void-return setters.
    generateBuilders = false

    // Whether to include dynamic getters, setters, and builders or to omit these methods.
    includeDynamicAccessors = false

    // Whether to include dynamic getters or to omit these methods.
    includeDynamicGetters = false

    // Whether to include dynamic setters or to omit these methods.
    includeDynamicSetters = false

    // Whether to include dynamic builders or to omit these methods.
    includeDynamicBuilders = false

    // What type to use instead of string when adding string properties of format "date" to Java types
    dateType = "java.time.LocalDate"

    // What type to use instead of string when adding string properties of format "date-time" to Java types
    dateTimeType = "java.time.LocalDateTime"

    // What type to use instead of string when adding string properties of format "time" to Java types
    timeType = "java.time.LocalTime"

    // A custom pattern to use when formatting date fields during serialization.
    // Requires support from your JSON binding library.
    customDatePattern = "yyyy-mm-dd"

    // A custom pattern to use when formatting date-time fields during
    // serialization. Requires support from your JSON binding library.
    customDateTimePattern = "hh:mm:ss"

    // A string containing any characters that should act as path delimiters when resolving $ref fragments.
    // By default, #, / and . are used in an attempt to support JSON Pointer and JSON Path.
    refFragmentPathDelimiters = "#/."

    // The sort order to be applied when recursively processing the source
    // files.  By default the OS can influence the processing order.   Supported values:
    //
    // - OS (Let the OS influence the order the source files are processed.)
    // - FILES_FIRST (Case sensitive sort, visit the files first.  The source files are processed in a breadth first sort order.)
    // - SUBDIRS_FIRST (Case sensitive sort, visit the sub-directories before the files.  The source files are processed in a depth first sort order.)
    sourceSortOrder = "OS"

    // Use builders on inner classes (creates an inner class that is classname + builder sort of like
    // lombok except you still need to new to inner class)
    useInnerClassBuilders = false

    // Constructor properties
    includeConstructorPropertiesAnnotation = false

    // Use the title as class name. Otherwise, the property and file name is used.
    useTitleAsClassname = false

    // Including type info
    includeJsonTypeInfoAnnotation = false

    // Required propert constructor
    includeRequiredPropertiesConstructor = false

    // All properties constructor
    includeAllPropertiesConstructor = false

    // Copy Instructor
    includeCopyConstructor = false

    // Optional for getters
    useOptionalForGetters = false

    // Parcable
    parcelable = false

    // Use Common Lang 3
    useCommonsLang3 = false

}

You can see I have tried a couple of things
project.evaluationDependsOn(":step2-prepare-files")
project.tasks.compileJava.mustRunAfter clean,
        ":step1-clean:myClean",
        ":step2-prepare-files:copyAllV1FilesToTemp",
        ":step2-prepare-files:copyAllV2FilesToTemp",
        ":step2-prepare-files:editV1Files",
        ":step2-prepare-files:editV2Files"

But these do not change when it runs.
How can I set this so it runs after the step2 subproject?

You probably need to remove the default dependsOn relationship to establish your own ordering. I forget the exact DSL syntax, but its something like abc.dependsOn.remove('xyz').
https://github.com/eirnym/js2p-gradle/blob/48448299fde75215819f643b6b8be2a40cab1e98/src/main/groovy/com/github/eirnym/js2p/GenerateJsonSchemaJavaTask.groovy#L57-L58

You probably need to remove the default dependsOn relationship to establish your own ordering. I forget the exact DSL syntax, but its something like abc.dependsOn.remove('xyz').
https://github.com/eirnym/js2p-gradle/blob/48448299fde75215819f643b6b8be2a40cab1e98/src/main/groovy/com/github/eirnym/js2p/GenerateJsonSchemaJavaTask.groovy#L57-L58

@ben-manes adding processResources dependsOn my previous subproject has done the trick.  Thanks.

@ben-manes adding processResources dependsOn my previous subproject has done the trick.  Thanks.

Thank you for a well-defined issue
I'll review my plugin and follow suggestions.

I'm not sure what happens so much, but if it comes from the default convention, we can relax it by splitting capabilities and conventions by providing a base plugin.

@PRAquilone there's a lot of things changed inside the plugin lately. Could you recheck if something left to fix here?

I can't reproduce the issue anymore. Please reopen if it's not fixed

I am trying to execute another task before generateJsonSchema2DataClass but everything returns an error.
I tried the following options:

generateJsonSchema2DataClass.dependsOn(myTask)
tasks.generateJsonSchema2DataClass.dependsOn(myTask)
project.tasks.generateJsonSchema2DataClass.dependsOn(myTask)
generateJsonSchema2DataClass.configure { dependsOn myTask }
tasks.withType(generateJsonSchema2DataClass).configureEach {  dependsOn(myTask) }

I am always getting the following error: Could not get unknown property 'generateJsonSchema2DataClass' for task set of type org.gradle.api.internal.tasks.DefaultTaskContainer.
except for the last two options which return:
Could not get unknown property 'generateJsonSchema2DataClass' for project ':objects' of type org.gradle.api.Project.

@sergeykad please, open a new issue with it

Hi! I'm getting this issue as well. Was there a new ticket created? If so, could it be linked here?

@corbetmw hi there
Could you describe the exact thing you're trying to achieve?

After meditating on Gradle API for a while, I found a way to make generated tasks discoverable for dependsOn and other magic you need.
This was implemented only for Java projects for now and later on I'll enable it for Android ones.
Generated tasks still always depend on resource generation and java/kotlin compilation is always depends on generated tasks.


Allow can? check on a single resource with a relation query

Closes #442

I couldn't get the spec to run locally, so I thought I would just create the PR and see what Travis says :-)

Please first fix the CI, then I'll look into it. Look at the README to know how to run tests locally using Appraisal. Thank you

Closing due to lack of feedback. Feel free to fix the CI and open again. Thank you.


how to get mAP and precision and recall values for tensorflow object detection API?

I have created a custom TensorFlow object detection api, now I want to get the
values of mAP and precision and recall? how?
using your video on youtube
https://www.youtube.com/watch?v=HjiBbChYRDw
@TannerGilbert

You can use the COCO detection metrics.

https://github.com/tensorflow/models/blob/master/research/object_detection/g3doc/evaluation_protocols.md#coco-detection-metrics
https://stackoverflow.com/questions/52982574/calculating-the-map-from-scores-obtained-in-tensorflow-object-detection-api


hanging on Jenkins is starting, Please wait

2.46.2
Docker commands that you execute
install plugins
docker restart jenkins2
Actual result
FINE: created build reference for deal-file-server #64 using jenkins.model.lazy.BuildReference$DefaultHolderFactory@ced1267
Jul 18, 2017 3:23:04 PM jenkins.security.ExceptionTranslationFilter doFilter
FINER: Chain processed normally
Jul 18, 2017 3:23:05 PM jenkins.model.lazy.BuildReference findHolder
FINE: created build reference for deal-web-data-server2 #38 using jenkins.model.lazy.BuildReference$DefaultHolderFactory@ced1267
Jul 18, 2017 3:23:06 PM jenkins.model.lazy.BuildReference findHolder
FINE: created build reference for deal-xiaoshouyi-server #378 using jenkins.model.lazy.BuildReference$DefaultHolderFactory@ced1267
Jul 18, 2017 3:23:06 PM jenkins.model.lazy.BuildReference findHolder
FINE: created build reference for deal-calendar-server #34 using jenkins.model.lazy.BuildReference$DefaultHolderFactory@ced1267

@michaelneale @carlossg @rtyler @kohsuke @ndeloof

please describe plugins you installed
please also docker exec in container to capture a thread dump

@zsjohny Did you solve the problem, Did you solve the problem?

@zsjohny I have the same problem, Did you solve the problem?

Not enough info to diagnose the issue

@KevinLoveGitHub if you have same issue please capture a java stack trace / container state to help understand the root cause.

@ndeloof Thanks，I changed the container network mode to host mode to solve this problem

not enough info


Hardcode more valid modifications commands

Codecov Report

Merging #37 (df07945) into master (117f847) will increase coverage by 70.22%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #37       +/-   ##
===========================================
+ Coverage        0   70.22%   +70.22%
===========================================
  Files           0       12       +12
  Lines           0      262      +262
  Branches        0       34       +34
===========================================
+ Hits            0      184      +184
- Misses          0       75       +75
- Partials        0        3        +3

Impacted Files
Coverage Δ

validator/schema/modification_schema.py
100.00% <ø> (ø)

validator/exceptions.py
100.00% <0.00%> (ø)

validator/schema/releases_schema.py
0.00% <0.00%> (ø)

validator/schema/__init__.py
100.00% <0.00%> (ø)

validator/format.py
100.00% <0.00%> (ø)

validator/support.py
82.22% <0.00%> (ø)

validator/schema/rpm_schema.py
100.00% <0.00%> (ø)

validator/distgit.py
100.00% <0.00%> (ø)

validator/github.py
100.00% <0.00%> (ø)

... and 3 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 117f847...df07945. Read the comment docs.


tests: Enable travis CI.

This enables Travis CI, which currently builds SMAUG and runs all the tests.

@xyzsam Do you know how to run CI automatically for PR?

I had to initialize the Travis CI project for this repo. It's active now. I kicked off a build to get things started. You may need to make a trivial change and repush to trigger the build.

What you can do is update the Travis CI badge in the README to this link: https://travis-ci.org/harvard-acc/smaug.svg?branch=master. That should get the process going.

What you can do is update the Travis CI badge in the README to this link: https://travis-ci.org/harvard-acc/smaug.svg?branch=master. That should get the process going.

Yes, added a commit for updating the Travis CI badge.

Going to rebase and merge to keep the experiments submodule commit separate.


KYLIN-4614 Broadcaster exclude itself

Pull Request Test Coverage Report for Build 6090

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
8 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.0006%) to 27.974%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-metadata/src/main/java/org/apache/kylin/metadata/cachesync/Broadcaster.java
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
3
78.42%

tool/src/main/java/org/apache/kylin/tool/query/ProbabilityGenerator.java
3
77.78%

Totals

Change from base Build 6065:
-0.0006%

Covered Lines:
26109

Relevant Lines:
93333

💛  - Coveralls

Close this temporarily.


frontend: Admin ui

This covers all the administrative actions for the circulation management, for example loan a book, return ...

Implemented in https://github.com/inveniosoftware/invenio-circulation/pull/44, https://github.com/inveniosoftware/invenio-circulation/pull/43, https://github.com/inveniosoftware/invenio-circulation/pull/41, https://github.com/inveniosoftware/invenio-circulation/pull/39. Are we ready to close this one or there is some missing functionality? @david-caro @egabancho @jirikuncar @mvesper


JSONDecodeError: Expecting value: line 1 column 1 (char 0) in api/api_usage.ipynb

Hi,

I set up NEMO as instructed. I was able to set up docker successfully.
I tried running the api/api_usage.ipynb notebook.
The "Run NER Model" section works as expected, but the rest of the models result in the following error:
JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I'm guessing that yap isn't available. Can you look at the docker output and share the error you see there?

nemo_1  | INFO:     <IP_ADDRESS>:64696 - "POST /multi_align_hybrid HTTP/1.1" 500 Internal Server Error
nemo_1  | ERROR:    Exception in ASGI application
nemo_1  | Traceback (most recent call last):
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/uvicorn/protocols/http/h11_impl.py", line 373, in run_asgi
nemo_1  |     result = await app(self.scope, self.receive, self.send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__
nemo_1  |     return await self.app(scope, receive, send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/fastapi/applications.py", line 208, in __call__
nemo_1  |     await super().__call__(scope, receive, send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/applications.py", line 112, in __call__
nemo_1  |     await self.middleware_stack(scope, receive, send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/middleware/errors.py", line 181, in __call__
nemo_1  |     raise exc from None
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/middleware/errors.py", line 159, in __call__
nemo_1  |     await self.app(scope, receive, _send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/exceptions.py", line 82, in __call__
nemo_1  |     raise exc from None
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/exceptions.py", line 71, in __call__
nemo_1  |     await self.app(scope, receive, sender)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/routing.py", line 580, in __call__
nemo_1  |     await route.handle(scope, receive, send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/routing.py", line 241, in handle
nemo_1  |     await self.app(scope, receive, send)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/routing.py", line 52, in app
nemo_1  |     response = await func(request)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/fastapi/routing.py", line 219, in app
nemo_1  |     raw_response = await run_endpoint_function(
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/fastapi/routing.py", line 154, in run_endpoint_function
nemo_1  |     return await run_in_threadpool(dependant.call, **values)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/starlette/concurrency.py", line 40, in run_in_threadpool
nemo_1  |     return await loop.run_in_executor(None, func, *args)
nemo_1  |   File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
nemo_1  |     result = self.fn(*self.args, **self.kwargs)
nemo_1  |   File "./api_main.py", line 356, in multi_align_hybrid
nemo_1  |     ma_lattice = run_yap_hebma(tok_sents)
nemo_1  |   File "./api_main.py", line 84, in run_yap_hebma
nemo_1  |     resp = yap_request('/yap/heb/ma', data)
nemo_1  |   File "./api_main.py", line 78, in yap_request
nemo_1  |     return requests.get(url+route, data=data, headers=headers).json()
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/requests/api.py", line 75, in get
nemo_1  |     return request('get', url, params=params, **kwargs)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/requests/api.py", line 61, in request
nemo_1  |     return session.request(method=method, url=url, **kwargs)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/requests/sessions.py", line 542, in request
nemo_1  |     resp = self.send(prep, **send_kwargs)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/requests/sessions.py", line 655, in send
nemo_1  |     r = adapter.send(request, **kwargs)
nemo_1  |   File "/usr/local/lib/python3.9/dist-packages/requests/adapters.py", line 516, in send
nemo_1  |     raise ConnectionError(e, request=request)
nemo_1  | requests.exceptions.ConnectionError: HTTPConnectionPool(host='yap', port=8000): Max retries exceeded with url: /yap/heb/ma (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f75c65257c0>: Failed to establish a new connection: [Errno -2] Name or service not known'))

Do I need to set up YAP before I docker-compose up?

No, docker compose builds and starts both yap and nemo. There is a separate containers for each. Is the yap container running? Can you share the full output of docker compose up?

I've just restarted it.
(base) sapir@sapir-MBP NEMO % docker-compose up
Starting nemo_yap_1 ... done
Starting nemo_nemo_1 ... done
Attaching to nemo_yap_1, nemo_nemo_1
yap_1   | 2021/08/20 17:18:21.909286 GOMAXPROCS:	4
yap_1   | 2021/08/20 17:18:21.920549
yap_1   | 2021/08/20 17:18:21.922557 Configuration
yap_1   | 2021/08/20 17:18:21.922603 Heb Lexicon:		/yap/src/yap/data/bgulex/bgupreflex_withdef.utf8.hr
yap_1   | 2021/08/20 17:18:21.922618 Heb Prefix:		/yap/src/yap/data/bgulex/bgulex.utf8.hr
yap_1   | 2021/08/20 17:18:21.922631 OOV Strategy:	Const:NNP
yap_1   | 2021/08/20 17:18:21.922661 xliter8 out:		false
yap_1   | 2021/08/20 17:18:21.922676
yap_1   | 2021/08/20 17:18:21.922704
yap_1   | 2021/08/20 17:18:21.922720 Reading Morphological Analyzer BGU Prefixes
yap_1   | 2021/08/20 17:18:21.923764 Found 102 tokens in lexicon file: /yap/src/yap/data/bgulex/bgupreflex_withdef.utf8.hr
yap_1   | 2021/08/20 17:18:21.923821 Loaded 101 prefixes from lexicon
yap_1   | 2021/08/20 17:18:21.923837 Reading Morphological Analyzer BGU Lexicon
nemo_1  | INFO:     Started server process [1]
nemo_1  | INFO:     Waiting for application startup.
yap_1   | 2021/08/20 17:18:24.318335 Found 514852 tokens in lexicon file: /yap/src/yap/data/bgulex/bgulex.utf8.hr
yap_1   | 2021/08/20 17:18:24.443766 Loaded 514852 tokens from lexicon
yap_1   | 2021/08/20 17:18:24.443823
yap_1   | 2021/08/20 17:18:24.444684 Configuration
yap_1   | 2021/08/20 17:18:24.444765 Beam:		Standard Beam [Not Aligned & Not Averaged]
yap_1   | 2021/08/20 17:18:24.444782 Transition System:	Morpheme-Based Morphological Disambiguator
yap_1   | 2021/08/20 17:18:24.444804 Iterations:		1
yap_1   | 2021/08/20 17:18:24.444819 Beam Size:		64
yap_1   | 2021/08/20 17:18:24.444836 Beam Concurrent:	true
yap_1   | 2021/08/20 17:18:24.444865 Parameter Func:	Funcs_Main_POS_Both_Prop
yap_1   | 2021/08/20 17:18:24.444878 Use POP:		true
yap_1   | 2021/08/20 17:18:24.444894 Infuse Gold Dev:	false
yap_1   | 2021/08/20 17:18:24.444911 Use Lemmas:		false
yap_1   | 2021/08/20 17:18:24.444925 Use CoNLL-U:		false
yap_1   | 2021/08/20 17:18:24.444963 No NNP Feat:		false
yap_1   | 2021/08/20 17:18:24.444981 Limit:		0
yap_1   | 2021/08/20 17:18:24.445002 Model file:		/yap/src/yap/data/md_model_temp_i9.b64
yap_1   | 2021/08/20 17:18:24.445032
yap_1   | 2021/08/20 17:18:24.445059 Features File:	/yap/src/yap/conf/standalone.md.yaml
yap_1   | 2021/08/20 17:18:24.445084
yap_1   | 2021/08/20 17:18:24.445101 Data
yap_1   | 2021/08/20 17:18:24.446731 Loading MD transition dependent feature group Past Morphemes Unigram
yap_1   | 2021/08/20 17:18:24.446804 Loading MD transition dependent feature group Past Morphemes Bigram
yap_1   | 2021/08/20 17:18:24.446856 Loading MD transition dependent feature group Past Morphemes Trigram
yap_1   | 2021/08/20 17:18:24.446894 Loading MD transition dependent feature group Next Lattice Unigram
yap_1   | 2021/08/20 17:18:24.446918 Loading MD transition dependent feature group Current Lattice Unigram
yap_1   | 2021/08/20 17:18:24.446956 Loading MD transition dependent feature group Previous Lattice Unigram
yap_1   | 2021/08/20 17:18:24.447016 Loading MD transition dependent feature group Next Lattice Bigram
yap_1   | 2021/08/20 17:18:24.447054 Loading MD transition dependent feature group Previous Lattice Bigram
yap_1   | 2021/08/20 17:18:24.447110 Loading POP transition dependent feature group POP
yap_1   | 2021/08/20 17:18:24.447134 Loading Lexical transition dependent feature group Lexical
yap_1   | 2021/08/20 17:18:24.447158
yap_1   | 2021/08/20 17:18:24.447186 Using Family HEBTB of Main_POS_Types [ [ADVERB BN BNT CD CDT JJ JJT NN NNP NNT RB VB] ]
yap_1   | 2021/08/20 17:18:24.447208
yap_1   | 2021/08/20 17:18:24.447244 Found MD model file /yap/src/yap/data/md_model_temp_i9.b64  ... loading model
nemo_yap_1 exited with code 137
nemo_1  | INFO:     Application startup complete.
nemo_1  | INFO:     Uvicorn running on http://<IP_ADDRESS>:8090 (Press CTRL+C to quit)

This is an out of memory error (137). Never had this happen for these containers. Docker is shutting down the yap containers while it's trying to load its models. Maybe you have other containers running that you can free up? I advise you try to google it a bit for possible solutions.

What system are you running on? Operating system, CPUs, RAM?

I'm using a mac, 16 GB RAM. Anyway, I'm able to use NEMO's API without docker. (Just started NEMO and YAP with the instructions and it's okay).

Oh that should be more than enough. Good to hear you made it work for you. Closing this for now


Removed unused buf_size argument to LimitedStream().

Unused since its introduction in 269e921756371bee6d35a967bc2ffe84d1ae39eb.

@ngnpope Thanks :+1:


WIP: UPSTREAM: 113971: wire context in rbac

This wires context into rule resolving in the authorization subsystem.
This is needed to fix https://github.com/kcp-dev/kcp/issues/1340

Wrong title for PR?

oh dear, yes thank you :D

xref https://github.com/kubernetes/kubernetes/pull/113971

upstream contribution is submitted https://github.com/kubernetes/kubernetes/pull/113971

/retitle UPSTREAM: 113971: wire context in rbac


Spring cloud security with CAS

Is there a plan to support CAS authentication with spring-cloud-security?

Please explain more in depth what you mean.

Sorry for being not clear. So, to configure CAS based authentication we need to configure a lot of beans like CasAuthenticationFilter, TicketValidator, AuthenticationProvider, SingleSignOutFilter etc.,
Is there a plan to make these configurations in-built with spring-cloud-security in order to avoid fuss in the code. Does this make sense?

Closing this due to inactivity. Please re-open if there's more to discuss.


Add command for ToDo, Deadline and Event

Codecov Report

Merging #138 (92240df) into master (f9a4163) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #138      +/-   ##
============================================
+ Coverage     66.77%   66.79%   +0.02%
  Complexity      478      478
============================================
  Files            91       91
  Lines          1610     1611       +1
  Branches        191      191
============================================
+ Hits           1075     1076       +1
  Misses          476      476
  Partials         59       59

Impacted Files
Coverage Δ

src/main/java/taskbook/logic/parser/CliSyntax.java
90.90% <100.00%> (+0.90%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


chore(main): release sinkchart 1.2.1

:robot: I have created a release beep boop
1.2.1 (2022-12-02)
Bug Fixes

represent non-prod dependencies (5e04c15)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sinkchart/releases/tag/sinkchart-v1.2.1 :sunflower:


Do not swallow errors on ec downloads

There's a script which is responsible for setting up the download service for Enterprise Contract. This script runs as part of the initContainer for the Deployment.
Prior to this commit, if there were any errors while executing the script, the script would wait forever.
This commit changes the behavior to allow the script to exit on errors and report some information about the failure. The idea is that if there is some sort of timing issue, OpenShift will retry executing the container again. This leverages that instead of entering a pseudo-deadlock state.
This is an attempt to fix RHTAPBUGS-1106, or to at least provide more information if the error does happen again.

@robnester-rh, needs the lgtm label. Can you comment with /lgtm ?
(This repo is super annoying...)

/lgtm


[feature request] Always on top player window

I'd like to suggest a player option to keep the window always on top during playback, there could also be the option to keep the player always on top only when it's actually reproducing something and not when paused.

There is View -> Float on Top. You want it to only be active during playback? This seems a bit confusing to me, do you know of any player that does this already?

I missed the View -> Float on Top option, I only searched under Window, my fault. The player I'm using is mpv and it has a lua script to achieve this behaviour here, there's also SMPlayer that does this. Essentially it only keeps the window on top of everything during playback and not when player is paused.

@devster31 This sounds potentially useful if you understand what's happening. But this feature seems hard to explain to new users, and the way it works also wouldn't be apparent immediately.
I could see this as a new option -- "Float on Top During Playback" in the menu. But this seems really niche, and maybe it's best left to be implemented as a plugin once we have support for that.


You should not use  outside a

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[ x] I am running the latest version
[ x] I checked the documentation and found no answer
[ x] I checked to make sure that this issue has not already been filed
[ x] I'm reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior
Demo should run after npm install and npm start
Current Behavior
Throws this error: "You should not use  outside a "
Failure Information (for bugs)
invariant
E:\material-dashboard\node_modules\babel-loader\lib\index.js??ref--6-oneOf-2!E:\material-dashboard\node_modules\invariant\browser.js:34
31 | } else {
32 |   var args = [a, b, c, d, e, f];
33 |   var argIndex = 0;

34 |   error = new Error(format.replace(/%s/g, function () {
| ^  35 |     return args[argIndex++];
36 |   }));
37 |   error.name = 'Invariant Violation';

render
E:\material-dashboard\node_modules\babel-loader\lib\index.js??ref--6-oneOf-2!E:\material-dashboard\node_modules\react-router-dom\es\Link.js:121
118 |   return React.createElement("a", _extends({}, props, {
119 |     onClick: this.handleClick,
120 |     href: href,

121 |     ref: innerRef
| ^  122 |   }));
123 | };
124 |

Steps to Reproduce

download dashboard
npm install
npm run

Context
Windows 7
NPM 5.6.0
Node 8.11.1
Browser Firefox 65
Terminal Cygwin Bash
Firewall: no firewall
Theme: default
Framework React
IP: private
Failure Logs
All Files of this repository and npm install dependencies

I have same problem :(

I found a solution. After I updated react-router-dom to version 4.4.0 - problem disappeared.

Thanks @rafull6
For any beginners like me, do the following:

In package.json find this line: "react-router-dom": "4.3.1",
Replace it with this line:    "react-router-dom": "4.4.0",
npm update
npm start

Done ;)

Hello there, guys,
Thank you for your interest in working with our products.
Sorry for the late response, we do not work on weekends.
In our package.json file we only install "react-router-dom". This package auto-installs "react-router". The problem is that is installing the latest version of "react-router", instead of the same version as "react-router-dom", and that is why you are having this error.
Run the following command:
npm i -E<EMAIL_ADDRESS>
And this will do the trick.
Best,
Manu


JWT auth for blogger gives error 403

I try to use JWT with Blogger API
var jwtClient = new google.auth.JWT(serviceAccount,
        /path/to/key
        null,
        ['https://www.googleapis.com/auth/blogger']
    ),
    blogger = google.blogger({version: 'v3', auth: jwtClient});

jwtClient.authorize(function(err, tokens) {
    blogger.posts.insert({
        blogId: 'id-number',
        title: 'title here',
        content: 'Just a simple test'
    }, function(err, resp) {
        if (err) throw err;
        console.log('Post inserted!');
    });
});

but get error { [Error: We're sorry, but you don't have permission to access this resource.] code: 403 }.
blogger.posts.get works just fine. The given scope allows to view and manage posts. So, I'm not clear what is wrong.

I'm have a similar issue when uploading a video to Youtube. If I use the FWT authentication mode, I'm getting a 401, however I can perform read operations correctly using FWT.
It's seems like some write operations are not support by this type of authentication and requires end user validation using OAuth. For server-to-server automation this is very painful.
Could someone confirm this?

You cannot use JWT for these types of requests. Any requests that is public can be authenticated in any way (e.g. API Key, OAuth2, JWT) however if you need to manage anything private (such as create a post, delete a post) you must be authenticated with OAuth2.
See here: https://developers.google.com/blogger/docs/3.0/using#auth
It also makes sense that you need to authenticate with a real Google account using OAuth2 rather than JWT to upload a youtube video because the youtube video is associated with a specific Google account.
Each API has their own section on authorization that is independent of the other APIs. You'll need to read the docs for each API in order to asses what your application needs in order to authenticate properly.

Thanks for the answer.
Therefore how it's possible to automate server-to-server tasks using OAuth when a web browser cannot we available? I've some possible, but all are ugly workarounds, such as using PhantomJS

I would suggest authenticating with OAuth2 using a browser and then storing the credentials (access token and refresh token) in a safe place and using them in your application. If your access token expires, use your refresh token to get a new one. Uploading a YouTube video shouldn't be a strictly server-to-server thing, because an end user needs to authenticate their account anyway. Even if that's your own YouTube account.
See here for the different flows you could use to accomplish that: https://developers.google.com/youtube/v3/guides/authentication#OAuth2_Flows

:+1: Thanks @ryanseys.
I had to spin up selenium server to did browser oauth. Will try to use refresh token.

Thanks for the tips.
I've created this packages to provide automation and remove headaches when dealing with OAuth2:
https://github.com/h2non/nightmare-google-oauth2
https://github.com/h2non/google-oauth2-token (CLI only)

@ryanseys I followed this sample code to get access token, setting access_type = 'offline' and scope = 'https://www.googleapis.com/auth/blogger', but I couldn't get the refresh token. Here's what I got back:
{
  access_token: 'ya29.xxx',
  token_type: 'Bearer',
  expiry_date:<PHONE_NUMBER>976
}

Do you know why so?


Scatterplot: click on legend should filter out points

Short description
When I click on a legend item in the Scatterplot, the category should be filtered out of the plot.
Maybe switch to the new legend right away, instead of using a plotly legend?

I just discovered, filtering by interacting with the legend no longer works, neither in the hexbin plot nor in the new scatter plot implementation


deprecate and archive this project

Is anyone aware of a CosmosDB session provider (using the more recent generation of @azure/cosmos libs like v3.6.2?
Just wondering about alternatives or half-baking my own...


(doc) Correcting minor typo.

 Issue by gep13
Monday Dec 12, 2016 at 09:24 GMT
Originally opened as https://github.com/Wyamio/Wyam.Web/pull/31

gep13 included the following code: https://github.com/Wyamio/Wyam.Web/pull/31/commits

 Comment by daveaglick
Monday Dec 12, 2016 at 22:01 GMT

:thumbsup:


(v0.9.6-release) Set parent result based on child result

fix the false positive issue caused by TAP parser

Fixes #4237
Signed-off-by: renfeiw<EMAIL_ADDRESS>

cherry pick: https://github.com/adoptium/aqa-tests/pull/4255


(DNM) Adding Unavailable Menu Item docs

✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[X] Filled out test instructions.
[X] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Goto the MenuTrigger Docs
Review the "Unavailable Menu Item" section
🧢 Your Project:
RSP

Should this be under Menu in addition to or rather than MenuTrigger?

Rob suggested this be added to MenuTrigger and we later Merge Menu and MenuTrigger docs.

I'm not seeing the popover in the docs for some reason.

This is the bug Rob warned us about. The position is wrong.


Contributing.MD and  CODE_OF_CONDUCT.md are almost the same

Contributing.MD and  CODE_OF_CONDUCT.md are almost the same so can be separated.

Oops! Seems like I copied in the wrong template. Thanks for pointing it out. I will fix it ASAP.
If you have any suggestions for what would help you contribute to this project more easily please feel free to make edits to the CONTRIBUTING.md file and raise a PR.

PR #4 updates the docs and improves the CONTRIBUTIONS.md file for an easier developer experience.


QSwitch: Use *OPC? instead of delays

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

Comparison is base (27abe4e) 21.53% compared to head (2fafb9a) 21.53%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #283   +/-   ##
=======================================
  Coverage   21.53%   21.53%
=======================================
  Files         163      163
  Lines       19361    19362    +1
=======================================
+ Hits         4169     4170    +1
  Misses      15192    15192

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


Add Files to Build Log

Agent Version and Platform
Version 2.122.1
Windows
VSTS Type and Version
On-Prem TFS
TFS 2018
What's not working?
I use Cake and MSBuild Binary Log. Currently the agent captures all console output and upload it to the server, but I also want to upload binary logs to the server. Is there a way to add a file to build log?
JobServerQueue.cs#L189 seems to be the method used by agent, but I don't know how I can use it.

@DSakura1987 try use the ##vso command from following doc
https://github.com/Microsoft/vsts-tasks/blob/master/docs/authoring/commands.md
##vso[task.uploadfile]local file path


Cannot Instantiate Package

Cannot instantiate the package due to the following error.
Repro Steps

Julia version 1.11.1
Got to RiskMDPs.jl folder
run the following command

julia --project=. -e 'using Pkg; Pkg.instantiate()'

Observe Error Below

Does this work for anyone else?

Forgot to read the README :1st_place_medal:

Still doesn't work?

Ok for some reason the following works (run it within the RiskMDPs directory.
julia
]activate .
]develop MDPs


Fix VB code generator

This is the pull request to fix #358

Hmmm... this looks like a regression in the VB compiler. The build integration tests for VB are clearly running and passing yet I can see in the latest VS 2019 builds same as you, that it introduces a compiler error.

I am using the latest VS2017 version (15.9.21) and I see the error. It also is correct that the error is shown, because it is a syntax error. According to the documentation the attributes have to be "on the same line" as the declaration. Breaking the line only works because it utilizes the implicit line continuation. The preprocessor statements interrupt the implicit continuation and so the error is correct.

It's definitely a vb compiler or .NET SDK issue. It only repros for .NET SDK style projects.
I filed https://github.com/dotnet/roslyn/issues/36365
In the meantime though, of course we want NB.GV to work for VB .NET SDK style projects, so we'll take the PR. :)


Duplicate Ubisoft Connect Entry made by NSLGameScanner.py

I noticed that I had two Ubisoft Connect entries every time I installed NonSteamLaunchers on my Steam Deck. I found out that it was the Battle Net scanner portion adding an 'unknown' game titled Ubisoft Connect as a duplicate entry. I ultimately found a file that was being identified. "C:/Program Files (x86)/Ubisoft/Ubisoft Game Launcher/retail/.flavor.info" and the "Flavor inferred was : unknown". Then it would say No exact match found. Using game ID of the first result: Ubisoft Connect (Program): 5270094.  It would subsequently add all of the Battle.net information for the launch of this game - that's what lead me to the Battle.net scanner failing and leading to this. The launch parameter would be --exec="launch unknown". The Launch options would be %command% "battlenet://unknown""
It looked like you were already working on a possible overhaul during my quick review of various issues open. Not sure if this is helpful but it fixed my duplicate entry problem.
Cheers.

I noticed with the original submission all Battle.net entries installed would not get automatically added to Steam as originally intended. My second submission accounts for this and still prevents a duplicate Ubisoft Connect entry.

@entropydo nice catch! Thank you!!!


[ML category based PROJECT PROPOSAL]-Early heart disease

Related Issue
#[ML category based PROJECT PROPOSAL] #9

Developing a model to predict early heart disease prediction by doing a comparitive analysis of different machine learning models to predcit heart disease with highest accuracy on defined and most important attributes
[x] GSSoC Participant
[x] Contributor

Closes: #9
Describe the changes you've made
A new folder consisting of Dataset of heart disease patient, Model file consisting of all models running on the dataset and a README file to understand what project is doing has been added to the branch.
The project involved analysis of the heart disease patient dataset with proper data processing. Then, different models was trained and predictions made with different algorithms KNN, Decision Tree, Random Forest,SVM,Logistic Regression etc. Variety of Machine Learning algorithms implemented in Python to predict the presence of heart disease in a patient. This is a classification problem, with input features as a variety of parameters, and the target variable as a binary variable, predicting whether heart disease is present or not.
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

This feature is satisfying the ML related projects described under World-of-AI README file.
How Has This Been Tested?
Different algorithms has been tested over the training dataset and validate the prediction over the testing dataset. The most accurate algorithms among  KNN, Decision Tree, Random Forest,SVM,Logistic Regression is Random Forest with 95%.
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshots

@adithya-s-k look into the request and feel free to share if you have any issues.

Ohkay, I will update it.

sure thing
also @poorvika11 the project structure has changed a little bit
please make sure you put your project under the machine learning folder

Can you please share the link of the appropriate structure?

Can you please share the link of the appropriate structure?

the project will be under the machine_learning folder
that's the only change

Ohkay
On Wed, May 24, 2023, 2:28 PM Adithya S K @.***> wrote:

Can you please share the link of the appropriate structure?
the project will be under the machine_learning folder
that's the only change
—
Reply to this email directly, view it on GitHub
https://github.com/CognitiveLab-tech/World-of-AI/pull/21#issuecomment-1560719587,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR5RGM7IOMSLM2FY6K6C4KLXHXESXANCNFSM6AAAAAAYJIE5R4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@adithya-s-k I have made another PL with all the changes included i.e. README file, branch folder and images set.

@poorvika11 shall i close this PR?


Add npm package name to npm install command :)

I just realized the way you are supposed to install it is first clone the repo. So my PR is not correct.


Feature/limit aoi size

Overview
Adds a new check in the analysis map view that validates the size of a geostore is less than 1 billion hectares. If true the save in my gfw button which opens the save area modal is disabled and displays a tooltip explaining why. Only applies when looking at a custom area.
Demo

Testing

draw a massive area and try and save it
analyze a country, wdpa, use and make sure you can still save those

Check my comments, otherwise looks good 👍 At some point would be top to do the same for widgets perhaps?

Updated. Going to merge.


Added support to configure database connection.

Very handy to log all activities for the application having multiple database connections.

👍

Is it possible to accomplish this by using a custom model and specify the connection in it's static bootmethod?

yeah, setting connection using setConnection() method in the constructor, using custom model is working fine.
public function __construct()
    {
        parent::setConnection('connection_name');
    }

But this feature which I've suggested is quite handy as well and easy to configure. any thoughts @freekmurze ?

Gonna hold of on this for now, as this behaviour is also easily implemented using a custom model.


Impossible to connect to Github with Vscode nor to access the extension

Type: Bug
I can't connect to github and search for extensions.
VS Code version: Code 1.70.2 (e4503b30fc78200f846c62cf8091b76ff5547662, 2022-08-16T05:35:13.448Z)
OS version: Windows_NT x64 10.0.22000
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8300H CPU @ 2.30GHz (8 x 2304)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.82GB (2.99GB free)

Process Argv
--crash-reporter-id 113f25ad-b839-4712-901e-2aa329596b23

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

vscode-language-pack-fr
MS-
1.69.7130919

@Djhoss, could you please provide a detailed description of the steps, as well as any errors that you are getting? Thanks!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


get_node() autocompletion malfunctions when single quotes are used.

Godot version:
Godot_v3.2-alpha1_win64
OS/device including version:
Windows 7 SP1
Issue description:
I've triggered autocomplete for this code
get_node('')
and got
get_node("."')
Works fine with double quotes.
If you trigger autocomplete for this code:
get_node(' )
you'll get
get_node("." )
not as bad as previously but i expected single quotes to be used.
Steps to reproduce:
Minimal reproduction project:
Any with at least the root node.
Related to https://github.com/godotengine/godot/issues/32593

I was unable to reproduce in 3.1.1 stable, possible regression?

It seems that this is the same for all types of autocompletion.

Godot version: 3.2.alpha3.official
text_editor/completion/use_single_quotes: disabled

@dalexeev yes, the two things that are broken currently are

autocompletion with $
autocompletion with functions ( which get_node() is )

The actual bug is located around line 6500 in text_edit.cpp, in the function _update_completion_candidates, where it is supposed to add the suggestion with single quotes to the completion_current variable but instead ( somehow ) pushes the suggestion with double quotes while the visual box uses single quotes.


Retrieval models definition using new RetrievalModelV2, InputBlockV2 and ModelOutput classes

Fixes #611
Fixes #805
Goals :soccer:

The objective of this PR is to implement a new version of high-level retrieval models API (MatrixFactorizationModelV2, TwoTowerModelV2, and YoutubeDNNRetrievalModelV2) using the new core API (such as RetrievalModelV2, InputBlockV2, and ModelOutput)

The tested functionalities for the three models are:
- Saving/loading retrieval models
- Saving /loading Top-k encoders defined from the trained retrieval models.
- Computing top-k evaluation of retrieval models
- Exporting item embeddings and the query tower.

Implementation Details :construction:

[x] Introduce new retrieval methods: MatrixFactorizationModelV2, TwoTowerModelV2, and YoutubeDNNRetrievalModelV2
[x] Ensure embeddings export is working for MatrixFactorizationModelV2 and TwoTowerModelV2
[x] Link Top-K encoder to retrieval models.

Testing Details :mag:

[x] Update all unit tests of retrieval models with the new API.
[x] Test export towers and embeddings works for MatrixFactorizationModelV2 and TwoTowerModelV2.
[x] Test top-k recommendation and evaluation of MatrixFactorizationModelV2 and TwoTowerModelV2.
[x] Test top-k recommendation and evaluation of YoutubeDNNRetrievalModelV2.
[x] Test MatrixFactorizationV2 and TwoTowerModelV2 with Binary and Regression tasks.
[x] Add integration tests for MatrixFactorizationV2 and TwoTowerModelV2.

Notes:

This PR does not refactor the old retrieval API

While exporting candidates embeddings is working, exporting the query embeddings from YoutubeDNNRetrievalModelV2 raises an error related to the batch-predict step. The error (more details can be found in the issue #801) is related to the apply method of dask-df does not support a list column with an unspecified dtype.

Integration tests showed a mismatch between the evaluation scores of the old and new API. After debugging the API, I found three differences that needed to be fixed in the new API (embedding regularization (fixed here), log-q correction (fixed here), and logits scaling (fixed here) )

I don't think we can close this issue https://github.com/NVIDIA-Merlin/models/issues/498 (in the sense that the issue is referring to the specific version of TwoTowerModel that will continue to have the loading issue even with this new version?). So that issue might exist until we've removed the old TwoTowerModel and migrated all the examples to use the new one?

I don't think we can close this issue #498 (in the sense that the issue is referring to the specific version of TwoTowerModel that will continue to have the loading error even with this new version?). So that issue might exist until we've removed the old TwoTowerModel and migrated all the examples to use the new one?

That's a good point! The focus of this PR is to implement a new version of retrieval models, such as TwoTowerModelV2. Removing the old API and updating the examples should happen in a separate PR. I removed the issue from the PR description and just add a comment about using TwoTowerModelV2 as an alternative solution.

rerun tests

rerun tests

rerun tests


Removed duplicate code

I removed all __request methods and implemented single _common_request -method. I also removed the deprecated key words. If they're still needed, they should be wrappers to _request rather than duplicating the code from _request. I also did some indent fixes based to pylint.
This change removes extra code, and makes maintaining easier. Also if you have to do some own extension for your SUT, then this simplifies also that. Overriding only one single method is easier than overriding multiple __request methods.

@teemuvesala seems like this branch is now out of date

@teemuvesala seems like this branch is now out of date

I allocated from my calendar time to fix those during next Tuesday. The "Cookie edit" merge changed the functions which I edited also.

@bulkan but do you think the approach of this PR is correct?
In case I can have a look into fixing conflicts.

Merged conflicts and fixed open points in the review so far #227


Upgrade to Kafka <IP_ADDRESS>

Rebased #9

@Crim I think from a process standpoint, moving forward we should bump the POM to the next SNAPSHOT first on master, add the heading to the changelog file and then rebase our branches off that to avoid merge conflicts as we file multiple branches for a bigger release back onto master.


Cant find git binary at windows 8

Catch an error:
Uncaught BufferedProcessError: Failed to spawn command `git`. Make sure `git` is installed and on your PATH

But git installed and PATH to git binary available at proccess.env from atom developer console. Change settings to full git path or path to git binary directory dont help.
Plgin version 5.6.1

Are you saying the 'Git path' option in the package settings isn't working for you?

@akonwi yes, more info on screenshot: http://joxi.ru/l2ZYJgpCkPZGmJ

What happens if you quote the value?

@envygeeks check screenshot, same error. I open pull request for fix.


[$250] No indication of scan in progress when clicking in to the expense when the receipt is scanning

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.64-0
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1713883047268849
Action Performed:

Create scan receipt expense as employee
Open the report when scanning is in progress as approver or employee

Expected Result:
Should have some indication for scanning
Actual Result:
No indication as receipt is scanning
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/ae2628c5-24e0-4b2e-8328-e6451a897840

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016bf55aab1c62849f
Upwork Job ID:<PHONE_NUMBER>230407168
Last Price Increase: 2024-04-30
Automatic offers:

rojiphil | Reviewer | 0
bernhardoj | Contributor | 0

Will review the proposal tomorrow and update.

@bernhardoj  Thanks for the proposal.
I agree we need to add support for  Scan in progress title and scanning status bar in MoneyReportHeader.  However, I am curious to know why we need to show scanning and next steps status bar together. Instead, don’t we need to show  scanning status bar only if all the requests are being scanned? Otherwise, we can show Next steps status bar as it is currently shown. What do you think?
Also, let us call MoneyReportHeaderScanStatusBar instead of MoneyRequestHeaderScanStatusBar.

Instead, don’t we need to show scanning status bar only if all the requests are being scanned?

The scanning status bar will only show if there is one request and it's scanning.
I also like to hide the Next Step when the scan status bar is shown, but that depends on the team decision.

The scanning status bar will only show if there is one request and it's scanning.

Since the Report Preview shows Scanning in progress when all requests are being scanned, we can expect the same in Expense Report too. So, let us do it this way unless we have good reasoning to do otherwise at the PR stage.

I think the reason this issue is created is because we don't know whether a one transaction report is in scanning or not. If we have multiple requests, then we don't need to write Scan in progress in the header anymore IMO.

If we have multiple requests, then we don't need to write Scan in progress in the header anymore IMO.

As seen here, we show Scan in progress in Report Preview when all the requests are being smart scanned. So, it is expected to see scan in progress status in the Expense Report in this scenario. no?

I think the reason this issue is created is because we don't know whether a one transaction report is in scanning or not.

hmm.. Not sure if I understood your comment. However, my understanding of the issue is that there is no indication of scan in progress in Expense Report when the Report Preview does. Are we on the same page on this?

My thought is, that in the transaction thread, you can see that the transaction receipt scan is in progress from the header.

But if you view the transaction from the (IOU/expense) one transaction report, the scan indication in the header doesn't appears. It's like an inconsistency since one transaction report looks like a transaction thread.

Btw, the scanning indication is now exist in the Amount and Merchant field.
I'm fine if we want to show Scan in progress when all transactions are scanning, but I think we should confirm with the team again.

I'm fine if we want to show Scan in progress when all transactions are scanning, but I think we should confirm with the team again.

That’s correct. The internal engineer/design team would also weigh in with their inputs. But, let’s deal with all this at the PR implementation stage:
As per my understanding, the following needs to be done:

Show Scan in progress title in Expense Report when all requests are being smart scanned.
Reasoning: The Report Preview in chat report shows as Scan in progress when all the requests are being smart scanned. Also, we display this title whenever the scanning... status bar is displayed.
Show Scanning… status bar instead of Next steps when all requests are being smart scanned.
Reasoning: Showing Next steps does not seem right when all the requests  are being smart scanned. Instead show scanning… status bar.

@bernhardoj proposal LGTM as indication of scanning can be shown via status bar and title in Expense Report. Further comments as mentioned above can be confirmed during the implementation.
🎀👀🎀 C+ reviewed

Issue not reproducible during KI retests. (First week)

Looks good, seems like this current issue is no longer reproducible but seems like there's some additional improvements that are discussed here

I just realized that if we show Scan in progress for the one transaction report title, then we will have both title and subtitle as Scan in progress

Is that fine?

I just realized that if we show Scan in progress for the one transaction report title, then we will have both title and subtitle as Scan in progress. Is this fine?

@bernhardoj I think this one on the Sidebar may be intentional. But how is it connected to our issue related to the status bar/title on the Expense Report?

But how is it connected to our issue related to the status bar/title on the Expense Report?

because both LHN title and report header title use ReportUtils.getReportName, so if we change it to show Scan in progress if all transactions are scanning, then that double Scan in progress... will happen.

because both LHN title and report header title use ReportUtils.getReportName, so if we change it to show Scan in progress if all transactions are scanning, then that double Scan in progress... will happen

@bernhardoj Ah! Got it. Thanks and that may make sense. Are you saying that showing Scanning in progress as title does not seem right? i.e. Point mentioned here is not required.

@rojiphil yep, I think we should just add the "Scanning" status bar for a one-transaction report. What do you think?

Sounds cool. So, point (1) here does not make sense. But I am unclear on what you meant by one-transaction report. Can you please bring the PR up? We can discuss further there.

PR is ready
cc: @rojiphil

both contracts paid


macOS: Implement NSTextInputClient

Issues this hopefully fixes:

#263
jwilm/alacritty#61
jwilm/alacritty#209
jwilm/alacritty#372
jwilm/alacritty#748 ✔
jwilm/alacritty#764 ✔
jwilm/alacritty#935 ✔
jwilm/alacritty#1040 ✔
jwilm/alacritty#1041 ✔
jwilm/alacritty#1064 ❌

(I'm not sure about all of these. Hopefully we can get feedback on most of them.)
Alacritty branch for easy testing: https://github.com/francesca64/alacritty/tree/macos-kb
@kwonoj are you interested in trying this out?

Note that this isn't perfect: https://github.com/tomaka/winit/issues/263#issuecomment-388913791

There's still future work that needs to be done, i.e. it's not currently possible to display a cute little underlined ´ prior to é appearing. This is far more bothersome when working with IME, but it's still a big improvement over the current state of affairs.

Cool thing, I was recently realized this is needed and chanhes arrived even before I dig into impl :)
I'll try some quick testing around tomorrow once I get to access to my mac machine.

Also (somewhat) related : https://github.com/servo/servo/issues/20770

I just gave it a try and it doesn't look like working as expected - I tried simple KR char composition to 한글 (added space to trigger composition end) , and belows are event I could observe:
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 79, state: Released, virtual_keycode: None, modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 46, state: Pressed, virtual_keycode: Some(M), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: ReceivedCharacter('ㅎ') }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 46, state: Released, virtual_keycode: Some(M), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 3, state: Pressed, virtual_keycode: Some(F), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 3, state: Released, virtual_keycode: Some(F), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 1, state: Pressed, virtual_keycode: Some(S), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 1, state: Released, virtual_keycode: Some(S), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 40, state: Pressed, virtual_keycode: Some(K), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: ReceivedCharacter('ㅏ') }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 40, state: Released, virtual_keycode: Some(K), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 5, state: Pressed, virtual_keycode: Some(G), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 13, state: Pressed, virtual_keycode: Some(W), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 5, state: Released, virtual_keycode: Some(G), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 13, state: Released, virtual_keycode: Some(W), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 49, state: Pressed, virtual_keycode: Some(Space), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: ReceivedCharacter('글') }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: ReceivedCharacter(' ') }
WindowEvent { window_id: WindowId(Id(140512591906368)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 49, state: Released, virtual_keycode: Some(Space), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }

Particular odd point is ReceivedCharacter, in order it emits

ㅎ
ㅏ
글

and it doesn't have specific rules, 1 emits when composition starts, but 2 is neither composition starts nor composition end (cause last consonant ᆫ for char 한 supposed to be char right before composition end), then for next char it doesn't emit any vowel / consonant but emits composited char once composition completes via pressing space key.

I tried with 390 Sebulshik, and I seem to be able to produce the desired sequence in Alacritty, just like in Terminal.app. ReceivedCharacter contains code points, not graphemes, so maybe that's why the events aren't what you expect?

But still in those cases, what's recommended way to process keyevt with compositions? In this case receivedcharacter is pretty much unusable, so applications have to filter it away and compose via keyboardinput only.
For cases like broswers implements composition* event, it surpresses keydown while composition's in progress so it is acceptable (filter out receivedchar) but

It is hard to figure out when composition is in progress with current keyinputevt (maybe this is up to application level to figure out)

Even if receivedchar supposed to display codepoint, current behavior does not emit all codepoints - each kr vowel/consonant have visibile codepoint, so hard to explain why some are received and some are not.

There's not currently a recommendation, and there's plenty of API design that needs to be done. I'm open to suggestions (and detailed explanations of requirements), but it's not something I can put much time into at present.

I see. I'll try to check few other things than receivedcharacter event as well.

So for non-emitting ReceivedCharaters, it seems insert_text()(https://github.com/tomaka/winit/pull/518/files#diff-58d7f5aca673e64eda4815ae176040b2R236)  itself is not even being called for certain composition status. I can't figure out exactly when it's not being called / why, probably worth to file separate issue instead of trying to dig out in this PR maybe?

Yeah, I'd like to get this merged and worry about further improvements afterwards.

Dead keys are not working for me at all with this version, also Japanese input also does not work. I could however switch keyboard while Alacritty was focused. I'm on Sierra(10.12.6)

Hi @francesca64 thanks for this awesome fix (now I can finally input tildes in spanish characters :) ). Two things I did notice when using your alacritty branch on Mac and U.S keyboard layout are: i) before I use to be able to hold the j key in vim and the cursor would go down lines as long as I don't release such key. Now in order to achieve such behaviour I need to hold and release j (i.e holding it down only works to go one line down but not further than that); and ii) for some reason now my  doesn't seem to work (whereas if I press  then I can actually mimic/send an  keystroke). Is that expected?
Once again thanks for such an amazing work. Btw Will there be a PR to alacritty with these changes?

I'm always on alacritty (not using terminal.app). The Enter issue happens only on tmux (and not on vanilla /plain alacritty). The j (or k) issue happens in both cases and inside vim or outside (for instance using less). It might have something to do with my own setup if you cannot reproduce it. Thanks!

I'll be sure to investigate that before releasing anything. I'm guessing the i/j thing happens with any key?

I'm guessing the i/j thing happens with any key?

Indeed.

I'll be sure to investigate that before releasing anything.

Thank you!

@petobens cargo update and let me know if it's fixed. (If it doesn't work, double check that it pulled in https://github.com/francesca64/winit?branch=macos-kb-regressions#a9860a92)

I was using Swedish Pro and trying to type e.g ´ procudes the following.
glutin event: WindowEvent { window_id: WindowId(Id(0)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 24, state: Pressed, virtual_keycode: Some(Equals), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }
glutin event: WindowEvent { window_id: WindowId(Id(0)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 24, state: Released, virtual_keycode: Some(Equals), modifiers: ModifiersState { shift: false, ctrl: false, alt: false, logo: false } } } }

The hardware keyboard in this case is a UK apple magic, worth nothing that equals is what is printed on the key used and what you get when using a UK layout.

@k0nserv that's not a regression, is it? If it's not a regression, then report an issue for it. I'm quite exhausted, so at the moment I'm only addressing things that block a release.

Nope it’s not a regression, but I guess it might be an edge case? Anyway thanks for the great work 👍

@francesca64 both fixes work :) Thank you!!


Navbar styling

Currently the buttons on the right of the navbar are hidden when you go to a small enough view width. This kinda prevents logging in on phones in the vertical orientation.

https://streamable.com/ss96m1
good enough for now


Enable cross-platform writes of las files

This pull request changes func_dlis_to_las.py to enable writing the .las files on windows,  linux, and other platforms.
I didn't make this change in the Jupyter files.  Let me know if I should make the change there too.
Let me know if this change could be accepted (or rejected) or
needs some additional changes before being approved and merged.
Thank you,
DC

Hi @aruss175,
Just checking in on this pull-request.  Is this something you would be interested to pull into DLISIO_Notebooks?  If not, that is okay too.   Also the checks are failing but the details are behind the Equinox firewall,  if you are interested to pull this in, let me know what needs to be fixed to enable the checks to pass, and I'll fix it.
Thank you!,
DC

Yikes, sorry, I've been out on vacation and there has been some heavy dlisio developement from the team with the 0.2.4 release.  I will accept the commit but I will need to do some updating then for the new release and that should hopefully fix the checks.

Great!,  Thank you for the merge.


Replace cassandra_protocol::frame::Frame with a more strongly typed CassandraFrame

Progress towards https://github.com/shotover/shotover-proxy/issues/445
Prereq to https://github.com/shotover/shotover-proxy/issues/473 and https://github.com/shotover/shotover-proxy/issues/476
There is a lot of logic changes in codec/cassandra.rs to convert the new format into message details, please dont think too hard about these changes as message details will be going away, as long as it passes the integration tests then that should be fine.
One difference to the implementation in https://github.com/shotover/shotover-proxy/pull/453 is the addition of:
pub enum CQL {
    Parsed(Vec<Statement>),
    FailedToParse(String),
}

This was required because we are now always round tripping through the new CassandraFrame so we need to allow for failing CQL parse which happens a lot at the moment.
Once full #453 lands there will be many cases where round tripping will not be required.
We may or may not want to keep this as is in full #453
For similar reasons, I also had to complete the full implementation of:
impl From<MessageValue> for cassandra_protocol::types::value::Bytes {
    fn from(value: MessageValue) -> cassandra_protocol::types::value::Bytes {
    }
}

due to the integration tests now hitting every case here.

Tests are parsing, this is now ready for review.


hotel (0.4.16) dies randomly (OS X)

The hotel daemon stops running at random times. Sometimes right after "hotel start" sometimes later (after hours).
This is what ~/.hotel/daemon.log says:
Exiting
Stop daemon
Remove pid file
/usr/local/lib/node_modules/hotel/lib/daemon/app.js:26
  var _req$headers$host$spl = req.headers.host.split(':');
                                              ^

TypeError: Cannot read property 'split' of undefined
    at parseReq (/usr/local/lib/node_modules/hotel/lib/daemon/app.js:26:47)
    at Server.<anonymous> (/usr/local/lib/node_modules/hotel/lib/daemon/app.js:68:21)
    at emitThree (events.js:110:13)
    at Server.emit (events.js:188:7)
    at onParserExecuteCommon (_http_server.js:400:14)
    at HTTPParser.onParserExecute (_http_server.js:368:5)

I of course did not stop hotel. Running hotel start after that results in:
/Users/me/Library/LaunchAgents/hotel.plist: service already loaded

I have to stop and start.

Hi @jeanmartin,
Thank you for reporting this issue.
There seem to be a request that doesn't have a host header, and so hotel fails to parse it.
But this code should handle only .dev requests, so there should always be a host.
Do you know the request that makes it fail? It would be really helpful if you could add some kind of logs to figure it out, for example:
// line 25 in /usr/local/lib/node_modules/hotel/lib/daemon/app.js
console.log(req.headers)

Then restart hotel and see what's the output.

Done. The last request before seems to be the proxy.pac:
21 Apr 14:32:52 - Serve proxy.pac
{ 'proxy-connection': 'keep-alive' }
Exiting
Stop daemon
Remove pid file
/usr/local/lib/node_modules/hotel/lib/daemon/app.js:27
  var _req$headers$host$spl = req.headers.host.split(':');
                                              ^

TypeError: Cannot read property 'split' of undefined
    at parseReq (/usr/local/lib/node_modules/hotel/lib/daemon/app.js:27:47)
    at Server.<anonymous> (/usr/local/lib/node_modules/hotel/lib/daemon/app.js:69:21)
    at emitThree (events.js:110:13)
    at Server.emit (events.js:188:7)
    at onParserExecuteCommon (_http_server.js:400:14)
    at HTTPParser.onParserExecute (_http_server.js:368:5)

Thanks! What did you do just before it crashed?
By the way, I'll release a fix for this today or this week-end.

Nothing. Seriously. I was checking mail, made no request to hotel or my app myself. However you never know if Chrome might have done it (e.g. because network connection was lost and reconnected again or something).
I will not update to your fix immediately and test a bit on the weekend trying to reproduce what exactly happens. Thanks for your fast response!

I've released the fix in v0.4.17.
No need to spend too much time, except out of curiosity :) let me know if you find out.
In any case, now the daemon checks if there's a host header and shouldn't fail anymore.


Want to add Competitive Programming by Felix Halim and Steven Halim ( one of the best books for competitive programming )

I would like to add Competitive Programming by Felix Halim and Steven Halim ( which is one of the top 5 recommended books by gfg on their list of competitive programming books )

Sure thing @HARSHITSHARMA18
I am assining you the issue. Make sure to close the issue after you make the PR.

Hey @mursalfk I have made the PR  , you can merge that


NPM package install size too large

Recently updated my @electric-sql/pglite from version 0.2.13 to 0.2.15 and the install size difference was huge, I believe double the size. It put my deployments over their size limit and I had to revert and go back to using 0.2.13.
When looking at the diff between these 2 versions, there seems to be a lot of new code added, so I'm unclear what exactly makes up the new package size difference.
Is it possible to reduce the install size of future versions? This package has become the largest npm package in my entire project.

Hey, thanks for the report. I'm ware it got a little larger, but we clearly need to address this.
the root cause what that the last version has some of the work required to make PostGIS possible, including much of its dependencies. These are in the package, but unused as the work is not complete. It's not loaded by the browser at all.
I think it's clear we need to roll this back, and move PostGIS to a separate package when we finally get it working.


Right Click Menu: Implementation Proposal

Context
Right click menu
Summary
Initial idea for right click menu.
Relevant Technical Choices
We'll be hooking into a lot of existing functionality, so I decided

This follows the quick actions menu architecture pretty closely.
Right click menus will need the following information to determine what items to show and what the items do:

the selected element (element that was right-clicked)
story context: to know what styles will be copied/the element information. May also need this for determining the front/back placement of elements in relation to each other.
page context: adding, removing, and duplicating pages

These items will hook into existing functionality most of the time. For that reason, I decided to follow the quick actions setup since it hooks into other contexts like the 'highlights' context.
To-do
A few questions that need to be talked about:

[x] Does right clicking an element select it? yes it does
[ ] (other pr)Investigate and possibly refactor the way the context menu opens. If we can leverage using the blur event then we will be able to click something under the 'mask' of the context menu

User-facing changes

Testing Instructions

Turn on the right click menu experiment
Go to editor
Add an element
Right right clicking the canvas should open the right click menu
Right clicking the element in the canvas should open the right click menu and select the element

Note: Right now you have to click away from the context menu to close it first before right clicking again. This will be fixed in a later ticket.
QA

[x] This is a non-user-facing change and requires no QA

Reviews
Does this PR have a security-related impact?
no
Does this PR change what data or activity we track or use?
no
Does this PR have a legal-related impact?
no
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #6156

I think we should split out the copy/paste/delete stuff into a new pr. It's complicated and we should isolate those changes to make sure they are reviewed more carefully.

I like the reducer update alot. Thanks for fixing the area getting watched for right clicks! I think this is a great foundation.
Something to add, either here or later is closing the menu when clicking outside of the canvas while menu is open (which is probably related to the updates you mentioned wanting to separate later on, figure I'd drop a note here anyways).

Yup, that will be fixed when we look into a different way to close the context menu 😄


feat: drop support for py2/py3 compatible libraries

This PR is inspired by https://github.com/splunk/addonfactory-ucc-generator/pull/224. As it contains 2 separate code changes, I decided to create this PR to separate the feature part.
This PR drops support for Python 2 in generated add-ons by ucc-gen command.

Codecov Report

Merging #225 (fcf65b1) into main (9858189) will increase coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #225      +/-   ##
==========================================
+ Coverage   55.98%   56.05%   +0.07%
==========================================
  Files          37       37
  Lines        2506     2492      -14
==========================================
- Hits         1403     1397       -6
+ Misses       1103     1095       -8

Impacted Files
Coverage Δ

splunk_add_on_ucc_framework/__init__.py
66.33% <0.00%> (+0.79%)
: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 9858189...fcf65b1. Read the comment docs.

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

v5.2.0
GitHub release

Your semantic-release bot :package::rocket:


chore : Small changes, replica nums, and route changes

These changes are intended to adapt the repository for MOTT use.

That was supposed to go to my own version not the bcgov version. My apologies


Typescript problem with G Element in Expo 46

Hello.
I'm using Expo 46 with React 18, and I have some problem with typescript.
This is a spinnet of used code:
<Svg height="20" width="90%">
          <Center>
            <Icon name="plane_3" size={24} inverted />
          </Center>
          <G>
            {dashes.map((_, index) => (
              <Rect
                key={index}
                x="0"
                y="10"
                width="8"
                height="3"
                fill={colors.secondaryContrast}
                translateX={spacing * index}
              />
            ))}
          </G>
</Svg>

An the error I get is:
Type '{ children: Element[]; }' has no properties in common with type 'IntrinsicAttributes & IntrinsicClassAttributes<Component<GProps, any, any>> & Readonly<GProps>'.
The versions I'm using:
    "expo": "^46.0.0",
    "react": "18.0.0",
    "react-native": "0.69.5",
    "react-native-svg": "12.3.0",
    "@types/react": "~18.0.0",
    "@types/react-native": "~0.69.1",

I believe this issue is similar to e.g. this one: https://github.com/react-native-svg/react-native-svg/issues/1798 and it should be solved since 12.4.4. Could you check if it works correctly in this version?

I believe this issue is similar to e.g. this one: #1798 and it should be solved since 12.4.4. Could you check if it works correctly in this version?

Upgrading to version 12.4.4 seems to resolve the issue for me. However I get the follow warning when starting Expo.
Some dependencies are incompatible with the installed expo version<EMAIL_ADDRESS>- expected version: 12.3.0 Your project may not work correctly until you install the correct versions of the packages. Install individual packages by running npx expo install<EMAIL_ADDRESS>I've opted for the patch described in https://github.com/react-native-svg/react-native-svg/issues/1798 until this gets resolved.

I believe this issue is similar to e.g. this one: #1798 and it should be solved since 12.4.4. Could you check if it works correctly in this version?

Upgrading to version 12.4.4 seems to resolve the issue for me. However I get the follow warning when starting Expo.
Some dependencies are incompatible with the installed expo version<EMAIL_ADDRESS>- expected version: 12.3.0 Your project may not work correctly until you install the correct versions of the packages. Install individual packages by running npx expo install<EMAIL_ADDRESS>I've opted for the patch described in #1798 until this gets resolved from the Expo side. I'll open an issue on their side and link it back to this for future reference.
Here is the issue opened with the Expo team expo/expo#19013

Thanks!

Ok so it seems like there is nothing more to be done on the side of react-native-svg. I will close this issue then. Feel free to comment here if something is wrong and we can always reopen it.


Added narrow() to Values

Towards #1065

Current coverage is 96.39%

Merging #1081 into master will decrease coverage by -0.07% as of b9bcb26

@@            master   #1081   diff @@
======================================
  Files           75      75
  Stmts         9444    9452     +8
  Branches      1733    1733
  Methods          0       0
======================================
+ Hit           9110    9111     +1
+ Partial        100      99     -1
- Missed         234     242     +8

Review entire Coverage Diff as of b9bcb26

Powered by Codecov. Updated on successful CI builds.

Note: The following classes do not have a narrow() method yet:

Either.LeftProjection, Either.RightProjection
Match.MatchValue, Match.MatchFunction

These are intended to be used as intermediate objects. narrow() may be added later for these if needed.

@danieldietrich why did you choose such a name (narrow) for this operation?
In scalaz it's widen:
https://github.com/scalaz/scalaz/blob/80ba9d879b4f80f0175b5f904ac4587b02400251/core/src/main/scala/scalaz/Functor.scala#L92-L97
In cats it's widen:
https://github.com/typelevel/cats/blob/88ccb7a67b884eecea8d41e7fa735d23f5788b5c/core/src/main/scala/cats/Functor.scala#L37-L58
In arrow-kt it's widen:
https://github.com/arrow-kt/arrow-core/blob/b9bac5a03562a8e792b95c976182e9094b8ae425/arrow-core-data/src/main/kotlin/arrow/typeclasses/Functor.kt#L227-L247

@danieldietrich why did you choose such a name (narrow) for this operation?
In scalaz it's widen:
https://github.com/scalaz/scalaz/blob/80ba9d879b4f80f0175b5f904ac4587b02400251/core/src/main/scala/scalaz/Functor.scala#L92-L97
In cats it's widen:
https://github.com/typelevel/cats/blob/88ccb7a67b884eecea8d41e7fa735d23f5788b5c/core/src/main/scala/cats/Functor.scala#L37-L58
In arrow-kt it's widen:
https://github.com/arrow-kt/arrow-core/blob/b9bac5a03562a8e792b95c976182e9094b8ae425/arrow-core-data/src/main/kotlin/arrow/typeclasses/Functor.kt#L227-L247


Refactor cairo1-run crate for library use

Is your feature request related to a problem? Please describe.
The current implementation of cairo1-run crate as a CLI restricts its usability in other projects that could benefit from its functionality as a library.
Describe the solution you'd like
Transform the cairo1-run crate into a standalone library that can be used by external projects. Ideally, the run function would return:

Output of program execution.
Relocated trace and memory.
Sierra (with debug info) and CASM programs.
CASM additional instructions (proof_mode_header, entry_code, libfunc_footer).
Diagnostics.
Compiler RootDatabase.

Motivation
We need it for https://cairovm.codes and https://walnut.dev — projects aiming to build a debugger for Cairo / Starknet.

Hello! We recently refactored the cairo1-run crate to also expose  a cairo_run_program function that takes a sierra program and returns the runner, vm, and return values after the run #1601

Closed by #1714


Linting

If I run jlpm lint on your branch I have
$ git status
Your branch is up-to-date with 'martinRenou/linting'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   .github/workflows/build.yml
	modified:   .github/workflows/check-release.yml
	modified:   .github/workflows/prep-release.yml
	modified:   .github/workflows/publish-release.yml
	modified:   packages/base/src/commands.ts
	modified:   packages/base/src/formdialog.tsx
	modified:   packages/base/src/mainview/mainview.tsx
	modified:   packages/base/src/mainview/mainviewmodel.ts
	modified:   packages/base/src/panelview/formbuilder.tsx
	modified:   packages/base/src/panelview/header.tsx
	modified:   packages/base/src/panelview/model.ts
	modified:   packages/base/src/panelview/objectproperties.tsx
	modified:   packages/base/src/panelview/rightpanel.tsx
	modified:   packages/base/src/toolbar/usertoolbaritem.tsx
	modified:   packages/base/src/toolbar/widget.tsx
	modified:   packages/base/src/tools.ts
	modified:   packages/base/src/types.ts
	modified:   packages/base/src/widget.tsx
	modified:   packages/schema/src/doc.ts
	modified:   packages/schema/src/interfaces.ts
	modified:   packages/schema/src/model.ts
	modified:   packages/schema/src/token.ts
	modified:   python/jupytergis_core/src/externalcommand.ts
	modified:   python/jupytergis_core/src/factory.ts
	modified:   python/jupytergis_core/src/index.ts
	modified:   python/jupytergis_core/src/jgisplugin/modelfactory.ts
	modified:   python/jupytergis_core/src/jgisplugin/plugins.ts
	modified:   python/jupytergis_core/src/plugin.ts
	modified:   python/jupytergis_core/style/index.css
	modified:   python/jupytergis_lab/src/index.ts
	modified:   python/jupytergis_lab/src/notebookrenderer.ts
	modified:   python/jupytergis_lab/style/base.css
	modified:   python/jupytergis_lab/style/index.css
	modified:   ui-tests/tests/notebook.spec.ts
	modified:   ui-tests/tests/ui.spec.ts

Looks like you should run it again

Indeed! Thanks for trying. Not sure what was going wrong, but starting again from scratch I could see the same as you/


[FTS Broadcaster] Add support for publishing forces in other frames on additional topics

This has to be cleaned a bit to split “additional topics” and filtering functionalities into separate PRs.
I would be glad to get some help with tests.
Documentation on new parameter is also missing.

@destogl are you ready to move this ahead?


Fixed typos in the update_project_provisioning

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Because I found typos in the update_project_provisioning. There is no issue associated with this.
Description
Fixed typos.

I signed it!

I signed it!


Allow requests to opt out of HTTPS enforcement

Our logic for authenticating a multipart request currently fails because preparing one entails passing requests with schemeless URLs to BearerTokenCredentialPolicy, which duly raises because these URLs don't start with "https".
As a workaround for this and other cases in which enforcing HTTPS is unnecessary or undesirable, this change allows individual requests to opt out of https enforcement via an enforce_https context option (i.e. a keyword argument to pipeline.run).

Would it be valuable to be more precise? Allow schema-less URLs (behind a flag, presumably), but still never allow URLs that start with "http://" ?

Allowing schemeless URLs is a reasonable way to solve the problem for storage but I think it's a weird special case for the general purpose bearer auth policy we have in azure-core. So, I think that (reasonable) solution belongs in the storage libraries.
I landed on allowing opting out of the enforcement entirely because it solves the problem at hand in a more general way I think will be useful when we find another similar scenario we want to support. For example, I'm running Azure Stack on a network I trust, and I care about TLS overhead.

@chlowell I didn't mean to suggest moving this, only making the "lax" mode here a little less lax

I follow you, what I meant is that I think configuration to disable validation entirely is more generally useful and less weird than configuration to allow malformed URLs. If we don't want the more general configuration (which is okay, no one's asked for it), I think storage clients should implement a custom policy.

Can one of the admins verify this patch?


[feature request] style: CSSProperties in the props

it works nice already, it's just not in the types.

You can send a PR to the types repo
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-slick/index.d.ts

no, thanks