Warnings of type "Files with such naming scheme are not part of BIDS specification." do not list full path

It's impossible to find the file in question

Hi @chrisfilo , can you elaborate on this?

location should be listed for those issues, like here

So, for every method from validator/bids.js that is returning an issue to the cli.js should we just carefully detect which method to return the file-path and debug this issue.
Is my understanding right?

No - this seems to be only specific to issue class NOT_INCLUDED. It's also
been observed in the web version of the validator so cli.js might have
little to do with it.
Best,
Chris
On Thu, Feb 22, 2018 at 5:57 PM, Rohan Goyal<EMAIL_ADDRESS>wrote:

So, for every method from validator/bids.js that is returning an issue to
the cli.js should we just carefully detect which method to return the
file-path and debug this issue.
Is my understanding right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/INCF/bids-validator/issues/357#issuecomment-367885668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkpx_hdrJ8XlVvT9LBt-1COOreKIg_ks5tXhsggaJpZM4QOxS4
.

Okay. 😄 Thanks

Hi @chrisfilo, .DS_Store file types are hidden files in macOS containing the folder's meta data. You can make the hidden files visible in macOS by running following command in terminal:
defaults write com.apple.finder AppleShowAllFiles -bool true
killall Finder

Then open your data folder in Mac's Finder app and you will see all hidden files including .DS_Store in it.
To hide them back, run:
defaults write com.apple.finder AppleShowAllFiles -bool false
killall Finder

Sorry for misinterpreting the issue, I just saw that https://github.com/INCF/bids-validator/pull/368 already resolves this and the actual issue is being addressed in https://github.com/INCF/bids-validator/pull/416


📝 Highlight correct line in tutorial docs/en/docs/tutorial/body-multiple-params.md

The tutorial section https://fastapi.tiangolo.com/tutorial/body-multiple-params/#multiple-body-params-and-query is about adding query parameters on top of (several) body parameters. In the sections before it, the tutorial had introduced 3 body parameters: item, user and importance. This section then focuses on adding the query parameter q, so it seems to me like the next line should be highlighted in the code snippets - the line defining q instead of the one defining importance.
Or, alternatively, we could highlight all 4 lines (the 3 body parameters + the additional query parameter)

Great, thank you! :bow:


QUESTION: can solve_least_squares be used to do least squares polynomial fitting?

Can solve_least_squares be used to do polynomial fitting like :
https://numpy.org/doc/stable/reference/generated/numpy.polyfit.html
https://numpy.org/doc/stable/reference/generated/numpy.polyval.html#numpy.polyval
?

There is an analytical solution to polynomial fitting, so the concept of having to provide a stop_strategy_type object doesn't seem right.

Yeah you should just use a linear solver if you want to do ols fits if polynomials.

This works fine:
dlib::matrix<float> dlib_polyfit(const vector<float>& x, const vector<float>& y, int order)
{
    DLIB_ASSERT(x.size() == y.size(), "bad input sizes. x and y must have matching sizes.");

    /*setup state*/
    dlib::matrix<float> Y = dlib::mat(y);
    dlib::matrix<float> X(x.size(),order+1);
    dlib::matrix<float> x_mat = dlib::mat(x);
    dlib::set_colm(X,0) = 1;
    for (int i = 1 ; i <= order ; i++)
        dlib::set_colm(X,i) = dlib::pointwise_multiply(dlib::colm(X,i-1),x_mat);

    dlib::matrix<float> X_t = dlib::trans(X);
    dlib::matrix<float> B = dlib::inv(X_t*X)*X_t*Y;

    DLIB_ASSERT(B.nr() == (order+1) && B.nc() == 1);
    return B;
}

float dlib_polyval(const dlib::matrix<float>& B, float x_hat)
{
    dlib::matrix<float,1,0> X(B.nr());
    X(0) = 1;
    for (long i = 1 ; i < B.nr() ; i++)
        X(i) = X(i-1)*x_hat;
    dlib::matrix<float> Y = X*B;
    DLIB_ASSERT(Y.size() == 1);
    return Y(0);
}

These functions, give or take some refactorisation, might be useful additions to dlib. @davisking, what do you think?

  It's not clear to me that anyone would use this if it was in dlib when there are the other options.  And like, if I was going to add something specifically for regressing 1D functions to scalars I would add something that fits a piecewise linear or some kind of spline.  That would be cool.  But polynomial regression isn't really very general and often gives bad results.  That's not to say I haven't used polynomial regression in real applications, but it was always in a specialized context.
As an aside, you don't and generally shouldn't make named temporaries with dlib::matrix.  It's unneeded and might make you lose out on optimization opportunities.  Like just do inv(trans(X)*X)*trans(X)*Y. It's definitely going to run faster than what you put there.  Dlib's matrix is also not like Eigen. I realize that in Eigen you have to make lots of temporaries because Eigen fails to handle aliasing and will give incorrect results.  However, dlib::matrix will always do the right thing.

Thanks for the comments. Would you say using rr_trainer is useful if you only have 10 samples ? It looks like rr_trainer adds a regularisation term.

So here is some context for what I'm doing. I'm currently doing some object tracking-by-detection. I have an object detector that spits out fresh, untracked, detections on each frame. I have an object tracker that uses max_cost_assignment with a CIOU cost function that matches tracklet predictions with fresh detections. For each matched tracklet/detection pair, the detection is assigned the tracklet's tracking number and the tracklet is updated with the detection. For unmatched detections, new tracklets are created. When a tracklet reaches a certain number of iterations without having been updated with a detection, it is removed from the buffer of tracklets. This is roughly how SORT and deepSORT work, and how i imagine other trackers work too. I've been experimenting with different types of tracklets. I have tried correlation_tracker from dlib. I have tried a kalman filter based tracklet using both opencv and dlib (they give roughly the same results). I have tried all the trackers from opencv (MIL, KCF, BOOSTING, etc). They all work fine to be honest. But i want to try other things too. Now i'm trying some polynomial regression. So if each tracklet cyclically buffers the last 10 detections say, i can use quadratic regression to predict the next detection. I'm tracking still objects, captured from a moving camera. So still objects move quadratically (roughly) within the camera frame, hence why polynomial regression seemed like a good idea to try. And indeed it all works. BUT, i was using liquid-dsp, a fantastic DSP library, which has polyfit and polyval functions. Since i'm using dlib anyway, i thought i could have 1 less dependency by defining polyfit and polyval in dlib.
Now that you mention rr_trainer, there are many other algorithms in dlib i could try. krls comes to mind to.

I also saw that you have a "learning to track" tutorial. But that requires training data, which I do not have. Otherwise that would have been oven baked ready to use.

Thanks for the comments. Would you say using rr_trainer is useful if you only have 10 samples ? It looks like rr_trainer adds a regularisation term.

Yes, it's useful when you have less data.  The regularization goes a long way.  It's also useful even when you have a lot of data.

To predict the next state of a track I would honestly use a Kalman filter (or an extended Kalman filter if you believe your motion dynamics require it).  A Kalman filter is an unassailable formalism in a lot of cases.

Just trying rr_trainer. How do you get a decision function out of a rr_trainer instance? I must have missed something in the docs.

OK using rr_trainer works. It's not obviously better than traditional polynomial fitting in my tests.

krls also works. Not necessarily better than rr_trainer, polyfit or kalman.

The best so far is correlation_tracker, but much more computationally expensive. Anyway, this is a tangent.


docs: fix broken link

Fixes broken link on the page "Writing Well-Structured Tests with Tap" that leads to "Coverage Maps".

Stumbled upon this as well!


Remove vim config

Vim plugin config is horribly out of date, unsupported, and broken.
Addresses #304

🍻


Add diabetes education to the list of note types (BeReferralPrescription)

@CyprienSmals
Salut Bart, aurais-tu le temps de regarder à ce code (temporaire) manquant ?
Il devrait être utilisé pour les deux templates :
Diabeteseducatie via conventiecentrum
Educatie en zelfzorg bij diabetespatiënten ‘zonder traject’
Comme spécifié dans le document de l’INAMI “Verwijsvoorschrift Business Rules_Nursing_V2.10.docx”, cela serait utilisé si “Alleen beschikbaar indien “Type educatie” = “Andere””.
Voilà la partie en question dans le document:
Merci beaucoup,
Cyprien

Available on the issue-265 branch, waiting for WG approval.

@CyprienSmals
what do you mean by "last"? The master branch? the earlyadopters branch? the issue-265 branch which contain the elements we have been talking about, but are not yet approved by the HL7 workgroup?

@bdc-ehealth
Yep sorry I meant the master branch, but it would be helpful to add it also to the earlyadopter branch; to allow integrators to use this note as part of the two templates for the MVP. I thought it has been approved by the HL7 workgroup since this issue has been closed and the https://github.com/hl7-be/referral/pull/266 has been merged. But maybe I am too soon

@CyprienSmals ,
can I put everything from the issue-265 branch to the earlyadopter branch, or do I have to cherrypick the commit with the new code?

For me you can put everything from issue-265 into earlyadopter branch, I do not see any breaking changes in it. Thank you @bdc-ehealth .

everything from the issue-265 branch is now in the earlyadopter branch.


ERROR: Unable to extract uploader id

Requirement already satisfied: youtube-dl in c:\users\hachi.ttl.ttd\appdata\local\programs\python\python311\lib\site-packages (2021.12.17)
[youtube] BaW_jenozKc: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I get the same...
youtube-dl --verbose https://www.youtube.com/watch?v=JuOWoGpqg1I [debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: [u'--verbose', u'https://www.youtube.com/watch?v=JuOWoGpqg1I'] [debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8 [debug] youtube-dl version 2021.12.17 [debug] Python version 2.7.18 (CPython) - Linux-5.10.0-23-amd64-x86_64-with-debian-11.7 [debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0 [debug] Proxy map: {} [youtube] JuOWoGpqg1I: Downloading webpage ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):   File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper     return func(self, *args, **kwargs)   File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info     ie_result = ie.extract(url)   File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract     ie_result = self._real_extract(url)   File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1794, in _real_extract     'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,   File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex     raise RegexNotFoundError('Unable to extract %s' % _name) RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Hello, I have same problem
youtube-dl --verbose https://www.youtube.com/watch?v=uikbtpVZS2s
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=uikbtpVZS2s']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.6 (CPython) - Linux-5.19.0-45-generic-x86_64-with-glibc2.35
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] uikbtpVZS2s: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python3.10/dist-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python3.10/dist-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/lib/python3.10/dist-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

A temporal fix can be.
PythonFolder\Lib\site-packages\youtube_dl\extractor\youtube.py (For windows)
Replace line 1794

A temporal fix can be. PythonFolder\Lib\site-packages\youtube_dl\extractor\youtube.py (For windows) Replace line 1794

Can you please give details step how and where to do this change in the code structure ?

I am getting the same error
AttributeError                            Traceback (most recent call last)
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\pydub\audio_segment.py:670, in AudioSegment.from_file(cls, file, format, codec, parameters, start_second, duration, **kwargs)
669 if start_second is None and duration is None:
--> 670     return cls._from_safe_wav(file)
671 elif start_second is not None and duration is None:
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\pydub\audio_segment.py:818, in AudioSegment._from_safe_wav(cls, file)
817 file, close_file = _fd_or_path_or_tempfile(file, 'rb', tempfile=False)
--> 818 file.seek(0)
819 obj = cls(data=file)
AttributeError: 'bytes' object has no attribute 'seek'
During handling of the above exception, another exception occurred:
AttributeError                            Traceback (most recent call last)
Cell In[10], line 49
46 video_id = 'JSggxkx6iDo'
48 # Identify the songs and their timestamps
---> 49 identified_songs = identify_songs(video_id)
51 # Print the identified songs
52 for song in identified_songs:
Cell In[10], line 17, in identify_songs(video_id)
...
--> 678         file.seek(0)
679 elif is_format("raw") or is_format("pcm"):
680     sample_width = kwargs['sample_width']
AttributeError: 'bytes' object has no attribute 'seek'
Output is truncated. View as a scrollable element or open in a text editor. Adjust cell output settings...
FileNotFoundError                         Traceback (most recent call last)
Cell In[16], line 51
48 video_id = 'JSggxkx6iDo'
50 # Identify the songs and their timestamps
---> 51 identified_songs = identify_songs(video_id)
53 # Print the identified songs
54 for song in identified_songs:
Cell In[16], line 19, in identify_songs(video_id)
16 audio_data, _ = process.communicate()
18 # Load the audio data with AudioSegment
---> 19 audio = AudioSegment.from_wav(io.BytesIO(audio_data))
21 # Initialize the ACRCloud recognizer
22 config = {
23     'host': ACR_HOST,
24     'access_key': ACR_ACCESS_KEY,
25     'access_secret': ACR_ACCESS_SECRET,
26 }
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\pydub\audio_segment.py:808, in AudioSegment.from_wav(cls, file, parameters)
806 @classmethod
807 def from_wav(cls, file, parameters=None):
--> 808     return cls.from_file(file, 'wav', parameters=parameters)
...
1524     self._close_pipe_fds(p2cread, p2cwrite,
1525                          c2pread, c2pwrite,
1526                          errread, errwrite)
FileNotFoundError: [WinError 2] The system cannot find the file specified
Output is truncated. View as a scrollable element or open in a text editor. Adjust cell output settings...
[youtube] vou3ULwk9VQ: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
RegexNotFoundError                        Traceback (most recent call last)
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\youtube_dl\YoutubeDL.py:815, in YoutubeDL.__handle_extraction_exceptions..wrapper(self, *args, **kwargs)
814 try:
--> 815     return func(self, *args, **kwargs)
816 except GeoRestrictedError as e:
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\youtube_dl\YoutubeDL.py:836, in YoutubeDL.__extract_info(self, url, ie, download, extra_info, process)
834 @__handle_extraction_exceptions
835 def __extract_info(self, url, ie, download, extra_info, process):
--> 836     ie_result = ie.extract(url)
837     if ie_result is None:  # Finished already (backwards compatibility; listformats and friends should be moved here)
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\youtube_dl\extractor\common.py:534, in InfoExtractor.extract(self, url)
533 self.initialize()
--> 534 ie_result = self._real_extract(url)
535 if self._x_forwarded_for_ip:
File ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\youtube_dl\extractor\youtube.py:1794, in YoutubeIE._real_extract(self, url)
1782 owner_profile_url = microformat.get('ownerProfileUrl')
1784 info = {
1785     'id': video_id,
1786     'title': self._live_title(video_title) if is_live else video_title,
1787     'formats': formats,
1788     'thumbnails': thumbnails,
...
597         exc_info = sys.exc_info()
--> 598     raise DownloadError(message, exc_info)
599 self._download_retcode = 1
DownloadError: ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Output is truncated. View as a scrollable element or open in a text editor. Adjust cell output settings...

In your case u need change this file, ~\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\youtube_dl\extractor\youtube.py](https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/Admin/Downloads/my-music-analysis-main/my-music-analysis-main/~/AppData/Local/Packages/PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0/LocalCache/local-packages/Python311/site-packages/youtube_dl/extractor/youtube.py
With Visual Studio Code u can access to the line 1794, the content of the line is 'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
replace it with 'uploader_id': None,

> /usr/local/lib/python3.10/dist-packages/youtube_dl/extractor/youtube.py
@rafae56038, thank you. It is works for me. But there is the following problem: low download speed
$ youtube-dl https://www.youtube.com/watch?v=uikbtpVZS2s
[youtube] uikbtpVZS2s: Downloading webpage
[download] Resuming download at byte 1103272
[download] Destination: 01 - Relational Model & Relational Algebra (CMU Intro to Database Systems _ Fall 2022)-uikbtpVZS2s.mp4
[download]   0.9% of 208.99MiB at 59.48KiB/s ETA 59:25^C

$yt-dlp  https://www.youtube.com/watch?v=uikbtpVZS2s
[youtube] Extracting URL: https://www.youtube.com/watch?v=uikbtpVZS2s
[youtube] uikbtpVZS2s: Downloading webpage
[youtube] uikbtpVZS2s: Downloading ios player API JSON
[youtube] uikbtpVZS2s: Downloading android player API JSON
[youtube] uikbtpVZS2s: Downloading m3u8 information
[info] uikbtpVZS2s: Downloading 1 format(s): 22
[download] Destination: 01 - Relational Model & Relational Algebra (CMU Intro to Database Systems ⧸ Fall 2022) [uikbtpVZS2s].mp4
[download]  27.4% of  208.99MiB at    7.94MiB/s ETA 00:19^C

OP and all commenters, especially those offering home-grown solutions: #30839 !


Search in Registry - Search by project title and Description

Forge task 36915

Submitted pr on qipr repo

Done, though there is a bug. Please view issues page on qipr project


Add Utility Network System Tables: Tables not added to map

Describe the bug
When the tool is run with the "add outputs to map" option unchecked in a user's environment profile the layers are not added to the map.
To Reproduce
Steps to reproduce the behavior:

Go to Project > Options > Geoprocessing and uncheck "Add output datasets to an open map"
Click ok
Open a new map
Open the Add Utility Network System Tables tool
Select a Utility Network (in my environment it was hosted in a FGDB)
Selecta all the tables
Run the tool
The tool completes without warnings or errors, but no tables are added to the map
I see that there are 4 table views created in the file geodatabase

Expected behavior
All four tables should be added o the map with the supplied names
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 Enterprise
Browser: N/A
Version: ArcGIS Pro 2.9.0

Additional context
Performed against a standalone Utility Network in a file geodatabase.

That's by design.
When Add output datasets to an open map is unchecked, then the results from GP tools are not added to the active map. Most (all?) GP tools honor this setting.

Because this tool is relying on that functionality to add the layer to the map it would make sense that either the tool would temporarily set that environment variable to be true then reset it once the layers had been added, or use another mechanism to add the layers to the map.

as designed


openjpeg: disable tests for aarch64 linux

Fails on hydra https://hydra.nixos.org/build/98332124

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

I don't see a rebuild on x86_64-linux https://github.com/NixOS/nixpkgs/pull/65926/checks?check_run_id=184880075 so should be good to master.

The log looks like the build is actually failing and the tests only fail because the tested binaries were not created. Am I missing something? Why is it reasonable to disable tests here?

It was reasonable to disable the tests because they always failed on aarch64-linux since they were enabled.

because the tested binaries were not created.

Care to explain what a tested binary is in this context?
This seems to build fine again

https://hydra.nixos.org/build/98513719#tabs-summary

Care to explain what a tested binary is in this context?

Well first the log says
Error(s) when building project

And then
Unable to find executable: /build/source/build/build/bin/opj_compress

So to me it sounds like the openjpg binaries failed to compile and the tests failed because the binaries were not present.

I've inspected the output by realizing the store path in the hydra link, and I can see the binaries, and I ran one of them on a aarch64 machine.

Curious. I can't find any details on the supposed build errors either :man_shrugging:
Well at least it seems to work.


Migration between different data center.

Is it possible to do the migration from one data center to another data center?

Hi,
Depends on what you need, you can use SRDF to migrate data, any flavor and do a fialover to the remote site and the data is available.
or
There is a non-disruptive migration feature on powermax and vmax all flash arrays that utilizes SRDF to accomplish this.  We support this in PyU4V 9.1.  There are caveats to be aware of at the host side to do a non-disruptive migration as the host actions have to be controlled.
https://www.dellemc.com/ar-sa/collaterals/unauth/technical-guides-support-information/products/storage/h17133-non-disruptive-migration-best-practices-and-operational-guide.pdf
Migration functions are under migrations module in PyU4V 9.1.


Performance slow down

Hi, Basil
Would you recommend something to boost rendering composables with these shadow effects some way?
10-12 simple composables in lazy column render ~50-70 ms,
compared to multiple drawRoundRect() for ex. they render ~7-8ms.
(CPU profiling of release build on Samsung A8+, RenderThread, RSBlur type)
HorizontalPager with several lists looks laggy during scroll animation.
Thanks!

Hi, thanks for commenting.
As for the problem you have, yes, that is the predicted outcome, since RSBlurShadow is drawing a shadow using the bitmap draw and RenderScript init process, which makes such lags. It is more for the static content.
Try to use the SoftLayerShadow instead or Elevation.
Or, if you want to use RSBlurShadow, then you have to create the following: create the instance of RenderScript library one time, and if you have static item heights, then you can create one bitmap and reuse it in each item. If the item's heights are dynamic, then it will be not possible to achieve easily.


[WIP] Add support for Texture transforms (center, rotate, repeat)

#91
Summary

Disable UV transform before rendering textures to RenderTarget2DArray
Add uv transform matrix for color map to MaterialsTexture
Transform UV coordinates in fragment shader before sampling color map

Tasks

[ ] Add support for metalnessMap transform
[ ] Add support for roughnessMap transform
[ ] Add support for transmissionMap transform
[ ] Add support for emissiveMap transform
[ ] Add. support for normalMap transform

This is great! Looks like a good start. Let me know when you're ready for some feedback / thoughts on organization and style - don't want to get ahead of things if you're still working on it. But nice work!
One quick note is I prefer not to commit changes to the package-lock.json in PRs so it would be best if you could revert that before we get this merged.

Cool! Feedback is welcome at this point. I imagine that the code for the other textures will be very similar so we might as well cover style / organizational stuff now. Also I will remove the package-lock and the debug texture that I added.

@gkjohnson The PR is ready for final review :) I just pushed some updates based on your feedback and added support for the last few texture transforms.

