can't delete bad virtualenv

[ x] I have searched the issues of this repo and believe that this is not a duplicate.
[ x] I have searched the documentation and believe that my question is not covered.

Question

Hi, first time using poetry today.
I went guns blazing and created a project using poetry new project-name and went ahead and added a package using poetry add package-name. A virtualenv was created - I noticed that this was for python2.7 but I wanted python3.7 (I know I should have read the docs first). I now know that I need to use pyenv as stated in the docs.
Knowing this I want to remove the bad virtualenv and create a new one for the correct version of python.
I went into /Users/me/Library/Caches/pypoetry/virtualenvs and manually deleted the folder for the venv (in this example it was called - package-name-py2.7).
However if I run poetry shell it still believes the virtualenv exists.
➜  package-name poetry shell
Virtual environment already activated: /Users/me/Library/Caches/pypoetry/virtualenvs/package-name-py2.7

I've tried clearing the cache but the following happens
poetry cache:clear --all

[MissingArguments]
Not enough arguments

cache:clear [--all] [--] <cache>

I've even tried deleting and recreating the project but it somehow still remembers the virtualenv.
I've also tried running the below to see if it creates a new virtualenv. But alas the old one still remains
pyenv install 3.7.0
pyenv local 3.7.0
poetry install

I'm running out of ideas. :(
Running on macOS Mojave v10.14 and running Poetry 0.12.2

Did you execute poetry shell before deleting your virtualenv? If so, you just have to exit the current shell and everything should be back to normal.

