Where's the test suite being assembled?

...but a test suite is now being assembled to vet them. - http://jsonapi.org/implementations/

https://github.com/endpoints/endpoints/pull/125

Please feel free to open issues over @ endpoints for this stuff!

There's tests here as well.

@tkellen @0x8890 Will https://github.com/endpoints/endpoints be the canonical local for tests?
That should be documented as part of json-api. Having a model for interop testing is as important – arguably more important – than the spec.

We're currently developing the integration suite against endpoints using https://github.com/json-api/json-api/blob/gh-pages/normative-statements.json. Once this is complete, we'll move it out to json-api and figure out how we can generalize it far enough to be pointed at any API. One hitch is that implementors will need to use http://github.com/endpoints/fantasy-database for their API. We don't support hypermedia quite well enough for the integration suite to be entirely generic.

@tkellen Great news.
I don't mind having to run tests with nodejs against my own impl.

@tkellen Could we make this bug – or another – a tracking issue for tests? Call it an "enhancement request".

It would not be too difficult to generalize testing, the implementation would just need to be aware of what types will be used for testing.
This is how I implemented tests:

Create all resources by posting to the API.
Run test.
Delete all resources. Repeat.

Note: not all tests need to run/pass, depending on the implementation.


Expose writing Stream on MediaFileInfo

Would it be possible to add Stream OpenWrite() method to the MediaFileInfo class in the same fashion as the Stream OpenRead() is? That would allow us e.g. to use CopyToAsync() for writing to the MTP device.
I am working on abstraction in my project which uses pure Streams for both download/upload transfers across multiple types of devices, so currently I will need to find some workaround, since only the void UploadFile(Stream stream, string path) is available at the moment (as far as I searched the library code).

MediaDevices is a wrapper for the Windows MTP implementation. This implementation is based on COM and is not thread safe. It is therefore not advisable to call the methods from different threads. It may work in individual cases but can lead to strange effects that are then difficult to localize. Therefore, unfortunately, do not use async here.


DOT Faucet on site doesn't transfer tokens

Having followed the getting started guide and funded my BTC testnet wallet I ran into this issue on https://beta.polkabtc.io/
Funding failed. You can only use the faucet once every 6 hours.
SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

I created a new account for PolkaBTC and selected it from the dropdown in the top right.
After clicking the DOT faucet link it waits a while and then shows this message.
I have gone through all available accounts and none show any available DOT tokens on the site.

Thanks for the bug report. Confirmed and we are investigating this right now.

Same problem here, and also meet "Could not connect to the Parachain" from both Chrome and Brave

Same issue. Had to use the faucet on discord

We had some load issues which I think are now mostly resolved. @antonva do you happen to have the response for the JSON parsing issue?

@gregdhill I didn't think to jot it down, I don't have it.

No worries, I'll close this issue for now. Feel free to re-open if you encounter the issue again.


Use environment variable for GNU Tools path

This PR changes the c/c++ Visual Studio Code extension's hardcoded paths for the GNU Tools Arm Embedded directory to an environment variable named ARM_GCC_ROOT.

Can you add some information to the developers guide for Windows users explaining how to setup their environment variable for this?

Added a step in the configuring-vscode doc instructing the user to create the environment variable.  Doesn't really belong in the developer's guide root doc as it only pertains to vscode and isn't Windows specific.

This has been working really well for me on Windows.  Recommend accepting.

Thanks!


AnonymousType

It seems like that XmlTyp(AnonymousType = true) is not supported.
        public class ClassA
        {
            public string ElementA { get; set; } = "TextA";
            public ClassB B { get; set; } = new ClassB();
        }

        [XmlType(AnonymousType = true)]
        public class ClassB
        {
            public string ElementB { get; set; } = "TextB";
        }

I want to prevent <B><ElementB>TextB</ElementB></B> and just get <ElementB>TextB</ElementB> inside ClassA. Is there another way to achieve this? I have to say, I don't like the standard Xml Serializer solution, because if I need a class C which should have Class B fully inside and not just his members, then I have to create a duplicate class for B but without the "AnonymousType" tag, that's why I would favour a member tag like XmlElement to be more flexible.

Ah... well, unfortunately, we do not offer anything out of the box.
Now, if there is only one property you need this for per class, then you could create an extension that could make a content property ala Xaml's ContentPropertyAttribute.
This is of course something that I have been meaning to do because it makes a good extension story.  Unfortunately it's not in there, but we'll strive to make it for v3 (I really hate punting work, sorry!).
Of course, I am of the opinion that having property names is a nice thing to have in your serialized object graph. 😇

Well, you do not need to apologize :), I thought about making my own Xml Serializer after I created my own XSD code generator, but I guess it would be better to help you with your framework. I only have 2 rare cases where I would need this option. In the UML model, I have an aggregation and in the XSD file it's just a group reference and I thought it would be nice to hide the XSD definition from the user and just give him the UML representation (give him the group reference as the underlying class, but resolve it's members like if they belong to the root class (ClassA) ;)

Another question, are XML types supported like:
[XmlText(DataType = "unsignedInt")]
public uint Value { get; set; }

Ah cool... sure if you manage to build an extension we will gladly accept and review any PRs you have. :)  Do note that v2 (current version) is pretty much in release mode.  We do have a v3 planned but it's pretty much a rewrite.
As for XML types... we do have our own system and derive it from CLR types, or the types defined in the member, if that makes sense.

Do you have an example for the XML types?
Because  [XmlText(DataType = "dateTime")] public DateTime Value { get; set; } = DateTime.Now; and         [XmlText(DataType = "date")] public DateTime Value { get; set; } = DateTime.Now; is not working for me. I would expect different output streams.

Ah, so what I was saying is that we have our own type system in this regard.  We always emit by default the value of a DateTime instance based on its registered converter, which is the result of XmlConvert.ToDateTime.  You can see this here.
So, by default, all DateTimes and DateTimeOffsets look the same, if that makes sense.
If you want, this would be a good place to use a custom serializer on a member to yield the result you are looking to achieve. :)  Actually, it would be an extension that would work in conjunction with custom member serializers.
I should point out that while we do not support every legacy/classic feature, the scope and intent of ExtendedXmlSerializer is to serialize/deserialize object graphs, rather than interop with other external data sources (which was classic XmlSerializer's primary concern, IMO).  Formatting text such as this is usually rooted in having to account for an external schema of some sort.  In this case, I would recommend using XSLT to get the data prepared correctly.

What's about [XmlText]? Seems like I have to stick with the standard XmlSerializer? :(

I guess I am not too familiar with XmlText.  Are you looking to format your DateTime in a specific way?  This can be done with extensions but there is nothing built yet for it, I'm afraid.

Ah alright... so it seems that this is indeed a mechanism to pull in content from an element and assign it to a property on a class.  Much like ContentPropertyAttribute as denoted earlier.  I am assuming the dateTime formats from the DataType property are used to properly read it in.
So what this is IMO is a ContentProperty, which could be read in via metadata (classic XmlTextAttribute) or via the ConfigurationContainer.
Maybe something like new ConfigurationContainer().Type<Subject>().Content(x => x.Property) and/or new ConfigurationContainer().Type<Subject>().Member(x => x.Property).AsContent().
Is there anything specific/magical about the XmlText that is missing here?  I think everything else is covered by EXS's type system.

Yeah, a content property and no it's not DateTime, most of the time it's just a string oder List. I have over 300 generated classes and about 70 of them use this kind of property. So using is no soultion for me and XmlText or XmlTextAttribute is not working, The property is treated like a regular element property ... hmm. YAX can handle this, but it has problems with collections of base classes (you don't have it). XmlSerializer can handle this all, but it's slow and you have set a lot of additional meta data informations when using inheritance and this is quite odd. Thanks for your help.

OK @justeff I ACCEPT YOUR CHALLENGE!!! 😆  Well, more like I feel your pain.  I myself have been spending the past week going through and trying to get all my identities aligned between GOOG and MSFT... and there is not a single chat solution that I am happy with between the two.  This is especially perplexing as GOOG has like 5 different chat applications and each one of them does something good, but none of them do it exactly right.
So I did some research on this and I cannot find an example of XmlText used as a string list.  Do you have a quick example you can share?  I am mostly interested in the xml output.

Just realized, list string makes no sense for me. The XML is standardized, if I get it from an external source it will be just one big string haha. I need only a content property then :)

OK COOL... let's see what I can do. 👍

Alright... I checked this out and I think I have a reasonable path forward here.  I've got some tasks that I have to wrap up for today, but I am going to aim to have this available for you by EOD tomorrow if that works for you.  Assuming there aren't any super-gotchas that impede the path, that is. 😇

Thank you Mike, you don't need to hurry up, I'll use XmlSerializer to get my stuff done and maybe I can switch to this framework later on and benefit from peformance gains :) Have a good day.

OK cool... maybe I will push it to the weekend then. 😇  I'm getting ready to switch out my hyper-v server to a Windows 10 install and that is going to take a little time.  In either case, I will certainly keep this thread updated.

Holy moly! Have I been in a rabbit hole. I am just now getting my environment back online and being able to attend to this. I hope to have this resolved today or tomorrow. We'll see how I do. 😛

Whew... brutal.  Alright.  this sums up how I feel about taking this (self-imposed) challenge:

😛
I got a rough draft in there for you to check out.  I'm not exactly happy with it, however.  It's a poor attempt at mimicking the classic serializer and it is fragile at best.  Unit tests pass but I doubt it will hold up to an actual real world scenario.
The good news is that it is all tucked away in an extension that has all of the components within it.  That's sort of the nice thing with this extension model: you can add something to the system that is suspect at best but it will only wreck your world if you ask for it. 😆
The other nice part about that is that it makes it easier for someone to dive in and perhaps fix future issues.  I say that in the hopes that I never have to look at this file again.  (kidding... sorta).
Anyways, please give this a try whenever you get the chance and let me know how well (or not) it works for you. 👍
Preview feed is here:
https://www.myget.org/F/wojtpl2/api/v3/index.json


[ClosureSpecializer] Don't release trivial noescape try_apply argument twice.

When a specialization is created, in the original function, releases are added in two different places:
(1) ClosureSpecCloner::populateCloned
(2) rewriteApplyInst
In the former, releases are added for closures which are guaranteed or trivial noescape (but with owned convention). In the latter, releases are added for closures that are owned.
Previously, when emitting releases at (2), whether the closure was trivial noescape wasn't considered.  The result was inserting the releases twice, an overrelease.
Here, fix (2) to recognize trivial noescape as not +1.
rdar://110058964

@swift-ci please test


Question: are volumes passed through command line argument mounted when entrypoint is executed?

I was wondering if my ENTRYPOINT command had access to volumes mountes with the -v flag when it is executed. I would like my entrypoint script to have different behaviour depending on if a file is present or not.

Yes the volume should be available to all processes within the container.
BTW, see https://docs.docker.com/faq/#where-can-i-find-more-answers   the mailing lists, or irc are usually a better place to ask questions. Github issues are usually for calling out problems not for asking questions.


Write ETL code to transfer CSV files to database

We have a bunch of webscraped CSV files in a standardized format. Before we can make cool visualizations for ACLU, we need to put data into a database. The database will make it easier to query whatever data we need to make plots.
Here is the database specification, if you have ideas for improvement please comment.
This is a great Issue to practice SQL programming! If you don't know much SQL, you can learn as you go. Also a great Issue for text processing.
We will start with SQLite (despite early thoughts of using Postgres) so you can get started writing code without installing a bunch of software.
If you want to help, post below or hit me up on our Slack channel. We'll discuss specific tasks you can do, depending on what we already finished.

[ ] the ETL should convert everything to lowercase before storing to database so we avoid possible uppercase/lowercase bugs in string comparison.

Steps to make minimal database to make just the plot of "how long misdemeanors are in jail before they are released" - most important to get done by January.

Specification in ETL readme for 1-table database that has booking & release timestamps, plus identifying info to uniquely identify each booking (different for each county). For each county we have webscraped, figure out what the unique identifying columns are.
Write Python code to totally ignore CSV rows that are felonies and to be able to flag if it’s a verified misdemeanor or not. Database includes a column that says it’s verified misdemeanors or now (we know for sure that booking is only composed of misdemeanors)
Write SQLite code to make the empty minimal database – copy mostly from init_database.py
For each row in pandas dataframe, use SQL to figure out if already in database.
If not in database, add it
If in database, update release date
Write logic to figure out if inmates dropped off the roster one day when they were previously on the roster (for certain counties only like Glynn/Richmond).

I'm working on it this week if you want to contribute post here.


CAD-971 check required tracers in LiveView

improved user experience when using "LiveView"
because the "TracerConfig" cannot be altered after it has been parsed from the configuration file, the node's startup process is only checking whether the required tracers are turned on. The missing tracer configurations are printed to the console and the startup is waiting for the user to acknowledge before continuing. If all required tracers are turned on then no interruption happens.

bors r+


Fix Issue related to total amount not matching in invoice report

Changes
https://github.com/ColoredCow/portal/issues/2733

Self reviewed


Rewrite ResourceManager in C++

This introduces an OOM when changing course in TTs. Too bad!

Can you confirm said OOM was not already in main?

Can you confirm said OOM was not already in main?

Yes.

OOM fix just dropped.

Implemented.


pgmigrate: add warnings when large objects present

Proposed changes in this pull request
Large objects are not compatible with logical replication as of writing
(https://www.postgresql.org/docs/15/logical-replication-restrictions.html).
Add some warning output if we detect the presence of large objects, and
also update our docs to indicate this.
Type (put an x where ever applicable)

[ ] Bug fix: Link to the issue
[X] Feature (Non-breaking change)
[ ] Feature (Breaking change)
[X] Documentation Improvement
[ ] Other

Checklist
Please put an x against the checkboxes. Write a small comment explaining if its N/A (not applicable)

[X] All the tests are passing after the introduction of new changes.
[ ] Added tests respective to the part of code I have written.
[X] Added proper documentation where ever applicable (in code and README.md).

Optional extra information

Related to https://github.com/aiven/aiven-db-migrate/issues/34

@kmichel-aiven Sure. The newlines etc in README come from my auto-formatter for markdown files. I'll disable it and add the warning.

@kmichel-aiven changes requested have been made.


Update examples (blurhash)

It seems blurhash has been removed - but the first example in the documentation still showcases how to use it:
OctoImage(
  image: CachedNetworkImageProvider(
      'https://blurha.sh/assets/images/img1.jpg'),
  placeholderBuilder: OctoPlaceholder.blurHash(
    'LEHV6nWB2yk8pyo0adR*.7kCMdnj',
  ),
  errorBuilder: OctoError.icon(color: Colors.red),
  fit: BoxFit.cover,
);

Yet there is no OctoPlacholder.blurHash(...) method anymore. Please update the examples!

same .blurHash is not defined

I'm also using OctoImage with blurHash and it works great for ^1.0.2. It'd be great if you could document how to migrate to ^2.0.0 to overcome this.

For those interested, I fixed on my side by introducing this class :
class OctoBlurHashFix {

  static OctoPlaceholderBuilder placeHolderBlurHash(String hash, {BoxFit? fit}) {
    return (context) => SizedBox.expand(
      child: Image(
        image: BlurHashImage(hash),
        fit: fit ?? BoxFit.cover,
      ),
    );
  }

  static OctoErrorBuilder errorBlurHash(
      String hash, {
        BoxFit? fit,
        Text? message,
        IconData? icon,
        Color? iconColor,
        double? iconSize,
      }) {
    return OctoError.placeholderWithErrorIcon(
      placeHolderBlurHash(hash, fit: fit),
      message: message,
      icon: icon,
      iconColor: iconColor,
      iconSize: iconSize,
    );
  }
}

Then, replace your usages of OctoError.blurhash and OctoPlaceholder.blurhash :
OctoImage(
    colorBlendMode: BlendMode.modulate,
    fit: BoxFit.cover,
    image: CachedNetworkImageProvider(imageUrl),
    placeholderBuilder: OctoBlurHashFix.placeHolderBlurHash(placeHolderBlurHashString),
    errorBuilder: OctoBlurHashFix.errorBlurHash(errorBlurHashString, iconColor: Colors.transparent),
 ),


Navigation needs updating for Zola 0.16 & later

@aaranxu can you give me access to create a PR to fix use of "higher/lower" in navigation as per
https://github.com/getzola/zola/blob/53ce6db057101b8a24631ac4212ab4f9f3d145ae/CHANGELOG.md#0160-2022-07-16
Or fix in templates/macros/docs-navigation.html
Thanks.

You saved my day. Thanks!


Transfer of variables to provider fails

Hi,
if I try to use environment variables in oneview-provider, it throws serveral errors:
# export TF_LOG_CORE=DEBUG; export TF_LOG_CORE=TRACE; export TF_LOG_PROVIDER=TRACE; export TF_LOG_PATH=logs.txt; export TF_VAR_endpoint=my-url-to-oneview; export TF_VAR_username=my-user; export TF_VAR_password=my-pass; export TF_VAR_ssl_enabled=false; export TF_VAR_ov_domain=my-domain

My terraform file - inspired from the README.md
# Create terraform OneView client
provider "oneview" {
       ov_username  = "${var.username}"
       ov_password  = "${var.password}"
       ov_endpoint  = "${var.endpoint}"
       ov_sslverify = "${var.ssl_enabled}"
       ov_domain    = "${var.ov_domain}"
       ov_apiversion = 3200
       ov_ifmatch = "*"
}

data "oneview_server_hardware" "server_hardware" {
  name = "my-server-name"
}

output "oneview_server_hardware_value" {
  value = data.oneview_server_hardware.server_hardware.uri
}

Result:
# terraform init
# terraform plan

Error: Reference to undeclared input variable

  on test.tf line 3, in provider "oneview":
   3:   ov_username  = "${var.username}"

An input variable with the name "username" has not been declared. This
variable can be declared with a variable "username" {} block.

Error: Reference to undeclared input variable

  on test.tf line 4, in provider "oneview":
   4:   ov_password  = "${var.password}"

An input variable with the name "password" has not been declared. This
variable can be declared with a variable "password" {} block.

Error: Reference to undeclared input variable

  on test.tf line 5, in provider "oneview":
   5:   ov_endpoint  = "${var.endpoint}"

An input variable with the name "endpoint" has not been declared. This
variable can be declared with a variable "endpoint" {} block.

Error: Reference to undeclared input variable

  on test.tf line 6, in provider "oneview":
   6:   ov_sslverify = "${var.ssl_enabled}"

An input variable with the name "ssl_enabled" has not been declared. This
variable can be declared with a variable "ssl_enabled" {} block.

Error: Reference to undeclared input variable

  on test.tf line 7, in provider "oneview":
   7:   ov_domain    = "${var.ov_domain}"

An input variable with the name "ov_domain" has not been declared. This
variable can be declared with a variable "ov_domain" {} block.

Any hint or ideas ?
Regards

Solved by adding declarations like this to .tf file:
variable "password" {
   type = string
}


Fix definition of historical user IDs

Spaces are allowed by Synapse and a space is a printable ASCII character.
Signed-off-by: Tulir Asokan<EMAIL_ADDRESS>P.S.: I might have created a user ID with spaces and joined some rooms with it a couple of months back. Conduit doesn't like such rooms 😿

Spaces are allowed by Synapse and a space is a printable ASCII character.

Citation needed.

Citation from #urandom:maunium.net
{
  "auth_events": [
    "$1rZlyzCq6xgG6h39hPjF41MozLNKpnWZc-nr9AZCydc",
    "$noQOOQLwKmVYY5VhKSU_AQuMsiFjeRZIT-pAY10b0Zk",
    "$IA3XkLB8FN1NonkIuw5m9ug10fGGcMPI_0MwE3bSZmw"
  ],
  "prev_events": [
    "$aX35Lvo5Pmigd3NJrTjxqsRHCb7J5TUfuZ1hUNC8yLs"
  ],
  "type": "m.room.member",
  "room_id": "!WgEsWTOEGkIcwYphxi:maunium.net",
  "sender": "@s p a c e:maunium.net",
  "content": {
    "membership": "join",
    "displayname": "S p a c e",
    "avatar_url": "mxc://maunium.net/GLgMPTLCbZlhbXHoHDebzCeo"
  },
  "depth": 9715,
  "prev_state": [],
  "state_key": "@s p a c e:maunium.net",
  "origin": "maunium.net",
  "origin_server_ts":<PHONE_NUMBER>853,
  "hashes": {
    "sha256": "TJ6Lxs2PvfXCyxgtUFjGOEKpJT9OGKAXJU6h3HEnoJw"
  },
  "signatures": {
    "maunium.net": {
      "ed25519:a_xxeS": "W34Z8w2N1Cky06XkK2u/kk48zKd24sw/NJK19UzIOx2nMTSKoozNuKHC+KeAQ+MhVZIhfPBJ1YNFI8rSPEtGDQ"
    }
  },
  "unsigned": {
    "age_ts":<PHONE_NUMBER>853
  }
}

I can join other rooms with that account too if you want :D

Needs a code citation, please.

extended_user_id_char = %x20-39 / %x3B-7E  ; all ascii printing chars except :

I'm not sure that synapse even enforces these limits :(.

(please stop breaking rooms by turning off what little validation synapse has)

I'm not sure that synapse even enforces these limits for events received over federation :(.

That sounds like the spec needs to be fixed to allow any user IDs and a new MSC needs to be made to restrict non-printable-ascii user IDs after room v7

#2828


chore: Remove unneeded condition

Details
When #1874 rearranged the validation tests, an unnecessary condition was left in place. it's unnecessary because the run should fail if either of the 2 previous test steps fail. Leaving the condition in place doesn't impact the overall result (thanks to the validation job), but it's both slower and inconsistent with the comment on lines 50 and 51.
Validation run is at https://dev.azure.com/accessibility-insights-private/Accessibility Insights (private)/_build/results?buildId=48628&view=results
Motivation
Avoid maintaining unnecessary stuff
Context

Pull request checklist

[n/a] Addresses an existing issue: Fixes
[n/a] Added relevant unit test for your changes. (yarn test)
[ ] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[ ] Ran precheckin (yarn precheckin)

Codecov Report

Merging #1900 (67d4241) into main (220d480) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 67d4241 differs from pull request most recent head ba1113c. Consider uploading reports for the commit ba1113c to get more accurate results

@@           Coverage Diff           @@
##             main    #1900   +/-   ##
=======================================
  Coverage   94.52%   94.52%
=======================================
  Files          38       38
  Lines        1059     1059
  Branches      141      141
=======================================
  Hits         1001     1001
  Misses         58       58


initial_size<1 or final_size<1 should be an error

We currently enforce that Ne>0, but it should be Ne>=1. Unless there is some circumstance under which 0 < Ne < 1 makes sense?

What about people things like working in coalescent units? Fractional Ne values < 1 might make sense then? I don't think there's much to be gained from putting an upper limit on it.

Happy to leave it as it is, I just noticed that hypothesis was often (mostly?) generating graphs that won't be useful. Eg. parameters at the edge of their range, like ~1e±308.

That's fine I think. No point in being too prescriptive unless we're preventing errors that we know will happen (rather than preventing stuff we think probably doesn't make sense)

Ok, closing. :)


CUDA error: out of memory

Hi thanks for sharing your work.
I'm trying to use your network with cityscapes dataset but i get a runtime cuda error during test forwarding (out of memory).
I'm using a gtx 1060 6gb and intel i5 7600. What do you think i can change to make it work properly? Which parameters should i change?
thanks for your help!

Try to lower the batch size using -b or --batch_size, or lowering the image resolution using  --height and --width


Visualizing fine tuning.

Hi, can someone please explain after finetuning process how to plot the training curves ?

@mindFlayer123 , did you find a solution for this already?


balena-yocto-scripts: Update balena-yocto-scripts v1.4.0

Changelog-entry: Update balena-yocto-scripts to v1.4.0
Signed-off-by: Florin Sarbu<EMAIL_ADDRESS>

The board is discontinued


[BUG] User not added to group

General information

OS: Windows
Hypervisor: Hyper-V
Did you run crc setup before starting it (Yes/No)? no
Running CRC on: Laptop

CRC version
# Put `crc version` output here

CodeReady Containers version: 1.30.1+376cc3c1
OpenShift version: 4.8.2 (not embedded in executable)
CRC status
# Put `crc status --log-level debug` output here

DEBU CodeReady Containers version: 1.30.1+376cc3c1
DEBU OpenShift version: 4.8.2 (not embedded in executable)
DEBU Running 'crc setup'
INFO Checking if crc-admin-helper executable is cached
INFO Checking for obsolete admin-helper executable
DEBU Checking if an older admin-helper executable is installed
DEBU No older admin-helper executable found
INFO Checking if running on a supported CPU architecture
INFO Checking minimum RAM requirements
DEBU Total memory of system is<PHONE_NUMBER>4 bytes
INFO Checking if running in a shell with administrator rights
DEBU Running '$currentPrincipal = New-Object Security.Principal.WindowsPrincipal([Security.Principal.WindowsIdentity]::GetCurrent());$currentPrincipal.IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator)'
INFO Checking Windows 10 release
DEBU Running '(Get-ItemProperty -Path "HKLM:\SOFTWARE\Microsoft\Windows NT\CurrentVersion" -Name ReleaseId).ReleaseId'
INFO Checking Windows edition
DEBU Running '(Get-ItemProperty -Path "HKLM:\SOFTWARE\Microsoft\Windows NT\CurrentVersion").EditionID'
DEBU Running on Windows Enterprise edition
INFO Checking if Hyper-V is installed and operational
DEBU Running '@(Get-Wmiobject Win32_ComputerSystem).HypervisorPresent'
DEBU Running '@(Get-Service vmms).Status'
INFO Checking if crc-users group exists
DEBU Running 'Get-LocalGroup -Name crc-users'
INFO Checking if current user is in Hyper-V group and crc-users group
DEBU Running 'Get-LocalGroupMember -Name crc-users -Member '0123456789''
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: Get-LocalGroupMember : Principal<PHONE_NUMBER> was not found.
At line:1 char:43

... lyContinue'; Get-LocalGroupMember -Name crc-users -Member '0123456789'

             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : ObjectNotFound:<PHONE_NUMBER>:String) [Get-LocalGroupMember], PrincipalNotFoundException