It looks like the "materialBall" example is broken -- looks like an issue in the render target array update?

Got catch. Looks like texture is undefined. When is that the case?

This code all looks great! Thank you again for the contributions -- nice work! Let me know if there's anything else you're interested in working on!
cc @bhouston

Awesome! Thanks for the guidance @gkjohnson I will pick another issue to work on shortly!


Question about interpolation in global-level fusion.

Hi! Thanks for contributing the code of your excellent work!
I have a question regarding the interpolation strategy in the volume rendering of global-level fusion.
I notice that here the alpha is derived by first estimating the alpha at the corner points and then interpolating. This is different from recent works that first interpolate features, and then query the alpha.
https://github.com/likojack/bnv_fusion/blob/17a64d7ba8a74946ff826e0603c5ff7827196a70/src/models/sparse_volume.py#L792
Does this strategy show better performance than the feature intepolation one? Are there any specific considerations?

We used corner points because each feature voxel is independent. We use the local coordinate rather than the global coordinate of the entire scene, which is different from some recent works.


build: Add cli as a default feature

This reverts a change from @aljazerzen, so I'll wait for his review. It's required for cargo install to work without a --features flag.
Even if we don't want this, I would vote to merge this, release 0.4.1, and then do the work to the docs & brew recipe so that works again

This actually isn't trivial, since the way cli.rs vs main.rs works has been changed. And I see a docstring that shows @aljazerzen knew that --features=cli was required. So I'll wait to hear from him. But we should decide / fix since currently the brew recipe is broken.
One option is to add add another crate prqlc which contains the CLI. When thinking through this before, I thought having a single crate with the compiler was easier for users though. In other projects, I've found it a bit annoying and unnecessary that running cargo install foo doesn't work, and actually someone has to run cargo install foo-cli...

I don't feel like adding --features=cli is too burdensome. The primary intention of the crate is to be used as a library (which should not include cli module) so it makes sense to say "hey I want the CLI flavor".
In initial intention here was to remove cli module from docs, but that was not possible, even if I removed cli from default features. So I changed the structure (and removed main.rs) so cli binary now essentially acts as its own crate, apart from dependencies, which have to be controlled via features.
And if we don't want CLI dependencies in the library build, installing cli will never be possible with just cargo install prql-compiler. Maybe it would make most sense to has cargo install prql-compiler --features=prqlc or cargo install prqlc.

OK, that's quite reasonable. I'll change the brew recipe instead.

Maybe it would make most sense to has cargo install prql-compiler --features=prqlc or cargo install prqlc.

Yes these are reasonable; any thoughts on which?

I prefer cargo install prqlc

+1 for cargo install prqlc

Closing in favor of #1549


Permission request for xl plugins

Description

Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above
https://github.com/jenkinsci/xlrelease-plugin
https://github.com/jenkinsci/xldeploy-plugin

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

@wlee1668 Please look into the PR for approval.

@ashish_thakur will be the new maintainer

yes, ashish_thakur is my jenkins community LDAP id which i have mentioned in the yaml and @ashish-thakur07 is my github id. So permission is required for ashish_thakur.


WIP Add dart fix for TextEditingController.buildTextSpan

WIP
Part of #74908
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

This has been blocked on https://github.com/dart-lang/sdk/issues/45475
There are a number of merge conflicts here too, so I am going to close this PR until the blocking issue has been resolved.


[Docs] Ingress Support by Cilium in EKS-A

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@chrisdoherty4 @chrisnegus @csplinter


Can't read room's topic

error TS2339: Property 'topic' does not exist on type 'string'.
const topic = await room.topic()
full src/start-web.ts
https://github.com/zhuangbiaowei/OSS-bot/blob/master/src/start-web.ts
line 65~70

You should use of instead of in.
let room of rooms

P.S. We strongly recommend to use roomList instead of rooms because the previous variable name is more clear for reading.


Eliminates global layout state.

Global state was preventing the Sprity from processing multiple PNG streams within the same invocation of node. The 'default' layout became a catchall for every PNG, leading to sprites containing images from multiple streams.
My solution was to attach layout to the options object, which is unique to the stream and hidden from the user.

Bump!

@shyndman Thank you!!!

If you're curious, we're using this over on http://www.github.com/google/material-design-icons
Thanks for the package. :)

Thats funny. I didn't even notice although I use the icons in some projects :)
Thanks for the icons ;)


NameError: undefined local variable or method `node'

Cookbook version
4.0.0
Chef-client version
12.13.37
Platform Details
Centos 7.2
Scenario:
Trying to install the aws-sdk and use the api
Steps to Reproduce:
I am writing a library to create s3 buckets
class Chef
  module BucketCreator
    class << self
      include Chef::Mixin::ShellOut
      include Opscode::Aws

      def create_bucket(bucket_name)
          require_aws_sdk
          s3 = Aws::S3::Client.new()
          begin
            s3.create_bucket(bucket: bucket_name)
          rescue Aws::S3::Errors::BucketAlreadyExists
               puts 'Max number retry reached'
          end
        end
      end
    end
  end
end

But when including it in a recipe and running it, I get the error:
[2016-09-19T13:02:52+00:00] ERROR: ruby_block[start-instance] (commons::start-instance line 3) had an error: NameError: undefined local variable or method `node' for Chef::BucketCreator:Module

The reason is that the library has not access to the Chef node object
Expected Result:
I should be able to run the require_aws_sdk method with no exception
Actual Result:
Error shown above.
Is there another way to do what I'm trying to achieve?
Thanks

I'm not getting the same results with centOS. Were you able to use other AWS services ?

While technically the libraries here can be used as true libraries and consumes elsewhere that's not something we're supporting going forward. I'd highly suggest just using the AWS SDK directly. It'll probably take less effort.


Bug: Privacy Link Doesn't Open Privacy Policy

Description
I noticed that the Privacy link isn't directed to the correct place anymore. Pretty minor bug. It tried to direct to https://github.com/files-community/Files/blob/main/Privacy.md instead of the new link, https://github.com/files-community/Files/blob/main/.github/PRIVACY.md
Steps To Reproduce

Open Files
Open Settings
Go to About
Scroll down and click the Privacy button

Requirements
Expected to open the correct link
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3007
Log File
Log file doesn't really show anything.
debug.log

Thank you for the bug report! I've already opened a PR to fix this.

Thank you for the bug report! I've already opened a PR to fix this.


return 00:00:00:00:00:00 sometimes

my friend if i used .all function sometimes return to me 00:00:00:00:00:00 on some users

This should be an issue with libuv c++ library. I think that patch has landed.

The module always returns zeroes on all interfaces on macOs.
{ en0:
     { ipv6: 'fe80::100f:5aa3:96e0:937',
       mac: '00:00:00:00:00:00',
       ipv4: '<IP_ADDRESS>' },
    awdl0:
     { ipv6: 'fe80::5c42:d3ff:fe9c:b379', mac: '00:00:00:00:00:00' },
    utun0:
     { ipv6: 'fe80::26d7:ed97:caf4:3d91', mac: '00:00:00:00:00:00' } }

I was able to fix this issue by changing this if the condition on line https://github.com/scravy/node-macaddress/blob/master/index.js#L134 to
if (!ifaces[iface].mac || ifaces[iface].mac === '00:00:00:00:00:00')

@sheshnath08 I have incorporated the check for a mac address comprised of all zeroes in 0f329b4, has been released along v0.4.3


SqlDataConnection Suppresses SQLMetal Warnings

Hello,
Feature request perhaps, but SqlDataConnection (in Microsoft.FSharp.Data.TypeProviders) suppresses SQLMetal warnings, as such, sometimes stuff is missing from the generated types, to rebug, one needs to run SqlMetal Manually and look for the errors, ideally these type of warnings should be surfaced at compile time.
e.g output of SQLMetal:
Warning : SQM1019: Unable to extract stored procedure 'Alerts' from SqlServer because its result set contains columns with the same name 'XXyyyId'.

Please move this bug to http://github.com/fsprojects/FSharp.Data.TypeProviders


Implement Deref trait for BigInt

Feature Request
Implement the Deref trait for the BigInt structure to allow users to conveniently access the underlying array.
Context
The Deref trait allows types to be treated like references. By implementing it for BigInt, users can easily access the underlying array when needed. This will simplifies part of the implementations where self.0 is used and can be replaced by self after the implementation.
Proposal
Add a Deref implementation for BigInt with the associated type Target set to [u64; N].
Example
use std::ops::Deref;

#[derive(Copy, Clone, PartialEq, Eq, Hash, Zeroize)]
pub struct BigInt<const N: usize>(pub [u64; N]);

impl<const N: usize> Deref for BigInt<N> {
    type Target = [u64; N];

    fn deref(&self) -> &Self::Target {
        &self.0
    }
}

fn main() {
    let big_int = BigInt::<3>([1, 2, 3]);

    // Using Deref to access the underlying array
    let array_ref: &[u64; 3] = &*big_int;

    println!("{:?}", array_ref); // Output: [1, 2, 3]
}

This is behaviour that we want to hide. See, eg, #122

This is behaviour that we want to hide. See, eg, #769

@Pratyush Great then if I understand well the proposed solution I imagine it will make sense to implement Deref for each individual Bs in order to manipulate them then more efficiently what do you think?

Deref seems poinless here since BigInt has a public field.  In fact .0 winds up better since ownership can change.
https://github.com/arkworks-rs/algebra/blob/master/ff/src/biginteger/mod.rs#L33
We've all abused Deref of course, but ideally Deref should be reserved for smart pointers, and avoided for other types.
https://doc.rust-lang.org/stable/std/ops/trait.Deref.html
https://rust-unofficial.github.io/patterns/anti_patterns/deref.html


Divide only the necessary dimensions

Check if the distance is greater than each dimension and divide only the ones that are too big.
TODO:

[x] Update the results figure of simple_tess recipe. The model file was wrong.

Checklist:

[x] Tests for new code
[ ] Benchmark
[ ] All tests pass
[ ] Changelog entry
[ ] Can be merged

Benchmark
Here is the result from the master branch:

Computer specs
model name	: Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz
MemTotal:        3950820 kB
Tesseroids version
tessgz (Tesseroids project) 1.1.1-52-gbc07a60
Model size: 1686 model.txt
Grid size: 2656 grid.txt
Time spent on potential:
1.41user 0.00system 0:01.42elapsed 99%CPU (0avgtext+0avgdata 996maxresident)k
0inputs+192outputs (0major+317minor)pagefaults 0swaps
Time spent on gz:
6.75user 0.00system 0:06.76elapsed 99%CPU (0avgtext+0avgdata 1068maxresident)k
0inputs+192outputs (0major+343minor)pagefaults 0swaps
Time spent on gzz:
49.22user 0.00system 0:49.27elapsed 99%CPU (0avgtext+0avgdata 1064maxresident)k
0inputs+192outputs (0major+341minor)pagefaults 0swaps

and here the current result from this branch:

Computer specs
model name	: Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz
MemTotal:        3950820 kB
Tesseroids version
1.1.1-81-g8a89aee
Model size: 1686 model.txt
Grid size: 2656 grid.txt
Time spent on potential:
2.43user 0.00system 0:02.44elapsed 99%CPU (0avgtext+0avgdata 996maxresident)k
0inputs+192outputs (0major+324minor)pagefaults 0swaps
Time spent on gz:
5.37user 0.00system 0:05.38elapsed 99%CPU (0avgtext+0avgdata 1088maxresident)k
0inputs+192outputs (0major+347minor)pagefaults 0swaps
Time spent on gzz:
37.90user 0.00system 0:37.94elapsed 99%CPU (0avgtext+0avgdata 1088maxresident)k
0inputs+192outputs (0major+347minor)pagefaults 0swaps

potential is slower, which might be attributed to the increase of the distance-size ratio from 0.5  to 1. This might out weight the benefit of using a stack vs recursion.
gz and gzz are faster even though the distance-size ration increased (1 to 1.5 for gz, 2.5 to 8 for gzz).

Seems that when there are a lot of divisions happening, the advantage of a stack becomes more pronounced.


Saving file requires additional input

When saving a file that has an error/warning that is being displayed, it requires another input.
https://github.com/TomDeneire/notify-diagnostics.nvim/assets/58049266/ab68bdfd-3a08-49b4-8bc9-d68258b6c3fc

Hi,
thanks for your feedback. I think you may be able to correct this by adding some extra autocommands to the settings:
https://github.com/TomDeneire/notify-diagnostics.nvim?tab=readme-ov-file#default-settings
Perhaps BufWritePre will do the trick.
In any case, I'm going to archive this project, because I don't use it myself anymore. It was more of an experiment anyways.
I use https://github.com/RaafatTurki/corn.nvim now, with a very similar effect as my plugin.
All the best,
Tom.


add fb & c4n favicon by twitter on nepamap

amit - just wanted to create the issue as you asked. we discussed this in slack a few days ago.

I added the facebook logo. Favicon size is too small.Do we have any SVG file for favicon?

@ravinepal I added facebook logo for now.

thanks, @studenton! appreciate it!


Fix docs rendering

The docs rendering seems to not work correctly, this needs fixing.

Closed with #15


run_with_psql doesn't use the configured connection info

I have postgres setup in docker, via tcp, but when I call run_with_psql it attempts to connect to a socket. Looking at the command, it seems that it never uses host/port stuff I configured. Wonder if this is intentional?

Yeah, at the moment the run_with_psql function basically assumes a local database. It would be a simple thing to add additional function signatures to match on the additional arguments needed for the host/port stuff.
Unless @robconery or @nurugger07 have something to say to the contrary, you are more than welcome to put in a PR, or if not, I can add something this weekend.
Feel up for it?

@xivSolutions You're right, and I wouldn't mind doing it. I realize however that there might be more to this story. If postgres is in docker, and psql is on the host (mac), it's likely their versions are mismatched, and this command shouldn't assume that simply calling psql will do the right thing. This makes it a bit more involved.

Added this for v3.


[openmm] Build OpenMM 7.2 beta as openmm-beta in main label; revert openmm recipe

This will push the OpenMM 7.2 beta to openmm-beta, like we did for previous betas.
This avoids the issue where pushing to openmm under the beta label overwrites the dev label and vice-versa.
Installation of the beta would require:
conda remove --yes openmm
conda install --yes openmm-beta

If this is what we did last time, then it may be what we have to do. We could also bump the version on the dev-recipes to something higher such as 7.2.1, so there would not be a clash and would avoid the openmm-beta recipe which has to get built, then deleted later

