sample1.py speech recognition script crashes on Linux

When we run the sample1.py speech recognition script in Linux command line, it is working fine.
But for our specific application, we require this sample1.py script to be called from a C program. When we call this program from the C interface, it gives the following error message without capturing and recognizing the speech.
Say something...
Speech Recognition canceled: CancellationReason.Error
Error details: Connection failed (no connection to the remote host). Internal error: 1. Error details: 2308. Please check network connection, firewall setting, and the region name used to create speech factory.
Is this a known issue or is there a workaround for this?

I was running the Python script as a system call from C program. Interestingly, if I just do a simple ping of an internet site from inside the same python script, the ping command is successful. This means that there is no network connectivity issue from that server and also from inside the python script. However the error comes in as mentioned in the issue report above "Connection failed (no connection to the remote host). Internal error: 1. Error details: 2308. Please check network connection, firewall setting, and the region name used to create speech factory.". This possibly means that there is an issue in contacting the speech engine site alone in this manner.
If we download and try the SPeech SDK C API and run the same way from my existing C program interface, the same error results. However the same sample code of C API Speech recognition library when run from the command line, is fully successful.

Can you share the relevant portions of your code so we can try to replicate the issue?
From: Vasanvs<EMAIL_ADDRESS>Reply-To: Azure-Samples/cognitive-services-speech-sdk<EMAIL_ADDRESS>Date: Thursday, 7. February 2019 at 11:14
To: Azure-Samples/cognitive-services-speech-sdk<EMAIL_ADDRESS>Cc: Christian Landsiedel<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Azure-Samples/cognitive-services-speech-sdk] sample1.py speech recognition script crashes on Linux (#155)
I was running the Python script as a system call from C program. Interestingly, if I just do a simple ping of an internet site from inside the same python script, the ping command is successful. This means that there is no network connectivity issue from that server and also from inside the python script. However the error comes in as mentioned in the issue report above "Connection failed (no connection to the remote host). Internal error: 1. Error details: 2308. Please check network connection, firewall setting, and the region name used to create speech factory.". This possibly means that there is an issue in contacting the speech engine site alone in this manner.
If we download and try the SPeech SDK C API and run the same way from my existing C program interface, the same error results. However the same sample code of C API Speech recognition library when run from the command line, is fully successful.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure-Samples%2Fcognitive-services-speech-sdk%2Fissues%2F155%23issuecomment-461360018&data=02|01|Christian.Landsiedel%40microsoft.com|8c5b21167ecc4519b6e508d68ce50496|72f988bf86f141af91ab2d7cd011db47|1|0|636851312584177749&sdata=%2FJJlA0y0imdfCU6C%2FB8nYlIl4vpOil4wisjjwOl2Dxk%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FApAZKTkOdUfYWXkk_dNSdj9xic8_bvYAks5vK_x4gaJpZM4ajlqv&data=02|01|Christian.Landsiedel%40microsoft.com|8c5b21167ecc4519b6e508d68ce50496|72f988bf86f141af91ab2d7cd011db47|1|0|636851312584187757&sdata=aIAaFrgYQ9yW92Mdev8%2F3H9dsy%2FXrvg1xLJZx9YBUpo%3D&reserved=0.

no feedback - closing


Seventeenth Community Call

Next call will be at Tue October 20th at 10am CEST (0800 UTC,1:30pm India, 5pm Korea). The time is selected to be accessible to European and Asian-based projects. We will alternate times, every 4 weeks at this time, and every 4 weeks (with 2 week offset), at a more US-friendly time - 6pm CET.
Agenda:

Testnet Plans

Stargate plans

Cosmos Ecosystem Summit

Open Q&A

Zoom link: https://zoom.us/j/627731807?pwd=aHlGZzBKUUsxd0hrdURUY2wrR2poUT09
(This is same for all meetings at Asia-friendly times)

The replay link is as below
https://vimeo.com/470099501
Thank you for attending and watching it everyone!


Maybe we should only train on the response part instead of prompt+response

Should I create a pr for this? By implementing a custom data collator similar to Stanford alpaca

I've been trying to implement this for the past few days — there might actually be a branch in this repo that does a version of this. Unfortunately I've been very busy with travel and my day job, so I haven't got around to fully testing, optimizing, and releasing...

@siriusctrl can you comment on this? https://github.com/tloen/alpaca-lora/issues/171
Its driving me crazy. :( I dont understand how these models are learning useful anything at all...

I think there are difference between "good questions" and "bad questions". I think it makes sense to train the response only for bad questions. For example: "What is the color of number 5?" is a bad question, and the response might be "number 5 does not have a color". We don't want to train on the question because it does not make sense, while we want to train on the response so that the LLM can produce a reasonable response for the nonsense question.
However, I think it is harmless to also train on good questions, e.g. "What is the color of Mars?"


autocomplete issue in kosmos [development-bcdb]

j.clients.ssh.test.[TAB]

should show properties of the sshclient
j.clients.sshkey.s

also issue here
in branch: development-bcdb

j.clients.ssh.test is a method, not an instance, choose another name e.g. j.clients.ssh.mine.
The second does not has any problems too.


Add John Kjell for external collaborator to TAG Security

See https://github.com/cncf/tag-security/pull/1033 for related approvals

Looks good to me! 👍🏼


Daylight saving time breaks detection of new activities?

I'm using antfs-cli with my Forerunner 310XT. From time to time, it tries to download not only the new files, but all activities on the device. This is particularly painful when you're affected by the problems described in #132 and #145 -- in this case such a download might run for the whole night. (There is also no convenient way to delete all activities on the device except for the last few that have not yet been transmitted...)
Now it occurred to me that the last time this happend was after the daylight saving spring change. Both devices (my computer and my Forerunner) use the same local time zone and automatically adjust to daylight savings time, but depending on how activity time stamps are stored in Forerunner internally, and how the antfs-cli sync operation compares these time stamps to local file system time stamps, this might cause all activities to be recognized as new.
Maybe it helps to convert all time stamps to UTC before comparing?

I'm observing the same issue (downloading all "old" activities), and this only starts to happen after the DST changeover.

@henningspruth same here, would not have realized it is DST related! It seems that all my old activities have somehow got new timestamps on the watch, hence the re-download, so it is possible that it's fundamentally a firmware issue (I'm assuming here antfs-cli cannot alter the activity timestamps on the watch itself).


Add a basic DesignSwitcher Roborazzi test

It will be used as a base to test basic testTags.
Been meaning to add one for a while.

It'll remove all of that.
That hacky code is already in most of the code base. When I'm talking about
fixing Unit tests, #502 is part of what I mean.
On Tue, Mar 5, 2024 at 11:54 AM Robert Lin @.***> wrote:

@.**** commented on this pull request.
In
designcompose/src/testDebug/kotlin/com/android/designcompose/DesignSwitcherBasicTests.kt
https://github.com/google/automotive-design-compose/pull/798#discussion_r1513260585
:

fun showChangeScreen() {

   with(composeTestRule) {

       onDCDoc(DesignSwitcherDoc).performClick()

       onNodeWithText("Change", useUnmergedTree = true).performClick()

       onNodeWithText("Load", useUnmergedTree = true).assertExists()

       captureRootRoboImage("ChangeFileScreen")

   }

}

@Test
fun canCheckACheckBox() {

   with(composeTestRule) {

       onDCDoc(DesignSwitcherDoc).performClick()

       onNodeWithText("Options", useUnmergedTree = true).performClick()

       captureRootRoboImage("OptionsScreenBeforeCheckingBox")

       onAllNodes(hasClickAction())[2].performClick() // Currently the only way to find it

This finds the third node that has a click action and clicks it?
In designcompose/build.gradle.kts
https://github.com/google/automotive-design-compose/pull/798#discussion_r1513261016
:

@@ -71,6 +75,23 @@ android {
add("META-INF/LGPL2.1")
}
}

// Hacky: GH-502

What will GH-502
https://github.com/google/automotive-design-compose/issues/502 improve
here that makes it less hacky?
—
Reply to this email directly, view it on GitHub
https://github.com/google/automotive-design-compose/pull/798#pullrequestreview-1917739237,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMM2SQARFD4EZAXR6NEJUDYWYBEFAVCNFSM6AAAAABDCEAEI6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMJXG4ZTSMRTG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Tim Froehlich
Test Engineer for DesignCompose and AAOS
Located in Austin TX, US Central Time
balance/froeht


Sticky Menu Appears only when scrolling from bottom to top.

Is there any quick way to do this currently it is appearing when scrolling from top to down I want menu appears when scrolling from bottom to top.

if you use Sticky with Bootstrap .nav .navbar, it may cause the problem! At least it's what happened to me.


Add notes to all fingerprint fields in our protos.

This change is

src/main/proto/wfa/virtual_people/common/event.proto line 40 at r2 (raw file):
Previously, wangyaopw (Yao Wang) wrote…
We definitely need to adjust the kotlin implementation a little bit when updating the data type of the virtual people id. Should be very simple update.

Node index is not necessarily consecutive, so in theory it can exceed int32.MAX_VALUE. In practice we expect this is mostly used by model compiler and wouldn't exceed.
The population pools offset can be larger than int64.MAX_VALUE, because it is arbitrary in the sense that MP may choose specific range for various purpose. That is actually the reason why we plan to update vpid type to match.

So let's just do it. If you update the proto in the common-repo, I can update both the c++ and kotlin labeler implementation.

we don't need to worry about node index since it is just used as the key in the nodeRef map. Reading uint32 as int32 doesn't matter.


MPX-24 - Add support for CoAP and DTLS

What does this do?
This is a feature as it adds coap proxy support without DTLS as well as DTLS temination for CoAP messaging.
Which issue(s) does this PR fix/relate to?
Resolves #24
Related issue https://github.com/absmach/magistrala/issues/1963
List any changes that modify/break current functionality?
None.
Have you included tests for your changes?
No
Did you document any new/modified functionality?
Yes, in incode comments
Notes
N/A

@1998-felix
It will be great if we provide example scripts to test the coap server and client, like how we have for MQTT, WS, HTTP with documentation in readme.md.

LGTM


OWLS-95240 - Reconcile running servers after the domain configuration change and fix for NPE

Changes to reconcile the running servers after a running domain is updated to delete a cluster and/or an independent managed server from the domain topology.. This change removes the server pods for the clustered servers and the independent managed servers which are not part of the domain topology. It also checks if the server config is null before invoking the health check to avoid the NullPointerException.
Integration test run - https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/7681/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/7685/console

The latest integration test run has 3 failures - https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/7712/ .
I ran the ItElasticLogging and ItElasticLoggingFluentd individually and they passed. The ItStickySession#testSameSessionStickinessUsingClusterService seem to be a known failure. I think the PR is ready to merge.


Image plots show proper message and disable button if image is not available

This image doesn't exist at in the experiment, refresh doesn't make much sense here.
Might depend on plots error handing cc @mattseddon @pared

Depends on https://github.com/iterative/dvc/issues/7692 & https://github.com/iterative/vscode-dvc/issues/1649.

@shcheklein revisit this, try to reproduce.
We still need error messages, or a signal that plot doesn't exist in the commit.

Can we treat the directory more like a dataset and each image more like a datapoint?

dvc plots diff does not know if there's an experiment running, and I don't think it should care. Displaying errors (or not) is contextual, so I think it's up to the extension to interpret.

@skshetry I don't think it's related to whether an experiment is running. One revision contains the file mispredicted/6599-... and the other does not.
@mattseddon Is there an error returned here? I don't see one when testing this. I think we just need to show a different message here than 'No plot to display: refresh` (something like N/A or no image at this revision). I see an output like this:
{
  "dir/img1.png": [
    {
      "type": "image",
      "revisions": [
        "workspace"
      ],
      "url": "/Users/dave/repo/dvc_plots/workspace_dir_img1.png"
    }
  ],
  "dir/img2.png": [
    {
      "type": "image",
      "revisions": [
        "workspace"
      ],
      "url": "/Users/dave/repo/dvc_plots/workspace_dir_img2.png"
    },
    {
      "type": "image",
      "revisions": [
        "main"
      ],
      "url": "/Users/dave/repo/dvc_plots/img1.png"
    }
  ]
}

@shcheklein What do you want to show in this scenario?
Some thoughts on what I'd expect:

Grouping by directory paths. Seeing all cats, dogs, muffins, etc. together would help organize it.
Showing the number of images per revision per directory. At least then I'd easily know whether there are more misclassified dogs or cats in each revision.

@mattseddon Is there an error returned here? I don't see one when testing this. I think we just need to show a different message here than 'No plot to display: refresh` (something like N/A or no image at this revision). I see an output like this:

I completely fluffed this. Didn't check the new output. Everything seems to be behaving as it needs to. I will open a PR to close this issue.

Some thoughts on what I'd expect:

Grouping by directory paths. Seeing all cats, dogs, muffins, etc. together would help organize it.

I can sort the images before sending them to the webview.

Showing the number of images per revision per directory. At least then I'd easily know whether there are more misclassified dogs or cats in each revision.

We should have this already in the plots tree.

Thanks @mattseddon! Not sure if we want a separate issue for the rest of the discussion for how to organize these images, which is probably not p1.

I can sort the images before sending them to the webview initially but the user can reorder them which makes future updates more difficult.

I meant not just sorting, but making them collapsible by directory.

Showing the number of images per revision per directory. At least then I'd easily know whether there are more misclassified dogs or cats in each revision.

~We should have this already in the plots tree.~ Where should this be shown?

🤔 I was thinking if we have collapsible directories, then we could put the number of images per revision next to those. For example, imagine the paths below are directories (cats, dogs, etc.) instead of files (I'm not tied to this UI, it's just the first idea I had):

Sorry folks, just catching up a bit. It feels like grouping, etc is a separate improvement that can be done (or not) independently from this ticket?
For this specific ticket - even an empty box (and we don't even need the border) is fine I think.


bug: Regression in meltano remove

Meltano Version
2.3.0
Python Version
NA
Bug scope
CLI (options, error messages, logging, etc.)
Operating System
NA
Description
Meltano 2.3.0 introduced a regression on meltano remove where it complains that the plugin is missing the settings_with_extras attribute.
The regression was most likely introduced in https://github.com/meltano/meltano/pull/6267 somewhere in the use of a feature flag:
https://github.com/meltano/meltano/blob/23a9099cefe148cb560d25e7ae335447858948df/src/meltano/core/plugin/settings_service.py#L63-L65
I still haven't figured out the root cause but might be related with trying to check for feature flags in the PluginSettingsService.
Other commands may be similarly affected, but at least invoke and run seem to be working fine.
Code
Traceback
Traceback (most recent call last):
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/behavior/canonical.py", line 138, in __getattr__
    value = self._dict[attr]
KeyError: 'settings_with_extras'

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

Traceback (most recent call last):
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/__init__.py", line 66, in _run_cli
    cli(obj={"project": None})
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.9/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/cli.py", line 35, in main
    return super().main(*args, windows_expand_args=False, **kwargs)
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.9/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/utils.py", line 540, in invoke
    super().invoke(ctx)  # noqa: WPS608
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.9/lib/python3.9/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.9/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.9/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/params.py", line 18, in decorate
    return func(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/params.py", line 53, in decorate
    func(project, *args, **kwargs)
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.9/lib/python3.9/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/remove.py", line 32, in remove
    remove_plugins(project, plugins)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/remove.py", line 39, in remove_plugins
    num_removed, total = remove_service.remove_plugins(
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin_remove_service.py", line 60, in remove_plugins
    removal_managers = self.remove_plugin(plugin)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin_remove_service.py", line 87, in remove_plugin
    DbRemoveManager(plugin, self.project),
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin_location_remove.py", line 86, in __init__
    self.plugins_settings_service = PluginSettingsService(project, plugin)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin/settings_service.py", line 63, in __init__
    with self.feature_flag(
  File "/Users/edgarramirez/.pyenv/versions/3.9.13/lib/python3.9/contextlib.py", line 119, in __enter__
    return next(self.gen)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 657, in feature_flag
    allowed = self.get(f"{FEATURE_FLAG_PREFIX}.{feature}") or False
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 444, in get
    value, _ = self.get_with_source(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 431, in get_with_source
    value, metadata = self.get_with_metadata(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 333, in get_with_metadata
    setting_def = setting_def or self.find_setting(name)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 598, in find_setting
    for setting in self.definitions()
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 568, in definitions
    for setting in self.setting_definitions
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin/settings_service.py", line 136, in setting_definitions
    settings = self.plugin.settings_with_extras
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/behavior/canonical.py", line 143, in __getattr__
    raise AttributeError(attr) from err
AttributeError: settings_with_extras

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

Traceback (most recent call last):
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/__init__.py", line 74, in _run_cli
    raise CliError(str(err)) from err
meltano.cli.utils.CliError: settings_with_extras
settings_with_extras

Thanks for opening @edgarrmondragon. @aaronsteers I'm assigning to Iteration 10 as a fixing regressions should be a high priority.

As noted in #6656 here this may be fixed by that PR. Will verify the fixed behaviour once #6656 merges 👍

@kgpayne No luck. #6656 didn't fix this. Seeing:
Traceback
$ meltano remove extractor tap-gitlab
2022-08-24T16:31:17.519641Z [info     ] Environment 'dev' is active
2022-08-24T16:31:18.183907Z [debug    ] Creating engine <meltano.core.project.Project object at 0x104d652d0>@sqlite:////Users/edgarramirez/edgar/meltano-dataops/.meltano/meltano.db

Removing extractor 'tap-gitlab'...

2022-08-24T16:31:19.861278Z [debug    ] settings_with_extras
Traceback (most recent call last):
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/behavior/canonical.py", line 177, in __getattr__
    value = self._dict[attr]
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/ruamel/yaml/comments.py", line 927, in __getitem__
    return ordereddict.__getitem__(self, key)
KeyError: 'settings_with_extras'

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

Traceback (most recent call last):
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/__init__.py", line 66, in _run_cli
    cli(obj={"project": None})
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/cli.py", line 38, in main
    return super().main(*args, windows_expand_args=False, **kwargs)
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/utils.py", line 541, in invoke
    super().invoke(ctx)  # noqa: WPS608
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/utils.py", line 559, in invoke
    super().invoke(ctx)  # noqa: WPS608
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/params.py", line 29, in decorate
    return func(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/params.py", line 78, in decorate
    func(project, *args, **kwargs)
  File "/Users/edgarramirez/Library/Caches/pypoetry/virtualenvs/meltano-SY7IjDqw-py3.10/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/remove.py", line 34, in remove
    remove_plugins(project, plugins)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/remove.py", line 41, in remove_plugins
    num_removed, total = remove_service.remove_plugins(
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin_remove_service.py", line 63, in remove_plugins
    removal_managers = self.remove_plugin(plugin)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin_remove_service.py", line 90, in remove_plugin
    DbRemoveManager(plugin, self.project),
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin_location_remove.py", line 88, in __init__
    self.plugins_settings_service = PluginSettingsService(project, plugin)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin/settings_service.py", line 65, in __init__
    with self.feature_flag(
  File "/Users/edgarramirez/.pyenv/versions/3.10.4/lib/python3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 658, in feature_flag
    allowed = self.get(f"{FEATURE_FLAG_PREFIX}.{feature}") or False
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 442, in get
    value, _ = self.get_with_source(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 429, in get_with_source
    value, metadata = self.get_with_metadata(*args, **kwargs)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 322, in get_with_metadata
    setting_def = setting_def or self.find_setting(name)
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 596, in find_setting
    for setting in self.definitions()
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/settings_service.py", line 566, in definitions
    for setting in self.setting_definitions
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/plugin/settings_service.py", line 138, in setting_definitions
    settings = self.plugin.settings_with_extras
  File "/Users/edgarramirez/meltano/meltano/src/meltano/core/behavior/canonical.py", line 182, in __getattr__
    raise AttributeError(attr) from err
AttributeError: settings_with_extras

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

Traceback (most recent call last):
  File "/Users/edgarramirez/meltano/meltano/src/meltano/cli/__init__.py", line 74, in _run_cli
    raise CliError(str(err)) from err
meltano.cli.utils.CliError: settings_with_extras
settings_with_extras

I think the fix is to replicate the patch Cody did on PluginInvoker.env in the construtor of PluginSettingsService, i.e.
- with self.feature_flag(
+ with project_settings_service.feature_flag(

@kgpayne @edgarrmondragon -- agreed with @edgarrmondragon's suggestion. Feature flags are generally at the project level and available only via the project_settings_service. The fix would be in this line
I'm realizing that it's not very intuitive to have access to the feature_flag method in the SettingsService base class. I'm not sure what all changes it would entail, but it might be worth moving that method into ProjectSettingsService only, if possible to avoid this in the future.

I'm realizing that it's not very intuitive to have access to the feature_flag method in the SettingsService base class. I'm not sure what all changes it would entail, but it might be worth moving that method into ProjectSettingsService only, if possible to avoid this in the future.

@cjohnhanson Thanks for jumping in! Yeah, it makes sense to implement feature flags exclusively at the project settings level. I'm almost sure it wouldn't cause any issues.

@kgpayne a good way to prevent future regressions would be to add a meltano remove at the end of this integration test:
https://github.com/meltano/meltano/blob/40a8066c3d3b5ad403bc08a4e51d195ffdd1e02a/docs/example-library/meltano-objects/index.md?plain=1#L1


connect command

:connect src snk src_lt snk_lt

same as #38 i think an entity with 2 targets and a connect/disconnect op is much more flexible and useful

relates to #36 too


Feature/test twist gate

Status
DEVELOPMENT
Description
Added tests for changes by #1609
Related PRs
#1609

In order to pass the test, it is necessary to fix autoware health checker. #2012

@sgermanserrano I added tests what I can add immediately although it is not perfect.

@sgermanserrano That is the test I can add right now.
Could you review this again?


GM can no longer select a player token to see what they can see.

This appears to be fairly new behavior. I have not found a setting that might revert it back to the original system.
Currently as a GM if you select a player's token that have PC vision enabled you no longer are able to see what they can see. Previously I would use this to see what their limits of vision are when describing rooms, or determining valid targets.

It sounds like you have GM vision turned on. Is the light on the left purple and fluorescent?

The light on the left is standard.
I checked the configuration settings and party vision is off.
I tried turning it on then off again and it did not work. When I did turn it on the light on the left did not look different.

Try disabling modules.

Done, no change.

Though I did not restart Foundry
I will try that

No change on restart

Seeing something odd. This was happening in two saved games. When I jumped in to the other one. I notice that the light bulb looks different (CF bulb vs incandescent) and then I clicked that I saw that a gm vision icon (the CFL) bulb was there (it was not in the other world). I did notice that Ctrl-G was the key stroke needed to toggle it. And that seems to have fixed that world.
I jumped back to the one I was using to show you and toggled Ctrl-G and now it seems to work, even has the CLF bulb where it was not showing up before.
Strange but I will take it. Thanks for the help. Sorry for the bother.

Dorako UI looks to be obscuring whether it's on, so you might open an issue there.


decoder and encoder shuffle table generator

This is so far a standalone utility that prints the decoder shuffle table to stdout, trying to match the existing one in the decoder source.  There's also an encoder shuffle table generator, and the lengths table.
The subroutines inside generate the actual shuffle table as the same uint8_t array used by the decoder, so they could be used to instantiate the table instead of the static declaration.

We can easily add the command line parameters later. We might also consider the possibility of just getting rid of the static table and having it generated on demand... though static tables have benefits from an engineering point of view.
I'll merge and create an AUTHORS file with your name in it.

I tweaked your executable somewhat, so that it generates something like usable C code. Feel free to change this if you want.


Region-wide search

A page-wide search would make navigation much easier. In order to ensure a good user experience, this search should deliver the results live.

Just to be sure: We are talking about a search in the the backend here and not providing a search API for apps? What should be searched? Menu items or the content of all pages? In all languages? I'm not yet sure right now what the most helpful solution is.

Yeah, a search for the backend. All entities should be searchable and I think it also makes sense to allow a search in all languages. To provide really good and fast search results, a normal SQL search will probably not be enough. Maybe it makes sense to set up elasticsearch or something similar. But this is also a discussion for the future ;)

@svenseeberg I suggest to move the issue to the Version 1.0 milestone as search fields are important feature that we have in wordpress as well. Your opinion?

I think this issue covers the global search in one region which should return arbitrary results (so no matter what content type)... this doesn't exist in WordPress in this way, right? Thus I think we only need the object-based search #63 and can implement this one later on...


Dart2js dart:mirrors does not include abstract members in declarations

import "dart:mirrors";

abstract class A {
  int foo();
}
main() {
  print(reflectType(A).declarations);
}

Prints {Symbol("A"): MethodMirror on 'A'} in dart2js and {Symbol("foo"): MethodMirror on 'foo', Symbol("A"): MethodMirror on 'A'} on the VM.
Not sure what is right, but it should be consistent at least.

Duplicate of #18721.


feat: introduce useSocial and useSocialProfile hooks

Description
This PR includes the following hooks for convenience:

useSocial() for easily accessing the Social SDK instance shared by the provider.
useSocialProfile() for easily accessing any account ID's social profile.

Context
Some time ago, something similar to this sdk has been created here and recently that module was ported as a separate repo here.
near-social-js has almost the same functionality (maybe even more) as near-social-db-sdk. That's why I decided to contribute to your module (and maybe collaborate in a future) to avoid duplicate modules.
Also, feel free to check out near-social-db-sdk on your own and leave your thoughts on what would you like to port to this module too
Type of change

[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 🏗 Build configuration (CI configuration, scaffolding etc.)
[ ] 🐛 Bug fix (non-breaking change which fixes an issue)
[ ] 📝 Documentation update(s)
[ ] 📦 Dependency update(s)
[x] 👩🏽💻 Improve developer experience
[ ] ⚡ Improve performance
[x] ✨ New feature (non-breaking change which adds functionality)
[ ] ♻ Refactor
[ ] ⏪ Revert changes
[x] 🧪 New tests or updates to existing tests

Hey @shelegdmitriy, thanks for the pull request! I'm glad to hear we can join forces on this SDK, maybe we can start a chat or schedule a call sometime to fully understand all your requirements and how you plan to use it. Also -- we discuss this project in the Build DAO telegram chat, under "NearSocialJS" topic, feel free to join!
Regarding the pull request -- my first feeling is that we should keep any react-specific hooks and components separate from the core package -- we can make this a monorepo and split this into several packages, for this: @near-social-js/react
Although I also don't want to slow down any progress -- I would be fine with merging this now into the v1 release with the understanding that this will change, but I think this will introduce more headaches on both sides if it's not implemented 100% as we want it. With that said, we're about to release v1 this week, and I think I'd recommend keeping these hooks and provider in your own app and reference v1 methods, until we feel confident + stable to move these into this package.
What do you think @kieranroneill @jaswinder6991 ?

Hey @shelegdmitriy, thanks for the pull request! I'm glad to hear we can join forces on this SDK, maybe we can start a chat or schedule a call sometime to fully understand all your requirements and how you plan to use it. Also -- we discuss this project in the Build DAO telegram chat, under "NearSocialJS" topic, feel free to join!
Regarding the pull request -- my first feeling is that we should keep any react-specific hooks and components separate from the core package -- we can make this a monorepo and split this into several packages, for this: @near-social-js/react
Although I also don't want to slow down any progress -- I would be fine with merging this now into the v1 release with the understanding that this will change, but I think this will introduce more headaches on both sides if it's not implemented 100% as we want it. With that said, we're about to release v1 this week, and I think I'd recommend keeping these hooks and provider in your own app and reference v1 methods, until we feel confident + stable to move these into this package.
What do you think @kieranroneill @jaswinder6991 ?

Thanks @elliotBraem for sharing your thoughts.
I completely agree with you here. We do not want to introduce framework specific concepts like Hooks to the core package.
I think this implementation would be better off in its own package. I am not sure if we would want to manage it in this repo just yet even in a monorepo for now.
That being said, we are also figuring out a lot of things about the library at the moment. Probably in the future we would have a clearer picture if we need anything like this.
Hopefully after v1, we can publish something for our next steps.
@shelegdmitriy we look forward to talking to you and understand what are your requirements and needs for social-db-sdk, we can indeed reduce duplicated efforts.

@kieranroneill @jaswinder6991 @elliotBraem thank you all for your feedback!  I agree with everything that you described here.

I think splitting this into a monorepo would be a good idea. The "core" work that is already in here is agnostic any JS environment and adding in React would cause issues for those that want to use this outside of React.

I like this solution and happy to improve this PR according to your suggestions. What is the next steps for me with this PR? Should I just close this one and re-open it in a monorepo? Do you want me to improve it with your suggestions and leave it here instead without closing?


[Generator] Removed the ability to configure getters for date columns

Now we use DateTime objects

Big -1 for me. Don't remove something that is not broken. The default setting for getters might need a change, but I see no reason for removing functionality.

Not sure if this configuration is still useful in a PHP 5.3 world... Moreover a getter should not have any parameters.
That's why I cleaned the code. But I agree we don't have to remove all functionnalities, if we can clean a bit the builders we should IMO.
So this PR won't be merged atm
:12, Francois<EMAIL_ADDRESS>a écrit :

Big -1 for me. Don't remove something that is not broken. The default setting for getters might need a change, but I see no reason for removing functionality.

Reply to this email directly or view it on GitHub:
https://github.com/propelorm/Propel2/pull/48#issuecomment-2721173

PR updated. No aside effect by removing this parameter :)


All comments country flags are  unknown

All comments country flags are unknown

Maxthon <IP_ADDRESS>00
Thanks

it is like that for about 4 months already, the google API no longer offers the ability to know the country of origin.

Yup, you can search for closed tickets about this..


implement decimal numeric literal

I've started on this in #208. It seems like a good way for me to familiarize myself with the code base.

Thank you. The codebase is sometimes messy. Any improvements are welcome!

So far the pull-request looks good.  You will need to determine the type of a decimal literal as you have noticed.  It gets sometimes hairly when literals are combined with operators, like 1 + 2 + 3 * 999.  Then somehow the contextual information needs to be passed along.
Maybe it's easier to introduce literals indicating types, like 100u8 and 100u256

Yes, it would be much easier to introduce literals with those endings. I'll do that for now and we can discuss changes to type inference in the future.

I've added two new expression types, DecLit8Exp and DecLit256Exp. Any ordinary-looking decimal literal is assumed to be uint256 but to write a uint8 literal one must add a u8 suffix as in 5u8.


upgrade hive

Signed-off-by: ldpliu<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/22810

/cc @elgnay @zhiweiyin318

/lgtm


Content Delivery Server (Azure WebApps) redirects are not working

Hi,
On our Content Delivery server the redirects are not working.
I've installed the module(v.1.8.2) on Sitecore 8.2 (rev. 161221) on a Azure Web Apps environment. On the Content Management Servers everything is working fine. The redirects are working and also the UrlRewriter module in sitecore is working fine for all databases (master and web).
However on the Content Delivery server the redirects are not working at all. I don't get any errors in the logs. The Hi.UrlRewrite.dll and the config files are existing on the Content Delivery server.
Any idea what could be going on and how I can fix this?

My guess is that you need to make sure the links database is set
correctly.  This module was first designed specifically for Azure.
On Thu, Aug 31, 2017, 7:47 AM Ricardo Wouda<EMAIL_ADDRESS>wrote:

Hi,
On our Content Delivery server the redirects are not working.
I've installed the module(v.1.8.2) on Sitecore 8.2 (rev. 161221) on a
Azure Web Apps environment. On the Content Management Servers everything is
working fine. The redirects are working and also the UrlRewriter module in
sitecore is working fine for all databases (master and web).
However on the Content Delivery server the redirects are not working at
all. I don't get any errors in the logs. The Hi.UrlRewrite.dll and the
config files are existing on the Content Delivery server.
Any idea what could be going on and how I can fix this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/iamandycohen/UrlRewrite/issues/36, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxmfylcX9vYbp3k_mewLPIAtiLMIqzGks5sdqtsgaJpZM4PIxwR
.

Thanks for your reply, it seems to work now a bit better. The linkdatabase configuration wasn't correctly. However on each redirect you must be logged in into Sitecore for some reason. When I go to a url which is configured as a redirect, I get the Sitecore login page (if not logged in). This also happens on the Content Delivery server where you can't login into Sitecore. So I need to be logged in into Sitecore for some reason.
The target page is just a page which is available without Sitecore login. For example even when I try to redirect to google.com, it also redirects to the Sitecore login page.
Any idea what could be going on and how I can fix this?

You have published all the redirects and their patent folders, correct?
On Tue, Sep 5, 2017, 6:51 AM Ricardo Wouda<EMAIL_ADDRESS>wrote:

Thanks for your reply, it seems to work now a bit better. The linkdatabase
configuration wasn't correctly. However on each redirect you must be logged
in into Sitecore for some reason. When I go to a url which is configured as
a redirect, I get the Sitecore login page (if not logged in). This also
happens on the Content Delivery server where you can't login into Sitecore.
So I need to be logged in into Sitecore for some reason.
The target page is just a page which is available without Sitecore login.
For example even when I try to redirect to google.com, it also redirects
to the Sitecore login page.
Any idea what could be going on and how I can fix this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iamandycohen/UrlRewrite/issues/36#issuecomment-327152791,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxmfzOIobiIAJvU6zqXYEMJZS6ymDyUks5sfTXUgaJpZM4PIxwR
.

Yes I have published them all. All items are correctly on the Web db. Also on the Content Management server when not logged in I have this issue.

Do you have any Modules installed?  Are you able to email me the contents
of your App_Config folder and also your web.config on all your servers?
CM, Publishing, Processing and CD?
On Wed, Sep 6, 2017, 9:53 AM Ricardo Wouda<EMAIL_ADDRESS>wrote:

Yes I have published them all. All items are correctly on the Web db. Also
on the Content Management server when not logged in I have this issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iamandycohen/UrlRewrite/issues/36#issuecomment-327509189,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxmf6FlcTJTobOlUHk2XNvItT8X9sl8ks5sfrHRgaJpZM4PIxwR
.

We don't have other modules installed yet. I'll send you an e-mail with the files.
Thanks for your help

Just a simple idea ....
Is your site configured correct - do you have configured for your site a loginPage?
And are there any security settings on the redirect items or the parent folder that prevent anonymous/everyone from reading the item(s)?

There is no login page configured for the current site where the redirects items are configured. The items have read rights for everyone. Same rights as the homepage for example.
See this example (I've only enabled the first redirect item Simple External Redirect):

If I may chime in, I'd like to ask if there's any particular instruction to install this module on the CD servers?

@mattyindustries I've confirmed that the workaround you propose above makes this module work with Sitecore 9. The web.configs included in Sitecore 9 by default also contain that section. I'm guessing it'll have to be a change going forward.

We have upgraded our website from sitecore 8.2 to Sitecore 9.0.1. Installed Url Rewrite 8.2 and delete the sample redirect and create own redirects. everything is working fine.
We have deployed on azure Paas webapps. on CD sever manually copy dll and configs.


In Step 7, default license for code should be MIT

Describe the bug
Currently, Creative Commons Attribution 4.0 Commons license is defaulted for Software code.  Instead, the MIT License should be defaulted.
Is this a regression?
No.
Steps to Reproduce

Go to Step 3 and add a data set with a datatype of "source code"
In Step 7 see that this data set has been defaulted with the CC BY license

Expected behavior
The MIT license should be defaulted for data sets marked as "source code"
Screenshots

@ChristianeStork please add/edit as needed


Update README.md

Addresses #12

@JohnGiorgi recent commit should address points you brought up here as well as final suggestion in #11 . Lmk what you think!

Cool, LGTM! I would write the tutorial notebook to follow the installation instructions of the readme rather than just pip install packages directly, but feel free to do that in another PR.


Add support for cross-platform targets

Currently a target can only be associated with one platform, however a common use case is to have multiple targets per-platform from a single source set to enable that target to be consumed on multiple platforms.
This feature would enable a target to specify multiple platforms, as follows:
YAML/JSON
targets:
  My framework:
    platform:
    - ios
    - tv

Specfile
target 'My framework' do
  platform 'ios'
  platform 'tv'
end

This would perform a few actions within struct:

For each platform, a target suffixed by the platform name would be generated:

My framework-iOS
My framework-tvOS

Each target would define the preprocessor variable PLATFORM_{PLATFORM_NAME}
Each target variant would share the same source set, the source set group will retain the original target's name

Great solution! Much better than currently defining duplicate targets with differing platforms. One wrinkle though would be how to reference Carthage dependancies as they are build per platform - location: Carthage/Build/tvOS/BrightFutures.framework. Another problem would be if that cross platform "target" referenced another cross platform "target". Struct would need to intelligently get it to reference the correct actual target.
Once possible solution is to have some sort of platformTargets list that acts a bit like a variant where you can specify overrides per platform that get used in the actually created target.

Hi @yonaskolb, yes we'll have to come up with a mechanism to only link to a specific platform's frameworks. Potentially we could namespace the references when using multiple platforms, for example:
targets:
  My framework:
    platform:
    - ios
    - tv
    references:
      ios:
      - location: Carthage/Build/iOS/BrightFutures.framework
      tv
      - location: Carthage/Build/tvOS/BrightFutures.framework

Or some sort of substitution system using mustache {{ platform }}, xcconfig style ${platform}/$(platform), yaml referencing, or any other style:
targets:
  My framework:
    platform:
    - ios
    - tv
    references:
      - location: Carthage/Build/${platform}/BrightFutures.framework

Generating multiple platform targets from 1 would also affect anything referencing those targets, like another target trying to pull in one of those targets as a dependancy

Yes we'd have to do a platform resolution step when resolving references to ensure that the correct target is selected as a reference.

I don't want to rush you at all, but when would you estimate something like this would be completed, along with a guesstimate of the actual 2.2.0 release?

Hi @yonaskolb. I have a full day to work on this tomorrow, so I recommend keeping an eye on the project over the next couple of days.
Chances are we can get this done very soon (days rather than weeks), but certain parts will require some thought.
Even if we don't end up releasing this week, we should still be able to get something usable in develop!

@yonaskolb since you're one of the first devs that'll be using this feature, I'll mention you on the pull request when it's ready to merge. If you could provide feedback at that time, that would ensure that the implementation of this feature is enough for your needs.


Video streams not working with newer bbb versions

fixed. problem was: Sec-WebSocket-Key was to short.


feat(GITOPSRVCE-753): Add alert rule/test for Cluster Agent Operation SLO

JIRA - https://issues.redhat.com/browse/GITOPSRVCE-753

Few points:

Please verify whether the alert should be fired when the value is lower than 10 or higher than 10, per @keithchong's comment.
Please check about the use of sum within the query. I'm not convinced it should be used, and if it does, I suspect we should not sum across clusters.
Note that sum is not summarizing over time. It summarizes the value at a given moment for all time series that fit the query. In this case, I think there might be only one time series per cluster, so I don't think sum should be used.
In Prometheus  tests cases, each entry on the input_series list is comprised of a metric along with its labels (series) and its values over time (values).
The values are provided as a string of space-separated numbers, each representing the value of the series at a given time, where the time increments between values is the interval value.

      - series: 'operationDB_rows_in_non_complete_state'
        values: '1x10'

is a shortened notation for value of 1 over 10 intervals. i.e.
      - series: 'operationDB_rows_in_non_complete_state'
        values: '1 1 1 1 1 1 1 1 1 1'

With interval of 1m, this means 10 occurrences of value 1 over ten minutes for this metric.
Also, the test will examine the rule at eval_time. In this case, it will find there 1 and the query will sum it with all time series for the 'operationDB_rows_in_non_complete_state', and in this case, there's only one such time series (only one element on the input_series list), so the result will still be 1.
The different test cases here should not differ by the number of entries, nor the evaluation time. They should differ by the values over time for the time series.
Reference: https://prometheus.io/docs/prometheus/latest/configuration/unit_testing_rules/

I have added my comments above for first 2  points, regarding lower than or higher than and regarding sum, once first 2 points gets cleared, I will look into the test comments, thank you

Hi @keithchong and @yftacherzog updated all the changes and updated dashboard as well and here is the PR, PTAL

@yftacherzog updated the changes , can you please take a look as well?


[RFC] Revisit/merge/deprecate pre_dispatch, dispatch, post_dispatch and run_stage functions from strategies and trainer

Proposed refactor
Discussion raised from #10896
After 2/n simplify spawning logic @awaelchli has done, we can take a step further to simplify hooks
It was in the plan of #10059 in step 4, "deprecate dispatch and post_dispatch"
Will it make sense to deprecate pre/post dispatch and run_stage and aggregate logic into dispatch in both trainer and accelerator?
Motivation

Pitch

Additional context

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Lite: enables pure PyTorch users to scale their existing code on any kind of device while retaining full control over their own loops and optimization logic.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, fine-tuning, and solving problems with deep learning.

Bolts: Pretrained SOTA Deep Learning models, callbacks, and more for research and production with PyTorch Lightning and PyTorch.

Lightning Transformers: Flexible interface for high-performance research using SOTA Transformers leveraging Pytorch Lightning, Transformers, and Hydra.

cc @justusschock @awaelchli @akihironitta @kaushikb11 @ananthsub

Alright, I looked at how/whether we can cleanly merge the setup() and pre_dispatch() methods and everything seems to work out quite well. I propose to merge them like so, with the on_fit_start call previously in the middle:
Before:
# trainer.py, _run():

...

self.training_type_plugin.setup(self)  # WE WANT TO MERGE THIS ...
call_hook("on_fit_start")
self.training_type_plugin.pre_dispatch()  #  ... AND THIS

...

After:
# trainer.py, _run():

...

self.training_type_plugin.setup(self)  # PRE_DISPATCH BECOMES JUST SETUP()
call_hook("on_fit_start")
...

Here is the PR showing how this would look like: #11137

Had to be very careful here, because various plugins who previously overrode the pre_dispatch() or setup() had to be adjusted depending on whether they called super().x, to avoid unwanted calls to trickle down the inheritance chain. For this reason, the PR might be a bit hard to review for anyone who is not fully on top of these details ^^


python3 at vn0.7.0

update to vn0.7.0, update sha256sum, allow python3

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


Indexing Skolems (without Service bindings)?

I have just started looking into API-X, so this may have a simple solution.  My implementation needs to have the Skolem IRIs indexed, because I am using them to create lists.
I do not require them to have service bindings, because they are just glue to build JSON-LD @lists that are subsequently constructed from a triplestore query.
I assume that the ontology for API-X is either only identifying "http://fedora.info/definitions/v4/repository#Resource" or filtering all other objects because "http://fedora.info/definitions/v4/repository#Skolem" is definitely not indexed.
EDIT: Alternative work around that I explored yesterday is possibly to implement the toolbox indexer and the apix-indexer separately (using different db graphs).  I built a docker image for the toolbox indexer yesterday, based mostly on your image that was quite a puzzle.  The sharing dependency is that they would need to use the same Fuseki instance.  I had to put Fuseki on 3030 both internally and externally, and your demo has in on 8080 internally ...
Of course, this is a small detail that applies only to the demo.
I really want to use API-X in the framework, so I will continue to work on figuring a way to do this.

Hi @christopher-johnson,
You may know that the API-X demo includes a minimal index for service discovery at http://localhost:3030/fuseki/service-index (if using docker-machine you'll need to substitute in the IP of your docker machine).  The index is included in the demo for a heretofore un-published exercise (we are planning to create a second demo this quarter which would include additional exercises around this index), but the index isn't used by API-X itself.  In fact, API-X doesn't rely on any external indexes to function.
So for those reasons, the demo doesn't include a full index of what is in the repository.

EDIT: Alternative work around that I explored yesterday is possibly to implement the toolbox indexer and the apix-indexer separately (using different db graphs).

You and I were probably trying to do the same thing yesterday :)
My suggestion would be to integrate the fcrepo-indexing-triplestore into the indexing docker container, and update the fuseki container with an additional dataset.  As you mention, this is quite a puzzle to the un-initiated, and to the initiated as well ;)
Unfortunately, this approach will ultimately probably not give you what you want, because I don't believe that fcrepo-indexing-triplestore indexes Skolems.
Ultimately, I think you would have to modify fcrepo-indexing-triplestore or otherwise write your own indexer in order to have Skolems indexed in the triplestore.
If it would help you to have an updated fuseki and indexing docker containers with fcrepo-indexing-triplestore working out of the box, I can probably tweak what I have and push updated docker images.  That would just require you to implement the actual indexing code, and install it into the indexing container.


Provide an active editor for images via window.onDidChangeActiveTextEditor

VSCode Version: 1.24
OS Version: Windows 10 x64

Steps to Reproduce:

Setup an extension to listen to window.onDidChangeActiveTextEditor
Open an image
Observe that the active editor is undefined

Since an image is backed by a file can some sort of editor be returned so that an extension can get the Uri and maybe the type of editor (i.e. image)
This hampers GitLens and other extensions from being able to deal with images -- showing history, diffs, etc.

Does this issue occur when all extensions are disabled?: No

I believe this is by-design for onDidChangeActiveTextEditor since images are not backed by text editors
Not sure if there are any existing issues tracking something like an activeResource that would have the uri of the current file

Yeah, I'm just dying for a "tab" model rather than an "editor" model.


Bulk enrollment

I’m looking for information on how to change the 20 devices limit that the Windows Configuration Designer advises of. I can’t find anything in the MSDocs articles for bulk enrollment or provisoning packages that advises on how to actually change this limit. I am not sure if it is related to the token itself or the account used to generate the token.
If anyone has any answer to this, if you could help that would be appreciated. Thanks!

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

ID: c8c7ef37-60ce-464e-a2ce-88a074280220
Version Independent ID: 844aa895-64a9-72f5-bd94-09c9089acc3c
Content: Bulk enrollment
Content Source: windows/client-management/mdm/bulk-enrollment-using-windows-provisioning-tool.md
Product: w10
Technology: windows
GitHub Login: @lomayor
Microsoft Alias: dansimp

Hello, @MSLuke
The issue is under investigation and you will be notified with any changes on the way.
Thank you for providing feedback.

@MSLuke
With the help of your feedback, the content has been updated with relevant changes accordingly. Thanks.


Migrate examples to 0.4.1

part of #116
This PR will migrate southpark/urbandict/flower
To test it, please use
python app.py -t index
python app.py -t query

Thank you @Joan for your great suggestions!

@YueLiu-jina Is this PR ready for merging? It looks like all the changes are fixed.

@nan-wang There's still a problem and Joan is taking a look in it. How about merge this first and fix problem in later PR?

@YueLiu-jina.
This is good to be merged. I will look into why matches do not get full text in the output but it is not blocking and maybe expected behavior


Add Nuget Package To Workflow

Nuget package needs to be done to the workflow

This has been done. Not the best but it works for now.


#742 net: pass ipv6 zone param to net.ListenUDP

Description
Pass ipv6 zone param to net.ListenUDP. Also checked that it's specified in all other calls of net.ListenUDP. Tested it locally with ipv6 link-local and global unicast addresses.
However, should I add a unit test for that @Sean-Der? I could add it in the gather_test.go, but it would be host specific and require an assigned ipv6 address.
Also don't forget to backport to v3, because ipv6 link-local addresses appeared in v3.
Reference issue
Fixes #742

@N1cOs Can you try<EMAIL_ADDRESS>now?

I will check in a few days, but on v4.0.2 ignoring link-local addresses definitely fixes the high CPU consumption on connection initiation


Fix buffered container children being clipped wrong on OpenGL

Closes https://github.com/ppy/osu/issues/22982
Closes https://github.com/ppy/osu/issues/23009

Please treat with caution. While I tested that this does work, I only kind of know what I'm doing with this one.
Testing:

Renderer/Platform
current master
this PR

D3D11 @ Windows

OpenGL @ Windows

Vulkan @ Windows

OpenGL (legacy) @ Windows

OpenGL (legacy) @ Android

Metal @ macOS

[x] Probably could use a sanity check on Metal too (iOS/macOS).

which is flipping all vertices vertically when rendering to a framebuffer, as opposed to rendering the framebuffer texture upside-down

It's hard to explain without going into RenderDoc and seeing it for yourself, but the purpose of that PR was to standardise UV coordinates such their origin is always as expected by the renderer.

If you flip UV coordinates, as we were doing previously, then you'd also need to flip any offsets on those coordinates (the blur direction). It's why we previously had the blur direction specified as a negative value for a clockwise direction, but I don't think that is a detail which the user should be concerned about.

Right, I've read through the blur logic and I understand now. Seems like it's a good direction then.

That being said, I don't know if -gl_Position.y is the correct way of implementing it as opposed to something like viewport.Height - gl_Position.y.

Flipping the sign of the Y coordinate should essentially be the same as viewport.Height - Y, since the bounds of the NDC is mapped to the bounds of the viewport rectangle, as far as I understand.


How to turn off features?

In the README it says features can be turned off modularly, but it doesn't say how. Deleting files maybe? Should probably be explained.

No, much easier. Open userChrome.css and you'll see a bunch of @import command with short description:
•••

To disable/enable some part of the style, add/remove "/* " before its @import. Example:
@import "userChromeReplaceEverythingWithPizza.css"                [ ENABLED ]
/* @import "userChromeReplaceEverythingWithPizza.css"             [ DISABLED ]

•••

/* ### ###  Elem Animations & Rounding  ### ### */
@import "userChromeAnimations.css";

/* ### ###    PhotonUI icons (.icon)    ### ### */
@import "userChromeIcons.css";

/* ### ###   Hide Bloaty/Useless Opts   ### ### */
@import "userChromeOptHider.css";

•••

Let's say you want to disable Photon icons. To do this, add /* before @import "userChromeIcons.css", like this:
•••

/* ### ###    PhotonUI icons (.icon)    ### ### */
/* @import "userChromeIcons.css";

•••

Do the opposite to (re-)enable a disabled feature. Hope this helps

I guess the issue is now resolved, also added the link to it in README for clarity


Flying mounts in LycanitesMobs stop working when you press the sneak button to go down.

I am currently running

SpongeForge version: 1.12.2-2838-7.2.4-RC4054
SpongeVanilla version:
Java version: 1.12.2
Operating System: PC

Plugins/Mods: SpongeForge, LycanitesMobs

Flying mounts in LycanitesMobs stop working when you press the sneak button to go down. Only happens with SpongeForge installed. Issue can be replicated easily by installing the two mods, summoning a flying mount, putting a saddle on it, and flying it, then trying to go down.

Duplicate of https://github.com/SpongePowered/SpongeForge/issues/3133


[FEATURE] new storage backend and logging

A new storage backend (Redis? TiKV? some other K/V ram-based db).
Also to prevent something like #26 from happening again...
Requirements
[  ] Resilient to data loss (ie. regular backups)
[  ] Resilient to down time... (Kubernetes?)
[  ] Consistently low latency (95% <5ms lookups at full load); needs testing
Ideas for Implementation

K/V db running on a HA kubernetes cluster (borrow university servers)
API: better error catching (request processing)

looks like time to do this...


ci: add forcetypeassert linter

This linter is required by Ant Group QA Team.
Signed-off-by: spacewander<EMAIL_ADDRESS>

Ignore the low coverage. Most of the type assert is already protected early so the branch is not hit.


proxy-nsmd-k8s always filters for local NSEs

Expected Behavior
FindNetworkService RPC calls should return all endpoints (NSEs) available in a domain for a NetworkService, even if the call is for a remote (interdomain) NSR.
Current Behavior
If there's local endpoints (NSEs), proxy-nsmd-k8s will only return those local endpoints (NSEs).
Failure Information (for bugs)

Steps to Reproduce

Deploy interdomain NSM with an interdomain NSR
Deploy a network service with NSEs in all domains
Trigger FindNetworkService() for the interdomain NSR
The NetworkServiceResponse will only contain the local domain's endpoints.

Context
The fix for #2169  (PR #2170) introduced this filter.
A potential fix that works in my setup is to wrap the localNSE filter in an proxy-nsmd-k8s env var setting:
diff --git a/k8s/pkg/proxyregistryserver/discovery.go b/k8s/pkg/proxyregistryserver/discovery.go
index 3072aa86..7f44e1c5 100644
--- a/k8s/pkg/proxyregistryserver/discovery.go
+++ b/k8s/pkg/proxyregistryserver/discovery.go
@@ -4,6 +4,7 @@ import (
        "context"
        "fmt"
        "os"
+       "strconv"
        "strings"

        "github.com/pkg/errors"
@@ -71,8 +72,10 @@ func (d *discoveryService) FindNetworkService(ctx context.Context, request *regi
                        return nil, dErr
                }
                for _, nsm := range response.NetworkServiceManagers {
-                       if url, urlErr := d.currentDomainNSMgrURL(ctx, d.clusterInfoService, nsm.Url); urlErr == nil && nsm.Url == url {
-                               return d.handleLocalFindCase(response, url), nil
+                       if isLocalNseFilter, _ := strconv.ParseBool(os.Getenv("PROXYNSR_LOCALNSE_FILTER")); isLocalNseFilter {
+                               if url, urlErr := d.currentDomainNSMgrURL(ctx, d.clusterInfoService, nsm.Url); urlErr == nil && nsm.Url == url {
+                                       return d.handleLocalFindCase(response, url), nil
+                               }
                        }
                        nsm.Name = fmt.Sprintf("%s@%s", nsm.Name, nsm.Url)
                        nsmURL := os.Getenv(ProxyNsmdAPIAddressEnv)

Failure Logs

@denis-tingajkin Could you have a look at this?

@tiswanso can you attach your CLIENT_NETWORK_SERVICE?

Also, I think the fix of the issue could be:
1. Resolve all NSEs
2. Replace 'local interdomain' NSEs with local NSEs


DDD with Symfony 4

Since Symfony is widely used by the PHP community, is it possible to provide a project using Symfony 4 with a folder-by-feature structure?

Sure, send some money.
Sorry if this sounds harsh, but this is an open source project and we have
only limited resources.
On Fri, Dec 7, 2018, 22:55 Kwadz<EMAIL_ADDRESS>wrote:

Since Symfony is widely used by the PHP community, is it possible to
provide a project using Symfony 4 with a folder-by-feature
https://dzone.com/articles/package-by-layer-for-spring-projects-is-obsolete
structure?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codeliner/php-ddd-cargo-sample/issues/53, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYEvGB5tACvU0np_T6N_NP9o1J3xnCtks5u2oF-gaJpZM4ZIlit
.

No worries, I supposed some maintainers use Symfony, since it's rare nowadays using PHP without any framework.

prooph maintainers are rare :)
Sascha-Oliver Prolic
Am Di., 11. Dez. 2018 um 18:03 Uhr schrieb Kwadz<EMAIL_ADDRESS>
No worries, I supposed some maintainers use Symfony, since it's rare
nowadays using PHP without any framework.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/codeliner/php-ddd-cargo-sample/issues/53#issuecomment-446143634,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYEvMVNmsmgFQuohiHOFHDlyugv6EHLks5u34MMgaJpZM4ZIlit
.

Oh some money would be nice. Good idea @prolic ;)
DDDesign itself is not about frameworks. Anyway, choosing a good framework to handle http requests, authentication, authorization, etc. is important for every project and can be part of tactical DDDesign, but it highly depends on existing team Skills. It's not the focus of this example project.
Feel free to fork the example and replace zend-expressive with Symfony. Send me a link to your fork and I'd be happy to add it to the readme.

I was more focus on the architecture than the framework components. Anyway, thanks for your answers. If I have the time to create an example using Symfony 4, I'll send you the link ;)


[CENT-331] unify checkVariables and checkState

checkVariables now handles both the original FiatTokenV1 verification and the new compact format for state verification.
-- Created objects ControllerState and MintControllerState
-- checkVariables checks what type of input it gets
-- unit tests call checkVariables with an array of contracts and state variables

Closing pull request. Fixed Issues in PR 219. (Need a separate PR because centre-tokens became public and repository detached).


Derive ZIO Test Gen[R, A] from Schema[A]

Work in progress to derive zio-test's Gen[R, A] to fix #56.
@jdegoes
John could you check if I'm on a right track?

@jdegoes
Hey John! Could you give a hint or two on how to deal with EnumN I took few attempts to implement it but all of them failed(. Oh, Also it seems I need to do nested pattern matching  on CaseClass[_] in order to cover it and there going to be 22 cases..)

BTW, as you can see I implemented both Enumeration and Record in the same way. Well, the implementation looks identical from Gen point of view sort of

Well, I need to rebase my PR due to recent changes in Schema so the comments of mine I made above is irrelevant

@igrebenik
For record, if you have n terms, you have to generate n values, and combine them all together to get the product.
For enum, if you have n terms, you have to pick one of them at random, and generate that one.
That's the main difference.
Also: with @thinkharderdev's work on DynamicValue, it's possible you can piggyback on that to get a derive for any type that has a Schema (because he implemented logic to make dynamic values given a schema).

@jdegoes  I did exactly that I just had the issue convincing the compiler to compile the code)

@igrebenik I see all checks have passed, can you resolve merge conflicts and I will take a look? 🙏

@jdegoes Done!
I couldn't implement Schema.Enumeration case on DeriveGen#gen. I added special function for that in the StructureGen called tuple but I couldn't make it compile 😅

@igrebenik I suggest we pair on finishing this one!

@thinkharderdev Can you take a look and approve if you're happy with these changes? Thank you!

Looks great! Thanks @igrebenik!

@jdegoes @adamgfraser @thinkharderdev I was able to do it thanks to you!!!

@thinkharderdev Hey Dan, should I update (resolve conflicts and whatnot) this pr. It's just I've got a feeling it became obsolete and needs to be drop due to recent changes and future plans?

Hey @igrebenik. I think it should be mostly still ok. I don't think there's any future plans that would invalidate this work.

Awesome! I will update it shortly


Middleware: compression

Using https://docs.rs/async-compression/0.1.0-alpha.1/async_compression/ we should write a middleware that sends Accept-Encoding parameters to the server, and decompresses the response.
Examples

https://github.com/rustasync/tide/blob/master/tide-compression/src/lib.rs

Hi, I would like to help with this as a learning experience if that's okay? I looked at the example code from the Tide implementation and will plan to use that as inspiration. The end goal would be something like:
let uri = "https://httpbin.org/get";  let string = surf::get(uri)     .middleware(ResponseCompression::default())     .recv_string().await?;
I used the name ResponseCompression (Happy to change it if you want) since I assumed this would not handle compression of the body for the outgoing request, just the response back with Accept-Encoding headers. The middleware would set the header with br, gzip, zstd and deflate and then decode the server response.
To test this I thought I could implement a stub HttpClient that could just respond with compressed data. Have I understood the issue correctly? Or did you mean that the middleware also should handle compression for outgoing requests and then set the Content-Encoding header for the request?

@Nehliin That would be fantastic, thanks! I feel like you're mostly on the right path here for sure!

Or did you mean that the middleware also should handle compression for outgoing requests and then set the Content-Encoding header for the request?

This isn't quite possible unfortunately because by default we can't know what the client accepts. HTTP/2 applies some compression by default to circumvent this, so I wouldn't worry too much about it. Decompressing server responses is by far the right priority here!

Okay great! I'll start this week

I didn't quite have as much time this week as I wished but I thought I might give some status update at least. I have implemented a first version of the middleware but polish is still needed before I'll have a PR up. I have also implemented the StubClient which tests the decompression and that the right headers are set. I will write some more tests and clean the code up a bit afterwards. I sure have a lot to learn working with streams so I would love to get some feedback to make it more idiomatic when I have something to show you.

@Nehliin yay, thanks for the update! -- and yeah for sure, happy to give feedback!

I changed my mind and thought I'd put up a WIP PR instead of trying to fix everything in isolation, the earlier the feedback the better!


JBDS-3997 Upgrade to vagrant 1.8.1 and compatible virtual box 5.0.26

test this

Should also adjust validation to new version.

Test failures aside, are we getting rid of the developers.redhat.com redirects?

I am temporary put direct links and created for redirects I'll create issues in RHDENG and then update them when they are in place.

Released to master as 9dff7ee after re-base.


Create Notification Subsystem and UI Presentation

There are specific types of system events that users should be notified of in the UI. This would include various types of failures, connectivity issues reaching various services (NVD, NSP, etc) and updates to dependency track that are available.
This enhancement may or may not be part of a large alert subsystem that would eventually tie into vulnerability monitoring. More research is required to determine if these things are separate or part of the same feature set.

Notifications, by their very nature, are PUSH mechanisms. In addition to pushing notifications to the UI/etc, there is a need to push items to external systems (i.e. Kenna #39 in an automated way, and to JIRA in the same workflow being designed in #65).
Other integrations for push include chatbots, etc.
The notification subsystem will need to be flexible enough to handle the wide variety of usecases currently planned.

Not to self: check out https://skywalkapps.github.io/bootstrap-notifications/

Leaning towards SSE instead of websockets

Twitter Poll: https://twitter.com/DependencyTrack/status/1010311341989654529

Outbound notification functionality is now complete. Notifications are generated for multiple types of events, they can be subscribed to in the UI, and are published to Slack, Microsoft Teams, Outbound Webhooks, Email, and the system console.
Alerting of notifications in the UI (for example in the top right of the screen whenever something happens) is out of scope for this ticket.


[pre-sve] We cant set UserCanConsent option in OTA at run time so TC-SU-2.3 fails

Problem

expected behavior
- DUT should obtain user consent before transferring the image.

actual behavior
- there is no option to obtain user consent at run-time

system configuration
-  DUT used : ESP32
-  app use: all-clusters-app
- chip-tool and DUT commit id : 425f1fb916ca154deeb75cd82cd5196778aba152 (pre-sve SHA)

logs:
- chip-tool : chip-tool-4.txt
- ota-requestor logs:
otar-1.txt
otar-2.txt
otar-4.txt
otar-6.txt
- corresponding ota-provider logs:
otap-2.txt
otap-6.txt

Bumping this issue to cert blocker as it's a request cert blocker discovered during pre-SVE tests.

Cert Blocker Review: tagging @franck-apple to notify ESP

@shubhamdp @jadhavrohit924 PTAL


Adds CLI option for custom true negative intent

Adds the -n, --true-negative-intent option to the compare command, which allows you to customize the intent name that will trigger true negative intents.
Fixes #211

Abandoning in favor of #274.


Custom menus steal focus when using Alt for column selection in the editor

Issue Type: Bug
On Windows,

Configure "editor.multiCursorModifier": "ctrlCmd"
Use a mouse to make a selection by mousedown + dragging
Press Alt to switch to doing a column selection
At the end, the menubar steals the focus

VS Code version: Code - Insiders 1.32.0-insider (c42088be3b557e5607f7cde6b3b10c0dba6763dc, 2019-03-04T05:18:19.410Z)
OS version: Darwin x64 18.2.0

@sbatten We should decide if we want to do anything for this release, otherwise we might want to pull the entry from the release notes, since this cripples the feature on Windows.

Closing. @alexandrudima  merged the PR.


Allow extensions to extend vue instance configuration

Description
Allow extensions to add elements like custom components, libraries, or error handlers for error tracking.
E.g.: https://github.com/cnotv/sentry-rancher-extension
Context
In the current implementation, extensions logic includes the Vue instance attribute but is not passed as an argument. It is impossible to inherit it within the extensions and add further capabilities.

Closing issue due discussion in PR https://github.com/rancher/dashboard/pull/12629


How do you write genosets?

Where I can find a guide/tutorial/reference to write useful GQLs queries for genosets?

readme has been updated


Unable to reproduce the machine translation results (WMT16) reported by Ott et al. (2018), Wu et al. (2019), and Fan et al. (2020)

❓ Questions and Help
I have been trying and failing to reproduce the results of the WMT16 En-De translation task, which was reported in Ott et al. (2018) Scaling Neural Machine Translation and was later used as a baseline for comparison in subsequent work: Wu et al. (2019) Pay Less Attention with Lightweight and Dynamic Convolutions and Fan et al. (2020) Reducing Transformer Depth on Demand with Structured Dropout.
I followed closely the instructions in the documentation to train a transformer_vaswani_wmt_en_de_big model on 16 GPUs using the following command:
fairseq-train \
    --distributed-port 12345 \
    --distributed-world-size 16 \
    wmt16/data-bin/wmt16_en_de_bpe32k \
    --arch transformer_vaswani_wmt_en_de_big \
    --share-all-embeddings \
    --optimizer adam \
    --adam-betas '(0.9, 0.98)' \
    --clip-norm 0.0 \
    --lr 0.001 \
    --lr-scheduler inverse_sqrt \
    --warmup-updates 4000 \
    --warmup-init-lr 1e-07 \
    --dropout 0.3 \
    --weight-decay 0.0 \
    --criterion label_smoothed_cross_entropy \
    --label-smoothing 0.1 \
    --max-tokens 3584 \
    --update-freq 8 \
    --fp16

After 20 hours of training (>150k steps), using checkpoint averaging (as instructed) I obtained only 27.79 BLEU4 (compound split) instead of the reported 29.29.
As it is not clear how many training steps Ott et al. (2018) and Fan et al. (2020) used in their experiments, I followed Wu et al. (2019) and made sure to use exactly the same training configuration as theirs:

Therefore, I trained the model on 8 GPUs using the following command:
fairseq-train \
    --distributed-port 12345 \
    --distributed-world-size 8 \
    wmt16/data-bin/wmt16_en_de_bpe32k \
    --arch transformer_vaswani_wmt_en_de_big \
    --share-all-embeddings \
    --optimizer adam \
    --adam-betas '(0.9, 0.98)' \
    --clip-norm 0.0 \
    --lr 0.001 \
    --lr-scheduler inverse_sqrt \
    --warmup-updates 10000 \
    --max-update 30000 \
    --warmup-init-lr 1e-7 \
    --dropout 0.3 \
    --weight-decay 0.0 \
    --criterion label_smoothed_cross_entropy \
    --label-smoothing 0.1 \
    --max-tokens 3584 \
    --update-freq 16 \
    --fp16

(Note that Wu et al. used 10K warmup steps and trained the model only for 30K steps.)
Using checkpoint averaging and a beam size of 8, I obtained only 27.53 in BLEU.
Could you please check this?
What's your environment?

fairseq Version (e.g., 1.0 or master): master
PyTorch Version: 1.8.1
OS (e.g., Linux): Linux
How you installed fairseq (pip, source): source
Build command you used (if compiling from source): pip install --editable ./
Python version: 3.8.2
CUDA/cuDNN version: 10.2/8.0
GPU models and configuration: V100

I've realized that there have been some major changes in the implementation of Transformer recently, this might be the cause...

@myleott @huihuifan Could you please have a look at this issue?


isAddress() does not confirm the checksum

Version: 3.11.1
Python: 3.5
OS: linux

What was wrong?
I expected an isAddress() call on an address with an invalid checksum to fail.
According to the docs:

If the address contains mixed upper and lower cased characters this function also checks if the the (sic) address checksum is valid according to EIP55

In [2]: Web3.isChecksumAddress('0x5B2063246f2191f18f2675cedb8b28102e957458')
Out[2]: False

In [3]: Web3.isAddress('0x5B2063246f2191f18f2675cedb8b28102e957458')
Out[3]: True

How can it be fixed?
Web3.isAddress is a passthrough to eth_utils.is_address. At first glance is_address doesn't do anything to check the checksum. Looks like the logic just needs to be added.

Waiting on the test update in: https://github.com/pipermerriam/web3.py/pull/235
Then will update in web3utils and here

Waiting for the next deploy

PR #235 has been merged.
Should be OK to close?..

Thanks for catching this @veox


Compile error by assigning an array to inner const

Given
class A
  B = []
end

Expected
no error
Actual

thread 'main' panicked at '[BUG] lvar `ary@0_' not found in ctx.lvars', lib/skc_codegen/src/gen_exprs.rs:464:32

Should I add a pair somewhere?

The story is little more complex. The compiler defines LLVM function for each constant (like @"init_A::B") but to support programs like this:
class A
  let x = 1000
  B = x + 10
  C = x * 20
end

the LLVM functions should not be per constant but per class.


Export/UI: improve "NLA Strips" option name/description

(Belated) followup to https://github.com/KhronosGroup/glTF-Blender-IO/pull/764#issuecomment-551403017.
The name and tooltip for the "NLA Strips" option is currently basically impossible to understand unless you already know what it does. We should try to improve it.
How does this look? I tried to make it both descriptive and concise.
Alternative name idea: Multiple Animations (organized by NLA)

Alternative text idea: "When on, create a glTF animation named 'Foo' by pushing all actions that should play onto NLA tracks named 'Foo'."

cc @donmccurdy

I like this new description, that reflects really better the feature. If Don validates it too, we need to update the documentation too.

Perhaps "Group by" rather than "Organize by"? Otherwise this looks good!

I'm good with "Group by". Should I go with that?

I changed it to "Group by" and updated the option name in the docs.
The docs about animation (all the stuff about "NLA Strip animations") could probably be reworked too. Basically I wrote those in reverse: given that the "NLA Strips" option meant "Export NLA Strip animations", I tried to write a definition for "NLA Strip animation" that would explain what that meant.

One suggestion, otherwise looks good to me too.


Repo Restructuring

The separate site and scrapers so they are in different top-level folders & package.json files.

Changes:

scraper: contains all the site scrapers and scraped info.
src: the static site
each of the folders has their own package.json so you'll need to run npm install on both, depending on what you're working on

For the most part, they are independent. The only intersection is with the multi-parse script under scrapers as that parses through all the canonical data. The parsed data is then saved under src/src/data, which is the dataset used by the site.


[ZEPPELIN-1722] Modernize visualization/transformation using ES6 class and module syntax

What is this PR for?
Modernize visualization/transformation using ES6 class and module syntax.
And remove global variable 'zeppelin'
What type of PR is it?
Refactoring
Todos

[x] - Use ES6 class and module syntax

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1722
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

\cc @1ambda @felizbear

I generally wish that api would be designed more like d3 or jquery, e.g. something along the lines of visualization(el).render(); without the need of calling it with new; it is more aligned with common practices in js libraries

Addressed comments.
I don't think style visualization(el).transform(...).render(); is important topic here.
Because this Visualization class is intended to be used by developer who want to create new visualization. And the developer doesn't need to call either visualization(el).transform(...).render();  or new ; because instantiation of visualization is managed by result.controller.js.

Addressed comments.
I don't think style visualization(el).transform(...).render(); is important topic here.
Because this Visualization class is intended to be used by developer who want to create new visualization. And the developer doesn't need to call either visualization(el).transform(...).render();  or new ; because instantiation of visualization is managed by result.controller.js.

Merge to master if there're no more discussions


Bar chart icons

Icon Request
1.
Icon name: Grouped bar chart
Tags of this icon: Grouped bar chart
2.
Icon name: Stacked bar chart
Tags of this icon: Stacked bar chart
3.
Icon name: Adjustable bar chart
Tags of this icon: Adjustable bar chart
4.
Icon name: horizontal bar charts
Tags of this icon: horizontal bar charts

Use case: Platform like tableau.com and developers.google.com/chart
Screenshots of similar icons:

Sorry, I only added 2 new bar-chart icons because the 24x24 pixel grid is too small to draw complexed icons and the complexed icon is out of the style of remixicon with is simple and clean.
bar-chart-grouped
bar-chart-horizontal


[Harpoon.nvim]: Issue with files in the git folder

Im using Harpoon2 in Neovim. Im facing an issue.
To share an example, the files are in two different folders. Both are under the same git repository.
root/folder1/ex1.lua and root/folder2/ex2.lua
when I open the harpoon list I do not see two files ex1.lua, ex2.lua as I expect. But I only see one depending on which folder my vim is in currently.
Does anyone know what could be the issue,? or if my understanding is wrong?

Harpoon2 gets it root directory from get get_root_dir function.
The default function just takes your vim.loop.cwd().
You have to change your get_root_dir to something like this if you want it to use your git root:
harpoon:setup({
	default = {
		get_root_dir = function ()
			local cwd = vim.loop.cwd()
			local root = vim.fn.system("git rev-parse --show-toplevel")
			if vim.v.shell_error == 0 and root ~= nil then
				return string.gsub(root, "\n", "")
			end
			return cwd
    		end
    	}
})

I tried the above solution, but still doesn't seem to work yet.

Hmmm, what is your output for :pwd?
Are you navigating to root/folder1 in nvim and then adding them?
and is your root folder a git folder?
It could also help to remove the harpoon.json ( should be in this folder: lua print(vim.fn.stdpath("data")))

yes, I deleted and tried it. they are in git folder
here is my dot files: https://github.com/nrupatunga/nvim

Hmmm, I had a look, but everything looks okay. Not sure what's wrong :/

The bug is in my config setting the autochdir=false fixed it.


Confusion to new comer: what version shall I use?

Thanks for the great work of putting up this amazing tool together.
As a new comer to this tool, I'm overwhelmed by the version changes and on going doc improvements. My question is, since V4 is in beta, and a lot of APIs are changing (correct me if I'm wrong), what version shall we use in production?
Thanks!

To be more specific, in V3's doc page, the API page is giving 404. And in V4's doc page, seems the author is still working on it :|

Hi, @oeddyo I think that it depends on what you're building. If it's an iOS only app, I'd definitely go with v4 (React Navigation transitions are a delight). On the other hand, if you need Android compatibility, v4 can't be used without workarounds.

Thanks a lot @diegocouto for commenting!
We are actually building for both android & ios. Thus I guess the option for us is to go with v3 then...
Thank you!

@oeddyo I see your confusion - v4-beta or unsupported-v3. This problem is going from quite strange React Native team behaviour - stop to support existing navigation and ask community for migration to beta ReactNavigation package. So I v4-beta is the same quality as beta React Navigation. If it works differently for Android compare to React Navigation then it is VERY strange because RNRF is just wrapper - you may check source codes - it just creates needed component tree and configuration for ReactNavigation. Probably the problem that Android doesn't support passing functions/images as parameters to actions (because images are passing now via props)? It is necessary just to debug it to find the reason...


README clarification

Adding the :with_deleted to a belongs_to association does not create two association accessors, you still need to explicitly create one including deleted records and (or one without).

I should also mention that I was developing entirely on ree-1.8.7-2011.03 while doing the upgrade for Rails 3.2.  Not sure if you want to add that to the README as well.

The tests hang on my REE build. Do they run on yours?

Yup:
$ rvm current
ree-1.8.7-2011.03@aap

$ ruby -Itest test/test_core.rb
Loaded suite test/test_core
Started
...
Finished in 1.193894 seconds.

16 tests, 110 assertions, 0 failures, 0 errors

This is on e4d3676bbf96f74b11f51bcb364e3bb613db8bd3 from master.

That's only test_core, but you're right — I was just able to run all of them successfully. I'm not entirely sure if I want to add it to the README though, since I don't intend to actively monitor 1.8 compatibility.


First version of 0.1.0 Beta

Now supports vanilla highcharts config
Much less code - should be much more stable and less bugs
Only support Highchart/Highstock >= 5.0.0
2-way axes binding no longer supported
loading property no longer supported

probably worth killing support for Angular <= 1.5 and start to use a component instead

@graingert good idea - will look into it.

@pablojim you can point 1.3 users at https://github.com/toddmotto/angular-component

probably also worth looking at babel for the source code.

Anyone using this in production yet? How stable would you guys say this is?

@VictorioBerra in general it's a lot simpler than the original version so I'd expect it to be more stable. That said it is new and I'm pushing quite large changes still!

@pablojim do you have any idea of a time-frame on merging the beta to master?

@pablojim any chance of including a function in your documentation to convert the old style object into the new?

@graingert  there is no new style object. It takes a normal highcharts object. Just remove your "option" property and move everything inside it out to the config object.

I'm not sure if I'm an idiot, but can we include an example of how to use the drilldowns?
As the whole config structure, it might be worth it to bring an example for this for new comers?

@Stenkilde Im not sure whats so confusing about this new version for people. Because @graingert was also asking about the "new config structure".
The configuration object is now IDENTICAL to the same one you would normal pass to your Highcharts constructor.  If you want to use drilldowns. Head over to the official highcharts documentation here and use their example drilldown configuration. NOTE: Make sure to include the drilldown plugin.
I have created a jsfiddle for you that demonstrates the use of drilldowns uses highcharts-ng2. Notice how the configuration object I pass to highcharts-ng2 is a copy/paste of the one from the official highcharts example I linked above.

@VictorioBerra I've never used the normal Highcharts constructor.

@VictorioBerra @pablojim is this what you mean?
function convertOldHighchartsNgToNew({ options, config }) {
    return Object.assign({}, config, options);
}

If you really need that, then yes that will work. Id recommend running a
conversion script against your saved configuration data using that instead
of hard coding it as a permanent conversion operation in your database.
Otherwise you will need stuff like if(config.options !== undefined) after
you start persisting your new config.
On Tue, Nov 29, 2016 at 9:37 AM, Thomas Grainger<EMAIL_ADDRESS>wrote:

@VictorioBerra https://github.com/VictorioBerra @pablojim
https://github.com/pablojim is this what you mean?
function convertOldHighchartsNgToNew({ options, config }) {
return Object.assign({}, config, options);
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pablojim/highcharts-ng/pull/529#issuecomment-263603817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACzG68zKjul_g-N4U_WJNH-o4ipi8LFNks5rDEaggaJpZM4K27on
.

@VictorioBerra our config is generated on the fly, I just didn't understand the changes to make.

@graingert No worries. In general now that this library accepts a regular highcharts config object it will be much simpler to use and configure. Now you can just use the HC docs and know that this library will work out of the box.

@pablojim @VictorioBerra what was the "2-way axes binding" because it's "no longer supported"?

@Stenkilde no silly questions - let us know if @VictorioBerra's jsfiddle doesn't work for you!
@VictorioBerra Thanks for providing the example jsfiddle - maybe we can automate the creation of these!
@graingert don't think I'll provide a conversion function as there are some subtleties around events and other config that might be hard to support.
The 2-way axis binding is where you previously could zoom in in the chart and have the config update to reflect the new axis values. This is no longer supported as I'm trying to move the component to a one way binding. Axes changes could be notified by events but this is not implemented yet.

"The chart does not fit into the parent container? How to fix that?" in the FAQ does not apply for the beta version. How could I fix this in the beta version?

Legend title bug with series IDs. https://jsfiddle.net/Lord_Zero/xkdk91mg/
Notice how there is no legend title. If you uncomment the ids in the config the title works.
The library should be ensuring IDs.


exlude all non-english pages from the search results

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

/kind event

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog
/area documentation
/area community
What this PR does / why we need it:
This PR removes from the search results all pages which are not in En. it avoids to have duplicated contents in the results.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Is there a way to make the search language aware? So if I have the language set to Japanese and I type something I get only the Japanese pages etc?

Is there a way to make the search language aware? So if I have the language set to Japanese and I type something I get only the Japanese pages etc?

I searched for and nothing like that in Lunr which is really limited. It also means, cause we don't have all pages translated, the search must fall back on EN pages. Not easy.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

LGTM label has been added.  Git tree hash: 63cd04d9241a0b39e42ceae90d723245096208c1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [Issif,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment