Issue with 'CRYSTALpytools.electronics ChargeDensity'

When plotting 2D charge density, I was running the ECHG calculation followed by a PATO + ECHG tag to calculate the charge density and charge difference at once, CRYSTALpytools cannot read in the output file properly, with error message below:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
Cell In[141], [line 3](vscode-notebook-cell:?execution_count=141&line=3)
      [1](vscode-notebook-cell:?execution_count=141&line=1) from CRYSTALpytools.electronics import ChargeDensity
----> [3](vscode-notebook-cell:?execution_count=141&line=3) fig = ChargeDensity.from_file('AF1_UDUD_PBE0/ECHG/echg_old.f25',
      [4](vscode-notebook-cell:?execution_count=141&line=4)                               output='AF1_UDUD_PBE0/ECHG/echg_old.outp').plot_2D()

File /opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/electronics.py:535, in ChargeDensity.from_file(cls, output, method, *files)
    [533](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/electronics.py:533) file.close()
    [534](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/electronics.py:534) if '-%-' in header: # 2D plot in fort.25
--> [535](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/electronics.py:535)     cls = Properties_output(output).read_ECHG(*files, method=method)
    [536](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/electronics.py:536) return cls

File /opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/crystal_io.py:2857, in Properties_output.read_ECHG(self, method, index, *f25_files)
   [2855](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/crystal_io.py:2855) ichg = np.where(headers>chg[use_idx])[0][0]
   [2856](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/crystal_io.py:2856) if len(spin) != 0:
-> [2857](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/crystal_io.py:2857)     ispin = np.where(headers>spin[use_idx][0][0])
   [2858](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/crystal_io.py:2858)     if method == 'substract': # pato substract
   [2859](https://file+.vscode-resource.vscode-cdn.net/opt/anaconda3/envs/crystalPytool/lib/python3.9/site-packages/CRYSTALpytools/crystal_io.py:2859)         index = np.array([ipato, ichg, ispin], dtype=int)

IndexError: invalid index to scalar variable.

The ECHG.d3 file is as follow:
ECHG
0
65
COORDINA
-4. -4. 0.0
4. -4. 0.0
4. 4. 0.0
MARGINS
1.5 1.5 1.5 1.5
END
PATO
0 0
ECHG
0
65
COORDINA
-4. -4. 0.0
4. -4. 0.0
4. 4. 0.0
MARGINS
1.5 1.5 1.5 1.5
END
END

The CRYSTALpytool python script as follow:
from CRYSTALpytools.electronics import ChargeDensity

fig = ChargeDensity.from_file('file_name/ECHG/echg_old.f25',
                              output='file_name/ECHG/echg_old.outp').plot_2D()

Thank you!

Dear Fei,
Thank you for identifying this issue. We can confirm that it is a bug of source code. Now it should have been addressed in the lateest version 24.11.2. Please refer to the last example of This Section.
Note: only the difference of charge density is returned with the subtract method for both spin-polarized and non-polarized systems since the difference of (\alpha-\beta) densities is somewhat 'meaningless' to us.
Kind regards,
CPYTools developers


Contributor Guide link in Community doc is broken

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2669
Originally created by @DouweM on 2021-03-23 15:58:07

View 2 previous comments from the original issue on GitLab


Issue with cryptography module

I have been facing this exclusive issue including crpytography component. It have no problem compiling the same but when we run the application - it fails. It gets the following errlr.
Traceback (most recent call last):
File "E:\Projects\SomeInternalProject\app.py", line 5, in
from ProcessManager import ProcessManager
File "E:\Projects\SomeInternalProject\ProcessManager.py", line 6, in
import globals
File "E:\Projects\SomeInternalProject\globals.py", line 8, in
from cryptography.fernet import Fernet
File "E:\Projects\SomeInternalProject\venv\lib\site-packages\cryptography\fernet.py", line 18, in
from cryptography.hazmat.primitives import hashes, padding
File "E:\Projects\SomeInternalProject\venv\lib\site-packages\cryptography\hazmat\primitives\padding.py", line 13, in
from cryptography.hazmat.bindings._padding import lib
ModuleNotFoundError: No module named 'cryptography.hazmat.bindings._padding'
Have tried various options
python -m nuitka --follow-imports --standalone --python-version=3.6 --experimental=use_pefile --experimental=use_pefile_recurse --experimental=use_pefile_fullrecurse --plugin-enable=pylint-warnings  --plugin-enable=numpy --recurse-to=cryptography --recurse-to=cryptography.hazmat --recurse-to=cryptography.hazmat.bindings app.py
and also
python -m nuitka --follow-imports --experimental=use_pefile --experimental=use_pefile_recurse --experimental=use_pefile_fullrecurse --plugin-enable=pylint-warnings  --plugin-enable=numpy --include-package=cryptography app.py
but every time we try to run the compiled output - we end up with the above issue.
Secondly - will like to mention that the cryptography module that was installed are all compiled to pyd.. and it dose not have source attached. No problem i can get the source code too if required - but this is something i am stuck in. Can someone please help me get out of this issue? PLS??

Well just realized i should also mention - the same when i try running inside the console .. i am not getting any error
(venv) λ python
Python 3.6.4 (v3.6.4:d48eceb, Dec 19 2017, 06:54:40) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from cryptography.hazmat.bindings._padding import lib
>>>

pls provide me with your app source.
I will try and dig thru the problem.

I suggest to close this thread as much more with the same topic was done here: https://github.com/Nuitka/Nuitka/issues/645

This is indeed a duplicate and works these days.


Fix for issues after recreating the control on iOS

If we recreate the control without changing the current BindingContext (for example recreating the control on orientation change, with changed width size), the control has't stored the previous state, so the position changing animation could be happen to the wrong direction.
Also, there was a NullReferenceException, because a delegate called during recreation, when the Element had a null value.

What about Android and UWP?

I haven't tested on UWP, but on Android it is working fine without any modifications. I've tested with a low-end device, and with a Nexus 5X, and it works as expected.

All right, i will incorporate your fix before doing the "pull request" for Xamarin.Forms. The plugin is about to officially become Xamarin.Forms CarouselView. Its being pulled into Xamarin.Forms core :)

Wow, that's very great! You plugin is awesome, so congratulations! :)

I hope to get more nominations now and finally become an MVP :)
Thanks

Can you provide a demo where this can be reproduced?

Unfortunately I don't have much time nowadays, but I'm almost completely sure that you can reproduce the issue in the following way:

Create a page with a ViewModel.
Add an eventhandler to the SizeChanged event of the page.
The content of the page has to be set in the eventhandler. (So the page is recreated on every orientation change. I've ensured that the recreation is made only when the orientation really changed, from portrait to landscape, or the opposite.)
Place a carouselview in the content, and bind it's Position property to a property in the ViewModel. (So the position will be set even if the page is recreated on orientation change)
Override the page's OnPropertyChanged method: if the property what is changed is the ContentProperty, set the ViewModel's position property to a certain value. (I've made this, to ensure that the view on the requested position in the CarouselView is already created, so the position change can be made without a problem. Maybe it's not a good decision, but I had problems with that with the official CarouselView.)

With the configuration above, the carouselview will be recreated on every orientation change, and it's position will be set to the value before the orientation change happened.
Because the view is recreated, the renderer's SetCurrentController method is called one time, but the local prevPosition is recreated too, with the default 0 value. (So the position changing animation will be always called to the forward direction.)
I'm not sure why the NullReferenceException happens, that would need more research. The only thing I'm sure about, is that the error happens in a reproducible way. (Maybe changing the orientation multiple times in a fast way can cause the error, because of the async method calls.)

You have missed out the Element null check at the SetCurrentPage method, in the latest version. Now I get the NRE again, when rotate the device. Can you fix this, please?

Is that the only place this fix is needed? Android or iOS?

Because i rotate the device and it doesn't happen

I think that's the only place. On iOS, like in this PR.
It happens because I recreate the page on orientation change, so it should be some async problem.

Okay, I think I found the problem. It occurs, if the Position is binded to the ViewModel, but the ItemSource isn't. (It's just set in the CarouselView's constructor.) In this case, the rapid rotation causes the NRE. If the ItemSource is also binded, the exception won't come. (It is the wanted behaviour, so my issue is not relevant anymore.)


extend to gamlss

Would it be difficult to make these extensions? https://en.wikipedia.org/wiki/Generalized_additive_model_for_location,_scale_and_shape

I've been reading about these and i believe this requires a LOT of work.
They sound really cool though.


Which package is it in Sublime's Package Control?

I can find no entry for any of these strings

terminal
sublime_terminal
sublime-terminal
browse terminal

I am using SublimeText 3, on Linux.

It should be discoverable via "Terminal".
Package Control doesn't list already installed packages via "Package Control: Install Package" so try "Package Control: List Packages" as well to see if it's already been installed

Thank you. Using 'List packages' did indeed reveal the program. (It is annoying, isn't it, that installed programs do not show up at all 'install packages'?) However, and this is why I had not noticed Terminal was installed, the installation had somehow broken. Removing Terminal and reinstalling it seems to have fixed everything.


FF-153

Coverage decreased (-0.9%) to 28.718% when pulling 422c5444f89dd5e9bb95c4a12f9d5140885c512b on loadxl_on_submit4dn into 89a09d0bd38f6247e0427d3f0c4974d55a84334e on master.

Coverage decreased (-0.9%) to 28.718% when pulling 131a1252b9b84456d0c8d3b2192228c27f1d8ad2 on loadxl_on_submit4dn into 41ba15ee60b606b017910b91bc701b2759a86318 on master.

Coverage decreased (-0.9%) to 28.718% when pulling 131a1252b9b84456d0c8d3b2192228c27f1d8ad2 on loadxl_on_submit4dn into 41ba15ee60b606b017910b91bc701b2759a86318 on master.


Fix devcontainer dotnet image tag

I want to pick up two mini-projects with dotnet interactive, namely support for F# 8.0 and Postgresql support #1968. My first test was to start up a the devcontainer and build the project, which failed because there registry doesn't contain a devcontainer/dotnet image with tag 0-8.0.
This PR changes both default tag in the Dockerfile and devcontainer.json file to 1-8.0-bookworm-slim, and for the VARIANT arg in the Dockerfile not to have the implicit prefix 0-.*.
I had two other problems in my test, one related to VSCode not correctly handling the .devcontainer.json file which I was able to fix by adding an initializeCommand (see https://github.com/microsoft/vscode-remote-release/issues/9302#issuecomment-1859139566) and the FSharp.Core.xml file couldn't be found in the NuGet cache (cleaning the cache didn't fix the issue but manually writing the file did), I don't think either issue requires a fix in the repo.

@dotnet-policy-service agree

Thanks for the feedback @WeihanLi and thanks for addressing it @colombod, I forgot to come back to this!


after option applies only to MySQL

Doesn't work with SQLite, and apparently is only available in MySQL (See this SO answer)

Codecov Report

Merging #1253 into master will increase coverage by 2.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1253      +/-   ##
==========================================
+ Coverage   72.32%   74.57%   +2.24%
==========================================
  Files          35       35
  Lines        5551     4732     -819
==========================================
- Hits         4015     3529     -486
+ Misses       1536     1203     -333

Impacted Files
Coverage Δ

src/Phinx/Migration/AbstractTemplateCreation.php
73.33% <0%> (-4.45%)
:arrow_down:

src/Phinx/Console/Command/Breakpoint.php
36% <0%> (-3.29%)
:arrow_down:

src/Phinx/Seed/AbstractSeed.php
46.34% <0%> (-1.58%)
:arrow_down:

src/Phinx/Console/Command/AbstractCommand.php
50.89% <0%> (-1.49%)
:arrow_down:

src/Phinx/Console/Command/Test.php
24.13% <0%> (-0.87%)
:arrow_down:

src/Phinx/Db/Table.php
95.87% <0%> (-0.37%)
:arrow_down:

src/Phinx/Db/Adapter/AbstractAdapter.php
80.32% <0%> (-0.27%)
:arrow_down:

src/Phinx/Db/Adapter/TablePrefixAdapter.php
79.54% <0%> (-0.07%)
:arrow_down:

src/Phinx/Db/Table/ForeignKey.php
97.91% <0%> (-0.05%)
:arrow_down:

src/Phinx/Migration/AbstractMigration.php
100% <0%> (ø)
:arrow_up:

... and 24 more

Continue to review full report at Codecov.

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


Smaller Fitting than GT?

Hi folks, @JanaldoChen @woo1 et al.,
I tried the demo for in-the-wild videos and found fittings seem smaller than expected. May I check if this is a common issue?
https://github.com/JanaldoChen/Anim-NeRF/blob/1342a9e2d02411a09acecac40ac325f38708b9c9/tools/vibe.py#L237
Thx & rgds,
I attached an example here,

solved by https://github.com/mkocabas/VIBE/issues/182#issuecomment-1892156211


[sdformat9] no absolute paths

For #20469

poke @autoantwort for response

Hm now it fails for me with  #22602

Huh I closed it? @PhoebeHui can you reopen it? :)

Thanks!


a little bug about loss

At CLAP loss: logits_per_text = logit_scale_a * text_features @ audio_features.T
I think logit_scale_a should be logit_scale_t??
https://github.com/LAION-AI/CLAP/blob/776b7dc95f2dc71775e9dc9804876e23265285b7/src/laion_clap/clap_module/loss.py#L194C1-L195C1
Thanks a lot and Great work!!

Hi, the naming of the variable is a bit confusing but we used an identical temperature scaling for both text and audio logits. Please also see: https://github.com/LAION-AI/CLAP/issues/109


[SUPPORT] BigQuery Sync error with --use-bq-manifest-file option when syncing Hudi table to BigLake table.

Describe the problem you faced
I have an issue when trying to submit Spark Job to sync Hudi table to BigLake table with --use-bq-manifest-file option I've built hudi-gcp-bundle-0.14.0-rc1(rc2).jar JAR file to run spark-submit. The error is shown with Exception: Exception in thread "main" java.lang.NoSuchMethodError. The ConfigProperty Class still has markAdvanced() method.
Expected behavior
A clear and concise description of what you expected to happen.
The spark-submit job will run successfully without error with  --use-bq-manifest-file option and generate a manifest file with the BigLake table in GCP.
Environment Description

Hudi version : 0.14.0

Spark version : 3.3.2

Hive version : 3.1.3

Hadoop version : 3.3.3

Storage (HDFS/S3/GCS..) : GCS

Running on Docker? (yes/no) : No

Additional context
Stacktrace
Exception in thread "main" java.lang.NoSuchMethodError: 'org.apache.hudi.common.config.ConfigProperty org.apache.hudi.common.config.ConfigProperty.markAdvanced()'         at org.apache.hudi.gcp.bigquery.BigQuerySyncConfig.<clinit>(BigQuerySyncConfig.java:57)         at org.apache.hudi.gcp.bigquery.BigQuerySyncConfig$BigQuerySyncConfigParams.toProps(BigQuerySyncConfig.java:163)         at org.apache.hudi.gcp.bigquery.BigQuerySyncTool.main(BigQuerySyncTool.java:157)         at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)         at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)         at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)         at java.base/java.lang.reflect.Method.invoke(Method.java:566)         at org.apache.spark.deploy.JavaMainApplication.start(SparkApplication.scala:52)         at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:973)         at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:180)         at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203)         at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90)         at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1061)         at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1070)         at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)

@thanhnd20 Can you please share your mvn command you have used to build the jar.  By error I can see that the gcp bundle jar may be conflicting with hudi bundle.

@ad1happy2go
The mvn command I have used to build Jar bundle packages: mvn -am -pl packing/gcp-hudi-bundle clean package DskipTest=true.
The complete command I have used to run spark-submit:
spark-submit \    --master yarn \    --packages com.google.cloud:google-cloud-bigquery:2.10.4 \    --class org.apache.hudi.gcp.bigquery.BigQuerySyncTool \    gs://<my_gs_folder>/hudi-gcp-bundle-0.14.0-rc1.jar \    --project-id <my_project_id> \    --dataset-name <my_datasetname> \    --dataset-location asia-southeast1 \    --source-uri gs://<my_gs_folder>/hudi_trips_cow/asia=* \    --source-uri-prefix gs://<my_gs_folder>/hudi_trips_cow/ \    --use-bq-manifest-file true
If you need me to provide any more information, please let me know.

The markAdvanced is a recent change so I had some thoughts about whether there is another jar included on the spark cluster with an older hudi version.

@thanhnd20  can you reply on the ticket about whether you are using other Hudi versions on your class path? cc @the-other-tim-brown @ad1happy2go -
following up on the slack thread: https://apache-hudi.slack.com/archives/C4D716NPQ/p1695734405920159?thread_ts=1694417492.831359&cid=C4D716NPQ

@thanhnd20 was able to fix this error by upgrading the Spark Hudi version in Spark Cluster. Thanks for the all the support from everyone.


Remove all usages of empty lists in kwarg defaults

@mambocab to review. #417 brought to my attention that this mistake was occurring in our codebase.

Looks good - the only instances I see of changing [] to None and not subsequently setting it are when the method isn't implemented. +1.


Legend order of chart #188 CO2 biomass

I find the legend order of chart #188 (co2 biomass) a little funny, it looks a bit random (see figure below). @DorinevanderVlies , am I right that you have created this chart? Maybe you know why it's like this?

Perhaps we should hide all zero/small series and their legends?

I think me and @jorisberkhout crated this chart together.
It becomes more clear when there is a significant amount of biomass in a scenario and the CO2 for biomass sliders are set:

Perhaps we should hide all zero/small series and their legends?
Great idea, maybe @grdw can help with this

Please have a look at https://github.com/quintel/etmodel/issues/2596 before making any changes to this chart.


about the size of prototype.

Thanks for your great work and code sharing. Here is one confusing I concerted.
I saw you define the objective_vectors in models/adaptation_modelv2.py with a size of  256
        self.objective_vectors = torch.zeros([self.class_numbers, 256])         self.objective_vectors_num = torch.zeros([self.class_numbers])
but I found that the channel of feature maps is 2048 in layer4.
self.layer4 = self._make_layer(block, 512, layers[3], stride=1, dilation=4, BatchNorm=BatchNorm) self.layer5 = self._make_pred_layer(Classifier_Module2, 2048, [6, 12, 18, 24], [6, 12, 18, 24], num_classes)
and in your command, you just use the output feature of layer4 as out['feat']
`    def forward(self, x, ssl=False, lbl=None):
_, _, h, w = x.size()
x = self.conv1(x)
x = self.bn1(x)
x = self.relu(x)
x = self.maxpool(x)
x = self.layer1(x)
x = self.layer2(x)
x = self.layer3(x)
x = self.layer4(x)
if self.bn_clr:
x = self.bn_pretrain(x)
    out = self.layer5(x, get_feat=True)
    # out = dict()
    # out['feat'] = x
    # x = self.layer5(x)

    # if not ssl:
    #     x = nn.functional.upsample(x, (h, w), mode='bilinear', align_corners=True)
    #     if lbl is not None:
    #         self.loss = self.CrossEntropy2d(x, lbl)
    # out['out'] = x
    return out`

so....which is the correct size of prototype and if it's 256 how to get the feature~

Hi ShifuShen,
I don't belong to the author team, but I might know the answer for your confusion.
The feature is extracted right before the last layer of Classifier_Module2 in deeplabv2.py. If you look at the forward function of Classifier_Module2, you might see that:
out['feat'] is input of layer nn.Conv2d(256, num_classes, kernel_size=1, padding=0, dilation=1, bias=False)
So it has 256 kernels in the end.

Hi ShifuShen,
I don't belong to the author team, but I might know the answer for your confusion.
The feature is extracted right before the last layer of Classifier_Module2 in deeplabv2.py. If you look at the forward function of Classifier_Module2, you might see that:
out['feat'] is input of layer nn.Conv2d(256, num_classes, kernel_size=1, padding=0, dilation=1, bias=False)
So it has 256 kernels in the end.

Got it  Thanks ~


Switches default zipkin-ui image to use the experimental new UI

This allows users to easily try the new UI, which can help it mature.
See https://github.com/openzipkin/zipkin-ui/issues/10

this build republishes docker-ui as the new code https://travis-ci.org/openzipkin/docker-zipkin/builds/187155633


Added Contribution Guide

Pull Request
Description
Added the first draft of the basis of contribution guidelines, for newcomers. We can keep updating it according to the needs of the organizations.
Fixes #7