We could also bump the version on the dev-recipes to something higher such as 7.2.1, so there would not be a clash and would avoid the openmm-beta recipe which has to get built, then deleted later

Ooh, good idea. That would allow people to upgrade in-place without mucking about with deleting openmm, installing openmm-beta, and then reinstalling all the things that depended on openmm that were wiped out by the uninstall.
@peastman : Which would you prefer?

If we really need two different recipes for dev and beta, is there some way we could at least avoid having so much duplication?  A lot of the files are identical, so could we contrive to have only one copy of them?

Can you explain exactly what the two options are, and what the advantages/disadvantages of each are?

Problem: We can't have a separate dev and beta version of openmm with the same package name because they overwrite each other. Any one package can have any number of labels, but the base package name is the starting point.
Option A:
Do what we did last time. Create a new package on the main conda-recipes channel called openmm-beta.
Pro:

Unique package which we can adjust and tinker with.
Its the solution which worked last time

Con:

Requires a different package name when installed through conda: openmm-beta vs openmm.
Requires us to delete the openmm-beta by hand when we are done with it.
Could cause confusion if you have openmm and openmm-beta installed at the same time.

Option B:
Bump the conda-dev-recipes version of openmm to a version higher than 7.2
Pro:

Easier to target specific package version while enforcing a single installable version on any given conda environment

conda install -c conda-forge -c omnia openmm <- main stable version
... -c omnia/label/dev openmm <- nightly dev version
... -c omnia/label/beta openmm <- 7.2 beta built by changes to the openmm recipe in the conda-recipes repo

No new packages when we have to then delete

Con:

Don't know what the appropriate bumped version number should be
We'll have to delete the old dev version from the cloud

How will each of these interact with other packages?  For example, as soon as we get the OpenMM beta up, I'll want to create a beta of PDBFixer that depends on the OpenMM beta.  And then I'll want to create a beta of OpenMM-Setup that depends on the betas of both OpenMM and PDBFixer.  And then users need to be able to type some command of the form conda install (...) openmm-setup and get the betas of all three packages (not the more recent dev versions).  Would either of these approaches present difficulties for that?

@peastman : OK, we should do what @Lnaden suggests and

update the dev builds to 7.2.1 dev label on openmm
push the beta builds to 7.2.0 beta label on openmm

That way, all you have to do is make sure you've added the beta label (but NOT the dev label!) and have your OpenMM-Setup and PDBFixer packages require >=7.2.0 if they need 7.2.0 specific features.
I'll close this PR and bump the dev package.


Distribute the test between android phone and tablet

Is it possible to distribute the test run between phone and tablet?
For example: 3 phone and 3 tablet are connected to the same machine and i want my tests to run only on phone and not on tablet as the app flow and apk is different

This can probably be done if adb can identify tablets separately.

This can probably be done if adb can identify tablets separately.

Have being facing a similar problem too, with a slit between emulators and real devices.
How about a solution, where we can provide a list of device id's that parallel calabash would run on.
Looking at implementing this at the moment

Check out https://github.com/rajdeepv/parallel_calabash/pull/7, it should do the trick.
I've tested locally against emulators and devices, and seems to work pretty well.


General DOM docs tasks for Fx68

See https://trello.com/c/aBs1o6Pg/86-dom-general-enhancements-fx-68 for the original trello card.
Acceptance criteria

[ ] Make target=_blank on a/area elements imply rel=noopener by default (will be put behind a pref?) — documented appropriately. HAS NOT LANDED FOR 68 AS OF JUNE 21

[x] Consider adding support for customizing scrolling behavior with Element.focus (FocusOptions / preventScroll) — documented appropriately.

[x] support document.cookie for all documents — documented appropriately.

[x] Support "noreferrer" for window.open() — documented appropriately.

[x] Fennec and Geckoview fire resize events on page-load while Chromium does not — documented appropriately.

[x] Remove MouseEvent.mozPressure. — documented appropriately.

[x] Inconsistent popup-blocker behavior with document.addEventListener document.body.addEventListener onclick auxclick — documented appropriately.

Created #1804 for the item that has not landed yet. Closing this one.


Broken demo link

The demo link in README seems not to work properly, I got a 404 after clicking it.

Thank you for pointing it out.
I will fix for this.


Windows support

At present, most configurations will fail for Windows.
Probably our best strategy is to download precompiled packages and use those.

With CMake 3.9-rc4 I successfully built with pre-downloaded Boost, HDF5 and FFTW (as per the wiki) and at commit 0eabb888fd22412a1fc418063c08ed67c9adc463
(command lines were in Powershell)
cd C:\Users\krisf\Documents\devel\SIRF-SuperBuild-CMake3.9-64
~/Documents/devel\CMake-Build\INSTALL\bin/cmake  -DBOOST_ROOT=C:/Users/krisf/Documents/devel/boost_1_63_0   -DUSE_SYSTEM_Boost=On -DUSE_SYSTEM_HDF5=On -DCMAKE_GENERATOR_PLATFORM=x64   -DCMAKE_INSTALL_PREFIX=C:\Users\krisf\Documents\devel\SIRF-SuperBuild-CMake3.9-64/INSTALL -DUSE_SYSTEM_FFTW3=On -DFFTW3_ROOT_DIR=C:/Users/krisf/Documents/devel/fftw-3.3.5-dll64 C:/Users/krisf/Documents/devel/SIRF-SuperBuild

Some difficulties encountered with HDF5:

Anaconda comes with a very old HDF5 which has a bug in its hdf5-config.cmake (pointing to a non-existent c:\aroot folder for zlib etc)
HDF5 1.10.0-patch1 has a bug that it cannot be installed with RelWithDebInfo (it has incorrect info for installing .ptb files). This is resolved in 1.10.1.
it seems impossible to force CMake to find a specific version by setting HDF5_DIR. This might be due to a bug in FindHDF5.cmake (it only goes for the config version if HDF5_ROOT is set) but even when I try to fix that, it still doesn't find it.
If CMake doesn't find the config, there are 2 problems when HDF5 is built with shared libraries (which is the default):

FindHDF5 (even in 3.9) doesn't add setting the preprocessor variable H5_BUILT_AS_DYNAMIC_LIB  to HDF5_DEFINITIONS when in Module-mode. We could fix that (see https://gitlab.kitware.com/cmake/cmake/merge_requests/1023)
Even if we'd set HDF5_DEFINITIONS, ISMRMRD/CMakeLists.txt (and Gadgetron) does not use it yet (see https://github.com/ismrmrd/ismrmrd/pull/92)

Summary:

when installing recent HDF5 libraries (in Program Files), it works with CMake 3.9, as long as we let CMake ignore our HDF5 version. Without the Program Files version, CMake finds the Anaconda-HDF5 and it breaks.
it might be possible to ignore most of these bugs if we'd build HDF5 static libraries

