Add important wiki links to README

Important project links to coding-style, build, faq, and code review process should be accessible from the project README.  The README is displayed on the first page when users come to this project site and the README is often the first thing users read after cloning the source.  Adding the links to the README would bring them front-and-center so that they are not overlooked by potential users.

fix by #402 .thanks uartie


Generic type not accepted as function return type

The following library fails to compile since T isn't recognized as a type.
library foo;

pub enum Foo<T> {
    A: T,
}

impl Foo<T> {
    fn bar(self) -> T {
    }
}

error
  --> return/src/main.sw:8:21
   |
 6 |
 7 | impl Foo<T> {
 8 |     fn bar(self) -> T {
   |                     ^^ Unknown type name.
 9 |     }
10 | }
   |

Closing as a duplicate of #814.


Implement support for TextFormat letterSpacing

Setting letter spacing on neko/c++ targets has no effect.

Closed in favor of: https://github.com/openfl/openfl/issues/1700


use JSONObject as the structure for claims

Debugging Test is failing -> same result as in main branch
Used JSONObject as an option instead of <reified T> which is know at compile time
Added functions to validate JSONObject structures used to construct claims in credentials and presentations
Added parameter claims in method createPresentation(credential: String, claims: JSONObject, releaseClaims: JSONObject, audience: String? = null, nonce: String? = null, holderKey: JWK? = null) used to validate against the structure of  parameter releaseClaims
added several test cases to cover some cases where validation should fail

Thank you very much for you PR. This is a cool new feature but it should be consistently integrated with no code duplication. As far as I can see only the createCredential method can handle a string as input now but I think the createPresentation and verifyPresentation method also has to be able to do this.

Hi Fabian, yes I am currently doing that and also trying to improve the tests. An update of the PR will follow soon.

There is still a problem because the commits are not signed. Can you please sign the commits? After that I will merge the PR.

There is still a problem because the commits are not signed. Can you please sign the commits? After that I will merge the PR.

As far as I can see, all my commits were signed. However, according to the github message, this needs to be done on the base branch. A workaround for that is that the "rebase and merge" option for the PR should be selected. Maybe you need to do that since the PR is merging into your repository? Otherwise, do you know what should I do to solve this from my side? unfortunately I do not have a lot of experience with working in github.


Review iOS APIs

As a developer, I need that all APIs matches through different environments.
Review iOS APIs to match Dart SDK APIs.
Acceptance criteria:
1 - Updated APIs, with equivalent parameters.


Autosize diagram

Is it possible to autosize the diagrams to the size of the display?

If you're using the Structurizr web renderer, you can press the c key to zoom to fit content -> https://structurizr.com/help/keyboard-shortcuts


Add more effective multi-auth backend tests

Exercise each of the backends through the MultiAuthBackend to ensure they're adhering to
the expected contract.

Coverage decreased (-38.06%) to 54.545% when pulling 136b04926626e098210f53d8f30d1a02e09931e9 on jcwilson:master into 9287440aedd8c8e91c3cf6ed7bf9fda36d0ddb5b on loanzen:master.


Whether I can use this code on windows10?

It depends on whether you have a windows-compatible Caffe, I suggest you use our new code based on MXNet, which support windows.

@Orpine ok. Thanks a lot.


Restart broken Olm sessions

@manuroe commented on Thu Dec 13 2018
Spec: https://github.com/matrix-org/matrix-doc/pull/1719
JS PR: https://github.com/matrix-org/matrix-js-sdk/pull/780
and this bug fix on this feature: https://github.com/matrix-org/matrix-js-sdk/pull/792

I believe I’m running into this from time to time as some users are unable to read my messages despite being verified. I was able to workaround it by blacklisting all of their devices (or just the affected one if you know), unblacklisting and re-verifying them. That seemed to reset the session and resolve it.

https://github.com/poljar/matrix-nio/blob/master/tests/encryption_test.py#L899 is a test showing the scenario.


Update map.less to fix flash messages

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (e.g. Closes #XXXX): Closes #8505

Description

Only one line has been added in the class "flashMessage".
Flash messages should be centered now instead of being offset.
Before:

After:

Is this the correct place as seen in the screenshot? I think to keep it consistent the message should go to the top of the page below the nav bar as in other pages.
/cc @hallaathrad Can you QA if this is the expected behavior?

@raisedadead That is where it is supposed to be on the map page

@cashhh Nice work!

@BerkeleyTrue Thanks alot!


chore: remove caching for synpress runs

Disable cache for synpress run to avoid issues with old version tests

Thanks, but why we need this?

I suspect that it can be the same issue with caching as we had with deployment, so I'd get rid of this.
Anyway, it's only +30 seconds for each run, so it's not critical.


[request] Rename commands for logging

Currently, if one types "git st" in the Command Pallete, the suggested commands begin with "GitSavvy: start logging".
This is very unlikely to be what the user wants in nearly all cases. The user is most likely to be expecting "git: status" to be the first suggested command. Also, since "git st" and "git s" are very often git aliases for status, this is the expected behaviour.
So, my suggested fix is:
Please rename "GitSavvy: start logging" and "GitSavvy: stop logging" to "GitSavvy: enable logging" and "GitSavvy: disable logging".
I don't think such a change will negatively anyone's workflow but will improve the UX for some others by making "git: status" on the command palette accessible with the same command as on the command line.

Seems like a reasonable request.  I've found that Sublime learns pretty quick what I actually want, since it re-orders the displayed options based on previous selections.  However, no need to make this more difficult than it needs to be.


Internationalization

Internationalization, 国际化, интернационализация, internationalisation, internationalisierung, 国際化, internacionalización, umiędzynarodowienie, internacionalização, 국제화, internazionalizzazione, internationalisering, internacionalizace...
If you understand any of these languages (or any other language) I would be really like if we could work together to translate this handbook.

cn.md should be change to zh-hans.md and zh-hant.md which stand for simplifeid chinese and tranditional chinese.

I will take care of Spanish, I am from Argentina but I will try to make it as plain Spanish as posible.

I'm setting up https://crowdin.com for translations, you'll probably just want to wait for that

Will wait then, post a comment here when is ready!

интернационализация

I'll take this one

I take 中文 zh-cn.md

I can help zh-cn.md

I can help with the Dutch translation if needed

I can help with russian translation.

I can help with Traditional Chinese 正體中文 zh-tw.md

国际化, I'm in.

国际化, I'm in.

国际化，I can help with simplified chinese.

国际化，I can help with simplified chinese.

Hey Everyone! I've setup Crowdin for translations, so if you want to contribute a translation you can do so now. I've updated this issue body above with instructions!

:boom: I've setup a digital ocean droplet to update this repo when translations are completed, please let me know if you notice something go wrong

Simplified Chinese is now available at: https://github.com/thejameskyle/babel-plugin-handbook/tree/master/translations/zh-CN thanks to @nightire

I can help with translate docs into Korean (ko-KR). I'll work soon!

@thejameskyle Just FYI, I've fix some broken sentences on Crowdin.

I can help with Hebrew translation

I'll help with French

I'll help with Italian!

This is so cool, thanks everyone for helping out! Lets keep it going!

I will translate to Romanian. Happy new year to everyone!

Please change "Romeno" to "Română", in the translated language list.
Thank you.

Whoops, fixed

Romanian finished.

That's awesome! It's times like this I really love open source

:hankey: You may have just received a notification that I uploaded new translation strings for a lot of stuff even though the strings haven't changed. I'm looking into it now

Would everyone here mind linking me to any language-specific JavaScript communities that we can reach out to? Any forums/newsletters/blogs/social networks?

I remember hearing about iojs translation<EMAIL_ADDRESS>

That's awesome

What about to add base hash to get a difference between original and translated ? such as https://github.com/interagent/http-api-design

@sotayamashita I'm sorry, I don't follow

well, what @sotayamashita means is like:
Portuguese version (based on fba98f08b5), by @Gutem
Spanish version (based on 2a74f45), by @jmnavarro
Korean version (based on f38dba6), by @yoondo
Simplified Chinese version (based on 337c4a0), by @ZhangBohan
Traditional Chinese version (based on 232f8dc), by @kcyeu
Turkish version (based on c03842f), by @hkulekci

but it won't be necessary in our case, @thejameskyle setup a automatic tracking & updating system, we, as translators, should do our work on Crowdin.

@nightire Oops. I do not understand how Crowdin works.

Again, zh-CN should be renamed to zh-hans (because zh-CN means Chinese used in mainland China but Singapore also use simplified Chinese), and zh-TW should be renamed to zh-Hant (because zh-TW means Chinese used in Taiwan but Hongkong and Macao also use traditional Chinese).

@hax the original translated version is named as zh-hans as you suggested. However, since @thejameskyle setup Crowdin as translation platform, he didn't use my version directly,  but I agree with you on naming conversions.

I found that Crowdin have zh-HK, zh-MO, zh-SG (though titled in "Chinese Traditional, Singapore" which is totally wrong) separately. In practice, I never see any project provide different translations with zh-HK vs zh-MO (because Macau is too small to have any significant difference with Hong Kong in language usage), or zh-CN vs zh-SG (because Singapore adopt mainland China standard). So it's the fault of Crowdin, but I expect this project could fix the problem in automation tool.

Fixed with https://github.com/thejameskyle/babel-plugin-handbook/commit/aa1a8d48420f1c9015a6fec5377761dd62a197b2 and https://github.com/thejameskyle/babel-plugin-handbook/commit/10d84b892382e6e720a12e17a70cde9e0e78bc4d

all links in Table of contents are broken now for any translation

Yeah, I have no idea how to fix that because they are just the heading titles which are based on the text and crowdin won't let me update the links per translation

well, this could be an opportunity to improve crowdin itself :laughing: maybe translators should update links manually on crowdin?

It's not currently possible, I can email them about it though

You can just add  to each heading and link to that in the TOC, although I think that wouldn’t work well with GitHub’s autogenerated IDs as there would be two IDs to reference a heading.

You can just add  to each heading and link to that in the TOC, although I think that wouldn’t work well with GitHub’s autogenerated IDs as there would be two IDs to reference a heading.

:+1: ... to avoid this, we can use a difference name from heading, or add a prefix to each IDs.

I'll contribute to Russian as well. I've noticed that there are a lot to do for now.

Count me inn on the Norwegian part.

The single file README.md in Finnish, translated completely. Probably can continue tomorrow with other parts.


Add Divider component and an example

This PR adds the Divider component and some examples for it.
The implementation can seem a little bit shaky, especially the fact that we have to define dividerWidth for the Divider if we want to show text, but I was trying to get it to look the same on most of the browsers, and we accomplish that.
I considered dividing the component into two components (Divider and DividerWithText), but finally, I decided not to split the component into two. I think it works best the way it is right now
What do you guys think? @iamhectorsosa @jvorcak

I expect that the divider would have a smaller base width that my email and it will still render centered, no?

The Divider unfortunately does not render centered if it has a smaller width than the email width.
What I instead did, I made one example smaller in width, and passed that width to the Divider's baseWidth as well so we can show more clearly how that prop can be used.


modifications of deployer support

I think we need several modifications of deployer (and download) support.

I don't think multiple deployers need to be chained as the output dictionary has already been processed well when it arrived at the deployer. I think that framework supports a single deployer is enough.
I'd like the single deployer gets an Option[Dictionary] as an input parameter and outputs a Seq[String] from process function.
Then framework transforms the Seq[String] to Array[Byte].
The series of procedures described above is started by framework when either Deploy or Download buttons is clicked.
If the user clicks Deploy button, framework transports the Array[Byte] to the registered server (Solr/ES). Otherwise (user clicks Download button), framework transports the Array[Byte] to web browser that the user is using.

I'll open some pull requests for each procedure.

Hi Yamamoto-san,
I've just pushed a branch framework_support_transfer_bytearry which includes a list of items described above except Download button stuff. Can you check it?

I'm going to merge this soon...


Decide how to handle references for images in citations

It is best practice to actually look at an original image rather than assuming that the FreeBMD transcription is correct.
It is best practice in genealogical citations to reference the original image, not the finding aid.
Can we achieve best practice - i.e. generate citations that include the image looked at (e.g. UKD-01/1841D1-0128.tif or 1837B3-B-0017.jpg, etc.
See also systemwide #154
Will this require tickboxing the image(s) looked at?  Would it be sufficient simply to list all the images which are associated with the record (in the citation, not the url)?

I think it will be fine to simply list all the images in the citation, and the researcher can edit if they want to specify the one looked at.

Decision made and new story created/updated so now closing.


Determine new naming conventions for IF

Story
As a user or developer I want clear naming conventions across the codebase that help me to understand how IF executes.
Rationale
We are refactoring the IF code and using the opportunity to update any outdated names.
Implementation details
Agree a set of names that need to be updated and their replacements
Priority
5/5
Scope
Will affect all of IF
Size
1.5 days
What does "done" look like?
New set of names agreed
Does this require updates to documentation or other materials??
New documentation will be required
What testing is required?
none
Is this a known/expected update?
expected

Closing as complete


Is it possible to change the port number and format?

Description:

In my application I need to receive data from tcp connection. I had some questions when I was testing this extension.

The port number seems to be fixed. Is there any way to change it without modifying the source code?
There might be some self-defined protocol in tcp message. Is it possible to define the format ourselves? (like writing a custom map extension)
The data source only sends data by responding a transmission request. Is it possible to make the source actively request for data (by sending tcp message)?
Thank you in advance!

Suggested Labels:

Suggested Assignees:

Affected Product Version:
2.0.17
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

@szjiajin,
Please find my answers below,

Port number can be overwritten as a system parameter if you are using SP please see the following doc on the setting system parameter. else you can set it using inMemoryConfig provider.
Yes, you can write a custom map extension and use it with TCP.
As of now, this is not possible.

Best Regards,
Niveathika

Closing this as it is answered


Adopting Swift 6 language mode

This work is not complete in its present form. The main app itself builds in Swift 6 language mode with these changes, but due to problems with transitive language mode affecting dependencies, the tests do not yet build.

I think we're really close now, or we might even be there with what we can do given the remaining warnings and expected upstream compiler changes.
I've added the output of a swift build --build-tests run to a gist for reference:
https://gist.github.com/finestructure/37cecd4c1b8b2c38c2850ff61358bb56
Total: 187 warnings

145 converting non-sendable function value to '@Sendable (from AppEnvironment)
42 code after 'throw' will never be executed (from throw XCTSkip, introduced to skip failing tests)

Xcode 16b2 counts this as a total of 41 warnings:

34 converting non-sendable function value to '@Sendable
7 code after 'throw' will never be executed

It's weird that Xcode counts differently and unfortunate that swift build doesn't give a count summary but that's what we've got.
@gwynne , I believe the converting non-sendable function value to '@Sendable warning is one that should go away with a compiler change/update, isn't it? I believe there was also a simple fix that we don't want to deploy because it'll be redundant with the compiler fix?

As to the tests failures, which I'm looking into, there are currently

78 failures across 26 tests
7 tests we're skipping due

a crash with NSInternalInconsistencyException in SnapshotTesting/AssertSnapshot.swift:317
a failure due to https://github.com/vapor/fluent-kit/issues/612

The Fluent issue probably affects at least some if not all of the remaining 26 test failures, TBD.

I've updated the gist after the last commit, we're now down to 0 concurrency warnings in our code 🎉

All tests are now passing with Xcode 16b3. It's possible now to use Xcode 16b3 as the daily driver for development. (We already build and run with Swift 6 in the base image for deployment, so 15.4 was technically lagging behind.)


Deprecate Localization.getLocalizedStringWithNamespace

Localization.getLocalizedStringWithNamespace will be removed in 4.0, but for now it should be deprecated.
This is my first time deprecating anything, so let me know if there's any step that I missed.

Add to "Deprecations" section of NextVersion.md.


Error when generate openapi.json, Object of type 'type' is not JSON serializable

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from pydantic import BaseModel

class BaseResponse(BaseModel):
    success: bool = True
    message: str = "Success"
    data: dict = {}
    execution_time_ms: float = 0.0

class Detect(BaseResponse):
    data: dict = {}

router = APIRouter()

@router.post("/detect", response_model=Detect, status_code=200)
def detect(
    image_file: UploadFile = File(None),
    image_base64: str = Form(None),
    image_url: str = Form(None),
    detection: bool = Form(...),
) -> Detect:
    return Detect(
        success=True,
        message="detected in the image",
        data=response_data,
        execution_time_ms=0,
    )

Description
 File "/home/sam/Documents/onp/app/venv/lib/python3.8/site-packages/fastapi/openapi/utils.py", line 418, in get_openapi
    definitions = get_model_definitions(
                  └ <function get_model_definitions at 0x7f72d5015c10>
  File "/home/sam/Documents/onp/app/venv/lib/python3.8/site-packages/fastapi/utils.py", line 28, in get_model_definitions
    m_schema, m_definitions, m_nested_models = model_process_schema(
    │         │                                └ <cyfunction model_process_schema at 0x7f72d54f6040>
    │         └ {}
    └ {'title': 'Detect', 'type': 'object', 'properties': {'success': {'title': 'Success', 'default': True, 'type': 'boolean'},...

  File "pydantic/schema.py", line 617, in pydantic.schema.model_process_schema

  File "pydantic/schema.py", line 658, in pydantic.schema.model_type_schema

  File "pydantic/schema.py", line 251, in pydantic.schema.field_schema

  File "pydantic/schema.py", line 219, in pydantic.schema.get_field_info_schema

  File "pydantic/schema.py", line 982, in pydantic.schema.encode_default

  File "pydantic/schema.py", line 986, in pydantic.schema.encode_default

  File "pydantic/json.py", line 97, in pydantic.json.pydantic_encoder

TypeError: Object of type 'type' is not JSON serializable

Since the error got cut off half way, I can not know which part of the model is referring to type. I've removed Optional from model, and make it as simple as possible. But this error still shows throughout the app, I'm not sure whether it's my BaseResponse issue or function definition issue. Any suggestion on how to debut the issue?
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.75.2
Python Version
3.8.10
Additional Context
No response

Detect() is inheriting from BaseResponse (not BaseModel), I'd guess the error is somewhere in the parent class.

I am not able to reproduce with the example you gave. Everything seems to work for me.

I am not able to reproduce with the example you gave. Everything seems to work for me. What is response_data in your code?

Thanks for testing. Now I suspect it may caused by dependency on the endpoint. I'll test this issue later and report back

Finally solved this problem, the issue was due to use BaseModel and combined with SDK return data not being in correct format. jsonable_encoder converted all the code into the right format.
Thank all for helping.


err with too many colons

How can I solve this error showed as bellow.  My tiller version is 2.7.2 swift version 0.5.2, and after go build it , run with "/swift run --v=3 --connector=direct --tiller-endpoint=http://<IP_ADDRESS>:44134"  It is defferent in v0.5.2 of the endpoint-address ?
2018/06/08 07:54:16 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:17 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:18 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:50 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:51 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:53 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:55 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:56 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:54:58 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }
2018/06/08 07:55:00 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: address http://<IP_ADDRESS>:44134: too many colons in address"; Reconnecting to {http://<IP_ADDRESS>:44134 }

@jeffchanjunwei, were you able to solve this issue?

yes，http://ip:port should be replaced with ip:port only


MWPW-133818: Integration of interactive web components

Integration of interactive web components with change background feature
Resolves: MWPW-133818
Test URLs:

Before: https://main--cc--adobecom.hlx.page/drafts/suhjain/imarquee?martech=off
After: https://change-bg-marquee--cc--suhjainadobe.hlx.page/drafts/suhjain/imarquee?martech=off

@suhjainadobe The unit tests are failing. Could you take a look?

Unit Test cases:

Unit Test cases:

You can add the following ignore pattern for the library files in .eslintrc.js as they will continue to throw lint errors and should not be validated by the linter
  ignorePatterns: [
    '/creativecloud/deps/*',
  ],

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

Comparison is base (1875ebf) 100.00% compared to head (8633e38) 100.00%.
Report is 45 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #108   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         1    -1
  Lines           14        90   +76
=========================================
+ Hits            14        90   +76

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


Adding back to top

@vishanurag Kindly review the issue and assign this task to me.

Already done


Ability to specify which npm version to use

Describe the Enhancement
I would like to be able to select the npm version to be used to install dependencies.
Possible Solution
Introduce a BP_NPM_VERSION environment variable.
Motivation
There are sometimes reasons to use an older or newer version of npm than the default shipped with Node.js, like breaking changes or new features.

Closing this issue as I found this issue requesting this feature in the nodejs buildpack repo: https://github.com/paketo-buildpacks/nodejs/issues/270


Question: Why not unrelate

Good afternoon, I already defined my models and their relationships, when I try to insert a graphic, instead of removing the relation of the element that is not in the object, try to erase it queestionnaire data in db
This is request
                "id": 74,
                "id_parent": 67,
                "name": "Otra Administración",
                "question": "question for new node",
                "children": [],
                "typification_elements": null,
                "questionnaire": []
            }
This is the db data
`{
                "id": 74,
                "id_parent": 67,
                "name": "Otra Administración",
                "question": "question for new node",
                "children": [],
                "typification_elements": null,
                "questionnaire": [
                    {
                        "id": 4,
                        "name": "Nuevo Cuestionario",
                        "active": 1,
                        "questionnaire_question": [
                            {
                                "id": 13,
                                "id_questionnaire": 4,
                                "id_questionnaire_question_type": 1,
                                "name": "Pregunta 1",
                                "text": "Pregunta 1",
                                "options": null
                            },
                            {
                                "id": 14,
                                "id_questionnaire": 4,
                                "id_questionnaire_question_type": 1,
                                "name": "Pregunta 2",
                                "text": "Pregunta 2?",
                                "options": null
                            }
                        ]
                    }
                ]
            }
`
`const options = {
    relate: ['questionnaire'],
    unrelate: ['questionnaire']
  }
return Typification
        .query(trx)
        .upsertGraphAndFetch(req.body, options)
`

That should work. Without more information about the models, I cannot really help you. Could you post your models here too?

No answer, closing.

@koskimas i can open again issue?
Sorry, i was sick


Sergey Romanenko just invited you to the morfy-cms/morfy chat on Gitter

Why am I being invited to Gitter for a project that I have never interacted with before?

Sorry, this is was wrong invitation


lieer: add module

Add 'programs.lieer', a tool for synchronizing a Gmail account with a
local maildir and notmuch database. Per-account configuration lives in
'accounts.email.accounts..lieer'.
Add 'services.lieer', which generates systemd timer and service units
to synchronize a Gmail account with lieer. Per-account configuration
lives in 'accounts.email.accounts..lieer.sync'.

Great work! I just added a few minor comments.

Thanks! Rebased to master in 9f46d516fa13df70768a3211827d14f456fa6e85.


switch "ups, offline" notification off

the "retry in 75 sec" notification should be a option (default off)

fixed since v2.12


EDAM operation not extracted

A quick look on https://docs.google.com/spreadsheets/d/1Nq_g-CPc8t_eC4M1NAS9XFJDflA7yE3b9hfSg3zu9L4/edit#gid=301051642 showed, that there are no EDAM operations present in the sheet.
But, e.g. https://bio.tools/antismash does have EDAM operations annotated.

Are there EDAM annotations in the wrapper?
This might be because we aren't extracting metadata from bio.tools yet.
Only the bio.tools ID is being extracted from the tool wrapper.

This is how ToolFinder extracts bio.tools metadata: https://github.com/AustralianBioCommons/australianbiocommons.github.io/blob/master/finders/toolfinder.py#L198

The script does look in bio.tools directly: https://github.com/bebatut/galaxy_tool_extractor/blob/ce4b94cd4404f7751fb15101598c4a0bcacd4b8c/bin/extract_galaxy_tools.py#L261-L275
And it works for the EDAM topic, but I need to check what the issue with the EDAM operation

It doesn't seem like there are any examples where it is working for operations.

I fixed it: https://github.com/bebatut/galaxy_tool_extractor/pull/4


rust-toolchain, windows build on CI

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 65.008%

Files with Coverage Reduction
New Missed Lines
%

core/src/api.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
1986

Relevant Lines:
3055

💛  - Coveralls


Builder::sort() does not convert property name to field name

Hi,
Seems the Builder::sort() method does not convert my property name in model class (private $firstName) to the real name in the document properties (first_name).
If I try this query, it does not sort:
public function getByFirstLetter($letter)
{
    $qb = $this->createQueryBuilder();
        ->sort('firstName', 'ASC')
        ->sort('lastName', 'ASC')
    ;

    return $qb->getQuery()->execute();
}

and the generated query in my Doctrine logs is:
db.clients.find().sort({ "firstName": 1, "lastName": 1 });

But if I create the following query, it does work:
public function getByFirstLetter($letter)
{
    $qb = $this->createQueryBuilder();
        ->sort('first_name', 'ASC')
        ->sort('last_name', 'ASC')
    ;

    return $qb->getQuery()->execute();
}

and the generated query in my Doctrine logs is:
db.clients.find().sort({ "first_name": 1, "last_name": 1 });

This is fixed now.


Data type not aligned

Dear author, during training the data type of img embedding is float16, but the input type for llama is bf16, I am wondering if this misalignment leads to data precision loss during training? Since float16 is much more precise than bf16.

Hi, thanks for your interest.
I thinks it's ok to use fp16 for img embedding since it can reduce GPU memory, and it's a common practice for other MLLMs like video-llama. However, it would be possiable to improve performance by using bf16.


Add basic configuration and info files

[x] README.md, stating your project title, styled with Markdown
[x] LICENSE file. Find template for it in plain text or Markdown format. You may use MIT licence for simplicity
[x] .gitattributes. Find some template for your repo on github
[x] .gitignore. Find typical template for your language (Python, Vim, Linux etc.) @ gitignore.io

You'll need to push a new branch (e.g. features/3-initial-config) with atomic commits and create a PR referencing this issue.

@webknjaz please review https://github.com/kpi-web-guild/django-girls-blog-serhii73/tree/features/initial-config

It is impossible to "review" a branch, only PR. Also you are not supposed to push to master.

Oh, you don't have master. So create it, create an empty initial commit there and then rebase your branch on top of it and create a new PR

I'm confused. @webknjaz can you just delete all my branches? I need a clean repository again.

https://help.github.com/articles/setting-the-default-branch/ - And I don't have a settings button.

I've switched default to master in settings

@webknjaz Thank you. https://github.com/kpi-web-guild/django-girls-blog-serhii73/pull/17 - I am create PR for this issue.
P.S. I right understand, in this project I don't need git commit like Resolves #4, right?

You may use keywords. You do not need a separate PR

@webknjaz https://github.com/kpi-web-guild/django-girls-blog-serhii73/pull/18


pkgid cmd: fix help msg with multiple packages

When there are multiple versions of a package that match a given spec
the command tells you to re-run it with the -p argument which does not
exist. The command appears to work without it properly.
Signed-off-by: Doug Goldstein<EMAIL_ADDRESS>

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks for the report! Perhaps though the -p argument could be added to cargo pkgid for consistency? This is pretty critical information for other commands like cargo build or cargo update, so I'd prefer to not lose the suggestion of -p where it's correct there.

Yeah we can do that.

@alexcrichton This should be good for another peek. I aimed to retain backwards compatibility for people that might have scripts that don't use the -p argument.

@bors: r+
Thanks!

:pushpin: Commit 1e7893c has been approved by alexcrichton

:hourglass: Testing commit 1e7893c with merge 02fed69...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing 02fed69e71806989a96ae372f5d0f7cbb9528c37 to master...


Demote redundant builtins to a standard library

As discussed in
#386

https://github.com/informalsystems/tnt/pull/386#discussion_r1039485690
https://github.com/informalsystems/tnt/pull/386#discussion_r1039482799

We have several builtin operators which should be simple to define as part of our standard library instead. E.g.,

notin
contains
neq
iff
implies
setBy: def setBy(m, k, f) = m.set(k, f(m.get(k)))

iiuc, there are advantages to moving this into the standard library:

It reduces the surface needed for anyone who wants to do a different implementation of TNT.
It should also reduce the surface area of our own implementation, reducing the chance for bugs in the TS.

Are there other gains/costs to consider?

See #388


Updating contact creates a new contact instead of updating

    const { account } = this.props
    const { address_book: addressBook } = account
    try {
      addressBook.contact_update(contactName, { name: contactName, recipient: contactAddress })

This should not create a new contact, can you list me your contacts prior to this call, and can you share the contactAddress and contactName when this happened?
Will add a unit test extra for it.

Don't think there was ever an issue in the code. Did change it a bit by storing it as a list instead of a dict, so we can fetch and check using the base-sensitivity name comparisons.
Added a unit test as well to ensure I can update an existing contact.
When you give the recipient in the opts parameter it will indeed create a new contact object under the hood. However, as long as you give the same name (in the opts param) as the name you specify the regular name parameter (as you do in your example), you'll be able to fetch the contact using the same name as you were used too.
The contact_update function also returns the contact object created and available for the specified name.
Still, I don't get what issue you really have here. As long as you do not try to store contact objects yourself (for more than just a direct usage within a props-flow), you should be good. Because if you do store objects for a longer time, and you update that contact while still having the object stored that used to represent the contact, then there will be indeed a disconnect between the 2.


allow non-Symbol object properties on RSAA

Here's the proposed change regarding Issue #188.
Dev setup was easier than expected and worked just as described! 👍Let me know if I need to change anything for convention, or add/edit additional tests or documentation.
This change only removes this requirement from the validateRSAA method, and from the documentation.  It doesn't seem as if anything else in the codebase requires the old behavior.
This is still a breaking change though and does effect the existing RSAA specification, so probably best grouped into a future major (v3+) release after ppl have time to discuss in this PR (and if we still want to make this change) :)

Thanks @darthrellimnad! This looks good to me.
Like I said in the issue, I don't know why this was part of the RSAA spec. At the same time, I don't see the harm in changing it.
Let's leave this open for a few days so people can chime in. If no issues are raised I'm happy to merge this in as-is 👍

@darthrellimnad I just cut a new next branch for a 3.0 release. Can you update this PR to target that branch instead?

Coverage remained the same at 100.0% when pulling d3741511aaed3b25ceea06cb36cd17055e6318a3 on darthrellimnad:major/issue-188-custom-rsaa-props into 3a19b1bed91135bf2caea4e285e48ab3678e23d7 on agraboso:next.

Awesome, thanks! I'm going to go ahead and merge this into next so I can publish a beta later today.

This has been released in 3.0.0-beta.0


Not working anymore on chrome, but working one Firefox and Edge.

Describe the bug
A clear and concise description of what the bug is.
VOD / Channel link
The url where you encountered the error
Browser
Chrome, Edge, Firefox, Brave, ...
Additional context
Add any other context about the problem here.

i use opera gx, all i did was clear the cookies and it worked again.

i use opera gx, all i did was clear the cookies and it worked again.

Deleting cookies doesn't work for me. I've tested it on two different PCs with different web browsers, but there's been no change

Please update to https://github.com/besuper/TwitchNoSub/releases/tag/0.7.1


WinUI 2 / 3 / WinAppSDK reorg broken links, regressions

From Twitter:

To reflect how #WinUI and the #WindowsAppSDK have evolved, we've decoupled the WinUI3 and WinUI2 API docs and merged WinUI3 into the #WinAppSDK docs. Let us know what you think.

This change--I couldn't immediately find the commit in the commit log--has broken all the "equivalent class" interior links throughout the docset(s). This is a big regression and makes finding useful WinUI documentation much more difficult.
Steps to reproduce:

Open https://docs.microsoft.com/uwp/api/windows.ui.xaml.visualstatemanager?view=winrt-22621
Click Equivalent WinUI class: Microsoft.UI.Xaml.VisualStateManager link
Observe 404

cc: @asklar @stevewhims @Karl-Bridge-Microsoft

Now I know why I was getting so many 404 errors.

@Karl-Bridge-Microsoft @asklar @stevewhims Would be great to at least get an ack. on this highly visible issue.

@riverar - thanks for following up on this. We’re trying to resolve a few expected and unexpected issues with this migration. Stay tuned.

@Karl-Bridge-Microsoft @stevewhims Is this something we can bump the priority for? Or revert and revisit later? This continues to impact production docs, broken links everywhere.

These broken link issues should be fully resolved. Please open a new issue if you're still encountering broken links.
Thanks.


Throw for uninstantiated dynamic module access

This is an alternative to #3, where instead of altering execution to ensure that checking export names on dynamic modules isn't possible before they have been instantiated, we always throw in this scenario.
To recap, the problem case as in #3 is:
a.mjs
import './b.mjs';
export * from 'dynamic';
console.log('a exec');

b.mjs
import * as a from './a.mjs';
console.log('b exec');
console.log(a);

Where dynamic is a dynamic module and if importing a.mjs first, we will find that b.mjs is executed before dynamic is executed, so that we will be logging the namespace exports of the dynamic module before it is executed.
The approach taken in this PR is using the fact that instantiate order exactly matches execution order, such that we can catch this problem during instantiation.
The check is that GetExportedNames is called on the dynamic module before it has instantiated, so we can do a null return here to indicate an error to then throw in the Runtime Semantics of GetModuleNamespace.
In addition, this fixes the fact that this was previously a bug in the instantiation process as well.
This comprehensively defends against invalid access to the dynamic module, so I would like to merge this soon and close out https://github.com/guybedford/proposal-dynamic-modules/pull/3.
Would value help on the review of the approach, and if this works ok in the spec format. The completion handling may need a second look.
//cc @bmeck @caridy

@GeorgNeis this implements the approach we discussed to deal with the reference issues in instantiate and execute. Would be interested to hear your thoughts on the approach.

I've merged this in, along with closing #3.

I just wanted to summarize the logic here again for reference as I was going through it again today:

If you have a reference to a dynamic module namespace or a namespace which reflects export * exports off of a dynamic module namespace then you must have that dynamic module as a dependency. The problematic case is where that dynamic module in question is unexecuted, and this only happens when there is a "circular leaf case" (as in the example here). We can comprehensively detect these circular leaf cases since instantiation order follows execution order, and the PR here ensures we throw whenever it happens in a dynamic module graph.


fix(orchestrator): Add lastRunId to overview endpoints

Return last workflow run id in /workflows/overview and /workflows/:workflowId/overview
Update v2 endpoints too.
Bump backend version to 1.6.3

Address: (FLPATH-1209)[https://issues.redhat.com/browse/FLPATH-1209]

@caponetto @batzionb is it ok to update the backend version?


Better handling of batch mode

In #17 we add some basic batch mode handling. It would be preferable to have a command line switch over this solution.

Basically look at the concept and idea of how things like apt/yum or similar implements the -y/--yes flag for automatically answer any questions that is raised to the user.


Lights Issue

Turning off the lights in google Chrome seems to leave a huge gap in between the player and the comments. Not sure if this is a huge issue considering I tend not to read the comments with the lights off, but thought i'd let you know anyhow.

duplicate of #1696

Duplicate of #1708


🚧Research and see what it will take for generating documentation

I have done the items below . . .

[X] I have updated the title by replacing the '<title>' section.

Description
This project item will deal with all of the research involved with generating documentation for Velaptor and other projects as well.
Once the research is done, use what was discovered to create other project items to get the work done.
Look Into:

[ ] DocFX

This will automatically generate documentation that can be pushed to GitHub pages.  GitHub pages is what will be used for the documentation resource for users

[ ] Look into what it will take to push the documentation changes to GitHub pages from a GitHub actions standpoint.  The idea is that every release will run a github action that will generate the documentation and then push it to GitHub pages
[ ] Look into Read The Docs which is a free service for hosting technical documentation for open source projects

Other:
Six Labors Image Sharp uses DocFX.
Acceptance Criteria
This issue is finished when:

[ ] Research data is collected to help drive further direction when it comes to documentation
[ ] Other project items created to do the work based on the research collected

ToDo Items

[X] Priority label added to issue (low priority, medium priority, or high priority)
[X] Issue linked to the proper project
[X] Issue linked to proper milestone

Issue Dependencies
No response
Related Work
No response

Groomed

Combined issue #22 into this card.


clr-tree-nodes dont overflow if bound by a bounding container

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
If the tree is very large and we want it to be should in a restricted space (a container which has fixed bounds of height and width and overflow set to auto), the tree should navigatable with scrolls on the container
Actual behavior
The tree can be scrolled vertically but not horizontally
Reproduction of behavior
https://plnkr.co/edit/37qhI0?p=preview
Environment details

Angular version: Not applicable

Clarity version: 0.10.0-alpha.3

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

I kind of have this fixed for my requirements though but would be better if the fix came from clarity
https://plnkr.co/edit/IQDp32?p=preview
check the app.component.css for the fix

The workaround doesn't work anymore, and that's a pretty tough fix as far as I can see. Marking as more complex than expected for now, we'll have to investigate more when we start working on it.
The problem is that the overall width changes when nodes are expanded or collapsed, but I think it's doable.

I tested this against latest master and don't see this problem if you put the tree view inside a container. It was rewritten in v2 and seems to be resolved. Closing as fixed unless we can get an updated demo.


Deploy Issue

I have below error when i run deploy command:  npx hardhat run scripts/deploy.ts
No need to generate any newer typings.
HardhatError: HH700: Artifact for contract "Greeter" not found.
at Artifacts._handleWrongArtifactForContractName (/Users/Waqar/Downloads/evm-master 2/node_modules/hardhat/src/internal/artifacts.ts:478:11)
at Artifacts._getArtifactPathFromFiles (/Users/Waqar/Downloads/evm-master 2/node_modules/hardhat/src/internal/artifacts.ts:592:19)
at Artifacts._getArtifactPath (/Users/Waqar/Downloads/evm-master 2/node_modules/hardhat/src/internal/artifacts.ts:275:17)
at async Artifacts.readArtifact (/Users/Waqar/Downloads/evm-master 2/node_modules/hardhat/src/internal/artifacts.ts:58:26)
at async getContractFactory (/Users/Waqar/Downloads/evm-master 2/node_modules/@nomiclabs/hardhat-ethers/src/internal/helpers.ts:91:22)
at async main (/Users/Waqar/Downloads/evm-master 2/scripts/deploy.ts:17:19)
Plz help

Deployment scripts aren't yet implemented, this script is attempting to deploy a generic hardhat test contract. The deployment script is not currently implemented, but will be once the initial legos are finalized.


Examples on how to export multiple app and network rules

In regards to Azure Firewall. I have a large data set of firewall config exported from my squid proxy, is there examples (script) or a way to export all that data into Azure firewall network rules (based on port) and app rules (based on fqdn) ?
Another issue is when a rule is applied, it automatically overrides all previous rules using automation is there is no append flag in powershell. How can one bypass this?
So far, I have a PS script that reads from a file and passes the config to Azure. I want to be able to put different rules in to the application collection rule. Eg: I have an app collection called AppCol1, I want to be able to put multiple rules by name into that app col.
Link to doc - https://docs.microsoft.com/en-us/azure/firewall/scripts/sample-create-firewall-test

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

ID: 2de5293a-ffcf-f22d-6c32-a0412dd1438b
Version Independent ID: 5c0414b8-690d-9cdb-16ef-d7d26724bfd2
Content: Azure PowerShell script sample - Create Azure Firewall test environment
Content Source: articles/firewall/scripts/sample-create-firewall-test.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@eosho I am not sure if we currently have a good way in the docs to bulk export a large list of rules. Your best bet is to do it how you are doing now, and add them via PowerShell. Here is a doc that outlines the process of creating an Azure Firewall and adding rules via PowerShell. As the doc outlines, you can use a Rule Collection to conglomerate multiple rules, and then you add multiple rule collections to a single firewall.

@TravisCragg-MSFT my current method isn't clean. I want to be able to demo this to my customer this week. Do you think you can help? I will ping you internally.

@eosho ping me if you need any further assistance. I am going to assign this issue to the author to potentially add examples of importing multiple rules via PowerShell or CLI.

@eosho
Going forward, Azure Firewall will use Firewall Policy, which will have an import function, and firewall policies can be applied to multiple firewalls. Firewall Manager is in preview. For more information, see https://docs.microsoft.com/en-us/azure/firewall-manager/
#please-close


Subtile mkv file

Hello,
Is it possible to read the subtitles embedded in an mkv file?

Hello,
Is it possible to read the subtitles embedded in an mkv file?

I‘m sorry, it not support.

Do you think it is possible to add this functionality?

Do you think it is possible to add this functionality?

目前实现不太可能。如果你想要实现这样的功能只能修改源生的插件，但是我现在集成的是Flutter官方的video-player插件。这个插件是封装了源生播放器，比如：iOS使用的是avplayer; Android使用的是exoplayer。
122/5000
Implementation is unlikely at the moment. If you want to achieve such a function, you can only modify the generated plugin, but what I have integrated is the official video-player plugin of Flutter. This plug-in encapsulates the source player, for example: avplayer is used in iOS; Android USES exoplayer.


feat: implement serialization module typings

Description:
implement serialization module typings
Related Issue:
resolve #120
Type of change:

[x] New feature / Refactor / Enhancement (non-breaking change which adds new typings)

Checklist:

[ ] All code follows the project's coding style and conventions.
[ ] I have performed a self-review of my own code.
[ ] My changes generate no new warnings or errors.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding updates to the documentation.

:tada: This PR is included in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Implement in-place merge sort

http://www.cs.ubc.ca/~harrison/Java/MergeSortAlgorithm.java.html

Hello,
I want to contribute by writing the code. Can you please tell me how to go about it. I know Java and the algorithm. But do not know how to contribute.

Hey! Thanks for your interest.
I have created a CONTRIBUTING.md you can take a look at. If you have any further questions, feel free to let me know.

Hey!
I wish to contribute to your repository. Can you please tell me the language you want the above-mentioned algorithm to be written in.
Thanks :)

@divyanshu-talwar It needs to follow the current implementation, so Python 2.

Okay thanks :)
Will get back to you in an hour or two.


typefilter-cache

Changes the cache creation and lookup for FilteredTypeResolvers to use its root class as key for the cache.

This is a duplicate of #64


fix #127

adds missing 'string' param in isEmptyValue check

Coverage remained the same at 88.235% when pulling 5551cb617adfa777581544b7aa45c3ad63911410 on zhaolins:master into def5a89bed084bbbb76c0222f1592acb6db9822f on yiminghe:master.

transform empty string to undefined for value of type number


Review Lab Culture chapter

@rwildermuth can you review the Lab Culture chapter? Thanks!

Under "Come prepared to be engaged," I think we could add "Be present (physically/technologically and mentally)." Also, I think the last sentence could be rephrased: "Try to consider how meetings can contribute something new to your work and productivity, rather than keeping you from them."
Under "Sustain a positive, safe learning environment," this may be a good place to express that we understand that not everyone learns in the same mode (text, numbers, drawings, etc.) and that we can learn how to adapt to the modalities of our lab members.
Under "Foster inclusivity within our group and greater community," this might be a good place to (re)state that we prioritize the safety of marginalized people over the comfort of the privileged majority.

Thanks Robert for reviewing and your suggestions.
For 1) I adjusted the final sentence. I think the first part of your comment is addressed by "come ready to do your work".
2) I added text recognizing that people learn in different ways.
3) I think this bullet is getting at something different from your comment (i.e. we are emphasizing our group's 'citizenship' rather than interaction and conduct). The text you refer to is important, but I think it is in it's right place in the code of conduct.


chore: upgrade deps

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Problem while building your documentation

Details

Project URL: https://readthedocs.org/projects/lima-doc/
Build URL (if applicable): https://readthedocs.org/projects/lima-doc/builds/6962290/
Read the Docs username (if applicable): same as GitHub

Expected Result
My documentation builds fine locally. Only the branch unsubmodule-doc is supposed to work (not master) at this point.
Actual Result
There was a problem with Read the Docs while building your documentation. Compilation fails after git clean -d -f -f command.

We see the following error here. I'm not sure if this is a wonky submodule config or a legitimate issue with our submodule handling:
Build failed with unhandled exception: Gitmodule path u'third-party/yat' did not exist in revision of parent commit HEAD

This isn't the only build we notice this type of error on, so this is pointing to a bug. Could you elaborate on the submodule config? That might help us determine the cause here, as I'm not quite sure what this error is currently.

Thank you for you kind support! It was indeed an issue with a ghost submodule. Deleting the submodule was the solution, not sure why I did not catch it when cloning and compiling locally. Thank you again.


Only download the last podcast

In my case ytpodcast can only download the latest video of the channel. And if in 1 day 1 channel upload 3 video then only 1 of them is downloaded and the other 2 are lost. This is certainly a problem. The url I use to config.yml is next:
https://www.youtube.com/feeds/videos.xml?channel_id=UCSQJX9lm4u92bx0XGpEIUiA
Some enhancement to introduce I do believe should be next:
1- Add the capacity to select how many times per day you want to check if new videos available

Right now you must use the url of YouTube for the channel or list... for example, https://www.youtube.com/@jordillatzer/videos. This will allow (in the future) download from other  services as Vimeo.
I add the url and download all the videos... please try it and tell me.

Hi, Thanks. I'll try and report you later. however use the URL in my opinion is not a good idea.
I do believe that if use the URL you won't be able to download the shorts and the lives"Directos". I assume you only will be able to download the videos under the category "videos":

So in the sample I show you will miss "SHORTS" and "EN DIRECTO". For me Twitch has much more interesting content than Vimeo. For me the service that you are developing is very useful.

However in case I'm right do not worry... I can create 1 feed for the videos, another feed for the shorts and another for the "lives"
May I ask how often the container updates? I think it's once per day But would be nice to know what time exactly


ScrollingObjectCollection.ManageVisibility takes too long when numerous objects in container

Describe the bug
Using the MRTK2 ScrollingObjectCollection, when the list of objects in the container is more than just a few (like in the example scenes) the ManageVisibility method, which is called every frame, takes too long and significantly impacts performance: from 60fps to 20fps!
To reproduce
Following instructions in page linked above, create a ScrollingObjectCollection. Add many buttons to its GridObjectCollection. Update the collection.
Using the Unity profiler, you can notice that the Camera pre-render or the LateUpdate (depending on the UseOnPreRender option) takes a lot of time.
Expected behavior
ManageVisibility is not called when the collection hasn't been scrolled.
This is the workaround I have implemented.

Hi @julienlang , unfortunately this issue does not meet the minimum requirements for addressing.  You may still reach out to our support team at https://aka.ms/hldevsupport and they will assist you with this issue.


Bangla language

I create bangla language and folder name (bn). this language work done . i think you add this bangla language .

Hello,
thank you, but validation.php is not translated ;)

