idea: Support more parameters for /v1/models/start

Problem Statement
Currently, /v1/models/start only accept prompt_template parameter
Feature Idea
Should accept more parameters like context_length
Scenario:

When the user sends start request, cortex will override parameters loaded from model.yml by request's parameters.
The content of model.yml will be kept as-is.

Need to add context-length vs ngl into start parameters

To support vision model, we also need to add mmproj parameter, type: string

API: accept

Eg test n_parallel


Update Composer dependencies (2022-02-09-00-07)

Visual regression test passed!
View the visual regression test report


feat: Change ERC20Token address to be alloy's Address instead of H160

Also change the VMPoolState attributes to be Address instead of H160
TODO: (in the upcoming episodes):

Replace ethers U256 with alloy U256 in the vm pool state
Abstract away ERC20Token logic (this one is evm specific)
Use alloy to encode and decode instead of ethers (in simulation contract)

@tamaralipows needs this in her current task so we decided to merge right away. Feel free to review it and I will address the comments in the next PR (which will be in the same scope anyway)


Support POST /_changes in Listener

From my understanding, the replicators communicate with sync_gateway via POST and I think iOS also supports this on the Listener. On Android, it would be nice to allow unified communication with the _changes endpoint via POST.

https://github.com/couchbase/couchbase-lite-ios/pull/1110/files

See #1188


Help wanted / question : "exclude ALL but some" resources

Hello!
First of all, thank you @ebaauw for your work! Second, I hope this is an acceptable way to ask for some help.
My challenge:
I would like to hide all resources linked to my Philips Hue bridge, EXCEPT one (or several) that I explicitly select/configure.
Why?
I have several Hue bulbs that I use through the Hue Bridge. I have one Osram Smart Plug that is also connected to the Hue bridge, but that doesn't show up on Homekit --> hence homebridge!
My homebridge with homebridge-hue plugin causes to expose my Hue Bridge bulb resources twice to Homekit, once per connected bridge. I thus want to hide all ressources in homebridge EXCEPT this Osram switch, and this without having to individually block each and every bulb.
In a nutshell, I am looking how to implement a "deny all, allow some" rule.
Thanks a lot!

What's your config.json?  By default, homebridge-hue only exposes non-Hue lights connected to a v2 Hue bridge.
For the "allow some", use a whitelist resourcelink, see https://github.com/ebaauw/homebridge-hue#configuration.


What does "point cloud distillation loss" mean for CroCo V2 initialization?

Nice work! I have a question about model training. In appendix it is mentioned that "for CroCoV2 initialization, we warm up the training by adding point cloud distillation loss from the DUSt3R model for 1,000 steps". However, I checked the DUSt3R paper but did not see a "point cloud distillation loss" term. So I wonder what does this loss exactly mean?

Hi, thanks for your interest in our work. The distillation loss is used by us instead of DUSt3R. You can find it here, it is similar with the point loss used by DUSt3R.

Is there any change of other hyperparameters to run this point cloud distillation loss apart from setting self.train_cfg.distill_max_steps=1000?


Timeout Values Working?

I have set the timeout values to an extremely low number (like .0001) to try to force a timeout. But the request does not timeout even though the response time was as high as two seconds. Are the timeout values being applied?

The value for connectTimeout should be a whole number value in milliseconds. On Android, if you pass a float it will be rounded up or down. If you pass a 0 value it will use the default values for the request handler.
The values are being applied here on Android and here in iOS.
Closing this unless you have a small reproduction of your issue since I'm not able to reproduce it.


[ECP-9532] Call to paymentMethods endpoint during customer/section/load

Is there a possibility of separating that call from customer section load request? As it is implemented at the moment, it might prolong the customer section load request in case the request to paymentMethods times out, or in case adyen server is under load which would be detectable only after customers would report delay in minicart update.
Not that we had any issues with it so far, it's just an idea of a possible improvement - of course if you see it as a possibility.

Or at least adding a configurable timeout when executing that request?

Hello @roby0404,
Thank you for raising this issue. It looks like a valid concern. We've created an internal investigation ticket for a potential improvement. You will be informed when the investigation is done.
Best Regards,
Can


Update README.markdown

adaptiveHeight option exists on the preview page but not in this RADME.markodown file.

Ping :)

great, thanks for that...
could you just make one more update; asNavFor also accepts a jQuery object.

Done, sorry for the late response :)

@gabskoro ... there's now a conflict , can't merge.
If you have time, please rebase :)


Increases timeout for alert dialog presence testing

LGTM, but I think 1 min feels a bit long. Would 30 seconds be fine? Just thinking about what counts as "reasonable" loading time.
FInal decision is up to you, this is just a data point. Feel free to merge once you've decided.


Added authsource option to allow for specifying authentication database

Hi @dcrosta, could you take a look to this pull request please ?

ping.

Added this functionality in #88. Sorry this has taken so long for me to get to all of these.


Improve global bone pose calculation in Skeleton3D

This change introduces a way to calculate global bone poses without recalculating all bones every time a pose changes. It defines the bone hierarchy as a nested set. This makes it possible to represent bone subtrees continuously within an array. With regard to the introduction of SkeletonModifier3D, this should improve performance when using many modifiers that depend on global poses.
Two fields are added to Skeleton3D::Bone:

nested_set_offset: defines the position within the nested set[^1]
nested_set_span: defines the size of the subtree (the bone itself and all descendants)

[^1]:  For imported scenes, nested_set_offset seems to be the same as the bone index. Presumably bones are already ordered this way. However, this may be different for manually created skeletons.
This shows the nested set values for a simplified skeleton:

How it works
Skeleton3D::bone_global_pose_dirty is an array containing a flag for each bone indicating whether the global pose needs to be recalculated. It can be indexed with Bone::nested_set_offset, followed by the descendant bones.
When changing the pose of a bone, the dirty flags of its subtree need to be set to indicate that the global poses of the bone and its descendants need to be recalculated. This is done by setting all flags in the range from Bone::nested_set_offset with length Bone::nested_set_span to true. This has some overhead[^2]. But it can be skipped if the bone is already dirty at index Bone::nested_set_offset. Because if a global pose of a bone is dirty, all of its descendants are dirty too.
[^2]: It could be implemented as some kind of bit field structure instead of an array, if it turns out that performance is a problem at that point.
Getting the global pose with get_bone_global_pose
When requesting the global pose of a bone, only the global poses of its parents that are dirty are recalculated and the dirty flag is cleared. In the best case (if the parent global poses have already been calculated) only a few global poses need to be recalculated.
Recalculating global poses with force_update_bone_children_transforms
By looping through Skeleton3D::nested_set_offset_to_bone_index it is possible to update the bone hierarchy without the need for recursion. By checking the dirty flag at the current index, only required bones are updated. The order also ensures, that parent bone poses are calculated before child bone poses.
Benchmarks
skeleton_3d.gd.zip

Test
Before
After

get_global_poses
12.67ms
12.61ms

set_all_poses
49.83ms
51.98ms

set_all_poses_reverse
87.56ms
89.03ms

set_and_get_global_poses_forward
747.6ms
27.8ms

set_and_get_global_poses_reverse
788.4ms
57.37ms

set_and_get_some_global_poses
29.64ms
15.22ms

Note: I ran the tests individually. Some values were way too high even on master when run in series.
TODO

[x] Improve performance of get_bone_global_pose
[x] Improve performance of force_update_bone_children_transforms
[x] Provide some performance tests
[x] Thread safety?

Tested on Animation_test.zip.

Master: 70 fps
This PR: 75 fps

As for thread safe (as I understand it you are talking about using the thread_local keyword) it is multi-thread safe. Although I do not think that this can cause bugs, since the multi-threaded scene tree is not implemented.

As for thread safe (as I understand it you are talking about using the thread_local keyword) it is multi-thread safe. Although I do not think that this can cause bugs, since the multi-threaded scene tree is not implemented.

I was wondering if there is a problem now that the logic is more complex. But if the scene tree is not multi-threaded then that's ok I guess.

As far as I know multi-threaded scene tree is implemented. I'm unable to get you the details, but it was done in an earlier release disabled by default.

As far as I know multi-threaded scene tree is implemented.

Right, the implementation is here tho, it would be cool in look in this for  skeleton, anim player/tree multithreading improvements.  https://github.com/godotengine/godot/pull/75901

Is this ready for review? I noticed it is still in draft stage but all the items are checked.

Yes, I forgot.

Fixed conflicts.

Thanks!

@detomon Do you think this can be done to the general 3d node caching system too?

@fire I'm not too familiar with how the Node3D hierarchy works. I will look into that.

https://github.com/godotengine/godot/blob/master/scene/3d/node_3d.cpp#L38


Refactor StaticServer constructor

Hi
This is a suggestion,
Too many conditions check in constructor, my suggestion below
let server = new StaticServer({port: 8080, localOnly : true, keepAlive : true });
class StaticServer {
          constructor(opts) {
                       if(opts)
                        {
				this.port = `${opts.port}` || PORT;
				this.root = opts.root || ROOT;
				this.localOnly = opts.localOnly || false;
				this.keepAlive = opts.keepAlive || false;
			 }
			else{
				this.port = PORT;
				this.root = ROOT;
				this.localOnly = false;
				this.keepAlive = false;
                   }
             }

....
}

let me know your feedback

Agreed this could be simpler, but don't think it is worth breaking changes.


chore(deps): clean up reth-stages deps

closes #1679

Codecov Report

Merging #1682 (34068cc) into main (279ad34) will decrease coverage by 57.58%.
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             @@
##             main    #1682       +/-   ##
===========================================
- Coverage   75.02%   17.44%   -57.58%
===========================================
  Files         378      378
  Lines       45463    45463
===========================================
- Hits        34107     7930    -26177
- Misses      11356    37533    +26177

Flag
Coverage Δ

integration-tests
17.44% <ø> (-4.11%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


export valuelink/tags.jsx as valuelink/tags

exporting a jsx file would require a compilation step for node_modules in other apps, other projects/react packages never export jsx file directly, they export the compiled version of jsx -> js, just like the export of valuelink.js

Good, I will make the change.

Fixed in 1.3.6


Airgapped installation of CP4I

Solves #311

Needs a review after merging conflicts. Will do the review in another branch.


[IMPROVE] Easy way to copy value

It would be great it there is an easy way to copy/paste the values of the subclasses, for example with the context menu.

Hi @jlaprof !
Copy and paste functionality has been added in the latest release.
https://github.com/mackysoft/Unity-SerializeReferenceExtensions/releases/tag/1.2.0


Add optional support for TLS

This should use wss:// instead of ws://. With alternative backend:
http.ListenAndServeTLS(":8443", certFile, keyFile, nil)
Next step is password

#4

I'll do this, but I want to set up an automated build first. Thanks!

For the intended use case, this feature is not so very important.


Connect to Teradata LDAP

Hi,
I'm trying to figure out how to connect to a Teradata database through LDAP using DBeaver. Anyone have any tips or advice on how to do this?
I installed a JDBC driver and got it to connect to LDAP, but I cannot point it towards Teradata (all I get is as displayed below).

I'm quite new at this so might be doing something fundamentally wrong.
Thanks!

Hello Pealp,
Ensure to have the correct version of the Teradata JDBC driver installed to the version of Teradata you are trying to access. You do not need to connect to LDAP first before connecting to Teradata rather you need to use the correct logging mechanism (i.e. LOGMECH) in the JDBC url setup,
Few things within the JDBC setup I suggest to fixed your problem:

Edit the URL template via the "Edit Driver Settings" to add an extra field for the LOGMECH settings. Try using the following template: jdbc:teradata://{host}[:{port}]/{database}[,{server}]

Once you modified the URL template, trying the following JDBC values (you may need the port number 1025 or not):

Hopefully with this you will successfully accessed Teradata via LDAP.
Thanks,

There isn't any activity about this ticket very long time. It will be closed. However, if you think that it is actual just let us know.

Hi,
I am facing the same issue. Wanting to use LDAP authentication to login to Teradata but looks like the above solution doesn't work I keep getting an error that the username or account or password is invalid. I can login using same credentials through Teradata studio but not Dbeaver.
Any solutions??

@isakalley Please post exact error message  (error stacktrace would be also helpful)

[Teradata JDBC Driver] [TeraJDBC <IP_ADDRESS>] [Error 1536] [SQLState HY000] Invalid connection parameter name <databse_name>,TMODE

@Veera93
I think the connection string from @SirBenJammin is missing database= part.
jdbc:teradata://{host}[:{port}]/database={database}[,{server}]

Even with this connection setting, [test connection] in the setting dialog gave me an error about username, password, account not being valid. But Interestingly it worked just fine from the main screen Database Navigator once the setting is registered.
In my environment, I also had to omit port number in the setting (but I kept it in the URL template).

Goto toolbar Database -> search Teradata and double click -> Check below parameters:

Now, toolbar search newly added Teradata, check below details:

Sorry for commenting on a closed thread, but this is the most relevant issue!
I am using the community edition: Version <IP_ADDRESS>205011839 and connection to teradata fails with error:
[Teradata JDBC Driver] [TeraJDBC <IP_ADDRESS>] [Error 1000] [SQLState 08S01] Login failure for Connection to TERA.DEV.COM Tue May 10 09:03:17 EDT 2022 socket orig=TERA.DEV.COM cid=2646bbd0 sess=0 java.net.ConnectException: Connection refused (Connection refused)  at java.base/java.net.PlainSocketImpl.socketConnect(Native Method)  at java.base/java.net.AbstractPlainSocketImpl.doConnect(Unknown Source)  at java.base/java.net.AbstractPlainSocketImpl.connectToAddress(Unknown Source)  at java.base/java.net.AbstractPlainSocketImpl.connect(Unknown Source)  at java.base/java.net.SocksSocketImpl.connect(Unknown Source)  at java.base/java.net.Socket.connect(Unknown Source)  at java.base/sun.security.ssl.SSLSocketImpl.connect(Unknown Source)  at com.teradata.jdbc.jdbc_4.io.TDNetworkIOIF$ConnectThread.run(TDNetworkIOIF.java:1563)
  Connection refused (Connection refused)
  Connection refused (Connection refused)

After fiddling around with the tera driver params, I realized it tries to use ssl to connect even though custom host and port are specified. To force non-ssl, ended up adding SSLMODE=DISABLE and it worked out file. Here are the final JDBC url: jdbc:teradata://TERA.DEV.COM/DATABASE=DEVDB,TMODE=TERA,LOGMECH=LDAP,SSLMODE=DISABLE,DBS_PORT=1025
Thanks a lot @tphani, the above JDBC url worked for me I have been struggling for the access for a while.


Latest version of PyGradle breaks Wheels dependencies

With PyGradle 0.6.21, I used to be able to specify a wheel dependency in my build.gradle file like this:
dependencies {
    python 'mygroup:my_python_module:0.0.8'
}

Now, when I try to upgrade to PyGradle 0.7.7, I get this error:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':installPythonRequirements'.
> Cannot calculate Python package extension from C:\Users\adagenais\.gradle\caches\modules-2\files-2.1\mygroup\my_python_module\0.0.8\c18e1f71e7ee593edfbbeea335e15328a7f620d\my_python_module-0.0.8-py2-none-any.whl using regular expression /\.tar\.gz|\.zip|\.tar|\.tar\.bz2|\.tgz/.

It seems like fixing the regex is enough to fix the issue. I'll submit a pull request with a fix soon.


Get tests passing for rc2

Get tests passing for v2.5.0rc2. Resolves #473. This pull request is based on #472 and should be merged after that one.
Note that this pull request is just to get the tests passing. We still need to:

Update documentation on updating expected images for test_diags.py
Refactor/restructure testing directories

@tomvothecoder I made this commit with --no-verify because of the error I sent you in Slack. That's why the CI/CD build is failing.

PR #472 has been merged. Once the pre-commit issues are fixed, the build should pass.

@tomvothecoder Thanks, I created #475 to do the other tasks (to be included for rc3).


chore: Fix warnings related to params of AshPhoenix.Form.submit

This PR removes warnings related to params of AshPhoenix.Form.submit:
As reminder, the warning was the following:
warning: The `params` option should be supplied at all times. This will be required in a future major release.
To silence this warning without providing `params`, you can pass `params: nil`.

For example:

    def handle_event("submit", %{"form" => params}, socket) do
      case AshPhoenix.Form.submit(socket.assigns.form, params: params) do
        ...
      end
    end

The form unassigned parameters are given to AshPhoenix.Form.submit when possible, otherwise params: nil is given.
Contributor checklist

[ ] Bug fixes include regression tests
[ ] Features include unit/acceptance tests

🚀 Thank you for your contribution! 🚀


Error: permission denied for table users

Looks like this is related to #470. I'm going to close this issue, but mention it in 470.


Compatibility with Windows (10)

I just tested this out on Windows 10 and it seems to be compatible apart from the command to show the window.

Thanks for reporting. Unfortunately, I have no Windows machine to maintain this simple tools. I recommend that every user on Windows or Linux should build own version with source.


Fix minor typo in notebook_launcher error message

Fixing a minor typo, num_process=...  -->  num_processes=... in the error message when num_processes is None for notebook_launcher.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Bug Report] ELTree checkbox margin-right not working

Element UI version
2.11.1
OS/Browsers version
macOS Mojave 10.14.5/Chrome 76
Vue version
2.6.10
Reproduction Link
https://jsfiddle.net/amkhsgyj/1/
Steps to reproduce
@import url("//unpkg.com/element-ui@2.8.2/lib/theme-chalk/index.css");
Change theme version
What is Expected?
Checkbox margin retrieval
What is actually happening?
No margin

I think it's fixed in 2.12.0.


enhancement Record detailed fail reason for schema change tasks

Proposed changes
Expose the error msg from BE as the real fail reason recorded for schema change tasks. To avoid too much memory usage, we just pick one among all to record.

run buildall


Need to test against all version numbers.

Test versions between 2004 and 2016, these have slightly different HTML than the versions tested against (2003, 2017). Should be easy to fix and would allow the tool to work with 90%+ of Odyssey sites which have no other major blocker.
notes on blockers for scraping here:
https://github.com/open-austin/Odyssey-Court-Records-to-JSON/blob/master/resources/texas_county_data.csv

did, basic manual testing. Only tested up to getting search results. Didn't test getting case data. I opened another issue to gather information needed for more in depth testing. Will write a testing utility when that work is done.


Invalid decoded text

Platform:  Mac
Mercury Parser Version: 2.1.0
Node Version (if a Node bug): 10
Browser Version (if a browser bug):

Expected Behavior
Parsed HTML should be properly encoded as per the original text
Current Behavior
The parsed html contains in invalid text .Might be because of decoding issue.
Steps to Reproduce

Fetch the html using any client
Pass that to the parse using Mercury.parse(url,{html:fetchedHtml})
Returned HTML contains incorrectly decoded text

Some Links:
https://www.newyorker.com/culture/the-new-yorker-interview/daenerys-tells-all-game-of-thrones-finale-emilia-clarke-beyonce
Detailed Description
I want to parse by fetching the html and giving to the parse instead of parser fetching the html.
Possible Solution
After looking at the code, it seem you are handling the case for browser only  i.e. only if the html is provided from the browser, the proper encoding is checked from the html file. Ideally it should be able to decode the text irrespective of whether the parser is running on a browser or not

Fixed it by passing the html as Buffer with utf-8 instead of string as mentioned in the README

Hi @farmaan-appachhi
Could you please provide an example?
I am facing pretty much the same issue.

For me problem was when I was trying to pass the local html as string. Using Buffer fixed the issue
Mercury.parse(url, {
        html: Buffer.from(html, "utf-8"),
        headers: {
            "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) " +
                "Chrome/60.0.3112.113 Safari/537.36"
        },
    })

@farmaan-appachhi
That saved my day (at least half), thank you
Could you provide the link where you found that?

I tried debugging the code. Took my 5-6 hour figure out the issue. If you
see the source code, they were using Buffer when fethcing the html but
local files was used just as string. That's how I figured it out
On Tue, Jul 30, 2019, 6:32 PM grigoriy-didorenko<EMAIL_ADDRESS>wrote:

@farmaan-appachhi https://github.com/farmaan-appachhi
That saved my day (at least half), thank you
Could you provide the link where you found that?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/postlight/mercury-parser/issues/425?email_source=notifications&email_token=AESIMEGJYBITDT7VUBYRV53QCA3XPA5CNFSM4HPSFQ5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3D4NWQ#issuecomment-516409050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AESIMEHW7YZFAESJT7KZXCLQCA3XPANCNFSM4HPSFQ5A
.

I can verify that using Buffer works. This should have been mentioned in the README.

+1 for Buffer works. It seems that the string should not be passed in for any case. or Mercury should detect the type and handle it specially.


Begin work on recipes

API | Common

Implementation might not be done for a while yet: https://github.com/SpongePowered/SpongeAPI/pull/1098#issuecomment-188300093

Anything I could help with?

@kashike You should close this one since the one for API is superseded.

Thought this was already closed - thanks @liach.


Error codes missing from ReadTheDocs as of 4.0.0

