Command on level not working

Hello,
I cant use this feature
commands:
    kit:
      command:
      - lp user [player] permission add wtk.job.bucheron10
      - msg [player] !Permission bucheron10
      levelFrom: 1
      levelUntil: 10

When i level from 1 to 2, the commands are not performed.
I am on spigot 1.13.2

This works properly, only you have a configuration error because it has more space than it needed.
The correct:
commands:
  kit:
    command:
    - lp user [player] permission add wtk.job.bucheron10
    - msg [player] !Permission bucheron10
    levelFrom: 1
    levelUntil: 10

There is an indentation problem when i pasted my config here, but actually the indentation is good in my file and is exactely the same as  you.

This working for me correctly when I reach the 2 level from 1 and gets the commands. Make sure that the commands you set are working even if you type them in the game.

Hello,
Yeah sorry, it does work, it was because i was using alias command instead the real command.
Another question, does this seems correct for you, except indentation
permissions:
    wtk.job.bucheron:
      value: true
      level: 0
    wtk.job.bucheron10:
      value: true
      level: 10

Because i have trouble to make this work too.
Thanks

Unfortunately, the permission and conditions section has some minor permission setup problems. The problem is that it does not give the permission to the permission system. Same as #217

excuse me if i'm joining the conversation (just let me know if i need to open a new issue..) but i do encounter the same problem that was firstly stated here (more or less) , i can get the reward+ message for level 5 , nothing at all for level 10, 15,20,25 are working correctly , then level 30 doesn't give anything...
jobexample.txt

Bucheron:
fullname: Bûcheron
shortname: B
description: Gagne des sous en plantant et coupant des arbres
ChatColour: DARK_GREEN
chat-display: shortjob
max-level: 200
leveling-progression-equation: 10*(joblevel)+(jobleveljoblevel4)
income-progression-equation: baseincome+(baseincome*(joblevel-1)*0.01)-((baseincome+(joblevel-1)*0.01) * ((numjobs-1)0.05))
points-progression-equation: basepoints+(basepoints(joblevel-1)*0.01)-((basepoints+(joblevel-1)*0.01) * ((numjobs-1)0.05))
experience-progression-equation: baseexperience-(baseexperience((numjobs-1) *0.01))
rejoinCooldown: 10
Gui:
Id: 271
Data: 0
commands:
# command name, just to have better idea what this do
joinconstructeur:
# Command its self, this will be executed from console, so all commands should work
# Possible variables are: [player] [jobname] [oldlevel] [newlevel]
command: lp user [player] permission set jobs.join.constructeur true
# When to execute this command first time
levelFrom: 80
# Until when to do this command
# This can be set to same level as levelFrom, so this command will be executed only once
levelUntil: 80
message:
command: msg [player] Vous avez débloqué le métier de Constructeur !
levelFrom: 80
levelUntil: 80
level5:
command:
- eco give [player] 20
- msg [player] Paie pour avoir atteint le niveau 5 de votre job !
levelFrom: 5
levelUntil: 5
level10:
command:
- eco give [player] 50
- msg [player] Paie pour avoir atteint le niveau 10 de votre job !
levelFrom: 10
levelUntil: 10
level15:
command:
- eco give [player] 90
- msg [player] Paie pour avoir atteint le niveau 15 de votre job !
levelFrom: 15
levelUntil: 15
level20:
command:
- eco give [player] 140
- msg [player] Paie pour avoir atteint le niveau 20 de votre job !
levelFrom: 20
levelUntil: 20
level25:
command:
- eco give [player] 200
- msg [player] Paie pour avoir atteint le niveau 25 de votre job !
levelFrom: 25
levelUntil: 25
level30:
command:
- eco give [player] 270
- msg [player] Paie pour avoir atteint le niveau 30 de votre job !
levelFrom: 30
levelUntil: 30
level40:

@DonSensiLeone I checked this, and this working correctly for me when I reach the 30 level. Do you send me the full job that this problem occurs?

No i didn't send you the whole job but it's the same things until level 200 , i'm not sure that you understood correctly so i'm going to try to explain it again : when me or players from my servers are joining this particular job , we can get the message + eco command for leveling to level 5 , then once we reach level 10 , nothing happens at all (no message and no eco command) , then we can reach each levels (15,20,25) and get the message and the eco command but then , as we reach level 30 , its the same than reaching level 5 , no message and no eco command .. and i get this 'error' as well on my test server as on my official server , maybe is it related to essentials ? i don't know.. i'm kinda lost there as i seem to not messing up my job setup (or maybe do i ?)  Here is a paste of the full job : https://pastebin.com/fhEDLcEn

Since this last message i bought CMI , replaced essentials and other plugins with this.. and tested the job and i still get the same leveling errors at the exact same darn levels :/ i made the change in my job config for cmi money give [player] amount

Hello,
Yeah sorry, it does work, it was because i was using alias command instead the real command.

@DonSensiLeone
Check this message.

This error happens too on other jobs (same config than this one) ,as Miner , same levels are impacted with commands not working correctly.

@DonSensiLeone
You have duplicated names like
      level10:         command:         - eco give [player] 50         - msg [player] Paie pour avoir atteint le niveau 10 de votre job !         levelFrom: 10         levelUntil: 10
and
      level10:         command:         - eco give [player] 1000         - msg [player] Paie pour avoir atteint le niveau 110 de votre job !         levelFrom: 110         levelUntil: 110

Ohh ! that would explain ! i try to change and keep in touch


Submitting to appstore requires the ability to report post

I was rejected from the app store since I did not have a way for content to be reported. Just a heads up for other users that maybe using Twift.
Guideline 1.2 - Safety - User Generated Content
We found in our review that your app includes user-generated content but does not have all the required precautions. Apps with user-generated content must take specific steps to moderate content and prevent abusive behavior.
Next Steps
To resolve this issue, please revise your app to implement the following precautions:

Require that users agree to terms (EULA) and these terms must make it clear that there is no tolerance for objectionable content or abusive users

A mechanism for users to flag objectionable content

Resources
Learn more about our policies for user-generated content in App Store Review Guideline 1.2.

oh wow 😬
Had no idea about this, thank you for letting us know.
Daym...I would expect as we are making a Twitter api app that there is no need for us to implement such a thing....hm...Gotta make sure the popup is there I guess 😅

I don't see that v2 has a way to report spam (there is in v1 thought) but can block or mute users. Can you expand on this Gotta make sure the popup is there I guess? I'm trying to figure out how I can meet this requirement.

Well, I guess on first launch you have to have  screen saying "to use this app you must agree to EULA terms bla bla...and have a link to the terms as well.
You can even frame it as "by closing this alert you agree" 😅
As for flagging....mute is quite ok i guess and should be good enough. They just frame it as "flagging" as that is usually an option that you have

Closing the loop on this, but blocking the user and then redirecting to the user profile that was blocked satisfied Apple Review policy.


[BUG] in integration tests with VW

Description

tests/integration/test_criteo.py .                                       [  6%]
tests/integration/test_movielens.py .........                            [ 66%]
tests/integration/test_notebooks_python.py ....F                         [100%]

=================================== FAILURES ===================================
______________ test_vw_deep_dive_integration[1m-expected_values0] ______________

notebooks = {'als_deep_dive': '/data/home/recocat/cicd/7/s/notebooks/02_model/als_deep_dive.ipynb', 'als_pyspark': '/data/home/rec...baseline_deep_dive.ipynb', 'data_split': '/data/home/recocat/cicd/7/s/notebooks/01_prepare_data/data_split.ipynb', ...}
size = '1m'
expected_values = {'exp_var': 0.266197, 'mae': 0.686706, 'map': 0.186626, 'ndcg': 0.681827, ...}

    @pytest.mark.integration
    @pytest.mark.parametrize(
        "size, expected_values",
        [
            (
                "1m",
                dict(
                    rmse=0.958504,
                    mae=0.686706,
                    rsquared=0.26606,
                    exp_var=0.266197,
                    map=0.186626,
                    ndcg=0.681827,
                    precision=0.5748,
                    recall=0.186626,
                ),
            )
        ],
    )
    def test_vw_deep_dive_integration(notebooks, size, expected_values):
        notebook_path = notebooks["vowpal_wabbit_deep_dive"]
        pm.execute_notebook(
            notebook_path,
            OUTPUT_NOTEBOOK,
            kernel_name=KERNEL_NAME,
            parameters=dict(MOVIELENS_DATA_SIZE=size, TOP_K=10),
        )
        results = pm.read_notebook(OUTPUT_NOTEBOOK).dataframe.set_index("name")["value"]

        for key, value in expected_values.items():
>           assert results[key] == pytest.approx(value, rel=TOL, abs=ABS_TOL)
E           assert 0.5077596367584887 == 0.186626 ± 5.0e-02
E            +  where 0.186626 ± 5.0e-02 = <function approx at 0x7f6355d952f0>(0.186626, rel=0.05, abs=0.05)
E            +    where <function approx at 0x7f6355d952f0> = pytest.approx

original error: https://msdata.visualstudio.com/AlgorithmsAndDataScience/_build/results?buildId=2781053
In which platform does it happen?

Azure Data Science Virtual Machine
Expected behavior (i.e. solution)

tests passing
Other Comments

I've been looking at this, the failure comes in the integration tests of "1m". I found that 19 days ago we changed the metrics, originally we had:
"1m",
            dict(
                rmse=0.9555,
                mae=0.68493,
                rsquared=0.26547,
                exp_var=0.26615,
                map=0.50635,
                ndcg=0.99966,
                precision=0.92684,
                recall=0.50635,
            ),

tbh the rating metrics seemed too high, and now we have:
            "1m",
            dict(
                rmse=0.958504,
                mae=0.686706,
                rsquared=0.26606,
                exp_var=0.266197,
                map=0.186626,
                ndcg=0.681827,
                precision=0.5748,
                recall=0.186626,
            ),

If you run now the notebook, we will get the first set of metrics again, @gramhagen do you remember why we changed the metrics?
The only change I could find in the notebook is pred_df.drop("rating", axis=1, inplace=True) in cell 3. This is the new notebook and this is the old one

this is a little fuzzy, i think it was due to the dataframes having identical column names ('rating') when we merged them inside the metrics code the true rating column was renamed, it looks like I'm not dropping that column in the final calculation used for the metrics?
perhaps the latest changes to the metrics code exposed this? I'll look into it, we may need to check for this case in the metrics calculations.