Just checked that adding -DHDF5_USE_STATIC_LIBRARIES=1 still needs a change to FindHDF5 as the relevant lines there are only enabled on Unix. Fix is in
https://gitlab.kitware.com/cmake/cmake/merge_requests/1023
So, sadly it seems that unless we can force CMake to find out Config-install (see https://gitlab.kitware.com/cmake/cmake/issues/17029), we can only build with an installed HDF5 in Program Files (at least until FindHDF5.cmake gets fixes)

Started testing with CMake 3.10.1 (which has the relevant fixes for FindHDF5.cmake).
Current problems:

had to fix detection of x64
CMAKE_INSTALL_PREFIX was not reset from the default (should normally have been set ok)
USE_SYSTEM_HDF5 was not correctly honoured (When I ran CMake, it said it was using the system version, but Visual Studio went ahead and downloaded source and compiled anyway).
ISMRMRD failed to link against own compiled HDF5 because that was compiled for 32-bit. message:

library machine type 'X86' conflicts with target machine type 'x64'

When building in Debug-mode, Python link fails, see https://github.com/CCPPETMR/SIRF/issues/42

Items 2 and 3 might have been because I ran CMake a few times (with deleting the cache and without) before settling on USE_SYSTEM_HDF5=On. It's strange/undesirable though. Will need to check again later.

confirmed that when deleting the cache, configuring, and then clicking USE_SYSTEM_HDF5 does work as expected (also for CMAKE_INSTALL_PREFIX). Not sure how to reproduce the above problems 2 and 3 so I am going to ignore them...

CTest fails because config option isn't passed on:
ctest -VV -C Release
...
test 1
    Start 1: SIRF_TESTS

1: Test command: "C:\Program Files\CMake\bin\ctest.exe" "test"
...
1:     Start 1: PET_TEST1
1: Test not available without configuration.  (Missing "-C <config>"?)
1: 1/3 Test #1: PET_TEST1 ........................***Not Run   0.00 sec
...
Can we add this option on other systems?

Going to the relevant subdir, PET tests work after setting PYTHONPATH! In Powershell
cd SIRF-prefix\src\SIRF-build\src\xSTIR
Env:PYTHONPATH = "C:\Users\krisf\Documents\devel\build-VC\SIRF-SuperBuild-CMAKE3.10\INSTALL\python"
ctest -VV -C Release
...
100% tests passed, 0 tests failed out of 1

MR tests fail with error
1: Traceback (most recent call last):
1:   File "fully_sampled.py", line 22, in <module>
1:     from pGadgetron import *
1:   File "C:\Users\krisf\Documents\devel\build-VC\SIRF-SuperBuild-CMAKE3.10\INSTALL\python\pGadgetron.py", line 41, in
<module>
1:     import pygadgetron
1:   File "C:\Users\krisf\Documents\devel\build-VC\SIRF-SuperBuild-CMAKE3.10\INSTALL\python\pygadgetron.py", line 17, in
 <module>
1:     _pygadgetron = swig_import_helper()
1:   File "C:\Users\krisf\Documents\devel\build-VC\SIRF-SuperBuild-CMAKE3.10\INSTALL\python\pygadgetron.py", line 16, in
 swig_import_helper
1:     return importlib.import_module('_pygadgetron')
1:   File "C:\Users\krisf\Anaconda2\lib\importlib\__init__.py", line 37, in import_module
1:     __import__(name)
1: ImportError: DLL load failed: The specified module could not be found.

This means that a DLL isn't in the Path. Sadly, Windows doesn't say which one. Obvious candidate is the ISMRMRD DLL but the following didn't help
$env:Path += ";C:\Users\krisf\Documents\devel\build-VC\SIRF-SuperBuild-CMAKE3.10\INSTALL\bin"

@evgueni-ovtchinnikov , do you know how to set the PATH correctly?

Compilation problems should be fixed with #64.
Still need to fix above DLL problem, and also create .bat and .PS1 files with appropriate settings. @evgueni-ovtchinnikov and @paskino can you find the correct settings and then creating the configure file?

Currently tested with USE_SYSTEM_BOOST=ON only, but USE_SYSTEM_FFTW ON/OFF, USE_SYSTEM_HDF5 `ON/OFF'.
old version of HDF5 (1.8.15) with out-of-date conda gives
LINK : fatal error LNK1181: cannot open input file 'C:\aroot\stage\Library\lib\z.lib' [C:\Users\krisf\Documents\devel\build-VC\SIRF-SuperBuild-CMAKE3.10\ISMRMRD-prefix\src\ISMRMRD-build\ismrmrd.vcxproj]

This is resolved with a conda update hdf5.

Regarding ctest, need to call ctest in the SIRF-build with -C ${CMAKE_CFG_INTDIR} probably. However, this might then fail when using Makefiles as CMAKE_CFG_INTDIR might be set to . as opposed to empty (I checked that on the VM, both ctest -C and ctest -C . work fine though, so this should be safe). @rijobro, can you try this on Mac?

[ ] let CMake abort if 32bit build and say to give generator

Closing this as very outdated. Updating #197 instead


add fusetools gitignore

Reasons for making this change:
Fuse is a developing frame for mobile application development, and I think adding this gitignore will help the new developers.  ^_^
Links to documentation supporting these rule changes:
https://www.fusetools.com/learn/fuse#
If this is a new template:

Link to application or project’s homepage: https://www.fusetools.com

New link confirming this: https://www.fusetools.com/docs/getting-started/getting-started#git

LGTM! I've been working in Fuse for many years, and this matches what we currently use.
(The build directory used to be called .build back in the days, but that gets deleted every time you clean your build anyway, so I'd prefer to just have build and .uno like in this PR.)

I'm going to close this out because I'm not sure of the overall value of maintaining this template in the repository.
@tim-hub if you feel like this should still be a template listed in the repository, please read the section about specialized templates and open a new pull request targeting the new location.


PreferenceActivity and Preference fragment overall style modification

Provide custom layout and style abilities for multipane PreferenceActivity.
Current looks on Kitkat:

Current looks on Lollipop and intended default behavior:

Will prolly just fix colors and alignments.

Using preference-v7. Implementation left to user.


Allow unicode parameter names in expressions

ASCII-convertible unicode parameter names are now accepted in the
SimulationResult class.
Resolves: pysb/pysb#278

Coverage increased (+0.003%) to 74.111% when pulling 3d4d1e2b428e6285fcebcd5ff3ada48b0b1d9a80 on alubbock:unicode_exprs into 1e33c98a7a51836436f0f1ee71e6183b2e4f1805 on pysb:master.

Coverage increased (+0.003%) to 74.111% when pulling bac91e1b1c62a764a3c770612a612ec2e3a389f0 on alubbock:unicode_exprs into 4ecc5d886d6529f0efc6d1f93da3d735d4a6007c on pysb:master.


Changing TestDefinitions to Jobs in e2e tests

Description
Changes proposed in this pull request:

Duplicating tests folder with name tests-new
Changing TestDefinitions to Jobs (apiVersion, kind, +restartPolicy, -disableConcurrency)
Changing config names in ha-aws-cleanup & free-aws-cleanup

Related issue(s)
See also #1043

Closing PR due to a change in the approach to the issue of removing Octopus. E2e tests will be moved to management-control-plane.


bugfix: llm - fixes issue on swap screen whereby target account was the From account name

📝 Description
In LLM on the Swap screen, if you changed the network fee then we were incorrectly displaying the target account with the From account name. This PR fixes that to not update the target account name when the network fee is changed.
I have also manually checked that you can still change the target account from one ethereum account to another, and it still works as expected.
❓ Context

Impacted projects: LLM
Linked resource(s): https://ledgerhq.atlassian.net/browse/LIVE-4781

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo
https://user-images.githubusercontent.com/119950081/206242523-ab475ab7-1c60-46ac-8891-536ee30a78d4.mp4

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

LGTM


replication with clickhouse as a subscriber causes a crash of the compute node

Steps to reproduce
Follow this instruction, but don't create a role and a database in Postgres, don't modify pg_hba.conf etc, use the neon default user and the default database instead. Crash will happen after the database connection creation in the ClickHouse.
Expected result
No crash
Actual result
Crash
Environment
Staging (ep-holy-fog-w2aoqekg)
Logs, links
Discussion:
https://neondb.slack.com/archives/C0756AKBMU0/p1720032475503459

To reproduce follow this instruction, but don't create a role and a database in Postgres, don't modify pg_hba.conf etc, use the neon default user and the default database instead. Crash will happen after the database connection creation in the ClickHouse.

Were you able to get core dump?

Were you able to get core dump?

Unfortunately https://github.com/neondatabase/neon/pull/8272 is not yet merged, still needs approve from the code owners

The backtrace is attached to the slack discussion.

#1  0x00007f5505fbeecf in NeonWALReadRemote (state=0x5652093cd200, buf=0x5652093cf200 "s", buf@entry=0x5652093cd200 "\023\321\002", startptr=140737453845920, startptr@entry=16777216, count=140737453845916, count@entry=8192, tli=tli@entry=1) at neon_walreader.c:373

so in https://github.com/neondatabase/neon/blob/main/pgxn/neon/neon_walreader.c#L373

With https://github.com/neondatabase/neon/pull/8360 merged, we should try again and confirm it doesn't crash with clickhouse

I confirm that starting a logical replication using ClickHouse doesn't cause crashes anymore.


Blink Led HIDL

Write the steps to blink LED by HIDL.

https://joaodanil0.github.io/Boards/VIM3/Android/Hal/HIDL/


build log output from /log/project/id/commit errors when waiting for pods to start

 Issue by bacongobbler
2017-07-06 23:04:10

This error seemed a little weird when considering other CI systems:
Error from server (BadRequest): container "acidrun" in pod "go-tester-1499382097536-7532df40" is waiting to start: PodInitializing
no logs for go-tester-1499382097536-7532df40

I had expected there to be a more positive "waiting..." message or a spinning wheel showing that we're just waiting for the job to start before shipping logs.

Comment by technosophos
2017-10-03 19:30:29

I think we can close this because the new API handles it differently. Feel free to re-open if I am missing something.


Remove some files that are not needed/used

Fairly trivial change. I don't see how these files are relevant anymore, but please go through the list to see if you disagree. I left things that may not be used anymore but still may contain some useful information.

I did not know that! Good to know.
But since you have moved the code elsewhere, the subfolders they refer to are not there anymore. 😄


[Feature Request] xUnit.net as default testing tool

System Information (Required)

.NET/C#

Issue Description
To make BotBuilder more open source and more cross-platform (Visual Studio Mac support) I propose start moving to xUnit testing tool.

No plans for this.


There is windows ARM64 binary release but not Linux (in snap)

Is your feature request related to a problem? Please describe.
$ sudo snap install moonlight
error: snap "moonlight" is not available on stable for this architecture
       (arm64) but exists on other architectures (amd64, i386).

Describe the solution you'd like
Wish to see the arm64 binary for Moonlight.
Describe alternatives you've considered
I wish to emphasize that Linux arm64 binary is more needed than Windows, because Windows users can use WOW64 to run x86 binary but Linux users don't have this advantage and have to seek out 64-bit native executables.
Additional context
The computer I am using is Lenovo X13s Snapdragon with Ubuntu 23.04

I'm also very interested in ARM64 Linux support...

Have you tried the flatpak version?


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Jens1809/ioBroker.pushbullet

[x] [E114] No adapter are allowed in the repo without admin3 support
[x] [E115] No license found in io-package.json
[ ] [E154] common.dependencies must contain{"js-controller": ">=2.0.0"} or {"js-controller": ">=3.0.0"}
[ ] [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: "npm owner add bluefox iobroker.pushbullet"
[ ] [E801] No repository about text found. Please go to "https://github.com/Jens1809/ioBroker.pushbullet", press the settings button beside the about title and add the description.
[ ] [E802] No topics found in the repository. Please go to "https://github.com/Jens1809/ioBroker.pushbullet", press the settings button beside the about title and add some topics.

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


Pre check for URL in target group with relabel config define or not

Current URL check take effect when relabel configs not defined at pre-check stage, just like this:
scrape_configs:
  - job_name: prometheus
    static_configs:
      - targets:
          - http://bad

If relabel configs defined as below:
scrape_configs:
  - job_name: prometheus
    relabel_configs:
      - source_labels: [__address__]
        target_label: __param_target
    static_configs:
      - targets:
          - http://bad

It's hard for troubleshooting, because the pre-check will pass through, and URL will be checked by populateLabels func in target.go, the error appears just in error log finally.
The URLs in scrape/alert target should be checked whether relabel config exists or not at pre-check stage.
It's necessary to expose the problem as much as possible at pre-check stage, such as promtool.
Signed-off-by: DrAuYueng<EMAIL_ADDRESS>

While I understand the intention, this seems to be fragile as it does not really reuse the code we use for the labels. is there a better way to do so, without re-implementing relabeling?
Maybe that Promtool itself should do an extra step with the static targets? maybe we should call populatelabels?

Agree on the term that should reuse the code for labels.
It's not pure enough to use populateLabels, because besides the URL check,
the func check other item such as scrape timeout/interval.
I think should extract func from populateLabels for relabel, like this:
func populateLabels(...) (...) {
	call DoPopulateLabels()
	call CheckAddress()
	call CheckTimeoutAndInterval()
	call ...
	return ...

}

So we can reuse the code for labels as DoPopulateLabels in pre check stage as well.

What's wrong with also checking for scrape interval?

It's has been checked in UnmarshalYAML func of config.go.

Another solution is add targets check as extra step in UnmarshalYAML of AlertmanagerConfig and ScrapeConfig.
The new check step reuse targetsFromGroup in target.go, alertmanagerFromGroup in notifier.go.

This check was present historically for the migration to prometheus 0.15. Before prometheus switched to YAML in 2015, it was allowed to put full http URI here.
Now, that check makes little sense. We could remove it, but I do not think we should make it more complex.

Nice.

@roidelapluie @LeviHarrison Latest submission make target check as a extra step in promtool checkConfig func. It's reuse TargetsFromGroup and AlertmanagerFromGroup which delegate targetsFromGroup and alertmanagerFromGroup in target.go and notifier.go.
The key issue is the address maybe changed after relabel process. It's need take a check step after that.
Whether address valid or not is according implementation of config.CheckTargetAddress which can expand to support full http URI as @roidelapluie mentioned.

The functions are still not exposed, and I feel like we would need to care about relabeling, as you have said (including in the tests).
Thanks!

@roidelapluie I think is't more suitable to refactor populateLabels func like this:
func targetsFromGroup(...) (...) {
	prepareBaseLabels()
	populateLabels()
}

func populateLabels(...) (...) {
	mergeScrapeLabels()
	relebal.Process(...)
	setDefaultLabels()
	addPortIfNecessary()
	checkAddress()
	checkTimeoutAndInterval()
	cleanMetaLabels()
        ...
	return ...
}

// Just relabel
func PopulateLabels(...) (...) {
	mergeScrapeLabels()
	relebal.Process(...)
	setDefaultLabels()
	addPortIfNecessary()
	cleanMetaLabels()
        ...
	return ...
}```

Promtool use PopulateLabels func for relabeling and get the __address__ for check.

Address check is  covered in TargetsFromGroup and AlertmanagerFromGroup func which is not clear enough.
Maybe the title should be changed as "Pre check for target after relabel process".

@roidelapluie
This PR(#9343) make scrape/alert relabel reusable.


"Cannot read property 'toLowerCase' of undefined" during run task

VSCode Version: Code - Insiders 1.9.0-insider (e68cede8b65dc918332493146c2572a7e132a451, 2017-01-27T20:36:09.447Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

vscode-clock
Compulim
0.0.1

mayaport
JonMacey
1.0.0

vscode-docker
PeterJausovec
0.0.11

code-settings-sync
Shan
2.4.2

project-manager
alefragnani
0.13.1

python
donjayamanne
0.5.6

vscode-material-theme
fredrikaverpil
0.0.3

theme-material-theme
jprestidge
1.0.1

prettify-json
mohsen1
0.0.3

mel
sator-imaging
0.1.1

trailing-spaces
shardulm94
0.2.11

jinja
wholroyd
0.0.7

Steps to Reproduce:

Use provided tasks.json and cmd+shift+p "Run task" "Cross-OS executable"

{
    "version": "0.1.0",
    "tasks": [
        {
            "taskName": "Cross-OS executable",
            "osx": {
                "command": "/foo/bar",
                "args": ["baz"]
            },
            "windows": {
                "command": "C:\\foo\\bar.exe",
                "args": ["baz"]
            },
            "isShellCommand": true,
            "isBackground": false,
            "echoCommand": true,
            "showOutput": "always"
        }
    ]
}

Expected behavior: Run /foo/bar baz on OS X and C:\foo\bar baz on Windows.
Actual behavior: Error: Cannot read property 'toLowerCase' of undefined (and the task seems to hang)

I noticed this all works fine if you supply a "dummy" command key to the JSON, however, why would you want to do that?
{
    "version": "0.1.0",
    "tasks": [
        {
            "taskName": "Cross-OS executable",
            "command": "dummy",
            "osx": {
                "command": "/foo/bar",
                "args": ["baz"]
            },
            "windows": {
                "command": "C:\\foo\\bar.exe",
                "args": ["baz"]
            },
            "isShellCommand": true,
            "isBackground": false,
            "echoCommand": true,
            "showOutput": "always"
        }
    ]
}

Reporting the same behaviour:

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-08T23:31:51.320Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

vscode-color
anseki
0.2.3

python
donjayamanne
0.5.8

Theme-TwoStones
gerane
0.0.2

Theme-wild-cherry
gerane
0.0.3

theme-onedark-sublime
joshpeng
0.3.6

cpptools
ms-vscode
0.10.1

Steps to Reproduce:

Use provided task.json
Output shows Cannot read property 'toLowerCase' of undefined
Add a "command": "dummy" as suggested by @fredrikaverpil
Still not working
Reload VSCode
Prompt for task termination
After reload it works

{
    "version": "0.1.0",
    "showOutput": "always",
    "echoCommand": true,
    "tasks": [
        {
            "taskName": "build",
            "isBuildCommand": true,
            "isShellCommand": true,
            "windows": {
                "command": "echo",
                "args": ["you"]
            },
            "linux": {
                "command": "echo",
                "args": ["big"]
            },
            "osx": {
                "command": "echo",
                "args": ["cat"]
            }
        }
    ]
}

On top of the dummy command suggested above,  for me it was not fixed until the editor was reloaded, some task needed termination.


Random CTD

Here is my crash log. It just happens randomly, hopefully its an easy fix.
https://pastebin.com/LiztCJH5

====================================================
SCANNING THE LOG FOR SPECIFIC (POSSIBLE) SUSPECTS...
LIST OF (POSSIBLE) PLUGIN SUSPECTS

COULDN'T FIND ANY PLUGIN SUSPECTS *

LIST OF (POSSIBLE) FORM ID SUSPECTS

Form ID: 0002B785 | [Fallout4.esm] | 19
Form ID: 000A1AD2 | [Fallout4.esm] | 2
Form ID: 001231B2 | [Fallout4.esm] | 2
Form ID: 0018DEB3 | [Fallout4.esm] | 9
Form ID: 0018DEDF | [Fallout4.esm] | 2

[Last number counts how many times each Form ID shows up in the crash log.]
These Form IDs were caught by Buffout 4 and some of them might be related to this crash.
You can try searching any listed Form IDs in xEdit and see if they lead to relevant records.
LIST OF DETECTED (NAMED) RECORDS

(char*) "Effects\Ambient\MistLowSlow.nif" | 3
(char*) "SetDressing\CanvasPainting01.BGSM" | 1
(char*) "setdressing\paintingsgeneric\PaintingGeneric_CanvasSwap.BGSM" | 2
Name: "BldWoodPSmRailStairsShort01RDest" | 7
Name: "BldWoodPSmRailStairsShort01RDest:11" | 3
Name: "DestNode:3" | 1
Name: "FemaleMannequinBurnt01" | 1
Name: "FootFXDirtSmall" | 2
Name: "Materials\SetDressing\PaintingsGeneric\PaintingGeneric_CanvasSwap.BGSM" | 1
Name: "MistLowSlow" | 11

Where in game did thsi crash happen? @Stationdown

Being investigated...

@Stationdown i think I found a fix. Can you reach out to me on discord

Here is the fix!


Current use of execution providers is suboptimal

Currently the library only operates with the WASM backend even in nodejs. However the better approach should be use node bindings in node and add the webgl EP to automatically fallback to wasm if webgl is not supported by the model.
Currenly in code:
let session = await InferenceSession.create(buffer, {
  // executionProviders: ["webgl"]
  executionProviders: ["wasm"]
});

Hack to at least improve in the web
let session;

try {
    session = await InferenceSession.create(buffer, {
        executionProviders: ['webgl', 'wasm']
    });
} catch (err) {
    session = await InferenceSession.create(buffer, {
        executionProviders: ['wasm']
    });
}

I can prepare a PR to improve the execution in node and web 😃

related to https://github.com/xenova/transformers.js/issues/22 and #20

Hi! Yes, please do make a PR :) any contributions are most welcome.
I just had a question about webgl support for some of the models. When initially testing with the webgl backend, I kept getting errors saying that certain operations weren't supported. It might have been because the models are quantised (8-bit), but either way, I wasn't able to get it working reliably enough.
Do you have any suggestions?

Hi! Yes, please do make a PR :) any contributions are most welcome.

Good! Let's do it!

Do you have any suggestions?

m open sourcing a project called aiui that transforms a model into an app, My first support was ONNX and it also happened to me. Not every layer is supported in webgl. It will depend on the model, but at least for the supported ones the speed will be very good.
Let me check what your script is doing.


Should BSC now be called BNB Chain

Is network id 56 now called BNB Chain?
{
    "name": "BSC",
    "alias": "bsc",
    "id": "56",
    "symbol": "BNB",
    "testnet": false,
    "providerUrl": "https://rpc.ankr.com/bsc",
    "explorer": {
      "api": {
        "url": "https://api.bscscan.com/api",
        "key": {
          "required": true,
          "hardhatEtherscanAlias": "bsc"
        }
      },
      "browserUrl": "https://bscscan.com/"
    },
    "blockTimeMs": 3009
  },

Closing this. I see that #63 address this issue


Broken backward compatibility

Hi!
For<EMAIL_ADDRESS>you should not use<EMAIL_ADDRESS>becase this packages does not support node <4. It contradicts your support. Working version for old node is 1.3.1.
Now I get error of eslint:

Oops! Something went wrong! :(
ESLint couldn't find the plugin "eslint-plugin-react". This can happen for a couple different reasons:

If ESLint is installed globally, then make sure eslint-plugin-react is also installed globally. A globally-installed ESLint cannot find a locally-installed plugin.

If ESLint is installed locally, then it's likely that the plugin isn't installed correctly. Try reinstalling by running the following:
npm i eslint-plugin-react@latest --save-dev

If you still can't figure out the problem, please stop by https://gitter.im/eslint/eslint to chat with the team.

And I get error if run simple script like:
> require('eslint-plugin-react');
TypeError: undefined is not a function
    at Object.<anonymous> (/Users/sigorilla/project/node_modules/jsx-ast-utils/lib/values/expressions/index.js:106:28)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/Users/sigorilla/project/node_modules/jsx-ast-utils/lib/values/index.js:17:20)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)

