Get adb and fastboot builds from source (via robotnix)

Describe the bug

One of the few things on my systems that isn't portable over to aarch64 is adb and fastboot.
One of the few remaining non-free packages (or at least requiring explicit opt-in) on my system is adb.

It seems like Robotnix already is building adb/fastboot, but I don't know how much work it would be to extract a 'minimal' piece that could be pulled into nixpkgs.
It would be nice to have these natively, freely built from source.
cc: @danielfullmer (and since I haven't had a chance to say it anywhere else, awesome talk, awesome project, thank you!)

related (for reference): https://github.com/NixOS/nixpkgs/issues/75603

I'll summarize some of the discussion in other issues (#75603 and https://github.com/danielfullmer/robotnix/issues/25), as well as the relevant slides from my talk at Nixcon 2020.
I currently have two working ways to build adb / fastboot from source:
Use the existing android build system (ckati + soong).
Try this using by building sdk/adb.nix from https://github.com/danielfullmer/robotnix

Pros:

maintainability (at least, when compared with option 2)

Cons:

Uses Google's prebuilt toolchains.  I believe these toolchains only support aarch64 via cross-compilation from x86_64.
Likely requires checking out all (or nearly all) of the AOSP source tree (>40GB of source) to build. See #75603 for additional discussion

Difficulty: 4/10.  This really just requires extracting a more minimal build script from robotnix (and potentially refactoring the updater script).

Use my experimental projects to convert the android build system to a nix-native build system: blueprint2nix + soongnix.
Try this by building the fastboot or adb attribute from the default.nix of https://github.com/danielfullmer/soongnix

Pros:

Uses nixpkgs toolchains. Supports native compilation on aarch64-linux. (Tested working on my pinebook pro)
Only requires checking out the subset of source that is actually needed to build the projects.

Cons:

Maintainability.  Each new android release may require serious updates for blueprint2nix and soongnix.

Difficulty: 7/10.  blueprint2nix / soongnix would need to updated to work with the latest platform-tools git tag from Google.  These projects also currently have nearly no documentation or comments throughout the source code. The code could likely use some serious refactoring as well.  Some (seemingly optional) modules used by adb or fastboot are currently unimplemented in soongnix (e.g. java_binary_host for fastdeploy)

I'm not exactly sure if/when I'll have time to work on this myself in the near future.  But, I'm available to help answer any questions if anyone else is interested in working on this..

I forgot about this issue. Since #124992 we have an android-tools package that should satisfy both 1. (AArch64 support) and 2. (built from source).
We can leave this issue open if we want a Robotnix build as well (which is a really awesome project! Thanks @danielfullmer) but at least we now have a viable alternative to the pre-built binaries from Google :)

I'll go one step further and close this issue. The android-tools fork in use solves the underlying problems AFAICT.
Feel free re-open if there is a reason we would want to use the soongnix tooling instead :).


fix: #14 to support some arch linux

fix: #14 by changing the root dir.

Rollback and fixed

LGTM, thanks.


StageRecap bugs out with RiskUI

As the title says. The gameplay itself is fine and nothing crashes, however on interaction with either the teleporter or any portal to leave the stage, the prompt to interact with either teleporter/portal disappears and the objective to take them gets checked off. However, the player's current gold is never converted to experience and they're never teleported anywhere, stranding them on the stage. Unfortunately, I have no log that would help out in bug fixing but if this could be looked into that'd be greatly appreciated.
Below is the mod list and what it looks like in practice.

It's hard to tell since I had to compress the GIF under 10mb, but the interact prompt that appears at the bottom right disappears upon interacting with both the portal and the teleporter.

Edit: This has been fixed with the most recent patch of StageRecap. Closing issue.


Support custom backends in forge crate

Component
Forge
Describe the feature you would like
❓ What are we doing?
We are building tooling that gives smart contract developers ways to create and verify zk-proofs in Solidity. We are utilizing revm with custom revm inspectors. We want to make it easy for developers to test their smart contracts using forge.
The problem
We have a few inspectors that are necessary for the vlayer contracts to work. There's currently no easy way to run forge with with custom revm inspectors.
What we had to do was copy a bunch of code to replace backend in test executor with our custom backend with custom revm inspector.
Unfortinately, it's very deep in the call tree with nested builders, which made it necessary for us to copy a lot of very slightly modified forge code.
main
  TestArgs::run()
    TestArgs::execute_tests()
      multi_runner = MultiRunnerBuilder::build()
      TestArgs::run_tests(multi_runner, ...)
        multi_runner.test()
          for each contract:
          multi_runner.run_test_suite()
            executor = ExecutorBuilder::build()
            runner = ContractRunner(executor, ...)
            runner.run_tests()
              for each test:
              runner.run_unit_test()
                executor.call()
                ... more calls inside executor...
                executor.call_with_env()
    We need to patch this
    ----->        Create backend with inspector
                  backend.call()

This makes it hard for us to keep it up to date with the current foundry codebase.
Another big issue is the fact that there're a lot of large files in the /bin dir, with makes it impossible to import them in any other crate.
Can it be improved?
Yes, we believe so! And we think that it would be useful for other developers, if the forge crate was more modular and easy to use.
But talking solely about our use case, here's a minimal set of changes that would make our life much easier:

Move (ideally all files from) /bin/cmd to /src/cmd
Create Executor and ExecutorBuilder traits
Accept dyn ExecutorBuilder in TestArgs::execute_tests and pass it down the tree
ExecutorBuilder::build() would return Box<dyn Executor>
ContractRunner would also keep Box<dyn Executor>

This way, we'd only need to reimplement call_with_env in the executor.
We will be happy to discuss this proposal and prepare the pull request!
Additional context
No response

Post-1.0  we are planning to work on forge-builder to greatly improve the way developers can interact with Foundry as a library. I would be supportive of an external PR but please be aware that it is not an area currently in focus pre-1.0.


BUG:mne.create_info function mismatch channel names with DigMontage

Describe the bug
In 0.19.2, mne.channels.read_montage has been replaced by mne.channels.make_standard_montage, which returns a DigMontage object.
However, mne.create_info with DigMontage can not match channel names in a case insensitive manner.
Steps to reproduce
import mne

montage = mne.channels.make_standard_montage('standard_1005')
info = mne.create_info(
    ['FPz'], 100,
    ch_types=['eeg'], montage=montage)

Actual results
~/miniconda3/envs/ml/lib/python3.7/site-packages/mne/utils/_logging.py in wrapper(*args, **kwargs)
     88             with use_log_level(verbose_level):
     89                 return function(*args, **kwargs)
---> 90         return function(*args, **kwargs)
     91     return FunctionMaker.create(
     92         function, 'return decfunc(%(signature)s)',

~/miniconda3/envs/ml/lib/python3.7/site-packages/mne/io/meas_info.py in create_info(ch_names, sfreq, ch_types, montage, verbose)
   1798         for montage_ in montage:
   1799             if isinstance(montage_, (str, Montage, DigMontage)):
-> 1800                 _set_montage(info, montage_)
   1801             else:
   1802                 raise TypeError('Montage must be an instance of Montage, '

~/miniconda3/envs/ml/lib/python3.7/site-packages/mne/channels/montage.py in _set_montage(info, montage, update_ch_names, set_dig, raise_if_subset)
   1564             info_names=set([info['ch_names'][ii] for ii in _pick_chs(info)]),
   1565             montage_names=set(ch_pos),
-> 1566             raise_if_subset=_raise,  # XXX: deprecated param to remove in 0.20
   1567         )
   1568

~/miniconda3/envs/ml/lib/python3.7/site-packages/mne/channels/montage.py in _check_ch_names_are_compatible(info_names, montage_names, raise_if_subset)
     91                 '\nYou can use `raise_if_subset=False` in `set_montage` to'
     92                 ' avoid this ValueError and get a DeprecationWarning instead.'
---> 93             ).format(n_ch=len(not_in_montage), ch_names=not_in_montage))
     94         else:
     95             # XXX: deprecated. to remove in 0.20 (raise_if_subset, too)

ValueError: DigMontage is a only a subset of info. There are 1 channel positions not present it the DigMontage. The required channels are: {'FPz'}.
You can use `raise_if_subset=False` in `set_montage` to avoid this ValueError and get a DeprecationWarning instead.

It seems reasonable to me to make the name comparisons case insensitive. Okay with you @agramfort ?

We doing the API change as mne requires unique channel names in case sensitive manner it was decided that case match should be enforced to avoid bugs. Maybe a param to set_montage could be added though. Something like match_case or something

I meet the same bug.
Channel naming rules are not uniform.
Such as standard_1005:   Fp1 & CP1.

you mean our default standard montage are not uniformly named?
maybe a parameter in set_montage is the way.

you mean our default standard montage are not uniformly named? maybe a parameter in set_montage is the way.

There isn't a 'set_montage' method for mne.create_info.
When I only need info instance what can i do?

@TanTingyi  larsoner has already added match_case keyword to _set_montage, which is called by set_montage and create_info. Although we can not directly use match_case in create_info, now we can create raw object as follows:
import numpy as np
import mne

data = np.random.rand(1, 200)
info = mne.create_info(['FPz'], 100, ch_types=['eeg'])

raw = mne.io.RawArray(data, info)

montage =  mne.channels.make_standard_montage('standard_1005')
raw.set_montage(montage, match_case=False)

@larsoner  Would you mind adding match_case option to create_info function in your PR?

PR from @larsoner is now merged. I guess this one can now be closed.

Agreed. I'd prefer to deprecate the montage argument in create_info rather than add new montage-related kwargs to it. Then set_montage would be the one place to go for this functionality.

sounds good to me

Thank you @drammock for the explanation. I have followed your instructions and it now works. Thanks again!

Hello there,
I'm trying to run set_montage in the BCI200 raw data but I'm still getting errors related  to channels name. Raw data channels are:
ch_names: Fc5., Fc3., Fc1., Fcz., Fc2., Fc4., Fc6., C5.., C3.., C1.., ...
Thus, when I ran
montage = mne.channels.make_standard_montage('biosemi64')
print(montage)
<DigMontage | 0 extras (headshape), 0 HPIs, 3 fiducials, 64 channels>
<bound method set_channel_types of <RawEDF | S003R03.edf, 64 x 20000 (125.0 s), ~9.8 MB, data loaded>>
raw.set_montage(montage)
I get the following error
ValueError                                Traceback (most recent call last)
 in ()