FullyQualifiedErrorId : PrincipalNotFound,Microsoft.PowerShell.Commands.GetLocalGroupMemberCommand

DEBU exit status 1
INFO Adding current user to group
DEBU Running 'Get-LocalGroupMember -Name crc-users -Member '0123456789''
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: Get-LocalGroupMember : Principal<PHONE_NUMBER> was not found.
At line:1 char:43

... lyContinue'; Get-LocalGroupMember -Name crc-users -Member '0123456789'

             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : ObjectNotFound:<PHONE_NUMBER>:String) [Get-LocalGroupMember], PrincipalNotFoundException
FullyQualifiedErrorId : PrincipalNotFound,Microsoft.PowerShell.Commands.GetLocalGroupMemberCommand

INFO Will run as admin: adding current user to crc-users group
DEBU Running 'C:\Users\XXXXXX~1\AppData\Local\Temp\crcScripts190497199\runAsAdmin.ps1'
DEBU Running '$sid = New-Object System.Security.Principal.SecurityIdentifier("S-1-5-32-578")
@([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole($sid)'
Please reboot your system and run 'crc setup' to complete the setup process
OS version
# Put `systeminfo` output here

OS Name:                   Microsoft Windows 10 Enterprise
OS Version:                10.0.19042 N/A Build 19042
Steps to reproduce

run Windows MSI installer, fix issue #2620 with workaround #2581 -> 2620 should NOT have been closed because the problem did not go away, the workaround may have caused this here or other problems!
run crc setup
reboot as instructed
loop - crc setup never completes because I am not logged in as a local user<PHONE_NUMBER>) but as a domain/cloud user (something\0123456789) - crc just does not get this, cannot find a non-existing local user, and keeps messing with local groups that it did not need before ...

Expected
completion of crc setup without any issues
Actual
loop, see above
Logs
see above

funny enough, crc setup found the full user (something\0123456789) the first time I ran it and added it to the crc-users local group that it had created (net localgroup crc-users lists it correctly). But crc setup cannot find that user again at any subsequent run. Is it looking for something different than before? A simple whoami would get it the right user.

@mraufer What is now the output of crc setup --check-only ?

output below as requested, plus user listing of relevant groups and local users:
crc setup --check-only
INFO Checking if crc-admin-helper executable is cached
INFO Checking for obsolete admin-helper executable
INFO Checking if running on a supported CPU architecture
INFO Checking minimum RAM requirements
INFO Checking if running in a shell with administrator rights
INFO Checking Windows 10 release
INFO Checking Windows edition
INFO Checking if Hyper-V is installed and operational
INFO Checking if crc-users group exists
INFO Checking if current user is in Hyper-V group and crc-users group
exit status 1
whoami
something\0123456789
net localgroup "hyper-v administrators"
Alias name     hyper-v administrators
Comment        .
Members
Administrator
something\0123456789
The command completed successfully.
net localgroup crc-users
Alias name     crc-users
Comment        Group for CodeReady Containers users
Members
something\0123456789
The command completed successfully.
net user
User accounts for \HOSTNAME
Administrator            DefaultAccount           defaultuser100000
Guest                     WDAGUtilityAccount
The command completed successfully.

Duplicate of #2620 #2581

This is does not look like a duplicate of #2581 to me.  #2581 is a locale issue.
#2446 provides a solution: reset your local USERNAME environment variable to include the domain prefix:
C:...> set USERNAME=something\0123456789
C:...> crc setup --log-level debug

@jackhu008 this should be fixed in newer CRC releases, if you still face this please open a new issue.


Increase regex version constraint compatibility

What type of PR is this?

/kind dependency-change
What this PR does / why we need it:
The current version constraint causes issues, specifying a dependency of 1 indicates compatibility with any other 1.*.* version of the regex crate, which I think is the case here.
This is related to discussion in https://github.com/containers/oci-spec-rs/issues/190#issuecomment-2383800512
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
Thanks!
Does this PR introduce a user-facing change?

None

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 29.02%. Comparing base (57570dd) to head (62fb917).
Report is 53 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
+ Coverage   26.61%   29.02%   +2.41%
==========================================
  Files          24       26       +2
  Lines        2788     3159     +371
  Branches     1516     1671     +155
==========================================
+ Hits          742      917     +175
- Misses        948     1021      +73
- Partials     1098     1221     +123

@kpcyrd do you mind a rebase?

Done!

Could you please release a new version? :) the latest version on crates.io still breaks cargo outdated -R for my project.

Could you please release a new version? :) the latest version on crates.io still breaks cargo outdated -R for my project.

Yeah: https://github.com/youki-dev/oci-spec-rs/pull/253


Fix for GSLB monitor not taking effect.

The "monitor" argument was being passed as a bunch of arrays due to the .sort call. The Dynect API was silently failing to set the monitor because of this.
The outcome of this PR is that instead of this being sent to the Dynect API,
[[:expected, ""], [:header, ""], [:host, "example.com"], [:interval, 1], [:path, "/up"], [:port, 80], [:protocol, "HTTP"], [:retries, 2]]

we will now send this,
"{\"protocol\":\"HTTP\",\"interval\":1,\"retries\":2,\"port\":80,\"path\":\"/health\",\"host\":\"example.com\",\"header\":\"\",\"expected\":\"\"}"

and Dynect will correctly accept and apply the new monitor settings.

thank you


keep args in the scope

It's common to return args and then access attributes as needed in subsequent lines (necessary as soon as more than 1 argument is passed via the cli)
https://github.com/core-unit-bioinformatics/devtools/blob/4302a211efca8e387a338d80581a9dd139b2dcee/update_template.py#L7

updated

I'll mark this as resolved. Assigning the members of args. to identically named variables in a scope of ~10 lines with a single access to the respective member seems a bit ... over the top ;-)


Remove padding

Hello!
Is there a way to remove excess padding from the HGCircularSlider view?

Hi @Natelegreat1, you can fix that by setting backtrackLineWidthequal to lineWidth

@HamzaGhazouani Good thinking but alas I do not mean the fact that the dark blue is thicker than the white part -- that, I want! :)
What I would like to get rid of is where I've drawn red lines. I want the HGCircularSlider UIView to siteToFit() to its containing view. Does that make more sense?

Hi @Natelegreat1, we have this padding because the thumbs (in the circular slider) should be inside the frame. Below, how we calculate the radius of the circular slider
/**
* The radius of circle
*/
    internal var radius: CGFloat {
        get {
            // the minimum between the height/2 and the width/2
            var radius =  min(bounds.center.x, bounds.center.y)
            // all elements should be inside the view rect, for that we should subtract the highest value between the radius of thumb and the line width
            radius -= max(lineWidth, (thumbRadius + thumbLineWidth))
            return radius
        }
    }

in your case, we use it like a progress view, so we don't need the thumbs
If you put thumbRadius = 0 & thumbLineWidth = 0 the padding will disappear.
Thanks

Amazing. That's exactly what I was looking for!


[feat] Generate Typescript Declaration from Structs in Rust

Describe the problem
Typically, when you create a struct on the rust side, you need to declare the same struct on the typescript side using a .d.ts file or having a dedicated interface file. This normally isn't bad if not a little bit time consuming, though it does open up the floodgates for human error which can slow progress down significantly on larger codebases.
Describe the solution you'd like
There is a rust crate that can do this, however, which is simply a derive macro in typescript-definitions.
my knowledge of rust (and this code base) is very limited however it may be a good idea to have #[tauri::struct] macro in the same way we have #[tauri::command], which would release typescript definition files should a typescript mode be enabled in the projects tauri.conf.json.
The developer could then simply attach this #[tauri::struct] to any struct that are supposed to be accessed by the front end (and while its there, it may be a good idea to make it derive serde::Serialize if you can, idk).
Alternatives considered
Realistically this should only be a minor annoyance (that being, creating the .d.ts files yourself) until the codebases start scaling massively although it would be a nifty little feature, so if you absolute gods consider something like this it probably wouldn't be the highest priority at the moment all things considered.
Additional context
Overall this is purely an idea to improve the developer experience and overall, nothing will happen if this doesn't.

Yeah i think for this specific use-case it makes more sense to promote existing solutions (maybe integrate them in the docs or an example).

Closing as duplicate of #3586.


client root info could not update when epoch=0

Now, our client inits root info with Arc::default(), so initial epoch value is 0.
So root_client couldn't refresh the initial root info from root_server because root_server's epoch is also 0.
This will cause 2nd node join the cluster is very very slow (because root_client still uses fake & empty default RootDesc until RootServer epoch changes for some reason.

So root_client couldn't refresh the initial root info from root_server because root_server's epoch is also 0.

It seems that the root cause is root server's epoch is 0, but the root server epoch cannot be 0 (by designed).


Add character: <3> to font:

Character : <3>
Font :

If no one is working on this issue you can assign this to me @AnshumanMahato


Question about rotation

i have a question about rotate points along Z axis, in OpenPCDet code, rotation matrix is following, i wonder if sina and -sina should change its location?

but i use some examples to check, the OpenPCDet code is correct, anyone could tell me why?

They use points@R instead of R@points, resulting in the matrix being transposed.

They use points@R instead of R@points, resulting in the matrix being transposed.

oh, yes. I got it now. Thanks, ziyi.


update examples/neural_doodle.py based on issues #3731

Re-implement the style loss function for better normalization of features from different CNN layers.
The result is better doodling qualities for some images.
See #3731 for details.

LGTM.


Update Atoms to 2.6.0

Expected Behavior
Project uses Atoms 2.6.0
Actual Behavior
It uses Atoms 2.4.0
Mention any other details that might be useful

Steps to reproduce the behavior

The log given by the failure.

closed via #96


fix(resize): correct check for overflow in wrapper

Was incorrectly checking for overflow using getBoundingClientRect(), which wasn't consistently returning the width of the child element if there was actually overflow in the parent.

Coverage increased (+0.1%) to 93.913% when pulling 3bb27a20932c458d29d17f77b50ce331ce6c9670 on overflow-fix into 24852b2b77126c52a94fefb33d71aa87a8b31803 on master.


TNT-40238 - Refactor subdomain and TLD extraction in Node.js SDK

Description
Adding support for complex domain parsing by utilizing the psl module. In order to keep at.js lightweight, this module is only installed on target-tools and the new function is imported into the Decisioning engine by the DecisioningConfig. The old lightweight, module-less, domain parsing will still occur by default if the new parsing function is not passed into the DecisioningConfig. Thus, at.js will be unaffected.
Related Issue
https://jira.corp.adobe.com/browse/TNT-40238
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Coverage increased (+0.09%) to 92.496% when pulling 1b41a89a7337c9831a7b7bfe2597856e05a99617 on ericfichtel:TNT-40238 into c1c1709637d636662a8e2f6cad77da40e93b0d70 on adobe:main.

looks good, feel free to merge when you are ready.  friendly reminder to "squash and merge".


Sort out any remaining issues with the chainlink integration

Problem
There is on going discussion around fees and an open bug #2583 .  What else needs to be done to complete this integration?
Solution

List all open issues and resolve them.

Alternatives
No response

Resolved.


Search Results link to old webguide subdirectory, not custom domain.

The search results appear to link to the old subdirectory for the webguide and not the new webguide.ucsb.edu domain. This bug should be addressed.

Hey, @loganfranken , does the site.baseurl and the Google Custom Search at https://github.com/ucsb-wsg/ucsb-wsg.github.io/blob/master/scripts/main.js#L92 need to be updated for the new webguide.ucsb.edu url?

Hey @tenken! Good catch!
Yeah, as far as I remember, Mihoko set up the custom search engine originally. She may be able to change it?
If not, we could always create a new one.

Ah yes, I recall now the old webguide was integrated with Google Custom Search (GCS) that powers also the www.ucsb.edu search. Eg, searching the campus will return webguide results from the legacy webguide URL. I also recall IA saying they could include a new subdomain such as webguide.ucsb.edu if it gets made and we notify them. I'll see if they can update the GCS to include the new domain.
Baring, that possibility I'm sure we could roll our own.

Email request sent to IA.

All those slashes in the directory names for site content ... are probably not a good thing ...
http://webguide.ucsb.edu/sitemap.xml

@loganfranken can you update the google custom search used on the site to use the new subdomain?

Hey @tenken!
Sorry, I'm confused: as far as I know, I don't administer the Google Custom Search used on the Web Guide so I can't update it. Did you hear otherwise?
If we want to just move forward with creating a new Google Custom Search catered to the new subdomain, I can totally do that.

Augh, sorry, you were totally right: I do administer this one.
Updated!
Looks like results are now pointing to the new subdomain, so I think we can close this whenever.
(Sorry again for the confusion)

Thanks for the effort! No worries :)


Ionic Package building android with --prod does not change file size or minify javascript

From @kamok on March 16, 2017 16:46
Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Two days ago, I built an .apk with this exact command, as copied from my ~/.bash_history: ionic package build android --prod --profile dev_profile --release. The .apk was 6.6 MB big, took 20 seconds, and was booting up in 6 seconds. This is good.
However, today, using the same exact command, using the same exact commit on Git (did a git checkout to the successful build). The .apk was 8.8 MB, took 20 seconds, and boot time at 16 seconds. This is bad.
I tried debugging this to see if somehow my --prod flag wasn't working. So I ran another build like so, without the flag: ionic package build android --profile dev_profile --release. This yielded a build time of 40 seconds (expected), and the file was, once again 8.8 MB with a slow boot time.
So I can conclude that --prod is being noted by the change in build time, but for some reason, AoT compiling and minifaction is not happening. So no change in boot time, no change in file size.
Only difference I can think of is the /platform directory which is gitignored. And the /node_modules, which I did a fresh npm install. No avail.
Had a co-worker do the same thing with the same repo + same command, adding --prod does not seem to change file size at all and other than taking 20 seconds faster to build there's no difference.
Expected behavior:
It should work like it did 2 days ago. I used the exact file, exact command.
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:

Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.1.4
Ionic App Lib Version: 2.1.2
Ionic App Scripts Version: 1.1.4
ios-deploy version: 1.9.0
ios-sim version: Not installed
OS: Mac OS X El Capitan
Node Version: v6.9.1
Xcode version: Xcode 8.2.1 Build version 8C1002

Installed platforms:
  android 6.1.2
  ios 4.3.0

Copied from original issue: driftyco/ionic#10808

From @kamok on March 16, 2017 20:3
In my config.xml, I use  <engine name="android" spec="latest"/>. I'm guessing this is COULD be the problem. However, having trouble finding out the version history changelog from 3/14 - 3/16 (today). Edit:  They did not make any changes. As seen on https://cordova.apache.org/blog/

Here's the logs on local. It seems like the issue with --prod is not just on Ionic Package. `ionic build android --prod --release" locally also uses dev mode:
$ ionic build android --prod --release -- --keystore="election-android.keystore" --storePassword=*****  --password=***** --alias="election-mobile"

><EMAIL_ADDRESS>ionic:build /Users/kamok/code/election/election-mobile
> ionic-app-scripts build

[17:18:39]  ionic-app-scripts 1.1.4
[17:18:39]  build dev started ...
[17:18:39]  clean started ...
[17:18:39]  clean finished in 4 ms
[17:18:39]  copy started ...
[17:18:39]  transpile started ...
[17:18:43]  transpile finished in 3.67 s
[17:18:43]  preprocess started ...
[17:18:43]  preprocess finished in less than 1 ms
[17:18:43]  webpack started ...
[17:18:43]  copy finished in 3.84 s
[17:18:55]  webpack finished in 11.94 s
[17:18:55]  sass started ...
[17:18:55]  sass finished in 834 ms
[17:18:55]  postprocess started ...
[17:18:55]  postprocess finished in 1 ms
[17:18:55]  lint started ...
[17:18:55]  build dev finished in 16.47 s
[17:18:59]  lint finished in 4.00 s
Running command: /Users/kamok/code/election/election-mobile/hooks/after_prepare/010_add_platform_class.js /Users/kamok/code/election/election-mobile

add to body class: platform-android

Running command: /Users/kamok/code/election/election-mobile/hooks/after_prepare/delete_test_assets.js /Users/kamok/code/election/election-mobile

Test build @ platforms/android/assets/www/build/test does not exist for removal

ANDROID_HOME=/Users/kamok/Library/Android/sdk

JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_111.jdk/Contents/Home

Subproject Path: CordovaLib

Incremental java compilation is an incubating feature.

:preBuild
 UP-TO-DATE

:preReleaseBuild UP-TO-DATE
:checkReleaseManifest

:CordovaLib:preBuild UP-TO-DATE

:CordovaLib:preReleaseBuild UP-TO-DATE
:CordovaLib:checkReleaseManifest

:CordovaLib:prepareReleaseDependencies