Update Bangla language .

Final bangla language please check . Thank you

Thank you


New comment does not focus on rename

Hi,
I’ve encountered a bug where newly added comments require me to manually rename them (F2) instead of having the focus on the text edition automatically while bAutoRenameNewComments is set to true.
While investigating, I found that the CommentToASCMapping did not contain the newly created comment node GUID resulting in this code never entering in the if block.
if (TSharedPtr<SAutoSizeCommentsGraphNode> ASCComment = FASCState::Get().GetASCComment(NewComment))
{
	ASCComment->RequestRename();
}

https://github.com/AutoSizeComments/AutoSizeComments/blob/866a5a0e0707749840f1bf70b63889a98f67326a/Source/AutoSizeComments/Private/AutoSizeCommentsGraphHandler.cpp#L782
I am using ue 4.27.2 source, plugin 3.2.3.

Hmm it does appear this function is not being called. I don't quite remember why this was required because the renaming seems to work for me on 5.3 and 4.27.
If you breakpoint inside SGraphNode::RequestRename it should be called in the engine when added to the graph from SGraphPanel::AddNode.
        if (bWasUserAdded)
	{
		// Add the node to visible children, this allows focus to occur on sub-widgets for naming purposes.
		VisibleChildren.Add(NewNode.ToSharedRef());

		NewNode->PlaySpawnEffect();
		NewNode->RequestRenameOnSpawn();
	}