Error in Object.assign. With previous version of jsx-ast-utils plugin worked correctly.

@sigorilla this is a bug in jsx-ast-utils; see https://github.com/evcohen/jsx-ast-utils/issues/31

Fixed with https://github.com/jsx-eslint/jsx-ast-utils/pull/32


feat(middleware): add alb as an event source

What did you implement:
Add ALB as a supported event source for the correlation IDs middleware.
Closes #130

How did you implement it:
Created a new event source detector specifically for ALB. Also modified API Gateway to exclude any ELB events as both event payloads, although quite similar, do have their differences.
On API Gateway, the headers are case-sensitive, but on ALB the header names you get in the request object are all lower-cased. The change here introduces a new const for detecting user agent. The correlation ID key is still preserved as User-Agent to maintain consistency. The tests also cover this.

How can we verify it:
Add a Lambda function behind an ALB target group to view correlations IDs being detected and forwarded on where required.
ALB event test added to verify.

Todos:

[x] Write tests
[x] Write documentation
[x] Fix linting errors
[x] Make sure code coverage hasn't dropped
[x] Provide verification config / projects / resources
[x] Enable "Allow edits from maintainers" for this PR
[x] Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

bump @simontabor @theburningmonk @loujaybee


Enhancement:

Please check for duplicates or similar issues before creating this issue.
What is your suggestion, and why should it be implemented?
Add Events to make events and modcharts
Add Psych Engine Support
Add 7 To Editior
Add Easy Modding
And Add Ctrl in Story mode and Freeplay
Thats All

