Restructure CI jobs

Using folders, it should be possible to simplify the layout of the jobs in Jenkins.
/ev3dev/                             - root folder, see below
       /build/                         - for openjdk builds
             /ev3-stretch-native/        - builds for ev3dev-stretch for EV3, building on ARMv8
                                /jdk11     - OpenJDK 11 build
                                /jdk12     - OpenJDK 12 build
                                /jdk13     - OpenJDK 13 build
                                /master    - OpenJDK Master build
             /ev3-buster-native/         - builds for ev3dev-buster for EV3, building on ARMv8
                               /jdk11      - OpenJDK 11 build
                               /jdk12      - OpenJDK 12 build
                               /jdk13      - OpenJDK 13 build
                               /master     - OpenJDK Master build
       /test/                          - for openjdk tests
            /ev3-stretch/                - for stretch
                        /TBD               - ???
            /ev3-buster/                 - for buster
                       /TBD                - ???
       /pipelines/                     - TBD, could be used for building & testing together

(optionally) the ev3 platform part could be removed, as no other platforms are currently used.
I think the ev3dev folder on the top level may be necessary. This is because views do not create a separate namespace (AFAIK); therefore having an ev3dev prefix should prevent collisions.

To address the issue with the changing distro versions in the latest jdk12 builds, I have changed the configuration to the following model:

openjdk12_build_ev3_linux_native: 12 for Stretch
openjdk12_build_ev3_linux_native_next: 12 for Buster
openjdk13_build_ev3_linux_native: 13 for Stretch
openjdk13_build_ev3_linux_native_next: 13 for Buster

I liked it.
Can we close this issue?

Not yet - the changes made so far are only temporary. The target state is sketched in the first post and it is not yet implemented; I will try to do so now.

I have built the structure here: https://ci.adoptopenjdk.net/view/ev3dev/job/ev3dev/.
There are two views in that folder: All and Build Summary. All shows the folder structure; Build Summary shows summary of builds in the subfolders.
The URL has doubled ev3dev because the ev3dev folder (which creates namespace isolation) is located in ev3dev view (which creates the tab on the main page). However, the view part is not mandatory; https://ci.adoptopenjdk.net/job/ev3dev/ will work too.
OpenJDK-EV3 will however have to be adjusted, as the move will change the URLs from where the host/boot JDKs are taken.

Maybe, so much folders in the last implementation:
https://ci.adoptopenjdk.net/view/ev3dev/
https://ci.adoptopenjdk.net/view/ev3dev/job/ev3dev/
https://ci.adoptopenjdk.net/view/ev3dev/job/ev3dev/job/build/
What is the idea for pipeline folder and test?
https://ci.adoptopenjdk.net/view/ev3dev/job/ev3dev/job/pipelines/
https://ci.adoptopenjdk.net/view/ev3dev/job/ev3dev/job/test/

Now, all builds appear for deletion, but we didn´t talk about the archive process to not loose any build from 9 & 10.

Yup; however there are already no build artifacts for deletion. I have cleaned the jobs a few months ago to save space on the Jenkins master.
I will reenable the jobs and try to run a new build. Do we want to build only for Stretch?

The test folder is intended to contain some automatic ARM-native tests, if we add some.
The pipelines folder is intended for orchestration of multiple builds & tests together.

Now that the builds depend on each other, I think the periodic builds could be ordered from JDK11 to JDK-Master.
Also, the tests that would be run should be chained to the build via a top-level pipeline job.

The new URLs are quite long; however it is not all that bad.

When the view is omitted, the URL is actually shorter - see https://github.com/ev3dev-lang-java/openjdk-ev3/pull/41/files.
The displayed paths of the jobs are now nicer:

Hmmm, we have one build of JDK9/JDK10 archived: https://github.com/ev3dev-lang-java/openjdk-ev3/releases/tag/v0.5.0. However it's probably a bit out of date.

I have some ideas regarding the URL length & complexity:

We could omit the extra ev3dev folder and instead use a shorter prefix (like elj) to the build, test and pipelines folders. This would be to prevent collisions with folders from AdoptOpenJDKFrom:
https://ci.adoptopenjdk.net/view/ev3dev/job/ev3dev/job/build/job/ev3-buster-native/job/jdk11/
To:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/ev3-buster-native/job/jdk11/

+ we could omit the ev3 and native part, as everything we now have is ARMv8 -> EV3.From:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/ev3-buster-native/job/jdk11/
To:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/buster/job/jdk11/

+ we could remove the separate folder for buster and stretch:From:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/buster/job/jdk11/
To:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/buster-jdk11/

+ we could omit the jdk prefix:From:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/buster-jdk11/
To:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/buster-11/

Alternatively, without the view prefix:https://ci.adoptopenjdk.net/job/eljbuild/job/buster-11/

I understand the refactoring, I like this schema:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/job/buster-jdk11/
do you like if we evolve like:
https://ci.adoptopenjdk.net/view/ev3dev/job/build/job/buster-jdk11/
https://ci.adoptopenjdk.net/view/ev3dev/job/test/job/buster-jdk11/
Do you like?

Hi,
there is a problem with that: the view/ev3dev part is optional and it does not isolate job names. The following two URLs would point to the same item:
https://ci.adoptopenjdk.net/view/ev3dev/job/build
https://ci.adoptopenjdk.net/job/build
The only difference is that the ev3dev view wouldn't show up in the upper navigation bar.
By doing this, we would take the generic build folder/job just for ev3dev and that feels strange to me.
On the other hand, this was the reason why I have introduced the second level ev3dev folder - for namespace isolation. The following two links would point to different items:
https://ci.adoptopenjdk.net/job/myFolder/job/myProject
https://ci.adoptopenjdk.net/job/myProject
I chose the elj alternative prefix to take the global eljbuild, eljtest and eljpipelines folders as they are unlikely to be taken and they refer to ev3dev-lang-java.
Regards

Hmm, I have clicked on some other builds in Jenkins and the URLs are long as well:
https://ci.adoptopenjdk.net/view/arm/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk12u/job/jdk12u-linux-arm-hotspot/
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-x64-openj9/

JDK9 build was successful: openjdk09_build_ev3_linux#72
JDK10 build should be OK too: openjdk10_build_ev3_linux#147

I have created the described folder structure:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljbuild/
https://ci.adoptopenjdk.net/view/ev3dev/job/eljtest/
https://ci.adoptopenjdk.net/view/ev3dev/job/eljpipes/

After https://github.com/ev3dev-lang-java/openjdk-ev3/pull/46 and https://github.com/ev3dev-lang-java/openjdk-ev3/pull/45 are merged, I would consider this issue finished.

Originally posted by @jabrena in https://github.com/ev3dev-lang-java/openjdk-ev3/pull/46#issuecomment-528930026

What is the idea for:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljtest/
and:
https://ci.adoptopenjdk.net/view/ev3dev/job/eljpipes/
?

The eljtest test folder is intended to contain replacements for the old test jobs.
The eljpipes pipelines folder is intended for chaining of tests after builds and optionally for scheduling all builds at once (so they could be ordered).

Finished


Cloud config should be optional for the xenorchestra_vm resource

@m0ps points out that not everyone wants to use cloud-init for provisioning instances. I personally don't like using Terraform's provisioning feature but that doesn't mean this provider should force people into doing it my way.

This is fixed in v0.7.0


update to latest tokens api

Signed-off-by: Carson Farmer<EMAIL_ADDRESS>

Also removes explicit token strings, which might be confusing to new users. Instead, just uses 'placeholder' token-string.

this meant to still be open @carsonfarmer?


267 subclasses of editorial work for extra content alex

Thanks Jürgen for your final translations. Like aligned, the update is ready for review and to be merged into the main branch.

Removed Interview Class like agreed.

I would like to have the dcterms:description annotations to the classes Trailer and MakingOf re-inserted, as we had decided to keep them, even if we want to add skos:defintions to all entitities in the long run and thereby replace the dcterms:definition. The reason for that was to understand the earlier semantical intention while being in the process of adding the skos:definitions. Once we have added skos:definitions to all entities, we can then probably remove the dcterms:descripitions without losing valuable information.

re-inserted dcterms:description annotations to the classes Trailer and MakingOf.

Noticed that the dcterm:description was removed for other subclasses of Editorial Extra as well. Should these be recovered as well?

Yes, please.
dcterms:description should only be removed, when we have completed the task to add skos:definitions to all classes.

Fixed wrong rdfs:prefLabel by replacing through skos:prefLabel.


Use -MMD instead of -MD, preventing system headers from showing up in .d files

Tried to do a git pull; make on a fairly stale, previously-built clone of icestorm. A minor change in my system headers prevented the compile from succeeding; .d files from the old build referenced system headers that had been moved/changed. Using -MMD instead of -MD to generate the .d files will exclude system headers, preventing this problem.

will exclude system headers, preventing this problem.

I don't quite see how that improves things. I'd say if system headers have changed you want to re-build the project, not run into obscure linker or runtime errors because some .o files have not been properly rebuilt.

In that situation, make clean; make would be successful. Does your make clean remove the .d files?

In that situation, make clean; make would be successful. Does your make clean remove the .d files?

Yes.

I thought I had tried that, but I just double-checked and it does indeed work.
Sorry for the useless PR.


Add missing 3D file fields

You can see it documented here https://doc.cognitedata.com/api/0.6/#operation/getRevision
And here https://doc.cognitedata.com/api/0.6/#operation/getSectors

@nsunderland-cognite Can you update the test data as well?

Thanks btw

Done. That field should've been optional but I've added to the test data anyway.

You probably want to bump the package as well (patch). See the readme file for a guide


Enhance aarch64 platform compatibility