Hi Sebastien!
Yes I did.
I tried but it has no effect ```
➜  package-name exit
exit:59: maximum nested function level reached


[feat] Deprecate test syntax that overrides RegressionTest methods

This feature is introduced to discourage users to override RegressionTest methods and use the hooks instead. If it is necessary, the user can still override the original methods but he has to use a new decorator that is also introduced in this pr.
Fixes #1130
Things to be done:

[x] implement feature
[x] unittests for the new feature
[x] fixing the original unittests, so that they don't throw warnings

Hello @ekouts, Thank you for submitting the Pull Request!

In the file reframe/core/decorators.py:

Line 79:67: E203 whitespace before ','

Do see the ReFrame Coding Style Guide

Moving to next sprint; it's still a wip.

@jenkins-cscs retry daint kesch

We will need to update the documentation as well.

Codecov Report

Merging #1228 into master will decrease coverage by 0.64%.
The diff coverage is 82.29%.

@@            Coverage Diff             @@
##           master    #1228      +/-   ##
==========================================
- Coverage   92.00%   91.35%   -0.65%
==========================================
  Files          83       84       +1
  Lines       12166    12495     +329
==========================================
+ Hits        11193    11415     +222
- Misses        973     1080     +107

Impacted Files
Coverage Δ

unittests/test_loader.py
83.67% <70.37%> (-16.33%)
:arrow_down:

reframe/core/pipeline.py
92.70% <94.11%> (+0.33%)
:arrow_up:

reframe/core/meta.py
100.00% <100.00%> (ø)

unittests/resources/checks/frontend_checks.py
99.22% <100.00%> (+<0.01%)
:arrow_up:

...ittests/resources/checks_unlisted/kbd_interrupt.py
100.00% <100.00%> (ø)

reframe/core/config.py
51.23% <0.00%> (-33.26%)
:arrow_down:

reframe/core/schedulers/torque.py
33.33% <0.00%> (-1.81%)
:arrow_down:

unittests/test_schedulers.py
95.37% <0.00%> (-1.49%)
:arrow_down:

reframe/core/schedulers/slurm.py
53.51% <0.00%> (-1.32%)
:arrow_down:

reframe/core/schedulers/pbs.py
67.85% <0.00%> (-0.82%)
:arrow_down:

... and 15 more

Continue to review full report at Codecov.

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


[Setup] requirements.txt based on poetry & pypi setup

Includes smaller fixes to prepare for publishing PyPi.


⚠ BeyondHD has degraded performance

In bfa7626, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 1885 ms

Resolved: BeyondHD performance has improved in f958965 after 7 minutes.


Add support for handling archive permisions on hardened servers

The module downloads the file as the puppet user (in this case root) and then attempts to untar the module as the confluence user. This fails if the server is hardened and restricts permissions on downloaded files. We have to work around this by adding the following heira:
system::execs:
    chown_jira_tarball:
        command: /bin/chown -R confluence:confluence /opt/staging/
        unless: /usr/bin/stat -c "%U:%G" /opt/staging/jira/atlassian-confluence-5.7.4.tar.gz | /bin/grep confluence:confluence
        require: Staging::File[atlassian-confluence-5.7.4.tar.gz]
        before: Exec[extract atlassian-confluence-5.7.4.tar.gz]

It would be good if the module handled this itself by explicitly ensuring the archive permission are correct before attempting the extract.
This is also the case for confluence and I'll raise it there as well.

@ghorsman can you provide a patch to the module?

@igalic I will look at this issue for all the Atlassian modules.
Possibly it can be resolved with some additional parameters being specified to the staging class.
@ghorsman Can you give me any specifics on how I could reproduce this issue? just give me some pointers on the os settings used to harden to the system to reproduce the issue.

Hello: What's the status of this?

We have a similar problem with it.  We have added a umask parameter to our staging::file and staging::extract modules (to set it to 022 for all execs).  In our case, our hardening had set the umasks to 077 which rendered the files put in place unreadable.  This was a problem when run by any user other than root.
A more direct way of passing permissions would be preferable, but I could possibly recommend the addition of a umask param to staging::file and staging::extract


Fixed singular/plural example to make even more sense

Good catch, thanks a lot for your help @vburghelea !


[PornHub] 2 Persistent Errors

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.10.22
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '--proxy', 'socks5://<IP_ADDRESS>:9050', '--max-downloads', '5', '--dump-json', 'https://www.pornhub.com/model/mini-diva/videos/upload']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.10.22
[debug] Python version 3.7.4 (CPython) - Linux-5.2.5-1-ARCH-aarch64-with-arch
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1, phantomjs 2.1.1
[debug] Proxy map: {'http': 'socks5://<IP_ADDRESS>:9050', 'https': 'socks5://<IP_ADDRESS>:9050'}
ERROR: Unable to download webpage: Remote end closed connection without response (caused by RemoteDisconnected('Remote end closed connection without response')); 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.
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 627, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 2237, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.7/urllib/request.py", line 525, in open
    response = self._open(req, data)
  File "/usr/lib/python3.7/urllib/request.py", line 543, in _open
    '_open', req)
  File "/usr/lib/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.7/site-packages/youtube_dl/utils.py", line 2724, in https_open
    req, **kwargs)
  File "/usr/lib/python3.7/urllib/request.py", line 1320, in do_open
    r = h.getresponse()
  File "/usr/lib/python3.7/http/client.py", line 1336, in getresponse
    response.begin()
  File "/usr/lib/python3.7/http/client.py", line 306, in begin
    version, status, reason = self._read_status()
  File "/usr/lib/python3.7/http/client.py", line 275, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"

---------------------------------------------------------------------------------------------------------------------

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--proxy', 'socks5://<IP_ADDRESS>:9050', '--max-downloads', '5', '--dump-json', 'https://www.pornhub.com/model/siasiberia/videos/upload', '--verbose', '--write-pages']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.10.22
[debug] Python version 3.7.4 (CPython) - Linux-5.2.5-1-ARCH-aarch64-with-arch
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1, phantomjs 2.1.1
[debug] Proxy map: {'http': 'socks5://<IP_ADDRESS>:9050', 'https': 'socks5://<IP_ADDRESS>:9050'}
ERROR: Unable to extract encoded url; 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/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/pornhub.py", line 235, in _real_extract
    'encoded url').split(';')
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 1005, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract encoded url; 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.

Description

I'm using youtube-dl to create a PornHub client that fetches videos over TOR for me and every single URL gives those 2 errors. I'm not using youtube-dl embedded, I'm using os.subprocess (seems like the options I'm using doesn't work together in embedded mode, I think it has to do with the download count when using both max downloads and skip download since I only want the first 5 video's JSON, but this is for another issue). Here is the URL list (all of them fail, sometimes with the first error, sometimes with the second):
https://www.pornhub.com/model/coffincouple
https://www.pornhub.com/model/jade_skyee
https://www.pornhub.com/model/littlereislin
https://www.pornhub.com/model/lyra-fae
https://www.pornhub.com/model/marcelin-abadir
https://www.pornhub.com/model/mattiedoll
https://www.pornhub.com/model/mini-diva
https://www.pornhub.com/model/rosieandalena
https://www.pornhub.com/model/siasiberia
https://www.pornhub.com/model/solazola
https://www.pornhub.com/model/tweetney
https://www.pornhub.com/pornstar/cherry-crush
https://www.pornhub.com/pornstar/elles
https://www.pornhub.com/pornstar/flora-rodgers
https://www.pornhub.com/pornstar/indigo-white
https://www.pornhub.com/pornstar/ivy-aura
https://www.pornhub.com/pornstar/leda-elizabeth
https://www.pornhub.com/pornstar/liz-vicious
https://www.pornhub.com/pornstar/lure-lady
https://www.pornhub.com/users/fioreholland1
Maybe PornHub is trying to avoid youtube-dl? Thanks.
Dumps here: https://anonymousfiles.io/PW1q5Njh/ (is it okay to be zipped?)

It's pretty obvious that all free proxies are banned.


$hasBeenInView? property

I think it would be a welcome addition to track whether or not an element with the in-view attribute has been in view at least once.
This way, I could have a callback function that only executes once [per session]:
.button(in-view='!$hasBeenInView && $inview && doSomething()')

You can easily achieve this with something like:
<any in-view="hasBeenInView = hasBeenInView||$inview"></any>

and use the hasBeenInView for whatever you need.


Can't seem to get test ads to show up

I used a placementId from the FAN dashboard and am running on a simulator, I'm getting nothing.
          import { BannerView } from 'react-native-fbads';

          <BannerView
            placementId="my_placement_id"
            type="large"
            onClick={() => console.log('click')}
            onError={(err) => console.log('error', err)}
          />

Am using React Native 0.37 with the latest version of this library 4.0.0. When I try using 3.1.1, I get the error: "Element type is invalid: expected a string (for built-in components) or a class/function for composite components) but got: undefined
What am I missing?

Oh what... 3.1.1 has no BannerView, and 4.0.0 isn't compatible with RN < 0.40?

On Android, I actually get an error that undefined is not an object (evaluating 'CTKNativeAdManager.init'). I ran react-native link react-native-fbads.

Got something to show up on my Android emulator. Just wasn't showing up on a real device for some reason. Still having issues with iOS. Adding FBAudienceNetwork.framework won't even build, so I tried it via CocoaPods.

I enabled some logging for iOS and am getting this:
[FBAudienceNetworkLog/FBAdProvider:146 thread:1 <error>] Ad request error: Error Domain=com.facebook.ads.sdk Code=2000 "Server Error" UserInfo={NSLocalizedDescription=Server Error, FBAdErrorDetailKey={
    code = 200;
    message = "(#200) Permissions error";
    type = OAuthException;
    url = "https://graph.facebook.com/network_ads_common/";
}}

Is it cause it doesn't realize it's a test device because the test device is not being properly added?

I got it!!!
This was crazy and way too difficult to set up. This was not documented anywhere. I had to read all over StackOverflow until I found this: http://stackoverflow.com/questions/33909430/how-to-test-facebook-audience-network-ads-over-testflight
I then added the snippets to my AppDelegate.m:
@import FBAudienceNetwork;

  NSLog(@"isTestMode1: %d",[FBAdSettings isTestMode]);
  NSString *testDeviceKey = [FBAdSettings testDeviceHash];
  if (testDeviceKey) {
    [FBAdSettings setLogLevel:FBAdLogLevelLog];
    [FBAdSettings addTestDevice:testDeviceKey];
  }

and voila!

What gives? Everyone else got this to work without doing all that nonsense?
Btw, running
React Native 0.37
FacebookSDK 4.18.0
react-native-fbads 3.1.1

Great you managed to sort it out! Yes, we have been missing addTestDevice in both docs and APIs and recently figured out the reason it wasn't working.
Feel free to submit a PR fixing docs, I'll accept straight away!

@grabbou Cool! I saw your new release. So am I correct in that version > 4.0.0+ only works with React Native > 0.40? So if I'm stuck with React native 0.37, I need to stick with 3.1.1 of this library, where I just have to manually add the test device for now?

Well, in that case I think you can keep it inside AppDelegate.m and mark as to be removed when upgraded.

@grabbou Cool, cool.
Well my Android app is all set up and good to go. I am able to show test ads, and when I run it in production, I am able to show live ads. Clicked on one and Android app was just approved for Facebook Audience Network.
On iOS, I was able to show the test ad with [FBAdSettings addTestDevice:testDeviceKey]; in AppDelegate.m, however I'm not seeing a real ad after I remove it and run the app in Production/Profiling mode. For iOS, does my app actually have to be on the app store to display the ad I can click to send the approval request? I noticed in general, iOS requires a little more to get more ads most of the time, for example in my simulators where only real AdMob ads are shown on the Android simulator, but the iOS recognizes it as test: https://cloud.githubusercontent.com/assets/13257645/23438537/4acbfcf8-fde0-11e6-8e6e-147dc48fcfed.png
Any suggestions would be appreciated. Thanks. * so close *

@grabbou I set my app to public in Facebook Developer Dashboard, waited a day, and ads suddenly started showing up live on iOS (not even on the app store yet).
Everything is hooked up successfully, ready to rock and roll!


Added AddressInfo fields to CreationTransaction

Closes #245
Changes:

Added creator_info, factory_info and implementation_info fields of AddressInfo type
Unit tests

I played around with this for a bit looking for a way of decoupling the address_info and composing the methods than just having a big one fetching everything. If you want I can open an issue to address this optimization sooner than later.

To close information loop: https://github.com/gnosis/safe-client-gateway/issues/292


OpenCV 3.2 has been released

Hopefully updating is just as simple as changing a few numbers in a few places and it will just work. :)

Yes, I hope so too but you never know :)
My plan is to push first new 3.1.0 builds with Python 3.6 support to PyPI (seems that the latest commit builds are now ok) and after that I will update OpenCV to version to 3.2. If everything works fine, I'll release the 3.2.0 builds to PyPI right after that. This is just a very slow process since the builds take a lot of time.

Looks like Linux and macOS require some tinkering to get the builds working with 3.2.0.

Linux is OK, failures were caused by dropped Python 3.3 support on Numpy side. Pasting macOS errors here for easier access:

/Users/travis/build/skvark/opencv-python/opencv/modules/core/src/hal_internal.cpp:111:17: error: no matching function for call to 'sgesv_'

                sgesv_(&m, &n, (float*)a, &lda, piv, (float*)b, &m, info);

                ^~~~~~

/System/Library/Frameworks/Accelerate.framework/Frameworks/vecLib.framework/Headers/clapack.h:5571:5: note: candidate function not viable: no known conversion from 'int *' to '__CLPK_integer *' (aka 'long *') for 1st argument

int sgesv_(__CLPK_integer *__n, __CLPK_integer *__nrhs, __CLPK_real *__a,

    ^

/Users/travis/build/skvark/opencv-python/opencv/modules/core/src/hal_internal.cpp:113:17: error: no matching function for call to 'dgesv_'

                dgesv_(&m, &n, (double*)a, &lda, piv, (double*)b, &m, info);

                ^~~~~~

/System/Library/Frameworks/Accelerate.framework/Frameworks/vecLib.framework/Headers/clapack.h:2688:5: note: candidate function not viable: no known conversion from 'int *' to '__CLPK_integer *' (aka 'long *') for 1st argument

int dgesv_(__CLPK_integer *__n, __CLPK_integer *__nrhs, __CLPK_doublereal *__a,

    ^````

It looks like the LAPACK implementation on macOS doesn't except integers as parameters for those functions...

3.2.0 builds are now in PyPI. I disabled LAPACK for macOS builds for now.


[eoa-balance-and-staking-pools] added eoa-balance-and-staking-pools strategy

Fixes # .
Changes proposed in this pull request:

added eoa-balance-and-staking-pools strategy

I have restored the original yarn.lock file

added a new pr https://github.com/snapshot-labs/snapshot-strategies/pull/1297


Remove dead code.

use_rematerialization doesn't seem to be used at all.

@riccardofelluga can you please take a look at this?


Allow deletion of custom log type if custom rule index is missing

Description
In the case when the detector index is present, but the custom rule index is missing, we want to allow the deletion of custom log types to be successful instead of throwing an error.
Issues Resolved
#700
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks like delete log type is wrongly wired.
If detector index does NOT exist, we seem to directly delete the log type instead of checking if there is a rule index.
https://github.com/opensearch-project/security-analytics/blob/8be2159f8913082f018b25af55ae10b54fb55cd7/src/main/java/org/opensearch/securityanalytics/transport/TransportDeleteCustomLogTypeAction.java#L176
https://github.com/opensearch-project/security-analytics/blob/8be2159f8913082f018b25af55ae10b54fb55cd7/src/main/java/org/opensearch/securityanalytics/transport/TransportDeleteCustomLogTypeAction.java#L227-L229
Plz add check for rule index here also. extract into a method if redundant.


mkvar

I got really bored of writing
var1 = Variable("var1")
var2 = Variable("var2")
etc.

so I threw together a quick function mkvar.py that automatically generate the variable initialization from a list. Not sure if this belongs in GPkit at all, but I thought I'd put it here in case anyone else wants it.

Interesting. I'm going to need to make a text-format for GPs for the web server. Alternately, this could be a macro for vim / sublime / etc.
Alternately alternately, there's the old "modifying the globals() dict" technique for doing this live...

What if the standard format were a csv, with the columns being the same as the current argument list to Variable?
This would let people keep track of their variables in Excel (and even google spreadsheets...), which some might find handy.

Inteeeresting. Works fine for me; @pgkirsch?

Would need to figure out VectorVariables... a separate csv might be simplest, though I do a lot of changing of variables from vector to non-vector...

Hmm, there's a couple ways I would do that; first option would be to have the columns be:
Variable name, Index (if vector), Lower bound, Upper bound
then for the constraints have the var name be CONSTRAINT, the index be either GP, SP, or OBJECTIVE,  the third column be the cs list and the fourth column be the exps list

Wait you would be trying to put Variable definitions and the text representation of a nomial into the same file / format? It seems to me those might want to be separate

As long as I can still initialize my variable list in a regular text editor that's fine with me. As far as I can tell it would just add a comma to the end of my current input file format right?
Whatever the solution, I still think that a modeler ultimately needs to be able to see the initialization of the variable in the same file as where the constraints are written, to be able to see inline comments/the label.

You wouldn't even need the comma :D

Proposal for CSV format: (# comments would not be present in the actual file
Variables  # optional. format is: name, shape, unitstr, label
foo  # semicolons are banned? or need to be escaped with a backslash?
bar, , m, length of bar
vector1, 3  # vector named vector1 of length 3
matrix, "3,3"  # 2d array named matrix of shape (3,3)

# optional arbitrary whitespace
Minimize  # required for a non-'1' objective
foo

Constraints  # optional; if blank, can be used to create a model with only variables, so as to also have mkvar's functionality.
foo >= 1 + bar  # spaces required
vector1;1 + vector1;2 + vector1;3 <= 1

Substitutions  # optional. format is variable name
bar, 3

Do we really want users to be writing their constraints, objective, and substitutions in csv files? (I thought the intent was to just support the Variables section).
And, even if we support the other sections, should they maybe go in separate files, so that standard csv parsing utilities like pandas. read_csv() can just work automatically, instead of needing to split into sections?

Hmm, good point.

Is the plan to leave this open, not merge it, but keep it as an example for the eventual csv reading functionality?

If that's ok with @pgkirsch

Can one of the admins verify this patch?

This PR can be closed because there is a more elegant vim solution. The command :%s/\(\w\+\)/& Variable(‘\1’)/g takes a list of variables
foo
bar
giraffe

and converts it into
foo = Variable(‘foo’)
bar = Variable(‘bar’)
giraffe = Variable(‘giraffe’)


logger.flush is not a function

Problem
When setting Fastify's logger option to false, an error is thrown when Fastify is terminated. The error message is TypeError: logger.flush is not a function.
Steps to Reproduce

Set Fastify's logger option to false in the configuration.
Start the Fastify app and use ctrl + c to cancel it.
Observe the error thrown referencing fastify-graceful-shutdown/index.js:45:14.

We don't expect that someone disables the logger. It is stated in the README that we rely on it. I'd open an issue on fastify. The best solution would be to enable a noop implementation when logger: false instead of dealing with undefined.

Fixed in https://github.com/hemerajs/fastify-graceful-shutdown/releases/tag/v3.5.3


PR - Feature fix icons for french and danish in metadata component

added missing translation

creating new PR for this one


Dtypes of some image parameters not fixed for some cases

One file:
{('intensity_max', '<f8'),
 ('intensity_mean', '<f8'),
 ('intensity_min', '<f8'),
 ('intensity_std', '<f8'),
 ('leakage_intensity_width_1', '<f8'),
 ('leakage_intensity_width_2', '<f8'),
 ('peak_time_max', '<f8'),
 ('peak_time_mean', '<f8'),
 ('peak_time_min', '<f8'),
 ('peak_time_std', '<f8')}

Another one:
In [6]: b - a
Out[6]:
{('intensity_max', '<f4'),
 ('intensity_mean', '<f4'),
 ('intensity_min', '<f4'),
 ('intensity_std', '<f4'),
 ('leakage_intensity_width_1', '<f4'),
 ('leakage_intensity_width_2', '<f4'),
 ('peak_time_max', '<f4'),
 ('peak_time_mean', '<f4'),
 ('peak_time_min', '<f4'),
 ('peak_time_std', '<f4')}

I have the fear, that this is because np.nan is a float64, but the calculated image parameters might be float64.

Checking 200 La Palma prod3 files, I get this distribution of dtypes in the telescope image parameter table for these columns:
In [16]: Counter(dtypes)
Out[16]: Counter({dtype('float32'): 2926, dtype('float64'): 878})

specifying a dtype everywhere is a good idea, even for scalars. Originally the definition of "Field" was perhaps too lightweight, in that it didn't require the developer to specify everything and just inferred things from the values.  Moving toward being more strict will avoid these problems.

Checking 200 La Palma prod3 files, I get this distribution of dtypes in the telescope image parameter table for these columns:

Did this have to do with which machine did the analysis?

All on the same machine. So I would think the float64 nan is the culprit

For now fixed in the merge tool by casting to the dtype of the first file.

My monologue a few weeks ago: #1411

closed as duplicate of #1411


csv import "sitename" not included in csv export

the csv import includes an id field "sitename", but the output csv only provides the "SiteRef" id of the hex point. a user wanting to tie this back to their points (e.g. for mapping) won't be able to do so.
We need to include sitename as the first field in the csv export.

Fixed


[Bdt] Error code summaries for each loop

Signed-off-by: Greg Balke<EMAIL_ADDRESS>Topic: tools_error_summaries

#
head
base
diff
date
summary

0
eb53850a
d49ab03f
diff
Sep 12 8:51 PM
3 files changed, 45 insertions(+), 13 deletions(-)

Reviews in this chain:
└https://github.com/BetzDrive/bldc-controller/pull/67 [Bdt] Error code summaries for each loop


About the plugin

Hello @pankajnirwan103 , thanks for your plugin, it works well.
Now I have to find the good receipt printer which works with Printershare or an other printing app... By the way, do you use your plugin with a receipt printer or just a basic printer ?
I have an other question about the printing app that you use in your example, do you think is possible to find an equivalent of this printing app but that we can configure directly in the app or in the plugin ? I just want as when I click on "print", I have to choose which printer and that's all.
Thanks for you help.
Cheers
Thierry

Hello pankajnirwan103!
We are developing an android app with a printing function.
Therefore we really would need your help.
Do you have any experience with woosim printers?
We are happy to pay you for your work.
Please contact us.
All the best
Mario


[prespecialized metadata] Adapted to new llvm ir.

@swift-ci please smoke test and merge

@swift-ci test windows platform

Keep in mind this will fail on some other failures on macOS, Linux will just fail to build, and I think that windows is going to fail on some other things as well. But lets just see if on macOS, Windows if this fixes the issues. We may want to wait for @JDevlieghere to fix the LLVM linux build so we can run a Linux version of this (to make sure it works there) before landing this.


Add out board motor from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of amankduffers, to add out board motor to CKAN.
Please direct questions about this pull request to amankduffers.

Closing for #2693


popover 报错

ElementUI version

OS/Browers version

Vue version

Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

我的代码：click 激活

控制台：

顺序反了，必须先 popover 后才能放 v-popover


Does not seem to work on the 15" “Early 2011” MacBook Pro

Hi,
I installed Aerial on my 15" early-2011 MacBook Pro, and, while it shows an animated preview in the Screen Saver Control Panel, the screen saver itself doesn't actually show on the screen when the screen saver is invoked.
This is on a 2GHz i7 with 16 GB of RAM and an AMD Radeon HD 6490M 256 MB
Any ideas?

Which version of the screensaver do you have installed?

I think I installed the current version--the one from the download link on
https://github.com/JohnCoates/Aerial

Jack
On Tue, Jul 5, 2016 at 1:03 AM Alistair McMillan<EMAIL_ADDRESS>wrote:

Which version of the screensaver do you have installed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JohnCoates/Aerial/issues/241#issuecomment-230412213,
or mute the thread
https://github.com/notifications/unsubscribe/ATWg9Q9WYgbseh1jO5sk6rP3JEJvj-w5ks5qSg_LgaJpZM4JEvWB
.

Hi @JackOnGitHubDotCom
The newest version is available here. https://github.com/JohnCoates/Aerial/releases/tag/v1.2beta5
It lets you cache the videos on your Mac. Which should eliminate any network issues.
Give that a try and let me know how you get on.

A

Alistair,
Thank you—that worked... but also, I think it popped up my Little Snitch
Configuration app with the Screen Saver Engine highlighted, and shown as
blocking any connections. I ALLOWED the connections, and that fixed it!
Thanks again!

Jack

On Wed, Jul 6, 2016 at 11:57 AM, Alistair McMillan<EMAIL_ADDRESS>
wrote:

Hi @JackOnGitHubDotCom https://github.com/JackOnGitHubDotCom
The newest version is available here.
https://github.com/JohnCoates/Aerial/releases/tag/v1.2beta5
It lets you cache the videos on your Mac. Which should eliminate any
network issues.
Give that a try and let me know how you get on.

A

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JohnCoates/Aerial/issues/241#issuecomment-230870535,
or mute the thread
https://github.com/notifications/unsubscribe/ATWg9STLwqJ70spnPDtLEzRw8dOK1TI4ks5qS_qfgaJpZM4JEvWB
.

@JackOnGitHubDotCom Glad it's working for you now. :)


Fix issue of missing packages when using Python compiled from source.

Debian Python packages reside at /usr/lib/python2.7/dist-packages (Python2) or /usr/lib/python3/dist-packages (Python3) while compiled Python resides at /usr/local/bin/python/ and packages installed at /usr/local/python<version>/site-packages. This means compiled Python cant access Debian Python packages.
This can be resolved by setting PYTHONPATH env variable to point to dist-packages.

@Page- , @petrosagg please review this


Update azure cli 2.0 syntax

Hi @advishnuprasad, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thank you!


No time for Custom Race (issue)

It's necessary to go to "laps" and then return to "time" so that the field appears

Hi Luis, what browser are you using?

I found this issue while working on a different feature. will try and back port it into v1.4


Fixes #1871, Update Enforcer to use the common gcp_api compute client.

Fixes #1871.

Remove one off compute API implementation from Enforcer
Update the Compute Insert/Update/Delete firewall rules to take an
optional retry argument
Update the common date_time library and remove the requirement for
the google python datelib module.
Switch enforcement from running multiple simultaneous operations to
running a single operation in blocking mode. This will reduce load on
the back end and should improve reliability of operations.

Here's a handy checklist to ensure your PR goes smoothly.

[x] I signed Google's Contributor License Agreement
[x] My code conforms to Google's python style.
[x] My PR at a minimum doesn't decrease unit-test coverage (if applicable).
[x] My PR has been functionally tested.
[x] All of the unit-tests still pass.
[x] Running pylint --rcfile=pylintrc passes.

These guidelines and more can be found in our contributing guidelines.

Tested on GCP, merging.


Fix personalization click handler

Description
We were causing a click collection call on every click because the Personalization was returning empty decisions meta data, and the event was considered not Empty.
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[x] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.

👍


feat: add i18n for pages which include button, label, and title

Fixes #10103
Motivation
Make the UI be easier for non-English speakers. I didn't finish all the translation work, please let me know if you have any ideas or suggestions.
Modifications

Verification
Manually verify.

There are many Chinese users. This is a good beginning！

There are many Chinese users. This is a good beginning！

Please feel free to give it a review. And thanks for your response.

Do we need to focus these? Or you will follow here later.

Consider this is the first PR of i18n. The most important meaning of it is having the i18n feature. And we will add all the Chinese words evantually.

OK. For users, consistency in UI style is very important.

OK. For users, consistency in UI style is very important.

Sure.

cc @terrytangyuan as a Lead (this would be a pretty big feature) and having responded to the original issue #10103.


<EMAIL_ADDRESS>hash check failed

PS C:\Users\silen> scoop install SumatraPDF
WARN  Scoop uses 'aria2c' for multi-connection downloads.
WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.
Installing 'SumatraPDF' (3.1.2) [64bit]
Loading SumatraPDF-3.1.2-64.zip from cache.
Checking hash of SumatraPDF-3.1.2-64.zip ... ERROR Hash check failed!
App:         extras/SumatraPDF
URL:         https://www.sumatrapdfreader.org/dl/SumatraPDF-3.1.2-64.zip
First bytes: 50 4B 03 04 14 00 08 00
Expected:    3c07b3fe267251d32a17f67ed5070018d69c8436df33e6db3ce757889b734676
Actual:      ddbf1477d401d6429e7a409ff5d048836f4a43a329229f01e6ba589b4907c726
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop-extras/issues/new?title=SumatraPDF%403.1.2%3A+hash+check+failed

Cannot reproduce from 4 different location.
scoop cache rm sumatrapdf; scoop update SumatraPDF -f


[BUG]: Not Getting Capacity Alerts if Approved Capacity is Null

Describe the bug
If the Approved Capacity is null, then you do not get any Capacity Alerts
Component (please tell us which flow or app you are experiencing issues with):

Solution Core
Version 2.0
App or Flow Get Capacity Alerts

To Reproduce
Steps to reproduce the behavior:
set the Approved Capacity to null
Expected behavior
If the actual capacity is > 0 and is less than the approved capacity is null, then I would have expected there to have been a Capacity Alert.  There is no capacity alert if the approved capacity is null.  This is because the "List Environment Capacity Information" has a filter in it which filters out record where the approved capacity is null (See screenshot below)
Screenshots

Hello. This is by design. We do not want to send alerts unless the admin has explicitly decided to add capacity limits for the environments. You just need to enter these to be some default value if you want to have alerts for all your environments.
Thanks for using CoE

How do you add these default values ?

From: Jenefer Monroe @.>
Sent: 14 April 2021 18:18
To: microsoft/powerapps-tools @.>
Cc: Nigel Price @.>; Author @.>
Subject: Re: [microsoft/powerapps-tools] [BUG]: Not Getting Capacity Alerts if Approved Capacity is Null (#932)
Hello. This is by design. We do not want to send alerts unless the admin has explicitly decided to add capacity limits for the environments. You just need to enter these to be some default value if you want to have alerts for all your environments.
Thanks for using CoE
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/powerapps-tools/issues/932#issuecomment-819684766, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACAXJUEOYVLMD3LWGCUIA5LTIXFADANCNFSM425XVHBQ.

If you're looking for a daily capacity report, you may find it easier to set up a separate flow following these steps: https://docs.microsoft.com/en-us/power-platform/admin/programmability-tutorial-create-daily-capacity-report


Fix incorrect appVersion

What this PR does / why we need it:
This PR fixes wrong image version defaulting for cert-manager-controller, cert-manager-webhook and cert-manager-cainjector images.
To reproduce the issue, check out release-0.10 tag and try to install cert-manager from checked out repo:
$ cd <path_to_cert_manager_chart>
$ helm install --name cert-manager --namespace cert-manager .

After some time execute kubectl get po -n cert-manager, notice that cainjector pod is in status ErrImagePull, and controller pod fails to start with Error: unknown flag: --webhook-namespace message in logs.
This is due to wrong appVersion set in both main and cainjector's Chart.yaml files. As Helm template for image tag defaults to using Chart.yaml's appVersion key value, it makes Helm download images of v0.1.0 instead of v0.10.1.
This patch fixes value of both appVersion keys, this allows Helm to pull correct image version for 0.10.x release (v0.10.1).
Release note:

NONE

/assign @simonswine

Hello! Thanks for the PR.
This configuration is actually intentional since we set the image version during our release process. Installation of cert-manager should always be done through the targets we create and never using the in-tree manifests. You can follow the helm installation guide here.
Going to close this now but feel free to ask questions either here or #cert-manager Kubernetes slack :)
/unassign @simonswine
/close


Fixes for CLI-321 withoug using BeanUtils

Similar to #215 but without using BeanUtils.
Implementation introduces Converter and Verifier interfaces.
Converter does as it says and converts from Str to object.
Verifier verifies that string values are proper before doing the conversion.  Checks are made when the string is added to the values in the Option object.
TypeHandler is reworked to use and manage the mapping of the Converter and Verifier instances to Classes.
Methods are provided to register Converter and Verifier for a class.
Tests added for Converters and Verifiers as well as TypeHandler changes.
Changes to earlier tests:
There was a TypeHandler test that expected the value "3,5" to throw and exception when it was retrieved, that now throws an exception when the value is initially added to the Option during command line parsing.  Test has been updated to account for this.
Some javadocs need to be completed.

Codecov Report
Attention: 26 lines in your changes are missing coverage. Please review.

Comparison is base (c1f473c) 93.20% compared to head (e02c36c) 91.58%.
Report is 1 commits behind head on master.

Files
Patch %
Lines

.../main/java/org/apache/commons/cli/TypeHandler.java
75.80%
13 Missing and 2 partials :warning:

src/main/java/org/apache/commons/cli/Option.java
80.76%
1 Missing and 4 partials :warning:

...in/java/org/apache/commons/cli/ParseException.java
55.55%
3 Missing and 1 partial :warning:

.../main/java/org/apache/commons/cli/CommandLine.java
80.00%
1 Missing :warning:

...a/org/apache/commons/cli/PatternOptionBuilder.java
93.33%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #216      +/-   ##
============================================
- Coverage     93.20%   91.58%   -1.62%
- Complexity      570      582      +12
============================================
  Files            21       23       +2
  Lines          1207     1296      +89
  Branches        215      218       +3
============================================
+ Hits           1125     1187      +62
- Misses           46       66      +20
- Partials         36       43       +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Hi @Claudenw
Thank you for the PR.
Run 'mvn' by itself before pushing to catch these build failures.

@garydgregory I think this might be ready for prime time now.

How about adding support for Path?

@garydgregory Sorry about the formatting.
What would you want to see for Path?
I think that Verifier class should become a static class that simply contains static common Predicate instances.
Where Verifier is currently used in the code it will become Predicate .
I don't think there needs to be a difference. At one time I had the Verifier throwing the ParseException, but that has been fixed, so there is no valid distinction.
What are your thoughts?
I also want to move everything in o.a.c.cli.converters into o.a.c.cli

Path: feature parity with File.

@garydgregory Sorry about the formatting.
What would you want to see for Path?
I think that Verifier class should become a static class that simply contains static common Predicate instances.
Where Verifier is currently used in the code it will become Predicate .
I don't think there needs to be a difference. At one time I had the Verifier throwing the ParseException, but that has been fixed, so there is no valid distinction.
What are your thoughts?
I also want to move everything in o.a.c.cli.converters into o.a.c.cli

Path: feature parity with File.

Which can be done later.

Hi @Claudenw
Hm, I think this PR is overly complicated.
Specifically, I do not think there should be a split between verifying and converting, there should not be a whole Verifier infrastructure IMO: The converters do their parsing, and in this PR, the syntaxes between verifying and parsing don't match anyway.
For a simple example, it is legal for an Integer to have a leading + character but the regular expression for integers doesn't allow it.
It's worse if I want to have a Double parameter since the RE for that is large and complex as documented in Double.valueOf(String) and it's certainly not what this PR does:
     *  final String Digits     = "(\\p{Digit}+)";
     *  final String HexDigits  = "(\\p{XDigit}+)";
     *  // an exponent is 'e' or 'E' followed by an optionally
     *  // signed decimal integer.
     *  final String Exp        = "[eE][+-]?"+Digits;
     *  final String fpRegex    =
     *      ("[\\x00-\\x20]*"+  // Optional leading "whitespace"
     *       "[+-]?(" + // Optional sign character
     *       "NaN|" +           // "NaN" string
     *       "Infinity|" +      // "Infinity" string
     *
     *       // A decimal floating-point string representing a finite positive
     *       // number without a leading sign has at most five basic pieces:
     *       // Digits . Digits ExponentPart FloatTypeSuffix
     *       //
     *       // Since this method allows integer-only strings as input
     *       // in addition to strings of floating-point literals, the
     *       // two sub-patterns below are simplifications of the grammar
     *       // productions from section 3.10.2 of
     *       // The Java Language Specification.
     *
     *       // Digits ._opt Digits_opt ExponentPart_opt FloatTypeSuffix_opt
     *       "((("+Digits+"(\\.)?("+Digits+"?)("+Exp+")?)|"+
     *
     *       // . Digits ExponentPart_opt FloatTypeSuffix_opt
     *       "(\\.("+Digits+")("+Exp+")?)|"+
     *
     *       // Hexadecimal strings
     *       "((" +
     *        // 0[xX] HexDigits ._opt BinaryExponent FloatTypeSuffix_opt
     *        "(0[xX]" + HexDigits + "(\\.)?)|" +
     *
     *        // 0[xX] HexDigits_opt . HexDigits BinaryExponent FloatTypeSuffix_opt
     *        "(0[xX]" + HexDigits + "?(\\.)" + HexDigits + ")" +
     *
     *        ")[pP][+-]?" + Digits + "))" +
     *       "[fFdD]?))" +
     *       "[\\x00-\\x20]*");// Optional trailing "whitespace"
     *
     *  if (Pattern.matches(fpRegex, myString))
     *      Double.valueOf(myString); // Will not throw NumberFormatException
     *  else {
     *      // Perform suitable alternative action
     *  }

All of that to say, that you don't have to worry about any of this if you let a converter do its usual parsing work without trying to short-circuit the process with an additional parsing step. IOW, don't parse input twice, we don't need verifiers.
Nit: @param Javadoc tags should be listed first.

@garydgregory I have added the Path to the standard converters handled by the TypeHandler and added documentation for the entire span of this change.
I also added a set of getOptionValue() methods that accept a Supplier to create the defaults.

In working on Cassandra Stress test tool where we are trying to move to commons-cli there are cases where we want to quickly identify incorrect textual input without having to go through expensive conversion processes.  This also gives an easy way to verify things like enum values and provide better error messages than if you just call enum.valueOf(str).  Providing the option of the verifier means that often the converter is a simple call to an existing method.

@garydgregory  requested  changes made.  Also found old 'verifier' documentation still in place and removed it.


Changed npm cmd for prereq

Tested on my Xubuntu 18.04 laptop:
If the installation of these node packages are global (with the -g option), hugo server --themesDir=../.. will throw an error, saying that the autoprefixer can't be found.  However, if this option is removed, the same hugo command will the site locally for preview.

@VincentTam can you please elaborate a bit why this is needed? In general I prefer to install the dependencies globally, as they may be used in other (hugo) projcts.

@VincentTam can you please elaborate a bit why this is needed? In general I prefer to install the dependencies globally, as they may be used in other (hugo) projcts.

@hanzei I've tested these procedures on both Xubuntu and Ubutnu 18.04 with NPM installed to reproduce the error.

Clone to a new repo.
Use npm list -g to verify if the packages are installed globally.  (True in the screenshot below.)
Change directory to exampleSite/.
hugo serve -themesDir ../..
See that the generated site has no CSS in local preview (default to port 1313).

@hanzei I've overlooked the following paragraph in Hugo's docs for PostCSS

If you are using the Hugo Snap package, PostCSS and plugin(s) need to be installed locally within your Hugo site directory, e.g., npm install postcss-cli without the -g flag.

Since README recommends latest extended version of Hugo, one might want to the snap app version of Hugo to automatically get the lastest build without manual update.  It is worth putting a notice for the convenience of snap users.

Thanks @VincentTam :+1:


Add Max Jonas Werner as maintainer

I'd like to become a maintainer of this repository. At the moment I'm working on adding edit functionality to the PullRequestClient (#165) with the plan to enhance its functionality more as part of on-going work on Weave GitOps OSS and Enterprise.

I don’t have a good explanation for why the Flux core maintainers team, of which you are part of, doesn’t have write access to this repo. go-git-providers is no different to other Git packages that we all maintain e.g. go-git. Let’s discuss this at the next dev meeting?


Import features from exisiting Entities

The following are not imported on the model when creating from Entities:

Existing Named Queries

Existing hashCode/equals

Existing toString method

ID Generator (i.e. TableGenerator)

It's just unmanageable to recreate all this in the imported model.

Hi @javydreamercsw ,
Java Class Reverse engineering is not supported for hashCode/equals/toString/constructor , custom method/property/annotation/snippet/comments/javadoc.
With the DB reverse engineering , you can not recover embeddable , mapped super class , converter etc.
For Named Queries and ID Generator , I tried and it is working fine , so can you attach sample java entity to reproduce the issue for Named Queries and ID Generator reverse engineering.

I'm aware it is not supported, is a request for that. Additional methods have no reason to be removed during the import. Same with additional constructors, comments, etc. All my requests were for Java Class Reverse engeneering.

This is an example of entity with annotations and id that didn't get imported. Created with NetBeans wizards. AssigmentType.pdf

Otherwise the Java Class Reverse engineering feature is kind of non-existent if you have to convert all those into modeler format. Works well if you start from database or a new diagram.
If it is expected for this to be broadly accepted by existing users it needs to provide this support or it'll be dropped really fast.

If you point me to the piece of code that does the importing I might take a look and help if not too involved.

I would probably try something like this to parse methods, etc. and act upon it.

Great , Please find the following link for Java Class reverse engineering snippet.
Java Class reverse engineering snippet:
https://github.com/jeddict/jCode/blob/master/JPA Specification/src/org/netbeans/jpa/modeler/spec/extend/JavaClass.java#L117
Root Call:
https://github.com/jeddict/jeddict/blob/master/JPA Classes Reverse Engineering/src/org/netbeans/jpa/modeler/jcre/wizard/JPAModelGenerator.java#L51
Clas Hierarchy :
	at org.netbeans.jpa.modeler.spec.extend.JavaClass.load(JavaClass.java:120)
	at org.netbeans.jpa.modeler.spec.ManagedClass.load(ManagedClass.java:34)
	at org.netbeans.jpa.modeler.spec.IdentifiableClass.load(IdentifiableClass.java:94)
	at org.netbeans.jpa.modeler.spec.Entity.load(Entity.java:186)
	at org.netbeans.jpa.modeler.jcre.wizard.JPAModelGenerator$1.run(JPAModelGenerator.java:51)
	at org.netbeans.jpa.modeler.jcre.wizard.JPAModelGenerator$1.run(JPAModelGenerator.java:41)
	....
	at org.netbeans.jpa.modeler.jcre.wizard.JPAModelGenerator.generateJPAModel(JPAModelGenerator.java:41)
	at org.netbeans.jpa.modeler.jcre.wizard.RevEngWizardDescriptor.generateJPAModel(RevEngWizardDescriptor.java:196)
	at org.netbeans.jpa.modeler.jcre.wizard.RevEngWizardDescriptor.lambda$instantiateJCREProcess$0(RevEngWizardDescriptor.java:125)
	at org.netbeans.jpa.modeler.jcre.wizard.RevEngWizardDescriptor$$Lambda$130.18955681.run

Call intiated from JPA Classes Reverse Engineering project and reverse engineering is passed to domain object available in JPA Specification project.

Forked and looking into it.

https://github.com/jeddict/NBModeler is also required to setup.

Had to learn the hard way that to even compile the code you need the following:

Need to work on NetBeans 8.2.

Need to have Jeddict installed.

Probably need to set up instructions on how to contribute ;)

Please check : https://jeddict.github.io/page.html?l=tutorial/Contribute
This doc helps to create custom code generator but may be it will help you to setup code.

Getting error when compiling jeddict:
/home/javier/NetBeansProjects/jeddict/JPA Modeler/src/org/netbeans/jpa/modeler/specification/model/scene/JPAModelerScene.java:95: error: JPAModelerScene is not abstract and does not override abstract method createTreeView() in IModelerScene
public class JPAModelerScene extends DefaultPModelerScene<EntityMappings> {
Note: Some input files additionally use or override a deprecated API.
Note: Some input files additionally use unchecked or unsafe operations.
1 error

Any idea?

Take the latest src again of NBModeler and Jeddict

I did everything on that tutorial already, but having the error above compiling/running jeddict.

There was conflict(api not-sync) between commit of two different project , please take the latest src again of NBModeler and Jeddict.

I thought it was something like that.
Had to stop for the day. Will take a look next week.
If it wasn't for that issue I could have had some good quality time with the code.

Sadly it seems way more convoluted than expected. I don't have the time to commit to contribute.

Fixed by commit : 720eea447108ef912a3859fc4e167411ea55c71e
Duplicate ticket : #275


Remember iOS

I added the basic DropView for iOS with two ImageViews as buttons. They need a tap effect, and to support file selection when tapping send. Receive should allow the user to scan a QR code to receive files. If we can achieve a similar animated effect as Android it would be nice.

Added QR code generation to iOS today


Problem with tag condition

Component
server
Describe the bug
In a pipeline with 3 commons steps (test, build & publish), I try to made the build & publish steps only run on tag : I have to a when: event: tag condition but in this case, these two steps are executed in no case, even if I create a new tag. I tried some alternatives, like add a when: tag: v* condition, but it's does not also work  (all steps are run even if on a simple push).
System Info
{"source":"https://github.com/woodpecker-ci/woodpecker","version":"0.15.0"}

Additional context
Example of problematic pipeline : https://ci.zionetrix.net/bn8/python-mylib/build/33/2

Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A join our Discord Chat Server.

I resolve my problem by removing branch: master extra condition and leave only the event: tag one. May be we could add in documentation that branch clause could not be use with a clause tag event.


MAPREDUCE-7342. Stop RMService in TestClientRedirect.testRedirect()

The test org.apache.hadoop.mapred.TestClientRedirect.testRedirect is not idempotent and fail if run twice in the same JVM, because it pollutes some states shared among tests. It may be good to clean this state pollution so that some other tests do not fail in the future due to the shared state polluted by this test.
Detail
Running TestClientRedirect.testRedirect twice would result in the second run failing due to the following assertion error:
INFO  [main] service.AbstractService (AbstractService.java:noteFailure(267)) - Service test failed in state STARTED
org.apache.hadoop.yarn.exceptions.YarnRuntimeException: java.net.BindException:

Problem binding to [<IP_ADDRESS>:8054] java.net.BindException: Address already in use

The root cause is that the RM server listening on port 8054 is started in the first run of this test, but hasn't been stopped when the test finishes. In the second run, when the test is trying to start the RMService, it fails because port 8054 is already in use, leading to the exception.
The suggested fix is to stop the RM server in the added overridden method RMService.serviceStop().
With the proposed fix, the test does not pollute the shared state (and passes when run twice in the same JVM).
JIRA link: MAPREDUCE-7342

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  2s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 26s

trunk passed

+1 :green_heart:
compile
0m 37s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
0m 33s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
0m 34s

trunk passed

+1 :green_heart:
mvnsite
0m 37s

trunk passed

+1 :green_heart:
javadoc
0m 28s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 25s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
0m 49s

trunk passed

+1 :green_heart:
shadedclient
13m 47s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 30s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
compile
0m 26s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
0m 26s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 25s

the patch passed

+1 :green_heart:
mvnsite
0m 28s

the patch passed

+1 :green_heart:
javadoc
0m 18s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 18s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
0m 50s

the patch passed

+1 :green_heart:
shadedclient
13m 39s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
134m 59s

hadoop-mapreduce-client-jobclient in the patch passed.

+1 :green_heart:
asflicense
0m 41s

The patch does not generate ASF License warnings.

207m 59s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2968/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2968

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux d8bd48a72748 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / bb83171934c16ae3491970fce324fabf2351204c

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2968/1/testReport/

Max. process+thread count
1333 (vs. ulimit of 5500)

modules
C: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient U: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2968/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Hi @aajisaka , could you please help reviewing this change? It resolves similar issues as #2499 . Thanks!

Committed. Thank you @lzx404243 for your contribution!


Generated file is still analyzed for Code Analysis

Every file in "Generated Files*.cs" is a generated file, so they should not be analyzed for Code Analysis warnings. Using a projection with CsWinRTIncludes and CsWinRTWindowsMetadata should not add a bunch of warnings, as it does today.
Describe the bug
To Reproduce

Project a winmd using CsWinRTIncludes and CsWinRTWindowsMetadata.
There will be dozens of SA1400, SA1127, SA1308, etc...

Expected behavior
No warnings for generated code.
Version Info
CS WinRT 1.2.6
Additional context
Maybe leveraging GeneratedCodeAttribute is all that is needed?

We could probably remove some, but which Code Analysis rules are on or off is a user preference, and might be different for each of their own projects, so I don't think we can remove the warning just by tweaking how we generate the source. The GeneratedCodeAttribute seems to be specifically for this scenario.


Integrate the HiFi 5 Neural Network Library v1.5.0

Integrated HiFi 5 NN Library kernels for below operators
Equal, NotEqual, Greater, GreaterEqual, Less and LessEqual.
Signed-off-by: Anirban Mandal<EMAIL_ADDRESS>Signed-off-by: Biswa Dhal<EMAIL_ADDRESS>Signed-off-by: Bhanu Prakash Bandaru Venkata<EMAIL_ADDRESS>Signed-off-by: Chaitanya Sanjay Muley<EMAIL_ADDRESS>Signed-off-by: Harinarayanan E V<EMAIL_ADDRESS>Signed-off-by: Harshavardhan Ravindra Joshi
<EMAIL_ADDRESS>

Closing old PR. Let's reopen if needed.


Any plan for 1.19.2?

Original mod stopping on 1.18, yours bump to 1.20. 1.19 sadly asked.

Hi! If i remember correctly 1.18 version actually works on 1.19.1 and 1.19.2! 1.19.3 version changed it's rendering API and so it required patches i did for 1.20.1. Sadly when i tested 1.19.3 with 1.20.1 version mod didn't work. If more people want me to add support to 1.19.3-1.19.4 i might look into it. I didn't add support for 1.19.2+ since majority of fabric mods for 1.19 sadly stuck on 1.19.2

My fault. I've test with original lootbeamfabric in 1.19.2 found it works fine.
btw, I test your updated in 1.19.4 (just edit fabric.mod.json) it works fine too. The API hasn't changed much between 1.19.4 and 1.20.

Oh thanks, for investagation! I'll leave this issue open to see if anyone would want a 1.19.3 version, for now i'll just update fabric.mod.json :+1:


HTML Report CSS styles in Opera 45

http://rt.goaccess.io/?20170531150753
Opera	45.0.2552.881
System:	Windows 10 64-bit (WoW64)
CSS breakout at http://rt.goaccess.io/?20170531150753

Thanks for reporting this. Have you tried clearing your cache? I tested this out on Win7, Opera 45.0.2552.888 (PGO) and it works fine.

Just wanted to check in and see if it clearing your cache and restarting your browser fixed the issue? Thanks

Closing this. Feel free to reopen it if needed.


UI is unresponsive and buggy : cluster is unusable

Rancher Version: 1.2-pre1 to 1.2-pre3
**Docker Version:1.11.2 on agents and 1.12.1 on my laptop
OS and where are the hosts located? (cloud, bare metal, etc):
ubuntu on baremetal servers
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node rancher, external db in a container on the same host than the server.
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
upgrade a rancher server from 1.1 to 1.2-pre-X (firstly tested on 1.2-pre3, but also present on 1.2-pre1 and 1.2-pre-2).
Results:
UI is unresponsive: the CPU is very high, the browser (firefox and chrome), launch a warn pop up saying that the script run a slow script, and propose to kill it.
when we try to continue, either a blank page is displayed, or the CPU continue to be at top, and the ui is displayed, but we cannot do any action....
when i try to revert to 1.1.3 or 1.1.4, an error page is displayed (with a plane moving around the page).
So, my cluster is unusable, unless a fix is provided in the next 1.2-pre-4.....
Vincent from the rancher team has confirmed the bug from the UI layer  (excerpt from his answer on the forum):

The two biggest things are typeify as you've already found (converting plain objects into Ember models) and filtered-sorted-array-proxy's updateContent (sorting & filtering removed/purged resources from most screens). A combination of UI and [backwards compatible] API changes will allow typeify to do less work (recursing over only the fields that have a type that could be another model) and eliminate most filtered arrays (by not returning things we don't to show in the first place).

Expected:
Provide quickly a fix to permit to use the UI, or provide a way to rollback to 1.1.X version.

Same problem here and I agree it's very painful. I have to wait for several minutes and it only loads sometimes for me. Other members of the team can't access Rancher anymore which has a negative impact on our productivity.
Any updates on the fix? Would be happy to test an early release ;)

There are changes going into a branch today but they require cattle from master and various things are still broken.. You'll want to wait for -pre4.

Same problem for me. It's very frustrating, and I already had 2 outages that I couldn't fix in time because the UI is basically impossible to reach.
If it helps, I have a big deployment (115 servers), and I'm open to answer any question to fix this!

Hi,
i think the only possible fix is to stop your 1.2-pre-X rancher server, rollback your mysql database to a previous state (when your rancher server was running a 1.1 release), and launch a 1.1 rancher server.
hope it helps,
Charles.

@clescot How do I roll back the DB? I had a lot of changes in the infrastructure/services since i switched to 1.2-pre-X..
Is there a way to just roll back the schemas changed during the 1.2-pre upgrade?

@lucacri , i don't know some way to only rollback  schemas changes done after the 1.2-pre upgrade.
To rollback your db, you need to have previously put a backup mechanism with some kind of retention.
i use with success this mysql backup to aws image : https://hub.docker.com/r/nickbreen/mysql-backup-cron/
cheers,
Charles.

You should note that a 1.2-pre4-RC1 release is out since few hours, but i don't know if it fix, although partially, our problem......
Charles.

Please do not use v1.2.0-pre4-rc1, there are lots of infrastructure changes that still need to be tested.

@deniseschannon I installed the pre4-rc1 just because my cluster is for all intents and purposes useless at the moment. Rc1 is still unusable.. Any way I can revert to 1.1.3?
I can't rebuild the server from scratch because I have 118 hosts

Various changes are in 1.2 to do less work to keep track of changes, spend less time rendering, load less data from the server, etc.  Post-1.2.0 will do paging to render less, as that is now the bulk of the time on the client side.

Version - master 10/19
Verified fixed
Still a few refresh issues need to be resolved, those will be entered separately.  Reopen if you are still having issues in next release.


Godot is randomly unable to write to config files and file cache

Godot 3.0 beta 1 and master and 2.1.4
Windows 10 64 bits
Every so often, under pretty random occasions, I see error messages in which I see Godot is unable to write to some config file or file cache, things like Unable to write to file <path>, file in use, locked or lacking permissions.
Example here, I got this one 1 second after opening a scene:

Another one with file cache, when I use Ctrl+S:

Same thing happens to me as well.
3.0.beta.build1

This happened to me too, but at one point the Windows 10 Firewall popped up and i gave permissions, since then the problem is not happening for me. Not sure what causes this.

Also happens in 2.1.4

I'll try to give it a check.

Sounds like is related to the run project bug, windows is doing something weird when trying to open (or close) files.

String is Unicode already, and c_str() returns a wchar_t*, so I don't see what's the issue there.

Not having this issue, but I'm on Windows 10 x64 1703, not the recent 1709 in case you are.
 I constantly have it turned off.

I have disabled Thumbs.db file generation and indexing service which are both known to lock files, mostly due to NTFS being so 💥 💥 slow.
I still get the issue using the latest master(as of 14/12/2017).

Does anyone still have this issue? Maybe you ran Godot as administrator and the files got locked.

Never ran Godot as administrator.

I tried running the godot 3 beta2 here on windows as admin but still seeing this permissions/lock issue.  I tried adding an exclusion to my anti virus (avast) too for the godot app data folder, it didn't seem to help resolve the issue.

Just to update, when I temporary disabled avast anti virus, the error stopped.  When I re-enabled it occurred again.  My folder exclusion didn't seem to be enough to stop the file error.

Can't Godot just try to save again after a second on the first failed attempt? That may work around the aggressive UAC and similar systems.

@Reduz. So far, I haven't had the problem with v3.0.beta.build2, but I haven't had a lot of coding time with it yet. I'll try to hammer it a bit this week and report back.
FYI, I never had the issue on v2.4 or 3.0.alpha.build1. It only occurred for me on 3.0.beta.build1, and happened very often after changing a script and then attempting to run (saving automatically before running). Often, it would take me to the project screen (no crash, just the project screen would pop up and I'd have to manually close it). After subsequent tries it would finally work.
I've never run Godot as admin, and all versions were used under the same conditions (same drive, security permissions, etc). OS is Win 10  btw.

Windows 10 vers 1709 - disabling AVG antivirus has so far fixed this for me

disabling "Safe Save On Backup Then Rename" fixed it for me as well. is there any risk to having this disabled?

If it crashes while saving you'll end up with a corrupted scene
On Feb 7, 2018 6:53 PM, "NarayanaWalters"<EMAIL_ADDRESS>wrote:

disabling "Safe Save On Backup Then Rename" fixed it for me as well. is
there any risk to having this disabled?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/14339#issuecomment-363923473,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgJ7bcwtyxnjAT9hTfxv4Sx52YXKW8Xks5tSht1gaJpZM4Q4pSb
.

anything I should/can do about it?

To prevent game engines from messing up your work is always good to use at least a local version control @NarayanaWalters

Another peculiar thing I've noticed is the non-linear correlation between failures and the delay between retry attempts. With the currently hardcoded 1s delay between attempts (the comment in the code says 100ms, but it really is 1 second if delay_usec takes microseconds as the name implies), I usually get 2-4 failures for one or two files per save attempt (i.e. rename finally works about 2 to 4 seconds after the first failed attempt for each file). With the delay set to 1ms, I still usually get 2-4 failures, with a maximum of 15 repeated failures I've seen so far (i.e. about 15ms after the first failure + whatever the processing delay within the loop is). With the delay completely removed, I still usually get 2-4 failures back to back, with a maximum of 12 failures I've been able to get back to back. So what regularly requires 2-4 seconds of delay to successfully rename a file when the delay between attempts is 1 second, resolves itself within about 2-4ms (or 15ms in the worst case I could reproduce) when the delay is set to 1 ms. If the delay is removed completely, the rename operation succeeds much faster on average, limited only by the computer's speed, after a similar number of retries.
So, here's my proposal:

Move the error reporting code so that it reports an error only if ALL rename attempts fail
Remove the call to delay_usec completely (or set it to 1ms if you're adamant about it)
Increase the max number of attempts to 100 (a safe, large number that is insignificant if there's no delay or only 1ms delay between attempts).

The current 1s delay, if left in the code, causes unduly long save times without really achieving much in my testing.
P.S. I would make these changes myself, but I'm not set up to do pull requests for Godot yet. All my testing was done using two scene files (my own custom scene, where I was able to reproduce 13 failures back to back with a 1ms retry delay and 12 failures back to back with no retry delay, and the official 3D Platformer Demo's "stage" scene, where I was able to reproduce 15 failures back to back with a 1ms retry delay, and 7 failures back to back with the retry delay completely removed -- I keep retrying, but I can't get the latter above 7; wait, I just got it to go up to 10!). One thing to note about the tests is that I am able to test many more times when the delay is 1ms or nil, as the save operation is much faster. It's hard to test many times with the default 1s retry delay, as it's so slow. The machine I'm using is quite fast and the scene files are written to a folder that resides on two SSDs setup in a RAID0 configuration.

Should we reopen this and move to 3.1 for discussing the changes suggested by @ArdaE above, or use one of the other issues (https://github.com/godotengine/godot/issues/16274, https://github.com/godotengine/godot/issues/6393)?
Recently ran into this as well, using a laptop I loaned from a colleague - I think it was the installed antivirus, but still, having Godot lock up for a few seconds if the files are not writable becomes annoying really quick.

On a related note, this line here:
https://github.com/fire/godot/commit/186e73c0249f50bb20e76c8d09bb21593089d642#diff-fda82590f36bad1a4bc15cabc0b68409R146
Actually waits for 1 second per attempt, not 100msec right now, one zero too many.. Should this be changed?

@profan Yup seems like a typo, should be fixed. Want to submit a PR?  You just need to unblock the file like this:

(right click -> properties, on file)

I got a new computer and noticed that when I save any file (in Godot or Sublime Text), it often takes half a second to save it, which is unusual. My new computer doesn't have any third-party antivirus installed, only has the built-in Microsoft one. I didn't get the error popup in 3.0.4 though.

It is happening often in Windows 10, even if I have "Safe save on backup an rename" flag toggled on or off.

I have only the Windows Defender antivirus enabled. However, it hasn't shown any warning or anything.

I have only the Windows Defender antivirus enabled.

That's enough to cause the issue.

@jahd2602, do you have Avast installed? The first time you disable Avast, it gets fully disabled; however, after that times out or you enable it, disabling Avast again doesn't really work. It reports itself as disabled, Windows reports it as disabled, and it probably doesn't give any virus warnings, but it still keeps checking files in the background, triggering this Godot-Avast incompatibility issue. If you have Avast installed, restart your computer, disable Avast, and try saving in Godot before Avast's disable times out (after that happens, like I said above, there's no fully disabling Avast again regardless of what its UI says and what it reports to Windows).

@ArdaE No, I don't have Avast or any other antivirus installed, other than the default Windows Defender. As @OvermindDL1 points out, probably Windows Defender is what causes the error. I saw the 100ms retrial, and maybe what's happening is that it fails at first, shows the error, then it tries again, and finally saves successfully. However, in this case, the pop-up should not appear. It should only appear when, after retrying, it still does not save.

@jahd2602, you must be using a stock build of Godot. It looks like someone did implement half of my suggestions from earlier on the master branch, which includes a fix to the location where a write failure is reported. If you know how to build Godot from source, give the master branch a try.

Ooops, yes, I am using 3.0.2, I will compile master then.

@jahd2602 Please ping me to close this if this is fixed on master :)

@mhilbrunner I've been using godot from master for 2 hours and the error hasn't appeared. So it is fixed, I think.

It's actually supposed to be fixed in 3.0.3 and 3.0.4 too, as #17869 was cherry-picked.

Suffering the same problem again v3.0.4.stable.official.d916135

Hola, yo tengo el mismo problema en 3.0.5, tanto en la versión normal como en la mono

@techtide & @JA-Developer, if you know how to build the code, try my complete set of suggestions from above. So far, the two bugs I've highlighted have been fixed (regarding the units of the delay and the location of the error reporting), but the following two suggestions that worked for me haven't been implemented:

Change the delay between rename attempts to 1 ms
Increase the max number of rename attempts to 100 (a safe, large number that is insignificant if there's only 1ms delay between attempts)

If these suggestions fix the issue for you without introducing any other issues, I think they should be considered seriously.
P.S. I know the better approach would be for me to just make a pull-request, but I'm not set up for that yet, and I'm too busy with my non-godot projects. Also, when I do find the time to experiment with Godot, it's almost always on my Mac where this issue doesn't exist.

This bug still persists in 3.1.1 stable. Platform: Windows 10 pro version 1809

Same here. Using 3.1.1 and still happens.

I'm using 3.2 and just ran into this problem. But, I'm not on a Windows machine and have no anti-virus running.

From Windows 10: the problem (or something very nearly like it) still exists in 3.2.1. Godot can't save any files or write new ones, making it impossible to use without running it in Admin mode!

@WolfGangsta I reopened #6393, please continue the discussion there.

Windows 7, Godot 3.2 stable, no anti-viruses or anything

It happens to me when I save with a visible detected code error. (Guess you shouldn't think your code will work even if the compiler says it's wrong)

I had the same problem, it didn't work even with godot turned off. I found that godot was not killed in the process.


IBXC transfer bug on Weaver Wallet

Describe the bug
When I'm trying transfer my testnet İBXC to other wallet ı'm get "early transaction time" notice.
Can you solve this problem?
Reproduction
.
System Info
Wallet

Logs
--test

Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

can you send the screenshot ?


Use main and dev branches for development

Use

dev branch for active development
main branch for stable commit and potential release (tagging)
disable/remove CI like auto-merge

and clean up current branch-X.Y after all new branches and CI work fine

dev branch was created and set as default branch with latest commits from previous default branch-22.08
main branch was synched to latest dev branch https://github.com/NVIDIA/spark-rapids-benchmarks/pull/96
removed auto merge: https://github.com/NVIDIA/spark-rapids-benchmarks/pull/95
updated CI jobs to new branch and cleaned unused branch-X.Y


How to access chart.zoomX correctly ?

my code :
<apexchart type="area" ref="chartRef" :options="chartOptions" :series="series"></apexchart>
and when I  call this.$refs.chartRef.zoomX() ,
it said "Uncaught TypeError: this.$refs.chartRef.zoomX is not a function".
cosole.log(this.$refs.chartRef)
exist, and It's a Proxy, but can't find property zoomX or other method.
How can I access this method ?
use Vue.toRaw to resolve the Proxy? but it does't work.
Thanks your look, help me

I can print rawChart Object  in Init function, code:
chart.value = new ApexCharts(el.value, config); const refChart = Vue.ref(chart.value); const rawChart = Vue.toRaw(refChart.value) console.log("[rawChart]",rawChart,rawChart.zoomX);
and what should I do next step?

emm, it's this.$refs.chartRef.chart.zoomX 🤣


Comments Syntax

Would be really nice to be able to write comments in a script :)
I guess something like:
// some comment

or...
# another comment

Let me know if you need more info.

I was also thinking about this and in addition to
// some comment

it would make sense to also enable
fn /* some comment */ add(arg1, arg2) { ... }

I was already thinking where one would best implement comments. The token parser seems too late for it since it already processes the code - somewhat - char by char hence some sort of preprocessor which strips comments.
Since newlines, carriage returns and unnecessary whitespaces are considered illegal it would make sense to strip them in addition to the comments (except there is some reason that the TokenParser considers them illegal) to make for more readable code the way it is currently provided in the Readme.
Here's the location of the nl/cr/whitespace token:
https://github.com/jonathandturner/rhai/blob/master/src/parser.rs#L466
If the usage of a preprocessor is approved I could create a PR for that. Shouldn't be too hard to implement.

Closing, added to TODO. Issue #26


schema-revision: options/axes.schema.json

origin and directions should be an array of numbers no?
if anatomicalLables is the semantic version of directions do we also need one for origin?

What was directions supposed to be again? (1 0 0), (0 1 0) and (0 0 1)? origin should be an array of number, yes. Also as a general comment to that (since it applies to several schemas): We should limit the items of the array to 3, right?
Like this:
{
"type": "array",
"minItems": 3,
"maxItems": 3
}
Or are there cases where this is not true and we need fewer/more items?

I would say no. anatomicalLabels is the semantic version of directions yes, but I don't think this applies to the origin. Terms like "ventral", "dorsal", "medial", "temporal" cannot be applied to an origin, but could be in relation to the origin (as in "ventral to the origin").

 The indexingParameters have units for the axesMin and axesMax, do we need units in the axes-schema then? Do directions need units?

summary of properties:

origin (array of integers (0 to 1), items count:3) @xgui3783 : correct ??? what is assumed here as 0, 0 , 0 (right-handed Cartesian coordinate system ? this is unitless correct?)
units (controlledTerm, count: 1)
mathematicalDirections (array of integers (0 to 1), items count: 3, array count: 3) @xgui3783 : correct ??? this is unitless correct?
anatomicalDirections (controlledTerm, count: 1)
indexingParameters (indexingParameters, count: 1)

@UlrikeS91 this one needs again a bit of input / clarification first.

@lzehl Can I clarify that axes.schema.json is meant to represent all axes in a coorindate space?

origin (array of integers (0 to 1), items count:3) correct ??? what is assumed here as 0, 0 , 0 (right-handed Cartesian coordinate system ? this is unitless correct?)

I struggle to understand why origin is needed at all. I would have understood the origin is by definition [0,0,0] in a 3D space. If a non 0 value is provided, then one must ask the question, the origin is [20, 20, 20], relative to what.

units (controlledTerm, count: 1) ( if origin and direction are unitless and units are defined in indexingParameters is unit here needed?)

This is something which I will need input from @skoehnen . From the indexingParameter issue, I have expressed by concerns for exposing the indexingParameter in SANDS. Even if we put that aside, IMO, unit is a property of axes, and not the other way around. (indexingParameter can be unitless in that case, inheriting/inferring the unit from axes)

mathematicalDirections (array of integers (0 to 1), items count: 3, array count: 3) correct ??? this is unitless correct?

I can't find reference to mathematicalDirections anywhere. to me, anatomicalDirections is sufficient, and makes supercedes mathematicalDirections

Yes! thanks that you brought this up. I did not understand this part as well completely, because it originates from some discussions we had with Lionel, way back...

yes, axes.schema.json was now meant to represent ALL axes
Feel free to make a counter suggestion if you think each axes should be represented individually

I (and Ulrike) have the same question about the origin...
I assumed that the relation was always to the technical origin of images (upper, left front corner is [0,0,0]?)

good to hear your feedback on indexingParameters and axes.
Does this mean:
a) unit will be property of axes
b) should minAxes, and maxAxes also be part of axes?
c) steps in xyz should be defined in indexingParameters
d) what else is needed in indexingParameters?

Feel free to make a counter suggestion if you think each axes should be represented individually

Now that i step back a bit, I think it does make sense to have the axes represented in a single object.

I assumed that the relation was always to the technical origin of images (upper, left front corner is [0,0,0]?)

Then shouldn't it reside in a description field of some kind? something like
{
  "@type": "https://schema.ebrains.eu/sands/Axes",
  "description": "This axes describes the histological scan of Big Brain 20, slice 1336. The origin, per computer image standard, Is the top left most point of the image"
}

Or would you prefer to have a number of controlled_terms for common origins? e.g. 2D image (top left), brain atlas (somewhere in the middle) etc?

a) unit will be property of axes

this indeed is my opinion

b) should minAxes, and maxAxes also be part of axes?

Hmm, I think I remember this now. I requested this to be placed in, because I needed to convert coodrinates from PIR to RAS. I don't know the answer to this question yet. Imposing a minAxes/maxAxes seems strange (afterall, axes, by definition, should extend indefinitely). I will perhaps have to thinking about this

c) steps in xyz should be defined in indexingParameters
d) what else is needed in indexingParameters?

Hmm, if they need to be somewhere, they will have to be in indexingParameters. Steps in Axes wouldn't make sense (unless we are talking about voxel/pixel resolution). But it should not affect users from placing a landmark between two steps.
I am still under the impression that indexingParameter should not be public facing, but I think we should reserve that discussion in person/on zoom.

for the description; this is not an option because it is a specific description of a specific use case not the on of a schema.
since the discussion with the origin seems to be a bit more complex I would prefer solving this one in a meeting
the min/maxAxes then seems to be indeed better placed in indexingParameters
general meaning and usage of indexingParameters should be also discussed in the meeting
[I just send out an email to you and Stefan for a meeting, early next week]

"units" should be "unit" because it's just defining what step "1" means in each direction
"indexingParameters" need to be removed here, and put to the image schema (cf. discussion in #34 )
"origin" needs to be clearly defined (origin in relation to what?)
"directionsMatrix" should be considered again? (@xgui3783 & @skoehnen please continue discussion)

@skoehnen I still struggle to understand how directionsMatrix solves the issues of:

how one defines origin (top left image? top left front post point of a volume?)
orientation, as in, whether the increment of the first index result in translation in the LR/AP/SI direction

"origin" needs to be clearly defined (origin in relation to what?)

I wonder if we need to define where the origin is, rather than how it is defined. Consider nifti transform header. It instructs programs how to find the origin, without explaining if the origin has any meaning what-so-ever.
For our purpose, can we not just say:
sands/core/coordinateSpace/v1.0.0/bad-b01-1
{
  "origin": [10, 20, 30],
  "unit": "mm",
  "anatomicalOrientation": "RAS",
  "defaultImage": "sands/core/image/v1.0.0/bad-b01-2"
}

Then, in principle, we find the most left, most posterior, most inferior voxel fo the image, and count 10 units to the right, 20 units towards the anterior, and 30 units towards the superior, and it would be our origin?

@skoehnen this sounds like it would work. Would it work for all cases (all anatomicalOrientations?) besides RAS?

feedback from SANDS meeting:
axes.schema.json
    anatomicalOrientation [expects: controlledTerm; count: 1]
    origin [expects: number; count: 3]
    unit [expects: controlledTerm; count: 1]

feedback from SANDS meeting:
axes.schema.json
    anatomicalOrientation [expects: controlledTerm; count: 1]
    origin [expects: number; count: 3]
    unit [expects: controlledTerm; count: 1]

merge into #40


Parent is required even for top-level resources

The following linters:

https://linter.aip.dev/133/method-signature
https://linter.aip.dev/133/http-uri-parent

expect Create methods to always have a parent.
Should the linters not warn on missing parent when the resource is a top level resource/collection item?

A parent field must be included unless the resource being listed is a top-level resource.
(https://aip.dev/133)

This is a good question and I am not sure what the best answer is here. The old linter also complained about this, but it was a Cloud-specific rule (any parent other than project got a complaint).
I think the right answer here might be to make the rules here more lenient but add a Cloud-specific rule around projects and locations. Thoughts?

+1

+1


core: Implement _GO_STANDALONE mode

Closes #147. Whereas that issue describes a _GO_NO_CD_ROOTDIR variable, I realized the implications of the change were bit more far-reaching, and also provided the opportunity to clean up the help and tab completion interfaces for standalone programs.
See the updates to README.md, most notably the new "Standalone mode" section, for details.

Coverage increased (+0.02%) to 95.224% when pulling 33c5eabdc0f938ba74a551252d95c719d94b27e4 on standalone into e2bd2709fe1be69cdb5110ee4e280012874b2e5c on master.


i given correct api but it showing incorrect api

data: {
error: {
message: 'Incorrect API key provided: sk-CmlKF***************************************jP2n. You can find your API key at https://platform.openai.com/account/api-keys.',
type: 'invalid_request_error',
param: null,
code: 'invalid_api_key'
}

I believe you have solved this. If not create a new issue.


Added checking for incompatable libraries

Summary
This pull request is for adding checking for incompatible libraries in client.run()  which raises Warnings if incompatible libraries are added
Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.

[x] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[x] This PR is a breaking change (e.g. methods or parameters removed/renamed) (potentially)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Hey! Please test your changes.

why did you close this btw?

I made the changes, however this loop turns the client object into a string for some reason


bug: Some rules flag based on Properties position on a resource

What is the problem?
To address the NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod, I add an Aspect to set the RetentionInDays to 180.
When applied to a stack with some L2 constructs, the affected Cfn resource sets those properties in different orders, and as a result the resource is marked as non-compliant
Reproduction Steps
import * as cdk from 'aws-cdk-lib';
import { Stack, StackProps, Aspects, IAspect } from 'aws-cdk-lib';
import { NetworkLoadBalancedFargateService } from 'aws-cdk-lib/aws-ecs-patterns';
import { ContainerImage } from 'aws-cdk-lib/aws-ecs';
import { CfnLogGroup, LogGroup } from 'aws-cdk-lib/aws-logs';
import { Construct, IConstruct } from 'constructs';
import { NIST80053R4Checks } from 'cdk-nag';

class CdkNagTestStack extends Stack {
  constructor(scope: Construct, id: string, props?: StackProps) {
    super(scope, id, props);
    new NetworkLoadBalancedFargateService(this, 'MyNlbFargateService', {
      taskImageOptions: {
        image: ContainerImage.fromRegistry('amazon/amazon-ecs-sample'),
      }
    });
    new LogGroup(this, "MyLogGroup");
  }
}

class LogRetention implements IAspect {
  public visit(node: IConstruct): void {
    if (node instanceof CfnLogGroup) {
      node.addPropertyOverride('RetentionInDays', 180); //Ensure RetentionInDays is set
    }
  }
}

const app = new cdk.App();
new CdkNagTestStack(app, 'CdkNagTestStack');
Aspects.of(app).add(new LogRetention())
Aspects.of(app).add(new NIST80053R4Checks())

When doing a cdk synth, there are a number of nag rules violated, but NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod is triggered on only one of the AWS::Logs::LogGroups
I comment out Aspects.of(app).add(new NIST80053R4Checks()), and run a cdk synth > tmp.yml
The NetworkLoadBalancedFargateService creates a number of resources, including a AWS::Logs::LogGroup
  MyNlbFargateServiceTaskDefwebLogGroup566BBF85:
    Type: AWS::Logs::LogGroup
    UpdateReplacePolicy: Retain
    DeletionPolicy: Retain
    Metadata:
      aws:cdk:path: CdkNagTestStack/MyNlbFargateService/TaskDef/web/LogGroup/Resource
    Properties:
      RetentionInDays: 180

This resource throws the NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod
However the MyLogGroup resource is synthed as:
  MyLogGroup5C0DAD85:
    Type: AWS::Logs::LogGroup
    Properties:
      RetentionInDays: 180
    UpdateReplacePolicy: Retain
    DeletionPolicy: Retain
    Metadata:
      aws:cdk:path: CdkNagTestStack/MyLogGroup/Resource

This resource doesn't throw the NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod
Only difference I notice is the Properties comes before the resource attributes when found as compliant, and after the resource attributes when not compliant
What did you expect to happen?
The Aspect to set Log group retention days would address the NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod rule regardless of where the Properties falls under the resource
What actually happened?
NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod was hit on the following resource
  MyNlbFargateServiceTaskDefwebLogGroup566BBF85:
    Type: AWS::Logs::LogGroup
    UpdateReplacePolicy: Retain
    DeletionPolicy: Retain
    Metadata:
      aws:cdk:path: CdkNagTestStack/MyNlbFargateService/TaskDef/web/LogGroup/Resource
    Properties:
      RetentionInDays: 180

cdk-nag version
2.13.1
Language
Typescript
Other information
No response

The problem has to do with this line.
node.addPropertyOverride('RetentionInDays', 180); //Ensure RetentionInDays is set
Property overrides always take place after template synthesis even though you applied the change during  Aspects in the validation phase.
In this case there is a easy fix for that; we can directly access the CDK Cloud Assembly and modify the value from the Aspect itself! Here is a modified LogRetention Aspect
class LogRetention implements IAspect {
  public visit(node: IConstruct): void {
    if (node instanceof CfnLogGroup) {
      Annotations.of(node).addInfo(`Retention Days changed from: ${node.retentionInDays}, to: 180`); // not needed, purely informational
      node.retentionInDays = 180;
    }
  }
}

With the optional info message we can see the following in the output
[Info at /CdkNagTestStack/MyNlbFargateService/TaskDef/web/LogGroup/Resource] Retention Days changed from: undefined, to: 180
[Info at /CdkNagTestStack/MyLogGroup/Resource] Retention Days changed from: 731, to: 180

"MyLogGroup" didn't cause a NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod error when using property overrides because the CDK sets an explicit retention days of two years by default, which passes the check.
Important Note
When using Aspects to fix an issue, order of operations matter. Aspects get applied in the order that they are specified. As an experiment you can try the following with the fixed LogRetention Aspect
const app = new cdk.App();
new CdkNagTestStack(app, 'CdkNagTestStack');
Aspects.of(app).add(new NIST80053R4Checks());
Aspects.of(app).add(new LogRetention());
Aspects.of(app).add(new NIST80053R4Checks());

app.synth();

The output will also be displayed in order. Below is an abbreviated output, we can see that due to ordering that NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod is triggered the first time the NIST80053R4Checks are run and not the second time since the LogRetention Aspect was run between those two.
...Snip...

[Error at /CdkNagTestStack/MyNlbFargateService/TaskDef/web/LogGroup/Resource] NIST.800.53.R4-CloudWatchLogGroupRetentionPeriod: The CloudWatch Log Group does not have an explicit retention period configured - (Control IDs: AU-11, SI-12).

...Snip...

[Info at /CdkNagTestStack/MyNlbFargateService/TaskDef/web/LogGroup/Resource] Retention Days changed from: undefined, to: 180

...Snip...

[Info at /CdkNagTestStack/MyLogGroup/Resource] Retention Days changed from: 731, to: 180

...Snip...

Thanks @dontirun !
Modifying the Cloud Assembly did the trick! 👍