Update versions in dockerfiles for 2.2

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8622/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8626/
Test PASSed.

alluxio-bot, merge this please


[Requirement] Review and fix the Vulnerabilities raised in Github

https://github.com/Vyshantha/multiscripteditor/security/dependabot

All vulnerabilities from bots are being regularly fixed and checked-in.


First PR

Copying files from scil + changing content for tractometer pages.
A lot of tweeking still needed. Nearly all links should be updated.
Things that would be nice:

Make subtitles bigger
Improve general style: banners and other. Main page.
Hyperlinks colors! (They color on hover but base color is not a visible choice)
Fix figcaption to make it the same size as the image.

@arnaudbore , It is more or less good when tested locally. Could serve as base, we will improve the style later.

Look very nice appart from the missing logo and the banner. We should try to push this asap. Once is done ... it's done :+1:


make TauLevels faster

calculating TauLevels is slow right now. Make it faster...either in the fortran, or using multiprocessing in python.

closed with #76


Update pip tools

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

To adhere to CONTRIBUTING, you will probably want to squash the two commits into one

Hey @jonringer, that isn't quite the case. CONTRIBUTING.md doesn't state anything on it but does refer to the  nixpkgs-manual, and the manual states

Make commits of logical units.

I believe this is what I've done.

I always took it as atomic "package changes" that all correlated to a given change. This way, all changes can easily be reverted if needed.

I thought that may be the case too, but then I don't see why not just let the commits be while in the PR state and then "squash and merge"

In that specific case I think it was fine to just push changes until it works and use the squash button at the end. Having separate commits helped me with the review as well.

yep git rebase -i ftw. A better pr/diff viewer also helps a lot. I'm partial to reviewable.io myself since it doesn't try to take over the development model like gerrit does.


Error happens when I try to load data

As a <user type>, I want to <task> so that <goal> (make this the title)
Expected behavior
Actual behavior
Steps to reproduce the behavior
I used jupyter notebook with anaconda python 2.7 on my local PC to do a project with pixiedust. Here are the errors when I tried to load data frame:
!pip install pixiedust
import pixiedust
df = pixiedust.sampleData("https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD")
Downloading 'https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD' from https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD
Downloaded 214106 of 214106 bytes
Creating pySpark DataFrame for 'https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD'. Please wait...
Successfully created pySpark DataFrame for 'https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD'
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 df = pixiedust.sampleData("https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD")
/home/azhang/anaconda/lib/python2.7/site-packages/pixiedust/utils/environment.pyc in wrapper(*args, **kwargs)
83             kwargs.pop("fromScala")
84             fromScala = True
---> 85         retValue = func(*args, **kwargs)
86         if fromScala and retValue is not None:
87             from pixiedust.utils.javaBridge import JavaWrapper
/home/azhang/anaconda/lib/python2.7/site-packages/pixiedust/utils/sampleData.pyc in sampleData(dataId)
78 def sampleData(dataId=None):
79     global dataDefs
---> 80     return SampleData(dataDefs).sampleData(dataId)
81
82 class SampleData(object):
/home/azhang/anaconda/lib/python2.7/site-packages/pixiedust/utils/sampleData.pyc in sampleData(self, dataId)
91             return self.loadSparkDataFrameFromSampleData(dataDefs[str(dataId)])
92         elif "https://" in str(dataId) or "http://" in str(dataId) or "file://" in str(dataId):
---> 93             return self.loadSparkDataFrameFromUrl(str(dataId))
94         else:
95             print("Unknown sample data identifier. Please choose an id from the list below")
/home/azhang/anaconda/lib/python2.7/site-packages/pixiedust/utils/sampleData.pyc in loadSparkDataFrameFromUrl(self, dataUrl)
126             "url": dataUrl
127         }
--> 128         return Downloader(dataDef).download(self.dataLoader)
129
130
/home/azhang/anaconda/lib/python2.7/site-packages/pixiedust/utils/sampleData.pyc in download(self, dataLoader)
150             try:
151                 print("Creating pySpark DataFrame for '{0}'. Please wait...".format(displayName))
--> 152                 return dataLoader(path, self.dataDef.get("schema", None))
153             finally:
154                 print("Successfully created pySpark DataFrame for '{0}'".format(displayName))
/home/azhang/anaconda/lib/python2.7/site-packages/pixiedust/utils/sampleData.pyc in dataLoader(self, path, schema)
103     def dataLoader(self, path, schema=None):
104         #TODO: if in Spark 2.0 or higher, use new API to load CSV
--> 105         load = ShellAccess["sqlContext"].read.format('com.databricks.spark.csv')
106         if schema is not None:
107             def getType(t):
AttributeError: 'NoneType' object has no attribute 'read'

@andrewzhang1 I cannot reproduce. This works for me with Python 2.7.11 and Spark 2.0.2 with Pixiedust 1.0.6.
import pixiedust
sftest = pixiedust.sampleData("https://data.sfgov.org/api/views/vv57-2fgy/rows.csv?accessType=DOWNLOAD")

@rajrsingh I attended David Taieb's demo yesterday, and I did  not use IBM's Data science cloud environment. By instead, I use my own local jupyter on my linux. I showed this error to David, and David told me there're might be a bug there. I like to confirm: no spark needs to be installed (or connected) on my local machines in order use pixiedust.

By the way, I'm not the only one getting this error by using local jupyter notebook.

@andrewzhang1 Yes, this is a valid bug that happens when running PixieDust on a plain Python Notebook without Spark.

sampleData() has been updated in commit https://github.com/ibm-watson-data-lab/pixiedust/commit/90937efb6e18dab7b2f2746c4442cd1d26528511 (PR #404) to successful load data into a pandas dataframe when in a notebook w/o Spark


Added the ability to set the Image with a UIImage

This change allows a user to be able to set the image with a UIImage instead of a file. This gives the user for example the ability to composite multiple images and then use that UIImage as the Panoramic Image instead of having to save the composited image.

this is great! thank you.
this brings up a good point - your function should be the "default"
instead of:
setImage:(NSString)
setImageWithImage:(UIImage)
should be:
setImage:(UIImage)
setImageWithName:(NSString)

is this not available with swift?


Update to match specs from TechWG discussion (#83)

Related Issue: #83
Summary of Problem:
In our TechWG meeting where we reviewed Steamclog usage, we decided on some updates to how LogLevelPresets were named and what their LogLevel mappings were:

Tech WG Doc:
https://coda.io/d/Tech-Working-Group_dq8QHMnEi6A/Aug-16th-2021_suivT#_luAHl
Proposed Solution:

Renamed LogLevelPresets
Renamed LogLevelPreset destinations to match spec (ie. remote instead of sentry)
Updated LogLevelPreset LogLevels
Steamclog now defaults to LogLevelPreset Debug for debug builds (was previously Firehose)

Testing Steps:
These should mostly be internal changes, but the sample app should now show the updated levels:

We should make sure we release the next version as a major version upgrade, as I think all projects need to change their current verbose logs to debug in order to utilize the logging level changes

We should make sure we release the next version as a major version upgrade, as I think all projects need to change their current verbose logs to debug in order to utilize the logging level changes

Yeah, I agree. I believe the goal was to make the next release 2.0, and then have the swift version bump up to match shortly (since I believe right now it's 0.x)?


Gem failsafe error about connecting to DB

Details in https://app.clubhouse.io/rollbar/story/22078/gem-failsafe-error-about-connecting-to-db

Closing this issue as we are unable to reproduce it.

We've actually been seeing this intermittently in production as well. The primary issue is that the error has basically no information. I have no idea how to go about debugging this. What's a guy gotta do to get backtrace?
Failsafe from rollbar-gem. ActiveRecord::ConnectionTimeoutError: \"could not obtain a connection from the pool within 5.000 seconds (waited 5.000 seconds); all pooled connections were in use\" in /app/vendor/bundle/ruby/2.5.0/gems/activerecord-<IP_ADDRESS>/lib/active_record/connection_adapters/abstract/connection_pool.rb:201:in 'block in wait_poll': build_item in exception_data'

Thanks @sdhull .
@waltjones Any ideas on 1) what might cause this situation, and 2) how rollbar-gem can better handle it?

@sdhull I'll take a look to see if more diagnostic info can be provided in these failsafe conditions.
@brianr It looks like the original error is the connection pool is exhausted, with unknown root cause. It looks like rollbar-gem should provide more info on failsafe conditions regardless of what error got us there.

@waltjones 👍
The other thing I'm wondering is whether this situation can result in a more positive outcome than a failsafe. I assume the db connection attempt is happening when rollbar-gem is trying to get the person data, or maybe in "custom_data_fn" or something like that. If that happens, I'd think we can still report the original stack trace and the contextual data that we are able to get; we just won't have the person data/whatever failed to build.

@brianr Good idea.

Super impressed how responsive you guys are. Thanks! Looking forward to seeing this move

Usually by the time we get to a failsafe it is because we tried making a payload with some information and even that failed. If there is an exception in say the person function we should already just ignore it and try to move on with that blank.

@sdhull can you bundle with branch wj-failsafe? This isn't a final fix, but should allow you to get the report for the original exception. If the original non-rollbar exception is also ActiveRecord::ConnectionTimeoutError that would be interesting to know.


The Code with mmcv is hard to understand.

Hello, dear authors. Thank you for your contribution to the field of scene graph generation, I find your work very interesting, but when I was about to reproduce your article or wish to follow your work, I was very uncomfortable with the opaqueness of the mmcv framework and could not modify the code properly, so I would like to ask if you could provide a pure pytorch version of the code or provide us with some reference materials for learning mmcv, I would appreciate it.

Thanks for the question.
The PSG task contains important components from detection and segmentation, which is not easy to be implemented by pure pytorch. MMdet is one of the best detection framework, which would make the code easier. If you want to dive deep in PSG and dense prediction tasks, I believe you should be familiar with the framework.
You can use "debug" mode in vscode to explore the mmdet and mmcv frameworks.
Good luck!


javasrc2cpg: detect configFiles and create nodes

In order to detect e.g. hardcoded secrets in configFiles, java2cpg creates ConfigFile nodes, and javasrc2cpg should do the same eventually.
e.g.
https://github.com/ShiftLeftSecurity/HelloShiftLeft -> java2cpg -> cpg.configFile.size -> 162

This was added in https://github.com/joernio/joern/pull/1638. For the HelloShiftLeft project, javasrc2cpg only finds the 4 config files in the repo and does not find anything for the dependencies. We could look for those when fetching dependency jars, but that feels like unnecessary extra complexity (please correct me if I'm wrong here)