Can you check if this is working for you? Also check if a new CustomEvent node is being auto renamed correctly, cause it should be using the same flow to do so.

I added the breakpoint and never went into the SGraphNode::RequestRename function, the custom event added prompted me for renaming on spawn and the name was applied effectively, same for the replicated rename (Client/Server/Multicast), they were working fine on a custom event.
When breaking on the NewNode->RequestRenameOnSpawn() I went it the following call stack (both for comment which didnt work and for the custom event which did).

I'm really not sure how this can happen. Looking at UEdGraphNode_Comment, in the constructor it sets bCanRenameNode to true. Is this somehow false only when the node is created? Because otherwise it wouldn't ever let you rename if bCanRenameNode is always false.
So if you double click on the comment node title, does it let you rename? When you double click it should call RequestRename as part of my comment node logic in SAutoSizeCommentsGraphNode::OnMouseButtonDoubleClick.
There is one other class which inherits from UEdGraphNode_Comment, UMaterialGraphNode_Comment but this doesn't set bCanRenameNode to false either 🤔 Just to make sure, this is happening on a regular blueprint graph right?

Double clicking lets me rename the comment as expected, in a regular BP graph yes.

Thanks for letting me know and sorry for the lack of replies. I was on holidays and forgot about this issue :/
My guess would be you have a setting change on your project? If you are sharing the settings between your team usiing a DefaultEditorPerProjectUserSettings.ini, please send me this and I can test it again for you.
If not, then it would be unlikely that both you and your colleague are running into the same bug.
However if you would like to try resetting your plugin settings, you can by going to:

%Project%/Saved/Config/WindowsEditor/EditorPerProjectUserSettings.ini

and searching in here for [/Script/AutoSizeComments.AutoSizeCommentsSettings]
By deleting the lines in this section, you can resetting the plugin back to default. Of course, make sure to keep a backup before you do this :)

It could perhaps be a plugin or your custom engine change interfering but I think that is more unlikely.

Indeed the settings were at fault.
When setting bEnableCommentBubbleDefaults=False the issue is gone, I’ll settle with the absence of bubble by default for now, comment renaming is way more important.
My complete ASC section in the project repository:
[/Script/AutoSizeComments.AutoSizeCommentsSettings]
DefaultFontSize=18
bUseDefaultFontSize=False
DefaultCommentColorMethod=Default
DefaultCommentColor=(R=1.000000,G=1.000000,B=1.000000,A=1.000000)
bAggressivelyUseDefaultColor=False
bUseRandomColorFromList=False
HeaderStyle=(Color=(R=0.500000,G=0.500000,B=0.500000,A=1.000000),FontSize=18,bSetHeader=False)
+PresetStyles=(Color=(R=1.000000,G=1.000000,B=1.000000,A=1.000000),FontSize=18,bSetHeader=False)
+PresetStyles=(Color=(R=1.000000,G=1.000000,B=0.100000,A=1.000000),FontSize=18,bSetHeader=False)
TaggedPresets=(("TODO:", (Color=(R=0.000000,G=1.000000,B=1.000000,A=1.000000))),("TOCLEAN:", (Color=(R=1.000000,G=0.000000,B=0.000000,A=1.000000))),("DEPRECATED:", (Color=(R=0.000000,G=0.000000,B=1.000000,A=1.000000),bSetHeader=True)),("DEV:", (Color=(R=0.000000,G=1.000000,B=0.000000,A=1.000000))),("PROTO:", (Color=(R=1.000000,G=0.500000,B=0.000000,A=1.000000))))
bHideCommentBubble=False
bEnableCommentBubbleDefaults=True
bDefaultShowBubbleWhenZoomed=True
bSnapToGridWhileResizing=True
bHideRandomizeButton=True

Thanks for letting me know. I can confirm something odd is going on with the bEnableCommentBubbleDefaults. Not immediately obvious what is causing this but I'll hopefully get this sorted out soon.

Feel free to ping me here or on discord (nado1764) should you need any additional environment details to recreate the bug or to try a fix on my setup.
Thanks for your work and your plugin


docker image

sudo nvidia-docker run -it -v $HOST-SIMPLEDET-DIR:$CONTAINER-WORKDIR rogerchen/simpledet /usr/bin/zsh
[sudo] password for koufeitian:
docker: Error response from daemon: invalid volume specification: '-SIMPLEDET-DIR:-WORKDIR': invalid mount config for type "volume": invalid mount path: '-WORKDIR' mount path must be absolute.
See 'docker run --help'.

Please refer to this.

Hey, refer to https://dongjuns.github.io/simpledet/object/detection/simpledet-tutorial/
I explained some docker information for simpledet users.


Add group option

Use case: gather many items, and process them in variable-size groups (where the consumer may take a variable amount of time), without substantially slowing down the producer. For instance, gather groups for batch database operations.
If there's a better way to handle this in the existing pull-stream ecosystem, I'm all ears.
Ended up making slightly breaking behavior to avoid an off-by-one error - group indicates it shouldn't receive more than hwm values.

thanks! merged into 0.2.0


Possible other libraries

Hey! I've been keeping a list of these for a while as I've come across. Some may not fit into your plan so I figure it would be easier to post here instead of spamming PR's.

Spectre.css
mini.css
Stack Exchange Stacks UI - No public repo for this one
Bulma
Blaze CSS
Help Scout - Covers several categories
Heroku Purple3
Element
UIKit

As per #41 I think the Stacks UI and Help Scout ones fit into the company specific. Heroku's is super nice but based on Tachyons

Thanks for collating these! Yeah most of them look relevant I would say

Thanks! I saw you added them so I'm going to close this one.


Custom Cassandra Docker Image

python / cqlsh is not working on the 3.7 custom image.  Need to include probably more python modules, not just python-min

Not going to do custom image at this point.  Will continue with C* image.


The Insert API now skips processing GroupMemberships of RawContacts that specified no GroupMemberships

Problem
Prior to the changes made for #320, the Groups table was being queried for every new RawContact that is being inserted even if the RawContact did not have any GroupMemberships. The API was wasting CPU, querying the Groups table for absolutely no reason.
This is one of the causes of slow insertion speeds of large quantities of RawContacts as discussed in https://github.com/vestrel00/contacts-android/discussions/317#discussioncomment-7061779
Even with the optimizations in #330, we are still wasting CPU by not guarding against an empty group memberships list.
Solution
Skip processing GroupMemberships during insertion of RawContacts for RawContacts that specified no GroupMemberships