modified:   teuthology/packaging.py

Signed-off-by: Dai Zhiwei<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@kshtsk ping?

I don't understand why this is required? Could someone give some explanation?


Crossbar Machines

Added crossbar machines baseline_v0_8_4 and baseline_v0_16_8.
The default machine is now baseline_v0_16_8. This is not the preliminary dimensions of a pod, but it is larger than the small machines and still has a reasonable simulation speed.

Depends on #347

Very cool!
On Mon, Jul 6, 2020 at 5:29 PM Dustin Richmond<EMAIL_ADDRESS>wrote:

Depends on #347
https://github.com/bespoke-silicon-group/bsg_manycore/pull/347
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/bespoke-silicon-group/bsg_replicant/pull/608#issuecomment-654529942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AANGKCQC3TB7VQU42LR2JT5VANCNFSM4OSB2JTA
.

Added comments to address concerns about changing hierarchies in #347.
https://github.com/bespoke-silicon-group/bsg_replicant/blob/a52d5bfa1f804b6b7d0baf445e4a986e021469ab/libraries/features/profiler/simulation/bsg_manycore_profiler.cpp#L50
https://github.com/bespoke-silicon-group/bsg_replicant/blob/a52d5bfa1f804b6b7d0baf445e4a986e021469ab/libraries/platforms/aws-fpga/hardware/cl_manycore.sv#L31
https://github.com/bespoke-silicon-group/bsg_replicant/blob/a52d5bfa1f804b6b7d0baf445e4a986e021469ab/libraries/platforms/aws-fpga/hardware/cl_manycore.sv#L343
Also merged a PR (#321) in basejump that catches missing hierarchies and "throws an exception" during RTL/simulation initialization to address concerns and improve usability.

@tommydcjung Do you have any feedback for this PR?

@taylor-bsg What machine do you want to be the default? 16_8 or 32_16?


Add ability to create phone intents

It would be convenient to be able to make phone calls/dial numbers.
Example API:
Shelly.phone()
    .number("1234 123 123")
    .dial()

Shelly.phone()
    .number("1234 123 123")
    .call()

Where call would call the provided number, and dial would dial the number but not the call.

This sounds like a really cool idea and one I can get behind.


Add ansi256 → ansi16

Output in Windows Terminal:

Fixes #92, Fixes #89

Thanks - could you add tests that check the result of ansi256.ansi16() to rgb.ansi16(ansi256.rgb()) to make sure it's as expected?
Something like:
for (let i = 0; i < 8; i++) {
    assert( cc.ansi256.ansi16(i) === (i + 30) );
}

for (let i = 8; i < 16; i++) {
    assert( cc.ansi256.ansi16(i) === (i + 90) );
}

for (let i = 16; i < 256; i++) {
    assert( cc.ansi256.ansi16(i) === cc.rgb.ansi16(cc.ansi256.rgb(i)) );
}

Couldn't get the third test to work but the other 2 passed fine.


enable metadata merging when no sample ids overlap

At present, sample metadata merging functions by intersecting the sample ids in the metadata files and as a result doesn't support metadata files with no overlapping sample ids. It would be useful to enable merge when no sample ids overlap. This would be particularly useful for pooled analyses, where samples from two or more different studies are integrated in a single analysis (e.g., contextualizing new data with pre-existing HMP or EMP samples). At present, this requires a manual merge of metadata files, which is error-prone and can lead to a proliferation of different metadata files (which is bad for a lot of reasons).
Challenges

In many cases, sample ids across studies will be entirely different. However there may be some commonly used sample names, for example control.1, blank.2, ... In this case, based on current behavior, the resulting metadata would be only the metadata associated with those samples (i.e., it would default to the sample id intersection behavior). This would result in users sometimes getting unexpected results that could require them to modify their input metadata (e.g., comment out records) or would require new functionality to tell QIIME 2 how to handle overlapping sample ids.

Hey @gregcaporaso - I'm pretty sure we can close this one as completed, correct? All of the functionality you mentioned is essentially included in this PR (just with a slightly different approach than what you mention above):
https://github.com/qiime2/q2-metadata/pull/60

Yep, thanks!

Yep, thanks!


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

Done


CSV feedback download doesn't work

For some reason, the system gives the file in Microsoft Excel format (instead of CSV) which is difficult to process.

wont fix unless more requests for the feature


How to include 3rd party libs

How to include 3rd party libs like ng2-translate, ng2-material and others?
After adding module to vendor.ts and build i have errors.
Inside your files i added:
vendor.ts
import {TranslateModule} from 'ng2-translate/ng2-translate';
export default {
   TranslateModule,
}

build.ts

globals:{
'ng2-translate/ng2-translate': 'vendor.TranslateModule'
}
external: [
        'ng2-translate/ng2-translate'
      ]

what am I missing?

I fixed it
import * as _angular_translate from 'ng2-translate/ng2-translate';
default as _angular_translate,
build
globals as 'ng2-translate/ng2-translate': 'vendor._angular_translate'
and external 'ng2-translate/ng2-translate'

To find examples on how to import 3rd party libs with rollup see here and here

https://github.com/rollup/rollup-plugin-inject


Pod install issue

Undefined symbols for architecture x86_64:
  "std::__1::__vector_base_common<true>::__throw_length_error() const", referenced from:
      std::__1::vector<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >, std::__1::allocator<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> > > >::allocate(unsigned long) in libYCMatrix.a(HaltonInterface.o)
      std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >::__append(unsigned long) in libYCMatrix.a(HaltonInterface.o)
  "std::terminate()", referenced from:
      ___clang_call_terminate in libYCMatrix.a(HaltonInterface.o)
  "operator delete(void*)", referenced from:
      std::__1::__vector_base<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >, std::__1::allocator<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> > > >::~__vector_base() in libYCMatrix.a(HaltonInterface.o)
      std::__1::__vector_base<unsigned short, std::__1::allocator<unsigned short> >::~__vector_base() in libYCMatrix.a(HaltonInterface.o)
      std::__1::__split_buffer<unsigned short, std::__1::allocator<unsigned short>&>::~__split_buffer() in libYCMatrix.a(HaltonInterface.o)
  "operator new(unsigned long)", referenced from:
      std::__1::vector<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >, std::__1::allocator<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> > > >::allocate(unsigned long) in libYCMatrix.a(HaltonInterface.o)
      std::__1::__split_buffer<unsigned short, std::__1::allocator<unsigned short>&>::__split_buffer(unsigned long, unsigned long, std::__1::allocator<unsigned short>&) in libYCMatrix.a(HaltonInterface.o)
  "___cxa_begin_catch", referenced from:
      ___clang_call_terminate in libYCMatrix.a(HaltonInterface.o)
  "___gxx_personality_v0", referenced from:
      +[HaltonInterface sampleWithDimension:count:] in libYCMatrix.a(HaltonInterface.o)
      Halton_sampler::init_faure() in libYCMatrix.a(HaltonInterface.o)
      std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >::resize(unsigned long) in libYCMatrix.a(HaltonInterface.o)
      std::__1::vector<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >, std::__1::allocator<std::__1::vector<unsigned short, std::__1::allocator<unsigned short> > > >::vector(unsigned long) in libYCMatrix.a(HaltonInterface.o)
      std::__1::vector<unsigned short, std::__1::allocator<unsigned short> >::__append(unsigned long) in libYCMatrix.a(HaltonInterface.o)
      Dwarf Exception Unwind Info (__eh_frame) in libYCMatrix.a(HaltonInterface.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I am getting a build error in iOS 9.2  ? just when installing the pod

Thanks for reporting this. I created a new blank project and tested this and indeed I am getting the same errors.
A workaround, if you are not using the Halton sampling functionality, is to manually remove from within Xcode the following files:
halton_sampler.h
HaltonInterface.h
HaltonInterface.mm
Then, remove the relevant import and method call in YCMatrix+Advanced.h and YCMatrix+Advanced.m.
After doing this the project compiles fine for me on iOS.
We're gonna need some help from someone better versed in Cocoapods to fix this..

Cocoapods are no longer supported. Closing this issue.


Test coverage

Checkpoint test coverage is extremely low - 0.5%.

WriteAheadFrontendLogger is also low - 15%


How to draw 2 lines with different lengths in the same window?

I've found that the viz.line can draw many lines with the same lengths in a window, but if there is a method to draw lines with different lengths in the same window? Thanks in advance~

You can append new lines of different lengths by using the update='append' argument and name='trace-name' for any trace name that isn't already in the plot. That will create a new line with that name.

Thank you very much! I've solved it.~


Fix inconstancy with data mappings file

at the moment the activities object is really made of 'domain and activities'

this causes inconsistencies like the one seen below

One interesting challenge is that having the domains in the source file will make it easier to edit manually (and understand)
This is related to 'Find better way to map the domain and practice' #90

This has been addressed by recent refactorings of data object


add test for python 3.11

Codecov Report

Merging #84 (4b6b3a7) into main (096605e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #84   +/-   ##
=======================================
  Coverage   97.77%   97.77%
=======================================
  Files           2        2
  Lines          45       45
=======================================
  Hits           44       44
  Misses          1        1

Flag
Coverage Δ

pytest
97.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Wayland Sway borders/shadows see-through to desktop

          I’ve just installed Lapce 0.3.1 on Alpine Linux v3.19 with Sway/Wayland desktop and I see the same bug – border and shadows are transparent, seeing the desktop background through them.

Originally posted by @jirutka in https://github.com/lapce/lapce/issues/466#issuecomment-2016878963
(I split this out as a separate issue from the original as that was with the previous UI lib, while this is with floem)

Also experiencing this on NixOS

Lapce 0.4.0
Sway 1.8.1

Can someone confirm with latest nightly or source build if that still happens (specifically the see through)?
I cannot reproduce it (although I'm running quite cursed set-up of Lapce inside Sway inside a Fedora container on Fedora/KDE)

Also if you can provide GPU details, as this particular issues tend to be tied specifically to hardware.

I built the latest master (c0fc11ec96ee85228f127e75517cdf6dbd754174) and tried it on my Alpine Linux v3.19 desktop with Sway. I can confirm that the problem with see-through borders and shadows is gone and I haven’t noticed any other issues with z-index so far.
Navigating UI elements using the keyboard still doesn’t work though.

other issues with z-index so far.

There are still some when using LSP suggestions or in other UI places, but we are still working on proper resolution.

Navigating UI elements using the keyboard still doesn’t work though.

Can you clarify a bit more? I presume it's about tab/shift+tab and arrow keys navigation in file tree or menus/dropdown?


[home] use an enum instead of ints #trivial

Forgot to do this in my last PR but it's a bit nicer.
Skip New Tests.

merging


New collections abstraction

Decided to go in a different direction with collections stuff.  Removed the CollectionsLibType stuff and replaced it with a new CollecitonsAbstraction which defines things like monadic operations on a collections library, as well as lifting AST nodes of particular types into this abstraction.
There are some warts to it... an interesting idea for this was that you could encode some type information in guardrail itself to make things a bit safer.  So you could do something like:
val five = new IntegerLiteralExpr("5").lift[Int]
val optFive = five.liftOptionalTerm[Option[Int]]
val plusOne = new LambdaExpr(new Parameter(new UnknownType, "number"), new BinaryExpr(new NameExpr("number"), new IntegerLiteralExpr("1"), BinaryExpr.Operator.PLUS)).lift[Int => Int]
val maybeFivePlusOne = optFive.map(plusOne)
val unliftedExpr: Expression = maybeFivePlusOne.value

... and then the generated code would be, for java stdlib:
Optional.ofNullable(5).map(number -> number + 1)

... or for java vavr:
Option.of(5).map(number -> number + 1)

However, it turns out that it's rare that we actually have types like Int lying around.  Usually it's a type that's going to be generated by guardrail, so we can't use it as a type parameter in guardrail itself (best we could do would be AnyRef), or it's some primitive type, but we don't have information as to what type it is anymore anyway (so now we just have to use Any).
There's definitely value to some of it: knowing that something is Option[Any] vs. Vector[Any] vs. even Future[Vector[Any]] can be useful to keep generation safer.
Another wart is that javaparser obviously doesn't expose type information for types contained in expressions.  So I could do
new IntegerLiteralExpr("5").lift[Int => String]

... and the machinery would allow that.
So there's probably some room for improvement here, but I do think we're going to hit a ceiling pretty quickly on how type-safe we can be.  But I still think this is a nice start, and is a decent base to build off of.
(Thanks of course go to @blast-hardcheese for the idea and initial implementation of it; this is definitely something I wouldn't've come up with on my own.)
Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

Codecov Report

Merging #887 (e263ffc) into master (7d12a01) will decrease coverage by 0.08%.
The diff coverage is 96.42%.

@@            Coverage Diff             @@
##           master     #887      +/-   ##
==========================================
- Coverage   83.45%   83.36%   -0.09%
==========================================
  Files          74       73       -1
  Lines        5488     5460      -28
  Branches      164      162       -2
==========================================
- Hits         4580     4552      -28
  Misses        908      908

Impacted Files
Coverage Δ

.../com/twilio/guardrail/languages/JavaLanguage.scala
50.00% <ø> (ø)

...nerators/Java/AsyncHttpClientClientGenerator.scala
94.38% <95.71%> (+0.07%)
:arrow_up:

.../twilio/guardrail/generators/Java/Dropwizard.scala
100.00% <100.00%> (ø)

...il/generators/Java/DropwizardServerGenerator.scala
95.04% <100.00%> (ø)

...o/guardrail/generators/Java/JacksonGenerator.scala
95.03% <100.00%> (ø)

...m/twilio/guardrail/generators/Java/SpringMvc.scala
100.00% <100.00%> (ø)

...ail/generators/Java/SpringMvcServerGenerator.scala
90.75% <100.00%> (ø)

...a/com/twilio/guardrail/generators/JavaModule.scala
100.00% <100.00%> (ø)

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 7d12a01...e263ffc. Read the comment docs.


unable to run tern on alpine images

while running tern on container image with alpine base image seeing following errors
    Layer 2:
            info: Layer created by commands: /bin/sh -c apk add --no-cache openjdk17
            info: Retrieved package metadata using apk default method.

            error: /bin/sh: export: line 0: JAVA_OPTS:: bad variable name

/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
    File licenses found in Layer:  None
    Packages found in Layer: None

To Reproduce
Steps to reproduce the behavior:

run tern scan from docker image (ternd report  -w test.tar -o test.txt)

seeing this error
2023-04-28 12:56:09,110 - ERROR - rootfs - Command failed. /bin/sh: export: line 0: JAVA_OPTS:: bad variable name

for image layer1 the output is clean
Layer 1:
info: Layer created by commands: /bin/sh -c #(nop) ADD file:9a4f77dfaba7fd2aa78186e4ef0e7486ad55101cefc1fabbc1b385601bb38920 in /
info: Found 'Alpine Linux v3.17' in /etc/os-release.
info: Retrieved package metadata using apk default method.
 File licenses found in Layer:  None
 Packages found in Layer:
 +------------------------+-------------+-------------------------------------------+------------+
 | Package                | Version     | License(s)                                | Pkg Format |
 +------------------------+-------------+-------------------------------------------+------------+
 | alpine-baselayout-data | 3.4.0-r0    | GPL-2.0-only                              | apk        |
 | musl                   | 1.2.3-r4    | MIT                                       | apk        |
 | busybox                | 1.35.0-r29  | GPL-2.0-only                              | apk        |
 | busybox-binsh          | 1.35.0-r29  | GPL-2.0-only                              | apk        |
 | alpine-baselayout      | 3.4.0-r0    | GPL-2.0-only                              | apk        |
 | alpine-keys            | 2.4-r1      | MIT                                       | apk        |
 | ca-certificates-bundle | 20220614-r4 | MPL-2.0 AND MIT                           | apk        |
 | libcrypto3             | 3.0.8-r3    | Apache-2.0                                | apk        |
 | libssl3                | 3.0.8-r3    | Apache-2.0                                | apk        |
 | ssl_client             | 1.35.0-r29  | GPL-2.0-only                              | apk        |
 | zlib                   | 1.2.13-r0   | Zlib                                      | apk        |
 | apk-tools              | 2.12.10-r1  | GPL-2.0-only                              | apk        |
 | scanelf                | 1.3.5-r1    | GPL-2.0-only                              | apk        |
 | musl-utils             | 1.2.3-r4    | MIT AND BSD-2-Clause AND GPL-2.0-or-later | apk        |
 | libc-utils             | 0.7.2-r3    | BSD-2-Clause AND BSD-3-Clause             | apk        |
 +------------------------+-------------+-------------------------------------------+------------+

subsequent layers it doesn't show the correct results

=======================================================================================
    Layer 2:
            info: Layer created by commands: /bin/sh -c apk add --no-cache openjdk17
            info: Retrieved package metadata using apk default method.

            error: /bin/sh: export: line 0: JAVA_OPTS:: bad variable name

/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
/bin/sh: export: line 0: JAVA_OPTS:: bad variable name
    File licenses found in Layer:  None
    Packages found in Layer: None

=======================================================================================
Expected behavior
Expected to see the results without errors.
Environment you are running Tern on
Enter all that apply

Output of 'tern --version'
Tern version 2.12.0
python version = 3.9.16 (main, Apr 12 2023, 07:00:42)
Operating System (Linux Distro and version or Mac or Windows)
Ubuntu
Vagrant file
Container OS
Python version (3.6 or higher)
Cloud environment (AWS, Azure, GCP)

Please attach files if they exist

tern.log
report.*

Hi @dnelaturi, thanks for the issue! Can you please provide the Dockerfile for your image so I may debug? From what I can see this is not an issue with the Alpine base image. Rather, I suspect there is some type of unique line in the Dockerfile that Tern's parser does not recognize or an environment variable that Tern cannot properly export, hence the /bin/sh: export: line 0: JAVA_OPTS:: bad variable name error that is throwing off the results.


Delegates reconnect logic to the XMPP adapter

The reconnect logic is currently broken, as far as I can tell. Currently, when a disconnect occurs, the onOffline method is invoked, which invokes the disconnect method, which closes the connection, which invokes the onOffline method, etc... until the stack is blown. Log messages look like:
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] DEBUG Disconnecting here
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] INFO Connection went offline
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] DEBUG Disconnecting here
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] INFO Connection went offline
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] DEBUG Disconnecting here
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] INFO Connection went offline
[Wed Sep 30 2015 12:15:20 GMT-0400 (EDT)] DEBUG Disconnecting here