@all-contributors please add @roshnaeem  for doc


Attempting to register an unknown factory: Need help

I have my route structure like this:
abc (route)
    |-- index
    |       |-- route.js (extends base.js)
    |       |-- template.hbs
    |
    |-- edit
    |       |-- route.js (extends base.js)
    |       |-- template.hbs
    |
    |-- base.js (extends Ember.Route)

Now when I am trying to write the unit test for my base.js I am getting the following exception:

Promise rejected before it exists: Attempting to register an unknown factory: route:abc/base

What's wrong I have done that it isn't able to register my base class as route.?
If I change my base.js to route.js then it starts working,  but what happen is that the afterModel inside base.js get executed twice when I make transition to abc/index(this is expected behavior). My base class contains code that is required by both of the index and edit route.
Is there any better way to deal with this.?

What are you using base for? I assume it is just for import Base from './base; export default Base.extend()?  If that is true, then you just don't need to add it to needs` at all, since it is not known to the resolver.

Yes I am extending my index and edit from Base.
And have moved duplicate code in base and extended index and edit from that.
May be I should use mixin instead of this Base..?

What you are doing is good, just don't list route:abc/base in needs listing because it isn't a thing known to the registry (its just something you import and extend from).

I haven't added route:abc/base in needs. I am basically writing the unit test for base.js.
moduleFor isn't recognizing base.js as route.

I think I would do the following:
import Base from '../some/path/here';

moduleFor('route:base', {
  integration: true,

  subject() {
    return Base.extend(...arguments);
  }
});

ok let me try this.
Thanks :)

Now getting this on Base.extend
Assertion Failed: Expected hash or Mixin instance, got [object Undefined]

that is probably because you're not passing any arguments to the subject() method when calling it


Project Is Not Building

there are some errors with the classes that should be imported:
'org.gradle.api.Plugin'
'org.gradle.api.Project'

bug-fixed


Error trying to build core-image-sato.

I am trying to build an example image, core-image-sato.
When trying to build, I get this.
pknopf@ubuntu:~/git/tegra-test2/build$ bitbake core-image-sato
Parsing recipes: 100% |######################################################################################################################################################################| Time: 00:00:47
Parsing of 902 .bb files complete (0 cached, 902 parsed). 2242 targets, 230 skipped, 0 masked, 0 errors.
NOTE: There are 3 recipes to be removed from sysroot jetson-tx1, removing...
NOTE: Resolving any missing task queue dependencies
ERROR: Nothing PROVIDES 'virtual/libgl' (but /home/pknopf/git/tegra-test2/poky/meta/recipes-graphics/libsdl/libsdl_1.2.15.bb DEPENDS on or otherwise requires it)
ERROR: tegra-libraries PROVIDES virtual/libgl but was skipped: incompatible with host aarch64-poky-linux (not in COMPATIBLE_HOST)
ERROR: lib32-mesa-gl PROVIDES virtual/libgl but was skipped: incompatible with machine jetson-tx1 (not in COMPATIBLE_MACHINE)
ERROR: mesa-gl PROVIDES virtual/libgl but was skipped: incompatible with machine jetson-tx1 (not in COMPATIBLE_MACHINE)
NOTE: Runtime target 'libsdl' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['libsdl', 'virtual/libgl']
NOTE: Runtime target 'packagegroup-core-x11-sato-games' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['packagegroup-core-x11-sato-games', 'libsdl', 'virtual/libgl']
ERROR: Required build target 'core-image-sato' has no buildable providers.
Missing or unbuildable dependency chain was: ['core-image-sato', 'packagegroup-core-x11-sato-games', 'libsdl', 'virtual/libgl']

It seems that the NVidia libraries provided are only 32bit. Is this correct? What does this layer even support multi-lib then? Why not just compile everything (even then kernel) at 32bit?

I was able to get it working using bitbake lib32-core-image-sato. However, I am getting some xorg driver errors. Something to do with NVidia not supporting XORG eabi version 20, whatever that means.

Any update on supporting 23.2?

I figured out how to work around the xorg ABI problem after I updated to the 24.1 drop - https://github.com/madisongh/meta-tegra/commit/718286340bd4609a527f386accb95c8fb6f200e1
This may work with the 23.2 base as well.
And yes, you have to use the lib32- prefix to get the 32-bit binaries.  For my use, I was mixing 32-bit and 64-bit userland binaries, hence the multilib configuration.
And sorry for not responding to these sooner - turns out I wasn't getting notifications about issues being opened, so I didn't know these were here.

At this point, the L4T R24.1/JetPack 2.2 drop has been integrated, which is fully 64-bit.  I'm not planning on supporting 32-bit kernel or userland binaries going forward.


animations: Climb against vertical slopes

Climbing against vertical slopes appears a bit odd. It needs to be fixed someway by either not allowing the avatar to climb vertically or by adding some animation for vertical climb
https://user-images.githubusercontent.com/35926530/145103206-7568a5e7-6dae-40b7-a089-4372dbc34b8c.mp4
.

Should be fixed with the new character controller.


Add lint option like "max-warnings"

Is your feature request related to a problem? Please describe.
I don't think this is a big problem, but this functionality seems to me very suitable for this package.
Describe the solution you'd like
I would like to have the max-warnings option for the lint command.
https://github.com/Redocly/redocly-cli/blob/main/packages/cli/src/commands/lint.ts#L126
By the link, the place where the script returns 0 or 1, depending on the number of errors:
totals.errors === 0

It would be great to make it look something like this:
process.exit(
  (
    totals.errors === 0 &&
--> total.warnings < argv['max-warnings'] <--
  ) || argv['generate-ignore-file'] ? 0 : 1
)

Describe alternatives you've considered
I had to change the strictness rules from warn to error

We currently have max-problems to set an upper limit on how many are detected; this can be useful if you want to see more or fewer errors (the default value is 100).
If your use case is to let you know if you have more than "X" number of errors, a more targeted approach could be to generate an ignore file for the errors/warnings you're already aware of, so that only new ones will show in the linting output. (docs for ignore file: https://redocly.com/docs/cli/commands/lint/#generate-ignore-file).
Would either of these cover your use case? If not, can you say a bit more about what you're trying to do?

Now your script returns code 0 when there are no errors and code 1 when there is at least one error. I would like the script to return 1 when the "max-warnings" option is exceeded

In our ci, the check is considered not passed if the script response code is not 0. And we want to make the openapi linting fall even if there are warnings

I think I understand the question better now, thanks for the extra information. We adopted industry-standard behaviour of only exiting with an error code if there was an error - any warnings are shown in the output. I think you have a couple of options to achieve what you want:

check the output of the tool, parse the line with the number of warnings/errors and apply your own logic
the other option is what I think you already chose - making the warnings into errors since they are an actual problem

I'll close this since I don't think there's more to for us to do on this, but do let us know if you need anything else.


Extend max Dart sdk range to <4.0.0

Hey, Can we extend the maximum Dart SDK range to support all the Dart 3 versions?
Currently, we support till 3.0.5, and the current stable latest is 3.0.6. It's also hard for you to update every time a new SDK version comes.
https://github.com/syncapod/tart/blob/f270fa6725dd078047378e416b6a771f4ca21caf/tart/pubspec.yaml#L6-L7
The solution:
environment:
	sdk: ">=2.15.1 <4.0.0"

@sschwartz96 can you please take a look?

Hi @xsahil03x, I will get this fixed over this weekend!

Hey @sschwartz96, any update?

Closed with #12


Components hidden by default, such as mobile navigation menus, get cached and cause flickering when navigating between pages.

In a scenario where a page has a component menu using {show: false} and x-show="show", when clicking on a link from that menu, turbolinks caches the page with the menu visible. This cause flickering when visiting the page afterwards unless disabling turbolinks cache.
For example page A -> navigating to page B (page A is cached with the menu open) -> navigating back to page A (Turbolinks temporary shows the cached version with the menu open first for half a second causing flickering).

I'm using something like this in my project and can PR it if you think it's a good approach
document.addEventListener('turbolinks:before-render', (event) => {
  event.data.newBody.querySelectorAll('[x-cloak]').forEach((el) => {
    el.setAttribute('was-cloaked', '');
  });
});

// Cloak elements again before caching
document.addEventListener('turbolinks:before-cache', () => {
  document.body.querySelectorAll('[was-cloaked]').forEach((el) => {
    el.setAttribute('x-cloak', '');
    el.removeAttribute('was-cloaked');
  });
});

This depends on using the x-cloak attribute, but that seems to be common practice for cases like this.

@jduff It seems to be a sensible approach, thanks for sharing.
Does it work on the very first visit when the page is loaded by the browser or do we also need to cover that case as well?
Happy to accept a PR, we can build on that if we need to.
Can we prefix the custom attribute with data-alpine- please?

Does it work on the very first visit when the page is loaded by the browser or do we also need to cover that case as well?

It doesn't look like it's running on the first page load. I've been trying to get that to work from a turbolinks:load event but it looks like Alpine has already run at that point and removed the x-cloak attribute. I came across the deferLoadingAlpine function here which seems to get it to work.
So something like this:
window.deferLoadingAlpine = (startAlpine) => {
  document.addEventListener('turbolinks:load', () => {
    document.body.querySelectorAll('[x-cloak]').forEach((el) => {
      el.setAttribute('was-cloaked', '');
    });

    startAlpine();
  }, {once: true});
};

This sets up the turbolinks:load in the defer callback to only execute once, it then does the x-cloak magic and once that's done it starts Alpine.
Does that make sense or is there a better way to do this?

do we need to wait for a turbolinks:load event? I assume it could just run.
deferLoadingAlpine is used by other libraries so I don't think we should just override it.  We need something like
const init = window.deferLoadingAlpine || ((callback) => callback())
window.deferLoadingAlpine = (callback) => {
    document.body.querySelectorAll('[x-cloak]').forEach((el) => {
      el.setAttribute('was-cloaked', '');
    });
    init(callback)
}

To be fair, since Alpine waits for the DOM ready events anyway and the javascript engine runs as a single thread, I suspect it could just go inside the init function (https://github.com/alpinejs/alpine-turbolinks-adapter/blob/master/src/bridge.js#L15) without involving events.
init() {
    document.body.querySelectorAll('[x-cloak]').forEach((el) => {
      el.setAttribute('was-cloaked', '');
    });

   //etc etc
}

Any thoughts?

That looks like it could work, I'll put together a PR tonight.
I was using the turbolinks:load event as an alternative to ready. I think we still want to have it happen after dom ready though, don't we? As far as I can tell init is called right away and it's possible the DOM isn't fully loaded yet, or am I missing something here?
I think either way I need to put together a PR and we can continue the discussion there :).

The idea is that the script is defered so init should run before domready but when all the html is there. I guess we can't force user to defer the script (expecially if they create a bundle by themselves).
I'm happy with the deferLoadinAlpine if it works.
Bonus point if we can write a test but it's not a blocker for this PR.

@jduff Unfortunatelly the change conflicts with the way Livewire work (Livewire is not deferred and the order of events is not correct).
I've got a new version at<EMAIL_ADDRESS>Do you have any use cases I can use to test it works correctly?
@zepfietje can you confirm it fixes your issue as well?

@SimoTod I just gave it a test in my project and it seems to be working correctly. I'm pretty sure the first test I added, https://github.com/SimoTod/alpine-turbolinks-adapter/blob/master/tests/integration/cloak.spec.js#L4, failed before I added the defer loading so if that's passing I think it covers the initial load case that I added the deferloading for.
I ran a quick test in my project with the script itself bother deferred and and not deferred and it worked in both cases for me.

Yeah, your PR works in normal cases but in a project using Livewire they both tries to defer alpine using an event but they don't work well together. Ready state interactive should trigger before domready (so Alpine starting) but after the whole dom has been downloaded. Hopefully it will serve our purposes. I need to do some cross browsing tests over the weekend but I'm glad it looks like it could work. The workaround for livewire user for now is to remove the defer attributes from the adapter library.

@zepfietje can you confirm it fixes your issue as well?

Seems to fix my issue!

Glad it solved it (again). It gets complicated when livewire is involved and I can't have unit testa for that. I still need to check how it behaves in ie11 and edge but I should be able to tag 0.2.1 tomorrow or Monday. 👍

Closing as it was fixed in 0.2.0

The merge #13  breaks the responsive x-cloak functionality. As I see it, the x-cloak="mobile" would be replaced by x-cloak="". Is there a way to opt out on this and write my own custom solution?

Thanks @heyfadii, can you send me an example please? X-cloak is a boolean attribute and it does not support the syntax you posted if i remember correctly so you cannot decide to apply it conditionally based on a value.

Sure. What I want to do is hide something conditionally, depending on the resolution. I found this example: https://dev.to/hus_hmd/alpine-js-responsive-x-cloak-3blp

Gotcha. Yeah, it's not the traditional use of x-cloak but i can support it, no problem. It will likely be tonight or tomorrow.

@heyfadii Can you confirm that<EMAIL_ADDRESS>does what you need? Thanks.

@SimoTod Sorry for the delay. Yes works! when can this be expected to be merged? Thank you for being lightning fast!

@heyfadii tonight/tomorrow.


Intermittent errors with HRESULT 800705b4

We're running into errors with our Docker images that are using ServiceMonitor.exe as an ENTRYPOINT.  The last thing in the Docker logs is:
ERROR: Failed to stop or query status of service 'w3svc' error [800705b4]
Any ideas what would cause this?  Is there a suggested work around?

Perhaps helpful: This seems to occur when you modify the contents of web.config right before you call ServiceMonitor.exe.  It might be some race condition since it doesn't always happen.

If I tail the logs in Docker, here's what I see:

So basically we transform some web.config, which causes the app pool to recycle.  We then start ServiceMonitor.exe, which can't query the status of w3svc and immediately exits, causing the container the exit.  This creates an endless loop in Service Fabric, since it then tries to restart the container again.
Expected: ServiceMonitor.exe should handle recycling app pools correctly.

Another annoying thing is when ServiceMonitor.exe exits with this error, it still returns an EXIT code of 0 so we can't trap the error and retry with a script.
Expected: $LASTEXITCODE should non-zero when the program exits with an error.

@MikeChristensen We'll look into this. Thanks for reporting. You can expect a response soon.

@MikeChristensen Would you make a simplified dockerfile, which is reproducing the issue and then share it so that I can reproduce the issue and then fix the issue?
I'd like to inform that the error 800705b4 you reported means that there was a timeout error.
ServiceMonitor uses 20 seconds timeout for retrying (3~4 times) to get the service status after restarting w3svc. If it fails to get the service running state in 20 seconds, it returns the timeout error.
There must be some reasons why it took longer time (more than 20 seconds) to make w3svc service being restarted, which causes the failure of ServiceMonitor.
Thanks,
Jeong Hwan Kim
Microsoft

@MikeChristensen BTW, I assumed that you ran into the issue when starting container. Thinking again, I might be wrong about that. Would you describe more detail? If you can give the repro steps, that will be very helpful for me to understand how the error can happen.

ServiceMonitor stops w3svc service first to update applicationhost config with the environmental variables first and then restart the service. When container starts, it also starts the all services. There could be a timing issue that entry program (stopping service) was executed before Service started. To work around it, could you please try adding "net stop w3svc" in your docker file.

you may also need to reset w3svc start type to manual in docker file, i.e., sc config w3svc start=demand

I've confirmed this workaround solves the issue.  Basically, set the startup behavior to demand in the dockerfile, and set the entrypoint to transform the config files then run ServiceMonitor.exe (which will also start the service).

what does this actually mean?

Basically, set the startup behavior to demand in the dockerfile, and set the entrypoint to transform the config files then run ServiceMonitor.exe

What do we have to do to fix the issue?

The first set means to add "RUN sc config w3svc start=demand" as the below example dockerfile and the second one to replace the last line ("ENTRYPOINT ["C:\ServiceMonitor.exe", "w3svc"]) with calling your own  batch file which does the configuration job and put "c:\servicemonitor w3svc" at the end of the batch file.
FROM mcr.microsoft.com/windows/servercore/iis:latest

RUN powershell -Command Add-WindowsFeature Web-Server
RUN sc config w3svc start=demand
EXPOSE 80
ENTRYPOINT ["C:\\ServiceMonitor.exe", "w3svc"]

That's perfect! Thanks for the example

I'm getting the following error with this example:
A positional parameter cannot be found that accepts argument
Any ideas?

Fixed the postional parameter issue with the following:
RUN powershell -Command Set-Service -Name aspnet_state -StartupType 'Manual'

However I still have the ServiceMonitor issue - it keeps preventing a container starting and then just randomly recycles several times.  I have other containers with the same setup that work fine for a bit and then will crash with the same error later down the line.
Someone suggested updating Service Monitor but again gave no instructions

Do you have the docker log for the error case?

Task Log: ERROR: Failed to stop or query status of service 'w3svc' error [800705b4]

I think you used the wrong service name (aspnet_state).  The right service name is w3svc as the following:
RUN powershell -Command Set-Service -Name w3svc -StartupType 'Manual'

800705b4 means timeout. It seams ServiceMonitor had trouble to access W3SVC service. Did you set W3SVC service to manual start mode in docker file? you said you had the same setup work fine for a bit and then will crash later,  could you please give more details? Could you please check whether your container image is compatible with the host OS?

Correct the start service to w3svc and startup type to manual and the service now just recycles over and over with nothing in the logs.  It looks like the  w3svc isn't starting and keeping the container open
Docker file:
FROM mcr.microsoft.com/dotnet/framework/aspnet:4.7.2-windowsservercore-ltsc2019 AS base
WORKDIR /inetpub/wwwroot
EXPOSE 80
FROM mcr.microsoft.com/dotnet/framework/sdk:4.7.2-windowsservercore-ltsc2019 AS build
WORKDIR /app
copy csproj and restore as distinct layers
COPY . .
RUN nuget restore ./Website.csproj -SolutionDirectory ./..
build app
RUN msbuild ./Website.csproj /p:Configuration=Release /p:DebugSymbols=false /p:DebugType=None /p:DeployOnBuild=true /p:PublishProfile=FolderProfile.pubxml
FROM base as runtime
ARG source
COPY --from=build /app/bin/Release/Publish/. ./
RUN powershell -Command Set-Service -Name w3svc -StartupType 'Manual'
COPY docker-entrypoint.ps1 /
ENTRYPOINT powershell -File C:\docker-entrypoint.ps1

I assume you tried to build you project and publish it inside container to IIS and then access app. Could you please check whether the build succeeded and content was dropped correctly? Could you please share docker-entrypoint.ps1  so that we can make sure ServiceMonitor is in the picture? If possible, please open another issue as your problem seems not related with ServiceMonitor itself.


cw20-merkle-airdrop add possibility to withdraw remaining tokens after expiration

after expiration owner can withdraw leftover funds for future usage

Looks good, but just some formatting issues: cargo fmt.


Federico: Watch socialwander.com

Federico requests the watch of the watch_keyword socialwander.com. See the Metasmoke search here and the Stack Exchange search here.

Coverage remained the same at 63.155% when pulling 8c81eb162a52f18814f517b867804c9e32254bd4 on auto-blacklist-1512487922.9234157 into ba69049c085bfff490a32ca3f11b83a18be699fe on master.

!!/approve


Convert build to use stack

I deleted all the old nix stuff and updated the README and the Vagrant setup. The overall setup is much simpler, and everything seems to work great. I believe it would now be trivial to have a Travis job for the project, but I haven't tried to hooking that up.

I had Travis working with Stack here: https://github.com/joshcough/Scrabble, but do get intermittent build errors for reasons I haven't figured out yet. The stack documentation for Travis is really easy, too.

Yup. I was never a big fan of Nix, but for a while it seemed to be the only way to use Reflex with GHCJS. Now that there's this example for using Stack, I didn't see any point in sticking with Nix.

Hmm. I remember you claiming to be a big fan of it for a while :) But, I'm glad to see you decided not to use it, because I was about to try to copy your example!

Ha, you can't trust anything I say obviously. :) I do still like the general idea of Nix quite a bit (and Unison uses some of those same ideas). But the devil is in the details.

Yeah, not going to lie the old state of things was not Nix at its finest. I am trying out some things in at https://github.com/forked-upstream-packages-for-ghcjs and https://github.com/Ericson2314/nixpkgs/tree/ghcjs which will hopefully improve the state of ghcjs for Nix and Stack alike---ghcjs-boot is terrifying, and the try-reflex-stack doesn't solve the problem of ghcjs and ghc library versions not quite aligning.

@Ericson2314 cool. The warnings about ghc library versions not aligning are kind of janky I agree. But it does still work okay and the setup is very simple so I'm not complaining. :)


fix(guide): Remove repl.it links from challenge related guide articles (Chinese)

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Related to issue #35479
This PR removes all repl.it Run code links from the Chinese challenge related Guide articles.

Thank you @RandellDawson.


Broken link

Hi,
trying to open http://www.mkonrad.net/projects/ogles_gpgpu.html leads to redirection to incorrect address https://mkonrad.netprojects/ogles_gpgpu.html. I guess something wrong with site configs (?)

Thanks for reporting, that was a mistake in the HTTP to HTTPS redirection config on my server...

Thanks, works fine now


Frame graph

Sorry, monster PR!
Warning: Everything is experimental at this stage! Everything may/will change before version 8.0 is released, so be warned that your code may break without notice if you use this new feature.

Reviewing the PR
There's still a lot to do, but I had to deliver a first version before continuing, so that the existing code could be reviewed and team members could work on extending the framework.
As the PR has over 18,000 changes, I think I should point out the most important code to review:

I had to make a number of changes in the core engine to properly handle multiple rendering targets. These are the files under Engines/.
The frame graph framework:

everything in the FrameGraph/ directory
the two files under packages/dev/gui/src/2D/FrameGraph/ and packages/gui/src/2D/advancedDynamicTexture.ts.

The new architecture of post-process classes:

The files PostProcesses/ThinXXXPostProcess.ts and PostProcesses/ThinXXXEffect.ts implement the core of post-processes / rendering effects. Note that only a few post-processes / rendering effects have been ported to the new architecture for the time being.
EffectWrapper has been extended to be the base class used by the new ThinXXX classes: Materials/effectRenderer.ts.

Apart from these files, there have been many changes in the configuration files to support the new NRGE (Node Render Graph Editor). Of course, there's also the NRGE itself (packages/tools/nodeRenderGraphEditor/) if you'd like to take a look!

Quick instructions to get started
To use a frame graph to render a scene, you need... :

set scene.frameGraph to the frame graph
set scene.useFrameGraph = true.
When a frame graph is used, it will use the new scene.customRenderFunction to render the scene.

This means that a lot of code is no longer executed (scene._evaluateActiveMeshes, scene._renderForCamera, many observables are no longer notified, etc).
A number of features are currently broken because of this (frozen meshes, snapshot rendering, and others). We'll be fixing this over the coming months.

Important: also set scene.cameraToUseForPointers to a proper camera!

There is no longer a “current active camera” when using the frame graph for rendering, which would last for the duration of scene.render and is normally used as the “camera to use for pointers” (scene.activeCamera is still used - a lot of code depends on it - but only when running RenderTargetTexture.render).

You can create a frame graph in 3 ways:

using the FrameGraphXXX classes: http://localhost:1338/#2VHZFE#3
by importing a node render graph from NRGE: http://localhost:1338/#T6F4ZE#55 (the node render graph: http://localhost:1344/#PSA9PS#42)
create it using the NodeRenderGraphXXX classes, by manually connecting the nodes => probably won't be used much

Note that you can also use frame graphs without setting scene.frameGraph / scene.useFrameGraph parameters: you can simply call FrameGraph.execute yourself when you want a graph to be executed.
In this example, a pass post-process is used to render the scene in a texture, which is then used by the frame graph to apply a number of effects (black and white, bloom effect, GUI).

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

LGTM once the geometry helper call is made a bit more efficient as discussed offline

Done!

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Snapshot stored with reference name:
refs/pull/15735/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15735/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15735/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15735/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15735/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15735/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/testResults/webgpuplaywright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Reviewer - this PR has made changes to one or more package.json files.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15735/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Reviewer - this PR has made changes to one or more package.json files.

Reviewer - this PR has made changes to one or more package.json files.

Should we merge the PR?

Yes


How do I create an empty directory?

This is what I tried:
curl -X PUT "http://localhost:8888/test/"
=> {"error":"can not to write to folder /test/ without a file name"}
curl -X POST "http://localhost:8888/test/"
=> {"error":"request Content-Type isn't multipart/form-data"}
curl -X PUT "http://localhost:8888/test"
=> creates a 0 byte file
curl -X POST "http://localhost:8888/test"
=> {"error":"request Content-Type isn't multipart/form-data"}

currently not possible.

Added a commit to support
curl -X POST "http://localhost:8888/test/"


EIA_API_KEY in GitHub Actions

The EIA_API_KEY is blank in the GitHub actions environment, causing four of the test_eia tests to fail.
This can be recreated locally:
export EIA_API_KEY=

pytest gridstatus/tests/test_eia.py

FAILED gridstatus/tests/test_eia.py::test_list_routes - KeyError: 'response'
FAILED gridstatus/tests/test_eia.py::test_rto_interchange - KeyError: 'response'
FAILED gridstatus/tests/test_eia.py::test_rto_region_data - KeyError: 'response'
FAILED gridstatus/tests/test_eia.py::test_fuel_type - KeyError: 'response'

Setting a valid value for EIA_API_KEY allows the build to pass locally.
Secrets are not shared with PRs from forked repos unless we use the pull_request_target trigger in GitHub actions (I think based on an explanation. So, my recommendation would be to change the pull_request to pull_request_target in ./.github/workflows/tests.yaml.

Or, if the key doesn't need to be kept secret we could set it as a repo variable. I don't think that's a high security risk since we aren't paying for the key.

we can make it a repo variable

Addressed in #321 by putting the API key directly into the GitHub actions file since the key does not need to be secret.


feat: add initial workflow status in config

Summary
closes #1306
For our case we now use the editional workflow,  we use it to preview our changes, but sometimes we just want to publish directly.
And this is not possible, we have to save, change status to ready and publish.
I would have preferred to do this issue #4054 but it seemed more complex for a first issue on the project :)
The goal here is to add the initial status in the configuration, then in our cas we can put PENDING_PUBLISH status by default and then publish directly.
Test plan

Checklist
Please add a x inside each checkbox:

[x] I have read the contribution guidelines.
[x] Code is formatted via running yarn format.
[x] Tests are passing via running yarn test.
[x] The status checks are successful (continuous integration). Those can be seen below.

I don't really know where to add the expected test, if you can guide me :)

Hello @bytrangle, thanks for your review.
I tried when I start the backend, and everything work right.
But inside test I can't find where I can test this behavior, in the file that you have pointed out to me it doesn't work because it's looks like it's not in "editorial_workflow".
I tried here :

https://github.com/netlify/netlify-cms/blob/dbf2920254fb3682e12463a6df8ded4b94b55be0/packages/netlify-cms-core/src/__tests__/backend.spec.js#L347

And here :

https://github.com/netlify/netlify-cms/blob/cb2ad687ee250937a9dffc21ebfd4c399c6d76db/packages/netlify-cms-backend-test/src/__tests__/implementation.spec.js#L48
But we can only access to the window.repoFiles where there is no the entry workflow state.

Can you give me a little help please?

On further investigation, I think the test should be in netlify-cms-core/.../actions/__tests__/config.spec.js.
I got the idea from the test "should set publish_mode from config". I've tried create a barebone config object with two props: publish_mode and default_workflow_status.
Then I called applyDefaults(config). This function takes the original config object parsed from config.yml file and mutate it so that certain important properties won't be undefined if the user forgot to add them. For example, config.publish_mode has to be SIMPLE_WOKFLOW by default.
Then I check if applyDefaults(config).default_workflow_status is equal to the value I passed to the namesake property of the config.
You'll also need to modify the applyDefaults(config) so that if the user sets publish_mode to editorial_workflow, and don't specify the default_workflow_status, the mutated config will have a property default_workflow_status set to draft.
Does that make sense?

Oh nice ! It's really clear, I will do that :)
Just for your last sentence, you really think it's useful with the following code ?

https://github.com/RomainOliva/netlify-cms/blob/e89358a11101377de3c0ca3eb7187a3cf8b545e1/packages/netlify-cms-core/src/reducers/config.ts#L38

With this line if there is no default_workflow_status it take the first status (draft one)
thanks again for your help

@bytrangle I just add a fixup commit with some change and the test, tell me if this sounds good to you :)

@krider2010 Look forward to your review.

Small up please, this feature would allow us to save a lot of time in our processes 🙈

@bytrangle Please 🙈

It is still relevant today 🙈

It is still relevant today see_no_evil

waiting for this to happen

@RomainOliva are you still interested in moving this forward?

Great! Can you solve the merge conflicts? They mostly emerged because of the package rename.

@martinjagodic  sorry for the delay, it's all good the PR is rebased :)


Unresolved critical extensions issue in PkixCertPathChecker

When overriding Check method of PkixCertPathChecker.cs I can't remove unresolved critical extensions I don't want to check.
Reson: unresolvedCritExts object is immutable ICollection.
Fix: It could be of type Org.BouncyCastle.Utilities.Collections.ISet, this way I can call Remove() method on collection.

Addressed by #35.


docs: update banner links

@shati-patel and @felicitymay: this should be the last change required to the existing CodeQL docs.

Preview: https://jenkins.internal.semmle.com/job/Docs/job/Generate-Sphinx/926/artifact/target/sphinx/learn-ql/index.html

Thanks. Looks great 😄


Update AppCmdOnTargetMachines.ps1

Bug: Given binding already exists for a different website.
function Does-BindingExists: added comma to the end of the binding string which falsely identified a used binding from another site that started with the same string.

Done

Thanks for fixing all the comments.
BTW: you can keep updating same PR by doing new commits to the branch on which you opened PR


New signup methods

I tried to add NoSignupAllowed and the coupled signup described in https://github.com/ephios-dev/ephios/issues/515#issuecomment-883290221.
Beware that this is not branched off of main.

Coverage decreased (-0.2%) to 90.636% when pulling 3e4a3d844627f4d83655df93b837f15fb4429eaa on new_signup_methods into 4071af406f77ede4d1fc430cd816cb0f81cb2438 on main.


feat: updated open source page copy

Changes to copy only.

Looks good, just please fix the lint for the max line length.

Can you do that for me, please? I know what you're talking about, but not how to do it.


Use prop-types and createReactClass to fix deprecation.

Was having issues testing this.  Using yarn link doesn't seem to work as it complains about missing dependencies.  Seems to do with some webpack step that isn't working.  Figured you might know.

Tested that this works in react-data-viz. Accepting.


Release 4.0.0

I would like to include https://github.com/voxpupuli/puppet_metadata/pull/129 as well for the release


Missing example output in README or docs

It would be useful to have an example of output to demo the project.
This looks like a great idea, thank you for sharing.

It has been done! Thanks for the good idea :).


Remove 'CSS @scroll-timeline'

The spec has changed and this at-rule no longer exists.
Fixes #6906.

Maybe this should instead be renamed to "Scroll-driven animations"?

Yeah I think it would make more sense to just rename/update this to reflect the spec's changes.

I added a new JSON file. No support data filled in. Maybe some of the previous links should be preserved.
Demos: https://scroll-driven-animations.style/#demos

This PR may be closed as the feature was removed in https://github.com/Fyrd/caniuse/commit/0a84cfd3b2b3abb650787abdc39faf382800e8cd


FIX: make deploy does not instantiate the operator

Closes: #11

This was a tricky issue:
$ docker run -it --entrypoint /manager kubestone:latest
standard_init_linux.go:207: exec user process caused "no such file or directory"

$ docker run -it --entrypoint /nonexistent kubestone:latest
docker: Error response from daemon: OCI runtime create failed: container_linux.go:344: starting container process caused "exec: \"/nonexistent\": stat /nonexistent: no such file or directory": unknown.

$ docker run -it --entrypoint /bin/sh kubestone:latest
/ # ls -l /manager
-rwxr-xr-x    1 root     root      40603890 Aug  4 05:10 /manager

/ # /manager
/bin/sh: /manager: not found

/ # /lib/ld-musl-x86_64.so.1 /manager
2019-08-04T05:34:54.203Z    ERROR    controller-runtime.client.config    unable to get kubeconfig    {"error": "could not locate a kubeconfig"}
github.com/go-logr/zapr.(*zapLogger).Error
    /go/pkg/mod/github.com/go-logr/zapr@v0.1.0/zapr.go:128
sigs.k8s.io/controller-runtime/pkg/client/config.GetConfigOrDie
    /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.2.0-beta.4/pkg/client/config/config.go:143
main.main
    /workspace/main.go:75
runtime.main
    /usr/local/go/src/runtime/proc.go:200

The problem is caused by this change: https://github.com/xridge/kubestone/pull/4/files#diff-3254677a7917c6c01f55212f86c57fbfL18-R19
By removing CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO111MODULE=on the created binary became a dynamically linked one. The builder image had a gnu libc, while the final one was shipped with musl (via alpine). This setup caused the odd standard_init_linux.go:207: exec user process caused "no such file or directory" error.
Adding back the necessary flags for the build instructs the compiler to create a statically linked binary.


Update dependencies and to 2020e

Resolves #27 and some others along the way

The auto-updater failed for some reason, but will look into that separately

I don't know what killed the emulator

I don't know what killed the emulator

Filed an issue here: https://github.com/ReactiveCircus/android-emulator-runner/issues/118

Filed an issue here: https://github.com/ReactiveCircus/android-emulator-runner/issues/118

Will review this tomorrow

Will review this tomorrow

Let me regenerate just in case

Let me regenerate just in case


Error when used with Jupyter Book: source_suffix '.ipynb' is already registered

Running this extension with Jupyter Book (a custom sphinx build), which already has ipynb registered as a file type, generates an error when building a book:
Extension error:
source_suffix '.ipynb' is already registered

Also described here: https://github.com/executablebooks/jupyter-book/issues/1344#issuecomment-1062544017
Offending line is presumably: https://github.com/jupyterlite/jupyterlite-sphinx/blob/f14181aac6772b285f0dbd8616c33a5db742c3c7/src/jupyterlite_sphinx.py#L274

Thanks for reporting. I wonder what should be done for fixing this.
We could either use the override option (see https://www.sphinx-doc.org/en/master/extdev/appapi.html#sphinx.application.Sphinx.add_source_suffix) to force using jupyterlite-sphinx suffix handlers, or we could back off if it's already registered.
I guess the second option is best. The end user can override that in their configuration anyway (see https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-source_suffix).
So maybe something like
try:
    app.add_source_suffix(".ipynb", "jupyterlite_notebook")
catch ExtensionError:
    # Show a warning explaining how to force "jupyterlite_notebook"?
    pass

Yeah, I was wondering if there was a way of looking up already registered source suffixes, other than raising an exception if you try to re-add one, but I can't spot anything from a quick look at the docs.
The override option is new to me (I wonder if there are any side effects from re-registering a suffix?).

Looking at the Sphinx code, we could probably do:
if '.ipynb' not in self.app.registry.source_suffix:
    app.add_source_suffix(".ipynb", "jupyterlite_notebook")
else:
    # Show a warning explaining how to force "jupyterlite_notebook"?
    pass

We can also probably do just like nbsphinx does https://github.com/spatialaudio/nbsphinx/blob/64cd5a4ddadc8fb03887f3fac821cdfd184d6246/src/nbsphinx.py#L1889-L1890

Actually what nbsphinx does does not work and collide with jupyterlite-sphinx, so I guess my proposition above is safer.


Theme broken with new Discord update

Only messages have the theme color, everything else is mostly the default theme with this theme's font. Messages also have broken profile pictures
To Reproduce
Install theme (tried with Vencord)
Screenshots

Infomation (please complete the following information)
Discord channel: Stable
OS: Windows
Mod: Vencord
Discord language: English
Additional context
Apparently a Discord update broke pretty much every theme. Seems to be affecting both BetterDiscord and Vencord.

Discord did an absolute massive class reroll. Most if not all of all my themes are broken. I'll fix it when I can/can be bothered.


ability to post GitHub output only on failures

It would also be great to have a way to report the output differently in case of "failures".
For example, the report as PR comment can be very noisy in a monorepo with multiple projects reporting as PR comments, but if we can set the PR comment to be only in case of "failure", that would already be a lot less noisy.

Originally posted by @tleunen in https://github.com/LironEr/bundlemon/issues/115#issuecomment-1216594565

Each output will accept
"always" / "on-failure" / "off"

default:
"reportOutput": [
  [
    "github",
    {
      "checkRun": "off",
      "commitStatus": "always",
      "prComment": "always"
    }
  ]
]

@tleunen what do you think?

Exactly what I had in mind, and you kept true/false in your PR to be backward compatible so this is great.
Thank you!

Feature available in<EMAIL_ADDRESS>


adding ira_laser_tools

I'd like ira_laser_tools to be indexed and documented on ros.org

This is failing CI because it's out of alphabetical order.

please create a separate PR for the rosdep key addition providing the information listed in the rosdep rule contribution guidelines

okay I do it, thanks

@pietrocolombo sorry for the delay on this. This is now conflicting, can you please rebase this branch for it to be merged? thanks!

Same as: https://github.com/ros/rosdistro/pull/18533#issuecomment-411901933

I rebase this branch now you can merge
thanks!

Github doesnt allow to reopen a PR if the branch has been force-pushed. Can you please open a new PR with this change?


[nri-metadata-injection] Webhook timeout can block Digital Ocean cluster upgrades

Bug description
Digital Ocean's clusterlint tool recommends setting webhook timeouts to be 1 to 29 seconds in order to avoid blocking automated cluster upgrades. The current default is 30 seconds.
Version of Helm and Kubernetes
I'm using flux2 v0.17.2 on Kubernetes 1.20.15-do.1, but I'm not sure the versions matter as this problem affects Digital Ocean Kubernetes clusters of all versions.
Which chart?
nri-metadata-injection v1.1.1 (technically, I am using bundle version 3.5.0)
What happened?
I ran the "operational readiness check" feature on my production Digital Ocean cluster in preparation of a Kubernetes version upgrade after installing the NRI bundle chart with mostly default values, and I got this error:

Mutating webhook with a TimeoutSeconds value smaller than 1 second or greater than 29 seconds will block upgrades.

Screenshot:

The other issue in the report is the use of hostpath volumes when the privileged flag is enabled, but that one is marked as warning only (and it's probably unavoidable given the nature of the integration).
What you expected to happen?

Please avoid saying "It should just work".

Well... it should ;) I should think changing the default value from 30 to 29 would be minimally disruptive and would save Digital Ocean users like myself an extra step.
How to reproduce it?

Here's my flux helm release entity:
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: newrelic-bundle
spec:
  releaseName: bundle
  chart:
    spec:
      chart: nri-bundle
      sourceRef:
        kind: HelmRepository
        name: newrelic
        namespace: flux-system
      version: 3.5.0
  interval: 1h0m0s
  install:
    remediation:
      retries: 3
  # Default values
  # https://github.com/newrelic/helm-charts/blob/master/charts/nri-bundle/values.yaml
  values:
    # Default values for nri-infrastructure.
    # This is a YAML-formatted file.
    # Declare variables to be passed into your templates.
    # enables newrelic-infrastructure
    infrastructure:
      enabled: true
    # enables nri-prometheus
    prometheus:
      enabled: false
    # enables nri-metadata-injection
    webhook:
      enabled: true
    # enables kube-state-metrics
    ksm:
      enabled: true
    # enables nri-kube-events
    kubeEvents:
      enabled: true
    # enables newrelic-logging
    logging:
      enabled: false
    # enables newrelic-pixie
    newrelic-pixie:
      enabled: false
    # enables pixie-chart
    pixie-chart:
      enabled: false
    # enables newrelic-infra-operator
    newrelic-infra-operator:
      enabled: false
    # enables newrelic-k8s-metrics-adapter
    metrics-adapter:
      enabled: false
    # IMPORTANT: the following values are set as global so that they
    # can be shared by other newrelic product's charts
    #
    #   - Specify either the New Relic keys or the secret which
    #   contains them.
    #
    #   - Specify the Kubernetes cluster name.
    #     https://docs.newrelic.com/docs/kubernetes-monitoring-integration
    #
    global:
      lowDataMode: true
      licenseKey: ENC[REDACTED,type:str]
      #   insightsKey:
      #   customSecretName:
      #   customSecretLicenseKey:
      #   customSecretInsightsKey:
      cluster: REDACTED
    #   # IMPORTANT: Set this to true when deploying in EKS Fargate. This will pull the sidecar injector operator chart to
    #   # automatically monitor fargate workloads, and also prevent the DaemonSet from being scheduled in Fargate nodes
    #   fargate: true
    # It is possible to configure dependencies values from this chart.
    # For example you can configure `verboseLog` variable of the newrelic-infrastructure chart installed as a dependency and enable openshift configs as follows.
    newrelic-infrastructure:
      privileged: true
      resources:
        limits:
          cpu: 100m
          memory: 256Mi
        requests:
          cpu: 10m
          memory: 64Mi

You can reproduce my exact screenshot by installing the chart with these values on any Digital Ocean Kubernetes (DOKS) cluster and clicking on the operational readiness check in the Digital Ocean console.
Alternatively, you can run clusterlint directly... although this check is a DOKS-specific issue.
Anything else we need to know?
I know it's an easy fix, but happy to send a PR. Not sure it makes sense for me to sign a CLA for a one line change that I can easily work around on my side, hence the bug report.

Hi @ianonavy,
If I understood you correctly, the only issue you have is with the timeout of the MutatingWebhook.
The timeout of the MutatingWebhook can be modified changing your values.yaml.
Redeploying changing that value should be enough to cover your needs. Did that solve your issue?

@kang-makes Yes, I've already done that. Not looking for support—this is a product suggestion to change the default to 29.

With a little bit of investigation, I saw you are completely right. It is not an issue only with Digital Ocean but with every Kubernetes cluster
https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/#timeouts

The timeout value must be between 1 and 30 seconds.

However, we are moving these helm charts (and virtually as many charts as we can) to the repository where the source code rests and we are deprecating these charts. So yes, this is an issue, we have to solve it and if you want to create a PR solving it it should be done here.
The charts are going to be changed during the next weeks.
We appreciate you taking the time to bring this to our attention.

Solved in newrelic/k8s-metadata-injection#171
Closing


Fix text shadows in the blueprint theme

The new blueprint theme adds shadows to text on the sidebar and navbar. This gives the text a blurry feel which is stressful on the eye (at least for me). The task is to either fix this or choose a theme which doesn't have accessbility issues.

Surely we can modify the CSS to  remove the shadows?

claim

propose PR #646


hugo server --appendPort=false stopped working in Hugo 0.31

Also, seems like the baseURL from CLI in hugo server is also not working as expected in "single host mode". It behaves correctly if you add the --bin option.

For baseURL the new rule will be:

If multihost, baseURL from command line is not supported
Else: Use that value (like before)

Thanks, I tried v0.31.1 and have no issue anymore !


Add Resource Tab & sub-organize via tabs

organize the content under Sub-tabs to be consumed will less words:

e-books

white papers

case studies

Conference/JUGs MP abstracts available for download

MP presentation template (at least 2 templates)

Video sessions recorded from past MP presentations added under Conference sub-tab

@aeiras They do not have their own pages to take them to in a sub menu.
These pages would need to be created to do this.

Lets add that as a new feature for Resources directly in the skelleton doc.
we won't work in adjutments on MP.io yet. Closing ticket. :) thanks for this follow up Ryan.

@rstjames lets re-open this ticket and further updates, drop them here :)
One thing is that Resources page is a space for:

Coders/MicroProfilers

Users/Clients

In some way this page must accomodate the two groups that at times are not the same people.

@rstjames we re-opened this ticket b/c it matters to get tab improved. Using this ticket, please share the screen shots you have thus far on options. :) show and tell via this channel might be the best form to enable quick feedback by anyone. thanks!

On next Community call, lets tackle the resources tab with an update @rstjames, thanks!

@aeiras I can add the sub page but most of these pages do not have their own page. I can link the menu item to a targeted section on each page if this helps.

@aeiras added a sub menu that links to each section on the page. Should we close this ticket?

@rstjames,
It looks great! Yes  lets close this ticket. Can you own sending the git issue link to the MP forum as a written update on website tab betterment? thank YOU :)

@rstjames,
Lets also add a sub tab link to the MP_Marketing gDrive:

@aeiras  I will try to finish up the MP design recourse page this week and I can add this to the sub sections. That page will have all google drive recourse.

your dedication and speed rock Ryan! thank YOU :)

Nicely updated, thanks Ryan! @rstjames


Palm prompt parsing issue

I think there maybe some issue in palm implementation. I'm using provider "palm:chat-bison-001" but some of my test cases always report API error. The same test cases works with other providers.
[FAIL] API response error: TypeError: Cannot read properties of undefined (reading '0'): {"messages":[{"author":"0","content":"You are an AI software developer. You will be provided with the TypeScript code that the user is working on, delimited by triple hashtags. The user is currently stopped at \"\". Please complete this line of code. Code only, no explanations.\n\n###\nimport Config from \"#config\";\nimport { Configuration, OpenAIApi } from \"openai\";\n\nexport default async function (history:[], instruction: string) {\n const configuration = new Configuration({\n apiKey: Config.OPENAI_API_KEY,\n });\n const openai = new OpenAIApi(configuration);\n\n const messages = await build\n\n}\n\n/*\n* Build messages for OpenAI based on instructions.\n*/\nasync function buildMessages (history: [], instruction: string) {\n switch (instruction) {\n case \"chat\":\n history.unshift({\n \"role\": \"system\",\n \"content\": \"You are a helpful assistant.\"\n });\n break;\n case \"summarize\":\n history.unshift({\n \"role\": \"system\",\n \"content\": \"Summarize user's message within 20 words.\"\n });\n break;\n case \"complete\":\n history.unshift({\n \"role\": \"system\",\n \"content\": \"Complete the following text within 60 words.\"\n });\n break\n case \"code\":\n history.unshift({\n \"role\": \"system\",\n \"content\": \"Complete the following code.\"\n });\n break\n default:\n //do nothing\n }\n return history;\n}\n###"}],"filters":[{"reason":"OTHER"}]}
Test Config:
- description: This test mainly focuses on reasoning-depth. It requires the LLM to understand the whole piece of code and infer that the intent of the user is to call buildMessages function.
  difficulties:
    context-length: 1
    reasoning-depth: 4
    instruction-compliance: 2
  threshold: 0.8
  vars:
    name: 002_code_completion
    prompt: file://testcases/002_code_completion_prompt.txt
  assert:
    - type: equals
      value: "Messages(history, instruction);"
      weight: 1
    - type: contains-any
      value:
        - "buildMessages(history, instruction);"
        - "Messages(history, instruction);"
      weight: 4

Prompt:
You are an AI software developer. You will be provided with the TypeScript code that the user is working on, delimited by triple hashtags. The user is currently stopped at "<cursor_is_here>". Please complete this line of code. Code only, no explanations.

###
import Config from "#config";
import { Configuration, OpenAIApi } from "openai";

export default async function (history:[], instruction: string) {
    const configuration = new Configuration({
        apiKey: Config.OPENAI_API_KEY,
    });
    const openai = new OpenAIApi(configuration);

    const messages = await build<cursor_is_here>

}

/*
* Build messages for OpenAI based on instructions.
*/
async function buildMessages (history: [], instruction: string) {
    switch (instruction) {
        case "chat":
            history.unshift({
                "role": "system",
                "content": "You are a helpful assistant."
            });
            break;
        case "summarize":
            history.unshift({
                "role": "system",
                "content": "Summarize user's message within 20 words."
            });
            break;
        case "complete":
            history.unshift({
                "role": "system",
                "content": "Complete the following text within 60 words."
            });
            break
        case "code":
            history.unshift({
                "role": "system",
                "content": "Complete the following code."
            });
            break
        default:
        //do nothing
    }
    return history;
}
###

I suspect there is some issue when parsing the prompt.

https://github.com/promptfoo/promptfoo/blob/0.24.1/src/providers/palm.ts#L120 is the parsing line: data.candidates[0].content
Looks like maybe it should be data.messages[0].content

@zhlmmc This appears to be an issue with PaLM or chat-bison-001 itself.  The API response does not include the candidates field, which is expected in the generateMessage response.
Using your prompt, I can also reproduce this failure in the Google MakerSuite.  So I don't think it's promptfoo:

I might suggest using vertex:chat-bison@001.  In my experience, the Vertex API seems more robust and it's the same underlying model.


Correct missing single quote

This missing quote in the default _configuration setup was keeping _configuration from loading by the plugin (throws an error)

@jgclark Fixing a bug in the default template creation which caused the config file to not load. May send you some other suggestions, but wanted to get this bug fixed right away.

BTW, ... now I've looked more closely I see this is for the auto-generated script. This will get overwritten next time someone use the build system.
I will as well fix this in the underlying script from @nmn (configuration.js), and I'm in the process of raising a wider PR that will include this.  Please do get involved in that discussion ...


ERROR VSCODE:INSTALL Failed: connect ETIMEDOUT <IP_ADDRESS>:443

code-server version: latest
OS Version: ubuntu 16.04

Description
I got some error message when I use docker build to build "Dockerfile" to docker image.
The build step is:
 yarn && NODE_ENV=production yarn task build:server:binary

And the erroe message:
INFO  BUILD:SERVER:BINARY Starting...
INFO  VSCODE:INSTALL Starting...
ERROR VSCODE:INSTALL Failed: connect ETIMEDOUT <IP_ADDRESS>:443
ERROR VSCODE:INSTALL Stack: Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1097:14)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I have added the http_proxy and https_proxy ENV in front of Dockerfile, but still got this error message.
How can I fix this error?
thx

the problem is that the default node http client doesn't understand http proxy env variables.
Possible solutions are available here: https://github.com/nodejs/node/issues/8381


Create Fermions_in_double_well_qiskit.ipynb

Attached you can find one tutorial on experiments in the Jochim group that I prepared for teaching. I am not sure how it fits into the series of tutorials, but it might be nicely integrated ...

I have now reworked the Qiskit part of the notebook to make full use of the functionalities provided by the package.


Workflow RPC method as request response (update in Temporal)

The semantic is like “update” api in Temporal.
It’s already available in go sdk for early access

Allow locking for persistence field (using validator in Temporal)
allow checking version in persistence field (same validator)


Bump to Electron 0.31.2

Current build wasn't working so I'm bumping yet again.
For some reason though, I'm not seeing output from kernels (nor errors). Work to be done still...

Migrated over to jupyter-js-output-area for dogfooding.
Not terribly happy with the interface here, but it'll do.
/cc @jdfreder

Not terribly happy with the interface here, but it'll do.

Output area's interface?

This was written with JDA, which let you pass messages directly to the <jupyter-display-area> DOM elements. Book keeping needs to happen separately with this.
One thing about output area is the complete lack of Promises, even though underneath transformime is using it. It returns a boolean, which leaves me skeptical.


CreateUser screen progress

submit users now dynamically renders usernames in dropdown as well as render previous users in dropdown based on household

APPROVED


Does the old chain have test nodes?

https://bombay-lcd.terra.dev No longer available

not related to terra.js


Accept vars in distgit > branch values

Substitute variables in distgit > branch declarations.
distgit:
  branch: foo-{VAR}

/lgtm


When using remotely via p11-kit "server", keys aren't found

This is a slightly convoluted scenario (but I need it to use this from a chroot). Dumping the data here, I will try to debug later

Don't enable in p11-kit (ie, take out aws-kms-pkcs11.module) from /usr/share/p11-kit/modules or /etc/pkcs11/modules

Setup a server:

p11-kit server --provider /usr/lib64/pkcs11/aws_kms_pkcs11.so pkcs11:token=test-sign-key -f
(I use -f to easily ctrl-C)
Copy somewhere the output P11_KIT_SERVER_ADDRESS

Setup a client in .config/pkcs11/modules/p11-kit-client.module as follow

module: /usr/lib64/pkcs11/p11-kit-client.so

paste the P11_KIT_SERVER_ADDRESS

From this point p11-kit should use the client token which talks to the server which talks to aws-kms-pkcs11:
$ p11tool --list-all pkcs11:token=test-sign-key
Object 0:
	URL: pkcs11:token=test-sign-key;id=%61%6C%69%61%73%2F%74%65%73%74%2D%73%69%67%6E%2D%6B%65%79;object=test-sign-key;type=cert
	Type: X.509 Certificate
	Label: test-sign-key
	ID: 61:6c:69:61:73:2f:74:65:73:74:2d:73:69:67:6e:2d:6b:65:79

$ p11tool --list-keys pkcs11:token=test-sign-key
No matching objects found

I think this is the problem: https://gitlab.com/gnutls/gnutls/-/issues/1340, I'll try a fix later in gnutls. We might need to add an option to our plugin to create a dummy second priv. key as a workaround..

With the above, it seems to work now. Closing.


Add type definitions for Phaser

Copied from photonstorm/phaser repo.
Referring to issue photonstorm/phaser/issues/1724.
This is not my work, just adding it here for the convenience.

phaser/p2.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/phaser
http://bower.io/search/?q=phaser
others?

[ ] has a test file? (phaser/p2-tests.ts or others)
[ ] pass the Travic-CI test?

phaser/phaser.comments.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/phaser
http://bower.io/search/?q=phaser
others?

[ ] has a test file? (phaser/phaser.comments-tests.ts or others)
[ ] pass the Travic-CI test?

phaser/pixi.comments.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/phaser
http://bower.io/search/?q=phaser
others?

[ ] has a test file? (phaser/pixi.comments-tests.ts or others)
[ ] pass the Travic-CI test?

can you create phaser/phaser.d.ts? it XX/XX.d.ts naming pattern is good for definition file manager. (e.g. tsd

I've amended my commit, renaming files to remove .comments part.
For the tests, I'm not sure what should be tested? I think testing each function would be a bit too much.
From what I found, phaser has two examples: 1, 2. I could include those two examples to assert that it compiles. But this would only tests phaser library, but not the p2 and pixi, upon which the library (and some developers) depend.

@markogresak please read http://definitelytyped.org/guides/contributing.html#tests
test code is useful for reviewing and test backward compat about next pull request.

I have read the guide, but I still haven't understood it fully. Therefore I asked the question in earlier comment.

@basarat

Therefore I asked the question in earlier comment.

@markogresak would a reference please. Specifically would love to have at least a simple test to document how to use the definition. Feel free to ask questions about any difficulty :rose:

I took the url form phaser's definitions so I guess it should be correct.
Btw for some reason, the first time I've amended my commit, the phaser and pixi definitions were not added, therefore I had to amend it again to fix my mistake.
I will try to put up an example to test at least some of the library's aspects, although I'm not yet sure how to test other libraries than phaser. I will try to ask for more help at phaser repo, because I'm just a beginner at phaser and I wanted to test out how it works with typescript.
All I wanted to do is get these definitions on tsd, I apologize for doing a poor job on reading the docs.

I took the url form phaser's definitions so I guess it should be correct.

I think this is the file you are looking for : https://github.com/photonstorm/phaser/blob/master/typescript/phaser.d.ts
You would probably need to bring in definitions for phaser & pixi & p2 based on https://github.com/photonstorm/phaser/blob/4b22f48c75eb08f46b3359181bad42ca5bce81b3/typescript/phaser.d.ts#L1-L2
for p2 and pixi would need to be moved into their appropriate directories. Prefer directory names based on npm (if those packages are indeed available on npm)

As in photonstorm/phaser#2008, full coverage of Phaser would be a very difficult thing to do and since this repo requires all declarations to have test files, I'm closing this PR.


GH actions should be run against the pull request coming from the outside of our bubble

Currently almost all the gh actions / workflows are run on push event. So that if I open a PR from a feature branch that exist on the upstream repo the checks will appear on the PR, because they were run during the push into that feature branch. However, if anyone opens a pull-request from they own fork, the checks are not there.
We can't assume the contributors to create their feature branches on our repo (b/c of the rights hell), instead we should run all our checks against all the PRs.
To my knowledge there aren't any security implications here, there no secrets in the repo or gh metadata if i am not mistaken and if one opens a pull request w/ a change to the workflow itself, say providing a docker image that will mine some shitcoin, these actions that appears in the new PR should not be run before actually merging it (or at least that's how Travis worked and hopefully it's not a wishful thinking :D)

That's a good catch, i was skipping pull_request event in GHA configuration before because it is triggering duplicating action(s) run with our standard dev flow(aka push and create PR). It totally makes sense to incorporate it, especially from external contribution standpoint


Issue with GC options

Hey, I am trying to use your roles, this one and the one for Confluence, but both seem to have issues with the extra options you set for the JVM all GC related stuff... Crowd and Bitbucket work fine so far. I am on Ubtuntu 18.04 machines, seems the OpenJDK does not support these options.
Are they really needed?
I think I am going to replace your 'setenv.sh' changes with the original one supplied by Atlassian and just pull in the tags for jinja...

OK I think I found my error, the problem is Java 11, so going to get 8... sorry for bothering...

Yes please refer to molecule test case as recommended setup reference.
My basic principle for Atlassian’s stuff + Java version is: upgrade to another version, e.g. from openjdk 8 to 11, once all its 6 stuffs are confirm to be supported, else keep it as is.
I have this conclusion since jdk support upgrade form 6 to 7 to 8... Don’t make it too early else just keep on fighting with some we’ll known bugs...

Fixed by https://github.com/alvistack/ansible-role-jira/commit/86df9c7f28511fe7af05da9681726b8e23b4395e


fix: CrewAI-based flows with no extra openai

This pull request creates LLM() and Tool() objects which are CrewAI compatible, based on the components in the Crew-Ai based flow (including openai api key, etc)
@ogabrielluiz this fixes the crewai issues in my experience - the root problem is that we were (are) attempting to use tools and agent LLMs that are not in the format that Crew AI expects. This PR attempts to build the crewai versions from the specifications of the input tool and LLM. It's not perfect, as you'll see, but i think it might be a better short term solution than the setting env like you pointed out. Let me know what you think. CC @NadirJ, note i did upgrade CrewAI to the latest release with this, but this is also compatible with the prior release as well.

I like this.
What do you say we put this logic in the BaseCrewComponent?
The LLM can be passed to the agent or to the Crew. Also, do you know if they support all llms we support?

This is the main problem right now i think. I didn't see an easy way to find the "api_key" attribute in an arbitrary embeddings model (the langchain version). For example, in OpenAI this is openai_api_key. But obviously others its surely different. I was hoping there was a generic api_key attribute or some lookup to determine it. If you have any ideas there, that would be great... i also pass arbitrary kwargs to the LLM() function, but that has a specific api_key parameter that needs to be filled in so im sure there's models that wont work as is.
As for putting it in BaseCrewComponent, i think thats a great idea! I can update that.

@ogabrielluiz not sure if this would be a suitable solution in general, but we could do something like:
    def _find_api_key(self, model):
        """Attempts to find the API key attribute for a LangChain LLM model instance.

        Args:
            model: LangChain LLM model instance.

        Returns:
            The API key if found, otherwise None.
        """
        # Define the possible API key attribute names
        key_patterns = ["api_key", "key", "token"]

        # Iterate over the model attributes
        for attr in dir(model):

            # Check if the attribute name contains any of the key patterns
            if any(pattern in attr.lower() for pattern in key_patterns):
                value = getattr(model, attr, None)

                # Check if the value is a non-empty string
                if isinstance(value, str) and value:
                    return value

        return None

I think our best bet for now would be to downgrade crewai to a version it worked. What do you think?

I think our best bet for now would be to downgrade crewai to a version it worked. What do you think?

I could be wrong about this but i believe one of the reasons for upgrading crewai was the move to langchain~=0.3.x.... so .... i  agree its the best solution, but if thats correct then it may not be an easy option. I'll double check that though

I think our best bet for now would be to downgrade crewai to a version it worked. What do you think?

I could be wrong about this but i believe one of the reasons for upgrading crewai was the move to langchain~=0.3.x.... so .... i agree its the best solution, but if thats correct then it may not be an easy option. I'll double check that though

I see... Well. I guess, for now we focus on the short term solution. Maybe if the user passes a model that we haven't added support yet, raise an error.

I just updated the PR to factor the code a little better. i search for an key or token partial match in the attributes list - this works with the ones i've tested, i believe it works with all of the models we support that require a key, BUT - obviously its not perfect by any stretch and may cause a false positive - with that said, i think its better than the non-working solution from before (which really only works with OpenAI anyway).
If only there was some sort of from_langchain() method for an agent LLM like they have for tools (and im using in this PR). But alas, i don't see that: https://github.com/crewAIInc/crewAI/blob/main/src/crewai/llm.py

They use litellm which can be ok to integrate later. @phact has experience with it.