"why should it be implemented" to make easy modding

Add Events to make events and modcharts
Is currently planned
Add Psych Engine Support
Not feasible (especially due to Lua scripts) and won't be coming any time soon
Add 7 To Editior
which one?
Add Easy Modding
what do u mean
And Add Ctrl in Story mode and Freeplay
Could happen but for now just use Options in the pause menu

"which one?" In Menu

"what do u mean" i am just idk how to put some

which one?
Main Menu

just press 7 in main menu?

yes


[BUG] Hitting enter on CompleOrderForm resets the whole form

Steps to reproduce:
On Finish-Dash-Ticketing branch

Add items to cart and check out
Fill out Complete Order Form
with one of the fields active, hit enter

Now hitting the return/enter button will go to stripe and payment page


Fix flexbox display in IE

From my point of view, this fixes #376 reliably. Also tested that the layout still behaves correctly with Chrome, Firefox, Safari and Safari on iPhone.

This is a pretty obscure set of changes, can you explain how and why this is a valid fix?

Well - the only "real" change I made was in commit 6bd99f1, where I did the following:

add align-items: stretch; to .kg-card-markdown: IE11 has quite some issues with flexbox as you can see on caniuse or stackoverflow and other sources. It seems that IE needs this additional hint to render the markdown content correctly and to me, it seemed the least obtrusive way of doing it (idea from this comment on stackoverflow - I tried multiple of the suggestions there). Other browsers like Chrome, Firefox and Safari seem to do this by default anyway - at least, rendering of posts was still correct after this change in those browsers.
set the max-width: 840px; to .post-full-content img and .post-full-content video, as the max-width of their surrounding box is 840px anyway. Otherwise, the images overlap on the right side in IE11 as it seems not to align the images correctly. Also here, this does still look good on the other browsers as far as I can tell.

About the other commits: I just tried to keep my branch mergeable to upstream by pulling some upstream changes. I can also rebase and create a new PR if you'd like.

I agree my changes might be a bit obscure, but I guess IE's flexbox handling is even more obscure. 😄 However, as I want my blog to look good also to users stuck with company-enforced IE11 browsers, I still believe it's worth to merge, especially since it does not break compatibility with other browsers.

Hey @stjosh 👋 This PR is in need of a rebase/rebuild 😄

Sorry Kevin, I should've replied to this sooner

set the max-width: 840px; to .post-full-content img and .post-full-content video, as the max-width of their surrounding box is 840px anyway. Otherwise, the images overlap on the right side in IE11 as it seems not to align the images correctly. Also here, this does still look good on the other browsers as far as I can tell.

This is changing the design for all users, not simply fixing a bug for IE11 users - that's why I was asking for more info. Casper's entire post design is predicated on these not being the same widths. So if your fix relies on making them the same width - well then it defeats the object of the code itself.
So while you're of course very welcome to use these changes on your own fork - atm I can't see that this is a valid fix for core. My feeling is that there is no way around the IE11 issues caused by the current flexbox implementation, and that the real fix will be to do a rewrite of the overall system used to display posts. Possibly with CSS grid.


api: Speed up container status query.

StatusContainer() does not need to iterate through all containers after
it has found a matching container to return the status for.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Coverage decreased (-0.4%) to 67.388% when pulling b9101f24e12416b7edcb75f206dc188ce5f1bcb8 on jodh-intel:speed-up-container-status-query into 93aa8226e6fa7e9bb2d553b571209f4f94e96348 on containers:master.

lgtm

LGTM

Coverage decreased (-0.04%) to 67.718% when pulling e9342359f0f3dc5772318a8ad09a5e296fcd1774 on jodh-intel:speed-up-container-status-query into 93aa8226e6fa7e9bb2d553b571209f4f94e96348 on containers:master.


Update TS submodule and add asym encryption

This commit pulls in new changes to the upstream TS project and adds
asymmetric encryption support for the provider.
Signed-off-by: Ionut Mihalcea<EMAIL_ADDRESS>

Looks like the initial issue I was hoping was fixed hasn't been fixed yet, hence the test failures on asym encryption. I'll keep this open and update it once that gets fixed and upstreamed 😬

This PR is not intended to go into the 0.8.0 release as there seems to be an issue with the underlying library. Will be fixed and merged for the next release.

Closing this PR in favour of a rebased version, PR to pop up shortly.