undefined:0
(null)

RangeError: Maximum call stack size exceeded

In any case, the XMPP client itself has reconnect logic, so I think it makes more sense to delegate the functionality there. It has the effect of resolving the stack overflow issue too.

Is this suppose to provide a different error when you type in the wrong password aswell?

Is this suppose to provide a different error when you type in the wrong password aswell? @alindeman

Maybe, I haven't tested this case myself yet. Are you seeing unexpected behavior with my patch?

Maybe, I haven't tested this case myself yet. Are you seeing unexpected behavior with my patch?

I have tested your patch with the wrong password. I have not had time to reproduse any disconnections or bad internet, so I loose connection with hubot, to test your patch. But I experience the same message and full callstack with wrong password. The reccurring disconnection message with a full call stack is anyway bad feedback for a user who has typed the wrong password :stuck_out_tongue:


Dask workers should run with same permissions as the user that launches the dask gateway

This issue addresses the comments and hiccups discussed here.
The need for this ticket is to keep track of the issue within the QHub repository.

I still think this issue should be addressed.

@costrouc Have we already adressed this feature?

@viniciusdc,
This has not yet been addressed.  I just checked on qhub v0.3.12, and confirmed dask workers are running on root.
Here is a probably incomplete and only partially tested example of what I did to check:

