[Feature]: Validation failures for snupkg should specify which pdb(s) were problematic

Related Problem
From Twitter: https://twitter.com/punker76/status/1487421607303946243
If a snupkg contains both portable and Windows PDBs, validation will fail as expected. However the message doesn't say which file was the problem and it's hard for the user to know.
The Elevator Pitch
The validation messages should specify which files within a snupkg had issues.
Additional Context and Details
No response

Merging this one with https://github.com/NuGet/NuGetGallery/issues/8968


Update dependencies

Pull Request Test Coverage Report for Build 3fe512719d65353302be435074ebc1f1cf122430-PR-706

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.178%

Totals

Change from base Build<PHONE_NUMBER>b6ed950379a3c7802b4d153beae39e:
0.0%

Covered Lines:
1481

Relevant Lines:
1524

💛  - Coveralls

Pull Request Test Coverage Report for Build 3fe512719d65353302be435074ebc1f1cf122430-PR-706

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.178%

Totals

Change from base Build<PHONE_NUMBER>b6ed950379a3c7802b4d153beae39e:
0.0%

Covered Lines:
1481

Relevant Lines:
1524

💛  - Coveralls


Swift 4 support

Hi,
Is this project last repo already extensively tested and passing tests with swift4 ?
I am getting EXC_BAD_ACCESS called at Action:100  :
    Observable             .combineLatest(executing, enabledIf) { !$0 && $1 }             .bind(to: enabledSubject)             .disposed(by: disposeBag)
in .bind(to...)
at: override func subscribe<O : ObserverType>(_ observer: O) -> Disposable where O.E == Element {         if !CurrentThreadScheduler.isScheduleRequired {             // The returned disposable needs to release all references once it was disposed.             let disposer = SinkDisposer()             let sinkAndSubscription = run(observer, cancel: disposer)             disposer.setSinkAndSubscription(sink: sinkAndSubscription.sink, subscription: sinkAndSubscription.subscription)
in disposer.setSink...
Thats my only problem using RxSwift and not sure where is the problem.

Hi there! I don't think anyone's tested this for Swift 4 yet (I know I haven't). The solution doesn't appear obvious, I'll try to take a look this weekend and get back to you.

I'm working on a Swift 4 PR. Coming right up.

BTW @leszarna I'm not sure your problem comes from Action itself. Would be nice if you could try and repro once we make the swift4.0 branch available.

@fpillet I've tried your PR and it works very well for me now.
@fpillet Thank you very much for your help.
Thank you @ashfurrow.
This is really excellent piece of abstraction so thank you all working this.
Regards
Leszek

Hi guys, I'm newbie with cocoapods and things like that :)
Xcode 9 is stable now so we have Swift 4 officially released.  I updated my RxSwift and RxCocoa to use swift4.0 branch, but how can I do it with Action?
When I start pod install I got errors like this:
`pod install
Analyzing dependencies
[!] Unable to satisfy the following requirements:

RxSwift (from https://github.com/ReactiveX/RxSwift.git`, branch swift4.0)required byPodfile`
RxSwift (= 4.0.0-beta.0) required by Podfile.lock
RxSwift (~> 4.0.0-beta.0) required by RxCocoa (4.0.0-beta.0)
RxSwift (~> 4.0.0-alpha.0) required by RxDataSources (2.0.0-alpha.0)
RxSwift (~> 2.0.0-beta) required by Action (0.1.0)
`

Hi @adamsmaka – I'd be happy to help with this. Can you post your Podfile? Or at least the relevant portions of it? It looks like we might need to updated RxDataSources's podspec to point to the new beta: https://github.com/RxSwiftCommunity/RxDataSources/blob/swift4.0/RxDataSources.podspec#L44

of course, here it is
  pod 'RxSwift', git: 'https://github.com/ReactiveX/RxSwift.git', branch: 'swift4.0'
  pod 'RxCocoa', git: 'https://github.com/ReactiveX/RxSwift.git', branch: 'swift4.0'
  pod 'RxDataSources', git: 'https://github.com/RxSwiftCommunity/RxDataSources.git', branch: 'swift4.0'
  pod 'Action'

Thanks! I've reproduced the issue and I think it's caused by Action's pre-release not being on CocoaPods trunk. We should probably put Action up on trunk – my machine's setup is... not working right now for Swift 4 😞  @fpillet could you push the current podspec to trunk?
In the mean time, @adamsmaka , you can use Action's pre-release podspec with the following:
  pod 'Action', git: 'https://github.com/RxSwiftCommunity/Action.git', branch: 'master'

I had some issues with the branch names in your pod file, but was able to get CocoaPods working with the following:
  pod 'RxSwift', git: 'https://github.com/ReactiveX/RxSwift.git', branch: 'rxswift4.0-swift4.0'
  pod 'RxCocoa', git: 'https://github.com/ReactiveX/RxSwift.git', branch: 'rxswift4.0-swift4.0'
  pod 'RxDataSources', git: 'https://github.com/RxSwiftCommunity/RxDataSources.git', branch: 'swift4.0'
  pod 'Action', git: 'https://github.com/RxSwiftCommunity/Action.git', branch: 'master'

I'm currently on travel, in crunch mode and away from my usual Mac so not possible for me before next monday unfortunately

ok so we are waiting...

No worries, I'll take a look 👍 In the mean time, everything works with the podfile I posted above.

This is already fixed in #106


allow custom codec registry for Fongo async

and set protected visibility to Rule.newFongo method to allow Rule subclassing, in order to update codec registry in rules

thanks


Mchammer implementation

Implementation of MCHammer optimisation in TopologyGraph.construct().
Questions we need to answer (some are commented too):

Allow the user to specify the many settings as options (and how?) or just predefine some safe defaults (currently the case)?
What about periodic systems? MCHammer does not modify the cell size.
Handle output dir in MCHammer run - currently outputs trajectory and plots (could be optional)

@lukasturcani have a look at the latest API whenever you can!

@lukasturcani have a look at the latest API whenever you can!

Currently, none of these optimizers can modify the periodic cell.
This should be a separate issue as it requires modifying the API of the PeriodicInfo class (from my experience).

Currently, none of these optimizers can modify the periodic cell.
This should be a separate issue as it requires modifying the API of the PeriodicInfo class (from my experience).

@lukasturcani on review, please check the docs for each topology graph with a recommended optimizer -- what are your thoughts on this?

@lukasturcani on review, please check the docs for each topology graph with a recommended optimizer -- what are your thoughts on this?


music.lt support

Music.lt is biggest Lithuanian website having largest and legal music archive.
Player is based in flash/javascript.

Can you provide some sample URL's where you can listen to music?


feat: add support for RDS ExtendedSupport cost component

Adds cost component to db_instance and rds_cluster_instance for an extended support cost. This cost is only shown if the instnce engine and version fall into AWS's extended support category, and the current date is within the extended support window.
This is currently only applicable for mysql 5.7, for which extended support started March 2024. However, I have added tests for the rest of the versions listed in AWS docs so that we catch them in future.
The implementation for this feature means we hard code the support window in the resource. This is less than ideal but is the simplest way to accomodate the functionality right now. The pricing API response does contain an effectiveStartDate which we could use to filter the prices.
I avoided this, however, as it would involve a decent amount of refactoring and introduce some new concepts on the price fetching end. With this approach we would essentially "filter" the returned prices from the HCPAPI as well as during the lookup. e.g. effectivePriceDate is after current date, e.t.c.
I definitely think this is something we should accomodate in the future, however I am hesitant to do it now. Nevertheless, if you think this is a good time to try this, I can have a stab at it.

ah I think I've missed the vCPU multiplier of this, @aliscott I'll add that now


Generation of TSV and NT should happen at the same step

@justaddcoffee In the merge.yaml, we can define multiple outputs.
Right now our Jenkins build creates TSV in one stage and converts to NT in another stage.
Was this on purpose? For the sake of pushing the TSV to S3 first?
The amount of time taken to load the entire TSV again into memory is significantly high.

thanks @deepakunni3! I didn't realize we could define multiple outputs. Let's definitely change merge.yaml to output both TSV and NT. Can you ping me when you have a moment to chat about this?

for example in our merge.yaml we can define:
destination:
   merged-kg-tsv:
      type: tsv
      filename: merged-kg
   merged-kg-nt:
      type: nt
      filename: merged-kg.nt

closed by #179


[Bug]: Resetting audio and video widget resets playState values but widget UI does not change play state

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When resetting audio and video widget, playState value resets to NOT_STARTED but the widget UI does not change play state
Steps To Reproduce

Set an audio and video widget to reset using a button trigger field like so : {{resetWidget('Audio1')}}
Drop a text widget on the canvas and bind Audio/Video widget playState to this widget
Press play on the audio/video widget
Now trigger resetWidget reset using button widget
Observe the playState on the text widget changes from current state of PLAYING/PAUSED to NOT_PLAYING but UI does not change

https://www.loom.com/share/68026b0a41664d9eb8fbca37f8b2fa7a
Public Sample App
No response
Version
Cloud

Similar behaviour is with camera widget too, where widget property resets to original values but the UI still shows ok and discard icons after widget reset


Better Github integration for saving the running container

Through #49 we already have some very basic support for saving your progress from within everware.
It currently allows you to

fork the pulled repo if you don't have a repo with that name
push all changes to an everware branch in your repo

It would be nice if we could implement the following improvements:

Allow users to push to the checked out repo directly if they have push access, offer them to fork the repo otherwise
Allow them to specify the name of the branch that we push to
Show all commits that have been made between the originally checked out repo and the current state
Allow more fine-grained control over what is commited (e.g. on a per-file basis).

Some of this was reverted by #73 and needs to be added back. If @ibab doesn't start a new PR for this, how do we preserve his authorship of the HomeHandler?

If you cherry-pick the commits, then the author field will be preserved and show up correctly on Github.


Fix empty objects in if statements

Fix #559 as discussed in https://github.com/DanielXMoore/Civet/issues/559#issuecomment-1624411555
{} will be treated as an empty block if it starts on the same line as the if, and if it's on an unindented line.  But if it's on an indented line, it will be treated like an object literal.
I feel like this is sufficiently JS compatible, because the difference between doing nothing and creating an empty object literal (and similarly for doing something vs. creating an object literal with that something) only matters when interacting with implicit returns or if expressions, which are already differences from JS anyway.
I also ran into a weird bug with if (x) {} else {}, but that's for another issue/PR.

Thanks, good catch! The missing { is pretty spooky.


Deploy to Digital Ocean is broken

The button/link points to https://app.getkaiwa.com/oauth/start, which is a GitHub pages 404 (after accepting the SSL cert for the wrong domain).

The whole http://app.getkaiwa.com site is a 404. In this repo (kaiwa-site) there's a python script to start a flask app that contains the oauth endpoints: https://github.com/digicoop/kaiwa-site/blob/475496be94aadd4701fef0e09d73a12ac2958337/app/app.py


References [Awaiting another PR to be merged]

Problem: In Sui, we used to:

Deserialize Public Keys every time we want to verify a signature
Copy a Public Key TWICE every time we want to verify a signature (once from copying out of the 'committee' struct, the second time copying into the narwhal references function, which does not take in references)

This was probably fine with ed25519, where the public keys were already serialized bytes,  but BLS12381 stores public keys in some other form - which adds extra cost very time we copy/serialize the public keys. Note that this happened 30 times for each certificate.
Fix: This PR now instead allows the commitee struct to provide references to previously deserialized public keys. It also relies on another PR on narwhal that lets references of public keys to be passed in for verification.
Relies on: #3586 and https://github.com/MystenLabs/narwhal/pull/662

State of this PR?

@kchalkias I think this is definitely at least a bit faster, as I changed nothing in the code except for passing public keys by references only.

@kchalkias It's not a big difference on a 4-validator setup. I think we can leave this hanging too if you don't want to merge it.

Parking this PR for now, as benchmarks have not proved that it is useful.


No option to update the custom user properties

Current situation
Once we set some custom properties for a user who submits a survey, we are unable to add additional properties or change the value of the existing properties to add a new value.
Why is this needed?
We as a product tracked the users who were of type A and B before, but now we would like to add a type C and also add a new segment and map the existing users to the segment and we are unable to do so.

Hi @pramod-practo, the end user's properties should update if you update an existing one or if you add more fields. We'll investigate this issue and get back to you.

@diegoserranoa Thank you. Any breakthrough ?

@pramod-practo seems like a bug on our side. We'll fix it and let you know when the new version of the SDK with the fix is available!
Sorry for any inconvenience this has caused you.

Hello @pramod-practo! Sorry for the late update. I've been looking more into this and I think I misunderstood your request. You want to be able to update end users properties without surveying them? Example: You surveyed an end user and set properties = { "name" : "Joe", "last_name" : "Adams" }. Now after some time you want to be able to add/modify the properties for this end user even if no survey shows up for him?
Sorry for all the confusion.

Something like this: http://docs.wootric.com/api/#update-end-user ?

Hi @pramod-practo. I would love to get this resolved and we apologize it has taken so long. It seems that what you guys want is to update the survey properties. You can do that through our API. The link I commented above is to update the end user's properties. To update a response's properties you can use this:
http://docs.wootric.com/api/#update-response
Please let me know if this is correct.

Hello @diegoserranoa, Sorry for the delay from my side. Here is an example.
User 1 takes a survey on day 1 and this sets the property, prop1 with value1. After a few days based on the settings, the user takes the survey again (say day 90). Now in addition to prop1 we would like to add another property prop2 with value2. We want to do it with the survey he sends for the second time.  Looks like we can do it with the API you have provided. But we are using the iOS SDK. Is there no way to do it with the SDK ?

Oh, this can be done with the SDK. You would just need to add the new value to the dictionary.
Example
Survey 1
  NSDictionary *properties = @{
                               @"prop1" : @"value1"
                               };
  [Wootric setEndUserProperties:properties];

Survey 2
  NSDictionary *properties = @{
                               @"prop1" : @"value1",
                               @"prop2" : @"value2"
                               };
  [Wootric setEndUserProperties:properties];

@diegoserranoa we tried this and it did not get updated for a user who had already answered a survey before. Gets updated for a new user.

Hmm I just tried this a couple of days ago and it worked. Is this what you want?
Screenshot 1

Screenshot 2

@pramod-practo maybe you can send me the code snippet and I can try it on my side?

@diegoserranoa hey... i tried the same code again and it seems to be working today. But do take a look at the screenshots below. I tried sending surveys with different values for the properties.

@diegoserranoa Thank you for looking into this issue. Unfortunately it is not recreated in the current environment. If i even face this again i will raise an issue.

@pramod-practo Glad I could help! And sorry it took us so long to help you. Please feel free to reach us with any issue you have.

Having same issue on Android. I am not able to modify the properties that are currently set. I see Wootric is hitting this endpoint and returning the old properties when survey is shown.
https://api.wootric.com/v1/end_users?email=<...>&os_name=Android&os_version=9&sdk_version=android-2.15.0


allow interactive worldspace uis

Objective
allow use of the built in focus/interaction systems for non-window-based uis
Solution

add a ManualCursorPosition component that users can add to cameras to specify the cursor position
read it in ui_focus_system
sort UiStack by camera order as well as z-order, so that blocking elements are not ambiguously resovled between cameras
add interactive_ui_texture example

Testing
see example:
https://github.com/user-attachments/assets/8a3dc068-7633-4b75-8d0b-0a389f726fe1

What is the criteria for adding features and examples? I can't help but feel we are bloating the examples past the point of usefulness, and relying on them over documenting features.

yeah i was on the fence for the example, i'm happy to remove it. I added it because

World space UI is generally useful, but this addition seems like it can't stand on its own without a lot of supporting code in an example.

a game-ready solution would involve a collision library, which we don't have as first party. a simple code block on the struct seemed insufficient. I felt like this was a reasonable compromise to demonstrate the intention without writing a whole game.

On it's own ManualCursorPosition doesn't explain how it should be used, or motivate why it needs to be first party.

if it's not first party then you have to duplicate/replace the whole focus system in order to get Interactions on non-window uis. if you think the doc comment can be clearer then please let me know what you think it should say.
excluding the example this pr adds about 20 lines of code, 15 of which are fixing an existing ambiguity wrt overlapping uis on multiple cameras, and it opens up functionality that i personally need. if consensus is against it then i'll just continue to patch locally, but I do generally try to upstream things that I need and that are more broadly useful.

My preference would be instead of making this a special case, expose the pointer position on the UI as the only interface. The default bevy_ui input plugin would drive this with the pointer position on the window, and for this use case, where the render target is not a window, but a texture, you would need to supply it yourself - like you are doing in the example.
In other words, decouple the focus system from the input system. The input system sets the pointer position on the camera's render target, and the focus system just consumes that, it doesn't care how it got there.
This would also be more in line with how things will work with mod_picking.

decouple the focus system from the input system

agreed and updated. shall i drop the example as well?

The example is useful IMO: it's good to teach, and it'll be valuable during refactoring.


ENH Add slides to introduce cross validation

Partially addresses #445.
The goal of this slides is to dynamically show the splitting into folds using KFold and SuffleSplit strategies.
It also works as a motivation of the concept "variability of an estimated generalization performance".
Finally, it encourages "data scientist on the wild" to have good practices for scoring.

I think they should be inserted after https://2425-246063957-gh.circle-artifacts.com/0/jupyter-book/python_scripts/02_numerical_pipeline_scaling.html but before the module quizz.

I think they should be inserted after https://2425-246063957-gh.circle-artifacts.com/0/jupyter-book/python_scripts/02_numerical_pipeline_scaling.html but before the module quiz.

The idea is to split the numerical pipeline scaling notebook into two to create a new notebook with only the cross validation, and then insert the video between them.
I am already working on such PR, but I need #477 to be reviewed first to avoid conflicts.

The idea is to split the numerical pipeline scaling notebook into two to create a new notebook with only the cross validation, and then insert the video between them.

This is done in #498. We can either merge that PR first and modify this PR to modify the _toc file, or merge this PR and then assign the place to the slides there.


Docker image not working

Hello everyone,
I'm currently trying to get approval from my work so I can contribute to this project. To get this, I need to scan the repository for any conflicting licences using scancode-toolkit. This was working so far for the repository. However, I need to also check any Dockerfiles of the project. I'm currently trying to run tern with scancode-toolkit, which is also able to scan docker images. (This will need to build the Docker image.)
The Docker image in the containers directory doesn't seem to work properly. Running docker build -t refgenie -f Dockerfile_refgenie . led to some errors.
These were the steps I've had problems with:
htslib
Problematic code:
# htslib 1.9 (tabix cmd)
        RUN apt-get install -y libz-dev libncurses-dev
    RUN wget -O ~/htslib.tar.gz https://github.com/samtools/htslib/releases/download/1.9/htslib-1.9.tar.bz2 && tar xjf ~/htslib.tar.gz && cd htslib-1.9 && ./configure && make && make install
    ENV PATH="/htslib-1.9:${PATH}"

Error message:
2021-06-11 12:57:21 (11.0 MB/s) - '/root/htslib.tar.gz' saved [1178859/1178859]

checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking for ranlib... ranlib
checking for grep that handles long lines and -e... /bin/grep
checking for C compiler warning flags... -Wall
checking for special C compiler options needed for large files... no
checking for _FILE_OFFSET_BITS value needed for large files... no
checking for _LARGEFILE_SOURCE value needed for large files... no
checking shared library type for unknown-Linux... plain .so
checking how to run the C preprocessor... gcc -E
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking for stdlib.h... (cached) yes
checking for unistd.h... (cached) yes
checking for sys/param.h... yes
checking for getpagesize... yes
checking for working mmap... yes
checking for gmtime_r... yes
checking for fsync... yes
checking for drand48... yes
checking whether fdatasync is declared... yes
checking for fdatasync... yes
checking for library containing log... -lm
checking for zlib.h... yes
checking for inflate in -lz... yes
checking for library containing recv... none required
checking for bzlib.h... no
checking for BZ2_bzBuffToBuffCompress in -lbz2... no
configure: error: libbzip2 development files not found

The CRAM format may use bzip2 compression, which is implemented in HTSlib
by using compression routines from libbzip2 <http://www.bzip.org/>.

Building HTSlib requires libbzip2 development files to be installed on the
build machine; you may need to ensure a package such as libbz2-dev (on Debian
or Ubuntu Linux) or bzip2-devel (on RPM-based Linux distributions or Cygwin)
is installed.

Either configure with --disable-bz2 (which will make some CRAM files
produced elsewhere unreadable) or resolve this error to build HTSlib.
The command '/bin/sh -c wget -O ~/htslib.tar.gz https://github.com/samtools/htslib/releases/download/1.9/htslib-1.9.tar.bz2 && tar xjf ~/htslib.tar.gz && cd htslib-1.9 && ./configure && make && make install' returned a non-zero code: 1
[lasse@localhost refgenie]$ sudo docker images
REPOSITORY    TAG       IMAGE ID       CREATED         SIZE
<none>        <none>    77278782c07f   4 minutes ago   493MB"

Potential fix:
Adding RUN apt-get install -y libbz2-dev liblzma-dev before the wget command seems to fix it for me.
epilog_indexer
Problematic code:
# epilog meth calling
        # TODO: Use public version after publication.
        #RUN wget -O epilog_indexer.py https://github.com/databio/epilog/blob/master/epilog/epilog_indexer.py
        ADD includes/epilog_indexer.py bin/epilog_indexer.py
        RUN pip install --user regex

Error message:
Step 28/38 : ADD includes/epilog_indexer.py bin/epilog_indexer.py
ADD failed: file not found in build context or excluded by .dockerignore: stat includes/epilog_indexer.py: file does not exist

Problematic code:
# UCSC twoBitToFa
        ADD includes/twoBitToFa bin/twoBitToFa
        RUN apt-get install -y libpng-dev

Error message:
Step 28/36 : ADD includes/twoBitToFa bin/twoBitToFa
ADD failed: file not found in build context or excluded by .dockerignore: stat includes/twoBitToFa: file does not exist

The last two error messages looks like they're originating from a tool which is not ready for public release yet. Uncommenting these, will result in a successful image. I would be grateful if you could provide/update the Docker image so I can build and analyze it with all it's dependencies. Let me know if you need more information.
Cheers

Can confirm that the HTSlib step failed. After adding the fix you suggested, I was able to build the image.
[mstolarczyk@Darias-MacBook-Pro containers](dev): make
mkdir -p includes
wget https://github.com/nsheff/docker/raw/master/includes/twoBitToFa -P includes
--2021-06-11 14:52:55--  https://github.com/nsheff/docker/raw/master/includes/twoBitToFa
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://raw.githubusercontent.com/nsheff/docker/master/includes/twoBitToFa [following]
--2021-06-11 14:52:56--  https://raw.githubusercontent.com/nsheff/docker/master/includes/twoBitToFa
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 4626021 (4.4M) [application/octet-stream]
Saving to: ‘includes/twoBitToFa.1’

twoBitToFa.1                                       100%[==============================================================================================================>]   4.41M  2.92MB/s    in 1.5s

2021-06-11 14:52:57 (2.92 MB/s) - ‘includes/twoBitToFa.1’ saved [4626021/4626021]

wget https://github.com/nsheff/docker/blob/master/includes/epilog_indexer.py -P includes
--2021-06-11 14:52:57--  https://github.com/nsheff/docker/blob/master/includes/epilog_indexer.py
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/html]
Saving to: ‘includes/epilog_indexer.py.1’

epilog_indexer.py.1                                    [ <=>                                                                                ] 161.64K  --.-KB/s    in 0.05s

2021-06-11 14:52:58 (2.96 MB/s) - ‘includes/epilog_indexer.py.1’ saved [165518]

docker build -t databio/refgenie -f Dockerfile_refgenie .
[+] Building 718.9s (34/34) FINISHED
 => [internal] load build definition from Dockerfile_refgenie                                                                                0.0s
 => => transferring dockerfile: 2.70kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:18.04                                                                                1.1s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 118B                                                                                0.0s
 => [ 1/29] FROM docker.io/library/ubuntu:18.04@sha256:67b730ece0d34429b455c08124ffd444f021b81e06fa2d9cd0adaf0d0b875182                                                                              0.0s
 => CACHED [ 2/29] RUN apt-get update && apt-get install -y wget git unzip                                                                                0.0s
 => CACHED [ 3/29] RUN apt-get install -y python python-pip                                                                                0.0s
 => CACHED [ 4/29] RUN apt-get install -y curl                                                                                0.0s
 => CACHED [ 5/29] RUN apt-get install -y libz-dev libncurses-dev                                                                                0.0s
 => [ 6/29] RUN apt-get install -y libbz2-dev liblzma-dev                                                                                6.3s
 => [ 7/29] RUN wget -O ~/htslib.tar.gz https://github.com/samtools/htslib/releases/download/1.9/htslib-1.9.tar.bz2 && tar xjf ~/htslib.tar.gz && cd htslib-1.9 && ./configure && make && make ins  69.8s
 => [ 8/29] RUN wget -O ~/samtools.bz2 https://github.com/samtools/samtools/releases/download/1.3.1/samtools-1.3.1.tar.bz2                                                                           2.3s
 => [ 9/29] RUN tar -xf ~/samtools.bz2                                                                                1.7s
 => [10/29] RUN cd /samtools-1.3.1 && make                                                                                52.6s
 => [11/29] RUN apt-get install -y libtbb-dev  # bowtie2 dependencies                                                                                4.1s
 => [12/29] RUN wget -O ~/bowtie.zip "https://downloads.sourceforge.net/project/bowtie-bio/bowtie2/2.3.0/bowtie2-2.3.0-linux-x86_64.zip?r=https%3A%2F%2Fsourceforge.net%2Fprojects%2Fbowtie-bio%2Ff  5.7s
 => [13/29] RUN unzip ~/bowtie.zip                                                                                1.9s
 => [14/29] RUN wget -O ~/bismark.zip https://github.com/FelixKrueger/Bismark/archive/0.17.0.zip                                                                                2.2s
 => [15/29] RUN unzip ~/bismark.zip                                                                                0.6s
 => [16/29] RUN wget -O ~/hisat.zip ftp://ftp.ccb.jhu.edu/pub/infphilo/hisat2/downloads/hisat2-2.0.5-source.zip                                                                                1.5s
 => [17/29] RUN unzip ~/hisat.zip                                                                                0.8s
 => [18/29] RUN cd hisat2-2.0.5 && make                                                                                380.6s
 => [19/29] RUN wget -O ~/kallisto.tar.gz https://github.com/pachterlab/kallisto/releases/download/v0.43.0/kallisto_linux-v0.43.0.tar.gz                                                             4.1s
 => [20/29] RUN tar -xf ~/kallisto.tar.gz                                                                                1.1s
 => [21/29] ADD includes/epilog_indexer.py bin/epilog_indexer.py                                                                                0.0s
 => [22/29] RUN pip install --user regex                                                                                8.8s
 => [23/29] ADD includes/twoBitToFa bin/twoBitToFa                                                                                0.1s
 => [24/29] RUN apt-get install -y libpng-dev                                                                                4.2s
 => [25/29] RUN wget -O ~/bwa-0.7.17.tar.bz2 https://github.com/lh3/bwa/releases/download/v0.7.17/bwa-0.7.17.tar.bz2                                                                                0.9s
 => [26/29] RUN tar -xf ~/bwa-0.7.17.tar.bz2                                                                                0.4s
 => [27/29] RUN cd /bwa-0.7.17 && make                                                                                14.1s
 => [28/29] RUN wget -O ~/STAR.tar.gz https://github.com/alexdobin/STAR/archive/2.7.1a.tar.gz && tar -xf ~/STAR.tar.gz && cd STAR-2.7.1a/source && make STAR                                       138.5s
 => [29/29] RUN apt-get install -y genometools                                                                                9.0s
 => exporting to image                                                                                6.0s
 => => exporting layers                                                                                6.0s
 => => writing image sha256:317c2098263ee5dd5305bb230cedd7dc09e3ee14d1d8b2b2d7173f7e3f196f86                                                                                0.0s
 => => naming to docker.io/databio/refgenie

I just tested it and can confirm that I can build the Doker image correctly with the make command. Thank you!


[graphviz] Update to 7.0.0

What does your PR fix?
Updates graphviz.

Which triplets are supported/not supported? Have you updated the CI baseline?

Fixes arm64-windows. Probably this was a general cross-build problem: the dot must be run once to create plugin configuration. This used to fail, but now it is guarded (upstream), and a message is printed instead (portfile).
staticcrt unsupported due to use of plugin.
uwp unknown (cascade).
ci.baseline.txt unchanged.

Does your PR follow the maintainer guide?
yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
yes

osx suffered from https://gitlab.kitware.com/cmake/cmake/-/issues/16154#note_1220397 again.

Depends on  #27551

@FrankXie05 added the requires:all-feature-testing

There are no features modeled in graphviz.

Thank you!


Attempts to access ref on any Native Base component results in crash.

Hi! Thanks for trying out NativeBase!
Due to the overwhelming requests and issues/questions we ask you to add the following details with your issue.
Please check the existing open/closed issues for a possible duplicate before creating a new issue :)
Also DO fill out the below form to give us a better idea about your environment and help us debug it quicker. Issues without the required details will mostly be unattended to.
react-native, react and native-base version
react-native: 25.0.0
react: 16.2.0
native base: 2.4.3
Expected behaviour
Ability to access ref shouldn't crash
Actual behaviour
Any attempts to access the ref of a native base component results in crash
Steps to reproduce (code snippet or screenshot)
<Tabs ref={t => { this._tabs = t }}>...</Tabs>

Screenshot of emulator/device
No screenshot abailable, app crashes.
Is the bug present in both ios and android or in any one of them?
Both
Any other additional info which would help us debug the issue quicker.
I've tried with a standard react native component and can get the ref fine, any native base component fails.

@RoninSTi checked this with native-base version 2.4.3.Checked ref using <Input/> component. Can you try upgrading react and react-native packages. If this issue still occurs, can you post a sample code for us to reproduce. Attaching a gif.
Gif


How to generate hrdhistogram output

Please help, I need your assistance to generate an output for loading and running a Mongodb benchmark. what command do i need to generate a HRDHistogram output, and how do i apply it. pls it is urgent

Closing as stale. Please reopen if you're still having trouble.


Potential memory leak in SoloEnhanced

From<EMAIL_ADDRESS>on May 15, 2015 08:56:05
What steps will reproduce the problem? 1. Install LeakCanary into the build
2. Start the instrumentation test (in my case, this is with Calabash for Android
3. Have the test pause for at least 30 seconds (to give LeakCanary time to analyze the heap dump) What is the expected output? What do you see instead? Leak Canary detects a leak:
LoginActivity has leaked:
In local.myapp:1.0:1.

local.myapp.activity.LoginActivity has leaked:

GC ROOT static sh.calaba.instrumentationbackend.InstrumentationBackend.solo

references com.robotium.solo.SoloEnhanced.activityUtils

references com.robotium.solo.ActivityUtils.activity

leaks local.myapp.activity.LoginActivity instance

Reference Key: 8e4b95da-43fb-4af5-8ac0-9dc9b7118036

Device: samsung samsung SM-G900H k3gxx

Android Version: 4.4.2 API: 19

Durations: watch=5134ms, gc=218ms, heap dump=1275ms, analysis=23376ms What version of the product are you using? On what operating system? Calabash 0.5.8, which uses robotium-solo-4.3.1.jar
on Mac OS X 10.10 (Yosemite) Please provide any additional information below. The original issue was reported to LeakCanary at https://github.com/square/leakcanary/issues/93#issuecomment-102206755 the maintainer believes this to be a "real bug"

Original issue: http://code.google.com/p/robotium/issues/detail?id=677

From<EMAIL_ADDRESS>on May 15, 2015 08:57:52
Sorry, "SoloEnhanced" is wrong, that's the Calabash extension of Solo (in order to override visibility of some fields)

From<EMAIL_ADDRESS>on May 16, 2015 02:04:22
Thanks for reporting this. Calabash is using an old version of Robotium. Do you also get this when trying the latest version of Robotium?
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on May 26, 2015 16:16:25
Will open up if more information is given
Status: Invalid


[BugFix] get_json_string parse json field in non-string type as string

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #6447,#6426
Problem Summary(Required) ：

This makes get_json_string parse json field in non-string type as string.

run starrocks_clang-format


failing tests when run by Jenkins

I refreshed my Jenkins environment and thought I run a number of jobs that I installed 1/2 years ago. One of them is bats.
Although the project run without failing tests 2 years ago I now get failing tests
49 tests, 42 ok, 7 not ok, 0 skipped, 0 Bail Out!
I can provide more details if needed, but before spending more time on it I thought I'd check in with you.

got similar issue on Debian 8. Some tests fails without any additional info, albeit everything is OK on Debian 7. Trying to find out why...


Forward listening and error events.

listening event is currently not forwarded which force the user to pass a callback to .listen().
This breaks certain patterns such as the use of event-to-promise.
The problem is worst for the error event: because it is not forwarded, it will crash the process.

I opened up a similar issue (#155) as well, there already is some event forwarding code which likely needs to be refactored.


Add Test task akin to Build, Clean, Rebuild

From @tmat on February 13, 2017 23:56
Currently dotnet test gets translated to invocation of VSTest task implemented in Microsoft.TestPlatform.targets as a call to Microsoft.TestPlatform.Build.Tasks.VSTestTask.
This implementation is different from the other tasks provided by the SDK that follow the outer/inner build pattern when the project targets multiple frameworks.
It would imo be better to have a consistent implementation of these basic tasks in the SDK, so that the user has a single mental model of how to parameterize them. For example, I might want to run tests on 32bit and 64bit architectures when targeting net46 but only 64bit when targeting .NET Core:
  <TargetFrameworks>netcoreapp1.0;net46</TargetFrameworks>
  <TestArchitectures Condition="'$(TargetFramework)' == 'net46'">x64;x86</TestArchitectures>

I propose a Test task that is based on the outer/inner build pattern is added to the SDK. The implementation could be something like:
<Project>
  <PropertyGroup Condition="'$(IsTestProject)' == 'true'">
    <TestArchitectures Condition="'$(TestArchitectures)' == ''">x64</TestArchitectures>
    <_TestTargetName Condition="'$(TargetFrameworks)' == ''">InnerTest</_TestTargetName>
    <_TestTargetName Condition="'$(TargetFrameworks)' != ''">OuterTest</_TestTargetName>
  </PropertyGroup>

  <Target Name="Test" DependsOnTargets="$(_TestTargetName)" Condition="'$(IsTestProject)' == 'true'" />

  <Target Name="InnerTest">
    <ItemGroup>
      <_TestArchitectureItems Include="$(TestArchitectures)" />
    </ItemGroup>

    <PropertyGroup>
      <_TestLogPath>$(ArtifactsTestResultsDir)$(MSBuildProjectName)_$(TargetFramework)_%(_TestArchitectureItems.Identity).txt</_TestLogPath>
    </PropertyGroup>

    <Delete Files="$(_TestLogPath)" />
    <MakeDir Directories="$(ArtifactsTestResultsDir)"/>

    <!-- Replace this Exec with a direct call to Microsoft.TestPlatform.Build.Tasks.VSTestTask -->
    <Exec Command='"$(DotNetTool)" test "$(MSBuildProjectFullPath)" --no-build --framework:$(TargetFramework) -- --platform:%(_TestArchitectureItems.Identity) > $(_TestLogPath)'
          LogStandardErrorAsError="true"
          WorkingDirectory="$(OutDir)"
          IgnoreExitCode="true">

      <Output TaskParameter="ExitCode" PropertyName="ExitCode" />
    </Exec>

    <Message Text="Test succeeded: $(MSBuildProjectName) [$(TargetFramework)|%(_TestArchitectureItems.Identity)]" Condition="'$(ExitCode)' == '0'" />
    <Error Text="Test failed with exit code: $(ExitCode); log: $(_TestLogPath)" Condition="'$(ExitCode)' != '0'" />
  </Target>

  <Target Name="OuterTest" DependsOnTargets="_SetTestInnerTarget;DispatchToInnerBuilds" />

  <Target Name="_SetTestInnerTarget" Returns="@(InnerOutput)">
    <PropertyGroup Condition="'$(InnerTargets)' == ''">
      <InnerTargets>InnerTest</InnerTargets>
    </PropertyGroup>
  </Target>
</Project>

Copied from original issue: dotnet/sdk#866

From @tmat on February 14, 2017 2:38
FYI @ManishJayaswal

From @tmat on February 13, 2017 23:56
@sbaid @srivatsn @nguerrera @piotrpMSFT

From @piotrpMSFT on February 14, 2017 7:17
@tmat I think this belongs in microsoft/vstest

Here's the current cross targeting for tests: https://github.com/dotnet/cli/blob/rel/1.0.0/resources/MSBuildImports/15.0/Microsoft.Common.CrossTargeting.targets/ImportAfter/Microsoft.TestPlatform.CrossTargeting.targets

Related: https://github.com/Microsoft/vstest/issues/326

Will use https://github.com/Microsoft/vstest/issues/326 to track this one. Closing this.


Use remarketing with search results

We want to use Google Adwords Dynamic Remarketing with our search results. Is there functionality available for this? Or do we need to custom build something? And if so, what is the best way to implement this?
See https://support.google.com/adwords/answer/3103357?hl=en retail section for more info please.

Hi @Maaark we do not have anything builtin with google adwords but we are very interested in the needs you have.
I have no knowledge at all about google adword or the page you linked unfortunately so you will have to explain it to me a bit more in detail what is it you want to achieve.
Can you detail a bit more for example what are the type of calls you would want to do to google adwords and how it relates to instantsearch.js/search?
Thanks

@vvo Adwords retargeting are those ad blocks on websites that display products you recently viewed. To do that Google Adwords needs to know what products a customer viewed/searched for.
To do so we need to implement some code that runs after a search. That code needs to contain this info:
var google_tag_params = {
ecomm_prodid: "REPLACE_WITH_STRING_VALUE",
ecomm_pagetype: "REPLACE_WITH_STRING_VALUE"
};
</script>

```

ecomm_prodid is an array with the product id's as in the shopping feed
ecomm_pagetype is a predefined string which is the same for all search queries on this page.

So the main thing is that the ecomm_prodid gets filled with the product id's after a search and that the listed code is executed.

Is this clear enough? Or do you have additional questions?

So let's say I type "mp3" in a product search interface, I get ten different mp3 players, you want to send all of their ids to google adwords. Is that it?

@vvo Yeah, that's all :)

Ok then to do so, you could do a small custom widget like this:
var adWidget = {
  render: function(params) {
    var hits = params.results.hits;
    var ids = hits.map(function(hit) { return hit.id });
    adWordsCallWith(ids);
  }
};

search.addWidget(adWidget);

Something like that would work, then about the calls to make to google adwords, its up to you. There I cannot really help you.
Let me know how it goes.

@vvo Thanks, that worked!


try to create directories for history if they don't already exist

It can be a poor user experience if every time I exit numbat I see an error message about the history failing to save, without any automatic fixes or suggestions.

hamir@Laptop:~/numbat$  cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.21s
     Running `target/debug/numbat`

  █▄░█ █░█ █▀▄▀█ █▄▄ ▄▀█ ▀█▀    Numbat 1.10.1
  █░▀█ █▄█ █░▀░█ █▄█ █▀█ ░█░    https://numbat.dev/

>>>  cos (25 deg)

  cos(25 degree)

    = 0.906308

>>>
>>>
Error while saving history to '/home/hamir/.local/share/numbat/history': No such file or directory (os error 2): No such file or directory (os error 2)
hamir@Laptop:~/numbat$  cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.20s
     Running `target/debug/numbat`

  █▄░█ █░█ █▀▄▀█ █▄▄ ▄▀█ ▀█▀    Numbat 1.10.1
  █░▀█ █▄█ █░▀░█ █▄█ █▀█ ░█░    https://numbat.dev/

>>>  cos (25 deg)

  cos(25 degree)

    = 0.906308

>>>
>>>
Error while saving history to '/home/hamir/.local/share/numbat/history': No such file or directory (os error 2): No such file or directory (os error 2)


Improve self-report skills request and rejection messages

client-display: when reporting skills

change input to text - support multiple lines
add validation - max number of characters

dashboard: display long messages gracefully

support long messages for approval requests as well as history

dashboard: when rejecting

add validation - max number of characters

:+1:

@cjmoses - I noticed that when the self report skill uses the "Honor System" that it shows the character limit warning even though there is no text area for the user to type into... we should remove the warning message for this scenario.

Did not notice this until just now. Will be addressed in #942

:+1: see #942 for removing the warning message


OTG portals get created deeper than those quotes that 14 year olds think they make

OTG Plugin or Mod
[Please specify if it's about Forge Mod or Spigot server Plugin placing "x" into []]

[x] OTG Mod for Forge
[ ] OTG Plugin for Spigot

OTG Version
[Please specify the full OTG version that you are running by placing "x" into []]

[ ] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[x] Other - 1.12.2 v8.3

Forge or Spigot Versions
<IP_ADDRESS>47
What other plugins are you running?
FTB Builders Paradise 1.6.0
Decocraft 2.6.3 (PTRLib 1.0.4)
Extra Utilities 2 1.9.9
Born in a barn v1.8-1.1
AI improvements 0.0.1b3
Vanilla fix 1.0.10-SNAPSHOT
FPS reducer 1.14.1
Description
Getting into an OTG portal creates and teleports you into a portal close to y=0. (Note: don’t know the exact specifics but it is possible to also spawn deep below the surface when coming back too, maybe via teleporting directly to that dimension)
Make a portal (maybe having a y of 92 will help), go through it
Well I expected to have the portal on the surface, not one deeper than Kola Superdeep Borehole (40,230ft deep)

Hey @SlimyPython , this should hopefully be fixed for v9. Please try the latest dev version from the OTG Discord at https://discord.gg/UXzdVTH. If the problem still occurs, feel free to reopen this issue, thanks!


Add a new component category: AIR_CONDITIONER

What's needed?
We want to incorporate air conditioners into our stack.
Proposed solution
Extend the enum ComponentCategory.
Use cases
Users have indicated that they want to read electrical and sensor (temprature) metrics using our APIs.
Alternatives and workarounds
No response
Additional context
No response

For some reason, the linking from PR #218 did not work. Closing it here instead.

This needs to be added to v0.6.x branch now.


Editor selection on edit page instead of system-wide

Describe the feature you'd like
Currently, choosing between the WYSIWYG and markdown editors is done system-wide as mentioned in #57. Would it be possible to let the user select the way they want to provide content instead of restricting this option for admins only, by adding an option to switch editor on the edit page?
Describe the benefits this feature would bring to BookStack users
Users would be able to select the editor they like instead of the system-wide editor.
Additional context
In my current situation, I have several users that do not like to use the markdown editor, while other users, including myself can't do without it, as I often migrate MD files from an older system. Having this feature would make it easier to cater to all the users of my BookStack instance.
The comment raised in #57 (https://github.com/BookStackApp/BookStack/issues/57#issuecomment-183544163) does mention this might be confusing to some users, but perhaps this ability to switch editors could then be disabled in the admin panel by the admin?
My thanks to all contributors. You guys are awesome.

Hi @nvhaver,
Thank you for your clear request. I'm going to close this as I feel this request is already covered by #842.
There is an item on the roadmap ("Editor Alignment & Review") at which point we will strongly consider this kind of functionality.


Serializer encountered maximum depth

Hi,
Saved to restsharp, i retrieve some json data from a rest api source.
I do my stuff with the related objects then i want to store into arango.
For that i follow the doc https://github.com/yojimbo87/dictator#convert-strongly-typed-object-to-document.
And it works for very little objects as mostly i got this error :
"Serializer encountered maximum depth of 20"
I don't know what is this maximum depth but i imagined 20 is not big either so i changed this setting;
        ASettings.JsonParameters.SerializerMaxDepth = 255;

Apparently it's not enough as i get "Serializer encountered maximum depth of 255".
I looked at some objects, some were not so little so i loop them to do the same thing on each of their element instead of the whole structure but still the same error.
This drives me to think that "Convert strongly typed object to document" is OK but with very little object then...
Does someone have a workaround ? Idea ?
I feel stuck this way.
Maybe i should use Newtonsoft.Json and convert my objects into json and store into arango, and practise the way back when i need.

Hi, can you paste here how the object looks like in JSON format?

Yes, here it is in JSON before i get it into an object :


Update request: openrct2 0.4.5 → 0.4.6

Package name: openrct2
Latest released version: 0.4.6
Current version on the unstable channel: 0.4.5
Current version on the stable/release channel: 0.4.4
[x] Checked the nixpkgs pull requests

Notify maintainers
@oxzi

Note for maintainers: Please tag this issue in your PR.

I attempted this one, and even though all the new title-sequences files are in the same place as the old ones, it fails to load those. And it fails to load the park files from the original games, that reads without any issues in 0.4.5.

Failure logs: rct2.log
Note that object conflict also happens in 0.4.5.

Bump commit: https://github.com/PedroHLC/nixpkgs/commit/719cbd1a52a5630d681a863d6b13c5aff65be81d

Suggestions/Enhancements:

Use SRI-hash field: https://github.com/PedroHLC/nixpkgs/commit/fa19ac6ee30e177df08a12ecf0075f017f5eb91a
Use "srcs" (solves #225894): https://github.com/PedroHLC/nixpkgs/commit/ce4d21859828ca424e1a127efdc48d63b2f6c66f

I experienced the same when I tried 0.4.6 with an overlay. Also could not connect to multiplayer games because he could not load the map. Hoped that somebody more knowledgeable in NixOS could solve this.

Please excuse the late reply. When 0.4.6 came out, I tried bumping the versions as I did before and ran in the same buggy behavior as @PedroHLC and @ToppDev did. Since then I procrastinated debugging it. I'm hoping that I'll find time this week. If someone else beets me, I would also be happy.


fixed etherscan contract label crawling

Fixed crawler, now it is not adding id manually
Changes

How to test these changes?

Related issues

Superseded by https://github.com/bugout-dev/moonstream/pull/239


Priority in Groups

Sometimes you need to add a player to several groups. For example: One constant, the second has a lifetime. The second must overlap the rights of the first at the time of its existence, this requires priority.
We use the rscPermissions  plugin.
Here it is realized, that's the field
inheritance_priority.
The absence of such a mechanism stops us from switching to your plugin.
Thank you in advance! We will wait for the update.

This exists already.
https://github.com/lucko/LuckPerms/wiki/Command-Usage#lp-group-group-setweight


Consistency in Naming "OpenAI"

I've observed some inconsistency in class naming regarding "OpenAI". Some classes use "OpenAI" while others use "OpenAi". Is there a specific naming convention or rule being followed? If not, I would like to suggest standardizing the naming to "OpenAI" for consistency.

completed in recent versions


Add --open parameter to start script

This causes executing the start script to open the angular
app automatically.

Hi @winksaville,
Unfortunately, after discussing this a bit offline, we found that this isn't something we want to implement.
It's trivial for you to change once you created a project and not something most people want. I'm going to close this PR. Thanks.

Thanks for considering it.


The az aks get-credentials command is failing for me

The az aks get-credentials command is failing for me
The command fails with the error below.
I also tried rotating the cluster certs with az aks rotate-certs but I still get this error.
In case it is related, please note that we enabled Modern auth and turned on Security defaults for our tenant this past Monday (11 Oct).
Describe the bug
Command Name
az aks get-credentials Extension Name: aks-preview. Version: 0.5.24.
Errors:
'name'
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 659, in execute
    raise ex
cli/core/commands/__init__.py, ln 722, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 715, in _run_job
    six.reraise(*sys.exc_info())
...
cliextensions/aks-preview/azext_aks_preview/custom.py, ln 3807, in merge_kubernetes_configurations
    _handle_merge(existing, addition, 'users', replace)
cliextensions/aks-preview/azext_aks_preview/custom.py, ln 3749, in _handle_merge
    if i['name'] == j['name']:
KeyError: 'name'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks get-credentials -g {} -n {}

Expected Behavior
Environment Summary
macOS-10.13.6-x86_64-i386-64bit
Python 3.8.7
Installer: HOMEBREW

azure-cli 2.17.1 *

Extensions:
aks-preview 0.5.24

Additional Context

Can you try either removing aks-preview (az extension remove -n aks-preview) or updating it (az extension update -n aks-preview), as we're up to 0.5.34?

@davidack Apologies for the late reply. You can check under the user profile for file ~/.kube/config. There are 3 required sections which must be present: clusters, contexts and users. I feel there seems to be an issue to read this config file. You can try to rename or delete the file and run the az aks get-credentials command and check if that works.
Alternatively you can pass the --file or --overwrite-existing parameter and run the command again. More Info here.
Please let us know if you still need any assistance on this. Awaiting your reply.

I am familiar with the proper format of my ~/.kube/config file. It has the 3 required sections, and each of the sections contains many entries with some variation in content because I manage multiple clusters from multiple cloud vendors. I am not sure what is causing az aks get-credentials to have trouble merging the new set of credentials into the existing ones, but I suspect one of the existing non-AKS entries contains data that az aks get-credentials does not expect.
As noted above, running az aks get-credentials -g GROUPNAME -n CLUSTER -f FILE to output the new credentials into a new file, then hand-merging the contents of that file into my real ~/.kube/config, works fine as a workaround. I will continue to do that until this problem is fixed.

@davidack Apologies for the late reply. Could you confirm if you have tried  --overwrite-existing parameter and checked if that helps ? Awaiting your reply.

I am unable to test this with the current Azure CLI version (2.36.0) on the machine where I originally reported it, because Azure CLI on MacOS only supports Homebrew as an install method, and Homebrew no longer supports MacOS 10.13.
I did try this using Azure CLI 2.36.0 on one of my newer machines, which is running MacOS 11.6. On that machine, az aks get-credentials is able to merge the new set of credentials into my existing large .kube/config.
I am not sure if it is the newer MacOS version or some change in the newer Azure CLI version that fixed the problem, but in any case it seems to be working OK now. Thanks for checking back.

@davidack Thanks for getting back. We will now proceed with closure of this github issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.


facebook login isn't visible

Hi, in my application i use the login with password + the facebook login.
If i use
https://github.com/studiointeract/accounts-basic
the facebook login button appears in the login form. But if i use this plugin instead of the other, the facebook button doesn't appear.
I have to use this plugin beacuse i'm using materialize but i need also of the facebook login button.
What i have to do?
Thanks!

+1

+1


fix onRequestPermissionsResult crash & add arabic support

Description

fix  issue #51 "Fatal Exception: java.lang.ArrayIndexOutOfBoundsException".
add Arabic support.

@jaiselrahman can you make this as quick release please? :)

Thank you for the pull request and sorry for the late reply.
Can you send just the commit https://github.com/jaiselrahman/FilePicker/pull/53/commits/5dc1f13ba4f6322fe6363251287f061aa8617e0f  in the pull request as the paging support has been added?


Make threading issue easier to reproduce

I think we can make the issue easier to reproduce: the threading issue triggered by not having the volatile keyword.
Inspired by what @Xenoprimate mentioned in issue #5763 and this stackoverflow post I have some suggestions to make to the example code:
Change DoWork to:
            public void DoWork()
            {
                bool work = false;
                while (!_shouldStop)
                {
                    work = !work; // simulate some work
                }

                Console.WriteLine("Worker thread: terminating gracefully.");
            }

Because doing IO on the worker thread seems to trigger fresh reads for some reason.
Then change the Thread.Sleep from 1 to 500.
Now on to the good parts:

Run the program in both debug and release. Everything works as expected. :)
Now remove the volatile keyword:

In Debug mode the program still works
In Release mode it hangs forever.

As an interesting bonus:
When asking a friend to run this code he ran it as part of our unit testing system and the issue was triggered in debug and with a thread sleep of only 1. This confirms that some variation is possible and to be expected.
Hopefully this example helps trigger the behavior for the most number of people.

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

ID: d226514d-c7cb-0d1b-b9dc-fb791399dc47
Version Independent ID: aa5d1078-7f89-98ce-7fdc-107c9ed3eebc
Content: volatile - C# Reference
Content Source: docs/csharp/language-reference/keywords/volatile.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Well done @joaoportela !  I verified that behavior on my machine with .NET Core 2.2.
Do you want to submit a PR with this fix? If not, I'll add this to our backlog.


TypeError Plugin: netlify-cli: memoizeOne is not a function

Describe the bug
Latest release v3.21.8 fails with TypeError Plugin: netlify-cli: memoizeOne is not a function.
To Reproduce
Steps to reproduce the behavior:

yarn add netlify-cli
yarn run netlify

Configuration
N/A.
Expected behavior
No error.
CLI Output
(node:3023) TypeError Plugin: netlify-cli: memoizeOne is not a function
module<EMAIL_ADDRESS>task: runHook init
plugin: netlify-cli
root: /private/tmp/nc/node_modules/netlify-cli
See more details with DEBUG=*
(Use `node --trace-warnings ...` to show where the warning was created)

Additional context
This is really a continuation of #2202. Opening a new issue because that one is closed and claimed to be fixed, which is confusing. (Btw I can confirm that v3.21.7 did fix the issue for a while, but it's broken again.)

Also experiencing this issue, on this post in the forum another user said downgrading to an older version resolved it. What's the best way to do that ? (sorry if that's a dumb question).
https://answers.netlify.com/t/typeerror-plugin-netlify-cli-memoizeone-is-not-a-function/36458/4

I'm pinning to v3.21.6 for now.
yarn global add<EMAIL_ADDRESS># or
npm install -g<EMAIL_ADDRESS>

Hi everyone,
Thanks for reaching out.
<EMAIL_ADDRESS>should fix this problem.
Please let us know if this does not.

Works now, thanks.


proposal: cli to install & run kubernetes applications as turnkey solutions

ProblemStatement: As a kubernetes engineer I would like to install, run & verify one or more applications (that includes required controllers) on my preferred kubernetes cluster. I want a cli that can help me achieve my goal.
NOTE: I do not want to deal with HELM or other tools to achieve this.
NOTE: I do not want to configure yamls to achieve my purpose.
NOTE: This cli should provide me an opinionated solution.
Use Cases

Ability to test my K8s controller using a mocked kubernetes control plane
Ability to install, run & test Cassandra on OpenEBS local PV on GKE
Ability to install & verify CStorPool operator in my E2E pipelines

Notes:

Mocked k8s control plane implies k8s api server binary, etcd binary & kubectl talking to each other

Draft Thoughts on CLI

This is an example of installing, running & verifying cassandra that makes use of OpenEBS local PV.

$ kubectl nocode download cassandra-pack
$ kubectl nocode run cassandra-pack
$ kubectl nocode verify cassandra-pack


Look into why graphql.planetary.pub has stopped syncing

The pubs data is out of date, indicating it isnt syncing properly

@mixmix can you give an update on this?

Sorry I didn't get to this. Have been waiting for the new system pattern to settle before debugging.
I assume graphql.planetary.pub is gonna be wound down as we move to a more room-centric model with many smaller graphql servers?
Let's watch how they behave

It seems like they are behaving. Closing.


fix URLs in README

Morning! @tachylatus the links in the README appear to work except for the release-notes directory link but it looks like there's now branch conflicts with this PR if you don't mind resolving and then I can approve it. Thank you!

@dcwhittaker I've fixed the merge conflicts.


kind::bool_ & kind::double_ can be renamed kind::boolean & kind::floating

The reason I named them bool_ and double_ is because it resembles the actual C++ type as closely as possible

That's good, but they are a bit... ugly. Trailing underscores are most often used for private members, so it just doesn't look right.

Yes I know what you mean. It isn't perfect. But the universe isn't perfect. Here are three ways of doing it:

kind::bool
kind::bool_
kind::boolean

1 is of course the best but also a non-starter since it uses a reserved keyword. That leaves 2 and 3. Number 2 more closely resembles the keyword as it is the same as the keyword just with an underscore appended to it. Number 3 lacks the underscore but requires thought.
For the double, things are more clear:

kind::double
kind::double_
kind::floating

As before, 1 is ideal but also a reserved keyword and thus unusable. Number 2 is the same name as the type but with an (ugly) underscore appended. Number 3 is really odd here because it is not the word double. Instead it is the word "floating" which is only distantly related to the type it describes.
I think there is a strong case to be made that the currently chosen set of identifiers is the best. The "rules" for figuring out the name of a kind are very simple:

write the type that you want (e.g. int64, string)
if that type is a reserved keyword, append an underscore

Now compare that with your proposed rules:

write the type that you want, unless
if that type is bool then write boolean, otherwise
if that type is double then write floating

Your rules require memorizing more trivia than the current rules.
I think my case is strong, but sometimes there are naming objections or suggestions which come up during a Boost formal review. During the formal review period, if you still feel that the names should be changed, then you should bring this up and link this issue so that others can weigh in on it.

Nothing actionable here


NumericUpDown Double

I try to enter double in NumericUpDown, but controls don't allow me to enter dot.

@l1pton17 you can specify the Culture with the latest alpha


Added a playbook to restart a deployment on certain prometheus alert

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Nikhil Maheshwari seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

We now support this logic so closing


Fix arrayless flatdeletion

Bug fixes:

When a row lacked any array column it was getting deleted.
Flattened limit wasn't being correctly applied.
Because of the way the result schema was being calculated, SHOW TABLES queries failed when executed using the driver.

+1


Error Running GSM8K Script

I tried to run the gsm8k script on my one a4000 machine with a llama-7b checkpoint. I followed the instructions to install llama environment. My running command is

CUDA_VISIBLE_DEVICES=0 torchrun --nproc_per_node 4 --master-port 1054 run_gsm8k.py --llama-ckpt $LLAMA_CKPTS/7B --speedup-confidence-batch-size 2

The error message is:

Traceback (most recent call last):
File "run_gsm8k.py", line 5, in
Traceback (most recent call last):
File "run_gsm8k.py", line 5, in
from rap.models import QueryLlama
File "/share/data/ripl/fjd/RAP/rap/models.py", line 8, in
from rap.models import QueryLlama
File "/share/data/ripl/fjd/RAP/rap/models.py", line 8, in
class QueryLM(ABC):
File "/share/data/ripl/fjd/RAP/rap/models.py", line 14, in QueryLM
class QueryLM(ABC):
File "/share/data/ripl/fjd/RAP/rap/models.py", line 14, in QueryLM
def query_next_token(self, prompt: list[str]):
TypeError: 'type' object is not subscriptable
def query_next_token(self, prompt: list[str]):
TypeError: 'type' object is not subscriptable

Any idea how to fix it? Thanks!

Please try any of the following:

Use python>=3.9, since list[...] is supported since then
Just delete the type hint : list[str], including other appearances in the code

Feel free to reopen if still not resolved.


npm echo Error 'Segmentation fault'

Hi there!
My operating system is Alpine Linux 3.6.2-extend_x86_64 and docker-ce version is 17.07.0-ce.
Any other container is running ok, but nodejs related images(container) not.
When I create a ghost blog container with library/ghost image, it will give a error echo Segmentation fault and the ghost container always can not start.
I create and into a nodejs contianer found that any command related npm like npm -v will echo 'Segmentation fault'.
There is a log fragment by another user who has the same problem. log fragment

This looks like https://github.com/nodejs/docker-node/issues/588.

@yosifkit Thank you!
Yes, it is the same problem and I fix it by build a new image with a Dockerfile:
FROM ghost:alpine
RUN apk add --no-cache paxctl && paxctl -cm `which node`

This issue should close, but I want to konw is it the paxctl will be added to the ghost officail image in the feature?


Add Rollout addon offline installation steps

Signed-off-by: zhaohuihui<EMAIL_ADDRESS>Description of your changes
add Rollout addon offline installation steps
I have:

[x] Read and followed KubeVela's contribution process.
[ ] Update sidebar.js if adding a new page.
[ ] Run yarn start to ensure the changes has taken effect.

Special notes for your reviewer

@ctran

please sign the DCO


Plans to publish pep440-rs

Hi! Love this. Thanks for building it!
I see the issue you have tagged in the readme as the blocker. Is there any update on that? Also is there anything I can do to help get this project published?

just waiting on https://github.com/PyO3/pyo3/pull/2786 to get merged an published, which i forgot to update

it's now published published on crates.io and pypi


Request new release for jvm target 11

The latest version 1.5.0 requires jvm target 17 which is a breaking change, as https://github.com/Tlaster/PreCompose/pull/122 is merged, a new release is important.

1.5.1 has just released with fallback to jvm target 11

@Tlaster Thanks!


e2e: enable local IE testing

specify IE driver in capabilities object for protractor;
if --browser=ie, download/update IE driver and temporarily add its
path to process.env.path;

Closes #452
@gghai, OK to merge?

Please go ahead with checkin


[KYUUBI #5478][AUTHZ] Support Hudi ShowHoodieTablePartitionsCommand

Why are the changes needed?
To close #5478. Kyuubi authz support hudi ShowHoodieTablePartitionsCommand

ShowHoodieTablePartitionsCommand: https://github.com/apache/hudi/blob/master/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/command/ShowHoodieTablePartitionsCommand.scala

How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?
No

ping @bowenliang123 @yaooqinn @pan3793

Codecov Report

Merging #5481 (ef276f3) into master (48bdc7d) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #5481   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         588     588
  Lines       33466   33466
  Branches     4401    4401
======================================
  Misses      33466   33466

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

Thanks, merged to master


Feature/22 wasm create service layer to facilitate data caching

implemented back button will come back to caching service

close #34


Fixed Usage in README.md

I read the usage and followed the step-by-step tutorial, but I couldn't create a new GitHub App. We must specify the Webhook URL or uncheck the box. Thanks!

Great catch! Thank you very much @yokra9!


"fleetctl start unit" can not work  when unit was crash or killed -9

I did the following attemption:
$ fleetctl list-units
UNIT		MACHINE			ACTIVE	SUB
hello.service	d16f9669.../<IP_ADDRESS>	active	running
$ fleetctl status hello.service
hello.service - Hello World
Loaded: loaded (/run/fleet/units/hello.service; linked-runtime)
Active: active (running) since Thu 2015-01-15 20:52:04 CST; 1h 16min ago
Main PID: 14128 (bash)
CGroup: /system.slice/hello.service
├─14128 /bin/bash -c while true; do echo "Hello, world"; sleep 1; done
└─20307 sleep 1
$ sudo kill -9 14128
$ fleetctl list-units
UNIT		MACHINE			ACTIVE	SUB
hello.service	d16f9669.../<IP_ADDRESS>	failed	failed
$ fleetctl list-unit-files
UNIT		HASH	DSTATE		STATE		TARGET
hello.service	e55c0ae	launched	launched	d16f9669.../<IP_ADDRESS>
$ fleetctl start hello.service
$ fleetctl list-units
UNIT		MACHINE			ACTIVE	SUB
hello.service	d16f9669.../<IP_ADDRESS>	failed	failed
I found  the unit file status is still launched. So "fleetctl start hello.service" is not working.
$ fleetctl list-unit-files
UNIT		HASH	DSTATE		STATE		TARGET
hello.service	e55c0ae	launched	launched	d16f9669.../<IP_ADDRESS>
If I use "fleetctl stop hello.service" first, then "fleetctl start hello.service" is  working.  Is this fleet policy or bug? thanks.

@holmes86 most of the fleetctl commands (load, start, stop, unload) are declarative, not imperative. When you run fleetctl start foo.service, you're instructing the cluster that the unit foo.service should transition to the "launched" state: this transition requires the unit be scheduled somewhere and started. Once the transition happens, subsequent calls to fleetctl start foo.service will have no effect.
We've discussed the confusing naming here: https://github.com/coreos/fleet/issues/1025

@bcwaldon I mean fleet change ACTIVE and SUB status when unit was exception. why not change unit DSTATE or STATE?

DSTATE is set by the user, not in reaction to any events in the cluster. STATE is a concept defined independently of the states exposed by any given instance of systemd [0], so it is not logical for that field to change in response to a systemd-level event. The ACTIVE and SUB fields are populated by systemd, not by fleet, so they change based on what's happening in the actual instance of systemd running your unit.
[0] https://github.com/coreos/fleet/blob/master/Documentation/states.md


Redirect to favicon.ico after login

Gitea version (or commit ref): 1.12.4
Git version: 2.26.2
Operating system: Gentoo/Linux
Using the Gentoo init script for OpenRC
Gitea built using the (gentoo ebuild)
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No

Log gist:
N/A

Description
When REQUIRE_SIGNIN_VIEW is enabled, any request to any URL (even if non existent) seems to redirect to /user/login and places a cookie containing the original URL to redirect after login.
This seems fine at first glance.
However, there is a race condition here: for example, if the browser requests a favicon.ico before the user is logged in, a cookie will be set with redirect_to=%2Ffavicon.ico.
Then, if the user navigates to the login page and submits the login form, they will be redirected to /favicon.ico.
Screenshots
NA
Steps to reproduce
To reproduce this, you need to make the browser request a favicon.ico. This only seems to happen when the page you request does not explicitly include an icon.
The easiest way I found is trying to open the swagger UI while logged out, and then login.

Make sure to set REQUIRE_SIGNIN_VIEW=true in app.ini
Use an incognito window, preferably in a new profile (e.g. chromium --incognito --user-data-dir=$(mktemp -d)
Make sure you are logged out; open developer tools (F12), go to "Network" tab and check "disable cache" and "preserve log"
Navigate to https://your-gitea/api/swagger
This should return HTTP 403. Since the content is JSON, chrome will hopefully also request a favicon.ico. (if you don't see chrome requesting a favicon, please try again or this won't work)
The favicon request will return HTTP 302 with location: /user/login and will include a set-cookie: redirect_to=%2Ffavicon.ico; Path=/. Since this is the not the page you navigated to (it's only a resource), the redirect is ignored. However, the cookie is set.
Manually navigate to the gitea home page (type the URL in the address bar) and click on the login button in the menu
You will now be at the login page (the URL is https://your-gitea/user/login?redirect_to=). The cookie remains unchanged.
Type a valid username and password, and click Sign in
This will make a POST to /user/login, including the cookie cookie: lang=en-US; i_like_gitea=d96a44102eca1a57; _csrf=Kbj8-XwU1_qYqn5_7cyXyYafpgs6MTYwNjc4NjMyMTgwNzY5NDg2Ng; redirect_to=%2Ffavicon.ico. Therefore, the reply will be HTTP 302 with location: /favicon.ico
Your browser will comply and redirect you to https://your-gitea/favicon.ico instead of a sane page like the homepage.
Additionally, since there is no favicon.ico, a 404 plain-text error is returned with the following headers:

content-type: text/plain; charset=utf-8
content-length: 19
content-disposition: attachment; filename=favicon.png

This leaves the user with an ERR_INVALID_RESPONSE message after login.
(Also note that it returns filename=favicon.png even though I requested a favicon.ico which supposedly doesn't exist. Maybe this is another issue in itself.)
Endpoint replies

REQUIRE_SIGNIN_VIEW=true

# curl -sI <IP_ADDRESS>:3000/favicon.ico | grep -E "HTTP|redirect|Location"
HTTP/1.1 302 Found
Location: /user/login
Set-Cookie: redirect_to=%2Ffavicon.ico; Path=/
# curl -sI <IP_ADDRESS>:3000/asdfasfadsfasdfdsfaf | grep -E "HTTP|redirect|Location"
HTTP/1.1 302 Found
Location: /user/login
Set-Cookie: redirect_to=%2Fasdfasfadsfasdfdsfaf; Path=/

REQUIRE_SIGNIN_VIEW=false

# curl -sI <IP_ADDRESS>:3000/favicon.ico | grep -E "HTTP|redirect|Location"
HTTP/1.1 404 Not Found
# curl -sI <IP_ADDRESS>:3000/asdfasfadsfasdfdsfaf | grep -E "HTTP|redirect|Location"
HTTP/1.1 404 Not Found

Simple workaround if using nginx as a reverse proxy for gitea:
	location /favicon.ico {
		return 302 /favicon.png;
	}

This prevents it from setting the cookie.

I guess we need to ensure that all static resources are never subjected to that redirect, e.g. by routing them before the route that triggers the login redirect mechanism.

同样，每次登陆就跳到favicon

@lanybass Please speak English.


(Ci) Bug fix with mypy check | adding stub file

Relates

#1465

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 89bf79ee37c35a639f29dc0235b68a23b33c4918
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: ca9603b352c8e8dead4ca239e05d78e8b9043ab9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Refactor message header alignment and "raw" types to fit a clear pattern

Is your feature request related to a problem? Please describe.
Aligned version of message headers currently in SB, shows different handling of the base type.
https://github.com/nasa/cFE/blob/9804b59eb34154b10bb2946bde7654f3d27f4b85/fsw/cfe-core/src/inc/cfe_sb.h#L150-L163
Describe the solution you'd like
See discussion below.
Describe alternatives you've considered
None.
Additional context
Brought up as part of #777/#998 review.
Requester Info
Jacob Hageman - NASA/GSFC (from CCB discussion)

Regarding discussion that came up in today's CCB - this may also affect/determine how we proceed with #689:
This primarily deals with the alignment of messages.  The fundamental concern - and what needs to be distinguished here - is what module is actually imposing the alignment requirement.
Messages themselves only need to be aligned sufficiently for the types they actually contain (i.e. a message containing only uint8 bytes needs no special alignment, while a message containing any uint32 value needs 32 bit alignment, and so forth.
However, the software bus implementation (CFE_SB) internally aligns all of its buffers, and it uses the (historical) CFE_SB_Msg_t* pointer type as its interface type for both CFE_SB_SendMsg() and CFE_SB_RcvMsg() as well as the other msg introspection calls.
Therefore -- since the extra alignement is actually driven by CFE_SB and its underlying implementation - my recommendation is to put the unions here.  So:

The msg module should provide CFE_MSG_Message_t, CFE_MSG_TelemetryHeader_t, and CFE_MSG_CommandHeader_t types.  In this case the CFE_MSG_Message_t should be the actual structure type, not a union.

The CFE_SB module should provide CFE_SB_Msg_t, and CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t - which are simply  variants of those types provided by MSG but aligned properly for usage by software bus.  (that is, they are union-ized here)

The "raw" messages in apps should be defined using only types from the MSG module (i.e. in the respective *_msg.h file).

But when instantiating a message buffer locally for sending (i.e. in telemetry-generating code) the buffer should be instantiated using CFE_SB_Msg_t as its base type.  This way - in the event that the telemetry message has low alignment requirements - it will not generate a warning, because the buffer will be aligned to CFE_SB_Msg_t level - if this is greater than the TLM message itself.

On the other hand - When receiving a command - it is OK to cast from CFE_SB_Msg_t* to the actual command type - because this should only be a downgrade in alignment, never an upgrade.

The reason for following this pattern is that the sizeof() operator - when applied to the message types - will work as expected, and won't reflect extra padding that might have been added due to runtime padding for SB.
For a specific case where this distinction matters - consider CFE_ES_Restart_t.  This command has a single uint16 payload, and is traditionally expected to be 10 bytes total in length when the CMD header is added.
When sent across the software bus interface - the buffer will actually be 12 bytes or maybe even 16.  But that extra padding is only necessary for runtime data alignment - the extra bytes should not be interpreted as part of the message.
If these patterns I'm suggesting above are followed then things should mostly work (I think, or at least as well as can be expected using C structs and not a proper data dictionary implementation.

If you aren't using the aligned headers to define the message, wouldn't we also need to make the raw headers end on the most restrictive boundary, so the payload will always start in the same location (and not break CFE_SB_GetUserData)?  For "instantiating a message buffer locally" do you mean do the union between the raw message and CFE_SB_Msg_t within the app?  So really going back to the whole message buffer pattern that I took out in nasa/sample_app#104 for example?
typedef union
{
    CFE_SB_Msg_t       MsgHdr;
    SAMPLE_APP_HkTlm_t HkTlm;
} SAMPLE_APP_HkBuffer_t;

If that's the case, don't CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t really have no use anymore?
@ejtimmon - sounds like this isn't settled yet...

we also need to make the raw headers end on the most restrictive boundary, so the payload will always start in the same location (and not break CFE_SB_GetUserData)?

If I'm interpreting correctly - you are referring to the possibility of a gap occurring between the header as defined by MSG and the actual start of data (for instance if the TLM header is 12 bytes and the payload contains a double which requires 8 byte alignment).
As you noted in #777 though, the CFE_SB_GetUserData() function is inherently fragile and one should use the actual structure definition for this.  (e.g. use something like offsetof(mymessage_t, Payload) or &mymessage->Payload).  It is only an issue when you are making assumptions about how the compiler may (or may not) pad something out.  (as an aside I'm tempted to say CFE_SB_GetUserData should be deprecated, but in the event that one moves toward a dictionary-based solution, it becomes more possible to implement this reliably).
Pedantically speaking there also no guarantee that using CFE_SB_CmdHdr_t (the union) will avoid implicit padding any better than using CFE_MSG_CommandHeader_t (actual type) does.  Because avoiding padding requires an assumption about what the worst-case alignment will be for any payload will be - and accounting for it in the respective struct/union.    So - as long as MSG types already account for this by explicitly making  CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t to be a multiple of 8 bytes instead of 4 - this is probably as good as can be done when using C structs to define messages.  Adding a union won't avoid padding any better.
Again, padding avoidance is based on the assumption that the most probable worst case payload alignment will be 8 bytes, and it is therefore unlikely that the compiler will add extra padding in this case.  "assumption", "probable" and "unlikely" being the key weasel-words here, because technically any compiler can add padding or extra alignment wherever it sees fit on the particular architecture it is compiling for.  (i.e. it is entirely plausible that some FPU out there works better with 16 byte alignment, particularly if you use the long double type).  As a C programmer there is no way you can absolutely guarantee that a compiler won't add padding between struct members - the C standard says that this is platform-defined.
My main point is - this can result in an unanticipated implicit padding gaps either way - whether the unionized "CFE_SB_CmdHdr_t" was used or the base structure was used to declare the struct.  Compilers can do what they want - developers can only guess at the most likely memory padding, nothing is guaranteed about the layout of bits within a struct that isn't directly specified in the C standard.
Hence why I would prefer to see the struct definitions in the *_msg.h header file(s) convey the intent of what the cmd/tlm packets contain in the simplest terms possible.  The distinction is that we are not sending a CFE_SB_Msg_t or CFE_SB_CmdHdr_t or CFE_SB_TlmHdr_t on the network - this exists only for internal use within SB for transport/API purposes.  The CFE_MSG_CommandHeader_t or CFE_MSG_TelemetryHeader_t are what is intended to be sent/received, so this is how the msg structure should be defined in an app.

If that's the case, don't CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t really have no use anymore?

I think so - these seem like surplus types now given the current design.  Perhaps we can remove them.

If my memory serves correctly, the main reason for making CFE_SB_Msg_t into a union in the first place was for two reasons:

Provide a modicum of type safety for messages - Having a non void pointer base type means the user must actually declare their buffer using this type or get a compiler warning.  The alternative being void* where anything goes (no warnings, no type checks).
But because of that type check - having it be a union with uint32 increased its alignment requirement and thus avoided a compiler warning in the command handlers - i.e. when casting from CFE_SB_Msg_t* to something else - this increases the likelihood that this would be a downgrade (or equal) in alignment, not an upgrade - so no warning.

We could simplify a lot by just making the interface type in SB to be void* instead of CFE_SB_Msg_t* - but this would have no type checking or enforcement so its bad for safety - but it sure would make things simpler.

We need to resolve this ASAP for Caelum... realizing it's not a perfect solution, we need something good enough.  I understand the lack of guarantee, but a fix that works for most systems is better than something that is "broken" on all of them.  I do dislike CFE_SB_GetUserData, but even if we deprecated it I also think it's undesirable to have the payload start at different locations depending on the alignment requirements of the payload (makes ground system databases more complex).

Anyways, in an attempt to move forward I'll implement the alignment suggestion on top of #998 so we can at least get a PR in to discuss and see how the pattern settles.

My recommendation is to use the envelope/post office analogy as I mentioned in https://github.com/nasa/cFE/pull/998#pullrequestreview-530210054

CFE SB API deals with envelopes
CFE MSG API deals with contents

Upon receipt of a message we examine the MsgID and "open" the envelope.  This can be a typecast to the CFE_MSG type (typically CFE_MSG_CommandHeader_t) or (preferably) by accessing the correctly-typed member within the CFE_SB_Msg_t envelope.
When sending a message we "wrap" the raw message in the envelope for sending.
The CFE SB API should only have abstractions for those basic items which are required for message transport and delivery such as its MsgId and size.  These should operate on a CFE_SB_Msg_t* type.  This would also be the right place to deal with any authentication data if we ever need to implement that for security reasons.
The CFE MSG API should have abstractions for all other metadata - timestamps, command codes, checksums, and any other relevant info.
The CFE SB API should probably also provide helpers (either macros or helper functions) to facilitate the proper "packaging" and "unpackaging" of these raw messages.  Right now this is basically just a typecast that is performed directly in the apps (e.g. casting of CFE_SB_Msg_t* to its specific command type within the command dispatcher) but this could be more refined.


Use GitHub Actions as the CI

Move away from Semaphore and use GitHub Actions as our CI for this project.
Semaphore doesn't have Python 12 yet for us to test against, it will update versions once a month, and with GitHub Actions we can run it on the CI today.
[skip changeset]

This will need some more work, like auto cancel builds when you push a commit on a branch when another build for an older commit is already running and scheduled builds so we can see if anything fails (and connect that to Slack if possible). But this PR is here to gauge our interest.

I am certainly interested! I think this can replace our Semaphore usage as-is.
Before merging this we should actually see the checks fail as well as succeed. I don't trust it being green unless I have also seen it being red.

@unflxw here's an example with a failure: https://github.com/appsignal/appsignal-python/actions/runs/7492027115


Update tests

Update tests to handle new input type

Is this the same as #26 ?

Same as issue 26


Safari cannot handle URL-encoded download filenames

Content-Disposition header wasn't encoded in accordance with RFC 5987.


Use wmctrl instead of xdotool for getting classname/activating windows

Thanks for making this, I was about to write something like this and found it.
You are saying in readme that xdotool is the slow part. And I noticed that you already run wmctrl -l.
If you run wmctrl -l -x instead, that will return the list of windows with class name which should be parsable. You can also use wmctrl to activate windows with -a option, e.g. wmctrl -x -a class.name.goes.here and then you don't need to run xdotool at all.

Hell yeah!
Didn't know about wmctrl -l -x. I'll do some test on my old 32bit laptop to see how it compared to xdotool search -classname <classname>, the hog.
Regarding using wmctrl to switch to the target window, I need to see how it behaves.
The xdotool windowactivate <window_id> isn't affecting performance much.


Creating a map with mixed datasets sharing alternatives generates wrong sql

Share a user A dataset X with user B.
Create with user B a map with previous X shared dataset and a dataset B owns.
Both layer sql are rewritten to select * from A.table

cc @javitonino @gfiorav @xavijam

Setting assignee because of worklog update ;-)

Thanks, totally forgot.


google-benchmark wrap linker error due to missing transitive dependency

Describe the bug
Targets which declare a dependency only google_benchmark_main_dep fail to link because the transitive dependency on lib_benchmark is missing.
To Reproduce
I created a minimal working example here:
https://github.com/hazelnusse/meson_google_benchmark.git
$ git clone https://github.com/hazelnusse/meson_google_benchmark.git
$ cd meson_google_benchmark
$ meson setup builddir
$ cd builddir
$ ninja -v
Output can be found here:
https://github.com/hazelnusse/meson_google_benchmark/blob/main/output.txt
Expected behavior
The link step should succeed. Instead, in order to make it succeed, both google_benchmark_main_dep and google_benchmark_dep need to be added as dependencies. This seems counter-intuitive since the dependency declarations already do that here:
https://github.com/mesonbuild/google-benchmark/blob/1.5.2/meson.build
I see 4 options:

my meson.build is incorrect
I should not expect the transitive dependencies to be captured in this case
There is some problem with the wrap file or the patch source file
Meson's transitive dependency handling has some sort of bug

If it is option 1 or option 2, I am curious what is wrong.

I've moved this from the main meson issue tracker to the wrap tracker, this looks like a very easy fix, patches are definately welcome here.

Hi @dcbaker where is the wrap tracker?


Avoid ?

Does the C++ specification allow you to forward-declare the std::hash class template? If so, this would let you specialize it without the need to include <functional>.

AFAIK there is no exceptions in the standard to the "no user declarations in std" rule for std::hash.

What about if you include <typeindex> (which is very lightweight) instead of <functional> to get the primary template?

This would work to write the specialization for variant but we would still need the implementation of std::hash for builtin types which is in <functional>.

UGH you're right.


CommonMark blockquote example 221 fails

According to commonmark spec

It is a consequence of the Laziness rule that any number of initial >s may be omitted on a continuation line of a nested block quote:

>>> foo
> bar
>>> baz

should be render to (By spec.commonmark.org and github comment system)

foo
bar
baz

but current implemention gives:

foo
bar

baz

Thank you for report. We test CM spec automatically. Need to check why this sample was missed.

The example in the origin issue is not the example 221 in CM.
The example 221 is:
>>> foo
> bar
>>baz

Notice that there is not space before baz. markdown-it works fine with example 221.

Yeah, example 221 use two >, but I test it with three, that's why autotest didn't report this issue.

Fixed, thanks for reporting.


[OCP-4.X] unused var in inventory

OPENSHIFT_INSTALL_VPC_CIDR_BLOCK does not appear to be in use anywhere else.
https://github.com/openshift-scale/scale-ci-deploy/blob/master/OCP-4.X/inventory.example#L39

@chaitanyaenr PTAL


FullScreen: Adjusting padding issue on account details page

Padding bottom should be increase to 20px in the section below!

WORK IN PROGRESS

Related PR posted!


Update documentation

closes #124
view updated user guide at https://www.samuelfangjw.com/tp/

Ready for review

Some minor style issues.

Thanks for pointing this out,  wasn't aware there was a style guide for md files. Will look through the style guide again and update.


Installing cli fails

OS: macOS 10.15
Go Version: 1.15
I just tried to install the cli to get the latest updates but it fails to build:
$ go get github.com/kyleconroy/sqlc/cmd/sqlc
# github.com/kyleconroy/sqlc/internal/engine/dolphin
../../go/src/github.com/kyleconroy/sqlc/internal/engine/dolphin/convert.go:343:14: n.Offset undefined (type *"github.com/pingcap/parser/ast".FuncCallExpr has no field or method Offset)

Not sure if this is just an issue with my environment

Duplicate https://github.com/kyleconroy/sqlc/issues/654


pull progress bar

As we do not know exactly the size to be downloaded, I've created the classes below to control the progress bar. It is based on the number of layers that will be downloaded. Each block is updated according to the progress of each layer.
DownloadPart
file: cli/download_part.js
Is a download block. Receives the status message docker remote and sends the tick to the progress bar.
SmartProgressBar
file: cli/smart_progress_bar.js
Manages a progress bar without knowing total size previously
output example
$ azk start python34

azk: ↑ starting `python34` system, 2 new instances...
azk: ✓ checking `azukiapp/python:3.4` image...
azk: ⇲ downloading `azukiapp/python:3.4` image...
azk: ⇲ comparing registry layers and local layers...
azk: ⇲ pulling 11/29 layers.
     [============================================     ] 91% 10/11
azk: ✓ completed download of `azukiapp/python:3.4`

how to test
# run progressbar tests
$ azk nvm grunt --grep='progressbar'

# download images
$ adocker rmi -f saitodisse/10mblayers; azk shell -i saitodisse/10mblayers
$ adocker rmi -f mongo; azk shell -i mongo

This closes #234, closes #119 and closes #317;

LGFM! Very beautiful. :100:

Tested in mac: LGTM