'NoneType' object has no attribute 'replace'

I got the error when I run my code on Linux Machine as follows:
Traceback (most recent call last):
  File "/c_search_recall/eda_jd.py", line 40, in <module>
    report.to_file('data/jd_eda_report.html')
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/profile_report.py", line 370, in to_file
    data = self.to_html()
           ^^^^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/profile_report.py", line 485, in to_html
    return self.html
           ^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/profile_report.py", line 283, in html
    self._html = self._render_html()
                 ^^^^^^^^^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/profile_report.py", line 400, in _render_html
    report = self.report
             ^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/profile_report.py", line 277, in report
    self._report = get_report_structure(self.config, self.description_set)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/report/structure/report.py", line 387, in get_report_structure
    render_variables_section(config, summary),
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/report/structure/report.py", line 162, in render_variables_section
    template_variables.update(render_map_type(config, template_variables))
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/report/structure/variables/render_text.py", line 100, in render_text
    overview_table_char, unitab = render_categorical_unicode(config, summary, varid)
                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/miniconda3/envs/recall/lib/python3.11/site-packages/ydata_profiling/report/structure/variables/render_categorical.py", line 139, in render_categorical_unicode
    category_alias_name = category_alias_name.replace("_", " ")
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'replace'

python version: 3.11
ydata-profiling version: 4.6.4
Please help me, thx!

HI @Crispinli ,
More detail is required as this seems to be related with the data that you are currently. Can you please provide the code that you are using as well as some information regarding the dataset?

I have the same issue
ydata_profiling in /usr/local/lib/python3.10/dist-packages (4.6.5)
import pandas as pd
from ydata_profiling import ProfileReport
df = pd.read_csv("/content/drive/MyDrive/linkdin_Job_data.csv")
Generate the data profiling report
report = ProfileReport(df, title='My Data')
report.to_file("my_report.html")
Summarize dataset: 100%
 27/27 [00:09<00:00,  1.72it/s, Completed]
Generate report structure:   0%
 0/1 [00:17<?, ?it/s]
AttributeError                            Traceback (most recent call last)
 in <cell line: 9>()