whoami.py
from dask_gateway import Gateway

gateway = Gateway()
options = gateway.cluster_options()
cluster = gateway.new_cluster(options)
client = cluster.get_client()
cluster.scale(1)

def whoami():
    import subprocess

    result = subprocess.run(["whoami"], capture_output=True, text=True)
    return result.stdout.strip()

results = client.run(whoami)
print(results)

client.close()
cluster.close()
del client
del cluster

It should also be noted that /home/conda is being mounted as (rw), so while users' jupyterlab instances cannot modify conda-store environments because they are not running as root, dask workers can.  So in our case, when we need to install our internal package on workers, but because we want to ensure it is not installed into the conda-store environment, we have to take special care to install the package in user mode and the install location added to the python system path, which we accomplish with the following:

pip-user.py
import os
import site
import sys

cmd = sys.executable + ' -m pip install -U --user "git+https://{token_string}github.com/{repo}.git{version_string}"'
os.system(cmd)
sys.path.append(site.USER_SITE)

So while I still think this issue should be fixed, now I am also curious if the mount of /home/conda should be changed to be read only?


doc: fix some typo

@halfcrazy  Why submit so many documents?

@halfcrazy  Please refer to the translation documents, according to the process to join the translation organization submitted, thank you.


update gaiad version to v9.1.0

pls note:

fixed v9 upgrade name
no upgrade name for v9.1.0 because no SoftwareUpgradeProposal

oops looks like the bot doesn't like
"name": null,
can do
"name": "none", or similar?

oops looks like the bot doesn't like "name": null, can do "name": "none", or similar?

null is not a string. But I'd prefer just 'v9.1' over 'none'; makes way more sense, IMO, even if it's not official.

I think in that case the full tag v9.1.0 probably would make the most sense. however I also think there maybe should be another field in the schema to indicate whether a consensus breaking upgrade has an upgrade proposal, because if not it must be handled differently than a scheduled upgrade/

I think in that case the full tag v9.1.0 probably would make the most sense. however I also think there maybe should be another field in the schema to indicate whether a consensus breaking upgrade has an upgrade proposal, because if not it must be handled differently than a scheduled upgrade/

Wait, so this was an unscheduled consensus-breaking upgrade? I assumed that would never happen, except for maybe emergency patches...
Still I think this having a name makes sense--proposal or not.


Add trace-level rollup support (outreach fork)

Updates honeycomb-beeline with a new method, incrementTraceRollup,
which can be used to accumulate running counters over the course of a
request, to be reported on the top-level trace span when the request is
finished.
The original, unpatched honeycomb-beeline had some not-well-documented
suport for rollups, but it doesn't work with async spans, and most spans
are async.  This patched version will work with async spans, though it
does assume that even async spans don't outlive their parent trace.
That's generally true for us, but it's also a stronger assumption than
the upstream libraries are willing to make.  Our counts may be lower
than expected if that expectation is violated.

We have reviewed this code elsewhere, and it will be reviewed again soon, so I'm going to go ahead and merge this.


FYI - Detecting Canvas Defender and recovering the original canvas value

Evaluating the privacy implications of a canvas fingerprinting countermeasure

In this post, we showed that contrary to what’s claimed in their article, Canvas Defender can easily be detected. We showed that detecting if a user has Canvas Defender installed can be checked by looking at the string representation of toDataURL. It can also be detected using the MutationObserver API. The advantage of this approach is that it enables to extract the noise vector used by Canvas Defender. Depending on the settings chosen by the user in the extension, this noise vector may remain the same until the user deliberately changes it. Finally, we showed how to recover the original canvas value, even in the presence of Canvas Defender.

Can mozilla and google recover the original canvas value if you use privacy.resistFingerprinting to spoof it?

Just as well we don't recommend canvas defender - personally I think it's premise is flawed. Haven't read the article yet
note: https://kkapsner.github.io/CanvasBlocker/test/detectionTest.html
I don't think anyone cares if their canvas is detected as fake - eg RFP's hash gives it away as a 10x10 white canvas. But ID'ing the actual extension by name could raise entropy. For RFP users, we're already in the RFP set.

Can mozilla and google recover the original canvas value

Good question. I have no idea.
For Canvas Blocker - @kkapsner
For RFP - @tomrittervg , @arthuredelstein

From Arthur

I think RFP is safe from this attack because, when RFP is enabled, the canvas image extraction APIs (such as toDataURL) are sanitized in native code.

 Thanks for sharing that link.
This also makes some of our userscripts easily detectable btw ...
Ctrl+K then enter history.__lookupGetter__('length').toString() to see the custom function that spoofs history.length

I think all the points in the article do not apply to CanvasBlocker. The point is that Firefox has some additional WebExtension APIs that allow addon-code to run in the (security) context of the web page (exportFunction). But if you create a WebExtension that also has to run on Chrome or even Edge you cannot use them.
I think I also did a test where I could extract the canvas fingerprint with Canvas Defender installed without any additional code - the addon code was just not loaded fast enough
@earthlng: these APIs would also allow the history.length to be spoofed in a sneaky fashion. To be honest: it would be really simple to include this in CanvasBlocker... ;)

Just realized they removed the exportFunction documentation: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/Content_scripts$compare?locale=en-US&to=1348378&from=1348018
Hopefully they do not want to ditch it.

Here my test page for the extraction: https://canvasblocker.kkapsner.de/test/settingsLoading.php
Canvas Defender:

CanvasBlocker:
 notice the fingerprint icon in the address bar (here I can give you the real data as they are faked ;))

Ctrl+K then enter history.__lookupGetter__('length').toString() to see the custom function that spoofs history.length