On ReadTheDocs, the current error codes page has a blank Grouping heading.
For the 3.0.0 release docs, this is where the table of error codes would be printed.

I found this issue while trying to find documentation on new code D417: Missing arguments in the function docstring, added in 6de6d938377e2db3bfe6f706fa2e3b90c412db76 as part of #357 (initial GoogleDoc convention).
I note that http://www.pydocstyle.org/en/4.0.0/error_codes.html#default-conventions is also missing documentation of the new google convention.

There must have been a problem with the included file docs/snippets/error_code_table.rst (which is not under version control), it is meant to have been generated via docs/conf.py

Ew, that sounds like a hack. Importing conf.py isn't supposed to have side-effects. Maybe ReadTheDocs is now caching the contents of conf.py?

Importing conf.py is the accepted way to run sphinx-apidoc on RTD, and probably the only way to have any dynamic content on RTD - so I can see why that would be done.
Looking at the RTD build logs might be instructive...

I checked the log of a recent build and it doesn't look like they use caching - indeed they append extra build config to each conf.py.

Oh, here's the issue:
WARNING: snippets/error_code_table.rst:1: (ERROR/3) Malformed table.

Looks like D415 broke the table - if so there are less fragile RST table layouts without the vertical bars which would work better here.

Self-contained snippet for testing:
$ python -c "from pydocstyle.violations import ErrorRegistry; print(ErrorRegistry.to_rst())" > error_code_table.rst

$ pip install restructuredtext-lint
$ restructuredtext-lint error_code_table.rst
ERROR error_code_table.rst:1 Malformed table.

@Nurdok Could you tag a 4.0.0.post1 release in master? Simply tagging will update ReadTheDocs stable (but no need to upload a new package to PyPI).


Lack of a mutable request-context object makes extension building difficult

I'm currently working on extension that adds server-backed sessions to Sanic. I'm running into the issue that because the request object is immutable, adding convenient extensions is tricky.
For example, let's say I have a Redis connection pool and I'd like to just automatically hook it up to the request object before each request, so I don't have to instantiate it in every view.
I'd like to do something like this:
import asyncio_redis

from sanic import Sanic
from sanic.response import text
from sanic_session import RedisSessionInterface

app = Sanic()

class Redis:
    """
    A simple wrapper class that allows you to share a connection
    pool across your application.
    """
    _pool = None

    async def get_redis_pool(self):
        if not self._pool:
            self._pool = await asyncio_redis.Pool.create(
                host='localhost', port=6379, poolsize=10
            )

        return self._pool

redis = Redis()

@app.middleware('request')
async def add_redis_to_request(request):
    request.redis_connection = await redis.get_redis_pool()

@app.route("/")
async def test(request):
    foo = await request.redis_connection.get('foo')
    return foo

if __name__ == "__main__":
    app.run(host="<IP_ADDRESS>", port=8000, debug=True)

Of course, I cannot, because the request object is immutable and won't let me add the redis_connection onto it. The Flask request object is immutable as well, but Flask provides the convenient g object for situations like this.
Are there any thoughts on adding a mutable request-context object? It'd be really helpful, especially since I'm interested in developing on this.

From a code perspective, this would only require initializing an empty g class in sanic.handle_request, and passing g into the middleware and handler methods. I'm happy to put out a pull request but it would be a minor API change, would like any thoughts.

You should be able to use the request as a dictionary object to store global things as done in #163


Update README.md

Updated path of badges to the newly named GitHub Actions files
Pre-PR checklist

[ x ] PR is not a duplicate
[ x ] Changes have been made in a separate branch. Branches MUST have descriptive names that start with either the fix/ or feature/ prefixes. Good examples are: fix/signin-issue or feature/issue-templates.
[ x ] PR is associated to an issue

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # https://github.com/modalityone/modality-issues/issues/15
Type of change
Please delete options that are not relevant.

[ x ] This change requires a documentation update

Final checklist

[ x ] I have performed a self-review of my code
[ x ] My changes generate no new warnings

Badges now fixed with the correct paths to the newly-named GitHub Actions.


Gardener AWS: Incorrect "zone" property is used

Description
The gardener AWS terraform template is using a variable aws_zone to set the zone which is never set, since hydroform sets the zone variable for all providers in gardener.
We need to use the zone variable aslo for AWS like the others.
Expected result
When setting the zone flag the Gardener AWS cluster uses that value in provisioning
Actual result
Validation is passed sucessfully but later terraform prompts to set the aws_zone variable and provisioning can't be done.

While working on this issue, gardener updated its validation, making a bug visible that we had until now in Azure.
This PR fixes it in the gardener terraform provider: https://github.com/kyma-incubator/terraform-provider-gardener/pull/73 and it will be integrated together with the AWS zone improvements into hydroform and the CLI


Fine-tuning mBART for unseen language

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
The code in the README for mBART provides code snippet for English Romanian task. Do we have to any modifications to the code to fine-tune the model for supporting unseen language either or source side or target side?
Code

What have you tried?
What's your environment?

fairseq Version (e.g., 1.0 or master):
PyTorch Version (e.g., 1.0)
OS (e.g., Linux):
How you installed fairseq (pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

@sshleifer I've seen you posted a lot of content on BART and mBART, do you know about this?
Sorry for troubling you.

@sshleifer I've seen you posted a lot of content on BART and mBART, do you know about this?
Sorry for troubling you.

From grep, I think the answer is no, the code in ./examples/mbart/README.md is all that the authors released.
Sorry! Interesting question :)

From grep, I think the answer is no, the code in ./examples/mbart/README.md is all that the authors released.
Sorry! Interesting question :)

In this paper, the authors talked about how to create mBART50 from mBART25.

We take the public available pretrained mBART model which was pretrained on 25 languages and extend its embedding layers with randomly initialized vectors for an extra set of 25 language tokens.

So i dont think you need to modify the fairseq code to fine-tune the model. But you may need some code to extend the embedding layer dimension of the checkpoint before you call fairseq-train

In this paper, the authors talked about how to create mBART50 from mBART25.

We take the public available pretrained mBART model which was pretrained on 25 languages and extend its embedding layers with randomly initialized vectors for an extra set of 25 language tokens.

So i dont think you need to modify the fairseq code to fine-tune the model. But you may need some code to extend the embedding layer dimension of the checkpoint before you call fairseq-train

Did you ever find a solution for this? I tried following the exact setup but getting an error that the architectures don't match, I'm assuming because I need to some extra code like thpun explained above - not sure if you already have a working implementation somewhere


duplicating where clauses that contains (unused) braces

This affects very specialized and a few months outdated codebases that

had to use #[allow(unused_braces)] (before it was recently lifted) to invoke macros from where clauses, and
there is an asterisk-based or two slash-based comment in the struct or enum definition.