yeah, couple of problems. if a column sharing the rating column name exists in the prediction dataframe it generates errors when calculating metrics. I've corrected this issue in #713, but there are also problems with how I'm generating the all pair evaluation set. I need to update the test metrics for all the versions of the notebook.  they are much lower now =(  ... but as you mention they were inexplicable high before, so it seems better

fixed in #722


switch to biblatex for bibliography

See https://github.com/livecomsjournal/article_templates/issues/83
Start using biblatex for bibliography handling, which provides a much
wider array of entities (@online, @software, ...).
This PR is still in the draft stage since the appearance of the bibliography has not yet been strongly customized to fit the current livecoms style.

@ltalirz I still think this is a good idea; what needs to be done to push forward with it?

hi @davidlmobley , see issues documented in https://github.com/livecomsjournal/article_templates/issues/83#issuecomment-943293356

Sorry, after properly re-reading that comment, its purpose was slightly different (highlighting some of the benefits that come for free when switching to biber).
Nevertheless, the screenshots can serve as how the citation styles currently differ. For example

the biber style does not include the grey + bold font for the first author
the biber style uses italic font, while the current style does not
...


Add dockerfile for cfdatabasetool

cfdatabasetool is needed to run the integration tests. We should be able
to run the integration tests by brining up rm services and required
external services via docker.
In the long run we would ideally be able to run the integration tests
by driving APIs instead of manipulating data via cfdatabasetool. Until
then it should be easy to bring into the stack from running tests
locally.
For make the docker image available to pull down by anyone we can
manually push up the image. It is unlikely to change often so this
shouldn't be a problem. If it does change often we should create a build
which publishes the docker image.

LGTM apart from the Codacy whinge about needing a docker file maintainer ... worth adding this?

Had a chat to @pricem14pc about it. The maintainer should probably be a team email but we don't have one at the moment. We could use @pricem14pc if we need one


tutorial_french issues in the Narrative Model

Hello! I have been using the codes recently to analyze text. Yet, I encounter an error when executing the "tutorial_french" file. I have already tried with different versions of Python, but the error (see below) is not solved. I have tried changing all of the parameters in the NarrativeModel, and found out that the issue is the clustering parameter. I tried reinstalling the hdbscan, but the error persists. For the tutorial_english I did not have any problem. I appreaciate your help in this issue.
Thank you in advance!
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[13], line 15
      1 from relatio.narrative_models import NarrativeModel
      3 m = NarrativeModel(
      4     clustering = 'hdbscan',
      5     PCA = False,
   (...)
     12     threshold = 0.3
     13 )
---> 15 m.fit(postproc_roles, weight_by_frequency = True)

File ~/anaconda3/envs/adsml/lib/python3.8/site-packages/relatio/narrative_models.py:158, in NarrativeModel.fit(self, srl_res, pca_args, umap_args, cluster_args, weight_by_frequency, progress_bar)
    156     print("No fitting required, this model is deterministic!")
    157 if self.clustering in ["hdbscan", "kmeans"]:
--> 158     self.fit_static_clustering(
    159         srl_res,
    160         pca_args,
    161         umap_args,
    162         cluster_args,
    163         weight_by_frequency,
    164         progress_bar,
    165     )
    166 if self.clustering == "dynamic":
    167     pass

File ~/anaconda3/envs/adsml/lib/python3.8/site-packages/relatio/narrative_models.py:362, in NarrativeModel.fit_static_clustering(self, srl_res, pca_args, umap_args, cluster_args, weight_by_frequency, progress_bar)
    355     if k not in [
    356         "min_cluster_size",
    357         "min_samples",
    358         "cluster_selection_method",
    359     ]:
    360         args[k] = v
--> 362 hdb = hdbscan.HDBSCAN(**args).fit(self.training_vectors)
    363 models.append(hdb)
    364 score = hdbscan.validity.validity_index(
    365     self.training_vectors.astype(np.float64), hdb.labels_
    366 )

File ~/anaconda3/envs/adsml/lib/python3.8/site-packages/hdbscan/hdbscan_.py:1205, in HDBSCAN.fit(self, X, y)
   1195 kwargs.pop("prediction_data", None)
   1196 kwargs.update(self._metric_kwargs)
   1198 (
   1199     self.labels_,
   1200     self.probabilities_,
   1201     self.cluster_persistence_,
   1202     self._condensed_tree,
   1203     self._single_linkage_tree,
   1204     self._min_spanning_tree,
-> 1205 ) = hdbscan(clean_data, **kwargs)
   1207 if self.metric != "precomputed" and not self._all_finite:
   1208     # remap indices to align with original data in the case of non-finite entries.
   1209     self._condensed_tree = remap_condensed_tree(
   1210         self._condensed_tree, internal_to_raw, outliers
   1211     )

File ~/anaconda3/envs/adsml/lib/python3.8/site-packages/hdbscan/hdbscan_.py:824, in hdbscan(X, min_cluster_size, min_samples, alpha, cluster_selection_epsilon, max_cluster_size, metric, p, leaf_size, algorithm, memory, approx_min_span_tree, gen_min_span_tree, core_dist_n_jobs, cluster_selection_method, allow_single_cluster, match_reference_implementation, **kwargs)
    820 elif metric in KDTREE_VALID_METRICS:
    821     # TO DO: Need heuristic to decide when to go to boruvka;
    822     # still debugging for now
    823     if X.shape[1] > 60:
--> 824         (single_linkage_tree, result_min_span_tree) = memory.cache(
    825             _hdbscan_prims_kdtree
    826         )(
    827             X,
    828             min_samples,
    829             alpha,
    830             metric,
    831             p,
    832             leaf_size,
    833             gen_min_span_tree,
    834             **kwargs
    835         )
    836     else:
    837         (single_linkage_tree, result_min_span_tree) = memory.cache(
    838             _hdbscan_boruvka_kdtree
    839         )(
   (...)
    849             **kwargs
    850         )

File ~/anaconda3/envs/adsml/lib/python3.8/site-packages/joblib/memory.py:349, in NotMemorizedFunc.__call__(self, *args, **kwargs)
    348 def __call__(self, *args, **kwargs):
--> 349     return self.func(*args, **kwargs)

File ~/anaconda3/envs/adsml/lib/python3.8/site-packages/hdbscan/hdbscan_.py:265, in _hdbscan_prims_kdtree(X, min_samples, alpha, metric, p, leaf_size, gen_min_span_tree, **kwargs)
    260 core_distances = tree.query(
    261     X, k=min_samples + 1, dualtree=True, breadth_first=True
    262 )[0][:, -1].copy(order="C")
    264 # Mutual reachability distance is implicit in mst_linkage_core_vector
--> 265 min_spanning_tree = mst_linkage_core_vector(X, core_distances, dist_metric, alpha)
    267 # Sort edges of the min_spanning_tree by weight
    268 min_spanning_tree = min_spanning_tree[np.argsort(min_spanning_tree.T[2]), :]

File hdbscan/_hdbscan_linkage.pyx:55, in hdbscan._hdbscan_linkage.mst_linkage_core_vector()

File hdbscan/_hdbscan_linkage.pyx:165, in hdbscan._hdbscan_linkage.mst_linkage_core_vector()

TypeError: 'float' object cannot be interpreted as an integer

Hi there,
This is a general bug on scikit-learn's side related to Cython versions.
See the issue here: https://github.com/scikit-learn-contrib/hdbscan/issues/600
We will have to let them figure this out on their side before we can do anything about it.
In the meantime, I would suggest relying on the KMeans algorithm.
Sorry for the inconvenience!
Best,
Germain


Problem with instalation

Dear developers , I am trying to install proso -apps locally to verify the behavior of an adaptive system, but when trying to install is returned me this error:
(proso-apps) matheus@matheus-VirtualBox:~/proso-apps$ make install
pip install flake8
Collecting flake8
  Using cached flake8-2.5.4-py2.py3-none-any.whl
Collecting pep8!=1.6.0,!=1.6.1,!=1.6.2,>=1.5.7 (from flake8)
  Using cached pep8-1.7.0-py2.py3-none-any.whl
Collecting pyflakes<1.1,>=0.8.1 (from flake8)
  Using cached pyflakes-1.0.0-py2.py3-none-any.whl
Collecting mccabe<0.5,>=0.2.1 (from flake8)
  Using cached mccabe-0.4.0-py2.py3-none-any.whl
Installing collected packages: pep8, pyflakes, mccabe, flake8
Successfully installed flake8-2.5.4 mccabe-0.4.0 pep8-1.7.0 pyflakes-1.0.0
flake8 --ignore=E501,E225,E123,E128,W503,E731 --exclude=*/migrations/*.py,*/static/bower_components,setup.py,docs/conf.py,.ropeproject .
./proso/models/environment.py:12:29: E901 SyntaxError: invalid syntax
./proso/models/item_selection.py:16:31: E901 SyntaxError: invalid syntax
./proso/models/option_selection.py:19:33: E901 SyntaxError: invalid syntax
./proso/models/prediction.py:8:33: E901 SyntaxError: invalid syntax
./proso_flashcards/flashcard_construction.py:23:27: E901 SyntaxError: invalid syntax
Makefile:78: recipe for target 'check' failed
make: *** [check] Error 1

All items have been installed , but in your own the python2.7 and python3.5
Could you help me about it ?

Hi Mateus,
you are probably the first user of proso-apps outside our research group. Firstly, I would like to apologize you for a lot of undocumented stuff and not so stable development. What exactly is your motivation to use proso-apps?
The problematic syntax in your case is:
class Example(metaclass=abc.ABCMeta)

This syntax is not supported by Python 2.7. If you want to run your own installation of proso-apps, version 3.X is required and we recommend version 3.5.
I would also recommend you to use pip to installed last released version.

Hello Papousek ,
I have studied the adaptive education for some time, and would like to understand the functioning of an adaptive system and I was very excited to learn that there is your group working on it openly .
The biggest problem is that I 've never worked with Linux and has three weeks I'm trying to run the code and can not , with your tips I will see if I can and if possible would like to contribute with you on something .
I made the request in transifex to help in the translation into Portuguese of Brazil

Interesting!
If you want to run a demo, load demo data from our geography app (https://outlinemaps.org) and run testproject:
./manage.py testproject/test_data/flashcards/geography.json
./manage.py runserver

After this import you can visit (for example) API for instructional policy:
http://localhost:8000/models/practice?html&filter=[["africa/state"]]&limit=1

Be aware we use caching a lot. Caching is disabled for testproject, so testproject is quite slow.


Create dartpad Mittelwert

mittelwert

@lucaforster, I'm not sure what this PR is meant to do. Can you provide any info?


Support NumPy 1.20.0

The several mypy tests are failing on Python >= 3.7 in CI since NumPy has been upgraded to 1.20.0.
Since NumPy dropped Python 3.6 from NumPy 1.20.0, only the tests run on Python >= 3.7 are affected.

Codecov Report

Merging #2026 (7be33c7) into master (c5dbc9b) will decrease coverage by 3.77%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2026      +/-   ##
==========================================
- Coverage   94.60%   90.82%   -3.78%
==========================================
  Files          50       50
  Lines       11280    11173     -107
==========================================
- Hits        10671    10148     -523
- Misses        609     1025     +416

Impacted Files
Coverage Δ

databricks/koalas/frame.py
95.68% <ø> (-0.90%)
:arrow_down:

databricks/koalas/ml.py
96.29% <ø> (ø)

databricks/koalas/indexing.py
92.53% <100.00%> (-0.10%)
:arrow_down:

databricks/koalas/internal.py
95.92% <100.00%> (-0.69%)
:arrow_down:

databricks/koalas/series.py
96.40% <100.00%> (-0.39%)
:arrow_down:

databricks/koalas/typedef/typehints.py
92.03% <100.00%> (-1.77%)
:arrow_down:

databricks/koalas/plot/plotly.py
10.52% <0.00%> (-85.53%)
:arrow_down:

...bricks/koalas/tests/plot/test_frame_plot_plotly.py
23.89% <0.00%> (-76.11%)
:arrow_down:

...ricks/koalas/tests/plot/test_series_plot_plotly.py
27.83% <0.00%> (-70.23%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
25.66% <0.00%> (-66.65%)
:arrow_down:

... and 26 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 c5dbc9b...7be33c7. Read the comment docs.

Here is the summary of investigation:
>>> pdf = pd.DataFrame(
...     {
...         "a": [1, 2, 3, 4, 5, 6] * 100,
...         "b": [1.0, 1.0, 2.0, 3.0, 5.0, 8.0] * 100,
...         "c": [1, 4, 9, 16, 25, 36] * 100,
...     },
...     columns=["a", "b", "c"],
...     index=np.random.rand(600),
... )
>>> kdf = ks.DataFrame(pdf)
>>> kdf.koalas.transform_batch(lambda pdf: pdf.c + 1)
org.apache.spark.api.python.PythonException: Traceback (most recent call last):
...
pyarrow.lib.ArrowTypeError: Did not pass numpy.dtype object
...
RuntimeError: ('Exception thrown when converting pandas.Series (int64) to Arrow Array (int64). It can be caused by overflows or other unsafe conversions warned by Arrow. Arrow safe type check can be disabled by using SQL config `spark.sql.execution.pandas.convertToArrowArraySafely`.', ArrowTypeError('Did not pass numpy.dtype object'))

Seems like there is an incompatible issue between PyArrow and latest NumPy (1.20.0).
See https://github.com/Azure/MachineLearningNotebooks/issues/1314.
Until now, people seem to choose the solution not using the latest version of NumPy.
So, I'd say maybe it would be better to keep the NumPy upperbound and pending this PR until the issue is resolved ?
WDYT, @ueshin @HyukjinKwon @xinrong-databricks ??

Can you file an issue in Apache Arrow side and confirm that this is an issue?

numpy 1.21.1 has been supported in pyspark-pandas module of Build and test. I will close the PR now. @itholic let me know if there are any concerns. :)


added image delete functionality

Codecov Report

Merging #42 (d51a908) into main (d59798f) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   81.18%   81.47%   +0.28%
==========================================
  Files          10       10
  Lines         776      788      +12
==========================================
+ Hits          630      642      +12
  Misses        103      103
  Partials       43       43

Impacted Files
Coverage Δ

image.go
75.00% <100.00%> (+4.16%)
:arrow_up:

smugmug.go
96.42% <100.00%> (ø)

Continue to review full report at Codecov.

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


Readme update for Firebase Crashlytics?

Love this plugin so much!!
Maybe the readme could be updated for using Firebase Crashlytics? When doing so, the API keys parts are not needed. Everything else is the same and works great.

Thanks for the feedback ! Never used Firebase Crashlytics yet :) also this plugin embed Fabric binaries for Crashlytics and not Firebase one (doesn't know if it matters or not ^^)
How firebase know which project it is if you don't have any API key or setup somewhere ?
If you want to put some sentences about setup with Firebase Crashlytics on the readme don't hesitate to make a PR with :) I'll gladly merge it

Works great so far with Firebase. Basically, you need to run your app once and firebase crashlytics works. I followed your readme but did not add both parts about the API keys. Fabric moving to firebase next year.

Ho ok :) I know that Fabric is moving totally to firebase next year, but for now this plugin is temporary since Google is supposed to publish one with real support for dart/flutter. So I'm waiting for them see what they will offer. Depending on that I'll migrate the plugin to Firebase :)
If it already work it's awesome ! :)

