MPI headers availability for HIPCC

On some machines, and I'll take the example of some recent HPE-Cray environment (Frontier, Lumi and Adastra); a set of modules and wrappers needs to be used if one does not what to get his hand dirty. Under this regime, I had two issues:

HIPCC is exposed but not supported (the underlying amdclang is) and the flags such as the one needed to introduce MPI are not automatically added. I didn't succeed in making HIPFLAGS work (https://cmake.org/cmake/help/latest/envvar/HIPFLAGS.html).
CMake can succeed in finding MPI (MPI_FOUND is TRUE) but not populate any MPI_CXX_* variable such as MPI_CXX_INCLUDE_DIRS. This is probably due to the usage of the Cray wrappers but still, according to the CMake documentation, it should be set (
https://cmake.org/cmake/help/latest/module/FindMPI.html).

The fix resolved the issues aforementioned but I did not try it on a significant amount of system/environment!

After some thought, I concluded it was not worth the while trying to use two compilers (say crayclang and amdclang) notably when OpenMP support is enable. The whole program/cmake logic breaks down..


Unable to Start/Stop HBOT through RPC

Environment: Linux Docker
I am trying start start/stop HBOT through Nodred Flows through MQTT inject node, after connecting both HBOT and nodred to the same topic. I can see HBOT on the logs saying that:
''rpc - could not unpack request message 'data'
My guess that probably my triggering command start/stop is recognized  by hbot instance, may be I didn't inject the right command to trigger.
I tried the following command on the inject node:
{
"name":"start",
"command":"start"
}
start, stop etc...
All didn't work

Hi 😁 I was doing the same from a node.js application and I had to go through the source code of commlib and the MQTT bridge itself to get it, but this is the format you're supposed to use when sending messages:
const createStartCommand = () => JSON.stringify(({
  header: {
    reply_to: RESPONSE_TOPIC,
    timestamp: Date.now(),
  },
  data: {}
}));

Where RESPONSE_TOPIC is a string constant specifying the topic on which you'd like to get the response sent to.
Hope this is helpful, let me know how it goes 😁


chore(main): release 2.6.18

:robot: I have created a release beep boop
2.6.18 (2024-10-08)
Bug Fixes

deps: update rust crate clap to v4.5.20 (e6a85b1)
Include filters for charts (ee4a6e0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/philipcristiano/et/releases/tag/v2.6.18 :sunflower:


Add PHP 8.4 formula

Build<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

:beers: @BrewTestBot has triggered a merge.


Update Pages “howto”

Automatically generated by Static CMS

👷 Deploy Preview for la-chicharra processing.

Name
Link

🔨 Latest commit
c83a01d895d439f2547c288f183239061a8d672b

🔍 Latest deploy log
https://app.netlify.com/sites/la-chicharra/deploys/662d8528c698b60008352371


feat: add reusable fibers and fiber pool

It's just an idea. On my PC for test:
memtier_benchmark -t 8 --pipeline=30 --hide-histogram --test-time=3000 --distinct-client-seed -d 256 --ratio=1:0
I've got 19% performance improvement

Fibers are created once when the benchmark is starting and continue running during the lifetime of the loadtest program.
Based on this PR I do not see how these changes can affect the performance of the backend, specifically the performance of processing of requests.

Fiber creation and destruction take a significant part of CPU in our code. In my PR, I create a pool that contains all unused fibers, and next time when we need a new fiber we just get already created and don't waste time.


Save Comment class out of CommentsHelper

This PR extracts Comment from CommentsHelper, benefits:

Decouple from module
More tests coverage

My future goal is to make:

lib/danger/request_source/github.rb
lib/danger/request_source/gitlab.rb

them be easily tested.
This is part of the work 🚧 to achieve the goal along the way 🛤.

oh yes, yeah definitely, this whole section of github.rb is just asking to be vastly simplified + refactored now we've got a bunch of it tested + production'd

oh yes, yeah definitely, this whole section of github.rb is just asking to be vastly simplified + refactored now we've got a bunch of it tested + production'd

🙆 will try to simplify in another PR.
--
Could re-run build for AppVeyor? Thank you 🙇

It needs a rebase anyway 👍


New JobLog message

new JobLog message type

repeated field in case implementations want to allow retries

metadata from Job message moved to JobLog

field intended to store worker specific metadata (worker ID, SGE job ID, etc.)

old JobLog message renamed to ExecutorLog

@adamstruck Can you add some commentary on the "why" and not just the "what" for this & the other recent PRs?

The why in this case is to define how retries should be captured by the API.  There should be one JobLog per instantiation of a task. If it fails and retries there should be another JobLog created and appended to the array.   This is also the reasoning behind moving the metadata field since the worker metadata may change between attempts.

Ah gotcha. thanks. :+1:

Added start and end time fields to the JobLog message. These fields will capture the time it takes to localize inputs, run each executor step, and upload outputs.


Allow getProps template to contain buses...

...in which case invoke push instead of set.

Codecov Report

Merging #26 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   99.65%   99.65%   +<.01%
==========================================
  Files           1        1
  Lines         582      584       +2
==========================================
+ Hits          580      582       +2
  Misses          2        2

Impacted Files
Coverage Δ

dist/karet.util.cjs.js
99.65% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update Buildscript

https://github.com/GTNewHorizons/ExampleMod1.7.10/pull/12 needs to be reviewed/merged first!

Buildscript builds, runClient runs, full modpack loads and runs. The cleanup done looks good but I'm no authority on whether it is up to the very latest best practices. I assume this would benefit from wide playtesting since this mod impacts a lot of recipes.


Add a live/ directory for building a live cd.

This is just a start, lots of things to clean up.
See README.md for more info

@joylatten , @hallyn this is a work in progress. primarily because i seem to be hitting a bug in stacker :-(.


A class for fixed first column of table

With table-responsive class, scrolling right and left moves its contents horizontally. Its readability would be not good when entire contents are being moved horizontally. The readability get worse when its contents get wider.
To fix first column of table could solve its problem. Many sports websites already provide fixed first column style table to display much information on a table like below.

Duplicate of #31458.


Add Bolloju-Bharathi's file

I created a branch my-first-branch and named file like 0000-01-02-Bolloju-Bharathi.md file and commited a file and added Add Bolloju-Bharathi's file creates pull request But pull request is not opening

I created a branch my-first-branch and named file like 0000-01-02-Bolloju-Bharathi.md file and commited a file and added Add Bolloju-Bharathi's file creates pull request But pull request is not opening


Image Uploading

First off: fantastic editor. Thanks so much for putting this up for everyone.
Ok, I may be incredibly dense here, but I can't get the "upload image" option to pop up when clicking the toolbar image option. I just get the "insert image" option (similar to the heroku demo you have up).
I have the helper set up on a form for my Page object:
<%= form_for :page do |form| %>
<%= form.bootsy_area :content %>
<%= submit_tag "update" %>
<% end %>
And I have it the include code on my model:
class Page < ActiveRecord::Base
include Bootsy::Container
...
end
I ran the initial install migrations, etc.  Editor works great, but I can't get the actual uploader to come in as an option.
Any help would be much appreciated.
Thanks!
Ben

Awesome @volmer, its a great gem, and the solution worked for me too.


Make use of http_proxy and https_proxy in bitbake environment

Add default https-proxy and proxy flags to NPM_FLAGS to forward on bitbakes usage of http_proxy and https_proxy.
Unless there is an easier location we should be doing this in? This is just a quick way to keep the npm proxy configuration within bitbake and not rely on external settings that I found while working from behind a corporate proxy. Trash this pull request if there is an easier way that I didn't notice or if there is a nicer way to implement.

Thank you for the pull request.
I suspect http_proxy and https_proxy variables are not guaranteed to be defined. Obviously all NPM calls fail if they are not defined.
There needs to be dynamic inclusion of these args to npm call only if they actually have value.


Bumped ccx-rules-ocp version

Codecov Report
Base: 73.46% // Head: 73.46% // No change to project coverage :thumbsup:

Coverage data is based on head (3e47388) compared to base (5587be8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #447   +/-   ##
=======================================
  Coverage   73.46%   73.46%
=======================================
  Files          11       11
  Lines         603      603
=======================================
  Hits          443      443
  Misses        135      135
  Partials       25       25

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


r buildpack: include R CMD javareconf

R CMD javareconf sets env vars to tell R where Java is for packages that depend on it, namely rJava. Could that line be added to the regular R install while the build is still root? If Java is not present it will be ignored, so shouldn't cause problems if people haven't installed java using apt.txt.
It is more difficult to handle this when the system is not root (I haven't figured it out yet anyway). @betatim @ryanlovett Thanks for all the work on the R build pack! Wondering if you or @cboettig would recommend a different approach.

Guess this can't work since java can't be installed until after the build switches from root to a user.


[Feature] [Rebased] Add support for in-url base64 encoded images

Rebased from master: https://github.com/pinterest/PINRemoteImage/pull/240

@k06a It looks like the CI can't build post-rebase. The method earlyReturnWithOptions has a different signature, and animatedImage has been replaced with alternativeRepresentation.
Could you make those changes when you have time?
    - ERROR | [PINRemoteImage/Core, PINRemoteImage/iOS, PINRemoteImage/OSX, and more...] xcodebuild: Returned an unsuccessful exit code. You can use `--verbose` for more information.

    - ERROR | [PINRemoteImage/Core, PINRemoteImage/iOS, PINRemoteImage/OSX, and more...] xcodebuild:  /Users/travis/build/pinterest/PINRemoteImage/Pod/Classes/PINRemoteImageManager.m:590:27: error: no visible @interface for 'PINRemoteImageManager' declares the selector 'earlyReturnWithOptions:url:object:completion:'

    - ERROR | [PINRemoteImage/Core,PINRemoteImage/OSX] xcodebuild:  /Users/travis/build/pinterest/PINRemoteImage/Pod/Classes/PINRemoteImageManager.m:585:27: error: unknown receiver 'UIImage'; did you mean 'CIImage'?

    - NOTE  | [PINRemoteImage/Core,PINRemoteImage/OSX] xcodebuild:  /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSBitmapImageRep.h:15:8: note: 'CIImage' declared here

@Adlai-Holler fixed, forgot to run tests after rebase.
@appleguy should I uncomment memory caching? Looks like new version uses MD5 hash for key.

@Adlai-Holler ready to merge? :)

@k06a I believe we should now go back to putting the image in the memory cache. Also paging @garrettmoon. We'll get this merged soon, sorry for the delay.

@Adlai-Holler just discovered, that new version of earlyReturnWithOptions:url:key:object:completion: can do a lot for me.

@k06a Excellent! Love the simplification.

Where can I find any roadmap to 3.0.0 release?

@k06a we don't have a public roadmap for 3.0. We're looking to get it out… soon. This change just went out in the 3.0 beta 4. In the future we plan to road map publicly.


Proposed schema changes

Here are some suggested changes/additions!

Would it be possible to add some of the headers that are present in schema.yml - for example, OPEN PEER REVIEW, to the YAML files themselves? Furthermore, condensing down some of the white space (ie, between data fields that share the same reference url) might reduce the need to scroll so much. Each dsc could then be started with the name of the variable for disambiguation
"Peer review policy url (valid url)" -> "Open peer review policy url (valid url)"
Add url for preprint-citation
In the later part of the files, preprint urls follow the data fields, but in the early part, they preceed them. Suggest standardizing these
preprint url fields descriptions could be standardized as well
would be good to add "unknown" to most fields to signify that this information can't be found. If users don't look for info, they can leave the field null. But if they do look and can't find, they could use unknown. The absence of public information is also important...
From #25: split journals into two fields like journals and journals-subpolicies

@jpolka can you make a pull request with your edits to the schema. Do not merge it, but once i have the commit I can take it from there. This will be the first time we are changing the schema after there are already some annotations. Here's the computational workflow I'm envisioning:

Update the blank template data structure for each policy with existing annotations
Apply the data fields taken from RoMEO

This means that any changes to to the RoMEO fields will get overwritten.

would be good to add "unknown" to most fields to signify that this information can't be found

Sounds reasonable to me! not specified would be an alternative, although perhaps unknown is a bit more general?

Would it be possible to add some of the headers that are present in schema.yml - for example, OPEN PEER REVIEW, to the YAML files themselves? Furthermore, condensing down some of the white space (ie, between data fields that share the same reference url) might reduce the need to scroll so much.

I will see what's possible here given the treacherous documentation and attitude of ruamel.yaml.

Each dsc could then be started with the name of the variable for disambiguation

If you did this, then I we wouldn't have to worry about section headers, and I could easily remove the newlines (which took me a very long time to add, but if they are more damaging then helpful, we should remove).

@jpolka IIRC you also mentioned the idea of adding a general comments field somewhere?

If you did this, then I we wouldn't have to worry about section headers, and I could easily remove the newlines (which took me a very long time to add, but if they are more damaging then helpful, we should remove).

Can we use \n for new lines in descriptions?

As per the discussion on Gitter with @cameronblandford, it would be good to add the following changes:
(Please also suggest different tags for the field, @tonyR-H ?)
Can enum be used in a str inside a seq, @dhimmel ?
field
  type: seq
  desc: "Field - add all that apply. Add a new line for each. Start each line with two spaces, a dash, and one space, like this: '  - ' (biology/chemistry/physics/computer science/medicine/math/social science/humanities)"
  sequence:
    - type: str
      enum: [the above] #Not sure this will work -  http://www.kuwata-lab.com/kwalify/ruby/users-guide.01.html#schema

visible
  type: str
  desc: "Used by moderators to determine whether record is shown in frontend (yes/no) - do not edit
  enum: [yes, no]

you could make it so any policy or journal can have multiple subjects and make sure people are liberal with their tagging

Regarding the field (discipline / subject) field, I'm not sure its a good idea to add this a manual field. Curating each policy with fields is a lot of work. Instead, I'd suggest that we look up subjects based on ISSNs of the journals covered by each policy. I think we could do this perhaps via Crossref or Scopus.

would adding an optional contributors field help incentivize people / hold them accountable you think?

The website should link to the source YAML file on GitHub which has a blame view. We could also calculate contribution summaries based on the git history for each YAML file.

how about some sort of visible or available-on-website or filled-out field that admins can flip, so it can be up to moderators' discretion when something becomes available in the database?

Currently, at http://transpose.surge.sh I don't see actual data (I think it's showing mock data). An alternative to a visible field would be to only display fields that are not null. Under the current proposal would all journals default to "not visible" and would have to be toggled to visible? I'm not against the idea, just that I'd need a little more guidance before implementing it.

@dhimmel surge.sh only hosts static sites, so I haven't been able to provide updates to that deployment for any of the more recent changes. I'll put up something today. Only displaying fields that aren't null is already implemented in the dynamic version 👍 Thinking about it more now, I think it would be even better to display every policy regardless of how much has been filled out, and then have a link to the relevant file in the repo for each policy somewhere on the policy detail page, in a sort of "Contribute to this policy's collected information here" way.

This discussion sounds good!
Re fields - while I think a tag system would permit more flexibility (especially for interdisciplinary journal) than a system of categories, I completely agree with your point about using existing data where possible! It does seem like category-name from Crossref, which is apparently derived from Scopus, would be the way to go.
BTW I just noticed these little typos we might want to fix next time around...
# Publisher name from SHERAP/RoMEO. Do not edit
publisher:

# Policy heading from SHERAP/RoMEO. Do not edit
policy-heading:

From the editathon -

Need “before review” for which version of preprint is ok to post
Need a "conditional" or "other" option for peer review identities revealed/published and reports published #32

@jpolka can you make a PR that modifies the schema according to the edithon points? Then I can take it from there?

@dhimmel, absolutely, will do. However I am now also finding myself wanting a way to track:
Are [X] published? [mandatory/optional/conditional/no]

If optional or conditional, who decides? [some combination of: author/reviewer/editor]
If optional or conditional, when is the decision made? [at submission/after process is complete/other]
If optional or conditional, any other details? [free text]
Are they assigned a DOI?
DOI Type [Crossref peer review / other]
DOI ontology [for this object/ one DOI for a bundle of objects]
Are they open access? [no/yes/for some articles]
What license? [free text]

Where X is:

Peer review reports
Editorial decision letters
Author responses to reviewers
Previous versions of the manuscript
Reviewer identities

I think @tonyR-H has some other wishes as well.

If optional or conditional, who decides?

I don't see a kwalify option for conditional or dependent fields. Therefore, I don't think we can make the schema only allow setting "who decides" if the field is set to optional or conditional. However, we could always ignore the contents of "who decides" in those cases at a later stage. We can still have comments letting users know to fill in the field if optional or conditional is set.
The following three categories make sense with the above fields:

Peer review reports
Editorial decision letters
Author responses to reviewers

I am not sure what "Previous versions of the manuscript" means in this context and I don't think all of the fields above apply to "Reviewer identities".
Other comments: what is "DOI ontology"?  What do you mean by "DOI Type"? Is this for the DOI registrar agency, e.g. Crossref, Datacite, ...?

Thanks @dhimmel - I mean whether all the review files and kept together in one DOI or whether each reviewer report/author response/decision letter has its own DOI Crossref peer review DOI type.
Good point about just ignoring input for "who decides" when it is not applicable. I think this is a reasonable approach regardless of whether we switch platforms to something that permits conditional responses, as discussed in #36.


<EMAIL_ADDRESS>always returns null

Not sure when this problem started, but I tested quite a few different entities(custom zombies, etc.) that I spawn from script containers.  They used to work, but now only return 'null'.
Thanks,
Uledrith

Ok, I think this can be closed.  I was spawning entities from 'task' scripts and while the .scriptname tag did work on these, I'm guessing that is not by design.  I didn't know about actualy 'type: entity' script containers so I'm converting to those and they do work.  Sorry...
On a related note, I'm curious if there is a way to put an item in hands or offhand inside one of these entity script containers.  I know you can do equipment: li@i@boots|i@legs|i@chest|i@helmet.  I'm trying to avoid using commands (i.e. - equip ...) after I spawn the e@entity.
Thanks! Uledrith

Equipment has new slot for offhand now. I believe hand is inventory contents?


Appstudio update rhtap-demo-component-pspn

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-pspn-on-pull-request-
creating pipelinerun rhtap-demo-component-pspn-on-pull-request- in namespace rhtap-demo-dev-ncbn-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-pspn-on-pull-request-
creating pipelinerun rhtap-demo-component-pspn-on-pull-request- in namespace rhtap-demo-dev-uqcd-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Use of angle brackets around file names for include statements

Would you like to replace any double quotes by angle brackets around file names for include statements?

What can be the reason not to use double quotes in include statements?

I suggest to reconsider the consequences of the following wording from the section “16.2 Source file inclusion” in the standard specification for the programming language “C++”.
…
The named source file is searched for in an implementation-defined manner. If this search is not supported, or if the search fails, the directive is reprocessed as if it read

#include <h-char-sequence> new-line
…

How do you think about to avoid “a duplicated file search” here?
Would you like to restrict the searched directories for header files of your software?

I am not sure that I understand the concern about “a duplicated file search" because any TBB header file should be found (otherwise, something is broken) and I believe the preprocessor is quite smart to optimize the search.
In accordance with the provided link <> "is used for system header files" and "" "is used for header files of your own program". The TBB header files are not a part of system header files, so the mentioned file should be included with the "" syntax (because it is one of the TBB header files).

How often will this software library belong to “your own program” (as a required component)?
Do you expect that such a software library will be used from a storage location where it was installed?

There are different opinions about the handling of the involved implementation-defined behaviour.

Will header files be also searched outside the specified include directories if double quotes are used for the discussed preprocessor statement?
Is there a speed difference measurable if a file is not found there and the search will be retried with “the angle brackets inclusion method”?

How often will this software library belong to “your own program” (as a required component)?

The term “your own program”  is not defined. Personally, I understand it as "everything that is not C++ implementation", .i.e. any software that uses C++ but does not implement C++.

Do you expect that such a software library will be used from a storage location where it was installed?

#include "tbb_stddef.h" is inside tbb/task.h. I.e. the library header files includes some other header files from the same library. It expects that the including file resides in the same directory. If not, there is some installation issue and the library cannot be used.

Will header files be also searched outside the specified include directories if double quotes are used for the discussed preprocessor statement?

The document says that "It searches for a file named file first in the directory containing the current file", i.e. the place where the library expects other parts of itself. If the including file is not found in the current directory, the installation is broken.

Is there a speed difference measurable if a file is not found there and the search will be retried with “the angle brackets inclusion method”?

When we use "" we expect that file will be found in "extended directories" comparing to “the angle brackets inclusion method”. Since "extended directories" should be searched first, I'd not expect performance impact of “the angle brackets inclusion method” (because it is not applied).

… "It searches for a file named file first in the directory containing the current file", …

From which storage location should a library file be picked up preferably?

Known installation directory
Random (or temporary) software development status

If the including file is not found in the current directory, the installation is broken.

Can it be that software build parameters were intentionally selected in the way that questionable library files should not be found there?
Should the use of “extended inclusion directories” be occasionally (or even mostly) excluded?

Let's clarify the discussed environment and consider possible options. When we are speaking about the library we also speak about an application that uses this library. Therefore, we have two questions to consider:

How the application finds the library header files?
How the included library (if found) finds its other parts (other header files related to this library*)?

*I suppose that the library has its own structure and may rely on this structure. If some parts of the library are separated/rearranged I consider such installation as broken and such library cannot be used.
Let's consider possible options:

The application finds library with <> syntax. It means that the library is located in some "system" location. It means that library possibly can include its internal with <> because its internal also in "system" location.
The application finds library with "" syntax. It means that the library can be located in some "extended" locations. In that case library can fail to find its internals with the <> syntax.

From the library perspective it does not know how it is included; however, it knows its structure and may rely that the current directory is a part of the "extended" locations. Hence, the library can use the "" syntax for the both options when includes its internals.
Let's consider the following example that demonstrates the hazard of <> syntax:
$ find . -type f -exec echo -----{}----- \; -exec cat {} \;
-----./ext.h-----
#warning ext.h
-----./lib/ext.h-----
#warning lib/ext.h
-----./lib/lib.h-----
#include <ext.h> //   <> SYNTAX
#warning lib/lib.h
-----./main.cpp-----
#include <lib/lib.h>

I.e. the current directory contains two files (main.cpp and ext.h) and the lib subdirectory with two files (lib.h and ext.h). Let's suppose that lib.h is a public header of some library that includes its internal file ext.h. If lib.h uses the <> syntax it can find another ext.h that is located in another directory:
$ g++ -c main.cpp -I.
In file included from ./lib/lib.h:1,
                 from main.cpp:1:
./ext.h:1:2: warning: #warning ext.h [-Wcpp]
 #warning ext.h
  ^~~~~~~
In file included from main.cpp:1:
./lib/lib.h:2:2: warning: #warning lib/lib.h [-Wcpp]
 #warning lib/lib.h
  ^~~~~~~

Pay attention that lib/lib.h includes ./ext.h not ./lib/ext.h.
Let's try the same example but lib.h uses the "" syntax:
$ find . -type f -exec echo -----{}----- \; -exec cat {} \;
-----./ext.h-----
#warning ext.h
-----./lib/ext.h-----
#warning lib/ext.h
-----./lib/lib.h-----
#include "ext.h" //   "" SYNTAX
#warning lib/lib.h
-----./main.cpp-----
#include <lib/lib.h>
$ g++ -c main.cpp -I.
In file included from ./lib/lib.h:1,
                 from main.cpp:1:
./lib/ext.h:1:2: warning: #warning lib/ext.h [-Wcpp]
 #warning lib/ext.h
  ^~~~~~~
In file included from main.cpp:1:
./lib/lib.h:2:2: warning: #warning lib/lib.h [-Wcpp]
 #warning lib/lib.h
  ^~~~~~~

Now, lib.h includes the correct ext.h.

If some parts of the library are separated/rearranged I consider such installation as broken and such library cannot be used.

I suggest to reconsider this kind of wording. - If specific details are changed, the corresponding software components should be adjusted as usual.

The application finds library with "" syntax. It means that the library can be located in some "extended" locations. In that case library can fail to find its internals with the <> syntax.

I propose to reconsider the interpretation of “extended file inclusion locations”.
The possibility for search failures exists for both inclusion methods generally.
Software build parameters can influence the selection of preferred storage locations for needed source files. Would you like to give any setting the priority then by a specific build option?

Let's consider the following example that demonstrates the hazard of <> syntax:

There are development and system configuration challenges to consider as usual.

Is an installed library version used finally?
Do you use a library variant as a simple source code bundle?
Which efforts do you try to separate library files from source files of a program in an application project?

I would appreciate another constructive feedback for this issue.

@elfring, this issue is closed because it is highly unlikely that the current practice will change.
For a general discussion about the preferred syntax for a C++ library to include its own, 3rd party, and system header files, it is better to use some public forum or mailing list. We will not continue the discussion here.


FR: Reference values of annotations in chart.yaml to a property included in the values.yaml file

Hello people,
I want to deploy more and more applications via helm. This requires a lot of meta information for the complete product ecosystem which is generated generically. Static information are stored in the values.yaml file. Information of the product ecosystem is stored in system.yaml files. The application will be deployed by referencing booth yaml files. The information will be merged together: helm install my-app repo/my-app --values values.yaml --values system.yaml.
The problem is, that relevant meta information about the product ecosystem does not belong to a deployment, daemon set or other resource. The information simply belongs to the release of the chart.
Now I tried to reference the value of an annotation to the corresponding value in system.yaml. Unfortunately helm does not interpret the template format at this point.
apiVersion: v2
name: my-application
description: My great application
type: application
version: 0.1.0
appVersion: 1.5.7

annotations:
  app.company.local/service-version: "{{ .Values.service.version }}"

I needed this function to be able to store meta information generically. Are there already plans for this? Otherwise, this would be a very useful feature.

Hi @volker-raschek. We've heard this idea before in different forms, however I haven't seen anyone interested in picking up this work. For example https://github.com/helm/helm/issues/10562, https://github.com/helm/helm/issues/4782, https://github.com/helm/helm/issues/4275, and https://github.com/helm/helm/issues/10022.
The first step to propose a new feature would be to write a HIP.
We intend HIPs to be the primary mechanisms for proposing major new features, for collecting community input on an issue, and for documenting the design decisions that have gone into the project. The proposal author is responsible for building consensus within the community and documenting dissenting opinions.
See Proposing an Idea for more information on this process.

We are interested in the same feature.
Our use case is getting annotations as proposed in https://github.com/helm/community/blob/main/hips/hip-0015.md but it doesn't support dynamic version setting:
annotations:
  example.com/images: |-
    - name: image-name
      image: image-name: {{ .Chart.AppVersion }}

We would like to use templating or convenient override like:
helm package ${CHART_PATH} --app-version ${VERSION}


fix sample sheet create columns

There is a problem when creating the sample sheet with the function create-samplesheet.r
It creates two columns named sentrix_row and sentrix_col that after when using read-samplesheet.r do not work.

I am getting this error: "Error: $ operator is invalid for atomic vectors"
when running this line: "pcs <- meffil.methylation.pcs(beta, verbose=T)"
what can I do to fix this?

This is due to a typo in a recent change I made.  Just fixed this yesterday, so you should just be able to reinstall from github.

From: Lidija0710 @.>
Sent: 16 November 2021 9:20 AM
To: perishky/meffil @.>
Cc: Subscribed @.***>
Subject: Re: [perishky/meffil] fix sample sheet create columns (#21)
I am getting this error: "Error: $ operator is invalid for atomic vectors"
when running this line: "pcs <- meffil.methylation.pcs(beta, verbose=T)"
what can I do to fix this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/perishky/meffil/pull/21#issuecomment-970076938, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACSRUULJKBD6XPNJ2ZLZOWTUMIO6XANCNFSM4JBPGYPA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Option to disable modulepreload

Clear and concise description of the problem
Hi
The current version of Chrome is pretty aggressive with the preload and modulepreload directive : it will download the JS first, then get the CSS files. Because CSS files are blocking page rendering, the preload directive will delay rendering.
You can use webpagetest to see the effect

The vertical green bar is the moment the page is displayed. Chrome rushes on the preloaded JS files first
then request the CSS files

In this test, it needs an additional few hundred milliseconds after the last blocking CSS file to calculate the page layout and displays it. If the JS files were not preloaded, the CSS would be downloaded sooner, and so the page first paint. We use SSR, so the first paint is good enough for the user to see and we can allow the user to see the page without JS for a few seconds.
Safari iOS and Firefox behave less aggressively and will just get the CSS first and the rest later.

Suggested solution
I would need to deactivate manually the modulepreload on the JS files, at least as long as Chrome has this behaviour.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Here are more statistics, on a global level this time :
https://almanac.httparchive.org/en/2021/resource-hints#correlation-with-core-web-vitals
The more you preload, the less your performance is good, as measured by the Core Web Vitals from the Chrome Report User eXperience (used by Google Search to rank websites).

Please really consider being able to turn off preload of JS in Vite.

@jpvincent do you know the historical rationale why <link type="modulepreload"> has been chosen?

Tracked down the generation of <link> tags here - if someone wants to explore the code:
https://github.com/vitejs/vite/blob/46b862a6880a6e83d1e0e451376177c0e5cab0ba/packages/vite/src/node/plugins/html.ts#L479

For understandting the issue more deeply, here is a historical Google Chrome article on modulepreload.

Here is my writeup how to transform generated HTML in SvelteKit hooks using cheetah. By changing <link type=modulepreload> to the end of <body> <script> tags I managed to reduce the page loading time significantly (4.0 -> 2.5 to FCP on mobile).

No, I dont have any rationale, my guess being that the team chose this because it is said to be a good practice. But here we need more nuance, as this default does not work for everyone.
Thank for you research in the Vite code and your proposed solution, I hope to test it in a few weeks and see if it fits.

The time between the first .js file and the last .css file on https://www.radiofrance.fr/ starting to load is 20ms on a page that takes 2s total. So we're talking about a 1% speedup at the very most. There's probably much bigger wins to be had.
I'm surprised there's 50 JS files being loaded. That 20ms would likely be smaller if there were fewer files. I would generally expect things not to be so granularly chunked, but it's hard to advise on that without access to the source.
There's some weird stuff in these files. E.g. it looks like https://www.radiofrance.fr/client/chunks/index-83a12101.js includes the package.json:
m={"babel-cli":"^6.26.0","babel-core":"^6.1.2","babel-eslint":"^8.0.1","babel-loader":"7.1.5","babel-plugin-transform-object-rest-spread":"^6.26.0","babel-plugin-transform-runtime":"^6.23.0","babel-polyfill":"^6.2.0","babel-preset-env":"^1.6.0","babel-register":"^6.22.0",chai:"^4.1.1",eslint:"^5.4.0","eslint-config-airbnb":"^17.1.0","eslint-loader":"^2.0.0","eslint-plugin-import":"^2.2.0","eslint-plugin-jsx-a11y":"^6.0.2","eslint-plugin-mocha":"^5.0.0","eslint-plugin-react":"^7.11.1",jsdom:"^12.0.0",mocha:"^5.0.5","pre-commit":"^1.2.2",replace:"^1.0.0","semver-regex":"^3.1.0",sinon:"^6.1.5",webpack:"^4.4.0","webpack-cli":"^3.1.0"},_={build:"webpack --display-modules",prepublishOnly:"babel ./src -d ./module && npm run replace",replace:'replace "rfplayer/src" "rfplayer/module" ./module -r --include="*.js"',test:"mocha",lint:"eslint ./src<EMAIL_ADDRESS>
As does https://www.radiofrance.fr/client/chunks/index-3715f060.js:
we={"babel-cli":"^6.26.0","babel-core":"^6.1.2","babel-eslint":"^8.0.1","babel-loader":"7.1.4","babel-plugin-rewire":"^1.1.0","babel-plugin-transform-object-rest-spread":"^6.26.0","babel-plugin-transform-runtime":"^6.23.0","babel-polyfill":"^6.2.0","babel-preset-env":"^1.6.0","babel-register":"^6.22.0",chai:"^4.1.1","chai-as-promised":"^7.1.1",eslint:"^4.9.0","eslint-config-airbnb":"^16.1.0","eslint-loader":"^2.0.0","eslint-plugin-import":"^2.2.0","eslint-plugin-jsx-a11y":"^6.0.2","eslint-plugin-mocha":"^5.0.0","eslint-plugin-react":"^7.4.0","event-emitter":"^0.3.5",jsdom:"^11.2.0",mocha:"^5.0.5","pre-commit":"^1.2.2",replace:"^1.0.0",rfplayer:"^3.0.0-alpha.7","semver-regex":"^3.1.1",sinon:"^4.5.0",webpack:"^4.5.0","webpack-cli":"^3.1.2"},ke={build:"webpack --display-modules",prepublishOnly:"babel ./src -d ./module && npm run replace",replace:'replace "rfplayer/src" "rfplayer/module" ./module -r --include="*.js"',test:"mocha",lint:"eslint ./src ./test"},Re="src/index.js",Le={type:"git",url:"git@gitlab.dev.dnm.radiofrance.fr/lecteur-commun/rfplayer-plugin-smarttag.git"},Ne="Radio France",Ce="CECILL-B",$e="https://nexus-production.dnm.radiofrance.fr/repository/npm-group/rfplayer-plugin-smarttag/-/rfplayer-plugin-smarttag-3.1.3.tgz",je="sha512-UT65/9LRJhQfgk65bNR3+acFsOFsZaus8KdFac80WDAnhh7mNt6Bq0saAditYTGs2/ZMH48iE+zgJsn2FFScxw==",Ue="rfplayer-plugin-smarttag@3.1.3";var xe={name:Pe,version:Ie,description:Ae,"rfplayer-plugin":{minimumCoreVersion:"3.0.0"},dependencies:Oe,module:Me,devDependencies:we,scripts:ke,main:Re,repository:Le,author:Ne,license:Ce,"pre-commit":["lint","test"],_resolved:$e,_integrity:je,_from:Ue}

You'd probably get just as big of a win by not including that stuff. I'm also wondering why babel and webpack would be used in a Vite project...
I also see Microsoft copyright notices and am not sure they really need to be included?
/*! *****************************************************************************
Copyright (c) Microsoft Corporation.

Permission to use, copy, modify, and/or distribute this software for any
purpose with or without fee is hereby granted.

 IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT,
INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR
OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
PERFORMANCE OF THIS SOFTWARE.
***************************************************************************** */

And then there's fun stuff like https://api.privacy-center.org/v1/locations/current?fields[]=country_code&fields[]=region_code which takes over 300ms to load a very tiny response of {"country_code":"US","region_code":"CA"}
I'm not sure an option to disable preloading would really be worth the added complexity especially when there's lower hanging fruit...

I'm not sure an option to disable preloading would really be worth the added complexity especially when there's lower hanging fruit...

Is the complexity of this issue of:

Changing <link type="modulepreload"> to <script defer>
Adding a new option for this
Adding a new unit test for the new option
Documenting the option

If you are open to new patches I am happy to get it done, but if the discussion is going to be stalled whether the issue is real or not, then it is going to be much more nuanced conversation.

We should properly justify adding each new option. Your list isn't complete. The cost of a new feature isn't the time it will take to add it, but the added complexity to the code base and how it will interact later with other features that will impact the long-term maintenance of the project.

This one reminded me of an old issue from vite1 days
https://github.com/vitejs/vite/issues/881
modulepreload fequiring a polyfill in most non-chromium browsers makes it a less than ideal candidate imho. If script tyoe module are correctly defered like the spec suggests that would be a clean alternative, no?

We need to disable it as well, but for different reasons. Our project is enormous and preloading every single module like this is not warranted.

The time between the first .js file and the last .css file on https://www.radiofrance.fr/ starting to load is 20ms on a page that takes 2s total. So we're talking about a 1% speedup at the very most. There's probably much bigger wins to be had.

hi @benmccann . That's HTTP/2 at play here : the 20 ms you are talking about is the moment Chrome is queuing the file (white color). Before that, there is a 100ms of wait. Take a look at the darker green color : that is the actual transit on the network.
I should add those informations : CSS total size is 16 Kb, latency is 70 ms and bandwith 12 Mbp/s. It takes 400 ms to get the last byte of those 16Kb of CSS on a fast network, because the network is busy getting hundreds of Kb of JS + 58 Kb of fonts before (all preloaded).
Here is a link to original test from the screenshot : https://www.webpagetest.org/result/210928_BiDcSQ_200daba0ee17c95a6eacff5d7518536e/5/breakdown/ so we can discuss further.
Delaying CSS delays First Paint, but on Chrome it also delays the start of the download of the images, because they chose to wait for the layout information before starting getting them.

@benmccann : thank you for taking a look at what's inside the JS files or about the latency on the server. The frontend is based on SvelteKit.
However the point here is that whatever is in those JS files, they should not block the rendering at all, respecting the philosophy of defer . The preload directive as currently used by Chrome is de facto a problem. They give no indication of reverting this behaviour to get back to what Safari and Firefox are still doing.
We should be able to opt-out of the preload "optimization" if we see it does not serve us.

I'm afraid I have no idea what I'm supposed to be looking at in the link you posted. Is it possible to use Chrome screenshots instead of webpagetest.org screenshots to discuss this?

Before that, there is a 100ms of wait.

I don't see that at all or am unsure what you're referring to. I just went to https://radiofrance.fr/ now. The JS started downloading at 830.35ms

The CSS started downloading at 854.03ms

That's a difference of less than 25ms.

the network is busy getting hundreds of Kb of JS + 58 Kb of fonts

So you're also suggesting the network is being saturated? That's a different claim than the one made before that CSS files don't start downloading until later. That's easier for me to believe, but there's been no clear evidence of that shared here.

Changing <link type="modulepreload"> to <script defer>
modulepreload requiring a polyfill in most non-chromium browsers makes it a less than ideal candidate imho. If script type module are correctly defered like the spec suggests that would be a clean alternative, no?

You can't just swap one for another. modulepreload only fetches, but doesn't execute the script whereas <script defer> does, so they're not synonyms for each other. What you could do is simply remove the modulepreload tags. This will mean that the JS is only downloaded when encountered. If script A depends on B depends on C depends on D, you will end up needing to make many round trips instead of getting everything up front

We should be able to opt-out of the preload "optimization" if we see it does not serve us.

We try to have best practices enabled by default, as few options as possible, and clear explanations of what options do and when to change them. When considering whether to add a new option it must be clear that it's helpful, that it shouldn't just be made the default for all cases, and if it is to be added then there needs to be clear documentation regarding the tradeoffs of setting it.

I found an issue in SvelteKit affecting the download order while investigating this: https://github.com/sveltejs/kit/commit/d138efe21692f5925f1e89afc0a33f42d6a1a711
Can we consider this issue as solved now?

I figured out how to disable it, you have to remove __vitePreload( ) in the index.js file. To be safe, I'm doing it dynamically using an IIS managed handler that would replaces all javascript files on the fly with the following:
jsFile = Replace(jsFile, "__vitePreload=", "__vitePreload=(e,a)=>{ return e(); },__oldVitePreload=");

and all html files on the fly with the following:
html = Regex.Replace(html, "^\s*<link rel=""modulepreload"".*?>$", "", RegexOptions.IgnoreCase | RegexOptions.Multiline);

I also set the cache-control header with max-age and immutable, such as this:
context.Response.Headers.Add("Cache-Control", "private, immutable, max-age=604800")

and now my importmap is working, everything is caching properly, and both methods to the preload non-sense are gone.

I found an issue in SvelteKit affecting the download order while investigating this: sveltejs/kit@d138efe
Can we consider this issue as solved now?

You can consider it solved when you add a way to properly disable your preload nonsense.

I found an issue in SvelteKit affecting the download order while investigating this: sveltejs/kit@d138efe
Can we consider this issue as solved now?

In theory it's fixing the original problem I had : First Paint being delayed by JS preloads in Chrome.
I'm waiting for a deploy of this patch and test it live on a page with real browsers.
The preloads could still slow down other requests like the main visible image, so having an option to disable preload always seems like a good idea to me, but let's wait for a deploy.

It should only be considered resolved when there is way to properly disable both preload features.

As mentioned earlier, a justification would need to be provided for adding a new option. Happy to consider, but we would need a detailed explanation of why it's required.

You have to remember that  does not work with an importmap [yet], but import...from and import( ) do work with an importmap

It might be better to file a new issue if this is the desired reason. Almost all the discussion here is about @jpvincent's issue, which is quite different. Please don't assume prior knowledge of importmaps when discussing since it's a new feature I haven't seen before

I'm waiting for a deploy of this patch and test it live on a page with real browsers

As a heads up, a new version of SvelteKit with the patch was released a few hours ago

The preloads could still slow down other requests like the main visible image, so having an option to disable preload always seems like a good idea to me,

Another alternative in this scenario would be to add a preload for the main image appearing higher in the document


Some videos won't download from Elonet

Checklist

[X] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2020.03.24
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE
C:\Programs\download>youtube-dl -v "https://elonet.finna.fi/Record/kavi.elonet_elokuva_107867"
[debug] System config: []
[debug] User config: ['-f', 'bestvideo[height<=?1080]+bestaudio', '--merge-output-format', 'mkv']
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://elonet.finna.fi/Record/kavi.elonet_elokuva_107867']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg git-2020-04-03-52523b6
[debug] Proxy map: {}
[generic] kavi: Requesting header
WARNING: Falling back on generic information extractor.
[generic] kavi: Downloading webpage
[generic] kavi: Extracting information
ERROR: Unsupported URL: https://elonet.finna.fi/Record/kavi.elonet_elokuva_107867
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\generic.py", line 3352, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://elonet.finna.fi/Record/kavi.elonet_elokuva_107867

Description

WRITE DESCRIPTION HERE
Most of the videos from Elonet download just fine with the Vimeo plugin (for example: https://elonet.finna.fi/Record/kavi.elonet_elokuva_123238), but some won't at all (for example: https://elonet.finna.fi/Record/kavi.elonet_elokuva_107867). There is a similar issue in the closed issues (https://github.com/ytdl-org/youtube-dl/issues/23304), but I can't comment on there.

I can confirm this. In fact, I can't get youtube-dl to work with any videos on Elonet. It used to work so it looks like Elonet changed something.


Handle expiries beyond the year 9999 by truncating them rather than blowing up

Fixes #28

https://github.com/mihu/redis-rdb-tools/commit/cd3b6542037897d4b8b913732e2b4006269b43ad

this was already merged in by 3470263b725ce651894f1331bed8133f261a42fc


[Critical] CommandAPI breaks with latest Spigot update

Describe the bug
After updating spigot to it's latest 1.13.2 version, every API command I try to run throws a NoSuchMethodException. Looks like spigot changed something you're working with?
Screenshots
Spigot version:

Console ouput:

(Here I'm using the /random command I made, but it throws the error with ANY commandAPI command)
I'm using the unreleased 1.8 CommandAPI, because it has some bug fixes I really need.
This bug however also occurs on 1.7.2.

I've fixed the issue for the unreleased version of 1.8. You can grab a copy of a pre-built unreleased 1.8 build here:
https://github.com/JorelAli/1.13-Command-API/tree/master/1.13CommandAPI/unreleased builds

Thanks for fixing this so fast 👍
Also I suggest posting a new release soon, as the latest release doesn't work on the latest spigot version.

I agree...
It's been difficult lately - the master repo is in a bit of a state and of course, it's been the Christmas period and everyone's busy. On top of that, the 1.8 documentation has yet to be written and I still have bugs and areas I have yet to research.


Added flag of Poland

With SVG asset.

Can we get SVG to display correctly in Github's README.md?


Changed AT to PRED in ion/services/dm

Reviewed and made some changes. will make a new PR from ooici-dm


Question on the StyleGAN version used

Hello,
In the "README.md" file, the url links which are linked to the StyleGAN point to the Github of the "StyleGAN V1", but on the Google Drive, some files seem to be rather linked to the "StyleGAN V2" (ex. "Datasetgan_release_checkpoints/styegan_pretrain/stylegan2-ffhq-config-f.pt" OR "datasetGAN_data_release/training_latent/car20/latent_stylegan2_encoder.npy").
The DatasetGAN can probably work with any version of StyleGAN, but for the research and code of this Github, which version of StyleGAN was used ?

ignore the latent_stylegan2_encoder.npy, please use stylegan1 to reproduce results in paper.


Use Common Sanitizers

FYI @billwert @samvaity
If you need a new proxy version with changes. Sing out, we'll just update the target_proxy_version.txt until we're happy with the outcome.
Follow up for search SDK tests: https://github.com/Azure/azure-sdk-for-java/issues/40141

/azp run java - core - ci

/azp run java - core

/azp run java - core

/azp run java - core

/azp run java - core

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-core-test

@samvaity After testing it seems that currently the logic of this PR does not allow for removal of sanitizers during record, but the javadoc suggests this should be possible.

/check-enforcer override

/check-enforcer override


Conversion script bugfixes

Updates the NeoX-to-HF conversion utilities to fix the following problems:

#1129 tweaks the default is_pipe_parallel behavior s.t. PP=1 models no longer are trained using PipelineModules, since MoE is not PP-compatible. This affects the save format of such checkpoints, so bringing the conversion script is_pipe_parallel detection logic in line with this.
I found a bug where we would get KeyError: no key 'final_linear.weight' could be found when trying to convert a model with tied input/output embeddings. So adjusted the script to now save input embeds into the LM head state when converting a tied embedding model.

I think it would be more desirable to preserve the PP=1 -> PipelineModule with one pipeline stage behavior and have pinged the PR creator on Discord to discuss.

should #1214 get closed and pulled into this?

@dmahan93 yep!
I agree with Stella that going back to the behavior of pp=1 operating as a PipelineModule is desirable though.


Rename Internal namespace to ToolKitInternal for some conformance issues

https://github.com/microsoft/DirectXTK/issues/451


how to clean the referenceLine?

Thank you for creating this project. I use some code in my project to create mpr.
But there are some problems, I need your help.
1.When I first generate mpr, the axial referenceLine does not exist from the beginning, only when I click on point。 I want to know how can I do so that it will exist from beginning
2.When I back to the another page then enter the mpr page to rebuild the mpr, the referenceLine didn't clean, I want to know how to clean it when destroyed
3.About how to generate mpr. I'd appreciate hearing them if you have any reading material suggestions.

when I click mpr button, create mpr

when I click back button still click mpr button, there was two referenceLine on each side

👋 This was really just a proof of concept I played around with. For series of any substantial size, or for true free-form rotation, it became pretty clear that this approach would not be performant enough.
Your best bet is to look at vtk.js, ami.js or three.js. For special cases where you want to leverage cornerstone tools for annotations, you could pass back your current slice with changes applied. There is also some work being done to create a more generic SVG Widget Manager that could make it possible to render tool's using a shared API across cornerstone-tools and vtk.js
Unfortunately, that's all the information I have for you. I don't have much bandwidth to allocate to this. If you reach out to my current employer, Radical Imaging, you could contract me/us out for services to assist with problems like this.

Thank you for your quick reply, your suggestion I will consider


python37Packages.identify: 1.4.3 -> 1.4.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-identify/versions.
meta.description for python37Packages.identify is: '"File identification library for Python"'.
meta.homepage for python37Packages.identify is: '"https://github.com/chriskuehl/identify"

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.4.5 with grep in /nix/store/6jb61zripar9x7a20lrckfhcx5dr9i1s-python3.7-identify-1.4.5
directory tree listing: https://gist.github.com/fab254ed70901d6d6571547f8ef3fab8
du listing: https://gist.github.com/f29bc076d32a488bdf1b61a95c1d9dc9

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)
3 package rebuild(s)
3 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
gitAndTools.pre-commit
python27Packages.identify
python37Packages.identify

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6jb61zripar9x7a20lrckfhcx5dr9i1s-python3.7-identify-1.4.5 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python37Packages.identify https://github.com/r-ryantm/nixpkgs/archive/af03dd20756af42ec8429e31d03a8125925ff869.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6jb61zripar9x7a20lrckfhcx5dr9i1s-python3.7-identify-1.4.5
ls -la /nix/store/6jb61zripar9x7a20lrckfhcx5dr9i1s-python3.7-identify-1.4.5/bin

@GrahamcOfBorg build gitAndTools.pre-commit


Login not working

@iliyaZelenko
I tested your repo in an attempt to get laravel passport + nuxt working. I am trying to get end-to-end login and a full oauth flow (et access token, use access token to call an API) working.
I went through the installation steps here ttps://iliyazelenko.github.io/laravel-nuxt/features/
but was not successful. Login is not working.
This is what i see in the console:
Access to XMLHttpRequest at 'http://localhost:8000/api/auth/user' from origin 'http://localhost:3000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
 Should help.
I also assume that SPA mode works fine, as frontend and backend have the same origin (host).

@iliyaZelenko just ran
$ php artisan vendor:publish --provider="Barryvdh\Cors\ServiceProvider"
and the CORS error went away but I am still seeing a 500 server error in console when I try to log in. See:
https://1ce.org/1#S10MVvISH

It's good that there is one less problem.
I would like to get more information about the response from the server
(response body), there should be a message.
пт, 30 авг. 2019 г. в 11:54, KP<EMAIL_ADDRESS>
@iliyaZelenko https://github.com/iliyaZelenko just ran
$ php artisan vendor:publish --provider="Barryvdh\Cors\ServiceProvider"
and the CORS error went away but I am still seeing a 500 server error in
console when I try to log in. See:
https://1ce.org/1#S10MVvISH
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iliyaZelenko/laravel-nuxt/issues/5?email_source=notifications&email_token=ADD67ROXNQYBW7FRH2KW4TTQHDN5PA5CNFSM4ISJMLY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5RBBTY#issuecomment-526520527,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADD67RI6VNKVNWBHUPGPIL3QHDN5PANCNFSM4ISJMLYQ
.

Here's the message and the stack trace
Request URL: http://localhost:8000/api/auth/login
Request Method: POST
Status Code: 500 Internal Server Error
Remote Address: <IP_ADDRESS>:8000
Referrer Policy: no-referrer-when-downgrade
Access-Control-Allow-Origin: http://localhost:3000
Cache-Control: no-cache, private
Connection: close
Content-Type: application/json
Date: Fri, 30 Aug 2019 09:34:22 GMT, Fri, 30 Aug 2019 09:34:22 GMT
Host: localhost:8000
Server-Timing: app; dur=115.83995819092; desc="Application", timeline-event-total; dur=117.09880828857; desc="Total execution time.", timeline-event-initialisation; dur=7.7898502349854; desc="Application initialisation.", timeline-event-boot; dur=3.3080577850342; desc="Framework booting.", timeline-event-run; dur=109.30895805359; desc="Framework running."
 You need to find a workaround I think.
пт, 30 авг. 2019 г. в 12:35, KP<EMAIL_ADDRESS>
Here's the message and the stack trace
Request URL: http://localhost:8000/api/auth/login
Request Method: POST
Status Code: 500 Internal Server Error
Remote Address: <IP_ADDRESS>:8000
Referrer Policy: no-referrer-when-downgrade
Access-Control-Allow-Origin: http://localhost:3000
Cache-Control: no-cache, private
Connection: close
Content-Type: application/json
Date: Fri, 30 Aug 2019 09:34:22 GMT, Fri, 30 Aug 2019 09:34:22 GMT
Host: localhost:8000
Server-Timing: app; dur=115.83995819092; desc="Application", timeline-event-total; dur=117.09880828857; desc="Total execution time.", timeline-event-initialisation; dur=7.7898502349854; desc="Application initialisation.", timeline-event-boot; dur=3.3080577850342; desc="Framework booting.", timeline-event-run; dur=109.30895805359; desc="Framework running."

Reply to this email directly, view it on GitHub
https://github.com/iliyaZelenko/laravel-nuxt/issues/5?email_source=notifications&email_token=ADD67RJWMRYXUCNO7SGJ2YTQHDSUVA5CNFSM4ISJMLY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5RERQY#issuecomment-526534851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADD67RJL2F7EY5TVSQ7MLYLQHDSUVANCNFSM4ISJMLYQ
.

@iliyaZelenko I'm happy to do a test for you when this is fixed

@iliyaZelenko i am actually using the default config, not the latest version of laravel


invalid characters error using create-react-native-app

Description
When running create-react-native-app an error is displayed: An unexpected error occurred: "https://registry.yarnpkg.com/@expo%2Fbunyan: The header content contains invalid characters".
Expected Behavior
Expected successful creation of new application.
Observed Behavior
Here is the transcript:
$ create-react-native-app AwesomeProject
Creating a new React Native app in /Users/bbibeault/Dropbox/Development/projects/native/AwesomeProject.

Using package manager as yarnpkg with yarn interface.
Installing packages. This might take a couple minutes.
Installing react-native-scripts...

yarn add v0.18.1
info No lockfile found.
[1/4] 🔍  Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@expo%2fbunyan: The header content contains invalid characters".
info If you think this is a bug, please open a bug report with the information provided in "/Users/bbibeault/Dropbox/Development/projects/native/AwesomeProject/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.
`yarnpkg add --dev --exact --ignore-optional react-native-scripts` failed```

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts:
<EMAIL_ADDRESS>/Users/bbibeault/Dropbox/Development/projects/native/AwesomeProject
└── (empty)

npm ERR! code 1

npm ls react-native:
<EMAIL_ADDRESS>/Users/bbibeault/Dropbox/Development/projects/native/AwesomeProject
└── (empty)

npm ERR! code 1

npm ls expo:
<EMAIL_ADDRESS>/Users/bbibeault/Dropbox/Development/projects/native/AwesomeProject
└── (empty)

npm ERR! code 1* node -v:

npm -v:
3.10.8

yarn --version:
0.18.1

watchman version:
0.1.8

Also specify:

Operating system:
Mac OS X Sierra

Phone/emulator/simulator & version:
none

Reproducible Demo
Obviously nothing to post here yet.

I think yarn 0.18.1 is like a year old, when it was still a wee baby of an OSS project. If you update to latest yarn (1.3.2) that will fix the problem. Sorry this is not a CRNA bug, it is a bug in the package manager, so I am closing the issue


Allow AL internal UniqueID to be used as attribute

If configured, map AL internal unique id as an attribute to be used
by satosa, microservices and other plugins.

One line was missing from the commit of @jkakavas. As this is just a trivial oversight, I prefer not to open a merge request to his tree but paste the solution here. Feel free to apply it or wait for him to update this pull request.
index ad4202e..b437129 100644
--- a/src/satosa/micro_services/account_linking.py
+++ b/src/satosa/micro_services/account_linking.py
@@ -32,6 +32,7 @@ class AccountLinking(ResponseMicroService):
         self.redirect_url = config["redirect_url"]
         self.signing_key = RSAKey(key=rsa_load(config["sign_key"]), use="sig", alg="RS256")
         self.endpoint = "/handle_account_linking"
+        self.config = config
         logger.info("Account linking is active")

     def _handle_al_response(self, context):```

LGTM


Keep minSdkVersion lower for 5.x

Is your feature request related to a problem? Please describe.
We can see that for version 5.x, minSdkVersion is bumped to 24. This means (as far as we can see) that we won't be able to update, as the minSdkVersion of our app is currently 23. This means we either have to bump minSdkVersion of our app to 24 or not be able to update to 5.x (and possibly miss bugfixes and new features).
There might be reasons to why this bump was needed, maybe you'll be able to enlighten us in the reasoning? I tried to search a bit in the issues and commits, but couldn't see anything specific.
Describe the solution you'd like
The best solution for us at the moment would be to lower minSdkVersion to at least 23 for version 5.x, as we would also like to start using that.
Describe alternatives you've considered
I'm not sure what support plans there are for 4.x, but if minSdkVersion cannot be lowered any further,

Thanks for the feedback @Anigif! The reason for bumping the min SDK version was that we adopted a few Java 8 features (in particular lambda functions) that seemed not to be supported in the API 23.
We will investigate again if there is a way to make this work even with API 23. If you have some suggestion how to do that, please shout!

Hi @matus-tomlein, I'm a colleague of @Anigif.
I'm not sure if this could solve the issue, but have you considered using desugaring? As far as I understand, desugaring allows you to use certain Java 8 features without restrictions on API level, so it might be something to look into.
Although I seem to be able to build the next branch when I set minSdkVersion back to 21 (as it was previously) without adding desugaring. Not sure if there's something I'm missing here, but it doesn't seem like a necessary bump? I changed it in all three modules and can build and run snowplow-demo-java and snowplow-demo-kotlin, and all the tests in snowplow-android-tracker are passing

Thanks @hakonschia for the tip, that's very useful! I have managed to get it to work on SDK version 23 when using desugaring.
The next branch builds even without desugaring but the problems for me arised when I tried to run the snowplow-demo-java and track some events. The demo is creating a GlobalContextsConfiguration which on the background now uses the Java 8 lambda functions.
We'll make a PR to roll back bumping the minSdkVersion.

Great to hear! Thanks for the quick replies

Hi @matus-tomlein, 5.0.0-beta.2 looks good! Builds and runs for us without any issues

Resolved in v5 stable release.


Review tech skills transfer section

Description
Definition of done

[ ] Schedule meeting to review Tech Skills Transfer section Mural
[ ]  Self-assess and update the Mural if need be
[ ] Publish the Mural

Attachments
https://app.mural.co/t/gsa6/m/gsa6/1570571286978/24b8c16b07a044353a1ae5fc723716dbac0ae6f4

Here are some updates on this card:

I scheduled the assessment with Imran, we got about 2/3rds done in the first session, and will be wrapping it up today after pairing.

We need to schedule additional time with darren. The assessment is fairly long and will likely take over an hour, so finding a spot has been tough.

@Jacklynn I will need to pair with you to learn how to update the Mural 😄


WG21: More pseudonyms

Do you want to require that specific capitalization?

Not necessarily, but I don't really see people capitalizing them differently either. @brycelelbach WDYT?


need help with configuring a splitter

Hey everyone,
I have been trying to use DOMSplitter to crawl some data from a page but it won't work and I am not sure I know how to talk to DOMSPlitter. So I figured I can explain my problem and what I am trying and have someone review my assumptions and correct me.
So, I have a page that contains 8 tables. I need to only crawl data from tables 1, 2, 3, 4, 8. I will have to exclude tables 5, 6, 7. For this purpose, I wrote the following importer configuration with comments accompanying each instruction indicating what I expect from each.
config.txt
I hope someone can spot the flaw on my understanding of splitter and help me sort this issue out. Thanks in advance!

I got it. I was working in postParseHandlers, and was expecting preParseHandlers' behavior.
Rubber duck debugging.


Add pofile build

From #55

Fixes string formating from last PR
Add static method on GitHubUtils to build pofile
Add build pofile before mkdocs build

We will be keeping this PR temporaly, but it should not be merged.

I think we have mixed some concepts in a wrong way...
The build of documentation it should be independent of github and the build of documentation have to have all the actions needed: install requirements, compile mo files, etc.

@ecarreras
We will be keeping this PR temporaly, but it should not be merged (the last PR with 'garbage').
Following PRs will clean a bit the repo :+1:


Define Calendar and User table

Only accounts for lectures, not sections

@pablofgaeta Clarification on this ticket please

@tanmaymittal This is for us to decide on the structure of our Calendar and User tables in the database. It'll just end up being a sequelize model when we implement it


Improve Performance of SplitBamByCellBarcode

Purpose

SplitBam is slow. We want to make it faster.
Changes

Completely rewrote SplitBam to take advantage of parallelization. The tool now takes a list of bams as input: SplitBam -b bam1.bam bam2.bam as well as an optional --num-threads to set the number of threads/cores to use.
Testing with the bam produced by the 4k_pbmc dataset, SplitBam task took 50 minutes using 8 threads. For reference, the old implementation took 217 minutes. Both of these tasks were run using HDDs, so we may see even more speedup when using SSDs in the parallelized implementation.
Review Instructions

No instructions.

The bam_hash changed because the new implementation orders tags slightly differently than the old version.


Cannot share a Null pointer

In GitLab by @Roussel-C on Jan 26, 2018, 08:48
In data_descriptor.cxx, in functions void Data_descriptor::share, we prevent sharing a null pointer independently of the access right.
I am suggesting to authorise the null ptr for safe cases (exposing with intent OUT or NONE).

see #58 a null data_ref is currently used to identify invalid accesses and sharing a null pointer might lead to unexpected behavior all over

closed via merge request !104

removed assignee

mentioned in commit 5daa53426dbc43061336f274c49efce55bcf5e71

closed via commit 5daa53426dbc43061336f274c49efce55bcf5e71

mentioned in commit 600d96190e013460337c0f0e7078274119baeec2


Editor loses padding value set on buttons in UI library after restart.

I created a button in a UI library setting the padding to 0 all around.
When the editor restarts the value of the padding is changed from zeros to 10 10 5 7.

I don't know much about the UI framework yet but this would seem to reset the padding everytime a button is constructed: https://github.com/xenko3d/xenko/blob/master/sources/engine/Xenko.UI/Controls/Button.cs#L31

Any luck here @Kryptos-FR ? If you point me where to look I can take a stab and  possibly push a fix.


Windows integration testing

Describe the problem
While developing the docker support and the database integration for the instance registry, we occasionally ran into some problems regarding docker for windows and slick in combination with MySQL Server on Windows machines. There has not been any further investigation yet.
Describe the solution you would like
We have to test both the docker functionality and slick (DB support) on windows machines. We have to find out how to avoid problems, and add any additional step that needs to be performed on windows to the readme file.

That's odd, i don't know why it would ever need a default value for the timestamp, since all events have a defined timestamp. But at least you are able to connect to your DB on your system. Which versions of SQL and Windows are you running on ?

I am using MySQL 5.7.19 in windows 10. It is one of the service of my WAMP .

Default timestamp problem in the windows MySQL is also fixed


ServiceEntry.location (MESH_EXTERNAL/MESH_INTERNAL) not used

This field is no longer used in pilot except in tests. We should consider removing/deprecating/hiding from the docs.

This is used not for automtls


any recent benchmark in using jsx vs jiffy?

Hi,
I am thinking to use jsx by default instead of jiffy in one project to increase its stability. To help the user choices I would like to provides an idea about the performances changes so they can do their own choice. Is there any recent benchmark and tool to do it that would allow me to compare both? I would indeed prefer to  use an existing tool instead of providing my own benchmark :)

there's some json benchmarking tools at https://github.com/devinus/poison but they probably need to be updated. if i have time this week i'll try to update them to use latest jsx/jiffy

It would also help if it was determined at what point jiffy locks up the scheduler, unless it has a stable version that uses the dirty scheduler now (I know changes were being worked on, but it seemed like it wasn't ready yet).  So you probably want to make sure that both really large and really small JSON data is tested.

With all the recent changes in jsx and jiffy, I'm sure https://exote.ch/~aseigo/elixir_json_parser_benchmark/json_comparison.html is no longer valid. But this issue doesn't seem to be, either.


Obfuscater Nuxt Build Error: v-for has invalid expression.

I'm trying to implement the obfuscator in my project, but it is converting the expressions in my v-for:

it is giving replace in the expression "in" replacing it with "kbiquzn"

This obfuscator is hard to pull off right to be honest
The way it works is basically it tries to replace all class names it detects between any quotes
Meaning if you have a class named in , you will have problem

The correct way would have been parsing the entire output html and replacing the class names after render, but im not sure how feasible is that

fixed in v1.1.5


Multi step support for step file

AC1:
As a user(developer or dev-in-test)
I want a stepFile with array type support
So that I can reduce lines of code in main test file(As if step File will have support of array type so i will write more steps in single external file and reduce main test file).
Suppose i am doing e2e testing in which around 20 api's are there for a single flow and after every api i am doing 15-20 db assertions.
Means total steps in a single scenario(flow) will have :  20(api) + (20(api) * 20 (db assertions)) = 420 steps, which is difficult to handle and understand in a single json file.
So currently i am doing is:
I have created a main test json file in which all api's are there and for db assertions i have created stepFile, which will reduce the number of lines in main test but i have to create more number of json file.
So if step file will have support of array type so that for every api's db assertions i can create a single stepFile and put all db assertions (for particular api) in that stepFile.

===
Hi,
Currently, only one step is supported in the step file. What if step-file will support Array type, means in a single step file we can write multiple steps
For example
main_test.json
{
    "scenarioName": "Multi Step File Test",
    "steps": [
          {
            "name": "first_test",
            "url": "",
            "operation": "",
            "customLog": "Contributing to Zerocode",
            "request": {
                "id": "12"
            },
            "assertions": {
            }
        },
        // Step File
        {
            "id": "step_file",
            "stepFiles": "${JSON.FILE:integration_test_files/multi_step/multi_step_file.json}"
        }
    ]
}

multi_step_file.json
[
  {
    "name": "fist_of_multi_step",
    "url": "",
    "operation": "",
    "customLog": "Contributing to Zerocode",
    "request": {
      "req": "12345",
      "id": "${$.first_test.request.id}"
    },
    "assertions": {
    }
  },
  {
    "name": "second_of_multi_step",
    "url": "",
    "operation": "",
    "customLog": "Contributing to Zerocode",
    "request": {
      "req": "54321",
      "res": "${$.fist_of_multi_step.request.id}"
    },
    "assertions": {
      "res": "${$.first_test.request.id}"
    }
  }
]

ISSUE-367 Multi step support for step file added  #368

Hi @anishjha93, Trying to understand the use-case for this. Also for other reviewer's benefit, can you put some details around when this particular feature is useful?
I have created a placeholder(in the description section) which shall help you to capture the detailed usecase for this.

Details added

Available in 1.3.18

Thanks @authorjapps

Hey Anish, can you please update the Wiki Page for this.
Here is the Wiki page link(TODO) :  https://github.com/authorjapps/zerocode/wiki/Reusing-Single-and-Multiple-Step-Files


Add support for QuickFix auto-commands and autowrite boolean.

Needed for parity with make and lmake.

Fixed in 7b427116f59eaf58f0a8eb2adb609e34308a6ac0


refactor: refactor

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1b605c5). Click here to learn what that means.
The diff coverage is 7.46%.

@@           Coverage Diff            @@
##             master     #51   +/-   ##
========================================
  Coverage          ?   3.06%
========================================
  Files             ?       2
  Lines             ?     163
  Branches          ?       0
========================================
  Hits              ?       5
  Misses            ?     157
  Partials          ?       1

Impacted Files
Coverage Δ

pkg/api/exec.go
0.00% <0.00%> (ø)

pkg/api/post.go
7.14% <9.61%> (ø)

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 1b605c5...8b5aad1. Read the comment docs.


feat: [IDP]: Added to docs on how to register a group entity

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66b3a61d730cc1202557cbc1--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66b44fbe4a01a28f8c5e2bfc--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66b451b522b02a8630a6437b--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66b48555295bb5bf4c69a23e--harness-developer.netlify.app


[CanvasKit] computeLineMetrics is unimplemented

Steps to Reproduce

Run the following code on Flutter web using CanvasKit:

final textPainter = TextPainter(
  text: TextSpan(
    text: 'bug',
  ),
  textAlign: TextAlign.center,
  textDirection: TextDirection.ltr,
)..layout();

textPainter.computeLineMetrics();

See the following exception:

══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
The following UnimplementedError was thrown during paint():
computeLineMetrics

When the exception was thrown, this was the stack:
..ddc_runtime/errors.dart 236:49  throw_
..flutter_web_sdk/lib/_engine/engine/canvaskit/text.dart 591:5                 computeLineMetrics
..flutter/src/painting/text_painter.dart 915:12                                computeLineMetrics

Version
This is true as of Flutter version 2.1.0-12.2.pre.

Hi @creativecreatorormaybenot

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.3, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
    • Flutter version 2.0.3 at /Users/tahatesser/Code/flutter_stable
    • Framework revision 4d7946a68d (5 days ago), 2021-03-18 17:24:33 -0700
    • Engine revision 3459eb2436
    • Dart version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • def0ad20 • android-arm64  • Android 10 (API 29)
    • macOS (desktop)        • macos    • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)           • chrome   • web-javascript • Google Chrome 89.0.4389.90

• No issues found!

[✓] Flutter (Channel master, 2.1.0-13.0.pre.233, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
    • Flutter version 2.1.0-13.0.pre.233 at /Users/tahatesser/Code/flutter_master
    • Framework revision 2f05751973 (64 minutes ago), 2021-03-24 02:57:01 -0700
    • Engine revision d59a01d2d3
    • Dart version 2.13.0 (build 2.13.0-150.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • def0ad20 • android-arm64  • Android 10 (API 29)
    • macOS (desktop)        • macos    • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)           • chrome   • web-javascript • Google Chrome 89.0.4389.90

• No issues found!

Thanks for filing the issue!


TimeoutException when downloading video

Is this error happening to somebody else?
I tried with the new --video-per-video flag, but i get the same error.
Thanks.
Downloading: Introduction_to_State_Management
Traceback (most recent call last):
  File "frontendmasters-dl.py", line 21, in <module>
    downloader()
  File "/Library/Python/2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Python/2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Library/Python/2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Python/2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "frontendmasters-dl.py", line 14, in downloader
    spider.download(course)
  File "/Users/manuelteixeira/Downloads/OS_FrontendMaster-dl-master/extractor/spider.py", line 45, in download
    self.download_course(course_downloadbale)
  File "/Users/manuelteixeira/Downloads/OS_FrontendMaster-dl-master/extractor/spider.py", line 181, in download_course
    download_file(subsection['downloadable_url'], file_path, self)
  File "/Users/manuelteixeira/Downloads/OS_FrontendMaster-dl-master/extractor/helper.py", line 21, in download_file
    self.browser.get(url)
  File "/Library/Python/2.7/site-packages/selenium/webdriver/remote/webdriver.py", line 320, in get
    self.execute(Command.GET, {'url': url})
  File "/Library/Python/2.7/site-packages/selenium/webdriver/remote/webdriver.py", line 308, in execute
    self.error_handler.check_response(response)
  File "/Library/Python/2.7/site-packages/selenium/webdriver/remote/errorhandler.py", line 194, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message: timeout
  (Session info: chrome=63.0.3239.84)
  (Driver info: chromedriver=2.33.506106 (8a06c39c4582fbfbab6966dbb1c38a9173bfb1a2),platform=Mac OS X 10.12.6 x86_64)

I keep getting the same

I've just downloaded the course - everything is fine

I debugged it and it hangs out for me at line
https://github.com/li-xinyang/OS_FrontendMaster-dl/blob/4064f90e766668b34bd8a1b27f09ee023b552546/extractor/helper.py#L24
So I removed that
self.browser.get(url)
temporaryURL = self.browser.current_url
self.browser.back()

and works like a charm.
BUT it was added as a fix by @philgetz in https://github.com/li-xinyang/OS_FrontendMaster-dl/pull/25
Deeper investigation needed here.

Sorry I mistakenly added this issue as resolved by PR.
This: https://github.com/li-xinyang/OS_FrontendMaster-dl/pull/56 resolves it but I must go deep with details in https://github.com/li-xinyang/OS_FrontendMaster-dl/pull/25


Fix Delay clock divider on Rev V chips

Looks good to me 👍  I recall we only checked AN5312 for changes, so maybe why this was missed.
bors r+


Llvm integrate 1ecfc12b0c67de24c160dd126b873efa8e51c7c7

There is another integration on-going https://github.com/iree-org/iree/pull/9700. The only failure happens in cmake_linux_riscv64. Should we land that one first to avoid dup fixes?

Yes, this is why I stopped on this one for now, I'll rebase it on top of
Diego's once it lands.
On Wed, Jul 6, 2022 at 6:39 AM Han-Chung Wang @.***>
wrote:

There is another integration on-going #9700
https://github.com/iree-org/iree/pull/9700. The only failure happens in
cmake_linux_riscv64. Should we land that one first to avoid dup fixes?
—
Reply to this email directly, view it on GitHub
https://github.com/iree-org/iree/pull/9707#issuecomment-1175772984, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNNU5COI4YQXVVSB3ZD2O3VSUEZVANCNFSM52S6JBNA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
N

This has served its purpose.


Fix social distancing bug (name trademarked by kamilprzyb)

BTW, for those who weren't following on the Discord channel: this bug caused players to disconnect as soon as another player came into view. It only occurred on builds compiled with MSVC, which had a different getTime() method.
Here's an image of the bug occurring:

According a decompilation of the client, the only instance this error message will be displayed is when the server time is out of sync.
(I know this should've been put into an issue or something, sorry about that)


Add pre requirement to package.json

Now we have small pre requirement on this moment. Maybe we can add the this pre requirements to package.json ?

What is the pre requirement?

@mhagmajer https://github.com/xFAANG/askql#prerequisites :)

@km4 oh yes, sure, if this can be said in package.json let's do it

ok, challenge accepted:P

Thanks @km4 and good luck 💪

@mhagmajer we have some pre requirement version for npm ? we can add too

I don't know about specific requirements, the Node version was picked at LTS for convenience. Wr use Node mostly for build tools

I close this issue. Work is merged.


ACK controller dashboard - creation/injection of service & credentials

Right now creation of AWS services are being done with helm using CRD's
I considered using kubeapps for installing AWS controllers. but It would be nice to have a native dashboard from AWS for service Install , creation and bind to a namespace & export the credentials inside the app
Kubeapps

Below is the screenshot from Openshit . You can create a s3 bucket with point & click ..  Is there anything available for EKS

Hi @bala151187 , if I understand well, you are requesting a native dashboard for managing ACK controllers and resources?

Hi @bala151187 , if I understand well, you are requesting a native dashboard for managing ACK controllers and resources?

Right ..

@bala151187 We have very high priority features for ACK controllers/runtime and unfortunately we will not be able to focus on a similar task anytime in the near future... Have you ever tried to use ArgoCD dashboard?

@bala151187 We have very high priority features for ACK controllers/runtime and unfortunately we will not be able to focus on a similar task anytime in the near future... Have you ever tried to use ArgoCD dashboard?

Yes.. but it's not intuitive as wat we are looking for ..