Ctrl-SHIFT-K btw. Holy cow. This is all a bit over my head. Does this mean that (aside from kkapsner's info on exportFunction which is a concern if it is removed), that crafty scripts can enumerate unique function names and procure a list of web extensions? That would be a high entropy FP
@kkapsner Wow dude, thanks. I have the utmost faith in you - you've proven it time and time again :heart:

RFP
fingerprint consistent (e5d9fd78536844cc8a4144ddb7a03eb9628f12c7c8b7828f942cadf6efb79ac0) -> good!

what on earth is e5d... - that's not the RFP 10x10 a273d6a847f0e2a57fa0161158f12fed

Am concerned about that exportFunction being removed. Found your ticket from a while ago ( https://bugzilla.mozilla.org/show_bug.cgi?id=1081385 ) .. component XPConnect, fixed in FF59. Doesn't seem like something they would deprecate. Maybe we should ask Boris

FYI @antoinevastel

@antoinevastel et al - https://www.ghacks.net/2018/07/04/study-on-the-effectiveness-of-fingerprinting-countermeasures/
PS: I bookmarked and read that paper a couple of days ago :+1: after reading this: https://freedom-to-tinker.com/2018/06/29/against-privacy-defeatism-why-browsers-can-still-stop-fingerprinting/

Regarding e5d...: I get this as well when disabling CB and using RFP - so it's the correct value. I use a different canvas size (220px × 30px) and therefore the hash is different (also I do not know which hashing algorithm panopticlick is using - I use sha256 of the data-URL).

Just realized they removed the exportFunction documentation

they moved it to https://developer.mozilla.org/en-US/Add-ons/WebExtensions/Sharing_objects_with_page_scripts

@kkapsner

these APIs would also allow the history.length to be spoofed in a sneaky fashion.

can you show me how, please? I can't wrap my head around how to exportFunction a getter in a way that still makes it possible to check the original history.length (to know what to return) while at the same time prevent the page from seeing the custom function.

I'm just going to @gorhill here becuz 1. it's really interesting that without exportFunction, JS can extract custom function names (I assume I got that right), and 2. i want piece of mind from the owner that uM, uBO are OK
Imagine grabbing the top 200 web extensions in AMO and working out which ones you could ID using this method (see link in OP), eg a unique function name - lets say you could easily ID 150 of them - that's a lot of entropy - unless I've completely missed the point (ELI5 if I have)

@earthlng: thanks for the new link - good to know that it is still documented.
For the "how to exportFunction a getter" you can check CB-Code as that is exactly what I do: https://github.com/kkapsner/CanvasBlocker/blob/master/lib/intercept.js#L179
You first store the original function (has to be slighly different in your case as you have to store the getter) and then you redefine the getter.
@Thorin-Oakenpants: I do not think that the situation is that bad. Most dddons do not need to provide functions that can be called by the website. But I have not checked and unfortunately all the anti-fingerprinting and some of the privacy addons have to. So if they are not carefully they might make things wore.

But I have not checked and unfortunately all the anti-fingerprinting and some of the privacy addons have to. So if they are not carefully they might make things worse.

Thanks. OK, I'll reword it then ... Imagine grabbing the top 50 "privacy / anti-FP'ing" web extensions in AMO and working out which ones you could ID using this method (see link in OP), eg a unique function name - lets say you could easily ID 30 of them - that's still a lot of entropy
I think this might require us to vet all our recommended addons, but I have no idea how to do that

I think this might require us to vet all our recommended addons, but I have no idea how to do that
One by one on an individual basis. Basically look in the code of the addon or the "bad" guys and see what of the browsers APIs they are using/changing.

might as well close this.  Basically look in the code of the addon or the "bad" guys and see what of the browsers APIs they are using/changing.

Yeah, but I can't read that - it's all Greek to me (actually, that's a terrible saying to use, because I actually took classical Greek at university) it's all Hungarian to me


network message text enhancement

refer to https://github.com/rancher/harvester-ui/issues/218#event-4070842582

[x] If the physical NIC is invalid in settings: on the Network list page: Change the text to say "Physical NIC value in Setting is invalid".  Have "Settings" be a link to the network settings.  this could be addressed in the UI side

[x] In the Network setting if you enter a wrong NIC, the message should say '[nic field info] is not a valid physical NIC."

close in favor of https://github.com/rancher/harvester-network-controller/pull/17


Encrypted twice

Although this should be impossible, I think I encrypted the same wallet twice. I updated to a new wallet version. While waiting for my coins to show up, I encrypted. Then my coins (that were already encrypted) showed up. (I assume at the same exact time that I was encrypting)  Therefore, it was encrypted twice and I am subsequently locked out. My original encryption no longer works since this "double encryption" happened.
Info: I used the same encryption password the whole time. I did not choose a new one. Just simply encrypted again.
Is there something in place that makes the system change the password entirely when this happens? Or the system doesn't know how to respond when this happens? Any suggestions on what may unlock the wallet at this point?

Hi,
I don't believe that's how it works -- the entire wallet is encrypted, not individual "coins". If you updated to a new software version, the software should have still recognized that your wallet was already encrypted and not allowed you to encrypt again.
Can you confirm you were using the Dash-Qt wallet, or was it another wallet?

I think that is the problem. Because it did let me even though it probably
shouldn’t have. I must have entered an encryption (even though it was the
same one) while it was syncing and before the coins showed up. Then once
they appeared, it was “double” encrypted. It was all in a matter for a
minute or less I’d say.
On Sunday, March 25, 2018, Nathan Marley<EMAIL_ADDRESS>wrote:

Hi,
I don't believe that's how it works -- the entire wallet is encrypted, not
individual "coins". If you updated to a new software version, the software
should have still recognized that your wallet was already encrypted and not
allowed you to encrypt again.
Can you confirm you were using the Dash-Qt wallet, or was it another
wallet?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashpay/dash/issues/2013#issuecomment-375986693, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AkBxYC-CO9fWIc6894lqvPXJTPyY3uYvks5th9BFgaJpZM4S6NUZ
.

I must have entered an encryption (even though it was the same one) while it was syncing and before the coins showed up
I'm not sure if that's possible but can't rule it out, as I'm not as familiar with the Qt wallet portion of the software.

My recommendation at this point is to locate your previous backups of wallet.dat and restore using that. Make sure you have everything backed up so that at any step you're not over-writing something that you can't get back.

It is the qt wallet. And unfortunately, I am new to this one so, so far I
only had one backup. It shows zero coins. That’s why I want to ask the
techs if they can help with this.
On Sunday, March 25, 2018, Nathan Marley<EMAIL_ADDRESS>wrote:

I must have entered an encryption (even though it was the same one) while
it was syncing and before the coins showed up
I'm not sure if that's possible but can't rule it out, as I'm not as
familiar with the Qt wallet portion of the software.
My recommendation at this point is to locate your previous backups of
wallet.dat and restore using that. Make sure you have everything backed
up so that at any step you're not over-writing something that you can't get
back.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashpay/dash/issues/2013#issuecomment-376007040, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AkBxYKdTR1am6LiKQ3BWhHurljTiutgTks5tiBM4gaJpZM4S6NUZ
.

I do, however, when i open them they either show a balance of zero coins
(the first backups before the encryption) or they are encrypted and I can
not open them.
On Wed, Apr 4, 2018 at 9:02 PM, Nathan Marley<EMAIL_ADDRESS>wrote:

Do you have any auto-generated backups in your "backup" folder?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashpay/dash/issues/2013#issuecomment-378790690, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AkBxYHdfjPsdEHtCRCv2JljxRU7vgL0kks5tlW0ZgaJpZM4S6NUZ
.

I would verify that the coins were not moved from the addresses you think they should have been in. Possible an attacker gained access to your wallet.dat

At this point, I think we should close this issue, since it's been two years w/o response


pp_calc always returning null

Inputting any beatmap id into pp_calc will always return null. Example below.
const data = await tools.pp_calc(3555372);
console.log(data); // null

I performed a GET request without using pp_calc to the corresponding url, https://pp.osuck.net/pp?id=3555372, which worked fine.

should be fixed

Seems to be working for me. Closing now.


Not visible when zoomed in

When using "tap to zoom" or "Smart Zoom" or whatever it's called, the status bar isn't visible.

Duplicate of #18.


AppState feature request: option to disable logs

A message is displayed in the console after each stage change. This is very helpful in some cases, but I would love to have an option to disable it.

It is disabled by default in v1.1.0.
I am working right now with a debug package. I will upload it soon.

Closed as this would be solved by https://github.com/worona/meteorflux/issues/22


Installer website lying to me

The installer website says neptune installed successfully, but when I open tidal there is no neptune options in the drop-down menu. Please help

same here

did you close and reopen tidal?

Sometimes when TIDAL updates to a major version (ex: 2.38 > 2.39), their file structure changes, and neptune is linked to the old version. The installer says it's installed because it exists, but in fact it's not in the right version (it's in this circumstances that neptune simply vanish after a TIDAL update).
The only solution I found was uninstall TIDAL, going to AppData and deleting all tidal-related folders and reinstall both again.
If you're installing for the first time, then idk :/ (maybe you had TIDAL opened when installing neptune or you didn't restart TIDAL...)

This issue is no longer present due to the existence of the modern neptune installer.


Fix test

Codecov Report

Merging #898 into xprotocol_stable will decrease coverage by <.01%.
The diff coverage is 0%.

@@                 Coverage Diff                  @@
##           xprotocol_stable     #898      +/-   ##
====================================================
- Coverage             55.45%   55.45%   -0.01%
====================================================
  Files                   234      234
  Lines                 23866    23869       +3
====================================================
  Hits                  13236    13236
- Misses                 9417     9419       +2
- Partials               1213     1214       +1

Impacted Files
Coverage Δ

pkg/proxy/proxy.go
8.13% <0%> (-0.21%)
:arrow_down:

pkg/log/logger.go
54.69% <0%> (-1.66%)
:arrow_down:

pkg/module/http2/frame.go
84.17% <0%> (-0.31%)
:arrow_down:

pkg/module/http2/transport.go
78.44% <0%> (-0.08%)
:arrow_down:

pkg/module/http2/server.go
81.7% <0%> (+0.14%)
:arrow_up:

pkg/network/connection.go
34.61% <0%> (+0.73%)
: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 d12e53a...bca1188. Read the comment docs.


Use standard library 1.6?

The stdlib has released its 1.6 version so I think we should migrate the dependency.

Related: #243, #267.

If the CI is happy, this could be closed via #266.

Fixed in #266 (and addressed #267).


Create Gitter channel

Some discussion is better done through slow-chat than through Github issues. Gitter seems to work quite nicely, with a pretty web UI and apps for various mobile platforms, integration with Github, and I'm already using it for some other projects.
What are your thoughts?

I personally have never used Gitter, but I have seen the link to it on many other Github projects.
I think we can give it a go, as it is free and you are absolutely right that some discussion would be better to have in a chat than in Github issues.
So I'll add a link to it and let's see how this works :-)


Reject support for component glogin

https://camel.apache.org/glogin.html

Found 0 matching posts for subject:glogin in Camel - Users within 700 days

Reject support for Google App Engine (GAE)

Not in catalog any more


Training times v1 vs. v2 vs. v3?

Hello, I see it took you this amount of time to train v1:

It took about 13-14 days to train the model up to 2,500k steps with two V100 GPUs.

Since v2 and v3 have faster inference, does that mean training them would be faster too?

Thanks for your interest!
Yes. Although V2 and V3 are trained faster than V1, there is no difference as much as the difference in inference speed because the discriminator takes longer during training.
When training V3, the training time is shortened by approximately 13~14% compared to V1.

Awesome, thank you @jik876!


fix Keep PartitionInfo compatible with the cloud mode

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


MSVC warnings: narrowing conversion, signed/unsigned mismatch

Description
The following in doctest.cpp can cause MSVC to raise C4245 (signed/unsigned mismatch) and C4838 (narrowing conversion). This is because EXCEPTION_ILLEGAL_INSTRUCTION and friends may be defined as either a DWORD (unsigned long) or NTSTATUS (signed long) depending on whether ntstatus.h is included.
Simplest solution would be to static_cast to DWORD.
    struct SignalDefs
    {
        DWORD id;
        const char* name;
    };
    // ...
    SignalDefs signalDefs[] = {
            {EXCEPTION_ILLEGAL_INSTRUCTION, "SIGILL - Illegal instruction signal"},
            {EXCEPTION_STACK_OVERFLOW, "SIGSEGV - Stack overflow"},
            {EXCEPTION_ACCESS_VIOLATION, "SIGSEGV - Segmentation violation signal"},
            {EXCEPTION_INT_DIVIDE_BY_ZERO, "Divide by zero error"},
    };

// winnt.h (included by windows.h)
#define STATUS_ILLEGAL_INSTRUCTION ((DWORD)0xC000001DL)

// ntstatus.h
#define STATUS_ILLEGAL_INSTRUCTION ((NTSTATUS)0xC000001DL)

// minwinbase.h (included by windows.h)
#define EXCEPTION_ILLEGAL_INSTRUCTION STATUS_ILLEGAL_INSTRUCTION

Steps to reproduce
#define WIN32_NO_STATUS // suppress macro redefinition warnings
#include <windows.h>
#undef WIN32_NO_STATUS
#include <ntstatus.h>

#include <doctest.h>

Extra information

doctest version: v2.4.1
Operating System: Windows
Compiler+version: MSVC 19.28, Windows SDK 10.0.19041.0

merged the PR - closing this now. Will release an official version..."soon"!


log more

@rultor release, tag is 0.0.12

@rultor release, tag is 0.0.12

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.12

@yegor256 Done! FYI, the full log is here (took me 7min)


Included assimp headers are out-of-sync with later assimp versions causing stack corruption in some cases

It looks like the assimp headers that you have in your external/assimp/assimp folder are pretty old (from the v3.2 of assimp). Using a newer version of assimp (it's now on v5.0.1) results in a mismatch between your version of the headers and assimp's. That causes stack corruption when using parts of the assimp API. That causes problems for me (in embedded Linux) in 'scenerendering' and 'skeletalanimation'.
I've changed our build to pull the v3.2 tagged version of assimp rather than the latest and those problems go away. The minimal change here would be to document that exactly v3.2 was required, I think you currently state that at least v3.2 is needed.

hi, Gary
How to pull the v3.2 tagged version of assimp?

I have removed ASSIMP and replaced it with a custom glTF loader based on the header-only tinyglTF, so closing this.


Sim

All I'm trying do is feed my 2 year old son ik you might not care but that all I do it for I can't work no 9-5 and take care of him all I'm trying do is learn and once I'm locked in I'm in nothing I search up or have read shows me context so I'm lost plz help


Easier Image Scaling

This fork allows a user to opt in to having images scaled to the screens scale using the SDWebImageOptions bit mask, instead of using an<EMAIL_ADDRESS>suffix in the image key. Images should also be scaled to @3x on the iPhone 6 plus using this method
Oh, and I also had to add SDWebImageTransformAnimatedImage to the SDWebImageOptions to get the project to compile

This complicates the APIs, I would not merge it in.


Batcher error on messages marked as failed

Hi guys, awesome library!
I have a pipeline where the message process can fail (on handle_message/3 callback).
If I just mark the message as failed with Message.failed/2 I get the error message was set to unknown batcher ....
I'm not sure if this is the expected behavior. If I mark the message as failed, I don't expect the batcher to be set as it's never going to be forwarded.

Hi @luisgabrielroldan.
That's not the expected behaviour. Can you post the configuration of your pipeline along with the code in your handle_message?

Hi @msaraiva
I can post the configuration, but I think is not really relevant.
In the processor code here:
https://github.com/plataformatec/broadway/blob/d80ab6fb4c56300886c45d64babcd1677f500702/lib/broadway/processor.ex#L77
The message is validated immediately no matter the state of the message. I think there is the problem.


Please invite me to the GitHub Community Organization

I am very interested in this project, please invite me to join, tk

Welcome， @chenhaonan-eth ！
Invited.


Licensing complexity seems an obvious barrier to uptake

The length and complexity of this incomplete FAQ highlights that the complexity of the licensing is a barrier to the uptake of these otherwise useful products. Your audience who will drive the success or failure of these products are busy knowledge workers who just want to get on with producing results - they dont have time for this complexity.  Its frustrating because it is holding back otherwise useful products from broader adoption.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a2cad5bd-4ffa-3671-0364-95e5b6f65256
Version Independent ID: 6d0cc055-b7be-02d7-3ba8-55e400ccd21b
Content: Power Apps and Power Automate licensing FAQs - Power Platform Admin center
Content Source: power-platform/admin/powerapps-flow-licensing-faq.md
Service: power-platform
GitHub Login: @dileepsinghmicrosoft
Microsoft Alias: dileeps

Please specify whether any of the power platform services are included in an E3 or E5 MICROSOFT 365 license

Please specify whether any of the power platform services are included in an E3 or E5 MICROSOFT 365 license
As I understood from the licencing guide, any F1-E1-E3 user may have up to 2000 of daily executions, but only standard connectors. Bye-bye nice time when for 5$ per month we could convert word to pdf or other premium stuff... The other way is to user Logic apps.


I need an example JSON submit Form POST

Is it possible to use JSON in a form that can send using the POST method to the server? And create a form using GET?

Sorry but I don't understand what you need, and how it is related to this library

I want create a FORM using JSON file format which DATA stored in JSON file will be sent via POST method to SERVER.

yes it is possible, once you obtain the JSON data from the form generated by the library (obtained from a JSON Schema that you specify) you can do whatever you want with it in javascript, from example sending its JSON representation using another form under your control, via AJAX, or websockets


Separate CodeFixes from analyzers to ensure analyzers don't depend on workspaces

Fixes #24

Codecov Report

Merging #96 into v15.8 will decrease coverage by 1.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            v15.8      #96      +/-   ##
==========================================
- Coverage   91.24%   89.91%   -1.34%
==========================================
  Files          12        9       -3
  Lines         651      446     -205
  Branches      115       91      -24
==========================================
- Hits          594      401     -193
+ Misses         24       23       -1
+ Partials       33       22      -11

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 ed28a9c...ad82e88. Read the comment docs.


Add 'list' and 'set' creation shortcuts.

(list 0 1 2 3 4)

Should be a shortcut for
(add 0 1 2 3 4 (default (list int)))

(set 0 1 2 3 4)

Should be a shortcut for
(add 0 1 2 3 4 (default (set int)))

This requires the instruction set to become set!.

Done in https://github.com/nsensfel/tonkadur/commit/034153205de05ef2a4facf7aded9912ce8dfb6b5.


Fix package permission checks for organizations

fixes #19986
Is there already a helper method like GetOrgUserMaxAuthorizeLevel which does this?

Backport in #20520


Add antctl command to dump OVS flows of a NetworkPolicy

Look up all cached flows of a NetworkPolicy from policyRuleConjunction
and conjMatchFlowContext. Execute ovs-ofctl dump-flows to dump the
flow strings.
Add NetworkPolicy name and Namespace to NetworkPolicyController rule
cache and policyRuleConjunction to look up a NetworkPolicy and its
flows.
Fix flow match string for TCP/UDP/SCTP port and register matches.

Example results:
antctl get of  -n ns1 --networkpolicy np1
FLOW
cookie=0x29000000000000, duration=4.446s, table=100, n_packets=0, n_bytes=0, priority=200,ip,reg1=0x5 actions=drop
cookie=0x29050000000000, duration=4.317s, table=90, n_packets=0, n_bytes=0, priority=190,conj_id=1,ip actions=resubmit(,105)
cookie=0x29050000000000, duration=4.340s, table=90, n_packets=0, n_bytes=0, priority=200,conj_id=1,ip,nw_src=<IP_ADDRESS>/24 actions=resubmit(,100)
cookie=0x29050000000000, duration=4.361s, table=90, n_packets=0, n_bytes=0, priority=200,ip,nw_src=<IP_ADDRESS>/16 actions=conjunction(1,1/3)
cookie=0x29050000000000, duration=4.383s, table=90, n_packets=0, n_bytes=0, priority=200,ip,reg1=0x5 actions=conjunction(1,2/3)
cookie=0x29050000000000, duration=4.404s, table=90, n_packets=0, n_bytes=0, priority=200,tcp,tp_dst=80 actions=conjunction(1,3/3)
cookie=0x29050000000000, duration=4.425s, table=90, n_packets=0, n_bytes=0, priority=200,udp,tp_dst=1234 actions=conjunction(1,3/3)

Example results:
antctl get of -n ns1 --networkpolicy np1
FLOW
cookie=0x29000000000000, duration=4.446s, table=100, n_packets=0, n_bytes=0, priority=200,ip,reg1=0x5 actions=drop
cookie=0x29050000000000, duration=4.317s, table=90, n_packets=0, n_bytes=0, priority=190,conj_id=1,ip actions=resubmit(,105)
cookie=0x29050000000000, duration=4.340s, table=90, n_packets=0, n_bytes=0, priority=200,conj_id=1,ip,nw_src=<IP_ADDRESS>/24 actions=resubmit(,100)
cookie=0x29050000000000, duration=4.361s, table=90, n_packets=0, n_bytes=0, priority=200,ip,nw_src=<IP_ADDRESS>/16 actions=conjunction(1,1/3)
cookie=0x29050000000000, duration=4.383s, table=90, n_packets=0, n_bytes=0, priority=200,ip,reg1=0x5 actions=conjunction(1,2/3)
cookie=0x29050000000000, duration=4.404s, table=90, n_packets=0, n_bytes=0, priority=200,tcp,tp_dst=80 actions=conjunction(1,3/3)
cookie=0x29050000000000, duration=4.425s, table=90, n_packets=0, n_bytes=0, priority=200,udp,tp_dst=1234 actions=conjunction(1,3/3)

As there is no priority in the output, would you mind to print the flows in the same table in the order of the priority?

As there is no priority in the output, would you mind to print the flows in the same table in the order of the priority?
Priority is included in the output flow strings. For network policy flows, I order the flows for a single rule as: (allow) action flow -> drop flow -> conjunctive match flows -> default drop flow. Does this sound good to you?

/test-all

Priority is included in the output flow strings. For network policy flows, I order the flows for a single rule as: (allow) action flow -> drop flow ->

What do you mean about "drop flow", is it for the "excepts" flows? If yes, I think the "excepts" should have the highest priority, and then the conjunction action flows.

Priority is included in the output flow strings. For network policy flows, I order the flows for a single rule as: (allow) action flow -> drop flow -> conjunctive match flows -> default drop flow

What do you mean about "drop flow", is it for the "excepts" flows? If yes, I think the "excepts" should have the highest priority, and then the conjunction action flows.

Yes. I know "excepts flow" has higher priority, but the current way is easier to implement and needs not to reorder flows by priority. Also feel it is not hard to understand.

Yes. I know "excepts flow" has higher priority, but the current way is easier to implement and needs not to reorder flows by priority. Also feel it is not hard to understand.

I want to confirm the "drop" flow you mentioned is the default drop flow, or the "excepts" flow. If it is excepts flow, it would be OK if this order makes the code simpler. But if it is the default drop flow, the flow table is different from others. Then it should be the last of the flows.

I want to confirm the "drop" flow you mentioned is the default drop flow, or the "excepts" flow. If it is excepts flow, it would be OK if this order makes the code simpler. But if it is the default drop flow, the flow table is different from others. Then it should be the last of the flows.

I meant excepts. Default drop is the last.

/test-all


Migrate to 0.3.0

Related issue(s)

Resolves #646
Description
Summary of changes

Dependency additions / changes

Test additions / changes

Checklist

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[ ] I have linked this PR to the issue(s) that it resolves.
[ ] I have reviewed the changes on GitHub, line by line.
[ ] I have ensured all changes are covered in the description.

Additional context

This PR is blocked by the following:

We need to publish a new version of acvm to crates.io (changing the acvm references in the relevant Cargo.toml's once done)
The aztec_backend migration needs to be merged (changing nargo/Cargo.toml once done)

acvm has marked a few methods as deprecated -- I will create an issue to fix these in this repo, as using the recommended functions will change the behaviour post-migration.

@guipublic please remember that this is blocking anyone using a non-default Brew prefix to be unable build from source. A fast resolution is appreciated.

I've fixed all of the TODOs and we are now using the published version of acvm.
Once the aztec_backend PR has been merged, I'll update the dependency in nargo from my branch to a commit on master

please remember that this is blocking anyone using a non-default Brew prefix to be unable build from source. A fast resolution is appreciated.

I am sorry but I don't understand this sentence! I don't know what a non-default Brew prefix means, neither why we have an urgent build issue nor what caused it. I would recommend though to use a default Brew prefix :)
@kevaundray I am reviewing the acvm and will create an issue with all my concerns so we can see what can be resolved quickly (I am fine with doing the changes myself), what can be delayed and what should be rejected (from my concerns I mean)

I am sorry but I don't understand this sentence! I don't know what a non-default Brew prefix means, neither why we have an urgent build issue nor what caused it. I would recommend though to use a default Brew prefix :)

I created an issue with all of the concerns we went through -- We can push a 0.4 or 0.3.2 to resolve those issues. Once the aztec_backend PR has been merged, this will get merged.


Trigger webhook programmatically without modifying the entity

I have custom API for Item that has subpages for Unit of measure and prices. I want to trigger the Item API webhook when price or unit of measure is added or modified. Is there a way to trigger the webhook via code without modifying the item record?

This repository no longer handles general questions and is only dedicated to bug reports for the latest preview version of the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group

This repository no longer handles general questions and is only dedicated to bug reports for the latest preview version of the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


[Feature Request] Support for "module", "es2015", "typings" fields

Hey, it would be really awesome to not only check for the "main" file, but also for the files defined in the "module", "es2015" as well as "typings" fields. For example, this would be especially interesting for people (like me) developing Angular 2+ libraries - which (at least since Angular 4) also contain the fields named above.
Do you have time to extend the library? Or should I give it a try and open a PR?

Hi @dominique-mueller,
Good idea, working on it on next branch :)
Hope to be able to release something soon.

It's published (v1.1.0) :wink: :tada:
I preferred to avoid making assumptions on which non-standard should be checked by default, instead pkg-ok now accepts additional fields as arguments.

Awesome! Works like a charm!


Rename ND_normalmap to ND_normalmap_float in 1.39

Renaming ND_normalmap -> ND_normalmap_float
Added upgrade mechanism to handle the node definition rename.

USD stages will systematically hardcode the nodedef string into the info:id attribute of the UsdShade nodes. This means all USD persisted data will need an upgrade path to 1.39, and also potentially require storing the MaterialX version somewhere in the document to know if an upgrade is required.
We need to start the discussion with the AOUSD WG because that might become an adoption blocker for MaterialX 1.39.

Thanks for that context, @JGamache-autodesk, and that's the perspective I was hoping you could provide here.  I wonder if the USD project could begin versioning its own documents when it upgrades to MaterialX 1.39, which would allow MaterialX 1.38 content embedded in legacy USD stages to be robustly upgraded to MaterialX 1.39.  The logic that @ld-kerley has proposed here would then work universally for MaterialX content, regardless of the file format in which it has been encoded.

We'll need to address the USD versioning question before the OpenUSD project can upgrade to MaterialX 1.39, as even the simplest possible update to a node interface (e.g. https://github.com/AcademySoftwareFoundation/MaterialX/pull/1659) will require that the MaterialX version of the source document is known.
My sense is that we should move forward with this specific change from @ld-kerley, keeping the need for USD versioning in our minds as an important topic for upcoming USD/MaterialX working group meetings.


Remember to re-enable SSL verification

SSL verification is currently turned off due to the fact that on the staging server the certificates don't match up. This should be fixed on the production server.

Server has been updated for the right SSL configuration, turned back on SSL verification


Merge Strategy creating model__dbt_tmp at the same S3 location as configured in config where my main table got created

Expected behavior
{{
config(
materialized = 'incremental',
incremental_strategy='merge',
unique_key= 'epoch',
properties={
"partitioned_by": "ARRAY['epoch']",
"location": "'s3a://bucket_name/table_name"}
)
}}
SELECT
from_unixtime(1619445982) as epoch_to_local ,
1619445982000 as epoch
With delta as a catalog, we are trying to do this, however post the table creation, during the second execution the same got failed.
The reason of the failure is during the second execution the merge is creating the model__dbt_tmp table in the same location where the main table is which should not be the case.
Actual behavior
The query shouldn't fail and the temp should use some directory other than the temp one.
Steps To Reproduce
{{
config(
materialized = 'incremental',
incremental_strategy='merge',
unique_key= 'epoch',
properties={
"partitioned_by": "ARRAY['epoch']",
"location": "'s3a://bucket_name/table_name"}
)
}}
SELECT
from_unixtime(1619445982) as epoch_to_local ,
1619445982000 as epoch
Use the following configuration and try to run the query twice
Log output/Screenshots
No response
Operating System
mac
dbt version
1.4.1
Trino Server version
398
Python version
Python 3.9.13
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

As mentioned on Slack using location as a table property is not supported as this creates an external table, ie data is not dropped on delete.
Please add the dbt.log for the initial run and the incremental run to confirm.

@aman29jha please provide logs, otherwise we will close the issue


path parameters is always symbol => string

path parameters are always symbol => string on Rails 4.  This is to be
in line with url_for parameters which are also symbol => string.
Setting to a string will cause errors on Rails 4, but using a symbol
should work for both.

:+1:


Added "C" book

This book is from a university called USP, in São Paulo, Brazil.
Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s)
For resources
Description
Added "C" book , scroll down the page and click on "Slides" to read the book.
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x ] Not a duplicate
[x ] Included author(s) if appropriate
[ x] Lists are in alphabetical order
[ x] Needed indications added (PDF, access notes, under construction)