:CordovaLib:compileReleaseAidl

UP-TO-DATE
:CordovaLib:compileReleaseNdk UP-TO-DATE

:CordovaLib:compileLint

UP-TO-DATE

:CordovaLib:copyReleaseLint

UP-TO-DATE

:CordovaLib:mergeReleaseShaders

UP-TO-DATE
:CordovaLib:compileReleaseShaders
 UP-TO-DATE
:CordovaLib:generateReleaseAssets
 UP-TO-DATE

:CordovaLib:mergeReleaseAssets

UP-TO-DATE
:CordovaLib:mergeReleaseProguardFiles UP-TO-DATE
:CordovaLib:packageReleaseRenderscript UP-TO-DATE
:CordovaLib:compileReleaseRenderscript

UP-TO-DATE
:CordovaLib:generateReleaseResValues UP-TO-DATE

:CordovaLib:generateReleaseResources UP-TO-DATE
:CordovaLib:packageReleaseResources
 UP-TO-DATE

:CordovaLib:processReleaseManifest

UP-TO-DATE

:CordovaLib:generateReleaseBuildConfig

UP-TO-DATE

:CordovaLib:processReleaseResources

UP-TO-DATE
:CordovaLib:generateReleaseSources UP-TO-DATE

:CordovaLib:incrementalReleaseJavaCompilationSafeguard

UP-TO-DATE

:CordovaLib:compileReleaseJavaWithJavac

UP-TO-DATE

:CordovaLib:processReleaseJavaRes

UP-TO-DATE

:CordovaLib:transformResourcesWithMergeJavaResForRelease

UP-TO-DATE
:CordovaLib:transformClassesAndResourcesWithSyncLibJarsForRelease

UP-TO-DATE
:CordovaLib:mergeReleaseJniLibFolders

UP-TO-DATE

:CordovaLib:transformNative_libsWithMergeJniLibsForRelease

UP-TO-DATE

:CordovaLib:transformNative_libsWithSyncJniLibsForRelease

UP-TO-DATE

:CordovaLib:bundleRelease

UP-TO-DATE

:prepareAndroidCordovaLibUnspecifiedReleaseLibrary

UP-TO-DATE
:prepareReleaseDependencies

:compileReleaseAidl

UP-TO-DATE
:compileReleaseRenderscript

UP-TO-DATE
:generateReleaseBuildConfig

UP-TO-DATE

:generateReleaseResValues

UP-TO-DATE
:generateReleaseResources UP-TO-DATE
:mergeReleaseResources

UP-TO-DATE
:processReleaseManifest

UP-TO-DATE

:processReleaseResources

UP-TO-DATE

:generateReleaseSources UP-TO-DATE

:incrementalReleaseJavaCompilationSafeguard

UP-TO-DATE

:compileReleaseJavaWithJavac

UP-TO-DATE

:compileReleaseNdk
 UP-TO-DATE

:compileReleaseSources UP-TO-DATE

:lintVitalRelease

:mergeReleaseShaders

UP-TO-DATE
:compileReleaseShaders

UP-TO-DATE

:generateReleaseAssets
 UP-TO-DATE

:mergeReleaseAssets

UP-TO-DATE
:transformClassesWithDexForRelease

UP-TO-DATE
:mergeReleaseJniLibFolders

UP-TO-DATE
:transformNative_libsWithMergeJniLibsForRelease

UP-TO-DATE

:processReleaseJavaRes
 UP-TO-DATE
:transformResourcesWithMergeJavaResForRelease
 UP-TO-DATE

:validateSigningRelease
:packageRelease

UP-TO-DATE
:assembleRelease

:cdvBuildRelease

BUILD SUCCESSFUL

Total time: 1.09 secs

Built the following apk(s):
	/Users/kamok/code/election/election-mobile/platforms/android/build/outputs/apk/android-release.apk
	/Users/kamok/code/election/election-mobile/platforms/android/build/outputs/apk/android-release-unsigned.apk

I figured out the issue. Simply upgrade the ionic-cli from 2.1.4 to 2.2.1. Now build uses production, my .apk is minified again, load up time is fast, and I can sleep again. I don't know if updating npm had to do with it, but what I did was remove npm and reinstall using homebrew. Reinstall cordova and ionicvia the new npm. Reason I did this was because regular old npm install -g ionic didn't work and it wouldn't upgrade away from 2.1.4.
Strange enough, the building of android ALSO works on ionic package. Before it was also not building with the --prod tag, or atleast I'm assuming it because my boot time for my app was slow, and app was not compressed from 8.8MB to 6.6MB.
Cheers.


Kibana 4 support - #42

#42

squash your commits maybe ?

@satterly Any thoughts on above... ?

Any further thoughts on getting this merged ? @mchv / @satterly

Thanks for your PR. However, we decided to just junk support for Kibana 3 so it made the template much simpler. No need for switch statements on version. See #45. Testing of this template would be appreciated and any bug reports or fixes will be welcomed.

@satterly Cheers was just about to close myself :)


Consider creating a truffle framework box on this template

Is your feature request related to a problem? Please describe.
So dapps (decentralized applications) are growing and are using react to create web apps to connect to the ethereum network. Truffle is the framework used to assist with this. there are templates called truffle boxes to create apps. I honestly think your react template is the best template out there and think you should integrate your amazing template
Describe the solution you'd like
I wanted to actually take a version of your boilerplate I have and create a box on it. But I thought I would let you know, maybe you want to, contribute yourselves.
Describe alternatives you've considered
Here are the steps to create a box. If you don't want to do this. Let me know, ill attempt to create one based on your template.
Additional context
Here is the react box I am using at the moment.

Excellent! We'd love to be be part of a Truffle box for dapps!
Personally what little time I can dedicate to OS work I try to dedicate to this project directly but I'd be happy to help out if any questions/issues come up.
I'll keep this open as I'm sure the rbp+dapps intersection will be relevant to a few people who come here.

Closing for lack of activity.


Augur State Bulk Sync Chunking / Parallel Request Optimization

Currently, Augur State syncs each DB's block chunks sequentially from lowest block to highest. We should optimize it to retrieve all the block chunks in parallel and then queue the DB writes.

Currently this kinda already happens because we request all log types separately.
I think the real task here is to decide which log types to group and fetch all at once so that we can minimize the number of log range requests.


Update orelse typeof in optionals

I was going through the example code for the orelse section in the options and noticed a discrepancy on the line try expect(@TypeOf(b) == f32);. Upon inspection, I got that the type of b is comptime_int.
Please let me know if this is incorrect. Or if something is missing.

Closed in favour of https://github.com/Sobeston/zig.guide/commit/51f80b5fc8f25aec2a470109ee3826ce1d8281da - I should have added testing for this before, thanks for the report!


Rebased version with copy media item feature.

@pstasiak - I'm excited about getting pasting to work in Signal-iOS. So thanks for working on this.

What you've implemented here seems to be pasting for location, but not photos/videos/animate-gifs (the media types currently supported by Signal). Adding support for these other media types opens a can of worms (see: https://github.com/WhisperSystems/Signal-iOS/issues/1132).

I believe that ideally this work should be done in the Signal repo, not in JSQMessageViewController - unless that substantially complicates things / breaks existing patterns.

I feel like this fork of JSQMessageViewController is untenable in the long term, and we should be working to minimize further changes to it, and if possible, switch to the upstream fork. If not possible, do what we can to minimize the number of files we're touching and localize changes to core files. Did you try making these changes to the Signal MessagesViewController?
The sooner we can get away from diffs like this the better:

Apart from updating JSQMVC to the latest upstream, did you consider making all other changes directly to Signal-IOS?

we've since unforked, so this is no longer relevant.


Allow user to trigger monitors through kraken-web.

do not merge yet

@soenkehahn can you review?

LGTM


[python] how to set number of thread for lightgbm.train()?

There seems no instruction on how to set number of threads in https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.train.html#lightgbm.train
I tried to set in lightgbm.train(.....,num_threads=8) but it gave me an error:
model = lgb.train(params, d_train, num_boost_round=config['num_iterations'], valid_sets=d_validation, early_stopping_rounds=50, verbose_eval=100,num_threads=config['num_thread'])
TypeError: train() got an unexpected keyword argument 'num_threads'

If parameter is not presented among function arguments, then you can set it via params or kwargs.
In your case:
params = {"num_threads":8}
model = lgb.train(params, d_train, num_boost_round=config['num_iterations'], ...)

Thanks!


No more chainable APIs?

I am in the process of upgrading my project to a newer version of Entitas and I noticed that 0.37.0 is generating void for the component methods, which means I can no longer chain Entity creation, ie, something like below no longer works;
context.ReplaceMoveInput(movement).IsDeleteOnExit(true);

Or for a specific entity,
e.AddTurnBased(i + 1, config.turnDelay)
 .IsAIMove(true)
 .AddFoodDamager(dmg)
 .AddSmoothMove(config.turnDelay)
 .AddAudioAttackSource(Audio.scavengers_enemy1, Audio.scavengers_enemy2);

Is there a replacement or new best practice for this functionality? Having a fluent style API was really nice for quickly setting up entities.

It's due to performance. See: https://gitter.im/sschmid/Entitas-CSharp?at=58a356d321d548df2c0c0367

Ok thanks for the link,
The transcript from Simon is as follows:

It’s actually both - Design and Performance.
Design:
I think Fluent Api makes sense when creating / configuring / building objects. That’s why Systems.Add() can still be chained. You do it once in the beginning. With entities you do more and different things all the time.
Performance:
Before I moved the whole code base to a type-safe approach, I did several tests regarding generics, methods, method returns. Methods which return sth are generally slower. But to be fair, you have to run this test a lot of times to see a performance difference, but generally it’s slower :wink:
Both those arguments might not seem strong on their own, but both of them point at the same direction which is removing the return.
But if you’re life depends on it, or you simply have a different opinion on that, it’s now a piece of cake to add your own custom code generator which can revive the returns :wink:

I think I'll look into using the Systems.Add() method he mentioned. Does anyone have an example?

I finished converting my example project over to the latest version of Entitas.
Still looking for an example of what Simon was referring to.

Hey @JamesMcMahon,
sorry for removing this, but the good thing is, if you're preferring a different output from the code generator you can easily customize this by adding and using a custom generator.
All current generators can be found here:
https://github.com/sschmid/Entitas-CSharp/tree/develop/Addons/Entitas.CodeGeneration.Plugins/Entitas.CodeGeneration.Plugins/Plugins/CodeGenerators
You can create a new class and get inspired (aka copy paste) by existing generators. Put all your custom generators into an Editor folder and you will be able to select them in the Entitas Prefs Window.
You could for example change the ComponentEntityGenerator and create a custom one that return the entity.
https://github.com/sschmid/Entitas-CSharp/blob/develop/Addons/Entitas.CodeGeneration.Plugins/Entitas.CodeGeneration.Plugins/Plugins/CodeGenerators/ComponentEntityGenerator.cs#L20-L43
    public ${ContextName}Entity Add${ComponentName}(${memberArgs}) {
        var index = ${Index};
        var component = CreateComponent<${ComponentType}>(index);
${memberAssignment}
        AddComponent(index, component);

        return this;
    }

Hope that helps

Ah, I see what Systems.Add was referring to now (that API being fluent but this change made the entities non-fluent).
Thanks for the answer.


Efficient computation of Jdiff - [merged]

In GitLab by @cmastalli on Aug 13, 2019, 14:21
Merges topic/Jdiff-computation -> cpp_devel
this PR I included a tailored efficient computation of Jdiff based on Jintegrate #223. This method improves the computation performance. Additionally, this PR includes few changes that removed Eigen memory allocation; further improvement are needed against latest version of Eigen.

In GitLab by @cmastalli on Aug 13, 2019, 14:22
enabled an automatic merge when the pipeline for 551bb284b24e6e3446f6091e8dd706f50a2c46cb succeeds

In GitLab by @cmastalli on Aug 13, 2019, 16:26
merged

In GitLab by @cmastalli on Aug 13, 2019, 16:26
mentioned in commit ba2f0146d6fc0b3fc2a96f8d2a9fc796d5ab2054


Update index.js

add Linkdin

the icon is not aligned properly...closing this pr..kindly align the linkedin icon properly and then reopen the pr


Handle gzip encoding for errors whilst using consume API in BufferFillerActor

Looks good!
Please solve the conflicts and I will approve it.

Create a new PR without code-styling conflicts: #595


feat: 1421 refactor text

Description
Closes #1421
Changelog
New

None

Changed

Text updates
Sorting checkbox alphabetically

Removed

None

How was this tested?

[ ] 🧠 Not needed
[x] 👀 Eyeball
[ ] 🤖 Added tests

What gif/image best describes this PR or how it makes you feel?

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend
Oracle-API

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Closing to free resources.  Reopen when ready!

I will reopen this when I review the PR

your front-end build failed, please check out the error in the action run

@ngunner15  your frontend build is still failing, please check the error in the action

@ngunner15 your frontend build is still failing, please check the error in the action, it's not an issue with GitHub btw

@ngunner15 Try locally with cd frontend && docker build . -t frontend.
It appears to be failing during transpilation (ts -> js conversion).  Looks like FavActivityType needs a description property added.
><EMAIL_ADDRESS>build
> tsc && vite build

src/config/FavouriteActivitiyMap.ts(10,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.
src/config/FavouriteActivitiyMap.ts(27,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.
src/config/FavouriteActivitiyMap.ts(36,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.
src/config/FavouriteActivitiyMap.ts(45,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.

@ngunner15 your frontend build is still failing, please check the error in the action, it's not an issue with GitHub btw

@ngunner15 Try locally with cd frontend && docker build . -t frontend.
It appears to be failing during transpilation (ts -> js conversion). Looks like FavActivityType needs a description property added.
><EMAIL_ADDRESS>build
> tsc && vite build

src/config/FavouriteActivitiyMap.ts(10,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.
src/config/FavouriteActivitiyMap.ts(27,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.
src/config/FavouriteActivitiyMap.ts(36,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.
src/config/FavouriteActivitiyMap.ts(45,5): error TS2353: Object literal may only specify known properties, and 'description' does not exist in type 'FavActivityType'.

Thanks for the info 👍


Desktop client's changelog problems

[ ] Download links to binaries older than v2.1.1 are broken (e.g. https://download.owncloud.com/desktop/stable/ownCloud-<IP_ADDRESS>83-setup.exe)
[ ] v2.2.1 links are not even present in the changelog.
[ ] All the Linux download links point to the stable version OBS repository

Also, It would be nice to have a delimiter (e.g. horizontal rule), clearly stating from which versions of the client is "deprecated"/have issues with the server: https://doc.owncloud.org/desktop/2.2/introduction.html#introduction:

Because of various technical issues, desktop sync clients older than 1.7 will not allowed to connect and sync with the ownCloud 8.1+ server. It is highly recommended to keep your client updated.

Note: some package managers, like chocolatey on Windows, rely on this links to offer some old versions of the client and therefore are unable to install.

Closing because referencing to old website before relaunch.


Add arm64-v8a and armeabi-v7a build target

This PR will add arm64-v8a and armeabi-v7a as additional build target. I think it can be useful for users to download smaller apk build compared to large universal apk build. Zeus for example provided them as the option in their  Releases.
Thank you.

Thank you @atrifat


[Improvement-11380][scp-host.sh] Set StrictHostKeyChecking=no option to ssh

Purpose of the pull request
This PR close: #11380
Even if the password-free account is configured, when the user uses the scp-host.sh script to log in to the machine for the first time, an error (Host key verification failed.) may occur.
So set the  -o StrictHostKeyChecking=no option to ssh in scp-hosts.sh.
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

Hi @rickchengx , to target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.

Hi @rickchengx , to target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.

Sure. Thanks for your advice, I've added the keyword close: #issue_id.


[DOC] Adding Instructions for using Keycloak JWT with DataProvider/Axios in Refine

Documentation issue
No, this is not related to an existing part of the documentation.
Describe the thing to improve
A clear and concise description of problems and what you would like to see.
I would like to suggest adding a section to the documentation or examples that demonstrates how to use Keycloak JWT with the DataProvider and Axios in Refine. Specifically, I'd like to see guidance on how to pass headers to the AxiosInstance, such as
request.headers["Authorization"] = Bearer ${token}`;

in the context of authentication with Keycloak.
Example:
axiosInstance.interceptors.request.use((request) => {

  // const { key, init } = useKeycloak();
  const token = localStorage.getItem(TOKEN_KEY);
  if (token) {
    if (request.headers) {
      request.headers["Authorization"] = `Bearer ${token}`;
      request.headers["Accept"] = `application/vnd.api+json`;
      request.headers["Content-Type"] = `application/vnd.api+json`;
    }
  }
  return request;
});

Describe the solution (optional)
To improve the experience for beginners and readers, I propose adding a dedicated section or example in the documentation that showcases the integration of Keycloak JWT with Refine's DataProvider and Axios. This section should provide step-by-step instructions, code samples, and explanations on how to set up authentication using Keycloak and pass authentication headers in Axios requests.

Hello @PitBult, we have some sections that mentions it already. If you see any improvement, we are open to contributions.
https://refine.dev/docs/tutorial/understanding-authprovider/create-authprovider/#setting-authorization-credentials
https://refine.dev/docs/api-reference/core/providers/data-provider/#error-format
https://refine.dev/docs/packages/documentation/data-providers/simple-rest/#usage


Collection level DEFAULT_BITSTREAM_READ policy is trickling down to bundles (and it shouldn't be)

Describe the bug
In a local DS7.5 instance, and in the DS7 demo server, if you create a collection with a non-anonymous DEFAULT_BITSTREAM_READ policy, any items you put in the collection will inherit the policy to both bitstreams AND  bundles. This is a problem because restricting access to the bundle makes the bitstream invisible to a user (as opposed to just displaying as restricted, with a little lock image) and makes in functionally unavailable for a user to request.
This situation is demonstrated in the demo server with
collection: images of dogs and item muffin!
To Reproduce
Steps to reproduce the behavior:

log in to DS7 demo
create a collection with DEFAULT_BITSTREAM_POLICY limited to group: administrator and other READ policies group: anonymous
deposit an item to the collection created in step 2
look at the item from a non-logged in browser.

Expected behavior
The DEFAULT_BITSTREAM_READ policy should be applied to bitstreams, not bundles, so in step 4, above you would ideally see the bitstream name, followed by an "access restricted" note and a lock. A user would be able to click on this and request a copy (assuming request a copy is enabled)
Related work
Link to any related tickets or PRs here.

@kshepherd : Pinging you on this as it sounds somewhat related to your recent work in this area with https://github.com/DSpace/DSpace/pull/8783   It sounds similar, but not sure it's the same issue you fixed recently.

This does sound related. @cm647 FYI we recently patched things so that new bitstreams would properly check for the collection's DEFAULT_BITSTREAM_READ policy instead of only inheriting READ from bundle.
This issue is related... It sounds like we should agree that bundles should.. hm. Inherit from DEFAULT_ITEM_READ? With default anonymous? That way, if the intention is for an item to be read, then we are also expecting to be able to list bitstreams in that item, even if they themselves are restricted.
Either that, or we should propose a DEFAULT_BUNDLE_READ policy that can explicitly set this (I can imagine a use case where people want metadata available, but files and the ability to list files restricted by default) and fall back to DEFAULT_ITEM_READ if not present.
Any thoughts?

@kshepherd : I think it is reasonable for Bundle to inherit from DEFAULT_ITEM_READ at this time.  If we find a use case for a separate DEFAULT_BUNDLE_READ we could add it later, but it doesn't seem necessary to me (at least not yet).

We (Cornell’s DSpace instance) definitely need the ability to have items with public metadata, and listed (but restricted) bitstreams. One of our use cases is having collections of research material that is available automatically to the campus community, but with bitstreams that should be listed & restricted for off campus users.
If there is a way to enact ‘request a copy’ that doesn’t require being able to see that a bitstream exists, we would have less of a need for listed & restricted bitstreams. But right now, in order to request a copy, users need to try to click on the bitstream, so without the bitstream to click on we have Serious Workflow Problems ☹  -Chloe
Chloe A. McLaren (she/her/hers)
Metadata Projects Librarian
Library Technical Services
Cornell University Library
From: Tim Donohue @.>
Date: Tuesday, May 30, 2023 at 12:05 PM
To: DSpace/dspace-angular @.>
Cc: Chloe McLaren @.>, Mention @.>
Subject: Re: [DSpace/dspace-angular] Collection level DEFAULT_BITSTREAM_READ policy is trickling down to bundles (and it shouldn't be) (Issue #2273)
@kshepherdhttps://github.com/kshepherd : I think it is reasonable for Bundle to inherit from DEFAULT_ITEM_READ at this time. If we find a use case for a separate DEFAULT_BUNDLE_READ we could add it later, but it doesn't seem necessary to me (at least not yet).
—
Reply to this email directly, view it on GitHubhttps://github.com/DSpace/dspace-angular/issues/2273#issuecomment-1568698701, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFRPLSRVWHQZ3SS3TFOEIC3XIYLEDANCNFSM6AAAAAAYPDOSOA.
You are receiving this because you were mentioned.Message ID: @.***>

additional information/use case: we are noticing that our two most common types of restricted items that employ request a copy- our most used service- are behaving the same way. 1. what chloe outlines above- a collection level IP restricted policy and 2. batch uploaded items assigned embargos. Both should have policies that restrict bitstreams and should show a file with a lock that a user can click on to initiate RAC, but both have metadata and no file. the first is related to the inherited collection policy as the ticket states. The embargo issue is related to our batch processing submissions. Chloe is following the default dspace process and specifying the field (dc.description.embargo). while ds 7 and ds6 are registering the policies in the same way, ds 7 is interpreting that policy differently. it's the same problem as the collection policy inheritance (we get a bundle policy instead of bitstream, and now the bundle policy hides our files)


chore(package): update @types/jasmine to version 2.5.51

Closes #73

Codecov Report

Merging #77 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #77   +/-   ##
=======================================
  Coverage   96.55%   96.55%
=======================================
  Files           6        6
  Lines          58       58
  Branches        3        3
=======================================
  Hits           56       56
  Partials        2        2

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 c607e6a...a9099d7. Read the comment docs.


Suggested corrections for violations

Whenever PSScriptAnalyzer (PSSA) is invoked on a PowerShell (PS) script, it runs a set of rules against the contents of the script. Each rule checks for some sort of violation. As of now, the violations need to be fixed manually by the script author. Many of the violations, however, are simple fixes that can be automated. For example, gci triggers the PSAvoidUsingAliases rule and can be easily rectified by replacing gci with Get-ChildItem.
Whenever there are violations in a script, PSSA invocation emits a list of DiagnosticRecord (DR) instances. Each DR instance corresponds to one violation of a rule. The DR also contains the extent of the violation. We want to provide the correction extent that can be used to remove the violation. Whenever, a rule processes a script syntax tree, it has, in most cases, all the information needed to correct the violation. Hence, it would be efficient for the rule itself to emit the correction extent such that we can tag on a corrections property to each emitted DR. There are three cases that such a correction extent needs to address:

Replace the violation text with correction text.

For Example, replace gci with Get-ChildItem

Add text to the script that corrects the violation.

For Example, should process related changes.

Ability to propogate the change within the entire script

For Example, consider the PSUseSingularNoun rule. It triggers whenever a function definition is found wherein the noun part is plural. To rectify the violation we can suggest the singular version of the noun and then apply the change to all the instances of the function in the script.

The primary motivation behind this feature is to allow editors to take advantage of this property to enable quick fix scenarios for PS scripts.

Love this.
A few thoughts:

Some changes will span multiple script files. As this feature is implemented, please consider providing support (parameters) to apply a broad change (like in scenario 3) across multiple files. A simple example is identifying a function in a module that has a pluralized noun, and fixing that function name across the entire module -- everywhere it is invoked, in an Export-ModuleMember call, and in the manifest.

Some changes will be breaking, such as renaming a function in a module that has been published, and PSScriptAnalyzer is about best practices.  It would be ideal if PSScriptAnalyzer could identify required changes (for example, when renaming a function/fixing the verb/noun, identify the extent(s) that need to be updated.  That might mean a rule emitting a violation with a collection of extents in a DR, maybe identifying dependencies between them with the root extent where the problem was found, and other extents that are impacted via modification of the root extent, either in a list via properties that define dependencies or in a tree structure or some such.  This may also include identifying code that could/should optionally be added (e.g. if you are fixing a naming violation of a function in a published script or module, you really should define an alias to map the old name to the new, corrected name), in which case the extent could identify the entry point where the code would be added along with the code that would be added.  That would allow for many editors to easily get functionality similar to plugins in Visual Studio, where correcting an auto-correctable violation involves making the base correction and walking through other recommended and optional corrections, approving or skipping each one as you pass through a single file, with an option to continue walking through the entire module that the file belongs to (assuming you have some logic to identify when a function that is defined in a ps1/psm1 file appears to belong to a module by some simple logic that analyzes the file structure where it is stored).

You might not hit these targets on the first go, but I wanted to add them while they were top of mind because I think this is what this feature/functionality should strive towards.

The model we've designed will all multi-file corrections in the first release.  A single rule will be able to make insertions, modifications, or deletions in an arbitrary set of files.  The example you give for renaming a function and finding all of the occurrences and changing them is one that we're actively investigating.
The idea of defining an alias to map the old name to the new one is great!  I think we'll be at the mercy of editors when it comes to giving the user optional, additional corrections while performing a primary one (like renaming a cmdlet and then getting the option to add an alias).  Right now VS Code doesn't give you the ability to do a multi-step correct, you just select a single correction item and it applies that.
I'm looking forward to this one!

@KirkMunro Those are great suggestions. Thanks!
As @daviwil mentioned, the current model that we have can allow multi-file edits. But for the first release of this feature we are planning to roll out support for only script-wide edits. This is due to the way PSSA works as of now. Having the ability to propagate a change within a module and across modules is something that will need a significant redesign of the PSSA engine. Nevertheless, these are great suggestions to have on our TODO list.

I agree with everything said so far in this thread with one caveat: we might want to hold off on implementing any auto-correct for changes which may require edits throughout multiple files.
As @KirkMunro suggests, that design is probably going to a bit trickier, and I don't want to paint ourselves into a corner here.

At present we have added this capability (see PR #508) for the following rules

[ ] PSAvoidUsingCmdletAliases
[ ] PSAvoidUsingPlainTextForPassword
[ ] PSMisleadingBacktick
[ ] PSUseToExportFieldsInManifest

Which other rules should be equipped with this functionality so that it can improve authoring/editing PS scripts experience?

I would be more suggestive of moving this functionality to its own cmdlet, Invoke-ScriptAnalyzerCorrection
Reason being for me is that it allows me as an author to either use it or not use it especially for the reasons listed by Kirk above.
I like the idea to be able to do this though and think it's great it's being put on the table but I'm a little concerned that doing so as an additional aspect to Invoke-ScriptAnalyzer will likely pollute it's real purpose - To inform the author about what they could be looking to improve in their own writing practices.

I don't think we need another cmdlet.  But the idea of applying auto-correct is interesting and makes me think back to tools like chkdsk (am I aging myself?).  That would allow users to get the details about what could be improved by default (like it does today, but with an identifier/flag in the default format applied to the output or a "[AutoCorrectible]" prefix/qualifier in the issue description that clearly indicates whether or not a rule is autocorrectable), and then PSScriptAnalyzer could be run again with an optional -AutoCorrect switch to apply the changes.  I think that would be a better approach than a separate cmdlet.

@kilasuit This feature only emits a property which can be consumed by external utilities (e.g. editors) to provide auto-correct capability. Invoke-ScriptAnalyzer by itself will not modify the scripts that it analyzes.

@kapilmb Thanks for that clarificatin, that makes more sense, so that editors can apply correction to one or multiple issues identified by this feature, leveraging their UI properly to highlight auto-correctible items and to show what the corrections look like before applying them.

A PR - #508 - that adds this feature was merged. Hence, I will close this thread and would like to point everyone to the new thread - #516 - where I would like to request community's input on what rules would be most valuable w.r.t the suggested corrections feature.


Cannot communicate with deis pods

After getting what I think is a clean install of kube-cluster, I ran the install_deis command.  At the very end, it failed with trying to register the admin user.  It appears the hostnames it is trying to use aren't resolving properly.  I believe this is because it is trying to resolve via my LAN dns server instead of the k8s one:
Registering Deis Workflow user ...
http://deis.<IP_ADDRESS>.nip.io/v2/ does not appear to be a valid Deis controller.
Make sure that the Controller URI is correct, the server is running and
your client version is correct.
Error: Get http://deis.<IP_ADDRESS>.nip.io/v2/: dial tcp: lookup deis.<IP_ADDRESS>.nip.io on <IP_ADDRESS>:53: no such host

I wasn't sure about the whole nip.io thing.  I read the docs on the DNS addon that you link to in your readme ( https://github.com/kubernetes/kubernetes/blob/release-1.2/cluster/addons/dns/README.md ), and it talked about using the <IP_ADDRESS> DNS server.  I followed the instructions on that page to run a simple busybox pod and tested out the names found there, but even with those, I still can't get deis to register.
Make sure that the Controller URI is correct, the server is running and
your client version is correct.

Error: Get http://deis-controller.deis/v2/: dial tcp: lookup deis-controller.deis on **<IP_ADDRESS>:53**: no such host
bash-3.2$

Is this a botched configuration on my end or is there some sort of conflict with my LAN DNS?

@deinspanjer the DNS addon is purely for kubernetes use, e.g. it allows to reach other pods via service name.
nip.io is used as there is no local domain to associate with deis.. platform.
It has to be your LAN DNS end.
can you check that all deis workflow components are running properly with:
kubectl --namespace=deis get pods

Well, all of the deis pods were running when I was working with this yesterday, but today, (after a sleep and restore of my MBP) when I run get pods, I see there are a few that are not running properly:
bash-3.2$ kubectl logs deis-workflow-manager-smsw1
Error from server: pods "deis-workflow-manager-smsw1" not found
bash-3.2$ kubectl --namespace=deis get pods
NAME                          READY     STATUS             RESTARTS   AGE
deis-builder-4v82h            0/1       Running            3          13h
deis-controller-mvj38         1/1       Running            4          13h
deis-database-7o6gd           1/1       Running            2          13h
deis-logger-fluentd-gtvrw     0/1       CrashLoopBackOff   4          13h
deis-logger-fluentd-h7hg4     0/1       CrashLoopBackOff   4          13h
deis-logger-z4xc7             1/1       Running            0          13h
deis-minio-hud5q              1/1       Running            0          13h
deis-monitor-grafana-otexz    1/1       Running            0          13h
deis-monitor-influxdb-r41dy   1/1       Running            0          13h
deis-monitor-stdout-f4gs0     1/1       Running            0          13h
deis-monitor-telegraf-rvrl0   1/1       Running            1          13h
deis-monitor-telegraf-ynp0q   1/1       Running            1          13h
deis-registry-nafns           1/1       Running            0          13h
deis-router-x7wix             0/1       CrashLoopBackOff   6          13h
deis-workflow-manager-smsw1   1/1       Running            0          13h
bash-3.2$ kubectl describe deis-router-x7wix
the server doesn't have a resource type "deis-router-x7wix"
bash-3.2$ kubectl describe pods deis-router-x7wix
Error from server: pods "deis-router-x7wix" not found

And further, the error I reported in issue 47 about the x509 error is back as well. :/  It seems that my problem was not in fact with the symlinked id_rsa key.

@deinspanjer the problems above are not really related to the the Kube (solo and cluster) apps, I will report the issues back to my team

@deinspanjer the problem is fixed with the latest Workflow v2.1.
Use v0.3.4 App which supports Workflow v2.1 install

still an issue on k8s v1.3 and workflow v2.1, the issue should be fixed in workflow v2.2 by https://github.com/deis/builder/pull/382


Change :datetime dtype to be {:datetime, time_unit}

Based on the comment https://github.com/elixir-explorer/explorer/pull/642#issuecomment-1623283607
This PR handles the first part

Change :datetime dtype to be {:datetime, time_unit} (as listed here). Change all conversion functions to handle datetimes with different time units.

Hi @sasikumar87, this looks great! I have dropped some comments. Also, in encoding.rs, we still pattern match on DataType::Datetime(TimeUnit::Microseconds, None) but I think it can be DataType::Datetime(_, None).

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Check IPv4 address being available

Running terraform in a 10.21.50/24 enterprise-LAN. terraform allocated IPv4 addresses that were already used.

The module creates PowerVC network ports to allocate the IP and macaddress. Not sure how the above case is possible.

The 10.2150/24 is a DHCP LAN.

@xxTReich in your setup, did you create a separate network for OCP or you are using an existing network which already has some LPARs running ?
From the description it looks like you are using an existing network which already has some LPARs running and have IPs assigned from the network.


Zooming on hovering on the products

Description
In the shop section, when we hover over the products we can apply the zooming animation to make it look more interactive.
Screenshots

Additional information
The above screenshot attached is what it will look like if we hover over the images.
🥦 Browser
Google Chrome
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

Please assign me to this issue, I am willing to submit a PR!

Hello @IAmTamal ,
Hope you are doing great!
I want to contribute to this project. Can you please assign this issue to me?

@wickedwisdom0911 - Don't zoom in, instead have a box shadow that appears on all the 4-side on hover
we can have #297f75 color and a pointer cursor !

Sure would get on it.  Once I am done, I will attach some demo Screenshots
to see does it match what you are looking for. You can assign me. Thanks.
On Mon, Oct 10, 2022, 8:53 PM Tamal Das @.***> wrote:

@wickedwisdom0911 https://github.com/wickedwisdom0911 - Don't zoom in,
instead have a box shadow that appears on all the 4-side on hover
we can have #297f75 color and a pointer cursor !
—
Reply to this email directly, view it on GitHub
https://github.com/IAmTamal/Milan/issues/447#issuecomment-1273478549,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASWHKPJIATL3EYIJHVSR7ADWCQYFZANCNFSM6AAAAAARBBEAII
.
You are receiving this because you were mentioned.Message ID:
@.***>


Slow executable startup time on MacOS

MRE:
from time import time

def main():
    st = time()
    print("Wowzers! It started!")
    et = time() - st
    print("Script time: ", et)

if __name__ == "__main__":
    main()

Terminal Output:
Moosems $ python3 -m nuitka test.py --standalone; time ./test.dist/test.bin
Nuitka-Options: Used command line options: test.py --standalone
Nuitka: Starting Python compilation with Nuitka '2.4.7' on Python '3.12' commercial grade 'not installed'.
Nuitka: Completed Python level compilation and optimization.
Nuitka: Generating source code for C backend compiler.
Nuitka: Running data composer tool for optimal constant value handling.
Nuitka: Running C compilation via Scons.
Nuitka-Scons: Backend C compiler: clang (clang 15.0.0).
Nuitka-Scons: Backend linking program with 6 files (no progress information available for this stage).
Nuitka-Scons: Compiled 6 C files using ccache.
Nuitka-Scons: Cached C files (using ccache) with result 'cache hit': 6
Nuitka: Keeping build directory 'test.build'.
Nuitka: Created binary that runs on macOS 13.0 (x86_64) or higher.
Nuitka: Successfully created 'test.dist/test.bin'.
Wowzers! It started!
Script time:  0.00015425682067871094
./test.dist/test.bin  0.04s user 0.03s system 5% cpu 1.143 total
Moosems $ time ./test.dist/test.bin
Wowzers! It started!
Script time:  3.695487976074219e-05
./test.dist/test.bin  0.04s user 0.01s system 95% cpu 0.057 total
Moosems $ time python3 test.py
Wowzers! It started!
Script time:  2.574920654296875e-05
python3 test.py  0.04s user 0.02s system 88% cpu 0.059 total

Nuitka version, full Python version, flavor, OS, etc. as output by this exact command.
2.4.7

How did you install Nuitka and Python
pip and python.org

The larger the project, the longer the initial startup time (first time the binary is run) takes. It might be something to consider looking into but I understand if this isn't a priority. I searched the docs but couldn't find anything on this. If you have any ideas on bringing down this initial time, I'm open to suggestions. Without --standalone it was still 0.704 total the first time and averaged about the same afterwards. Maybe this also is a my end thing and, if so, feel free to close this.

In your example given, I am failing to identify a difference when I try it. The time output makes no sense. You can use --experimental=show-startup-time to look into the details. But since you cannot even provide the output of --version even though it clearly asked for that, I feel like you are not good at providing information.

@kayhayen Respectfully, there is no lack of information. The line Nuitka: Starting Python compilation with Nuitka '2.4.7' on Python '3.12' commercial grade 'not installed'. in the traceback shows both the Nuitka and Python version, the system is evident by the name, the flavor is a python.org install which was also stated, and it is very clear that it takes 1.5 seconds for the executable to run which is crazy given that the script itself only runs for 2.574920654296875e-05 to 0.00015425682067871094 seconds depending on how many times the executable has been run. Nuitka 2.4.10 still has this though the first run now generally takes about 1.15 seconds.
Wowzers! It started!
Script time:  0.00019788742065429688
2024-10-18 05:05:33.370 main(): Exited from __main__.
2024-10-18 05:05:33.370 main(): Calling Py_Exit.
2024-10-18 05:05:33.374 Nuitka_at_exit(): Called by C exit()
./test.dist/test.bin  0.03s user 0.04s system 5% cpu 1.226 total

It seems that, as I showed previously, this is not an issue with the script itself running but something about the executable startup time.

When people say "respectfully", what follows is usually not respectful, and this seems to be no exception. The only respectful answer would be to provide the missing information. So why should I work on this.


removed static tests replaced by dynamic ones

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 98.962%

Files with Coverage Reduction
New Missed Lines
%

theta/include/compact_theta_sketch_parser_impl.hpp
2
80.85%

common/include/bounds_binomial_proportions.hpp
3
93.94%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
15629

Relevant Lines:
15793

💛  - Coveralls


Make "default" an admin namespace

Not sure at what point this got broken since I'm pretty sure it used to work, but anyway, we're currently not actually using VNID 0 for namespace "default", so it doesn't work...

@rajatchopra @pravisankar

We decided not to merge https://github.com/openshift/origin/pull/4370 and that caused this issue.
It will be good, if we let the plugins to decide the sane defaults for AdminNamespaces.
Instead of assigning 'default' namespace in ovssubnet/common.go, do it in multitenant.go (https://github.com/openshift/origin/pull/4370/files#diff-d8a8179e40bbd42ad8c40b0fb54e2b4eR36)
kc.AdminNamespaces = append(kc.AdminNamespaces, "default") before calling kc.StartMaster(...)

ok, so this would be an origin change then


Add support for less

Hi!
Saw your post on /r/webdev, really like how minimal this is and would like to use it for a few of my projects. I've added support to expose a tipograf.less so it can be imported by people running less builds and allow for the base variables to be customised.
I couldn't figure out a way to automatically test this so what I've done is added a step to build the less into css and open up the demo webpage with the styles loaded for visual inspection.
npm run test:less will do this.
Please let me know if there's anything I've missed and I'll be happy to fix it up. 😄
Cheers,
Aquib.

Hi @aquibm !
Glad you like the repo. I was playing to add support for Less. Thank you 👍


⚠ SceneTime has degraded performance

In 21a38fd, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 12443 ms

Resolved: SceneTime performance has improved in e339779.


[Bug] Incorrect setting in tb-mqtt-transport.yml for AWS microservices configuration

When MQTTS is required, there are sections in the tb-mqtt-transport.yml file that need to be uncommented. One of these is incorrect and causes the service to crash on start-up:
- name: MQTT_SSL_KEY_STORE
  value: "/ssl-config/server.pem"

should be:
- name: MQTT_SSL_PEM_CERT
  value: "/ssl-config/server.pem"

Hello, @greg-mcnamara
Thanks for the note. Fixed.


Remove Hilt

https://github.com/square/anvil
https://gpeal.medium.com/dagger-anvil-learning-to-love-dependency-injection-on-android-8fad3d5530c9
https://github.com/gpeal/Anvil-Sample

May need to study again what the DI is.


Ingame GUI's don't work when LuckPerms is enabled

Hey
Ingame GUIs like /enderchest and /pv 1 don't work when LuckPerms is enabled.

You need to give the proper permissions.


Do not depend on "rails" gem, depend on parts of Rails that RailsRealFavicon uses

Description
RailsRealFavicon specifies a dependency on rails gem - https://github.com/RealFaviconGenerator/rails_real_favicon/blob/32fa257ff3c27a0ab26f92675dcad934bd8f55cb/rails_real_favicon.gemspec#L20
rails is just an "umbrella gem" that depends on all the components of Ruby on Rails framework - https://github.com/rails/rails/blob/86ca5b254109e64edeb6e142e656b8a8c247cac8/rails.gemspec - ActiveSupport, ActionPack, ActiveRecord, ...
I want to avoid installing actioncable gem in my Rails application. When RailsRealFavicon specifies rails gem as a dependency then actioncable becomes my transitive dependency automatically and there is no way to avoid installing it.
Proposal
Depend only on these parts of Rails that RailsRealFavicon actually uses, for example:
gem.add_dependency "railities", ">= 3.1.0"
gem.add_dependency "actionview", ">= 3.1.0"

I'm happy to make a PR if you accept the change.

That makes sense. I would be glad to merge your PR!

No news of this issue. I admit I don't know how to handle it, apart from a trial-and-error process until I find the minimum set of dependencies (which I won't do :) ).
Don't hesitate to submit a PR if this is something you are willing to do.


add bac001 contracts and update Changelog

Codecov Report

Merging #179 into release-rc2 will decrease coverage by 0.08%.
The diff coverage is 75.00%.

@@                Coverage Diff                @@
##             release-rc2     #179      +/-   ##
=================================================
- Coverage          29.33%   29.24%   -0.09%
+ Complexity           322      319       -3
=================================================
  Files                110      110
  Lines               3351     3344       -7
  Branches             223      223
=================================================
- Hits                 983      978       -5
+ Misses              2291     2289       -2
  Partials              77       77

Impacted Files
Coverage Δ
Complexity Δ

...va/com/webank/wecross/resource/ResourceDetail.java
92.59% <ø> (-0.96%)
12.00 <0.00> (-2.00)

...ain/java/com/webank/wecross/stub/ResourceInfo.java
58.33% <ø> (+4.48%)
13.00 <0.00> (ø)

...bank/wecross/restserver/response/StubResponse.java
71.42% <66.66%> (ø)
3.00 <1.00> (ø)

...m/webank/wecross/restserver/RestfulController.java
70.17% <100.00%> (ø)
12.00 <0.00> (ø)

...ain/java/com/webank/wecross/resource/Resource.java
65.00% <0.00%> (-2.50%)
15.00% <0.00%> (-1.00%)