7 # Generate the data profiling report
8 report = ProfileReport(df, title='My Data')
----> 9 report.to_file("my_report.html")
8 frames
/usr/local/lib/python3.10/dist-packages/ydata_profiling/report/structure/variables/render_categorical.py in render_categorical_unicode(config, summary, varid)
137         summary["category_alias_char_counts"].items(), key=lambda x: -len(x[1])
138     ):
--> 139         category_alias_name = category_alias_name.replace("_", " ")
140         cats.append(
141             FrequencyTable(
AttributeError: 'NoneType' object has no attribute 'replace'

@stnvapm Were you able to find a solution for this error?

Hi @stnvapm ,
can you please share the python packages that you have installed in your environment as well as their versions?
If you could share more details on your data as well that would be helpful to reproduce.

I get a similar error when trying to call profile.to_file("report.html") on a profile with a pyspark dataframe (summarizing seems successful:
 in to_file
    data = self.to_html()
 in to_html
    return self.html
 in html
    self._html = self._render_html()
 in _render_html
    report = self.report
 in report
    self._report = get_report_structure(self.config, self.description_set)
 in get_report_structure
    render_variables_section(config, summary),
 in render_variables_section
    template_variables.update(render_map_type(config, template_variables))
 in render_real
    "value": fmt_numeric(
 inner
    return func(arg, *args, **kwargs)
.venv/lib/python3.10/site-packages/ydata_profiling/report/formatters.py", line 248, in fmt_numeric
    fmtted = f"{{:.{precision}g}}".format(value)
TypeError: unsupported format string passed to NoneType.__format__

Here's my pyproject.toml:
[project]
requires-python = ">=3.10"
dependencies = [
    "dask[dataframe]>=2024.10.0",
    "pyspark>=3.5.3",
    "setuptools>=75.2.0",
    "ydata-profiling[notebook,unicode]>=4.11.0",
]

Hope I'm not hijacking this issue, but it seemed stale anyway and also seems to be an issue with NoneType handling.
Also installing ydata-profiling[pyspark] tells me the extra pyspark doesn't exist, so I installed manually.


neon: Add qtbl with vqtbl[1234]{,q} intrinsics

neon: Add qtbl with vqtbl[1234]{,q} intrinsics

My tbx commit seems to be included but I don't know why. I completely deleted my branch and even my fork, pulled from master and recreated the branch from master.

@nemequ I think I have finished with all the tbl, tbx, qtbl and qtbx intrinsics now. As they weren't merged I kept on piling stuff in ;-)
They are now out of date wrt master, do you want me to squash them and rebase on the current master?


Configuring through ARM template is possible

The SQL advisor configuration is documented in the ARM template reference. I am able to get this to work, although it seems to fail randomly with Server automatic tuning settings from previous request have not propagated to all databases yet. Please try again in few minutes..

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5ef45327-7cdf-fc60-2a04-b785bc12f4cf
Version Independent ID: b4f8c915-9651-14df-92d7-fa1387780a41
Content: Enable automatic tuning for Azure SQL Database
Content Source: articles/sql-database/sql-database-automatic-tuning-enable.md
Service: sql-database
Sub-service: performance
GitHub Login: @danimir
Microsoft Alias: danil

Thanks for the question. We are currently investigating and will update you shortly.

@sugarjig Are you making multiple changes via multiple executions of the ARM template or are you making a bunch of changes with one execution of the ARM template? Or is this the case randomly, as you stated? Is Geo-Replication enabled in this environment? I am trying to get a sense of what might be going on, either pending updates have yet to be implemented or the Auto Tuning process is running during the ARM template execution, as it periodically gathers information to be able to make recommendations. Thank you for any additional details can provide.
Also, it might be worth adding ARM template option to the documentation, if it is a legitimate means to manage this functionality. I am making the content owners aware (@danimir @CarlRabeler) of this doc-enhancement opportunity.

I've tried different variations of Enabled, Disabled, and Default, but each time I am setting values for CreateIndex, DropIndex and ForceLastGoodPlan in a single execution of the ARM template. Each time, it looks like one of them succeeds, while the other two fail with the error message. Geo-replication is not enabled in this environment.
I've going to wait overnight and try again in the morning to see if it is indeed an issue with propagation.

Each time, it looks like one of them succeeds, while the other two fail with the error message.

Meant to add: the one that succeeds changes between runs.

@Mike-Ubezzi-MSFT  This option is not officially supported. Although customer found it through the ARM, this was not officially tested nor staffed to be supported. Thanks.

I find this very confusing because the ARM reference documentation includes this feature (https://docs.microsoft.com/en-us/azure/templates/microsoft.sql/2014-04-01/servers/advisors). I'm surprised that other customers haven't run into this issue, given that the API version is from 2014. Should we suggest to the ARM team that they remove this documentation?
On a more general note, I have come to expect that all resource configuration is possible in ARM templates. I find it very frustrating when this is not the case. I realize it's possible to achieve this particular configuration via the REST API and Powershell cmdlets, but I find the declarative paradigm of ARM to be superior. I know I'm not alone in this, and I hope Microsoft dedicates some resources to making Azure more consistent in this regard.

@sugarjig I hear you and agree that all resources should be ARM configurable, but this is not the case. @Mike-Ubezzi-MSFT Can we ask the ARM folks to remove this reference please?

@tfitzmac Can you help out here? The document: Microsoft.Sql servers/advisors template reference is detailing functionality that is not fully supported. The functionality is configurable via ARM template but updates to that functionality via ARM should not be documented. There is an ask (per previous comments) to remove reference to this page. @danimir Do you want this entire doc removed? Thanks!

I'll remove it.

@sugarjig We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.

The advisors reference article has been removed.


Snakefile: automatically copy config.default.yaml on first use

analogous to PyPSA-Eur
https://github.com/PyPSA/pypsa-eur/blob/bb3477cd693d1c8e77e75e61a1a7e1a4647e6a3c/Snakefile#L1-L5

Leave as is because PyPSA-Eur-Sec installation also requires setting config.yaml of PyPSA-Eur which does not happen automatically if integrated as submodule.


update non-command role requirements

About the PR

Part 2 of #17366. Emisse requested that it also double intern caps, map specific tuning of that might need to happen after this PR
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Non-command role requirements have been modified.

K A C H O W


The problem about 'config.file'

When I wanted to use image_demo.py for image inferencing, I found that the config file was missing, could you please provide this file? Thanks a lot

The config file is located at ./configs/qpic/qpic_r50_150e_hico.py. I do not understand what is missing.

Sorry, could you give a sample command to explain how to operate image_demo.py? Thanks a lot.

Sorry for late reply. I commit the latest version and you can refer the README.md for more details.

1

------------------ Original ------------------
From: "Xepsiloner";
Date: 2022年4月23日(星期六) 下午5:52
To: "yingkaining/mmhoidet";
Cc: "kaining"; "Comment";
Subject: Re: [yingkaining/mmhoidet] The problem about 'config.file' (Issue #1)

Sorry, could you give a sample command to explain how to operate image_demo.py? Thanks a lot.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


remove some unused variables, update linter config

Thank you @larshp . Regards, Krishna

will probably find more, but I'm trying to keep the changes small so they are easier to review 😄


Run antivirus scan - this is not shown in the UI

Hi,
i created a custom detection rule triggering an Antivirus Scan on a Server 2019.
you can see Eventid 1000,1001 but no Info in the UI during the Scan (like if you start a quick scan manually)
maybe this is on purpose, but then the Info could help.
Further it would be nice to know which OS are supported, i couldn´t get 2016 or below to scan. But i guess that´s an OS limitation?
Kind Regards Patrick

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dbf510e0-742e-fce6-787e-18eb8a22f42d
Version Independent ID: 23b77843-cf19-aacb-47af-1f51792c76a0
Content: Create and manage custom detection rules in Microsoft 365 Defender
Content Source: microsoft-365/security/defender/custom-detection-rules.md
Product: m365-security
Technology: m365d
GitHub Login: @schmurky
Microsoft Alias: maccruz

@schmurky Please help us in resolving this issue. Thanks

Hello, this has to do with MDE logic, I suggest contacting for these two questions. MDE support.


Custom user selector

Hey!
I am currently using slack API's user_select component for user selection.
Is it possible to create a user selector with one ot two more additional options?  Roles are just static strings for my the slack app. I haven't found any built in solutions yet but I think it might  be a custom selector implementation. Should I implement the whole user list functionality handled by slack client? (I mean user pagination, filtering for keypress event  n  the text area and so on?)
Reproducible in:
pip freeze | grep slack
python --version
sw_vers && uname -v # or ver
The slack_bolt version
slack-bolt==1.5.0
slack-sdk==3.5.0
Python runtime version
3.8.5
(Paste the output of python --version)
OS info
Darwin Kernel Version 20.4.0:
(Tell what actually happened with logs, screenshots)
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hi @tomeszmh! Yes, it's definitely possible and you're correct: to accomplish this, you'll have to implement your own custom selector. 🙂
Using an external_select element, when the user clicks on the select, you can populate the options as you see fit. To make things quicker and more efficient, you might consider grabbing the entire list of users and storing those beforehand. No need to implement the keypress and text area that displays the character the user is searching the list on: you get those out of the box!
Here is an example of external_select that might be useful. Let us know if this helps!


Crate models!

https://user-images.githubusercontent.com/11138884/143252116-210b4895-c4b5-48fd-b23d-ed7b54534bce.mp4
https://user-images.githubusercontent.com/11138884/143252123-19798d16-dd5a-4562-b77d-d6a648bcd818.mp4
Crates!

These are super hot, awesome work


updated the version and sha

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


creating virtual wires of type 'bridge' fails

For the topo file in attachment, and with version 0.13.0, the connection of bridge virtual wire fails with the error:
INFO[0001] Creating virtual wire: srl04:e1-1 <--> testbridge:eth1
ERRO[0001] could not lookup "srl04": Link not found
The same topology file works with e.g. version 0.9.2.
Am I missing something ?
Topology file and debug log in attachment.
topology.yaml.txt
deployment_log.txt

Hey @ackew
yes, there was a typo that was fixed in #385
the release with these changes was not yet issued, as a workaround you can swap the endpoints in your topo file:
    - endpoints: ["testbridge:eth1", "srl04:e1-1"]

Thanks Roman.  Crystal clear !
From: Roman Dodin @.>
Sent: Monday, April 19, 2021 1:44 PM
To: srl-labs/containerlab @.>
Cc: Acke, Willem (Nokia - BE/Antwerp) @.>; Mention @.>
Subject: Re: [srl-labs/containerlab] creating virtual wires of type 'bridge' fails (#388)
Hey @ackewhttps://github.com/ackew
yes, there was a typo that was fixed in #385https://github.com/srl-labs/containerlab/pull/385
the release with these changes was not yet issued, as a workaround you can swap the endpoints in your topo file:
- endpoints: ["testbridge:eth1", "srl04:e1-1"]

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/srl-labs/containerlab/issues/388#issuecomment-822402802, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGBFZIKI6M34YNPTR2B4TMDTJQJQDANCNFSM43FT3HZA.


New syntax and semantics of the @_specialize attribute

This PR implements the syntax and semantics of the new @_specialize attribute.
One can use the @_specialize attribute to force creation of specializations of a given function for a given set of generic parameters.
The new @_specialize attribute has a syntax like this:
@_specialize(exported: true, kind: full, where K == Int, V == Int)
@_specialize(exported: false, kind: partial, where K: _Trivial64)
func dictFunction<K, V>(dict: Dictionary<K, V>) {
}

If exported is set, the corresponding specialization would have a public visibility and can be used by clients.
If exported is omitted, it's value is assumed to be false.
If kind is full it means a full specialization.
If kind is partial it means a partial specialization.
If kind is omitted, its value is assumed to be full.
One can use in the where clause of the @_specialize attribute the newly introduced layout constraints like _Trivial, _Trivial(N) or _RefCountedObject, etc.
Partial specializations are supported syntactically, but their SIL level support is not committed as part of this PR. It will be provided as a separate PR.

@slavapestov This is the second chunk of the #6486. It contains only the @_specialize related changes. No significant code changes compared to the parent PR. Could you please review?

@swift-ci Please smoke test

I see the SIL changes are in later commits. Do the earlier commits for the parser change compile and pass tests already?

@slavapestov

I see the SIL changes are in later commits. Do the earlier commits for the parser change compile and pass tests already?

I'm not sure what you are asking about. The tests in the last commit cover both Swift and SIL parsing of the @_specialized attribute.

I'm not sure what you are asking about. The tests in the last commit cover both Swift and SIL parsing of the @_specialized attribute.

Sure, but the earlier patches that only touch the parser could go in independently of the later changes right? Just trying to get as much of this checked in as quickly as possible :-)

Just looked over the latest patches. The parser/serialization/AST bits LGTM now.

@slavapestov I removed all Generics.cpp and related changes. The PR contains now only parsing, type checks and tests. Eager specializer and related tests are disabled for now.

@swift-ci Please smoke test

@swift-ci Please smoke test

This looks great. I added a couple of minor nit-picks but feel free to address them in a follow-up PR.

@slavapestov Thanks for reviewing the new version of the PR. I'll add these last two fixes you suggested.
BTW, any news on the conformance lookup issue? Have you looked at the example I sent you? Is it a bug in the conformance lookup implementation? Or may be the SubstitutionMap was created in a wrong way?

I saw your note about the conformance lookup issue but I haven't investigated it yet. Thanks for the detailed explanation though!

@swift-ci Please smoke test

@swift-ci Please smoke test Linux


Implement watcher

Watcher that will run script every n minutes/hours/days and alert user that the new offer is avaliable

#9 - resolved ✅


Ignoring setting when date_formats is stated

dateparser.parse("201704", date_formats=['%Y%m'], settings={'PREFER_DAY_OF_MONTH': 'first'})
Out[83]: datetime.datetime(2017, 4, 30, 0, 0)

It should have been:
datetime.datetime(2017, 4, 1, 0, 0)

Closing this a it is fixed since dateparser==0.7.4 (https://github.com/scrapinghub/dateparser/pull/611)


fix: set DD_SERVICE env

We need to set an explicit name for the service because, without this env datadog tries to guess it and now uses its chart name. It was ok when we had only one deployment. Now, we have more than one, and we should be able to split those logs/metrics with the service field.

Codecov Report

Merging #397 (757821b) into batching/main (a1750ed) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@              Coverage Diff               @@
##           batching/main     #397   +/-   ##
==============================================
  Coverage          43.22%   43.22%
==============================================
  Files                 30       30
  Lines               7272     7272
==============================================
  Hits                3143     3143
  Misses              4129     4129

Flag
Coverage Δ

test
43.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 a1750ed...757821b. Read the comment docs.


Increase nofiles

This PR was successfully deployed to production by CircleCI build 182 at 2017-03-31 14:56 BST.


Add e2e tests for apps in any namespace feature

Add e2e tests for apps in any namespace feature

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.47%. Comparing base (0c9c0a9) to head (daaed4b).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #844   +/-   ##
=======================================
  Coverage   73.47%   73.47%
=======================================
  Files          31       31
  Lines        3114     3114
=======================================
  Hits         2288     2288
  Misses        692      692
  Partials      134      134

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

@jannfis @chengfang We would need the PR https://github.com/argoproj-labs/argocd-image-updater/pull/854 to be merged in. I tested the e2e test with the code changes in the PR and it works successfully.
I will check with the contributor to fix the unit tests in the PR.


terminal not running my c++ code

Type: Bug
I have done every single process to install g++ in my laptop but it keep showing same issue
VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
AMD Ryzen 3 5300U with Radeon Graphics          (8 x 2595)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.33GB (3.16GB free)

Process Argv
--crash-reporter-id 9b264b81-76bf-433c-a4b2-98b52ffd0c50

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

cpptools
ms-
1.13.9

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
vsccc:30610678
pyindex848:30577860
nodejswelcome1cf:30587006
3biah626:30602489
f6dab269:30613381
pythonsymbol12cf:30657549
a9j8j154:30646983

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Update document for Bots V2

This includes the allow/deny changes and noting that requests without user agents are errors that should be handled.

Closed by https://github.com/arcjet/arcjet-docs/pull/49


fix line-length failed

`def format_record(record: dict) -> str:
"""
Custom format for loguru loggers.
Uses pformat for log any data like request/response body during debug.
Works with logging if loguru handler it.
Example:
>>> payload = [{"users":[{"name": "Nick", "age": 87, "is_active": True}, {"name": "Alex", "age": 27, "is_active": True}], "count": 2}]
>>> logger.bind(payload=).debug("users payload")
>>> [   {   'count': 2,
>>>         'users': [   {'age': 87, 'is_active': True, 'name': 'Nick'},
>>>                      {'age': 27, 'is_active': True, 'name': 'Alex'}]}]
"""`

demo/_logger.py:44:1: E501 Line too long (138 > 120 characters)

Unfortunately E501 isn't a "fixable" issue right now. If you take a look at the index in the README, the rules with the hammer and wrench in the "Fix" column are fixable.


Close comparison between float numbers within objects

Do you want to request a feature or report a bug?
Feature request
What is the current behavior?
.toEqual doesn't provide close comparison between float numbers within objects. I'm aware of toBeCloseTo but it's just too much work for me to unpack all the sub (and sub-sub) properties within the object I want to test.
What is the expected behavior?
Maybe .toEqual could receive another parameter for the precision tolerated by tests, or maybe another explicit matcher like .toEqualWithFloats could be added.
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
That's reproducible with the default jest configuration. I'm using jest v19.0.2.
Test case
test('compare floats within objects', () => {
  expect({ a: 0.30000000000000004 }).toEqualWithFloats({ a: 0.3 });
});

Jest already can do that :). See http://facebook.github.io/jest/docs/en/expect.html#tobeclosetonumber-numdigits

@thymikee it's slightly different from my proposal. The link you provided only does that for a single value, I have an object with lots of properties that contain floats. unpacking each value and testing it would be really laborious. do you think it makes sense?

I don't see the reason we should add this kind of matcher, as this use case is very specific.
And Jest also has expect.extend just for such occasions — to create your own matcher, useful inside your tests.

Fair enough. I just have tons of tables with floats because I'm building a UI for charts!
I was trying to reuse the jest tooling to create my own matcher via extend but I'd have to copy and paste code or write my own traversal function. I ended up just hardcoding some piece of data in my test so it always generates the same values and then toEqual becomes useful again.
Thanks for the response! <o/


Can't resolve components in my project

Hi there,
First of all, thanks a lot for this setup. It's great!
I am running into a problem where my components can't be resolved when I try to import them in my ionic project.
Some more info:

I am building my Stencil project with yarn build:stencil
I ran yarn link in my Stencil project
I ran yarn link <project-name> in my ionic vue project
I am importing the web component with

import { MyComponent } from "<project-name>"

When  I load the page, I see the following error in the console: Failed to resolve component: MyComponent
Am I missing something here

can you share your index file of your build?
cheers


is this an issue with inverse kinematics?

Describe the bug
the whole model comes out rather scrambled when used in Aframe
To Reproduce
Steps to reproduce the behavior:

export to gltf 2

import as asset
<a-asset-item id="robot_arm1_model" src="assets/robot_arm/robot_arm1.gltf"></a-asset-item>

use asset:
<a-entity id="arm_model" gltf-model="#robot_arm1_model" scale="0.01 0.01 0.01" animation-mixer></a-entity>

See error
something very hideous :(

Expected behavior
a proper jointed moving robot_arm extracting an item from a box
Screenshots

.blend file/ .gltf
robot_arm1.zip
Version

OS: [Win10 / Apache 2]
Blender Version [2.79]

Additional context
the robot arm is animated using inverse kinematics

You have to use "force sample animations" if you want bake bone constraints on your animations.

What is the method in the current Blender 2.83 where "force sample animations" is not an export setting for the built-in gltf/.glb exporter?

The new name is "Always sample animation", and is ON by default

Thank you @julienduroure .


Can I use Infinite Scroll Pagination for DataTable?

Hi.
My app is build with DataTable, is it possible to use Infinite Scroll Pagination ?
I don't see any possible way to do this, but maybe someone here will know.
Thanks.

yes, you can use a PagedSliverBuilder to make your own implementation.

yes, you can use a PagedSliverBuilder to make your own implementation.

Do you have an example of how to do this?


Offline Training

Thank you for the repo. Can the algorithms be trained offline?

Hi,
All algorithms are currently compatible with the MO-Gymnasium API. So although multi policy algorithms are made to be trained offline for various trade-offs, we do not support training on a fixed dataset.


ci: build RPC image automagically

This will build a warnet-rpc image when merging into main branch

I hate workflows

Ok this does work, but currently it will just push a new "latest" tag when we merge things into main branch, see here for example.
I wanted to have each pr get it's own tag, but thats difficult when PRs come from forks (you have to use a GitHub "environment", and I cannot get it to read secrets out of the environment correctly).


fix the bug

std::numeric_limits::quiet_NaN() will return 0 when T is float
when a[i] == 0 && b[i] !=  0, accm += 0, it sould be accm += b[i]

Cannot replicate the issue


[T4A2][W09-B1] Ng Yong Sheng

You missed one part of the exercise. :-p
Some comments added. Please close the PR after reading comments.


Rebuild pricing input component using GOV.UK Design System components

https://trello.com/c/yld9eBvJ/6-3-rebuild-pricing-form-template-using-govuk-design-system-components-in-supplier-frontend
Design
A new pricing input macro dmPricingInput has been created in the supplier frontend. I haven't added it to digitalmarketplace-govuk-frontend because it is only used in the supplier frontend. The template is based on the date input component in govuk-frontend.
Another separate macro has also been created, clPricingInput (suggestions on a better name welcome), to take input from the content loader and create a form. It was split into two macros in this way partly to keep the implementation of dmPricingInput clean, and also as an experiment into how we can create content loader macros for other form components in future.
Things I decided not to implement:

message parameter (only used for declarations, this question is not used for declarations)
question_number parameter: this doesn't appear to be used anywhere in the old template

Screenshots

Before (with bug)
After

Checklist

[x] Write macro for pricing input component
[x] Write macro to take content loader manifest and instantiate pricing input component
[x] Replace call of digitalmarketplace-frontend-toolkit pricing macro in toolkit_forms with new macro
[x] Fix tests

@lfdebrux tests fixed, dropped the commit with the temporary endpoint. Please have a look.

I'd double check the functional tests (specifically the @supplier-framework-application scenario for G12). I had a local failure but it was an unhelpful 'Too many failures (SystemExit)' message.

I'd double check the functional tests (specifically the @supplier-framework-application scenario for G12). I had a local failure but it was an unhelpful 'Too many failures (SystemExit)' message.

I'll take a look, thanks for flagging this up

I think it's this step, that's looking for a class that is no longer used. I'm happy to raise a PR to address it. https://github.com/alphagov/digitalmarketplace-functional-tests/blob/master/features/support/form_helpers.rb#L252

@lfdebrux @phantas Great stuff! This should be ready to go once https://github.com/alphagov/digitalmarketplace-functional-tests/pull/761 is merged 👍


Update chart external-dns to 1.14.4

This PR contains the following updates:

Package
Update
Change

external-dns
patch
1.14.3 -> 1.14.4

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Error on alpine

ps: unrecognized option: p
BusyBox v1.25.1 (2016-10-26 16:15:20 GMT) multi-call binary.

Usage: ps [-o COL1,COL2=HEADER]

Show list of processes

	-o COL1,COL2=HEADER	Select columns for display

I don't have access to alpine to test on, but there error is coming from the call to get memory usage:
https://github.com/delano/redis-dump/blob/master/lib/redis/dump.rb#L31-L33


SR and TTS connection bug

platform android
java sdk versin 1.1.7
during 8 hours of none stop SR and TTS test
I found there is a high chance of getting connection error, like can't connect to server.
for TTS the error msg is like this:
Error:Connection failed (no connection to the remote host). Internal error: 1. Error details: Code: 7. USP state: 2. Received audio size: 0 bytes. ErrorCode:ConnectionFailure
for SR the error msg is like this:
Connection failed (no connection to the remote host). Internal error: 1. Error details: Code: 7. SessionId: 4f4c2054a40f4f1b849e95790361ad85

Thanks for reaching out. I know the service team is constantly working on making sure their "up" time is as high as possible. Do you have any other information on this? Does it happen every time or just some times? Does it always occur at the same period of time?
Without more concrete information this doesn't really sound like something that the SDK team can help with. It may be the service or related to the specific environment. If we don't have enough to go on or find out there isn't anything we can do, we will close this issue.

Hi, this problem occurs randomly.
Is there a limit on how many socket connections we can have with one
subscription key?
If there is a limit , what should I do to make every user can connect to
server when needed.
On Wed, Jul 14, 2021 at 6:25 AM Christopher Schraer <
@.***> wrote:

Thanks for reaching out. I know the service team is constantly working on
making sure their "up" time is as high as possible. Do you have any other
information on this? Does it happen every time or just some times? Does it
always occur at the same period of time?
Without more concrete information this doesn't really sound like something
that the SDK team can help with. It may be the service or related to the
specific environment. If we don't have enough to go on or find out there
isn't anything we can do, we will close this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1134#issuecomment-879446425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AELYV6TGAZIZPOPTUVAQBKDTXS4NRANCNFSM46XCFPNA
.

Hi @incomingflyingbrick there is no limit.
One thing might be to check that all the recognizer objects are closed and properly cleaned up.
Otherwise without more information it would be hard to look into.
You could set the log file property and generate some logs, though with your large testing it may get too large.
speechConfig.setProperty(PropertyId.Speech_LogFilename, "./sdk.log");
Are you able to show your code?

@incomingflyingbrick Can you please provide more information as requested? Also be sure to update your SDK to the latest 1.18.0 release, just in case.

Closing the item since no further updates received, please open a new issue if more support is needed.


Usage with jest

I'm trying to figure out how to use this module with jest
I'm mocking out my lazy prisma client with jest:
const prismaClient = getPrismaClient()
await truncateAllTables(prismaClient)

const prismaTestingHelper = new PrismaTestingHelper(prismaClient)
let prismaClientProxy = prismaTestingHelper.getProxyClient()

class PrismaClientProxy {
  get() {
    return prismaClientProxy
  }
  get c() {
    return prismaClientProxy
  }
}
jest.unstable_mockModule("@service/repo/client", () => {
  return {
    PrismaClient,
    Prisma,
    prisma: new PrismaClientProxy(),
  }
})

But when my test tries to use the proxyClient, it has all of the models inside of the proxy object but they're all undefined. Any idea why that might be?

How does your original @service/repo/client look like, and how are you accessing it? Does it work if you mock it with
jest.unstable_mockModule("@service/repo/client", () => {
  return {
    PrismaClient,
    Prisma,
    prisma: prismaClientProxy,
  }
})

(without the separate PrismaClientProxy object)?

I have a lazy wrapper around my prisma client
import type { PrismaClient as PrismaClientType } from "@prisma/client"
import { LazyInstance } from "@selfage/lazy_instance"
import { getPrismaClient } from "./util/getPrismaClient"
import PrismaClientPkg from "@prisma/client"

const { PrismaClient, Prisma } = PrismaClientPkg
export { PrismaClient, Prisma }

// doubly-lazy prisma client :)
class LazyPrismaInstance extends LazyInstance<PrismaClientType> {
  // lazy way to get the lazy instance
  get c() {
    return this.get()
  }
}

export const prisma = new LazyPrismaInstance(getPrismaClient)
```

But I fail to see why it would affect me getting `undefined` for all of the models

The error I get in my test is:
    TypeError: Cannot read properties of undefined (reading 'findUnique')

Which is like prisma.user.findUnique so it's not just undefined when printing
I'll take a look at the example repo
unstable_mockModule is for ESM, and you need to do it before async importing (const { foo } = await import('./myModule')) the module you're testing for the mocks to work

I just updated my example repo to use unstable_mockModule and it works - so I guess the issue is somehow related to your specific use-case. You could try to call something like
const prismaClient = getPrismaClient()
await truncateAllTables(prismaClient)

const prismaTestingHelper = new PrismaTestingHelper(prismaClient)
let prismaClientProxy = prismaTestingHelper.getProxyClient()

const user = await prismaClientProxy.user.findUnique(/* ... */);

and if this works, then the problem has something to do with the mock.
If you have a reproduction repo, then I can take a look. But otherwise, I currently don't have an ideas why it wouldn't work in your case.

I'm making the switch to vitest because it has better mocking support for ES modules. Thanks for your help


#296 Load JRuby properties from ./.jrubyrc and ~/.jrubyrc

One step further:
This PR parses the files ./.jrubyrc and ~/.jrubyrc when asciidoctorj is started via the CLI.
(IMO integrators should set these properties themselves, either use the same method or define own properties.)
Fortunately JRuby already provides a public static method to do this in org.jruby.Main that is simply used to accomplish this.
Additionally extended the version output (-V) to show the JRuby version and RUBY_VERSION.

Very nice! This looks like it might be the most convenient way for users to set these values.
I would prefer if the output of -V fit more with the new output being used in Asciidoctor 1.5.3. Here's how it now looks:
Asciidoctor 1.5.3.dev [http://asciidoctor.org]
Runtime Environment (ruby 2.2.1p85 (2015-02-26 revision 49769) [x86_64-linux]) (lc:UTF-8 fs:UTF-8 in:- ex:UTF-8)

I'm thinking perhaps it could look like this:
AsciidoctorJ 1.5.3 [http://asciidoctor.org]
Runtime Environment: jruby 1.7.19 (1.9.3p551)

Notice I changed "Asciidoctor" to "AsciidoctorJ".
The template for the Runtime Environment value is:
jruby JRUBY_VERSION (RUBY_VERSION)

Sound good?

I propose this template because it is consistent with the output of jruby -v (without all the trailing superfluous info).

Sounds good!
Changed the output to look like this now:
Asciidoctor 1.5.2 [http://asciidoctor.org]
Runtime Environment: jruby 1.7.20 (1.9.3)

That looks wonderful.

:beers:

Oh, forgot the J in AsciidoctorJ.
Pushed c3b77ebded735123cdcc3cebcec094b3c919cb41


Highlights not appearing after opening notebook search result

Have a notebook with rendered markdown or outputs. Close it.
Search for something in it using the search widget.
Open the result. Sometimes, the decoration won't be there. I've also seen instances where it's there and then it disappears suddently.

Strange, I now see the decorations there, but the results get doubled. Will try to fix this in this issue.

seeing search flicker when I click on a notebook output search result. Also are the search options ever supposed to render two lines- like in this case I searched for something with two lines but there was a single line for the search result.
https://github.com/microsoft/vscode/assets/26030610/85d648da-65a3-473a-b2af-1e3ebbcb2a91

Hmm, that's a weird flicker, I think this is part of https://github.com/microsoft/vscode/issues/191568#issuecomment-1698227903, so I'll track the flicker there.
Seems that the decoration persists after the flicker, which should be good enough for this issue.


Consider a more efficient string manipulation approach

Moving big strings around can be bad.
It is probably better to persist as soon as possible and reload only when necessary.
The ideal situation is to have short lived objects holding web data.

Following advice 7.4.4 from Stroustrup's Tour of C++, it's better to keep returning strings by value and trust on the move semantics.
Also the big strings are relatively short-lived in the program.
C++11 rocks.


trackAppOpenedWithRemoteNotificationPayload not recording app push open for iOS7

Hello,
I have an issue where trackAppOpenedWithRemoteNotificationPayload is being called in iOS7 but no push open event is shown in Parse.com.
The issue is detailed here on SO: http://stackoverflow.com/questions/32103788/parse-not-sending-push-open-notifications-for-ios7
I have followed the guide for the relevant Push setup with Parse.
When testing the app on an iOS7 device and iOS8 device the same method is called when opening from a push:
    - (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler {
        DebugLog(@"%s",__PRETTY_FUNCTION__);
        if (application.applicationState == UIApplicationStateInactive) {
            [PFAnalytics trackAppOpenedWithRemoteNotificationPayload:userInfo];
        }
        if (completionHandler) {
            completionHandler(UIBackgroundFetchResultNoData);
        }
        [PFPush handlePush:userInfo];
    }

It appears that although the [PFAnalytics trackAppOpenedWithRemoteNotificationPayload:userInfo]; method is called on iOS7 the data is not shown in Parse.
For example, checking the Push that has been sent shows the amount of Opens. If I send a push to an iOS7 device and iOS8, the same part of code is being called. However, it is only recorded for the iOS8 version. Is there anything else I should be doing?
We have followed the relevant guide: https://parse.com/docs/ios/guide#push-notifications
This is also implemented:
    - (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo {
        DebugLog(@"%s",__PRETTY_FUNCTION__);
        if (application.applicationState == UIApplicationStateInactive) {
            // The application was just brought from the background to the foreground,
            // so we consider the app as having been "opened by a push notification."
            [PFAnalytics trackAppOpenedWithRemoteNotificationPayload:userInfo];
        }
        [PFPush handlePush:userInfo];
    }

Thanks

Hey @StuartMorris0, can you please verify that the check for application.applicationState == UIApplicationStateInactive passes on iOS 7 and 8?
The code behind this (inside PFAnalytics) as you might have seen doesn't have specific logic handling between iOS 7 and 8, so what I think happens is that on iOS 7 - the app is backgrounded and reports UIApplicationStateBackground there, instead of UIApplicationStateInactive.
Try modifying the code snippet to use something like this and let me know if it works.
    - (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo {
        DebugLog(@"%s",__PRETTY_FUNCTION__);
        if (application.applicationState != UIApplicationStateActive) {
            // The application was just brought from the background to the foreground,
            // so we consider the app as having been "opened by a push notification."
            [PFAnalytics trackAppOpenedWithRemoteNotificationPayload:userInfo];
        }
        [PFPush handlePush:userInfo];
    }

Hey @nlutsenko. I can confirm that when I run the application connected to XCode, close the application (send to background), then send a Parse Push.
The push is received and used to open the app on both iOS7 and iOS8 devices.
didReceiveRemoteNotification is then called for both, when stepping through the code on each device I can see that the method is being called. However for iOS7 it is not recorded in Parse, for iOS8 it is.
However, good news... I discovered the issue.
So... to test notification permissions it can be a pain on a real device. The steps to request the permission again once already done for an app is to Reset the device, Change the date to a date in the future, reset again, then reinstall the app and the permission will be requested again.
This was complete on both the devices iOS7/iOS8. It appears that the iOS7 device with a future date was not sending those Push opens. But the iOS8 one was. Maybe the iOS7 one was slightly ahead in terms of a date in the future (maybe too far).
However, setting the iOS7 date back to the correct date the push open is recorded as expected. Strange, but worth noting here. Thanks


integrate ads into Redot engine for funding

Tested versions
N/A
System information
N/A
Issue description
as a fork of the Godot engine, Redot has immense potential to grow and serve the developer community.  however, sustaining development and ensuring long-term viability necessitates a reliable funding model. i propose the integration of advertising support into the engine to create a potential revenue stream.
benefits:

funding for development: ads can provide a continuous source of income, helping to support ongoing development and maintenance.
community growth: with funding, the engine can attract more contributors, leading to faster development and improved features.
optional monetization for developers: by integrating ad services, developers can choose to monetize their games more easily, creating an ecosystem that benefits both the engine and the developers.

proposed implementation:

select ad networks: research and integrate popular ad networks (e.g., AdMob, ExoClick, Unity Ads) that offer easy-to-use sdks for game developers.
documentation:  develop comprehensive documentation detailing the process of implementing and customizing ads within games built on the Redot engine. this documentation should include step-by-step guides, code examples, and best practices to ensure developers can easily integrate ads without extensive technical knowledge. i suggest something like this:

extends Node

var aD_loaded = false
var ad_network = null # where we keep ad stuff or whatever
var random_number = 0 # just a random number

func _ready():
    load_ad_stuff()
    if aD_loaded == true:
        ad_network.show()
    else:
        print("oops, no ad loaded :(")

func load_ad_stuff():
    ad_network = preload("res://path/to/ads.gd").new() # might fail
    aD_loaded = true # let's just assume it worked tbh

func displayAdNow():
    if aD_loaded:
        if random_number < 0:
            ad_network.show_ad() # how does this work again?
        else:
            print("ads are not ready or something i dont know how this works honestly") # no clue

func _on_button_pressed(): # should be connected to a signal
    random_number = randi() % 10 # generate a random number
    displayAdNow() # show ad, i think

func random_event():
    if randi() % 5 == 0:
        print("random event happened")

opt-out system: ensure that the ad feature is optional for developers.

conclusion:
integrating advertisements into the redot engine could provide a crucial funding source while simultaneously fostering a thriving developer community. by creating an ecosystem that benefits both the engine and its users, redot can enhance its appeal to new developers and encourage innovation within the platform. i believe this proposal is a vital step toward ensuring the long-term success of redot.
thank you for considering this suggestion! i look forward to your thoughts and feedback on this proposal.
Steps to reproduce
N/A
Minimal reproduction project (MRP)
N/A

Are we talking about something like unity but in another wrapper? I personally originally chose godot because it doesn't have any ads or licence fees, have it's code published, can be modified and I don't want to write anything from scratch Suggestions like that while make sense from individual contributors perspective contradicts the purpose. My position is that people who use project by themselves for commercial use invest in its development. It's same reason they become stale, but do we need new no ads fork?

the person that opened this issue is a known bad actor, don't pay too much attention to it

wtf... why are we talken about this? No, No, No, No, No, No, No, No, No , NO

opt-out system: ensure that the ad feature is optional for developers.
tbh opt in would be okay for those who want to support but the default shouldnt be opt out ads because that isnt exactly in the spirit of collaboration and open source

ads arent the way to do it i mean how many people you know run ad blockers. if you wanna place ads in a game ok its up to you but in editor ads are a no go. so many people wouldn't even give the engine a chance if they have to see ads distracting from development.
As an open source community and a new fork at that development and support come from dedicated volunteers working hard to support a project they believe in.  Maybe in the future they could take donations to help with costs, but at the moment we'll put our trust in those giving up their time to make this project the best that it can be.

Are we talking about something like unity but in another wrapper? I personally originally chose godot because it doesn't have any ads or licence fees, have it's code published, can be modified and I don't want to write anything from scratch Suggestions like that while make sense from individual contributors perspective contradicts the purpose. My position is that people who use project by themselves for commercial use invest in its development. It's same reason they become stale, but do we need new no ads fork?

the person that opened this issue is a known bad actor, don't pay too much attention to it, it's clearly a troll issue

for instance modrinth hs ads on its platform redot shoud too idk ur just arguing in bad faith


Angular 17 Support

Can this plugin be upgraded to support the new control flow syntax?
Or  is this not maintained anymore?

FYI: I'm working on the new control flow syntax

parser : https://github.com/dlvandenberg/tree-sitter-angular/tree/feat-control-flow
fork : https://github.com/dlvandenberg/nvim-treesitter-angular/tree/feat-control-flow


Propagate M_UNKNOWN_TOKEN errors

When the proxied server returns a 401 error with M_UNKNOWN_TOKEN, it would be nice if the proxy could forward it to its clients, along with the soft_logout parameter if present, so that clients can detect it and properly react to it.

We do now send M_UNKNOWN_TOKEN when /whoami returns 401, but this doesn't propagate soft logout flags.


Heltec ESP32 sends to TTN but does not get data

The connection information even arrives at the TTN, but does not show the data. I am using the same code obtained here. What may be the problem?

I don't know if this part send the data, I tried with values fixed too.
`
void do_send(osjob_t* j)
{
// Check if there is not a current TX/RX job running
if (LMIC.opmode & OP_TXRXPEND)
{
Serial.printf("OP_TXRXPEND, not sending\r\n");
}
else
if (!(LMIC.opmode & OP_TXRXPEND))
{
TempAndHumidity newValues = dht.getTempAndHumidity();
lpp.reset();
newValues.temperature = 11;
newValues.humidity = 40;
lpp.addTemperature(1, newValues.temperature);
lpp.addRelativeHumidity(2, newValues.humidity);
Serial.printf("Temperature : %.2f, Humidity : %.2f\r\n", newValues.temperature, newValues.humidity);
display.clear();
display.drawString(0, 0, "Temperature: ");
display.drawString(90, 0, String(newValues.temperature));
display.drawString(0, 20, "Humidity  : ");
display.drawString(90,20, String(newValues.humidity));
display.display();
#if 0
if (GPS.fix)
{
float lat = GPS.latitudeDegrees;
float lon = GPS.longitudeDegrees;
float alt = GPS.geoidheight;
Serial.printf("GPS lat = %.2f, lon = %.2f, alt = %.2f\n", lat, lon, alt);
lpp.addGPS(3, lat, lon, alt);
}
#endif
LMIC_setTxData2(1, lpp.getBuffer(), lpp.getSize(), 0);
Serial.printf("Packet queued\r\n");
}
}
`

Hi,
I do not quite know how we can help you. This page is for supporting the single channel gateway but you have a question regarding an ESP32 sensor node. Do you use the single channel gateway or another gateway?
If the single channel gateway is used, it is possible to see the messages received and even decode them (#define _LOCALSERVER 1).
But if you are not using a single channel gateway you better ask on the TTN forum for ESP32 nodes.

Yes, I using the single channel getaway in my WiFi LoRa 32, I will try on this part #LOCALSERVER1
if here is not this local appropriate I apologize

#define _LOCALSERVER 1
see ESP-sc-gway.h and fill in your sensor in sensor.h

I solved the problem, it was not a code, but a TTN configuration. Thanks =)


CA combinations wrong

The CA combinations doesn't show the right number. For example I have a tower that projects 900+1800+2100+2600mhz but since the new update It only shows 1800mhz all the time. Or showing only 3CA when Im connected to 4CA.
So the projecting of the LTE frequency in the app are not right.

NetMonster Core fully depends on Android SDK and does directly interact with RIL. As you mention device software updates might break some functionalities.
If you think that issue is on library's side please send bug report from NetMonster application. In order to do that, go to app's settings, tap 5 times on 'Version' and then hit 'SEND' button on top-right corner. Make sure that your device is connected to at least two carriers when generating the report.

Ok I'm Noob at this
😁😁😁

Sorry Just kidding I'm too wasted to Concentrate... Or not wasted enough. Euh never mind I'm GOING TO finish My Dilirium bear


ManualValidation task doesn't send email notifications

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: ManualValidation
Environment

Server: Azure Pipelines
Agent: hosted

Issue Description
I have this job:
  - job:
    displayName: Manual validation
    pool: server
    timeoutInMinutes: 4320 # 3 days
    steps:
    - task: ManualValidation@0
      timeoutInMinutes: 1440 # 1 day
      inputs:
        notifyUsers: |
<EMAIL_ADDRESS>        instructions: 'Please validate the build configuration to resume.'
        onTimeout: 'resume'

It works insofar as I can choose to reject or resume in the web GUI – but I get no email notification. I've tried with my personal work email as well as a generated email for one of our Teams channels (which I have confirmed is working; shown in the above example). I have waited for several hours and also checked my Junk Email folder in OWA.
Task logs
The ManualValidation task doesn't seem to log anything.
Error logs
No errors logged.

It worked with a colleague's email, so we think the email address must be registered in DevOps in some way, although we're not quite sure how.

I confirm, that external (in sense of Azure Devops) emails aren't processed by this task, no information is sent. If email which is registered inside the Azure Devops itself is used, notification is sent.
Some notices about emails for notifyUsers settings will be enough for me.

@ds-ms can you check this and assign it appropriately

@SimonAlling  Is this in an On-prem version? I cannot get the gui to pause let alone send email, it is as if the task is not read or processed. Azure On-Prem 2020.0.1

@SimonAlling I've gotten this to work by first inviting the email address of the teams channel to the Azure DevOps project.


Build against a newer gdal

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

This is related to https://github.com/bioconda/bioconda-recipes/pull/15134 wherein r-rgdal was being built against a libgdal version that was pulling in an incompatible version of proj4.

Done in #21


Data Grid - Column Header - Add tooltip on hover

Great call outs @jgough & @joshuarrrr  - I think I would prefer, for the short term, to provide the tooltip on mouseover vs wrapping. Since there are variations of the look & feel for Data Grid - and - builders can set some rows to not be resizeable, we would need to engage in some more exploration for table header wrapping - especially now that it used to power visualizations that, on a dashboard, could end up somewhat small depending on how the user lays out the visualization on their dashboard, and I think we can make more meaningful progress using the tooltip. The tooltip also feels like the right thing to do for accessibility?
Lets do some exploration @joshuarrrr to see if there is any interference with the click action (right now I see that the entire header is clickable to get the pop-over to show for the sort actions).

Chose wrong comment to base issue off of, therefore closing as not planned to start a fresh version. Whoopsie!


fix(dashboard): Fix cluster name for application dashboard

Goal
Use cluster name on dashboard

:tada: This PR is included in version 1.6.12 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(TDI-39227): avoid cast exception

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-39227
What is the new behavior?
Avoid cast exception in the studio.
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:


Styling

When picking up item, time counter should blink in green for a few seconds

Possible solution: Add property to Game called style. On picked up item create setInterval every 120ms to swap styler between normal and green. When drawCounter is called on the next animation frame prints on normal or green depending on style value.

Valid solution. Fixed


Template modifications are not injected on reload

[x] I've read the guidelines for Contributing to Roots Projects
[x] This request isn't a duplicate of an existing issue
[x] I've read the docs and followed them (if applicable)
[x] This is not a personal support request that should be posted on the Roots Discourse community

Description
Template modifications are not taken into account with laravel mix in watch mode.
Steps to reproduce

I installed Sage 10.
I launch development mode with yarn start.
I modify one of the files in resources /views/.
The page is reloaded but the modifications made in the html code do not go back

Expected behavior: Template modification should be injected at page reload
Actual behavior: No template modification added.
Reproduces how often: 100%
Versions
Latest sage 10-dev-master
Additional information
Environment: lando (docker) with wordpress recipe (php 7.3, nginx, mariadb). Bedrock install.
The page reload is done if I add a get parameter in the url (eg localhost:8080?v=2)
I tried various configuration (watch: true on browserSync)

@AmphiBee have you found any solution to this error?

This should be fixed on the latest version.

I was having the same scenario. I've mange to fix it by realizing that opcache was enabled in my localhost.
So, if you are on a development environment make sure to disable opcache or downsize the time revalidate in revalidate_freq to 0.
Ways to fix it:

On .htaccess include this if Apache is configured to to manage php_flag inside .htaccess: php_flag opcache.enable Off
You can also disable in php.ini: opcache.enable=0 or opcache.revalidate_freq=0

Hope it helps.


Static binary release

I noticed you have rpm and deb packages available on releases page. Would it be possible to also provide statically built linux amd64 binary? Useful for easy inclusion into docker images.

Yes, it is possible. May be in 0.10 release.

Static binary has provided in 0.10.0 release


Adding probes to the cert-manager pods

Is your feature request related to a problem? Please describe.
As part of Kubernetes' best practices, I'd like to set Readiness and Liveness probes on all the containers deployed on my infrastructure. As of now, the charts in cert-manager lack this capability.
Describe the solution you'd like
An easy solution would be hard coding the probes to the cert-manager templates (cert-manager and cert-manager-cainjector).
A more complete solution would be making those probes customizable via values.yaml.
Describe alternatives you've considered
At the moment the only alternative to set up probes would be manually patching the deployments after helm install, which isn't the best practice devops-wise, and unfeasible for automated solutions on a wide array of clusters.
/kind feature

This is an interesting thing to think about.
How would we define the readiness and lifeness of a Kubernetes operator?
Since the controller and cainjector are not serving any traffic we can not test a port to be ready. Also how would we handle the state if the controller is up but is not the leader.
How would we actually verify that the controller is up and working? We don't want to add probes that give a false sense of security.
We have these on the webhook where it makes sense as it is an actual HTTPS endpoint serving traffic, which has services relying on it reporting it's state so endpoints can be controlled.
I took a look at how other Kubernetes operators do this and all ones I use also don't have these probes.
Feedback welcome!

We actually ran into this too just now and were surprised that there is no health check.
There is a metrics endpoint available (https://github.com/jetstack/cert-manager/blob/master/deploy/charts/cert-manager/templates/deployment.yaml#L106), so a possibility would be to scrape that endpoint to check if the container is up & running. There's quite a few tools that do it like that afaik.
In the case we just had, the metrics-endpoint stopped serving traffic when the controller went into some kind of busy-loop. We noticed sometime later and killed / restarted the pod, which resolved the issue. It would be nice if this was handled by the probes instead.
Maybe it's not possible to distinct between ready & alive, but maybe that's also not needed.

interesting idea, i never had CM lock up before so couldn't probe it. I do wonder with the recent refactor of our metrics if it would still be the same. Will try to look into it
/area deploy

/priority important-longterm

This issue is also valid if you want to use Cert manager in clusters that should be SOC 2 compliant because they're checked according to OWASP and missing liveness and readiness probes qualifies as Security misconfiguration. So it would be really nice if Cert manager just had some kind /status endpoint that could be used for those probes.

xref https://github.com/jetstack/cert-manager/pull/4133

/remove-lifecycle stale

I just had the second cert-manager not reconciling for some reason any more. I had to delete and restart the pod. A liveness probe would have catched this incident.
With kube-builder, we're using a function that validates the cache is in sync and register this as liveness probe in our daemons when starting the managers.
/remove-lifecycle stale

/remove-lifecycle stale

Note that probes would likely not be a replacement for running something like cmctl check api in CI before applying resources as you'd still need to verify that the whole system (controller, webhook and cainjector) functions before resources can be applied.
I can see that probes could be useful if there sometimes is a need to restart the pod- would be good to understand more in what cases this happens and perhaps look at how other projects do it if there are open source examples.

/remove-lifecycle stale

So this also caught my attention since this is the last Helm Chart in my cluster NOT having probes. Is there anything the community can help with? A simple first example that is coded into the HelmChart is very easy to contribute. Replacing the cmctl check api functionality with this is a whole different size of a contribution though. Any advice from maintainer side to this topic?

Bump.
We are also encountering this. The cert manager itself has the metrics port that can be checked via tcp probes.
The cainjector however does not expose any ports and does not have usable tools installed to perform checks via an exec probe.
The best approach in my opinion would be to just add health check endpoints to all services and probe them.

/remove-lifecycle stale

any news on this request? Liveness and readiness probes are very important for reliability ...

There's been further discussion in Slack:

https://kubernetes.slack.com/archives/CDEQJ0Q8M/p1665603954227279

And in:

https://github.com/cert-manager/cert-manager/pull/5670#pullrequestreview-1235725002

We are reaching the conclusion that there may be a useful liveness probe that we can add, based on the leader election library:
https://github.com/kubernetes/client-go/blob/v0.26.3/tools/leaderelection/healthzadaptor.go#L25-L36

Nope. This one stays open. :)

/remove-lifecycle stale

/remove-lifecycle rotten

Not having the /healthz endpoint tied to the leader election means that in case of a cert-manager upgrade, the Prometheus scraper may hit the wrong /metrics endpoint until the upgrade has finished. I don't think the /metrics endpoint should work when the process hasn't been elected.
Would anyone want to work on making the /healthz endpoint fail when the leader election fails?

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale


Fix the order of environment variables for injection

Since the traversal order of the map is unstable, the order of env after injection is unstable, resulting in different results of injecting the same Pod.
[x] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

Since the traversal order of the mapping is unstable, the order of env after injection is unstable, resulting in different results of injecting the same Pod.

@wzshiming: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
92094559979569d2f5721b91602adac7e196daf5
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


do not review do not merge

Follow-up on https://github.com/redhat-appstudio/e2e-tests/pull/126/files

/retest

/retest

/lgtm


Define capabilities and API for sl-pagination

This issue is where discussion about all of the things we want to change about the sl-pagination component should occur and the final definition of its capabilities and API are defined.  As these items are discussed and vetted, issues will be created to work any individual tasks, with this issue serving as the overall source of progress.
To indicate that a capability of the component needs to be verified, add a comment with **VERIFY** as the first line.
To propose a change to the component, add a comment with **PROPOSAL** as the first line.
To indicate that something needs to be researched, add a comment with **RESEARCH** as the first line.
As you are leaving comments regarding any of these VERIFY, PROPOSAL, or RESEARCH comments, link back to the comments link in your own so that the conversation can be more easily followed (in absence of threaded conversation support in Github).  This should be done on the first line of the comment via **RE: <url>**
Capabilities and API

[ ] ARIA support
[ ] ...

Tasks
Adding them here until issues are created

Update documentation for each change
Complete review of documentation against capabilities

Issues tracking these efforts
https://github.com/softlayer/sl-ember-components/issues?q=is%3Aopen+is%3Aissue+milestone%3A"v0.13.0+(Final+push+for+a+1.0.0+release)"+label%3Asl-pagination

PROPOSAL
support "pagination-lg" and "pagination-sm" classes for sizing

PROPOSAL
show pages like:
http://getbootstrap.com/components/#pagination
instead of one unclickable area that lists current page of total

PROPOSAL
add a property for maximum pages to show and show floor(maximum/2) before current and floor(maximum/2) after current

PROPOSAL
add a default declaration for changePage action

PROPOSAL
require totalPages in init

@SpikedKira Is https://github.com/softlayer/sl-ember-components/issues/1266#issuecomment-165810270 addressed by https://github.com/softlayer/sl-ember-components/issues/564 ?


Translation fixes

Fixes:

don't apply translate filter on a dynamic expression
don't apply __() on a dynamic expression

In neither of those cases above would the string be collected by gulp gettext-extract.

Euwe Backport conflict:
$ git cherry-pick -e -x -m 1  9912ba3
error: could not apply 9912ba3... Merge pull request #227 from mzazrivec/translation_fixes
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

$ git status
On branch euwe
Your branch is up-to-date with 'upstream/euwe'.
 as appropriate to mark resolution)

	deleted by us:   client/app/components/blueprints/blueprints-list.html
	deleted by us:   tests/blueprints/blueprints-list.directive.spec.js

@chessbyte Things seem to be different in euwe branch, I'll create a separate PR just for the euwe branch.

Backported to Euwe via https://github.com/ManageIQ/manageiq-ui-self_service/pull/231


Activity is null

General Troubleshooting

[X] I have checked for similar issues.
[X] I have updated to the latest JDA version.
[X] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
Sometimes the current activity of a member is detected, sometimes don't.

Expected Behavior
The activity should be detected.

Code Example or Reproduction Steps
    if (member.getActivities().size() >= 1) {
        return member.getActivities().get(0).getName();

Code for JDABuilder or DefaultShardManagerBuilder Used
        this.jda = JDABuilder.createDefault("Token")
                .setAutoReconnect(true)
                .setStatus(OnlineStatus.ONLINE)
                .setActivity(Activity.watching("activity"))
                .enableIntents(GatewayIntent.GUILD_PRESENCES)
                .enableCache(EnumSet.of(CacheFlag.ACTIVITY))
                .build();

Exception or Error

N/A

You are using the default member cache policy MemberCachePolicy.VOICE so the activities cannot be tracked. In order to track activities for members, you have to cache the members.
You can use setMemberCachePolicy(MemberCachePolicy.ONLINE) to only cache members who are currently online.

Worked, thanks for the quick help!


Obsolete single step process for ADP ribosylation

Obsolete:

GO:0018120 peptidyl-arginine ADP-ribosylation
GO:0018122 peptidyl-asparagine ADP-ribosylation
GO:0018123 peptidyl-cysteine ADP-ribosylation
GO:0018424 peptidyl-glutamic acid poly-ADP-ribosylation
GO:0018312 peptidyl-serine ADP-ribosylation

Look into:

GO:0006471 protein ADP-ribosylation
GO:0070213 protein auto-ADP-ribosylation
GO:0140289 protein mono-ADP-ribosylation
GO:0070212 protein poly-ADP-ribosylation

To be completed.

We have 7 EXP annotations in Dicty and I will look into these in the next days. Please don't obsolete yet.

Dear all,
The proposal has been made to obsolete the following single step process for protein ADP ribosylation:

GO:0006471 protein ADP-ribosylation
GO:0018120 peptidyl-arginine ADP-ribosylation
GO:0018122 peptidyl-asparagine ADP-ribosylation
GO:0018123 peptidyl-cysteine ADP-ribosylation
GO:0018424 peptidyl-glutamic acid poly-ADP-ribosylation
GO:0018312 peptidyl-serine ADP-ribosylation
GO:0140289 protein mono-ADP-ribosylation

There are 84 manual annotations to these terms that still need review, see https://github.com/geneontology/go-annotation/issues/4225

Contributor
(Assigned by)

BHF-UCL
11

CACAO
4

dictyBase
1

FlyBase
3

GeneDB
1

GR
1

MGI
9

MTBBASE
1

RGD
4

TAIR
1

UniProt
45

There are 6 InterPro mappings and 1 Unirule, see https://github.com/geneontology/go-annotation/issues/4225
We are opening a comment period for this proposed obsoletion. We’d like to proceed and obsolete this term on July 5th, 2022. Unless objections are received by July 5th, 2022, we will assume that you agree to this change.
Thanks, Pascale

I'm done with Dicty, but not yet reflected in QuickGO. I also disputed an annotation from CACA and there are still IBA

Obsoleted
GO:0006471 protein ADP-ribosylation
GO:0018120 peptidyl-arginine ADP-ribosylation
GO:0018122 peptidyl-asparagine ADP-ribosylation
GO:0018123 peptidyl-cysteine ADP-ribosylation
GO:0018424 peptidyl-glutamic acid poly-ADP-ribosylation
GO:0018312 peptidyl-serine ADP-ribosylation
GO:0140289 protein mono-ADP-ribosylation
Still need to

[ ] look into
GO:0070213 protein auto-ADP-ribosylation
GO:0070212 protein poly-ADP-ribosylation
For now I moved these under 'protein post-translational modification

[ ] Review annotations and obsolete
GO:0010835 regulation of protein ADP-ribosylation
GO:0010836 negative regulation of protein ADP-ribosylation
(now that we dont have the regulates closure in AmiGO anymore I forget to do these)

[ ] Review annotations and obsolete

protein de-ADP-ribosylation


jaeger exporter build is broken

Hello again. thanks for recently fixing up jaeger! I've had issues updating to master and 0.6.0 with the build for the jaeger exporter, it appears some files need to be re-genned and I do not consider myself an expert here, I hope it is easy to fix. repro:
$ cd exporter/jaeger
$ git rev-parse HEAD
12aa45c27dd5d6f81b3d5d193c3e0a427263056d
$ go build
# go.opencensus.io/exporter/jaeger/internal/gen-go/jaeger
internal/gen-go/jaeger/agent.go:79:20: too many arguments in call to oprot.Flush
        have (context.Context)
        want ()
internal/gen-go/jaeger/agent.go:122:13: too many arguments in call to oprot.Flush
        have (context.Context)
        want ()
internal/gen-go/jaeger/jaeger.go:2146:13: too many arguments in call to oprot.Flush
        have (context.Context)
        want ()
internal/gen-go/jaeger/jaeger.go:2163:14: too many arguments in call to oprot.Flush
        have (context.Context)
        want ()
internal/gen-go/jaeger/jaeger.go:2176:14: too many arguments in call to oprot.Flush
        have (context.Context)
        want ()
internal/gen-go/jaeger/jaeger.go:2190:23: too many arguments in call to oprot.Flush
        have (context.Context)
        want ()
$ head internal/gen-go/jaeger/jaeger.go
// Autogenerated by Thrift Compiler (0.11.0)
// DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING

I am not sure that I know what I am doing :)
thank you

Do you use dep? You might want to use dep to fix the versions and play around with different Thrift versions.

Thanks @ramonza - am using dep, I will try that. as a sanity test, I've attempted to build against this repo outside of the larger project and its dependencies to reproduce the above error just against the base repo. fwiw I see the noted files causing errors checked into the repo here, too https://github.com/census-instrumentation/opencensus-go/blob/master/exporter/jaeger/internal/gen-go/jaeger/agent.go#L79 and am not sure if I am missing something. it seems all that is needed to be done is change the parameters passed to oprot.Flush to go away, but the file seems generated and did not want to PR that myself as it was not clear to me how to generate them.

you might want to pin thrift and open census on the latest from the default branches of both projects since that definitely works according to travis CI

yea, have locked where we needed to, we are good, thanks. trying to help upstream now :)
it appears that updating thrift has fixed this. all i needed was a coffee... sigh, mondays. thank you!

thank you for taking the time to work through the issue. I know it's really frustrating, we also experienced a lot of problems with this dependency and we considered vendoring thrift or moving to the HTTP endpoint.

@rdallman Can you share which version of thrift was used? I wasn't able to get it working with 0.10.0, 0.11.0 and master. I used dep aswell.

hey @lukasmalkmus am using 272470790ad6db791bd6f9db399b2cd2d5879f74 for thrift. cheers.

Thanks fore the fast response. Unfortunately I can't get it to work. Maybe it's just late and I need a coffee 😄 Since thrift isn't a direct dependency I used an [[override]] in the Gopkg.toml but the version I get doesn't expose the interface wanted by the generated code in the opencensus package.

rats. when i was battling this I did end up doing a full:

rm -rf $GOPATH/pkg/dep
rm -rf vendor/
rm Gopkg.lock

and the relevant bits of Gopkg.toml are as you describe:
[[constraint]]
  name = "go.opencensus.io"
  version = "0.6.0"

[[override]]
  name = "git.apache.org/thrift.git"
  revision = "272470790ad6db791bd6f9db399b2cd2d5879f74"

sometimes dep can be mean, maybe nuking could help. with glide it was usually the magic touch required sigh

Ah thanks! I used the wrong import path (the thrift mirror on Github) which I did use in another project.  Would have taken me a night and a full can of coffee to get that by myself. Cheers mate 👍


Unable to add columns to Table resource

What happened?
Attempting to define a table with the nodejs SDK as:
new databricks.Table('name', {
  // ...
  columns: [{
    name: 'x',
    position: 0,
    typeName: 'string',
    typeText: 'varchar(64)',
  }],
})

This results in the following error:
cannot update table: UpdateTable Missing required field: UpdateTable Missing required field: column_0.type_name

Expected Behavior
The plugin should correctly map typeName to type_name before sending to the databricks API
Steps to reproduce
Create any databricks.Table resource with the nodejs Pulumi SDK
Output of pulumi about
CLI
Version      3.55.0
Go Version   go1.19.5
Go Compiler  gc
Plugins
NAME    VERSION
nodejs  unknown
Host
OS       darwin
Version  13.2.1
Arch     x86_64
This project is written in nodejs: executable='/usr/local/bin/node' version='v19.7.0'
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hey @mattfysh, thanks for opening an issue! I'm not sure I understand how your comment relates to this issue. Can you clarify what might be impossible?

Closing due to inactivity.


iam.Policy: return empty set on missing key.

Closes #3346.

Do we really need a defaultdict to solve #3346?

Nah if it's OK with @jonparrott it's OK with me


dockerize

BTW we shouldn't merge this into main as-is… We should keep using the tagged TPC-H DB files for consistency, and also re-enable TPC-H ×2 and ×4.

Just deleted the branch, but the content of this PR is memorialized in the pldi23-artifact-compiler tag: https://github.com/kovach/etch/releases/tag/pldi23-artifact-compiler


Press n and enter in numi -> loops

My fault...do not enable integration with alfred. Sorry ;)

The issue should be addressed with the latest Alfred extension fix. Closing for now.


Retry if build failed

I am in a situation where my jenkins builds sometimes randomly fails, so I added  the ability to retry if that happens.
you can select the number of retries from the plugins gui settings now.
I keep a count of the times a job tried to build, the trigger to try a new build, if one needed, is when pulling rss build results.
If you try to build again from the gui(for example if you want  different parameters), it will stop the previous job from retrying.

Merged in #177
Thanks for contribute


Add order descriptions via order item index

Mostly ready for review, just cleaning up comments
Can mix and match o/ and oi/ as long as at least one is present
E.g.

add  p/98765432<EMAIL_ADDRESS>a/311, Clementi Ave 2, #02-25 o/Chocolate Cakes o/2 oi/2 n/Dina t/friends t/owesMoney d/01/01/2022

add  p/98765432<EMAIL_ADDRESS>a/311, Clementi Ave 2, #02-25 oi/2 n/Dyna t/friends t/owesMoney d/01/01/2022

add  p/98765432<EMAIL_ADDRESS>a/311, Clementi Ave 2, #02-25 o/chocolate cake n/Deeena /friends t/friendly d/01/01/2022

Had quite a lot of fake data and temporary stuff before the storage pr was done - it should be removed soon

Codecov Report

Merging #108 (005229e) into master (142c801) will decrease coverage by 0.52%.
The diff coverage is 69.51%.

@@             Coverage Diff              @@
##             master     #108      +/-   ##
============================================
- Coverage     72.65%   72.13%   -0.53%
- Complexity      529      650     +121
============================================
  Files            87       98      +11
  Lines          1686     2128     +442
  Branches        183      254      +71
============================================
+ Hits           1225     1535     +310
- Misses          406      505      +99
- Partials         55       88      +33

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/seedu/cakecollate/MainApp.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

.../java/seedu/cakecollate/commons/core/Messages.java
66.66% <ø> (ø)
1.00 <0.00> (ø)

...late/logic/parser/DeliveryStatusCommandParser.java
80.00% <0.00%> (-20.00%)
3.00 <1.00> (+1.00)
:arrow_down:

src/main/java/seedu/cakecollate/model/Model.java
100.00% <ø> (ø)
1.00 <0.00> (ø)

...in/java/seedu/cakecollate/model/order/Address.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...main/java/seedu/cakecollate/model/order/Email.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

.../main/java/seedu/cakecollate/model/order/Name.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...a/seedu/cakecollate/model/util/SampleDataUtil.java
31.42% <0.00%> (-5.24%)
3.00 <0.00> (ø)

...edu/cakecollate/storage/JsonAdaptedOrderItems.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

src/main/java/seedu/cakecollate/ui/HelpCard.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

... and 44 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 c31c436...005229e. Read the comment docs.


Update the Redmine image in version 3.4.7

Please update the redmine image in version 3.4.7

Pushed https://github.com/sameersbn/docker-redmine/releases/tag/3.4.7
Build currently running https://hub.docker.com/r/sameersbn/redmine/builds


feat(dot): dots have new stroke and fill

:tada: This PR is included in version 1.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Change the layout of the doxygen-generated documentation

Move from index-based layout to treeview-based one.
Automatically take the first sentence of the documentation as the
brief doc.
Always generate both brief and detailed description sections, and
always sort them alpabetically.
Disable the generation of huge and overly complex call and caller
graphs.
Make the project title title-cased.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

I admit that it is a very subjective matter what looks good. So, please take these as my personal preferences. Happy to drop this PR if others don't like this look.

I think the improvement of the content is more important than the improvement of the style. :)

I agree, at least partially. My problem was that I couldn't grasp the content because of the style.
But then, what now with this PR? Update & merge, close/drop, or just let it linger here for a while more?

@akiss77, how can I check the difference between this PR and the current style? How can I generate the doxygen documentation?

Type doxygen in the root directory of the project and check the results indocs/doxygen/html.

I'm Ok with the changes. You can land if you'd like to do. Just a "feature request". Is it possible to align the right parenthesis to the end of the last attribute?
Current style in doxygen
uint64_t __attr_const___ jrt_extract_bit_field	(uint64_t 	container,
                                                 size_t 	lsb,
                                                 size_t 	width)
                                                 )

What I'd like to see:
uint64_t __attr_const___ jrt_extract_bit_field	(uint64_t 	container,
                                                 size_t 	lsb,
                                                 size_t 	width)

@LaszloLango I've digged in the sources of doxygen and found it hard-coded in the htmlgen where that closing ) is written. It only goes to the same line if the function has 0 or 1 arg.

@akiss77, thanks for checking it.

LGTM :)

LGTM


Add reverted commits

This pr is to add the reverted commits in branch release/spring-cloud-azure-4.4.0, which are:

https://github.com/Azure/azure-sdk-for-java/commit/20b8ccd739bec6a8a9a7a14ece66907047b59342
https://github.com/Azure/azure-sdk-for-java/commit/601e145280661f0d9ab308a7fcc42e9e07115a79
https://github.com/Azure/azure-sdk-for-java/commit/60771871289ce784356bf721feede72e50759234

/azp run java - spring - tests

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-autoconfigure


Example tests are not fetched by the exercism client

We recently added example tests to this track, #420, #568, however it's been overlooked that because they have the word example in the filename these example tests are not being fetched by the exercism client.
I'm not sure where the constraint for this is located within the exercism codebase, so don't know about a solution that could allow us to keep these files named as they are.
Otherwise we can just name them something else, or move the example tests into the main test files, and they'll still run.

where the constraint for this is located within the exercism codebase

https://github.com/exercism/trackler/blob/master/lib/trackler/implementation.rb#L9

ah, ok, thanks @petertseng
Ideally it would be good to keep the example tests in a separate file named example_test.go as this appears to be the convention used in the language. This would mean changing the above regex to something like example(?!_test) (see https://regex101.com/r/ifNpuG/1). I don't know what unintended side effects on other languages this would have. Do you know @petertseng @kytrinyx? Would it be worthwhile opening an issue regarding this in the trackler or discussions repos?
If we can't go this route then we have these two options:

use a different filename, for instance sample_test.go or ex_test.go (these are not good, I know, purely illustrative) which is problematic because it introduces a convention that is exercism specific, not Go specific.
keep them in the standard test file <exercise>_test.go, which is preferable, but in exercises where there are example tests alongside the standard tests, currently only hello-world and clock, this could make those test files rather long, though this is not a problem for any future exercises that only have example tests.

We can use a custom regex in the config.json file at the root of the repository.
https://github.com/exercism/request-new-language-track/blob/master/config.json#L6

Ah, amazing, thanks @kytrinyx. I assume that regex isn't a complete replacement for the one in Trackler, so in the config.json, should it be the filename, ie example_test or example_*_test? Does it need the file extension?

@robphoenix The regex is a complete replacement for the one in Trackler. If the config has a regex, then we use it. If not, we fall back to /test/i.
It doesn't add any further restrictions at the boundaries (unless the regex in the config does), so you can leave the extension off if you want.
Not all tests will have example in them, so you'd need a regex that handles both example_tests and other tests.

Sorry, but I think I'm a little confused.
I was thinking of the regexes linked to by @petertseng here: https://github.com/exercism/trackler/blob/master/lib/trackler/implementation.rb#L9
    IGNORE = [
      Regexp.new("HINTS\.md$"),
      Regexp.new("example", Regexp::IGNORECASE),
      Regexp.new("\/\.$"),
      Regexp.new("/\.meta/")
    ]

but you're referring to this regex, right?https://github.com/exercism/trackler/blob/master/lib/trackler/track.rb#L78
    def test_pattern
      if config.key?('test_pattern')
        Regexp.new(config['test_pattern'])
      else
        /test/i
      end
    end

Does one override the other? As in, if we define a test_pattern regex, (say /\w+_test/i) will everything matching it then not be ignored? But then the current regex /test/i should cover example_test.go shouldn't it? But it's still ignored because of the IGNORE array?
So is it then that anything with example in it is ignored after the test_pattern has been matched, and so the test_pattern regex is redundant as files with example will still be ignored no matter what the test_pattern is?
I've had a look through the Trackler, exercism.io & x-api code, but I don't know Ruby (not much of an excuse though) and I couldn't figure out how a filename passes through the test_pattern regex and the list of regexes to ignore (does that make sense?)
I feel like I'm being really dense here, but I think in this case I'm going to need it explaining to me like I'm 5! Sorry, if I'm missing the obvious.

if we define a test_pattern regex, (say /\w+_test/i) will everything matching it then not be ignored?

Ooooooh. Sorry.
The answer is I have no idea, I will have to write a test to make sure that if it matches the test_pattern then it will never be ignored.

OK, verified. You are absolutely correct. If it's in the ignore pattern, it is unconditionally ignored, whether or not it matches the test pattern.
I'm going to make that change.

OK, PR open here: https://github.com/exercism/trackler/pull/32

Amazing, thanks @Insti & @kytrinyx you both rock! :guitar: I'll open a PR to close this today.

closed by #645

I've just tried an exercism fetch go --all with an empty exercism directory and the current master branch of the cli and the example tests are still not being fetched.

Have the changes been deployed yet?

Ok, cool. Thanks for looking into this @Insti, I appreciate it.

http://x.exercism.io/v2/exercises/go/hello-world contains example_hello_world.go

sigh, I obviously meant example_helloworld_test.go


KHR extensions - ability to turn off exporting

I disagree with the current way KHR extensions are automatically added to the export if default values are changed.
If the specular value is changed from the default, then it will automatically add the KHR_materials_specular AND the KHR_materials_ior extensions.
Describe the solution you'd like
A feature added to export dialog - checkbox and to turn off auto KHR extensions.
Describe alternatives you've considered
Returning specular and tint to default is not viable - no way to search for materials that have changed from default values.
Additional context
None

Hello,
This addon is based on behavior to export as close as possible what the user see, including material nodes. If use changed the default value, it makes sense to include this information into the exported file.
I am not sure I want to add more UI option to be able to turn extension off, as these extensions are a great enhancement to PBR materials.
A solution should be to add some hooks to enable/disable extensions, and creating an hook-addon with UI to choose extensions to keep.

Ok will try the route of adding a hook to block what I don't want. Closed