doesn't appear to be a complete book.

Why not? It has a PDF with all the 162 pages. Have you scrolled down the page and clicked on "Slides" as I said on the description?

If you click on "Sumario", it will give you this page, it has all the  chapters of the book

The last item of the "Sumario" is "Árvores binárias de busca" - "Desempenho dos algoritmos", and it is the last thing the PDF has, i checked the other parts and I am sure it is a complete book

Not trying to be rude, just saying it is a free and complete book hehehe.


support sprites / contact sheets

ie a plugin version which loads one image and references the required sprite / image according to  timeline position

Hi there. Thanks for a great product! (both Flowplayer and this plugin).
I'm customizing the flowplayer-thumbnails plugin to accommodate a single JPG using sprites for a project, so I could submit a PR once everything's in place.
What are your thoughts on the API design? I'd propose a new sprite property, which would include metadata about the shape of the sprite grid.
For example, this configuration would work with this contact sheet (taken from Youtube), but is flexible to handle other grid shapes.
thumbnails: {
  template: 'thumbnails/contact-sheet.jpg', // single file, not necessarily a template
  sprite: {
    thumbnailWidth: 80,     // width of thumbnail within overall image
    thumbnailHeight: 45,    // height of thumbnail within overall image
    thumbnailsPerRow: 10,   // so plugin knows when to look at next line of thumbnails
  }
}