Readme updated with your notes :) thanks again


couchdb2.0 is not sending charset=utf-8 in response headers of json

HTTP/1.1 200 OK
Server: CouchDB/1.6.0 (Erlang OTP/19)
ETag: "GG61GZEDIDA2MOVW8O6204F7"
Date: Wed, 26 Apr 2017 07:53:28 GMT
Content-Type: text/plain; charset=utf-8
Content-Length: 0
Cache-Control: must-revalidate
HTTP/1.1 200 OK
Cache-Control: must-revalidate
Content-Length: 0
Content-Type: application/json
Date: Wed, 26 Apr 2017 07:53:34 GMT
Server: CouchDB/2.0.0 (Erlang OTP/19)
X-Couch-Request-ID: undefined
X-CouchDB-Body-Time: 0

Thank you for your contribution and interest to this problem!
Unfortuanelly, I have two things to notice:

You need to create a new feature branch for your PR from the branch to which you want it get merged (it's 2.0.x now), but...
application/json is always UTF-8 by default and his MIME type doesn't accepts any parameters per RFC-7159. See sections 8.1 and 11 for more information.

I think there is nothing need to do and everything should works fine. Do you have any issues caused by missed charset on MIME type? And why?

Hello, the Chromium version 57.0.2987.98 Built on Ubuntu , running on Ubuntu 17.04 (64-bit) does not show correctly the accent characters.

utf-8 is the defined charset when interpreting application/json, this is unnecessary.
https://www.ietf.org/rfc/rfc4627.txt
"3.  Encoding
JSON text SHALL be encoded in Unicode.  The default encoding is
UTF-8."


Add eventually! macro to help de-flake telemetry tests

Closes #615.
Based on @olix0r's suggestion in https://github.com/runconduit/conduit/issues/613#issuecomment-376024744, this PR adds an eventually! macro to retry an assertion a set number of times, waiting for 15 ms between retries. This is loosely based on ScalaTest's eventually.
I've rewritten the flaky telemetry tests to use the eventually! macro, to compensate for delays in the served metrics being updated between client requests and metrics scrapes.

@carllerche with regards to the name, it's called eventually! because a similar function in ScalaTest is called that, and @olix0r initially suggested "we should have something like ScalaTest's eventually". I'm definitely open to changing it, though I agree assert_eventually! is a bit lengthy --- how about just assert_eventual or eventual_assert? I'm not sure if I like that better.

Okay, I've changed the name to assert_eventually! as per @carllerche. Will merge when CI passes.


make voidFn() return value explicit

This is to get tree shaking when using SolidStart's getRequestEvent() function e.g.
export default () => {
	const requestEvent = getRequestEvent()

	if (requestEvent) {
		// server code here
	}

	// ...
}

For some reason Rollup can't statically analyse a function without an explicit return value. So before this change, the server code gets bundled into the client, but after this change, Rollup is able to see that getRequestEvent() always returns undefined meaning that requestEvent is always undefined meaning that the if statement can be tree shaken away for the client build.

Odd but can't hurt.


[added] isNested prop

Changes proposed:

adds isNested: bool (optional) prop, which gives you the option of not removing the modal backdrop.

Why? This enables Russian doll nesting, so you can have multiple modals open on top of each other.
Acceptance Checklist:

[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[ ] If the commit message has [changed] or [removed], there is an upgrade path above.

Coverage increased (+0.06%) to 86.758% when pulling 4c17fc42995ece0e8a332ec74e41ea3bca5285e6 on Jarch09:add-nesting into 1b561fc54f107df583281667eb92e7975ec33ba8 on reactjs:master.

Hey you have a chance to check this out?

So is this going to be pulled at any point?
Does this change only remove the background effect or does it also affect modal stacking order?


Add pm2 to prod env

See:

https://stackoverflow.com/questions/28942614/should-i-use-forever-pm2-within-a-docker-container
https://github.com/Unitech/pm2/issues/1879

prefer using docker --restart=always  than pm2 start dist/server.js or forever start dist/server.js


[REQ] Output of a script

Basically, I was thinking: wouldn't it be nice to be able to have a command output being displayed?
something like
label {
    monitor =
    exec = uptime -p
    interval = 30
    color = rgba(200, 200, 200, 1.0)
    font_size = 25
    font_family = Noto Sans

    position = 0, 80
    halign = center
    valign = center
}

To be clear, exactly what custom/script does in waybar
This would allow for basically everything to be displayed in hyprlock: uptime, time, CPU/memory usage, battery percentage, blah blah blah without the need for custom widgets

and exec_once might also be nice (e.g. fetching a quote). If vaxry is interested, I just did some config stuff in one of the prs and could therefore come up with another one

I mean, a way to get that would just be interval = once, thats how it works in waybar at least
but sure

The term can be arbitrary, but hyprland already uses these fields.. but I do not have any strong feelings about it.

I see, but then the case where both exec and exec_once are present should also be handled

I see, but then the case where both exec and exec_once are present should also be handled

From a label/widget perspective this does not make to much sense to have both at the same time present.

5370762eb3f27ecfdb9b4d1f82a9c91c21c88730

text = cmd[update:1000] echo "$(date)"

please note to never run commands that don't exit unless you wanna have a really bad time.


ECDSA signature verification fails when the y coordinate is large

coincurve 13.0.0 fails when the y coordinate is large. You can check this
with the following teste vectors from Google Wycheproof:
{
  "algorithm" : "ECDSA",
  "generatorVersion" : "0.8r12",
  "numberOfTests" : 380,
  "header" : [
    "Test vectors of type EcdsaVerify are meant for the verification",
    "of ASN encoded ECDSA signatures."
  ],
  "notes" : {
    "BER" : "This is a signature with correct values for (r, s) but using some alternative BER encoding instead of DER encoding. Implementations should not accept such signatures to limit signature malleability.",
    "EdgeCase" : "Edge case values such as r=1 and s=0 can lead to forgeries if the ECDSA implementation does not check boundaries and computes s^(-1)==0.",
    "MissingZero" : "Some implementations of ECDSA and DSA incorrectly encode r and s by not including leading zeros in the ASN encoding of integers when necessary. Hence, some implementations (e.g. jdk) allow signatures with incorrect ASN encodings assuming that the signature is otherwise valid.",
    "PointDuplication" : "Some implementations of ECDSA do not handle duplication and points at infinity correctly. This is a test vector that has been specially crafted to check for such an omission."
  },
  "schema" : "ecdsa_verify_schema.json",
  "testGroups" : [
    {
      "key" : {
        "curve" : "secp256k1",
        "keySize" : 256,
        "type" : "EcPublicKey",
        "uncompressed" : "046e823555452914099182c6b2c1d6f0b5d28d50ccd005af2ce1bba541aa40cafffffffffef9fb6d2a5a98c1f0da272af0481a73b62792b92bde96aa1e55c2bb4e",
        "wx" : "6e823555452914099182c6b2c1d6f0b5d28d50ccd005af2ce1bba541aa40caff",
        "wy" : "00fffffffef9fb6d2a5a98c1f0da272af0481a73b62792b92bde96aa1e55c2bb4e"
      },
      "keyDer" : "3056301006072a8648ce3d020106052b8104000a034200046e823555452914099182c6b2c1d6f0b5d28d50ccd005af2ce1bba541aa40cafffffffffef9fb6d2a5a98c1f0da272af0481a73b62792b92bde96aa1e55c2bb4e",
      "keyPem" : "-----BEGIN PUBLIC KEY-----\nMFYwEAYHKoZIzj0CAQYFK4EEAAoDQgAEboI1VUUpFAmRgsaywdbwtdKNUMzQBa8s\n4bulQapAyv/////++fttKlqYwfDaJyrwSBpztieSuSvelqoeVcK7Tg==\n-----END PUBLIC KEY-----",
      "sha" : "SHA-256",
      "type" : "EcdsaVerify",
      "tests" : [
        {
          "tcId" : 366,
          "comment" : "y-coordinate of the public key is large",
          "msg" : "4d657373616765",
          "sig" : "304502203854a3998aebdf2dbc28adac4181462ccac7873907ab7f212c42db0e69b56ed8022100c12c09475c772fd0c1b2060d5163e42bf71d727e4ae7c03eeba954bf50b43bb3",
          "result" : "valid",
          "flags" : []
        },
        {
          "tcId" : 367,
          "comment" : "y-coordinate of the public key is large",
          "msg" : "4d657373616765",
          "sig" : "3046022100e94dbdc38795fe5c904d8f16d969d3b587f0a25d2de90b6d8c5c53ff887e3607022100856b8c963e9b68dade44750bf97ec4d11b1a0a3804f4cb79aa27bdea78ac14e4",
          "result" : "valid",
          "flags" : []
        }
      ]
    }
  ]
}

and proof of concept:
from coincurve.keys import PrivateKey, PublicKey
from coincurve.utils import bytes_to_int, int_to_bytes_padded, verify_signature

from cryptography import exceptions
from cryptography.hazmat.backends import default_backend
from cryptography.hazmat.primitives import hashes
from cryptography.hazmat.primitives.asymmetric import ec
from cryptography.hazmat.primitives.asymmetric import utils

from hashlib import sha256

import json

def main():

    with open("ecdsa_secp256k1_sha256_test_y_coordinate_large.json", "r") as read_file:
        secp256k1_sha256_test = json.load(read_file)

    for test_group in secp256k1_sha256_test["testGroups"]:
        comp_str = test_group["key"]["uncompressed"]
        co_x = test_group["key"]["wx"]
        co_y = test_group["key"]["wy"]

        public_key = PublicKey(bytes.fromhex(comp_str))

        for test in test_group["tests"]:
            print("Loading test", test["tcId"], "with result:", test["result"], "\n")

            ### using python coincurve

            signature = bytes.fromhex(test["sig"])
            message = bytes.fromhex(test["msg"])
            print("\tcoincurve: ", public_key.verify(signature, message))

            ### using python cryptography

            curve = ec.SECP256K1()
            algo = ec.ECDSA(hashes.SHA256())

            pubnum = ec.EllipticCurvePublicNumbers(int(co_x, 16), int(co_y, 16), curve)

            data = bytes(bytearray.fromhex(test["msg"]))
            public_key_2 = pubnum.public_key(default_backend())
            signature = bytes(bytearray.fromhex(test["sig"]))

            try:
                public_key_2.verify(signature, data, ec.ECDSA(hashes.SHA256()))

            except exceptions.InvalidSignature:
                print("\tcryptography.io: False")
            else:
                print("\tcryptography.io: True")

if __name__ == "__main__":
    main()

where I compare the output of coincurve with the python cryptography
library.

https://github.com/ofek/coincurve/issues/68#issuecomment-633665607


[SPARK-21062] There are some minor mistakes in log

https://issues.apache.org/jira/browse/SPARK-21062
There are some minor mistakes in log.For example,"Offsets not available on leader" should be "Offsets are not available on leader" in kafkautils.

Can one of the admins verify this patch?

See JIRA; this isn't an improvement, and should be closed.


This aligns with some users' values.

Thanks for the pull request! I've edited the doc to reflect this idea and will continue doing so.


Create CollectGCDump Collection Rule Action

There's a fair number of similarities between this and the CollectDump Action. Most significant edits are either reorganization of existing functionality or an adjustment of code used for that action to fit this one. Once all the actions have been finished, I think there may be some benefit in trying to refactor similar code in the actions/tests into reusable methods, but for the sake of working on the actions simultaneously I've refrained from doing some of that for the time being.
Side note: I had originally attempted to rebase my changes based off main at Justin's request, but ended up having issues several times that broke my code. I ultimately merged with main, which appears to have worked fine; however, if there are issues and a rebase is necessary, we can close this PR and I can take another stab at rebasing.
Closes #658

/azp run


Heterogen cluster

Just for review (Please do not merge!)

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/690/

I think the name templateGroups is misleading because it suggests that a templateGroup contains multiple templates so I think this should be changed to instanceGroups instead

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/691/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/692/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/694/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/695/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/696/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/697/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/698/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/699/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/700/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/701/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/702/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/703/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/704/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/705/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/706/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/707/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/708/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/709/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/710/

all test done db migration needed before the merge.

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/711/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/712/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/713/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/714/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/716/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/717/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/718/


IPropertyStore, RECT, etc. generated with non-public visibility

Actual behavior
IPropertyStore and other types are generated with internal visibility, which is unexpected and hard to work with.
Expected behavior
Public visibility like other types and methods generated by CsWin32.
Repro steps
NativeMethods.txt content:
IPropertyStore

Results in:
// ...
namespace Windows.Win32
{
	namespace UI.Shell.PropertiesSystem
	{
		[Guid("886D8EEB-8CF2-4446-8D02-CDBA1DBDCF99"),InterfaceType(ComInterfaceType.InterfaceIsIUnknown),ComImport()]
		[global::System.CodeDom.Compiler.GeneratedCode("Microsoft.Windows.CsWin32", "0.3.49-beta+91f5c15987")]
		internal interface IPropertyStore
// ...

Context

CsWin32 version: 0.3.49-beta
Win32Metadata version (if explicitly set by project):
Target Framework: net8.0-windows10.0.19041.0
LangVersion (if explicitly set by project): [e.g. 9]

Just realized there's a NativeMethods.json (schema), where I can specify "public": true. Neat. Maybe we can add a schema link to the README?

Public visibility like other types and methods generated by CsWin32.

CsWin32 doesn't generate anything as public by default unless the C# specification requires it (e.g. operators). Or if you see otherwise, it's a bug in the generator, as the goal is to not add any public types to the receiving assembly unless the project opts in, as you've discovered.
I'll add a link to the schema as you suggest.

Thanks @aarnott, the "public": true is working great stepping stone as I migrate a large project over.


Feature/openapi add yaml support

Description
Added the yaml file support when importing OpenApi collection
Linked to this : https://github.com/usebruno/bruno/issues/729
Contribution Checklist:

[X] The pull request only addresses one issue or adds one feature.
[X] The pull request does not introduce any breaking changes
[X] I have added screenshots or gifs to help explain the change if applicable.
[X] I have read the contribution guidelines.
[X] Create an issue and link to the pull request.

@survivant Can you sync your fork with the main. There are some conflicts in package-lock.json
Delete the package-lock.json, npm install and push your changes.

@helloanoop done

Thanks for this @survivant !


Title: Doodle Jump Game

Is your feature request related to a problem? Please describe.
We might need more exciting adn kind of addictive games which involves visible movements!
Describe the solution you'd like.
Doodle Jump is a popular and addictive game known for its simple yet engaging mechanics, which can attract a wide audience. By incorporating this game, we can create a more captivating and enjoyable experience for our users
Describe alternatives you've considered.
.
Add any other context or screenshots about the feature request here.
will implement this popular game

@AnanyaJain25 Thank you for providing your insight but it is not required as of now

hey @swapnilsparsh May I know whats the issue with this.. It will be same as other projects completely based on Javascript

@AnanyaJain25 All the best 👍🏻 and make sure to use only HTML, CSS and JS with proper all screens responsiveness

@AnanyaJain25 All the best 👍🏻 and make sure to use only HTML, CSS and JS with proper all screens responsiveness

yes sir ! Sure sir


fix(Input.TextArea): TextArea autoResize not work while composition updating

Types of changes

[ ] New feature
[x] Bug fix
[ ] Enhancement
[ ] Documentation change
[ ] Coding style change
[ ] Component style change
[ ] Refactoring
[ ] Test cases
[ ] Continuous integration
[ ] Typescript definition change
[ ] Breaking change
[ ] Others

Changelog

Component
Changelog(CN)
Changelog(EN)
Related issues

Input
修复 Input.TextArea 组件 autoSize 属性在非英文输入法时可能不生效的问题。
Fix the issue that the autoSize of the Input.TextArea may not take effect while typing non-English texts.
Close #1084

Checklist:

[ ] Test suite passes (npm run test)
[ ] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others should be submitted to main branch)

Other information

Shall this thread get closed?


Bug: shadow_color > 0 modifies whole color output

background {
    color = 0xff212121
}

label { #1
    color = 0xFFD4BE98
    shadow_passes = 2
    shadow_size = 3
    shadow_color = rgb(255,0,0)
    shadow_boost = 1.2
}

label { #2
    color = 0xFFD4BE98
    shadow_passes = 1
    shadow_size = 3
    shadow_color = rgb(0,0,255)
    shadow_boost = 1.2
}

oh and maybe
hyprctl monitors -j | jaq '.[].currentFormat'
"XRGB8888"

can help

ah yes, I should've watched more captain disillusion.
Alpha premultiplication, gets me every time.


Use action bar instead of chat to announce shooting stars?

Hi hello! I'd like to suggest using the action bar above the toolbar (right above where item names come up when you scroll through them on the toolbar) to show shooting star announcements, instead of in the permanently-logged chat.
The text may need to be shortened (A shooting star has appeared in the sky - how lucky!) to suit, but I think it would be a more seamless experience

Changed in 2.0.0, thanks for the feedback!


EACCES when browsing extensions

From @hinogi, https://twitter.com/con_SSC/status/740876561835032576

[...] ext install gives an error "connect EACCES <IP_ADDRESS>:443"
[...] I think I'm not behind a proxy, the stable works just fine

Possibly related to #5596
--
@hinogi I just want to confirm a few things:

You have both the insider and stable releases installed in the same machine
You can both browse and install extensions in the stable release
You can't even browse extensions in the insider release
Your stable installation doesn't have any proxy settings configured (both global and workspace settings).

Can you confirm all those details for me? Thanks!
Also, can you run ping marketplace.visualstudio.com in the command prompt?

I cannot ping marketplace.visualstudio.com I get a timeout there, I'm not sure if ICMP is active on the other side. tracert hops near the destination but cannot reach it if I look at the IP ranges.
After Hop  23   137 ms   136 ms   133 ms  ae86-0.sn2-96cb-1b.ntwk.msn.net [<IP_ADDRESS>] it does not continue.
The rest I can confirm.

Yes, timeout makes sense. But it does resolve to <IP_ADDRESS> correct? It there anyway you could run a network analysis tool like Wireshark in both the insider and stable cases so we figure out what the difference it?

yes, it is not resolved. Anything special I should be looking for?

This is the stable version

This is the insider version

It looks pretty much like a certificate issue

Since both are accessing the same URL... this is sounding more and more strange.
I apologise for asking once again, but can you just triple check that you don't have any proxy settings in your stable, especially the http.proxyStrictSSL? Make sure you check your workspace settings too, in case you have a workspace open.
Any chance you can export those logs, so I can open them in Wireshark and take a closer look?

I have saved the logs but I don't want to put them up here of course.
both have the same configuration
//-------- HTTP configuration --------
// The proxy setting to use. If not set will be taken from the http_proxy and https_proxy environment variables
"http.proxy": "",

// Whether the proxy server certificate should be verified against the list of supplied CAs.
"http.proxyStrictSSL": true,

That seems to be the default configuration. I meant the contents of both .vscode/settings.json contained within your workspace and the settings.json file that opens on the right when you open the User Settings.
Also, I assume you have no http_proxy and https_proxy environment variables set?
And another thing, can you install an extension on stable?

I don't have a .vscode/settingsjson and I don't have any env. variables for proxy, and the user settings do not contain any http entries.

How about that last part? Can you install an extension on stable? And what version is your stable installation?
Sorry for all the questions... and thanks for the help by the way!

No problem, I have a lot extensions in stable. I have currently version 1.2.1 I can install, uninstall, update etc.

Stranger and stranger. Insiders is practically equal to stable, with regards to the gallery.
Is some firewall blocking the Insiders build? Can you disable any firewall/antivirus temporarily and see whether that is the case?

I cannot I'm afraid. I have no access to the antivirus program or firewall if there is one.

Alright let's try something else then. Can you download the latest insider release from here, and try there?
https://vscode-update.azurewebsites.net/latest/win32-archive/insider
This URL will download Code as an archive, instead of as a Setup executable. You can just extract and execute it from there directly.

Downloaded, extracted, same error on ext install

How about the stable archive?
https://vscode-update.azurewebsites.net/latest/win32-archive/stable
What happens if you rename the executable from the insiders to Code.exe and vice versa to the stable?

stable works as intended, I can use ext install.
renaming the insider executable didn't change anything.

Totally forgot about the devTools:
workbench.main.js:115 connect EACCES <IP_ADDRESS>:443: Error: connect EACCES <IP_ADDRESS>:443
at Object.exports._errnoException (util.js:890:11)
at exports._exceptionWithHostPort (util.js:913:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1061:14)
workbench.main.js:88 connect EACCES <IP_ADDRESS>:443: Error: connect EACCES <IP_ADDRESS>:443
at Object.exports._errnoException (util.js:890:11)
at exports._exceptionWithHostPort (util.js:913:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1061:14)

OK, so I'll try to compile it all down:

Error is connect EACCES, happening on Windows
Always happens on the insider executable
Never happens on the stable one
Both stable and insider builds do exactly the same when talking to the Marketplace

Googling a bit for "connect EACCES" windows yields firewall issues:

http://stackoverflow.com/questions/26192865/npm-install-eacces-error-on-windows
felixge/node-mysql#858

I suggest once again to disable any firewall on the system. Also check the Windows firewall. Usually, when a new application comes along there is a firewall permission dialog that will let the user block the application from making network requests. It's possible that this dialog was dismissed thus leaving the Insiders release unable to contact the Marketplace.

Okay now it works after adding it to windows firewall exception. The stable version is not in there but works.

cc @egamma

Hold the phones. It is not the firewall but Kaspersky Security 10. -.-

Webcontrol, Mail-Anti-Virus and IM-Anti-Virus is blocking Visual Studio Insiders

Good to know we have a culprit.

Also, it is an invisible one, no entries in any reports of Kasperski nor in the messages telling you it blocked something.

I can narrow it further down. Kaspersky blocks https traffic for apps that it doesn't know. It doesn't report that nor gives feedback to the user, it just cuts off the connection. If you disable the supervision on port :443 the extension connection works.


"self signed certificate" issue when creating new DB connections

I noticed in v0.22.10 when I create a new connection I get a "self signed certificate" issue when testing the connection. This seems to be due to the "rejectUnauthorized" setting under "node-pg driver specific options". It says that the default behavior should be set to True but it should only have an effect if "requestCert" is set to True (requestCert is False by default).
When I check and uncheck the "rejectUnauthorized" setting my connection works as it did before the update (this forces that setting to be False. This seems like a bug since "rejectUnauthorized" should only have an effect if "requestCert" is set to True (it is not by default).

Tried installing for the first time running against a local pgsql instance and get the same error. Checking / unchecking makes the test work, but doesn't work to actually run any queries which give the same error message.

I also encountered this issue. I discovered that you can only work around the issues by check and uncheck both rejectUnauthorized and requestCert. Then everything works after this hack.

You can manually edit the settings and remove the pgOptions.ssl stuff which also disables the use of SSL.
Remove this:
"pgOptions": {
    "ssl": {}
},

Or change it to:
"pgOptions": {
    "ssl": false
},

Quick note to get the fix above to work cause this had me confused, the "pgOptions" setting (I thought it must be a sqltools setting) is inside the connection settings so you fill everything with the gui, then select Open settings at the bottom and then Edit in settings.json there you'll find the part that needs to be modified, works like a charm.

For RDS (and all other postgres with required ssl) there is a workaround
$ export NODE_TLS_REJECT_UNAUTHORIZED='0'
$ code .
or setup NODE_TLS_REJECT_UNAUTHORIZED='0' environment variable in any other way.
It'll be great to have option to allow unauthorized certs to be used without providing CA directly in connection settings.


Added validation on streetNumber

https://nanopay.atlassian.net/browse/NP-1868
Added a validation on streetNumber itself so that wizard view's next/save buttons become disabled when error message thrown.

no necessary


Allow filtering arrays with numeric keys

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#18395

Thanks!


Allow custom docker images

FEATURE REQUEST
It would be nice if we could point to a custom, modified variant of the provided docker images.
For instance, related to BiaPyX/BiaPy#64, I tried to patch the code in the local docker image pulled from the biapyx/biapy repo with the same changes. I then committed the new image (barely modified from the original, now new entrypoint, etc.). I tagged the newly commited image identically to the previous one: biapyx/biapy:latest-11.8, however, I could not get (or trick) the GUI to use it since it checks the Repo Digest, which my custom image does not have since I haven't pushed it to the biapyx/biapy repo.
I tried cloning and modifying the GUI repo code to not do the check, but unfortunately while I could get the GUI to run (MacOS 14.3.1, sonoma), the text was not visible (it was all there, but all white):

Therefore, I was stuck using the provided .app bundle, which works great (text renders properly, etc), but I could not modify the code to skip the docker image check.

Thank you for your comment and your interest in BiaPy. I completely understand what you're saying, but the GUI is designed for users without a computer science background, so they can run BiaPy simply and clearly. Adding this will create confusion for them. A user, like in your case, who can delve into the code for a bug, fix it, and create a Docker container, can run the modified BiaPy code in other ways, such as through the command line or directly with Docker.

Got it. If you'd allow to make one clarification on the idea: my original desire was to patch biapy in the docker image you provide, commit and create a new image, and then share the new image with others in the lab, along with the workflow file and model weights I generated. My preference was to share just those three things I made outside of the GUI, and have them use the GUI directly (so that they don't have to work with the CLI or notebooks). In this particular case it was just a minor bug fix, and I didn't want them to have to wait until the new .app bundle was released before using the workflow file and trained model I generated from the GUI.
One of my lab members pointed out to me that while sharing a workflow file and model weights is fine, sharing arbitrary docker images may introduce security issues that you all would not be comfortable with. For that reason, along with the additional complexity that custom images may introduce, I would totally understand now wanting to incorporate this feature. Just wanted to clarify on my original intent.

Thank you for your understanding. I'm closing this issue.


All Resources page - wide-ranging results

Can we isolate resources in this listing?
This should just be resources. Some blog posts, howto and, what is are currently displaying.
The first three results are
/resources/1
/resources/26
/resources/27
which link to empty pages.

OK, should be fixed!


ChannelCandidateInvestigator - how does it work ?

Looking in the logs I noticed this:
plugin-clboss: ChannelCandidateInvestigator: Best candidates: 02df9bc4315c733658aaa910e7a857b0c34a1541dfb31bdffdd5eefa08c9da0f26
(24), 03afdfd5020decc582a9aedfc5190403117ec83cc0e5993a1e5bfb8448b7c5ee59(24), 029da80a069c1e6e854e127c022d4d6a725b3a8a5e7feb297e9ae1f336a2b74a8e(24), 03505902c83
ba1971c44e15736db0843271ec50a7741d0b15122f9e0d6d8a7ca98(24) ...

The first few nodes have 1 or 2 channels with very little capacity. I think it'll try to open a channel to 02df9bc4315c733658aaa910e7a857b0c34a1541dfb31bdffdd5eefa08c9da0f26 when fees drop but what's the reasoning behind it ?
How does the ChannelCandidateInvestigator work ? What determines if it's a good candidate ?

The candidates are chosen by ChannelFinder algorithms:

ChannelFinderByPopularity - looks for nodes with high number of peers, and selects one of the peers of popular nodes (not the popular nodes directly, but the popular ones tend to also be the peers of each other).
ChannelFinderByDistance - runs a Dijkstra from your node, and selects nodes that are distant from you (i.e. high fees coming from you).
ChannelFinderByListpays - looks at your payments, and selects nodes according to the number of payments you made to that node.
ChannelFinderByEarnedFee - looks at your peers and how much their out_fee_msat_per_day is, and selects a peer of high-earnings peer.

Candidates are actually pairs of nodes: the proposal, which is the actual node to be channeled with, and the patron, a node which we think is a good reason why we should connect to the proposal.
Here is how the above algorithms indicate as patron:

ChannelFinderByPopularity - the popular node is the patron.
ChannelFinderByDistance - the Dijkstra algo generates a shortest-path tree.  The patron of a distant node is its parent (a peer of that node that is one step closer to us than the leaf) in the shortest-path tree.
ChannelFinderByListpays - no patron --- it passes candidates through ChannelCandidateMatchmaker, which selects an existing candidate, gets the shortest path between the new candidate and the old candidate, and selects the first hop from the new candidate as the patron.
ChannelFinderByEarnedFee - our peer that had high earnings is the patron.

The patron is important because that is how we estimate the flow of the candidate.  It is not shown in most of the ChannelCandidateInvestigator logs, or in the clboss-status output, but it would probably be a good idea to do so.  The patron is stored in the database along with the proposal (i.e. the actual candidate).
After that, they go through ChannelPreinvestigator, which does two things:

check if it is up right now.
check if it has capacity, by using the Dowser.  The Dowser estimate must be at least 5mBTC.

The Dowser estimates the capacity of the proposal towards the patron.  This is done by doing a few getroutes from the proposal to the patron, finding the smallest channel along the path, then taking its capacity and dividing it by the route length + 1.  It only takes into account short routes of 1 or 2 hops, and sums up all the estimated per-route capacities.
You can access the Dowser algorithm via clboss-dowser.  First arg is the proposal, second is the patron.
Then, they enter ChannelCandidateInvestigator, which only really checks their uptime (i.e. we can connect to them).
Later, when the ChannelCreationDecider decides that creating some channels would be nice, this will trigger the ChannelCreator, in particular the ChannelCreator::Planner.  The planner takes the list of candidates from ChannelCandidateInvestigator, and runs the Dowser over the top candidates to determine their flow.  In particular, it checks if the Dowser result is at least 5mBTC, and if the Dowser results in less than that, will reject the node from channeling.  It then uses the Dowser result to tell it how much to put in a channel to a candidate.
Looking at your log, 02df9bc4315c733658aaa910e7a857b0c34a1541dfb31bdffdd5eefa08c9da0f26 is indeed a low-capacity node, but it is connected to rompert, which is fairly popular, so it might have been ChannelFinderByPopularity.  The ChannelFinderByDistance also tends to select low-capacity nodes, since distant nodes tend to be low-capacity.
(It would probably have been a good idea to record the algorithm that proposed a node for channeling lol).
On the other hand, we can observe that patrons are always peers of the proposal.  So if rompert (02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b) is the patron of the proposed node, we can ask the Dowser about its estimate, which on my system is:
$ lightning-cli clboss-dowser 02df9bc4315c733658aaa910e7a857b0c34a1541dfb31bdffdd5eefa08c9da0f26 02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b
{
   "amount": "49250000msat"
}

That is below the 0.005BTC limit, meaning the ChannelCreator::Planner will reject the node if ChannelCreationDecider decides to create channels now.  Looking at 1ml, the 02df9bc4315c733658aaa910e7a857b0c34a1541dfb31bdffdd5eefa08c9da0f26 proposal seems to have had some small 0.001 channels in the past, which are closed now; at the initial preinvestigation it probably had just enough capacity to squeeze past the 0.005BTC minimum.
ChannelCandidateInvestigator does not check Dowser flow, since there is the possibility of a channel shock where a node closes most of its channels, but then re-establishes them later, so I did not want to eliminate candidates just due to low flow until just before the ChannelCreator::Planner gets to them.  ChannelCandidateInvestigator is strictly about uptime, since investigating uptime is really what is time-consuming --- estimating flow is something you can do just before actually making a channel, and is more relevant during actual channel creation.
(on the other hand, users of clboss might worry, so maybe I can just drop low-flow candidates at ChannelCandidateInvestigator; I do not expect channel shocks to be common, but who knows? It happened in the past...)
In any case, even if we end up making a channel to it --- for example if it makes a few more small channels in the vicinity of rompert --- then Dowser will still give it a fairly low flow of about 0.005BTC, so clboss will make a channel to it that is only about 0.005BTC, which is a reasonable risk, as it has a 0.01 channel with popular node rompert.

@ZmnSCPxj Thanks for the detailed description, it makes more sense now. I guess it would be valuable to turn the above into a document, either appended to the README or created inside a docs folder for the noobs like me. I can work on it if you think it's useful?

Sure, I suggest putting in a new docs/ directory, and link from the README.md.

There is probably a lot more I need to explicate in its algorithms as well.  Most of my notes are scattered in various comments in lots of scattered modules.  I suppose that is the weakness of the overall bus-based architecture --- algorithms involve the behavior of lots of small modules, and you have to kinda be able to imagine the interactions between the many small modules.

@ZmnSCPxj - are you available on IRC or something like that? I have some questions wrt fees and don't want to spam you with issues on GH.

Sorry, had to do some reinstalls, will set up IRC again.

See also: https://zmnscpxj.github.io/clboss/03-investigation.html


Use github issue templates to minimize erroneous RFC proposals

There are quite a few submitted RFCs that do not follow the guidelines or use the template in this repo. Perhaps a github issue template may be used to pre-fill the issue with all the relevant information, instructions, and a pre-RFC checklist and may help minimize these "fly-by" RFCs.
https://help.github.com/en/github/building-a-strong-community/configuring-issue-templates-for-your-repository

There are no requirements for issues in this repo whatsoever, and I don't think there's a desire to change that. If there were, this would be a great idea, but doing this would, IMHO, be misleading.

Actual RFCs would be via PR; is it worth having a GitHub template for those?


closes #11

Demo for validation of classification including feature selection

I think the demo should be changed.

The sample and the loss functions should be called directly without storing their handles in a variable.
Use proc_flaten on fv instead of explicit reshaping.
Add a proc_ function for feature selection, e.g., transfer proc_fs_FisherCriterion from the old toolbox
Demonstrate correct validation in two ways: (a) performing feature selection via the 'proc' of crossvalidation (I hope that works), and (b) explicit cross-validation as now. See also the demo_feature_selection.m of the old toolbox.


Inequality bugfix

When constraint lower-bound is equal to its upper-bound add_linear_ineqaulity_constraint gives a wrong message. This PR will fix this issue and adds inequality constrain constraint as equality.

Codecov Report

Merging #904 (edce3da) into main (4007db9) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #904      +/-   ##
==========================================
+ Coverage   82.80%   82.83%   +0.03%
==========================================
  Files          79       79
  Lines        6664     6664
==========================================
+ Hits         5518     5520       +2
+ Misses       1146     1144       -2

Impacted Files
Coverage Δ

dimod/binary/binary_quadratic_model.py
91.01% <100.00%> (ø)

dimod/discrete/discrete_quadratic_model.py
95.45% <100.00%> (+0.56%)
:arrow_up:

Continue to review full report at Codecov.

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

This closes #905?

yes it closes #905

Can you add a release note?


no magic added

I can't add the undying enchantment to the armor. Is this a mistake or am I doing it wrong?

versiyon amethyst_imbuement-1.20.1-03

Undying isn't an armor enchantment. It it added to the Totem of Amethyst only.

I understand. I have the spell descriptions mod and in the description it said it was added to the armor. I guess this explanation fooled me :D


NATSTimeoutException when we supply a non existing or wrong cluster id

Hi,
not sure if i m going to get my answer as i m not seeing a lot of activities around Nats Client using .net,
while i m trying to use Nats streaming server i start getting a misleading exception ( NATSTimeoutException )
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout)
at NATS.Client.Connection.request(String subject, Byte[] data, Int32 offset, Int32 count, Int32 timeout)
when i provided a wrong cluster id , is this the intent or just triggering the wrong exception
Thanks

I have encountered this one as well and it can be somewhat difficult to track down if you are not aware of the importance of cluster id. However a NATSTimeoutException, IMO, is the correct exception. An error message saying that this coule be because if the IP/Hostname, Port OR cluster id might help narrowing this down earlier.

This issue seems to be a common one.  Would something like a ClusterUnavailableException with the cluster name in the message sound reasonable?

thanks for your answers guys
yes it make more sense to have an exception that contain a cluster as we need to separate exception between nats streaming connection (IStanConnection)  and nats connection (IConnection)  because nats connection doesn't really care about the cluster ID
Thanks

Hi all, I created a PR for this, #111 , does this look like it'll help?  After further thought, I didn't want to change the exception because that would be a breaking change in the API.

yes it will diffenetly help and better than the previous exception
thanks Colin for this enhancement


npm install fails on nyc installation

npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><3.0.0 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Linux 3.13.0-88-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! node v6.3.0
npm ERR! npm  v3.10.3
npm ERR! path /test-app/your_new_app/node_modules/nyc/node_modules/foreground-child/node_modules/which/bin/which
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod

npm ERR! enoent ENOENT: no such file or directory, chmod '/test-app/your_new_app/node_modules/nyc/node_modules/foreground-child/node_modules/which/bin/which'
npm ERR! enoent ENOENT: no such file or directory, chmod '/test-app/your_new_app/node_modules/nyc/node_modules/foreground-child/node_modules/which/bin/which'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent

Sorry people. This was happening because I was using vagrant and trying to load the project into the synced folder. The problem here is that locking isn't supported when syncing files in this way and so npm install fails (and will fail) in weird places.
To get around this, for anyone else who runs into the same issue and wants both synced files and locally installed node modules, you should create a node modules folder outside of the synced folder and link to it. Something like:
$ mkdir ~/node_modules/
$ cd ~/your_new_app/
$ ln -s ~/node_modules/


[Feature] The poseexplode function support like hive.

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
Need poseexplode function support, the implementation of this function in hive https://blog.csdn.net/dzysunshine/article/details/101110467
Use case

Related issues
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hello, I want to try this issue, could you assign it to me? Plz~

@luozenglin  I'm really sorry that the task was unassigned because I made a mistake. Could you please assign this task again? Thank you.

@luzhijing I'm really sorry that the task was unassigned because I made a mistake. Could you please assign this task again? Thank you.

I raised a discussion about this question, but no one has answered it, and the function of returning two columns may not be possible at the current stage. The link is as follows:https://github.com/apache/doris/discussions/12680

Now doris can't return two columns, so I don't know  such output can meet your needs？

https://github.com/apache/doris/pull/43221


Another implementation of Randomized

@mohsen-ghaffari1992 in a push to try to resolve bug #59, I have proposed to replace Randomized with Randomized2 based on probula.  This has actually uncovered some design flaw with Randomized (it was not properly abstracted so its design details were leaking to symsim code). As a result this required a massive amount of changes.
I had no time to try the entire test suite, so I do not even know if this is correct  (and it may have positive or negative impact on convergence)
But simple maze seems to work, and I was able to run a million episodes.  So try using this branch for your experiments.

@wasowski
Thanks!
I tested a maze 100x100 for 1000 episodes with timeout=2000, it perfectly is running. I still did not study how good is the results but it is very hopeful that now we can run both long trials and high number of episodes.

@mohsen-ghaffari1992
there are many tests failing so I am trying to chase bugs and push corrections to this branch. I think we will merge this branch after the deadline.

concrete.simplebandit.Bandit is an Agent.agent.observe (step (s) (a)._1) ∈ ObservableState *** FAILED *** (44 milliseconds)
[info]   ArrayIndexOutOfBoundsException was thrown during property evaluation.

Gaussian Simple Bandit seems to fail because of a bug in spire (issue #233). Do we need Gaussian for the paper anywhere @mohsen-ghaffari1992 ?

No, we do not need the gaussian.
Best,
Mohsen
From: Andrzej Wąsowski @.>
Date: Saturday, 13 January 2024 at 13.57
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>, Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
concrete.simplebandit.Bandit is an Agent.agent.observe (step (s) (a)._1) ∈ ObservableState *** FAILED *** (44 milliseconds)
[info]   ArrayIndexOutOfBoundsException was thrown during property evaluation.
Gaussian Simple Bandit seems to fail because of a bug in spire (issue #233https://github.com/itu-square/symsim/issues/233). Do we need Gaussian for the paper anywhere @mohsen-ghaffari1992https://github.com/mohsen-ghaffari1992 ?
—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890449338, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWRKZCXWWFLTSVGGZS43YOKAENAVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQGQ2DSMZTHA.
You are receiving this because you were mentioned.Message ID: @.***>

OK. Then we fix this later.
Now cartpole was failing because of an assertion, but as far as I can see the assertion was wrong (the last requirement for state invariant was
require (pv <= PvMin)

I switch this to PvMax and now seems to work, but this one still uses memory intensively. Even with 10K episodes I am running out of memory.

Do we use cartpole in the paper?
With what number of episodes?
Did it work before or also timed out?

Thanks! That is fine.
No problem. We do not evaluate on CartPole.
Best,
Mohsen
From: Andrzej Wąsowski @.>
Date: Saturday, 13 January 2024 at 14.28
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>, Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
OK. Then we fix this later.
Now cartpole was failing because of an assertion, but as far as I can see the assertion was wrong (the last requirement for state invariant was
require (pv <= PvMin)
I switch this to PvMax and now seems to work, but this one still uses memory intensively. Even with 10K episodes I am running out of memory.

Do we use cartpole in the paper?
With what number of episodes?
Did it work before or also timed out?

—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890459372, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWROT5MSI23Z2HCQJLVLYOKDZJAVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQGQ2TSMZXGI.
You are receiving this because you were mentioned.Message ID: @.***>

but I think there is something buggy, which means me question on all results.  WindyGrid and CliffWalking are also not able to complete even one episode, and these should work just fine! CliffWalking works very well in ADPRO on an extremely similar design, so something is fishy.

I checked the CliffWalking and the problem is initialize function. You check it is not final but sometimes it is out of board.
I believe the same issue for WindyGrid.
Best,
Mohsen
From: Andrzej Wąsowski @.>
Date: Saturday, 13 January 2024 at 15.50
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>, Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
but I think there is something buggy, which means me question on all results. WindyGrid and CliffWalking are also not able to complete even one episode, and these should work just fine! CliffWalking works very well in ADPRO on an extremely similar design, so something is fishy.
—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890478560, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWROG434B27O6RP6RA23YOKNMNAVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQGQ3TQNJWGA.
You are receiving this because you were mentioned.Message ID: @.***>

Hmm... this is weird.  The test for validity (CliffWalkingIsAgent) is passing, and we only produce states within the board? How can you see that this is happening?
(I suspected the dreaded function tailRecM in Randomized2, but looking at it for several hours, I start to believe that it is correct).

I just printed the states that are sending to step function and noticed the value is out of bound. Since, state is created by initialize function, we should check it. The point is that we check the state is not final, but we do not check whether it is valid!
Best,
Mohsen
From: Andrzej Wąsowski @.>
Date: Saturday, 13 January 2024 at 20.00
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>, Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
Hmm... this is weird. The test for validity (CliffWalkingIsAgent) is passing, and we only produce states within the board? How can you see that this is happening?
(I suspected the dreaded function tailRecM in Randomized2, but looking at it for several hours, I start to believe that it is correct).
—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890716128, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWRJ536CP6ZOZUIAKOZTYOLKWFAVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQG4YTMMJSHA.
You are receiving this because you were mentioned.Message ID: @.***>

But maybe I do not understand what valid states are.
  def initialize: Randomized2[CWState] = { for
    x <- Randomized2.between (0, BoardWidth  + 1)
    y <- Randomized2.between (0, BoardHeight + 1)
    s = CWState (x, y)
  yield s }.filter { !this.isFinal (_) }

I do not understand how this can create states outside the board. Also I have assertions now checking when the state is created in CWState, so if I try to create an illegal state everything crashes with an exception, but I do not see these exceptions. For example:
scala> CWState(20,20)
java.lang.IllegalArgumentException: requirement failed: Out-Of-Width x: ¬(20 ≤ 11)
  at scala.Predef$.require(Predef.scala:337)
  at symsim.examples.concrete.cliffWalking.CWState.<init>(CliffWalking.scala:18)
  at symsim.examples.concrete.cliffWalking.CWState$.apply(CliffWalking.scala:16)
  ... 42 elided

So I think this is not happening.  We already had this problem that everything was hanging during the first episode (or it seems, after some debugging that this might be during the second episode). But I do not remember what was the reason.

Oh - some progress.  I just discovered that I was misguided. It is the evaluation that is hanging, not learning! I have not been looking at the evaluation code, as I was sure it was learning.  I will look into eval. It might be the same problem as with maze, that the early policies are bad and they need a timeout.

If it is hanging, then the issue is probably the same.
When I was testing, it reported error that I found it is for required in move function. That is how I became with initialize. However, the error that I am talking about can be due to moving from one step to another. Because we do not check the valid state for return of the move.
Best,
Mohsen
From: Andrzej Wąsowski @.>
Date: Saturday, 13 January 2024 at 20.29
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>, Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
Oh - some progress. I just discovered that I was misguided. It is the evaluation that is hanging, not learning! I have not been looking at the evaluation code, as I was sure it was learning. I will look into eval. It might be the same problem as with maze, that the early policies are bad and they need a timeout.
—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890747332, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWRJWHVWDZHIXTCUGMSTYOLOC5AVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQG42DOMZTGI.
You are receiving this because you were mentioned.Message ID: @.***>

Now we check it every time (in cliffwalking) when a new state is constructed. So this checks both when moving and when initializing. I will look into eval.

[like]  Mohsen Ghaffari reacted to your message:

From: Andrzej Wąsowski @.>
Sent: Saturday, January 13, 2024 7:36:34 PM
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>; Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
Now we check it every time (in cliffwalking) when a new state is constructed. So this checks both when moving and when initializing. I will look into eval.
—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890748646, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWRMALIY3RMSHKIPCNVLYOLO4FAVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQG42DQNRUGY.
You are receiving this because you were mentioned.Message ID: @.***>

OK. I added time horizon to mountaincar, cliffwalking, and windygrid. Now everything seems to behave. I think this is ready for your experiments.  I hope this branch will allow all the experiments you need.  I definitely hit heap problems still, but I believe we can handle larger cases now.
Some tests are still failing (about 10). All of them are due to randomness or the spire bug mentioned above (with Gaussian).  I will stop now and move to other things.

Thanks Andrzej!
You put a real effort on this, I appreciate it.
Best,
Mohsen
From: Andrzej Wąsowski @.>
Date: Saturday, 13 January 2024 at 22.50
To: itu-square/symsim @.>
Cc: Mohsen Ghaffari @.>, Mention @.>
Subject: Re: [itu-square/symsim] Another implementation of Randomized (PR #232)
OK. I added time horizon to mountaincar, cliffwalking, and windygrid. Now everything seems to behave. I think this is ready for your experiments. I hope this branch will allow all the experiments you need. I definitely hit heap problems still, but I believe we can handle larger cases now.
Some tests are still failing (about 10). All of them are due to randomness or the spire bug mentioned above (with Gaussian). I will stop now and move to other things.
—
Reply to this email directly, view it on GitHubhttps://github.com/itu-square/symsim/pull/232#issuecomment-1890777127, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASQKWRONGGNBY2OKYXQ5LWDYOL6S5AVCNFSM6AAAAABBY25HNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQG43TOMJSG4.
You are receiving this because you were mentioned.Message ID: @.***>


adding example json file

three example json files TonyStark, Steverogers, and BuckyBarnes.

LGTM, thanks @yuyuan223 !


Something about the submit of test_data results on EvalAi

Hi I have read the submit guideline on Eval Ai. But it seems that the submit guideline is not completive. What should I really record in [results] of the JSON file? Or what format should I use in the JSON file.

@lemon-prog123 Hi, there is something wrong with the guideline. Sorry for that and we will fix it later. The format should be like this:
{
"version": "1.0",
"challenge": "ego4d_looking_at_me",
"results":[
{
"video_id": video id,
"unique_id": unique id for each frame,
"track_id": track id,
"label": predict label,
"score": score
},
...
{
"video_id": video id,
"unique_id": unique id for each frame,
"track_id": track id,
"label": predict label,
"score": score
},
],
}

@lemon-prog123 Hi, there is something wrong with the guideline. Sorry for that and we will fix it later. The format should be like this: { "version": "1.0", "challenge": "ego4d_looking_at_me", "results":[ { "video_id": video id, "unique_id": unique id for each frame, "track_id": track id, "label": predict label, "score": score }, ... { "video_id": video id, "unique_id": unique id for each frame, "track_id": track id, "label": predict label, "score": score }, ], }

Thank you for your answer! And I also want to know that whether the test phase on Eval Ai will be the final dataset used for competition. Or we will have another dataset for this competition.

@lemon-prog123 Hi, there is something wrong with the guideline. Sorry for that and we will fix it later. The format should be like this: { "version": "1.0", "challenge": "ego4d_looking_at_me", "results":[ { "video_id": video id, "unique_id": unique id for each frame, "track_id": track id, "label": predict label, "score": score }, ... { "video_id": video id, "unique_id": unique id for each frame, "track_id": track id, "label": predict label, "score": score }, ], }

Thank you for your answer! And I also want to know that whether the test phase on Eval Ai will be the final dataset used for competition. Or we will have another dataset for this competition.

Yes, it will be the final dataset used for the competition.

I have checked it out and there are exactly 2063259 pictures in videos_challenge. I have unzip the orignal_file again and ensure that there are 2063259 pictures in the test set.  Is that right for the number of test set?

ok I have checked it out. Thank you !


Beta Testing

Profile Information Update
Issue: Error messages "failed to update profile" and "request failed with no response, please try again later" appear when I try to update my profile information.
Steps to Reproduce:

Navigate to profile settings.
Attempt to update profile information.
Observe the error messages.

Hi

Hello @Joy-Adah i wil love to work on this

@Joy-Adah stop creating issue please without permission

@MSghais sorry about that. I assumed it was okay since it was specified in the beta testing issue.

@MSghais Please how do I submit my beta testing bug report?


Inform the user why the content isn't available

Display an informative error about why the content isn't available.
At least:
The file can not be found
The file is in the wrong format
There is no associated content for this verse

@PhotoNomad0 Investigate the potential reasons why content isn't available.

@birchamp For scripture content failures, I found these readily identifiable conditions:

Project manifest not found
Project manifest not valid
Book not found in Project
Scripture Verse not Translated

Please tweak the above messages for comprehension.

@PhotoNomad0

This project manifest failed to load. Please confirm that the correct manifest.yaml file exists in the project.
The manifest for this project is invalid.
This book can not be found in the project.
There is no associated content for this verse.

Test on slow connection to see if it creates flashing error messages.

Notes:
for testing messages:

in single-scripture-rcl styleguidist, in scripturePane, change HebrewExample Object
to simulate untranslated verse, change verse to 177, should see message
reset verse to 166 - should see verse text
to simulate untranslated chapter, change chapter to 151, should see message
change chapter and to 150:6 - should see verse text
to simulate missing book, use invalid projectId, such as psah, should see message:
reset projectId to psa - should see verse text
test invalid organization - change resource.owner to unfoldingWor
reset organization to unfoldingWord - should see verse text

v0.2.0 build cbad9c9
I am seeing exactly what was posted by PhotoNomad0 in the 2 immediately previous comments.
I wrote up issue #163 to point out some related issues.

Works as expected in v0.9.0 build 008141c


(IMPORT FAILED) ComfyUI-LTXVideo

installing with latest update of comfyui error
(IMPORT FAILED) ComfyUI-LTXVideo

I tried it all, easynodes still gives me errors. I saw that there is now a new command for installing packages for portable Comfy, but i honestly don't know where to execute it. I tried it inside the customnode folder, the python_embed ... Please help.

Run command with administrator privileges

Ok solution is to install via comfy manager. Didn't think of that. Maybe also code was update, I'm not sure. Yesterday it didn't work after trying a lot when just git cloning.
Now i can play around with the provided workflows finally.

Here's your response formatted for GitHub:
I am experiencing the same issue. I installed the module using the Manager, but it doesn't work.
I've tried the following steps:
Deleted and reinstalled the module.
Updated all dependencies.
Manually installed the requirements.txt file.

Despite this, I encounter an error when installing the ltx_video module. Upon starting ComfyUI, I get the following error:
Traceback (most recent call last):
  File "D:\_ComfyUI\ComfyUI\nodes.py", line 2027, in load_custom_node
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "D:\_ComfyUI\ComfyUI\custom_nodes\ComfyUI-LTXVideo\__init__.py", line 3, in <module>
    from .transformer import LTXVModelConfigurator, LTXVShiftSigmas
  File "D:\_ComfyUI\ComfyUI\custom_nodes\ComfyUI-LTXVideo\transformer.py", line 15, in <module>
    from ltx_video.models.transformers.transformer3d import Transformer3DModel
ModuleNotFoundError: No module named 'ltx_video'

Cannot import D:\_ComfyUI\ComfyUI\custom_nodes\ComfyUI-LTXVideo module for custom nodes: No module named 'ltx_video'

for me this happened due to the removal of the long-deprecated pkgutil.ImpImporter class, the pip command was not working for python 3.12
i installed python 3.11 and it was working after that

(IMPORT FAILED) ComfyUI-LTXVideo
how to fix it? :(

unable to install, same issue.

me too
Cannot import A:\ComfyUI\custom_nodes\ComfyUI-LTXVideo module for custom nodes: No module named 'ltx_video'


Adding Constraints in the argmin version is unclear / missing

@relf thank you for the amazing crate! I just wanted to ask how to add constraints in the latest 0.3 version? I am not sure if this is missing, or unclear in the documentation. Thank you!

Hi. No it is not missing but definitely not very well documented. You have an example on this page where objective and constraint are defined as:
struct ParaboloidProblem;
impl CostFunction for ParaboloidProblem {
    type Param = Vec<f64>;
    type Output = Vec<f64>;

    // Minimize 10*(x0+1)^2 + x1^2 subject to x0 >= 0
    fn cost(&self, x: &Self::Param) -> Result<Self::Output, Error> {
        Ok(vec![10. * (x[0] + 1.).powf(2.) + x[1].powf(2.), x[0]])
    }
}

As you may know, the argmin framework does not handle constrained optimization, but fortunatly I can implement this by defining the output of the cost function as a vector (the first component is the actual cost value, the remaining components are constraints values expected to be positive at the end of the optimization). Here on this example we minimiez subject to one constraint (x0 >= 0) hence the presence of x[0] as the second component of cost function output vector.

Oh nice I see. So the argmin package supports multi-variable objectives? Also, to what extend is it a solution guaranteed to satisfy the constraints? I suppose the traditional cobyla package supports constraints "natively" (by traversing the gradient projection)?

You have to check the argmin solvers but I am not sure there is one handling multi-objective optimizations. In the case of this cobyla crate I use the fact that the CostFunction trait allows to define output as a vector to pass also constraints values.

Also, to what extend is it a solution guaranteed to satisfy the constraints?

Well... The software is provided as is without any warranty 😉

I suppose the traditional cobyla package supports constraints "natively" (by traversing the gradient projection)?

The COBYLA method is a gradient-free algorithm.


LayerList/LayerListViewModel not there yet?

I am using ESRI JS API 4.x in TypeScript (2.1). I'd like to use LayerListViewModel.
It is announced and documented here as part of the 4.2 release
https://developers.arcgis.com/javascript/latest/api-reference/esri-widgets-LayerList-LayerListViewModel.html
Yet I cannot find anything related to the LayerList widget in the .d.ts
Did it not make it in the latest 4.2 code update?

This is now fixed at: https://github.com/Esri/jsapi-resources/blob/master/4.x/typescript/arcgis-js-api.d.ts#L8068-L8071
Thank you for reporting this.


Ask me anything?


Introduce domain flag for non standard cluster domains

This would address #2011 added this to facilitate discussion.

This change is

I signed it!

anything I can do to help this along ?

Ping @bprashanth. Would you please take a look at this? Thanks!

Sorry for the delay LGTM. If you'd like a new image, please send a follow up pr bumping up the image in the Makefile.


FixedChannelPool throw java.lang.IllegalStateException: Channel is unhealthy not offering it back to pool

Hi, i am trying to write some TCP stuff with FixedChannelPool. The documentation says i have to return any channel that borrow from pool even if the channel already dead, but when i do return the channel, FixedChannelPool throw java.lang.IllegalStateException: Channel is unhealthy not offering it back to pool.
Is this bug ?
if i don't release the channel to FixedChannelPool its never give any connection when dead channel count = max_pool_size.
if i just ignore the exception, everything works fine, but i think this should never happend.
Expected behavior
i expect no exception thrown
Actual behavior
throw  java.lang.IllegalStateException: Channel is unhealthy not offering it back to pool.

Minimal yet complete reproducer code (or URL to code)
public class MaybeBug {
    @Test
    public void test() throws InterruptedException, ExecutionException{
        Bootstrap config = new Bootstrap();
        config.remoteAddress("localhost", 1234);
        config.option(ChannelOption.TCP_NODELAY, true);
        config.channel(NioSocketChannel.class);
        config.group(new NioEventLoopGroup(1));
        FixedChannelPool pool = new FixedChannelPool(config, new AbstractChannelPoolHandler() {
            @Override
            public void channelCreated(Channel ch) throws Exception {
                /**
                 * ignore
                 */
            }
        }, 2);
        Channel fresh = pool.acquire().get();
        fresh.close().get();
        pool.release(fresh).get();
    } }
Netty version
4.1.5
JVM version (e.g. java -version)
1.8
OS version (e.g. uname -a)
Windows 10
NB: sorry for bad english.

@fian46 please check https://github.com/netty/netty/pull/6843

Fixed by https://github.com/netty/netty/pull/6843