----> 1 raw.set_montage(montage)
 in set_montage(self, montage, match_case, match_alias, on_missing, verbose)
2 frames
/usr/local/lib/python3.7/dist-packages/mne/utils/check.py in _on_missing(on_missing, msg, name, error_klass)
870     on_missing = 'warn' if on_missing == 'warning' else on_missing
871     if on_missing == 'raise':
--> 872         raise error_klass(msg)
873     elif on_missing == 'warn':
874         warn(msg)
ValueError: DigMontage is only a subset of info. There are 64 channel positions not present in the DigMontage. The required channels are:
['Fc5.', 'Fc3.', 'Fc1.', 'Fcz.', 'Fc2.', 'Fc4.', 'Fc6.', 'C5..', 'C3..', 'C1..', 'Cz..', 'C2..', 'C4..', 'C6..', 'Cp5.', 'Cp3.', 'Cp1.', 'Cpz.', 'Cp2.', 'Cp4.', 'Cp6.', 'Fp1.', 'Fpz.', 'Fp2.', 'Af7.', 'Af3.', 'Afz.', 'Af4.', 'Af8.', 'F7..', 'F5..', 'F3..', 'F1..', 'Fz..', 'F2..', 'F4..', 'F6..', 'F8..', 'Ft7.', 'Ft8.', 'T7..', 'T8..', 'T9..', 'T10.', 'Tp7.', 'Tp8.', 'P7..', 'P5..', 'P3..', 'P1..', 'Pz..', 'P2..', 'P4..', 'P6..', 'P8..', 'Po7.', 'Po3.', 'Poz.', 'Po4.', 'Po8.', 'O1..', 'Oz..', 'O2..', 'Iz..'].
Consider using inst.set_channel_types if these are not EEG channels, or use the on_missing parameter if the channel positions are allowed to be unknown in your analyses.
However, they are EEG channels and positions must be known, thus:
Datasamples I'm using are officially available at https://physionet.org/content/eegmmidb/1.0.0/
Must I rename channels as per indicated in this thread?

You can try mne.datasets.eegbci.standardize(raw) before setting the montage or you can just remove the channel names like so raw.rename_channels({ch: ch.replace('.', '') for ch in raw.ch_names})

Thanks a lot Alex.
The while point of renaming channels was because I'm stuck at
ica.plot_components(outlines="skirt")
that returns the error
WARNING:root:Did not find any electrode locations (in the info object), will attempt to use digitization points instead. However, if digitization points do not correspond to the EEG electrodes, this will lead to bad results. Please verify that the sensor locations in the plot are accurate.
RuntimeError                              Traceback (most recent call last)
 in ()
----> 1 ica.plot_components(outlines="skirt")
7 frames
 in plot_ica_components(ica, picks, ch_type, res, vmin, vmax, cmap, sensors, colorbar, title, show, outlines, contours, image_interp, inst, plot_std, topomap_args, image_args, psd_args, reject, sphere, verbose)
 in plot_ica_components(ica, picks, ch_type, res, vmin, vmax, cmap, sensors, colorbar, title, show, outlines, contours, image_interp, inst, plot_std, topomap_args, image_args, psd_args, reject, sphere, verbose)