Do you have any thoughts or anything else I should know?

@heychrisek - thank you very much, the PR is looking pretty good.
I can take care of stuff like backwards-compatibility with old IEs etc (we don't transpile this small plugin).
Give me a couple of days, and I believe we can integrate this.

@heychrisek - thanks again for your contribution. It's now in. Sorry for delay, I'm on sick leave.

Thanks @blacktrash . No problem at all. Welcome back from leave!


Add Rodrigo and Carlos as translation maintainers

For Brazilian Portuguese

Closing this PR and will submit another which has a table of 'official' reviewers and maintainers/backups instead


Parse Cloud Spanner raw string and byte prefixes

This PR solves #656

Coverage increased (+0.004%) to 83.653% when pulling d27fc7177d14bbad43bf3266a6ddc37ece38a239 on olavloite:parse-cs-raw-string-and-byte into 1d13897883c144892b510314856394012bc0ebb7 on JSQLParser:master.

Good job. May I squash all your commits? If you want to have it under control by yourself, you should provide a one commit pr.
Cheers Tobias

Hi Tobias, Thanks for the reply. Yes, please feel free to squash the commits.
Cheers,
Knut Olav


Disable Lin Out_channel test under FreeBSD

As the title says, this PR disables the negative Lin Out_channel test under FreeBSD.
As #401 documents, this negative test is a source of false CI alarms.
Closes #401.

Superseded by #431


Handle tax by each message instead of by transaction

By Terra data structure, we should handle tax per msg, not per transaction.

@maruf0011 How about now?


Youfone integration interferes with other integrations

The Youfone integration interferes with other integrations, among which:

Aurora
Open Meteo
NextDNS
Electricity Maps

These integrations typically become unavailable within 2 hours when the Youfone integration is configured.
API calls to the remote server are malformed in some way (probably the POST data).
This behaviour started when I updated the integration from the previous to the current version on oct 8.
See this bug at HA Core for the full report:
https://github.com/home-assistant/core/issues/128228
No error messages from the Youfone integration.

Hi, can you test with this beta: https://github.com/geertmeersman/youfone/releases/tag/v0.12.5-beta.2

@geertmeersman Is still happening on 0.12.5-beta.2. Running the integration now with debugging enabled.

Yes, the other integrations systematically break after Youfone data is refreshed.
The integrations you mention use aiohttp, so it's quite strange they interfere.
I'll try to debug it on my side and suppose you'll need to revert to a working youfone version :-)
I changed it because the certification checks were blocking I/O calls