This was closed in https://github.com/vestrel00/contacts-android/commit/43275c6b5a9b7d0685738d3fa91a6ac3472ca0b6#r128691349

This is included in release 0.3.2.


Remove uniform lane index requirement for Wave

The HLSL compiler team has agreed to this change, which is more consistent with the later specifications, but failed to update this document. This is consistent with conformant implementations.

@pow2clk : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Mentioning @tex3d so he can see this and weigh in if needed.


Consider using Boost.PFR

Boost.PFR allows tuple-like access to aggregate types without the need to use any "ADAPT" macro. In my parser use cases I use aggregates for representing sequences, and would benefit from not having to use those macros.
Downsides:

another Boost dependency
slower compile times
compile errors from Boost.PFR will show up as errors from Boost.Parser.

Feel free to close this issue at any time. This is just a suggestion.

I already had this idea to make the example from #28 Easier to write, but I couldn't remember the name PFR!  So, thanks for saving me a lot of searching.  I decided to reimplement the bits of it that I needed.


StorIOContentResolver does not observe changes of uris on API < 16

We're using only onChange(boolean, Uri) which was added in the API 16, so on devices with API < 16 we're skipping all notifications…
Solution — override also onChange(boolean).
Priority: critical.

Fixed by #550.


Add test cases for Exercise

Codecov Report

Merging #87 into master will decrease coverage by 1.41%.
The diff coverage is 0.00%.

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


Programmatically open comment widget with API?

Hi there wondering if there is a way to open up the comment widget via the JS API without the user having to click an annotation.
Something like this would be super helpful
anno.loadAnnotations(url).then(function(annotations) {
   let annotation = annotations.find((annotation) =>  annotation == id);
   anno.openWidget(annotation)
});

Sorry anno.selectAnnotation(annotation_id) was what I was looking for.


Added albumName option for capture() function

Hi Loch,
Thanks for react-native-camera, great project.
I've added a little patch that allows you to set an album name that appears within the camera roll. it's only for Android, sorry but I'm not an iOS guy.
Tackles issue #356
Thanks, cheers
Miguel

Hi @lwansbrough
I've updated the PR so that it merges without conflicts. New refactor for adding Android video support, simplifies things.
Cheers,
Miguel

Is the 'albumName' option for capture() function available for iOS? Or if it isn't, can anyone help me with saving an image to a particular album?


How can I track the line number of the error

I was trying to load some .js files to my program, so I called duk_peval_file() function:
 if(duk_peval_file(ctx, "myfile.js") != 0)
 {
 	printf("eval failed: %s\n", duk_safe_to_string(ctx,-1));
 }

Since I would like to see the error message if eval failed, I asked the program to print out script errors. Sometimes the error message contains the line number where error occurs, like:

"eval failed: SyntaxError: parse error (line 42)"

But somethimes the error message just looks like this:

"eval failed: TypeError: undefined not callable"

Since no line number info is shown, it is difficult for me to figure out where the error occurs. It is possible to use print statements to locate the error but it is very time consuming and inefficient. So I am just wondering if there is way to help me track the line number for any type of error? I appreciate it if you can help me with this. Thank you in advance.

Error objects have a few properties which you can look up:

.fileName and .lineNumber will point to a (usually) useful file/line associated with the error cause.
.stack contains a stack trace which provides file/line information for all call stack levels

You can read these properties off the error using duk_get_prop_string():
if (duk_peval_file(ctx, "myfile.js") != 0) {
    duk_get_prop_string(ctx, -1, "stack");
    printf("Eval failed:\n%s\n", duk_safe_to_string(ctx, -1));
    duk_pop(ctx);
}
duk_pop(ctx);

In Ecmascript code one would do something like:
try {
    eval("aiee");
} catch (e) {
    print(e.stack);
}

You need to read the .fileName and .lineNumber properties of the Error object.  So:
duk_get_prop_string(ctx, -1, "fileName");
duk_get_prop_string(ctx, -2, "lineNumber");
/* ... err fileName lineNumber */
const char* msg = duk_safe_to_string(ctx, -3);
const char* filename = duk_safe_to_string(ctx, -2);
int linenum = duk_to_int(ctx, -1);
/* error handling here */
duk_pop_3(ctx);

Haha, @svaarala beat me. :)

Thanks for your help, @svaarala and @fatcerberus. The problem has been solved, it saves me lots of time.

You're welcome :) I'll add a note to duk_pcall() API document because it's quite a basic thing related to protected calls.


Replace use of tsinfer.SampleData with sgkit

This should greatly facilitate all sorts of analyses of the genotype data.

For now, the quickest way to take advantage of sgkit is to get everything into VCF files before comparisons.
For example,
beagle_vcf_file = "../analysis/beagle/target.beagle.vcf.gz"
beagle_zarr_file = "target.beagle.zarr"
vcf_to_zarr(beagle_vcf_file, beagle_zarr_file)
beagle_ds = sg.load_dataset(beagle_zarr_file)
beagle_ds

Addressed in a separate repo (see tsimpute issue 99).


ci: lighthouse

Successor of #927
This is not the final copy of the action probably, this is just to get started, we can improve it as we use :D
@eddiejaoude you need to do this before this PR can be merged: https://github.com/GoogleChrome/lighthouse-ci/blob/main/docs/getting-started.md#github-app-method-recommended

Can we use an existing Action like this one? https://github.com/marketplace/actions/lighthouse-audit

It seems to not be maintained well @eddiejaoude
__
  Summary: :hammer_and_wrench: This code needs a fix  |  Tags: Not maintainable

I'm going to close this for now, hopefully we can find an action that does something similar instead.


feat: add opentelemetry 1.0.0RC3 release

update opentelemetry with release 1.0.0RC3

:tada: This PR is included in version 1.225.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


"0 final doses" in doses per day/week charts

https://mathiasbynens.github.io/covid-19-vaccinations-germany/#doses-per-day
https://mathiasbynens.github.io/covid-19-vaccinations-germany/#doses-per-week

Fixed in 9c0133e33ec4cda0886b62e1b03673e62b7009ee + subsequent imports.


Enable the data sync job to run automatically in prod

Can we please enable the data sync job to run automatically in prod?
Testing started in prod with corps and data is not currently getting copied back to cprd.
Could we please prioritize this request? Thank you
@deetz99 @thorwolpert

Done


Not marked as python3.4 compatible in PyPi

Hi there,
http://docs.python-requests.org/en/latest/ says there is support for python 2.6-3.4, but PyPi lists only 2.7.  I discovered this when i used the tool caniusepython3 and saw requests on the list of packages holding me back.

Yeah, all our trove classifiers seem to be missing on PyPI. Looks like it was a while since it was registered. Resolved now. =)
Thanks for the report!


Door sensor timeout

Since v6, there seems to be a change of behavior in sensor timeout.
I happen to leave door open all night to refresh a room and automatically turn off the heat pump.
When I wake up in the morning, the heat pump should be still off but that's not the case. What is the timeout for a sensor to be considered "dead"? Can we extend it?
Thank you

I have also seen something like that but because I have some automation, I wonder if one off the automation tries to turn on the heat pump.
May be there is something to enhance or fix. I will have a look.

In my case i have several schedules to change the Preset e.g for night.
If preset changes and the window is already open, then is the action (in my case the thermostat should choose the preset für frostprotection if window is open) ignored.

Hello, I change the behavior on https://github.com/jmcollin78/versatile_thermostat/releases/tag/6.3.4.
Please give it a try and I think this will solve this issue.
Please reopen, if you find out the issue is still there.
Thank you for your report.


[Add School] Adding usmba domain

Add usmba domain

Add usmba domain

Add usmba domain

@Ardouz11 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host usmba.ma not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


bumps ansible to 2.15.4

Issue #, if available:
Description of changes:
We are bumping to the new image-builder in eks-a which requires a slightly newer ansible.
https://github.com/aws/eks-anywhere-build-tooling/pull/2695
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Support new csproj format projects.

Hello, we tried to use this promising tool to analyse dependencies in our projects, but found that the tool does not work for .NET Standard .csproj files.
Is there any interest in the community for this? Any alternative?
Thanks.

Hi,
I think it should be relatively simple to implement this. The problem is that I have some other tools for which I have to add support .NET Standard support and which are more important to me (e.g. https://github.com/rsuter/NuGetReferenceSwitcher).

Ok it was simple... v2.0.0 now supports .NET Standard. Please test and report any problems.

Wow, so fast. We tried it and it was working correctly... Thank you! 😄


Revisiting default parameter settings?

Hello all,
I came upon a recent JMLR paper that examined the "tunability" of the hyperparameters of multiple algorithms, including XGBoost.
Their methodology, as far as I understand it, is to take the default parameters of the package, find the (near) optimal parameters for each dataset in their evaluation and determine how valuable it is to tune a particular parameter.
In doing so they also come up with "optimal defaults" in Table 3, and an interactive Shiny app.
This made me curious about how the defaults for XGBoost were chosen and if it's something that the community would be interested in revisiting in the future.

@thvasilo Seems to be a good read at weekend. ;-)

I'll ping the main author @PhilippPro in case he wants to chime in on recommended defaults.

Hey @thvasilo ! I think the defaults in xgboost are not chosen to have the best performance. The user also has to specify nrounds by himself. Probably the defaults are chosen to provide a basic version of gradient boosting. Maybe it would be nice if some good defaults would be at least described in the help section. It is a bit nasty if you want to use a package and have to search the internet to make it work properly. ;)
There is also an autoxgboost package (https://github.com/ja-thomas/autoxgboost), but it is not working very well, sometimes it provides nonsense results and performance is worse than other auto tune packages such as tuneRanger or liquidSVM. See here for a graph of the benchmark that I did on some regression datasets: https://github.com/PhilippPro/tuneRanger/blob/master/benchmark/figure/rsq_results.pdf
The benchmark is described here: https://github.com/PhilippPro/tuneRanger/blob/master/benchmark/figure/rsq_results.pdf

This paper is very interesting thanks @PhilippPro. There are a few issues with directly adopting the default parameters from Table 3. The paper refers to classification datasets only. How confident are we that these parameters are effective for regression/ranking? This actually brings up a very interesting question: are our regularisation parameters invariant to learning objective? For example min_child_weight will be much more restrictive in binary classification where the Hessians take on very small values, as compared to squared error regression where the Hessian is a constant.
The default value of 4168 boosting rounds with a low eta is almost certainly better but imposes a much higher computational burden. What if someone goes to test the algorithm and it takes 5 minutes to run? I still think it might be a good idea to use a higher default, but this is just a consideration.
What is the role of dataset size in the effectiveness of hyperparameters? I feel like small datasets benefit strongly from regularisation where large datasets often do not seem to benefit at all.
@PhilippPro if you are interested in going further with this I would love to adopt your work in xgboost. I think there is potential here to dramatically improve the results for a large portion of our user base.

Hi @RAMitchell ! I am interested. I agree with you, that the nrounds parameter is too high. I think it would be better to optimize nrounds in a restricted hyperparameter space, e.g. set the maximum nrounds to 300.
And yes, regression (can you do ranking in xgboost currently?) is another problem that should be considered. There is also the option to use other defaults for classification and regression, as done for example in several random forest packages.
The other thing you mention is the problem of hyperparameters dependent on dataset characteristics. Here you should be very sure about the relationship to set hyperparameters data dependent, e.g. on the number of observations or the number of features p (e.g. in random forest mtry is set to the square root of p). A paper which tries to set these defaults empirically can be seen here, but maybe it is better to specify this rule by "hand" (e.g. humans looking at some plots that show the relationship between dataset characteristics, hyperparameters and performance). I do not know much about theoretical considerations in this field regarding xgboost.
@pfistfl are the results of the new bot already usable, could we use this for the purpose here?

@RAMitchell
I could rerun the results with a restricted nrounds. To what value should I restrict it? Or what would be your idea?

I think 100-500 is more reasonable - the goal would be for the user's application not to hang unreasonably when they first try the library. We have several places in language bindings where default parameters arise, we could start with Python APIs (or whatever your preference is) and worry about the rest later.
Another way of approaching this would be for us to provide some kind of dictionary of preset parameters in our API. This gives more flexibility and choice.
This is all really about user experience and lowering the barrier to be able to train models effectively. This would go hand in hand with examples and documentation.

@PhilippPro any update on this? Here is what I propose: rerun with 500 rounds and a couple of regression datasets. Confirm run-time is acceptable. Set these as the default parameters, create a documentation page on parameter tuning with a few notes on methodology, how you arrived at these and linking to your paper.

@PhilippPro any update on this? Here is what I propose: rerun with 500 rounds and a couple of regression datasets. Confirm run-time is acceptable. Set these as the default parameters, create a documentation page on parameter tuning with a few notes on methodology, how you arrived at these and linking to your paper.

I have not forgotten it, but I currently do not have a lot of time. I can only rerun it on the existing datasets, as I have the data for this, but this is not a problem. Your proposition is good, I will follow it, thanks.

@RAMitchell I got some results now which are not really stunning. ;)
I created a blog post, where I describe the results:
New xgboost defaults
Currently I am repeating the 5-fold CV to get more stable results and will update the results in the blog post tomorrow. Future work (as described in the blog post) will be a bit more interesting.

Awesome work thanks! I think there is moderate evidence for changing the default parameters settings. Shall we re-evaluate once you have results from CatBoost/LightGBM?
Note that we are mostly focusing new development on "tree_method":"hist" and"tree_method":"gpu_hist" which exposes the "grow_policy":"lossguide"/"depthwise" parameter. One of these growth policies might be definitively better on your datasets.

Yes, that's fine. The results I got today (with 10 times repeated 5-fold CV) were slightly better for my defaults (75% better in case of Spearman's Rho), I updated the post.
I will be happy to evaluate your new parameters once they are readily implemented in the package.
I think Catboost will be better in the default mode, but I will see the results soon.

@RAMitchell how do you feel about the switch to a max_depth of 11 which the blog post suggests? Wouldn't that risk very large memory consumption? Would overfitting be an issue or is that taken care of early stopping/loss guide?

I looked at the graph again and thought a bit about the results. In the section with low R-squared the default of xgboost performs much worse. These are datasets that are hard to fit and few things can be learned. The higher eta (eta=0.1) leads to too much overfitting compared to my defaults (eta=0.05). If you would set nrounds lower than 500 the effect would be smaller. You could leave eta=0.1 but set nrounds to a lower default value. Or leave it like it is, withouth specifying it.
I am not sure how big is the effect for max_depth. You could leave it on 5 to get better runtimes.
The subsample and colsample parameters could be safely set to values between 0.6 and 0.8, I guess there is not a big danger here.

I'm not too worried about memory consumption from tree depth. As mentioned by @PhilippPro over fitting is also offset by lower learning rates and sampling.
@PhilippPro the "grow_policy":"lossguide"/"depthwise" parameters already exists for both "tree_method":"hist" and"tree_method":"gpu_hist". We may even make "tree_method":"hist" the default learning mode at some point.


How to get the code coverage of ChakraCore by executing js testcases?

Hi, I'm working on fuzz testing, and now i have a batch of js testcases, i want to get the code coverage of ChakraCore by executing testcases. (not coverage of testcases, i don't care it.). Because I want to use the increase of engine coverage as one of my performance metrics.
I've looked at a lot of info, and it looks like it can be done with some C/C++ tools, but I'm not familiar with C/C++, So could you tell if there is such a feature built in ChakraCore? If not,  can you give me some advice about how to meet my needs?
Thanks guys, I'll be waiting for your reply. :laughing:

The engine can only give you data on JavaScript it runs, not on its own sources, for that you would need an external tool (with compiler support).
Assuming you have access to Linux environment, gcov is a classic example of a code coverage tool for native code. However, since ChakraCore uses Clang instead, you probably want to take a look at the Source Code Coverage in Clang docs. That is probably the path of least resistance. You can conditionally add instrumentation flags that Clang supports via CMake.
On Windows, this needs to be a Visual Studio plugin, and looks like there are options. I guess you can also use llvm-cov, but you would first have to change the build to use LLVM instead of MSVC toolchain.

@ppenzin
Thanks for your advice. I will try to do it with clang first, thank you again.

@ty5491003 we will be happy to support coverage in the official builds - if you are going to be able to turn it on, we can help you with build plumbing (CMake options and so forth). Also if you have any questions about builds - feel free to ask.


Adds JUMO to FinTech category

Seems good, but it's sad that you still can't filter for Porto positions :(

Thanks @matosf!


How to scrape a content in a search page

I need insert a search keyword. Then push a search button and scrape content
for a only one element - the count of results.
In this website
https://www.rbauction.com/other?id=ci&s=Y2l8Tj0wJk50az1FcXVpcG1lbnQuZW4mTnR0PSZOdHg9
I do it like this
change search string:
$(".keyword-input-wrapper #rba-keyword-input").val("A40D");
submit a form:
$("form#rba-keyword-search")[0].submit();
and get data like this from the result.
$("div.summary-result-container h2.float-left span#summary-result-count").text()

Your problem is that submitting the form redirects you to a different page, hence you're losing the scope and need to reload artoo in the new page. You could use artoo's ajaxSpiders to collect the results from within you initial page, but it won't work as the resulting pages also load results dynamically individually.
You'll need some serverside scraper, you cane xperiment with https://github./com/medialab/sandcrawler that runs artoo on a phantom engine, but note it's an alpha not completely documented project yet :/

thanks for reply.
2016-08-24 18:19 GMT+03:00 Benjamin Ooghe-Tabanou<EMAIL_ADDRESS>:

Your problem is that submitting the form redirects you to a different
page, hence you're losing the scope and need to reload artoo in the new
page. You could use artoo's ajaxSpiders to collect the results from within
you initial page, but it won't work as the resulting pages also load
results dynamically individually.
You'll need some serverside scraper, you cane xperiment with
https://github./com/medialab/sandcrawler http://sandcrawler that runs
artoo on a phantom engine, but note it's an alpha not completely documented
project yet :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/medialab/artoo/issues/250#issuecomment-242102396, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHbMow8iAV88WvleXkM-U69tuVchcw1oks5qjGEDgaJpZM4Jr_xi
.