/usr/local/lib/python3.7/dist-packages/mne/channels/layout.py in _auto_topomap_coords(info, picks, ignore_overlap, to_sphere, sphere)
688         # Get EEG digitization points
689         if info['dig'] is None or len(info['dig']) == 0:
--> 690             raise RuntimeError('No digitization points found.')
691
692         locs3d = np.array([point['r'] for point in info['dig']
RuntimeError: No digitization points found.
Then when I googled it, I found set_montage command as a solution for that.  https://github.com/mne-tools/mne-python/issues/4835
However, that didn`t solve the problem. at least in my case
My final goal is to create a script to parse EEG data and build a file structure, which I could easily apply any sample of EEG signals/data from channels Fp1 and Fp2 in the following classification algorithm to decide where left or right command was   imagined by the subject.
https://github.com/wmvanvliet/neuroscience_tutorials/blob/master/eeg-bci/3. Imagined movement.ipynb

Does that make any sense at all?

I'm trying to read an eeg data in csv format, but the digitized points still at zero. The acquisition equipment is the Emotiv epoch one, with 14 channels. The channel names are: ch_names = ['AF3', 'F7', 'F3', 'FC5', 'T7', 'P7', 'O1', 'O2', 'P8', 'T8', 'FC6', 'F4', 'F8', 'AF4']. Anyone with the same issue?

@Riccellisp @iuri could you post to https://mne.discourse.group ? These seem more like usage questions than bugs

please use the forum and not the bug tracker to discuss usage questions.
thank you

Message ID: @.***>

i am getting the same error and I've spent a lot of time to find answers or examples to solve the error but i couldn't find helpful resources. How was your issue resolved ?

Hello Maitha,
As indicated by Alexandre, we must use the forum for this type of questions.
https://mne.discourse.group/ https://mne.discourse.group/
https://mne.discourse.group/t/parsing-edf-dataset-to-mat-matlab-file-structure/5071/6 https://mne.discourse.group/t/parsing-edf-dataset-to-mat-matlab-file-structure/5071/6

On Jul 31, 2022, at 3:04 PM, Maitha Khanji @.***> wrote:
i am getting the same error and I've spent a lot of time to find answers or examples to solve the error but i couldn't find helpful resources. How was your issue resolved ?
—
Reply to this email directly, view it on GitHub https://github.com/mne-tools/mne-python/issues/7169#issuecomment-1200471937, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEZZ5J5E7H7V6KMQXQHPW3VW25ZXANCNFSM4KBNGL2A.
You are receiving this because you were mentioned.


[downloader/f4m] Improved metadata handling

Metadata may be missing, and correctly write the metadata tag length

I don't have any samples, but Section 11.17 of F4M manifest specification (available at https://www.adobe.com/devnet/hds.html) defines  as optional ("This element MAY be present"). So I think youtube-dl should handle a missing metadata gracefully.

This should indeed improve youtube-dl without any downside I'm seeing. Merged, thanks!

I don't know how I could miss all that info when I read the spec. Thanks!


COMP: Do not explicitly instantiate Array in itkArrayTest

Array is intantiated and used throughout the toolkit and this
instantiation conflicts on macOS arm64 in symbol exports resulting in:
ld: warning: direct access in function 'itk::MetaDataObject<itk::Array >::GetMetaDataObjectTypeName() const' from file 'lib/libITKCommon-5.2.a(itkMetaDataObject.cxx.o)' to global weak symbol 'typeinfo name for itk::Array' from file 'Modules/Core/Common/test/CMakeFiles/ITKCommon1TestDriver.dir/itkArrayTest.cxx.o' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
ld: warning: direct access in function 'itk::MetaDataObject<itk::Array >::GetMetaDataObjectTypeName() const' from file 'lib/libITKCommon-5.2.a(itkMetaDataObject.cxx.o)' to global weak symbol 'typeinfo name for itk::Array' from file 'Modules/Core/Common/test/CMakeFiles/ITKCommon1TestDriver.dir/itkArrayTest.cxx.o' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
ld: warning: direct access in function 'itk::MetaDataObject<itk::Array >::GetMetaDataObjectTypeInfo() const' from file 'lib/libITKCommon-5.2.a(itkMetaDataObject.cxx.o)' to global weak symbol 'typeinfo for itk::Array' from file 'Modules/Core/Common/test/CMakeFiles/ITKCommon1TestDriver.dir/itkArrayTest.cxx.o' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
ld: warning: direct access in function 'itk::MetaDataObject<itk::Array >::GetMetaDataObjectTypeInfo() const' from file 'lib/libITKCommon-5.2.a(itkMetaDataObject.cxx.o)' to global weak symbol 'typeinfo for itk::Array' from file 'Modules/Core/Common/test/CMakeFiles/ITKCommon1TestDriver.dir/itkArrayTest.cxx.o' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.

@imikejackson take a look!

wow, nice find! +1


Update request: jacktrip 1.10.1 → 2.2.2

Package name: jacktrip
Latest released version: 2.2.2
Current version on the unstable channel: 1.10.1
Current version on the stable/release channel: 1.10.1
[X] Checked the nixpkgs pull requests

Notify maintainers
@iwanb
https://github.com/jacktrip/jacktrip/releases/tag/v2.2.2

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

Add a :+1: reaction to issues you find important.

Revised ticket to 2.2.0 release

Revised ticket to 2.2.2 release


Included makefiles

Included Makefiles in all the individual folder which can build automatically using travis CI whenever a PR arrives.

Hey @Panky-codes thanks for this PR :blush:
Also are there any changes that i need to know ?

Hi @Bhupesh-V . Nothing specifically. Whenever a PR arrives, travis will compile all the .cpp file in the folders (algo, stack, queue, vector, list ). In case if you add a new folder in the future, add the makefile in that new folder from any of the already existing folder and add the lines at the end in the travis.yml file under script as follows: && cd ../<new_folder> && make.
Right now we are using c++11 (In make file see the flag -std=c++11) to compile to code which I think covers all the basic STL functionalities. You could also see the travis output from my PR to see how this script is working. :) I hope it answers your questions.


Website is fully white.

Just forked the portfolio, and im really impressed. I've always been awful at Web Dev so seeing such a good portfolio for completely free is just amazing.
I forked the repo, edited stuff to fit my needs, basically just removing most the socials and the sample experience and education text.
Yet when I upload it to github pages, the website is completely white. I'm not sure if this is an easy fix or a known issue, but a response back wouild be nice.
heres the website if you wanna see... https://xdNuggets.github.io/portfolio
and heres the repo: https://github.com/xdNuggets/portfolio

Thank you a lot for the kind words. Means a lot.
Let me help you with the issue you are facing. I believe it's an easy fix.
Can you please check the source of your forked repo by visiting https://github.com/xdNuggets/portfolio/settings/pages? If it's not Github Actions change it to Github Actions please.

It indeed wasn't Actions. I assumed the deployment part was unchanged so I went with my usual deployment process

Let me know after changing it. Don't forget to trigger the workflow manually or by pushing any new commit.

I changed it to github actions. Do I select Jekyll or just static html?

Ah nvm! It works! Thanks for the quick responses, and thank you again for such a good portfolio for absolutely free

Glad I could help. The Actions part is pretty new to GitHub so haven't able to document it fully. Thanks for bringing the issue up. Will add it to the readme soon.


[enhancement] Support Lambda Execution via Docker containers

I think something like this can be used: https://github.com/lambci/docker-lambda
to provide true support for lambda functions...I think right now it expects everything to be in a exec'able string, which means it doesn't support multiple files as noted in https://github.com/spulec/moto/issues/981

This is done right?

yep!


[feature] npm builder: provide tarball name as workflow output

We may want to provide the name of the tarball created, so that users can download it if they want to.

Will handle this on #888


any wiki available for connecting with dialogflow?

Hi. if any documentation available for integrating with dialogflow?

Hello @soorajpazeekal, we will probably add this in the future, but there's a lot of other overdue maintenance we have to do first.
HINT: take a look at the Sagas file and Redux Saga to see how you can do said connection.
Feel free to Fork and make a PR if you'd like to contribute.


test: set thread count before grt to use parallel repair_antennas

@maliberty I see this failure in the CI:
./test/regression jpeg_sky130hd
jpeg_sky130hd (tcl) *FAIL* RSZ::hold_buffer_count   66 >   50

But I can't reproduce it on my GCP machine. We have another source of possible non-determinism since this metric is generated before grt. I can't download the artifacts of this design from the CI. Is there a way to access it? It would be nice to see where the discrepancies begin.

Does https://jenkins.openroad.tools/job/OpenROAD-Public/job/PR-5345-head/3/artifact/test/results.tgz contain it?

Does https://jenkins.openroad.tools/job/OpenROAD-Public/job/PR-5345-head/3/artifact/test/results.tgz contain it?

It has only for sky130hs/gcd.

Closing it due to instability in the results. Will create a new PR from the latest master branch to check if the instability persists.


handle the emit in try/catch

issue #46

may I know when the next version will go live?

First week in 2018.


add a gif on editing a photo using the starter project

include import
a couple edits
zoom
and export

2cf9f500cc6d02c109670cf404be0798a229ba42


[S3 Upload] Add content disposition field

When uploading to S3, it would be great to override the default inline value with attachment. S3 objects cannot have their metadata changed after and so it quires copying the s3 object with any metadata changes.
Related: https://github.com/livekit/egress/issues/518

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix link redirects to wrong site

The "Write queue storage messages (Output binding)" was redirecting to blobstorage example site instead of queue storage example.
Opened feedback #48717

@gyulaszucs : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for the heads-up, @gyulaszucs!
#sign-off


All letters assigned to new layers

Every letter is added to a new layer. Add all letters to a single layer, and add a layer flag to select which layer.

Resolved in v0.0.7


every thing is f9 what was the issues....

Challenge Link to External Pages with Anchor Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.12 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp

 View more cat photos

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

@junaidshah111 thanks for the issue but if you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room.
Hint: your code is wrong. Please take a look at the test that is failing and make sure you have the right link

You need an a element that links to http://freecatphotoapp.com

Closing as not an issue with the site. Happy coding!


Unable to pair with chip-all-clusters-app with chip-tool pairing (on same RPi)

Problem
Unable to pair with chip-all-clusters-app with chip-tool pairing, when running both on the same raspberry .
I am not sure if I am doing anything wrong here.

Commit sha: deb227fcd3dd560e11b14fb9ad46e86b638dcfb0

Error messages (full logs attached at the end):
chip-all-clusters-app:
[1629756499.336188][51812:51812] CHIP:SVR: OnSessionEstablishmentError: ../../src/protocols/secure_channel/PASESession.cpp:741: CHIP Error 0x0000003A: Key confirmation failed
[1629756499.336207][51812:51812] CHIP:SVR: Failed in SPAKE2+ handshake

chip-tool pairing:
[1629756499.318308][51817:51822] CHIP:SC: Failed to verify peer's MAC. This can happen when setup code is incorrect.
[1629756499.318397][51817:51822] CHIP:EM: Piggybacking Ack for MsgId:00000000 with msg
[1629756499.318561][51817:51822] CHIP:SC: Failed during PASE session setup. ../../src/crypto/CHIPCryptoPALOpenSSL.cpp:1273: CHIP Error 0x000000AC: Internal error
[1629756499.318635][51817:51822] CHIP:TOO: Secure Pairing Failed
[1629756499.319556][51817:51822] CHIP:TOO: Pairing Failure: ../../src/crypto/CHIPCryptoPALOpenSSL.cpp:1273: CHIP Error 0x000000AC: Internal error
[1629756499.320012][51817:51817] CHIP:TOO: Run command failure: ../../src/crypto/CHIPCryptoPALOpenSSL.cpp:1273: CHIP Error 0x000000AC: Internal error

Steps to reproduce:

Clone connectedhomeip repo
run source scripts/activate.sh
run sudo apt-get install build-essential libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev

This was needed to install missing dependencies for zap-tool

run ./scripts/tools/zap/run_zaptool.sh
run ./gn_build.sh
run ./out/debug/standalone/chip-all-clusters-app
Open new shell on same RPi and run ./out/debug/standalone/chip-tool pairing onnetwork 0 20202021 3840 ::1 5540

Logs (CLI Output)
chip-tool pairing.log
all-cluster-app.log

Thanks to @tcarmelveilleux for suggesting me to run rm /tmp/*.ini, it fixed the issue


Modification suggestion

Hello,  thanks for your codes first. I am a starter in audio aspect, and I really insterested in this. It is really helpful to read your code. And it seems that the problem you mentioned is that the parameter mu in utils.py line 35 is an int type, which leads to the output are all zeros, and you can just change it to float32 will give a nice results.

Thanks! Let me try that and share the results.


Error: node_modules directory is missing. Please run npm install in your project directory.

 Choose a template: expo-template-blank
? Choose which workflow to use: managed
√ Please enter a few initial configuration values.
Read more: https://docs.expo.io/versions/latest/workflow/configuration · 100% completed
[13:42:46] Extracting project files...
[13:42:50] Customizing project...
[13:42:51] Installing dependencies...
npm ERR! path git
npm ERR! code ENOENT
npm ERR! errno ENOENT
npm ERR! syscall spawn git
npm ERR! enoent Error while executing:
npm ERR! enoent undefined ls-remote -h -t<EMAIL_ADDRESS>npm ERR! enoent
npm ERR! enoent
npm ERR! enoent spawn git ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR!      C:\Users\008\AppData\Roaming\npm-cache_logs\2019-02-19T08_13_23_010Z-debug.log
[13:43:23] Process exited with non-zero code: 1
[13:43:23] Set EXPO_DEBUG=true in your env to view the stack trace.
C:\Users\008\Desktop\firstdb>cd AwesomeProject
C:\Users\008\Desktop\firstdb\AwesomeProject>npm start

@ start C:\Users\008\Desktop\firstdb\AwesomeProject
expo start

[13:45:00] Starting project at C:\Users\008\Desktop\firstdb\AwesomeProject
[13:45:06] Expo DevTools is running at http://localhost:19002
[13:45:06] Opening DevTools in the browser... (press shift-d to disable)
[13:45:12] Error: node_modules directory is missing. Please run npm install in your project directory.
[13:45:12] Couldn't start project. Please fix the errors and restart the project.
[13:45:12] Set EXPO_DEBUG=true in your env to view the stack trace.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ start: expo start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      C:\Users\008\AppData\Roaming\npm-cache_logs\2019-02-19T08_15_12_166Z-debug.log
C:\Users\008\Desktop\firstdb\AwesomeProject>npm install
npm ERR! path git
npm ERR! code ENOENT
npm ERR! errno ENOENT
npm ERR! syscall spawn git
npm ERR! enoent Error while executing:
npm ERR! enoent undefined ls-remote -h -t<EMAIL_ADDRESS>npm ERR! enoent
npm ERR! enoent
npm ERR! enoent spawn git ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR!      C:\Users\008\AppData\Roaming\npm-cache_logs\2019-02-19T08_15_37_721Z-debug.log

am facing the same problem

Hey. It's working now.
I am not install git in my PC.
download and install in your PC
Link : https://gitforwindows.org/
On Sat, Feb 23, 2019 at 9:25 PM wackyizzy<EMAIL_ADDRESS>wrote:

am facing the same problem
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/23528#issuecomment-466663952,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AscMHkCwErjuIXB_FzvF_8ARo0kzL9OFks5vQWRugaJpZM4bCeMn
.

I'm still seeing this issue when deploying from gitlab pipeline. Is there a reason this isn't installed as an npm  dep?

I faced the same issue. My problem was that git was not installed on my windows environment. Installing git solved my issue.

facing the same problem and i have git installed in my windows

me to i have this error when i try to run expo start,someone can help me.thank you

Hello there,
I just signed the petition "Marvel Indian: We want Tony Stark's / Iron
Man's old Tamil dubbed voice" and wanted to see if you could help by adding
your name.
Our goal is to reach 10,000 signatures and we need more support. You can
read more and sign the petition here:
http://chng.it/6qgHybJWXG
Thanks!
Mahinthan

please help

a

please help for this problem

Did you find how to avoid this problem?

a

please help for this problem

Did you find how to avoid this problem?

yes, i uninstall the react-native and reinstall, and change USB cable change, its work.

I avoided this problem with the environment PATH variable. I added twos news paths on it. Look:


Documentation

Please add to service files:
Documentation=file:/usr/share/doc/xlogin/README

Maybe a nice manpage is also helpful for users.
Documentation=man:xlogin(8) file:/usr/share/doc/xlogin/README

Currently, we don't install the README and there is no man page. I am unsure whether writing one would be worthwhile as xlogin was really designed as a stop-gap solution. Using .xinitrc-defined sessions is a bit weird in a systemd setting and the way we managing environment variables (e.g. DISPLAY) and bus addresses shows just how imperfect the current landscape is. In my opinion this has more to do with lack of consensus in the systemd camp than with xlogin, which is also indicated by the fact that xlogin has been the best `workaround' for about four years now!
The part of xlogin that is not a hack is the x-daemon script which actually does something useful in an elegant way. This script, however, is under 100 bytes(!) so maybe it can serve as its own documentation. Admitted, every line uses something clever and non-trivial. In the end, it just spawns X, passing the command line arguments, and terminates when X is started. Does anyone desire documentation for that?


Add Feature to Open AWS Console in Private Tab

Description
I propose adding a feature to allow users to open the AWS Console in a private tab. This enhancement will enable users to manage multiple AWS accounts simultaneously without session conflicts. Currently, opening multiple AWS accounts requires continuous logging in and out, which is cumbersome. By enabling a private tab option, users can seamlessly switch between accounts, improving efficiency and user experience.

@sergiovm I'm assuming you're on chrome. I could add a setting to open profiles in incognito mode, but this would only allow you to open 2 consoles at a time, 1 in incognito and 1 in normal.
If you need to open more than 2 consoles, I recommend checking out the firefox version of this extension that uses tab containers.

@WTFender thanks for your reply. In fact, I'm using Safari but I guess it behaves like Chrome in this regard. Thanks for the tip, I'll try Firefox with this extension.

@WTFender In safari, aren't incognito tabs independent of each other? It seems like you should be able to have multiple incognito tabs, each with its own separate aws login

@kamatsuoka thank you - I'm light on safari knowledge! Maybe I can pass the SSO session to private tabs.
The safari extension is just a port of the chrome one for now, so I'm going to close this, but will think about it as more safari issues pop-up.


Defect prediction replication in CI pipeline

DPT is meant to be used straight from IDE but also in an automated manner inside a CI/CD pipeline.
In https://radon-h2020.github.io/radon-defect-prediction-api/ is described how to use DPT using it as a docker container, by accessing its API. Although this functionality has been tested successfully passing a standalone file to the API calls, the  possibility of passing the whole csar file as input to the DPT has not been implemented, and therefore additional steps of file handling have to be added.
Could you provide information on how to access and identify the files inside the csar structure that makes sense to be tested for defects in an automated manner? Practically, how the defect prediction plugin performs this functionality.
Enchancement
Similarly to the DPT plugin in IDE, where we invoke the prediction to a csar archive and we receive defects analysis for each tosca file inside it, it is desired to have the same functionality when running DPT using the containerized version, by just providing the csar deployment package.

@AlexSpart ok, I ve moved this issue to https://github.com/radon-h2020/radon-defect-predictor and I will close it once the feature is ready. So you'll get notified.


Fixed #27755 -- Added ModelAdmin.get_inlines() hook.

Continue the pr 7920, for ticket 27755.
Test, release note, and versionadded annotation added.

@pope1ni, Thanks for your comment :+1:
Done

@pope1ni Sorry!. Fixed.

@felixxm Sorry for the delay.
Requested changes are done.

@felixxm Done!

@hramezani Thanks!
@pope1ni Does it look good for you? Any final comments?


How about the training time-consuming of hard samples mining module?

Hi @Tobias-Fischer , may I ask about the time-consuming during training? especially how the cache parameter effects the hard sample mining module at here.
I try to use this scheme in my own retrieval task but it cost much more time.
By the way, I use the default cache parameters

Are you asking how long it took to train the model? Also, you say it takes “much more time” - compared to what?

Oops, I mean the time consumption when the hard sample mining is used, compared to the training without mining. I found there are two triplet samples generation modes, especially for negative sample generation, with or without network forward. Code at here. Besides, I want to know more about the cached subset style mine strategy, can you give some related papers? Thanks.

Hi,
As written in the comments, if there is no network used, then there is no hard negative mining being done: https://github.com/QVPR/Patch-NetVLAD/blob/cba383478e6b656c76a8c5034a3681f69ab59ddc/patchnetvlad/training_tools/msls.py#L421-L423
The relevant code is from https://github.com/mapillary/mapillary_sls which also has a nice paper attached where you can read more.


Select All (Ctrl +A)  #Feature Request

Feature Request?

[x] Does this feature exist in another mail client or tool you use? Yes it does

In the Mail pane I cannot use Ctrl+A to select all mails. I wanted to delete all the mails from Deleted Items folder and tried this but it didn't work. I hat to use Shift+Select to select all the mails.
If we can enable Ctrl+A to select all visible mails it will be more user friendly.
Regards
Senasum.

Just came here to request the same thing. Ctrl+A highlights the mail address of the currently selected email, not the expected behaviour of selecting all emails.

Hey, yes, unfortunately this is a known issue we're working on right now. I'm merging this issue into #886. All of the select-all issues are related to an ongoing fix to better address keymaps and menus. We'll be updating #886 with the status of that fix.

Thanks, we look forward to use a more developed version soon.

Any news? If ctrl+a / cmd+a is hard to implement at least a "select all button"?


Using 10.0.11 CONTOSO, I cannot configure as in Instruction - "Set up the main account"

Following the instructions, I used DEMF.
Instructions are: Go to General ledger > Chart of accounts > Accounts > Main accounts.
Use the Quick Filter to find records. For example, filter on the Main account field with a value of '170150'.
In the list, click the link in the selected row.
Click Edit.
Select or clear the Require payment reference check box.
Click Save.
I can find the Main Account '170150', but the "Require payment reference check box" is not visible for me.
Has this behaviour changed since the writing of this Document?

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

ID: 598d636f-acab-3386-42c5-e28a85d2e60b
Version Independent ID: 5c300c83-a872-6569-6748-64bcacc261d5
Content: Set up mandatory payment references - Finance | Dynamics 365
Content Source: articles/finance/localizations/tasks/set-up-mandatory-payment-references.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @EvgenyPopovMBS
Microsoft Alias: epopov

@BernhardRaether, thanks for providing this feedback.
@EvgenyPopovMBS, please update the content to address this feedback.

@neserovleo, please look at the documentation issue.

Bug 477865 has been logged to track this issue.


ControlAddIn Examples

For one of my customers I am creating some prototype ControlAddIn objects using an extension.
I am very impressed with the ease of deployment and so is my customer with the result we got in two days of programming.
The examples I can find online are helpful and it got me started. They all seem to use jQuery.
In one of the threads about ControlAddIn objects Dennis Frostlander suggests to use Angular or React.
Can you provide a simple example of both?
I would love to also use TypeScript but that probably also requires a compiler to be added to the build process that converts the ts files to js files.
Can you please share your thoughts on how and where this is relevant and which frameworks you would suggest using?
Where do you think is the flipping point where instead of ControlAddIn development we should use Asp.NET Core instead?

Please elaborate further, where does dotnet core come into place in the context of a js addin? Dotnet core is a server side framework. And if you want to use frameworks like angular or react, there is no difference when using them in the context of a control addin. So you can just follow the wide range of tutorials out there. And I would suggest vue.js.

I was afraid that the last sentence was ambiguous.
With ControlAddIns you can customize the look and feel. For example, render a factbox or a subpage in HTML5.
However, if your customer requires a full-page UI like a POS I can imagine you would use a Single Page Framework like Angular and the API.
But in the Windows client (aka RTC) there is an option to start the client with the ControlAddIn full-size. This has been implemented as per request of companies like LS Retail.
I am curious what the recommendation would be and what other peeps are using.
You can connect an Angular app directly to the API endpoints, but you can also use ASP.Net Core as a server-in-the-middle. I've used both and I am curious what others do/suggest.

@dennisfrostlander , any updates on this?

@StanislawStempin , @kalberes ?

Unfortunately, we don't have samples showing how to use Angular/React or other similar frameworks with control add-ins.
It's hard to give a general recommendation on which framework to use - it really depends on the use case, and as it was mentioned here, there are lots of comparisons/tutorials online. Angular or React, by the way, are not the only options. Typically a new frontend framework appears by the time you finish reading the documentation for the previous one :)
Using TypeScript should be no problem with the popular frameworks. They usually incorporate typescript compilation within their development tools.

Ok, thanks. We'll start trying out different options and share our experiences back with the community. I think it's going to be the next big thing to get this to work properly and make Business Central do miracles.


[K12] Picklist metadeploy installer health check

POC to act as "health check" for picklist values for customers in a nice experience.
https://metadeploy-stg.herokuapp.com/products/k12/b60fd21d00eaa48a2268b300fe7e967369cc218d
Very open to presentation/implementation suggestions.
Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata
Testing Notes

@MatthewBlanski @davidmreed Since it seems like this won't get merged, is it ok if I close this out?

@MatthewBlanski @davidmreed Since it seems like this won't get merged, is it ok if I close this out?

Let's close it out and purge the branch, then.

Per comment above, I will close this PR and purge the branch


Code coverage

Gwen-web doesn't seem to have a code coverage feature (see cypress).

We had never considered that someone might want to use Gwen for code coverage purposes.

We had never considered that someone might want to use Gwen for code coverage purposes.

Understood, it's a completely separate concern but an essential part of a comprehensive test platform. I wonder if it is possible to integrate as an addition like cypress does?

You can instrument your code, run Gwen tests over it, and view your coverage without requiring any special integration in Gwen. And you can use any coverage tool of your choice without being locked into whichever one a potential integration solution we introduce might be coupled to. Being a separate testing concern, our preference would be to keep it out of Gwen and allow users freedom of choice over which coverage tools they wish to use.

Would be great if you could try it and let us know how it goes.

@JohnSColeman have you had a chance to try above? Would be keen to know how it went.

We'll take this as a comment. Closing.

Sorry, too busy with other work to focus on this.


app.commands forcing typescript error check

Steps to reproduce:

developing a normal plugin in VS Code

When trying to refer to this.app.commands a typescript error is shown.

From what I can guess, this is because app.commands is not intended to be a part of the API, even though it can be run in the console, and is still accessible in typescript.
Whatever the reason though I believe that if it IS accessible it should be documented, and it should be visible.
The reason is simple: otherwise it's considered deep magic. You can only do it if you can discover you can do it with the proper research, which isn't fair to any developer.
ie. if the reason is to discourage plugin developers from having dependencies on other plugins, that should not be up to the API to decide. Maybe community plugins shouldn't have such dependencies, but personal use plugins should very much encourage the automatic use of plugin commands.
and so it follows that if there is any other deep magic obsidian can do that is not exposed, it should be.

You can access pretty much anything inside Obsidian's internals but that doesn't mean we want to add it to the typescript documentation for everyone to use. The documented APIs are intended for long-term support and we usually don't make changes to them without significant effort for backward compatibility, whereas private APIs are not intended for public use and are subject to change anytime.


Add option to disable cron hooks

Add an option to disable the cron hook routes.
eg. c::set('gcapc-cron-hooks-enabled', false);
See also: #11

Closed by commit https://github.com/blankogmbh/kirby-git-commit-and-push-content/commit/2118b281fc4353888ea7f630c2876693c23bd073


Default process disallows visiting text nodes

The default behaviour is an optimization to not visit text nodes.   There was a note, saying
     THIS IS NON CONFORMANT BEHAVIOUR JUST FOR DISCUSSION OF A POSSIBLE CHANGE TO THE
     SPECIFICATION.

ISO Schematron 2016 has not changed, therefore the behaviour should default to visit.

Discussed in #46.


enable deep linking for linux

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
pull request for
Linux: Support protocol deep linking #627
Issue link
Linux: Support protocol deep linking #627
Test Cases
Additional Notes

@dunkla @jasonblais Do we have a scheme like mattermost://? IS this a feature we support at all?
Just to understand things properly.
I'll test the commit by building the app on my linux box to see if this change anything in practice.

Thanks @wget!
Yes, Mac and Windows support mattermost:// links - when used, it opens the Desktop App instead of a browser window
Related PRs:

https://github.com/mattermost/desktop/pull/616
https://github.com/mattermost/desktop/pull/680
https://github.com/mattermost/desktop/pull/785

Hi @dunkla, thank you for submitting the PR!
I tried this but the deep linking didn't work for me.
Environment:

Ubuntu 18.04

Steps:

Extract .tar.gz
Execute ./create_desktop_file.sh then copy Mattermost.desktop to ~/.local/share/applications/.
Open the desktop app.
Enter the URL in Chromium: mattermost://community.mattermost.com/core/channels/desktop-app
The app should be activated, but that didn't happen.
xdg-open also didn't perform.

Are there any extra steps to use deep linking?

@yuya-oc The patch is working properly, the issue seems to come from the electron app:
(master)[wget@hermes ~]$ xdg-open mattermost://community.mattermost.com/core/channels/desktop-app
(process:4556): Gtk-WARNING **: 18:40:55.597: Locale not supported by C library.
        Using the fallback 'C' locale.
(electron) 'app.makeSingleInstance(cb)' is deprecated. Use 'app.requestSingleInstanceLock() and app.on('second-instance', cb)' instead.
In development mode, deeplinking is disabled
Added Extension:  React Developer Tools
/usr/bin/mattermost-desktop: line 3:  4556 Segmentation fault      (core dumped) electron /usr/lib/mattermost-desktop/app.asar --disable-dev-mode "$@"

@wget The log is weird. app.makeSingleInstance() is no longer used in the master branch. And the deep linking should be disabled in development build even when --disable-dev-mode is used. So I guess that a wrong executable (probably your local repository's one) was launched by xdg-open.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@dunkla Do you have questions about the above provided feedback?

I didn't know you still wanted feedback from me. I thought @yuya-oc said it all.
Nevertheless, i tested it again (fresh build with above patch) and it works for me.

@yuya-oc any other feedback on this PR?

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@dunkla Sorry for the delay. Would you please merge master intro this PR? There has been a lot of changes recently that might have also affected this PR. After that we can test again.

I merged master into the pr.
npm test was fine, the built packages work as expected.

Thanks @dunkla!
@wget @yuya-oc Please review, when you have the time

@hanzei @dunkla Hello everyone. Sorry for the delay.
The xdg open and mattermost:// links are now working fine from my side. HOWEVER the channel change doesn't happen automatically.
If I'm in the default channel Town-Square on the Mattermost Community instance for example, and I open mattermost://community.mattermost.com/core/channels/desktop-app, the app is being launched, but we are not being switched to the right Mattermost instance and the right channel.
If this is not intended (a bug), it will need to get fixed first before merging.

@dunkla Sorry for the delay. Would you please merge master intro this PR? There has been a lot of changes recently that might have also affected this PR. After that we can test again.

And for the record, it's better to rebase rather than merging for this use case :)

@wget I'm lost there, because i cannot reproduce your problem.
Anybody else here, that can confirm/reproduce the problem?

@deanwhillier Can you help with debugging this?

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@yuya-oc or @deanwhillier wondering if you have time to provide troubleshooting/debuggin assistance?

I'll give it a try locally and see what I get!

Checked with the latest 4.3.0-rc2 and the problem persists:
Repro steps:
Create an html document with this:

    Desktop app deep linking test

Open Mattermost Desktop in a channel other than the Desktop app channel.
Click the link from the HTML file. While Mattermost desktop opens, you are not being redirected to the Desktop app channel.
cc @svelle

I'm not able to reproduce bug where desktop app does open but does not push state to update channel.
Seeing success when running repro steps as recommended above:

Think about reasons why others see it fail though...
The handleDeepLink method can only be called if it's associated webview ref exists in MainPage and I have seen instances where the opposite is the case.
In that case the following, which handles pushing the state to expected channel, would not be called:
    webview.executeJavaScript(
      'history.pushState(null, null, "' + relativeUrl + '");'
    );
    webview.executeJavaScript(
      'dispatchEvent(new PopStateEvent("popstate", null));'
    );

Unfortunately the app is only setup to register the scheme when the package is installed and not when running electron via CLI, so debugging will be a bit tedious.
I'm either going  to spend some time setting up app to register mattermost:// protocol when running electron through CLI, which is possible, or I'm going to target suspected culprit with well-placed logs.
For example,
line 298 MainPage.jsx
self.refs[`mattermostView${parsedDeeplink.teamIndex}`].handleDeepLink(parsedDeeplink.path);`

assumed that self.refs[mattermostView${parsedDeeplink.teamIndex}]. is true, but I've observed it to not always be the case.

Checked with the latest 4.3.0-rc2 and the problem persists:

@wget So I can try to repro more closely, are you saying that you pulled the repo at tag 4.3.0-rc2 and pulled in dunkla's commits?

@deanwhillier @Willyfrog what are the next steps on this PR?

@esethna I added a review from @devinbinnie as he has worked on deep linking, once we have all the changes needed to include it, we should decide on next steps, as I'm not sure the original author has seen the activity on this PR.

I'm closing this PR as most of the code base has changed, but I'll gladly reopen if it can be reused somehow


Support test data in YAML files

Currently ddt supports feeding test data from a file, but that file has to be in JSON format.
Support for YAML seems like a good addition.

Since YAML is not supported in Python's standard library, this would have to be an optional extension, since I do not want to force a YAML parser as a dependency for ddt. ddt should stay very lightweight and be very fast to install.

BUMP!
I agree with not making it a "required" dependency and an optional one.
The JSON part can be used as it is currently used.
For YAML, if someone tries to use it, without PyYAML (or whichever library) installed, you can raise an error telling the user to install the same, otherwise it should just work as expected.

Closed by #42


Need README.md for account template

https://github.com/Jumpscale/ays9/tree/master/templates/ovc/account
Explaining how it works, also clarify in which cases admin group membership is required

Docs are already there. Only users have group memberships although we can add users documentation for relevant templates.


Where to report feature requests or bugs that affect both CovPass and CWA or that are conceptual issues

Your Question
Where should the community report feature requests, bugs or conceptual issues that affect both CovPass & CWA?
I'm thinking about issues like https://github.com/corona-warn-app/cwa-wishlist/issues/573 & https://github.com/corona-warn-app/cwa-wishlist/issues/574

I guess in the dcc overview

Best practice would probably be to create issues in both organization's repositories (i.e. CWA and https://github.com/Digitaler-Impfnachweis).

@heinezen
Okay. Would you like to add this to the wishlist issues I linked to above?

I consider this as solved but would like to ask the community managers to clearly communicate this in the respective issues that are opened here, in the https://github.com/coronawarnapp organization.
Closing as answered.


fix: add patch roles to all updates roles

to avoid things breaking if the code does a patch instead of update

/approve


Magic 8-Ball Simulator

Ask a question and get a whimsical response.
The digital Magic 8-Ball provides random answers to user questions, like "Yes," "No," or "Ask again later." It could even include fun animations to replicate the real 8-Ball experience.

Assign the issue to me please @sumanth-0 . Thank you.


Return the result of this.dispatchEvent in fire

This PR changes the fire method so that it returns the value of executing this.dispatchEvent(), which allows users to know if at least one of the event handlers which handled the dispatched event called `Event.preventDefault().
It also adds tests for both when preventDefault() is used and isn't used within the callback.

Thanks!


TypeError

I am getting the following error on my Node-Red Flow... I was sending a flow.start using an inject node.
TypeError: Cannot create property 'low' on string '100'
Here is my upsc output...
homeassistant:~/.node-red# upsc<EMAIL_ADDRESS>Init SSL without certificate database
battery.charge: 100
battery.charge.low: 10
battery.charge.warning: 20
battery.mfr.date: CPS
battery.runtime: 1680
battery.runtime.low: 300
battery.type: PbAcid
battery.voltage: 14.1
battery.voltage.nominal: 12
device.mfr: CPS
device.model: UPS OR700
device.type: ups
driver.name: usbhid-ups
driver.parameter.pollfreq: 30
driver.parameter.pollinterval: 2
driver.parameter.port: auto
driver.parameter.synchronous: no
driver.version: 2.7.4
driver.version.data: CyberPower HID 0.4
driver.version.internal: 0.41
input.transfer.high: 140
input.transfer.low: 90
input.voltage: 123.0
input.voltage.nominal: 120
output.voltage: 123.0
ups.beeper.status: enabled
ups.delay.shutdown: 20
ups.delay.start: 30
ups.load: 29
ups.mfr: CPS
ups.model: UPS OR700
ups.productid: 0601
ups.realpower.nominal: 400
ups.status: OL
ups.test.result: Done and passed
ups.timer.shutdown: -60
ups.timer.start: -60
ups.vendorid: 0764

If you look at the source code, the string low is not there… So that's probably not coming from this node, it's from something further down your pipeline that processes it.

nut-ups.js.txt
just replace nut-ups.js with this
an you will get
21 Nov 20:06:53 - [info] [debug:4f4a9c73.7bc654]
{ battery_charge: '95',
battery_charge_low: '10',
battery_charge_warning: '30',
battery_date: '2009/12/23',
battery_runtime: '24320',
battery_type: 'PbAc',
device_mfr: 'POWERCOM Co.,LTD',
device_model: 'HID UPS Battery',
device_serial: '004-0D9F-000',
device_type: 'ups',
driver_name: 'usbhid-ups',
driver_parameter_pollfreq: '30',
driver_parameter_pollinterval: '2',
driver_parameter_port: 'auto',
driver_parameter_productid: '0004',
driver_parameter_synchronous: 'no',
driver_parameter_vendorid: '0d9f',
driver_version: '2.7.4',
driver_version_data: 'PowerCOM HID 0.5',
driver_version_internal: '0.41',
input_frequency: '50.0',
input_voltage: '230.0',
input_voltage_nominal: '220',
output_frequency: '50.0',
output_voltage: '230.0',
output_voltage_nominal: '220',
ups_beeper_status: 'enabled',
ups_date: '2009/12/23',
ups_delay_shutdown: '20',
ups_delay_start: '60',
ups_load: '5',
ups_mfr: 'POWERCOM Co.,LTD',
ups_model: 'HID UPS Battery',
ups_productid: '0004',
ups_serial: '004-0D9F-000',
ups_status: 'OL CHRG',
ups_test_result: 'Done and passed',
ups_timer_shutdown: '0',
ups_timer_start: '0',
ups_vendorid: '0d9f' }
the problem was in read_only object properties
https://stackoverflow.com/questions/27519836/uncaught-typeerror-cannot-assign-to-read-only-property

This was fixed in #2. Released v1.0.2.

not fixed.
fresh install of contrib
21 Nov 19:48:00 - [red] Uncaught Exception:
21 Nov 19:48:00 - TypeError: Cannot assign to read only property 'low' of 95
at /home/pi/.node-red/node_modules/node-red-contrib-nut-ups/nut-ups/nut-ups.js:26:38
at Array.forEach (native)

Make sure you actually install 1.0.2. "Fresh install" might install from cache.


running environment

Hi,could you please show me the ways to how to  run the great project on our own environment?
Thanks a lot!

I'm trying to show the way to how to run this project in README.md.
Could you let me know what kind of troubles you met when you ran scripts described in README.md?
Without this information, it's difficult for me to help you.

oh yes,I want to run it on my local environment with my computer,but i do not know what the environment should I build and how to start and run it?thanks a lot!

Sorry for my late response. If you want to run it on your computer, you need to install required python libraries by yourself. Unfortunately, I've already removed my python virtual environment and I'm not sure an exact list of libraries you need. I'd recommend you to start from anaconda's python 3.6 and  arrow-cpp by conda install arrow-cpp=0.9.* -c conda-forge and run my script with a down sampled dataset such as first 1000 rows of the original train and test datasets. You'll be able to see what are libraries you should install. After that, you just need to run scripts that are not related to Docker (e.g., scripts/convert_csv_to_feather.py, scripts/click_id_mapper.py, and run.py).


[docs] Add side nav, dabble in mobile responsiveness

Add side nav (fake routes for now, links 404)
Start on mobile responsiveness (no hamburger menu yet, not dialled in spacing yet)
Tweak text-sm line height
Refine quick nav a11y

https://deploy-preview-857--base-ui.netlify.app/new/components/dialog

@vladmoroz Sidebar scroll doesn't bubble up to the window when you reach the top/bottom of the scroll area?

@vladmoroz Sidebar scroll doesn't bubble up to the window when you reach the top/bottom of the scroll area?

Do you want it to? I have a slight preference to disable that, but could go either way. Restored scroll chaining in the recent commit, lmk if that feels better.

@vladmoroz Feels much better imo. I don't see how it would work the other way, just felt like I couldn't find my way around the website and the website was fighting me.


Support for ending tags

Hi,
twig-cs-fixer reports error:
 KO ./resources/views/pages/account/account.twig
 ------- --------------------------------------------------------------------------------------------------------------------
  FATAL   9    | <div class="container">
          10   |     {% livewire.component 'account-panel' %}
          >>   | File is invalid: Unexpected "livewire" tag (expecting closing tag for the "block" tag defined near line 9).
          11   | </div>

It is possible to support ending tags like {% endlivewire.component %} or functions {{ livewire.component('account-panel') }}?

Hi there! We currently don't have plans to add support for ending tags. Tags like {% include %} in Twig for instance can also exist without an ending tag. Perhaps it's something that can be set to allow in the twig-cs-fixer?


Exception ticking World Component brings game to 1 TPS

Describe the bug
In case we find this again or find a way to reproduce. Im having an error where i get constantly the following exception (reducing TPS to 1/s)
It happened right after i left one base i destroyed via pods. I might test it later because people rarelly leave destroyed bases by pods. I didnt had much surviving muffallos 😢
Loading the game completely fixed the issue
I know this report is almost useless. Puting it here case i find more details
To Reproduce
Steps to reproduce the behavior:
Dunno
Error Log
Exception ticking World Component: RimWorld.Planet.TileTemperaturesCompSystem.Threading.ThreadAbortException
  at (wrapper dynamic-method) RimWorld.Planet.TileTemperaturesComp.RimWorld.Planet.TileTemperaturesComp.WorldComponentTick_Patch0(RimWorld.Planet.TileTemperaturesComp)
  at RimThreaded.RimThreaded.ExecuteTicks () [0x00ca0] in <60167be910e14ffd90c81ad404a744e2>:0
Verse.Log:Error(String, Boolean)
RimThreaded.RimThreaded:ExecuteTicks()
RimThreaded.RimThreaded:ProcessTicks()
RimThreaded.<>c:<CreateWorkerThread>b__128_0()
System.Threading.ThreadHelper:ThreadStart_Context(Object)
System.Threading.ExecutionContext:RunInternal(ExecutionContext, ContextCallback, Object, Boolean)
System.Threading.ExecutionContext:Run(ExecutionContext, ContextCallback, Object, Boolean)
System.Threading.ExecutionContext:Run(ExecutionContext, ContextCallback, Object)
System.Threading.ThreadHelper:ThreadStart()

Mod List

A lot of Vannilla extended content
Alien races

Naga
harpy
androids
dragonian
moyo
astoriel
hisa

zoom plus
dubs mint menu
open the windows
roads to rin
git-up mods
Ticks per second

Screenshots

I ran into this issue as well but with a very different mod list. Here is a link to the issue I submitted with the same error https://github.com/cseelhoff/RimThreaded/issues/279 (idk if I can link these issues any other way)

please test with the newest version of RT and re open the issue if needed


Error working with accounts from default genesis config

Description
Can't use accounts specified in default genesis config.
Steps To Reproduce

Run a node on a network of v0.1.11 full nodes - make sure it is fully synced with a network.
Open a CLI wallet that connects to one of the net nodes, configured w the pub and private keys of the 2 tap accounts via its json data file.
Set tap1 or tap2 account as current.
INFO.

Actual Behavior
$ info
 INFO 00e 14:46:01.637 ▶ request: http://localhost:9090/v1/nonce, body: { "address": "0xc4eabd379d646beb6e065934cf7e9de59f8d43e7"}
 INFO 00f 14:46:01.640 ▶ response body: {"error":"account does not exist","message":"account does not exist","code":2}
 ERRO 010 14:46:01.640 ▶ failed to get account info: /nonce response status code: 500

Local alias:  tap1
Address:  0xc4eabd379d646beb6e065934cf7e9de59f8d43e7
Balance:
Nonce:
Public key: 0xc007a95dc5b50c3ebbb8ff6bc4eabd379d646beb6e065934cf7e9de59f8d43e7

Expected Behavior
API to return correct balance and nonce of the tap account and no error.
Environment
OS X

Looks like this was a tn-c2s16-34 specific issue and not due to changes in genesis accounts code in v0.1.11. We should still try to understand how genesis accounts could not be used on it. @AmitShaul @ilans

I don't think this issue can be caused by the network, it's a local issue. The node runs a genesis setup procedure when it sets up a new mesh, i.e. when the node starts up for the first time.
This procedure should update the tap accounts balance and nonce in the new global state.
I don't know why this didn't work this time, but I think that it would take considerable time to investigate and we're not sure it will ever happen again. Let's put this issue on hold and investigate only if we see this again. Closing for now.


"The evaluation period has expired" for the provided package from the repository

When try to build and run the following Dockerfile, I got the error.
https://github.com/Microsoft/mssql-docker/blob/master/linux/preview/RHEL/Dockerfile#L36
It looks like the repository is an old one. The url should be https://packages.microsoft.com/config/rhel/7/mssql-server-2017.repo.

Yes, you are correct.  Thanks for pointing this out.  I've fixed it.


Not receiving verification email

I'm a new user I've tried 3 or 4 times in the last 24 hours to register. I am not receiving the verification email. I've checked my spam folder. Thanks

Which site?

Mike,
I think it has something to do with my webhost email, at
laughingsquid.net... there must be some filtering that is happenning before
getting forwarded to my gmail? I was trying to register for hub.zilla. So I
switched to my gmail address and then I got the verification.
so I'm all sorted and I doubt it is anything on your end.
Thanks for the quick response!
Tom
On Tue, Apr 10, 2018 at 4:51 PM, Mike Macgirvin<EMAIL_ADDRESS>wrote:

Which site?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/redmatrix/hubzilla/issues/1061#issuecomment-380282853,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABptubMao9Y4rkE3GBoBP9tP9FYRnUZeks5tnUWRgaJpZM4TOd72
.

--
TOM SEPE
SEPE PRODUCTIONS
ART_DESIGN_FABRICATION
+1<PHONE_NUMBER> <+1%20510%20499%200946> mobile
<EMAIL_ADDRESS>http://tomsepe.com

Glad you got through.
Something that happens quite frequently is that an ISP will shut down a spammer, and one of our new sites ends up getting their old IP address.  The fact that spammers are always on the move means there are a large number of these blocked IPs in the pool and the odds of getting one for your new Hubzilla site are unfortunately quite high.


CITATIONS.cff (plural) along with unique-id    field

I am making two issues cards. This is the first one because it is the simpler of the two. However, they are related.
The other issue card is here: https://github.com/citation-file-format/citation-file-format/issues/345
Currently, it is only possible to have a single CFF file called CITATIONS.cff  .  The CFF 'specifications' should include the possibility of having mutiple citations in a file called CITATIONS.cff . These would be separated by the YAML document separator "---".  I am attaching the example that comes out from the code in the CiteSoft issues card (I am appending ".txt" to the filename so that I can attach it to github).
CITATIONS.cff.txt
A single cff file is perfectly fine when somebody is downloading a citation from a repository's webpage (or files). However when somebody is using a software and that software needs to export out multiple references, they should each be unique citation-file-format entries. This is not the same as transitive software references ( https://github.com/citation-file-format/citation-file-format/issues/267  ).  For anyone who wishes to understand why it is different, see the CiteSoft issues card and documents attached there.  https://github.com/citation-file-format/citation-file-format/issues/345
Additionally , there may certainly be other usages where people want to make a 'library' with multiple citation-file-format entries.
Writing and reading such files is not challenging, CiteSoft has such code in it https://github.com/citation-file-format/citation-file-format/issues/345. All that is required is having the citation-file-format specifications allow multiple citations in such a file, along with the related issue of unique-id field, described below.

There is a second change that I think should be made in conjunction with the multiple citations change. The CFF schema should add a field ("attribute") at the top/root level called unique_id.  That way, when somebody is appending to a CITATIONS.cff file, the file can be scanned to make sure that no other entry has the same unique-id. Please note that this means two different software versions can be cited in the same CITATIONS.cff as long as they have different unique-id's. This is intentional. Alternatively, two versions could have the same unique-id with only the newer version number being retained.  Why am I suggesting a unique-id field?   This is related to the identifiers issue: https://github.com/citation-file-format/citation-file-format/issues/102   in most cases, a developer will want to use a URI/URN that they are sure nobody else will use (it can be a URL or a DOI, for example).  However, we should not constrain people to be required to have that. Requiring that is an unecessary barrier to software citations. To better explain what I mean, let me give several examples of example unique-id below:
unique-id: 'AdityaSavaraLikesFrogs'
unique-id: 'https://github.com/AdityaSavara/CheKiPEUQ'
unique-id: 'https://doi2cff.readthedocs.io/en/latest/readme.html'
unique-id: 'https://doi.org/10.5281/zenodo.1206048'
unique-id: '10.5281/zenodo.1206048'
unique-id: 'doi:10.5281/zenodo.1206048'
Note that the first one seems absurd -- but that we can be confident nobody else will use it.
Note that the last three are basically the same. Note that these are strings with no 'rules' (except no disallowed character strings). Within the unique-id, one can define the URI type or not.
Why do I suggest this system, and why am I putting this in the same issues card as CITATIONS.cff?
Because (a) we need a way to ensure no duplicates when making a CITATIONS.cff , and we should not have to parse the entire entry to understand if there is a duplicate. This philosophy is important for CiteSoft and adoption in languages beyond python.  (b) we do not want to make our own URI system. We also do not want to catalog nor to police when someone is "allowed" to create a URN (like who gets to make a doi).
The solution (as far as I can see it) is to have a unique-id field where people are instructed to put in anything that they are sure nobody else will use, but that we strongly suggest they use a doi or a URL that is uniquely associated with the topic of that citation entry. We should even suggest but not require they follow the same format as urns ("unique-id:doi:10.5281/zenodo.3539765"). However, by having a single unique-id field, which is not required to be a URN, when a CITATIONS.cff file is compiled by CiteSoft or by bibliographic softwares, duplicates can be removed.

Hi Aditya, thanks for making these issues. I still have to disgest some of it, but regarding unique-id: do I understand correctly that there is no other purpose to it than to avoid duplications in an array of citable items? If so, I think there is no need to introduce an additional key for it, and instead tooling can take care of making sure there are no duplications. For example, when a tool (like citeme, CiteSoft) collects data on what subset of all references are relevant for a given purpose, they can just look at the array of dictionaries that are already in the array of citable items. Something like:
>>> items = list()
>>> item = {'a': 4}
>>> item in items
False
>>> items.append(item)
>>> items
[{'a': 4}]
>>> item in items
True
>>>

As long as you have, say, less than a few hundred references, I don't think it's worth the trouble to add extra functionality to it, but I guess that's up to the tool developers. If I had to implement something like this, I would simply use hashing based on the actual values of the dict. That is equally performant to asking users to supply a value while avoiding any ambiguity.
See e.g. https://computersciencewiki.org/index.php/Hashing for more on hashing.

I am grateful that you've already taken time to start considering this.
Yes, you understand correctly that the purpose of the unique-id is primarily to avoid duplications in an array of citable items. I'll explain the couple of considerations that I came across in making CiteSoft that made me feel a uniqueID field was needed.
(a) Somebody might use different versions of the same software, but they might have the same "citation".  In that case, the dictionaries would not match even though the software's author only wants to have their paper [or whatever] cited once.
(b) Two people might independently create CFF files for the same reference, and then the dictionaries would be different by the unique-id would not be different. I did think about just making the unique-id a custom identifier [and I did mention this possibility in my documents], however see next point.
(c) The idea that parsing these files is easy is a very python centric view. Python is unusually good at handling strings. With other languages it's more of a pain, so it's easier to just read the lines looking for one that starts with unique-id and seeing if the files  match.
Because of how CiteSoft in particular works, I don't think the unique-id field is critical for CiteSoft, but it may be useful for other tools.
Even if unique-id is not not embraced, I do think that, including files which multiple citations in it ( CITATIONS.cff ) as an accepted part of the standard ( https://github.com/citation-file-format/citation-file-format/files/7180164/CITATIONS.cff.txt ) would be useful.


[feature request] handling secret

I think the webserver secret should have a more secure way to handle the secret variable. Examples like the ones defined on the website suffer from the issue described here.
I can think in two ways of doing It:
First: change the examples to generate a random number every time the app starts (easy to put on examples. If the comments agree I can do this easily).
Second: a more complicated approach would be including It on the API, creating a table on the database that stores a strong encrypted key that needs to be decrypted somehow when running the webserver (one way would be using a db_secret='SECRET' to decrypt the key from db. This would be secure even if no one changes the password because the password stored is generated randomly and is secure. But of course, if someone has access to the table, could retrieve the key, but the surface area is much smaller already).

Thank you for raising this point @nullhack. I get the point of the article you mention but the way I see it, developers should be responsible of choosing a strong secret and keep it safe. This approach is widely used in big and battle-tested frameworks like Flask, Django, Ruby on Rails...
However, I agree this should be mentioned in a warning/danger notice in the documentation to raise developers attention that they should take extra-care about this.


Problem Number | Title | Language

Title - SOLVING MAXIMUM-AREA-HISTOGRAM PROBLEM IN PYTHON
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] Python

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

May I work on this issue?  Could you please assign it to me?

May I work on this issue? Could you please assign it to me?

Yeah sure!!!

@kaushalendra-pandey Please go through the contribution guidelines. And edit the title accordingly.

[x] Assigning to @RaymondWaterlooLi

[x]  Assigning to @RaymondWaterlooLi

Thank you.


Add powerai-based multiarch Dockerfile

This Dockerfile will create an empty (i.e., no deep learning frameworks) IBM PowerAI-based container with the MAX framework software installed.
Note that currently powerai 1.6.0 is gpu-based only, meaning any subsequent images based FROM this one will need access to GPU hardware (usually via nvidia-docker).

This looks good to me. What do you think, @bdwyer2 ?

Looks fine to me.
@ajbozarth are we able to have DockerHub build an image tag off this Dockerfile in our auto build? e.g. max-base-powerai ?

Creating a max-base-powerai build would be great. Ideally, I'd like to make the build multiarch (x86+ppc64le). I can add the logic in your travisfile to do that, but do you know if you have ppc64le builders available?

yeah we can set up a second auto build on a second docker file. Once this is merged I'll create a auto build for a tag called max-base-powerai built on the new dockerfile.

I've added a .travisfile to this PR after discussion with @ajbozarth . There's not a great way to get Docker Hub's automated build system to build multiarch images (or even just power images), so I've instead modified the travis builder to build those images.
You'll need to modify your travis builder with the following variables:
REPO: Whatever docker registry/repo we're pushing to (I'm assuming 'codait')
REPO_USER: The dockerhub user for authorizing with the above repo
REPO_PASS: And the dockerhub password
Let me know if there are any other changes here you'd prefer.

Just on an initial glance it seems that this would push a new image to docker hub even on PR travis builds, but overwriting the master build. Am I misreading the travis file?

Good call @ajbozarth. I've added a conditional so that it doesn't push on pull requests.

sorry I haven't had enough time to properly review this yet, I've requested other members of the MAX team to also take a look so this doesn't get blocked on me. I just want to make sure we don't let a minor edge case in the travis build mess up anything downstream since this is a large update to the yaml file.

Understandable! Let me know if I can answer any concerns. Just to be clear, this does the standard build/test for the vanilla max-base image that the previous travisfile performed, but while it pushes the new powerai based images, it does not push the vanilla max-base image it builds. That would still (presumably) be built by your DockerHub builder.
If y'all want it pushed from Travis instead, that could of course be done.

We'll discuss today internally

So based on our slack convo I made some updates/suggested edits to your travis.yml and am copying them into this comment below:
language: python

python:
  - 3.6

os:
  - linux
  - linux-ppc64le

env:
  - DOCKERFILE=Dockerfile IMAGE=test
  - DOCKERFILE=Dockerfile.pai IMAGE=powerai

matrix:
  exclude:
  - env: DOCKERFILE=Dockerfile IMAGE=test
    os: linux-ppc64le

services:
  - docker

before_install:
  - if [[ "$TRAVIS_OS_NAME" == "linux-ppc64le" ]]; then
       ARCH=ppc64le;
    fi
  - if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
       ARCH=x86_64;
    fi
  - mkdir -p ~/.docker
  - echo '{"experimental":"enabled"}' | tee ~/.docker/config.json
  - sudo service docker restart

install:
  - docker build -f "$DOCKERFILE" -t codait/max-base:"$IMAGE"-"$ARCH"-latest .

before_script:
  - docker run -it --rm -d -p 5000:5000 codait/max-base:"$IMAGE"-"$ARCH"-latest
  - sleep 10

script:
  - docker ps | grep codait/max-base:"$IMAGE"-"$ARCH"-latest

after_success:
  - if [[ "$IMAGE" != "test" && "$TRAVIS_PULL_REQUEST" == "false" ]]; then
      echo "$DOCKER_PASS" | docker login -u "$DOCKER_USER" --password-stdin;
      docker push codait/max-base:"$IMAGE"-"$ARCH"-latest;
    fi

jobs:
  include:
    - stage: "Docker Manifest"
      script:
      - echo "$DOCKER_PASS" | docker login -u "$DOCKER_USER" --password-stdin;
        docker pull codait/max-base:powerai-ppc64le-latest;
        docker pull codait/max-base:powerai-x86_64-latest;
        docker manifest create codait/max-base:powerai-latest codait/max-base:powerai-ppc64le-latest codait/max-base:powerai-x86_64-latest;
        docker manifest push codait/max-base:powerai-latest;

Notes based on the above code:

I changed how IMAGE is used and named since we want to have tags in the main max-base repo rather than a separate repo.
The build matrix with an excludes remains since it is seemingly the best way to do the 3 builds: original test build, and the powerai build on both x86 and ppc64le
I updated the docker login to the secure format
I moved the manifest chunk into a new stage called "Docker Manifest" that will run after the previous 3 builds finish. Note that the powerai builds push their image in the first stage then pull both in the manifest stage

Is what I wrote look ok to you? if so feel free to update your code with it. It (and your current code) have one large flaw remaining afaik. It wont push to docker hub on PR builds or the current test build, but travis runs on and push to any branch or tag, so we need to further restrict the docker push to just master and tag releases and make the docker tag name match appropriately (ie powerai-x86_64-latest and powerai-latest vs powerai-x86_64-v1.1.2 and powerai-v1.1.2). This could most likely be done with a few if statements and setting another var in the before_install step.

Ok so as a followup on my last comment I'v tried writing up a version with the checks for master/tag. I plan to create that fork now and test it, but here's the travis.yml I'm testing:
language: python

python:
  - 3.6

os:
  - linux
  - linux-ppc64le

env:
  - DOCKERFILE=Dockerfile IMAGE=test
  - DOCKERFILE=Dockerfile.pai IMAGE=powerai

matrix:
  exclude:
  - env: DOCKERFILE=Dockerfile IMAGE=test
    os: linux-ppc64le

services:
  - docker

before_install:
  - if [[ "$TRAVIS_OS_NAME" == "linux-ppc64le" ]]; then
      ARCH=ppc64le;
    fi
  - if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
      ARCH=x86_64;
    fi
  - if [[ "$TRAVIS_BRANCH" == "master" ]]; then
      VERSION=latest
    else
      VERSION=$TRAVIS_BRANCH
    fi
  - mkdir -p ~/.docker
  - echo '{"experimental":"enabled"}' | tee ~/.docker/config.json
  - sudo service docker restart

install:
  - docker build -f "$DOCKERFILE" -t codait/max-base:"$IMAGE"-"$ARCH"-"$VERSION" .

before_script:
  - docker run -it --rm -d -p 5000:5000 codait/max-base:"$IMAGE"-"$ARCH"-"$VERSION"
  - sleep 10

script:
  - docker ps | grep codait/max-base:"$IMAGE"-"$ARCH"-"$VERSION"

after_success:
  - if [[ "$IMAGE" != "test" && "$TRAVIS_PULL_REQUEST" == "false" ]] && [[ "$TRAVIS_BRANCH" == "master" || "$TRAVIS_BRANCH" == "$TRAVIS_TAG" ]]; then
      echo "$DOCKER_PASS" | docker login -u "$DOCKER_USER" --password-stdin;
      docker push codait/max-base:"$IMAGE"-"$ARCH"-"$VERSION";
    fi

jobs:
  include:
    - stage: "Docker Manifest"
      if: branch = master OR tag IS present
      script:
      - if [[ "$TRAVIS_BRANCH" == "master" ]]; then
          VERSION=latest
        else
          VERSION=$TRAVIS_BRANCH
        fi
        echo "$DOCKER_PASS" | docker login -u "$DOCKER_USER" --password-stdin;
        docker pull codait/max-base:powerai-ppc64le-"$VERSION";
        docker pull codait/max-base:powerai-x86_64-"$VERSION";
        docker manifest create codait/max-base:powerai-"$VERSION" codait/max-base:powerai-ppc64le-"$VERSION" codait/max-base:powerai-x86_64-"$VERSION";
        docker manifest push codait/max-base:powerai-"$VERSION";

So I forked the repo and have done some through testing of the travis/docker side of this (feel free to check out my docker images and make sure they work from the user side of things). My updated yml file can be found here: https://github.com/ajbozarth/MAX-Base/blob/master/.travis.yml
While testing I made quite a few edits to the file to get it working (from the code pasted in my above comment). I'll highlight some of them here:

moved chunk to enable experimental mode to the stage that uses it.
made various edits to the manifest stage such as fixing the conditional, separating out script lines for easier debugging, and making sure the stage doesn't do all the builds.

The final thing I'm still working on is that I've discovered that TRAVIS_OS_NAME=linux for both x86 and ppc64le so the current if block setting ARCH doesn't work. Based on preliminary googling we may have to completely restructure how the build matrix is written so we can differentiate between the two in the build steps

So I decided to keep at it and I solved the os issue by using matrix includes and the docker images it built can be found here: https://cloud.docker.com/repository/docker/ajbozarth/max-base/tags
Also I found that how we were enabling manifest didn't work, but discovered and switched to a different method: force update of docker then enable via env var.
As of now the .travis.yml in my repo work as intended for this PR. If you can test that the images in my docker hub repo work as you intend them to then we can move forward with my updates.

This looks great, and works in my repo. You can check the images at mborgard/max-base:powerai-latest, and my basic function testing confirmed they work as expected.
As we discussed, I had to add dist: xenial to get it to build, but that seems to be an IBM Travis issue. So if you're building this elsewhere, the .travisfile currently in this PR should work.

I created the docker user codaitbot and filled in it's credentials in DOCKER_USER and DOCKER_PASS for both travis-ci.org and travis-ci.com, please make sure to note in each follow up PR for the models that I will need to make that change.

No that this is merged (and since there's no open work on the repo) I will be cutting a new minor release. Any references in the models should point to the new release rather than latest.
I am also watching to see if having both .org and .com running will break this docker portion before cutting said release.


Support mysql connection and schema creation.

issue moved to jira https://aerobase.atlassian.net/browse/ARB-8


Bugfix in LRUCacheTest.java

The list is 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 and list.get(3) will return 4 which will not be equal to cache.lookup(3) having an expected value of 3. The correct check I assume should be assertEquals(list.get(2), cache.lookup(3)); which pushed key 3 on top of recently accessed and cache.insert(1, 1); will pop of key 4 due to capacity exceeded. We assert for a null of key 4 in next step of test case.

@gardncl Can you please review this change. Thanks


Updated to DND 4.1.1 and it is not working

I fully understand that this module is not being maintained but it is a KICK ASS module and the very slight off chance this can be fixed I would be so mega grateful.
Here is the error from the players point of view.

They are still getting the help card in chat if they want it.

Appreciate the compliment; can't say no to a request like that, lol. I'll take a look this weekend and see what I can do for ya!

Thank you so much, really appreciate it.

Just an update - dnd5e v4.1 changed the whole roll pipeline around, so the module dev community is still collectively wrapping their heads around how it all works now as we play catch up. I've been looking at this over the weekend, and it seems like there's a couple snags in the system preventing the module from actually making the roll blind. But as soon as those get resolved (or I can find a workaround) I'll try to get an update out for you!

Definitely feel free to share your workarounds and anything you were able to dig up! Worst case it's something I've already tapped as an issue, but it may help anyone else who stumbles onto this ticket in the near future find a temporary solution.

No worries, I was wrong with the file, it is in dnd5e.mjs and I have provided a screenshot of the rectification. I think there is an issue in for it currently.
The word 'message' is missing from line 31572.
In the screenshot I have added it.

Okay, cool. Yeah that's the same thing we were able to track down yesterday. I did find the issue on the main system repo, looks like that fix is slated for 4.1.2, so that shouldn't be too long a wait.

Yeah, so I made some macros for the players and they are so used to clicking skill checks from TAH or character sheet it was a bit of fun getting them to use macros. More of a reason why your module is great. lol
Thank you for the update.

Just checking in again since dnd5e 4.2.1 dropped a few hours ago; I have a mostly working version of the module right now, but there's a few wrinkles I need to iron out still (like macros using the old roll syntax not working as intended). I'll try to get those things fixed up this week, and have an update out for you this weekend.

Version 0,9.3 is now available and should be working fine with dnd5e v4.1.2 (and only v4.1.2).
I'll leave this ticket open until I have confirmation that everything looks good for ya, so lemme know once you've tested it.

Tested and works perfectly, thank you very much.

Alright awesome. Thanks for your patience! Happy gaming!


[1.34.0] Command-F to search items in responses no longer works

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
When performing Command-F in a JSON result, the search dialog no longer appears.
.bru file to reproduce the bug
No response
Screenshots/Live demo link
1.33.1

1.34.0

Hi @fantpmas - thanks for raising this. We'll have it sorted in the next release (likely next week).

Hi @fantpmas - thanks for raising this. We'll have it sorted in the next release (likely next week).

Please fix this asap.

This issue has been resolved in v1.34.1, so I'm closing this ticket.


C:\Users\Komal Sharma\OneDrive\Desktop\Free\phive/vendor/autoload.php

not found

Copy from AI response:
The "Laravel vendor/autoload.php missing" error is a common issue indicating problems with your Laravel project's dependencies.  Here's a breakdown of why this occurs and how to fix it:
Why the Error Happens:

Incomplete Installation: The vendor directory, containing all project dependencies (including autoload.php) might be missing if you didn't fully install Laravel or you interrupted the installation process.
Git or Other Version Control: If you cloned a Laravel project from a repository like Git, the vendor directory is often excluded because it's large and can be regenerated.
Accidental Deletion: You may have inadvertently deleted the vendor directory or the autoload.php file.

Troubleshooting and Solutions

Install Dependencies Using Composer:

Open your terminal (or command prompt) and navigate to your Laravel project's root directory.

Execute the following command:
composer install

This will automatically download and install all the necessary dependencies listed in your composer.json file into the vendor directory.

Regenerate Autoload Files:

If the vendor directory exists, but autoload.php is still missing, run:
composer dump-autoload

This command regenerates the class autoloading files needed for your project to function.

Check Permissions:

Ensure your web server and the user running the project have the necessary permissions to write to the project directory. You may need to use chmod or chown to adjust permissions if needed.

Clear Configuration Cache (if applicable):

Sometimes cached configuration can cause issues. Run:
php artisan config:clear

Additional Tips:

Always use Composer to manage your Laravel project's dependencies. Don't manually copy or move the vendor directory.
Consider a fresh Laravel installation if other solutions fail, then copy your application-specific code into the new project.

If the problem persists, provide the following information for further assistance:

The exact error message you're seeing.
Your Laravel version.
Your operating system.
The steps you've already taken to try and fix the problem.