SMHE-729: cucumber test additions for test alarms

Signed-off-by: Andre Bolder<EMAIL_ADDRESS>Small change for adding a PlatformKey for test alarm type

https://jenkins.fdp.osgp.cloud/view/Nightly Build/job/gxf-nightly-build/483/


Change pep8 to pycodestyle

Due to some confusion, the pep8 package has been renamed to pycodestyle. https://pypi.org/project/pycodestyle/.
The change was requested here: https://github.com/PyCQA/pycodestyle/issues/466

This means that the recent changes made to pep8 were not captured, causing the test_pep8.py to fail


Only use dictionary update if the new value is a dictionary

We are missing the case where you want to replace a value
in cassandra.yaml that is normally a dict, with a single key.
For example, with just "None".
Fixes #434.

CI looks fine.


Use js.native members in NodeFS

org.scalajs.testsuite.compiler.SourceMapTest is failing in the bootstrap tests.

It seems that using a @js.native member protects the module from being captured as this:
-        $i_fs.close(fd$1, arg1$1);
+        (0, $i_fs.close)(fd$1, arg1$1);

Is this working as intended? (I'd assume that for ES modules this doesn't make a difference, for CommonJS it might).

In fact, the source map test fails, because the linking fails:
org.scalajs.linker.runtime.UndefinedBehaviorError: java.lang.ClassCastException: undefined is not an instance of java.lang.Integer
  at $throwClassCastException(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:26:9)
  at $uI(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:369:51)
  at org.scalajs.linker.NodeOutputFile$NodeOutputChannel.write(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:52183:7)
  at org.scalajs.linker.interface.unstable.OutputFileImpl.writeLoop$1(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:56637:112)
  at {anonymous}()(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:56646:169)
  at scala.scalajs.runtime.AnonFunction1.apply(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:64214:45)
  at scala.concurrent.impl.Promise$Transformation.run(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:101552:27)
  at scala.scalajs.concurrent.QueueExecutionContext$PromisesExecutionContext.scala$scalajs$concurrent$QueueExecutionContext$PromisesExecutionContext$$$anonfun$execute$2(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:70332:18)
  at {anonymous}()(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:70324:24)
Caused by: java.lang.ClassCastException: undefined is not an instance of java.lang.Integer
  at $throwClassCastException(/localhome/jenkins/sjsjenkins/workspace/scalajs-main_PR-4068@3/test-suite-linker/.2.13/target/scala-2.13/scalajs-test-suite-linker-fastopt.js:26:67)
  ... 8 more

I suspect this simply throws before the source map is written.

Ah yes, that is working as intended, since all our @JSImports are specified as if they were JavaScript imports, and if we use actual modules in JS:
// foo.mjs
export function foo() {
  return this;
}

// main.mjs
import { foo } from "./foo.mjs"

console.log(foo());

$ node main.mjs
undefined

we can see that this is not bound for such a method call.

org.scalajs.linker.runtime.UndefinedBehaviorError: java.lang.ClassCastException: undefined is not an instance of java.lang.Integer
  at $throwClassCastException(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/scalajs-test-suite-linker-fastopt.js:26:9)
  at $uI(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/scalajs-test-suite-linker-fastopt.js:369:51)
  at org.scalajs.linker.NodeOutputFile$NodeOutputChannel.write(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/file:.../scala-js/linker/js/src/main/scala/org/scalajs/linker/NodeOutputFile.scala:44:37)
  at org.scalajs.linker.interface.unstable.OutputFileImpl.writeLoop$1(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/file:.../scala-js/linker-interface/shared/src/main/scala/org/scalajs/linker/interface/unstable/OutputFileImpl.scala:29:43)
  at {anonymous}()(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/file:.../scala-js/linker-interface/shared/src/main/scala/org/scalajs/linker/interface/unstable/OutputFileImpl.scala:33:28)
  at scala.scalajs.runtime.AnonFunction1.apply(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/file:.../scala-js/library/src/main/scala/scala/scalajs/runtime/AnonFunctions.scala:27:38)
  at {anonymous}()(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/https:/raw.githubusercontent.com/scala/scala/v2.12.11/src/library/scala/concurrent/Future.scala:307:25)
  at scala.scalajs.runtime.AnonFunction1.apply(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/file:.../scala-js/library/src/main/scala/scala/scalajs/runtime/AnonFunctions.scala:27:38)
  at {anonymous}()(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/https:/raw.githubusercontent.com/scala/scala/v2.12.11/src/library/scala/concurrent/impl/Promise.scala:41:17)
  at scala.scalajs.runtime.AnonFunction1.apply(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/file:.../scala-js/library/src/main/scala/scala/scalajs/runtime/AnonFunctions.scala:27:38)
Caused by: java.lang.ClassCastException: undefined is not an instance of java.lang.Integer
  at $throwClassCastException(.../scala-js/test-suite-linker/.2.12/target/scala-2.12/scalajs-test-suite-linker-fastopt.js:26:67)
  ... 9 more

The problem seems to be this diff:
-      $i_fs.write(this.Lorg_scalajs_linker_NodeOutputFile$NodeOutputChannel__f_fd, ta, 0, $uI(ta.length), (void 0), arg1$1);
+      $uI((0, $i_fs.write)(this.Lorg_scalajs_linker_NodeOutputFile$NodeOutputChannel__f_fd, ta, 0, $uI(ta.length), (void 0), arg1$1));

The facade for write has a wrong Int return type. It seems when calling it on a native module, no unbox is inserted. Calling a native member inserts that unbox. The unbox fails (correctly).

There was a mima problem.


Added tzdata Alpine package

Adding support for TZ environment variable in alpine as per #124

@bluemalkin Thanks for the contribution - merging it!


[gettext-libintl] Link CoreFoundation on apple

Cherry-picked from https://github.com/microsoft/vcpkg/pull/38658.

Side note:
The single list item "-framework CoreFoundation" avoids absolute SDK paths while building in CMake, and is used elsewhere in vcpkg. But the embedded space as linker option separator isn't entirely accurate and may cause problems when link libs are interpreted by other programs.
TIL https://cmake.org/cmake/help/latest/command/find_library.html:

If the library found is a framework, then <VAR> will be set to the full path to the framework <fullPath>/A.framework. When a full path to a framework is used as a library, CMake will use a -framework A, and a -F<fullPath> to link the framework to the target.

Until the CMake behavior is better covered in vcpkg's check and fixup of absolute paths, I will stick to the established practice.

Thanks!


Improve download server selection algorithm