Hence likely to be a lower priority. But I'm reporting this in case it's connected to any other/potential bigger issues.
Run cargo fmt from recent nightly (noticed with nightly as of Feb 2, 2023, chances are the problem existed before; the bug still exists as of Feb 15, 2023) on a project with lib.rs containing the following (which does compile; or get the code from https://github.com/peter-kehl/fmt_bug_where_braces):
#![allow(incomplete_features)]
#![feature(generic_const_exprs)]

macro_rules! num_slots {
    ($c:expr) => {
        1
    };
}

#[allow(dead_code)]
#[allow(unused_braces)]
struct S<const C: usize>
where
    [(); { num_slots!(C) }]:, {
    /* An asterisk-based, or a double-slash-prefixed, comment here is
       required to trigger the fmt bug.

    A single-line triple-slash-prefixed comment (with a field following it) is not enough - it will not trigger the fmt bug.

    Side note: If you have a combination of two, or all three of the
    above mentioned types of comments here, some of them disappear
    after `cargo fmt`.

    The bug gets triggered even if a field definition following the
    (asterisk-based, or a double-slash-prefixed) comment, too.
    */
}

That injects an extra line num_slots!(C) }]:, {, which makes the project not compile anymore:
cargo fmt
git diff
diff --git a/src/lib.rs b/src/lib.rs
index ce30fe0..44f77eb 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -12,6 +12,7 @@ macro_rules! num_slots {
 struct S<const C: usize>
 where
     [(); { num_slots!(C) }]:, {
+    num_slots!(C) }]:, {
     /* An asterisk-based, or a double-slash-prefixed, comment here is
        required to trigger the fmt bug.

(Initially reported as https://github.com/rust-lang/rust/issues/107610. The source that triggered the bug is not a part of rust-lang/rust itself, but it's from my experimentation.)

Thank you for the fix.
When is this likely to be in nightly, or do I need any special steps?
I'm on Manjaro/Arch Linux x64, using rustup as it comes from Manjaro/Arch packages. rustup update gave me cargo 1.69.0-nightly (17b3d0de0 2023-02-17). When I run cargo fmt, I still have this problem (duplicated and misplaced part of where bound).

And, while we're on it, and if you happen to know:
I have this problem when building my fork of rust-lang/rust with its x.py (I actually use x). In my config.toml I have channel = "nightly". Would that be using the version of nightly that I have, or does it pull nightly for its own purposes and independent of nightly that I use for ordinary development?

Thank you. All fine. Take care.


Rust SDK: Handle send errors on disconnect

When the stream is shutting down, route receives a DisconnectedError and
then tries to forward this error on to all of its channles. However,
these channels tend to already have been cleaned up, causing a send
error. This commit handles this send error by logging it, rather than
panicing. This panic was seen during shutdown.
Signed-off-by: Adam Ludvik<EMAIL_ADDRESS>

This is an example of the thread where the router was running doing a panic on shutdown:
   Compiling sawtooth_sdk v0.1.0 (file:///Users/ludvik/git/sawtooth-core/sdk/rust)
    Finished dev [unoptimized + debuginfo] target(s) in 7.64 secs
     Running target/debug/deps/sawtooth_sdk-9f084f67474c56e7

running 1 test
start()
block()
message()
peer()
stop()
ALL CHECKS PASSED
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: "SendError(..)"', libcore/result.rs:945:5
test consensus::zmq_driver::tests::test_zmq_driver ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 12 filtered out

@grkvlt I believe this fixes the seth-rpc shutdown error you were seeing


Touch Bar support

Depends on https://github.com/electron/electron/pull/8095.
Would be cool to display the elapsed recording time and a stop button in the Touch Bar.
If you don't have a native Touch Bar, you can use: https://red-sweater.com/touche/

https://github.com/electron/electron/issues/7781#issuecomment-278121145

it is here finally https://electron.atom.io/blog/2017/03/08/touch-bar-support but the version of electron still in beta, I would love to work on this once you guys decide to move to Electron v1.6.3+ and of course when the version is out of the beta

@Khaledgarbaya feel free to bump the dependency to 1.6.3 🙌

@matheuss @skllcrn any preferences on what you want to show in the TouchBar.
I was thinking about start/stop recording mainly

Note the limitation that we can only be in the Touch Bar when the Kap window is focused, so not during recording.


[COMMONS-1] Created commons project with credential and shell executi…

…on utilities

@TechnologyBrewery/technology-brewery-maintainers
WIP.  Interested in opinions on the general structure/approach.  Obviously needs tests.
Example (also WIP) usages:

https://github.com/TechnologyBrewery/habushu/pull/51
https://github.com/TechnologyBrewery/orphedomos/pull/23


[Snyk] Upgrade: @babel/cli, @babel/core, @babel/node, @babel/plugin-transform-runtime

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade multiple dependencies.
👯 The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

@babel/clifrom 7.22.9 to 7.22.10
1 version ahead of your current version
22 days agoon 2023-08-07

@babel/corefrom 7.22.9 to 7.22.10
1 version ahead of your current version
22 days agoon 2023-08-07

@babel/nodefrom 7.22.6 to 7.22.10
1 version ahead of your current version
22 days agoon 2023-08-07

@babel/plugin-transform-runtimefrom 7.22.9 to 7.22.10
1 version ahead of your current version
22 days agoon 2023-08-07

Release notes

Package name: @babel/cli

7.22.10 - 2023-08-07v7.22.10 (2023-08-07)
Thanks @ DimitriPapadopoulos, @ FabianWarnecke, @ JoshuaKGoldberg, and @ YYvanYang for your first pull requests!
🐛 Bug Fix

babel-plugin-transform-typescript

#15799 [ts] Strip type-only namespaces (@ nicolo-ribaudo)
#15798 [ts] Fix compiling extended exported nested namespace (@ nicolo-ribaudo)

babel-helper-create-class-features-plugin, babel-plugin-proposal-decorators, babel-plugin-proposal-destructuring-private, babel-plugin-transform-class-properties, babel-plugin-transform-class-static-block, babel-plugin-transform-new-target, babel-plugin-transform-private-methods, babel-preset-env

#15701 Memoize class binding when compiling private methods and static elements (@ JLHwung)

💅 Polish

babel-cli

#15824 Add meta object to @ babel/eslint-plugin (@ JLHwung)

babel-traverse, babel-types

#15661 Improve the type definition of path.isX (@ liuxingbaoyu)

babel-generator, babel-types

#15776 improve SourceLocation typing (@ JLHwung)

🏠 Internal

Other

#15818 build: generate flow typings in prepublish job (@ JLHwung)
#15777 chore: bump dev dependencies and remove .eslintignore (@ JLHwung)

babel-cli, babel-core, babel-generator, babel-helper-builder-react-jsx, babel-preset-env, babel-standalone

#15794 Enable @ typescript-eslint/no-redundant-type-constituents rule (@ JLHwung)

babel-helper-compilation-targets

#15811 Remove @ babel/core peerDep from helper-compilation-targets (@ nicolo-ribaudo)

babel-parser

#15793 Use const enum in babel-parser (@ JLHwung)

babel-plugin-transform-runtime, babel-traverse, babel-types

#15716 chore: Use typescript-eslint@v6 with reworked configs (@ JoshuaKGoldberg)

🔬 Output optimization

babel-plugin-transform-block-scoping, babel-plugin-transform-parameters, babel-plugin-transform-regenerator

#15746 Reduce transform-block-scoping loops output size (@ liuxingbaoyu)

Committers: 9

Babel Bot (@ babel-bot)
Dimitri Papadopoulos Orfanos (@ DimitriPapadopoulos)
Huáng Jùnliàng (@ JLHwung)
Josh Goldberg ✨ (@ JoshuaKGoldberg)
Nicolò Ribaudo (@ nicolo-ribaudo)
Tianlan Zhou (@ SuperSodaSea)
@ FabianWarnecke
@ YYvanYang
@ liuxingbaoyu

7.22.9 - 2023-07-12v7.22.9 (2023-07-12)
🐛 Bug Fix

babel-plugin-transform-typescript

#15774 fix: Infinity in enums (@ liuxingbaoyu)

babel-helper-module-transforms, babel-helper-remap-async-to-generator, babel-helper-replace-supers, babel-helper-wrap-function

#15771 Simplify helper dependencies (@ JLHwung)

💅 Polish

babel-generator

#15757 recordAndTupleSyntaxType defaults to "hash" (@ coderaiser)

🏠 Internal

#15748 Migrate to eslint.config.js (@ JLHwung)
#15758 Use Prettier 3 stable (@ nicolo-ribaudo)

Committers: 5

Babel Bot (@ babel-bot)
Huáng Jùnliàng (@ JLHwung)
Nicolò Ribaudo (@ nicolo-ribaudo)
@ liuxingbaoyu
coderaiser (@ coderaiser)

    from @babel/cli GitHub release notes

Package name: @babel/core

7.22.10 - 2023-08-07v7.22.10 (2023-08-07)
Thanks @ DimitriPapadopoulos, @ FabianWarnecke, @ JoshuaKGoldberg, and @ YYvanYang for your first pull requests!
🐛 Bug Fix

babel-plugin-transform-typescript

#15799 [ts] Strip type-only namespaces (@ nicolo-ribaudo)
#15798 [ts] Fix compiling extended exported nested namespace (@ nicolo-ribaudo)

babel-helper-create-class-features-plugin, babel-plugin-proposal-decorators, babel-plugin-proposal-destructuring-private, babel-plugin-transform-class-properties, babel-plugin-transform-class-static-block, babel-plugin-transform-new-target, babel-plugin-transform-private-methods, babel-preset-env

#15701 Memoize class binding when compiling private methods and static elements (@ JLHwung)

💅 Polish

babel-cli

#15824 Add meta object to @ babel/eslint-plugin (@ JLHwung)

babel-traverse, babel-types

#15661 Improve the type definition of path.isX (@ liuxingbaoyu)

babel-generator, babel-types

#15776 improve SourceLocation typing (@ JLHwung)

🏠 Internal

Other

#15818 build: generate flow typings in prepublish job (@ JLHwung)
#15777 chore: bump dev dependencies and remove .eslintignore (@ JLHwung)

babel-cli, babel-core, babel-generator, babel-helper-builder-react-jsx, babel-preset-env, babel-standalone

#15794 Enable @ typescript-eslint/no-redundant-type-constituents rule (@ JLHwung)

babel-helper-compilation-targets

#15811 Remove @ babel/core peerDep from helper-compilation-targets (@ nicolo-ribaudo)

babel-parser

#15793 Use const enum in babel-parser (@ JLHwung)

babel-plugin-transform-runtime, babel-traverse, babel-types

#15716 chore: Use typescript-eslint@v6 with reworked configs (@ JoshuaKGoldberg)

🔬 Output optimization

babel-plugin-transform-block-scoping, babel-plugin-transform-parameters, babel-plugin-transform-regenerator

#15746 Reduce transform-block-scoping loops output size (@ liuxingbaoyu)

Committers: 9

Babel Bot (@ babel-bot)
Dimitri Papadopoulos Orfanos (@ DimitriPapadopoulos)
Huáng Jùnliàng (@ JLHwung)
Josh Goldberg ✨ (@ JoshuaKGoldberg)
Nicolò Ribaudo (@ nicolo-ribaudo)
Tianlan Zhou (@ SuperSodaSea)
@ FabianWarnecke
@ YYvanYang
@ liuxingbaoyu

7.22.9 - 2023-07-12v7.22.9 (2023-07-12)
🐛 Bug Fix

babel-plugin-transform-typescript

#15774 fix: Infinity in enums (@ liuxingbaoyu)

babel-helper-module-transforms, babel-helper-remap-async-to-generator, babel-helper-replace-supers, babel-helper-wrap-function

#15771 Simplify helper dependencies (@ JLHwung)

💅 Polish

babel-generator

#15757 recordAndTupleSyntaxType defaults to "hash" (@ coderaiser)

🏠 Internal

#15748 Migrate to eslint.config.js (@ JLHwung)
#15758 Use Prettier 3 stable (@ nicolo-ribaudo)

Committers: 5

Babel Bot (@ babel-bot)
Huáng Jùnliàng (@ JLHwung)
Nicolò Ribaudo (@ nicolo-ribaudo)
@ liuxingbaoyu
coderaiser (@ coderaiser)

    from @babel/core GitHub release notes

Package name: @babel/node

7.22.10 - 2023-08-07 Read more

7.22.6 - 2023-07-04v7.22.6 (2023-07-04)
🐛 Bug Fix

babel-compat-data, babel-helper-compilation-targets, babel-preset-env

#15727 Add opera mobile compat data (@ JLHwung)

babel-plugin-transform-optional-chaining

#15739 Fix transform of delete a?.b in function params (@ nicolo-ribaudo)

babel-helper-split-export-declaration, babel-plugin-transform-modules-commonjs

#15736 fix: Default export for duplicate names (@ liuxingbaoyu)

babel-compat-data, babel-preset-env

#15726 update compat-data sources (@ JLHwung)

babel-helpers, babel-plugin-proposal-explicit-resource-management, babel-runtime-corejs3, babel-runtime

#15705 Fix handling of sync error in @@ asyncDispose (@ nicolo-ribaudo)

babel-standalone

#15707 fix: Support transforming Explicit Resource Management in stage-2 (@ liuxingbaoyu)

babel-core

#15626 fix: Works correctly with --frozen-intrinsics (@ liuxingbaoyu)

🏠 Internal

babel-helper-create-class-features-plugin, babel-plugin-transform-classes

#15700 Minor class transform cleanups (@ JLHwung)

🔬 Output optimization

babel-plugin-bugfix-v8-spread-parameters-in-optional-chaining, babel-plugin-transform-class-properties, babel-plugin-transform-optional-chaining, babel-plugin-transform-typescript

#15740 Compress output for optional chain with multiple ?. (@ nicolo-ribaudo)

babel-plugin-proposal-destructuring-private, babel-plugin-proposal-do-expressions, babel-plugin-proposal-pipeline-operator, babel-plugin-transform-class-properties, babel-plugin-transform-nullish-coalescing-operator, babel-plugin-transform-optional-chaining, babel-plugin-transform-private-property-in-object, babel-traverse

#15741 Inject tmp vars in the params list of IIFEs when possible (@ nicolo-ribaudo)

Committers: 5

Babel Bot (@ babel-bot)
Huáng Jùnliàng (@ JLHwung)
Joe Maffei (@ joemaffei)
Nicolò Ribaudo (@ nicolo-ribaudo)
@ liuxingbaoyu

    from @babel/node GitHub release notes

Package name: @babel/plugin-transform-runtime

7.22.10 - 2023-08-07 Read more

7.22.9 - 2023-07-12 Read more

    from @babel/plugin-transform-runtime GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Issue solved in #111


[SPARK-28243][PYSPARK][ML]remove setFeatureSubsetStrategy and setSubsamplingRate from Python TreeEnsembleParams

What changes were proposed in this pull request?
Remove deprecated setFeatureSubsetStrategy and setSubsamplingRate from Python TreeEnsembleParams
How was this patch tested?
Use existing tests.

Test build #107187 has started for PR 25046 at commit 8eeb6fa.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12355/
Test PASSed.

Test build #107187 has finished for PR 25046 at commit 8eeb6fa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107187/
Test PASSed.

@huaxingao I think you're probably right on this but can you remind us here why you also remove subsampling rate? the feature strategy setter is still on the Scala side; is it also meant to just move rather than go away?

@srowen Sorry I didn't make it clear in the PR description.
On Scala side, initially, both setSubsamplingRate and setFeatureSubsetStrategy were in trait TreeEnsembleParams in treeParams.scala. These two setters were deprecated and moved from trait TreeEnsembleParams  to RandomForestClassifier/Regressor, GBTClassifier/Regressor in 3.0.0.
In this PR, I did the same thing on python side, I moved setSubsamplingRate from TreeEnsembleParams  to RandomForestClassifier/Regressor and GBTClassifier/Regressor.  setFeatureSubsetStrategy is already in python RandomForestClassifier/Regressor and GBTClassifier/Regressor, so I simply removed it from TreeEnsembleParams.

OK this is really a follow up of https://github.com/apache/spark/commit/4aa9ccbde7870fb2750712e9e38e6aad740e0770#diff-6b8a041f558af2b7bc50d930b1ad2670 then. I wonder if we missed any other setters that were removed in Scala? but this seems OK. CC @mgaido91 FYI

I initially did this https://github.com/apache/spark/pull/21413 so I know I marked this TreeEnsembleParams.setFeatureSubsetStrategy deprecated and need to remove it later on. When I removed it I saw setSubsamplingRate and removed it too.
I will also move all the other deprecated setters in https://github.com/apache/spark/commit/4aa9ccbde7870fb2750712e9e38e6aad740e0770#diff-6b8a041f558af2b7bc50d930b1ad2670 too. Do you prefer me to do it in this PR or have a separate PR? @srowen

You can do it here if it's also just the Pyspark part of the change for consistency. Thanks!

thanks for checking! Yes, it would be great to check them all. Thanks.

I made modifications in python to match the changes in https://github.com/apache/spark/commit/4aa9ccbde7870fb2750712e9e38e6aad740e0770#diff-6b8a041f558af2b7bc50d930b1ad2670. However, I didn't move the following 4 setters:
HasMaxIter.setMaxIter
HasSeed.setSeed
HasCheckpointInterval.setCheckpointInterval
HasStepSize.setStepSize
The reason that I didn't change these 4 setters is because besides DecisionTreeClassifier/Regressor, GBTClassifier/Regressor, RandomForestClassifier/Regressor (these are the files that changed in https://github.com/apache/spark/commit/4aa9ccbde7870fb2750712e9e38e6aad740e0770#diff-6b8a041f558af2b7bc50d930b1ad2670), quite a few other classes in PySpark also implement these HasXXX, for example, LogisticRegression, GaussianMixture, KMeans, LDA, etc. also implement HasMaxIter and rely on its setMaxIter. So I guess I will leave HasMaxIter.setMaxIter and the other 3 setters as is.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12979/
Test PASSed.

Test build #107865 has started for PR 25046 at commit 0dd3ba6.

Test build #107865 has finished for PR 25046 at commit 0dd3ba6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107865/
Test PASSed.

Merged to master

@huaxingao @srowen @mgaido91
I agree that we should remove those setter from the py side. However, we should not directly touch param/shared.py, instead we have to modify _shared_params_code_gen.py and then run python _shared_params_code_gen.py > shared.py.
This is caused by that the _shared_params_code_gen.py will automatic generate both the setter and the getter, while in the scala side, only getter is generated.
And in the scala side, DecisionTreeParams is not placed in shareParam.scala.
There are too many design conflicts between the class hierarchy of scala and py,  it's too confusing that can not be maintained easily. Maybe it is time to re-org the py side to keep it in line the scala side.
I found this when I'm adding Implement Tree-Based Feature Transformation https://github.com/apache/spark/pull/25383 in the py side.

There are too many design conflicts between the class hierarchy of scala and py, it's too confusing that can not be maintained easily. Maybe it is time to re-org the py side to keep it in line the scala side.

Yes, I do agree with you. we could also think to have a script which generates both APIs, in order to be sure that they are in sync. WDYT?

Oh, hm, OK. I am not even sure if shared.py is in sync with what the script produces then. If it's easy to fix later or separately, great, but not even sure this is a good strategy for maintaining the code.

I will modify  _shared_params_code_gen.py and use it to generate shared.py. I am out of the town and will work on this after I come back on 8/15.
I also agree there are too many design conflicts between the class hierarchy of scala and py. It's a good idea to re-org the py side to keep it consistent with the scala side.

@mgaido91  It is a good idea. I think we may start with some script that only check the parity.
@srowen shared.py is not in sync with _shared_params_code_gen.py now.
@huaxingao I tend to change this part in https://github.com/apache/spark/pull/25383,  maybe by moving DecisionTreeParams out of shared.py.  Hope you could help reviewing.


Repo sync for protected CLA branch

The pull request is created from main638066595551561138sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit a1de468:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Add Game: m,n,k-game

This PR contains an implementation of m,n,k-game, which is a generalization of tic-tac-toe and Gomoku.
The parameter defaults of m=15, n=15, and k=5 make the m,n,k-game identical to Gomoku. When set to m=3, n=3, and k=3, it is identical to tic-tac-toe.
I hope this game is a welcome addition to OpenSpiel.

Thanks!


Cheapest product link displaying twice

When working with the latest build the cheapest product link is displaying twice. @aadithya-naresh Kindly check this.

Will look into it


First line in test output obscured by heading

The first line in the test output window is obscured by the test name:

For record-keeping purposes, this bug is known and was discussed in another issue in Nexus Unit Testing (https://github.com/TheNexusAvenger/Nexus-Unit-Testing/issues/4). I have seen this bug myself, and I hope that I can figure out a fix at some point.

This should now be addressed in the newest version.

Awesome, thank you.
From: TheNexusAvenger<EMAIL_ADDRESS>Sent: Wednesday, 6 January 2021 10:57 AM
To: TheNexusAvenger/Nexus-Unit-Testing-Plugin<EMAIL_ADDRESS>Cc: psdillon<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [TheNexusAvenger/Nexus-Unit-Testing-Plugin] First line in test output obscured by heading (#2)
This should now be addressed in the newest version.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://apac01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FTheNexusAvenger%2FNexus-Unit-Testing-Plugin%2Fissues%2F2%23issuecomment-754972903&data=04|01||9019bff09314434c0fd108d8b1d586b2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637454877939760440|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Vpkt4aqqHHPglgDj2Mx3whnFkxYspvHj8d7e8iuF9Xw%3D&reserved=0, or unsubscribehttps://apac01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA4YI5MVDKNHOAFBJXFR6ZTSYORK7ANCNFSM4OZBO7DQ&data=04|01||9019bff09314434c0fd108d8b1d586b2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637454877939770437|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ywg9Vk8MBzBJFoGZTRPAfN3DUlHU8mpuyh3%2BDg2dR8M%3D&reserved=0.

Awesome, thank you.
From: TheNexusAvenger<EMAIL_ADDRESS>Sent: Wednesday, 6 January 2021 10:57 AM
To: TheNexusAvenger/Nexus-Unit-Testing-Plugin<EMAIL_ADDRESS>Cc: psdillon<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [TheNexusAvenger/Nexus-Unit-Testing-Plugin] First line in test output obscured by heading (#2)
This should now be addressed in the newest version.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://apac01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FTheNexusAvenger%2FNexus-Unit-Testing-Plugin%2Fissues%2F2%23issuecomment-754972903&data=04|01||9019bff09314434c0fd108d8b1d586b2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637454877939760440|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Vpkt4aqqHHPglgDj2Mx3whnFkxYspvHj8d7e8iuF9Xw%3D&reserved=0, or unsubscribehttps://apac01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA4YI5MVDKNHOAFBJXFR6ZTSYORK7ANCNFSM4OZBO7DQ&data=04|01||9019bff09314434c0fd108d8b1d586b2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637454877939770437|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ywg9Vk8MBzBJFoGZTRPAfN3DUlHU8mpuyh3%2BDg2dR8M%3D&reserved=0.


Creation of Dataset fails

Describe the bug
The Node.js Version needs to be updated. CloudFormation logs
resource handler returned message: "The runtime parameter of nodejs12.x is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (nodejs18.x) while creating or updating functions. (Service: Lambda, Status Code: 400, Request ID: 60891d40-7199-4e22-a3ca-21ba40f2ef36)" (RequestToken: e89d2d8c-8cc9-e15d-1fa0-91623b8e4abc, HandlerErrorCode: InvalidRequest)
How to Reproduce
Create a dataset from Data.all console

Expected behavior
Dataset is correctly created.
Your project
No response
Screenshots

OS
linux
Python version
3.7+
AWS data.all version
1.5.0
Additional context
No response

Thanks @marjet26 I was able to reproduce the issue. We are working on a fix as this is a priority issue

It is related to this issue: https://github.com/awslabs/aws-dataall/issues/425


Logging using logrus

Closes #15

Depends on #13

After you fix linting and conflicts this can be merged


Add Location and Map

http://www.mcmenamins.com/CrystalBallroom

done.  @lenadroid knocked that out. Thx!


When trying to start a Google login flow on this specific device I get ERROCODE 10

Step 2: Describe your environment

Android device: Redmi 6
Android OS version: 9.0
Google Play Services version: lastest release updated
Firebase/Play Services SDK version: lastest release updated
FirebaseUI version: updated

Step 3: Describe the problem:
When trying to start a Google login flow on this specific device I get ERROCODE 10.
When I click on google login, the flow appears, I click on the google account that I want to login and the error code 10 occurs to me.
Steps to reproduce:

Click on flow
Click on account
Error is happened, this error: ErrorCodes.EMAIL_LINK_CROSS_DEVICE_LINKING_ERROR

Expected Results:

What did you expect to happen?
On other devices, such as the emulated S9 +, everything goes well and the login is carried out with complete success.

Relevant Code:
This code is the flow of logging with Google account on FirebaseAuth
protected void onStart() {
      super.onStart();
      GoogleSignInAccount account = GoogleSignIn.getLastSignedInAccount(this);
      if(account != null){
          //Toast.makeText(getApplicationContext(), "User already logged", Toast.LENGTH_LONG).show();
      }else{
          //Toast.makeText(getApplicationContext(), "New User", Toast.LENGTH_LONG).show();
      }
  }

      googleLogin.setOnClickListener(new View.OnClickListener() {
          @Override
          public void onClick(View v) {                startActivityForResult(AuthUI.getInstance().createSignInIntentBuilder().setAvailableProviders(Collections.singletonList(new AuthUI.IdpConfig.GoogleBuilder().build())).build()
                      , REQUEST_LOGIN_GOOGLE); // This error occurs after this intent
          }
      });

Example of error occurred on Google Firebase Test Lab devices, error code 10.

Update:
I just implemented using EXACTLY the example on the Android library website, inserting my key in a String in the application, however, the error persists, some devices log in successfully, others occur this error incessantly, until then without correction.
Link of example documentation android

I just noticed something else, my SHA1 key in firebase is DIFFERENT from my Play Console SHA1 key, can this conflict?

Two keys, one generated from Gradle and other generated from Play Console app Bundles

@LukeDaniel16 yeah this does sound like it could be a SHA1 issue, have you made sure all of those different SHA1 are in the Firebase console?
Error Code 10 is likely from Google Sign In, not FirebaseUI, where it means DEVELOPER_ERROR:
https://github.com/googlesamples/google-services/issues/360

@LukeDaniel16 yeah this does sound like it could be a SHA1 issue, have you made sure all of those different SHA1 are in the Firebase console?
Error Code 10 is likely from Google Sign In, not FirebaseUI, where it means :
googlesamples/google-services#360DEVELOPER_ERROR

I have a sure that every SHA-1 was added on Firebase Console, i checked the SHA-1 on Android Studio Gradle and check on Play Console too, i linked the playstore app to firebase, the error still goes on ... I'm lost, this basically only occurs on Xiaomi devices...

I think the option that I use in Android Studio may be incorrect, when generating the Bundle I use the Generate Signed Bundle option, maybe I shouldn't generate the signed bundle because the bundle is already signed automatically by Google Play Console, right? So I should use Build Bundle.

Same problem... Currently i believe that problem is in Firebase Test Lab that can't use Google Credentials correctly.

@LukeDaniel16 did you download the right google-services.json file and put it in app directory of your android app. When you add SHA in firebase console you have to download the latest google-services.json  from firebase and add it to your app for the app to recognize you.


⚠ Bofrost.de has degraded performance

In 6254be2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1219 ms

Resolved: Bofrost.de performance has improved in cd1b476 after 13 minutes.


Submit a Cloud Definition ~ Scalability

Word: Scalability
Content: One of the core design principles or best practice needed in a highly available environment. You can scale up (increase the VM size) or scale out (increase the number of VMs) when a workload of a system is overloaded. For example, a shoe release on a site increases traffic for the web server, one way to mitigate this is using an auto scale to scale out more VMs or web servers and have your load balancer distribute the traffic evenly.
Author Name: Athina (Artist) Thomas
Author Link: https://linktr.ee/artistyay?fbclid=PAAaYkcfWyjCkgqM-JdaL03_xhx3bZTl7wT5a2wQpgWAmAqmCPG6nqr8mk_70
Learn More URL: (https://www.geeksforgeeks.org/scalability-and-elasticity-in-cloud-computing/)
Category: general, compute
Abbreviation: N/A

Scalability has already been submitted, thank you :) please check https://zealous-flower-0f27b070f.2.azurestaticapps.net/ for submitted words.


Contribution to coding challenge #124 : Evolution Aquarium

My Contribution to coding challenge #124.
added small Creatures, Predators, Avoiders, Food, Poison etc.

This is so cool! Do you have a write-up or anything that explains all the different behaviors etc.? Great work! ❤

Actually I don't have a write-up for now but the code is very simple.
If the creatures eat poison their health goes down, and if they eat food their health goes up (inverse for Predators).
I also added a behavior(); function which handles food(like) and poison(dislike) perception.
All the agents have a defineFear(); function which handles the behavior.
And there are Male(green) and Female(pink) creatures, and they grow big as they eat more food and when they became adults they reproduce(); (if they are close enough) over time.
And for the “Predators” they also use defineFear(); but inversely.


fix: allow authorization header

Allow authorization header. This is required for IMS authentication.

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

v1.51.4
GitHub release

Your semantic-release bot :package::rocket:


site 502 Bad Gateway

@chenguolin thanks, we're having some scaling issues and I had to restart the server.


Site dash

Add a site specific dashboard

Some notes:

Add some /reasons bars (https://chat.stackexchange.com/transcript/message/43170371#43170371)
Optimize the heck out of the reason count graph (https://chat.stackexchange.com/transcript/message/43174491#43174491)
Fix the autoflag info (https://chat.stackexchange.com/transcript/message/43170618#43170618)

More feedback (https://chat.stackexchange.com/transcript/message/43174752#43174752)


[Question]: AssertionError can not find model for

Describe your problem
i tried
git clone https://github.com/infiniflow/ragflow.git
cd ragflow/
pip3 install huggingface-hub nltk
python3 download_deps.py
docker build -f Dockerfile -t infiniflow/ragflow:dev .
then i use docker compose up -d to start server,
when i start conversation , i meet error ERROR: Can't find model for 9a0a65bc5ec611efac090242ac1e0006/embedding/maidalun1020/bce-embedding-base_v1


fix: now supporting allowed email/domain combo rules

Before submitting this PR:
Checklist

[x] No breaking changes
[x] Tests pass
[x] New features have new tests
[ ] Documentation is updated

Breaking changes
Avoid breaking changes and regressions. If you feel it is unavoidable, make it explicit in your PR comment so we can review it and see how to handle it.
Tests

please make sure your changes pass the current tests (Use the make test or the make watch command).
if you are introducing a new feature, please write as much tests as possible.

Documentation
Please make sure the documentation is updated accordingly, in particular:

Workflows. Workflows are Mermaid sequence diagrams
Schema. The schema in a Mermaid ER diagram
Environment variables. Please adjust the .env.example file accordingly
OpenApi specifications. We are using inline JSDoc annotations

Now supporting combination of allowed email/domain rules as described in the docs
<EMAIL_ADDRESS>AUTH_ACCESS_CONTROL_ALLOWED_EMAIL_DOMAINS=nhost.io,example.com

In the above example, users with the following emails would be able to register<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>whereas<EMAIL_ADDRESS>won't.

I am afraid this change isn't correct. I am closing in favor of https://github.com/nhost/hasura-auth/pull/510


IntegrationWithExsistingApps Android Update

Added permission code to handle the user permission accepted or denied cases which causes potential app crashing if not handled. Only a small piece of code added to the guide to resolve an issue I was facing with permission.
The app that I was testing following the guide available, it was crashing showing permission denied error on the android monitor, and there was no step for how to resolve the issue. So I tried out a solution so thought would share it on the official Guide to help people who are stuck with the same issue.

cc @hramos @lacker


systemd: check if the file exist on the target

Thank you.  We'll be reviewing this.  The 2023.2 release will be coming soon and this code has changed slightly.  The bbappend has been removed from meta-kria and moved to meta-petalinux.
Did you find this issue with petalinux, or with a different distribution configuration using meta-kria?

Did you find this issue with petalinux, or with a different distribution configuration using meta-kria?

I am not using the petalinux and just meta-kria with their layer dependencies in the Linux microPlatform (LmP) distribution. The  LmP distro is provided by the layer meta-lmp that support have other BSP layers but it shouldn't be relevante for the issue.


Import of existing KeyVault not possible due to case-sensitive state comparison

I would like to import an existing KeyVault into our stack. That does not seem to be possible though, as I can't seem to get matching states.
After aligning all other properties, I'm stuck with one diff in the preview:
  ~ properties: {
      ~ sku: {
          ~ name: "Standard" => "standard"
        }
    }

I cannot seem to work around this, since using SkuArgs with a lower-case standard produces:
error: azure-nextgen:keyvault/latest:Vault resource 'xyz' has a problem: 'properties.sku.name' is 'Standard' but expected one of ['standard','premium']

I noticed the case-sensisitivy issue with the PermissionsArgs too, but in this case there doesn't seem to be a constraint to lower-case on pulumi-side.
Any way to work around this for the Sku argument too?
Expected behavior
Comparisons on properties where case is treated different in Pulumi and on Azure should probably be handled case-insensitively.
Current behavior
See above.
Steps to reproduce
I expect this should be reproducible by creating any KeyVault e.g. in the Azure Portal and trying to import that.

Sorry for the inconvenience. Two options to try:

Add properties.sku.name to the ignoreChanges option of the resource.
Use the import command pulumi import azure-nextgen:keyvault/latest:Vault sample-vault <your-keyvault-id>

Meanwhile, we'll see what we can do about the validation.

Thank you for the quick reply, both options in the end still fail because of the standard vs Standard issue:
error: azure-nextgen:keyvault/latest:Vault resource 'sample-vault' has a problem: 'properties.sku.name' is 'Standard' but expected one of ['standard','premium']

I see... I guess we'll have to relax our internal check.

The check will be relaxed in 0.7.0


Show correct float metric value on flink UI.

What is the purpose of the change
This PR makes flink ui show float metric value correctly.
Before this PR:  (outPoolUsage always 0 or 1)

After this PR:

Brief change log
Verifying this change
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot run azure


Support other Doctrine flavours

Solves #72.
@dbu I'm not really sure of how I should wrap it up with the command. Should we pass an option to the command? Have several commands?
Doctrine flavours:

[x] Doctrine ORM
[x] MongoDB
[x] PHPCR

Other tasks:

Update tests

[x] Doctrine ORM
[x] MongoDB
[ ] PHPCR

[x] Add deprecation notice for previous command
[x] Update Doc & Upgrade guide

in phpcr we namespaced our commands with doctrine:phpcr, for example doctrine:phpcr:mapping:info. i think its going to be easier to have separate commands per storage layer. with some factory methods, you could reuse most of LoadDataFixturesCommand. (btw, the name of that command seems a bit wrong, as its not saying its about doctrine. but it is specific to doctrine orm...)
i would propose:

hautelook_alice:doctrine:load (and keep the fixtures:load command registered for BC but deprecate it)
hautelook_alice:doctrine:phpcr:load
hautelook_alice:doctrine:...:load (not sure what mongo / couch call their commands)

you could do a compilar pass removes any of the commands services that require a doctrine flavour that is not available. (you will notice when the manager registry service is missing)

oh, and great that you work on this!

Hm, didn't though on the compiler pass thing to remove unused things, good to hear.
Well for the command name although there's no "Doctrine" in it's in the Doctrine namespace so it would be kind of redundant IMO.
Regarding CouchDB, I didn't found any reference in doctrine/data-fixtures.
For the PR I'm not sure on when I'll be able to finish it as the next weeks promise to be very busy, but I'll do my best to include it in the 1.0.0 release.

would be great to bring it in.
RE command name: i did not mean the command class (and if you move to more injection you might even avoid tying it to  a specific doctrine flavour). the name of the command is a problem, as its called hautelook_alice:fixtures:load which does not say doctrine although its specific to doctrine.
lets ignore couch. no idea how active it is...

Indeed, and as we may add a support for Propel in the next versions it would make more sense too.

@dbu ready for review :)
Still have some tests to do though

awesome! one detail: please typehint the DocumentManagerInterface for phpcr
to allow for decorated dm's.
apart from that lets merge and make sure the phpcr-odm and cmf docs
reference this bundle in the relevant places.

@dbu somewhat ending up refactoring the Finder to create and interface and split it into FixturesFinder and BundlesResolver.
I've added a Document/Product entity for tests with Doctrine ODM and Document/Task for tests with Doctrine PHPCR-ODM. But can't get it work with Task, I get the following error:
[Doctrine\ODM\PHPCR\Id\IdException]
  ID could not be read from the document instance using the AssignedIdGenerator.

Not sure from where the error is coming from, do you have any idea?

the default ids are not autogenerated, you need to set it explicitly or set
it to auto generated id. its in the phpcr odm doc, hope you find it. i am
travelling atm and only have my mobile phone with me...

Erg, kind of not working unsure why. But don't worry if you can take a look later it's fine for now I need to update the tests anyway so it's not that urgent.

@dbu there's only how to set this ID thing and then it's good

I'll be merging it for now. There's only the tests for PHPCR that are skipped otherwise is all green.

In loading the latest from master, and only the ODM library and ODM Bundle, I'm getting the following error on the console:
[Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException]
  The service "hautelook_alice.doctrine.generator.loader_generator" has a dependency on a non-existent service "hautelook_alice.doctrine.orm.fixtures_finder".

I've got the ODM & Alice bundles enabled in the AppKernel, and the following config block:
hautelook_alice:
    locale: en_US       # Locale to used for faker; must be a valid Faker locale otherwise will fallback to en_EN
    seed: 1             # A seed to make sure faker generates data consistently across runs, set to null to disable
    persist_once: false # Only persist objects once if multiple files are passed
    db_drivers:
        orm: false #attempting to manually disable ORM to prevent error
        mongodb: true
doctrine_mongodb:
    connections:
        default:
            server: "%mongo_database_host%"
            options: {}
    document_managers:
        default:
            auto_mapping: true

Is ORM still a dependency after this merge? Or am I missing a configuration option somewhere?

ORM is definitely not a dependency, but your configuration should be good. Which version are you using? If it's not already the case, try to upgrade to the @beta.3 and open a ticket if the problem persists.

I'm currently using dev-master. I'll switch to @beta.3 and report back with a ticket if the problem doesn't resolve itself.

The beta.3 has just been release so chances are hight that the bug is still
present.
On Thursday, September 10, 2015, Daniel Lakes<EMAIL_ADDRESS>wrote:

I'm currently using dev-master. I'll switch to @beta.3 and report back
with a ticket if the problem doesn't resolve itself.
—
Reply to this email directly or view it on GitHub
https://github.com/hautelook/AliceBundle/pull/86#issuecomment-139307194.


Related work

Hello! Thank you so much for the contribution of this repo.
I'm so interested in this work, and I'm suveying papers with key words like "captioning anything" or "instance level captioning" or "per pixel captioning". Would you like to recomand some related work to me?

As far as I know, dense captioning is the most related topic, which generates captions at the region/object level. Scene graph generation is another way to describe the image at the instance level, which considers the instance as graph nodes and relationships as edges.
Here I list several early seminal works

image-based: DenseCap: Fully Convolutional Localization Networks for Dense Captioning
video-based: Dense-Captioning Events in Videos
3D data-based: Scan2cap: Context-aware dense captioning in rgb-d scans

Some recent works to combine LLMs and fine-grained visual experts for dense captioning generation:

Socratic Models: Composing Zero-Shot Multimodal Reasoning with Language
Visual Clues: Bridging Vision and Language Foundations for Image Paragraph Captioning
ChatGPT Asks, BLIP-2 Answers: Automatic Questioning Towards Enriched Visual Descriptions


Jetty JavaDoc broken link reported during build "configure" stage

Armeria build constantly print following messages to the output:
> Configure project :
Download https://www.eclipse.org/jetty/javadoc/current/package-list
Download failed: 404 Not Found
Download https://www.eclipse.org/jetty/javadoc/current/element-list
Download failed: 404 Not Found

Apparently https://www.eclipse.org/jetty/javadoc/current/package-list no longer exists.

Thanks for the notice!
Yeah, I guess we can use https://javadoc.io/doc/org.eclipse.jetty/jetty-server/latest/ instead.
They recently released version 11, so we will handle it at the next release. 😄

Thanks for the notice!
Yeah, I guess we can use https://javadoc.io/doc/org.eclipse.jetty/jetty-server/latest/ instead.
They recently released version 11, so we will handle it at the next release. 😄


[CLI] Links to sift-cli-linux are invalid for release 1.5.1

I'm following the installation instructions for installing SIFT onto a clean install of Ubuntu 16.04.03, When I try to download the sift-cli-linux file I get a 404 error from an AWS site. I've also tried transferring the "/usr/local/bin/sift" from the OVA version to the clean ubuntu version but this returns the error:
    connect ECONNREFUSED <IP_ADDRESS>:443

(which looks like it does not find the download site).
Any ideas?

Do you have something overriding DNS in your environment.
Sent from my iPhone

On Nov 20, 2017, at 16:17, Cruc1bl3<EMAIL_ADDRESS>wrote:
I'm following the installation instructions for installing SIFT onto a clean install of Ubuntu 16.04.03, When I try to download the sift-cli-linux file I get a 404 error from an AWS site. I've also tried transferring the "/usr/local/bin/sift" from the OVA version to the clean ubuntu version but this returns the error:
connect ECONNREFUSED <IP_ADDRESS>:443

(which looks like it does not find the download site).
Any ideas?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


accessibility issue using dataAdaptor

I am using select2 V4.0.12, we are loading, I am able to achive multiselect with lazyloading and everything works fie, but problem is when we try to access using screen reader like JWAS and Chrome VOX, it does not read option and looses focus.
Please find the stack overflow link as well
https://stackoverflow.com/questions/60279373/select2-accessibility-issue-using-dataadaptor
I dont find much example on google using data-adaptor methods which implements, multiselect with lazy-loading. Please help with link if you have any

It honestly looks like you should actually be using the stock data adapters for Select2 but instead of trying to hack around custom filtering with array data, do it with the AJAX adapter and make your own custom transport. That way you can read from the local array instead of reaching out to a remote data source, and you'll still get pagination (which it looks like you're looking for).

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


URLSegmentFilter is replacing too many characters

Originally reported by @thats4shaw

Forward slashes, question marks, equal signs, hashes and colons were never actually being replaced with hyphens due to the order of the default_replacements array.
The regex rule before it was always blasting it away.

PRs

https://github.com/silverstripe/silverstripe-framework/pull/10976

@emteknetnz I've updated the issue description to include the information from the PR. Please try to avoid issues that just say "parent issue for " - it's a simple matter to copy the relevant information to the issue, and then the issue itself has some value beyond just being on a board.


[Discussion] Would it be feasible to port this for other libc implementations/generalize?

I have tried to dig into the source code but sadly, my Makefile knowledge did not suffice to actually understand most of it.
But I wondered how heavily is this geared towards musl? Would it be possible/feasible to replace musl with another libc implementation to build other toolchains?

this repo is owned by the maintainer of musl libc. if you're looking for something more general with multiple libc/compiler/etc. support, there are other choices.

buildroot :  https://buildroot.org/
openembedded / yocto : https://www.openembedded.org/wiki/Main_Page and https://www.yoctoproject.org/

these are both geared towards generating full OS deployments/system images, with buildroot being easy to get started with and yocto being the ahh let's say "most enterprise ready." but the compilers from either should be generally usable.

Thanks. I am not really looking into building my own OS. What I am going for is creating docker images with cross-compilers.

the approach used here is fairly generic.i basically copied it in shell-form in the mingw recipe in sabotage linux. this spared me from having to install stuff during the build of the single components.

Okay, thanks. Then I will try it again and see what I can come up with.

As an aside, Debian provides cross-compilers for systems the project supports.
root@8fb02abdb197:~# lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 12 (bookworm)
Release:        12
Codename:       bookworm

root@8fb02abdb197:~# ( apt search linux.*arm64 2>/dev/null ; apt search g.*aarch64 2>/dev/null ) |& grep -iE '^(g(\+\+|cc)|linux)'
linux-libc-dev-arm64-cross/stable 6.1.4-1cross1 all
g++-11-aarch64-linux-gnu/stable 11.3.0-11cross1 amd64
g++-12-aarch64-linux-gnu/stable 12.2.0-14cross1 amd64
g++-aarch64-linux-gnu/stable 4:12.2.0-3 amd64
gcc-11-aarch64-linux-gnu/stable 11.3.0-11cross1 amd64
gcc-11-aarch64-linux-gnu-base/stable 11.3.0-11cross1 amd64
gcc-11-plugin-dev-aarch64-linux-gnu/stable 11.3.0-11cross1 amd64
gcc-12-aarch64-linux-gnu/stable 12.2.0-14cross1 amd64
gcc-12-aarch64-linux-gnu-base/stable 12.2.0-14cross1 amd64
gcc-12-plugin-dev-aarch64-linux-gnu/stable 12.2.0-14cross1 amd64
gcc-aarch64-linux-gnu/stable 4:12.2.0-3 amd64
gccgo-11-aarch64-linux-gnu/stable 11.3.0-11cross1 amd64
gccgo-12-aarch64-linux-gnu/stable 12.2.0-14cross1 amd64
gccgo-aarch64-linux-gnu/stable 4:12.2.0-3 amd64


Failing to use mp execution

I am trying to use the MPExecutor but I am getting the following error:
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "/data1/alferre/cc_net/cc_net/execution.py", line 145, in global_fn
    return f(*args[1:])
  File "/data1/alferre/cc_net/cc_net/mine.py", line 347, in _mine_shard
    output=tmp_output if not conf.will_split else None,
  File "/data1/alferre/cc_net/cc_net/jsonql.py", line 435, in run_pipes
    initargs=(transform,),
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/multiprocessing/context.py", line 119, in Pool
    context=self.get_context())
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/multiprocessing/pool.py", line 176, in __init__
    self._repopulate_pool()
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/multiprocessing/pool.py", line 241, in _repopulate_pool
    w.start()
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/multiprocessing/process.py", line 110, in start
    'daemonic processes are not allowed to have children'
AssertionError: daemonic processes are not allowed to have children
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/data1/alferre/cc_net/cc_net/__main__.py", line 24, in <module>
    main()
  File "/data1/alferre/cc_net/cc_net/__main__.py", line 20, in main
    func_argparse.parse_and_call(parser)
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/site-packages/func_argparse/__init__.py", line 72, in parse_and_call
    return command(**parsed_args)
  File "/data1/alferre/cc_net/cc_net/mine.py", line 509, in main
    regroup(conf)
  File "/data1/alferre/cc_net/cc_net/mine.py", line 364, in regroup
    mine(conf)
  File "/data1/alferre/cc_net/cc_net/mine.py", line 271, in mine
    ex(_mine_shard, repeat(conf), hashes_files, *_transpose(missing_outputs))
  File "/data1/alferre/cc_net/cc_net/execution.py", line 174, in __call__
    global_fn, zip(itertools.repeat(f_name), *args)
  File "/home/alferre/anaconda3/envs/mtdev/lib/python3.7/multiprocessing/pool.py", line 748, in next
    raise value
AssertionError: daemonic processes are not allowed to have children

I am running the following command
python -m cc_net mine --config /home/alferre/data/cc_net/config/config_alex.json

And this is my config file:
{
    "output_dir": "/home/alferre/data/cc_net/data_alex",
    "dump": "2019-09",
    "num_shards": 1,
    "num_segments_per_shard": 1,
    "hash_in_mem": 2,
    "mine_num_processes": 4,
    "lang_whitelist": [
        "pt"
    ],
    "execution": "mp",
    "target_size": "32M",
    "cleanup_after_regroup": false
}

Sorry for the delay, I'm just back from paternity leave. Did you solve your issue in the meantime?
I've never seen such error.
I'll try to reproduce.
Can you share more information about your execution environment ? OS and specific python version ?
Thanks.

I think the error is that you can't have a Multiprocessing process that starts its own pool of process.
In your case that mean you should use "mine_num_processes": 1 if you want several process handling several shards in parallel, or "execution": "debug"if you want several process working on the same shard.
Can you try those ?

Just type the command  export PYTHONOPTIMIZE=1 and run it  in the terminal. Then run your python -m cc_net mine --config /home/alferre/data/cc_net/config/config_alex.json

Hi, I got similar error when using execution mp as follows:
Traceback (most recent call last):
  File "/usr/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "/ssd2/dongzhe/cc_net/cc_net/execution.py", line 145, in global_fn
    return f(*args[1:])
  File "/ssd2/dongzhe/cc_net/cc_net/mine.py", line 218, in _hashes_shard
    file=conf.get_cc_shard(shard),
  File "/ssd2/dongzhe/cc_net/cc_net/jsonql.py", line 449, in run_pipes
    for res in results:
  File "/ssd2/dongzhe/cc_net/cc_net/jsonql.py", line 296, in map
    for x in source:
  File "/ssd2/dongzhe/cc_net/cc_net/process_wet_file.py", line 199, in __iter__
    for doc in parse_warc_file(iter(f), self.min_len):
  File "/ssd2/dongzhe/cc_net/cc_net/process_wet_file.py", line 117, in parse_warc_file
    for doc in group_by_docs(lines):
  File "/ssd2/dongzhe/cc_net/cc_net/process_wet_file.py", line 89, in group_by_docs
    for warc in warc_lines:
  File "/usr/lib/python3.7/gzip.py", line 300, in read1
    return self._buffer.read1(size)
  File "/usr/lib/python3.7/_compression.py", line 68, in readinto
    data = self.read(len(byte_view))
  File "/usr/lib/python3.7/gzip.py", line 493, in read
    raise EOFError("Compressed file ended before the "
EOFError: Compressed file ended before the end-of-stream marker was reached
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/ssd2/dongzhe/cc_net/cc_net/__main__.py", line 24, in <module>
    main()
  File "/ssd2/dongzhe/cc_net/cc_net/__main__.py", line 20, in main
    func_argparse.parse_and_call(parser)
  File "/home/dongzhe/dz_venv_3.7/lib/python3.7/site-packages/func_argparse/__init__.py", line 72, in parse_and_call
    return command(**parsed_args)
  File "/ssd2/dongzhe/cc_net/cc_net/mine.py", line 509, in main
    regroup(conf)
  File "/ssd2/dongzhe/cc_net/cc_net/mine.py", line 364, in regroup
    mine(conf)
  File "/ssd2/dongzhe/cc_net/cc_net/mine.py", line 257, in mine
    hashes_groups = list(jsonql.grouper(hashes(conf), conf.hash_in_mem))
  File "/ssd2/dongzhe/cc_net/cc_net/mine.py", line 206, in hashes
    ex(_hashes_shard, repeat(conf), *_transpose(missing_outputs))
  File "/ssd2/dongzhe/cc_net/cc_net/execution.py", line 174, in __call__
    global_fn, zip(itertools.repeat(f_name), *args)
  File "/usr/lib/python3.7/multiprocessing/pool.py", line 748, in next
    raise value
EOFError: Compressed file ended before the end-of-stream marker was reached

when I ran the command:
python -m cc_net mine --config config/marathon.json

where the marathon.json looks like:
{
    "dump": "2019-09",
    "num_shards": 1600,
    "lang_whitelist": ["en", "ja", "zh"],
    "lm_languages": ["en", "ja", "zh"],
    "mine_num_processes": 1,
    "execution": "mp",
    "num_segments_per_shard": -1,
    "task_parallelism": 96,
    "target_size": "4G"
}

btw, export PYTHONOPTIMIZE=1  was ran in command beforehead.


sound: Add new sampling rates 12000Hz and 24000Hz

These values were just merged upstream in oasis, let's add them.
https://github.com/oasis-tcs/virtio-spec/commit/37c6a406678a5ee891fdf5671298cb4fcfa517f2
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR have Signed-Off-By trailers (with
git commit -s), and the commit message has max 60 characters for the
summary and max 75 characters for each description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

We don't use these rates at our supported rates currently, so a new release can wait for the time being.


add new snippet script src

hello @giovanigenerali
thank you for the PR, another user has submitted a PR 3 days ago to add this and I have commented on his request to correct a mistake he has, we might also remove the standard script tag and replace it with this script with a src property since its the most popular and default for the web stack.
if that request doesn't get merged I will take your PR.
Thank you

Hello @abusaidm don't worry about it, if you receive that merge it's ok. Tks


Cannot boot application flashed to nrf52840_pca10059

Describe the bug
For some reason Zephyr chooses to flash it at 0x1000 not 0x0 for scenario without bootloaded.
To Reproduce
Steps to reproduce the behavior:
Build and flash dongle.
Expected behavior
Should boot from 0x0 if no bootloader support enabled.
Impact
Showstopper.
Screenshots or console output
N/A
Environment (please complete the following information):
zephyr ncs: 0bf5263b0522bb5cfac84eefdfdee86dc2c67e3e (upstream d3bb3cf)
Additional context
N/A

Probably somebody change configuration in DTS yet again...

Probably this one 500fa722fb4ae971f2f0668bc02af9c001168e42. I guess only one configuration is assumed and other ignored.

Hi @lemrey , we don't use MBR and so would like load address be 0 if no mcuboot and slot0 if mcuboot is present. As it is now we cannot boot our dongle at all as FLASH_LOAD_OFFSET points to incorrect place in memory.

The problem is BOARD_HAS_NRF5_BOOTLOADER. Can this be converted into project specific options instead of a board option?

nrf52840_pca10059 has no debugger, so the default behavior is different from other boards; you have to use nrfutil to program it. This behavior is documented (here), as are the instructions to have it instead be linked from 0x00.
Can you achieve the desired behavior by selecting the fstab-debugger partition file in the board DTS file?

Hi @lemrey , we have jlink connector mounted on this board. There are pads for it so such action is in line with board design.  We can access board using a debugger. Why limiting board use to a specific configuration only when there is absolutely no need for it?
I understand forcing boot step can be convenient for some group of users who are not willing to mount a jlink connector. But this also has serious drawbacks: it complicates process (yet another tool to use), increases bootup time (additional step) and it uses flash. So it would be great if it would be optional.
I can force the behavior by setting CONFIG_BOARD_HAS_NRF5_BOOTLOADER=n but it would be better if this variable would be called CONFIG_NRF5_BOOTLOADER_PERSENT as it would imply project/system feature rather than board's.

Hi @pdunaj, the functionality is not limited to a specific configuration, simply, there is a default configuration. The reasoning behind the default configuration is that the board comes with no debugger and a bootloader, so I thought that Zephyr application for nrf52840_pca10059 should support that configuration by default.
I don't understand how having CONFIG_NRF5_BOOTLOADER_PRESENT, defaulting to off would be any different from what we currently have (except for a different default setting). Why setting CONFIG_BOARD_HAS_NRF5_BOOTLOADER=n in your project (and selecting the provided fstab-debugger partitions), is not an acceptable solution for you?
I understand that you have a different use case than the default one, but if we change the default I am sure in one month there is going to be another issue asking why aren't applications working with the bootloader.

Hi @lemrey , the default is fine and I agree it should stay as is to avoid confusion. We can set the configuration option to disable bootloader.
As for the name, I have impression (maybe it is wrong) that options HAS_xyz just like refer to characteristic (compatibility) of HW and so should not be modified by application configuration. If presence of the bootloader is optional having "HAS_" in the name may be misleading. I may be totally wrong here however.

We are setting the TEXT_SECTION_OFFSET in our configuration file. So far it was working and it seems that the latest change should not really affect it. Probably the order in which values are applied is different now.
I disabled BOARD_HAS_NRF5_BOOTLOADER and dongle boots. Let's leave it as is until somebody else will complain about the name.


enhancement: add pr checks

Description
sensitive file checks
What problem we're solving?
What is the proposed solution?
Additional notes

create tag ci failing