Can you try with this one: https://github.com/geertmeersman/youfone/releases/tag/v0.12.5-beta.5

Yes seems fixed now.

Thank you very much Geert!


git-lfs support

Currently user needs to have git-lfs installed locally.

@kkumar23 The change looks good. But keep in mind that we won't be able to upstream it to SymbiFlow as we cannot use git-lfs there.

@mkurc-ant Well in that case need to explore other options, meanwhile this will be a temporary fix.

@mkurc-ant : With this option it installs the git-lfs. But when we run any design it does not identify the git-lfs for the repo.
gzip: /home/kkumar23/DD_0906/symbiflow-arch-defs/build/quicklogic/qlf_k6n10/devices/gf12/qlf_k6n10-qlf_k6n10_gf12.rr_graph.bin.gz: not in gzip format

@kkumar23 From what I know one needs to manually do git lfs pull inside the qlfpga_symbiflow_plugins submodule directory after the submodule has been initialized.
I've found that supposedly when one clones the SymbiFlow repository for the first time one may do it using git clone --recurse-submodules. This should automatically do lfs pulls on submodules.


Not synced the deleted spent time.

I deleted a time entry on the web interface, but it still appears in the extension after syncing.

This is a little bit tricky (because the Redmine API) but will be fixed in the next release. Keep tight.


do not override the content type if set

Dio is overwriting the contentType header if one is set in the options. It should not overwrite the content-type if a header is explicitly set for that.
New Pull Request Checklist

[x] I have read the Documentation
[x] I have searched for a similar pull request in the project and found none
[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)
[x] I have added the required tests to prove the fix/feature I am adding
[x] I have updated the documentation (if necessary)
[x] I have run the tests and they pass

This merge request fixes / refers to the following issues: ...
Pull Request Description
...

This issue has been fixed on our fork of dio, dio_http.
Package is available here: https://pub.dev/packages/dio_http

awesome... thanks @jgoyvaerts!

This is not a proper fix in dio. Why this happens?
I have same issue. I see one more issue here: https://github.com/flutterchina/dio/issues/1257

This is not a proper fix in dio. Why this happens?
I have same issue. I see one more issue here: #1257

Unfortunately this package is no longer being maintained, we moved everything over to a fork (dio_http)[https://github.com/dart-tools/dio_http] . Can you try that package and see if it solves your issue?

@jgoyvaerts First time I hear about this. How is this not maintained, it's one of the most popular packages? I

Explained here, here and here

@jgoyvaerts OK. Thank you for you help. The dio_htttp works.


Updated OperatorEnum to use any data type (not just Numbers)

Changes:

Added a TypeInterface module. If you want to use an OperatorEnum on a specific data type, you have to implement all the functions from that module.
Added an on_type argument to @extend_operators, which restricts the type parameter of AbstractNodes   that have the operators defined on them.
Added an example with the Scalar-Vector-Matrix type as a  test

By the way, I wonder if using a 0-dimensional array instead of a scalar could simplify some code? I'm not sure if it would or not (see bottom example for a caveat), but you could try it if interested.
You can make 0-dimensional arrays with:
julia> zeros(Float64)
0-dimensional Array{Float64, 0}:
0.0

julia> fill(0.0)
0-dimensional Array{Float64, 0}:
0.0

julia> x = Array{Float64}(undef); x .= 0.0; x
0-dimensional Array{Float64, 0}:
0.0

To index a zero-dimensional array, you use []:
julia> x = zeros(Float64)
0-dimensional Array{Float64, 0}:
0.0

julia> x[] = 1.0
1.0

julia> x
0-dimensional Array{Float64, 0}:
1.0

Perhaps this simplifies some things like splatting because size(x) is an empty tuple in the case of 0-dimensional arrays.
However there are some issues with zero-dimensional arrays... For example, op.(x) does not return a 0-dimensional array, but a scalar:
julia> x = zeros(Float64)
0-dimensional Array{Float64, 0}:
0.0

julia> x .+ x
0.0

julia> x + x
0-dimensional Array{Float64, 0}:
0.0

which might make other parts of the code more complex...
So, completely up to you. Just another option to consider!

It seems like 0-dimensional arrays aren't used much so it might run into some weird behavior... Maybe just better to stick with your existing approach of scalars.

I think the line causing an error:
        @test_throws ErrorException ex(randn(1, 1))

can just be removed. This line just asserts that (up to this point) you couldn't use this type of behavior. But now you can! So that check can be turned off.

Also I wonder if SVM is the right name for it because of the overlap with https://en.wikipedia.org/wiki/Support_vector_machine. Maybe just "Tensor" is a better name for that struct. Or DynamicTensor maybe (emphasising that the dimensionality is "dynamic" and is stored as a runtime value rather than fixed to the type)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

145 of 166 (87.35%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.6%) to 94.904%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
25
26
96.15%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/OperatorEnumConstruction.jl
60
66
90.91%

src/TypeInterface.jl
10
19
52.63%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
2272

Relevant Lines:
2394

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

145 of 166 (87.35%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.6%) to 94.904%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
25
26
96.15%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/OperatorEnumConstruction.jl
60
66
90.91%

src/TypeInterface.jl
10
19
52.63%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
2272

Relevant Lines:
2394

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

145 of 159 (91.19%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 95.182%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
25
26
96.15%

src/Strings.jl
4
6
66.67%

src/TypeInterface.jl
10
12
83.33%

src/Evaluate.jl
26
29
89.66%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
2272

Relevant Lines:
2387

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

145 of 160 (90.63%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 95.142%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
25
26
96.15%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/TypeInterface.jl
10
13
76.92%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
2272

Relevant Lines:
2388

💛  - Coveralls

Thanks again! I'm going to do a few minor tweaks and then merge. In particular I think we may want to refer to "scalars" rather than "numbers" because e.g., "count_number_constants" could be read as "count number of constants" rather than "count the number of scalars". So I think we should rename them to be "count_scalar_constants" if that's okay with you? I'll make a commit now

Okay implemented some suggestions! I just thought we might as well rename some of the key functions. For example, count_constants is now count_constant_nodes, and count_number_constants is now count_scalar_constants. Let me know what you think.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

191 of 208 (91.83%)  changed or added relevant lines in 12 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.5%) to 95.061%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/TypeInterface.jl
10
13
76.92%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
2271

Relevant Lines:
2389

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

193 of 210 (91.9%)  changed or added relevant lines in 12 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.5%) to 95.065%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/TypeInterface.jl
10
13
76.92%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
2273

Relevant Lines:
2391

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

211 of 229 (92.14%)  changed or added relevant lines in 12 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.5%) to 95.062%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/ValueInterface.jl
28
32
87.5%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
2291

Relevant Lines:
2410

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

226 of 243 (93.0%)  changed or added relevant lines in 15 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 95.137%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/ValueInterface.jl
29
32
90.63%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
2289

Relevant Lines:
2406

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

226 of 243 (93.0%)  changed or added relevant lines in 15 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 95.137%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/ValueInterface.jl
29
32
90.63%

src/OperatorEnumConstruction.jl
60
66
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/OperatorEnumConstruction.jl
1
90.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
2289

Relevant Lines:
2406

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

224 of 239 (93.72%)  changed or added relevant lines in 15 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 95.371%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/ValueInterface.jl
29
32
90.63%

src/OperatorEnumConstruction.jl
58
62
93.55%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2287

Relevant Lines:
2398

💛  - Coveralls

There are some issues with defining OperatorEnum on BroadcastedFunction – seems you can't re-define an OperatorEnum on the non-broadcasted version. But we can fix that later and I'll just print a warning for now that this feature is unstable

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

229 of 243 (94.24%)  changed or added relevant lines in 15 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 95.417%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/NodeUtils.jl
28
29
96.55%

src/Expression.jl
10
12
83.33%

src/Strings.jl
4
6
66.67%

src/Evaluate.jl
26
29
89.66%

src/OperatorEnumConstruction.jl
63
66
95.45%

src/ValueInterface.jl
29
32
90.63%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2290

Relevant Lines:
2400

💛  - Coveralls