When dowloading a large file, the backend periodically requests new download servers from mailru dispatcher, e.g. when a chunk download fails. The current implementation in the serverList.take procedure (https://github.com/ivandeex/rclone/blob/master/backend/mailru/mailru.go#L2037) may result in sticking with an overloaded server. This issue is to improve this situation and make the the backend leave a failing server earlier. In particular, we should request a new download server whenever

need new server (the current chunk is done)
the old server lock is is expired (even if the server is till in the active list)

Fixed by commit c4bf7a46d4b3d839b1806f2a08c8535f72b28855


Add test cases for manual command

Added test cases for manual command, handling all success cases and one failing test case.

Codecov Report

Merging #112 (bf8a7c3) into master (b262d70) will increase coverage by 1.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #112      +/-   ##
============================================
+ Coverage     50.44%   51.48%   +1.04%
- Complexity      414      431      +17
============================================
  Files            96       96
  Lines          1919     1919
  Branches        229      229
============================================
+ Hits            968      988      +20
+ Misses          864      844      -20
  Partials         87       87

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/ManualCommand.java
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Avoid non-image files.

Is your feature request related to a problem? Please describe.
When there are non-picture files in the folder, it will raise <PIL.UnidentifiedImageError>.

Describe the solution you'd like
The filter_images_in_image_folder_path() function on line 64 of the gallery.py code seems to be unable to avoid non-picture files, so that an exception is thrown.
Additional context
This project is very useful. The built-in image viewing program is somewhat inefficient, and I hope your project can continue to be optimized.

This should be avoided since the filtered images are only added if they are image files. And as you see it’s a png. But I can add a try catch for this issue to handle broken image files

Hi. Thanks for your reply.
I forgot to mention earlier that I use Mac OS, and this type of file(dot underscore ._ files) will only appear on that system.
The is_valid_image_extension() function on line 890 of the gallery.py code just judge the suffix of the file.
Since this kind of file is not a image file, PIL will throw an exception after reading it.
Personally, I patched this code here to avoid the problem. If possible, maybe you can improve it a little in the next update, although there are very few people who use Mac to play stable diffusion.😭😭😭
line 890 of gallery.py:
def is_valid_image_extension(filename):
    ...
    if sys.platform == 'darwin' and filename.startswith('._'):
        return False
    return filename.endswith(".jpg") or filename.endswith(".jpeg") or filename.endswith(".png")


Intent creation with multiple training phrases having same parameter, creates the mentioned parameter as list

I'm creating an intent with two different training phrases containing the same parameter.
As follows:
TrainingPhrases=[
  {
     parts: [
        {
          text: "ciao mi chiamo"
        },
        {
          alias:"nome"
<EMAIL_ADDRESS>          isList:false
          text:"Marco"
          userDefined:true
        }]
  },
  {
     parts:[
        {
           text: "Buongiorno sono"
        },
        {
           alias:"nome"
<EMAIL_ADDRESS>           isList:false
           text:"Silvia"
           userDefined:true
        }
     ]
   }
]

Besides uncorrect JSON notation that might be here, as you can see "Silvia" and "Marco" are always referring to the same parameter that's "nome".
Once I load everything, the parameter is created as a list:

Why?

A Part doesn't know about isList
https://cloud.google.com/dialogflow/docs/reference/rpc/google.cloud.dialogflow.v2#google.cloud.dialogflow.v2.Intent.TrainingPhrase.Part
Only the Parameter does.
When you create the parameter, you should mark isList: false then.
https://cloud.google.com/dialogflow/docs/reference/rpc/google.cloud.dialogflow.v2#parameter


Bug on Windows 10 (build 1809)

Windows 10 (build 1809) doesn't recognize tls12 or tls13 as specified in your module
[String]$AllProtocols = 'tls12,tls13'	## Https encryption
What worked for me is replacing this line with:
[String]$AllProtocols = 'Tls12'	## Https encryption

If you change that given line 81 mentioned above to the one below, you have the best of both worlds, uses TLS 1.3 if possible, but uses TLS 1.2 if the client doesn't support anything newer:
[String]$AllProtocols = if (([Net.SecurityProtocolType]).DeclaredMembers.Name -contains "Tls13") { 'tls12,tls13' } else { 'tls12' }	## Https encryption - minimum Tls 1.2 needed

and if I would test it before I post an answer, I would have spotted that it needed to be put inside an @( ) like this:
[String]$AllProtocols = @(if (([Net.SecurityProtocolType]).DeclaredMembers.Name -contains "Tls13") { 'tls12,tls13' } else { 'tls12' })	## Https encryption - minimum Tls 1.2 needed

Modified as suggested by Robby-Swartenbroekx in version 1.5.


Proposal: Add typings / linter / static checker

As soon as more and more developers is going to contribute to the project it will be nice to have a way to make some issues more obvious. That is why I propose to add typings / linter / static checker to the project. Before working on that I want to discuss does it makes sense or not.

i don't think it's neccessary at this point


Document DynamicNodeEncoding and attributed intrinsic

These features are now available in master, but should be documented in README or separate doc files before 0.4 release is tagged.

This was implemented in #85


css minify script issue with removing empty {}

In the CSSMin php
// ------------------------------------------------------------------------
// Remove empty blocks.
// ------------------------------------------------------------------------
// $_source = preg_replace('/}[^/}]+{}/', '}', $_source);
I had to comment out because I like to have some media query in some css empty while developing. The removing of the empty brackets lead to unexpected results with "open" media queries.

This is related to our intension to switch to a third-party CSS compressor (#34). I think, the problem will be solved automatically.

I just tested the two cases which somatonic reported and the new YUI CSS compressor handles those correctly. The first case results in
.text{color:#fff}

The second case results in no output as the resulting style is empty.

Soma, can you please verify if the current version of AIOM (3.2) fixes this issue?

Sorry - I read over the answer of @hwmaier. In this case, I think this issue is solved. @somatonic: If the problem still occurs, feel free to reopen this issue.


header button

Hello,
is the header list (h1...h6) already implemented in ngx-wig?
How can i add heading to the text.
Thanks

@veteran1 this feature is not currently baked in the library. However, you could create a plugin for this. Check out this guide how to do it 😃

ok, i will take look,
thanks for your feedback

@veteran1 , any update on this?

@arcadius, no I did not create the plugin since it was not a mandatory requirement for me.


Cap flake8 bugbear version

What does this PR do?

Cap bugbear version due to https://github.com/PyCQA/flake8-bugbear/issues/108
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Issue fixed, https://github.com/PyCQA/flake8-bugbear/pull/109
Closing this PR.


hide peerpods and peerpodconfigs internal objects

based on:
https://docs.openshift.com/container-platform/4.12/operators/operator_sdk/osdk-generating-csvs.html#osdk-hiding-internal-objects_osdk-generating-csvs

cc @bpradipt @jensfr @pmores for reviewing

premerge - I don't see the PeerPod APIs.  They're invisible as they should be.

/lgtm

pre-merge success!
/lgtm
/approve


Edit url fails to fetch data to be updated

TypeError: Cannot destructure property 'post' of '(intermediate value)' as it is undefined.
Figured out the bug but still needs fixing


Ensembles

Feature request: Make it possible to train ensembles of density estimators instead of a single density estimator.
There are two ways of implementing this, either naively instantiating multiple density estimators that form the ensemble, or by adding batch dimensions to weight matrices are training a single big density estimator. The latter can be significantly faster but requires flexibility wrt to weight matrices in the density estimator implementation used (e.g. nflows).
References:

https://arxiv.org/abs/1612.01474
https://arxiv.org/abs/1805.09294

Hi! Actually, this is we have also been doing with @JoeriHermans for SNRE/AALR. Using an ensemble of ratio estimators results in smoother posterior and can also be quite informative as a diagnostic (high variance often suggests a poor fit and therefore an unreliable posterior).

Hi, thank you for pointing this out!
Since ensembling can be so useful (improves robustness/accuracy, for active learning, and also diagnostics as you mention), I've been thinking some more about a minimal implementation.
Using joblib, it's very simple to run any inference algorithm that is part of sbi in parallel (assuming enough memory/compute). For example:
from sbi.examples.minimal import simple
from joblib import Parallel, delayed

ensemble_size = 10
posteriors = Parallel(n_jobs=-1)(delayed(simple)() for i in range(ensemble_size))  # Returns list of NeuralPosteriors

In order to sample and score the ensemble posterior, it might be useful to introduce a new class, i.e., a NeuralPosteriorMixture:
import math
import torch
from typing import List
from torch import Tensor
from sbi.types import Shape

class NeuralPosteriorMixture:
    def __init__(self, posteriors: List):
        self.posteriors = posteriors
        self.num_components = len(posteriors)
        self.weights = torch.tensor(
            [1.0 / self.num_components for _ in range(self.num_components)]
        )

    def sample(self, sample_shape: Shape = torch.Size(), **kwargs):
        num_samples = torch.Size(sample_shape).numel()
        idxs = torch.multinomial(self.weights, num_samples, replacement=True)
        samples = []
        for c in range(self.num_components):
            sample_shape_c = (
                torch.Size((len(idxs == c),)) + torch.Size(sample_shape)[1:]
            )
            samples.append(self.posteriors[c].sample(sample_shape_c, **kwargs))
        return torch.cat(samples)[idxs]

    def log_prob(self, theta: Tensor, **kwargs):
        return torch.logsumexp(
            torch.stack(
                [
                    math.log(self.weights[c])
                    + self.posteriors[c].log_prob(theta, **kwargs)
                    for c in range(self.num_components)
                ]
            ),
            dim=0,
        )

    def set_default_x(self, x: Tensor):
        for c in range(self.num_components):
            self.posteriors[c].set_default_x(x)

The code above should already work and with additional tests and documentation, could turn into a first version of ensembles for sbi. As outlined in the top comment, more efficient implementations are certainly possible -- this one is just the lowest effort one in terms of code I could think of.

@jan-matthis, @janfb Will assign myself to this. I will stick to the simple implementation suggested by @jan-matthis, as there is no point doing batched weights I guess, due to the impending deprecation of nflows in sbi.

I think this issue requires quite some discussion on how to best implement this first

Alright I suggest we talk about this tmr? :)

Yup perfect!

@michaeldeistler Implemented a first working version of what we talked about this morning and made a PR (#612).


This commit supports spdm-requester-emu communicate with other endpoints

via MCTP in kernel. It also supports sending single command GET_VERSION to test the interface.
Tested:
Send command:
spdm_requester_emu --trans MCTP_KERNEL --eid 20 --cmd GET_VERSION
Sample output:
spdm_requester_emu version 0.1
trans - 0x5
spdm_cmd - 0x01
connect success!
context_size - 0x21d8
libspdm_send_spdm_request[0] msg SPDM_GET_VERSION(0x84), size (0x4):
0000: 10 84 00 00
Platform port Transmit transport_type: 00 00 00 05
Platform port Transmit size: 00 00 00 05
Platform port Transmit buffer:
05 10 84 00 00
Platform port Receive buffer:
05 10 04 00 00 00 03 00 10 00 11 00 12
libspdm_receive_spdm_response[0] msg SPDM_VERSION(0x4), size (0xc):
0000: 10 04 00 00 00 03 00 10 00 11 00 12
Client stopped

@NguyenTanNhutQuang , please fix CI failure.
Please note that spdm-emu need to be built in both Linux and Windows OS environment.
I feel you need to have a way (maybe MACO) to exclude Linux kernel code if it is Windows/Visual Studio.

@NguyenTanNhutQuang , please fix CI failure.
Please note that spdm-emu need to be built in both Linux and Windows OS environment. I feel you need to have a way (maybe MACO) to exclude Linux kernel code if it is Windows/Visual Studio.

Hi jyao1,
Is there anyway that I can check CI on my local server before push to github? Thank you

@steven-bellock , do you have any idea?


Avoid showing the groupThreads on the graphs

The metrics related to a groupThread is not shown any more on the graphs. This is fixed on the frontend side, by filtering the labels of curves to show on each chart. The metrics related to a Sample are prefixed by "spl_" and those related to threads are suffixed by "threads". This means that we don't tolerate to show the labels that don't start with "spl" and don't finish with "_thread".

There's the new Screen shots.
The image bellow show the test plan configuration

And there's an example of a graph


Manual changes to make DatastoreClient use ChannelPool

LGTM
I like the new simpler Create() methods.
Although I'm wondering if we should switch round the parameters, i.e. have DatastoreSettings as the first param, and Endpoint/Channel as the second; as I expect the settings to be used far more than the endpoint/channel.

Well for Create(Channel, Settings), the settings have to be second as Channel is required.
For Create(ServiceEndpoint, Settings) I'm not sure... it depends if people connect to emulators, test environments etc.
Possibly something to bring up with other teams.


Add rule newline-per-chained-call

Summary
Include a rule to break lines when functions (CallExpression) have more than one chained call.
Why do we need a custom rule?
The ESLint newline-per-chained-call is not so flexible to have some behaviors that we thought is necessary.
Examples:

When the default value of ignoreChainWithDepth is surpassed only the following chained calls would break the line:
❌  ESLint newline-per-chained-call
// Entry code
foo().bar().baz().quz()

// Output code
foo().bar().baz()
    .quz()

✅ Croct newline-per-chained-call
foo().bar().baz().quz()

// Output code
foo()
    .bar()
    .baz()
    .quz()

When the chained calls were only MemberExpression:
❌  ESLint newline-per-chained-call
// Input code
'&:focus-visible': {
    borderColor: ({colors}): string => colors.ui.style.shadow.ring.emphasized,
    boxShadow: ring('emphasized', false, '1px'),
},

// Output code
'&:focus-visible': {
    borderColor: ({colors}): string => colors
        .ui
        .style
        .shadow
        .ring
        .emphasized,
    boxShadow: ring('emphasized', false, '1px'),
},

✅ Croct newline-per-chained-call
// Input code
'&:focus-visible': {
    borderColor: ({colors}): string => colors.ui.style.shadow.ring.emphasized,
    boxShadow: ring('emphasized', false, '1px'),
},

// Output code
'&:focus-visible': {
    borderColor: ({colors}): string => colors.ui.style.shadow.ring.emphasized,
    boxShadow: ring('emphasized', false, '1px'),
},

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Which scenario do the existing rule do not support that required us to implement a custom one?

Which scenario do the existing rule do not support that required us to implement a custom one?

Sorry my bad, I totally forgot to update the description of the PR.
I've added some examples on why we created a custom rule.

If we decide to adopt the \n, please fix the other rule's style in another PR as it's unrelated to this.


typo for array index in https://github.com/KhronosGroup/OpenCL-CTS/blob/main/test_conformance/math_brute_force/unary_double.cpp#L292,L293

https://github.com/KhronosGroup/OpenCL-CTS/blob/main/test_conformance/math_brute_force/unary_double.cpp#L292,L293
input is
cl_double *p = (cl_double *)gIn + thread_id * buffer_elements;
instead of (cl_double *)gIn)[j],
output ref is
cl_double *r = (cl_double *)gOut_Ref + thread_id * buffer_elements;
instead of gOut_Ref.
Can you please double check? Thanks very much.
+@svenvh, @bashbaug

I think the current expressions are wrong indeed, thanks for reporting it!  Ideally we'd align with unary_float.cpp and use s[j] and t[j].
Are you happy to open a PR to fix it?

I think the current expressions are wrong indeed, thanks for reporting it! Ideally we'd align with unary_float.cpp and use s[j] and t[j].
Are you happy to open a PR to fix it?

Sure! Thanks for quickly reply.

Looking through the other tests, I think test_conformance/math_brute_force/macro_unary_double.cpp may have a similar issue (it's a multi-threaded test that directly indexes into gIn without taking the thread ID into account).

All issues have been fixed by #1995 now.


Update ethers-v6 target to support Result methods

fixes #817

Thanks! Merged as : https://github.com/dethcrypto/TypeChain/pull/866

Sorry, no this is not merged yet.

@krzkaczor please let me know anything you need on this before it can be merged? Thank you.

@shirotech can you rebase with current master and run test:fix and fix all the errors. Please follow contributing guide when in doubt.

Are there any updates here?


feat(compute): Disk snapshot samples

I'm still not a big fan of how this sample uses t.Run(), but I won't stop you from merging.

What is a better way to use t.Run()?

I am basically following the testing pattern already laid out, but if there's a better way I'd be happy to use it for future PRs, and perhaps convert the existing tests to it later if it provides tangible benefits.


docs incorrectly claim no support for top-level "networks:" in docker-compose.yml

Context in FD thread: https://www.flowdock.com/app/rulemotion/r-supervisor/threads/4RnQtHlrcrx2g4zJvJ6KwkwX8h_
You can (despite docs saying not being supported) define a networks: stanza in your compose file at the top-level doing more than just name changing. You can change the actual subnet.
If your docker-compose.yml has:
networks:
  default:
    ipam:
      config:
        - subnet: <IP_ADDRESS>/24
          gateway: <IP_ADDRESS>

, then you will see:
root@21edf97:~# ip r s
default via <IP_ADDRESS> dev enp0s3  metric 100
<IP_ADDRESS>/24 dev enp0s3 scope link  src <IP_ADDRESS>  metric 100
<IP_ADDRESS>/24 dev balena0 scope link  src <IP_ADDRESS>
<IP_ADDRESS>/24 dev resin-dns scope link  src <IP_ADDRESS>
<IP_ADDRESS>/25 dev supervisor0 scope link  src <IP_ADDRESS>
<IP_ADDRESS> dev resin-vpn scope link  src <IP_ADDRESS>
<IP_ADDRESS>/24 dev br-254e87c07e29 scope link  src <IP_ADDRESS>

(the default network has successfully been redefined away from the default 172.17.x.x to 192.20.x.x).

Related to #1172


Add 2 new optional configuration to slack credentials:  slack_retr…

…y_reason_header and slack_retry_number_header

Change 2 default header value to : "x-slack-retry-reason" and "x-slack-retry-num"

Proposed changes:

...

Status (please check what you already did):

[ ] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the PR, we'll give it a review as soon as possible

@cuongnda Thanks for the PR. Can you please shortly explain, why we should change the default headers?
Apart from that the PR looks good. You just need to format the code.

Hi, the reason we need to change the default header is that it is currently different to the header that slack used. Please check this following thread: https://github.com/RasaHQ/rasa/issues/3882

This doesn't seem to work for me, is this a correct syntax:
  errors_ignore_retry: |
    - "http_timeout"


Suport "rename" annotate directive in schema

Add "rename" feature to the annotate directive in GraphQL type.
MarketData2(helloWorld=md1.hello_world)

type MarketData1 {
   hello-world: String! @annotate(rename="helloWorld", exclude=true)
}

type MarketData2 {
  helloWorld: String!
}

md1: MarketData1
md2 = md1.transform()

I'd expand this in the ability to add schema transformations:
Here are some common transformations:

rename field while transforming

field/type aliases for the types that can't be represented in programming langs
rename the field to the desired name

exclude selected fields
only include selected fields
transform array of pair into a map and vice-versa
Can in future support many such transformations if needed to


chore(deps): package-lock maintainance

Updates dependencies in package-lock.json, first commit is actually #4620. This will be rebased once it's merged.

Whoa, a 41k line change to package-lock?

Ah, something must've gone wrong - I actually just ran npm update.
I'll look into it once we're done with #4620

I see that in #4601 part of the change to package-lock included this:
diff --git a/package-lock.json b/package-lock.json
index 6cc71665..503a0f15 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -10,7 +10,6 @@
       "hasInstallScript": true,
       "license": "Apache-2.0",
       "workspaces": [
-        "api",
         "packages/*",
         "experimental/packages/*",
         "experimental/examples/*",

I don't know that that is problematic, but it is weird.

I don't know that that is problematic, but it is weird.

I think I know why this happens: It must be the prepare release script, we remove the API there so that lerna can do it's thing. We'll likely need to sync package-lock again after we do that.

I don't know that that is problematic, but it is weird.

I think I know why this happens: It must be the prepare release script, we remove the API there so that lerna can do it's thing. We'll likely need to sync package-lock again after we do that.

#4623 should take care of this.

I actually just ran npm update.

I tried that too, and my diff is 72k! :)
% git diff | wc -l
   72979

I don't really know what to say. I think the huge diff is from shuffling around where deps are installed. E.g. in my 72k line diff, all the mocha@10 deps were shuffled from the top-level node_modules/ to the separate .../node_modules/ dirs for each package... and the one usage of mocha@9 moved to the top-level.
I guess we just take it and move on.

I actually just ran npm update.

I tried that too, and my diff is 72k! :)

New highscore :D

% git diff | wc -l
   72979

I don't really know what to say. I think the huge diff is from shuffling around where deps are installed. E.g. in my 72k line diff, all the mocha@10 deps were shuffled from the top-level node_modules/ to the separate .../node_modules/ dirs for each package... and the one usage of mocha@9 moved to the top-level.
I guess we just take it and move on.

I'll try to enable packge-lock maintainance on renovate bot. Maybe it can manage to produce a diff that's a bit less daunting to look at. :thinking:

PR to enable lock file maintenance, let's see what it does: #4628 :slightly_smiling_face:

Closing this for now as I think going the renovate route will be better.


Auto-sync to iCal feed?

Hi,
Any chance of integrating this capability (http://www.redmine.org/plugins/redmine_add_to_calendar) to have todo's with a deadline added to a calendar?  This way any calendaring app can show the todo.

It's unlikely that I'll implement it but I would love to accept a pull request :)

Happily waiting for pull requests...


Integrating CIFTI templates into FMRIPREP installation (without online download)

Hi there,
I am using FMRIPREP under a HPC environment with no Internet connection, and processing failed during CIFTI generating stage because of the restriction of connection, as attached below. I am just wondering if it is possible to integrate CIFTI templates (and other possibly required templates) into FMRIPREP installation (and container) so it can be run without Internet completely. Thank you!
================================================================
Node: fmriprep_wf.single_subject_C207_wf.func_preproc_ses_Baseline_task_nback02_run_01_wf.bold_grayords_wf.gen_cifti
Working directory: /project/oathes_group/fmriprep/temp_fmriprep/fmriprep_wf/single_subject_C207_wf/func_preproc_ses_Baseline_task_nback02_run_01_wf/bold_grayords_wf/gen_cifti

Node inputs:

TR = 1.0
bold_file = <undefined>
subjects_dir = <undefined>
surface_bolds = <undefined>
surface_density = 32k
surface_target = fsLR
volume_target = MNI152NLin6Asym

Traceback (most recent call last):
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connection.py", line 159, in _new_conn
    conn = connection.create_connection(
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/util/connection.py", line 84, in create_connection
    raise err
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/util/connection.py", line 74, in create_connection
    sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 670, in urlopen
    httplib_response = self._make_request(
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 381, in _make_request
    self._validate_conn(conn)
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 978, in _validate_conn
    conn.connect()
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connection.py", line 309, in connect
    conn = self._new_conn()
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connection.py", line 171, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x2b7b826e71f0>: Failed to establish a new connection: [Errno 110] Connection timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/appl/python-3.9/lib/python3.9/site-packages/requests/adapters.py", line 439, in send
    resp = conn.urlopen(
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/connectionpool.py", line 726, in urlopen
    retries = retries.increment(
  File "/appl/python-3.9/lib/python3.9/site-packages/urllib3/util/retry.py", line 446, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='templateflow.s3.amazonaws.com', port=443): Max retries exceeded with url: /tpl-fsLR/tpl-fsLR_hemi-L_den-32k_desc-nomedialwall_dparc.label.gii (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x2b7b826e71f0>: Failed to establish a new connection: [Errno 110] Connection timed out'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/appl/python-3.9/lib/python3.9/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/appl/python-3.9/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
    result = self._run_interface(execute=True)
  File "/appl/python-3.9/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
    return self._run_command(execute)
  File "/appl/python-3.9/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
    result = self._interface.run(cwd=outdir)
  File "/appl/python-3.9/lib/python3.9/site-packages/nipype/interfaces/base/core.py", line 428, in run
    runtime = self._run_interface(runtime)
  File "/appl/python-3.9/lib/python3.9/site-packages/niworkflows/interfaces/cifti.py", line 104, in _run_interface
    annotation_files, label_file = _get_cifti_data(
  File "/appl/python-3.9/lib/python3.9/site-packages/niworkflows/interfaces/cifti.py", line 229, in _get_cifti_data
    for f in tf.get(
  File "/appl/python-3.9/lib/python3.9/site-packages/templateflow/api.py", line 78, in get
    _s3_get(filepath)
  File "/appl/python-3.9/lib/python3.9/site-packages/templateflow/api.py", line 220, in _s3_get
    r = requests.get(url, stream=True)
  File "/appl/python-3.9/lib/python3.9/site-packages/requests/api.py", line 76, in get
    return request('get', url, params=params, **kwargs)
  File "/appl/python-3.9/lib/python3.9/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/appl/python-3.9/lib/python3.9/site-packages/requests/sessions.py", line 530, in request
    resp = self.send(prep, **send_kwargs)
  File "/appl/python-3.9/lib/python3.9/site-packages/requests/sessions.py", line 643, in send
    r = adapter.send(request, **kwargs)
  File "/appl/python-3.9/lib/python3.9/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='templateflow.s3.amazonaws.com', port=443): Max retries exceeded with url: /tpl-fsLR/tpl-fsLR_hemi-L_den-32k_desc-nomedialwall_dparc.label.gii (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x2b7b826e71f0>: Failed to establish a new connection: [Errno 110] Connection timed out'))

The tricky thing about this is depending on what command line options are used, different files might be required. I do think it's reasonable to ensure all the CIFTI files (at least for 91k grayordinates) are present.
However, see https://github.com/nipreps/fmriprep/issues/2521#issuecomment-907587408 for a workaround.

Thank you so much for the help!


Interface for using te::concept_map for static dispatch

Have you considered providing an interface that'd allow using the concept maps of this library for static dispatch, basically to provide a compile-time mechanism for customization points? Shouldn't be too hard to implement, judging by looking through the implementation.
I'm mainly interested in this because I've been working on a customization points thingy myself, although with slightly less Hana and slightly more Boost.PP (which is currently mostly necessary for my vision of the interface due to how we don't have reflection yet), and I'd love to be able to compare bananas to bananas.
The main difference between what I did (which I can hopefully get online soon) and what you did that I've been focused more on providing a customization point mechanism that sucks somewhat less than what's been available to date, and you've focused on providing runtime behavior; this is in essence very similar and requires a lot of common features.
Just few comments/questions not exactly related to the issue, I hope you don't mind putting them here:

Are you going to present a talk about this at C++Now? (I've proposed a talk that's supposed to talk about my approach, so hopefully we can get some discussion about both the approaches if we both end up giving talks at the conference.)
I believe (but I'm biased, so what do I know? :D) my end user interface is nicer than yours, unless until you hit templates, where the macro'd approach starts to get not very customizable at one point. More on point though, did you consider how te might look like when reflection becomes available? Mine should be mostly 1:1 identical, I wonder if yours is as easily translatable at the end user interface level.
Have you thought about solving the problem of having to exit all of your namespaces to specialize the variable template in namespace te from some levels of end-user namespaces?

Have you considered providing an interface that'd allow using the concept maps of this library for static dispatch, basically to provide a compile-time mechanism for customization points?

Yes, I have, but I have been focusing on the application to type erasure exclusively so far. Technically speaking, we could even generate "archetypes" of some sort using these concept definitions, which would enable some sort of separate checking. But that would require writing all your code using compile-time strings to refer to methods, which is messy.
Without going so far down that road, however, it would be possible to ask whether a type models a concept, and then basically implement a library version of concepts lite on top of that. I fear that using this at any kind of non-trivial scale would be prohibitively slow at compile-time, though.

Just few comments/questions not exactly related to the issue, I hope you don't mind putting them here:

I'm planning on doing some refactoring of concepts and concept maps, which should make it easier to implement. But, at least for now, the focus is on type erasure.

Are you going to present a talk about this at C++Now?

No, not specifically. I plan to use the type erasure library as an example of using Hana in a non-trivial way, but I won't talk in depth about the library.

I believe (but I'm biased, so what do I know? :D) my end user interface is nicer than yours, unless until you hit templates, where the macro'd approach starts to get not very customizable at one point.

I would love to see your interface.

More on point though, did you consider how te might look like when reflection becomes available?

When full reflection is available, what we could do is use structs to define runtime concepts. The method signatures and names would be used in place of the DSL I'm using right now. Also, we would then most likely be able to generate the wrapper on top of te::poly automatically, or with very little user input. Finally, we might be able to generate some default concept maps automatically, but that's not clear to me yet. Overall, that would be quite nice, but it would solely be a "wrapper" on top of the existing logic.

Have you thought about solving the problem of having to exit all of your namespaces to specialize the variable template in namespace te from some levels of end-user namespaces?

No, and that's a real pain. The only solution I can think of is to use ADL in some way, since that's the only piece of C++ I know of that can lookup across namespaces like it does, but then you'd need to define your concept map in a function and place that function in an associated namespace of the type you're defining the concept map for. Something like:
namespace lib {
  struct SomeConcept : ... { };
}

namespace my {
  struct Foo { };
  constexpr auto concept_map(lib::SomeConcept, Foo) {
    return te::make_concept_map<lib::SomeConcept, Foo>(
      "f"_s = []() { ... }
    );
  }
}

// Then, in `te`, we create the vtable with something like this, which picks
// up the right `concept_map` because of ADL:
auto vtable = te::vtable<lib::SomeConcept>{concept_map(lib::SomeConcept{}, my::Foo{})};

This solution doesn't work because we need to construct Foo{}, etc..., but the idea can maybe be tweaked. I'm not sure how much I like the idea of defining a function that returns the concept map.

You can see my WIP here, though it doesn't yet compile with clang <5 due to some obscure injected class name bug that Richard Smith fixed on trunk recently: reaver-project/reaverlib#43.

Thanks for sharing. I personally don't like your approach better, as I find the macros make it harder to see what's going on. More magic, basically. But hey, I guess we're both the worst persons to judge on that since we're biased.
In any case, deriving from the type class definition is not an option in my case, because I want to get rid of inheritance and classic vtables.

There's inheritance and classic vtables in the current implementation of erased, but : public typeclass doesn't involve vtables - it's just to inherit the implementation of the structure I'm using to avoid ADL and other such tricks. It's like derives in haskell.
I'd say both our involve magic, and my point of view is that the end user needs to understand just the interface, not its implementation.

There's inheritance and classic vtables in the current implementation of erased, but : public typeclass doesn't involve vtables - it's just to inherit the implementation of the structure I'm using to avoid ADL and other such tricks.

Ah, sorry, I did not look at the implementation closely enough.

I'd say both our involve magic, and my point of view is that the end user needs to understand just the interface, not its implementation.

Sure.

I'm going to close this now. We can still talk here without problem, but I just want to close the issue.


fix(parser): should parser error when function declaration has no name

#3461  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite


improv: override Tracer auto-capture response/exception via env vars

Issue #, if available: Feature parity with Java powertools PR https://github.com/awslabs/aws-lambda-powertools-java/pull/249
Description of changes:

Adds env var support to override default behaviour in Tracer of auto-capturing both response and error:

Environment variable
Description

POWERTOOLS_TRACER_CAPTURE_RESPONSE
disable auto-capture response as metadata (e.g. "true", "True", "TRUE")

POWERTOOLS_TRACER_CAPTURE_ERROR
disable auto-capture error as metadata (e.g. "true", "True", "TRUE")

Checklist

[ ] Meet tenets criteria
[x] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #259 (8ab3507) into develop (1131541) will decrease coverage by 0.07%.
The diff coverage is 87.50%.

@@             Coverage Diff             @@
##           develop     #259      +/-   ##
===========================================
- Coverage    99.81%   99.74%   -0.08%
===========================================
  Files           76       76
  Lines         2763     2749      -14
  Branches       113      114       +1
===========================================
- Hits          2758     2742      -16
- Misses           4        5       +1
- Partials         1        2       +1

Impacted Files
Coverage Δ

aws_lambda_powertools/tracing/tracer.py
98.83% <87.50%> (-1.17%)
:arrow_down:

aws_lambda_powertools/logging/logger.py
100.00% <0.00%> (ø)

aws_lambda_powertools/metrics/metrics.py
100.00% <0.00%> (ø)

aws_lambda_powertools/logging/formatter.py
100.00% <0.00%> (ø)

aws_lambda_powertools/utilities/batch/base.py
100.00% <0.00%> (ø)

...ws_lambda_powertools/middleware_factory/factory.py
100.00% <0.00%> (ø)

...mbda_powertools/utilities/parser/models/kinesis.py
100.00% <0.00%> (ø)

...a_powertools/utilities/parser/models/cloudwatch.py
100.00% <0.00%> (ø)

...ertools/utilities/validation/jmespath_functions.py
100.00% <0.00%> (ø)

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 1131541...8ab3507. Read the comment docs.

Codecov Report

Merging #259 (8ab3507) into develop (1131541) will decrease coverage by 0.07%.
The diff coverage is 87.50%.

@@             Coverage Diff             @@
##           develop     #259      +/-   ##
===========================================
- Coverage    99.81%   99.74%   -0.08%
===========================================
  Files           76       76
  Lines         2763     2749      -14
  Branches       113      114       +1
===========================================
- Hits          2758     2742      -16
- Misses           4        5       +1
- Partials         1        2       +1

Impacted Files
Coverage Δ

aws_lambda_powertools/tracing/tracer.py
98.83% <87.50%> (-1.17%)
:arrow_down:

aws_lambda_powertools/logging/logger.py
100.00% <0.00%> (ø)

aws_lambda_powertools/metrics/metrics.py
100.00% <0.00%> (ø)

aws_lambda_powertools/logging/formatter.py
100.00% <0.00%> (ø)

aws_lambda_powertools/utilities/batch/base.py
100.00% <0.00%> (ø)

...ws_lambda_powertools/middleware_factory/factory.py
100.00% <0.00%> (ø)

...mbda_powertools/utilities/parser/models/kinesis.py
100.00% <0.00%> (ø)

...a_powertools/utilities/parser/models/cloudwatch.py
100.00% <0.00%> (ø)

...ertools/utilities/validation/jmespath_functions.py
100.00% <0.00%> (ø)

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 1131541...8ab3507. Read the comment docs.

Addressed all your comments and some other oversights I've noticed too - Here's the doc too. Over to you @nmoutschen

Addressed all your comments and some other oversights I've noticed too - Here's the doc too. Over to you @nmoutschen


create readme.it

created readme.it I re-read it a few times and hope there are the less number of errors as possible but if there is some1 just tell me without problems

hi, please make sure path to images and all links are correct; you might need to edit relative paths

path edited (sorry for the mistake) but when I go to the 'Files changed' section and show the preview of the edited file the images seems not working as before, whereas if I do some changes on the file and then show the preview it is right
Tell me if there is still the problem


Permission denied for exec in rootless docker

Context:

ubuntu 20.04
docker rootless
running docker-compose run --user="geth" execution (or any service with a script as entrypoint)

I got the following result:
Creating eth-docker_execution_run ... done
standard_init_linux.go:228: exec user process caused: permission denied
ERROR: 1

Different error without the --user parameter:
su-exec: /usr/local/bin/docker-entrypoint.sh: Permission denied

It works when permissions are set this way in the dockerfile:
...
COPY ./docker-entrypoint.sh /usr/local/bin
RUN chmod 755 /usr/local/bin/docker-entrypoint.sh  # added line

Is there a better way of fixing this ?

Interesting. I haven't tried rootless docker. I'll need to play with this a little.

Okay I installed rootless docker-ce and I cannot reproduce this, with the default user or with a geth user. Tried with besu and geth.

Ok weird. When running docker info | grep rootless do you see rootless as a result ?

In a nutshell yes, I guarantee this was a rootless install.
A lot of the entrypoint scripts are gone and the remaining ones are all 775 already in the filesystem and get copied into the images that way. Does this issue persist for you?

Closing as there is no further activity.


GDAL detection broken

I have GDAL both on the PKG_CONFIG_PATH and on the CMAKE_PREFIX_PATH, but do not give explicit cmake options. This leads to GDAL both being discovered and not:
-- Found GDAL: /work2/00434/eijkhout/gdal/installation-gdal-3.7.0-clx-gcc12/lib64/libgdal.so (found version "3.7.0")
-- Found OpenMP_C: -fopenmp (found version "4.5")
-- Found OpenMP_CXX: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
CMake Warning at submodules/dephier/submodules/richdem/CMakeLists.txt:75 (message):
  Compiling RichDEM without GDAL!

CMake Warning at submodules/dephier/CMakeLists.txt:28 (message):
  GDAL is required to build the DepHier example programs; it is not needed to
  use the DepHier library in another project.

CMake Warning at CMakeLists.txt:37 (message):
  GDAL is needed to build some of the FSM performance tests and example
  programs; it is not needed to use the FSM library in another project.

Ok, adding -D USE_GDAL=ON helps. But now the build breaks on libproj stuff not being found. Should you add a test for that?


signed binaries for goreleaser

mimics the signed binaries by goreleaser as implemented in celestia-app so node can also generate signed binaries
@MSevey confirmed that we have an org level secret for GPG_SIGNING_KEY and GPG_PASSPHRASE so should require no repo setup
implementation for app: https://github.com/celestiaorg/celestia-app/commit/934fdeda062229f5c150e9dce74f05b0557ea123
Will follow up with some work to ensure this workflow actually gets triggered (currently the steps before do not pass and generate binaries)
closes: https://github.com/celestiaorg/celestia-node/issues/2679

Changes lgtm. Actions unfortunately still don't really have a great way to test beside live testing.
For this one, I'd recommend making a new clean fork of the repo so that you can merge this change into that fork. Then update the CI workflow to bypass the tests so that the release is triggered so that you can then verify that the binaries and release are generated and then you can download and verify the binary as well.

@MSevey replicated and got this setup working and signing on one of our test tools https://github.com/ramin/waypoint/actions/runs/6645867234/job/18058097433#step:5:54, really only thing i can't test is that our GPG secrets are available and correct, so we'll have to do it live. Will get this entire pipeline running next so we can see it generate some artifacts


feat: DYN takes a memory address instead of digest on stack

Closes #1091
This change breaks how dyncall works (i.e. dyncall compiles to call(dyn)), since after the call, the memory is wiped since we entered a new context, and hence placing the procedure hash in memory before the call doesn't work.
We might need to do #1478 before we can merge this.
Left to do:

[ ] fix docs
[ ] fix test dynexec_with_procref()
[ ] cleanup TODO(plafer)

This change breaks how dyncall works (i.e. dyncall compiles to call(dyn)), since after the call, the memory is wiped since we entered a new context, and hence placing the procedure hash in memory before the call doesn't work.
We might need to do https://github.com/0xPolygonMiden/miden-vm/issues/1478 before we can merge this.

Ah yes - good point! I didn't think of this. Indeed, we may need to do #1478 first.

@bobbinth docs are ready for review

I'll merge this PR tomorrow (after miden-base has been migrated to the current next).

Let's hold off on merging this until we have a corresponding PR in miden-base (otherwise it'll break miden-base). @Fumuran will start working on this in the next day or so.


Support for WebSockets permessage-deflate extension?

Does the current version of VerneMQ support the permessage-deflate extension (https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression-19)? If not, are there any plans for this in the future?

Hi, compression for websockets is currently no supported, though it would not be a hard thing to add. However, the websocket server (cowboy 1.x) we currently use does not support the permessage-deflate and we'd have to upgrade to cowboy 2.x to do that which isn't planned for VerneMQ 1.x at the moment and VerneMQ 2.x doesn't have a set release date yet other than 'some time during 2018'. See also this issue for more information https://github.com/erlio/vernemq/issues/607


Declaration of files with suffixes is incorrect

Describe the bug
Hi, I'm facing an issue with the auto-import.d.ts file when generating files with the suffixes as: foo.composable.ts or bar.helper.ts, etc.
The problem is that the file removes the suffix, which causes an "any" type for my files, specifically within the declare global {} block. I've tried investigating the issue on my own but can't seem to understand the problem. Below, I'll list what I have attempted.
Problem:

What solved my problem was changing this line in the unplugin/unimport library:
function toTypeDeclarationItems(imports, options) {
  return imports.map((i) => {
+    const from = options?.resolvePath?.(i) || (i.typeFrom || i.from).replace(/\.[a-zA-Z]+$/, "");
-    const from = (options?.resolvePath?.(i) || i.typeFrom || i.from).replace(/\.[a-zA-Z]+$/, "");
    return `const ${i.as}: typeof import('${from}')${i.name !== "*" ? `['${i.name}']` : ""}`;
  }).sort();
}

However, when I forked the unplugin/unimport repository to create a pull request, it broke all tests hahaha. So, the issue might not be related to this change, or it could be happening due to how I am using the library.
So, I forked the unplugin-auto-import repository and tried to reproduce my problem. Surprisingly, everything worked well in the playground folder.
Lastly, I created a project on StackBlitz and another on my computer using bun create vite, and both exhibited the same problem I'm facing in my project.

To reproduce the error, run rm auto-import.d.ts; npm run build
Observe how the file is generated: foo.composable.ts is written as foo instead of foo.composable within the declare global {} block.
Reproduction
https://stackblitz.com/edit/vue3-vite-starter-srm5lm
System Info
System:
    OS: Linux 6.6 Arch Linux
    CPU: (12) x64 AMD Ryzen 5 5600X 6-Core Processor
    Memory: 19.51 GB / 31.25 GB
    Container: Yes
    Shell: 5.9 - /usr/bin/zsh
  Binaries:
    Node: 18.12.0 - ~/.asdf/installs/nodejs/18.12.0/bin/node
    Yarn: 1.22.19 - ~/.asdf/installs/nodejs/18.12.0/bin/yarn
    npm: 10.2.0 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: 8.12.0 - ~/.asdf/installs/nodejs/18.12.0/bin/pnpm
    bun: 1.0.11 - ~/.bun/bin/bun
  Browsers:
    Chromium: 120.0.6099.71

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[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, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

maybe you use basename(filename, extname(filename)) instead regexp


mustRunAfter not respected for dependencies of finalizer tasks

I'm using Gradle 5.6.1 on RHEL7 (Linux), and encountered a weird problem on a real project, which I reproduced on a very simplified test project: test-parallel.zip.
This test project has 3 subprojects, all doing the same: making a bear find a bee hive, steal it, check if the honey in it tastes good, and then throw away the hive. (Excuse me for not having a more meaningful example.)
The reason I have this many tasks is that in the real project I use the gradle-docker-plugin, which makes me create a task for each docker operation. And I wanted those tasks to be flexible, so they depend on each other, but a few tasks can be issued separately too (depending on command-line flags), so that's why you see mustRunAfter in places where a dependOn association would be just fine too.
Now, all tasks run fine when executed sequentially:
$ gw bearCheckIfHoneyTastesGood --continue --no-parallel

> Task :test1:bearFindBeeHive
> Task :test1:bearStealBeeHive
> Task :test1:bearInspectBeeHive
> Task :test1:bearRequireBeeHive
> Task :test1:bearCheckIfHoneyTastesGood
> Task :test1:bearThrowAwayBeeHive
> Task :test2:bearFindBeeHive
> Task :test2:bearStealBeeHive
> Task :test2:bearInspectBeeHive
> Task :test2:bearRequireBeeHive
> Task :test2:bearCheckIfHoneyTastesGood
> Task :test2:bearThrowAwayBeeHive
> Task :test3:bearFindBeeHive
> Task :test3:bearStealBeeHive
> Task :test3:bearInspectBeeHive
> Task :test3:bearRequireBeeHive
> Task :test3:bearCheckIfHoneyTastesGood
> Task :test3:bearThrowAwayBeeHive

When I run them in parallel, I expect that there is only one bee hive stolen at a time (in the real project the docker containers would conflict on resources). However, I see build failures occurring with a random probability, like:

multiple bee hives stolen at the same time (mustRunAfter not respected across projects)

sometimes without this detected in the Gradle code (I assume that is my fault for not using a thread-safe data type, or is it?)

Task bearRequireBeeHive executed before bearInspectBeeHive within the same project (mustRunAfter not respected within the same project), which is even weirder.

$ gw bearCheckIfHoneyTastesGood --continue

> Task :test2:bearFindBeeHive
> Task :test1:bearFindBeeHive
> Task :test3:bearFindBeeHive
> Task :test2:bearStealBeeHive
> Task :test1:bearStealBeeHive FAILED
> Task :test2:bearRequireBeeHive FAILED
> Task :test1:bearInspectBeeHive
> Task :test1:bearRequireBeeHive
> Task :test1:bearThrowAwayBeeHive
> Task :test2:bearInspectBeeHive
> Task :test2:bearThrowAwayBeeHive
> Task :test3:bearStealBeeHive
> Task :test3:bearInspectBeeHive
> Task :test3:bearRequireBeeHive
> Task :test3:bearCheckIfHoneyTastesGood
> Task :test3:bearThrowAwayBeeHive

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* Where:
Build file '/home/rkrisztian/projects/testProjects/test-parallel/test1/build.gradle' line: 16

* What went wrong:
Execution failed for task ':test1:bearStealBeeHive'.
> Only one bee hive should be stolen at a time.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* Where:
Build file '/home/rkrisztian/projects/testProjects/test-parallel/test2/build.gradle' line: 34

* What went wrong:
Execution failed for task ':test2:bearRequireBeeHive'.
> Bee hive not OK

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

* Get more help at https://help.gradle.org

BUILD FAILED in 516ms

I need your help with this: is this a Gradle bug, or am I doing anything wrong?
Thanks in advance.

It seems to me that what happens here is the following:
There is only a finalizedBy relationship between bearCheckIfHoneyTastesGood and bearThrowAwayBeeHive, so Gradle doesn't know up-front that bearThrowAwayBeeHive needs to run, since it is not part of the requested tasks.
And since it isn't part of the required tasks, e.g. :test1:throwAwayBeeHive doesn't block :test2:bearStealBeeHive from running.
In theory, Gradle knows that the finalizer will run at some point if one of the requested tasks executed or will be executed. So we could make this use-case work.
Do you want more something like shared resources which can only be used by one task at a time? In Gradle 6.0, we introduce a concept for exactly this: https://github.com/gradle/gradle/pull/9914
Maybe this is better suited for your needs.

Thanks for the explanation.
In theory, yes, the shared resources concept would help for this problem, if we can make sure that the same resource can be (atomically) reserved by all tasks executed between bearStealBeeHive and bearThrowAwayBeeHive, because the first task allocates the resources, and the last task frees it up in our case. So if there is still any possibility of race conditions between project test1 and test2, then our problem is still not solved, and it is not clear at a first glance that the PR can handle this too. I read the bug report that originated the PR though and it was very similar to my problem.
However, I would like to know  if the phenomenon that lead to task bearRequireBeeHive executed before bearInspectBeeHive is really the same and can be solved the same way, as the difference there is that those tasks were executed within the same project, so it is not obvious to me if the shared resources concept would help that case too.
In general, is there a documentation on mustRunAfter that it may not work for finalizer tasks? I tried to look up this information, and I found none, which lead me to this "bug" in the first place. So if there is none, I would at least ask for a little documentation update. Because someone else might run into the same problem in the future again, for something different, which may or may not be solved with a shared resource configuration.

There is some workaround you can use: you can add a mustRunAfter between the task which is finalized and the task which should run after it as well. That should give you the desired effect. So in your case, :test2:bearStealBeeHive.mustRunAfter(':test1:bearCheckIfHoneyTastesGood') should do the trick.

In theory, yes, the shared resources concept would help for this problem, if we can make sure that the same resource can be (atomically) reserved by all tasks executed between bearStealBeeHive and bearThrowAwayBeeHive, because the first task allocates the resources, and the last task frees it up in our case.

IIUC, shared resources currently wouldn't solve this problems currently. Though We may implement exactly what you describe above in the future.

I tried your suggestion, but it seems that no matter what I do, my mustRunAfter requests are still ignored. And I'm also failing to see how the suggestion results in correctness. I need to make sure that only one project executes any task between bearStealBeeHive and bearThrowAwayBeeHive (inclusively).
That's what I've been trying to say regarding "atomic" resource allocation for a task graph, i.e. that a group of tasks collectively want to use a shared resource (with the first task allocating it, last task freeing it up), and not to be treated as a list of individual tasks each one expected to let go of the resource after executed, which is not true in my case.

The following works for me:
In the top-level build.gradle, I ensure that the first task of the chain and finalizer of the first task of the chain runs after the last task of the chain and the finalizer of the last task of the chain:
private setTaskOrderBetweenProjects(projectBefore, projectAfter) {
    tasks.getByPath("$projectAfter:bearStealBeeHive").mustRunAfter "$projectBefore:bearThrowAwayBeeHive"
    tasks.getByPath("$projectAfter:bearStealBeeHive").mustRunAfter "$projectBefore:bearCheckIfHoneyTastesGood"
    tasks.getByPath("$projectAfter:bearRequireBeeHive").mustRunAfter "$projectBefore:bearThrowAwayBeeHive"
    tasks.getByPath("$projectAfter:bearRequireBeeHive").mustRunAfter "$projectBefore:bearCheckIfHoneyTastesGood"
}

Moreover, in the individual test projects, I make sure that bearRequireBeeHive runs after bearStealBeeHive as well, which is finalized of bearInspectBeeHive:
task('bearRequireBeeHive') {
      mustRunAfter bearInspectBeeHive
      mustRunAfter bearStealBeeHive

...
}

After these changes, the tasks execute as expected.

Thank you so much, the same worked in my real project too!
Although we could at least find a workaround, having to configure this many associations is a bit less than ideal, to be honest. One would imagine that the following would normally be enough:
    tasks.getByPath("$projectAfter:bearStealBeeHive").mustRunAfter "$projectBefore:bearThrowAwayBeeHive"

(I didn't add the bearRequireBeeHive one, because in my real project, we would run only the full chain in parallel. Partial chain executions are only considered for individual subprojects due to resource conflicts.)
Therefore, I would say, please consider at least documenting this limitation of mustRunAfter, if not adding full support for task graphs with finalizers, as said above. It is no longer urgent for me, knowing that there is a fine workaround.

Sadly the given workaround may break once tasks.getByPath gets removed from Gradle due to the pushing of the task configuration avoidance API. Plus my memory is getting vague here, but I've read somewhere that it's not a good design to call such a method because we are digging into the internals of other projects.

FWIW, I've just hit this issue.
It would be grand to get this fixed because it's actually quite surprising to have mustRunAfter ignored when the argument is a task that is a finaliser and for it to only happen when running in parallel mode so you end up scratching your head and doubting your build configuration... In the meantime I will use the workaround of also adding mustRunAfter for the finalized task as well, thanks for providing that suggestion @wolfs.

I tried with Gradle 7.6 and this seems to work now: https://ge.gradle.org/s/zgn35dykqcus2/timeline


CommonJS support

It would be great to have CommonJS support. Angular has it and it makes the use of dependencies much easier. See: http://angularjs.blogspot.com/2015/02/new-angularjs-releases-140-beta5-and.html
Possible usage example:
var angular = require('angular');
angular.module('myApp', [
  require('angular'),
  require('angular-gantt'),
  require('angular-gantt-table-plugin')
]);

I would need help for this. It's very easy to make a module for a single NPM package and use it with Browserify but for our plugins it would be necessary to expose multiple modules in a single package. Somehow I didn't see how to do this yet.

I'm in favor of switching to a better module management, and more globally a new build system.
To go forward and prepare angular 2 migration #350, we could switch directly to SystemJS and ES6 in the project. SystemJS is the ES6 module loader used in angular 2. SystemJS can load any ES5 modules packages (AMD, CommonJS, ...), so we should then rely on npm packages for dependency instead of bower.
It requires some build script to provide ES5 CommonJS modules to users of angular-gantt, but it seems doable with babel.

Some links on subject:

http://ilikekillnerds.com/2015/07/jspm-vs-webpack/ (That's why I'm not in favor of webpack, let's jump an historical step instead ...)
http://martinmicunda.com/2015/02/09/how-to-start-writing-apps-with-es6-angular-1x-and-jspm/

One advantage for webpack though : It supports things outside of pure javascript scope, like stylesheets and precompilation. But it seems not possible to use both Webpack and SystemJS.

I'm definitely in favor of System.js. It removes the development build step and is future proof. It also allows export of CommonJS, AMD and other module formats to keep compatibility with current module systems.

@Schweigi Maybe it's time to start migrating to angular 2 as it's in beta now ... I'll start reading this about ngUpgrade and ngForward to engage angular-gantt in this process ... But don't expect anything before February I think :p.

Closing in favor of #732.
Fun to see how things change quickly in JS world, System.js was the hype 2 years ago, but now it seems to just suck and has lost versus Webpack. For now ...


Add footer section and in contact us page

I would like to work on it, Should be similar to the home page or I can customise it.

I would like to work on it, Should be similar to the home page or I can customise it.

ya sure @shindesuyog go ahead ...add footer and make it responsive ....and u are free to customize the contact us page
add some cool stuffs

I would like to work on it, Should be similar to the home page or I can customise it.

ya sure @shindesuyog go ahead ...add footer and make it responsive ....and u are free to customize the contact us page
add some cool stuffs

and if u want illustrations image then ask me ill send u

@shindesuyog any updates ??

I am working on it , will make pull request by tomorrow.

ok @shindesuyog

I would like to work upon this issue. Please assign me this to improve footer section and contact us page @Ayan-thecodeking

I would like to work upon this issue. Please assign me this to improve footer section and contact us page @Ayan-thecodeking

hey @aditya687 just create another issue for improvement is contact page i'll assign u

Ok, I'll try to do contact page


Overwriting types classes

Right now we can easily add our own resolvers, own object types, args, inputs, etc.
However, we can't easily change and replace implementation of one object type or input type, like simplifying StringFilter or some UserWhereUnique input.
It would be possible to allow providing alternative implementation of such types, e.g. with proper validation:
@InputType()
class UserWhereUnique {
  // alternative body, fields and decorators used
}

const typeReplacementMap = {
  inputs: {
    UserWhereUnique
  },
};
export default typeReplacementMap;

So the generated artifacts would always use that provided type instead of the default generated ones:
@ArgsType()
export class UserFindOneArgs {
  @Field(() => typeReplacementMap.inputs.UserWhereUnique)
  name!: UserWhereUnique;
}

This will make the generated artifacts harder to eject and use as standalone code but will reduce the need to do the eject at all.

So the only option right now to actually change an input type would be to reimplement it completely and then use it in a custom resolver, right?

Will this issue allow us to add a new field to an WhereInput? Today we need to implement the whole resolver, right?


deploy hbase on another server,which file did i modify?

hi, i have 3 servers to deploy pinpoint
server1 :  i deployed hbase
server2:   i  deployed collector and web
server3:   i deployed  agent on it which in  a testAPP
before started collector and web , i had change hbase.properties
hbase.client.host=server1ip
hbase.client.port=2181
when i start the collector and web , i see logs printing

did i miss modify any args?

Hello, @Rnic
Just looking at the log, it looks like the connection has been refused from the embedded zookeeper in hbase.
Have you noticed, there are two hbase.properties. one for Pinpoint-Collector and one for the Pinpoint-Web. If you have modified both of them, please let me know which has produced those logs that you've posted, the Collector? Web?.

hi,@RoySRose   thank u for your information.
i had changed two hbase.properties  files,  the picture is from pinpoint-Collector's hbase.properties.
the log always print "connection refused",did't see any another things.
by the way ,  i see the log from pinpoint-web ,it seems like the same question.
you mentioned the embedded zookeeper ,did i need to change anything about the embedded zookeeper?

hi, @Rnic
If you had changed two properties files from Collector and the Web. I think that is all you need to do.
Even if you connect to the embedded zookeeper. you don't really need to change anything.
I've mentioned the embedded zookeeper, because I see the word 'zookeeper' in your log file. Since the connection has been refused from Pinpoint-Web to the Hbase. I think you should start from there. Maybe firewall? or anything else?

@RoySRose  thank u for ur help again, i checked the firewall they are closed, and i will try to figure it out


Doesn't see Falcon 40b converted, to be a GGML file..ctransformers problem?

On Ubuntu Focal..
Current binding: c_transformers
Current model: None
Menu:
1 - WizardLM-Uncensored-Falcon-40b.reference
2 - Install model
3 - Change binding
4 - Back
Enter your choice: 1
You selected model: WizardLM-Uncensored-Falcon-40b.reference
Loading a reference model:
/home/michael/models/WizardLM-Uncensored-Falcon-40b
error loading model: unknown (magic, version) combination: 04034b50, 08080000; is this really a GGML file?
falcon_init_from_file: failed to load model

I have both the original, as well as the converted, and quantized versions in that directory.
drwxrwxr-x 1 michael michael         1206 Jul 22 07:27 .
drwxrwxr-x 1 michael michael          140 Jul 21 20:04 ..
-rw-rw-r-- 1 michael michael          719 Jul 21 20:04 config.json
-rw-rw-r-- 1 michael michael         2511 Jul 21 20:04 configuration_RW.py
-rw-rw-r-- 1 michael michael          111 Jul 21 20:04 generation_config.json
-rw-rw-r-- 1 michael michael<PHONE_NUMBER>89 Jul 22 07:21 ggml-model-falcon-40b-wizardlm-f32.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER>0 Jul 22 07:30 ggml-model-falcon-40b-wizardlm-qt_k_m.bin
drwxrwxr-x 1 michael michael          182 Jul 22 06:57 .git
-rw-rw-r-- 1 michael michael         1477 Jul 21 20:04 .gitattributes
-rw-rw-r-- 1 michael michael           14 Jul 21 20:04 latest
-rw-rw-r-- 1 michael michael        47131 Jul 21 20:04 modelling_RW.py
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:01 pytorch_model-00001-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:01 pytorch_model-00002-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:02 pytorch_model-00003-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:01 pytorch_model-00004-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:02 pytorch_model-00005-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:02 pytorch_model-00006-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:02 pytorch_model-00007-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:02 pytorch_model-00008-of-00009.bin
-rw-rw-r-- 1 michael michael<PHONE_NUMBER> Jul 21 22:21 pytorch_model-00009-of-00009.bin
-rw-rw-r-- 1 michael michael        39264 Jul 21 20:04 pytorch_model.bin.index.json
-rw-rw-r-- 1 michael michael         1051 Jul 21 20:04 README.md
-rw-rw-r-- 1 michael michael          305 Jul 21 20:04 special_tokens_map.json
-rw-rw-r-- 1 michael michael          207 Jul 21 20:04 tokenizer_config.json
-rw-rw-r-- 1 michael michael      2734597 Jul 21 20:04 tokenizer.json
-rw-rw-r-- 1 michael michael        13129 Jul 21 20:04 trainer_state.json
-rw-rw-r-- 1 michael michael         5051 Jul 21 22:02 training_args.bin
-rw-rw-r-- 1 michael michael        24043 Jul 21 20:04 zero_to_fp32.py

Easy fix? Suggest that when the 'path' is chosen, if it doesn't see a GGML file, that maybe it can prompt for the actual name of the GGML file of choice.
Couldn't load model. Please verify your configuration file at /home/michael/Documents/lollms/configs or use the next menu to select a valid model
Binding returned this exception : Failed to create LLM 'falcon' from '/home/michael/models/WizardLM-Uncensored-Falcon-40b/training_args.bin'.
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/lollms/app.py", line 75, in load_model
    model = ModelBuilder(self.binding).get_model()
  File "/usr/local/lib/python3.10/dist-packages/lollms/binding.py", line 285, in __init__
    self.build_model()
  File "/usr/local/lib/python3.10/dist-packages/lollms/binding.py", line 288, in build_model
    self.model = self.binding.build_model()
  File "/home/michael/Documents/lollms/bindings_zoo/c_transformers/__init__.py", line 114, in build_model
    self.model = AutoModelForCausalLM.from_pretrained(
  File "/usr/local/lib/python3.10/dist-packages/ctransformers/hub.py", line 157, in from_pretrained
    return LLM(
  File "/usr/local/lib/python3.10/dist-packages/ctransformers/llm.py", line 214, in __init__
    raise RuntimeError(
RuntimeError: Failed to create LLM 'falcon' from '/home/michael/models/WizardLM-Uncensored-Falcon-40b/training_args.bin'.

personal_models_path: /home/michael/Documents/lollms/models
Binding name:c_transformers
Model name:WizardLM-Uncensored-Falcon-40b.reference
Please select a valid model or install a new one from a url

It isn't clear from the errors, but it is actually bubbling up from ctransformers, and have filed ticket there as well.
Issues with ctransformers_llm_create and Falcon-40b model #60
Only posting here to see if perhaps it is not called with correct parameters, but it 'appears' this isn't a problem with that.
Ticketing here for others trying to use ctransformers with Falcon for now.

Okay, the problem here appears to lie in the way AutoModelForCausalLM is called..  When using a ctransformers method, we should be calling it with a model_file argument. See https://github.com/marella/ctransformers/issues/60.  Otherwise, it can return any .bin from the path, including training_args.bin, which of course will fail ggml checks.

And to continue, I have the actual filename in the reference file
cat ~/lollms_data/models/llama_cpp_official/falcon-40b-ggml.reference

~/models/WizardLM-Uncensored-Falcon-40b/ggml-model-falcon-40b-wizardlm-qt_k5.bin

Howeer, it doesn't see that it properly sees that as a file.  Instead, earlier that was parsed down to model_path, and model_name was not preserved.. So when we run..  _find_model_path_from_dir, the optional filename is not passed in.  AutoModelForCausalLM.from_pretrained(), which was called after get_model_path
if self.config.model_name.endswith(".reference"):
     with open(str(self.lollms_paths.personal_models_path / f"{self.binding_folder_name}/{self.config.model_name}"), 'r') as f:
                model_path = Path(f.read()

That means if you have the file name in a .reference it will not work. ?

Well, guess I am still talking to myself.. ;) Went up and down this code, and there doesn't seem to be a 'recommended' way to handle setting the actual filename from the config, for the ctransformers binding (or any binding).  I think the work is all about pulling down the needed models, but when you have a directory of different quantized models you are kind of hooped.
The transformers binding 'should' be doing more that the simple get_model_path(self), which in itself could use a little validation.It should only read the .reference file once if it exists, and set both path and filename if the .reference file contains a fullpath to a model.
Not really sure why the transformer bindings were decided that they should NOT be part of the library itself, and placed in the users lollms personal settings.. Seems like a strange design choice.
/c_transformers/init.py needs to be updated. .
To get on with things, since noone is commenting, no use making a pull request.. I just hacked get_model_path() to set self.model_file, and the zoo.. init.py, to pass that in when it calls AutoModelForCasual.  NOTE, this would ALSO work if get_model_path could return whatever is in the .reference file, but that goes against my grain, as the function could be used elsewhere, where it really does need the 'path'

Hi there, I am sorry. I has loads of issues opened and yours was down and I was both in vacation, then was coding so i didn't see it.
I have dixed your problem. It was just about the code that determines the type of the file out of its name. Now Wizard stuff is considered llama so all of his code is now accepted as llama.


display info for icfp prepopulated data page

Before submitting this PR as ready for review, please make sure:

[x] Work items have been linked
[x] Your code builds clean without any errors or warnings
[x] You have run all unit/integration tests and they pass
[x] Your branch has been rebased onto master

ticket
This PR adds:

displays school and finance info for icfp prepopulated data page
integration tests amended to account for required setup for this page

outstanding required work to complete ticket

post the plan
validation
retrieval of the plan if it exists

Use AB#[ticket] to link the work items.


1572/menu thumbnails

Ticket: #1572

Coverage increased (+0.006%) to 94.067% when pulling d8de769cfbb6cd9a3b5f1c09015575e4a7ffdac0 on 1572/menu_thumbnails into 8d18547b193e7fd0ebdb629f9130a22dbb191493 on master.

Coverage increased (+0.006%) to 94.067% when pulling 5518b414ca999ee831a1fefe5473f8af7127253c on 1572/menu_thumbnails into 8d18547b193e7fd0ebdb629f9130a22dbb191493 on master.

Coverage increased (+0.006%) to 94.067% when pulling 5518b414ca999ee831a1fefe5473f8af7127253c on 1572/menu_thumbnails into 8d18547b193e7fd0ebdb629f9130a22dbb191493 on master.

Coverage increased (+0.006%) to 94.067% when pulling 5518b414ca999ee831a1fefe5473f8af7127253c on 1572/menu_thumbnails into 8d18547b193e7fd0ebdb629f9130a22dbb191493 on master.

Coverage increased (+0.005%) to 94.287% when pulling 6904c9c086e88922f2c359260cc5d1dad779b1a2 on 1572/menu_thumbnails into 7bef80a6e1dfbdf2da9f1525b5e4187cafeb5541 on master.

Coverage increased (+0.004%) to 94.285% when pulling 5ee64eb1935127746520af6e0044d987fdee77dd on 1572/menu_thumbnails into 37765c19c3c856e47708315b8c4f784ceb988826 on master.

Coverage increased (+0.03%) to 94.315% when pulling 5ee64eb1935127746520af6e0044d987fdee77dd on 1572/menu_thumbnails into 37765c19c3c856e47708315b8c4f784ceb988826 on master.


Add files via upload

I have created service link 1 issue #11 please review it and give some improvement feedback

Kindly Write proper issue number at PR time so that I can merge it

sir kindly check again

I had already raised a issue you have to mention that at the PR time. Read all the things very carefully.I think you can see that at the eñd of the PR submission.
Try it.if you stuck then I will help you to make a successful PR


FORTECH-108: Fix to compile JS documentation properly

Fix to compile JS documentation properly
Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>

@adrian-cocoara-lgp fixed the issue. I just created the PR based on his patch.

Actually, we have to apply left: -50%  in my commit -https://github.com/enactjs/agate/pull/346/commits/766de8de3f4ce7363aeaf018e22ca64410e3bad7. But due to the overflow: hidden in the parent DOM element, the divider could be clipped. We can't remove the overflow css style. So I just simply apply left: 0 here.

:shipit: